Cannot read property 'start' of undefined (related to<EMAIL_ADDRESS>update)

Here is my file which can't even be "started" since recent a babel update (6.5.1?)
You will not need anything but my file itself to reproduce (I guess the parsing fail somewhere)...
Looks like it's the same kind of issue that just popup here https://github.com/istarkov/babel-plugin-webpack-loaders/issues/15
import test from "ava"; import "babel-core/register"

import feed from "../feed"

test("statinamic/lib/md-collection-loader feed helper", (t) => {

  t.is(
    feed({
      feedOptions: {
        title: "test",
        site_url: "http://statinamic.test/",
      },
      destination: "feed.xml",
      collection: [
        {
          title: "One",
          date: "2015-01-01",
          description: "One",
          __filename: "page/one.md",
          __url: "/page/one/",
        },
        {
          title: "Two",
          date: "2015-12-31",
          description: "Two",
          __filename: "page/two.md",
          __url: "/page/two/",
        },
      ],

      // for testing
      xmlOptions: { indent: "  " },
    }),

    /* eslint-disable max-len */
    `<?xml version="1.0" encoding="UTF-8"?><rss xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">
  <channel>
    <title><![CDATA[test]]></title>
    <description><![CDATA[test]]></description>
    <link>http://statinamic.test/</link>
    <generator>RSS for Node</generator>
    <lastBuildDate>${ (new Date()).toUTCString() }</lastBuildDate>
    <atom:link href="http://statinamic.test/feed.xml" rel="self" type="application/rss+xml"/>
    <item>
      <title><![CDATA[One]]></title>
      <description><![CDATA[One]]></description>
      <link>http://statinamic.test/page/one/</link>
      <guid isPermaLink="true">http://statinamic.test/page/one/</guid>
      <pubDate>Thu, 01 Jan 2015 00:00:00 GMT</pubDate>
    </item>
    <item>
      <title><![CDATA[Two]]></title>
      <description><![CDATA[Two]]></description>
      <link>http://statinamic.test/page/two/</link>
      <guid isPermaLink="true">http://statinamic.test/page/two/</guid>
      <pubDate>Thu, 31 Dec 2015 00:00:00 GMT</pubDate>
    </item>
  </channel>
</rss>`,
    "should generate a feed from a collection"
  )
})

Here a minimal test case
~/Sync/Development
❯ mkd ava-fail

~/Sync/Development/ava-fail
❯ node --version
v5.5.0

~/Sync/Development/ava-fail
❯ npm --version
3.7.1

~/Sync/Development/ava-fail
❯ npm i<EMAIL_ADDRESS>
~/Sync/Development/ava-fail 3m 43s
❯ pbpaste > fail.js

~/Sync/Development/ava-fail
❯ ava fail.js

TypeError: /Users/MoOx/Sync/Development/ava-fail/fail.js: Cannot read property 'start' of undefined
    at /Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/whitespace.js:29:19
    at Whitespace._findToken (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/whitespace.js:99:30)
    at Whitespace.getNewlinesBefore (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/whitespace.js:28:22)
    at CodeGenerator.printComment (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:321:34)
    at CodeGenerator.printComments (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:377:12)
    at CodeGenerator.printLeadingComments (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:231:10)
    at CodeGenerator.print (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:81:10)
    at CodeGenerator.printJoin (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:193:12)
    at CodeGenerator.printList (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:258:17)
    at CodeGenerator.ObjectExpression (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/generators/types.js:48:10)
    at CodeGenerator._print (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:155:17)
    at CodeGenerator.print (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:91:10)
    at CodeGenerator.printJoin (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:193:12)
    at CodeGenerator.printList (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:258:17)
    at CodeGenerator.CallExpression (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/generators/expressions.js:142:8)
    at CodeGenerator._print (/Users/MoOx/Sync/Development/ava-fail/node_modules/babel-generator/lib/printer.js:155:17)
From previous event:
    at Api.run (/Users/MoOx/Sync/Development/ava-fail/node_modules/ava/api.js:148:4)
    at Object.<anonymous> (/Users/MoOx/Sync/Development/ava-fail/node_modules/ava/cli.js:115:5)
    at Module._compile (module.js:413:34)
    at Object.Module._extensions..js (module.js:422:10)
    at Module.load (module.js:357:32)
    at Function.Module._load (module.js:314:12)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:139:18)
    at node.js:999:3

Workaround
import test from "ava"; import "babel-core/register"

import feed from "../feed"

test("statinamic/lib/md-collection-loader feed helper", (t) => {

  const feedXML = feed({
    feedOptions: {
      title: "test",
      site_url: "http://statinamic.test/",
    },
    destination: "feed.xml",
    collection: [
      {
        title: "One",
        date: "2015-01-01",
        description: "One",
        __filename: "page/one.md",
        __url: "/page/one/",
      },
      {
        title: "Two",
        date: "2015-12-31",
        description: "Two",
        __filename: "page/two.md",
        __url: "/page/two/",
      },
    ],

    // for testing
    xmlOptions: { indent: "  " },
  })

  /* eslint-disable max-len */
  const feedXMLexpected = `<?xml version="1.0" encoding="UTF-8"?><rss xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:content="http://purl.org/rss/1.0/modules/content/" xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">
  <channel>
    <title><![CDATA[test]]></title>
    <description><![CDATA[test]]></description>
    <link>http://statinamic.test/</link>
    <generator>RSS for Node</generator>
    <lastBuildDate>${ (new Date()).toUTCString() }</lastBuildDate>
    <atom:link href="http://statinamic.test/feed.xml" rel="self" type="application/rss+xml"/>
    <item>
      <title><![CDATA[One]]></title>
      <description><![CDATA[One]]></description>
      <link>http://statinamic.test/page/one/</link>
      <guid isPermaLink="true">http://statinamic.test/page/one/</guid>
      <pubDate>Thu, 01 Jan 2015 00:00:00 GMT</pubDate>
    </item>
    <item>
      <title><![CDATA[Two]]></title>
      <description><![CDATA[Two]]></description>
      <link>http://statinamic.test/page/two/</link>
      <guid isPermaLink="true">http://statinamic.test/page/two/</guid>
      <pubDate>Thu, 31 Dec 2015 00:00:00 GMT</pubDate>
    </item>
  </channel>
</rss>`

  t.is(
    feedXML,
    feedXMLexpected,
    "should generate a feed from a collection"
  )
})

FYI, I am facing this issue on multiple (all?) projects (eg: https://travis-ci.org/postcss/postcss-import/jobs/108972544#L631) poke @hzoo

Looking..
Also a simplier snippet from what you had that fails
import test from "ava";

test("a", t => {
  t.is(
    {
      // for testing
    },
    1
  );
});

If the comment is removed that error is gone.

@hzoo I came to the same conclusion
"fix" for postcss-import: https://github.com/postcss/postcss-import/pull/172

And then fixing that by reverting that PR only lead me to find an acorn error?
hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:945
  throw err;
  ^

SyntaxError: Unexpected token (1:17)
    at Parser.pp.raise (/Users/hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:943:13)
    at Parser.pp.unexpected (/Users/hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:1503:8)
    at Parser.pp.parseIdent (/Users/hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:670:10)
    at Parser.parseIdent (/Users/hzhu/dev/test/node_modules/acorn-es7-plugin/acorn-es7-plugin.js:93:19)
    at Parser.pp.parsePropertyName (/Users/hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:554:127)

Not sure about this

hzhu/dev/test/node_modules/power-assert-formatter/node_modules/acorn/dist/acorn.js:945

@twada Any chance power-assert has something to do with this issue?

@hzoo acorn.input becomes t.is({ // comment }, 1) in your case. Forcing a newline in concise mode at https://github.com/babel/babel/blob/master/packages/babel-generator/src/printer.js#L300 should help.

Related? sindresorhus/xo#79

@sindresorhus Understood. I'll fix babel-plugin-espower ASAP.
Thanks @hzoo @gzzhanghao !

Just released babel-plugin-espower 2.1.2 to fix the problem. Thank you for reporting and suggestions.

@MoOx would you reinstall ava and try again?

@BarryThePenguin Yup.
@twada Thanks for the quick response. That did it for me.

Thanks guys! This is now ok.


Updated documentation for DenseDesignMatrixPyTables.

Content looks good to me, there's one Travis error to fix.

Great, thanks!


fixed yestheoryuploaded ping

If webhooks count as bots this should not return an error and we should no longer need to use hasRole.
Used the test YesBot by forcing it to send a message in #yestheory-uploaded which pinged the group.

This actually might not work I checked the discord.js docs and webhooks are not logged in as bots? So message.author.bot might always return false, I'm not sure of this I gotta read the docs a bit better.
if (message.webhookID) Will have to replace it with this I think :')

Both webhookID or message.author.bot work but the first one will only return true if the message is from a webhook that serves our goal better.

webhookID was a really nice catch!

webhookID was a really nice catch!

I used my big brain for once :D


another storage to help synchronizing with external storage

I use specs with nphysics (actually a fork on my own but 0.8 nphysics will impl things..) I have to synchronize RigidBodyHandle in specs world and in nphysics world.
My needs are:

for insert everything is sync: the method to create RigidBodyHandle component takes nphysics world and create them altogether
for remove I accept that some object are still alive in nphysics world but have no component in specs. and then I want to regularly sync things by removing those objects that have been deleted from specs world. But to do so I have issues:

I tried to implement using flagged storage in current master but I can't get the deleted component I just have the index of the entity deleted and that is not enough:
extern crate specs;

use specs::prelude::*;

#[derive(Debug)]
struct Vel(usize);

impl Component for Vel {
    type Storage = FlaggedStorage<Self, VecStorage<Self>>;
}

fn main() {
    let mut vels = vec![];
    let mut world = World::new();
    world.register::<Vel>();
    let mut removed_id = world.write::<Vel>().track_removed();

    vels.push(0.0);
    let e0 = world.create_entity().with(Vel(0)).build();

    {
        // remove an entity
        println!("entity removed:  {:?}", e0);
        world.delete_entity(e0).unwrap();

        // add an entity
        vels.push(0.3);
        let e1 = world.create_entity().with(Vel(1)).build();
        println!("entity created:  {:?}", e1);

        // try to find removed component
        let mut removed = BitSet::new();
        world.write::<Vel>().populate_removed(&mut removed_id, &mut removed);

        let entities = world.entities();
        for (e, _, v) in (&*entities, &removed, &world.read::<Vel>()).join() {
            // this does show we found new component instead
            println!("component of removed ? {:?}: {:?}", e, v);
        }
    }
}

The solution I'll implement is to create a new storage that keeps deleted component in some cache that I'll clear regularly (with removing their nphysic body).
This implementation is straightforward but might be needed by others so that is why I open this issue and also to know if you have better solution.

There's no too obvious solution as far as I can tell with my limited knowledge of nphysics.
I think I need some more information to give a good recommendation. I'll look into it later.

I don't think it is  related to nphysics, in my example I use an external storage of velocities.
The solution I found is to have a new storage like FlaggedStorage but without tracker field and with cache: Vec<C> field with remove method inserting a clone of component into cache (this requires that component is clonable) and a method to get access to cache.

Well I think your solution is a pretty solid one, don't see any issues with it.

Yes I actually opened the issue because synchronizing with external storage can be a standard pattern. But this kind of RetainedStorage may be specific to nphysics or physic engine in general that allow incoference such as present in physic world but absent in specs world until next manual maintain.
then I don't know about this pattern in general, maybe it should be addressed specifcally.
This issue was more about showing a solution to a specific instance of this pattern. Feel free to close it

As inserting to cache happens only when the components are removed, it should be possible to not do any clones, right? Or am I missing something.

The API requires you to return the removed component.

Ah that would prevent it then. We could probably change remove to return associated type which storages could use to change the behaviour, but that would be overkill.

Yeah, that would complicate generic code a lot.

usually when you work with an external storage the spec component is just a handle, cloning it is very usually very cheap.

Adding a cache type system is certainly possible, but connecting that to the logic is much harder to do generically. The basic problem is that you can't update the cache in separation, you need to update the cache in lock step with the sync logic, otherwise you're back in the same rabbit hole.

What about adding a way to send data along with the TrackedStorage events? This could be a clone of the component, some kind of unique id or just ().
Rough sketch: add a trait
trait TrackData<C: Component> {
    fn new(component: &C) -> Self;
}

impl<C: Component> TrackData<C> for () {
   fn new(component: &C) -> Self { () }
}

Then add a type parameter to FlaggedStorage defaulting to () that defaults to ().

Another issue I've run into with synchronising with external storages is that if you are reading both the insert and remove event channels then you cannot tell which order the events came in. It is possible to get multiple InsertFlags and RemovedFlags for the same component, which makes it awkward to work out whether the component ended up being added or removed.

Process remove first, then insert. If you read every frame it's safe.

What do you mean by 'safe'?

What version of specs are you testing? My test code is below, and it prints several add and removed events.
extern crate specs;

use specs::prelude::*;

pub struct Comp;
impl Component for Comp {
    type Storage = FlaggedStorage<Self>;
}

pub struct CompSystem {
    inserted_id: ReaderId<InsertedFlag>,
    removed_id: ReaderId<RemovedFlag>,
}

impl<'a> System<'a> for CompSystem {
    type SystemData = WriteStorage<'a, Comp>;

    fn run(&mut self, comps: Self::SystemData) {
        for event in comps.removed().read(&mut self.removed_id) {
            println!("removed: {:?}", event);
        }

        for event in comps.inserted().read(&mut self.inserted_id) {
            println!("inserted: {:?}", *event);
        }
    }
}

pub struct AddRemoveSystem(Entity);

impl<'a> System<'a> for AddRemoveSystem {
    type SystemData = WriteStorage<'a, Comp>;

    fn run(&mut self, mut comps: Self::SystemData) {
//        comps.insert(self.0, Comp);
        comps.remove(self.0);
        comps.insert(self.0, Comp);
        comps.remove(self.0);
        comps.insert(self.0, Comp);
        comps.remove(self.0);
    }
}

fn main() {
    let mut world = World::new();
    world.register::<Comp>();

    let mut ar_sys = AddRemoveSystem(world
        .create_entity()
        .with(Comp)
        .build()
    );

    let mut comp_sys = {
        let mut comps = world.write_storage::<Comp>();
        CompSystem {
            removed_id: comps.track_removed(),
            inserted_id: comps.track_inserted(),
        }
    };

    ar_sys.run_now(&mut world.res);
    comp_sys.run_now(&mut world.res);
}

0.11, but using the storage.populate_{insert/removed/modified} like in the examples, I think that might be your issue.

Are you using a bitset with populate_{insert/removed/modified}? That would explain why you get only one event for each index

Yep

Can I take a look at your code? When I insert, then remove I still get both events.

It's not a simple test, so it's not tiny.
https://pastebin.com/aDctj9aQ
The relevant code is in the impl<'a, S> System<'a> for ZSortSystem<ZComponent<S>>, and there is a test in there I'm using to check it. Run the tests with cargo test -- --nocapture or the printlns in the test won't show up.

I think the reason you get that behaviour is you join over the entities resource when you go through insertions, so you don't see insert events for entities that no longer exist.

I've published a crate specs-mirror to hopefully help with this use-case. I've designed it with my own use case in mind but it should work with something like nphysics with a bit of tweaking.

I think the reason you get that behaviour is you join over the entities resource when you go through insertions, so you don't see insert events for entities that no longer exist.

Taking out entities still gives the same result for me.
I gave it a try with your code, replacing the CompSystem impl with:
impl<'a> System<'a> for CompSystem {
    type SystemData = WriteStorage<'a, Comp>;

    fn run(&mut self, comps: Self::SystemData) {
        let mut removed = BitSet::new();
        let mut inserted = BitSet::new();

        comps.populate_removed(&mut self.removed_id, &mut removed);
        comps.populate_inserted(&mut self.inserted_id, &mut inserted);

        for index in &removed {
            println!("Removed: {}", index);
        }

        for index in &inserted {
            println!("Inserted: {}", index);
        }
    }
}

Outputs:
Removed: 0
Inserted: 0

So that seems to be working as expected for me, I'm not sure why it's not working for you :(

Is there anything that needs to be fixed?

I skimmed through this issue and there seems to be some confusion about the FlaggedStorage. Events get pushed once they come in, so there's definitely no way for them to disappear.
Huh, at the moment I'm myself unsure how you need to process them..

I've added a PR that is going to demonstrate how you keep track of the inserted components. Right now I don't know how it was done, and I couldn't find any examples in Amethyst. I hope this isn't some oversight.

The proposed storage is implemented in https://crates.io/crates/specs-mirror, closing.


Add a solution to problem 1590

The idea is to work with numbers modulo p instead of the numbers themselves. First, we compute the prefix sums modulo p, enabling O(1) computation of the sum modulo p for any subarray. Let target be the sum modulo p of the subarray to be removed, which equals the sum modulo p of the entire array. If target is 0, the array is already divisible by p, and we return 0. Otherwise, we try all possible end indices for the removed subarray. For each end index, we find the latest prefix index that removing the prefix array results in a remaining array with a sum modulo p equal to target. We track the resulting length for each end index and return the minimum length, or -1 if no such subarray is found.

Little supplement showing that the complement is unique:
(s - v + p) % p = target
=> s - v1 = target when s >= v or s - v2 + p = target when s < v
=> v1 = s - target and v2 = s - target + p
=> v2 = v1 + p
=> v2 % p = v1 % p


Add the ability to access legacy dats over the network (including dat-dns)

Currently the dat converter can only access local dats. It should be possible to hit the network as well.

🙏 👍

Okay, dat:// can now read from the network. Next up, rendering the page.


Improve msg when attempting to use result of void

Motivation
Closes #6787
Test plan
I think the existing tests already cover this. TBD.

Hey @froydnj,
Huh I just noticed that. Do you guys aim to steer clear of method_missing throughout the whole codebase?

Huh I just noticed that. Do you guys aim to steer clear of method_missing throughout the whole codebase?

We do, both in Sorbet's codebase and Stripe's codebase generally.  We generally think it introduces too much magic runtime behavior and can be better modeled with statically analyzable functions and/or a different design.

I agree with that motivation. I have some battle scars from particularly bad method_missing implementations (sub-classing BasicObject and accidentally implementing #hash to return an xml node like <hash/>? pandemonium.)
I don't think there's any alternative approach here. Kind of by definition, we want to be hooking onto any method call that can be made, and #method_missing is exact API for doing just that. Personally, I find this to be an acceptable exception-to-the-rule. If you disagree, shall I close this PR and related issue?


Minor refactor the column type interface

I think the column type desired a dedicated type instead of an expression.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

62 of 83 (74.7%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.09%) to 60.243%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

parser/ast.go
27
48
56.25%

Files with Coverage Reduction
New Missed Lines
%

parser/ast.go
4
56.3%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.09%

Covered Lines:
6334

Relevant Lines:
10514

💛  - Coveralls


Add Sentry issue tracking

There's a full description of the changes in this PR here.

Closes #14.

Looking good, but Raven in Rails isn't set up for backend events. Can you make sure you add the proper initializer for Sentry on the backend?

Raven is already be set up by adding the gem and setting the ENV variable: https://docs.getsentry.com/hosted/clients/ruby/#configuration.
It's already collecting backend errors locally:

Hmm, can you make this more explicit by adding an entry for SENTRY_DSN in
secrets.yml and configuring it in the initializer?
On Tue, Aug 16, 2016 at 9:51 AM, Max Wofford<EMAIL_ADDRESS>wrote:

Raven is already be set up by adding the gem and setting the ENV variable:
https://docs.getsentry.com/hosted/clients/ruby/#configuration.
It's already collecting backend errors locally:
[image: screenshot_2016-08-16_09 50 57]
https://cloud.githubusercontent.com/assets/5891442/17707819/fea160ea-6396-11e6-8710-4e6bb74e02dc.png
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/hackclub/lecture-hall/pull/33#issuecomment-240164591,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8j-AuzWwwXF7wkaZ7IX4f4lAds9zaIks5qgeqrgaJpZM4JksSI
.

--
Zach Latta
https://zachlatta.com http://zachlatta.com

@zachlatta Will do

Thanks for merging @zachlatta


Missing file when attempting to build package

Hello,
When I used makepkg, an error occured in build() when executing 'publish' - the following file couldn't be found:
 /home/me/cavalier/src/cavalier/NickvisionCavalier.Shared/org.nickvision.cavalier.in
Using a file manager, I can confirm it wasn't directly there. However, I found the required file within NickvisionCavalier.Shared/Linux. After copying it to the expected location, the package was made without error.
I'm new to Linux, so I was wondering if this was as fault of mine or a bug.

What was the script you use? If you used the one from AUR, it is out of data and needs to be updated to support V2023.11.0 (which is what introduced that change)

Yes, I believe I used git clone https://aur.archlinux.org/cavalier.git
That would make sense. Thank you!


Updated azure-communication-phonenumbers version after release.

API change check
API changes are not detected in this pull request.


Interpolate the variable holding all the fixup commits.

This makes sure that the expressions is evaluated based on the content of the var insteaf of the string "commit_messages"
Fixes the current bug where the ci thinks that we have fixup commits even when we don't, since the string "commit_messages" isn't empty.

Dog accepted


Add and detect AbuseRateLimitError.

It's similar to RateLimitError, but it's a different type of rate limit
error. It is documented at:
https://developer.github.com/v3/#abuse-rate-limits
Parse and include the Retry-After header value, if present. According
to GitHub support, its type is an integer value representing seconds. (/cc @izuzak FYI.)
Helps #431.

Some notes on this.
I'm implementing this based on GitHub documentation (and a clarifying/confirmation email I've sent to support@github.com). I've not had the opportunity to test it for real because I don't want to try to reproduce an abuse rate limit situation on purpose just for testing this PR.

I like it.  Thank you, @shurcooL!
Merging.


[FLINK-31296][table-planner] Add JoinConditionEqualityTransferRule to…

… stream optimizer
What is the purpose of the change
Add JoinConditionEqualityTransferRule to stream optimizer
Verifying this change
This PR is covered by the exist tests.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

@godfreyhe can you help review this pr ?

Hi @swuferhong could you help review this pr when you are free ?


fixed exclude files copy/paste error

Source Files in the Exclude example were a pattern, not a discreet set of files. If I've misunderstood, let me know!

The point of that particular example is to say, if you have any number of .log and .bak, in a tools and tools\build folder, and want to exclude all of them (regardless of filename), then use wildcards in the .nuspec entries.  Perhaps is would help to say something like "tools*.log (for example, build.log and test.log)" under Sources? Just giving a single filename is not the intent.

Thanks...looks more reasonable now. Thanks for the contribution!


Re-sign signature and follow redirects

On redirections (503), the default http client will retry the request on
the new Location.  However, the signature will be invalid since it
includes the original Host in the canonical string.
Retry the request after resigning with the new host.  This change
disables the default client retry mechanism and retries the request in
the executeMethod retry loop.  Unfortunately this incurs a retry delay
penalty.
Fixes #867

@fdawg4l this is not a correct fix a correct fix is posted here https://github.com/minio/minio-go/pull/829

I would suggest you close this PR and test #829 on your end.

@harshavardhana you're very right.  I like that patch much better than this.  Thanks!


Added new package langium-sprotty

This is work in progress for integrating Langium with Sprotty. I will test it with the "states" example in sprotty-vscode.

I will add an example to sprotty-vscode that demonstrates how this works. But I cannot add that example before this is merged and a snapshot version is published.

I adapted the code to the new multi-language support (#311). This required considerable changes. I will test it next week.


Block trap detour trap activation when there is outstanding request

It's breaking the timing of RouteChangePending.proceed().


Make lumX available via npm with browserify

First of all, I'd like to say thank you for this project. It is already far better than angular-material (widget set, look out of the box, performance) and hopefully will continue to improve.
I would love to be able to use lumx via npm+browserify, without bower. It's rather unfortunate that there is no consensus on packaging and dependency management for the frontend development. It appears (to my novice eye) that npm+browserify and bower are two top choices at the moment, and it also appears that npm and browserify are both very versatile and ready to evolve to support frontend more while bower is rather naive (nothing wrong with that, it's just that it doesn't do enough).
Initially I started my new project with angular-material and I was able to get all javascript dependencies set up with browserify, having single <script> element in index.html to load browserify-produced bundle.
I found angluIar-material lacking and was very glad to discover lumX. Trying to set it up with browserify I ran into to two problems. One of course is the topic of this issue - lumX doesn't seem to be available as npm module. Another is that even though all lumX's dependencies are available as npm modules, velocity.js ('velocity-animate' in npm registry) is expected to be a jquery plugin (dropdown menu element is looking for .velocity() method) and for some reason velocity doesn't register as jquery plugin when these components are loaded via browserify's require(). It appears that velocity.js itself no longer requires jquery and mere "presence" of jquery doesn't make it register as plugin.
I was able to get lumX (0.2.11) to work by adding all the dependencies as <script> elements, but using <script> and relying on browser globals feels like a step backwards compared to using modules with require().

+1 I know that the NPM guys have started making the transition to handling front-end libraries as well. This is why Google has started publishing Angular (and all it's related modules) on both NPM and Bower.

+1, please do this. I'm not using browserify. I'm using webpack. It's amazing. I highly recommend it. Either way, I can use webpack to get my bower packages, but I much rather have everything use npm. Please publish a version on npm :-) I recommend this method

+1
I think npm is evolving to support front-end and I am already using it for front-end dependencies.
Please do this.

Note, the angular team is going to do module.exports = 'moduleName', so I highly recommend that this project does the same so when I require('lumX') it gives me the name of the module, not the module itself.

+1, regardless of potential use of browserify, npm is as much used as bower for front-end packages management. And if you're on another stack than nodejs and already have a package manager, like bundler for Ruby or Composer for PHP, since npm is required for Bower, it's makes 3 package managers... And seriously Bower doesn't provide any feature that npm does not have.
And since the package.json is very similar to the bower.json, it's really easy to publish!

+1, also using Webpack. Luckily, all of the LumX Bower dependencies have existing npm modules already.  For anyone with a similar issue, you can load the LumX dependency with the imports-loader.
 global.$ = global.jQuery = require('jquery');
  require('jquery.transit');
  require('velocity-animate');
  require('angular');
  global.moment = require('moment');
  require('imports?angular!./bower_components/lumx/dist/lumx.js');

+1, please do this.

+1

I have a full gulp-browserify bundle for LumX that I was planning on submitting soon.
Will allow us to get up and running quickly, and dramatically cut down the gulp file

Sorry, I really was deep in other project, I'll try to manage this ASAP.

+1

NPM is gonna be pretty much the "de facto" standard way to distribute browser modules, so this should be published there as well

p.s. not just browserify, the same thing that's on bower should be on npm as well

+1 Webpack prefers NPM (CommonJS)

+1 for NPM!

+1 👌

+1

+1

+1 - The only reason I need bower at the moment is Lumx, all other dependencies are bundles using Webpack and NPM

+1

+1

+1

+1

So I was able to npm install lumx but it doesn't pull down the right release, it doesn't include the dist/ dir, can you update the version for npm?

Any updates on this? Basically all of the client-side dependencies you can think of are already in the NPM.

Hats off guys !!!
Only using Webpack + NPM. Now I need bower for just Lumx.
Please do this !

It seems someone published your framework on npm https://www.npmjs.com/package/lumx , I'd really like to have the possibility to get lumX from npm as well.
Any news on it ?

@kentcdodds Any chance you can share with us how you manage to use lumx bower with webpack ?

Unfortunately not, it's been a long time since I've used Bower and I strongly recommend against using Bower these days.

No worries, but indeed I just discover this https://github.com/bower/bower/issues/2298#issuecomment-302792690
cc @malexandre I think this should be a good enough reason for lumX to be on NPM as well no ?


Split feature work into separate parsing & compilation steps

This is my first PR that actually adds a new Work implementation. This was relatively painless, but slightly confusing in a few places:

how work creation works is not very well documented or obvious.
there are a few places where I think a trait could simplify things significantly. Mainly It feels like all of the add_x_job methods want to be generic, so that you just have something like Workflow::add_job<T: WorkOrWorkMakerOrWhaterver>, and this calls a trait method like Work::create.
the add_x_job implementations are kind of opaque and it's not clear what you're supposed to do here. Another benefit of a trait is that it gives you somewhere to document what is expected.
the need for everything that gets stored in a context to serialize itself is a papercut. In the cast of the AST it is cheap to recreate and inefficient to serialize, and it would be nice if this were expressible.

Overall though not too bad, and I got this working without much struggle.

Want to file an issue for context items that don't serialize? - I agree that would be a good change.
+1 to the add job bit; I've been gradually making those more self-similar with roughly that intent.

opened #697 and #698.


feat(core): warn if private APIs are not enabled, closes #3481

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[ ] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Notarizing still works, but AFAIK the app can't be published to the App Store.

Ah okay, well I'll add a sentence to the docs about it 👍🏻


Support a "stub_distribution" metadata field

This allows us to override the generated and uploaded distribution name,
e.g. if the name is already in use.
Cf. python/typeshed#9246

Closing in favor of #79.


 It would be very helpful if you could suggest a solution.
Thanking you!
Kind Regards
Rangeet

Hello
I would start with a more simple pipeline first and see, what happens:
gst-launch-1.0 tcambin name=tcam0 ! video/x-raw,format=GRAY8, width=640, height=480,framerate=30/1,binning=2X2 !  videoconvert ! x264enc ! mp4mux ! filesink async=true sync=false name=save-sink location =/home/kevin/test.mp4 -e
And see, what happens.
I see, you show a live video window too. Do you see only one image there too?
Stefan

Hi Stefan
I tried the new one, the code is running but no running window pops up. Its shows up to redistribute latency... in the terminal only.
Need help!
Thanks
Rangeet

Hi Rangeet
there is no window, because the pipeline does not have one. More interresting: What is in the test.mp4 in kevins's directory?
Stefan

Dear Stefan,
the video has been saved and it works fine. But, is it possible anyhow to record and view the video simultaneously?
That would be great!
Thanks!
Rangeet

Hello Rangeet
If video saving works fine now, start enhancing your GStreamer pipeline with an ximagesink as you tried at beginning.
I suggest to to that step by step, so start with showing the video in the ximagesink, but do not add video saving.
Then start adding the tee and so one.
Do that step by step and always check the results you get.
It is all GStreamer usage. It is not camera related.
 Somehow error pops up or one shap shot happens. Need some solution !
Thanks !
Regards
Rangeet

Hello Rangeet
You need a "tee" filter : https://gstreamer.freedesktop.org/documentation/coreelements/tee.html?gi-language=c
It splits the GStreamer Pipeline into branches. One for display and another for video capture. You used that already in your first post.
Searching in the internet for "Gstreamer tee examples" you will find
https://stackoverflow.com/questions/45873407/gstreamer-tee-with-different-capabilities-on-each-branch
and more.
It is GStreamer usage, not a camera issue.
Stefan

Hello Rangeet,
I am sorry, but I am not familiar with the queue parameters. You may look at https://gstreamer.freedesktop.org/documentation/coreelements/queue.html?gi-language=c for the documentation.
Stefan

Hello Rangeet
I made some tests on my own.

I am sorry, I have no solution for this issue. Thus, I suggest you ask the question with the videotestsrc pipeline above at https://stackoverflow.com/ or at https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues.


Adjustments for small screens

Allow time display to be customized a bit more. We can optionally show the date/timezone, we can also use a short-time-date prop. What happens is clearer than with the smAndDown prop. Also slightly offsets the mouse icon to look more 'over the button' than 'in the button'

Something about this breaks the middle "show me what the nova will look like" button on mobile. It takes you to the T CrB location, but it does not switch the layer btw Tycho & USNO. @johnarban or @Carifio24 can one of you please take a look?
This is otherwise a much improved small screen layout!

Yes, that was an intentional change - I thought i put a note, but I needed to remove the button which toggles the tycho layer in order to make things fit properly. There wasn't enough space for all 3 large buttons, especially when the 'T CrB is Set' chip is visible. It is gone only for very small landscape screens ( < 330 px)

So for me, the button is visible, but it doesn't behave how it should. You can press it, and it will take you to T CrB if it wasn't already centered on it, but the layer views don't change. I see why we'd want to get rid of the button all together for small screens, but we don't want a visible but non-working button.


Update tapir-core, tapir-http4s-server, ... to 1.1.3

Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-newtype
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-swagger-ui

from 1.1.2 to 1.1.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.1.2).
You might want to review and update them manually.
project/Dependencies.scala

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #264.


Remove sudo chmod hack from Makefile

We need a permanent solution to some lingering permissions issues in the dash-pipeline/Makefile. sudo chmod is a temporary workaround. Docker containers need to be built w/o a developer's username and permissions baked in. ;)

Fixed by https://github.com/Azure/DASH/pull/202


Support Vim under Windows

Support for Neovim under Windows has been added in #40. However, the plugin does not work in Vim under Windows yet.

Running the following in Vim on Windows
let term = term_start('NONE') | wincmd p
let termtty = term_gettty(term, 1)
call writefile(['Test'], termtty, 'b')

results in
E482: Can't create file \\.\pipe\vim-10940-in-2


Feature Request: Support for expo-av

Hey there - I was sent here by Mux Support when enquiring about Mux SDK support for Expo's video player, expo-av. I was instructed to open a feature request, and I'm hoping to get some information on whether there'll be any support for expo-av in the future.
It's maybe worth noting that I also looked into replacing the expo-av player with react-native-video, but I found that in order to replace it, I would have to eject my project from Expo.
I did get some playback working when wrapping the expo-av player in the react-native-video Mux SDK. However, the main issue I am seeing is that the video player remounts when any state vars are updated in onProgress (or onPlaybackStatusUpdate with expo-av). This results in playback stuttering/looping as the player remounts on each poll.
I hope this info helps, and if there's anything I can do to help support expo-av with Mux, I'm more than happy to help. This issue really puts my work on halt, so any info on this is very appreciated. Thanks!

Hi @petracles thanks for the issue. <mux-player> and <MuxPlayer /> React are both for web right now. But we would the equivalents on the native platforms iOS, Android, React Native, Flutter, etc.

I did get some playback working when wrapping the expo-av player in the react-native-video Mux SDK

That is the Mux Data SDK for react-native-video? Sounds like a bug in that SDK that we'll need to patch in that SDK to play nice with expo-av

Hey @dylanjha thanks for the quick response.
Yeah correct, I'm referring to the Mux Data SDK for react-native-video. If there is an easy fix to make the expo-av player compatible, that would be huge and would unblock my Mux Data integration for my project.
Let me know if there's anything I can do to help with this! Happy to share any source or test any fixes/suggestions you have in mind.
Thanks again!


Add margin shorthand helper function

https://developer.mozilla.org/en-US/docs/Web/CSS/margin

This has been added and released in v9.1.0


Add VA IaaS API

Adding API for Vulnerability assessment on IaaS.
The swaggers and examples in this PR have been approved in the private repository - https://github.com/Azure/azure-rest-api-specs-pr/pull/1597
(There are minor renaming changes)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

add "WaitForARMFeedback" since new API version added

hi @PhoenixHe-msft could you pls help to look at the SDK Schema CI failure it failed at SDK Automation step, thanks.https://dev.azure.com/azure-sdk/internal/_build/results?buildId=576163&view=logs&j=c3091266-fad0-55aa-282d-90fffef43473&t=12dc5c1e-bc2e-5f06-18e4-c23fee099af4

hi @changlong-liu could you pls look at the Python SDK breaking change label? thanks.

hi @changlong-liu Changlong Liu FTE could you pls look at the Python SDK breaking change label? thanks.

Hi @yungezz , the Python breaking changes have already exist in previous merged PR(https://github.com/Azure/azure-rest-api-specs/pull/10893). I think this PR is fine.

This has already been approved in the private repo (https://github.com/Azure/azure-rest-api-specs-pr/pull/1597/files) . The same APIs are being brought into the public repo now.
Signing off from ARM side.

/azp run


Multi gallery select

Please take a moment to fill out the following:
Fixes # . Rebase and minor reimplementation of #355 to account for project structure change
Changes Proposed in this pull request:

Allows multiple image selection for iOS/Android/UWP (stubbed for Tizen)

1cd38ec should address https://github.com/jamesmontemagno/MediaPlugin/issues/598

@jamesmontemagno is this one good to go?

rebased over the last merge and retested on iOS.

@jamesmontemagno any plans on merging this?

I was going to look into implementing this. Thank God I check the pull request list. @jamesmontemagno Could you please please merge this?

Just wanted to say thank you @pfaucon for implementing this. Not to mention preserving exif data in iOS, often overlooked. Can't wait to see this merged!

@motoko89 I'm not sure why the target isn't recognizable, on my side it builds but only on Windows, but it was like that before I made any changes.  Are you able to build the project from master?

@motoko89 I'm not sure why the target isn't recognizable, on my side it builds but only on Windows, but it was like that before I made any changes. Are you able to build the project from master?

Yeah I know you didn't change anything regarding building. I can't build from master either

Maybe try installing Android studio or the target framework?

I added the files into respective projects directly and it works now. Need to do a few tweaks. Can't wait for this to merge

Hello is this going to be merged we need this ASAP

@jamesmontemagno

Merge?

Any updates on this?

@pfaucon Need to merge master

@pfaucon Need to merge master

Yes, that is true; I dont know if @jamesmontemagno is gonna merge or not...
@pfaucon  can you merge and if possible build it so that we can consume your changes and test??

I'm still waiting for the moment :)

I am working on a PR for cancelation tokens, which is looking good, then will pull this in.

Unreal I'm itching for this as well...

@jamesmontemagno  It seems this feature is enabled in v<IP_ADDRESS>
Can you let me know how to use it please?

@jamesmontemagno It seems this feature is enabled in v<IP_ADDRESS>
Can you let me know how to use it please?
@pfaucon Any answer here please?

You can just add the nuget package to your solution and wherever you were using _media.PickPhotoAsync(options), replace it with _media.PickPhotosAsync(options)


Add transparent layers to example app

@johnpryan Looks I need a bit of description to close this issue? Could you give some? Happy to work on :)

#453 should make this easier...


Tutorial 2. TypeError: 'Example' object is not iterable

Hi,
When I use my own text dataset, which format is tsv, like following:
pos	"Bromwell High is a cartoon comedy. It ran at the same time as some other programs about school life, such as ""Teachers"". My 35 years in the teaching profession lead me to believe that Bromwell High's satire is much closer to reality than is ""Teachers"". The scramble to survive financially, the insightful students who can see right through their pathetic teachers' pomp, the pettiness of the whole situation, all remind me of the schools I knew and their students. When I saw the episode in which a student repeatedly tried to burn down the school, I immediately recalled ......... at .......... High.  STUDENT: Welcome to Bromwell High. I expect that many adults of my age think that Bromwell High is far fetched.
pos	"Homelessness (or Houselessness as George Carlin stated) has been an issue for years but never a plan to help those on the street that were once considered human who did everything from going to school, work, or vote for the matter. Most people think of the homeless as just a lost cause while worrying about things such as racism, the war on Iraq, pressuring kids to succeed, technology, the elections, inflation, or worrying if they'll be next to end up on the streets.

But what if you were given a bet to live on the streets for a month without the luxuries you once had from a home, the entertainment sets, a bathroom, pictures on the wall, a computer, and everything you once treasure to see what it's like to be homeless? That is Goddard Bolt's lesson.

Mel Brooks (who directs) who stars as Bolt plays a rich man who has everything in the world until deciding to make a bet with a sissy rival (Jeffery Tambor) to see if he can live in the streets for thirty days without the luxuries; if Bolt succeeds, he can do what he wants with a future project of making more buildings. The bet's on where Bolt is thrown on the street with a bracelet on his leg to monitor his every move where he can't step off the sidewalk. He's given the nickname Pepto by a vagrant after it's written on his forehead where Bolt meets other characters including a woman by the name of Molly (Lesley Ann Warren) an ex-dancer who got divorce before losing her home, and her pals Sailor (Howard Morris) and Fumes (Teddy Wilson) who are already used to the streets. They're survivors. Bolt isn't.  Maybe they should give it to the homeless instead of using it like Monopoly money.

Or maybe this film will inspire you to help others."
pos	"Brilliant over-acting by Lesley Ann Warren. Best dramatic hobo lady I have ever seen, and love scenes in clothes warehouse are second to none. The corn on face is a classic, as good as anything in Blazing Saddles. The take on lawyers is also superb. After being accused of being a turncoat, selling out his boss, and being dishonest the lawyer of Pepto Bolt shrugs indifferently ""I'm a lawyer"" he says. Three funny words. Jeffrey Tambor, a favorite from the later Larry Sanders show, is fantastic here too as a mad millionaire who wants to crush the ghetto. His character is more malevolent than usual. The hospital scene, and the scene where the homeless invade a demolition site, are all-time classics. Look for the legs scene and the two big diggers fighting (one bleeds). This movie gets better each time I see it (which is quite often)."
pos	This is easily the most underrated film inn the Brooks cannon. Sure, its flawed. It does not give a realistic view of homelessness (unlike, say, how Citizen Kane gave a realistic view of lounge singers, or Titanic gave a realistic view of Italians YOU IDIOTS). Many of the jokes fall flat. But still, this film is very lovable in a way many comedies are not, and to pull that off in a story about some of the most traditionally reviled members of society is truly impressive. Its not The Fisher King, but its not crap, either. My only complaint is that Brooks should have cast someone else in the lead (I love Mel as a Director and Writer, not so much as a lead).
pos	"This is not the typical Mel Brooks film. It was much less slapstick than most of his movies and actually had a plot that was followable. Leslie Ann Warren made the movie, she is such a fantastic, under-rated actress. There were some moments that could have been fleshed out a bit more, and some scenes that could probably have been cut to make the room to do so, but all in all, this is worth the price to rent and see it. The acting was good overall, Brooks himself did a good job without his characteristic speaking to directly to the audience. Again, Warren was the best actor in the movie, but ""Fume"" and ""Sailor"" both played their parts well."

I follow the Appendix A- Using torchtext with your own dataset to load my own dataset. When input:
TEXT.build_vocab(train_data)

It shows error:
TypeError: 'Example' object is not iterable

I guess it maybe about the dataset for not doing data cleaning. Could you please provide me some guidance on how to pre-process the dataset with torchtext?
Thanks in advance!

Can you post the code you are using? You might be defining the fields incorrectly.

@bentrevett
Thank you for your response.
Here is my torchtext defining code:
# 1. data.Field()
TEXT = data.Field(tokenize='spacy', include_lengths=True)
LABELS = data.LabelField()

# 2. data.TabularDataset
train_data, test_data = data.TabularDataset.splits(path=dataset_path,
                                                   train="train.tsv",
                                                   test="test.tsv",
                                                   fields=[('labels', LABELS), ('text', TEXT)],
                                                   format="tsv")

# 3. Split train_data to train_data, valid_data
train_data, valid_data = train_data.split(random_state=random.seed(SEED))

# 4. data.BucketIterator
train_iter, valid_iter, test_iter = data.BucketIterator.splits((train_data, valid_data, test_data),
                                                               batch_size=BATCH_SIZE,
                                                               device=device,
                                                               sort_key=lambda x: len(x.text))

# 5. Build vocab
TEXT.build_vocab(train_data)
LABELS.build_vocab(train_data)

It runs successfully, but when I input this code to see text, text_lengths in every batch:
for batch in train_iterator:
    text, text_lengths = batch.text
    print(text)

It shows error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-11-fb59b3b1eadd> in <module>
----> 1 for batch in train_iterator:
      2     print(batch)
      3     text, text_lengths = batch.text

/usr/local/lib/python3.7/site-packages/torchtext/data/iterator.py in __iter__(self)
    140         while True:
    141             self.init_epoch()
--> 142             for idx, minibatch in enumerate(self.batches):
    143                 # fast-forward if loaded from state
    144                 if self._iterations_this_epoch > idx:

/usr/local/lib/python3.7/site-packages/torchtext/data/iterator.py in pool(data, batch_size, key, batch_size_fn, random_shuffler, shuffle, sort_within_batch)
    284     for p in batch(data, batch_size * 100, batch_size_fn):
    285         p_batch = batch(sorted(p, key=key), batch_size, batch_size_fn) \
--> 286             if sort_within_batch \
    287             else batch(p, batch_size, batch_size_fn)
    288         if shuffle:

TypeError: '<' not supported between instances of 'Example' and 'Example'

Are you using the latest version of TorchText? I've managed to run your code without any errors.

Finally it works.
The error doesn't affect the training with BiLSTM, though I don't know why it shows that error.
Thank you for your guidance and patience! I will close this issue.


Add ability to disable caching of transforms in Gradle

When a transform fails to upload to the remote build cache for whatever reason, caching for transforms and Gradle tasks is disabled for the rest of the build.
An example: 413: Request Entity Too Large when a transform produces a very large output would cause the remote build cache to be disabled for the rest of the build.
Request
Add an API similar to what one would do with a Gradle task: outputs.cacheIf { false } in order to disable caching of the transform.

How would this method be called? How would you select which transforms you'd want to disable caching for? Can you give an example?

I don't know all the specifics but maybe it could be called cacheIf { false } just like the method for Gradle tasks.
Without any visibility in to transforms in builds scans I would simply disable the transform which caused the entity too large exception which I would see in the logs. Once transforms appear in build scans, I would disable all those which are more disk based transforms and take longer to retrieve from the cache than to re-execute.

Transforms don't have the same sort of IDs as tasks. With tasks, you can do tasks.named("lajos").outputs.cacheif { false } or tasks.withType(JavaCompile) { outputs.cacheIf { false } } but there are no equivalents for transforms. E.g. we don't have transforms.withType() nor do have transforms have names the way tasks have. So the question is still how you'd imagine explaining to Gradle which transforms to not cache.

Considering our plans with fixing negative savings by speculative execution in the foreseeable future, do you think we can put this away and check if we still need it once we have that feature?

Sounds good to me! Thanks


Bulk Data for Integration Testing

Use Case Background

Data set will support testing of integration between Drupal and Salesforce
Account, Affiliation and Opportunity record types are important
Account Membership Status controls signed in user's access to pricing, specific content etc

Generate records from following sobjects

Accounts (record type 1)
-  records that represent at least 3 different values from custom field, with at least two records representing 2 specific values

Accounts (record type 2)

Contacts
- Need Contact to have multiple Affiliations, for both Affiliation record types, with both Current and Former Status

Affiliations (record type 1)

Affiliations (record type 2)

Opportunities (record type 1)
- Need Parent Campaign created

Misc Notes

Ability to opt for default values for fields or override and set custom values
Ability to SAVE data spec template
Ability to select what is pushed to the Org and what can be exported

This is basically made moot by the existence of Snowfakery.


fix(package.json): replace repository.url with new URL in all package

🚨 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
Due to the community package repository move, I changed repository.url to a new URL to make it easier to refer to the source code from the npm page.
Related https://github.com/dotansimha/graphql-code-generator/discussions/8600

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
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Thanks for the fix @MH4GF! ⚡


ref: throw an error when an object is invalid

Q
A

Bug fix?
N

New feature?
N

BC breaks?
N

Deprecations?
N

Fixed tickets
N

Throwing an error on invalid object, showing that it uses less code.

We finally found a solution ;-) Tks


Method list not updated for included mixins

If I specify --embed-mixins or do something like:
#@!parse include MyMixin

The mixin methods are documented directly on the class page as if they were written inline, but they don't show up in the the method list for the classes where it's included. I suppose one could argue that as being correct behavior, but I find it a little confusing/misleading.
I'm using yard <IP_ADDRESS>.

The only concern would be that adding embedded methods to the method list would cause a huge method list explosion for certain projects that might rely on it heavily, like Rails, for instance.
The other point is that just like regular inheritance, the methods don't actually exist on that object, they exist in the parent in the inheritance tree. We don't duplicate methods inherited from regular superclasses, so I'm not sure why YARD wouldn't behave the same for mixins.
The template logic for embedding mixins is simply a presentation layer convenience, it shouldn't affect the overall structure of where code lives.

Hmm, I see your point. I guess we can close this then.


Build dist with poetry

https://github.com/appliedAI-Initiative/pymetrius/blob/5523ddbd97420d6e27cc7333bdc4972f1ca32ca0/{{cookiecutter.project_name}}/.github/workflows/publish.yaml#L26-L29

@MischaPanch I did not really understand this.
Also, we are missing the step to publish github pages I guess.

This is for publishing an artifact to pypi. Previously it was done by setup tools, now it has to be done by poetry. I'm sure there's a bunch of explanations online how to do that, should be in official poetry docs

Done in 5fe7a71d54


[Tech Debt]: Remove zombie code from Sidebar.tsx file

What is needed?
The file Sidebar.tsx has two state values managed by useState() hooks named searchInput and searchMode
These values were created some time ago before search functionality had been extracted to the useSidebarSearch() hook. However, they're still present and being used functionally in the file's control flow. These files should be removed and any valid functionality should be replaced with data returned from and managed by useSidebarSearch.ts

can i get this assigned to  me


Fix windows build with icu tag

In mingw64, icu comes with libicuin instead of libicui18n library.

Codecov Report

Merging #1017 (7eefebc) into master (ae2a61f) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1017      +/-   ##
==========================================
- Coverage   46.17%   46.04%   -0.14%
==========================================
  Files          11       11
  Lines        1490     1490
==========================================
- Hits          688      686       -2
- Misses        666      667       +1
- Partials      136      137       +1

Impacted Files
Coverage Δ

sqlite3.go
52.97% <0.00%> (-0.23%)
:arrow_down:

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 ae2a61f...7eefebc. Read the comment docs.


tensorflow_backend.py add backend deleted in Keras

workaround for keras_backend change:
https://github.com/fchollet/keras/commit/6b106ab4ec9a1c0eb3e24ae590ce63f84022ad40
Should fix this properly to match the master backend.

replaced by https://github.com/farizrahman4u/keras-contrib/pull/168


chore(api): Add GraphQL IAM integ target

Issue #, if available:
Description of changes:

Add separate target for GraphQL IAM integration testing
Need to add separate targets for each README, GraphQL UserPool, REST UserPool, REST IAM, GraphQL Lambda Auth, etc

Check points: (check or cross out if not relevant)

[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all target using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

the changes are now in https://github.com/aws-amplify/amplify-ios/pull/2119


Implement PID File for go-dnscollector to Enable SIGHUP Configuration Reload

Implement PID File for go-dnscollector to Enable SIGHUP Configuration Reload
Description:
Now that SIGHUP is implemented on the go-dnscollector side (see), we (pDNSSOC) need to integrate this functionality for configuration reload without restarting the process. This will allow us to make changes on-the-fly, such as reading updated files, by sending a SIGHUP signal to the process without having to restart the process manually.
Background:
It became common practice for daemonized processes to use SIGHUP for configuration reload. This allows changes to be made without terminating the process.
Current Implementation Steps:

Find the process number of go-dnscollector.
Run the following command to reload the configuration:kill -SIGHUP 21371  # Assuming 21371 is the PID of the running go-dnscollector

Challenge:
The PID changes every time the process restarts, which makes it difficult to automate this with a static command.
Proposed Solution:
Implement a PID file in go-dnscollector that stores the current PID of the running process. This PID file will be created on startup and can be referenced to send the SIGHUP signal.
Benefits:

Simplifies the process of sending SIGHUP signals.
Eliminates the need to restart the process for configuration changes.
Potentially allows us to remove the dependency on supervisord, as it is currently used only for restarting on config changes.
Follows a good and common industry practice.

Additional Notes:

Confirm if Denis has implemented the PID file functionality. If not, this issue should prioritize that implementation.
If we (pDNSSOC) can ditch supervisord, a Python script inside fetch-iocs can monitor file changes and send the SIGHUP signal accordingly.

References:

Run Program and Generate PID

Action Items:

[ ] Implement PID file creation in go-dnscollector.
[ ] We (pDNSSOC) will have to create a script to read the PID from the PID file and send the SIGHUP signal.

This sounds perfect for us. Thanks!

feature implemented in master branch

The issue seems to be resolved. Thanks for your quick action!


pip install deploy-agent doesnt work

Based on the wiki documents I understood that deploy-agent would be available through pypi, but running pip install deploy-agent doesn't work. Is the intended usage to install manually via local install?

I think we don't put it on pypi although it was installed via pip from our internal repo. The external usage is either installing manually or uploading to private internal repo


Make sbt-revolver an AutoPlugin

It will be starting from the upcoming 0.8.0-RC1.

Yay! When will 0.8.0-RC1 be released?

I already released it to bintray but I just asked for inclusion into sbt-plugin-releases. So, for now you could give it a try by adding a custom resolver
resolvers += Resolver.url("jrudolph-sbt-plugins", url("https://bintray.com/jrudolph/sbt-plugins"))(Resolver.ivyStylePatterns)

Actually, for some reason the redirection to dl.bintray.com does not work automatically, so, you need to make it
resolvers += Resolver.url("jrudolph-sbt-plugins", url("https://dl.bintray.com/jrudolph/sbt-plugins"))(Resolver.ivyStylePatterns)

Thanks!


[Telink] Update Docker image (Zephyr update)

Change overview

drivers: bluetooth: hci_b9x: Fixed samples compilation
boards: riscv: telink: Add support 4 buttons
soc: riscv: telink_b9x: Fix retention functionality
boards: riscv: telink: Move HEAP_MEM_POOL_SIZE into Kconfig
soc: telink: b9x: Added RF calibration at startup

Testing
Tested manually.
Steps:

Build image
Run docker
Check if Telink examples able to be built successfully

@andy31415 Could you help to create new Docker Image?


Restore per-device options being reachable through HyprCtl.

Description
It seems we lost access to per-device config options since the new curly braces format was adopted.
So this does not work anymore:
hyprctl getoption device:my-device:sensitivity
hyprctl keyword device:my-device:sensitivity 1

Getting/setting these values on-demand is important for accessibility, at least mouse sensitivity.
As far as I can tell, the issue lies in HyprCtl.cpp:
const auto VAR = g_pConfigManager->getHyprlangConfigValuePtr(curitem);

It only checks for "ConfigValue", but per-device options are stored as "SpecialConfigValue" in ConfigManager.cpp:
m_pConfig->addSpecialConfigValue("device", "sensitivity", {0.F});

Any chance this lost functionality will be restored?

hyprctl keyword device:my-device:sensitivity 1

the syntax has been changed a while back and is documented on the wiki:
hyprctl keyword 'device[my-device]:sensitivity 1'

for getoption, there's #5724

It's your project, I respect 100% if you just don't feel like solving this issue, but I have some more information.
The current state of affairs seems to be that the functionality was lost due to refactoring, an attempt has been made to reimplement half of it, i.e. the keyword part, but it currently doesn't work, at least not for everyone, and is not documented in a very obvious place.
Setting per-device values with hyprctl:
The new syntax is not mentioned in the wiki on the hyprctl page, nor on the keywords page under device-specific configs. I still haven't been able to find it. But at least it's mentioned in this issue, so people can find it.
However, the command you mentioned doesn't work. I can set this in the .conf:
device {
    name = kensington-eagle-trackball
    sensitivity = -0.5
    accel_profile = flat
}

But none of these commands do anything. I just get an ok and the sensitivity remains the same:
hyprctl keyword 'device[kensington-eagle-trackball]:sensitivity' -1
hyprctl keyword 'device[kensington-eagle-trackball]:sensitivity' 0
hyprctl keyword 'device[kensington-eagle-trackball]:sensitivity' 1
hyprctl keyword 'device[kensington-eagle-trackball]:sensitivity' 999

I can even put any garbage in there as the device name, and it just says ok:
hyprctl keyword 'device[otter-nose-pointer]:sensitivity' 1

Getting per-device values with hyprctl:
The solution for getoption you linked to is a thread of people talking about how they also are missing this functionality, and suggesting just programmatically changing the .conf file, which is circumventing hyprctl.
In conclusion:
We could just change the .conf file in script for getting/setting these values and not bother with hyprctl altogether, I suppose that's an actual solution. Again, I respect if you're just not interested in making this work with hyprctl, but if that's the case, I'd highly recommend making it clear in the wiki that it's no longer supported, on purpose.
Version info, if it matters:
Hyprland, built from branch  at commit 918d8340afd652b011b937d29d5eea0be08467f5  (flake.lock: update).
Date: Tue Jun 25 12:06:02 2024
Tag: v0.41.2, commits: 4886

it doesnt work cuz I forgot you need the -r flag to hyprctl

True, this works:
hyprctl -r keyword 'device[kensington-eagle-trackball]:sensitivity' -1

That's definitely better than using sed to write changes to the .conf file. So all we need is the documentation available somewhere it can easily be found, as well as a reimplementation of getoption.

This doesn't work, though:
hyprctl -r keyword 'device[kensington-eagle-trackball]:sensitivity' -0.5

It seems it doesn't accept decimals.

So all we need is the documentation available somewhere it can easily be found

MRs welcome

It seems it doesn't accept decimals. If all this is mentioned in the wiki, I'm sorry, but I just can't find it anywhere.

- is parsed as an option marker, typical shell behavior:
hyprctl -r -- keyword 'device[kensington-eagle-trackball]:sensitivity' -0.5

Brilliant, thanks!
PR here: https://github.com/hyprwm/hyprland-wiki/pull/728


"Documenting..." messages should use \ on windows

Discovered in https://github.com/rust-lang/cargo/pull/4627#issuecomment-339289446, the "Documenting" messages aren't using \ on windows and should be:
failures:
---- doc_multiple_targets_same_name stdout ----
	running `C:\projects\cargo\target\debug\cargo.exe doc --all`
thread 'doc_multiple_targets_same_name' panicked at '
Expected: execs
    but: expected to find:
[DOCUMENTING] foo v0.1.0 (file:///C:/projects/cargo/target/cit/t10/foo[/]foo)
did not find in output:
 Documenting bar v0.1.0 (file:///C:/projects/cargo/target/cit/t10/foo/bar)
 Documenting foo v0.1.0 (file:///C:/projects/cargo/target/cit/t10/foo/foo)
    Finished dev [unoptimized + debuginfo] target(s) in 0.68 secs
', C:\Users\appveyor\.cargo\registry\src\github.com-1ecc6299db9ec823\hamcrest-0.1.1\src\core.rs:31:12
note: Run with `RUST_BACKTRACE=1` for a backtrace.

@matklad it looks like the "Compiling" messages don't use windows paths either, should they?
I'm basing this on the observation that this test, and many others, don't use square brackets either....

@carols10cents Yeah, I think ideally all paths should use file separators, native to the current platform.
But this is an ultra cosmetic things actually, so it might be better to fix something else instead of this :)

Are you talking about file URLs? Those use forward slashes even on Windows, AFAIK. For example, calling UrlCreateFromPath on "C:\D\E F\G.txt" will produce "file:///C:/D/E%20F/G.txt".

@hban that makes sense actually... Which actually brings the question of why do we show URLs here?
I think that instead of
 Documenting foo v0.1.0 (file:///C:/projects/cargo/target/cit/t10/foo/foo)

The user should see
 Documenting foo v0.1.0 (C:\projects\cargo\target\cit\t10\foo\foo)

If anyone wants to fix it, the relevant Display impl is over here:
https://github.com/rust-lang/cargo/blob/a5b13e82db645bb5af33fc59d67da63ea30a17af/src/cargo/core/source/source_id.rs#L366
We need to make sure that this impl is not relied on by anything important, for example, that it doesn't accidentally leak to the lockfile. It seems to be the case, but it's better to double check.
Note that after the change the paths in Documenting ... and in lockfile will look slightly different, but that is probably not problematic.

If this happens we'll want to update https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#overriding-dependencies (search for "note the file:// in the build output")

Thanks for picking this up!
I would consider using:
let root = if cfg!(windows) { r#"C:\"# } else { "/" };

in front of the [..].
Btw, beware of this change: https://github.com/rust-lang/cargo/commit/d5fc8dc3a77b39d7bb3a5671c539d06b9dcc61fa#diff-b03a0182b2cc2abbd16483fb77b37a6d.

So, that check doesn't quite work because the file may be on a different drive. The closest I've got right now is a function in tests\testsuite\support\mod.rs that looks like this:
/// Gets a match expression suitable for matching the root directory on the platform.
pub fn root_match () -> &'static str {
    // On Windows, we need to use a wildcard for the drive,
    // because we don't actually know what it will be.
    if cfg!(windows) { r#"[..]:\"# } else { "/" }
}

And then at each of the call sites you need to do something like this:
p.cargo("build")
        .with_stderr(format!(
            "\
[UPDATING] registry `{root}[..]`
[COMPILING] bar v0.1.0 ({root}[..])
[COMPILING] foo v0.0.1 (CWD)
[FINISHED] dev [unoptimized + debuginfo] target(s) in [..]
", root=root_match()),
        ).run();

I can do that, but it's a bunch of now-failing tests to update, so I figured I'd check to see if that sounded crazy before I did the work.
As for the CWD thing, I think it already just kind of accidentally works because the CWD replacement is done both for the URL and for the actual CWD path, which will match what what we're converting the URL to.

So the CWD macro is doing two things, both of which against you (in a sense).
First of all it replaces, as you said, both the URL and the path, exactly what this issue is looking to change, so it makes all the tests agnostic to this change.
But, more importantly, that includes the root, which means that that match won't match. That's what I think is causing those now-failing tests.
So you're going to kind of have to find a way to avoid the CWD macro, or opt-out of it, or something 😕.

Ah, I see. That makes sense. But that’s because the replace is happening on the output. Could we change it to do “CWD -> Actual CWD Path” on the input?
Maybe that’s the solution to the other problem, too. Instead of replacing at the call site, just expand the uh...’template system’ to include ‘[[ROOT]]’ (and probably add some delimiters like that) and then do the match replacement in the evaluation.
Thoughts?


Docs problem on Setting up blog in an existing project

This isn't a code problem, but a tiny docs issue.
on https://middlemanapp.com/basics/blogging/ the instructions for activating the extension in an existing project are:
activate :blog do |blog|
  # set options on blog
end

Unfortunately, this doesn't work out of the box. This error is thrown:
NoMethodError at /blog/2012/01/01/example-article
undefined method `link' for nil:NilClass
[snip]/gems/middleman-blog-4.0.2/lib/middleman-blog/helpers.rb: in tag_path, line 86

Maybe the docs could be updated to include these two template options, so that the example works as specified?
activate :blog do |blog|
  # set options on blog
  blog.tag_template = "tag.html"
  blog.calendar_template = "calendar.html"
end

PS: love the project, thank you!

thanks @adamdill just what I needed

ℹ This issue is stale because it has been open for more than 90 days with no activity. It will be closed in 30 days if no further activity occurs. Thank you for your contributions.


curl does not follow redirects

Makefile in js/lib should read like this:
# Download copies of all third-party libs and check hashes

all:
        -curl -LK libs.txt
        -md5sum --check --quiet md5sums.txt

clean:
        -rm *.js

This repo is deprecated and archived. Therefore, this issue is now closed. Thank you.


[Bug]  alloy-dyn-abi::TypedData should populate Resolver EIP712Domain type fields when existent

Component
dyn-abi
What version of Alloy are you on?
0.5.2
Operating System
Linux
Describe the bug
When creating a alloy-dyn-abi::eip712::TypedData::from_struct() with a Eip712Domain the internal Resolver field is not populated with the domain type fields used.
This creates issues for recovery using the TypedData object signed (tested with browser injected wallet).

In my implementation im doing
let mut typed_data = TypedData::from_struct(&struct_object, Some(domain_object)); let _ = typed_data.resolver.ingest_string(typed_data.domain().encode_type());
to populate domain type fields


@deck.gl/jupyter-widget: Add additional browser tests

For #3640

Background
Addition of more tests to detect changes that break @deck.gl/jupyter-widget

Change List

Add three additional layer tests
Clean up canvas between tests

Coverage decreased (-2.2%) to 81.58% when pulling df2b162dc86f79f27b418b683516ba59b182cca0 on ajd/at-at-syntax-4 into 207b5e925719fa87c2ddce9ebf4e113449b2e9ad on ajd/at-at-syntax-3.


Doesn't seem to work with libraries in the user library directory on windows

Describe the bug
The tool fails to inline libraries that are in the SCAD user library directory (C:\Users<user>\Documents\OpenSCAD\libraries)
To Reproduce
Steps to reproduce the behavior:

install BOLS in user scad library
create an scad file with the only line "include <BOSL2/std.scad>;"
run scadi
inlined file is empty

Additional context
It does successfully inline files that are in the same directory as the target file.

Thank you for reporting this. I will look into adding library folder detection. Initially, I avoided this since it requires special handling for different operating systems.

It's working great, thanks.
In-lining a large library like BOSL2 seems to cause SCAD to hit a recursion limit, but I'm guessing that's an openSCAD bug/limitation rather than anything to do with your tool.
(I've attached the test file, and in-lined version, in case you wanted to see it)
hex-grid-test.txt
inline-hex-grid-test.txt

@Loki44 I tested this on my development server and can reproduce your results. This would be considered a bug, so I will create a new issue you can track.

Inlining confirmed to work with the user's Documents folder.

@Loki44 I have created a new issue for the OpenSCAD error: #51.


Expression was too complex to be solved in reasonable time; consider breaking up the expression into distinct sub-expressions

Xcode 9.0
UIColorExtension.swift
let results = (0...3).map{ (toComponents[$0] - components[$0]) * abs(multiplier) + components[$0] }
Expression was too complex to be solved in reasonable time; consider breaking up the expression into distinct sub-expressions

pod version was old


Upgrade Issue with bringing in kube-registry-proxy

From @vdice on March 8, 2017 21:12
In order to switch over from our in-house registry-proxy to the official/upstream kube-registry-proxy (as original PR https://github.com/deis/workflow/pull/734 proposed) we will need to sort out the following issue when upgrading.
v2.12.0 release candidate testing showed that after a Workflow install that uses the in-house variant of deis-registry-proxy (say, v2.11.0), when one goes to upgrade (helm upgrade luminous-hummingbird workflow-staging/workflow --version v2.12.0), although the deis-registry-proxy pod appears to have been removed, the new luminous-hummingbird-kube-registry-proxy sometimes does not appear due to a host port conflict:
 $ helm ls
NAME                	REVISION	UPDATED                 	STATUS  	CHART           	NAMESPACE
luminous-hummingbird	4       	Wed Mar  8 14:01:02 2017	DEPLOYED	workflow-v2.12.0	deis

$ kd get po,ds
NAME                                        READY     STATUS    RESTARTS   AGE
po/deis-builder-574483744-qnf44             1/1       Running   0          24m
po/deis-controller-3953262871-jqkmd         1/1       Running   2          24m
po/deis-database-83844344-m5x4x             1/1       Running   0          24m
po/deis-logger-176328999-d7fxc              1/1       Running   9          1h
po/deis-logger-fluentd-0hqfs                1/1       Running   0          1h
po/deis-logger-fluentd-drfh6                1/1       Running   0          1h
po/deis-logger-redis-304849759-nbrdp        1/1       Running   0          1h
po/deis-minio-676004970-g2bj9               1/1       Running   0          1h
po/deis-monitor-grafana-432627134-87b1z     1/1       Running   0          24m
po/deis-monitor-influxdb-2729788615-q67f9   1/1       Running   0          25m
po/deis-monitor-telegraf-6q562              1/1       Running   0          1h
po/deis-monitor-telegraf-rzwnv              1/1       Running   6          1h
po/deis-nsqd-3597503299-94nhx               1/1       Running   0          1h
po/deis-registry-756475849-v0rmw            1/1       Running   0          24m
po/deis-router-1001573613-mk07g             1/1       Running   0          13m
po/deis-workflow-manager-1013677227-kh5vt   1/1       Running   0          25m

NAME                                          DESIRED   CURRENT   READY     NODE-SELECTOR   AGE
ds/deis-logger-fluentd                        2         2         2         <none>          1h
ds/deis-monitor-telegraf                      2         2         2         <none>          1h
ds/luminous-hummingbird-kube-registry-proxy   0         0         0         <none>          24m

 $ kd describe ds luminous-hummingbird-kube-registry-proxy
Name:		luminous-hummingbird-kube-registry-proxy
Image(s):	gcr.io/google_containers/kube-registry-proxy:0.4
Selector:	app=luminous-hummingbird-kube-registry-proxy
Node-Selector:	<none>
Labels:		chart=kube-registry-proxy-0.1.0
		heritage=Tiller
		release=luminous-hummingbird
Desired Number of Nodes Scheduled: 0
Current Number of Nodes Scheduled: 0
Number of Nodes Misscheduled: 0
Pods Status:	0 Running / 0 Waiting / 0 Succeeded / 0 Failed
Events:
  FirstSeen	LastSeen	Count	From			SubObjectPath	Type		Reason		Message
  ---------	--------	-----	----			-------------	--------	------		-------
  25m		25m		2	{daemonset-controller }			Normal		FailedPlacement	failed to place pod on "k8s-agent-fbf26383-0": host port conflict
  25m		25m		2	{daemonset-controller }			Normal		FailedPlacement	failed to place pod on "k8s-master-fbf26383-0": host port conflict

Copied from original issue: deis/workflow#766

From @bacongobbler on March 8, 2017 21:14
let's see if we can distill this into a base case which we can hopefully ship a PR and functional test upstream to helm.

From @vdice on March 8, 2017 23:2
It is a possibility that this is due to a k8s regression (been running v1.5.x in my testing); perhaps related: https://github.com/kubernetes/kubernetes/issues/23013

From @vdice on May 2, 2017 17:10
Adding this to the v2.15 milestone.  We'll want to re-try this on a v1.6.x cluster.  As it stands, we've added deis/registry-proxy back into CI as features have come in with the Workflow v2.14 milestone.


Feature/guard against basic authentication

Motivation:
Issue #33
All Submissions:
Note: CONTRIBUTING.md seems to be missing?
I did not see a relevant PR.
Changes to Core Features:
The explanation is in the issue; I have run tests locally and did not break them.
I have not provided new tests.

About the CONTRIBUTING.md, it is typical for all PySnippet organization projects.


Add raw locations to the Subscriber SDK

enable listening for raw locations in the Subscriber SDK see https://github.com/ably/ably-asset-tracking-android/pull/495
┆Issue is synchronized with this Jira Task by Unito

@AndyNicks this linked PR https://github.com/ably/ably-asset-tracking-android/pull/495 has nothing to an example app as the github label might suggest - or am I wrong? :)

No you are right. Thanks for spottuing it.


compileFunc bugfix: public a recursive function

Codecov Report

Merging #119 (7ab63df) into main (9af9f93) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   87.25%   87.27%   +0.01%
==========================================
  Files          10       10
  Lines        3257     3262       +5
==========================================
+ Hits         2842     2847       +5
  Misses        330      330
  Partials       85       85

Impacted Files
Coverage Δ

cl/compile.go
94.63% <100.00%> (+0.07%)
:arrow_up:

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 9af9f93...7ab63df. Read the comment docs.


crash when opening video settings tab (neoforge 1.21.1)

https://mclo.gs/FZJ66n3

https://streamable.com/knago3

After testing mods, it's incompatibility with More Culling.

yes,txny says he would push a fix for that


Upgrade to Metrics 0.11.0.Final

@jsanda here locally, this has failed on /hawkular/metrics/status returning Not Found. Could you or somebody from metrics have a look what is wrong?

@ppalaga, Hawkular Metrics has automated tests for the status endpoint: StatusITest.groovy. 0.11.0 was released with all the tests passing. There must be something else happening there.

I restarted the build looked at the entire log. I see the status endpoint being deployed and I do not see anything in the logs about 404 on that endpoint.
@ppalaga, can you please link to that error in the last build?

No one even trusts the travis integration tests anymore!

Merging since the error has nothing to do with metrics but with:
testComplexScenario(org.hawkular.integration.test.HystrixCommandsITest)  Time elapsed: 14.476 sec  <<< ERROR!

Merged #782

This is something UI is waiting for.


DatePicker does not work with reactive forms (fix included)

v 4.15.0
Date Picker does not work with reactive forms
This is because
if (this.isFlatpickrLoaded() && this.flatpickrInstance.config) {

are both null at time of writevalue, so the value can never be set
https://github.com/IBM/carbon-components-angular/blob/master/src/datepicker/datepicker.component.ts#L313
Now the good news is that this is easy to fix just wrap it in a setTimeout like this (this allows the view to be completely initialized
writeValue(value: (Date | string)[]) {
    this.value = value;
	setTimeout(() => {
            if (this.isFlatpickrLoaded() && this.flatpickrInstance.config) {
                this.setDateValues(this.value);
            }
        }, 1)
  }

Thanks for having this fixed

the 1 is not needed
	writeValue(value: (Date | string)[]) {
		this.value = value;

		setTimeout(() => {
			if (this.isFlatpickrLoaded() && this.flatpickrInstance.config) {
				this.setDateValues(this.value);
			}
		});
	}

Thanks for hunting down the fix! I should have a PR up shortly 👍


Add Jetstack to the users section

Spoke to @bwplotka today. We here at Jetstack are happy users of Thanos.
Thanks for the hard work on it :+1:

Thanks!


[FEATURE] move wasmcloud.toml to a different directory for a tinygo component

Affected project(s)

[ ] documentation
[ ] examples
[X] wasmCloud host
[X] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Is your feature request related to a problem? Please describe.
When i create a tinygo component, I am not able to move the wasmcloud.toml to a different directory other than the root.
Describe the solution you'd like
I would like to have the ability to move the wasmcloud.toml to a different directory. I want to abstract out the specifics of wasmcloud host and provide an abstraction to the developer that looks like he/she s writing plain business logic in components.
Describe alternatives you've considered
No response
Additional context
No response

I think what would be the most useful thing to do here is add a working_directory or build_directory project configuration field to the wasmcloud.toml manifest that wash will use do change directory into (absolute, or relative to the wasmcloud.toml file) before attempting to run build commands.
This is a great first issue, and should almost entirely take place in the wash-lib crate

I think in general for this issue, it would be valuable to be able to move the wasmcloud.toml, wadm.yaml, build and wit directories to other places and ensure that wash build can work out of the box


Bug: Upload Function of DataLakeFileClient Resulting in Corrupted File with 15 KB Size on ADLS Gen2 Account.

storage-file-datalake:
Package Version:
Windows:
[ ] react

version:

[ ] chrome

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Upload Function of DataLakeFileClient Resulting in Corrupted File with 15 KB Size on ADLS Gen2 Account
To Reproduce
const customPipeline = newPipeline(new AnonymousCredential());
const dataLakeServiceClient = new DataLakeServiceClient(
url,
new AnonymousCredential(),
customPipeline
);
const fileSystemClient =
dataLakeServiceClient.getFileSystemClient(containerName);
const fileClient = fileSystemClient.getFileClient(blob.name);
const aborter = Azure.Aborter.none;
fileClient
.upload(blob.stream, {
pathHttpHeaders: {
contentType: 'application/octet-stream',
},
abortSignal: aborter,
onProgress: (TransferProgressEvent) => {
let percentComplete = TransferProgressEvent.loadedBytes / blob.size;
let loadedBytes = TransferProgressEvent.loadedBytes;
emitter({ percentComplete, loadedBytes });
},
})
Expected behavior
File should be uploaded with proper file size.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

The blob object was not proper so it's working now,


Upgrade Dependencies

@dotressel please provide a description why this was done next time, so it is documented for the future.
(The dependency updates were for Backstage 1.5.1.)


Implement autoplay

Features

Set autoplay delay via prop

This can be set to false at runtime to stop advancing.  AKA, a watcher should be attached to this that kills setTimeout

Suspend advancing when window de-focused

May happen automatically with a setTimeout based approach
Concern is that we don't queue up a bunch of advances if you switch to a different browser tab and then return

Stop advancing on hover (set by prop)

Required for https://app.asana.com/0/1199094171432804/1201499866805760

Closed by #28


Add snap config

What was wrong?

Write snapcraft.yaml configuration to allow for linux distribution via snap

Cute Animal Picture

ping @pipermerriam for :eyes:

Should this include a section in the documentation on how to install?  I don't really know anything about snaps so can't really give this a meaningful review....


Symbolicated Stack Trace On Device

Hi Karl @kstenerud - I am trying to use KSCrash to obtain a symbolicated stack trace on device for an iOS app. If the app crashed in the previous run then during the current run I want to be able to display the stack trace in a pop up kind of dialog or at least write the stack trace information to my own log file. Is this possible using KSCrash?
To understand KSCrash better, I started off with the Advanced example provided along with the KSCrash download and used the console installation. The advanced example prints the unsymbolicated stack trace in JSON format in the "Output window" of Xcode? Can the console installation print fully symbolicated stack trace(with line numbers)?

On-Device-Symbolication should still work if you're not using bitcode.

On-Device-Symbolication should still work if you're not using bitcode.

@mickeyl
I have had any luck with this. Can you please suggest what needs to be done to get the on-device symbolication working? Like is there a flag or something that needs to be set. Any info on this is appreciated.

Frankly, the only thing I had to do to get it working with KSCrash was to adjust the settings as indicated in the README:

On-device symbolication requires basic symbols to be present in the final build. To enable this, go to your app's build settings and set Strip Style to Debugging Symbols. Doing so increases your final binary size by about 5%, but you get on-device symbolication.

It then worked right out of the box for me. Note that I didn't use it myself for several years now (have to use bitcode now).


Multiple gpus training

One question: when I ran the training, I always used 1 single GPU because when I tried to use more than one the usage was always at 0%. Does the code work when all the GPUs are set to "EXCLUSIVE" mode?

Have you specified device_ids?

yes, I have always used the command that's on the readme.
It may be a problem of the server when I run it, but I just wanted to understand if the code worked when the GPUs where in exclusive mode as that may be an issue with the server.

Sorry, I have no idea what thar exclusive mode means. So you may try to see if some simple cifar multi gpu works for you. And if simple cifar with synchronous bn works.

@alessiapacca may you please share with us on which database you managed to train the network? What version of Python are you using?

@Eliot04 hey I trained with Vox dataset and I used python 3.6.4

@alessiapacca Super helpful. Thanks.

@alessiapacca I tried to use distributed data parallel to accelerate the training, and it semms to be working. Maybe you can try this too. (but synchronized BatchNorm may have problem when dist data parallel is used, i did not test it)

@alessiapacca I tried to use distributed data parallel to accelerate the training, and it semms to be working. Maybe you can try this too. (but synchronized BatchNorm may have problem when dist data parallel is used, i did not test it)

@SystemErrorWang How to use distributed data parallel to accelerate the training ？I put the model and datasets to DDP. but it seams to be  not working. the GPU usage was always at 0%

@SystemErrorWang How to use distributed data parallel to accelerate the training ？I put the model and datasets to DDP. but it seams to be not working. the GPU usage was always at 0%

I modified the code with this repo: https://github.com/rosinality/stylegan2-pytorch
adopted the ddp part of the stylegan2 code, combined with the First-Order Motion Model training code
It would spend some time to read the code, but unfortunately my previous code is missing because I changed my job now
I believe it's practical and not difficult, wish you good luck!