deserializing a slotmap compiled to .wasm causes error.

Slotmap works just fine in web assembly with just one exception. It appears that deserialization with serde does not work. (Serialization works just fine).
I really have no idea if this is a problem with Slotmap, Serde, or wasm_bindgen it really is above my head.
Anyway. this code  ::
        let mut sm: SlotMap<PlayerKey, u32> = SlotMap::with_key();

        sm.insert(3);

        let j = serde_json::to_string(&sm).unwrap(); // serialization works fine

        print!("{}",j );

        let sm2 : SlotMap<PlayerKey, u32> = serde_json::from_str(&j).unwrap();

causes this error  only in a wasm environment ::
`panicked at 'attempt to shift left with overflow', /Users/rebo/.cargo/registry/src/github.com-1ecc6299db9ec823/slotmap-0.3.0/src/normal.rs:1068:31
Stack:
Error
at Module.__wbg_new_59cb74e423758ede (webpack-internal:///./crate/pkg/index.js:547:21)
at __wbg_new_59cb74e423758ede (http://localhost:8000/app.js?56a04bc9da773126182b:801:101)
at console_error_panic_hook::Error::new::hf9155aeb30e0df7c (wasm-function[11765]:22)
at console_error_panic_hook::hook_impl::h19b17601d076c52f (wasm-function[2120]:100)
at console_error_panic_hook::hook::hd8298b7121f5bbfe (wasm-function[12932]:38)
at core::ops::function::Fn::call::hced78bec1b5a708c (wasm-function[11784]:45)
at std::panicking::rust_panic_with_hook::hefb6d16d56fedd94 (wasm-function[3093]:265)
at std::panicking::continue_panic_fmt::ha605a3bfcbd9f700 (wasm-function[6701]:116)
at rust_begin_unwind (wasm-function[14788]:3)
at core::panicking::panic_fmt::hc16a7652262fbed2 (wasm-function[12041]:70)`

Does WebAssembly have a 32-bit usize? Now there's definitely an issue in my code that I compute 2^32 - 1 as (1 << 32) - 1 as this won't work on 32-bit systems.
However... how did this compile?  This should give a compile-time error: https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=f233a1f5f93c82a748d1f0d8da58e34c .

This issue should be fixed in the latest commit. It's going to take a bit (probably at least a couple days to a week) for me to publish 0.4, I have some other changes I'd like to make for that.

Thank you very much for looking at this, unfortunately the example above still panics for me on deserialize when using the latest commit on master.
I have created a small sample app that demonstrates this phenomena.
https://github.com/rebo/sm_issue

What is the panic traceback?

It's a little bit different:
RuntimeError: unreachable
at __rust_start_panic (wasm-function[1114]:1)
at rust_panic (wasm-function[1004]:31)
at std::panicking::rust_panic_with_hook::hefb6d16d56fedd94 (wasm-function[246]:304)
at std::panicking::continue_panic_fmt::ha605a3bfcbd9f700 (wasm-function[533]:116)
at rust_begin_unwind (wasm-function[1103]:3)
at core::panicking::panic_fmt::hc16a7652262fbed2 (wasm-function[843]:70)
at core::panicking::panic::h1f84f7eb49aeaec9 (wasm-function[627]:100)
at slotmap::normal::serialize::<impl serde::de::Deserialize for slotmap::normal::SlotMap<K,V>>::deserialize::hd239fa69a2c00838 (wasm-function[30]:1967)
at serde_json::de::from_trait::hc4dda8f146982609 (wasm-function[54]:121)
at serde_json::de::from_str::h2a6f0f9869081e1d (wasm-function[485]:93)

Alright I tracked it down, it was the exact same issue, just in two other places. I wasn't using a local repository when doing my first fix, so I used Github search in the repo to see if there were any other instances. Unfortunately these instances never showed up. Could you try again with the latest commit?

Again thanks for getting to this so quickly. Looks like we are not quite there yet, here is the latest trace from a new panic than the last one (but similar to the first one lol).

index.js?91f7:408 panicked at 'attempt to shift left with overflow', /Users/rebo/Coding/Projects/Rust/stiff/slotmap/src/normal.rs:1090:31

Stack:

Error
    at Module.__wbg_new_59cb74e423758ede (webpack-internal:///./crate/pkg/index.js:547:21)
    at __wbg_new_59cb74e423758ede (http://localhost:8000/app.js?d0084421dcdfb4bedfd4:801:101)
    at console_error_panic_hook::Error::new::hf9155aeb30e0df7c (wasm-function[11764]:22)
    at console_error_panic_hook::hook_impl::h19b17601d076c52f (wasm-function[2120]:100)
    at console_error_panic_hook::hook::hd8298b7121f5bbfe (wasm-function[12931]:38)
    at core::ops::function::Fn::call::hced78bec1b5a708c (wasm-function[11783]:45)
    at std::panicking::rust_panic_with_hook::hefb6d16d56fedd94 (wasm-function[3092]:265)
    at std::panicking::continue_panic_fmt::ha605a3bfcbd9f700 (wasm-function[6701]:116)
    at rust_begin_unwind (wasm-function[14785]:3)
    at core::panicking::panic_fmt::hc16a7652262fbed2 (wasm-function[12040]:70)

(are you sure you're on the latest commit?)

@rebo  I believe you do not have the latest version as the line number in that panic message is a line I fixed.

Hi Orlp and nabijaczleweli ,
I am happy to report this works fine now. Tested under slotmap 0.4 and latest nightly.
Many thanks for fixing it so quickly.


Feature Request - Create a setter method to sets the encoding when parsing the response as a Document

Hi,
I'm using your library in some projects here, it's great.
It would be great to have a setter method to set the encoding when parsing a response as a document.
For example, if I need to execute a post request and parse the response as a Document with the ISO-8859-1 encoding, I have to do this:
private Document executeRequest(String value2) throws IOException {
    return Jsoup.connect(DEFAULT_URL)
        .timeout(DEFAULT_TIMEOUT)
        .data("param1", "value1")
        .data("param2", value2)
        .userAgent(DEFAULT_USER_AGENT)
        .method(Method.POST)
        .execute()
        .charset(CHARSET_ISO_8859_1)
        .parse();
}

It would be great something like this:
private Document executeRequest(String value2) throws IOException {
    return Jsoup.connect(DEFAULT_URL)
        .timeout(DEFAULT_TIMEOUT)
        .data("param1", "value1")
        .data("param2", value2)
        .userAgent(DEFAULT_USER_AGENT)
        .responseEncoding("ISO-8859-1") // <-- This is the setter method I'm suggesting, something like that
        .post();
}

The postDataCharset method sets the charset when sending a POST request, but not for parsing the response as a document.
Of course the method name is your choice.
What do you think?
P.S: @krystiangorecki This is the issue with the correct description.

Thanks, makes sense. Is the site not setting the response encoding in a header or meta though, or is jsoup parsing it incorrectly? Trying to understand the root issue.

In my tests, the site I was scraping didn't bring the encoding in response headers neither in html, but I knew beforehand the proper encoding was ISO-8859-1.
As far as I know, jsoup parses documents as UTF-8 when it can't detect the document encoding, right?
I don't think it is a jsoup bug, more likely a site problem.
This is the site if you want to check it out.


Write an article relating to 3Box and ETH SF

Maybe announcing the bounties we're offering before the hackathon, and another article describing the projects that used 3Box during the hackathon, or learnings from interviews conducted during the hackathon. There might be two different posts here. Will eventually split this story into two if that is the case.


import workers.new worker script

now w/ wrangler2 and typescript

@eidam this doesnt deploy to staging environment, but i had the other tokens in place


cargo doc and cargo test erroneosly detect a documentation test

Problem
The following code snippet causes cargo test to fail.
pub mod m1 {
    /** This const

    has a multiline document string.
    */
    pub const GOOD_DOC: u32 = 1;

    pub mod m2 {
        /** This const

        has a multiline document string.
        */
        pub const BAD_DOC: u32 = 0;
    }
}

The failure is as follows:
running 1 test
test src/lib.rs - m1::m2::BAD_DOC (line 11) ... FAILED

failures:

---- src/lib.rs - m1::m2::BAD_DOC (line 11) stdout ----
error: expected one of `!`, `.`, `::`, `;`, `?`, `{`, `}`, or an operator, found `a`
 --> src/lib.rs:12:5
  |
3 | has a multiline document string.
  |     ^ expected one of 8 possible tokens

error: aborting due to 1 previous error

Couldn't compile the test.

failures:
    src/lib.rs - m1::m2::BAD_DOC (line 11)

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.03s

error: doctest failed, to rerun pass `--doc`

Moreover, cargo doc generates the documentation of BAD_DOC constant as if it contained a code snippet
.
Interestingly, the GOOD_DOC constant is treated correctly both in cargo doc and cargo test, i.e., no doc-test is created for it and documentation is generated without a code snippet.
This behavior is strange because the doc string is exactly the same in both constants. The only difference is that BAD_DOC is nested in another module m2.
The expected behavior is to treat BAD_DOC the same way as GOOD_DOC:

there should be no doc-test generated and cargo test shouldn't fail
cargo doc shouldn't generate a code snippet

Steps
No response
Possible Solution(s)
No response
Notes
No response
Version
cargo 1.77.1 (e52e36006 2024-03-26)
release: 1.77.1
commit-hash: e52e360061cacbbeac79f7f1215a7a90b6f08442
commit-date: 2024-03-26
host: x86_64-unknown-linux-gnu
libgit2: 1.7.2 (sys:0.18.2 vendored)
libcurl: 8.5.0-DEV (sys:0.4.70+curl-8.5.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Ubuntu 23.10 (mantic) [64-bit]

See also
pub mod m1 {
    /** This const

    has a multiline document string.
    */
    pub const GOOD_DOC: u32 = 1;

    pub mod m2 {
        /** This const

        has a multiline document string.
        */
        pub const BAD_DOC: u32 = 0;
    }

    pub mod m3 {
        /** This const
         *
         * has a multiline document string.
         */
        pub const GOOD_DOC: u32 = 0;
    }
}

Note that m3::GOOD_DOC uses * prefixes on the lines and it works.
Likely whats happening is that * is needed to establish the indent level.  Without that, the raw indentation is passed along to the markdown parser and it is being treated as an indented code block.
However, I'm  unsure how m1::GOOD_DOC isn't treated as an indented code block.
Moving this over for the rustdoc team to triage in case the inconsistency between m1::GOOD_DOC and m2::BAD_DOC is unexpected.


Fix syntax errors for<EMAIL_ADDRESS>Reformat codebase with dhall<EMAIL_ADDRESS>

Context
Fields that are the same name as dhall keywords broke in dhall-1.33 if not using backticks. In this codebase that meant: with and if.
Description
👋 This makes importing this library possible for<EMAIL_ADDRESS>I assume you want all the diff noise of a new formatter in one commit vs subsequent PRs.

@thebritican sorry for the late review!

No worries!


[$500] Log in - New accounts on domain control that have had a bill split with them cannot login on a domain controlled account.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @kevinksullivan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1697856462347089
Action Performed:

Split bill with new user that is using an email on a domain controlled account
Go try to sign up with that user you split a bill with in 1

Expected Result:
User can sign up and get a magic code
Actual Result:
Error message, blocked from signing in
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01237b5b3352b4c2e4
Upwork Job ID:<PHONE_NUMBER>396441088
Last Price Increase: 2023-12-09

Proposal
Please re-state the problem that we are trying to solve in this issue.
New users that have had a bill split with them cannot login to a domain controlled account.  This occurs on Get Started on the homepage on staging.
What changes do you think we should make in order to solve the problem?
Make sure the bill split account are assigned a account id that belongs to the domain controlled account if the domains are the same.  It seems the account details are not able to be retrieved.
https://github.com/Expensify/App/blob/fcee2073f096ad60dd9f30c536342de7d3893282/src/pages/ValidateLoginPage/index.js#L46
What alternative solutions did you explore? (Optional)
Change error mess

Hey @0xmiroslav can you please review the proposal above? Thanks!

@0xmiroslav can you please review the proposal above?

Please let me know if you accept since this is overdue.

@0xmiroslav, any updates?

Waiting for proposals.

I am working on a new proposal.

I need help recreating the issue:
https://expensify.slack.com/archives/C01GTK53T8Q/p1699217765377179

Hey @0xmiroslav can you please follow up on the commend above? Thanks!

Waiting for proposals!

Still waiting for proposals.

Still waiting for proposals.
FYI I'm OOO until Nov 27.

Still waiting for proposals!

Still waiting for proposals!

Still waiting for proposals!

@0xmiroslav @debragail, can you please clarify what scenarios the bug can occur in? Is this bug isolated to splitting a bill with someone?

Bump @0xmiroslav @debragail on my question above. thanks!

@0xmiroslav @debragail, can you please follow up on my questions above?
Thanks!

Hey, @isabelastisser and @0xmiroslav, I wanted to provide my investigation findings. I hope it helps.
This issue requires a backend fix and possibly some frontend changes to match because the user of the Gmail-controlled account can solve it by inviting the user of the new domain-controlled account to the domain on Old Dot. The problem might be the server is incorrectly authenticating optimistically created domain-controlled accounts. Instead of splitting a bill, I started a chat and got the same error message from the server.
Furthermore, I got the same error message when using a Gmail-controlled account to a domain-controlled employee account (like the description) and a Gmail-controlled admin account to a domain-controlled employee account.
Could you verify that my analysis is correct? Thanks 😄
Error message from server

Hey, @isabelastisser and @0xmiroslav, I wanted to provide my investigation findings. I hope it helps.
This issue requires a backend fix and possibly some frontend changes to match because the user of the Gmail-controlled account can solve it by inviting the user of the new domain-controlled account to the domain on Old Dot.
The problem might be the server is incorrectly authenticating optimistically created domain-controlled accounts.
Instead of splitting a bill, I started a chat and got the same error message from the server.
Furthermore, I got the same error message when using a Gmail-controlled account to a domain-controlled employee account (like the description) and a Gmail-controlled admin account to a domain-controlled employee account.
Could you verify that my analysis is correct?
Thanks 😄
Error message from server

Thanks for the details @Tony-MK! @0xmiroslav, can you please review and follow up on the explanation above?

Do we need an Expensify engineer to work on this @0xmiroslav ?

@isabelastisser should I still work on the issue?

yes, at least on backend part. Hope @blimpich would help with this issue 🙏

Acknowledging this. I think this is worth working on but I am currently prioritizing other issues. Thank you @0xmiroslav for summarizing the state of this issue at the moment. I don't know when I'll have time to work on this, but hopefully sometime next week.

This will be reviewed next week.

Demoting to weekly until I know that I can work on this daily.

Issue not reproducible during KI retests. (First week)

Not overdue.

Issue not reproducible during KI retests. (Second week)

Not overdue. Not reproducing so might be a non-issue. Will let this sit for now.

@blimpich I'll close this since it cannot be reproduced.


Try to resist inexistent data in the mine

For some unknown reason, the mine can contain data about a node X after removing X from the cluster. However, when minions try to get some data about X (like the IP address), that information is not there and we get an exception. This problem seems to appear only when generating the /etc/hosts file.
This PR tries to resist that data missing by just returning empty values and skipping these hosts in /etc/hosts. Some hosts will have an entry for node X in  their /etc/hosts while some others will not...
https://bugzilla.suse.com/show_bug.cgi?id=1091361
bsc#1091361

I haven't tried it yet, but code wise LGTM


Include improved TS library in this package

In the main projects actually using the route-snapper, I'm not using https://github.com/dabreegster/route_snapper/blob/main/route-snapper/lib.js. Instead there's a newer TS version of this library, split into a few pieces:

https://github.com/dabreegster/ltn/blob/main/web/src/common/snapper/route_tool.ts: the main TS replacement library
https://github.com/dabreegster/ltn/blob/main/web/src/common/snapper/RouteSnapperLayer.svelte: a svelte-maplibre styling for drawing the editing operations on a map
https://github.com/dabreegster/ltn/blob/main/web/src/common/snapper/SnapPolygonControls.svelte: really simple Svelte UI with instructions and an undo button

At least that first piece should be included in the NPM package here. The other two pieces could be listed as example references, but since they're Svelte specific, not much code, and make UI choices about wording / styling that's not likely to be general, I don't think they belong in the package.
I need to figure out how to include a TS file in an NPM package correctly. The NPM package is partly generated by wasm-pack and already includes some auto-generated type declarations, so it has to play well with that. Shouldn't be too hard though.
CC @tordans -- if you'd want to use the TS library and not just copy in the short-term, I'll prioritize this

FYI I started looking into out how to add the library to ReactMapGL https://visgl.github.io/react-map-gl/ in an Astro Project. Any hints on that are welcome, did anyone add it to a react project, yet?

I was wondering if the package is following the recommendations on how to export methods, because it looks like VS Code / Typescript magic that allows to "import package" when using a function seems to not work. Using the import used in your svelte example I get…

I've got some ideas to simplify things here -- I'm going to try building + publishing just the WASM library as one NPM package, then making a second package with the nice TS wrapper. I'm having trouble finding a minimal working template to do that, though -- I've been trying the Vite vanilla TS starter, but it doesn't appear suitable for publishing libraries, as it's insisting on an index.html and main.ts. Happen to know any simple templates or guides to follow? Personal recommendations win over outdated Medium articles ;)


Use the hash of the BlobTx as a reference in the mempool and event system

Tendermint was designed around the hash of the transactions that users submitting, and the hash of the tx included on chain, be the same. For us, this is not the case, as a user submits a single transaction that contains the encoded sdk.Tx that will be included onchain along with the blob, but these things are separated before they are included in the block. Similar to what we do currently, where we keep track of each transaction with the original transaction's hash, this PR uses the hash of the encoded sdk.Tx that gets included on chain. This way, after the transaction gets included, we can hash it without having to lookup the relevant blob.

closing as redundant to #892


extension of ingestion and subscription end points

SRS 4.5.0 (1.11)
Extension of /ingestion and /subscription end points
Added postman collections
Added Junit Testcases

Can one of the admins verify this patch?

ok to test

retest this please

retest this please


5.) User authentification example code broken

Hi,
the 5-user-authentication.md consists of the following example code:
Login users
export class AppComponent {
  constructor(public af: AngularFire) {}

 login() {
    this.af.auth.login();
  }

  logout() {
     this.af.auth.logout();
  }
}

We get the following error:
[ts] Property 'af' does not exist on type 'AppComponent'.

Looks like af is not being added automatically to the this scope automatically.
Should it be :
export class AppComponent {
  af: AngularFire;
  title = 'app works!';

  item: FirebaseObjectObservable<any>;

  constructor(af: AngularFire) {
    this.af =  af ;
    this.item = af.database.object('/item');
  }

  login() {
      this.af.auth.login();
    }

  logout() {
     this.af.auth.logout();
  }
}

But there must be a better way to initialize af since the Angualr 2 docs say that it is not good practice to do such things in the constructor and rather use:
  ngOnInit() {
  }

But how to inject public af: AngularFire to ngOnInit()
please?

I believe instantiating the AngularFire class in the constructor makes sense instead of ngOnInit().
See http://stackoverflow.com/a/35763811/728287

So You should use constructor() to setup Dependency Injection and not much else. ngOnInit() is better place to "start" - it's where/when components' bindings are resolved

What if you add the @Component decorator, does it work in the constructor?

What if you add the @Component decorator, does it work in the constructor?
? it already woks in the constructor like this:

  constructor(af: AngularFire) {
    this.af =  af ;
    this.item = af.database.object('/item');
  }

Ok I found the issue , the af variable in the constructor needas to be private or public
constructor( private af: AngularFire) {


Port SQLite database driver from CI3

Will SQLite 3 be the min version?

I haven't looked into the specifics of why we'd need to keep v2 around, but, yes, it was my intention to just do 3.

Sweet, I'll try to start getting somewhere with it

Awesome! Let me know if you run into any questions.


Option to filter out SVG images

Hi, I've been working with some pdf's in PDFminer that when processed cause very high memory usage, this seems to be due to the amount of objects created by analyzing pages with SVG images.
Steps to reproduce:
>>> from pdfminer.high_level import extract_pages
>>> for page in extract_pages("/home/antoine.local/Downloads/pdf_with_svg_image.pdf"):
...   print(len(page._objs))
...
100029

To me, 100k objects from a single figure seems obscene, and this runs into memory issues on my machine, especially when I try to scan multiple pages with SVG's.
My ideal solution would include limiting the amount of objects that can be gathered from a single figure, but I'm not sure how feasible this is.:
>>> from pdfminer.high_level import extract_pages
>>> for page in extract_pages("/home/antoine.local/Downloads/pdf_with_svg_image.pdf", max_figure_object_amount=1000):
...   print(len(page._objs))
...
1000

Any other options? Or maybe a pointer to what code causes this? I could try and see if I could add support myself and create a PR or fork if necessary.

I'm not sure if the issue is caused by the svg images.
Can you share the PDF such that we can investigate?

Great example!
I used python tools/pdf2txt.py pdf_with_svg_image.pdf and it runs for a long time, although there is no text.
I think the preferred solution here is similar to #455, being able to ignore the images altogether in the layout algorithm.

Hmm yes, so this looks like a duplicate then if I understand that other ticket correctly? In which case this can be closed? Or do you envision something else being required here


Update shapeless to 2.3.7

Updates com.chuusai:shapeless from 2.3.6 to 2.3.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.chuusai", artifactId = "shapeless" } ]

labels: library-update, semver-patch

Codecov Report

Merging #972 (b33c228) into develop (b43c0f3) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #972      +/-   ##
===========================================
- Coverage    89.57%   89.54%   -0.04%
===========================================
  Files          119      119
  Lines         9871     9871
  Branches       511      511
===========================================
- Hits          8842     8839       -3
- Misses        1029     1032       +3

Impacted Files
Coverage Δ

...scala/com/twitter/algebird/PredecessibleLaws.scala
71.42% <0.00%> (-14.29%)
:arrow_down:

...main/scala/com/twitter/algebird/monad/Reader.scala
52.94% <0.00%> (-5.89%)
:arrow_down:

...src/main/scala/com/twitter/algebird/Interval.scala
73.04% <0.00%> (-2.61%)
:arrow_down:

...a/com/twitter/algebird/immutable/BloomFilter.scala
95.08% <0.00%> (ø)

.../scala/com/twitter/algebird/immutable/BitSet.scala
96.78% <0.00%> (+0.21%)
:arrow_up:

.../main/scala/com/twitter/algebird/HyperLogLog.scala
90.04% <0.00%> (+0.43%)
:arrow_up:

.../main/scala/com/twitter/algebird/BloomFilter.scala
94.24% <0.00%> (+0.44%)
:arrow_up:

Continue to review full report at Codecov.

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


Whether stage-2 pre-train model(llavaphi-2.7b-finetune) is released?

Will do.

same ques

Sorry. I haven't had enough bandwidth for this lately. But it will be about this week.

🎉 We release all stage2 models, cheching our model zoo.


Bug in Line 3 not reading color blue " waypont

Challenge Waypoint: Use CSS Selectors to Style Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.101 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

ERROR: (Your h2 element should be blue.)
Color blue issue.docx

This is a duplicate of #4885. We have a PR in the works that will solve this.
Thanks and happy coding!


Add more fan options

Please add the possibility to disable the fan for the first "X" layers to make first layers better stuck on the bed.

Sorry for the delay @leobel96 . I have been tied up with a job transition.
Do you think it's an acceptable solution to just have the desired value set in the slicing profile? Obviously if you regularly adjust this setting, from model to model, it can be very annoying to use slicing profile for the purpose. If this is the case, can you elaborate why you need to make frequent adjust to this setting (disable fan for first X layers)?

You are right, it's something I'll never adjust it but I haven't cura installed on my pc so I should install the old cura, set everything and then save config and upload it. I'd like to uninstall all my slicers and only use Octoprint-slicer. In this way I can also slice and print all my projects from everywhere also from my smartphone.

@leobel96 IT would be worth keeping a compatible version of Cura on your PC so you can make new slicing profiles. You never know when you might want to get into a new filament, or tweak some obscure setting that Ken's plugin does not give you access to.

@John-Mc Yes, it's true

If you want to be geeky about it, you can ssh to your octopi and find the xxx.profile file, and directly edit the settings there, as in attached screenshot.
It's a shame on everyone working on 3D printing softwares, myself included, that our users are still required to jump through all these hoops to get things done. :(

I definitely want to get to a point where users only need to go to one place to do everything, as opposed to having to keep an old software to work around problems. However, it requires a much better (and bigger) planning than this plugin warrants.
Since the original purpose of this plugin is to provide an easy and quick way for users to slice directly within OctoPrint, I hesitate want to add options that are not used often. Since you already have a workaround, I'll just close this issue for now @leobel96

@kennethjiang thank you for the suggestion!! I'll try that. Anyway I agree with everyone said. I'll edit using SSH or cura, no problem.


fix(install): Ensure error bubbles up when executing only 1 recipe and failing

@zlesnr Regardless of what message we display to the user, I think we need to return an error here to exit with a non-zero code.


Missing sections of the supposed API?

Looking at the wish.go file, there seems to be many helpful printing functions that aren't actually included in the built package - am I missing something? Only the NewServer, Error, and Fatal ones are exposed.
See https://pkg.go.dev/github.com/charmbracelet/wish#readme-logging

Please excuse my 1am brain – those functions just haven't been released yet. Time for bed 🙃


Running tutorial_train.py on Tesla V100 16Gb results in out-of-memory during image logging

Running tutorial_train.py (with fill50k dataset) on Tesla V100 16Gb results in out-of-memory during image logging (DDIM Sampler).
I am running with batch_size=1 and accumulate_grad_batches=4 (OOM if batch_size>1).
When training, 15654MiB of the total GPU memory (16160MiB) are used.
Any suggestions?
Thanks!

The error message:
  File "/home/ubuntu/ControlNet/cldm/cldm.py", line 397, in log_images
    samples_cfg, _ = self.sample_log(cond={"c_concat": [c_cat], "c_crossattn": [c]},
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/ControlNet/cldm/cldm.py", line 413, in sample_log
    samples, intermediates = ddim_sampler.sample(ddim_steps, batch_size, shape, cond, verbose=False, **kwargs)
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/models/diffusion/ddim.py", line 103, in sample
    samples, intermediates = self.ddim_sampling(conditioning, size,
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/models/diffusion/ddim.py", line 163, in ddim_sampling
    outs = self.p_sample_ddim(img, cond, ts, index=index, use_original_steps=ddim_use_original_steps,
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/models/diffusion/ddim.py", line 211, in p_sample_ddim
    model_uncond, model_t = self.model.apply_model(x_in, t_in, c_in).chunk(2)
  File "/home/ubuntu/ControlNet/cldm/cldm.py", line 339, in apply_model
    eps = diffusion_model(x=x_noisy, timesteps=t, context=cond_txt, control=control, only_mid_control=self.only_mid_control)
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/ControlNet/cldm/cldm.py", line 30, in forward
    h = module(h, emb, context)
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/modules/diffusionmodules/openaimodel.py", line 84, in forward
    x = layer(x, context)
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/modules/attention.py", line 334, in forward
    x = block(x, context=context[i])
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/modules/attention.py", line 269, in forward
    return checkpoint(self._forward, (x, context), self.parameters(), self.checkpoint)
  File "/home/ubuntu/ControlNet/ldm/modules/diffusionmodules/util.py", line 114, in checkpoint
    return CheckpointFunction.apply(func, len(inputs), *args)
  File "/home/ubuntu/ControlNet/ldm/modules/diffusionmodules/util.py", line 129, in forward
    output_tensors = ctx.run_function(*ctx.input_tensors)
  File "/home/ubuntu/ControlNet/ldm/modules/attention.py", line 272, in _forward
    x = self.attn1(self.norm1(x), context=context if self.disable_self_attn else None) + x
  File "/home/ubuntu/anaconda3/envs/control/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/ubuntu/ControlNet/ldm/modules/attention.py", line 177, in forward
    sim = einsum('b i d, b j d -> b i j', q, k) * self.scale
RuntimeError: CUDA out of memory. Tried to allocate 1024.00 MiB (GPU 0; 15.78 GiB total capacity; 12.46 GiB already allocated; 1024.00 MiB free; 13.61 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

all duplicate concerning "RAM and out of memory exceptions (OOM)":
https://github.com/lllyasviel/ControlNet/issues/21
https://github.com/lllyasviel/ControlNet/issues/33
https://github.com/lllyasviel/ControlNet/issues/191
https://github.com/lllyasviel/ControlNet/issues/236
https://github.com/lllyasviel/ControlNet/issues/241
https://github.com/lllyasviel/ControlNet/issues/247
https://github.com/lllyasviel/ControlNet/issues/294
https://github.com/lllyasviel/ControlNet/issues/301


Please remove implicit operator InputOnlineFile(stream)

It confuses people more than help them.
Nobody ever wants to send a nameless file without an extension defining the type.
Additionally, you could make the fileName parameter mandatory (no default null value) on:
public InputOnlineFile(Stream content, string? fileName = null)

I do agree that all SendXXXAsync() methods would benefit from this approach.
But there are few cases, where file name should not be mandatory:

a certificate parameter in setWebhook
a photo parameter in setChatPhoto
*sticker* parameters in createNewStickerSet, sendSticker , uploadStickerFile , addStickerToSet, setStickerSetThumb
all thumb parameters...

Should we enforce users to specify FileName in some methods with new NamedStreamFileSomething type?

I did some changes in InputFile type hierarchy to align it with the Bot API:
https://github.com/TelegramBots/Telegram.Bot/tree/karb0f0s/input_file

certificate parameter in setWebhook
this one is InputFileStream, not InputOnlineFile

for the other, if type is implicit regardless of filename/extension, user can still pass an explicit null as filename.
in any case, if you don't accept my second suggestion (InputOnlineFile mandatory 2nd argument), the removal of implicit operator will force the dev to get interested in the InputOnlineFile constructors and find out the filename parameter if they need it.

I don't really like the idea of passing null around.
Giving this whole issue some though it seems that the only method that actually requires file extension is SendDocument. For all other cases/media types extension is not important or relevant.
I'd love to see some prototype/suggestion on how this case should be handled.

then maybe have an overload of SendDocument take Stream and filename arguments instead of InputOnlineFile

Resolved by #1147.


As a data developer, I want to investigate missing AMI data, to improve the tool's data quality.

Description
Median household income as a percent of area median income has 7.4% of the values missing. This might be due to a bad join in the ETL, or the data fundamentally being unavailable.

@emma-nechamkin are we doing this?


Fix dropdown direction issue on webkit

#1016

Should be solved with https://github.com/angular-ui/ui-select/pull/1212


Document advanced filter behaviours

The filter documentation already describes the basic work of a filter chain and the protocol filters, where each request follows through A -> B -> C and each response flows back to through C -> B -> A.  What we are missing is documentation
describing the other abilities of the filter:

request filters can, at their own volition. send their own response (i.e. an error response).
filters can make their own requests to the broker

etc.
Originally posted by @k-wall in https://github.com/kroxylicious/kroxylicious/pull/406#discussion_r1244948459

Keen to do this


voted_sensors_update: fix mag rotation

Yet another fix for mag rotations...
fixes a wrong index for _mag_device_id: previously, driver_index was used
(the CAL_MAG param index), but the correct index is the uorb topic
instance.
Tested on a setup where the board is 180deg rotated, but the external mag faces forward direction.
@mhkabir

@bkueng Thanks. What was the exact issue which you found? On what hardware?

OK, nevermind I see it now.


Fix mypy errors created with latest numpy

Summary
numpy 1.22.0 released for python > 3.7 added mypy errors.
Errors fixed in this PR
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.5%) to 82.271%

Files with Coverage Reduction
New Missed Lines
%

qiskit_finance/data_providers/wikipedia_data_provider.py
3
75.0%

qiskit_finance/data_providers/exchange_data_provider.py
7
65.08%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
710

Relevant Lines:
863

💛  - Coveralls


Not compatible with Umbrel?

Just installed, Error message upon launch:
UNABLE TO FETCH THE NODE TIER
pool is not running: rpc error: code = DeadlineExceeded desc = context deadline exceeded
Same when choosing Pool from the left menu:
This subserver encountered an error and failed to start up.
rpc error: code = DeadlineExceeded desc = context deadline exceeded

0.11.0-alpha
Log shows no errors:
2023-10-19 18:53:53.632 [INF] LITD: Handling static file request: /
2023-10-19 18:53:56.670 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetInfo
2023-10-19 18:53:59.303 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetInfo
2023-10-19 18:54:10.827 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/SubscribeTransactions
2023-10-19 18:54:10.845 [INF] LITD: Handling gRPC web request: /litrpc.Status/SubServerStatus
2023-10-19 18:54:10.871 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/SubscribeChannelEvents
2023-10-19 18:54:10.874 [INF] LITD: Handling gRPC web request: /litrpc.Status/SubServerStatus
2023-10-19 18:54:10.895 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetInfo
2023-10-19 18:54:10.918 [INF] LITD: Handling gRPC web request: /looprpc.SwapClient/Monitor
2023-10-19 18:54:10.922 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetInfo
2023-10-19 18:54:10.924 [INF] LOOPD: Monitor request received
2023-10-19 18:54:14.215 [INF] LITD: Handling gRPC web request: /poolrpc.Trader/NodeRatings
2023-10-19 18:54:14.218 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/ListChannels
2023-10-19 18:54:14.226 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/ListChannels
2023-10-19 18:54:15.575 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/PendingChannels
2023-10-19 18:54:15.584 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.605 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.611 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.670 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.672 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.695 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/ChannelBalance
2023-10-19 18:54:15.697 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/PendingChannels
2023-10-19 18:54:15.724 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.749 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.751 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.809 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.870 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/GetChanInfo
2023-10-19 18:54:15.872 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/ChannelBalance
2023-10-19 18:54:15.915 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/WalletBalance
2023-10-19 18:54:16.014 [INF] LITD: Handling gRPC web request: /lnrpc.Lightning/WalletBalance
2023-10-19 18:54:16.030 [INF] LITD: Handling gRPC web request: /litrpc.Sessions/ListSessions
2023-10-19 18:54:16.053 [INF] LITD: Handling gRPC web request: /looprpc.SwapClient/ListSwaps
2023-10-19 18:54:16.135 [INF] LITD: Handling gRPC web request: /litrpc.Sessions/ListSessions
2023-10-19 18:54:16.158 [INF] LITD: Handling gRPC web request: /looprpc.SwapClient/ListSwaps

Could you try upgrading to 0.12.0? Recently released on Umbrel.

I uninstalled and installed again. Now I cannot log in with the default password. Nothing happens when O press Submit. Tried on the phone with Opera.

Why does it keep asking me to Connect to Terminal even though I already connected?

Yes, I see no errors

So, you are good to go after upgrading? Everything working as anticipated? If so, I will close this issue. Thanks for reporting!

Why does it keep asking me to Connect to Terminal even though I already connected? And creates every time a new session... I don't like something running on my node that lets connect to it from the web. I better uninstall.

You're welcome to use the local UI version of Terminal and not connect to anything. If you want a more robust feature set (Autofees, Autoloop, and more), you can connect out to terminal.lightning.engineering. More info in our docs. The connection to Terminal on the Web is done through Lightning Node Connect, which is an end-to-end encrypted mechanism for connecting a Lightning Node and a web browser. We can develop faster and react to user feedback more by developing on the web, but you can still just use the local UI as you are now if you so desire.

I'm going to close this issue for now as it sounds like you ended up seeing no errors per the message above. We can always re-open it if you encounter any issues.


React 16.2.x runtime throws an exception on ES7 autobind render()

Any usage of this component with the latest React causes:
Cannot assign to read only property 'render' of object '#<ProxyComponent>'

This PR changes render = () => syntax to regular method declaration in handleFocus component.

@azproduction thanks for the PR!


What do i have to do to enable VTT Subtitles Support

Current behavior
I am trying to setup the subtitles into react native video based player but when trying to make it run on iOS is not working. Does anybody knows how to run it.
Reproduction steps
Run In iOS with another react native component called AFVideoPlayer and adding to the control bar a toggle Subtitles button is not working
Expected behavior
Hide and show the Selected subtitles (selectedTextTrack) clicking a button to change the state from { type: 'title', value: 'English'} to {type: 'disabled', value: ''}
Platform
Which player are you experiencing the problem on:

iOS

Video sample
Link to see the issue in action

AFVideoPlayer is using react-native-video 2.0.0 so it's possible we fixed some bugs or added additional support for subtitles since then. Also, if you're using a set of controls on top of RNV, you'll need to work with them to get the bugs worked out.
I would create an example that uses RNV directly, make sure the subtitles work then fix the bug in AFVideoPlayer and submit a PR to them or add more info here if it's a bug in RNV.


Request for dark mode/How to turn enable dark mode

I will be glad if someone in the team can please tell me how to add a dark mode feature or make the theme dark by default, thanks xD.

https://www.spectralwebservices.com/blog/adding-a-dark-mode-button-to-your-ghost-blog/
Best Regards,
Jen
JEN WALLACETech Help for Seniors &
*IT Support for Entrepreneurs  *
(303) 647-1411**www.DenverIT.com https://denverit.com/
On Thu, May 9, 2024 at 9:31 AM Olaoluwa @.***> wrote:

I will be glad if someone in the team can please tell me how to add a dark
mode feature or make the theme dark by default, thanks xD.
—
Reply to this email directly, view it on GitHub
https://github.com/huangyuzhang/Fizzy-Theme/issues/82, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACOXYOAGHI4J5L24DWI4G3ZBOJFRAVCNFSM6AAAAABHPAXKC6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI4DOOJTGQYTMOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks


When will code be released?

also very interested!

looking forward to the code  !!!!

@msftgits @cclin0 sorry to bother you, do you have any idea when this code will be released?
Thanks!

Hello, author, how can I guarantee the correctness of the segmenter mask when t=0? If the first frame is not segmented correctly, what happens to the subsequent frames

looking forward to the code !!!!


Update adding-windows-nodes.md

sync with the english version with same format.

/lgtm

/approve

/approve

can this commit be merged now?


.anbf grammar

Hi, I'm working on building a pretty deep neo4j integration with Elixir. I'm pretty new to grammars and I have only found parsers that support anbf grammars. I'm not familiar with the difference between abnf and ebnf.
Is this something that is easy to add? It looks like you are generating the .enbf from an xml file.
Thanks!

Hello @venkatd and thanks for your question.
The thing with grammars is that each parser or tool seems to have their own interpretation of BNF-like syntax. We went with EBNF because it has an ISO standard (ISO14977), but we do not expect that the generated artifact (the EBNF grammar) will be readily consumed in its raw form. Rather we'd expect some tool to have to rewrite it into the target flavour of EBNF syntax, and I think it would be possible to do so for ABNF as well.
It would probably be possible to add a custom ABNF-writer, however, which perhaps is what you really were after. You're correct that we are generating the grammars from an XML specification. This XML specification is what we regard as the source of truth (although it is not currently complete), rather than the generated BNF grammars. We do currently have two such writers, one that uses ISO14977 format, and one that uses Antlrv4 format (http://www.antlr.org/). I don't see any immediate blockers to adding another subclass of BnfWriter, AbnfWriter or similar, that would output the grammar in the ABNF format. From the little I've gathered on ABNF so far, it seems to be similar enough to EBNF that the current ISO14977 implementation could be of good use in such an effort.
As for our own plans, we do not currently intend on adding more output formats, but we welcome both the discussion and possible contributions, if you are interested in implementing something like this yourself.
All the best
Mats

ABNF is defined in RFC5234, like Mats says it should be pretty simple to implement a generator for that format.

I'm closing this as answered. Please reopen if there are any follow-up questions.


[Desktop] Link window to trail icon

should reimplement it using system api for windows and, at least, macos. We can leave mouse event detecting as a fallback.


Add a formatted JSON preview in preview pane

📝 Add a formatted JSON preview in preview pane
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
When you click on a JSON file with the preview pane open, it should show a formatted version of the JSON even when the JSON is not formatted itself.

Possible duplicate of #1527

yup, valid 100% we'd want this.  #1527 would handle this where a Monaco based solution would 'just work' for most dev files.  We need to do a proof of concept to prove this


import tvm now requires pytest

With the merge of #6331, import tvm now requires pytest. I created this issue just to check whether this is something intentional or something that we want to fix.
The chain from import tvm to import pytest happens due to the from .import testing on python/tvm/__init__.py. There is nothing actually done with that import.
https://github.com/apache/incubator-tvm/blob/a4ebb16ed76bfea4ce4eed7be7ea73d4a01027e2/python/tvm/__init__.py#L53-L56
Within python/tvm/testing.py then there is the import pytest. I was thinking that we might want to remove these lines from __init__.py, so that we don't load tvm.testing and will only import it when required. I'm happy to submit a PR removing those lines, in case there is an understanding that it makes sense.
cc @tqchen

I agree. thanks @leandron !

Good catch.
When I was adding the pytest-related functions, I originally put them in tests/python/__init__.py so they would be local to the tests. Unfortunately, doing this meant that you could no longer run files directly. Instead you'd have to specify the module path. For example, python tests/python/unittest/test_ir_type.py would become python -m tests.python.unittest.test_ir_type.


Command to list active forgotten virtual machines

1minutetip has an option -r which gives you a list of your vm's that you started and did not kill.
tmt should have something similar in case you forget to finish a machine and it stays active somewhere.

How about a whole new area, instead of one?
$ tmt list-runs --active

Agreed, this could be definitely useful for other provision methods as well. It could also be something like
tmt status --running
tmt status --finished
tmt status

The last one could provide an overview of available runs.

Great idea! I just checked how 1minutetip implements it and it queries for a specific machine name using openstack. Do you think this approach would be applicable to other provisions (I am not sure about testcloud and vagrant, it would definitely work with container and minute)? But perhaps it would be better to keep status somewhere in the run directory (it would be modified during the finish step) rather than relying on naming of machines.

Yes, vagrant has global status.
But more generally- we could step into every folder and check whether the guest is running (as a backup way of status-polling f.e.). I'm incline not to rely on our status file, but that can be turned off by default (if there is a "state" file), and could be triggered by f.e. --force-check.

@psss do you think having to check for every machine state in every workdir would be complicated / slow?

I think checking all available test run workdirs (under the default location) should be ok and relatively simply/fast. We just check for runs where provision is done and finish is not done.

Sounds good to me and should be simple to implement (since we store the status of the step in step.yaml). I can try implementing this. What should the output be? Simply list out the paths (ids) to workdirs?

What about listing one line per each run/plan combination? It could look like this:
disc prov prep exec repo fini
done done done done done done /var/tmp/tmt/run-225 /plans/basic
done done done todo todo todo /var/tmp/tmt/run-225 /plans/core
todo todo todo todo todo todo /var/tmp/tmt/run-225 /plans/helps

The done and todo labels could be nicely colored to quickly see what has been already completed.

What about listing one line per each run/plan combination? It could look like this:
disc prov prep exec repo fini
done done done done done done /var/tmp/tmt/run-225 /plans/basic
done done done todo todo todo /var/tmp/tmt/run-225 /plans/core
todo todo todo todo todo todo /var/tmp/tmt/run-225 /plans/helps

Wouldn't just the last 'done' step suffice? This does not look much like user-friendly output to me.

Or maybe print current running step if running and todo/done for not started/finished ? E.g.
done /var/tmp/tmt/run-225 /plans/basic
exec /var/tmp/tmt/run-225 /plans/core
todo /var/tmp/tmt/run-225 /plans/helps

The reason for this was originally to be able to spot and kill machines that you left running, that sometimes happen if you get disconnected or you need more machines to run a test in background,....
Take a look at 1mt -r output:
1minutetip -r
[ INFO  ] Select instance which you want to reattach by providing its number
1	user-1MT-RHEL-7.2-updates-123456-56789-2020-10-27-10-52 <IP_ADDRESS>
Which instance to attach? (number):
if you then reattach you can kill it.
How would you use the output you are showing to kill the machine?
The ID could maybe be first and separated by spaces for easy copying by double click.

How would you use the output you are showing to kill the machine?

In order to finish the reserved guest you would do the following:
tmt run -i /var/tmp/tmt/run-225 finish

Ok then.

I like @lukaszachy's approach with a single value but still I see use cases when the long output would be useful as well. For example I fire up a bunch of integration jobs and want to know what's their status. Perhaps we could support -l and --long options to enable the detailed output?

And, just brainstorming, we could also have tmt status --finish to finish all running jobs?

And, just brainstorming, we could also have tmt status --finish to finish all running jobs?

It's not obvious what it does IMHO, could we use instead something like --run finish?

I like @lukaszachy's approach with a single value but still I see use cases when the long output would be useful as well. For example I fire up a bunch of integration jobs and want to know what's their status. Perhaps we could support -l and --long options to enable the detailed output?

Or different short option? Not to confuse it with -l as --last.

Or different short option? Not to confuse it with -l as --last.

What about simply verbose (so -v or ---verbose)? I feel like that verbose exactly describes that it gives more information (in this case information about status of individual steps)

What about simply verbose (so -v or ---verbose)?

Yeah, actually that's better and consistent with the rest.

Based on the discussion in this issue and our conversation on IRC, I propose the following solution:
The command tmt status will have 2 general options:

--verbose/-v that adds information about step status to the output (see examples below)
path that changes where the search begins. Defaults to /var/tmp/tmt. This can be useful as working directory of tmt can be set to anything so the users may want to provide a custom location. Example: Running tmt status . would list status from CWD.

Then there will be mutually exclusive filters (by default all disabled, so all runs and plans will be written out). For starters, we could start with the following filters:

--unfinished-guest that lists out plans that have provision step done but the finish step is todo. In other words, there is a running VM somewhere that should be cleaned up. This is what @Athwale suggested in this issue. We could later add a way to finish the machines more easily (I am not really sure where such option would fit).
--active lists out all plans where at least one step is todo.
--finished the inverse of active, lists out all plans where all steps are done.

Now for some examples making use of what @psss and @lukaszachy suggested:
> tmt status
status id
done   /var/tmp/tmt/run-225  /plans/basic
exec   /var/tmp/tmt/run-225  /plans/core
todo   /var/tmp/tmt/run-225  /plans/helps

> tmt status --verbose
disc prov prep exec repo fini
done done done done done done  /var/tmp/tmt/run-225  /plans/basic
done done done todo todo todo  /var/tmp/tmt/run-225  /plans/core
todo todo todo todo todo todo  /var/tmp/tmt/run-225  /plans/helps

> tmt status --unfinished-guest
exec   /var/tmp/tmt/run-225  /plans/core

> tmt status --unfinished-guest --verbose
done done done todo todo todo  /var/tmp/tmt/run-225  /plans/core

> tmt status --active
exec   /var/tmp/tmt/run-225  /plans/core
todo   /var/tmp/tmt/run-225  /plans/helps

> tmt status --finished
done   /var/tmp/tmt/run-225  /plans/basic

The output could obviously be colored (use green for done, yellow for todo, keep the coloring consistent with the rest of tmt commands).
What do you think about this? Any feedback how to make it better? Any ideas for an option/command that would remove all unfinished guests?

@FrNecas, thanks much for putting this together. Looks good to me in general. I've got a couple of comments/ideas:
Perhaps it would be good to make here the run id handling consistent with tmt run command. There it is possible to provide only directory name, for example like this:
tmt run -i alpha
tmt run -i errata-1234
tmt run -i ksh-tier1
tmt run -i ksh-tier2

So I would expect to be able to do:
tmt status alpha
tmt status errata
tmt status ksh

In order to change the default root directory I'd propose to implement a new option. Perhaps --root or something similar:
tmt status --root /my/custom/run/directory

For unfinished guests I'd suggest something shorter. What about one of these?
tmt status --abandoned
tmt status --deserted

For the simple output we could show full step name instead of four-letter abbreviations:
> tmt status
status  id
execute /var/tmp/tmt/run-001
prepare /var/tmp/tmt/run-002
finish  /var/tmp/tmt/run-003
done    /var/tmp/tmt/run-004

And one more thing: Shall we display individual plans always?

Optionally, we could also introduce a dedicated options --plan to enable displaying individual plans.

@psss thanks for the feedback. I agree that listing status of one run directory should be possible.
I feel like adding tmt status --root could be confusing for the users. There is already a "global" option tmt --root, right? Adding 2 options of the same name under different commands can't be any good. Using the top-level root for status would be inconsistent with how it works in other subcommands.
I agree with using --abandoned instead of --unfinished-guest and with using the full step name in the non-verbose output. The 2 verbose levels also look good to me, in a lot of cases (for example when wanting to shut down unfinished provisions) a user will only care about the run id as they would run tmt run --id /var/tmp/tmt/run-001 finish.

And one more thing: Shall we display individual plans always?
IMHO proposed -v for plans and -vv for state table is nice.

As discussed on the hacking session the clean up functionality (e.g. shutting down all/recent abandoned containers/machines) should be handled in a dedicated command outlined in #436.

Closing as tmt status has already been merged and the idea to automatically finish all plans with active provision has been proposed in #436


can not import

can not import bignumber by "import BigNumber from 'bignumber.js';" in version 7.1.0

I can import but it throws TypeError: bignumber_js_1.BigNumber is not a constructor when using new BigNumber(..)

i have found workaround:
import BigNumber from 'bignumber.js/bignumber';

7.0.1 , 7.0.2 work fine
but 7.1.0 , The following error occurred

"export 'BigNumber' was not found in 'bignumber.js'

when i use import BigNumber from 'bignumber.js/bignumber.mjs'
it throw error:
WEBPACK_IMPORTED_MODULE_1_bignumber_js_bignumber_mjs.a is not a constructor"

Looks like that BigNumber variable is never assigned in .mjs file
adding BigNumber = clone() before export default BigNumber solved the issue for me.
But I am not sure whether it is correct fix or not

create-react-app's webpack config has issues with .mjs files, eg.
https://github.com/szimek/signature_pad/issues/257
https://github.com/icd2k3/react-router-breadcrumbs-hoc/pull/14
With the change in 7.1.0 I now have to explicitly load the .js version.
import { BigNumber } from 'bignumber.js/bignumber.js';

Yes, @akostiv is right. It seems I accidentally removed the BigNumber = clone() from bignumber.mjs in v7.0.0. I will fix tomorrow.
That still will leave the problem of node.js not supporting the module field in the package.json, so it won't find the bignumber.mjs file if you just do import BigNumber from 'bignumber.js'.
Further information on that still to be digested at these links:
https://github.com/nodejs/node/issues/17300
https://github.com/nodejs/node-eps/blob/master/002-es-modules.md#44-shipping-both-esm-and-cjs
https://github.com/nodejs/node-eps/pull/60
https://github.com/nodejs/node-eps/blob/master/002-es-modules.md

Just published v7.2.0 which should fix the import issues.

Or you could do
import {BigNumber} form 'bignumber.js'

This issue still exists as I'm using the latest v7.2.1 version and import {BigNumber} from 'bignumber.js'.
The workaround
import BigNumber from 'bignumber.js/bignumber';

does work.

For me I'm having issues loading bignumber with Jest while testing though bundling it with webpack for production seems to work.

You should probably open a new issue for this with more information. @bodymindarts

To get anywhere with these import issues much more detail is required about the context in whicih this library is loaded - node.js/browser/bundler/transpiler/typescript etc.
Anyway, I am considering going back to the original, simpler configuration, i.e. removing the module and browser fields from the package.json, changing main: 'bignumber' back to main: 'bignumber.js', and moving the bignumber.mjs file to its own directory so it won't be loaded by mistake.

I am on version 7.2.1 and trying to use the library inside the angular app, so it is TypeScript.
I am using the following import:
import BigNumber from 'bignumber.js/bignumber';
it does resolve 'is not a constructor' issue, so constructing BigNumber works fine, but calling isEqualTo gives 'isEqualTo is not a function' error.
Any ideas? Let me know if you need anything else, I am happy to assist you with providing more details if needed.
Is there any lib version that does not have these issues in typescript environment? I would be happy to downgrade.

@dawka
If isEqualTo is not a function it sounds like you are using an older version of this library.
Typescript imports work just fine: you can use just the standard import BigNumber from "bignumber.js" or import {BigNumber} from "bignumber.js" loading from a node_modules folder.
The issue is probably with the "inside the angular app" part and/or your project's configuration. I am not familiar with angular. If you open a new issue on this and provide fuill details someone may be able to help.

My bad, I'm using typescript! So the import works fine, sorry I didn't specify that (or realize that is whats resolving the issue for me).

Still seeing this when trying to use bignumber.js in React.
npm install bignumber.js --save
...
+<EMAIL_ADDRESS>added 1 package and updated 1 package in 42.022s

In MyEntity.js:
import BigNumber from "./bignumber.mjs";

Webstorm hint:
Cannot resolve file 'bignumber.mjs'

Running MyEntityTest.js with jest:
Cannot find module './bignumber.mjs' from 'MyEntity.js'


add estonia sample data for the tartu workshop

at pygeoapi we have a tradition to add localised data related to the foss4g location, since Tartu is a european one, and the main one is later this year in brazil, not sure if we need to go through the effort...
mind we should have small files, because they will stay in git history, even if removed

a dataset with bathing waters in Estonia extracted from https://avaandmed.eesti.ee/datasets/joogiveeallikad having a CC-BY license
bathingwater.csv.zip
projection epsg:3300


Fix concurrency issue: Only alter completions dictionary while locked

Looks good to me, thanks for catching this!


LP tool to Submit, Amend, Cancel & View LP orders

LPs would benefit from having a tool that allows them to change their provision/shape and watch it in real time.

We already have a tool to allow us to monitor a single party in a single market. However it is missing a few bits of information that might be better viewed in a new tool just for visualising the overall picture for a given market. Such things include:

Number of liquidity providers
Total amount committed
Fee levels
Target commitment for the market


Travis not utilizing updated packages?

I've run into a strange problem. A package I maintain is not able to be installed on Travis because of an apparent namespace issue:
Building with: R CMD build --resave-data --compact-vignettes=gs+qpdf
$ R CMD build  .
* checking for file ‘./DESCRIPTION’ ... OK
* preparing ‘poppr’:
* checking DESCRIPTION meta-information ... OK
* cleaning src
* installing the package to build vignettes
* creating vignettes ... ERROR
Loading required package: adegenet
Loading required package: ade4

   /// adegenet 2.0.0 is loaded ////////////
   > overview: '?adegenet'
   > tutorials/doc/questions: 'adegenetWeb()'
   > bug reports/feature resquests: adegenetIssues()

This is poppr version <IP_ADDRESS>.230. To get started, type package?poppr
OMP parallel support: available

This version of poppr is under development.
If you find any bugs, please report them at https://github.com/grunwaldlab/poppr/issues
Loading required package: ape
Error in unloadNamespace(package) :
  namespace 'ape' is imported by 'poppr', 'adegenet', 'pegas', 'phangorn' so cannot be unloaded
Quitting from lines 205-212 (mlg.Rmd)
Error: processing vignette 'mlg.Rmd' failed with diagnostics:
Package 'ape' version 3.3 cannot be unloaded
Execution halted

The command "R CMD build  ." failed and exited with 1 during .

The odd thing is that this error states ape version 3.3, while the log states that version 3.4 is installed.
This check works on my machine as well as win-builder. I can't seem to figure out what the source of the error is. Is there some sort of hidden cache that is happening in travis?

so there's no special caching -- version 3.3 got installed as a binary dependency. when we install the dependencies for the package itself, we pick up a newer version as a source package.
IIUC, the issue here is a mix of binary dependencies and Depends. i think the binary version of adegenet ultimately loads the binary version of ape; when you call library("ape") at runtime to load the newer source package, it tries to unload the old one, but can't, and badness ensues. this would line up well with the fact that this only happens on travis, which (I'm guessing) is the only place you're doing binary package installs?
one easy workaround is to just avoid picking up adegenet and friends as binary packages, and install from source. it takes a bit longer, but it works. i'm happy to just send a PR if that works for you.
i think getting a newer version of adegenet and ape as binary packages would be another fix.

Thanks for looking into this. I tried switching to binary packages a while back because my builds were taking northwards of 30 minutes, so I can live with 11. A PR definitely works for me. Thank you for checking this out!


External account button is overlapping in French

Describe the bug
The text in the "External account" button is overlapping outside the button .
To Reproduce
Steps to reproduce the behavior:

Go to 'https://reach4help-staging-web-client.firebaseapp.com/'
Select French
Press "Continuez"

Expected behavior
The text should stay inside the button.
Desktop (please complete the following information):

OS: W10 Pro
Browser Chrome
Version 83.0.4103.116

ZenHub Information
This information is updated automatically. To modify it, please use ZenHub.

This issue is technically fixed, but it has created a new issue as the new elements that have been added are not translated.

Will enter new issue.


New version of CLI fails to apply migrations with error: must be member of role "supabase_admin"

Describe the bug
Supabase won't start locally with new version of CLI and Docker images.
We've developed our app with the older CLI (1.29.3). Now a new teammate is blocked from setting up their local environment because the new CLI (1.82.2) and the accompanying images fail to apply our migrations and start the local dev setup.
We are able to reproduce the issue and confirm that 1.29.3 works fine and 1.82.2 fails by installing the CLI as a dev dependency.
To Reproduce
Steps to reproduce the behavior:

supabase start

Expected behavior
Applying the existing migrations and starting the local environment.
Screenshots
Attached.

Desktop (please complete the following information):

OS: macOS Ventura 13.2.1
Version of CLI: v1.82.2

As far as I can find, there are is no documentation for how to migrate old migrations to work with the new CLI.

Hi @ozozozd, we made a security patch to hosted projects so that all tables are owned by postgres role instead of supabase_admin. You _remote_commit.sql file doesn't seem to have the latest platform changes.
You can try to manually edit _remote_commit.sql file to change any alter table statements that use supabase_admin to OWNER TO 'postgres' instead.
If that fails, you may also drop supabase_migrations.schema_migrations table on your hosted project, remove all files in supabase/migrations, then run supabase db remote commit again. This will reset your migration history to the latest snapshot.

Thank you @sweatybridge. Is there any downside to dropping the supabase_migrations.schema_migrations and resetting the migration history?

Feel free to reopen if you are still running into issues.

You can try to manually edit _remote_commit.sql file to change any alter table statements that use supabase_admin to OWNER TO 'postgres' instead. Or simply comment out those alter owner statements since the default owner is postgres.

This solved my issue, adding some text here to help with search since OP provided a screenshot:
ERROR: must be member of role "supabase_admin" (SQLSTATE 42501)
At statement 19: ALTER TABLE "public"."accounts" OWNER TO "supabase_admin"

had the same issue and decided to follow the following steps:

drop supabase_migrations.schema_migrations table
remove all files in supabase/migrations
then run supabase db remote commit again

this borked my Golang services, luckily it's on staging, so I'm not freaking out. I suspect it changed the roles for my user against these tables. This is the error I get:
ERROR: relation "table" does not exist (SQLSTATE 42P01)

I've tried:
ALTER TABLE public.table OWNER TO "postgres";
GRANT SELECT, INSERT, UPDATE, DELETE ON public.table TO "postgres";

but it intermittently fixes the issue, processes one table query, then eventually fails with the next. It's a big whac-a-mole game. Idk, if it has to do with a legacy configuration where previously my db had granted permissions to supabase_admin. I see that was discontinued.
@sweatybridge, any thoughts to remedy this issue?

If anyone encounters the issue I received. This did the trick:

execute SELECT pg_catalog.set_config('search_path', 'public, extensions', true);
restart your db, not ideal, but it was staging, so was fine having down time

must be some cached db connection that was causing the issue.


NotebookProfileViewer: Diff from ref. is a confusing name

Summary
In NotebookProfileViewer, for summary_drift_report, the column name Diff from ref. is a confusing name, as difference gives the idea that the lower the difference, the higher the similarity between profiles, which is not the case.
What is the expected correct behavior?
It should be clear that these values are the same used for the Drift Detected in X Features section in the top of the page, and that these are the p-values calculated through KS or Chisquared test.

What about "Difference using p-value"?

Fixed in https://github.com/whylabs/whylogs/pull/570


Add a HeadingComponent to standardise headings

Trello card
Trello-3870
Context
We have inconsistencies around how we display headings on the website. We also currently embed captions within the heading tag which is not accessible. In an effort to make headings more consistent going forward we're introducing a HeadingComponent and also moving the caption outside the heading as part of this.
Changes proposed in this pull request

Add heading component

Adds a HeadingComponent to help ensure consistent headings throughout the site, specifically when used with a caption.

Remove subheadings/subtitles

These no longer appear to be referenced so we can remove the style and presentation code in the layout.

Use HeadingComponent for captions

Anywhere we use a caption we want to render it via the HeadingComponent for consistency.
Remove all content from left_section of PromoComponent as we no longer want to display any here and it contained a caption.

Update content guidance for headings

Guidance to review
I started replacing all heading tags with the new component but the number of changes is enormous; I think it would be better to do this gradually as we go in and change content.
We can add more to the HeadingComponent to support box headings/highlighting/margins but I'll pick this up in a follow-up PR.
The headings updated so far are in the promo sections, the home page, the 'why teach' sections and the content page headings.

@gazcarr updated if you could recheck

is the gap still there @ethax-ross ?

@gazcarr it looks like the review app didn't refresh after my last update, I've re-ran it and its updated now

Looks fine to me and I'm happy for this to be merged after @gazcarr has reviewed

@gemmadallmandfe @ethax-ross Looking good to me


The preview doesn’t refresh automatically

The preview doesn’t refresh automatically.
I’m trying to fix this issue.
If you want to refresh the preview, please click the refresh button in your browser.

I've fixed this problem.


chore: add missing fonts, dedup italics

Adds missing hosted fonts for Inter and combines the regular and italic styles into one.

Closing in favor of #163.


Open source Sidekick

If Sidekick does end up going down for good, would you be okay with open sourcing the code so other people can run their own instances of it? Sidekick is still the best dice-rolling bot for the features I use it for, and I'd love to be able to continue using for the 3-5 servers I use it in, even if I have to support future API changes myself.

Not currently interested in publishing pronto the old code. If someone wants to help with a pluggable JS engine / architecture, then the old Sidekick might be republished as one of the JS plugins.
( cf. Pluggable in https://github.com/ArtemGr/bounty/blob/main/notes/my-net.md )


[CSL-1563] Use TH to fetch/print git revision

This PR provides a simple TH-based framework (Pos.Util.CompileInfo) to fetch the compile time data like git revision. Previous solution with cpp definition led to unneeded recompilations and now this problem is solved.

Also there was a request to print revision when --version is passed.


Various updates to the generated code

Hi - this is a fairly diverse set of changes; I was playing with changes to the generated code output. I'll create this as a draft PR - this probably shouldn't land as is, but we may want to pull portions out into separate PRs. Some notes (on this PR and on package:intl_translation):

it doesn't look like bin/generate_from_arb.dart responds to -h or --help
we get a type error in bin/generate_from_arb.dart (fixed below) if an ARB file contains a Map where we expect a String (for example, see the @scriptCategory value in the example/lib/messages/material_en.arb file below)
I added an example/ directory in this PR in order to better understand changes to the generated files - the checked in generated code can act as goldens
bin/generate_from_arb.dart is unable to generate from an ARB file unless you have a corresponding dart file that uses the Intl.message messages (see lib/example_messages.dart); this was super surprising to me
various minor changes to the generated code format (see the before and after commits for the differences)

I think its a good idea to add more end-to-end/golden tests.

Makes sense; I can reformulate this into separate PRs.
What's the state of the copybara sync? Could I send PRs or should I contribute internally?

I suspect the reason it expects a corresponding Dart file is that it attempts to prune out any messages that are not referenced from the Dart code. Motivated by Google projects that had very large existing translation catalogs shared with Dart projects that only needed a small subset. I'd think that ought to be able to be skipped, but it might not be obvious.

Ah, thanks for the info! That corresponds with some of the comments I saw in the code when I traced through to find out what was happening (generating files w/ no symbols).


Missing placeholder in glide

Hello your Library is working fine but it has missing placeholder image functionality in glide
SliderView.java line 152, 155
Please add
.apply(new RequestOptions().placeholder(placeholderImageDrawable))
Because of missing Placeholder if image at the specified URL doesn't exist it will show blank screen

You can check this library https://github.com/OpenSooq/Pluto
it has the ability to have full control over your views and layout and changing the placeholder is easy


Improve the error to identify whether client side or server side connection limit is hit in MySQL

Description:
In the current ballerina MySQL module, the same error message is provided when the connection limit is hit on the client-side and server side. We do not have a way to determine which limit was hit, is it the client-side limit or the server-side limit.
I have to use the maxOpenConnections parameter in MySQL options to set the client-side limit and was able to observe the following error message when the limit was hit.
Error while executing SQL query: select * from assets_mapping where input_id="SAMSUNG" limit 1. error while getting the connection for SQLClientConnector. HikariPool-1 - Connection is not available, request timed out after 30000ms..

Then I increased the client-side connection limit and then changed the server-side connection limit with set global max_connections =2;, I get the same error message
Error while executing SQL query: select * from assets_mapping where input_id="SAMSUNG" limit 1. error while getting the connection for SQLClientConnector. HikariPool-1 - Connection is not available, request timed out after 30000ms..

It would be great if the error messages are different so we can know what limit was hit, whether it is from client-side or from the MySQL server-side
Related issue: wso2-enterprise/internal-support-ballerina#201

With the improvements in https://github.com/ballerina-platform/module-ballerina-sql/pull/584, we can provide more specific error messages to the user on why the connection failed.
e.g. when the connection is rejected by the server due to the maximum connection limit being reached
In MySQL with driver version 8.0.29
Error while executing SQL query: SELECT * FROM Customers. error while getting the connection for SQLClientConnector. Data source rejected establishment of connection,  message from server: "Too many connections".
In PostgreSQL with driver version 42.4.1
Error while executing SQL query: SELECT * FROM Customers. error while getting the connection for SQLClientConnector. FATAL: sorry, too many clients already.
The user will be responsible for determining the cause from the error message.

@kaneeldias IMO at this point this is enough since the user can take decisions based on the error message.
I don't think processing the error message to create Ballerina Error is right think forward, since these messages are dependant on both driver and driver version.


Refactoring - separating classes into modules

I did a major refactoring to the whole codebase. Created modules for comparator, diff tool, service, lib and the plugin and separated classes to the modules.
Comparator and diff is still dependent on the plugin for now, but I will continue with refactoring it, so it is only dependent on the lib module.

It kille dthe CI.... The gihub ci fiels will neeed to adjust.

Looks good to me. Please merge. Do not forget to rebase chartjs-1 branch on top of this new set!!
Its better to do it now, then later, after more compelx changes go in...

Please merge.
Was this meant to me? I dont have write access to this repo...

Do not forget to rebase chartjs-1 branch on top of this new set!!
Sure!

merged and will rise your privlidges.


Weird actions

In documentation it says to use
  destroy.delegate="myEventCallback($event)"       edit.delegate="myEditImplementation($event)"
but they are not events but bindable
you can only use
 destroy.call="myEventCallback($event)"       edit.call="myEditImplementation($event)"
but then you don't have the second argument (index)

@natrim That looks like an issue in the docs, I'm sorry about that. Perhaps what you're looking for is the method of defining actions (giving you more control). If you want the event, it has to be implemented (should be simple enough). Want to give it a go?

Haha. I have got the same issue. So I have tried use edit.call() and destroy.call() function for edit, destroy instead of ...delegate() as displayed in documentation. So please update it. Thanks.

@nvtin The docs are in the docs directory, feel free to PR

@RWOverdijk I have just made a pull request for update document. Please check and approve it
https://github.com/SpoonX/aurelia-datatable/pull/99

Docs updated!


Intercity trains despawn instead of returning to an outside connection

As @aubergine18 reported here (#161), intercity trains cannot return to an outside connection after unloading. However, it is currently unclear what is causing this and whether it is a bug in TMPE or not.

It's happening with some, but not all, cargo trains too - in some situations.
Should be reproducible on the save linked in #140 which if memory serves has two cargo stations (one in Garnet district, one in Forestry district) both served by a the same cargo track which loops round in Forestry (ensuring the forward-only train can turn round and leave the map).
After an external train deposits it's cargo:

If the station where it offloaded cargo onloads new cargo to the train, the train will leave the map as expected
If, however, there is no cargo to put on the train, and the train has no cargo remaining onboard, the train will despawn.

I've not checked, but maybe passenger trains are showing same trait - despawning if there's no passengers for the "leaving city" journey?

It seems to also be happening with cargo ships (and possibly passenger ships too, but not tested those yet). If an external cargo ship offloads goods, it then despawns rather than returning to whence it came.

I finally found the reasons why this is happening. It's going to be a bit technical here.
Passenger trains
Passenger trains despawn under certain circumstances if the the train station where they arrive is located too close to the edge of the map. Of course, this is possible if the 81 tile mod is enabled for example, and in general there should be unproblematic.
When passenger train stations are built next to the edge of the map the game assumes that the train reached an outside connection. If there is no are no passengers to pick up the game assumes that the train may safely despawn because

it's senseless to send empty trains to the player's cities and
because a regular player could not witness the train despawning anyway.

However, the map bound condition CO once wrote uses a rather rough metric. [technical stuff ahead] It checks whether the following condition is true for the last known world position (lastFramePosition) of the train in question:
max(abs(lastFramePosition.x), abs(lastFramePosition.z)) > 4800
(code reference: PassengerTrainAI.ArriveAtTarget)
A possible solution would be therefore to reduce the threshold (4800) to some lower value. I would first have to check if other mods already provide "old-fashioned" (=non-"Harmony") detours for the PassengerTrainAI.ArriveAtTarget method (@bloodypenguin I have not checked but I strongly assume that public transport related mods like Improved Public Transport 2 override it). If that's the case this would lead to further mod incompatibilities which I do not want to accept.
Cargo trains
I have not tested cargo trains as extensively as passenger trains but a different condition seems to apply here. Cargo trains waiting at a cargo hub despawn when the hub is not able to provide any goods within ~16 seconds. [technical stuff ahead] The check is performed in the method CargoTrainAI.ArriveAtDestination() (starting with the call to CargoTruckAI.FindNextCargoParent).
Same story: If other mods override the ArriveAtDestination method for cargo trains I will not take any action here.
I will postpone the issue for at least version 1.11.
But when trains unload their passengers

That's a really nice investigration! Yes, IPT2 indeed overrides ArriveAtTarget methods, and I can try to implement the fixes there

Issue affects cargo road vehicles too.

I've noticed this is happening a lot less after the recent updates - maybe due to improved path finding increasing the likelihood that there's cargo/cims to load on to trains/ships/etc?

moved


Installation no longer works since the office update

Hi,
Unfortunatly since the office update, this package no longer work because it uses kernel32.
I had to remove all packages to be able to open my excel file.
Since, I can't call API REST.
So in my version of excel I can't install this package but I can open installer file.
It's really a shame because this code is a great job.
Regards

I've always found it easier just to import the *.bas files from the src folder of the repo than to use the installer.

I've always found it easier just to import the *.bas files from the src folder of the repo than to use the installer.

Can you elaborate on this?  Also unable to install.


Voice Control inserts spaces in input fields

Issue: Voice Control inserts spaces in input fields.
Platform: macOS 11.0.1 (20B29)
Browser: Safari 14.0.1 (166<IP_ADDRESS>.8). (Only tested in the native browser, as other vendors will likely refuse to fix anyway.)
Example page: Design System » Date input » Example
Why: When the user inputs a value, Voice Control automatically inserts a leading space, leading to validation errors and a confused user.
Suggest: Client-side trim of leading whitespace on blur()
Example 1: Input of a date on the example page:

Insert the day with the keyboard as a baseline test: 11. Then switch to using Voice Control to enter the month and year.
“01”
“Next field”
“2021”

This actually tries to submit the following three values:

"11"
" 01"
" 2021"

☝ Note the leading spaces in the inputs where Voice Control was used. Visually, they’re barely noticeable, but these will cause validation issues and force the error state. Here’s a screenshot from Safari’s web inspector, showing those values.

Example 2: Picking a country from the autocomplete:

France (No results)
sigh
“Select word. Delete previous character.”
France (1 Result)

Note: This issue doesn’t occur on mobile devices, but only because Voice Control completely refuses to see the inputs on the example page.

iPadOS 14.3 (18C66)
iOS 14.3 (18C66)

I've also seen this happen with Dragon if a user pauses between numbers or letters. I've seen it come up a few times, and using blur() as @philsherry  suggested could be a nice solution for this particular instance.
However, I think there is a guidance or education piece which needs addressing around sanitising input data before validation as it will affect more than just the date component. I've seen it happen on bank account fields and National Insurance number fields, so it then becomes a bit of a can of worms trying to fix it for people rather than getting them to understand their own validation requirements. It also means you'd still have the issue if JS didn't load.
It almost feels like we should have patterns for this kind of thing. We have visual patterns and coded components, but we don't have common sanitisation, regex's, wording etc.

I've also seen this happen with Dragon if a user pauses between numbers or letters. I've seen it come up a few times, and using blur() as @philsherry  suggested could be a nice solution for this particular instance.
However, I think there is a guidance or education piece which needs addressing around sanitising input data before validation as it will affect more than just the date component. I've seen it happen on bank account fields and National Insurance number fields, so it then becomes a bit of a can of worms trying to fix it for people rather than getting them to understand their own validation requirements. It also means you'd still have the issue if JS didn't load.
It almost feels like we should have patterns for this kind of thing. We have visual patterns and coded components, but we don't have common sanitisation, regex's, wording etc.

Having talked about this as a team, we think this is best addressed on the server-side by trimming or otherwise ignoring leading and trailing spaces as part of the validation process where appropriate. We think there'd be scope to add something about this to the 'Help users to recover from validation errors' pattern.
I've created an issue to represent this work – https://github.com/alphagov/govuk-design-system/issues/1450.
@philsherry it seems like this issue should be reported to Apple – have you raised a bug in Webkit / Radar?

Having talked about this as a team, we think this is best addressed on the server-side by trimming or otherwise ignoring leading and trailing spaces as part of the validation process where appropriate. We think there'd be scope to add something about this to the 'Help users to recover from validation errors' pattern.
I've created an issue to represent this work – https://github.com/alphagov/govuk-design-system/issues/1450.
@philsherry it seems like this issue should be reported to Apple – have you raised a bug in Webkit / Radar?

@36degrees Just done.
AX: Voice Control inserts leading spaces in input fields

@36degrees Just done.
AX: Voice Control inserts leading spaces in input fields

To add - after testing, it's not just leading and trailing whitespace, but whitespace between the digits. Dictating with Voice Control I get 2 0 2 1. I think this is probably rather unexpected and teams already stripping leading and trialing whitespace by default are unlikely to catch this.
It would be good to explicitly mention this on the date input component.

This is reported as having been fixed 'in shipping software' on the original Webkit bug as of February this year.
Testing it myself, following the steps outlined above, I am no longer seeing leading spaces nor spaces between characters when using Voice Control in Safari 15.5 (176<IP_ADDRESS>.8) on macOS 12.4 (21F79).
I think we can therefore close this issue as resolved. Thanks to @philsherry for taking the time to report it to us and to the Webkit team.

(I've also tested dictating 'two zero two one' and I get 2021 in the text input rather than 2 0 2 1, so I believe that's been fixed too)


Cannot use custom php ini

I've setup a ddev project with the following config:
APIVersion: v1.6.0
name: test-project
type: php
docroot: web
php_version: "7.2"
webserver_type: apache-cgi
router_http_port: "80"
router_https_port: "443"
xdebug_enabled: false
additional_hostnames: []
additional_fqdns: []
mariadb_version: "10.2"
webcache_enabled: false
nfs_mount_enabled: false
provider: default

I created a file at .ddev/php/my-config.ini with:
[PHP]
max_execution_time = 240
display_errors = Off

After I ddev rm  && ddev start i check phpinfo() and I still get default values for PHP config
I've tried to enter the container and under /etc/php I get my custom my-php.ini inside 7.2/fpm folder and not under 7.2/cgi folder as I think would be expected.
Am I doing anything wrong?
My version reports
bgsync          drud/ddev-bgsync:v1.6.0
cli             v1.6.0
commit          v1.6.0
db              drud/ddev-dbserver:v1.6.0-10.2
dba             drud/phpmyadmin:v1.6.0
ddev-ssh-agent  drud/ddev-ssh-agent:v1.6.0
docker          18.09.3-ce
docker-compose  1.23.2
domain          ddev.local
router          drud/ddev-router:v1.6.0
web             drud/ddev-webserver:v1.6.0

When you do a ddev start does it show something like Using custom PHP configuration: [/Users/rfay/workspace/d7git/.ddev/php/my-config.ini] ?

I tried with your exact example and it worked fine. Could you please provide the pwd from your .ddev/php directory?

Yes it shows the message every time I start
On Wed, Mar 20, 2019, 13:23 Randy Fay<EMAIL_ADDRESS>wrote:

When you do a ddev start does it show something like Using custom PHP
configuration: [/Users/rfay/workspace/d7git/.ddev/php/my-config.ini] ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/drud/ddev/issues/1499#issuecomment-474826281, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADCDYdVx2not-h60zCMn5PXQ7SL22cQRks5vYjZYgaJpZM4b_Rwf
.

Please cd .ddev/php and provide the output of pwd.
How are you looking at phpinfo? Just a phpinfo(); file?
I just noticed you're using apache-cgi and tried it, and I seem to get your result, so it sounds like a bug with the rarely-used apache-cgi configuration. I'd recommend changing to apache-fpm for the short term. I'll update the title.

I have a similar issue by using webserver_type: apache-fpm and trying to change the vhost configuration in .ddev/apache/apache-site.conf.
I have this message on start-up :
Using custom apache configuration in apache/apache-site.conf
Custom configuration takes effect when container is created,
usually on start, use 'ddev restart' if you're not seeing it take effect.

But the apache configuration in the container does not change.
I'm using ddev v1.9.1 (which is amazing!!!!!! thank you so much for it).

Sorry it was my bad. I used an environment variable in a wrong way...
I was doing
	Alias "/$SERVICE_NAME/$APP_NAME" "$WEBSERVER_DOCROOT"

instead of
	Alias /${SERVICE_NAME}/${APP_NAME} "$WEBSERVER_DOCROOT"

So my alias was not working.
You can ignore my previous comment.

I just tested this with apache-cgi and I don't see any problems with custom php usage, so closing this for now, but happy to follow up here or elsewhere.


new package: gawk 5.0.1

Description of package
Checklist

[x] Latest upstream version.
[x] I agree to maintain this package (required).
[x] I agree to notify KISS if I can no longer maintain this package (required).

I'd rather we didn't use /opt at all. Can this not simply be installed as gawk?

I'm open to other ideas, but where do you propose we put all this stuff, then without breaking busybox?
I have packaged not only gawk, but also awk, and gnucoreutils, findutils, diffutils, and sbase and ubase. All of which will conflict busybox if installed to to /usr/bin
I'm open to other locations, but let's just make sure it's consistently implemented.

And yes we could simply install this as /usr/bin/gawk , let me know if you'd prefer that, but however, the rest of the stuff still needs to be somewhere either 1. later on PATH than /bin and /usr/bin . or 2. off of PATH completely (my preference is definitely the latter)

one way to do this if you prefer no /opt would be to install all of these tools with a 'g' prefix
https://www.freshports.org/sysutils/coreutils/
that's the approach freebsd takes
However, the down side is losing the ability to quickly modify PATH for a user. Currently, for example my PATH for my user account "adam" looks like this:
/home/adam/.nix-profile/bin:/opt/gnu/coreutils/bin:/opt/gnu/findutils/bin:/opt/gnu/diffuti
ls/bin:/opt/gnu/gawk/bin:/opt/gnu/grep/bin:/opt/gnu/patch/bin:/opt/awk/bin:/home/adam/bin/bin:/usr/bin:/bin:/usr/bin
if i did this to root I'd have a broken system, but it's fine for the regular user, and it gives me a better Emacs experience, because a lot of Emacs packages expect gnu tools, sadly. But i'm pretty locked in to Emacs, and busy box with Emacs is a nightmare.

but then the user would have to symlink them all manually, one by one, into his own specially directory full of symlinks, assuming the user wanted to do what I am currently doing. A pain, but not the end of the world at all.

At that point I think it's better to use user repository for that. The main package will use the g prefix to avoid conflicts with packages from main repo, then you'd use symlinks in your personal pkgdir (e.g. /path/to/repo/coreutils/versions -> /var/db/kiss/community/versions). So you'd only have to maintain your build file in your personal repo, and maintain the "upstream" build, version, checksums, depends (if any), and sources in the community repo.

Okay, sounds like the 'g' prefix may be the goto solution for this. @dylanaraps what say you? If so, I'll get to work on it, and I'll also work on getting the rest of the suckless and gnu software out of /opt (so need to bring it up again, if this is how we choose to resolve this one.)

Yup. I agree with the g prefix.

fixed

The depends file can be removed now. :)

done 23e4c2c