Fix for the intermittent URLSession Test Failures on async wait.

An intermittent test-failure is observed in TestURLSession.test_downloadTaskWithURLAndHandler where the test times out. JIRA bug: https://bugs.swift.org/browse/SR-4647
In URLSessionTask, the api ‘startNewTransfer’ https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/NSURLSession/NSURLSessionTask.swift#L456 calls the setter for ‘currentRequest’ https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/NSURLSession/NSURLSessionTask.swift#L142 which posts an async task of barrier type on the concurrent queue ‘taskAttributesIsolation’.
Further in the same api ‘startNewTransfer’, ‘configureEasyHandle’ api is called which calls the getter on ‘currentRequest’ https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/NSURLSession/NSURLSessionTask.swift#L570 which posts a sync task https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/NSURLSession/NSURLSessionTask.swift#L138 on the same concurrent queue ‘taskAttributesIsolation’.
Reading the dispatch_barrier_async api reference https://developer.apple.com/reference/dispatch/1452797-dispatch_barrier_async it says: “Calls to this function always return immediately after the block has been submitted and never wait for the block to be invoked. When the barrier block reaches the front of a private concurrent queue, it is not executed immediately. Instead, the queue waits until its currently executing blocks finish executing. At that point, the barrier block executes by itself. Any blocks submitted after the barrier block are not executed until the barrier block completes.”
In case of the test-failure, when the async task is posted no other tasks exist: Please refer to the stack-trace in the bug link: https://bugs.swift.org/browse/SR-4647
Instrumenting Foundation code, it is seen that the test is hung on the sync task and then fails with timeout.
Looks like the async barrier request task never completed on the dispatch queue which blocked the sync request task causing the timeout.
This fix uses the simpler dispatch semaphore to carry out the short critical sections in NSURLSessionTask to overcome the intermittent failure seen.

@swift-ci test

This work around looks good to me. Though parallel reads of the property will no longer be possible, it may not matter much because, as noted in the comment above, the critical sections are small - just one read/write.
Another point to note here is that though this particular failure does show up locally, quite consistently, on a 2CPU box, we do not know if this was the actual (and only) reason for the CI failures. Nevertheless, I think we should enable the URLSession tests with this and monitor the CI failures for a few weeks. Thanks.

I am merging this one now. Since it is enabling the URLSession tests we will have to monitor any CI failures over the next few weeks. Thanks.

@swift-ci test and merge


Make bunyan sentry stream compatible with node 0.12

Coverage remained the same at 100.0% when pulling 8a2896cc325dc61fccb4b30d552d243dba13f565 on Intelligration:feature/giles/node-compatibility into fdfb61b53465ac3686564f0e57bff07c82cdace2 on transcovo:master.

Whoops - wrong repo.


CompletionStage interop methods are misleading

I've been playing around with the interop methods that take CompletionStage and I think they can be misleading.
For example, a naive user could create code like CompletableFuturesNotOk where it expects future2 to be called after future1 has completed but this is not what will happen. Because both interop methods take eager CompletableStage instances, the moment they're called the futures will be launched right there and then. So it could easily happen that future2 is executed before future1 has completed.
The correct way to deal with this would be for the future2 to be launched when future1 has completed, this can be done as shown in CompletableFuturesOk.
I think you should have more generic different interop methods for CompletionStage that take lazy functions that create CompletionStage. For example, here's one I've created for BiFunction instances:
static <A, B, V> Maybe<V> biFutureToMaybe(A a, B b, BiFunction<A, B, CompletionStage<V>> f) {
   return new Maybe<V>() {
      @Override
      protected void subscribeActual(MaybeObserver<? super V> observer) {
         f.apply(a, b).whenComplete(
            (v, t) -> {
               if (t != null)
                  observer.onError(t);
               else if (v != null)
                  observer.onSuccess(v);
               else
                  observer.onComplete();
            }
         );
      }
   };
}

You could have just one method that takes Supplier<CompletionStage> but you'd often have users having to create lambdas that capture external variables.
Cheers,
Galder

Sounds like you actually misunderstood the CompletionStage API instead. Unlike the RxJava constructs, CompletionStage and CompletableFuture represent an ongoing or already terminated computation. In other terms they are eager and RxJava is generally deferred: one has to call subscribe to get the operations going.
RxJava has tools to make computation deferred: defer, fromCallable, flatMap, using, etc. For example, if there is a method that starts some computation and returns a CompletionStage of it, you can defer its creation/execution until subscription time, and thus its conversion to an RxJava type with defer for example:
Single.defer(() -> SingleInterop.fromFuture(methodReturningStage(params)))
.subscribe(/* ... */);

suggestion to improve/extend your

Your post didn't sound like a feature request but as an bug/complaint of the library deliberately/erroneously misleading the user about what the method does.

Also your biFutureToMaybe violates the Maybe protocol.

Also your biFutureToMaybe violates the Maybe protocol.

Through further testing I realised ^
The code I use now looks like this:
   static <A, B, T> Maybe<T> toMaybe(A a, B b, BiFunction<A, B, CompletionStage<T>> f) {
      return Maybe.create(source -> {
         f.apply(a, b).whenComplete(
            (x, t) -> {
               if (t != null) {
                  source.onError(t);
               } else if (x != null) {
                  source.onSuccess(x);
               } else {
                  source.onComplete();
               }
            }
         );
      });
   }

^ seems to work fine, or would you use Maybe.defer instead?

If you don't want the result to be cached then it is fine.

What do you mean the result being cached?


self host on runnpod

Hi, can you please add documentation to self-host on runpod? it offers GPU prices cheaper than aws.

Hi @Stealthwriter , we currently don't have any plans on supporting runpod. LLM Engine works on k8s, so if runpod supports k8s, then it can almost work - we'd need to find substitutes for S3.


python-sdk: Work in progress initial commit of a python sdk

Work in progress: Initial commit of all the features discussed in the open feature specification

@toddbaert @justinabrahms I think this initial implementation (without hooks and evaluation context) is now ready for final review.
If you're ok with the approach, we'd like to get this merged and add the hooks and context in separate PRs.
Let me know if you'd rather keep it all to a single PR though and we can do that instead.

I support merging this as is. :)

I support merging this as is. :)

As do I.


Could this mod be published on Modrinth?

I have multiples reasons I think it could benefits both users and you:

Modrinth is for what I know, a pretty good platform
Modrinth has it's own client and lot of people use it instead of curse, so posting it there, would allow other players to be able to use your mod.
Extra points and revenues for you
More people can enjoy the mod

(I'm not sponsored by them, It's just that because of some mods like you're not being avalaible on modrinth, it makes pack making harder, because you can only use modrinth mods on modrinth packs)

Thanks for the suggestion, I'll add this mod to Modrinth. Soon.

The mod is still being moderated.

https://modrinth.com/mod/achievements-optimizer


move contents of HACKING.md to a doc/HOWTO, and reference list of files

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

This change was a suggestion in #17926.

I just noticed that this file isn't referred to from anywhere...  is there a risk that it will get lost in oblivion?

I just noticed that this file isn't referred to from anywhere... is there a risk that it will get lost in oblivion?

Was HACKING.md referenced before?
It's at the top-level, so maybe it would get noticed, and it points at the other files.

I just noticed that this file isn't referred to from anywhere... is there a risk that it will get lost in oblivion?

Was HACKING.md referenced before? It's at the top-level, so maybe it would get noticed, and it points at the other files.

I hasn't been referenced before, we did indeed rely on its top-level location for it to be noticed.  This move makes it less directly visible...

I just noticed that this file isn't referred to from anywhere... is there a risk that it will get lost in oblivion?

Was HACKING.md referenced before? It's at the top-level, so maybe it would get noticed, and it points at the other files.

I hasn't been referenced before, we did indeed rely on its top-level location for it to be noticed. This move makes it less directly visible...

The file is STILL THERE.  It just references content elsewhere now.


iOS Application based on this API

Hi Team,
I would like to create an iOS App using this API. I am seeking your permission to use this API for the app. I am willing to create it as part of covid19india project.

No problem.


Create PCA_analysis.qmd

Performing PCA Analysis

I'll give this an initial review shortly!

This is a great start!  The code chunks in here look like they're missing the backticks (```) to indicate the start and end of code chunks and also the heading tags (e.g. #) for some of the text.  Mind adding those in please @royano19 ?

This is a great start! The code chunks in here look like they're missing the backticks (```) to indicate the start and end of code chunks and also the heading tags (e.g. #) for some of the text. Mind adding those in please @royano19 ?

Sure Let me check

The build is failing as we have a couple of missing package dependencies, but we can take a look at making a PR to your fork to update this as it's a little but of a weird process.  Will ping once we have something together!

yes I donno first time it is passing all the checks but now it's not working. strange

Hello! @royano19 @thisisnic i created a PR with the changes to the renv.lock that should fix the issue. https://github.com/royano19/CAMIS/pull/2
@royano19 if you merge it, you'll see the changes in this PR to the original CAMIS repo

@ines-gimeno-molina @thisisnic I have merged. Now what else I need to do?

The checks are now passing @royano19 @thisisnic

Thanks @ines-gimeno-molina . @thisisnic you can have a review.


Resolve day-1 experience related bugs

https://msazure.visualstudio.com/Unified Platform KPIs/_queries?tempQueryId=1399c6d3-39c4-4982-ba88-30a1a1705e47

[ ] #21488: Resolve day-1 experience related bugs - Key Vault
[x] #21480: Resolve day-1 experience related bugs - gremlin
[ ] #21479: Resolve day-1 experience related bugs - Event Hubs
[ ] #21476: Resolve day-1 experience related bugs - App Config
[ ] #21478: Resolve day-1 experience related bugs - Misc
[ ] #21477: Define solution for coordinating version numbers between "latest release" and "content on docs.microsoft.com"
[x] #21475:  Resolve day-1 experience related bugs - GitHub

Closing in favor of #21759


no-unused-labels

Current dependencies on/for this PR:

main

PR #130   👈

This comment was auto-generated by Graphite.


Fixes API change from 2015-02-18

This add backward compatibility and handles the current api's swap of 'cards' to 'sources'.

thanks @toadkicker!

I'm not too familiar with the stripe setup, but I'm getting an error on this: TypeError: Cannot read property 'data' of undefined.  It doesn't look like the customer object in the first expression is returning a cards property for me.
The customer.sources.data[0] does the trick for me though in any case, so I just swapped the order of the two expressions.

@benrlodge Stripe changed their API to 'sources' but as far as I could tell the rest of the data structure is the same. If customer.cards is false it should move onto try to set it as customer.sources. What's the node version you're running against?

It's not customer.cards that it's failing on, it's the data property on cards, customer.cards.data. Because thecards property doesn't exist, checking for any property on it will return undefined, as a reference is unresolvable if it's base is undefined. If we want to keep the expression check in that order, something like this should do the trick - var card = customer.cards ? customers.cards.data[0] : customer.sources.data[0];

https://github.com/eddywashere/node-stripe-membership-saas/pull/16


useHashPath: true {{pathFor}} won't render hash urls properly

I am on the file:// protocol, so i have no way to use history.pushState().
But if i set Iron.Location.configure({useHashPaths: true}); and use e.g. {{pathFor route='dashboard'}} it renders me /dashboard instead of #!dashboard

The problem seems to be that iron router changes the link to hashes when catching the click event. Though on the file:// protocol in chrome, it seems like this is not happening, instead it redirect to the actual URL immediately, So it would be better to actual render the correct link like #!dashboard.


Make language and translator configuration flexible

A proposal:
I am wondering if the configuration and translators could be configured so that for the most basic needs developers could set up default translators for specific language pairings. It's possibly a niche situation but Icelandic for example does not have great support in most machine translation services, and the specific local Icelandic ones have no support for translation between languages other than Icelandic to/from some other language.
Also, I think it would be valuable for developers to have a few default translation actions available to import and connect.
For example, it would be nice to have an action that auto-translates and publishes, and then another one that would auto-translate and save as a draft, etc.
So we would need to have some configuration options for developers to set a default translator class and a default action.
Then, for more specific needs, there could be a more detailed mapping between languages to specify a translator class and even the action to perform.
For the developer, this could look like something along the lines of:
WAGTAIL_TRANSLATE_DEFAULT_CLASS = "myapp.translators.DeepLTranslator" # None by default
WAGTAIL_TRANSLATE_DEFAULT_ACTION = "wagtail_translate.actions.translate_and_draft" # this could be the default
WAGTAIL_TRANSLATE_MAPPINGS = [ # default empty
    {
        "class": "myapp.translators.IcelandicTranslator",
        "pairs": ["is-en", "is-pl", "en-is", "pl-is"],
        "action": "wagtail_translate.actions.translate_and_publish",
    }
]

For those wanting to create their own actions, the translate mappings could help them pick the correct translator class, by using something like:
from wagtail_translate import get_translator_class
Translator = get_translator_class(source_lang, dest_lang)

Things to develop to achieve this

[ ] basic settings reading and applying defaults if the developer does not override them
[ ] A function get_translator_class that checks mappings and returns the correct translator class
[ ] create actions for the provided default use cases that pick the correct translator class based on source and destination language

Let me know what you think

Hi @saevarom,
Thank you for your proposal! Appreciated!
I acknowledge the need to simplify the onboarding story. The current situation is a hurdle. However, I believe that the flexibility offered by the signal receiver aligns better with the diverse requirements of multi-language projects. I just have to communicate the advantages better.
While additional settings might seem to simplify the initial setup, they can also make it harder to do additional customisations. The signal receiver provides developers direct access to the behaviours. Allowing them to implement their specific project requirements.
Going forward, I'll simplify the signal receiver example, and will add additional examples for:

Auto-translate and publish vs. auto-translate and save as a draft
Offloading to background workers
Custom Translator classes
Synchronized page tree
Storing the source locale on the translated object
Lock, translate, unlock

Your requirement for multiple translation services can be implemented like this:
# Your signals.py
from django.dispatch import receiver
from wagtail.models import Page, TranslatableMixin
from wagtail_translate.translators.deepl import DeeplTranslator
from myapp.translators.special import IcelandicTranslator
from wagtail_translate.signals import copy_for_translation_done  # For Wagtail < 6.2

@receiver(copy_for_translation_done)
def my_translation_done_receiver(sender, source_obj, target_obj, **kwargs):
    source_language_code = source_obj.locale.language_code
    target_language_code = target_obj.locale.language_code

    mapping = {
          ("en", "is"): IcelandicTranslator,
          ("is", "en"): IcelandicTranslator,
    }
    translator_class = mapping.get((source_language, target_language_code), DeeplTranslator)
    translator = translator_class(source_language_code, target_language_code)
    translated_obj = translator.translate_obj(source_obj, target_obj)

    if isinstance(translated_obj, Page):
        translated_obj.save_revision()
    else:
        translated_obj.save()

Does that make sense?

I made a PR introducing a default app.
https://github.com/allcaps/wagtail-translate/pull/14
Readme rendered
https://github.com/allcaps/wagtail-translate/blob/4bee3b81e015a99a42ba0ea0faa38831251bbb3d/README.md
Maybe you can have a look? And provide feedback?

Interesting, will take a look!


Rename TypeVariable, Arc and Handle type and info classes #2369

This also cleans up some the type test files.

Part of #2369


Simplify Tap Dance actions

The functionality added in tapdance.h was made primarily with Clip Studio Paint in mind, back when I used a pen display without its own shortcut keys.
Since then, I've upgraded to a model that has all the keys I need on its bezel, so the complicated tap dances (and the issues they bring with them) aren't really needed anymore.
I would like to keep the dual functionality tap/hold keys (quick Control+Z/X/C/V), but since the tap dance itself isn't needed anymore, see if it can't be simplified so that double or triple tapping one of those keys doesn't have to be a special case in the handling function.

Closed with d122d193febe3f209faf03ff49652ae1f66f7ee3.
All tap dance actions have been removed in the restructuring.


Bug? Outer iterations using W_init and H_init cause index error

Hello, me again - I may have come across a bug, though of course it could simply be a mistake on my end. I was trying to replicate a line of code from the matlab demo where iterations are forced by and outside for-loop (i.e., each internal max_iter =1) What is happening is that when I try to run many iterations with W_init and H_init set to the previous W and H, I immediately receive this following error:

File "/home/jsherman/miniconda3/envs/mfadae/lib/python3.7/site-packages/seqnmf/seqnmf.py", line 58, in seqnmf
X[mask] = X_hat[mask]
IndexError: boolean index did not match indexed array along dimension 1; dimension is 96 but corresponding boolean dimension is 102

I am wondering if it has to do with the line below
ln 44  T = X.shape[1] + 2 * L
My simple reason is that for my case L is 3, so 2*L is 6 which is the amount of the index error
The original matlab shows
  % zeropad data by L         X = [zeros(N,L),X,zeros(N,L)];         [N, T] = size(X);
Their iterative code is
[W, H, ~,loadings,power]= seqNMF(X,'K',K,'L',L,...
            'lambda', .00002,'lambdaL1W', 0, 'lambdaL1H', 0, ...
            'lambdaOrthoW', 0, 'lambdaOrthoH', 0.1, 'maxiter', 1, 'showPlot', 0); `

for iteri = 1:1000

    if iteri<20 || mod(iteri,10)==0
        clf; SimpleWHPlotSpectrograms(W(:,:,:),H(:,tstart:end), [], 0)
        set(gcf, 'color', [0 0 0])
        title(['Iteration ' num2str(iteri)]); drawnow
        cdata = print(gcf, '-RGBImage', '-r300');
        [w, h, ~] = size(cdata);
        cdata = reshape(cdata, w*h,3);
        cdata(mean(cdata,2)==255,:)=1;
        cdata = reshape(cdata,w,h,3);
        step(obj, cdata);
    end
    [W, H]= seqNMF(X,'K',K,'L',L,...
            'lambda', .00002,'lambdaL1W', 0.5, 'lambdaL1H', 0,...
            'maxiter', 1, 'showPlot', 0,...
            'lambdaOrthoW', 0, 'lambdaOrthoH', 0.1, 'W_init', W, 'H_init', H, 'SortFactors', 0);

Mine is
W, H, cost, loading, power = seqnmf(w_s, K=self.source_num, L=3, Lambda=0.001, lambda_L1W=0, lambda_OrthH=0,\
            max_iter=1, sort_factors=0)
for i in range(0, 1000):
    W, H, cost, loading, power = seqnmf(w_s, K=self.source_num, L=3, Lambda=0.001, lambda_L1W=0, lambda_OrthH=0,\
    W_init=W, H_init=H, max_iter=1, sort_factors=0)

This could well be a bug-- as I had mentioned, this library hasn't been debugged or tested with any sort of rigor.  For what it's worth, I think the right way to set up some initial tests would be something like:

Load in the sample dataset from the MATLAB toolbox
Modify the MATLAB code so that it saves the output of each line
In the Python version, load in the same sample dataset, and verify that every line of the Python version matches each saved output from the MATLAB version


Version 1.0.5: can't extract "big" files

I've seen that there are some open issues that have something to do with the extraction of "big" files.
I've read that the problem has been introduced in 1.0.3 and that a revert on a commit (callback changes #37) was performed so that the problem is now solved.
I'm using version 1.0.5 but the problem is still there.
My .tar.gz archive contains a 537,60 KB PDF file that shows the problem: only 407 KB (exactly 416768 bytes) gets extracted, then the .extract call stops.

I've investigated and I've found the following (maybe I'm wrong... but I'll give it a try):

the revert changed line 59 (index.js) from
this._bubble(stream2, stream1);
to
this._bubble(stream1, stream2);

line 59 (index.js) in master is
this._bubble(stream1, stream2);

line 59 (index.js) inside the 1.0.5 release seems wrong
this._bubble(stream2, stream1);

It seems that the release has been created before the revert... Is that the case?

Solved: I've changed line 59 (index.js) on my production server with no luck. Downgrading to version 1.0.2 solved my problem. I'll stay there for a while.

+1
Also downgraded to 1.0.2 after noticing that – sometimes, not always – extracting an archive failed.


Changed queue polling in _run_function_in_parallel(), reworked upgrade_tools_all() to run tools upgrades in parallel.

Needs some testing.  Firing multiple upgrade tools actions seems to cause sporadic connection issues with provider (current work around is to try to fire event twice before throwing error).

@rmgoodberlet Testing still needs to be done on this since it's still failing in parallel. Not sure if this is the best way to do it. Closing this PR. Let's discuss more about it at work.


3026-3126 CSN/H Suggestion

Hello shindimitry,
Thanks for the great work, i do appreciate a lot. But i wonder is it possible to change messages "60 degrees (Fire) to 60 Degrees Take Ice (or just "Take Ice") and 40 degrees (Ice) to 40 Degrees Take Fire (or just "Take Fire" ). I know not much really needed. Buıt instead of what it is (cuz it is obvious that boss gives fire and its able to see the degree bar is high or low), that might be better. Just a suggestion tho o/

Hi,
This messages simply informs that there will be an AOE applying fire or ice to all players. Which debuff to take to whom depends on the situation. In hard mode it is not always necessary to change your debuff. Therefore, asking everyone to change the debuff at once is incorrect (in particular, for hard mode).

May be added an additional message to notify that the temp is rising or falling.

Yeah might last notice can be as well as you said.

This messages added in the latest update.


Feature: Add awslabs/ssosync

This package also provides releases for Windows platform, however Windows releases are packaged using zip not tgz and I don't see support for multiple install types or multiple urls
Should I also update cache, or is that done separately?

cache has been updated; you can binenv update && binenv install ssosync


Test against multiple Rails versions

I've added an Appraisals file (and generated all of its gemfiles) to enable testing against all of the currently-supported Rails versions (as per Rails' Maintenance Policy).
I added these to the Travis config (so we'll see if they pass 😅).
I also added a default rake task since there wasn't one previously. This makes it easier to test locally -- bundle exec rake will run the spec task against all appraisals -- and also works on Travis for each individual build combination.

I'm gonna close this due to inactivity.


Run Arguments

Mr's.
After new update;
the use of "Run Arguments" Stopped working in "toggle run options" for set arguments.
Att,
Wesly Silva.
ENG Software.

opps
sorry that i replaced the input element with atom-text-editor element in the last update but forgot to change the getter setter of the element text content in run option view
i will fix it as soon as possible

Thank you Mr. Lau.
his solution was quick and efficient.
sorry not to have given so fast feedback when your :)
2016-05-22 13:02 GMT-03:00 Lau Cheuk Yin<EMAIL_ADDRESS>
Closed #6 https://github.com/tomlau10/gcc-make-run/issues/6 via 20cef04
https://github.com/tomlau10/gcc-make-run/commit/20cef04aee0c428eac80a9a38e46eb0aedfc4c39
.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/tomlau10/gcc-make-run/issues/6#event-668247662


community: Remove targets_table from C* GraphVectorStore

Description: Remove the unnecessary targets_table parameter

@cbornet

This looks like a breaking change. I realize that you're managing this integration, so merging in. If dangerous you could also raise a deprecation error next time and/or a warning that the parameter has been removed.

We ensured a deprecation path for the code. In new versions of ragstack-ai-knowledge-store, targets_table is allowed at init and ignored and will output a deprecation warning.
It's a breaking change on the implementation schema though. We think it's still possible to do it at this stage.


Interview - Summary

Interview - summary of following interviews:
Interviews:

Interview 1 - AutoDoc

Interview 2 - AutoDoc

Interview 3 - AutoDoc

Interview 4 - AutoDoc

Interview 5 - AutoDoc

Interview 1 - TypeDesigner

Interview 2 - TypeDesigner

Interview 3 - TypeDesigner

Interview 4 - TypeDesigner

Interview 5 - TypeDesigner

First draft in youtrack: https://youtrack.sprinteins.com/articles/TBW-A-59/ADR-Interview-Summary
After Feedback will be published to Zulip.

Published to Zulip. Final Summary here: https://github.com/sprinteins/oscd-plugins/issues/170


Add indicator that a vote caused the proposal to be approved.

Is your feature request related to a problem? Please describe.
It's difficult to find which vote caused a proposal to be approved. Can we add an icon or indicator that a vote caused the proposal to be executed?

Just thinking - I suppose this should be the last vote in proposal and we know its timestamp so I think we can handle this

Implemented


migrate whole library to new arch

Description
PR with all the changes for migration to new arch gathered together.
Checklist

[ ] I have tested this on a device/simulator for each compatible OS
[ ] I updated the documentation with running yarn generate in the root folder
[ ] I added/updated a sample - if a new feature was implemented (/example)

Screenshot OR Video

@WoLewicki thank much for the effort. I'm a bit concerned about the amount of changes. It will be very hard to review/manage this.
Also please use RNMBX prefix instead of MBX as MBX prefix is already used  by some Mapbox components.

This PR is just a draft with all the changes I make. We will for sure split it to smaller PRs so it is much easier to test and review. I'll also change the names of components 🚀

This PR is just a draft with all the changes I make. We will for sure split it to smaller PRs so it is much easier to test and review. I'll also change the names of components 🚀

Cool that sounds great. One issue is that I'm working on supporting the v11 api. See #3040, not sure what's the best way to make merges easier for both of us.

I think we can handle it in each component one-by-one when migrating them.

I think we can handle it in each component one-by-one when migrating them.

👍

Closing in favor of many other PRs based on this one :rocket:


interp: false positive for redeclaration after failed assignment

% yaegi
> f := myawesomefunc
1:28: constant definition loop # as expected
> func myawesomefunc() {
println("I Exist")
}
1:14: myawesomefunc redeclared in this block # unexpected

Actually, this is somehow already fixed by #802 , i.e. with yaegi compiled on top of this PR:
% yaegi
> f := myawesomefunc
_.go:1:28: constant definition loop
> func myawesomefunc() {
println("I Exist")
}
: 0xc0000a4530
> f()
_.go:1:28: undefined: f(/_.go)
> myawesomefunc()
I Exist
: 0xc0004103b0
>
>
> f = myawesomefunc
_.go:1:28: undefined: f(/_.go)
> f := myawesomefunc
: 0x10b7970
> f()
I Exist
: 0xc000178750


Remove RequirementsVerifier

Description

This removes the RequirementsVerifier, which prevents startup when the application is presented with fewer resources than the minimum requirements.
The API server is now able to run on much fewer resources than the vanilla version. Refusing startup when presented with "too few" resources is not necessary anymore.
Addressed Issue

N/A
Additional Details

N/A
Checklist

[x] I have read and understand the contributing guidelines
~This PR fixes a defect, and I have provided tests to verify that the fix is effective~
~This PR implements an enhancement, and I have provided tests to verify that it works as intended~
~This PR introduces changes to the database model, and I have added corresponding update logic~
~This PR introduces new or alters existing behavior, and I have updated the documentation accordingly~

@VithikaS Ha, good spot! Removed it, and also the RequirementsException class that is no longer needed.
In fact, I found that none of the classes in the exception package are actually used anywhere. So more code to delete, yay!


Network: Fix regression in GetNetworkLoadBalancers

Introduced with df55120483f11031266ab67c5a35f977813e11db
Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Build finished.

jenkins: test this please

jenkins: test this please


TP09: Switch "Automatic Mode" missing

Hello, in the past I had the Dyson Purifier Cool (TP07), which I connected using this integration.
After a repair of the TP07, Dyson could only provide me a Dyson Purifier Cool Formaldehyde (TP09) as replacement.
Now I connected this TP09.
Now I am missing the switch for the automatic mode.

Is there a chance to get the switch active?
Cheers, red

I beleive you have to click on the button "Dyson Fan" and auto will appear in a drop down menu.

Nope, no dropdown, no auto mode.
These are the entities that are available in Home Assistant:

BTW: The formaldehyde sensor (which is specific to this device) is even not available.

It's kind of buried but it should be there. This is also a formaldehyde machine but I don't see formaldehyde numbers either.

Hi @Anto79-ops, you made my day!!
I really found the dropdown you've mentioned. It was a bit hidden. Therefore I just attach this little instruction as a gif for others that even don't find it.

Thanks!!

And for people who are looking for a way to switch on the auto mode:
service fan.turn_on provides some settings to turn on the auto mode, e.g.
data:
  preset_mode: Auto
target:
  entity_id: fan.dyson_fan

or

The provided solution seems more like a workaround. Could we get an actual entity to set/toggle the preset mode without having to go into the device settings or creating an automation?


no mc edit for mc 1.5.X?

I need a version for mc 1.5.X for a project and i thoght it existed but i cant seem to find it .
can anyone send me a link or tell me if it exists?

The latest version will probably work it just hasn't been tested in that version so there might be problems. Back up your world and give it a try.


Implement support for Highlight annotations

Fixes #5018.
Highlight annotations are basically nothing more than the already implemented Text annotations. Therefore a Highlight annotation can be represented as a Text annotation according to the PDF specification at https://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/pdf_reference_1-7.pdf#page=633&zoom=auto,-306,268.
Note that for the issue mentioned in this PR the annotations itself are fixed with this PR, but the annotation content is rotated. This is a separate issue (due to the transform on annotText that also affects child elements such as the annotation content) and I think it also related to #3826. This should be filed as a follow-up issue.

Any news?

No news yet, someone has to review #5585 before I can continue with my work on the annotation layer.

@timvandermeij Will the work you do on the annotation layer fix the issue described in #3826?
We experience the same issue with PDFs including highlighting (it is rotated).
Highlighting is very important in our use case.

No, my work is more focused on refactoring. This is a bugfix, so I'm not currently working on that.


BCEL-268: Add StackMapTable-Generator

I used some code from the ASM-framework which already supports Stackmaptables and applied small changes to fit the BCEL-framework.
The code is not complete yet. There are many things to do, like implementing support for arrays and static fields/functions.
I would love any kind of feedback<EMAIL_ADDRESS>)

Sorry, by copying code without appropriate note you are violating ASM license.
Open source != copy/paste.

Thanks for the review.
Complying with the ASM license should be easy. It's not more than copy/paste their license on top of the source.

@iloveeclipse thank you for having a look at this.
@JanUlrich this is a big addition. I need to ask you to sign an Individual Contributor License Agreement as described in CONTRIBUTING.md. Please make sure you don't violate IP of other parties.

@JanUlrich @iloveeclipse I have created https://issues.apache.org/jira/browse/LEGAL-267 to make sure we're not violating any licenses here.

@JanUlrich please reach out to me if you're still interested in getting this into BCEL. Our legal team gave some advices on this contribution:

if the code is copied from ASM, we need attribution in NOTICE.txt
if the code has been changed, we need to make clear which parts have been changed
it would be best to depend on ASM code instead of coping it

Given what you said earlier:

Nearly all of the copy/paste code was modified.
Only parts of the switch-case-statement in the "applyStatement" method is from the ASM-codebase.
In retrospect I had to alter nearly every single line of the copied code. If I change the switch-case to a if-else not a single line would be the same as in the ASM-codebase.
Majority of the code was added by me based on the ASM-documentation about stackmaptables (see: [1])

It sounds to me that this is "new" code, which was written in the spirit of the ASM documentation. In this case the files only need an ASF header. Furthermore we can add attribution in JavaDoc and in NOTICE.txt.
In any case, if you want this major contribution to be included, I have to ask you to sign an Individual Contributor License Agreement.
Thank you!

Benedikt Ritter:

@JanUlrich please reach out to me if you're still interested in getting this into BCEL.

I'm definitely interested!
But there is no hurry. Some functionality is still missing in the code
and there are propably some bugs to fix and tests to write.

In any case, if you want this major contribution to be included, I have to ask you to sign an Individual Contributor License Agreement.

I'm on it. I will send a signed ICL Agreement to<EMAIL_ADDRESS>

@JanUlrich it is still not clear to me where the code is coming from. Did you write it from scratch based on the documentation on the ASM website or is it copy pasted from ASM?


Could not get unknown property 'DEFAULT_NDK_API_LEVEL'

Hi!
When I try to build my react-native app for android I get this error:

FAILURE: Build failed with an exception.

Where:
Build file<EMAIL_ADDRESS>line: 89

What went wrong:
A problem occurred evaluating project ':@emurgo_react-native-haskell-shelley'.

Could not get unknown property 'DEFAULT_NDK_API_LEVEL' for object of type com.nishtahir.CargoExtension.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

I'm using version ^4.0.0-beta.3 of react-native-haskell-shelley.
Any ideas what could be the issue here? Thanks!

Go to Cardano page to report this issue for the live representative to review this and resolve via Live Chat

Hi @PiotrNap! Check please env configuration from https://github.com/Emurgo/yoroi-mobile , also how do you build it for android ?

Hi @lisicky . I'm sorry for the late response, life got a bit busy.
I run react-native run-android to build the app. I was able to solve the above issue by adding androidXAnnotation = "1.1.0" to my build.gradle file :)
However, now I'm faced with a different problem. The build process stops at this error message:

error: failed to download base64 v0.20.0
Caused by:
unable to get packages from source
Caused by:
failed to parse manifest at /home/piotr/.cargo/registry/src/github.com-1ecc6299db9ec823/base64-0.20.0/Cargo.toml
Caused by:
failed to parse the edition key
Caused by:
supported edition values are 2015 or 2018, but 2021 is unknown
FAILURE: Build failed with an exception.

I've tried upgrading Rust to the latest version and changing edition to 2021 inside react-native-haskell-shelley/rust/Cargo.toml but that didn't helped.
Do you have any suggestions as to what else I could try?
I'd highly appreciate it!

@PiotrNap try to set any 1.6X version (1.63 by rustup override set 1.63.0 for example)  https://doc.bccnsoft.com/docs/rust-1.36.0-docs-html/edition-guide/rust-2018/rustup-for-managing-rust-versions.html
If it won't help, try to clone and compile CSL from source and provide information about installed rust versions.

Hi, my apologies for late response. I completely forgot about this ticket after some time. Updating the rust version and making sure all required targets are installed did the trick. I'll close this one.
Thanks for your help!


playerProfile calls returning 500

nba.stats.playerProfile({
  PlayerID: 201939
}).then(profile => {
  // do something
}, error => {
  console.log(error)
})

i'm seeing an error like
[Error: 500 Internal Server Error – {"Message":"An error has occurred."}]

i'm guessing the API changed underneath you, but i haven't done any digging yet.
thanks nick!

😞 Yea I noticed this too – it's causing the test suite to fail. I'll take a spin through stats.nba.com and see if I can find them doing a valid request. Usually the error messages are pretty descriptive (they'll tell you which field is missing or malformed) but this one is a head-scratcher.

ha! yeah, it doesn't give you much to work with.
i just clicked around stats.nba.com a bit, and found a request to http://stats.nba.com/stats/playerprofilev2?LeagueID=00&PerMode=PerGame&PlayerID=203076
since they're using a v2 url now, i'm guess this change affects nba-client-template instead of this repo
hope this helps! ⭐

nice find -- thanks!!

fixed & published in 3.1.1 -- @brandly thanks for the assist 🏀
(travis seems to be flaking out on something unrelated -- the actual tests are passing)


[FLINK-31513][DOC] update a path in README

https://issues.apache.org/jira/browse/FLINK-31513
Documentation

Does this pull request introduce a new feature? (yes / no) no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) doc

@flinkbot run azure


Support Top-level Await

Is Your Feature Request Related to a Problem? Please Describe
Beaker's APIs use Promises extensively. Asynchronous IIFEs are very common in Beaker websites to take advantage of await syntax during initialization logic.
Solution You'd Like
Top-level await. Currently a Stage 3 ECMAScript Proposal, top-level await has experimental support in V8.

For platform features like this, I think the best thing we can do is wait for Chromium and then Electron to implement it -- but if anybody knows how we can enable it sooner, I'm open to it

Top level await is currently being formalized in the HTML spec and browser vendors. It'll probably be easy to get into Beaker once electron gets it through Chromium.
https://github.com/whatwg/html/pull/4352#pullrequestreview-425632594


ResolveTagHelperRazorGenerateInputs target gets invoked even when no inputs change

This was causing the RazorTagHelper task to run even when there were no changes to any csharp files.

c2932674216e0dd72b60991fa303fbfd8867d3b1


[495] Setup Vagrant for development

Story / Bug id: https://github.com/CodeForPoznan/volontulo/issues/495
Reference person: @riv2q
Description:
Adding application setup option via vagrant
New imports / dependencies:

psycopg2>=2.6
pytz==2016.7

What tests do I need to run to validate this change:

vagrant up
vagrant ssh
./manage.py runserver <IP_ADDRESS>:8000 --settings=volontulo_org.settings.dev

Travis's tests are failing here.

@riv2q
You've changed dev's settings, that they use Postgres. Please, create separate settings file for Vagrant.
See:
1.01s$ coverage run --source='./apps' manage.py test --settings=volontulo_org.settings.dev -v 3
nosetests -v --verbosity=3
nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
Creating test database for alias 'default' ('test_volontulo')...
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.4.2/lib/python3.4/site-packages/django/db/backends/base/base.py", line 130, in ensure_connection
    self.connect()
  File "/home/travis/virtualenv/python3.4.2/lib/python3.4/site-packages/django/db/backends/base/base.py", line 119, in connect
    self.connection = self.get_new_connection(conn_params)
  File "/home/travis/virtualenv/python3.4.2/lib/python3.4/site-packages/django/db/backends/postgresql_psycopg2/base.py", line 176, in get_new_connection
    connection = Database.connect(**conn_params)
  File "/home/travis/virtualenv/python3.4.2/lib/python3.4/site-packages/psycopg2/__init__.py", line 164, in connect
    conn = _connect(dsn, connection_factory=connection_factory, async=async)
psycopg2.OperationalError: FATAL:  role "volontulo" does not exist

will do

Current coverage is 91.26% (diff: 100%)

No coverage report found for master at 7ba096e.

Powered by Codecov. Last update 7ba096e...2e4ad17


Local storage fix

Fix for case when we might encounter Put being called multiple times on the same output file.
The issue was that first time Put was called it hard links the file. The second time its called hard linking fails since the dest already exists. This results in a fallback to copyFile  where os.Create is called on dest thus resulting in the truncation of the source file (since source and dest refer to the same inode).

My concern here is that these two Put() calls could be from two different tasks, right? What's the expected behavior of two tasks writing to the same file, sequentially? I would expect the second to overwrite the first, although I could also see a case for raising an error, but if I understand correctly, this code does neither of those options.

os.SameFile is checking that the two inode references are the same.   If they are not then the file will be overwritten.

test was updated to make things more clear.


HM Configuration Tool LAN not status

Hallo,
is there an problem with the "HM Configuration Tool LAN" Adappter in the latest version ? The Status is not Displayed in Version <IP_ADDRESS>71022.
I have  3 of these adapters, byed bevor the Langateway was on the marktet ! Is this Device supportet now?
And i tried to delete the 3 devices reboot and add it new, no luck.
greedings
Jake
PS: I tried to add it as Lan Gateway (read this in an foum) to test but don't work also!

Are you only using the HM-LAN-CFG without HM-MOD-UART? If yes, please read https://github.com/jens-maus/RaspberryMatic/issues/177, the issue is fixed and close this one. If not, please give more details about your structure and some logs here.

hello,
yes i have only the 3 Lan gateways, but i don't have an error message when i configure an device.
Short way, YES this works on my System too, But if i restart the PI the file still is changed back to default and only the Name "Virtual Devices " is filled in.
tanks
Jake
PS: Try to by the hm-occu-sdk module


Subsequent calls on ScaleTransition causes element to be invisible

When calling animationController.forward() and then animationController.reverse() multiple times in a row, the element ends up not being drawn.

Minimal app
import 'package:flutter/material.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(

        primarySwatch: Colors.blue,

        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Input(),
      ),
    );
  }
}

class Input extends StatefulWidget {
  Input({Key key}) : super(key: key);

  @override
  _InputSate createState() => _InputSate();
}

class _InputSate extends State<Input>
    with TickerProviderStateMixin {
  AnimationController _animationControllerCheckMark;
  AnimationController _animationControllerSend;
  Animation<double> _animationCheckmark;
  Animation<double> _animationSend;

  bool isAnimating = false;

  @override
  void initState() {
    _animationControllerCheckMark =
        AnimationController(vsync: this, duration: Duration(milliseconds: 150));
    _animationControllerSend =
        AnimationController(vsync: this, duration: Duration(milliseconds: 150));
    _animationCheckmark =
        Tween<double>(begin: 0, end: 1).animate(_animationControllerCheckMark);
    _animationSend =
        Tween<double>(begin: 1, end: 0).animate(_animationControllerSend);
    super.initState();
  }

  void sendMessage() {
    isAnimating = true;
    _animationControllerSend
        .forward()
        .then((value) => _animationControllerCheckMark.forward());

    Future.delayed(Duration(seconds: 2), () {
      _animationControllerCheckMark
          .reverse()
          .then((value) => _animationControllerSend.reverse().then((value) {
                isAnimating = false;
              }));
    });
  }

  @override
  Widget build(BuildContext context) {
    return RaisedButton(
        onPressed: sendMessage,
        child: Stack(alignment: Alignment.center, children: [
          ScaleTransition(
              key: Key("ScaleTransition1"),
              scale: _animationCheckmark,
              child: Icon(Icons.check)),
          ScaleTransition(
              key: Key("ScaleTransition2"),
              scale: _animationSend,
              child: Text("SEND")),
        ]));
  }

  @override
  void dispose() {
    _animationControllerCheckMark.dispose();
    _animationControllerSend.dispose();
    super.dispose();
  }
}

Logs

[√] Flutter (Channel stable, 1.22.5, on Microsoft Windows [Version 10.0.19041.804], locale fr-FR)
    • Flutter version 1.22.5 at E:\Flutter
    • Framework revision<PHONE_NUMBER> (2 months ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\MrJ\AppData\Local\Android\sdk
    • Platform android-30, build-tools 29.0.2
    • Java binary at: E:\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[!] Android Studio (version 4.1.0)
    • Android Studio at E:\Android\Android Studio
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[!] Connected device
    ! Device emulator-5554 is offline.

! Doctor found issues in 2 categories.

Somehow screen recording doesn't work on my emulator, if someone would be so kind and post the result, that would certainly make things clearer. Just start the app and click the button everytime you see "SEND".

Hi @themadmrj
Here I tried several times, nothing odd
https://user-images.githubusercontent.com/48603081/108728099-3f4fe980-754f-11eb-9eef-9d8291eaa10f.mp4

flutter doctor -v
  ╔════════════════════════════════════════════════════════════════════════════╗
  ║                 Welcome to Flutter! - https://flutter.dev                  ║
  ║                                                                            ║
  ║ The Flutter tool uses Google Analytics to anonymously report feature usage ║
  ║ statistics and basic crash reports. This data is used to help improve      ║
  ║ Flutter tools over time.                                                   ║
  ║                                                                            ║
  ║ Flutter tool analytics are not sent on the very first run. To disable      ║
  ║ reporting, type 'flutter config --no-analytics'. To display the current    ║
  ║ setting, type 'flutter config'. If you opt out of analytics, an opt-out    ║
  ║ event will be sent, and then no further information will be sent by the    ║
  ║ Flutter tool.                                                              ║
  ║                                                                            ║
  ║ By downloading the Flutter SDK, you agree to the Google Terms of Service.  ║
  ║ Note: The Google Privacy Policy describes how data is handled in this      ║
  ║ service.                                                                   ║
  ║                                                                            ║
  ║ Moreover, Flutter includes the Dart SDK, which may send usage metrics and  ║
  ║ crash reports to Google.                                                   ║
  ║                                                                            ║
  ║ Read about data we send with crash reports:                                ║
  ║ https://flutter.dev/docs/reference/crash-reporting                         ║
  ║                                                                            ║
  ║ See Google's privacy policy:                                               ║
  ║ https://policies.google.com/privacy                                        ║
  ╚════════════════════════════════════════════════════════════════════════════╝
[✓] Flutter (Channel stable, 1.22.6, on macOS 11.2.1 20D74 darwin-x64, locale en-GB)
    • Flutter version 1.22.6 at /Users/tahatesser/Code/flutter_stable
    • Framework revision 9b2d32b605 (4 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5
[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, set ANDROID_SDK_ROOT to that location.
      You may also want to add it to your PATH environment variable.
[!] Xcode - develop for iOS and macOS (Xcode 12.4)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods
[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
[✓] VS Code (version 1.53.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.19.0
[✓] Connected device (1 available)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E • ios • iOS 14.4
! Doctor found issues in 3 categories.

I cannot reproduce this issue on the latest stable channel.
Can you please upgrade to the latest  stable and try again
flutter channel stable
flutter upgrade --force
flutter doctor -v
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


nile send failing with "TypeError: expected str, bytes or os.PathLike object, not int"

Hello!
Using MacOS with Python 3.8.5,
nile version 0.4.0
I have exported a environment variable TEST_PRIV to 123 and successfully run nile setup TEST_PRIV with the following output:
$ nile setup TEST_PRIV
🚀 Deploying Account
⏳ Deployment of Account successfully sent at 0x0636ab47cf615028c92a64e9aa9630d5bc93a335aa0ae174fbce2e3f22773cfc
🧾 Transaction hash: 0x40b2b24dd47c1d86079c9c8ecb1f30d556c6132e1247ef7063a82854f63a3fd
📦 Registering deployment as account-0 in localhost.deployments.txt
(base)

now I wanted to interact with a deployed contract and hence running nile send
nile send TEST_PRIV token1 mint 0x0636ab47cf615028c92a64e9aa9630d5bc93a335aa0ae174fbce2e3f22773cfc 1000

and I get the following error:
Traceback (most recent call last):
  File "/Users/redacted/opt/anaconda3/bin/nile", line 8, in <module>
    sys.exit(cli())
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/nile/cli.py", line 96, in send
    account.send(contract_name, method, params)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/nile/core/account.py", line 62, in send
    call_or_invoke(
  File "/Users/redacted/opt/anaconda3/lib/python3.8/site-packages/nile/core/call_or_invoke.py", line 40, in call_or_invoke
    subprocess.check_call(command)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/subprocess.py", line 359, in check_call
    retcode = call(*popenargs, **kwargs)
  File "/Users/redacted/opt/anaconda3/lib/python3.8/subprocess.py", line 340, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/Users/redacted/opt/anaconda3/lib/python3.8/subprocess.py", line 854, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/Users/redacted/opt/anaconda3/lib/python3.8/subprocess.py", line 1637, in _execute_child
    self.pid = _posixsubprocess.fork_exec(
TypeError: expected str, bytes or os.PathLike object, not int
(base)

Could any one provide any pointers on how to fix this please? thank you!
Please also let me know if any other information would be helpful for debugging, happy to provide!

same issue happens here

Fixed on https://github.com/OpenZeppelin/nile/pull/77.


Fix running api server locally

I downloaded this PR and Porch now runs locally for me.

I downloaded this PR and Porch now runs locally for me.

With the upgrade k8s apiserver, the ApplyTo(config *server.RecommendedConfig) requires that the *restclient.Config be set.
https://github.com/kubernetes/apiserver/blob/master/pkg/server/options/recommended.go#L123
With how the run is standalone-debug mode was being set up, the RecommendedOptions.CoreAPI was being set to nil
https://github.com/nephio-project/porch/blob/main/pkg/cmd/server/start.go#L128
meaning that the default restclient would not be setup.
https://github.com/kubernetes/apiserver/blob/master/pkg/server/config.go#L318
So the fix was to remove the nil set above.

Thanks @efiacor

/approve
/lgtm


Could not find a declaration file

Confirmed the contents are in node_modules: @slipmatio/control-knob/...
<template>
  <div class="app">
    <control-knob />
  </div>
</template>

<script>
import ControlKnob from '@slipmatio/control-knob'

export default {
  components: {
    ControlKnob
  }
}
</script>

It renders like this:

I tried both .js and .ts Vue projects. The only issue I see but couldn't resolve is:

Could not find a declaration file for module '@slipmatio/control-knob'. <EMAIL_ADDRESS>implicitly has an 'any' type.
Try npm i --save-dev @types/slipmatio__control-knob if it exists or add a new declaration (.d.ts) file containing declare module '@slipmatio/control-knob';Vetur(7016)
Could not find a declaration file for module '@slipmatio/control-knob'. <EMAIL_ADDRESS>implicitly has an 'any' type.
Try npm i --save-dev @types/slipmatio__control-knob if it exists or add a new declaration (.d.ts) file containing declare module '@slipmatio/control-knob';ts-plugin(7016)

The error is not related. I finally noticed that the default state has everything in the same color. Following the example, different colors did the trick.
 <ControlKnob
    v-model="value"
    svg-class="select-none"
    bg-class="text-[#868686]"
    rim-class="text-[#393939]"
    value-arch-class="text-[#53d769]"
    tick-class="text-black"
    value-text-class="text-gray-50 text-[30px] font-normal font-mono" />

This is wrong as well. Ignore my confusion. The configuration should be in this form.
 <ControlKnob
      v-model="knobValue"
      :options="{
        imageSize: 100,
        minValue: 40,
        maxValue: 80
      }"
    />


Test import_short_absolute fails on Windows

The commit
https://github.com/dart-lang/sdk/commit/227b74cb55e441dbf39b43815187c166cb2570de
introduces a test in pkg/analyzer/test/src/summary/resynthesize_common.dart that fails on Windows.

Working on it.

This is tricky.  The test which was added asks analyzer to analyze a file containing an import statement of the form:
import "/a.dart";

The analyzer converts this to the URI file:///a.dart, and then attempts to use the path package to convert it to a file path.  On Linux and mac, the path package converts this back to /a.dart, which seems reasonable.  On Windows, it converts it to a.dart, which seems wrong, because a.dart is a relative path.
Note that the Uri.toFilePath(windows: true) method has different behavior: it converts file:///a.dart to \a.dart.  I'm not sure if that's any more meaningful than a.dart (since it's an absolute path without a drive letter), but at least it doesn't look like an absolute path.
I will file a bug against the "path" package and commit a temporary workaround that disables this test on Windows.


Output of link checker workflow is not publicly visible

For osunyorg/idn-website#7 we have integrated a link checker workflow at the end of the build pipeline, to check deployed artifacts for abandoned references.
Unfortunately it has turned out that these are not visible to the content maintainers of the website, who do not posess GitHub credentials to view the results.
Lucky us the action used has a function to surface the results as public issues, which people can then act upon.

The current implementation posts check results back to the PR that triggered them:
https://github.com/degrowth/idn-website/pull/23#issuecomment-2207047293


Connect GitLab to Casdoor with LDAP

Hello!
I am trying to connect GitLab to Casdoor with LDAP since this way UserSync available. There is issues on the way, but I hope you are interested in success same as me. Each problem by in separate issue.
During login process GitLab makes such request - see info in GitLab documentation
LDAPMessage searchRequest(4) "uid=3bc2c5de-058d-4888-a862-ff04aaa8e442,cn=user_btkqev,ou=GROUP,dc=sso,dc=DOMAIN,dc=com" baseObject
    messageID: 4
    protocolOp: searchRequest (3)
        searchRequest
            baseObject: uid=3bc2c5de-058d-4888-a862-ff04aaa8e442,cn=user_btkqev,ou=GROUP,dc=sso,dc=DOMAIN,dc=com
            scope: baseObject (0)
            derefAliases: neverDerefAliases (0)
            sizeLimit: 0
            timeLimit: 0
            typesOnly: False
            Filter: (userAccountControl:1.2.840.1135<IP_ADDRESS>3:=2)
                filter: extensibleMatch (9)
                    extensibleMatch userAccountControl
                        matchingRule: 1.2.840.1135<IP_ADDRESS>3
                        type: userAccountControl
                        matchValue: 2
            attributes: 1 item
                AttributeDescription: dn

In response Casdoor closes connection with messages in the logs:
2024/10/07 18:09:16 Connection client [81] from <IP_ADDRESS>:55506 accepted
2024/10/07 18:09:16 <<< 81 - BindRequest
2024/10/07 18:09:16 >>> 81 - BindResponse
2024/10/07 18:09:16 <<< 81 - SearchRequest
2024/10/07 18:09:16 ******************************
2024/10/07 18:09:16 **** ROUTE MATCH ;  SEARCH****
2024/10/07 18:09:16 ******************************
2024/10/07 18:09:16 >>> 81 - SearchResultDone
2024/10/07 18:09:16 Error readMessagePacket: EOF
2024/10/07 18:09:16 client 81 close()
2024/10/07 18:09:16 client 81 close() - stop reading from client
2024/10/07 18:09:16 client 81 close() - Abandon signal sent to processors
2024/10/07 18:09:16 client [81] request processors ended
2024/10/07 18:09:16 client [81] connection closed
2024/10/07 18:09:16 Connection client [82] from <IP_ADDRESS>:55518 accepted
2024/10/07 18:09:16 <<< 82 - BindRequest
2024/10/07 18:09:16 >>> 82 - BindResponse
2024/10/07 18:09:16 Error reading Message : runtime error: index out of range [47] with length 47
	3081ab0201026381a5045d7569643d33626332633564652d303538642d343838382d613836322d6666303461616138653434322c636e3d757365725f62746b7165762c6f753d62657467616d696e67732c64633d73736f2c64633d786d796265742c64633d636f6d0a01000a0100020100020100010100a92f8116312e322e3834302e3131333535362e312e342e3830338212757365724163636f756e74436f6e74726f6c83013230040402646e

I suppose if answer with empty response (do not close connection silently) GitLab can simply proceed to next steps.

@baznikin you should debug the code to locate the code line for error: Error reading Message : runtime error: index out of range [47] with length 47

I am not good enough to do debug, but I believe it came from upstream library you are using, there are issue for this case - https://github.com/vjeantet/ldapserver/issues/26. Looks like it is unmaintained - no updates for years. This particular issue was opened 6 years ago...
I suppose there is 2 options for you to implement better protocol handling - change library or make a fork to maintain it on your own.... I am not Golang programmer and can't help you with it

WORKAROUND: Turns out, this request from GitLab can be skiped if set LDAP config option active_directory: false

We have updated the LDAP import lib (not resolving this issue): https://github.com/casdoor/casdoor/commit/05830b9ff662c0db8a11c8976e54483ec07f6cb1

For all people, who want to implement LDAP auth because you can't autocreate user in GitLab during first sign in - it is not true, you CAN create user in GitlLab during first sign in!
https://docs.gitlab.com/ee/integration/omniauth.html#configure-common-settings
for helm values
global:
  appConfig:
    omniauth:
       allowSingleSignOn: ['openid_connect']


Bug: RegExpMatchArray is missing properties index and input

TypeScript Version:  3.7.2
Search Terms: RegExpMatchArray, input, index, length, regexp, regex, result objects
Code
const match = 'a'.match(/a/)
if ( match ) console.log(match.input.includes('a'), match.index.toFixed(2))

Expected behavior: It to work.
Actual behavior: Type checks fails because RegExpMatchArray does not define index and input
Playground Link: Playground Link
Related Issues: none
For reference of the index and input properties, see:

RegExp result objects have some non-numerical properties already, which named capture groups may overlap with, namely length, index and input.

https://github.com/tc39/proposal-regexp-named-groups

When I run this in the playground, I get "true 0.00" in the console, which is expected behavior.

Correct, the code will run, but the typescript type checking warns Object is possibly 'undefined'.(2532)

The error is correct.  Typescript does define RegExpMatchArray.input and RegExpMatchArray.index, but warns that they might be undefined, because they might be undefined:
const m = 'a'.match(/a/g)
if (m) {
    console.log(m.input === undefined) // prints true
    console.log(m.index === undefined) // prints true
}

Interesting. Is there any further documentation on their behaviour anywhere?

spec
In short, if the regexp has the global flag set, then String.prototype.match won't have input or index.

Could we update the type definition then to have input and index set if the global flag is unset? The introspection could be done using https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/global

Regex flags aren't propagated into the resulting type. I believe this is the only one that has any effect, so it'd be a tough sell to add a new feature to make these properties get detected


parse_osc11 break terminal following system-wide dark mode preference

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
misc
Description
Hello o/
Mini parse_osc11 implementation introduce a bug that breaks terminal background "restore" when following system-wide dark mode preferences:
https://github.com/user-attachments/assets/d085e224-a762-47f6-b16e-0e1f568fffd5
This issue does not occur when using the parse_osc11 function directly from https://github.com/neovim/neovim/blob/0d3f2c4904d552635da07a6e2b4e75520b514940/runtime/lua/vim/_defaults.lua#L430
https://github.com/user-attachments/assets/218497d5-5132-49b3-8041-1198ba6319e6
Neovim version
0.10.0
Steps to reproduce
Using e.g. Fedora 40 in VM
cat << EOF >> ~/.config/kitty/kitty.conf
# BEGIN_KITTY_THEME
include current-theme.conf
# END_KITTY_THEME
EOF

mkdir -p ~/.config/kitty/themes

cat << EOF > ~/.config/kitty/themes/neovim-dark.conf
# vim:ft=kitty

# Basic
foreground              #E0E2EA
background              #14161B
selection_foreground    #E0E2EA
selection_background    #4F5258

# Cursor
cursor                  none
cursor_text_color       background

# Border
active_border_color     #E0E2EA
inactive_border_color   #E0E2EA

# Tab
active_tab_foreground   #E0E2EA
active_tab_background   #14161B
inactive_tab_foreground #C4C6CD
inactive_tab_background #2C2E33

# Palette

# black
color0 #07080d
color8 #4f5258

# red
color1 #ffc0b9
color9 #ffc0b9

# green
color2  #b3f6c0
color10 #b3f6c0

# yellow
color3  #fce094
color11 #fce094

# blue
color4  #a6dbff
color12 #a6dbff

# magenta
color5  #ffcaff
color13 #ffcaff

# cyan
color6  #8cf8f7
color14 #8cf8f7

# white
color7  #eef1f8
color15 #eef1f8
EOF

cat << EOF > ~/.config/kitty/themes/neovim-light.conf
# vim:ft=kitty

# Basic
foreground              #14161B
background              #e0e2ea
selection_foreground    #14161B
selection_background    #9B9EA4

# Cursor
cursor                  none
cursor_text_color       background

# Border
active_border_color     #14161B
inactive_border_color   #14161B

# Tab
active_tab_foreground   #14161B
active_tab_background   #E0E2EA
inactive_tab_foreground #2C2E33
inactive_tab_background #C4C6CD

# Palette

# black
color0 #07080d
color8 #4f5258

# red
color1 #590008
color9 #590008

# green
color2  #005523
color10 #005523

# yellow
color3  #6b5300
color11 #6b5300

# blue
color4  #004c73
color12 #004c73

# magenta
color5  #470045
color13 #470045

# cyan
color6  #007373
color14 #007373

# white
color7  #eef1f8
color15 #eef1f8
EOF

cat << EOF > ~/.local/bin/kitty-daemon
#!/usr/bin/env bash

if ! command -v kitty > /dev/null 2>&1; then
  exit 1
fi

gsettings monitor org.gnome.desktop.interface color-scheme | \
  xargs -I % bash -c "$HOME/.local/bin/kitty-color-scheme" &2>>/dev/null true
EOF

cat << EOF > ~/.local/bin/kitty-color-scheme
#!/usr/bin/env bash

if ! command -v kitty > /dev/null 2>&1; then
  exit 1
fi

mode=`gsettings get org.gnome.desktop.interface color-scheme`

d="Neovim-Dark"
l="Neovim-Light"

case $mode in
  *dark* )
    kitty +kitten themes --reload-in=all $d 2>/dev/null
    ;;
  *default* | *light* )
    kitty +kitten themes --reload-in=all $l 2>/dev/null
    ;;
esac
EOF

chmod +x ~/.local/bin/kitty-daemon
chmod +x ~/.local/bin/kitty-color-scheme

cat << EOF > ~/.config/systemd/user/kitty-daemon.service
[Unit]
Description=Foo

[Service]
Type=forking
Environment=\"PATH=$HOME/.local/bin:/usr/local/bin:/usr/bin:/bin\"
ExecStart=$HOME/.local/bin/kitty-daemon
RemainAfterExit=yes

[Install]
WantedBy=default.target
EOF

systemctl --user enable kitty-daemon.service
systemctl --user start kitty-daemon.service

Then run neovim and change desktop to dark or light mode and exit

Thanks for the issue!

Mini parse_osc11 implementation introduce a bug that breaks terminal background "restore" when following system-wide dark mode preferences:

Can you, please, clarify what exactly you think is broken? If it is that the color is set back to the light one, then it is by design in order to support wider range of terminal emulators.

This issue does not occur when using the parse_osc11 function directly from https://github.com/neovim/neovim/blob/0d3f2c4904d552635da07a6e2b4e75520b514940/runtime/lua/vim/_defaults.lua#L430

You could not have used parse_osc11 function directly, because it returns incompatible output. Chances are, that the alternative solution uses \027]111\027\\ to restore "original" terminal background color which work well in this scenario. This is not the case with setup_termbg_sync(), which (as said above) is expected.

Thanks for the issue!

Mini parse_osc11 implementation introduce a bug that breaks terminal background "restore" when following system-wide dark mode preferences:

Can you, please, clarify what exactly you think is broken? If it is that the color is set back to the light one, then it is by design in order to support wider range of terminal emulators.

This issue does not occur when using the parse_osc11 function directly from https://github.com/neovim/neovim/blob/0d3f2c4904d552635da07a6e2b4e75520b514940/runtime/lua/vim/_defaults.lua#L430

You could not have used parse_osc11 function directly, because it returns incompatible output. Chances are, that the alternative solution uses \027]111\027\\ to restore "original" terminal background color which work well in this scenario. This is not the case with setup_termbg_sync(), which (as said above) is expected.

That could be it, indeed, if it return invalid input then it does indeed make sense

If it is that the color is set back to the light one

That'd make sense; would you be open to have a compatibility option that we can opt-out for modern terminal that can handle the initial request properly?

Sorry for the noise, this can be either closed or converted to a feature request :)

would you be open to having a compatibility option that we can opt-out for modern terminals that can handle the initial request properly?

Sorry, no. I don't changing default terminal color is a frequent enough use case to warrant adding extra configuration. If this is annoying enough for you, I think it would be better to just use custom autocommands:
vim.api.nvim_create_autocmd({ "UIEnter", "ColorScheme" }, {
  callback = function()
    local normal = vim.api.nvim_get_hl(0, { name = "Normal" })
    if not normal.bg then return end
    io.write(string.format("\027]11;#%06x\027\\", normal.bg))
  end,
})

vim.api.nvim_create_autocmd("UILeave", {
  callback = function() io.write("\027]111\027\\") end,
})

Closing as both not an issue and not planned.

I don't changing default terminal color is a frequent enough use case to warrant adding extra configuration

This is not about changing the default terminal color, this is about how it break every terminal that follow OS dark-light preference, this is non-trivial IMO

This is not about changing the default terminal color, this is about how it break every terminal that follow OS dark-light preference, this is non-trivial imo 🤔

The effect of following OS dark-light preference in terminal emulator is to effectively change its color scheme (which means changing its "default" background color). If using "\027]111\027\" OSC won't help, then there are even more reasons to not support this use case.

If using "\027]111\027" OSC won't help, then there are even more reasons to not support this use case.

Are we misunderstanding each other? Just to be clear, \027]111\027\ does work properly—which is basically what I'm asking for here: a way to send it instead of sending, e.g.
    H.termbg_init = H.termbg_init or bg_init
    local reset = function() io.write('\027]11;' .. H.termbg_init .. '\007') end

where H.termbg_init is #162430

Are we misunderstanding each other? Just to be clear, \027]111\027\ does work properly

Then my suggestion is to use it inside custom autocommands instead of setup_termbg_sync().
I don't see it as a good idea based solely on this use case to introduce configuration for an approach with less terminal emulator coverage, which should also be implemented, documented (introducing unnecessary confusion), and tested.


BUG Can't choose import file

When trying to import, the system file chooser displays all files grayed out, even the .json export file. Therefore, no file can be selected for import.
Using Android 9.
The export function works as expected.

have the same issue with import = impossible to use the app as it is ...

Thank you for reporting can you provide more details os version, mobile and app version


Need code owners for opentemetry-operator helm chart

The operator helm chart need more attention.
@open-telemetry/operator-maintainers please let us know who can help with maintaining it and can be added as a code owner.

I don't think any of the current Operator maintainers did anything regarding the helm chart for it.

@erichsueh3 from AWS has also been working on the operator helm chart. He can help maintain the releases for the helm chart to stay current with the Collector versions.

@dmitryax I can be part of maintaining otel-operator helm chart :)

@dmitryax @alolita I can also help out!

Thanks @VineethReddy02 and @erichsueh3. I think we currently have at least couple of issues related to the operator:

https://github.com/open-telemetry/opentelemetry-helm-charts/issues/132
https://github.com/open-telemetry/opentelemetry-helm-charts/issues/69
and this one that would be very useful for both charts:
https://github.com/open-telemetry/opentelemetry-helm-charts/issues/78

Feel free to take any of them

Another issue reported https://github.com/open-telemetry/opentelemetry-helm-charts/issues/145

@jaronoff97 and @abelperezok would you be interested in taking a more active role in the operator chart?

closing as @Allex1 is now code owner.


Regex in template

Hi
Is it possible to use regex in the template section?
10x!

Can you explain / show what you are trying to accomplish?

it doesn't matter anymore, I used another way to achieve my goal.
10x anyway!


Fix the version of jackson-databind to <IP_ADDRESS>

Before fixing the version:
snyk test

✗ Denial of Service (DoS) [High Severity][https://security.snyk.io/vuln/SNYK-JAVA-COMFASTERXMLJACKSONCORE-2421244] in<EMAIL_ADDRESS>introduced by<EMAIL_ADDRESS>><EMAIL_ADDRESS>

After fixing the version:

snyk test

> ✔ Tested 13 dependencies for known issues, no vulnerable paths found.

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
:memo: Please visit https://docs.snowplowanalytics.com/docs/contributing/contributor-license-agreement/ to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


[Bug]一个worker不能属于多个工作组 one worker can’t belong different worker groups

1.3.2版本一个新的功能是一个worker能属于多个工作组
https://github.com/apache/incubator-dolphinscheduler/pull/3277
但我实际测试，在worker.properties里设置worker.groups=default,group1
在页面上还是只有default这个工作组
A new feature of version 1.3.2 is that a worker can belong to multiple working groups
https://github.com/apache/incubator-dolphinscheduler/pull/3277
But I actually tested that in config file worker.properties Setting in worker.groups=default,group1
On the UI, only default worder groups
worker.properties

install_config.conf

WEB UI

Which version of Dolphin Scheduler:
-[1.3.2-release]

My test is OK. After you change the worker configuration, did you restart the worker service?

My different worker groups are OK, After you change the worker configuration, did you restart the worker service?

Yes, all service are restarted. Can you take a screenshot of the work groups in worker.properties and install_config.conf

There is no problem with your configuration. Are you sure it is the latest code of 1.3.2-release? You can check whether the worker groups are the latest code through the pr:https://github.com/apache/incubator-dolphinscheduler/pull/3277

There is no problem with your configuration. Are you sure it is the latest code of 1.3.2-release? You can check whether the worker groups are the latest code

Maybe it's what you said. The first time I use  script install.sh update the directory of 1.3.1-release. Just now I installed  in a new directory and modify the config file. As a result, the work groups are right.
Thanks for your help.


Build CJS, update package.json

Supercedes https://github.com/KittyGiraudel/a11y-dialog/pull/659; just opening for reference

Closing in favor of #702. Thank you for the super insightful comments here!


Change setup script to use pip install instead of setup.py develop

Description:
Using python setup.py develop did not manage to install the required dependencies.
This updates script/setup to use pip install -e . instead in order to resolve the required dependencies.
Checklist:
If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hi @hoh,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!


BIGTOP-3804: Add quicklinks for HDFS/YARN/MapReduce

Description of PR
1 add quicklinks for HDFS/YARN/MapReduce
2 fix the bug that cann't access to http://mr-historyserver-host:19888/logs
How was this patch tested?
Tested in local vm cluster, including kerberos authentication
hdfs quicklink

yarn quicklink

mr quicklink

mr logs ui

For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'BIGTOP-3638. Your PR title ...')?
[ ] Make sure that newly added files do not have any licensing issues. When in doubt refer to https://www.apache.org/licenses/

Please keep same style for name NameNode Logs, ResourceManager logs and JobHistory logs.

Please keep same style for name NameNode Logs, ResourceManager logs and JobHistory logs.

Thhank for your review. fixed

+1.LGTM

Merged, thanks @smallyao

Merged, thanks @smallyao

Thanks @guyuqi


Return InvalidArgument code when Piped tries to operate deleted app

What this PR does / why we need it:
Updating a deleted application shouldn't handled as an internal error.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

🚀
/lgtm

Fixed

/approve

Sorry. I made a minor fix so PTAL

/approve


How to do Fulltext search on the below Document with ArangoDB?

{
"rootElement": {
"names": {
"name": [
"Haseb",
"Anil",
"Ajinkya",
{
"city": "mumbai",
"state": "maharashtra",
"job": {
"second": "bosch",
"first": "infosys"
}
}
]
},
"places": {
"place": {
"origin": "INDIA",
"current": "GERMANY"
}
}
}
}
If I had the document like the above example and I want to search the value like "mumbai" or "infosys" then how would I do the indexing and search for the same. Help would highly be appreciated. Thanks in advance :)

closing as duplicate. http://stackoverflow.com/questions/35322859/how-to-do-fulltext-search-on-the-below-document-with-arangodb/35323834#35323834


Make file backend an optional feature (fix #5)

Codecov Report

Merging #7 into master will decrease coverage by 40.84%.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           master    #7       +/-   ##
========================================
- Coverage   40.84%    0%   -40.85%
========================================
  Files          17    17
  Lines         404   329       -75
========================================
- Hits          165     0      -165
- Misses        239   329       +90

Impacted Files
Coverage Δ

efivar/src/error.rs
0% <ø> (ø)
:arrow_up:

efivar/src/lib.rs
0% <0%> (ø)
:arrow_up:

efivar/src/store/vendor_group.rs
0% <0%> (-100%)
:arrow_down:

efivar/src/store/store_value.rs
0% <0%> (-100%)
:arrow_down:

efivar/src/store/guid_group.rs
0% <0%> (-100%)
:arrow_down:

efivar/src/store/memory.rs
0% <0%> (-93.11%)
:arrow_down:

efivar/src/efi.rs
0% <0%> (-76.93%)
:arrow_down:

efivar/src/store/file.rs
0% <0%> (-73.92%)
:arrow_down:

efivar/src/sys/linux/mod.rs
0% <0%> (-54.17%)
:arrow_down:

efivar/src/store/variable.rs
0% <0%> (-42.11%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


update-pipeline Docker Hub logins

I'm using update-pipeline with --push-images like this:
pachctl update-pipeline -f annotate.json --push-images --password XXXXX -u dwhitena

I can do this once and only once successfully.  After that once successful --push-images, I get the following:
dwhitena@dirac:annotate$ pachctl update-pipeline -f annotate.json --push-images --password XXXX -u dwhitena
Pushing dwhitena/chess-annotate:48e986766098493cb1bd4ce4b09029ec, this may take a while.
toomanyrequests: too many failed login attempts for username or IP address

Despite using the correct credentials and not having any other processing trying to log into Docker Hub.  Thus, it seems like something in the --update-images process is causing failed logins.  If I wait 10 minutes or so, the problem goes away (I'm assuming this is the window for Docker Hub).

Hmm, that is very strange indeed. I can't think of where the failed login attempts would be coming from given that it clearly succeeds in at least one of these attempts.

This issue seems like it's relevant: https://github.com/docker/docker/issues/22188

Between that issue from @jdoliner and some debugging, I was able to determine that this had nothing to do with Pachyderm or pachctl.  I had changed some setting to my local Docker login, and we trying failed logins when pushing images.


build(deps): bump gorm.io/gorm from 1.24.5 to 1.25.1

dependabot PR: #38

Codecov Report
Patch coverage has no change and project coverage change: -0.09 :warning:

Comparison is base (dfa325f) 27.14% compared to head (e53865e) 27.05%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   27.14%   27.05%   -0.09%
==========================================
  Files           2        2
  Lines         619      621       +2
==========================================
  Hits          168      168
- Misses        436      438       +2
  Partials       15       15

Flag
Coverage Δ

go-1.18.x
27.05% <0.00%> (-0.09%)
:arrow_down:

go-1.19.x
27.05% <0.00%> (-0.09%)
:arrow_down:

go-1.20.x
27.05% <0.00%> (-0.09%)
:arrow_down:

macOS
27.05% <0.00%> (-0.09%)
:arrow_down:

ubuntu
27.05% <0.00%> (-0.09%)
:arrow_down:

unit
27.05% <0.00%> (-0.09%)
:arrow_down:

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/dialect/dialect.go
25.70% <0.00%> (-0.09%)
:arrow_down:

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


Alert in cart always visible

Hi,
There is no logic for showing/hiding cart alert in templates/cart/index.html

From what I see this piece of markup is used on the JS side cart.js, but I don't see this message anyway when doing changes in my cart. Maybe there is something wrong. @dominik-zeglen could you check it when you have time?

Thanks for pointing it out!
It seems that alerts are generated by JS, but they are not working atm. I'll fix it in a moment.


Post task not executed

Describe the Issue
Post task (reboot) is not included in main task, so it is skipped and the handler doesn't run when expected to.
Expected Behavior
Post task included, flush handlers executed
Actual Behavior
Post task skipped, handlers not executed at end of role execution.
Control(s) Affected
None
Environment (please complete the following information):

branch being used: [e.g. devel]
Ansible Version: [e.g. 2.10]
Host Python Version: [e.g. Python 3.7.6]
Ansible Server Python Version: [e.g. Python 3.7.6]
Additional Details:

Additional Notes
None
Possible Solution
Include task post in main.yml

resolved in #44  by @mfortin !


try to parse host and port from single argument to :ClojureConnect

Small change to parse a host and port if there is a single argument to :ClojureConnect

Neat! Will take a look at this ASAP, thank you!


Create 'relaton-ogc' to fetch OGC documents

@gbuehler mentioned that there is an existing OGC database of all publications.
We can take advantage of that either by:

directly querying against a web endpoint that exposes that data source, or
that data source can be transformed into a static YAML file like https://github.com/CalConnect/standards.calconnect.org/blob/master/_input/admin.yaml

Either way we will need a new gem.
@gbuehler how can we obtain this database? If it's a static one, how do we keep it updated?

@ronaldtse The endpoint always gives a JSON dump of the current Standards, Best Practice, Engineering Reports and Discussion Papers. It can be found at:  https://portal.opengeospatial.org/public_ogc/api/docs.php

@gbuehler that's great. Does the docs.php endpoint also allow querying, such as fetching by document identifier?

I was going to say that @ghobona mentioned that the JSON dump can also be found at:
https://github.com/opengeospatial/NamingAuthority/blob/master/incubation/bibliography/bibliography.json

And we have a question on the "type" attribute. These values from “type” seem to have some meaning.
AS => Abstract Specification
BP => Best Practice
CAN => ?
CC => ?
CR => ?
CS => Community Standard
DP => Discussion Paper
DP-Draft => Draft DP
IPR => ?
IS => ?
ISC => ?
ISx => ?
Notes => Notes?
ORM => ?
PC => ?
PER => ?
POL => ?
POL-NTS => ?
Primer => Primer?
Profile => Profile
RFC => (Isn’t RFC a document stage?)
Retired => (Retired is a stage?)
SAP => ?
TS => ?
WhitePaper => White paper?
The D-* ones seem to be Drafts?
D-AS => Draft AS
D-ATB => Draft ?
D-BP => Draft BP
D-DP => Draft
D-IS => ?
D-ISC => ?
D-ORM => ?
D-Profile => Draft Profile
D-RFC => Draft RFC?
D-RP => ?
D-SAP => ?

Thanks @ghobona for the reply. The types are listed here now:
{
  "AS" => "Abstract Specification",
  "BP" => "Best Practice",
  "CAN" => "Candidate Standard",
  "CC" => "Conformance Class",
  "CR" => "Change Request",
  "CS" => "Community Standard",
  "DP" => "Discussion Paper",
  "DP-Draft" => "Draft Discussion Paper",
  "IPR" => "Interoperability Program Report - Engineering Specification",
  "IS" => "Implementation Standard",
  "ISC" => "Implementation Standard Corrigendum",
  "ISx" => "Extension Package Standard",
  "Notes" => "Notes",
  "ORM" => "OGC Reference Model",
  "PC" => "Profile Corrigendum",
  "PER" => "Public Engineering Report",
  "POL" => "Policy",
  "POL-NTS" => "Policy - Name Type Specification",
  "Primer" => "Primer",
  "Profile" => "Profile",
  "RFC" => "Request for Comment",
  "Retired" => "Retired document",
  "SAP" => "Standard Application Profile",
  "TS" => "Test Suite",
  "WhitePaper" => "Whitepaper"
}

The "D-" prefix means that a document is deprecated.

@ronaldtse should we update the JSON dump daily as we do in relaton-nist https://github.com/relaton/relaton-nist/issues/20#issue-449410651 ?

@andrew2net yes we should to ensure it is updated. I believe we can use ETag cache checks to ensure the remote response is updated before fetching the full data again.

@ronaldtse in the JSON dump there "identifier"=>"18-073r2". Assume we should use identifier to search documents. Do we need to fetch references with year included? like 18-073r2:2019

OGC references "18-073r2" implies it is edition 2, so the year is not necessary. Thanks!

The endpoint https://portal.opengeospatial.org/public_ogc/api/docs.php returns 982 documents.
The https://github.com/opengeospatial/NamingAuthority/blob/master/incubation/bibliography/bibliography.json returns 979.
The first one returns 3 documents more but doesn't support ETag. Which one should we use?

Thanks @ghobona for the reply. The types are listed here now:
{
  "AS" => "Abstract Specification",
  "BP" => "Best Practice",
  "CAN" => "Candidate Standard",
  "CC" => "Conformance Class",
  "CR" => "Change Request",
  "CS" => "Community Standard",
  "DP" => "Discussion Paper",
  "DP-Draft" => "Draft Discussion Paper",
  "IPR" => "Interoperability Program Report - Engineering Specification",
  "IS" => "Implementation Standard",
  "ISC" => "Implementation Standard Corrigendum",
  "ISx" => "Extension Package Standard",
  "Notes" => "Notes",
  "ORM" => "OGC Reference Model",
  "PC" => "Profile Corrigendum",
  "PER" => "Public Engineering Report",
  "POL" => "Policy",
  "POL-NTS" => "Policy - Name Type Specification",
  "Primer" => "Primer",
  "Profile" => "Profile",
  "RFC" => "Request for Comment",
  "Retired" => "Retired document",
  "SAP" => "Standard Application Profile",
  "TS" => "Test Suite",
  "WhitePaper" => "Whitepaper"
}

The "D-" prefix means that a document is deprecated.

@ronaldtse not all of the types match to the grammar
DocumentType =
"standard" | "standard-with-suite" | "abstract-specification" | "community-standard" | "profile" | "best-practice" | "engineering-report" | "discussion-paper" | "reference-model" | "user-guide" | "policy" | "guide" | "amendment" | "technical-corrigendum" | "administrative"

Shouldn't we add new types to grammar?

@ronaldtse there are D-ATB and D-RP types. They are likely deprecated ATB and RP documents. But there aren't such types in the type list.

The endpoint https://portal.opengeospatial.org/public_ogc/api/docs.php returns 982 documents.
The https://github.com/opengeospatial/NamingAuthority/blob/master/incubation/bibliography/bibliography.json returns 979.
The first one returns 3 documents more but doesn't support ETag. Which one should we use?

I think we just use the second one for now. It gets updated weekly or something.

Shouldn't we add new types to grammar?

Yes you are right! We need to update the grammar (both Relaton and in Metanorma). Ping @opoudjis .

@ronaldtse there are D-ATB and D-RP types. They are likely deprecated ATB and RP documents. But there aren't such types in the type list.

@ghobona @gbuehler could you help let us know what are the D-ATB and D-RP type documents? Thanks!

@ronaldtse there are duplicates identifiers:
 "16-079",
 "12-128r14",
 "12-128r12",
 "12-176r7",
 "04-094",
 "15-078r6",
 "14-115",
 "14-095",
 "06-042",
 "07-147r2",
 "12-066",
 "09-000",
 "07-036",
 "12-006",
 "09-025r1",
 "07-110r4",
 "03-105r1",
 "04-021r3",
 "07-006r1",
 "06-104r4",
 "11-122r1",
 "06-035r1",
 "05-134",
 "03-006r3",
 "01-009",
 "02-058"

l checked 12-128r14 and foud 12 entries. It seems the only difference is title though.

For 12-128r14 the duplicate identifiers are from Conformance Classes (CC). These are not standards but rather specification elements. So Conformance Classes should not be considered standards.
For 16-079, the entries are for different stages of the document. The document was first registered as a Discussion Paper (DP) and then it was developed into an Implementation Standard (IS).
For 12-128r12, the Conformance Classes and Corrigenda have the same identifier as their source standard.
Hopefully this helps.

For 16-079, the entries are for different stages of the document. The document was first registered as a Discussion Paper (DP) and then it was developed into an Implementation Standard (IS).

Should we search by the stages or just fetch the latest document?

For 12-128r12, the Conformance Classes and Corrigenda have the same identifier as their source standard.

Should be Corregeda considered standards?

@ghobona Thanks! Some follow up questions:

Could you also help explain what D-ATB and D-RP documents are?

Are Conformance Classes (CC) citable? If so, is there a way of differentiating them other than using a comparison of title? For example, 12-128r14-cc1 or 12-128r14/cc1?

Corrigenda are usually uniquely citable. For example, ISO uses a syntax like "ISO 1234/Corr 1:2015" to uniquely refer to a document. Should OGC corrigenda be also citable?

How shoud we treat following documents?

14-115 both two entries look similar. The same date and type.
14-095 two entries have the same type but different dates. Suppose we should take the latest.
03-105r1 three entries with the same date and type
04-021r3 two entries with the same date and type
01-009 two entries with SAP and SP types and the same date


Mobile PJ set up

In this project . I added these features. See in below
-Add HTML
_Add CSS
_Linter
-Figma template
-git

project setup


Typescript typings

What is the current behavior?
I'm trying to upgrade from ~1.3.1 to the latest package version ~1.4.5 and all the imports from this package are now throwing the following error:
File<EMAIL_ADDRESS>is not a module.ts(2306)

What's happening that seems wrong?
Looking into the index.d.ts from both versions they're definitely different, and the version from 1.3 was exporting the Deepgram module and now it doesn't seem so.

Steps to reproduce

Start a typescript project
Install the latest version from the SDK: yarn add @deepgram/sdk@latest;
Import the sdk import { Deepgram } from '@deepgram/sdk';
Typescript will give an error.

Expected behavior
I'd expect to import not only the main Deepgram class, but also many other different types such as: PrerecordedTranscriptionResponse, WordBase, etc.
Please tell us about your environment

Operating System/Version: MacOS 11.5.2
Language: TypeScript 4.7.4

Other information
https://stackblitz.com/edit/typescript-vbkaj4?file=index.ts you might need a bit for it to fully load the linter but the result will be the same as the image:

Amazing find and detail @sircon! We're on this now.

Thanks again for reporting this @sircon. Version 1.4.7 has just been released and should have resolved this.
I tested in a StackBlitz that you can check out here: https://stackblitz.com/edit/typescript-g4e1dm?file=index.ts

Thanks for the fix @MichaelJolley but... although the Deepgram works now, there are some issues with other types. In version 1.3.1 I could import PrerecordedTranscriptionResponse and WordBase form your package and now it doesn't seem to find it anywhere. Looking at your dist folder, it looks like you are missing two important folders you had before types/index.d.ts and the transcription/index.d.ts.

Do you think you can add these too?

For reference, how it looked before:

I could just copy & paste the interfaces, but obviously having it always up to date with the package would be the ideal situation!

Ugh. Can't believe I missed that. 1.4.8 should have them restored.

Works great @MichaelJolley, thanks for the quick fixes!