DTM batching

The main goal here is to avoid having to emulate batching for every JTAG probe we add; instead queue commands within the DTM and selectively batch or write sequentially based on probe capabilities.
The extension trait technique (coined as "IDET"'s by @daniel5151, see the gdbstub docs) seems to work quite well here, probes can opt into the batching functionality, and there is a single branching path in execute to determine what to do with the probe. However, in the batching path, I'm not sure how to resume after resetting the dmi status (@bjoernQ perhaps you have an idea what to do here).
This is by no means complete, but what are your thoughts on this approach?
Edit: Also @bjoernQ would you mind testing the performance vs master? I don't think this should make a big difference, but better to check :)
Closes #790

FTDI performance is still fine here - tested with Sipeed JTAG adapter and BL602
In batching mode there is no real way out of an error and all the batched commands after the first error will error (for RequestInProgress since it's sticky). The only thing that can be done is repeat everything (or at least from the first error on) and avoid running into the error by giving the target more time
this is from the RV Debugger Spec

If a debugger sees this status, it needs to
give the target more TCK edges between Update-
DR and Capture-DR. The simplest way to do that
is to add extra transitions in Run-Test/Idle.

From my tests regarding #790 it's best to avoid running into the situation since retries are very expensive (even on Jlink w/o batching)

In batching mode there is no real way out of an error and all the batched commands after the first error will error (for RequestInProgress since it's sticky). The only thing that can be done is repeat everything (or at least from the first error on) and avoid running into the error by giving the target more time

Agreed, that's where I was heading - I guess I'll need to modify the batch API to allow being able to start again.

From my tests regarding #790 it's best to avoid running into the situation since retries are very expensive (even on Jlink w/o batching)

Thanks for the pointer; out of curiosity if logged how many resets I was getting... nearly 250! I added some code to increment the idle cycle time every reset and ended up improving my flashing time considerably (relatively speaking, my algo is still slow).

I guess I'll need to modify the batch API to allow being able to start again.

Sounds good to me.

I added some code to increment the idle cycle time every reset and ended up improving my flashing time considerably (relatively speaking, my algo is still slow).

Great - thought about something similar - good you are on it
I also tested your branch with BL602/Jlink and the performance seems good:
Wrote 16000 bytes in 1.4441618s (11079.09 bytes/s)
Read  16000 bytes in 2.8785059s ( 5558.44 bytes/s)

vs 0.11.0
Wrote 16000 bytes in 1.7198036s ( 9303.39 bytes/s)
Read  16000 bytes in 3.4725317s ( 4607.59 bytes/s)

Let me know if there is anything new to test with BL602 and/or FTDI

I've been using this locally for a few days and seems to be working well. This is now ready for review/testing.

I wonder if the IDET and the two codepaths in DTM could be avoided by having the "batch emulation" in the default trait method.
It's not possible to do that with the current "schedule" and "execute" methods, because theyrequire storing state inside the impling struct, which is not possible from default trait methods.
However, it is possible with a fn write_register_batched(&mut self, writes: Vec<Blah>) -> Result<Vec<Response>, Error>.

@Dirbaio I just pushed some changes that implement what you suggested. I do think it is a cleaner approach overall.
@bjoernQ mind testing once more with your ftdi adapter?

Looks simpler indeed :) There's one last thing that seems simplifiable, otherwise IMO this is good to go :)

A test with the Pine64 JTAG (which needs the adjustment from #807 btw) and BL602 doesn't look too good unfortunately:
probe-rs/probe-rs on  dtm-batching-experiments [!] is 📦 v0.11.0 via 🦀 v1.52.1
❯ cargo run --features ftdi --example ram_download -- --address 0x22010000 --size 4000 --chip riscv --protocol jtag
    Finished dev [unoptimized + debuginfo] target(s) in 0.28s
     Running `d:\projects\probe-rs-mabez\probe-rs\target\debug\examples\ram_download.exe --address 0x22010000 --size 4000 --chip riscv --protocol jtag`
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', probe-rs\src\probe\ftdi\mod.rs:694:43
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
error: process didn't exit successfully: `d:\projects\probe-rs-mabez\probe-rs\target\debug\examples\ram_download.exe --address 0x22010000 --size 4000 --chip riscv --protocol jtag` (exit code: 101)

this is the same on 0.11.0
Wrote 16000 bytes in 13.1917037s ( 1212.88 bytes/s)
Read  16000 bytes in 26.9076264s (  594.63 bytes/s)
Verification succesful.

this is the same on current master
Wrote 16000 bytes in 481.0215ms (33262.55 bytes/s)
Read  16000 bytes in 960.2465ms (16662.39 bytes/s)
Verification succesful.

if it helps here is the panic with RUST_BACKTRACE=1
D:\projects\probe-rs-mabez\probe-rs>cargo run --features ftdi --example ram_download -- --address 0x22010000 --size 4000 --chip riscv --protocol jtag
   Compiling probe-rs v0.11.0 (D:\projects\probe-rs-mabez\probe-rs\probe-rs)
    Finished dev [unoptimized + debuginfo] target(s) in 15.34s
     Running `target\debug\examples\ram_download.exe --address 0x22010000 --size 4000 --chip riscv --protocol jtag`
thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', probe-rs\src\probe\ftdi\mod.rs:694:43
stack backtrace:
   0: std::panicking::begin_panic_handler
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\std\src\panicking.rs:493
   1: core::panicking::panic_fmt
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\core\src\panicking.rs:92
   2: core::panicking::panic_bounds_check
             at /rustc/9bc8c42bb2f19e745a63f3445f1ac248fb015e53\/library\core\src\panicking.rs:69
   3: core::slice::index::{{impl}}::index<core::option::Option<fn(alloc::vec::Vec<u8, alloc::alloc::Global>) -> core::result::Result<u32, probe_rs::probe::DebugProbeError>>>
             at D:\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\slice\index.rs:182
   4: core::slice::index::{{impl}}::index<core::option::Option<fn(alloc::vec::Vec<u8, alloc::alloc::Global>) -> core::result::Result<u32, probe_rs::probe::DebugProbeError>>,usize>
             at D:\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\slice\index.rs:15
   5: alloc::vec::{{impl}}::index<core::option::Option<fn(alloc::vec::Vec<u8, alloc::alloc::Global>) -> core::result::Result<u32, probe_rs::probe::DebugProbeError>>,usize,alloc::alloc::Global>
             at D:\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\alloc\src\vec\mod.rs:2381
   6: probe_rs::probe::ftdi::{{impl}}::write_register_batch
             at .\probe-rs\src\probe\ftdi\mod.rs:694
   7: probe_rs::architecture::riscv::dtm::Dtm::execute
             at .\probe-rs\src\architecture\riscv\dtm.rs:95
   8: probe_rs::architecture::riscv::communication_interface::RiscvCommunicationInterface::execute
             at .\probe-rs\src\architecture\riscv\communication_interface.rs:1242
   9: probe_rs::architecture::riscv::communication_interface::RiscvCommunicationInterface::perform_memory_write_sysbus<u32>
             at .\probe-rs\src\architecture\riscv\communication_interface.rs:788
  10: probe_rs::architecture::riscv::communication_interface::RiscvCommunicationInterface::write_multiple<u32>
             at .\probe-rs\src\architecture\riscv\communication_interface.rs:1225
  11: probe_rs::architecture::riscv::communication_interface::{{impl}}::write_32
             at .\probe-rs\src\architecture\riscv\communication_interface.rs:1696
  12: probe_rs::architecture::riscv::{{impl}}::write_32
             at .\probe-rs\src\architecture\riscv\mod.rs:546
  13: probe_rs::core::{{impl}}::write_32
             at .\probe-rs\src\core\mod.rs:211
  14: ram_download::main
             at .\probe-rs\examples\ram_download.rs:77
  15: core::ops::function::FnOnce::call_once<fn() -> core::result::Result<tuple<>, anyhow::Error>,tuple<>>
             at D:\.rustup\toolchains\stable-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\ops\function.rs:227
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
error: process didn't exit successfully: `target\debug\examples\ram_download.exe --address 0x22010000 --size 4000 --chip riscv --protocol jtag` (exit code: 101)

Thanks for the backtrace @bjoernQ, that was a mistake of mine to not update the ftdi impl to use the right transforms. Hopefully it should be working now.
I simplified the impl, by removing the CommandResults and DeferredResult trait, in favour of concrete types. I think this approach is simpler, but probably less flexible. If we don't have a use case for more flexibility now, we can always add the traits, or similar traits back when we need it.

Nice - already merged - works fine for me


error[E0277]: the trait bound {integer}: SampleRange<_> is not satisfied

The issue shows on Windows 11.
I changed scheduler.rs.
add line
use rand::{Rng, distributions::Uniform};

and change line 97
let idx = rng.gen_range(0, list.len());

into
let idx = rng.sample(Uniform::new_inclusive(0, list.len()));

The problem will be solved.

Thank you for your reminder. https://github.com/zhanglun/pavo/commit/95ff60e8d22db39d9cea763499fc43f053ae9a68
Please help me to test it if you can. I will close the issue if it solved

Yep. It was solved.


Configuration

This adds configuration for api_auth, which allows more flexibility in the construction of the Authorization header and the Signature. See configuration.rb or the README for the best overview of the changes.
Also, this uses the method ActiveSupport::SecurityUtils.secure_compare does to prevent a timing attack, as discussed in #56

Thank you so much for spending time on api-auth!
I'd like to consider each subcomponent of the PR is isolation in separate PRs if possible (i.e. the different factories and signer and what not).
While I'm open to the idea of more fine grained configuration like this, I'd prefer it not be done via a global config. Doing it via config instances instead of a global config would allow for finer grained controls and would account for situations in which API-Auth needs to handle multiple use cases or needs to generate signature for multiple endpoints with different configs all within the same App. Does that make sense?
Thanks!

Yeah, makes sense. And I can split up the PR too. 👍

Closing since this PR has gone a little stale. I'd love to merge these in as smaller PRs per the discussion above, though.


Steve's UV undersides are inconsisent between models

As part of some unknowable, source-accuracy reason or something, Steve (assuming all variants+Zombie, excluding Alex/Enderman) have the bottom UV of their heads flipped vertically.
My pointers tell me that along with basing their default Steve on the pre-shaven version, they retained the UV behavior as before the following update:

This quirk does not apply to Alex's skin, and I'm uncertain regarding Enderman's (which is unique as-is).
Along with the underside of the head, I'd assume & check for the same situation for all other under-side faces.
The 2 leftmost are the same texture, 2:1 format,
Middle is as viewed in the Minecraft Launcher.
Rightmost is a 1:1 edit of the same skin applied to the Alex model.

And these are the skins used in the above screenshots:

Fixed in 1.1.0


path completion

Can the path completion with tab be changed so that you do not always see the complete path so far in the pop-up, but only the further options? With long paths, the end of the path (which then contains the files I need to choose from) fall off the screen or I have to start scrolling.

Hi, can you reopen this on the jupyter/notebook repo? JupyterHub is specifically the multi-user wrapper for notebook servers. And I would call it a general issue for long completions, with paths as a specific common case. Thanks!


Can't get GS working for some reason

I have been using gridster and all well, somehow I can't get gridstack going..it just ends up blank div.
It's probably something obvious but it's not jumping out at me.
Here is a fiddle.
https://jsfiddle.net/dschueler/vutut21k/

You need to load underscore.js as well.
On Wed, Mar 25, 2015 at 10:20 AM, dschueler<EMAIL_ADDRESS>wrote:

I have been using gridster and all well, somehow I can't get gridstack
going..it just ends up blank div.
It's probably something obvious but it's not jumping out at me.
Here is a fiddle.
https://jsfiddle.net/dschueler/vutut21k/
--
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/104.

Also, your gridstack is getting a height of 0px on load.  Maybe investigate
why that is happening.
On Wed, Mar 25, 2015 at 10:21 AM, Adam Verheyen<EMAIL_ADDRESS>wrote:

You need to load underscore.js as well.
On Wed, Mar 25, 2015 at 10:20 AM, dschueler<EMAIL_ADDRESS>wrote:

I have been using gridster and all well, somehow I can't get gridstack
going..it just ends up blank div.
It's probably something obvious but it's not jumping out at me.
Here is a fiddle.
https://jsfiddle.net/dschueler/vutut21k/
--
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/104.

I loaded underscore and still no output. Does this fiddle look correct?

Also remove the script tags from your js area.  Familiarize yourself with
how jsfiddle works and make sure youre loading all of your components
correclty first.
On Wed, Mar 25, 2015 at 10:33 AM, dschueler<EMAIL_ADDRESS>wrote:

I loaded underscore and still no output. Does this fiddle look correct?
--
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/104#issuecomment-86051725
.

Adam, I  have loaded underscore and still no output.
If you want to take a look at my fiddle check this out.
https://jsfiddle.net/dschueler/vutut21k/#&togetherjs=PCvoLFgRpi

On Mar 25, 2015, at 10:22 AM, Adam Verheyen<EMAIL_ADDRESS>wrote:
You need to load underscore.js as well.
On Wed, Mar 25, 2015 at 10:20 AM, dschueler<EMAIL_ADDRESS>wrote:

I have been using gridster and all well, somehow I can't get gridstack
going..it just ends up blank div.
It's probably something obvious but it's not jumping out at me.
Here is a fiddle.
https://jsfiddle.net/dschueler/vutut21k/
--
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/104.

—
Reply to this email directly or view it on GitHub https://github.com/troolee/gridstack.js/issues/104#issuecomment-86047859.

Don Schueler
<EMAIL_ADDRESS>Mobile:<PHONE_NUMBER>

Adam, I admit I am new to fiddle and getting used to it’s way.
I removed script tags and loading resources from the External Resources section and Frameworks and Extensions section
I appreciate your help.
Thanks..Don

On Mar 25, 2015, at 10:34 AM, Adam Verheyen<EMAIL_ADDRESS>wrote:
Also remove the script tags from your js area. Familiarize yourself with
how jsfiddle works and make sure youre loading all of your components
correclty first.
On Wed, Mar 25, 2015 at 10:33 AM, dschueler<EMAIL_ADDRESS>wrote:

I loaded underscore and still no output. Does this fiddle look correct?
--
Reply to this email directly or view it on GitHub
https://github.com/troolee/gridstack.js/issues/104#issuecomment-86051725
.

—
Reply to this email directly or view it on GitHub https://github.com/troolee/gridstack.js/issues/104#issuecomment-86052407.

Don Schueler
<EMAIL_ADDRESS>Mobile:<PHONE_NUMBER>

This still didn't work for me no matter what I did...EXCEPT one thing. I took the source from the source folder and embedded it into my page and VOILA! It all works as I would expect. If I include the minified version directly in source OR from https://cdnjs.cloudflare.com/ajax/libs/gridstack.js/0.2.2/gridstack.min.js
it does NOT work. I wonder if there is some issue with the minified versions????

Hi,
Please check it out: https://jsfiddle.net/ssfk1kfa/1/
Note, that lodash support announced in 0.2.3 only. Please also note the order of js files... gridstack.js must be included after jquery, jquery ui and underscore.


Patient names are wrong on canvas

The zip files for Patients 1 and 3 are reversed. Fix the links so that they have the correct files

@dbampalikis I assume this has been done already :-) . Can this Issue be closed?


Add dependabot

Similar to https://github.com/MeltanoLabs/meltano-map-transform/blob/main/.github/dependabot.yml, in order to keep dependencies up to date.

Closed by https://github.com/MeltanoLabs/tap-github/pull/262


Add verusfmt --check to CI

This PR adds verusfmt --check to the CI for vstd
With this commit, we should prevent drifts from happening from the initial reformat PR (#1004).
There were some minor changes to verusfmt since then (primarily fixes of indentation of code after a verus! macro), which causes a minor amount of whitespace churn in this PR, but I've marked that commit as authored by verusfmt (reminder to myself to add to .git-blame-ignore-revs right before merging; also to use a merge commit, rather than a squashing)

Added documentation as a separate section, since verusfmt will soon be applied to other files too (such as syntax.rs)---will have those changes in a separate PR, to not slow this particular PR down.
I also did a rebase onto main to update the reformatting commit, since it seems like main's vstd has moved on since #1004 while waiting on the CI being activated, and thus vstd had diverged from being properly formatted.

Thank you. Looks good, but the current instructions would only work on *nix and we have Windows users (and developers).
I added the equivalent file search to vargo, and a check that the user has a recent-enough verusfmt that we can bump occasionally in #1024. If that looks good, merge it in here, and then I think we can merge this.

I looked over that PR, and it indeed would be useful to add verusfmt to vstd fmt; however, some updates might be necessary there.
Just so that this PR is not blocked on that, I added instructions for Windows to the docs in addition to clarifying that the prior instructions were for *nix.
Additionally, I just did a rebase of this PR against latest main, to re-run verusfmt against all the updates to vstd on main.

Thanks! Did a last rebase and added the commit to .git-blame-ignore-revs. As soon as the CI goes green, will merge this in.


Pick command should be able to pick multiple notes

In 0.4.11, show command can show multiple notes at once.  If pick command could pick multiple notes, it is convenient to specify multiple notes like the following:
rbnotes pick | rbnotes show

In 0.4.11, pick command uses a picker program like fzf or peco internally.
fzf need the -m option to enable selecting multiple items.  On the other hand, peco is able to select multiple items by default with hitting ctrl-[SPACE] to select.  So, if specifying peco as picker program, the current implementation of pick could use to select multiple items.
I'm not familiar with other picker programs and their option settings to enable selecting multiple items.  It doesn't seem to be good hard-coded specifying for the picker option.  Instead, modify for pick command to refer the configuration setting for the picker option.
The configuration setting looks like:
:picker: "fzf"
:picker_option: "-m"


Enhancement: Android version?

First of all, thank you for this amazing plugin, it simplified my workflow drastically. I am just wondering here, by any chance are you planning on having this working on Android as well?

I'm also interested in Android support, but I have no recent Android device that has enough capability to run NDI.
Anyway, thanks for the input. I understood that there is some demand for it.

Just out of curiosity, which Android device are you going to use?

Thanks for the information. It looks interesting.


classs error

To keep track of composer installation, $mxc=new has bug in MxcSpot() operation.
Uncaught Error: Class 'MxcSpot' not found

To keep track of composer installation, $mxc=new has bug in MxcSpot() operation.
Uncaught Error: Class 'MxcSpot' not found

I didn't find out, is there something wrong with your PHP environment? composer is 2.0？

I don't think there is a problem in php. running other api
Composer version 2.0.11
PHP 7.1.9 on Windows NT / 10.0

I don't think there is a problem in php. running other api
Composer version 2.0.11 PHP 7.1.9 on Windows NT / 10.0

It is recommended that you try it in linux, windows will have unknown errors


Add message count option to Debug status

This includes the commits from #3544 and does the final tidy, including i18n

Coverage decreased (-0.04%) to 68.5% when pulling 973b31521e27a705b60c1242d3a5e563457b5c46 on pr_3544 into 62e730b62135860bd5fa3274c7c226baf11434b9 on dev.


#51027 Update sampling telemetry note for excludedTypes

Closes #51027
The note incorrectly states "Log sampling may cause some executions to not show up in the Application Insights monitor blade. To avoid log sampling, add samplingExcludedTypes: "Request" to the applicationInsights value.".
The property table below shows "excludedTypes" under the "samplingSettings" value, not "samplingExcludedTypes" under the "applicationInsights" value.
The note should read "Log sampling may cause some executions to not show up in the Application Insights monitor blade. To avoid log sampling, add excludedTypes: "Request" to the samplingSettings value.".

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

Thanks @collinstevens for providing a fix here. I've cherry-picked your commit into a PR in our private repo to be included in some other changes to this article.  You'll get credit for your commit, and should see the updates live by this afternoon. Thanks again! #please-close


com.android.chrome

If I search for google or chrome, I don't find the app listed in the title

looks like the problem is here:
https://github.com/onyxbits/raccoon4/blob/a7d78d4f71be221bd0ab343bf2668d10d2c7da25/src/main/java/com/akdeniz/googleplaycrawler/Utils.java#L186-L202
somewhere in there, a value some are calling voiceCapable needs to be set


ampbyexample.com i18n support

Is there way to contribute translation for ampbyexample.com i18n?
I cannot find any way to translate this example website.

Unfortunately, we currently don't support translated versions of ampbyexample.com. We are currently planning a site re-design as part of which it might become possible. Closing this issue for now, but thanks a lot for the offer!


[Suggestion] Make you able to reverse search images on right click (like TinEye extension)

Like this:

It wouldn't be more than a couple lines of code I think, so I could probably add it myself if needed

Yes, feel free to submit a PR.

@Grooty12 FYI support for Kagi in the Search by Image extension was added a couple months ago: https://github.com/dessant/search-by-image/issues/248


Use of uninitialized variables?

Hello Jiri,
I'm building your examples (though not with your makefiles) , and am getting compilation warnings from GCC 10 about how calculate_norm is passed without initialization to launch_jacobi_kernel in mpi_overlap/jacobi.cpp:
bool calculate_norm;
// ... snip ...
while (l2_norm > tol && iter < iter_max) {
    // ... snip ...
    if (use_hp_streams) {
        launch_jacobi_kernel(a_new, a, l2_norm_d, (iy_start + 1), (iy_end - 1), nx,
                             calculate_norm, compute_stream);
    }
    // ... snip ...
    calculate_norm = (iter % nccheck) == 0 || (!csv && (iter % 100) == 0);
    // ... snip ...
}

something similar is happening in single_threaded_copy/jacobi.cu

Thanks for the report. Indeed calculate_norm is uninitialized in the first iteration. Will look into a fix.

@jirikraus : This might be enough, but I haven't verified it.

Fixed with 3a7090ac0499a544ab157a6319af1c56fbba16d5. Closing this.


Fix incorrect initialization of columnSkipList for missing columns

Some columns to cstore table might be added after some data was ingested. This makes earlier stripes to not to have appropriate value for ColumnBlockSkipNode. Earlier work initialized them to be default values. However initialization code was faulty and did not initialize all blocks properly.
This issue was found by a user.
Fixes #135

:shipit:


Refactor noise residual handling in simulated data

Avoid applying map2alm twice on the noise maps.  This fixes a discrepancy at low
ell in the variance of an ensemble of noise sims.

Results agreeing

Looks good to be merged.


Add shortcut to category words contact and task

Commands can now use t and c instead of task and contact
Fixes #128

Codecov Report

Merging #167 (58b4b2e) into master (115ca10) will increase coverage by 0.02%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master     #167      +/-   ##
============================================
+ Coverage     70.40%   70.43%   +0.02%
- Complexity      720      722       +2
============================================
  Files           116      116
  Lines          2264     2266       +2
  Branches        289      291       +2
============================================
+ Hits           1594     1596       +2
  Misses          546      546
  Partials        124      124

Impacted Files
Coverage Δ

...k/logic/parser/contacts/ContactCategoryParser.java
57.14% <ø> (ø)

...askbook/logic/parser/tasks/TaskCategoryParser.java
75.00% <ø> (ø)

...ain/java/taskbook/logic/parser/TaskBookParser.java
90.00% <75.00%> (+1.11%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix: print SurfaceLine label, not id

@tormy pointed out that id was used instead of label for GeomodelLabelsLayer drawLineLabel() and drawAreaLabel(). drawAreaLabel() was already fixed though in recent commit.

We don't use the GroupAreas stuff, so I'm not sure of the expected behavior and what should be used as label. So please have a close look at it again.


2.0.1 is super unstable

Has anything changed lately that would impact stability?  I'm on Sublime@3126 with<EMAIL_ADDRESS>on<EMAIL_ADDRESS>and Sublime has been crashing a hell of a lot today.

nothing changed on our side today. can you share a log

Which log would be helpful?  Where would I find it?

I concur. It seems to be an issue with the actual typescript compiler, not specifically this plugin.
I can confirm that reverting back to https://github.com/Microsoft/TypeScript-Sublime-Plugin/tree/3475f8febcdd02b10477903652dd82a03cb3acac is the last stable release I've found usable.


SAR on GPU

Add a new algorithm that computes SAR on a GPU. Maybe use Gilbert's code from blads_sar repo?

hey @anargyri @eisber @yueguoguo, is anyone working on this at the moment? if not I would like to take the task when I come back in January

From Max:
You could try using GPU for block-matrix multiplication to scale SAR, i.e. if you have two large matrices A and B which cannot fit into memory, you can split them up into blocks, perform multiplication on GPU by batching through the blocks and then combining the results to disc again. I can point you to pyTorch and TensorFlow SAR implementations but from what I recall they were slower than CPU for anything over 1M records.

Perhaps something like this https://github.com/bamos/block but I haven't tried any of these libraries recently.

https://pandas.pydata.org/pandas-docs/stable/enhancingperf.html

closing because stale issue, feel free to reopen if needed


No notification for someone sharing its location with me

Steps to reproduce
Probably related to Benjamin-Loison/element-android/issues/2.

Where are you starting? What can you see?
What do you click?
More steps…

Outcome
What did you expect?
What happened instead?
Operating system
No response
Application version
No response
How did you install the app?
No response
Homeserver
No response
Will you send logs?
Yes

Maybe because with a too simple implementation it would trigger a notification at each location update, related to Benjamin-Loison/element-android/issues/21.


esphome: 2021.12.3 -> 2022.1.0

Motivation for this change
https://github.com/esphome/esphome/releases/tag/2022.1.0
Untested
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

ERROR Error copying file /nix/store/54ilg87nvk4wkrxl1qv5y5jjpqbj5vvq-esphome-2022.1.0/lib/python3.9/site-packages/esphome/components/esp8266/post_build.py.script to .esphome/build/esphome-02f841/post_build.py: [Errno 2] No such file or directory: '/nix/store/54ilg87nvk4wkrxl1qv5y5jjpqbj5vvq-esphome-2022.1.0/lib/python3.9/site-packages/esphome/components/esp8266/post_build.py.script'


Make RTN17 tests resilient to concurrent requests

In the test, we were checking that each fallback host got exactly one request. But that's more than we need to check; we just need to check that all fallback hosts are used, in the fake-random order we set.
That extra strictness was causing the test to fail sometimes, as some request would sneak in during the test and duplicate a fallback host. This is because we first set each fallback host as rest.prioritizedHost, then check if it works, so in between that, all requests are sent to that host.
I still don't know why did some extra requests happened since I couldn't reproduce this on my machine and we have limited visibility into CI, but the point is, there's no reason why that should affect the test.
Fixes #931.

Superseded by #961.


Add the connector component type to help charts

Add  theconnector component type to help charts. See https://github.com/open-telemetry/opentelemetry-collector/tree/main/connector

cc: @djaglowski


Added line in Contributing document to improve clarity

I added another line about cd'ing into the Platform-Game folder. I'm not 100% sure if it works the same for other OSes? But hopefully this should help keep the instructions clear for beginners who aren't very familiar with the command prompt.

@Germlord Thanks for adding that, is does make it more clear!


feat(remix-serve): Allow opting out of using source-map-support

This PR adds a way to opt out of using source-map-support when using remix-serve, which can cause issues with certain observability providers like Sentry.
Closes: https://github.com/remix-run/remix/discussions/9752

[x] Docs
[ ] Tests

Testing Strategy:
I had a hard time coming up with a way to test this in the existing testing framework. I am not too familiar with the code base. If someone can point me into the right direction I am happy to add tests! :)

Thank you for the PR but remix-serve is intentionally opinionated/unconfigurable so it can be the simplest possible solution for folks who don't want to do any customizations.
There is already a way to opt-out of source map support by using the express adapter with your own server.js file which you can customize as needed.

It's a production-ready, but basic Node.js server built with Express. If you find you want to customize it, use the @remix-run/express adapter instead.


source code for training of senseBert

guys are u planning to release sensebert training code and dataset , if so when @yoel-zeldes @oriram

Hi @papasanimohansrinivas , we'll not release the training code, as it's proprietry.
Basically the code is straightforward - we add another head, parllel to BERT's Masked LM head.
The training data is exactly the same like BERT's (Wiki and BookCorpus), with the exception that we add supersense labels to masked words (according to WordNet), as described in the paper.


Support multiple addresses in filters

Motivation
According to the ETH Wiki a filter should be able to define a contract address or a list of addresses from which logs should originate. The current state allows to define only one single address.
Solution
I added the ValueOrArray enum to the Log types and refactored some ethers-contract code with the mentioned enum.
To check the replacements there is also a new integration test in the ethers-contract package, that will deploy multiple contracts and check if the logs that originate are from the corresponding addresses.

hey - I wanted to make some formatting changes and rebase this to master, but couldn't push to your branch. closing and re-opening there!

No problem @gakonst  thanks for maintaining my commit 👍


Fixed violation of the unique constraint for v68 migration

as title

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4297   +/-   ##
=======================================
  Coverage   20.09%   20.09%
=======================================
  Files         153      153
  Lines       30696    30696
=======================================
  Hits         6168     6168
  Misses      23586    23586
  Partials      942      942

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 e2721b6...6554ff1. Read the comment docs.

@bkcsoft, I agree that it is incorrect. Tried to fix all these problems.

@lunny, Is sess.Init() correct or I've to use sess = x.NewSession(); defer sess.Close() every times?

@axifive yes please do revert xorm update

need @daviian 's approval.

@axifive Please fix suggestions from @techknowlogick otherwise LGTM


Finalize tool name

The CLI command (the binary) was originally sm but it's now smcli. The help message is therefore contradictory:
> ./smcli -h
sm is your terminal's connection to the Spacemesh network.

The sm CLI provides an ergonomic set of tools for managing a node and wallet.

Usage:
  sm [command]

Available Commands:
  completion  Generate the autocompletion script for the specified shell
  help        Help about any command
  wallet      A brief description of your command

Flags:
      --config string   config file (default is $HOME/.smcli.yaml)
  -h, --help            help for sm
  -t, --toggle          Help message for toggle

Use "sm [command] --help" for more information about a command.

Pick a name and harmonize these.

@lrettig Happy to fix it.
Do you want to go the fancy way with debug.BuildInfo to retrieve the module name auto-magically? Or do you prefer a constant which is used across the project (and can be easily overwritten when compiling)?

Yeah, I understood this tool more as the latter than the former but up to you. In the meantime I am happy to just provide a fix where the current name (smcli) is defined in a constant and used across the code base. Making it easy to change the name later either in the code or at compilation.
Would that be ok for you?

@0xjac i think that's a great idea. feel free to put me as a reviewer. thanks


Schema Change: Key behavior when components are null

If a key has an optional component that is missing or empty, keep the delimiter, e.g. consider schema (part1)-(part2)-(part3) and part2 is missing, the key would be part1--part3.  The reason for it is to avoid potential collision between keys.

Closed by #69


Run cilium-cli inside Docker

Run cilium-cli inside a container in preparation to merge cilium-cli repo to cilium repo as proposed in CFP-25694 ^1.
Move "Install Cilium CLI" step after the cluster creation step so that cilium-cli can access .kube/config file.
Run "aws configure" for workflows that run on an EKS cluster.
Set --conn-disrupt-test-restarts-path flag. The default path under /tmp doesn't work, as cilium-cli can only access the current working directory when running inside a container ^2.

this pull request is for workflows that are owned by github-sec and ci-structure without any additional owners. i opened separate pull requests for workflows that have additional owners:
servicemesh: https://github.com/cilium/cilium/pull/33724
clustermesh: https://github.com/cilium/cilium/pull/33749
ipsec: https://github.com/cilium/cilium/pull/33750
perf: https://github.com/cilium/cilium/pull/33752

/test

/test


Error 403 for 7-day weather forecast

Hi,
For a couple of weeks now I have been getting the following error message where the 7-day forecast should be:
"Cannot get 7 day weather forecast
Request failed with status code 403"
Has the weather provider changed something?
Edit: Same with the Docker version.
Regards
Max

I'm experiencing the same problem but don't know when it started.

I have the same issue, wondering if its related to the tomorrow.io free account only gets a 5 day forecast?

Could be. Have they changed that recently?

I just logged into my Tomorrow.io account and it shows no calls for my account.  Anyone know if the current weather is coming from Tomorrow or Mapbox?

I'm so glad I'm not the only one.  Been looking for this issue for at least a week or two.  I'm making the API call into tomorrow.io but just getting the 24 hour.

Let's hope someone can help :D

I messaged him on reddit, I'm hoping it's an easy fix... I'm no coder.

@jbeckley11 ... can't thank you enough for figuring this out issue and coming up with the fix.

@k5map To rebuild the project, navigate into the client folder and do npm run prod
Then go back to the app directory and do npm start

@jbeckley11 ... thanks for the commands.  When I run "npm run prod", I get this error:
pi@pi-Weather:~/pi-weather-station/client $ npm run prod

<EMAIL_ADDRESS>prod
webpack -p --env.BUILD_PRODUCTION=1

sh: 1: webpack: not found
npm ERR! code 127
npm ERR! path /home/pi/pi-weather-station/client
npm ERR! command failed
npm ERR! command sh -c webpack -p --env.BUILD_PRODUCTION=1
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/pi/.npm/_logs/2023-06-04T11_48_42_756Z-debug.log
Any ideas what could be wrong?

@k5map   not sure if its the correct way to do it but this is how I got around the webpack error.. :
from the client folder run
sudo npm install
sudo npm run prod

@ZeroCool-Dade ... that did it, Much Thanks!

#39
7 day forecast changed to 5 day forecast to meet new API requirements

@elewin... this is not a big deal but did you also update file client/src/components/weatherCharts/DailyCharts/index.js line 30 from 7 Day to 5 Day?

Yep!
https://github.com/elewin/pi-weather-station/pull/39/commits/da05477d54831834e1fc600f6ed6af68c0c4038a

@elewin Are you able to merge the fix with Sean Riggs's Docker fork?

Thanks all for their help in resolving this and @elewin for updating the code.

Yep. Thanks a lot guys :)

What about docker image?

Not sure. I haven't gotten any response regarding this topic. I guess you need to do change it yourself for now.
Install nano inside the container and change it (it's Alpine Linux btw).


Notification not working

I did all procedure as mentioned in doc also started push module by calling
FirebasePushModule.activateForFirebase();
in main activity but notifications not coming to receiver and sender have below error on logcat -
V/FirebasePushHandler: Push response: Response{protocol=h2, code=401, message=, url=https://fcm.googleapis.com/fcm/send}
Please help me to sort out notification thing.

@bensmiley , the link you gave for the notifications is referring to an obsolete or deprecated "compile" index.
Push Notifications
Add the following to your build.gradle
Add the library
Gradle
compile 'co.chatsdk.chatsdk:chat-sdk-firebase-push:4.4.8'
Manual Import
compile project(path: ':chat-sdk-firebase-push')
Enable the module
In your main class onCreate method add:
FirebasePushModule.activate(); (This I have done but the message notifications are still not showing up.)
and @paakjis , the reference code you made there on line 39 is missing from the new repo as of December, 2018.
So how do I get the notifications to work?


[POL-22] Testing outside issue

Footer

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog :^)

POL-22

Comment from GitHub to Linear ?


Various compiler warnings

And GCC6:
c:/Rtools/mingw_64/bin/g++  -std=gnu++11 -I"C:/PROGRA~1/R/R-devel/include" -DNDEBUG -I. -Ic   -I"C:/Users/Jeroen/Documents/R/win-library/3.4/Rcpp/include" -I"d:/Compiler/gcc-4.9.3/local330/include"  -D__USE_MINGW_ANSI_STDIO   -O2 -Wall  -mtune=core2 -c JsonVisitor.cpp -o JsonVisitor.o
JsonVisitor.cpp: In member function 'void facebook::graphql::ast::visitor::JsonVisitor::printChildList(const const_iterator&, size_t)':
JsonVisitor.cpp:41:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for (int ii = 0; ii < numChildren; ++ii) {
                       ^

g++ -fsanitize=address,undefined,bounds-strict -fno-omit-frame-pointer -fno-sanitize=object-size,vptr -std=gnu++11 -I/usr/local/lib/R/include -DNDEBUG -I. -Ic -I/usr/local/include -I"/usr/local/lib/R/site-library/Rcpp/include"   -fpic  -pipe -Wall -pedantic -O2 -mtune=native -c lexer.cpp -o lexer.o
lexer.lpp: In function ‘int yylex(yystype*, yy::location*, yyscan_t)’:
lexer.lpp:89:33: warning: format ‘%x’ expects argument of type ‘unsigned int*’, but argument 3 has type ‘int*’ [-Wformat=]
     sscanf(yytext + 1, "%x", &ch);

And on Xcode8:
lexer.cpp:796:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register yy_state_type yy_current_state;
        ^~~~~~~~~
lexer.cpp:797:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *yy_cp, *yy_bp;
        ^~~~~~~~~
lexer.cpp:797:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *yy_cp, *yy_bp;
        ^~~~~~~~~
lexer.cpp:798:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register int yy_act;
        ^~~~~~~~~
lexer.cpp:856:4: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
                        register YY_CHAR yy_c = yy_ec[YY_SC_TO_UI(*yy_cp)];
                        ^~~~~~~~~
lexer.cpp:1326:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *dest = YY_CURRENT_BUFFER_LVALUE->yy_ch_buf;
        ^~~~~~~~~
lexer.cpp:1327:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *source = yyg->yytext_ptr;
        ^~~~~~~~~
lexer.cpp:1328:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register int number_to_move, i;
        ^~~~~~~~~
lexer.cpp:1328:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register int number_to_move, i;
        ^~~~~~~~~
lexer.cpp:1460:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register yy_state_type yy_current_state;
        ^~~~~~~~~
lexer.cpp:1461:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *yy_cp;
        ^~~~~~~~~
lexer.cpp:1468:3: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
                register YY_CHAR yy_c = (*yy_cp ? yy_ec[YY_SC_TO_UI(*yy_cp)] : 1);
                ^~~~~~~~~
lexer.cpp:1493:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register int yy_is_jam;
        ^~~~~~~~~
lexer.cpp:1495:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register char *yy_cp = yyg->yy_c_buf_p;
        ^~~~~~~~~
lexer.cpp:1497:2: warning: 'register' storage class specifier is deprecated and incompatible with C++1z [-Wdeprecated-register]
        register YY_CHAR yy_c = 1;
        ^~~~~~~~~

Sorry for the slow response, I've just got back from vacation. These all look benign enough to me. I'll patch the signed/unsigned comparison when I get a chance.

Just pushed fixes for the warnings about register and sign comparison in JsonVisitor.cpp.

I am still seeing one warning:
c:/Rtools/mingw_64/bin/g++  -std=c++0x -I"C:/PROGRA~1/R/R-33~1.1/include" -DNDEBUG -I. -Ilibgraphqlparser   -I"C:/Program Files/R/R-3.3.1/library/Rcpp/include" -I"d:/Compiler/gcc-4.9.3/local330/include"  -D__USE_MINGW_ANSI_STDIO   -O2 -Wall  -mtune=core2 -c libgraphqlparser/lexer.cpp -o libgraphqlparser/lexer.o
/Users/swolchok/libgraphqlparser/lexer.cpp: In function 'int yy_get_next_buffer(yyscan_t)':
/Users/swolchok/libgraphqlparser/lexer.cpp:727:18: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
/Users/swolchok/libgraphqlparser/lexer.cpp:1426:3: note: in expansion of macro 'YY_INPUT'
/Users/swolchok/libgraphqlparser/lexer.cpp: In function 'yy_buffer_state* yy_scan_bytes(const char*, int, yyscan_t)':
/Users/swolchok/libgraphqlparser/lexer.cpp:1948:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]

I can't fix warnings in flex generated code manually.


Institute of Polish Language (Polish Academy of Sciences)

https://ijp.pan.pl/en/studia/

@novacellus First stage of verification passed, official domain is ijp.pan.pl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@novacellus We generally do not grant educational subscriptions to research institutes. Would you please clarify how many PhD students are in the institute, if any? How many of them need software development tools for their educational needs? Thank you.

@novacellus You might miss my previous comment. Please react.

@novacellus I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Use sbt directly in github actions

Do we need nix scripts after these changes?

I think yes. nix wasn't there just for CI, but for local development as well


Running in nested mode overwrites systemd environment variables

When running Hyprland nested for dev/debug it sets systemd  env variables DISPLAY and WAYLAND_DISPLAY to the nested values, which means any future systemd launched programs have the wrong environment and likely won't work.
Maybe just don't do the systemctl import-environment if running nested? Or allow it to be disabled in the config, since you shouldn't really be running nested without a separate config anyways

Woops, missed it. There are two places and one was missing the check. Thanks. a5a0434fffcd1b691506f694e5e9f9091b859a85


Refactor spo file roleassignment add to use util instead of calling other command, Closes #5268

Closes #5268

Thanks for another awesome PR 💪👍. You are on 🔥 today. We will review it ASAP

setting to draft due to the following observations
I will provide more guidance how we should proceed with this PR in near future


Unable to see my DevOps organizations

Hi,
Here is the link to docs
Steps to reproduce:

Click Azure devOps organizations in portal

Click My DevOps organizations

This is what I see:

Copy/past:
Enhancement failed for 'tfs.controls._type_3'.
c @ vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:69
me?mkt=en-US:43 Uncaught Error: Cannot find template with ID home-index
    at L.b.C.L.b.v.makeTemplateSource (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:141)
    at L.b.C.L.b.v.renderTemplate (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:141)
    at T (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:78)
    at b.j.Ka (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:146)
    at g (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:113)
    at Object.L.b.j (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:115)
    at Object.L.b.va (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:146)
    at update (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:148)
    at X.b.j.W (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:82)
    at g (vss-bundle-view-vBB4fHYsTDVdoaQgkQnyrST3swm8Q6mS5LnimHBgtDvk=:113)
ImageSprite.png:1 Failed to load resource: the server responded with a status of 404 ()

After several refreshes, I'm able to see my organization. So, it is not critical.
Thanks

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

ID: b1b9dd76-4131-bf69-f9bc-c7f7ef562609
Version Independent ID: 8d3bfbae-3398-c0a2-ca4d-c4782a3b4370
Content: Create a new organization, project collection - Azure DevOps
Content Source: docs/organizations/accounts/create-organization.md
Product: devops
Technology: devops-accounts
GitHub Login: @chcomley
Microsoft Alias: chcomley

@captain-yossarian Thank you for your feedback! We will review and update as appropriate.

@captain-yossarian That is the correct page, however it should look something like this:

Are you able to get this consistently?  I've found one other similar report.  If you are still running into this please let us know as we'd like to work with you to get this resolved.

@kobulloc-MSFT no worries, after I made 5-6 page refreshes it worked.
This erro is not deterministic

@captain-yossarian Thank you for the update!  I'll follow up with the product group to make sure that they are aware of the issue.
From a documentation standpoint it doesn't look like there's anything that needs to be updated so we'll close this as resolved, but if you have any additional questions about the documentation just tag me in your reply and we'll continue the conversation.


Refacto Assistant Data Source selection: folders & website under a parent

Description
Today there are several issues on our Assistant Data Sources selection modal:

If you have a lot of folders or websites the modal is not super easy to use.
There is no way to unselect a folder.
We allow on the UX to select a subpage of a website while in reality we select the full website resulting in a weird UX bug.

After a Slack thread with Ed he shared the Figma showing that for websites and folders it should be one Folder entry grouping all Folders & then a list of checkboxes.: https://www.figma.com/file/QOxHwppG64GtPocpDhS6Y7/Product?type=design&node-id=4225-28162&mode=design&t=arGAa9c9HJkzH1IH-0

If I click on Folders:

If I click on Websites:

PR is deployed on Front Edge 🙇🏻♀
It's not perfect because there's no search bar on the subscreens websites & folders but I think it's better than the bugs and we can add a search bar later if we get some user requests.
Risk
/
Deploy Plan
/

About the search bar: Ideally I would move it to the Folder & Website modal, that's where it would be the most useful. I can two in two steps (remove from this PR, add back on another one) so that I can merge this fix quicly  👍🏻
About the back button: Thanks for catching, I'll add a commit!
Thanks for the quick review 🙇🏻♀
(Leaving to go to Lorraine now but will try to apply feedback from the car)

About the search bar: Ideally I would move it to the Folder & Website modal, that's where it would be the most useful. I can two in two steps (remove from this PR, add back on another one) so that I can merge this fix quicly 👍🏻

Agreed

About the back button: Thanks for catching, I'll add a commit!

:+1:

Feedback applied 🙇🏻♀
-> Remove search bar from  Select Data sources in modal.
-> Add the search bar to the both Select Folders & Selected Websites modals, it was only a few line of codes
-> On the Select Folders & Selected Websites modals, since it's instant save we don't display Save/Cancel but just the Cross. Clicking on it returns to the previous page, the Select Data sources in  modal.
Front Edge currently deploying so easy to test again!


flutter build web needs to check for existence of index.html before attempting to parse it

This is the number 2 crasher on dev:
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/commands/build_web.dart#L102
there is some verification further down that needs to be hoisted up

If index.html doesn't not exist, should it just be ignored? or something else should be done?

We already exit if index.html is missing:
https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/web/compile.dart#L31
But this check happens after we attempt to read data from the file. Either we move the check to before L102 in build_web.dart, or vice versa


Added segmind integration

Title:

Adding support for Segmind's text to image generation models.

Related Issues:

#238

Closes #238


Add dylibs and so libs to support different architectures

Let's wait for G to review this before merging


Lfs_file_sync is returning LFS_ERR_NOSPC with alot of memory left to it's disposal.

Hello,
I'm facing som issues which I'm having a hard time understanding. I've been using lfs for some time now without experiencing this issue before, but lately this issue has been surfacing more than once.
I'm downloading data into several files, one at a time, around 10 files with approx. 80 KB data in each file. The first 9 files are succesfully filled with data and closed correctly, but when I'm trying to close the last file, I will get the LFS_ERR_NOSPC error code returned from the lfs_file_close-function, and the same thing happends if I try to call lfs_file_sync before lfs_file_close as well. The problem is consistent until I re-format the filesystem.
As I'm debugging the lfs_dir_commitattr-function I can see that off + dsize is larger than end, resulting in the following return:
dsize = 16
commit->block = 7712
commit->off = 490
commit->begin = 0
commit->end = 504
if (commit->off + dsize > commit->end) {
return LFS_ERR_NOSPC;
}
However, there is no way that my device is out of memory. There is a total of 8388608 blocks (512B each) to lfs's disposal.
When calling lfs_fs_size, I can verify that only 1960 blocks are in use and 8386648 blocks are left.
Any help would be greatly appreciated.

@Karlhead Were you able to solve this? I'm having the same issue.