py-ard 1.0.9 :snowflake:

Destination channel: Snowflake
Links

PKG-3763
release-notes: https://github.com/nmdp-bioinformatics/py-ard/releases
release-notes-tagged: https://github.com/nmdp-bioinformatics/py-ard/releases/tag/v1.0.9
changelog: https://github.com/nmdp-bioinformatics/py-ard/blob/master/CHANGELOG.md
license:

https://github.com/nmdp-bioinformatics/py-ard/blob/master/COPYING
https://github.com/nmdp-bioinformatics/py-ard/blob/master/LICENSE

requirements-tagged:

https://github.com/nmdp-bioinformatics/py-ard/blob/v1.0.9/requirements-tests.txt
https://github.com/nmdp-bioinformatics/py-ard/blob/v1.0.9/requirements.txt
https://github.com/nmdp-bioinformatics/py-ard/blob/v1.0.9/setup.py

Explanation of changes:

Add a new feedstock
Fix indentation
Add the upper bound <2.0.0 for pandas as it's not compatible (tests failed), see https://github.com/nmdp-bioinformatics/py-ard/issues/227#issuecomment-1498143195

Reminder, please don't merge this. See https://anaconda.atlassian.net/browse/PKG-3763?focusedCommentId=198324.


[incubator/buzzfeed-sso] Add support for GCP BackendConfig

Is this a new chart
No
What this PR does / why we need it:
To be able to configure GCP LB backend service (e.g timeouts) for applications behind buzzfeed-sso Proxy.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @darioblanco

@darioblanco @komljen can I have your reviews for this PR please ?

@darioblanco @komljen any chance in reviewing this PR ?

/assign @komljen

/ok-to-test

/lgtm

@anas-aso sorry this one took so long. I am available in the #charts channel in the Kubernetes Slack space if I am not reacting to an assignment. I totally overlooked it.


Errors reference

Hello,
I'm getting an error #008 with a link, but when i click on the link, 404 appears.
Is there any other reference for errors?

(stub-2)warn("[React Flow]: Couldn't create edge for target handle id: "undefined", edge id: edge:1 '-->' 1. Help: https://reactflow.dev/error#008")

It would be nice to get some more information on whats going on (or at least not a broken link :) ), even though error message is pretty self explanatory.

Duplicate of xyflow/web#266  I'll try to fix this today.

should work again!


Question : Default Admin credentials

Hey ! Hope you're doing well, I wanted to know if it's possible to include admin default admin credentials. Especially with environment variables like ADMIN_EMAIL=<email> and ADMIN_PASSWORD=<secret>. Why with environment variables? Because in some cases, users might need to deploy shynet on a docker instance without having shell access. Environment variables are common knowledge and does not require knowing Django Admin.
What do you think of this approach? And do you have any ideas on how to integrate it into shynet (as a workaround to avoid having to use the instance shell) ? I'm thinking of adding this administrator login at startup.
Thank you for your time and this quality project. Have a nice day!

Hi @MatteoGauthier! This is certainly something we could consider. I'm hesitant to add default credentials (as in admin username and password password, for example), but I do think it would be OK to check whether certain environment variables are set at startup and create an admin user with those credentials if none already exists. Would someone be willing to open a PR with this change?


Check how many days ago a project was reviewed.

What's the feature?
A way to see how many days ago a project was reviewed.
What problem is the feature intended to solve?
Udacity will be removing certification of reviewers that did not have reviewed a particular project for more than 90 days.
Is this feature similar to an existing feature in another tool?
Maybe certs.
Is this a feature you're prepared to implement, with support from the urcli team?
Yes.  I would like to discuss first what would be the best way to do it. We could add an extra column in the urcli certs table showing how many days ago each project was reviewed, but I think that could overload the command. Maybe a new command could be implemented.
@trolster , what are your thoughts on this?

I found a workaround to show the projects reviewed in the last 90 days.
urcli revenue --from $(date --d="90 days ago" -u '+%Y-%m-%d')

I think this solves my issue and a new feature does not need to be developed.


Run everything on 1.10

#384 points out that the workaround I introduced in https://github.com/compintell/Mooncake.jl/pull/377 only works on 1.11. I didn't catch this because, so far, I've only been running a subset of CI on 1.11 in order to keep CI times down. This PR will

change our CI set up so that all tests are run on LTS and release (preventing this kind of problem from recurring), and
fix the problem on 1.10.

If it looks like the amount of time spent running CI is becoming prohibitive again, I'll have to look at ways to speed it up.

This is getting there, but there are a few issues on 1.10 that need dealing with. I've made some progress today, and hope to finish off tomorrow.

@lcontento it looks like we're nearly there -- down to one error. I have to call it a day now -- should be able to resolve in the morning!


FloorMap: fix currect fontWeight

Idea Description

Implementation(noset)
Figma(500)

TODO

[ ] set fontWeight 500
https://github.com/DroidKaigi/conference-app-2023/blob/80a347f106357b0a0097fb5e3ede80facf560df6/feature/floor-map/src/main/java/io/github/droidkaigi/confsched2023/floormap/section/FloorMap.kt#L46-L55

Thanks! You are so reliable!

Can I try this issue?

Sure! Go ahead!

I think this issue is already done by #835.
https://github.com/DroidKaigi/conference-app-2023/blob/773d5f3649c022e227c4529e37f52fd47c8af1f6/core/designsystem/src/commonMain/kotlin/io/github/droidkaigi/confsched2023/designsystem/theme/Type.kt#L56

@takahirom
I think this issue is closeable.💁♂

Thanks 🙇


add support for outbound Redis set operations

This adds support for the SADD, SREM, and SMEMBERS Redis commands, useful for apps which need to manage groups of unique identifiers (e.g. chat apps).
Signed-off-by: Joel Dice<EMAIL_ADDRESS>

@lann FYI

This seems fine, but I have to wonder if we'd be better off with a generic "exec" function rather than a function per redis command...

This seems fine, but I have to wonder if we'd be better off with a generic "exec" function rather than a function per redis command...

Yeah, I was thinking the same thing.  Redis a lot of operations.  It's nice to have a bit of static type and arity checking, though.  Maybe there's some kind of machine-readable Redis IDL we could generate WIT files and tests from.

I've run make test-outbound-redis and manually tested the tinygo-outbound-redis example to make sure there are no regressions.  I'm going to go ahead and merge this since set operations are broadly useful.
I do agree that having a separate function for every operation is not sustainable unless it can be automated somehow, so a good follow-up PR would be to add a general-purpose exec function to cover the myriad other ops, transactions, etc.


Relation fields doesn't get generated when used as a mixin

Hi,
when using .relation as this:
export default createMixin((UserRelationshipMixin) => {
  UserRelationshipMixin
    .relation("user", UserModel, {
      references: ["id"],
      fields: ["userId"],
    })
    .int("userId", { unique: true })
});

the resulting model is missing those fields. Please look into example:
https://codesandbox.io/s/exciting-leaf-bxqzvt?file=/src/models/Session.model.ts
As you can see, Token model has manually added those fields and everything is fine. Session model on the other hand is using mixin, and it lack those fields.
Thanks in advance :)

Thanks, will begin looking into this.

@Daidalos117 Went ahead and addressed the bug, thanks for the report! This will be out in the next release.

Nice! Thank you! I wanted to fix it by myself, but it would probably took me forever 😄 Good job tho.


Bazel image: Please add 'zip' and 'file'

Which image/versions are related to this issue/feature request?
cgr.dev/chainguard/bazel:6.1.1 & 6.2.0
Issue/Feature description
Official bazel distribution assumes presence of 2 tools in the underlying environment:

zip (https://github.com/bazelbuild/bazel/blob/85dcafeba92479bee5b2cd2cb3f33f84be7de7f7/tools/test/test-setup.sh#L430)
file (https://github.com/bazelbuild/bazel/blob/85dcafeba92479bee5b2cd2cb3f33f84be7de7f7/tools/test/test-setup.sh#L396)

Neither of them is provided by busybox:
[2023-06-05 07:30:49] external/bazel_tools/tools/test/test-setup.sh: line 391: file: command not found
[2023-06-05 07:30:49] Could not create "/home/host_user/out/bazel_out/f8087e59fd95af1ae29e8fcb7ff1a3dc/sandbox/processwrapper-sandbox/26/execroot/{redacted}/bazel-out/k8-fastbuild/testlogs/projects/shared/test_module/test.outputs/outputs.zip": zip not found or failed

There is long standing issue in bazel: https://github.com/bazelbuild/bazel/issues/13230 for using or internal 'zipper' impl.

Looks like a linked PR was merged.  Can we close this out @sfc-gh-ptabor ?


ServiceBus not respecting environment variables HTTP[S]_PROXY

Package Name: azure-servicebus
Package Version: 7.6.1
Operating System: Linux (ubuntu 20.04)
Python Version: 3.9.12

Describe the bug
The Service Bus SDK ignores setting of variables HTTP_PROXY and HTTPS_PROXY.
To Reproduce
Steps to reproduce the behavior:
0. Set up a proxy server. You can use for example squid in docker
a. Terminal 1 - run proxy
docker run --rm --name squid alpine /bin/sh -c "apk --no-cache add squid && squid -N"
b. Terminal 2 - run scripts here
export http_proxy="http://$(docker inspect squid | grep '"IPAddress":' | head -n 1 | cut -d'"' -f4):3128"
export https_proxy="http://$(docker inspect squid | grep '"IPAddress":' | head -n 1 | cut -d'"' -f4):3128"

Send a message to a service bus topic via azure-servicebus SDK
The Azure Service Bus SDK is not communicating via the proxy.
If your network is setup to only allow communication via the proxy, you get

azure.servicebus.exceptions.ServiceBusConnectionError: Unable to open authentication session on connection b'SBSender-<REDACTED ID>'.

Please confirm target hostname exists: b'[<REDACTED SBN NAME>.servicebus.windows.net]'

Expected behavior
Service bus SDK should respect the the environment variables HTTP_PROXY and HTTPS_PROXY and route all traffic through the specified proxies, as specified by Microsoft Azure SDK documentation.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Hi @mbelak-dtml thanks for your issue, we'll take a look asap.

Hi, yes, I have been able to use a proxy using the provided example.
However, it has taken some time to debug this issue since the Azure SDK documentation states that all client libraries should follow the proxy configuration provided by environment variables and the error message is not very helpful.

Awesome! @mbelak-dtml we will look into aligning the two to avoid this in the future, thank you! Issue Created

Since the proxy.py sample works and a new issue has been created to resolve issues with documentation, looks like this issue has been addressed.


Document the domain interface

What are domains?
How do they relate to everything else?
How to use them in depth

The new approach makes this redundant


Disable filteredRelationCount feature in prisma > 4.16.0

Bug description
Our team upgraded from 4.14.1 to 5.0.0 and ran into bugs due to our usage of typegraphql-prisma and the new default enabled filteredRelationCount feature released to GA in 4.16.0.
This feature results in autogenerated types containing many new required getters that we do not need. Is there a way to disable this?
See the related issue we opened on typegraphql-prisma: https://github.com/MichalLytek/typegraphql-prisma/issues/408
How to reproduce
Upgrade to 4.16.0 or beyond
Expected behavior
I want to be able to disable filteredRelationCount
Prisma information
Prisma 5.0.0
Environment & setup

OS: macOS
Database: MySQL
Node.js version: 18.12.1

Prisma Version
5.0.0

Generally, we have no plans to disable any functionality we added, unless it is incorrect or broken. But if I understand correctly, this is doing exactly what we want it to - via typegraphql-prisma this just leads to more generated code than before? Isn't this more a configuration for typegraphql-prisma to skip generating these getters that you do not need?

@janpio I would agree that the feature is working correctly. I can't quite tell at this point if:

we need to modify typegraphql-prisma to accommodate for the new code
maintain our types manually without using typegraphql-prisma
flag to disable this feature in prisma
ensure that the autogenerated getters using the new filteredRelationCount feature are marked as optional in the autogenerated type (I anticipate this would be a change in typegraphql-prisma)

For our team, having typescript types autogenerated using typegraphql-prisma for each of the models in the schema.prisma is a big value add. I'll be happy to provide our solution when we get there, but until then, we are blocked on moving from 4.14.1 to prisma >= 4.16.0.
I appreciate the response.

We talked about this internally, and don't really see a viable path to implement this in Prisma itself.
The challenge here is that all these features are interconnected. If you disable one, even if you do it really cleanly, in theory all other features can break as well. Disable another one, same situation again. So to confidently ship something like this we would need to test Prisma Client with each feature disabled individually, and then also with all combinations 😨 We already run into a similar problem with our preview feature flags that add functionality, and had to do quite a dance to cover that.
So unfortunately for now our suggestion is that this is implemented on a deeper level instead by Prisma itself.
We'll keep this issue open though (with a slightly renamed title) to collect further feedback and especially use cases. Maybe this is actually a super common problem, and we should find a way to make this possible if there are enough people asking and making a good case for it.

@janpio thanks for discussing with the team. It seems reasonable to just leave this open for feedback. Hopefully it is not a common issue. Solving it in typegraphql-prisma or in peoples own codebase is reasonable. I did successfully test two possible solutions for this. These are relevant for others who may come across this

Run a post:generate script to clean up the types that typegraphql-generated to no longer include the getters. I did a quick and dirty version that fixed the issues that we ran into. We did not opt to use this approach since it felt brittle, but it was certainly the smallest amount of effort.

const path = './node_modules/@generated/type-graphql/resolvers/outputs'
console.log('Processing all generated *Count files to remove getters.')
try {
  ;(async () => {
    // eslint-disable-next-line @typescript-eslint/no-var-requires
    const fs = require('fs')
    const files = await fs.readdirSync(path)
    files.forEach(async (file) => {
      if (file.includes('Count')) {
        const filePath = `${path}/${file}`
        const fileContent = await fs.readFileSync(filePath, { encoding: 'utf8' })
        const newContent = fileContent.replace(/get(.+?);/g, '')
        fs.writeFileSync(filePath, newContent)
      }
    })
    console.log('Successfully processed all generated *Count files to remove getters.')
  })()
} catch (err) {
  console.error('Error processing generated *Count files to remove errors', err)
}

Replace include: { _count: true } with _count: { select: xyzRelation: true } in addition to type coercion after the prisma operation. ex:

 return (await context.prisma.xyzModel.findMany({
        include: {
          _count: {
            select: {
              xyzRelation: true,
            },
          },
        },
      })) as xyzModel[]

For this solution we had to refactor anywhere we were using _count: true in a prisma operation. We also had to do type coercion (which feels a little bit wrong) in a number of places.
Probably the best solution is the one we did not pursue yet.

Patch typegraphql-prisma to make these get* functions optional in the type. We may investigate this option further if we see runtime issues due to our type coercion solution.


Extract author

Find first valid value (trimmed length in interval 0..100) of:

meta[name="author"]
[rel="author"]
[class="author"], [class="writer"], [class="writtenby"]
[id="author"], [id="writer"], [id="writtenby"]

Looks like we're getting support in #49

Check out v1.0.0 and see if it meets your needs! :)


German localization is missing code of conduct

This is a Bug Report

Problem:
https://kubernetes.io/de/community/code-of-conduct/ is a missing page. This page is important because it provides the CNCF code of conduct, and that is an important element of an inclusive community.
Proposed Solution:

Add German code of conduct page to website

Page to Update:
https://kubernetes.io/de/community/code-of-conduct/

Additional Information:
/priority important-soon
/language de
/kind bug

/remove-prority important-soon
/remove-kind bug
/priority important-longterm

/remove-priority important-soon

/assign

Has been fixed with #18981

Thanks for the update @alexzimmer96
/close


Is there a way to emit an event from a Objective-C class method?

Description
I'm looking for a way to call sendEventWithName from a class method of an interface that inherits from RCTEventEmitter.
Reason:
I want to send an event to JS from any place of Objective-C native code of my app using a native module.
So the scenario would be:
@interface LocalEventEmitter : RCTEventEmitter<RCTBridgeModule>
- (NSArray<NSString*>*)supportedEvents;
+ (void)emit:(NSString*)data;
@end

and in the emit method I would like to emit the event.
I'm using the class method so I can call it from anywhere, without creating the instance as it would require some React Native context that I don't have.
However, I'm stuck at:https://github.com/facebook/react-native/blob/a70354df12ef71aec08583cca4f1fed5fb77d874/React/Modules/RCTEventEmitter.m#L53-L61
I know the reasoning behind this, but I can't find a way to somehow "keep" the context of React Native and use it in my class method to correctly send the event.
So my question is:
How to emit the event from iOS native module using class method?
Version
0.69.1
Output of npx react-native info
System:
OS: macOS 12.4
CPU: (10) arm64 Apple M1 Pro
Memory: 1.74 GB / 32.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 18.4.0 - ~/.nvm/versions/node/v18.4.0/bin/node
Yarn: 1.22.18 - /opt/homebrew/bin/yarn
npm: 8.12.1 - /opt/homebrew/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK:
API Levels: 28, 29, 30, 31, 32
Build Tools: 28.0.3, 29.0.2, 30.0.2, 30.0.3, 31.0.0, 32.0.0, 32.1.0
System Images: android-28 | Intel x86 Atom, android-28 | Intel x86 Atom_64, android-28 | Google Play Intel x86 Atom, android-28 | Google Play Intel x86 Atom_64, android-29 | Intel x86 Atom_64, android-29 | Google APIs Intel x86 Atom, android-30 | Google APIs Intel x86 Atom_64, android-31 | Android TV ARM 64 v8a, android-32 | Google APIs ARM 64 v8a
Android NDK: 21.3.6528147
IDEs:
Android Studio: 2021.1 AI-211.7628.21.2111.8139111
Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
Languages:
Java: 11.0.11 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.0.0 => 18.0.0
react-native: 0.69.1 => 0.69.1
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

Create the app
Implement basic native module on both Android and iOS app that is to emit events to JS
For Android keep the reactContext in a static variable to use it for emitting event
For iOS send the event from a class method

Snack, code example, screenshot, or link to a repository
@implementation LocalEventEmitter

RCT_EXPORT_MODULE()

- (NSArray<NSString*>*)supportedEvents
{
    return @[@"newDataReceived"];
}

+ (void)emit:(NSString*)data
{
    [[[self alloc] init] sendEventWithName:@"newDataReceived" body:@{@"summary": data}];
}
@end

also I want supprt

(NSArray<NSString *> *)supportedEvents

to use viewController uuid, can not set it. not addListener with same type only first got callback

Let me bump it then, please. I would still use some advise regarding this.


Web interface - Simple desk page number non-integer value

Describe the bug
There is a bug with the previousPage() function which causes the decimal places ".53333333333333" to appear with the Page Number.
I couldn't find a reference to this issue on the forum or GitHub.

To Reproduce
Steps to reproduce the behavior:

Run QLC+ with the -w flag Web Interface Docs
Using Chrome, navigage to http://<IP_ADDRESS>:9999/simpleDesk
Shrink the window
Hit refresh the page
Click the back arrow next to the page indicator
The page indicator will put the page number and a bunch of decimal places of garbage.

Expected behavior
Single integer values to appear, such as when using the nextPage() button.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop:

OS: Windows 11
QLC+ Version: 4.13.0
Hardware setup: N/A

Additional context
I believe the problem is with simpledesk.js within this function:

Additionally, it seems you can keep pressing the back button into negative values:

Do you have a custom number of faders set for Simple desk?

Do you have a custom number of faders set for Simple desk?
Note to myself: this is the referenced line of code https://github.com/mcallegari/qlcplus/blob/master/webaccess/res/simpledesk.js#L127

No only standard

After playing around with the latest release, either the bug isn't present anymore or it is impossible to replicate.
Closing to declutter the issues tab. If I find it occurs again I'll reopen.


gfortran build failure (dependency of lapack

After new pull today (08/13/2020), gfortran, a dependency of lapack, fails to build.
E:\Libs\vcpkg>vcpkg install ceres[eigensparse,lapack,tools]
Computing installation plan...
The following packages will be built and installed:
ceres[core,eigensparse,lapack,tools]:x64-windows

lapack[core]:x64-windows
lapack-reference[blas-select,core,noblas]:x64-windows
vcpkg-gfortran[core]:x64-windows
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Starting package 1/4: vcpkg-gfortran:x64-windows
Building package vcpkg-gfortran[core]:x64-windows...
Could not locate cached archive: C:\Users\Kevin.Carroll\AppData\Local\vcpkg\archives\b3\b3aedb32ae55a0c39d901fa6b0296b15faa7c0a5.zip
-- The Fortran compiler identification is unknown
-- No Fortran compiler found on the PATH. Using MinGW gfortran!
-- Acquiring MSYS Packages...
-- Acquiring MSYS Packages... OK
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/bin/libgfortran-5.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/bin/libquadmath-0.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/bin/libwinpthread-1.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/bin/libgcc_s_seh-1.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/debug/bin/libgfortran-5.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/debug/bin/libquadmath-0.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/debug/bin/libwinpthread-1.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/debug/bin/libgcc_s_seh-1.dll
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/winpthreads
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/winpthreads/COPYING
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/winpthreads/mingw-w64-libraries
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/winpthreads/mingw-w64-libraries/winpthreads
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/winpthreads/mingw-w64-libraries/winpthreads/COPYING
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/gcc-libs
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/gcc-libs/COPYING.LIB
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/gcc-libs/COPYING.RUNTIME
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/gcc-libs/COPYING3
-- Installing: E:/Libs/vcpkg/packages/vcpkg-gfortran_x64-windows/share/vcpkg-gfortran/license/gcc-libs/README
CMake Error at ports/vcpkg-gfortran/portfile.cmake:50 (file):
file INSTALL cannot find
"E:/Libs/vcpkg/downloads/tools/msys2/msys64/mingw64/share/licenses/crt/COPYING.MinGW-w64-runtime.txt":
No such file or directory.
Call Stack (most recent call first):
scripts/ports.cmake:79 (include)

Error: Building package vcpkg-gfortran:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: vcpkg-gfortran:x64-windows
Vcpkg version: 2020.06.15-nohash
Additionally, attach any relevant sections from the log files above.

duplicate of #12888

Please remove your cached MSYS install and retry. Normally it is located in downloads/tools/msys2

I removed my cached MSYS and retried the install.  MSYS was pulled down, but the error persists.
Here is the output and the vcpkg\buildtrees\lapack-reference\config-x64-windows-out.log:
E:\Libs\vcpkg>vcpkg install lapack
Computing installation plan...
The following packages will be built and installed:
lapack[core]:x64-windows

lapack-reference[blas-select,core,noblas]:x64-windows
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
Starting package 1/2: lapack-reference:x64-windows
Building package lapack-reference[blas-select,core,noblas]:x64-windows...
Could not locate cached archive: C:\Users\Kevin.Carroll\AppData\Local\vcpkg\archives\3e\3efd478cfc4b6e71d4d5bab59e76004378608593.zip
-- Using cached E:/Libs/vcpkg/downloads/Reference-LAPACK-lapack-v3.8.0.tar.gz
-- Cleaning sources at E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source E:/Libs/vcpkg/downloads/Reference-LAPACK-lapack-v3.8.0.tar.gz
-- Using source at E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean
-- The Fortran compiler identification is unknown
-- No Fortran compiler found on the PATH. Using MinGW gfortran!
-- Configuring x64-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
Command failed: ninja -v
Working Directory: E:/Libs/vcpkg/buildtrees/lapack-reference/x64-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
E:\Libs\vcpkg\buildtrees\lapack-reference\config-x64-windows-out.log

Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:300 (vcpkg_execute_required_process)
ports/lapack-reference/portfile.cmake:56 (vcpkg_configure_cmake)
scripts/ports.cmake:79 (include)
Error: Building package lapack-reference:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: lapack-reference:x64-windows
Vcpkg version: 2020.06.15-nohash
Additionally, attach any relevant sections from the log files above.
[1/2] cmd /c "cd ../../x64-windows-dbg && "E:/Libs/vcpkg/downloads/tools/cmake-3.17.2-windows/cmake-3.17.2-win32-x86/bin/cmake.exe" "E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean" "-DUSE_OPTIMIZED_BLAS=ON" "-DCBLAS=OFF" "-DCMAKE_GNUtoMS=ON" "-DCMAKE_Fortran_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gfortran.exe" "-DCMAKE_C_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gcc.exe" "-DCMAKE_Fortran_FLAGS_INIT:STRING= -mabi=ms -m64 " "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/toolchains/mingw.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=E:/Libs/vcpkg" "-D_VCPKG_INSTALLED_DIR=E:/Libs/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=E:/Libs/vcpkg/packages/lapack-reference_x64-windows/debug""
FAILED: ../../x64-windows-dbg/CMakeCache.txt
cmd /c "cd ../../x64-windows-dbg && "E:/Libs/vcpkg/downloads/tools/cmake-3.17.2-windows/cmake-3.17.2-win32-x86/bin/cmake.exe" "E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean" "-DUSE_OPTIMIZED_BLAS=ON" "-DCBLAS=OFF" "-DCMAKE_GNUtoMS=ON" "-DCMAKE_Fortran_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gfortran.exe" "-DCMAKE_C_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gcc.exe" "-DCMAKE_Fortran_FLAGS_INIT:STRING= -mabi=ms -m64 " "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/toolchains/mingw.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=E:/Libs/vcpkg" "-D_VCPKG_INSTALLED_DIR=E:/Libs/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=E:/Libs/vcpkg/packages/lapack-reference_x64-windows/debug""
CMake Error at CMakeLists.txt:3 (project):
The Ninja generator does not support Fortran using Ninja version
1.8.2

due to lack of required features.  Ninja 1.10 or higher is required.
-- Configuring incomplete, errors occurred!
See also "E:/Libs/vcpkg/buildtrees/lapack-reference/x64-windows-dbg/CMakeFiles/CMakeOutput.log".
[2/2] cmd /c "cd .. && "E:/Libs/vcpkg/downloads/tools/cmake-3.17.2-windows/cmake-3.17.2-win32-x86/bin/cmake.exe" "E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean" "-DUSE_OPTIMIZED_BLAS=ON" "-DCBLAS=OFF" "-DCMAKE_GNUtoMS=ON" "-DCMAKE_Fortran_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gfortran.exe" "-DCMAKE_C_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gcc.exe" "-DCMAKE_Fortran_FLAGS_INIT:STRING= -mabi=ms -m64 " "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/toolchains/mingw.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=E:/Libs/vcpkg" "-D_VCPKG_INSTALLED_DIR=E:/Libs/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=E:/Libs/vcpkg/packages/lapack-reference_x64-windows""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "E:/Libs/vcpkg/downloads/tools/cmake-3.17.2-windows/cmake-3.17.2-win32-x86/bin/cmake.exe" "E:/Libs/vcpkg/buildtrees/lapack-reference/src/v3.8.0-954f10683f.clean" "-DUSE_OPTIMIZED_BLAS=ON" "-DCBLAS=OFF" "-DCMAKE_GNUtoMS=ON" "-DCMAKE_Fortran_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gfortran.exe" "-DCMAKE_C_COMPILER=E:/Libs/vcpkg/downloads/tools/msys2-20200812/msys64/mingw64/bin/gcc.exe" "-DCMAKE_Fortran_FLAGS_INIT:STRING= -mabi=ms -m64 " "-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2019/Professional/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/toolchains/mingw.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v142" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=E:/Libs/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-D_VCPKG_ROOT_DIR=E:/Libs/vcpkg" "-D_VCPKG_INSTALLED_DIR=E:/Libs/vcpkg/installed" "-DVCPKG_MANIFEST_INSTALL=OFF" "-G" "Ninja" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=E:/Libs/vcpkg/packages/lapack-reference_x64-windows""
CMake Error at CMakeLists.txt:3 (project):
The Ninja generator does not support Fortran using Ninja version
1.8.2

due to lack of required features.  Ninja 1.10 or higher is required.
-- Configuring incomplete, errors occurred!
See also "E:/Libs/vcpkg/buildtrees/lapack-reference/x64-windows-rel/CMakeFiles/CMakeOutput.log".
ninja: build stopped: subcommand failed.

Now the error is a duplicate of #12888
If you want a temporary workaround doing: vcpkg fetch ninja should be good enough until #12895 is merged

@rkcarroll Thanks for posting this issue. Now PR #12895 has been merged, does this issue still occur?

I did a new pull this morning (08/17/2020) and got a clean install.  Thanks!!!
From: Lily<EMAIL_ADDRESS>Sent: Monday, August 17, 2020 3:03 AM
To: microsoft/vcpkg<EMAIL_ADDRESS>Cc: Kevin Carroll<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/vcpkg] gfortran build failure (dependency of lapack (#12892)
@rkcarroll https://github.com/rkcarroll  Thanks for posting this issue. Now PR #12895 https://github.com/microsoft/vcpkg/pull/12895  has been merged, does this issue still occur?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/vcpkg/issues/12892#issuecomment-674727841 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC72OZUE4WSER6ZFRDGHBLSBDP47ANCNFSM4P6QA4OQ .  https://github.com/notifications/beacon/AAC72OYFRNOZG6M7JYORFVLSBDP47A5CNFSM4P6QA4O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFA3YPII.gif

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.


fixing font scaling issues

It seems the font scaling wasn't quite working properly on macOS, causing the command entries to be twice as tall as they need to be, and misalignment when highlighting:

The two-line change now allows this to work properly:

Hi, sorry, I'm not super versed in pull requests and see that my continued work on my fork shows up as additional commits here. I had intended to add those as a separate pull request.
The additional changes since I created this pull request do a few things:

added an optional second parameter to CommandPalette which specifies the search string hint for the InputText
i improved the spacing and alignment of the the command entries to better match that in ImGui ListBoxes and Menus
I added support for icons and shortcuts to be displayed along with each command name. These are optional, and when not specified the behavior should be the same as before. (Note that, just like ImGui menu items, the shortcuts are only for display. The logic to trigger the commands via the shortcuts is up to the user)

Here is a screenshot of the improvements

If you want to see a demo, the latest version of HDRView's dear-imgui branch has this.
Also not sure whether these changes should be a pull request to your fork, to directly to the original repo.


Clean up example samples

Why are these changes needed?

This PR cleans up the "complete" and "autoscaler" sample yamls a bit.
Unnecessary pod spec fields are removed without sacrificing the completeness of the examples.
The idea is to make the configuration look less intimidating.
Related issue number

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] This PR is not tested :(

great. These configuration are covered by codes. Looks good to me

We could consider automatically configuring the init container and pre-stop hook in the 0.4.0 release.

Just going to double check that the configs don't have any issues before merging.


Unfindable spirv-builder crate and undocumented target in latest git version

Expected Behaviour
use the spirv-builder crate like in the documentation
Example & Steps To Reproduce

follow the steps described in the documentation
the spirv-builder = "0.1.0" crate is some kind of zombie crate that doesn't have the spirv_builder mod in it. Other versions are not found at all
if added through the git, it requires an undocumented target parameter, supposedly for the cargo command.

System Info

Rust: rustc 1.54.0-nightly
OS: Windows 10 Version 10.0.19041 Build 19041
GPU: RTX 3090 eGPU
SPIR-V: SPIRV-Tools v2021.2-dev v2021.1-31-gc05f7441

Backtrace

Backtrace

Caused by:
  process didn't exit successfully: `[redacted]\target\debug\build\rust_engine-d11c877f42ad3630\build-script-build` (exit code: 101)
  --- stderr
  error: failed to run `rustc` to learn about target-specific information
  Caused by:
    process didn't exit successfully: [redacted]\.cargo\bin\rustc.exe - --crate-name ___ --print=file-names -Z codegen-backend=[redacted]\target\debug\rustc_codegen_spirv.dll -Zsymbol-mangling-version=legacy --target "SPIR-V 1.0" --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=cfg` (exit code: 1)
    --- stderr
    error: Error loading target specification: Could not find specification for target "SPIR-V 1.0". Run `rustc --print target-list` for a list of built-in targets
  thread 'main' panicked at 'Did not find output file in rustc output', [redacted]\.cargo\git\checkouts\rust-gpu-e0a37a82a46176e6\486a3ce\crates\spirv-builder\src\lib.rs:458:10
  stack backtrace:
  note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
Process finished with exit code 101

For those encountering the problem, the target is: spirv-unknown-spv1.4, 1.4 can be changed to the appropriate version.

Ah, looks like we forgot to update that bit of the documentation when we added the target parameter. The targets are documented here in the book, but yeah, we should update that page.
Also, you are correct in that we do not publish spirv-builder to crates.io because we don't feel it's ready yet - I'm kind of baffled how what we've been saying of "use the git version" hasn't gotten into that page, thanks for pointing that out.

@khyperia thank you :)


Add Brotli size to minsize task output

I think it's worth adding Brotli to the output snapshot.
It can help validate minifier passes/changes are net positive, or dispel that they're net negative (#376 seems to be such a case).

Lovely! Thank you!


Is there a check for consistent spacing for newline escapes?

Here is an example which is not flagged as of 0.2.0 with default options:
set var {one \
         two\
         three  \
         four\
         five   \
         six \
         seven
}

Is there a check for this? I think by default I would expect a single space:
set var {one \
         two \
         three \
         four \
         five \
         six \
         seven
}

Or maybe a user could override with the number of spaces (e.g. 0):
set var {one\
         two\
         three\
         four\
         five\
         six\
         seven
}

And in an advanced case maybe it could check for block alignment:
set var {one   \
         two   \
         three \
         four  \
         five  \
         six   \
         seven
}

Thanks!

This check doesn't exist yet, but I think it's a good suggestion - thanks! I'll go ahead and implement it and close this issue once it's done.
I also agree that a single space would be my baseline expectation. I'll probably start with that to minimize proliferation of options, but open to adding more configuration flexibility if you think alternative styles are fairly common.

Thinking a bit more about this -tclint may not want to check this inside curly braces, since the spacing does technically affect the contents of the braced word. For example, these two commands have a different result:
puts {one \
      two}
# output: one  two

puts {one\
      two}
# output: one two

In practice it may not matter, since I think most people default to "" for displayed strings (the main place where I imagine subtle whitespace differences count), but I'm not sure. That being said, I also think in most cases the curly braces make the newline escapes unnecessary! It depends how commands interpret their arguments, but in the example of sta::define_cmd_args which I think spurred this issue, it seems like they might not be needed: https://github.com/The-OpenROAD-Project/OpenROAD/blob/master/src/rcx/src/OpenRCX.tcl#L54-L64.
I guess that does beg the question of whether tclint could flag these unnecessary newline escapes... which unfortunately I think would be tricky given it's command-specific.

Very fair... I think it could still check newline escapes outside of curly braces. Maybe there's a larger issue of flagging when to use curly braces vs. quotes? Seems difficult though.

Likewise with inside double quoted strings.


from_reader implementation using &mut Read

I have some types where I cant consume the reader and thus not use the from_reader function - would it be possible to change the implementation so use it uses a &mut Read?

Could you share a minimal example of the code you would like to be able to write that doesn't work currently?

Playground still has no CBOR support.
But the structs field is private and it is a network socket.

Would it be possible to put together a minimal example using std::fs::File or something else basic as the reader, to show what doesn't work? It's fine if it needs to be built outside of the playground.

probably related: https://rust-lang-nursery.github.io/api-guidelines/interoperability.html#generic-readerwriter-functions-take-r-read-and-w-write-by-value-c-rw-value

@pyfisch had the proper solution - kudos to you :D


Fix overriding handlers issue.

When a handler is passed as a prop to the Autocomplete module, the same handler is passed to it's child.
So, when the child's handler handles an event, it propagates the event to the handler of the props. Thus, the handler of the Autocomplete module was overwritten.
To avoid that issue, we should pass to the child the handler of the Autocomplete module, which in turns it will call the handler of the props.

Hello, sorry I was in holidays, I will check this asap.


Add possibility to use @defer/@stream on a fragment spread or an inline fragment

Commit is on top of c2ee0ba295135e83757d3794925b1594a8733c7c (the latest release) as for some reason I can run warp in master i keep getting an:
➜  ~ curl 'http://localhost:9000' -H 'Accept-Encoding: gzip, deflate, br' -H 'Content-Type: application/json' -H 'Accept: application/json' -H 'Connection: keep-alive' -H 'DNT: 1' -H 'Origin: file://' --data-binary '{"query":""}' --compressed
INTERNAL_SERVER_ERROR%

PS: feel free to close this if it makes no sense

This is not really supporting @defer on the Fragment.

Agreed i don't know what I am doing :-)

Just checked, indeed with this the response isn't streamed back but i get a full json instead.

the end goal is to stream those fragments (which isn't working now) and to provide the label back on the subsequent chunk:
{"data":{"node":{"__typename":"User","id":"726995694","timeline_list_feed_units":{"edges":[{"node":
.....
{"label":"ProfileCometTimelineFeed_user$stream$ProfileCometTimelineFeed_user_t

We will reopen this once when standard is properly defined. Thanks for your help


update the key in guidelines component

This PR resolves #163
Updated the individual guidelines to have the correct key as their indices in that array. This way the guidelines are bound to their indices.

Now when the key is console logged, we see only the indices of the guidelines as keys as shown -

I would request @charithccmc and @wdevon99 to please review this PR :)


Add text content to sections in main

Content provided in "content.odt" in assets folder. All text content prepared to be copied into document. Note the code keywords should be set using a span (see SNIPPETS for the span to copy/paste)

Section headers have been added along with sub-section menu lists. Structure in place for main content to be added.

Main content added in 2 sections with multiple relevant sub-sections. Footer and Advertising banner (other resources) content also added


To enable async logging mechanism for a specific logger

Describe your problem(s)
As per the current implementation of the APIM 3.1.0 server, we can not enable the async logging mechanism for a specific logger. It throws the below error when enabling async logging via the log4j2.properties configuration.
org.ops4j.pax.logging.pax-logging-api [log4j2] ERROR : Log4J2 configuration problem: com.lmax.disruptor.dsl.Disruptor.<init>(Lcom/lmax/disruptor/EventFactory;ILjava/util/concurrent/ThreadFactory;Lcom/lmax/disruptor/dsl/ProducerType;Lcom/lmax/disruptor/WaitStrategy;)V Ignored FQCN: org.apache.logging.log4j.spi.AbstractLogger
java.lang.NoSuchMethodError: com.lmax.disruptor.dsl.Disruptor.<init>(Lcom/lmax/disruptor/EventFactory;ILjava/util/concurrent/ThreadFactory;Lcom/lmax/disruptor/dsl/ProducerType;Lcom/lmax/disruptor/WaitStrategy;)V
at org.apache.logging.log4j.core.async.AsyncLoggerConfigDisruptor.start(AsyncLoggerConfigDisruptor.java:231)
at org.apache.logging.log4j.core.config.AbstractConfiguration.start(AbstractConfiguration.java:296)
at org.apache.logging.log4j.core.LoggerContext.setConfiguration(LoggerContext.java:579)
at org.apache.logging.log4j.core.LoggerContext.start(LoggerContext.java:285)
at org.ops4j.pax.logging.log4j2.internal.PaxLoggingServiceImpl.configureLog4J2(PaxLoggingServiceImpl.java:439)
at org.ops4j.pax.logging.log4j2.internal.PaxLoggingServiceImpl.updated(PaxLoggingServiceImpl.java:272)
at org.ops4j.pax.logging.log4j2.internal.PaxLoggingServiceImpl$1ManagedPaxLoggingService.updated(PaxLoggingServiceImpl.java:581)
at org.eclipse.equinox.internal.cm.ManagedServiceTracker$1.run(ManagedServiceTracker.java:279)
at org.eclipse.equinox.internal.cm.SerializedTaskQueue$1.run(SerializedTaskQueue.java:40)

Describe your solution
As per the official log4j2 document 1 , to enable async logging, disruptor-3.3.4.jar or higher is required. Currently, the available disruptor bundle version in the APIM 3.1.0 is 3.3.2.wso2v1. Therefore we need to upgrade the disruptor bundle version and then we can enable the async logging for a specific logger by defining the logger type as below in the log4j2.properties configuration.
Please refer to the below sample log4j2 configurations to enable the async logging for the synapse-wire logger
logger.synapse-wire.type = AsyncLogger
logger.synapse-wire.name = org.apache.synapse.transport.http.wire
logger.synapse-wire.level = DEBUG

We are already using the disruptor version 3.4.2.wso2v1 in the latest public branch. Hence closing.


Support export/output of test run results for CI/CD integration

Feature Request
Description
Details are TBD
Describe the solution you'd like

Describe alternatives you've considered

Additional context

By means of #64 we now have what we need to expose the results to a CI pipeline.
The result files will be exposed through a webserver that job the runner (Jenkins, CircleCI, ...) can hit and download the data from and then move it to the appropriate storage service for archival.
That's a generic approach that should work with most scenarios. Until we have more specific needs, we shouldn't have to add more complexity.

By means of #64 we now have what we need to expose the results to a CI pipeline.
The result files will be exposed through a webserver that job the runner (Jenkins, CircleCI, ...) can hit and download the data from and then move it to the appropriate storage service for archival.
That's a generic approach that should work with most scenarios. Until we have more specific needs, we shouldn't have to add more complexity.


Customize Styleguide footer

The footer currently shown on the styleguide says

"Generated with Vue Styleguidist"

which isn't helpful at all. A better footer is needed which is more informative.
I particularly like the footer of the Carbon design system.

Our own Footer component is pretty cool too.
This would also resolve #43.

Hi, Can I work on this issue?

Sure, take it up

Hi, does this footer needs to be implemented on each page? I saw the elements, "Generated with Vue Styleguidist" is there on almost every page.

Yeah, the idea is to replace the "Generated with Vue Styleguidist" with our footer.

This plan has been dropped in favour of #6, which eliminates Vue Styleguidist entirely.


Provide step-by-step CLI-only instructions to install and setup rebar

Goal: to provide Step-by-step instructions to install and setup Digital Rebar from the command line
Tasks:

[ ]  provide Step-by-step instructions to install and setup Digital Rebar from the command line

[ ]  include configuring the subnet for DHCP, workflows, and any other required setup

Please review: https://provision.readthedocs.io/en/tip/doc/quickstart.html
This has all the steps for install DRP on an existing system via the CLI.  DHCP is not needed for Packet.


Adapt file headers to Lint specifications

What is the expected enhancement?
When running the test suite, specifically 'lint' tests for code quality, the same message is displayed for every file in the project:
************* Module qiskit.tools.visualization qiskit\tools\visualization.py:1:0: C5001: File header should match regex "(?:(?:#[^\n]*)?\n)*# This code is part of Qiskit.\n#\n# \(C\) Copyright IBM [0-9, -]*.\n#\n# This code is licensed under the Apache License, Version 2.0. You may\n# obtain a copy of this license in the LICENSE.txt file in the root directory\n# of this source tree or at http://www.apache.org/licenses/LICENSE-2.0.\n#\n# Any modifications or derivative works of this code must retain this\n# copyright notice, and modified files need to carry a notice indicating\n# that they have been altered from the originals.\n" (invalid-file-header)
This message, with some variations, appears hundreds of times in the log. That makes it a bit difficult to keep track of the other code smells that should be fixed. Setting a new format for the headers, or making Lint ignore this issue, would remove hundreds of log entries in one go.

I have never seen this issue using make lint in the source directory, nor when directly using pylint on a source file.

It appears then I use tox, that runs both the unit tests and lint.

@rafamartinc What platform are you testing on?

On Windows 10. It was just easier to use tox there. But when I switched to Ubuntu, make lint did not yield this error, so I'm not sure about what causes this... Any further details I can provide, that may be of help?

@rafamartinc this is a pretty common issue with pylint (one of many reasons I opened #1179). Pylint is very sensitive to the environment it's run in. The only way to get pylint to behave consistently is to pin pylint and anything packages or tools that it uses to an exact version. We've been doing this in the constraints file, but we're likely missing several packages there. But even with pinning pylint and astroid it still behaves differently for me between my desktop, laptop, and CI (although I've never experienced an issue with the header rule check).
Assuming you only have one version of python3 installed on your local system can you run pip freeze and .tox/lint/bin/pip freeze (from the repo root) and paste the output from both here. That will let us see all the packages in your tox venv vs what's installed in your system site-packages and we can add additional pins to constraints.txt based on that if needed.

Oh, and the other key difference I forgot is python version. We run in CI with python 3.5 (although I'm going to push a PR to update that to 3.7 shortly) and pylint will flag errors based on different versions (mostly because of stdlib differences, but sometimes for more subtle reasons). For example, the most recent example is:
on python 3.7 (which is my local default) lint will fail with:
************* Module qiskit.pulse.commands.command
qiskit/pulse/commands/command.py:32:4: W0221: Parameters differ from overridden '__new__' method (arguments-differ)

because 3.7 changed the signature for __new__ in the abc.ABCMetaclass after python 3.5 (not sure if it was in 3.6 or 3.7)


[Improvement]LocalStorage init use multi thread #71

issue: #71

Test case is needed for this improvement

Could you modify the description according to our pull request template? We will use the description as commit message,  the commit message will provide more information to help people understand the code.

Codecov Report

Merging #72 (c2b8a6d) into master (aa18be0) will increase coverage by 0.57%.
The diff coverage is 81.81%.

@@             Coverage Diff              @@
##             master      #72      +/-   ##
============================================
+ Coverage     56.39%   56.96%   +0.57%
+ Complexity     1173     1052     -121
============================================
  Files           149      136      -13
  Lines          7953     6755    -1198
  Branches        761      649     -112
============================================
- Hits           4485     3848     -637
+ Misses         3226     2692     -534
+ Partials        242      215      -27

Impacted Files
Coverage Δ

...he/uniffle/server/storage/LocalStorageManager.java
54.25% <81.81%> (+3.62%)
:arrow_up:

...storage/handler/impl/DataSkippableReadHandler.java
81.25% <0.00%> (-3.13%)
:arrow_down:

...org/apache/uniffle/server/ShuffleFlushManager.java
76.70% <0.00%> (-1.71%)
:arrow_down:

...e/uniffle/server/storage/SingleStorageManager.java
65.57% <0.00%> (-1.64%)
:arrow_down:

.../apache/uniffle/coordinator/ClientConfManager.java
91.54% <0.00%> (-1.41%)
:arrow_down:

.../java/org/apache/hadoop/mapreduce/RssMRConfig.java

...java/org/apache/hadoop/mapred/SortWriteBuffer.java

...n/java/org/apache/hadoop/mapreduce/MRIdHelper.java

...mapreduce/task/reduce/RssInMemoryRemoteMerger.java

...pache/hadoop/mapreduce/task/reduce/RssShuffle.java

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Embedding First Frame as Thumbnail/Cover Art

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I am unsure how/if it is possible to embed the first frame of the video instate of the video thumbnail.
I know using ffmpeg might be able to do something like that but I looked around but couldn't find any answers to do it with yt-dlp without running a secound line.
I'm asking this because of youtube music thumbnails are low res. (Sorry For Any Spelling errors)
Verbose log
No response

I know using ffmpeg might be able to do something like that but I looked around but couldn't find any answers to do it with yt-dlp without running a secound line.

How do you do this with ffmpeg?

I know using ffmpeg might be able to do something like that but I looked around but couldn't find any answers to do it with yt-dlp without running a secound line.

How do you do this with ffmpeg?

 ffmpeg -i <url> -map 0:a:0 -map 0:v:0 -id3v2_version 3 -metadata:s:v comment="Cover (Front)" out.mp3

Then the equivalent yt-dlp command would be
yt-dlp --use-postprocessor FFmpegCopyStream --ppa "copystream:-id3v2_version 3 -metadata:s:v comment=Cover\ (Front)" "URL"

Noted Thank You.

Changed title so other can find this issue easier when searching.

Can you help me here I can't seem to work it out.
-x
--audio-format mp3
--audio-quality 320K
--format "bestvideo+bestaudio/best"
--concurrent-fragments 50
--sponsorblock-mark all
--embed-metadata
--use-postprocessor FFmpegCopyStream --ppa "copystream:-id3v2_version 3 -metadata:s:v comment=Cover\ (Front)" "URL"
--embed-thumbnail
-o "D:\Downloads\%(artist,uploader,channel)s\%(artist,uploader,channel)s - %(track,title)s.%(ext)s"

Can you help me here I can't seem to work it out.
-x
--audio-format mp3
--audio-quality 320K
--format "bestvideo+bestaudio/best"
--concurrent-fragments 50
--sponsorblock-mark all
--embed-metadata
--use-postprocessor FFmpegCopyStream --ppa "copystream:-id3v2_version 3 -metadata:s:v comment=Cover\ (Front)"
--embed-thumbnail
-o "D:\Downloads\%(artist,uploader,channel)s\%(artist,uploader,channel)s - %(track,title)s.%(ext)s"

I don't think 50 concurrent fragments is needed for one audio file.
I used:
yt-dlp -N 2 --windows-filenames --check-formats --no-write-subs --no-write-thumbnail -f "ba" -o "%(title)s-%(artist)s[%(id)s.%(ext)s" --use-postprocessor FFmpegCopyStream --ppa "copystream:-id3v2_version 3 -metadata:s:v comment=Cover\ (Front) [URL]"

You could modify it so it matches your code more closer.
I no longer use yt-dlp to embed metadata and cover images, so will not work if anything has changed that might have broken it but it seem to work fine. I use a second program to add coverart and metadata as it has higher res coverart and more detailed metadata.

I use a second program to add coverart and metadata as it has higher res coverart and more detailed metadata.

Might I ask what program is that? Since it is better, also pls state if it can add metadata and coverart in batch or manually one-by-one.

Might I ask what program is that?

I use a app on my android phone called 'AutomaTag'.
It can do batch but it's manual(You can select multiple files but it will show you them one by one for you check and press next).
The Full auto batch is Paid tho... I don't use the full auto because it can misidentify songs as it uses the title and/or the metadata in the file to try to find the right one(It does shows more than one option for you to pick.)


Fetch latest "current" browser agent from v1228 onwards

Replace manually updated agent snippet strings with runtime fetch for the current scripts from CDN.
Also, the configs should go before the agent, so re-arranged the order.

@metal-messiah @cwli24
hi!
are there any plans to release 2.3.1 with gatsby 5 support
I am seeing this warning

@metal-messiah can you clarify reasoning behind closing this pr?


Actually make api protos a submodule and update them to latest master

What was changed
In title
Why?
I needed the latest versions of the API protos and realized that although the docs say that the protos are a submodule they weren't.

We should just change the docs to say subtree and include the instructions for updating them - submodules are comparatively a pain in the butt to update properly IMO. Subtrees on the other hand just work when someone pulls the repo. Unless we've got a compelling reason to switch I think I'd prefer to keep it as it
This is the right command for updating I believe:
git subtree pull --prefix protos/api_upstream/ git://github.com/temporalio/api.git master --squash

Subtrees are a different type of pain from my experience but I'm fine with using them here since we never edit them directly from this repo.
I was confused by the docs and didn't realize we were using subtree.

I'll update the readme in https://github.com/temporalio/sdk-core/pull/195

Whoops, didn't mean to close, we can use this to update the subtree


No description of zip, packageheader and manifest file

There is no documentation on how the zip file should look like, particularly the packageheader and manifest files. There is no way to tell what each attributes means and if they are mandatory or not.
Any proper product should be accompanied with solid documentation.

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

ID: 01b2cd6b-f885-7461-fbdf-cf81f9e50d86
Version Independent ID: 8df7a6ae-67db-5d29-fb6a-0820704ff3ef
Content: Recurring integrations - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/recurring-integrations.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

Thank you for the input on our documentation. We really appreciate it.

Created bug 582880 to track this issue.

Thank you for the feedback.
I believe the documentation you are looking for is in the topic, Process and consume data packages in Dynamics 365 Finance and Operations apps solution. The recommended approach is to automatically create the data package, zip file, as described in step 6-10 in the beginning of the above topic. There is no need to try to manually create the data package file.


revert: vite upgrade

This reverts commit 4d5f54b5cf86e1a0c9ab38d202b6c63da8d23808, reversing changes made to 3d3ed2f8d9cee8ed5336f5e5f33c340ed087fa2e.
PR Checklist

[ ] Includes changes to desk files
[ ] Describes how you tested the PR locally (test ship vs livenet)
[ ] If a new feature, includes automated tests
[ ] Comments added anywhere logic may be confusing without context

I just pushed something like this


Modernize djbdns_rr

Use:

converge_by to support why run mode
state_attrs to support chef reporting
use_inline_resources

use_inline_resources added, but still no why-run


feat: RBAC authentication contract and role library

Roles.sol is a library for convenience functions on the Role struct that maps addresses to whether or not they have this role
RBAC.sol is a contract providing storage and accessors for the roles, indexed by strings (which are compacted in memory)

Questions

is it worth indexing roles by an Enum instead? This would require some casting to uints
are events something RBAC.sol should be concerned with?

looks like a flaky test. is there a re-run option?

Thanks @Shrugs!
I was thinking of having RBAC be Ownable and exposing admin functionality to the owner. I feel like it will always be needed and we should aim to reduce boilerplate. What do you think?

is it worth indexing roles by an Enum instead? This would require some casting to uints

I understand the motivation, but I think I prefer strings to uints. They both have issues though... If we go with strings we should probably recommend that users of the library define constant variables with their role names.

are events something RBAC.sol should be concerned with?

Do you mean events as in e.g. RoleAssigned(string, address)? Yes, absolutely. I think RBAC should be like a more general Ownable.

@frangio what do you think about making RBAC ownable, but using its own access control scheme? i.e., not using Ownable, but providing a default "owner" role and making msg.sender owner by default, and providing an onlyOwner modifier and transferOwnership method? I think it'd be weird if RBAC depended on Ownership.sol.
I'll add the events 👍

Although, how important is the owner public variable? If we want to match the Ownable interface, we'd need an owner() getter, which requires storing the current owner address.
We can either keep track of the owner (in which case we just inherit from Ownable and call it a day) or we can deviate from the interface and end up with something like:
// should we prefix this with Log to match current standards, or keep it as it is to match the previous interface? if we're going to deviate from the ownable interface at all, I'd prefer to update this to include Log as well.
event OwnershipTransferred(address indexed previousOwner, address indexed newOwner);
function isOwner(address addr) external returns (bool);
function transferOwnership(address addr) onlyOwner public;
modifier onlyOwner();

and users can see if a specific address is owner by calling isOwner(address).

Hm, okay, it might be a bit weird/confusing to have two different permission mechanisms in one contract (Ownable + RBAC).
Let's not worry about matching the Ownable interface, and just have a specific role who can add and remove addresses from roles. Since we're deviating from Ownable, how about calling the role:
const ADMIN_ROLE = "admin";
I'm guessing this means making RBAC#addRole (etc.) not internal but public and only callable by an "admin". The view functions should also be public IMO. This makes it very easy to use I think.

If we're going to bake in the ability for admin's to add/remove roles, there are two api surfaces; the internal, no-constraint CRUD operations and the onlyAdmin management operations.
So I've added adminAddRole and adminRemoveRole which are onlyAdmin public, which should cover most usecases.
I also rebased on master.

NO longer interest

@frangio I've updated the event names and tested them, as well as rebasing on master to include the recent changes

Great! I'm ready to merge once the linter errors are fixed. 😃

😅 good catch; I should set up a precommit hook instead of hoping my editor catches it.

Hehe. We should look into how to make that easy for other developers too. 🙂
Thanks @Shrugs!

Xxcccc
On4 December 2017 23:30:47 GMT+08:00, Francisco Giordano<EMAIL_ADDRESS>wrote:

Merged #580.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/OpenZeppelin/zeppelin-solidity/pull/580#event-1370704621


Configurable edgex secret store

PR #20 adds the support for edgex-security option set to on/off. This is fine in most settings but is not extensible to allow other ways to provide the secret. Moreover, it isn't so intuitive because it sets the "edgex-security" to off even though the redis credentials may be passed via other means to allow secure connections. What it really does is to disable redis credentials setup. The redis credential setup involves (1) querying credentials from a vault (local, default port) using the vault token provided by the content interface and (2) injecting credentials into config files.
We can get inspiration from toml secret store section in configuration files of other services. E.g. device-mqtt:
[SecretStore]
Type = "vault"
Host = "localhost"
Port = 8200
Path = "device-mqtt/"
Protocol = "http"
RootCaCertPath = ""
ServerName = ""
# Uncomment to seed service secrets file
# SecretsFile = "/tmp/edgex/secrets/device-mqtt/secrets.json"
DisableScrubSecretsFile = false
TokenFile = "/tmp/edgex/secrets/device-mqtt/secrets-token.json"
    [SecretStore.Authentication]
    AuthType = "X-Vault-Token"

No longer relevant. EdgeX secret store usage is now disabled with config.edgex-security-secret-store=false.


fix(v2): expand broken admonitions

Motivation
Closes #2452 and fixes the templates used by @docusaurus/init
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan

Created a new project using @docusaurus/init using the fixed template.
Previed updated docs
Ran prettier-docs and confirmed the admonitions formatting was not changed

Yeah I think it should be
:::important

This is important

:::

Correct. The regex I ran had a typo and it was late enough I didn't notice ☕
I've fixed it to be the actual proper formatting.


Fix accuraccy issue in resnet50

I still see some accuracy issues for resnet50i1 with ef4189f in the concat-fix branch.  Different results, but still not accurate:
50000 top1: 58 top5: 289
Overall - top1: 0.00116 top5: 0.00578

inceptioni1 still gives expected results
50000 top1: 38823 top5: 46883
Overall - top1: 0.77646 top5: 0.93766

TF resnet50v2i1 now gives expected results (it used to core dump)
50000 top1: 35384 top5: 45007
Overall - top1: 0.70768 top5: 0.90014

TF mobilenetv2i1 also now gives expected results (it used to core dump)
50000 top1: 35059 top5: 44694
Overall - top1: 0.70118 top5: 0.89388

With latest fixes, now see expected accuracy for resnet50i
50000 top1: 37655 top5: 46261
Overall - top1: 0.7531 top5: 0.92522

Checking other models and will update this comment when done.


glbajajgroup.org

GL Bajaj Group Of Instituions ,Mathura
URL: glbajajgroup.org

@ownway08 glbajajgroup.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!

@ownway08 glbajajgroup.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!

I had made a new request please review it.


[WIP] Arraylias integration - Rotating frame -

Summary
I started to migrate Arraylias to rotating_frame.py.
The first difficulty is dealing with csr_matrix and BCOO.
I created a new file of arraylais_matrix and used alias.register_function for handling both of them.
Details and comments
will be updated as needed.

The to_array, to_csr, to_BCOO, and to_numeric_matrix_type are things that we'll need to think carefully about when converting to arraylias. Dynamics has been very loose with array types, and that has been partly enabled by these functions, which are used throughout the package to funnel a variety of types (including lists, lists of arrays, etc) into array and sparse types. It would be nice to not need these methods, but I'm not sure to what extent we can avoid using them.
One option would be to just start being very strict about types. E.g. the RotatingFrame methods could all require the inputs be ArrayLike (assuming csr_matrix and BCOO become registered Arraylias types). The reason I hesitate with fully going this direction is that it's natural to specify operators in models in a variety of ways, either as raw arrays/sparse matrices, qiskit Operator/QuantumState instances, qutip Qobj, or lists of these. We could just force users to convert these to their desired array types in most places, and make the to_* functions into publically available helper functions. I wouldn't want to do this everywhere, e.g. in Solver.solve, there is some high level handling of QuantumState types that I wouldn't want to get rid of.
Aside from these concerns, I don't think we want to register csr_matrix or BCOO as straight 'numpy'/'jax' types. Both require special syntax, so I think it will be necessary to register them as their own libs. E.g., in arraylias_state.py, I think we can do something like (with suitable try blocks in case JAX isn't installed):
DYNAMICS_ALIAS.register_type(csr_matrix, lib="scipy_sparse")

"""
register required custom versions of functions for csr type here
"""

DYNAMICS_ALIAS.register_type(BCOO, lib="jax_sparse")
"""
register required custom versions of functions for BCOO type here
"""

I think for scipy, there are also multiple different sparse types that we'll need to register (they get transformed into each other). The scipy.sparse function issparse is used to check for all of these types. Maybe we can look there for all of the types.
Some considerations with functions that need to be registered:

For some reason, the sparse types in scipy use * for matrix multiplication instead of @. There are places in RotatingFrame where the issparse checks for a sparse type, and a line is implemented using *. I think this type checking can be eliminated with Arraylias if we convert matrix multiplications in RotatingFrame to be written with unp.matmul, and we register lambda x, y: x * y as the definition of matmul for lib='scipy_sparse'.
The BCOO type has been set up interestingly in JAX to use the sparsify transformation. This takes a JAX function meant to act on dense arrays, and converts it into one that also works with sparse arrays. (It doesn't work with absolutely everything in JAX, but it does for everything we'll care about.) Checkout the JAX documentation on this, and also the operator_collections.py file for examples in Dynamics. I think the things being registered in operator_collections.py can probably be registered in the main arraylias_state.py file to be used throughout the rest of Dynamics.
The asarray method will need to be registered in both cases, and I think this is what your 'csr_matrix' registered function should be.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: DanPuzzuoli:x: to24toroYou have signed the CLA already but the status is still pending? Let us recheck it.

close and move to https://github.com/Qiskit-Extensions/qiskit-dynamics/pull/282.


Impossible to serialize associations with EntitySerializer

In updateQuery method, updateSelectQueryPart called without last param, which is false be default
https://github.com/orocrm/platform/blob/master/src/Oro/Component/EntitySerializer/EntitySerializer.php#L343

The "to-one" associations are added by the next code. Please see https://github.com/orocrm/platform/blob/master/src/Oro/Component/EntitySerializer/EntitySerializer.php#L347-L366. The "to-many" associations are loaded by another method. Please see 'loadRelatedData'.
Could you please describe the issue in more details?

No response from @ishakuta yet so I'm closing it for now.


Document whether Compile avoidance for Kapt is still needed as of Gradle 6.8

gradle 6.8 has been released and mention compilation avoidance by default for Kotlin.
https://docs.gradle.org/6.8/release-notes.html#performance-improvements
Kapt, the Kotlin annotation processor supports compilation avoidance
https://kotlinlang.org/docs/reference/kapt.html#compile-avoidance-for-kapt-since-1320
by adding this gradle property
kapt.include.compile.classpath=false
Is this compilation avoidance now obscolete/redundant or is it complimentary/hortogonal with the 6.8 improvements ?
aside: I've heard that gradle team is working on an idiomatic/best practices project, maybe the answer to this issue should be included in such project ?

Afaiu, setting the property kapt.include.compile.classpath=false is still required. That property prevents the annotation processors that are declared with dependencies { ... kapt(...)} from being included in the compile classpath (they are only added to the kapt config then). This is a setting of kapt, not Gradle, and the latest kapt documentation still says they same:

However, compile avoidance can't be used for annotation processors discovered in the compile classpath since any changes in them require running the annotation processing tasks.
To run kapt with compile avoidance:

Add the annotation processor dependencies to the kapt* configurations manually as described above.

Turn off the discovery of annotation processors in the compile classpath by adding this line to your gradle.properties file:

kapt.include.compile.classpath=false

Gradle 6.8 only seemed to add compile avoidance for the Kotlin build script (build.gradle.kts) itself:

Until now, any change to build logic in buildSrc required all of the build scripts to be recompiled. This release introduces compilation avoidance for Gradle Kotlin DSL scripts.
Compilation avoidance will cause Gradle to only recompile build scripts when a change to shared build logic impacts the ABI (application binary interface) of the classpath of the build script. Changes to private implementation details of build logic, such as private methods or classes, bodies of non-private methods or classes, as well as internal changes to precompiled script plugins, will no longer trigger recompilation of the project's build scripts.
Compilation avoidance also applies to changes in any JAR on the build script's classpath. That includes JARs added by plugins define in included builds and JARs added directly via the buildscript {} block.

I'm not sure how to test whether compile avoidance for kapt is working though.

Closing as there is nothing to be done in Gradle at this point.


Add a scan QR Code from camera feature

In the spend page, it will be usefull to had a scan QR code from camera (or import image/screenshot)

which crate should we use for get the camera stream?


When using SSE there is not enough information to exercise the API

This issue summaries a request from @olinux and makes some proposals for a solution:
Using SSE in kg resources you get back the projectUUID and the resource ID information. However, our API does not support to use the projectUUID on the URI (/v1/resources/{projUUID}/_/{resourceID}).
There are several solutions:

On the admin service, add a lookup endpoint: /v1/projects/_/{projectUUID}/lookup
On the kg service support both labels and UUIDs /v1/resources/{orgUUID}/{projectUUID}/_/{resourceId}
In this case, we have to add the organizationUUID to the SSE (which we have to add anyway in order to support SSE per organization).
On the kg service support both labels and UUIDs, with the character _ for the organizationUUID /v1/resources//{projectUUID}//{resourceId}

Some other suggestions are welcomed.

I think that since the UUID uniquely identifies the project we could simplify the API call to /v1/resources/{projectUUID}/_/{resourceId}. In this case we wouldn't need to add the organizationUUID to the SSE for this use case.

I updated the first comment to include @wwajerowicz option, but keeping the API segments consistent

I agree that (since the project determines the organization), the comment of @wwajerowicz is valid for the (technical) resolution of the resource. The information about the organization would be interesting in SSEs nevertheless, since we make use of it for dispatching. Not providing it in the SSE would mean, that we need to resolve the project first to extract the organization information. Adding it to the SSE therefore would simplify our code a lot and I can imagine other consumers of the SSE might be interested in this kind of information as well.

I agree that adding this information to the events would be very useful. With respect to the API, I think we can efficiently handle UUIDs in the segments and I generally like the idea of using _ as a wildcard for cases where the organization is not needed:
/v1/resources/_/{project_uuid}/_/{resource_id}
/v1/projects/_/{project_uuid}
/v1/orgs/{org_uuid}
etc.
When the wildcard is not used, we can check if the provided organisation and project match correctly. I have mixed feelings about the use of mixed labels and uuids, but since they do not pose a technical challenge or feature ambiguity we can allow them.

@olinux the final solution has been the following:

Added _organizationUuid to the Events through SSE
Added the possibility the exercise the APIs using UUIDs or labels indistinguishably

Those two items should fix your issues.


Please merge https://github.com/dart-lang/sdk/commit/11d340c51d7d916c3c4068c341c6e5fe48a6bf40 to stable

Please merge
https://github.com/dart-lang/sdk/commit/11d340c51d7d916c3c4068c341c6e5fe48a6bf40
to the 1.18 stable branch.
This CL fixes issue https://github.com/dart-lang/sdk/issues/26948 raised by the fiber team.
@crelier @zanderso

I have created a new dev channel release, 1.19.0-dev.1.0, which includes this fix.  I would like to see it go through all our testing, including internal testing, before patching to stable.  Does this sound good to you?

Sounds good to me.Thanks.

Merged in 1.18.1


Articullate api is responding very slow

Issues
Hi I am using articulate api version 0.0.1 along with rasa NLU version 0.13.0. Whenever i call agent converse api i get very slow response i also did the stress testing by sending 25 concurrent requests results of that test is attached

You can clearly see from the above image that the mean response time is about 25 seconds.
I am using CentOS machine with 4 cores and 8 gb memory other services are also running on that machine but they are taking very less whenever i start the stress test the CPU usage of articulate shoots upto about 300% and average load reaches upto 4.
What could be the problem here ?
Do let me know if any other information is required

Can you tell me how many domains you have? can we get a screenshot/video of docker stats while the test is taking place?

Same question asked on SO: https://stackoverflow.com/questions/54634557/spg-articulate-api-is-responding-very-slow

Hi sorry for the late reply actually i couldnt be able to test again on the same machine so i setup another machine with 8 cores and 16 GB RAM and tested on it the cpu usage reduced to about 100 to 150 % during test but the response time remained approximately same . There is only one domain in articulate. screenshots of docker stats during test and test results are attached kindly see

Can you describe your network/Docker setup? I notice you're connecting using https, but to a local IP?
I am working up some info on this issue today.

yes the api is listening on https with self signed certificates and this machine is on LAN.

I just noticed that your docker stats doesn't include the full stack, can you post a screenshot showing everything. The API CPU usage actually isn't what I want to see, I want to see what Rasa is doing.

Hi I have again tested it and recorded stats of Rasa, api, duckling and Redis. Please see the attached video. Kindly download the zip file and extract the video. Also results of stress test are also attached

stress-test recording.mov.zip


Start from curlconverter when implementing curl => Dart

I added Dart support to the curlconverter project that generates http client code from various languages. You can start there if you're implementing the same functionality in Dart.

@csells Good idea :) The URL's changed somwhat since that JS file has been renamed to a TS file 🤪 https://github.com/curlconverter/curlconverter/blob/master/src/generators/dart.ts


Bug diagonal mouvement

https://github.com/MehdiZouitine/gym_ma_toy/blob/0a692b59d2367c6201f3a4d6c0a47da82852939b/gym_ma_toy/envs/coop/game_base.py#L393
We cannot say that a diagonal movement is possible if we only know that both vertical and horizontal movement are available.
Here is the following case of error that can lead to the superposition of two agents:
If the diagonal agent A1 is in position (x,y) and (x-1,y) and (x,y+1) are empty then we cannot say that (x-1,y+1) and empty.
We need to check in addition to that that the diagonal box is empty.

fixed


seperated css and imported in html file

Signed-off-by: Sanket Gawande<EMAIL_ADDRESS>


IPython extension

Upon receiving the hup signal gets you an IPython shell that can load the environment of the function calling main-loop. Perfect to change a few hyperparameters by hand, or just to force a checkpoint.

Very nice! Since this is blocks-extras, I am happy to merge it, if you wish, you can add some docs later.

I have a report that Theano don't allow to kill execution with ctrl-C in ipython. Do you know about this?
https://github.com/Theano/Theano/issues/3064
It is dangerous to interrupt a Theano function call. I'm not sure this always go well. If a function have many updates, it could happend that some of them get applyed, but not all.
Just in case you have some information on that.

The code does two things: install a tiny signal handler which just sets a flag and the IPython runner which is run between batches, when it should be safe.
I don't know if the signal handler can break theano calls - it should exit very fast.

Ok, I think what you do is safe at a high level. I think the problem with
Theano is that it is a C function and that isn't interuptible by default.
At least, not in the way we do it. If you know how to do this, it could be
useful.
On Thu, Aug 6, 2015 at 6:58 PM, Jan Chorowski<EMAIL_ADDRESS>wrote:

The code does two things: install a tiny signal handler which just sets a
flag and the IPython runner which is run between batches, when it should be
safe.
I don't know if the signal handler can break theano calls - it should exit
very fast.
—
Reply to this email directly or view it on GitHub
https://github.com/mila-udem/blocks-extras/pull/17#issuecomment-128534788
.


Swap mapper iterates for 20 trials even if it finds a depth 1 circuit in the first try

What is the expected enhancement?
A common case is that with a single swap, you get the job done. In this case, there is no need to do more trials. The current swap mapper does 20 trials regardless, however. The fix is simple.

I think this is fixed by the current StochasticSwap pass. @awcross1?

@1ucian0 yes, StochasticSwap breaks out of the trial loop now if the depth of the swap circuit for a given layer is 1

great, closing.


fix screening package build

#294  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


Parameterize the index type of maps and sets

The index type is a new generic parameter defaulting to usize:
pub struct IndexSet<T, S = RandomState, Idx = usize>
pub struct IndexMap<K, V, S = RandomState, Idx = usize>

In no_std builds, S has no default, but still Idx = usize.
This may be used to optimized the storage size in the raw table, for
example using a u32 index if you know that u32::MAX capacity is
sufficient for you. It may also be useful to define a newtype index
custom to a particular domain, which only needs to implement the new
Indexable trait to be useful here.

Nice idea and pretty smooth implementation.
With the into/from usize conversion, we insert a lot more panic sites in the code. And I'd like to ensure we can't panic in places that leave the map inconsistent. (Is this a too onerous goal?)
The first usize conversion in .pop() looks like a problem. But it is actually guarded (global invariant) if we just ensure the map length never grows larger than what the index size can handle. However, it is not apparent if/how the insert method ensures this.
Then we document that insert panics if the index type's limits are exceeded etc.

Nice idea and pretty smooth implementation.
With the into/from usize conversion, we insert a lot more panic sites in the code. And I'd like to ensure we can't panic in places that leave the map inconsistent. (Is this a too onerous goal?)
The first usize conversion in .pop() looks like a problem. But it is actually guarded (global invariant) if we just ensure the map length never grows larger than what the index size can handle. However, it is not apparent if/how the insert method ensures this.
Then we document that insert panics if the index type's limits are exceeded etc. (And .extend() and .collect() et.c. 🙁 I'm just sad that it gets ugly in the details.)

@bluss it's not clear to me that a smaller index type needs special treatment – after all, the current logic also panics if an allocation can't be met (for whatever reason, possibly capacity > usize).

@bluss it's not clear to me that a smaller index type needs special treatment – after all, the current logic also panics if an allocation can't be met (for whatever reason, possibly capacity > usize).

@malthe That's too hand-wavy, the exact line and place where we can panic matters for which inconsistencies can show up in the data structure.

@malthe That's too hand-wavy, the exact line and place where we can panic matters for which inconsistencies can show up in the data structure.

@bluss but if we panic, what's left of the program? Is it not just going to halt?
Another issue I found was that it feels inconsistent that len() should return a usize when the index parameter is something different.

@bluss but if we panic, what's left of the program? Is it not just going to halt?
Another issue I found was that it feels inconsistent that len() should return a usize when the index parameter is something different.

Panic is recoverable, so that's why it matters.

Panic is recoverable, so that's why it matters.

@cuviper I have begun some rebasing against the current master on this one, but let me know if you already have something cooking.

Len should probably continue to return usize, just to note - without looking deeper.

@malthe Sorry, I don't know how far you got, but I just pushed a rebase myself. I was responsible for a lot of the conflicting changes anyway, so I had a good idea how to solve things.
I have mixed feelings whether the length and capacity should be usize or Idx. As usize, there's a small bonus that the maximum like 255u8 is a usable index, since we can still report length 256. As Idx, we would at least need Indexable to tell us its maximum value, so we can saturate when RawTable over-allocates. (e.g. RawTable::with_capacity(224) will allocate 256 and report capacity 224 exactly, but with_capacity(225) allocates 512 and reports capacity 448 -- too high for Idx = u8.) It might make some of those Idx <-> usize panics easier to deal with if the maximum is a sentinel though.

@cuviper yeah I think it has to be usize – but perhaps there could be a function that returns a range, i.e. 0..length - 1 which then would be type-safe on the index type.

To get some real data, I tried to use this branch in rust-lang/rust#96751 and its perf infrastructure. For instructions, there were a few small wins, but nothing better than 1% improvement, and one case regressed 1.68%. Maybe that's a good thing that it was mostly neutral, since this has added usize conversions in a lot of places. But max-rss didn't improve either, just looks like noise.
It's possible that I just didn't make good use of it, but those were all the places I could find that clearly dealt with smaller indexes. Maybe there are more significant maps that would be happy limited to u32::MAX items, just as the compiler often does with its IndexVec.
Overall, I'm not inclined to merge this if there's no clear winner, as the new parameter is pretty invasive.


Present Pinned Tracks in a Track Group

Sokatoa Issue
Presents Pinned Tracks in a minimalized collapsible track group.  Now also supports reordering tracks within this group.

Somehow it evens restores from a previous session in Sokatoa without any additional work

Since I'm storing the reorder by using the order of the pinnedTracks array in the state (which we use to track which tracks are pinned already), when we restore from the list, its in the same order.


Email fetching not working after restart

I had this issue already with the 1.9.12 version and thought it would be fixed in 1.9.14, so I am opening now an issue.
I have set up one "email" which fetches an IMAP account. This works fine, but when I stop and start the container again, it stops working. The strange thing is that i receive a "Mail Fetch Failure Alert" mail (Error: IMAP Authentication cancelled) which is send with the credentials of this fetching account that doesn't work. After entering and saving the password of that account, the fetching works again. I would have thought that the password is stored in the DB and so there is no influence on the stopping/destroying the container, as long as the mysql server is running?
Has anybody else run into this issue?

Are you setting the INSTALL_SECRET environmental variable?
OSTicket stores an encrypted version of the IMAP account password in the database. This is the SECRET_SALT variable in the source code. The value to use is declared and read in from the settings file.
This image will automatically generate a weak random value when you start a container if this is not specified. However, if you start a new image, a new value will be generated and this will not match the previous value, hence decryption of the IMAP password would fail.
If you set INSTALL_SECRET to some random value with sufficient entropy then this should prevent this problem.

Closed due to no response. Feel free to re-open if the issue isn't resolved.

@martin-css thank you very much! it seems that i missed that part in the installation instructions. sorry for bothering. and sorry for the slow reply, i was on holiday.

No problem, glad the problem is fixed.


Mean airway pressure is missing?

Hi, I checked airway pressure in lab table. It looks like mean airway pressure is missing, am I right? @tompollard @alistairewj

Not sure but I wouldn't think mean airway pressure would be stored in the lab table. It's likely in a respiratory table which we are currently working on releasing.