Fix swift tests to work with in-tree LLDB

Several of the swift repl tests assume that lldb does not exist next to the swift driver in the binary directory. This patch updates the tests to remove that assumption. This allows the swift tests to correctly pass if LLDB is built in-tree.

@swift-ci please test

I don't understand the assumption here. I have some other things I need to do today, so I can look into it. But if you can explain the mechanism without me needing to engage my brain = ). It would be appreciated.

These tests are all checking for invocations of the legacy swift repl. If LLDB is in the directory next to swift they fail because it instead invokes the LLDB repl. This patch makes the tests pass by hard linking the swift driver into a temp directory before running those tests so that swift doesn't find LLDB sitting next to itself.

@llvm-beanz Can you change the commit message to be what you just wrote. Then it is completely self explanatory.

Also just in case, I know that @jrose-apple often times has opinions in this area. So we should see if he has any strong feelings.

@gottesmm I've updated the commit message with the explanation above.
@swift-ci plese smoke test

Would help if I could spell...
@swift-ci please smoke test

@swift-ci please smoke test

SGTM! Thanks!

LGTM as well. Thanks, Chris.


A0399 specimen and disease name

https://data.tst.oncomx.org/allbiomarkers/biomarker/A0399
Urinary bladder cancer panel lists specimen type and disease name as Lung Cancer, please confirm if this is correct.

Yes, this is an error.
I saw this error in the file before sending it to you and thought I had fixed it; good catch.
Corrections:
Specimen type:	urine (UN:0001088)
Disease name:		urinary bladder cancer (DOID:11054)

Please take a look at these panel files and let me know if we should revert back to the "original" from April 2022:
Current file: https://data.tst.oncomx.org/OMX_000095
April 2022 file:
human_cancer_biomarker_panel_vysis_urinary_bladder_cancer_042522.csv

Yes, all these values (Specimen type: lung (UN:0002048), Disease name: lung
cancer (DOID:1324)) are incorrect.
Correct values:
Specimen type: urine (UN:0001088)
Disease name: urinary bladder cancer (DOID:11054)
Also, I've noticed some issues with BCOs that we should discuss
Daniel Lyman, PhD
Senior Research Associate
Cancer Biomarkers Project Lead, HIVE-Lab
MS in Bioinformatics and Biochemistry Program Coordinator
Department of Biochemistry and Molecular Medicine
School of Medicine & Health Sciences
The George Washington University
Ross Hall
2300 Eye Street N.W.
Washington, DC 20037
Phone lab:<PHONE_NUMBER>
MS Program Office hours: T/W 9:00am-5:00pm
On Thu, Jan 26, 2023 at 10:37 AM Karina Martinez @.***>
wrote:

Please take a look at these panel files and let me know if we should
revert back to the "original" from April 2022:
Current file: https://data.tst.oncomx.org/OMX_000095
April 2022 file:
human_cancer_biomarker_panel_vysis_urinary_bladder_cancer_042522.csv
https://github.com/biomarker-ontology/biomarkerontology/files/10510782/human_cancer_biomarker_panel_vysis_urinary_bladder_cancer_042522.csv
—
Reply to this email directly, view it on GitHub
https://github.com/biomarker-ontology/biomarkerontology/issues/3#issuecomment-1405194166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4WWYUUBYBGT667HEG4BUP3WUKKZRANCNFSM6AAAAAAUFO6U64
.
You are receiving this because you were assigned.Message ID:
@.***>


lazygit_config_file_path overwritten

When I set vim.g.lazygit_config_file_path = '~/.config/lazygit/config.yml' lazygit override it to ''
Expected behavior
Use config from lazygit_config_file_path
Desktop (please complete the following information):
MacOS Big Sur
nvim 0.8
lazygit latest

You’ll need the following as well I think: vim.g.lazygit_use_custom_config_file_path = 1

lazygit.nvim overwrride lazygit_use_custom_config_file_path it as well.(set it to 0)
After I commented Plug 'kdheepak/lazygit.nvim' , variables set fine


Move reporter requirements to the project under test for Mocha strategy

This strategy was the easiest to move, so I started here. It's coordinated with the addition of https://github.com/learn-co-curriculum/curriculum-guide-writing-tests to the Curriculum Guide (already in) and https://github.com/learn-co-curriculum/node-js-npm-lab.
The main issue that I can see is that this change will break older Mocha labs. While we're migrating those labs to conform to these new requirements, I've kept some backwards-compatibility here.

nice! before we merge this in, let's have kaitlin take a QA pass at it, but then i'm all for it

@kvignali, if there's bandwidth, I would love to get this (or a fixed/improved version of it) in now that we're working on the JavaScript and Node.js tracks. Thanks!
cc @loganhasson

@pletcher bah, sorry i spaced on getting this in. will all our existing curriculum work out of the box with this?

They should -- I can try to carve out time to verify. The else branch preserves current functionality for labs that don't mention ".results.json" in their test script.

seems legit to me 👍

Hey all, I just added another change -- hopefully it's minor, but happy to put the brakes on if it seems too much.
I added a check for node_modules/ and an npm install run if the directory doesn't exist. This is because a lot of these labs will have some basic version of dependencies that students won't need to finish the lab but will need to run the tests -- basically just a hacky check so that we can install if needed.

💥


[FEATURE REQUEST] Add account highlighting when the account has not been reconciled

I'm dealing with a lot of accounts and I hate that I can't easily see which account remains to be reconciled.
Similar to this ticket: https://github.com/toolkit-for-ynab/toolkit-for-ynab/pull/2002
It would be nice to have a small indicator that shows which of the accounts have been reconciled.

I think this feature would work great as an alternative option to the "Emphasize Uncleared Accounts" feature.
I've added a card to Trello to track this here.


WebGlTile in a group => opacity of group ignored (Tile are always 100% opacity)

May be this bug is fix by https://github.com/openlayers/openlayers/issues/13507
See https://codesandbox.io/s/wms-image-custom-proj-forked-dfc1vm?file=/main.js
The map contains a single layer of type Group (ol/layer/Group).
The group contains a single layer of type WebGlTile (ol/layer/WebGlTile ).
After 3 seconds the code change the opacity of the group (from 1 to 0.5)
=> The WebGlTile layer has still opacity to 100% !
Note : If I change the type of the layer from WebGlTile to Tile (ol/layer/Tile), the layer opacity is well apply. The issue is only for WebGlTile

It is working with 6.14.1 https://codesandbox.io/s/wms-image-custom-proj-forked-67ox89

Fixed by #13509


Update scrollspy usage to have height requirement

Not sure if this is a workaround or actual usage, but without explicitly setting the height requirement on the spied-on element, the scrollspy will not work:
http://stackoverflow.com/questions/13134013/how-to-use-bootstrap-scroll-spy

Closing due to lack of response from OP.


Dynamically Allocated Executor

Implementation for #3334

I'm not sure that we should do this. My reasoning:

You can already use alloc to allocate tasks with the current public API with Box::leak(Box::new(TaskStorage::new())). This allows using generics and type inference and everything. Note this allows some things that the API in this PR doesn't allow:

You can reuse the storage by spawning it again when a task exits, if you keep the TaskStorage around.
You can use another allocator, it doesn't have to be Box. Anything that'll give you a &'static TaskStorage will work.

Dynamically allocated tasks still have severe downsides, mainly that you can't free the memory. This applies to both this PR and the TaskStorage API. This is unfixable, it's due to the core executor design (tasks living forever removes the need for refcounting wakers, which is what allows embassy-executor to be so fast+small). "Fixing" it would require adding refcounts and vtables so the executor design would basically collapse to the same as async-task.

I think no matter what we do embassy-executor is going to be a bad alloc executor, therefore we shouldn't encourage it. It's fine if it's possible to use TaskStorage with alloc, because it's "advanced" use that you have to go out of your way to do. Adding a more visible API like in this PR is basically telling people "hey go use this". They'll run into the limitation, or maybe not realize task storages are leaked and run into OOMs.
I do agree alloc has its use cases. We might want an embassy-alloc-executor crate, as a completely separate executor implementation. This way people would decide upfront whether they want alloc, picking between a GOOD alloc executor and a GOOD no-alloc executor. No weird limitations. The "alloc executor" would have the same design as async-task, fully refcounting tasks, or perhaps use async-task itself.
I do think we should expand docs on how to use TaskStorage with alloc. Crate docs, examples, FAQ entries would help. It's quite neat and there's little awareness of it.

@Dirbaio interesting, I'll experiment with the approaches you mentioned!


blocking the recovery of the ssh key

I try again.
it's almost deployed. it's just at the end , it retrieves the ssh key !!!! and it's still like that.
Jun 23 08:45:18 maas conjure-up/openstack: [DEBUG] 00_deploy-done:13 - Running deploy-done for OpenStack installation.

Moula.

Jun 23 12:36:04 maas conjure-up/openstack: [ERROR] Showing dialog for exception:#012Traceback (most recent call last):#012  File "/usr/share/conjure-up/conjure/juju.py", line 385, in do_add_all#012    params=params)#012  File "/usr/share/conjure-up/macumba/v2.py", line 113, in _request#012    'Params': params})#012  File "/usr/share/conjure-up/macumba/v2.py", line 124, in call#012    return self.receive(req_id, timeout)#012  File "/usr/share/conjure-up/macumba/api.py", line 126, in receive#012    raise ServerError(res['Error'], res)#012macumba.errors.ServerError: (ServerError(...), 'application "neutron-openvswitch" not found')#012#012During handling of the above exception, another exception occurred:#012#012Traceback (most recent call last):#012  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run#012    result = self.fn(*self.args, **self.kwargs)#012  File "/usr/share/conjure-up/conjure/juju.py", line 36, in _decorator#012    return f(*args, **kwargs)#012  File "/usr/share/conjure-up/conjure/juju.py", line 388, in do_add_all#012    exc_cb(e)#012  File "/usr/share/conjure-up/conjure/controllers/deploy/gui.py", line 25, in __handle_exception#012    app.ui.show_exception_message(exc)#012  File "/usr/share/conjure-up/conjure/ui/init.py", line 17, in show_exception_message#012    self.frame.body = ErrorView(ex.args[0])#012  File "/usr/share/conjure-up/ubuntui/views/error.py", line 23, in init#012    Padding.center_80(Color.error_major(Text(error))),#012  File "/usr/lib/python3/dist-packages/urwid/widget.py", line 836, in init#012    self.set_text(markup)#012  File "/usr/lib/python3/dist-packages/urwid/widget.py", line 879, in set_text#012    self._text, self._attrib = decompose_tagmarkup(markup)#012  File "/usr/lib/python3/dist-packages/urwid/util.py", line 375, in decompose_tagmarkup#012    tl, al = _tagmarkup_recurse(tm, None)#012  File "/usr/lib/python3/dist-packages/urwid/util.py", line 416, in _tagmarkup_recurse#012    raise TagMarkupException("Invalid markup element: %r" % tm)#012urwid.util.TagMarkupException: Invalid markup element: ServerError(ServerError(...), 'application "neutron-openvswitch" not found')
Jun 23 12:36:05 maas conjure-up/openstack: [DEBUG] 00_deploy-done:13 - Running deploy-done for OpenStack installation.


Game crashes during spinning animation after completing Cosmic Ocean.

Love the app. Thanks so much for it.
I'm on Windows 11, running on Steam conventionally. I've been able to warp to and beat Hundun and have my progress saved without any issues.
But when I warp to 7-98 and make it through the door, I spin for a bit and then the game crashes.
I suspect this is unavoidable behavior (it's possible that warping to 7-98 directly doesn't let the game know what it needs to know for the ending constellation), but just creating this issue in case it's a bug that can be resolved.

Just tested and works fine for me on the latest whip build, even when warping directly from Main Menu

I just got a Win 11 PC so I can see if that's specific to that, but I kinda doubt it.

Thanks!
Just tried again. Still an issue on my end.
Is there a debug mode I can enable so I can copy and paste more helpful output?
Right now, Overlunky just looks like this for me -
C:\Users\hi\Desktop\code>overlunky
[info] Overlunky version: 0.6.1
[info] Searching for Spel2.exe process... Pro tip: You have to launch it yourself.
[info] Found Spel2.exe PID: 4648
[debug] Allocated memory: 0x27201ca0000
[debug] Injecting DLL into process... C:\Program Files (x86)\Steam\steamapps\common\Spelunky 2\Overlunky\injected.dll
[debug] Calling: 0x7ffef4b8f500
[debug] Calling: 0x7ffe451daae0

Looks like you're possibly using 0.6.1 which is from Feb. Have you tried the WHIP build? https://github.com/spelunky-fyi/overlunky/releases/tag/whip

Thanks!
Just tried. Crashed again in the same way!
C:\Users\hi\Desktop>overlunky
[info] Overlunky version: e653f1e
[info] Searching for Spel2.exe process... Pro tip: You have to launch it yourself.
[info] Found Spel2.exe PID: 33452
[debug] Allocated memory: 0x26500000000
[debug] Injecting DLL into process... C:\Program Files (x86)\Steam\steamapps\common\Spelunky 2\Overlunky\injected.dll
[debug] Calling: 0x7ffef4b8f500
[debug] Calling: 0x7ffe4509f7f0

Interesting, I can reproduce this on my Win11 machine. I'll try to figure out what's up, but not sure if I will get any fix done today.

I have no idea if we've tried to fix this or not, but I'm trying to clean up in here so what's up? The only consistent constellation crash I know of was loading the ending with level_count == 0, but going through the door should increment it anyway.


loading sprites issue

Hi,
i am using pixi 4.6 with osx, i can reproduce this "issue" with both firefox, chrome and safari
please note the "issue" here, i think that i need help investigating whats happening.
in my code i load everything with preloadjs, i can confirm that everything seems to be loaded (preloadjs fire the correct event) and then pick up the data from a local cache.
when i create pixi objects (of spritesheet, of bitmapdata, spine) sometimes (it's not easy to reproduce it) a random object get black with green box as the bitmap data it's not ready to be used.
i have try to do a super ugly setTimeout right after preloadjs fire the onComplete event, to give pixi more time to warm up (and to dirty check if it's a problem of not yet laoded data) but with any delay the problem happen again.
the game it's quite complex, and it doesn't happen with a simple test case so it's very hard to isolate this issue. also it's not repeatable and i am having quite a lot of difficulties :(
can someone point me in the right direction or give some suggestion on what i should investigate more?
tnx

Could you show me your code ?

@ramarro123 going to close since it's been a long time! If you've still got any issues don't hesitate to come back and raise a new issue or ask a question at http://www.html5gamedevs.com/forum/15-pixijs/


Leverage existing csharp source code generator

Translate GDScript to csharp
The following repos uses ClassDB to extract method and thier correspondence Ng parameters , properties and enum to source generate cscharp codes.
https://github.com/Delsin-Yu/CSharp-Wrapper-Generator-for-GDExtension/pull/30#pullrequestreview-2039579976

First, we need a good coverage of gdscript parser using c#
Second, we need to interface with dotnet Roslyn to source generate c# conversation
Third, we need to examine the repos provided to include Godot specific context of Array and Dictionary to cast from Gdscript to c# codes

@raulsntos
Do you see this repos relevant for Roslyn use case for the official Gdscript to CS conversion?

@ZerxZ
Do you see this repos relevant for automated or to speed up the porting of Gdscript examples user for demonstrating c++ Gdextension into c# integration tests code generation?

@elamaunt
Consider using gdunit4Net vs2022 test Adapter for writing assert statements with Fluent support to simplify the automated integration test code generation


TypeError: Arguments to path.join must be strings

First off, great job on this package so far.  This thing is pretty awesome.  I'm not very skilled in these things so far, but I've noticed some change in the behavior very recently
This error is thrown when I try to build:

Ooops. Something went wrong.
Arguments to path.join must be strings
TypeError: Arguments to path.join must be strings
at f (path.js:178:13)
at Object.filter (native)
at Object.win32.join (path.js:183:38)
at Object.module.exports.isEligable (C:\Users\BaSiK-i5.atom\packages\build\lib\tools\atom-build.js:10:29)
at C:\Users\BaSiK-i5.atom\packages\build\lib\build.js:180:23
at arrayFilter (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\lodash\index.js:1409:13)
at Function.filter (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\lodash\index.js:6289:14)
at Object. (C:\Users\BaSiK-i5.atom\packages\build\lib\build.js:179:18)
at Object.tryCatcher (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\util.js:24:31)
at Promise._settlePromiseFromHandler (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\promise.js:454:31)
at Promise._settlePromiseAt (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\promise.js:530:18)
at Promise._settlePromises (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\promise.js:646:14)
at Async._drainQueue (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\async.js:182:16)
at Async._drainQueues (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\async.js:192:10)
at MutationObserver.Async.drainQueues (C:\Users\BaSiK-i5\AppData\Local\atom\app-0.210.0\resources\app.asar\node_modules\babel-core\node_modules\bluebird\js\main\async.js:15:14)

Here's what I have in the .json file:

{
"cmd": "W:\handmade\code\build-atom.bat",
"args":[],
"sh": false,
"env": {},
"errorMatch": "(?<file>.+)\((?<line>\d+)\)"
}

My bet is that I'm doing something wrong - I have a lay-mans understanding of Regex (read: noob), so I removed it to see if it reacted the same way.  It does.
After my build attempts threw this error a couple times, I went to check the .json file to see if I had accidentally deleted something, and after I checked that it was fine, I tried to compile again.  It worked!
I've confirmed that this behavior persists - trying to run build fails until I open the .json file and compile from there.
I realize that a lot of what's in that error is found elsewhere, but hopefully you can point me in the right direction.  Thanks for the help.

Thanks for the praise, and thanks for taking the time to provide such a detailed description!
Hmm, could it be that you don't have a project opened? I believe this function is the reason it doesn't work and most likely because atom.project.getPaths() return an empty array. This is definitely a bug in atom-build which I will address (with improved error handling).
However, if you have no active file, and no active project - what do you expect to build?

I'm not sure that's it,  but I could be misunderstanding the way the active
file/project is identified in atom.   I currently have it set to always
open the project I've been working on.   Pretty much,  every time I open
the editor I have the project folder in the tree view,  and my most recent
files up and running.
I just had a quick look at the error again and it pointed me at your
'target' code.  This is embarrassing,  and I'm not sure how it happened.  I
remember accidentally using that keybind  + t and spazzing out with
my mouse because I didn't know what it was.   I just used it again,
clicked on "default"  and it started working again.   I don't have any
other targets listed,  so I'm not sure how it could have been set to
anything else.   Would typing garbage into it have broken something?
I'll have to check it out again later tonight.
On Mon, Jun 22, 2015, 4:44 AM Alexander Olsson<EMAIL_ADDRESS>wrote:

Thanks for the praise, and thanks for taking the time to provide such a
detailed description!
Hmm, could it be that you don't have a project opened? I believe this
function
https://github.com/noseglid/atom-build/blob/master/lib/build.js#L136-L148
is the reason it doesn't work and most likely because
atom.project.getPaths() return an empty array. This is definitely a bug
in atom-build which I will address (with improved error handling).
However, if you have no active file, and no active project - what do you
expect to build?
—
Reply to this email directly or view it on GitHub
https://github.com/noseglid/atom-build/issues/104#issuecomment-114075906
.

I understand you consider this closed, but I looked into this after the fact.  I DID have a project open every time I tried this, and the whole "target" thing is supposed to be default, according to what I've read in here.  Can you think of ANY other reason that the call to the function you mentioned would fail?
I recreated the error and went into the dev tools to see what the call to getActiveTextEditor() was returning.  It was functioning as expected.  TextEditor is reading the active editor tabs and putting them in the array, as expected, but the check in your code still failed them.  For now, I've simply turned this (if (!textEditor || !textEditor.getPath()) { /* your code here*/}) off, and it's working just fine.
I haven't pulled your latest commit to see if it changes anything.  I'll look into that tomorrow morning.
I appreciate your help - like I said before, I'm fairly new at this, so I apologize for the misunderstandings that come with that.


BUG: != applied to disjunction before structural cycle is concrete

cue version: 42346df8456e476a484323b2ae8128fec8e512d4
The following yields an error:
Value :: 0 | 1

foo: Value
foo: != bar
bar: Value
bar: != foo

:!cue eval bug.cue --all
foo: empty disjunction: invalid value 0 (excluded by !=(0 | 1)):
    ./scratch.cue:4:6
    ./scratch.cue:1:9
bar: error evaluating bound:
    ./scratch.cue:6:6

shell returned 1

However, the following doesn't:
Value:: 0 | 1

foo: Value
foo: != bar
bar: Value
bar: != foo

bar: 0

:!cue eval bug.cue --all
Value :: 0 | 1
foo:     1
bar:     0

likely related https://github.com/cuelang/cue/issues/242

The error message may be unhelpful, but that does seem like an invalid cycle to me.
I am guessing that you expected unevaluated CUE. That is strictly more correct in the above case, although in both cases, exporting results in an error.
Note that at attempting to making

Here are test cases cover this issue https://github.com/rudolph9/cue/compare/bd2173eb2c9ffa4b75bc43c9b36dbd19caf0e196...rudolph9:gh-242

Testing out the 0.3.0-alpha1, the bug appears to be fixed!
cell: a:  0 | 1
cell: a:  != cell.b
cell: b:  0 | 1
cell: b:  != cell.a
cell: b:  0

:!cue eval --concrete fiename.cue
cell: {
    a: 1
    b: 0
}

cell: a:  0 | 1
cell: a:  != cell.b
cell: b:  0 | 1
cell: b:  != cell.a
cell: a:  1

:!cue eval --concrete fiename.cue
cell: {
    a: 1
    b: 0
}

I'll leave the issue open until changes are included in a non-alpha release.

See https://github.com/cuelang/cue/blob/master/cue/testdata/cycle/issue241.txtar

This issue has been migrated to https://github.com/cue-lang/cue/issues/241.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


(HC-33) Support adding element when array does not exist yet

This commit changes the behavior for applying an array_element when
the array does not exist in the target file yet from displaying an error
to creating the array and populating it with the element.

Ping @nwolfe and @fpringvaldsen for review.

:+1:

:+1:


util: Make get_custom_time_in_minutes throw error for unknown unit.

Follow-up for this PR comment: https://github.com/zulip/zulip/pull/30510#discussion_r1803859377.
The helper function get_custom_time_in_minutes still returns the time value from the input, so that the app doesn't break when the error is thrown. Currently, this is only used in the invite users modal, but will likely be used in other modals where a user can set a custom time with different time units (minutes, days, etc.).

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


Percentagefix following PR #903

This will work after accepting PR #903

@ffurano can you remove your username from the config file?

I think you mean users.demo.json
sorry, it slipped in

@ffurano can you rebase with master and fix the conflict?


Could not find the built application bundle at build/ios/iphonesimulator/Runner.app

Issue
Flutter run is building .app but looking for Runner.app to launch for iOS simulator
Steps to Reproduce

flutter run on an ios simulator; am using iPhone 11 simulator

Logs
 $ flutter run
Launching lib/main.dart on iPhone 11 in debug mode...

Running pod install...                                             38.1s
Running Xcode build...

 └─Compiling, linking and signing...                        53.3s
Xcode build done.                                           227.8s
Could not find the built application bundle at build/ios/iphonesimulator/Runner.app.
Error launching application on iPhone 11.

Build generated by flutter run
 $ ls build/ios/iphonesimulator/
Praja.app

flutter doctor -v
[✓] Flutter (Channel unknown, v1.15.17, on Mac OS X 10.15.4 19E287, locale en-IN)
    • Flutter version 1.15.17 at /Users/dheeraj2dj/flutter
    • Framework revision 2294d75bfa (6 weeks ago), 2020-03-07 00:28:38 +0900
    • Engine revision 5aff311948
    • Dart version 2.8.0 (build 2.8.0-dev.12.0 9983424a3c)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/dheeraj2dj/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] IntelliJ IDEA Community Edition (version 2019.3.4)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] VS Code (version 1.36.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.4.1

[✓] Connected device (3 available)
    • iPhone 11  • 9EF4EB5C-3E5E-4E51-9287-A651536F880B • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-4 (simulator)
    • Chrome     • chrome                               • web-javascript • Google Chrome 80.0.3987.163
    • Web Server • web-server                           • web-javascript • Flutter Tools

! Doctor found issues in 1 category.

I have same issue that flutter can't run my app on iOS, I have been working hard to find out but seem not work, I think this is cause of the Xcode

Have solved this by removing (moving) ios folder and recreating it by flutter create -i swift .
Ofcourse, I had to copy Info.plist, GoogleService-Info.plist & other necessary signing & capabilities. But it worked!


Update documentation on .ps1 usage

Shorten command to use powershell -ex AllSigned instead of using Set-ExecutionPolicy in the inline script

Usage of AllSigned is more secure than the initial Bypass in that signatures of loaded modules will be verified. Both configurations will work with the install script.

Clarify when script signature is checked
Document advanced usage of the script

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/137

View Changes | Compare Changes

Project: todo-csharp-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-mongo -b pr/137

View Changes | Compare Changes

Project: todo-csharp-sql
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-sql -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo-aca
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-aca -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo-swa-func
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-swa-func -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo-aca
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-aca -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo-swa-func
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-swa-func -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo -b pr/137

View Changes | Compare Changes

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-csharp-cosmos-sql
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-cosmos-sql -b pr/137

View Changes | Compare Changes

Project: todo-csharp-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-mongo -b pr/137

View Changes | Compare Changes

Project: todo-csharp-sql
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-csharp-sql -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo-aca
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-aca -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo-swa-func
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo-swa-func -b pr/137

View Changes | Compare Changes

Project: todo-nodejs-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-nodejs-mongo -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo-aca
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-aca -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo-swa-func
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo-swa-func -b pr/137

View Changes | Compare Changes

Project: todo-python-mongo
Remote: azure-samples
Branch: pr/137
You can initialize this project with:
azd init -t Azure-Samples/todo-python-mongo -b pr/137

View Changes | Compare Changes

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/137/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-137


Missing Step

The step for creating the role is missing this is what it should be
az cosmosdb sql role definition create
--resource-group $resourceGroupName
--account-name $cosmosName
--body '{
"RoleName": "Read Cosmos Metadata",
"Type": "CustomRole",
"AssignableScopes": ["/"],
"Permissions": [{
"DataActions": [
"Microsoft.DocumentDB/databaseAccounts/readMetadata"
]
}]
}'

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

ID: 9dc95f42-25e6-42ba-a3d9-94ed896d7383
Version Independent ID: 00936b2f-959b-b55f-60d3-deaa7bea6e17
Content: Use system-assigned managed identities to access Azure Cosmos DB data
Content Source: articles/cosmos-db/managed-identity-based-authentication.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

Hello @Jptasznik!
You are correct that we are missing a step. We will update this article as soon as we can and get the change live.
#assign:seesharprun


Add distance_from_spawn and distance_from_coordinates block/entity conditions

Add the distance_from_spawn and distance_from_coordinates entity and block conditions. They're the same but with different default values.
Fields
The 17 fields are as follows, only 2 are mandatory:

Field
Type
Default
Description

reference
String
world_origin (DFC)world_spawn (DFS)
The point to compare the distance to with.

offset_x/x
Double
0.0
By how much the reference should be offset on the X axis.

offset_y/y
Double
0.0
By how much the reference should be offset on the Y axis.

offset_z/z
Double
0.0
By how much the reference should be offset on the Z axis.

ignore_x
Boolean
false
Considers the distance on the X axis to be 0.

ignore_y
Boolean
false
Considers the distance on the Y axis to be 0.

ignore_z
Boolean
false
Considers the distance on the Z axis to be 0.

shape
String
euclidean
See notes below

scale_reference_to_dimension
Boolean
true
Whether or not to test for the coordinate the reference would have had if it "took a nether portal" to its new coordinates.

scale_distance_to_dimension
Boolean
false
Whether or not to scale the calculated distance based on the scale of the dimension the entity/block is in (e.g. multiplies the distance by 8 in the nether, such as if this flag is true, it checks for the distance you'd have to travel in the overworld from a portal at the block/entity to a portal at the reference point.

comparison
Comparison

How the calculated distance should be compared to the specified value.

compare_to
Double

The value to compare the Y position of the block to.

result_on_wrong_dimension
Boolean
optional
If set, will override the result of the comparison if the entity/block being tested is not in the reference's dimension.

round_to_digit
Integer
optional
If set, rounds the result to the closest number with the specified amount of digits after the comma. Negative numbers work (e.g. -2 rounds to multiples of 100).

reference is future-proofed to add player_spawn and player_natural_spawn once the modify_player_spawn power type is fixed. Note that given the current two references, the scale of the reference's dimension is supposed to be 1.0 since it's the overworld in both cases. If player/power spawns are implemented, the power will need to be reworked a little (some scaling to do in places, and I imagine two new fields), though I'm of the belief that it won't require breaking backward compatibility.
offset_<axis> and <axis> fields add up to each other (used as a cheap method of aliasing) to move the reference point (mostly useful with world_origin). The difference is that semantically, for DFS we deal with an offset, and for DFC while it still technically is an offset from 0 0 0 it's more meaningful to think of the coordinates directly (example below). Thus the aliases. My idea would be to only mention the "offsetful" syntax in the documentation for distance_from_spawn, and only the "offsetless" syntax in that of distance_from_coordinates.
ignore_<axis> can be used for testing the distance in 1D/2D; setting this and the one for Z to true makes the condition equal to height outside of edge cases (dimensions with a coordinate scale of 0 or wrong shape or can't get World from WorldView).
The shape is the same as for block_in_radius except it adds aliases for the academic names of the distances, and euclidean distance, which it defaults to. These changes were not added to the aforementioned condition, maybe they should. I'm not sure then whether to have the default value different for the two conditions, or to change the default for this one to be the Chebyshev (cube) distance.

Diagram

Example/test powers
Check out the power added to testdata too.
Generic demonstrative power (different from the one in testdata):
{
	"type": "apoli:action_over_time",
	"interval": 1,

	"rising_action": {
		"type": "apoli:execute_command",
		"command": "tellraw @s \"You are now in a radius of 8 blocks around 256 64 32\""
	},

	"falling_action": {
		"type": "apoli:execute_command",
		"command": "tellraw @s \"You are now outside the 8-block radius around 256 64 32\""
	},

	"condition": {
		"type": "apoli:distance_from_coordinates",
		"x": 256,
		"y": 64,
		"z": 32,
		"shape": "sphere",
		"comparison": "<",
		"compare_to": 8
	}
}

Test power (different from the one in testdata):
{
	"type": "apoli:action_over_time",
	"interval": 1,
	"entity_action": {
		"type": "apoli:if_else",

		"if_action": {
			"type": "apoli:execute_command",
			"command": "setblock ~ 255 ~ yellow_stained_glass keep"
		},
		"else_action": {
			"type": "apoli:execute_command",
			"command": "setblock ~ 255 ~ light_blue_stained_glass keep"
		},
		"condition": {
			"type": "apoli:distance_from_spawn",
			"shape": "cube",
			"ignore_y": true,
			"comparison": "<",
			"compare_to": 4
		}
	}
}

Result (the low FPS is because I was testing on capped FPS):

I wish I could split up the testCondition method, but it appears non-trivial given the edge cases that warrant early returns - else the code would be more complex or unnecessarily allocate objects.

I'm aware it's probably better to register the shape field as an actual shape and not a string. If you're willing to translate that (either way check out the bullet point), feel free to edit the branch, else I'll take a look myself at some point.

holy shit

Apart from my previous comment, I might replace the offset_x, ... and x, ... parameters with a vector data type in the future (maybe before release?). But that is nothing for you to worry about right now! :D
Other than that great PR. :) Thanks for putting in the effort of commenting it thoroughly and providing test powers. :) Will help lots in keeping this up-to-date with any changes!


Use min/max for string length constraints instead of minLength/maxLength

Current:
import { string, Schema } from 'schematox'

const static = { type: 'string', minLength: 1, maxLength: 2 } as const satisfies Schema
const programmatic = string.minLength(1).maxLength(2)

Desired:
import { string } from 'schematox'

const static = { type: 'string', min: 1, max: 2 } as const satisfies Schema
const programmatic = string.min(1).max(2)

It simply feels better and consistent with number schema

ArraySchema also has minLength and maxLength properties, considering that having verbose minLength on string  is not feels so bad anymore


Fix comments webhook panic

Closes #11940

ping LG-TM

@cornelk thank you for the PR :) would you be able to send this as a backport to the release/v1.12 branch?

@techknowlogick yes I will send a backport later


In archive schema, don't use account identifiers where token is implicitly the default

Otherwise, there's the possibility of an invalid state

Done.


rpm: change default binary compression to zstd

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
RPMs in our build system are currently being sequentially compressed with gzip level 9 by default. We can save time building packages, and reduce rpm file sizes, by switching to a different compression type. This PR sets %_binary_payload w7T0.zstdio in our macro file to change the default to zstd level 7 multithreaded compression. zstd compression is very fast, and can reduce RPM file sizes from ~3% - ~15%.
Reasoning for choosing zstd level 7:
zstd seems to give us the best balance of reduced build time, as well as reduced file size. I ran some tests with zstd compression levels of 3 (the default), 19 (the value used in Fedora 31), and 7 (something in-between). I found that using 19 provides smaller file sizes, at the expense of much worse build times. Level 7 reduces the file size significantly while requiring only slightly more build time than 3. See below for a comparison of compression type/levels when building our kernel package (kernel-<IP_ADDRESS>-1.cm2.x86_64) (which has among the longest build times of our core packages). Note that xz-7 compression has an advantage in the smallest file size, but is still much slower than zstd-7.

Compression Type
Compression level
Kernel-debuginfo RPM size
Time to generate kernel-debuginfo

gzip
9
913 MB
14 min 16 sec

xz
7
447 MB
6 min 12 sec

zstd
3
718 MB
39 sec

zstd
7
650 MB
43 sec

zstd
19
529 MB
21 min

For reference, Fedora changed from gzip to xz in Fedora 12 details. They changed to zstd, with compression level 19, in Fedora 31 details
Description and sample compression options: macros.in
Build time improvements:
These build times are for x64 OB buddy builds of some core mariner packages that take the longest to build:
kernel - 34 min (gzip) - 20 min (zstd)
pytorch - 69 min (gzip) - 57 min (zstd)
nodejs18 - 50 min (gzip) - 48 min (zstd)
File size reduction:
rpms.tar.gz file size changes, from main branch. This contains all core mariner RPM packages;
gzip - 21,010 MB
zstd - 17,884 MB
toolchain_built_rpms_all.tar.gz
gzip - 1,600 MB
zstd - 1,462 MB
Change Log

Change mariner-rpm-macros to set %_binary_payload w7T0.zstdio
Change toolchain manifests

Does this affect the toolchain?

YES
Test Methodology

Full build: 2.0.20230707-anphel-390037

To check the compression:
$ rpm -qp --qf "%{PAYLOADCOMPRESSOR} %{PAYLOADFLAGS} %{PAYLOADFORMAT}\n"    https://packages.microsoft.com/cbl-mariner/2.0/prod/base/x86_64/Packages/k/kernel-<IP_ADDRESS>-1.cm2.x86_64.rpm
gzip 9 cpio

$ rpm -qp --qf "%{PAYLOADCOMPRESSOR} %{PAYLOADFLAGS} %{PAYLOADFORMAT}\n"  ../out/RPMS/x86_64/kernel-<IP_ADDRESS>-12.cm2.x86_64.rpm
zstd 7T0 cpio

Does rpm have a dependency on zstd? Do we ship zstd on every image that ships rpm? Are there any other risks of taking this?

Does rpm have a dependency on zstd? Do we ship zstd on every image that ships rpm?

Yes, our rpm-libs package requires  zstd. And we've built rpm with zstd support since Mariner 1.0.

This generally looks great to me! I just have these 2 questions:

zstd seems like a clear winner. Could you help educate us on how we arrived on level 7 compression as the default? Is that worth benchmarking as well to find the right compression time vs. decompression time vs. size trade-off? (Or perhaps that's something that was already done / already exists somewhere?)
From the referenced links, it sounds like this has a chance of speeding up decompression/install times too! I'd be curious to know if any non-negligible impact was seen in image build, say, as that's a process that does quite a bit of package installation.

I've updated the PR description with an explaination of why I chose zstd level 7. For the image build times, it's possible they will improve. However, with the test runs I performed so far, the build time seems roughly the same.

Do you know if there is any way to swap in pigz here if its available? That was one of the biggest improvements to tooling stuff we found (outside rpm build).

Do you know if there is any way to swap in and test pigz here? That was one of the biggest improvements to tooling stuff we found (outside rpm build). We would need to add it to our chroot but its WAY faster than normal gzip (pigz = Parallel Implementation of Gzip)
edit: Looking at the macro files it seems like its a bit black-box unfortunately.

Had a dig, looks like this is using parallel zstd so we are not likely to see much improvement beyond this (the T0 in w7T0.zstdio means # cpu threads).

Do you know if there is any way to swap in and test pigz here? That was one of the biggest improvements to tooling stuff we found (outside rpm build). We would need to add it to our chroot but its WAY faster than normal gzip (pigz = Parallel Implementation of Gzip)
edit: Looking at the macro files it seems like its a bit black-box unfortunately.

Had a dig, looks like this is using parallel zstd so we are not likely to see much improvement beyond this (the T0 in w7T0.zstdio means # cpu threads).

Right, even if we could use pigz, the zstd compression shrinks the file sizes quite a bit over gzip.


Update synergy to 2.0.11

@vitorgalvao You mean this one?
After making all changes to the cask:

- [x] `brew cask audit --download {{cask_file}}` is error-free.
- [x] `brew cask style --fix {{cask_file}}` left no offenses.
- [x] The commit message includes the cask’s name and version.

Usually I use cask-repair which does this automatically. But this time (and the last time) they changed the capitalization for the synergy download file, so I had to edit the formula and create the PR manually.
Is there a convenient place where I could find the template in cases like this? I dug it out from an old PR of mine now, but maybe it would be good to have in a place like the contribution guide?

Usually I use cask-repair which does this automatically. But this time (and the last time) they changed the capitalization for the synergy download file, so I had to edit the formula and create the PR manually.

You can still use cask-repair for that. Either give it the --edit-cask flag, or just before submitting, when it asks you to confirm that everything is correct, press e.

I should add that I created the PR using hub. Maybe that's the problem?

It is. They have an issue on their tracker regarding adding support for PR templates.

When creating a PR through the Github GUI, will the template be added automatically?

Yes.

I see, that's great information, thanks a lot! I'll use the --edit-cask flag next time, then.


Fix docs, response instead of req in ExampleRedditAPI.md

Just req -> response.

Good catch, that is indeed more in line with the other examples!

Thanks very much for taking the time!


Snippets for the following steps in the default suite were not generated (check your configuration)

If your context does not implement SnippetAcceptingContext, and you have undefined steps, when you run Behat you will get:
--- Snippets for the following steps in the default suite were not generated (check your configuration):
...

I don't like "check your configuration" part, it can be missleading, as in this case, fix would be to just implement SnippetAcceptingContext.

I am agree with @umpirsky. It is not clear.

I also agree. I lost some time looking through config settings trying to correct this.


Pass attempt 0 to DelayForBackoff in AfterDelay

Passing 1 to DelayForBackoff results in the delay being doubled. If a delay
of 1 minute is used, it is unexpected that the polling delays for 2
minutes.
Encountered while debugging https://github.com/kubernetes/kubernetes/issues/35180#issuecomment-273085063

cc: @garimakhulbe @mcardosos
@codablock and others are reporting that the VM Client is not polling properly. From the linked issue, there's question as to whether or not CRP is reporting the header, whether go-autorest is respecting it, and then what looks like a bug with 1 being passed here instead of 0.
Could you take a look? Thanks!

Thanks for digging this up, @codablock & @colemickens. The change seemed pretty obvious to me, so I went ahead and merged it in. I'll sync-up with the rest of the Azure-SDK-for-Go team, and publish a patch if they sign-off.


Why show this on Windows 11

Prerequisites

[X] Verify that this is not a Windows issue;
[x] Refer to the System requirements;
[X] I do not use a homebrew Windows image;
[X] I did not tweak Windows before that could cause system instability;
[X] Attach a created log. You may find it in the Logs folder.

Steps To Reproduce
Run the .exe
Expected behavior

Actual behaviour
idk
Windows Version
Microsoft Windows 11 Pro 10.0.22000 22000
SophiApp version
<IP_ADDRESS>
SophiApp release type
SophiApp
Logs
Windows 11 Pro 21H2 build 22000.856
Computer name: ADMINISTRATOR
User: Peter
User domain: ADMINISTRATOR
User culture: Spanish (Colombia)
User region: Colombia
App version: <IP_ADDRESS>
App is release: True
App folder: "D:\Downloads\SophiApp"
App localization: EN
App theme: DARK
19.08.2022 14:04:21 Debug mode is: False
19.08.2022 14:04:21 Active view is: Loading
19.08.2022 14:04:21 Advanced settings is visible: False
19.08.2022 14:04:21 The "UWP for all users" switch state is: UNCHECKED
19.08.2022 14:04:21 The OS conditions checkings started
19.08.2022 14:04:21 OsVersionCondition run result: True
19.08.2022 14:04:21 The next condition to be run: OsBuildVersionCondition
19.08.2022 14:04:21 OsBuildVersionCondition run result: True
19.08.2022 14:04:21 The next condition to be run: OsFilesCorruptedCondition
19.08.2022 14:04:21 OsFilesCorruptedCondition run result: False
19.08.2022 14:04:21 The next condition to be run: RebootRequiredCondition
19.08.2022 14:04:22 Active view is: OsFilesCorrupted
19.08.2022 14:04:22 It took 0 second(s) to check the OS conditions

Hi! It is shown if some services were removed from Windows. Let's check: run this as admin in PowerShell one by one and send output
Get-CimInstance -ClassName MSFT_MpComputerStatus -Namespace root/microsoft/windows/defender

Get-Service -Name Windefend, SecurityHealthService, wscsvc

AMEngineVersion                 : <IP_ADDRESS>
AMProductVersion                : 4.18.2104.10
AMRunningMode                   : Not running
AMServiceEnabled                : False
AMServiceVersion                : <IP_ADDRESS>
AntispywareEnabled              : False
AntispywareSignatureAge         :<PHONE_NUMBER>
AntispywareSignatureLastUpdated :
AntispywareSignatureVersion     : <IP_ADDRESS>
AntivirusEnabled                : False
AntivirusSignatureAge           :<PHONE_NUMBER>
AntivirusSignatureLastUpdated   :
AntivirusSignatureVersion       : <IP_ADDRESS>
BehaviorMonitorEnabled          : False
ComputerID                      : A3A82DA5-F27D-4961-B0FE-DF126D0EBD54
ComputerState                   : 0
FullScanAge                     :<PHONE_NUMBER>
FullScanEndTime                 :
FullScanStartTime               :
IoavProtectionEnabled           : False
IsTamperProtected               : False
IsVirtualMachine                : False
LastFullScanSource              : 0
LastQuickScanSource             : 0
NISEnabled                      : False
NISEngineVersion                : <IP_ADDRESS>
NISSignatureAge                 :<PHONE_NUMBER>
NISSignatureLastUpdated         :
NISSignatureVersion             : <IP_ADDRESS>
OnAccessProtectionEnabled       : False
QuickScanAge                    :<PHONE_NUMBER>
QuickScanEndTime                :
QuickScanStartTime              :
RealTimeProtectionEnabled       : False
RealTimeScanDirection           : 0
TamperProtectionSource          : Signatures
PSComputerName                  :

Get-Service: Cannot find any service with service name 'SecurityHealthService'.
Status   Name               DisplayName

Stopped  Windefend          Servicio Antivirus de Microsoft Defen…
Stopped  wscsvc             Centro de seguridad

You removed one Defender related service. That's why launching this app was failed. This is the Bible: if Windows is destroyed, we play safe and block ability. How did it happen that Defender was destroyed and WMI cache with it?

I uninstalled it myself, because it conflicts with some activators or removes them without asking and well, I use independent antivirus

Ah, removed? :) So add your activators (I know this feeling) to the exclusion list is for the weaks? :) Why you didn't do that?
PowerShell is a powerful to manage Defender. Even if you do not know it, why you didn't just added your files manually? this is so strange.
Anyway, the issue, cannot be fixed. My advice: re-install the OS, deactivate Defender—not remove it (if you really want it). Add your activators and so on manually or by code to the exclusion list. Profit!


undecoded string fields from hdf5

Hi,
When I get a string field by silx_io.get_data(path-to-it), the returned value is actually of bytes type and needs decoding. This happens when I read from hdf5. And this is despite the Type is reported as string:

When I read from spec, the result is indeed str.
In my view, strings always should be decoded, so I consider it a bug (the hdf5 part, spec is fine).

In my opinion the "bug" is at the writer side: Sardana.
If not fixed by Sardana people, it will be automatically fixed by them when they will pass to Python 3. In terms of NeXus, both are accepted although it is preferred to have utf-8 strings.
With their current implementation, they are going to generate different output depending on the file being written under Python 2 or Python 3. What is in my opinion a bug at their side.

You will find a lot of literature at https://github.com/h5py/h5py/issues/379

indeed, it is a lot :)
You're probably right, this is on the Sardana side.
Thanks!

I never try to use get_data to read strings. There is probably a conversion to do here for HDF5 field stored as ascii. But the field can then contain dirty sequence of bytes. And i don't know if it's a good idea to abstract h5py.
But i would prefer to read strings as strings, and not as an abstract sequence of bytes.


Cannot get Instrumentation tests to run with androidx

I was wondering if you had run your instrumentation tests since migrating to androidx?
I'm getting:
"Test running failed: Instrumentation run failed due to 'Process crashed.'
Empty test suite."
If I reset to just before the androidx migrate commit, "git clean -dfx" then reopen in Android Studio, the tests run just fine.
I tried the solution for the related issue at https://github.com/pot8os/Kotlin-Espresso-sample/issues/7 but this didn't help. I didn't expect it to since it was posted before the migration to androidx.

Found the problem. Tried to push a branch but don't have write access.
In app/build.gradle change:
testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
to:
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"

@mylesbennett Thanks for your feedback! Can you make a pull request against the repo?


TFE: Proposal Asset: Dialog for errors - formatting

Attempt at posting an asset proposal with following payload:
{   "rationale": {     "title": "Add USDT Coin (USDT)",     "description": "Proposal to add USDT Coin (USDT) as an asset"   },   "terms": {     "newAsset": {       "changes": {         "name": "USDT Coin",         "symbol": "USDT",         "decimals": "18",         "quantum": "1",         "erc20": {           "contractAddress": "0xb404c51bbc10dcbe948077f18a4b8e553d160084"         }       }     },     "closingTimestamp":<PHONE_NUMBER>,     "enactmentTimestamp":<PHONE_NUMBER>   } }
Get the following transaction Failed dialog

Suggest we add formatting: punctuation and spacing if possible to enable it to read more clearly when multiple issues with payload

It maybe that given we are reworking this area - this scenario will be avoided by our UI forcing user to enter valid fields and relevant timestamps?

Given my last comment above - I will use this ticket to raise any further minor issues - rather than create a lot of noise about issues that may not actually exist after the reworking that is taking place in this area -


Enable bundling of personal scripts while preventing committing them

Previously, the bundler would ignore any scripts marked as personal (i.e., personal_*). However, this means it's nearly impossible to share scripts.
Now, you can run the bundler locally (described here) and it will bundle those scripts as well. Git will now also still ensure those scripts won't be committed into the repository.
As a side effect, this PR also allows one to create personal libraries. Any file in the library can be prefixed with personal_ and it will not be committed into the repo.

I know we should really do code reviews, but this one seems minor enough that I'm going to go ahead and merge it. That way I don't accidentally commit a bunch of personal scripts I shouldn't make public yet if I get around to doing another PR today.
If there is an issue, leave a comment. Happy to fix it in a follow-up PR.


Apple AirPlay

Would love to see AirPlay support in addition to https://github.com/fluid-player/fluid-player/issues/265

@leroybm thanks 😁 the reason I did open this was that videojs required a separate plugin for both


fix: redirect to native URI in Brave

This is a cosmetic fix that ensures IPFS resources are redirected to
ipfs:// or ipns:// URI and that URI is displayed in the browser UI
in the address bar.
Due to the way Brave implemented URI cloak for local HTTP-based gateway
provided by go-ipfs managed by the browser itself the regular redirect
done via webRequest API did not produce native URI in address bar.
However, tabs.update does the trick, so we use that for root requests.
We can remove this in the future, if webRequest behavior is fixed.
cc @autonome (mitigates issue we both discovered independently)

I'm fast-tracking this to Beta channel, so we can ship it together with #959 before BRave 1.19 hits Stable this week :dash:

I'm fast-tracking this to Beta channel, so we can ship it together with #959 before BRave 1.19 hits Stable this week :dash:


Packaging typescript app (lib.d.ts not found)

Hi, I am trying to package a typescript app that was compiled to .js using the typescript compiler tsc. The package seems to be created without any issues, however when I try to run the exe I get the error:
TypeScriptError: File 'C:/snapshot/script/node_modules/typescript/lib/lib.d.ts' not found.

So I added every .d.ts files as assets in package.json:
"assets": "**/*.d.ts"

...but that didn't work.
Sorry I am new to pkg, so any help would be appreciated.
Thank you!

Typescript should stay on devDependencies in package.json
{
  "name": "command",
  "version": "version",
  "bin": "result.min.js", // the compiled javascript
  "dependencies": {
  },
  "devDependencies": {
    "pkg": "4.3.1", // since latest version 4.3.4 have bug with node9, so you should use 4.3.1 instead.
    "typescript": "3.0.3", // compiler
  },
  "scripts": {
    "compile:prod": "webpack --mode=production", // compile, I use webpack, but tsc should work fine as well
    "build": "yarn compile:prod && pkg ." // build the executable command
  },
  "pkg": {
    "assets": [""] // just in case.
  },
}

That worked, thanks.


[ABW-3163] main settings screen re-organisation

Description
Related tickets
https://radixdlt.atlassian.net/browse/ABW-3163
https://radixdlt.atlassian.net/browse/ABW-3167
https://radixdlt.atlassian.net/browse/ABW-3173
Main settings redesign and re-organisation.

new UI for items
no olympia import card anymore
oragnised in spaced sections
big chunk of the change is just package rename to match name of new sections
new items are leading to empty screens now, changes will come in later PRs
we no longer query backup status at this settings level or entities which seed phrase needs backup
security center stub screen
moved seed phrases/ledger wallets under Settings/Security Center/Security Factors

How to test

Check 1st level wallet settings
Verify that "Preferences" are redesigned.
Verify security factors

Video
https://github.com/radixdlt/babylon-wallet-android/assets/118203440/7d1024af-f68c-4954-afcd-8ee8c880de98
PR submission checklist

[x] I have tested 1st level settings

I suggest a package structure that mirrors the UI:
settings/
     securitycenter/
         securityfactors/
               seedphrases/
               ledgers/
         backup/
      personas/
      approved dapps/
      linkedconnectors/
      preferences/
          guarantees/
          accounthiding/
          gateways/
      ...

At the moment we have under the settings package, these two packages that are not part of the new UI: accountsecurity ? appsettings ?
Although I strongly believe that it is easier to navigate to the certain screen in our codebase if we already know how it looks in UI, sometimes a deeply nested package/file can be annoying and If the rest of the team thinks we should avoid it then I am fine!

accountsecurity and appsettings will go away in next PR where I will reorganize all items - some of account security items will be under troubleshooting which is blank screen for now


Read and add data object size and modify time to HeadObject response

Read and add data object size and modify time to HeadObject response

This has been merged.  Closing this issue.


where can place init code in totaljs ?

// Example<EMAIL_ADDRESS>var Agent = require('sqlagent/pg').connect('connetion-string-to-postgresql');
var sql = new Agent();

Can i place this code in where on application ?, in model, controller, or any else ?
it call one or each query on model ?

Hi @lyquocnam,
I store SQLAgent initialization into the definitions/database.js e.g. https://github.com/totaljs/helpdesk/blob/master/definitions/database.js.

thanks @petersirka , got it !


How long is expected to take to complete the pre-training stage (on single GPU) ?

As titled, it seems it takes forever in the pre-training stage on the EMNLP2017 WMT News dataset.
What is a reasonable time for that?
Is there any configuration / hyper-parameters that I need to be aware of?

I think it’s because the BLEU evaluation has taken much time. So I suggest to use a smaller sample_size here, such as sample_size= 50 or 100.

@weilinie
It doesn't quite help after using a smaller sample_size=50.
The estimation completion time is still over 300 hours.
BTW, I am using tensorflow==1.15, does it matter?
May I know what hardware specification were you using when you produced the result.

That looks weird. Can you disable the BLEU score evaluation and see if it runs much faster? I didn’t test the code on TF1.15, so I’m not sure right now if it matters. For your reference, I mainly ran experiments on ubuntu 16.04 with GPU GTX 1080ti for ~1.5 days to get the real data results.

Closing it now. Feel free the reopen it if you have further questions.


Can't push HA -> Companion app notifications with images

I am trying to use this script in order to push notifications from my Eufy cameras using their image entities to my Android phone, but only text notifications appear without any images:

service: notify.mobile_app_fuatx3pro
data:
message: Motion detected
data:
image: /api/image_proxy/image.entrance_event_image

Try this in place of your image: line - Obviously replace your homeassistant url to your own and if not using ssl chnage to http:// instead of https:// - Have struggled with this a few times, and this seems the most reliable way to ensure the image shows up on the notification.
image: >-
  https://YOURHOMEASSISTANTURL/api/image_proxy/image.entrance_event_image?token={{state_attr('image.entrance_event_image','access_token')}}

Try this in place of your image: line - Obviously replace your homeassistant url to your own and if not using ssl chnage to http:// instead of https:// - Have struggled with this a few times, and this seems the most reliable way to ensure the image shows up on the notification.
image: >-
  https://YOURHOMEASSISTANTURL/api/image_proxy/image.entrance_event_image?token={{state_attr('image.entrance_event_image','access_token')}}

Tried it, doesnt work either.

I am not sure what is wrong, but it is working well for me here. I would choose the first approach over as home assistant itself is taking care of image generation for push notificaiton.
This setup has been working fine for me 99% of the time.
if you can enable debug log for all home assistant and share more logs, we can discuss further.


Add edit icon, delete duplicate policies icons

Adding edit icon so EE can supply non-Mui specific colors for dark mode. Adjust Pipelines and Delivery icons for EE dark mode
before:

after:

Unfortunately the Pipelines icon is in a tough spot - the fill on the middle section can not be hardcoded to white, but once it's transparent we get that extra filled-in section on the wings. Needs an adjustment in Figma.
Dark Mode Edit button:

I believe we need to keep it as it's, there's a huge difference in Icons as

but there's a chance to enhance the PolicyMode to mock the old styles color fill or size .
@AsmaaNabilBakr

Cool - made an issue here - we can't have the same icon twice so the svg will need to support both styles https://github.com/weaveworks/weave-gitops/issues/3782

Some tests are failing. The order of imports.

oop thanks Olga - fixed.


More convenient shortcut key for opening new Tab(s).

Description of the new feature/enhancement
A more suitable way to open new tabs, with the latest release as of now, the CTRL+Tab to switch between tabs was brought back, but to open new tabs it is still set to ctrl+shift+1 and show on.
For default new Tab it is set to Ctrl+Shift+T which doesn't appear much convenient. I believe we need the Ctrl+T way to open a new tab, which brings consistency in the way new tabs open in every other software.

Proposed technical implementation details (optional)

Well it is simple I just believe that CTRL+TAB is there for changing tabs then I guess we pretty much need CTRL+T to open a new tab with default and CTRL+1, CTRL+2, etc for the others.
Though CTRL+1, 2, etc are optional CTRL+T is a requirement I believe.
More Details
I understand it pretty well, that we can change it using the JSON keybindings, but I am requesting it as a default behavior and not as a customization.

Thanks for the suggestion! We've played with the defaults pretty extensively over the last year, and we've found that the Ctrl+Shift+key versions of the keyboard shortcuts are almost always better. The main reason is that a lot of commandline applications already have actions bound to Ctrl+key key chords. If the Terminal were to bind these keys by default, then most users would have to opt out of these keybindings. It's been less painful to instead have the few users who want to use Ctrl+key chords for the Terminal shortcuts to opt-in to them.

@zadjii-msft Well I understand the fact that you're stating.
But I recently shifted from the Ubuntu to Windows, just because of the Terminal and the Linux Subsystem being available on Windows.
And Ubuntu Terminal, as well as other Linux env have a Ctrl+key for the new tab, and I still believe it won't be a major problem with Ctrl+T since it is the most prominent for new tabs among all the terminals and CLI dev know this fact.
But Yet, if the dev team thinks it's for the best, then I think we have to assume it.


File size fix

fixes https://github.com/node-formidable/formidable/issues/607
@mlorthios @kmorte review and try this branch

This would be a breaking change so version 4.0.0

I am going to test this for a week

Made it a non breaking change by changing the new defaults


sniffio 1.3.0

Recipe directory diff between current master and this update:
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
index 0697768..61907b0 100644
--- a/recipe/meta.yaml
+++ b/recipe/meta.yaml
@@ -1,5 +1,5 @@
 {% set name = "sniffio" %}
-{% set version = "1.2.0" %}
+{% set version = "1.3.0" %}

 package:
   name: {{ name|lower }}
@@ -7,7 +7,7 @@ package:

 source:
   url: https://pypi.io/packages/source/{{ name[0] }}/{{ name }}/{{ name }}-{{ version }}.tar.gz
-  sha256: c4666eecec1d3f50960c6bdf61ab7bc350648da6c126e3cf6898d8cd4ddcd3de
+  sha256: e60305c5e5d314f5389259b7f22aaa33d8f7dee49763119234af3755c55b9101

 build:
   number: 1

Jira ticket:  (-)
The upstream data:
Actions:

Notes:
*
Additional info:

Package's statistics

Priority D | effort: easy | Category: other | subcategory:  | pkg_type: python
Outdated platfroms: 'linux-64', 'linux-aarch64', 'linux-ppc64le', 'osx-64', 'osx-arm64', 'win-64'
Latest version: 1.3.0
Release on PyPi:

version: 1.3.0
date: 2022-09-01T12:31:36

PyPi history
Popularity:

3 months downloads: 808143.0
All time:  5233108 downloads -  sniffio  1.3.0  Sniff out which async library your code is running under

Other checks:

[ ] Check the pinnings

[ ] Verify that the build_number is correct

[ ] Verify if the package needs setuptools

[ ] Verify if the package needs wheel

[ ] NO pip in test

[ ] Verify the test section

[ ] Verify if the package is architecture specific

[ ] Verify that private modules are not mentioned in the recipe For example: (_private_module)

[x] license_file: LICENSE.APACHE2 is present

[x] license_family Apache is present

[x] License: Apache-2.0
[x] License is spdx compliant

Check if the license identifier has correct name from the SPDX License List

Identifier

Apache-2.0

Check dependency issues:

../aggregate/sniffio-feedstock/recipe/meta.yaml
Dependencies: ['pip', 'python']
noarch:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-ppc64le:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

linux-s390x:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

osx-arm64:

py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

win-64:

py3.7:  No issues found
py3.8:  No issues found
py3.9:  No issues found
py3.10:  No issues found

Package Build Score: 17

Links:

Anaconda Recipes feedstock
Update branch
conda-forge recipe
PyPI
Diff between upstream and feature branch
Diff between origin and feature branch
The last merged PRs

Updating the recipe:
If the recipe needs additional modification the update branch can be modified. Note that the PR diffs are not updated with these changes.
git clone -b 1.3.0 git@github.com:AnacondaRecipes/sniffio-feedstock.git

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b4gm6935gy/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_b4gm6935gy/clone/recipe/meta.yaml:30: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_b4gm6935gy/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
ERROR: /tmp/abs_b4gm6935gy/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
Errors were found


Unable to solve challenge

Hi,
I'm trying to get html from this site:
https://www.netfreex.club/
but I receive an error. It looks like the challenge can't be solved.
I can see through the html the challege cf_chl_jschl_tk , but this site:
https://ilgeniodellostreaming.tw/?s=casa+di+carta (accessible only by Italian IP)
use the same challenge, and here it works.

Your mentioned site throws captcha after solving js-challenge. It works if you provide a 2captcha API key. And your Italian sites work globally it seems.


Option request : do not train word vectors

I would enjoy to experiment with an option that would do the following:

randomize the network, especially the word vector layers.
train the network WITHOUT updating the word vector layers.

Would it be hard to code ?

Hi @EtienneAb3d
this is an issue more related to the low-level tensor2tensor framework that we use as our core model. For this reason it is probably best to ask directly to the t2t community that could definitely be more helpful!
If you succeed into train an engine, regardless on how you did it, you can "plug" it into ModernMT without any problem. In fact, this is what we actually do: our training pipeline at the end encapsulates the t2t-train call.
Feel free to re-open the issue if you think there can be something we can do on ModernMT side!
Thanks,
Davide


Upload fusion dataset

(nuscenes dataset) Failed to upload dataset on open source version.

Fix.
It is expected that when uploading Lidar Fusion datasets, they can be displayed without a camera config file.

I have a same question,  what is the format of camera config file

You can refer this
https://docs.basic.ai/lidar/Coordinate-System#coordinate-system-1


More improvements

Moved from Libextract to Goose for text-extraction
Cleaned up asyncio calls
JSON-LD as fallback for date-extraction

Coverage increased (+0.02%) to 99.657% when pulling 5b5d2339b95146affc2d18765e8bacebff4817ce on mborho:master into ccd430d549e3172b0810193014c1e9e44567a08d on psolbach:master.


Line numbering in Jupyter Notebooks

First, let me say I'm using the Insiders version. I don't have this problem with the regular edition of VS Code.
I just can't see line numbers in any cells of my Jupyter Notebooks.
I press L with the cell selected but no numbering is visible.

Please upvote #96742. Closing as a duplicate.


Add constraints to HouseEducatorMapping

Follow-on to https://github.com/studentinsights/studentinsights/pull/2090, thanks failed Travis build! :) https://travis-ci.org/studentinsights/studentinsights/builds/429781187?utm_source=email&utm_medium=notification

selfie


Support for AXI MCDMA?

Hi!
Is your API support AXI MCDMA?

Hi!
I have not tested it with MCDMA. I assume that doesn't support.


Allow override of serving port by env var

Can now set HISHTORY_SERVER_PORT but will default to 8080.
A couple of caveats -

this is my first public PR so it's totally possible I've messed it up.
I've barely written any Go and don't have a working Go environment at present so this  might not actually work but I thought it was likely at least more helpful than just raising an issue :)

Thank you for the contribution! This LGTM and passed tests, so going ahead and merging. :)


Localization as a crate feature (compile-time option)

Enhancement's description:
Allow compiling without localization support for a lighter executable size.
Expected behavior / output:
By disabling a localization feature, the binary will compile without localization.

No interest, since the binary is now back to 1.2MB.


Documenting models

The user need to have a more friendly way to see the models.

#45 Some information in the platform
#10 Uploading pdfs
#52 Creating documentation from the model content

mentioned in issue #45

Added ~76 label


DOMException: Failed to execute 'appendBuffer' on 'SourceBuffer': This SourceBuffer has been removed from the parent media source.

DOMException: Failed to execute 'appendBuffer' on 'SourceBuffer': This SourceBuffer has been removed from the parent media source.
at msePlayer.readPacket (http://<IP_ADDRESS>:8083/pages/multiview/full?controls:681:34)
at WebSocket._this.ws.onmessage (http://<IP_ADDRESS>:8083/pages/multiview/full?controls:615:19)

I have the same problem. After some check we have notice that this was a behavior on a pc with lower performance. (16gb RAM memory) It seams that browser can't deal good enough with packets if pc doesn't have high performance. Is it possible?


!radar no longer opens the image

!radar responds with the url of an image. After a minor recent change #383 Slack no longer loads the image into the chat client.

Does it work if you do something like:
attachments = [{"text": "", "image_url": image_url}]
bot.post_message(channel, "", attachments=attachments)


chore: :passport_control: Remove DB and Design codeowner locks

Due to changing teams, it makes sense to remove the truss DB and design codeowner settings.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 8a065ebc3b2ee06b61ca52f6b6504e09ab1f76c3


Ideas for AST changes

We want to collect ideas about AST changes.

Change boxing rules: https://github.com/swc-project/swc/pull/9158

merge ForInStmt/ForOfStmt into forXStmt
use Function everywhere

const obj = {
    x() {}, // Function ✅
    set a(v) {}, // should use Function
    get b() {}, // should use Function
}

function foo() {} // Function ✅
const a = () => {} // should use Function
const b = () => expr; // We should use another form of representation, and it can be conveniently converted into a Function.

SWC AST Viewer

Merge ForInStmt/ForOfStmt into forXStmt

Use 👍 or 👎 to express your opinion.

Try using an arena allocator (eg bumpalo) for heap allocated values like Vec and Box. Would require passing lifetimes around unfortunately but apparently this is where oxc gets some of its performance wins. Could be worth testing to see how much impact it has.

Class
The attribute is_abstract should belong to the ClassDecl, but now it is an attribute of the Class.
SWC AST Viewer

BindingIdent should not have ident: Ident.
PrivateName should not have id: Ident.

Rename: ImportStarAsSpecifier => ImportNamespaceSpecifier.

Rename variants in TypeScript enums.

Fix TypeScript namespace vs module

Remove Key and merge PrivateName into PropName

JSX => Jsx

or

Ts => TS

Take trait => Default trait

Perhaps I should open a separate issue for this, but would you consider supporting a way to include custom types of expressions in the AST? For example:
enum Expr {
  // ...
  Custom(Box<dyn CustomExpr>)
}

trait CustomExpr: Eq, Hash, ..., swc_ecma_codegen::Node {}

The use case for this that I have is to store placeholder references in the AST for dependencies such as require, new URL(...) and other expressions that might get replaced by a bundler.
I did something similar to this in lightningcss, except using generics instead of dyn. But I'd recommend not doing that - it was quite painful to propagate the generics everywhere.


Update globalStyles.tsx

Html background was not displaying correct color based on the theme provided

I see a few changes need to be made to correct background colors on dark mode. Closing for now


Make several params to WorkloadIdentityCredential optional and read them from the environment instead.

I also haven't seen any credential reading from the environment variables (with an exception of EnvironmentCredential which behaves like that by design).
Did you see this behavior in some other language SDK?

Yes, for WorkloadIdentityCredential, the other language SDKs do behave this way and read from the environment variables, leaving the parameters optional.


support requiring less

Hey, where should I add tests?

Hey, where should I add tests?

test/core/test/require.test.ts

Hey @sheremet-va the failing tests seem unrelated to my change, also can you please review? because of this project's fast pace, the branch keeps getting out of date


[feat]: AlicanAkyuz/Submission Flow Photo Upload

The type of this PR is: Feature
This PR resolves SWA-86
Description
This PR rebuilds the photo-upload step of the artwork submission flow.
Video IOS and Convection
https://user-images.githubusercontent.com/42584148/153413225-5024b2bb-e5bb-4003-bffc-1d7d12fc8dc6.mov
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Dev changes
Since this feature is behind a feature flag, the changes apply to devs only.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 5711f59d5362e889cd688f4453eb6b553ff0ac5b

The PR looks good.
Here are a few potential improvements that we could discuss with the rest of the team before implementing.
Photo Placeholder format:

Show upload status

Display error message if the photo upload is unsuccessful

Add an X symbol on top right so that user can stop the upload in case their changed their mind (eg by accident uploaded the wrong file, upload takes too long)

Sometimes a Photo upload takes a longer time: Maybe add a spinner "This takes longer than expected, please hold on..."

When you close the app does the submission stop? If so we should ask them not to close the app while their data is being processed.

Display an error/info message if the user tries to upload an unsupported format

What happens if the user tries to upload one photo and then directly another one? If this is not supported, Add Photo could be disabled.

Upload takes a long time (at least for me)

Add info message: "Please wait while we upload your photo"

Maybe we can display the size of the photo that's being uploaded (so that the user knows it's a big file for example and can expect it to last longer)

Other comments:

On Artwork Details screen, maybe "Rarity" could be a dropdown instead of a modal
Location autosuggest looks a little weird (bigger letters)

Personally I'd be fine with merging this PR and opening another PR with the suggested enhancements if we decide it's something we want to implement.
Also since this is not a trivial ticket I would also consider ask for a review from Mounir, Pavlos or Brian.


Removing DATABASE_PORT from docker-compose example

DATABASE_PORT is useful when running manually without docker-compose but
the DATABASE_HOST db is referring to the docker-compose postgres which
is on the network in created in the docker-compose.

This brings up the question as to whether we need to expose the port at all on the postgres. I am assuming grafana is using it.

Nope, exposing the database port is not required in order to run the application. It may be useful if one would want to access the db from outside the docker container, though.

But I'm all for keeping the docker-compose.yml simple. Let's remove the DATABASE_PORT from the example.

True, but for the sake of the example probably not needed.

Yeah, you're right. Let's also remove it. Do you want to include it in this PR?

Done.

Cool, thanks!


Waterline associations issues

Waterline version:0.11.0
Node version:6.9.5
NPM version:3.10.10
Operating system:win8, ubuntu16.04

Hi guys, I'm using waterline associations and have a trouble:
I have 3 related models, and try to create'em one by one by bulk (Model.create([data])).
There 2 issues I found
1. Model.find().populate() doesnt collect all associated models, but model.findOne does.
2. Array more that 10 values of associated models when create(update and model.association.add(model)) return sometimes 9 sometimes 10 associations, but it's have to be more.
Here is testing data and outout:
Addin
{ name: 'addin one' },
{ name: 'addin two' },
{ name: 'addin three' },
{ name: 'addin four' },
{ name: 'addin five' },
OrderItem
{ name: 'orderitem one', addins: [{ id: 1 }, { id: 1 }, { id: 1 }, { id: 1 }, { id: 1 }] },
{ name: 'orderitem two', addins: [1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1], }, //27 values
{ name: 'orderitem three', addins: [1, 1, 1, 2, 3], },
{ name: 'orderitem four', addins: [1, 2, 3, 4, 5], },
Order
{ orderitems: [1, 2, 3, 4] },
{ orderitems: [1, 2, 3, 4] },
{ orderitems: [1, 2, 3, 4] },
RESULT:
MySQL OrderItem.find().popoulate("addins") result:
orderitem two  has  1  of addins
orderitem one  has  1  of addins
orderitem three  has  3  of addins
orderitem four  has  5  of addins
MySQL OrderItem.findOne().popoulate("addins") result:
orderitem two  has  10  of addins
orderitem one  has  5  of addins
orderitem three  has  5  of addins
orderitem four  has  5  of addins
localDiskDb OrderItemLDB.find().popoulate("addins") result:
orderitemLDB one  has  1  of addins
orderitem two  has  1  of addins
orderitemLDB three  has  3  of addins
orderitemLDB four  has  5  of addins
localDiskDb OrderItemLDB.findOne().popoulate("addins") result:
orderitemLDB one  has  1  of addins
orderitem two  has  1  of addins
orderitemLDB three  has  3  of addins
orderitemLDB four  has  5  of addins
there https://github.com/DryRuskFromLS/waterline-fail  I create a test - project.

Ok, fixed!

Ok, fixed!


Some wrong code in get_started document

On your get started website http://www.tensorflow.org/get_started/basic_usage.html#basic-usage
I found a small piece of wrong code:
In the sample code of Feeds section:
input1 = tf.placeholder(tf.types.float32)
input2 = tf.placeholder(tf.types.float32)

I tried them and python gave the AttributeError: 'module' object has no attribute 'types'
After reading the document http://www.tensorflow.org/resources/dims_types.html
I think it should be tf.float32 instead of tf.types.float32.  Probably it's a version issue?

Thanks for pointing this out! We have a fix in the pipeline, and it should appear on the website soon.

Thanks for the prompt reply!

Again I found very minor wrong number in the tutorial: http://www.tensorflow.org/tutorials/mnist/beginners/index.html#mnist-for-ml-beginners
It mentions the shape of training images are [60000, 784], but actually [55000, 784]; also for the labels: [55000, 10] instead of [60000, 10].

Thanks! it does help!!!!
Now I'm studying the newly tensorflow version based on the windows platform, and some demo codes of the guide TENSORFLOW.ORG gave do have some error, probably this guide is for python version 2.x, but the new tensorflow is based on python version 3.5.x.....

@Careenn If you have links to the tutorials that don't work in Python 3.5.x, please send them in a new issue (or sending a pull request that fixes them would be even better!).


Parser accepts invalid numerical values

The spec states:

"Similar to integers, you may use underscores to enhance readability. Each underscore must be surrounded by at least one digit."

I think we currently allow leading and trailing underscores: This should be fixed.
Auditing that part of the code to makes sure there aren't any other issues regarding what we allow/disallow regarding ints and floats would be a good thing in general.
This should probably be looked at together with #2
The code for this is in the Int, float, datetime arm of the parse_val() function in parser.rs

Hey,
I am still pretty new to this. But I'll give a try at this issue during the week-end.


Pin commands panel

The VS Code CMake extension has a Pinned Commands section in their action bar's panel.

The  "pinned commands" is not a CMake novel idea on pinning commands. It could be used by anyone. I am adding this feature request here to gauge interest by others.

What interest are you trying to gauge exactly with this issue? Are you referring to a specific VS Code extension that this team maintains?

@joshspicer does https://github.com/microsoft/vscode-cmake-tools/issues/3977 shed some light?

Unfortunately i'm still a bit confused. I understand you like what vscode cmake tools did here, are you asking for a way to pin any VS Code command and click it to run?  I'm seeing various extensions on the marketplace that might fit your need. Would something like this extension work?

are you asking for a way to pin any VS Code command and click it to run

Yes.

I understand you like what vscode cmake tools did here

Not necessarily so, I will most likely not use it myself. I added this issue here since CMake hade implemented such a section to "click-and-run-a-command" section in the GUI since I assume their users wanted it. I feel it is malplaced in a specific extension, as any user might want a "click-and-run-command" part of the GUI to pin their favorite commands. I personally prefer ctrl+shift+p and typing the command.

Would something like this extension work?

You are correct, and it does solve it. I was obviousl sloppy here, so I sincerely apologize for wasting your time.


KAFKA-9630; Replace OffsetsForLeaderEpoch request/response with automated protocol

This PR migrates the OffsetsForLeaderEpoch request/response to the automated protocol. It also refactors the OffsetsForLeaderEpochClient to use directly the internal structs generated by the automated protocol. It relies on the existing tests.
It seems that we could also refactor the broker (api layer, fetcher) to directly use the internal structs of the generated protocol but, as it is more involved, I propose to address it in a separate PR.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@chia7712 @hachikuji Thanks for your comments. I have addressed them. Could you have another quick look?

@dajac The updates LGTM.

@dajac +1 to nice updating. sorry that my commit causes conflicting files to you.

Failed test seems unrelated to this PR. Merging to trunk.


Honor the [@printer] annotation for variant types.

This patch makes ppx_deriving_show honor [@printer] annotations
on the cases of a variant type, closing
https://github.com/whitequark/ppx_deriving/issues/77.
In particular, consider the following code:
type t =
| First [@printer fun fmt _ -> Format.fprintf fmt "first case"]
| Second
[@@deriving show]
let () =
Format.printf "%a<EMAIL_ADDRESS>pp First pp Second
It now outputs "first case, Printer_variant.Second".

Looks good but please make indentation consistent with the rest of that file.

Thanks!


[Feature] Support EVA

Motivation
paper: https://arxiv.org/abs/2211.07636
code: https://github.com/baaivision/EVA
issue: https://github.com/open-mmlab/mmclassification/issues/1219
Modification
In addition, I fixed a bug in BEiT that caused ffn's add_identity to be True. fixed in https://github.com/open-mmlab/mmclassification/pull/1234
And fixed sklearn deprecated error.
Checklist
Before PR:

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[x] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[x] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[x] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[x] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects, like MMDet or MMSeg.
[x] CLA has been signed and all committers have signed the CLA in this PR.

the official repo  https://github.com/baaivision/EVA has released MAE style EVA model, Could you convert them all together?

@Ezra-Yu
You mean 'eva_psz14' and 'eva_psz14to16' ?
https://github.com/baaivision/EVA/tree/master/eva
I thought this convert script was fine for its, any improvements?

@Ezra-Yu You mean 'eva_psz14' and 'eva_psz14to16' ? https://github.com/baaivision/EVA/tree/master/eva

this https://github.com/baaivision/EVA/tree/master/eva#eva-l-learning-better-mim-representations-from-eva-clip; eva_l_psz14 and eva_l_psz14_21k_ft

I got it.

@Ezra-Yu Done!

@mzr1996 It looks good.


Remove collects.forEach

This tracks removing the forEach function defined in collections.ts. Instead you should use Object.entries() with a for...of loop
Current consumers:

[ ] src/vs/base/test/common/collections.test.ts @mjbvz
[x] src/vs/editor/browser/config/migrateOptions.ts @alexdima
[x] src/vs/platform/extensionManagement/common/extensionTipsService.ts @sandy081
[x] src/vs/platform/extensionManagement/electron-sandbox/extensionTipsService.ts @sandy081
[x] src/vs/workbench/api/browser/mainThreadDialogs.ts @jrieken
[x] src/vs/workbench/api/browser/mainThreadTask.ts @meganrogge
[x] src/vs/workbench/api/browser/viewsExtensionPoint.ts @sandy081
[ ] src/vs/workbench/api/test/browser/extHostTypeConverter.test.ts @mjbvz
[x] src/vs/workbench/contrib/extensions/browser/fileBasedRecommendations.ts @sandy081
[x] src/vs/workbench/contrib/snippets/browser/snippetsFile.ts @jrieken
[x] src/vs/workbench/contrib/tasks/browser/runAutomaticTasks.ts @meganrogge
[x] src/vs/workbench/services/actions/common/menusExtensionPoint.ts @sbatten
[x] src/vs/workbench/services/configuration/browser/configurationService.ts @sandy081
[ ] src/vs/workbench/services/configurationResolver/browser/baseConfigurationResolverService.ts @alexr00
[x] src/vs/workbench/services/extensionManagement/browser/webExtensionsScannerService.ts @sandy081
[ ] src/vs/workbench/services/extensions/common/abstractExtensionService.ts @alexdima

@alexr00 Assigned this item src/vs/workbench/services/configurationResolver/browser/baseConfigurationResolverService.ts to you, assuming you own it.

@sandy081 I now own tasks, so can take care of that one


Consdier only storing keys in wg.conf

After looking at the db folder I see my keys are stored in plain text. Ideally the keys would only be stored in wg.conf which is locked down appropriately.

Yes, for now, it is just simple like that. We don't have any method for encoding or encrypting atm. Any suggestion?

Another solution is we don't store the key in the DB. But then we have to use the same existing key in wg.conf when we update the file.


Update Multi tenancy with foreign keys.md

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Wrong tslint depricated warning

I believe there is a wrong deprication warning here.
This is my code:
  @ApiOkResponse({ type: RegisterByEmailResponseDto })
  @ApiException(() => UserAlreadyExistsException)
  @ApiException(() => InternalServerException)
  async register(
    @Body() registerInfo: RegisterDto,
   ...

And this is the warnning:
Deprecated: This decorator signature will be removed in future versions. Please provide HttpExceptions as function:

\@ApiException(() => BadRequestException, [options])eslintimport/no-deprecated

Hi @rostamiani, thank you for opening this issue. I believe this could be a problem with TSLint having issues detecting the deprecated overloaded decorator signature properly. As TSLint is deprecated and a migration to ESLint is advised: Is it possible for you to migrate your NestJS backend to ESLint?
We orientated at this comment and had no issues migrating to ESLint in our NestJS project.


Add support for nested expressions

Fixes #53
Parameters for helpers are parsed as mustache expressions if enclosed in parentheses.

LGTM