Cannot Access WSL2 File System from Explorer at Startup [Build 20215]

Environment
Windows build number: 10.0.20215.1000
Your Distribution version: 20.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce

Every new startup of Windows 10 Build 20215 has this issue.

Expected behavior

WSL2 file system should be automatically connected to Windows10 via 9p at startup.
Actual behavior

On a new startup of Windows 10 Build 20215: In the Explorer, the  Penquin icon is there with Ubuntu folder inside. But clicking on the folder shows folder not accessible.
In Windows Terminal, my Ubuntu terminal would start with ~ directory when under Windows Build 20201. But now it starts with /mnt/c/User/XXX in Windows Build 20215. I CAN cd into Ubuntu file system without problem though.
Run wsl --shutdown and restart WSL2 in Powershell as admin solve the problem. Now Explorer can access the Ubuntu file system and Windows Terminal (Ubuntu Terminal actually) starts correctly with ~ correctly.
Restart Windows will reproduce this issue again.

P.S.: My Ubuntu is on an independent internal disk other than Drive C:. It has worked fine since Build 20150 until now. I did skip the builds after 20201 and before 20215.

same +1

same +1

same +1

Thank you to report it. I tried but I didn't reproduce the problem. Can you help to collect wsl logs by Feedback Hub?
When the problems happen, what's the output of wsl -l -v? If the distro is up and running, can you login to the distro and dmesg |grep 9p
When you click Ubuntu and other distro in File Explorer, and if distro is down, it brings up the distro automatically.  When folder is 'not accessible', most likely the distro is not up or 9p service is not up correctly.
@benhillis Do you know about  ~ is changed to /mnt/c/User/XXX?

Hi @licanhua ,
Thanks for you response. I've uploaded my screen record to Feedback Hub.
Here is the output for  wsl -l -v:
PS C:\WINDOWS\system32> wsl -l -v
  NAME            STATE           VERSION
* Ubuntu-20.04    Running         2

I can login to the distro via Windows Terminal and here is the output for dmesg |grep 9p:
(base) xxxx@DESKTOP-XXXXXXX:~$ dmesg | grep 9p
[    0.942551] 9p: Installing v9fs 9p2000 file system support
[    0.942558] FS-Cache: Netfs '9p' registered for caching
[    0.985865] 9pnet: Installing 9P2000 support
[    1.616591] 9pnet_virtio: no channels available for device drvfs
[    1.722666] 9pnet_virtio: no channels available for device drvfs
[    1.829245] 9pnet_virtio: no channels available for device drvfs
[    1.935976] 9pnet_virtio: no channels available for device drvfs

Any suggestions?

Add @SvenGroot

PS C:\Users\benja> wsl -l -v
  NAME      STATE           VERSION
* Ubuntu    Stopped         2

Cannot start WSL either:
The remote procedure call failed.

@dg520 Your log is not enough for troubleshooting. no channels available for device drvf here means Linux has problem(or waiting for PCI ready) to mount windows file system like /mnt/c. But your problem is to access Linux file system.
@B3Kay If you are with Windows Insider build, you may hit #5907. Fix is on the way

Hi @licanhua ,
Could you please let me know what I can do to provide more related logs? Do I need something like dmesg |grep 9p but run on the Powershell side?
Best,
Dadi

collect wsl logs by Feedback Hub and then share the feedback hub link.

@licanhua Here is my feedback hub link

@dg520 Thanks to provide the log, but I didn't find the log related to WSL. I want to confirm with you if you were doing like this:

After your machine is up, start Feedback Hub, fill the information, and 'start recording'
Open File Explorer to reproduce the problem
Run 'wsl -l -v' to check the status
stop recording and finish send out feedback.

@licanhua , I tried your instruction again. Does this link contain the log now?

@dg520 this log is valid. thanks. I will follow up with this if there is update.

Duplicate of #5902

@xds95 I think this is different than #5902. WSL2 is running in my case and can be used in via terminal. It is just not accessible in explorer on a new Windows start.

I think it's a different issue too?
@dg520 From the log, it's related to user's token. Have you disabled UAC? You can see the settings from Change User Account Control settings
What's your logon account, built-in administrator account, or a non-administrator account?

@licanhua Thanks for the insight. My UAC is left as default (Notify me only apps try to make changes to my computer (default)). My logon account is the built-in administrator account, with my Microsoft account logged at the same time.

PS C:\Users\benja> wsl -l -v
  NAME      STATE           VERSION
* Ubuntu    Stopped         2

Cannot start WSL either:
The remote procedure call failed.

Same Build 20221, since 20211 ...

remote procedure call failed is related to #5902. The fix is on the way, and please check that issue for update

@licanhua I seems found and partially fixed the issue of my case.
I have a ps1 script to automatically start WSL2 and its services, run by Windows Task Scheduler. The script looks like:
###############################################################
#Start Ubuntu Services, mimic systemd in a real Ubuntu machine
###############################################################
iex "wsl -u root sudo rstudio-server start";
iex "wsl -u root sudo service nginx start";
iex "wsl -u root sudo service ssh start";

###############################################################
#Forward WSL2 ports to Windows ports
###############################################################
$remoteport = bash.exe -c "ifconfig eth0 | grep 'inet '"
$found = $remoteport -match '\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}';

if( $found ){
  $remoteport = $matches[0];
} else{
  echo "The 'wsl-port-forwarding' script exited, the ip address of WSL 2 cannot be found";
  exit;
}

#All the ports you want to forward
$ports=@(8762,8772);

#[Static ip]
#You can change the addr to your ip config to listen to a specific address
$addr='<IP_ADDRESS>';
$ports_a = $ports -join ",";

for( $i = 0; $i -lt $ports.length; $i++ ){
  $port = $ports[$i];
  iex "netsh interface portproxy delete v4tov4 listenport=$port listenaddress=$addr";
  iex "netsh interface portproxy add v4tov4 listenport=$port listenaddress=$addr connectport=$port connectaddress=$remoteport";
}

And I set this scheduler task as Run whether user is logged on or not. This used to work fine before Windows Build 20215, meaning 1) WSL2 would start; 2) the WSL2 service would be activated, 3) the ports would be forwarded; and 4) Windows explorer can visit WSL2 system.
Since Build 20215, 1), 2) and 3) still works but 4) does not work. I've tried with and without Run with highest privileges selected.
The workaround I've found is to change Run whether user is logged on or not to Run only when user is logged on. Then Windows will be able to visit WSL2 without a permission error.
Is there a way to make this work at system startup without logged on?

Start WSL service without user login is really a key feature and many of users required for that. Hope that WSL team could managed to implement that.


Revert accidental merge of dummy PR#99.

Hi @kutschke,
You have proposed changes to files in these packages:

/

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for 9a447a1673cdecb3a4fb315493b14eda13f94a34: build  (Build queue has 1 jobs)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The tests passed at 9a447a1673cdecb3a4fb315493b14eda13f94a34.

Test
Result
Details

merge
:white_check_mark:
Merged 9a447a1673cdecb3a4fb315493b14eda13f94a34 at e02a4b7711b133cfbd70635f0c0669c7f5ece511

build (prof)
:white_check_mark:
Log file. Build time: 14 min 38 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 9a447a1673cdecb3a4fb315493b14eda13f94a34 after being merged into the base branch at e02a4b7711b133cfbd70635f0c0669c7f5ece511.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


Property duplication

There is an incorrect possibility to have a BSONDocument  with two (or even more) same properties:
scala> (BSONDocument("name" -> "Initial Value") ++ BSONDocument("name" -> "Changed Value")).elements.toList
res14: List[reactivemongo.bson.BSONElement] = List((name,BSONString(Initial Value)), (name,BSONString(Changed Value)))

I am using the <IP_ADDRESS>.akka23 version.
The reactivemongo driver even successfully inserts in the mongo instance such documents.
From the mongo console:
> db.users.find({})
{ "_id" : ObjectId("54832b6b148d380065a1c00d"), "name" : "Initial Value", "name" : "Changed Value"}

Hi @kaygorodov , I'm not sure to understand why you say it's incorrect. MongoDB allows such document, so the drive behaves in the same way for me.

Hi @cchantep, I checked the docs and it says that BSON format allows having such documents, but MongoDB process won't ever add duplicate fields.
From official docs:
BSON documents may have more than one field with the same name. Most MongoDB interfaces, however, represent MongoDB with a structure (e.g. a hash table) that does not support duplicate field names. If you need to manipulate documents that have more than one field with the same name, see the driver documentation for your driver.
Some documents created by internal MongoDB processes may have duplicate fields, but no MongoDB process will ever add duplicate fields to an existing user document.
http://docs.mongodb.org/manual/core/document/

From official docs:
BSON documents may have more than one field with the same name.

So we agree that the DB itself allows it.

Most MongoDB interfaces, however, represent MongoDB with a structure (e.g. a
hash table) that does not support duplicate field names.

Most but not all :) (not a requirement as DB accepts it).
I think that if the most common case is to avoid such duplicate, then some config options or alternate document functions could be added, to keep both approaches.

Close it for now. This can be discussed on the mailing list. If more feedback can be reopened.

We were also bitten by this. I agree that this is a MongoDB bug, not the driver's, but this behavior by MongoDB really sucks!


chore: upgrade deps

upgrade vite&typescript that won't be upgraded by renovate
Also, upgraded lockfile (pnpm dedupe)

/ecosystem-ci run

/ecosystem-ci run

/ecosystem-ci run

/ecosystem-ci run


Update about new release?

Update about new release.
Hello Timmy ! Great work.
Is there any confirm date of new release?

Sorry, not really. As soon as I can get to it. I think the master branch is in pretty good shape (so feel free to use that), but I want to spend some time doing more testing before the release.


hidden prop of Tabs component is ignored

Detailed description

Describe in detail the issue you're having.
I can't hide the Tab Content rendered by the Tabs component when the Tab isn't currently selected.  This causes problems because it forces all my child components to load even if the user isn't viewing them.

Is this issue related to a specific component?
Tabs

What did you expect to happen? What happened instead? What would you like to see changed?
I assumed that the "hidden" prop of the Tabs component would keep the child Tab content from rendering if it's not currently showing from the description of the prop?  Is that the intention of that property?

I checked the source of the Tabs component but it appears that the hidden property is ignored, and no matter what all the Tab Content components are rendered into the Dom with aria-hidden attached to them.

What browser are you working in?
Chrome

What version of the Carbon Design System are you using?
6.106.0

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/nk30xz0p3m

Notice that tab2 content is actually in the DOM even though I pass the hidden prop

Just responding to this, after it was linked here: https://ibm-safer-payments.slack.com/archives/C2K6RFJ1G/p1554229626107000 https://ibm-safer-payments.slack.com/archives/C2K6RFJ1G/p1554229740107400
We've had a similar issue, that the components still technically existed and were just hidden, so we created a wrapper which only adds the react component to the tab if the tab is actually selected (may need to be generalized a bit before being used)
export function TabsWithRenderingWrapper(
  Components: Array<React.ReactElement<ITabbableComponent>>,
  selectedTab: number,
  handleTabSelection: (tabNumber: number) => void
) {
  const tabs = Components.map((element, index) => {
    return (
      <Tab
        key={element.props.tabLabel}
        label={element.props.tabLabel}
        tabIndex={index}>
        {selectedTab === index && element}
      </Tab>
    )
  })
  return (
    <Tabs selected={selectedTab} onSelectionChange={handleTabSelection}>
      {tabs}
    </Tabs>
  )
}

@JSoet To see what to do with this issue, would like to elaborate why you wanted to avoid rendering non-selected tab contents (panels)? Thanks!

We are using a 3rd party component and this component didn't properly update after being hidden, it would calculate some widths when it was originally mounted and would calculate them incorrectly because it was hidden, and then when it was unhidden it would still use those widths and there was no way to get it to 'update' itself...
(Obviously ideal would've been to get it fixed in the other component, but this would've been a much larger task and the above workaround was the easiest thing that we figured out to do)

Cool thanks @JSoet for your response!


ReferenceError: document is not defined

Hi!
only one string
const app = new Application();
and i have error
ReferenceError: document is not defined     at new AccessibilityManager (D:\test\node_modules@pixi\accessibility\src\AccessibilityManager.ts:96:21)
an error appears if the monorepe uses
pixi 7.2.4 for frontend and @pixi/node for backend
dep: "@pixi/node": "7.2.0"
start: node dist/main.js
I tried to run a test project with an example from the pixijs/node readme.md file
however, it also didn't work with an error
PS D:\test2> node index.js
file:///D:/test2/node_modules/@pixi/extract/lib/Extract.mjs:51
const imageData = new ImageData(new Uint8ClampedArray(pixels.buffer), width, height);
^
ReferenceError: ImageData is not defined
at _Extract.canvas (file:///D:/test2/node_modules/@pixi/extract/lib/Extract.mjs:51:23)
at file:///D:/test2/index.js:38:6
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Same issue.
Seems the code doesn't actually work which is sad :(

Hey, this is a node-canvas issue, solution here: https://github.com/Automattic/node-canvas/issues/2003#issuecomment-1356245951

Although the above solutions didn't work for me, the following did:
import { Canvas, ImageData } from 'canvas';
import { Application } from "@pixi/node";

const app = new Application();
// ...

const pixels = new Uint8ClampedArray(app.renderer.extract.pixels(app.stage));
const imageData = new ImageData(pixels, chosenWidth, chosenHeight);

const canvas = new Canvas(chosenWidth, chosenHeight);
const ctxt = canvas.getContext('2d');
ctxt.putImageData(imageData, 0, 0);

await stream.pipeline(
    canvas.createPNGStream(),
    fs.createWriteStream(outputPath),
)

app.destroy();


fixed auto-scroll for emotion attributes

1. Why is this change necessary?
If the emotion component has a lot of attributes, the backend window doesn't have a scrollbar.
2. What does this change do, exactly?
Adds autoScroll to ExtJS.
3. Describe each step to reproduce the issue or behaviour.
Add emotion attributes.
4. Please link to the relevant issues (if any).
None.
5. Which documentation changes (if any) need to be made because of this PR?
None.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-22471
Please use this issue to track the state of your pull request.

Hey @kekster ,
thank you for your contribution :blue_heart:


dreport: added new plugins to collect additional data for BMC dump

Pulled the following commits from master and also changed to use ZSTD compression in downstream
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52899
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52913
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52891
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52849
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52944
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/53082
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52594
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52943
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52852
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52792
https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-debug-collector/+/52850

Can you run  dreport -v and make sure no errors reporting, before merge.

Looks, like the last commit, is not required, right? did not see any changes

ZSTD enablement : Cleaner approach will be providing option in dreport  , and bazsed on the  compression type enable the compression type , for IBM code ( ghe)you can pass zstd type. That is celaner way.  The code added here seems to not cleaner.

ZSTD enablement : Cleaner approach will be providing option in dreport , and bazsed on the compression type enable the compression type , for IBM code ( ghe)you can pass zstd type. That is celaner way. The code added here seems to not cleaner.

agree for master we need to provide option

Devender: can you add description on Known , warnings/errors ( if any with this PR).  Also need Matt's input on esel support.

At present following warnings are dispalyed the same will be fixed in a new patch set, as this patchset has over 13 commits we want this to goahead, and these warnings are not critical.
Thu Apr 21 12:54:11 UTC 2022 ERROR: Failed to collect Hardware dumps
Thu Apr 21 12:54:11 UTC 2022 ERROR: Failed to collect SBE dumps
Regarding below warning
"Thu Apr 21 12:54:11 UTC 2022 ERROR: Failed to collect Attribute list"
corresponding functionality is removed for now and will be added after fixing the attributes command or plugin

ibm_elogall  script is removed based on @spinler approval, matt said "don;t need ibm_elogall anymore, this was just for witherspoon, I am ok if you remove from witherspoon as well, since we aren;t shipping out of master"


how to clear circles and lines after the first input?

after the user insert first input ('123' for example ) - how can i clear the board?
i tries to use onReset() methd - but its not working.

i solved it by adding intrerval prop and by fixing the  resetActive() method in the source code. instead of :
this.state.lines = [];
i placed:
this.setState({lines: []});


Approver isn't added the first time in intent

https://github.com/warden-protocol/wardenprotocol/assets/59832414/14b9d9ef-209c-48ac-8b15-391fa8be3fe9

Should be fixed in this PR https://github.com/warden-protocol/wardenprotocol/pull/290

Fixed


Tab sorting doesn't work, if tabs use default title

If the title of the tabs are the default titles (Tab 1, Tab 2 ...), the sorting of the tabs does not work:

@wesleybl It works, but the widget is misleading.
The widget will automatically create titles for its options, so these titles are automatically generated. Of course they'll renumber once dropped.


Compatible with torch.export?

Is this implementation compatible with torch.export? I want to deploy SAM in a serverless fashion but the compile step causes long cold start times.

The model has 0 graph breaks so indeed torch.export should work, if you run into any issues please let us know!

@dbbert I'm going to close this now since the issue seems to be resolved, but please reopen if that's not the case.


Add support for failover

Laravel provides failover configuration for the mail drivers.
That means, in case the mail driver doesn't work, it would retry with the next registered driver to send emails.
Doc: https://laravel.com/docs/10.x/mail#failover-configuration
Currently, it doesn't work with mailersend.
The reason it doesn't work is because Symfony's Mail Transport catches exceptions through TransportExceptionInterface.
However, when something goes wrong, MailerSend throws MailerSendException.
Ref: https://github.com/symfony/mailer/blob/7.0/Transport/RoundRobinTransport.php#L56-L60
And because of that, it doesn't find the exception and can not retry with another fallback driver.
Now to make this work, we need to throw TransportException instead of MailerSendException.

@kavanpancholi I think we can close this one right?

Yes for sure. Closing now.


feat: support install poetry plugins

Close #53
Inspired by #69, but remove poetry version check and comma-delimited support
Also, add poetry 1.8.1 to test list

This looks solid @waketzheng. Thanks! I'll try to review this sometime this week 👍

Awesome, thanks for the contribution @waketzheng!


Proposal to remove race condition between InlineCompletionProvider and CompletionProvider display

Originally described here, but splitting out into a new issue: https://github.com/microsoft/vscode/issues/131940#issuecomment-931815784. Sorry for the delay!
This proposal attempts to resolve the race condition that exists between inline completions and autocomplete completions, and remove the ambiguity between which will be shown in the case of an autocomplete trigger character being typed.

When typing normally on a line the inline completion is shown and updates as you type
When an autocomplete trigger character is typed, request autocomplete and inline completion in the background
Delay display for inline completion and autocomplete results, which may require a network round trip
If no autocomplete result "agrees" with inline completion prefix, show autocomplete popup, otherwise show inline completion.

Agreement as described above simply means that the inline completion begins with the text from an autocomplete result.  In the case of Copilot this means the AI is on track, so no autocomplete popup should be shown.
/cc @hediet @isidorn @hsenag

This will mean autocomplete shows later than it would otherwise, right? (as we have to wait for the inline completion before deciding whether to show it) That might be fine, just wanted to be explicit about it.
/cc @SmallLemur

if IntelliCode extension smarter than Copilot , what will happen?
or,
If Copilot smarter than IntelliCode.
I have always wondered if there is repression if one  is too smarter than the other.
Who should follow who ?


Better error reporting in azure devops for sentinel builds

Is your feature request related to a problem? Please describe.
After adding content for ASIM parsers, azure devops pipeline complains that
Template Id: ASimAuditEventIllumioSaaSCore is not valid in Line: 356 col: 25 Errors: The incomplete fragment is unexpected., Code: 'KS198', Severity: 'Error', Location: '31121..31122' Expected: True Actual:   False
From this message it isnt clear, which file this is pointing to.
Also, what does location 'x..y' mean? What does the code KS198 mean?
Will be helpful if filename is present.

Hi @ashwinvenkatesha, Could you please let us know is that related to any PR? can you share PR no here.

Hi @v-sudkharat
https://github.com/Azure/Azure-Sentinel/pull/10994/checks?check_run_id=29130521606
this is from an older commit

@ashwinvenkatesha, The pipeline validations checks are pass for the above shared PR, can you check once.

I understand the pipeline runs have passed for the pr in the recent runs
But not for that run_id isnt it.
If you open that link I shared above, you will see
Kqlvalidations.Tests.KqlValidationTests.Validate_ParsersFunctions_HaveValidKql(fileName: "ASimAuditEventIllumioSaaSCore.yaml", encodedFilePath: "L2hvbWUvdnN0cy93b3JrLzEvcy9QYXJzZXJzL0FTaW1BdWRpdE"...)

Template Id: ASimAuditEventIllumioSaaSCore is not valid in Line: 356 col: 25
                    Errors: The incomplete fragment is unexpected., Code: 'KS198', Severity: 'Error', Location: '31121..31122'
Expected: True
Actual:   False

This is the issue I am referring to, this error reporting makes no sense, since line 356, col 25 never had the error.
The error was due to an extra semicolon at the end of the file, after many hours of frustration I realized the semicolon was not needed and that is when the next run went clean.
https://github.com/Azure/Azure-Sentinel/pull/10994/files/8eed458b0ee8b6e43bc4eb31035b500cb4b31495#diff-ae739a11b071ad0e82212c2ccc5a1063bbaec58629fd63e28f8de52ad4b31d03, check out ASimAuditEventIllumioSaaSCore.yaml, it has an extra semicolon at the end.

tagging @mkchiliveri for visibility.
@v-rusraut @v-sudkharat I think this issue is more related to improving our error reporting during github validations.


Unsupported .MD syntax

I try to convert https://github.com/NancyFx/Nancy.wiki.git
1 [[localfile]] unsupported. I replace manually [[url]] on [url](url)
2.i added manually  alias  c# for highlight .js
3. Not found pdf output file. Html file looks not very good (in opera and chrome)

@lobster2012-user [[]] is not supported, cause lwc relies on marked and it can't recognize it. Please see this thread https://github.com/chjj/marked/issues/471.
Instead you can check my PR - https://github.com/limedocs/limedocs-wiki-converter/pull/2, it allows you to use full links to .md files in you wiki and they will be treated as page indicators.
About PDF output, make sure you did install http://wkhtmltopdf.org/downloads.html and add it to your PATH. I can see both html and pdf outputs.

Closing as this seems fully replied


Windows TCC examples don't build

E.g. trying to compile the minimal C example:

When compiling the example with the WebUI library TCC compiled:
tcc: error: undefined symbol 'strtoll'
tcc: error: undefined symbol 'SleepConditionVariableCS'
tcc: error: undefined symbol 'opendir'
tcc: error: undefined symbol 'closedir'
tcc: error: undefined symbol 'strtoull'
tcc: error: undefined symbol 'IsWow64Process'

GCC compiled library:
tcc: error: invalid object file
tcc: error: library 'webui-2-static' not found
mingw32-make: *** [Makefile:27: release] Error 1

The errors are similar when trying to use tcc with V.
If they can be fixed, it might help to get tcc running over there.

Edit:
Checking out earlier stable tags like 2.3.0, the TCC example doesn't build as well.
With 2.3.0 we run into less errors
tcc: error: undefined symbol 'opendir'
tcc: error: undefined symbol 'closedir'
tcc: error: undefined symbol 'IsWow64Process'

And checking over at civetweb it's related to what author is stating here: https://github.com/civetweb/civetweb/issues/1158#issuecomment-1546845523
Maybe we should remove TCC from the supported list and the examples until it works?

All those issues are related to missing header files. I suggest removing the TCC support from WebUI.
At the end of the day, any user can add headers files to TCC and compile the lib and examples.


Fix QN MPO * MPS

This fixes the "densitymatrix" version of MPO * MPS with QNs. Fixes #320.
I've also included a few other things in this PR (improvements I came across while rewriting _contract_densitymatrx):

Fixed #307 (return 0 for an QN ITensor with no indices and no blocks).
Add a lot of priming, tagging and similar functions for manipulating indices of MPOs and MPSs (this utilizes metaprogramming to make it easier to generically add new functionality, instead of the ad hoc way in the C++ version).
Adds a keyword argument to setindex! for MPO and MPS called set_limits which defaults to true. When false, it turns off setting new limits when replacing one of the ITensors, if you know it doesn't change the orthogonality.

I just pushed a small formatting improvement to the mismatched arrow error message. Otherwise looks good to merge. Nice to have these fixes!

In the latest, I made the following changes:

Rename siteinds(A::MPO, B::MPS) to unique_siteinds(A::MPO, B::MPS) to get a vector of the site indices of A that are not shared with B (the previous one was a bit ambiguous, and doesn't distinguish from common_siteinds(A::MPO, B::MPS)).
Add common_siteinds(A::MPO, B::MPS) to get a vector of the site indices shared between A and B.
Add functions like prime_common_siteinds!(A::MPO, B::MPS) to prime all site indices shared between A and B (works with all prime and tag related functions, and sim).
Remove functions like prime_linkinds!(A::MPO, B::MPS) which primes the link indices of A and B as well as the site indices shared by A and B. I think that name it is a bit ambiguous given the functionality, and can be done with prime_linkinds(A); prime_linkinds(B); prime_common_siteinds(A, B). We could bring back that functionality in a more compact way, but I would rather be on the conservative side right now and come up with a better name.

Those changes all sounds really good. I agree that prime_linkinds is a confusing name if it also primes indices which could be considered "site". And similar with the other changes you mentioned. By the way, those functions which find all of the indices in common between two MPS or MPO's and do things like replace links with similar ones etc. are really useful for "hardening" MPO and MPS algorithms against all possible user inputs. They've been great to have!


Cannot find module 'electron-devtools-installer' when running "npm start"

When launching FlameCast using npm start on a Linux development box, the following error dialog appears:

electron-devtools-installer is not present in package.json in the root of the project. However, it is present under devDependencies in /app/package.json (which is where electron lives).
After dismissing the dialog, FlameCast still launches and likewise a new browser tab launches with the same content.
System:

Ubuntu 21.04
Linux Kernel: 5.11.0-36-generic
Gnome 3.38.5
X11
AMD Ryzen Pro 4750U
32GB RAM

@smphayward is this issue still happening? I ran npm start on my Windows machine and it works. If still happening, might be specific to Linux.
Since it's 15 days old, just want to ensure it's still breaking

I cloned FlameCast (origin/main) onto my Windows VM and got the same error.
This is a fairly fresh install of Windows. I had to install node on it just to run the code.
As with my Linux machine, after dismissing the error dialog the electron app runs as expected.
My guess is that there is some dependency that doesn't properly come down with "npm i".

In package.json, devDependencies was missing an entry for "electron-devtools-installer".
On my Windows VM, I ran the following command in the flamecast directory:
npm install electron-devtools-installer --save-dev
This added the missing dependency to devDependencies.

Finally, when launching with "npm start" it did not give the error! w00t!

@dorival I've actually managed to fix the issue. I'm going to make a PR shortly.


feat(nav): remove sidecar

@BerkeleyTrue be sure to make the "help" button on all of the challenge views open the appropriate chat room in a new tab as well.


Change iris dimensions to match the report, and fix encoded polynomial block configurations

Dimension Changes
Currently we're using iris dimensions of 160x80, but the production dimensions are slightly different. So this PR changes those sizes to match the production system.
Design Changes
As part of this change, we need to fix some design issues:

[x] Split iris bit configurations and encoding configurations
[x] Avoid confusion between the number of rows in a full iris code, and the number of rows per encoded block

So this PR creates two different kinds of configs:

FullBits, MediumBits, TinyTest - the bit sizes and dimensions of the full iris codes and plaintexts
FullRes, MediumRes, TinyTest - the block sizes and polynomial sizes for encoding (and encryption)

These design changes are needed for ticket #89.
Cleanups
It also improves performance and ease of use:

[x] make polynomial sizes tight, and automatic, by calculating them from the iris size
[x] Rename config types to be consistent with each other

And fixes some bugs:

[x] Calculate polynomial sizes from the encoded size plus padding
[x] Add missing impl EncodedConf for MiddleRes
[x] Add missing tests and benchmarks for Test*/Middle* (or Middle*/Full* if TinyTest doesn't work)

[x] Remove a hard coded TestRes in some tests

[x] Add missing types and inputs/outputs to test failure logs
[x] Cleanup redundant tests and benchmarks

Close #102
Part of #89, #100

Some benchmarks will change because we changed the size of the iris code, for example:

Benchmark suite	Current: d38c37e	Previous: b2b5a96	Ratio
Plaintext full match/random 657171 ns/iter (± 56739)	975758 ns/iter (± 7168)	0.67


*: test, docs with go1.6+

etcd v3 uses http/2, which doesn't work well with go1.5

lgtm

fixed. merging after travis passes.


Fixed CSS for inversion

Lol I didn't saw the pull-request, thanks, even if the project is dead! 😂


Fix selfupdate on Windows

On Windows, a process can't change or delete a file if it is executing.
selfupdate command was failing due to this limitation. This change
uses a background process on Windows so that it repeats moving the file
until the execution finishes. The background process deletes itself at
the end using an exceptional behavior of Windows where a bat script can
delete itself.

@komamitsu can I ask you to take a look?

I left some comments. But totally LGTM 👍


remove_trailing_whitespace is removing one trailing newline at each run

Which configuration?
Type: builtin
Filetype: none
Formatter: any remove_trailing_whitespace
Configuration:
require("formatter").setup {
  filetype = {
    ["*"] = {
      function()
        return require("formatter.filetypes.any").remove_trailing_whitespace()
      end
    }
  }
}

Expected behavior
yo

in hex 796f 0a0a 0a
Should be unchanged by the formatter
Actual behaviour
yo

in hex 796f 0a0a
if you have another run of the formatter, then one more newline is removed
yo

in hex 796f 0a
From here on new runs don't change anything
Additional context
I tried but couldn't find the reason after a quick check of the code.
I feel this is an issue coming from the way the data from stdin is ingested.

I've confirmed that I experience the same behavior. ➕


Feature/offline study v1 3

Note: this branch cannot be used for robomimic datasets, since breaking changes are included in gripper model as https://github.com/ARISE-Initiative/robosuite/pull/213
  File "wrappers.pxi", line 1051, in mujoco_py.cymj.PyMjModel.site_name2id
ValueError: No "site" with name gripper0_ee_x exists. Available "site" names = ('table_top', 'robot0_ee', 'robot0_ee_x', 'robot0_ee_z', 'robot0_ee_y', 'gripper0_ft_frame', 'gripper0_grip_site', 'gripper0_grip_site_cylinder', 'cube_default_site').

This problem is mentioned in robomimic repository, too

https://github.com/TMats/robomimic/blob/develop/docs/miscellaneous/troubleshooting.md#known-issues

making this PR pending


Automated cherry pick of #3220: fix: azure ListObject should skip current folder

Cherry pick of #3220 on release/2.11.
#3220: fix: azure ListObject should skip current folder

Codecov Report

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

@@              Coverage Diff               @@
##             release/2.11   #3221   +/-   ##
==============================================
  Coverage                ?   7.13%
==============================================
  Files                   ?     476
  Lines                   ?   77276
  Branches                ?       0
==============================================
  Hits                    ?    5514
  Misses                  ?   71299
  Partials                ?     463

Flag
Coverage Δ

#aFlag
7.13% <ø> (?)

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 7903796...76e8685. Read the comment docs.

/lgtm
/approve


auth:password-reset-confirm-success never seems to be broadcast

I'm using ng-token-auth on my mobile app (ionic/angular) with Devise Token Auth on rails backend.  I've been struggling with the password reset process. 'auth:password-reset-confirm-success' is never broadcast after I click on the reset email link.  I may just be doing this all wrong, but I wanted to do the password reset from the app after the user hit the confirm link in the email, rather than posting a web form, having them reset there, then go back to the app, etc.
UPDATE: To get the broadcast, you need to have ValidateToken run.  my $authProvider.configure:
.config(function($authProvider,apiUrl){
              $authProvider.configure({
                        apiUrl: apiUrl,
                        forceValidateToken: true,
                        storage: 'localStorage'
                  });
})

Then after making the reset request call, I change the state back to SignIn, which forces a page load and ValidateToken, and I get the broadcast.  Now, onto handling the password reset from the app.
Never mind, it doesn't work the way I had imagined.  I am unable to make password reset work between my mobile app and the rails server.

I am trying to achieve just the same, my optimal password reset flow would be:

user requests reset mail from the app
user receives email and clicks the reset link from the mail
user is redirected to the app from there (whether it's emulated on my localhost in dev environment or on the app on a real device)
user is signed in for one session and is asked to set a new password

however, I can't get the bit in 3. working where the user is signed in after the click on the link. any help or clarification whether this can even work as I hope is highly appreciated.


Service List sort order correction

Service List default sort order based on service name.

Codecov Report

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

@@            Coverage Diff            @@
##           master    #127      +/-   ##
=========================================
- Coverage   66.68%   66.6%   -0.08%
=========================================
  Files          18      18
  Lines        3956    3956
=========================================
- Hits         2638    2635       -3
- Misses       1153    1155       +2
- Partials      165     166       +1

Impacted Files
Coverage Δ

server/service/governservice.go
64.07% <0%> (-1.46%)
:arrow_down:

server/service/tag.go
71.01% <0%> (-0.49%)
:arrow_down:

server/service/instances.go
66.84% <0%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage decreased (-0.05%) to 71.004% when pulling b9f81289630f87771d58f6fab4ae72cd1e34c6e5 on vibakar:master into fa2904b7ecf07309db36891751dd41f1e91868cf on ServiceComb:master.


Bug with provider-selection-enabled MFA

Description: Correcting Bug when selection menu is enabled and there are multiple MFA available for a service
Test case:

Configure cas.authn.mfa.provider-selection-enabled=true
No global MFA provider is set
Configure a service with multiple MFA:
for example:
multifactorPolicy: {
multifactorAuthenticationProviders: [
'mfa-gauth',
'mfa-u2f',
'mfa-simple'
],
failureMode: 'CLOSED',
bypassEnabled: false,
forceExecution: false,
bypassTrustedDeviceEnabled: false,
_class: 'org.apereo.cas.services.DefaultRegisteredServiceMultifactorPolicy'
},
Authenticate to this service
Authenticate with user/password
Select the MFA to use
Authenticate with the selected MFA

Result:
After successfully authenticate to the MFA, the service could not validate the Service Ticket
CAs throws an INVALID_AUTHENTICATION_CONTEXT error
the logs show "Attempting to match requested authentication context [mfa-composite] against [[]]", where  is the real selected MFA by user
Expected result:
Successfull service ticket validation
Correction:
When a service is configured with a ChainingMultifactorAuthenticationProvider (mfa-composite), the DefaultMultifactorAuthenticationContextValidator fails
The patch detects this case and replace the requestedContext (mfa-composite) by the context of the real selected MFA from the authentication attributes

Thank you for the patch. Please re-open the pull request when you have added test cases.
Instructions:
https://apereo.github.io/cas/development/developer/Test-Process.html

Could i have any help to make a test ?
I have no idea how to implement a complex test for this !

Sure. What kind of help do you need? Take a look at the document I posted earlier, and study the existing tests. Then, checkout the documentation for JUnit and Puppeteer. Those are the best docs we can offer.

Hi,
the test to do is the full process:

enable mfa selection
configure a service with 2 mfa
authenticate with a user/password
select the mfa using the selection menu
authenticate with the mfa selected
catch the ticket
validate the ticket

All the parts seeams already tested separately but " select the mfa using the selection menu"
Do you have any clue to do this ?
Regards,
Thierry Perpère
Direction des Systèmes d'Information
Chef de projet - Expert en infrastructures
Pôle Infrastructures Systèmes et Applications Transverses
+33 1 69 15 70 77
[ http://www.universite-paris-saclay.fr/ | www.universite-paris-saclay.fr ]
De: "Misagh Moayyed" @.>
À: "apereo/cas" @.>
Cc: "thierrytiti" @.>, "Author" @.>
Envoyé: Lundi 31 Mai 2021 16:39:43
Objet: Re: [apereo/cas] Bug with provider-selection-enabled MFA (#5153)
Sure. What kind of help do you need? Take a look at the document I posted earlier, and study the existing tests. Then, checkout the documentation for JUnit and Puppeteer. Those are the best docs we can offer.
—
You are receiving this because you authored the thread.
Reply to this email directly, [ https://github.com/apereo/cas/pull/5153#issuecomment-851534065 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/ACBIQXOMPMTQ3NR3CR4ZZU3TQONS7ANCNFSM452JQDRA | unsubscribe ] .

Look at the collection puppeteer tests; I believe there is a test case in there for this purpose.
Also, make sure you run your changes through the styling guidelines.
Also, if this issue is in fact true; your changes are invalid; you need to make sure the correct identifier from the composite is recorded in the authentication at authentication tim, not at validation time.

Thank you,
You're right for my workaround, it's not completly the good way to do things.
My goal is simply to to help the project move forward.
I'll firstly try to add the test with user selection.
Thierry Perpère
Direction des Systèmes d'Information
Chef de projet - Expert en infrastructures
Pôle Infrastructures Systèmes et Applications Transverses
+33 1 69 15 70 77
[ http://www.universite-paris-saclay.fr/ | www.universite-paris-saclay.fr ]
De: "Misagh Moayyed" @.>
À: "apereo/cas" @.>
Cc: "thierrytiti" @.>, "Author" @.>
Envoyé: Lundi 31 Mai 2021 17:13:45
Objet: Re: [apereo/cas] Bug with provider-selection-enabled MFA (#5153)
Look at the collection puppeteer tests; I believe there is a test case in there for this purpose.
Also, make sure you run your changes through the styling guidelines.
Also, if this issue is in fact true; your changes are invalid; you need to make sure the correct identifier from the composite is recorded in the authentication at authentication tim, not at validation time.
—
You are receiving this because you authored the thread.
Reply to this email directly, [ https://github.com/apereo/cas/pull/5153#issuecomment-851552347 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/ACBIQXN3L5HR4J5ARLIMFQTTQORSTANCNFSM452JQDRA | unsubscribe ] .

You can create a new test based on this:
https://github.com/apereo/cas/tree/master/ci/tests/puppeteer/scenarios/mfa-provider-selection
And then replicate the whole setup as you described.


Recipe to revert the box back to sheeting

I got caught that I hesitate to craft boxes because cannot be reversed. If the node could be reversed the boxes could be used as compact storage for plastic, and maybe used more..
Since the cheapie upstream seems to be dead, do you accept my change in your fork?

Absolutely, thanks @bell07! I'm mainly a creative player myself, so I def. appreciate PRs from those playing survival (though I do want to get into survival at some point!) :smiley_cat:


Fix several typos in the documentation

Typos and copy-paste erros found when translating the class reference.
Note: The reason for deleting See also [method merge] is that this sentence has already appeared in the first paragraph.

Thanks!


[Feature request] Support for "Date Added" and "Favourite" fields

Hello, would it be possible to add support for the date added and "loved" fields to carry over to navidrome?
From what I've experienced, as of right now, the "date added" is not overwritten by the date added field migrated from iTunes, and it would be nice to have the "loved" field from iTunes show up as a favourite in navidrome

From a look at itunestoND.py and the output of each it_song_entry I can see that the key "Loved" exists if only if it's true

So I believe it would be simple to check each it_song_entry for the field like loved = it_song_entry.find('key', string='Loved') and a simple check if(loved != None)
Also the date added field seems to be formatted in the exact same way as the last added field:

I'm just unfamiliar about how to implement this with the existing way you update playstatistics, so some advice or assistance would be appreciated.
Thanks

Regarding transferring your hearts...I did not write that into the script. If you look in the write_to_annotation function, there is a line:
annotation_entries.append((generate_annotation_id(), userID, item_id, entry_type, play_count, play_date, rating, 0, None))
The last two things in that list (0 and None) tell the Navidrome database that there is no heart (the 0). None is the database field that holds the "Starred at" date-time value.
If there is a 1 in the second to last field instead of a 0, then it will be hearted in Navidrome.
Example of how time is format in the database: 2023-03-05 04:19:20.
So you could pick the hearts out of your itunes library and write a 0 or 1 to the database field. I don't really know what point there is to the heart time stamps. To simplify the code, you can probably just take the current time and format it like the example.
Anyhow, the way the program works. It parses the Library.xml, and writes all that data to 3 different python dictionaries - one for artists, albums, and the songs themselves. (the update_playstats) function.
Right now, those dictionaries only track playcounts, last played date and song ratings. So you could add to those dictionaries the keys "heart" and "heart time" for instance.
Once those dictionaries are done being made, then the database actually gets written with the write_to_annotation function
So you'd need to change that function (the line annotation_entries.append((generate_annotation_id(), userID, item_id, entry_type, play_count, play_date, rating, 0, None))
And substitute your new variables in for the last 2 entries there.
That would probably work.

As for the syncing the "date added", in the Navidrome database that's a different table in the "created_at" and "updated_at" columns, so that's a whole other can of worms.


Pip install does not work

Hi, did you manage to install the politeness package ? From what I have tried, the pip install does not work. And I cannot figure out how to use it in bash once you have linked the url to the stanford coreNLP server

What exactly is the pip command that you are trying to run, and what's the error message? I also had some trouble pip installing some of the packages but figured out and was able to run model.py(the Python3 version).

The politeness functionality has now transitioned to ConvoKit (https:://convokit.cornell.edu). Try using that instead. Thanks.

The init.py is empty?
init.py
@sudhof sudhof initial model implementation code commit
9d30e18 on 27 Sep 2014
0 lines (0 sloc)  0 Bytes


Grammar fixes

Patrick decided that nulls should be handled as a special case in tools rather than in the lexer/parser. The parser will just treat null like any other identifier, but if a tool sees null in a meta section and there is no declaration for null, then the tool should substitute the null value.
Patrick also pointed out that import aliases were added in v1.0.
Finally, we decided on new names for the grammar files (WdlDraft2Parser/Lexer and WdlV1Parser/Lexer).
I also split out the functions that translate between CST and AST into separate classes because I will need to use them independently of the parser in the generator tool that I'm currently developing.

Note that the CI/CD test isn't passing. It is probably because of the compilation of antlr4 files.


Refactor/Simplify Code

Overview
The pull request is just code refactors. I also didn't test the code which I don't know if it will work but I'm sure you can review if there is any significant changes
Note: What's the difference between writing {} then just None

For python 3.8 and earlier the added syntactic sugar seems unsupported as shown by by the integration tests. TypeError: unsupported operand type(s) for |=: 'dict' and 'dict'
In principle the changes look good to me but at least Python 3.8 according to PEP 569 is not in end of life until October 2023. Not sure what our policy here should be? maybe @vaphes has an opinion?

We can leave this PR until py version 3.8 is no longer in use. It's up to you if you want to wait or close


Support morph browsers.

Description
This pr adds support for browsers targeting a morphTo.

This does not yet work for nested modules (example: admin.content.pages) ..


Fix multiprocessing support for Windows binary

The black binary generated for Windows builds would fail on
reformatting multiple files due to a Windows-specific
multiprocessing issue. Fix by calling freeze_support() as
described in Python docs.

I didn't create an issue for this, not sure if that's a required part of the workflow?  If that's preferred, let me know and I'll do it and link this PR to it.

You probably want to add this in blackd.py as well

Pull Request Test Coverage Report for Build 827

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 96.064%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

black.py
1
2
50.0%

Totals

Change from base Build 824:
-0.03%

Covered Lines:
2904

Relevant Lines:
3023

💛  - Coveralls

@zsol Added now. At the moment, this binary is not built in CI, but I built it locally with C:\\Python36\\python.exe -m PyInstaller --clean -F --add-data blib2to3/;blib2to3 blackd.py and verified that it fails before the fix and works after.

Thanks!


adding material to begginner path

Related Issue
No related issue (will make sure to creat one first going forward)

Information about the related issue

Closes: #
Describe the changes you've made

Type of change
Changes adds a short written path at beginning pf beginner dev path. Plan to extend this in future to give info on creating dev environment. Also adds a new section which has some recommended reading.
What sort of change have you made:

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

How Has This Been Tested?
Not a code change

Checklist

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

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Code of Conduct

[x] I agree to follow this project's Code of Conduct

Next time, please don't merge it until 2 or more maintainers have approved. Thanks. 🙂


Piping between two Win32 processes in Bash doesn't work

Please use the following bug reporting template to help produce actionable and reproducible issues. Please try to ensure that the reproduction is minimal so that the team can go through more bugs!

A brief description
I tried executing "/mnt/c/Windows/System32/ipconfig.exe|/mnt/c/Windows/System32/clip.exe" and got back "INFO: Type "CLIP /?" for usage."

Expected results
Nothing to be printed to the terminal and return straight to the prompt for a new command.

Actual results (with terminal output if applicable)
"INFO: Type "CLIP /?" for usage."

Your Windows build number
14959

Steps / All commands required to reproduce the error from a brand new installation
/mnt/c/Windows/System32/ipconfig.exe|/mnt/c/Windows/System32/clip.exe

Strace of the failing command
None as this is an issue with Win32 interop

Required packages and commands to install
None

See our contributing instructions for assistance.

Duplicate of #1255


Dimension table stick in GPU memory and refresh after update trigger

Is your feature request related to a problem? Please describe.
To avoid repeat dimension table transfer
Describe the solution you'd like
Dimension table is very static, usually got update every several hours, we can lazy load dimension table in GPU and keep there until there is new data update.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

we might want to make it a non-default configurable option, user should only enable it if a table is know to be used frequently, since device memory is scarce resource.


Fix false-positives that non Rails formats are offended on Rails/ToSWithArgument

We had Rails/ToSWithArgument Cop to replace to_s with to_fs where Rails' extended functionality is used, but there are frequent false-positives where to_s is used with non Rails extended formats like value.to_s(2) (Integer#to_s).
To prevent this, it would be better to check its 1st argument so that only cases that appear to use Rails-extended functionality are offended.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[ ] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Let me provide a little more background.
To prepare to set config.active_support.disable_to_s_conversion = true (which is also set by load_defaults 7.0), I've tested this Cop against the source code of all the Gems that my Rails app depends on, and I noticed that there were so many false-positives. I ended up inspecting 20000+ Ruby files and found that 2 files still used the Rails extended to_s 😨, and there are 200+ false-positives about Integer#to_s.
This is why I decided to add this fix.


sanitycheck: add new _TEXT_SECTION_NAME_2 section to whitelist

Add _TEXT_SECTION_NAME_2 to the whitelisted list of linker sections.

what's generating this section?

@AdithyaBaglody I see this section is something you added to ARM's linker.ld, can you provide some background information?

Codecov Report

Merging #11955 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11955      +/-   ##
==========================================
+ Coverage   48.07%   48.17%   +0.09%
==========================================
  Files         281      281
  Lines       43347    43360      +13
  Branches    10391    10392       +1
==========================================
+ Hits        20839    20887      +48
+ Misses      18350    18318      -32
+ Partials     4158     4155       -3

Impacted Files
Coverage Δ

include/logging/log_core.h
100% <0%> (ø)
:arrow_up:

include/logging/log_msg.h
90.9% <0%> (+1.81%)
:arrow_up:

misc/printk.c
84.93% <0%> (+3.12%)
:arrow_up:

subsys/logging/log_core.c
74.67% <0%> (+4.8%)
:arrow_up:

subsys/logging/log_output.c
69.4% <0%> (+9.58%)
:arrow_up:

Continue to review full report at Codecov.

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

@andrewboie I don't think this is a non-expanded variable. @AdithyaBaglody seemingly wanted to break up .text into two separate sections. See commit message here: https://github.com/zephyrproject-rtos/zephyr/commit/91c5b84cd5aa58bef2025815956ee02022a08ac8#diff-14daecb8cf4c8f78809f57a31b3d21cc


Does this work with RHEL 6 targets?

Compatibility section says RH 7.  Has it been tested with RH 6?

The puppet plugin hasn't been fully tested with RHEL 6, but it should work without issues. Both RHEL 6 and 7 are supported as Delphix target host environments for most databases.


classes prop not working with emotion css.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
In the docs it says:

Emotion's css() method works seamlessly with Material-UI.

But when I try this:
/** @jsx jsx */ jsx;
import { jsx, css } from "@emotion/core";
import { makeStyles } from "@material-ui/core/styles";

const useStyles = makeStyles({
  root: css`
    display: none;
  `
});

The code (TypeScript) won't compile. I get this error:
[ error ] ERROR in /home/felix/dev_projects/mewo.dev/schuemeyer2/schuemeyer/pages/index.tsx
82:30 Argument of type '{ root: SerializedStyles; }' is not assignable to parameter of type 'Styles<Theme, {}, "root">'.
  Type '{ root: SerializedStyles; }' is not assignable to type 'Record<"root", CreateCSSProperties<{}> | ((props: {}) => CreateCSSProperties<{}>)>'.
    Types of property 'root' are incompatible.
      Type 'SerializedStyles' is not assignable to type 'CreateCSSProperties<{}> | ((props: {}) => CreateCSSProperties<{}>)'.
        Type 'SerializedStyles' is not assignable to type 'CreateCSSProperties<{}>'.
          Index signature is missing in type 'SerializedStyles'.
    80 | `;
    81 |
  > 82 | const useStyles = makeStyles({
       |                              ^
    83 |   root: css`
    84 |     display: none;
    85 |

Expected Behavior 🤔
makeStyles() and emotion's css method should work together.

Steps to Reproduce 🕹
See the use of makeStyles() in this code sandbox (It should highlight the error in the editor):
https://codesandbox.io/s/emotion-typescript-example-v03mj?fontsize=14
Context 🔦
I'm trying to find out if we could use mui for our projects that use emotion for styling, but I'm stuck with this issue.
Is there maybe a plugin or something that I have missed to get emotion and mui work seamlessly together?
Your Environment 🌎

Tech
Version

Material-UI
4.3.3

React
16.9.0

Browser
Firefox

TypeScript
3.5.3

"@emotion/core"
10.0.16

I'm trying to find out if we could use mui for our projects that use emotion for styling, but I'm stuck with this issue.

@feluxe See https://material-ui.com/guides/interoperability/#emotion

@oliviertassinari Thanks for your reply, but I already saw the page that you linked to, I have actually quoted it above ;-) The given example doesn't solve the issue that I have.
To put it simple:
I know that this works:
<Tabs css={css`padding: 5px;`} ...

But how do I use emotion with the classes prop, e.g.:
<Tabs classes={{ root: ???, indicator: ??? }} ...

I tried with makeStyles (see code example and codesandbox above), without luck.

Maybe you don't need to use the classes API, what about the global class names?

I'm not sure if it would suffice. I'll see if there is a way to transform the css method result into something that can be passed to the classes prop.

@feluxe I don't think that it's possible. I would personally use the global class names, like .MuiTabs-indicator in your example, or provide tokens, e.g. indicator to the classes keys.

@feluxe Have you found what you've being searching for?

@defusioner I didn't have the time to dig deeper into the issue. We decided not to use mui for now, because we have a lot of code that uses emotion already.

@defusioner You can target the global class names. e.g.
<Button
  css={css`
    background: linear-gradient(45deg, #fe6b8b 30%, #ff8e53 90%);
    border-radius: 3px;
    border: 0;
    color: white;
    height: 48px;
    padding: 0 30px;
    box-shadow: 0 3px 5px 2px rgba(255, 105, 135, 0.3);
    & .MuiButton-label {
      color: papayawhip;
    }
  `}
>
  Emotion
</Button>

@defusioner I didn't have the time to dig deeper into the issue. We decided not to use mui for now, because we have a lot of code that uses emotion already.

I've started to use emotion with @emotion/react package which provides css and jsx functions, but got blocked with mui classes override in the runtime (not theme). I've looked through and finally I'm using emotion basic package which provides both css and cx (classnames emotion helper).
So finally I create a Component.css file that exports styles using styled-components-like syntax and then I simply use them inside the component with cx.

@defusioner Does the emotion package correspond to a deprecated older version of the library? It should be working with the latest version. We have a demo in the documentation.
https://material-ui.com/guides/interoperability/#emotion

@oliviertassinari I'm speaking about https://www.npmjs.com/package/emotion package that is the agnostic version of emotion library. This is the latest one (10.x as all others).
What is missing in the doc, is this example (passing an emotion class to the mui component):
import { css, cx } from 'emotion'

const noWrap = css`
  ...styles
`

const conditionalStyle = css`
  ...styles
`

<Tooltip
      title={helpText}
      placement={placement}
      enterTouchDelay={0}
      leaveTouchDelay={3000}
      classes={{
        popper: cx(noWrap),
        tooltip: cx({ [conditionalStyle]: condition })
      }}
    >
...
</Tooltip>

@defusioner Thanks for the link, so it's not deprecated! The Tooltip case is interesting, as the tooltip escape the parent hierarchy. This makes the usage of the css prop a challenge. We document ta workaround for styled-components: https://material-ui.com/guides/interoperability/#portals. I don't know how we can approach the same problem with emotion 🤔.
In the case of @feluxe, it's much simpler, the tab children can be accessed with child CSS selectors (.MuiTabs-root, .MuiTabs-indicator).

@oliviertassinari I like much the approach of separating css-in-js "classes" into a separate file. I also find styled components approach of "decorating" components not brilliant in the way the final code looks like.
I've liked the approach of this cx(css) usage because it keeps component's tree clean and is usable naturally as if there was no css-in-js.

@oliviertassinari Thanks a lot for your example it works perfectly! For future reference, a code sandbox with this example can be found here.


get a copy of response.text()

Is there a way to get a copy of response.text(), allowing the data to be used in two places?
I'm working on a load test framework, and I'd like to optionally write the response headers and body to a log file when errors are detected, to aid in debugging. However, I also need to return the complete response to the load test so it can do its own processing of it.
Here is a simplified version of what I'm trying to do:
async fn log(response: Result<Response, Error>) {
    let body = match &response {
        Ok(r) => {
            match &r.text().await {
                Ok(b) => Some(b.to_string()),
                Err(_) => None,
            }
        },
        Err(_) => None,
    };
    // @TODO optionally write this (and other stuff) to a log file
}

This does not compile, failing with the following error:
481 |                 match &r.text().await {
    |                        ^ move occurs because `*r` has type `reqwest::async_impl::response::Response`, which does not implement the `Copy` trait

error[E0507]: cannot move out of `*r` which is behind a shared reference

Is there a way to get a copy of the response text? (I'm also hoping to get a copy of the response headers). Perhaps there's a way to grab a raw, unprocessed copy of it?

The body of the response is not buffered, it is streamed. Once it's been read into a string, a copy isn't kept, otherwise that would force a potentially giant copy of memory even if most people didn't need it.

Okay. That makes sense. Thanks for the quick reply. I’ll ponder a way to log useful information.


add custom range transform

to: @jq @tddft

LGTM, some nites

@jq @TDDFT Updated the PR to address comments.

LGTM


focus window problem

I've just installed Quicksilver (for Lion), and discovered a serious issue... I hope someone can help.
The focus window is the ONLY window that displays.  All other windows get hidden when I switch apps.
I use a multi-screen setup, and like to have two or three windows open, for example a browser, a word-processor and email.
Now, when I switch to the Word Processor, my mail and browser disappear. and vice-versa.
Help!

Hi there,
We're glad to here you're trying out Quicksilver, and hope you like it.
This is one of the settings in Quicksilver, and can be turned on and off.
If you:

go into the preferences (invoke Quicksilver and press ⌘, )
go to the 'preferences' pane'
click 'extras on the left hand side'
there is an option to 'hide other applications when switching'. Turning
this off should fix your problem.

Is this the first time you have installed Quicksilver?  This will hopefully be
less confusing.
Thanks
On 5 December 2011 17:39, gfiremark <
<EMAIL_ADDRESS>
wrote:

I've just installed Quicksilver (for Lion), and discovered a serious
issue... I hope someone can help.
The focus window is the ONLY window that displays.  All other windows get
hidden when I switch apps.
I use a multi-screen setup, and like to have two or three windows open,
for example a browser, a word-processor and email.
Now, when I switch to the Word Processor, my mail and browser disappear.
and vice-versa.
Help!

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/issues/608

No, I upgraded from an older version of QS, but never encountered this before.
On Dec 5, 2011, at 9:44 AM, Patrick Robertson wrote:

Hi there,
We're glad to here you're trying out Quicksilver, and hope you like it.
This is one of the settings in Quicksilver, and can be turned on and off.
If you:

go into the preferences (invoke Quicksilver and press ⌘, )
go to the 'preferences' pane'
click 'extras on the left hand side'
there is an option to 'hide other applications when switching'. Turning
this off should fix your problem.

Is this the first time you have installed Quicksilver?  This will hopefully be
less confusing.
Thanks
On 5 December 2011 17:39, gfiremark <
<EMAIL_ADDRESS>
wrote:

I've just installed Quicksilver (for Lion), and discovered a serious
issue... I hope someone can help.
The focus window is the ONLY window that displays.  All other windows get
hidden when I switch apps.
I use a multi-screen setup, and like to have two or three windows open,
for example a browser, a word-processor and email.
Now, when I switch to the Word Processor, my mail and browser disappear.
and vice-versa.
Help!

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/issues/608

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/issues/608#issuecomment-3020207

Thanks!
and, no... this was an upgrade from an earlier version (much earlier, I think).  But I've never seen this behavior before.
It seems to be OK, now.
-Gordon Firemark
On Dec 5, 2011, at 9:44 AM, Patrick Robertson wrote:

Hi there,
We're glad to here you're trying out Quicksilver, and hope you like it.
This is one of the settings in Quicksilver, and can be turned on and off.
If you:

go into the preferences (invoke Quicksilver and press ⌘, )
go to the 'preferences' pane'
click 'extras on the left hand side'
there is an option to 'hide other applications when switching'. Turning
this off should fix your problem.

Is this the first time you have installed Quicksilver?  This will hopefully be
less confusing.
Thanks
On 5 December 2011 17:39, gfiremark <
<EMAIL_ADDRESS>
wrote:

I've just installed Quicksilver (for Lion), and discovered a serious
issue... I hope someone can help.
The focus window is the ONLY window that displays.  All other windows get
hidden when I switch apps.
I use a multi-screen setup, and like to have two or three windows open,
for example a browser, a word-processor and email.
Now, when I switch to the Word Processor, my mail and browser disappear.
and vice-versa.
Help!

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/issues/608

Reply to this email directly or view it on GitHub:
https://github.com/quicksilver/Quicksilver/issues/608#issuecomment-3020207


Only build for vesctl supported platforms?

The release action builds binaries for all supported Terraform/OpenTofu platforms, but the only resources currently exposed require vesctl which is published in mac and linux amd64 flavours.

Related to #22 - if other experimental resources and/or data sources are added then this should be closed as wontfix.

Not stale


Unexpected warning about reactive variable used outside of tracked scope

I've encountered an issue where the solid/reactivity is warning about a reactive variable being used outside of a tracked scope and I can't seem to identify the reason for the warning. After some exploration It seems like perhaps it is a false-positive.
The point of failure is inside a Context that I've built for my application but the same behaviour is exposed when using the reference implementation for using Context Providers from the official SolidJS tutorial, here: https://www.solidjs.com/tutorial/stores_context?solved
The context module from the tutorial:
import { createSignal, createContext, useContext } from "solid-js";

const CounterContext = createContext();

export function CounterProvider(props) {
  const [count, setCount] = createSignal(props.count || 0),
    counter = [
      count,
      {
        increment() {
          setCount(c => c + 1);
        },
        decrement() {
          setCount(c => c - 1);
        }
      }
    ];

  return (
    <CounterContext.Provider value={counter}>
      {props.children}
    </CounterContext.Provider>
  );
}

The linter warning is the following:

The reactive variable 'props.count' should be used within JSX, a tracked scope (like createEffect), or inside an event handler function.

The reference example uses the context in the main application like this:
import { render } from "solid-js/web";
import Nested from "./nested";
import { CounterProvider } from "./counter";

function App() {
  return <>

Welcome to Counter App

    <Nested />
  </>
};

render(() => (
  <CounterProvider count={1}>
    <App />
  </CounterProvider>
), document.getElementById("app"));

Any insight you may have about this problem would be appreciated. Thanks!

Hello--this warning is intentional, but you can ignore it if you understand that any changes to the prop will not affect the signal. Or, you can rename the prop to initialCount or defaultCount which will silence the warning. See my answer here.


Invalid HTML attributes in rendered slides

I really wanted to use MDX-Deck for my next series of talks, including An Event Apart. However, there is some invalid HTML in the final output that I can't live with. Any way to prevent these attributes from being rendered?
color="heading" font-size="4"
You can see these invalid attributes in the live example: https://jxnblk.com/mdx-deck/#0

Perhaps data attributes would ease this issue without having to make substantial changes?

Thanks for checking the library out! This is a long-standing issue with styled-components and how it whitelists valid HTML attributes (some legacy, some I'm unsure about) to determine which props are passed on to the underlying HTML element. Chiming in on this thread might help: https://github.com/styled-components/styled-components/issues/439
That said, I've been pondering and working on a major update to mdx-deck that should get rid of these random attributes. But for the current version, all rendered MDX HTML components can be overwritten with a theme.components object in a custom theme. If you'd like some guidance on how to "reset" all the default components, I'd be happy to create an example

I raised a pull request yesterday on styled-components that provides a mechanism to deal with this issue: https://github.com/styled-components/styled-components/pull/2392

With the changes in v2, the DOM elements should no longer include these extra attributes – the beta can be installed with npm i mdx-deck@next if you'd like to give it a shot

Woo! Awesome, I'll check it out!


Add progress reporting for named requirement resolution

What's this?

It just adds a "[Build]" output in the CLI if we have to build a full wheel here (rare), like we do elsewhere.


[Bug+minor visual] Right sidebar position: Graph Canvas Menu over the window and bluebar on the wrong side

Branch:
1.0-rc
Expected behavior:
When sidebar is on the right, the Graph Canvas Menu should be moved next to the window:

And also the bluebar should appear on the right:

Actual behaviour:
Graph Canvas Menu over the window:

And bluebar on the left:

no longer relevant


Publishing VSIX should be done automatically only from master and next branches

AppVeyor shouldn't upload VSIX for builds unless it was pull request on master or next branch.

Done!


Help

Why it is obfuscated ?

we need to check if that will steal our robux

its clearly obvious why its obfuscated lmao
defo a scam


added openzfs caveat for el capitan

Fixes #14256
@vitorgalvao not sure how to make this caveat conditional, not a ruby guy. Feel free to fix and merge as needed

We could do it with a simple conditional but I have no quarrel with it appearing in every version. I actually think it might be better in this case, so no one is caught off guard.

Please remove any caveats. We are now SIP compliant.
https://github.com/ilovezfs/openzfsonosx-installer/commit/fe930aa0add6cb4c709cbab4d828101ceed13889
https://twitter.com/openzfsonosx/status/656095561490132994
https://openzfsonosx.org/wiki/Downloads
https://openzfsonosx.org/forum/viewtopic.php?f=20&t=2353


Improve JS Api

As @genaris has highlighted in # the API for parsing objects could be improved.
currently you have to do the following to load a credential definition:
rsCredentialDefinitions[credDefId] = CredentialDefinition.load(JSON.stringify(credentialDefinitions[credDefId]))

It would be nice if load took a json object instead of a string. We also discussed that e.g. Presentation.create could just take json objects as input, and do the loading in the Presentation class so the user doesn't need to care about the object handles etc..
I think we should make the API just json objects wherever we can

I think it will be useful to implement these improvements before https://github.com/hyperledger/aries-framework-javascript/issues/1331, as otherwise users will need to care about freeing a lot object handles that we most likely will not be useful other than being inputs as some classes like Presentation.

@genaris did #129 and #130 fix this?

Yes, mostly. For some few cases the handle clearing by the app will still be needed (for instance the one of the resulting presentation in the example)


Ability to render GroundOverlays on top of map content such as buildings

Is your feature request related to a problem? Please describe.
GroundOverlays always render below map content (specifically buildings that are enabled via isBuildingEnabled = true in MapProperties for instance) and there isn't any way to make GroundOverlays be rendered on top (i.e. their z index is always below everything else on the map).
Describe the solution you'd like
Add an API that offers the ability to for GroundOverlays to be rendered above other map content, and especially for buildings as that is needed for my team's use case where we want buildings to be drawn below our GroundOverlay image.
Describe alternatives you've considered
We tried working with the various shape APIs (Circle, Polygon, and Polyline) to see if any of those rendered above map content and found that none of those do either, though those also don't offer the same functionality we need which only GroundOverlay offers anyways.
Additional context
Below you can see examples of how buildings are always rendered on top of GroundOverlay, Circle, and Polygon:

GroundOverlay
Circle
Polygon

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@zalexbag Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.

@lucanicoletti yes I did test changing the zIndex to a few different non-zero values including Float.MAX_VALUE and buildings still render on top of the GroundOverlay.

Oh and one other point of context to add to that, in the Maps Android SDK documentation for Shapes in the Z-index section they state the following:

The z-index specifies the stack order of this shape, relative to other overlays (other shapes, ground overlays and tile overlays) on the map. An overlay with a high z-index is drawn above overlays with lower z-indexes. Two overlays with the same z-index are drawn in an arbitrary order.
Note that markers are always drawn above other overlays, regardless of the z-index of the other overlays.

This is referring to shapes, but I would infer the behavior is the same for other overlay types and in this case for GroundOverlay. So the way I see it, the zIndex is meant to layer with regards to other overlays but has no effect when non-overlay elements are displayed like buildings or (as they mentioned) map markers.


feat: move and add theorems on String

Move helper simp theorems on String.Pos from Mathlib4 to Std4.
Move String.utf8GetAux.inductionOn from Mathlib4 to Std4.
Add new helper simp theorems on String.Pos.

I removed @[simp] from lemmas that unfold Pos without applying some kind of extensionality rule first. I don't think we want to just unfold this abstraction everywhere.


Bridge Incentive/Fee Model

Due to the efficient nature of merkle mountain ranges, transactions on harmony only need one block relayed later in the epoch in order to be verified. This introduces a tragedy of the commons situation to the game theory/economics/incentives of the bridge under a traditional fee model. The severity of this problem would depend how exactly how expensive relaying harmony blocks to ethereum is.
 This situation is at its worst for the first transaction after a block has been relayed.
Conversely, a user paying the lowest possible fee will still have their transaction relayed if others come in paying higher fees. The most extreme example being one participant paying the whole relay fee alone to confirm that an important transaction will cross the bridge as fast as possible, and the others crossing the bridge for free.
This is in essence, the freeloader problem associated with the tragedy of the commons, with the commons being the combined bridge fee that needs to add up to enough to incentivize a relay to relay the block to ethereum. Even if this does not happen significantly and the relay threshold is low, this still has the potential to add latency to the bridge if some participants fully understand the economics of the fee model.
The fee model I am proposing solves this problem with an "Ante" type of system to boost incentives for users to pay higher fees. More details on this including examples and perhaps some graphs to follow on this issue page soon.

Lets run an example.
Alice wants to cross the bridge.
The current cost for a relayer to be willing to relay a block is 100 tokens.
There are 40 other people waiting to cross the bridge currently. For simplicity's sake lets assume to worst case senario and say that they are all freeloaders and do not care at all when their transaction crosses the bridge.
The maximum charge for the ante is 5% of the fee pool. The time to reach maximum is 3 hours.
Alice submits her transaction with a fee of 50 tokens.
A grace period of 5 minutes is extended. (This period allows for relayers to relay the block if Alice had contributed enough for the block to be relayed on her own. This prevents attacks by submitting large fees then relaying the block yourself.)
After 5 minutes, the ante starts to increase linearly for all waiting users.
After 45 minutes, the other 40 users are paying 1.25% to match toward Alice's fee.
This totals to 25 tokens, or .625 for each user. The total relay bounty is now 75 tokens.
After 90 minutes, the other 40 users are paying 2.5% to match toward Alice's fee.
This totals to 100 tokens, or 1.25 tokens for each user. The total relay bounty is now 100 tokens.
A relayer picks up and relays the current block from the harmony blockchain. All 41 users including Alice are now able to verify their transactions using the MMR of the harmony epoch, and the merkle root of each block.
The users call in on the ethereum side of the bridge to complete the bridge crossing and unlock their tokens on etheruem.
The total fee paid by Alice was 50 tokens. Each other user only paid 1.25 tokens.
Alice's fee had a multiplier of 200% toward the bridge fee.
Alice still paid significantly more than the other users in the system. This reflects her high willingness to pay to have her transaction relayed.
The amount Alice paid for her fee had a direct impact on how soon her transaction crossed the bridge. Alice's transaction would have been relayed later if she paid 40 tokens and earlier if she paid 60 tokens.
Other users only paid a small portion of what Alice paid, while still contributing to the fee.
All the numbers in this example could be adjusted as necessary to find what we think is an optimal setup for efficient bridging. Don't necessarily think of the times, prices, or number of users as what we might see in the real operation of this system.


App height does not properly adjust with the keyboard open

Framework7 version:  3.0.0
Vue.js version: 2.0.0
Platform and Target: iOS 12 Cordova app

Describe the bug
They keyboard covers form inputs that are towards the bottom of the screen / the app height does not properly adjust with the keyboard open.
To Reproduce
Steps to reproduce the behavior:

Install Kitchen Sink and publish production build with Vue
Remove the black bars in the info.plst so its all fullscreen as described here
Run in iOS 12 iPhone X with Cordova and open Messages
See error

Expected behavior

Also, preferably without the nav bar disappearing but thats a minor issue at this point...
Actual Behavior

Additional context
Running iOS 12 beta on a real iPhone X. Same issue in simulators. The issue is also causing wonky behaviour with the form navigation buttons (the up/down buttons on the iOS keyboard to move between input fields), especially when these fields are towards the bottom. Same behaviour can be observed in Inputs and Form Storage pages.
Don't know if relevant but I noticed that when the app has the top & bottom black bars removed the keyboard additionally shows the Change language icon and the microphone icon; notice the difference in keyboards between the two gifs.

These issues are not related to Framework7. For keyboard issue use some of cordova keyboard plugins to handle this and shrink app accordingly. For issue with app running in part of window in iPhoneX, you need to specify correct size splash screen


Model Checkpointing + FSDP causes Cuda OOM

Bug description
I'm using FSDP and model checkpointing (default settings for both). My model has 254 million parameters. I'm not sure why but when I run Trainer.fit() it will successfully run the first epoch, then it will hit a CUDA OOM on the first backward pass of the second epoch. And this problem goes away when I disable model checkpointing, making me think it is a bug with FSDP model checkpointing. After all why should model checkpointing cause a CUDA OOM? Furthermore the problem persists with both state_dict_type='sharded' and regular model checkpoints.
Also the fact that the OOM happens AFTER the model checkpointing (during the next backward()) makes me think that the model checkpointing could be causing some kind of memory leak?
What version are you seeing the problem on?
v2.3
Error Message(s) (abbreviated):
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/pytorch_lightning/loops/optimization/automatic.py", line 239, in backward_fn
[rank4]:     call._call_strategy_hook(self.trainer, "backward", loss, optimizer)
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/pytorch_lightning/trainer/call.py", line 311, in _call_strategy_hook
[rank4]:     output = fn(*args, **kwargs)
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/pytorch_lightning/strategies/strategy.py", line 212, in backward
[rank4]:     self.precision_plugin.backward(closure_loss, self.lightning_module, optimizer, *args, **kwargs)
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/pytorch_lightning/plugins/precision/precision.py", line 72, in backward
[rank4]:     model.backward(tensor, *args, **kwargs)
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/pytorch_lightning/core/module.py", line 1103, in backward
[rank4]:     loss.backward(*args, **kwargs)
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/torch/_tensor.py", line 525, in backward
[rank4]:     torch.autograd.backward(
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/torch/autograd/init.py", line 267, in backward
[rank4]:     _engine_run_backward(
[rank4]:   File "/home/dsdeigh/miniforge3/envs/uqops+proxy/lib/python3.10/site-packages/torch/autograd/graph.py", line 744, in _engine_run_backward
[rank4]:     return Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
[rank4]: torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 7.57 GiB. GPU

Is this problem solved? I have just met the same one, maybe its related to the ckpt-save part


Add FlightCore to Northstar installer list

Adds FlightCore to list of Northstar installers.
TODO:

[ ] Add to compatibility matrix
[ ] ???

Keeping as draft as I wanna improve crash logging on FlightCore before I add it to wiki ^^


Final Group Capstone Project

Hello Code Reviewer 👋 👋

For this capstone project, we are made up of a  group of 4 team members. We are;

Asha
Elarh
Samuel
Rohit

This is a link to our Kanban Board and a screenshot has been attached to this issue

Hi Team,
In my opinion, you are ready to move on to the next step! ⏭
Good luck! 🎉
Highlights

Met all requirements 👍
Like that you separated the back and frontend task

Optional suggestions

Consider using labels to help further identify and filter tasks by type

Feel free to leave any questions or comments in the Issue thread if something is not 100% clear.
Cheers and Happy coding!👏👏👏

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update interactive-cli pom.xml

I've had the repo cloned for some time and after pulling the last changes "maven clean package" was giving me compile error only for interactive-cli:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project interactive-cli: Compilation failure
[ERROR] /Users/admin/Documents/librespot-java/interactive-cli/src/main/java/xyz/gianlu/librespot/interactivecli/Main.java:[82,33] constructor FileConfiguration in class xyz.gianlu.librespot.FileConfiguration cannot be applied to given types;
[ERROR]   required: java.io.File,java.lang.String[]
[ERROR]   found: java.lang.String[]
[ERROR]   reason: actual and formal argument lists differ in length

Which was weird because CI was passing. Right?
Then I looked closer to maven output:
[INFO] librespot-java 0.6.0 ............................... SUCCESS [  0.124 s]
[INFO] librespot-java common 0.6.0 ........................ SUCCESS [ 12.953 s]
[INFO] librespot-java core 0.6.0 .......................... SUCCESS [  5.190 s]
[INFO] librespot-java api 0.6.0 ........................... SUCCESS [  2.865 s]
[INFO] librespot-java api client 0.6.0 .................... SUCCESS [  3.634 s]
[INFO] librespot-java interactive CLI 0.5.2 ............... FAILURE [  0.097 s]

interactive-cli's clear difference was it's version. Then I looked at the latest build output and I noticed it's lacking the version:
[INFO] librespot-java ..................................... SUCCESS [  7.569 s]
[INFO] librespot-java common .............................. SUCCESS [01:05 min]
[INFO] librespot-java core ................................ SUCCESS [ 15.453 s]
[INFO] librespot-java api ................................. SUCCESS [  7.872 s]
[INFO] librespot-java api client .......................... SUCCESS [ 19.800 s]
[INFO] librespot-java interactive CLI ..................... SUCCESS [06:03 min]

So I looked at the source and found out changing the parent version did indeed fix my problem and wanted to shared my thoughts in the form of a PR.

I wonder why Travis didn't catch that.


Visualisation error in hourly_stacked_area chart

Some curves show minor errors in the visualisation when they go to 0 on the new hourly_stacked_area charts. It mainly seems to occur for battery curves.
Example on the price-sensitive-flex branch:

Example on the innovations-flex branch:

I knew I'd seen this problem before. It was fixed on the price-sensitive-flex branch, but the change wasn't incorporated into the innovations-flex branch. 🤦
@mabijkerk Your screenshot from the price-sensitive-flex branch shows the problem is present there, but I'm not able to reproduce it myself. The lack of a translation for the chart title and battery charging and discharging being the same colour makes me wonder if this is an old screenshot? It looks similar to the II3050 regional governance scenario which, for me, doesn't have this problem:

It was fixed on the price-sensitive-flex branch

My mistake, I thought it was still open there. The chart shown above for the price-sensitive-flex branch is indeed an old screenshot.
When I run the scenario now (II3050 National governance), the series do stack nicely:


AcroField checkbox problem with the check status

Hi,
I have a problem with editing an existing file.
When i want to check a checkbox on my pdf file everything is fine when i open the generated pdf on my computer with basic viewer software.
But when i open in adobe reader, the checkbox are unchecked.
Its very strange.
I use V1.2.1
the code for checking checkbox :
field.set(PDFName.of('V'), PDFName.of('Yes'));
field.set(PDFName.of('AS'), PDFName.of('Yes'));
Is someone have any ideas of the cause ?

I'm having the same issue with check boxes and also with text fields, I'm just using the example project downloaded from here. I tested it on Acrobat DC and Foxit reader, the same issue.

Hello @zepekenio and @samneace1212! I've updated the example in #109 to resolve this issue. Please see my comment here.

pdf-lib now has form creation and filling APIs that should be used instead of the above example(s). See the form filling JSFiddle for a working example. Additional information is available in the README and API docs. The PDFCheckBox.check method is of particular relevance to this issue.


What are the intended OSs/versions supported? dependencies?

Got error when executing in WS2012 R2: The program can't start because
api-ms-win-core-libraryloader-I1-2-1.dll is missing from your computer. Try reinstalling the program to fix this problem.

Should be addressed by https://github.com/microsoft/latte/releases/tag/v1.0.0, which is built for OS versions down to Win7.


Tree: use sealed

Description
Use api-extractor sealed in tree where appropriate.
Also avoids one use of undefined where a more specific type is possible.

I had to scope creep this slightly since my little cleanup for unknown -> json broke the fences so I moved our json types.


Exploits allows literally every E2 on the server to be downloaded without any perms

Another E2 stealing exploit, but this one is much worse. You can literally just download every E2 on the server. Looking at git blame it looks like this exploit has been in wiremod for about 10 years.
Demonstration here:
https://youtu.be/5e4-p2dfDqo

here is the commit that broke it, which means it's 2 years old https://github.com/wiremod/wire/commit/ec99005dd480becb35eccd7840cce160a0dcad66#diff-b21127288048d7221b5ae884a93b665d1149b0c0289ca3221671bd85b2e6f89aL121


[shuffle] Make shuffle mapper outputs deterministic

What do these changes do?
This PR make shuffle mapper outputs and order deterministic, so that we can read shuffle data by index instead of data keys in future, which will reduce shuffle meta overhead.
Related issue number
https://github.com/mars-project/meps/pull/2
https://github.com/mars-project/mars/issues/3054
https://github.com/mars-project/mars/pull/3040
https://github.com/mars-project/mars/issues/2916
Check code requirements

[ ] tests added / passed (if needed)
[ ] Ensure all linting tests pass, see here for how to run them

@wjsi
If there are lots of shuffle partitions, (shuffle_proxy_key, mapper_id, reducer_id) will have a big meta overhead. And it will be difficult to make mars support shuffle which has PB-scale data. If context insert order is not an appropriate assumption, how about order all shuffle outputs by reducer index in mars.services.subtask.utils.iter_output_data?

@wjsi If there are lots of shuffle partitions, (shuffle_proxy_key, mapper_id, reducer_id) will have a big meta overhead. And it will be difficult to make mars support shuffle which has PB-scale data. If context insert order is not an appropriate assumption, how about order all shuffle outputs by reducer index in mars.services.subtask.utils.iter_output_data?

That's acceptable.


vip_network_id non-existing parameter for elbv2

The following mapping seems to be wrong in https://github.com/opentelekomcloud/python-otcextensions/blob/master/otcextensions/sdk/elb/v2/load_balancer.py
 #: VIP netowrk ID
 vip_network_id = resource.Body('vip_network_id')

According the documentation https://docs.otc.t-systems.com/elb/doc/download/pdf/elb-api.pdf this parameter does not exist in ELBv2
Due to this update the functional tests are also failing for octavia-proxy on Zuul

I doubt. This class is actually not used at all. We should clearly drop it (and maybe it causes some conflict), but the error (would be nice to see the error also) is most likely caused by trying to set this parameter while invoking the SDK/OTCE

the functional tests of octavia-proxy clearly fails with error when using the latest version of otcextensions:
b'{"faultcode": "Client", "faultstring": "Validation failure: Unrecognized attribute(s) \'vip_network_id\'", "debuginfo": null}'

using older version 0.20.0 the tests passes
This parameter vip_network_id mapping was introduced in version 0.21.0

you can try locally dropping this file. I assume nothing will change

you are right, the error in octavia tests has nothing to do with the parameter in otcextensions, but with the latest updates in the octavia_proxy code...going to make an issue there...
regardless of the error in octavia, I believe this parameter vip_network_id should not be in elbv2
and btw the error in octavia comes directly from neutron:
RESP BODY: {"NeutronError": {"detail": "", "type": "HTTPBadRequest", "message": "Unrecognized attribute(s) 'vip_network_id'"}}

we are not really overriding octavia resources that we use for v2. Generally octavia-proxy should not even try passing vip_network_id. You get the error cause something tries to initialize it.
If this does not work perhaps we need to do same as with v3 and reimplement it on our side

fixed in octavia-proxy