Bitmap too large to be uploaded into a texture (3031x4169, max=4096x4096)

Hi ,  First your library is so useful.
When I choose large crop size area then I got this error ,
Bitmap too large to be uploaded into a texture (3031x4169, max=4096x4096)
Some time also get  out of memory error.

This is getting after onGetCroppedImageComplete done, I tried to set bitmap to my ImageView here is error .
I picked image from gallery  .  Kindly advise how to solve this issue.

Android devices have a limit to the size of image they can render, see BitmapUtils.calculateInSampleSizeByMaxTextureSize to how I get that limit value, so you need to limit the size of the image you are getting.
It seems you are handling a very large images so I advice to use sampling, use .getCroppedImageAsync(int reqWidth, int reqHeight) overload to limit cropped image size.
See Loading Large Bitmaps Efficiently on what sampling is.
Note, by design getting cropped image has no built-in protection from out-of-memory error, you can do it using the requested width/height starting with large value and lowering it until success.

Thanks for your immediate response,
Now I pass with Width & Height  by using below , Now its working fine.
int xDim=iv_chooseImage.getWidth();
int yDim=iv_chooseImage.getHeight();


Update set-ansi-defaults-transact-sql.md

Add filtered indexes to the list of things with required ANSI SET options. Having any of these set will cause the following index create statement to fail.
--SET QUOTED_IDENTIFIER OFF;
--SET NUMERIC_ROUNDABORT ON;
--SET ANSI_WARNINGS OFF;
--SET ANSI_PADDING OFF;
--SET ANSI_NULLS OFF;
--SET CONCAT_NULL_YIELDS_NULL OFF;
DROP TABLE IF EXISTS #t;
CREATE TABLE #t(id bigint)
CREATE INDEX t ON #t(id) WHERE id = 0;
GO

Thanks, @erikdarlingdata!

FWIW this PR (like many others) is tagged with "do-not-merge" flag [by the prmerger-automator bot], but has been merged anyway, which smells like a contradiction. Maybe the actual approver of any PR should clear that flag (or a bot should automagically clear it as assumed the approver knows what s/he is doing? Hmm.

@DickBaker Thank you for your feedback.


Children method fails on receiving big list of children

In file github.com/samuel/go-zookeeper/zk/conn.go on line 545 we grow buffer size to blen if blen is bigger than current buffer. Then in several places we decode stuff like:
_, err := decodePacket(buf[16:16+blen], res)
which is obviously panics - because we get past slice boundary.
Fix is to change line 545 to look like:
buf = make([]byte, blen+16)
Can't create a pull request right now. But would appreciate if somebody could fix this in mainstream.

I created a pull request for this. https://github.com/samuel/go-zookeeper/pull/90


APEXMALHAR-2484 Support of PartFileWriter for writing the part files

@shubham-pathak22 @yogidevendra By mistakenly, I closed the PR https://github.com/apache/apex-malhar/pull/611. . Please review and merge.

Looks good to me. Will merge it in a day if there are no further comments from the community

@chaithu14 Please check why Jenkins check is failing.

@ilganeli Thanks for reviewing. Incorporated the review comments and updated the PR.

@ilganeli Please merge, if you are okay with the changes.

LGTM. Thanks.

For how to merge PRs see:  http://apex.apache.org/contributing.html

@ilganeli Please mark the JIRA https://issues.apache.org/jira/browse/APEXMALHAR-2484 resolved and version 3.8.0 (upcoming). @chaithu14 @ilganeli please verify and manually close this PR,  the commit id in the tree is different from the one in the PR.

Will do. Why would the commit Id be different and how would I avoid this in future? I followed the commit guidelines step by step.

From: Pramod Immaneni<EMAIL_ADDRESS>Sent: Friday, May 5, 2017 7:58:28 PM
To: apache/apex-malhar
Cc: Ganelin, Ilya; Mention
Subject: Re: [apache/apex-malhar] APEXMALHAR-2484 Support of PartFileWriter for writing the part files (#614)
@ilganelihttps://github.com/ilganeli Please mark the JIRA https://issues.apache.org/jira/browse/APEXMALHAR-2484 resolved and version 3.8.0 (upcoming). @chaithu14https://github.com/chaithu14 @ilganelihttps://github.com/ilganeli please verify and manually close this PR, the commit id in the tree is different from the one in the PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/apache/apex-malhar/pull/614#issuecomment-299600650, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABaiUXSY5xTDkOFbGW-EbspcxoDLQW20ks5r27ekgaJpZM4NKAFS.

The information contained in this e-mail is confidential and/or proprietary to Capital One and/or its affiliates and may only be used solely in performance of work or services for Capital One. The information transmitted herewith is intended only for use by the individual or entity to which it is addressed. If the reader of this message is not the intended recipient, you are hereby notified that any review, retransmission, dissemination, distribution, copying or other use of, or taking of any action in reliance upon this information is strictly prohibited. If you have received this communication in error, please contact the sender and delete the material from your computer.

Probably happened because you may have rebased it. Ask the contributor to rebase when the PR is ready to be merged. There may be still some lag before you merge it and additional commits may have gotten into the master in which case you can merge it with an automatic merge commit. This process is not perfect but at-least it reduces the occurrences of these merges. If you see conflicts have contributor rebase and fix again.


Format like P4-14 document

[DO NOT MERGE]
This experimental pull request uses the same look and feel as the P4_14 spec.
It requires:

Xelatex
Fonts mentioned in the README

Let's not merge in case it breaks the build for others. But if you are curious, feel free to try it out.

@cc10512 Would you do a dry run building the documents using this branch and let me know if the new build with fancy fonts works for you?

Can you please rebase this branch?
To make sure that the TOC shows correctly in PDF, you could remove h* from the tex style:
--- a/p4-16/spec/P4-16-draft-spec.mdk
+++ b/p4-16/spec/P4-16-draft-spec.mdk
@@ -31,7 +31,7 @@ pre, code {
 body.madoko {
   font-family: UtopiaStd-Regular;
 }
-title,titlenote,titlefooter,authors,h1,h2,h3,h4,h5 {
+title,titlenote,titlefooter,authors {
   font-family: Helvetica;
   font-weight: bold;
 }

Also, the Dockerfile will need to be updated with at least the following:
apt install -y texlive-math-extra
sudo apt install lmodern
mkdir -p ~/.fonts/truetype/UtopiaStd/
curl -o  ~/.fonts/truetype/UtopiaStd/UtopiaStd-Regular.otf <from somewhere that doesn't have captcha>
wget https://tug.org/fonts/getnonfreefonts/install-getnonfreefonts
texlua install-getnonfreefonts
getnonfreefonts-sys luximono
sudo fc-cache -f -v

@cc10512 I cherry-picked the commit (414510f) from your branch. Can this go now? If so, will you approve?


Add Answer for Freetext Question Types

Right now free-text question types (SA, Essay) don't use their answers field.
Instead, we can allow a single simple string that we can output in answer keys.
This can serve as a rubric/instructions for graders.

Complete in ff6a5ec0ab3c16a83e141d46cb8d33b7e7f92d40.


Fix for Patch & Wait

Fix to prevent all attributes being sent with request when using
model.save(attrs, { patch: true, wait: true });

Related issue here:
https://github.com/AmpersandJS/ampersand-model/issues/13

I say this is a bug fix rather than an enhancement.  The expected behaviour when sending PATCH requests is to only send the specified attributes.

Yeah this makes sense. Should it have a quick test?

@wraithgar Let me check if there is an existing test in the backbone tests that are pulled in. If not I'll get one written ASAP.

Added tested for with PATCH and save with PATCH and wait

Hey @nickryall I want to review this PR (finally) - can you rebase all commits into 1?

@pgilad  Apologies, I just saw your comment. Sure I'll get this done today.

Rebased and re added new PR #58. Thanks!


fix: update Code of Conduct governance document link in CODE_OF_CONDUCT.md

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions.

Description of the Change
"Code of conduct governance" text and link seem to be connected incorrectly. It has been updated.
Before: https://github.com/openjs-foundation/bootstrap/blob/master/proposals/stage-1/CODE_OF_CONDUCT/FOUNDATION_CODE_OF_CONDUCT_REQUIREMENTS.md
After: https://github.com/openjs-foundation/cross-project-council/blob/master/FOUNDATION_CODE_OF_CONDUCT_REQUIREMENTS.md
Alternate Designs
N/A
Why should this be in core?
N/A
Benefits
People will be able to click on the links, and get to the content they expected.
Possible Drawbacks
N/A
Applicable issues
N/A

Coverage increased (+0.06%) to 93.849% when pulling ccd2244f6e94b9d989c6d4a9f573e9e9c3d3ac37 on sujin-park:fix-coc-document-link into 78d979db7c41b68941518f026b5cad7d59376ada on mochajs:master.


sign-conversion warnings

would be nice to eliminate warning like this:
rapidcsv.h:1062:32: warning: conversion to ‘std::__cxx1998::vector<std::__debug::vector<std::__cxx11::basic_string<char> >, std::allocator<std::__debug::vector<std::__cxx11::basic_string<char> > > >::size_type’ {aka ‘long unsigned int’} from ‘ssize_t’ {aka ‘long int’} may change the sign of the result [-Wsign-conversion]

there are a lot of them.

Thanks for pointing this out, it makes sense. Will prepare a fix.

A fix for this is under review in pull request #112 - will update this issue once it's integrated to master branch.

The sign conversion warnings should be fixed in master branch now, with above commit. Please let me know if you still encounter any warnings.


Make sure question mark is included in query string

Fixes #1672. This was edited in the GitHub GUI so bear with me :wink:

This pull request does not fix #1672. The question mark is still missing in 1.0 stable.


Guzzle Truck Factor

Dear Guzzle contributors,
I am Master's student investigating the importance of the idea of truck factors in Software Engineering.
As you know, truck factor (aka bus factor) is defined as “the number of people on your team who have to be hit by a truck before the project is in serious trouble”.  By "serious trouble", we mean that it will very difficult to maintain the project (e.g., bugs will take more time to be fixed, new features will be delayed etc).
I would appreciate your contribution by answering two questions:

According to your experience, what number best represents the truck factor of Guzzle?

Who are the developers responsible for this truck factor (i.e., the name of the developers who are responsible for the TF result answered in the previous question)?

Mívian Ferreira
Master's Student
Applied Software Engineering Group (ASERG)
UFMG, Brazil
http://aserg.labsoft.dcc.ufmg.br/

@2mferreira I am not completely sure if it's the perfect timing to ask these questions :smile:
The Guzzle maintainer team is changing at the moment. @mtdowling decided to move to the background and the PHP HTTP group is stepping in for maintenance. There are four active maintainers of the group, but it's still unclear who exactly will contribute to Guzzle in the near future.
I would say the bus factor is pretty good in our case, because everyone knows about every project, yet we have separate responsibilities in the group, so we can easily replace each other in case of absence (be it a holiday, or God save us from that, but an actual bus accident).
All in all, I am not sure if we can answer your questions at the moment.

This type of question isn't really an issue. Closing.


Add multi api client to remove dependency on azure-common

Duplicate of https://github.com/Azure/azure-sdk-for-python/issues/10179


Add test script for bypassing backend authentication issue

Description

Refs #4115.
Readiness checklist

[ ] I added/updated unit tests (and they pass).
[ ] I added/updated integration/compatibility tests (and they pass).
[ ] I added/updated comments and checked rendering.
[ ] I made spot refactorings.
[ ] I updated user documentation.
[ ] I ran task all, and it passed.
[ ] I ensured that PR title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

#4313


Install ruby in container the way other repos do

Summary
Ruby installation in tests/Dockerfile uses a ppa repository added in the Dockerfile. We install ruby differently in other repos. We should standardize our installations or add a comment explaining why we need to do it this way here.
Steps to Reproduce

Go to tests/Dockerfile
read the line that starts apt-add-repository -y ppa:/busybox/ruby-ng (currently line 19 as I write this)

Expected Results
A clear explanation of exactly where ruby is being installed from and why it is done this way here, or a replacement with the normal method of installing ruby in a container.

See https://conjurhq.slack.com/archives/C0A2NF134/p1614031664013600 for @izgeri 's thoughts on this.


Union/Struct/Enum Definitions Inside a Function Scope

Well, that one is a bit complicated in the context of the Norm.
enum e_endian
{
	LITTLE,
	BIG
};

enum e_endian	which_endian(void)
{
	union
	{
		unsigned char	var2[2];
		unsigned short	var1;
	}	u_endian;// This should be the correct indentation
//	}					u_endian;
	u_endian.var1 = 1;
	if (u_endian.var1 == u_endian.var2[0])
		return (LITTLE);
	else
		return (BIG);
}

I assume that the indentation and the alignment of the union should be as it is. Not sure of it's name's though.
It returns many errors, and a variable scope inside a function breaks norminette.

Hello, union is not allowed inside the scope of the fonction, but it needs to return a clear error.

I forgot about that, thanks.


switch morphs when clicked, and zooms when closed

Describe the bug
switch morphs when clicked, and zooms when closed
Steps to reproduce
https://github.com/tusen-ai/naive-ui/assets/20943608/fe955ca9-8f54-451a-aab2-742da62908b3
Link to minimal reproduction

System Info
windows

Used Package Manager
npm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

Provided reproduction is a minimal reproducible example of the bug please.

@Volankey
https://codesandbox.io/p/sandbox/awesome-paper-vf8lsf?file=%2Fsrc%2FDemo.vue%3A49%2C31
When ellipsis + switch is used in a table, clicking switch will deform


Rename OnAppStart to OnAppReady

More clear on the purpose of the call.
init() functions run when the app starts, but we want to run certain code AFTER the app is ready, not after it starts.

@notzippy prefers the current naming because it's before the server starts. We need to clarify further.

With Before / After instead of interceptors would it make sense to replace OnAppStart mechanism by a special first class function that would be invoked automagically? Like:
type Controller struct {
	...
}

func (c *Controller) Before() revel.Result {
	...
}

func (c *Controller) SomeAction() revel.Result {
	...
}

func (c *Controller) After() revel.Result {
	...
}

func Ready() {
	...
}

Sometimes your init code is not related to a particular controller @alxkchr

That's why it's a function, not a method. Or is you concern that you can have only 1 Ready function per package?

What purpose would the Ready function perform ?

That couldn't be done by
func init() {
  revel.OnAppStart(Ready)
}

What purpose would the Before / After actions perform that couldn't be done by interceptors? That's consistency.

revel.OnRevelReady() or revel.OnRevelStart() or revel.OnInit() or revel.OnStart()?

I like revel.OnReady(func(){}) or revel.OnStart(func(){})
That said, not sure if this is a direly needed feature


I2C module crash due to writing to NULL pointer (IDFGH-7003)

Environment

Development Kit:     Custom
Kit version: Custom
Module or chip used:  ESP32-WROOM-32
IDF version (run git describe --tags to find it):
// v4.4-348-gc45dee0a7a
Build System:        CMake|idf.py
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
// (crosstool-NG esp-2021r2-patch3) 8.4.0
Operating System:     Linux
Using an IDE?: Yes, eclipse with IDF plugin
Power Supply:         Regulated 3.3V

Problem Description
Tracked down the issue to the i2c.c file, where the code is trying to write a structure using a pointer without verifying if it's NULL.
Found a fix that was done for the same isse in the same function at https://github.com/espressif/esp-idf/commit/0cfa7ca20f40ac481495d4312953329ec5b00960
static void IRAM_ATTR i2c_master_cmd_begin_static(i2c_port_t i2c_num)
{
    i2c_obj_t *p_i2c = p_i2c_obj[i2c_num];
    portBASE_TYPE HPTaskAwoken = pdFALSE;
    i2c_cmd_evt_t evt = { 0 };
    if (p_i2c->cmd_link.head != NULL && p_i2c->status == I2C_STATUS_READ) {
        i2c_cmd_t *cmd = &p_i2c->cmd_link.head->cmd;
        i2c_hal_read_rxfifo(&(i2c_context[i2c_num].hal), cmd->data + cmd->bytes_used, p_i2c->rx_cnt);
        /* rx_cnt bytes have just been read, increment the number of bytes used from the buffer */
        cmd->bytes_used += p_i2c->rx_cnt;

        /* Test if there are still some remaining bytes to send. */
        if (cmd->bytes_used != cmd->total_bytes) {
            p_i2c->cmd_idx = 0;
        } else {
            p_i2c->cmd_link.head = p_i2c->cmd_link.head->next;
            p_i2c->cmd_link.head->cmd.bytes_used = 0;
        }

The line where is crashing is the next one, that is located in the last else (line 1289 in my I2C.c file)
p_i2c->cmd_link.head->cmd.bytes_used = 0;
Expected Behavior
Execute the code only when the pointer is not NULL
Actual Behavior
Trying the write the bytes_used field when head is NULL
Steps to reproduce
Not pretty sure, the issue only appeared when an I2C slave was connected to the I2C bus. And the code being executed would be this one
I fixed it by copying the fix in the mentioned commit in that line
p_i2c->cmd_link.head = p_i2c->cmd_link.head->next;
if(p_i2c->cmd_link.head) {
    p_i2c->cmd_link.head->cmd.bytes_used = 0;
}

Code to reproduce this issue
Part of the code, can't copy the whole code due to company's NDA

    /* Create cmd link to store the streaming bits */
    i2c_cmd_handle_t cmd = i2c_cmd_link_create();

    /* Set the start bit */
    i2c_master_start(cmd);

    /* Send the slave address with the read bit set and ack verification enabled */
    i2c_master_write_byte(cmd, (address << 1) | READ_BIT, ACK_CHECK_EN);

    /* Read the first byte containing the packet size */
    i2c_master_read_byte(cmd, data_rd, ACK_VAL);

    /* Execute the I2C communication */
    ret = i2c_master_cmd_begin(I2C_MASTER_NUM, cmd, I2C_TIMEOUT_MS/portTICK_PERIOD_MS);

Debug Logs
Guru Meditation Error: Core  0 panic'ed (StoreProhibited). Exception was unhandled.

https://github.com/espressif/esp-idf/issues/8548

Hi @luis0visa ,
As @AxelLin stated, this issue is known and currently being fixed.
In your example though, you forgot the STOP signal at the end (was it intended?), this is why is error is happening. Moreover, if you are only willing to perform simple I2C transactions, you can use the following functions, which are tested and working:
esp_err_t i2c_master_write_to_device(i2c_port_t i2c_num, uint8_t device_address,
                                     const uint8_t* write_buffer, size_t write_size,
                                     TickType_t ticks_to_wait);
esp_err_t i2c_master_read_from_device(i2c_port_t i2c_num, uint8_t device_address,
                                      uint8_t* read_buffer, size_t read_size,
                                      TickType_t ticks_to_wait);
esp_err_t i2c_master_write_read_device(i2c_port_t i2c_num, uint8_t device_address,
                                       const uint8_t* write_buffer, size_t write_size,
                                       uint8_t* read_buffer, size_t read_size,
                                       TickType_t ticks_to_wait);

More info in the documentation

Hi there,
You guys are right, my issue is the same that @AxelLin pointed out, I didn't find it when looking for it.
And @o-marshmallow, yes it was intentional at that point, I updated the code to reproduce the issue to include the rest of the code where the stop is appended, what is being done is to read the slave using a custom "protocol", where the first step is to read one byte where the slave will indicate the number of bytes that it will reply, then the master compares it to the size indicated by the user (to verify that the provided buffer is big enough to get the whole transmission), and then read the slave finishing the communication.
This would be the read function, the equivalent custom write function is kind of the same, but the ESP32 (master) starts sending one byte with the size of the data, finishing the communication, and initiating a new one after waiting some little time (to allow the slave to set up and be ready for the reception. This is done like this due to some issues, even though it's supposed to be like the read (one stream).
This code was working in previous IDF versions though, thank you guys, this issue could be marked as duplicaded or closed as you consider it better.

@luis0visa I understand, thanks for the details :+1:
Closing this issue then


Game crash

Hi,
I did a git clone https://github.com/e8johan/supermemory
an add this to godot 3.2.3
If I run the game got this error:

How can I fix this ?

Hi @ds2k5 !
It seems like a new warning/error is Godot. I believe I was on 3.1.x when developing this. Just do what it says, and change shape_id to _shape_id, and I think it should be fine.

Thanks
I did but did not work for me


can we add gRPC over TCP not HTTP2, TCP is has less protocal overhead. windows WCF support both TCP and HTTP2

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context about the feature request here.

NO. The grpc protocol requires multiplexing and per stream flow control and reinventing both of these over TCP will get us back to something like HTTP/2.


update asterisk-stable and asterisk-lts

Motivation for this change

include debian patch to remove warnings that clutter journald logs
bump to latest upstream versions

Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

At first look, seems fine. Out of curiosity, why disable the md5 verification in the makefile?

Thanks for reviewing @yorickvP and yeah i forgot to mention about the MD5 verification.
The upstream makefile now downloads an MD5SUM.TXT  next to the pjsip package url.
I propose removing this step instead of maintaining another hash each time we bump versions.

Thank you


feat(pytorch): Add init container config to avoid DNS lookup failure

What this PR does / why we need it:
This PR is to add the init container, which checks if the master is already running.
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #1482

/hold

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

65 of 83 (78.31%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 27.639%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controller.v1/pytorch/envvar.go
13
15
86.67%

pkg/controller.v1/pytorch/pytorchjob_controller.go
3
7
42.86%

pkg/controller.v1/pytorch/initcontainer.go
48
60
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
1689

Relevant Lines:
6111

💛  - Coveralls

/hold cancel

It looks good to me. Feel free to unhold it  when it's ready to merge.
/lgtm
/hold

/unhold


Fix README

[x] Create a README

This is complete. I have also merged all of the routing related repos together.


Insecure resource loading over http in example

The example for the theme (https://jamigibbs.github.io/phantom/) does not load any of required resources. This is due to these being requested over http and not https, and the browser refusing to load these resources due to that. For my browser, I used Chrome 74.0.3729.131.

Thanks @matthiasludwig 👍


NPE in textfilter

Appears to be caused by AbstractRollingAppender.getFrequency() downcasting a null Long to a long.
Setting the frequency to big value, e.g. 9999d is a work around.


Add storage hosts biham and nihal

S'ad al-Biham -- Lucky Star of the Young Beasts (https://en.wikipedia.org/wiki/Theta_Pegasi)
An-Nihal -- The camels quenching their thirst (https://en.wikipedia.org/wiki/Beta_Leporis)

nihal will be used for cube, biham will be mainly archives and npm.
castor and sirius are now deprecated, their homes and repos will be moved to biham.

:+1: thanks!


" in env vars makes custom api to fail to deploy

This is fixed. We now have a regex hat we check all env vars against.


File Names are wrong

Whenever i export scene as GLTF, it renames the textures into the wrong file extensions

Hey, please post

Unity version
SRP version
UnityGLTF version
reproduction file (e.g. .unitypackage)

Thank you!

is fixed with: d1ffc441def00d73c9f9dc677d1eefd0e38e50f1


Legacy JS support removed - intentional?

klock 2.1.1 appears to no longer include support for Kotlin/JS's legacy compiler. I'm guessing this is due to this commit:
https://github.com/korlibs/easy-kotlin-mpp-gradle-plugin/commit/f84b257fe8582e73783868d3cd88ce2e168e889e
I wanted to ask what the reasoning was behind this change. My project (https://github.com/jpd236/kotwords) still uses the legacy compiler; the IR compiler is still pretty rough around the edges from my testing - it's explicitly marked as an alpha in the official docs - and doesn't seem to be a trivial change. The docs here recommend that libraries compile to both. Wouldn't it make sense here to release both versions until IR becomes more stable and prevalent?

I removed it intentionally. But I can enable it again

In theory if everything goes well 0.15.3 should include BOTH artifacts: https://github.com/korlibs/klock/actions/runs/874689849
I'm closing the issue. Feel free to reopen if you find some issues with this

It seems that at least 2.3.1 does not contain the legacy version. At least Gradle is complaining in my project.

It seems that 2.2.2 contains the version for legacy backend, 2.3.0 is the fist version without

@jschneider no new additions on 2.3.0. The release is there because was triggered in korge-next for a consistent release to save me time on releases:
https://github.com/korlibs/korge-next
You can safely use 2.2.2 that should be binary compatible with all Kotlin 1.5.x versions.


Fix cross platform RPM builds

Explain the changes

This PR restructures how we do the RPM builds today.

What changed?

Instead of fetching NooBaa runtime dependencies in the Dockerfile, we fetch them in the RPM build process.
We make sure the environment has all the dependencies that NooBaa requires for the build process

Why changed?

This approach allows generating a true architecture independent SRPM, hence a SRPM can be generated once (per RHEL version) and then rpmbuild --rebuild  can be used to generate RPMS for different architectures.
This approach allows us to use mock to build the RPMs

Note: Usage remains same, that is use, make rpm. This will generate a SRPM and a RPM. Use the SRPM to generate the RPMs for other architectures. The command also supports make rpm SRPM_ONLY=true which is significantly faster and jsut generates SRPMs.

[ ] Doc added/updated
[ ] Tests added

FYI: I had to disable BUILD_S3SELECT to get this to pass when testing locally.

FYI: I had to disable BUILD_S3SELECT to get this to pass when testing locally.
@tangledbytes I have invited you to my fork of the repo, you should be able to update this PR if necessary.


tmux-fingers window does not show matches in tmux 3.1c

I'm using tmux-fingers with a relatively new version of tmux (3.1c). When I invoke prefix+F, I get a [fingers] window, but all of the stuff that should be copyable is blanked out.
Example, before invoking tmux-fingers:

After invoking tmux-fingers:

The health-check and hints scripts don't return anything.

Is tmux-fingers compatible with tmux 3?

Seems to be fixed by upgrading from 3.1c -> 3.2.

usually this is fixed by resourcing your tmux.conf


Dont await TenantId task for detectors that don't need tenantid

ASE Lens fails to work on ASE that has failed the creation step. This is due to the fact that the environment doesn't have stabilized roles so tenant ids will return empty which cause a 500 on the API.
There are some detectors such as DeploymentDetector, ManagamentHealthDetector that can function without tenant ids.
Find a way to not break the API for detectors that don't need tenant id, rather just return partial data.

example:
https://applensv2.azurewebsites.net/hostingEnvironments/drj-test-ase2-ilb/aseAvailabilityAnalysis
https://applensv2.azurewebsites.net/hostingEnvironments/drj-test-ase1-ilb/aseAvailabilityAnalysis


Snapshot between brs-023 and brs-027 must be coordinated

We need to define where to implement the logic to persist basis data for a process run.
When a D03 or D04 is run we should not store charges related data (aggregation)
When we run a D05 or D32 we should store a complete basis data set (wholesale)
Proposed solutions:

Store data based on process type in the aggregation process
Create a separate job with the responsibility of storing basis data based on the process type

Share data between multiple databricks jobs: https://medium.com/@kar9475/data-sharing-between-multiple-spark-jobs-in-databricks-308687c99897

Database must be changed to correspond to image below:

When data preparation job is created, the snapshot must be inserted into the database.

When other jobs are created they need a reference to a snapshot.


Questions on VAD (sample_rate, length of channels, device)

I have questions about VAD pre-trained model.

Sample rate
Assume that given the sample rate of random wav file as not 16kHz, how can I change to use for VAD pre-trained model (get_speech_prob_file function)? It seems like get_speech_prob_file function only allows to get the file path of the wav file.

Because the get_speech_prob_file function allows only the file path.
Do I have to load first and resample that wav file, and save, and load it again?

Length of channels
If the wav data has [2, T], how could I use the get_speech_prob_file function? It seems like only allowing the [1, T] of the wav file. Do I have to load first and cut as [1, T] and save, ... and load it again?

Assume that I can't use torchaudio's save function by changing the 2 channel wav to 1 channel, how can I solve it?

Device
I wanna use the VAD pre-trained model as Cuda device, but I got errors when I load the wav file as below:

Because the input format of get_speech_prob_file is only available at filepath (e.g., /home/data.wav). So I don't know how to make this file on Cuda available.
How can use this as cuda? Only can perform on CPU?

The most hurry things that I want to handle are the number 2 that is the length of channels. If I cannot use the torchaudio.save in a limited environment, how can I use the 2 channels of wav file (stereo)? Could you help me? @mravanelli

Hi @kaen2891,
thank you for the interest in SpeechBrain.
Let me reply to your questions:

Sample rate
The supported sample rate depends on the one used for training the model. In the vad-crdnn-libriparty case, the sample rate is 16 kHz.  If your signal is at a higher sampling rate (e.g., 44.1 kHz or 48 kHz), you can downsample it to 16 kHz before feeding it into the VAD. If your sampling rate is smaller (e.g., 8 kHz) you can upsample at 16 kHz.
The upsampling is a bit more critical because you lose some information but normally works well anyway. To resample a signal,  you can use torchaudio or sox.  So yes, you have to load, resample, and save the resampled version on disk before using the VAD. As an alternative, you can do the resampling on the fly right after reading each chunk of signal, after this line.

Length of channels
The system expects single-channel audio. You can save the signal as single-channel audio or you can select the channel right after reading the signal here.

Device
The device mismatch issue is already solved in the latest dev branch. Make sure you have it installed:

git clone https://github.com/speechbrain/speechbrain/
cd speechbrain
pip install -e .

The fix will appear in the main branch (i.e. the one that you use with pip install) when we will release the next version of speechbrain (which will be issued soon).

Thanks for replying, @mravanelli
But I also got an error with VAD that is installed with the git clone version.

I don't know how to solve this.
The input wav file is 16kHz, mono channel.

Hi,
my feeling is that your input signal is not in the expected format (you might need to transpose the input signal, unsqueeze it,or, something like that).
You can try to double check it in this way:

Try to run the example provided in the HF repository with the provided signal example (see code at the end of this message). Do it work?

If it works, I suggest to make sure the dimensionality of your signal is compatible with the format used in the provided example (see pretrained_model_checkpoints/example_vad.wav). You can check the dimensionality of the provided example with torchaudio in this way:

import torchaudio
sig, fs = torchaudio.load('example_vad.wav')
sig.shape
torch.Size([1, 4794408])

You can run the code above with your signal to make sure the format is [1, time_steps]. If not, change the format of your signal ti match the expected one (using operations like transpose, unsqueeze, etc)

Code Snippet with the provided example:
from speechbrain.pretrained import VAD
VAD = VAD.from_hparams(source="speechbrain/vad-crdnn-libriparty", savedir="pretrained_models/vad-crdnn-libriparty")

# 1- Let's compute frame-level posteriors first
audio_file = 'pretrained_model_checkpoints/example_vad.wav'
prob_chunks = VAD.get_speech_prob_file(audio_file)

# 2- Let's apply a threshold on top of the posteriors
prob_th = VAD.apply_threshold(prob_chunks).float()

# 3- Let's now derive the candidate speech segments
boundaries = VAD.get_boundaries(prob_th)

# 4- Apply energy VAD within each candidate speech segment (optional)

boundaries = VAD.energy_VAD(audio_file,boundaries)

# 5- Merge segments that are too close
boundaries = VAD.merge_close_segments(boundaries, close_th=0.250)

# 6- Remove segments that are too short
boundaries = VAD.remove_short_segments(boundaries, len_th=0.250)

# 7- Double-check speech segments (optional).
boundaries = VAD.double_check_speech_segments(boundaries, audio_file,  speech_th=0.5)

@kaen2891, is everything fine now?


Implement OTP Input

Describe the feature
Port input-otp to shadcn-svelte.
This component has been out for quite some time, but I've noticed, that the svelte port of shadcn does not have it currently.

Hello;
Related/duplicate of: https://github.com/huntabyte/shadcn-svelte/issues/1107


Drop pylint-gui

Dropping pylint-gui as it was dropped upstream in 1.7.0.
ref: https://github.com/PyCQA/pylint/blob/pylint-1.7.0/doc/whatsnew/1.7.rst

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy
[ ] Ensured the license file is being packaged.

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

@conda-forge-admin, please re-render.


stdin: Make sure that the buffer read from stdin is NULL terminated

Read one less byte from stdin to make sure the buffer is NULL
terminated.
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

LGTM

Actually, we can do much better than this I believe.
I'm guessing the red flag is because send_proxy_message() ends up doing a strlen on the stdin data. That looks like a bug, stdin data may well be binary data (say a targz'ed data stream to move files to the container). We also have to go through the data to figure out its length.
Instead of doing the above, we should reuse the length from read() as the payload length, not measure the data as if it were a string.

@dlespiau You are right, we are handling binary data from stdin. I'll rework this PR.

Opened #22 for this. Closing this one.


Display log location in the VM Migration Task Popover

While the code existed all along to display the location of the log in the Migration task popover, it was never displaying (because of using the wrong flag taskCompleted)
Before -

After -

ahh yes agreed... did not sample any of those here. thanks @AllenBW !

Yes, I missed that case. Will address it now.
Thanks @AllenBW

@AllenBW Can you test again when you get a chance?
Log: should not be displayed for your above use case.

Gaprindashvili backport details:
$ git log -1
commit 30187576eae4f2dcb750e7f12481dd497a80efc4
Author: Allen Wight<EMAIL_ADDRESS>Date:   Tue Jul 10 16:03:11 2018 -0400

    Merge pull request #480 from AparnaKarve/display_log_location

    Display log location in the VM Migration Task Popover
    (cherry picked from commit 558b2266d1b0750670552600047fe61a0f8e6c44)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1608750


Increase readability for curry function solution

Hi @mimol91, Thank you for taking time and contributing to Currying in JavaScript. However, few points to note:

Variable arity is intentionally been used to show new developers what arity is and where we are using it. It is verbose information for sake understandability.
if more than expected parameter are passed we should expect result based on actual arity of the function.
One more important point, once you get a partial function and is executed with all parameter, its subsequent calls with remaining parameter should provide the result.

I have created a JSbin with original and you curry function for your more clarity: https://jsbin.com/vajuhux/edit?js,console
Do let me know if you have any suggestion or want to discuss any point.
Cheers..


Slides 21

Hey Andy,
the latex is not rendering properly, i have added all the equations, the curly braces is not going to the second line. That's the only issue. Everything else seems fine.

thanks

can you point out the latex issue? I don't see where it is.

oh on the last slide? you needed single backticks around the $$.


[Bug]: Home Button is not working

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When you click on Home button on the main page It will give 404 Error.
Add Screenshots

Page is routing to index.html instead of Bobble-AI/index.html .
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

I can solve this issue assign me

/assign this task to me @mansiruhil13

/assign

@mansiruhil13 pls assign this task..


Fix bug with provider not set on GCP

If you set variables using terraform.tfvars file and not environment variables - you will get an provider not set error.

Will be fixed in 1.0-RC2


cannot send stream over the buffered channel [programming error]

Description
"jsonPayload": {
    "target": "datasources::bigquery",
    "threadId": "ThreadId(2)",
    "filename": "crates/datasources/src/bigquery/mod.rs",
    "timestamp": "2023-08-04T20:43:45.916355Z",
    "line_number": 316,
    "level": "ERROR",
    "threadName": "server-thread-0",
    "message": "cannot send stream over the buffered channel [programming error]"
}

Was this the result of a query that was running?

I don't know, saw the qa alert and created the ticket. Not able to check logs atm. Wasn't running any queries or even using qa at the time it occurred.

Is this still an issue?

Is this still an issue?

I have not seen it since, will wait for @vrongmeal's input

We can re-open this if required. I think I got a little far with this but still couldn't figure it out. Since we've not seen this for a long time, we can close it for now.


Create Boards doc set

Create Boards doc set within the product docs.
@chenilim: This is the migrated content from the Focalboard site to the docs site. Once this is merged, I'll review and update the content. If you could take a close look at the overview section and possibly add more content that'd be great. :)

@chenilim, gentle reminder on this to review. :)

@chenilim, I've split out the Focalboard content but not sure where you envisage it to live. You can view a preview here: http://mattermost-docs-preview-pulls.s3-website-us-east-1.amazonaws.com/4979/guides/boards.html. It could work in its own section called "Working with the Focalboard App". LMK what you think.


update sparseGPT

Type of Change
feature, API not changed
Description
Fine-tunable sparseGPT pruning, a branch of experimentation.

Type of Change
feature or bug fix or documentation or validation or others API changed or not
Description
detail description
Expected Behavior & Potential Risk
the expected behavior that triggered by this PR
How has this PR been tested?
how to reproduce the test (including hardware information)
Dependency Change?
any library dependency introduced or removed

Please update description and fix CI.

@WeiweiZhang1  do we still need this PR?


Auto docs with Sphinx

Set documentation with a sphinx

[ ] Automate docs update
[ ] Automate version read from the init.py
[ ] Correct modules in the index page


Using the N1MM ID field for the duplicate QSO detection

N1MM now sends an ID in the contacting record. This appears to be a 32 byte GUID. I suggest that the duplicate detection can be simplified to use the ID to lookup the contact. This is important because the duplicate table is made up of the following fields:
def checksum(data):
    """
    generate a unique ID for each QSO.
    this is using md5 rather than crc32 because it is hoped that md5 will have less collisions.
    """
    hval = data['timestamp'] + data['rxfreq'] + data['txfreq'] + data['operator'] + data['mode'] + data['call']
    return int(hashlib.md5(hval.encode()).hexdigest(), 16)

But the deletes record from N1MM does not send everything in the hash.
From the N1MM UDP Documentation:
<?xml version="1.0" encoding="utf-8"?>
<contactdelete>
    <app>N1MM</app>
    <timestamp>2020-01-17 16 :43:38</timestamp>
    <call>WlAW</call>
    <contestnr>73</contestnr>
    <StationName>CONTEST-PC</StationName>
    <ID>a1b2c3d4e5f6g7h</ID>
</contactdelete>

So when a contact is corrected and N1MM sends the contact delete and then contact replace, the collector cannot delete the entry from the duplicates hash table.
If it uses the ID field, then it can be deleted.
If it is not deleted, when N1MM sends the contact replace packet (which has the same fields as contactinfo, it would flag as a duplicate unless the timestamp of the contact or callsign were changed.
So by just using ID for duplicates detection, it would allow the deletion/replace change process to work.
Where this is really important is that if N1MM resends all the contacts to UDP, it sends out contact delete and contact replace packets.
This is the sequence sent by N1MM on a rebroadcast:
'<?xml version="1.0" encoding="utf-8"?>\r<contactdelete>\r\t<app>N1MM</app>\r\t<timestamp>2022-06-27 19:45:53</timestamp>\r\t<call>W2IU</call>\r\t<contestnr>22</contestnr>\r\t<StationName>DESKTOP-4K20B9M</StationName>\r\t<ID>fcbdeb9c-c69d-4479-90b1-2944b6cd76be</ID>\r</contactdelete>

2022-06-27 19:59:36.086 INFO     DELETEQSO: W2IU, timestamp =<PHONE_NUMBER>
Then it sends the contact replace
<?xml version="1.0" encoding="utf-8"?>\r<contactreplace>\r\t<app>N1MM</app>\r\t<contestname>FD</contestname>\r\t<contestnr>22</contestnr>\r\t<timestamp>2022-06-27 19:45:53</timestamp>\r\t<mycall>W4TA</mycall>\r\t<band>14</band>\r\t<rxfreq>1420000</rxfreq>\r\t<txfreq>1420000</txfreq>\r\t<operator>NY4I</operator>\r\t<mode>USB</mode>\r\t<call>W2IU</call>\r\t<countryprefix>K</countryprefix>\r\t<wpxprefix>W2</wpxprefix>\r\t<stationprefix>W4TA</stationprefix>\r\t<continent>NA</continent>\r\t<snt>59</snt>\r\t<sntnr>2</sntnr>\r\t<rcv>59</rcv>\r\t<rcvnr>0</rcvnr>\r\t<gridsquare> </gridsquare>\r\t<exchange1>1E</exchange1>\r\t
NLI
\r\t<comment></comment>\r\t<qth></qth>\r\t<name></name>\r\t<power></power>\r\t<misctext> </misctext>\r\t<zone>5</zone>\r\t<prec></prec>\r\t<ck>0</ck>\r\t<ismultiplier1>0</ismultiplier1>\r\t<ismultiplier2>0</ismultiplier2>\r\t<ismultiplier3>0</ismultiplier3>\r\t<points>1</points>\r\t<radionr>1</radionr>\r\t<run1run2>1</run1run2>\r\t<RoverLocation> </RoverLocation>\r\t<RadioInterfaced>0</RadioInterfaced>\r\t<NetworkedCompNr>0</NetworkedCompNr>\r\t<IsOriginal>True</IsOriginal>\r\t<NetBiosName>DESKTOP-4K20B9M</NetBiosName>\r\t<IsRunQSO>0</IsRunQSO>\r\t<StationName>DESKTOP-4K20B9M</StationName>\r\t<ID>fcbdeb9cc69d447990b12944b6cd76be</ID>\r\t<IsClaimedQso>1</IsClaimedQso>\r</contactreplace>

And then in the collector log, we get a duplicate message because the item was not deleted upon the contact delete.
2022-06-27 19:59:36.118 DEBUG    duplicate message

I'm glad to code this up but wanted to see if you agree Jeff before I do.

This sounds great to me.  Calculating the hash is way more expensive than simple string comparison, so this sounds like a win.  Please proceed.

This seems timely as Tom sent out a notification this morning stating that as of July 5 build, the  message will no longer be sent and it is now advisable to use GUID to identify rows.

Yes that was my doing. I noticed the packets came out of order so the combination of contact delete/replace and the seen set were not very useful. The ID GUID allows it to work better.  After discussing it with the team, they opted to skip the delete/replace sequence since that predated the ID GUID.
My thought was that double packets are rare so I just change the database code to do an INSERT or REPLACE and made ID NOT NULL and UNIQUE.
I am just using the database as the duplicate detection mechanism as seen did not seem to be adding anything. Again, simply because in my experience, double messages are rare (unless one does a rebroadcast or has one station set to Broadcast All). The overhead is not too great in that case.
cursor.execute(
'insert or replace into qso_log \n'
'    (timestamp, mycall, band_id, mode_id, operator_id, station_id , rx_freq, tx_freq, \n'
'     callsign, rst_sent, rst_recv, exchange, section, comment, msgID)\n'
'    values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?);',
(calendar.timegm(timestamp), mycall, band_id, mode_id, operator_id, station_id, rx_freq, tx_freq,
callsign, rst_sent, rst_recv, exchange, section, comment, msgID))
I have some significant changed in thew works to support all of this. I am finalizing my testing and will have a pull request by the weekend.
Tom NY4I

On Jun 29, 2022, at 8:50 AM, KD4Z @.***> wrote:
This seems timely as Tom sent out a notification this morning stating that as of July 5 build, the message will no longer be sent and it is now advisable to use GUID to identify rows.
—
Reply to this email directly, view it on GitHub https://github.com/n1kdo/n1mm_view/issues/39#issuecomment-1169940359, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6TWSW2RODO7S5CULXP3GDVRRBBDANCNFSM5Z7UWHRQ.
You are receiving this because you authored the thread.

I looked at your branch.  Can you name the new column 'n1mm_id' or 'qso_id' because I don't think it is really the msgID.
Otherwise I like it.  Thank you!  Jeff

Sure
Principal Solutions Architect
Better Software Solutions, Inc.
727-437-2771

On Jul 1, 2022, at 1:19 PM, Jeff Otterson @.***> wrote:

I looked at your branch. Can you name the new column 'n1mm_id' or 'qso_id' because I don't think it is really the msgID.
Otherwise I like it. Thank you! Jeff
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Just an update. We are moving so I have this code but have to complete its testing. I will get to that after we move in September or October. It will be done well in time for Winter Field Day.

Winter Field Day is coming!  I hope your move went well, Tom.
One of the changes I am going to add at some time is the ability to control which displays are shown.  Feedback from one local club is that they were interested in the raw scores, but not so much the per-operator scores.  Personally, I like the per-operator scores, but the plan is to make which panels are shown (and perhaps for how long) configurable.
73, Happy New Year.

Hi. I was just thinking about this project as I found my PC after misplacing it.
That sounds like a great change. I will take a look this weekend to catch up on what I was working on before I moved.
Thanks,
Tom

On Dec 31, 2022, at 2:24 PM, Jeff Otterson @.***> wrote:
Winter Field Day is coming! I hope your move went well, Tom.
One of the changes I am going to add at some time is the ability to control which displays are shown. Feedback from one local club is that they were interested in the raw scores, but not so much the per-operator scores. Personally, I like the per-operator scores, but the plan is to make which panels are shown (and perhaps for how long) configurable.
73, Happy New Year.
—
Reply to this email directly, view it on GitHub https://github.com/n1kdo/n1mm_view/issues/39#issuecomment-1368268411, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC6TWSWRI5ZDVTVZLWJO47TWQCB6JANCNFSM5Z7UWHRQ.
You are receiving this because you authored the thread.

Thank you for your code contribution.  This issue seems to now be fixed.


Feature karma https webserver

Specifies 2 new config options: protocol and httpsServerOptions to allow Karma web server to run on https if desired. Default is still http.

@Dignifiedquire I have made adjustment for deeper integration. The config value is not available in the unit tests as they test some classes directly without going through config. Changed implementation to use function arguments where possible, and updated unit tests accordingly. Feels more in line with how the app is set up.

Should this line be updated to 'http[s]?:\\/\\/...'? I am not sure what this function does, but it's the last hardcoded occurrence of http:.

Should this line be updated to 'http[s]?:\/\/...'? I am not sure what this function does, but it's the last hardcoded occurrence of http:.

Yes this needs to be updated as well

Changes made. The travis build seems a bit flaky but I don't think it's caused by my changes.

Needs rebase onto latest master

Tests added and pulled in latest master.

Thanks @KrekkieD now squash it all together and we can get this shipped :)

I believe, after some Googling on how to do that properly after pulling in commits from master, it is done ;)

Thanks :octocat:

Released in v0.13.5


fix: compilation hash should hash hot index to avoid hmr loop

Summary
Because the hash of [hash].hot-update.js/json is previous compilation hash, the diffrent compilation hash maybe at same .hot-update.js file, it will cause hmr loop if you cached .hot-update.js, because the compilation hash isn't correct. For detail:
Status A(hash: A) -> Status B(hash: B) will generate A.hot-update.js
Status A(hash: A) -> Status C(hash: C) will generate A.hot-update.js
The status is different, should generate different hash for .hot-update.js, you can see webpack at https://github.com/webpack/webpack/blob/main/lib/HotModuleReplacementPlugin.js#L425.
So use compilation hash update hot_index to fix it.

Test Plan
Added.

!canary

Is this related to https://github.com/web-infra-dev/rspack/issues/3554?

Is this related to #3554?

Maybe it is, let user have a try?


Update dependencies

Pull Request Test Coverage Report for Build 6770

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 31.524%

Totals

Change from base Build 6756:
0.0%

Covered Lines:
1862

Relevant Lines:
5521

💛  - Coveralls


[WIP] Fix #4082 Auto-correct empty class definitions

Resolves: https://github.com/rubocop-hq/rubocop/issues/4082
Part I:
# bad
class FooError < StandardError
end

# good
class FooError < StandardError; end

I am not sure that all test cases covered.
Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Just for the reference, style link https://rubystyle.guide/#single-line-classes

@denys281 Ping :-) I see the build is failing as well.

Will check! It seems, that work was unfinished :)

@denys281 Great! Ping us when we should review the PR again.

There is one edge case here.
Consider:
class Foo
  # Note:
  #
  # Some extensive comments about Foo and why it
  # is not implemented, etc. How would we deal with
  # this if the cop were to implement auto-correct?
end

Is this empty or not empty? 🙂

There is one edge case here.

Move docs above class definition?

@denys281 Great! Ping us when we should review the PR again.

@bbatsov  ping :)

@bbatsov  ping?:)

There's a lot of activity and @bbatsov has been a bit behind recently, but I can comment that the style guide has been changed in the meantime (sorry, and it's my "fault" 😅).
The best would be that this cop can be configured, either for preference of one-line, or two-line, or either, with default for two-line, and the same choice of settings for exception classes?

Hm, well it need to be added some configuration options. Currently it is only one configuration regarding Exceptions only cases.

Right, but this cop can not be merged as is, since it currently enforces the contrary of what the style guide now says...

Understood. So what was changed?

Ok, I see it https://github.com/rubocop-hq/ruby-style-guide#single-line-classes. So is it worth to change this PR to match new style guide? In case yes, what requirements could be?

So is it worth to change this PR to match new style guide? In case yes, what requirements could be?

Yes, it is worth it. Yes, it should match the new style guide, by default, but it would be nice to have a config.
As I said, ... "The best would be that this cop can be configured, either for preference of one-line, or two-line, or either, with default for two-line, and the same choice of settings for exception classes"

I'm closing this PR due to no recent activity. Feel free to re-open it if you ever come back to it.


Move 'ml' and 'cloud' outside of dvc/dvc

This will allow us to separate core dvc logic from ml use cases and create first real user for dvc python API.

Closing as it is no longer relevant.


Test PR build [run-systemtest]

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Fixed :)


Add ability to specify different listen.owner / listen.group from pool's user and group

Add ability to specify different listen.owner / listen.group from pool's user and group
Added all defaults to be absolutely compatible to the last available version.
These new options allow for a more secure setup where web server may be running by a different user to prevent unauthorized file manipulation...
Also noticed that there was a type in one of the files, where value of USER was being assigned instead of GROUP where GROUP was expected.
This commit is absolutely backwards compatible, although I didn't have an ability to test it, edited all files directly on github.com
Please accept this merge request, I can't deploy new servers without these changes. Thank you.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@virtuman

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Are the Rubocop lints the only thing holding this back? These fixes are holding back an upgrade on my end, as well.

Sorry about Advertising
I need contributors for my project. If you would like to help the project is located here.
https://github.com/poKe-Websites/poKe-CMS
Thank You :)

Closing since #169 was merged


Support of Custom Translate and Transliteration in LUDown Translate

Tool
Name: Ludown
Describe the problem
Currently, LUDown translate uses the standard Microsoft Translator API.  There are scenarios where a custom translator performs better and needed. The same applies to custom transliteration.
Describe the solution you'd like
Provide in the "ludown translate" an option where a custom translator and transliteration web endpoint
can be used for translation and transliteration.
Describe alternatives you've considered
N/A.
Additional context
N/A.

@daden-ms this can be very tricky in that utterances with labelled entity values need to translated and mapped back correctly. E.g. There is explicit dependency on word alignment available as part of the translate service call that needs to be mapped back.
Do not have any plans to add this kind of capability into the core of Ludown CLI itself. That said, Ludown is available as a library and so it should be possible for you to wrap the parse and refresh command groups with appropriate translate/ transliterate modules.


refactor _get_tif_tolerances

For the three tests I ran as part of #23 , these tolerances result in:
names = ['hyp3-testing-9Apv1Hj', 'hyp3-testing-evcd52X', 'hyp3-testing-pIcNMBf']

hyp3-testing-9Apv1Hj/main hyp3-testing-9Apv1Hj/develop -- fail
hyp3-testing-evcd52X/main hyp3-testing-pIcNMBf/develop -- pass
hyp3-testing-pIcNMBf/main hyp3-testing-pIcNMBf/develop -- pass

hyp3-testing-9Apv1Hj/main hyp3-testing-evcd52X/main -- fail
hyp3-testing-evcd52X/main hyp3-testing-pIcNMBf/main -- pass
hyp3-testing-pIcNMBf/main hyp3-testing-9Apv1Hj/main -- fail

hyp3-testing-9Apv1Hj/develop hyp3-testing-evcd52X/develop -- fail
hyp3-testing-evcd52X/develop hyp3-testing-pIcNMBf/develop -- pass
hyp3-testing-pIcNMBf/develop hyp3-testing-9Apv1Hj/develop -- fail

Alright, since #23 passes all 9 of the above, I think the "right" (whatever that means) tolerances is somewhere between these two -- I'm going to merge and then slowly move back towards the tolerances in #23 until all the main v main and dev v dev runs pass


fix: metadata_collector metric are being overwritten by collectors

Fixes #1707

double checking our earlier conversation - you confirmed this changed does not affect autosupport?


[omnibus] strip symbols

What does this PR do?
This PR strips the symbols to the binaries/libs on linux.
The current problem with the PR is these symbols are currently deleted after being stripped, we don't want to do this, we want to zip them up and upload them somewhere. So that's pending.
Motivation
Decrease package size.
Additional Notes
Please make sure we save the symbols somewhere.
Sister PR: https://github.com/DataDog/omnibus-ruby/pull/91 (all heavy lifting done there)

This sister PR will also need to be merged before this one: https://github.com/DataDog/omnibus-ruby/pull/94


FileManager.filePermits semaphore is unfair, blocking scans under high load

User reported seeing messages in their logs where it was taking hours to reserve files for scan. Prior to 1.9.0 the FileManager.filePermits semaphore was fair, but was changed in this commit. The commit message says, "Also in this change the file managers semaphore was switched to non-fair.  The fair semaphore is expensive and can cause contention." Fair locks and semaphores have a penalty, which seems unnecessary when not running in a high contention heavy-load situation. However, when in that situation, they allow for many threads to make some progress vs a small number of threads making progress and the others doing nothing.
@keith-turner - would like your thoughts on this as you were the author of the referenced commit. Was the penalty for the fair lock severe?

@dlmarion I can work on this if you want to assign it to me

@keith-turner - would like your thoughts on this as you were the author of the referenced commit. Was the penalty for the fair lock severe?

I can not remember the specifics of what I was doing.  I don't know if my comment was based on performance testing or just reading about the performance for the fair semaphore.  Given the bad situation that is happening sometimes, it seems like it would be best to switch it to fair.
Should this change need to be made in 1.10.x?

Should this change need to be made in 1.10.x?

Maybe. I'll let @ddanielr evaluate that. If it's easy, then we should probably do that.

Should this change need to be made in 1.10.x?

Maybe. I'll let @ddanielr evaluate that. If it's easy, then we should probably do that.

I"m still looking into this code so I don't have a complete solution yet.
The FileManager code is extremely similar between 1.10 and 2.1 so if the fix is constrained to just that class then I don't see why it couldn't be released to 1.10.x considering this problem's impact.
@dlmarion do you know if the user was experiencing these extended scan blocks while using Scan Executors?

Yes, it was during scanning

From the java docs:
Generally, semaphores used to control resource access should be initialized as fair, to ensure that no thread is starved out from accessing a resource. When using semaphores for other kinds of synchronization control, the throughput advantages of non-fair ordering often outweigh fairness considerations.

Thread starvation for scans seem extremely problematic in a high load environment. Especially so once you add in expected behavior of the Scan Executors thread pools.
A fair semaphore ensures this can't happen by enforcing a FIFO queue for resource requests once all of the available file permits are reserved.
This code is pretty stable between 1.10 and 2.1.x so given the impact to scans, this bug should be fixed in the 1.10.x line.

I added 2.1.1 as a version in which this is fixed, since the fix for this will appear in both the 1.10.3 and the 2.1.1 patch releases.


FixCode: for missing witnesses with default definitions, avoid inserting stubs.

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test


FormBuildRadio should support Direction Parameter

This List should support direction param so we could define axis.horizontal vs vertical

Hi @WilliamCunhaCardoso,
This is a nice idea. But how exactly to implement it would be a bit of an issue. For example how many radios to show in a line, the width of the items, how to handle overflows, etc.
More ideas, suggestions on how to handle this, or better yet a PR will be highly appreciated.

I was about to raise a similar issue as well.
@danvick I think usually when we use the horizontal radio button options, we are already aware that we will only have few and fixed options (e.g. genders - male, female, others). So I think without the overflow handling, this would still be a great feature (maybe can just put some note on the API).

Kindly use FormBuilderRadioGroup field.


Simple cipher: fix comment grammar

The original comment said: "Here we take advantage of the fact that plaintext of "aaa..." doesn't output the key." If I understand correctly, this is the opposite of what was meant, that is, "Here we take advantage of the fact that plaintext of "aaa..." outputs the key."

More thanks!


Updated the code, state_size was pointing to the wrong value

JIRA: MADLIB-1325
state_size was not updating and always has the wrong value ( -1)
Updated the code to point it to the correct value.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/934/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/941/

No tolerance :
madlib=# SELECT madlib.mlp_classification(
madlib(#     'iris_data',      -- Source table
madlib(#     'mlp_model',      -- Destination table
madlib(#     'attributes',     -- Input features
madlib(#     'class_text',     -- Label
madlib(#     ARRAY[5],         -- Number of units per layer
madlib(#     'learning_rate_init=0.003,
madlib'#     n_iterations=25,
madlib'#     tolerance=0',     -- Optimizer params
madlib(#     'tanh',           -- Activation function
madlib(#     NULL,             -- Default weight (1)
madlib(#     FALSE,            -- No warm start
madlib(#     TRUE             -- Not verbose
madlib(# );
INFO:  Iteration: 1, Loss: <1.57403083382>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 2, Loss: <1.18209701678>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 3, Loss: <0.805838101786>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 4, Loss: <0.456743716108>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 5, Loss: <0.269013324379>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 6, Loss: <0.176825519276>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 7, Loss: <0.12831471301>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 8, Loss: <0.0995580174594>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 9, Loss: <0.0808205589525>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 10, Loss: <0.0679497236685>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 11, Loss: <0.058451236734>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 12, Loss: <0.0510716510938>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 13, Loss: <0.0453356242116>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 14, Loss: <0.040657636203>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 15, Loss: <0.0368149574231>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 16, Loss: <0.0336085514468>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 17, Loss: <0.0309155266308>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 18, Loss: <0.0286056560803>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 19, Loss: <0.0266028934429>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 20, Loss: <0.0248412801724>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 21, Loss: <0.023296777543>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 22, Loss: <0.0219201410755>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 23, Loss: <0.0206909609214>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 24, Loss: <0.0195874516687>
CONTEXT:  PL/Python function "mlp_classification"
 mlp_classification
--------------------

(1 row)

tolerance=0.1 :
madlib=# SELECT madlib.mlp_classification(
madlib(#     'iris_data',      -- Source table
madlib(#     'mlp_model',      -- Destination table
madlib(#     'attributes',     -- Input features
madlib(#     'class_text',     -- Label
madlib(#     ARRAY[5],         -- Number of units per layer
madlib(#     'learning_rate_init=0.003,
madlib'#     n_iterations=25,
madlib'#     tolerance=0.1',     -- Optimizer params
madlib(#     'tanh',           -- Activation function
madlib(#     NULL,             -- Default weight (1)
madlib(#     FALSE,            -- No warm start
madlib(#     TRUE             -- Not verbose
madlib(# );
INFO:  Iteration: 1, Loss: <1.47220611074>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 2, Loss: <1.24578776394>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 3, Loss: <0.887573516227>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 4, Loss: <0.530981965126>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 5, Loss: <0.306411380721>
CONTEXT:  PL/Python function "mlp_classification"
INFO:  Iteration: 6, Loss: <0.195035410704>
CONTEXT:  PL/Python function "mlp_classification"
 mlp_classification
--------------------

(1 row)

LGTM
@hpandeycodeit @njayaram2   Just to confirm:  in the case of grouping, all groups must meet the tolerance threshold before training is ended?


Install admission webhook for v0.1.0-rc3 kubefed controller manager

@abhat I think it is safe to close this issue, right?
@qinpingli wdyt?

@sohankunkerkar Yes, we can close this issue.


Demo version

What is the deno version of supabase edge functions? How can I update it?? I cannot run "npm:express" super hard to install "firebase-admin"

Hey there, we're currently pinning the Deno version to v1.20.3 because of some issues with deno bundle. Eventually we'll always use the latest stable Deno version. You can track this issue here: https://github.com/supabase/cli/issues/490


Cirrus: Support testing with F31

Add F31 + Drop F29
Depends on: #3754 #4028 #4028 #4154 #4222 #4825
Fixes #5285
Fixes #3447
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

F31 VM setup packaging failures:
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] No match for argument: atomic-registries
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package bzip2-1.0.8-1.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package findutils-1:4.6.0-24.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package gnupg2-2.2.17-2.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] No match for argument: golang-github-cpuguy83-go-md2man
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package iproute-5.2.0-2.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package libseccomp-2.4.1-1.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package procps-ng-3.3.15-6.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package python-unversioned-command-3.7.4-5.fc31.noarch is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] No match for argument: python2-future
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package python3-dateutil-1:2.8.0-3.fc31.noarch is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package python3-psutil-5.6.3-2.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package which-2.21-15.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Package xz-5.2.4-6.fc31.x86_64 is already installed.
[+0148s]
    fedora-31: [2019-08-30T09:12:56-04:00] Error: Unable to find a match: atomic-registries golang-github-cpuguy83-go-md2man python2-future

All: Earliest working F31 cache-image build is fedora-31-libpod-5654728358232064 that should fire up successfully with hack/get_ci_vm.sh.  Please play!
(I will work on stripping the xfedora / crun stuff and just install crun)

Note to me:  We need both runc and crun testing happening in F31 and F30.

runc tests will not pass in F31 until runc supports cgroupsv2 or you need to boot the kernel in cgroupv1 mode

or you need to boot the kernel in cgroupv1 mode

Yeah, that's what I'm worried about needing to do.  The only way that's possible (the booting part) is by using a custom, snowflake image - a.k.a. a PITA.
We currently do this for giuseppe's playground in F30 and I was hoping to be able to drop that in this PR...now it sounds like I need to ADD another snowflake image (the 'P' part of the PITA).  It complicates automation and ongoing maintenance - and therefor impacts reliability, and flakes, and sleepless nights, and ... :disappointed:
Maybe we can "strategically" decide to not ship runc with F31 and drop crun from F30 :smile_cat:
@rhatdan is there significant risk added if we were to only test crun in F31 and only test runc in F30?
(with the understanding we may need to swap testing to runc in F31 later if patches are accepted)

note to me: Still need to address the what/how/where of crun and runc.  Last commit does:

Test crun in F31
Test runc in F30 and Ubuntu

/hold

I am fine with only testing crun on f31 and runc on f30.  Once we get to f31 and f32 and runc has support for cgroupfs, we can start testing with both.

Got a new F31 cloud-image on the download site, produced a new F31 base image.  Rolling new cache-images now.

note to me: Need new crun 0.10 to prevent VM lockups and test timeouts

note to me: Ubuntu failures expected (no builds happening w/ packaged conmon).  Need #4222 to help investigate system-test failures (varlink process died?)

grumble grumble grumble Even with the official F31 cloud images, I've got VMs locking up during integration testing (different test every time).  For example: https://cirrus-ci.com/task/6425946084605952
Oddly, it always seems to happen about 20-25minutes into testing :confounded:

linked https://bugzilla.redhat.com/show_bug.cgi?id=1767539 kernel panic in F31

Attempting to avoid kernel panic by switching to deadline scheduler on F31.

/hold cancel

@giuseppe @haircommander mind taking a peek at these two F31 rootless failures (they're similar)?

special_testing_rootless TEST_REMOTE_CLIENT:false
special_testing_rootless TEST_REMOTE_CLIENT:true

Note: I already temporarily disabled podman-in-podman testing because of massive numbers of failures.  So debugging that will need to come also / after this PR merges.  I'm also mentioning this in case it's related to the rootless test failures.

(Peter I mentioned you because I seen the word "conmon" in some of the error messages :smile: )

This is still something we want.

Status: Working on verifying the BFQ kernel patch (this PR reproduces the panic).  Will resume here once the fix is confirmed (and I no longer need this PR to repro the problem)...

...no dice, patch does not fix problem.  Now reported/tracking upstream: https://bugzilla.kernel.org/show_bug.cgi?id=205447
To allow libpod to move forward, I'll put my workaround back in place, build fresh VM images, and go from there.  Then the kernel panic debugging can continue in parallel.

note to me:  PR took too long, it's F30 and F31 base images got pruned, need to re-add them.

/approve

LGTM

/lgtm


{Packaging} Drop Alpine docker image

Alpine is deprecated in https://github.com/Azure/azure-cli/pull/29628
We also removed it in release pipeline.

Packaging

Should we remove alpine.dockerfile too?


ModuleNotFoundError: No module named 'future'

C:\Users\Doyle\PycharmProjects\DjangoTest>python manage.py makemigrations
Traceback (most recent call last):
File "manage.py", line 15, in
execute_from_command_line(sys.argv)
File "C:\Users\Doyle\venv\DjangoTest\lib\site-packages\django\core\management_init_.py", line 371,
in execute_from_command_line
utility.execute()
File "C:\Users\Doyle\venv\DjangoTest\lib\site-packages\django\core\management_init_.py", line 347,
in execute
django.setup()
File "C:\Users\Doyle\venv\DjangoTest\lib\site-packages\django_init_.py", line 24, in setup
apps.populate(settings.INSTALLED_APPS)
File "C:\Users\Doyle\venv\DjangoTest\lib\site-packages\django\apps\registry.py", line 89, in populate
app_config = AppConfig.create(entry)
File "C:\Users\Doyle\venv\DjangoTest\lib\site-packages\django\apps\config.py", line 90, in create
module = import_module(entry)
File "C:\Users\Doyle\AppData\Local\Programs\Python\Python36\lib\importlib_init_.py", line 126, in i
mport_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 978, in _gcd_import
File "", line 961, in _find_and_load
File "", line 950, in _find_and_load_unlocked
File "", line 655, in _load_unlocked
File "", line 678, in exec_module
File "", line 205, in call_with_frames_removed
File "C:\Users\Doyle\PycharmProjects\DjangoTest\xadmin_init.py", line 4, in
from xadmin.sites import AdminSite, site
File "C:\Users\Doyle\PycharmProjects\DjangoTest\xadmin\sites.py", line 3, in
from future.utils import iteritems
ModuleNotFoundError: No module named 'future'

it'ok.you can slove it :  pip install --upgrade future
others: you should pip others framework. ex. six,django-import-export,django-reversion,
django-formtools,future,httplib2,six


Add Font Awesome CSS to the Jekyll template

Please add the link to the font awesome CSS to the Jekyll template.
https://use.fontawesome.com/releases/v5.15.4/css/all.css

Would you want it as a default on all the pages or as a property on page-to-page basis?

That's a good question.  I don't think there is much harm in making it available, but if it's rarely used, then it's a waste.  Why don't we put it at the page level, then people can use it with intension.

Agree. And I'm thinking of a feature that I could push to GCWeb-Jekyll in the future where you could have a property for Font Awesome (FA) itself and you could choose which version of FA to use on your page. That could facilitate the migration to newer version of FA in the long run.


Review: Verify dev environment is deployed correctly

Coordinate with PnP-Deploy team to verify environment setup

Related issue #653

The backend pool for the Application Gateway created gets the URL "bu0001a0008-00.aks-ingress.contoso.com".
Tracking this as we get a service deployed to see if this needs to be changed

@jkeane showed the solution of changing "bu0001a0008-00.aks-ingress.contoso.com" to "bu0001a0008-00.ngsa-monitoring-dev.com" that will be included in an upcoming PR.
This is the only issue that I saw in the initial configuration and it has been updated


Update data-retention-cleanup.md

Fixed a few typos and small consistency changes

@rwestMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
Thanks @myatt83, we're planning to refresh this content in the next while, but this is a good change in the meantime.


Update the yaml schema definition and related docs

The tye.yaml has a schema definition file (tye-schema.json) found here: https://github.com/tye-v2/tye2/tree/main/src/schema.
The README.md describes how it can be used in VSCode to get schema validation.

The following should be fixed:

[ ] update the readme (and any doc pages if any) with an url pointing to the file in the tye2 repo

[ ] https://github.com/tye-v2/tye2/blob/main/docs/reference/schema.md

[ ] update the tye-schema.json to reflect what is currently possible to define in the tye.yaml

[ ] For example the routes enumeration is shown as an error, althow the tool accepts it's values

Could you go with the PR?


Upgrading CLI versions causing problems w/ polyfills

[ ] 1.x (For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x ] 3.x
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
After upgrading CLI versions I'm getting this error that I haven't encountered before.
I've tried multiple times to reinstall the Ionic CLI and ran npm install/update and haven't had any success in resolving this issue.

    @ionic/cli-utils  : 1.8.1
    ionic (Ionic CLI) : 3.8.1

global packages:

    Cordova CLI : 7.0.1

local packages:

    @ionic/app-scripts : 1.3.0
    Cordova Platforms  : android 6.2.3 ios 4.4.0
    Ionic Framework    : ionic-angular 3.4.2

System:

    Android SDK Tools : 25.2.5
    Node              : v6.10.0
    npm               : 3.10.10
    OS                : Windows 10

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having?
Also, I am moving this issue to the ionic-cli repo since this appears to be an issue with the CLI.
Thanks for using Ionic!

This issue was moved to ionic-team/ionic-cli#2640


[SourceKit] Save clang module validation time

Pass -fbuild-session-timestamp and -fmodules-validate-once-per-build-session to ClangImporter so that the module validation happens only once for the SourceKit lifetime.
rdar://problem/59567281

@swift-ci Please smoke test

sourcekitd-test -req=sema <..> == -copy-file <..> == -req=sema <..>

The second sema invocation probably needs some extra '-D' argument to force creation of a new AST, make sure it's not just re-using the first AST. Generally this test line should be failing without your changes.

Probably also good to do a similar test using complete request, to verify we avoid system header validation for code-completion.

We should also make sure we have a test for modifying a user header, where we expect to see the changes immediately.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 465d20d2220572b10a296e421131dc25c84b4203

Build failed
Swift Test Linux Platform
Git Sha - 465d20d2220572b10a296e421131dc25c84b4203

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 465d20d2220572b10a296e421131dc25c84b4203

Build failed
Swift Test Linux Platform
Git Sha - 465d20d2220572b10a296e421131dc25c84b4203

@swift-ci Please smoke test

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c091605bf0e6aac52ff405f942b5af35036dc0b8

Build failed
Swift Test OS X Platform
Git Sha - c091605bf0e6aac52ff405f942b5af35036dc0b8

@swift-ci Please test Windows

@swift-ci Please smoke test

Windows testing is running in https://ci-external.swift.org/job/swift-PR-windows/996/

Windows failure is unrelated. Merging


Release synthtool 0.4.0

This pull request was generated using releasetool.

Release is at https://github.com/GoogleCloudPlatform/synthtool/releases/tag/synthtool-0.4.0


Log overflow with logs

Describe the bug
Looking at the log - there are multiple lines appearing again and again that presumably needs to be in DEBUG:
e.g.:
2024-08-11 10:03:27.711 INFO (Thread-79) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 404
2024-08-11 10:03:30.272 INFO (Thread-81) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 405
2024-08-11 10:03:32.761 INFO (Thread-82) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 406
2024-08-11 10:03:35.240 INFO (Thread-84) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 407
2024-08-11 10:03:37.714 INFO (Thread-86) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 408
2024-08-11 10:03:37.737 INFO (Thread-86) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 409
2024-08-11 10:03:40.244 INFO (Thread-88) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 410
2024-08-11 10:03:42.762 INFO (Thread-90) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 411
2024-08-11 10:03:45.255 INFO (Thread-92) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 412
2024-08-11 10:03:45.276 INFO (Thread-92) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 413
2024-08-11 10:03:47.721 INFO (Thread-93) [custom_components.dreame_vacuum.dreame.map] Decode P map 2 414

2024-08-11 10:04:11.078 INFO (Thread-110) [custom_components.dreame_vacuum.dreame.device] Property CLEANED_AREA Changed: 16 -> 17
2024-08-11 10:04:17.223 INFO (Thread-114) [custom_components.dreame_vacuum.dreame.device] Property BATTERY_LEVEL
2024-08-11 10:04:20.417 INFO (Thread-116) [custom_components.dreame_vacuum.dreame.device] Property CLEANING_TIME Changed: 13 -> 14

To Reproduce
Using Dreame F9 - start a cleanup.
Expected behavior
2-3 log lines indicating that the cleanup has started
Screenshots
If applicable, add screenshots to help explain your problem.
Additional Information (please complete the following information)

Model Name [e.g. dreame.vacuum.p2028]:  dreame.vacuum.p2008
Firmware Version [e.g. 1156]: 1107
Home Assistant Version: 2024.8
Configuration Type [With or without map support]: With
Errors or warnings shown in the HA logs (if applicable):

🤷🏻  OK


about training codes

非常感谢你的这个具有开创性和启发性的工作。我希望可以follow and replay your work.
尤其是关于如何去train parallelSpeechCLIP的部分。但是我并没有在repository中找到。
请问，能不能大概提供一下training codes on how to call the proposed speech encoder and image encoder of CLIP，之后又是如何计算contrastive loss, followed by how to backward propagation.
或者提供一些reference codes/blogs about training codes.
非常感谢！

I have completed this project already. Thanks for your works as well.


DPIv2: Video Plots + Tasks

Incredible apologies for the delay, but this should be all the necessary plots + constants for the video part of the paper. Let me know if other analyses are needed. I will PR the multimodal_analysis stuff sepertely/or add it here.
How to Review:
It's arranged by commits so I would look at those instead of the files directly.

Video plots are in here
Constants: here
Task Summaries: won't be too useful to look at cause there are > 100 jsons but incase

@ktiwary2 great start! A couple minor things:

You'll notice that in re-generating the data summaries you deleted the inferrred metadata and bibtex info that we had generated for each one. Could you make sure to add your new metadata without deleting those parts?
We would like all plots to be generated through multimodal_analysis.ipynb, rather than duplicated code in its own notebook. When we are generating the same or concatenated figures for all 3 modalities we need to have the same code responsible for those, so we don't need to make the changes in multiple places. Could you merge it into multimodal_analysis, so we are using the same pipelines?
Ideally we should be building out multimodal_analysis, not adding too much stuff to it. Wherever possible please edit or add any code to multimodal_util, so that the analysis notebook is simply calling the data processing and plotting functions.

Instead of rebasing, I pulled and that's why there are so many commits here. I will fix this so that the PR is self contained.
In the mean time let me know what the feedback is on the following notebooks:  multimodal_analysis.ipynb and from video_analysis.ipynb.
Link to the multimodal and video specific plots: https://drive.google.com/drive/folders/1LUf4SRD53QLH8VmpnZX--K2Lwe5AOs00?usp=sharing

Closing this; please see https://github.com/Data-Provenance-Initiative/Data-Provenance-Collection/pull/181 which resolves the issues above + the comments.


[Hugo Migration] Migrate Notebook posts to Academic's "Documentation" setup

Mostly posting here for posterity, @causallycausal and @ionlights have already finished this.

We still need to test the generalizing of our work on this. This partially involves editing autobot, but an incoming commit will have some demo files to work from.


doc: boards: Edit nxp board and debug probe docs

Incorporates minor documentation edits recommended by David Kinder after
PR #14000 was merged.
Signed-off-by: Maureen Helm<EMAIL_ADDRESS>

Codecov Report

Merging #15088 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #15088   +/-   ##
=======================================
  Coverage   52.87%   52.87%
=======================================
  Files         309      309
  Lines       45267    45267
  Branches    10451    10451
=======================================
  Hits        23934    23934
  Misses      16558    16558
  Partials     4775     4775

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 1eead93...a874864. Read the comment docs.


[Feature Request] Screen Module button "switch' Option

It would be great to have an option on the screen button module to make its appearance more like a 'switch'. With simple 'on/off' item the button is hard to read. It would also be nice to allow 'inverting' the on/off on the display for a system that will be turned on with no redstone signal vs a system that can be turned on with redstone signal.
Here's a bad mockup left being current and the right being requested feature.

A Redstone Lever module would be very nice indeed

A workaround meanwhile is to use a Redstone Module that displays the current status (on vs off).

You are all aware that screen buttons can already be in 'toggle' mode right?

Yep, but I believe OP's opinion is that it can be hard to tell which state it's in at a glance.
I don't really have an opinion on the matter myself, although I might find being able to change the button's text based on the on/off status.

With the above pictures, it is hard to tell which state something is in. Especially when things can differ and be on or off with or without a Redstone signal. It would just be nice to have some more functionality if possible.

Currently it's hard to tell at a glance if a button is toggled.  I love OPs suggestion, or even just an extra text field for a button module's toggled state.  e.g. Off when not pressed, On when pressed etc..
Also an invert signal option would be really handy when controlling machines that are on by default.


Cannot use within another directive

I have a directive which creates an editiable form via templateUrl and none of the directives will work..

Nevermind this was due to an outter form element .. works inside directive..


yask: Support for acoustic adjoint operator and misc

Support for temporary grids
tests for reverse loops, temporary grids, adjoint operator
Misc improvements all over the yask backend
Small fixes to some DSE routines

All good, so this one is up next - again depending on our old friend Travis.

Merged.


feat: consider string number as milliseconds

consider string number as milliseconds when create a Moment

@0Ncoder0 I'm not 100% sure this is a good change. If you know this is a milliseconds string, why don't you convert to number before passing to moment? This input comes from a dev, not from a user (I hope) so the dev can handle a + sign...
Also it doesn't have tests.


Correct YAML examples to use upper case enum of areaType

Problem description
To correct YAML examples to use upper case enum of areaType:

CIRCLE

Expected action
Correction of the YAMLs to reflect the correct enum
Additional context
enum for areaType introduced in #132

Correct the following to reflect radius as well, instead of accuracy:
    example:
        areaType: Circle
        center:
          latitude: 50.735851
          longitude: 7.10066
        accuracy: 50000

Thanks, have to be fixed. We have to create a PR for it


Issue with inout port of constraint file

This problem was found primarily trying to make Amaranth HLS work with CC GateMate. The issue is that Amaranth, based on the Python code of the module, creates the given constraint file but it doesn't have the option to get pin direction data because no other vendor uses pin direction declaration inside the constraint file. The declaration is done in RTL.
I tried getting over this problem by using Pin_inout generally. But when doing the simple blinky example in Amaranth doesn't work. But when manually setting the pin directions correctly to Pin_in or Pin_out where needed it works fine.
Then I tried a simple Verilog Blinky example where I change Pin_out of the LED on the Olimex GateMate board to Pin_inout and then the example doesn't work.
Why is that? Why doesn't it work and what changes in the post-placement/routing netlist or bitstream to make it not work?
What change in the netlist makes it work again?

What exactly do you want to achieve by specifying an output port as Pin_inout?
Pin_inout defines a bidirectional pin, as described in the documentation and here. In addition, your definition will at least lead to such a warning, as P&R cannot explicitly assign the pin.

Pin_inout works perfectly well with bidirectional pins, CC_IOBUF, to be more precise. #20 is therefore not an error. I will get back to you with a solution to the other discussion about the CCF format after consultation with my team.

What exactly do you want to achieve by specifying an output port as Pin_inout?

In my case I have tried to get PS2 working on Olimex board, and managed to get keyboard running as it only needs input data.
Then I have tried mouse core that needs inout out - changed in to inout in verilog and ccf, but did mouse did not work.
Then I tried with working keyboard code that is working if ccf is set as input, but it does not work if I just change pins to inout in ccf.

Thank you @goran-mahovlic , can I find the mouse core in this repository? I'd like to take a look at it.

PS2 core is here - https://github.com/trabucayre/GateMate_demos/tree/main/colorBarVGAPS2
It is will create Keyboard example that stops working if pins in ccf are set inout https://github.com/trabucayre/GateMate_demos/blob/main/colorBarVGAPS2/colorBarVGAPS2.v#L7

With parameter ps2core being set to 2 , we will instantiate the ps2kbd module.
As you can see in the module ports, there is no inout port at all:
https://github.com/trabucayre/fpgalibs/blob/4672011c87b7ea3c89218b7537bf5d90429b26a9/hdl/ps2kbd.v#L7C1-L15C3
Yosys log confirms:
     CC_ADDF                        53
     CC_BUFG                         1
     CC_DFF                        122
     CC_IBUF                         4 // only input ...
     CC_LUT1                        10
     CC_LUT2                        74
     CC_LUT3                        23
     CC_LUT4                        39
     CC_MX4                         14
     CC_OBUF                        17 // ... and output buffers
     CC_PLL                          1

With parameter ps2core set to 0, however, we will describe the behavior of an IOBUF:
Yosys log confirms:
     CC_ADDF                        87
     CC_BUFG                         1
     CC_DFF                        177
     CC_IBUF                         2
     CC_IOBUF                        2 // here is our IOBUF
     CC_LUT1                        19
     CC_LUT2                       113
     CC_LUT3                        25
     CC_LUT4                       102
     CC_MX4                         14
     CC_OBUF                        17
     CC_PLL                          1

P&R will map these buffers into IOBUF cells.
Btw., with the latest P&R release, you can now specify the NET keyword instead of the pin direction to automatically determine the direction:
Net  "ps2clk" Loc = "IO_WB_A0";
Net  "ps2data" Loc = "IO_WB_B0";

https://colognechip.com/downloads/cc-toolchain-linux.tar.gz
https://colognechip.com/downloads/cc-toolchain-win.zip

With the new CC p_r update that introduced the NET keyword. The Amaranth Blinky example works immediately without any p_r errors.
I am continuing the work to make Amaranth work on GateMate boards.


Error at startup

Expected Behavior
Program running
Actual Behavior
print '[#] something rustles nearby!'
SyntaxError: Missing parentheses in call to 'print'
Steps to Reproduce
Pokecli.py -a google -u user -p pass - location location
Other Information
OS: win10
Git Commit: (run 'git log -n 1 --pretty=format:"%H"' and paste it here)

you're using python 3 and we work with python 2

Please close the issue if this resolves your issue.

@DokterPhill Follow the Readme. Feel free to open a new issue when necessary.


Refactor open ephys package

User story
As a developer, I want each of the packages in the metadata-mapper to have a JobSettings.
Acceptance criteria

[ ] Create a module for models in the open_ephys package
[ ] Create a JobSettings class that inherits BaseJobSettings in new module
[ ] Move the parameters currently being inputted into the ETL to JobSettings, and add a Literal field job_settings_name
[ ] Pass job_settings to ETL and refactor ETL methods to retrieve info from job_settings
[ ] Add OpenEphysJobSettings to SessionJobSettings Union in models.py

Sprint Ready Checklist

[ ] 1. Acceptance criteria defined
[ ] 2. Team understands acceptance criteria
[ ] 3. Team has defined solution / steps to satisfy acceptance criteria
[ ] 4. Acceptance criteria is verifiable / testable
[ ] 5. External / 3rd Party dependencies identified
[ ] 6. Ticket is prioritized and sized

Notes
Look at bergamo package as example

Arjun's going to work on this!

working

Closing this issue as completed. Carter's refactor in #174 finished the first few criteria, and there's a new issue for the remaining part (adding it to GatherMetadata job #176)


Hosted mode fix in cert-controller

a reusable GitHub workflow for running the Kind E2E tests in hosted mode was added. We need to now trigger these tests from the component repos as part of the PR CI in releases 2.7+.
Modify the governance-policy-framework release-2.7, release-2.8, and main branches to consolidate this as a new job in the existing reusable workflow file.
This change would apply to the release-2.7, release-2.8, and main branches for the following repos:
https://github.com/stolostron/governance-policy-propagator
https://github.com/stolostron/governance-policy-framework-addon
https://github.com/stolostron/config-policy-controller
https://github.com/stolostron/iam-policy-controller
https://github.com/stolostron/cert-policy-controller
Ref: https://issues.redhat.com/browse/ACM-5211

/cherrypick release-2.8