Add "alerts" tag to identify measures for alert email

Currently measures have to be in the "core" tag in order for alerts to be sent.  This may create issues with additional measures, such as zuclopenthixol, which we may choose not to add to the core group.
Following discussion with @sebbacon I have set up an "alert" tag, which we can add to all measures that are suitable (i.e. most of them).  We'll need to change the code to reflect this when triggering the alert.
One thought though - would it be better for us to have a "no_alert" tag instead, so that we exclude measures which aren't suitable, rather than include ones that are?  @sebbacon @evansd @inglesp ?

We don't want to use a separate tag for this, unless we want "not for alert" to appear in the list of tags for a measure or in the dropdown for "Show measures categorised as..." (which I think would be meaningless for a user).
Instead, we should make a new no_alert attribute on the measure definition, and filter out those measures where this is True when sending alerts.
Does this sound sensible?

Good point. I concur.

Eminently
On Tue, 8 Oct 2019, 10:25 Peter Inglesby<EMAIL_ADDRESS>wrote:

We don't want to use a separate tag for this, unless we want "not for
alert" to appear in the list of tags for a measure or in the dropdown for
"Show measures categorised as..." (which I think would be meaningless for a
user).
Instead, we should make a new no_alert attribute on the measure
definition, and filter out those measures where this is True when sending
alerts.
Does this sound sensible?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ebmdatalab/openprescribing/issues/2127?email_source=notifications&email_token=AEORKZ6LZVZZQO4FWQG7ARDQNRGYVA5CNFSM4I6NVLE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEATQ4EQ#issuecomment-539430418,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEORKZ26WBVP6VS2VHYYRKLQNRGYVANCNFSM4I6NVLEQ
.

I'd got the impression (although I'm not sure from where) that we now want to include all current measures in the alerts.  Is that correct?  @richiecroker

I think that'd be helpful.  We don't currently include individual LVM in there, and I think we should.  @brianmackenna do you agree?

So yes, let's include all LVM, but exclude the lzomnibus

OK, thanks!

There are also two measures tagged as "papers" which aren't currently in "core":
https://openprescribing.net/measure/tamoxifen/
https://openprescribing.net/measure/nimodipine/

Thanks for checking @evansd.

Although is this an oversight?  Is there any reason why they shouldn't be included in alerts?

yes, they're not strictly measure measures...

Just to mention I recall it was a positive decision to exclude individual LP measures from alerts. Not saying it was a good decision. Just that there was a discussion.  I guess it was because many of them, being low numbers, would result in lots of false positives for change over time.

Fair enough, but I now think it'd be useful for individual alerts to start to go out, as we're some time down the line on the programme.

Fixed by #2128.


feat(meta): docs and contributing improvements

Migrates project to pnpm
Adds prettier and associated config
Tweaks issue templates
Adds CONTRIBUTING.md

Basic contributing guidelines, as requested in #371


Pinning a grpc client

Hello there, this library is super cool for me, I don't know why doesn't have more starts :(
A little question: is possible to do pinning with a grpc client ?
There's a secure client for grpc here, but I got lost trying to make it fit with this lib, it looks like it's possible 'cause both handle the idea of tls.Configs in some points.
Other question, are you using this library in production ? I have used it for my toy projects, but now I have intentions to use it in a real project at work.
Thanks in advance!!

Thanks for the support!
I think you could pass the dialConf.NewDialer() from the example into a grpc.WithDialer and then put that after the grpc.WithTransportCredentials of the grpc.Dial.
What I've done with grpc in production is to have a dedicated certificate authority and only trust the root cert, as in kelsey's example. I'm not too sure of the utility using cert pinning in that context. I was mostly thinking of using this package on forks of public service api golang libraries (like the github or digitalocean) but haven't really gotten around to that yet.

Awesome, double thanks!!! yes it looks that WithDialer() should work and yes it makes a more sense just trust in your own ca


Examples using AWS

Rather like the people in this thread:
https://github.com/BD2KGenomics/toil/issues/682
I'm interested in running CWL workflows through Toil with many platforms supported (SGE, LSF, AWS). The documentation mentions "Commercial clouds + Amazon Web Services (including the spot market)" but I can't find any examples of Toil workflows creating EC2 instances to use for execution. How would I do I this with a CWL workflow? If I can get a bit of help with this I'd be happy to make some recipe-style examples and make a PR to put them in the repo.
I've also tried running CWL workflows on my local SGE grid. It looks like each individual CWL job runs in a docker container invoked (I think) through cwltool sandboxjs.py:
trynodes = (["xnodejs"], ["node"], ["docker", "run",
                                   "--attach=STDIN", "--attach=STDOUT", "--attach=STDERR",
                                   "--interactive",
                                   "--rm",
                                   "node:slim"])

this looks like there are a few constraints:

Docker must be working and supported on each individual compute node, which means it won't work where the nodes are running operating systems that aren't fully docker compatible (like CentOS6, which has a kernel that is tool old)?
Docker must work without needing sudo.

Is this correct?
Peter

Hi Peter,
Toil currently relies on cgcloud to provision the cluster in EC2. In a nutshell, you would install and configure cgcloud-core and cgcloud-toil as described here:
https://github.com/BD2KGenomics/cgcloud/blob/master/core/README.rst
https://github.com/BD2KGenomics/cgcloud/blob/master/toil/README.rst
And then run
cgcloud create -IT toil-latest-box
cgcloud create-cluster toil ...
cgcloud rsync toil-leader userscript.py :
cgcloud ssh toil-leader
At the moment, it is not possible to run CWL jobs on a cluster of machines. You can track our efforts on making that possible here:
https://github.com/BD2KGenomics/toil/pull/709
https://github.com/BD2KGenomics/toil/issues/711
@tetron, do you have any insight on Peter's CWL/Docker-related questions?

Hi @psaffrey-illumina,
You're correct that each compute node needs Docker installed and usable by the compute user (typically by putting the user it a docker group).  However, the code in sandboxjs.py you quoted is actually there to run Node.js for executing Javascript expression in the CWL document, and not running general purpose jobs.  For that, you want job.py.  (The sandbox.js code will use the node:slim docker image as a fallback if nodejs isn't installed already).

@psaffrey-illumina, please reopen if you feel your question was not answered satisfactorily.


[FLINK-25839][e2e] Print the previous logs of restarted pod for k8s e2e tests

This PR is trying to print the previous logs of restarted pod. It is very useful to debug the failed K8s related e2e tests(e.g. FLINK-25839).

cc @gaoyunhaii Could you give a pass for this simple change?


/tpr - command for teleporting to specified mc region coordinates

I would like to see command that teleports you to specified minecraft region for example:
/tpr 3200,-6500 or /tpregion 3200,-6500.
Describe alternatives you've considered
Maybe make this as t++ addon that contains extra utility commands?

Duplicate of #3
My opinion of the matter hasn't changed, this should be implemented in CubicChunks instead.
It is also just a matter of multiplying by 256 for 3dr or 512 for 2dr and mca, so you can do it pretty easily with /tp.

this is definitely beyond of the scope of either terra++ or cubic chunks.


Implement setFormType for read only and disable state

Would be great to be able to set the entire form as read only or disabled, programmatically by client API or by a business rule.
Don't get why this is not available yet.

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

ID: f9e7128f-8c02-38ef-60cd-df48cbd7894a
Version Independent ID: b3a92845-c46b-165e-4021-d449f39689bd
Content: getFormType (Client API reference) in model-driven apps - PowerApps
Content Source: powerapps-docs/developer/model-driven-apps/clientapi/reference/formContext-ui/getFormType.md
Service: powerapps
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

Thanks for your feedback. For product-related feedback, visit https://ideas.powerapps.com.
Closing this issue.


add a path flag

Users should be able to specify a custom path to where they want the new project generated without changing GO_INIT_PATH or GOPATH.

added in v0.4.0


No CVEs for tourchserve

PyPI
Snyk
Mend
OSV

I believe this is now available as https://osv.dev/vulnerability/GHSA-8fxr-qfr9-p34w.

https://osv.dev/list?ecosystem=&q=torchserve is now returning two results


fix(deps): update gomod

This PR contains the following updates:

Package
Type
Update
Change

github.com/IBM-Cloud/bluemix-go
require
digest
6ead1f8 -> 3386b53

github.com/IBM-Cloud/power-go-client
require
patch
v1.5.8 -> v1.5.9

github.com/IBM/platform-services-go-sdk
require
minor
v0.56.4 -> v0.59.0

Release Notes

IBM-Cloud/power-go-client (github.com/IBM-Cloud/power-go-client)
v1.5.9
Compare Source
What's Changed

Bump github.com/IBM/platform-services-go-sdk from 0.55.0 to 0.56.0 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/298
Bump github.com/IBM/platform-services-go-sdk from 0.56.0 to 0.56.2 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/300
Bump github.com/IBM/platform-services-go-sdk from 0.56.2 to 0.56.3 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/304
Bump github.com/IBM/go-sdk-core/v5 from 5.15.0 to 5.15.1 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/305
Bump github.com/IBM/platform-services-go-sdk from 0.56.3 to 0.56.4 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/308
Update swagger client and models to service-broker v1.138.0 by @powervs-ibm in https://github.com/IBM-Cloud/power-go-client/pull/309
Update swagger client and models to service-broker v1.138.0 by @powervs-ibm in https://github.com/IBM-Cloud/power-go-client/pull/310
Update swagger client and models to service-broker v1.138.0 by @powervs-ibm in https://github.com/IBM-Cloud/power-go-client/pull/311
Bump golangci/golangci-lint-action from 3 to 4 by @dependabot in https://github.com/IBM-Cloud/power-go-client/pull/314
Enable shared-processor-pool functionality on stratos by @ismirlia in https://github.com/IBM-Cloud/power-go-client/pull/312
Update swagger client and models to service-broker v1.139.0 by @powervs-ibm in https://github.com/IBM-Cloud/power-go-client/pull/316
Add Bulk Attach, Delete & Detach Volume by @michaelkad in https://github.com/IBM-Cloud/power-go-client/pull/313

Full Changelog: https://github.com/IBM-Cloud/power-go-client/compare/v1.5.8...v1.5.9

IBM/platform-services-go-sdk (github.com/IBM/platform-services-go-sdk)
v0.59.0
Compare Source
Features

Partner Usage Reports: add new service to project (#302) (3d62fcf)

v0.58.0
Compare Source
Features

Partner Billing Units: add new service to project (#301) (d430463)

v0.57.0
Compare Source
Features

Catalog Management: re-gen service after recent API changes (#298) (f7f8acc)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 1.28.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Bug] Function coalesce if value is decimal or double return wrong value

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.2.6-release
What's Wrong?
create SQL :
CREATE TABLE test_ifnull (
id int(11) NULL COMMENT '',
t_decimal DECIMALV3(26, 9) NULL,
test_double double NULL
) ENGINE = OLAP
DUPLICATE KEY(id)
COMMENT 'test_table'
DISTRIBUTED BY HASH(id) BUCKETS 1
PROPERTIES (
"compression" = "ZSTD"
);
insert SQL:
insert into test_ifnull VALUES(1,11111.11111,2222.22222)
test SQL :
select id,t_decimal,test_double,coalesce(t_decimal,test_double) as co_dou,coalesce(test_double,t_decimal) as co_dei from test_ifnull

What You Expected?
return right value
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

thanks, we will fix it.

fixed.   https://github.com/apache/doris/pull/23691
This issue currently only occurs in version 1.2.6, but 2.0.0 is fine.


Update to latest cake

We have to run with skip verification. It is not nice. Please update.

@Cheesebaron let me see if the update breaks anything and I will update it.

I have something ready for a PR. Will wrap it up and create it.

Take a look at https://www.nuget.org/packages/Cake.AndroidAppManifest/1.1.0 and give me feedback @Cheesebaron


stored value in create a field method

Hi
stored value keys "true" or "false" typed in your document
this is not working
I've tried as, its working : (->stored('YES'))
$request = new OpenSearchServer\Field\Create(); $request->index('index_name')         ->name('titleStandard')         ->indexed(true)         ->analyzer('StandardAnalyzer')         ->stored('YES')         ->copyOf('title'); $response = $oss_api->submit($request)

Hi, we just made a fix by allowing both textual and boolean parameter (as we did for the indexed property). The main reason is that 'stored' should also support 'COMPRESSED' value.
We just updated the master and the 3.0 branch, can you check that ? If it will be fixed we will create a new tag.


Add setup.py

#43

I'd not explicitly test the installation as part of the "tests" but by running travis et.al. and installing acme-tiny there.

Should the version really be hard coded to 1.0? I plan on packing this up in an RPM, and it would be sad if there are multiple versions of the script that are version 1.0, given that there has not been any tagged release, yet. Have you considered using setuptools_scm?

I'd not explicitly test the installation as part of the "tests" but by running travis et.al. and installing acme-tiny there.

It should be part of the test suite in such way that it's also tested for if running outside Travis. I don't care how that's implemented; the most natural way for me was to simply make it a Python unit test. What's the disadvantage of this approach?

Should the version really be hard coded to 1.0? I plan on packing this up in an RPM, and it would be sad if there are multiple versions of the script that are version 1.0, given that there has not been any tagged release, yet.

Ultimately that's the maintainer's choice but it makes sense to me. The program seems to be pretty feature complete and I believe it should use Semantic Versioning.

Have you considered using setuptools_scm?

Nope, due to the lack of experience with that tool. It's probably a good idea to switch to it though!

Fair enough, I just have never seen that in any other Python project. (E.g take a look what pretty much any django/django related project does)

This now doesn't hardcode the version number anymore but uses the git revision/tag as a basis for the version number. Thanks to @rspeed for the improvement

After tagging a release, this command creates a wheel that works in both Python 2 and 3:
pip wheel --build-option --universal<EMAIL_ADDRESS>

Hello, we are planning to use this library in a project (https://github.com/OCA/server-tools/pull/347). If you can merge this and make it pip-installable we would thank you a lot :innocent:

I'll second the downstream request.  Currently waiting on this PR to add this software to our official package repository in Gentoo.

You know what, I'll simply submit the PyPI package myself. However, submission seems to be broken currently (HTTP 500), so I'll try again in a few days.

OK, it's now on PyPI. @diafygi contact me anytime if you're interested in taking over ownership of the package.

I can't find it. How's it called?

It's called acme-tiny: https://pypi.python.org/pypi/acme-tiny
PyPI's search function doesn't list it. I don't know why that is.

Maybe someone could file a bug report here: https://bitbucket.org/pypa/pypi/issues I don't have a Bitbucket account

It was already reported. Seems like pypi is being deprecated and this will never be fixed. However in http://warehouse.python.org it gets correctly listed.

Can I get a sitrep on this? @jonashaag I don't think squatting on the project name is a good idea unless you are the maintainer..

This isn't squatting, I simply upload any released version of acme-tiny without modification to PyPI. I'm happy to hand over control to a maintainer.

Btw everyone, since I'm not really watching this project anymore, whenever you need a new PyPI release, please just ping me real quick.


Fix, regression inside MetalRenderPrimitive

This fixes a crash in material_sandbox introduced in #3695.
Fixes #3706. I'll also cherry-pick this into rc/1.9.19.

Wups, sorry about that!!

Wups, sorry about that!!

All good, I missed it too :)


Implement UpdateEl for iterators

I noticed that UpdateEl is not implemented on iterators.
Would be great to add an implementation.
That saves me a annoying collect.
div![
  // Want:
  items.iter().map(render_item)
  // Required now:
  items.iter().map(render_item).collect::<Vec<_>>()
]

Agree.

I'm not sure if it's possible to differentiate the type of an Iterator. Getting conflicting requirements issues, where the compiler can't tell if we should add Els, Listeners, or text, due to all of them being iterators.

The only way how I can compile:
fn render_item(number: &i32) -> El<Msg> {
    div![ format!("{}", number) ]
}
fn view(_: &Model) -> El<Msg> {
    let items = vec![1,2,3];
    div![
      items.iter().map(render_item)
    ]
}

Is to implement UpdateEl for Map:
impl<Ms, T, U> UpdateEl<El<Ms>> for std::iter::Map<T, U>
    where
        T: Iterator,
        U: FnMut(T::Item)->El<Ms>
{
    fn update(self, el: &mut El<Ms>) {
        el.children.append(&mut self.collect());
    }
}

But I don't know how to make it more generic, so it's not very useful in the current form.

I've exhausted all my ideas.
@theduke Do you have an advice how to implement it? Otherwise I recommend to close it.

Let's leave open for now. I don't have a fix now, and suspect their may not be one... But this would be a syntax improvent. Maybe some macro magic we haven't thought of?

You motivated me to do some more investigation.
The most generic implementation for iterators I can come up with:
impl<Ms, I, U, F> UpdateEl<El<Ms>> for std::iter::Map<I, F>
    where
        I: Iterator,
        U: UpdateEl<El<Ms>>,
        F: FnMut(I::Item)->U
{
    fn update(self, el: &mut El<Ms>) {
        self.for_each(|item|item.update(el));
    }
}

It let us to use iter().map(f) when f maps to something that implements UpdateEl<El<Ms>>.
So it should cover the most used case when you want to iterate over a list of something from model and render its items.
(I tested it only with some examples so there can be some problems.)

I think that we need to wait for specialization:

RFC: https://github.com/rust-lang/rfcs/blob/master/text/1210-impl-specialization.md
Forum post + link to example: https://users.rust-lang.org/t/how-to-use-specialization-features/6023

It should resolve conflicting implementations between Iterator and structs which implement Iterator and also allow us to implement UpdateEl for ToString. However I don't know if it allow e.g. to differentiate associated types.

Maybe some macro magic we haven't thought of?

I don't know if some macro magic resolves it, but if you feel very adventurous, you can try TypeId or some Any magic with downcast.

Nice! That appears to work on my end for Els. Using map in this way is, I imagine, a common use-case; not having to explicitly use collect is nice.

Notes:

We can probably use the same "trick" like the one in styles! https://github.com/David-OConnor/seed/pull/273/files with traits as a workaround for unstable specialization.
I have type problems with .map when I was also using itertools. I don't know exact problem (yet).

TODO: Fix commented-out UpdateEl implementation for Filter - https://github.com/seed-rs/seed/blob/master/src/virtual_dom/update_el.rs#L147


hard to connect

hard to connect basic nodes,could BTCQ update more nodes please？

addnode=<IP_ADDRESS>:3333
addnode=<IP_ADDRESS>:3333
addnode=<IP_ADDRESS>:3333
addnode=<IP_ADDRESS>:3333
addnode=<IP_ADDRESS>:3333

In the configuration section, in the "NETWORK" section activate "port mapping using UPnP" you will have better connections with nodes and fast synchronization


Figure our if we need any "Terms of Use" and/or "Privacy Policy" for MTB Calnedar

If you look at current mtbcalendar.com they have this:
http://www.mtbcalendar.com/abouts/terms
In particular they state:

Neither we nor any third parties provide any warranty or guarantee as to the accuracy, timeliness, performance, completeness or suitability of the information and materials found or offered on this website for any particular purpose. You acknowledge that such information and materials may contain inaccuracies or errors and we expressly exclude liability for any such inaccuracies or errors to the fullest extent permitted by law.
Your use of any information or materials on this website is entirely at your own risk, for which we shall not be liable. It shall be your own responsibility to ensure that any products, services or information available through this website meet your specific requirements. In other words always contact the promoter or check their website before attending a race.

How to do this task
I wonder what if we don't post anything like this, should we worry from legal standpoint? E.g. find a layer in cycling network for consultation and google around.

PP Ver.2 (RCN-MTB-00.1)

@Restuta give me the link on TOS you like (remember?)

this is not the link I wanted to share (I can't find that one), but it's a good example of extensive policy + nice design for that page we can use as well https://coolors.co/legal


Webpack 5 incorrectly compiles CJS module marked

Bug report
What is the current behavior?
When loading marked (CJS) through a pnpm monorepo package with Webpack 5, it compiles incorrectly. marked compiles to:
/***/<EMAIL_ADDRESS>/*!******************************************************************************!*\
  !***<EMAIL_ADDRESS>***!
  \******************************************************************************/
/***/ ((module, __unused_webpack_exports, __webpack_require__) => {

"use strict";
module.exports = __webpack_require__.p + "static/media/marked.79f9f27bbd94805ba150.cjs";

/***/ }),

According to the maintainers of marked this looks like a bug in Webpack.
If the current behavior is a bug, please provide the steps to reproduce.
There's a minimal repo showcasing the bug here:
https://github.com/laustdeleuran/marked-2265-module-import-export-issue
And some deeper context in this issue in marked:
https://github.com/markedjs/marked/issues/2265#issuecomment-1451124186
What is the expected behavior?
marked gets compiled correctly.
Other relevant information:
webpack version: 5
Node.js version: 14 or 16
Operating System: MacOS
Additional tools: pnpm and create-react-app

Haven't dug too deep, but it looks like this might be related to #16680 or #16570

It looks like this might have something to do with webpack incorrectly compiling .cjs files in packages that are "type": "module"
In @laustdeleuran's repo, If I remove "type": "module"  and rename marked.cjs to marked.js everything works fine.
Detailed Steps to reproduce:

Open repo https://github.com/laustdeleuran/marked-2265-module-import-export-issue
Run pnpm i
Open File<EMAIL_ADDRESS>Edit line 24- "default": "./lib/marked.cjs"
+ "default": "./lib/marked.js"

Remove line 6- "type": "module",

Rename File-<EMAIL_ADDRESS>+<EMAIL_ADDRESS>

Run cd some-cra
Run pnpm start

Any update on this, @Webpack? Seems like we have a legit bug here with minimal repros, steps to reproduce and a potential path for resolution. Is there anything else we can do to move this forward? Thanks!

Hm, seems like this might actually be an issue in create-react-app: https://github.com/webpack/webpack/issues/16776

@laustdeleuran Can you reproduce this bug without CRA? Because we have tests for these cases

@alexander-akait Sorry for the noise. After some more investigation, I do think this is a create-react-app issue and not a webpack issue.
There's some more context in https://github.com/facebook/create-react-app/issues/12700, including a couple of PRs to solve this issue, as well as a couple of workaround.
Closing this issue.


isBusy/IsNotBusy?

Hello :)
it's not an issue just maybe a suggestion.
In BaseViewModel you have "isBusy" and "isNotBusy"....
wouldn't just one be enough?
cheers! :)

It is a nice thing to have so you don't need an inverse converter. Sometimes you want to only show things when not busy :)

yes I understand but why not to use one field? for example: bool isBusy
if(isBusy)
{
// do stuff when busy
}
else
{
// do stuff when not busy
}
that is simpler, right??

Well, that is what I do, but in XAML if you are showing an element when something is not  busy.
Here is an example:
https://github.com/jamesmontemagno/app-coffeecups/blob/2985698505fe35e874fd86ca1da746ce48fbf665/CoffeeCups.Shared/View/CoffeesPage.xaml#L33
If I didn't have this then you would need a value converter.


Some questions about multithreading

Can I load the model only once for multiple asynchronous image processing? Will using multithreading have any impact if possible?

Actually, I want to develop upper computer software for machine vision. This is my first attempt to use Rust instead of C++, and I am currently testing its feasibility

Sounds cool! You should carefully review the ort crate for more information! Similarly, you can use projects like candle and burn for model inference.

They have some strict requirements for me, a beginner like Rust. Can I use your code if I only need to run the model at software startup and transfer images when the camera triggers photography

thanks


interbase/firebird query-builder replace feature

it's mention on firebird docs (http://www.firebirdsql.org/file/documentation/reference_manuals/fblangref25-en/html/fblangref25-dml-update-or-insert.html)

Mentioned in firebird doesn't mean it is or should be supported in CI, especially if it is something firebird-specific.

ohh. ok.


[Question] Using MPC for parameter generation.

Are there plans to support distributed parameter generation? Or could I maybe modify the Powers of Tau library (https://github.com/ebfull/powersoftau) to generate parameters for the SW6 curve?

Hmm currently we don’t have an implementation of the powers of Tau, but it should (hopefully) be a mechanical change to translate @ebfull’s repo to use Zexe instead. I know that @kobigurk was also interested in this.

My desire is to adapt https://github.com/kobigurk/phase2-bn254/ to the Zexe curves.
It's originally based on Zcash's repo, which then Matter Labs took and added support for larger parameters by working in chunks. We use it for https://github.com/weijiekoh/perpetualpowersoftau, where we generate parameters for 2^28 constraints on BN254.
Our specific repo adds a workflow for doing phase2 (which was quite manual before) and support for interoperability with circom, so we can get out-of-the-box circom circuit on Zexe curves.
@brunoffranca if you want to help out, that would be awesome, we didn't get to tackling it yet.

@kobigurk actually we might be able to help each other. We at Nimiq are also interested in adapting the powers of tau protocol to the ZEXE curves. And your perpetual powers of tau repo also has a lot of functionality that it's useful to use. Can I email you so that we can talk more at length?

@kobigurk actually we might be able to help each other. We at Nimiq are also interested in adapting the powers of tau protocol to the ZEXE curves. And your perpetual powers of tau repo also has a lot of functionality that it's useful to us. Can I email you so that we can talk more at length?

Sure, feel free to reach out at<EMAIL_ADDRESS>:)


[T5A1][T13-B1] Hong Bangwu

Added new use cases, user stories and non-functional requirements to developerGuide

@bangwu93
Some comments added. Please ack & close the PR after reading comments.


[BUG] bond_options parameter not working

Harvester 0.3.0-rc1
According to the documentation the following configuration should be used to configure the bond mode to balance-rr :
install:
  mode: create
  networks:
    harvester-mgmt:       # The management bond name. This is mandatory.
      interfaces:
      - name: ens5
      default_route: true
      method: dhcp
      bond_options:
        mode: balance-rr

This has no effect.  In order to successfully set the bond interface mode the configuration has to modified by changing bond_options to bond_option:
      method: dhcp
      bond_option:
        mode: balance-rr

@weihanglo I updated the doc. Do you think bond_options (with s) makes more sense? If so, we can update the field in the code and correct the doc again in the future.

Since there is more than 1 bond option the plural form makes more sense. See also https://github.com/harvester/harvester/issues/1324

Agreed. And #1324 also seems reasonable to me. We'll add more options and make it plural. Thanks!

Hi @janeczku , for currently we use bond_option (without s) to configure, sorry it was typo in documentation.
(fixed in https://github.com/harvester/docs/pull/26)
The configure option will be update to bond_options (with s) when #1324 implemented, please use the issue to track its status.


Maintain helm chart

helm chart is outdated, is it being maintained or has a plan to maintain by etcd-operator team.
for example

etcd is outdated 3.1.8
etcd-operator is outdated 0.3.3
it still uses tpr not crd

No.
We don't have the bandwidth to maintain it at the moment.

@raoofm @hongchaodeng
i labeled this as future. it is a good thing to do, but not high priority for now.

@xiang90 thnx, was just checking as it was confusing to refer the chart and the examples here.

@xiang90 @hongchaodeng I assumed it is being maintained as it was referred here https://github.com/coreos/etcd-operator/blob/master/doc/user/op_guide.md#installation-via-helm without any warning of usage :)

In case you do not maintain the chart anymore, please put a message in the readme stating so to save people wasting time.
I've spent quite a few hours now getting it working. It breaks completely with kubectl v1.8 due to the use of TPRs and it also does not spawn a cluster, i.e. cluster.enabled=true does not work. This breaks my helm setup.

we never try to maintain it. someone sent us a pr to add it there.
we can:

make it clear that it is an externally maintained project
remove it from etcd-operator reference.

@Nitrogenycs
you should file a PR to where the chart is. The maintainer is probably more active there.

@Nitrogenycs you are right that chart is kind of broken or it isn't the right way of doing it. I had spent a day as well thinking that is an approved chart as I found it in etcd-operator guide and figuring out what am I missing as it was in the stable directory. Probably it should be moved to incubator again.

we never try to maintain it

@xiang90 it would be a great help to have it there. I even gave it a shot but the spec changed quite frequently. I think option 1 is good so that at least if any user wants to maintain it they see the link in the project.

we made it clear that we are not maintaining it in #1663.


opaec++: add class except for runtime exceptions

except encapsulates a location in the source code along with an
fpga_result flag. except overrides std::exception::what() to provide an
informative message about the exception.

Hmm, but wouldn't that just test the C++ runtime's ability to throw/catch?


Add support for loading extra operators

It would be nice if we could allow users to specify additional operators to be available with @cycle/time, so that users could create their own operators or so that we could ship some operators that are uncommonly used but can be added as needed.
One possible way we could do this is to create a makeTimeDriver function that takes an extras object:
import {makeTimeDriver} from `@cycle/time`;
import tween from `@cycle/time/extra/tween`;

const drivers = {
  Time: makeTimeDriver({extras: {tween}})
}

mockTimeSource could also take this extras object.
An extra would need to be a function that takes in the schedule object (used for adding to the schedule, like a listener), the currentTime() function, and possibly a function to get the time source (to allow building on top of core operators). It would need to return a function.
One thing I am unsure of is how to make this play nicely with TypeScript. Perhaps there is a way to tell TypeScript that there will be extra operators in the TimeSource from the provided extras object but I am unsure.
This is a prerequisite for #13.

I think the whole approach to scheduling and the virtual timing could be simplified. If xstream doesn't want it yet to introduce virtual time ok, then I would do the following:

Follow simplified singleton approach that cyclejs prefers (for example with adapt)
Create virtual setTimeout which could be imported from import { setTimeout } from '@cycle/time'
In normal situation this setTimeout would be a normal setTimeout
Create import { setVirtual } from '@cycle/time', when you call setVirtual(), it makes setTimeout to use virtual time.
Take all needed operators's code from xstream replace there setTimeout with setTimeout from '@cycle/time' (actually just add import { setTimeout } from '@cycle/time' on top of each file.

And use those operators in the manner you like import {delay} from '@cycle/time' or Time.delay.., maybe moved to extra, whatever.
When running test just call setVritual() and all time in @cycle/time operators would become virtual, this would also allow to use this virtual setTimeout singleton in tests for doing imperative timeout stuff.

@whitecolor one thing I can see being hard with this approach is how would we support RxJS and most.js?

Well as for Time.periodic(..) things may support just via adapt

@staltz what do you think about my comment?

@whitecolor your suggestion would probably work, but I'm not sure I like it.

Follow simplified singleton approach that cyclejs prefers (for example with adapt)

I disagree that Cycle prefers a global singleton approach. A clear design principle of Cycle is to use drivers to control side effects and global state. There are two parts of global state in Cycle that I know of. One is setAdapt and the other is the internal counter in isolate.
If we used global singleton for this, why not the same for the DOM driver?
It's also problematic for cycle-restart, which actually swaps out the time driver each time the application is restarted, so there is a period where two co-exist.
I'll grant that it would simplify things a bit and make it easier to build new operators and adapt old ones, but I really, really dislike global state.

There are two parts of global state in Cycle that I know of. One is setAdapt and the other is the internal counter in isolate.

isolate I believe is a different thing it is application logic level. While setAdapt is the framework level thing, it is a singleton in the context of application runtime environment.
I too didn't (don't?) like but. I as applications/service tend to be smaller in scope, it might be ok, global setAdapt would be a rub if you would need/want to use multiple stream libraries in one cycle application/environmental context. And it simplifies things kind of.
So agreeing with setAdapt means agreeing with using only one stream library in cycle application.

I'll grant that it would simplify things a bit and make it easier to build new operators and adapt old ones, but I really, really dislike global state.

This only one part of my suggestion global singletons like setVirtual or setTimeout, agreeing with it means agreeing that you only use one time scheduler in the application - in practice I believe it corresponds well to setAdapt approach and it is too framework level thing.
And this thing could introduce in xstream itself. Agreeing with it would mean that you use only xstream in you app and only one time scheduller real or virtual.
But if xstream doesn't implement scheduling itself yet, another part of my suggestion was no take the implementation of operators from cycle, and not implement them again with some new approach (as they are implemented now in cycle-time) - this would just be less work and potential bugs. And returning to global singleton problem - potentially this could be implemented without it, but it would like do setAdapt not a singleton thing - maybe not so needed.
I would really encourage @staltz to think about introducing virtual timing in xstream something like I described. Maybe then special virtual time driver would be not so needed.
For example, for Rxjs application I would prefer standard test/virtual scheduler approach.

This only one part of my suggestion global singletons like setVirtual or setTimeout, agreeing with it means agreeing that you only use one time scheduler in the application - in practice I believe it corresponds well to setAdapt approach and it is too framework level thing.

It's also problematic for cycle-restart, which actually swaps out the time driver each time the application is restarted, so there is a period where two co-exist.

There is not always a guarantee that there is one time scheduler in play at once.

this would just be less work and potential bugs

It would be more work, as the time based operators in @cycle/time exist and work. In fact, there are some bugs present in xstream's time based operators that are fixed in @cycle/time, like periodic getting out of sync in background tabs.
I'm not going to say that there are no bugs in the @cycle/time operators, but I have done a lot of testing and they work well. If you know of any defects, please report them.
Personally I am not a fan of the setAdapt approach. It makes some things easier but some things harder, and personally I was fine with using adapters explicitly.

For example, for Rxjs application I would prefer standard test/virtual scheduler approach.

Which means you either need to use global state or inject a scheduler, which has the same tradeoffs as here. Except if you want to change stream libraries you have to rewrite your entire test suite.
I really dislike global state. I think it causes surprising behaviour, makes it harder to compose applications, and setting a further precedent for it might mean we're more likely to use it in future.

Having a single virtual scheduler would also make it impossible to parallelize tests as well.

Having a single virtual scheduler would also make it impossible to parallelize tests as well.

Well, yes, but:

tests usually are run in different runtime context, for example, I run browser application test in the node, because good cycle tests would use mocked driver sources abstracted from real effects.

if you want for example to run tests in the browser in the same window you still can implement it in a different context:

for example, web/service worker.
implement using completely separate bundle, in this case, singleton modules will be global NOT for browser window but for module context (unless they explicitly not use window/global object).

So As I said this is a limitation that can be taken. And it seems to me quite reasonable to take if it really simplifies dev experience.

https://github.com/avajs/ava runs tests concurrently, which would break with a singleton scheduler.
You can also run tests concurrently with mocha.

And this not popular approach anyway.

If it speeds up a large test suite, why wouldn't you do it?
I'm somewhat tired of discussing this. I've thought about it quite a bit and I prefer the tradeoffs of dependency injection over globals.
I've listened to your opinion, and I have taken it into consideration. I want to attempt to implement extras as I originally conceived, and if that doesn't work I will explore alternative avenues.

I don't get why should parallel tests be broken by single virtual sceduller that just abstracts time? What is the difference for scheduler where schedule requests come from one tests or multiple tests? Of course, it may depend on scheduler implementation, but I see currently no problem with implementing the universal scheduler that should not be coupled to the particular cycle app or test running.

I'm late for the discussion, but I just read everything. xstream won't get its own virtual scheduler because @cycle/time is precisely xstream's virtual scheduler. It all comes from the realization that time is an effectful I/O (think of it as querying the system clock inside the computer). So import {delay} from 'xstream/extra/delay'; doesn't make the dependency to effectful I/O explicit. But sources.Time.delay(100) does.
On a slightly different subject, @Widdershin what if extras would work like this?
import whatever from 'my-extra-whatever-operator';

function main (sources) {
  // ...
  const b$ = a$.compose(whatever(sources.Time, 100 /* ms */))
  // ...
  return sinks;
}

That would solve TypeScript usage, since normal extra xstream operators also work like that, and respect TypeScript type inference.

I like that @staltz, let's do it!


refactor: use meta/ansible-roles.yml for determining superficial requirements

can be merged once all roles have been refactored as per linked issue
Closes https://github.com/JonasPammer/ansible-roles/issues/27

removed labels as per linked commits found above


pgsql: operator does not exist: character varying = integer

What steps will reproduce the problem?
Set emulatePrepare=true in db component config
Execute query:
Model::find()->where(['title' => 1])->one()

Where title column has type string
db=# select column_name, data_type from information_schema.columns where table_name = 'table';
  column_name   |     data_type
----------------+-------------------
...
 title          | character varying
...

What is the expected result?
SQL Query:
SELECT * FROM "table" WHERE "title"='1'

What do you get instead?
SQL Query:
SELECT * FROM "table" WHERE "title"=1

And exception
yii\db\Exception with message 'SQLSTATE[42883]: Undefined function: 7 ERROR:  operator does not exist: character varying = integer
LINE 1: SELECT * FROM "table" WHERE "title"=1
                                          ^
HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts.
The SQL being executed was: SELECT * FROM "table" WHERE "title"=1'

Every time explicitly indicate the type is inconvenient :(
What helped
https://github.com/yiisoft/yii2/blob/master/framework/db/Schema.php#L266
...
'integer' => \PDO::PARAM_STR,
...

In overridden schema class
When searching by a string value on a numeric column, there is no error
Additional info

Q
A

Yii version
2.0.14

PHP version
7.2.3

Operating system
Debian GNU/Linux 9

PostgreSQL
9.6.6

Hi, try using quote literals on value in clausule where
Example
$query = Model::find()->where(['title' => "1"])->one()

Indeed, types matter.


Task: Database Structures and code design

BASIC DATA STRUCTURES **to be improved

user database
id
firstname
lastname
date of birth
username
##password(hashed)

post database
id
date/time
userID
content
likes //possibility

//For following database a prediction

following
id
followerUserID
followingUserID
however this may become inefficient over time

Like(reaction) table

id
time stamp (date/time)
postID
userID


Update page. User flow name in twitter callback URL must be LOWER CASE even if mixed case on AZURE.

Please update this article to note that the user flow entered in the callback URL must be lower case even if it shows upper case in the azure portal.  Twitter authentication will not work if the user flow name is mixed case.

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

ID: 383fed5d-cba7-610d-8ce8-0329a45cfa2c
Version Independent ID: 52da97ee-890c-b696-d532-2a265f7390d0
Content: Set up sign-up and sign-in with a Twitter account - Azure AD B2C
Content Source: articles/active-directory-b2c/identity-provider-twitter.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@ChrisAWith Thank you for your feedback . We will engage the content team and update the thread.

Dear @ChrisAWith, thanks for the feedback. Not sure I really go you, The doc says:

Oh… LOL… terribly sorry I completely missed that and I read it like 15 times.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: Yoel @.>
Sent: Tuesday, August 31, 2021 5:18 AM
To: @.>
Cc: Chris @.>; @.>
Subject: Re: [MicrosoftDocs/azure-docs] Update page. User flow name in twitter callback URL must be LOWER CASE even if mixed case on AZURE. (#80325)
Dear @ChrisAWithhttps://github.com/ChrisAWith, thanks for the feedback. Not sure I really go you, The doc says:
[image]https://user-images.githubusercontent.com/20111866/131493443-e85f6351-398b-4816-892b-e2784ba70f77.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/80325#issuecomment-909142789, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVC5PHLMWSKNETOLPLI3KXTT7S277ANCNFSM5DAX2SMQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

#please-close


RFC: breaking changes for the coming major release version 6

This project is written in Groovy, but it seems that Kotlin is better option to code with Gradle API, so I want to rewrite this project with Kotlin 1.9.0 or later.
I hope that it can mostly keep binary compatibility, but it may not, so it's better to do it as a part of major release (spotbugs-gradle-plugin v6).
Let me know if you think it's nice to introduce more breaking changes, then I'll mark your issue with the milestone for v6.
When you submit PRs for v6, please use the beta branch as their base branch. Thank you!

Trying to understand what you are stating here.  Why does the code need to be groovy or kotlin instead of just java?  Why is the build tool forcing one?  I understand the gradle build itself being groovy or kotlin because that is what it is, but why does the code have to be as well?  I'm coming from maven and maven doesn't force me to use java at all.  So I'm having a hard time understanding why gradle would want users to do such a thing and make it even harder for contributors that don't know groovy well or worse haven't even heard of kotlin.  Ultimately both run on the jvm so to me its making it more complicated in general.
Where I'm coming from this is that the maven plugin is IMO a mess because it was written in groovy and I see zero reason that was done.  It should have been java and then I would not be the only person supporting it.  Most won't even attempt contribute due to groovy.  Nothing ever suggested it should have been that way IMO.  What I do assume is that original author wanted to try something new and this change here feels a lot like the same.
So that leads me to this.  We missed you for a long time here, I get it other life priorities, but for those of us that might be left around, is this going to be easy or hard?  For example, you have foo magic releasing this repo, but appears spotbugs does not have foo magic and was not up to date on how to even remotely release given it stated 'travis' all over and we still don't know how to release it.  I'm sure I can but its a learning curve if no one shows me how at least one time.  So I kind of see a repeat here.  Its great you are back but wondering what the cost this will be to supporters and contributors in general.

one thing to keep in mind https://youtrack.jetbrains.com/issue/KT-57669/Add-Java-20-to-JvmTarget
jdk 21 is GA next month.  We are getting behind and it does not help if gradle is not keeping up there either.  Kotlin supports 20 now but gradle not.

one thing to keep in mind https://youtrack.jetbrains.com/issue/KT-57669/Add-Java-20-to-JvmTarget
jdk 21 is GA next month. We are getting behind and it does not help if gradle is not keeping up there either. Kotlin supports 20 now but gradle not.

According to https://github.com/gradle/gradle/pull/25777 Gradle 8.3 should be ready to build on JDK 20.

Why does the code need to be groovy or kotlin instead of just java?

Gradle provides two options to code its buildscript: Groovy and Kotlin (XML for the case of Maven). And it is going to support Kotlin as the default language to code buildscript, refs https://github.com/gradle/build-tool-roadmap/issues/51
For plugins, we can use any JVM language including Java, Groovy, and Kotlin. But for Groovy and Kotlin, Gradle provides several features to shorten its implementation, like https://github.com/gradle/build-tool-roadmap/issues/50. This would be nice reason to use Groovy or Kotlin instead of Java.
And Groovy is relatively inactive and buggy for years (e.g. #523 and https://discuss.kotlinlang.org/t/bug-cannot-use-kotlin-enum-from-groovy/1521) so I believe that the Kotlin is the best language to choose.

Thanks will read over this.
Sent from my Verizon, Samsung Galaxy smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Kengo TODA @.>
Sent: Monday, August 21, 2023 7:00:41 PM
To: spotbugs/spotbugs-gradle-plugin @.>
Cc: Jeremy Landis @.>; Comment @.>
Subject: Re: [spotbugs/spotbugs-gradle-plugin] RFC: breaking changes for the coming major release version 6 (Issue #926)
Why does the code need to be groovy or kotlin instead of just java?
Gradle provides two options to code its buildscript: Groovy and Kotlin (XML for the case of Maven). And it is going to support Kotlin as the default language to code buildscript, refs gradle/build-tool-roadmap#51https://github.com/gradle/build-tool-roadmap/issues/51
For plugins, we can use any JVM language including Java, Groovy, and Kotlin. But for Groovy and Kotlin, Gradle provides several features to shorten its implementation, like gradle/build-tool-roadmap#50https://github.com/gradle/build-tool-roadmap/issues/50. This would be nice reason to use Groovy or Kotlin instead of Java.
And Groovy is relatively inactive and buggy for years (e.g. #523https://github.com/spotbugs/spotbugs-gradle-plugin/pull/523 and https://discuss.kotlinlang.org/t/bug-cannot-use-kotlin-enum-from-groovy/1521) so I believe that the Kotlin is the best language to choose.
—
Reply to this email directly, view it on GitHubhttps://github.com/spotbugs/spotbugs-gradle-plugin/issues/926#issuecomment-1687173186, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHODI4ZCJJO3ADJMKCVMQ3XWPSBTANCNFSM6AAAAAA27VOEE4.
You are receiving this because you commented.Message ID: @.***>

As part of all of the work for version 6, what will be the minimum Gradle version you will support?
In the plugins I've written, I've had to drop support for Gradle < 7.6 due to Jackson dependencies, but I don't think you will have that issue.  Still, I had previously had to drop support for Gradle < 7.0 due to Gradle API changes.
My employer's convention plugin auto-applies and configures your plugin, so we definitely want to track these changes.


Generated XLSX corrupted

Hi,
Great project, I love it!
I encountered the issue of being unable to open the generated XLSX with Excel 2019, the error stating file was corrupted. However, updating NPOI and SharpZipLib packages to their latest versions (NPOI version 2.4.1 and SharpZipLib version1.2.0) resolved the issue for me, so I thought it'd be good idea for you to update them in repo, too.
Cheers,
Antonia

Thanks for your feedback. I will do.


Tasklist doesn't read the correct status of UWP process

Trying to read the status of suspended UWP process, but tasklist sees it as running:

OS: Windows 10 Pro 20H2, build 19042.685

As discussed on Discord, this is because UWP suspension is not the same as Win32 suspension, and tasklist only checks the latter.
It should be updated to also check UWP suspension. This can be done by calling NtQuerySystemInformation to get PROCESS_EXTENDED_BASIC_INFORMATION, and then the status of UWP suspension will be in the IsFrozen bit-field.

As discussed on Discord, this is because UWP suspension is not the same as Win32 suspension, and tasklist only checks the latter.
It should be updated to also check UWP suspension. This can be done by calling NtQuerySystemInformation to get PROCESS_EXTENDED_BASIC_INFORMATION, and then the status of UWP suspension will be in the IsFrozen bit-field.

@sylveon Thanks for the solution 🙂. This should help @torquerrr

@AvriMSFT Can we reopen this? I think there are still actionable items on this, as per @sylveon's comment.


[Prism] Remove unused code

This PR addresses #31793 removing unused code later discovered not needed.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Tried to add :runners:prism:java:build to Java PreCommit, there is at least one flaky (2/2) test:
givenPrismLocation404_thenThrows
value of           : getMessage()
expected to contain: NotFoundException
but was            : java.io.IOException: Server returned HTTP response code: 500 for URL: https://example.com/i/dont/exist.zip
	at app//org.apache.beam.runners.prism.PrismLocatorTest.givenPrismLocation404_thenThrows(PrismLocatorTest.java:119)

In general, PrismLocatorTest relies on real URLs (https://github.com/apache/beam/releases/tag/v2.57.0/apache_beam-v2.57.0-prism-darwin-amd64.zip, https://example.com/i/dont/exist.zip) which are not a good unit test. Can we use mock there?

funny that "fix xxxx" phrase also leads to close a PR, not only an Issue


Investigate why expanding a tree in the explorer would make the extension host a noticeable number of seconds slower

Extracted from here.
Investigate why expanding a tree in the explorer would make the extension host a noticeable number of seconds slower. e.g. language features such as squiggles also suffer from the slow extension host...

Closing to clean up since this has been around for 2 years. If someone still sees this problem please ping :)


JSON format wrong API key handling

When using an invalid API key with the json format, the steam API won't return a json response and will simply display regular HTML, this commit prevents node.js from crashing and actually calls the callback with an error when this situation happens.

Awesome - thanks :) merged and published to npm as 0.2.5


How to Implement Authentication on GraphQL request

Hy, I'm new learner python,..
I'm try to use fastapi with GraphQL. Actually, I don't have any problem when build graphQL API until I try to implement Authentication.
I dont know, how to add authentication there, I'm try to looking up in graphene documentation (middleware, decorator, etc), issue, etc. But there, I just found how to implement graphene in Django Framework authentication.
So, I  have a few question.

How to authenticate GraphQL route ?
How to authenticate just on several mutation or Query?

Can you please use the issue template? Did you see this?
You will be better helped if you follow the issue template.

I have a starter graphql project with auth here https://github.com/matthewchung74/full-stack-fastapi-graphql
it's still (wip) but all the graphql auth stuff is in place. hope it helps.

hy @aalifadv  @matthewchung74, thank you. Let me try first.
I think this reposite will be good reference https://github.com/matthewchung74/full-stack-fastapi-graphql. include docker setup.

you might want to look at this https://github.com/tiangolo/fastapi/issues/2855 . i might create another starter using https://ariadnegraphql.org/ since I think subscriptions might be easier to build in. i couldn't get them working with graphene.

You can probably check out Strawberry, there are docs in FastAPI: https://fastapi.tiangolo.com/advanced/graphql/
This problem would probably relate to the GraphQL library directly, not so much to FastAPI.

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


is there a list of publicly hosted fcd / lcd / terrad combos? do all validators host them? (COL-5)

is there a list of publicly hosted fcd / lcd / terrad combos? do all validators host them? wondering if the one built into terra station is the only one available COL-5

In the Terra discord they posted this information
https://discord.com/channels/464241079042965516/897536156651106384/927845734785622016

Short answer is no, not all validators make them public.


Drawer not closing on click outside

Looking at the docs, I can't find a property that enables you to do this very commonly requested behavior.
am I missing something?

Do you mean tapToClose={true}?


Zen mode timer +hidden

I also have a suggestion for you.
During testing I left some chats hidden and forget about it.
I'm not sure whether chats should be hidden after WA refreshing or after computer reboot.
My idea is to add timer to Zen mode.
Like Sleep mode on mobile devices.
For example, I come to work and set time for 8 hours(or till 18:00) and after that time all chats become unhidden again.

This is sorta like #42 , someday will be implemented by an evolution of that. Closing for now.


Adds built-in graph states to flamingpy.utils

Context for changes
Adds five built-in graph states (star, GHZ, ring states, linear, and bell).

New features:

Adds five built-in graph states.

Bug fixes:

Fixed small bug in draw_EGraph that did not allow plotting EGraphs with non-integer coordinates.

Documentation changes:

Mention these functions in the run_graph_states.py tutorial.

Example usage and tests
from flamingpy.utils import graph_states

graph_states.star(10).draw()

graph_states.ghz(15).draw()

graph_states.ring(10).draw()

graph_states.linear(7).draw()

graph_states.bell().draw()

Performance results justifying changes

[X] Not Applicable

Workflow actions and tests
Added new unit tests (flamingpy/tests/unit/test_utils_graphstates.py) for testing the EGraphs of the states mentioned above.
Expected benefits and drawbacks
This will allow new users to familiarize themselves with flamingpy.
Expected benefits:

Users can define graph states from several state families.

Possible drawbacks:

Can't do much with them at the moment.

Related Github issues

[X] Not Applicable

Checklist and integration statements

[X] My Python and C++ codes follow this project's coding and commenting styles as indicated by existing files. Precisely, the changes conform to given black, docformatter and pylint configurations.

[X] I have performed a self-review of these changes, checked my code (including for codefactor compliance), and corrected misspellings to the best of my capacity. I have synced this branch with others as required.

[X] I have added context for corresponding changes in documentation and README.md as needed.

[X] I have added new workflow CI tests for corresponding changes, ensuring codecoverage is 95% or better, and these pass locally for me.

[X] I have updated CHANGELOG.md following the template. I recognize that the developers may revisit CHANGELOG.md and the versioning, and create a Special Release including my changes.

Can't merge because of code coverage :( but I've added unit tests to all new functions of this PR. I will try implementing more tests to increase code coverage.


Fix httpx deprecation warnings

Lots of these: https://github.com/simonw/datasette/actions/runs/8258529260/job/22590924898
  /opt/hostedtoolcache/Python/3.12.2/x64/lib/python3.12/site-packages/httpx/_client.py:1426:
  DeprecationWarning: The 'app' shortcut is now deprecated. Use the explicit style 'transport=ASGITransport(app=...)' instead.

 Set cookies directly on the client instance instead.

I can handle both of these in the datasette.client class, such that I don't have to change how dozens of tests that use datasette.client.get(..., cookie=...) work across many different plugins.


My Profile Page - Text Fields

Web Path
Home Page -> John Doe -> My Profile

Requirement
When you click on the text fields, the label should move up.
See: http://materializecss.com/forms.html

This issue was moved to Code4SocialGood/c4sg-web#56

Hi Jenny,
I am having following error.
[image: Inline image 1]
[image: Inline image 2]
Thanks,
Sabrina
On Thu, Mar 16, 2017 at 8:32 PM, Code for Social Good <
<EMAIL_ADDRESS>wrote:

Closed #461 https://github.com/Code4SocialGood/C4SG-Obsolete/issues/461.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Code4SocialGood/C4SG-Obsolete/issues/461#event-1003779588,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATkfjBHo4iDq7tjm6Rz9wO6YfAqKYabqks5rmeKqgaJpZM4MWJNe
.


refactor: Fixes to dependency tooling ansible playbooks

Updated home directory in playbooks to use use global home variables
Added logic in wmla120.py to only execute playbooks when eng mode is set
Added GEN_DEPS_PATH to wmla120.py
Added deps_local_path to ansible local vars
Added global path to generate dependencies path in lib.genesis

Updated Ansible tasks with new global dependencies from local vars
Updated proper task calling in install procedure for pre and post gather

Dev-wmla120 has since been sunset where changes. Resubmitting pull request under latest master branch:
https://github.com/open-power-ref-design-toolkit/power-up/pull/449
https://github.com/open-power-ref-design-toolkit/power-up/pull/448
https://github.com/open-power-ref-design-toolkit/power-up/pull/447


Split the train and val/test pipeline.

Dear reaserchers, I would like to express my gratitude for your excellent work on this project. I find it extremely interesting and valuable. However, I am wondering if you have ever tried to separate the finetuning and inference into the traditional training and testing. For example, first finetuning the model with a batch of data, and then using the finetuned model for inference. Because I found that the current method requires a long time for finetuning before generating each image (compared to the sampling time). I have tried it myself, but the results were not satisfactory. I would greatly appreciate any experience or advice you could share with me.
I am looking forward to your reply. Thank you in advance!
Best regards.

Hi, thanks for bringing up your concern! This method needs to take minutes to finetune the model for each input, which is indeed time-consuming.
According to your description, maybe you want to finetune the model for multiple subjects (eg, A, B, C ...) simultaneously, so that you can directly run the model to generate A, B or C at test time? If I understand correctly, I would say I haven't tried this solution myself but i also suppose the results will not be good, since I really do not see any works trying to fit multiple subjects through single-time finetuning with shared model weights, this process may not be easy to learn.
But there could be some alternatives for this issue:
(1) Train a feed-forward model that directly takes subject image as input condition at inference, this is much faster, like Paint-by-Example, AnyDoor, IP-adapter etc.
(2) Maybe you can train LoRA models, one LoRA for each subject. These Loras are plug-and-play which can be directly loaded to base model when needed. In other words, First you pretrain a set of LoRA models {A, B, C}. When you want to generate A, then load A for inference, no finetuning needed.

Thank you for your prompt response. In fact, in my task, I need to erase the same subject in a scene, such as erasing all people in different images. This is a challenging problem, because the content to fill after erasing depends on the specific context of the image. Your method is far ahead, and the effect is better than related works, because each image allows the model to learn the context of the content during generation, thus generating a more realistic filled background. Thank you again for your contributions to the community. If you have any relevant experience or good practices on this kind of erasing and inpainting model, I hope you can share with me. Thank you!

Hi, thanks for bringing up your concern! This method needs to take minutes to finetune the model for each input, which is indeed time-consuming.
According to your description, maybe you want to finetune the model for multiple subjects (eg, A, B, C ...) simultaneously, so that you can directly run the model to generate A, B or C at test time? If I understand correctly, I would say I haven't tried this solution myself but i also suppose the results will not be good, since I really do not see any works trying to fit multiple subjects through single-time finetuning with shared model weights, this process may not be easy to learn.
But there could be some alternatives for this issue: (1) Train a feed-forward model that directly takes subject image as input condition at inference, this is much faster, like Paint-by-Example, AnyDoor, IP-adapter etc. (2) Maybe you can train LoRA models, one LoRA for each subject. These Loras are plug-and-play which can be directly loaded to base model when needed. In other words, First you pretrain a set of LoRA models {A, B, C}. When you want to generate A, then load A for inference, no finetuning needed.

share with me

Hi, thanks for bringing up your concern! This method needs to take minutes to finetune the model for each input, which is indeed time-consuming.
According to your description, maybe you want to finetune the model for multiple subjects (eg, A, B, C ...) simultaneously, so that you can directly run the model to generate A, B or C at test time? If I understand correctly, I would say I haven't tried this solution myself but i also suppose the results will not be good, since I really do not see any works trying to fit multiple subjects through single-time finetuning with shared model weights, this process may not be easy to learn.
But there could be some alternatives for this issue: (1) Train a feed-forward model that directly takes subject image as input condition at inference, this is much faster, like Paint-by-Example, AnyDoor, IP-adapter etc. (2) Maybe you can train LoRA models, one LoRA for each subject. These Loras are plug-and-play which can be directly loaded to base model when needed. In other words, First you pretrain a set of LoRA models {A, B, C}. When you want to generate A, then load A for inference, no finetuning needed.

Thank you for your prompt response. In fact, in my task, I need to erase the same subject in a scene, such as erasing all people in different images. This is a challenging problem, because the content to fill after erasing depends on the specific context of the image. Your method is far ahead, and the effect is better than related works, because each image allows the model to learn the context of the content during generation, thus generating a more realistic filled background. Thank you again for your contributions to the community. If you have any relevant experience or good practices on this kind of erasing and inpainting model, I hope you can share it. Thank you!

Okay, I have read some works which might be related to your topic,
Inpaint Anything: Segment Anything Meets Image Inpainting [https://github.com/geekyutao/Inpaint-Anything]
Inst-Inpaint: Instructing to Remove Objects with Diffusion Models [http://instinpaint.abyildirim.com/]
Maybe you have read these works before, I am not an expert in this field but i hope it will be helpful for your research

Thank you for your patience in answering and helping, I wish you all the best.
I will close this issue and try the solution you provided.


Psr0Fixer, Psr4Fixer - ignore "new class" syntax

Adds support for PHP 7 anonymous classes.

Hi, thanks for PR !
please, could you retarget it against 2.2 base branch ?

Done

nope, base branch is still master

please fix issues reported by CIs

Fixed :)

I have updated your branch with 2.2 to have newest CircleCI config

@keradus It is a big surprise for me that you can write to my branch. But now I see 3 commits there and I do not know if it's okay.

@dg
On the right side of the PR there is a checkbox to (not) allow maintainers to make changes to you work.
Just a little FYI for the future ;)

I see it, great!

From a maintainer side, you even have a button "Update branch" (near "Merge PR") so I could merge base branch (here 2.2) to your branch directly from website.

But now I see 3 commits there and I do not know if it's okay.

that's perfectly fine, I will squash them during merging

please fix tests ;)

Thank you @dg.


compilation problem - Module not found

Hi ,
compilation  yarn   && yarn build:app returns errors.
basing on  https://github.com/appbaseio/dejavu/issues/279 I did git submodule update --recursive --remote
but still get errors:
ERROR in ./app/src/components/SearchPreview.js
Module not found: Error: Can't resolve '../batteries/components/BaseContainer' in '/Users/user/Downloads/dejavu/app/src/components'
 @ ./app/src/components/SearchPreview.js 6:0-66 20:166-179
 @ ./app/src/App.js
 @ ./app/src/index.js
 @ multi (webpack)-dev-server/client?http://localhost:1358 (webpack)/hot/dev-server.js ./app/src/index.js

ERROR in ./app/src/components/SearchPreview.js
Module not found: Error: Can't resolve '../batteries/components/SearchSandbox' in '/Users/user/Downloads/dejavu/app/src/components'
 @ ./app/src/components/SearchPreview.js 5:0-66 25:25-38
 @ ./app/src/App.js
 @ ./app/src/index.js
 @ multi (webpack)-dev-server/client?http://localhost:1358 (webpack)/hot/dev-server.js ./app/src/index.js

ERROR in ./app/src/components/SearchPreview.js
Module not found: Error: Can't resolve '../batteries/components/SearchSandbox/containers/Editor' in '/Users/user/Downloads/dejavu/app/src/components'
 @ ./app/src/components/SearchPreview.js 7:0-77 29:25-31
 @ ./app/src/App.js
 @ ./app/src/index.js
 @ multi (webpack)-dev-server/client?http://localhost:1358 (webpack)/hot/dev-server.js ./app/src/index.js

ERROR in ./app/src/reducers/index.js
Module not found: Error: Can't resolve '../batteries/modules/reducers' in '/Users/user/Downloads/dejavu/app/src/reducers'
 @ ./app/src/reducers/index.js 21:0-62 45:3-20
 @ ./app/src/store/index.js
 @ ./app/src/App.js
 @ ./app/src/index.js
 @ multi (webpack)-dev-server/client?http://localhost:1358 (webpack)/hot/dev-server.js ./app/src/index.js

Resolved basing on https://github.com/appbaseio/dejavu/blob/dev/CONTRIBUTING.md.
cd dejavu git submodule init git submodule sync git submodule update --recursive --remote


Error updating to Keychain

2024-12-17 11:07:37.273519 -0800	FirezoneNetworkExtensionmacOS	startTunnel(options:completionHandler:): Error! appleSecError(call: "SecItemUpdate", status: FirezoneKit.Keychain.SecStatus.status(FirezoneKit.KeychainStatus.wrPerm))

I don't think this will be an issue in 1.4.0 since this appears to happen when going from sysex -> appex during testing, but noting here to look into just in case.

Won't be a possible codepath in 1.4.0


Field Guard & Hide content in introspection for InputObject

Description of the feature
I really like the field guard and hidden content features. It is really useful to display additional data on the same schema.
Would it be possible to have those features on InputObject ?
For example, we can think of a request in which we want to filter by ID, but this property is only visible to the admins. On the requested object, this field will not appear for the average user, but it will still appear in the input object which is annoying.
Code example (if possible)
#[derive(InputObject)]
pub struct UserFilter {
    pub name: Option<String>,
    #[graphql(visible = "is_admin")]
    pub id: Option<String>,
}

InputObject already supports the visible property.

#[derive(SimpleObject)]
#[derive(InputObject)]
pub struct UserFilter {
    pub name: Option<String>,
    #[graphql(visible = "is_admin")]
    pub id: Option<String>,
}

pub fn is_admin(ctx: &Context<'_>) -> bool {
    ....
}

Great, I don't know why it didn't work when I tried with the visible property.
I'm leaving the issue open for the guard feature, I think it's still very useful.

Hi,
In addition to making the Guards available for the InputObject, it would be nice to be able to define guards when using SimpleObject and Inputobject simultaneously


fixed extractAssignment to also recognize commits with added/removed files

The current extractAssignment function only examines the commit.Modified field. This doesn't capture commits that only rename/add/remove files. This commit fixes that by also examining the commit.Added and commit.Removed fields in order to extract the labX name.

Coverage decreased (-0.1%) to 35.84% when pulling 2f841c97266559da1e08c2bc245ad5eb83ee13b9 on commit-added-removed-fields-hotfix into cde25ec7656c22a8611ffa42d8ea3f797889d3b3 on master.


Fix user_email from wrong command line number

According to the order prescribed in action.yml, destination_directory is option 4 and user_email is option 5

Thank you for the fix! Sorry it took me so long to respond, I was not watching this repo, so I did not know about this pr until now.


Logging on heroku

We might want to install a logging addon on heroku.
When Crafatar crashes or an error occurs and we don't immediately notice it, we can hardly see the cause because heroku stores only 1500 lines of logs.
It looks like both Logentries and Papertrail allow us to search in the logs and set up alerts.
I think I would prefer Logentries after comparing the free plans.

Switched from Logentries to Papertrail since Logentries was a limited demo


Field moisture 22C 10/22 high CH4 variability

Finally for now, number 2:

Raw:

Distribution:

Just...a lot of variability between cores, in particular AL 5 which was super high the first measurement and then super low the next time.

CH4 is variable. Not inclined to 'do' anything unless we know there's a problem.


How can I call a controller action from xxDialog.ts?

Hi;
I need help calling a controller action in Home controller, Edit action with parameter 1 using xxDialog.ts and open it in another tab.
Is there a way to do it?
Thank you

A controller action is called by calling its route. So simply create a  with the href set to the route of the controller action. You can submit parameters via URL (see general ASP.NET MVC examples). To ensure it's opened in a new tab add the target="_blank". But be warned: Popup filters might block this (new tab/new window is literally the same, in the end the client settings decide if the link is attempted to open in a new tab or a new window.
In case you refer to Dialog Tabs of Serenity, the approach will be different (simplified):
Create a tabbed dialog template (see the example included in serenity)
Create a custom models file (analog to the examples included in serenity) with its own request & response class for that special case.
On page load:
Create the request, call the service with the request -> assign the value(s) of the response to the
elements on the tab.


Update Eigen to 8719b9c5bc1a97e62d675c02495ed72dda6fae73 to fix compiling error

This PR tries to update Eigen to 8719b9c5bc1a97e62d675c02495ed72dda6fae73.
The reason to update Eigen is to fix the build error for custom ops (See error below).
The issue is that in Eigen there was a bug that uses if defined(EIGEN_ARCH_PPC) incorrectly (should be if EIGEN_ARCH_PPC).
I have created a PR in Eigen https://gitlab.com/libeigen/eigen/-/merge_requests/131 and the PR has already been merged.
This PR is a follow up in tensorflow repo to bump the Eigen to the latest version.
The error before this PR, when building a custom ops:
Execution platform: @local_config_platform//:host

Use --sandbox_debug to see verbose messages from the sandbox
In file included from tensorflow_io/core/kernels/io_optimization.cc:22:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/compiler/mlir/mlir_graph_optimization_pass.h:20:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/core/common_runtime/function_optimization_registry.h:23:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/core/common_runtime/device_set.h:23:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/core/common_runtime/device.h:35:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/core/framework/allocator.h:26:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/tensorflow/core/framework/numeric_types.h:20:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/third_party/eigen3/unsupported/Eigen/CXX11/Tensor:1:
In file included from bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/unsupported/Eigen/CXX11/Tensor:14:
bazel-out/darwin-fastbuild/bin/external/local_config_tf/include/unsupported/Eigen/CXX11/../../../Eigen/Core:334:10: fatal error: 'src/Core/arch/AltiVec/MatrixProduct.h' file not found
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
Target //tensorflow_io/core:optimization failed to build
INFO: Elapsed time: 4.778s, Critical Path: 4.54s

Signed-off-by: Yong Tang<EMAIL_ADDRESS>

The error is caused by https://stackoverflow.com/questions/60864626/cannot-fetch-eigen-with-bazel-406-not-acceptable
I think adding the mirror in https://storage.googleapis.com/mirror.tensorflow.org will solve the issue.

@mihaimaruseac The test failures are related to Bazel 's inability to download gitlab directly. In the past that was resolved through TensorFlow's own bazel mirror ( https://storage.googleapis.com/mirror.tensorflow.org ).
I don't have access to add items to this mirror. Wondering if you could help placing the eigen archive file into the mirror?

Sure. Was planning to do that on import, but it seems currently copybara is broken due to MLIR/LLVM so I'll have to import manually.


Specificity of the placeholder styles...

Is there a reason for the specificity of the placeholder styles? IE:
input::-webkit-input-placeholder,
textarea::-webkit-input-placeholder {
  color: #cacaca; }

input::-moz-placeholder,
textarea::-moz-placeholder {
  color: #cacaca; }

input:-ms-input-placeholder,
textarea:-ms-input-placeholder {
  color: #cacaca; }

input::placeholder,
textarea::placeholder {
  color: #cacaca; }

I have to use !important to override these styles because I do not use input before the style. Is there a reason I should be using input to style the placeholder? Why not use this:
::placeholder{
  color: #cacaca; }

If there is a reason, I'd love to know so I can improve my code. :)
Also, in my opinion, the color is too light and should be a touch darker.

Makes sense, if you want to write a PR we can pull it in.
As for the color, we have three standard shades of gray we use. As I recall, the $medium-gray was too dark, which is why we went with $light-gray.

No worries on the color, that's more of an opinion and one that is easy to override. I'll look at a PR for this but I'm swamped for the next few days so anyone is welcome to go ahead. :)


CI for Approval

Create approval-workflow.yml
name: Team Renaissance's approval workflow

Move to https://github.com/andyof/github-actions-for-ci/pull/6


How do you get access to i18n (t, changeLanguage) in an /page/api route ?

I need to have access to i18n inside an api route (serverless).
module.exports = new NextI18Next ({
  // ignoreRoutes: ['/data/', '/_next/', '/static/', '/public/', '/api/'],
  ignoreRoutes: [ '/data/', '/_next/', '/static/', '/public/' ],
  localePath: path.resolve ('./public/static/locales'),
  localeSubpaths,
  otherLanguages: [
    'de',
    'fr',
  ],
});

I have removed '/api' in the ignoreRoutes. The req object doesn't seem to be populated with i18n.
Am i missing something?

The middleware logic occurs on the appWithTranslation HOC, so your API routes won't work in this way. Please check the RFC below for future support.
https://github.com/vercel/next.js/discussions/17078


Add doc on verifying container images from the bundle manifest file

Issue #, if available:
https://github.com/aws/eks-anywhere-internal/issues/1257
Description of changes:
We are going to sign all the container images that we use using AWS CLI and notation CLI. Adding a doc on how to verify container images from the bundle manifest file.
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


website: Update toggle switch documentation & demos

Add toggle switch demos for the different options.
Update documentation, include adding hyperlinks to other mentioned components.

I noticed the toggle switch props only list label, labelPosition, and setFocus.  Why aren't icon, size showing?

I noticed the toggle switch props only list label, labelPosition, and setFocus. Why aren't icon, size showing?

Probably a bug in how we get props.


Update changelog for 2.4 release

The 2.x CHANGELOG is now empty, given that the 2.4 release was created
and all 2.4 changes have moved to that version's release notes file.
This also brings in the 2.4 release notes.
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

@kotwanikunal I decided to take a different approach and bring the 2.4 release notes into this branch and empty out the changelog. I think this makes more sense. We shouldn't duplicate data between release notes and the changelog. With this approach, the changelog just tracks unreleased changes.


milvus multiple vector fields  in the same collection

support milvus multiple vector fields  in the same collection  and  hybrid search

@qingcelei could you provide more details and a sample of what you would like to see for this capability?


Add missing 0.9.6 compatibility to Colors

Sorry to trouble you again! https://github.com/JuliaGraphics/ColorTypes.jl/pull/143#issuecomment-567206533

Thanks again, @fredrikekre! :heart:


Enhance documentation

Create a dedicated website with demo/gif etc instead of readme.
If we can provide a minimum HASS environment with demo component to see it live instead of video etc, would be better.

This have to be splited, it's the next main milestone that cannot be reduced as a simple issue.


Fix git tarball strategy for private repos

See https://github.com/travis-ci/travis-ci/issues/3923

Downloading a tar to run tests is a bad idea.

People often exclude their tests folder from the archives.

@GrahamCampbell this tar is from GitHub, and is just a shallow clone of the repo.

Normally when you download tars from github, things like tests are excluded. For example, try downloading laravel/framework, and you'll see what I mean.

https://github.com/laravel/framework/archive/5.1.tar.gz

@GrahamCampbell Interesting. Is this behavior documented somewhere on GitHub? Do keep in mind that this is an opt-in, so we expect the user to know what they are opting in to.

Not sure if's documented, but it's done when users mark files as excluded to save on download size of archives. These archives are exclusively intended for distribution for use, which is why people exclude their tests. They're not at all meant for this purpose.

So, maintainers of the repository chooses to exclude them from archive, right? I tend to think this is not a concern for the purpose of this PR.

The point I'm getting at is that you're misusing these archives if use them in this way.

What purposes are these archives supposed to serve? It is for convenient distribution of the source code, no? It seems to me that we can use them for testing if the maintainers choose to do so.

It is for convenient distribution of the source code, no?

For distribution for actual use, not to run it's own tests.

@GrahamCampbell I am afraid that we are not going to agree on this.
I believe that running test constitutes "actual use". Whether or not the archive includes tests is completely up to the maintainers (who decides what's in .gitconfigure); if the tests are in the archive, I believe that the archive is fit for running tests. When the maintainers of the repository decides to use the git tarball strategy to run the tests on Travis, they have decided that it is a good idea to do so. If you, a laravel/framework maintainer, don't want to use the tarball strategy to get the source code for your tests on Travis CI, you don't have to.

I believe that running test constitutes "actual use".

But it doesn't? That is literally the definition. The definition of that archive is to exlude the tests and anything unrelated to public use of the package.

It is your archive's definition. Not someone else's. As I said before, this is an opt-in strategy.

It's the definition everyone uses. All package managers use that definition for example.

Thanks for excluding me from "everyone".
Package managers most likely just need the binaries (some have "source" packages, yes), and their view is drastically different from CI platforms, which need to actually run tests. It stands to reason that the optimal "archives" for package managers and the CI platforms are different.
You are welcome to open an issue to discuss whether using tarball strategy to pull down source is a good idea, but that is not the purpose of this PR.

The documentation for git archive does not mention any specific use cases for excluding files or archiving in general. Although GitHub has exposed this feature as a method for distributing releases, the git protocol allows an archive to be requested for any commit, which suggests a general design that does not exclude development work flows.
@BanzaiMan This seems like a straightforward bug fix. Is there anything that can be done here to get this merged or closed?

Now obsolete!


Investigate a solution to push messages on critical issues to users

We should implement automation so that we can tag GitHub issues and run automatically post the message by linking the issue; this should happen through multiple mediums like Mattermost, RSS feed, a mailing list, etc.
We should figure out if we want to write a bot to run somewhere, use the GitHub bot, or different things.

Example: https://chat.charmhub.io/charmhub/pl/bosbio6y17drpnq73ubocxab1h


add a link to docs

in addition to the Git link and the Demo link we should add a link to a doc.

there is an "URL" filed which is meant to point to e.g. Docs pages but it looks like we don't display this link in our beautiful interface


Fix SnappyOutputStreamTest for big-endian platforms

Fix issue #119.

Looks good to me. Thanks for the fix!


Mask of the first frame without gui

Hello, thanks for your contribution, regarding provinding a mask without an interface and when the video has different camera angles, like consider a clip from a show where scene might change with different characters(considering person is the class of interest and multiple people appearing at the same time which I should track simaltaneously), how can I do that to provide the first frame mask of each scene and camera angle change?

eval.py supports using multiple input masks containing different objects (ref. YouTubeVOS evaluation does this). If the different camera angles contain substantially different content, you might be better off creating another evaluation session whenever the scene changes (e.g., treat them as separate videos). Otherwise the memory from a previous scene might creep in.
As a side note, our new project Cutie might work better.


Add "advanced" docs for generic classes

This is (partially) addressed in PR #101

I'm going to call this complete for now; we've got docs on every base class and method (that matter - I'm happy leaving out the really weird stuff like the base metaclass), some pretty class hierarchy diagrams, and it's all wrapped up in its own chapter. Maybe in future we can add some recipes for the generics but that can be another ticket for another release.


Re-enable wildcard read test

Problem
Per recent runs, we can nolonger reproduce #13750
Closes #13750
Change overview
Re-enable the test.
Testing
Update the test to see if the server crashes when doing wildcard read.

fast track: this adds a test


Limitations

Are there any limitations to joining the tadi web apart from having the slippy mindset?

I have a website with 2 files for the same page, would that work for the tadi web?


Update cardinality-estimation-sql-server.md

Just a simple typo: there was "can be can be" instead of "can be"

@pituach : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit 6bde4fa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/relational-databases/performance/cardinality-estimation-sql-server.md
:white_check_mark:Succeeded

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

Nice catch @pituach, thanks! Your commit will be merged and this article update is in flight. After the updated article is published, GitHub will list you as an article contributor.
Thanks again for this feedback, your suggestions help improve our documentation.
#sign-off

Docs Build status updates of commit 6bde4fa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/relational-databases/performance/cardinality-estimation-sql-server.md
:white_check_mark:Succeeded

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


Update monitor-vm-reference.md

Add description for the newly added dimension.

@ju-shim
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way 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


Design BinRead backtrace output

Here is a possible mockup:

Raw text minus ansi escape sequences:
 ╺━━━━━━━━━━━━━━━━━━━━┅ Backtrace ┅━━━━━━━━━━━━━━━━━━━━╸

 0: Reader reached the end of the file
     at src/entry.rs:100
  ┄───╮
   98 │  #[br(big)]
   99 │  #[br(calc = index_count)]
  100 ⎬╴ indices: Vec<u32>,
  ┄───╯

 1: While parsing field 'entries' in FileHeader
     at src/file_header.rs:6
  ┄───╮
    5 │  #[br(big, calc = entry_count + 1)]
    6 ⎬╴ entries: Vec<Entry>,
  ┄───╯

 2: While parsing field 'header' in MyFile
     at src/my_file.rs:15
  ┄───╮
      │  #[br(little)]
   12 │  struct MyFile {
      …
   15 ⎬╴     header: FileHeader,
  ┄───╯

 ╺━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╸

(Excuse github's mono font having horrible box drawing character support, any sane terminal won't have this issue)
And here is what condensed mode will look like:
 0: Reader reached the end of the file
     at src/entry.rs:100
 1: While parsing field 'entries' in FileHeader
     at src/file_header.rs:6
 2: While parsing field 'header' in MyFile
     at src/my_file.rs:15

Some things that came to mind immediately to think about were: (1) ensure field name is always in the failure message itself, (2) rendering where multiple fields are on one line in the source, (3) being able to see the contents of related variables.
Also FWIW my workflow is going to be either receiving a report from an end user (so wanting only compact output) or just clicking on the source of the failure to view it in the IDE so don’t expect to derive much personal benefit from flashy output.

I haven't used it myself, but it might be worth taking a look at miette which seems like it would do a bunch of heavy lifting with respect to actually formatting things.
example:


About the inference speed

Hi, dear authors!
This is a great job. May I ask can the model support real-time user interaction (>25fps)?
Thanks a lot.

Sorry for late reply.
The algorithm currently does not support real-time user interaction. Generation consumes about 0.4s, while editing consumes about 10s. After generation or editing, the rendering time is the same as EG3D.
Thanks for your attention.


Register version 0.3.2

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/118479
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.3.2 -m "<description of version>" 9cffcd79a142c3b05e05a7599df874eaddb5b3e3
git push origin v0.3.2


refactor: remove unneeded '/' for URL construction

This is a very minor follow up of https://github.com/openlayers/openlayers/pull/15792 that removes the unneeded second / when constructing the helper URL instance for appending the collections parameter on OGC API Layers. See also https://github.com/openlayers/openlayers/pull/15792#discussion_r1592822864

Thanks, @jansule !


Issue removing all options from a remote multi-select in v4.0.0

I have a remote multiple select box, enhanced by select2. Here's a snippet from my Rails view:
= select_tag 'settings[product_ids]'<EMAIL_ADDRESS>'id', 'name', @settings.product_ids), multiple: true, include_blank: true, data: { token_field: { source: catalog_products_url, options: { placeholder: 'Choose a product...' } }.to_json }

I select 3 items and save. It works.
I remove one item and save. 2 items remain.
I remove the last 2 items and save. Those same 2 items remain.
There seems to be a problem saving the state when all options have been removed.
Here's a screencast: http://i.imgur.com/XNHnflZ.gifv
It seems like a bug to me but I may be doing something wrong.

Duh. The browser doesn't submit the values of a multi-select if nothing is selected!
http://stackoverflow.com/questions/13137280/select-multiple-submit-issue-post-is-empty-if-not-selected