Make v1 6 0

What does the code in this PR do / what does it improve?
New release test processing is currently running, but dali is super slow. Also postpone to push tags after test processing passed.

Coverage increased (+0.03%) to 93.801% when pulling 2cdaa76d6faf2771ae4944ab4872de6cd5079d06 on make_v1_6_0 into 8a8d46957175867220fccc094b2287ad8190e538 on master.


[Test] Region prompt

region 0 / region 1 / background
lora for region 0/1
ip adapter for each region
controlnet_openpose

region 0 / region 1 / background
lora for region 0/1
ip adapter for each region
controlnet_openpose

region 0 / region 1 / background
lora for region 0/1
ip adapter for each region
controlnet_openpose

region 0 / background
lora for region 0
ip adapter for each region
controlnet_openpose

This is so great and I'm testing for it, may I understand that we can use lora in each region but  backgroynd?

background is also a region
The remainder after cutting out each region's mask is the background.
# new lora_map format
  "lora_map": {
        "path_to_models/models/Lora/zs_Abdiel.safetensors": {   # setting for abdiel lora
            "region" : ["0", "background"],            # target region. Multiple designations possible
            "scale" : {
                # "frame_no" : scale format
                "0": 0.75           # lora scale. same as prompt_map format. For example, it is possible to set the lora to be used from the 30th frame.
            }
        },
        "path_to_models/models/Lora/zs_Amanazoko.safetensors": {  # setting for amanozako lora
            "region" : ["1"],            # target region
            "scale" : {
                "0": 0.75
            }
        }
  },

Thanks!
if there are 2 region to control, I name region 0,background for background, and region1, region2 for other character?
in this case how do I name those region in the region map section?
also one more question, if you can setup lora by region, does it possible to setup controlnet in region only?
thanks!!!!!

motion lora

Thanks!
so my understand as below, please correct me if I'm wrong, thanks!
"lora_map": {
"path_to_models/model_00"
"region" : ["0", "background"],   *--->this means prompt will use root setup?
...
}
},
"path_to_models/model_01": {  # setting for amanozako lora
"region" : ["1"],
...
}
},
"region_map" : {
I don't need to setup region 0 because it already been setup at root?
"1":{}

xxxx
}

"background" = root

Great! BIG THANKS!!!!

do you have the configs for the first, second, or third generation here?


XPathEvalError-82-add-namespace

Solve this:
https://github.com/Shoobx/xmldiff/issues/82

I think this is trying to solve a problem that doesn't exist.


olli spec refactor

This is a refactor of the olli spec and internal structure nodes.
Changes:

consolidates the FacetedChart and Chart types into one type that can be distinguished by checking 'facet' in spec
makes most fields in the spec optional. the only required field in the olli spec now is data. what does this mean? users who want to create accessible textual structures don't need to create a visualization first or even use an adapter. instead, they can supply data without any visual information and olli will still render a tree, with descriptions that don't reference a visualization
exposes the structure of the tree as part of the spec. this field is optional, so users will still get the default structures for free using adapters or with defaults. but users can also provide custom structures by writing their own olli spec or merging a custom structure spec with the olli spec output by an adapter
new, predicate-based interface for structure nodes. this drastically simplifies the structure node. there are now two types of nodes: predicate and groupby. predicate nodes represent a selection over data, and groupby nodes take a field and generate child equal-predicate nodes for each value / bin of the field.
instead of generating a table within the tree, users can now press a key to open a dialog view that contains a table. users can open a table at any predicate node, or the root for the full dataset. using a separate dialog that's not part of the tree makes the table work better with built-in screen reader keys because tree navigation key bindings won't conflict.

TODO:

[x] fix the vega and observableplot adapters

The new APIs are not backwards compatible so will have to bump up to 2.0.0. But users who use the built-in adapters and upgrade both core and adapters to 2.0.0 should experience no issues, since the adapters will take the same input and generate the new spec.

addresses these tickets

https://github.com/mitvis/olli/issues/7
https://github.com/mitvis/olli/issues/74
https://github.com/mitvis/olli/issues/51
https://github.com/mitvis/olli/issues/50
https://github.com/mitvis/olli/issues/47
https://github.com/mitvis/olli/issues/45
probably https://github.com/mitvis/olli/issues/43 (needs more testing)

Added in selections in #78, which addresses https://github.com/mitvis/olli/issues/49


Question mark characters

Hey all,
Thanks for VimBox! I'm just now trying to get back into using vim to see what's changed.
I followed the README, but I just noticed that there are several weird characters in the status line:

Any idea what those are?
Thanks,
Alexis

This is using vim-airline, which expects that you use patched fonts by default. VimBox includes them and the installation instructions have a step which would enable them.

With the fonts, it looks awesome like this.
It would be great if someone would send a PR that detects if the user is using a guifont that supports airline arrow symbols and then automatically turns on the arrows based on that.

Reopen if needed.


edge function geo is not working (undefined values)

Middleware in my api folder.
import { NextResponse } from "next/server";

export async function middleware(req) {
  const { nextUrl: url, geo } = req;

  console.log(url);
  console.log(geo);

  if (url.pathname === "/api/ip-info") {
    const country = geo.country || "";
    const region = geo.region || "";

    url.searchParams.set("countryCode", country);
    url.searchParams.set("region", region);

    return NextResponse.rewrite(url);
  }

  return NextResponse.next();
}

Logs from Vercel.
/api/ip-info

{ city: undefined,
  country: undefined,
  region: undefined,
  latitude: undefined,
  longitude: undefined }

What am I missing?
The request is triggered via the CookieConsent consentTypeRequired function.
    <>
      <Head>
        <meta
          name="viewport"
          content="minimum-scale=1, initial-scale=1, maximum-scale=5, width=device-width, shrink-to-fit=no, viewport-fit=cover"
        />
      </Head>
      <DefaultSeo {...SEO} />
      <CookieConsentProvider consentTypeRequired={consentTypeRequired}>
        <Component {...pageProps} />
        <CookieConsent />
      </CookieConsentProvider>
    </>

Is there anything I could simplify? Currently I have a fallback in the ip-info api that uses external services to determine the geo. However that causes initial delays.

The same as #34

Should work correctly within a paid plan.

This is being changed this week, should be resolved soon!

This should be fixed! 🎉


Issues with table rendering: extra s and lost html markup

During table conversion from markdown to html, an html
 elements are inserted by the plugin inside the table cells.
I find this unexpected and undesirable.
Could you please consider and remove unspecified elements from the table cells?
On a related issue. At the moment, standard table rendering allows a certain mix of markdown and html markup. This works with tables as well. For example, the following markup would be properly rendered:
> Number of patients (%) | Drug 1
(n=130) | Drug 2
(n=128)
> ----|:----:|:----:
> **Total number of TEAEs** | 593  | 591
>
Study drug-related
 | 74 (57) | 64 (50)

Alas, during the plugin rendering those html elements are lost. Can this be fixed?

This should be fixable. The root of the problem is that I am using the markdown converter+renderer built into Obsidian's API which isn't the most ideal. I suspect this is also the cause of #7 and I will try to migrate to a different markdown parser with the same featureset

I didn't end up changing the markdown renderer but instead used a hacky solution of removing the
s. This problem is now resolved by disabling this in settings:

nvm I didn't actually finish the whole issue - html rendering now moved to #13


CSSNano Minify Script Running AutoPrefixer a second time

We're running Autoprefixer earlier in the build process, and CSS Nano is currently running it a 2nd time and overriding any custom Autoprefixer options (back to default) that ran the first time. One way to fix this is in the "minifyStyles.js" gulp task file change the nano line to:
.pipe(cssnano({autoprefixer: false}))

Hey Brad, mind trying this in the latest setup? This seems to be working okay on my end!
/* Source CSS */
div {
  display: -webkit-flex;
  display: -ms-flexbox;
  display: flex;
  box-shadow: 0 0 2px #ff0;
  -webkit-transform: translate(-50%, -20%);
          transform: translate(-50%, -20%);
}

/* Minified CSS */
div{display:-webkit-flex;display:-ms-flexbox;display:flex;box-shadow:0 0 2px #ff0;-webkit-transform:translate(-50%,-20%);transform:translate(-50%,-20%)}

Hi Aaron, I gave this another try this morning and it still looks like once you run a build to the "dist" folder CSS Nano sets Autoprefixer settings to default (last 2 versions). For example, in the gulp task file "styles.js" try setting your autoprefixer on line #19 to "autoprefixer({ browsers: ['last 3 version', 'Explorer >= 9'] })"
That adds the "-ms-" prefix to CSS transforms and works great in the _tmp stylesheet, but once you run a build into dist the "-ms-" prefix will be gone. The fix I mentioned above seems to do the trick and prevents autoprefixer from running a second time.
Is that the case for you as well? Maybe I need to adjust something in my setup?

Ooooh, wow you're right! I'll correct this in the latest starter version. Thanks for reporting!


bug: cors should be at header_filter phase

Issue description
the cors headers can be overridden by upstreams if it's put at rewrite phase.
Environment

apisix version (cmd: apisix version): 1.4
OS: Linux

Minimal test code / Steps to reproduce the issue

Create a backend which override the cors headers, such as Access-Control-Allow-Origin
Configure cors plugin to proxy to the upstream setup in step 1
Request the uri configured

What's the actual result? (including assertion message & call stack if applicable)
the cors header will be determined by the upstream
What's the expected result?
the cors header should be determined by the cors plugin.
This is related to antoher issue: https://github.com/apache/incubator-apisix/issues/1528
I don't think it's good practice to change the phase of cors plugin to make sure it works with the auth plugins.
The cors plugin worked in header_filter phase since it needs to rewrite those headers after apisix gets response from upstream. If we change it to be at the rewrite phase, it's possible the header got overridden by upstream or the header is discarded somewhere.

@ShiningRush do you have to look at this issue?

@JanLi-air many thx for your report this bug

It related with #1704, I will fix it in this week.
I think user should not concern at which phase cors plugin work, they just care whether the cors plugin overwrite upstream's header.
BTW: make sure cors plugin work fine with auth plugins is a better way, because the Browser just report the cors error  when authenticated failed.
It is unfriendly to front-end engineer

if cors plugin needs to work with auth plugins, i think we can set header both at header_filter phase and after auth failure.
But since auth plugin will return before cors plugin, and we need to keep plugins independent with each other, we'd better write cors headers at both rewrite phase and header_filter phase.

Yes, you are right : )
The cors plugin will write header in rewrite phase or both rewrite and header_filter phase, it depend on is_overwrite_upstream.
But now here is a strange problem that it seems if we write header in rewrite phase and upstream also write header , client will get a duplicate headers, it confuse me.

it depends on how we write the header. because in http you can have multiple headers with identical name, so if you want to override you need to first delete the existing one.


Generic with extends and return will fail compilation

TypeScript Version:  2.4.2
class A {

}

export class B {
    public method<T extends A>(val: T): Promise<T> {
        return new Promise(resolver => {
            resolver(new A());
        });
    }
}

It fails with this
Argument of type 'A' is not assignable to parameter of type 'T | PromiseLike'.
Type 'A' is not assignable to type 'PromiseLike'.
Property 'then' is missing in type 'A'.
This code compiled just fine in 2.3
It looks similar to this old bug which compiled fine with 2.3, but fails with 2.4
https://github.com/Microsoft/TypeScript/issues/13993

Not reproducible in the playground.

This is working as intended and is an effect of #15104. Type A is not assignable to type T because T represents some type that derives from A.
We also now handle #13993 correctly (by reporting an error), but for a different reason. It was fixed by #16368.

@ahejlsberg then maybe I am looking at this where T : A C# thing but in TypeScript?
The documentation does not make any other mention of this? https://www.typescriptlang.org/docs/handbook/generics.html
Is it possible in 2.4?

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.

@mhegazy could you kindly let me know if there is such a thing as I asked in the previous comment?
I am staying on 2.3 until I figure this out...

Is it possible in 2.4?

no. this is flagged as error in 2.4 +


methods should be underscore_case

I think I may have caused some confusion when changing things. According to PEP8, class names should be CamelCase (source) and instances and methods should be underscore_case (source), so the syntax should be:
dsetview.lazy_slice[1:10:2,:,0:50:5].lazy_transpose([2,0,1]).lazy_slice[25:55,1,1:4:1,:]

The methods use the correct naming convention now.


use offset strategy to select random data

Related to https://github.com/zooniverse/Panoptes/pull/1358#issuecomment-139301488 - this technique should randomly offset into the available data, select a window over the data and randomly sample into that window.

so this strategy seems to scale linearly but i'd say becomes too slow at larger offsets with larger table spaces. I tested synthetic data up to 2 million set member subjects (2xworkflows, 2x subject sets with ~50% assignment of the data into the selecting workflow - so probably not super realistic of the way we select data).

see, https://github.com/zooniverse/Panoptes/pull/1358#issuecomment-139619412 for details on the selection distribution vs timing benchmarks.

closing in favour of #1358


provide tighter control over number of database connections

This provides tighter control over the number of db connections established by tiled.
In particular, it makes the sqlalchemy max_overflow parameter configurable and avoids creating a separate engine during app startup. This limits the number of established connections to the value expected from the configuration.

Power-cycled to re-run CI now that the main branch is passing again.


Add GET_LATEST_BAZEL_VERSIONS method

It returns a certain number of bazel versions. It allows you to test the
latest 3 minor versions of Bazel without testing all the patch versions
or having to hardcode any values.

Thanks for the review. PTAL


testAutorefreshAfterBackgroundUpdate often times out on verify-ios7-static

Almost always.

We haven't seen this happen in a long time, so closing! :tada:


refreshAppRegistrations before OTA

When the trollstore drops the signature, the OTA will fail.
The system freezes and can only be forced to restart

Please explain this better, it seems like a lot of your mesage got lost in translation. What do you mean by "When TrollStore drops the signature"?

When you need Persistence Helper

There is still a problem, maybe jbd crashed


test(e2e-tests): adds FF for check your answers

test(e2e-tests): adds FF for appellant submission check your answers
Ticket Number

UCD-1318
Description of change
adds FF for appellant submission check your answers
Checklist

[ ] Requires infrastructure changes
[ ] If adding or remove environment variables (e.g. in docker-compose.yaml) then I have updated the appropriate Helm chart
[ ] I have updated the documentation accordingly
[x] My commit history in this PR is linear
[ ] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

ty

ty


bevy_reflect: Trait casting

Objective

Note: The majority of this PR was implemented by @kabergstrom. They kindly asked that I head up the PR due to possible unavailability to make any requested changes. But all credit should go to them!

Trait Casting
Currently the best way of converting a reflected type to a trait object is to go through a multi-step process wherein a user must:

Mark the trait with #[reflect_trait]
Import the generated ReflectSomeTrait
Mark the implementor structs with #[reflect(SomeTrait)]
Get the stored ReflectSomeTrait from the registry
Use the ReflectSomeTrait to cast your reflected value to &dyn SomeTrait

Example Code
#[reflect_trait]
trait SomeTrait {
  fn foo(&self);
}

#[derive(Reflect)]
#[reflect(SomeTrait)]
struct MyStruct {}

fn try_cast(value: &dyn Reflect, registry: &TypeRegistry) -> &dyn SomeTrait {
  let reflect_some_trait = registry
    .get_type_data::<ReflectSomeTrait>(value.type_id())
    .unwrap();

  reflect_some_trait.get(&*value).unwrap()
}

This is both complicated and annoying to code out, with lots of places to go wrong. Ideally there would be a way for casting to Just Work™.
Solution
Trait Casting
Added the following methods to TypeRegistration:

has_trait_cast<T: ?Sized + 'static>(&self) -> bool
Returns whether or not a given value can be cast to a given trait object.
let is_implemented: bool = type_registration.has_trait_cast::<dyn SomeTrait>();

trait_cast<'a, T: ?Sized + 'static>(&self, val: &'a dyn Reflect) -> Option<&'a T>
Attempts to cast a given value to a given trait object.
let obj: &dyn SomeTrait = type_registration.trait_cast::<dyn SomeTrait>(&some_struct).unwrap();

Together, these methods can turn the previous code:
let reflect_some_trait = registry
    .get_type_data::<ReflectSomeTrait>(value.type_id())
    .unwrap();

  reflect_some_trait.get(&*value).unwrap()

into the following:
let reflect_some_trait = registry
    .trait_cast::<dyn SomeTrait>(&value)
    .unwrap();

This is a small improvement, but it's not where the magic of this PR happens. That takes place during type registration.
Type Registration
The biggest difficulty with handling traits using Bevy's reflection system is that each trait has to be registered per implementor— at the implementor's definition. This can be difficult to maintain as implementors need to import the generated ReflectMyTrait structs.
To get around this issue, this PR adds the register_all macro, which allows multiple traits and types to be registered at once, and let the compiler figure out which types can register which trait.
#[derive(Reflect)]
struct Foo;
#[derive(Reflect)]
struct Bar;

trait SomeTrait {}
impl SomeTrait for Foo {}

register_all! {
  traits: [SomeTrait],
  types: [Foo, Bar]
}

// Registers:
// Foo with SomeTrait trait cast
// Bar with nothing
register_types(&mut type_registry);

The register_all registers a trait cast for each trait that a given type implements. It then generates a public function called register_types which can be used to mass register all types.
Type Data
While the focus of this PR is trait casting, we can't forget about traits which cannot be cast into dyn MyTrait objects. Those traits are often represented as type data, which can be inserted into a type's registration. Since this PR added register_all for traits, it made sense to also give the ability to register type data as well.
#[derive(Reflect)]
#[reflect(SomeTrait)]
struct Foo;
#[derive(Reflect)]
struct Bar;

#[reflect_trait]
trait SomeTrait {}
impl SomeTrait for Foo {}

register_all! {
  data: [ReflectSomeTrait],
  types: [Foo, Bar]
}

// Registers:
// Foo with ReflectSomeTrait type data
// Bar with nothing
register_types(&mut type_registry);

Note: The only required field is types, so both traits and data can be included or excluded as necessary.

Benefits
Aside from just being more compact, the macro automatically allows usage of the new trait_cast methods, making it much easier to cast types to their respective traits. It also makes using managing reflected traits a lot simpler (users don't need to worry about importing/exporting the proper ReflectMyTrait structs).
Additionally, it makes it much easier to reflect third-party traits. Instead of manually creating the struct (or using a macro to do it for you), you can simply add it to the list of traits in the register_all! macro (same goes for third-party structs as well).
Considerations
One downside of this is that it ends up generating a fair amount of code (~40 lines per type per trait) as each type needs to check if it implements a given trait. This means that the total number of generated "blocks" is equal to the number of types times the number of traits.
However, it might be worth it for widely used traits or when used more granularly.

Sample Output
pub fn register_types(registry: &mut bevy_reflect::TypeRegistry) {
  {
    let type_registration = match registry.get_mut(::std::any::TypeId::of::<Foo>()) {
      Some(registration) => registration,
      None => {
        registry.register::<Foo>();
        registry.get_mut(::std::any::TypeId::of::<Foo>()).unwrap()
      }
    };
    if let Some(cast_fn) = {
      {
        trait NotTrait {
          const CAST_FN: Option<for<'a> fn(&'a Foo) -> &'a dyn SomeTrait> = None;
        }
        impl<T> NotTrait for T {}
        struct IsImplemented<T>(core::marker::PhantomData<T>);
        impl<T: SomeTrait + 'static> IsImplemented<T> {
          #[allow(dead_code)]
          const CAST_FN: Option<for<'a> fn(&'a T) -> &'a dyn SomeTrait> =
            Some(|a| a);
        }
        if IsImplemented::<Foo>::CAST_FN.is_some() {
          let f: fn(&dyn crate::Reflect) -> crate::ErasedNonNull =
            |val: &dyn crate::Reflect| {
              let cast_fn = IsImplemented::<Foo>::CAST_FN.unwrap();
              let static_val: &Foo = val.downcast_ref::<Foo>().unwrap();
              let trait_val: &dyn SomeTrait = (cast_fn)(static_val);
              crate::ErasedNonNull::new(
                trait_val,
                core::any::TypeId::of::<dyn SomeTrait>(),
              )
            };
          Some(f)
        } else {
          None
        }
      }
    } {
      {
        type_registration.register_trait_cast::<dyn SomeTrait>(cast_fn);
      }
    }
  };
  {
    let type_registration = match registry.get_mut(::std::any::TypeId::of::<Bar>()) {
      Some(registration) => registration,
      None => {
        registry.register::<Bar>();
        registry.get_mut(::std::any::TypeId::of::<Bar>()).unwrap()
      }
    };
    if let Some(cast_fn) = {
      {
        trait NotTrait {
          const CAST_FN: Option<for<'a> fn(&'a Bar) -> &'a dyn SomeTrait> = None;
        }
        impl<T> NotTrait for T {}
        struct IsImplemented<T>(core::marker::PhantomData<T>);
        impl<T: SomeTrait + 'static> IsImplemented<T> {
          #[allow(dead_code)]
          const CAST_FN: Option<for<'a> fn(&'a T) -> &'a dyn SomeTrait> =
            Some(|a| a);
        }
        if IsImplemented::<Bar>::CAST_FN.is_some() {
          let f: fn(&dyn crate::Reflect) -> crate::ErasedNonNull =
            |val: &dyn crate::Reflect| {
              let cast_fn = IsImplemented::<Bar>::CAST_FN.unwrap();
              let static_val: &Bar = val.downcast_ref::<Bar>().unwrap();
              let trait_val: &dyn SomeTrait = (cast_fn)(static_val);
              crate::ErasedNonNull::new(
                trait_val,
                core::any::TypeId::of::<dyn SomeTrait>(),
              )
            };
          Some(f)
        } else {
          None
        }
      }
    } {
      {
        type_registration.register_trait_cast::<dyn SomeTrait>(cast_fn);
      }
    }
  }
}

Example
As an example of how this works, bevy_reflect::impls now exports a registrations module:
mod registrations {
    use crate as bevy_reflect;
    use crate::erased_serde::Serialize;
    use crate::register_all;

    register_all! {
        traits: [Serialize],
        types: [
            bool,
            char,
            u8,
            u16,
            u32,
            u64,
            u128,
            usize,
            i8,
            i16,
            i32,
            i64,
            i128,
            isize,
            f32,
            f64,
            String,
            Option<String>,
        ]
    }
}

Because we register all these types with a Serialize trait cast, we can use it in our ReflectSerializer as easily as:
// `unwrap()` used for conciseness here
let registration = self.registry
  .get(self.value.type_id())
  .unwrap();
let serializable = registration
  .trait_cast::<dyn erased_serde::Serialize>(self.value)
  .unwrap();

I'm also not a fan of register_all, it's really not the direction I like to see any library move in. Why can't this be a part of the #[reflect(Trait)] syntax?

I'm also not a fan of register_all, it's really not the direction I like to see any library move in. Why can't this be a part of the #[reflect(Trait)] syntax?

Many traits are not implemented for a struct in the same crate as the struct is declared.

Finally giving this the attention it deserves. My current thoughts:
1. The trait casting improvements are welcome and great ergonomics.

2. Something like the register_all! macro is essential to reduce boilerplate, but it's syntax feels alien to me and it's hard to tell what's going on. Generating a named function that you must call definitely feels wrong. Can we do something method-like on type registeries? Can we pass in a type registry to the macro?

3. Why have all the `Serialize` reflect_value calls been removed but not the others?

Regarding 2, I suppose we could require the user to make their own function, and then they invoke the macro inside of it to expand the code.

Something like the register_all! macro is essential to reduce boilerplate, but it's syntax feels alien to me and it's hard to tell what's going on. Generating a named function that you must call definitely feels wrong. Can we do something method-like on type registeries? Can we pass in a type registry to the macro?

Yeah that's totally fair. What if it looked more like:
let mut app = App::new();
register_all! {
  app,
  traits: [SomeTrait],
  types: [Foo, Bar]
};
// ...

Where we instead pass the app in directly? Or maybe it should be the registry 🤔

Why have all the Serialize reflect_value calls been removed but not the others?

The reason is that, with the current serialization system, only ReflectRef::Value types are actually use the impl'd Serialize. Otherwise, they always use the reflected form. This is addressed in #5723.

I'm also not a fan of register_all, it's really not the direction I like to see any library move in. Why can't this be a part of the #[reflect(Trait)] syntax?

I agree. I think register_all! is great for mass registration, keeping everything in one place, and registering for foreign types. However, I think whatever we do, the #[reflect(SomeTrait)] syntax needs to be updated to support the trait casting registration (maybe as #[reflect(dyn SomeTrait)]?).

Where we instead pass the app in directly? Or maybe it should be the registry 🤔

I like this much better! I think it should be the registry though.

(maybe as #[reflect(dyn SomeTrait)]?).

This syntax makes sense to me. We'll need good docs to support it though!

Closing this out as stale. I also don't know if it's the direction we want to go.


Update home.ts

Excellent tutorial. Adding the following changes fixes the tab showing after logout.

Thanks @whoknowsjas and congratulations on your first PR!


Rate limit for OTP logins

I've rebased over master and removed the offending doc.html json patch. Can you have a quick final look?

Looks good.


[Question] Problems getting the user access token

I'm having a problems trying to get the user access token, always null even if the user accepted my aplication. In my local aplication the package works fine but on my server I just cant get this user access token. Any ideia what it could be?

The reason for this is that according to the docs, the access token is saved on session with the key 'fb_user_access_token'.
You will have to manually add this access token to the \SammyK\LaravelFacebookSdk\LaravelFacebookSdk again on another request.
I'm working on a PR about this right now, because I've experienced the same problem.

My problem was the the server block of nginx, I had to add the configuration so it could accept parameters from facebook, basically it was bad configuration.

Oops! It seems my suggestion was off target. Glad you solved it.


Package restore failed for 'Template10'.

When trying to install the Template10 Nuget package to new Blank App (Universal Windows) project I get the following error:
Package restore failed for 'Template10'.

I uploaded a new NuGet version. See if that makes a difference.

Hi Jerry, I'm afraid not - I still get the same error:
This is what's at the end of the console output:
All packages are compatible with UAP,Version=v10.0 (win10-x64-aot).
Package restore failed for 'Template10'.
Package restore failed. Rolling back package changes for 'Template10'.

So far you are the only person to report this. I have just uploaded a new ver to NuGet again. I can't be sure this will work either. But, what's going on with your project?! Any suggestions?

I've had another report that's says the package only works targeting x86 - fails if you are targeting x64

I am fixing the XAML parse error today in NuGet (took a while to track that one down) and I think the changes will also resolve this. But I will check after the update. If so, I will close this.

Please try v1.0.4 now in NuGet

It's still not working for me.
Steps I've taken:

Visual Studio 2015 - New Project -> Blank App (Universal Windows)
Nuget 3.0 - install Template 10 1.04

When installing the error list shows:
"Package restore failed. Rolling back package changes for 'Template10'.
Don't know why I'm still having this issue... perhaps I'm being stupid?!


[FLINK-9963] [table] Add a string table format factory

What is the purpose of the change
Add a string table format factory
Brief change log

add StringRowFormatFactory

Verifying this change
This change added tests and can be verified as follows:
test cases in flink-formats/flink-string
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency):  no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers:  no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented?  no

@twalthr, It seems that this pr can be more generic to SingleColumnRowFormat, so FLINK-9984 can be a concrete implementation.

@Xpray do you think you find time to update this PR?

@twalthr I'll update this PR in two days. Best.


Split out wine build image

It would be easier to split the wine build out into its own image, so we more easily rebuild the docker image without having to build wine each time.

done in 70ba7924151c0ff82e5726d093fb2ce433b8f60f


Add material table dynamically from array with [dataSource]

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular6 updated with angular material 6.2.1
Showing dynamic table properly with an array but how to connect 1st table belongs to the first pagination in the angular material.
Code:
HTML:
array of each index

    <ng-container matColumnDef="start">
         Cycle Start
         {{element.start}} {{element.no}}
    </ng-container>

    <ng-container matColumnDef="end">
         Cycle End
         {{element.end}}
    </ng-container>

<mat-paginator #paginator [pageSizeOptions]="[5, 10, 20]"></mat-paginator>

TypeScript:
displayedColumns: string[] = ['start', 'end'];
@ViewChildren(MatPaginator) paginator = new QueryList<MatPaginator>();

Below table view for 3 dynamic material table data

pagination is showing like below for 3 material table..

Please give your suggestions how to proceed pagination and sorting connetivity with the dynamic array  datasource
Thanks,
In advance

SOLVED!

Please keep GitHub issues for bug reports / feature requests. Better avenues for troubleshooting / questions are stack overflow, gitter, mailing list, etc.

@badrinath389 Could you please provide your solutions here? Thanks


Setting message priority while Publishing (not only Sending)

Thank you very much for adding this feature. As I can see SendContext has SetPriority method now, but could you also add a method to set it with Publish method?

Is it not in the publish context? It should be, it's the same inheritance chain so it should pick it up. I'll verify.

So, you can specify the priority when publishing:
bus.Publish(new Message(), x => x.SetPriority(27));

The extensions are in the RabbitMqTransport namespace, so I'll move them out so that they resolve correctly.

Thank you for clarification.

How is possible to publish higher priority messages to one queue and lower priority messages to another queue?
Currently the situation I have is following:

I have two queues, one with x-max-priority: 1 and another with x-max-priority:2
when I publish message with priority 2, it goes to queue with x-max-priority: 2
when I publish message with priority 1, it goes to both queue


[Feature] Cursor position

It would be great to have the ability to specify the cursor location once the text has been expanded. This would work really well for programming snippets, for example.
Let's say I'm adding a tag in HTML, I would want the cursor to be in between the opening and closing tags, rather than at the end.
<p>[Cursor]</p>

It's already possible. See the #{cursor} variable.

Thanks, @xmichelo! I tried looking for the documentation but didn't find the variables.


Laravel for exception and error display 500

Laravel Version: 5.4.33
PHP Version: 7.0.22
Database Driver & Version: did not use

Description:
php-fpm + nginx
When I use the composer to install laravel, I use throw new \Exception in the code or intentionally wrong grammar, the system directly reported 500 errors, after I check, I found in Illuminate\Foundation\Bootstrap\HandleExceptions set_exception_handler does not work, the handleException no work, But the laravel.log normal record, after tracking, HandleExceptions in the constructor is the normal implementation, and now my solution is in App\Exceptions\render echo output error display, and I used in other places using the set_exception_handler function Did not take effect, but I directly use php to write a single file to test the function is no problem, so I would like to know what the reasons
my .env the
APP_NAME=Laravel
APP_ENV=local
APP_KEY=base64:zBtKxMNj8k/Xbo8bfn6fiephkCvoM7S+aHEs9vjW4r0=
APP_DEBUG=true
APP_LOG_LEVEL=debug
APP_URL=http://localhost
Steps To Reproduce:
use composer to install laravel, in routes\web.php file, add throw new \Exception in the code or intentionally wrong grammar, the system directly reported 500 errors

This repo is for bug tracking. Use the forums or slack channel for solving your issue

Please ask on the forums, this repo is for bug reporting only. You can use https://laracasts.com/discuss or https://laravel.io/forum which are forums with a very large community of developers helping each other.

ok, thanks


MM-49368 - When pasting a link from github it clears the undo history

Fixes https://github.com/mattermost/mattermost-server/issues/22039
Demo video
https://www.loom.com/share/5bb0587f30964a84b3a3cd44916b7c10
Release note
Fixes undo history bug when pasting a link from GitHub

Hello @gitstart,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

@hmhealey feedback has been addressed.

@hmhealey changes have been made to AdvancedCreateComment

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

Hey there!, Thank you for your PR.
Webapp is now part of mattermost monorepo now at https://github.com/mattermost/mattermost-server/tree/master/webapp Would you be interested in opening the PR over there?

The web app now lives in the webapp/channels over in https://github.com/mattermost/mattermost-server. We decided to move it to be a monorepo to make both development and our CI pipelines easier, but it's unfortunately disruptive right now. We've updated our setup docs and added some steps for migrating to working with the monorepo.
Copying the changes over there hopefully shouldn't be too hard. There's a few changes to file paths (such as the code living in a dedicated src subdirectory), but most things should look the same as they do now.
Let us know if you could use any help with this. Hopefully the process is smooth.

@hmhealey This PR has been moved  to the monorepo https://github.com/mattermost/mattermost-server/pull/22719 cc @M-ZubairAhmed


Error when multiple versions are installed through sub dependencies

ember-resources throws following error when multiple versions of ember-modify-based-class-resource are installed through sub dependencies.
Assertion Failed: type may not overlap with an existing usable
The debugger showed that class-based trying to register twice.
This happened when 1.1.0 and 1.1.1 were both installed through sub dependencies (ember-data-resources & other).
I was able to fix it by just pinning to 1.1.1 and therefore eliminating the duplicate.
A bit vague but just wanted to leave a notice for anyone stumbling into the same problem.

Thank you! This is helpful information!


Length of string

Title of programme
Write a JavaScript programme to find the length of string.
Additional information
Input  : abcd
Output : 4

Code of Conduct

[X] I have read and agree to the Codinasion project's Code of Conduct

hey @harshraj8843 ,you can just do e.g
let name= "harsh raj";
console.log(name.length);
hope this helps

hey @harshraj8843 ,you can just do e.g
let name= "harsh raj"; console.log(name.length);
hope this helps

Hey @Godwindaniel10, could you please raise a PR for this 👍🏻


Ignore certain filetypes

Seagoat is taking a very long while to scan a repository - and by the looks of it, it is stuck on the translation files which would add little value to the task at hand. Is there a way to ignore them?
Log: https://dpaste.com/4UK6GPF8D (started at 8:40, stopped it at 10:40 with little progress done)

I think that this would be a great feature indeed, it could be probably easy to add after this issue is fixed: https://github.com/kantord/SeaGOAT/issues/180

It got me thinking, are you sure that translation files would add little value?
I guess it depends on the use case, but I think it could actually be a great way to find things as the translation text could hold way more semantic information than the code itself. The downside that I can see is that you will find the translation file, not the code file. For me as a dev that would not be a dealbreaker, since once I have the translation key, it's trivial to find where it's used. And perhaps there could be an additional feature that actually finds the code related with translation keys as well

it is stuck on the translation files which would add little value to the task at hand.

It depends on what you mean by stuck, SeaGOAT prioritizes files that have been changed recently, or that get changed often. In this case translation files are generated by an online platform? I guess maybe there could also be a feature that tries to detect if a folder is meant to store generated/downloaded code or it is code that is manually changed. Or perhaps this could be inferred based on the commit author

A related issue is that I would like SeaGOAT to ignore directories that contain intermediate products and final distributions, which are nearly always uninteresting... I thought that perhaps gitignore would be a good source for these but I agree that sometimes the gitignore files are still interesting for semantic search.  (Though ignoring the gitignore files would probably be a sensible default behavior which could be config'ed off)

Yeah, I think that it would be possible to either ignore these directories/files, that's pretty easy to implement. And it would also be easy to allow the user to configure these things.
But I am wondering if it's possible to come up with a relatively reliable heuristics to detect these scenarios.

Apart from ignoring, it's also possible to "penalize" such files, which would mean:

They would appear generally lower on result pages, even if they are considered good results otherwise. They would still show up when they are very good results, and they could be on the top of the result page when no other good results are found
They would be deprioritized for being analyzed, so the server would only start analyzing them once all other files have been analyzed

the original issue was fixed in https://github.com/kantord/SeaGOAT/pull/268
see documentation: https://kantord.github.io/SeaGOAT/latest/configuration/#server


python311Packages.wordcloud: disable failing tests

Description of changes
Fix build (https://hydra.nixos.org/build/241777043)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 270389 run on x86_64-linux 1

4 packages built:

python310Packages.wordcloud
python310Packages.wordcloud.dist
python311Packages.wordcloud
python311Packages.wordcloud.dist


Changing the host/source field

The library adds the host field by default to hostname::get()? value which then is by default used by Graylog as the source of the message – is there any way to customize it, set the value to something else?

Not currently, but it should be very easy to add it - we just need to add another setter to the builder and default to the existing behaviour.

Thanks for so quick reply!
Are you willing to do this? Or should I contribute a PR?


[5.5] Add and stop middleware methods

Hey guys
Just an idea -- I was thinking about a Route::startMiddleware('foo'). So, for instance, you could do something like this:
Route::startMiddleware('auth');

Route::startMiddleware('cors');

Route::get('cors');

Route::stopMiddleware();

Route::get('foo', 'FooController@index');

Route:stopMiddleware();

Route::get('users', 'UsersController@index');

cors would have auth and cors middlewares applied.
foo would have auth middleware applied.
users would have no middlewares.
The foo route would have the auth  middleware applied. As far as I know the only way to do thi right now is using the group method, right?
Sorry if something like this already exists — wasn't aware!

Like you said, groups allow for this already and it seems more clear on what's going on.
Route::group(['middleware' => 'auth'], function($r) {
$r->group(['middleware' => 'cors'], function($r) {
$r->get('/cors', 'CorsController@index');
});
$r->get('/auth', 'AuthController@index');
});
Router::get('/users', 'UserController@index');

Yeah, I thought it would better for a scenario I have, but right now I don't think it's needed. Gonna close this.


viewer pattern

Hello,
a common pattern in relay is viewer type, which models the current user and all graphql queries are fields of it. I am facing a difficulty as most of the creation mutation returns viewer on the payload as part of fat query. This creates a circular reference, as viewer contains all the query resolvers. Any thoughts on how can I split the resolvers in modules (which works fine for me, except the viewer pattern), and still allow mutation of entities access to viewer.
thanks,
bsr.

One sol I found was to create a new resolver by embedding the viewer at the top level.
type CreateWidgetResolver struct {
	*widget.MutationWidgetResolver
	*viewerFieldResolver
}

this is acceptable, considering fat query will be deprecated in relay 2.

closing as above sol is satisfactory. please comment if a better way.


ISPN-5232 Entry iterator returns entries created in current TX but does

not apply the filter/converter

Transaction context values now check filter/converter
Fixed issue when multiple context values were present we only chose 1

https://issues.jboss.org/browse/ISPN-5232

Looking...

Looks good. Will remove the unused stuff myself and integrate it. Thanks @wburns!


Wallet Connect SafeApp doesn't paste url in clipboard

Wallet Connect SafeApp doesn't paste url in clipboard. It is very invariable when this happens, some times it does, sometimes it doesn't.
Environment

Browser: Brave
Wallet: Safe browser
Safe: Yes
Environment: Mainnet

Steps to reproduce

Go to dapp you want to connect and select wallet connect
Copy WC url to clipboard
Go to wallet connect safe app on the safe
Paste wallet connect url

Expected result
I expect to see that the Safe is connected to the desired app.
Obtained result
Loading spinner shows (pictured below), and then the space to paste the url is blank again.
Screenshots
Loading spinner:

Console error (seems to just be GA being blocked by brave):

You can reproduce this in some apps when you reconnect your wallet, and the WalletConnect fails to generate the new session URL. It could be on the dApp side or WalletConnect side, but 99% is not a Safe App issue.


Updating E000295.yaml to adjust phone validation

Adding javascript for phone validation

nice! Generally we only put numbers in these fields, stripping other things server-side before it hits the yamls, which tends to work for just about every form.


Fix error when installing from homebrew

WHAT
Same as the title
WHY
Before this, installation would fail with an error:

Error: An exception occurred within a child process:
Errno::ENOENT: No such file or directory - hcledit

Thanks for the contribution! However, the brew formula is generated by goreleaser so we should fix it within the goreleaser config.

I think this is fixed by https://github.com/goreleaser/goreleaser/pull/2541 and since this workflow uses latest, it'll be updated next time a release is tagged in this repo 👍


updated scripture resources package version

This change is

package.json, line 17 at r2 (raw file):

    "create-coverage-badge": "bash scripts/create-badge-json.sh"
  },
  "version": "0.9.0-beta.7",

Any time we make a change in this app, we need to manually increment the build number since we are not publishing with NPM.


Cant delete completed goals

There is no way to delete a main goal. I say main as in the parent drop down that has all the goals in it. You can right click a specific goal and remove it but u cant do that to the parent ones.

uhmm, its been nearly 8 months and you still cant delete stuff?


[Improvement][Server] When the server exist in the dead server list of zk,need stop service byself

Describe the question
When the server exist in the dead server list of zk,need stop service byself
What are the current deficiencies and the benefits of improvement

A clear and concise description of the current deficiencies and the benefits of this improvement.

Which version of DolphinScheduler:
-[1.3.5]

i want to resolve this issue, assignt it to me.

i want to resolve this issue, assignt it to me.

good job,  looking forward to your PR


mlserver dockerfile COPY failed

I'm not sure if it's a bug or intended  (if it's intended I think we need to add Documentation on how to work with the mlserver dockerfile command)
I'd like to make a report that the Dockerfile can't be used as it is after executing mlserver dockerfile .
I got this error when executing docker build -t mlserver-demo -f Dockerfile .
Step 5/15 : COPY     ./environment.ym[l]     ./environment.yam[l]     ./conda.ym[l]     ./conda.yam[l]     .
COPY failed: no source files were specified

the fix that I made is by removing most of the COPY part on the Dockerfile (after the part below are deleted I can build the Docker image)
COPY \
     ./environment.ym[l] \
     ./environment.yam[l] \
     ./conda.ym[l] \
     ./conda.yam[l] \
     .
COPY \
     --chown=1000 \
     --from=env-builder \
     /envs/base.tar.g[z] \
     ./envs/base.tar.gz
COPY \
     ./settings.jso[n] \
     ./model-settings.jso[n] \
     ./requirements.tx[t] \
     .

here my directory structure:
.
├── Dockerfile
├── Makefile
├── models
│   └── text-clf.joblib
├── model-settings.json
└── requirements.txt

it's a simple sklearn pipeline model in the models directory
{
    "name": "text-clf",
    "implementation": "mlserver_sklearn.SKLearnModel",
    "parameters": {
        "uri": "./models/text-clf.joblib",
        "version": "v0.1.0"
    }
}

Hey @MuhammadAgf ,
The Dockerfile used by MLServer requires buildkit to be enabled (i.e. the new Docker backend). There're several ways to enable it, one of them is by setting the DOCKER_BUILDKIT=1 env var before calling docker build, e.g.
DOCKER_BUILDKIT=1 docker build ....

We'll keep this one open to add a note to the docs about buildkit.


Quotation marks: add on option to replace them by \enquote{} on export

Some of my references it Zotero have in their titles (or in other fields), some quotations marks. They are correctly exported by BBT but, in the final output produced by LaTeX, that can cause certain issues. Straight quotes are not parsed correctly as being the opening or the closing quote by LaTeX… (the reason, I read, is that " is right quotation mark in some encoding). Nested quotes can't be managed correctly, punctuation (with French guillemets) is deficient (nbsp needed).
Personally I use the csquotes package and its \enquote{} command (which switches automatically from one type of quotes to another if they're nested and handles French punctuation, with babel, perfectly.)
Would it be possible to add, perhaps as an option, the replacement of quotes by \enquote{} (or any other command defined by the user)? I.e. check if a word is wrapped by some quotation marks and replace them.
Quotations marks I use are: « » (guillemets) and " " (straight), and, rarely, “ ”. Other languages use:  ‘ ’ and „ “. So the list of quotation marks might have to be defined by the user too.
(In French, guillemets «» comes with a non-breaking space after the opening and before the closing mark. As I wrote above, this is handled by csquotes and babel, so the space must be removed on export if you decide to implement my suggestion).
A .json and a .bib to test (but with the actual behaviour of the exporter).

The guillemets and the fancy quotes would be the easy case as I can recognize them independent of context, but the straight quotes are tricky. Straight quotes are also sometimes used to denote inches for example.

Indeed, but I think it's rare in titles. Is it possible to rely on the context (figures), and the fact that they're not paired? Geography: « 49° 30′ 00″ »
https://en.wikipedia.org/wiki/Prime_(symbol)

You wouldn't believe the amount of variety present in bibliographies :/. In
a 3-quote situation there'd be no way of deciding short of some fairly
spectacular AI. How about if I just do the guillemets and fancy quotes?
Then you could force csquotes by replacing straight quotes.

Could you give http://tempsend.com/6D23630EAA a try? It will replace single and double guillemets (including a single breaking or non-breaking space if present) and left and right double quotes if you set the preference to on (defaults to off). It does not replace straight quotes or single left quotes, as the single right quote is simply the apostrophe -- too common a character to mess with.

Great, I'll change the remaining straight quotes in my library. Thank you!

This hasn't been merged into the main branch yet, I'm putting together a
test case in the morning, then I'll release it when all pass. Does it do
what you need?

Yes,that does the trick. As for the test, you can use the data in the file above.

Which file is that?

A .json and a .bib to test (but with the actual behaviour of the exporter).

(updated with the \enquote{} markup and to fit the test guidelines -- timestamp, etc…)

Thanks, I'd overlooked those, that helps tremendously. It just struck me though -- I can replace the guillemots safely, but in the case of the fancy quotes, “ can be either open or close quotes (see your sample above). That makes it once again tricky to just assume they can be replaced safely without regards to context.

In my sample above, Searle's article is enclosed by straight quotes. Is that your remark ?

Ah no sorry, I meant “ ”. vs „ “.

I see.
I think that you can rely on the fact that an opening quote has a space before, and the closing quote has its space after. Except the guillemet, but it's more straightforward with them (though there might be some languages which use them in the opposite way).
Or, that could be user-defined (hidden pref is ok): code «»,“” would mean that there are two pairs to take into account, and the first defined is the opening quote.

It won't always have spaces around them; a comma can for example follow a closing quote. Human language is frustratingly flexible. The hidden pref is what I've gone with; a simple string of character pairs, no separating comma. Also no enabling in the GUI anymore; if the csquotes pref is non-empty, it is applied. Tests are running, if they go green I'll release.

Travis is unresponsive at the moment, so it will probably take a few tries to get a release out. I'll try again tonight.

So, I can have as extensions.zotero.translators.better-bibtex.csquotes the value «»“” ?

Yep.

I don't know if it's been released yet, Travis has been acting up.

Shouldn't such quotes translate to \mkbibquote{...} rather than \enquote{...}?

Just tried this, setting extensions.zotero.translators.better-bibtex.csquotes to the value "". The .bib file now replaces each quotation with Enquote$>$ and Enquote$<$ instead of the proper \enquote{...}. Using BBT 5.0.180 and Zotero 5.0.54.

csquotes only works with two distinct quote characters.

Try with left/right double quotation marks (https://www.cl.cam.ac.uk/~mgk25/ucs/quotes.html).

Makes sense, thanks!


Add cleanup function that turns off channel event listeners when component is unmounted

This is a tiny update mostly aiming to improve the ✨ developer experience ✨
This warning shows up on the new schedules page:

We don't want a memory leak! useEffect()'s first argument can be used to return a function that will fire when the React component is unmounted, so we use that here to turn off the event listeners that are activated by the setupChannels call in the <Map> component. At least, I think that's how channels here work. :)

@phildarnowsky Ready for your review.


Documentation suggestion: more specificity, less fog

Hi Mark!  I'm struggling to get this working.  It appears that pkger is right for my application, but I don't know yet, and it's taking me awhile to find out.
Which leads me to make a suggestion.
This section of the readme is very confusing.  I'm including this as a screen shot because we can't reference blocks in .md files like source code.

Here the section begins with a negative, referring perhaps to an alternate use case?  The section continues with "Alternatively..." which I am guessing is the main use case?
The first suggestion is make this two sections:

To include referenced  files at package time, do this...
To include unreferenced files at package time, do this..

The second suggestion is explain why is the pkger list command used here?  Ostensibly the user is expecting to add a file to the binary here, given the section title.   Maybe also explain why two -include arguments are used?  Is this not re-entrant?  Would not two successive pkger -include statements work?   And why would we want an -include to reach out to github for a static file in the very first example in the readme?  This example seems highly convoluted, even fabricated.  It just doesn't make sense, given its very poor introductory text.
All I'm trying to do is, include a static json file in my binary, to be marshalled into a type.  I imagine this would be a very common and simple use case.  But these documented examples seem to mostly get in the way.
I'm sure I'll figure this out.  I just want to share this unvarnished first impression of trying to do something that should be slam-dunk easy.

I was going to raise a similar issue.
What I was expecting from the readme was a step by step intro of:

Explain ideal structure for a go program with a static file of directory to pkg
How to tell pkger about the static file or directory
How to access the files using pkger

The initial section ends up pointing to API docs with no explanation or code sample and the alternative command line I couldn't get to work.


Last Week in Pony - February 10, 2018

The recording of the developer sync from February 5, 2019 is available here:
https://pony.groups.io/g/dev/files/Pony Sync/2019_02_05/pony_sync_february_05_2019.m4a

The exuberant ctags Pony module that John Mumm created has a new home under the Pony organization. ctags support is awesome if you are using Vim or Emacs with Pony.
https://github.com/ponylang/pony-ctags

GitHub user @kastiglione submitted his first PR against the Pony website. Improvements to the LLDB cheatsheet. Thanks @kastiglione.

The implementation of RFC 61: Add modulo and floored division operators has been merged to master and will be available in the next release.

The Reasonable Coding Twitch stream will be looking at Pony on February 11th. Check out https://www.twitch.tv/ostera/events for more information.  If you are in the US, you'll have to get up very early to watch live, but, folks in Europe should be good with the time.

Pony stable, our little dependency manager that could, just had version 0.2.0 released: https://www.ponylang.io/blog/2019/02/pony-stable-0.2.0-released/


[FORSETI] Cloud SQL scanner functionality

Related to Forseti Security Issue #3624
Scanner
Cloud SQL Scanner
Description: Cloud SQL instances can be configured to grant external networks access. The Cloud SQL scanner supports a blacklist mode, to ensure unauthorized users don’t gain access to your Cloud SQL instances.
Python Violations Summary
Generate a violation if any of the following fields do not match:

Instance_name
Authorized_networks
Require_ssl

1:1 Rego | Python Parity Status
It looks like the Rego rule does check for authorized_networks and require_ssl, but it does not check for the instance_name.
Rego work to do
Create or edit existing Rego rules to do the following:

https://github.com/forseti-security/policy-library/blob/master/policies/templates/gcp_sql_allowed_authorized_networks_v1.yaml

https://github.com/forseti-security/policy-library/blob/master/policies/templates/gcp_sql_ssl_v1.yaml

Allow to check for instance name parameter --> currently only looks at authorized_networks and require_ssl.

Does this policy cover the check for requiring SSL?

Looks like it- thanks Gregg!

For instance name checking, I think this should cover it: https://github.com/forseti-security/policy-library/blob/master/policies/templates/gcp_enforce_naming_v1.yaml

The Forseti scanner allows you to target specific instances to check if SSL is disabled or has a public IP. The Rego template that exists follows best practices to check all instances.


pull request for e2e test at 12/07/2016 22:15:01

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ragged.boolean_mask() gives a different result when executed in a model

mask = tf.ones((1, 100, 300), dtype=bool)

data = tf.ragged.boolean_mask(
    data=tf.ones((1,100,300,12)),
    mask=mask
).to_tensor()

print(data)

This gives me tf.Tensor(......, shape=(1, 100, 300, 12), dtype=float32) when executed normally, but when it is executed in a model it gives me Tensor("model/attention/RaggedToTensor/RaggedTensorToTensor:0", shape=(None, None, None, 12), dtype=float32).
Why is this? How do I get it to return the correctly masked tensor in a model?

@surGeonGG
We see that you have not filled the issue template, please fill the issue template which provides us with the tf version, stand alone code the replicate the issue or is possible share a colab gist with the error faced.

Replicated here
https://colab.research.google.com/drive/1HaOtA209X062l7OHn3JPtI2O_uGEcc8O?usp=sharing

@surGeonGG
if Dense is first layer then you have to set shape/size of input data. You can see it even in example in Dense documentation.
Please refer to below links for reference:
#35464 #33422 #23748 link

I'm confused. Dense is not the first layer. The first layer is an lstm. The shape information becomes (none, none, none) after I call...
 mask = tf.ones((1, 200, 12), dtype=bool)     x = tf.ragged.boolean_mask(         data=x,         mask=mask     ).to_tensor()

Figured it out! I simply use x = tf.ensure_shape(x, (*shape here*)) afterwards to make sure the shape is correct.
Thanks for the help.


fix(multi-account): account switch on wagmi

Description
Our Wagmi client have a connection status mismatching issue while switching between accounts and re-calling the syncAccount method. The status value is getting undefined, thus, the new account is not properly set.
This PR introduces changes to fix this along with some function props for multi-chain functionality. The multi-chain prop update is not fixing something, it just makes sure that we are okay when we switch to multi-chain architecture.
Type of change

[ ] Chore (non-breaking change that addresses non-functional tasks, maintenance, or code quality improvements)
[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)

Associated Issues
For Linear issues: Closes APKT-xxx
For GH issues: closes #...
Showcase (Optional)
If there is a UI change include the screenshots with before and after state.
If new feature is being introduced, include the link to demo recording.
Checklist

[ ] Code in this PR is covered by automated tests (Unit tests, E2E tests)
[ ] My changes generate no new warnings
[ ] I have reviewed my own code
[ ] I have filled out all required sections

Account switching is failing to function correctly in both my local @web3modal/wagmi 5.1.6 setup and the web3modal-laboratory online testing platform.


Feature/cu mfce60 gamepad control

Gamepad control established
Centered the way all inputs are used inside Player.h defining InputActions:
MOVEMENT,  ORIENTATION,  SWITCH, SHOOT, ABILITY_1, ABILITY_2, ABILITY_3
Gamepad control will be used if the bool inside PlayerController.h named useGamepad is true and a Gamepad is connected
useGamePad bool inside PlayerController can be set through a public method SetUseGamepad()
This method should be called from some UI button
Orientation is slightly more complicated, established a way for gamepad and mouse behaviors to coexist

Task linked: CU-mfce60 Player control with gamepad


ECRBackend delete_repository ignoring force argument

Bug Description
Recently in issue #4128 has been made a improvement to the delete_repository method in the ECRBackend class (moto==2.2.1 >=), but when we try to delete a repository with images inside, the force execution flow has not been implemented, which has broken our test/CI pipelines this week raising:
botocore.errorfactory.RepositoryNotEmptyException: An error occurred (RepositoryNotEmptyException) when calling the DeleteRepository operation: The repository with name 'abc-job-dev' in registry with id '123456789012' cannot be deleted because it still contains images

When we run this mocked code:
ecr_client = boto3.client("ecr")
ecr_client.delete_repository(
   repositoryName=job.image,
   force=True,  # remove all repository images
)

Basically, the temporary solution has been to hardcode moto to 2.2.0.

Thanks for raising this @victor-mariano-leite - will mark it as a bug.

@bblommers you can assign me, will create a PR today


Normalize with timezone

What type of PR is this?

What does this PR do / why is it needed ?

Which issue(s) this PR fixes:

Fixes #
Other notes for reviewers:
Does this PR introduce a user-facing change?

Can we add tests for uses with and without the flag? You should  be able to add tests to TestResultNormalizer to test both flows

Change not needed - timezone was not the issue in date formatting


DRAFT: fix(buffer): compile time

Fixes #88

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.38%. Comparing base (a22b04d) to head (c8a94d7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   99.38%   99.38%
=======================================
  Files          58       58
  Lines        6044     6048    +4
=======================================
+ Hits         6007     6011    +4
  Misses         37       37

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[MRG] Add save method for EpochsTFR.

Closes https://github.com/mne-tools/mne-python/issues/3810

Codecov Report

Merging #4182 into master will increase coverage by <.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master    #4182      +/-   ##
==========================================
+ Coverage   86.17%   86.18%   +<.01%
==========================================
  Files         356      356
  Lines       64057    64070      +13
  Branches     9748     9751       +3
==========================================
+ Hits        55202    55217      +15
+ Misses       6161     6159       -2
  Partials     2694     2694

Continue to review full report at Codecov.

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

LGTM
don't forget to update what's new

Thanks @jaeilepp !

+1 for merge from me, as well (just please update whats_new as @agramfort suggests)

just please update whats_new as @agramfort suggests

It's been done already. See if it's good enough.

thx @jaeilepp


Simple tutorial does not work

https://github.com/kube-logging/kube-logging.github.io/blob/master/content/docs/quickstarts/single.md
FluentbitAgents are not processed, No fluentd are deployed.
Setting logging.enabled: true fixes it

In the example we suggest to create the resources manually, not through logging.enabled: true

Mmpf, I my previous tests the fluentbitagent resources were not being reconciled. I tried again in a kind clusters, and it worked. I must have missed something. Sorry for that, closing.


Shop

interface for purchasing buildings or producing more humans
each component would have an image, a title, description, price, and something that shows the rate of return

Done


Deletes special character if cursor is next to one with ciw

Sorry if this is a duplicate issue, I tried searching for it and couldn't find anything. This issue has to do with the command ciw in normal mode, aka change in word. If your cursor is next to a special character, such as (, ", or ? (for example), doing ciw before that character will also delete said special character, which it should not. Ex:
doing ciw in the following string, when your cursor is on the t
some_object.present?

results in the following
some_object.

however, it should result in
some_object.?

with the cursor between the . and the ?

This is because of an issue in Atom: https://github.com/atom/atom/issues/9851

As stated in the README, this package is no longer maintained and is deprecated. We recommend that people use the vim-mode-plus package instead. Because of this, we are archiving this repository and closing all issues and pull requests. Thanks very much for your support and contributions!