Allow for a configurable strategy for granting refresh_token

We should consider allowing for a configurable strategy for granting refresh_token.
The solution should take into consideration for allowing refresh_token to be granted to public clients, but would be the responsibility of the consuming application to configure the strategy.
Related gh-297

@jgrandja , I was able to get refresh tokens for a public client as shown in your test case.
However, I was not able to use them afterwards:
I call the token endpoint with the refresh token and wiht grant_type "refresh_token", but I get an InsufficientAuthenticationException ("Full authentication is required to access this resource").
During debugging I have seen so far, that OAuth2TokenEndpointFilter and OAuth2RefreshTokenAuthenticationConverter are not even called at all.
My guess is that the reason is the missing OAuth2ClientAuthenticationToken, since none of the AuthenticationConverters applies here, neither ClientSecretBasicAuthenticationConverter, nor ClientSecretPostAuthenticationConverter. And especially not PublicClientAuthenticationConverter , since it is only for PKCE. But this is not applicable when sending a refresh_token to the token endpoint, but only for authorization_code grant. Likewise, an according AuthenticationProvider is missing, since PublicClientAuthenticationProvider only works for PKCE again.
So, the AuthorizationFilter fails due to missing authentication, and OAuth2TokenEndpointFilter, which is afterwards, is never reached.
There are extension points for both, AuthenticationConverters and AuthenticationProviders. So, would it be the right way to write a converter that looks up the client by the refresh token and then a provider that authenticates the public client by that?
I guess that could work, but I am a bit afraid to open some new security holes here.
Update:
I haved solved it as pointed out above, by a custom AuthenticationConverter and AuthenticationProvider, which are configured like this:
authorizationServerConfigurer.clientAuthentication(ca -> {
            ca.authenticationConverters(converters ->
                     converters.add(new PublicClientRefreshTokenAuthenticationConverter()));
            ca.authenticationProviders(providers -> PublicClientRefreshTokenSupportingAuthenticationProvider
                    .createAndInsert(providers, registeredClientRepository, authorizationService));
        });

The converter looks for "refresh_token" grant and checks the refresh token parameter. If both are present, an unauthenticated OAuth2ClientAuthenticationToken is created. I had to tweak a bit here: Since a client id is required, I provide some fake id not used by real clients.
The authentication provider looks for ClientAuthenticationMethod.NONE and for the fake id. Then it uses the OAuth2AuthorizationService to look up the OAuth2Authorization by the refresh token. Then, it gets the registered client from it. Finally, it creates an authenticated OAuth2ClientAuthenticationToken with the registered client.

@stefanocke PublicClientRefreshTokenSupportingAuthenticationProvider is not needed as you can (and should) leverage the existing OAuth2RefreshTokenAuthenticationProvider. You only need to provide a custom AuthenticationConverter that ultimately returns an OAuth2ClientAuthenticationToken instance where OAuth2ClientAuthenticationToken.isAuthenticated() is true. You can see a similar implementation here.

@jgrandja , thanks for your feedback. I am not sure I understand you instructions.

You say, I need an authenticated OAuth2ClientAuthenticationToken.
For this, I need the RegisteredClient.
In you example, the RegisteredClient is determined by a DeviceClientAuthenticationProvider
Very similar to this, I have the PublicClientRefreshTokenSupportingAuthenticationProvider, which also determines the RegisteredClient and creates an authenticated OAuth2ClientAuthenticationToken.
The OAuth2RefreshTokenAuthenticationProvider is still used later, but not for client authentication, but for authenticating the resource owner (like in all other cases)
Of course I could try to summarize my Provider and Converter into a Converter that immediately gives an authenticated OAuth2ClientAuthenticationToken. But is this really the right way considering that in all other cases and even in your example it is separated?

@stefanocke

I have no client id, but a refresh token, which was issued in the context of a certain client.

See also https://datatracker.ietf.org/doc/html/rfc6749#page-47 : There is no clientId parameter here in the request. So if there is no client auth, we can only determine the client by the refresh token.

Even for public (unauthenticated) clients, the client_id parameter MUST be sent to identify the calling client. The client_id parameter will then be compared to client_id associated to the refresh_token, to prevent token replay.
In section 6, Refreshing an Access Token:

If the client type is confidential or
the client was issued client credentials (or assigned other
authentication requirements), the client MUST authenticate with the
authorization server as described in Section 3.2.1.

Although, the spec does not state anything about public clients, the token endpoint is a protected resource and therefore requires some form of client authentication. For public clients, then client_id is required at a minimum.

@jgrandja , in Section 3.2.1. we have:

A client MAY use the "client_id" request parameter to identify itself
when sending requests to the token endpoint.  In the
"authorization_code" "grant_type" request to the token endpoint, an
unauthenticated client MUST send its "client_id" to prevent itself
from inadvertently accepting a code intended for a client with a
different "client_id".

IMHO MAY != MUST, so I can not rely on the "client_id" request parameter.  (It is only MUST for "authorization_code" grant type, but not for "refresh_token")
Of course, if it is there, I should use it. But otherwise, I have only the refresh token to identify the client.
But isn't this quite independent from the question regarding separating the logic into a Coverter and Provider? The Converter just does the HTTP request parameter extraction and is dumb otherwise. The Provider looks up the RegisteredClient, like in your example.

@jgrandja I have a similar scenario and wanted to create a new access token for a public client using a refresh token. Now I have taken the following example from you:

PublicClientRefreshTokenSupportingAuthenticationProvider is not needed as you can (and should) leverage the existing OAuth2RefreshTokenAuthenticationProvider. You only need to provide a custom AuthenticationConverter that ultimately returns an OAuth2ClientAuthenticationToken instance where OAuth2ClientAuthenticationToken.isAuthenticated() is true. You can see a similar implementation here.

I have integrated a self-implemented AuthenticationConverter and set the existing OAuth2RefreshTokenAuthenticationProvider as the provider, but the scenario does not work in this case.
The self-implemented converter returns an OAuth2ClientAuthenticationToken (as specified in your example), but the provider class OAuth2RefreshTokenAuthenticationProvider only supports the following type of token: OAuth2RefreshTokenAuthenticationToken. The problem is that the provider is not pulled in the ProviderManager due to the above-mentioned circumstance.
Here is the code from the OAuth2RefreshTokenAuthenticationProvider class (254-256).
@Override
public boolean supports(Class<?> authentication) {
   return OAuth2RefreshTokenAuthenticationToken.class.isAssignableFrom(authentication);
}

If I am making a mistake, I would be grateful for any help. Possibly the topic could have been discussed at Stackoverflow, but I thought that it might fit to this issue, since I take up the topic.
Thanks for the help or information.

@stefanocke @TryAndErrorBot @mberwanger
See this comment


No response received by Bot Framework Emulator V 3.5.25

I experience a problem with the following setup. It might also be a problem with the Bot Connector or VS template:
.Net Microosft.Bot.Connector package 1.1.0
Bot Emulator 3.5.25
Visual Studio 2015 and the Bot Template
When trying to use a new Visual Studio .Net Bot Application Project with the emulator, your message gets posted from the emulator to the default controller of your application but the Message.Type is "message" rather than "Message" as expected by the template
if (message.Type == "Message")
{
                // calculate something for us to return
                int length = (message.Text ?? string.Empty).Length;

                // return our reply to the user
                return message.CreateReplyMessage($"You sent {length} characters");
}

After changing this the message gets processed and a reply is created. But this reply is not received by the emulator. When checking the message the reply message just has the user information:
Has the message format been changed or is there some information missing from the message being send by the emulator?

Solution: Manually update the Visual Studio Bot Application template to the latest version by downloading from http://aka.ms/bf-bc-vstemplate


Use snapkit with swift2.2 - renamed debug identifiers

@ChristopheFernandes was this intentional? Renaming them to this would be a regression as it is now supposed to be #file and #line?

@robertjpayne sorry i wanted to PR on my repo :(

@ChristopheFernandes no worries! Happens more often than you probably think! 👍


Cleanly sectioned codes

OH1879 and UT1884 are now cleanly sectioned and can be removed from the fuzzy sectioning function for next time we run this. This shouldn't affect the visualizations you're working on right now--no need to re-hash.

Done.


AppLayout in mobile view does not display side navigation as expected

Describe the bug
When using the AppLayout component and viewing on a mobile device or small screen size, there are a number of issues.

The side navigation is displayed when first landing on the page (it should be hidden on a small screen)
The side navigation is hidden behind the top header bar - this causes the close icon hidden and header text in the side navigation to be hidden
Clicking on a link the side navigation keeps the side navigation open - clicking on a link does not take the user to the screen they expect to see

Versions
Which version of aws-northstar npm package is installed?
@aws-northstar/ui 1.1.2
To Reproduce
Please check code sandbox here:
https://codesandbox.io/s/strange-ritchie-8j5ssq?file=/src/App.tsx&resolutionWidth=320&resolutionHeight=675
Expected behavior

Side navigation hidden by default on mobile view
Side navigation shifted down slightly so that it is not hidden behind the top header
Side navigation links automatically close the side navigation when clicked (only on mobile/small screen)

Screenshots

Desktop (please complete the following information):

OS: Mac OS - Ventura 13.4
Browser: Chrome
Version 114.0.5735.133

Smartphone (please complete the following information):

Device: iPhone 11
OS: iOS 16.5
Browser: Safari

🎉 This issue has been resolved in @aws-northstar/ui version 1.1.3 🎉


springboot mvc example csproj possible?

would be great (if possible?) to have a minimal mvc springboot example.. (if possible)
https://docs.spring.io/spring-boot/docs/current/maven-plugin/reference/htmlsingle/

This is most definately out of scope for this project, at least. I'd be happy to open a repository under ikvmnet for something though, but that something would need to be more clearly defined. I'd move the conversation for this to Discussions also.


Bigtable: INTERNAL: HTTP/2 error code: INTERNAL_ERROR Received Rst Stream as a retryable status code

Long lived connections are sometimes disconnected via a RST frame. These should be handled similar to UNAVAILABLE. Also need to make sure that GOAWAYs are handled similar.
This is the same issue as https://github.com/googleapis/cloud-bigtable-client/issues/2177

We are experiencing a "goaway", which is mentioned above in a comment. This is after the process has been running for over an hour.
Caused by: com.google.api.gax.rpc.UnavailableException: io.grpc.StatusRuntimeException: UNAVAILABLE: HTTP/2 error code: NO_ERROR
Received Goaway
max_age
        at com.google.api.gax.rpc.ApiExceptionFactory.createException(ApiExceptionFactory.java:69)
        at com.google.api.gax.grpc.GrpcApiExceptionFactory.create(GrpcApiExceptionFactory.java:72)
        at com.google.api.gax.grpc.GrpcApiExceptionFactory.create(GrpcApiExceptionFactory.java:60)
        ... 23 more

We are performing checkAndMutateRow operations. Are we being thrown this error be because these operations are considered non-idempotent so not internally retried? When such an error happens can it be safely concluded that the operation was not attempted, so perhaps retried idempotently?

We are facing the same issue as @shikhar with checkAndMutateRow operations. However, we have a retry mechanism on top of the BT call and our logic depends on the return value (whether the mutation was applied or not; duplicated mutation would return "not applied"). In the several cases that I checked after several of our retries, the mutation was finally applied without duplication.
However, I would much appreciate it if there were no errors or at least if the error was more descriptive. I would be more than happy to take some measures to prevent it from happening in our setup but at the moment I am clueless as to what is the cause.

GOAWAY is a different issue from the "Received rst stream" error. For checkAndMutateRow, we'll need to investigate if the requests have ever reached cloud bigtable service when these errors happen.

GOAWAY is a different issue from the "Received rst stream" error. For checkAndMutateRow, we'll need to investigate if the requests have ever reached cloud bigtable service when these errors happen.

We are still seeing this issue using the latest bigtable library. I can get a list of timestamps and a GCP bigtable instance if that is helpful.
StatusRuntimeException: INTERNAL: HTTP/2 error code: INTERNAL_ERROR
Received Rst Stream
    at io.grpc.Status.asRuntimeException(Status.java:521)
    at com.google.cloud.bigtable.grpc.async.AbstractRetryingOperation.onError(AbstractRetryingOperation.java:210)
    at com.google.cloud.bigtable.grpc.async.AbstractRetryingOperation.onClose(AbstractRetryingOperation.java:175)
    at com.google.cloud.bigtable.grpc.scanner.RetryingReadRowsOperation.onClose(RetryingReadRowsOperation.java:211)
    at com.google.cloud.bigtable.grpc.io.ChannelPool$InstrumentedChannel$2.onClose(ChannelPool.java:210)
...
(26 additional frame(s) were not displayed)

IOExceptionWithStatus: Error in response stream
    at com.google.cloud.bigtable.grpc.scanner.ResultQueueEntry$ExceptionResultQueueEntry.getResponseOrThrow(ResultQueueEntry.java:100)
    at com.google.cloud.bigtable.grpc.scanner.ResponseQueueReader.getNextMergedRow(ResponseQueueReader.java:105)
    at com.google.cloud.bigtable.grpc.scanner.ResponseQueueReader.getNextMergedRow(ResponseQueueReader.java:111)
    at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:80)
    at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:35)
...
(24 additional frame(s) were not displayed)

RuntimeException: com.google.cloud.bigtable.grpc.io.IOExceptionWithStatus: Error in response stream
    at org.apache.hadoop.hbase.client.AbstractClientScanner$1.hasNext(AbstractClientScanner.java:95)
    at com.alloymetrics.service.compute.query.aggregation.AggregationQuerySpliterator.tryAdvance(AggregationQuerySpliterator.java:166)
    at java.util.stream.StreamSpliterators$WrappingSpliterator.lambda$initPartialTraversalState$0(StreamSpliterators.java:294)
    at java.util.stream.StreamSpliterators$AbstractWrappingSpliterator.fillBuffer(StreamSpliterators.java:206)
    at java.util.stream.StreamSpliterators$AbstractWrappingSpliterator.doAdvance(StreamSpliterators.java:169)
...
(18 additional frame(s) were not displayed)

com.google.cloud.bigtable.grpc.io.IOExceptionWithStatus: Error in response stream

Working on a fix for client-core: https://github.com/googleapis/java-bigtable-hbase/pull/3000


[dv/chip] Add an apply_reset hook for chip_callback_vseq

Hi,
This small PR is adding additional hooks for the chip_callback_vseq before and after the apply_reset in the chip_base_vseq to be used by the extending environments.
It has been found that the hooks in the dut_init are needed but not enough.
Thanks!
Signed-off-by: Eitan Shapira<EMAIL_ADDRESS>

Thanks!


Prevent buffer overflow in package.c (#1177)

Here we go. Hope this builds on all platforms (I am not certain about the availability of strl{cpy,cat} on Windows).
Cheers :)

The strl* functions are only available on BSD-like OS.

Yup, painfully noticed that... I'll take care of this.

Thanks!


Object literal may only specify known properties

Environment
Nuxt project info:                                                                                10:11:52

Operating System: Linux
Node Version:     v16.19.0
Nuxt Version:     3.0.0
Nitro Version:    1.0.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
root/tsconfig.base.json
{
  "compileOnSave": false,
  "compilerOptions": {
    "rootDir": ".",
    "sourceMap": true,
    "declaration": false,
    "moduleResolution": "NodeNext",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,

    "noPropertyAccessFromIndexSignature": false,

    "esModuleInterop": true,

    "importHelpers": true,
    "target": "ESNext",
    "module": "ESNext",
    "lib": [
      "es2017",
      "dom"
    ],
    "skipLibCheck": true,
    "skipDefaultLibCheck": true,
    "baseUrl": "."
  },
  "exclude": [
    "tmp",
    "dist"
  ]
}

app tsconfig.json
{
  // https://v3.nuxtjs.org/concepts/typescript
  "extends": "../../../tsconfig.base.json"
}

Describe the bug
TypeScript complains about the configuration of this plugin in IntelliJ IDEA / WebStorm

When I press STRG + B to jump into the source, it correctly points to this library
import { NuxtModule } from '@nuxt/schema'
declare module '@nuxt/schema' {
  interface NuxtConfig {
    ["auth"]?: typeof import("@sidebase/nuxt-auth").default extends NuxtModule<infer O> ? Partial<O> : Record<string, any>
    ["primevue"]?: typeof import("@sfxcode/nuxt-primevue").default extends NuxtModule<infer O> ? Partial<O> : Record<string, any>
    ["telemetry"]?: typeof import("@nuxt/telemetry").default extends NuxtModule<infer O> ? Partial<O> : Record<string, any>
  }
  interface RuntimeConfig {
   app: {
      baseURL: string,

      buildAssetsDir: string,

      cdnURL: string,
   },
  }
  interface PublicRuntimeConfig {
   version: string,

   apiUrl: string,

   primevue: {
      config: {
         ripple: boolean,
      },
   },
  }
}

What am I missing?

Hey @mklueh 👋
Thanks for the q - this one may be a bit hard to debug as it could be related to many things, e.g., your editor setup. Can you ensure that:

nuxi prepare ran,
volar / whatever you are using is correctly configured?

Other than that I can say: I don't think at the moment that this is related to nuxt-auth but rather to the setup. This could be verified by trying to configure a different nuxt module and seeing if it also gets the red ssquiggles.

@BracketJohn thanks, it was indeed related to my wrong tsconfig setup and it's now working 👍
https://github.com/nuxt/framework/issues/10107


TRPO: correct the params positions of diagonal gaussian kl function.

Background
The KL divergence (Kullback–Leibler divergence) is a measure of how one probability distribution is different from a second.

And the KL divergence is asymmetric, which means

To convey the fundamental asymmetry of the KL divergence,  is described as the divergence of P from Q or the divergence from Q to P.
Spinup code
The diagonal_gaussian_kl function implemented in spinup, calculates the KL divergence  from N1 to N0:
# spinningup/spinup/algos/trpo/core.py

def diagonal_gaussian_kl(mu0, log_std0, mu1, log_std1):
    """
    tf symbol for mean KL divergence between two batches of diagonal gaussian distributions,
    where distributions are specified by means and log stds.
    (https://en.wikipedia.org/wiki/Kullback-Leibler_divergence#Multivariate_normal_distributions)
    """
    var0, var1 = tf.exp(2 * log_std0), tf.exp(2 * log_std1)
    pre_sum = 0.5*(((mu1- mu0)**2 + var0)/(var1 + EPS) - 1) +  log_std1 - log_std0
    all_kls = tf.reduce_sum(pre_sum, axis=1)
    return tf.reduce_mean(all_kls)

which equals to:

Why I use the term from N1 to N0 here is because of the fundamental asymmetry of KL divergence, which means

When invoking the diagonal_gaussian_kl function at the mlp_gaussian_policy function,
# spinningup/spinup/algos/trpo/core.py

def mlp_gaussian_policy(x, a, hidden_sizes, activation, output_activation, action_space):
    ...
    d_kl = diagonal_gaussian_kl(mu, log_std, old_mu_ph, old_log_std_ph)
    ...

the first two params represent the new policy and the last two params represent the old policy.
So the d_kl is the symbol of KL divergence from old policy to new policy, which means:

Problem
In the original TRPO paper, the trust region constraint is defined as:

where  represents the KL divergence from new policy to old policy. But in the code, the d_kl symbol calculates the KL divergence from old policy to new policy, which is incorrect.
So the params to invoke the diagonal_gaussian_kl function is in a wrong order, we need correct it.
Another equation in the original TRPO paper also confirms me in that the param order is wrong:

Bug fix
There are two options to fix this bug:

Change the param order when invoking the diagonal_gaussian_kl function:
We can replace
d_kl = diagonal_gaussian_kl(mu, log_std, old_mu_ph, old_log_std_ph)
with
d_kl = diagonal_gaussian_kl(old_mu_ph, old_log_std_ph, mu, log_std).

Keep the param order not changed, change the inside logic of diagonal_gaussian_kl function:
We can change the implementation of diagonal_gaussian_kl to get the correct KL divergence.
The new diagonal_gaussian_kl function is:

# spinningup/spinup/algos/trpo/core.py

def diagonal_gaussian_kl(mu0, log_std0, mu1, log_std1):
    """
    tf symbol for mean KL divergence between two batches of diagonal gaussian distributions,
    where distributions are specified by means and log stds.
    (https://en.wikipedia.org/wiki/Kullback-Leibler_divergence#Multivariate_normal_distributions)
    """
    var0, var1 = tf.exp(2 * log_std0), tf.exp(2 * log_std1)
    pre_sum = 0.5*(((mu0- mu1)**2 + var1)/(var0 + EPS) - 1) +  log_std0 - log_std1
    all_kls = tf.reduce_sum(pre_sum, axis=1)
    return tf.reduce_mean(all_kls)

which equals to:

When invoking it with d_kl = diagonal_gaussian_kl(mu, log_std, old_mu_ph, old_log_std_ph), the d_kl is the symbol of KL divergence from new policy to old policy:

The first option is simple, so I take it.
Reference

Kullback-Leibler divergence
Trust Region Policy Optimization

Hello! Indeed, you are correct that KL-divergence is not symmetric, and the original paper does things one way, and here we do things a slightly-different way. However, I counter-claim that this is not actually a bug---there is no error, from either a mathematical standpoint (in terms of the theoretical bounds on which TRPO is ostensibly based), or from an outcomes standpoint (in terms of the difference between using KL divergence in one direction or the other in TRPO).
From the mathematical standpoint: the theoretical bounds involved in TRPO in an earlier step use a symmetric divergence (the TV divergence), and only afterwards bound that by a KL divergence. The KL for either order can be used, so the choice of direction is arbitrary in theory.
From an outcomes standpoint: the TRPO computation depends on the KL divergence through its Hessian. It can be shown that the Hessian for KL, when evaluated with both arguments at the same distribution with the same parameterization (P=Q), is the same whether you were taking gradients with respect to the first distribution's parameters or the second distribution's parameters. Therefore, the parameter-space update direction you compute (H^{-1} g) is the same for either KL direction. The only place where a difference in calculation could occur for different choices of KL direction, then, is in the line search, which evaluates KL analytically (instead of with the Hessian). But the second-order approximation is good for small parameter distances, so we should expect that both KL directions will be approximately equal around the trust region, and so there will be virtually no difference in line search outcomes.

@jachiam
Thanks for your explanation. :)


Upgrade deps

upgraded react, bootstrap
code cleanup and refactor

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


How to dump_token_embeddings?

I have trained a model based on token input, and delete the 'char_cnn' parameters from options. Now, I want to extract features from trained ELMO, but I can't produce the 'embedding_weight_file' which is necessary for 'BidirectionalLanguageModel' init when its 'use_character_inputs' is False (because I have delete 'char_cnn', and haven't used char_embedding in my training graph)
I try to use function 'dump_token_embeddings()' to create 'embedding_weight_file', but internal of this function, it still need the 'char_cnn' parameters.
`
def dump_token_embeddings(vocab_file, options_file, weight_file, outfile):
'''Given an input vocabulary file, dump all the token embeddings to the
outfile.  The result can be used as the embedding_weight_file when
constructing a BidirectionalLanguageModel.'''

with open(options_file, 'r') as fin:
    options = json.load(fin)
max_word_length = options['char_cnn']['max_characters_per_token']

vocab = UnicodeCharsVocabulary(vocab_file, max_word_length)
batcher = Batcher(vocab_file, max_word_length)
ids_placeholder = tf.placeholder('int32', shape=(None, None, max_word_length))
model = BidirectionalLanguageModel(options_file, weight_file)
...

`
Does this function just write for the model with char_embedding?
And if I want to dump token embeddings, how do I need to modify this function?

change UnicodeCharsVocabulary to Vocabulary
the Batcher to TokenBatcher
placeholder
the BidirectionalLanguageModel

But if I cerate the BidirectionalLanguageModel without 'char_cnn', it will need 'embedding_weight_file' not None... This is deadlocked

OK, I have fixed this problem, I have modify codes of BidirectionalLanguageModel
`
comment blow codes
    # if not use_character_inputs:
    #     if embedding_weight_file is None:
    #         raise ValueError(
    #             "embedding_weight_file is required input with "
    #             "not use_character_inputs"
    #         )

and change file reading in
token_embedding_file -> weight_file
def _pretrained_initializer(varname, weight_file, embedding_weight_file=None):
...
if varname_in_file == 'embedding':
with h5py.File(weight_file, 'r') as fin:
`
and modify BidirectionalLanguageModelGraph
`
previews under else is None
    if embedding_weight_file is not None:
          ...
    else:
        # +1 for padding
        self._n_tokens_vocab = options['n_tokens_vocab'] + 1

`

hi, I also meet with this problem, thank you for your solution! But I am wondering about the code
self._n_tokens_vocab = options['n_tokens_vocab'] + 1
why do we need to +1 for the _n_tokens_vocab as padding? Thank you!


[sigh] Even if using wildcard provisioning profile, WKAppBundleIdentifier is rewritten

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
I'm using app groups with iOS and Today extension. so I'm building with following configuration.

iOS app

bundle identifier is com.my.app
use com.my.app provisioning profile. (Same as bundle identifier)

Today extension

bundle identifier is com.my.app.today
use com.my.app.today provisioning profile. (Same as bundle identifier)

Watch app

bundle identifier is com.my.app.watch
use com.my.* provisioning profile. (Different form bundle identifier)

Watch app extension

bundle identifier is com.my.app.watch.extension
use com.my.* provisioning profile. (Different form bundle identifier)

In this case, when resign, the WKAppBundleIdentifier in Info.plist of Watch app extension becomes com.my.*.
When trying to installing application from deploygate etc., the following error is output to device log and installation will fail.
installd(MobileSystemServices)[52] <Notice>: 0x16dfdb000 -[MIBundle validatePluginMetadataWithError:]: 552: WatchKit 2 app extension's NSExtension.NSExtensionAttributes.WKAppBundleIdentifier value does not match WatchKit app's bundle ID (found "com.my.*"; expected "com.my.app.watch").

Idea
Submited a pull request #7110
thank you.

@yukiasai Thanks for contributing! We are a little behind on pull requests but will work with you soon over there. I hope you have a wonderful week!

@yukiasai The PR looks good but it would be great to get some verification from the community that this meets their signing needs before merging. 👍

@yukiasai, @asfalcone
Hey there ~
I'm not sure what's the process, but this fix actually breaks our resigning.
I'm looking into more details, but once I have identified actual reason, what do I do next?
Do I reopen this issue or open a new one?

I'm not sure on this code:
if [ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" && ! "$NEW_REF_BUNDLE_ID" =~ \* ];
              then

Is it correct bash syntax?
I'm trying the following and getting errors:
 ~ git:(master) ✗ if [ au.com.commbank.commbank.apnscrypto =~ \* ]; then echo "Match"; fi
zsh: ~ not found
 ~ git:(master) ✗ if [ "au.com.commbank.commbank.apnscrypto" =~ \* ]; then echo "Match"; fi
zsh: ~ not found
 ~ git:(master) ✗

So here's what I got in the end:
 ~ git:(master) ✗ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app"; if [[ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" ]] && ! [[ ${NEW_REF_BUNDLE_ID} =~ '\*' ]]; then echo "YES"; else echo "NO"; fi
YES
 ~ git:(master) ✗ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app.*"; if [[ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" ]] && ! [[ ${NEW_REF_BUNDLE_ID} =~ '\*' ]]; then echo "YES"; else echo "NO"; fi
NO
 ~ git:(master) ✗

I'll follow up with PR to sigh to use this syntax.

zsh is apparently not the same as bush :/
bash-3.2$ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app.*"; if [[ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" ]] && ! [[ ${NEW_REF_BUNDLE_ID} =~ \* ]]; then echo "YES"; else echo "NO"; fi
NO
bash-3.2$ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app"; if [[ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" ]] && ! [[ ${NEW_REF_BUNDLE_ID} =~ \* ]]; then echo "YES"; else echo "NO"; fi
YES
bash-3.2$

still current code seems to be causing error in bash 3.2
bash-3.2$ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app.*"; if [ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" && ! "$NEW_REF_BUNDLE_ID" =~ \* ]; then echo "YES"; else echo "NO"; fi
bash: [: missing `]'
NO
bash-3.2$ REF_BUNDLE_ID="app.prod" NEW_REF_BUNDLE_ID="app"; if [ "$REF_BUNDLE_ID" != "$NEW_REF_BUNDLE_ID" && ! "$NEW_REF_BUNDLE_ID" =~ \* ]; then echo "YES"; else echo "NO"; fi
bash: [: missing `]'
NO
bash-3.2$

sigh 1.12.1 has been released with the fix. Thanks @mgrebenets

This seems to have broken our signing. Embedded extensions resigning is no longer working for us when using a non wildcard identifier.

@jvashishtha Could you please open a new issue with full details? You can link to this issue if you think it is related. Thanks 👍

+1. @jvashishtha
We're using 12.1 for resigning our app, which has watchkit app and extension.
Just resigned on couple of ours ago.
So would be good to have it as separate issue with more details.

I opened #7977 .


Setting range broken for dates

On latest master I am getting a new failure when trying to set the range on time dimensions.
import numpy as np
import pandas as pd
import holoviews as hv

hv.extension('bokeh')

df = pd.DataFrame({
    'x': pd.date_range('2017', '2018', periods=100),
    'y': np.sin(np.linspace(0, 2*np.pi, 100))
})
hv.Curve(df).redim.range(x=(pd.Timestamp('2017-05'),pd.Timestamp('2017-09')))

error output:
/Users/jsignell/holoviews/holoviews/core/util.py in dimension_range(lower, upper, hard_range, soft_range, padding, log)
    883     lower, upper = max_range([(lower, upper), soft_range])
    884     dmin, dmax = hard_range
--> 885     lower = lower if dmin is None or not np.isfinite(dmin) else dmin
    886     upper = upper if dmax is None or not np.isfinite(dmax) else dmax
    887     return lower, upper

TypeError: ufunc 'isfinite' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Thanks, we wrote our own isfinite implementation which also lives in core/util.py for precisely this reason but it seems its not used consistently throughout. Should probably grep for np.isfinite and replace it whereever it makes sense.

Fixed in https://github.com/ioam/holoviews/pull/3045


TrailStartTime moves up 1 year from actual data start

age at 1st marriage has data for 1970 for tto (Trinidad & Tobago) and 1973 for lby(Libya).
When setting these years as trailStartTime in the url Vizabi will turn these in 1971 and 1974 in both url and graph.
They should stay on 1970 and 1973 and show that data in Vizabi.
data:
https://github.com/open-numbers/ddf--gapminder--gapminder_world/blob/master/output/ddf/ddf--datapoints--age_at_1st_marriage_women--by--geo--time.csv
url with 1970 and 1973 trailStartTime:
http://www.gapminder.org/tools/bubbles#_state_time_value=2005;&entities_select@_geo=tto&trailStartTime=1970;&_geo=lby&trailStartTime=1973&labelOffset@:-0.146&:-0.017;;;;&marker_axis%2F_x_which=children%2F_per%2F_woman%2F_total%2F_fertility&domainMin:0.84&domainMax:9.22&zoomedMin:0.86&zoomedMax:8.7&scaleType=linear;&axis%2F_y_which=age%2F_at%2F_1st%2F_marriage%2F_women&domainMin:12.5&domainMax:33.66&zoomedMin:13&zoomedMax:33.66;&size_extent@:0.02&:0.41;;&color_palette_america=%23ebcc21&sub%2F_saharan%2F_africa=%234e7af0&east%2F_asia%2F_pacific=%23f03838&south%2F_asia=%2335d1d1&middle%2F_east%2F_north%2F_africa=%235be56b&europe%2F_central%2F_asia=%23f49d37;&which=geo.geographic%2F_regions

https://github.com/Gapminder/vizabi/pull/1681

Opened the link in issue description first time: correct behaviour
Opened the link second and every time after: incorrect behaviour (trail starts at 1971 and 1974 instead of 1970 and 1973)

@jheeffer  did you test it on stage or on prod? updates are not in prod yet
the link in the issue leads to prod

good one haha, my fault, stage works yes.


feat: update macOS SDK to 11.3

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

Thank you! Have you tried compiling this image and also running it with the profile you want? If so, could you share the output with me here? :)

Thank you for the ideas in this PR! I have adopted the idea for tests in my PR #4 and also bumped to OSX 11.3!


Show percentages in 'Registered VMs with Free Space <35%' report

Before:

After:

I will put together a sanity spec for this in another pr.
@miq-bot add_label bug, reporting
/cc @martinpovolny

Added fix for 'Unregistered VMs with Free Space < 35%'. The very same problem.

Closing/reopening since #8684 was merged to fix master


dep init fails with prepare solver: at least one package must be present in the PackageTree

Running dep init on a repository fails with:
Searching GOPATH for projects...
prepare solver: at least one package must be present in the PackageTree

The error message should be a little more detailed.
the GOPATH contains /home/mildred/Projects/go:WORKDIR/aws-config-elbv2+/gopath. All the needed dependencies are located in /home/mildred/Projects/go while WORKDIR/aws-config-elbv2+/gopath is only containing a symlink WORKDIR/aws-config-elbv2+/gopath/src/github.com/squarescale/aws-config-elbv2 -> WORKDIR/aws-config-elbv2+. The current directory is WORKDIR/aws-config-elbv2+/gopath/src/github.com/squarescale/aws-config-elbv2

this message should now be improved by the new handling we have in place for this.


Integrate footprint of owned website

https://www.websitecarbon.com/ allows people to judge how much websites they control are generating.

Closing due to inactivity


Update the comments endpoint

The implementation should be based on and tested with v2.8 of the Facebook Graph API

Obsolete and deprecated code should be removed

etc.

Also see https://github.com/abjerner/Skybrud.Social/issues/34


[Question] How to find credentials in Watson NLU?

My all set up are done. Now I don't understand, How do I get my username and password?
natural_language_understanding = NaturalLanguageUnderstandingV1(
    version='2017-02-27',
    username="",
    password="")

In that section i provide my Ibm id and password.
In my Watson NLU API I found API key, service id, URL, and all but no username or password.  I think that's why I am getting an error.
ConnectionError: HTTPSConnectionPool(host='gateway.watsonplatform.net', port=443): Max retries exceeded with url: /natural-language-understanding/api/v1/analyze?version=2017-02-27 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fc1321dfa00>: Failed to establish a new connection: [Errno -2] Name or service not known'))
Can someone please help me out? @kant @dolph
Thanks in Advance!!

Use ibm_watson instead of  watson_developer_cloud Also use IAMAuthenticator
authenticator = IAMAuthenticator('{apikey}') natural_language_understanding = NaturalLanguageUnderstandingV1(     version='2021-08-01',     authenticator=authenticator )


Fix hidapi back end

In HidApiUSB call the parent class's constructor so all the required
fields are present.  In particular the addition of the member
'packet_count'  fixes crashes when running hidapi.
The HID API interface was broken by the patch:
d2b5c06d4cde08c2ed8a1121c055d3e6e56fb756 -
"Improve dapTransfer speed by queuing packets"

works for me now, thanks! :+1:


A little overtuned

Fiery Grip should be once per short or long rest.
Change the d6s in Explosive Intent to d4s, and the stunned effect should last 12 seconds, not until the target's next turn.

Maybe the extra damage from Defensive Hearth should be advantage on con and strength saving throws, too.


Read the clipboard without authorization when use flutter_form_builder

When opening a page that uses form builder for the first time, it will prompt that the page needs to read the clipboard

Android 11 Miui 12.5 21.1.7

flutter issue
https://github.com/flutter/flutter/issues/74139


Adding Distil-Whisper

Thought I'd add an issue here for adding Distil-Whisper to use in this project as it looks very promising.

I made sure to be up nice and early for when the weights are released today!
After some validation testing we'll be adding support for it.
Note that it is only available in medium (English only) and large.

distil-whisper is only available in medium.en and large-v2 currently. WIS doesn't have the concept of an English only model (I'll work on this) but if you want to test large-v2 it is available.
Put that in your WIS models directory and restart WIS.


How can we dynamically reload plugin at run time while backend application is running.

While backend java application is running i want to reload the updated plugin at the same time if you have any solution please answer.

We don't support that.

Is the above mention problem is possible by using classloader or any other way if you have any idea about that please tell me.
One of the possible solution i got by using Java Watchservice to auto reload properties is it the correct way i can solve?

If you know how JVM, javaagent and skywalking work, you will find this does not like your mentioned way.

If you want to change a property value, it is easy, but reload plugin is totally different.

yes i am asking about reload the plugin.
In order to load a plugin, you have used classloader so if I improvise the code to reload on certain condition when any plugin update is it work?

That is not the case. You think this too simple


Alternative Encoding for Bytes to QR Digits: 7 bytes into 17 digits

Thanks for your excellent work!
I just wanted to raise that there is an alternative encoding for QR codes that has an even better efficiency for encoding data bytes into digits for use in QR while also having a simpler implementation.
In the Fido 2.2 draft, there are [QR assisted transactions](https://fidoalliance.org/specs/fido-v2.2-rd-20230321/fido-client-to-authenticator-protocol-v2.2-rd-20230321.html#hybrid-qr-initiated:~:text=15 digits%2C respectively.-,func digitEncode,-(d%20%5B%5D ) that also use QR codes to transfer binary data (in their case cbor) and have about the same requirements as here:

transfer a valid URL (in their case with the FIDO: scheme, so that a phone opens this in the right app.)
as tight packing as possible

They use pack 7 bytes (=56bits) and pack them into 17 digits.
So this compares as

encoding
payload bits
encoded digits
encoded bits
overhead

binary
8
8
8
0%

kernel13b4d
39 (3*13)
12
12*3.333=40
2.5%

fido56b17d
168 (3*56)
51
51*3.333=170
1.2%

For Version 40 codes, this is about 35 bytes more compressed payload.
Since this takes whole multiples of 7 bytes, it also makes the implementation easier, as there is no sub-byte shifting necessary.
Adam Langley describes his reasoning on this blog post: https://www.imperialviolet.org/2021/08/26/qrencoding.html
The sample code from the specification in Go is available here: https://fidoalliance.org/specs/fido-v2.2-rd-20230321/fido-client-to-authenticator-protocol-v2.2-rd-20230321.html#hybrid-qr-initiated:~:text=15 digits%2C respectively.-,func digitEncode,-(d%20%5B%5D
I condensed that encoding code in Python for readability (the referenced Go code is probably more efficient but imho less readable):
def fido56b17d_encode(input: IOBase|bytes|bytearray) -> str:
    if isinstance(input, (bytes, bytearray)):
        input = BytesIO(input)
    output = StringIO()

    while chunk := input.read(7):
        q, r = divmod(len(chunk) * 17, 7 )
        num_digits = q + 1*(r>0)
        digits, = struct.unpack("<Q",chunk.ljust(8,b'\0'))
        output.write(str(digits).zfill(num_digits))

    return output.getvalue()

and the decoding code, again in Python:
def fido56b17d_decode(input: TextIOBase|str) -> bytes:
    if isinstance(input, (str)):
        input = StringIO(input)
    output = BytesIO()

    while chunk:=input.read(17):
        byte_length = len(chunk) * 7 // 17
        output.write(struct.pack("<Q", int(chunk,10))[:byte_length])

    return output.getvalue()

I'm not sure if this is still any help at this point in time, or if this ship has sailed already, though.
QR Encoding efficiency for the URL in front of the payload
If the submitted URL before the digit encoded payload is uppercase (more precisely, only using characters from this set<PHONE_NUMBER>9ABCDEFGHIJKLMNOPQRSTUVWXYZ $%*+-./:), it could use alphanumeric encoding which can be represented in 11 bits per 2 characters (so ~70% of binary encoding).
So that would be
HTTPS://KDJ0C.GITHUB.IO/PANIC-REPORT/A/x86-64/V/6.11.0-RC5-NEXT-20240830/ZL/

instead of
https://kdj0c.github.io/panic_report/?a=x86_64&v=6.11.0-rc5-next-20240830&zl=

This case would give you another 52 bytes for the payload. The downside is that this would not have query params, but the payload encoded in the path, which makes the browser app side less flexible (e.g. use a single page app or render the stuff on the server)
As Adam Langley's writes on his blog:

DNS names are case insensitive and “an implementation should accept uppercase letters” for the scheme says RFC 3986. Maybe it just looks odd and that's not worth the bits, though?

Thanks a lot for this very detailed information.
Before doing this, I spent some time looking for some existing solution to this problem, but to my surprise I didn't find any, that's why I came up with my own solution.
Also the QR panic code is currently more a demo, I don't think it's used in production, so it's probably still time to change the encoding. I will see if I can do that, it's better to converge on the same solution. The encoder is in rust, and the decoder in javascript, but I should be able to do it from the python and go sample.
Regarding the main url, I considered changing the encoding for the parts without '&' and '=' but you lose a few bytes each time you change the encoding which makes the gain not worth it. Using path instead of url parameter is a smart trick, but gives less flexibility for the web server. Also the all caps url makes it looks suspicious when your qr code apps asks if the url is legitimate. (Even if it's already quite cryptic with the big payload as digits). So I'm unsure about this one.


Full Process Post Deploy

Hi there,
Thanks again for your work on this - looks really great and i'm just getting round to attempting to set up a deployment pipeline for one of our models.
One thing I am not clear on is if a deploy achieved by something like follows:
start /wait TabularEditor.exe "C:\Projects\MyModel\Model.bim" -D ssasserver databasename -O -C -P -V -W

Will actually do a process on the deployed model?
With the scale of my model meaning it isn't too much resource I would like to deploy and then ensure a full process is done each time the database is deployed.
Is this possible in current command line functionality?
Regards,
Adam

Currently, Tabular Editor will not do any kind of processing, but the functionality is in the works for version 3.0.
You'd have to add a subsequent step to your pipeline to perform the processing, for example using PowerShell and the Invoke-AsCmd cmdlet.


Would be good to be able to delete multiple simulations at once

Having to go into each simulation to delete can be a little time consuming if there are many simulations. We might want to add a "select all" checkbox in the simulation list view and the a delete option.

Could select items using meta+click and which would toggle a delete button at the top. e.g. wunderlist:

Thats the sort of interaction I was thinking of
On Mon, Mar 7, 2016 at 12:59 PM, TristanWright<EMAIL_ADDRESS>wrote:

Could select items using meta+click and which would toggle a delete button
at the top. e.g. wunderlist:
[image: screen shot 2016-03-07 at 10 59 08]
https://cloud.githubusercontent.com/assets/13318121/13578187/a11fa818-e453-11e5-8197-9873ed6c45fa.png
—
Reply to this email directly or view it on GitHub
https://github.com/Kitware/HPCCloud/issues/211#issuecomment-193371921.


Improving the UI for the share on twitter

Hello i noticed the share on twitter thing is somewhat very small and i felt sort of uncomfortable to click on it,
I would like to ask if i can work on this and make a little clearer UI, i am a beginner and not that good at coding so i would like to just help for the design :'D

also i recommend adding your follow me in the footer section, i saw that there were people already doing the UI like color palette  which i highly recommend, the colors right now make it look like a ppt slideshow XD.
If possible could you assign this to me under GSSoC24 :).

Please assign me this task under GSSoC24


Tell users where bulk data to import is

For example, openedxstats/openedxstats/apps/sites/test_data/
edx_sites_csv_7-11-16.csv

Addressed in most recent docs update


Upgrade to commons-io 2.8.0 [PAXLOGGING-319]

Jean-Baptiste Onofre created PAXLOGGING-319

Fixed in: 1.11.8, 2.0.7
Votes: 0, Watches: 1

A test dependency only.


Update config for quantecon.build.lectures

This PR updates the conf.py file to include some settings required for compilation in the quantecon.build.lectures build chain.
jupyter_allow_html_only allows for inclusion of html targeted only blocks as a passthrough to nbconvert
jupyter_target_html adjusts the behaviour of some markdown objects to support more html features for passthrough to nbconvert

Thanks @mmcky, will go ahead and merge.


Remove pyparsing pinning requirement

By using a Regex parser instead of building the rule we are able to get rid of the concurrency issue while loading multiple graphs.
From Paul McGuire, Pyparsing author :

That KeyError is being raised inside the collections.OrderedDict code, and the key is extracted just a couple of lines prior to the place where the exception is raised, so we know it is a valid key. But there is a (small) window of time in which the same key could be popped by two different threads, and so the second thread will fail as the key will no longer be there.

I've found in other parsers that building up numeric values with Combine, Word, etc. is a real slowdown compared to using a Regex, and there is not that much loss in readability or maintainability. These re's aren't that long or complicated.

Cool! Just remove unused imports, it breaks tests.

I haven't tested but I like the approach.

There you go!

Would be nice to have someone benchmark reads after this change.

Would be easy to write a benchmark for grammer, but I strongly suspect that grammar parsing is blazing fast compared to the I/O made when rendering the target.

This looks good, but I'm not sure what the implications are of disabling packrat

Please see #1828, this becomes a real issue as pinning doesn't work in some circumstances.

@DanCech That I don't know. What I have is just words from pyparsing author that performance shouldn't be impacted. Is that your question?

@ybizeul do you mean on chat or something you can link us to?

@DanCech Yes in my original post, last quote

@ybizeul doesn't appear to mention packrat?

Look, I'm trying to pretend I know what we're talking about here. Obviously you got me :-D
I assumed "packrat" was the other chain parsing method, which I dropped in favor of regular expression. That's not what it is right?

After reading https://pyparsing-public.wikispaces.com/Frequently Asked Questions-What the heck is "packrat parsing"? I think that as long as performance isn't affected adversely we shouldn't see any other issues from not using packrat.

As said earlier, I don't think we should worry about performances here, most of the time is spent doing I/O anyway.

Agreed - parsing metrics' grammar is not most heavyweight operation on graphite. But I agreed with @obfuscurity - we should benchmark that, at least on separate test.

@ybizeul would you have time to write a small benchmark for that ?

@iksaif Not sure how to do that?

@ybizeul I guess simply try to call grammar.parseString(target) ~10000 times with and without the patch and record the time :)

Ok, both runs with 1000 iterations.
Legacy :
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.total_data 2s
netapp.perf.MBP.clus1.node.clus1-01.aggr.disk_busy 3s
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.read_latency 4s
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.write_latency 4s
netapp.perf.MBP.clus1.node.clus1-01.processor.avg_processor_busy 6s
sumSeries(netapp.perf.MBP.clus1.node.clus1-01.vol_summary.total_ops) 7s

Patched:
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.read_latency 1s
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.total_data 1s
netapp.perf.MBP.clus1.node.clus1-01.vol_summary.write_latency 2s
netapp.perf.MBP.clus1.node.clus1-01.processor.avg_processor_busy 2s
sumSeries(netapp.perf.MBP.clus1.node.clus1-01.vol_summary.total_ops) 2s
netapp.perf.MBP.clus1.node.clus1-01.aggr.disk_busy 1s

Very nice! Can you share the benchmarking script?

I didn't know how to isolate it, so I don't really have a script.
What I did is this in graphite/render/evaluator.py (and I reinstalled since then but you'll get the idea) :
def evaluateTarget(requestContext, target):

  sec_ = calendar.timegm(time.gmtime())

  for x in xrange(1, 1000):                                  # Loop
    tokens = grammar.parseString(target)
  result = evaluateTokens(requestContext, tokens)

  log target + " " + calendar.timegm(time.gmtime()) - sec_ # pseudo code
  if type(result) is TimeSeries:
    return [result] #we have to return a list of TimeSeries objects

  else:
    return result

Then just loaded a dashboard page

So, is that enough to merge? @graphite-project/committers ?

I think so :+1:


regexes

If we’re not going to have ceylon/ceylon-spec#390, i suggest we add a regex library to the Ceylon SDK that simply uses the platform’s regexes. As far as I know, Javascript regexes are a strict subset of Java regexes, so the regexes themselves shouldn’t be a problem – you’d just need a uniform API to work with them.

I got this which is fully functional: https://github.com/quintesse/ceylon-tako-sdk/blob/master/source/org/codejive/ceylon/regexp/regexp.ceylon which are native wrapeprs around the implementations available in each platform.

@quintesse cool, very nice.

So shall I add it to the SDK?

its really cool something like that in sdk, but in my opnion, i would like more with RegularExpression name ;)

I think I saw a convention somewhere to distinguish between:

Regex or RegExp – what you get in most programming languages
Regular Expression – original theoretical computer science meaning;
Chomsky-3, equivalent to finite automaton or regular grammar (much more
limited than most Regexes)

But I’m not sure how common that distinction is.
On 22.09.2015 00:07, Diego Coronel wrote:

its really cool something like that in sdk, but in my opnion, i would
like more with RegularExpression name ;)
—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon-sdk/issues/182#issuecomment-142123789.

Regex

@gavinking am I sensing resignation in your stance against regexes in language/sdk?

@luolong I have no problem with supporting platform native regexes in the SDK.

(And I have never had FTR.)

@luolong there was never resistance against having regexes, the resistance was against a very large amount of time in creating our own implementation when there were like a gazillion other things to do. But of course something would be needed at some point and it's there where Gavin made it clear he saw much better opportunities for other technologies. But we all knew we'd have regexes at some point in some form.

So Gavin, do I include this in the SDK as ceylon.regex::Regex?

Sure.

Done.


python -m sklearnex my_application.py VERSUS patch_sklearn()

Hi developers and maintainers! First of all, I love this improvement and it really speeds up my programs by 100x! It's amazing.
To my question (not a bug): Say I have a program (my_application.py) that imports packages from Scikit-learn, e.g., from sklearn.ensemble import RandomForestRegressor and runs some subsequent code.
Question: What is the difference between executing the program via the Terminal by calling python -m sklearnex my_application.py OR editing the program itself, e.g. by
from sklearnex import patch_sklearn
patch_sklearn()
from sklearn.ensemble import RandomForestRegressor

Looking forward to hearing from you!

@napetrov, I hope it's okay I ask a quick follow-up question.
I'm writing a package in which is use sklearn quite intensively. I have a base class and some child classes. I could add
from sklearnex import patch_sklearn
patch_sklearn()

to the top of my base class module, but that could lead to the patching being done several times when I instantiate multiple child classes. What would you recommend doing here? If patching multiple times doesn't hurt performance in any ways, I don't mind it---especially if there's a way to silence the message "Intel(R) Extension for Scikit-learn* enabled (https://github.com/intel/scikit-learn-intelex)".
Looking forward to your answer!

@muhlbach  -  there is no harm in patching multiple times.

Thanks for the prompt reply!

@muhlbach  - i'm looking for users to conduct short interview to get understanding on what problems you are solving currently and how our product help you there. If you have interest - please contact me via<EMAIL_ADDRESS>thanks!

I tried adding
from sklearnex import patch_sklearn
patch_sklearn()

to the top of my base class, but it gives the following error. For reproduction, run script "tests.py" from here
Traceback (most recent call last):
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/onedal/svm/svm.py", line 35, in <module>
    from _onedal4py_dpc import (
ModuleNotFoundError: No module named '_onedal4py_dpc'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "tests.py", line 34, in <module>
    from mlregression.mlreg import MLRegressor
  File "/Users/muhlbach/Repositories/mlregression/src/mlregression/mlreg.py", line 8, in <module>
    from .base.base_mlreg import BaseMLRegressor
  File "/Users/muhlbach/Repositories/mlregression/src/mlregression/base/base_mlreg.py", line 5, in <module>
    from sklearnex import patch_sklearn
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/sklearnex/__init__.py", line 18, in <module>
    from .dispatcher import patch_sklearn
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/sklearnex/dispatcher.py", line 26, in <module>
    from .svm import SVR as SVR_sklearnex
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/sklearnex/svm/__init__.py", line 21, in <module>
    from .svr import SVR
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/sklearnex/svm/svr.py", line 25, in <module>
    from onedal.svm import SVR as onedal_SVR
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/onedal/svm/__init__.py", line 17, in <module>
    from .svm import SVC, SVR, NuSVC, NuSVR, SVMtype
  File "/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/onedal/svm/svm.py", line 47, in <module>
    from _onedal4py_host import (
ImportError: dlopen(/Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/_onedal4py_host.cpython-38-darwin.so, 2): Symbol not found: __ZN6oneapi3dal17polynomial_kernel2v113compute_inputINS1_4task2v17computeEEC1ERKNS0_2v15tableESB_
  Referenced from: /Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/_onedal4py_host.cpython-38-darwin.so
  Expected in: /Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/../../libonedal.dylib
 in /Users/muhlbach/opt/anaconda3/envs/main/lib/python3.8/site-packages/_onedal4py_host.cpython-38-darwin.so
(main) muhlbach@Muhlbach-MacBook-Pro src %


Add blocking option for subreddits

Similar blocking as to how it is done for user submissions, but for subreddits. Next to "hide user submissions" tag, also have a "hide subreddit links". This is mostly for on /r/all or the frontpage where you may not want to see a specific popular subreddit. Perhaps ignore this ban if the current page is the subreddit's (such as if you hide /r/programming but are on /r/programming).

Fixed with #38


Update Composer dependencies (2020-07-07-00-10)

Visual regression test passed!
View the visual regression test report


Default values for type parameters are not used as type arguments

Search terms

type-parameters, type-arguments
Expected Behavior

Default values for type parameters are used as type arguments.
Actual Behavior

When a class is extending the binding of a template class with some type parameters getting no type arguments, the default values for those type parameters are not used and instead the type parameters are shown on the page of the class, which makes the class look generic while it isn't.
Example:
class TemplateClass<A = {}, B = any> {
}

class ConreteClass extends TemplateClass<string> {
}

Results in a typedoc page for ConreteClass where ConreteClass has a type parameter B, instead of it being derived from TemplateClass<string, any>.
Steps to reproduce the bug

Create the documentation for the code above.
Environment

Typedoc version: 0.17.8
TypeScript version: 3.9.6
Node.js version: 12.13.0
OS: Windows 10 (64-bit)

I started working on this and things are looking good in my small test project. Will have to test the changes on a bigger React project though.

I realized that there are several bugs involved here so I moved parts of the original issue to #1345.


chore: update docs

Fixes relative links at https://registry.terraform.io/modules/GoogleCloudPlatform/lb-http/google/latest

Is the lint passing in master? I didn't change any of the files that the linter is complaining about...
I see the same lint check failing in other PRs such as https://github.com/terraform-google-modules/terraform-google-lb-http/pull/471

@apeabody thx. I think this is related to rollback of cft dev tool. I will merge this one.


Zoom animation bug on empty map

In master version (04.03.2013) Leaflet after zoom animation on empty map is not working using zoom-control and mouse wheel
http://jsfiddle.net/OriginalSin/FJ7RM/4/
If you press any keyboard button for panning (for example, up button), both ZoomControl and zoom using mouse wheel is restored.
This bug in not reproduced in v0.5.1.

Testing Leaflet 1.0.0 stable
https://github.com/OriginalSin/Leaflet1.0.0-test


PMID:20826461

normal number of nuclei in ascus
( NB this is independent of whether the spores from correctly, in these exp the nucleie are OK but the spores don't always form)

wrong tracker?

yup


fix: allow builds without metrics

Closes https://github.com/n0-computer/iroh/issues/149

hmm, this is a lot of code 😥

hmm, this is a lot of code 😥

It is, got any better ideas?
My alternative is to make a null metrics implementation and switch on that with features. Might or might not be better but should be more contained.

My alternative is to make a null metrics implementation and switch on that with features.

I think that might be nicer, I am worried about things like the resolver code having cfgs all over the place.

My alternative is to make a null metrics implementation and switch on that with features.

I think that might be nicer, I am worried about things like the resolver code having cfgs all over the place.

Yeah it's not pretty. And a bit tedious to annotate every call to every metrics call ever. I'll do another branch for it and see where that goes.


Error running script (Delete-NavApplicationObject) in container.

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Describe the issue
I am getting an error trying to run a script in a container.  I suspect it maybe related to accessing a remote SQL server (#1518).  I am attempting step 2 in Task 9 of Upgrading the Data to Business Central: Single-Tenant Deployment
I get the same error when running the Delete-NAVApplicationObject command from PowerShell after entering the container.
Any assistance would be appreciated.

: [27590710] Unable to process table changes because the Microsoft Dynamics NAV Development Environment cannot connect to the specified Microsoft Dynamics NAV Server instance:
Server Name:
erver Instance:
anagement Port: 0
Microsoft Dynamics NAV Development Environment will detect a running server automatically.
Check that:
. The Microsoft Dynamics NAV Server is running
. The management service is configured
. The firewall is open for the management port if remote
. The server is running with only one tenant
. Microsoft Dynamics NAV Development Environment is running as Administrator, or
. UAC is turned off.
at Delete-NAVApplicationObject, C:\Program Files (x86)\Microsoft Dynamics NAV\140\RoleTailored Client\Microsoft.Dynamics.Nav.Ide.psm1: line 473

This is the container's hosts file (without the standard comments):

<IP_ADDRESS> dbserver.containerhelper.internal
<IP_ADDRESS> host.containerhelper.internal

$databaseName = 'vect001 upgrade'
$DatabaseServer = 'dbserver.containerhelper.internal'
$user = 'Docker'
$uPass = '6Hash######'

        Invoke-ScriptInBCContainer -containerName $config.Container -scriptblock {
            param ($databaseName, $databaseServer, $user, $uPass)
            Delete-NAVApplicationObject -DatabaseName $databaseName -DatabaseServer $DatabaseServer -userName $user -password $uPass `
                -Filter 'Type=Table;Id=470|824|825|827|828|829|1510' `
                -SynchronizeSchemaChanges Force -Confirm:$false

Scripts used to create container and cause the issue
$sqlserver'; . 'c:\run\SetupDatabase.ps1'" } `
Clear-Host

$containerName = 'vect14017'
$password = '<password>'
$securePassword = ConvertTo-SecureString -String $password -AsPlainText -Force
$credential = New-Object pscredential 'admin', $securePassword
$auth = 'UserPassword'
$artifactUrl = Get-NavArtifactUrl -nav '2016' -cu '45' -country 'gb'
$databaseServer = 'dbserver.containerhelper.internal' #per #1518
$databaseInstance = ''
$databaseName = 'vect001 upgrade'
$databaseUsername = 'Docker'
$databasePassword = '<password>'
$databaseSecurePassword = ConvertTo-SecureString -String $databasePassword -AsPlainText -Force
$databaseCredential = New-Object pscredential $databaseUsername, $databaseSecurePassword
$licenseFile = 'F:\License\NAV2018License.flf'
New-BcContainer `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -databaseServer $databaseServer -databaseInstance $databaseInstance -databaseName $databaseName `
    -databaseCredential $databaseCredential `
    -licenseFile $licenseFile `
    -dns '<IP_ADDRESS>' `
    -updateHosts

Full output of scripts
BcContainerHelper is version 1.0.15
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 19.03.12
Docker Server Version is 19.03.12
Removing container vect14017
Removing vect14017 from host hosts file
Removing vect14017-* from host hosts file
Removing f:\bccontainerhelper\Extensions\vect14017
Fetching all docker images
ArtifactUrl and ImageName specified
Image internal:onprem-14.17.44663.0-gb-nodb already exists
Using image internal:onprem-14.17.44663.0-gb-nodb
Creating Container vect14017
Version: 14.17.44663.0-gb
Style: onprem
Multitenant: No
Platform: 14.0.44656.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1577 (ltsc2019)
Host OS Version: 10.0.17763.1577 (ltsc2019)
Using process isolation
Using locale en-GB
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file F:\License\BC14License.flf
Additional Parameters:
-v "F:\Clients\VECT001\Shared":C:\Shared
Files in f:\bccontainerhelper\Extensions\vect14017\my:
- AdditionalOutput.ps1
- AdditionalSetup.ps1
- license.flf
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container vect14017 from image internal:onprem-14.17.44663.0-gb-nodb
fc8b7c7b785eb0c395ee608b34b9da74882a0b0c8049e88d736bbc01c5a07d3f
Waiting for container vect14017 to be ready
Initializing...
Setting dbserver.containerhelper.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is vect14017
PublicDnsName is vect14017
Using NavUserPassword Authentication
Starting Internet Information Server
Import Encryption Key
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 7DB673E07D01E8D3674BBDFBC7CF5BDB9C7316F7
Modifying Service Tier Config File with Instance Specific Settings
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Container IP Address: <IP_ADDRESS>
Container Hostname  : vect14017
Container Dns Name  : vect14017
Web Client          : http://vect14017/NAV/
Dev. Server         : http://vect14017
Dev. ServerInstance : NAV
Setting vect14017 to <IP_ADDRESS> in host hosts file

Files:
http://vect14017:8080/ALLanguage.vsix

Container Total Physical Memory is 28.0Gb
Container Free Physical Memory is 19.3Gb

Initialization took 145 seconds
Ready for connections!
Reading CustomSettings.config from vect14017
Creating Desktop Shortcuts for vect14017
Creating .net Assembly Reference Folder for VS Code
Copying DLLs from C:\Windows\assembly to assemblyProbingPath
Copying DLLs from C:\Program Files\Microsoft Dynamics NAV\140\Service to assemblyProbingPath
Copying DLLs from C:\Program Files (x86)\Microsoft Dynamics NAV\140\RoleTailored Client to assemblyProbingPath
Copying DLLs from C:\Program Files (x86)\Open XML SDK to assemblyProbingPath
Container vect14017 successfully created

Use:
Get-BcContainerEventLog -containerName vect14017 to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName vect14017 to get debug information about the container
Enter-BcContainer -containerName vect14017 to open a PowerShell prompt inside the container
Remove-BcContainer -containerName vect14017 to remove the container again
docker logs vect14017 to retrieve information about URL's again
Creation time 209 seconds
Creating User Docker
Assigning Permission Set SUPER to Docker```

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Additional context**
- does it happen all the time? YES
- did it use to work? outside of a container it works.

Don't you need to specify -NavServerInstance as well?
Looked in the docs - sorry, but I don't know a lot about these commands.

Not normally but I have tried adding that and still have the problem.

You tried to specify all 3 parameters?
When I look inside the compile-objectsInNavContainer - it specifies all 3
-NavServerName localhost   -NavServerInstance $ServerInstance
-NavServerManagementPort "$managementServicesPort"

I have and had the same results.
$databaseName = 'vect001 upgrade'
$DatabaseServer = 'dbserver.containerhelper.internal'
$user = 'Docker'
$uPass = '6Hash######'

Delete-NAVApplicationObject -DatabaseName $databaseName -DatabaseServer $DatabaseServer -userName $user -password $uPass -Filter 'Type=Table;Id=470|824|825|827|828|829|1510' -SynchronizeSchemaChanges Force -Confirm:$false -NavServerInstance 'NAV' -NavServerName 'vect14017' -NavServerManagementPort 7045 -Verbose -SuppressElevationCheck Yes

VERBOSE: Delete application objects from vect001 upgrade database.
VERBOSE: Running command: & "C:\Program Files (x86)\Microsoft Dynamics NAV\140\RoleTailored Client\finsql.exe" --% Command=DeleteObjects,SynchronizeSchemaChange
s=Force,Filter="Type=Table;Id=470|824|825|827|828|829|1510",LogFile="C:\Users\ContainerAdministrator\AppData\Local\Temp\NavIde\5e0c6a52-2d08-4775-a38c-1507c09c7
66d\naverrorlog.txt",ntauthentication=No,username="Docker",password="6Hash######",ServerName="dbserver.containerhelper.internal",Database="vect001 upgrade",NavS
erverName="vect14017",NavServerInstance="NAV",NavServerManagementport=7045,SuppressElevationCheck="Yes" | Out-Null
: [27590710] Unable to process table changes because the Microsoft Dynamics NAV Development Environment cannot connect to the specified Microsoft Dynamics NAV
Server instance:
Server Name:
erver Instance:
anagement Port: 0
Microsoft Dynamics NAV Development Environment will detect a running server automatically.
Check that:
. The Microsoft Dynamics NAV Server is running
. The management service is configured
. The firewall is open for the management port if remote
. The server is running with only one tenant
. Microsoft Dynamics NAV Development Environment is running as Administrator, or
. UAC is turned off.
+ CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
+ FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Delete-NAVApplicationObject

So it works from a BC install on the Docker Host so it has to be related to how the container is configured.
$databaseName = 'vect001 upgrade'
$DatabaseServer = 'dbserver.containerhelper.internal'
$user = 'Docker'
$uPass = '6Hash######'
$navServerInstance = 'BC140'
$NavServerName = 'localhost'

Delete-NAVApplicationObject -DatabaseName $databaseName -DatabaseServer $DatabaseServer -userName $user -password $uPass -Filter 'Type=Table;Id=470|824|825|827|828|829|1510' -SynchronizeSchemaChanges Force -Confirm:$false -NavServerInstance $navServerInstance -NavServerName $NavServerName -NavServerManagementPort 7045 -Verbose -SuppressElevationCheck Yes

PS C:\windows\system32>
VERBOSE: Delete application objects from vect001 upgrade database.
VERBOSE: Running command: & "F:\Program Files (x86)\Microsoft Dynamics 365 Business Central\140\RoleTailored Client\finsql.exe" --% Command=DeleteObjects,Synchr
onizeSchemaChanges=Force,Filter="Type=Table;Id=470|824|825|827|828|829|1510",LogFile="C:\Users\todd.scott\AppData\Local\Temp\NavIde\1b74ad98-d3fa-40c0-bf6a-e6d0
10584142\naverrorlog.txt",ntauthentication=No,username="Docker",password="6Hash######",ServerName="dbserver.containerhelper.internal",Database="vect001 upgrade"
,NavServerName="localhost",NavServerInstance="BC140",NavServerManagementport=7045,SuppressElevationCheck="Yes" | Out-Null
PS C:\windows\system32>

When running inside the container, you would set NavServerName to localhost (you are on the machine with the NavServer) and you should replace ntauthentication=No with ntauthentication=0

NavServerName = localhost does not seem to correct the issue.  ntAuthentication is NOT a parameter on the Delete-NAVApplicationObject cmdlet.  It is put there by the Microsoft.Dynamics.Nav.Ide.psm1 that "translates" the cmdlet to a FINSQL command line.
Updated script...
$databaseName = 'vect001 upgrade'
$DatabaseServer = 'dbserver.containerhelper.internal'
$user = 'Docker'
$uPass = '6Hash######'
$navServerInstance = 'NAV'
$NavServerName = 'localhost'

Delete-NAVApplicationObject -DatabaseName $databaseName -DatabaseServer $DatabaseServer -userName $user -password $uPass `
    -Filter 'Type=Table;Id=470|824|825|827|828|829|1510' `
    -NavServerInstance $NavServerInstance `
    -NavServerName $NavServerName `
    -NavServerManagementPort 7045 `
    -SynchronizeSchemaChanges Force `
    -SuppressElevationCheck Yes `
    -Confirm:$false `
    -Verbose

I have recreated the containers a number of times and was still having an issue.  Now it works!?!


Revert "feat: add marisa-trie"

Reverts home-assistant/wheels#711
@vermut So sorry about this, but I've must have been sleeping.
This PR was added to the wrong repository, it should be added to this repository instead:
https://github.com/home-assistant/wheels-custom-integrations
The good news is though, the CI passes and we can build the wheels 👍

../Frenck

easy - https://github.com/home-assistant/wheels-custom-integrations/pull/695


Exposure logging doesn’t work

My exposure logging doesn‘t work anymore.

Hey @tiny-slayer
Thanks for the report!
How long are you seeing this issue already?

For your information: There is also a FAQ entry on this error:  https://www.coronawarn.app/en/faq/results/#ENError13

Hi Tim, it’s been a week. I get the error when I click on “restart”. The last update was on 4th of June, I guess since then I get that error.

Hey @tiny-slayer, thanks for the feedback! Could you try turning exposure logging off completely for 24 hours, and then reactivating it again? This might help. Please note, that the app does not collect any random IDs if you turn exposure logging off.
Could you please also let us know:

iOS version
App version
iPhone model

Thanks!

Hi again,
IOS version is iOS 15.3.1
App version is 2.23.2 (0)
Phone model is iPhone 11

Yes, I did already. I’m going to turn it on again tomorrow.

It still doesn’t work:(
Now there is another error.

@thomasaugsten Please, could you have a look at this?

Please deactivate the ENF for 24 and activate the error report/ Fehlerbericht an activate the ENF after 24 error. We need an error report of the no exposure window error

Hi, i have the same issue. Can you support in fixing it?

@Lulara14
Please, could you enable error logging like this:

German steps

In der CWA auf dem Reiter "Status" ganz nach unten scrollen
Auf "App-Informationen" scrollen
Auf "Fehlerberichte"
Auf "Starten" tippen
Einmal machen, dass der Fehler auftaucht
Schritte 1-3 wiederholen
Auf "Fehlerbericht senden" tippen
Auf "Einverstanden und senden" tippen
Nach unten scrollen
Auf "ID-Historie" tippen
Die jeweilige ID hier in den GitHub Issue stellen

English steps
1. In CWA, scroll down to the very bottom of the "Status" tab
2. Tap on "App Information"
3. Tap on "Error Reports"
4. Tap "Start"
5. Do the steps that cause the error
6. Rebelst steps 1-3
7. Tap on "Send Error Report"
8. Tap on "Agree and Send"
9. Scroll down
10. Tap on "ID History"
11. Let us know the corresponding ID in this GitHub issue

After you did this, please perform the following steps to fix the problem:

German steps

iOS-Einstellungen öffnen
Bis zu "Begegnungsmitteilungen" scrollen & "Begegnungsmitteilungen" antippen
Auf "Corona-Warn-App" ("Aktive Region") tippen
Auf "Corona-Warn-App" tippen
Runterscrollen
"Begegnungsinformationen teilen" deaktivieren
Etwas mehr als 24 Stunden warten
Schritte 1-5 wiederholen
"Begegnungsinformationen teilen" aktivieren
Oben auf den blauen Text "Als aktive Region festlegen" tippen
CWA öffnen und prüfen ob alles wieder funktioniert

English steps

Open the iOS Settings
Scroll down to "Exposure Notifications"
Tap on "Corona-Warn-App" ("Active Region")
Tap on "Corona-Warn-App"
Scroll down
Deactivate "Share Exposure Information"
Wait for a bit more than 24 hours
Perform steps 1-5
Activate "Share Exposure Information"
Tap on the blue text above "Set As Active Region"
Open up CWA and check if everything works again

Hey @Lulara14 & @tiny-slayer, does the error still occur for you? Or can we close this issue?

It is solved now! Tks
Sent from my iPhone

On 5. Aug 2022, at 15:25, Tim @.***> wrote:

Hey @Lulara14 & @tiny-slayer, does the error still occur for you? Or can we close this issue?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

The error is not happening anymore, thank you!
Kindest regards,
Aizhan
Sent from my iPhone

On 5. Aug 2022, at 15:25, Tim @.***> wrote:

Hey @Lulara14 & @tiny-slayer, does the error still occur for you? Or can we close this issue?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

@thomasaugsten Please, could you close this issue?


Trying to run from the automatic version of vlanmandic result in clip error

Good afternoon,
I tought I would try to run the merger under vladmandic port of auto1111 webui and it is giving me a CLIP error...
(venv) PS D:\automatic\extensions\sd-webui-bayesian-merger> python.exe .\bayesian_merger.py
Loading sac+logos+ava1-l14-linearMSE.pth
Loading ViT-L/14
Error executing job with overrides: []
Traceback (most recent call last):
  File "D:\automatic\extensions\sd-webui-bayesian-merger\bayesian_merger.py", line 37, in main
    bo = cls(cfg)
  File "<string>", line 5, in __init__
  File "D:\automatic\extensions\sd-webui-bayesian-merger\sd_webui_bayesian_merger\optimiser.py", line 127, in __post_init__
    self.scorer = AestheticScorer(self.cfg)
  File "<string>", line 4, in __init__
  File "D:\automatic\extensions\sd-webui-bayesian-merger\sd_webui_bayesian_merger\scorer.py", line 79, in __post_init__
    self.load_model()
  File "D:\automatic\extensions\sd-webui-bayesian-merger\sd_webui_bayesian_merger\scorer.py", line 144, in load_model
    self.load_clip()
  File "D:\automatic\extensions\sd-webui-bayesian-merger\sd_webui_bayesian_merger\scorer.py", line 155, in load_clip
    self.clip_model, self.clip_preprocess = clip.load(
AttributeError: module 'clip' has no attribute 'load'

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

I know you are developping for auto1111 but many users are now moving away from it because it is becoming stale... and falling behind from a support and feature point of view. Perhaps you might consider making this great extension work under vladmandic fork:
https://github.com/vladmandic/automatic
This is a fresh install of vladmandic with no customization... so it is as pure as can be.

Many thanks. I will give that a try...

Hummm... looking at pip freeze it look like vladmandic install clip @ git+https://github.com/openai/CLIP.git@d50d76daa670286dd6cacf3bcd80b5e4823fc8e1
I tried uninstalling that and dod a pip upgrade -r requirements.txt and it installed the standard clip==0.2.0 but still the same error.
I guess I can always run the merger from within the auto1111 repo...


[BUG] Gremlin queries that may need pk.

Describe the bug
I have some queries that timeout after 10min, which I think we may need to use pk index to speed up.
900 million vertices.
g.V().hasLabel('%s').has('id', '%s')  # timeout
g.V(%s) # timeout
g.V(%s).bothE(%s). # timeout

These cases have been fixed.


-n and -e cannot be used together

Your environment
λ ruby -v
ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-darwin22]
λ rdbg -v
rdbg 1.7.1

Describe the bug
I was trying to execute some command with -c and specify a break point using -e so that I don't need to write the script in the rdbg again.
It works but I still need to manually run continue, to avoid this, I thought -n could help. However, after some tests I turns out -n and -e cannot be used together.
My current workaround is to put continue in -e:
rdbg -c -e 'b OpenStruct#initialize'$'\n''continue' -- ruby -rostruct -e 'OpenStruct.new'

To Reproduce
rdbg -n -c -e 'b OpenStruct#initialize' -- ruby -rostruct -e 'OpenStruct.new'

Expected behavior
process won't exit and attached rdbg shell on OpenStruct#initialize, output should look like:
[122, 131] in /usr/local/Cellar/ruby/3.2.2/lib/ruby/3.2.0/ostruct.rb
   122|   #   data = OpenStruct.new(hash)
   123|   #
   124|   #   data   # => #<OpenStruct country="Australia", capital="Canberra">
   125|   #
   126|   def initialize(hash=nil)
=> 127|     if hash
   128|       update_to_values!(hash)
   129|     else
   130|       @table = {}
   131|     end
=>#0    OpenStruct#initialize(hash=nil) at /usr/local/Cellar/ruby/3.2.2/lib/ruby/3.2.0/ostruct.rb:127
  #1    [C] Class#new at -e:1
  # and 1 frames (use `bt' command for all frames)

Stop by #0  BP - Method  OpenStruct#initialize at /usr/local/Cellar/ruby/3.2.2/lib/ruby/3.2.0/ostruct.rb:126

Additional context
process exists and prints nothing.

The current -e command feature is not run the command at the beginning, but run the command at the first pausing.
So the current behavior is intentional.
I agree that the current semantics is not easy... Maybe you expect that it is written in ~/.rdbg.
Introduce new option for that?


Allow the user to conditional view total

Need to show totals of expenses expenses show db|csv --total

Thanks for your contribution @kingzbauer !
There seems to be an issue with accuracy in the CSV total value.  The image below shows a sample that I created to verify the changes. While the database shows the right value, the CSV seems to have updated it to the nearest integer:

Also an aesthetic suggestion would be to make cost the last column, so we don't have the incomplete border as seen in the image.
Let me know what you think. Thanks!

Thanks for your contribution @kingzbauer !
There seems to be an issue with accuracy in the CSV total value.  The image below shows a sample that I created to verify the changes. While the database shows the right value, the CSV seems to have updated it to the nearest integer:

Also an aesthetic suggestion would be to make cost the last column, so we don't have the incomplete border as seen in the image.
Let me know what you think. Thanks!

Aah, I see, lemme fix that.
For reordering the columns, that's more straight forward for sql. I think for csv, each record would need to be reordered. This is no issue for a small list but for a longer row set maybe. I'll go with the naive approach of reordering the csv row by row.

Aah, I see, lemme fix that.
For reordering the columns, that's more straight forward for sql. I think for csv, each record would need to be reordered. This is no issue for a small list but for a longer row set maybe. I'll go with the naive approach of reordering the csv row by row.

My suggestion would be not modify the underlying CSV file but just change the order in which they are displayed by changing the table.Append() calls.  I’m not sure if that’s you mean changing row by row.

My suggestion would be not modify the underlying CSV file but just change the order in which they are displayed by changing the table.Append() calls.  I’m not sure if that’s you mean changing row by row.

Not changing the csv, just changing the []string to reflect the desired order

Not changing the csv, just changing the []string to reflect the desired order

Great! Thanks 😀

Great! Thanks 😀

Updated this.

Updated this.

Thanks a lot @kingzbauer ! Really appreciate your contributions 😀

Thanks a lot @kingzbauer ! Really appreciate your contributions 😀


Clojure REPL in async environments does not eval output

When trying to print things or get values from an async implementation usually the thing you wanna spill out will not show unless is the last action of a function.

(ns hospital.aula6
  (:use [clojure pprint])
  (:require
   [hospital.model :as h.model]))

(defn cabe-na-fila?
  [fila]
  (-> fila
      count
      (< 5)))

(defn chega-em
  [fila pessoa]
  (if (cabe-na-fila? fila)
    (conj fila pessoa)
    (throw (ex-info "Fila já está cheia" {:tentando-adiconar pessoa}))))

(defn chega-em! [hospital pessoa]
  (let [fila (get hospital :espera)]
    (alter fila chega-em pessoa)))

(defn simula-um-dia
  []
  (let [hospital {:espera (ref h.model/empty-queue)
                  :laboratorio1 (ref h.model/empty-queue)
                  :laboratorio2 (ref h.model/empty-queue)
                  :laboratorio3 (ref h.model/empty-queue)}]
    (dosync
     (chega-em! hospital "guilherme")
     (chega-em! hospital "ana")
     (chega-em! hospital "paulo")
     (chega-em! hospital "paloma")
     (chega-em! hospital "maria")
     (chega-em! hospital "david"))
    (pprint hospital)))

(simula-um-dia)

(defn async-chega-em! [hospital pessoa]
  (future
    (Thread/sleep (rand 5000))
    (dosync
     (pprint "Tentando adicionar pessoa" pessoa)
     (chega-em! hospital pessoa))))

(defn simula-um-dia-async
  []
  (let [hospital {:espera (ref h.model/empty-queue)
                  :laboratorio1 (ref h.model/empty-queue)
                  :laboratorio2 (ref h.model/empty-queue)
                  :laboratorio3 (ref h.model/empty-queue)}]
    (async-chega-em! hospital 15)
    (pprint hospital)))

(simula-um-dia-async)

Since is not a private code I will submit it here to fast testing with the actual environment, it's a tutorial, and the empty queue is just a way to create an empty queue without all that code that is needed to create it.
(def empty-queue clojure.lang.PersistentQueue/EMPTY)

So you are expecting to see "Tentando adicionar pessoa" in the log and you are not? Do you see the output in you Clojure nREPL window instead? Because you're print is in a different thread to the nREPL thread it means Conjure doesn't get told about the output, so in that case it normally goes to the original nREPL you are connected to.
Running :ConjureOutSubscribe inside your editor may fix that problem and redirect ALL console output into Conjure.
If you are expecting to see values in :laboratorio1 etc though, you will not when they're pprint-ed at the end of simula-um-dia-async. When you call pprint the code inside async-chega-em! has not run yet. So you will need to store the references to the queues or the hospital map somewhere and pprint it later when the future has completed.
That part is just how Clojure (and all async programming) works and isn't something I can change I'm afraid.  So! If your question is about the println not showing up, I hope the command I mentioned helps, if it's about the async code not running in time, I'm afraid that's just a normal programming thing and you'll need to make your code check periodically or wait (using https://clojuredocs.org/clojure.core/deref etc) until the async code is complete before printing it.
I hope this helps!

I'm trying to see the "Tentando adicionar pessoa".
I did try to use the command you mentioned, nothing different happened, the output still get's "ignored" like you said it's running in another thread.
Another thing is, looking at the nREPL started by :Lein I cannot see there too, even after waiting some time, it's not a game ending behavior but in IntelliJ if you wait for some time the output gets there, so I think it should be doable.


HKDFBytesGenerator misleading and dead Code

see https://github.com/bcgit/bc-java/blob/master/core/src/main/java/org/bouncycastle/crypto/generators/HKDFBytesGenerator.java#L75
HMAC Key will be anyway set either to an empty byte array with hashLen or the salt byte array.

code removed.


Request to rebuild the image for docker hub

Please, consider rebuilding the image and pushing latest version to Docker Hub.
Latest Alpine Linux 3.12 includes easy-rsa version 3.0.7-r0 which fixes some annoying bugs, while Alpine 3.11 sticks to easy-rsa 3.0.6-r0.
That would be most appreciated. Thanks!
Fixes: #528
Caused by: https://github.com/OpenVPN/easy-rsa/issues/261#issuecomment-599529357

However, when I tried to rebuild manually from Alpine Linux 3.12 / easy-rsa 3.0.7-r0 on my server, I got new problems.
$ docker-compose run --rm openvpn ovpn_initpki produces:
Easy-RSA error:

The file '/etc/openvpn/vars' was not found.

I am highly uncertain about the cause of it, as well as I am not sure my building environment is compatible with that of Travis CI.
Tests from tests/run.sh mostly fail too:
testing kylemanna/openvpn:latest
        'paranoid' [1/8]...failed
        'conf_options' [2/8]...passed
        'client' [3/8]...
Easy-RSA error:

The file '/etc/openvpn/vars' was not found.
Unable to find "test1", please try again or generate the key first
==> Config match not found: ^tun-mtu\s\+1337
failed
        'basic' [4/8]...failed
        'dual-proto' [5/8]...failed
        'otp' [6/8]...failed
        'iptables' [7/8]...failed
        'revocation' [8/8]...failed

@ratijas : I have updated Dockerfile on my fork, tp-ng branch, that use easy-rsa 3.0.7 and based on alpine:3.12.0 image. Give it a spin and let me know if it works for you.
https://github.com/neofob/docker-openvpn/blob/tp-ng/Dockerfile
TODO: Make a separate branch for PR.

@neofob I am confused by all this checkout and cleanup about easyrsa. They have normal GitHub releases page, and there is 3.0.7 tag with tgz archive — is there a reason not use that one?
https://github.com/OpenVPN/easy-rsa/releases/tag/v3.0.7
https://github.com/neofob/docker-openvpn/blob/16beaf939de00309047573a36dd458cac999eb54/Dockerfile#L14-L20
For now I turned back to using openvpn-install script. But for the same of docker experiments I'll spin up a new server to try it out, but probably not before the next week.

@ratijas : iirc, at the time I was working to get it working, there were 2 reasons for that:

There was not easy-rsa version 3.0.7 in alpine image 3.12.0. I think that the rc0 didn't work.
I resorted to use git route and it worked.

Yeah, I'll check out the package on releases github page and give it a spin.
Thanks for pointing that out...

As I pointed out in the first post here, newer Alpine image already has easyrsa 3.0.7, so might just take that for base. Maybe things have changed again since, who knows...

Also, what's the deal with that lonely cd &&?
https://github.com/neofob/docker-openvpn/blob/16beaf939de00309047573a36dd458cac999eb54/Dockerfile#L15

Haha, it must have been copy & paste or something...

@ratijas : I just tested with custom built docker image. So removing this line ENV EASYRSA_VARS_FILE $OPENVPN/vars and updating these 2 variables work now with alpine:3.12.0 and alpine easy-rsa package.
Feel free to make a PR if your change(s) work.


Fix: Binary sevctl was absent from debian packages

Solution: Build sevctl using an upstream version of Rust (the version in Debian is not supported), and bundle it in the Debian packages.
Add a setting in aleph-vm with the path of the bundled binary.

Test are failing
error: could not find `Cargo.toml` in `/opt/packaging/sevctl` or any parent directory
make: *** [Makefile:51: target/sevctl] Error 101
make: *** [Makefile:93: all-podman-debian-12] Error 2
ls: cannot access 'packaging/target': No such file or directory


Project Structure revision

Problem
Somehow Smoothie's project structure has become very confusing lately.

CategoryCache and OrderedNutrientsCache are under pages/product/
Some cards are in pages/product some in card/.../ (also some product related ones there)
We have lib/widgets and lib/generic_lib for widgets and a lot of widgets are just spread across other folders

Proposed solution
Looking at all the files we currently have I propose a separation by features with some exceptions.
Meaning that we split all the Pages including theire specific widgets into theire own folders e.g scan, product_page,  user_managment ...
But I would keep a dedicated lib/widgets folder for common widgets like SmoothCard etc and keep the database folder (you could even see it as a own feature)
I'd like to hear your opinions on that @monsieurtanuki @g123k @cli1005 @AshAman999 @bhattabhi013 @vik4114 @teolemon

I agree with the suggestions, though not sure if we would separate widgets by features or just keep them separate in the widgets folder. To me keeping widgets separately makes more appealing and easy then making them separate by functionality

Instead of lib/widgets and lib/generic_lib, would be better to club them all in one widgets folder and maybe make further subdirectories thereon.
Some suggestion on the helper folder, It would make more sense if all files related to certain tasks are separated into their directors, for example, analytics, launch URL everything is there, it would be better if all of them get to be at their sub directories, like everything related to analytics there in the lib/helpers/analytics
Even in the pages directory, it is too much clutter there, would be better to do some cleanup there as well.
Had saw one project something like this, felt cleaner, if we could do something like this
lib

lib/network

Still thinking much before i make further suggestions,
Till then waiting for everyone else's suggestions

I don't say that the files are always in the good folders (I don't even have an opinion on that), but I read somewhere about a deadline on May 23rd and now I have limited time to devote to Smoothie. In short: to me, that's not a priority at all.

Hi @M123-dev,
I also think that there is a need to re-structure the project.
I also have some suggestions regarding the structure e.g.

Separation on the basis of functionality and in case of exception, the widget should be placed on the nearest/parent page.
A single widget folder for widgets.
Create a separate folder for every page.

I'll share a detailed structure soon.
I think @AshAman999 did a good job with folder structure and I also believe that we should wait for others suggestions.

This has gotten a bit stale, understandably, there were many other things with higher priority.
However, I think that it is still quite a big problem, I personally have trouble finding my way around the folders, instead I always search for the class names and find my way around, but for new contributors I imagine it to be much more difficult.
It's not a big refactoring per se, it's moving around files and fixing imports, the bigger part is finding a suitable system,
Here is an example of my suggestion (according to this article, it's called feature-first):

background
cache
database
widgets (very common widgets e.g. SmoothCard, SmoothScaffold, ...)
scanner

cards

settings

user_management
food_preferences
...

product_page

knowledge_panel

product_addition
product_edit
lists

cards

hunger_games
onboarding

The main point being, if I want to change something in the lists, I can work in the list's folder, why search together all the needed files in different folders if we can combine them.


npm package.json typo

There is a , on the last dependency:
"dependencies": {
    "mantra-core": "^1.2.0",
    "react": "^0.14.7",
    "react-dom": "^0.14.7",
    "react-komposer": "^1.3.0",
    "react-mounter": "^1.0.0",
    "react-simple-di": "^1.0.1",
    "eslint": "1.10.x",
    "eslint-plugin-react": "3.15.x",
  }

That doesn't allow npm to parse the json properly.

Fix is in master. Will be released in v0.3.6.


Please update for node pool deployment

Would be good to see how loadbalancer/ingress is set up with multiple node pools.

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

ID: c22b6ec6-7a27-4029-ff8f-674a40c0e552
Version Independent ID: bd2884f3-831a-3761-0502-5eb1594872bd
Content: Tutorial - Create a Kubernetes cluster with Azure Kubernetes Service (AKS) using Terraform
Content Source: articles/terraform/terraform-create-k8s-cluster-with-tf-and-aks.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

Thanks for the feedback! We are currently investigating and will update you shortly.

@ellispritchard I will assign this to the content author to review and see if we can add those steps/ create a doc around that.

@ellispritchard Thank you for the feedback. I'll take a look at this and talk to the engineering team about putting this in our backlog. I'll keep this issue open until we update you per our discussion.

This is a request for future articles and/or modifications to existing articles. It's not a documentation bug. #please-close.


Update how-to-mfa-number-match.md

clarify the MSCHAPv2 behavior

@justinha
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off