Adjust for Queue v8

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.33%. Comparing base (0ea0be3) to head (89220f5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage     83.33%   83.33%
  Complexity      100      100
=========================================
  Files             9        9
  Lines           312      312
=========================================
  Hits            260      260
  Misses           52       52

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add Skiff resources

As in the contributing guidelines, I added Skiff - end-to-end encrypted mail and collaboration - to the relevant sections with a brief description.

@amilich  Added Skiff mail in f150ef76d213adf974fcbd6da7fcf118ea9308af :)

@amilich Added Skiff mail in f150ef7 :)

Fantastic!

Its privacy policy is problematic

@hwyedba4 can you elaborate please? Thank you!


Lagging of Kernel Execution

VS Code Version: 1.55.0
OS Version: Darwin x64 19.5.0

Steps to Reproduce:

SSH to a remote computer (Not sure if relevant)
Using the VSCode Notebook, connect to a kernel created by Python venv.
Run a series of cells.
Successful output of some cells, and forced stopping of process by user.
Unable to execute any further cells when running cell using Shift + Enter.
Stopped all cell.
Experimented with simple printing of ("hello") in Python.
Cell does not execute. VS Code indicates at the bottom "Executing Cell".

Does this issue occur when all extensions are disabled?: Yes/No

Successful output of some cells, and forced stopping of process by user.
Unable to execute any further cells when running cell using Shift + Enter.
Stopped all cell.

Can you explain what you are doing a little more, step by step? I can't repro an issue.
Also please include the logs from the Jupyter output channel, it might help.
cc @DonJayamanne


V0.6

Description

Motivation and Context

Fixes #
How Has This Been Tested?

Checklist:

[] I have added a Signed-off-by.
[] I have either added documentation to cover my changes or this change requires no new documentation.
[] I have either added unit tests to cover my changes or this change requires no new tests.
[] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by:

Please submit your work to https://gerrit.hyperledger.org/


Creating type aliases for public type will make the alias public regardless of it's visibility modifier

Description:
Aliasing a public type will make that aliased type public even though the alias has it's own visibility modifier.
Steps to reproduce:
module abc
type Foo int;
type Bar int | string;

module cde
import abc;
abc:Foo foo = 9;
abc:Bar bar = "dd";

above code works
Affected Versions:
0.990.3 and latest master

Tried the same on 1.0.4 and got the following errors.
$ ballerina build -a
Compiling source
        maryam/abc:0.1.0
        maryam/cde:0.1.0
error: maryam/cde:0.1.0::main.bal:3:1: attempt to refer to non-accessible symbol 'Foo'
error: maryam/cde:0.1.0::main.bal:3:1: unknown type 'Foo'
error: maryam/cde:0.1.0::main.bal:4:1: attempt to refer to non-accessible symbol 'Bar'
error: maryam/cde:0.1.0::main.bal:4:1: unknown type 'Bar'

The multiple errors should be handled with #9344.


Fix bug in localhost fallback connection

Beet certificate expired, triggered me to finally fix the localhost fallback connection

I'll update the certs repo as well


Zip operator functionality issue

It seems like Zip operator not raising onComplete until all its underlying observables are completed
This behaviour is greatly differs from previous rx implementations
For example: Zip observable array with N elements and infinite timer. In current implementation zip will never raise onCompleted
(but it should've when timer fires N+1st event, according to rxmarbles)
Is this behaviour intentional in R3's implementation?

It seems that the Zip operator in RxNet behaves in such a way that it completes when OnNext is emitted if all other observables except itself are in the OnCompleted state.
For example, if two infinite timers are included in the Zip, it is unlikely to complete.
That in itself seems like a strange behavior...

Thanks for your answer!
I was just wondering if it's intentional behaviour in new implementation because i noticed that some of my observables using zip were broken after migrating from UniRx
something like that:

now it will never call oncompleted (but previously (in unirx) oncomplete was called when the source observable ends and the number of timer ticks matched the number of elements)
If it is intentional new behaviour, that's ok. Maybe mention this changes in docs?

I think it's good to complete Zip somewhere, as it will never move and will only continue to accumulate in the Queue forever.
I think it's particularly beneficial in the case of the left-right pattern, so I'll revert some of the behavior (in your pattern, it will behave the same as UniRx).
Waiting for OnCompleted in Nth-Zip doesn't seem to be very appropriate behavior for UniRx either, so I may make changes.
I will report and implement after a little more thought on the behavior. Thank you!

v 1.2.0 changes Zip, ZipLatest, CombineLatest behavior.


Unit testing uses incorrect cache of DMOD package.

https://github.com/NOAA-OWP/DMOD/blob/f8aaea7d4933a9ac1dc014ec26503ee25b75cc7a/.github/workflows/testing_and_validation.yml#L33
The cache key is static, so as long a the cache has been built the same one will be used each time.  But since the DMOD packages are installed into this venv, if multiple changes to various package components are made in subsequent PR's, the testing is likely to fail.
Need to invalidate this venv cache when DMOD packages are changed, or not cache DMOD installs and only cache external dependencies.

Closed by #51


Add poetry to project

This will make dep changes much simpler :)

Looks good--could you please fix that ruff issue?
ruff check . --fix

should auto-fix it :)


soc: arm: atmel: sam4s: Setup external crystal only once

If the SoC uses an external main clock it should not be set both in the bootloader and the application.
Skip if the internal RC oscillator is disabled.

Please could you clarify the issue and how this resolves it?
I'm not keen on the "internal oscillator is disabled, therefore everything is configured correctly and as desired" assumption here... it may work for you, but it'll very probably bite someone else in subtle ways.
I'd much prefer a "reset to internal RC oscillator, and reconfigure the external crystal oscillator" approach, if possible.

I'd much prefer a "reset to internal RC oscillator, and reconfigure the external crystal oscillator" approach, if possible.

I've modified the code to switch to the MAIN clock (reset value) before doing the initialization.
CC: @nandojve
Two follow-up questions:

Should the same be done for sam3x/sam4e/same70/samv71?
The same70 and samv71 actually have a check, is this preferred over resetting to the MAIN clock:

	/*
	 * Setup main external crystal oscillator if not already done
	 * by a previous program i.e. bootloader
	 */

	if (!(PMC->CKGR_MOR & CKGR_MOR_MOSCSEL_Msk)) {
		// ...
	}

Do it for all of them! Just tested it on ATSAM4E16C with MCUBoot, works perfectly :)

Hi @pdgendt ,
If you want to proper fix this you need to take care of more things. All this code was implemented before MCUboot to any SAM device. In fact, the support was added not to long ago but officially we still not support on Zephyr, besides MCUboot already have all necessary details in place to make it available.
In regards of your PR, you need to setup the whole chain of registers to make sure that we can support correctly any bootloader. In the future, LPM could disable the on-chip RC oscillator and then when you select the MCK from it you stuck the device if the system do a call to the reset vector.
That said, you must assume that everything was complete reconfigured and system made a call to the reset vector:
1- Enable the internal OSC with 12MHz
2- Select MOSCSEL to internal OSC
3- Set MCK to MAINCK
4- Set EFC0->EEFC_FMR = EEFC_FMR_FWS(0);
5- Enable Clock Failure Detector
6- Configure all other clocks (except disabling the internal OSC)
7- Set EFC0->EEFC_FMR = EEFC_FMR_FWS(5); /* 120MHz */
8- Set MCK to PPLACK
9- Disable internal OSC to save power
It will be nice if you can update all the series.

Hey @nandojve,

If you want to proper fix this you need to take care of more things. All this code was implemented before MCUboot to any SAM device. In fact, the support was added not to long ago but officially we still not support on Zephyr, besides MCUboot already have all necessary details in place to make it available.

Understood, I will try to do the changes and test on sam4s, for me MCUboot is the needed use case, so official support would be nice. I do need the changes from https://github.com/zephyrproject-rtos/zephyr/pull/64215 for my board.

It will be nice if you can update all the series.

It would even be nicer to convert the Kconfig options to device tree devices and properties, and share common code between the SoCs. But this might be too big of a step to take at once.

5- Enable Clock Failure Detector

@nandojve, the datasheet has the following:

Are we guaranteed the slow RC is active?
Section 29.13 Main Clock Failure Detector however states it will be activated.

Anyway, is there an example on how the failure detection should be done as I'm not entirely sure how this works.

@bjarki-trackunit would you mind verifying the current state for the SAM4E if this still works?
@nandojve I think the only missing part is the clock failure detection now.

Any particular reason to no update same7x samv7x?

I can try, but the code was already a bit different.

Any particular reason to no update same7x samv7x?

I can try, but the code was already a bit different.

If you can incorporate the equal parts is already fine. Then we have same sequence for all series.

@nandojve I think I've made all requested changes, except the set_source with prescaler. The datasheet states that the prescaler should be set depending on the source:

For this reason I think it makes sense to keep these separate so we have this freedom in the calling code.
I can only test using SAM4S SoC, please validate the other too.
@bjarki-trackunit Can you test for the SAM4E SoC?

Hi @pdgendt ,
Needs rebase : )

@nandojve I don't have a samv71 to test, can you suggest a fix?

@nandojve I don't have a samv71 to test, can you suggest a fix?

Yes, I'll check it soon.

Hi @pdgendt ,
Just apply this patch : )
diff --git a/soc/arm/atmel_sam/common/soc_pmc.h b/soc/arm/atmel_sam/common/soc_pmc.h
index 719d728848..3be251a3d0 100644
--- a/soc/arm/atmel_sam/common/soc_pmc.h
+++ b/soc/arm/atmel_sam/common/soc_pmc.h
@@ -316,6 +316,8 @@ static ALWAYS_INLINE bool soc_pmc_osc_is_ready_main_xtal(void)
  */
 static ALWAYS_INLINE void soc_pmc_switch_mainck_to_xtal(bool bypass, uint32_t xtal_startup_time)
 {
+       soc_pmc_osc_enable_main_xtal(xtal_startup_time);
+
        /* Enable Main Xtal oscillator */
        if (bypass) {
                PMC->CKGR_MOR = (PMC->CKGR_MOR & ~CKGR_MOR_MOSCXTEN)
diff --git a/soc/arm/atmel_sam/samv71/soc.c b/soc/arm/atmel_sam/samv71/soc.c
index 6e82b9fe2f..5b446a8549 100644
--- a/soc/arm/atmel_sam/samv71/soc.c
+++ b/soc/arm/atmel_sam/samv71/soc.c
@@ -45,7 +45,6 @@ static ALWAYS_INLINE void clock_init(void)
                soc_supc_slow_clock_select_crystal_osc();
        }

-
        if (IS_ENABLED(CONFIG_SOC_ATMEL_SAMV71_EXT_MAINCK)) {
                /*
                 * Setup main external crystal oscillator.

@nandojve applied

I made another round with SAM4S, SAM4E and SAMV71. Everything working!
Thank you so much for this work @pdgendt : )


sys.stdin as UTF-8

Now I use this with nkf and pour into stdin with CP932 encoding:
$ cat sample.txt | nkf -s | python -m text2docx

because any UTF-8 Japanese text was not accepted:
$ cat sample.txt | python -m text2docx
(...)
UnicodeEncodeError: 'utf-8' codec can't encode character '\udc87' in position 45: surrogates not allowed

I think this UnicodeEncodeError occurs only Windows.
What about Linux occurs this error?
I will try to install on Linux vm later.

TextIOWrapper reconstruction seems to work better.
sys.stdin = io.TextIOWrapper(sys.stdin.buffer encoding='utf-8')

But maybe in future it is like to bothers us.
I choose to add new arg which can remove this treatment.

merged #4


feat(nd): support returning per_curiam key

This is just a small change to collect "per_curiam" now that we can ingest it in Courtlistener.
Changes the scraper and the test files

also we can use the same trick you suggested before if we change the regex to
regex = r"(?P<volume>20\d{2})\s(?P<reporter>ND)\s(?P<page>\d+)"

This is good but there a few problems.

we can improve the citation extraction which I pointed out above.
There are a tremendous number of duplicates on their page.  I count roughly 25/100.  They appear to list the same case every time even when consolidated.

one 4 times - another 16.  The only way I see being able to do this is to extract the title from the document.  or ... use the first one you get and then extract if it you can.  Its actually relatively easy to get most of hte names but when they issue corrections the PDFs are total junk

I will take advantage of this more complete refactor to implement a backscraper and address a gap #1098


feat: Run pytest in parallel

So that tests run as fast as possible, we want to run them in parallel.

Reference
Code review checklist

Removing root catalog (or changing test assertion so that root catalog may contain multiple parent or child item from another test) is a prerequisite for this piece of work. Currently, some of the tests assert that the root catalog may only contains one or two expected parent or child item. When running tests in parallel, there is a high chance that an adjacent test would write a different parent or child item to said root catalog, thus making some of the test assertion to fail.


Carrotreport

I haven't been receiving a /carrotreport prompt when joining the server for the last few days, as well as when I manually use /carrotreport, it no longer comes up with a link to the report. It simply just says "The report is being prepared..." and never actually comes. It used to take a few seconds, but I have entered the command and waited for long periods of time just to be sure, so I don't think it's just a delay at this point. This is a very useful tool for keeping up with stock, and it has also helped identify exploits and dupes in the past, so it is useful to have. I was told to provide a screenshot, so I guess I will provide the manual command not coming up with anything.

So I was banned a couple of days ago for mining in the wilderness. I mined with a fortune 2 diamond pick axe and I got 11 iron and 37 coals and some cobblestome and some sand stone, but I was thrown into sand and teleportations out. There then I was banned for no other reason than mineing, I mined with my flavorite diamond pick axe and got banned. please with out cherry on top because I am alergic to cherries unban me please.

Hellblox, you need to make your own report, not respond to someone else's. I will give you a few to copy what you have here then I will be deleting it along with my response

Can you attach your logs please to the report?

Do you mean chat logs? I could provide logs before it stopped working and after to show the difference, but other than that I'm not sure what to provide.

debug.log exactly, sometimes the error is client-side

For me,
[15:26:40] [Server thread/INFO] [minecraft/DedicatedServer]: [Master][Staff]Skelecpt: Welcome [15:26:40] [pool-3-thread-1320/ERROR] [Sponge]: The Scheduler tried to run the task CarrotShop - Report owned by Plugin{id=carrotshop, name=CarrotShop, version=1.19-hotfix1-S7.1-MC1.12.2, description=A SignShop-like shop plugin for Sponge, url=https://github.com/TheoKah/CarrotShop, authors=[Carrot], source=./mods/CarrotShop-1.19-hotfix1-S7.1-MC1.12.2.jar}, but an error occured. com.google.gson.JsonSyntaxException: com.google.gson.stream.MalformedJsonException: Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 12 path $         at com.google.gson.JsonParser.parse(JsonParser.java:65) ~[JsonParser.class:?]         at com.google.gson.JsonParser.parse(JsonParser.java:45) ~[JsonParser.class:?]         at com.carrot.carrotshop.ShopsLogs.lambda$generateReport$0(ShopsLogs.java:246) ~[ShopsLogs.class:?]         at org.spongepowered.api.scheduler.Task$Builder.lambda$execute$0(Task.java:139) ~[Task$Builder.class:1.12.2-2838-7.2.2]         at org.spongepowered.common.scheduler.SchedulerBase.lambda$startTask$0(SchedulerBase.java:197) ~[SchedulerBase.class:1.12.2-2838-7.2.2]         at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511) [?:1.8.0_262]         at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_262]         at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_262]         at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_262]         at java.lang.Thread.run(Thread.java:748) [?:1.8.0_262] Caused by: com.google.gson.stream.MalformedJsonException: Use JsonReader.setLenient(true) to accept malformed JSON at line 1 column 12 path $         at com.google.gson.stream.JsonReader.syntaxError(JsonReader.java:1559) ~[JsonReader.class:?]         at com.google.gson.stream.JsonReader.checkLenient(JsonReader.java:1401) ~[JsonReader.class:?]         at com.google.gson.stream.JsonReader.doPeek(JsonReader.java:542) ~[JsonReader.class:?]         at com.google.gson.stream.JsonReader.peek(JsonReader.java:425) ~[JsonReader.class:?]         at com.google.gson.JsonParser.parse(JsonParser.java:60) ~[JsonParser.class:?]         ... 9 more

@Hiroku Does it require an update, or is the scheduler simply the specific task breaking? I can reproduce it with another user

The carrotreport is working normally again btw


Bash not starting

Bash is not opening after closing the scheduled apt package index update task.
I tried restarting but its still not working. Whenever I run bash the command prompt simply becomes unresponsive.
Build : 14393.51
This is what happened
1> restarted computer for some reason. apt package index update started in a command prompt window. Seemed unresponsive, so I closed it.
2> tried running bash but nothing happens. It simply shows the cursor. Ctrl+C or any other combos don't work.
3> Tried restarting. still doesn't work.
[ possible related issue #422 ]

Bash is not opening after closing the scheduled apt package index update task.
I tried restarting but its still not working. Whenever I run bash the command prompt simply becomes unresponsive.

Are you using a third party firewall?  Some are known to block network traffic and cause this issue.

No third party firewall.

UPDATE:
I tried uninstalling bash using lxrun /uninstall /full
I uninstalled properly.
Now when I try to install it using either bash or lxrun /install, it gets stuck at extracting filesystem...

Same windows build, same issue here. Installation originally hung at "extracting filesystem" for a few hours...killed it, uninstalled, attempted to reinstall, same issue. No luck on my end so far.

Same windows build here. It installed fine originally and continued working fine across reboots. But after I rebooted yesterday it's now hanging. I don't remember seeing anything to do with apt-get update and I've completely disabled avast and rebooted multiple times.
While it's hung I have two init processes, one hanging off the lxssmanager svchost and the other a child of the first init. The latter sits at 25-30% cpu for as long as I've been patient to leave it (15 mins).
I've followed the instructions at https://github.com/Microsoft/BashOnWindows/issues/410#issuecomment-221463487 and sent an e-mail as instructed - I know it wasn't explicitly requested for this issue, but I figured it wouldn't hurt. The trace starts before bash is started and was stopped after it was hung for around a minute (iirc). Be warned: the zip is ~100MB because lxcore_kernel.etl is about 4GB!
I subsequently attempted a full reinstall and it hung during extraction.

I can also confirm this. I just checked it again and there are indeed two init processes. Also same behaviour with cpu usage (the first does not use any, the latter around 25%).

@aidanhs Thank you, I received your traces and will take a look this morning.

Additional notes:

I totally removed and reinstalled the linux subsystem feature, no joy
I created a new user, they also hang on install with the same isse

Hey there.
I have the same build and also got stuck without any errors.
I tried to install the "Bash on Ubuntu on Windows" feature with cmd and windows powershell.
But every time I got stuck at "extracting files".
Is there any workaround?

My bash has suddenly started working again ... I don't know what triggered it ... I restarted my pc a couple of times today and later for some reason opened powershell and typed bash and somehow it was working ... I uninstalled and reinstalled and its still working properly.

It has stopped working again after a restart

Hi, I finally found some info (this issue, #849) on this problem.  With Google I've found nothing.
I have the identical problem.  bash.exe just hangs despite multiple install/deinstall, etc.  Including an uninstall "-Full" using PowerShell of the LXSS subsystem.  Also, a newly created user using bash.exe hangs after "extracting filesystem", etc. just as previously described.
I traced the binary of bash.exe, using Process Explorer (sysinternals).  It hangs forever on WaitForMultipleObjects().  I compiled the "outbash" project (found on github, it's a reimplementation of bash.exe) and when running it, it had the same behavior.  It hung forever on WaitForMultipleObjects().
It seems to be sending a command line to some localhost port (some service?) but the service is never responding.  I also get the AptPackageIndexUpdate window popping up, it does the apt-get update, however the window hangs on the screen until I hit the close button.
This is all after using LXSS quite successfully from the release of the Anniversary Update.  It just randomly stopped working as described.  I suspect the KB3194496 cumulative update.  However, unfortunately, W10 has no way to roll back system updates which I know of.  For the time being I'll just continue with the MSYS2 project and use Linux on VirtualBox with a local Windows X11 server.
Thanks, Alex

Hi, I had a look at the debug of "cbwin" again.  It may be it's simply calling C:\Windows\System32\bash.exe and waiting for that.  But something's clearly waiting for something ... "HANG"!

I found my solution, in another thread involving "multiple pagefiles" which is now marked "closed with workaround."  Sorry I can't recall which number it is.
It was the solution to the exact problem of bash.exe hanging, AptPackageIndexUpdate window appearing and freezing, etc., at least for me.
I had changed to a C-drive pagefile of 800-1024, and in addition a system-managed size for drive D:, E:, and F:, which are all 1.8TB drives (C: is 512GB SSD).  I had then tried all uninstalls, reinstalls, etc., thinking nothing of changing the pagefiles.  I had suspected the usual: latest windows update, AVG antivirus hook DLL, etc.
Reading through the other issue, they suggested having only one pagefile as a workaround.  I simply reconfigured the pagefiles to one only on C: (system managed) and "None" on the other three.  Reboot, then bash.exe immediately shows me the Linux prompt.
Great day, thanks!
Alex

HI started using Bash on ubuntu on windows. It was working well since two months, suddenly it stopped working. I couldn't able to figure it out what exactly happened.  could you please help me to resolve this issue

@chanduch06 - Can you please provide a little bit more information?  What behavior are you experiencing?

Closing this out since @coldasparagus has resolved his issue.

I decided to just start a shell prompt and type bash in it. The error message was about the "legacy mode enabled". To change it:

Right click on the cmd prompt window.
In the option tab, uncheck the "Use legacy console"
Apply the changes and restart cmd.exe
type bash.
Installation will start.

a bit more details about @coldasparagus 's explanation

Run windows + R
type SystemPropertiesAdvanced
click settings in performance group
In advanced tab, deselect the automatically manage paging file size for all drives
Select custom size and enter 800 for initial size, 1024 for maximum size.
Apply changes and restart.

And there you go

Changing the page settings from System-managed to 800-10240 fixed it for me as well.
However, I'd like to consider this as a work-around, and not a real fix please.

A fix for this issue was included in creators update.

Unfortunately, the issue still occurred on my machine after installing the Creators Update. The workaround fixed it for me (for now).

I just tried the virtual memory fix by @coldasparagus and @imkimchi. I only have one drive and it's a 256gb SSD. The fix worked for me as well.

I just usedthe virtual memory fix by @coldasparagus and @imkimchi - works like a charm thx 👍

Hi, I'm on the last update, I tried @imkimchi fix but each time, after using bash a while, bash just handle and not responding.
I use bash to run a node.js server, when I close the bash term, localhost is still handling.
What I have to do please ? It's very annoying, and my windows was recently reset to factory (reinstall)

This worked for me as well; 10.0.16299 Win10 pro. I have a fast M2 SSD drive and 32GB of system RAM so this is surprising. The automatically managed page file reported itself as being 5120MB.
Potentially related: other programs seemed really slow to launch (under essentially idle system load) and I noticed Service Host: Local Session Manager was hogging a constant 10% of my CPU. Changing to manual pagefile both brought back WSL bash and removed the local session manager issue.

Had similar issue on
Win10Pro Build17074.rs_prerelease_flt180116-1539
was using the ubuntu subsystem perfectly fine for a while and today it stopped working for no apparent reason. following @imkimchi steps fixed it. thanks.

Thank yo @imkimchi @coldasparagus !!!

@imkimchi thank you for the clear directions! Worked for me. It just started happening yesterday on a 3-week old system.

@imkimchi Thanks! Work good for me.

@imkimchi Thanks! Work good for me.

Not enough memory will occurs and applications crash was observed after this setting take effect.

Happened to me today on version 10.0.16299.461. @imkimchi solution worked for me as well.
@benhillis Why is this issue not being re-opened? Looks like it still affects newer releases.

I would like to expand on @imkimchi solution - instead of a custom size between 800 and 1024 MB, I would suggest selecting the System managed size option or setting a much higher max size (for me the "system managed" size is 16 GB which is the same as my RAM).
With 16 GB RAM, having a page-file of only 1024 MB is rather tight and might cause many applications to crash / not start -- especially on a developer's workstation.
The workaround still works, as it seems to be mostly related to having one page-file instead of multiple files.

@benhillis 	I've also started seeing this issue intermittently. I'm on Windows 10 Pro 1809 (build 17763.316)

Already a single system-managed page file but same problem.
I tried restarting, removing/reinstalling ubuntu app, nothing.
After maybe 3-4 reinstalls, the store was telling me it was installed, but ubuntu.exe was not found and wsl.exe still hanging. Then I rebooted once more and was able to reinstall ubuntu properly.
It was a weird issues and still no clues about the real fix in this mess.

The solution as suggested in https://github.com/Microsoft/WSL/issues/849#issuecomment-322163360 did work for me. But after running for about a month, wsl has stopped responding again. I don't think the Creators Update has this fix (not doesn't have the complete fix).

I have the same problem in 1809, so I do not think Creators Update really fix it. #849 works for me.

A better solution is to stop and start the LxssManager service. Be patient, it may timeout sometimes.

I have had similar issues with this as well. The solution for me was to disable fast startup.

I have tried the same recommendations from https://github.com/microsoft/WSL/issues/849#issuecomment-322163360 and from https://github.com/microsoft/WSL/issues/849#issuecomment-501054122 and I still cannot get WSL bash to work.  I see the task start briefly then die in my task manager.  Can @benhillis please reopen this and fix it please?  I am using
OS Name	Microsoft Windows 10 Enterprise
Version	10.0.17134 Build 17134

Hello!
I have the same problem as @cmosguy, on the same OS version.
I've noticed that everytime I rebooted my computer, my Debian distro worked well. But as soon as I close it, I'm unable to get it back, I have to reboot again.
commands 'wsl' and 'bash', both in the cmd and powershell as administrators changed nothing to that problem. Changing the page/memory thing changed nothing. I couldn't stop and start again the LxssManager. (it says Stopping for a while and that's all). Disable Fast Startup changed nothing either.
OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.17134 N/A Build 17134

https://github.com/Microsoft/WSL/issues/849#issuecomment-322163360
This fix worked for me...why does this happen? Some memory fragmentation issue? Would love an explanation...

The workaround is to do a FULL reboot of Windows10, hold SHIFT and click reboot.
Normal reboot/shutdown action in the start menu will not shutdown win10 fullly, for the "fast boot" mechanism saves memory from the last session. The other comment mentioned changing system memroy page size which makes win10 do a full reboot too, works but unnecessary.
By holding SHIFT key and click the Shutdown/Reboot in start menu, win10 will go into maintain mode, choose Shutdown from the menu. Then start normally again.

Tried that and I still can not start the Ubuntu bash

Yesterday installed WLS, today it stopped working, tried combo of pagefile reconfigure + shift-reboot, started working 🤞

After I updated my Win10 to 18362.1082.
I can not make it working as usual.
Now I use bash --debug to "work around" :)

Ensure your default WSL distribution is properly configured. See what your default is with wslconfig /l. My default was set to docker. wslconfig /s Ubuntu-18.04 set Ubuntu as the default, which return the bash command as well as the Open Linux shell here context menu to working order.

@mobiusxs Had the fix.  This is still an issue on modern installs with Docker because Docker does bad things and changes your config so the wsl command stops working until you reset the default tht windows requires to use the shell you want.
Docker shouldn't be breaking installs like this, but all it takes is to update docker and WSL will break in my tests.

https://github.com/microsoft/WSL/issues/849#issuecomment-322163360
This worked for me too

Why is this issue closed?

@benhillis why closing the issue? this is not fixed

Same issue on Build 21370 Release 21024-1611. #849 (comment) worked for me. I definitely agree with everyone above, this issue shouldn't be closed...

This is ongoing... filed formal feedback... not expecting much since msft is total radio silence on this issue unexpectedly... very disappointing! The rest of msft's new turn has resulted in greater transparency but we don't ever get any from WSL team.

I just had this happen to me - I DID have a Windows update queued up. Plenty of RAM, WSL worked fine before, status all looked good. Rebooting fixed it.
That should not happen. Sorry I didn't collect any logs :( if it happens again I will.

This problem is still makes out until now.

Found a solution without restarting from StackOverflow:

First get the PID of svchost.exe running LxssManager, open the cmd as administrator and run:
tasklist /svc /fi "imagename eq svchost.exe" | findstr LxssManager
Grab the returned PID, then run task manager as administrator, in the details tab, search for the svchost.exe containing the PID, right click it and select 'end process tree'.

Been banging my head on this for hours...


Fix GetValue to fall back to BindInstance when Section.Value is null

Hi @GunterGlad, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Where's the bug for this?

And the test to go with it?

The bug is in our current dev environment. Would you like me to add a test case?

I'd like you to do a few things:

file a bug describing the issue.
write a test whenever you fix any issue.

We may not even want to fix the bug (the behavior might be by design). So filing it is a good way to see how the maintainers feel about it.

Here is the issue:
https://github.com/aspnet/Configuration/issues/409
Sorry, dont have time to write tests for you right now.


Use a QuadTree to optimise rendering

This PR attempts to implement the use of a QuadTree within AnnoCanvas.
By using a Quad Tree structure, we can speed up collision detection, as we can easily throwaway parts of the canvas that will never intersect. It means we can do collision detection without needing to check every single object in the PlacedObjects list.
Additionally, a "viewport" can be added, which would mean we would only have to process what was on screen at the time, and functions like drag all or zooming can be more easily implemented - rather than moving all the objects themselves, we can just shift the viewport, by drawing all objects with a specific offset.
Alongside the rendering improvements, this change should also allow us to add scroll bars to the canvas when its wrapped in a ScrollViewer.
To Do:

[x] Implement collision checking
[x] Check if _oldObject position bug is fixed
[x] Fix issue with viewport rect
[x] Cache GridRect in LayoutObject for use in GetObjectAt
[x] Handle items which are offscreen but still project an influence radius/range
[ ] Write tests for QuadTree
[x] ~Calling "Normalise" on large layouts is now very slow as we need to reindex the entire QuadTree.~ Solved by adding an overload to the QuadTree that allows deriving bounds from the objects themselves during a re-index operation.
[x] Implement ScrollViewer support

[x] Align grid to viewport
[x] Invert scroll direction
[x] Invert panning direction
[x] Update scroll bars on resize

[x] Handle when placing an object outside the quadtree (increase bounds and re-index)

This change is now ready. The final thing I added was a couple of options to the General Settings page to invert the panning and scrolling direction.

Can we close #43 if this PR gets merged? It would still be awkward, but scrollbars would allow a touchpad to move around the canvas.


Remove as many external dependencies as possible

The following external dependencies can be replaced by a Qt functionality:

GLEW, can be replaced by QOpenGLExtraFunctions
FreeType, might be replaceable by QFont+QPixmap+QPainter

The following external dependencies may be embedded into our repo as a submodule:

Eigen
pysigset

Other external dependencies that should be removed:

numpy (only needed for the test window, there must be some way to generate the buffers without it)
texinfo

Hell yeah!

Just integrated the Eigen source code into the repo. Sadly, there is no official git mirror for the Eigen project, therefore I had to perform a full copy of the Eigen library (which has about 5MB uncompressed).

Just integrated the Eigen source code as a submodule. Since there is no official git mirror for the Eigen project, I created a fork of my own.
I'm thinking about doing the same with pysigset, which will remove two dependencies from the installation procedure (pip3 and pysigset itself).


pkgs.nexusmods-app: 0.4.1 -> 0.6.0

Description of changes

https://github.com/Nexus-Mods/NexusMods.App/releases/tag/v0.6.0
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Needs more testing.
CLI interactions work but there's some GUI issues.


fix(contributors): SQL query to return members without a tier

Coverage remained the same at 64.375% when pulling 728a11f29ad181f120a07864825b91565f3b9f7a on fix/contributors-without-tier into 6bad41fbe8054fa05b93ec465c525189cc1d9c7e on master.


error when running generator

After installing the dependencies with npm install -g yo karma webpack webpack-dev-server typings generator-cg-angular15 and running yo I keep getting this error:
readline.js:982
            throw err;
            ^

Error: Cannot find module 'ansi-regex'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/local/lib/node_modules/generator-cg-angular15/node_modules/yeoman-generator/node_modules/chalk/node_modules/strip-ansi/index.js:2:38)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

I have already updated node, npm, yo and did some investigation work but with no success so far.

Obviously, it had nothing to do with the generator itself. It was a problem with the global npm packages that were migrated from another version of node which was installed under nvm but the cg-angular15 generator was run out of nvm scope.
Basically, a npm cleanup will solve issues like this, ie: npm cache clean, rm -rf /paths-to-npm/node-modules and checking whether there still are npm packages installed on the system (looking also in /usr/local..., ~/.nvm/..., etc)


Integrate Combined Source Code View

The first couple of commits are various small refactorings. The actual integration work happens in the last two commits (integration and test).
Test: There is actually a quite comprehensive unit-like test. On top I did a lot of manual testing.

First nit that I am doing here as a separate comment: I would squash the commits "Move TryAskingTheUserAndReadSourceFile to separate module" and "Use SourcePathsMappingUI module in OrbitQt". The first commit actually adds code and the second one removes code. Having both in one commit makes it more clear that the code was actually moved.

Thank you all for the reviews. I will rebase this, clean up all the fixup commits, conduct another round of testing, and merge it when done.


Person ReId models update

This is update for new person ReID models.
@snosov1, @sovrasov

Jenkins ok to test

@DmitriySidnev please also update https://github.com/opencv/open_model_zoo/blob/develop/models/intel/index.md


KAFKA-2643: Run mirror maker ducktape tests with SSL and SASL

Run tests with SSL, SASL_PLAINTEXT and SASL_SSL. Same security protocol is used for source and target Kafka.

Mirrormaker tests passed with all protocols - http://jenkins.confluent.io/job/kafka_system_tests_branch_builder/211/

Thanks @rajinisivaram. @granders, can you please review this?

Thanks @rajinisivaram, this looks good to me

LGTM.

@gwenshap @junrao Any of you available to review and potentially merge this? @granders and @benstopford reviewed it 6 days ago.

sorry for the delay! checking.

@gwenshap no worries about the delay, thanks for checking!


Support xpu device for 3.x ipex static

Type of Change
feature
API not changed
Description
add support to xpu device for 3.x ipex static
Expected Behavior & Potential Risk
How has this PR been tested?
local test
Dependency Change?

Will raise a PR with new design.


Get mouse trixel position lined up right

add the spawn particle at mouse position command as proof that mouse position is lined up correctly

Works but there is a tiny bit of error around game resolution vs output resolution.


Syntax error in 1.10 pre release

Hi,
I tried the 1.1.0 pre-release, but I run into some problems
I am running ruby 2.5.0 and vagrant 2.0.3 on Arch Linux.
Here is what I did:
wget https://github.com/cogitatio/vagrant-hostsupdater/releases/download/1.1.0/vagrant-hostsupdater-1.1.0.gem
vagrant plugin install vagrant-hostsupdater-1.1.0.gem
vagrant up
I get the following output:
Vagrant failed to initialize at a very early stage:

The plugins failed to load properly. The error message given is
shown below.

/home/tjaart/.vagrant.d/gems/2.5.0/gems/vagrant-hostsupdater-1.1.0/lib/vagrant-hostsupdater/HostsUpdater.rb:235: syntax error, unexpected end-of-input, expecting keyword_end

@tjaartvdwalt thanks for testing this. I will check it out and release an patch shortly. Would you be available to test it locally again for me?

I updated the release with a fix. Thank you again for testing and reporting! I'm going to close this issue, if anything else pops up just open a new issue.


Misc comments

After reading some comments in the code, it seems we have some issues to discuss:

The @law for dispersion is not working. We should at least try to identify which is the problem.
The doubt abut why a minus in the Neumann term. I think the answer is simple, since the problem is nonlinear, we are providing the solver the residual. The Neumann term is one of these terms, and in the residual it comes with a minus, right?

On the other hand, there are some additional developments:

Create tests with a method of manufactured solutions that will tell us whether the implementation is correct.
Implement the transient version of the driver using GridapODEs.

With regard to profiling:

Compare the computation time of the integration/assembly of Gridap vs FEMPAR. (Take into account that the first run involves JIT, consider a one warm-up solve first.)
Consider using a cheaper iterative e.g. multigrid solver or a PETSc solver. Right now, we are using the Julia \, not sure how efficient it is going to be.

What else?

One performance comment
I think you don't need a @law here:
https://github.com/gridapapps/GridapGeosciences.jl/blob/f77ed1002890856e8efe96513c31037da49d448d/src/DarcyElectricTransport.jl#L117
You can just use a function in terms of x, i.e.,
const n_src = ...
const xsrc = ...
const sigma_s = ...

function fe(x)  # Source as exponential function
  s = 0
  for i in 1:n_src
    s += ((-1)^(i+1))*σ_m*exp(-((x[1]-xsrc[i,1])^2+(x[2]-xsrc[i,2])^2)/(2*σ_s^2))
  end
  return s
end #function

Replace
n_src = 2
xsrc = zeros((2,2))
xsrc[1,:] = [0.4,1.0]
xsrc[2,:] = [0.6,1.0]

with
const n_src = 2
const xsrc = zeros(0.4,1.0,0.6,1.0)

for performance

One issue that is important in terms of performance/correctness @dagarcial
I think you don't need a @law here:
https://github.com/gridapapps/GridapGeosciences.jl/blob/f77ed1002890856e8efe96513c31037da49d448d/src/DarcyElectricTransport.jl#L117
You can just use a function in terms of x, i.e.,
const n_src = ...
const xsrc = ...
const sigma_s = ...

function fe(x)  # Source as exponential function
  s = 0
  for i in 1:n_src
    s += ((-1)^(i+1))*σ_m*exp(-((x[1]-xsrc[i,1])^2+(x[2]-xsrc[i,2])^2)/(2*σ_s^2))
  end
  return s
end #function

And then
function res_electrostatics(c,ϕ,ξ)
   ∇(ξ)*σ(c,∇(ϕ)) + ξ*fe
end

One issue that is important in terms of performance/correctness @dagarcial
I think you don't need a @law here:
https://github.com/gridapapps/GridapGeosciences.jl/blob/f77ed1002890856e8efe96513c31037da49d448d/src/DarcyElectricTransport.jl#L117
You can just use a function in terms of x, i.e.,
const n_src = ...
const xsrc = ...
const sigma_s = ...

function fe(x)  # Source as exponential function
  s = 0
  for i in 1:n_src
    s += ((-1)^(i+1))*σ_m*exp(-((x[1]-xsrc[i,1])^2+(x[2]-xsrc[i,2])^2)/(2*σ_s^2))
  end
  return s
end #function

And then
function res_electrostatics(c,ϕ,ξ)
   ∇(ξ)*σ(c,∇(ϕ)) + ξ*fe
end

Fixed

One issue that is important in terms of performance/correctness @dagarcial
I think you don't need a @law here:
https://github.com/gridapapps/GridapGeosciences.jl/blob/f77ed1002890856e8efe96513c31037da49d448d/src/DarcyElectricTransport.jl#L117
You can just use a function in terms of x, i.e.,
const n_src = ...
const xsrc = ...
const sigma_s = ...

function fe(x)  # Source as exponential function
  s = 0
  for i in 1:n_src
    s += ((-1)^(i+1))*σ_m*exp(-((x[1]-xsrc[i,1])^2+(x[2]-xsrc[i,2])^2)/(2*σ_s^2))
  end
  return s
end #function

And then
function res_electrostatics(c,ϕ,ξ)
   ∇(ξ)*σ(c,∇(ϕ)) + ξ*fe
end

I modified the code in accordance with this comment.

Replace
n_src = 2
xsrc = zeros((2,2))
xsrc[1,:] = [0.4,1.0]
xsrc[2,:] = [0.6,1.0]

with
const n_src = 2
const xsrc = zeros(0.4,1.0,0.6,1.0)

for performance and put the whole driver in a function for JIT compilation

Fixed. However, I code this part considering that in the future the number of sources (electrodes) will be large. I do not feel comfortable filling a 1D array to set the coordinates of 100 electrodes or more.
In the future, it must be created at least a function that goes from a list with coordinates as columns to a vector, if the vector is the optimal way.
I do not understand what is "put the whole driver in a function for JIT compilation"

You can always create your array using a function with whatever you want and then create an immutable Tuple out of it:
a = # create here your 100x100 array
const b = Tuple(a)

For the driver I mean
# Define the params you want in your driver, but it is not much important
function my_solver(order,partition)
# and all your driver here
end

order = [2,5,9,27]
partition = 100
my_solver(order,partition)

I push a new version of the code. This version is computing a steady-state Henry problem version (density-driven flow problem).
Consider that the formal henry problem is on time. However, since it reaches a steady-state work to check the code before implement the time discretization.
Solving this problem I found and issue. I haven't checked numerically, just graphically. While concentration obtained with Gridap looks similar to the papers solutions, the velocity field is not the same. I am not sure the reason, but I assume that is not a problem of Gridap but a formulation problem.
This version fixed the problem I had with the dispersion coefficient. The problem was related with the term of the formula that has the outer product of u (velocity) over the norm of u. Usually, for me uxu/|u| should not has any problem when u=0, but in this case, it does.
Additional issues:

About the Robin boundary conditions, I observed that Gridap uses FESource to impose the Neumann boundary conditions. But there is not an explicit way to tell if the integration is over the domain or over the boundary. The robin bc required a boundary integration at the LHS, including the unknown of the potential field. I can not add directly that term to the residual since Gridap will understand that the integral is over the domain, and I can not add it to the Boundary function since I required to add the unknown of the potential field. Can you tell me what is te correct way to add this kind of terms to the code?.

About the time discretization, I test the code with GridapODEs but I couldn't make it work. Do you know if ODEs is working for multiphysics?.

Hi @dagarcial

Solving this problem I found and issue. I haven't checked numerically, just graphically. While concentration obtained with Gridap looks similar to the papers solutions, the velocity field is not the same. I am not sure the reason, but I assume that is not a problem of Gridap but a formulation problem.

With this information, I can't help much. The methods you are using have been extensively tested by other developers for many different problems. So, I wouldn't expect it to be a Gridap issue, but you never know. It would be nice to identify better the problem.

This version fixed the problem I had with the dispersion coefficient. The problem was related with the term of the formula that has the outer product of u (velocity) over the norm of u. Usually, for me uxu/|u| should not has any problem when u=0, but in this case, it does.

0/0 is mathematically undetermined, so once you have it in a code, in the very rare situation it does not provide an exception (what I always expect) you would get rubish.

Additional issues:
About the Robin boundary conditions, I observed that Gridap uses FESource to impose the Neumann boundary conditions. But there is not an explicit way to tell if the integration is over the domain or over the boundary. The robin bc required a boundary integration at the LHS, including the unknown of the potential field. I can not add directly that term to the residual since Gridap will understand that the integral is over the domain, and I can not add it to the Boundary function since I required to add the unknown of the potential field. Can you tell me what is te correct way to add this kind of terms to the code?.

Look e.g. at this tutorial
https://gridap.github.io/Tutorials/dev/pages/t006_dg_discretization/#Tutorial-6:-Poisson-equation-(with-DG)-1
Note that you have a triangulation and a boundary triangulation, and a quadrature and a boundary quadrature, and then you create FETerms with the triangulation/quadrature for the bulk or boundary triangulation/quadrature for boundary terms. All you need is in this tutorial.

About the time discretization, I test the code with GridapODEs but I couldn't make it work. Do you know if ODEs is working for multiphysics?.

Yes, @oriolcg has used it for a transient fluid-structure interaction problem and Navier-Stokes incompressible.
https://github.com/gridap/Tutorials/blob/transient_nsi/src/transient_inc_navier_stokes.jl
But there is always the possibility of a bug. You could push the transient example, and create an issue with the problem. I can take a look.


Please bump ws to avoid recompilation on reinstalls

See https://github.com/websockets/ws/issues/461 ... thanks

1.4.0 uses upgrades ws to 0.7.1 to fix this (thanks to @mcollina)

Grazie!


Fix documentation tip for network/debug/remote_host editor setting

The documentation for network/debug/remote_host did not match the behavior, there is no <IP_ADDRESS> option and the correct selection for the described behavior is the device's local IP address.

Thanks!


Selection/cursor-change and edits racing condition

A known issue of this approach/example is a racing condition which makes cursors stuck/out-of-correct-position sometimes when a selection-change cursor update (which goes through cursors socket) is quickly followed by some edits (which goes through ShareDB and its socket) - like using the back arrow key with Ctrl to go to the beginning  of a word and immediately start typing.
How to replicate:

Open two side-by-side windows on https://quill-sharedb-cursors.herokuapp.com;
On one of them try to move to the beginning of a word with the keyboard shortcut (Ctrl/Option+Left Arrow) and inserting some text immediately after;
If you do these things quick enough, the cursor on the other client might become off-position.

My take is that:

Sometimes, the insert operation gets processed first (and vice-versa) on server (which moves cursor forward) and then the message with absolute position (from the arrow caret move) comes in and places the cursor behind from it is on the other client;
Sometimes the cursor doesn't even move from the place of start, meaning that it only gets updated/shifted on the insert operations, like the server hasn't ever received the caret change update/message in the first place;

One solution for this was if... cursor updates could be transmitted through ShareDB (probably as ephemeral metadata like stated on share/sharedb#128).
Still needs more testing.

@pedrosanta
I noticed this issue too. The cursor is sometimes sent before the sharedb delta operation is updated. So it sometimes forces the cursor onto a new line or it hides the cursor.
What do you think about this as a temp solution?
the quill.on 'selection-change' even is only called on initial focus and when you have selected or deselected text. It is not called on each character typed.
So the cursor updating is really dependent on doc.on 'nothing pending', debouncedSendCursorData event. Since the original timeout was really high, it caused issues.
So I reduced the timeout to 1ms. Basically always updating the cursor whenever operations have competed. This way the cursor always is updating right after the delta's have been applied.
In the selection change event I added an extra check if range.length isnt oldRange.length to make sure the selection-change event only updates when the actual selecting or deselecting of text has happend.
@pedrosanta In my testing it seems this solutions works better, and just wondering what your thoughts are?
debouncedSendCursorData = utils.debounce((->
      range = quill.getSelection()
      if range
        console.log '[cursors] Stopped typing, sending a cursor update/refresh.'
        sendCursorData range
      return
    ), 1)
    doc.on 'nothing pending', debouncedSendCursorData

quill.on 'selection-change', (range, oldRange, source) ->
      if range.length isnt oldRange.length
        sendCursorData range
      return

But I do notice one downside is the constant sending of cursor data via sockets...
Otherwise the cursor module shifts the cursor to the correct spot depending on the delta, not requiring the cursor update.
So reducing the timeout does help cursor placement but also uses more network resources...

Hey @sferoze, thank you for the comments. I've been a bit low on time in the latest few days, but let me catch this up and I'll get back to you. 👍


world.getSignText

In the world.yml the called method getSign does not exist. this has to be getSignText
and
This is the result of this method:
{"result":"success","source":"world.getSignText","success":[Ljava.lang.String;@6221b030}

Does java support ()[] syntax? I was considering using a call like this as a workaround:
{
			"name": "getSignTextLine",
			"desc": "Gets all the lines of text on a sign.",
			"returns": ["String", "The line of text on a sign."],
			"call": "this.getSignText(0,1,2,3)[4]",
			"args": [
				["String", "The world the sign is in."],
				["int", "The x-pos of the sign."],
				["int", "The y-pos of the sign."],
				["int", "The z-pos of the sign."],
				["int","The line # to get from the sign."]
			]
		},

I don't even get a result from getSignText :(

It looks like getSignText returns String[], when it SHOULD be returning List because String[] isn't even supported by JSONAPI.
Alternatively, support for converting String[] into JSON should be added.

{ u'error': u'The server is offline right now. Try again in 2 seconds.',
u'result': u'error',
u'source': u'world.getSignText'}
2012-02-20 21:40:19 [INFO] [JSONAPI] [API Call] <IP_ADDRESS>: method=world.getSignText?args=["dreckbuddler-survival", 248, 70, -314]
2012-02-20 21:40:19 [SEVERE] Exception in thread "Thread-50


league.getParticipantEntry() returns null

    RiotAPI.setMirror(Region.NA);
    RiotAPI.setRegion(Region.NA);
    RiotAPI.setAPIKey("banana");

    Summoner summoner = RiotAPI.getSummonerByName("SpiritedDusty");

    for (League league : LeagueAPI.getLeagueEntriesBySummoner(summoner)) {
    	if (league.getQueueType() == QueueType.RANKED_SOLO_5x5) {
    		System.out.println(league.getParticipantEntry());
    	}
    }

Pretty much what the title says. getParticipantEntry() returns null. I'm assuming it's supposed to return the LeagueEntry for the summoner object.

So this was the documented behavior for getParticipantEntry() on a league retrieved using getLeagueEntries, since these methods only return one entry per league. I've updated it to return the LeagueEntry instead.
This change will be in the next release, or you can get the most up-to-date snapshot at robrua.com/orianna


MUMUP-2873 Adds MyUW Lacrosse Theme back into the Java Build

Coverage remained the same at 39.199% when pulling 86de41a11597a7656ce3c47145a00d347e7e88b9 on MUMUP-2873-Lacrosse-Needs-Themes-Too into db79415b451747f50434faa87bc776b9afed6d90 on master.


Type checking - Complain if proven wrong

One idea is to use type checking where it complains if there is a proof that a variable has the wrong type. This makes it possible to specify as much type information you like.
fn foo(arr: array<number>, x: number) {
    arr[0] = x
}

fn foo(arr: array<number>, x: string) {
    arr[0] = x // ERROR: Expected `number`, found `string`
}

array can optionally specify the inner type, such that the following is ok:
fn foo(arr: array, x: number) {
    arr[0] = x
}

Conversion rules:

array <=> array<T>
option <=> option<T>
result <=> result<T>

Objects could have a struct declaration, where the known types of members are declared. This does not lock the type of the object, but controls how the object is used in various contexts. When the type of an object is specified, it is only allowed to access the members as specified by the type.
struct Point {
  x: number,
  y: number,
}

fn add(a: Point, b: Point) -> Point {
  // Requires the fields `x` and `y`
  return Point { x: a.x + b.x, y: a.y + b.y }
}

fn foo(a: Point) {
  // Not allowed because only the fields `x` and `y` can be accessed
  a.z := 0
}

The as keyword can be used to cast from one object type to another. This is required when an object is used as one type in one context and needs to be converted for calling another function. If an object is created in same scope or returned from a function, and the members are provably not a subset, then as will give an error message.
struct Foo { z: number }

a := Point { x: 0, y: 0 }
b := a as Foo // ERROR: Expected `Foo`

Conversion rules:

object <=> T
T <=> U only when T as U

A first class function requires a type. Lifetimes in function type uses indices instead of argument names. There must be one , in the function type for each argument. fn() has 0 arguments, fn(,) and one argument, fn(,,) as 2 etc.
fn foo(a, b: 'a) { ... }

fn bar(f: fn(,'0,)) { ... }

To call a function or assign it to a member of an object, the type of the object must be known.
struct Foo {
  say_hello: fn(string,),
}

x := Foo { say_hello: say_hello } // Checks the type of function

fn foo(x) {
  x.say_hello("hello") // ERROR: Type must be known for `x`
  x.say_hello = say_hello // ERROR: Type must be known for `x`
  x.say_hello := say_hello // ERROR: Function can not be added as member, use `=`
}

Accessing a first class function without calling or overwriting it is not permitted. This is to avoid run time information about first class functions to keep them safe. If we look at the case when types are unknown, it is not safe to copy the first class function. However, the run-time can not tell whether types are known in a given context or not. Therefore, when you attempt to access a first class function an error message will appear at run-time to prevent overwriting a first class function with another type.
For example, when copying a from one function to another object:
fn foo(x: Foo, y: Foo) {
  x.say_hello = y.say_hello // ERROR: Function can not be accessed from `y`
}

fn foo() {
  f := say_hello // OK, since `say_hello` is not copied from somewhere
  g := f // ERROR: Function can not be copied
}

(Needs more thinking: Copying an object with unknown type that contains a first class function, into an object treated later with a different function type. Perhaps compute a unique number for each function signature that gets checked at run-time?)
When casting to a type that contains a first class function not existing in the previous type, an unsafe_as keyword must be used.
x as Foo // ERROR: `Foo` contains a function not existing in `x`, use `unsafe_as`
x unsafe_as Foo // OK

Closing.


isMounted is deprecated

Warning: Tappable: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

Closing in favour of #111


Login failed for user

Hallo I hope someone could help me,
i installed the paperless home assistant addon.
If i try to login, I allways get the message that username and password are not matching.
in German:
"Ihr Benutzername und Kennwort stimmen nicht überein. Bitte versuchen Sie es erneut."
Log says this:
2024-01-30 20:51:41,150] [INFO] [paperless.auth] Login failed for user xxx from private IP
I deinstalled and reinstallad the addon several times. and tried other Superuser names.
Is there anything I could do?
Kind regards Max

If you forgot to edit configuration before starting the add-on it might be set to null null
I'll add an input validation to avoid that
In the meanwhile you can either try to connect to null account or delete the add on configuration folder


Added Alexis comments and very small changes to code

Pull request for data

@datalorax

Thanks Alexis! Really good feedback again. I think specifically thinking about intended audiences and best practices for communicating the info is a good thing to think about.


encoding/gob: data missing while encoding an empty *string

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8.3
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux/amd64
What did you do?
https://play.golang.org/p/3akEQ76r55
What did you expect to see?
dest.B should be ""
What did you see instead?
nil

Basics

... Pointers are not transmitted, but the things they point to are transmitted; that is, the values are flattened. ...

Encoding Details

... If a field has the zero value for its type (except for arrays; see above), it is omitted from the transmission. ...

It is working as intended/documented. (*string -> string, string == "" -> not encoded.)

Seems like @cznic is right, this is working as intended. Closing.


ValueError: Expecting byte string of length 1, got a 'str'

Upgraded from Monterey 12.2 to 12.3 (Python 2 removed in the process), and installed macadmins Python (https://github.com/macadmins/python).
now unable to set profiles:
--- Setting Color Profile(s)...
found icc PRINT profile, setting /Library/ColorSync/Profiles/Displays/wel-6b01-27_it089041_5k 160cdm_PRINT_14-01-2022.icc for iMac
/Applications/Utilities/SetColourProfile.app/Contents/Resources/tools/customdisplayprofiles:27: DeprecationWarning: This function will be removed in PyObjC 9, switch to the modern metadata system
objc.parseBridgeSupport("""
Traceback (most recent call last):
File "/Applications/Utilities/SetColourProfile.app/Contents/Resources/tools/customdisplayprofiles", line 221, in
main()
File "/Applications/Utilities/SetColourProfile.app/Contents/Resources/tools/customdisplayprofiles", line 202, in main
profile_url = Foundation.CFURLCreateFromFileSystemRepresentation(None, profile_path, len(profile_path), False)
ValueError: Expecting byte string of length 1, got a 'str'
no projector found
no external display found

Scrap my issue :) found the new branch & built from that: https://github.com/timsutton/customdisplayprofiles/tree/pyinstaller
Not quite sure how the previously uncompiled? customdisplayprofiles script I had implemented was working TBH!
Nonetheless, this has been a crucial backbone in several colour managed student labs I manage - long may it continue 👍
Thanks for this great tool!

Not quite sure how the previously uncompiled? customdisplayprofiles script I had implemented was working TBH!
Nonetheless, this has been a crucial backbone in several colour managed student labs I manage - long may it continue 👍

Ah, you found it :) I'd begun hacking that together but had put it down for a while, so I'd have been fine to merge your changes in first to have something that at least works. But maybe I'll just land what I have in its current state.


Default podman compose up configuration to a known model with cached data and demo mode enabled.

I think it may help end users who are evaluating Kai if we make the configuration as simple as we can for an easy evaluation.
For this reason I'm proposing we:

Enable DEMO_MODE to True by default in compose.yaml
Ensure we have valid cached data for running docs/scenarios/demo.md for the default model

The goal is to allow an end user to run podman compose up without modifying any env vars or kai/config.toml and then be able to use cached data for a quick evaluation of Kai.

Hey @jwmatthews , Do you mean this file docs/scenarios/demo.md? As the link mentioned above is not found.

I'm not too sure about having KAI__DEMO_MODE enabled by default. Enabling demo mode is fairly trivial (KAI__DEMO_MODE="true" podman compose up). This way, it's explicit that they're using cached data, and could avoid potential confusion if they want to generate solutions via their LLMs.

I'm not too sure about having KAI__DEMO_MODE enabled by default. Enabling demo mode is fairly trivial (KAI__DEMO_MODE="true" podman compose up). This way, it's explicit that they're using cached data, and could avoid potential confusion if they want to generate solutions via their LLMs.

I am concerned that while this is easy it is another data point of something to document/know and does add a potential friction when a user 'forgets' or doesn't see and runs without explicitly enabling demo mode.  I believe this is against how we should think for the expected persona to use podman compose up.
Please consider this from the expected user perspective of a new individual unfamiliar with Kai, first time they come in to see what it can do.  They are not yet sure if Kai is worth their time/attention...they want/need a clear signal Kai has potential to help so its worth learning more.  I would like to focus the experience of consuming podman compose up for this specific user perspective.
Then I would assume once the user has seen value in Kai, they will be motivated to read docs and explore configuration options and customize as they desire.  At this point I can see them opting in to disable demo_mode if they choose to regenerate code fixes, but I think this is the less likely desire from the user running podman compose up.  I think the consumer of podman compose up is not interested in nuances of regenerating fixes in real-time, I think they are primarily focused on assessing potential value of Kai so they can quickly show this value to others or decide for themselves if Kai is worth more of their attention.
Let's consider that I expect we have 2 user personas as of today.

Evaluator:  First time user of Kai, they are trying to determine if Kai provides value and is worth more time/energy to dive in

Deployment: podman compose up

Contributor:  They believe in Kai's vision/approach and are determined to learn/extend/use.  This may be a user helping us to improve Kai via feedback or a developer improving functionality

Deployment: make run-server

I think you're making a good point about simplifying the experience for Evaluators, but I believe there's another user group we should consider. These are users who are past the evaluation stage but aren't quite Contributors yet—they expect to use Kai in a more operational capacity, but not in demo mode.
For these users, having KAI__DEMO_MODE enabled by default might be confusing. They’d likely expect to work with live data and generate solutions with their LLMs without needing to reconfigure. While it's great to optimize for Evaluators, we might unintentionally create friction for those users who are ready to move beyond evaluation and use Kai more fully.
If we're set on having the default compose.yaml having KAI__DEMO_MODE=true, then perhaps we could create another compose-no-demo.yaml or something that disables it. Then they can run it with podman compose -f compose-no-demo.yaml up.
However, I believe it would be better for KAI__DEMO_MODE to be an opt-in feature rather than opt-out. This way, users are explicitly choosing to enter demo mode, which reduces the risk of confusion for those who intend to use the full capabilities of Kai from the start

I am fine with leaving KAI__DEMO_MODE=true in the compose.yaml for now, as long as we document this behavior somewhere, and intend for it to be reversed at some point in the future.


"simple demo" is broken

FYI
The "simple demo" is broken. The "You've selected" names always stays as "none".

tks! fixed at https://github.com/willgm/ngSelectable/releases/tag/1.0.5


Check the disabled status when adding a log provider

Please provide a description of this PR:
By checking the disabled status when adding a log provider, this PR fixes an issue where disabling a log provider through Istio telemetry API would not work. Otherwise, a disabled log provider may still be added to the log configuration and cause the disabling to not work as expected. The test case in this PR verifies that with the fix this PR, a disabled log provider will not be added to the log configuration.
To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[X ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

In response to a cherrypick label: #45366 failed to apply on top of branch "release-1.16":
Applying: Check the disabled status when adding a log provider
Using index info to reconstruct a base tree...
M	pilot/pkg/model/telemetry.go
M	pilot/pkg/model/telemetry_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/model/telemetry_test.go
CONFLICT (content): Merge conflict in pilot/pkg/model/telemetry_test.go
Auto-merging pilot/pkg/model/telemetry.go
CONFLICT (content): Merge conflict in pilot/pkg/model/telemetry.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Check the disabled status when adding a log provider
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45371

In response to a cherrypick label: #45366 failed to apply on top of branch "release-1.17":
Applying: Check the disabled status when adding a log provider
Using index info to reconstruct a base tree...
M	pilot/pkg/model/telemetry.go
M	pilot/pkg/model/telemetry_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/model/telemetry_test.go
CONFLICT (content): Merge conflict in pilot/pkg/model/telemetry_test.go
Auto-merging pilot/pkg/model/telemetry.go
CONFLICT (content): Merge conflict in pilot/pkg/model/telemetry.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Check the disabled status when adding a log provider
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #45372

In response to a cherrypick label: new pull request created: #45373


ResampleImageBySpacing causing unintended flip

ResampleImageBySpacing caused an AP and LR flip, due to ANTs handling of nifti header - as in #433

Will be resolved directly on the interface upstreamed to NiPype


Resize causes expandos to clip

Describe the bug
Resizing the dialog explorer causes the expandos to sometimes clip:

Version
https://github.com/microsoft/BotFramework-Composer-Nightlies/releases/tag/v1.1.1-nightly.178300.83c0804

@beyackle can you please take a look at this issue with the other project tree issues?

I'll take a look. I think it's a CSS problem.


[PT Run] Refactoring: combined all NativeMethods.cs files for plugins…

… into Wox.Plugin/Common/Win32/
Summary of the Pull Request
What is this about:
Moving common files used by plugins to Wox.Plugin/Common/ (currently the only common file is NativeMethods.cs).
What is included in the PR:
All NativeMethods.cs combined to Wox.Plugin/Common/Win32/.
How does someone test / validate:
By compiling.
Quality Checklist

[x] Linked issue: Resolves #15596
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

I am not a friend of splitting constants, enums and flags in two different files. Especially as there are duplicated values I think.
Can we consolidated them into one file. For me it's not clear what should be added where in the future.

I don't really like having enums and structs inside of a class, maybe we can have ConstantsAndStructs.cs and put all of them there? (in the namespace directly)

@crutkas
Thoughts?

@cyberrex5
What is the state here? Do you implement a ConstantsAndStructs.cs?

well i was waiting for the review then id implement the requested changes, but i could just do the ConstantsAndStructs.cs file now

not sure why the spell check is failing. STGM is in expect.txt

Also why are we still on c# 7.3 on .net 5?

@cyberrex5 ... lets do compartmentalize our upgrades.  I'm all for upgrading the languages more so since we are going to .net 6 in a few days.  Lets get the runtimes upgrades first

I feel like this should be one giant NativeMethods.cs  then, no?  I don't understand why we are doing some of the refactoring like it is being done

I'd also prefer everything in NativeMethods.cs than having something called ConstantsAndStructs.cs

I don't have a preference how it should be at the end. But it should be as simple as possible.

nothing its done

Great. Can you please fix the spelling that we can review and merge. Thank you.

Great. Can you please fix the spelling that we can review and merge. Thank you.

The bot is bugged out, [STGM is in expext.exe)[https://github.com/microsoft/PowerToys/blob/7984718b08df14d2fd4551b36e0c3e8256e41247/.github/actions/spell-check/expect.txt#L1946]

Great. Can you please fix the spelling that we can review and merge. Thank you.

The bot is bugged out, STGM is in expect.exe.

Try changing from STGM to stgm.

~Try changing from STGM to stgm.~ The wird is twice in the expect file. Let's remove one.

Isn't it case sensitive? if not there are other duplicates.

And I don't find the enum definition. Where is it moved to?

It's in NativeMethods.cs


Try to get everything working "cross-platform"

Cross-platform means -- at least -- Windows (with Visual Studio) and OS X (with Xamarin Studio).

The primary issue appears to be VB.NET support. This is very problematic for testing, but the core libraries DO compile without issue (at least using Xamarin Studio).


Issue #556 - Fix and improve API documentation links

Changed the docs shield to link to the fxgl-base JavaDocs (where
GameApplication is).
Added textual links to the fxgl and fxgl-base JavaDocs.

Codecov Report

Merging #557 into master will decrease coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #557      +/-   ##
===========================================
- Coverage     27.06%   26.9%   -0.16%
+ Complexity     1449    1446       -3
===========================================
  Files           253     253
  Lines         12568   12568
  Branches       1543    1543
===========================================
- Hits           3401    3382      -19
- Misses         8870    8889      +19
  Partials        297     297

Impacted Files
Coverage Δ
Complexity Δ

...ava/com/almasb/fxgl/core/collection/ObjectMap.java
39.83% <0%> (-3.99%)
35% <0%> (-3%)

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 8f83ca4...072d830. Read the comment docs.


sync: mutex.TryLock

This has come up several times on the mailing list. Looking for something like this to
be provided in sync.Mutex:

func (m *Mutex) TryLock() (locked bool) {
	locked = atomic.CompareAndSwapInt32(&m.state, 0, mutexLocked)
	if locked && raceenabled {
		raceAcquire(unsafe.Pointer(m))
	}
	return
}

Now you could use CompareAndSwapInt32 yourself. But it gets tricky when you need to use
Lock in one function, and TryLock in another on the same mutex.

The typical use-case is something like this:

if m.TryLock() {
    // do something
} else {
    // do something else
}

It's very similar to select default for channels:

select {
case <-c:
// do something
default:
// do something else
}

The difference being that with such a function you can do this with a primitive.

Other languages provide this functionality:

Java:
http://docs.oracle.com/javase/7/docs/api/java/util/concurrent/locks/Lock.html#tryLock()
Objective-C:
https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSLock_Class/Reference/Reference.html#//apple_ref/occ/instm/NSLock/tryLock

A "LockBeforeTime" would also be useful, but I understand that's more
difficult to implement.

In case someone else comes along here, I think this would work for a general-purpose non-blocking semaphore:
select {
    case v := <-tryLockSem:
        // do something
        tryLockSem <- v
    default:
        // did not get the lock
}

It's not as efficient as it could be, but it's very simple and easily understandable as it uses basic language features (and is flexible enough to allow any type in the channel).

How about creating a pool of resources (say connections or buffers) ?
Code would look like this:
func (pool *Pool) getResource() {
   for _, resource := range pool {
      if resource.lock.Try() {
         return resource
      }
   }

   pool.Lock()
   if pool.len() < MAX_POOL_SIZE {
      r := NewResource()
      pool.add(r)
      pool.Unlock()
      return r
   }

   pool.Unlock()
   // Stand in the waitgroup for resources to be freed
   // and return it
}

This is like sync.Pool but for resources that you don't want deallocated when free.
I guess the alternative using channels would have the resources send themselves
over a channel when free? Something like this:
func (pool *Pool) getResource() {
   pool.lock.Lock()
   refer pool.lock.Unlock()

   if len(pool.freeResources) != 0 {
      return <- pool.freeResources
   }

   if pool.len() < MAX_POOL_SIZE {
      return NewResource()
   }

   return <- pool.freeResources
}

func (pool *Pool) freeResource(r *Resource) {
   pool.freeResources <- r
}

The good side of using channels is that the "WaitGroup logic" is straightforward. A detail is that you always lock the pool for adquiring resources, which might have an impact if resources have a really short work-cycle. The only real disadvantage with the second case that I can see is that freeing a resource is locking if you reach channel capacity. But then again, you can always call freeResource in a goroutine if you can't estimate the buffering you are going to need. Although this would seem weird, considering you have MAX_POOL_SIZE, think of a case when MAX_POOL_SIZE is a soft limit, that is, you have two different calls for adquiring resources; getResource() and getResourceNow(). The second method does not honor the limit and is used on crucial situations when you cannot be bothered by that limit.
Would this pattern be correct?

If you are asking a question, please ask on the mailing list<EMAIL_ADDRESS> Please don't ask questions on the issue tracker.  And particularly please don't comment on closed issues; if you want to reopen one, bring that up on the mailing list as well.  Thanks.

I think a simple CAS can be employed for try-lock semantics.
// the lock guard
lockGuard := int32(0)

// code race for locking
if atomic.CompareAndSwapInt32(&lockGuard, 0, 1) { // try lock
    // do something when lock is acquired...
    lockGuard = 0
}


Unable to build Intel gptj Docker image due to installation errors

Hi all,
I'm trying to run Intel benchmarks on my server. I have the following error when trying to build gptj Docker image using this script:
#12 [conda 1/1] RUN curl -fsSL -v -o ~/miniconda.sh -O  https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh  &&     chmod +x ~/miniconda.sh &&     ~/miniconda.sh -b -p /opt/conda &&     rm ~/miniconda.sh &&     /opt/conda/bin/conda install -y python=3.9 cmake==3.22.1 &&     /opt/conda/bin/conda install -c conda-forge -y jemalloc==5.2.1 gperftools==2.10 pybind11==2.10.4 &&     /opt/conda/bin/conda install -c intel -y mkl==2023.1.0 mkl-include==2023.1.0 intel-openmp==2023.1.0 &&     /opt/conda/bin/conda clean -ya  &&     pip3 install --pre torch==2.1.0.dev20230711+cpu torchvision==0.16.0.dev20230711+cpu torchaudio==2.1.0.dev20230711+cpu --index-url https://download.pytorch.org/whl/nightly/cpu &&     pip install setuptools==58.2.0 &&     pip install cpuid nltk evaluate protobuf absl-py rouge-score==0.1.2 tqdm numpy cython sentencepiece accelerate &&     pip install git+https://github.com/intel/neural-compressor.git@a2931eaa4052eec195be3c79a13f7bfa23e54473

...

ERROR: Could not find a version that satisfies the requirement torch==2.1.0.dev20230711+cpu (from versions: 2.2.0.dev20231010+cpu, 2.2.0.dev20231204+cpu, 2.2.0.dev20231205+cpu, 2.2.0.dev20231206+cpu, 2.2.0.dev20231207+cpu, 2.2.0.dev20231208+cpu, 2.2.0.dev20231209+cpu, 2.2.0.dev20231210+cpu, 2.2.0.dev20231211+cpu, 2.2.0.dev20231212+cpu, 2.2.0.dev20231213+cpu, 2.3.0.dev20231214+cpu, 2.3.0.dev20231215+cpu, 2.3.0.dev20231216+cpu, 2.3.0.dev20231217+cpu, 2.3.0.dev20231218+cpu, 2.3.0.dev20231219+cpu, 2.3.0.dev20231220+cpu, 2.3.0.dev20231221+cpu, 2.3.0.dev20231222+cpu, 2.3.0.dev20231223+cpu, 2.3.0.dev20231224+cpu, 2.3.0.dev20231225+cpu, 2.3.0.dev20231226+cpu, 2.3.0.dev20231227+cpu, 2.3.0.dev20231228+cpu, 2.3.0.dev20231229+cpu, 2.3.0.dev20231230+cpu, 2.3.0.dev20231231+cpu, 2.3.0.dev20240101+cpu, 2.3.0.dev20240102+cpu, 2.3.0.dev20240103+cpu, 2.3.0.dev20240104+cpu, 2.3.0.dev20240105+cpu, 2.3.0.dev20240106+cpu, 2.3.0.dev20240107+cpu, 2.3.0.dev20240108+cpu, 2.3.0.dev20240109+cpu, 2.3.0.dev20240110+cpu, 2.3.0.dev20240111+cpu, 2.3.0.dev20240113+cpu, 2.3.0.dev20240114+cpu, 2.3.0.dev20240115+cpu, 2.3.0.dev20240116+cpu, 2.3.0.dev20240117+cpu, 2.3.0.dev20240118+cpu, 2.3.0.dev20240119+cpu, 2.3.0.dev20240120+cpu, 2.3.0.dev20240121+cpu, 2.3.0.dev20240122+cpu, 2.3.0.dev20240123+cpu, 2.3.0.dev20240124+cpu, 2.3.0.dev20240125+cpu, 2.3.0.dev20240126+cpu, 2.3.0.dev20240127+cpu, 2.3.0.dev20240128+cpu)
#12 167.7 ERROR: No matching distribution found for torch==2.1.0.dev20230711+cpu

Can anybody provide a solution/workaround for this? Can changing the torch version to a latest dev version affect the validity of the benchmark results?
I also had to edit the Dockerfile by adding a -y to some conda install commands to prevent a hang when waiting for user confirmation.

We had a fix for this which was to build pytorch from the git hash corresponding to the wheel file. Will just double check everything and share the recipe shortly.


NDIS interface provider providing NDIS interface services...

The words "interface" and "provider" and "service" are used so frequently in any given sentence that it becomes difficult to tease out the specific meaning, at least to someone trying to become familiar with the material. "NDIS" and "NDISIF" and "NDIS IF" may also appear multiple times in a sentence. Maybe it's just me, but it seems like this page could benefit from a writer armed with a thesaurus.

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

ID: b5e565aa-dbbf-2662-b638-ff1b9242de81
Version Independent ID: e0449c99-60ca-4894-bb70-8d41cd231e0f
Content: NDIS Network Interface Architecture - Windows drivers
Content Source: windows-driver-docs-pr/network/ndis-network-interface-architecture.md
Product: windows-hardware
GitHub Login: @duncanmacmichael
Microsoft Alias: dumacmic

Hi @druske, thanks for the feedback. I can see how this might be a bit confusing for someone learning the material, so I've made some minor style and organizational changes to help emphasize clearer meaning Unfortunately, I can't change the wording too much since the definitions are what they are, but hopefully this helps. Those changes should be live later today, Redmond time.
If you're new to NDIS and network drivers in general, I'd suggest starting with the Roadmap for Developing NDISDrivers which should help you acclimate to general terminology and ideas. In addition, the pages surrounding this one in the table of contents should help clarify exactly what NDIS network interface providers are and what they do (all the pages under the NDIS Network Interfaces doc node).
If you have specific suggestions for how to clarify the content, you're more than welcome to edit the page and submit a PR that I can then review and merge for you. Thanks!


Develop

Codecov Report

Merging #6 into master will increase coverage by 2.67%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             master     #6      +/-   ##
==========================================
+ Coverage     97.32%   100%   +2.67%
+ Complexity       58     57       -1
==========================================
  Files             2      3       +1
  Lines           112    104       -8
==========================================
- Hits            109    104       -5
+ Misses            3      0       -3

Impacted Files
Coverage Δ
Complexity Δ

src/SegmentChecker.php
100% <100%> (+3.79%)
14 <14> (-30)
:arrow_down:

src/Expression.php
100% <100%> (ø)
15 <13> (+1)
:arrow_up:

src/Validator.php
100% <100%> (ø)
28 <28> (?)

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 3ee70f0...3b1b9f2. Read the comment docs.


New Metric: Percent Recovered

This would ideally just return the percent of recovery relative to the recovery target and could be calculated using the delta index recovery metric and the target recovery value.

I might misunderstand, but doesn't the R80P index cover this?


Support AWS Deployment

Description
There is heavy AWS support in the project but deployment isn't strongly documented. For instance, I need to create a dockerrun.aws.json file in order to launch the services on elastic beanstalk (I'm still struggling to configure this properly). Would be great to have an option that injects some packaging scripts for deploying to AWS.
Make the project ready to be zip'd and uploaded directly into AWS ELB.
Rationale
This should be implemented because of AWS' popularity. Should be much simpler to deploy to the cloud right out the gate (although I understand this adds the dependency of keeping up with any AWS deployment config changes).

ELB is no longer supported in this project, see 793a46ba452

Would still be great to add built-in ECS support in lieu of ELB (sounds like @pydanny is using it for his apps). I'm having trouble figuring out how to set up all the containers in the Task Definition for the default configuration with celery and redis.

I'm trying to set up ECS @steverecio with AWS GitHub Actions so that you only need to git push. My only problem is 1. I don't want to go serverless and 2. Blue/Green deployment plus ECR means using ONE Dockerfile. Have you got any luck? I'm documenting my finds here: https://github.com/Andrew-Chen-Wang/cookiecutter-django-ecs-github
The only progress as an AWS newbie is mangling through the ECS cluster configuration. It seems like you can add volumes there instead of having volumes in the Dockerfile themselves so that the volumes can persist between builds.


add passthrough download mode,  add option to limit image proxying to a specific protocol

add passthrough download mode
It's meant to be used by image proxies that are backed by picoFeed functions
like the image proxy in miniflux.
add option to limit image proxying to a specific protocol
reverts d3785fc and implements an option to
set the targeted protocol.
One use case where the image proxy should be used with HTTPS urls is referrer
cloaking.

BTW: why are you using these image proxies :) ? do you know of the xss risk?

why are you using these image proxies

prevent mixed content warnings when connected via https to miniflux; remove the referrer to bypass hotlinking protection which is used by http://www.neowin.net/ for example

do you know of the xss risk?

Yeap. That's why miniflux uses CSP to prevent the execution of inline javascript.

You can add JavaScript in svg images ;D

https://www.owasp.org/images/0/03/Mario_Heiderich_OWASP_Sweden_The_image_that_called_me.pdf

@Raydiation How do you handle this problem (mixed content warnings) with Owncloud?

I've tried to handle it but there are no real solutions. What I do for instance is to default to prepend https when a user just enters www.site.com.
There are different security levels for mixed content: active and passive, see https://developer.mozilla.org/en-US/docs/Security/MixedContent
In regards to passive content, a MITM attack could only alter the images in the worst case but not access or modify the page. The important thing is to prevent the active content such as scripts, links, iframes, css url values and objects with the data attribte. Everything is blacklisted by content filtering using HTMLPurifier except iframes from youtube and vimeo which are usually added using https.
Now that I mention it, I should probably only whitelist https iframes xD https://github.com/owncloud/news/blob/master/appinfo/application.php#L127

Pushed a fix to ownCloud News for https iframes btw :)

BTW not all browsers fully support CSP. Firefox only recently started to support it and IE is a real problem, check http://caniuse.com/#search=csp
And dont forget the android browser, only android 4.4+ supports it

Well I've checked the image proxy with the rough svg http://www.kresin.me/javascript.svg. At least firefox, chrome & internet explorer 11 do not display the alert. That doesn't mean that it not possible, but at least it is not that easy.

One more word on this. We can only limit the attack surface, but finally it's the browsers job to prevent javascript execution from the img src attribute for example.

Btw you are just silencing the error message with the proxy. The real problem is that the original source does not offer https. Now if you are putting an image proxy in between you are still vulnerable to mitm, the problem is just shifted to the image proxy and even worse: your users think that this is now safe although it isn't really ;)
The only thing that probably helps is to write a tool that raises flags for http resources in feeds, something like the w3c validator that makes it easy to test for insecure feeds. That could apply some pressure

I'm glad that we have this discussion and generally I share your security concerns.
I've done my first MITM attack approximately a decade ago using arp spoofing and still doing MITM attacks to analyse/understand the inner workings of TLS encrypted protocols. It's a mess to see how many applications, which targeting a single hardcoded endpoint, accept certificates for this endpoint issued from any CA. Since I'm a network/system guy, I've way more knowledge about abusing the protocol layer (MITM) than the application layer (XSS).
First of all we should summarise in what situations the images proxy is tend to be used:

only if the user is connected via https to miniflux
only for images
only for images served via http (there is one possible exception to this, but this needs to be discussed later)

Furthermore we're talking about public feeds, which contain images that are used to illustrate an article. It's not about authentication data or two factor authentication qr code images.
Last but not least, the image proxy is default of and needs to be enabled by the user.

Now if you are putting an image proxy in between you are still vulnerable to mitm, the problem is just shifted to the image proxy and even worse: your users think that this is now safe although it isn't really ;)

Nope, it doesn't change the game! Independent of the usage of an image proxy, the user has no chance to notice that (s)he is under MITM attack if the images are served via HTTP.
But the image proxy moves the MITM attack vector to the connection between the server where miniflux runs and the endpoint that serves the image. Using the image proxy, it is possible to verify that unencrypted content isn't altered on the way from the miniflux server to the users browsers.
If we have a look at the server side, it's getting even worse. I assume that johnny average uses a shared hoster and simply drops of the php files into his public_html dir. Johnny has no control which certificates the local server trusts, may the php binary is altered to accept any certificates. That is the point where a MITM attack is a easy game.
On the other side I assume that shared hoster know there job. Based on my personal experiences it's more likely that the users (desktop) system is compromised by malware.

The only thing that probably helps is to write a tool that raises flags for http resources in feeds, something like the w3c validator that makes it easy to test for insecure feeds.

I'm not sure what you mean by "flags". I'm assuming you are talking about some kind of notification to the user? I even doubt that this would change anything to the majority of the users.  The average user:

doesn't care about encryption/security (I've nothing to hide)
confirms warnings faster than I could read the text
ignores errors till nothing works anymore

A few days ago someone came up with a patch to disable certificate validation completely, because his windows php binary had no ca certificates bundled. That's how users try to fix there problems.

That could apply some pressure

If you are talking about the feed creator, forget it. My personal experience is, that companies changing things only if there profit is affected. RSS feeds are more less a disaster for companies that make profit with there textual content. Especially in conjunction with content grabber. No page impressions, no advertisement and so on.

I read a bit about the svg vulnerability. As far I understood it, it is limited to svg in conjunction with object or embed tags. The image tag is not vulnerable.
@Raydiation It's all well. Every notice about a possible security problem is welcome.
@fguillot I've pushed the miniflux counterpart to https://github.com/mkresin/miniflux/tree/imageproxy. I tend to use the referrer cloaking even with content served via https. Well, it breaks the end-to-end encryption, but otherwise the images are not shown for the user.

I'm not entirely happy with the disablePassthroughMode() function. Neither am I sure that the function is really needed nor that the function name is good one. Maybe it should be named something like enableDownloadMode().
I've started using the Client classes of picoFeed for another project. Have you ever considered to split of the Client classes into a picoDownload?

@mkresin I didn't plan to make a picoClient or picoDownload library, usually people use popular libraries like https://github.com/guzzle/guzzle or https://github.com/kriswallsmith/Buzz


How do I access file modification time as template variable?

I would like to use the file modification time of any file in _posts to display:

Last updated: Jan 21, 2010

at the top of my posts.
Is this possible?
I know I could add this template variable into the YAML Front Matter myself, but that's annoying. I'd only want to do that to override the file modification time.
Thanks!
Matt

#154 is that what are you looking for?