Plugin changes and breaks other fields

I'm using this plugin alongside a few others, one being "Banner".
Every time I update the banner url with the quotations for the syntax, it removes the quotes, thus breaking the url.
(ex: banner: "https://wallpapers.com/images/high/dark-academia-ireland-library-afhnkau6zvbpzu1o.webp" => banner: https://wallpapers.com/images/high/dark-academia-ireland-library-afhnkau6zvbpzu1o.webp)
My current workaround is to just save the images, but that's taxing on my storage space for the number of files I link.

Hi,
Thank you for reporting this issue. What version of obsidian are you using?
I am using an Obsidian official api to modify the front matter, so if this is causing an issue, I might have to report it back to either the banner plugin for not reading the string correctly, or the obsidian api repo.
Technically, the url is valid, even without the quotation marks

I am currently up to date with version 1.3.7 of obsidian.
I notice that it's also changing some of my formatting for titles and authors using the "Book Search" plugin, as these problems don't occur with the plugin disabled.
Example from one of the book notes:
title: "Key Person of Influence: The Five-Step Method to Become One of the Most Highly Valued and Highly Paid People in Your Industry"
title: "Key Person of Influence: The Five-Step Method to Become One of the Most   Highly Valued and Highly Paid People in Your Industry"
(A new line is added after "Most")

Same here, break cover field when it's too long

description: Use Unicode in Windows Terminal with Clion (CMake)
permalink: cpp/snippets/config/terminal_windows_unicode
created: 2023-07-24T08:17
updated: 2023-08-14T15:52
cover: 🧑🏻💻 Coding/💫 C - C++/🌟 C++/Snippets ✂/Config 🔩/_💾Assets/Pasted  image<PHONE_NUMBER>4156.png
is transformed in

description: Use Unicode in Windows Terminal with Clion (CMake)
permalink: cpp/snippets/config/terminal_windows_unicode
created: 2023-07-24T08:17
updated: 2023-08-14T15:52
cover: 🧑🏻💻 Coding/💫 C - C++/🌟 C++/Snippets ✂/Config 🔩/_💾Assets/Pasted
image<PHONE_NUMBER>4156.png

Feel free to report it on the obsidian api repo, as this is expected behavior from their side, see https://github.com/obsidianmd/obsidian-api/issues/110#issuecomment-1500356605

Read continuously reported issue with that in the obsidian forum.


Add explicit anchor ids in docs

Fixes #3888.
As a preparation for #3890.

Merging soon unless people have concerns :)

Thanks Thad for the in-depth review! I have not thought much about whether we should stick to the automatically generated anchor ids or if it is better to prettify them.
I guess it is fine to tweak them, but have you checked if the ones you changed were linked to anywhere? We would need to update the corresponding links then.

@wetneb I checked and did find links, but I did not also update those links.  (There were only a few, mostly in each same doc file referenced)

Then I'd say let's just do those changes in another PR, which simultaneously changes the anchors and updates the links, otherwise the docs will be broken in the meantime.


feat: add handling for the interface field, when an Interface field is an object that has already been instantiated, use the type of the instantiated object itself instead of interface{}.

feat: add handling for the interface field, when an Interface field is an object that has already been instantiated, use the type of the instantiated object itself instead of interface{}.
feat: add handling for the interface field, when an Interface field is an object that has already been instantiated, use the type of the instantiated object itself instead of interface{}.

feat: add handling for the interface field, when an Interface field is an object that has already been instantiated, use the type of the instantiated object itself instead of interface{}.

type BaseMeta interface {
    // Scan....
}
type Meta1 struct {
}
// meta1 Scan ...
type Meta2 struct {
}
// meta2 Scan ...
type Hello struct {
    Meta BaseMeta `gorm:"type:JSON;column:meta"`
}

// ...
// hello := &Hello{ Meta: &Meta1{} }
// db.First(hello).Error...

Hi @jinlongchen
Can you add more information about what's it used for?

sometimes, we need to store different types of data in a JSON field. however, using a json.RawMessage in code can be inconvenient. to address this issue, we define multiple types of structs which inherit from a base interface. we then use this base interface as the field type, enabling us to easily work with these structs via a type assertion without having to unmarshal the json.RawMessage.

Checkout gorm.io/docs/serializer.html, it should work for your cases.


Load Data in Power BI Desktop - Step sequencing issue

See screenshot below

@VitalyMCT  please provide more explanation of what you expect to see and what is wrong with this as is.

The highlighted block belongs to step 7 rather than after step number 8.

@AngieRudduck Thanks for the update. Since as you noted there is still a (new) issue with the Markdown I recommend either keeping this item open or opening a new issue. This is in keeping with standard GitHub conventions.


Saving optional enum (nil) triggers: [ WARNING ] bind $3 type (UNKNOWN 0) does not match expected parameter type (TEXT)

As discussed here: https://discordapp.com/channels/431917998102675485/684159753189982218/704938144323076136
When inserting nil for an optional enum:
enum Status: String, Codable {
  case ok
  ...
}

...

    @Enum(key: "status")
    var status: Status?

...

I observe the following message in the console:
[ WARNING ] bind $3 type (UNKNOWN 0) does not match expected parameter type (TEXT)

There seems to be no adverse effect and saving non-nil enum works fine.

@finestructure can you try using @OptionalEnum instead of @Enum? I'm a bit surprised @Enum is working there actually.

Sure! I’ll give that a go and report back.

Hmm yeah I'm getting this error when I try to use @Enum.
Cannot convert value of type 'EnumProperty<Foo, _>' to specified type 'Status?'

Only @OptionalEnum works for me and I'm not able to recreate the warning there.

I'll try and create a test case in the project but it'll take moment because I've now moved on to using @Enum and var status: Status with a default case none instead of optional.

FYI: I've just tried with @OptionalEnum and it works fine. I also can't reproduce the original error anymore. Not sure if it was an update in the meantime that fixed it or if something else changed 🤷
I'll close this now - thanks for taking a look! :)


Docker build issue for release 2.6.1

The dockerfile line 35 "RUN dnf install -y util-linux" complains dnf command not found during the build on Openshift. I replace with yum, and it passed.

It sounds like you're overriding default base image in your builds and not using the latest fedora based image; https://github.com/BCDevOps/backup-container/blob/master/docker/Dockerfile#L2.  Is your build still using a redhat based Postgres image as it's base?

I had this same issue when I first build the image on OpenShift and my build was still referencing the previous redhat based image which was overriding the FROM command in the docker file and replacing the quay.io/fedora/postgresql-14:14 with the previous used redhat image.

close. have to add BASE_IMAGE_FOR_BUILD=quay.io/fedora/postgresql-14:14  when process the build template on Openshift otherwise it will use centos12


New debate format for German "Jugend Debattiert"

Jugend Debattiert is a German debate format which is mainly used in secondary schools to teach students from an early age how to debate. Jugend Debattiert can be translated to "youth debating" but since it is the official name of the competition, the German name is also used internationally. The debate consists of three main parts: four opening speeches, one big free debate of 12 minutes, and four closing speeches. Participants are divided into two groups of two.
Here is an article on wikipedia about the competition: https://de.wikipedia.org/wiki/Jugend_debattiert_(Deutschland)
By submitting this pull request, I agree to license my contributions under the MIT License.

@czlee I am not sure why the checks are failing. Could you look over it?

Hi, the problem here is that where you go jd.openingSpeech or jd.closingSpeech, capital letters aren't allowed. I've left a couple comments where this problem occurs. You should be able to just click them to add them into a new commit. 😄

Ah yes, thanks @daankoning! I thought it might be that but wanted to get a computer to be sure before saying it myself 😊

fwiw I don't mind if the file name is jugend-debattiert.xml (it doesn't need the german- prefix) but if you prefer/have already distributed it with german-jugend-debattiert.xml that's cool too. Just wanted to mention the option 😎

All done, thanks so much for submitting @mirzapolat! Hopefully you can see it in the downloads list in the app now.
Thanks also to @daankoning for jumping in to advise when I wasn't available!


refactor!: logging only in test scope

The log crate is now a dev dependency. Logging
only happens when running in the test scope - cfg(test). This eliminates the risk of logging being secretely enabled by other dependencies, revealing potential secrets.

Based on https://github.com/Xuanwo/serde-env/discussions/34

@Xuanwo looks like two jobs timed out before a suitable runner was a CV available. Would you mind restarting them ? TY.

Hi, I feel like we can remove those log entirely.

That's the best solution really. I'll do that.


Design Definition for New Learn Page (Combined Courses + Programs views)

@moiz994 - this is what @sdaitzman is looking into


Filling in relations in "related()"

Let's say I have a House with Persons in it, so the House model has (using model registry):
persons: function () {
    return this.hasMany("Person");
}

And the Person model has:
house: function () {
    return this.belongsTo("House");
}

Now if I query a house and get its people:
House.forge({id:id}).fetch({withRelated["persons"]}).then(function (house) {
    var people = house.related("persons");
});

Then for each person in people, the parent house won't be fetched.
Is there a way to fetch it?
The real reason (and maybe there is a better way) is that I'm trying to write code that operates on a collection of people, but it needs to not care how that collection was obtained. So, for example:
function processPeople (people) {
    var ps = people.toJSON();
    for (p of ps) {
        // p.house needs to be valid.
    }
}

// this works, since house is fetched per person:
Person.fetchAll({withRelated:["house"]}).then(function (people) {
    processPeople(people);
});

// this doesn't work, since house is not fetched per person:
House.forge({id:id}).fetch({withRelated["persons"]}).then(function (house) {
    processPeople(house.related("persons"));
});

How can I get the above code to work in the latter case?

It appears it's not possible, so I refactored a bit, basically like:
function processPeople (people, house) {
    var ps = people.toJSON();
    var hs = house ? house.toJSON() : undefined;
    for (p of ps) {
        let p_house = p.house ? p.house : hs;
        // use p_house instead of p.house
    }
}

// this works, since house is fetched per person:
Person.fetchAll({withRelated:["house"]}).then(function (people) {
    processPeople(people);
});

// now in this case, we pass the house to processPeople()
House.forge({id:id}).fetch({withRelated["persons"]}).then(function (house) {
    processPeople(house.related("persons"), house);
});

Actually reopening this as a feature request.


Added waitDuration in the InkWell widget to wait before starting the hover animation (Resolves #149812 )

This PR introduces a new feature to the InkWell widget in Flutter, allowing developers to specify a waitDuration before the hover animation is shown. This parameter provides more control over user interactions, which is particularly beneficial for desktop platforms where hover effects are more prevalent.
Key Benefits:

Performance: Delaying the hover effect in data-intensive applications can prevent unnecessary background processing.
User Experience: A delayed hover effect reduces visual noise and distractions for interactive dashboards and gaming interfaces.
Accessibility: Introducing a delay can help users with motor impairments by preventing accidental hover triggers, making the application more accessible.
Context-Specific Interactions: In scenarios like educational tools or e-commerce websites, a delayed hover effect ensures that additional information or actions are only shown when the user is genuinely interested.

Changes:

Added waitDuration property: This property allows developers to set a duration that a pointer must hover over the InkWell widget before the hover animation is triggered.
Updated _InkResponseState class: Modified the state management to incorporate the waitDuration functionality, ensuring the hover animation respects the specified delay.

Fixes #149812.
Example Usage:
InkWell(
  onTap: () {
    // Handle tap
  },
  waitDuration: Duration(milliseconds: 500), // Hover animation will show after 500ms
  child: Container(
    padding: EdgeInsets.all(16.0),
    child: Text('Hover over me!'),
  ),
);

Video:
https://github.com/user-attachments/assets/a8cf46e8-80c5-4d65-9b33-24e48f8847c5
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

@TahaTesser After watching the bounty hunter videos, I've decided to contribute. Thanks for the amazing guidelines. Would you please check this PR when you have time?

@TahaTesser After watching the bounty hunter videos, I've decided to contribute. Thanks for the amazing guidelines. Would you please check this PR when you have time?

Thanks for contributing!

This enhancement provides more control over the user interaction experience, which is handy for scenarios where immediate hover feedback is not desired.

Could you please some real world use cases for this?  i don't see any convincing explanation in the issue proposal.
Before adding another new parameter we should some background.

Could you please some real world use cases for this? i don't see any convincing explanation in the issue proposal.
Before adding another new parameter we should've some background.

@TahaTesser the use cases for this variable are the same as the tooltip. Hovering over a dashboard with many buttons can become visually overwhelming if every button responds immediately to hover events. A delayed hover effect can help reduce visual noise

@momshaddinury Is there any framework or app that has this delayed hover capability?

@momshaddinury Is there any framework or app that has this delayed hover capability?
I'm curious how useful is this?

@TahaTesser yes, your queries makes sense. I found react native to have similar functionality.
Here is the link: https://www.npmjs.com/package/react-hover-observer

@TahaTesser yes, your queries makes sense. I found react native to have similar functionality.
Here is the link: https://www.npmjs.com/package/react-hover-observer

This is a small third-party package with last update from 6 years ago. Not a framework feature.
I'll leave the PR as it is for now. Let the Flutter team triage this.

@TahaTesser yes, your queries makes sense. I found react native to have similar functionality.
Here is the link: https://www.npmjs.com/package/react-hover-observer

This is a small third-party package with last update from 6 years ago. Not a framework feature.
I'll leave the PR as it is for now. Let the Flutter team triage this.

Alright, thanks.

Hey @momshaddinury thank you for contributing! Looking into this in triage today, I am not sure there is enough need to justify adding this to the framework. Can we discuss further in the issue and see if more folks can enumerate use cases for this?

Thank you for your contribution. I'm going to close this PR for now since we have not heard back in a while, and we aren't sure we want to add this to the public API. Please feel free to follow up in the linked issue above. Thanks!


Add outputVerbosity into configuration

Discussion from https://github.com/SimplrJS/scss-bundle/pull/28

Maybe adding yet another configuration point, e.g. outputVerbosity would do?
The CLI gets a bit cluttered, but I suppose if you're using all of the options, you either use the configuration file or consume scss-bundle via TypeScript either way.

PR: https://github.com/reactway/scss-bundle/pull/65


python312Packages.tencentcloud-sdk-python: 3.0.1152 -> 3.0.1153

Diff: https://github.com/TencentCloud/tencentcloud-sdk-python/compare/refs/tags/3.0.1152...3.0.1153
Changelog: https://github.com/TencentCloud/tencentcloud-sdk-python/blob/3.0.1153/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

python311Packages.tencentcloud-sdk-python
python311Packages.tencentcloud-sdk-python.dist
python312Packages.tencentcloud-sdk-python
python312Packages.tencentcloud-sdk-python.dist


terra-modal-manager is not responsive in Edge browser

Bug Report
Description

terra-modal-manager is not responsive in Edge, but it works fine in chrome and firefox. I encountered this issue while running wdio tests for my component.
Steps to Reproduce

Open the terra-modal-manager examples in Edge browser.
Resize the window to mobile view.
Click on the Disclose(Large)

Additional Context / Screenshots

Expected Behavior

Environment

Browser Name and Version: Edge
Operating System and version (desktop or mobile): Mobile

I believe this is a long known issue with edge:
https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/14375647/

With edge converting to chromium we should re-evaluate this issue.

I tested it with Microsoft Edge 42.17134.1.0 and I don't seem to see the issue.

@dkasper-was-taken, I think we can close this issue.

Closing. as we are not able recreate this issue


Error in list item

Can't bind to 'item' since it isn't a known property of 'list-item'.

If 'list-item' is an Angular component and it has 'item' input, then verify that it is part of this module.

'list-item' must a custom build angular2 component, outside of this library. A sample list item is give below or check the demo app for list-item.component.ts
import { Component, Input } from '@angular/core';

export interface ListItem {
    index?: number;
    name?: string;
    gender?: string;
    age?: number;
    email?: string;
    phone?: string;
    address?: string;
}

@Component({
    selector: 'list-item',
    template: `

{{item.index}}

{{item.name}}

                {{item.age}} / {{item.gender}}
                {{item.email}} | {{item.phone}}

{{item.address}}

    `,
    styleUrls: ['./list-item.scss']
})
export class ListItemComponent {
    @Input()
    item: ListItem;
}

What if we want to achieve this using div tag???

Got it . Please include what the list-item in the documentation so that there is no confusion. Thanks

I have added to this to documentation. Thanks for bringing this to my notice.
Yes, you could also use a div tag if you like to do so.
<virtual-scroll [items]="items" (update)="viewPortItems = $event">

{{ item }}

</virtual-scroll>

let items[] = ['Item1', 'Item2'];

If we give
{{ item }}
we will still get the following error
"Can't bind to 'item' since it isn't a known property of 'div'. "
so simply
{{item}}  is enough. This worked for me

@rintoj Hi, I did what you mentin below and it still doesn't work. I tried to move (update) expression to another method in my .ts file but it seem my app can't reach the update event


Scrolling stops on first scroll

On new 116 Kiwi
Pixel 5
Pixel 3a
Ta S8
Sample url https://www.economist.com/china/2023/06/01/china-goes-from-zero-covid-to-zero-restrictions
If you move the page to scroll down the screen locks and you have to release and do it again

Still happening on 5845.27


Add support of docker "swarm mode"

Roadmap:

implement docker DTOs (possibly already finished)
extends DockerService for new api methods
make new cluster kind and do some refactoring. This feature allow to connect for different nodes for orchestration, but current RealCluster does not support this

Investigation show that we need to wait docker 0.13 release due to https://github.com/docker/docker/pull/27910


Help connecting to known devices

Hi folks,
I've had much more luck since moving from the CSR dongle to the RPi3 internal adapter (yay).
What I'm seeing now is more of a timing issue that may be related to my newness to Node-style coding.  I'd like some help getting this done right, and I'll be happy to write up a guide in the Wiki when it's working.
I have 8 devices I'm working with: 7 BLE bulbs and an adafruit Bluefruit chip (which is irrelevant here).  I know all of their UUIDs and I have logical Ids (positions in the room) that map to them.  Here's my current approach:

Start Discovering at process startup
In on("discover) I connect(), then discoverServices() to see if it has a bulb Characteristic and, if so, I store the Peripheral & Characteristic objects in a map and set that entry to "valid".
I set a timer to periodically adjust the lights (per logic that's not relevant) every 200ms
When I see I have all the devices, I stopScanning().

What I'm seeing is that if I do the logic in #2 above, I miss the discovery event for some of the lights.  So I am re-working this to JUST store the peripheral object in the map and then loop through those after I have all 7 bulbs.  However, I'm finding that EVEN THAT connect() call is enough to make me miss discover events.
Given that I have the addresses, is there a better approach?  I don't see a way to connect() to peripherals by UUID (unless I'm missing something), and I LIKE the discover() approach because it would work well if the lights are turned off and on (they'd be removed & replaced in the map).
Sample code is attached... it relies on onecolor, but that could easily be removed
When the connectAndSetup() call (line44) is commented out, I get this output:
pi@pi3:~/fadecandy/examples/node $ node test.js
State Changed: poweredOn
discovered: b4:99:4c:55:38:d1
discovered: b4:99:4c:64:85:26
discovered: b4:99:4c:64:84:cb
discovered: b4:99:4c:64:83:f4
discovered: b4:99:4c:64:7d:bd
discovered: b4:99:4c:64:85:01
discovered: b4:99:4c:64:83:35
discovered: f2:bb:cf:78:2c:85

When I uncomment it, I get this:
Cpi@pi3:~/fadecandy/examples/node $ node test.js
State Changed: poweredOn
discovered: b4:99:4c:64:7d:bd
discovered: b4:99:4c:55:38:d1
discovered: b4:99:4c:64:83:f4
Connecting to b4:99:4c:64:7d:bd
null
Found a Bulb - b4:99:4c:64:7d:bd
checking for the map
Connecting to b4:99:4c:55:38:d1
null
Found a Bulb - b4:99:4c:55:38:d1
checking for the map
Connecting to b4:99:4c:64:83:f4
null
Found a Bulb - b4:99:4c:64:83:f4
checking for the map
#the output stops here

It looks like if that the first attempt to connect causes the scanning/disconnect to fail: I've attached the hcidump output of that second attempt.
Any thoughts here?  Again, this seems like a usage that will be pretty common - people who know what peripherals to expect and want to configure them on discovery.
Thanks and again, I'll be happy to write
test.js.txt
hcidump.out.txt
up what the end result is as a guide.
Bill

After adding lots of safety checks and JUST saving the peripheral, waiting until all had been connected,  I have this connected to 6 of 7 90+% of the time. and 7 of 7 50% of the time.  When it fails I see this message in hcidump:
> HCI Event: Command Status (0x0f) plen 4
    LE Read Remote Used Features (0x08|0x0016) status 0x00 ncmd 1
> HCI Event: Disconn Complete (0x05) plen 4
    status 0x00 handle 68 reason 0x3e
    Reason: Connection Failed to be Established

Now, some googling tells me that that error 0x3e can "just happen" sometimes (yuk).  However, don't see this error being returned (all 'error' parameters are null), so I don't know how to retry this...
Any help would be appreciated.  I've included the new code for reference
Bill
test.js.txt

Given that I have the addresses, is there a better approach? I don't see a way to connect() to peripherals by UUID (unless I'm missing something), and I LIKE the discover() approach because it would work well if the lights are turned off and on (they'd be removed & replaced in the map).

This is the right approach, noble is based on the OS X/iOS CoreBluetooth stack which requires scanning + discovering prior to connecting.

Any thoughts here? Again, this seems like a usage that will be pretty common - people who know what peripherals to expect and want to configure them on discovery.

I would suggest trying out calling noble.startScanning(...) again after the connect is done so that any devices that were missed can be discovered again.

Now, some googling tells me that that error 0x3e can "just happen" sometimes (yuk). However, don't see this error being returned (all 'error' parameters are null), so I don't know how to retry this...

Please provide a full HCI dump to share more context.

I'm closing this due to lack of activity. If you are still interested please re-open with the requested info.


Translate advanced settings to Ukrainian

As of now, advanced/extended settings' text is still in English when Ukrainian is selected. This PR fixes that slight issue.

Plz resolve conflicts, and PR only to 0.0.x, docs into docsimprovement

I see it's already translated there, closing this PR then.


switching to herokuish. closes #1091

naive switch to herokuish. @josegonzalez need help with the packaging stuff. maybe just drop it since its a simple docker pull now?

What do you mean by that? I'm on irc if it helps :)

@progrium any reason you don't tag versions in docker hub? looks like the latest tag is the release branch but it would be nice to be able to pin to a specific version.
https://registry.hub.docker.com/u/gliderlabs/herokuish/builds_history/172630/

I think that we can drop the deb package for herokuish

tests are failing on building herokuish from master.
Start building herokuish
Cloning into '/tmp/herokuish'...
remote: Counting objects: 905, done.
remote: Compressing objects: 100% (5/5), done.
remote: Total 905 (delta 0), reused 0 (delta 0), pack-reused 900
Receiving objects: 100% (905/905), 990.77 KiB | 0 bytes/s, done.
Resolving deltas: 100% (334/334), done.
Checking connectivity... done.
Sending build context to Docker daemon 44.54 kB

Sending build context to Docker daemon

Step 0 : FROM heroku/cedar:14
Pulling repository heroku/cedar

INFO[0000] Error pulling image (14) from heroku/cedar, Driver btrfs failed to create image rootfs 511136ea3c5a64f264b78b5433614aec563103b4d4702f3ba7d4d2698e22c158: Failed to create btrfs subvolume: file exists

make: *** [stack] Error 1 ./tests/ci/parallel_runner.sh returned exit code 2

Awaiting a response from CircleCI.

It can't be automated. What if we tried not pinning? Heroku doesn't pin
their environment.
On Tuesday, April 21, 2015, Michael Hobbs<EMAIL_ADDRESS>wrote:

@progrium https://github.com/progrium any reason you don't tag versions
in docker hub? looks like the latest tag is the release branch but it would
be nice to be able to pin to a specific version.
https://registry.hub.docker.com/u/gliderlabs/herokuish/builds_history/172630/
—
Reply to this email directly or view it on GitHub
https://github.com/progrium/dokku/pull/1128#issuecomment-94594704.

--
Jeff Lindsay
http://progrium.com

I am wondering if we should really rename the plugin hooks to include herokuish ??
This will force many users to update the plugins ...
Should we either keep the naming of buildstep and just change the image? Or if we already rename the hooks, can we rename them so they are not dependant on the docker image name ??

I'm fairly ambivalent on the hook name. @josegonzalez thoughts?

@progrium It would make debugging issues much easier - oh you got this working in release 0.0.5 and 0.0.6 doesn't work? - if we pinned. Even if it's just "version 1" versus "version 2".
@michaelshobbs changing hook names here doesn't make sense, unless our next release is going to be 0.4.0. Which I'm fine with, just saying. We may want to hold off if we're going to move to buildkit though, since thats yet another name change :)

I do think dokku would probably be best served to push this change in the next major release. However, if we're going to rename the hook, we should probably only do it once and name it something like -buildpack instead as that doesn't attach it to an underlying image name and is more reflective of what we're actually doing. i.e. buildpack vs. Dockerfile

I do agree - when it comes to plugins and hooks let`s create a feature outline for the major release and merge everything related to that together. So we break it only once

@progrium ping on tagging releases?

@progrium how would i go about building the herokish docker image from master as opposed to the release tgz referenced in the Dockerfile?

I have a tool now that programmatically adds tags for versions on Docker
Hub for Automated Builds from CI, but I haven't had time to add it.
On Tue, Jun 2, 2015 at 7:30 PM, Michael Hobbs<EMAIL_ADDRESS>wrote:

@progrium https://github.com/progrium how would i go about building the
herokish docker image from master as opposed to the release tgz referenced
in the Dockerfile?
—
Reply to this email directly or view it on GitHub
https://github.com/progrium/dokku/pull/1128#issuecomment-108140114.

--
Jeff Lindsay
http://progrium.com

@progrium it would also be cool to have a nightly build (based on master) for testing with dokku; similar to how we were testing with buildstep.

Yeah, I'm going to make sure herokuish has master, latest, and specific version tags.

@michaelshobbs can you rebase this and also change the hooks to say buildpack instead of herokuish?

@josegonzalez where do you want that changed?

currently there is no way to actually build herokuish from master because the dockefile is pinned to a release. i'm going to take a stab at a PR for that now.

references https://github.com/gliderlabs/herokuish#35

refs: https://github.com/gliderlabs/herokuish/pull/35

Per IRC convo, we'll call the old pluginhooks as well for a patch release and then deprecate in 0.4.0.

What else does this depend upon for being released? Do I need to update our release process to do herokuish now?

What are you referring to wrt the release process?

Well, do I need to make a deb package of herokuish now?

Yeah I think so

I think this is ready to go pending tests.

I don't think this is bc-break anymore

@josegonzalez ping. is the debian side of this ok? i don't know what i'm doing there. can you double check it before we merge?

@josegonzalez the release branch of gliderlabs/herokuish needs to be merged to master to make the tests pass

You mean we need a new release? The release branch has already been merge.

Hrm. We did something wrong then.
https://github.com/gliderlabs/herokuish/pull/52

@progrium we'll need to get the docker hub image built. can i get contributor access to gliderlabs/herokuish and docker hub access as well? (would be good if @josegonzalez had access as well)

what is @josegonzalez's docker hub user?

catmilla is my docker hub user (maybe josegonzalez, but lord knows docker makes it difficult to figure that out).

Seems legit with ^ fix :) . I'll make one more small release and then merge this in for a new release!


Github sponsorship detection requires public sponsorship

Prerequisites

[X] I'm using latest version: v1.3.0
[X] I've checked Known issues
[X] I've checked Open issues

Description
When sponsoring, the app is unable to detect the sponsorship if 'Private' is selected. Changing to 'Everyone' and restarting the app detects the sponsorship. Would suggest adding this to the README, I got stuck on it for a while.

Hi @bharrisau and thanks for the report. Github API actually allows me to also get private ones with includePrivateso I'll do that

Should work now :)


Errors in CT Spreadsheet

During use of the CT spreadsheet noticed that:

#Row 154 onwards in the Role column you start using “Class” instead or “Entity”.
Address entity Logical Data Model name is AliasCode, should be Address.

Also #73


feat(cli): Oauth tokens plain text fallback

Linked issues

Fixes #74

Changes

CLI

Fallback to storing access + refresh OAuth tokens in plain text if the device doesn't have keyring capabilities

Additional comments
Testable with release v0.4.0-alpha.3

Ignore the checks' failure. I initiated the PRs targeting main by mistake.


Port messaging extension - configure settings from C# to JS

Port message extension - configure settings
https://github.com/microsoft/botbuilder-dotnet/tree/master/tests/Teams/ComposeMessagingExtensions

This is very similar to the action based extension scenario. I'm not sure who did that one, but it should be fairly straight forward to do. The manifest in this one does have 1 difference:
"canUpdateConfiguration": true,

I'll do this one.

Done.

Closing: pr submitted, awaiting review https://github.com/microsoft/botbuilder-js/pull/1265


Update to channels 0.16.x

Fixes #407

By the way I am aware of the misspelled branch name, I just don't know how to rename it without breaking anything

By the way I am aware of the misspelled branch name, I just don't know how to rename it without breaking anything

That's fine, it won't be mentioned in the merge commit.


Switch to pip and update to 0_2_2

Checklist

[ ] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

In the future will use a fork, but the package has already been published so will merge.


Use ccache

Includes https://github.com/skvark/opencv-python/pull/146 and is supposed to be applied on top of it.
Includes a workaround for https://github.com/skvark/opencv-python/issues/148 (which won't be needed once it's solved).
Ccache brings rebuild time from 15 to 5 minutes (for a regular build).
The ccache stats printed at the end are aggregate stats -- since the creation of the cache. I wasn't sure if zeroing them at the start would be more useful.

I think the aggregate stats are ok.

Great! Travis should be now trying to make exactly 3 builds in master, so after they finish, we should have caches for everything. Even manual restarts won't be needed if there are no irrelevant errors.

Thanks for all the hard work again!


Add hardware metrics for cisco (fan)

What does this PR do?

See title
Motivation

NDM-2294
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached
[ ] If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

I was thinking about just factoring out this testing logic (it has to be present due to changes in _cisco-generic.yaml/snmprec), which is the same for all profiles where the profile extends _cisco-generic (or any other profile that itself extends _cisco-generic). Like adding assert_extend__cisco_generic (and similar) to utils.py. What do you think ?

I was thinking about just factoring out this testing logic (it has to be present due to changes in _cisco-generic.yaml/snmprec), which is the same for all profiles where the profile extends _cisco-generic (or any other profile that itself extends _cisco-generic). Like adding assert_extend__cisco_generic (and similar) to test_e2e_core_profiles/utils.py. What do you think ?

you don't have to test the metrics for every profile that extends _cisco-generic.yaml (directly or indirectly)
having the vendor metadata cisco present in the payload is enough to test the extend

you don't have to test the metrics for every profile that extends _cisco-generic.yaml (directly or indirectly) having the vendor metadata cisco present in the payload is enough to test the extend

But what is the purpose of all those functions in test_e2e_core_profiles/utils.py and how do we know which metrics are needed to be checked anyway (and added to snmprec) ?

Indeed, as Nouha suggestion. We mainly want to assert that we indeed extended the profile (e.g. assert a single metric or a metadata), but don't need to assert all metrics from the extended profile.

But what is the purpose of all those functions in test_e2e_core_profiles/utils.py

The assert_extend_* functions test_e2e_core_profiles/utils.py are here to make it easier to assert extends. You just need to include specific OID in snmprec and call the assert_extend_*() function.

how do we know which metrics are needed to be checked anyway (and added to snmprec) ?

Those are just some arbitrary metrics from the extended profile.
It's just usually the most simple metric possible from that profile.
The OID to be added to snmprec correspond to the OIDs necessary to generated the expected metrics.

In this PR, you can probably only tests the new metrics in test_profile__cisco_generic.py, remove the assertions/snmprec entries for the other profiles :)

yes, but for all the profiles that extend _cisco-generic, there are no such functions. That's why I was thinking about adding one.

@lizaerem Yes, you can possibly add a assert_extend_cisco_generic() to test one of the metric from _cisco-generic.yaml


attention layer weights in DIN

Hi, Could you please show me how to get the weights of SequencePoolingLayer to visualize attention?
Given a test sample, I am interested in seeing what items (keys_emb_list) had the biggest impact on the target item (query_emb_list) @shenweichen

any idea how to solve this problem?

set return_score=True in AttentionSequencePoolingLayer can get the attention score of DIN

I am also interested in getting the attention layer weights. I have tried setting return_score=True. However, I am still not seeing anything returned by calling model.get_layer('attention_sequence_pooling_layer').get_weights()? Do you mind sharing some code snippets to extract attention weights? Thanks!


Adding active state to navigation links?

Is there a way to add active state to links on a one page site?

@curvedb, please see issue #64


Error adding GPG Key

Description
When I go to add my public gpg key at https://gitea.domain.rocks/user/settings/keys I'm getting this error:

An error occurred:
AddPublicKey, Error 1264 (22003): Out of range value for column 'expired_unix' at row 1

Gitea Version
1.20.1
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
2.40.1, Wire Protocol Version 2 Enabled
Operating System
Official Docker container
How are you running Gitea?
Official Docker container
Database
SQLite

Thanks for this report, could you share your public key that you uploaded?

@techknowlogick  Sure thing:
-----BEGIN PGP PUBLIC KEY BLOCK-----

mDMEZLxUJhYJKwYBBAHaRw8BAQdAPC+7z3gBawAIuttYV4fbjwlgqBsC782r5TUP
8i9JDv20JUouIFNjb3R0IEVsYmxlaW4gPHN0YXJkb2dnQGdtYWlsLmNvbT6IkwQT
FgoAOxYhBBbsV4Q3umu57TrprgSKH0n69vOCBQJkvFQmAhsjBQsJCAcCAiICBhUK
CQgLAgQWAgMBAh4HAheAAAoJEASKH0n69vOCdmQBANo460PNWuxgd079E5eWsY/c
nX8xevEAP7UFkPzvOOckAP0SiTdae7kZUz6dLKuFopXyiayzNnYVfjYzfSd3twva
Bbg4BGS8VCYSCisGAQQBl1UBBQEBB0B6oqJZgzL5aDCn8nW+XBNQSQraYmr91BNS
xFVD5PWbEgMBCAeIeAQYFgoAIBYhBBbsV4Q3umu57TrprgSKH0n69vOCBQJkvFQm
AhsMAAoJEASKH0n69vOCkTgBAPtlGp4LsiI/qDSDkFyFYMvYb9P42c6RdxA8vqj9
xi5MAP96F64ZxRjYja3YfCHZaE6rowKcaIefQSml8o6twQGyAA==
=ZlkT
-----END PGP PUBLIC KEY BLOCK-----

Oops, just realized that I had forgotten that I was successful at switching the db from sqlite to my mariadb server. Updated the OP.

If it helps any, here's my table:

Maybe those columns should be bigint but not int(11). How did you convert the database?

@lunny
You the man. =)
Changed it to bigint and re-tried to enter my key and it worked. I wish I could remember exactly how I did the conversion but it was awhile ago. I vaguely think I might remember the Gitea docs had some sort of instruction how to do it?
One questions, as I'm not the hugest SQL guru; when I go to change the column to bigint here:

and delete the "11" from the Length column, leaving it blank; then click Save, it ends up saving as bigint(20) rather than just 'bigint`. Any idea why? And is that fine, or?


GetPlatformVersion

Please add the GetPlatformVersion Version again.
Formerly this function was located in the Tenant Management Codeunit.

@JesperSchulz

Hi @ChrisBlankDe ,
The GetPlatformVersion returned a very technical term about the platform build number used internally by MS. The only place it was actually used is page 1920 "Container Sandbox Environment", where the function has been moved to as a Local function. We would like to know the justification for you to use it. If you simply wish to have an application identifier, would it be enough for you to use any of the procedures defined in either of the two APIs:

codeunit 457 "Environment Information"
codeunit 417 "Tenant Information"

These can be found in the module Environment Information.
Best regards.

@DuttaSoumya

Hi @ChrisBlankDe ,
What I meant was that the user cannot do much about, in most cases does not even understand, the version details for the platform. In case of error, the application service for the user can be pulled up by the troubleshooter, and the app service always runs on a specific app + platform version.
One more thing, could you please point me to the help topic on this procedure- we may have overlooked it when taking this function out.
Cheers,
Dutta.


Display the commit hash when showing gauge version

To identify the commit that built the release without having to check ci builds display the commit sha in the gauge version command
Expected behavior
$ gauge version
~> gauge version
Gauge version: 0.9.0.nightly-2017-07-05 be81241

Plugins
-------
html-report (4.0.1)
java (0.6.4)

Actual behavior
The commit hash is not displayed.

related #584, which is waiting for golang 1.9 (see comments).
All the more important when we support #698
Also, should we include which branch it was built?

Oh  didn't see that.
I can close this in favor of  #584 only that we always show a commit from wherever it is built.
Can't the sha be used to identify the branch?


Added Image Download Button

Description
Once all the operations are processed, an image download window is popped again and again. Instead, a download button is added so the user can download the processed image without executing the operations again.

Feature: #118

Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Also, include screenshots for verification and reviewing purpose.
https://user-images.githubusercontent.com/19731056/224710947-d4c94eb1-17b7-4ff8-beac-ef8ea573d18d.mov
Checklist:

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

i wanna work on this issue

@abdullahranginwala When we are executing the pipeline it always shows the image save option. So by this issue what we are expecting is removing that image save option dialog and adding a separate image download option. So your approach is fine. But need some modifications to do. Add that image download button under the preview panel.
As well as don't try to fix several issues in one PR. I see that you have added undo and redo functions also. Try to fix one issue with one PR.
Thank you!

Thank you for your feedback! I have already made a PR for the undo and redo function which has been merged. There are some sync issues with my local repository. I'll fix that and implement the other feedback as well

@kaveeshadinamidu I've implemented the feedback but I'm not sure how I can remove the 'undo/redo feature' commit. The PR for that particular issue has already been merged into the master branch but it got added here due to some merge conflicts. I've tried to clean up and fix the conflict. Would you recommend that I squash all the commits or is this good?

I want to  work on this issue.


[FEAT]: Defining structure (properties) of a model attribute of type Object

First of all, thank you for the amazing library, it helped me a lot with enforcing the single table design pattern in DynamoDB! :)
I was wondering if you had any thoughts on adding support for defining a structure of a model attribute that has a type of an Object. Specifically, I have a property in my model defined as follows:
location: { type: Object, required: true }
What (in my case) would be very helpful feature is to be able to define the properties of that location object and then have those properties added to the TypeScript type by the library. For example I would want to add those properties to my location object:
zip: { type: String, required: true }
state: { type: String, required: true },
city: { type: String, required: true },

Would this be something that's useful and possible to implement?

Thanks for the feedback. All suggestions to improve are really welcome.
Your suggestion is definitely useful. We're taking increased TypeScript support for things like this step by step and are keen to continue to strength this support. However TS only has so much support and we've not yet stumbled on the solution. If you know of a real TS typing wizard, let us know.
Ideally, we'd like TS types for all nested properties regardless of depth. Still working on how to do that and not sure if that is possible yet.

This issue is really closely related to #76. I'm going to close this issue and deal with it via #76.


nav bar disappears when changing route

Got the basename solution from https://github.com/ReactTraining/react-router/issues/7128.  With the basename in the Routes element, the  element will disappear after clicking on the nav link.  We need a way to have the nav receives a basename and stays on the page after changing route.   In router 5.0, we can do that by setting basename in the BrowserRouter level.   Is there a way to make the nav bar stays after changing route and applying the basename on the link in 6.0?
// url where new router is created: https://my-site/who/users
const RootModule = () => {
  return (
    <main>
      <BrowserRouter>
        <Routes basename="who/users">
          <nav>
            <Link to="">Home</Link>
            <Link to="who/users/about">About</Link>
            <Link to="who/users/users">Users</Link>
          </nav>
          <Routes>
            <Route path="/" element={<Home />} />
            <Route path="who/users/about" element={<About />} />
            <Route path="who/users/users" element={<Users />} />
          </Routes>
        </Routes>
      </BrowserRouter>
    </main>
  );
};

Thanks.

No as of now, this is a dupe of #7216.


Add query validation for GraphQL Interfaces through Ballerina Distinct Service Classes

Description:
$title

Closing since this is no longer valid issue


calendar width collaps

is only your module displayed on the screen the broad collapses. you should use "min-width: 240px;" in your css sheet to catch this error.
regards

+1.  In fact, the rounded corners get clipped up until ~275px.  I wound up putting a min-width: 300px; in my custom.css.


Fixed #1129 - missing validation for folder plugin when folder field are empty

Based on issue #1129

Fixed in #1165


Resource deletion should break test case when a resource reaches the 'DELETE_FAILED' state

Changes:

enforce delete yarn environment from test to be able to identify yarn environment deletion issues
enforce TestFailException in case of DELETE_FAILED status during resource terminations

I think the integration-test/src/main/java/com/sequenceiq/it/cloudbreak/testcase/e2e/hybrid/AwsYcloudHybridCloudTest.java tearDown is not necessary anymore if we move back the:
.given(CHILD_ENVIRONMENT_KEY, EnvironmentTestDto.class, CHILD_CLOUD_PLATFORM)
.when(environmentTestClient.cascadingDelete(), RunningParameter.key(CHILD_ENVIRONMENT_KEY))
.await(EnvironmentStatus.ARCHIVED, RunningParameter.key(CHILD_ENVIRONMENT_KEY))

block to the test.


asterisk issue fixed.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Exchange-Server-2013/connect-or-restore-a-deleted-mailbox-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/create-user-mailboxes-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/manage-permissions-for-recipients-exchange-online-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/remote-domains-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/review-the-voice-mail-calls-in-your-organization-exchange-2013-help.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Exchange-Server-2013/connect-or-restore-a-deleted-mailbox-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/create-user-mailboxes-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/manage-permissions-for-recipients-exchange-online-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/remote-domains-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/review-the-voice-mail-calls-in-your-organization-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/track-messages-with-delivery-reports-exchange-2013-help.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Exchange-Server-2013/connect-or-restore-a-deleted-mailbox-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/create-user-mailboxes-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/manage-permissions-for-recipients-exchange-online-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/remote-domains-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/review-the-voice-mail-calls-in-your-organization-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/track-messages-with-delivery-reports-exchange-2013-help.md
:white_check_mark:Succeeded
View

Exchange-Server-2013/unified-messaging-exchange-2013-help.md
:white_check_mark:Succeeded
View

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


⚠ edjufy Backend has degraded performance

In d1d8558, edjufy Backend ($EDJUFY_BACKEND_SYSTEM_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 2331 ms

Resolved: edjufy Backend performance has improved in 45477e1 after 7 minutes.


Does this lib works with React Native ?

Hi guys
I am working on mobile app development ,
I need a stripe payment
Does this lib works with React Native ?

Hi there! Sorry for the delay in response. Please see this response: https://github.com/stripe/react-stripe-elements/issues/21#issuecomment-307163466


Web interface to update site content

The web interface accessible while the development server should allow for the dynamic creation of new content.
Perform CRUD (Create, Read, Update, Delete) operations against an Event

[x] Add new events to the database from the web interface
[x] Update existing events in the database from the web interface
[x] Delete an event from the database from the web interface

Perform CRUD operations against a Project

[x] Add new projects to the database from the web interface
[x] Update existing projects in the database from the web interface
[x] Delete a project from the database from the web interface

Closes with #13 .


tf.keras.layers.Add can add tensors with dimension ()

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0-dev20190731
Python version: 3.6.7
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
when I add multiple tensors with shape (), tf.keras.layers.Add complains that

File "/home/xieyi/.local/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 680, in call
self._maybe_build(inputs)
File "/home/xieyi/.local/lib/python3.6/site-packages/tensorflow_core/python/keras/engine/base_layer.py", line 1905, in _maybe_build
self.build(input_shapes)
File "/home/xieyi/.local/lib/python3.6/site-packages/tensorflow_core/python/keras/utils/tf_utils.py", line 299, in wrapper
output_shape = fn(instance, input_shape)
File "/home/xieyi/.local/lib/python3.6/site-packages/tensorflow_core/python/keras/layers/merge.py", line 95, in build
batch_sizes = [s[0] for s in input_shape if s is not None]
File "/home/xieyi/.local/lib/python3.6/site-packages/tensorflow_core/python/keras/layers/merge.py", line 95, in
batch_sizes = [s[0] for s in input_shape if s is not None]
IndexError: tuple index out of range

Describe the expected behavior
I expect tf.keras.layers.Add can process tensors of any dimension.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
#!/usr/bin/python3

import tensorflow as tf;

a = tf.constant(1);
b = tf.constant(2);
c = tf.constant(3);
d = tf.keras.layers.Add()([a,b,c]);

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

I have tried on colab with TF version 2.0.0-dev20190731, 2.0.0.dev20190915 ,2.0.0-rc1 and was able to reproduce the issue.Please, find the gist here.Thanks!

@breadbread1984 I think this was resolved in recent tf-nightly. I ran it without any issues. Please take a look at the colab gist.
Can you please verify once and close the issue if this was resolved for you. thanks!


Driver independence

This is a merge of SnabbCo#1234 to lwaftr, resolving a minor conflict in snabbvmx's setup.lua.

I have tested this on snabb1 and verified that it hasn't broken our lwaftr use cases, and that other tests pass.  Fingers crossed for the bots!

The build failure appears to be a flaky property-based test, unrelated to the contents of this patch.  Merging!


Fix kGPIO_GetKey constant at zkvm/platform/io.h

The constant looks suspicious and its value doesn't match to correspondent constant in io.rs

@jbruestle Is it OK if we drop this GPIO entirely? I don't think any of the new examples use this feature anymore.

The new zkvm implementation uses ecalls rather than GPIO. The GetKey functionality has also been dropped.


Performance with five repeats

The experimental result in your article is the average of 5 repeats, but there is no trace of 5 repeats in the code. I tried your model initially, and I found that every time the performance was very unstable and the variance was extremely large, what could be the reason for this?
For example, In res14 dataset, sometimes the result is F1:0.70368, sometimes F1:0.41351

hi @BD-MF, thanks for your attention. The reason may be that the saved best model is overwritten by other experiments when you running several experiments at the same time. So you can rename the saved model with different datasets and rounds as the prefix.

The problem has been solved, thank you very much.

You're welcome.

I can not find parameters about random seeds, maybe this can lead to unstable results as well.


Regression with latest Amplify version for Auth and API

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
I use OAuth authentication and HostedUI on my application.
With the "aws-amplify" and "aws-amplify-react" latest versions, a "signIn" event sent to the Hub component and the rest of the application after OAuth authentication is not sent anymore,  as it was with the following versions:

"aws-amplify": "^0.3.3",
"aws-amplify-react": "^0.1.43",

Latest versions that not work for me:

"aws-amplify": "^0.3.4",
"aws-amplify-react": "^0.1.47",

In addition, when simulating the event by testing the userSession, it happens that the API component is not anymore able to connect to AppSync with the Connect component. I guess it a consequence of the first issue but I have no clue.
When I downgrade to  "aws-amplify": "^0.3.3" and aws-amplify-react": "^0.1.43", everything is working well.
I didn't have the time analyse deeper the issue and I prefer to referenced quick because it seems to be a blocking one for those who use HostedUI.
What is the expected behavior?
After signin, token parsing and session loading, AuthClass should fire a SignIn event as it was before and Auth and API configuration should be loaded properly to enable AppSync API call.
Which versions of Amplify, and which browser / OS are affected by this issue? Did this work in previous versions?
Latest versions that not work for me:

"aws-amplify": "^0.3.4",
"aws-amplify-react": "^0.1.47",

You can turn on the debug mode to provide more info for us by setting window.LOG_LEVEL = 'DEBUG'; in your app.

@OlivierPT could you try<EMAIL_ADDRESS>as the latest version has some fix around Auth.

Hi,
Unfortunately, there is no change with<EMAIL_ADDRESS>:(
I use the Hub component to listen to "auth" events as mentionned in the documentation:
Hub.listen('auth', this, 'SignInAWS');`
and then:
`onHubCapsule(capsule) {
logger.debug('capsule:', capsule);
const { channel, payload } = capsule;
if (channel === 'auth') { this.onAuthEvent(payload); }
}
onAuthEvent(payload) {
const { event, data } = payload;
logger.debug('data:',data);
switch (event) {
case 'signIn':
logger.debug('user signed in');
this.props.onStateChange('signedIn');
break;
case 'signUp':
logger.debug('user signed up');
break;
case 'signOut':
logger.debug('user signed out');
this.redirectToHostedUI();
break;
case 'signIn_failure':
logger.debug('user sign in failed');
this.redirectToHostedUI();
break;
default:
logger.debug('default');
logger.debug('event:',event);
}
}
`
But I never receive any event despite the use is signed (I can get UserInfo in parallele).
Do I need to change something after the update?

@OlivierPT are you using withAuthenticator? I noticed if you are using that hoc to wrap your app component, then this app class won't be registered into Hub before you sign in so you will miss the signIn event. Also please make sure you have bind this with onHubCapsule like this.onHubCapsule = this.onHubCapsule.bind(this);

Hi @powerful23 ,
I don't use withAuthenticator but <Authenticator> so I guess it is the same :
Currently, my code is like that:
<Authenticator hideDefault="true">           <SignInAWS {...this.props} checkTimeout={3}/>           <AppAuthPiece /> </Authenticator>
So, I tried to put my  component (that listen to the auth events outside the  component and I also added the bind line in the constructor of my component
`

`
=> But no change, no event...
Moreover, I think it is an issue to not receive events inside the  before being signedIn since some (most) of the event are before being signed. Isn't it?
Also, I add a workaround to send myself the signIn event if I find a valid user session, then, my App is showing up but I get an error when trying to use an AppSync API.
Again, this is fully working with<EMAIL_ADDRESS>I would love to understand what change!

I just did few other tests... And the issue seems to come from aws-amplify-react and not from aws-amplify since my code is working well with
"aws-amplify": "^0.4.1",
"aws-amplify-react": "^0.1.43"

I hope it can help...

@OlivierPT Thanks for your feedback! So which version of aws-amplify-react is working for you? 0.1.43?

Yes, it is 0.1.43. This configuration
"aws-amplify": "^0.4.1", "aws-amplify-react": "^0.1.43"
With this configuration, I have also a "new" and annoying behavior: when switching between screen (using react-router, client-side rendering), the component associated to authState = "signIn", "signedOut"... is showing up quickly and then my app is back...
So... I guess something change in the way to manage authState and the associated events...

Hi,
I re-tried to update my project to "aws-amplify": "^0.4.1" and "aws-amplify-react": "^0.1.47" today and I don't get the error as mentioned in this issue! I didn't had to update/change any part of my code so... maybe the issue was coming from a dependencie!?
Anyway, I close the issue.
Thank you


Support resuable configuration snippets options in cfn-compose.yml file

@rbalman Still looking for description 😄


Implementing Monotonic using embedded_time.

Hi. Not sure if this is the right place for this, but hopefully you can help. I'm working on an rp2040 project, and want to implement Monotonic using the embedded_time crate to avoid additional dependencies. (embedded_time is already used internally by the rp2040 HAL). I've found a very straightforward implementation using fugit (see https://github.com/korken89/rp2040-monotonic). Is there an easy way to do this with embedded time, without having to change the embedded_time crate.
Whatever types I try to use for Instant and Duration don't seem to line up with the trait bounds to make it work, but there's a good chance I'm misunderstanding one or both of these libraries.
So far I've defined Clock as follows:
pub struct Clock {
    pub(crate) timer: Timer,
}

impl embedded_time::Clock for Clock {
    type T = u64;

    const SCALING_FACTOR: embedded_time::rate::Fraction = Fraction::new(1, 1_000_000);

    fn try_now(&self) -> Result<embedded_time::Instant<Self>, embedded_time::clock::Error> {
        Ok(Instant::new(self.timer.get_counter()))
    }
}

and am using type Instant = embedded_time::instant::Instant<Clock> in my Monotonic impl. The issue is then no type seems to meet the requirements for Duration. If I use Microseconds<u64>, I get:
error[E0271]: type mismatch resolving `<Instant<monotonic::Clock> as Sub>::Output == Microseconds<u64>`
  --> src/monotonic.rs:31:20
   |
31 |     type Instant = Instant<Clock>;
   |                    ^^^^^^^^^^^^^^ expected struct `Microseconds`, found struct `embedded_time::duration::Generic`
   |
   = note: expected struct `Microseconds<u64>`
              found struct `embedded_time::duration::Generic<u64>`
note: required by a bound in `rtic::Monotonic::Instant`
  --> /home/nickclare/.cargo/registry/src/github.com-1ecc6299db9ec823/rtic-monotonic-1.0.0/src/lib.rs:39:30
   |
39 |         + Sub<Self::Instant, Output = Self::Duration>;
   |                              ^^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `rtic::Monotonic::Instant`

Alternatively, If i try type Duration = Generic<u64>, I get:
error[E0277]: the trait bound `embedded_time::duration::Generic<u64>: FixedPoint` is not satisfied
  --> src/monotonic.rs:31:20
   |
31 |     type Instant = Instant<Clock>;
   |                    ^^^^^^^^^^^^^^ the trait `FixedPoint` is not implemented for `embedded_time::duration::Generic<u64>`
   |
   = note: required because of the requirements on the impl of `Sub<embedded_time::duration::Generic<u64>>` for `Instant<monotonic::Clock>`
note: required by a bound in `rtic::Monotonic::Instant`
  --> /home/nickclare/.cargo/registry/src/github.com-1ecc6299db9ec823/rtic-monotonic-1.0.0/src/lib.rs:38:31
   |
38 |         + Sub<Self::Duration, Output = Self::Instant>
   |                               ^^^^^^^^^^^^^^^^^^^^^^ required by this bound in `rtic::Monotonic::Instant`

In the worst case I can just use the fugit dependency, but I'm just trying to understand if what I'm trying to do is supposed to be possible and I'm missing something, or if it would require changes in one or both of these libraries (rtic and/or embedded_time). The book suggests it should be possible, but I haven't found any examples of it being done.
Thanks
Nick

Hi!
Which version of embedded time are you using? 0.13 release branch I've heard is the recommended one, as discussed here: https://github.com/rtic-rs/cortex-m-rtic/issues/578
I have not seen much happening regarding embedded_time, which is unfortunate. I have personally migrated over to fugit, so I might be biased :P

We close this for now, pleas re-open if fugit would not work for you.


[Proposal] Generic Time Series Engine in Pinot

Hi folks, we are excited to share our proposal for supporting a Generic Time Series Engine in Pinot: https://docs.google.com/document/d/1SBDDf71QZINYUjAbRSWguNMfbrWRGfdcF1JPi8SJZlM/edit
You can find a quick overview of the proposal below.
Context
At Uber, we have seen an increase in Observability use-cases for Pinot (for both alerting and monitoring). Pinot is commonly used for High-Cardinality Metrics use-cases at Uber, since our in-house system for Observability (M3), isn't optimized for them.
One of the feedbacks from the customers has been that SQL tends to get quite complex for time-series analysis, and it not only hampers productivity, but is also lacking certain features commonly available in Observability systems (esp. Gapfill).
Earlier this year, we internally built out native support in Pinot for M3QL, Uber's open-source query language. The results were quite promising: we were able to support all of M3QL's features natively in Pinot.
We hence kick-started this effort to bring these improvements to OSS Pinot.
Language Agnostic
Our proposal is such that one will be able to implement plugins that can re-use the Generic Time Series Engine to support any time-series query language (PromQL, M3QL, etc.). This will allow each stakeholder to develop their own language plugins.
For more please refer to the doc.
cc: @kishoreg

@ankitsultana  - can you open the design doc for comment access please ?

@siddharthteotia done

First of all, this is a great initiative. I've been working in Devo Inc, a company that is focused on the timeseries market (trying to compete with Splunk) and there we designed our own streaming SQL database that supports both SQL and LINQ expressions. The later is closer to the M3QL language proposed in the design document and the language most customers used to use.
I've also created an internal proposal in StarTree to discuss about this feature. It can be read here. My document is not focused on the language itself. I believe SQL is expressive enough to be able to execute this kind of queries, but I understand that the market is not there and supporting M3QL and specially PromQL will be probably more useful to attract users.
I have some concerns about the designed document:

My main concern about the proposed design document is that I don't think we need to add too many new concepts on top of the multi-stage query engine (MSQE). Instead, what I think we need to add is some key functions and some special operators (specifically for aggregate, window and join) that should work in streaming fashion.
I think the design is too broad and ambitious. I would split it in the following:

Add streaming capabilities to the current MSQE. Basically the tumble functions and window base aggregate, join and probably window functions.
Add the plugable syntax/protocol support. There is where we would think about how to add M3QL, etc.
Add the ability to extend aggregates. The current design assumes this is going to be needed to support other syntaxes, but I don't think this should be part of the time series design given that is also something that would help normal queries. We have recently added the ability to create new scalar and transformation functions using typesafe and it was a hard job to do, so I think we should think about long term support when adding a way to add aggregate functions.

Anyway I know I'm late to the party and several key PRs have been merged so it doesn't make sense to discuss too much about the past, but please take me also in consideration for future developments.
Again, I don't want to sound very pessimistic. I think this is an awesome feature for Apache Pinot and it is great to see it progress that fast!

@gortiz thanks Gonzalo for sharing your views. There are several points made by you here and in the design document, but I think it's best to first establish the broader picture in which we are doing this.
As you know, most companies have been using Pinot for OLAP on mainly "events". The idea being that you can power user-facing dashboards or other end-user App UI elements like counters based on Realtime data with low latency.
At least at Uber, over the last 2 years, we have seen an increase in the number of users who are using Pinot for alerting and monitoring ("metrics"). And recently we also shared in the Uber Meetup that we are now using Pinot for our logging platform too. (side-note: Pinot specifically solves the problem of High Cardinality Metrics)
The term being used these days to cover this landscape of use-cases is MELT: Metrics, Events, Logs and Traces.
Pinot is able to solve all of these problems, but its capability to solve all of these problems differs. What our proposal aims to do is to immediately improve Pinot's ability to tackle the Metrics and the Logging problem, and IMO bring it up to the state of the art in the HICAM space. (note that CH also announced in their latest release that they plan to support PromQL).
re: your proposal about Streaming queries in Pinot, I think it's talking about a different problem.
re: your specific points about this not being necessary and we can leverage the MSE or add enable some SQL extensions, I shared the transformNull (gapfill) example in the doc which should help clear that up. For other readers, see the comment thread at the bottom of this doc
Finally, I think this time-series engine may stick out like a sore thumb right now, but once we integrate it with the Multistage Engine Shuffle Framework, I think it will sit very neatly with the rest of the code. And from a use-case perspective, it takes Pinot forward and expands its capabilities around use-cases that it theoretically can already support, but can't support as well. (side-note: from a high-level, I am thinking in the direction that we will have a common Operator interface in the SPI that will be oblivious of the data-model: relational, time-series, etc.)
A potential future direction is that we may make the entire engine (above the V1 server level engine) pluggable, but it might make sense to consider that after making sure that the time-series engine becomes a success first.


Set consistency level on connection to both nodes in do_upgrade

Joel, which test should I run 1000x times to confirm this fixes the issue?

I multiplexed a test that displayed failures fairly often here. It passed 500 times, and the change looks good to me.
+1


ci: Clean up workflow

This is nicer with a clearly split up matrix with arguments; and dropping the version number out of the GHA name avoids us needing to update the branch protection when bumping versions.

(edited the branch protection rules again)


Invalid module path in botbuilder-ai

Versions
botbuilder-ai 4.5.0
Linux
Describe the bug

> tsc
node_modules/botbuilder-ai/lib/qnamaker-interfaces/index.d.ts:12:15 - error TS2307: Cannot find module './qnamakerTraceinfo'.
12 export * from './qnamakerTraceinfo';

To Reproduce
Steps to reproduce the behavior:

Write code using botbuilder-ai.
e.g. import * as appInsights from 'botbuilder-applicationinsights';
execute tsc
See error

[bug]

I'm getting the same issue while trying to build on Linux with our pipeline. It's just a naming issue I think...
export * from './qnamakerTraceinfo'; should be: export * from './qnamakerTraceInfo';. Builds fine on mac but it's not case sensitive. Bit of an issue for us to be honest.

A quick update:

PR #1090 has been merged to master. Thanks @fukuhiro!
The PR has been cherry-picked over to the 4.5 branch. See the merge here.
The resulting file (in 4.5) is seen here - with the correct casing.
The 4.5.3 Build has run.
The 4.5.3 Packages are on the daily build server now. For example, BotBuilder 4.5.3 is here.

@dmvtech is testing these packages tonight, with a before/after check.
Plan is to release to NPM in the morning, provided testing goes as expected.

Confirmed working on Ubuntu 19.04.

The 4.5.3 SDK has been pushed to NPM.
Thanks everyone for reporting this, providing a good repo case, and even submitting the PR to fix the issue.


ERROR: unimplemented base type encountered in renderer scene cull when running --doctool

Godot version
4.0.alpha (8a1e5980116355024cd7a7ce0c15db7d4ecb200a)
System information
Mageia 9 Linux, x86_64
Issue description
When running --doctool (and possibly other run modes), the following error is printed to the terminal:
ERROR: unimplemented base type encountered in renderer scene cull
   at: instance_set_base (servers/rendering/renderer_scene_cull.cpp:619)

And when the process terminates:
ERROR: 21 RID allocations of type 'N13RendererDummy11MeshStorage9DummyMeshE' were leaked at exit.

This bug was introduced recently (didn't happen 2 weeks ago).
Steps to reproduce
godot --doctool
Minimal reproduction project
No response

Not entirely sure why but at some point support for creating a dummy mesh object was added to the dummy renderer however this implementation is incomplete and has loads of flow on effects. Trying to fix them all...

Looks like it was likely introduced by https://github.com/godotengine/godot/commit/3c55637459ee2939d67ac9fb87e0749b9caa7467 which allocates meshes in the Dummy Renderer but does not implement a way to free them


Don't duplicate code in the README

See #14

answered in the pull request (now closed)

Clarified wording in #16 .


Can we handle hold on on going call when we get call from other app and accept that call?

[x] I have read the Getting Started and Getting Started(with call invitation)
[x] I have already searched for the same problem

Environment

Technology
Version

Flutter version
Latest

Plugin version
Latest

Android version

iOS version

macOS version

Xcode version

Google Chrome version

Device information:
Description
Expected behavior:
Call should be go on hold if i will busy on other call or voip call
Current behavior:
Problem:
When we receive another call from an app like WhatsApp while an active call is ongoing in our Flutter app, the issue arises where our app’s call is not placed on hold. Ideally, when we answer the WhatsApp call, our app’s ongoing call should be automatically placed on hold.
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat

Problem:
When we receive another call from an app like WhatsApp while an active call is ongoing in our Flutter app, the issue arises where our app’s call is not placed on hold. Ideally, when we answer the WhatsApp call, our app’s ongoing call should be automatically placed on hold.
https://github.com/user-attachments/assets/c378e950-76af-450d-ad78-3fcd2d276c3f
This the demo video, when we're in call and I got call on my mobile. App call is still running in background
https://github.com/user-attachments/assets/71b9b650-ee2d-43fe-8b8f-2eced29e0825
Demo video for WhatsApp call as well. It's allowing us to continue talking in this app as well and even on WhatsApp call as well.

You can try to listen to this event. When the phone microphone is interrupted, an event will be thrown.

ZegoUIKitPrebuiltCallInvitationService().init(
  ...
  events: ZegoUIKitPrebuiltCallEvents(
    audioVideo: ZegoCallAudioVideoEvents(
      onRemoteMicrophoneExceptionOccurred: (
        ZegoUIKitUser targetUser,
        ZegoUIKitDeviceExceptionType? type,
      ) {
        ///
      },
    ),
  ),
);

Ok i will test it

@yoer its working fine


Windows - Problem Downloading

I tried to download this song in Windows10 the result is this:
C:\Python27\Scripts>spotdl.py

Enter a song/cmd: one direction taken

Taken - One Direction - Lyrics
4,047,501 Bytes [100.00%] received. Rate: [ 737 KB/s].  ETA: [0 secs]
Converting Taken_-One_Direction-_Lyrics.m4a to mp3
The system cannot find the path specified.
A folder named "Music" appeared in C:\Python27\Scripts\     ...but empty
this all works fine in my Zorin OS which is an ubuntu base but I wanna try it on my windows since it's my primary OS. any help?

Oh yeah. I am wrong. This repository should be extracted inside the Python folder not in Python/Scripts..... Thanks!

@cassaregh It's all good :)


Not able to update query parameter from navigation guard

Version
3.0.2
Reproduction link
jsfiddle.net/bqaxhgkc
Steps to reproduce
Inside beforeEnter try to modify query parameters by passing next({ query: {'my-param': 'new-value'} })
What is expected?
Query parameter will be updated with the new value and the URL will also update.
What is actually happening?
URL still having the same old value for my-param.

Even though the query parameter value is not updated in the URL, this.$route.query['my-param'] value is updated with the new value.

I'm realizing that calling next like the fiddle should create an infinite redirect, the same as doing
next({ name: 'foo', query: {'my-param': 5} })

which does create an infinite redirect. So I think this is technically a bug and I don't know why I didn't comment the workaround back then, but you can do this:
beforeEnter: (to, from, next) => {
  // avoid an infinite redirection
  if (String(to.query['my-param']) === '5') return next()
  // make sure to include `name`
  next({ name: 'foo', query: {'my-param': 5} })
    }

Hello, I am trying to do something similar... but not quite. Basically I have a situation where I do have query parameters not tied to any route in particular, but that I would like to carry over through routes if they are present.
I am trying this, but is not working:
router.beforeEach((to, from, next) => {

	const query = { ...to.query, ...configurationService.getURLCarryOverParams()};

	// Replace the query params adding the non-route related query params that
	// we want to carry over from page to page
	next({
		query: query
	});
});

There is any workaround for this situation? I also tried to assign the "query" of the "to" route but it is read only.

@JohnBernardsson I would override the router's push method to implicitly pass the previous query so you don't have to write this everywhere:
router.push({ name: 'thing', query: { ...$route.query }})

@posva you mean overriding the library push method itself? Interesting... ok, I'll give it a go, see how that works.
Thanks!

Just in case someone reaches this, and have the same problem (or something similar), I easily solved overriding the push method as @posva mentioned.
Code uses ES6 classes:
/**
 * @class
 * Extends Vue Router, to add specific push logic
 */
export class CustomVueRouter extends VueRouter {

	/**
	 * Overrides the Vue Router push method, to allow carrying over to all our pages
	 * some queryparams, if they were present in first load
	 * @param {RawLocation} location
	 * @param {Function} [onComplete]
	 * @param {ErrorHandler} [onAbort]
	 */
	push(location, onComplete, onAbort) {

		const
		staticQueryParams = {
			// Logic to get the params to carry over
		},
		newLocation = {...location,
			query: {
				...(location.query || {}), ...staticQueryParams
			}
		};

		// Calls the VueRouter push method
		super.push(newLocation, onComplete, onAbort);
	}
}

This is a major flaw with Vue Router. It really shouldn't be this difficult in updating passing a query string. I've spent hours dealing with every problem everyone has mentioned in this thread.
Needs to be completely redone, because this is just terrible.


Fix broken skip-push on docker-build-publish

Lost part of the keyword it seems.

Thank you for contributing to openstack-infra/jenkins-job-builder!
openstack-infra/jenkins-job-builder uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Use logging module instead of deprecated spider.log()

[Current coverage][cc-pull] is 83.50%

Merging [#1][cc-pull] into [master][cc-base-branch] will increase coverage by 0.89%

@@             master         #1   diff @@
==========================================
  Files             2          2
  Lines            92         97     +5
  Methods           0          0
  Messages          0          0
  Branches         23         23
==========================================
+ Hits             76         81     +5
  Misses            8          8
  Partials          8          8

Powered by Codecov. Last updated by [058154c...9015018][cc-compare]
[cc-base-branch]: https://codecov.io/gh/scrapy-plugins/scrapy-magicfields/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/scrapy-plugins/scrapy-magicfields/compare/058154ceaf63f9687f8691ea389bf2f9f30bb0c7...901501868b522432d53b574c8d5e02a19fc9e3dc?src=pr
[cc-pull]: https://codecov.io/gh/scrapy-plugins/scrapy-magicfields/pull/1?src=pr


STAR-432. Add options for consistency-level and serial-consistency-level

Allows overriding the default consistency-level (ONE) and the default serial-consistency-level (SERIAL) on the command line.
Co-authored-by: Robert Stupp<EMAIL_ADDRESS>

Started https://jenkins-stargazer.build.dsinternal.org/job/ds-cassandra-build/318/ with - hopefully - all the CQLSH tests


How to add language, for example: Chinese。

Description of the new feature/enhancement

Proposed technical implementation details (optional)

How to add language, for example: Chinese。

I'm sorry, but I'm not sure I understand the question/? The Terminal is already translated into Chinese. It should automatically launch in your system locale, but if you'd like to change the language manually, you can do so via the settings:

34/5000
I mean I compile my own code how to add languages. The dev version I compiled myself is only in English.

I mean how do I add language when I compile the codes. The dev version I compiled myself is only in English.

I'm not sure the final localization files are published anywhere other than in the package itself. The translated resources all come from an internal loc service, and aren't part of the repo itself. I suspect if you're running dev builds, you might just be out of luck here 😕

Thank you for your answer。


Dependency cycle error if sources are managed exclusively by puppet

Description
If you use the purge option of the main ::apt class, and want to manage sources exclusively in Puppet, then you fall in a dependency cycle error when using this module.
Problem
The sources are purged, they need to be updated, but they wait for apt::conf of this module and apt::conf require the Package['unattended-upgrades'] that can't be installed because sources are purged or wait for  Class['apt::update'] .
Testing code
class base::apt inherits base {

  $ubuntu_location = 'http://archive.ubuntu.com/ubuntu'
  $ubuntu_repos = 'main universe'

  class { 'apt':
    purge => {
      'sources.list'   => true,
      'sources.list.d' => true,
      'preferences'    => true,
      'preferences.d'  => true
    },
  }

  apt::source { 'ubuntu':
    location    => $ubuntu_location,
    release     => $lsbdistcodename,
    repos       => $ubuntu_repos,
    include_src => true,
  }

  apt::source { 'ubuntu_updates':
    location    => $ubuntu_location,
    release     => "${lsbdistcodename}-updates",
    repos       => $ubuntu_repos,
    include_src => true,
  }

  apt::source { 'ubuntu_security':
    location    => $ubuntu_location,
    release     => "${lsbdistcodename}-security",
    repos       => $ubuntu_repos,
    include_src => true,
  }
}

My base class includes many modules and some of them use also the ::apt class, like the docker one garethr-docker and the unattended-upgrades.
So, I need to execute apt_update before installating anything.
class base {
   Class['apt::update'] -> Package<| |>
    include 'base::apt'
    include 'unattended-upgrades'
    include 'base::install' # list of packages to install
}

Then my node is simply:
node default {
    include 'base'
}

Removing requirements on Package['unattended-upgrades'] resolved the issue.
See this commit.
But maybe there is a better way to go ?

How about
Class['apt::update'] -> Package<| tag != 'unattended-upgrades' |>

That doesn't scale (I think), but then the catch-all collector is a dangerous beast in any case.

The sources are purged, they need to be updated, but they wait for apt::conf of this module and apt::conf require the Package['unattended-upgrades'] that can't be installed because sources are purged or wait for Class['apt::update'] .

As far as I can see this would only be a problem if you've explicitly defined Class['::apt::update'] -> Package <||>. It wouldn't cause a dependency cycle just because of the package requirement.

I think this is fixed as a side effect of 67ee9eca5749fc54d8060cb7b87c8bd56c20765c. Since we no longer notify Apt::Update by default (as that was rather pointless) it should have resolved this dependency cycle.

Closing due to the above commit.


[Loggable] issue with userentity

Hi,
I versioned "lastlogin" field from userentity to get user connexion history.
But it is a particular case and I have the following issue: username is always NULL at this event because when user logins, username is not yet set to the loggable listener.
I think I located the reason.
in class "LoggerListener" on the function "onKernelRequest(GetResponseEvent $event)":
/**
 * Set the username from the security context by listening on core.request
 *
 * @param \Symfony\Component\HttpKernel\Event\GetResponseEvent $event
 */
public function onKernelRequest(GetResponseEvent $event)
{
    if (HttpKernelInterface::MASTER_REQUEST !== $event->getRequestType()) {
        return;
    }

    if (null === $this->tokenStorage || null === $this->authorizationChecker) {
        return;
    }

    $token = $this->tokenStorage->getToken();

    if (null !== $token && $this->authorizationChecker->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
        $this->loggableListener->setUsername($token);
    }
}

When I want to version "lastlogin" field from Userentite, I'm in the first case because:
$event->getRequestType() == HttpKernelInterface::MASTER_REQUEST
and unfortunately,
$this->loggableListener->setUsername($token); is not set...
How can I solve this case ?
thanks for your help
regards

Hi, you should try to switch the event priority if it gets invoked before security token is set. or try to debug your event listener to ensure that the token gets set and when..

Hi,
thanks a lot for your suggest.
Please could you say me how to switch event priority ?
About listener, I'm using stof/doctrine-extensions-bundle which gives "loggerListener"
the problem remains the same:
/**
 * Set the username from the security context by listening on core.request
 *
 * @param \Symfony\Component\HttpKernel\Event\GetResponseEvent $event
 */
public function onKernelRequest(GetResponseEvent $event)
{
    if (HttpKernelInterface::MASTER_REQUEST !== $event->getRequestType()) {
        return;
    }

    if (null === $this->tokenStorage || null === $this->authorizationChecker) {
        return;
    }

    $token = $this->tokenStorage->getToken();

    if (null !== $token && $this->authorizationChecker->isGranted('IS_AUTHENTICATED_REMEMBERED')) {
        $this->loggableListener->setUsername($token);
    }
}

not sure if it is still valid example, but have a look. I can't debug it for you though, since I just cannot replicate the context.


Add Relative Path for the AppShell

New Feature Proposal
Add capability to Emulator to load app shell from a relative path to the Pilet.
For now, we can use an app shell using an absolute path.
Description
We can add an attribute to <AppShell /> like ../../path/to/shell
Background
This will allow faster dev/test on local environment.

For now, we can use an app shell using an absolute path.

That is not true. You can use NuGet package paths. This is also the recommended way.

So when working on a Solution with Shell and Pilet, we have to pack the nuget and copy it to the %userprofile%/.nuget/packages folder, no ?

I don't know. Well, at some point you need to pack and distribute; when that is would be up to you. The ideal situation is that a local shell is detected and just used.


[TRAH] Sergei / TRAH 3146 / Add performance metrics for trader's hub new repo

add startInitTimers to main and create performance metrics file

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 21 (23.81%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 49.037%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/utils/performance-metrics-methods.ts
4
20
20.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
885

Relevant Lines:
1495

💛  - Coveralls


ConvexHull is throwing an exception for 4D points

Hi,
The program below is throwing exception like:
System.IndexOutOfRangeException: Index was outside the bounds of the array.    at MIConvexHull.MathHelper.CalculateFacePlane(ConvexFaceInternal face, Double[] center) in C:\Users\Muti\Desktop\MIConvexHull\MIConvexHull.NET Standard\ConvexHull\MathHelper.cs:line 118
`using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace ConsoleApp1
{
class Program
{
public static class Tools { public static int DBG = 0; }
    static void Main(string[] args)
    {
        var dbg = Tools.DBG;

        List<Point> points = new List<Point>();

        foreach (var x in Enumerable.Range(1, 3))
            foreach (var y in Enumerable.Range(1, 3))
                foreach (var z in Enumerable.Range(1, 3))
                    foreach (var t in Enumerable.Range(1, 3))
                        points.Add(new Point(x, y, z, t));

        var hull = MIConvexHull.ConvexHull.Create(points);

        if (hull is null)
            dbg = Tools.DBG;

        var hullResult = hull.Result;

        if (hullResult is null)
            dbg = Tools.DBG;

        var hullPoints = hullResult.Points.ToList();

        if (hullPoints is null)
            dbg = Tools.DBG;
    }

    public class Point : MIConvexHull.IVertex
    {
        private double[] Values = new double[] { };

        public Point(double a, double b, double c, double d)
        {
            Values = new double[] { a, b, c, d };
        }
        public double[] Position
        {
            get
            {
                return Values;
            }
        }
    }

}

}
`

whoops, that shouldn't have happened. I fixed it now and pushed an update to nuget (<IP_ADDRESS>9) and to this repo (https://github.com/DesignEngrLab/MIConvexHull/commit/d5f3a77489f7f6c29ad954b29bfeaf8e4f4b646b).
I fear that you may not be satisfied with the answer however. As you likely know the correct answer to your problem is 16 vertices, but MIConvexhull returns 17. this is because the current implementation does not produce either the maximal or minimal set of vertices - it is more based on speed. In your problem, all but one of the 81 vertices is on the convex hull, but many points like: {1,3,1,2} would be along some hyper-edge - and not a corner like {1,1,1,1}.
The minimal set would remove these side points. The maximal set would include these. And there are different used cases in which you would want one or the over.
Right now, it's all about speed. This is because it has been derived for uses in geometry where the shape is need - not necessarily the membership decision about each vertex.
Well, this is becoming a long-winded answer but I hope it helps you understand where we're coming from. Until we get around to having an input setting to switch between: maximal, minimal, or fast; perhaps you can post process the convex hull to find any vertices that are along edges.

Thanks @micampbell . It is solved.


Does Ensemble model use only 2 models?!

While I was using your code, I noticed that the lightgbm tree only shows 2 models out of all models in ensemble method.
Here's what tried to do:
from deepface import DeepFace
import lightgbm as lgb
home = DeepFace.functions.get_deepface_home()
ensemble_model_path = home+'/.deepface/weights/face-recognition-ensemble-model.txt'
deepface_ensemble = lgb.Booster(model_file = ensemble_model_path)
lgb.plot_tree(deepface_ensemble,orientation='vertical',
        show_info=['split_gain','internal_value','internal_count','internal_weight','leaf_count','leaf_weight','data_percentage'])
lgb.plot_importance(deepface_ensemble)

This gives the following tree and importance plots

Does this mean that the tree only uses VGG_Face_cosine & OpenFace_cosine to get the score values?
Also, How to get score values instead of distances when doing only single model? (maybe a feature to add later?!)

OK
I managed to get all 112 trees by addeing tree_index= to the lgb.plot_tree() function
for itrees in range(deepface_ensemble.num_trees()):
    lgb.plot_tree(deepface_ensemble,tree_index=itrees,orientation='vertical', show_info=['split_gain','internal_value','internal_count','internal_weight','leaf_count','leaf_weight','data_percentage'])
lgb.plot_importance(deepface_ensemble)

thanks for the catch!


[stable/rabbitmq-ha] add readme about ingress

What this PR does / why we need it:

add readme about ingress

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/lgtm
/assign


version 0.2.2

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@dfm Three of the builds are timing out because no output was received over 10 min during build. Is there a way to increase this limit on circleci? I had to increase it on Travis using the travis_wait command.

I'm not sure. I just opened a pull request to your fork with a possible fix...

@dfm Your fix didn't quite do the trick; I had to increase the timeout in config.yml. Working now!


add maven-assembly-plugin dependency to readme

i had to add this dependency to my pom.xml to get it to work, thought i'd save others the hassle of googling

currently in the process of moving, once i get a chance to test this out and everything i’ll merge it

alright

Alright checking this out now

I have never in my life seen a <type>maven-plugin</type> dependency before and can't seem to find any information about it... could you show me what told you that this was needed and how to reproduce your issue when it is not included?

i forgot what article said it but from my testing intellij complains it doesn't exist without using the dependency

Sorry but I really don't see the use in this and I've never seen this pattern before, if you can find information on why it's actually needed (I tried to look for myself and failed) then I might consider merging