jetty-6 downgrade

Embedding jetty-9 caused conflicts with Tomcat 6 (and a few Tomcat 7 versions) as jetty-9 pulls in servlet 3.1 which gets dropped by tomcat.
Hence going back to jetty-6 based on servlet 2.5

NetflixOSS » karyon » karyon-pull-requests #35 SUCCESS
This pull request looks good

in the same time, Jetty-6, wait you were on Jetty-7 originally

Jetty-7 causing the same compatibility problem with new Jetty in the same area, servlet-api 2.5. I hope I won't be shot in here, could you maybe provide two set of binaries ( Jetty 7 and Jetty 9+ ) ?

We have v2.3.0 w/ jetty7
I think we can build one w/ jetty-9 commit sha if we need a release. For
now - jetty-6 seems a safer approach w.r.t existing tomcat 6 version
compatibility
On Tue, Apr 21, 2015 at 4:04 PM, maksimlikharev<EMAIL_ADDRESS>wrote:

Jetty-7 causing the same compatibility problem with new Jetty in the same
area, servlet-api 2.5. I hope I won't be shot in here, could you maybe
provide two set of binaries ( Jetty 7 and Jetty 9+ ) ?
—
Reply to this email directly or view it on GitHub
https://github.com/Netflix/karyon/pull/217#issuecomment-94967329.

--
Amit Joshi<EMAIL_ADDRESS>

Honestly it was a big sight of relief, when I saw Jetty 9 in the webadmin, that was causing so much pain ( that is Jetty 7 servlet-api 2.5 ), pretty much we would have to build our own version every time we switch to new Karyon version. I'm sure other people who are not using tomcat  feel the same.
Basically parallel versions would really, really help a lot, but I know it's more work for you.
Thank you.


Fixing bug : we were pushing the previous build

Had to run it twice to make it work: fixing
Need to move _site after the build, not before

Jenkins job istio.github.io/presubmit passed


chore: remove extraneous whitespace characters

This change removes extraneous whitespace (tab, space) characters from lines in //:zsh.completion.

CI failure for golangci-lint / lint appears to be caused by a timeout (re-running it will likely resolve it). If a maintainer is not around, I can rebase this commit and force push to my tree in order to trigger a new run (but this will re-run every job, which can/should be avoided if possible).

This is a generated file that is overwritten on every release. Please remove the extra whitespaces from the source instead.

sorry, haven't had much time to address this recently. i'll get back to this sometime in the next week or two.


Use plugin with its own private API

hello I wanted to ask if you can use this plugin to be able to call a own private server without using gmc or ios,use your own exsample api:
 $.ajax({
            type:"POST",
            url: "your_url",
            data:data,
            success: function (response){
                var myarray = $.parseJSON(response);

            }
        });

push.on('notification', function(myarray ) {
    console.log(myarray .message);
    console.log(myarray .title);
    console.log(myarray .count);
    console.log(myarray .sound);
    console.log(myarray .image);
    console.log(myarray .additionalData);
});

and you can price everything ?
thanks if you can not do this how do I fix ?
Thanks again

@eros2187 sorry, I'm not quite sure what you are asking. Do you want to use this plugin with a 3rd party push service like PushWoosh, Parse or Amazon SNS? Or are you looking to write your own service that totally bypasses Google's GCM and Apple's APNS?

hello Simon MacDonald yes!! Or are you looking to write your own service that totally bypasses Google's GCM and Apple's APNS
is this possible? as in creating an ajax call that returns me an array that allows me to be able to use the plugin ..
for me that would be great !!

@eros2187 it is possible to write your own push service. I've been part of an effort to do just that before on Android in order to guarentee delivery and level of service but my recommendation after participating in that project is, just don't do it. Trying to build your own GCM or APNS is just madness.
Building your own service you can make an ajax call to which in turn sends that data to your device through GCM or APNS is more that doable. Is that more in line with what you want to do?

already I do this but there are limits of messages both in ios that gcm .. unfortunately .. I wanted to use this plugin in order to have a push my service would be enough to create an ajax call on a server and to reclaim the json data poie use current plugins but I lack the knowledge suitable ..
I used this plugin https://github.com/katzer/cordova-plugin-local-notifications.git
creates local notifications problem is that if the app is closed push no more are displayed .. because the call ajax is not performed unlike most of this plugin that will always work even if the app is closed
so I'm stuck ..
I think it is a problem that many incontrono ..
how could I do for you? ce solution to this Simon MacDonald
Thanks for your time

@eros2187 what you are asking is possible but out of scope for this plugin. I don't recommend you trying to write your own push service to work around GCM or APNS.


Mix images or transfer style without using prompt

Hi.
Can you think of a way to infer between two images?
I thought subtracting the latency between two images, but that doesn't seem to work because it doesn't infer the error (noise prediction).

I have a use case I’m trying to think how to use SD for. Fashion. Take a
model in one photo and an article of clothing in another. Can we merge them
together in a nice way with a prompt and say inpainting guidance?
Seth
On Wed, Dec 28, 2022 at 7:49 PM Bazza @.***> wrote:

Hi.
Can you think of a way to infer between two images?
I thought subtracting the latency between two images, but that doesn't
seem to work because it doesn't infer the error (noise prediction).
—
Reply to this email directly, view it on GitHub
https://github.com/bes-dev/stable_diffusion.openvino/issues/112, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3Y67FK2ZBMCFPCVO373L73WPTUY5ANCNFSM6AAAAAATLWK224
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I think you can use an image VAE as a tokenizer (in addition to the prompt) but I don't really know how to calculate this error.
This would at least produce an image the reference style.

maybe the functionality we both want is to be able to do img2img but with
more than 1 input image in addition to the prompt and the model. I'm just a
hobbyist user I have a technical background but haven't written code in 15
years so can't really create that myself.
On Sat, Dec 31, 2022 at 8:26 AM Bazza @.***> wrote:

I think you can use an image VAE as a tokenizer (in addition to the
prompt) but I don't really know how to calculate this error.
This would at least produce an image the reference style.
—
Reply to this email directly, view it on GitHub
https://github.com/bes-dev/stable_diffusion.openvino/issues/112#issuecomment-1368222419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3Y67FPBBEZ7UIFAXAWMOG3WQA7B5ANCNFSM6AAAAAATLWK224
.
You are receiving this because you commented.Message ID:
@.***>

That already tries and creates a literal mix between the images.
I think if you find the space of latency it would be more appropriate, not so literal but a space where you find what the prompt indicates


feat: Added Job Board Starter

Hi @boris-becomes

@gomandev  The issue with the graphql pointers was on our end and it's fixed. You have to update the package: npm i -D gatsby-source-bcms@latest so it's "gatsby-source-bcms": "^3.0.4"

@gomandev The issue with the graphql pointers was on our end and it's fixed. You have to update the package: npm i -D gatsby-source-bcms@latest so it's "gatsby-source-bcms": "^3.0.4"

@boris-becomes Thank you, the fixes has been made.

@gomandev

Can we make the transition on dropdown and make the click outside close it?
Autocomplete missing on the Home page on the Locations search field - compare with the live version when you start typing any available location
Fix autocomplete for the Search field - autocomplete value is wrong ( try typing "Wa" for "Waitress" )
Handle error state on Contact page fields on empty submit
On the About Us page the content for the About Us section is missing
Update the domain link on the PageWrapper.tsx component

@gomandev
* Can we make the transition on dropdown and make the click outside close it?

* Autocomplete missing on the Home page on the Locations search field - compare with the live version when you start typing any available location

* Fix autocomplete for the Search field - autocomplete value is wrong ( try typing "Wa" for "Waitress" )

* Handle error state on Contact page fields on empty submit

* On the About Us page the content for the About Us section is missing

* Update the domain link on the PageWrapper.tsx component

@boris-becomes The fixes has been made. apologies for the back and forth.

@gomandev
I still don't see autocomplete on the "Search" field on the Home page. If I type "wa" I'm expecting to get the "itress". And for the "Locations" field It works only if I start typing the location name from the start, but if I type "Tulun" I'm also expecting to get the autocomplete for the rest of the "Tulungagung".
Everything else is great!

@gomandev
I still don't see autocomplete on the "Search" field on the Home page. If I type "wa" I'm expecting to get the "itress". And for the "Locations" field It works only if I start typing the location name from the start, but if I type "Tulun" I'm also expecting to get the autocomplete for the rest of the "Tulungagung".
Everything else is great!

@boris-becomes My bad, the issue has been resolved.


update the entered changes

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for contributing @ankushsoni96 - In the future be sure to check off the items in the PR template
Thanks, and happy coding!


⚠ Samsung has degraded performance

In e7fb4e0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2645 ms

Resolved: Samsung performance has improved in ba6cc7c.


Branch 136860272

Jenkins, test this please.

@tensorflow-jenkins test this please.

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please.


Show filename before path in "Open File" instead of path

Hi guys! Thank you for a great project!
When I use "Open file" and have lots of nesting folders, my path fill all available space so I can't see the exact name of the file.

For example in VSCode the filename stands before path so you can clearly see it.

I wondered if it makes sense as UI improvement. If yes - I would be glad to make a PR.

Thanks! I think we've agreed that this is how we want it. Just need to get around to actually doing it.

Related: #758

Fixed with https://github.com/onivim/oni2/pull/3251:


[DS-4126] master: Optimize docker builds

Port of https://github.com/DSpace/DSpace/pull/2307
Also resolved a bug.  The docker override of the solr web.xml for dspace/dspace:dspace-7_x-jdk8-test was out of sync with the web.xml for the branch.
Images have been built on DockerHub at terrywbrady/dspace.

verifyImages.sh has been run
 ===== dspace-7_x-jdk8 =====
DSpace version:  7.0-SNAPSHOT
           JRE:  Oracle Corporation version 1.8.0_181
  ---> http://localhost:8080/xmlui
  ---> http://localhost:8080/oai/request
  ---> http://localhost:8080/solr
  ---> http://localhost:8080/rest
 ===== dspace-7_x-jdk8-test =====
DSpace version:  7.0-SNAPSHOT
           JRE:  Oracle Corporation version 1.8.0_181
  ---> http://localhost:8080/xmlui
  ---> http://localhost:8080/oai/request
  ---> http://localhost:8080/solr
  ---> http://localhost:8080/rest

@PTrottier and @pnbecker , fyi.  This port exists.
Patrick, this also resolves the startup of /solr in the dspace:dspace-7_x-jdk8-test image.
A pre-build version is available at terrywbrady/dspace:dspace-7_x-jdk8-test

@PTrottier since that issue relates to Angular, those issues should be captured in a new issue.  This PR just affects the back end.

:+1:
I already reviewed the original PR and this looks similar and great to me.


cmd/bosun: Button to download config to local machine

Relies on browser's save dialog. Intent is user can save wherever they choose and upload to git or server or whatever.

 The following files have changed:
cmd/bosun/web/static.go

Looks okay to me once it passed the build.


Updates for 1.1 native SDK

Drafting a PR here while we are finishing up the underlying 1.1 native SDK updates. These are the core changes I believe we'll need to make for:

simplify the analytics listener property formatting since it is handled in the native SDKs now
add additionalAutoProperties at startup to capture the _applicationFramework and _applicationFrameworkVersion (screenshot example below)
update android example for target sdk 33 update (Compose 1.3.0)

This won't build yet since it uses code changes in the 1.1 SDK (config.additionalAutoProperties) that I have only tested locally by changing to local project references.


Update README to reflect changes introduced in PR #34

Update README to document the new meta feature introduced by https://github.com/akurtovic/InstaRaider/pull/34

I'll update README and submit another pull request.
Since the dependencies and installation instructions are for Debian, I suppose I should add a line detailing how to install gexiv2 on a Debian system but I don't have one installed.
I assume this would work but I'm not sure:
sudo apt-get install gexiv2
Is that right?

@izilly
sudo apt-get install gir1.2-gexiv2-0.10
worked for me


Changes related to the next Meilisearch release (v1.3.0)

Update Meilisearch version for latest release

bors merge


Template update

Closes #4
I've updated the pipeline template from 2.5 to 2.8.

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Added a few commits to address issues actually found in https://github.com/sanger-tol/ensemblgenedownload/pull/6 and a commit to enable the cleanup profile for our Tower tests and close #7

Unrelated query: What is this "Run pipeline with test data" CI that keeps running but does not complete? I saw this on treeval pipeline too.

Unrelated query: What is this "Run pipeline with test data" CI that keeps running but does not complete? I saw this on treeval pipeline too.

Unrelated query: What is this "Run pipeline with test data" CI that keeps running but does not complete? I saw this on treeval pipeline too.

This is one of the CI requirements we set through the pipeline website health page. Obviously we run it twice with the different version of nextflow, Github doesn't match them.  We can add the version to the requirement as well, probably just the latest version?

I think your comments abut merging the pipelines make sense. I already have the changes regarding --outdir lined up, but I look into that after.


The site address in document is expired, need update.

The site www.easyframework.cc is invalid. shoule be update to address of github.com.

将文档中旧的链接只想最新的网站地址。（github）


Yasnippet functions not working

yasnippet functions don't seem to be working for me even though I have it installed already.
* org-mode

** prose: insert prose                                                    :p:

#+BEGIN_PROSE
$0
#+END_PROSE

The above set up inserts $0 literally without the tab stops.

You need to have yas-minor-mode enabled in the buffer. Otherwise yankpad won't use it.

Ok, issue's been resolved. Thank you.


Add flag that prevents self issue assignment

Add a new doNotAssignToSelf flag that prevents issue/PR self-assignment.

Done in v1.7.0


Type checking rule for ---

We have the following type checking rule for e1 --- e2.
G, e1 --> G', t1    G, e2 --> G'', t2  Gf = G' + G''
---------------------------------------------------------------    where G' + G'' = G''
                G, e1 --- e2 --> Gf, t2

The definition of + on G is incorrect for this case:
{
  let x = 1;
  ---
  let y = 2;
};
x + y // error with `x` is unbound.

Figure out the right + and implement it.

As a starting point, it might be worth looking around at other papers that use a linear or affine type system in an imperative language. The usual formulation for the lambda-calculus does not have "result contexts," which are necessary to support imperative-style variable definitions. Those papers doubtless need to figure out, for ;-style sequential composition, how to both support "building up" a context with declarations while "dividing up" contexts when checking both sides of this composition.
(I don't have any specific paper recommendations right now, but I may be able to dig some up…)

This program doesn't work:
let bucket_idx = 0;
---
bucket_idx := 10

Just trying to get some clarity on the motivation behind the original typechecking rule. If G' + G'' = G'' then evaluating e1 --- e2 under G is essentially evaluating e2 under G, as if e1 was never there.
In addition, having Gf be the intersection of G' and G'' (as in the comment in TypeCheck.scala) wouldn't help the bucket_idx example.
As far as I can tell, wouldn't the simplest solution be evaluating e2 under G' and then allowing the resulting environment to be G''? Want to make sure I'm not misunderstanding the purpose of ---.

No, the original type checking rules is just plain wrong.
We want a few things:

— should “recover” the banks between e1 and e2. Concretely, this can be implemented as checking both of them in the original G to get G1 and G2.

anything in parallel sequence with e1 — e2 should not be able to use anything used by the merge of G1 and G2.

any declarations from G1 should be available in G2.

Something that’ll help is writing down a few examples and asking if they should or shouldn’t typecheck.
For an implementation, we’ll probably have to use endScope after e1 is checked to get any new bindings B and add them to the G. This means e2 is checked under G + B.

I was thinking a little bit about what makes this hard today and doodled a few notes:

I was trying to sketch the rules for any generic composition operator (think of ; as being some composition operator for any language, not necessarily our ; or ---).
In the upper-left, I wrote my impression of how these rules usually go in "normal" functional languages. In the lower-left, I showed how this has to change in a substructural type system. The "U with plus" operator is a disjoint union on contexts, so Gamma1 and Gamma2 have to be two "pieces" of the overall Gamma—and together cover all of Gamma.
In the upper-right, I sketched how the rule usually looks for checking composition of statements in an imperative language. I probably should have used "c" instead of "e" to remind us that we're dealing with commands. The idea is that the form of the judgment needs to change so you can build up a "result context," which might include newly-declared variables. So we have something like:
\empty |- let x = 5 -| { x }

That is, the rule for declarations needs to "add" something to the context. The composition rule I've shown in the upper-right uses Gamma' to check e2 because it might need to use declarations from e1.
The lower-right is meant to show why designing a substructural version of the imperative composition rule is hard. The question is, what do you use as the input context for checking e2? On the one hand, you want to use Gamma2 because that's how you "divide up" Gamma. On the other hand, you want to use Gamma', because that's how you pick up new declarations that appear in e1. Neither one really suffices!
It seems like what the rule needs is some of both: get "new" declarations from Gamma' and "old" declarations from Gamma2. But figuring out what's "new" and "old" in each context seems pretty messy…

@apurvakoti @sampsyo and I talked about this and we came to the decision that the source level typechecking should use addScope and endScope to extract the new bindings added in e1 and use add them to the type checking for e2.
The type checking rule for the core calculus will not thread through variables in this magical way. If a let binding needs to be used in both the branches, the code should write this:
let x = 1;
{
  x := 10;
  ---
  x := 11
}

The source level type checking rule can be viewed as doing this "hoisting" implicitly.

Ok, I can work on an implementation for this.


Allow "summing up" of values with the same key

Short example:

When two assets both have USB_C: 2 we want to be able to have a command or query that gives the sum as a result, in this case something like onyo get --sum ASSET_A ASSET_B should result in USB_C: 4.

This is IMO a strong requirement to have asset folders like discussed in #18. If there is no way to sum up multiple values, for example the number of CPUs of multiple servers bundled up in one cluster (which is then the asset folder), then the whole idea of asset folders does not help, but instead isolate information from each other.
This might also be interesting to find out how much storage a group folder contains or similar casees, were you do not care about in which file something is, or if it is used in 1 or 5 laptops, like the information currently provided by onyo get.

Possible problems (which stop us from just looping over a list and summing up values) are obviously:

How to handle this for values were it does not make sense to sum up numbers (e.g. dates, boolean)
Values with units (e.g. 8GB).
How to handle empty values (e.g. RAM: "", RAM: )
Values which are not possible to sum up at all (e.g. USB: 2 and USB: yes)

Implementation-wise, I can see this being used at least in 2 different situations:

onyo get --keys RAM --sum <group_a> to get the RAM of all devices in group_a together
onyo cat <asset_dir> to automatically sumerize all YAMLs in an asset folder as if it would be one asset file (hence the problem if e.g. two files contain a string defining an inventory number or other conflicts)

(I could imagine that for values that are not possible to sum up, we would just use lists to concatenate these into lists? ping #23)

As mentioned in https://github.com/psyinfra/onyo/issues/18#issuecomment-1571492258, I'm not sure we should bother really. This boils down to interprete the meaning of particular keys:

How to handle this for values were it does not make sense to sum up numbers (e.g. dates, boolean) ...

And that's not our business, I'd argue. It's the user who decides what keys to put in there and how to interprete them.
We can already provide a list of the values that would need summing via get. What's missing is better ways to format get's output (not including the asset path in this case): See #358
Summing them up can be done outside onyo.

The proper way to do this is to pipe onyo output into another tool. The user can choose from a variety of tools that will meet their needs (including addressing the type of corner cases that Tobias listed), any of which will be infinitely more capable than anything we could (reasonably) add to Onyo itself.
For example:
❱ onyo get --keys path type ram --path jsc | awk '{ sum += $3 } END { print sum }'
22528

I will close this, as I believe both Ben and I are both strongly on the same page. @TobiasKadelka if you disagree, please re-open and we can discuss further. :-)


CSI: Recover the workloads with stale COS Volume

Test Results: https://github.com/ambiknai/csi-plugin-operator/pull/1

lgtm


handle systems with blueshifted [OIII] 5007

fastspec /global/cfs/cdirs/desi/spectro/redux/guadalupe/healpix/main/bright/323/32332/redrock-main-bright-32332.fits \
  -o fastspec.fits --targetids<PHONE_NUMBER>9303775

Lots of development has helped address this issue, most recently #166. With the 2.5.0 version of the code, the example posted here (clearly an AGN) now looks pretty good @stephjuneau @Ragadeepika-Pucha @VFawcett. It's actually really cool to see how much better the modeling has become! Clearly, this object still has a clear [OIII] 5007 outflow, but that will need to be supported with either custom code (e.g., @Ragadeepika-Pucha's emfit) or a specialized FastSpecFit module.


Dynamically setting Embed source errors on Nexus 6P

[x] Issue does not already exist
[x] Issue observed on https://plyr.io

Expected behaviour
You should be able to setup Plyr using the markup for one kind of source, namely the HTML5 <video> element, and switch to another type of source, namely YouTube using plyr#source method without raising an error on plyr#media.
Actual behaviour
In Chrome 64 on the Nexus 6P plyr is throwing an error when you load YouTube using plyr#source looking at the logs, it seems to happen where the loadeddata, canplay, and canplaythrough events should be thrown. The embed is successful and the video is playable.
I also observed the same behavior on Plyr.io when switching to Vimeo. I have little experience working with Vimeo so I've concentrated on YouTube here.
This is an edge case, as this is a phone that could and should be updated to Android 8.0. My issue is mostly struggling to differentiate this ignorable error from another error that could be fatal. The way the error events are proxied, the only information that comes through is that an error type event was raised on plyr#container.
Environment

Browser: Chrome
Version:  64
Operating System: Android
Version: 6.0.1

Players affected:

[ ] HTML5 Video
[ ] HTML5 Audio
[x] YouTube
[x] Vimeo

Steps to reproduce

Setup plyr using HTML5 video with debug option enabled
use plyr#source to switch to a YouTube video
observe the error event in the log

Relevant links
Logs
Log on Nexus 6P when switching to YouTube
[Plyr] Config {enabled: true, debug: true, autoplay: false, loop: false, seekTime: 10, …}
plyr.js:1 [Plyr] Chrome 64
plyr.js:1 [Plyr] AJAX loading absolute SVG sprite
plyr.js:1 [Plyr] Injecting custom controls
plyr.js:1 [Plyr] Native fullscreen enabled
plyr.js:1 [Plyr] Caption track found; URI: https://cdn.plyr.io/static/demo/View_From_A_Blue_Moon_Trailer-HD.en.vtt
plyr.js:1 [Plyr] TextTracks supported
plyr.js:1 [Plyr] event: setup {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: ready {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: loadeddata {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: canplay {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: canplaythrough {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] Cancelled network requests
plyr.js:1 [Plyr] event: emptied {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: timeupdate {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: loadstart {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: progress {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: loadedmetadata {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
www-widgetapi.js:88 Unrecognized feature: 'autoplay'.
Xa @ www-widgetapi.js:88
W @ www-widgetapi.js:76
Y @ www-widgetapi.js:96
oe @ plyr.js:1
(anonymous) @ plyr.js:1
(anonymous) @ plyr.js:1
O.object.e.onYouTubeIframeAPIReady @ plyr.js:1
(anonymous) @ www-widgetapi.js:118
(anonymous) @ www-widgetapi.js:118
plyr.js:1 [Plyr] event: error {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
bTqVqk7FSmY:1 Error parsing header X-XSS-Protection: 1; mode=block; report=https://www.google.com/appserve/security-bugs/log/youtube: insecure reporting URL for secure page at character position 22. The default protections will be applied.
plyr.js:1 [Plyr] Native fullscreen enabled
plyr.js:1 [Plyr] event: volumechange {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: timeupdate {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: ready {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}

Xa @ www-widgetapi.js:88
W @ www-widgetapi.js:76
Y @ www-widgetapi.js:96
oe @ plyr.js:1
(anonymous) @ plyr.js:1
(anonymous) @ plyr.js:1
O.object.e.onYouTubeIframeAPIReady @ plyr.js:1
(anonymous) @ www-widgetapi.js:118
(anonymous) @ www-widgetapi.js:118
bTqVqk7FSmY:1 Error parsing header X-XSS-Protection: 1; mode=block; report=https://www.google.com/appserve/security-bugs/log/youtube: insecure reporting URL for secure page at character position 22. The default protections will be applied.
plyr.js:1 [Plyr] Native fullscreen enabled
plyr.js:1 [Plyr] event: volumechange {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: timeupdate {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}
plyr.js:1 [Plyr] event: ready {getOriginal: ƒ, getContainer: ƒ, getEmbed: ƒ, getMedia: ƒ, getType: ƒ, …}

Seems like a duplicate of #754. You should notice the actual error thrown is a big clue: Unrecognized feature: 'autoplay'
This error is thrown by youtube, not Plyr.
Phones rarely support autoplay with audio unless you mute it. This is being implemented for desktop browsers as well (Chrome 64 removed support for autoplay with audio unless you visited the domain and played media before). I would never use it, and haven't gotten this issue either.
Related links:
https://developers.google.com/web/updates/2016/07/autoplay
https://www.macrumors.com/2017/09/19/safari-11-released-macos-sierra-os-x-el-capitan/
https://venturebeat.com/2017/09/14/chrome-will-no-longer-autoplay-content-with-sound-in-january-2018/

@friday
I'm getting the Unrecognized feature: 'autoplay' as a warning from the YouTube code on Chrome on my Mac, as well but plyr is, correctly, not treating that as an error.
Also, switching to Vimeo embed also causes plyr to throw an error but there's no Unrecognized feature: 'autoplay' warning happening there.


Added [Feature]: a toString() method for enumerated values in Java

Closes https://github.com/accordproject/concerto/issues/100

Changes

 override the toString method inside the javavisitor enumDeclaration function
 Updated snapshots for required changes

Author Checklist

[ ] Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
[ ] Vital features and changes captured in unit and/or integration tests
[ ] Commits messages follow AP format
[ ] Extend the documentation, if necessary
[ ] Merging to main from fork:branchname

hey @mttrbrts @sanketshevkar can you please review and give some feedback

coverage: 98.739% (+0.001%) from 98.738%
when pulling 7c14d706387e371d22c36ccdb17fda08b4a2f1e7 on subhajit20:feature/javavisitor_enumDeclaration_toString
into a2463e0d7d0363653d9c195b80ab812d34a14692 on accordproject:main.


Task/GitHub actions

This just tracks hanami#611 to allow me to suggest improvements,
but with the knowledge of whether they work

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1fb1874). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   97.13%
=========================================
  Files             ?       37
  Lines             ?     1325
  Branches          ?        0
=========================================
  Hits              ?     1287
  Misses            ?       38
  Partials          ?        0

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1fb1874). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   97.13%
=========================================
  Files             ?       37
  Lines             ?     1325
  Branches          ?        0
=========================================
  Hits              ?     1287
  Misses            ?       38
  Partials          ?        0

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1fb1874). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.05%
=========================================
  Files             ?       35
  Lines             ?     1268
  Branches          ?        0
=========================================
  Hits              ?     1218
  Misses            ?       50
  Partials          ?        0

Continue to review full report at Codecov.

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1fb1874). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.05%
=========================================
  Files             ?       35
  Lines             ?     1268
  Branches          ?        0
=========================================
  Hits              ?     1218
  Misses            ?       50
  Partials          ?        0

Continue to review full report at Codecov.

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


NDViewer crash on Windows when clicking buttons

ndv version: main branch (locally cloned)
Python version: 3.10
Operating System: Windows 11
Installation options: pip install .[pyside,vispy]

Description
I'm currently launching the viewer with a local script after installing ndv from my fork. Whenever I click the "composite" or "3D/2D" buttons, the script just crashes with no thrown exception or error message.
Crash after clicking composite...
https://github.com/user-attachments/assets/031a2d52-fb32-4f2b-b789-19915e80a5df
... and after clicking 3D/2D.
https://github.com/user-attachments/assets/9a3b46a7-9429-4b1c-bb9b-2c604ee7d287
What I Did
test script on Windows:
import ndv
from qtpy.QtWidgets import QApplication
from ndv import NDViewer

images = ndv.data.cells3d()

app = QApplication([])
viewer = NDViewer(images)
viewer.show()
app.exec_()

Is it a bug I forgot to notice in the backlog?

Forgot to add: when running in debug mode and putting breakpoints in the incriminated code sections (a.k.a. the callbacks of those pushbuttons), it reaches the breakpoint with no problem and keeps going.

As mentioned on zulip, please try pyqt6 and pyside6<6.8 and see if you still get it

This may also be an issue with OpenGL on your system... to debug, it would be helpful to override sys.excepthook so that it hopefully doesn't crash out to console (but the fact that no trace back shows suggests this is an even lower level crash).  Will try to reproduce locally when I can

yeah, can confirm same behavior here, pyside6 not working well on windows.  (pyside6 has generally always been harder to support :/)  i'll leave this open until i have time to debug.  i almost always use pyqt6, so I need to also check on macos linux.  we do test against them all on ci. but that doesn't always catch the same stuff


ARCHITECTURES_2_TASK is limiting the tasks able to be deployed with HF DLC

Issue:
Customer is deploying HF ID: BAAI/bge-m3 with Task: sentence-similarity using:
model_builder = ModelBuilder(
    model=“BAAI/bge-m3”,
    schema_builder=SchemaBuilder(sample_input, sample_output),
    model_path=path, #local path where artifacts will be saved
    mode=Mode.LOCAL_CONTAINER,
    env_vars={
        "HF_TASK": "sentence-similarity"
    }
)

model_builder.deploy()

But getting the following from within the HF DLC:
ModelBuilder: DEBUG:     2024-02-13 00:05:57,567 [INFO ] W-BAAI__bge-m3-58-stdout
com.amazonaws.ml.mms.wlm.WorkerLifeCycle - ValueError: Task couldn't be inferenced from XLMRobertaModel.Inference Toolkit
can only inference tasks from architectures ending with ['TapasForQuestionAnswering', 'ForQuestionAnswering',
'ForTokenClassification', 'ForSequenceClassification', 'ForMultipleChoice', 'ForMaskedLM', 'ForCausalLM',
'ForConditionalGeneration', 'MTModel', 'EncoderDecoderModel', 'GPT2LMHeadModel', 'T5WithLMHeadModel'].Use env
`HF_TASK` to define your task.

Root Cause:
ARCHITECTURES_2_TASK mapping is too constraining and does not include all admissible pipeline tasks:
https://github.com/aws/sagemaker-huggingface-inference-toolkit/blob/80634b30703e8e9525db8b7128b05f713f42f9dc/src/sagemaker_huggingface_inference_toolkit/transformers_utils.py#L79
All we do is pass the task to get_pipeline from within the handler_service.py as far as I can see: https://github.com/aws/sagemaker-huggingface-inference-toolkit/blob/80634b30703e8e9525db8b7128b05f713f42f9dc/src/sagemaker_huggingface_inference_toolkit/handler_service.py#L115
Shouldn't the logic be the following?:
if "HF_TASK" provided -> set task to "HF_TASK"

else:
    fetch architecture from config.json
    if architecture is not in ARCHITECTURES_2_TASK -> throw error
    set task to mapped task

This way we allow for a best effort deployment / a.k.a pass through for the HF_TASK to get_pipeline(). If that fails, then so be it we should propagate the right error messaging to the customer stating that we tried the best and that xyz went wrong.

Hello,
task sentence-similarity is sentence-transformers specific which is not yet supported.

@philschmid  Is there a reason why we do not support all the tasks?

ack so will this eventually be supported via get_pipeline? Or will it only be limited to:
from sentence_transformers import SentenceTransformer
model = SentenceTransformer("BAAI/bge-m3")

We might for on that in the future but for now the simplest is to create a inference.py, see here https://www.philschmid.de/custom-inference-huggingface-sagemaker

@philschmid there are a number of multimodal tasks missing here (object detection, text to speech, audio classification etc.)

Where is "here"?

Is there a way to extend this map further for new use-cases? This will also save customer effort to create the inference.py for these formats.

What map?

@philschmid there are a number of multimodal tasks missing here (object detection, text to speech, audio classification etc.)

Where is "here"?
ARCHITECTURES_2_TASK https://github.com/aws/sagemaker-huggingface-inference-toolkit/blob/80634b30703e8e9525db8b7128b05f713f42f9dc/src/sagemaker_huggingface_inference_toolkit/transformers_utils.py#L79-L93

Is there a way to extend this map further for new use-cases? This will also save customer effort to create the inference.py for these formats.

What map?
ARCHITECTURES_2_TASK

The ARCHITECTURES_2_TASK Is not actively used since its not longer always possible to derive the task from a model since the same "architecture" can have multiple tasks. Thats why we always (every example, guide, docs etc.) set the TASK.

There are two things here

Tasks being restricted by old implementation - fix is out for that.
Some transformers don't have tasks that have been exposed via pipeline tag yet - the option you provided is one solution (another could them be exposed as pipeline tag on a release cycle). I think we would need a list of libraries that do need custom inference to prioritize such an implementation. If custom inference is needed, is this the long term plan for such tasks. This looks to be more as a problem of supporting tasks in extremely nascent stage of development.

Customer has two options here

Set the HF task via env variables
Provide inference.py for non task models like sentence similarity.


Swift.EncodingError.Context(codingPath: [], debugDescription: "Top-level Data encoded as string JSON fragment.", underlyingError: nil)

This was an accident when I converted the custom model using an archive to binary using "Defaults" for storage. This is a problem I recently used with Xcode 9.4.1, Swift 4.0. I didn't see this problem before using Xcode 9.2. The version of Defaults that I am using now is 0.0.8.
I also looked at this document "https://git.snooey.net/Mirrors/swift/blob/a8ba0772cd2f887c2b48549172d2c447140e06d8/stdlib/public/SDK/Foundation/JSONEncoder.swift"
I don't quite understand why this error caused my storage binary type to fail.
I hope you can help me.
Thanks

Hi, @niuxinhuai
I hope this is no longer an issue and that you were able to fix it. I'm sorry it took so long for the reply. If it's still something you'd like to discuss, please be so kind and re-open the issue and I will do my best to assist you.
Thank you 🙏


Wanted: option to disable wrapping in mc.{prev,next}Cursor

I was gonna make this suggestion after clearing up #69, but it seems that you might be a step ahead of me with the CursorContext:seekCursor refactor. :)
But basically, I was going to point out that it's not so easy to disable the wrapping from the mc.{prev,next}Cursor mappings, which are now made more complex by handling the enabled/disabled cursors. So it would be convenient to be able to pass something in so that the trues hard-coded in https://github.com/jake-stewart/multicursor.nvim/blob/7b2be6ed76c1a104fb0b1d82fb5f83f4e8c24198/lua/multicursor-nvim/examples.lua#L430 and https://github.com/jake-stewart/multicursor.nvim/blob/7b2be6ed76c1a104fb0b1d82fb5f83f4e8c24198/lua/multicursor-nvim/examples.lua#L436 could be overridden. In the end, I'm imagining being able to do something like
vim.keymap.set("n", "<right>", function() mc.nextCursor({ wrap = false }) end)
vim.keymap.set("n", "<left>", function() mc.prevCursor({ wrap = false }) end)

Existing mappings like vim.keymap.set("n", "<right>", mc.nextCursor) could remain backwards compatible by defaulting the opts to wrap = true.

@ajvondrak I think I'd rather have wrap as a regular param, instead of an options dict.
So you would do mc.nextCursor(false) to disable wrapping. if the wrap parameter is nil, then it can default to true.

sorted

Works for me. Thanks!


TEAMNADO-4245 => Prod Stable

Codecov Report
Base: 89.54% // Head: 89.54% // No change to project coverage :thumbsup:

Coverage data is based on head (4875f07) compared to base (39ee416).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##           prod-stable     #119   +/-   ##
============================================
  Coverage        89.54%   89.54%
============================================
  Files               42       42
  Lines              660      660
  Branches           122      122
============================================
  Hits               591      591
  Misses              69       69

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Multiple version documentation

We should provide the documentation of all the versions that are released.
There should be UI to switch between the different versions of the documentation and proper management of the documentation versions. There should also be 2 special tags: latest and stable to allow fast access to the documentation of both development and stable release respectively.
Note: This is not necessary until before release of version 2.0.0

I'd say latest and stable instead of dev and latest

I'd say latest and stable instead of dev and latest

Unless I'm understanding this wrong

I'd say latest and stable instead of dev and latest

Edited :+1:

Might be worth it making a separate docs repo for this, could also possibly do something with different deployed environments if we wanted to get real fancy

Might be worth it making a separate docs repo for this, could also possibly do something with different deployed environments if we wanted to get real fancy

Might be useful. This also means we could move the docs from https://github.io/hikari-py/hikari into https://github.io/hikari-py. But yea, good idea :+1:

Little update on this, it might take a while.
I have noticed a lot of issues with the current website and will be remaking it from the ground up. Small prs might come in once in a while, but don't expect anything soon.

Blocked by #808


Provide introspection on hyper::Error

I have an application in which I want to try to set up a server on a specific port, but if that port is taken (i.e., an AddrInUse error is raised), it should fall back to any port (i.e., port 0). Unfortunately, since hyper's error type is now opaque (as of https://github.com/hyperium/hyper/commit/5d3c472228d40b57e47ea26004b3710cfdd451f3), I can no longer easily see if that was the cause of an error raised by Server::try_bind.
While I could construct a TcpListener myself and then pass that listener's Incoming to Server::builder, that's an unfortunate amount of extra boilerplate. Perhaps it would make sense to expose an as_io_error(&self) -> Option<&io::Error> on hyper::Error?

I saw mention of this in IRC, it seems you can't downcast the cause because Error::cause() doesn't return a Error + 'static, right?
What if hyper::Error were to get a cause_static (sorry, naming is hard) method, kind of like std::io::Error::get_ref, to allow downcasting?

That'd be nice, but is there a particular reason you don't want to expose the io::Error through an explicit as_io_error method? As someone pointed out on IRC when I asked about it, cause() isn't really intended to be used as a way to figure out if a particular error occurred. Part of this is because causes form a chain, and you don't technically know how deep you have to go. And more importantly, the error nesting could change between versions.

I'm not especially against some form of as_io_error, I was just thinking that people may wish to access the errors they created themselves, like from a Payload or Service, and suggested something that might solve all cases.

Oh, yeah, I think having both is probably a good idea! I think my concern about cause nesting would make it pretty difficult to extract user errors from behind a hyper::Error nonetheless, but at least it would be possible!


Update validator and tests

NOT READY FOR MERGE

It looks like a lot of the AaC structure is becoming hard coded into the validator.  How can we avoid that and use the AaC specification of itself to drive the validation?  The test would be to change the AaC spec and successfully validate itself without having to change anything in the validator.


[📚] Your documentation feedback title (CHANGE ME)

Please describe your documentation issue or suggested improvement in detail here and provide links to any pre-existing/relevant documentation and screenshots if necessary:

Hi @Duc56789
When opening an issue, please try to provide all of the information indicated in the template.
Closing this issue, since no information has been provided.
Thank you.


Update standard 8x8 font to include all CP-1252 characters

This is in 1.04 RC3 so marking as 'Done'


Range Slider filtering issue... When using single vs double figures...

I have a couple of Range Sliders on my site connected to filter.js for filtering.
One is an Age Range filter (range from 2 to 19)
The other a distance Range Slider (in Miles 1 to 20)
Both sliders work if the range is between 0 and 10 or 10 and above...
However you cannot filter for say values 5 and 15, basically no results are shown.
#completelyPuzzled?
Any help greatly appreciated...
Please feel free to take a look...
http://everest.usvolleyballinstitute.com/index.php?option=com_usg_programs&Itemid=102&view=programs&section=all&zip=07733

Nobody???

I recently had a similar issue. Make sure the values in your range are integers and not strings. My json integer values were coming back as strings, silly. Easy fix though.


Use Docker Action Releaser to handle releases

Inspired by yesterdays tomfoolery, I made a thing.
It solves a couple of problems for us:

No more forgetting to move tags around when adding stuff to the action.yml which can break stuff.
Users will be able to lock down to a specific version. Currently, every version uses the 1 Docker image so the running code is always the latest release no matter what version of TagBot you have.

I'm gonna let this sit for a bit, I have to write some tests in the other repo. But it is working in the general case already.

this is still happening eventually, but i want to change how the action works first. the current push-to-master approach is ugly.

Ok, I'm finally just closing this. If I ever come back to this idea it'll just specifically for this repo without trying to do something general for others to use.


CRV_HYBRID has the brake error bits always set high

Todo: Find their replacements, if available.

Codecov Report

Merging #4 into honda-bosch-longitudinal will increase coverage by 0.30%.
The diff coverage is 100.00%.

@@                     Coverage Diff                      @@
##           honda-bosch-longitudinal       #4      +/-   ##
============================================================
+ Coverage                     64.51%   64.81%   +0.30%
============================================================
  Files                           183      183
  Lines                         14637    14637
============================================================
+ Hits                           9443     9487      +44
+ Misses                         5194     5150      -44

Flag
Coverage Δ

#test_car_models
59.25% <100.00%> (+1.24%)
:arrow_up:

#unit_tests
60.56% <100.00%> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

selfdrive/car/honda/carstate.py
98.42% <100.00%> (ø)

selfdrive/car/honda/hondacan.py
67.85% <0.00%> (-5.36%)
:arrow_down:

selfdrive/car/ford/radar_interface.py
96.22% <0.00%> (-1.89%)
:arrow_down:

selfdrive/car/honda/carcontroller.py
84.12% <0.00%> (-1.59%)
:arrow_down:

selfdrive/car/hyundai/carstate.py
85.98% <0.00%> (+0.93%)
:arrow_up:

selfdrive/athena/athenad.py
59.43% <0.00%> (+1.20%)
:arrow_up:

selfdrive/controls/controlsd.py
83.12% <0.00%> (+1.24%)
:arrow_up:

selfdrive/controls/radard.py
99.32% <0.00%> (+1.34%)
:arrow_up:

selfdrive/controls/lib/alertmanager.py
100.00% <0.00%> (+4.08%)
:arrow_up:

selfdrive/car/hyundai/carcontroller.py
76.19% <0.00%> (+4.76%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


LEAN Timezone issue

Hello
I have some discussion with Alexandre Catarino in QuantConnect community about LEAN timezone related issue. Alexandre suggests to open a ticket and here it is. Beside the (potential) problem, I am also making a few suggestions as well.
A. LEAN chart display time issue
An example will be easier to describe the situation.
Take the default "SPY" security data (2013/10/7) comes standard with LEAN download.
We know "SPY" started at 9:30am New York time that day (and every trading day indeed). Furthermore, assume the user operates at Shanghai/UTC+8 timezone, so the timezone setting of the host OS where LEAN is running is UTC+8.
We could express 9:30am NY time interchangeably in other timezones and they all mean the same instant of time.
9:30am NY = 13:30 UTC = 21:30 Shanghai/UTC+8
So if we put that "SPY" data into LEAN and plot a chart of price using one of the three timezones we specified above, the chart should either starts at 9:30am,  13:30 or 21:30 depends on which one of the three timezones is specified in the algorithm.  Chart starts at any other time/timezone combinations would considered as incorrect.
Currently various combinations of settings in LEAN will lead to chart starting time others than what described above. I have performed an analysis and tabulate the result in the following community question, please refer that for details.
Discussion Forum: Different behavior for custom data plot using QC in cloud and desktop
B. cut the tie of chart display with timezone setting of the host OS where LEAN is running.  In other word, change host OS timezone should not have any effect on chart display.
C. inconsistency between LEAN and QuantConnect. Same algorithm running on LEAN and QuantConnect with have different chart display.
D. need documentation on where time zone could be set inside LEAN ( e.g. SetTimeZone, AddData, SecurityExhange, etc) and how they affect the algorithm and chart display. Also suggestion on default setting or best practice of how they should be set.
E. legend/label in chart showing the current timezone. Current it is hard to tell especially if you have multiple streams of data from different timezone. Making users very confusing and hard to collaborate across multiple timezone. This is especially true if you clone/receive an algorithm created by somebody else
F. allow user to specify chart display timezone they preferred (Or is that already a setting serving that purpose?)

Thanks for reporting @gvickh, this is addressed in PR #751. All charting is now stored in UTC making it ignore the OS timezone. QuantConnect/Embedded charts are forced into -5 Hours offset from UTC this is not configurable at this time but we may consider it given user demand.


fix: enable import of esm build

Node's ESM import requires file extensions for all imports: https://nodejs.org/api/esm.html#mandatory-file-extensions. Without the .js extensions, a runtime error is thrown when cashscript is imported by esm projects.
This also updates electrum-cash, where some interfaces have changed (and if another package tries to share configuration with ElectrumNetworkProvider, they'll get type errors, as this one is more restrictive).
Finally, the explicit exporting of the network classes (in src/index.ts) solves an issue that esm dependents encounter: The requested module 'cashscript' does not provide an export named 'ElectrumNetworkProvider'.
(And if you're interested in migrating to ESM, hopefully this gets you a lot closer!)

Ah, I also added 'import/extensions': ['error', { js: 'always' }], in you eslint config; that would make it easier to notice when a rogue import would break esm imports at runtime.

A project using this PR: https://gitlab.com/GeneralProtocols/anyhedge/library/-/merge_requests/193


undefined GetOpenAPIDefinitions in make process

I cloned metrics-server in GOPATH/github.com/kubernetes-incubator/ and
ran 'make' but it failed. The error is "pkg/apiserver/config.go:45:72: undefined: 'github.com/kubernetes-incubator/metrics-server/pkg/generated/openapi'.GetOpenAPIDefinitions".
I check the help of openapi-gen.go.
It will output the package in './' if $GOPATH is not set.
So I add a GOPATH variable and it works.

How are you building outside a gopath?

I ran make inside gopath.


Batch right-retracts in accumulate nodes

I have identified what looks like a quick and easy performance optimization for Clara accumulator node retraction logic.
As implemented, I think that clara.rules.engine/right-retracts are not retracting facts in a batch/collection-based style.  This is in contrast to how the c.r.e/right-activate does perform accumulation logic in batches.
I'd like to have stronger benchmarks for some future changes, but I think for this one I can make a pretty brute-force example to demonstrate the speed to be gained.
  There are some other notable spots where time is spent now when I look at the before vs after profiling results, but I'd like to address each concern separately where appropriate.
I have ran these several times repeatedly in the same process as well to ensure there was some "JVM warmup" done.  The results seem consistent.
The PR here is very simple.  It just ensures that we do retraction logic in batch for clara.rules.engine.AccumulateNode and c.r.e.AccumulateWithJoinNode.

Love it! Merged and thanks!
As for future benchmarks, it might be worth adding them to the clara-benchmark project at https://github.com/rbrush/clara-benchmark. I had put that together when doing some initial profiling and comparison to Drools, but it makes sense to add more to it as we find things. It uses the criterium library to warm up the JVM, which I've found to be consistent and reliable.

I messed around with clara-benchmark a while back, so I am a little familiar with it.  I will look into getting something set up there that demonstrates this accumulator path.  I have 1 or 2 other accumulator optimizations ideas I'm working through, so it would be good to have those laid out there.
+1 to criterium.  It seems to be a commonly used lib for this sort of thing.


Unusual camelCase repository name breaks vgo

The x/vgo prototype enforces strict casing on imports. spf13/viper imports spf13/jwalterweatherman, which is not the same casing as the github repo. This breaks importing the library.
Arguably vgo should not be enforcing case. However, even you, the author, don't use the correct case, which suggests to me maybe in this case the github repo should just be renamed to what people actually import it as. WDYT?

I'm not the author, but I have an opinion: The camel-case makes a good test case for the x/vgo and simliar. So I vote to keep as is.

My question would be: what do we gain from this test case? AFAICT, this repository is the only popular repository out there that violates the casing convention. Allowing mismatched case packages is problematic, because it will only work with vgo in specific situations ("is the repo on github", basically).
The generic module hosting mecanisms described for vgo have as a requirement "must be able to use a dumb static content server," which means there must be no arbitrary transformations between the imported name and the name of the thing hosting the modules.
In theory, vgo could add a bunch of special-casing for just Github... But otoh, this really is only a problem during the onboarding phase of vgo. Post vgo adoption, any such mismatching would be found instantly, and corrected before the module is even published. So, my question is: is it worth the pain to support this special case, if it results in a less consistent vgo UX, when github trivially allows renaming of repositories?

AFAICT, this repository is the only popular repository out there that violates the casing convention.

That I doubt. But as I said, I'm not the owner of this repo. If it somehow stops me from using vgo when that time comes, then I will maybe revisit this problem. But vgo is an early prototype, there are lots of "case issues" and other stuff to be ironed out.

@bep seems to be fixed in a recent version of vgo. @danderson would you consider trying go get -u golang.org/x/vgo and if it works consider closing this issue?

The "canonical Go import" part of this module has already been github.com/spf13/jwalterweatherman -- I say this because that is how it always has been in Hugo, the origin of this module. This is also the module name used in go.mod, which works fine for most.
So, the only correct thing to do is to rename this repository to all lowercase. I will get @spf13 's attention about this and get it done.


Is there a way to quickly notify a Core 0 FreeRTOS task from ulp? (IDFGH-9362)

Is your feature request related to a problem?
Hi @ESP-Marius  Could you please gaveme some help？
Since release 5.0 of esp idf, driver/rtc_cntl.h has been moved to esp_private/rtc_cntl.h.
Does ESP IDF 5.0 offer an alternative (of rtc_isr_register()) in order to raise an ISR from ulp coprocessor and handle it in the main C program on CP0?
I tried ETS_RTC_CORE_INTR_SOURCE, but the esp_intr_alloc() call failed.
Is there a way to quickly notify a Core 0 FreeRTOS task from ulp?
I couldn’t find any definition of “RTC_CNTL_RTC_SW_CPU_INT” in the esp-idf-v5.0 folder.
Do you have an example of triggering this interrupt from ulp assembly code ?
Do you have an example of handling this interrupt in core 0 C code ?
For memory I use ESP IDF V5 framework, on a ESP32-U4WDH chip.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Hi,
On ESP32 it seems like ULP_CP_INT would be usable for this purpose. As of now there is no ULP API wrapper for this interrupt, so you still need to use rtc_isr_register.
But I agree that we should provide a wrapper for this, as ideally #include "esp_private/rtc_ctrl.h" shouldn't be used in user code.
How about changing this issue to be a generic feature request for this feature?


Smartcard logon

A big PR that enables NLA smartcard logon with both real smartcards and smartcard emulation.
Note: there's still a concern for automatically computing containerName under non windows hosts. So smartcard logon works perfectly with smartcard emulation as we know which containerName it gives under windows (Private Key 00). Anyway there's still some work to do the computation with real smartcard under other platforms (as under Window NCryptEnumKeys gives the name directly), as we may do some smartcard requests:

with PIV cards mix the card GUID and the container id
with GIDS cards use the key name (and in this case providing the reader name is also required)

@hardening did some refactoring, pushed to https://github.com/akallabeth/FreeRDP/tree/smartcardlogon

@hardening remaining is the thing why you did have a runtime switch for SmartcardEmulation as that is a compile time only option

@hardening ok, do have some compatibility failure. Connecting with current release/master versions is fine, but with this pr connections fail:
/tmp/clang/bin/xfreerdp /v:192.168.xx.yy /u:demo2 /d:rdtest /p:xxx
[11:09:33:087] [733291:733292] [INFO][com.freerdp.core] - freerdp_connect:freerdp_set_last_error_ex resetting error state
[11:09:33:087] [733291:733292] [INFO][com.freerdp.client.common.cmdline] - loading channelEx rdpdr
[11:09:33:087] [733291:733292] [INFO][com.freerdp.client.common.cmdline] - loading channelEx rdpsnd
[11:09:33:087] [733291:733292] [INFO][com.freerdp.client.common.cmdline] - loading channelEx cliprdr
[11:09:33:087] [733291:733292] [INFO][com.freerdp.client.common.cmdline] - loading channelEx drdynvc
[11:09:33:089] [733291:733292] [INFO][com.freerdp.client.x11] - Property 263 does not exist
[11:09:34:493] [733291:733292] [INFO][com.freerdp.primitives] - primitives autodetect, using optimized
[11:09:34:512] [733291:733292] [INFO][com.freerdp.core] - freerdp_tcp_is_hostname_resolvable:freerdp_set_last_error_ex resetting error state
[11:09:34:512] [733291:733292] [INFO][com.freerdp.core] - freerdp_tcp_default_connect:freerdp_set_last_error_ex resetting error state
[11:09:34:519] [733291:733292] [INFO][com.freerdp.core.nego] - RDP_NEG_RSP::flags = { [0x1f] EXTENDED_CLIENT_DATA_SUPPORTED|DYNVC_GFX_PROTOCOL_SUPPORTED|RDP_NEGRSP_RESERVED|RESTRICTED_ADMIN_MODE_SUPPORTED|REDIRECTED_AUTHENTICATION_MODE_SUPPORTED }
[11:09:34:531] [733291:733292] [WARN][com.freerdp.crypto] - Certificate verification failure 'self signed certificate (18)' at stack position 0
[11:09:34:531] [733291:733292] [WARN][com.freerdp.crypto] - CN = RD3.rdtest.local
[11:09:34:532] [733291:733292] [INFO][com.freerdp.crypto] - known_hosts=1
[11:09:34:664] [733291:733292] [ERROR][com.winpr.sspi.Kerberos] - error while getting credentials
[11:09:34:665] [733291:733292] [ERROR][com.winpr.sspi.Kerberos] - Kerberos credentials not found and could not be acquired
[11:09:34:665] [733291:733292] [WARN][com.winpr.negotiate] - No Kerberos credentials. Retry with NTLM
[11:09:34:665] [733291:733292] [WARN][com.winpr.sspi] - InitializeSecurityContextA status SEC_E_NO_CREDENTIALS [0x8009030E]
[11:09:34:665] [733291:733292] [WARN][com.freerdp.core.nla] -  InitializeSecurityContext failure status SEC_E_NO_CREDENTIALS [0x8009030E]
[11:09:43:677] [733291:733292] [ERROR][com.freerdp.core.connection] - Timeout waiting for activation
[11:09:43:677] [733291:733292] [ERROR][com.freerdp.core] - rdp_client_connect:freerdp_set_last_error_ex ERRCONNECT_ACTIVATION_TIMEOUT [0x0002001C]
[11:09:43:680] [733291:733291] [WARN][com.freerdp.core.nla] - DeleteSecurityContext status SEC_E_SECPKG_NOT_FOUND [0x80090305]

@hardening the WITH_GSSAPI check does not work, as only winpr/config.h.in has a #cmakedefine WITH_GSSAPI

@hardening pushed a few fixes for some warnings/wrong header locations, but I´ve found some leftovers (specifically cred.h) which need to be fixed.

@hardening you did overwrite the fixes I had already pushed ;(

Hum wait for a moment, because your last changes have over-ridden some of my changes

Ok I did the changes. It remains the concerns with Unicode vs Ascii calls under windows, but we can address that in further PR. I will need it anyway as my final target needs to have it working under win32.

@freerdp-bot test

This is much needed and much appreciated.  Anyone wanna post documentation on how to properly do this?
Seems like here would be a good place:
https://github.com/FreeRDP/FreeRDP/wiki/smartcard-logon


Video section looks odd

The Video section on the info page looks a  bit odd.

Hey, I would like to fix this issue. Can you please assign this to me?

Yes @arpita221b  go ahead and fix it!


Receiving Notifications when locked screen?

Is it posible to somehow receive, and/or display notification in any way when the device's screen is locked (and inactive)?
I get notifications when app is opened, in background and even when closed on my iOS device but it doesn't seem to register anything when closed and screen is locked (and inactive).
Anyone any thoughts on this, couldn't really find anything in OneSignals docs on this.

Nevermind, found a way to do it. Just haven't played around with the settings enough I guess.

@almadsen how did u solve ethis?


Implemented toggle button(slider) for dark mode/light mode functionality

Issue Title: Enhancing User Experience with Dark/Light Mode Toggle Button
resolve issue: #45
Type of change ☑
feature
What sort of change have you made:
Made a new feature of Dark/light mode functionality

added css for the toggle slider(button) in index.css.
added a toggle button in the App.jsx.

What sort of change have you made:

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

Checklist: ☑

[X] My code follows the guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
Describe how have you verified the changes made.
I had run the website on my local machine Twice and their is no sign of warnings or errors.

@ChaithanyaKrishna28  I've reviewed your changes & here is my feedback:

You should stick to the tech stack of the project, we are using TailwindCSS for styling purpose and hence there should not be custom external css  as you added in the index.css and Menu.css file, add the corresponding TailwindCSS classes and remove the style rules from index.css and menu.css

This dark mode is not looking good, Make it appealing for the users, it must be soothing to the eyes, for example the tools like brush, thickness are having white color on silvery background which is obviously not a good styling practice you should have make it dark, same goes with color palatees, you should have added some background to separate them from the background.

Make these changes and commit them, I'll then review again.

Again, Best of luck.

@ChaithanyaKrishna28  I've reviewed your changes & here is my feedback:

You should stick to the tech stack of the project, we are using TailwindCSS for styling purpose and hence there should not be custom external css  as you added in the index.css and Menu.css file, add the corresponding TailwindCSS classes and remove the style rules from index.css and menu.css

This dark mode is not looking good, Make it appealing for the users, it must be soothing to the eyes, for example the tools like brush, thickness are having white color on silvery background which is obviously not a good styling practice you should have make it dark, same goes with color palatees, you should have added some background to separate them from the background.

Make these changes and commit them, I'll then review again.

Again, Best of luck.

Thank you for the detailed feedback! I'll make sure to address each point in my next commit. I'll remove the custom external CSS and integrate TailwindCSS classes as suggested. I'll also work on improving the dark mode for better user experience, ensuring it's soothing to the eyes. Your encouragement for ongoing open-source contribution is noted, and I'll continue to contribute to the project beyond this PR. I'll make these changes and commit them for further review.


fix(configuration/#1826): initial value of editor.insertSpaces deviates from default

Fixes #1826. The default value itself was fixed in #1580

We really ought to generate the initial configuration file from the defaults, to avoid issues like this.

Thanks for catching and fixing this, @glennsl !


fix #1986

This needs tests before we can merge it.

I'd need #2011 merged first, the test doesn't pass yet without

Please read CONTRIBUTING.md; that is not an appropriate test for a feature like this.

Ok, from what I understand it should be a wpt test, so I'll search which to enable, thanks

For adding a test, I added Element.matches
if fails differently than before (it was failing on a :invalid) selector
Failed in "In-document Element.matches: Attribute presence selector, matching title attribute, case insensitivity (with no refNodes): #attr-presence [*|TiTlE]":
The string "#attr-presence [*|TiTlE]", is not a valid CSS selector

SyntaxError: The string "#attr-presence [*|TiTlE]", is not a valid CSS selector
 ...

Yes, a web platform test in to-upstream would be good.

I added a stupid test, but well I think I should remove it, Timothy already covers #1986 case in https://github.com/tmpvar/jsdom/blob/master/test/web-platform-tests/to-upstream/svg/element-svg.html#L19

I mean, we need a test that fails without this change, and passes with it. If Timothy's test covers it already, then this change isn't needed, it sounds like, and we should close this pull request?

on master branch I get the issue only for fragments, like #1986 described:
~/dev/jsdom$ node
> var {JSDOM} = require('.')
> var frag = JSDOM.fragment(`<svg></svg>`);
> frag.querySelector("svg")
null
> var frag = JSDOM.fragment(`<div><svg><foreignObject/></svg></div>`);
> var div=frag.firstChild
> div.querySelector('svg')
null
> var {window: {document}}=new JSDOM('
<svg><foreignObject/></svg>
')
> var div = document.querySelector('div')
> div.querySelector('svg')
SVGSVGElement {}

The SVG PR adds those new SVG classes instead of HTMLUnknownElement.
I don't understand well, but this PR only fixes it for fragments, so I'll need more classic unit tests

No, please don't do that. If we can't figure out how this is exposed to web code, we have a bigger bug.

using jsdom ^11.1.0
var frag1 = JSDOM.fragment(`<svg><foreignObject/></svg>`);
var frag2 = JSDOM.fragment(`<div><svg><foreignObject/></svg></div>`);
frag1.querySelector('svg') // null
frag1.querySelector('*') // HTMLUnknownElement

frag2.querySelector('svg') // null
frag.querySelector('div').querySelector('svg') // HTMLUnknownElement

I managed to reproduce the issue in a web platform test as follows:
<!DOCTYPE html>
<meta charset="utf-8">
<title>querySelector on template fragments with SVG elements</title>

<template id="template1">

</template>
<template id="template2"><svg></svg></template>
<template id="template3">
<svg></svg>
</template>

I will push this to this branch so we can merge.


Clarify README how Token is used and tracked by Article

I'm having trouble figuring out how my domain object should implement the "token" concept. The current README shows Article->getToken() as a method, but it isn't clear to me from the documentation how that token is used by the workflow and its nodes.
Does Article->getToken() return a unique identifier for the Node the Article is currently at? Would it return a value like article.draft or article.deleted ?
If it is the current workflow state, in which example class would you set the token via Article->setToken($currentNodeToken)?

This is exactly the idea: the token value is the name of the node the instance is currently on.
The setToken() method should be called in every subscriber.


Include Loved flag

Hi!
This is a feature proposal: can you include "loved" flag in the export?
Thanks in advance!
Tom

The parameter to set is extended https://www.last.fm/api/show/user.getRecentTracks
Set that on the request and modify the response parsing to add a 'loved' column


Remove duplicated article about memory map

The article was already updated on Developer Portal when we migrated the content.
We keep the alias to older article, so that redirects from blog.espressif.com works.
The content could be deleted.
PTAL @marekmatej

Looks good! Thanks for the fix


IF else condition

Operating system: ✍ TODO
Last upstream commit (run git log --author="Robert Lord" | head -n 1): ✍ TODO
Browser version(s): ✍ TODO
Ruby version (run ruby -v):  ✍ TODO

✍We are trying to provide a if else condition based on the parameter that we have passed in config.rb now i have to add a page based on this value please do guide me how shall i proceed

Hi! You should be able to do this with erb — maybe ask in the forum?
https://spectrum.chat/slate


Color temperature for lights

Hi, its possible to implement the color temperature in the light class accessory?
Thanks in advance!

There's no temperature option exactly.
Both the hue and saturation are set when you change either a Colour or a Temperature in the Home app.
So you can just use hueX.

Thanks for the quick response! if i use hueX parameter i will obtains this control?
https://i.redd.it/0k0ath5jr8dx.png
Thanks!

Yep

Has anyone used this? Is so can I get an example config?


Introduce bug,feature request and support templates

Amended.
fyi, checks won't run on this PR as it's only .md changes.


DM26 Completed Monitors detected as Failed in Part 12 Test 2

The Part 12 DM26 test triggers false positive outcomes for completed monitors after engine operation.
The criterion in part 12 test 2 exhibits the same conceptual error discussed for Part 2 test 8.
Additional details are provided below.

Dear Dan and Paul,
DM26 is queried in Part 12, after engine operation.  Failures for completed monitors have been observed.
Start Test 12.2 - DM26: Diagnostic readiness 3
09:30:02.8123 Destination Specific DM26 Request to Engine #1 (0)
08:18:27.8825 18EA00F9 [3] B8 FD 00 (TX)
08:18:27.8890 18FDB800 [8] 00 00 00 37 C0 16 E0 1E
DM26 from Engine #1 (0): Warm-ups: 0, Time Since Engine Start: 0 seconds
FAIL: <IP_ADDRESS>.a - Engine #1 (0) reported Misfire as 'not complete this cycle' when it reported it as 'complete this cycle' in part 11
End Test 12.2 - DM26: Diagnostic readiness 3
The proposed text for Part 12 uses the same content that was proposed for DM26 in Part 2.
6.12.2 DM26: Diagnostic Readiness 3
<IP_ADDRESS> Actions
a. DS DM26 [(send Request (PGNPG 59904) for PGNPG 64952 (SPNSPs 3303-3305)]) to each OBD ECU.
<IP_ADDRESS> Fail Criteria
a. Warn, if any individual required monitor, except Continuous Component Monitoring (CCM) is supported by more than one OBD ECU.
b. Info, if any response for any monitor supported in DM5 by a given ECU is reported as “0=monitor complete this cycle or not supported” in SP 3303 bits 1-4 and SP 3305 [except comprehensive components monitor (CCM)].
c. Fail if NACK not received from OBD ECUs that did not provide a DM26 message.
Sincerely Yours,
Eric Swenson

<IP_ADDRESS> Actions
a. Global DM26 [send Request (PG 59904) for PG 64952 (SP 3301)].
i. Record time since engine start.
ii. Separately start tracking time in software to compare to reported values later in part 11.
<IP_ADDRESS> Fail Criteria
a. If more than one ECU responds, fail if times (since engine start) differ by > 2 seconds.
b. Fail if no OBD ECU provides a DM26 message.
<IP_ADDRESS> Actions
a. DS DM26 [send Request (PG 59904) for PG 64952 (SPs 3301-3305)] to each OBD ECU.
b. Record all monitor readiness this trip data (i.e., which supported monitors are complete this trip or supported and not complete this trip).
c. Display composite status for support and enable bits for responses received from OBD ECUs.

Superseded by #1098

Document change complete for composite display. Superceded.


[UX Test]: ComputationalTool property priorities

Issue Name
ComputationalTool property priorities
Test Purpose
To identify the properties which are useful for searching/filtering and display in the user interface
Test Approach
The test will consist of asking users to sort cards with properties (and their definitions) into categories ranging from not important to very important.
Test Cost
$100
Test Time
2-3 days for set up, 1 day for collection, 2-3 days for analysis
Results URL
https://docs.google.com/spreadsheets/d/1M0CGS-nuPwx-0-3PJ6CO4wJ_mWyPIHXLljCb4N25o4I/edit?gid=0#gid=0
https://docs.google.com/presentation/d/19uFLs9Y1s99cKTFIiocx0dlfd3JF7AeLXAf481L4kxA/edit#slide=id.g1f804f59111_0_3
Related WBS task
https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/29
For internal use only. Assignee, please select the status of this issue

[ ] Not yet started
[ ] In progress
[ ] Blocked
[ ] Will not address

Status Description
No response
Test status check list

[ ] This test has yet to be discussed between NIAID, Scripps, Leidos
[ ] This test has been discussed/reported between NIAID, Scripps, Leidos
[X] This test has been scoped out
[X] This test has been drafted in the platform in which it will be conducted
[X] This test has been conducted
[X] The results of this test have been analyzed

Results: https://docs.google.com/presentation/d/19uFLs9Y1s99cKTFIiocx0dlfd3JF7AeLXAf481L4kxA/edit#slide=id.g1f804f59111_0_3

I am marking this issue as pending close out and will close it after a week if there are no additional changes needed.


Whether an image can be provided for fvm install

The original
I'm a Chinese user so you know
Just like nvm
  NVM_NODEJS_ORG_MIRROR=https://nodejs.org/dist nvm install 4.2

@zq0904 you might be looking for this https://fvm.app/docs/getting_started/configuration#different-flutter-repo

@zq0904 you might be looking for this https://fvm.app/docs/getting_started/configuration#different-flutter-repo

Thank you! That solved my problem


reconcile_transaction() somehow breaks the "rules" functionality on actual

Checks

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest version of actualpy.

Reproducible example
from actual import Actual
from actual.queries import get_accounts, get_transactions, reconcile_transaction

csv_data = [["Date","Payee","Notes","Category","Amount","Cleared", "imported_ID"]]

with Actual(base_url=URL_ACTUAL, password=PASSWORD_ACTUAL, file=FILE_ACTUAL) as actual:

accounts = get_accounts(actual.session)
for account in accounts:

     for row in csv_data:
        # here, we define the basic information from the file
        date, payee, notes, category, amount, cleared, imported_ID = (
            datetime.strptime(row[0], "%Y-%m-%d").date(),  # transform to date
            row[1],
            row[2],
            row[3],
            decimal.Decimal(row[4]),
            row[5] == "Cleared",
            row[6],
        )
        t = reconcile_transaction(
            actual_session,
            date,
            account,
            payee,
            notes,
            category,
            amount,
            imported_ID,
            cleared=cleared,
            imported_payee=payee,
            already_matched=added_transactions,
        )
        added_transactions.append(t)
actual.commit()

Log output
No response
Issue description
I am using reconcile_transaction() to import bank data into Actual.
Everything seems to be working fine, but the "rules" funcionality to rename payees do not work.
One thing I also noticed is when I import .ofx or .csv inside Actual, the payee name gets bolded, but it is not the case for the payees names of imported transactions through this API.
Expected behavior
After actual.commit() the newly created transactions should be renamed based on Actual rules automatically.
Installed versions

actualpy version: 0.4.0 - Actual Server version: 24.9.0

I understand your use case, maybe adding a run_rules on commit would be enough, but it's not optimal because this API tries to mimic adding entries via frontend, and if you add a transaction via frontend, no rule will be run (as far as I could test).
Wouldn't something like this work for you?
from actual import Actual
from datetime import datetime
import decimal
from actual.queries import get_accounts, reconcile_transaction, get_ruleset

csv_data = [["Date", "Payee", "Notes", "Category", "Amount", "Cleared", "imported_ID"]]

URL_ACTUAL = "http://localhost:5006"
PASSWORD_ACTUAL = "mypass"
FILE_ACTUAL = "CSV Importer"

with Actual(base_url=URL_ACTUAL, password=PASSWORD_ACTUAL, file=FILE_ACTUAL) as actual:
    accounts = get_accounts(actual.session)
    ruleset = get_ruleset(actual.session)
    added_transactions = []
    for account in accounts:
        for row in csv_data:
            # here, we define the basic information from the file
            date, payee, notes, category, amount, cleared, imported_ID = (
                datetime.strptime(row[0], "%Y-%m-%d").date(),  # transform to date
                row[1],
                row[2],
                row[3],
                decimal.Decimal(row[4]),
                row[5] == "Cleared",
                row[6],
            )
            t = reconcile_transaction(
                actual.session,
                date,
                account,
                payee,
                notes,
                category,
                amount,
                imported_ID,
                cleared=cleared,
                imported_payee=payee,
                already_matched=added_transactions,
            )
            actual.session.flush()  # flush to load the ids and relationships
            ruleset.run(t)  # run the rule here
            added_transactions.append(t)
    actual.commit()

It worked now! Appreciate your help!

Caught a new bug, now it's when running ruleset.run(t). The code example is basically the same as you sent above.
The error happens when my budget has rules with "imported payee - any option - name" (similar to issue #67) or has transactions automatically added by schedules rules.
This raised due to scheduled transactions. Ir only happens then the date from a transaction "t" is equal a trasaction added by a schedule. When I delete the schedule, it stops the error:

AttributeError: 'NoneType' object has no attribute 'date'
Traceback:
File "/usr/local/lib/python3.11/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 88, in exec_func_with_error_handling
result = func()
^^^^^^
File "/usr/local/lib/python3.11/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 590, in code_to_exec
exec(code, module.dict)
File "/app/streamlit_app.py", line 91, in
pluggy_sync(URL_ACTUAL, PASSWORD_ACTUAL, FILE_ACTUAL, start_date, end_date)
File "/app/functions.py", line 256, in pluggy_sync
data_to_actual(csv_data, actual.session, account)
File "/app/functions.py", line 163, in data_to_actual
ruleset.run(t)  # run the rule here
^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 555, in run
the order 'pre' -> None -> 'post'. You can provide a value to run only a certain stage of rules."""
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 544, in _run
for t in transaction:
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 495, in run
def run(self, transaction: Transactions) -> bool:
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 490, in evaluate
def evaluate(self, transaction: Transactions) -> bool:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 490, in
def evaluate(self, transaction: Transactions) -> bool:
^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 302, in run
attr = get_attribute_by_table_name(Transactions.tablename, self.field)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/rules.py", line 185, in condition_evaluation
# https://github.com/actualbudget/actual/blob/98a7aac73667241da350169e55edd2fc16a6687f/packages/loot-core/src/server/accounts/rules.ts#L302-L304
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/schedules.py", line 166, in is_approx
before = self.before(date)
^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/schedules.py", line 233, in before
return self.do_skip_weekend(dt_start, before_datetime).date()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

This raised due to " imported payee - any option - name " rule. Deleting the rule, stops the error:

ValidationError: 1 validation error for list[function-after[check_operation_type(), function-after[convert_value(), Condition]]] 0.type Input should be 'date', 'id', 'string', 'number' or 'boolean' [type=enum, input_value='imported_payee', input_type=str] For further information visit https://errors.pydantic.dev/2.9/v/enum
Traceback:
File "/usr/local/lib/python3.11/site-packages/streamlit/runtime/scriptrunner/exec_code.py", line 88, in exec_func_with_error_handling
result = func()
^^^^^^
File "/usr/local/lib/python3.11/site-packages/streamlit/runtime/scriptrunner/script_runner.py", line 590, in code_to_exec
exec(code, module.dict)
File "/app/streamlit_app.py", line 91, in
pluggy_sync(URL_ACTUAL, PASSWORD_ACTUAL, FILE_ACTUAL, start_date, end_date)
File "/app/functions.py", line 256, in pluggy_sync
data_to_actual(csv_data, actual.session, account)
File "/app/functions.py", line 136, in data_to_actual
ruleset = get_ruleset(actual_session)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/actual/queries.py", line 604, in get_ruleset
:param s: session from Actual local database.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/pydantic/type_adapter.py", line 135, in wrapped
return func(self, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/pydantic/type_adapter.py", line 384, in validate_json
return self.validator.validate_json(data, strict=strict, context=context)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Do your schedule happens to have an end date?

Sure, here it is:

I generated a new release, could you update and check if it still fails?

It failed 😔 However, there is improvement! It did not failed now because of the "imported payee" rule. It only fails because of a scheduled rule.
Oh, while typing here I was testing to find exactly what was causing the error and I found it! It's this date rule inside the schedule:

It was set to "before". Changing to "after" or unselecting it stops the error.
The error output:

AttributeError                            Traceback (most recent call last)
Cell In[4], line 15
13 start_date = "2024-09-08"
14 end_date = "2024-09-10"
---> 15 pluggy_sync(URL_ACTUAL, PASSWORD_ACTUAL, FILE_ACTUAL, start_date, end_date)
File [z:\docker\code-server\config\actual-pluggy-py\functions.py:257](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:257), in pluggy_sync(URL_ACTUAL, PASSWORD_ACTUAL, FILE_ACTUAL, start_date, end_date)
[255](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:255)             if pluggy_status: # if pluggy connection failed, do nothing
[256](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:256)                 print(f"Starting Actual reconciliation for {accName}")
--> [257](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:257)                 data_to_actual(csv_data, actual.session, account)
[259](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:259) actual.commit()
File [z:\docker\code-server\config\actual-pluggy-py\functions.py:164](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:164), in data_to_actual(csv_data, actual_session, account)
[162](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:162) actual_session.flush()  # flush to load the ids and relationships
[163](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:163) # print(t)
--> [164](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:164) ruleset.run(t)  # run the rule here
[165](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:165) added_transactions.append(t)
[167](file:///Z:/docker/code-server/config/actual-pluggy-py/functions.py:167) if t.changed():
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:594](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:594), in RuleSet.run(self, transaction, stage)
[592](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:592) if stage == "all":
[593](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:593)     self._run(transaction, "pre")
--> [594](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:594)     self._run(transaction, None)
[595](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:595)     self._run(transaction, "post")
[596](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:596) else:
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:583](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:583), in RuleSet._run(self, transaction, stage)
[581](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:581)         rule.run(t)
[582](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:582) else:
--> [583](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:583)     rule.run(transaction)
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:534](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:534), in Rule.run(self, transaction)
[531](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:531) def run(self, transaction: Transactions) -> bool:
[532](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:532)     """Runs the rule on the transaction, calling evaluate, and if the return is True then running each of
[533](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:533)     the actions."""
--> [534](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:534)     if condition_met := self.evaluate(transaction):
[535](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:535)         splits = self.set_split_amount(transaction)
[536](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:536)         if splits:
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:529](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:529), in Rule.evaluate(self, transaction)
[527](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:527) """Evaluates the rule on the transaction, without applying any action."""
[528](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:528) op = any if self.operation == "or" else all
--> [529](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:529) return op(c.run(transaction) for c in self.conditions)
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:529](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:529), in (.0)
[527](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:527) """Evaluates the rule on the transaction, without applying any action."""
[528](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:528) op = any if self.operation == "or" else all
--> [529](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:529) return op(c.run(transaction) for c in self.conditions)
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:322](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:322), in Condition.run(self, transaction)
[320](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:320) true_value = get_value(getattr(transaction, attr), self.type)
[321](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:321) self_value = self.get_value()
--> [322](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:322) return condition_evaluation(self.op, true_value, self_value, self.options)
File [c:\git\actualpy.conda\Lib\site-packages\actual\rules.py:200](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:200), in condition_evaluation(op, true_value, self_value, options)
[198](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:198)     interval = datetime.timedelta(days=2)
[199](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:199)     if isinstance(self_value, Schedule):
--> [200](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:200)         return self_value.is_approx(true_value, interval)
[201](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:201) else:
[202](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:202)     # Actual uses 7.5% of the value as threshold
[203](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:203)     # https://github.com/actualbudget/actual/blob/243703b2f70532ec1acbd3088dda879b5d07a5b3/packages/loot-core/src/shared/rules.ts#L261-L263
[204](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/rules.py:204)     interval = round(abs(self_value) * 0.075, 2)
File [c:\git\actualpy.conda\Lib\site-packages\actual\schedules.py:166](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:166), in Schedule.is_approx(self, date, interval)
[164](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:164) if date < self.start or (self.end_mode == EndMode.ON_DATE and self.end_date < date):
[165](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:165)     return False
--> [166](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:166) before = self.before(date)
[167](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:167) after = self.xafter(date, 1)
[168](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:168) if before and (before - interval <= date <= before + interval):
File [c:\git\actualpy.conda\Lib\site-packages\actual\schedules.py:233](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:233), in Schedule.before(self, date)
[231](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:231) if not before_datetime:
[232](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:232)     return None
--> [233](file:///C:/git/actualpy/.conda/Lib/site-packages/actual/schedules.py:233) return self.do_skip_weekend(dt_start, before_datetime).date()

Hello @tcpr1, thanks for your last message, I could finally reproduce the reason why it was crashing. Turns out I was comparing the wrong cutoff date when resolving the before weekend mode.
It should be fixed in the soon to be released version 0.5.1.

No more errors!

Hey @tcpr1 , I'll then close this issue with https://github.com/bvanelli/actualpy/pull/85 after including more documentation and one example using your use case as a base.


The 1.0.3-fix crashes when searching in "Android System"

are you on android 12.1?
"Android 12L (AKA Android 12.1) has patched the ability to create fabricated overlays as the shell user. Only system apps and root can create fabricated overlays now."

I am getting this issue and I am on Android 12L but with root access (Magisk 25.1). Even with root access, is this issue non-recoverable on Android 12L ?


Update to 1.1.0

[1.1.0] 21-10-2024
Added

Cross-link to Expansy Enzymes

Changed

Updated README to clarify scope of package

Removed

Removed id-mappings file: not needed anymore
Removed html-generation option from CLI
Removed html template generation
Removed dependencies related to generation of static html pages

@Adafede Removing the html generation stuff, makes more sense to have it in the new mite_web.


SqlReturn uses incorrect value

In pr #27 SqlReturn changed
    SQL_INVALID_HANDLE = -2,
to     pub const INVALID_HANDLE: SqlReturn = SqlReturn(-1);
Is this correct?

Thanks for politely phrasing it as a question, but no this is an accidental change.  The downstream libraries pretty much prevent INVALID HANDLES by design, how on earth did you find this? Luckily it should not be to hard to write a test for this to prevent that error in the future.

Fixed in 0.17.1. Thanks for reporting!