Error when using fit() with a huggingface model

I am trying to follow the examples for using promptify with huggingface, but I run into this error trying to run the code:

library was installed directly from github and everything from requirements.txt is installed
collab notebook with reproduceable error

I'm facing the same issue.

I have fixed the issue, Please check now :)

Thank you @monk1337. I see a new error popping up as below: ( not sure if need to raise a new issue for this)

The steps to repro are exactly the same as in the initial issue.
Thanks!

Any updates on this? Getting the same error

set max_new_tokens=None
example
model    = HubModel(api_key = "", max_new_tokens=None)

@monk1337
I am getting this error on just loading the model.This was not the case earlier.

@monk1337 I am getting this error.This was not the case earlier.

Similar error
from promptify import Prompter, HubModel, Pipeline

sentence = "are left and right twix the same thing"

model        = HubModel()
prompter     = Prompter('ner.jinja') # select a template or provide custom template
pipe         = Pipeline(prompter , model)

result       = pipe.fit(sentence,
                          domain  = 'all',
                          labels  = None)
print(result)

Error Error in generating prompt: Model not found. Please choose the model from : ['text-davinci-003', 'gpt-3.5-turbo', 'gpt-35-turbo', 'claude-instant-1', 'claude-2', 'command', 'command-light', 'command-medium-beta', 'command-nightly', 'command-xlarge-beta']
Promptify installed by pip3 install git+https://github.com/promptslab/Promptify.git

Same error, same code only model settings differs:
model = HubModel(model="cointegrated/rut5-base", max_new_tokens=None)

Same error, any updates on this?


problem with manual editing

when property 'value' is set input becomes disabled for manual input and i can change date only by picking from calendar. Are there any solution to change date by manual editing and by picking from calendar?
<DatePicker                             locale="ru-ru"                             onChange={e => {                                 this.setState({startDate: e._d})                             }}                             value={this.state.startDate}                         />

use selected= instead of value= to set it


Added warning message about Mbed OS Py3 compatability

New output:
$ mbed new .
[mbed] WARNING: Usage of Python 3 requires Mbed OS >= v5.9
---
[mbed] Creating new program "test" (git)
[mbed] Adding library "mbed-os" from<EMAIL_ADDRESS>at branch/tag "latest"
...

The warning is great.
But can we still catch the traceback error?

@MarceloSalazar The particular traceback error that you hit will actually be resolved once https://github.com/ARMmbed/mbed-os/pull/7092 is merged.
Continuing the issue conversation in here,

This should be ok as quick/initial fix. My main interest is to catch the traceback error and display a message.

In what regard? Tracebacks can occur regardless of Python version.

The particular traceback error that you hit will actually be resolved once ARMmbed/mbed-os#7092 is merged.

I understand 7092 will fix the problem from 5.9+, which is good.
But I believe we'll still hit the error with <=5.8, unless we include a fix in CLI.
Is there anything we could do in CLI to catch the trace back for this particular use case?

Btw, I'm happy to get this PR done and continue the conversation in 686.

@cmonr Could you expand the message to
WARNING: You're using Python 3 which requires Mbed OS 5.9 and above. If you're using Mbed OS 5.8 and earlier versions, you will get Python errors when compiling, testing and exporting.

@cmonr Could we print this message only on an error? that way we don't clutter up perfectly working projects

If it's harder, feel free to clutter things  up for now.

@theotherjimmy Let's get this in as a short-term solution. I'd like to add a mini-patch that detects the version by reusing the functionality from mbed releases and triggers a message if incompatibility is detected.

LGTM! Thanks


Windows 32-bit builds missing

Seems we only have 64-bit builds on Windows, but are missing 32-bit builds. Is that intentional or is this a bug?

Just realized this is intentional, was looking at an old copy of the feedstock. Closing this out.


Received a sentence with an invalid checksum

I have made sure the baud rates of the device and the one in nmea_serial_driver.py are the same (115200). I am using a novatel PwrPak 7. Any help would be appreciated!

I'm getting this too, any solution?

Judging from the error messages, you seem to be getting empty messages. Without more information, I don't think anyone can help you debug this further, but I don't see anything that indicates a bug in the driver software.


Time zones seem to be backwards

I am currently in GMT +1. It shows my time correctly when I select GMT -1.
When we were attempting to use this feature to send times to each other it seems to reverse the GMT +/- for everybody viewing it.

Examples of the issue:
Using the relative time (x minutes ago) we can see where the issue is.

I have input my own time zone and time, and it is giving me odd results.

In this second screenshot you can see it now shows my time.

D'oh! I did some light research and found the identifiers are intentionally inverted in those names:

https://stackoverflow.com/a/39781455
https://gitter.im/moment/moment-timezone?at=5630ed7fe16589782d931772

I'll push an update shortly to map these to the proper labels.

Thank you for raising this issue, the fix is now live. As an added bonus, I sorted all GMT time zones to the start of the list to make them easily selectable by simply scrolling.


New-ADOPSServiceConnection

Create a service connection from Azure DevOps to Azure (required for #2 )

@Hardstl any updates on this? Is there anything I can help with?

Part of the challenge in this is the dynamic nature of the different types of service connections. It would be easier to ensure support a single type of service connection than to support any type of service connection. 🤔


sources: add regression test for issue##1693

There was a regression with the secrets adding of rhsm for the curl source. This was my mistake (sorry!). Here is a regression test that would have prevented this (if we have had it earlier).

Had to rebase because of https://github.com/osbuild/osbuild/pull/1685


Apply the "datatables" plugin to all the tables in Portus

The PR #599 introduced the "datatables" plugin, that allows us to make tables sortable on a specific column. As mentioned in the pull request, we should extend this to every table in Portus.

Closing since this is no longer relevant.


Proposal: fileReplacements generator support

Proposal
configuration.fileReplacements currently supports { replace: string, with: string }.
This proposal wants to add support for { replace: string, withGenerator: string }, where "withGenerator" is the path to the generator file.
The generator file exports a function, which generates the code/content that should replace the file defined in "replace".
Motivation
This would enable more reusability for build-time configuration. The generator could be configured via environment variables. (See #3855 or #4318)
Open Questions

Is this even a feature that should be supported?
Is "withGenerator" an appropriate name for this functionality?
Should *.ts and/or esm be supported as generators?
Should an cli argument like --envVar=ENV_VAR=some_value be supported or should this be left to tools like cross-env?

Implementation Proposal
This would be my first contribution to angular-cli. I'm not overly familiar with the code base, so I will most likely need some guidance, if this proposal is accepted.
For this I'll provide an outline of my proposal implementation:

schema.d.ts: New interface GeneratorFileReplacement and add it to FileReplacement
normalize-file-replacements: Adapt normalizeFileReplacements
build-options.ts: Union type for fileReplacements
typescript.ts: Adapt hostReplacementsPaths to contain with or withGenerator
angular_compiler_plugin.ts: Split normal replacement and generator replacement potentially with a new (delegator) host implementation

The potential host implementation is my big question mark. Would it work to implement a host similar to ResolverHost, that delegates most functionality, but overrides read(path: Path) (and other appropriate methods), which executes the generator?

That's exactly what I need, and according to #4318 it's a highly requested feature.
Please implement this.
Thanks.

Due to shifting priorities I have not yet spent any time on this issue.
I should be able to provide an initial implementation by end of december. However, whether that will be accepted depends on the Angular CLI team.

@kyubisation I'd recommend you to write a design document so that we can review it first; here's a template https://goo.gl/YCQttR.

In my eyes this is no longer necessary. At least my requirements can be met with fileReplacements and the builders API.


Token method of unsubscribing

Would it be possible to add a feature where you can create a token when subscribing to a topic, and use that token to unsubscribe later on? Chaining isn't a good option for me as I unsubscribe at a different point in my app, not when the topic is received. Here's an example of an ideal syntax, showing the added advantage of allowing short arrow functions:

var subscriptions = [];
subscriptions.push(events.on('foo', ()=> {
  // function 1
));
subscriptions.push(events.on('bar', ()=> {
  // function 2
)));

for (var token of subscriptions) {
  events.off(token);
}

Currently the cleanest way to do the same thing is less concise (unless you can see a better way):
var subscriptions = [];

var function1 = ()=> {};
var function2 = ()=> {};
subscriptions.push({type: 'foo', handler: function1});
subscriptions.push({type: 'bar', handler: function2});
events.on('foo', function1);
events.on('bar', function2);

for (var token of subscriptions) {
  events.off(token.type, token.handler);
}

I'm aware there is a wildcard * but this would cancel all subscriptions to that topic, even if there are other parts of the application that might still need the topic (for example subscribers to a scroll event)

you can just unbind them all if you want. Only pass in the type, it will remove all the handlers.


fix(karma-webpack): don't error when webpackMiddleware config not provided

This PR contains a:

[x] bugfix
[ ] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
Error is thrown if no webpackMiddleware. You can see here the change that makes the CI pass again.
Breaking Changes
N/A
Additional Info
Regression introduced in 4.0.0

Perfect thanks. Just need you to sign the CLA so I can merge 😊

Seems to be stuck on the CLA check, @evilebottnawi any ideas?

Sometimes it is buggy, just reopen issue

Nice one!


Packager

In place of #111 should be reviewed after #114

Renamed app to lib
Uses electron-packager.
Tested on Linux and works.
Imports siad from Gopath bin folder on release.
IMPORTANT: Apparently for the icon to work on windows and the packager to work for windows release ran from Linux, wine has to be installed.

definitely looks like an improvement to me :+1:

merge conflicts

@DavidVorick bloop

need to delete resources/default_app from the release
I also think you don't need resources/atom.asar, not 100% sure on that though
You should also include 'siac' in the Sia/ folder, not just 'siad'.
 The linux ones should be put in a '.tar.gz' and everything else should be put in a '.zip'

https://github.com/loopline-systems/electron-builder#how-we-use-it-so-far to make installers is perhaps the next step


housekeeping on troubleshooting.md

add troubleshooting guide (author @zwwhdls)

Codecov Report
Base: 46.69% // Head: 46.69% // No change to project coverage :thumbsup:

Coverage data is based on head (dc8676e) compared to base (b80e312).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #477   +/-   ##
=======================================
  Coverage   46.69%   46.69%
=======================================
  Files          27       27
  Lines        2844     2844
=======================================
  Hits         1328     1328
  Misses       1417     1417
  Partials       99       99

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

经讨论, 结论:

保留单机版, 甚至日后还可以考虑进一步简化(我觉得可以去掉一些 commands reference 内容)
分布式版教程里, 对所有话题都完整描述介绍, 在其他篇章中如有需要引用教程流程中的步骤(比如, 创建文件系统), 也将直接饮用分布式版教程的小节.


Fix payment states of offline payment methods

Fixes #547

@franckc Have added a payment_type and removed the dummy Transaction record for Offline payment. Would you like to take another pass?

We probably need a backfill script for the new column. Adding it

Merging this in and going to deploy to staging/prod


Move jenkins build scripts out of build-script repo

This is completed. New scripts are at https://github.com/spring-cloud/spring-cloud-stream-jenkins-jobs

@mbogoevici please close this issue when the GH webhook for the repo is added.

webhook added.


Add-AzureAccount mentioned in article is not part of Module Az - need MSI installer for Module Azure

[Enter feedback here]
Hi,
Could a link to the Azure Module MSI installer be put somewhere in this article please?
I cannot find this on GitHub, or anywhere else
This module contains Add-AzureAccount (which is not in the Az module)
This command is one of the troubleshooting steps mentioned in the article
I need to install this on a machine that can't get out to the Internet
Many thanks in advance
Cheers,
Matt

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

ID: 0a216181-0864-5435-b27d-75513f4308a2
Version Independent ID: bd4fb194-d6af-e442-5ced-793e09c3fa83
Content: Install Azure PowerShell with PowerShellGet
Content Source: docs-conceptual/azps-5.2.0/install-az-ps.md
Service: azure-powershell
GitHub Login: @mikefrobbins
Microsoft Alias: mirobb

@MattRalph One of the neighboring install pages is Install with MSI, which you can find here:
https://docs.microsoft.com/en-us/powershell/azure/install-az-ps-msi?view=azps-5.2.0
You'll find instructions to get to the install link under Install or update on Windows using the MSI Package which will take you here:
5.2.0 MSI:

Az-Cmdlets-<IP_ADDRESS>762-x64.msi

Az-Cmdlets-<IP_ADDRESS>762-x86.msi

Hi,
That is for the Az module, which does not include the Add-AzureAccount command - screenshot of where this command is used, below, and link to page which gave option to post GitHub feedback
I am after the Azure module
I have this module on my computer, and can see the AddAzureAccount command on that
I'm trying to get an MSI installer for the Azure (not Az) module so I can install it on a server that is not able to access online repositories, I can't update it, install it with PowerShell commmands
Cheers,
Matt
[cid:7d5f6f74-c760-4918-bd88-1b4655f8a3d7]
Troubleshoot Azure Automation runbook issues | Microsoft Docshttps://docs.microsoft.com/en-us/azure/automation/troubleshoot/runbooks
[https://docs.microsoft.com/en-us/media/logos/logo-ms-social.png]https://docs.microsoft.com/en-us/azure/automation/troubleshoot/runbooks
Troubleshoot Azure Automation runbook issues | Microsoft Docshttps://docs.microsoft.com/en-us/azure/automation/troubleshoot/runbooks
In this article. This article describes runbook issues that might occur and how to resolve them. For general information, see Runbook execution in Azure Automation.. Diagnose runbook issues
docs.microsoft.com

From: kobulloc-MSFT<EMAIL_ADDRESS>Sent: Friday, 11 December 2020 7:57 AM
To: MicrosoftDocs/azure-docs-powershell<EMAIL_ADDRESS>Cc: Matt Ralph<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs-powershell] Add-AzureAccount mentioned in article is not part of Module Az - need MSI installer for Module Azure (#1656)
@MattRalphhttps://github.com/MattRalph One of the neighboring install pages is Install with MSI, which you can find here:
https://docs.microsoft.com/en-us/powershell/azure/install-az-ps-msi?view=azps-5.2.0
You'll find instructions to get to the install link under Install or update on Windows using the MSI Packagehttps://docs.microsoft.com/en-us/powershell/azure/install-az-ps-msi?view=azps-5.2.0#install-or-update-on-windows-using-the-msi-package which will take you here:
5.2.0 MSI:

Az-Cmdlets-<IP_ADDRESS>762-x64.msihttps://github.com/Azure/azure-powershell/releases/download/v5.2.0-December2020/Az-Cmdlets-<IP_ADDRESS>762-x64.msi

Az-Cmdlets-<IP_ADDRESS>762-x86.msihttps://github.com/Azure/azure-powershell/releases/download/v5.2.0-December2020/Az-Cmdlets-<IP_ADDRESS>762-x86.msi

[image]https://user-images.githubusercontent.com/37556655/101828952-1dfe6600-3ae7-11eb-8f39-ce753279c596.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs-powershell/issues/1656#issuecomment-742794253, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALRHIG22P7KRLPGC6OP5GHDSUEY2RANCNFSM4UUQ4KBA.

@MattRalph Ah, perfect.  I'm seeing that under step 2 of Resolution.  Let me look into that for you.

@MattRalph The Azure PowerShell Service Management module is only for the classic deployment model. This module allows you to work with your classic Azure deployments that have not been converted to Azure Resource Manager. There is no MSI for the module.
You can use Save-Module to download the Azure PowerShell Service Management module from a system that has access to the Internet. Then copy it to a location in the $env:PSModulePath on the system that does not have Internet access.
Also, notice that the referenced doc gives two different options, one for Azure Service Management and another for Azure Resource Manager depending on whether or not you're using the classic deployment model.
$Cred = Get-Credential
#Using Azure Service Management (Option 1)
Add-AzureAccount –Credential $Cred
#Using Azure Resource Manager (Option 2)
Connect-AzAccount –Credential $Cred

Let us know if this helps and if you need further assistance.


Extension on ColumnBuilder not working

i have many table had user relation so i create extension to add them for more code readable :
extension ColumnBuilder_<Builder, ResultColumn extends Column<ResultDartType>,
    ResultDartType> on ColumnBuilder<Builder, ResultColumn, ResultDartType> {
  Builder addUserRelation({bool nullable = false}) {
    return this.customConstraint(
      '${nullable ? 'NULL' : ''} REFERENCES users(id) ON DELETE CASCADE');
  }
}

and now i use it like that
abstract class UserEntry extends BaseEntity {
  IntColumn get userId => integer().named('user_id').addUserRelation()();
}

and when i delete the user from database all entry with reference on it still in database
but when change it like that work fine
abstract class UserEntry extends BaseEntity {
  IntColumn get userId => integer()
      .named('user_id')
      .customConstraint('REFERENCES users(id) ON DELETE CASCADE')();
}

am I doing it wrong or is there a bug ?

Unfortunately moor does not support that approach. It has to know the table structure at compile time, so it just looks at the source code to see what's going on. It doesn't actually run the column builders, so complicated control flow like extensions won't work.
If all tables referencing User have a column named userId, you can extract that into a mixin:
mixin UserReference on Table {
  IntColumn get userId => integer().named('user_id').customConstraint('NOT NULL REFERENCES users(id) ON DELETE CASCADE')();
}

mixin NullableUserReference on Table {
  IntColumn get userId => integer().nullable().named('user_id').customConstraint('REFERENCES users(id) ON DELETE CASCADE')();
}

Then, you could use abstract class UserEntity extends BaseEntity with UserReference for instance.


feat: accessibility enhancements

Summary
Accessibility enhancements (Overflow,Tags, Tooltip)
ADO Story
Link here (if applicable).
Checklist

[x] Used Conventional Commit messages as outlined in the contributing guide.

[ ] Noted breaking changes (if any).

[x] Documented/updated all props, events, slots, parts, etc with JSDoc.

[x] Ran the analyze command to update Storybook docs.

[x] Added/updated Stories with controls to cover all variants.
[ ] Ran test locally to address any failures.
[ ] Added/updated the Figma link for the Story's Design tab.
[ ] Added any new component exports to the src/index.ts file

:tada: This PR is included in version 1.48.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-next.22 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add .only and .skip modifiers

Please add .only and .skip modifiers.
This would help a lot while debugging or developing new features.
As a work around, currently one can comment out failing or not yet working tests to skip them. But this can create unused imports and these can be removed by formatters automatically. I just try to explain that comment / uncomment is not the same as .skip.

Hey @mrazauskas 🙋🏻♂
About skip, I'm developing this feature allowing conditions to enable it or not. For example:
test.skip(isWindows, ('My Test') => {
  // ...
});

Still thinking about the best way to organize the params.

On the other hand, Poku doesn't map the tests before running them, it just allows the standard JavaScript flow. I need to elaborate an approach to make it possible to use only without compromising this principle.

Regarding the flow. I think, it is enough to define conditional tests:
test("conditional test", () => {
  if (isWindows) {
    return;
  }

  // etc.
});

One can print a message, or not. This works.

What if importing only() (as an alias for describe() and test()) would have some side effect that would make it work? Just an idea. I don’t know how to implement this. It feels like side effect thing.

What if importing only() (as an alias for describe() and test()) would have some side effect that would make it work?

I'll play with that.

All ideas when mapping tests always come back to this problem:
describe(() => {
  console.log('Start');

  it('Test 1', () => {
    // ...
  });

  it.only('Test 2', () => {
    // ...
  });

  it('Test 3', () => {
    // ...
  });

  console.log('End');
});

Output:
Start
End
Test 2

Although it should be:
Start
Test 2
End

I'm thinking something like:
import { describe, test } from 'poku';

describe.modifiers.only.enable();

test('', () => {});

test.only('', () => {}); // ✅

test('', () => {});

That's way, there is no organizeImports conflicts.
It's important to note that if there is no ".only", nothing will be performed (maybe force an error in this case).
The same idea when using .only without explicitly defining the modifier.

Maybe:
describe.modifiers.only.enable();
test.modifiers.only.enable();
it.modifiers.only.enable();

It's also interesting to note that my idea is not to map the tests if these option are passed, but rather to internally ignore the tests that don't come from only.

Hm.. It is easy to forget to remove that line. Also it looks verbose and not really clear why only must be specified separately for describe() and test()?
What about something simple:
import { modifiers } from 'poku';

modifiers({ only: true });

In the other hand, this feature could be only available via test runner. For instance, Node.js test runner has the --test-only flag. Never tried it, but it looks like Bun has something similar: bun test --only.

By the way, take a look at this note: https://bun.sh/blog/bun-v1.1.26#test-only-skips-all-other-tests
I don’t know how that is implemented, but seems like DX got improved in the latest version of Bun.

import { modifiers } from 'poku';

modifiers({ only: true });

That way, both test runner and direct tests would work, but we'd have the same problem by forgetting to remove that line line.

For instance, Node.js test runner has the --test-only flag. Never tried it, but it looks like Bun has something similar: bun test --only.

This approach would only work with the test runner, but despite this, I feel that this is a suitable alternative, even if it doesn't work with the direct test:

No unnecessary code changes
No need to change imports
A temp flag like --only makes sense for a debugging process
It can be used without changing the script, e.g: npm run test -- --only

Also, it would be easy to warn runs without the test runner that .only will be run as a common test (no modifiers).
I'll try to elaborate on that idea first.

I don’t know how that is implemented, but seems like DX got improved in the latest version of Bun.

This is done at Zig level: https://github.com/oven-sh/bun/commit/781998cf003ae4b61adc864c908990825112e037 (from our side, it would be like interpreting the file before executing it).

Just an update:

The skip is available in v2.6.0.
About the .only:
It was quite simple to solve the question of whether to allow .only with the test runner or without it.

I opted to use the --only flag.
If no flags are passed, but there are .only tests, I opted to cause a failure in the test in question.

Feel free to give your opinion 🙋🏻♂

Sounds good to me. Especially the idea about throwing an error, because one can commit some .only tests accidentally, hence getting an error in CI is great thing!


Add internationalization support

The Kiali standalone project hasn't introduced i18 support.
But OpenShift Console has support, and at some point in the plugin lifecycle, this topic should come to the backlog.
Here the recommended approach would be to take a look at what other plugins are doing:
https://github.com/kubevirt-ui/kubevirt-plugin/blob/main/INTERNATIONALIZATION.md
Also, this topic is different for Kiali as in these first steps; this plugin uses a mix of approaches to reuse the Kiali standalone UI plus adding native pages using the dynamic plugin SDK.

Not applicable since internationalization support is going to be implemented in #kiali/kiali


getAuthenticationStatus() stuck on isLoading: true and 0 connectionAttempts on expo / rn

Describe the bug
Like the title says, isAuthenticated, isLoading and connectionAttempts aren't updated.
To Reproduce
I am able to sign up, sign in successfully and through debugging see my user's session.
However when I reload the app and land on my home screen, getAuthenticationStatus() always returns isLoading: true, isAuthenticated: false and connectionAttempts: 0.
Expected behavior
I expect the isAuthenticated, isLoading and connectionAttempts to be updated
Screenshots

Desktop (please complete the following information):

OS: macOS
Browser: Chrome
Version: 110.0.5481.100 (Official Build) (x86_64)

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
I am nhost-js on expo and using expo-secure-store as the clientStorage
export const nhost = new NhostClient({
  authUrl: process.env.AUTH_API,
  graphqlUrl: process.env.GRAPHQL_API,
  storageUrl: process.env.STORAGE_API,
  functionsUrl: process.env.FUNCTIONS_API,
  clientStorageType: "expo-secure-storage",
  clientStorage: SecureStore,
});

That sounds correct. When you reload the page, the first thing that happens is that the JS SDK makes a network request to ensure the user is actually signed in.
And you'll see that as isLoading: true.
Try to create a button that executes the same function and see what it returns.

In React Native, you can use the helper hooks that are designed to react to changes in the authentication state behind the scenes, so you will not need to worry about calling onAuthStateChanged event listeners.
Here's an example for React:
https://docs.nhost.io/reference/react/protecting-routes
This is not exactly the same in React Native, but you get the point.

I'll close this issue. Feel free to open another one if the problem persists.


Export failing

Exporting a module with and without images produces this error. The plugin is installed correctly and the general exportor works fine on this module. This plugin has previously successful worked with this module also. Not sure what is different!
File "shapefile.py", line 47, in
import numpy
ImportError: No module named numpy

This sounds like the server needs an update. Are you using a local or
online server?
Best,
Adela
On Tue, 11 Aug 2020, 05:25 mickmorrison<EMAIL_ADDRESS>wrote:

Exporting a module with and without images produces this error:
File "shapefile.py", line 47, in
import numpy
ImportError: No module named numpy
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FAIMS/kmlExporter/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ7DNLN3BOSBDOY25STPILSAC22VANCNFSM4P2SVKUA
.

Using a server on sack. I probably need to adjust the pip install to account for version of ubuntu. Will deal with it when I can.


Support serializing v4 notebooks

Not sure if this is an API surface area you want to take on, but I was poking around with nbformat for juv and it would be useful.
Usage in: https://github.com/manzt/juv-rs/blob/aa250b57d49cc44a3a83bdd31281f024789f6308/Cargo.toml#L12

Not sure if this is an API surface area you want to take on, but I was poking around with nbformat for juv and it would be useful.

Yes definitely! We'll be using it in Zed.

The only thing I think we'll have to add is serializing outputs and source as multiline strings. We only handle the deserialization of that.


Stash indicator

Hey. Looks awesome.
Can stash indicator be added?
Some prompts just show it's something in git stash and some shows how many stashes created.
This is how it was done in Pure https://github.com/pure-fish/pure/pull/239/files

I considered adding a stash indicator at some point, but since I rarely use the stash, I didn't follow through.
If more people want it, I guess, why not?

If you feel like working on this and sending me a PR, I'll definitely take a look.


Show-NetFirewallRule help contains probably outdated references to Server 2012 in all versions I see

I am told this is the right place to submit this docs issue.  Thanks.
[Enter feedback here]

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

ID: a52ef452-126c-ca4b-a534-e135da614476
Version Independent ID: 317e554b-0edf-bc4d-21d9-a518799b0bcf
Content: Show-NetFirewallRule (NetSecurity)
Content Source: docset/winserver2022-ps/netsecurity/Show-NetFirewallRule.md
Product: w10
GitHub Login: @JasonGerend
Microsoft Alias: **jgerend

Oops:  I had failed to note that Get-NetFirewallRule shows the same issues, unless both are correct as is, which seems unlikely.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


How did you copy the params?

Sorry to bother you again~ After I converted the model to caffe, I tested it and the result is so low.
So I checked the code, and there are some questions there, for example, how did you copy the params in BN layer?
The code is that:
` def _convert_BatchNorm(node,graph,err):
epsilon = node.attrs.get("epsilon", 1e-5)
scale = node.input_tensors[node.inputs[1]]
bias = node.input_tensors[node.inputs[2]]
mean = node.input_tensors[node.inputs[3]]
var = node.input_tensors[node.inputs[4]]
node_name = node.name

input_name = str(node.inputs[0])
output_name = str(node.outputs[0])

if input_name==output_name:
    inplace = True
else:
    inplace = False

bn_layer = myf("BatchNorm", node_name+"_bn",[input_name],[output_name],eps = epsilon, use_global_stats = True, in_place=inplace)
scale_layer = myf("Scale", node_name, [output_name],[output_name],in_place=True,bias_term=True)

graph.channel_dims[output_name] = graph.channel_dims[input_name]

return bn_layer,scale_layer

`
And the "myf" is:
`       def init(self, type_name, layer_name, inputs,outputs, **params):
    self.type_name = type_name
    self.inputs = inputs
    self.outputs = outputs
    self.params = params
    self.layer_name = layer_name
    self.ntop = self.params.get('ntop', 1)
    # use del to make sure kwargs are not double-processed as layer params
    if 'ntop' in self.params:
        del self.params['ntop']
    self.in_place = self.params.get('in_place', False)
    if 'in_place' in self.params:
        del self.params['in_place']

`
I found that there is no any codes to copy the params, so how did you do that?

I have this question because I think maybe the problem is the params in BN and scale layer are given inversely.

you can see the code in https://github.com/MTlab/onnx2caffe/blob/master/onnx2caffe/_weightloader.py#L38-L48

Thanks!


Reserve API Endpoint

@AmineDerbal ! ✋
This PR concerns #28
I think it satisfies the requirements!

You have linters errors


Enforce consistent JSX opening tag spacing

Closing tag spacing is already enforced:
$ echo '<t/>' | standard -
standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  <text>:1:3: A space is required before closing bracket

Let's do the same for the opening tag:
$ echo '<t >hi</t>' | standard -
no error!

Is there a rule for this in eslint-plugin-react?

This will be part of standard v14!


Completion item info often reports types as 'any' when quickinfo on the completed item reports otherwise

Started typing out a variable name, incorrectly got any as the type:

Accepted the completion, got the correct type:

I think this is due to how resolveLocation works; this check seems a bit questionable:
if ((isExpression(parent) || isObjectLiteralMethod(node)) &&
    isContextSensitive(<Expression>parent))

@CyrusNajmabadi has a PR out with #2475 which should fix this.


Cannot get select multiple to work

Hi i've been at this for several hours and cannot get the select to work. Here what i got going so far:
					<lx-select ng-model="Current.status" lx-choices="Choices.status" lx-label="Lorem Ipsum" lx-multiple="true" flex>
						<lx-select-selected>
							{{ $selected }}
						</lx-select-selected>

						<lx-select-choices>
							{{ $choice }}
						</lx-select-choices>
					</lx-select>

var Portal_Main_App = angular
.module('Portal_Main_App',['ngParseExt','ngMaterial','md.data.table','ngMask','ngMaterialDatePicker','summernote', 'lumx'])
.controller('Portal_Main_Controller',LUMX_Controller)
.config( function($mdThemingProvider) {
	$mdThemingProvider.theme('default').primaryPalette('green').accentPalette('blue').warnPalette('red');
});

LUMX_Controller.$inject = ['$scope', '$http','$filter', '$mdDialog', '$q','LxNotificationService'];

function LUMX_Controller($scope, $http, $filter, $mdDialog, $q, LxNotificationService ) {

$scope.Choices = [];
$scope.Current = [];
$scope.Current.status = ["Open"];
$scope.Choices.status = ["N/A","Open","Ongoing","Closed"];

}

Anyone have any ideas?

Thanks @akinjide for answering this old issue (that wasn't on our radar anymore, sorry for that @Owizardo)


Update README.md

Fixed a few grammar errors.

Thank you for contributing to my app! I apologize as I haven't checked my email or GitHub in a couple days. As such, I hadn't seen your pull request until now. I've made a major update to the readme and added the actual application to the repository since you made the pull request. Merging these files is no longer a useful endeavor, however, I will double-check my current readme for grammatical errors. If you find any that I didn't catch, feel free to make another pull request. Thank you!


Hover preview for note ref should only show reference

Please Select if your Request is Either something new or an Enhancement

[x] Enhancement of an existing Feature.
[ ] Request of a new feature.

Please select the area your request applys to. (Multiple selections are Possible)

[ ] Onboard. Initial vault setup and import/export pods
[ ] Create. Note creation, lookup, snippets and templates
[x] Retrieve. Backlinks, references, graph view
[ ] Structure. Refactoring, multi-vault and schemas
[ ] Publish. Sharing your repo with the world
[ ] Misc (Choose this if your not sure)

Is your feature request related to a problem? Please describe

Hover preview currently shows the whole note, should only show the reference area

I haven't seen this being mentioned in release notes, but hover previews work like this already (showing only referenced content).

Confirmed. It's done.


Correct typos

 Issue by Akshar-Patel
Thursday Jun 01, 2017 at 04:16 GMT
Originally opened as https://github.com/android/android-CardView/pull/7

Akshar-Patel included the following code: https://github.com/android/android-CardView/pull/7/commits

 Comment by googlebot
Thursday Jun 01, 2017 at 04:16 GMT

Thanks for your pull request.  It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify.  Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please let us know the company's name.

 Comment by Akshar-Patel
Thursday Jun 01, 2017 at 04:20 GMT

I signed it!

 Comment by googlebot
Thursday Jun 01, 2017 at 04:20 GMT

CLAs look good, thanks!


Auto-upgrade to <IP_ADDRESS>

Base PullRequest
default branch (https://github.com/http4k/http4k-connect/tree/master)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

./upgrade_http4k.sh <IP_ADDRESS>

Changed files

Changed file:

gradle.properties

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Codecov Report

Merging #79 (633ebdb) into master (26515aa) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #79   +/-   ##
=========================================
  Coverage     61.87%   61.87%
  Complexity      631      631
=========================================
  Files           376      376
  Lines          4073     4073
  Branches        277      277
=========================================
  Hits           2520     2520
  Misses         1387     1387
  Partials        166      166

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 26515aa...633ebdb. Read the comment docs.


Ensure the firmware targets that device type and version

when i update nrf5182 BLE，send error Ensure the firmware targets that device type and version。with the nrf tool box。

I had solved it already.


Using my own obj files for training

I have been trying to use my own dataset with meshcnn but I am not able to identity how I can make your code to accept my dataset.
I created a new dataset in the dataset folder it has the same directory structure as the shrec16 dataset but when I run the code after changing the dataroot directory to datasets/anchor_obj in train.sh file, I get the following error:
File "/home/sawaiz/Documents/Lab/Projects/MeshCNN/MeshCNN-master/data/init.py", line 12, in CreateDataset
dataset = ClassificationData(opt)
File "/home/sawaiz/Documents/Lab/Projects/MeshCNN/MeshCNN-master/data/classification_data.py", line 19, in init
self.get_mean_std()
File "/home/sawaiz/Documents/Lab/Projects/MeshCNN/MeshCNN-master/data/base_dataset.py", line 32, in get_mean_std
for i, data in enumerate(self):
File "/home/sawaiz/Documents/Lab/Projects/MeshCNN/MeshCNN-master/data/classification_data.py", line 34, in getitem
edge_features = pad(edge_features, self.opt.ninput_edges)
File "/home/sawaiz/Documents/Lab/Projects/MeshCNN/MeshCNN-master/util/util.py", line 22, in pad
return np.pad(input_arr, pad_width=npad, mode='constant', constant_values=val)
File "/home/sawaiz/.local/lib/python3.10/site-packages/numpy/lib/arraypad.py", line 748, in pad
pad_width = _as_pairs(pad_width, array.ndim, as_index=True)
File "/home/sawaiz/.local/lib/python3.10/site-packages/numpy/lib/arraypad.py", line 518, in _as_pairs
raise ValueError("index can't contain negative values")
ValueError: index can't contain negative values
I feel I have to process my data in a certain way but I am not sure. Can you please clarify how I can do that?
My dataset is a face dataset. I used mediapipe facial landmarker to get the facial landmarks of a face in 3d and then convert the point cloud into a mesh using open3d library. I can access the obj files and see the objects. The number of vertices and faces are variable in all these files.

Hi! I get the same error.And I processed my data,like simplifying to the same faces,but it still didn't work.Were you able to solve the issue?

Nope. I gave up


Use new adin driver in L2 and BCMP

This commit gets hello world to build, but other apps will need similar changes.
Pulls in https://github.com/bristlemouth/bm_core/pull/40 which should be merged first, then this PR updated with the bm_core merge commit before merging this one.
I wanted to get reviewer eyes on this before I go changing all the apps.

OK, so CI is finally passing. I'm trying to run hello world on a dev kit and getting adin SPI errors during init. I'll debug that next week. ✌


'ascii' codec can't encode character '\u201c' in position 7: ordinal not in range(128)

getting this error on MAC system (Apple) based on the code below: -
    def _llama_parser(self, file_path):
        """
        Function for extracting full text from a local PDF using Llama Parser
        """
        parser = LlamaParse(
            api_key=self.api_key,
            result_type="markdown",  # "markdown" and "text" are available
            verbose=True,
            language="en"
        )
        file_extractor = {".pdf": parser}
        documents = SimpleDirectoryReader(input_files=[file_path], file_extractor=file_extractor).load_data()
        return documents[0].text

It seems to be a quote issue. Make sure all your double-quote (") in your code and environment variables are not automatically replaced by the fancy version on Mac (“).


Unable to locate stylesheet

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
16
Description
Hello,
I have an Angular project in version 16 that I've just migrated to version 17.
My problem is during the build, in fact I have 2 warnings coming from the Verona(Template file) : assets/layout files
▲ [WARNING] Unable to locate stylesheet: C:\assets\layout\layout\preloading.css
▲ [WARNING] Unable to locate stylesheet: C:\assets\layout\styles\theme\theme-light\indigo\theme.css

these errors appeared during migration to Angular 17...
I was able to remove the warnings by using the webpack builder proposed on Angular 16 which is @angular-devkit/build-angular:browser and not the one used (recommended) on Angular 17: @angular-devkit/build-angular:application or browser-esbuild.
I checked the Angular configuration files, but I couldn't find a solution.
Do you have any idea where this problem might be coming from?
Relative & Absolute path ?

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.1701.1 (cli-only)
@angular-devkit/core         17.1.1 (cli-only)
@angular-devkit/schematics   17.1.1 (cli-only)
@schematics/angular          17.1.1 (cli-only)

Anything else relevant?
All package in my JSOn file is update

This seems like a bug but we'll need to look at a reproduction to find and fix the problem. Can you setup a minimal repro please?
You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


Rearrange CI config

This PR moves rake internal_investigation from CI into Linting where (IMO) it belongs.

cc @Earlopain

This looks reasonable to me, the cleanup is nice.

One thing I was asking myself while working on CI was if that many jobs for checking that are even needed. They take up a few minutes during runs and if the the only expectation is that the code is formatted correctly then that doesn't need to be confirmed 14 times. Would the test suite not be enough?

I agree that we use the internal_investigation task as a fix of linting and verification on top of the specs. I don’t know if that extra verification is necessary (did it ever catch anything the specs didn’t?) but hopefully someone else will remember.

One thing I was asking myself while working on CI was if that many jobs for checking that are even needed. They take up a few minutes during runs and if the the only expectation is that the code is formatted correctly then that doesn't need to be confirmed 14 times. Would the test suite not be enough? I guess what I'm saying is it either is testing something and should be part of the other file, or it really is just linting and doing that once is ok

Likely on oversight on our part when the CI got migrated in the past. That's now the 3rd CI Platform (Travis, Circle, GHA)...
I agree that things like linting and formatting should be done just once, everything else is just a waste of time.


Add support for changing options

I noticed the chart doesn't redraw when changing the options (for example switching from a stacked to an unstacked bar chart through the stackBars option).
I modified AngularChartistCtrl.prototype.renderChart into
    AngularChartistCtrl.prototype.renderChart = function (element, chartType, chartOptions) {
        if (chartType) {
            this.chartType = chartType;
        }
        if (chartOptions) {
            this.chartOptions = chartOptions;
        }

        return Chartist[this.chartType](element, this.data, this.options, this.responsiveOptions);
    };

and the watch into
                scope.$watch(function () {
                    return {
                        data: scope.data(),
                        chartType: scope.chartType,
                        chartOptions: scope.chartOptions()
                    };
                }, function (newConfig, oldConfig) {
                    console.log('data changed into %o', newConfig);
                    var newData = newConfig.data;
                    var oldData = oldConfig.data;

                    var newChartType = newConfig.chartType;
                    var oldChartType = oldConfig.chartType;

                    var newChartOptions = newConfig.chartOptions;
                    var oldChartOptions = oldConfig.chartOptions;

                    if (newData !== oldData) {
                        chart.update(newData);
                    }

                    if (newChartType !== oldChartType || newChartOptions !== oldChartOptions) {
                        chart = Ctrl.renderChart(elm, newChartType, newChartOptions);
                    }

                }, true);

but now this error occurs sometimes:
TypeError: Cannot read property 'currentOptions' of undefined
    at Line.update (chartist.js:1085)
    at Object.fn (angular-chartist.js:80) —> this is chart.update(newData);
    at Scope.$get.Scope.$digest (angular.js:14225)
    at Scope.$get.Scope.$apply (angular.js:14488)

(which incidentally made me notice — because of the line numbers — that installing angular-chartist.js via npm creates a dependency on chartist 0.5.0 instead of the current 0.7.1)
Do you have any thoughts as to why this happens?

Not sure...There is an outstanding PR that resolves the first issue you are discussing. I asked that it be updated to follow conventions for this repo with no response as of yet. If you are interested in making a PR with the fix, go for it. It is as simple as you made it in your example.
As to the random error, it could just be that it is an out of date dependency. Feel free to make a separate PR for that if you'd like. Otherwise, I will try and get to it this weekend.

The errors don't seem to crop up when updating the chart instead of rerendering it, as shown in @rolandschuetz' solution to this problem. I created a PR #37.

Closed via 3cebded66f73506aa42ea0707a94d556d2cb0662


dry-run support for kudo install ... command

Add dry-run support for kudo install command similar to what is available for kudo init.
What would you like to be added:
That I can run kudo install with a dry-run option to get my hands on the yaml manifest. The command would look like follows
kubectl kudo install kafka --instance=kafka --dry-run -o yaml
Why is this needed:
Its needed to be able to create a helm chart for like kudo-kafka.

This is really needed. I also don't understand why kudo install creates an instance of my operator.

@dahu33 that is an interesting point, could you elaborate on that more? there's probably some docs missing. Basically you need instance for the operator to actually run... you can have multiple instances of the same operator, e.g. kafka in dev, kafka in prod

@alenkacz I'm creating an operator for my app. When I install the operator I do not expect to get an instance, I expect to operator "to be installed in k8s". Then, I can create as many instances I need of my operator with the parameters of my choice. IMO these are two distinct operations that should not be regrouped together in a single CLI command.
Also, I think the kudo CLI should be outputting yaml by default instead of applying them directly to k8s. Not only that approach would be safer but it would also be easier to understand what kudo is doing under the hood.

oooh, I see @dahu33 for that use case, you should run install with --skip-instance ... then next time you run install, only instance will be created. You're right that right now these two are in one command. Could you open an issue explaining your use case so we can have a discussion about separating those? I am not against that at all...
Yes, that is what dry run will be for. There I kind of feel that the default behavior of applying is fine - it's more in line with how kubectl works - there you also by default apply, and only explicitly with flags you get the yamls back. But it's also up for discussion, this is just my personal opinion :)

Duplicates #1492


feat: Make Popper instance available to child context

This allows this kind of usage:
const Demo = ({ close, children }) =>
  <span>
    <Target>👇</Target>
    <Portal>
      <ClickOutside onClickOutside={close}>
        <Popper>
          {({ popper }) =>
            <ResizeAware onResize={popper.scheduleUpdate}>
              {children} 👋
            </ResizeAware>
          }
        </Popper>
      </ClickOutside>
    </Portal>
  </span>

Thanks! Thoughts on changing it to popperInstance or maybe just instance so it's more descriptive?

Yes whatever makes sense to you it's fine.

Talked with my coworkers and instead of passing the instance down. I'd feel better just passing down scheduleUpdate. This way someone doesn't get access to methods like destroy and we can keep those private for React to handle.

I'll get an update out today with this functionality.

Okay cool, that's fine too.


WindowsAuthenticationAzureServiceTokenProvider.GetTokenAsync thows when using Microsoft.IdentityModel.Clients.ActiveDirectory 4+

Original Issue: https://github.com/AzureAD/azure-activedirectory-library-for-dotnet/issues/1283
All our machines are domain joined and corpnet user/group identities are replicated to Azure AD under onmicrosoft.com directory.
We've been using this code:
var azureServiceTokenProvider = new AzureServiceTokenProvider();
var authenticationCallback = new KeyVaultClient.AuthenticationCallback(azureServiceTokenProvider.KeyVaultTokenCallback);
var keyVaultClient = new KeyVaultClient(authenticationCallback);
keyVaultClient.GetSecretsAsync(<url>).Dump(); //this throws

to access KeyVault no problem with these packages:
  <package id="Microsoft.Azure.KeyVault" version="3.0.0" targetFramework="net461" />
  <package id="Microsoft.Azure.KeyVault.WebKey" version="3.0.0" targetFramework="net461" />
  <package id="Microsoft.Azure.Services.AppAuthentication" version="1.0.3" targetFramework="net461" />
  <package id="Microsoft.IdentityModel.Clients.ActiveDirectory" version="3.19.8" targetFramework="net461" />

However we recently had to update to "Microsoft.IdentityModel.Clients.ActiveDirectory" version="4.0.0-preview" and the authentication workflow we have been using stopped working (ActiveDirectory 4.1-preview also fails). se we have to reference Microsoft.IdentityModel.Clients.ActiveDirectory version 4+ in most of our solutions. However Microsoft.Azure.Services.AppAuthetnication references Microsoft.IdentityModel.Clients.ActiveDirectory 3.14.2 which still had the ActiveDirectory.Platforms.dll.
This is the failure:
Parameters: Connectionstring: [No connection string specified], Resource: https://vault.azure.net, Authority: https://login.windows.net/72f988bf-86f1-41af-91ab-2d7cd011db47. Exception Message: Tried to get token using Active Directory Integrated Authentication. Access token could not be acquired. Could not load file or assembly 'Microsoft.IdentityModel.Clients.ActiveDirectory.Platform, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. The system cannot find the file specified.
Even with a binding redirect this fails because the Platforms namespace is now merged into ActiveDirectory.
The simple fix would be to update the Microsoft.IdentityModel.Clients.ActiveDirectory in Azure.Services.AppAuthentication.
PR to fix this issue: https://github.com/Azure/azure-sdk-for-net/pull/5007

(The PR was merged and then reverted)
Issue is still ongoing but I believe there was a conversion offline with yourself and @nonik0. You mentioned it will be addressed in the future when the whole package updates to ADAL 4 or MSAL... I look forward to it.

Whats the status on this? Would like to update the ActiveDirectory package to the latest version instead of keeping it at 3.19.8.

@Ofekw @Zenuka Wanted to let you know that we just released the newest version 1.3.0 of AppAuth, where the ADAL dependency has been updated to 4.3.

Awesome!


Update package.json

I think that one is right.
in scripts,
parcel build = parcel-build

I appreciate this but I've retired devmod-v4, and as such will be no longer updating it. Feel free to look at the current version of devmod.


Check if any more blockchain explorers beside Etherscan for Ethereum mainnet supports the "token" slug

i.e. modify the implementation of RPCServer.etherscanTokenDetailsWebPageURL(for:)if we can move some of the enum cases to be grouped with.main`.
Buzz @oa-s if you need help

Can this be further explained? I'm to look at all the RPC.etherscanWebpageRoot urls and see if they have a method for looking up tokens? Then expand RPC. RPCServer.etherscanTokenDetailsWebPageURL(for:) to include them? @hboon

@eviltofu The implementation for etherscanTokenDetailsWebPageURL(for:) returns 2 different URL formats based on self/RPCServer like this now:
func {
    switch self {
    case .main:
    case many many, the rest here:
    }
}

Roughly speaking, see if we can make it like this:
func {
    switch self {
    case .main, .oneMore, .twoMore:
    case many many, the rest here:
    }
}

This is because the first version (which .main uses) is a better URL — it provides more information — but every chain's blockchain explorer website is slightly different so they might not support it. So it's mostly manual work, print out the list of URLs returned by this function for the 29 (besides .main) chains we support and see if they can be modified to use the first version and modify the switch statement accordingly.


Update the QuickStart section of README.md to indicate how to access the Accumulo shell.

One the user has run the uno commands to install Accumulo and its dependencies add additional information indicating how to access the Accumulo shell.

I've updated the readme with this information on my uno mirror located at https://github.com/jmark99/uno.

Closed by #154


[Test Only][Not Merge] Public Validation

@irushx : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit 4ac6d91:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

azure-sql/index.yml
:x:Error

Details

azure-sql/index.yml

Line 3, Column 14: [Error: yaml-syntax-error] While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@irushx : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit f1984af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

azure-sql/index.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


container/service drop down should have scroll bar when there are too many containers to display

Server version - Build from master - jun/03
When adding services , select "service with name"/"container with name" option in "Scheduling" tab.
Container/service drop down should have scroll bar when there are too many containers to display.

Tested with server version - V0.52.0-rc4
container/service drop down now has scroll bar when there are too many containers/service names  to display in scehduling tab.


Trim repo size

[x] Split repo. Internal tracking URL
[x] move PDFs
[ ] Migrate sample to samples
[x] remove PDF, map, and min files  #9182
[x] Request approval to use VS Code Acrolinx extension in aspnet/AspNetDocs repo
[x] Create announcement in aspnet/Announcements for the repo split

remaining work in #11917


viz server binding to explicit IP

Quoted from @benclifford

i want to open an issue that --external would more conventionally be an explicit IP to bind to, where you might choose <IP_ADDRESS> or you might choose a specific IP address of an interface - that's how lots of other server software works

This has been implemented and merged in #1029


Support for global configuration

Currently all cppcheclipse settings (except binaryPath) are in 'disabled' state by default. So when user creates every new Workspace, he gets clean configuration in Window->Preferences->C++->cppcheclipse->Settings panel.  There's no way to set them to some alternate default state for new Workspaces, because all changes that can be made here will be immediately saved in current workspace's preferences, and you have to reconfigure'em every time 'from scratch'. It would be nice to have possibility to save their default values to configuration/.settings/com.googlecode.cppcheclipse.core.prefs (just like binaryPath variable) with some buttons like e.g. 'save defaults' and 'load defaults', or (maybe even better) to add checkbox like 'load defaults for new workspaces' at main settings screen (Window->Preferences->C++->cppcheclipse)

There's no way to set them to some alternate default state for new Workspaces

What about the pluggin_customization.ini
That's a way to global settings be populate as defaults for new workspaces

Could not get it working even providing global customization file via -pluginCustomization /tmp/cppcheclipse.ini:

com.googlecode.cppcheclipse.core/languageStandardC=c99              #IGNORED
com.googlecode.cppcheclipse.core/binaryPath=/bin/true                     #IGNORED

Also any modifications made to global prefs file (<IDE_ROOT>/configuration/.settings/com.googlecode.cppcheclipse.core.prefs) have no effect (except binaryPath and automaticUpdateCheck values):

automaticUpdateCheck=false    #PROCESSED
binaryPath=/bin/true                   #PROCESSED
languageStandardC=c99            # IGNORED
eclipse.preferences.version=1

These are my customizations inside the plugin_customization.ini file:
## Cppcheck (cppcheclipse) settings
com.googlecode.cppcheclipse.core/binaryPath=${eclipse_home}/../extras/cppcheck/cppcheck.exe
com.googlecode.cppcheclipse.core/automaticUpdateCheck=false
com.googlecode.cppcheclipse.core/automaticUpdateCheckInterval=monthly
com.googlecode.cppcheclipse.core/languageStandardC=c11
com.googlecode.cppcheclipse.core/languageStandardCpp=c++11
com.googlecode.cppcheclipse.core/useInlineSuppressions=true
com.googlecode.cppcheclipse.core/followSystemIncludes=true
com.googlecode.cppcheclipse.core/followUserIncludes=true

And I can setup more things here, I keep it in the same folder as the eclipse.ini file (in the root of the eclipse), and I changed the eclipse.ini with these lines:
-pluginCustomization
plugin_customization.ini

It works very reliably for me.


ENH: special: add _ufuncs._iv_ratio_c function

Reference issue
Toward gh-20253.
What does this implement/fix?
Adds a private function _iv_ratio_c to compute 1-iv(v,x)/iv(v-1,x) for v >= 1 and x >= 0. One of v or x may be inf but not both.
This is complementary to _iv_ratio (gh-20457) but offers better accuracy for large x, where _iv_ratio(v,x) is close to 1.
The purpose of this function is to be used to compute the inverse of _iv_ratio using a univariate root finder accurately when the function value is close to 1. Preliminary results show that this helps to reduce the relative error (of the inverse) from a few hundred machine epsilon to a few machine epsilon.
Algorithm
Following the lines of gh-20457, let $R_\nu(x) := I_\nu(x) / I_{\nu-1}(x)$ for $\nu \ge 0.5$ and $x > 0$. We make use of the Perron continued fraction
$$
R_\nu(x) = \frac{x}{x+\gamma+f(\nu,\gamma;x)}
$$
where $\gamma \in \mathbb R$ is an arbitrary constant and
$$
f(\nu,\gamma;x) = (2\nu -\gamma ) + \frac{-(2\nu+1)x}{2(\nu+x)+1+{}} \frac{-(2\nu+3)x}{2(\nu+x)+2+{}} \frac{-(2\nu+5)x}{2(\nu+x)+3+{}} \cdots
$$
The desired result is
$$
1-R_\nu(x) =\frac{\gamma+f(\nu,\gamma;x)}{x+\gamma+f(\nu,\gamma;x)}
$$
It is shown that the sequence of convergents, $(f_n)_{n=0}^\infty$, monotonically decreases toward $f$, and that
$$
\nu-0.5 <  \gamma+ f < \gamma+ f_n < 2\nu
$$
for all $n \ge 1$. The continued fraction $f$ is evaluated by $f_n$ for some $n \ge 1$ such that $|f_n-f_{n-1}| \le \epsilon |f_n|$, where $\epsilon > 0$ is a given tolerance. The desired result is then evaluated by
$$
1-R_{\nu,n}(x) := \frac{\gamma+f_n}{x+\gamma+f_n}
$$
Truncation error.  When $1-R_\nu(x)$ is evaluated by $1-R_{\nu,n}(x)$, the relative truncation error is
$$
\left| \frac{1-R_{\nu,n}(x)}{1-R_\nu(x)} - 1\right|= \frac{x(f_n-f)}{(x+\gamma+f_n)(\gamma+f)}
$$
When the termination condition is satisfied, it is shown that
$$
\begin{align*}
f_n-f \le \epsilon | f_n| \frac{x}{2\nu-0.5}
\end{align*}
$$
(Note that the above bound is very coarse for large $x$.) Together with the fact that
$$
\frac{x}{x+\gamma+f_n} < \frac{x}{x+\gamma+f}=R
$$
and that
$$
\frac{1}{\gamma+f}=\frac{R}{x(1-R)},
$$
the relative truncation error may be bounded by
$$
\left| \frac{1-R_{\nu,n}(x)}{1-R_\nu(x)} - 1\right| < \frac{R^2}{1-R}\frac{\epsilon |f_n|}{2\nu-0.5}
$$
This bound can grow arbitrarily large as $R \rightarrow 1$, unless $f_n$ converges to $0$ “fast enough”. Thus, unlike _iv_ratio, the accuracy of _iv_ratio_c is not guaranteed even if infinite precision arithmetic were used. We proceed to test the accuracy numerically.

Accuracy. We test the accuracy of _iv_ratio_c numerically for each choice of $\gamma \in \{0, \nu-0.5, 1.5\nu-0.15, 2\nu\}$ and for various choices of $\epsilon$. For each $\gamma$ and $\epsilon$, a few values of $\nu$ are tested; for each $\nu$, 100,000 values of $x$ are chosen such that $R_\nu(x)$ is "approximately" evenly spaced over $(0,1)$. The computed result is then compared against reference values produced by mpmath.
The results show that the choice $\gamma=0$ consistently produces the smallest relative error. The accuracy improves with smaller $\epsilon$ but "flattens" after $2^{-56}$  Therefore, we choose $\gamma=0$ and $\epsilon=2^{-56}$ in this PR.
The following table summarizes the relative error of $1-R_\nu(x)$, computed by the code of this PR. The errors are reported in units of $10^{-16}$. Numbers in parentheses indicate the value of $R_\nu(x)$ around which the peak error is attained.
-------------------------------------------------------------------------
                 Min              Max            Mean     RMS   Max
     v  func     err              err             err     err  eval
-------------------------------------------------------------------------
 0.500  ivr    -3.18 (0.5515)   3.15 (0.5212)   -0.00    0.90    23
 0.500  ivrc   -5.02 (0.7572)   4.63 (0.7472)    0.00    1.05    23
-------------------------------------------------------------------------
 0.501  ivr    -3.17 (0.5514)   3.43 (0.5216)    0.06    0.92    66
 0.501  ivrc  -3215.72 (0.9999) 651.37 (0.9997)   -0.42   23.11    66
-------------------------------------------------------------------------
 1.000  ivr    -3.14 (0.3324)   3.20 (0.5566)   -0.00    0.87    54
 1.000  ivrc   -8.67 (0.9242)   8.99 (0.9202)    0.01    1.49    54
-------------------------------------------------------------------------
 1.500  ivr    -3.00 (0.6172)   2.77 (0.5610)   -0.01    0.77    51
 1.500  ivrc   -5.60 (0.9343)   5.43 (0.8562)    0.02    0.96    51
-------------------------------------------------------------------------
 1.501  ivr    -3.30 (0.6232)   3.32 (0.6146)    0.00    0.79    51
 1.501  ivrc   -5.82 (0.9296)   6.51 (0.9846)    0.01    1.15    51
-------------------------------------------------------------------------
 3.142  ivr    -2.97 (0.5637)   3.20 (0.5531)   -0.01    0.75    43
 3.142  ivrc   -4.71 (0.9855)   5.34 (0.9130)    0.02    0.93    43
-------------------------------------------------------------------------
10.000  ivr    -2.77 (0.0681)   2.74 (0.1278)   -0.01    0.77    32
10.000  ivrc   -4.24 (0.8439)   4.89 (0.8396)    0.01    0.87    32
-------------------------------------------------------------------------
99.990  ivr    -2.94 (0.5050)   2.93 (0.5238)   -0.01    0.74    24
99.990  ivrc   -4.69 (0.9824)   3.58 (0.8696)    0.01    0.84    24
-------------------------------------------------------------------------

The table shows that for $\nu \ge 1$, the relative error of _iv_ratio_c still looks reasonable if $x$ is not too large.
Tail accuracy. For really large $x$, it is too slow to compute the reference answer using mpmath. Instead, we use the asymptotic expansion
$$
R_\nu(x) = 1-\frac{\nu-0.5}{x}+\frac{(\nu-0.5)(\nu-1.5)}{2x^2} + O(x^{-3})
$$
for $\nu > 0.5$, and $R_{0.5}(x) \equiv \tanh(x)$. We test a few large $x$ for each choice of $\nu$. The result is summarized below.
v=0.500 x=1.00e+02 iv_ratio_c=9.41e-20 mp=2.77e-87 rel-err=3.40e+67
v=0.501 x=1.83e+109 iv_ratio_c=5.45e-113 mp=5.45e-113 rel-err=1.11e-13
v=1.000 x=1.83e+112 iv_ratio_c=2.73e-113 mp=2.73e-113 rel-err=4.89e-16
v=1.500 x=1.57e+112 iv_ratio_c=6.36e-113 mp=6.36e-113 rel-err=8.19e-17
v=1.501 x=3.67e+112 iv_ratio_c=2.73e-113 mp=2.73e-113 rel-err=7.98e-17
v=3.142 x=1.45e+113 iv_ratio_c=1.82e-113 mp=1.82e-113 rel-err=2.47e-16
v=10.000 x=1.31e+113 iv_ratio_c=7.27e-113 mp=7.27e-113 rel-err=2.60e-16
v=99.990 x=2.19e+114 iv_ratio_c=4.55e-113 mp=4.55e-113 rel-err=1.85e-16

Asymptotic expansion
From the asymptotic expansion of modified Bessel functions (NIST, 10.40.1, 10.17.1)
$$
I_\nu(z) =\frac{e^z}{\sqrt{2\pi z}} \sum_{k=0}^\infty \frac{\left(\frac12 -\nu\right)_k \left(\frac12+\nu\right)_k }{ 2^k k!} z^{-k}
%(-1)^k a_k(\nu) z^{-k}
$$
where
$$
(a)_k := \prod_{i=0}^{k-1} (a+i)
$$
is the rising factorial, we can find (using the rules of multiplication and division of asymptotic expansion) that
$$
\frac{I_{\mu+1/2}(x)}{I_{\mu-1/2}(x)} = 1-\frac{\mu}{x} +\frac{\mu(\mu-1)}{2x^2}+\frac{\mu(\mu-1)}{2x^3}+O\left(\frac{1}{x^4}\right)
$$
It follows that
$$
1-\frac{I_{\mu+1/2}(x)}{I_{\mu-1/2}(x)} = \frac{\mu}{x} -\frac{\mu(\mu-1)}{2x^2}-\frac{\mu(\mu-1)}{2x^3}+O\left(\frac{1}{x^4}\right)
$$
Two special values that make the series finite are $\mu=0$ ($\nu=0.5$), for which
$$
\frac{I_{0.5}(x)}{I_{-0.5}(x)} = \tanh x, \,
1-\frac{I_{0.5}(x)}{I_{-0.5}(x)}= \frac{2e^{-2x}}{1+e^{-2x}},
$$
and $\mu=1$ ($\nu=1.5$), for which
$$
\frac{I_{1.5}(x)}{I_{0.5}(x)} =\frac{1}{ \tanh x}-\frac{1}{x}, \,
1-\frac{I_{1.5}(x)}{I_{0.5}(x)}= \frac{1}{x}-\frac{2e^{-2x}}{1-e^{-2x}}
$$

Below is an analysis on why the complement of $R_\nu(x) \equiv I_\nu(x)/I_{\nu-1}(x)$ is needed for accurate numerical inversion. Here "numerical inversion" means using an iterative solver (Newton, Brent, bisection, etc) to locate the root by repeatedly evaluating $R_\nu(x)$ at candidate roots.
Intuitively, since $R_\nu(x)$ is very flat for large $x$, many $x$'s would evaluate to the same function value numerically. This makes it impossible to find an accurate numerical inverse by evaluating $R_\nu(x)$ alone.
This idea can be formalized by studying the condition number of $R_\nu(x)$ with respect to $x$. To simplify notations later, denote
$$
r_\nu(x) := R_{\nu+0.5}(x) = I_{\nu+0.5}(x) / I_{\nu-0.5}(x)
$$

I finally had some time to look into this. This is looking pretty good, but I don't think you're going to be able to get away with only using the continued fraction method here. For large x, why not use the asymptotic expansion since you consider it good enough to use for calculating reference values?
I wrote a script that evaluates the large x examples you gave above, I didn't have an issue with calculating the results in mpmath directly using my machine:

import numpy as np
import pandas as pd

from mpmath import mp
from scipy.special._ufuncs import _iv_ratio_c

def mp_iv_ratio_c(v, x):
    with mp.workdps(1000):
        v, x = mp.mpf(v), mp.mpf(x)
        result = float(mp.one - mp.besseli(v, x)/mp.besseli(v - 1, x))
    return result

def iv_ratio_c_asymptotic(v, x):
    return np.where(v != 0.5, (v - 0.5)/x - (v - 0.5)*(v - 1.5)/(2*x*x), 2*np.exp(-2*x))

cases = [
    (0.5, 1.00e2),
    (0.501, 1.83e109),
    (1.0, 1.83e112),
    (1.5, 1.57e112),
    (1.501, 3.57e112),
    (3.142, 1.45e113),
    (10.0, 1.31e113),
    (99.0, 2.19e114),
]

rows = []
for v, x in cases:
    reference = mp_iv_ratio_c(v, x)
    observed = _iv_ratio_c(v, x)
    asymptotic = iv_ratio_c_asymptotic(v, x)
    relative_error1 = abs(reference - observed) / abs(reference)
    relative_error2 = abs(reference - asymptotic) / abs(reference)
    rows.append([v, x, reference, observed, asymptotic, relative_error1, relative_error2])

columns=["v", "x", "reference", "_iv_ratio_c", "asymptotic", "relative_error",
         "relative_error_asymptotic"]

df = pd.DataFrame(rows, columns=columns)
print(df.to_markdown(tablefmt="github", floatfmt=".17g"))

The script generated this table:

v
x
reference
_iv_ratio_c
asymptotic
relative_error
relative_error_asymptotic

0
0.5
100
2.7677930534734751e-87
nan
2.7677930534734751e-87
nan
0

1
0.501
1.8299999999999999e+109
5.4644808743169454e-113
nan
5.4644808743169454e-113
nan
0

2
1
1.83e+112
2.7322404371584698e-113
2.7322404371584698e-113
2.7322404371584698e-113
0
0

3
1.5
1.5699999999999999e+112
6.3694267515923567e-113
6.3694267515923567e-113
6.3694267515923567e-113
0
0

4
1.5009999999999999
3.57e+112
2.8039215686274506e-113
2.8039215686274506e-113
2.8039215686274506e-113
0
0

5
3.1419999999999999
1.4499999999999999e+113
1.8220689655172414e-113
1.8220689655172414e-113
1.8220689655172414e-113
0
0

6
10
1.31e+113
7.2519083969465646e-113
7.2519083969465646e-113
7.2519083969465646e-113
0
0

7
99
2.19e+114
4.4977168949771689e-113
4.4977168949771683e-113
4.4977168949771689e-113
1.2830090845780939e-16
0

The asymptotic expansion works well in all of these cases. You'll need to figure out how to decide between the continued fraction and asymptotic expansion based on the values of $v$ and $x$. You only wrote out how the error term of the asymptotic expansion depends on $x$ for $v$ held constant. I suggest looking also on how it depends on $v$ in order to inform your decision.

Thanks very much for the comments and test script @steppi !
I did take a quick look at the asymptotic formula, but didn’t easily find a suitable cut-off between the asymptotic and continued fraction. It seems to me that the range of “large” but not “extremely large” $x$ is the tricky part!
Let me look into this more closely!

This looks very promising. I cannot comment on the algorithms itself, but could you maybe produce a two dimensional plot of the relative error for logarithmically scaled arguments (something like here)? Color coding would also be fine, 3D plotting can be tricky in matplotlib.

Good idea @dschmitz89 ! Let me try that after I get a satisfactory result for $0.5 < \nu < 1$.
I use the following script to compute the asymptotic expansion:
import sympy
from sympy.series.formal import FormalPowerSeries

t = sympy.Symbol('t')

def a(v,k):
    if k == 0:
        return sympy.Rational(1, 1)
    else:
        half = sympy.Rational(1, 2)
        return sympy.rf(half-v,k)*sympy.rf(half+v,k)/(2**k*sympy.factorial(k))

def iv_fps(v):
    f = sympy.besseli(v, 1/t) / (sympy.exp(1/t) / sympy.sqrt(2*sympy.pi/t))
    k = sympy.Symbol('k', integer=True)
    half = sympy.Rational(1, 2)
    aks = sympy.SeqFormula(a(v,k), (k, 0, sympy.oo))
    tks = sympy.SeqFormula(t**k, (k, 0, sympy.oo))
    return FormalPowerSeries(f, t, 0, 1, (aks, tks, 0))

def ivr_fps(v, n):
    f1 = iv_fps(v)
    f2 = iv_fps(v-1)
    return sympy.collect(sympy.simplify(f2.inverse().truncate(n)*f1.truncate(n)), t)

v = sympy.Symbol('v')
print(sympy.latex(1 - ivr_fps(v, 6)))

Output:
$$
-t \left(\frac{1}{2} - v\right) - t^{2} \left(\frac{v^{2}}{2} - v + \frac{3}{8}\right) - t^{3} \left(\frac{v^{2}}{2} - v + \frac{3}{8}\right) - t^{4} \left(- \frac{v^{4}}{8} + \frac{v^{3}}{2} + \frac{v^{2}}{16} - \frac{9 v}{8} + \frac{63}{128}\right) - t^{5} \left(- \frac{v^{4}}{2} + 2 v^{3} - \frac{5 v^{2}}{4} - \frac{3 v}{2} + \frac{27}{32}\right) + O\left(t^{6}\right)
$$

Good idea @dschmitz89 ! Let me try that after I get a satisfactory result for 0.5<ν<1.
I use the following script to compute the asymptotic expansion:
Output:
−t(1/2−v)−t2(v22−v+38)−t3(v22−v+38)−t4(−v48+v32+v216−9v8+63128)−t5(−v42+2v3−5v24−3v2+2732)+O(t6)
The accuracy is more challenging the closer ν is to 0.5, so I test the asymptotic expansion (up to x−5) together with _iv_ratio_c for v = np.nextafter(0.5, 1), against reference values produced by mpmath using 1000 digits of computation. The result shows there's still a large accuracy "gap" for the range of 1<x<2000, where both _iv_ratio_c and the asymptotic expansion has low accuracy: (relative error in parentheses)

Is the first $1/2 -v$ term maybe the source of the error at $v=1/2$?

Is the first 1/2−v term maybe the source of the error at v=1/2?

I’m not sure, but that $(1/2-\nu)$ is the leading term of the asymptotic expansion, and I evaluate the asymptotic expansion using mpmath to ensure accuracy. I think the expansion is doing a good job, but the continued fraction is inaccurate for $\nu \approx 0.5$ and $x$ as small as $1$, and we can’t hope for an asymptotic expansion at infinity to work for $x =1$!

For $v \approx 0.5$ and small $x$ you can just do (ive(v-1, x) - ive(v, x))/ive(v-1, x)?  I'd suggest trying to find the region where the direct calculation (ive(v-1, x) - ive(v, x))/ive(v-1, x) is most advantageous.

For v≈0.5 and small x you can just do (ive(v-1, x) - ive(v, x))/ive(v-1, x). I'd suggest trying to find the region where the direct calculation (ive(v-1, x) - ive(v, x))/ive(v-1, x) is most advantageous.

I tried out this method, once using scipy.special.ive ("method1"), and once using mp.besseli rounded to float64 ("method2"). The accuracy is slightly improved, but still less than ideal for $\nu \approx 0.5$ and $x$ near 15:

import numpy as np
import scipy.special
from mpmath import mp
mp.dps = 1000

def ivrc_mp(v, x):
    v = mp.mpf(v)
    x = mp.mpf(x)
    return float(1 - mp.besseli(v,x)/mp.besseli(v-1,x))

def ivrc_direct(v, x):
    a = scipy.special.ive(v-1, x)
    b = scipy.special.ive(v, x)
    return (a - b) / a

def ivrc_direct_mp(v, x):
    a = float(mp.besseli(v-1, x))
    b = float(mp.besseli(v, x))
    return (a - b) / a

def compare_methods(v):
    print(f'{v=}')
    x = 2000
    while x > 0.25:
        reference = ivrc_mp(v, x)
        method1 = ivrc_direct(v, x)
        method2 = ivrc_direct_mp(v, x)
        err1 = method1 / reference - 1
        err2 = method2 / reference - 1
        print(f'{x=:8.2f} mp={reference:.2e} '
              f'{method1=:.2e} ({err1:.1e}) '
              f'{method2=:.2e} ({err2:.1e})')
        x /= 2

compare_methods(np.nextafter(0.5, 1))

Output:
v=0.5000000000000001
x= 2000.00 mp=5.55e-20 method1=0.00e+00 (-1.0e+00) method2=nan (nan)
x= 1000.00 mp=1.11e-19 method1=0.00e+00 (-1.0e+00) method2=nan (nan)
x=  500.00 mp=2.22e-19 method1=0.00e+00 (-1.0e+00) method2=0.00e+00 (-1.0e+00)
x=  250.00 mp=4.45e-19 method1=0.00e+00 (-1.0e+00) method2=0.00e+00 (-1.0e+00)
x=  125.00 mp=8.92e-19 method1=0.00e+00 (-1.0e+00) method2=0.00e+00 (-1.0e+00)
x=   62.50 mp=1.79e-18 method1=0.00e+00 (-1.0e+00) method2=0.00e+00 (-1.0e+00)
x=   31.25 mp=3.61e-18 method1=0.00e+00 (-1.0e+00) method2=0.00e+00 (-1.0e+00)
x=   15.62 mp=5.36e-14 method1=5.21e-14 (-2.8e-02) method2=5.36e-14 (2.1e-04)
x=    7.81 mp=3.27e-07 method1=3.27e-07 (-4.4e-09) method2=3.27e-07 (2.0e-11)
x=    3.91 mp=8.09e-04 method1=8.09e-04 (-1.5e-12) method2=8.09e-04 (-5.5e-14)
x=    1.95 mp=3.94e-02 method1=3.94e-02 (-9.3e-15) method2=3.94e-02 (1.3e-15)
x=    0.98 mp=2.48e-01 method1=2.48e-01 (-2.3e-15) method2=2.48e-01 (-1.1e-16)
x=    0.49 mp=5.47e-01 method1=5.47e-01 (-5.6e-16) method2=5.47e-01 (0.0e+00)

It seems the reason is that $I_\nu(x)$ and $I_{\nu-1}(x)$ are very close, such that their difference is orders of magnitude smaller than their value, so even if their value is evaluated to high accuracy, there's not enough precision left for their difference.
It seems (an accurate implementation of) logiv could help here!

I also tried a "brute force" method, where I expand the $n$-th convergent of the continued fraction symbolically into a rational function ($n$-degree polynomial in $x$ over $n$-degree polynomial in $x$), and then evaluate the rational function numerically. This improves accuracy because the rational turns out to comprise only positive terms, so it's free of cancellation error. Even so, as many as 100 terms are needed to achieve an accuracy of a few epsilon.

import numpy as np
import sympy
from mpmath import mp
mp.dps = 1000

def fn_rat(v, x, n):
    """Return the expanded rational function of f_n(v, x)"""
    q = sympy.poly(1*x)
    p = q - q
    for k in range(n, 0, -1):
        a = sympy.poly(-(2*v+(2*k-1))*x, x)
        b = sympy.poly(2*v+k + 2*x, x)
        p, q = a*q, b*q+p
    p = 2*v*q + p
    return p, q

u = sympy.Symbol('mu')
x = sympy.Symbol('x')
p, q = fn_rat(u+sympy.Rational(1,2), x, 100)

def ivrc_rat(v, x):
    mu = v - 0.5
    fn_expr = str(p / q)
    fn_val = eval(fn_expr)
    return fn_val / (x + fn_val)

def compare_methods(v, func1, func2):
    print(f'{v=}')
    x = 250
    while x > 0.25:
        reference = ivrc_mp(v, x)
        val1 = func1(v, x)
        val2 = func2(v, x)
        err1 = val1 / reference - 1
        err2 = val2 / reference - 1
        print(f'{x=:8.2f} mp={reference:.2e} '
              f'{val1=:.2e} ({err1:.1e}) '
              f'{val2=:.2e} ({err2:.1e})')
        x /= 2

compare_methods(np.nextafter(0.5, 1), ivrc_direct, ivrc_rat)

Output: ("val2" is from rational expansion; relative error in parentheses)
v=0.5000000000000001
x=  250.00 mp=4.45e-19 val1=0.00e+00 (-1.0e+00) val2=4.45e-19 (-2.2e-16)
x=  125.00 mp=8.92e-19 val1=0.00e+00 (-1.0e+00) val2=8.92e-19 (4.4e-16)
x=   62.50 mp=1.79e-18 val1=0.00e+00 (-1.0e+00) val2=1.79e-18 (-6.7e-16)
x=   31.25 mp=3.61e-18 val1=0.00e+00 (-1.0e+00) val2=3.61e-18 (6.7e-16)
x=   15.62 mp=5.36e-14 val1=5.21e-14 (-2.8e-02) val2=5.36e-14 (-5.6e-16)
x=    7.81 mp=3.27e-07 val1=3.27e-07 (-4.4e-09) val2=3.27e-07 (-1.1e-16)
x=    3.91 mp=8.09e-04 val1=8.09e-04 (-1.5e-12) val2=8.09e-04 (-1.1e-16)
x=    1.95 mp=3.94e-02 val1=3.94e-02 (-9.3e-15) val2=3.94e-02 (0.0e+00)
x=    0.98 mp=2.48e-01 val1=2.48e-01 (-2.3e-15) val2=2.48e-01 (-1.1e-16)
x=    0.49 mp=5.47e-01 val1=5.47e-01 (-5.6e-16) val2=5.47e-01 (-2.2e-16)

After various experiments it appears to me that for _iv_ratio_c to achieve relative error below $2.22\times 10^{-16}$ for all $\nu > 0.5$ and $x > 0$, I have to either compute the continued fraction using double-double arithmetic, or symbolically expand the continued fraction and evaluate the (very large) rational function using double precision. The latter approach is rather formidable, so this leaves only the double-double arithmetic a viable solution.
(The most difficult input is v = np.nextafter(0.5, 1) and x = 26 (approximately). It requires 105 terms to achieve the desired accuracy.)
Now there appears to me two options:

Restrict the domain of $\nu$ to $\nu \ge 1$. This is the domain of $\nu$ relevant for the vonmises-fisher distribution. For these $\nu$, evaluating the continued fraction with double precision is sufficient to achieve the desired accuracy for _iv_ratio_c.

Restrict the domain of $\nu$ to $\nu \ge 0.5$, and use double-double precision to evaluate the continued fraction of _iv_ratio_c for $0.5 < \nu \le 1$, and double precision for $\nu \ge 1$. A small problem with this choice is that I don’t seem to readily find a double-double implementation in SciPy. Is it available somewhere? Or shall I
adapt from some external library?

Which choice would you prefer? @steppi
And @dschmitz89 , I have not forgotten your suggestion to plot an error graph by values of $\nu$ and $x$. I will do that after the implementation for small $\nu$ is settled.

Thanks for all of the effort you've put into this @fancidev.

Which choice would you prefer? @steppi

I think it's fine to restrict to $\nu >= 1$, since the function is private, and that's all that's needed for the vonmises-fisher distribution. If we ever want to make this public, we should try to expand the domain though.
double-double arithmetic is in SciPy, but is kind of well hidden. I's included within Cephes (which has been integrated into the header only C++ library we're working on.). It's in this header file: dd_real.h, and here's an example where it is used, https://github.com/scipy/scipy/blob/2050d49e6b1fd512bd7eba35d3368bb5678bb7f2/scipy/special/special/cephes/struve.h#L179-L182.

And @dschmitz89 , I have not forgotten your suggestion to plot an error graph by values of

My understanding is that @dschmitz89, wanted to see this plot for different methods to help settle the choice of implementation. If we restrict to $\nu >= 1$ it's no longer necessary though.

Thanks for the comments @steppi ! Indeed dd_real is the “external library” I was referring to; glad to see it’s already there!
Extending the domain is an interesting exercise for me. Let me try the double-double arithmetic, possibly combined with asymptotic expansion. If it works, I’ll also take a look at whether it is possible/easy to extend the domain to $\nu > 0$ or even $\nu \in \mathbb{R}$ with sufficient accuracy.

@fancidev : just wanted to mention that scipy.special does not require and cannot achieve machine precision for every function in every domain, especially for non canonical ones like this. What you are doing here is a substantial improvement over what we have so far, so if double-double precision does not work out that is not too bad. :)

The double_double from cephes works nicely! And iv_ratio_c can achieve machine precision with double-double arithmetic (for v < 1).
There are still a few pending points to update:

Update doc string with extended domain
Add test cases for parameters in extended domain (especially the difficult ones)
(Optional) Investigate the accuracy for v < 0.5
(Optional) Implement an accurate fma for double_double

I made a simple notebook to display the graph of $I_\nu(x)/I_{\nu-1}(x)$ for real $\nu$ and $x$:
https://gist.github.com/fancidev/503dfa3a3733834db1b981292d0fca0b

@fancidev, I've found that the double_double arithmetic currently doesn't compile in NVRTC so I've removed the iv_ratio test case from text_xsf_cuda.py here https://github.com/scipy/scipy/pull/21210/commits/124f12e2c6859817f438d7f4af81855df0831c79. It's not really an issue since we don't use iv_ratio on CUDA yet anyway, and the plan is to macroize out the double_double arithmetic on CUDA.

Thanks for reviewing and all the great ideas and suggestions @steppi !

@fancidev, I've found that the double_double arithmetic currently doesn't compile in NVRTC so I've removed the iv_ratio test case from text_xsf_cuda.py here 124f12e. It's not really an issue since we don't use iv_ratio on CUDA yet anyway, and the plan is to macroize out the double_double arithmetic on CUDA.

I’d like to help make it work but I don’t have an NVidia graphics card to test it. Or maybe it is possible to “cross compile”? Let me check that. It seems to me there should be a way to get double_double to work on GPU as it’s not doing anything funky.

I’d like to help make it work but I don’t have an NVidia graphics card to test it. Or maybe it is possible to “cross compile”? Let me check that. It seems to me there should be a way to get double_double to work on GPU as it’s not doing anything funky.

Yes, there should be a way to get it to work. I expect the double_double arithmetic wouldn’t be very efficient on GPU, but it might be worth testing out. Thanks for the offer to help. I’ll see what options there are for you to test things on GPU.


First automated smoke test using Docker setup

There is currently no automated integration test setup in the Leda Distro build workflow.
By running qemu and the smoketests (in tests/), we can run some simple first tests whether the system boots up and the services are started.
By using the docker-compose setup, we can also run robot tests in a system-test, which includes the download of update bundles from a "remote" location. See https://github.com/eclipse-leda/leda-distro/tree/main/resources/docker

With the merge of the docker-snapshot pull request (#44), there is now also a dockerized Robot Framework test setup available.
See https://eclipse-leda.github.io/leda/docs/build/tests/ for documentation.


add readme to dockerhub

https://hub.docker.com/r/tremor/tremor has no readme, @Agares pointed out it would help people.
from https://github.com/wayfair-tremor/tremor-www-docs/issues/9

Would you mind also adding more details on the dockerhub page? Currently the overview is empty, would be great to at least have information about the distro that the image is based on :)

We should add one

Added a readme https://hub.docker.com/repository/docker/tremorproject/tremor

Thank you! Would you mind also documenting where does the Dockerfile live? I assume it's this one: https://github.com/wayfair-tremor/tremor-runtime/blob/master/Dockerfile but that's only a guess :)

That's an excellent idea! I didn't think of that :) I added it!


Fix mikrotik detecting capsman support

Proposed change

Type of change

[ ] Dependency upgrade
[x] Bugfix (non-breaking change which fixes an issue)
[ ] New integration (thank you!)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #31749
This PR is related to issue:
Link to documentation pull request:

Checklist

[X] The code change is tested and works locally.
[X] Local tests pass. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.
[ ] I have followed the development checklist
[ ] The code has been formatted using Black (black --fast homeassistant tests)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly.
Updated and included derived files by running: python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt.
Updated by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

The integration reached or maintains the following Integration Quality Scale:

[ ] No score or internal
[ ] 🥈 Silver
[ ] 🥇 Gold
[ ] 🏆 Platinum

@engrbm87 ok?

@engrbm87 ok?

Approved.


Hovering on imported module shows path from TypeScript folder.

VSCode Version: 1.46.1
OS Version: Windows 10

Steps to Reproduce:

Require modules like yargs, validate in a nodejs project. (Installed in project folder)
On hover, TypeScripts node_modules path is shown instead of project's node_modules folder.
Packages not in TypeScript's node_modules folder has no such issue.

Hovering on yargs:

Hovering on chalk:

Does this issue occur when all extensions are disabled?: Yes

On hover, TypeScripts node_modules path is shown instead of project's node_modules folder.

TS shows actual path to file the provides type info for that module. Chalk has built-in types.
On the other hand yargs doesn't have types in package.json, therefore it you have @types/yargs in your package.json or even in node_modules definitions will be used with ATA.
To provide suggestions etc. for npm modules that don't provide own types or don't have corresponding @types package in your project node_modules TS (used by VS Code for both JS and TS) will download types packages and provide info for them.
See Typings and Automatic Type Acquisition

@IllusionMH The version of the types for yargs is 15.0.5 but the version I'm using is 12.0.2. Is this going to cause any problems if they made changes to the API?


Async conn events

[x] fix failing tests deadlock

Tests passing with a workaround... going to not merge this by itself, but instead along with the state update because there is a chance this is strictly less correct as-is. With the design of the coming state work, the incorrectness should be impossible.

The work here was originally included in https://github.com/holochain/tx5/pull/77 - but largely had to be removed because of the go pion deadlock when closing things during event callbacks.


Devolutions.RemoteDesktopManager version 2<IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Updated version of Remote Desktop Manager Enterprise

OK I'll fix that immediately.

Please use wingetcrate update -i Devolutions.RemoteDesktopManager -v 2<IP_ADDRESS> -u https://cdn.devolutions.net/download/Setup.RemoteDesktopManager.2<IP_ADDRESS>.msi when there is a existing package.

@OfficialEsco Should I create new PR when I use command with correct syntax?

Please use wingetcrate update -i Devolutions.RemoteDesktopManager -v 2<IP_ADDRESS> -u https://cdn.devolutions.net/download/Setup.RemoteDesktopManager.2<IP_ADDRESS>.msi when there is a existing package.

@OfficialEsco Should I create new PR when I use command with correct syntax?

That would probably be easier yes

OK, I'll do that, this is first time I tried to updated package in repo so that's why I did this mistake.

OK, I'll do that, this is first time I tried to updated package in repo so that's why I did this mistake.

Understandable :)
When doing wingetcreate new make sure the program you are about to add does not exist, search with both winget search and on GitHub.
If the package exist copy the PackageIdentifier and do wingetcreate update -i "PackageIdentifier" -v "version" -u "url"

OK, fixed version of manifest from this PR is in new PR with number #16216