Issue with terms using 'telomeric RNA' - two different meanings

Hi, I am annotating a paper (PMID 23935072) describing a human protein that binds to human TERRA (telomeric repeat-containg RNA), and have discovered that at the moment there are two different meanings for the phrase 'telomeric RNA' in GO:
In this term, 'telomeric RNA' refers to TERRA:
GO:0097393 telomeric RNA transcription
The synthesis of telomeric RNA from a DNA template. A telomere is a complex of DNA and proteins that seals the end of a chromosome.
PMID:22139915 (S.pombe paper)
But in this term, 'telomeric RNA' refers to TERC, the RNA template component of telomerase
GO:0070034 telomeric RNA binding
Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it
PMID:16884717 (S.pombe paper)
One solution would perhaps be to replace 'telomeric RNA' with 'telomeric template RNA' in all term names referring to TERC, as is already the case for some, eg.
GO:0003721 telomeric template RNA reverse transcriptase activity
Catalysis of the extension of the 3' end of a DNA strand by one deoxynucleotide at a time. Cannot initiate a chain de novo; uses the RNA subunit of the telomerase enzyme complex as its template.
PMID:11812242
And similarly, replace 'telomeric RNA' with 'telomeric repeat-containing RNA' (and add TERRA as synonym) in all terms that refer to TERRA?
If you agree this is a way forward, then could I also request a new term that would allow me to annotate PMID 23935072 and others, something like:
GO:NEW telomeric repeat-containing RNA binding
Interacting selectively and non-covalently with long non-coding RNA molecules transcribed from subtelomeric regions in most eukaryotes. Telomeric repeat-containing RNA (TERRA) molecules consist of subtelomeric-derived sequences and G-rich telomeric repeats.
(PMID: 20655916)
thanks, Jess
@NancyCampbell
@ValWood

hi,
Sounds reasonable. The second paper we don't appear to have curated so I'm not sure how it came to be the referencing  paper in the GO term...we will process this paper soon...
Val

Stumbled on this while helping a CACAO student with something completely unrelated... Shouldn't the template RNA be telomerase RNA not telomeric RNA?

Hi Jim, that would be simpler and also fit better with definition of the template RNA in RNACentral: telomerase RNA component (TERC), telomerase RNA
http://rnacentral.org/rna/URS00004A7003/9606
So updated suggestions would be:

Replace 'telomeric RNA' with 'telomeric repeat-containing RNA' (and add TERRA as synonym) in all terms that refer to TERRA

Replace 'telomeric RNA' and 'telomeric template RNA' with 'telomerase RNA' in all term names referring to TERC

Create new term, something like:
GO:NEW telomeric repeat-containing RNA binding
Interacting selectively and non-covalently with long non-coding RNA molecules transcribed from subtelomeric regions in most eukaryotes. Telomeric repeat-containing RNA (TERRA) molecules consist of subtelomeric-derived sequences and G-rich telomeric repeats.
(PMID: 20655916

thanks, Jess

Hi Everyone,
'telomeric RNA binding' is defined as: Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it. So, it looks like that term refers to either the 'telomerase RNA' or the 'telomeric repeat-containing RNA'.
So I propose that we leave that term as is and create two children, 'telomeric repeat-containing RNA binding' and 'telomerase RNA binding' with the added specification. Does this make sense?
In the meantime here is what I have done to the other terms:
[Term]
id: GO:0003721
-name: telomeric template RNA reverse transcriptase activity
+name: telomerase RNA reverse transcriptase activity
namespace: molecular_function
def: "Catalysis of the extension of the 3' end of a DNA strand by one deoxynucleotide at a time. Cannot initiate a chain de novo; uses the RNA subunit of the telomerase enzyme complex as its template." [EC:<IP_ADDRESS>, PMID:11812242]
synonym: "telomerase, catalyst" EXACT []
@@ -478000,21 +478000,23 @@
[Term]
id: GO:0097393
-name: telomeric RNA transcription
+name: telomeric repeat-containing RNA transcription
namespace: biological_process
-def: "The synthesis of telomeric RNA from a DNA template. A telomere is a complex of DNA and proteins that seals the end of a chromosome." [GOC:al, PMID:22139915]
+def: "The synthesis of telomeric repeat-containing RNA from a DNA template. A telomere is a complex of DNA and proteins that seals the end of a chromosome." [GOC:al, PMID:22139915]
+synonym: "TERRA transcription" EXACT [PMID:22139915]
is_a: GO:0006351 ! transcription, DNA-templated
created_by: paola
creation_date: 2012-10-23T10:06:17Z
[Term]
id: GO:0097394
-name: telomeric RNA transcription from RNA pol II promoter
+name: telomeric repeat-containing RNA transcription from RNA pol II promoter
namespace: biological_process
-def: "The synthesis of telomeric RNA from a DNA template by RNA Polymerase II (Pol II), originating at a Pol II promoter." [GOC:al, PMID:22139915]
-synonym: "telomeric RNA transcription from Pol II promoter" EXACT []
+def: "The synthesis of telomeric repeat-containing RNA from a DNA template by RNA Polymerase II (Pol II), originating at a Pol II promoter." [GOC:al, PMID:22139915]
+synonym: "telomeric RNA transcription from Pol II promoter" RELATED []
+synonym: "TERRA repeat-containing RNA transcription from RNA pol II promoter" RELATED [GOC:22139915]
is_a: GO:0006366 ! transcription from RNA polymerase II promoter
-is_a: GO:0097393 ! telomeric RNA transcription
+is_a: GO:0097393 ! telomeric repeat-containing RNA transcription
created_by: paola
creation_date: 2012-10-23T10:09:15Z

Hi,
I think 'telomeric RNA binding' refers only to the telomerase template RNA at present, as TERRA is transcribed directly from telomeric DNA (they are lncRNAs of variable length/sequence, starting in the subtelomeric regions refs. original 2007 paper PMID 17916692, recent review 25926849) - so not really complementary to the telomerase template RNA molecule, of which only a small part is  complementary to telomeric DNA.
So I think it would be good if possible to have a new term as suggested for TERRA binding, as there are several proteins that are involved in telomere maintenance through this mechanism, and would be good to capture this newly discovered process in GO if possible.
Updated term suggestions look fine - small suggestion:
this
+synonym: "TERRA repeat-containing RNA transcription from RNA pol II promoter"
could be simplified to:
+synonym: "TERRA transcription from RNA pol II promoter"
thanks

OK. I think I got it. I editied the synonym as you suggested and added:
[Term]
+id: GO:0061752
+name: telomeric repeat-containing RNA binding
+namespace: molecular_function
+def: "Interacting selectively and non-covalently with long non-coding RNA molecules transcribed from subtelomeric regions in most eukaryotes. Telomeric repeat-containing RNA (TERRA) molecules consist of subtelomeric-derived sequences and G-rich telomeric repeats." [GOC:dph, PMID:20655916]
+synonym: "TERRA binding" EXACT [PMID:20655916]
+is_a: GO:0003723 ! RNA binding
+created_by: dph
+creation_date: 2015-11-13T07:55:06Z

Hi,
I am annotating this paper PMID 16507993 figure 1A. I want to annotate mTERT IDA telomeric RNA binding. This does not exist (I think it did before). Anyway, what I can find instead is GO:0070034
Name 	telomerase RNA binding
Ontology 	Molecular Function
Definition 	Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it.
As mentioned above, now there is a specific TERRA binding term:
+id: GO:0061752
+name: telomeric repeat-containing RNA binding
+namespace: molecular_function
+def: "Interacting selectively and non-covalently with long non-coding RNA molecules transcribed from subtelomeric regions in most eukaryotes. Telomeric repeat-containing RNA (TERRA) molecules consist of subtelomeric-derived sequences and G-rich telomeric repeats." [GOC:dph, PMID:20655916]
+synonym: "TERRA binding" EXACT [PMID:20655916]
+is_a: GO:0003723 ! RNA binding
+created_by: dph
+creation_date: 2015-11-13T07:55:06Z
So, does it not make sense to change the definition of
GO:0070034
Name 	telomerase RNA binding
Ontology 	Molecular Function
Definition 	Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it.
to
GO:0070034
Name 	telomerase RNA binding
Ontology 	Molecular Function
Definition 	Interacting selectively and non-covalently with the telomerase RNA template.
however if the previous definition is maintained, then,
+id: GO:0061752
+name: telomeric repeat-containing RNA binding
should be a child of GO:0070034 (though this does not make sense with its current name). And a sibling created which is the existing
+id: GO:0061752
+name: telomeric repeat-containing RNA binding.
Am i missing something? was it agreed to change the definition of
GO:0070034
Name 	telomerase RNA binding
and I missed it!!
Sorry to throw a spanner in the works!
Thanks
Nancy

Hi Nancy,
So are you suggesting changing changing GO:0070034 back to telomeric RNA binding, retaining the def as 'Definition Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it.'? Then making the new 'telomeric repeat-containing RNA binding' GO:0061752 a child of it and creating a new child called 'telomerase RNA binding'? I would be ok with this.
-D

Hi David,
I just looked at the manual annotations that are available to GO:0070034 and they all 'mean' telomerase RNA binding. So the new name 'means' what it should mean in context of current annotations (and it makes sense somehow). But the definition itself does not i.e. the last bit referrring to "or to RNA complementary to it" is wrong. I guess there is no harm in inserting a new term which is a child of RNA binding but is parent to both: NEW telomeric RNA binding term parent to both GO:0070034 & GO:0061752 and give it the wider definition  Interacting selectively and non-covalently with the telomerase RNA template or to RNA complementary to it (currently assigned to GO:0070034). This could be a safe way to keep the current annotations as they are but assigned to a better named term with a more specific definition.
So I do agree with you that a parent term is a very good idea but due to the current annotations, maybe it can be done with a tweak?
Thoughts?
Thanks David

I could just tweak the definition of the current term. 'Interacting selectively and non-covalently with the telomerase RNA template'
I think that is what Jess was getting at.

I am happy with that. Thanks David.

Hi, I definitely agree with this tweak to definition:
GO:0070034
Name telomerase RNA binding
Ontology Molecular Function
Definition Interacting selectively and non-covalently with the telomerase RNA template.
BUT I don't think we should have a NEW telomeric RNA binding parent term, as it wrongly implies that these two RNAs are different subtypes of the same class of RNA, or are complementary to each other, neither of which is the case? In my opinion it would be clearer just to keep both these terms as children of the more general RNA binding as they are now, ie
parent GO:0003723 RNA binding:
children
GO:0061752
Name telomeric repeat-containing RNA binding
GO:0070034
Name telomerase RNA binding
My original point was that the phrase 'telomeric RNA' had been used in GO to refer to two completely different types of RNA molecule (TERRA, any lncRNA transcribed directly from telomeric DNA, variable lengths and sequences, and the telomerase template RNA, the specific 451bp RNA encoded by the TERC gene, chr3q in humans) - TERRA isn't complementary to the telomerase RNA template as only a small part of the TERC transcript is complementary to telomeric DNA.
hope that makes sense?
thanks, Jess

OK. I tweaked the definition and left the structure as is. Thanks for all of your help on this.
-David

Thanks David.

Hi David,
Please could you add the following GOC refs to the new term below in QuickGO:
GOC:BHF, GOC:BHF_telomere and GOC:jbu
Thanks very much and apologies for forgetting to include this at the time of the request,
bw Jess
GO:0061752
telomeric repeat-containing RNA binding
Ontology	Molecular Function
Definition	Interacting selectively and non-covalently with long non-coding RNA molecules transcribed from subtelomeric regions in most eukaryotes. Telomeric repeat-containing RNA (TERRA) molecules consist of subtelomeric-derived sequences and G-rich telomeric repeats.

Hi Jess,
I must have done this, but forgot to close the ticket because the xrefs are associated with GO:0061752.
-D


Use path module so windows doesn't attempt to run renderer tests with…

… plain mocha. r=vporof

Coverage remained the same at 42.34% when pulling 23da1182e90eec464ebaabd1c86a73ab02322f6f on jsantell:fix-unix-specific-path into 8f3facd893cf45b566132cce7a0d4b49e621543e on mozilla:master.

Handled in #476


Refactor GetHostZoneUnbondingMsgs to sub funcs

Closes: #329 #417
Context and purpose of the change
Refer from #329 there a lot of logic in GetHostZoneUnbondingMsgs that makes it difficult to read.
Brief Changelog
Create sub funcs with separate processing logic:

GetUnbondingAmountAndRecords: get total unbonding amount & all unbonding record of hostzone
DistributeUnbondingAmountToValidators: distribute unbonding amount to all validators
SplitDelegationMsg: wrap to msgs

Author's Checklist
I have...

[x] Run and PASSED locally all GAIA integration tests
[ ] If the change is contentful, I either:

[ ] Added a new unit test OR
[ ] Added test cases to existing unit tests

[ ] OR this change is a trivial rework / code cleanup without any test coverage

If skipped any of the tests above, explain.

Future plan

[ ] add test for new funcs
[ ] refactor GetHostZoneUnbondingMsgs test to table-driven & add more edge case

@sampocs I just formated code, add unit-tests for sub funcs, could u pls check it out

Friendly bump on this PR @hieuvubk, looks like there are some outstanding comments/recommendations from Sam, and you pushed new commits since his comments. Do you feel you've addressed them?
I'm also happy to review this if it would be helpful. Just let me know @sampocs.

open


No Informations can be fetched

Adapter turns green, but can not fetch informations from polestar. there is an error in the log:
{"errors":[{"message":"Cannot query field \"consumerId\" on type \"MyStarCar\".","locations":[{"line":70,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"engine\" on type \"MyStarCar\".","locations":[{"line":71,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"exterior\" on type \"MyStarCar\". Did you mean \"exteriorImageUrl\"?","locations":[{"line":72,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"exteriorCode\" on type \"MyStarCar\". Did you mean \"exteriorImageUrl\"?","locations":[{"line":73,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"gearbox\" on type \"MyStarCar\".","locations":[{"line":75,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"interior\" on type \"MyStarCar\".","locations":[{"line":76,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"interiorCode\" on type \"MyStarCar\".","locations":[{"line":77,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"interiorImageUrl\" on type \"MyStarCar\". Did you mean \"exteriorImageUrl\"?","locations":[{"line":78,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"packageCode\" on type \"MyStarCar\". Did you mean \"package\"?","locations":[{"line":82,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"pdfUrl\" on type \"MyStarCar\".","locations":[{"line":83,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"status\" on type \"MyStarCar\".","locations":[{"line":84,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"steering\" on type \"MyStarCar\".","locations":[{"line":85,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"wheels\" on type \"MyStarCar\".","locations":[{"line":87,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}},{"message":"Cannot query field \"wheelsCode\" on type \"MyStarCar\". Did you mean \"modelCode\"?","locations":[{"line":88,"column":3}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED"}}]}

This correct the adapter can only show general data of the car but no actual data. The connection via mqtt to get actual values is not done

Yes I know, but there are no data fetched.

Yes only in the log so far

@TA2k It seems the messages are serialised in protobuf format. I found a whole bunch of .proto files depicting all kinds of requests and responses while decoding the Polestar Android APK. They are part of the Volvo vocmo SDK, where the app seems to be built upon. The app's actual Java files also seem to consume parts of these files, matching feature parity. It's all pretty heavily obfuscated though, so not easy to find out what's exactly done and how.
Would you happen to still have some of the messages you received? My car doesn't arrive until 24 feb this month, but I'd already like to take a look to see what we can find out. Does the rest of the connection process work? Do you own a Polestar yourself?
I think the first thing to do is to figure out how they define which message is received. Probably utilising an enveloping/wrapper message. I think at least reading events would be easy if we found that part out. Requesting updates may be a bit harder.

It seems it's a lot more complex than just protobuf decoding the strings. It seems the mobile device first needs to be enrolled with the API. It will in turn get a private key/cert pair. With this cert/key pair it can sign encrypted enveloped messages (through asn1 encoding) which contain protobuf messages.
I suppose these envelopes are also the way the MQTT communication is authenticated.
At least.. That's what I'm guessing based on the decompiled app source. It's pretty sophisticated. :-)

Yes you are right this is also my assumption
here you can see a mqtt connection

There also two files in the android shared keystore
(CACERT_xxxxdevicePairingKey and CACERT_xxxxcatalogKey.
I think this files are used to encrypt the mqtt traffic
Unfortunately I cannot see the de and encryption in Frida it looks like they are doing his own stuff in libvocmo-lib.so
Because of the complexity I don't think this is good way to pursuit. I saw you can connect your polestar with the tibber app it would be interesting how this looks like or waiting for an api like Volvo offers.

I've done some more digging. I've found the lib they use for those message crypto bindings (libvocmo-lib.so). It is indeed compiled C/C++ code, and bundled with the APK for all supported platforms (including x86_64).
It seems they use a project called 'djinni' (https://djinni.xlcpp.dev/) for interfacing from the apps with the lib. Conveniently, they kept the 'bindings' unencrypted inside the Android source code;

So in theory, one could simply take libvocmo-lib.so and implement those Djinni bindings, use their official library for decoding/signing the messages. This would remove the need to reverse engineer the lib file which seems to be hard.
Some more references; https://www.dropbox.com/s/k1b214bsqgpx4e6/Djinni Talk - CppCon 2015.pdf?dl=0.


SightLayer.computeSight is deprecated since FVTT 0.8.2

It's worth noting that while Foundry 0.8 still contains the deprecated SightLayer.computeSight function, that function won't be called anymore. As a result the vision caching done by this module won't take effect until TokensVisible changes over to the new function.
Additionally, Foundry 0.8 does a lot less unnecessary vision recalculations than 0.7 did, so it might be worth revisiting whether vision caching still brings any performance benefit or not.

TokensVisible 2.3.0 now uses the new function in 0.8 and the old one in 0.7
in 0.7 this feature makes a massive difference in scenes with many tokens, and if I fork the code then I wont be able to continue supporting 0.7.x without creating extra work.
The many of the unnecessary vision recalculations are removed in 0.8, however occasionally the cache still gets a hit, for whatever that's worth.  (maybe not much)  In some cases someone may want to use it.  Possibly if there are a a large number of walls.
thank you for bringing this to my attention.


[RSPEED-275] Add local history management and improve the command

Now we can save/view and query (basic) the history locally. The history module allows for modularity to switch between LocalHistory and a future implementation of a RemoteHistory that will be stored in the servers.
We also improved the history command to show in the terminal the conversation history by having the following modes:

All history
First history
Last history

More modes will be implemented in the future, but for now, this is enough.

Jira Issues:

RSPEED-275

Checklist

[ ] Jira issue has been made public if possible
[ ] [RSPEED-] is part of the PR title
[ ] PR title explains the change from the user's point of view
[ ] Code and tests are documented properly
[ ] The commits are squashed to as few commits as possible (without losing data)

/packit build


improve e2e test

fixes #101

coverage: 95.037%. remained the same when pulling e7c2cfc371d0c12b735da4ae758034070141f6bc on improve_e2e_test into c859e7748bcd97505d046ff82fb354357a4da830 on main.

Nice!
I few suggestions:

Test waits until it receives first non-zero sample, then receives 10000 samples, then checks that each one is either zero or has correct value; it means that if we received one non-zero sample, and then 10000 zeros, the test will pass. Instead of this, I suggest to remove upper limit for how much samples we receive, and instead place lower limit of how much non-zero samples we want. So we will receive samples, check that each one is either zero or correct, and repeat this until we accumulate N non-zero samples. This way the test will check that there are a lot (N) of good samples, and at the same time it won't be bound to a particular loss ratio. Losses may happen on a loaded CI worker.

100 samples is not very interesting number - they will fit in one packet. I suggest to wait until we receive something like 10K non-zero samples, so that it will be many packets.

To avoid unintuitive part with 0.5, we can work with integers:
valueL = int(samples[i*2] * 100)
assert((prevValue + 1) % 100 == valueL)

When assertions fails, it makes sense to report index and maybe dump the whole stream or block.

Let's replace magical number 2 with a constant like NumChan(nel)s.

When assertions fails, it makes sense to report index and maybe dump the whole stream or block.

--- FAIL: TestEnd2End_Default (0.42s)
    e2e_test.go:146:
                Error Trace:    /Users/andreybushmin/workspace/roc/roc-go/roc/e2e_test.go:146
                Error:          Not equal:
                                expected: 2
                                actual  : 3
                Test:           TestEnd2End_Default
                Messages:       prevL: 1, valueL: 2, index: 2, samples: [0=0.01, 1=-0.01, 2=0.02, 3=-0.02, 4=0.03, 5=-0.03, 6=0.04, 7=-0.04, 8=0.05, 9=-0.05, 10=0.06, 11=-0.06, 12=0.07, 13=-0.07, 14=0.08, 15=-0.08, 16=0.09, 17=-0.09, 18=0.10, 19=-0.10, 20=0.11, 21=-0.11, 22=0.12, 23=-0.12, 24=0.13, 25=-0.13, 26=0.14, 27=-0.14, 28=0.15, 29=-0.15, 30=0.16, 31=-0.16, 32=0.17, 33=-0.17, 34=0.18, 35=-0.18, 36=0.19, 37=-0.19, 38=0.20, 39=-0.20, 40=0.21, 41=-0.21, 42=0.22, 43=-0.22, 44=0.23, 45=-0.23, 46=0.24, 47=-0.24, 48=0.25, 49=-0.25, 50=0.26, 51=-0.26, 52=0.27, 53=-0.27, 54=0.28, 55=-0.28, 56=0.29, 57=-0.29, 58=0.30, 59=-0.30, 60=0.31, 61=-0.31, 62=0.32, 63=-0.32, 64=0.33, 65=-0.33, 66=0.34, 67=-0.34, 68=0.35, 69=-0.35, 70=0.36, 71=-0.36, 72=0.37, 73=-0.37, 74=0.38, 75=-0.38, 76=0.39, 77=-0.39, 78=0.40, 79=-0.40, 80=0.41, 81=-0.41, 82=0.42, 83=-0.42, 84=0.43, 85=-0.43, 86=0.44, 87=-0.44, 88=0.45, 89=-0.45, 90=0.46, 91=-0.46, 92=0.47, 93=-0.47, 94=0.48, 95=-0.48, 96=0.49, 97=-0.49, 98=0.50, 99=-0.50]

output example (changed assert to fail)
all other suggestions are fixed

Thanks!!


Unable to find a valid app

Got 1.2.10 from the archive and tried running it. When it starts updating I immediately get the following error
Edit: Same with setup version

What? 1.2.10? Were at 2.0.1 now. https://github.com/dustinblackman/Championify/releases


[2/3] Timer abstraction: add OneShotTimer/PeriodicTimer drivers, Timer trait

This is still a bit messy overall, but I plan to clean things up in the next and final PR in this series. Refactoring and documentation will be improved in said PR.
At this time OneShotTimer is working correctly both when backed by TIMGx and SYSTIMER. PeriodicTimer is working when backed by TIMGx, however there is one small issue remaining when backed by SYSTIMER.
Given the following example:
#![no_std]
#![no_main]

use esp_backtrace as _;
use esp_hal::{
    peripherals::Peripherals,
    prelude::*,
    timer::{systimer::SystemTimer, PeriodicTimer},
};
use esp_println::println;

#[entry]
fn main() -> ! {
    let peripherals = Peripherals::take();

    let systimer = SystemTimer::new(peripherals.SYSTIMER);
    let mut periodic = PeriodicTimer::new(systimer.alarm0);

    println!("Countdown 5 times using PeriodicTimer...");
    periodic.start(1000.millis());
    let mut i = 5;
    while i > 0 {
        println!("{i}");
        nb::block!(periodic.wait()).unwrap();
        i -= 1;
    }
    periodic.cancel().unwrap();
    println!("Done!");

    loop {}
}

Both 5 and 4 are immediately printed, and then the countdown continues as expected from there. Not sure why the first iteration is clearing right away, still looking into it.

Solution to the systimer periodic timer problem seems to be this:

i.e. this seems to fix it for me (tested on C6 only for now)
    fn load_value(&self, value: MicrosDurationU64) {
        ....

        if auto_reload {
            // Period mode

            systimer
                .target_conf(CHANNEL as usize)
                .modify(|_, w| unsafe { w.period().bits(ticks as u32) });

            #[cfg(not(esp32s2))]
            systimer
                .comp_load(CHANNEL as usize)
                .write(|w| w.load().set_bit());

            // Clear and then set SYSTIMER_TARGETx_PERIOD_MODE to configure COMPx into period mode
            systimer.target_conf(CHANNEL as usize).modify(|_,w| w.period_mode().clear_bit());
            systimer.target_conf(CHANNEL as usize).modify(|_,w| w.period_mode().set_bit());
        } else {
            // Target mode

                     .....

5 is still printed immediately but not 4


Switch to a more explicit core data type; ditch STM

This is a WIP branch that switches to a more explicit Widget representation and ditches stm in the process.
The reasons are twofold:

This representation separates views from effects. This means that we can discharge effects without having to emit a new view/DOM at every step. A purely effectful view won't ever emit a DOM change. The current implementation will emit a new DOM on every effect, which is pretty wasteful.
stm is slow. Simple MVars with threads are much faster, although I suspect more care needs to be taken in regards to async exceptions. Informal tests with 10k elements revealed MVars to be an order of magnitude faster than stm.

Note, the first implementation still depended on stm but uncovered this rare bug in GHC and so served as motivation to experiment with a pure MVar implementation.
What do you think?

Thanks for the PR! I will get back to you soon after I've had time to look at it.

Introduced a Forever constructor and a single-child fast path in orr.
Regarding async concurrently' seems to not rely on stm currently, although there's also an stm version behind a #define flag. I suspect it's probably not a good idea to rely on library internals too much; maybe just copy over the exception masking logic?

Introduced a Forever constructor and a single-child fast path in orr.

I still need to find time to look at it in detail, but just to yak shave a little, forever reminds me of Control.Monad.forever. A better name might be something like never, or blocked, or stuck (I think I like this one best).

Regarding async, concurrently' seems to not rely on stm currently, although there's also an stm version behind a #define flag. I suspect it's probably not a good idea to rely on library internals too much; maybe just copy over the exception masking logic?

Yeah I am in favor of copying over the code. Async also seems to pull in other deps via hashable which can be avoided.

I also tidied up the code a little and included some comments.

@pkamenarsky I merged this to allow moving this forward, since it seems like the right direction to take, even though I haven't had a lot of time to look at it myself. Thanks!


⚠ Bofrost.at has degraded performance

In 6042da7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1353 ms

Resolved: Bofrost.at performance has improved in a543c61.


Update main.go

Updated the syros-deployctl version in deployctl/main.go from "undefined" to "1.3.0" to better reflect the correct version.

You can set the version is set at build time:
go build -ldflags "-X main.version=$(APP_VERSION)" -o dist/syros-deployctl github.com/stefanprodan/syros/deployctl

@bogdan-radocea I've published the deployctl binary with the correct version on GitHub releases.
curl -o /usr/local/bin/syros-deployctl -L https://github.com/stefanprodan/syros/releases/download/v1.3.0/syros-deployctl
chmod +x /usr/local/bin/syros-deployctl

syros-deployctl -h

VERSION:
   1.3.0


Support static routing_key

This Pull Request makes the routing keys work with static values, as mentioned in issue #21
How it works
If the configuration of the routing key starts with an uppercase letter, it uses parameter keys from the endpoint URL
Otherwise, it is used as a static value.
Routing key documentation: https://www.krakend.io/docs/backends/amqp-producer/#routing_key

Hello @alombarte @kpacha
Could you please review my PR if you have some time?

ouch! I reviewed it 2 weeks ago and forgot to click the submit button...
I think it would be better to add a new configuration parameter instead of checking if the value starts with an uppercase, because you may want a static routing key that starts with an uppercase

@kpacha Thanks for your feedback!
Do you mean something like this?
type producerCfg struct {
	queueCfg
	Mandatory        bool   `json:"mandatory"`
	Immediate        bool   `json:"immediate"`
	ExpirationKey    string `json:"exp_key"`
	ReplyToKey       string `json:"reply_to_key"`
	MessageIdKey     string `json:"msg_id_key"`
	PriorityKey      string `json:"priority_key"`
	RoutingKey       string `json:"routing_key"`
	StaticRoutingKey bool   `json:"static_routing_key"`
}

Have a boolean parameter to define whether the routing_key value is static?
Let me know if you have something different in your mind, and I will implement it!

@lennondtps yes, that's exactly what I had in mind. I think that could offer the complete feature.

@alombarte @kpacha This will be merged in master or you will create another branch?

Hi @lennondtps , this will be merged into master and released with KrakenD 2.5 which is out within this quarter


How to exclude a entire subfolder and its contents from the assets manifest

[Enter feedback here]
I have been unsuccessful in figuring out how to exclude an entire subfolder and its contents from the generated assets manifest. Would you show an example on this page, please?

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

ID: 7c8c235e-1ce4-046f-e848-294096361922
Version Independent ID: 6240f626-97d4-ace0-5e34-20e1c3592f86
Content: Build Progressive Web Applications with ASP.NET Core Blazor WebAssembly
Content Source: aspnetcore/blazor/progressive-web-app.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @comfycoder ... Could you open an issue on the engineering repo asking this? Open it at ...
https://github.com/dotnet/aspnetcore/issues
Add a cc: @guardrex to your issue so that I can follow the discussion.
Leave this issue open. When we get an answer there, I'll react over here on this issue.

This seems like an MSBuild question. You'll want to do something like this, except instead of removing from <Content>, you need to remove from <ServiceWorkerAssetsManifestItem>.

@SteveSandersonMS, sounds like you're saying that this is an edge case that we won't be covering. Is that correct?

No, just saying it's a question about MSBuild rather than Blazor :) See MSBuild docs for info about how to manipulate its item groups.

Oh, if you're asking whether I think this should be covered in Blazor docs, I'd say no, since it's not our goal in Blazor docs to document all the ways you can manipulate things with MSBuild.

Actually my apologies, I was wrong. There isn't a good supported way to remove items from the list after it's been populated, because it's populated inside the Blazor build internal targets, and we don't support hooking into the before/after states of those targets.
Anyone who really wants to do this can look at the sources at https://github.com/dotnet/aspnetcore/blob/master/src/Components/WebAssembly/Build/src/targets/ServiceWorkerAssetsManifest.targets#L103-L113 to see how the behavior could be modified in custom MSBuild targets, but please be cautioned that we don't consider the underscore-prefixed target names to be public API, so we may change them in any future release.
If we get more customer feedback that further extensibility is needed here we can certainly look into adding a feature for it in the future.

Thanks @SteveSandersonMS
@comfycoder, I'm going to close as not actionable for the docs, but that should give u enough info to do something ..... well ...... 💀 dangerous at your own risk! 💀 :smile: lol

If we get more customer feedback that further extensibility is needed here we can certainly look into adding a feature for it in the future.

I opened the issue linked above ☝ to see if that drums up interest from any other customers.


华为盒子上频繁出现 unexpected end of stream

Before Issue

Please search on the Issues
Please search on the wiki
Please set FileDownloadLog.NEED_LOG=true and review the Logcat output from main process and :filedownloader process ( pay attention to Warn and Error level logcat)

Issue

What problem do you get?
Which version of FileDownloader are you using when you produce such problem?
How to reproduce such problem?
Do you set FileDownloadLog.NEED_LOG=true?
Could you please reproduce this problem and provide all main process and :filedownloader process logcat
Can you fix it by yourself and request PR, if not, what's problem do you get when you try to fix it

P.S. If you don't know how to get :filedownloader process, it's recommended to using pidcat to just filter all your application logcat, or define process.non-separate=true on filedownloader.properties

请在Issue前认真的跟进上面提到的建议，这样将可以极大的加快你遇到问题的处理。

准确来讲 unexpected end of stream 是一个服务端的错误，你可以看 这里 了解更多。
你的描述里是在华为盒子上频繁出现，那么可以建议你参考 这里 的方案进行解决。


fix: Use bucket KMS key arn if provided for W&B managed bucket, always use that key even if empty for customer provided buckets

This is a fix for customers using SSE Encryption.

This PR is included in version 4.21.5 :tada:


$GIT_INDEX_FILE is set after newly starting VS Code as a git editor

Version: 1.65.0-insider (Universal)
Commit: 1d618b2df1cf08f0e3ab706353f0a8f6f1424795
Date: 2022-02-18T05:15:42.132Z
Electron: 16.0.9
Chromium: 96.0.4664.174
Node.js: 16.9.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 20.6.0
$  which git
/usr/bin/git

$  git --version
git version 2.32.0 (Apple Git-132)

A wrong  $GIT_INDEX_FILE interferes with the git feature of VS Code. For example, files are displayed as untracked.

Steps to Reproduce:

Set VS Code as a git editor in ~/.gitconfig with:

[core]
    editor = code -nw

Close VS Code.
On Terminal.app, execute git commit filename.txt, which newly starts VS Code as a git editor
Open VS Code Terminal. Execute echo $GIT_INDEX_FILE. We can see $GIT_INDEX_FILE is set.
Close the window.
Open a new window of VS Code.
Open VS Code Terminal. Execute echo $GIT_INDEX_FILE. We can see $GIT_INDEX_FILE is still set.
The wrong  $GIT_INDEX_FILE would interfere with the git feature of VS Code.

Workaround:
Set
[core]
    editor = env -u GIT_INDEX_FILE code -nw

Ref:

https://github.com/git/git/blob/e6ebfd0e8cbbd10878070c8a356b5ad1b3ca464e/commit.c#L1715-L1736
https://github.com/git/git/commit/8babab95af7c01d9ea75c97ee0df40e5a2170b83


[otbn] Extend testplan as discussed in testplan review meeting

@weicaiyang:

can you also add how you verify the idle?

Good point. I'll add something to the DV document and push it up in a separate PR.


Create PATCH /shipping-instructions/{ID} which relies on JSON patch

Evaluate the JSON patch approach (e.g. https://www.baeldung.com/spring-rest-json-patch) while working on the model level (i.e. write our own patching).
Complications:

Track creation vs. updates vs. deletions

Cover:

[ ] Handle patch of cargoItems

[ ] CargoItem
[ ] CargoLineItem

[ ] Handle patch of shipmentEquipments

[ ] ShipmentEquipment
[ ] Equipment
[ ] ActiveReeferSettings
[ ] Seals

[ ] Handle patch references

[ ] Handle patch of documentParties

[ ] DocumentParty
[ ] Party

[ ] Handle patch of shipmentLocations

We have decided to drop PATCH


Why I can't use this extensions?

What did you do?
I installed SwifterSwift using CocoaPods
What did you expect to happen?
I want to use extensions
What happened instead?
SwifterSwift installed but I can't use extensions
SwifterSwift Environment

SwifterSwift version: last version
Xcode version: last version
macOS version running Xcode: HighSierra
Swift version: 4.2
Platform(s) running SwifterSwift: IOS

ScreenShot

You might have to add
import SwifterSwift
At the top of your file

You might have to add
import SwifterSwift
At the top of your file

But it's an extension. I think, i don't need to import SwifterSwift.
And i already tried. Not working.

I removed pod and install manually.. I can use extensions now..


take timestamp from arduino

Looks generally ok, but does not address a problem I considered when thinking about this. The 16-bit millis value will rollover quite frequently. I think we need to keep a local counter with infinite depth and adjust that value by the difference between the current millis() value and the previous millis() value.
localTime = lastLocal + (currMillis - lastMillis)
but we have to deal with the rollover condition and deal with that appropriately.
if currMillis < lastMillis: some 16-bit adjust subtraction here.

Ok. I agree you are correct. I had it in my head from previous readings that millis() was 16-bit. I am wrong.


GraphQL parse/validation caching

Since GraphQL parse and validate phase are static (does not depend on the user input like variables), we can implement caching for both phases (for parser: String => query::Document, for validation: String | query::Document => Vec<ValidationError>).
This might be more relevant once https://github.com/graphprotocol/graph-node/pull/3057 lands, because if we'll add more validation rules it might add some (probably minimal) overhead.
Usually it's implemented with LRU.

That's a good idea! For our purposes, I think it would be enough to keep a HashSet of the shape hashes of the queries that were successfully validated. We already calculate that hash (a u64) for all queries anyway.

https://github.com/graphprotocol/graph-node/pull/3188#issuecomment-1021490277


subprocess: don't use shell

There is no need to.

Please pull changes from upstream/dev

@Silverarmor done


Create favicon

If you have any imaginations on this, or any descriptions of what it should convey, let me know. Can work on it.

@ranitripathi5 I wanted something which related with fractals like sierpinski triangle or mandelbrot set, but you are open to use your creativity


cleanup static nginx files

This removes unreferenced js files from the nginx html directory, and updates the default API index.html page to include instructions on installing manage. This includes an outdated jqueery version -t he only component that uses jquery appears to be oc-id, and they pull it in at build time via the jquery-rails gem
ping @chef/lob /cc  @raskchanky @jcreedcmu

Note that the CI failure above looks transient, and occurred in post-test cleanup. Re-running it.

This is what the api index page looks like with these updates.

Updated image after editing the intro paragraph.

:+1:

Is this good to merge?  Would love someone from VI to confirm.

:+1:

Some text updates from marketing inbound. I thought I pushed them already.
Will do so when I'm at my desk
On Mon, Sep 21, 2015, 12:04 PM Nathan L Smith<EMAIL_ADDRESS>wrote:

[image: :+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/chef/chef-server/pull/524#issuecomment-142028266.


[11.x] Standalone usage of illuminate/collections is broken

Laravel Version
11
PHP Version
8.2
Database Driver & Version
No response
Description
We maintain a package that uses illuminate/collections and nothing else from Laravel. I wanted to test if it's compatible with Laravel 11, so I installed master. But now I get this error:

Fatal error: Trait "Illuminate\Support\Traits\Dumpable" not found in /var/www/html/vendor/illuminate/collections/Traits/EnumeratesValues.php on line 56

Related: https://github.com/laravel/framework/pull/47122
Steps To Reproduce

Start an empty/basic Composer-project;
composer require "illuminate/collections:^11.0";
Try to use Collections;
See the error.

Fixed.


Add nmslib and lucene dashboard to website

Description
Use share-> Embed code to copy dashboards from https://benchmarks.opensearch.org/ to  be available as part of website
Issues Resolved
#2871
Check List

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

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@AMoo-Miki Can you please help us approving and merging this PR?

Can i get review for this? Thanks


Add support for usernames already defined in ssh config

Is it possible to add support for usernames already defined in the ssh config?  It isn't technically "passing" it but ssh just uses it.  This is how I believe Ansible works.  This would be very helpful so (a) usernames aren't hardcoded and (b) a username configured already for the server in ssh so you don't duplicate the definition elsewhere.
An easy example is ec2-user for Amazon Linux instances.  I already have this defined in my ssh config.  I'd like to avoid defining it in another spot.

Please check this.
https://github.com/pstadler/flightplan/issues/117#issuecomment-178592529
PRs are welcome, as I don't have much time to working on open source projects currently.

As an FYI (and shameless plugging), flightplan-sshconf provides support for reading from SSH config files to fill unpassed connection options.


docs: grammatical updates

A bunch of non-substantive updates to the documentation to improve readability and flow.

Bad merge, full of typos


Add support for flush interval

The bulk helper supports a flushInterval option as well. It would be nice to support it!

I'd like to take this if possible


theme closure not working with combination of default and custom colors/styles

note the differences in lines 288 to 290 in this working version from the gist which doesn't mix default and custom colors
https://gist.github.com/e7183320922c275869c1b51ccb7363c3
and now it breaks if you mix them i.e. make it default + custom colors in lines 288 to 290
      fill: theme => ( {
        'primary': theme( 'colors.blue.500' ),
        'secondary': theme( 'colors.blue.900' ),
        'ternary': theme( 'colors.grey-900' ),
      } ),

Once you need object dot notation, and once the key is a string. Haven't had time to check theme yet but I'd guess this is where it fails.

Not sure what the problem is here, I copied the config file from the gist and everything compiles as expected.
Switching to mixed dot/dash notation also compiles, although the colors are different which is what I'd expect.


Suppress improper_ctypes_definitions for compiler_builtins

It causes a lot of lint warnings, we should suppress for now as it's an upstream crate.

@JohnTitor: no appropriate reviewer found, use r? to override


Use futures::TryStream for eventually::AggregateExt and eventually::ReadStore

The current implementation of eventually::ReadStore returns a futures::Stream object on stream() method.
However, futures::Stream assumes the stream of Future objects can't fail, and the futures are always successfully returned (when ready).
This is, of course, not the case when the store is, instead, streaming data from an external data store (e.g. database): the network can fail at any time.
Such failure should and must be represented in the code, for:

eventually::AggregateExt::async_fold
eventually::ReadStore
and, as a result, eventually::command::dispatcher::Dispatcher

Since the only supported event store implementation thus far is in-memory, using futures::Stream doesn't cause an issue.
But in order to supply support for other event store technologies (Redis, Postgres, etc.) we need to solve this issue.

Partially solved in #29


fix(filterFilter): do not filter objects properly in case a property is an array

In JavaScript, an array is a special type of object, therefore typeof [] returns object.
Added corresponding unit tests.
Changed condition for array type to isArray.

Looks correct to me, but could you please fix the indentation in your tests? 2 space indentation

LGTM, just waiting on travis


Hide live streams / don't crash on claims without files #1176

The latest SDK has a parameter to include only claims with files. We'll need to pass that everywhere to hide live stream claims until we are ready to handle them.
If someone does open the claim, don't crash, and maybe show a hyperlink to odysee.com via browser for now.
There is a breaking change in it for connection status, see release notes.

#59 will fix this and is in progress, so I'm just going to close this


[Compute] az vmss create/update: Add new parameters --v-cpus-available and --v-cpus-per-core to support VMSize customization

Description

VMSS VM Customization Properties https://github.com/Azure/azure-cli/issues/20786
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Compute


Make tempo field optional

The field should also be ignored while "linking" notes (both during integrity check and after adding), unless more than one candidate is suitable, in which case the nearest value should be chosen.

Tested:

tempo fields is optional while creating interval from MI2A
error with tempo field is not displayed on integrity check if interval was added from AnkiDroid without tempo
tempo field is ignored while "linking" notes
if more than one candidate is suitable, in this case the nearest value is chosen


Add support for soft sector skew

Some platforms, like the Apple II and CP/M, tend to use soft sector skew where the physical sectors get mapped to a logical sector via a table which isn't on the disk. FluxEngine needs to be told what that table is before it can do anything about it.

It's backwards. Here's a mapping that worked for me:
comment: 'Applies standard AppleDOS soft sector skew to the Apple II format'
is_extension: true

sector_mapping {
	trackdata {
		mapping { physical: 0  logical: 0  }
		mapping { physical: 1  logical: 7  }
		mapping { physical: 2  logical: 14 }
		mapping { physical: 3  logical: 6  }
		mapping { physical: 4  logical: 13 }
		mapping { physical: 5  logical: 5  }
		mapping { physical: 6  logical: 12 }
		mapping { physical: 7  logical: 4  }
		mapping { physical: 8  logical: 11 }
		mapping { physical: 9  logical: 3  }
		mapping { physical: 10 logical: 10 }
		mapping { physical: 11 logical: 2  }
		mapping { physical: 12 logical: 9  }
		mapping { physical: 13 logical: 1  }
		mapping { physical: 14 logical: 8  }
		mapping { physical: 15 logical: 15 }
	}
}

This, together with #489, let me get a complete working Apple DOS 3.3 image that is usable in mame.
Note that for best use in emulators, it would be nice to be able to use the appledos sector mapping when the .do extension is specified and the prodos sector mapping when the .po extension is specified. Is this feasible to implement?
I've prepared a pull request that updates the mapping, which I see you grabbed already while I was writing this up. yay. Thank you!

Thanks for doing this!
Changing the mapping by extension would be annoying --- not hard, but it'd need two bespoke imagereader/writer subclasses which automatically apply the mapping files. Doable but annoying...


No rules_cc - master.zip file not found 404 error

System information (Please provide as much relevant information as possible)

OS Platform and Distribution (e.g. Linux Ubuntu 16.04, Android 11, iOS 14.4): ubuntu~18.04
Compiler version (e.g. gcc/g++ 8 /Apple clang version 12.0.0): gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)
Programming Language and version ( e.g. C++ 14, Python 3.6, Java ): c++
Installed using virtualenv? pip? Conda? (if python): Pip
MediaPipe version: v0.8.5
Bazel version: bazel 3.7.2
OpenCV version (if running on desktop):  4.5.2

Command
bazel build -c opt --copt -DMESA_EGL_NO_X11_HEADERS --copt -DEGL_NO_X11 mediapipe/examples/desktop/hand_tracking:hand_tracking_gpu
Error Message or log
WARNING: Download from https://github.com/bazelbuild/rules_cc/archive/master.zip failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 404 Not Found
ERROR: An error occurred during the fetch of repository 'rules_cc':
Traceback (most recent call last)

thats probably because you dont have the latest pull.
They made few changes, including the WORKSPACE file. So, look at the commits and either make those changes to your local workspace file or just take a pull.
and also, run this command as well. bazel clean --expunge

Now i'm getting following error
ERROR: /home/Downloads/mediapipe/mediapipe/calculators/tensor/BUILD:627:11: C++ compilation of rule '//mediapipe/calculators/tensor:image_to_tensor_converter_gl_buffer' failed (Exit 1): gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG -ffunction-sections ... (remaining 67 argument(s) skipped)
tried replacing https://github.com/google/mediapipe/issues/1673#issuecomment-797863638 same error.

i am not sure about this error. it'd be better if u pull latest release, and then try again.

i have tried latest version v0.8.5 , v0.8.4  and also master build, i see the same error from all the versions. Do you think i'm missing any installation that is missing in the install.md file.

Hi @shivSD, Its because your version of GCC is too old. Have a look at this  https://github.com/google/mediapipe/issues/1684#issuecomment-794590159

Because https://github.com/bazelbuild/rules_cc just changed its master branch name to main.
This problem literally made all the mediapipe versions before v0.8.5 (included) useless.
@sgowroji you need to lunch a new version ASAP.

@zwfcrazy, Thank you for the update. We will update you soon.

Hi @sgowroji I'm working on GCC 7.5 version which is latest part of the ubuntu 18.04. Please can you recommend the gcc version.
Compiler version (e.g. gcc/g++ 8 /Apple clang version 12.0.0): gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)

modify the WORKSPACE under the mediapipe root folder.
urls = ["https://github.com/bazelbuild/rules_cc/archive/master.zip"], -> urls = ["https://github.com/bazelbuild/rules_cc/archive/main.zip"]


Write CSRD processor to parse first datapoint from CSRD reports

This is the issue to create our first processor component in our architecture, that processes CSRD documents linked in carbon.txt files.
It's the CSRD processor linked in the picture below, and in this google slide deck of diagrams

Acceptance Criteria

[ ] We have a processor that is able to read CSRD documents passed into it, and return a data structure saying if the desired datapoints exist.
[ ] the processor is called at the appropriate part of the validatino lifecycle, using the pre-defined plugin hooks (see the plugins hooks mentiond in issue #41)
[ ] We have sensible, safe errors returned for common failure modes - i.e. no file, unparsable document, no datapoints and no data, datapoint but no meaningful data.

Link back to trello card - https://trello.com/c/7ku5gdTs/467-write-processor-to-validate-first-datapoint-in-csrd-documents


Updated index file

Updated link in index.yml.
Updated metadata.

Docs Build status updates of commit 9b0d492:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/index.yml
:white_check_mark:Succeeded
View

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

#sign-off and merged.


Still supported?

Error at line 55 of framework/model/connect/DBConnector.php
On installation and clicking Create for a new connector.

@asecondwill sorry, but we're currently not actively using the module on any of our projects, so we're not able to spend time on it - pull requests are of course welcome.

ok, thanks for the info.


Bug:  After withdrawing - Back button misfires

After withdrawing the application when I go back to view the application it takes me to the Review Page.  From the review page as I hit the back button I can get up to the Organization Profile page but clicking back at this page misfires, and keeps me on the page

Can't reproduce this exactly, but I have a similar behaviour, where the "Back" button takes me back and forth between pages 16 and 17

Additional detail: If your application is withdrawn, and you click on "Organization Location" in the stepper, then it moves the form 1 page forward

I have the same issue as AntBush, the stepper does not help with this bug.  The bug exists for received applications too i.e. submitted applications after the intake is closed


TypeError: unbound method init() must be called with RecordingHTTPConnection instance as first argument (got VerifiedHTTPSConnection instance instead)

Attempting to archive this group as a non-owner, I get a bunch of errors which ultimately boil down to:
2019-11-07 17:45:21.301 CST INFO requests.packages.urllib3.connectionpool Starting new HTTPS connection (4): groups.yahoo.com
Traceback (most recent call last):
  File "./yahoo.py", line 718, in <module>
    archive_db(yga)
  File "./yahoo.py", line 310, in archive_db
    db_json = yga.database()
  File "/Users/brettneese/Development/git/github.com/IgnoredAmbience/yahoo-group-archiver/yahoogroupsapi.py", line 156, in get_json
    r = self.s.get(uri, params=opts, verify=VERIFY_HTTPS, allow_redirects=False, timeout=15)
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/sessions.py", line 395, in get
    return self.request('GET', url, **kwargs)
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/sessions.py", line 383, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/sessions.py", line 486, in send
    r = adapter.send(request, **kwargs)
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/adapters.py", line 334, in send
    timeout=timeout
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/packages/urllib3/connectionpool.py", line 475, in urlopen
    conn = self._get_conn(timeout=pool_timeout)
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/packages/urllib3/connectionpool.py", line 217, in _get_conn
    return conn or self._new_conn()
  File "/Users/brettneese/Library/Python/2.7/lib/python/site-packages/requests/packages/urllib3/connectionpool.py", line 654, in _new_conn
    **extra_params)
  File<EMAIL_ADDRESS>line 1243, in __init__
    source_address)
TypeError: unbound method __init__() must be called with RecordingHTTPConnection instance as first argument (got VerifiedHTTPSConnection instance instead)

Looks like my Python version is 2.7.16 on OSX, and I double checked that my cookies were in order.

I was getting this same error with python2.7 on Ubuntu. When I tried python3.4 instead the error went away but unfortunately I got a different error.

Unfortunately, I'm unable to reproduce this, and unsure what the cause of the issue could be.

@IgnoredAmbience no worries! I will try a few different python versions to see if I can get it to work

FYI moving from 2.7 to 3.4 on Linux Mint 17.3 fixed the issue for me and I did not hit any further errors.

Can also confirm that moving to a 3.x version solved the problem and I was able to archive the group I was trying to! Closing the issue, anyone who needs it can find it via search. Thanks all!


Rebalancing SetToken Refactor [WIP]

Pull Request Test Coverage Report for Build 6322

171 of 171 (100.0%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 99.865%

Totals

Change from base Build 6287:
-0.1%

Covered Lines:
1072

Relevant Lines:
1072

💛  - Coveralls


Handle Middleware if user not login

Hello, I using \Bican\Roles\Middleware\VerifyRole::class, and got a problem about if user not login I want to redirect to auth/login but this class is alway throw to RoleDeniedException.
So best way to handle this thing is I should use middleware like this.
'middleware' => ['auth', 'role:admin'],

that right ?
Thanks.

@dogrocker Then in app/Exceptions/Handler.php, you would do:
/**
 * Render an exception into an HTTP response.
 *
 * @param  \Illuminate\Http\Request  $request
 * @param  \Exception  $e
 * @return \Illuminate\Http\Response
 */
public function render($request, Exception $e)
{
    if ($e instanceof \Bican\Roles\Exceptions\RoleDeniedException) {
        // you can for example flash message, redirect...
        return redirect('auth/login');
    }

    return parent::render($request, $e);
}

So best way to handle this thing is I should use middleware like this.

'middleware' => ['auth', 'role:admin'],

Yes, you would apply this to your route. However, if you are referring to a resource route, see here.


Change QC::Queue interface to support setting the conn adapter in the constructor.

Fixes #249

this makes a breaking change to the constructor. What about using top_bound as means to pass both the actual top_bound or a options hash (possibly containing the key :top_bound). We can then print a deprecation warning for non-Hash arguments.
Also I think we should deprecate conn_adapter= on the Queue. As I see it, it's trivial to just spin up a new queue instance when you want to use a different adapter.

I love @senny's approach here.

Closing as it's old, plus the original idea in #249 seems bette


Does this support caching GET requests?

Curious if this is meant to only handle the actually http requests, or if it handles caching.
If not, is there a way to hook into the http requests to intercept and write a caching plugin for it?

Hi,
I want to cache particular request but it should get call once its parameter gets update.
I am using restangular cache is working for me but once my parameter change request does not get call.


try to fix classloading issues on runLocal and test with java 9

fixes #251

@lihaoyi @robby-phd can any of you review this PR?

Looks good to me.

Looks good to me; just update the comments and fill in the PR description with a proper explanation of what's going on, then feel free to merge


RSS for blog

Not sure what blogging software is being used, it's homebrewed, right? It would really be nice if it had RSS :)

medium now!


Prevent simultaneous editing of comments and issues

fixes #22907
Tested:

[x] issue content edit
[x] issue content change tasklist
[x] pull request content edit
[x] pull request change tasklist

Thank you for the feature! It is good for end users. However there are some blocking problems in my mind at the moment:

I do not think it's right to use XORM "version" here. Otherwise, all issue-model code needs to call NoVersionCheck again and again, and it might cause more breakings.
The comment content field is the same as issue content field, but comments are stored in another table.

Should we have a complete design before only "fixing" the issue?

The "version" is quite unclear and would mislead developers.

I can see the "version" is only for "content"
But there are still other fields like "title", which can be changed simultaneously
And there are still other "related models" like "labels", "assignees" can be changed simultaneously
So I do not think we should use "version" for "issue content"

@wxiaoguang thank you for review. do you have any suggestion? maybe a 'content hash' colum as @drsybren said? or any generic solution?

TBH I haven't thought about the details carefully.
Some brief ideas (just my opinion):

Use ContentVersion, add it for both issue and comment tables, and this field is only used for "content editing".
Re-use the "issue content history", use the last issue content history record as reference.
Hash (sha256) the issue/comment's content as reference.

For the frontend mechanism: I would strongly suggest to make it independent of backend and totally transparent (do not calculate "version+1" in frontend). It could use an abstracted variable like data-content-version-tag, and frontend code only gets it from backend and sends it back to backend, then frontend doesn't need to share any knowledge with backend, and all of these "ideas" above work with this mechanism.

Thank you for the feature! It is good for end users. However there are some blocking problems in my mind at the moment:
1. I do not think it's right to use XORM "version" here. Otherwise, all issue-model code needs to call NoVersionCheck again and again, and it might cause more breakings. (it is also related to point 3)

2. The comment content field is the same as issue content field, but comments are stored in another table.

   * Should we have a complete design before only "fixing" the problem for "issue"?

3. The "version" is quite unclear and would mislead developers.

   * I can see the "version" is only designed for "content"
   * But there are still other fields like "title", which can be changed simultaneously
   * And there are still other "related models" like "labels", "assignees" can be changed simultaneously
   * So I do not think we should use "version" for "issue content"

I would agree since only content has the version, we should use a name like content_version column but not version. And looks like XORM based version will require careful changes in other places. Maybe just revert back to user-defined version. Sorry, @metiftikci .
But I don't think it's necessary to put comment content version changes in the same PR. Another PRs are better. Because most of the edit box needs column like this. i.e. comment, release note, user information, org information and etc.

@wxiaoguang changed Version to ContentVersion as int. Do i need to do something else? frontend gets new version number from backend

Overall it looks good to me. Thank you for the updates.
Actually, I still think we should make the "comment" also support "content version" in one PR: they share the same code, and I do not think it's worth to add another migration for "comment" table.
And one more thing, I think it's worth to write some test code to cover ChangeIssueContent's behavior.

@wxiaoguang added comment content version and some test but i am not familiar. waiting for any change request or suggestions

@lunny i think fail is not related to pr, right ?

@lunny i think fail is not related to pr, right ?

I think so. Let me restart the failed jobs.

Looks like the CI failures are related.

My bad. fixed migration. now looks like fixed

(ps: no need to merge with every main commit manually, maintainers and the Gitea bot will do that 😄 )


Broken leg

Manhattan Cafe: My leg hurts.
Game version: 1.18.2
Mod version: umapyoi 1.0.18

Fix on road, stand by.


Suggestion: store key in URL fragment rather than param

Hi! Pretty cool project you've got here, thanks for sharing!
In commit ec8d20473e98302dc6ca3b758d8fcb3506db9a02 you added a feature that allows the encryption key to be embedded in the shared URL. As a suggestion to keep the E2E encryption safe, I think it would be better to move it into the fragment (window.location.hash) rather than a query parameter. This is because query parameters are sent to the request made to the web server - which defeats the point of E2E encryption - but hash/fragment identifiers are not.
Thanks!

Hi,
That is a good catch, I totally missed that! Thanks for creating the issue.
I created the PR #12  to fix it, which I will merge soon and then update the webserver as well as the client. Just need to check for backward compatibility since people with the old client will still get the ?key=123 query parameter :sweat_smile:

Should be fixed now, e.g. https://cli2cloud.com/8TGxG8#key=123


Use Release tags instead of plain tags to mark new versions

Dear @cleydyr, can you please use the Release feature of GitHub to mark new versions? That would enable users to receive notifications upon new releases:

Done. Published the first GH release v1.4.0.

Great. Thanks a lot! :-)


Uploading thousands of files at the same time

If I upload thousands of files at the same time, I am having an issue where some files dont end up in s3 bucket but slignshot responds with success. Why does it happen? Does it have to do with the signature being generated at the same time due to multiple pararel uploads?

What is your use case here? Are you doing any kind of throttling/error checking as you go?


Make maya-apiservice a configurable value

FEATURE REQUEST
The openebs provisioner assumes that the maya-apiserver runs within in the same cluster and its service name is "maya-apiserver-service". This hardcoding of the name could conflict with other services running in the cluster.
https://github.com/kubernetes-incubator/external-storage/blob/2fd77da/openebs/pkg/v1/maya_api.go#L65
Expected change:
(a) Provide the service name as an ENV variable to the provisioner.
The other alternatives could be:
(b) make the maya-apiserver details part of the storageclass
(c)  pass the maya-apiserver URL as env (so that provisioner can connect to an external maya-apiserver)

@kmova having Maya API server URL in Storage Class is good to have. This will enable use of multiple apiservers in a cluster or talking to a apiservers across different cluster. However even labelselector  can be used in Storage Class to talk to a particular maya API server within a cluster. Label selector seems to be kubernetes way


[Refactor] scheduler rolling batch refactor

Description
Testing

tested rolling_batch_integration.yml in my ec2 machine
Added a unit test case for scheduler rolling batch

@KexinFeng


Fix: pass env vars to Docker

What it solves
Hosting safe-react in a container where the build process creates static files it needs a way to pass ENV vars into the Dockerfile.
How this PR fixes it
ARGs in Dockerfile.
All sorts of API keys seem to be exposed or it allows it to be exposed (like the infura API key) which I don't think should be used this way - I'm not much of a frontend dev tho so does it get obfuscated somehow?
export const getRpcServiceUrl = (): string => {
  const { rpcUri } = getChainInfo()
  return formatRpcServiceUrl(rpcUri, INFURA_TOKEN)
}

How to test it
Screenshots

I have read the CLA Document and I hereby sign the CLA

recheckcla

I'm not much of a frontend dev tho so does it get obfuscated somehow?

Nothing in the browser is private. The Infura key can be easily seen in the network requests.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
9 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 32.072%

Files with Coverage Reduction
New Missed Lines
%

src/components/Root/LegacyRouteRedirection.tsx
9
29.17%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
3071

Relevant Lines:
8495

💛  - Coveralls

I'm not much of a frontend dev tho so does it get obfuscated somehow?

Nothing in the browser is private. The Infura key can be easily seen in the network requests.

Is that an issue that's tracked somewhere? I guess all requests should be handled by the gateway?

This hasn't been tracked. Good point though. I know Infura actually allows to lock the key to whitelisted origins.
I'll make a ticket for our infra devs (it's not public).

Hi @InoMurko If you want to set a env variable you can create a .env file:
 cp .env.example .env

you can add the values in the .env and then execute the build
docker-compose build

and run the project
docker-compose start

If you want to run the application locally, this should be sufficient. Let me know if this works for you or if you want to do something different

Hi @InoMurko If you want to set a env variable you can create a .env file:

 cp .env.example .env

you can add the values in the .env and then execute the build

docker-compose build

and run the project

docker-compose start

If you want to run the application locally, this should be sufficient. Let me know if this works for you or if you want to do something different

In my experience this does not work. Env variables need to be present at build and afaik, there's no other way to present ENV vars to the build process. Your approach only pushes ENV vars to the runtime - which is already too late for the static files.

In my experience this does not work. Env variables need to be present at build and afaik, there's no other way to present ENV vars to the build process. Your approach only pushes ENV vars to the runtime - which is already too late for the static files.

Hi @InoMurko we provide a way to execute the app locally using docker because in the past we had some issues with the new M1 Mac devices. We prefer to avoid specifying the env values in another file different from the .env.example file (in case we add / remove a var env we can forget to update the dockerfile as well )
I tested it in Windows and in my M1 Mac and it seems that for me the env vars are with the correct values:
I create the .env with my custom env variables, I execute docker-compose build and docker-compose start and the app launches in the http://localhost:3000 with the custom values used in the .env file.
Could you please provide more context on what steps you are running? because my assumption is that you only want to run the application locally with custom env vars.

@DaniSomoza absolutely correct. I think I was somehow overwritting my env file and hence, didn't see the changes I was trying to apply.


Edge artifacts

Allow option for removal of the beginning and end of the signals based on filter size

Should that take place in the filterfn not in the pacfn?

right

see comments in issue #20


ci: lint and test

Lint markdown and yaml files for syntax errors
Test templates against non-default configurations for correct syntax
Update readmes to reflect latest changes in release
Separate testing and release workflow, make workflow callable

Signed-off-by: Moritz Röhrich<EMAIL_ADDRESS>Describe your changes
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.
[x] CHANGELOG.md has been updated should there be relevant changes in this PR.

what is the target release for this PR?


connecttimeout and readtimeout will be reset when client has any configuration

Describe the bug
After the bug (Read timeout from application properties is not used when client is configured  ) was fixed in v2.2.4.RELEASE, connect timeout and readtimeout of LoadBalancerFeignClient cannot work when client has any configuration.
Sample
When feign client not config, the readtimeout and connecttimeout load by IClientConfig is 1000ms / 1000ms (default, or configed to others ) ( see LoadBalancerFeignClient#L97)
//LoadBalancerFeignClient#L97
IClientConfig getClientConfig(Request.Options options, String clientName) {
		IClientConfig requestConfig;
		if (options == DEFAULT_OPTIONS) {
			requestConfig = this.clientFactory.getClientConfig(clientName);
		}
		else {
			requestConfig = new FeignOptionsClientConfig(options);
		}
		return requestConfig;
	}

and
//FeignClientFactoryBean with bug 323:
if (config.getConnectTimeout() != null && config.getReadTimeout() != null) {
			builder.options(new Request.Options(config.getConnectTimeout(),
					config.getReadTimeout()));
		}

It was worked before bug 323 fixed in FeignClientFactoryBean.java. But now when client has any configuration,  readtimeout and connecttimeout (1000ms/1000ms) will be reset to 10000ms and 60000ms. For example, put a Logger leave in properties:
feign.client.config.default.loggerLevel=FULL

Hello, @izdt  2.2.x is no longer supported. Could you please verify if the issue can be reproduced on a supported version (3.1.4, Spring Cloud dependencies 2021.0.4) and let me know here?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Hello, @izdt 2.2.x is no longer supported. Could you please verify if the issue can be reproduced on a supported version (3.1.4, Spring Cloud dependencies 2021.0.4) and let me know here?
@OlgaMaciaszek
It occurs when use org.springframework.cloud.openfeign.ribbon.FeignLoadbalancer . Since ribbon has removed from spring-cloud-openfeign in version 3.x , the bug will not be trigged. I verified version 3.1.4 , this bug does not exist anymore.


Add Rust on Espressif discovery journey

Hi! Since we at Espressif have something similar to the discovery books, but in a different format, I thought that it might be worth mentioning the resources in the discovery book.
If you want to keep this for the current content only, feel free to close the PR. Also, not sure if the format in which I added it, is the best, so also open to any suggestions!

This feels a bit out of place to me. I think a reference like this is more fit for an entry in one of the areweXyet websites (which I don't thing exists for embedded, but should)

Wow, this got closed pretty fast. I would still advocate for including a reference in the Discovery Book for the following reasons:

At RustNL we discussed to cover sync and asnyc embedded Rust in the discovery book, so at least referring to the "second dimension" of of embedded Rust - no_std and std - feels natural to me
Especially the RISC-V devices offer a comfortable way of getting into embedded Rust and they are well supported and easily accessible targets
Awesome Embedded Rust is really broad and overwhelming for "young players" and I from my perspective these two points (std + sync, std + async) in the design space could and should be mentioned in the Discovery Book as well
Training resources are already available

Documentation and the Discovery Book is currently under discussion. What about discussing the if, how and where there as well?

I guess Ill reopen it for others to jump in an give an opinion

I think adding an "Alternatively" or similar word might be an improvement.

I'm happy to use another wording, what about "Alternative Discovery Books"?

Any input here? @eldruin @sirhcel happy to make any changes if required

Please give us a bit more time to transition to the newer discovery-mb2 book; we're still sorting out exactly how we want to handle other "Discoveries".


4.0.0 Version crashes for some instances (Windows 10)

See here:
timetable_73MB.zip

Smaller instances are working fine.

@sfiruch:

I think it'd make sense to discuss Windows-specific bugs in the repo of the fork. Also, can you test whether disabling shrinking helps?

How do I disable it?

if it is shrink=0, then it dows not work

With verbose=3 one can see this:

It seems to work with
--congruence=false

It also works with:
--congruencebinaries=false

Thanks for reporting this. I provided updated binaries, which should solve the problem.
Root cause is/was that the Radix sort does stack allocation, which is limited to 1MB by default (which is a sensible limit). I changed the code to do heap allocations instead. In my experiments I didn't measure any negative performance effects.


[infra] NDK install fails on MacOS GitHub runner

Run nttld/setup-ndk@6a7e9c5494a25c4842ca1011f710300132eacd94
  with:
    ndk-version: r[2](https://github.com/dart-lang/native/actions/runs/8906568850/job/24458937965?pr=1114#step:4:2)6b
    add-to-path: true
    link-to-sdk: false
    local-cache: false
  env:
    DART_HOME: /Users/runner/hostedtoolcache/dart/[3](https://github.com/dart-lang/native/actions/runs/8906568850/job/24458937965?pr=1114#step:4:3).3.4/arm64
    PUB_CACHE: /Users/runner/.pub-cache
Error: Error: Unsupported arch "arm6[4](https://github.com/dart-lang/native/actions/runs/8906568850/job/24458937965?pr=1114#step:4:4)"

I believe the MacOS bots were x64 before. It looks like they silently switched to arm64.
We run the arm64 MacOS bot less often because it wasn't free:

https://github.com/dart-lang/native/pull/847

We should check if its free now.
Also, we probably want to keep running things on x64 MacOS.

Yep, the default is arm64 now: https://github.com/actions/runner-images/issues/9741

Tracking issue for the GitHub action we're using to install the NDK:

https://github.com/nttld/setup-ndk/issues/548

However, an NDK should already be pre-installed on the bot:

https://github.com/actions/runner-images/blob/main/images/macos/macos-14-arm64-Readme.md

Hello! As stated in the table the only intel-based macOS-14 runner is macos-14-large (doc). There are no plans to introduce macOS-14 intel standard runners as of now.

https://github.com/actions/runner-images/issues/9741#issuecomment-2075259811
@liamappelbe Maybe we should then change the weekly builder to be an x64 one? macos-14-large

It's fixed https://github.com/nttld/setup-ndk/pull/549

The GitHub bot already has an NDK pre-installed, so we're using that one now.
https://github.com/dart-lang/native/pull/1114/files#diff-67b78d3322afa689c9b62e854680e8087184f61dd7787f43ceb53f471eef6704
We can get back to installing one manually if we want a specific version of the SDK in the future.


Spike - Measuring Theming Implementation

To successfully achieve our goal of launching Dual Branding (theming) across ecommerce digital experiences, we need to follow two strategies: 1. New development utilizes semantic theming tokens and 2. Legacy development is upgraded to semantic theming tokens.
This Spike is important as it is a first step towards measuring the efficacy of strategy 1 + 2, and allows us to measure our progress towards being 100% theme ready. If we aren't able to measure how pervasive our tokens are implemented, we aren't able to coordinate and plan our roadmap to Dual Brand live and report progress to leadership.
Def of Done:

Time-boxed spike to ~3 points of effort
2 Code repositories are evaluated (Content and Flight Search)
Measurement approach is documented
Results of measurement are documented
Tradeoffs and accuracy of measurement are documented

We need to get the chrome extension before we can start on this.


fix: Ensure JSON response is properly parsed in generate function when format is set to 'json'

When I set format as "json", my response looks like this:
{'model': 'llama3', 'created_at': '2024-06-28T05:44:32.12715Z', 'response': '{"results": [\n{\n"entity_type": "IMEI",\n"text": "06-184755-866851-3"\n}\n]}\n\n \n\n ', 'done': True, 'done_reason': 'stop', 'context': [], 'total_duration':<PHONE_NUMBER>, 'load_duration': 2316500, 'prompt_eval_count': 437, 'prompt_eval_duration':<PHONE_NUMBER>, 'eval_count': 32, 'eval_duration':<PHONE_NUMBER>}
As you can notice, although it indeed returns a JSON object, the previous code parses the 'response' as a string (hence the presence of \n characters and type checking result).
After I added some code to correctly parse the JSON response, the response now looks like this:
{'model': 'llama3', 'created_at': '2024-06-28T09:16:56.417739Z', 'response': {'results': [{'entity_type': 'IMEI', 'text': '06-184755-866851-3'}]}, 'done': True, 'done_reason': 'stop', 'context':
It becomes cleaner and matches my expected format.

We should have better serialization with Pydantic now. If you still run into this after the new release feel free to re-open this!


data-drag-delay wrong behavior.

I set "data-drag-delay" to 500. When i fast drag and drop node not in tree area, node keep holding under cursor.

Closing this issue due to inactivity. See #815 for more information.
Please create a new issue is this still occurs with the current version of the component.


求 大佬 适配 MiniCPM-2B，感谢

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
求 大佬 适配 MiniCPM-2B，感谢 。请问近期有无计划
Expected behavior
No response
System Info
No response
Others
No response

mark

--template cpm


[Bug} File size shown in formats window doesn't match yt-dlp output

Example:
https://www.youtube.com/watch?v=DfLW4pdvn-s
The ytdlp-interface format window shows video format 315 has file size of 2.27 GB. But running command line of:
yt-dlp --list-formats "https://www.youtube.com/watch?v=DfLW4pdvn-s"
shows video format 315 has file size of 6.27 GB... which is the correct file size of the downloaded file. This is just one example but I have seen similar discrepancies with other youtube videos.

Thanks for reporting this, I've identified the bug and I'm working on a fix.


datasets

Hello, is there any other way to download the preprocessed dataset?

Hi @kingkingofall,
Yes, the preprocess files can be downloaded using the S3 URI's provided in README.
Please let me know if you have any additional questions!


Fix phone number constraint

Initially, phone number can take any values of numbers more than 3 digits long.
Now, we restrict phone numbers to be between 3-10 digits to ensure that only valid phone numbers are keyed in.
Closes #121.

UserGuide is updated


When I load up visual studio it says: Updates are disabled because you are running the user-scope installation of Visual Studio Code as Administrator, BUT IM NOT!

Type: Performance Issue
I just want to be able to update visual studio code without having to delete and reinstall it.
VS Code version: Code 1.86.0 (05047486b6df5eb8d44b2ecd70ea3bdf775fd937, 2024-01-31T10:28:19.990Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
12th Gen Intel(R) Core(TM) i9-12900H (20 x 2918)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
63.77GB (53.33GB free)

Process Argv
--crash-reporter-id 3d14ffc6-2af6-4956-b22e-487ee3d259ee

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   102	 12500	code main
    0	    78	  2760	fileWatcher [1]
    0	    40	  5244	   utility-network-service
    0	   174	  6380	window [1] (Visual Studio Code [Administrator])
    0	   157	 11996	   gpu-process
    0	    28	 13240	   crashpad-handler
    0	   100	 19164	shared-process
    0	    98	 19596	window [2] (Issue Reporter)
    0	   183	 21340	extensionHost [1]
    0	    86	 22748	     "C:\Users\Administrator\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\Administrator\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=21340

Workspace Info
;

Extensions (14)

Extension
Author (truncated)
Version

better-comments
aar
3.0.2

bootstrap5-vscode
Anb
0.4.4

vscode-tailwindcss
bra
0.10.5

path-intellisense
chr
2.8.5

prettier-vscode
esb
10.4.0

copilot
Git
1.184.0

copilot-chat
Git
0.12.2

bootstrap5-snippets
Han
1.2.5

vscode-kubernetes-tools
ms-
1.3.16

vscode-yaml
red
1.14.0

LiveServer
rit
5.7.9

polaris-for-vscode
Sho
0.9.8

open-in-browser
tec
2.0.0

vscode-icons
vsc
12.7.0

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
welcomedialog:30910333
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
pythontbext0:30879054
accentitlementsc:30995553
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
fegfb526:30981948
bg6jg535:30979843
ccp1r6:30993540
dsvsc020:30976470
pythonait:31006305
gee8j676:31009558
chatpanelc:31018788
dsvsc021:30996838
da93g388:31013173
pythoncenvpt:31022790

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.88.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

uhhh is it because its app data is inside of my administrator file and not the user program file directory on the local storage.

Also thats so weird like this is my computer why can I not run it as administrator and still download apps?

See https://github.com/microsoft/vscode/issues/157624#issuecomment-1222300708

I understand that I’m using the older version that’s not what I asked.
On Sun, Apr 28, 2024 at 1:54 PM VSCode Triage Bot @.***>
wrote:

Thanks for creating this issue! It looks like you may be using an old
version of VS Code, the latest stable release is 1.88.1. Please try
upgrading to the latest version and checking whether this issue remains.
Happy Coding!
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/211585#issuecomment-2081633058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFZQL3NZOQ4CZ5XTTEJXT5DY7VHW3AVCNFSM6AAAAABG5HMODGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBRGYZTGMBVHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I've linked explanation why you cannot update user setup when launched as admin not sure why you reply to old standard bot's warning.
Check in Task Manager Details tab if it is runned as elevated user (you may need to enable hidden column Elevated)

This did not populate into the email I recieved I do apologize thank you so much! Happy coding!


Manual Input/Auto parameters not working after updating

Hadn't pulled a new build of the mp4 automator in a year, figured it was time to grab a new one, and also update to Python 3+ instead of 2.7 that I'd been running, and noticed that when running manual.py with input and auto parameters, it's still asking for path and confirmation for guessit to tag the file.
I tried uninstalling python entirely (all versions), cleaned up their folders, deleted the mp4 automator folder and reinstalled everything, but that didn't help. Anyone who can help me figure out what's wrong?
Manual.py -i .\Downloads -a
Manual processor started.
Python 64-bit 3.8.3 (tags/v3.8.3:6f8c832, May 13 2020, 22:37:02) [MSC v.1924 64 bit (AMD64)].
Guessit version: 3.1.1.
C:\Python38\python.exe
Loading config file C:\sickbeard_mp4_automator\config\autoProcess.ini.

> Enter path to file:
#: .\Downloads
Processing file .\Downloads\Funny.People.2009.mp4
Matched movie title as: Funny People 2009-07-31 (TMDB ID: 20829)
> Proceed using guessed identification from filename?
# [y/n]: y

It actually looks like no python arguments are being passed (notice the input failed too). I suspect this is a python problem that happens with windows sometimes.
https://stackoverflow.com/questions/2640971/windows-is-not-passing-command-line-arguments-to-python-programs-executed-from-t
Check and see if any of those solutions fix things

Thanks mdhiggins, had to add %* to HKEY_CLASSES_ROOT\Applications\python.exe\shell\open\command and it started working.


XML support removed without proper replacement

XML support was removed but only replacement seems to be manual adding of tags.
Why it bothers us?
Our projects contains tens of macros for rendering components etc. I cannot ask every coleague current and new to click out all the macros and filters.
Not just it is tedious but also can lead to missclick if it is paired or unpaired macro and fixing means going to settings.
Removal of XML also closes door to autocompleting parametrs of macros which XML supported but plugin not yet implemented.
As far as i know, even the pro version doesnt offer replacement.
Can you please explain the decission to remove XML properly? Thanks.

XML configuration was removed because it was bad implemented here and there were problems with it, it was main reason. And I also see it as a feature for teams and therefore one that should only be in the Pro version.
Pro version offers replacement: https://mesour-intellij-plugins.github.io/latte.html#xml-config and it has more options. And here https://mesour-intellij-plugins.github.io/latte.html#migration is also migration guide where is described what has been changed compared to the free version.


[CSM] Disable IMDS from drift events by default

What does this PR do?

This PR disables IMDS events from drift events by default (but it leaves the dump configuration untouched).
Motivation

Let's start small to minimize the impact :)
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Use JSX instead of React DOM factories to fix React warning in >= 15.6

Fixes the following warning in React 15.6 and above:
“Warning: Accessing factories like React.DOM.a has been deprecated and will be removed in v16.0+. Use the react-dom-factories package instead.  Version 1.0 provides a drop-in replacement. For more info, see https://fb.me/react-dom-factories”

Great, thanks alot. Merge and will publish later today.

@fisshy do you think you can publish soon a new release on npm repo 👼 ?
Thx! ❤

New version out @shouze :) v1.5.3

@fisshy thx a lot!


Source Predicates Missing From YAML Config File

While working on #312 (KG2.8.4), the build failed due to missing source predicates:
Source predicate curie is missing from the YAML config file: RO:0002596
Source predicate curie is missing from the YAML config file: RO:0002570
Source predicate curie is missing from the YAML config file: CL:4030045
Source predicate curie is missing from the YAML config file: NCIT:has_dipg_dmg_permissible_value
Source predicate curie is missing from the YAML config file: OBO:HANCESTRO_0330
Source predicate curie is missing from the YAML config file: NCIT:is_pcdc_hl_permissible_value_for_variable
Source predicate curie is missing from the YAML config file: CL:4030044
Source predicate curie is missing from the YAML config file: NCIT:has_pcdc_hl_permissible_value
Source predicate curie is missing from the YAML config file: NCIT:is_dipg_dmg_permissible_value_for_variable
Source predicate curie is missing from the YAML config file: CL:4030046
Source predicate curie is missing from the YAML config file: NCIT:is_pcdc_os_permissible_value_for_variable
Source predicate curie is missing from the YAML config file: OBO:HANCESTRO_0308
Source predicate curie is missing from the YAML config file: OBO:HANCESTRO_0301
Source predicate curie is missing from the YAML config file: DRUGBANK:carrier
Source predicate curie is missing from the YAML config file: NCIT:has_pcdc_os_permissible_value
Source predicate curie is missing from the YAML config file: RO:0002336
There are source predicate curies missing from the yaml config file. Please add them and try again. Exiting.

Since the KG2.8.4pre build finished successfully, I am going to close this issue as completed.


Account inactive?

Details

Project URL:
Build URL (if applicable):
Read the Docs username (if applicable):

Expected Result
A description of what you wanted to happen
Actual Result
When I try to login this morning it says my account is inactive, yesterday I logged in without any problem. What happened?
A description of what actually happened

Thanks @madskinner. Can you fill out the content in the issue template a bit more? Did you have a specific project that you were trying to access or work on?

Project URL:pub2sd2
Build URL (if applicable):
Read the Docs username (if applicable): i assume its either 'madskinner' or email address<EMAIL_ADDRESS>
Expected Result
login and see build status/view docs
A description of what you wanted to happen
login and see build status/view docs
Actual Result
When I try to login this morning it says my account is inactive, yesterday I logged in without any problem. What happened?
A description of what actually happened
When I try to login this morning it says my account is inactive, yesterday I logged in without any problem.


Refactor and Expand UEFI code

Updated the UEFI code to align more with normal boot-code.
Fixed multiple taps for UEFI
Fixed ISO to only be used when using install
Removed duplicate code.
Added sectorsize as setting instead of hardcoding to 512 for UEFI.
Request for feedback, I have most certainly missed or changed something that does not align to the project.

Hi,
There is not Fixed multiple taps for UEFI relevant code in the PR. Is the PR complete? Or have I missed something?
BR, Radek

The pci="$(__guest_prepare $name)" takes care of and returns all tap devices (as well as other standard devices) assigned to the guest. So instead of doing something custom for the UEFI code i reused an already existing function for that exact purpose.
I tested with multiple tap devices and got bhyve -c 2 -H -w -m 8G -s 0,hostbridge -s '1,ahci-hd,/dev/zvol/zroot/iohyve/win10/disk0,sectorsize=512' -s 2,virtio-net,tap5 -s 3,virtio-net,tap6 -s '4,fbuf,tcp=<IP_ADDRESS>:5900,w=800,h=600,wait' -s 5,xhci,tablet -s 31,lpc -u -l com1,/dev/nmdm1A -l bootrom,/iohyve/Firmware/BHYVE_UEFI.fd/BHYVE_UEFI.fd ioh-win10as the bhyve command,

Hi,
You are right. I overlooked this part local pci_args=$(__guest_get_bhyve_cmd "$pci" )
BR, Radek

=) Awesome! Thanks for looking over the code! I appreciate it.
Br
Mikael

Looking good! This will probably be merged soon!
I am looking to boot from ISO even if not installing, as that is the only way to get Linux UEFI guests to boot after installation (and destroying). Booting a "rEFInd" ISO before the zvol fixes this as suggested by @EpiJunkie on the mailing list.
I also heard that this should be fixed sooner than later, so we'll just cross that bridge when we come to it.

Awesome!
You should be able to boot from an attached CD just using the install command, and then use the select boot medium uefi hotkey and select CD.

I'm going to merge, but we might make a few changes. If a ZFS property (like an iohyve property) is not set, it is returned with a value of -, not an empty string. Also, instead of failing, if no sectorsize property is present, we should simply not set one. (This feature is only needed by some older Windows versions).


Some minor improvements

Improve LongLiteralExpressionVisitor
C# 11 supports unsigned right shift
Using logging instead of Console.WriteLine
Improve AddCommentsTrivias
Update Microsoft.CodeAnalysis.CSharp dependency
Add integration tests
Preserve the original folders of the source project in the output folder
Add GitHub action to build the project automatically

This is great, thank you @VahidN!


Automator: update istio/api@master dependency in istio/istio@master

Generated by Automator - 2020-04-09T16:15:39+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
e4692a3d0f163829865164220972e450aa9fddfe
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


cmake v3.20.0,  cmake --build --config Release does not function as required.

Describe the bug
this command does not work properly as before:
cmake --build . -j --config Release -- -m -v:m
this command does work properly:
cmake --build . -j --config Release -- -p:Configuration=Release -m -v:m
There must be something wrong with CMAKE VERSION 3.20.0, I guess.
Environment

OS: Windows
Compiler: msvc 2019

@tom2068, could you report this issue to CMake instead? Or if you use cmake in VS, you can create a feedback tickets to VS.
BTW, I tried it with CMake latest release version 3.20.1, it looks worked.
E:\vcpkg\PhoebeHui\manifest\build>"C:\Program Files\CMake\bin\cmake.exe" --build . --config Release
Microsoft (R) Build Engine version 16.9.0+5e4b48a27 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.
Checking Build System
main.vcxproj -> E:\vcpkg\PhoebeHui\manifest\build\Release\main.exe
Building Custom Rule E:/vcpkg/PhoebeHui/manifest/CMakeLists.txt

PhoebeHui, Thank you very much!


rustfmt swallows comments inside impl ... where

Example:
trait Foo {}

impl<T> Foo for T
// goodbye, cruel world
where T: Copy + Clone {}

rustfmt turns this into:
trait Foo {}

impl<T> Foo for T
where
    T: Copy + Clone,
{
}

$ rustfmt --version
0.9.0-nightly ( )

This also happens with comments after where.