Support for response multiValueHeaders locally

Issue #, if available:
Addresses is #830, supporting multiValueHeaders locally.
Description of changes:
Merges lamdba response headers and multiValueHeaders in accordance with guidance from here.
https://docs.aws.amazon.com/apigateway/latest/developerguide/set-up-lambda-proxy-integrations.html#api-gateway-simple-proxy-for-lambda-output-format

You can use the multiValueHeaders key to specify all of your extra headers, including any single-value ones.
If you specify values for both headers and multiValueHeaders, API Gateway merges them into a single list.
If the same key-value pair is specified in both, only the values from multiValueHeaders will appear in the merged list.

Within base64 logic, splits Content-Type by ;, to prevent issues with malformed Content types.
content_type = lamba_response_headers['Content-Type'].split(";", 1)[0]

[ ] TODO: Add tests for the _should_base64_decode_body change - Guidance needed

Checklist:

[x] Write unit tests (Tests for functionality of merger and update to parse_lambda tests for ensuring the component is called)
[x] Write/update functional tests (Unsure what to add - Guidance needed)
[x] Write/update integration tests (Unsure what to add - Guidance needed)
[x] make pr passes
[x ] Write documentation - No adjustment required (unless there is a supported features list somewhere?)

Please let me know what else is required/missing! :smile: This PR plays well with #741, which is for requests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @jfuss, could you let me know what else is required for this? Thanks!

@martysweet I really would love for this to work.  when I'm testing it using both your branch here, and apigw, they do act different.  this code creates a comma-seperated list in a single header (which chrome, at least, doesnt like), and apigw creates multiple headers with the same key (in my tests I'm using set-cookie, but I'm sure it's for anything that it applies to)

and apigw creates multiple headers with the same key (in my tests I'm using set-cookie, but I'm sure it's for anything that it applies to)

@everyonce, this isn't true, actually. Set-Cookie is a special case, according to the HTTP spec headers aren't supposed to appear multiple times, but they allow Set-Cookie since it's different. API GW seems to follow this as well, only creating multiple headers for Set-Cookie and instead creating a list for other headers, like X-Foo. I've just tested it and using X-Foo definitely creates a comma-separated list.

Closing in favour of #1166


feat(app): add automatic backup feature

:tada: This PR is included in version 2.38.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ARROW-1531: [C++] Use forward references for appending std::array in FixedSizeBinaryBuilder

You can still pass them: http://en.cppreference.com/w/cpp/utility/forward

If a call to wrapper() passes an rvalue std::string, then T is deduced to std::string (not std::string&, const std::string&, or std::string&&), and std::forward ensures that an rvalue reference is passed to foo.
If a call to wrapper() passes a const lvalue std::string, then T is deduced to const std::string&, and std::forward ensures that a const lvalue reference is passed to foo.
If a call to wrapper() passes a non-const lvalue std::string, then T is deduced to std::string&, and std::forward ensures that a non-const lvalue reference is passed to foo.

I mean that this code no longer works:

FixedSizeBinaryBuilder builder(type);

std<array, SIZE> val = ...;

builder.Append(val);

You would have to do builder.Append(std::move(val));. Might be wrong, should try compiling it before I put my foot in my mouth =)

That should still work, because that's a non-const lvalue which would be deduced to
std::array<uint8_t, 16>&

I tried compiling:
TEST_F(TestFWBinaryArray, Builder2) {
  InitBuilder(8);

  std::array<uint8_t, 8> val;

  ASSERT_OK(builder_->Append<8>(val));
}

../src/arrow/array-test.cc:1207:39: error: no matching member function for call to 'Append'
  do { ::arrow::Status s = (builder_->Append<8>(val)); if (!s.ok()) { return ::testing::internal::AssertHelper(::testing::TestPartResult::kFatalFailure, "../src/arrow/array-test.cc", 1207, "Failed") = ::testing::Message() << s.ToString(); } } while (0);
                            ~~~~~~~~~~^~~~~~~~~
../src/arrow/builder.h:743:10: note: candidate function [with NBYTES = 8] not viable: no known conversion from 'std::array<uint8_t, 8>' (aka 'array<unsigned char, 8>') to 'std::array<uint8_t, 8UL> &&' (aka 'array<unsigned char, 8UL> &&') for 1st argument
  Status Append(std::array<uint8_t, NBYTES>&& value) {

Can you change the title? I'm going to cut the 0.7.0 RC after these tests pass and I can merge this

done

Appveyor build in progress, will merge once that's looking good: https://ci.appveyor.com/project/cpcloud/arrow/build/1.0.286

Appveyor build is green: https://ci.appveyor.com/project/cpcloud/arrow/build/1.0.286

I rebased this PR since then, just FYI


[Meru] Invite by email

Description
Implement / make sure the invite by email flow has the same look and feel as the one designed on Figma.
Email copy and design needs to be adjusted too.

[x] Design the emails and make them functionals.
[x] Design the landing page once the user clicks on the email.
[x] Make sure the invite flow works.

Screenshots

@Duncid can you provide a design for the email templates?

@flvndvd do you confirm the screens are done?
As for the emails I think it's lower priority than many other things. Feel free to drop IMHO

Yep screens are done.

Invite template is done as well and live.


Help setting up

Hey James,
I'm writing an app where the user pays for 'walking games', and I'm struggling setting up the IAP's, I understand it is a non-consumable item that I'm using and I have set that up in the play console, as well as having an active release with the permissions set up. However I'm struggling to get anything else set up, would you be able to help me. Cheers in advance

Hi @Saxby-source I had similar problems setting up in-app purchases. In the play console, go to settings -> mail-lists and add your google play account. Then go to your app -> to your active release -> tester and add the mail there too. A very important point is, that you need to accept to participation for testing. The link is in the bottom from the tester page.
Another important point is, you need to sign every apk. You can do this in the Android-Package signing options in visual studio. If it still doesn't work, maybe this link helps..
Regards.

Thank you for your response, ill give it a try
On Sun, Nov 22, 2020, 3:32 PM WorldOfBasti<EMAIL_ADDRESS>wrote:

Hi @Saxby-source https://github.com/Saxby-source I had similar problems
setting up in-app purchases. In the play console, go to settings ->
mail-lists and add your google play account. Then go to your app -> to your
active release -> tester and add the mail there too. A very important point
is, that you need to accept to participation for testing. The link is in
the bottom from the tester page.
Another important point is, you need to sign every apk. You can do this in
the Android-Package signing options in visual studio. If it still doesn't
work, maybe this link
https://jamesmontemagno.github.io/InAppBillingPlugin/TestingAndTroubleshooting.html
helps..
Regards.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jamesmontemagno/InAppBillingPlugin/issues/316#issuecomment-731765448,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEPDIYRB4BAWYFTCB3YCWN3SREVJ5ANCNFSM4SKIYR4Q
.

What is the problem? I don't get it, can you post a code snippet or something similar?

This is my code for a single purchase page, it works fine until I put multiple 'links' to it, that's when i get the error 'you ar enot connected to google play store.
`public partial class PuchasePage : ContentPage
{
bool isPurchased;
string skuId;
ContentPage navigateTo;
public PuchasePage(string productId, string gameDescription, string gameImgSource, ContentPage navTo)
{
InitializeComponent();
skuId = productId;
navigateTo = navTo;
gameImage.Source = gameImgSource;
DescriptionTxt.Text = gameDescription;
CheckPurchase(skuId);
}
async private void CheckPurchase(string productId)
{
var connected = await CrossInAppBilling.Current.ConnectAsync();
if (!connected)
{
// Could not connect, alert the user
await DisplayAlert("ALERT", "Could not connect", "Try again");
return;
}
var purchaseCheck = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);
if (purchaseCheck?.Any(p => p.ProductId == productId) ?? false)
{
Purchased.IsVisible = true;
}
else
{
PurcahseBttn.IsVisible = true;
}
}
async private void PurchaseProduct(string productId)
{
try
{
var connected = await CrossInAppBilling.Current.ConnectAsync();
if (!connected)
{
// Could not connect, alert the user
await DisplayAlert("ALERT", "Could not connect", "Try again");
return;
}
// Try to purchase item
var purchase = await CrossInAppBilling.Current.PurchaseAsync(productId, ItemType.InAppPurchase, "applyLoad");
if (purchase == null)
{
// Not purchased, alert the user
await DisplayAlert("ALERT", "Could not purchase " + productId, "Try again");
}
else
{
// Prucahsed, alert the user
var id = purchase.Id;
var token = purchase.PurchaseToken;
var state = purchase.State;
await DisplayAlert("Awesome", "Purchased item " + productId, "Ok");
isPurchased = true;
}
}
catch (Exception)
{
// An error has occured, alert user
await DisplayAlert("Alert", "Error: could not process payment", "Try again");
        }
        finally
        {
            // Disconect CrossInAppBilling
            await CrossInAppBilling.Current.DisconnectAsync();
        }
        CrossInAppBilling.Dispose();
    }

    private void PurcahseBttn_Clicked(object sender, System.EventArgs e)
    {
        if (!isPurchased)
        {
            PurchaseProduct(skuId);
        }
        if (isPurchased)
        {
            PurcahseBttn.IsVisible = false;
            Purchased.IsVisible = true;
        }
    }
    async private void Purchased_Clicked(object sender, EventArgs e)
    {
        await Navigation.PushModalAsync(navigateTo);
    }
}`

The first point is, that you also need to disconnect after Checking the purchases.
The second point is, that I would use an async Task for PurchaseProduct instead an async void. Add an async to the  PurcahseBttn_Clicked Method. Then add the await keyword to the PurchaseProduct(skuId) call.
You need to make sure, that the purchase check is complete before purchasing..
You can look at the documentation if you need any examples.
I think it should work with this, please test:
public partial class PuchasePage : ContentPage
{
    bool isPurchased;
    string skuId;
    ContentPage navigateTo;
    public PuchasePage(string productId, string gameDescription, string gameImgSource, ContentPage navTo)
    {
        InitializeComponent();
        skuId = productId;
        navigateTo = navTo;
        gameImage.Source = gameImgSource;
        DescriptionTxt.Text = gameDescription;
        CheckPurchase(skuId);
    }
    async private void CheckPurchase(string productId)
    {
        try
        {
            var connected = await CrossInAppBilling.Current.ConnectAsync();
            if (!connected)
            {
                // Could not connect, alert the user
                await DisplayAlert("ALERT", "Could not connect", "Try again");
                return;
            }
            var purchaseCheck = await CrossInAppBilling.Current.GetPurchasesAsync(ItemType.InAppPurchase);
            if (purchaseCheck?.Any(p => p.ProductId == productId) ?? false)
            {
                Purchased.IsVisible = true;
            }
            else
            {
                PurcahseBttn.IsVisible = true;
            }
        }
        catch (Exception)
        {
            // An error has occured, alert user
            await DisplayAlert("Alert", "Error: could not process payment", "Try again");
        }
        finally
        {
            // Disconect CrossInAppBilling
            await CrossInAppBilling.Current.DisconnectAsync();
        }
    }
    async private Task PurchaseProduct(string productId)
    {
        try
        {
            var connected = await CrossInAppBilling.Current.ConnectAsync();
            if (!connected)
            {
                // Could not connect, alert the user
                await DisplayAlert("ALERT", "Could not connect", "Try again");
                return;
            }
            // Try to purchase item
            var purchase = await CrossInAppBilling.Current.PurchaseAsync(productId, ItemType.InAppPurchase, "applyLoad");
            if (purchase == null)
            {
                // Not purchased, alert the user
                await DisplayAlert("ALERT", "Could not purchase " + productId, "Try again");
            }
            else
            {
                // Prucahsed, alert the user
                var id = purchase.Id;
                var token = purchase.PurchaseToken;
                var state = purchase.State;
                await DisplayAlert("Awesome", "Purchased item " + productId, "Ok");
                isPurchased = true;
            }
        }
        catch (Exception)
        {
            // An error has occured, alert user
            await DisplayAlert("Alert", "Error: could not process payment", "Try again");

        }
        finally
        {
            // Disconect CrossInAppBilling
            await CrossInAppBilling.Current.DisconnectAsync();
        }
    }

    private async void PurcahseBttn_Clicked(object sender, System.EventArgs e)
    {
        if (!isPurchased)
        {
            await PurchaseProduct(skuId);
        }
        if (isPurchased)
        {
            PurcahseBttn.IsVisible = false;
            Purchased.IsVisible = true;
        }
    }
    async private void Purchased_Clicked(object sender, EventArgs e)
    {
        await Navigation.PushModalAsync(navigateTo);
    }
}

awesome, so after a bit of testing it's gotten me past the initial problem, however, I'm not going to close the post until I've done more through testing. Cheers for all the help :P


AssistedInject crashes without message when using with class with generic parameter

AssistedInject fails without explanation when I try to use it with various classes that use generics. The simplest case that fails for me with Dagger 2.24, AssistedInject 0.5.0 and Kotlin 1.3.50:
class AClass<A> @AssistedInject constructor(
    @Assisted dependency: String
) {

    @AssistedInject.Factory
    interface Factory {

        fun create(dependency: String): AClass<String>
    }
}

The only error that appears is the following:
error: cannot find symbol
@dagger.Module(includes = {AssistedInject_PresentationModule.class})
                           ^
  symbol: class AssistedInject_PresentationModule
FAILURE: Build failed with an exception.

This is just a simplified version of what I was trying to do initially, which also fails with no meaningful message:
abstract class Base<T>
class Wrapper<A, B>

class AClass<A, B> @AssistedInject constructor(
    @Assisted dependency: (A) -> B
) : Base<Wrapper<A, B>>() {

    @AssistedInject.Factory
    interface Factory {

        fun <A, B> create(dependency: (A) -> B): AClass<A, B>
    }
}

I wrote this test which passes:
  @Test fun stuff() {
    val input = JavaFileObjects.forSourceString("test.Test", """
      package test;

      import com.squareup.inject.assisted.Assisted;
      import com.squareup.inject.assisted.AssistedInject;

      abstract class Base<T> {}
      class Wrapper<A, B> {}
      interface KFunction1<A, B> {}

      class Test<A, B> extends Base<Wrapper<A, B>> {
        @AssistedInject
        Test(@Assisted KFunction1<? extends A, ? super B> dependency) {}

        @AssistedInject.Factory
        interface Factory {
          <A, B> Test<A, B> create(KFunction1<? extends A, ? super B> dependency);
        }
      }
    """)

    val expected = JavaFileObjects.forSourceString("test.Test_AssistedFactory", """
      package test;

      import java.lang.Override;
      import $GENERATED_TYPE;
      import javax.inject.Inject;

      $GENERATED_ANNOTATION
      public final class Test_AssistedFactory implements Test.Factory {
        @Inject public Test_AssistedFactory() {
        }

        @Override public <A, B> Test<A, B> create(KFunction1<? extends A, ? super B> dependency) {
          return new Test<A, B>(dependency);
        }
      }
    """)

    assertAbout(javaSource())
        .that(input)
        .processedWith(AssistedInjectProcessor())
        .compilesWithoutError()
        .and()
        .generatesSources(expected)
  }

Can you find the Java stub that is generated for AClass in your build/ directory and show it?

Here's stub file from build/tmp/kapt3/stubs/debug/com.... I only removed @kotlin.Metadata annotations
For the first sample:
package com.example;

import java.lang.System;

public final class AClass<A extends java.lang.Object> {

    @com.squareup.inject.assisted.AssistedInject()
    public AClass(@org.jetbrains.annotations.NotNull()
    @com.squareup.inject.assisted.Assisted()
    java.lang.String dependency) {
        super();
    }

    @com.squareup.inject.assisted.AssistedInject.Factory()
    public static abstract interface Factory {

        @org.jetbrains.annotations.NotNull()
        public abstract com.example.AClass<java.lang.String> create(@org.jetbrains.annotations.NotNull()
        java.lang.String dependency);
    }
}

For the second one:
// AClass
package com.example;

import java.lang.System;

public final class AClass<A extends java.lang.Object, B extends java.lang.Object> extends com.example.Base<com.example.Wrapper<A, B>> {

    @com.squareup.inject.assisted.AssistedInject()
    public AClass(@org.jetbrains.annotations.NotNull()
    @com.squareup.inject.assisted.Assisted()
    kotlin.jvm.functions.Function1<? super A, ? extends B> dependency) {
        super();
    }

    @com.squareup.inject.assisted.AssistedInject.Factory()
    public static abstract interface Factory {

        @org.jetbrains.annotations.NotNull()
        public abstract <A extends java.lang.Object, B extends java.lang.Object>com.example.AClass<A, B> create(@org.jetbrains.annotations.NotNull()
        kotlin.jvm.functions.Function1<? super A, ? extends B> dependency);
    }
}

// Base
package com.example;

import java.lang.System;

public abstract class Base<T extends java.lang.Object> {

    public Base() {
        super();
    }
}

// Wrapper
package com.example;

import java.lang.System;

public final class Wrapper<A extends java.lang.Object, B extends java.lang.Object> {

    public Wrapper() {
        super();
    }
}

Also maybe irrelevant, but all classes are in an android library module with only single debug variant

Ah, okay I think I found a problem in the simplified example! Still not in the second one though...

▶ java -version
java version "1.8.0_202"
Java(TM) SE Runtime Environment (build 1.8.0_202-b08)
Java HotSpot(TM) 64-Bit Server VM (build 25.202-b08, mixed mode)

I have both targets for all projects set to Java 8, but if I can somehow check that my configuration is correct I can do that.
I can only see that in your test case variance is different:
KFunction1<? extends A, ? super B> dependency
While in my stubs it's
Function1<? super A, ? extends B>

Ah, yeah I just guessed. Will try with the real thing, although since there's one of each it shouldn't make much of a difference.

Narrowed in on the cause. The fix isn't so trivial though, but I'm working on it.

Seems like this is closed by #125, but fyi the test you added in #123 wasn't un-ignored and still fails. Not sure if it's the test that's incorrect or this was closed by mistake


Tests JsonHelperSpec.shouldCleanSelectedChars() + .shouldEscapeNewLine() fail on Windows

The above tests fail on Windows, because the input to JsonHelper is system-dependent (via System.getProperty("line.separator")), but the test expectation isn't.
Message of test shouldEscapeLine:
org.javalite.test.jspec.TestException: Test object: java.lang.String == <line 1\r\nline 2> and expected java.lang.String == <line 1\nline 2> are not equal, but they should be.
One solution could be to make the test expectation system-dependent as well like this:
String escapedLineSep = System.getProperty("line.separator").replace("\n", "\\n").replace("\r", "\\r");         the(result).shouldBeEqual("line 1" + escapedLineSep + "line 2");
Another solution: hard-code the separators in the input to JsonHelper and introduce a second test with hard-coded line separators of Windows.

@cschabl that is strange, since I expected System.getProperty("line.separator") to produce a system-independent result. Maybe this: https://github.com/cschabl/activejdbc/blob/31defe717b691062ac9f0212c79359ce779a9d71/javalite-common/src/main/java/org/javalite/common/JsonHelper.java#L239-L239 needs to be fixed to achieve portability? I do not use Windows though

@ipolevoy I can't tell if the code you are pointing to needs to be adjusted.  To me, it is a unit test issue. There is another test, that fails on Windows: ToJsonSpec.shouldSanitizeJson(). The file bad.txt is read on the byte-level. So line separator characters are preserved. This problem could be avoided by configuring the Git client appropriately. But the default of the Windows Git client is to adjust line breaks to the system on checkout.
shouldSanitizeJson() can be made system-independent like this:
p.set("name", Util.readResource("/bad.txt").replaceAll("\r\n", "\n"));
or to read the resource with a reader that supports reading by line and normalize the line breaks:
private String readResource(String resourcePath) {
        InputStream is = Util.class.getResourceAsStream(resourcePath);
        StringBuilder sb = new StringBuilder();

        try (LineNumberReader reader = new LineNumberReader(new InputStreamReader(is, "UTF-8"))){

            while(true) {
                String line = reader.readLine();
                if (line == null) {
                    break;
                }
                if (reader.getLineNumber() > 1)
                {
                    sb.append("\n");
                }
                sb.append(line);
            }
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
        return sb.toString();
    }

I reproduced both problems on Windows under VBox, will look into it.

@cschabl, I made corresponding changes and the build passes on Linux and Windows. However, the shouldSanitizeJson did not have a problem for me on Windows. I added your suggestion, can you check it on yours?

@ipolevoy , these tests do pass now on my box. There is a final test affected by this line separators problem: FreemarkerSpec.  The test methods shouldRenderList() and shouldRenderRowProcessor() fail on my machine, because of Windows line separators in template file list.ftl. You could fix it like this, e.g.

@cschabl just send in another PL, I will accept


ARROW-7354: [C++] Fix crash in test-io-hdfs

@ursabot crossbow submit test-conda-python-3.7-hdfs-2.9.2

Why was the code wrong before (was something changed regarding multithreading when using the Status-based APIs)?

The test was using concrete class HdfsReadableFile which doesn't inherit the deprecated methods. So file->ReadAt(3, 3, &buffer) was calling HdfsReadableFile::ReadAt(int64_t position, int64_t nbytes, void* out)...

Oh I see, thanks


Add link to DISCLAIMER.md

This PR adds a link to the current (develop) version of the DISCLAIMER.md file.
My other thoughts would be to link to the master branch, so that the disclaimer people read has passed at least one release window - or to generate a page on the website based on the markdown file itself.

I would be okay with this but I'm wondering if we can put it in another place - putting it in that list of links doesn't feel quite right to me.  E.g. between the controls and the start of the grid or somewhere around the "X Free SVG icons for popular brands" message?

This PR adds a link to the current (develop) version of the DISCLAIMER.md file.
My other thoughts would be to link to the master branch, so that the disclaimer people read has passed at least one release window

I think it should link to the version on the master branch.

or to generate a page on the website based on the markdown file itself.

That could be cool in the long run, but I think linking to it is a good first step.

So I've tried a few variants, styling is obviously subject to some additional work.

In the Main Menu
Under X free SVG icons
Below the control bar, before the icon grid

Considering that we probably want to move away from the "large" header, something along the lines of the last option is probably the way to go. Any other thoughts on this @simple-icons/maintainers?

Given it's importance, should we look into trying to position it somewhere near the search form so we can make it sticky? If not then including in the menu as it currently is in the PR makes most sense to me with option 2 being a close second for me.

Given its importance, should we look into trying to position it somewhere near the search form so we can make it sticky?

I'd personally not be a fan of making this sticky, or at least using any additional vertical sticky space.

including in the menu as it currently is in the PR makes most sense to me with option 2 being a close second for me.

I think I agree, we can always update this later.

In the interest of having some link to the disclaimer on the website I'll be merging this in now. But, I'll immediately reopen the related issue with an updated title to keep this topic alive 🙂


Complete should end all loops

Should also consider remaining loops and adjust Reverse accordingly for PingPong (consider current Reversed status to determine whether this tween started running backwards.
DOTween doesn't allow completing of infinite loops, but that seems confusing. Better to end the current loop IMO.
Also is it possible to just do Complete().Kill()? instead of Kill(true).

Pretty sure that Kill(true) doesn't even make sense if Complete() ends all loops actually. They can be completely orthogonal. So long as Complete().KillSync() forces the changes to be flushed and callback to fire, then it's fine.


aws: Ensure ARNs have the correct partition

Not all ARNs belong to the 'aws' partition based on their region. This ensures that all ARNs have the correct one.

/hold
until this is tested

LGTM

/hold cancel

/lgtm


DropDown Menu in Dialog not fully Displayed

Hi,
we have the following scenario:
let items = [{value:"1",text:"First Select"},{value:"5",text:"Selected"},{value:"7",text:"other Item"},{value:"8",text:"Cool Item"},{value:"9",text:"Bla"}];

<Dialog  ref="dialog" title="TestSelect">

"Bla bla bla bla"

<DropDownMenu menuItems={items} autoScrollBodyContent={true} displayMember="text" valueMember="value"}/>

</Dialog>

Which causes the following situtation: Not all dropDownItems are displayed:

Either should:

the Dialog grow dynamically as the Dropdown is opened

or:

the Dropdown should be renderred in the foreground.

+1
Having this same issue. Is there a way to set a maxHeight on a DropDownMenu the way you can with normal menus?

+1 set the dropmenu z-index  is nouse

can anybody help, thanks

I have the same issue when using menus inside Cards.

+1

Same here (inside cards).
I know that DropDownMenu is being rewritten (#1158), is this use case being considered as part of that effort?

I have the same issue.

+1

+1

:+1:  Same thing happens when using dropdown menu as a table row action.

+1

Fixed in #2150.


sections overlap on cell phone

When the sections have nontrivial content, their contents seem to overlap on my iphone 7. Is there an easy solution in bootstrap? Thank you

Hi! I had the same problem. It's not a responsive site. I ended up using media queries to make it responsive. If you put the following css into scrolling-nav.css it should work.
 @media screen and (max-width: 500px) {     body {         height: auto;     } }
The problem is that the sections are inheriting their height from body to be 100%. But on smaller screens 100% is not enough room for the contents of the section. So, if you place it on auto it will make it as large as necessary to accommodate the contents. However, if you don't have enough content on larger screen sizes, it will not fill the entire height of the page. So, a media query lets you change things if the size of the width of the screen meets a certain criteria.
Let me know if you have any questions! If it works for you, I'll see if I can't create a pull request for that solution.

This is fixed up in the BS4 update.


[in_app_purchase] PurchaseParam#sandboxTesting is not used to switch iOS sandbox environment

Steps to Reproduce

I'm using in_app_purchase plugin version 0.3.4+3, and I notice that PurchaseParam has a confusing param.
sample code:
final PurchaseParam purchaseParam = PurchaseParam(
  productDetails: productDetails,
  sandboxTesting: true,
);
await InAppPurchaseConnection.instance
    .buyNonConsumable(purchaseParam: purchaseParam);

PurchaseParam has sandboxTesting param. This document says:

The 'sandboxTesting' is only available on iOS, set it to true for testing in AppStore's sandbox environment. The default value is false.

But this explanation seems to be wrong. In fact, it is only used to test Ask to Buy feature.
    await _skPaymentQueueWrapper.addPayment(SKPaymentWrapper(
        productIdentifier: purchaseParam.productDetails.id,
        quantity: 1,
        applicationUsername: purchaseParam.applicationUserName,
        simulatesAskToBuyInSandbox: purchaseParam.sandboxTesting,
        requestData: null));

https://github.com/flutter/plugins/blob/939b9a6f00367238131ad8dcc8b704f4e3e5ba26/packages/in_app_purchase/lib/src/in_app_purchase/app_store_connection.dart#L52
Furthermore, simulatesAskToBuyInSandbox isn't actually used because of typo.
    payment.simulatesAskToBuyInSandbox =
        [[paymentMap objectForKey:@"simulatesAskToBuyInSandBox"] boolValue];

https://github.com/flutter/plugins/blob/f903d4cf0342b894e44143b02d993bf298b8abe4/packages/in_app_purchase/ios/Classes/InAppPurchasePlugin.m#L182-L183
Correct key is simulatesAskToBuyInSandbox, but actual is simulatesAskToBuyInSandBox.
Expected results:
sandboxTesting is used to switch sandbox environment.
Actual results:
sandboxTesting is used to switch "Ask to Buy" feature, and furthermore, it is not working.
To switch sandbox environment, switch provisioning profile. This seems to be an iOS specification, so there seems to be no way to change it.

Hi @oboenikui, i trying to implementing this plugin in my app, one month ago this work but now is asking for parents permissions.
could you please help me to fix my code to works again?
i'm adding the lines:

@alexlovar You mean that you don't want to test "Ask to Buy" feature, right?
If you set true to sandboxTesting parameter, the transaction simulates "Ask to Buy" feature.
So set false like:
final PurchaseParam purchaseParam = PurchaseParam(
  productDetails: productDetails,
  sandboxTesting: false, // or delete this line
);
await InAppPurchaseConnection.instance
    .buyNonConsumable(purchaseParam: purchaseParam);

Regardless of the value of sandboxTesting, you will actually be connected to Sandbox environment unless your app is released to the App Store.

thanks @oboenikui .
i'm using the sample files buyapp.dart from this repository. i can't find the way to set the params in the sample
i'm using the sample files buyapp.dart from this repository.
i'm adding the lines in the void _listenToPurchaseUpdated(List purchaseDetailsList) line 415 from buyapp.dart sample file.

You need to use PurchaseParam with buyNonConsumable method.
For more details, see main.dart of in_app_purchase example.
https://github.com/flutter/plugins/blob/d82940f02fc306707cce66e4e7111c0bc6ca6fc6/packages/in_app_purchase/example/lib/main.dart#L256


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3461

Questions ?
Please refer to the Backport tool documentation


[1619] Add schools component and render it in the record page

Context

https://trello.com/c/thldQc4L/1620-add-schools-section-to-trainee-show-page-ie-non-draft

Changes proposed in this pull request

Builds the schools component to be used in the record page and /check-details

Guidance to review

Needs a bit of manual prep to set up the trainee as the school work is still in flux. But you'll need to attach a school record as the lead_school or employing_school on a trainee and head to their record page to see it.

 Add Schools section to trainee show page (i.e. non-draft)


Add reset() and set_mode()

add reset() which will trigger a soft-reset
add set_mode() to be able to switch between normal, forced and sleep mode

hiya - we're closing all PRs and deprecating this library. that library uses forced mode only, we noticed that it would self-heat otherwise. if you're up for it, we'd love a PR for the reset() funciton! :)
https://github.com/adafruit/Adafruit_CircuitPython_BME280


Images/docker-compose.yml do not match in 3.4

Describe the bug
After installing a stable version 3.4 for production according to the instructions, the images do not match the docker-compose file.
Instructions:
https://github.com/OpenSlides/OpenSlides/tree/stable/3.4.x
How to Reproduce
Starting from scratch:
git clone https://github.com/OpenSlides/OpenSlides.git --recurse-submodules
cd OpenSlides/docker
./build.sh all
printf "DJANGO_SECRET_KEY='%s'\n" \
  "$(tr -dc 'a-zA-Z0-9' < /dev/urandom | head -c 64)" > secrets/django.env
cp secrets/admin.env.example secrets/adminsecret.env

The build.sh all results in:
Successfully built images:

openslides/openslides-datastore-writer:latest-4
openslides/openslides-client:latest-4
openslides/openslides-backend:latest-4
openslides/openslides-proxy:latest-4
openslides/openslides-autoupdate:latest-4
openslides/openslides-auth:latest-4
openslides/openslides-media:latest-4
openslides/openslides-manage:latest-4
openslides/openslides-datastore-reader:latest-4

So far no errors. But then I am supposed to use
m4 docker-compose.yml.m4 > docker-compose.yml
but there is no docker-compose.yml.m4, so I download directly from the github directory.
After generating the docker-compose.yml, docker-compose up results in
Pulling client (openslides/openslides-client:latest)...
ERROR: The image for the service you're trying to recreate has been removed. If you continue, volume data could be lost.
Consider backing up your data before continuing.
Continue with the new image? [yN]y
Pulling client (openslides/openslides-client:latest)...

ERROR: pull access denied for openslides/openslides-client, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

If I change the docker-compose.yml to use latest-4 instead of latest, it will end up complaining about openslides/openslides-repmgr:latest not existing, which indeed was not build by the "build.sh all". So I gather, that the docker-compose does not match. The latest-4 additionally makes me suspicious that this is not the 3.4 but version 4, although I don't see how else I would the 3.4 version.
Expected behavior
Openslides containers get started.
System information

OpenSlides-Version: 3.4
OS: Linux 5.4.0-67-generic #75-Ubuntu SMP Fri Feb 19 18:03:38 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

The instructions are wrong, instead of "git clone https://github.com/OpenSlides/OpenSlides.git --recurse-submodules" they should specify the branch:
git clone https://github.com/OpenSlides/OpenSlides.git --recurse-submodules -b stable/3.4.x


MINOR: Fix typo on introduction page

@guozhangwang From PR #34 on kafka-site repo

LGTM and merged to trunk.


The instructions for get_dataset_version.ipynb#Download-Dataset-Assets are a sadness

The instructions for the Download Dataset Assets cell state:
# Uncomment code below to locally download all dataset version assets

# download_assets([dataset_version], log_version_id=True)

When the line is uncommented, it then fails because download_assets is commented out in the Import Dependencies cell.
from src.dataset import (
    # download_assets,
    get_dataset,
)

I recommended uncommenting both. Was there a rationale to not demonstrate downloads by default?

Removing comments so that the assets download when the notebook is run... but also reverting sample id to a fake uuid, in keeping with all other notebooks.


gennodejs: 1.0.1-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository gennodejs to 1.0.1-0:

upstream repository: https://github.com/RethinkRobotics-opensource/gennodejs.git
release repository: https://github.com/RethinkRobotics-release/gennodejs-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.21
previous version for package: null

gennodejs
* Package name-switch from genjs to gennodejs based on feedback
* When deserializing, creates an instance of the message
  rather than using json.
* Update genjs for install
  -No longer includes hardcoded paths in generated message files
  -Actually install stuff
* Contributors: Chris Smith, Ian McMahon, chris-smith

Should I add an additional commit here to remove genjs?

You can either add it to this PR or create a separate one.

Alrighty, I'll have that up in this one momentarily.

This is the package rename from genjs --> gennodejs. Could you please clean out the debs and docs for genjs. Thanks!

Please also add doc and source entries.

Please also take the time to create a package page in the wiki for it (http://wiki.ros.org/gennodejs). The package header will then be populated by the doc job.

@tfoote Can you please remove all old artifacts of genjs (doc, source and binary packages).

Thanks for the pointers, Dirk. I created the package template wiki page and added the source and doc info for rosdistro.


Use gem to merge and sign PRs.

What
We've extracted this into a gem, so there's no longer a need to maintain
the script in this repo.
See https://github.com/alphagov/paas-bootstrap/pull/82 for the same change in paas-bootstrap.
How to review
Code review.  The gem itself has already been reviewed here: https://github.com/alphagov/paas-github_merge_sign/pull/1
Optionally, you could use this branch to merge a PR to verify it all works.
Who can review
Not me.

Used to merge #996 👍


Issue 2307 database creation fixes

Two fixes for https://github.com/YOURLS/YOURLS/issues/2307

patch by teyeheimans to report db-creation errors
patch by me to lower key size to 191, to accommodate mysql 5.6 databases using utf8mb4

Thanks!
Can you elaborate on the 191 chars bit, to "accomodate utf8mb4 on mysql 5.6" ? What is it exactly?

OK, read a bit about utf8mb4 and this (weird?) 191 char limit. Thanks a lot.
The PR needs a slight reworking: in order to trigger the DB upgrade, you need to bump the DB version (arbitrary number, 483 will be fine), to trigger a redirection to the upgrade page. A newer function in functions-upgrade.php must reflect the newer DB version: don't change function yourls_upgrade_482(), create a newer one that will be run (yourls_upgrade_483() for instance)
Tell me if that's clear or if I can help

Oh, IC. But I'm sorry, I'm no longer interested. Now that it worked, it turned out you can only have one redirect per URL, which is a showstopper for me, so I looked at a different software.

@Seegras You can, though. It's just a config change away. See #1901.

Some of this PR is included in #2690


3 initiatives

There are 3 initiatives in this section. 1. Azure Policy initiative, 2. Azure initiatives. 3. security initiative.
Are these 3 different things? Or same thing?
If these 3 are different things, please explain differences more. If these 3 are same thing, please use only 1 for preventing confusing.

What is a security initiative?
An Azure Policy initiative is a collection of Azure Policy definitions, or rules, that are grouped together towards a specific goal or purpose. Azure initiatives simplify management of your policies by grouping a set of policies together, logically, as a single item.
A security initiative defines the desired configuration of your workloads and helps ensure you're complying with the security requirements of your company or regulators.
Thank you.

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

ID: a799321a-5fd7-61eb-1922-c7ff881c60a9
Version Independent ID: 00540900-567b-f5be-a3f1-20e14f67f074
Content: Understanding security policies, initiatives, and recommendations in Microsoft Defender for Cloud
Content Source: articles/defender-for-cloud/security-policy-concept.md
Service: defender-for-cloud
GitHub Login: @ElazarK
Microsoft Alias: elkrieger

@takeakitake
Thanks for your feedback! We will investigate and update as appropriate.

@takeakitake Good point. They are all the same. We'll rewrite the section.

#reassign: @bmansheim

#please-close


fixed-size strings

Hi,
Interesting project.
I wonder how you could define a string of a fixed length, or even an array of structs?
Regards

Hi! I do not develop this anymore, but for such features, you could check construct  and construct-typing!


Add AsyncLoader to load and update value periodically

Fixes #5506.
Motivation:
AsyncLoader  can be useful in the following situations.

When it is necessary to periodically read and update information from a file such as resolv.conf .
When data is not valid after a certain period of time, such as an OAuth 2.0 access token.

We already have an implementation for that on AbstractOAuth2AuthorizationGrant.java.
However, I hope to generalize it and add new features to use it in various cases.
Modifications:

Add AsyncLoader to load and update value periodically

Result:

Closes #5506.
Now user can load, cache, update value periodically by using AsyncLoader

@ikhoon really thank you for polishing PR & address comments 🙇 🙇

@jrhee17 The code has been changed after your approval. I was wondering if you want to check the code again or if you already have done.

Gentle ping @trustin. The usage of obtrude*() has been removed. PTAL. 🙇♂

I was wondering if you want to check the code again or if you already have done.

I think it should be fine to go ahead as I don't think the changes are in a critical path 👍


Yoroi for iOS is unable to sent any amount of Native Tokens out of Nine Quintillion.

I've sucessfuly minted 9,223,372,036,854,775,807 tokens which I've called ATHOMs.
Yoroi for iOS is unable to sent any amount of ATHOMs from the Wallet containing bit more than Nine Quintillion, no matter 1 or One Quintillion. It just hangs after hitting "confirm" button.

Dear Sebastien, I'm totally disagree with your point. For example Daedaus Flight is able to send up two Ten Qadrillion Assets, althougth also having issue to send more.
If user can mint 9,223,372,036,854,775,807 tokens, user should be able to work with this tokens. Who are we to decide he should avoide it? There can be a usecases.
I have to add at the moment I've checked Yoroi, I've sent the sagnificant part of 9,223,372,036,854,775,807 tockens with Daedaus Flight, and in case of Yoroi statement it has to deal with near max value is not correct, it is not even close to this limit as it was less by hundred of quadrillions...

I would append also that during this exercise I've found issues in every piece of IOHK software I've used: cardano-node, Daedalus and Yoroi, which clearly shows Verifacation across the whole software team has a way to improve.
I'm not aware of any Bug Bounty program in Cardano ecosystem, so I did this exercise Pro Bono to benefit the community.
And It's not a kind of a good manner to close the eyes on subject clearly stating software you develop has an obvious bug.

I haven't "closed my eyes on the subject" which is why this issue is not closed. I'm just letting you know that unless the Cardano specification is changed, there are limitations to how tokens with near signed 64-bit values are handled and so it's best to avoid these. This is not a personal decision from the wallet perspective -- this is is a inherit limitation of the system.


nextGenConfig init changes

nextGenConfig init changes
Reasons for changing nextGenConfig:
Currently when a user is configuring next-generator, they need to manually create a file called: "nextGenConfig.json", And fill in the correct information. This is acceptable, but not the best user experience. They should be able to easily generate this from a build in command that feels natural.
Requirements to close this issue:

[x] Create a interactive cli which generates a default nextGenConfig.json automatically. Even if the information entered is standard and not dynamic for now.
[x] This command should be called by entering "nextGen init" in the terminal.
[x] Jest tests written for correct functionality.

In progress, To be completed by Jan 9th, 2022.

Skipping detailed jest tests at this time.


Menu item disabled property added

Close https://github.com/radzenhq/radzen-blazor/pull/1240

Hello Enchev,
I tried it also today to get my code working, I was on the completely wrong path, I guess
So I thought getOpenEvents tells the Menu which Icon is clickable and which is not, but now I realize it tells the menu bar before the menus are loaded. So my code can't work because I am checking the current state and therefore onclick is already set to clickable?
Or is this guess also wrong?
Thanks for adding it, very cool to see how you did it

Hi @Neoxflo,
There are two click event related methods. The first one will raise the Blazor component Click event and the second one will handle open/close of the menus with plain JavaScript.

Ahhh okay now I see, thanks for implementing this, I am pretty sure I didn't get this working by myself.


Fix request: clone default branch, not just "master"

I realized it clones exactly master branch of a repo, I guess it supposed to clone default branch (master branch isn't always equals to default branch of a repo).

Did you read the documentation? It has examples for just about everything. You can give it a branch, or tag.
" Using a non-master branch Plug 'rdnetto/YCM-Generator', { 'branch': 'stable' }

@kodaman2: I understood this as a proposal to make the default branch to clone (i.e. when no branch was given explicitely like you suggested), the default branch of the repository (which often but not always is master). I think this is a sensible thing to do and also what I would expect. What e.g. happens, when I make production my default branch and entirely remove any master branch? Why would vim-plug require explicit branch specification for this repository if git provides a way to ask 'If I have no clue which branch to take, which one should I pick?'. :wink:

I misunderstood OP question, sorry.
Looking into the code vim-plugin has "master" as a string variable. L103. Will have to modify the command where it calls git to get default branch instead which is entirely possible, but to be honest most vim plugins have default as master, I don't know why you would change the default to something else other than master.
L554-L561
I guess there are far better ways to handle from which branch does it clone by vim-plug. vim-plug could select based on latest tag, or latest branch, but then again latest branch might be a WIP. Would be far ideal to pull the latest version tag.

I think default should be what the repository owner sets as default, so the 'default' git branch as proposed here. That said I agree that naming this branch master is a very common convention and if implementing this is non-trivial, I'd totally understand keeping master hard-coded.

Maybe give the option to the user on the config file, if they want plugins to search for default or master could be a good idea in the case for those plugins which master is not the default.
Or do this check when installing a plugin where master is not the default give a warning and let user select branch?

Did you read the documentation? It has examples for just about everything. You can give it a branch, or tag.
" Using a non-master branch Plug 'rdnetto/YCM-Generator', { 'branch': 'stable' }

I know how I can do it, I'm about that default behavior isn't perfect.

How many plugins have you found which the default isn't master out of curiosity.

@kodaman2 My own fork of a plugin while I'm keeping master branch up-to-date to original repo.

@kodaman2 Well, I just realized there are more:
grep ^Plug plugins.vim \
  | sed "s/^Plug '\([^']*\)'.*$/\1/" \
  | parallel 'printf "%s: " "{}"; git remote show -- "https://github.com/{}" | grep "HEAD branch" | sed "s/[^:]*: //"' \
  | grep -v ': master$'

unclechu/haskell-vim: my-fork
Lokaltog/vim-distinguished: develop

This was discussed several times in the past. You should be able to find some related issues here. While I do not disagree with the suggestion, implementing it correctly without complicating the code too much is actually not trivial (e.g. branch validation in PlugStatus, how to find out if the default branch has been changed in the origin after we cloned it, etc.), so it's not exactly something I'd like to look into now. But patches are welcome.


macOS Monterey build failed

external/png_archive/pngwutil.c:14:10: fatal error: 'pngpriv.h' file not found
#include "pngpriv.h"

I have the exact same error on Fedora 38


move CheckPassed and CheckFailed to integration folder

move CheckPassed and CheckFailed to integration folder in order to make it available for integration tests

Just saw that moving CheckPassed to test.h is not sufficient in order to use it e.g. in get-random.int.c:
In file included from test/integration/get-random.int.c:3:0:
test/integration/get-random.int.c: In function ‘test_invoke’:
test/integration/test.h:5:5: error: implicit declaration of function ‘DebugPrintf’ [-Werror=implicit-function-declaration]
DebugPrintf( NO_PREFIX, "\tpassing case:  " );
^
Should I move all test helper functions like "DebugPrintf()" to test.c?

The CheckPassed and CheckFailed are only used in tpm_client and they eventually will be removed after we clean it up. The integ tests do simple if (rc != TSS2_RC_SUCCESS) and print_fail() in case of  failure.


[WIP] Buffer: Initial implementation of buffer accounting for H2

Signed-off-by: Kevin Baichoo<EMAIL_ADDRESS>
Commit Message: Track the number of bytes allocated in buffers of H2 streams.
Additional Description: This is still a WIP, will need to add some tests. Moreover, this currently have each stream charge itself for bytes in its buffers, while really we'd want to change the downstream stream that originated the upstream streams.
Risk Level: medium
Testing: To be included
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
Related #15791

/assign @antoniovicente

I've now simplified the slice coalescing @antoniovicente, slice charges do not transfer over.

LGTM. @antoniovicente any other comments?

Also LGTM, no additional comments.

/retest


Add a menu option to download SnapGene DNA file format files

It could be useful to be able to download SnapGene DNA files.  This would use the SnapGene server to convert GenBank files to SnapGene DNA files.  Note the SBOL would need to be first converted to GenBank.  This is all handled now in the write function in libSBOLj.
Note that upload of SnapGene DNA files is automatically supported via libSBOLj's read functionality.

Essentially, add a snapgene option to the download dropdown. Would need to check but SnapGene has their own file type? Would probably be a download plugin.

Supported via a plugin


CumulusCI incorrectly persists extracted data when using Autoincrement Primary Keys and a step contains an after lookup

CumulusCI does not correctly rewrite lookups to autoincrement PKs for after steps. This is because rewriting is done after each step, meaning that after references are not resolvable at the time CumulusCI attempts to rewrite.
The outcome is an extracted data set that incorrectly contains Salesforce object Ids in relationship fields which are not resolvable at load time, breaking the data set.
To fix this issue, we will move rewriting object Ids to the end of the extraction, instead of occurring with each step.
Reference W-7898122.

Fixed in #2002.


_jukebox_core_download_prebuild_libzmq_with_drafts installation routine copies everything in /home/pi into /usr/local

See https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/6b72a21f5c5e5906ded14eb87a79b763d6a5f2a9/installation/routines/setup_jukebox_core.sh#L61
I think the script should only rsync include and lib folders, not all contents of ~. Maybe the script should download and unpack the archive into a temporary location as well.

Thanks a lot. I'll have a look. This should definitely be corrected!

I think this was a fluke, and ZMQ_TMP_DIR ended up being unset for some reason on my system. I did not see the same again when retrying the install. I think this can be closed, sorry about the false alarm!

ZMQ_TMP_DIR ended up being unset for some reason on my system

Any idea how that happened? Did you do anything special during the installation?

I think what happened, is that the directory already existed on my system (from a previous install). The code only CDs into that directory, when mkdir ${ZMQ_TMP_DIR} succeeds.
When the directory already exists, mkdir will fail and the rest of the following line will not be executed: cd ${HOME_PATH} && mkdir ${ZMQ_TMP_DIR} && cd ${ZMQ_TMP_DIR}.
I guess it would be good to remove the temporary directory after that step has completed.

Good analysis! Definitely looks like something we should make more robust in the install script. I am reopening this issue. @pabera: over to you, again.

Closing as this should be fixed in #1749
feel free to reopen, if the issue still occurs.


Option to select all matching entries in multi selects

Hi again :)
I would like to be able to select all matching entries, tried using code from
https://github.com/select2/select2/issues/195#issuecomment-240130634 (jsbin showing it), but when I use it, no options previously inside form are listed.
Any simple way to make it work, or that's more complicated issue?
Edit: Thrown the code inside the form template, if that's relevant.

It works for me when I select "California", then open select2 again and click "Select All". It moved a bit, but it's there. How to reproduce your issue exactly ?

I tried to use it on modelform which has 2 autocomplete fields. Person, that is directly taken from model and Report, that queries reports based on person chosen.
JS is simply dumped to this form template js section.
When I'll be back home I'll try to reproduce it on test project.

The same behavior can be observed in test project - added script to select2_gm2m template:
https://github.com/marekjedrzejewski/django-autocomplete-light/commit/3cb7006577221d75dd8fb8ac24a09d69edb04e0b
and 'select all' button appears within form fields, but options are no longer there.

@jpic Did you have any chance to look at this?

Any insight on this? :)

I needed this and i needed this with implemented with ajax(because of the large select options).
Here is my overly hacky/completed solution for this:
In form field i keep the default widget and don't use dal widget. Also I set the queryset to none since I will be using the ajax to get the items.
forms.py

    terms = ModelMultipleChoiceField(
        queryset=Term.objects.none(),
        required=False,
        # widget=autocomplete.ModelSelect2(url='dove:term_autocomplete')
    )

Difference in view class is that I set the paginate_by to a high number so I don't have to deal with pagination.

# views.py
class TermAutocomplete(autocomplete.Select2QuerySetView):
    paginate_by = 1000

    def get_queryset(self):
        # Don't forget to filter out results depending on the visitor !
        if not self.request.user.is_authenticated:
            return Term.objects.none()

        qs = Term.objects.all()

        if self.q:
            qs = qs.filter(label__icontains=self.q)
        return qs

In javascript:

I manually initiate the select2 on the select field.
We get objects with ajax call. However objects when displayed in select2 dropdown don't have their pk.
Using the select2 templateResult I added back object pk to dropdown li elements.
After search is made I added at the top of the search field a "Select All" button.
Since there are no options in select we add the options using the pk's we added with template.
Lastly select them all using .val()

function formatResult(r) {
    return $('' + r.text + '');
}

  $("#id_terms").select2({
      minimumInputLength: 3,
      templateResult: formatResult,
      ajax: {
        url: "{% url 'dove:term_autocomplete' %}",
	dataType: 'json',
        cache: false,
	// Additional AJAX parameters go here; see the end of this chapter for the full code of this example
      }
  });

  var terms_search_field = $("#id_terms").data("select2").$selection.find('.select2-search__field');
  var terms_label =  $("#geneset-form-div")[0].querySelector("label[for='id_terms']")
  terms_label.innerHTML += '<button type="button" id="clear-all-terms">Clear All</button>'

  $("#clear-all-terms")[0].addEventListener("click", function(){
    $("#id_terms").val(null).trigger("change");
  });

  terms_search_field[0].addEventListener("keyup", function(){
      if (terms_search_field.val().length >= 3) {
        var results = $("#select2-id_terms-results")[0];
        if (!results.innerHTML.includes("select-all-matched-terms")) {
          results.innerHTML = '
<button type="button" id="select-all-matched-terms">Select All</button>
' + results.innerHTML
        }

        $("#select-all-matched-terms")[0].addEventListener("click", function(){
          var matches = [];
          $.each($(".select2-results__options").find('li'), function(i, item) {
              var term_id = item.children[0].id.split("-").splice(-1)[0];
              var term_text = item.children[0].textContent;
              if (term_id.length > 0 && term_id != "terms"){
                if ($('#id_terms').find("option[value='" + term_id + "']").length) {
                    $('#id_terms').val(term_id).trigger('change');
                } else {
                    // Create a DOM Option and pre-select by default
                    var newOption = new Option(term_text, term_id, true, true);
                    // Append it to the select
                    $('#id_terms').append(newOption).trigger('change');
                }
                matches.push(term_id);
              }
          });
          $("#id_terms").val(matches).trigger("change");
          terms_search_field[0].value = "";
          $("#id_terms").select2("close");
        });
      }
  // Do something
  });


Is there a way to directly remove the huge splat? These splats are affecting almost my entire scene.

Is there a way to directly remove the huge splat? These splats are affecting almost my entire scene. I don't want to use reset_every property because it causes the splat to move towards the camera, resulting in more floaters. refine_every property seems to be unable to remove the huge splat that is far away.

have you tried the mcmc strategy? python simple_trainer.py mcmc


Can't install via npm install shelltest

npm install shelltest
npm ERR! fetch failed https://registry.npmjs.org/shelltest/-/shelltest-1.0.0.tgz
npm WARN retry will retry, error on last attempt: Error: fetch failed with status code 404
npm ERR! fetch failed https://registry.npmjs.org/shelltest/-/shelltest-1.0.0.tgz

For me it works only via using command
npm i git+https://github.com/jolshevski/shelltest.git

Does this work as intended or it's some problem with publishing of the npm package?

For me it works only via using command
npm i git+https://github.com/jolshevski/shelltest.git

Yeah that's strange - thanks for the heads up. I bumped the version to 1.0.1 and re-released; now it's installable from NPM.


Publishing latest updates

Pushing PnP PowerShell update and SharePoint name fixes.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Disable-MailPublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Enable-MailPublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Enable-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Export-JournalRuleCollection.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Export-TransportRuleCollection.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EmailAddressPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-IntraOrganizationConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailPublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxExportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxExportRequestStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxImportRequestStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxRestoreRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MailboxRestoreRequestStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MapiVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-MoveRequestStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-PublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-PublicFolderClientPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-PublicFolderDatabase.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-PublicFolderItemStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-PublicFolderStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-Recipient.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-RoleGroup.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-AutodiscoverVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-FederationTrust.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-HybridConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-Mailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MailboxExportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MailboxImportRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MailboxRestoreRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MailboxSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MapiVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MigrationBatch.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-MoveRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-OfflineAddressBook.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-OwaVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-PartnerApplication.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-PublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-RetentionPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-RetentionPolicyTag.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-SendConnector.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-ThrottlingPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-TransportRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-WebServicesVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-ApplicationAccessPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-PublicFolder.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Remove-PublicFolderClientPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Resume-MoveRequest.md
:white_check_mark:Succeeded
View (exchange-ps)

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


add deployer owner resource

How to categorize this PR?

/area deployers
/kind enhancement
/priority 3
What this PR does / why we need it:
First part of https://github.com/gardener/landscaper/issues/246 .
Implements the shared custom owner crd.
Also enhance the crd generation so that crd's can be written to different directories
Special notes for your reviewer:
Release note:

NONE

If ownership objects could be modified doesn't this result in a quite complicated synchronisation problem if one deployer is just working on an item and another is taking over?

Owner objects are not for synchronisation. Their main purpose is that the responsibility for targets could be modified during runtime.
Also these owner objects are not part of the Landscaper but part of the deployer. So e.g. they live in the fenced cluster when the deployer is deployed their.
So they are just local configuration.
With the current approach, using annotations or labels, we have the same issue. There could be multiple deployers reacting to the same deploy items.


added supdock to UI terminal section

What's Up, Dock(er)? A slightly more visual way to interact with the docker daemon. Supdock is a wrapper for the docker command meaning you can still use all of the other docker commands without issues.

https://github.com/segersniels/supdock
Most of the user interface enhancing community projects for Docker take Docker that slight bit too far. Supdock aims at optimising your workflow without bringing too much overwhelming changes to your standard docker usage.
By submitting this pull request I confirm I've read and complied with the below requirements.
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.

I have read and understood the contribution guidelines

The project submitted conforms to the quality standards outlined in the contribution guidelines

Go to the contribution guidelines and read it again.

RULES (because you probably didn't read the guidelines):
Links will be tested with awesome_bot. Please make sure your URL to be listed is unique (do some search queries), otherwise put a link at the end of README.md file.

FORMAT

Use the following format: [Name](Link) - Description.
[Name] should be the title of the project
(Link) should be pointing to a GitHub repository, not to a website
Description. should end with a full stop/period
For GitHub repositories, add the name of the author with a link to its GitHub account after the description, like By [@author](GitHub account)

ORDER: The entries are in lexical order (English letter order)
LABEL (Optional): Add one of the following icons next to the project name like [Name](Link) :icon: - Description.

:heavy_dollar_sign: indicates if one needs a paying subscription to use the project.
:construction: indicates that the project is a work in progress/beta release.
:skull: indicates that the project is obsolete, but still worth mentioning it.

@segersniels Nice project! Well done. 👍
Thank you for contributing


Blend Map export

Am not sure if its bug or am missing something but why is exported blend map transparent with black spots and not like depicted in export dialog RGBA blendmap?

It is because we use all R,G,B,A channels for texture splatting in renderer.
Dialog RGBA blendmap show you image without alpha channel for visually good looking.
The transparent with black spots what you are looking in the filesystem is caused by alpha channel of the blendmap.
It is normal behavior :)

Oh I see, so how do I go about making it look like without alpha channel?

Well,, there is shortcut for removing alpha channel, such as opening it with paint tool and saving it again.
But why do you want to see blendmap with only RGB channels?
Removing alpha channel from blend map means removing one terrain layer from the result.

Good point, I need to keep all 4....

Thank you for your interest to my repo.
Seems like working well.
I'll close the issue. Please re-open the issue whenever you need help 😁


Implement additional support for build

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #19, #20, #21
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
The changes in this PR include:

Adding a tsconfig.json file to support typescript users
Remove all legacy IE support
Update version of chalk package

Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Strict Null Checks

Will force us to handle null and undefined
See docs

Don't use non-null assertions


Use weak self references in service completion blocks

Fixes: #335
In the four table view controllers use a weak self reference in all of the completion blocks passed to service calls. This should help prevent crashes when remote calls finish after the view is dismissed and a table animation exception is thrown.
This is a little hard to review with StatsDemo due to the nature of stats being the root view controller. In reality, the best way to test is to point WPiOS at the local directory on your disk:
pod 'WordPressCom-Stats-iOS', :path => '../WordPressCom-Stats-iOS/'

Needs Review: @koke

I can't get it to crash now :clap: :shipit:

This is a good case where I felt memory leaks wouldn't happen (and didn't) but lifecycle problems came about not using weak references.
Thanks @koke :smile:


Perfect Idler

Would it be possible to add
Game.Unlock("Perfect idling") as an option in the settings? Might be considered cheating by some, but still..

Hmm, just as an option?  It is borderline cheating, though it is a cheat built into the game itself.  I'll ponder on this.


Add description column to saved searches table

Fixes #951
This change modifies the database schema to add an optional description column to the existing saved searches table. This new column will allow users to store a brief description of their saved search, making it easier to understand its purpose later.
Other changes:

Add new type: OptionallySet. Previously we allowed callers to let the spanner layer know that we want to update one field by setting a pointer. That worked on fields that were not optional. But since the description can be null, I added this new type to capture that.
Modified tests to use the new description field and ensure that saving and retrieving descriptions works as expected.

Whoops. Thanks for that. I edited the description.


Feature/rebase main #8622

What this PR does / why we need it:
Rebase of PR: #8622 to main
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Is it going to be reviewed soon? I need this feature :)

Closing in favour of https://github.com/helm/helm/pull/10944.


Review pagination SQL tests of Oracle

Content: After #4885 is fixed, this issue begins to process.
Difficulty Rating: ★★☆☆☆
How to fix this one?

[ ] Remove the comments from all pagination SQLs of Oracle in ./incubator-shardingsphere/sharding-sql-test/src/main/resources/sql/supported/dml/select-pagination.xml and ./incubator-shardingsphere/sharding-sql-test/src/main/resources/sql/supported/dml/select-pagination-group-by-order-by.xml
[ ] Run SQLParserParameterizedTest to find targets to fix
[ ] Modify the parsing results of those sql cases to in ./incubator-shardingsphere/shardingsphere-sql-parser/shardingsphere-sql-parser-test/src/test/resources/sql/dml make SQLParserParameterizedTest  pass

This issue has a prerequisite issue.

@jingshanglu Hi could you take charge of this one?


download dataset, dir_metadata_cache

Hi,
I download dataset from here, however I cant see "train_tags.txt" and "development_tags.txt". Also I'm not quite sure that how should I set "dir_metadata_cache" in notebooks 1 and2.

Hi!
Tags files can be found inside train.tar.gz and development.tar.gz respectively.
Regarding metadata, there is a description in the readme section: https://github.com/yandex-research/shifts/tree/main/sdc#caching-dataset-metadata-for-downstream-analysis.


GooglePropertyExtractor - VisionAPI not returning full RGB information

This look more like a bug with GoogleVisionAPI, but for some images (er rather just one), only "red" and "green" are returned in the response, but no blue, leading to a KeyError. I'm going to investigate if its because blue = 0, or because of a random fluke. It would be good to make the extractor robust to these possible responses.
color = {'color': {'red': 68, 'green': 3}, 'score': 0.007824038, 'pixelFraction': 0.014705882}

https://github.com/tyarkoni/pliers/blob/a7ea0d2c69de92bf74388295de0a18311bf0a294/pliers/extractors/api/google.py#L112

This is def a bug in their API, because even when I give the image to the online demo, blue value is "undefined"

Looks like this issue was experienced by the Google API as well. https://github.com/GoogleCloudPlatform/google-cloud-python/issues/2757
They seem to think undefined = 0, so seems like an easy fix to default to zero if no value is present in the response.


Fix for JInterface implementation.

Fixes #547

This one appears to depend on removing Python 2.7 unless I want to reformulate.


Script task - Only supports standard .NET Framework APIs?

It says the Script task Only supports standard .NET Framework APIs, so clearly we are talking about ".NET Framework", and the standard APIs are a subset, but what is that subset?
Are we by any chance talking about ".NET Standard", which I think shouldn't really be labelled as ".NET Framework"
I refer to the Microsoft .NET terms defined here https://dotnet.microsoft.com/learn/dotnet/what-is-dotnet-framework, but they may have changed after lunch.

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

ID: 7bab227c-0449-76f4-b232-96b9f0b658dd
Version Independent ID: df73cb55-55e4-8e76-2943-44168d589624
Content: Limitations and known issues for SSIS on Linux - SQL Server
Content Source: docs/linux/sql-server-linux-ssis-known-issues.md
Product: sql
Technology: linux
GitHub Login: @lrtoyou1223
Microsoft Alias: lle

@sqlgreen -- Darren, thank you for your feedback.
@lrtoyou1223 -- Leo, please look into this issue.

Hi @sqlgreen. The article says "standard .NET Framework APIs", meaning the standard APIs that are included in .NET Framework.
#please-close


Simulate a long network request for just one endpoint

How would I go about doing this in mirage? I tried setting the return value in a setTimeout but I guess mirage is not resolving when I return something? I don't want to set the timing on everything just this one endpoint. What would that process look like?

Currently, this is not supported, though this is a feature in the backlog.
Closing (duplicate of #55).


feat: client side support

This PR

WIP of client side support

Related Issues

Fixes #371
Notes

Follow-up Tasks

How to test

We've decided to not provide client-side deployment abstractions.


can you share your pretrained_model?

can you share your pretrained_model?

Thanks for your interests! I'll need to figure out the internal code permission for the checkpoint release. Stay tuned :)

anything update? @hushell

@KyleWang-Hunter
Sorry for late reply! Just uploaded 3 checkpoints on Meta-Dataset: https://huggingface.co/hushell/pmf_metadataset_dino


WIP: Detect orchestration results when mods has been provided as argument

We usually run the orchestrations manually as:
salt-run state.orchestrate orch.kubernetes
And this case was detected properly. This change adds support for also
detecting the orchestration result when the orchestration is called using
the salt-api on Velum (the general case):
salt-run state.orchestrate mods=orch.kubernetes
With this change we support both return results.

How I tested this: follow the instructions at https://github.com/kubic-project/velum/pull/243#issuecomment-313443754, but instead of running:
docker exec -it $(docker ps | grep salt-master | awk '{print $1}') salt-run state.orchestrate orch.kubernetes

Run:
docker exec -it $(docker ps | grep velum-dashboard | awk '{print $1}') entrypoint.sh bundle exec "rails runner \"require 'velum/salt'; Velum::Salt.orchestrate\""

This will mimic a call to the orchestration using the salt-api backend implementation in Velum.

I can confirm this works as expected with the instructions above: removing WIP flag.


fix: mutation does not delete cache entry

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of
the pull request.
Description

the actual cache entry found for a responseId has never been deleted

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take
action on it as appropriate
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can
reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Environment:

OS:
@graphql-mesh/...:
NodeJS:

Checklist:

[x] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works, and I have added a
changeset using yarn changeset that bumps the version
[ ] New and existing unit tests and linter rules pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose
the solution you did and what alternatives you considered, etc...

Sorry for the late review :)
The fix is already out. I am merging the PR for the credit :)


Implement blacklist_words and mandatory_words in filter.

Improve the filter and targeting of interaction by check the biography text:
Example of filter.json
{
    "blacklist_words": ["sex", "nude", "hot"],
    "mandatory_words": ["pasta", "food", "pizza"]
}

Skip the user if bio contains one word in blacklist array
Interact with the user only if contains at least on of mandatory words in bio

I don't know if blacklist_words and mandatory_words are good keyword and if the logger.info are write correctly (in english I mean). Maybe mandatory_words can be required_word?

When we merge this, we need to add in #16 too since they are the same basically. If someone else adds this, please be sure to credit @ugur-kellecioglu in the commit. If you don't know how to do this; let me know.

i set the alphabet LATIN

but i think we should check the names when an account has no biography
  (it followed that account)
i noticed that when there are only numbers in biography _find_alphabet method return DIGIT and it is same for emojis it returns the emoji name like SPLASH

@ugur-kellecioglu
https://github.com/GramAddict/bot/blob/a2b3716cace81d7384d62843c526e7e47a954096/GramAddict/core/filter.py#L151
Why not +1 and indent all under the line 117
if field_blacklist_words is not None or field_mandatory_words is not None or field_spesific_alphabet is not None:
So you can remove: biography_text = self._get_profile_biography(device)
Just for prevent to call twice _get_profile_biography


Case insensitive string functions

Re https://github.com/mattwparas/steel/issues/65
Does not include string-ci>?, string-ci>=? because Steel does not include string>?,string>=?. I can add them if you want?
I was originally deferring the case insensitive checks to the non-ci versions, but since they accept a SteelString and to_lowercase() returns a String, it seemed not worth re-wrapping the types (?) and I just repeated the comparison code.

Though re-reading this, probably it is best to rewrap to_lowercase() -> SteelString so if < etc is ever overloaded we pickup the behaviour?

Does not include string-ci>?, string-ci>=? because Steel does not include string>?,string>=?. I can add them if you want?

If you could add those as well that would be great! Also per the second point, SteelString is really just a wrapper around Rc<String> - so I think we are safe to just use the to_lowercase() -> String - converting to the SteelString I think won't be necessary for our case

closes #65

Cheers for the quick shepherding.


Require plugin to be applied on root project

The plugin now requires to be applied to the root project to ensure
that each Nexus repository is only declared once. It then adds a Maven
repository for publishing to the root and all subprojects that have the
maven-publish plugin applied.

It would be good to add a simple multi-project build to the e2e suite (probably using Kotlin DSL as the single-module one uses Groovy DSL).

Done.

I see that you have an interesting weekend :). Thanks @vlsi for your input!
It could be handled separately, but can we do something about the warnings the new e2e project?

I see that you have an interesting weekend :). Thanks @vlsi for your input!

Indeed, that was very helpful! 👍

It could be handled separately, but can we do something about the warnings the new e2e project?

Looks like it's a known issue: https://github.com/gradle/gradle/issues/9178


feat: add unchanged resources

Proposed changes
There is a difference between an unchanged resource and a resource that is not showing in the report.
A unchanged resource means the resource from the snapshot has a match in the destination org but is identical.
Similar to other IaC tools, unchanged resources should be display in the preview so the user understand these resources were processed.

Note: I did not know what color to use for unchanged resources. Maybe grey is not the right one...

Testing

[ ] Unit Tests:

[ ] Functionnal Tests:

[ ] Manual Tests:

CDX-389

Was added in #258 while refactoring reportViewer


Issue #SB-0000 fix: Merge release-3.6.0 to release-3.7.0

@vrayulu Merging this PR as this is a blocker for release.
@rajeevsathish Please make sure the coverage report at 3.7.0


Flaky kongintegration test TestParsersGoldenTestsOutputs/expressions_router

Example failure on CI: https://github.com/Kong/kubernetes-ingress-controller/actions/runs/6614735467/job/17965379108
        	Error Trace:	/home/runner/work/kubernetes-ingress-controller/kubernetes-ingress-controller/test/kongintegration/containers/kong.go:84
        	            				/home/runner/work/kubernetes-ingress-controller/kubernetes-ingress-controller/test/kongintegration/parser_golden_tests_outputs_test.go:66
        	Error:      	Received unexpected error:
        	            	context deadline exceeded
        	Test:       	TestParsersGoldenTestsOutputs/expressions_router

In the run I found these messages:
    kong.go:103: failed validating Kong version: HTTP status 404 (message: "no Route matched with those values")
    kong.go:103: failed validating Kong version: HTTP status 404 (message: "no Route matched with those values")
    kong.go:103: failed validating Kong version: HTTP status 404 (message: "no Route matched with those values")
    kong.go:103: failed validating Kong version: HTTP status 404 (message: "no Route matched with those values")

are repeating. The code here is validating Kong version, but the response seems to come from the proxy server of Kong gateway. I think it would be some problems brought by the testcontainer package, bug or incorrect usage made the requests sent to Kong proxy instead of Kong admin API.


Enable processing of Updates in Server

Requires #1607, Requires #1603

@Qup42 I am already playing around with this (in conjunction with #1597) and it works like a charm. I have one request:
Currently, the API call simply returns a string "Update successful". It would be great if for Accept: application/qlever-results+json, it would return a JSON with (for now) the following fields:

A field time with the following subfields, all values in milliseconds: parse (the time for parsing the query), where (the time for evaluating the WHERE clause, or zero if there is none), update (the time for adding the triples to the delta triples and locating them in the blocks), and total (the total time for the update operation).
A field delta-triples with the following subfields: before (the number of delta triples before the update operation),  after (the number of delta triples after the update operation), and difference (the value of after minus the value of before).
A field located-triples with the following subfields: pos with two subfields blocks-total (the total number of blocks in that permutation) and blocks-affected (the number of blocks with at least one located triple), and the same for every other permutation.

I am assuming that all this information is available relatively easily. If not, please let me know. Please also let me know if you think that this belongs in a separate PR.
And somewhat related: In my original (by now, ancient) draft of this, there was a command to clear all delta triples. For example, it could be activated via cmd=clear-delta-triples (it should require the access-token, of course). Would that be easy to add here or should that (also) be in a separate PR?

@hannahbast adding some timing information and further metadata to the update response is already on my ToDo list. Points 1 (time) and 2 (delta-triples) should be no problem. I would extend the delta-triples field by the number of added/removed fields, as these are not knowable otherwise. For located-triples I can't estimate how hard this is.
My current plan was to do the additional info in a separate PR. The PR activating the updates in the server was supposed to be small, so that it can be merged quickly. Before adding this additional information I wanted to have a look at what other engines do here. To be able to run updates some changes are required in qlever-ui, so there was little use to every it earlier at that poing.
But this can of course change depending on e.g. how long it will take to merge #1597 and how much value the additional info would provide for you for while benchmarking.
Clearing the updates is just very little code, I'll add that to this PR.


UI overflow, google stuff removal, adhere to current API.

Fix horizontal overflow (issue https://github.com/visual-framework/vf-core/issues/2122), remove google stuff, fix api update.
The test regarding on pending jobs is not working. I tried to run cypress here and it opened an app and gave me some error. I think the test might not be correct but I don't know how to set it properly.

I have also added the background you suggested.

You can check it online already: http://<IP_ADDRESS>/jobs/21

Nice =) Thank you!


Fix removal of compliance comments for testing purposes.

For testing, we need to insert the comment // Noncompliant in lines where there are rule violations (this is mandatory according to Sonar).
We have a method that removes this comment from the source code after Sonarqube-repair repairs it. However, this method only works properly when we have the simple comment // Noncompliant, and test cases fail when we have a "bigger" comment like // Noncompliant {{Cast one of the operands of this addition operation to a "long".}}. This PR proposes a fix for that.

Thanks a lot Fernanda, + looking at the build of this PR, found #45

Thank you for merging, @monperrus. I will investigate the recently discovered issue.


⚠ Bofrost.at has degraded performance

In 1e0f8d3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1634 ms

Resolved: Bofrost.at performance has improved in c8d5872.


Battery indicator shows wrong value

A fix has been made for this, but please add your comments here if it works or not with your watch.

I have a B5000D and the battery marks me badly. Moreover, it appears degrees Celsius instead of percentage. Because?

The degree Celsius is a separate temperature measurement of the watch. It
is a new indicator. If you put your watch in ice, this reading will slow
lower, but it takes up to half an hour to update.
About the battery, are you saying it is not showing properly? Is it showing
too low?
Thanks for your feedback.
On Tue., Jul. 18, 2023, 12:44 Thv1337, @.***> wrote:

I have a B5000D and the battery marks me badly. Moreover, it appears
degrees Celsius instead of percentage. Because?
—
Reply to this email directly, view it on GitHub
https://github.com/izivkov/CasioGShockSmartSync/issues/43#issuecomment-1640573297,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7M37UZZSP4JDXCKE77QV3XQ24O3ANCNFSM6AAAAAA2LLUBP4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Seems like did not get a lot of feedback, so I will consider it working.


Add property to disable some test resources resolvers

This commit introduces a property to disable a test resolver. It was implemented in a backwards compatible fashion, by introducing a new subtype of resolver. Resolvers which can be disabled need to implement the ToggableTestResourcesResolver interface and provide a name.
This name is used at resolution time, to figure out a test resources property. For example, if the name of the resolver is foo, then at resolution time, we'd look into the test resources configuration for the property:
test-resources.foo.enabled
and ignore the resolver if that property is false. This flag should only be used for debugging purposes.
Fixes #399

I think the test-resources: properties are only resolved from the files.

It will work if your test implements TestPropertyProvider (see https://micronaut-projects.github.io/micronaut-test-resources/latest/guide/#extensions-core)

Cool, I assume this will work with https://github.com/micronaut-projects/micronaut-data/blob/master/data-r2dbc/src/test/groovy/io/micronaut/data/r2dbc/postgres/PostgresRepositorySpec.groovy

Yes it should


near and storage tracing - wip

Hm, I'm not 100% sure you actually handled the upgrading - i see direct self.account.get calls.
I think the easiest way is to keep account_deposit as old field and create new lookup map with VersionedAccount. And just when loading - check back if it exists and upgrade.

I've asked @evgenykuzyakov  about possible strategies few weeks ago. I think if we won't use indexer then then doing byte computation for figuring out the structure version is too error prone.
So, depending which strategy we will take. If we want to do it lazy (so user will do migration when he will use the account next time) then we will need to duplicate the accounts - if we will use indexer then we will do a migration function. We had a chat about it in telegram. Here I wanted to verify the AccountDeposit structure and update.

#27 adds storage handling without needing to upgrade Account data structure. This fixes the issue short term and removes need to keep track storage_used from it for now.