train lego exampl with neus_blender.yml catch non-zero size error

thx for great works. when i train lego example with neus_blender.yml , i got the non-zero size error, i tried change the seed number, but that did not work.
vmin, vmax = mesh_coarse['v_pos'].amin(dim=0), mesh_coarse['v_pos'].amax(dim=0)
IndexError: amin(): Expected reduction dim 0 to have non-zero size.

I believe this is a duplicate of https://github.com/bennyguo/instant-nsr-pl/issues/22. Please check it out to see if that gives you your answer.

您好，邮件已收到!--------------------------------------------吴建宝   @.***

hello, I meet the same problem. And I found that I must set the 'up_est_method'  to 'ground'  and set 'center_est_method'  to 'point', or the error will happen.

您好，邮件已收到!--------------------------------------------吴建宝   @.***


Rearrange list of all test cases alphabetically.

Easier to know where to add new one.
Avoiding duplications by mistakes.

nice! i should have looked at the pull requests before fixing this in mine.


unify the variable names that point to the same object in all programs

Related link
Discussion: https://github.com/orgs/aicers/discussions/20
issuse: https://github.com/aicers/patio/issues/22
Modifications

Modify roots to handle as a single root where it used to handle as an array.
Change config fields name

before
after

roots
root

ingest_address
ingest_srv_addr

publish_address
publish_srv_addr

graphql_address
graphql_srv_addr

https://github.com/orgs/aicers/discussions/20#discussioncomment-8990905 로 인한 내용 반영을 위해 repoen합니다.

해당 이슈는 https://github.com/orgs/aicers/discussions/20#discussioncomment-8990905  에 대한 수정사항이 없어서 다시 close합니다.


Different Field for Std Err

For example:
{
	"input": "there",
	"source": "print('hello' + 1)",
	"language": "python"
}

should probably respond with
{
  "ran": true,
  "language": "python",
  "version": "3.6.9",
  "output": "",
  "error": "Traceback (most recent call last):\n  File \"code.code\", line 1, in <module>\n    print('hello' + 1)\nTypeError: can only concatenate str (not \"int\") to str"
}

If this feature gets selected for development, I can help with it.
Thanks!

If this feature gets selected for development, I can help with it.
Thanks!

this is actually a planned feature already. for backwards compatibility output will stay as is and two new fields stdout and stderr will be introduced.

this is actually a planned feature already. for backwards compatibility output will stay as is and two new fields stdout and stderr will be introduced.

I guess this issue should be closed since stdout and stderr are now part of the response :)
This was a quick transformation from planned to implemented !

I guess this issue should be closed since stdout and stderr are now part of the response :)
This was a quick transformation from planned to implemented !

Yep, next on the list is stdin, which won't be hard, but will require a lot of copy pasting

Yep, next on the list is stdin, which won't be hard, but will require a lot of copy pasting


feat: fix bug that login by saml provider can not find application

fix: #1662

Unfortunately this does not work. The last call to /api/login fails with the following stacktrace:
subOwner = built-in, subName = admin, method = POST, urlPath = /api/login, obj.Owner = , obj.Name = , result = allow
2023/03/23 09:25:29.403 [I]  (<IP_ADDRESS>: ) subOwner = built-in, subName = admin, method = POST, urlPath = /api/login, obj.Owner = , obj.Name = , result = allow
2023/03/23 09:25:29.462 [C]  the request url is  /api/login
2023/03/23 09:25:29.462 [C]  Handler crashed with error runtime error: invalid memory address or nil pointer dereference
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/panic.go:1038
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/panic.go:221
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/signal_unix.go:735
2023/03/23 09:25:29.462 [C]  /go/src/casdoor/object/saml_sp.go:40
2023/03/23 09:25:29.462 [C]  /go/src/casdoor/controllers/auth.go:360
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/reflect/value.go:543
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/reflect/value.go:339
2023/03/23 09:25:29.462 [C]  /go/pkg/mod/github.com/beego/beego@v1.12.11/router.go:897
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/net/http/server.go:2879
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/net/http/server.go:1930
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/asm_amd64.s:1581

What can I do to help debug this further?

@leo220yuyaodog @hsluoyz
Unfortunately this does not work. The last call to /api/login fails with the following stacktrace:
subOwner = built-in, subName = admin, method = POST, urlPath = /api/login, obj.Owner = , obj.Name = , result = allow
2023/03/23 09:25:29.403 [I]  (<IP_ADDRESS>: ) subOwner = built-in, subName = admin, method = POST, urlPath = /api/login, obj.Owner = , obj.Name = , result = allow
2023/03/23 09:25:29.462 [C]  the request url is  /api/login
2023/03/23 09:25:29.462 [C]  Handler crashed with error runtime error: invalid memory address or nil pointer dereference
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/panic.go:1038
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/panic.go:221
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/signal_unix.go:735
2023/03/23 09:25:29.462 [C]  /go/src/casdoor/object/saml_sp.go:40
2023/03/23 09:25:29.462 [C]  /go/src/casdoor/controllers/auth.go:360
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/reflect/value.go:543
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/reflect/value.go:339
2023/03/23 09:25:29.462 [C]  /go/pkg/mod/github.com/beego/beego@v1.12.11/router.go:897
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/net/http/server.go:2879
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/net/http/server.go:1930
2023/03/23 09:25:29.462 [C]  /usr/local/go/src/runtime/asm_amd64.s:1581

What can I do to help debug this further?

casdoor provider:

keycloak 6.0.1:

test demo:
https://user-images.githubusercontent.com/71440988/227167768-e5a218de-b767-46ca-946a-756b431f7094.mp4
According the stacktrace, I guess casdoor does not get userinfo from the samlResponse.


[BUG] Pretrained weights unavailable for EfficientNet > B3

While trying to fine-tune some EfficientNets, I realized that networks larger than B3 don't have pre-trained weights available. This is with a fresh install of timm on Ubuntu:

B0-B2 work as intended. The code for timm seems to imply that B4-B8 should be there. I'd greatly appreciate a pointer here :) Thanks in advance.

By the way, thanks for the fantastic work!

Just for reference -- B3 works:

@aljoschaleonhardt not a bug, worth checking the code before filing issues.
https://github.com/rwightman/pytorch-image-models/blob/master/timm/models/efficientnet.py#L84-L294
There are multiple model definitions w/  weights > b3 ... including the largest L2 models, just not the native PyTorch variants that I trained myself (which don't have a tf_ prefix). This is the only reproduction of these models that includes all of the variants of original TF weights (to my knowledge).

@aljoschaleonhardt not a bug, worth checking the code before filing issues.
https://github.com/rwightman/pytorch-image-models/blob/master/timm/models/efficientnet.py#L84-L294
There are multiple model definitions w/  weights > b3 ... including the largest L2 models, just not the native PyTorch variants that I trained myself (which don't have a tf_ prefix). This is the only reproduction of these models that includes all of the variants of original TF weights (to my knowledge).

Also: https://github.com/rwightman/pytorch-image-models/blob/master/results/results-imagenet.csv

Sorry, I checked the wrong part of the code base -- many apologies for wasting your time on this one :/ Again, thanks for the lovely repo.


Dropdown won't open in IE if openOnFocus is false

Steps

I have a dropdown with search and selection enabled and openOnFocus={false}
I open the website in Internet explorer 11
I click the dropdown with the mouse

Expected Result
The dropdown should open when clicked.
Actual Result
I get a javascript error from this row in the code:
if (options[nextIndex].disabled) return _this.moveSelectionBy(offset, nextIndex);
saying that options[nextIndex] is undefined
The dropdown don't open
Version
0.64.5

Thanks for the report.  Can you please paste the full error or a screenshot of it?  Also, a working codepen will help us debug more thoroughly.
@layershifter I'm guessing this is due to different event handling on windows, could you confirm?

I'll check it on my PC tomorrow.

Sadly my Internet Explorer is locked to Swedish (work computer) so the error messages was not so usable. I will try to reproduce the bug on codepen tomorrow.

So here is a pen http://codepen.io/cattermo/pen/zNeqpY
The dropdown does open some times but it's very unreliable and seems to depend a bit on where I click (on the text, on the arrow, on the whitespace etc).
I'm on Windows 7 with Internet Explorer 11.0.9600.18499
I can't reproduce the javascript error, I will have to look into why that occurs in our app but not in the pen.

Thanks, getting closer here.  Could you list specifically what happens when you click each of those parts?  Example:

clicking text, opens dropdown
clicking dropdown icon, does not open
clicking whitespace inside the search field, throws error

Something like this will help us track down which event handlers need looked into.

My results are:

clicking once on placeholder text, does not open
double clicking on placeholder text, opens dropdown (!)
clicking once on arrow, does not open
double clicking on arrow, opens dropdown (!)
clicking on whitespace, opens dropdown
double clicking on whitespace, opens and closes dropdown (as expected)

In summary everything works fine when clicking on the whitespace (between placeholder text and arrow) but clicking the arrow or the placeholder text requires a double click to make the dropdown open.

We still have this problem in IE11 if anyone has the time to take a look...

@cattermo I lost this issue from my sight. I've checked Dropdown's docs, but can't reproduce issue. Can you make a codepen with mininal code that shows error in IE?

The bug can still be reproduced in the pen here http://codepen.io/cattermo/pen/zNeqpY (updated with semantic-ui-react 0.68.2)
I still have the same results as my comment from feb 14 says.

@cattermo Oh, I missed that code was already there. It's a known bug with onBlur and IE (facebook/react#3751).
PR is here (#1667), will be awesome to get a feedback.


Elral/issue36_2

This is an alternative pull request to #36.
In this case the config will be loaded again from the EEPROM into the already processed configBuffer (all "," and ";" are replaced by NULL), then it is transfered using already existing CMDMessenger function and at least the configBuffer will be processed again. Smaller changes in onResetBoard() and loadConfig() are required. They original functions are uncomment with additional comments. Advantage is that no changes in CMDMessenger is required, disadvantage is that a connected LCD will be switched ON / OFF every time the connector connects or requests the config.
As also changes in the CMDMessenger for wrapping the #defines are required, I would prefer to add the required function as described in the pull request #36 and to merge not this one.
This pull request will superseed pull request #13.

@elral I think it's safe for you to close this PR, I assume you are also more inclined to go with your first option for fixing this.


Export useIsomorphicLayoutEffect + docs

Closes #377

:tada: This PR is included in version 9.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix typo in startup.md

The following sentence:

IStartupFilter allows a different component call...

Is missing the word "to" and should read:

IStartupFilter allows a different component to call...

PS: the diff view for this PR seems to indicate I made a change with the text ::: moniker-end but I don't know how that happened or what it even means...just wanted to point it out here for the reviewers!

@ericmutta  thanks! Nice fix.

@ericmutta  thanks! Nice fix.

@ericmutta  looking forward to more of your PR's for ASP.NET Core docs. Thanks again.

@ericmutta  looking forward to more of your PR's for ASP.NET Core docs. Thanks again.

Many thanks for following up @Rick-Anderson - you and the team have done an amazing job of documenting ASP.NET :+1:

btw @ericmutta ... That 'moniker-end' Line 283 change is a new line (invisible character) change at the end of the file. GitHub prefers a new line on the last lines of files. When you edited the file, the new line was added ✨ magically ✨ by the edit ... the character change just can't be seen there 👻 on the diff.

btw @ericmutta ... That 'moniker-end' Line 283 change is a new line (invisible character) change at the end of the file. GitHub prefers a new line on the last lines of files. When you edited the file, the new line was added ✨ magically ✨ by the edit ... the character change just can't be seen there 👻 on the diff.

Thanks for the explanation @guardrex, I have learned something new today about GitHub :smile:

Thanks for the explanation @guardrex, I have learned something new today about GitHub :smile:


Release 0.1.7

Coverage remained the same at 38.739% when pulling f21841512fff48975379a9894c7ffb35f2db8a3f on develop into a79e498e1707f094907c04ef0671e05874930a45 on master.


Add FromReflect trait to convert dynamic types to concrete types

Dynamic types (DynamicStruct, DynamicTupleStruct, DynamicTuple, DynamicList and DynamicMap) are used when deserializing scenes, but currently they can only be applied to existing concrete types. This leads to issues when trying to spawn non trivial deserialized scene.
For components, the issue is avoided by requiring that reflected components implement FromResources FromWorld (or Default). When spawning, a new concrete type is created that way, and the dynamic type is applied to it. Unfortunately, some components don't have any valid implementation of these traits.
In addition, any Vec or HashMap inside a component will panic when a dynamic type is pushed into it (for instance, Text panics when adding a text section).
To solve this issue, this PR adds the FromReflect trait that creates a concrete type from a dynamic type that represent it, derives the trait alongside the Reflect trait, drops the FromResources FromWorld requirement on reflected components, and enables reflection for UI and Text bundles. It also adds the requirement that fields ignored with #[reflect(ignore)] implement Default, since we need to initialize them somehow.

This is definitely worth considering, but its also a pretty big change to the "scene/entity" lifecycle so I'll need to spend some time considering the design. I'll get to this as soon as I can, but I probably won't prioritize this for the 0.5 release.

To help track here are some related:
Issues:
#2215
PRs:
#2216 (attempts to fix)
#2359 (blocked)
#1429 (blocked)

I updated the PR and trimmed it down, I hope it feels more like a small improvement to fix an issue and less like a big change to the "scene/entity" lifecycle :-)
What I kept

add the FromReflect trait, which calls itself recursively on fields to construct a concrete copy of a possibly dynamic value
derive FromReflect along with Reflect, add a Default bound to ignored fields
manually implement FromReflect on SmallVec, Vec, Hashmap and Cow<'static, str>, fix List::push() on SmallVec and Vec
add a test to deserialize containers which items do not implement FromWorld/Default
change FromWorld bound to FromReflect in ReflectComponent (this could also be trimmed)

What I removed

reflect and enable deserialization on Text (that was my original motivation, but I will make a separate PR)
use FromReflect in Reflect::clone_value() implementations  (I think it should be done, but we can talk about it later)

This PR was discussed here: https://discord.com/channels/691052431525675048/745805740274614303/869276523503435817
I think the main point was: "Reflect should be derivable for every type, including those which cannot be constructed from a combination of FromReflect and Default impls" (from cart)
Suggested changes are:

don't use FromReflect in ReflectComponent
split FromReflect into its own derive

For future reference, I wanted to use FromReflect in ReflectComponent because I had several cases where a Component could not correctly implement Default/FromWorld. For instance, components that contain an Entity must implement Default with a fake Entity in order to be reflected (see Parent).

Have you seen the technique used in #2491? Perhaps we could use it here as well, to get elegant handling of the various initialization methods.

Have you seen the technique used in #2491? Perhaps we could use it here as well, to get elegant handling of the various initialization methods.

I don't think so unfortunately, there is no wrapping type like Local in this case.
After thinking a bit more about it, introducing FromReflect (especially as a separate derive) may not be worth the effort if the only remaining use is in List::push(). We might as well "fix" the issues by requiring that types inside containers that need reflection must implement Default, the same way we already require that reflected components must implement FromWorld. It's a small change that could be reverted later if we find a better solution, but we would need to impl Default for Entity because of Children, which I don't like.

Here is how the alternative looks like: https://github.com/Davier/bevy/commit/e621fcc41a597e53c2dcb04ab284f0542c32c58f
Please tell me which way you prefer, I'd be fine with either at this point.

but we would need to impl Default for Entity because of Children, which I don't like.

Other users structs containing Entity are very common too. The other option here is actually to use niches to make Option<Entity> the same size as Entity. IIRC @cart and @BoxyUwU were discussing this in some other thread; a NonZero u64 would work fine for us here IMO and Option<Entity> comes up in enough other places that I want this anyways.
Then, we could change Children to store Option<Entity>, giving us the correct semantics: it starts explicitly uninitialized.
Last time this came up, there was some frustration that we can't assign None to u64::MAX and the risk of off-by-one errors that can create. But given that the alternative is to create a dummy Entity id that is only used for defaulting or bloating the storage cost of every entity-containing component we need to serialize, that very much seems like the lesser of the evils.

I hate having to handle the invalid None case everywhere even more than initializing to an invalid entity id TBH

I reverted the modifications to the reflection example, since FromReflect will not be expected to always be derived along Reflect. Should I make a specific example deriving it for a struct that is inside a container, or is that too niche?
Also, should I do the refactoring to deduplicate the code between derive_reflect and derive_from_reflect? Or is it be better left for another PR, like #2377?

bors r+


mc_rtc compatibility with ROS2 Humble

Hi,
Because of long-term support for ROS2 Humble, I'm considering using it for Ubuntu 22.04. Is mc_rtc compatible with it?
Thank you

Yes.
This has been fully tested on ROS2 Iron on Ubuntu 22.04, and all relevant tools have been ported to ROS2 (mc_rtc's rviz panel and GUI widgets, surface/convex visualization, etc). The superbuild installation also supports ROS2.


@ngx-rx/state

Restructure state management logic under new package state.
Refactoring

[ ] Move reusable logic of rxjs-state into @ngx-rx/ts-etc and @ngx-rx/rxjs-etc
[ ] Keep duplicate in @ngx-rx/rxjs-state until we move it into another repo.

Public API

[ ] RxState

[ ] $
[ ] set
[ ] connect
[ ] get
[ ] select
[ ] hold

[ ] Operators

[ ] select
[ ] stateful
[ ] distinctUntilKeysChanged

Docs

[ ] API
[ ] Tutorial

please do also not forget the link in the main readme.md under the "Angular" headline

Thx @BenjaminDiscoVergy  did it!


issue with BehaviorInverter

hello i've been messing around with your plugin doing some basic stuff to learn how it works but the inverter is throwing an error in the code and i dont know how to fix it, i hope u can help with this because its a really cool plugin

I haven't touched any gdscript/godot programming for a few months, but my first guess is it looks like a syntax issue on your line 14. I would try with and without the typeof or extends check and see if one of those is the issue

seems like it's extends, should be changed to result is BehvError
considering this was reported to godotengine https://github.com/godotengine/godot/issues/11596


Running Downloader fails if the ticker_list includes only 1 ticker

setting the ticker to "AAPL" (str) will fail, because of the _base class's calc_time_zone


Allow creation of dragon sq shield

Description:

allows the creation of dragon sq shield
fixes #430

Changes:

[] I have tested all my changes thoroughly.

@jamosaur tests are failing


Drop Arc around SyncWaker

Related code:
https://github.com/Thomasdezeeuw/heph/blob/09f5ff59581f9727208c1030d334c365dfff06a2/src/actor/sync.rs#L145
Because internally Thread already uses an Arc.

We'll have to do some unsafe trickery, so not doing this now.


Add error handling for queuing functions

Following on from discussions in #10, this PR demonstrates a possible implementation of improved error handling, focusing just on one group of errors at present - those returned by libcamera_camera_queue_request.
There's a couple of ways of doing this, depending on how much one wants to read into the libcamera code.
The simplest solution is just to map all error numbers to a rust enum, e.g.
#[derive(Debug, thiserror::Error)]
pub enum LibcameraError {
    #[error("No device found")]
    ENODEV,
    #[error("Access error")]
    EACCES,
    #[error(transparent)]
    Unexpected(#[from] io::Error),
    // etc
}

// then map return codes to rust errors

let ret = libcamera_camera_queue_request(req);
match ret {
    0 => Ok(()),
    v if v == -libc::ENODEV => Err(LibcameraError::ENODEV),
    v if c == -libc......
    // catch all
    v => Err(LibcameraError::Unexpected(io::Error::from_raw_os_error(v)))
}

That would still be better than just returning an io::Error which would need to be interrogated with io::Error.kind() to yield the particular type.
In the pull request, I've demonstrated a fuller approach which gives specific reasons for why the code produced an error \but\ at the expense of reading through the libcamera C code and following all the error paths.
I've be interested to hear thoughts on how I've implemented it in the PR versus any other preferred way.

Just to add, I've added a basic test which relies upon a camera device being present... I'm not sure how to emulate that on CI.

Yes, adding sudo ldconfig after building libcamera is required lest we get the error: error while loading shared libraries: libcamera-base.so.0.0.4: cannot open shared object file: No such file or directory.
Apparently this should have been fixed, so I've re-raised the question: https://github.com/kbingham/libcamera/issues/17

I've just added an example for CameraManager, using a slightly different API (impl'ing from_raw_os_error directly on CameraManagerError to move error handling logic closer to the error type itself). This works when there are no clashes between libcamera error codes and their meanings, as is the case here.

In the pull request, I've demonstrated a fuller approach which gives specific reasons for why the code produced an error \but\ at the expense of reading through the libcamera C code and following all the error paths.
I've be interested to hear thoughts on how I've implemented it in the PR versus any other preferred way.

This is exactly how I think it should be done. I don't think there is much use of just re-writting io::Error like in the example.


Web version source code ?

Tried to search for emscripteen in repo, didnt find anything..
Are you planning to improve web version?
Url encoded .vcv file would be killer feature.

Wow, that would be amazing, especially using the web version with the WebAssembly!

bump! I'd love this as well. I'm trying to get it going from a fork of this repo with minimal success

Hello,
All the code is in this repo. To build, you need to set WEB environment variable and just run make dev / make, like so (on osx/linux) "WEB=1 made dep". There may be some paths to emscripten needed to be set, I'm not sure. I believe it is compiling into WebAssembly.
Unfortunately when working on the web version, I discovered that audio processing in web browsers still had issues, and I was experiencing audio artefacts even with simple patches. I remember discussions that audio processing in browser is not running at high/realtime priority (at least in browser/os combinations I was using). This is why I stopped developing the web version. Maybe this issue (and some other smaller ones) has already been resolved in current versions of browsers, I don't know.
If anyone wants to continue this work, feel free to ask any specific questions about the build process or the code, I'll help with what I still remember.

@pronvit Should be better by now, we have AudioWorklets and WebAssembly is getting more and more support.


[Solved] [Error] Downloading stash for Hardcore

Turns out my race character was moved to hardcore and it gave me an error for some reason, when i deleted it was solved.

For some reason race characters cause GGG's API to freak out, glad it's sorted.


typos discovered by IntelliJ inspections

Typos and minor code cleanups.

Current coverage is 100% (diff: 100%)

Merging #88 into master will not change coverage

@@           master   #88   diff @@
===================================
  Files          13    13
  Lines         827   827
  Methods       726   726
  Messages        0     0
  Branches      101   101
===================================
  Hits          827   827
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 71e22bb...87da834


Remove BLT_EXPORT_THIRDPARTY workaround

This resolves the following FIXME by incorporating the functionality added to BLT in https://github.com/LLNL/blt/pull/436:
https://github.com/LLNL/serac/blob/4dbb20800e4914ccfb5b42903f65f589cb03b610/examples/using-with-cmake/CMakeLists.txt#L7-L10
Fixes #470
Possibly this would have been noticed earlier if we had opened a GitHub issue instead of just leaving it as a TODO?

Codecov Report

Merging #472 (993c8f6) into develop (4dbb208) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #472   +/-   ##
========================================
  Coverage    92.64%   92.64%
========================================
  Files           47       47
  Lines         2435     2435
========================================
  Hits          2256     2256
  Misses         179      179

Continue to review full report at Codecov.

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

This resolves the following FIXME by incorporating the functionality added to BLT in LLNL/blt#436:
https://github.com/LLNL/serac/blob/4dbb20800e4914ccfb5b42903f65f589cb03b610/examples/using-with-cmake/CMakeLists.txt#L7-L10
Fixes #470
Possibly this would have been noticed earlier if we had opened a GitHub issue instead of just leaving it as a TODO?

Yeah, I try to catch this stuff and add issues as I see them, but I know we have some hanging out in the code base. We should probably do a search to grab all the TODO items.


Error cordova building with barcodescanner

Hi,
I'm trying to use phonegap-plugin-barcodescanner, i install it with the command line :
cordova plugin add com.phonegap.plugins.barcodescanner
and it works without problem, but when i try to build my project i have this error :
:processDebugResources FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':processDebugResources'.

com.android.ide.common.internal.LoggedErrorException: Failed to run command:
/home/pascall/www/android/android-sdk-linux/build-tools/22.0.1/aapt package -f --no-crunch -I /home/pascall/www/android/android-sdk-linux/platforms/android-22/android.jar -M /.........../platforms/android/build/intermediates/manifests/full/debug/AndroidManifest.xml -S /.........../platforms/android/build/intermediates/res/debug -A /.........../platforms/android/build/intermediates/assets/debug -m -J /.........../platforms/android/build/generated/source/r/debug -F /.........../platforms/android/build/intermediates/res/resources-debug.ap_ --debug-mode --custom-package io.cordova.hellocordova -0 apk --output-text-symbols /.........../platforms/android/build/intermediates/symbols/debug
Error Code:
132

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 43.013 secs
/.........../platforms/android/cordova/node_modules/q/q.js:126
throw e;
^
Error code 1 for command: /.........../platforms/android/gradlew with args: cdvBuildDebug,-b,/.........../platforms/android/build.gradle,-Dorg.gradle.daemon=true
ERROR building one of the platforms: Error: /.........../platforms/android/cordova/build: Command failed with exit code 8
You may not have the required environment or OS to build this project
Error: /.........../platforms/android/cordova/build: Command failed with exit code 8
at ChildProcess.whenDone (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/cordova/superspawn.js:134:23)
at ChildProcess.EventEmitter.emit (events.js:98:17)
at maybeClose (child_process.js:735:16)
at Process.ChildProcess._handle.onexit (child_process.js:802:5)
any idea ?
Thanks for your help
Best Regards
Pascall

Hi @mrbinr, are you still seeing? I tried using the same command as you and was unable to reproduce the issue. Also, "com.phonegap.plugins.barcodescanner" is the old plugin name. The new version is now called "phonegap-plugins-barcodescanner". If you are still seeing the issue try installing using the new plugin name.

Thanks for your answer, i gonna try it and will give you the feedback
Best Regards
Pascall

No movement in about a year. Closing this issue

i am trying to build o ionic android app but i don't have success. The return in my terminal is this bellow:
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':transformClassesWithDexForDebug'.

com.android.build.api.transform.TransformException: com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_60.jdk/Contents/Home/bin/java'' finished with non-zero exit value 2

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 24.663 secs
Error: Error code 1 for command: /path/to/app/platforms/android/gradlew with args: cdvBuildDebug,-b,/path/to/app/platforms/android/build.gradle,-Dorg.gradle.daemon=true,-Pandroid.useDeprecatedNdk=true
Someone can help me?


Create searchd.service

Systemd init file for searchd to put on /lib/systemd/system/

Going to be getting rid of Vagrant, so closing this.


Cannot start nuxt:  Cannot read properties of undefined (reading 'feed')

Hi !
I have a error when install nuxt 3:
Cannot start nuxt:  Cannot read properties of undefined (reading 'feed')
Thanks

Because the module is not yet compatible with Nuxt3. Follow #114  #106


Removeunused

Related issue: #1182
Type of change: other enhancement
Impact: no functional change
Development Phase: implementation
Release Notes
Remove unused imports.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: ucbjrl:x: tamplerYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: ucbjrl:x: tamplerYou have signed the CLA already but the status is still pending? Let us recheck it.

@tampler, could you sign the CLA so we can get this PR merged?

A bit outdated PR, but it's okay. I'll add more.
Signed CLA, thank you


Improve UI of Fitness centre Website

@pranjay-poddar I would like to improve the whole UI of the Fitness centre Website by adding extra features to each section. I will add animations to this website as well.
Please assign me this issue under GSSoC'23.

@pranjay-poddar Please review and merge it.


Add Rate filter to tabular view

Release Type: Non Breaking Feature
Fixes x-team-internal.atlassian.net/browse/XP-2472
Related to x-team-internal.atlassian.net/browse/XP-2434
Related to x-team-internal.atlassian.net/browse/XP-2459 (back-end)
Description
Include postLabel even when no label is provided to InputField
Previously we would only see the postLabel if a label prop was
provided to the InputField component. This is no longer the case.
This was done so that we could use it this way for the Rate filter feature
(see GIFs/screenshots over at https://github.com/x-team/auto/pull/1635)
Checklist

[x] set yourself as an assignee
[x] set appropriate labels for a PR (In Review or In Progress depending on its status)
[x] move respective Jira issue to the IN REVIEW column
[x] make sure your code lints (npm run lint)
[x] Flow typechecks passed (npm run flow)
[x] my change requires a change to the component's documentation (.snap)
[x] component's documentation (.snap) is changed or added accordingly
[x] if you've fixed a bug or added code that should be tested, add unit tests
[x] if any snapshots have been changed, verify that component still works and looks as expected and update the changed snapshot
[x] manually tested the component by running it in the browser and checked nothing is broken and operates as expected!

Related PRs

branch
PR

XP-2472-rate-selector-filter
Auto

XP-2459-filter-max-rate
Auto API

Impacted Areas in Application
The InputField component
Screenshots
Before

After

Rebased and resolved conflicts


Frontend optimizations

Minimize JavaScript in overlay (for example angular.js)
Use SASS, jsHint, jscs (and integrate them into the build processes)

I tried to move all JS code into dnn.js, but only the code from HostSettings worked out. When I try to move the JS from Overlay.aspx, angular is missing. When I try to move the JS from Toolbar.html, $.connection is undefined.

Improved JavaScript include order and moved JavaScript for toolbar to dnn.js.
dnn.js won't work for overlay, created new issue for this improvement, see #17


investigate user reports of "stalled" tests

Several users of the NDIA instance have reported "stalled" tests. This happens consistently but not every time the test is conducted. Mac, Windows, and Linux browsers have been confirmed to have this issue.
Upon investigation, this issue is confirmed to be a problem with mlab-ns providing clients hostnames for cloud nodes that are offline. The server request is provided with no error, but when a test is attempted, the hostname for the NDT service cannot be resolved.
Tracking in https://github.com/m-lab/mlab-ns/issues/223
https://speed.digitalinclusion.org
Using M-Lab Server ndt-iupui-mlab1-chs0c.measurement-lab.org
main.js:60 Location received
main.js:25 Test started.  Waiting for connection to server...
main.js:25 WebSocket connection to 'wss://ndt-iupui-mlab1-chs0c.measurement-lab.org:3010/ndt_protocol' failed: Error in connection establishment: net::ERR_NAME_NOT_RESOLVED

Per the issue comments in the mlab-ns repo, this appears to be resolved.

Some users still experience stalled tests. Firefox on Ubuntu does not appear work correctly. We think that this may be related to the way Firefox shares location temporarily or permanently. It may be necessary to have a different method of location for Firefox users. @rudietuesdays or @jheretic could you advise on the issues Firefox users are experiencing?

Confirmed more info from @nkinkade:

In firefox. I went to about:config and changed geo.provider.network.url  from "https://www.googleapis.com/geolocation/v1/geolocate?key=%GOOGLE_LOCATION_SERVICE_API_KEY%" to "https://location.services.mozilla.com/v1/geolocate?key=%MOZILLA_API_KEY%" and the test started working.
It seems that indeed the fundamental issue was the site not being able to obtain a geolocation and just hanging.

This behavior can be generally reproduced by selecting to "Use my browser location" and then denying the browser request (or more generally, having it blocked as is probably more common with Firefox users). The geolocation API call should have a timeout value supplied in order to avoid this in the future.

@jheretic: I don't believe the browser should have been blocking the request. It's a Firefox profile I have which is configured to pretty much accept anything, do no blocking that Firefox likes to do, but then to delete every single save setting on exit.
When I clicked on "Use my browser location", Firefox popped  up a dialog asking if I wanted to share my location, to which I selected yes.
It also seems odd to me that it would work with Mozilla's location service, but not Google's, which is the default in Firefox.
This may be user error on my part. I haven't dug into much deeper than what you see in my comment.

This has been reported by multiple partners on the NDIA project and confirmed by @nkinkade and myself. I think we need to consider this an interrupt priority to fix now. @laiyi-ohlsen do you agree? We can discuss if needed on today's check in call.

+1

Another report, lmk if contacting the user for more info is useful!
Was just trying to take your survey which I saw on LinkedIn... kept getting this “wheel of wait” for the speed test.

So I ran SpeedTest from my iPad Pro and got this...

Just went back to your survey page, in Safari, and the wheel of wait is still turning and won’t let me submit my survey.

Deployed to production.

Confirmed in FF and Chrome, this fix does allow users of both browsers to run the test, whether they consent to location or not. However, #162 and #163 are still happening for all submissions. The submissions API endpoint is down and a 500 error is presented in the browser console output.


Add crystal syntax highlighting

I added language definition of Crystal.  This Pull Request is my first contribute to highlight.js, so it may have some problems :)

Hi! Thanks for the contribution! It looks fine on the first glance apart from one bit: the Category: common in the .js header. This category is special as it holds only languages we decide to include in the CDN build. The category field is not required anyway so please remove it from the definition. Thanks!

Ok. I removed it.

Merged! This was an easy one, thanks for taking your time to follow the check list!


How to point to an agent withs spring-cloud-consul ? Is this way really highly available?

How to point to an agent withs spring-cloud-consul ? Is this way really highly available ?
Are there any examples ？
Beg for your replay ,thank you !

The default configuration is to point to an agent. Can you describe more of what you are looking for?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


feat:moved display url to different route path

#81

Issue:
Module Warning (from ./node_modules/eslint-loader/index.js):

/Users/nlothain/dev/verida/account-explorer/src/router/index.ts
  14:18  warning  Insert `,`  prettier/prettier

✖ 1 problem (0 errors, 1 warning)
  0 errors and 1 warning potentially fixable with the `--fix` option.

You may use special comments to disable some warnings.
Use // eslint-disable-next-line to ignore the next line.
Use /* eslint-disable */ to ignore all warnings in a file.

Issue:

Go to front page and search for DID
Click on DID that is found. Link goes to old (non /did) URL

@nick-verida,
I have made the requested changes.

@andy-verida @ram-verida can you please review this change


Rake error deploying to Heroku with 0.3.10

We've been using the USE_FREEDESKTOP_PLACEHOLDER=true approach and it's worked fine for our apps that are deployed elsewhere. But today I tried this approach for a Heroku-deployed app, and I get the following error. The bundle exec rake -P command works fine locally, but the dummy.xml file it's apparently looking for in the Heroku build/deploy environment is not there on my local system either.
My app details:
$ egrep "^ruby|^gem .rails| rails |mimemagic" Gemfile* .ruby-version | grep -v ':#'
Gemfile:ruby '2.7.3'
Gemfile:gem 'rails', '~> 5.2.4', '>= <IP_ADDRESS>'
Gemfile:gem 'mimemagic', '~> 0.3.10'
Gemfile.lock:      mimemagic (~> 0.3.2)
Gemfile.lock:    mimemagic (0.3.10)
Gemfile.lock:    rails (<IP_ADDRESS>)
Gemfile.lock:  mimemagic (~> 0.3.10)
Gemfile.lock:  rails (~> 5.2.4, >= <IP_ADDRESS>)
.ruby-version:ruby-2.7.3
$ heroku config:get USE_FREEDESKTOP_PLACEHOLDER
true
$ env | grep FREEDESK
USE_FREEDESKTOP_PLACEHOLDER=true
$ ls ~/.rvm/gems/ruby-2.7.3/gems/mimemagic-0.3.10/dummy.xml
ls: cannot access /home/bjn/.rvm/gems/ruby-2.7.3/gems/mimemagic-0.3.10/dummy.xml: No such file or directory
$ bundle exec rake -P | grep db:version
rake db:version
$ bundle -v
Bundler version 1.16.6

The Heroku build/deploy error:
-----> Compiling Ruby/Rails
-----> Using Ruby version: ruby-2.7.3
-----> Installing dependencies using bundler 1.17.3
       Running: BUNDLE_WITHOUT='development:test' BUNDLE_PATH=vendor/bundle BUNDLE_BIN=vendor/bundle/bin BUNDLE_DEPLOYMENT=1 BUNDLE_GLOBAL_PATH_APPENDS_RUBY_SCOPE=1 bundle install -j4
       Using rake 13.0.1
[...]
       Using mimemagic 0.3.10
[...]
       Bundle complete! 25 Gemfile dependencies, 62 gems now installed.
       Gems in the groups development and test were not installed.
       Bundled gems are installed into `./vendor/bundle`
       Removing bundler (1.17.3)
       Bundle completed (0.47s)
       Cleaning up the bundler cache.
-----> Installing node-v12.16.2-linux-x64
-----> Detecting rake tasks
 !
 !     Could not detect rake tasks
 !     ensure you can run `$ bundle exec rake -P` against your app
 !     and using the production group of your Gemfile.
 !     rake aborted!
 !     Errno::ENOENT: No such file or directory @ rb_sysopen - /tmp/build_f6907ac1/vendor/bundle/ruby/2.7.0/gems/mimemagic-0.3.10/dummy.xml

Looking at how USE_FREEDESKTOP_PLACEHOLDER is implemented, I don't think you can get it to work on Heroku (or any other platform where build step is separated from run step).
Maybe instead you could try creating an empty file manually (somewhere in your code) and point FREEDESKTOP_MIME_TYPES_PATH env var to it?


GGML_ASSERT: /build/go-llama/llama.cpp/ggml-cuda.cu:6670: src0->type == GGML_TYPE_F16 9:27AM DBG GRPC(ggml-model-q4_0.gguf-<IP_ADDRESS>:33649): stderr SIGABRT: abort

LocalAI version:

Environment, CPU architecture, OS, and Version:

Linux iZwz98q2j698koh4a9ht5lZ 3.10.0-1160.92.1.el7.x86_64 #1 SMP Tue Jun 20 11:48:01 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
I download and use quay.io/go-skynet/local-ai:master-cublas-cuda12
Describe the bug

I Use PostMan

I Get Error In LocalAI

To Reproduce

Expected behavior

Logs

Additional context

I entered the container and ran it alone
CGO_ LDFLAGS="- cublas - cubart - L/usr/local/cuda/lib64/" LIBRARY_ PATH=$PWD C_ INCLUDE_ PATH=$PWD go run/ Examples - m "/models/gml model q4_0. gguf" - t 14- ngl 44- n 4096
It works good.

@mudler  please help me。I almost succeeded

the new docker image fix it


"Decorator" applicability

Continuation of https://github.com/decentraland/unity-explorer/pull/572#discussion_r1567263219 and https://github.com/decentraland/unity-explorer/pull/572#discussion_r1567259333
Last two points:

If ConnectiveRoom is a core:

Can it be used as a core without interfaces?
Can it have the name with Core
Only referenced from ArchipelagoRoom, GateKeeperSceneRoom, FixedConnectiveRoom from where it should be used by design. Then it will be clear how the core is connected with additional functionality as the core can't be used in practice without the corresponding higher level IConnectiveRoom

For me it's confusing that the logic of ForkArchipelagoIslandRoom acts as a factory but it's a decorator instead.
Why it can't be a factory then? Then it will be clear that we are using factory instead of IArchipelagoIslandRoom.
There should be some common sense limitations for using Decorator pattern.
I truly believe that on the consumer side it's more predictable if the factory is used directly and not something abstract hidden behind several layers of nesting.
The same concern related to RenewableConnectiveRoom. Why it should implement IConnectiveRoom interface when it acts as a factory.
Why do we place it on the same layer as a decorator?

Why these functions are delegates with captures?
you told me you had cross-references issues
I don't remember that I pointed it's a cross references issues, do you mean about IRealmNavigator?
When you break the layering and try to flatten everything on the decorator (model layer) you circumvent it by introducing captures.
It can be easily made with IConnectiveRoomFactory, captures aren't deal here, it can be removed without struggles.
StartAsync function can be replaced with CreateAsync and it will become a factory. Factory can have switched or be a factory of factories (Abstract Factory) but it's still a factory that should operate on its own layer.
Who will be responsible of explicitly creating new rooms? IRoomHub? It's not responsibility of it on my opinion. I repeat: ForkArchipelagoIslandRoom is not a factory, it uses factories (factory methods currently).
_What disadvantages I see in delegates with captures?
You hide the real flow of dependencies, potentially hiding the problem with references
You cheat with abstraction layers
The code becomes less obvious, readable and maintainable as it's not clear what is hidden behind Action and Func_
As I stated before: Func can easily be replaced with direct IConnectiveRoomFactory if it's required. Didn't you see this line? It's not a factory, it uses factory methods to fork (like if else) behaviour depending on the response. It composes factories but it's not a factory. I didn't create separated objects for factories because it would bring unnecessary objects


#14790: Adding CrossfadeImage

Description
👋 This is my first PR, feedback and pointers greatly appreciated.
I've implemented a variant of FadeInImage that provides a crossfade animation instead of a fade out/fade in animation. These changes are directly based on the existing FadeInImage class (particularly the tests) and the animation behaviour for the crossfade is provided by AnimatedCrossFade.
I believe we could invest in this area to make the separation of responsibility cleaner. The behaviour of both FadeInImage and CrossfadeImage are very similar: load two images in a stack and fade to the second image using a given animation strategy when the second image has loaded. I'm imagining a way where we supply the animation strategy as a constructor argument to FadeInImage but I couldn't work out a way to separate this sensibly and maintain a good signature (each animation type could require different parameters), is there an approach I could try?
Happy to incorporate feedback, thank you for reviewing :)
Related Issues

Fixes #14790

Tests
I added the following tests:

Tests are heavily inspired by the existing tests for FadeInImage but specific animation checks are removed as the animation details are the responsibility of AnimatedCrossFade.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

This could also easily live in an external third-party package. I am not convinced we should have it in the framework.

Added it here as the FadeInImage is in the framework already but I agree that it could live externally.
An alternate option would be to extract the animation strategy out of FadeInImage and have it be provided somehow. I don't think there's a way to do that without introducing a breaking change. My thinking for this:

Each animation strategy will likely need different args and I'm unsure if we can provide a sensible, backwards-compatible default option.
We can't guarantee that the existing args will nicely map to all possible strategies (e.g. fadeInDuration and fadeOutDuration aren't relevant to the cross-fade so one would be redundant).

I'd like to take a pass at abstracting out the animation strategy as I'd like to investigate the first bullet point here (there's probably a solution but I'm worried it might not be nice).
In general, what're your thoughts on making FadeInImage parameterised by an animation strategy @goderbauer ? And thanks for taking a look at this, I appreciate it!

I've explored this today and have provided an alternative approach in https://github.com/flutter/flutter/pull/57385. This alternative approach extracts the animation builder out, abstracting the animation strategy instead of contributing a new class to the framework for every animation combination.

Like I said on the other PR: I don't think we need this in the framework for now as this can easily provided and build in a third-party package.


Support Implicit Member Expression

Hi @nmdias
Can Key class inherit a class like following, for example?
public class AnyKey {}

public final class Key<ValueType: Codable>: AnyKey {}

This makes it possible to do following.
extension AnyKey {
    static let key = Key<String>("someKey")
}

defaults.set("Codable FTW 😃", for: .key)

My goal is like #9.
Thanks.

Thank you for the suggestion @KyoheiG3, it worked out really well 😃


any plans to add a calendar component?

Duplicate of #782

#782 is for a date picker. I was thinking more along the lines of google calendar

Is there a component specified for calendars in material design? To my
knowledge there isn't.
On Sep 14, 2015 4:10 PM, "Sonny To"<EMAIL_ADDRESS>wrote:

#782 https://github.com/google/material-design-lite/issues/782 is for a
date picker. I was thinking more along the lines of google calendar
—
Reply to this email directly or view it on GitHub
https://github.com/google/material-design-lite/issues/1598#issuecomment-140192254
.


Vue official extension for Vue 3 does not do syntax highligting

Vue - Official extension or vue-tsc version
2.0.21
VSCode version
1.90.0
Vue version
3.4.0
TypeScript version
x
System Info
No response
Steps to reproduce
The extension has several bugs. It fails to highlight both braces and curly braces properly, and it sometimes gives misleading signals by incorrectly marking some components in red.
open vs code and you will see it sometimes fails to highlight some elements.
Link to minimal reproduction
No response
Any additional comments?
No response

Could you please provide some screenshots?

Could you please provide some screenshots?

Isn't the highlight correct?

Isn't the highlight correct?

No, curly braces and parenthesis are supposed to be yellow not white.

What theme are you using? BTW the second image failed to upload.

What theme are you using? BTW the second image failed to upload.

@

The issue is probably this code /s
const isTrue = false;
const isFalse = true;

On a serious note, could you share a public repo with a minimal reproduction?

codesnap | adp | 1.3.4
laravel-extra-intellisense | ami | 0.6.6
vscode-tailwindcss | bra | 0.10.5
laravel-goto-view | cod | 1.3.11
vscode-eslint | dba | 2.4.4
composer-php-vscode | DEV | 1.47.15512
phptools-vscode | DEV | 1.47.15512
profiler-php-vscode | DEV | 1.47.15512
es7-react-js-snippets | dsz | 4.4.3
prettier-vscode | esb | 10.4.0
auto-close-tag | for | 0.5.15
auto-rename-tag | for | 0.1.10
vscode-phpfmt | kok | 1.2.12
rainbow-csv | mec | 3.12.0
dotenv | mik | 1.0.1
inline-fold | moa | 0.2.6
laravel-goto-components | nao | 1.2.0
color-highlight | nau | 2.8.0
laravel-blade | one | 1.36.1
material-icon-theme | PKi | 5.3.0
vue-vscode-snippets | sdr | 3.1.1
vscode-blade-formatter | shu | 0.24.2
svg-preview | Sim | 2.8.3
cody-ai | sou | 1.20.3
errorlens | use | 3.18.0
volar | Vue | 2.0.21
vue | Wsc | 1.0.26
and here is the repo: https://github.com/vusal5555/vue-js-learning

That looks like an issue with an installed extension. I recommend creating a new vscode profile and installing your extensions one by one until you can reproduce the issue.

My guess is that Vue - Official and Vue 3 Support - All In One are having conflicts with each other but I may be wrong.

Well, yes, my guess was right. I disabled the Vue 3 Support - All In One extension and the issue was resolved.


Use an insert queue to avoid series flood causing heavy lock contention

This change adds a shard series insert queue used to batch inserts into the shard series map without sacrificing delays to insert the series.
This is important as during floods of new IDs we want to avoid acquiring lock to insert each individual series and insert as many as possible all together acquiring the lock once.
It was experimented also sleeping for a very short duration, i.e. 1ms, during the insert loop and it added enough latency even just with 1ms that it hurt it more than the previous implementation of just acquiring the lock for each series insertion.
The batching as it is without any sleep and just relying on a notification trigger and hot looping when being flooded improved the overall perf by a factor of roughly 4x during floods of new series. (4x less perf degradation during a flood).
cc @xichen2020 @cw9 @prateek @ben-lerner

Coverage increased (+0.2%) to 77.259% when pulling 7bb1706b2b534969597371b944870e6ae7919b8e on r/db-shard-insert-queue into b14bf9109493d146d0871f92efcf040a30fa31e4 on master.


filestore: Add remove_dir_all_where method

This will be useful for extensions, even if it's not used in core trussed

on top it simply wraps a store:: primitive
so: lgtm


[Snyk] Upgrade bootstrap from 5.2.3 to 5.3.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade bootstrap from 5.3.0 to 5.3.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-07-26.

Release notes

Package name: bootstrap

5.3.1 - 2023-07-26 Read more

5.3.0 - 2023-05-30Release v5.3.0 (#38657)
* Bump version to 5.3.0
* Dist

    from bootstrap GitHub release notes

Commit messages

Package name: bootstrap

2a1bf52 Release v5.3.1 (#38956)
8e5dada Examples: improve spinner buttons accessibility (#38958)
11b3806 Fix bundlewatch values (#38954)
57ddb42 Fix postcss plugin options (#38947)
cc5a8a9 fix
ea21de1 Docs: Update float responsive examples
92d779f Update devDependencies and regenerate package-lock.json
01dacaf dashboard: fix offcanvas md display (#38952)
4271d22 Docs: remove v4 reference in homepage
1c80bd9 Fix some typos in Customize > Sass doc (#38948)
d45cc7e Support `Home` and `End` keys in tabs (#38498)
8fcfce1 Build(deps-dev): Bump eslint-plugin-unicorn from 47.0.0 to 48.0.0 (#38928)
a814733 Build(deps-dev): Bump rollup from 3.26.2 to 3.26.3 (#38931)
fd0a4e6 Build(deps-dev): Bump @ babel/core from 7.22.8 to 7.22.9 (#38930)
86c5371 Build(deps-dev): Bump hugo-bin from 0.111.2 to 0.111.3 (#38929)
fb348b6 Build(deps-dev): Bump postcss from 8.4.25 to 8.4.26 (#38927)
83e5ec8 Build(deps-dev): Bump @ babel/cli from 7.22.6 to 7.22.9 (#38922)
ae255aa Build(deps-dev): Bump @ babel/preset-env from 7.22.7 to 7.22.9 (#38920)
cf074cd Build(deps-dev): Bump terser from 5.19.0 to 5.19.1 (#38921)
303e567 Build(deps-dev): Bump @ rollup/plugin-commonjs from 25.0.2 to 25.0.3 (#38923)
9205ffe Build(deps-dev): Bump eslint from 8.44.0 to 8.45.0 (#38919)
101ff73 Docs: fix carousel carousel colors of carousel examples in dark mode
b3da789 Fix dark mode rendering of Cheatsheet examples (#38905)
3ea4168 Streamline a certain amount and increase presentations in different directions (#38604)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Er konflikter så lukker denne og så tar vi den når dependabot kommer med den.


Migrate to webpack 4 🎉

webpack v4 was released at the beginning of the year, this is a quick update of the webpack related packages and the config file. Unlike webpack v3, which was not a major upgrade over webpack v2, webpack v4 has a whole bunch of changes and features massive speed improvements.
See: https://medium.com/webpack/webpack-4-released-today-6cdb994702d4
Config Changes

Added "mode" configuration option
Removed "process.env.NODE_ENV" from DefinePlugin as it's now defined with "mode" configuration
Moved "noEmitOnErrors" from plugins to optimizations.
Removed "json-loader"

Removed "json-loader"
webpack >= v2.0.0 has 'native'/direct JSON support. Since webpack >= v4.0.0 it's
a module.type (no JS wrapper needed anymore). It's suggested to remove json-loader.
See: https://github.com/webpack-contrib/json-loader/issues/65#issuecomment-368409135
Updated Packages

friendly-errors-webpack-plugin: 1.6.x -> 1.7.x
webpack: 3.x -> 4.x
webpack-node-externals: 1.6.x -> 1.7.x

I've tested all of the examples and everything seems to work. I ran a quick test using awesome-typescript-loader v4 and it worked, so this should also fix this issue https://github.com/jaredpalmer/backpack/issues/122
Let me know if anything doesn't work.
Thanks.

@jaredpalmer Any reason why this hasn’t been accepted?

@cyruseftos when it release?

I hope it release the sooner the better.
Some dependencies are out of date, feel uncomfortable to see tip appears when npm install.

 I'm currently on version "^0.4.3" of backpack-core and I just see webpack version is still "^3.0.0", so it's still not running on version 4


Review and discuss display portion of MinVR

In an upcoming Skype call would like to walk through the current approach to handling displays, and this could also include a walk through of how the plugins are setup.  Similar to the walkthrough of events that we did at the end today.

We did this on a past call. Closing issue.


How to open modal not user button?

I want to use JQuery to open it but don't need any other buttons to open it.
Ex:

Cras justo odio, dapibus ac facilisis in, egestas eget quam. Aenean eu leo quam. Pellentesque ornare sem lacinia quam venenatis vestibulum. Praesent commodo cursus magna, vel scelerisque nisl consectetur et.

But not
<button class="inline">Show</button>

Cras justo odio, dapibus ac facilisis in, egestas eget quam. Aenean eu leo quam. Pellentesque ornare sem lacinia quam venenatis vestibulum.  Did you figure it out?

I'm looking for similar functionality. Did you figure it out?

I tried to fix the source but it failed! I had to trigger the submit button to open the modal, which was silly

Have you tried
$('.inline').modaal({
	content_source: '#inline'
	start_open: true
});

as per the docs?
See: https://github.com/humaan/Modaal#32-programatically-open-a-modaal

@pzi Yes, but I want to open the modal based on actions not when the website first loads. Something like they complete a form and then show up the modal if the form submits, as an example.

CodePen example of triggering the form on form submit:
https://codepen.io/pzi/pen/MWKXgxg
HTML
<form>

    Foo
    <input />

  <button>Submit</button>
</form>

...

CSS
#inline-content {
  display: none;
}

JS
// init modaal
$(".trigger").modaal({ content_source: "#inline-content" });

$("form").on("submit", (event) => {
  event.preventDefault();
  // open modaal on submit
  $(".trigger").modaal("open");
});

Disclaimer: haven't used Modaal that much, so unsure if the "trigger" being on the same as the modaal content is legit, but it seems to work. Can always chuck the trigger class on another element if it's an issue.


syntax highlighting + intellisense not working for project created using dotnet script init

I tried creating a scripting project using dotnet script init and then opened it on VS Code. When I open main.csx, syntax highlighting appears for a brief moment and then disappears as soon as the following line appears in the Output window:
[Info  - 4:50:53 PM] [Program]Language server initialized

Initially I thought this might be an Omnisharp issue, but the syntax highlighting + intellisense seem to work just fine on a conventional console application.
Should I expect syntax highlighting + intellisense to work with dotnet-script? If so what might be going on here?

Hi. This is probably because you have the C# Dev Kit installed and hence OmniSharp is not used here.
Step 1. Make sure that the C# Dev Kit extension is uninstalled
Step 2. Make sure that "Use Omnisharp" is ticket in the settings.

Microsoft basically killed OmniSharp (and dotnet-script language services) when they decided to write their  backend.
It did not land well in the OSS community. Most of the discussion can be found here
https://github.com/dotnet/vscode-csharp/issues/5276

I have not fully understood this myself. Although Microsoft is developing their own closed source LSP-based host to provide language services, there is still people from Microsoft working with OmniSharp-Roslyn which we now need to opt in to use.
There is also an effort to bring LSP to Omnisharp through this project https://github.com/OmniSharp/csharp-language-server-protocol. This is also mostly driven by Joey Robichaud which is employed at Microsoft.
So to me it seems that maybe they are keeping Omnisharp alive just in case this new thing goes south. I don't know 😃

It's supposedly on their table, let's see if they can get the integration in. I was interested as well and found the issue on their open-source repo for that: https://github.com/dotnet/vscode-csharp/issues/5722


[DC-584] Drop participants lacking both "The Basics" and any EHR data

Putting this up for initial feedback, will add tests once I've gotten directional confirmation.

"too many updates" achilles_test failure, rerunning. @mark-velez let me know if this looks reasonable as-is. The integration test I've added would need to be reworked when transitioning to a common integration test package, but it can be run manually locally and it passes.


Rename "phoenix" references to "kitura"

Several places in source and samples makes references to "pheonix".  These should be renamed to "kitura".

I have a pending PR (https://github.com/IBM-Swift/Kitura-CouchDB/pull/5) which changes phoenix to kitura_test_db

Fixed in PR #5


Trigger job manually

when trigger job manually like
Scheduler.TriggerJob(JobKey.Create(jobkey), job.JobDataMap);
throws exception
Quartz.SchedulerException: Job threw an unhandled exception.
---> System.NullReferenceException: Object reference not set to an instance of an object.
at Quartz.Core.JobRunShell.Run(CancellationToken cancellationToken)
--- End of inner exception stack trace --- [See nested exception: System.NullReferenceException: Object reference not set to an instance of an object.
at Quartz.Core.JobRunShell.Run(CancellationToken cancellationToken)]
there is no problem in the job's execute method. it's just console log.

@sezerkorkmaz
Hi You should use Microsoft DI. The one possible working solution :
 services.AddSilkierQuartz(it =>
            {
                var section = Configuration.GetSection("Quart");
                foreach (var config in section.GetChildren())
                {
                    it.Add(config.Key, config.Value);
                }
            });
            services.AddQuartz(it => it.UseMicrosoftDependencyInjectionJobFactory());


psql Library not loaded: /usr/local/lib/libpq.5.dylib

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed casks are deprecated or disabled.
You should find replacements for the following casks:
  ksdiff

Warning: A newer Command Line Tools release is available.
Update them from Software Update in System Settings.

If that doesn't show you any updates, run:
  sudo rm -rf /Library/Developer/CommandLineTools
  sudo xcode-select --install

Alternatively, manually download them from:
  https://developer.apple.com/download/all/.
You should download the Command Line Tools for Xcode 15.1.

Warning: Some installed formulae are deprecated or disabled.
You should find replacements for the following formulae:
<EMAIL_ADDRESS>
Warning: Unbrewed header files were found in /usr/local/include.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

➜  ~

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.2.20
ORIGIN: https://github.com/Homebrew/brew
HEAD: c2ed3327c605c3e738359c9807b8f4cd6fec09eb
Last commit: 2 days ago
Core tap JSON: 30 Apr 17:13 UTC
Core cask tap JSON: 30 Apr 17:13 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 3.1.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 14.0.3 build 1403
Git: 2.44.0 => /opt/homebrew/bin/git
Curl: 8.1.2 => /usr/bin/curl
macOS: 13.5.1-arm64
CLT: <IP_ADDRESS>.1.1683849156
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
I install postgres 16 and run it. When I try the command psql, I got this error

What happened (include all command output)?
dyld[28821]: Library not loaded: /usr/local/lib/libpq.5.dylib   Referenced from: <4A4C9EBE-1868-317F-A86C-2DAE62CA9064> /usr/local/bin/psql   Reason: tried: '/usr/local/lib/libpq.5.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/usr/local/lib/libpq.5.dylib' (no such file), '/usr/local/lib/libpq.5.dylib' (no such file), '/usr/lib/libpq.5.dylib' (no such file, not in dyld cache) [1]    28821 abort      psql
What did you expect to happen?
postgres database should run
Step-by-step reproduction instructions (by running brew commands)
brew install postgresql@16
brew services start postgresql@16
brew services list
psql

What's the output of psql --version? I'm guessing it's not 16.x, since postgresql@16 is keg-only.
Also check the output of ls -l /usr/local/bin/psql.

That's not Homebrew's psql, and I should've noticed earlier that you're running on an ARM64 Mac, for which Homebrew installs exclusively under /opt/homebrew.
You're looking for /opt/homebrew/opt/postgresql@16/bin/psql instead.

Yeah, that's the psql you want to run.

How can I run that one ?

Just like that: /opt/homebrew/opt/postgresql@16/bin/psql.

Now I got database not found, how can I create/edit database if I can't run postgres ?

You're asking us how to use PostgreSQL at this point which is not an issue with Homebrew.

I just to do this rm -rf /usr/local/bin/psql
And remove all postgres that were installed by homebrew
Then install postgres app from https://postgresapp.com/downloads.html
Now my psql commpand works


[ABW-3654] - Update screens navigation transitions

Jira Ticket
Description
This PR updates navigation transitions between screens.

only minor thing I noticed is that when you go from persona edit back to persona details, persona details screen seem to blink/fade in instead of stay in place when persona edit slide down. But looking at the code it seem your setup is correct

Good catch. Fixed it.


Test function that updates performance reports for spray operators

Test that performance_report actually updates the performance reports for spray operators if it receives data for the same spray_operator in the same date

Fixed by #480


Remove unnecessary calls to OpenPrivatePersonalDetailsPage

Details

We now return these details in OpenApp, so I'm removing extra calls to fetch this data.
Fixed Issues

$ https://github.com/Expensify/App/issues/23873
$ https://github.com/Expensify/App/issues/36647
Tests

[ ] Verify that no errors appear in the JS console
[ ] Same as QA

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console
[ ] In the profile page, under private personal details, set Legal Name, DOB, Address for a test account.
[ ] Log out and Log back in
[ ] Open the following pages:
[ ] Profile page
[ ] Card Details Page (if account has an expensify card)
[ ] Verify details are displayed ✅

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

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

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Deep linking into forms that relay on private personal details does not work correctly since default values are taken into consideration only on the first render
https://github.com/Expensify/App/assets/16493223/7a66f12b-db22-4a30-95ed-a85d9d5ea39a

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/16493223/e216b4b1-56c8-4886-bf54-ad7847063529

MacOS: Desktop

@s77rt I fixed the deep linking bug

Tests well. @grgia Can you please complete the checklist?

For the failing test, can you try merge main

Failing ts test is effecting all current open PRs (can be ignored)

Thanks @s77rt  for your patience while I was OOO! I'll work on updating this PR

@s77rt apologies for the delay on updating this, I've created an ADHOC build for testing

@grgia Can you check the 2 above comments, they don't seem resolved

Addressed comments @s77rt

No need to block on me!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/grgia in version: 1.4.62-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/grgia in version: 1.4.62-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/grgia in version: 1.4.62-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 1.4.62-17 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Playground: Suggestion: Strip top-level export keywords when emitting JS

Stripping top-level export keywords from emitted JS in the Playground would make it easier to paste and work with code from a real file (which likely has one or more exports).

Repro:
Type any code into the Playground that exports anything from the top level (not within a namespace) and click Run

Expected behavior:
The code should run ignoring the exports
Actual behavior:
The code chokes on the exports and will not run until they are removed

Just going to add here: In discussion it's also been noted that this can be solved by wrapping all the playground code in a namespace

Hey! I've created the above mentioned PR which solves this issue. Will be awaiting review. Thanks


Cannot find module 'xml-escape'

Hello there!
I get this on hexo s command
ERROR Plugin load failed: hexo-generator-podcast
Error: Cannot find module 'xml-escape'

Will you help me, please?

Please install that dependency manually for the time being. I'll update package.json in time.

Dependencies in package.json have been updated.


Can you support AndroidX?

Android support package name already changed

yes, I started a new version of this lib that will use the AndroidX and is all Kotlin. I'll have it pushed up in a few days.

I would be happy to contribute to the AndroidX and Kotlin version, @kayvannj if you need the help.

Hey @levibostian, I haven't gotten around to it but yes it would be greatly appreciated if you can contribute. Feel free to push up a PR and I'll review it.

I would be happy to contribute to the AndroidX and Kotlin version, @kayvannj if you need the help.

Is there a specific branch you have begun work on that I can continue with? Or would I need to start from scratch?

My first stab at it is in this branch https://github.com/kayvannj/PermissionUtil/commits/kotlin-wip
But I don't recall if it's finished up or not. the switch to kotlin and android-x can be a major version so don't worry about breaking the API.


Let NCCL ID store fate share with CollectiveGroup.

Right now the id actor is detached and deleted by "supposedly" the last user of the actor.
However when execution order is not determined, this will crash the entire workload if the killed named actor is accessed again.
This change make the actor fate-share with worker rank 0. so Rendezvous would not need to kill it.
Closes #907

Not quite familiar with ray. Could you please explain when the new store is deleted?

Not quite familiar with ray. Could you please explain when the new store is deleted?

now that this store lives on the CollectiveGroup, it will get deleted when CollectiveGroup gets deleted with ref counting.
is that acceptable?

thanks. gonna merge this for now to stabilize our CI.
let me know if you guys have any other comments.


Backstack - order issues

As designed, no matter where the users are, when they click "back", they should go back to Message screen immediately.
However, in the app (the 25th commit), if the users click contact and then profile, clicking "back" in the Profile screen will just direct them to contact (it should be the Message screen).

This issue is not longer valid since I used a different strategy to do it. This issue will be potentially merged to issue #5 .


Overall roadmap for Noports

Outline the internal roadmap for Noports

Moving this to P0, the scope of this roadmap has grown tremendously in the past week... It's highly important that everyone is made aware of all the moving parts.


Modify the testcase_name in Html report content as default constant string “Testcases Name”

Since  testcase_name in Html report content  maybe the output path string.

I need this fix :/
Noones maintaining this codebase?


Identity changes for AuthN work

Mostly just switching from ClaimsIdentity -> ClaimsPrincipal, AuthenticationScheme -> AuthenticationType everywhere
cc @divega @lodejard @Tratcher @blowdart

Needed to make some changes to the Controllers to adapt to the new world (cc @rustd)
Logged in check change:
(old)User.Identity.IsAuthenticated => (new)User.IsLoggedIn()
IsLoggedIn is a new Identity extension method on IPrincipal from Identity which looks for a ClaimsIdentity with the ApplicationCookie's authentication type]

Renamed IsSignedIn => IsLoggedIn for consistency

Removed old IClaimsIdentity extension methods and moved them to IPrincipal ...
User.Identity.GetUserId => User.GetUserId
User.Identity.GetUserName => User.GetUserName

Looks good to me. :shipit:

Merged 3aaa6283653a9d44b746767ec04fb7159553cb9e


ATTN can not import create prameter how can I fix this

You probably had another Sollumz version previously installed, make sure to uninstall any versions you have and install the newest one. Afterward, restart blender just in case.
If you still have issues, ask in the Discord server. The issue tracker is for actual bug reports.


Web -  After refreshing the page, the task title and description fields do not automatically focus

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

Action Performed:

Open a chat.
Navigate to the "Assign task" section.
Enter a title and create a task.
After creating the task, click on the task title or description field.
Refresh the page.
Notice that the task title and description fields do not automatically focus. (Note: This issue is not present in the production)

Expected Result:
After refreshing the page, the task title and description fields should automatically focus
Actual Result:
After refreshing the page, the task title and description fields do not automatically focus as expected
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/bc744b25-dfdb-4e6a-863d-980fb2760597
https://github.com/Expensify/App/assets/93399543/a9f37978-6822-407d-a47d-1fed3588d5e8
Expensify/Expensify Issue URL:
Issue reported by: @ayazhussain79
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693003307227389
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

I don't think this is a blocker and TBH I think we should do nothing here

it is working fine on Production but not in staging

Resolved on staging with a web-E CP. Closing issue. If it persists feel free to reopen.


Minutes 2021-07-21

Re question 1. "how does an agent know all the resources it has access to?" above, which was not answered during the call.
In my view it is not possible for an agent to know all the resources it has access to, as those will span multiple pods. Perhaps that should be brought up in an issue or discussion, since we did not cover it in the session @justinwb ?
We should go over the minutes and find other unanswered questions that point to issues that need discussiong. (Perhaps some have already been discussed in existing issues and we can link to them).

@bblfish it's the plan (re: creating issues, linking to existing ones and discussing those points).
I extracted three points, there might be more.
See: https://github.com/solid/authorization-panel/blob/minutes-2021-07-21/meetings/2021-07-21.md#actions

@bblfish as per our discussion during the panel this week, this issue was created to acknowledge that conversations about new access modes could take place (we're not meant to discuss anything there, but I think the topic deserves a few more thumbs up).
I would propose creating an issue for discussing/exploring the potential of a ControlRead access mode.
Are you ok with me starting that or would you suggest something else?


Fix for settings test and iframes

Added keyword "select_frame_with_value"
Removed some commented and blank lines
Removed some unused imports
Added namespace to one of the objects
Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata
Testing Notes

@jessingrass -- I see that this is a fix for some of the robot stuff, and we have the robot tests failing here. Do we have a handle on our automated testing for EDA? I've got this rather large label change PR that Docs is hoping to get in, but when I run the robots (completely different branch) things are not pretty at all. I was hoping that some of these changes y'all are making here would help clean that up. Not sure if I should merge this branch into the label branch to see or if y'all would rather just have me wait until we have a plan for the EDA robot stuff. I'd love some guidance here.

Leave it for now? @Safiyab has returned from leave, so she will be working on stabilizing these tests in the near-term. The failing tests here are related to the work in this specific PR that still needs to be corrected.

@boakley Ready for re-review :)

@boakley @Safiyab @jessingrass -- Thank you all for working on this. As soon as our merge freeze is lifted, we will get this merged.

@jessingrass -- looks like this is all approved, is it okay to merge I assume? cc: @boakley @Safiyab

@jessingrass -- looks like this is all approved, is it okay to merge I assume? cc: @boakley @Safiyab

Yes please. I saw on the Product Release calendar that Merge Freeze lifts at 1 PT, so I was waiting for that.

@jessingrass -- looks like this is all approved, is it okay to merge I assume? cc: @boakley @Safiyab

Yes please. I saw on the Product Release calendar that Merge Freeze lifts at 1 PT, so I was waiting for that.

awesome! RelEng just lifted our freeze, I prefer having QE merge just to stick with our process but let me know if you'd rather me merge it.


Proposed fix for Issue 73

Fix for KeyError on message acknowledgment-- avoid duplicate keys on message delivery tags.

Thanks!  I've merged it, but I also changed it to only use UUIds for the SQS transport rather than every virtual transport


[Feature]helm repository and image registry support authenticate registry with credential specified

Is your feature request related to a problem? Please describe.
After configuring the link, username and password, we don't know if the link works or if the username and password are correct, so it is necessary to add an authentication interface to verify the connectivity of the link and the correctness of the user name and password.

Describe the solution you'd like
add an authentication interface to verify the connectivity of the link and the correctness of the user name and password.

Describe alternatives you've considered
Refer to go-containerregistry to implement image registry verification

Additional context

Can you describe your issue more clearly?

Can you describe your issue more clearly?

@wangyikewxgm done

@wonderflow

Can you describe your issue more clearly?

Has been redescribed, please review

@barnettZQG

The config template for the image registry looks like:
import (
	"encoding/base64"
	"encoding/json"
	"strconv"
)

metadata: {
	...
}

template: {
	output: {
		...
	}

	parameter: {
		// +usage=Image registry FQDN, such as: index.docker.io
		registry: *"index.docker.io" | string
		// +usage=Authenticate the image registry
		auth?: {
			// +usage=Private Image registry username
			username: string
			// +usage=Private Image registry password
			password: string
			// +usage=Private Image registry email
			email?: string
		}
		// +usage=For the registry server that uses the self-signed certificate
		insecure?: bool
		// +usage=For the registry server that uses the HTTP protocol
		useHTTP?: bool
	}
}

This definition only checks the parameter type. But many configs need validation. So, maybe we could define the validation rule in the template:
import (
  "vela/validation"
)
...
template: {
	output: {...}
        validation: validation.#ImageRegistry & {
           $params: parameter
        }
	parameter: {...}
}

When creating a config, we should check the condition validation.$response.result === true, If not, return the  validation.$response.message.
This proposal depends on CueX. /cc @FogDong @Somefive


Patch cached Redis manifest regression

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Coverage decreased (-0.8%) to 82.644% when pulling a91b1df1296b20c9ecb9a0175e552c5aa22f0d17 on 126-redis-manifest-regression into 218fa0f979f588364d9ad3dd197603dad4cf6de6 on main.


Document OnDemand 2.0 upgrade and installation

Set base branch to branch used for #491

Fixes #459

What about #490 and enabling powertools?

I added a commit just now to ensure that CentOS / RHEL repos are enabled. The repo name powertools is specific to CentOS so have to document what to enable for RHEL too.