onset/offset shift on onsets_and_frames result

Hi, i'm evaluating pretrained onset and frames transcription model.
Using the pretrained model, I transcribed MAESTRO dataset(test split) and saved the piano roll and midi. When I draw the histogram of note onset/offset errors compare to ground-truth midi, I found that estimated onsets are shifted around -16ms, while offsets are shifted +16ms (the center of the histogram of onset / offset errors was not zero-centered).
I suspect it is caused by sequence_to_pianoroll quantization, because start_frame is calculated with round down here https://github.com/tensorflow/magenta/blob/dc04ef3ff6a410559106fcf539a259153aae518d/magenta/music/sequences_lib.py#L1813
which might lead time shift of half frame(512/16000/2 = 16ms) when it converted back into time.
I also checked hparam for min_frame_occupancy_for_label is set 0.0.
I also observed that mean onset F1 score improved (.9473 -> .9494) when I shifted prediction 16ms forward.
Is it right that the pretrained model is trained on shifted label? (+16ms on onset, -16ms on offset) if it is, should I shift the onset & offset to get correct result?

I've checked with lasted version(master) today with one of wav file in MAESTRO set (MIDI-Unprocessed_03_R1_2008_01-04_ORIG_MID--AUDIO_03_R1_2008_wav--4.wav). Here's the piano roll of ground truth(above) and transcirption(below) .
Staccato notes at the beginning becomes longer (in both direction)

Histogram of onset errors are also seems to be shifted.

Interesting! Maybe we should switch to using np.round (bankers' rounding) for both the onset and offset in sequence_to_pianoroll during training. Do you think that would fix the issues you're seeing?

I think min_frame_occupancy_for_label parameter in sequence_to_pianoroll is determining the relation between sequence head and first frame in the piano roll.
https://github.com/tensorflow/magenta/blob/85ef5267513f62f4a40b01b2a1ee488f90f64a13/magenta/models/onsets_frames_transcription/data.py#L224
When min_frame_occupancy_for_label == 0.5 might be the situation we are thinking of.
But I think it doesn't have to be like that, because label annotation can be determined arbitrary and  may have small or big effects on model performance. (have you already tested?)
I think we can compensate the effect of min_frame_occupancy_for_label when we decode piano roll to sequence. Maybe in pianoroll_to_note_sequence? https://github.com/tensorflow/magenta/blob/dc04ef3ff6a410559106fcf539a259153aae518d/magenta/music/sequences_lib.py#L1927
I'm not sure why offsets are delayed, while onsets are shifted forward. But maybe it happens with similar reason.

I think using min_frame_occupancy_for_label would just shift the error distribution forward by a frame and using np.around would result in a more unbiased distribution of quantization errors.
@jesseengel do you have any thoughts here?


cannot un-WIP or change PR title

Description
since new version of gitea 1.21.9, i can't un-WIP a PR. i get the error
JavaScript error: Cannot read properties of undefined (reading 'endsWith') (https://gitea.raphaelpiccolo.com/assets/js/index.js?v=1.21.9 @ 20:181341). Open browser console to see more details.

i also cant change PR title. i get no error, but it does nothing
inspecting the ajax request says :

Une erreur s’est produite:

ChangeTitle, exit status 128 - fatal: ambiguous argument '': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
 - fatal: ambiguous argument '': unknown revision or path not in the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'

Gitea Version
1.21.9
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
2.40.1
Operating System
ubuntu on the host, i dont know what is in the docker image
How are you running Gitea?
docker compose
gitea:
        image: gitea/gitea:latest
        container_name: gitea
        restart: always
        volumes:
            - ./gitea:/data

Database
MySQL/MariaDB

Here are some workarounds: https://github.com/go-gitea/gitea/issues/29997#issuecomment-2015053135

Duplicate of https://github.com/go-gitea/gitea/issues/29997#issuecomment-2015053135, pinned that one.

Ref #29997
Sorry for the inconvenience.

It could be fixed by updating to pre-1.21.10:

Docker image: use gitea/gitea:1.21-nightly and gitea/gitea:1.21-nightly-rootless
Binary: download from https://dl.gitea.com/gitea/1.21/


Check 2017-present WQP data initially tagged as "preliminary"

Consideration of data tagged in WQP as "Preliminary"

Just to clarify, is the "preliminary" tag within the harmonized dataset? I'd expect that checks on those data would now be complete, so we could pull the current values for those samples labeled "preliminary".
For 2019-present data labeled as preliminary, I can see a few options:

We learn from the step above how much the preliminary vs. approved data differ. We base our decision to use preliminary or not from that analysis.
We use/do not use preliminary data following the same guidelines as used for harmonization.
We do not use preliminary data for data quality concerns. If there are few preliminary samples, this can be a good option.

Originally posted by @jds485 in https://github.com/USGS-R/drb-inland-salinity-ml/issues/3#issuecomment-953928129

Update based on figures in #3 and #19, there are many preliminary data 2000-present, and data from ~2020 - present have provisional labels per USGS practice for newly collected data. So, option 3 on this list is not going to work for this project.

We currently use approach 2 on the list above and rely on the flags generated during data harmonization when it comes to filtering sites, since the figures in #3 show that a substantial chunk of data initially tagged as 'preliminary' in WQP is never revisited and approved. I'm going to close this issue for now.


MM-51188 Use mux.Router

Summary
Migrate JIRA codebase to use mux.Router
Ticket Link
https://mattermost.atlassian.net/browse/MM-51188

Codecov Report
Patch coverage: 68.68% and project coverage change: +0.40 :tada:

Comparison is base (a7f0e41) 29.96% compared to head (2f8fa78) 30.36%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #923      +/-   ##
==========================================
+ Coverage   29.96%   30.36%   +0.40%
==========================================
  Files          49       49
  Lines        7520     7524       +4
==========================================
+ Hits         2253     2285      +32
+ Misses       5078     5048      -30
- Partials      189      191       +2

Impacted Files
Coverage Δ

server/command.go
16.06% <0.00%> (ø)

server/plugin.go
9.45% <0.00%> (-0.03%)
:arrow_down:

server/setup_flow.go
0.00% <0.00%> (ø)

server/webhook.go
31.09% <0.00%> (ø)

server/http.go
65.53% <81.57%> (+21.21%)
:arrow_up:

server/issue_parser.go
70.00% <100.00%> (ø)

server/subscribe.go
66.11% <100.00%> (+0.10%)
:arrow_up:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@hanzei @mickmister, I have implemented the review comments and some additional findings like using checkAuth as part of the router handler. Re-requesting your reviews.

@srkgupta I may be mistaken but I believe something is broken here. When I try to smoke test by installing a cloud instance, the .json which provides the installation details seem to no longer be reachable and returns a 404. This causes the install from the Jira side to fail.
Please let me know if you can reproduce this or perhaps there is some unrelated cause I have overlooked.
Steps:

Install the latest production release and enable the plugin
Run /jira instance install cloud https://mmtest.atlassian.net
In the ephemeral post note that a link in generated to https://dkh-local.ngrok.io/plugins/jira/instance/aHR0cHM6Ly9tbXRlc3QuYXRsYXNzaWFuLm5ldA==/ac/atlassian-connect.json Your MM URL will differ but clicking the link open a .json object in a new tab with installation details
Upgrade to MM-51188_mux-routerbranch
Run connect command again (optional as the lik address has not changed)
Observed the same URL now goes to a 404 when clicked

Thanks @DHaussermann for catching this issue. I have fixed the issue and pushed a new commit. It should resolve the issue. Can you please specifically check the entire OAuth1 & the Connection flows. The HTTP methods for some of these API endpoints were not clearly defined previously. Hence any HTTP method like GET, POST request would work. However with the new mux.Router approach, since its specifically defined, just want to be sure that these are correctly working now.

Great work @srkgupta :+1:

Hi @DHaussermann
I looked into this in detail and found that the URLs for certain paths started with /instance/{instance_id} and hence had to be handled in a slightly different way. I have fixed this issue and have tested a basic Jira Cloud setup flow and it is working fine now. Please recheck now and let me know if you see any other issues.


(docs): externals array still referenced

This PR https://github.com/webpack/webpack.js.org/pull/2100 didn't remove all references to array in externals.
https://github.com/webpack/webpack.js.org/blob/2eb1c0e3c355a52a10cc54dba2dab30958fb7758/src/content/configuration/externals.md#array
https://webpack.js.org/configuration/externals/#array
https://github.com/webpack/webpack.js.org/blob/2eb1c0e3c355a52a10cc54dba2dab30958fb7758/src/content/configuration/externals.md#combining-syntaxes
https://webpack.js.org/configuration/externals/#combining-syntaxes
Which is confusing to users, as it seems array is still supported (but doesn't seem to be)

Looking into it.


Add an example of performing validation directly in the component to "Business logic validation" section of "Blazor forms and validation"

I was working on a simple Blazor app recently (two single-form pages) and I needed to add some fairly simple custom validation to a form (ensure at least one of a set of checkboxes was checked). I was surprised that the documentation guided me to create a custom validator component for such a basic scenario.
I suggest we update the "Business logic validation" section to include content that shows how to add basic custom validation to a form so that it correctly interacts with Blazor's built-in form validation logic, without having to encapsulate that logic in a separate component or validation attribute. This would be desirable in cases where the form's model is defined within the component hosting the form (either as members directly on the component or in a sub-class), rather than a model class that is separate for the purposes of reuse, etc.
The approach involves configuring an EditForm to use a declared EditContext and ValidationMessageStore instance, and then setting up a handler for the OnValidationRequested event of the EditContext, inside of which the custom validation logic is performed and the result of which is stored in the ValidationMessageStore instance.
Example:
@page "/example"

Form with custom validation

<EditForm EditContext="editContext" OnValidSubmit="HandleValidSubmit">
    <DataAnnotationsValidator />

        Name
        <InputText class="form-control" @bind-Value="model.Name" />

                    <InputCheckbox id="option1" class="form-check-input" @bind-Value="model.Option1" />

                        Option #1

                    <InputCheckbox id="option2" class="form-check-input" @bind-Value="model.Option2" />

                        Option #2

                <ValidationMessage For="() => model.Options" />

    <button class="btn btn-primary" type="submit">Save</button>
</EditForm>

@code {
    Model model;
    EditContext editContext;
    ValidationMessageStore validationMessageStore;

    protected override void OnInitialized()
    {
        model = new Model();
        editContext = new EditContext(model);
        editContext.OnValidationRequested += editContext_OnValidationRequested;
        validationMessageStore = new ValidationMessageStore(editContext);
    }

    void editContext_OnValidationRequested(object sender, ValidationRequestedEventArgs args)
    {
        validationMessageStore.Clear();

        // Put custom validation logic here
        if (!model.Options)
        {
            validationMessageStore.Add(() => model.Options, "At least one option must be selected");
        }
    }

    void HandleValidSubmit()
    {
        // This will only be called when all validation has passed

    }

    class Model
    {
        [Required]
        public string Name { get; set; }

        public bool Option1 { get; set; }
        public bool Option2 { get; set; }
        public bool Options => Option1 || Option2;
    }
}

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

ID: eef47457-c8cf-74a4-b884-c1fe5c2ef52d
Version Independent ID: 09a02d10-2834-467d-d8c2-3fd19ceb3863
Content: ASP.NET Core Blazor forms and validation
Content Source: aspnetcore/blazor/forms-validation.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

cc: @pranavkm ... any additional guidance? ... any sneaky 😈 or hidden 🐉 for this scenario that I'll need to call out?


Implement an idea about guarding from obsolete mutations

I will not merge it, just let me know what you think, ping @ognen

Closed this one, since it was done in master and only as an idea. Changes as discussed will be committed to develop.


[UI] SSO OIDC

Motivation and context

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into the CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

actions_users/issue_1810_login_logout failed because changed social auth API is used in UI, but not on the server(it will be removed in another PR), so I guess we can skip it


1.18.2 Axolotl and Turtle's vision is permanently zoomed, even if its in water

Yeah, I should really fix this as it is really annoying, especially when you are morphed as a turtle or as a spider.

This issue has now been fixed with commit cd46f4bf8faf3e4c06187eebb5a586bfaa69bfe5.
I should note however that there is still some weird behaviour when you go in and out of webs really quickly as a cave spider.

Alright, fixed the weird FOV behaviour when a cave spider is in a cobweb in this commit: bd7db2af7fab3bacfcd5ec4bab083b0b5f37536c. Finding this issue was actually surprisingly difficult (took me 1-2h or so), but now I think there shouldn't be any FOV issues anymore. If you however still encounter problems regarding the FOV in future releases, please contact me.


more than one @IndexType in model

In sample/app/indexing/Team.java the index type is being set like this:
@IndexType(name = "team")
How can this be set to several or all types? I tried
@IndexType(name = "teama,teamb,teamc") for several types
@IndexType(name = "*") for all types
When leaving @IndexType out and just setting @IndexName I get a NullPointerException.

I was just discussing the same issue with @eduardofcbg who is also developing a Play2/ES module which works with the current Play 2.4.1 version at https://github.com/eduardofcbg/playframework2-elasticsearch
There is no need to enhance the @IndexType here and the issue can be closed.


Upload objects/images to the site

#15 All of the hyperlinked images are added in the metadata now

This issue has changed to become later issues


rustpython cannot be installed via cargo because all rustpython-vm cargo package versions were yanked

Summary
See the title.
This is the output for the cargo install rustpython command:
    Updating crates.io index
  Installing rustpython v0.1.2
error: failed to compile `rustpython v0.1.2`, intermediate artifacts can be found at `/var/folders/jc/g95ml13n3fb8qz480wwp01xr0000gn/T/cargo-install6HITH8`

Caused by:
  no matching package named `rustpython-vm` found
  location searched: registry `https://github.com/rust-lang/crates.io-index`
  required by package `rustpython v0.1.2`

And here we can see how the package versions were yanked: https://crates.io/crates/rustpython-vm/versions

Oh, it's already documented.


solved

hi all,
i have solve the problem  Thanks.

@andrewarnier then can you please close the issue yourself, thanks


add boolean to clearAgreeShowDialog

I'm leveraging this lib after I show my own dialog on whether they're liking the app or not. Essentially need a way to set AgreeShowDialog for both cases -- if they hit no, never show the app rate dialog again.

Thx just a minute!

Thx!

+1 how do we get this pushed to maven central?

@bblia @rogerhu I'll publish new version in the evening. Stay tuned.

Thank you so much! :)
On Thu, Oct 8, 2015 at 6:22 PM Shintaro Katafuchi<EMAIL_ADDRESS>wrote:

@bblia https://github.com/bblia @rogerhu https://github.com/rogerhu
I'll publish new version in the evening. Stay tuned.
—
Reply to this email directly or view it on GitHub
https://github.com/hotchemi/Android-Rate/pull/71#issuecomment-146731486.

Sorry for the late. 0.5.4 will be out soon!


(PUP-8471) Prevent TypeSet loader from being replaced

Before this commit, the TypeSet loader that provides internal access
to the types defined locally in the set was replaced by the loader
that is associated with the file containing the type set source. This
replacement happened in the TypeParser when resolving the types
contained in the TypeSet. This commit prevents that replacement from
taking place.

CLA signed by all contributors.


Pass retry count at executeForPromise

It can be useful for logging scenario:
polly().retry(2)
  .executeForPromise((count) => {
  if (count > 0) {
    logger.info(`Retry number ${count}`);
  }
  })

Codecov Report

Merging #73 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master     #73   +/-   ##
======================================
  Coverage    97.4%   97.4%
======================================
  Files           1       1
  Lines          77      77
======================================
  Hits           75      75
  Misses          2       2

Impacted Files
Coverage Δ

src/polly.js
97.4% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @jincod
I like the suggestion but there are different ways of using Polly and they need to have the same behavior. The execute(), executeForPromise() and executeForNode() for both retry and waitAndRetry` would all need to do the same here.
Thanks,
Maurice

Hi @mauricedb,
I've added count for execute and executeForPromiseWithDelay, but I'm not sure about executeForNode and executeForNodeWithDelay.
I see 2 possible options:
First without changing API, but very confusing: count arguments goes after callback
polly()
    .retry(2)
    .executeForNode(function (cb, count) {
        console.log(count);
        fs.readFile(path.join(__dirname, './hello.txt'), cb);
    }, function (err, data) {
        //...
    });

or better API, but has breaking changes:
polly()
    .retry(2)
    .executeForNode(function (count, cb) {
        console.log(count);
        fs.readFile(path.join(__dirname, './hello.txt'), cb);
    }, function (err, data) {
        //...
    });

What do you think about this?

Hi @jincod,
I would go for the first option as it doesn't change the API and break exsting code. A second reason is that most users will not be interested in using the count parameter but have to use the callback. If the callback is the second param they will always have to include an unused first parameter.
BTW One thing I was thinking about was ading more info like the delay time so maybe passing an object with the count would be better than just a count itself. What do you think?

@mauricedb I've implemented second option and pass count as field object's

@jincod Looking good.
The only two thing missing are:

[ ] Unit tests to make sure the count value is as expected
[ ] The updated TypeScript definition file

With those I will merge the PR.
Thanks

Dropped Node 4 and 5 support with the new syntax but that should be fine.
Thanks

@mauricedb thank you!


Repo sync for protected CLA branch

The pull request is created from main637916022728922729 to main to fix git push error for protected CLA branch

Docs Build status updates of commit 79dd5b6:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


API for argument-based assertions

The only reason why I use gomock instead of this is that I can't simply do stuff like
v.EXPECT(1).RETURN(2)
v.EXPECT(2).RETURN(2)
v.EXPECT(3).RETURN(2)

I mean I have to write some if statements and so on. Looks like there's no easy way to configure several calls of mock. It looks especially complicated in a go-way-ish table-driven-tests

I guess, supporting the same API as gomock is not a goal for moq. If you wish to have such an API, the simplest thing will be to stick with it.
That being said, there are small helpers that may simplify your life for use cases like the one above. Have a look at my comment in https://github.com/matryer/moq/issues/160#issuecomment-963643394.
The "pop" functionality mentioned in the above comment can now be moved to a generic function like this:
func pop[T any](t *testing.T, s *[]T) T {
        t.Helper()

        if len(*s) == 0 {
                t.Fatal("queue is already drained")
        }

        item := (*s)[0]
        *s = (*s)[1:]

        return item
}

With this, it becomes easy to place multiple different return values into a list (slice), which then can be placed in the test table for the table-driven tests.


Fixed: Adding an extra & character at the end of the password value

Hi,
I was using your repository and encountered CORS error. In the browser's console it showed me the query string that was generated against the data I provided. In the message, I noticed that there was an extra '&' character in my password which was added during encoding url. I have  performed a fix for this problem in this pull request, kindly review.
Thanks
Ali

Hi @alihasan94, thank you for using my repo. I hope it helps you. I merged your changes.

Thank you 👍


black square bug

this is somewhate intentional since the frames are based of trapdoors very closely, this way there are more distinguishable.


EdgeDetection crashes Raspberry Pi 3

Using Detect and running a very basic program on the Raspberry Pi 3, will result in the whole Raspberry Pi locking up (and needing a forced restart).
This only happens on a pin with very many rises/falls, currently I am using this for a RF receiver, so there are quite a lot of rises/falls.
The code I am using is this ...
package main

import (
	"fmt"
	"time"

	rpio "github.com/stianeikeland/go-rpio"
)

// This program is just going to be
// a POC of the sender working.
// This should and WILL be cleaned up
// I do not care, I will not fucking
// write the report without having
// the time to do this properly

// Raspberry Pi 3 GPIO
// https://i.imgur.com/PPFf2ul.png
const rx_pin_nr = 2

var preamble = [8]int{1, 0, 1, 0, 1, 0, 1, 0}
var sfd = [8]int{1, 0, 1, 0, 1, 0, 1, 1}
var rx_pin rpio.Pin

func main() {
	// initialize go-rpio
	err := rpio.Open()

	if err != nil {
		fmt.Println(err)
	}

	// set our pin as a output
	rx_pin = rpio.Pin(rx_pin_nr)
	rx_pin.Input()
	rx_pin.Detect(rpio.RiseEdge)

	for {
		// sendData(test)
		fmt.Println("Running stuff")
		time.Sleep(100 * time.Millisecond)
	}
}

func check_sync() {

}

Not even doing anything with it. I don't know what the solution would be, but I think that we could get the interrupt etc. if we used such a system as epoll (http://man7.org/linux/man-pages/man7/epoll.7.html), I know it is uses for another Go system for GPIO interaction here - https://godoc.org/github.com/kidoman/embd

Edit, it is done using this - https://golang.org/pkg/syscall/#EpollCreate1

Hi,
this seems to be the same issue as in #33, (which I am not able to reproduce, unfortunately). Does it freeze every time as soon as Detect method is called, or only when you connect it to RF receiver and generate some edges?
Could you provide me more info about your environment?

Using the newest version of raspbian downloaded from raspberrypi.org three days ago (and ran update/upgrade)
Going a direct go run main.go when running my code.

But I will check number 1 tomorrow to make sure! :)

Setup

Raspberry Pi 3 model B v1.2
RF sender/receiver here
Power supply for RPi here - sometimes relevant for issues on the RPi..

Running the latest version of Raspbian (2018-11-13-raspbian-stretch-lite), having the latests updates using sudo apt update && sudo apt upgrade -y
pi@hub:~ $ uname -a
Linux hub 4.14.79-v7+ #1159 SMP Sun Nov 4 17:50:20 GMT 2018 armv7l GNU/Linux
pi@hub:~ $ lsb_release -a
No LSB modules are available.
Distributor ID:	Raspbian
Description:	Raspbian GNU/Linux 9.6 (stretch)
Release:	9.6
Codename:	stretch
pi@hub:~ $

Code execution
Running the code below WITHOUT the RF receiver attached, prints "works" and then exists the program and everything works fine.
But attaching the RF receiver, it will print out works and then just completely freeze up and I cannot ping the host, nmap it, ssh into it or anything. I need to unplug the power and plug it in again.
Currently I am just using the polling method, and checking the state each time.
There are not that many rises and falls, but it does of course still have some jitter sometimes.
main.go
package main

import (
	"fmt"
	"time"

	rpio "github.com/stianeikeland/go-rpio"
)

func main() {
	// initialize go-rpio
	err := rpio.Open()

	if err != nil {
		fmt.Println(err)
	}

	// set our pin as a output
	rx_pin := rpio.Pin(2)
	rx_pin.Input()
	rx_pin.Detect(rpio.RiseEdge)
	fmt.Println("Works..")
}

Output from hexdump
00000000  7e 00 00 00 3f 00 00 00  01 00 00 00 40 00 00 00<EMAIL_ADDRESS>00000010  40 00 00 00 00 00 10 00<EMAIL_ADDRESS>00000018

Final notes
If you lived nearby, I would be able to give you a module for testing (if the university agrees), but doesn't look like it.

That is really some kind of magic.
Maybe pi just can't handle faster changes.
You can edit the test file and use pin in Clock mode, and gradually increase the frequency to see if there is problem in that. Actually I can do this myself.
Also we can try to use "asynchronous" GPAREN/GPAFEN registers for detection instead of "synchronous" GPREN/GPFEN registers. The difference is the synchronous looking for 011/100 patterns, and the other dont - so they should be able to detect even shorter pulses. Just change lines 351/352 in rpio.go to
renReg := p/32 + 31
fenReg := p/32 + 34

See chapter 6.1 in https://www.raspberrypi.org/app/uploads/2012/02/BCM2835-ARM-Peripherals.pdf if you are interested in details.

@Drahoslav7 awesome, i'll have results in a few minutes after testing dtoverlay=gpio-no-irq

@Drahoslav7 my code works like a charm the dtoverlay setting, thank you

I'm happy we figured it out.
Hope it works for @eyJhb as well. Feel free to reopen this if not.

@Drahoslav7 haven't had time to test it, currently in the mid of getting all the code to work together before the deadline.
Will test it out :) - But wouldn't it be awesome, to have actual interrupts?

I had this issue on my Pi 4B running the program as root. Adding dtoverlay=gpio-no-irq to the boot config fixed it. Using go-rpio v4.6.0.


Columns with Foreign keys  are not filtered in data view and reference panel of SQL Server

System information:

Operating system (distribution) and version Windows 10 Home
DBeaver version 6.2.0

Connection specification:

Database name and version SQL Server Express
Do you use tunnels or proxies (SSH, SOCKS, etc)? no

Describe the problem you're observing:
Columns with Foreign keys  are not filtered in data view and reference panel of SQL Server
Steps to reproduce, if exist:
-Open a table with foreign keys in data view
-try to filter column with Foreign key using filter icon on column header.
Expected result: Selected value is filtered
Actual result: value is not filtered, a window with a mistake is shown

dbeaver-debug.log
.log
Include any warning/errors/backtraces from the logs

Not reproduced any more


simplified dedicated node guide

Related Issue

#484

Work Done

simplified dedicated machines steps in manual

Closing in favor of #514
We don't need a dedicated node section anymore.


fix a table

@yoichinak : Thanks for your contribution! The author, @ckarst, has been notified to review your proposed change.

Thank you for fixing this @yoichinak 👍
#sign-off

@JennieHubbard: I'm sorry - only the author of this article, @ckarst, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Broker service locator improvements

Issue #51
https://github.com/loekd/ServiceFabric.PubSubActors/issues/51

@danijel-peric  Thank you for this contribution! Can you please submit the broker service locator changes separately first? And please have a look at the rebase conflicts.


Pallet Label (if different to type) should be indexed for search

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
fixes #3297
As the file node and file in node now have different palette label to type name, the search does not reveal them.
this modification adds the palletLabel (when different to  type name) to the search index.

Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage increased (+0.03%) to 67.339% when pulling fdc4219b681ec2798d08e04d8a126f48cd6ba781 on Steve-Mcl:search-find-filenode into 5b428bb8e669dedc967a41d963b767a5fac559a0 on node-red:dev.


Provide the ability to check the status of external systems

The biorepository acts essentially as a proxy to external systems, identifying and deidentifying on-the-fly. It would be useful to know the status of downstream systems to provide insight to their performance and availability. Principally, REDCap is a system that is heavily used and which sometimes experiences degraded performance.
Consider the use of Celery to establish a status check queue. Storing the results to redis for easy retrieval.

We're monitoring redcap status as of c20af66. Will investigate the monitoring of other external systems in their own issues. Closing.


Find and replace with regular expressions extremely buggy

Issue Type: Bug
I really love VS Code. But for this one buggy feature. I'm not sure it's just me or others too, find and replace with regular expressions is extremely buggy. It is the only bad thing for me with VS Code. The options just don't work according to the documentation. Like for example, you select a block of text and want it to find and replace within the selection, but it replaces in the entire document. This sometimes just keeps happening frustrating me. Can you please check and fix?
VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:33:29.235Z)
OS version: Darwin x64 16.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz (4 x 2500)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 4

Memory (System)
16.00GB (0.98GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (26)

Extension
Author (truncated)
Version

vscode-tomcat
ada
0.10.0

vscode-database
baj
1.5.2

pgformatter
bra
1.3.0

sql-formatter
cym
1.0.2

java-decompiler
dgi
0.0.2

xml
Dot
2.3.1

vscode-install-vsix
fab
1.1.2

gc-excelviewer
Gra
2.1.25

java-properties
ith
0.0.2

vscode-github
Kni
0.30.0

rainbow-csv
mec
0.4.2

anaconda-extension-pack
ms-
1.0.0

python
ms-
2018.7.1

vscode-docker
Pet
0.1.0

vscode-spring-boot
Piv
0.4.0

java
red
0.29.0

vscode-yaml
red
0.0.14

vscode-icons
rob
7.24.0

vscode-java-debug
vsc
0.10.0

vscode-java-pack
vsc
0.3.0

vscode-java-test
vsc
0.7.1

vscode-maven
vsc
0.10.0

vscode-spring-initializr
vsc
0.3.0

vim
vsc
0.15.7

markdown-pdf
yza
1.2.0

markdown-all-in-one
yzh
1.6.0

(2 theme extensions excluded)

Let's track this issue in https://github.com/Microsoft/vscode/issues/44702


To find paddle serving model

Where can I find pyramid_box_lite_server_mask to be able to use with  paddle_serving_server.serv

we don't have the model in our model zoo. but you can transfer the model yourself with https://github.com/PaddlePaddle/Serving/blob/develop/doc/INFERNCE_TO_SERVING.md


GLTF Morph Targets

What problem does this solve or what need does it fill?
Bevy supports skeletal animation but not morph-target animation. Morph target animation is widely used for face animation rigs and is actually much simpler mathematically than skeletal animation, but requires a larger amount of memory.
What solution would you like?
Bevy should (in order of criticality):

Load GLTF mesh primitive's morph targets into vertex attributes CPU-side
Load the initial weights stored on GLTF Mesh nodes into a bevy mesh struct
** Morph target names tend to be stored in "extras"."targetNames" in GLTF files
** Providing an API for enumerating available targets would be very useful
Provide a CPU-side API for blending Meshes with morph targets (then up to users to update in the scene)
** This can be useful for raycasting and cpu-side scene exporting etc even when GPU-morphing is implemented
Provide GPU-side morph blending with the help of a shape gains uniform on Mesh objects

What alternative(s) have you considered?
An outside plugin to handle this would - as far as I understand it - require duplicating much of the work the existing GLTF loader does. Once loading is possible, CPU blending could be a third-party plugin I am sure.
For integration into the rendering pipeline (GPU-side blending) and other plugins, I think Bevy having an understanding of morph targets right down in it's Mesh primitives will prove very valuable.
Additional context
I'd be quite interested to have a tinker into some work on this front - however guidance into the intended APIs around Mesh/SkinnedMesh would help me be confident to open a PR for at least part of this work. I don't think I have the knowledge or time to try and integrate into the shaders and standard pipeline properly.
three.js for instance supports morph targets as part of it's root Mesh class: Mesh.morphTargetInfluences and supports CPU morphing BufferGeometryUtils.computeMorphedAttributes despite generally performing morphing in shaders:  morphtarget_vertex.glsl.js

Since opening this I see: https://github.com/bevyengine/bevy/pull/3722 and https://github.com/bevyengine/bevy/issues/4026 which more or less cover this. I'd be inclined to keep this issue open to specifically mention morph targets due to the wide scope of #4026 but I'm happy to close as duplicate if that's easier for the maintainers.


Should multi-select values be selectable?

hey! this incredibly minor issue was raised by someone here, but i'm happy to contribute a fix if it's something that should be fixed:
If you have a multiselect (eg, the top select on https://vue-multiselect.js.org/) and add a value, it's not possible to highlight the value to copy and paste it, even though it looks a bit like it should be possible because it's a text element not an input.
Steps to replicate:

Add "License" to multiselect
Try to copy "License" to be pasted elsewhere

Should this be possible? It would reduce the size of the area where you can click to active the select (or maybe it's possible to detect highlighting?), but it would make copying and pasting values from the multiselect possible.

I would need a second look on that - the issue might be because of amousedown events rather than click that is triggering the multiselect. If this is to be changed I think we would also need to modify the rest of the elements and how the component manages the active state (currently through focus). Would you like to take a look at the current v3? The implementation of the active state changed there and I wonder what's your opinion on that both regarding this issue and possible further accessibility improvements.


All controllers pass logger in the ctx

Is there a related GitHub Issue?
#1935
What is this change about?
This PR updates the org and space controllers to pass the logger to helper functions via the context.
Does this PR introduce a breaking change?
No
Acceptance Steps
Deploy and run tests.
Tag your pair, your PM, and/or team
@kieron-dev

So we could ignore the error / use the discard logger function. But then we might miss when we've broken the context wiring. I suppose introducing assertions in tests that particular messages are logged would solve that, though I'm not sure people are generally keen about testing log messages.

@kieron-dev I've switched to using the discard function and have added a basic logging test for each controller. Let me know what you think.

The new pattern looks good. I think the log tests should test one or more of the logs that actually take the logger from the context though. The observed generation log always seems to come from a logger which has no chance of being a discard logger.

Good point. I've updated the tests that can assert on log messages from helper functions to use them. Unfortunately, some of the controllers don't log anything in the helper functions. Should we add a debug log line to them so that we can test that everything is wired up properly?


Added distribution build scripts.

Signed-off-by: dblock<EMAIL_ADDRESS>Description
Alternative to #402 in support of https://github.com/opensearch-project/opensearch-build/pull/1957
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/1950
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Close this as it is resolved in opensearch-project/opensearch-build#1957 opensearch-project/opensearch-build#1959

See https://github.com/opensearch-project/opensearch-build/pull/1959#issuecomment-1097517420. I'll reopen.


Depth Chart breaks after zooming in and out

I seem to be able to break the order book by scrolling to the limit (i.e. zooming all of the way in and all of the way out).

We've talked about coming up with a heuristic to decide the initial zoom level before. There's no obvious right way to do it so let me come up with something and we can see how it behaves

Fairly sure this is fine now


[Bug Report] Queue loosing tracks after a while

Describe the bug
I noticed that my Musicbot is loosing some tracks after some hours. In this case 3 of 24 songs where dropped
To Reproduce
I have the bot always on and tried songs from a yt playlist or single songs one after another.
Expected behavior
Songs stay in the playlist :-)
Debug Info
HW: pi4b 4gb
SW:
Linux raspberrypi 5.10.17-v8+ #1421 SMP PREEMPT Thu May 27 14:01:37 BST 2021 aarch64 GNU/Linux
openjdk version "11.0.11" 2021-04-20
OpenJDK Runtime Environment (build 11.0.11+9-post-Debian-1deb10u1)
OpenJDK 64-Bit Server VM (build 11.0.11+9-post-Debian-1deb10u1, mixed mode)
Starting parameters:
nohup java -Dnogui=true -jar JMusicBot-0.3.4.jar &
nohuplog.txt

It looks like some tracks just failed to play. There are a multitude of reasons that could cause this, such as:

a temporary drop in the bot's internet connection
youtube temporarily blocking the bot or preventing a video from loading
request timeouts
This is fairly normal behavior. A newer release of JMusicBot may help with this, but occasionally tracks having issues is to be expected.


Re-enable skipped notebook test

https://github.com/microsoft/vscode/blob/806c22dcc9134964aa999be0b46790e24c93893e/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts#L897
I skipped this when enabling the new markdown renders

The issue seems to be timing related. If I add a delay:
diff --git a/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts b/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts
index 935e3e8a57f..557f8694ee4 100644
--- a/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts
+++ b/extensions/vscode-api-tests/src/singlefolder-tests/notebook.test.ts
@@ -1106,7 +1106,7 @@ suite('Notebook API tests', function () {
                await saveAllFilesAndCloseAll(secondResource);
        });

-       test.skip('multiple tabs: different editors with same document', async function () {
+       test.only('multiple tabs: different editors with same document', async function () {
                const resource = await createRandomFile('', undefined, '.vsctestnb');
                await vscode.commands.executeCommand('vscode.openWith', resource, 'notebookCoreTest');
                const firstNotebookEditor = vscode.window.activeNotebookEditor;
@@ -1116,6 +1116,7 @@ suite('Notebook API tests', function () {

                await splitEditor();
                const secondNotebookEditor = vscode.window.activeNotebookEditor;
+               await new Promise(resolve => setTimeout(resolve, 500));
                assert.strictEqual(secondNotebookEditor !== undefined, true, 'notebook first');
                assert.strictEqual(secondNotebookEditor!.selection?.document.getText(), 'test');
                assert.strictEqual(secondNotebookEditor!.selection?.document.languageId, 'typescript');

then the test pass again.
I suspect this has something to do with some of the awaits we added to make sure the webview is resolved before showing it


feature(offers view): sponsors info page

Resolves #616
What does this PR do?
Info page mobile version
How should this be tested?

Make run
Go to sponsor info page

@Edgar097  Can't check this pr until #640 is solved


waffle.io Badge

Merge this to receive a badge indicating the number of issues in the ready column on your waffle.io board at https://waffle.io/activenetwork-automation/vcoworkflows
This was requested by a real person (user gkra) on waffle.io, we're not trying to spam you.

Coverage remained the same at 79.41% when pulling 0fd0245ac6670f608ba97224948861ee31e19408 on waffle-iron:master into 5a369efc2b67251c88e28155569e92842db3a29d on activenetwork-automation:master.

this should be ok to merge once #18 is merged.

depends on #18


Removing packages since they can be restored

@paolosalvatori any particular reason why this is not merged? There's really no reason to keep packages checked in. Thanks.

Yes, I was busy and I couldn't merge your code before today ;)

All good, thank you :smiley:


examples: unpinned dependencies

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

Also the pidstore should be added to the dependencies (as in my PR https://github.com/inveniosoftware/invenio-marc21/pull/11/files#diff-86d2f29b7151f1dfb6b158f5f495ba35R37).

@switowski I have added tests for example app. If you see any problem please let me know.

@jirikuncar good idea with tests for the app! The code LGTM (I see that the pidstore  was actually added in one of the commits), I will merge it after travis gives the :green_heart:

@switowski :shipit:

When testing the example app, npm tries to write to the system-wide /usr/lib and /usr/bin places.  Could be changed perhaps?

mock is needed for tests and is missing in the test requirement list. Can you add it?

Otherwise LGTM.

When testing the example app, npm tries to write to the system-wide /usr/lib and /usr/bin places.  Could be changed perhaps?

I don't know how.

mock is needed for tests and is missing in the test requirement list. Can you add it?

Added.

Thanks. One more thing: let's add tests/test.db to .gitignore.

Thanks. One more thing: let's add tests/test.db to .gitignore.

IMHO it is not related with this PR but I have added.

IMHO it is not related with this PR but I have added.

Yeah, it's just a clean-up for the past PRs... likemock, hence together.  Thanks.


Signup /setup flow not working

I followed the instructions to get up and running the CI, but it haven't take me through a flow to set up a new CI user.

Oh that's interesting. We're actually working on onboaring using the new frontend. Have you used the FASTLANE_CI_ERB_CLIENT ENV variable to use the legacy front-end for onboarding?

@KrauseFx No I haven't use that ENV variable. As soon as I ran the CI on localhost:8080 it prompted me with the login fields. I already set up the angular frontend as explained in the README. I don't know if it has something to do with but I had Jenkins running as a Daemon listening on port 8080.

Yeah makes sense. Sorry we haven't finished onboarding on the new frontend yet. For now, can you launch the server using
FASTLANE_CI_ERB_CLIENT=1 bundle exec rackup

and see if that works (for onboarding). Once you set fastlane.ci up you can remove that ENV variable again


Capped market notional value overridden if not press enter

Description
Manually choosing a notional value on a capped market does not work, it uses whatever is on the slider.
Steps to Reproduce

Go to a capped market and choose a notional value and long/short

Expected behavior
See that the value of the order is the one from the slider and not what you entered.
Screenshots
https://github.com/vegaprotocol/frontend-monorepo/assets/100769129/ba3ef356-882f-47cb-85e9-64d99f80a4cd

I think you're misunderstanding what is being shown.
The notional is being set in the ticket but the values in the order book and the submit button are still the SIZE that is being submitted on the order.  We derive that size from the notional but will never match the notional ... unless of course price = 1.
There are some bugs around here still which we are working through, but if I'm interpreting this video correctly I think the above is the issue.

Dan has confirmed that this is working now


IpVersions/UdpListenerImplTest.UdpGroBasic/IPv4 flaked

IpVersions/UdpListenerImplTest.UdpGroBasic/IPv4 failed recently in an (unrelated) PR
[ RUN      ] IpVersions/UdpListenerImplTest.UdpGroBasic/IPv4
test/common/network/udp_listener_impl_test.cc:535: Failure
Actual function call count doesn't match EXPECT_CALL(os_sys_calls, recvmsg(_, _, _))...
         Expected: to be called at least once
           Actual: never called - unsatisfied and active
Stack trace:
  0x559e51b74dd3: testing::internal::GoogleTestFailureReporter::ReportFailure()
  0x559e4f3f8d8b: testing::internal::Expect()
  0x559e51b85c1f: testing::internal::UntypedFunctionMockerBase::VerifyAndClearExpectationsLocked()
  0x559e4f64a4b8: testing::internal::FunctionMocker<>::~FunctionMocker()
  0x559e4f63d0ca: Envoy::Api::MockOsSysCalls::~MockOsSysCalls()
  0x559e4f3deba9: Envoy::Network::(anonymous namespace)::UdpListenerImplTest_UdpGroBasic_Test::TestBody()
  0x559e51b69e40: testing::internal::HandleSehExceptionsInMethodIfSupported<>()
  0x559e51b64f71: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x559e51b4be56: testing::Test::Run()
  0x559e51b4c82d: testing::TestInfo::Run()
... Google Test internal frames ...

test/common/network/udp_listener_impl_test.cc:610: Failure
Actual function call count doesn't match EXPECT_CALL(listener_callbacks_, onReadReady())...
         Expected: to be called once
           Actual: never called - unsatisfied and active
Stack trace:
  0x559e51b74dd3: testing::internal::GoogleTestFailureReporter::ReportFailure()
  0x559e4f3f8d8b: testing::internal::Expect()
  0x559e51b85c1f: testing::internal::UntypedFunctionMockerBase::VerifyAndClearExpectationsLocked()
  0x559e4f4a148a: testing::internal::FunctionMocker<>::~FunctionMocker()
  0x559e4fbc8bd6: Envoy::Network::MockUdpListenerCallbacks::~MockUdpListenerCallbacks()
  0x559e4f409e4e: testing::NiceMock<>::~NiceMock()
  0x559e4f3fd0df: Envoy::Network::UdpListenerImplTestBase::~UdpListenerImplTestBase()
  0x559e4f3d3a37: Envoy::Network::(anonymous namespace)::UdpListenerImplTest::~UdpListenerImplTest()
  0x559e4f3e672b: Envoy::Network::(anonymous namespace)::UdpListenerImplTest_UdpGroBasic_Test::~UdpListenerImplTest_UdpGroBasic_Test()
  0x559e4f3e6754: Envoy::Network::(anonymous namespace)::UdpListenerImplTest_UdpGroBasic_Test::~UdpListenerImplTest_UdpGroBasic_Test()
  0x559e51b5ea50: testing::Test::DeleteSelf_()
  0x559e51b69e40: testing::internal::HandleSehExceptionsInMethodIfSupported<>()
  0x559e51b64f71: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x559e51b4c881: testing::TestInfo::Run()
  0x559e51b4d0e9: testing::TestSuite::Run()
... Google Test internal frames ...

test/common/network/udp_listener_impl_test.cc:611: Failure
Actual function call count doesn't match EXPECT_CALL(listener_callbacks_, onData(_))...
         Expected: to be called at least once
           Actual: never called - unsatisfied and active
Stack trace:
  0x559e51b74dd3: testing::internal::GoogleTestFailureReporter::ReportFailure()
  0x559e4f3f8d8b: testing::internal::Expect()
  0x559e51b85c1f: testing::internal::UntypedFunctionMockerBase::VerifyAndClearExpectationsLocked()
  0x559e4fbebd58: testing::internal::FunctionMocker<>::~FunctionMocker()
  0x559e4fbc8bf6: Envoy::Network::MockUdpListenerCallbacks::~MockUdpListenerCallbacks()
  0x559e4f409e4e: testing::NiceMock<>::~NiceMock()
  0x559e4f3fd0df: Envoy::Network::UdpListenerImplTestBase::~UdpListenerImplTestBase()
  0x559e4f3d3a37: Envoy::Network::(anonymous namespace)::UdpListenerImplTest::~UdpListenerImplTest()
  0x559e4f3e672b: Envoy::Network::(anonymous namespace)::UdpListenerImplTest_UdpGroBasic_Test::~UdpListenerImplTest_UdpGroBasic_Test()
  0x559e4f3e6754: Envoy::Network::(anonymous namespace)::UdpListenerImplTest_UdpGroBasic_Test::~UdpListenerImplTest_UdpGroBasic_Test()
  0x559e51b5ea50: testing::Test::DeleteSelf_()
  0x559e51b69e40: testing::internal::HandleSehExceptionsInMethodIfSupported<>()
  0x559e51b64f71: testing::internal::HandleExceptionsInMethodIfSupported<>()
  0x559e51b4c881: testing::TestInfo::Run()
  0x559e51b4d0e9: testing::TestSuite::Run()
... Google Test internal frames ...

[  FAILED  ] IpVersions/UdpListenerImplTest.UdpGroBasic/IPv4, where GetParam() = 4-byte object <00-00 00-00> (397 ms)

https://dev.azure.com/cncf/envoy/_build/results?buildId=147448&view=logs&jobId=8bf29878-a4cc-50f7-4e84-2255e6fd4065&j=8bf29878-a4cc-50f7-4e84-2255e6fd4065&t=10423dec-52d0-5521-c797-f153bdaa8e53

Looks like this was @yugantrana but he hasn't been involved for a while.  @danzh2010 can you either pick this up or suggest someone who can?

I'm not able to repro this locally, proposed a potential fix in https://github.com/envoyproxy/envoy/pull/29855.


feat: store window size and forecast horizon in dataset

Summary of Changes
Window size and forecast horizon are no longer specified in the input conversion but in the time series dataset.
The intuition is that a dataset describes

what to predict (target, forecast horizon),
with what (time, features, window size).

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

v0.26.0
GitHub release

Your semantic-release bot :package::rocket:


Autoinject: support 1.1 labels

For upgrade/migration, add a flag to allow one 'environment' to support the 1.1 injection labels.
This will allow a smooth upgrade.
istio-env label allows namespaces to target a specific environment.

@costinm in how for do we need this in the installer? I'd suppose for the upgrade case there will be an Istio 1.1 installed with the 1.1 helm charts and hence supporting 1.1 injection labels...


Streamz + Panel Plotting Issue

Im trying to plot a simple chart with streamz + panel in jupyter lab but there seems to be an issue at the end: there is an extra line that goes back to the starting point. This is the code that I wrote and the plot that I get:
beacon_sdf=sDataFrame(example=pd.DataFrame({'interval_duration_sec':[],'sim_time_sec':[]},index=pd.DatetimeIndex([])))

def beacon_Line(beacon_sdf):
    return beacon_sdf.hvplot.line('sim_time_sec','interval_duration_sec')

beacon_Line(beacon_sdf)

hvplot_stream = beacon_sdf.stream.map(beacon_Line)

hvplot_pane = pn.pane.Streamz(hvplot_stream, height=350, always_watch=True,widgets={'Simulation Time(sec)': pn.widgets.FloatSlider})

def emit():
    for index,rows in beacon_df.iterrows():
        beacon_sdf.stream.emit(pd.DataFrame({'interval_duration_sec':rows['interval_duration_sec'],'sim_time_sec':rows['sim_time_sec']},index=pd.DatetimeIndex([pd.datetime.now()])))
        sleep(0.01)

hvplot_pane.add_periodic_callback(emit,period=500,timeout=2)

Does anyone have any thoughts? The source data is not the problem.

cc @philippjfr

@philippjfr could you help on this? Thanks!


Add "Deactivate" to enterprise API, for testing

Note that enterprise.Deactivate also calls DeleteAll(), to avoid invalid cluster states (e.g. auth on but enterprise state = NONE)

Get a verbal LGTM. I'll submit this when it passes CI


python3Packages.hg-commitsigs: init at unstable-2021-01-08

A Mercurial extension for signing changesets.
If people want to be able to use this extension they have to manually
set their PYTHONPATH to
/nix/store/...-profile/lib/python3.8/site-packages/hgext3rd.  It
would be nice if we could make the Mercurial package automatically
read things from this directory without any intervention from the
user.
CC: @lukegb

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Sorry for the excessively slow review - I didn't see this in my GH notifications inbox and I junked all my email at the time.
Since this PR, I've added a "withExtensions" mechanism to the Mercurial derivation to support loading things like this; it works with hg-evolve for instance, but not with this one, because this doesn't use the standard Python building mechanism so it doesn't export the attributes it expects.
If you're still interested in this, would you mind switching this to use the standard Python buildPythonPackage thing?
You should be able to test it's working with e.g.
nix-shell -p '((import ./. {}).mercurial.withExtensions (pm: [ pm.hg-commitsigs ]))' --run "hg --config extensions.commitsigs= help verifysigs"

This has to be moved out of python-packages.nix.

Never mind, it just has to use the correct interpreter: https://github.com/NixOS/nixpkgs/pull/172975


Elaborate on managing Conda python, channels

This pull request addresses the requirements outlined in Assignment 2 (A2) for the course CSC301 - Introduction to Software Engineering, submitted by Yuxuan Mu. The focus of this assignment is on managing Conda environments effectively.

Hi Yuxuan, I fogot to add citations for my pr #600, it's already merged. Could you add the citation section for me.
References

Conda Official Documentation. Available at: Conda Documentation
Existing work and examples can be found in the Conda README directory within the repository. For specific examples and configurations, refer to the Conda README in this repository.

will do that right now. Thank you for feedback.

I'd suggest to edit the reference list to keep consist with the other documents for Conda in the repo

Also, the reference list is needed for each .md file for the consistent aim.


Manage recipes (Archival)

manage (view all recipes) grid styling updated

scrollbar, inset shadows, quality of life
added title and action bar to the grid to enable custom table controls

added simple selection control to the grid (checkboxes)

adds action links to add, archive, and view archived in the grid

adds IArchivable

interface implemented by Recipe
added IQueryable extensions and updated the repo to filter by archived recipes
adds ef migration

added EmptyGuidGuard for repo actions with bulk operations


Styling

How should styling be done?

Now you can do that by adding a custom class and you can also disable bootstrap
Closing #2


Missing access modifier for ApmScopeManager class

Describe the bug
I tried manually activating a span.
When calling new ElasticApmTracer().scopeManager().activate(span, finishSpanOnClose) I get a compile time error (see details below).
To Reproduce

Copy the above code (In another package than co.elastic.apm.opentracing).
Run the java compiler

Expected behavior
The code should compile fine.
Debug logs
Compile time error, so no logs to attach.

Click to expand
[error] /path/to/ElasticApmTracingProvider.java:64: Unable to emit reference to method activate in class ApmScopeManager, class ApmScopeManager is not accessible in class ElasticApmTracingProvider
[error]     tracer.scopeManager().activate(span, finishSpanOnClose)

In my opinion, this should be accessible for API users, but I could be wrong.

Hi and thanks for the issue!
You should only call new ElasticApmTracer() during initial initialization and thereafter refer to the io.opentracing.Tracer interface. For example, you can create a bean of that type or use the io.opentracing.util.GlobalTracer to register the ElasticApmTracer. When actually using the tracer, you can just call io.opentracing.util.GlobalTracer#get, which returns a io.opentracing.Tracer.
But to make a working example for this one-liner, call ((io.opentracing.Tracer) new ElasticApmTracer()).scopeManager().activate(span, finishSpanOnClose)

Thanks for clearing this up, works as expected! :+


PR #427 introduced invalid json

You can see here that the JSON parsing failed https://travis-ci.org/Showndarya/Hacktoberfest/builds/438744297
The reason is that you cannot have a trailing comma in JSON, only Javascript objects.
Also there should be a warning about the lowercase filename but that's another issue

@jukefr I corrected that. It is a success now. Thanks a lot for the help.


doc: update all YAML blocks

I noticed a lot of the YAML blocks are outdated for example in their indentation, but maybe also in their actual contents?

[  ] Look for ```yaml code blocks throughout the docs and verify they are correct by reproducing the example commands they come from.

updating all the YAML code blocks. @shcheklein @jorgeorpinel

Cool @ryokugyu, thanks! Just open a PR and reference this issue with Fixes #480 in the PR description when ready, so we can all help review the changes.

Cool @ryokugyu, thanks! Just open a PR and reference this issue with Fixes #480 in the PR description when you have changes you'd like us to help review.
already did this. This is the link for PR.

p.s. I guess you can also assign yourself here?
nope.

Let's go this as we go. No way we can do this at once.


proc: bugfix: Truncate stacktrace when FDE of a frame can not be found

Instead of returning an error when FDE of a frame can not be found,
just truncate the stack trace.
Fixes #462
I also did a bit of refactoring of the stacktrace code:

made GoroutineStacktrace a method of struct G
made stacktrace a method of StackIterator
renamed StackIterator to stackIterator
factored out logic to obtain a stackIterator from a goroutine that's
used by both (*G).Stacktrace and by (*G).UserCurrent

@aarzilli what's the underlying cause of this issue? Why do we encounter PCs we cannot reconcile with an FDE? I haven't dug into this, but is it a bug in our FDE lookup, or in the information generated?

runtime.mcall (which happens to be at the bottom of the stack in the related issue) is apparently one of those functions that never returns. In general I think it's better to show a truncated stack trace rather than a error message when we can't decode a stack frame.

@aarzilli makes sense, but in that case we should print a warning that the stack trace has been truncated. Additionally, I think that runtime.mcall should be considered a "top of stack" function, which should signify that we can stop collecting frames.

Ok, I've changed proc.(*stackIterator).Next so that it will return an extra frame with File "?" and Line -1 for all undecodable frames except when the undecodable frame is the top of the stack (in that case we lose no informations in returning the error directly).
A truncated stack trace will look like this:
(dlv) stack
0  0x000000000045d6a9 in runtime.epollwait
   at /usr/local/go/src/runtime/sys_linux_amd64.s:440
1  0x0000000000428acc in runtime.netpoll
   at /usr/local/go/src/runtime/netpoll_epoll.go:67
2  0x00000000004323e3 in runtime.findrunnable
   at /usr/local/go/src/runtime/proc.go:1955
3  0x0000000000432b7a in runtime.schedule
   at /usr/local/go/src/runtime/proc.go:2072
4  0x0000000000432e16 in runtime.park_m
   at /usr/local/go/src/runtime/proc.go:2137
5  0x0000000000459b09 in runtime.mcall
   at /usr/local/go/src/runtime/asm_amd64.s:233
6  0x000000000091e800 in (nil)
   at ?:-1

Additionally I have added runtime.mcall to the "top of stack" functoins.

Found a bug while investigating issue #468 (stacktrace returned 0 stack frames and no errors because PCToLine didn't find anything), I've added a commit to address it.

I've disabled the test on windows since there are no stack frames to print.

Merged.


Update to 1.7.6

Changelog
=========

1.7.6
-----

Changes:
 - Improvement: sqldiff, ignore proxy models in diff (with cli option to include them if wanted)

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.


Auto relist all items on transferlist - show how many times it has been listed and not sold

READ BEFORE YOU ADD AN ISSUE: Do not request autobuyer features, because they are considered as cheating. Issues regarding autobuyers will be closed immediately.
Expected behavior:
Give a detailed explanation of the expected behavior.
Current behavior:
Give a detailed explanation of the current behavior.
Metadata:

Script version:
Browser:
OS:

Duplicate of #42
Automatically relisting is unwanted behavior because it could be considered as 'botting'.


Initial implementation of igor plugin

This allows igor to monitor given git repositories (GitHub only for now). This functionality may need to move to clouddriver in the future, but moving it shouldn't be too difficult.

@nabuskey can we close this PR?


Allow providing secrets as paths

I've been using sops-nix to manage my secrets, but I cannot use the same pattern with this flake, since all the configuration values that I would like to keep secret (serverName, password, permittedList, adminList) must be specified directly.
It would be great if we could specify all of these values indirectly, either through environment variables or using file paths.

The only piece of information that really needs to be kept secret is the server password, and upstream makes this impossible by only taking this as a commandline argument (unless there is some undocumented way of putting it in a configuration file).

Fair enough. Could we use a .env file instead? I've seen this pattern used for a flake like restic.

The only piece of information that really needs to be kept secret is the server password

You are right, although it couldn't hurt to be able to keep the name of the server and player lists private too. Less important obviously.

Okay, I think I found a way that doesn't require any changes here: https://github.com/pborzenkov/nix-config/blob/09c522c49baca298211eb7414ef5cd3025abe474/nixos/machines/rock/valheim.nix#L2.
Thank you! I'll close this now.


Wrong inferred initial value in reduce

🔎 Search Terms
"reduce initial value", "wrong inferred initial value in reduce"
🕗 Version & Regression Information

This changed between versions ______ and _______
This changed in commit or PR _______
This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________
I was unable to test this on prior versions because _______

⏯ Playground Link
https://www.typescriptlang.org/play/?#code/MYewdgzgLgBGCuBbARgUwE4QILvQQwE8YBeGAbQEYAaGeMAE1QDMBLMVemgJhoGYa6jVu04wArAF0AsAChZAG1Sw2LKCzzyAahvioAXHCRp0JGAAZZs0JFgQkiPOiKkEKDNlyEAdOg7xgqAAUgXjAwDQAbjqoAJQkAHwwAN6yMGkwvlDw6GAwocAwANQwgVHyujAA-JXmMbIAvjQqahra5bFAA
💻 Code
const numbersArray = [1, undefined, 2, 3, undefined, 5]

let initialValue: number = 0

const summary = numbersArray.reduce((acc, value) => {
    return acc + (value ?? 0)
}, initialValue)

🙁 Actual behavior
acc is number | undefined instead of number
🙂 Expected behavior
Since an initialValue is provided the type of acc should be inferred from it and not from the first element of the numbersArray.

A value to which accumulator is initialized the first time the callback is called. If initialValue is specified, callbackFn starts executing with the first value in the array as currentValue.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce

How does TypeScript infer the type of the accumulator in reduce()?
TypeScript infers the type of the accumulator based on the initial value provided (if any) and the return type of the reducer function. If no initial value is provided, it infers the type from the first element of the array.
https://www.geeksforgeeks.org/typescript-array-reduce-method/

Additional information about the issue
No response

I believe this is working as intended.  reduce() has several overloads, and the one used does not have an additional type argument for the initialValue because the provided value is compatible with your array type.
A workaround is to explicitly choose the overload with the type argument by providing it: numbersArray.reduce<number>(...)


Allow negative indices in Concat/Split ops

@caffe2bot test this please

FYI master is broken, this looks good.

@caffe2bot retest this please


[eas-cli] Makes eas.json configuration to only run on update:configure

Checklist

[x] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
This PR is a bug fix. I landed a PR where when running eas update:configure, we set the channel property in build profiles. However, I placed that logic in a place that is also called when running eas update .... (We want to set up expo-updates and configure app.json if needed when running eas update)
This is not the desired behavior for this configuration however. We only want to set up channels in build profiles when a developer is running eas update:configure. By removing this from the eas update command, it allows developers to set a channel in their native code, then use EAS Build, and not have their native channel overwritten.
How

Takes the ensureEASUpdateIsConfiguredInEasJsonAsync() function out of the shared configuration and call it in the update:configure command.

Test Plan

Create an app with npc create-expo-app myApp
Run eas build:configure
Run eas update. Expect that there will be no channels configured in eas.json, while app.json will be configured.
Run eas update:configure. Expect that eas.json is configured with channels.

/changelog-entry bug-fix Makes eas.json configuration to only run on update:configure

Merging because I got two checkmarks and I think I answered @wkozyra95's question adequately. If you still think that this is an issue, please let me know and we can discuss/revert


Revert "jsk_apc: 0.8.0-1 in 'indigo/distribution.yaml' [bloom]"

Reverts ros/rosdistro#11696

@wkentaro fyi


Brunch-Built module has local dependency issues when imported in another project

Description
I'm working on a Vue plugin called Animated Vue and I'm building my module with brunch. I've installed it into another project I had already working with brunch, and when I try to build it I get errors such as the following:
14:44:23 - error: Processing of node_modules/animated-vue/dist/animated-vue.js failed. Error: Could not load module './bounce' from '/Users/rjuliani/dev/wannaworkthere/application/node_modules/animated-vue/dist'. Make sure the file actually exists.

Expected behavior
The build process completes successfully and does not have issues with animated-vue's internal dependencies.
Actual behavior
I get an error where Brunch can't find a dependency that's internal to animated-vue, and is already inside dist/animated-vue.js
You can check out animated-vue's repo and my project can be found here
Environment

Brunch: 2.10.8
Node.js: 6.9.5
NPM: 3.10.10
Operating system: MacOSX
Code editor: Atom

package.json contents
animated-vue
{
  "name": "animated-vue",
  "version": "0.1.2",
  "description": "A set of transitions for Vue2 based on animate.css",
  "main": "dist/animated-vue.js",
  "scripts": {
    "build:dev": "brunch b",
    "build:production": "brunch b --production",
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/radical-dreamers/animated-vue.git"
  },
  "keywords": [
    "Vue",
    "animate.css",
    "es6",
    "javascript",
    "Vue2",
    "transitions"
  ],
  "author": "Rodrigo Juliani<EMAIL_ADDRESS>  "license": "MIT",
  "bugs": {
    "url": "https://github.com/radical-dreamers/animated-vue/issues"
  },
  "homepage": "https://github.com/radical-dreamers/animated-vue#readme",
  "devDependencies": {
    "babel-brunch": "^6.0.6",
    "babel-eslint": "^7.1.1",
    "babel-plugin-transform-runtime": "^6.22.0",
    "babel-preset-es2015": "^6.22.0",
    "babel-preset-es2016": "^6.22.0",
    "babel-preset-latest": "^6.22.0",
    "babel-preset-stage-3": "^6.22.0",
    "brunch": "^2.10.6",
    "eslint-brunch": "^3.12.0",
    "uglify-js-brunch": "^2.10.0",
    "vue-brunch": "^2.0.0"
  },
  "dependencies": {
    "vue": "^2.1.10",
    "animate.css": "^3.5.2"
  }
}

Project
{
  "name": "wannaworkthere",
  "version": "1.0.1",
  "description": "UI for the wannaworkthere package",
  "main": "public/app.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "start": "brunch w --server",
    "build:local": "brunch b",
    "build:production": "PRODUCTION=true brunch b --production"
  },
  "repository": {
    "type": "git",
    "url": "git+https://gitlab.com/srodriki/wannaworkthere.git"
  },
  "keywords": [
    "vue",
    "brunch",
    "vuex",
    "vue-router"
  ],
  "author": "Rodrigo Juliani<EMAIL_ADDRESS>  "license": "MIT",
  "bugs": {
    "url": "https://gitlab.com/srodriki/wannaworkthere/issues"
  },
  "homepage": "https://gitlab.com/srodriki/wannaworkthere#README",
  "devDependencies": {
    "babel-brunch": "^6.0.6",
    "babel-plugin-transform-runtime": "^6.22.0",
    "babel-preset-es2015": "^6.22.0",
    "babel-preset-es2016": "^6.22.0",
    "babel-preset-latest": "^6.22.0",
    "babel-preset-stage-3": "^6.22.0",
    "brunch": "^2.10.6",
    "copycat-brunch": "^1.1.0",
    "process-env-brunch": "^1.4.5",
    "sass-brunch": "^2.10.4",
    "vue-brunch": "^2.0.0"
  },
  "dependencies": {
    "animate.css": "^3.5.2",
    "animated-vue": "^0.1.2",
    "axios": "^0.15.3",
    "bulma": "^0.3.2",
    "font-awesome": "^4.7.0",
    "izitoast": "^1.1.1",
    "moment": "^2.17.1",
    "store": "^1.3.20",
    "vee-validate": "^2.0.0-beta.25",
    "vue": "^2.1.10",
    "vue-router": "^2.2.0",
    "vuex": "^2.1.2"
  }
}

brunch config contents
animated-vue

module.exports = {
  paths: {
    public: 'dist',
    watched: ['src']
  },
  files: {
    javascripts: {
      joinTo: 'animated-vue.js'
    },
    stylesheets: {
      joinTo: 'animated-vue.css'
    },
    templates: {
      joinTo: 'animated-vue.js'
    }
  },
  plugins: {
    babel: {
      presets: ['latest', 'stage-3']
    },
    vue: {
      extractCSS: false
    },
    eslint: {
      pattern: /^src\/.*\.js?$/,
      warnOnly: true
    },
    uglify: {
      mangle: false,
      compress: {
        global_defs: {
          DEBUG: false
        }
      }
    }
  },
  npm: {
    styles: {
      'animate.css': ['animate.css']
    }
  }
}

Project
module.exports = {
  files: {
    javascripts: {
      joinTo: {
        'vendor.js': /^(?!app)/, // Files that are not in `app` dir.
        'app.js': /^app/
      }
    },
    stylesheets: {
      joinTo: 'app.css'/*{
        'vendor.css': /^(?!app)/,
        'app.js': /^app/
      }*/
    },
    templates: {
      joinTo: 'app.js'
    }
  },
  plugins: {
    babel: {
      presets: ['latest', 'stage-3']
    },
    vue: {
      extractCSS: false
    },
    sass: {
      mode: "native",
      precision: 8,
      options: {
        includePaths: [
          'node_modules/bulma'
        ]
      }
    },
    copycat: {
      'fonts': ['node_modules/font-awesome/fonts']
    }
  },
  npm: {
    styles: {
      'izitoast': ['dist/css/iziToast.css'],
      'font-awesome': ['css/font-awesome.css'],
      'animate.css': ['animate.css']
    }
  }
}

Other useful files, when present (log, bower.json etc.)
Brunch build stack
15:00:19 - error: Processing of node_modules/animated-vue/dist/animated-vue.js failed. Error: Could not load module './bounce' from '/Users/rjuliani/dev/wannaworkthere/application/node_modules/animated-vue/dist'. Make sure the file actually exists.
  at friendlyRequireError (/Users/rjuliani/dev/wannaworkthere/application/node_modules/deppack/lib/resolve.js:69:9)
  at err2 (/Users/rjuliani/dev/wannaworkthere/application/node_modules/deppack/lib/resolve.js:90:17)
  at /Users/rjuliani/dev/wannaworkthere/application/node_modules/browser-resolve/index.js:265:24
  at /Users/rjuliani/dev/wannaworkthere/application/node_modules/resolve/lib/async.js:55:18
  at load (/Users/rjuliani/dev/wannaworkthere/application/node_modules/resolve/lib/async.js:69:43)
  at onex (/Users/rjuliani/dev/wannaworkthere/application/node_modules/resolve/lib/async.js:92:31)
  at /Users/rjuliani/dev/wannaworkthere/application/node_modules/resolve/lib/async.js:22:47
  at FSReqWrap.oncomplete (fs.js:123:15)

Consider not having dist. Sources are enough.
Brunch was not supposed to be used for building modules. CommonJS bundling is not state-of-the art right now. Will fix sometime later.

@shvaikalesh
Thanks for your answer.
I was actually hoping to build my package as I wanted people not using ES6 to still be able to use it (i.e. just include it in a script tag from their html), very much like VueJS's library works.
So, if I were to leave it as src-only, am I right assuming the users would need to actually build the thing with other package builders as the only means to using this plugin?

Also, if I do as you said, I get the following error:
rocessing of node_modules/animated-vue/src/index.js failed. SyntaxError: 'import' and 'export' may only appear at the top level (5:4)
Which means brunch is somewhat not transpiling the code... for my module alone. Any ideas how to fix that?

There are a lot of people, myself included, that would prefer ES6 (because evergreen browsers/electron/phonegap) and import/export (because tree shaking). And those people would have to build it, yeah. We are trying to make it as easy as it can be with Brunch.
For <script src= use there are services like https://wzrd.in.

@shvaikalesh I understand, and totally agree. I've opened another question, in which you also commented, because I'm having issues building that module as a dependency of another web app. Maybe you can help me sort it out?
Thanks a lot!


Dependency cannot resolve

memory, fclabels and happy fail to build with stack,
reported mtl unusable due to shadow dependency of base
Using stack under macOS, brew has ghc 8.4.2 and stack has its own 8.2.2. I suppose they do not interfere with each other.
Source is fetched with git clone git://address --recursive --depth=1
--  While building custom Setup.hs for package happy-1.19.9 using:
      /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/setup/setup --builddir=.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
    Logs have been written to: /Users/soylens/Cloned/haskell-ide-engine/.stack-work/logs/happy-1.19.9.log

    [1 of 2] Compiling Main             ( /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/Setup.lhs, /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/setup/Main.o )

    /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/Setup.lhs:48:22: warning: [-Wdeprecations]
        In the use of ‘rawSystemProgramConf’
        (imported from Distribution.Simple.Program):
        Deprecated: "use runDbProgram instead"
       |
    48 |   let runProgram p = rawSystemProgramConf (fromFlagOrDefault normal (buildVerbosity flags))
       |                      ^^^^^^^^^^^^^^^^^^^^
    [2 of 2] Compiling StackSetupShim   ( /Users/soylens/.stack/setup-exe-src/setup-shim-mPHDZzAJ.hs, /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/setup/StackSetupShim.o )
    Linking /private/var/folders/rd/70fb1k0j5fg0fxlwwdhhgx580000gn/T/stack15119/happy-1.19.9/.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS>/setup/setup ...
    clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
    clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
    Configuring happy-1.19.9...
    clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
    Preprocessing executable 'happy' for happy-1.19.9..
    Building executable 'happy' for happy-1.19.9..
    <command line>: cannot satisfy -package-id mtl-2.2.2-8XubxMJDT8QLsstvlNotkc:
        mtl-2.2.2-8XubxMJDT8QLsstvlNotkc is unusable due to shadowed dependencies:
          base-<IP_ADDRESS>
        (use -v for more information)

--  While building custom Setup.hs for package fclabels-<IP_ADDRESS> using:
      /Users/soylens/.stack/setup-exe-cache/x86_64-osx/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.2.2 --builddir=.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
    Logs have been written to: /Users/soylens/Cloned/haskell-ide-engine/.stack-work/logs/fclabels-<IP_ADDRESS>.log

    Configuring fclabels-<IP_ADDRESS>...
    clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
    Preprocessing library for fclabels-<IP_ADDRESS>..
    Building library for fclabels-<IP_ADDRESS>..
    <command line>: cannot satisfy -package-id mtl-2.2.2-8XubxMJDT8QLsstvlNotkc:
        mtl-2.2.2-8XubxMJDT8QLsstvlNotkc is unusable due to shadowed dependencies:
          base-<IP_ADDRESS>
        (use -v for more information)

--  While building custom Setup.hs for package memory-0.14.16 using:
      /Users/soylens/.stack/setup-exe-cache/x86_64-osx/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.2.2 --builddir=.stack-work/dist/x86_64-osx/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
    Logs have been written to: /Users/soylens/Cloned/haskell-ide-engine/.stack-work/logs/memory-0.14.16.log

    Configuring memory-0.14.16...
    clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
    Preprocessing library for memory-0.14.16..
    Building library for memory-0.14.16..
    <command line>: cannot satisfy -package-id basement-0.0.7-IRcTCDmYoctGxt4FUYqdQc:
        basement-0.0.7-IRcTCDmYoctGxt4FUYqdQc is unusable due to shadowed dependencies:
          base-<IP_ADDRESS>
        (use -v for more information)

Any solution to this problem?

If the project does not build under stack, then it will not magically build using hie.

This is a known issue with happy. The resolution there was to install with the 64-bit version of ghc.


componentWillReceiveProps has been renamed, and is not recommended for use Warning

Its not a bug, but a warning.
I received react warning from Password Field with componentWillReceiveProps Method.
This component will be updated?
See below the warning:
Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: PasswordField
how can i solve this?

Let's keep this open until it is solved. Since you alreaedy looked up how to solve this (we need the second option here), you could submit a PR to remove the warning. :wink:

Let's keep this open until it is solved. Since you alreaedy looked up how to solve this (we need the second option here), you could submit a PR to remove the warning.
Edit: Oh wow, I didn't see that this has been open for over a month. I'll fix it asap.

Don't worry, I can solve this.
I submit a PR coming soon.


Android TV Build failed, package com.brentvatne.react does not exist

Current behavior
Describe what happens when you encounter this issue.
:app:compileDebugJavaWithJavac
:app:compileDebugJavaWithJavac - is not incremental (e.g. outputs have changed, no previous execution, etc.).
E:\Code\mmb\android\app\src\main\java\com\mmb\MainApplication.java:6: error: package com.brentvatne.react does not exist
import com.brentvatne.react.ReactVideoPackage;
                           ^
E:\Code\mmb\android\app\src\main\java\com\mmb\MainApplication.java:27: error: cannot find symbol
        new ReactVideoPackage()
            ^
  symbol: class ReactVideoPackage
2 errors
:app:compileDebugJavaWithJavac FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.

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

Reproduction steps
I have run react-native link react-native-video
rnpm-install info Linking react-native-video android dependency
rnpm-install info Platform 'android' module react-native-video has been successfully linked

Here is the package.json
{
  "name": "mmb",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "start": "node node_modules/react-native/local-cli/cli.js start",
    "test": "jest"
  },
  "dependencies": {
    "react": "16.3.1",
    "react-native": "0.55.4",
    "react-native-video": "^3.0.0",
    "react-navigation": "^2.18.0"
  },
  "devDependencies": {
    "babel-jest": "23.4.2",
    "babel-preset-react-native": "4.0.0",
    "jest": "23.4.2",
    "react-test-renderer": "16.3.1"
  },
  "jest": {
    "preset": "react-native"
  }
}

Expected behavior
Should Run as the others
Platform
Which player are you experiencing the problem on:

Android ExoPlayer

Does anyone have the same issue? Or could anyone please help with that? Thanks

This looks like a linking issue. Even though you did the link step, sometimes it won't work properly. I would do react-native unlink react-native-video and try again. Also try restarting Android Studio as it will sometimes get in a state where it doesn't pick up the dependencies properly.

Check that you have the react-native-video entry in both your settings.gradle and app/build.gradle. Since it's not working properly, you may need to add the lines manually as described in the install instructions.

This looks like a linking issue. Even though you did the link step, sometimes it won't work properly. I would do react-native unlink react-native-video and try again. Also try restarting Android Studio as it will sometimes get in a state where it doesn't pick up the dependencies properly.

what does unlinking and 'trying again' mean? yarn add? Do you delete the auto added entries in each of the files (e.g. MainActivity etc?)

"react-native": "0.55.4", I don't think this is compatible with react-native-video.
You should try to add following content in react-native.config.js from react-native-video package source:
module.exports = {
dependency: {
platforms: {
android: {
packageImportPath: 'import com.brentvatne.ReactVideoPackage;',
packageInstance: 'new ReactVideoPackage()',
},
},
},
}


feat: update github.com/choleraehyq/pid to support Go 1.18

I tried the Getting Started section, and run go run . inside the hello directory. Then I got the following error output:
# github.com/choleraehyq/pid
../../../../../pkg/mod/github.com/choleraehyq/pid@v0.0.12/pid_go1.5_amd64.s:28: expected pseudo-register; found R13
../../../../../pkg/mod/github.com/choleraehyq/pid@v0.0.12/pid_go1.5_amd64.s:29: expected pseudo-register; found R14
asm: assembly of<EMAIL_ADDRESS>failed

It used to work in the past, so I think maybe it relates to the recent update of my local Golang version from Go 1.16 to Go 1.18.
I noticed kitex-examples depend on github.com/choleraehyq/pid indirectly, the corresponding version is v0.0.12. And they began to support Go 1.18 from v0.0.13. So I update the version, and rerun go run ., it works now.
Detailed release of github.com/choleraehyq/pid could be found here: https://github.com/choleraehyq/pid/releases

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks, Kitex had upgraded this repo, when the version is released we will upgrade kitex.  But this pr I can approve first


Allow multiple $dateFormat

Laravel Version: 5.4
PHP Version: 7.1

Feature:
Wouldn't it be great if models would allow different date formats for different attributes. The $dates array can be rewritten to support attribute names as keys and formats as values.
class MultipleDateFormats extends Model
{
    protected $dates = [
        'expiration' => 'm-y',
        'birthday'   => 'm-d-Y',
        'default_date'
    ];
}

For suggestions and feature requests please use the https://github.com/laravel/internals repo.
Thanks :)


feat: add global data provider for several libraries in runtime

If there are coupple of i18n libraries with diffrent versions in runtime we have global data provides for transfering language settings between instances.

approve


interface to collect necessary stats for building a k8s readiness probe.

Describe you feature request
Is your feature request related to a problem? Please describe.
I am very new to frankenphp so could certainly have missed something in the docs that allow me to do this.
I am trying to deploy frankenphp to k8s. I would like to sanely configure readiness probes to keep instances from getting overloaded with requests. I think this would predominantly come down to having 2 key metrics:

how many of the workers are currently handling a request
how many requests are queued up waiting for workers

Additional metrics may be good:

avg delay before queued requests are dispatched?

Describe the solution you'd like
I would like to have some sort of interface (http, cli that can be invoked, etc) which would export the above and perhaps more. I would then write a script to be executed which would retrieve that info, allow for some threshold of queued requests waiting for a thread, and if above the threshold k8s would stop sending traffic to that instance until things settle a bit.
Ideally the metrics would exclude any requests that are for non-php workers (ie: static files, etc).
Perhaps the caddy admin port already has some of this data?
Describe alternatives you've considered
I can have an endpoint in my app, but that seems less than ideal as each check would itself take up a thread/worker.

Hi, indeed, we expose a Prometheus endpoint for this usage, and metrics about workers will be available in the next release: #966.

@travisghansen see: https://github.com/dunglas/frankenphp/blob/main/docs/metrics.md

Thanks, I read that in the PR but I don’t see the above mentioned stats in there..am I missing something? I am essentially looking for some direct insight into the backlog (specifically the backlog of requests that will be handled by php workers…which may be impossible to know pre-emptively, I am unsure of the internals at play).

It's possible to know, but not possible for metrics to know in any sort of stable way. For example, if your prometheus scrape happens to be between the time it gets sent to the workers and the time it gets picked up by the worker, it will be greater than zero. However, really, it is just "bad timing."
If this is fine, then we can add it. But it may flap around quite a bit and not be useful, although I guess the "average trend" could be useful.

Yeah understood. There are a lot of factors at play there that would/could dramatically. The avg time I mentioned could help mitigate the problems you have pointed out with the number…especially if direct values were exported putting the numbers in different time buckets/spans. There is definitely a lot of nuance to the numbers and which to show so further discussion is definitely warranted IMO. For example maybe a better number is how many requests could not be immediately dispatched in the last 1 seconds, 3 seconds, 5 seconds.
Having said that, the ‘current number’ is still very useful and k8s provides the basic nobs to really help with that by giving the ability to set the frequency in which the check is run and the rise/fall numbers. Using those 3 pieces it is possible to create a liveness probe which very closely mimics an average over a desired period of time.
I also think the numbers could prove incredibly valuable for really locking in hpa logic as well. Closely related to liveness but very distinct concepts.
Thanks for the consideration!

Looking at how things work, this would put metrics directly into the "hot path" of request handling, which I would like to avoid. However, we can passively detect whether workers are stalling (requests are coming in faster than workers can respond) and how bad it is.
So what about a metric like:
frankenphp_[worker]_stalled_[1,3,5]: 0-1
Where the number shows a % over the last 1-5 seconds. This number is a representation of how full the worker request buffer is. In low utilization it is always 0. Once it goes above zero, latency tends to grow exponentially in my experiments.
What do you think about that?

I think that number is fantastic! Great idea.
Which number goes into the hot path? Is it trying to gather the amount of time each of the stalled requests has been waiting?
If I understand the number correctly I think there is 1 piece of missing context which would be incredibly useful (either a separate metric or encapsulated in a different metric/name altogether) which is the magnitude of the problem. I suppose this would only be relevant when the proposed stalled metric is 1..if I am 100% stalled, how big is the backlog?
I am completely new to frankenphp so am a little foggy still about how the dispatching/queue works so may be off on my thinking so please correct my understanding as necessary. For example I don’t know if the requests are simply round robin’d to each worker as they come in OR if there’s more intelligence scheduling in front of the workers (ie: do I technically have 1 backlog or do I have N backlogs, 1 for each worker?) doing least connection style logic etc.

#269 Linking to related issue.

@travisghansen it looks like this is already doable due to a 'logic bug'.
frankenphp_[worker]_busy_workers will go above the number of workers if there are more requests than workers due to being incremented before we ever send the request to the worker. So, you can use frankenphp_[worker]_busy_workers - frankenphp_[worker]_total_workers to get the backlog, where <= 0 means no backlog.


Update RMRKRoyalties wtih RoyaltyDetails struct

I updated the RMRKRoyalties with the RoyaltyDetails struct which I use to avoid the limit of too many params in the factories.
Also, I added public getters for royalty recipient and royalty percentage.

This is only needed for the factory contracts so we'll keep it clean and not do it here.
We'll only add the RMRKRoyalties.getRoyaltyRecipient method.


Pkg: Fonts should be opt-in

Fonts are currently hard coded into all.scss. They should be explicitly set into a theme or fonts file so they can be replaced. Now they always get downloaded whether you want them or not.
Also, should set an explicit font stack for each font type.

This is good! We should include the fonts as part of the theme settings and not directly on the library. The library will work with the standard platform font.

@saadmhybris Since we are tying fonts to specific themes, I recommend creating a Techne theme file at scss/theme/techne.scss. In that file, only include the fonts and the tn-fonts variable declaration - remove the !default - from _settings.scss
In _settings.scss, change tn-fonts to the browser defaults only. Then in all.scss uncomment the theme line and set that as your theme file. This will essentially preserve the Techne look and feel which is intentionally buried in core/_settings.scss but separates the fonts.
I think this is better than hardcoding them in the starter HTML head.


New language avaible

Good night
I made a translate of Bludit to Portuguese. I'm new in gitHub and I don't know how I can send it to became part of the project.
pt_BR.zip
I hope that is useful.
Thanks,
ASB

Hi, thanks, here you have a guide to push content in the repository.
https://varlogdiego.com/post/how-to-contribute-to-a-github-project


updatehub: Require peer verification with DTLS

DTLS without peer verification offers no security whatsoever (and is
arguably worse than not using DTLS in the first place).
Change the verification option to require this peer verification.  To
use this, it may be necessary to install and use a root certificate.
Signed-off-by: David Brown<EMAIL_ADDRESS>

Hi David, thank you for this update.
CC @otavio


Add restart functionality and Game Over handling

Hi,
I’ve added the restart functionality and game over state handling to the game. Here’s a summary of the changes:
Implemented the ability to restart the game using the 'R' key.
Added game over state detection with a "Game Over!" message displayed in red.
Disabled all controls during game over except for the restart functionality.
Updated the main game logic to integrate new game over state management.
Created a new plus.md file to describe the added features.
Modified model.py to include a gameOver method for game over detection.
Files added:
plus.md (New file with feature descriptions)
Files modified:
tetris_game.py (Main game logic updates)
model.py (Added game over detection method)
Please review the changes and let me know your feedback. I would appreciate it if you could take a look and provide your comments.
Thank you!

@LoveDaisy I know you're busy, but if you could take a quick look at this PR, it would really help us keep things moving forward. Thanks so much for your time and effort!

sorry for my late reply. I was on a vacation and didn't check my email in the past several days. The PR is merged. Thanks for your enthusiastic works!

Thank you for taking the time despite your busy schedule, and have a great vacation!


Bug: showing preview laggs behind when searching through the result

RTFM Checklist

[X] I have searched exisiting issues / discussions
[X] I have read the Wiki including the Advanced section
[X] I have read man fzf / I am well versed in shell fzf

Operating system
mac 14.3
Shell
zsh
Neovim version (nvim --version)
0.9
Fzf version (fzf --version)
0.44.1
Output of :lua print(os.getenv('FZF_DEFAULT_OPTS'))
nil
Is the problem reproducible with mini.sh?

[ ] My issue is reproducible with mini.sh
[ ] My issue IS NOT reproducible with mini.sh
[ ] I have not tested with mini.sh (not relevant, requires LSP, Windows, etc)

Fzf-lua configuration
require('fzf-lua').setup({
})

Describe the bug / steps to reproduce
When searching files, live-grep, live-grep-native, live-grep-glob, and returning a list of 50+ files (the amount really doesn't matter tbh), and i go up/down through the list, the preview is not immediately updated.
An example would be holding down arrow down/up button going through the list, it only updates when i stop pressing the buttons.

An example would be holding down arrow down/up button going through the list, it only updates when i stop pressing the buttons.

There is a denounce mechanism in place by design, IMHO it doesn’t make any sense to load the previewer while scrolling.
If you want to disable the denounce, use winopts.preview.delay, you can test how that feels with:
:FzfLua files winopts.preview.delay=0

If you want that permanently use `setup’.