Azure AD Linux Integration Side Effects

According to this document, "this feature is not recommended for production workloads".
If this approach would be used on a production VM as an alternative authentication method to local accounts what would be the side effects ?
Example, local (service ) accounts would be only used for deployment activities while Azure AD account would be used for troubleshooting / management tasks. What would be the risk of installing Azure AD extension ?
Please could you provide a raw estimate about when thi feature should be released in GA ( i.e. one year, some months, etc)
Thanks

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

ID: bb8e2623-1f79-aee3-bd97-8b5f647b6980
Version Independent ID: 6346b54d-cc8d-0b79-a6b8-f9144cfa2459
Content: Log in to a Linux VM with Azure Active Directory credentials - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/login-using-aad.md
Service: virtual-machines-linux
GitHub Login: @SanDeo-MSFT
Microsoft Alias: sandeo

@giovannifl Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@SanDeo-MSFT Can you please check and add your comments on this doc update request as applicable

#please-close The public preview feature described in this article was deprecated August 15, 2021. Thank you!
Investigating/working on this in AB#1958451


[back-port][v0.9] fix: return error when verify empty cert chain

back-port #308 to 0.9.X

bors try

CI failed with:
  thread 'main' panicked at '"x509_san_other_name_union_(unnamed_at_vendor/include/mbedtls/x509_crt_h_115_5)" is not a valid Ident', /home/travis/.cargo/registry/src/index.crates.io-6f17d22bba15001f/proc-macro2-1.0.24/src/fallback.rs:693:9

If I remember correctly, this is caused by a combination of bindgen/clang-11. Updating the compiler to clang-12 should resolve the issue. Switching to GitHub Actions as part of this PR is a bit weird.

bors try

CI failed with:
  thread 'main' panicked at '"x509_san_other_name_union_(unnamed_at_vendor/include/mbedtls/x509_crt_h_115_5)" is not a valid Ident', /home/travis/.cargo/registry/src/index.crates.io-6f17d22bba15001f/proc-macro2-1.0.24/src/fallback.rs:693:9

If I remember correctly, this is caused by a combination of bindgen/clang-11. Updating the compiler to clang-12 should resolve the issue. Switching to GitHub Actions as part of this PR is a bit weird.

Actually it's odd because there is actually no change on bindgen crate and clang11. What is only changed is rust toolchain.

bors cancel

bors try-

bors try

bors try

bors try

bors try

bors try

bors try

bor r=[xinyufort]


[Feature] Ability to Export a Single Rule

IF individual rules could be exported then it'd make it easier to add them to the list of rules

Also if there were some way to break up the json (in the repo before it gets combined into one rule) based on names then people could contribute to the list a bit easier

In next release I will add import for 1 rule
and folder where need to save rules by files

<3


Increase production data volume to 1TB

Increase production data volume to 1TB
🗣 Description
Increase production data volume to 1TB.  We have almost filled up 200GB already.
💭 Motivation and Context
We have almost filled up 200GB already.  The size must be increased.  The spice must flow.
🧪 Testing
I (very carefully) deployed this change to production and resized the file system successfully.
🚥 Types of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (causes existing functionality to change)

✅ Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

This change has been successfully deployed to prod-a.


Packaging tweaks

Checklist

[X] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Big changes in this PR are:

Add myself as a recipe maintainer. (It's my package upstream).
Make pyyaml and toml optional dependencies; the rationale for this is in the commit message.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Tranparent Button

Fixes Issue
Closes #1617

Changes proposed

Screenshots

Demo vedio:
https://github.com/Spyware007/Animating-Buttons/assets/98798977/11599fd0-b90f-455e-9d47-b1655dffdbd3

Checklist

[x] You've linked this PR to the correct issue.
[x] You have checked that the code is working correctly.
[x] You ⭐ the repository!

Note to reviewers

@Spyware007 @adity2911 Please review it there are no animation just hover effecr

@adity2911 please review and merge it

This was made clear before only that buttons PR will be suspended for the time being.
Wait for us to finish the readme.md file so that the new instructions can be followed.


Create release for a fix

Hi,
Can you make a release with the differents fix like: https://github.com/WyriHaximus/TwigView/commit/c786956cd52a0f440efa8acdfb60fda0275faaa3 ?

Yes they will be release, planning on releasing 4.0 due to some BC changes in the code since the last release. And want to add a few extra's for 4.0

In the meantime, you can pin the version in composer to dev-master, e.g. in composer.json :
"wyrihaximus/twig-view": "dev-master"

Ok think you

FYI I've added some things I want to add to the 4.0.0 mile stone: https://github.com/WyriHaximus/TwigView/milestone/2

Just released 4.0 and 3.3.1 with this and another fix


Relative url for comment embed code

Comment embeds doesn't work right now because the src url is relative. The pull request #113 fixes that.

Thanks!


OSD-9819 - Handle delete before requesting clusterdeployment

Completes: https://issues.redhat.com/browse/OSD-9819

Certman operator contains a race-condition which potentially allows clusterdeployments to be deleted before the certificaterequest controller can retrieve the object. At present, the clusterdeployment is requested before delete operations are handled, and causes the controller to crash when no valid CD is found for a certificaterequest.
These changes reorder the operations to handle certificaterequest deletion before requesting the clusterdeployment, eliminating the race-condition which would otherwise block a cluster deprovision.

/lgtm


Fix and re-enable ArtifactTransformParallelIntegrationTest

The test has been flaky.
Fixes https://github.com/gradle/gradle-private/issues/3283.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test and merge

Your PR is queued because #16995 is being built.

OK, I've already triggered a build for you.

Pre-tested commit build failed.


Set disableLineTextInReferences=true in TS user preferences

In https://github.com/microsoft/TypeScript/issues/51017 and https://github.com/microsoft/TypeScript/pull/51081 we introduced this option to speed up references, but then never set it in VS Code. Oops!

(Ignore the branch delete/close/restore/reopen; I was cleaning up my fork and accidentally forgot to ignore this branch.)


Support for Java libraries

I'm a developer of Apache Camel K and we have often issues with first time users that struggle to install it, because they need to configure an external container registry for our image builder.
I looked at Nixery and found it very promising for improving the getting started experience.
I was wondering if a schema like this can be added:
nixery.dev/java/org.apache.camel_camel-core_3.0.0/org.apache.camel_camel-timer_3.0.0

That would be generic for any java project, not only for us. Libraries could be downloaded from maven central directly.
Having it on the default instance of nixery.dev would improve our getting started experience because we won't require anymore any special configuration for first time users.
Best effort guarantees are ok for our use case.
If this is in line with your expectations from your project, I can also work on implementing this (just need some pointers to get started).

Hi @nicolaferraro,
I think one way to do that would be to write Nix derivations for these Java packages, as explained in the nixpkg manual. If the Nix derivations get included in the official nixpkgs you could use them directly with Nixery. Otherwise you could host them on your own fork of nixpkgs and point your own copy of Nixery to that fork.
I hope this helps!


Unknown constant pool tag in classfile module-info.class

Currently the fast-classpath-scanner fails for JPMS modules, as the Java 9 Spec defines two additional constant pool tags for module-info classes (see https://docs.oracle.com/javase/specs/jvms/se9/html/jvms-4.html#jvms-4.4): CONSTANT_Module (19) and CONSTANT_Package (20).

@wuetherich Thanks for finding and reporting this, and thanks for the associated PR #142.
The PR is great for now. However, for full JDK9 support (#36), I'm guessing that at least CONSTANT_Package will probably have to be parsed, since I'm guessing that allows JDK 9 to break the requirement that classfiles paths match the package name, or something?

@wuetherich PS if you know anything about JDK9, I would love your comments on #36. I have yet to dive headlong into that.

Published in 2.6.1. Please also see comments/questions above. Thanks!


Make text input component example id's unique

Fixes: #451

You can preview this change here:
Built with commit 2fa1888ea5aacfd9772030bc42b18c1c93999a84
https://deploy-preview-453--govuk-design-system-preview.netlify.com


ParseException on the converter

The ParseException declared on the convert method is not useful as sometimes it is difficult to work out the index to throw this exception. Besides, most of cases, an IllegalArgumentException is thrown. I suggest to add the throws IllegalArgumentException in the javadoc so that it can be decided by end users whether to deal with this exception or not.

Not quite sure I get what you mean.
IllegalArgumentException is also fine for me.
The reason for replacing the own ConverterException with any existing standard exception is that we do not treat it anyway. And a user can neither.
The real question is: If some configured value is simply wrong (e.g. '202-14-37' for a date) what can the user do? Is it worth catching this exception? Who is responsible for logging the error? Or do we just let it blow up?
My thirst thinking is that I would let it blow up and just clearly define who is responsible for logging a meaningful message IF a value cannot get converted properly.

I think we should let it blow up. If the user decides to catch it, it is his choice. I tried to explain the ParseException is less useful, as it is checked exception and you cannot wrap another exception inside (see its constructors).

ok I see re ParseException. Missed that.

Great, thanks.

Fixed


A wallpaper module

Fix #15
I tried to make the README part as concise as possible, but it was hard.

The modern tabbar style also has to adjust to the new helpers path by replacing helpers.prrect with helpers.shape.prrect right?

The modern tabbar style also has to adjust to the new helpers path by replacing helpers.prrect with helpers.shape.prrect right?

Damn, I missed these with the merge! Fixed.

Damn, I missed these with the merge! Fixed.

Looks good. Thank you :+1:

Looks good. Thank you :+1:


Typewriter effect done

Issue #55 solved
I have added a typewriter effect on the main heading keeping in mind the original styling of the text
Before

After
https://github.com/thestarsahil/Counsellor/assets/104912087/826ce72a-4423-48be-ac2a-5277dfe0cae5
Thank you for this opportunity @thestarsahil

Really sorry sir for not noticing that. Can i do the necessary changes in some days as my exams are going on ryt now @thestarsahil

@Anshika-Jain02
No problem at all! I completely understand that exams take priority. Take your time to prepare for your exams and once you're done, you can work on the necessary changes to the signup page. Whenever you're ready to work on the project, feel free to get back to it. Good luck with your exams!

Thank you sir @thestarsahil


ISPN-15291 Functional map with simple cache

https://issues.redhat.com/browse/ISPN-15291
The first commit handles the notification part. The second commit adds a WriteOnlyMap implementation with a simple cache and tests for all the simple variants.

merged thanks @jabolina !


Implemented script to remove old batch-api jobs

Script to remove old jobs, it will free some memory and clean keys in Redis

Tested in development:
ubuntu@cdb-dev:/vagrant/repos/CartoDB-SQL-API$ node batch/maintenance/remove-old-batch-jobs.js
  > using "development" environment +0ms
  > redis "<IP_ADDRESS>:6379" +3ms
  > going to scan jobs +4ms
  > found "2" jobs +33ms
  > -------------------------------------------------- +2ms
  > fetching job "batch:jobs:e191d3b6-3f71-4239-9804-0f7f8f6784df" +1ms
  > job "batch:jobs:e191d3b6-3f71-4239-9804-0f7f8f6784df" found (user:cdb, status: done, created_at: 2019-04-09T15:58:58.586Z, updated_at: 2019-04-09T15:59:00.110Z) +5ms
  > job "batch:jobs:e191d3b6-3f71-4239-9804-0f7f8f6784df" is younger than two days, keeping it +0ms
  > -------------------------------------------------- +1ms
  > fetching job "batch:jobs:341cd441-184b-4e34-840d-14a3e2e0502e" +0ms
  > job "batch:jobs:341cd441-184b-4e34-840d-14a3e2e0502e" found (user:cdb, status: done, created_at: 2019-04-08T20:21:54.947Z, updated_at: 2019-04-07T15:21:54.947Z) +5ms
  > job batch:jobs:341cd441-184b-4e34-840d-14a3e2e0502e is older than two days, removing it +0ms
  > job "batch:jobs:341cd441-184b-4e34-840d-14a3e2e0502e" removed +1ms
  > -------------------------------------------------- +0ms
  > summary: { found: 2, removed: 1 } +0ms
  > done +3ms

And staging:
[azew] ubuntu@api-blue-02-st:~/www/node-sql-api/current$ NODE_ENV=staging node batch/maintenance/remove-old-batch-jobs.js
  > using "staging" environment +0ms
  > redis "mtr02-st.azew.cartodb.net:6379" +3ms
  > going to scan jobs +10ms
  > found "2" jobs +457ms
  > -------------------------------------------------- +1ms
  > fetching job "batch:jobs:027f9328-f4d7-4ada-9058-4cb3d6dcf9ab" +0ms
  > job "batch:jobs:027f9328-f4d7-4ada-9058-4cb3d6dcf9ab" found (user:dgaubert, status: done, created_at: 2019-04-09T16:25:04.124Z, updated_at: 2019-04-09T16:25:04.786Z) +4ms
  > job "batch:jobs:027f9328-f4d7-4ada-9058-4cb3d6dcf9ab" is younger than two days, keeping it +0ms
  > -------------------------------------------------- +0ms
  > fetching job "batch:jobs:b3000d49-4777-4518-a588-cf62621ff63c" +0ms
  > job "batch:jobs:b3000d49-4777-4518-a588-cf62621ff63c" found (user:dgaubert, status: done, created_at: 2019-04-09T16:25:38.024Z, updated_at: 2019-04-09T16:25:38.568Z) +3ms
  > job "batch:jobs:b3000d49-4777-4518-a588-cf62621ff63c" is younger than two days, keeping it +0ms
  > -------------------------------------------------- +0ms
  > summary: { found: 2, removed: 0 } +0ms
  > done +3ms

Thinking out loud: using this script, we are removing jobs in Redis, but we could still have the jobs in memory and due to it, we could have unexpected behaviours. What do you think?

Forget my previous comment, it should work


"Modified on: " field should not depend on local update but on the effective modification

For example, if a developer does a "git checkout", which updates the content of the file, then the parser thinks that the modification date is the date of the checkout.
Ideally, the modification should be based on the github repository (if possible).

http://stackoverflow.com/questions/6795070/is-there-a-way-in-git-to-obtain-a-push-date-for-a-given-commit

Git hooks.
https://www.atlassian.com/git/tutorials/git-hooks/
Ill try that.

could you please change:
Modified on  Mon Sep 28 16:33:53 2015

for:
Modified on:  2015-09-28 16:33:53

make sure to manage in case there is no modif.txt file

let's change the approach since there is a problem with having developers manage their own motif.txt. How about this:
http://stackoverflow.com/questions/2179722/checking-out-old-file-with-original-create-modified-timestamps/2179876#2179876

ok, after all, let's forget about modification date and just display version.txt.


Release

merges develop > main 🚀

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


ubuntu-24.04 claims ruby versions, but none are installed

Description
The Readme for ubuntu-24.04 says that Ruby 3.2.3 is available on the image. This must be the installed system ruby because no others are listed as available.
If you look here, you can see that there are no versions of ruby.
You can also see this in my azure pipelines output on ubuntu-24.04:
Starting: UseRubyVersion
==============================================================================
Task         : Use Ruby version
Description  : Use the specified version of Ruby from the tool cache, optionally adding it to the PATH
Version      : 0.248.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/tool/use-ruby-version
==============================================================================
##[error]Version spec 3.2 for architecture %s did not match any version in Agent.ToolsDirectory.
Available versions: /opt/hostedtoolcache

If this is a Microsoft-hosted agent, check that this image supports side-by-side versions of Ruby at https://aka.ms/hosted-agent-software.
If this is a self-hosted agent, see how to configure side-by-side Ruby versions at https://go.microsoft.com/fwlink/?linkid=2005989.
Finishing: UseRubyVersion

Given that 2 different minor versions are available on ubuntu-22.04, it seems like a strange oversight to exclude any selectable versions of ruby from the toolset.
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[X] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
Version: 20241117.1.0
Is it regression?
ubuntu-22.04
Expected behavior
Expected ruby 3.2 and ruby 3.3 to be selectable with UseRubyVersion.
Actual behavior
No rubies are selectable with UseRubyVersion.
Repro steps
Create a pipeline with UseRubyVersion in azure-pipeline.yml on an ubuntu-24.04 image on a microsoft hosted agent.
          - task: UseRubyVersion@0
            inputs:
              versionSpec: "3.2.3"
              addToPath: true

Hi @martinemde Thank you for bringing this issue to our attention. We are looking into this issue and will update you on this issue after investigating

Hi @martinemde The Ruby version 3.2.* has been added in toolset-24.04 , and new image with the changes has been deployed. Could you please retry by forking latest code from main branch. We are closing this issue, thanks.

Thanks for the quick fix.


Add flag for slow target value function

In this issue, you mentioned that using a slow target network for the value function could prevent the loss from diverging.  Would it be possible to add a flag to enable this option?  Or could you please describe how you implemented this feature? Thanks!

This is implemented in the DreamerV2 repository: https://github.com/danijar/dreamerv2


Too deep namespace

Inside 'searching' folder we have subfolders with just one js file. Maybe better to make folder structure not so deep like this:
searching/
-- binarysearch.js
-- recursive-binarysearch.js
-- knuth-morris-pratt.js
-- longest-increasing-subsequence.js
-- quickselect.js
-- maximum-subarray-divide-and-conquer.js
-- maximum-subarray.js
?

Ok, lets do it this way.


Solve the problem of different kinds of build errors slowing down the build process

🚀 Feature
Implement a toggle switch which ignores/moves/removes faulty md/doc files found while compiling into static html.
Have you read the Contributing Guidelines on issues?
Yes
Has this been requested on Canny?
No
Motivation
At our company we are currently using docusaurus as a poc. We have lots of documents from prior flat file cms systems. Our problem is, that in order to compile successfully we have to go through every build step, catch errors found in the md files and remove them afterwards. This is currently done recursively because each "build step" yields different categories of errors. This increases the consumed time of our CI pipelines which is not optimal. To catch these faulty files via regex queries in the console output is also not the neatest way of handling this. Some faulty files are repairable easily, for example broken img links can just be removed. This is still tedious to do it for every bug so we decided to just delete them.
API Design
We now need a way of finding out whats wrong with our documents before building. Currently investigating different md linters for that, but maybe this could also be built into saurus? ex. cli tool for scanning a file which automatically removes lines which it cant compile, proceeds to the next build step and outputs the errors of the different error "hierarchies"/ categories. And if the linter doesnt catch our problems a way in which the compiler ignores/moves/removes these documents so it can build in ONE go would be really helpful.
Have you tried building it?
As i said currently we can only catch these errors via recursive building and removing the whole files. I also tried to look into the packages that scan/ compile these files but i felt that the code was spread on different packages which made it harder to find all aspects which docusaurus is using to compile.
We are ready to build this for you/ cooperate if you think its a necessary/ good feature.

I'm not sure to understand what you mean by "recursively compile" and "build steps".
We don't have a good way to detect errors early in the build process, and some errors are actually only reported by our own tooling (ie Webpack / Node.js reporting broken paths).
Detecting broken things asap is definitively something we should improve so that users have a quick feedback loop, but we are not there yet.
What I understand is that you'd like a "fail-safe" mode: you prefer deploying a broken Docusaurus site (with console warnings so that you are aware of the problems) rather than failing the CI build.
Here I'll need an exhaustive list of the distinct errors you encounter in practice, and the expected output of such a fail-safe behavior for each error.
An example:

If a .md document is not valid markdown/MDX, then I prefer to have the site render an empty page rather failing

(I'll be on holiday so I'll likely answer in a few weeks)

Thanks for your answer! I'll attach some code we currently use to rerun the build process when im in office on monday. Additionally i'll append some console logs that identify the errors.
"Here I'll need an exhaustive list of the distinct errors you encounter in practice, and the expected output of such a fail-safe behavior for each error."
Yeah it could be done like this and as a result we could have a config to tell what to do with the specific errors ... eg: "throw/warn/error/remove"
But i guess some errors are so crucial that you cant continue the compilation process with the file so you'd have to delete it
But nevertheless a single switch which just deletes the faulty file and logs it would be the easiest solution, because when the 3rd party packages are updated they might introduce new code scanning methods which may lead to new error categories and we'd have to maintain a list of probable errors which seems to be time consuming. I think its also the easiest to implement.

mdErrors.txt
deleteFaultyMds.py.txt
Here is the script we currently use to build docusaurus. Yes its extremly crude and not very error tolerant but it just represents a POC. The most important method is the following

in this method the build process is executed and depending on the returned observed error from the logs, a different
removal process is executed and the method is run again.
In the attached file, i appended some markdown text and the resulting error logs. They are sorted in the sequence they occur. I didnt attach them, but missing image links also cause the build to fail.
As a solution instead of just deleting the faulty file there ofc are many other solutions like:
-displaying and empty file
-displaying and empty file with just the error log output
-display the page with errors (probably not possible because the different packages which build and
optimize the page consecutive require a clean state in order to pack and bundle the file correctly?)

I have a similar problem where the Crowdin auto translations are problematic. E.g. Make sure to save your <0>chart layout</0> on Tradingview after successfully installing an update so you can ensure it will stay becomes Assurez-vous d'enregistrer votre mise en page</0> <0>sur Tradingview après avoir installé avec succès une mise à jour afin de vous assurer qu'elle restera. I have no idea why it does that since the order of words didn't even change much, but I ended up having a lot of tags being in the wrong place. While I can fix the French translation myself, I'm pretty clueless about other languages like German, so it would be cool if we can catch the page build errors and have some fallback (e.g. for i18n, fall back to base locale; for other cases, display a dedicated "something went wrong" page), while we look for translation contributors to inspect the issue.

regarding my problem - wouldnt it be the easiest to just implement a "force" build mode in which it excludes problematic files after each step? Like in my script but without the need to rerun the whole build process. And if the files are already indexed in some minified packed js file they ofc need to be removed/excluded from there too (maybe keep the sitebar entries but forbid navigation to the file and show a tooltip).

MDX v2 (Docusaurus v3) comes with a CommonMark compatible mode, which should throw much less errors compared to MDX regular compilation format.
CommonMark is much more fail-safe, and able to ignore things that seem invalid, like unclosed html/JSX:

I'm going to close assuming this is enough.
Useful links:

CommonMark support RFC: https://github.com/facebook/docusaurus/issues/3018
Some current limits: https://github.com/facebook/docusaurus/issues/3018
MDX v2 PR: https://github.com/facebook/docusaurus/pull/8288


Fix bug with calculations on small timescales

Proposed changes
When performing simulations on small timescales (e.g., final_time=1e-7), some bugs appear due to the use of np.isclose with the default parameters (atol, rtol). For example (see below), computation stops without reaching the final time. As @RemDelaporteMathurin suggested, the current PR attempts to fix the bug by setting atol=0 in np.isclose.
Types of changes
What types of changes does your code introduce to FESTIM?

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Code refactoring
[ ] Documentation Update (if none of the other choices apply)
[ ] New tests

Checklist

[x] Black formatted
[x] Unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments
MWE:
import festim as F
import numpy as np

my_model = F.Simulation(log_level = 40)

my_model.mesh = F.MeshFromVertices(np.linspace(0, 1, 10))

my_model.materials = F.Material(
    id=1,
    D_0=1,
    E_D=0.1,
)

my_model.T = F.Temperature(1000)

my_model.settings = F.Settings(
    absolute_tolerance=1e-10,
    relative_tolerance=1e-10,
    final_time=1e-7,
)

my_model.dt = F.Stepsize(
    initial_value=1e-10,
)

my_model.initialise()
my_model.run()

Stops at 90%:
Defining initial values
Defining variational problem
Defining source terms
Defining boundary conditions
Time stepping...
90.0 %        9.0e-08 s    Ellapsed time so far: 3.2 s

Very nice MWE @KulaginVladimir , would you mind making use of it in a test that catches the bug?
I think you can do something like:
def test_small_timesteps_final_time_bug():
    """Some description"""
    my_model = F.Simulation(log_level = 40)

    my_model.mesh = F.MeshFromVertices(np.linspace(0, 1, 10))

    my_model.materials = F.Material(
        id=1,
        D_0=1,
        E_D=0.1,
    )

    my_model.T = F.Temperature(1000)

    my_model.settings = F.Settings(
        absolute_tolerance=1e-10,
        relative_tolerance=1e-10,
        final_time=1e-7,
    )

    my_model.dt = F.Stepsize(
        initial_value=1e-10,
    )

    my_model.initialise()
    my_model.run()

    assert np.isclose(my_model.t, my_model.settings.final_time, atol=0.0)


Samsung SmartThings Smart Plug

Please can you add support for the Samsung SmartThings Smart Plug, which is already supported by zigbee2mqtt - https://www.zigbee2mqtt.io/devices/GP-WOU019BBDWG.html (support has been added for the F-APP-UK-V2, but not for this one).
MQTT message with device information
PUT here the device data you receive when publish message with topic
zigbee2mqtt/bridge/config/devices, as example
{"ieeeAddr":"0xccccccfffe3cc886","type":"Router","networkAddress":6053,"model":"GP-WOU019BBDWG","friendly_name":"Cabin_Heater","manufacturerID":4456,"manufacturerName":"LDS","powerSource":"Mains (single phase)","modelID":"ZB-ONOFFPlug-D0005","hardwareVersion":1,"softwareBuildID":"2.3","dateCode":"20190716-165","lastSeen":1585916990652}]

MQTT messages from device
Put here examples of MQTT messages with different states you've received from device, as example
info  2020-04-03 13:34:52: MQTT publish: topic 'zigbee2mqtt/Cabin_Heater', payload '{"power":0,"current":0,"voltage":0,"linkquality":13,"state":"OFF"}'

MQTT state change commands
See here - https://www.zigbee2mqtt.io/devices/GP-WOU019BBDWG.html

If you add a row in file /adapters/samsung/init.py at the end of the list,than it supports your power socket:
samsung_adapters = {
.
.
'F-APP-UK-V2': OnOffSwitchAdapter,  # SmartThings Outlet
'GP-WOU019BBDWG': OnOffSwitchAdapter, # SmartThings Outlet (new)
}
new = your device entry

"init.py"

init.pi

I've done that, and it works fine for on/off, but this switch has power monitoring, so that doesn't capture the power or usage info.

I see did not considered the additional features. In that case you have to created a new class.
Change the _init_.pi file
from adapters.samsung.GPWOU019BBDWG import GPWOU019BBDWG
'GP-WOU019BBDWG': GPWOU019BBDWG, # SmartThings Outlet (new)
==create new file GPWOU019BBDWG.py============
from adapters.on_off_switch_adapter import OnOffSwitchAdapter
from devices.sensor.current import CurrentSensor
from devices.sensor.voltage import VoltageSensor
from devices.sensor.kwh import KwhSensor
class GP-WOU019BBDWG(OnOffSwitchAdapter):
def init(self, devices):
super().init(devices)
self.devices.append(VoltageSensor(devices, 'volt', 'voltage', ' (Voltage)'))
self.devices.append(CurrentSensor(devices, 'ampere', 'current', ' (Current)'))
self.devices.append(KwhSensor(devices, 'power', ['power'], ' (Power)'))
==
you can take the inri/SP120 as example code, you need to modify the class name en file
Be aware changes for your own risk. I'm not an expert ..

Done, could you please update plugin and check

Hi Stanislav, I saw in the payload three attributes:
payload '{"power":0,"current":0,"voltage"
Therefore I suggested to create a new class. I was not aware about the limit generic kWh class that you presented

Brilliant - thank you!


Issue when updating from 4.1.4 to 4.3.0 - webparts not updated

PROVISIONING NOTE: If you are having issues with the SharePoint look book while trying to provision M365 learning pathways please do not post that issue to this repo as we cannot help you here. Please post all questions and issues for the SharePoint look book to SharePoint/sp-provisioning-service GitHub Issues List.

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.

Describe the bug
After updating the latest version from 4.1.4 to 4.3.0, I am getting an error information in the CustomLearningAdmin.aspx page that the verion is not latest. When looking at the content packs, the latest updates are not there for 2021. The appcatalog package has been updated and published. The information about the webparts in the CustomLearningAdmin.aspx still shows as 4.1.4, but the app on the site shows 4.3.0 with no ability to get the latest content or update
To Reproduce
Run all pre-reqs and install 4.1.4.
Install package 4.3.0
Expected behavior
Webpart version is updated, and the latest content is available
Screenshots

Learning Pathways version number
4.3.0

Hello @J-Liner, If you could check a few things it would be helpful. 1) Can you check in the App Catalog where you installed the new SPPKG file that there were no errors. 2) When you go to the CustomLearningAdmin.aspx page do you get any error in the browser console or on the screen other than the upgrade message? If there are errors in the console please provide those. You may also want to try updating the cache. You can do this by deleting the entries in the CustomConfig list. Let me know what comes up.

@dcashpeterson really appreciate the quick response. I've been looking into your suggestions, and here is what I have so far.

The SPPKG didnt have errors when installing/publishing
The CustomLearningAdmin.aspx page loaded without errors (validated through F12)
I am trying to clear our the CustomConfig list, but I keep getting an error when loading the page.

I'll post back once I get the CustomConfig list sorted

@dcashpeterson really appreciate the quick response. I've been looking into your suggestions, and here is what I ahve so far.

The SPPKG didnt have errors when installing/publishing
The CustomLearningAdmin.aspx page loaded without errors (validated through F12)
I am trying to clear our the CustomConfig list, but I keep getting an error when loading the page.

I'll post back once I get that sorted

I was able to clear out the CustomConfig list, and it repopulated. The JSON was interfering with the page load, possibly too much content for a list entry. I created a new list view, and removed the column. Clearing the list did not resolve the version issue.
Additional things I've tried:

re-installed the app (cleared the recycle bin just in case).

Validated` App Instances details that the app installed correctly with the right version

forcing an update using "SitePages/CustomLearningAdmin.aspx?forceUpdate=v3"
Validated the CDN, and Site URL tenant properties (still correct)
Validated properties stored on the site

App installed for Microsoft 365 learning pathways. InError: False
_PnP_ProvisioningtemplateInfo {"TemplateId":"CUSTOMLEARNING","TemplateVersion":0.0,"TemplateSitePolicy":null,"ProvisioningTime":"2021-09-23T04:24:08.7145208+00:00","Result":true}

I still cannot find where the CustomLearningAdmin.aspx is pulling the "Current Web Part Version" from the image above.
The major problem is that the content is only through Sept 2020. I dont know that fixing the webpart version will resolve that issue.

@J-Liner you have definitely checked all the things I would have asked you to check. It sounds like there are a few issues here. 1) the webpart version number doesn't seem to be updating and 2) the content isn't getting picked up. Let's tackle number 2 first. Check if you have the following playlist https://.sharepoint.com/sites//SitePages/CustomLearningViewer.aspx?cdn=Default&playlist=479705db-2e50-4a8b-8a2e-a2a3d59ce61b&asset=8c923c9d-4ef2-849d-a399-4de716ecbbaf That was part of the latest update. Unless you are using a custom content pack that is a copy of the previous version you should get the updates automatically. There is an issue we are working through with the Success Center that hasn't been updated but the rest of the playlists have.
For number 1, the webpart version is stored in the webpart manifest but also in the customconfig list. Look at the entry in there. You can use the network tab and look for WebPartVersion. My guess is that there was an issue in the update and something errored out.
Try that and let me know what you see.

The playlist above did load:

I had cracked open the SPPKG earlier to validate we had the right package (sometimes that happens), and it is indeed the Version="<IP_ADDRESS>". I didn't see an entry for updating the app package but that isn't necessarily required.
The CustomConfig JSON is outside the boundry for the field at 32k characters. Inspecting the JSONData field looked well formed, and the site is working so that isn't the problem. Do you want me to include the config value here? it is really long.
Viewing network/console there were several CORS errors when view the webpart.
The Admin page always loads clean.
From the CustomLearningViewer.aspx link above:

From work-remotely.aspx:

The webpart is loading just fine. I couldn't find the version number in its response. What I did find is that there are old resources from previous versions in the client side assets for the app. Maybe that is related?

@J-Liner I need to do some more digging into the code. I will let you know what I find out.

@J-Liner The web part version gets set through the manifest of the webpart. When the webpart loads on the page the manifest for the webpart is loaded into the context of the page. The admin webpart retrieves that value from the context and puts it in the cache. What you are seeing in the Client Side Assets library are indeed old file and those are not the current files. This leads me to believe that there was an error when the webpart was installed. Have you tried updating the sppkg file in the app catalog already? If the update was successful you would see files named m365LPString_en-us-GUID.js. If you haven't try that and check the console while you are doing it. Are you using a site collection app catalog or the tenant app catalog?

@dcashpeterson I did not try re-installing/republishing through the app catalog because I dont have permissions for that. I'll get that done today and post back the findings.

@J-Liner Did you get the web part reinstalled and did that resolve the issue?

@dcashpeterson Still working on getting a GA. Just responded to an email 5mins ago.

@dcashpeterson We were able to redeploy the package, and the assets are now correct. The admin page is referencing the old webpart now. Will fix that and post an update

@J-Liner I am going to go ahead and close this. If this is still an issue feel free to open a new issue and address this one.


Allow updating subscribers in more order statuses

Before:
Subscribers could be added or removed only if an order was in draft.
After:
Subscribers can be added or removed if an order is in draft, quote_awaiting_acceptance, quote_accepted or paid.
They still can't be changed if the order is complete or cancelled.

Codecov Report

Merging #667 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #667   +/-   ##
========================================
  Coverage    96.53%   96.53%
========================================
  Files          128      128
  Lines         3957     3957
  Branches       341      341
========================================
  Hits          3820     3820
  Misses          96       96
  Partials        41       41

Impacted Files
Coverage Δ

datahub/omis/order/serializers.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


less annoying debugger

@olydis,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


cannot resolve includeVideo(true)

Hi all,
I am using  com.github.esafirm.android-image-picker:imagepicker:1.12.0 version, and when I add includeVideo() to builder I get an error "Cannot resolve method"
Kind regards,
Alexandr

It should be there
https://github.com/esafirm/android-image-picker/blob/master/imagepicker/src/main/java/com/esafirm/imagepicker/features/ImagePicker.java#L161
Anyway, you should use the new version


Serialization issue when using missing

The following round-trip serialization has an error when reading the data back in. The error is ERROR: MethodError: no method matching Union{Missing, String}(::String).
using Catlab

@present SimpleSch(FreeSchema) begin
    X::Ob
    Attr1::AttrType
    attr::Attr(X,Attr1)
end

@acset_type SimpleData(SimpleSch)

simple_data = @acset SimpleData{Union{String,Missing}} begin
    X=3
    attr=["a",missing,"c"]
end

write_json_acset_schema(SimpleSch, "./simplesch.json")
write_json_acset(simple_data, "./simpledata.json")

SimpleSch1 = read_json_acset_schema("./simplesch.json")

@acset_type SimpleData1(SimpleSch1)

simple_data1 = read_json_acset(SimpleData1{Union{String,Missing}}, "./simpledata.json")

On versions
  [227ef7b5] ACSets v0.2.13
  [134e5e36] Catlab v0.16.5

@olynch or @epatters do you have any ideas on how we could address this? The issue is when parsing the json acset we hit these methods in StructTypes https://github.com/JuliaData/StructTypes.jl/blob/main/src/StructTypes.jl#L524-L526 and it tries to call Union{String,Missing}(). Once again, union types being a source of strange behavior in Julia.


Proposal: extract all string between two separator

A string api to extract all words between two chartacters.
use case:
str :=  "[classname] [function-name] [randomstring] somethingmore"
splits := strings.extractbetween(str, '[', ']')
splits will be ["classname", "function-name", "randomstring"]

This functionality seems to have somewhat narrow application and could be handled using a regular expression:
re := regexp.MustCompile(`\[[^]]*\]`)
ss := re.FindAllString("[classname] [function-name] [randomstring] somethingmore", -1)
for _, s := range ss {
	fmt.Println(s[1 : len(s)-1])
}

What's the justification that this needs to be special-cased in the standard library?

You could also implement this with a mix of strings.Split and TrimPrefix/TrimSuffix.

yeah, i used Split to implement this. and of course there are other ways to implement it. I was wondering if this is worth including in the standard library.
Coming from a java background, i know Apache common-langs provides an api in stringutils library and it is really handy.
https://github.com/apache/commons-lang/blob/5cdac9cfd5a74b0a52ebde32798b973c6edbaa79/src/main/java/org/apache/commons/lang3/StringUtils.java#L3060

Perhaps publish this as a third party package first, or as part of a third-party layer on top of strings. Then you can start judging if it's popular enough to consider moving into the standard library.
I personally have never had this use-case, nor have I seen any code implementing it, so I default to thinking it doesn't belong in the standard library. That is, unless you have evidence that this is a common need in Go code out there.

https://golang.org/doc/faq#x_in_std

aha... this makes more sense. I was looking for guiding principles that defines if something should be a part of standard library or not.
thank you everyone for all your inputs. happy to publish at third party package on top of strings.


[quote] - auto-instrumentation produces more span than expected since #644

Bug Report
The quoteservice now produces 14 spans on each request. This used to be 2 spans before #644 was merged. Fixing this may require updates to the PHP SDK itself.
Screenshot of a trace snippet with the quoteserivce involved showing 14 spans for a GetQuote request

/cc @brettmc

In progress, I've got it down to 3 spans which feels like a sane amount (remembering that it's being instrumented by a generalized auto-instrumentation for the framework that quoteservice uses, not specialized for the demo):


[SPARK-22667][ML] Fix model-specific optimization support for ML tuning: Python API

What changes were proposed in this pull request?
Python CrossValidator/TrainValidationSplit:
With base Estimator implemented in Scala/Java
→ Convert base Estimator to Scala/Java object, and call the JVM fit() (as in Weichen’s comment)
With base Estimator implemented in Python
→ Python needs the same machinery for multi-model fitting and parallelism as Scala.  We can call directly into it. New API added:
class Estimator:
  def parallelFit(self, dataset, paramMaps, threadPool, modelCallback):

Note This PR also fix the # TODO: persist average metrics as well in CV/TVS. Because the testsuite need to check consistency of avgMetrics so this need to be fixed.
If this need backport to old spark version, I can split it to a separate PR.
How was this patch tested?
Existing UT already covers each code paths which need test.

@MrBago @jkbradley I think this PR need to be reviewed and merged first, before reviewing #19627
Because this PR change some critical code path.

Test build #84372 has started for PR 19857 at commit 980c8ec.

Test build #84372 has finished for PR 19857 at commit 980c8ec.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #84373 has started for PR 19857 at commit c6f2250.

Test build #84373 has finished for PR 19857 at commit c6f2250.

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

Merged build finished. Test PASSed.

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

The design of this issue changed. @MrBago will take this over.


Create list of all commands

If needed, write an automated utility to do this for us.
We need this to make checklist for project spawning changes.

fixed in f5453f9b2142588de29ecc0e6ff0e6464ac21ece


Move auth functions to auth.ts

Including:
REDIRECT_URI_OOB
oauth2Client
getAPICredentials
authorizeWithLocalHost
authorizeWithoutLocalHost
Note: I am exporting LOG from index.ts just until we move it to /src/commands.ts
Also, though the tsconfig.json file still has this:
  "files": [
    "index.ts",
    "tests/test.ts",
    "src/utils.ts"
  ]

It still compiles /src/auth.ts into /src/auth.js. Might be worth looking into.
Signed-off-by: campionfellin<EMAIL_ADDRESS>Works on #134

[x] npm run test succeeds.
[x] npm run lint succeeds.
[ ] Appropriate changes to README are included in PR.

Pull Request Test Coverage Report for Build 136

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
255 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.6%) to 12.068%

Files with Coverage Reduction
New Missed Lines
%

index.js
255
6.47%

Totals

Change from base Build 135:
0.6%

Covered Lines:
130

Relevant Lines:
801

💛  - Coveralls

Thanks for the detailed comments. LGTM.


Using prefix in QueryService's addCriteria

The Anno4j QueryService allows definition of prefixes. On the other hand it is not possible to use previously defined prefixes in the value argument of e.g. QueryService.addCriteria(String ldpath, String value).
So for example the following query would not return a previously persisted resource http://example.org/myAnno:
qs.addPrefix("ex", "http://example.org/")
.addCriteria(".", "ex:myAnno")
.execute(Annotation.class);
Resolution of resources in prefix notation would make querying more concise.

Hi @fischmat !
I have pushed a (preliminary, as I am sure there would be a more thorough solution down there in Marmotta :)) fix to the develop branch!
https://github.com/anno4j/anno4j/commit/e6e8f57f079446b98dcc816f467e2bc681fcfb0b
Please check if your things work with that solution!

Now it works :)
I think this way querying is more straightforward.


feat : updates on landing page

Description
I feel the empty space should be filled or  text should take up full space.
Screenshots

Additional Information
No response

Any Ideas @KaKashi1210 ?

this repo uses framer motion right ? so it will be nice if we can use text animation that writes devs in tech community

Hello ,
I'm harsh. I want to work on this issue under Gssoc'23. I am really eager to work on open source projects and contribute to this community.I've taken a look at the codebase and I believe I can make some valuable improvements. I'll start working on it right away.
Skills
HTML5, CSS3, JS, Bootstrap | node js mongodb | Php, mysql | MERN | Responsive Design
Can you please assign this issue to me.
@himanshu1221

@KaKashi1210 Go ahead, just add little bit animation

@KaKashi1210 Are you working on it ?

Im really sorry for replying late, i am busy with exams  can you please assign it to someone else.

@AbhiPatel10 please assign this to me i am interested....

@AbhiPatel10 please assign this issue to me.


rust-cardano-crypto is used in react-nativce

The RN project cannot run after the rust-cardano-crypto is loaded

Did I miss something？

I don't think wasm is supported on iOS and android yet, which make it using react-native difficult at the moment

@BenLakes you shouldn’t use the wasm library in iOS and Android. WebAssembly is not supported in iOS and the compatibility in Android differs depending on the OS version.
If you still want to try it out, you should change the default package manager for Haul. It doesn’t load because the size of the wasm file is too big.

@BenLakes in Emurgo we will release a react-native library that uses cardano-rust. If you want I can ping you here when it’s ready. it should be this or next week.

@nicarq  Thank you. We are now anxious to use this library in the react native project. Please inform me immediately if you are ready.

@BenLakes https://github.com/Emurgo/react-native-cardano enjoy! :)

@nicarq  Thank you


Resubscribe all subscriptions on reconnection

This ensures that the service resubscribes to all existing subscriptions of the service ever needs to reconnect

I think this should work. The only thing is, we should use self.sub(id, name, params), rather than self.sub(name, params).

Right. I've updated it and tested, and its working well.


Marking last article read changes folder selection

I keep a smart folder named "_Unread Articles" at the top for all unread articles. That's the folder I keep selected. After spacebarring through all articles and the last one gets marked as read, Vienna 3.1.5 is now changing the selected folder from that one to the parent folder of that last article. I believe it started doing this with the previous release. It was fine a couple weeks ago.
It's pretty annoying. Sure it only happens once a night, but it happens every night and I shouldn't have to reselect my preferred folder every night.

That sounds like a dup of #636 which was just re-opened.

On Jun 20, 2016, at 8:41 PM, sjmills<EMAIL_ADDRESS>wrote:
I keep a smart folder named "_Unread Articles" at the top for all unread articles. That's the folder I keep selected. After spacebarring through all articles and the last one gets marked as read, Vienna 3.1.5 is now changing the selected folder from that one to the parent folder of that last article. I believe it started doing this with the previous release. It was fine a couple weeks ago.
It's pretty annoying. Sure it only happens once a night, but it happens every night and I shouldn't have to reselect my preferred folder every night.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Fixed in 3.1.6


Image fails to upload - Branch protection rule(s)

Error:
When trying to upload an image when Github branch protection rule(s) are in place i.e requires approval we get the following error message.
API_ERROR: 2 of 2 required status checks are expected. At least 1 approving review is required by reviewers with write access.

To Reproduce:

Step 1:  Create a new Post
Step 2: Try to upload an image (even with no PR at this point)

Details:

Netlify CMS version:  "netlify-cms": "^2.9.1",
GitHub

Notes:

If I fork the repo + removing the rules I can upload images.

Will be resolved by https://github.com/netlify/netlify-cms/issues/1344.


TranscriptsDisabled exception

Hi,
I am trying to use this API to get the manually created transcript in Hebrew of YouTube videos.
I am using these code:
transcript_list = YouTubeTranscriptApi.list_transcripts([video_id]) transcript = transcript_list.find_manually_created_transcript(['he'])
I tried it with many different videos with manually created captions, one video for example: https://www.youtube.com/watch?v=i5_5gGimcio
I also tried 'iw' for the language code.
I keep getting the TranscriptsDisabled exception.
What am I doing wrong?

Hi @dankeinan1, list_transcripts() should receive a single video id instead of a list of video ids. The name is pluralized because it returns multiple transcripts not because it takes multiple inputs 😊
I realise that a lot of people get confused with input types, that why I am planning to add type assertions in #125.
I will close this for now, let me know if you're still having issues.


[VS Code] Joining collaboration session: Collaboration session not found.

%0A%0A%23%23 Error:%0AJoining collaboration session: Collaboration session not found.%0A%0A%23%23 Steps to Reproduce:%0A1.%0A2.%0A%0A||Version Data|%0A|-:|:-|%0A|**extensionName**|VSLS|%0A|**extensionVersion**|1.0.311|%0A|**protocolVersion**|2.2|%0A|**applicationName**|VSCode|%0A|**applicationVersion**|1.34.0|%0A|**platformName**|Windows|%0A|**platformVersion**|10.0.14393|

Hi @romeromr10, thanks for the feedback! The issue title says that the session you were trying to connect was no longer active. Do you think the session did end at the time of your connection or you think that's a real issue?
Thanks!

As noted above, most likely the host stopped sharing before you tried to join, so I'm going to go ahead and close this, since the error is expected in that case. If that is not what happened, feel free to reopen, and please provide more details about your experience.


App crash at ClusterStatus

As reported from the community:
App crash at /cluster/5d0e86ac3c1d30f483817124a0e5d134
Version 5.4.5-latest.20220405.1
ClusterStatus (http://localhost:63386/build/chunks/5.js:2817:9)
    at div
    at ClusterView (http://localhost:63386/build/chunks/5.js:3085:9)
    at Route (http://localhost:63386/build/Lens.js:71644:29)
    at Switch (http://localhost:63386/build/Lens.js:71810:29)
    at main
    at div
    at NonInjectedClusterManager (http://localhost:63386/build/chunks/5.js:4294:49)
    at Route (http://localhost:63386/build/Lens.js:71644:29)
    at Switch (http://localhost:63386/build/Lens.js:71810:29)
    at ErrorBoundary (http://localhost:63386/build/Lens.js:228144:9)
    at Router (http://localhost:63386/build/Lens.js:71309:30)
    at NonInjectedRootFrame (http://localhost:63386/build/chunks/5.js:4382:9)
    at ThemeProvider (http://localhost:63386/build/Lens.js:592515:24)
Error stack:
TypeError: message.trim is not a function
    at http://localhost:63386/build/chunks/5.js:2889:248
    at http://localhost:63386/build/Lens.js:4208:23
    at Array.map (<anonymous>)
    at Proxy.<anonymous> (http://localhost:63386/build/Lens.js:4207:36)
    at ClusterStatus.renderAuthenticationOutput (http://localhost:63386/build/chunks/5.js:2889:76)
    at ClusterStatus.render (http://localhost:63386/build/chunks/5.js:2916:22)
    at allowStateChanges (http://localhost:63386/build/Lens.js:1535:12)
    at http://localhost:63386/build/Lens.js:9915:84
    at trackDerivedFunction (http://localhost:63386/build/Lens.js:2084:18)
    at Reaction.track (http://localhost:63386/build/Lens.js:2613:18)
    at ClusterStatus.reactiveRender [as render] (http://localhost:63386/build/Lens.js:9913:14)
    at qi (http://localhost:63386/build/Lens.js:540465:188)
    at pi (http://localhost:63386/build/Lens.js:540464:173)
    at ck (http://localhost:63386/build/Lens.js:540547:427)
    at bk (http://localhost:63386/build/Lens.js:540528:347)
    at ak (http://localhost:63386/build/Lens.js:540528:278)
    at Tj (http://localhost:63386/build/Lens.js:540528:138)
    at Lj (http://localhost:63386/build/Lens.js:540521:163)
    at http://localhost:63386/build/Lens.js:540401:115
    at exports.unstable_runWithPriority (http://localhost:63386/build/Lens.js:540612:343)
    at gg (http://localhost:63386/build/Lens.js:540400:325)
    at jg (http://localhost:63386/build/Lens.js:540401:61)
    at ig (http://localhost:63386/build/Lens.js:540400:428)
    at Wj (http://localhost:63386/build/Lens.js:540522:190)
    at reactionScheduler (http://localhost:63386/build/Lens.js:2740:12)
    at http://localhost:63386/build/Lens.js:2741:14
    at Bh (C:\Program Files\Lens\resources\extensions\lenscloud-lens-extension\dist\renderer.js:115:141185)
    at reactionScheduler (http://localhost:63386/build/Lens.js:2740:12)
    at runReactions (http://localhost:63386/build/Lens.js:2709:3)
    at endBatch (http://localhost:63386/build/Lens.js:2377:5)
    at Atom.reportChanged (http://localhost:63386/build/Lens.js:823:5)
    at ObservableArrayAdministration.notifyArraySplice_ (http://localhost:63386/build/Lens.js:3977:16)
    at ObservableArrayAdministration.spliceWithArray_ (http://localhost:63386/build/Lens.js:3916:58)
    at Proxy.push (http://localhost:63386/build/Lens.js:4102:9)
    at EventEmitter.<anonymous> (http://localhost:63386/build/chunks/5.js:2878:33)
    at EventEmitter.emit (events.js:376:20)
    at Object.onMessage (electron/js2c/renderer_init.js:81:1017)

Hello, I had the same issue because I was not using user impersonation correctly.
I am using Teleport to access my cluster and multiple Kubernetes users linked to my Teleport user.
After setting only one user and loging again, the bug disappeared.
It would be nice to add a feature to Lens (or maybe i didn't see it) to allow user impersonation, in the same way it can be used with the --as option.


WPRDC SQL: Date range

Right now we have a number of datasets (at least the 311 data) that sorts the WPRDC dataset by date and gets the most recent N-number of results. This is not the same as getting all the results from this month--N could be more than this month, or it could be less. Right now we filter out the dates on the client side, but that means that we're either getting too much data from the server (which is a waste), or we're not getting enough data for the month. We need to add to the SQL query that we send some kind of check to make sure we only get the appropriate dates. This will likely go within the dataset object's primaryFiltering property.
We should take also out any limits we have placed, and we should remove the client-side code that check to make sure the data is in the proper range (although we can use that check now to log if we have any extra records while we develop this solution).
This can help you get started: http://stackoverflow.com/questions/14208958/select-data-from-date-range-between-two-dates

@cey10 I noticed you're interested in being a collaborator, but you haven't picked up an Issue yet--would you be interested in taking this one? If you need any help, we can work on it together.

@vonbearshark I would.  I'll look at it and let you know if I have any questions, thanks!

@vonbearshark hey JJ. I think I understand what we're trying to do and how that's accomplished, but I'm not really sure how to implement that into the code that is already here. Could you explain how data is fetched from the server? Is the date limit related to options in the fetchWPRDCData function?

We have this collection of datasets here: https://github.com/Pitt-CSC/PantherView/blob/master/main.js#L224. Each one has a .primaryFiltering field, for example, the city police data: https://github.com/Pitt-CSC/PantherView/blob/master/main.js#L227. This will eventually be concatenated into the SQL part of the URL we send to the WPRDC server, here: https://github.com/Pitt-CSC/PantherView/blob/master/main.js#L349. So you could add the SQL to .primaryFiltering. It means we'll have to duplicate the query for each dataset that needs a date, but that's not too bad, and it helps us prevent asking for a date from datasets that don't need one, because we manually define it.
Once the URL is constructed with the "sql" parameter in it, we send a request to the WPRDC server. They have configured their server to parse and execute the commands we provide via the "sql" parameter.
Does that clarify things?

Hey @vonbearshark, check out PR #123 for my solution to this issue. The arbitrary limitation has been removed, and the SQL is now properly retrieving only the records that were created within the last 30 days. The client-side pruning of the data (i.e. records from the last day, records within the last week) is still in-place.

Commented there, too, but can we remove the limitation on the other sets, too?

Closed by #123


Changes the satis.json generation to match comnposer.json validation and the HTTP status code of webhook responses.

Closed #34

Thanks for this, @Bonscho. I ended up fixing these issues in b73c5a3 and 8d42a31 using a little bit different methodology, allowing the site name and homepage parameters to be specified in config.yml.
Thanks for using Packages!


Support null fields in MappingDocumentDbConverter

Hi there,
Is it possible to support null fields in MappingDocumentDbConverter? I can see in JsonSerializable, it will check the null value and store it when does the mapping.
        if (value == null) {             this.propertyBag.put(propertyName, JSONObject.NULL);         }
So i would expect the same behaviour in MappingDocumentDbConverter? Correct me if I am wrong 😄
Thanks,
Robbie

@rgong1987
Actually, the behavior should be the same as JsonSerializable, the MappingDocumentDbConverter will obtain the value of property( of cause it is null). And pass it to Json.
targetDocument.set(field.getName(), mapToDocumentDBValue(accessor.getProperty(property)))
For example, if one field is null, we may get null here.
mapToDocumentDBValue(accessor.getProperty(property).

@Incarnation-p-lee  ah yes, i got it wrong. It is a bit mess with my groovy class which made the property to be null (the above line)
Thanks for your reply!

@rgong1987
Never mind, and Thank you~

Hi @Incarnation-p-lee ,
Just out of curiosity, is there any reason you are throwing error when property is  null (by Assertion ) instead of ignoring unknown properties?
I know this is not designed for groovy, but it really makes me struggle to bypass some groovy default properties... 😞


Support snforge 0.19 – install universal-sierra-compiler correctly

Cannot find universal-sierra-compiler binary. Make sure you have USC installed https://github.com/software-mansion/universal-sierra-compiler and added to PATH (or set at UNIVERSAL_SIERRA_COMPILER env var): cannot find binary path
https://github.com/CarmineOptions/governance/actions/runs/8299664916/job/22715883695

Hey @tensojka
I see you're using foundry-rs/setup-snfoundry@v2, but a new v3 version came out recently. Could you check if the issue persists after the change?

Now it works. I simply used the snippet from README, so I suggest updating it there. Thanks


regro-cf-autotick-bot erroring with v0.22 update

Comment:
I know you've all been working hard on release automation 🤩, so I just wanted to make you aware that the PyPI-to-conda automation isn't currently working.
Cartopy appears in the conda-forge version updates list under the Errored heading, with this text:
2.00 attempts - The recipe did not change in the version migration, a URL did not hash, or there is jinja2 syntax the bot cannot handle!

Please check the URLs in your recipe with version '0.22.0' to make sure they exist!

We also found the following errors:

 - could not hash URL template 'https://pypi.io/packages/source/C/Cartopy/Cartopy-{{ version }}.tar.gz'

Caropy developers didn't include a .tar.gz file with the latest PyPI release:
https://pypi.org/project/Cartopy/#files

This means we will have to switch to getting source from GitHub, which is not ideal but not that big a deal.

They moved to publishing wheels upstream, maybe they forgot to add the sdist? @trexfeathers can you confirm of that is the case? While we can move to GitHub it would be nice to keep publishing the sdist on PyPI too.

I already made an issue: https://github.com/SciTools/cartopy/issues/2228


removed HRA from titles

Tools Overview page: Remove CCF from all button cards
Tools Overview page: Remove HRA or Human Reference Atlas from button cards
Tools Overview page: Update HRA Organ VR gallery to Organ Gallery in VR
Tools Overview page: Add (APIs) at the end of the APIs button card.
Tools navigation menu: Update HRA Organ VR gallery to Organ Gallery in VR
Tools navigation menu: Update HRA Millitome to Millitome
Tools navigation menu: Update Application Programming Interfaces to Application Programming Interfaces (APIs)
API page: Update the title card's title from Human Reference Atlas Application Programming Interfaces to Application Programming Interfaces (API)
Millitome page: Update the title card's title from Human Reference Atlas Millitome to Millitome
Performed all the above changes

Made all the changes mentioned above.


Code coverage of handlers

Related to https://github.com/icubam/icubam/issues/109, currently www handlers are shown as mostly not being covered by tests,
https://codecov.io/gh/icubam/icubam/tree/520140e5d480e8f2605544530c8ff42e08461294/icubam/www/handlers
aside for the version and static endpoints,

We should add tests to at least render the corresponding pages.
The same applies to BO  https://codecov.io/gh/icubam/icubam/tree/520140e5d480e8f2605544530c8ff42e08461294/icubam/backoffice/handlers
though there it's strange since https://github.com/icubam/icubam/blob/520140e5d480e8f2605544530c8ff42e08461294/icubam/backoffice/server_test.py#L50 should have tested them. I suspect this test is not working as expected, which would also explain some strange behavior I was seeing when testing the /operational-dasboard endpoint https://github.com/icubam/icubam/pull/159#issuecomment-609042864
Either that or tornado do not allow to compute code coverage properly for requests originating in tests.

The test coverage for the BO handlers is super low for sure.

The tests indeed do not work properly. It seems related to authentication, because exposing the endpoints without authentication seems to make the test actually run.
My bet is that there are two problems:

the bo handlers test does not pass authentication credentials correctly
the authentication system returns a 302 in order to redirect to the login page, probably triggering a redirection in the test which then reaches a 200 on the login page and sees it as a success

I'll try to fix the second problem, then the first.

the authentication system returns a 302 in order to redirect to the login page, probably triggering a redirection in the test which then reaches a 200 on the login page and sees it as a success

Ahh, that would explain it! Thanks for investigating.

Maybe we should check that the final response URL is the same as the request URL.

Maybe we should check that the final response URL is the same as the request URL.

It would probably work, but I have a slightly more robust patch, which follows redirections only for tests where it is expected.
So I confirmed the second point above, and fixed it. Now working on the first (fixing authentication).
It could be a good idea to wait that I fix the tests before merging other backoffice-related PRs, since the tests are actually not effective.

Ok, the bo tests should be fixed by #256. Took me a while to figure out how to fix authentication. If everything goes fine, I'll apply the same treatment to www, so that we can add more tests and finally close this issue.

Thanks you @maximedenes ! This will certainly address most of https://github.com/icubam/icubam/issues/197
Could you sync with master to see if CI now passes with /list-tokens fix? For the /operational-dashboard I'll look into it in a separate PR.


Remove manual File.opAssign

The bug requiring it was fixed already and the current implementation violated DIP1000 (but was not detected due to another bug...).

Blocking dlang/dmd#12260

CI failures seem unrelated(?)

Ugh, this is the issue with CI's set up by others that I don't understand.

I tried to restart the build, it won't let me on Travis, have no UI to do so on AppVeyor. Maybe a force-push?

@schveiguy Done

Thanks

I don't need a release for this, right? buildkite works with master?

BuildKite uses the latest tag IIRC.

Tagged a new release.

Thanks!


New Component : List Entry

To Do:

Styling component when there are errors and playing back the error message
Tests
Displaying "Optional"

Closing this PR for now (but keeping the branch for future reference).


show custom crash activity

i added this line into main activity:
throw new RuntimeException("Boom!");
and configure library on application class:
   CaocConfig.Builder.create()
           .backgroundMode(CaocConfig.BACKGROUND_MODE_SILENT) //default: CaocConfig.BACKGROUND_MODE_SHOW_CUSTOM
           .enabled(false)
           .showErrorDetails(false)
           .showRestartButton(false)
           .trackActivities(true)
           .minTimeBetweenCrashesMs(2000)
           .errorDrawable(R.mipmap.ic_launcher_round)
           .restartActivity(null)
           .errorActivity(CrashActivity.class)
           .eventListener(new CustomEventListener())
           .apply();

manifest:
   <activity
       android:name=".Ui.CrashActivity"
       android:configChanges="screenSize|orientation"/>

restartActivity is null here and after launch application that cause of restarting to loop and i don't see crash activity

Hi @MahdiPishguy, you are setting .enabled(false), this disables the crash interception mechanism and is not supposed to be used by the majority of apps. You copied the customization example, but you should only set the parameters you wish to change, not all of them.


Fix InvalidCastException in GetAllElements method

Tests:

Hi @PaulHatch ,
I hope you are doing well! Could you please review this and consider publishing a release on NuGet? Thank you in advance!

Hey @ArminShoeibi ,
Thanks for the contribution. It's been some time since I looked at this since we moved away from using Vault, but I guess the underlying API is now returning a JsonElement rather than something cast-able to string?
There's an opportunity for a bit more pattern matching here:
public IReadOnlyCollection<XElement> GetAllElements()
    {
        try
        {
            var response = RunSync(() => _vault.V1.Secrets.KeyValue.V2.ReadSecretAsync(_path, null, _mountPoint));
            return response.Data.Data.Values.Select(e => e switch
                {
                    string v => XElement.Parse(v), // this line might not be necessary is string is never returned now
                    JsonElement {ValueKind: JsonValueKind.String} j when j.GetString() is var s => XElement.Parse(s),
                    _ => throw new InvalidCastException($"Expected JSON string, but got {e.GetType()}.")
                }).ToList()
                .AsReadOnly();
        }
        catch (VaultApiException e) when (e.StatusCode == 404)
        {
            return Array.Empty<XElement>();
        }
    }

The test here is requiring a running Vault server on localhost, which isn't something (easily) available to the CI service. The test runs as part of the build script, so this will fail and can't be published as is even if merged. A mock could be used instead.

Hey @ArminShoeibi , Thanks for the contribution. It's been some time since I looked at this since we moved away from using Vault, but I guess the underlying API is now returning a JsonElement rather than something cast-able to string?
There's an opportunity for a bit more pattern matching here:
public IReadOnlyCollection<XElement> GetAllElements()
    {
        try
        {
            var response = RunSync(() => _vault.V1.Secrets.KeyValue.V2.ReadSecretAsync(_path, null, _mountPoint));
            return response.Data.Data.Values.Select(e => e switch
                {
                    string v => XElement.Parse(v), // this line might not be necessary is string is never returned now
                    JsonElement {ValueKind: JsonValueKind.String} j when j.GetString() is var s => XElement.Parse(s),
                    _ => throw new InvalidCastException($"Expected JSON string, but got {e.GetType()}.")
                }).ToList()
                .AsReadOnly();
        }
        catch (VaultApiException e) when (e.StatusCode == 404)
        {
            return Array.Empty<XElement>();
        }
    }

The test here is requiring a running Vault server on localhost, which isn't something (easily) available to the CI service. The test runs as part of the build script, so this will fail and can't be published as is even if merged. A mock could be used instead.

You’re right; the API now returns a JsonElement instead of a string.
I appreciate the pattern-matching suggestion I have committed to that change..
Regarding the test, you’re correct that it currently depends on a running Vault server locally, which isn't ideal for CI. Would it be possible to disable this specific test in the CI pipeline? Alternatively, we could explore using TestContainers to mock the Vault server, allowing the test to run without requiring a live instance.

OK, so let's add something like [Fact(Skip = "Requires a running Vault server")] to the test, then it can be released.

OK, so let's add something like [Fact(Skip = "Requires a running Vault server")] to the test, then it can be released.

Thanks @PaulHatch , It's done.


Using BTreeSet instead of BTreemap

Isn't it better to use BTreeSet<(Instant,String)> here ?

That would work too, yes.


[raylib] Disable Default Sanitizers on Debug Builds

Fixes #39996
Follow up to #40100
Sanitizers should be defined in a toolchain file instead.
Tested on a custom toolchain file that enables asan and ubsan on Linux.

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Note #40209

OH, that's neat, do you need help with that PR?

There's also #40209, it also disables the sanitizers and makes other adjustments to the port like devendoring the external header files.
Maybe it should be merged instead...

Good point, I'm going to merge the other PR instead to avoid merge conflicts since I've approved both PRs.

Closed in favor of the other PR.


Revert "Add check against admission webhooks installed in helm integration tests"

Reverts istio/istio#35696
This was breaking post submits. I'll try and investigate it and fix it later.

@jacob-delgado: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
58e28b42cf0d98420bc5e1dda04ef905f8a23b6e
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


System.Numerics Namespace description

Hi,
on page https://docs.microsoft.com/en-us/dotnet/api/?view=netcore-3.1, you can see:

There are commas but no text between the commas.
Cheers
PS: Also here:

Thanks for your feedback. This is a known issue.
Duplicate of #637
/cc @mimisasouvanh


I can't create ServiceAccountStaticAccessKey - No argument or block type is named "name"

apiVersion: iam.yandex-cloud.upjet.crossplane.io/v1alpha1
kind: ServiceAccountStaticAccessKey
metadata:
  annotations:
    crossplane.io/external-name: sa-static-key
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"iam.yandex-cloud.upjet.crossplane.io/v1alpha1","kind":"ServiceAccountStaticAccessKey","metadata":{"annotations":{},"labels":{"testing.upbound.io/example-name":"sa-static-key"},"name":"sa-static-key"},"spec":{"forProvider":{"description":"static access key for object storage","serviceAccountIdRef":{"name":"sa-crossplane"}},"providerConfigRef":{"name":"provider-upjet-yc"}}}
  creationTimestamp: "2024-02-18T16:02:37Z"
  generation: 3
  labels:
    testing.upbound.io/example-name: sa-static-key
  name: sa-static-key
  resourceVersion: "634052479"
  uid: 6a76de49-d45e-401f-85fd-bbf86a73b757
spec:
  deletionPolicy: Delete
  forProvider:
    description: static access key for object storage
    serviceAccountId: {{MY_ID_SA}}
    serviceAccountIdRef:
      name: sa-crossplane
  initProvider: {}
  managementPolicies:
  - '*'
  providerConfigRef:
    name: provider-upjet-yc
status:
  atProvider: {}
  conditions:
  - lastTransitionTime: "2024-02-18T16:02:38Z"
    message: 'observe failed: cannot run refresh: refresh failed: Extraneous JSON
      object property: No argument or block type is named "name".'
    reason: ReconcileError
    status: "False"
    type: Synced

---

apiVersion: iam.yandex-cloud.upjet.crossplane.io/v1alpha1
kind: ServiceAccountStaticAccessKey
metadata:
  labels:
    testing.upbound.io/example-name: sa-static-key
  name: sa-static-key
spec:
  forProvider:
    description: static access key for object storage
    serviceAccountIdRef:
      name: sa-crossplane
  providerConfigRef:
    name: provider-upjet-yc

---

apiVersion: iam.yandex-cloud.upjet.crossplane.io/v1alpha1
kind: ServiceAccount
metadata:
  annotations:
    meta.upbound.io/example-id: iam/v1alpha1/serviceaccount
  labels:
    testing.upbound.io/example-name: sa-crossplane
  name: sa-crossplane
spec:
  forProvider:
    description: service account for crossplane
    folderId: b1guugseh3jmtmgukcs5
  providerConfigRef:
    name: provider-upjet-yc
  writeConnectionSecretToRef:
    name: upjet-creds
    namespace: crossplane-system

Hi, any progress in working on this bug?

Hi. Unfortunately, it took us a little longer to get to work on your issue. The solution has been tested and is in the attached pull request which we will accept and release a new 0.1.6 tag with the fix.
Thanks for your issue, it helps us to be better!


OptionalNumber and PaddingValue are not exposed

Description
When subclassing flet.View, one has to collect and then pass a lot of parameters to the init() method of it in order to not lose any functionality. Thankfully, VSCode is smart enough to create such a statement automatically and even add the required type hints.
Two of the type hints cause problems though: OptionalNumber and PaddingValue are both not found in the flet package. Weirdly enough, the former should be there as it is imported in the __init__.py of flet_core and the __init__.py of flet simply imports everything from flet_core. The latter seems to have been forgotten though.
Screenshot

Flet version (pip show flet):
0.4.0.dev1117

In an even weirder turn of events, OptionalNumber is now suddenly recognized as part of flet. Will make a PR to include PaddingValue as well and then close this.


implement OpenFinex API client

A minimal functionality includes:

authentication
mng_create_order
mng_cancel_order
listen to mng_trade events and trigger the verification and settlement of matches
listen to order events and update the enclave's orderbook mirror (because orders may only be partly executed by a trade / match)

Remarks:

Listening to events should happen in its own thread, spawned at worker startup.
create and cancel order should be performed with minimal latency

Finex engine management (S2S) websocket protocol
Authentication
The management websocket authenticate use standard JWT - RS256
JWT=$(barong-jwt --uid IDBD12DEB15B)
wscat --connect 'ws://localhost:8080/api/v2/finex/ws' -H "Authorization: Bearer ${JWT}"

Example of JWT:
eyJhbGciOiJSUzI1NiIsInR5cCI6IkpXVCJ9.eyJhdWQiOlsicGVhdGlvIiwiYmFyb25nIl0sImVtYWlsIjoiYWRtaW5AYmFyb25nLmlvIiwiZXhwIjoxNjE3OTU2MjU3LCJpYXQiOjE2MTc5NTI2NTcsImlzcyI6ImJhcm9uZyIsImp0aSI6IjE2MTc5NTI2NTciLCJsZXZlbCI6MywicmVmZXJyYWxfaWQiOm51bGwsInJvbGUiOiJhZG1pbiIsInN0YXRlIjoiYWN0aXZlIiwic3ViIjoic2Vzc2lvbiIsInVpZCI6IlU0ODcyMDU4NjMifQ.ROjIs7OhcsX86ZxxVy5vnBCNheEtOENpI9g7eN4BgbGlz5U3XLg7D9cILw8FEsmfzTf5pSzMRrx59_rD9sI4abim7Nn7eJ8gay7E-vQdFfeu6720ZQkbM3w5hIpq2x6pubzUu88ZCVZA2mLEw2XuQFVtDBCEBT5yWCPIOFSsuVN6wSuANvR2ZpJytJHvzSpeGv_9BxZC0ZtAR4PaP2f_iyX1Zw-71QxB4-BxdZeCUp17ic82DQ7TQU_GgYoQNTQFVBHc_oouj_3XB_B1PIYvbSAoPkS1JalqrwJRIw0nFQfTe09OazACyjGyQpsQqsVBN6WlEjYViV30cGquDRATYg

Resources:

Analyse and generate JWT: www.jwt.io
Tool to generate JWT for testing: https://github.com/openware/barong-jwt

Create orders
Method: mng_create_order
Arguments:

№
Name
Data type
Description

1
uid or nickname
string
User UID or nickname to identify the user

2
market_id
string
Market identifier for order ex: "btcusd"

3
market_type
string
Market type ex: "trusted"

3
order_type
string
- market: "m"- limit: "l"- Post only (Must not fill at all or is canceled): "p"- Fill or kill (Must fully match at a given price or is canceled): "f"

4
side
string
Used to specify order side, "buy" or "sell"

5
quantity
string
An amount that placed within the order

6
price
string
Main (limit) price of the order (optional)

Example:
[1,42,"mng_create_order",["14dQ6XGcrk4njhYB7ihcjHyyKbFKUVCXt5vffTV9yAWcgrbu", "btcusd", "trusted", "l", "buy", "0.250000", "9120.00"]]

Response:
[2,42,"mng_create_order",["1245-2345-6798-123123"]]

Error example:
[2,42,"error",["Message describing the error"]]

Cancel orders
Method: mng_cancel_order_bulk
Arguments:

№
Name
Data type
Description

1
uid or nickname
string
User UID or nickname to identify the user

2
market_id
string
Market identifier for order ex: "btcusd"

3
orders IDs or UUIDs
string
List of order IDs or UUIDs to cancel

Example:
[1,42,"mng_cancel_order",["btcusd", ["1245-2345-6798-123123", "1245-2345-6798-123124"]]]

Events
Subscription to events
TODO
Order update events

Argument
Description

Market ID
Market unique identifier

ID
Unique order ID

UUID
Unique order uuid

Side
"buy" or "sell"

Kind
"bid" or "ask"

State
Current state of the order

Type
Order type

Price
Order price

Average Price
Average execution price

Volume	Current
order volume

Origin Volume
Origin volume

Executed Volume
Executed volume

Trades Count	Order
trades count

Timestamp
Order creation timestamp

[4,"order",["btcusd", 2, "7acbbc84-939d-11ea-a827-1831bf9834b0", "buy", "bid", "done", "limit", "1", "1", "0", "1", "1", 1,<PHONE_NUMBER>]]

Trades events

Argument
Description

Market ID
Market unique identifier

ID
Unique trade ID

Price
Trade execution price

Amount
Trade execution amount

Funds
Trade funds, amount * price

Maker OrderID
Maker's trade order ID

Maker OrderUUID
Maker's trade order UUID

Taker OrderID
Taker's trade order ID

Taker OrderUUID
Taker's trade order UUID

Maker Side
Maker order side

Timestamp
Trade timestamp

[4,"mng_trade",["btcusd", 1, "1", "1", "1", 2, "55d78eee-939e-11ea-945f-1831bf9834b0", 3, "55d78eee-939e-11ea-945f-1831bf9834as", "buy",<PHONE_NUMBER>]]

Debug helpers
The following functions are not strictly needed but it may help debugging if we let OpenFinex know about deposit and withdrawal events:
Deposit funds
Method: mng_deposit
Arguments:

№
Name
Data type
Description

1
uid or nickname
string
User UID or nickname to identify the user

2
currency_id
string
Currency identifier of the deposit

3
amount
number
Amount to deposit

3
tx_id
string
Transaction ID (optional)

Example:
[1,42,"mng_deposit",["14dQ6XGcrk4njhYB7ihcjHyyKbFKUVCXt5vffTV9yAWcgrbu", "usd", "1500.000000000000", "0x1234657890123456789"]

Response:
[2,42,"mng_deposit",[124]]

Error example:
[2,42,"error",["Message describing the error"]]

Withdraw funds
Method: mng_withdraw
Arguments:

№
Name
Data type
Description

1
uid or nickname
string
User UID or nickname to identify the user

2
currency_id
string
Currency identifier of the deposit

3
amount
number
Amount to withdraw

3
tx_id
string
Transaction ID (optional)

Example:
[1,43,"mng_withdraw",["14dQ6XGcrk4njhYB7ihcjHyyKbFKUVCXt5vffTV9yAWcgrbu", "usd", "1500", "0x1234657890123456789"]

Response:
[2,43,"mng_withdraw",[124]]

Error example:
[2,43,"error",["Message describing the error"]]

Finally tested it, and a trade event successfully modified the free balances of accounts, hence closing this issue.


Details of getting facebook login to work

So I created an app on facebook.
Copied the key and pasted that in the code.
Turned on authentication for facebook in the Mobile App and checked everything
Ran the code. Clicked Login. And I get

I fixed it partly.. Didn't completely save the authentication piece before. So I login and I do not see errors but nothing happens.. and the breakpoint at OnLoginComplete is never reached

oh. soon as I saved that it just hit the breakpoint..

Also can't find hello api on the callback... Well atleast the CRUD on the highscores table works..

@punkouter

I see you are using 'http://' -> I recommend you use 'https://' instead for mobile app services. But not sure if the auth errors are because of just that?

Also this Unity 'referenced script' is missing error is annoying. I get this too and have added this as issue #15. Remove the broken script from the GameObject and try to reattach to repair the link.

Let me know if any of those 2 pointers fix the issues you encountered. I reckon 'https://' will be needed for Facebook auth as I remember having problems myself with 'http://'

App ID AND Secret? I thought it was only one... Ill look at it again ..

@punkouter will be closing this issue as I have added developer guide link to readme.
http://www.deadlyfingers.net/azure/azure-app-services-for-unity3d/
Please reopen if this is still an issue.


Mini-refactoring of the error functions, following PR #225

Codecov Report

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

@@           Coverage Diff           @@
##           master     #231   +/-   ##
=======================================
  Coverage   74.25%   74.25%
=======================================
  Files           7        7
  Lines         268      268
=======================================
  Hits          199      199
  Misses         64       64
  Partials        5        5

Continue to review full report at Codecov.

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

Check the CONTRIBUTORS.md files, choose one username not two :)

Yeah, that is because in Github I used a different username than in my local box :-| ...fixed now


Operator logs - Version always shows "ver: unknown"

In the dapr-operator logs, the version in log messages always shows as unknown
In what area(s)?
/area operator
What version of Dapr?
1.5.x
Expected Behavior
Version should be accurate
Actual Behavior
Version shows unknown
Steps to Reproduce the Problem
Check dapr-operator logs
Release Note
Fix dapr-operator logging version

Can you attach a screenshot?

It is a common bug, as we only add version info in cmd main.go, like https://github.com/dapr/dapr/blob/master/cmd/operator/main.go#L23-L39,
And define new one in internal, like https://github.com/dapr/dapr/blob/master/pkg/operator/api/api.go#L46.

Those loggers defined at package level will not be set correct version until https://github.com/dapr/kit/blob/main/logger/options.go#L74(including these defined in main package). This func get all loggers and sets the correct version by calling https://github.com/dapr/kit/blob/main/logger/dapr_logger.go#L54.
If we can ensure that all log info is outputed after this function, the version info will be all correct.
But I think it may be better to set the version in logger package at build time, too.

Besides, this function should be called after logger.DaprVersion = version.Version(), but in operator it is called in init func.


fix(chat): name duplicated attached file from empty prefix (Issue #1234)

Description:
Name the duplicated attached files for imported conversations from the empty prefix.
Issues:

Issue #1234

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


Support -fno-exceptions

I am thinking of using this library in a situation where I have to compile with -fno-exceptions. However, this is currently not possible due to the usage of std::stoull in src/ipv4_address.cpp:parse_ipv4_number (and maybe the usage of expected, but I think it has an exception-free variant). For the former issue, std::from_chars could be an alternative to std::stoull.

I like the idea of a no exceptions version of this library, and though I’ve never tested the design should be pretty close to allowing it. However, as you pointed out, unless the dependencies are also possible to compile with exceptions disabled, then I don’t see how to make it work. I may reach out to the author of to::expected.
Your other suggestion is implementable and I would be happy to review a PR.


The icon refresh for MicaForEveryone

Aim at provide a icon matched the Fluent Design

the icon is very nice 😍
but compared to the original, we lost the idea of transparency that MFE adds :(

the icon is very nice 😍 but compared to the original, we lost the idea of transparency that MFE adds :(

Adding transparency is available

wow


Meyda Web Invalid Array Length

Hey. I'm using Meyda in my project modV - love it, great work! But on trying to update modV to use the npm version of Meyda so I can stay up to date, I keep running into:
meyda.js:2412 Uncaught RangeError: Invalid array length
Tracing this back, the variable 'numFilters' is undefined within the function 'createMelFilterBank' and following the traceback, looks like it was never set.
I'm initialising Meyda within modV with:
self.meyda = new Meyda.createMeydaAnalyzer({
	audioContext: aCtx, // AudioContext
	source: microphone, // MediaStreamSource
	bufferSize: 512
});

It fails at this point.
Just for clarification, I'm most definately using the file at "node_modules/meyda/dist/web/meyda.js" (tried both the un-minified and the minified versions)
Let me know if I can help!
Best,
Sam

Hey Sam!
Thanks for letting us know. I am not sure why it's happening off the top of my head but we'll look into it ASAP.
Cheers,
Nevo

Hi Sam, could you paste the traceback if you still have it? What browser were you using?

Google Chrome (Mac) Version 49.0.2623.87 (64-bit)
Here's the traceback, through 4 functions.

Google Chrome (Mac) Version 49.0.2623.87 (64-bit)
Here's the traceback, through 4 functions.

 Can you give us a JSON representation of the state of the self object when the MeydaAnalyzer function defined on line 2195 is called?

Hi Hugh, how would I see the garbage collection level? I didn't realise JavaScript had different levels of Garbage collection. I'll get you the self object asap.

The JSON representation of self on line 2195 is {}.

I was definitely wrong about the GC.
Do you have a branch that you're working on that I can clone? Might be the best way for me to fix this.


Vertical text alignment fix

An attempt has previously been made to fix this in #2938, but I don't think it worked as intended.
This fix has been working well for me an I hope it can help others.

thx for your pr, this fix has been merged to html2canvas-pro


how to prepare dataset？

Thanks for your excellent work.  When I was preparing the data, I encountered a problem.

I downloaded the coco data set according to the instructions of STAC（https://github.com/google-research/ssl_detection）
But file image_info_unlabeled2017.json is missing when running your code（bash tools/dataset/prepare_coco_data.sh conduct）.
Can you help me fixed it?

This file can be downloaded from http://images.cocodataset.org/annotations/image_info_unlabeled2017.zip. I will update the tutorial to make it clear:sweat_smile:.

@MendelXu Thanks for your reply. I fixed the problem. However, I ran into another problem.

can you help me？

I wonder if parameters like 'epoch_length' and so on shold be changed if I use another dataset in coco format instead of coco.Tanks for your help!

Thanks! I understand it now!

Yeah, I solved the bug based on your suggestion! Thanks a lot!


Fixed the naming for interactive disconnected slug

@artdude543, thanks for your PR! By analyzing the blame information on this pull request, we identified @ProbablePrime and @connor4312 to be potential reviewers

lgtm. Thanks!


Added NoopExceptionListener that does nothing when an exception happens

With this class, the configuration exception_listener: Sentry\SentryBundle\EventListener\NoopExceptionListener will disable exceptions being logged by sentry. Uncaught exceptions should still be logged, because they produce a fatal error by default.

This can be closed since the same was done in #247.


Revision suffix validation logic prevents suffixes that start with number

Describe the bug
Use the following command to create a new containerapp or a revision on existing app. Make sure that the revision-suffix starts with a number.
export BLUE_COMMIT_ID=0b699ef
az containerapp create --name $APP_NAME
--environment $APP_ENVIRONMENT_NAME
--resource-group $RESOURCE_GROUP
--image mcr.microsoft.com/k8se/samples/test-app:$BLUE_COMMIT_ID
--revision-suffix $BLUE_COMMIT_ID
--env-vars REVISION_COMMIT_ID=$BLUE_COMMIT_ID
--ingress external
--target-port 80
--revisions-mode multiple

This results in a validation error
Related command
az containerapp create
Errors
Invalid Container App revision name '{value}'. A revision name must consist of lower case alphanumeric characters or '-', start with a letter, end with an alphanumeric character and cannot have '--'.
Issue script & Debug output
N/A
Expected behavior
The command should allow revision suffix that starts with a number
Environment Summary
Latest version
Additional context
No response

Thank you for opening this issue, we will look into it.

PR: #6448

Hi @ruslany
the revision suffix can not start with number:

@ruslany Thanks for reaching out to us and reporting this issue. Since you have filed the PR already which includes the fix. I am assigning this issue to you.

@ruslany This PR has been merged. Closing.


proposal: Go 2: Some proposals for circular import

Author background

Would you consider yourself a novice, intermediate, or experienced Go programmer?
10 year
What other languages do you have experience with?
20 year for C/C++ & JAVA & PHP

Related proposals
The problem of circular import has always bothered me.
I have defined an interface, but the interface refers to another package.
Some of the functions implemented in this package refer to this interface.
So I've been thinking. Since the interface does not involve the implementation.
So is it possible to use a separate type to apply to the interface package to prevent the problem of circular import
an example for this like:
interface anInterfacePackage

import(
    somepackage "packages"
)

type ANewInterface interface{
    PacageFuncA() somepackage.FuncA
}

This must be a separate folder, only interface packages are allowed.
No functional implementations are allowed.
Even if it refers to another package, it tells the compiler that it does not need to import this package anymore.
The interface does not involve implementation so it can be implemented by any struct.
It is the implementation that needs to refer to the specific package.
This allows the interface to be extracted separately, without the need to consider the problem of package conflicts.
Most of the time the role of the interface is not used to solve this problem?
The idea may not be perfect, I hope you discuss it.
A better solution to the circular import problem can be sorted out

Duplicate of #30247


Modify PYTHONHOME configuration not to depend on anaconda version

#99

The CIs warn build failure, however, I guess it is not related to this PR...


feat: Module::add_item

Description
Problem
Resolves #5875
Summary
I'm a bit blocked on this because I'm not sure how to collect something in a module but later elaborate it as if it's inside the function we are currently in.
Additional Context
Documentation
Check one:

[ ] No documentation needed.
[x] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@jfecher I think I'm going to need some help on this one. Here's the code I'm trying it with:
#[run]
mod other {}

struct Foo {

}

fn run(m: Module) {
    m.add_item(
        quote {pub fn get_foo() -> Foo {
            Foo {}
        }}
    );
}

fn main() {
    let foo = other::get_foo();
    println(foo);
}

The issue is that Foo isn't found. I'm trying to elaborate the function as if it's in run, and I'm trying to add it to other. I tried something but it doesn't seem to be working (it's not meant as the definitive code, I was just trying some things).