[storage-file-datalake] Allow DataLakePathClient.move() use sas to authenticate source and destination

Fix issue #12758

I forgot to add the fallback logic for re-using source SAS. Will add it.

Would you mind adding some test cases? I can continue working with it if needed.

source with '/' in the file path

source with 'versionid' or 'snapshot' in the query parameters - probably will fail but is expected.

source and destination both with sas

source with shared key credential, destination with sas - the current behavior is the service would authenticate with the shared key credential.

source with shared key credential, destination without sas

destinationPath url encoded

destinationPath not encoded

I have added several tests and ran them through my own storage account in recording mode (turn on recording mode, configure my storage account credentials in ENV, run the tests). Could you help me check if the tests are written properly?

/azp run js - storage-file-datalake - tests

Changes look good to me. 👍
We also need to record the newly added test cases to pass the CI via setting the TEST_MODE to record in .env and then run the tests to generate the recordings.

/azp run js - storage-file-datalake - tests

It seems like the recorder auto-redacts my private information. I have added the recording files after running those tests in record mode with my storage account.

/azp run js - storage-file-datalake - tests

Fixes #12758
Also, would you mind adding a changelog entry for this?

Fixes #12758
Also, would you mind adding a changelog entry for this?

Added a bullet in the "Unreleased" section.

Added a bullet in the "Unreleased" section.


Feature: reuse Swing fonts on desktop

There isn't an easy way to take a font used from Swing and apply it to CfD too. That makes it very hard to integrate with existing Swing apps.

This was done by me in Skiko. Closing as completed.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[FEATURE] Add CI to test bpftrace

Is your feature request related to a problem? Please describe.

bpftrace is an important tool bpftime currently support. We need a CI test to ensure future development will not bring new bugs with bpftrace.
Describe the solution you'd like

Describe alternatives you've considered

Add a CI to test some bpftrace scripts in GitHub Actions.
The CI file can be similar to https://github.com/eunomia-bpf/bpftime/blob/master/.github/workflows/test-examples.yml
And the bpftrace commands can be taken from https://github.com/eunomia-bpf/bpftime/blob/master/example/bpftrace/README.md.
Provide usage examples

Additional context

Hey I am already onto a workflow related issue I think I got the idea what needs to be done, feel free to assign this to me!

Thanks @Sanket-0510 !

Hey @yunwei37 I am already onto a workflow related issue, I think I got the idea what needs to be done, feel free to assign this to me!

Hello, how's this issue going?

I have exams this week, If anyone else wants to work on this feel free to assign this to them.

Test shows error on my site:

ubt 22.04 Jammy / make release-llvm-jit / bpftrace version 0.9.4

Tried some other versions of ubuntu (23.04 as bpftime requires), bpftime of other compilation commands (vm or not, jit or not), bpftrace versions(0.15+). Error stills.
Could you please give further information about your successful test?

Test shows error on my site:

ubt 22.04 Jammy / make release-llvm-jit / bpftrace version 0.9.4

Tried some other versions of ubuntu (23.04 as bpftime requires), bpftime of other compilation commands (vm or not, jit or not), bpftrace versions(0.15+). Error stills.
Could you please give further information about your successful test?

Please enable ENABLE_LLVM_SHARED when configuring using CMake. This should be enabled by default in the recent versions of bpftime


remove prow job for go-repo-template.

What this PR does / why we need it:
remove prow job for go-repo-template, since we have to move these jobs to travis to support multi-arch build.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Specify your PR reviewers:

/cc @gyliu513 @morvencao @clyang82
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/assign @gyliu513

/lgtm


Add securityContext and podSecurityContext values

In some clusters, a validation controller can prohibit creating pods that have not dropped capabilities, or require SELinux, AppArmor, or some other security feature. This exposes the Kubernetes SecurityContext feature so that it can be configured via helm values.

make securityContext and podSecurityContext configurable
add changelog entry

When installing StackStorm through helm, I get the following error on the MongoDB container:
> kubectl logs -f -n stackstorm stackstorm-mongodb-0
Advertised Hostname: stackstorm-mongodb-0.stackstorm-mongodb-headless.stackstorm.svc.cluster.local
Pod name matches initial primary pod name, configuring node as a primary
mongodb 15:09:55.49
mongodb 15:09:55.49 Welcome to the Bitnami mongodb container
mongodb 15:09:55.49 Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-mongodb
mongodb 15:09:55.49 Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-mongodb/issues
mongodb 15:09:55.50
mongodb 15:09:55.50 INFO  ==> ** Starting MongoDB setup **
mongodb 15:09:55.52 INFO  ==> Validating settings in MONGODB_* env vars...
mkdir: cannot create directory '/bitnami/mongodb/data': Permission denied

As far as I can tell, this is related to https://github.com/bitnami/bitnami-docker-mongodb/issues/177 and could be fixed if this merge request was merged :)


Task/delete trillium start

Resolves #243

delete the /trillium/start page
remove all associated tests
update hidden redirects to go to trillium/rent now
remove it from the Cypress test

✂ Perfect, good work! ✂


Fix some mark sweep pruner bugs where nodes that should be kept were being swept

PR description
Adds integration tests to catch bug where the mark storage was being cleared before a mark began. Instead, the mark storage is now cleared upon preparing the mark sweep pruner
Fixes bug where the pending marks where not being checked while pruning was occurring. By removing the flush in sweepBefore, the existing tests catch the bug.
Fixed Issue(s)

https://pegasys1.atlassian.net/secure/RapidBoard.jspa?rapidView=28&projectKey=PIE&modal=detail&selectedIssue=PIE-1945

This needed to be rolled back because of DCO tagging.  #50 is the PR that will cover it.


Fix excessive synchronization in contract on Dagger.DArray

Closes #123

Can we add some tests? :)

Yeah, we should. Currently, we are trying to fix some things in order to make the code work.

Updated the code to the last Tenet and Dagger changes. Also it now calls Tenet.contract instead of OMEinsum.EinCode.
@jofrevalles I also added some tests.


how to display airflow job on dr.elephant

I have config airflow baseurl in SchedulerConf.xml,and restart dr.elephant。I run a job on airflow,when I  open dr.elephant webUI, I only can see a hive job ,but I can't see airflowjob .So, what more config  need I to config ? else someting I don't know. thank you! I have deal it one day!!!!!!!

how to modify airflow hive-operator ?What should I modify in hive operator?where should I modify in hive operator?


Added README-hi.md for README in Hindi Language.

It is the README.md in Hindi Language Mostly Spoken In Asian Countries, Especially India. It Currently Covers Topics Till Route Management But Would Will Gradually  Translate It Completely.

Thank you so much for this.
It would be great to have the specific files for state management, dependency injection and routes translated as well, but the main document is already a huge improvement.


Crash with WAILA

https://paste.dimdev.org/ezipewoxaj.mccrash
Don't have much information at this time regarding this, just forwarding the crash from a user of the modpack this is in.

Additional info from user

Would you happen to have a link to the modpack so I can try to recreate the bug?

https://www.curseforge.com/minecraft/modpacks/dojopack
There you go :)

Any news on this issue or estimates for a potential fix?

Hi, sorry for the delay.. I've took a look into this and I haven't been able to recreate this issue. What I will do, is I will place an extra precaution at the position that the error occurred so that this issue should not happen again. The new version should release shortly. If it does happen again, be sure to let me know!


Can not use video on cloud storage - "Invalid 'manifest.jsonl file structure': 'extension' is required, but not found".

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Created a matifest.jsonl for mp4 video. Uploaded both to S3 cloud storage and created a cloud storage in cvat server.
Then created a project and a task and selected a cloud storage with corresponding video manifest.
So system gives me an error:
Could not fetch content for cloud storage #5
Error: Request failed with status code 400. "Invalid 'manifest.jsonl file structure': 'extension' is required, but not found".

Contents of manifest.jsonl is
{"version":"1.1"}
{"type":"video"}
{"properties":{"name":"video25.mp4","resolution":[1920,1200],"length":3270}}
{"number":0,"pts":0,"checksum":"4815365990e6644fa58ee9f52383ad3d"}
{"number":12,"pts":3072,"checksum":"d7e7a38c416541a9040acc3165fd6b79"}
{"number":24,"pts":6144,"checksum":"3fa58d757b91c17a2a225f1d4d1316a7"}
{"number":36,"pts":9216,"checksum":"be37551a52327874fdcf2ebc8114689b"}
{"number":48,"pts":12288,"checksum":"be0564207bb2dbcf5f04995d5c3b6f8b"}
{"number":60,"pts":15360,"checksum":"57dbecc197fd5358fa2e6802231fa2a8"}
{"number":72,"pts":18432,"checksum":"a486049853a6b6785096d8cd76887374"}
{"number":84,"pts":21504,"checksum":"3a58ee985c2259b0a67786aa8e71692f"}
{"number":96,"pts":24576,"checksum":"7a07a6e27085e8c30390afa8d7279f78"}
{"number":108,"pts":27648,"checksum":"ec3879dd92c4f00256788d4b6b47ceed"}
...

 No
Operating System and version (e.g. Linux, Windows, MacOS): Linux bright 5.19.12-arch1-1

Duplicate of #4460, #4853

I will close the issue as a duplicate.


[Benchmark] Add MMSearch illustration in README

Add MMSearch illustration in README

Hi @CaraJ7. Wanted to ask if you are planning to add the MMSearch benchmark to VLMEvalKit? For the repository README we generally recommend making changes at the end after contributing the base code.


Add AR Retinal Resolution [ARRR] axis

Overview
The AR Retinal Resolution proposal aims to improve the user experience of AR/VR headsets by enabling resolution-specific enhancements. As the resolution of these headsets continues to evolve towards retina resolution (60 PPD), it will become increasingly challenging for designers to create scalable experiences that cater to a wide range of devices. Traditionally, designers have optimized text for the latest technology, resulting in substandard experiences for users with previous-generation devices.
To address this issue, the proposal suggests using typefaces with resolution-specific adjustments that are duplexed (uni-width). Some of the adjustments in AR One Sans include:

Addition of flaring at stroke ends to reduce the rounding effect at the ends due to missing pixels (in low resolution). It optimizes the shape of letters minimizing the rounding effect that can occur with rectangular strokes, making them appear more like rounded rectangles.
Light traps to reduce excess glow at the intersection of strokes.
Large x-height for a lower resolution to increase the counters and apertures and balance the negative space inside the letters due to flaring.

This approach empowers designers to create a single design (UX) that can be rendered across a large range of headsets, regardless of resolution since there is no change in page layout across devices. By mapping the AR retinal resolution (ARRR) to the PPD resolution of the headsets, users can enjoy the best possible experience as the typeface is rendered based on their device's resolution.
Overall, the AR Retinal Resolution proposal seeks to provide a solution to the challenge of designing scalable experiences for a wider audience by enabling designers to create designs that work across a range of devices, resulting in a better user experience for all users.
Image

GIF illustrating the flaring and change in x-height on ARRR axis.
Proposed Axis Details
Tag: ARRR
Name: AR Retinal Resolution
Valid numeric range: 0 to 60
Description: Resolution-specific enhancements in AR/VR typefaces optimize text without changing the width of the letters, maintaining consistent spacing and kerning, and preserving layout and line breaks. This ensures a consistent design and layout across different resolutions, making designs accessible and easy to read, regardless of the headset's resolution.
Scale interpretation: In the proposed resolution-specific adjustments for AR/VR typefaces, the lower value of the axis corresponds to the PPD of low-resolution headsets, while the upper value corresponds to 60 PPD, which is considered retina resolution. Beyond this point, the effect of adjustments on legibility and readability becomes negligible. Therefore, by limiting the adjustments to this range, designers can optimize the legibility and readability of their typefaces for a wide range of headsets while minimizing the amount of adjustment work required.
Recommended Use: Is to map the ARRR to the resolution of headsets and render the typefaces according to that.
Reference PPD values of headsets:
Varjo XR-3: 70 PPD in its focus area and 30 PPD in its peripheral area.
Hololens 2 (AR): 47 PPD
Oculus Quest (VR): 14.4 PPD
PSVR (VR): 9.6 PPD
Other Supporting Information
Pixel Density in AR/VR headsets
VR headset PPD table
Importance of retinal resolution in AR/VR

@vv-monsalve @davelab6 Let's continue the discussion here :)

Bringing here the previous comments in the upstream issue to continue the discussion here :)
@davelab6 question:

Is there a way to convert ppd to Points? Would there be a font with both opsz and ARRR axes?

@niteeshy answer:

I'm not too sure about the conversion of PPD to points. To calculate the HMD’s pixel density in PPD (the number of pixels per degree it presents to the eye), the number of pixels in a horizontal display line has to be divided by the horizontal field of view provided by the lens. Example: Oculus Quest has a resolution of 1440x1600 per-eye and FOV of 93° horizontal, so the PPD come out to be 15.31 (1440/93).
I was looking at PPD as a reference unit since it would be easy to understand based on the retinal resolution concept. Any specific reason to convert it into points?
Yes, there is a possibility of having opsz alongside ARRR as it can make it easy for designers to deliver size-specific variants/ adjustments.

@davelab6

Per (2) then I think this axis regisration should proceed

Taking the above, so far these would be the
Axis metadata fields
#(ARRR) based on https://github.com/niteeshy/ar-one-sans
tag: "ARRR"
display_name: "AR Retinal Resolution"
min_value: 0
default_value: 0
max_value: 60
precision:
fallback {
  name: "Default"
  value: 0.00
}
fallback_only: false
description:
  "Glyphs rotate up and down, negative values tilt down"
  " and positive values tilt up, in the Y dimension."
  " Resolution-specific enhancements in AR/VR typefaces"
  " to optimize rendering across the different resolutions of the headsets"
  " making designs accessible and easy to read."

I'm hesitant about the min value. It should be the lowest value available in the market. Taking the details provided above, probably 9PPD

@vv-monsalve There are a few headsets which have a lower resolution as well. Zero is too extreme and unfeasible, so we can use 5 as the min value. In AR One Sans I have used 10 as min supported value.
Here most of the designs are approximate adjustments so decimal values does't seem to be necessary.

@MariannaPaszkowska, this is the new AR-related axis proposal ;)

In AR One Sans I have used 10 as min supported value.

We can take this as a minimum value reference then.

That sounds good. Anyways the readability of text below 10 PPD will not be very good. Usage wise: 10 PPD ARRR value can be default.

After today meeting this would be the metadata fields
#(ARRR) based on https://github.com/niteeshy/ar-one-sans
tag: "ARRR"
display_name: "AR Retinal Resolution"
min_value: 5
default_value: 10
max_value: 60
precision: 0
fallback {
  name: "Default"
  value: 0
}
fallback_only: false
description:
  " Resolution-specific enhancements in AR/VR typefaces"
  " to optimize rendering across the different resolutions of the headsets"
  " making designs accessible and easy to read."

This is good for the axis registry, but will need a lot of explanation in GFK Glossary page :) Only one small suggestion here:

headsets

headsets,


Confused: cbarrick's version vs. this?

npm install gulp-run gets cbarrick's deprecated version, but npm home gulp-run comes here.  If you've taken over, can you publish a new version to true things up?

The reason this is an issue is that gulp prints Warning: gulp-run is deprecated on every run, which is causing people to file bugs on my project.

Just a warning. Currently, the next version is not ready for production. So you need to wait a few days.

So... have "a few days" passed already? Any help still needed?

Done.


Missing files

Thank you for sharing your models.

from trafficintelligence import moving, cvutils, storage
ModuleNotFoundError: No module named 'trafficintelligence'

I got these files already!


License

Project does not have a license file. This might be required as part of putting project on PyPI. Need to check and if required add one to the repo.

Added license


replaced regedit with winreg

This has been failing nix builds all week.  What is the status of this PR?

Looks fine.

@Ajay-MS, @kmkumaran  I am closing this PR as went ahead with optional packages approach.

Taking care of this changes in a different PR


Question: about signup workspace owner and signin members

Description
Hi everyone, I'm starting a new project which will allow only the workspace owner to signup by installing the app, and store the credentials if I'm not wrong oAuth will do that for me, am I right?
the question is: I want after the workspace owner completes the signing up, all the members in that workspace will be able to signin and use my app under his workspace (like slack), which means he is able to switch workspaces also in my app if the workspace owner has signed up.
What do I need for this? any help?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Hi @NurdinDev, thanks for asking the question!
Until your app is installed into a workspace by its admin user, your app is unable to know whether an installing user is an admin. So, one feasible way to realize your secure installation flow would be:

Use CallbackOptions.afterInstallation to check the installing user's authority (calling users.info API and check if is_admin is true). If the user is not an admin, your app can revert the installation by calling auth.revoke API and return false to skip saving the installation plus display error page to the user. Refer to https://slack.dev/node-slack-sdk/reference/oauth#callbackoptions for details.
As for a workspace that its admin already installed your app, your app can check the condition in CallbackOptions.afterInstallation and then proceed by returning true.
Your app may need to delete all the installations once a workspace's admin removes your app from the workspace. You can  utilize "tokens_revoked" event delivery for triggering this operation.

I hope this was helpful to you.

Thanks, @seratch it's very helpful
Regarding the part about deleting all installations, why should I have many installations in the same workspace?

@NurdinDev hm,

all the members in that workspace will be able to signin and use my app under his workspace (like slack)

I might misunderstand this part.
I thought that this means that, after admin enables your app in a workspace, each individual user may do either their OAuth flow with user scopes or Sign in with Slack (OpenID Connect). In this case, your app's datastore needs to store multiple user-level installation data in addition to the admin's installation for bot scopes.
If you meant "Once an admin installs your app, anyone in the workspace can use the app and no one needs to go through the OAuth flow", your app will have only one installation data per workspace.

@seratch Sorry for confusing you with my question, I couldn't explain it well.
The main idea is to install the slack app on each workspace registered on my website, and the users in the workspace are able to login into my website, and for example, "send messages from the website"
imagine the flow is similar to slack when you register you have to create a workspace and then a subdomain will be created for your workspace, same here but when you register on my website you will integrate your slack workspace instead of creating one, and then, for instance, the website will create a dashboard and fetches all the users from the workspace, which will allow anyone in this workspace to login.
I think each tenant should have a separate slack app installed in my case! 🤔

@NurdinDev Thanks for your reply.
In this case, Sign in with Slack (OpenID Connect) is the way to go for non-admin users who sign in on your website. Your website can check the workspace's ID (team_id) in its response. When the tenant for the workspace is already created, the signed-in user can access dashboards and so on.
Only admin users need to go through the OAuth flow with bot scopes such as channels:history, chat:write, files:read, and so on (see the full list here).
As for the OpenID Connect authentication, unfortunately, bolt-js does not support it. So, please directly go ahead with bolt-js's underlying @slack/oauth package. Here is a simple example app demonstrating how it works: https://github.com/slackapi/node-slack-sdk/tree/main/examples/openid-connect
I think that I've answered all of your questions here. Would you mind closing this issue? I'd suggest doing some prototyping first. And then, whenever you have further questions, please feel free to create a new question issue!

Thank you @seratch for your time, It was very helpful to ask you and validate some questions in my mind.


Fixed android toolchain with newest NDK

Describe the pull request
ANDROID_NATIVE_API_LEVEL has been removed from NDK cmake toolchain so we now have to use ANDROID_PLATFORM

What does your PR fix?

Using latest android NDK

Which triplets are supported/not supported? Have you updated the CI baseline?

Only affects android triplets

Does your PR follow the maintainer guide?

Yes

why not setting both, to preserve compatibility with older ndk?

I noticed some weird stuff when setting both. Some versions of the NDK do not execute the same code when both are set (checks are made in the toolchain if both variables are set or not set).
ANDROID_PLATFORM is actually here since a long time but it was not clear which variable was "the new one" when I first added it to this script.

cc @talregev @tronical @jherico @huangqinjin @jamiebk for review this PR.

@AenBleidd

CC @holmescn @m-kuhn

If I recall correctly, since NDK 19, ANDROID_NATIVE_API_LEVEL is same as ANDROID_PLATFORM, as stated in https://developer.android.com/ndk/guides/cmake#android_platform. But before that, ANDROID_PLATFORM only supports android-NN format.
Seems the latest NDK dropped ANDROID_NATIVE_API_LEVEL for new toolchain, but still support for legacy toolchain. Sorry to bother @DanAlbert, is that intentional?

If I recall correctly, since NDK 19, ANDROID_NATIVE_API_LEVEL is same as ANDROID_PLATFORM, as stated in https://developer.android.com/ndk/guides/cmake#android_platform. But before that, ANDROID_PLATFORM only supports android-NN format.
Seems the latest NDK dropped ANDROID_NATIVE_API_LEVEL for new toolchain, but still support for legacy toolchain. Sorry to bother @DanAlbert, is that intentional?

File an NDK bug please.

Building with this change failed on compiler-detect at least on armv6:
CMake Error at /home/runner/work/boinc/boinc/3rdParty/android/android-ndk-r15c/build/cmake/android.toolchain.cmake:408 (message):
  Invalid Android platform: 16.
Call Stack (most recent call first):
  /home/runner/work/boinc/boinc/3rdParty/android/vcpkg/scripts/toolchains/android.cmake:46 (include)
  /home/runner/work/boinc/boinc/3rdParty/android/vcpkg/scripts/buildsystems/vcpkg.cmake:215 (include)
  /usr/local/share/cmake-3.21/Modules/CMakeDetermineSystem.cmake:124 (include)
  CMakeLists.txt:2 (project)

So in order to fix the usage of the newest NDK, a little bit more work is required here.

Filed bug https://github.com/android/ndk/issues/1610.

I think CMAKE_SYSTEM_VERSION should change both:
ANDROID_NATIVE_API_LEVEL and ANDROID_PLATFORM
Then it will support old and new android sdk.
Can you try?

I think CMAKE_SYSTEM_VERSION should change both: ANDROID_NATIVE_API_LEVEL and ANDROID_PLATFORM Then it will support old and new android sdk. Can you try?

The issue with setting both is that the 2 variables are not fully compatible (in syntax), depending on the NDK version used by the caller.
I noticed some weird issues/errors when trying to set both at first.
Not sure what is the best solution here, just that for people updating to the latest NDK, vcpkg is not working anymore.

If both not working, then we need to separate the update according to NDK version.
old version (let say 21 and less) will update the old var: ANDROID_NATIVE_API_LEVEL
The new version will update the new var: ANDROID_PLATFORM
You can take the version of the ndk from the cmake (or the path) that user will providing you.
Can you try?

You can take the version of the ndk from the cmake (or the path) that user will providing you.
Can you try?

I unfortunately won't have the time to test this at the moment.
I'm not sure it will be that easy as you have to define the variables before including the NDK cmake toolchain, so it might be difficult to get a value from it before including it, to define said variable based on what has been retrieved.
Maybe someone has a better idea, but currently vcpkg is broken if you get latest NDK. Setting both variables will break vcpkg for people with an older NDK depending on what is set in their CMAKE_SYSTEM_VERSION

You don't have to include the cmake to find version number. maybe simple parse from one of the txt/cmake file for ndk version.

I look on the android.toolchain.cmake inside the ndk 15. we actually can use ANDROID_PLATFORM
we need to set it to 'android-16` and not just 16.
@AenBleidd Can you try and let us know?

@talregev, not sure I got your idea.

@AenBleidd I am writing you in discord.

ok. I understand the confusion.
the ANDROID_PLATFORM also work in old ndks but in needed to add the prefix android- before the platform number that it will work.
For backward compatibility and not confuse others, I am suggesting another change:
to check if CMAKE_SYSTEM_VERSION it just a number, and if yes, add it android- prefix to the number.
if(CMAKE_SYSTEM_VERSION MATCHES "[0-9]+$")
    set(ANDROID_PLATFORM android-${CMAKE_SYSTEM_VERSION} CACHE STRING "")
else()
    set(ANDROID_PLATFORM ${CMAKE_SYSTEM_VERSION} CACHE STRING "")
endif()

We change our triplets to
set(VCPKG_CMAKE_SYSTEM_VERSION android-x) and it working. (also working before and after your change).

So everyone approved this changes?

I think before we can refactor the entire build process, any patches would temporarily work. Although Android NDK is not evolved so fast, version fragmentation is serious. Issue #21345 tried to discuss and resolve this problem from scratch. But any effort is welcome.

@JackBoosY, I'm ok with these changes.

@JackBoosY Please consider my suggestion before merge.

ok. I understand the confusion. the ANDROID_PLATFORM also work in old ndks but in needed to add the prefix android- before the platform number that it will work. For backward compatibility and not confuse others, I am suggesting another change: to check if CMAKE_SYSTEM_VERSION it just a number, and if yes, add it android- prefix to the number.
if(CMAKE_SYSTEM_VERSION MATCHES "[0-9]+$")
    set(ANDROID_PLATFORM android-${CMAKE_SYSTEM_VERSION} CACHE STRING "")
else()
    set(ANDROID_PLATFORM ${CMAKE_SYSTEM_VERSION} CACHE STRING "")
endif()

@christophe-calmejane Can you try my suggestion?
https://github.com/microsoft/vcpkg/pull/21465#issuecomment-975966155

I quickly checked and it fails if you set CMAKE_SYSTEM_VERSION to android-15 for exemple

There is no android-15. the number is no ndk. we  using android-16.

https://developer.android.com/ndk/downloads/revision_history
from ndk 18 google drop the support for android 15

Android NDK r18b
Support for ICS (android-14 and android-15) has been removed. Apps using executables no longer need to provide both a PIE and non-PIE executable.

Sorry I just used any random number when replying, use any number you want, it fails with your change if you set android-NN

use any number you want, it fails with your change if you set android

Can you tell me what ndk are you using, and past here the error?

Since I have less knowledge with Android, I will follow the opinions of most people.

use any number you want, it fails with your change if you set android

Can you tell me the ndk version are you using, and past here the error?

Your code causes android-19 to be transformed to android-android-19 which is bad.
It might just be as simple as to add strict matching "^[0-9]+$" but I don't have time this week to test more

Thank you that you find error in my regular expression. I meant if the var contain only digits. can you fix  it?

if(CMAKE_SYSTEM_VERSION MATCHES "[0-9]+$")
    set(ANDROID_PLATFORM android-${CMAKE_SYSTEM_VERSION} CACHE STRING "")
else()
    set(ANDROID_PLATFORM ${CMAKE_SYSTEM_VERSION} CACHE STRING "")
endif()

@christophe-calmejane
I understand the my error
Also read here:
https://cmake.org/cmake/help/latest/command/string.html#regex-specification
Can you test it with the fix?
if(CMAKE_SYSTEM_VERSION MATCHES "^[0-9]+$")
    set(ANDROID_PLATFORM android-${CMAKE_SYSTEM_VERSION} CACHE STRING "")
else()
    set(ANDROID_PLATFORM ${CMAKE_SYSTEM_VERSION} CACHE STRING "")
endif()

Sorry for the long delay, I'm gonna test some changes as soon as I find a bit of time, hopefully before the end of the week.

I pushed a new commit, including your proposal. Seems fine on my side.

@talregev What's your opinion now?

I am happy with the changes. It also support back compatibility.
@christophe-calmejane Thank you for your changes and fix.
@JackBoosY Can you merge it please?

Maybe we should also update our docs.

Maybe we should also update our docs.

I don't see anywhere any of these variables are mentioned in our docs for Android.

Thanks for the bugfix!


Auth backend saml clock discrepancies

This is a pull request about SAML configuration
It allow to configure acceptedClockSkewMs option.
Closes #6608
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I dont understand why this fail : https://github.com/backstage/backstage/actions/runs/1071270059

Looks like a swagger dependency release broke the e2e tests however, we've fixed it in #6626 so you should be able to go ahead and rebase from master and the build pass.


Upstream v3.20.0

Fixes #300
Fixes #297
Update to upstream v3.20.0 with @lukehoban's patch to replace the github.com/hashicorp/vault dependency.
Unfortunately, the dependency trim does not resolve #197.

Snyk rejects on security but not license.


better icon for "discrete" switch

Currently the "discrete" switch at the rightmost of the knob controller combo is implemented using this code:
Image(imageVector = if (discrete) Icons.Default.CheckCircle else Icons.Default.Close,
        "send", Modifier.size(32.dp))

It is not looking very obvious. But I have no idea what could be better. We cannot hover mouse cursor on mobiles so it must be self-explanatory.

It is "fixed" as bee5b13cc227a8983f729f39833b7cb62feea617


Flashy/Incorrect bevior on theme change in flutter web deployment on mobile devices

Describe the bug
As discussed in other issues we can prevent a flashy bevior on Theme changes and set initial values when passing it directly
into the AdaptiveThemeWidget when passing an savedThemeMode before runApp.
However, I am currently experiencing a flash in the browser status bar and browser bar when switchting from light to dark mode and vice versa.
There is no flash when I build for Android
There is no flash when I build for Web and access with DesktopBrowser (tested with Chrome and Firefox)
THERE IS A FLASH When building for web and accessing through a  Browser on a mobile device! (Tested with three phones, two of them recent Android phones and one recent Iphone)
It is very hard to reproduce and to demonstrate because of this but for demonstration purpose I post my sample site where it is deployed. You need to open this using your mobile phones browser Test Site. Just hit the theme change button and should see the incorrect bevior.
https://github.com/BirjuVachhani/adaptive_theme/assets/9419583/6133f083-e567-48c0-8999-1e203d706a85
To Reproduce
Steps to reproduce the behavior:

Go to Test Site with your phones browser
Click on the theme change button
3 See the flash in the status bar and browser bar.

Expected behavior
No flash and same expeierence as on a desktop browser
Smartphone (please complete the following information):
1 Recent Iphone using browser  (no data because not my device)
1 Android Phone from 2021 using browser  (no data because not my device)
1 Android Phone Huawei P30 Pro using browser (my phone)
Additional context
I am using adaptive_theme: ^3.3.0
This is the code of used main.dart file
void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  final savedThemeMode = await AdaptiveTheme.getThemeMode();
  runApp(MyApp(savedThemeMode: savedThemeMode));
}

class MyApp extends StatelessWidget {
  final AdaptiveThemeMode? savedThemeMode;
  const MyApp({super.key, this.savedThemeMode});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return AdaptiveTheme(
        light: ThemeData(
          colorSchemeSeed: Colors.green,
          brightness: Brightness.light,
          appBarTheme: const AppBarTheme(
              systemOverlayStyle: SystemUiOverlayStyle(
                  systemNavigationBarColor: Colors.green,
                  systemNavigationBarIconBrightness: Brightness.dark)),
          useMaterial3: true,
        ),
        dark: ThemeData(
          colorSchemeSeed: Colors.green,
          brightness: Brightness.dark,
          appBarTheme: const AppBarTheme(
              systemOverlayStyle: SystemUiOverlayStyle(
                  systemNavigationBarColor: Colors.green,
                  systemNavigationBarIconBrightness: Brightness.light)),
          useMaterial3: true,
        ),
        debugShowFloatingThemeButton: true,
        initial: savedThemeMode ?? AdaptiveThemeMode.light,
        builder: (theme, darkTheme) => MaterialApp(
              title: 'FoodSnapAI',
              theme: theme,
              darkTheme: darkTheme,
              home:
                  const MyHomePage(title: 'FoodSnapAI page - work in progress'),
            ));
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

  @override
  void initState() {
    super.initState();
  }

  void _toggleDarkMode() {
    final manager = AdaptiveTheme.of(context);
    if (manager.mode.isLight) {
      manager.setDark();
    } else {
      manager.setLight();
    }
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.primaryContainer,
        title: Text(
          widget.title,
        ),
        actions: [
          Container(
              margin: const EdgeInsets.only(right: 20.0),
              child: IconButton(
                  onPressed: _toggleDarkMode,
                  icon: Icon(AdaptiveTheme.of(context).mode.isDark
                      ? Icons.dark_mode
                      : Icons.dark_mode_outlined)))
        ],
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            BuildingCard(),
          ],
        ),
      ),
    );
  }
}

@alexw92 Would you also mind providing a screen record of what you're seeing? Would be much more helpful for me trying to reproduce! Thanks!

@BirjuVachhani Sure, I just recorded a short video with my phone which shows the issue. It is attached above

I am just passing by.
On android 13, Edge browser v116 it works fine.
https://github.com/BirjuVachhani/adaptive_theme/assets/119455938/ec2e8107-4717-4d36-b5f8-fb70e499f20e

thanks @giga10 for your feedback.  I updated the dependency.
So for me it looks like that the status bar is onTheme change quickly changed to green before that change is taken back immediately. Also shouldnt Chrome itself go dark in darkmode? When I open github on my phone the chrome bar is dark for instance. I have do clue what I am doing wrong here.

I checked on chrome and safari on iOS 16 and it seems to be working just fine both in light mode and dark mode!
https://github.com/BirjuVachhani/adaptive_theme/assets/20423471/343f9521-824b-4062-ac5e-eea18ffb8562
https://github.com/BirjuVachhani/adaptive_theme/assets/20423471/c0c5ee01-8854-4c39-91cf-ae7941f72aee

AFAIK..this is not something that can be controller by Flutter. I'll try to test on an Android device!

thanks @BirjuVachhani very interesting. It seems even in the second video the status bar gets colored too which I never got working on the devices I have at hand. On all 3 I tested it it on it was like i posted above unfortunately.

@alexw92 Is this still an issue for you with latest Flutter verison?

I couldnt reproduce it ever since :D It seems to work fine now. I am not completely sure what the reason was in the first place since this was some time ago :D But it works for me too now!


psl_is_public_suffix2(): allow checking for suffixes not in the list

Add a PSL_TYPE_NO_STAR_RULE type to check for suffixes without the '*'
rule. This allows checking for suffixes that are not in the PSL.

Coverage remained the same at 0.0% when pulling 84614b178556fc44ef43956e2bff5407a6689b41 on csaavedra:no-star-rule-type into 179ca703b22d68cd11dfc22542e6c2a2247eca8b on rockdaboot:master.

Great ! LGTM :-)
Travis mourns, because i added a C99 feature recently and this library is pure C89 (to build on legacy platforms).  I just pushed a fix.
Adding tests would be needed to see if it's working as expected.
I suggest to extend test-is-public.c and test-is-public-builtin.c in the following way:

Add a second result (e.g. result2) to struct test_data.
Copy the loop and call psl_is_public_suffix2 with the new flag or'ed with the ANY flag and amend the if construct to test result2.
Optional: Add some more tests to the array as you like.

And then rebase to see Travis CI succeed.

The tests will fail for a handful of cases; but I think the expectations I wrote are correct. This might be related to the fact that there is a new codepath that might not have been tested before. But please, correct me if the expectations I added and are failing are not correct. Same if there are any other added expectations that are wrong.

Fix some smaller things in your last commit and added TLDs to the data. Before we skipped them because of the star rule, so the new tests failed on single TLDs from the PSL.
I will make up a new release in the next days.
Thanks again for your contribution !


Embedded entities

Hello
I have a question about couchdb-odm
Is it possible to make embedded entities like doctrine-orm?
/** @Entity */
class User
{
    /** @Embedded(class = "Address") */
    private $address;
}

/** @Embeddable */
class Address
{
    /** @Column(type = "string") */
    private $street;

    /** @Column(type = "string") */
    private $postalCode;

    /** @Column(type = "string") */
    private $city;

    /** @Column(type = "string") */
    private $country;
}

If yes could you provide an example?

See https://www.doctrine-project.org/projects/doctrine-mongodb-odm/en/1.2/reference/annotations-reference.html#embedone


Add Chi La Khong Cung Nhau by Tang Phuc x Truong Thao Nhi

Song Name
Chi La Khong Cung Nhau
Artist Name
Tang Phuc x Truong Thao Nhi
Source
Youtube
Youtube Link
UqKVL56IJB8
Direct File Link
No response
Song ID
529148
Start Offset [ms]
11500
Extra
Created from Auto Nong.
Created for level:
Downloaded: Yes

accept


RuntimeError happened during local deployment and startup.

I can deploy the app under bluemix and works well. But
When I try to run the app with this command locally:
python manage.py runserver localhost:8080
I got the following error:
raceback (most recent call last):
File "/Library/Python/2.7/site-packages/django/utils/autoreload.py", line 226, in wrapper
fn(*args, **kwargs)
File "/Library/Python/2.7/site-packages/django/core/management/commands/runserver.py", line 116, in inner_run
self.check(display_num_errors=True)
File "/Library/Python/2.7/site-packages/django/core/management/base.py", line 426, in check
include_deployment_checks=include_deployment_checks,
File "/Library/Python/2.7/site-packages/django/core/checks/registry.py", line 75, in run_checks
new_errors = check(app_configs=app_configs)
File "/Library/Python/2.7/site-packages/django/core/checks/urls.py", line 13, in check_url_config
return check_resolver(resolver)
File "/Library/Python/2.7/site-packages/django/core/checks/urls.py", line 23, in check_resolver
for pattern in resolver.url_patterns:
File "/Library/Python/2.7/site-packages/django/utils/functional.py", line 33, in get
res = instance.dict[self.name] = self.func(instance)
File "/Library/Python/2.7/site-packages/django/core/urlresolvers.py", line 417, in url_patterns
patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
File "/Library/Python/2.7/site-packages/django/utils/functional.py", line 33, in get
res = instance.dict[self.name] = self.func(instance)
File "/Library/Python/2.7/site-packages/django/core/urlresolvers.py", line 410, in urlconf_module
return import_module(self.urlconf_name)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/Users/dliu/workspace/openwhisk-tutorial/deploy_settings/urls.py", line 19, in
from whisk_tutorial import views
File "/Users/dliu/workspace/openwhisk-tutorial/whisk_tutorial/views.py", line 8, in
from . import utils
File "/Users/dliu/workspace/openwhisk-tutorial/whisk_tutorial/utils.py", line 4, in
from whisk_tutorial.models import TutorialUser
File "/Users/dliu/workspace/openwhisk-tutorial/whisk_tutorial/models.py", line 7, in
class TutorialUser(models.Model):
File "/Library/Python/2.7/site-packages/django/db/models/base.py", line 102, in new
"INSTALLED_APPS." % (module, name)
RuntimeError: Model class whisk_tutorial.models.TutorialUser doesn't declare an explicit app_label and isn't in an application in INSTALLED_APPS.
So what might be the potential issue, I'll search for some clue.

Please contact Zlati to see if he has any insight

Could you please tell us a bit more about what you're trying to do?
What are you trying to run? What is manage.py?
Thank you.

Yes, David is trying to make sure the tutorial app can run locally and had the above failure but am not sure he worked with Zlati to confirm his environment. Also, I believe David is on vacation for this week...

I could not replicate the issue. I had the app installed on two different machines apart from mine and it worked fine. David we can work offline if you are still experiencing this.
I recommend installing the exact module versions that are located in the requirements.txt and trying again. The command to do it is pip install -r requirements.txt. Also, Python 2 is the supported version with this app. I will update the documentation to reflect that.

@ZmG , thank you, I'll reinstall the as the command above.
@mrutkows @ioana-blue ,thank you.

@ZmG , thank you. Yes, the issue resolved, I reinstalled the packages, I got the following outputs, so I guess maybe my Django version is newer.
vincentdembp:openwhisk-tutorial dliu$ sudo pip install -r requirements.txt
Password:
The directory '/Users/dliu/Library/Caches/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/Users/dliu/Library/Caches/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Collecting Django<1.7,>=1.6 (from -r requirements.txt (line 1))
Downloading Django-1.6.11-py2.py3-none-any.whl (6.7MB)
100% |████████████████████████████████| 6.7MB 58kB/s
Collecting South<0.9,>=0.8 (from -r requirements.txt (line 2))
Downloading South-0.8.4-py2.py3-none-any.whl (135kB)
100% |████████████████████████████████| 139kB 4.2MB/s
Collecting Unipath==0.2.1 (from -r requirements.txt (line 3))
Downloading Unipath-0.2.1.tar.gz (149kB)
100% |████████████████████████████████| 151kB 90kB/s
Requirement already satisfied (use --upgrade to upgrade): django-markdown-deux in /Library/Python/2.7/site-packages (from -r requirements.txt (line 4))
Requirement already satisfied (use --upgrade to upgrade): markdown2 in /Library/Python/2.7/site-packages (from django-markdown-deux->-r requirements.txt (line 4))
Installing collected packages: Django, South, Unipath
Found existing installation: Django 1.9.7
Uninstalling Django-1.9.7:
Successfully uninstalled Django-1.9.7
Found existing installation: Unipath 1.1
Uninstalling Unipath-1.1:
Successfully uninstalled Unipath-1.1
Running setup.py install for Unipath ... done
Successfully installed Django-1.6.11 South-0.8.4 Unipath-0.2.1

Close it as resolved.


Bump expat-sys

expat-sys 2.1.0 is broken.

Seems fine.
@bors-servo r+

:pushpin: Commit 148e98c has been approved by Ms2ger

:hourglass: Testing commit 148e98c with merge fc2e81b...

:broken_heart: Test failed - travis

https://github.com/rust-lang/cargo/issues/2337

:pushpin: Commit 36e0d1d has been approved by Ms2ger

@bors-servo r=Ms2ger

:hourglass: Testing commit 36e0d1d with merge 8119dfb...

:sunny: Test successful - travis


auth folder

Thanks for updating the project. I installed the plugin, but I'm missing api/auth/saml in the api folder. Thoughts?

try againg osticket cache clear.

Hello and thank you so much for your work on the osticket saml connection. We where waiting for this functionality for a long time. As @bssher described in a earlier message, we experience the same. Authentication results in a callback/redirect from the IDP to page-url/api/auth/saml which does not exist. Apache2 log: "POST /api/auth/saml HTTP/1.1" 404. Help on this subject will be greatly appriciated. regards

Hi, the original poster closed this issue but I am having the same problem and I do not see a resolution on this.  Can this be looked into further?


[Enhancement]Support iceberg local disk metadata cache

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #18111
Problem Summary(Required) ：
iceberg metadata cached in memory cause high miss rate and  sql plan will take a lot of time, so I  change cached medium, save metadata(metadata.json,   manifest list,  manifest file) in local disk
Config usage:
iceberg_fileio_cache_disk_medium_dir:   which dir to save iceberg metadata
iceberg_fileio_cache_max_total_size:   total cache size, default 2GB
iceberg_fileio_cache_max_content_length:    max  single file size to cache
iceberg_disk_cache_expire_after_access_day:   expiration policy, default 7 day
Checklist:

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

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

Bugfix cherry-pick branch check:

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

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

[FE PR Coverage Check]
:disappointed: fail : 45 / 89 (50.56%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/io/IcebergCachingFileIO.java
31
70
44.29%
[113, 114, 115, 116, 117, 119, 120, 121, 122, 123, 124, 125, 128, 186, 188, 192, 193, 194, 195, 196, 209, 210, 243, 244, 277, 289, 292, 296, 297, 298, 299, 300, 301, 305, 306, 308, 319, 320, 321]

:large_blue_circle:
com/starrocks/connector/iceberg/io/IOUtil.java
10
15
66.67%
[95, 99, 100, 101, 102]

:large_blue_circle:
com/starrocks/common/Config.java
4
4
100.00%
[]

run starrocks_fe_unittest

[FE PR Coverage Check]
:disappointed: fail : 45 / 89 (50.56%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/io/IcebergCachingFileIO.java
31
70
44.29%
[113, 114, 115, 116, 117, 119, 120, 121, 122, 123, 124, 125, 128, 186, 188, 192, 193, 194, 195, 196, 209, 210, 243, 244, 277, 289, 292, 296, 297, 298, 299, 300, 301, 305, 306, 308, 319, 320, 321]

:large_blue_circle:
com/starrocks/connector/iceberg/io/IOUtil.java
10
15
66.67%
[95, 99, 100, 101, 102]

:large_blue_circle:
com/starrocks/common/Config.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test


Allow for object-storage-specific chunking?

Hello,
This module seems awesome. The only big problem I see for me is that I cannot use it to upload to my OpenStack Swift... Even if both Swift and Resumable.JS share all the principles to make it work it would require tweaks. Basically, it needs to use a different address for the GET request because, unless the storage is open to everyone, the request needs a signature forged by a server with correct access.
Here is the info for OpenStack https://docs.openstack.org/developer/swift/overview_large_objects.html
I am pretty certain it would also be useful if we could do the same thing for S3 and friends...
I would be happy to help define/develop this feature if you find it suitable for this exact module. Maybe it would be better to make it a different tool that would use resumable's API behind the scenes?
Thanks for your input.

There has been a few requests about this before -- and the consensus has been that even if it's an awesome feature, it's a feature that's out of scope for this library. It would be us building (and supporting!) interaction with services that we don't use ourselves.

I understand, thank you for the fast feedback.
2017-02-23 18:45 GMT+01:00 Steffen Tiedemann Christensen <
<EMAIL_ADDRESS>
Closed #364 https://github.com/23/resumable.js/issues/364.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/23/resumable.js/issues/364#event-974782722, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKrNZlrkEFDq1-XRFhXHTNpERJnlYhZmks5rfcXRgaJpZM4MKHEK
.

Did any developments arose since then within resumable.js?
@Salketer : Did you encountered a convenient alternative?

@drzraf I migrated to https://github.com/flowjs/flow.js which is an actively maintained alternative


Upgrade Kotlin and coroutines to stable

Can probably clean up the kotlinx maven repos while you're at it :)

Ignore me I'm imagining things


display rename displayKey in options doc

update

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Working

jangeador: I just saw this. Hope I did not miss anything too important :)

I was trying to pull request from yours to mine. I am still trying to figure out "the github" so it was a pr made by mistake.


Automatically assert that no temporary files are left after test suite runs

Current Behavior
Right now temporary file garbage collection has to be verified manually by checking that the tmp directory is clean after running make test.
Proposed Behavior
This works, but it would be great if automated tests could verify this.

Something something artifact cache something something. I don't think we SUPER care about this at this point


(H)DBSCAN

Some interest in a parallel / distributed version of this

https://github.com/scikit-learn-contrib/hdbscan
http://nbviewer.jupyter.org/github/scikit-learn-contrib/hdbscan/blob/master/notebooks/How HDBSCAN Works.ipynb
http://nbviewer.jupyter.org/github/scikit-learn-contrib/hdbscan/blob/master/notebooks/Comparing Clustering Algorithms.ipynb

Haven't looked to see if it's parallelizable.

I believe an approximate version of this is parallelizable. Some of the relevant techniques are described in this paper: http://www.sysml.cc/doc/105.pdf
This is certainly of interest to me -- I had been planning something similar before the paper came out, and had already coded up a python version of NNDescent. That version is not parallel, but making it so would be interesting to try. One catch that comes to mind is that that implementation is currently using Numba instead of Cython for performance. Did the maintainers here have any strong feelings on Numba dependencies?

Great!
I’m happy to accept numba as a dependency (I have a branch moving our Cython k-means to numba. Using just numba would simplify our build and release process. )

I'll see what I can put together then. I can't make any promises on getting it done soon, but I will definitely put it in the queue of things to work on.

Skimming the paper at http://www.sysml.cc/doc/105.pdf
https://github.com/lmcinnes/pynndescent would be great to make dask-friendly and include in dask-ml. If possible I'd like to re-use it for a top-level KNeighborsClassfier/Regressor API, and inside SpectralClustering (as an alternative to K-Means).

I chatted with @lmcinnes about this.  It seems like this is slightly blocked on improved slicing of dask arrays with out-of-order indices.  For example the following was inefficient
In [1]: import dask.array as da

In [2]: x = da.random.random((1000000, 100), chunks=(10000, 100))

In [3]: import numpy as np

In [4]: index = np.arange(len(x))

In [5]: np.random.shuffle(index)

In [6]: %time y = x[index]
/home/mrocklin/Software/anaconda/bin/ipython:1: PerformanceWarning: Slicing with an out-of-order index is generating 9900 times more chunks
  #!/home/mrocklin/Software/anaconda/bin/python
CPU times: user 4.48 s, sys: 188 ms, total: 4.67 s
Wall time: 4.68 s

In [7]: y.numblocks
Out[7]: (989999, 1)

So would solving issue ( https://github.com/dask/dask/issues/3409 ) address this?

Possibly.  It might also be that @lmcinnes would want to do more targetted
indexing than an entirely random shuffle.
On Sun, Jul 15, 2018 at 1:18 PM, jakirkham<EMAIL_ADDRESS>wrote:

So would solving issue ( dask/dask#3409
https://github.com/dask/dask/issues/3409 ) address this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/158#issuecomment-405108411, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszNY72sUiQpbPD5plw_7y33lEn5eCks5uG4dfgaJpZM4TOvYA
.

Although it may also be that we can improve things today if we index directly many times, but make sure that the index is sorted.
In [1]: import dask.array as da

In [2]: x = da.random.random((1000000, 100), chunks=(10000, 100))

In [3]: import numpy as np

In [4]: index = np.random.randint(0, len(x), size=10000)

In [5]: %time x[index].numblocks
/home/mrocklin/Software/anaconda/bin/ipython:1: PerformanceWarning: Slicing with an out-of-order index is generating 100 times more chunks
  #!/home/mrocklin/Software/anaconda/bin/python
CPU times: user 73.6 ms, sys: 3.8 ms, total: 77.4 ms
Wall time: 76.8 ms
Out[5]: (9907, 1)

In [6]: index.sort()

In [7]: %time x[index].numblocks
CPU times: user 4.74 ms, sys: 47 µs, total: 4.78 ms
Wall time: 4.15 ms
Out[7]: (100, 1)

I've laid out a plan in https://github.com/dask/dask/issues/3409#issuecomment-405254656 to improve things a bit.  That should be accessible for others to contribute if they're interested.


What about CHANGELOG?

I use aerys as backend driver in one my project, and done upgrade from ^0.4.3 to ^0.7.1.
Without changelog it was some kind of challenge)
Keep changelog it is good practice.
To my mind when project has the changelog it is convenient for the user.
PS: Now I prepare small patch to aerys and can started it for unreleased from last version.

Currently we usually have the changelog directly in the releases section, as CHANGELOG.md tends to be forgotten to be updated. Unfortunately, @bwoebi doesn't tag releases with a summary. :stuck_out_tongue_winking_eye:
Amp and Artax already have a CHANGELOG.md, so we can just add it here as well.


(feat) safer download

refactored the download of SECP256K1 source

@ofek This is ready, just a proposal to remove the extractall() that generates a safety warning

I'm okay with the current way, I understand there is a warning but it is basically "safe"


[Do not merge] Fix Xcode 8 beta 4 errors and warnings

For after Travis gets updated.

Nice! Thanks for this! I guess we'll have to keep an eye out on when they update Travis. Feel free to let me know on here if you see before me.

I contacted Travis and they said they are working on adding beta 4. They will email me directly when it's updated.

Beta 4 is available now. Can you resolve these conflicts and push, so I can merge these changes @robtimp?

Current coverage is 84.99% (diff: 88.23%)

No coverage report found for master at b54a998.

Powered by Codecov. Last update b54a998...a03a3dd


HeartBeat request is not properly url encoded

Hi all,
Describe the bug
I am migrating an application that uses Jersey 1 to RestTemplate.
During migration I have encountered heart beat requests returning 400 Bad request status code and upon further debugging found out, that the sent request is not complete.
The application uses instanceId in format of service#version and that is the problem.
Using spring-cloud-netflix-eureka-client 3.1.2 I tested this scenario with both Jersey 1 and 2 passing.
Sample
I forked the repo and modified testcase AbstractEurekaHttpClientTest adding some characters that have to be escaped into instanceId.
testSendHeartBeat now failed for RestTemplate with 400 status code (as expected), however WebClient test still passed with 200 although logs showed 400 returned as well (no idea why, I did not proceed down the rabbit hole).
I replaced String concatenation in both WebClient and RestTemplate with UriBuilder and UriComponentsBuilder where deemed appropriate.
Feel free to check it out: https://github.com/NewAgeCZ/spring-cloud-netflix/commits/fix/urlencoding
Best regards!

PRs welcome

Hi @spencergibb. PR has been available for some time now https://github.com/spring-cloud/spring-cloud-netflix/pull/4163
I am happy to jump in and assist if @NewAgeCZ does not have time at present.

Hi @ZIRAKrezovic,
I have been waiting for the PR review to come... almost forgot about this fix. If the code changes are approved, I can handle rebasing onto lastest upstream.
Lets wait for the review and I will notice you afterwards if my time schedule would not allow me to react in reasonable time.


TH2-4916

transport protocol support

these changes were merged to TS-1127 is PR#29 ( https://github.com/th2-net/th2-codec-csv/pull/29 )


is there a "continue on error" option?

Thank you for this issue.
Currently, npm-run-all doesn't have the option to continue on an error.
I will add it.

That would be great. Thanks!


"STORES" environment variable doesnt work at start

Now goes through every store at the beginning, then after initial spam it will go back to the configured variables.

Is this really fixed? I'm still experiencing this.

Not sure, seems to work fine for me. I would make sure you have the latest code.

This is what it looks like for me. I'm on the latest source and have run npm install after a git pull. Is there a cache I should try clearing?

Interesting. OK, what does your .env look like?

** All configuration variables are optional **
Read https://github.com/jef/nvidia-snatcher#customization for help on customizing this file
#############################################################################################
BROWSER_TRUSTED="false"
COUNTRY="usa"
DESKTOP_NOTIFICATIONS="false"
EMAIL_PASSWORD="redacted"
EMAIL_TO="redacted"
EMAIL_USERNAME="redacted"
HEADLESS="true"
IN_STOCK_WAIT_TIME="0"
LOG_LEVEL="info"
LOW_BANDWIDTH="false"
MAX_PRICE_SERIES_3070="600"
MAX_PRICE_SERIES_3080="900"
MAX_PRICE_SERIES_3090="1600"
MICROCENTER_LOCATION="web"
NVIDIA_ADD_TO_CART_ATTEMPTS="10"
NVIDIA_SESSION_TTL="60000"
OPEN_BROWSER="true"
PAGE_BACKOFF_MIN="10000"
PAGE_BACKOFF_MAX="3600000"
PAGE_SLEEP_MIN="5000"
PAGE_SLEEP_MAX="10000"
PAGE_TIMEOUT="30000"
PHONE_CARRIER="tmobile"
PHONE_NUMBER="5035503453"
PLAY_SOUND="alert.mp3"
PUSHOVER_TOKEN="redacted"
PUSHOVER_USER="redacted"
PUSHOVER_PRIORITY="1"
SCREENSHOT="true"
SHOW_ONLY_BRANDS="asus, nvidia"
SHOW_ONLY_MODELS="rog strix, rog strix oc, strix, tuf, tuf oc, founders edition"
SHOW_ONLY_SERIES="3080"
STORES="amazon, bestbuy, newegg"
USER_AGENT="Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36"

I did a fresh clone and pull, same results. Most likely something to do with my config or the base code.

Nothing wrong, i closed this already but it was reopen for some reason.

I think it's with the code, because I usually test with minimal setups. I just used the STORE=amazon and starting using all the stores.
Probably has something to do with this: https://github.com/jef/nvidia-snatcher/commit/503d76f3c7d01368fc978af4ab02924d3d036ac0, but still need to debug.

Wow... Oops. This is not a problem.

If you notice, you get the other stores in the beginning, but the stores in search are the ones you've specified.
I don't believe this is an issue.

I wouldn't say it's a bug in that it's a blocking issue. But I would say it's probably unintended behavior. It didn't used to do this, was a recent change.

It also does this where it will randomly interject 1 or 2 searches at stores that I've not configured.

Doesnt effect functionality at all. I had closed this but it was reopened for some reason.
https://i.imgur.com/X4Vx5Xi.png

Fixing the logic to not evaluate any stores that are not in the list.


Propose to merge release-1.12 into master

sig-docs-zh team has worked on release-1.12 for quite a long time, we should seriously think about merge the branch to master and work on the release-1.13 (release-1.14 was out recently).
This issue is created to address the effort of doing the merge and serve as place to discuss it.
/cc @zacharysarah @kubernetes/sig-docs-zh-owners
/cc @xichengliudui @SataQiu @tengqm

Looking at the list of open Chinese language PRs, I see PRs against release-1.12 (#13234, #13500) and master (#13298, #13251). Have you been monitoring PRs to ensure that they're merged to the same branch?

Agree with, about open Chinese PRs.

Merge the currently mergable PRs
Turn off PRs that have not responded for a long time

I agree with @chenrui333 .
But as @zacharysarah said, there may be some problems in this process, such as conflicts.
Is there a good way to deal with these problems?

Certainly, we definitely need to address the opening PRs (especially the ones with base branch as release-1.12). I do monitor the open PRs. :)
This issue can be a place that we can track down the process as it goes.
Regarding the merge conflicts, we can fix as it goes (think about the sig-docs release, it would be like that). But I do think we should consider the contributors' activities, in the sense of Github's approach of counting the contributions would be against the master branch. My preference would be merge commit instead of prow's squash-and-merge way.
Let me know if that makes sense.

+1

Kicking off a slack discussion on this: https://kubernetes.slack.com/archives/CA1MMM1HU/p1555973690018400

checkin

This work has been done, so close this.
/close


Address issues from the code review of plugin-support

pin stevedore
remove deprecated list
clean up detritus after removing extension manager forcing
available_apps should only return apps
fix fragile test for help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 91.731%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
29684

Relevant Lines:
32360

💛  - Coveralls


unable to consume topic, dns error

Hi! When running kt topic the command connects successfully and prints out the topics on my kafka cluster, but when doing kt consume -topic my-topic I get the following error:
sarama client configuration &sarama.Config{Net:struct { MaxOpenRequests int; DialTimeout time.Duration; ReadTimeout time.Duration; WriteTimeout time.Duration; TLS struct { Enable bool; Config *tls.Config }; SASL struct { Enable bool; Handshake bool; User string; Password string }; KeepAlive time.Duration }{MaxOpenRequests:5, DialTimeout:30000000000, ReadTimeout:30000000000, WriteTimeout:30000000000, TLS:struct { Enable bool; Config *tls.Config }{Enable:false, Config:(*tls.Config)(nil)}, SASL:struct { Enable bool; Handshake bool; User string; Password string }{Enable:false, Handshake:true, User:"", Password:""}, KeepAlive:0}, Metadata:struct { Retry struct { Max int; Backoff time.Duration }; RefreshFrequency time.Duration }{Retry:struct { Max int; Backoff time.Duration }{Max:3, Backoff:250000000}, RefreshFrequency:600000000000}, Producer:struct { MaxMessageBytes int; RequiredAcks sarama.RequiredAcks; Timeout time.Duration; Compression sarama.CompressionCodec; Partitioner sarama.PartitionerConstructor; Return struct { Successes bool; Errors bool }; Flush struct { Bytes int; Messages int; Frequency time.Duration; MaxMessages int }; Retry struct { Max int; Backoff time.Duration } }{MaxMessageBytes:1000000, RequiredAcks:1, Timeout:10000000000, Compression:0, Partitioner:(sarama.PartitionerConstructor)(0x1256d60), Return:struct { Successes bool; Errors bool }{Successes:false, Errors:true}, Flush:struct { Bytes int; Messages int; Frequency time.Duration; MaxMessages int }{Bytes:0, Messages:0, Frequency:0, MaxMessages:0}, Retry:struct { Max int; Backoff time.Duration }{Max:3, Backoff:100000000}}, Consumer:struct { Retry struct { Backoff time.Duration }; Fetch struct { Min int32; Default int32; Max int32 }; MaxWaitTime time.Duration; MaxProcessingTime time.Duration; Return struct { Errors bool }; Offsets struct { CommitInterval time.Duration; Initial int64; Retention time.Duration } }{Retry:struct { Backoff time.Duration }{Backoff:2000000000}, Fetch:struct { Min int32; Default int32; Max int32 }{Min:1, Default:32768, Max:0}, MaxWaitTime:250000000, MaxProcessingTime:100000000, Return:struct { Errors bool }{Errors:false}, Offsets:struct { CommitInterval time.Duration; Initial int64; Retention time.Duration }{CommitInterval:1000000000, Initial:-1, Retention:0}}, ClientID:"kt-consume-emeliasmith", ChannelBufferSize:256, Version:sarama.KafkaVersion{version:[4]uint{0x0, 0xa, 0x0, 0x0}}, MetricRegistry:(*metrics.StandardRegistry)(0xc42004c890)}
2019/08/08 13:11:17 Initializing new client
2019/08/08 13:11:17 client/metadata fetching metadata for all topics from broker localhost:9092
2019/08/08 13:11:17 Connected to broker at localhost:9092 (unregistered)
2019/08/08 13:11:17 client/brokers registered new broker #1 at kafka.company.local:9092
2019/08/08 13:11:17 Successfully initialized new client
2019/08/08 13:11:17 Failed to connect to broker kafka.company.local:9092: dial tcp: lookup kafka.quandoo.local on <IP_ADDRESS>:53: no such host
2019/08/08 13:11:17 client/metadata fetching metadata for [my-topic] from broker localhost:9092
2019/08/08 13:11:17 Failed to connect to broker kafka.company.local:9092: dial tcp: lookup kafka.company.local on <IP_ADDRESS>:53: no such host
Failed to read start offset for partition 0 err=dial tcp: lookup kafka.company.local on <IP_ADDRESS>:53: no such host

I've no idea what's causing this as if I can list topics, surely I can consume them? My cluster has no authentication requirements.

hi @ThisIsMissEm -- sorry for the delay, i've only recently resumed work on my github projects. reading through the logs it looks like you can connect to the broker that's on your localhost, but not a "remote" one at 'kafka.company.local' - correct? if you're trying to connect to a bigger cluster it might be worth checking with your kafka admin? the issue is a networking one though from what i can tell - e.g. maybe the advertised hostname or ip is incorrect? i hope you found a solution by now though! i'm closing this issue as I don't think it's a kt issue, but please feel free to re-open the issue if i can be of help!

Sorry to revive an old issue, but I'm having a similar problem and I think I've found the root cause. In -verbose mode I see an attempt to look up the hostname on my home router (<IP_ADDRESS>) but not on the one(s) provided by the VPN (<IP_ADDRESS>, <IP_ADDRESS>); if I use the IP address of the broker(s) then everything works fine.
The net package docs say that the pure Go resolver "sends DNS requests directly to the servers listed in /etc/resolv.conf", i.e. if that doesn't include the VPN servers then it won't be able to look them up. There's an alternative implementation called cgo but unfortunately the netgo build tag prevents me from overriding it — "The decision can also be forced while building the Go source tree by setting the netgo or netcgo build tag" — and I've confirmed that myself:
$ GODEBUG=netdns=cgo+2 kt consume --brokers kafka.example.tech:9092 -topic foo -offsets newest -verbose
...
go package net: built with netgo build tag; using Go's DNS resolver
go package net: hostLookupOrder(kafka.example.tech) = files,dns

Unless netgo can be removed somehow (I don't know the intricacies of Go builds or why it was chosen) I think these are the only workarounds:

add the broker hostnames to /etc/hosts
add the VPN-provided DNS addresses to /etc/resolv.conf (as at least Cisco's VPN software doesn't do this itself)
use the IP addresses of the broker hosts.


Skip QC if label file is empty

Check if the label file is empty (i.e., no lesion was drawn in MS patient); if so, skip QC.
Resolves: https://github.com/spinalcordtoolbox/manual-correction/issues/60

By doing that we accept the creation of empty masks for all the tasks. The user needs to be aware of that maybe.

By doing that we accept the creation of empty masks for all the tasks. The user needs to be aware of that maybe.

Good point! Do you think it would be better to allow the empty mask only for the FILES_LESION key?

also, don’t we want to check MS lesion mask even if it is empty? otherwise we will miss false negatives.

also, don’t we want to check MS lesion mask even if it is empty? otherwise we will miss false negatives.

This is an excellent point! I initially thought that sct_qc does not allow to create QC if the mask is empty. But I was mistaken! The empty mask is not allowed for the sct_qc -p sct_label_utils flag. But, with the sct_qc -p sct_deepseg_lesion flag, empty QC can be created; investigation here.

The QC is now skipped only for functions which do not support empty labels (sct_label_utils, sct_label_vertebrae, sct_detect_pmj, commit). In other words,  an empty label mask is supported for sct_deepseg_sc, sct_deepseg_gm, and sct_deepseg_lesion.

Another point is, do we want to keep empty masks for every tasks ?

Another point is, do we want to keep empty masks for every tasks ?

Good point! I would say that it is the responsibility of the user to correct/create a label when using the manual_correction.py script. If the user decides to keep the label empty, for example, in case of no MS lesion or no SC compression, it is absolutely fine. But it is true that, for instance, for disc labeling, the empty mask does not make much sense. However, I would not complicate this PR since this PR is about preventing the QC from failing.

I understand but this falling is also currently the only warning/error that we have regarding empty masks

I understand but this falling is also currently the only warning/error that we have regarding empty masks

I added the following warning:
https://github.com/spinalcordtoolbox/manual-correction/blob/8adb17220f2e2a76f3efab857f3639355e3957c5/manual_correction.py#L595


Update website - 7/31

Audit and update for content, links, 2016 graphics, turnoff links

Let me know how I can help with this!

Well underway. looking for feedback, please!

I was looking at it on mobile today and noticed that it's hard to navigate
to pages about the event. The homepage is a blog post and is confusing to
me. visitors will need to see details about the event front and center.
Jason
On Mon, Jul 25, 2016 at 6:28 PM, Chris Mathews<EMAIL_ADDRESS>wrote:

Well underway. looking for feedback, please!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/CityCampNC/issues/119#issuecomment-235105727,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADgoV-APow_Ao04aF83pNFAQXA4Jb5hnks5qZTiDgaJpZM4JPNWD
.

thanks. That's the kind of feedback I need. So we lose the blog posts and do a one pager layout? about - venue - taste - unconference - sprint/hackathon

I just checked it out with Chrome on my PC.  The layout looks good.  The
content about the NC Open Pass early bird on the welcome page needs to be
updated.  I think it would be good to mention that Data Jam will be with
City Camp.  We've taken a stab at that at www.ncdatapalooza.com.  Just let
Lizzy and I know if you've got ideas about how we incorporate that into the
Datapalooza website.
Zach
On Tue, Jul 26, 2016 at 8:11 AM, Chris Mathews<EMAIL_ADDRESS>wrote:

thanks. That's the kind of feedback I need. So we lose the blog posts and
do a one pager layout? about - venue - taste - unconference -
sprint/hackathon
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/CityCampNC/issues/119#issuecomment-235249165,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQ6IxnUkc7zw_dRm79m_yCjki76D8OYJks5qZflmgaJpZM4JPNWD
.

--
Zach Ambrose
<EMAIL_ADDRESS>919-438-2752 (office)
919-616-9969 (cell)
www.AmbroseStrategy.com
https://www.linkedin.com/in/zambrose

What do we need to do to get this ready to go before the weekend?

Jason needs to change the ip address associated with wwww.citycampnc.org


Minor refactoring and Editor bug fix

Codecov Report

Merging #81 (a76f296) into master (8f3858b) will decrease coverage by 0.03%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   77.73%   77.69%   -0.04%
==========================================
  Files          41       41
  Lines        1509     1511       +2
==========================================
+ Hits         1173     1174       +1
- Misses        336      337       +1

Impacted Files
Coverage Δ

...time/Containers/ConfigurableDependencyContainer.cs
81.25% <0.00%> (-0.86%)
:arrow_down:

...Framework/Runtime/Dependencies/ObjectDependency.cs
100.00% <100.00%> (ø)


Move to Java 7

Hi!
I was wondering if we could move to Java 7 for the next release of Jest? It might be handy to avoid this sort of issues: https://github.com/searchbox-io/Jest/issues/412 : we could use try with resources.
I'll propose a PR if you think it's a good idea.
--
Guillaume

@gsmet I think, due to a long break at our release cycle (not happy with it), we should keep Java 6 and make a release with some of the issues fixed to improve stability
But as you said moving to 7 is a must (actually even 8 is getting old) therefore after next release we can quickly create another release with only 7 move,
Does it make sense ?

@ferhatsb yes, makes sense!

Done, from 2.4 min version is java 7.


READ.me Updated

Please read how to contribute first!
Description
Submission description here
Checklist

[x] I've checked the issue list BEFORE deciding what to submit
[x] I've read the contribution guidelines
[x] I've edited the README.md link to my code

Don't forget to visit our website!
Feel free to add this badge after your submission is accepted!

@bagladivyang03 I think there is no change in the file.

Problem is solved na
U have merged na?

@bagladivyang03 Earlier this PR was having only empty single line commit and rest things are already added so I think this has to be closed now.


Painting black on white with a tolerance of zero freezes everything

I can't explain why this happens and I don't know how to debug this, as my whole browser just freezes. Super strange, it really only happens with tolerance 0 and white on black. While this script looks unmaintained, I thought it'd be nice to just document it at least :D

I've been struggling with this issue for awhile.  I've tried numerous times to fix it and always end up either failing or making other things worse.  I know this lib looks unmaintained but I've gone through the algo so many times in the past 2 years and I admit defeat.  Looking for someone to help out here, so if you'd like to take a shot at it I'd be much appreciative!


numpy dependency missing?

magenta/models/sketch_rnn/utils.py has import numpy as np, but magenta/tools/pip/setup.py doesn't list it as a dependency.

Yeah, we should probably add that. It hasn't been a problem because we depend on tensorflow, which depends on numpy, but we should list our dependency explicitly.


add correlationBlogpost

add Blogpost: Pearson correlation coefficient, Correlation Matrices and Heatmap.

@kMutagene  is everything looking good?


Mouseover on links in citation pop-up shows wrong cursor

When mousing over Citation or Bibtex in this Citation pop-up for publications, the mouse cursor should change to a pointer finger (currently changes to a flashing edit cursor)

This happens at least on the landing page (haven't tested it on the pubs page)

@jonfroehlich

is this what you envision?

Yes. Thank you. And awesome post with an animated gif and everything. Please keep up this high quality work!! 👍🏽
Sent from my iPhone

On Jul 11, 2018, at 9:12 AM, johnsonkuang<EMAIL_ADDRESS>wrote:
@jonfroehlich
is this what you envision?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Pull request submitted and merged, closing issue


InvalidAuthenticationToken: CompactToken parsing failed with error code: 80049217

Love this tutorial and could reproduce it fine up to and including the "sign out" bit. After doing the code changes for "refreshing the access token" and "using the mail API", I can still sign in and out, but the app fails when I try to display the inbox. When I click on the Inbox tab, I get the following error in the debug section: "{
"code": "InvalidAuthenticationToken",
"message": "CompactToken parsing failed with error code: 80049217",
"innerError": {
"request-id": "2644e612-c299-4204-bf6c-3861ee3f741e",
"date": "2019-01-31T15:44:57"
}
}"
Also I get the following error on my server console: "Error: C:\Dev\minutes\mstut\node-tutorial\views\error.hbs: Can't set headers after they are sent.
at validateHeader (_http_outgoing.js:491:11)
at ServerResponse.setHeader (_http_outgoing.js:498:3)
at ServerResponse.header (C:\Dev\minutes\mstut\node-tutorial\node_modules\express\lib\response.js:767:10)
at ServerResponse.send (C:\Dev\minutes\mstut\node-tutorial\node_modules\express\lib\response.js:170:12)
at done (C:\Dev\minutes\mstut\node-tutorial\node_modules\express\lib\response.js:1004:10)
at C:\Dev\minutes\mstut\node-tutorial\node_modules\hbs\lib\hbs.js:93:9
at Object.done (C:\Dev\minutes\mstut\node-tutorial\node_modules\hbs\lib\async.js:74:20)
at C:\Dev\minutes\mstut\node-tutorial\node_modules\hbs\lib\hbs.js:88:18
at C:\Dev\minutes\mstut\node-tutorial\node_modules\hbs\lib\hbs.js:69:11
at Object.done (C:\Dev\minutes\mstut\node-tutorial\node_modules\hbs\lib\async.js:74:20)"
I am at a loss here. Could I ask for your help?

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

ID: 53b47c58-f96e-dca0-6ae6-0e22bda41a07
Version Independent ID: 760e409d-5f01-7bf2-0c8e-a05e75afa711
Content: How to use Outlook REST APIs in a Node.js app - Outlook Developer
Content Source: docs/rest/node-tutorial.md
Product: outlook
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

@JensSteckhan I'm glad you like it, but sorry you're having trouble :). That error is returned from the Microsoft Graph API. Whenever you have a token problem like that, it's always helpful to parse the token and see if there's something wrong with it (as the error indicates). Can you try parsing it at https://jwt.ms?
Also, what type of account are you signing in with, an Office 365 account, or a personal Outlook.com account?

Thank you for getting back to me, I was getting rather desperate over here! I am using my company's Office 365 account. When I enter the token string at jwt.ms, nothing happens. I guess that is a sign that I am doing something wrong?

Hmm yes, the string should parse out to a JSON web token. Let's try this: can you go to Graph Explorer and try signing in with that same account, then see if you can make a simple API call?

Hi Jason, thank you for helping me here. Yes, I can sign in to Graph Explorer with my company account, make API calls and get meaningful results :-)
Just to avoid a possible misunderstandings: in auth.js, the function getTokenFromCode logs the token to the console. It is of type 'Bearer' and has a 676 character ASCII string as access_token. I took that string and pasted into the upper text field of https://jwt.ms .
Nothing happened.

I looked into the code in getTokenFromCode in auth.js one more time. It is supposed to take an auth code, post it via simple-oauth2 to the auth server and retrieve an access token, right?
This is the code:
let result = await oauth2.authorizationCode.getToken({
code: auth_code,
redirect_uri: process.env.REDIRECT_URI,
scope: process.env.APP_SCOPES
});
const token = oauth2.accessToken.create(result);
const access_token = token.token.access_token;
I had initialized oauth2 with the following values:
tokenHost: 'https://login.microsoftonline.com'
authorizePath: "<my company's tenant>/oauth2/authorize"
tokenPath: "<my company's tenant>/oauth2/token"
I had also maintained the client ID and secret in the .env file.
However, when I run the code, log access_token to the console and paste it to jwt.ms, the result is invalid. Also, I looked via Fiddler at the calls. I can't see any post to the auth server. So somehow, the magic "get me an access token for my auth code" seems to fail and I have no idea why.

Ah, I think I see the problem. Based on your paths, you're using Azure's V1 auth endpoints, but this sample is expecting to use the v2 endpoints. The v1 endpoints don't understand the scope query parameter, and instead expect a resource URL. Try changing them to:
authorizePath: "<my company's tenant>/oauth2/v2.0/authorize"
tokenPath: "<my company's tenant>/oauth2/v2.0/token"

This is assuming that you registered in the Application Registration Portal (per https://docs.microsoft.com/outlook/rest/node-tutorial#register-the-app)

You made my day!!
Thank you, that did the trick.

Glad that fixed it. Happy coding!

@jasonjoh For me the token is correct, it parses correct in jwt.ms. It also gives that it has Calendar Read/Write permissions. I took this token from Graph API developer site "https://developer.microsoft.com/en-us/graph/graph-explorer". However, when i pass it through code, it gives me the below error:
{
"error": {
"code": "InvalidAuthenticationToken",
"message": "CompactToken parsing failed with error code: 80049217",
"innerError": {
"request-id": "e45d2a93-e6fe-4c33-adc9-64cb4e3d3bc4",
"date": "2019-08-30T10:14:26"
}
}
}
Not sure what am i missing.

I just removed the bearer from authorization headers and put only access token. It worked...

Thanks for that workaround, @RoyRK07. This seems to be a bug with the Graph API. I hope someone from MS can log the issue.


Fix/discriminator classifier nn dimensions

PR Checklist

[X] Referenced issue is linked (closes #468)

Description of changes
When trying to use the DiscriminatorClassifierSpace, a bunch of errors are thrown. It appears that initially, a LabelEncoder was employed, encoding the labels to train the neural network as integers, but subsequently, it was partially replaced by one-hot encoding. This PR modifies the usage to exclusively use one-hot encoding and fixes several downstream errors that occurred.
Technical details
Precisely, the following things are changed:

Fix loss calculation: CrossEntropyLoss only works on one-hot encoded labels with some preprocessing, which I added.
get_embeddings maintains the .obs annotations of the AnnData object
Fixed BatchNorm1D dimensions: Previously, the number of neurons was provided as parameter to the batch norm layer, which throws an error since the parameter should specify the number of channels which is 1.
Silence several warnings: Set shuffling for validation and test dataloaders to False; provide the batch_size when logging during training or testing.

FAILED tests/tools/_perturbation_space/test_discriminator_classifier.py::test_discriminator_classifier - RuntimeError: mat1 and mat2 must have the same dtype, but got Double and Float

is still open


refactor: refactor submit_starknet_transaction our from KakarotEthApi to KakarotStarknetApi

submit_starknet_transaction method was part of the KakarotEthApi trait, but it is better suited in KakarotStarknetApi trait, this PR does this refactor.
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing
[ ] Other (please describe):

What is the current behavior?
submit_starknet_transaction method is part of the KakarotEthApi
Resolves: #233
What is the new behavior?

submit_starknet_transaction method is part of the KakarotStarknetApi

Does this introduce a breaking change?

[ ] Yes
[x] No

@bajpai244 if we are closing 233 with this PR, can you make another issue for updating the function signatures from KakarotEthApi? The signature should only contain Reth types.

@bajpai244 if we are closing 233 with this PR, can you make another issue for updating the function signatures from KakarotEthApi? The signature should only contain Reth types.

@greged93 created here.
I have commented you can assign it to me.

@greged93 I guess it makes more sense to do #237 here as well. So, I am pushing the required changes here.


Q20+ data（FASTQs and BAMs） need

Hello! I want to obtain Q20+ data (FASTQs and BAMs format) for experimental purposes. However, I found on the main page that almost all of them need AWS to download, is there any other way or website? Since I am a student without a VISA card, I cannot register an AWS account.
Hope to get your help

Hi,
you can download the Q20+ data with the aws command line interface, e.g.
aws s3 sync --no-sign-request s3://ont-open-data/giab_lsk114_2022.12/analysis/wf-human-var-output/hg004_sup_v4/ .
The data link for Q20+ is available on this page.


Error converting html to string.

 Issue by tspier
Sun Jul 18 23:13:34 2021
Originally opened as https://github.com/codelucas/newspaper/issues/904

I'm not sure if it's an issue with the HTML of the website, if there's an issue parsing Tajiki, or something else, but I tried scraping http://www.jumhuriyat.tj/index.php?art_id=44635 on the Heroku demo page and received the following notice: Error converting html to string.

Works in 0.9.3


hl2 rtx appears to do nothing other than change the render engine to vulkan.

:53.336] info:  Found default config for: Source Engine
[18:35:53.336] info:  Trying to open config file: c:\program files (x86)\steam\steamapps\common\half-life 2\bin.trex\bridge.conf
[18:35:53.344] info:  Found user config file: c:\program files (x86)\steam\steamapps\common\half-life 2\bin.trex\bridge.conf
[18:35:53.345] info:  Effective configuration:
[18:35:53.345] info:    presentSemaphoreMaxFrames  = 1
[18:35:53.345] info:  ==================
[18:35:53.346] info:  NVIDIA RTX Remix Bridge Client
[18:35:53.346] info:  ==================
[18:35:53.346] info:  Version: remix-0.2.0
[18:35:53.347] info:  Loaded d3d9.dll from C:\Program Files (x86)\Steam\steamapps\common\Half-Life 2\hl2.exe
[18:35:53.348] warn:  Injected DirectInput8Create proc detected!
[18:35:53.358] info:  DirectInput8 hook attached.
[18:35:53.360] warn:  Injected DirectInputCreate proc detected!
[18:35:53.367] info:  DirectInput hook attached.
[18:35:53.367] info:  Initializing new shared memory object.
[18:35:53.392] info:  Initializing new shared memory object.
[18:35:53.404] info:  Launching server with GUID f40cf259-51fc-4356-b0dc-e62f1953db8d
[18:35:53.439] info:  Process set as DPI aware
[18:35:53.440] info:  Sending SYN command, waiting for ACK from server...
[18:35:53.978] info:  Ack received! Handshake completed! Telling server to continue waiting for commands...
[18:35:54.525] info:  Creating a thread-safe D3D9 device.
[18:35:58.646] info:  Message channel UWM_REMIX_BRIDGE_REGISTER_THREADPROC_MSG handshake complete.
[18:35:58.646] info:  Remix UI deactivated.
[18:35:58.646] info:  Remix UI deactivated.
[18:36:02.857] info:  Remix UI activated.
[18:36:04.667] info:  Client window became inactive, disabling timeouts for bridge client...
[18:36:05.838] info:  Client window became active, reenabling timeouts for bridge client...
[18:36:10.125] info:  Client window became inactive, disabling timeouts for bridge client...
[18:36:11.351] info:  Client window became active, reenabling timeouts for bridge client...
[18:36:16.572] info:  Remix UI activated.
[18:36:17.818] info:  Client window became inactive, disabling timeouts for bridge client...
[18:36:18.838] info:  Client window became active, reenabling timeouts for bridge client...
[18:37:14.714] info:  Client window became inactive, disabling timeouts for bridge client...
[18:37:15.735] info:  Client window became active, reenabling timeouts for bridge client...
[18:37:18.521] info:  Client window became inactive, disabling timeouts for bridge client...
[18:37:19.671] info:  Client window became active, reenabling timeouts for bridge client...
[18:39:46.076] info:  Client window became inactive, disabling timeouts for bridge client...
[18:39:54.128] info:  Client window became active, reenabling timeouts for bridge client...
[18:40:03.077] info:  Remix UI deactivated.
[18:40:06.315] info:  Remix UI activated.
[18:40:08.051] info:  Remix UI deactivated.
[18:40:12.804] info:  Client window became inactive, disabling timeouts for bridge client...
[18:40:14.621] info:  Client window became active, reenabling timeouts for bridge client...
[18:40:23.458] info:  Remix UI activated.
[18:40:26.897] info:  Client window became inactive, disabling timeouts for bridge client...
[18:40:28.568] info:  Client window became active, reenabling timeouts for bridge client...
[18:40:34.395] info:  Client window became inactive, disabling timeouts for bridge client...
[18:40:42.399] info:  Client window became active, reenabling timeouts for bridge client...
[18:40:52.155] info:  Remix UI deactivated.
[18:40:54.290] info:  Remix UI activated.
[18:40:55.917] info:  Remix UI activated.
[18:40:57.145] info:  Client window became inactive, disabling timeouts for bridge client...
[18:41:40.511] info:  Client window became active, reenabling timeouts for bridge client...
[18:41:48.956] info:  Remix UI activated.
[18:41:50.337] info:  Client window became inactive, disabling timeouts for bridge client...
[18:41:51.393] info:  Client window became active, reenabling timeouts for bridge client...
[18:42:01.592] info:  Client window became inactive, disabling timeouts for bridge client...
[18:44:51.592] info:  Client window became active, reenabling timeouts for bridge client...
[18:44:53.780] info:  Remix UI activated.
[18:44:55.274] info:  Client window became inactive, disabling timeouts for bridge client...
[18:44:58.878] info:  Client window became active, reenabling timeouts for bridge client...
[18:45:11.451] info:  Remix UI activated.
[18:45:13.350] info:  Remix UI activated.
[18:45:26.867] info:  Client window became inactive, disabling timeouts for bridge client...
[18:50:18.687] info:  Client window became active, reenabling timeouts for bridge client...
[18:50:23.203] info:  Client window became inactive, disabling timeouts for bridge client...
[18:50:29.479] info:  Client window became active, reenabling timeouts for bridge client...
[18:51:03.769] info:  Client window became inactive, disabling timeouts for bridge client...
[18:51:10.184] info:  Client window became active, reenabling timeouts for bridge client...
[18:51:29.044] info:  Client window became inactive, disabling timeouts for bridge client...
[18:35:53.461] info:  Found default config for: Source Engine
[18:35:53.462] info:  Trying to open config file: c:\program files (x86)\steam\steamapps\common\half-life 2\bin.trex\bridge.conf
[18:35:53.462] info:  Found user config file: c:\program files (x86)\steam\steamapps\common\half-life 2\bin.trex\bridge.conf
[18:35:53.462] info:  Effective configuration:
[18:35:53.462] info:    presentSemaphoreMaxFrames  = 1
[18:35:53.462] info:  ==================
[18:35:53.462] info:  NVIDIA RTX Remix Bridge Server
[18:35:53.463] info:  ==================
[18:35:53.463] info:  Version: remix-0.2.0
[18:35:53.463] info:  Running in x64 mode!
[18:35:53.463] info:  Launched server with GUID f40cf259-51fc-4356-b0dc-e62f1953db8d
[18:35:53.463] info:  Server started up, waiting for connection from client...
[18:35:53.463] info:  Registering exit callback in case client exits unexpectedly.
[18:35:53.463] info:  Registering message channel for asynchronous message handling.
[18:35:53.463] info:  Initializing D3D9...
[18:35:53.464] info:  Message channel MessageChannelServer established.
[18:35:53.977] info:  D3D9 interface object creation succeeded!
[18:35:53.978] info:  Loaded D3D9 from c:\program files (x86)\steam\steamapps\common\half-life 2\bin.trex\d3d9.dll
[18:35:53.978] info:  Version of d3d9 loaded is DXVK
[18:35:53.978] info:  Sync request received, sending ACK response...
[18:35:53.978] info:  Done! Now waiting for client to consume the response...
[18:35:53.979] info:  Handshake completed! Now waiting for incoming commands...
[18:35:57.331] info:  Server side D3D9 Device created successfully!
[18:36:04.667] info:  Client window became inactive, disabling timeouts for bridge server...
[18:36:05.838] info:  Client window became active, reenabling timeouts for bridge server!
[18:36:10.126] info:  Client window became inactive, disabling timeouts for bridge server...
[18:36:11.351] info:  Client window became active, reenabling timeouts for bridge server!
[18:36:17.818] info:  Client window became inactive, disabling timeouts for bridge server...
[18:36:18.839] info:  Client window became active, reenabling timeouts for bridge server!
[18:37:14.714] info:  Client window became inactive, disabling timeouts for bridge server...
[18:37:15.735] info:  Client window became active, reenabling timeouts for bridge server!
[18:37:18.521] info:  Client window became inactive, disabling timeouts for bridge server...
[18:37:19.671] info:  Client window became active, reenabling timeouts for bridge server!
[18:39:46.076] info:  Client window became inactive, disabling timeouts for bridge server...
[18:39:54.128] info:  Client window became active, reenabling timeouts for bridge server!
[18:40:12.804] info:  Client window became inactive, disabling timeouts for bridge server...
[18:40:14.621] info:  Client window became active, reenabling timeouts for bridge server!
[18:40:26.898] info:  Client window became inactive, disabling timeouts for bridge server...
[18:40:28.569] info:  Client window became active, reenabling timeouts for bridge server!
[18:40:34.396] info:  Client window became inactive, disabling timeouts for bridge server...
[18:40:42.399] info:  Client window became active, reenabling timeouts for bridge server!
[18:40:57.145] info:  Client window became inactive, disabling timeouts for bridge server...
[18:41:40.511] info:  Client window became active, reenabling timeouts for bridge server!
[18:41:50.338] info:  Client window became inactive, disabling timeouts for bridge server...
[18:41:51.393] info:  Client window became active, reenabling timeouts for bridge server!
[18:42:01.593] info:  Client window became inactive, disabling timeouts for bridge server...
[18:44:51.592] info:  Client window became active, reenabling timeouts for bridge server!
[18:44:55.275] info:  Client window became inactive, disabling timeouts for bridge server...
[18:44:58.878] info:  Client window became active, reenabling timeouts for bridge server!
[18:45:26.867] info:  Client window became inactive, disabling timeouts for bridge server...
[18:50:18.687] info:  Client window became active, reenabling timeouts for bridge server!
[18:50:23.205] info:  Client window became inactive, disabling timeouts for bridge server...
[18:50:29.479] info:  Client window became active, reenabling timeouts for bridge server!
[18:51:03.770] info:  Client window became inactive, disabling timeouts for bridge server...
[18:51:10.184] info:  Client window became active, reenabling timeouts for bridge server!
[18:51:29.045] info:  Client window became inactive, disabling timeouts for bridge server...

Changes the sky to black too.

and capturing usd does absolutely nothing.

RTX Remix requires some setup. Please refer to the community Discord channel for further help. https://discord.gg/rtxremix