LOOKING FOR MAINTAINERS

Me and @Perdiga are the creators of this package,  however, we are not working with Flutter anymore, but we believe in the future of this package.
If you're working with flutter, and using this library, please consider becoming a Maintainer.
Please talk to me or @Perdiga  if you're interested.
Thanks a lot

hello @sonikro @Perdiga I am interested in maintaining this project.
would it be possible to set up a short informational video meeting so I can get some questions answered?
I'm a huge fan of your work so far, great job.

hello @sonikro @Perdiga I am interested in maintaining this project.
would it be possible to set up a short informational video meeting so I can get some questions answered?
I'm a huge fan of your work so far, great job.

Hello there. I'm glad to hear that. Feel free to contact me at<EMAIL_ADDRESS>so we can schedule a video call. I'm on timezone UTC-3.

hey @marktefftech are you currently maintaining this project? If so, would you want to start working on the web version and also null-safety upgrade for the official SDK.

Hello @Teajayy007 over the next few months I will looking to improve this project and will prioritize null-safety

Hello @Teajayy007 over the next few months I will looking to improve this project and will prioritize null-safety @marktefftech if you need an extra hand let me know. I can start working on the authentication for flutter web. Also, I think it already supports null-safety it just needs to be updated on the pub.dev officially.

@Teajayy007 that would be awesome. please shoot me an email and we can connect and decide on the best next steps<EMAIL_ADDRESS>

@marktefftech  @Teajayy007  Hi is there a way to replicate or make this run? I'm really interested but i have not been able to replicate even the example on flutter as it states that:
"
Warning
──────────────────────────────────────────────────────────────────────────────
Your Flutter application is created using an older version of the Android
embedding. It is being deprecated in favor of Android embedding v2. Follow the
steps at
https://flutter.dev/go/android-project-migration
to migrate your project. You may also pass the --ignore-deprecation flag to
ignore this check and continue with the deprecated v1 embedding. However,
the v1 Android embedding will be removed in future versions of Flutter.
"
if there is could you publish lates version to pub.dev?

does your flutter application support null-safety?

things to try:  upgrade your android SDK or change your min and targetSDKVersion to 21 and 30 respectively in build.gradle
if your flutter app supports null-safety, depend on the package from github like so and not pub.dev:
flutter_okta_sdk:
git:
url: https://github.com/sonikro/flutter-okta-sdk.git
ref: develop

@epinos Thanks for your comment. This is a community-supported project so appreciate your understanding on some issues.
The SDK does work with exceptions. Many of these have been fixed in recent PR's. I am trying to push out a build very soon.
In the interim for immediate debugging assistance you can see this project, which runs successfully on both platforms.

Thanks guys i really appreciate it. If theres anyway that i could help let me know :)


Fix #1409

The problem is that, when you read the file in /tmp like /tmp/node1 for me,
    "ansible_env": {
        "HOME": "/root",
        "LD_LIBRARY_PATH": "/opt/bin/pypy/lib:",
        "LOGNAME": "root",
        "MAIL": "/var/mail/root",
        "PATH": "/usr/bin:/bin:/usr/sbin:/sbin",
        "PWD": "/root",
        "SHELL": "/bin/bash",
        "SHLVL": "2",
        "SSH_CLIENT": "<IP_ADDRESS> 57727 22",
        "SSH_CONNECTION": "<IP_ADDRESS> 57727 <IP_ADDRESS> 22",
        "USER": "root",
        "_": "/opt/bin/python"
    },

The PATH didn't has /opt/bin in it, so the bootstrap can't find pip in PATH, we need set the execute of pip with {{ bin_dir }}

due https://github.com/kubernetes-incubator/kubespray/pull/2725, closed


Wrench doesn't work.

Steps to reproduce:

Pick up the Wrench (the active item that "gives a free item after using 3D printer and then breaks it")
Go to any 3D printer
Press a button to activate your active equipment.
Interact with 3D printer

Expected behavior: I get a free item that a certain 3D printer prints and then 3D printer breaks, becomes non-interactable. One charge of the Wrench is consumed.
What happens instead: no charge is consumed, 3D printer works as usual, taking one of my items and printing its shown item. It doesn't break after the first use, meaning that the Wrench has absolutely no effect.

Same issue here.

Just wanted to say I had it work in singleplayer with no problems as part of a large modpack so this may be a mod conflict rather than an inherent bug. Not yet confirmed working in multiplayer

Did you try looking at a printer with the wrench equipped? It should create this indicator over it, like a Royal Capacitor over an enemy.

I was playing multiplayer at the time, and the host tried, I tried, and one other person tried.
That icon did not show at that time.

The mod now ensures that a custom target searcher exists on the equipment slot every frame, preventing this bug from happening. Closing until this issue occurs again

Still happens. Turns out DebugToolkit somehow makes half of the mod work.


[LMS-488] Integration test

Adding unit testing and integration testing

SHOULD: We should add a way to locally run the test_extractor_integration tests. We can maybe add load_dotenv in conftest.py and provide guidance to either add a local .env file for tests in the same directory or provide the path to a local .env file.


release 22.2.0: problem with checksum files

Description
Hi, the content of https://dbeaver.io/files/22.2.0/checksum/dbeaver-ce_22.2.0_amd64.deb.sha256 does not match my locally calculated checksum for https://github.com/dbeaver/dbeaver/releases/download/22.2.0/dbeaver-ce_22.2.0_amd64.deb – the sha256 file contains 01deb3834681699775e28ec722ec509a1e275a69e62624c9612f47553f370c5f  and I get 00f5f29e380dc0898fda8d156f65b7bdb18596940e5fc2bb388d208cd6fed29a – and this used to match in many previous versions. Has something been changed in the way these files are distributed?
DBeaver Version
22.2.0
Operating System
Debian Linux
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Same issue.
File has unexpected size (111471768 != 111472372). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
  Hashes of expected file:
   - SHA512:3fda9c99ccef233bb8097f13ea2938bf3dcdc69cd9d870622ac1d99498e03aaf015966ae943f0e0aaf0a0f74004e595a8996fd52f6ed3f90dfb2e4b9ba5fd7b0
   - SHA256:00f5f29e380dc0898fda8d156f65b7bdb18596940e5fc2bb388d208cd6fed29a
   - SHA1:2dc90741cadc3922e6f1756cf026b30bff265cbd [weak]
   - MD5Sum:a42c218879fb007796aab5f1c8053f47 [weak]
   - Filesize:111472372 [weak]
E: Failed to fetch https://dbeaver.io/debs/dbeaver-ce/./dbeaver-ce_22.2.0_amd64.deb  File has unexpected size (111471768 != 111472372). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
   Hashes of expected file:
    - SHA512:3fda9c99ccef233bb8097f13ea2938bf3dcdc69cd9d870622ac1d99498e03aaf015966ae943f0e0aaf0a0f74004e595a8996fd52f6ed3f90dfb2e4b9ba5fd7b0
    - SHA256:00f5f29e380dc0898fda8d156f65b7bdb18596940e5fc2bb388d208cd6fed29a
    - SHA1:2dc90741cadc3922e6f1756cf026b30bff265cbd [weak]
    - MD5Sum:a42c218879fb007796aab5f1c8053f47 [weak]
    - Filesize:111472372 [weak]

A new version has been released (22.2.1) and this time the sha256 file at dbeaver.io matches the software file at Github. So the issue itself no longer exists. But I worry about the fact that this issue does not seem to have been noticed by anyone, judging by the lack of responses.

I should rephrase that: The issue does still exist for version 22.2.0.

It was a result of emergency hotfix version release.
We didn't re-uploaded github artifacts at that time.
Files on dbeaver.io are ok.
I don';t think that we should patch the past for now..

Yes, I can live with “the past remaining unpatched”, But this comment still helped me. I have an update script for dbeaver that downloads the .deb package and the checksum. I bleieve that when I wrote that script, the only place to download the .deb file was from Github (it’s possible that I overlooked something.) Now when I download this file from the website the download comes from dbeaver.com. I have now adapted my script to use https://download.dbeaver.com/community/${version}/dbeaver-ce_${version}_amd64.deb and I’m optmistic that in the future I am less likely to run into a discrepancy between that file and https://dbeaver.io/files/${version}/checksum/dbeaver-ce_${version}_amd64.deb.sha256. So I’m happy that I didn’t close the issue myself and grateful for the response. :-)


[dx11] add graphics pipeline creation & shader module creation

Adds basic pipeline creation and some plumbing for creating/converting from hal to dx11 state. The shader stuff is mostly from the dx12 backend without the root signature stuff, need to investigate how much patching needs to be done on the output (SM 5.0 vs 5.1)
PR checklist:

[ ] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:

@msiglreith Updated the error handling, but I couldn't get map() to work (.map() on Option<T> has to return Option<_>, but the create_pixel_shader returns Result<T, _> and so on).
I'll revisit this later but I think at least for now it reads more clear than having a series of transform operations on the return value and nested ? in closures.


Add support for custom service name

What does this PR do?
Closes https://github.com/DataDog/expo-datadog/issues/24
Adds a new serviceName option to support overriding the service name that is used when uploading the sourcemaps.

Adds serviceName option
Adds tests for withAndroidSourcemaps
Adds tests to check serviceName being added on iOS and Android
Adds tests to make sure datadog sdk configs are preserved if any other plugins set them

Motivation
When uploading sourcemaps by default the Service used will be the application's bundle identifier. This does not work for projects that set a custom serviceName in the RUM configuration during runtime initialization. For datadog to un-minify error logs the Service linked to the sourcemap needs to match the serviceName used by the RUM logs.
Additional Notes
Android
~For Android, the serviceName is set in the datadog sdk config in the gradle config. Since both withAndroidProguardMappingFiles and withAndroidSourcemaps can both edit the datadog sdk config the order these plugins run in matters. Currently, because the order is determined by a hardcoded map (ERROR_TRACKING_CONFIG_PLUGINS_MAP), the order is guaranteed. However, we still have to account for withAndroidProguardMappingFiles not being run if you pass in the config androidProguardMappingFiles: false to the expo plugin.~
Android Datadog config has moved to its own plugin so we don't have conflicts in which configs are applied.
iOS
For iOS, there is an existing method of overriding the serviceName. You can also specify a SERVICE_NAME_IOS environment variable. This is documented for the underlying datadog-ci command here: https://github.com/DataDog/datadog-ci/tree/master/src/commands/react-native#customize-the-command
Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)
[x] If this PR is auto-generated, please make sure also to manually update the code related to the change

add the option to dSYMs upload as well

Thanks for the feedback @louiszawadzki ❤
I've updated the PR to account for the Android config plugin as well as the iOS cli flag position.

add the option to dSYMs upload as well
For the dSYMs I was not able to find how to pass the service name for these uploads.

I'm not seeing a Service option in the underlying cli command here https://github.com/DataDog/datadog-ci/tree/master/src/commands/dsyms

Sorry you are correct about dSYMs, we use the file UUID to match it against errors, so there is no need to specify the service in this case! I'll take some time today to review and test your PR :)
By the way, if needed you can create a free Datadog Trial account here: https://www.datadoghq.com/free-datadog-trial/ :)

I think you're nearly there, you just need to sign all your commits as per our open source policy and we can merge!

@louiszawadzki Sorry about that. Updated all the commits to be signed.

Thanks a lot for all your work!
I will take care of the release and the documentation :)
We usually don't backport new features in the expo plugin as Expo upgrades are easier than RN upgrades, but I saw you mentioned it in the issue you opened.
Do you have issues when upgrading from Expo 49 to Expo 50?

Thanks a lot for all your work!
I will take care of the release and the documentation :)

Thanks @louiszawadzki, happy to help out. 😃

We usually don't backport new features in the expo plugin as Expo upgrades are easier than RN upgrades, but I saw you mentioned it in the issue you opened.
Do you have issues when upgrading from Expo 49 to Expo 50?

Ah, yes. Thank you for calling this out. While I sort of agree with the approach to simplify maintenance. Expo 49 is still an officially supported sdk version by Expo. Expo tends to support three versions at a time to allow time to upgrade (currently sdks 48, 49, & 50)
For larger projects with lots of dependencies and custom expo plugins it takes a bit of time to upgrade from my experience.
This specific issue ends up being a bit of a pain to patch since this plugin went through a minor refactor from 49 to 50.
It might be useful to backport changes to older sdk versions since this is missing functionality that makes it really hard (almost impossible) to get proper error logs.


unmanaged-cluster should allow for a skip-preflight flag

Feature Request
Some users might be just outside the CPU and memory limit for their environment when deploying unmanaged-cluster. Or, they might not be deploying very resource hungry apps to their cluster.
In those cases, we should allow the user to still deploy with a --skip-preflight flag
cc @stmcginnis for the suggestion in #3222
Describe alternatives you've considered
We should probably also log a warning informing the user that this is outside of the scope of what has been tested and verified for unmanaged-cluster

I believe we have this at
https://github.com/vmware-tanzu/community-edition/blob/fce1955a52c47c689f3220dfca29035dba5cc279/cli/cmd/plugin/unmanaged-cluster/cmd/create.go#L68
philosophically, we should always allow users to skip preflight as we can never predict corner cases.
@stmcginnis can you advise whether this satisfies the issue? If so, close it?

Yes, I believe that meets the need, and worked for the referenced issue. Was there something else you were getting at here @jpmcb ?
I'm going to close this since I don't think there is anything else to do, but please reopen if we're missing something yet.

Whoops! Spoke too soon, assumed we didn't have it.
Sorry for the noise! 🙊


Documentation Niche Configuration file>Full example

niche_vlaanderen version: 1.05b
Python version: 3.6.4
Operating System: Windows

Documentation
Is it possible to mention the floodplains block, as described higher, explicitely in the full example configuration file? And as such also in the text that is generated by the command line niche --example?

Closed in 0dd7d444f2ec4175b80a67466980a98d91c6e3d6


The demo floating button on the official website is invalid

Describe the bug
官网点击 浮动按钮 Float Button 时候，然后再点击案例的button后切其他组件仍然显示 浮动按钮

Steps to reproduce
1，点击组件的浮动按钮选项
2，案例的浮动按钮点击几次
3，再点击侧边栏就无效了

Link to minimal reproduction
https://www.naiveui.com/zh-CN/os-theme/components/float-button
System Info
npx envinfo --system --npmPackages 'naive-ui,vue' --binaries --browsers

Used Package Manager
npm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

repeat with https://github.com/tusen-ai/naive-ui/issues/5933
fix in https://github.com/tusen-ai/naive-ui/commit/a5c3b902cdc2ab8895f5c4c4e07c21d5a719abc8


add supprot to windowns/macOS/Linux

add support to many architectures to be able to use overlock.

if want/needed i can add workflow to pr as well.


Fix Maven build and test

"mvn test" is currently failing. I suspect it's an issue with Guava 20.0-SNAPSHOT:
https://github.com/google/guava/commit/1ef638f369964c1d1f56bd0ae588535c6abd99a4
Fix by skipping default mvn compile and test in pom-gwt.xml.
Also remove conflicting import statement and duplicate testGlobJs*() methods
from CommandLineRunnerTest.java to fix the Maven build.

Let's not "fix" it by just not compiling. If 20.0 is available maybe that will work.

@MatrixFrog The GWT compiler will still be invoke anyway with "mvn".  just disabled the default maven compile / test which uses the java compiler and does the same work as in pom-main.xml.
But yeah, let's hope Guava will fix this soon.

@MatrixFrog Ok seems like we have to use Guava 20.0-SNAPSHOT in both pom-main.xml and pom-gwt.xml for now.

ok lgtm once travis is done


Try re-enabling beta Rust tests in CI

Motivation
On 29 July, Zebra's beta Rust tests started failing, due to a beta rust bug (https://github.com/rust-lang/rust/pull/87483).
But the stable CI tests and beta zebra-chain build were fine.
Actions
We disabled the beta Rust tests in PR #2542.
We should try re-enabling beta Rust around mid-August. That way, we can get any bugs fixed before the current Rust beta release becomes stable in mid-September.

I removed docs from this ticket, because they aren't impacted by the beta rust bug.

I confirmed that this Rust PR fixes the test build failure we're seeing: https://github.com/rust-lang/rust/pull/87483
So we should be good to re-enable beta once it has been backported.

This has now been backported: https://github.com/rust-lang/rust/pull/88190


7z uncompress_archive_file fail

When I process 7z files, some 7z files will have errors, but if I use the command to decompression, it is normal.
thread 'main' panicked at 'Failed to get the file: Extraction("Truncated 7-Zip file body")'
let written = uncompress_archive_file(&mut source, &mut target, "tree/branch2/leaf")
        .expect("Failed to get the file");

Do you have a simple test case?

I didn't receive feedback since January, so I am closing it. Feel free to ask for a reopening if this still happens.


link-to.net isn't working

Link
https://link-to.net/623961/529.833070924929/dynamic?r=aHR0cHM6Ly9icm90aGVycGFyYWRpc2Uub25saW5lL25lQ1NFaW1Bb2E=
Version
Manifest-v3
What browser(s) are you seeing the problem on?
Firefox browsers
What platform are you seeing the problem on?
Computer (Windows, Linux, MacOS)
Anything else?
No response

Dupe #1419


javascript.suggest.names doesn't work if javascript.implicitProjectConfig.checkJs is enabled

This issue also happens in Insiders v1.39.0

VSCode Version: 1.39.0
OS Version: macOS 10.14.6

Steps to Reproduce:

Set "javascript.suggest.names": true
Set "javascript.implicitProjectConfig.checkJs": true
Notice that word based suggestions (in JS files) does not work
Set "javascript.implicitProjectConfig.checkJs": false
Notice that word based suggestions (in JS files) now work

Expectation:
Type checking (javascript.implicitProjectConfig.checkJs) should not have anything to do with word based suggestions (javascript.suggest.names).
Does this issue occur when all extensions are disabled?
Yes

This is by design but I'll update the documentation to make this more clear.
The reason: checkJs requires performs semantic analysis of the code. This means it needs to understand where symbols are defined and verify that they are used in a way that makes sense. For the example, consider the js:
//@ts-check
{
	const foo = 123;
}
{
	foo // trigger suggest here
}

Check js will consider the second usage of foo to be an error because it cannot find any variable called foo that is valid in the second scope.
However suggest.names is designed to suggest names of symbols from the entire file, even if they are not valid in the current location (such as suggesting foo inside the second block). That makes its behavior incompatible with checkJs. So rather than showing suggestions that would produce an error, we drop them


Change Username / Password authentication labels - Twilio Integration

Nikhil suggested that instead of Username / Password, can we have account SID and auth token in the datasource screen.
Note: It should be checked if we have control of the Labels “username” and “password” when creating a DataSource. Because require significant changes in SaaS App.

The Username and Password labels are defined by the Integration Datasource (check the Fetch_Integration_Dataosources query on the SaaS App).
To change this for the Twilio Integration and to allow customization for future integration, The SaaS App should include inputs to edit the default values of those fields.


Camera orientation cube in scene editor

Camera orientation cube in scene editor
Description

Add a clickable camera orientation cube in scene editor
Allow to ajust 2D camera rotation using the cube

Platform

Web Version [X]
Windows [X]
Linux [X]

Implemented!


Add React Next 2019

Add React Next 2019
Move past conferences to relevant section

The conference page seems to be updated right now. Thanks for sending PR and sorry for the late turn around time.


Use addon-dev declarations plugin for d.ts output

After landing https://github.com/embroider-build/embroider/pull/2200 in v7.1.0, we use the new plugin to emit type declarations.
Fixes #215

Yay!


fix(userspace): bump RLIMIT_MEMLOCK in any case

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area libscap-engine-modern-bpf
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR fixes an issue with libbpf. This issue is reported here<EMAIL_ADDRESS>We are applying the same patch of bpftool, i.e. we bump RLIMIT_MEMLOCK in any case even when the system uses a memcg-based memory accounting.
Which issue(s) this PR fixes:
https://github.com/falcosecurity/falco/issues/2626
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

LGTM label has been added.  Git tree hash: dc51338737348e0df124f981b8a6ab26ed51af11

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, incertum, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum,jasondellaluce]

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

/milestone 0.11.3


Renew Auth appears to be broken for new Auth0 tenants

Observed behavior (please include a screenshot if possible)
When a browser tab has remained open overnight and the auth token has expired, reloading the web page causes the page to appear frozen on a blank page. It appears that the renewAuth function is erroring out:
https://github.com/ibi-group/datatools-ui/blob/dfa01a0c21dbc5de8e5b9dc89471700a30759823/lib/common/user/Auth0Manager.js#L362-L389
Expected behavior
On page reload, the expired token should trigger a token renewal and the web page should reload normally.
Steps to reproduce the problem
Run localhost server with localhost configuration. Allow token to expire (perhaps to expedite this for testing, the token expiration time can be shortened via the Auth0 console). Reload web page.
Any special notes on configuration used
localhost config. May need to modify token expiration time to reproduce.
Version of datatools-ui and datatools-server if applicable (exact commit hash or branch name)
Running latest commites for deploy-to-ec2 branches, but I believe this also exists in latest dev.

The localhost callback urls weren't setup correctly. I have changed this in the Auth0 management profile. Please re-open if this is still a problem.


.NET 6 support

Considering the fact that .NET 6 is ready for production, I think it would be great if this package supports it.
Otherwise, the apps that have installed Microsoft.EntityFrameworkCore >= 6.0.0 will not be able to run Moq.EntityFrameworkCore.
Also, I needed to add Microsoft.NET.Test.Sdk to Moq.EntityFrameworkCore.Tests project in order to run the tests.

I will try to check it until end of this week.


Fix broken links and reference to conformance image

Conformance image is now maintained upstream and not in the
heptio/kube-conformance repo; fixed a reference to this.
In addition, some links to examples were in a format that would
link the docs between github *.md files but will not work on
the docs site. Updated those since the docs site is the primary
way these would be consumed.
Fixes #863

Codecov Report

Merging #864 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #864   +/-   ##
=======================================
  Coverage   46.83%   46.83%
=======================================
  Files          75       75
  Lines        4860     4860
=======================================
  Hits         2276     2276
  Misses       2452     2452
  Partials      132      132

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4737048...0c2ae45. Read the comment docs.


DRILL-5326: Unit tests failures related to the SERVER_METADTA

adding of the sql type name for the "GENERIC_OBJECT";
changing "NullCollation" in the "ServerMetaProvider" to the correct default value;

@laurentgo I changed the names for the server meta method.

LGTM

@laurentgo , @vdiravka , thanks. I'll run regression & merge.
+1


Error message for invalid string literal type could be improved

TypeScript Version:  2.0 Beta
Code
// A *self-contained* demonstration of the problem follows...
interface Foo {

    method(value: 'aa'): void;
    method(value: 'bb'): void;
    method(value: 'zz'): void;
    method(value: 'last'): void;
}

const x: Foo = {} as any;

x.method('bar'); // Error line

Expected behavior:
A more meaningful error message, e.g.
Argument of type "'bar'" is not assignable to parameter of type "'aa' | 'bb'|...|'last'"
Actual behavior:
Argument of type "'bar'" is not assignable to parameter of type "'last'"
I am finding this error message confusing, because the compiler picks the value of the last overload in order to report the error, and there is nothing special about the last value.

I've rated this at moderate difficulty because overload resolution isn't the simplest part of the compiler.
For what it's worth, I don't think the suggested error message is the best because it implies you could actually pass in a value of type "aa" | "bb" | ... | "last", but that's not the case.
Also see @sandersn's issue at #6541.


error for test_vistas.py

I try the test_vistas.py, something is wrong.
Traceback (most recent call last):
  File "test_vistas_single_gpu.py", line 311, in <module>
    main()
  File "test_vistas_single_gpu.py", line 188, in main
    probs, preds = model(img, scales, args.flip)
  File "/home/mingmin/anaconda2/envs/py36/lib/python3.6/site-packages/torch/nn/modules/module.py", line 477, in __call__
    result = self.forward(*input, **kwargs)
  File "test_vistas_single_gpu.py", line 135, in forward
    sem_logits = self._network(x, scale)
  File "test_vistas_single_gpu.py", line 117, in _network
    x_up = functional.upsample(x, size=scaled_size, mode="bilinear")
  File "/home/mingmin/anaconda2/envs/py36/lib/python3.6/site-packages/torch/nn/functional.py", line 1891, in upsample
    return interpolate(input, size, scale_factor, mode, align_corners)
  File "/home/mingmin/anaconda2/envs/py36/lib/python3.6/site-packages/torch/nn/functional.py", line 1985, in interpolate
    return torch._C._nn.upsample_bilinear2d(input, _output_size(2), align_corners)
TypeError: 'float' object cannot be interpreted as an integer

Then i modify the code in the function _network(self, x, scale):
 scaled_size = [s * scale for s in x.shape[-2:]]
to
scaled_size = [int(s * scale) for s in x.shape[-2:]]
No error occurs. But the output image is wrong. No right segmentation result.
Can you help check the segmentation model and code ?

@mingminzhen thank you for reporting this issue. We should cast the scaled_size to int, but I would do it with round. Anyway, I was using the test_vistas script with no problems to predict Vistas images. Are you running the same?

@mingminzhen when you download the samples you get in the same folder both the image and the ground-truth segmentation. Is it possible that the image you show here is actually the network applied to the ground-truth annotation?

I copy the images in the directory: /home/mingmin/data/mapillary/samples.
The groundtruth is not copied.

@mingminzhen this is the output I get:

It's weird. I didn't modify anything. I use cuda9.2 and pytorch 4.0.

@mingminzhen can you try with test_vistas_single_gpu.py?

@rotabulo I copy the image (samples/Asia/tlxGlVwxyGUdUBfkjy1UOQ.jpg) to another dir. And test it.

https://drive.google.com/open?id=1bpSF5Wpj9MTwC3k-n87_KXFQPZeyk5ZH   This is the image i put in a dir called test (only one image in this dir).

On my end this keeps working..

Is it possible that the pretrained model is different from which you use?
I check the code carefully. It seems no errors in the code.

@mingminzhen I tested with both pytorch v0.4.1 and v0.4.0 and I have cuda 9.2 too. It works in all cases.

@mingminzhen I am using the one we provide on the webpage and also checked that the md5 sum matches.

@rotabulo I try to change another machine and test it. It gets the same output. I test it on Anacoda 2, python3.6, pytorch v0.4.1 and cuda9.2. I feel so weird why it shows wrong output.

@mingminzhen what GPUs are you using?

@rotabulo i use one 1080 ti.

@mingminzhen I tried also with a 1070 and it worked. I don't think it's the GPU.

@rotabulo I use the cudnn<IP_ADDRESS>. I will check it again.

@rotabulo  Hi, I test the gpu code by following code:
import torch
import inplace_abn

input = torch.randn(100,100,150,200).cuda()
print('x:',input.size())

xsum1 = input.sum(dim=0).sum(dim=-1).sum(dim=-1)/(input.size(0)*input.size(2)*input.size(3))
print('sum1:',xsum1.size())
print('xsum:', xsum1.sum(),xsum1[0], xsum1[1])

xsum, xssum = inplace_abn.mean_var(input)
print('xsum:',xsum.sum(), xsum[0], xsum[1])

The result is as follows:
x: torch.Size([100, 100, 150, 200])
sum1: torch.Size([100])
xsum: tensor(-0.0041, device='cuda:0') tensor(-0.0000, device='cuda:0') tensor(-0.0007, device='cuda:0')
xsum: tensor(-284.8568, device='cuda:0') tensor(-0.0041, device='cuda:0') tensor(211.0134, device='cuda:0')

It seems there is error for the cuda code in mean_var  cuda implementation code.
Can you help check that?
In addition, the cpu code works well.
import torch
import inplace_abn

input = torch.randn(100,100,150,200)
print('x:',input.size())

xsum1 = input.sum(dim=0).sum(dim=-1).sum(dim=-1)/(input.size(0)*input.size(2)*input.size(3))
print('sum1:',xsum1.size())
print('xsum:', xsum1.sum(),xsum1[0], xsum1[1])

xsum, xssum = inplace_abn.mean_var(input)
print('xsum:',xsum.sum(), xsum[0], xsum[1])

The result is
x: torch.Size([100, 100, 150, 200])
sum1: torch.Size([100])
xsum: tensor(-0.0015) tensor(-0.0008) tensor(-0.0007)
xsum: tensor(-0.0015) tensor(-0.0008) tensor(-0.0007)

@rotabulo I try to use cpu to do test. it works well.

@mingminzhen I checked the test code you posted, and it works on our side, i.e. we experience no issue. I invite you to try with some other machine/gpu.

@rotabulo  I found the reason for that at last. It's because my driver is 390, but it's insufficient for cuda9.2 runtime. But there is no error shown in the code. So i suggest add some error check code for each kernel function.
  cudaError_t cudaError = cudaGetLastError();
  if(cudaError != cudaSuccess)
  {
    printf("  cudaGetLastError() returned %d: %s\n", cudaError, cudaGetErrorString(cudaError));
  }

This will be very useful!


bug: mo-doc is failing to emit doc comments for classes

(see HashMap and Random.finite documentation for examples).
Plus, when running make base, I get lots of (benign) errors about failing to extract documentation for test files)
@rvanasa can you take a look?

Labelling it P1 since it affects our public doc....
(found doing make base in motoko/doc)

Oops, wrong repo. Moved here https://github.com/dfinity/motoko/issues/3684


Graph evaluator caching

Graph evaluator caching component from former PR
Graph evaluator caching
The GraphNode struct now has an additional property, the AbstractArray cache, not initialised during object creation.
This enables the result of each node evaluation to be stored and retrieved without the need for a dictionary lookup.
A GraphNode can now also have the constant property set to true during evaluation if it is an operator node with constant operand nodes. In such a case the val property will also be set to the result of the single operation required. This behaviour won't occur in sufficiently optimised expressions, as a subexpression consisting of operators with constant operand nodes could be replaced with a single constant node. This doesn't meaningfully mutate the expression during the evaluation, as the constant property is otherwise meaningless for nodes with degree >0.
In terms of the evaluator itself, there are two implemented versions: one using Base.map, and one using LoopVectorization.vmapnt which is used when the LoopVectorization package is loaded. Some rough tests I did found that, on a graph with no shared subexpressions, the graph evaluator takes 1.4x as long as the corresponding (with or without LoopVectorization) tree evaluator. I might do some more work on reducing that, one method would be to quickly identify whether shared subexpressions exist in the expression and choose a corresponding evaluator. The relative performance of the graph evaluator operating on an expression with shared subexpressions is entirely dependent on the number and size of the shared subexpressions but has a theoretically unbounded performance improvement.

Can test locally by typing ]test into the REPL. Also for standardising the formatting you can use https://domluna.github.io/JuliaFormatter.jl/stable/#Quick-Start

Also see https://docs.julialang.org/en/v1/manual/performance-tips/#man-performance-abstract-container regarding the ::AbstractArray{T} for cache.


Xy calibration goodies2

test for laser zeroing verifies if the Si peaks is at the correct position
additional method for peak matching using scipy linear_assignment
CalibrationModel and XCalibrationComponent options to select the match_method
demo notebook

@georgievgeorgi  @nicocopez if you could go through the demo notebook and tell

what is not clear
what you want to change and wondering how , etc
then I will add explanations (or additional code)

Resolved the review comments
refactoring of the protocols/calibration into (semi)separate classes
optional plots in tests (disabled by default)
fixed bug in match_peaks_cluster in interpretation of clustering results
added options for match methods
added options for interpolators - we now have RBFInterpolator, PchipInterpolator, CubicSpline . Of these, only PchipInterpolator guarantees monotonic calibration curve, and when interpolating.  But Ne peaks don't cover the entire range, so we need (some) extrapolation. Hence added extrapolation option.
option what to do if the results of applying the calibration curve is nonmontonic - ignore, set nans, drop nans, raise error

@georgievgeorgi @kerberizer


Can you clarify the license of your code?

Thanks for the excellent code.
Can you clarify the license so that I can understand if it's OK to use?
Shai

Just saw the LICENSE file. duh!

You can do what you wish with it.
It would be nice to have the repo mentioned on the credits/thanks page though. :)


Disable auto-issue-linking in dnceng Azure DevOps workitems

This feature is incorrectly linking numbers found in GitHub mirror repositories to the unrelated Azure DevOps workitems. This then causes the workitem to be closed incorrectly.

I believe it's this feature. Resolve work items on commit.
Unfortunately, it is a per-repo setting.

yes, last year I did a bunch of repos manually, but I guess there are new repos 😞

For my understanding. What is the work here? To go into every repository in dnceng/internal and click the setting? or only for repositories owned by engineering services?

This mentions that it's enabled by default on a per-repo basis. https://learn.microsoft.com/en-us/azure/devops/repos/git/resolution-mentions?view=azure-devops#enable-commit-mention-work-item-resolution
I don't think this is the way to go. Moving this to triage for discussion.

I decided I don't want to champion this in triage. I went through the list of 145 repos with active mirroring enabled and made sure the setting was turned off.


Prevent deleting data when non-existing customer

Prevent deleting data when non-existing customer, eg. $customer->id = 0

All specific price not associated to a customer will be deleted
All cart rules not associated to a customer will be deleted
All anonymous customer thread and messages will be deleted

Bug reported from Addons

Hello
Thank for this PR.
As said these methods are public (also deleteDataFromPrestashop). These methods can be used to create a cron and anonymize old accounts (our use case).
An other case, any ajax call can send an erroneous id_customer.
Other use case on db overload and query partially broken and not secured.

@Matt75 in which case the id_customer = 1 ?

@sarahdib Problem is only when id_customer = 0 ;)
As said by @camlafit, on cron job or when url is manually changed.
For example ps-admin/index.php?controller=AdminAjaxPsgdpr&token=0fdd3c1562cd6e510b7f8f1bc9b35d56&ajax=true&action=DeleteCustomer&delete=customer&value=0 (Do not forget to change token by yours in this example)

@Matt75  yes but I can't reproduce a case with  id_customer = 0 in order to test. Do you now when a customer id is 0 ?

@sarahdib No need to have a customer id = 0 ... it never happens but if id_customer = 0 is given, gpdr module delete :

All specific price not associated to a customer will be deleted
All cart rules not associated to a customer will be deleted
All anonymous customer thread and messages will be deleted

@Matt75 Thanks :)

\o/


Update index.html

Pull request description
Update of contact person for the Drinking Water program.  Mark Jensen has retired and Deidre Beck is his replacement.

👍


Compress option not working in CLI - possible conflict with other options

Using the -c flag within the CLI does not actually compress a given input file under v1.8.3.
E.g. let's consider the compression example on the cssnano homepage:
input.css
h1::before, h1:before {
    margin: 10px 20px 10px 20px;
    color: #ff0000;
    -webkit-border-radius: 16px;
    border-radius: 16px;
    font-weight: normal;
    font-weight: normal;
}
/* invalid placement */
@charset "utf-8";

We'd expect cssnext -c input.css to give us this:
@charset "utf-8";h1:before{margin:10px 20px;color:red;border-radius:1pc;font-weight:400}

However we just get back the original input.css (non-compressed).
After some digging I found that adding the following flags in addition to -c seems to help, so I imagine there's a conflict with how cssnano is being called:
Hacky workaround - disable everything but compression:
> cssnext -c input.css --no-custom-properties --no-calc --no-custom-media --no-media-queries-range --no-custom-selectors --no-color-rebeccapurple --no-color-hwb --no-color-gray --no-color-hex-alpha --no-color-function --no-font-variant --no-filter --no-rem --no-pseudo-elements --no-pseudo-class-matches --no-pseudo-class-not --no-pseudo-class-any-link --no-color-rgba --no-autoprefixer
The above delivers compressed output, but it obviously isn't ideal given we're effectively disabling all the other features within cssnext.
Has anyone run into the same issue?

Might be related to this: https://github.com/cssnext/cssnext/issues/194

Not sure how it's related since nobody raised this issue until now.
@dhruvbhatia do you get the same issue without the charset rule?

Thanks for the quick responses!
Yes I get the same issue, even without the charset rule. I.e. Even basic input isn't compressed.
If you can't reproduce then perhaps there's something wrong at my end? Are you able to produce the expected output in the example above?

On 16 Aug 2015, at 1:57 am, Maxime Thirouin<EMAIL_ADDRESS>wrote:
Not sure how it's related since nobody raised this issue until now.
@dhruvbhatia do you get the same issue without the charset rule?
—
Reply to this email directly or view it on GitHub.

I tested historical versions in clean environments and discovered that the unexpected behaviour only started in v1.8.0 (compression worked fine prior).
This coincides with the upgrade to cssnano v2.x - #166 as per the v1.8.0 release notes.
v1.7.1 works fine so using that for the time being, but I'll keep digging and see if I can isolate further..

I confirm that starting from v1.8.0, compression does not work. I use cssnext as a PostCSS plugin.

@ben-eb seems correct to me based on my testing. When I changed postcssPlugin names to be unique, everything started working again.

Tests were passing when 1.8.3 were released, and now the same tests are broken with a fresh install.
So this is related to a breaking change introduced by a recent cssnano "minor" update (that should have been major ?).
I will fix that by addressing #194

You can work around this by passing this config to cssnext:
{
  compress: {filterPlugins: false}
}

Fix released as 1.8.4


Swapping images and resetting Reel within a page

Hi, I'm working with your plugin (thanks, BTW, for making such a cool project available on Github) and am hoping you can share some insight on how you were able to execute the image switching and plugin resetting you achieved in your example page (http://test.vostrel.cz/jquery.reel/test/sampler.html).
From what I can tell you're storing the values in the page but I can't quite find where you're making the call to do the swap and reset the plugin. Could you tell us a bit about how you achieved this within the page?
Thanks in advance,
william.mincy

Basically there is a
 in the body, where the visible part of
s is used for "navigation" segment at the top (the links). Besides it there are some invisible segments including three <xmp> tags representing HTML, CSS and JS example code you then get to see on the page under the example image.
When the example is rendered all three segments get evaluated and used to prepare and construct a call to .reel() function. Basically it takes #the_one #image image tag, sets its src to URL from the HTML segment and applies dimensions from the CSS segment. Then options object from JS segment is used as a single parameter to the .reel() call.
Please note, that the URL runs the development version of the code, so some bits may differ from the 1.1.3. Especially the need to .trigger("teardown") event first in order to allow for re-use of the stage.
The plan is to allow easy re-use by mere re-calling .reel() on the node again without any other constrains.

You may also be interested in the revised examples/ folder in the 1.2alpha distribution, where my aim was to extract the understandable essence of what it takes to get reeling, which to my mind is much more clean and concise compared to the previous.


Replace empty string with null

For integers, rewrite empty string with null values and strip whitespace. For strings these type of actions are performed server-side, but as the long-term goal is to move functionality from server to client-side I chose to put this code there.
Fixes #1085

LGTM


Fix description: Add support for sanity tests for hv_balloon driver.

	 1. check if the driver is loaded correctly
	 2. check if the driver has registered correctly

Signed-off-by: Shradha Gupta<EMAIL_ADDRESS>

@shradhagupta5 can you check verify_initrd_modules, does it meet your requirement, if not, please add logic in this test case, thanks!
lisa\microsoft\testsuites\core\hv_module.py

@shradhagupta5 can you check verify_initrd_modules, does it meet your requirement, if not, please add logic in this test case, thanks! lisa\microsoft\testsuites\core\hv_module.py

Thanks Lili, would go through them and make necessary changes

@shradhagupta5 hi, is this PR still valid?

Hi Lili, Yes. I fill fix all the conflicts and revive the request.


UserTimeline pagination fix, more null exception handling, general enchancements.

Unit-test
Created (and now passed) DuplicateAuthorObjectInDatabase_ListOfCheepsIsNotEmpty (name might be inaccurate). Was very useful during pagination-fix.
UserTimeline pagination fix.
Problem lied in thinking that retrieving an Author obj. such that: var author = GetAuthorByName("<Name>") would also automatically include the author's list of cheeps, but this lead to an empty list (and therefor a count of 0 cheeps -> max 1 totalPages).
These cheeps lie in a different table (Authors vs. Cheeps), so EntityFrameworkCore methods must be used to ensure that the list of cheeps are added in author.Cheeps.

Also ensured explicit connection between an author and their cheeps in Cheep.cs using ForeignKey data annotation, and in ChirpDBContext.cs using OnModelCreating. Meaning that a new migration has been added and applied: AuthorCheepsFix.

ChirpDBContext & In-Memory Testing
As per Patrick's #109 , I've simplified ChirpDBContext such that it doesn't handle its own setup, which in turn simplified how we could use / establish connection to an in-memory database in InMemoryWebApplicationFactory.
AuthorRepository enchancements
Attempted to adhere to a more consistent method-naming-scheme, so i.e. GetAuthorsCheeps -> GetCheepsByAuthor.
Method GetAuthorWithCheeps(string authorName) makes it easier to test access to author-specific cheeps.
Attempted to simplify and hopefully better the performance of GetCheepsByAuthor.
IntegrationsTest
Under the test of whether we can access our public timeline, /public has been changed / as the latter is our root / homepage, and where we populate a shared, public timeline of all cheeps. Else we'd be checking if a user called public had any cheeps on their personal usertimeline. As such, public's Timeline was also changed to Public Timeline.

Oh, and I received 0 warnings and 0 errors. All tests passed.
Resolves #105 . Resolves #106 .

Lemme look at smth first -- no merge!!


[Feature] Customize confirm prompt

It would be nice to be able to customize the text in the confirm prompt, possible even the options but I would be happy with changing the "Do you want to quit?" text in the configuration.

I agree. I modified it so that it can be changed.
https://github.com/yutkat/confirm-quit.nvim/commit/f15f6d728d385a3d2efa22098e9a45b8a2b20144
require("confirm-quit").setup({quit_message = "aaa"})


CoreML conversion fails: "ValueError: Only channel and sequence concatenation are supported."

Hi,
First of all, thanks for going through all the hassle of converting the model to keras so I don't have to :)
I tried to do the keras to coreml conversion, but it fails with the following error:
55 : activation_14, <keras.layers.core.Activation object at 0x12fc5a350>
56 : concatenate_1, <keras.layers.merge.Concatenate object at 0x12fbee610>
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Library/Python/2.7/site-packages/coremltools/converters/keras/_keras_converter.py", line 504, in convert
    predicted_probabilities_output = predicted_probabilities_output)
  File "/Library/Python/2.7/site-packages/coremltools/converters/keras/_keras2_converter.py", line 278, in _convert
    converter_func(builder, layer, input_names, output_names, keras_layer)
  File "/Library/Python/2.7/site-packages/coremltools/converters/keras/_layers2.py", line 511, in convert_merge
    mode = _get_elementwise_name_from_keras_layer(keras_layer)
  File "/Library/Python/2.7/site-packages/coremltools/converters/keras/_layers2.py", line 79, in _get_elementwise_name_from_keras_layer
    raise ValueError('Only channel and sequence concatenation are supported.')
ValueError: Only channel and sequence concatenation are supported.

Do you have an idea what I could do to fix that? Or would it be possible to send me the converted coreml model directly?
After looking through the notebook again, it seems like you changed the keras / coremltools packge. Can you share those changes / include them in the source?
Additionally, how well does the core ml model work? Did it generate the correct embeddings for the lfw images?

Hi @galli-leo,
Since Coremltools only supports channel-last model, and the model I uploaded is channel-first, I just pushed the channel-first model, you can try it again with channel-last one.
But there is another problem with Coremltools, it doesn't support Lamda Layer (form Keras), so I altered Coremltools source code a little bit to make it work. Per your request, I have included the CoreML Model.
CoreML model works very well on iOS devices, it generates exactly the same embeddings as Torch / Keras version, and it runs about 30FPS on iOS devices.
Let me know if you have any other questions.
Victor

@iwantooxxoox @galli-leo Is the coreML model available in the repo? Let me know the location otherwise, kindly include it. Thanks.

@iwantooxxoox That sounds very cool (cannot seem to find the model though :/). Could you also include the changes you made to the coreml source code? I want to try and convert a tenserflow version I found, since it seems to have better accuracy than the openface one. Thus your changes may be helpful :)
Anyways, I also tried my luck converting the torch version directly (see: https://github.com/prisma-ai/torch2coreml/pull/9). Unfortunately the numbers are a bit off when comparing torch vs coreml. Just out of curiosity, do you happen to know why that might be?

@pkwete, @galli-leo,
The CoreML model has been included in the model folder


User Reporting

User Reporting includes a user with all the possible attributes, filter and order possibilities. (Profile, Email, Authentication Methods, Last Login, etc.)

[ ] User Reporting Console Design
[ ]  User Reporting Projection
[ ] User Reporting CSV Export
[ ] User Reporting API
[ ] User Reporting Console Integration

tracked in #2301 this will be closed


Fix/permission docs

Description of the Change
Added description for can_get_peers permission + updated permission examples to work with the updated iroha python library.
Benefits
Wee better docs
Possible Drawbacks
No docs for removing peers yet. Will add as soon as the command is merged

This pr requires CI env modification. The merge is postponed


I cant't chage a memo to public, the error is "disable public memos system setting is enabled"

Describe the bug
I can change the memos from private to public before, but I can't remember the version that I used. Now I have upgrade to v0.20.1, but I can't change any memo to public anymore. It seems no related setting in the system. Has anyone else encountered the same problem?
Steps to reproduce

edit or create a memo;
change the visible to 'public'
you will see the alert box at the top right corner "disable public memos system setting is enabled"

The version of Memos you're using
0.20.1
Screenshots or additional context

Hi @hankluck, could you check the setting item about Disable public memos

Hi @michaelien , thank you for your help!
It works when I disable the button "Disable public memos".
Maybe there was a problem of cache before, I am not sure, because I used to switch this button before it was also invalid. I used the simplified Chinese language at that time.But changed to English, and then disable this function will play a role.
My problem has been solved, and this Issue can be closed.


Only ADA funds not showing correctly

Imported a wallet from Daedalus to Nami. All assets and coins are showing correctly. Only ADA is not. (22 in stead of 690). When connecting Nami to Taptools it shows 690 ADA, When connecting to muesliSwap it shows 22 so I cannot swap what I want to. Wallet is staked to TVV1N. What am I missing?

@ElrohirMelwasul Talk to Cardano Support on the Live support chat so that your issue can be looked into.


runtime/pprof/internal/protopprof: TestCPUProfileParse sleeps for 30s

This means that these tests take 30s even in -short mode. Seems undesirable.

/cc @matloob

Would this be fixed by skipping these tests in -short mode?

It's just the one test, TestCPUProfileParse. You could skip it, but can it be shortened instead somehow?

I've been on a mission to save a second here and a second there all across the standard library. A 30 second test defeats that whole mission. I don't know what this test is trying to do, but surely there's a better way.

The test needs to run for a bit of time to collect profile samples, but it certainly doesn't need to run for 30s. 30s was just the length of time the profiles are collected for in the tests on the original Google version of this code.

It may be helpful to look at tests like runtime/testdata/testprogcgo/pprof.go which test the profiler but limit how long they run.

This test has been deleted.

This test has been deleted.

There is now a new TestCPUProfileParse in runtime/pprof_test.go that sleeps for 5 seconds always: https://github.com/golang/go/blob/9a5bddd7ed57596a259f3896dd31ea30e331027d/src/runtime/pprof/pprof_test.go#L665. That is much better than 30 seconds, but all the other tests in that package seem to try and complete in 200ms in short mode. Maybe this new test should do the same?

That test will be deleted in https://go-review.googlesource.com/33136. Thanks.


Mapping $id or $ref into properties.

I've got some inner objects that are using $id as property names that need to be mapped to an object.  Normally I would modify the json serializer settings to ignore references, but this seems to not be exposed to us when using the Snowflake client?
Short version, given an object like so:
         "emid": {             "$id": "63484e6eab783af4c4072539" }
With a poco:
	public class Emid 	{ 		[JsonProperty("$id")] 		public string id { get; set; } 	}
In the final deserialized object coming back from QueryAsync, that ID will alway be null, is there a setting I'm overlooking?

Hello! You can override json serializer options - you can pass them in SnowflakeClient constructor or you can set it directly with this method: SnowflakeDataMapper.SetJsonMapperOptions(jsonSerializerOptions).


Add fork to query params

See this issue
Added the option to provide a fork query parameter to the following pages:

Precompiles
Playground
Reference Table

Also added the fork query parameter to the permalink copy feature of Playground.

Currently the PR enables you to use query params but it doesn't actually use it when you click the "link" button on an opcode or just when you browse the latest fork, should that change?

Currently the PR enables you to use query params but it doesn't actually use it when you click the "link" button on an opcode or just when you browse the latest fork, should that change?

I wasn't sure if this is wanted behavior or not, but I guess it might be useful for sharing opcodes/precompiles since they don’t have a “copy permalink” button like the playground.
Will add that.

Yeah, I could see reasons to go either way on it.  so totally fine keeping it as is/simpler

Yeah, I could see reasons to go either way on it. so totally fine keeping it as is/simpler

Already committed the change :)
I see the value in adding the fork to the URL to make sharing URLs between users more “transparent”.


Use encrypted token for TravisCI

The SAUCE_ACCESS_KEY should be encrypted and also the Slack token.
https://docs.travis-ci.com/user/sauce-connect/
https://docs.travis-ci.com/user/notifications/#Configuring-slack-notifications
It might be a good idea to create new ones, encrypt and add them and revoke access for the old ones.
https://github.com/chaijs/guidelines/blob/master/secure-tokens.md

Hi @DanielRuf,
Thanks for underlining this :wink:
It should be OK in the next release, as all CI stuff is under review.
Closing the issue.


[bug] KEYCODE_MEDIA_NEXT does not trigger with Capability.NEXT not set as notification capability on Android

Describe the Bug
The bug is described here: https://github.com/podverse/podverse-rn/issues/1758
for ios, RNTP behaves as normal. for android Event.RemoteSkipNext will simply not fire (via bluetooth, as notification button is hidden) when Capability.next is not set as a notification capability, because MediaSessionConnector.QueueNavigator does not enable the next/previous actions then. This is fine for most apps but not for podverse, where the next/previous buttons should be replaced by forward/backward buttons.
Steps To Reproduce
Take out Capability.Next in notifications and try next/previous with a bluetooth headset on android
Code To Reproduce
like https://github.com/lovegaoshi/KotlinAudio/commit/43a23de5ad884471968547d762b0d0d62d12f40d#diff-aac2898870a901482661b6a0bfd73dbbab38fc17fc1f8591a6fce7ab31f9c76a
Replicable on Example App?
yes.
Environment Info:
not relevant
How I can Help
I override onMediaButtonEvent for mediaSessionCompat.Callback (I think I wrote this part for the android auto PR?) to specifically handle skip next and previous events for podverse. unsure what the best practice is though.
https://github.com/lovegaoshi/KotlinAudio/commit/43a23de5ad884471968547d762b0d0d62d12f40d#diff-9c7fc38508b88eca04e110589f9f3c473b526e10a0451c87796dc53f9a9f20f8

It's a bit more complicated; skip next and previous, if enabled, will
always show buttons. This happens in both exoplayer2 and media3, and media3
has an option to disable certain action buttons, though poorly documented
I remember seeing commits of the capability revamp but then reverted too
On Fri, Oct 20, 2023, 11:37 PM David Chavez @.***>
wrote:

Right, at the moment the capabilities are both used to determine what the
notification shows /and/ media session actions. I have a PR in the works
that reworks and separates those two: #2076
https://github.com/doublesymmetry/react-native-track-player/pull/2076
—
Reply to this email directly, view it on GitHub
https://github.com/doublesymmetry/react-native-track-player/issues/2180#issuecomment-1773693990,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZMOVVXXQTOL7CB5QC3YF7DYANUTZAVCNFSM6AAAAAA6J5APGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTGY4TGOJZGA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


[TC-532] add tenancy check to ds safe-update endpoint

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/142/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/103/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-whatchanged/3/


cmd/internal/obj/ppc64: T/S bit is being flipped when using extended mnemonics in VSX instructions

What version of Go are you using (go version)?

go1.12

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
linux/ppc64le
linux/ppc64

What did you do?
CL160429 uncovered a bug in the assembler, where bit 31 of XX1-form instructions is flipped when using extended mnemonics (FP/VMX registers as arguments, instead of VSX).
What did you expect to see?
F0-F31 must map into VS0-VS13 and V0-V31 must map into VS32-VS63.
What did you see instead?
The problem makes FP register arguments become VMX registers (or, in VSX terms, VS0-VS31 into VS32-VS63). This doesn't cause an illegal instruction, but may cause register clobbering.

I have a fix and will submit as soon as the tree reopens.

Ups, sorry, tagged the wrong issue number. Move along...


Added listening presence support

SONG_NAME = <Activity>.details
SONG_ARTIST = <Activity>.state
Instead of "Listening Spotify" it displays "Listening to $SONG_NAME by $SONG_ARTIST"

Thanks a lot!


Remove https://www.w3.org/TR/selectors-api/

See https://github.com/tidoust/reffy-reports/issues/3#issuecomment-386557144

Thanks, @foolip.
FWIW, note I will actually keep the Selectors API spec in the W3C crawl and report: one of the goals of Reffy is to highlight inconsistencies that we have, typically here the fact that the spec needs to be obsoleted.


[6.0] Use table.* instead of * as default for select queries

When fetching an Eloquent model using a join query, it automatically uses select * which ends up using a joined models id as the main id, and causes unexpected issues when saving.
For example, if i used the following query to fetch a user: (I know this query doesn't make much sense, its just an example).
$user = User::query()
    ->join('account', 'account.id', '=', 'user.account_id')
    ->where('account.id', '=', 7)
    ->first();

I would now like to change a column on the user model:
$user->name = 'Test';
$user->save();

Instead of updating the user i expected (e.g. 20), it actually updates user 7 (which is actually the account.id).
I think this behaviour can easily cause unexpected bugs, and could be prevented with a safer default.
--
This PR changes ->get() and ->cursor() to default to using select "table".* unless otherwise specified using the ->select() method.
Note: This will cause a breaking change for users that don't specify a select and rely on columns that are automatically returned by select *.

No plans to change this behavior so close to release.

There has been a release now, maybe this could be reconsidered?

Another upvote for accepting this pull request to fix the issue where Eloquent objects are silently hydrated with wrong data; just ran into this today while implementing a custom Filter in Laravel Nova. Due to the issue fixed by this pull request, adding a simple join to the query builder in the Nova Filter's apply method directly and silently makes Nova present corrupted data objects to the end user.

Another upvote here. The fact this doesn't even merit a warning in the docs is terrible!

Another upvote. Had the same issue.


Zoom restriction fails with table and tables definition on layer.

Having both a table and tables defined causes the lookup method to fail.
"table": "dev.retail_points",
"tables": {
  "11": null,
  "12": "dev.retail_points"
}

Should be resolved in this PR. https://github.com/GEOLYTIX/xyz/pull/565


How can do a internal request that like subquery in nginx or asynchttpclient in tornado?

Maybe requests or libcurl or urllib?

Hi @codezq, you can use the requests library along with gevent, or if you are hosting with twisted.web.wsgi, you can use the crochet library to perform requests via twisted. You might also try aiohttp.wsgi if you are on py3k.

Thanks a lot! Now I use the requests library.


Backing image enhancement

longhorn/longhorn#2530

Adding error message reporting and retry backoff. Convert to draft for now.

These PR's that add new controllers + crds are always pretty large.
We should consider splitting them into  2 PR's.
PR1: With CRD + Boilerplate Code (datastore, etc) and boilerplate Controller (enqueues, constructor) no sync logic yet.
PR2: actual implementation of the sync logic + monitoring etc.
This way PR1 can go early into the code base with a quick check review, since it's more or less just boilerplate.
Then PR2 can be reviewed in depth or via in person code walk through, since this is where actual code will run and do stuff.
What do you guys think @innobead @shuo-wu ?

These PR's that add new controllers + crds are always pretty large.
We should consider splitting them into 2 PR's.
PR1: With CRD + Boilerplate Code (datastore, etc) and boilerplate Controller (enqueues, constructor) no sync logic yet.
PR2: actual implementation of the sync logic + monitoring etc.

It's a nice suggestion. But for this PR, splitting it now will lead to tons of extra modifications and commits adjustments, which will consume lots of time. Besides, there are 2 other PRs highly rely on this PR. I think we can follow this pattern next time.
BTW, the controller modifications/commits are in the latter part of this PR. We can focus on these commits then.

@shuo-wu Agree I wasn't suggesting to do this to this PR, makes sense not to touch this one.

@innobead /hold merge @shuo-wu will merge this after he had a chance to do a full test.

May need to update vendor longhorn/backing-image-manager before merge.

Had a review session with @shuo-wu LGTM, we identified some smaller changes.
And scheduled a PR review session for Monday, to follow up and get this merged.


Conflict between default kubernetes rollout

I've tried wave for the first time and looks like there is a possible conflict between default kubernetes rollout when you modify configmap/secret reference. So basically k8s starts its rollout and then wave makes a change and starts another rollout.
I don't think it is critical because I can't come up with a case where it is, but maybe this can cause some unforeseen race conditions. And as far as I understand the logic of wave there is no way around such situations because it needs to update the annotation in any case?
Also, want to add that in my particular case I need to monitor only one secret because configmap updates are managed by another process, so the ideal way is to configure wave to watch only specified resource (secret) and unfortunately there is no such a feature...

Are you referring to a situation where both a ConfigMap are a Deployment are updated roughly at the same time?
In those situations it is possible that, depending on ordering, a new Pod will start, still using the old configuration, then immediately gets replaced by one using the new one.
Without Wave this could still be a problem, causing some Pods to run with different configuration from others. Wave merely ensures in these cases that the state everything settles on eventually is the desired one.
Everything in Kubernetes is eventually consistent and so such situations should be expected. Applications should make use of PodDisruptionBudgets and empoy techniques that allow code to function with both old and new versions of a configuration set in order to avoid being negatively impacted.

@gargath I'm referring to the situation when configmap reference changes in the deployment. For example you have a deployment which references configmap1 and you change it to configmap2, k8s starts its rollout which is then superseded by wave at the same time. As you mentioned I think this is not a problem and expected by k8s, but just a thought. Maybe ideally it should be handled in admission webhook which injects new checksum on edit to prevent double rollout, but probably it's not worth it.

We should add an optional mutating webhook to implement this. It would prevent restarts in all cases where configmaps/secrets exist prior to creation of the deployment. In my opinion that would be worth implementing.
Unfortunately, this will not solve the case for most helm charts where you install configmap + deployment at the same time. Especially when you add other controller (such as cert-manager) into the mix. In those cases the pod might actually be pending until those secrets exist and start with the first version but wave will restart them right away.

Should be fixed by #155

@toelke this one can be closed. I guess I should have commented in the PR instead of here.


global: fixes and tx configuration

@tiborsimko should I remove the I18N support (last two commits)?

@alizeepace yes, please, including babel.ini, .tx, and catalog configuration from setup.py.


[SPARK-18960][SQL][SS] Avoid double reading file which is being copied.

What changes were proposed in this pull request?
In HDFS, when we copy a file into target directory, there will a temporary ._COPY_ file for a period of time. The duration depends on file size. If we do not skip this file, we will may read the same data for two times.
How was this patch tested?
update unit test

cc @zsxwing

Test build #70456 has started for PR 16370 at commit 1d248c3.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70456/
Test FAILed.

@AmplabJenkins retest it please

retest this please.

Test build #70473 has started for PR 16370 at commit 1d248c3.

Test build #70473 has finished for PR 16370 at commit 1d248c3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70473/
Test FAILed.

Merged build finished. Test FAILed.

unrelated errors, retest this please.

Test build #70476 has started for PR 16370 at commit 1d248c3.

Test build #70476 has finished for PR 16370 at commit 1d248c3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70476/
Test PASSed.

Merged build finished. Test PASSed.

LGTM.

A similar PR got refused as this is not the correct way to stream files: https://github.com/apache/spark/pull/3203
You should always move files instead of copying them.

@zsxwing Thanks for your reminder!!
In some ways, we really can evade this issue, just like not use -cp. But this is an user-side behaviour, we can not ensure every users know and use correct ways to move data. It may confuse users if they do not know this issue. Besides, current changes is so tiny that do not have any harm to codes. So, IMHO, it is OK to add the check for protection. What do you think？

I don't see a harm in ignoring these files, other comments notwithstanding. In fact, I think it's mandatory. The HDFS copy mechanism basically is copy (to a .COPYING file), then move (rename). This is exactly what anyone copying from a different FS would have to do anyway manually as you can't rename a file that isn't already on the FS, and don't necessarily have ability to write anywhere else.

@zsxwing Is there any farther feedback？

@srowen It looks like @zsxwing has no time to feed back.

Merged to master

Sorry for the delay. LGTM.


Chapter 4

Code example for Ch. 4 Embedding Models

requests.exceptions.SSLError: HTTPSConnectionPool(host='openaipublic.blob.core.windows.net', port=443): Max retries exceeded with url: /encodings/cl100k_base.tiktoken (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: self signed certificate in certificate chain (_ssl.c:1002)')))


[Add] UI GENERATORS RESOURCE

Site URL
https://chakra-ui.com/
Description of the above provided site
Chakra UI is a simple, modular and accessible component library that gives you the building blocks you need to build your React applications.
Category
Frontend
Sub Category
ui generators
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[ ] I am a GSSoC'23 contributor

assign me
I'm gssoc contributor

assign me I'm gssoc contributor

do you still want to work on this issue?

No, you can assign it to someone

@Ameya02 @Bhanuteja01 who is willing to work?

@rupali-codes
I'm willing to work on this
Assign me

done, please finish it as soon as possible


Update component status table to use nimble icons

Pull Request
🤨 Rationale
Update the component status table to use the nimble-table-column-icon rather than emoji icons.
👩💻 Implementation

Update component status table to use icon column
Update markdown table that is the legend for the icons
Add an enum for framework-specific status to avoid hard-coding strings

🧪 Testing
Verified that the storybook docs looked correct
✅ Checklist

[ ] I have updated the project documentation to reflect my changes or determined no changes are needed.

@m-akinc, can you provide a buddy review for this PR?


Documentation unclear about Preview

[Enter feedback here]
The page makes clear that if one has "an earlier preview version of SSMS 19 installed" then one must uninstall it. It then goes on to note that "You can see if you have a preview version by going to the Help > About", but fails to note what hint on that window will indicate a preview version (versus a non-preview version which presumably does not need to be uninstalled).

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

ID: 45daf2f0-bb22-338f-062b-a9fabcb71a92
Version Independent ID: b6e909fe-15d9-c1cf-c018-d4f6f36ff31f
Content: Download SQL Server Management Studio (SSMS) - SQL Server Management Studio (SSMS)
Content Source: docs/ssms/download-sql-server-management-studio-ssms.md
Service: sql
Sub-service: ssms
GitHub Login: @erinstellato-ms
Microsoft Alias: erinstellato

@BillMedland The version of SSMS 19 would include the text "Preview", for example, SSMS 19 Preview 3.

@BillMedland -- Bill, thank you for your feedback.
@erinstellato-ms -- Erin, thank you for clarifying.
I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


"client_class" and "callback_url" - not defined

Hi all,
I am trying to implement social auth using "django-rest-auth".  I followed the instruction, however once I sent access_token and code to "/rest-auth/facebook/" endpoint a I get: 'Define callback_url in view' error. I have checked the code and see that both "client_class" and "callback_url" - are required to have in view.
Is there anywhere I can read about it?
Many thanks,
Andrew

Also, I am authenticating on front-end using FB Login Dialog. However once it's done I get only accessToken and signedRequest info. How can I get the code? Or signedRequest is basically the code?

Ok, my fault! I was sending accessToken instead of access_token. Everything is working now!
Thanks for such a great app!


Make build fails at $(GOCMD) mod download

LocalAI version:
1.17.1
Environment, CPU architecture, OS, and Version:
MINGW64_NT-10.0-19044 DESKTOP-HSTM119 3.3.6-bec3d608-341.x86_64 2023-02-22 08:29 UTC x86_64 Msys
Describe the bug
Make build fails at $(GOCMD) mod download (go-rwkv)
This is happening for go-rwkv

go.mod file for go-rwkv

main go.mod file (after replace: is run)

To Reproduce
Just run make build
Expected behavior

Logs

Additional context
I would like to run without docker.
Downloaded MINGW64 & Go, not sure if there is any other dependencies
Working folder is C:\Projects\LocalAI

Hi, @intermag0. I believe you are trying to build the project on Windows without using container. The Makefile contains many of shell scripts may do not support by windows platform. I suggest you build the it in container environment. And here is our document for supporting the windows https://localai.io/basics/build/#windows-compatibility


Path-completion gets stuck in Mac OS X

I have installed zsh with Oh-my-zsh yesterday. Every time there is a longer path (say with 3 or 4 directories), the path completion gets stuck. It shows up after about 8-10 seconds, though.
I am using iTerm2 with robbyrussell theme.

I have found that if I disable compinit, then the problem goes (along with the feature to cycle through completions). Any suggestions?


[RoleDefinition Update] Cannot update existing role definitions via cli

Environment summary
Install Method: How did you install the CLI? (e.g. pip, interactive script, apt-get, Docker, MSI, nightly)
Answer here: pip
CLI Version: What version of the CLI and modules are installed? (Available with az --version)
Answer here: 2.0.2
OS Version: What OS and version are you using?
Answer here: Windows 10
Shell Type: What shell are you using? (e.g. bash, cmd.exe, PowerShell)
Answer here: bash
Description
We want to use azure-cli update existing custom role definition. The following command can create custom roles:
az role definition create --role-definition <definition_file>.json

When we call update command, the debug log shows a few issues:
# here's what we executed, $payload is a json
az role definition update --role-definition <definition_file>.json --name <role-definition-uuid-to-update> --scope '/subscriptions/<subscription-id>' --debug

The debug log shows:
Command arguments ['role', 'definition', 'update', '--role-definition', '{REMOVE SENSITIVE DATA}', '--name', 'xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx', '--scope', '/subscriptions/xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx']
Current active cloud 'AzureCloud'
{'active_directory': 'https://login.microsoftonline.com',
 'active_directory_graph_resource_id': 'https://graph.windows.net/',
 'active_directory_resource_id': 'https://management.core.windows.net/',
 'batch_resource_id': 'https://batch.core.windows.net/',
 'gallery': 'https://gallery.azure.com/',
 'management': 'https://management.core.windows.net/',
 'resource_manager': 'https://management.azure.com/',
 'sql_management': 'https://management.core.windows.net:8443/'}
{'azure_datalake_analytics_catalog_and_job_endpoint': 'azuredatalakeanalytics.net',
 'azure_datalake_store_file_system_endpoint': 'azuredatalakestore.net',
 'keyvault_dns': '.vault.azure.net',
 'sql_server_hostname': '.database.windows.net',
 'storage_endpoint': 'core.windows.net'}
Registered application event handler 'CommandTableParams.Loaded' at <function add_id_parameters at 0x7ffebc2f59b0>
Registered application event handler 'CommandTable.Loaded' at <function add_id_parameters at 0x7ffebc2f59b0>
Successfully loaded command table from module 'role'.
Application event 'CommandTable.Loaded' with event data {'command_table': OrderedDict([('ad group show', <azure.cli.core.commands.CliCommand object at 0x7ffebc2718d0>),
 ('ad sp reset-credentials', <azure.cli.core.commands.CliCommand object at 0x7ffebc271810>), ('ad sp delete', <azure.cli.core.commands.CliCommand object at 0x7ffebc2716
10>), ('ad user delete', <azure.cli.core.commands.CliCommand object at 0x7ffebc271850>), ('role definition delete', <azure.cli.core.commands.CliCommand object at 0x7ffe
bc264ed0>), ('ad group list', <azure.cli.core. [...]
Application event 'CommandParser.Loaded' with event data {'parser': AzCliCommandParser(prog='az', usage=None, description=None, version=None, formatter_class=<class 'ar
gparse.HelpFormatter'>, conflict_handler='error', add_help=True)}
Application event 'CommandTableParams.Loaded' with event data {'command_table': OrderedDict([('ad group show', <azure.cli.core.commands.CliCommand object at 0x7ffebc271
8d0>), ('ad sp reset-credentials', <azure.cli.core.commands.CliCommand object at 0x7ffebc271810>), ('ad sp delete', <azure.cli.core.commands.CliCommand object at 0x7ffe
bc271610>), ('ad user delete', <azure.cli.core.commands.CliCommand object at 0x7ffebc271850>), ('role definition delete', <azure.cli.core.commands.CliCommand object at
0x7ffebc264ed0>), ('ad group list', <azure.cli.core. [...]
Application event 'CommandParser.Parsed' with event data {'args': Namespace(_command_package='role', _jmespath_query=None, _log_verbosity_debug=False, _log_verbosity_ve
rbose=False, _output_format='json', _parser=AzCliCommandParser(prog='az role definition update', usage=None, description=None, version=None, formatter_class=<class 'arg
parse.HelpFormatter'>, conflict_handler='error', add_help=True), _validators=[], command='role definition update', func=<function handler at 0x7ffebc26eb90>, properties
_to_add=[], properties_to_remove=[], properties [...]
Getting management service client client_type=AuthorizationManagementClient
msrest.pipeline : Adding 'log_request' callback before event: 'request'
msrest.pipeline : Callback to overwrite original call: False
msrest.pipeline : Adding 'log_response' callback after event: 'response'
msrest.pipeline : Callback to overwrite original call: False
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - Authority:Performing instance discovery: https://login.microsoftonline.com/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - Authority:Performing static instance discovery
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - Authority:Authority validated via static instance discovery
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - TokenRequest:Getting token with client credentials.
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - TokenRequest:No user_id passed for cache query
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:finding with query: {"_clientId": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"}
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:Looking for potential cache entries:
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:{"_clientId": "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"}
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:Found 0 potential entries.
requests.packages.urllib3.connectionpool : Starting new HTTPS connection (1): login.microsoftonline.com
requests.packages.urllib3.connectionpool : https://login.microsoftonline.com:443 "POST /xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/oauth2/token HTTP/1.1" 200 1335
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:Get Token Server returned this correlation_id: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:Adding entry AccessTokenId: SENSITIVE DATA
adal-python : xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx - OAuth2Client:Added entry is MRRT
msrest.pipeline : Configuring request: timeout=100, verify=True, cert=None
msrest.pipeline : Configuring redirects: allow=True, max=30
msrest.pipeline : Configuring proxies: ''
msrest.pipeline : Evaluate proxies against ENV settings: True
msrest.pipeline : Configuring retry: max_retries=3, backoff_factor=0.8, max_backoff=90
msrest.http_logger : Request URL: 'https://management.azure.com/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/providers/Microsoft.Authorization/roleDefinitions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx?api-version=2015-07-01'
msrest.http_logger : Request method: 'GET'
msrest.http_logger : Request headers:
msrest.http_logger :     'Connection': 'keep-alive'
msrest.http_logger :     'Accept-Encoding': 'gzip, deflate'
msrest.http_logger :     'Accept': 'application/json'
msrest.http_logger :     'User-Agent': 'python/2.7.6 (Linux-3.4.0+-x86_64-with-Ubuntu-14.04-trusty) requests/2.13.0 msrest/0.4.6 msrest_azure/0.4.7 authorizationmanagem
entclient/0.30.0rc6 Azure-SDK-For-Python AZURECLI/2.0.2'
msrest.http_logger :     'Authorization': 'Bearer SENSITIVE DATA'
msrest.http_logger :     'x-ms-client-request-id': 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
msrest.http_logger :     'CommandName': 'role definition update'
msrest.http_logger :     'Content-Type': 'application/json; charset=utf-8'
msrest.http_logger :     'accept-language': 'en-US'
msrest.http_logger : Request body:
msrest.http_logger : None
requests.packages.urllib3.connectionpool : Starting new HTTPS connection (1): management.azure.com
requests.packages.urllib3.connectionpool : https://management.azure.com:443 "GET /subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/providers/Microsoft.Authorization/r
oleDefinitions/e487ce99-a95d-4b7e-a697-619d4246fc4f?api-version=2015-07-01 HTTP/1.1" 200 None
msrest.http_logger : Response status: 200
msrest.http_logger : Response headers:
msrest.http_logger :     'Cache-Control': 'no-cache'
msrest.http_logger :     'Pragma': 'no-cache'
msrest.http_logger :     'Transfer-Encoding': 'chunked'
msrest.http_logger :     'Content-Type': 'application/json; charset=utf-8'
msrest.http_logger :     'Content-Encoding': 'gzip'
msrest.http_logger :     'Expires': '-1'
msrest.http_logger :     'Vary': 'Accept-Encoding'
msrest.http_logger :     'x-ms-request-id': 'xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
msrest.http_logger :     'X-Content-Type-Options': 'nosniff'
msrest.http_logger :     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
msrest.http_logger :     'Set-Cookie': 'x-ms-gateway-slice=productionb; path=/'
msrest.http_logger :     'Server': 'Microsoft-IIS/8.5'
msrest.http_logger :     'X-Powered-By': 'ASP.NET'
msrest.http_logger :     'x-ms-ratelimit-remaining-subscription-reads': '14880'
msrest.http_logger :     'x-ms-correlation-request-id': 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
msrest.http_logger :     'x-ms-routing-request-id': 'UKWEST:20170404T113645Z:xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'
msrest.http_logger :     'Date': 'Tue, 04 Apr 2017 11:36:45 GMT'
msrest.http_logger : Response content:
msrest.http_logger : Body contains chunked data.
string indices must be integers, not str
Traceback (most recent call last):
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/main.py", line 37, in main
    cmd_result = APPLICATION.execute(args)
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/application.py", line 157, in execute
    result = expanded_arg.func(params)
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/azure/cli/core/commands/arm.py", line 309, in handler
    opres = setter(client, **setterargs) if client else setter(**setterargs)
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/azure/mgmt/authorization/operations/role_definitions_operations.py", line 200, in create_or_update
    body_content = self._serialize.body(role_definition, 'RoleDefinition')
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/msrest/serialization.py", line 297, in body
    data = _convert_to_datatype(data, data_type, self.dependencies)
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/msrest/serialization.py", line 171, in _convert_to_datatype
    localtypes) for key in data
  File "/home/jing/lib/azure-cli/local/lib/python2.7/site-packages/msrest/serialization.py", line 171, in <dictcomp>
    localtypes) for key in data
TypeError: string indices must be integers, not str

By looking at the debug log, it seems the az role definition update command send a get request instead of PUT.
This is the line that highlight the request method is "GET":
https://gist.github.com/activars/ccf06ecfe2bb46316ffbcd82aafe471b#file-output-sh-L58

Any estimation about when this can be resolved?
There's no portal or easy API to perform the task, we are mostly blocked at the moment.

@activars, I will take a look today and let you know

@yugangw-msft  thank you for the quick response.

@activars, the role definition update command  was written through generic update style whose mechanism will not work out with role update, or will be hacky to use. I will rewrite next week. At the same time, you can workaround through the following 2 steps

in your <definition_file>.json , update the Name with the real name  of role definition, which should be a guid
run az role definition create --role-definition

Do you know when this merged fix is going to be released?

It has been released

Thats awesome. I was looking for GitHub issue number in the 2.0.3 release note, didn't managed to located it before asking here.
Thanks for the quick response and quick turn around!

On Apr 24, 2017, at 17:55, Yugang Wang<EMAIL_ADDRESS>wrote:
It has been released
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


feat: use LazySpannerInitializer instead of DCL

The Spanner client library now includes a lazy initializer for Spanner instances that can be used with Google Cloud Functions and other libraries that want to create an instance the first time one is needed and reuse this instance for subsequent requests.
Fixes #2862

cc @saturnism

thanks! /cc @ace-n


Limit Engine Correctness Check

Check the limit result is a subset of the full result, current check may not work for floating point numbers due to weird serialization issues.

How can I reproduce this issue? I assume the issue originates from the float precision problem but I can't test my fix locally.

Although there are no errors currently, enhancing the code could help prevent potential problems. The related test is 'test_limit_engine'.


Can't get tests/test_unittest.py to create screenshots on AWS Device farm

I tried the code @ tests/test_unittest.py and uploaded the bundle zip to AWS device farm. Ran it on a single android device but couldnt get the screenshots to be generated. The test passes though. Please see below screens for test results on AWS DeviceFarm -
https://i.stack.imgur.com/GbxYp.png
https://i.stack.imgur.com/eacob.png
Please guide to fix.

Missed a slash / before filename..


FEATURE delete vs move ambiguous if both provided

Providing the script with the flags -mv /path/to/dir -d causes the script to move the files and afterwards error out when printing stats about the deleted files.
Moved 0 files(s) to "/home/alisot2000/Desktop/test-dirs2/output"
Traceback (most recent call last):
  File "/home/alisot2000/Desktop/lv2/lib/python3.12/site-packages/difPy/dif.py", line 1034, in <module>
    se.delete(silent_del=args.silent_del)
  File "/home/alisot2000/Desktop/lv2/lib/python3.12/site-packages/difPy/dif.py", line 690, in delete
    print(f'Deleted {deleted_files} file(s)')
                     ^^^^^^^^^^^^^
UnboundLocalError: cannot access local variable 'deleted_files' where it is not associated with a value

Suggestions for improvement:

Define a priority (and document it) which argument takes precedent if both are provided
Raise an Error if both are specified.

Hi @AliSot2000,
Thanks a lot for flagging this! This will definitely be considered for the upcoming update.


#16516 : supported mutually exclusive alert glossary/term and tag

Describe your changes:
Fixes #16516

I worked on supporting mutually exclusive alert glossary/term and tag

Type of change:

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@harshach Can you review the text and info banner here for the Mutually Exclusive field? Thanks.

We should show that message when someone turns on the mutual exclusive.
message
"
Warning: If you enable ‘Mutually Exclusive’ for a classification, users will be restricted to using only one tag to apply to a data asset. Once this option is activated, it cannot be deactivated.”


bedrock-runtime streaming not working

Hi,
I just tried running:
import boto3

client = boto3.client(service_name="bedrock-runtime",
                        aws_access_key_id=AWS_ACCESS_KEY_ID,
                        aws_secret_access_key=AWS_SECRET_ACCESS_KEY,
                        aws_session_token=AWS_SESSION_TOKEN,
                        region_name="ap-southeast-1")

@ell.simple(model="anthropic.claude-3-5-sonnet-20240620-v1:0", api_params={"stream":True}, client=client)
def hello_from_bedrock_streaming():
    """You are an AI assistant. Your task is to respond to the user's message with a friendly greeting."""
    return "Say hello to the world!!!"

hello_from_bedrock_streaming()

and it returns nothing. It works fine when I set api_params={"stream":False}.

Not sure why, could you please guide how to fix this?

Hi, which version of ell-ai did you use? Using 0.0.13 on my side works.

Hmm, strange.. I also used the latest version this morning (0.0.13).

@brnaba-aws can you give me an api key to test with?


chore: release 1.2.294

:robot: I have created a release *beep* *boop*
1.2.294 (2024-02-19)
Bug Fixes

deploy: dont strip non-whitespace after --- in manifests (1b4c7f2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/werf/releases/tag/v1.2.294 :sunflower:


Init image for SubstrateVM.

Added an init image for SubstrateVM compilation.
The init --init-image fnproject/fn-java-native-init generates a template with docker runtime building a SubstrateVM image for Java Maven project.

I demoed SVM support at the Hamburg JUG yesterday using this PR and it worked nicely! 😃 I've got a couple of comments:

The name of the generated binary is fn which could lead to confusion as our cli is called fn.  How about func which is the expected name in Go (func.go) and Node.js (func.js)?

The init image generated Dockerfile hardwires the name of the target class and method but should be picking it up from the func.yaml file to align with the existing JDK behaviour.

To @shaunsmith:

The name of the generated binary...
Yes, I will change the fn to func.

The init image generated Dockerfile hardwires the name of the target class and method but should be picking it up from the func.yaml file to align with the existing JDK behaviour.
As I understand the cli it's not possible in the docker type runtime. The cmd from the func.yaml is inserted by the the cli only for language helpers by writeTmpDockerfile in dockerBuild

@tzezula You're right, my thinking is wrong here.  But I see the class and method name also appears in the reflection.json.  If the purpose of the init image is to create a scaffolding that a user can use to get started quickly and modify I think we should use the name of the function as the name of the class so it's close to what they are trying to get to.  We don't do this now in the old --runtime boilerplate but it's something @delabassee has been asking for.  The move to init-image give us a chance to improve our code generation. :)  So if you take the function name and use it for the class name, put it in the Dockerfile, and in the reflection.json then at least we're a little closer to our end goal.
For Java, the obvious next step would be for the user to be able to specify the package they want the function class to be in.  @mjg123 here's a good use case for supporting additional parameters to init-image!

The function name is passed into the init image through the FN_FUNCTION_NAME environment variable.  An init-image that unpacks a tar is just the simplest example of how this can work.  You're in a container and can run whatever code you want to generate the output.  In the example the command that's run is CMD ["cat", "/init.tar"] but it could be anything.  The next simplest thing after tar is probably a shell script that uses sed to substitute in the name of the function into your template files, builds a tar file, and then cats it to stdout.

@shaunsmith Thanks! The FN_FUNCTION_NAME was the missing part.
Fixed.

I've updated the PR to resolve the comments.

fn renamed to func
Using ${FN_FUNCTION_NAME} when generating the template.
Using the release.version for fdk.version
Added rm -rf /var/lib/apt/lists/* to apt-get...

Questions: The artefacts for the version 1.0.64 from the release.version are not available in the https://dl.bintray.com/fnproject/fnproject/.

the version in release.version is the forthcoming release (next) not the previous version. we had a glitch with our release script which stopped 1.0.65-1.0.69 from being pushed, this is fixed.

the release script (and on each branch) already does the following prior to each build:
# We need to replace the example dependency versions also
# (sed syntax for portability between MacOS and gnu)
find . -name pom.xml |
   xargs -n 1 sed -i.bak -e "s|<fdk\\.version>.*</fdk\\.version>|<fdk.version>${release_version}</fdk.version>|"
find . -name pom.xml.bak -delete

so any fdk.version properties referenced in pom files should be up-to-date

I'm just testing this with the latest FDK version 1.0.70 and I see
the build works fine but then I see:
Building image foofn:0.0.1 .
Exception in thread "main" com.oracle.svm.core.jdk.UnsupportedFeatureError: Unsupported method java.lang.Class.getAnnotationsByType(Class) is reachable: The declaring class of this element has been substituted, but this element is not present in the substitution class
        at java.lang.Throwable.<init>(Throwable.java:265)
        at java.lang.Error.<init>(Error.java:70)
        at com.oracle.svm.core.jdk.UnsupportedFeatureError.<init>(UnsupportedFeatureError.java:31)
        at com.oracle.svm.core.jdk.Target_com_oracle_svm_core_util_VMError.unsupportedFeature(VMErrorSubstitutions.java:109)
        at java.lang.Class.getAnnotationsByType(Class.java:3434)
        at com.fnproject.fn.runtime.EntryPoint.run(EntryPoint.java:81)
        at com.fnproject.fn.runtime.EntryPoint.main(EntryPoint.java:43)
        at com.oracle.svm.core.JavaMainWrapper.run(JavaMainWrapper.java:163)

Is there something I can do to fix this (this code was added recently?)

@zootalures I am just solving it. It was introduced by:
FnFeature[] features = method.getTargetClass().getAnnotationsByType(FnFeature.class);

Doh - is that specific API not supported or is it a more general annotations issue?

I am now building the SubstrateVM from revision dca98069dc57c458b8bcb3392cf9e66316dd37b6 which is the fix of missing Class.getAnnotationsByType(Class). When there will be a release candidate containing the fix (1.0rc8) I will create a follow up pull request to build the SubstrateVM from the release candidate 1.0rc8.

Merge the other change I think this can be merged after a rebase/test and we can iterate from there