[v2.9] Port fix 478

Porting https://github.com/rancher/rancher/pull/45767

Need to bump RKE1 but we're currently blocked with RKE1 tag builds failing, see https://github.com/rancher/rke/actions/runs/9511782925/job/26218563519 / https://github.com/rancher/rke/actions/runs/9511782925/job/26218563359

@snasovich What version of RKE1 do we need to bump to?

Closing i favor of https://github.com/rancher/rancher/pull/45940.

@snasovich What version of RKE1 do we need to bump to?

@jakefhyde , https://github.com/rancher/rke/releases/tag/v1.6.0-rc8 cut a couple days ago or anything newer should do fine as it includes https://github.com/rancher/rke/pull/3610
We should now be unblocked.
Edit: I see you closed this PR just as I was typing my response and fact-checking.


Fix Options Pattern, issue #74.

Use AddOptions for ProxyOptions instead of AddTransient.

Thanks for the PR! However it contains two separate changed that I want to consider separately. Could you send them as independent PRs please?

Sorry, I have created another pull request #80 .


Don't use ERROR log when it is not an error

We log some information as ERROR even if it's not actually an error and we have a way how to overcome this.
Let's take a look at the logs and check if all logs marked as errors are really errors.
e.g. Can't describe this repository, are there any git tags?

yup, can't describe is pretty annoying

(And it's probably sent to Sentry if I am not mistaken...;)


feat: add option to persist rule index

What type of PR is this?
Feature
What package or component does this PR mostly affect?
cmd/gazelle
What does this PR do? Why is it needed?
Add the ability to persist the RuleIndex across runs to allow updating only the specified directories when indexing is required.
Which issues(s) does this PR fix?
Ref: #1181

Just to re-iterate a few points and limitations of this:
This is EXPERIMENTAL with no api guarantees. 99% chance there will be breaking changes. You should never commit this file to source-control.
When this EXPERIMENTAL feature is used:

the resolver.Imports/Embeds() calls for the entire workspace (the "rule index") is written to disk
when specifying dirs (default: .) on the command line only those directories are updated in the "rule index"
only the dirs specified on the command line will be traversed, therefore:

only those dirs will have Imports/Embeds() invoked
only those dirs will have Fix(), GenerateRules() invoked
only those dirs will have BUILDs updated, warnings generated etc.
only those dirs will have things like resolve warnings outputted

If resolver.Imports/Embeds() have side-effects then this won't work very well when they are cached and not invoked. Don't do that.
There is no cache invalidation here. It is entirely up to the user to pass the correct list of dirs that require reprocessing.
FUTURE: if those updated dirs changed what rules they generate and the Imports/Embeds() of those rules change it is possible that other rules outside dirs are now referencing something incorrect. I think if Imports/Embeds() of those dirs change we might want to re-run Fix + GenerateRules for all directories.

@fmeum did you have any opinions on this going forward? Is this something you think should live in gazelle?
For large repos even simply walking the fs can be slow (even after the changes making it concurrent) so any type of incremental BUILD-generation needs to run on a subset of the workspace which is the goal of this PR.

I could very well see this live in Gazelle. Do you think that any of the two recent proposals https://github.com/bazel-contrib/bazel-gazelle/issues/1891 and https://github.com/bazel-contrib/bazel-gazelle/issues/1907 could substantially improve the situation for your large repos? If so, I would prefer to explore this route first, mostly since serialization, even if experimental, often ends up constraining future efforts to change the persisted data structure. Not doing unnecessary work in the first place could save us the complexity of having to persist this work.


Update tag-support.md

MachineLearningServices compute doesn't support tags.

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

@is-smit - it looks like that resource type is already in the table a few rows down from where you added it.

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

@is-smit - it looks like that resource type is already in the table a few rows down from where you added it.

Thanks for catching this, made incorrect change. I've updated the commit

@is-smit - thanks for this update.
#sign-off


Add new service 'snapshot' for camera

Description:
Add a new service call to crap snapshot from a camera. Cleanup service names.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#3834
If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Future PR: add a default template that writes to <config dir>/camera_snapshots/<timestamp>_<entity_id>.jpg

Ability to add time stamp and tag to a file name would be very useful when using these images for training a machine learning classifier. This could be achieved using an automation, but would be a nice option

Looks like entity_id object is not available in automations/scripts, only in dev-service.
Is it possible to use camera name in a template for filename in automations?
automation:
  - alias: All cameras today
    trigger: {platform: time, at: '16:00'}
    action:
      service: camera.snapshot
      data_template:
        # entity_id: "{% for item in states.group.cams.attributes.entity_id %}{{ item }}, {% endfor %}"
        entity_id: camera.one, camera.two
        filename: '/Data/cams/{{ now().strftime("%Y-%m-%d %H.%M.%S") }} {{ entity_id.name }}.jpg'

No support in old PRs.


Sass @while challenge

Describe your problem and how to reproduce it:
When you type in @while, and begin to create the conditions, it starts an infinite loop.
I was able to pass it by creating the code in an html comment, and then deleting the comment.
Add a Link to the page with the problem:
https://learn.freecodecamp.org/front-end-libraries/sass/apply-a-style-until-a-condition-is-met-with-while
Tell us about your browser and operating system:

Browser Name:
Chrome
Browser Version:
Version 69.0.3497.100
Operating System:
Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

I am also experiencing this issue. I bypassed it by typing my code into my editor and then copy/pasting to the browser.
Tell us about your browser and operating system:
Browser Name:
Chrome
Browser Version:
69.0.3497.100 (Official Build) (64-bit)
Operating System:
macOS High Sierra, version 10.13.6

The problem arises if the user types the closing } before correctly creating the contents of the loop. Since this is a common syntax-error-avoidance habit formed by coders, we see campers run into this problem on a regular basis. It would be helpful if autorun could be disabled on these challenges.

Same issue here


python3Packages.censys: 2.1.2 -> 2.1.3

Description of changes
https://github.com/censys/censys-python/releases/tag/v2.1.3

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 163379 run on x86_64-linux 1

3 packages built:

python310Packages.censys
python39Packages.censys
theharvester


Create a LESS version of our color palette

I have this already in a few of our projects. It’s definitely in the womenshealth repo.

Closed in 9ee63a142f053372984caf71723375a80a3cf57d


Realm accessed from incorrect thread

Hi guy,
I have problem with realm when I try to write many object.
dispatch_async(QUEUE, ^{
        dispatch_semaphore_wait(self.semaphore, DISPATCH_TIME_FOREVER);
        [self fetchObject:^(Object* object) {
            dispatch_semaphore_signal(weakSelf.semaphore);
            if ([parentObject isEqual:weakSelf.level2Categoties.lastObject]) {
                [weakSelf saveCacheToDataBase];
            }
        }];
    });

- (void)saveCacheToDataBase {
    __weak __typeof(self) weakSelf = self;
    dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_BACKGROUND, 0), ^{
        RLMRealm* realm = [RLMRealm defaultRealm];

        [realm beginWriteTransaction];
        [realm addOrUpdateObjectsFromArray:weakSelf.rootCategoties];
        [realm addOrUpdateObjectsFromArray:weakSelf.level2Categoties];
        [realm addOrUpdateObjectsFromArray:weakSelf.level3Categoties];
        [realm commitWriteTransaction];

        DLogError(@"SUCCESS!");

        weakSelf.isLoading = NO;
        if (weakSelf.callBack) {
            dispatch_async(dispatch_get_main_queue(), ^{
                weakSelf.callBack(weakSelf.rootCategoties);
            });
        }
    });
}

I get the exception "Realm accessed from incorrect thread". But I didn't know why this happens.
Do you guys have any suggestion for me in this case. Thanks!
Realm version:  0.102.0
Xcode version: 7.3.1
iOS/OSX version: iOS 8.4/ OSX 10.1.4

The documentation's section on Threading discusses the fact that persisted Realm objects are confined to the thread they were created / fetched on. The backtrace of where the exception is raised will pinpoint the object that is being incorrectly shared across threads. From the code you've posted it seems likely to be related to weakSelf.rootCategoties as you're accessing it from both a global queue and the main queue, and it appears to contain persisted Realm objects.

I have tried as bellow:
    NSArray* categoriesLv1 = [self.rootCategoties copy];
    NSArray* categoriesLv2 = [self.level2Categoties copy];
    NSArray* categoriesLv3 = [self.level3Categoties copy];

    dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_BACKGROUND, 0), ^{
        RLMRealm* realm = [RLMRealm defaultRealm];

        [realm beginWriteTransaction];

        [realm addOrUpdateObjectsFromArray:categoriesLv1];
        [realm addOrUpdateObjectsFromArray:categoriesLv2];
        [realm addOrUpdateObjectsFromArray:categoriesLv3];
        [realm commitWriteTransaction];

        DLogError(@"SUCCESS!");

//        weakSelf.isLoading = NO;
//        if (weakSelf.callBack) {
//            dispatch_async(dispatch_get_main_queue(), ^{
//                weakSelf.callBack(weakSelf.rootCategoties);
//            });
//        }
    });

It is still not working.

Note that the problem is sharing the objects contained within rootCategoties and such across threads, not with sharing the array itself. The call to -copy just creates a new container. The objects contained within are still being shared across threads.

I got it. Thanks for your help!


LyricsX v1.1.4 from Github repo

[X] brew cask audit --download {{cask_file}} is error-free.
[X] brew cask style --fix {{cask_file}} reports no offenses.
[X] The commit message includes the cask’s name and version.
[X] Named the cask according to the token reference.
[X] brew cask install {{cask_file}} worked successfully.
[X] brew cask uninstall {{cask_file}} worked successfully.
[X] Checked there are no open pull requests for the same cask.
[X] Checked the cask was not already refused in closed issues.
[X] Checked the cask is submitted to the correct repo.

Please don’t collapse the template. It’s useful that the sections be kept as we made done, for easier parsing.

Ok, sorry for messing that up (first submissions).
Regards

Il giorno 07 ago 2017, alle ore 20:00, Vítor Galvão<EMAIL_ADDRESS>ha scritto:
Please don’t collapse the template. It’s useful that the sections be kept as we made done, for easier parsing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/caskroom/homebrew-cask/pull/37467#issuecomment-320735795, or mute the thread https://github.com/notifications/unsubscribe-auth/AFOjWMBdXP0QRenIWtzPiCPfI5NccuGrks5sV1DMgaJpZM4OvvTH.

Thank you for your contribution!


For Issue #9: Add toasts / pop-ups

Added error and success toasts in the following routes
- add-event-space
- explore-event-space
  - [spaceId]
- add-event
- manage-event
- register-event
  - [registerId]
- update-event

A simple preview:

Made sure that the toasts are nice and visible, can stack and easy to modify for later. Also, they aren't that intrusive so the user won't be overwhelmed with them nor are they appearing for every single action only the main ones. They also match your existing UI
(PS: Due to the auto-format some tailwind classes got rearranged)

This is on mobile

Bro I told you to add this at the top let's see preview

Bro I told you to add this at the top let's see preview

I commented about it on the issue and you said "Okay" so I went with it, but I'll make changes if you want

No worries

I will merge in soon something is wrong in auth

No worries

If you feel like changing it later to something different if you change up your UI it's easily doable too
// Available positions:
// top-left, top-center, top-right, bottom-left, bottom-center, bottom-right
<Toaster position="top-center" />

I will merge in soon something is wrong in auth

Yep ok no worries

@Eshan05 Please make PR to dev branch

@Eshan05 Please make PR to dev branch

Ok changed the base branch

add level labels @TejasNasre
Regards,
Community Manager Gssoc-ext


Mobiilivihko Redux refactoring

(October 1, 2024) Progressing on the sending the documents. Next is to edit the document

Now editing works, tracking features work. Started using "development builds" instead of "expo go" for developing the app. There's still some unreliable thunk actions, trying to get this sorted asap.

Test version 1.5.10 has been released for Android, today I'll release also the iOS test version.


[Feature] Custom metric for masked accuracy

Description
Metric used to calculate accuracy for the masked language model task.
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Job PR-503/5 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-503/5/index.html


Adjust model database type mapping

adjust definitions according to feedback from real world usage (ci.codeberg.org)
close  #1892
close  #1865

Codecov Report
Patch coverage has no change and project coverage change: -0.05 :warning:

Comparison is base (cef135e) 39.31% compared to head (6ae398e) 39.27%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1949      +/-   ##
==========================================
- Coverage   39.31%   39.27%   -0.05%
==========================================
  Files         177      177
  Lines       10725    10725
==========================================
- Hits         4217     4212       -5
- Misses       6214     6219       +5
  Partials      294      294

Impacted Files
Coverage Δ

server/model/pipeline.go
0.00% <ø> (ø)

server/model/step.go
0.00% <ø> (ø)

server/model/workflow.go
0.00% <ø> (ø)

...tastore/migration/020_parent_steps_to_workflows.go
59.45% <ø> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Use equal area Mollweide projection for computing fraction of overlap between inventory and model grid cells

Compute the fraction of the intersected area in the equal-area Mollweide projection.
Potentially, we want to avoid projecting the inventory cell onto the rotated pole grid first, then again onto the Mollweide.
However, this is not a costly step. Moreover, we already project all the corners of each cell individually, which is also wasteful since a corner belongs to 4 cells, so we project it 4 times.

However, this is not a costly step. Moreover, we already project all the corners of each cell individually, which is also wasteful since a corner belongs to 4 cells, so we project it 4 times.

We have to move from numpy arrays for storing grid coordinates to polygons for each grid cell anyway, if we want to implement #3 with arbitrary shaped inventory cells.


ci: update to internal semantic release action

:tada: This PR is included in version 1.0.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


parallaxed realtedlayout strange behaviour

the parallax scroll view contains two views
relative view with image and staff
and linear layout with rest info
still whn scrolled down parallax keeps overlaying part of screen where " mac street 1 " is


Replace recommendation service by wildfly-swarm/microprofile

Not "replace", in addition to :-)

I think, the addition would be nodejs.
something like customer (springboot)-> preference (wfswarm) -> recommendation (vertx) -> finance (nodejs)

No, ideally we would have
customer-preference-recommendation
boot-boot-boot
swarm-swarm-swarm
node-node-node
vertx-vertx-vertx
the average person is going to focus on one of those runtimes/stacks.


The new affiliations should be visible within the next 7 days.


[CON-184] it’s also confusing that the brush is only a hover state but the delete is a button, instead of both being buttons

From SyncLinear.com | CON-184

If already selected, don't include paint brush


Plex Sensor stays unavailable after server is turned on

The problem

When I turn on my Plex server the sensor entity stays at 'unavailable'. I left the server on for multiple days and I never changed to '0 Watching'. However if I restart HA and my server is on by any chance the sensor will switch to '0 Watching'. It seems like the sensor only checks the server connection on booting.
Environment

Home Assistant release with the issue: 0.105.5
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.): Hass.io
Integration causing this issue: Plex
Link to integration documentation on our website: https://www.home-assistant.io/integrations/plex/

Problem-relevant configuration.yaml

#Plex
plex:
  token: XXXXXXXXXXXXXX

Traceback/Error logs

2020-02-20 16:49:06 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.
2020-02-20 16:49:19 WARNING (MainThread) [homeassistant.setup] Setup of recorder is taking over 10 seconds.
2020-02-20 16:49:24 ERROR (Thread-7) [plexapi] http://XX: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
2020-02-20 16:49:34 ERROR (Thread-6) [plexapi] http://XX: HTTPConnectionPool(host='XX', port=28320): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x71513950>, 'Connection to XX timed out. (connect timeout=10)'))
2020-02-20 16:49:34 ERROR (Thread-5) [plexapi] http://XX:32400: HTTPConnectionPool(host='XX', port=32400): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.HTTPConnection object at 0x71523870>, 'Connection to XX timed out. (connect timeout=10)'))
2020-02-20 16:49:34 ERROR (Thread-2) [plexapi] https://XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct:32400: HTTPSConnectionPool(host='XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct', port=32400): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.VerifiedHTTPSConnection object at 0x71513210>, 'Connection to XXX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct timed out. (connect timeout=10)'))
2020-02-20 16:49:34 ERROR (Thread-3) [plexapi] https://XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct:28320: HTTPSConnectionPool(host='XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct', port=28320): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.VerifiedHTTPSConnection object at 0x71523770>, 'Connection to XXX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct timed out. (connect timeout=10)'))
2020-02-20 16:49:35 ERROR (Thread-4) [plexapi] https://XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct:8443: HTTPSConnectionPool(host='XX.a0446c53d0724cfda26cf61a37d59ffe.plex.direct', port=8443): Read timed out. (read timeout=10)
2020-02-20 16:49:35 ERROR (MainThread) [homeassistant.components.plex] Plex server could not be reached: Bakbeest
2020-02-20 16:49:42 ERROR (MainThread) [homeassistant.components.plex] Plex server (http://XX:32400) could not be reached: [HTTPConnectionPool(host='XX', port=32400): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7294f9b0>: Failed to establish a new connection: [Errno 113] Host is unreachable'))]

Additional information

As I understand the series of events:

Plex server is off
HA is started
HA's Plex integration fails to connect
Plex server is started
HA's Plex integration does not detect the Plex server being available
If HA is restarted everything connects properly

@donpoedel Can you confirm this is the scenario you're encountering?

As I understand the series of events:

Plex server is off
HA is started
HA's Plex integration fails to connect
Plex server is started
HA's Plex integration does not detect the Plex server being available
If HA is restarted everything connects properly

@donpoedel Can you confirm this is the scenario you're encountering?

I confirm this is the scenario.


Fix broken & redirecting contributing links

This should hopefully make https://ci.openmicroscopy.org/view/Docs/job/CONTRIBUTING-merge-docs/ green and also clean up redirecting URLs and references to old docs URLs.
Once the doc job migration and reconfiguration is complete, I'll also need to remove references to the linkchecker parser from the ci-docs page unless we find a way to get that working again.

https://ci.openmicroscopy.org/view/Docs/job/CONTRIBUTING-merge-docs/1224/ is now green with this and #1807 included


Issue with latest RN 0.56-rc.3

Not entirely sure what the issue here is, but it seems in babel 7 it does not like the export default class { ... } syntax, as I am unable to build a production release.
In development it is working fine, but trying to build is throwing me a
/Users/dan/code/app/node_modules/react-native-root-siblings/index.js: /Users/dan/code/app/node_modules/react-native-root-siblings/index.js: Exporting local "_default", which is not declared. (This is an error on an internal node. Probably an internal error.)

Fixed by adding any gibberish after the class keyword (export default class Chicken { ... })

the same issue~

same issue, that will cause CD bug....

@deehuey @mailworks @joey3060 when are you guys going to merge these changes and release the fix? We are having real problems building our app because of this!

when is the merge happening? It blocks the building of the app

Fixed in https://github.com/magicismight/react-native-root-siblings/commit/1f95f1552e30fc661d9bf4f8f54ae88aad6b5b21


Refactored static docker compiler Dockerimage

I refactored the docker compiler:

Query GitHub API (curl + jq) to get the latest git tag, so there is no need for GO.
Installed requirements according to the original git package.
Use checkinstall to create an installable .deb package which in turn is installed on a clean ubuntu image.

The disadvantage of this docker image is that when run with a different user than root, git will have problems storing config and credentials because // is assumed as home directory.
Example build call: docker build --pull --tag git:latest
Example run call: docker run --init --rm --tty --interactive --user $(id -u):$(id -g) --volume $(pwd):/app --workdir /app git:latest clone https://github.com/git/git.git

Love it. Thanks for contributing!


RuntimeError: CUDA error: no kernel image

when I run torchpack dist-run -np 8 python tools/test.py configs/nuscenes/det/transfusion/secfpn/camera+lidar/swint_v0p075/convfuser.yaml pretrained/bevfusion-det.pth --eval bbox
It encounter a problem as mentioned below:
RuntimeError: CUDA error: no kernel image is available for execution on the device CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect. For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
I think everything before the BEV pooling kernel (especially the sparse convolution backbone) runs .But I do not know how to solve it.

RuntimeError: CUDA error: no kernel image is available for execution on the device
CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Primary job  terminated normally, but 1 process returned
a non-zero exit code. Per user-direction, the job has been aborted.

mpirun detected that one or more processes exited with non-zero status, thus causing
the job to be terminated. The first process to do so was:
Process name: [[27088,1],1]
Exit code:    1
I got this same

Do we have more information about the environment? E.g. type of GPU, version of CUDA, cuDNN and PyTorch? Also, please make sure that you install the correct versions of required libraries indicated in our README.

My CUDA version is 11.1, my PyTorch version is 1.9.0 and I'm pretty sure that my environment is aligned with cu111. The whole conda env is installed following README.
This problem occurs while I trying to inference with a pretrained BEV segmentation models using
torchpack dist-run -np 8 python tools/test.py configs/nuscenes/seg/fusion-bev256d2-lss.yaml pretrained/bevfusion-seg.pth --eval map
The error is actually happens after the QuickCumsumCuda.apply, and it goes like this
 File "/cbh/bevfusion/mmdet3d/models/vtransforms/base.py", line 167, in bev_pool     x = bev_pool(x, geom_feats, B, self.nx[2], self.nx[0], self.nx[1])   File "/cbh/bevfusion/mmdet3d/ops/bev_pool/bev_pool.py", line 96, in bev_pool     x = x.permute(0, 4, 1, 2, 3).contiguous() RuntimeError: CUDA error: no kernel image is available for execution on the device CUDA kernel errors might be asynchronously reported at some other API call,so the stacktrace below might be incorrect.

my CUDA version is 11.3.
GPU:GP107 [GeForce GTX 1050 Ti]
pytorch:1.10.0
other required libraries are installed indicated in your README
but do not intall cudnn

and like issue #26.I  removing all -gencode=arch=compute_XX,code=sm_XX in the setup.py.

and like issue #26.I removing all -gencode=arch=compute_XX,code=sm_XX in the setup.py.

I also tried but it doesn't work

I also change "distributed"as I think I need to run single_gpu_test()

Hi folks,
Would you mind also taking a look at this issue? For @junwenjiao's environment, I think you probably want to add something like compute_61 and sm_61 (corresponding to GTX1050Ti) to the setup file and rerun the setup develop command.
Besides I try to find some time setting up a Docker for the environment. But it takes some time.
Best,
Haotian

Hi folks,
Would you mind also taking a look at this issue? For @junwenjiao's environment, I think you probably want to add something like compute_61 and sm_61 (corresponding to GTX1050Ti) to the setup file and rerun the setup develop command.
Besides I try to find some time setting up a Docker for the environment. But it takes some time.
Best, Haotian

Hi Haotian, corresponding to GTX 1080ti, what should I add to the setup file?

Sorry for the late reply. For 1080Ti, it is also compute_61 and sm_61.

thanks for your help.I try it but there is nothing happened.

My problem is solved by using compute_61 and sm_61, thx

Glad to hear that. I'm closing this issue because it is resolved.


Add confirmation to bicep publish

Is your feature request related to a problem? Please describe.
At the moment bicep publish overwrites modules in the registry if you run a publish and specify the same version as an existing module in the registry. I can see that users will overwrite modules by mistake.
Describe the solution you'd like
If a module already exists with the same version the users should be prompted with a confirmation. And potentially have a --force option as well.
EDIT (antmarti): Alternative suggestion from #8426:

Is your feature request related to a problem? Please describe.
bicep publish does not lock down the published module, which enables it to be overwritten. This is dangerous in CI/CD pipelines that consume from the registry automatically. Imagine a bad actor overwriting a widely consumed module in the registry to add themselves as User Access Administrator
Currently, I am handling this in our CI/CD process by running Update-AzContainerRegistryTag immediately after publishing.
Describe the solution you'd like
bicep publish .\mybicep.bicep --target myacr.azurecr.io/mybicep:v1.2.3 --immutable would set the -DeleteEnabled $False -WriteEnabled $False on the published tag and verify the module tag is writable before attempting to publish.
reference:
Container Registry Image Lock
Update-AzContainerRegistryTag

Why not both? IMO immutability should be the default, with the option to override. As a break glass scenario, the container registry owner could always update the tag lock and delete or update the module, but that should be an edge case. There should be options within bicep itself, as well as container storage security measures.

I'd like to take this issue and implement it, but before I spend too much time I'd like to know if any decision has been made to make it default to read-only with --force as an option, or to keep it like it is and add an --immutable option 🙂

@PalmEmanuel -- if we can account for both the locking and unlocking of artifacts, I think we are good to go with you proceeding. Let us know if you have any other questions along the way.

I confirmed with the ACR team and unfortunately the AcrPush permissions allow the user to lock and unlock artifacts in the registry. This means that we can't protect a registry from a compromised identity overwriting everything by locking artifacts since the attacker can simply unlock each artifact beforehand. This means that we should not implement the --immutable switch until a more granular permission model is available from ACR.
That said, confirmation and --force switch would be great additions to the tool. @PalmEmanuel, feel free to submit that PR.

@PalmEmanuel Sounds good. Would you mind creating a separate issue for the --immutable switch, so we can continue the discussion?


When providing the custom map layer to MapView, map flickers on load.

Is this a bug report?
Yes

Have you read the Installation Instructions?
Yes
Environment
react: 16.0.0-alpha.12,
react-native: 0.48.3,

Steps to Reproduce

Start the application.
You can notice a flickering in Map.

Expected Behavior
Map should load with the custom map layer without flickering.

Actual Behavior

Take a look at this:
https://giphy.com/gifs/xT1Ra3GVr4RouOadWM/html5

@saxenanickk can you give a snippet of your code to reproduce the issue ? thank you !


Drawer not visible

I setup everything, and so far I'm not getting errors and the Home Component displays correctly. Unfortunately there is no menu-icon at the top and I'm to able to slide out the drawer.
So it is kind of invisible... Do I need to change something to activate inside in the components?

For everyone who had the same "issue":
look inside the App_Ressources folder and download the icons (ic_menu_black.png). You'll need them, otherwise the drawer won't work.


Unable to download Facebook videos

URL which failed: https://www.facebook.com/jerseydemic/videos/10153749930515681/
youtube-dl version: 2015.10.18
Output:
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'https://www.facebook.com/jerseydemic/videos/10153749930515681/', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.10.18
[debug] Python version 2.7.6 - Linux-3.13.0-65-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] exe versions: ffmpeg 1.2.6-7, ffprobe 1.2.6-7, rtmpdump 2.4
[debug] Proxy map: {}
[facebook]<PHONE_NUMBER>0515681: Downloading webpage
ERROR: An extractor error has occured. (caused by KeyError(0,)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 291, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/facebook.py", line 145, in _real_extract
    video_data = params['video_data'][0]
KeyError: 0
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 661, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 297, in extract
    raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(0,)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

It's fixed in Git but not included in the latest release yet. Please wait patiently for the next version.


Use cast instead of transmute

NonNull supports cast(), so there's no need for transmute.

Thanks for the PR. I'm fine with the change, that said I'd like to understand the motivation for the change a bit better. For example one thing I like about transmute is you immediately see and enforce the type you cast from and to, and if you change something you have to go through each relevant transmute and update them, re-evaluating if they are still valid. With cast that would not be a compile time error right?

Completely unrelated, I just realized you are the creator of gifski a project I've used quite a bit, thank you for the great project <3

The idea behind this is that transmute is a more powerful, more general-purpose function, and it has power of transmuting types that aren't compatible.
cast() is more specific and directly expresses what is happening, and is implemented only on pointer types.
In your case I think all transmutes are correct. However in general there are cases where transmute of exotically sized pointers is not well-defined (they don't have guaranteed layouts), but cast (which can change pointer layout) is allowed.

Sounds good. I'm looking into the miri failure that seems unrelated to this change. Please update the PR to be compatible with rustc 1.36 as tested by old_rust_rustc_1_36.

You should be able to rebase on main now and the miri test should run as expected.

The CI for this project is a bit dated, and the triggers were wonky. I took the liberty of fixing it on main and rebasing your branch. Change looks good otherwise.

ptr: self.joined_ptr.as_ptr().cast::<u8>() requires ptr_cast feature which is not available for rustc 1.36. The machinery for polyfilling it only for old rustc versions and if the users opt into the old_rustc feature is there, but I wonder if it is overkill. Thoughts?

Sure, I've replaced that with a regular pointer cast.
BTW, I expect 1.36, and all rust versions below 1.56, to be completely dead and unusable: https://lib.rs/stats#rustc

I'd be careful about such assumptions, also I have an upstream dependency that specifically asked for this feature https://github.com/Voultapher/self_cell/issues/30. As long as it seems reasonably possible to keep the minimum required rustc version the same, I plan on doing so.

I've merged the change, thanks for the PR. I'm not sure if this change has any immediate impact on users, do you think a new release is required?

Thanks. No need to rush with a release.


LS features fail due to unexpected character in package version

Description:

Steps to reproduce:
Affected Versions:
2201.1.0
OS, DB, other environment details and versions:
MacOS Monterey

The exception oriiginates from io.ballerina.projects.SemanticVersion -> from class due to the invalid version given here 2.3.0_temp. The ProjectException thrown here is not handled in the io.ballerina.projects.PackageVersion -> from.
This can be fixed by replacing,
PackageVersion.from(SemanticVersion.from(versionString));

with,
PackageVersion version;
try {
    version = PackageVersion.from(SemanticVersion.from(versionString));
} catch (ProjectException e) {
    version = ProjectUtils.defaultVersion();
}

However I wasn't able to recreate the issue so far. The error has been occurred when running org.ballerinalang.langserver.codeaction.providers.imports.PullModuleCodeAction when pulling a package from the central. However, the bal pack command throws an error if the version is not in the semver format.

This is due to a corrupted package(s) in the remote repository of ballerina user home(.ballerina). When we download a package from central if the download process is interrupted (internet connection failure) the corrupted pack is saved with the temp suffix.  When we try to load such packs from the ballerina user home using a resolution request ILLEGAL CHAR exception is thron. I think the Project API behavior is correct. Form the LS side in the LSPackageLoader we can explicitly catch and handle this exception and proceed with loading the remaining packages.

Fixed for 2201.1.x with https://github.com/ballerina-platform/ballerina-lang/pull/37218


Triage HN discussion for new issues

Discussion over on HN mentions a few issues that do not seem to be in the issue tracker.
https://news.ycombinator.com/item?id=22035258

Support HTML as input
Provide example inputs and outputs to quickly see what's possible
Maybe provide instructions to run Jupyter Notebook as quick start in documentation
Are any of the questions answered in the HN conversations suitable for inclusion in docs/FAQ?
It is reported that the link to the Google Vision documentation in 2.1 Extractor Tools of your documentation is broken

Feel free to break these out into separate issues, or ignore completely if already covered by existing issues!
HTH

(Apologies for the 'bug' label, I picked the wrong issue template)

Thanks for the report!
I just fix the broken link for Google Vision.


In the local cluster on dashboard --> Cluster details --> Kubectl Shell (happens in a downstream cluster also)
The kubectl shell window opens but is disconnected.
Error in console:

TypeError: Cannot read property 'providerOs' of undefined
    at f.getSocketUrl (KubectlShell.vue:20)
    at ContainerShell.vue:230
    at d (runtime.js:45)
    at Generator._invoke (runtime.js:274)
    at Generator.O.forEach.t.<computed> [as next] (runtime.js:97)
    at r (asyncToGenerator.js:3)
    at l (asyncToGenerator.js:25)
    at asyncToGenerator.js:32
    at new Promise (<anonymous>)
    at asyncToGenerator.js:21

On master-head commit id: f33188d
No error seen when Kubectl was launched on downstream cluster on the Local cluster.


Add option to not join pythonNamespaces

As many seed don't require the upseto join namespaces, and running packegg without it is a lot faster, I'd like to add the option to not join namespaces for the egg building.
The seed running however, should take into account that the seed being packed can be in a different repo.

+1 to Ido's comments.
I'd love to see the day that seeds simply run with PYTHONPATH=/usr/share/strato-noded or any other directory that already contains everything it needs.
But until then, shipit.

@motty-stratoscale  if node already contains what it needs yuo can pass the "exclude_package" parametrer that will avoid packing stuff that is already there, + inside the seed itself you can set the python path to whatever you like

This was rebased (and not merged) onto master.


make executable

can you make its executable version. The exe files are made for non-tech users or lazy tech-users to make things simple and easy to use.

Don't know how to do that, but I'll leave this issue open in case someone else wants to implement this

What is the difference between AI and human text?


data: HiLakshya

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

:)

Thank you for merging the PR!
:)


feat: refactor planning actions

:tada: This PR is included in version 1.28.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updating the repo

no breaking changes

 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.0 out of 4 committers have signed the CLA.:x: apichick:x: ludoo:x: NotArpit:x: dgulliYou have signed the CLA already but the status is still pending? Let us recheck it.


Add prop-types as dependency

Packages context, core & redux use prop-types as dependency but when you want to uses these packages, you must add prop-types dependency  into your project.
In some case, isn't necessary & prop-types doc recommand to include it as dependencies.
https://github.com/facebook/prop-types#how-to-depend-on-this-package

The build break
 lerna info Installing external dependencies lerna ERR! npm ci exited 1 in '@axa-fr/react-oidc-context' lerna ERR! npm ci stderr: npm ERR! Cannot read property 'prop-types' of undefined
Do you have an idea why @arnaudforaison ?

@guillaume-chervet was due to package-lock as always ;-)


Why the RouteInfoManager.java use nested try...catch

Just as the topic said.I don't know why the code use nested try...catch, maybe because unlock operation can throw an exception?
The example code is there:
Now
    public byte[] getAllTopicList() {
        TopicList topicList = new TopicList();
        try {
            try {
                this.lock.readLock().lockInterruptibly();
                topicList.getTopicList().addAll(this.topicQueueTable.keySet());
            } finally {
                this.lock.readLock().unlock();
            }
        } catch (Exception e) {
            log.error("getAllTopicList Exception", e);
        }

        return topicList.encode();
    }

Expected
    public byte[] getAllTopicList() {
        TopicList topicList = new TopicList();
        try {
            this.lock.readLock().lockInterruptibly();
            topicList.getTopicList().addAll(this.topicQueueTable.keySet());
        } catch (Exception e) {
            log.error("getAllTopicList Exception", e);
        } finally {
            this.lock.readLock().unlock();
        }
        return topicList.encode();
    }

getAllTopicList

I agree with u, unlock can throw exception indeed.

getAllTopicList

I agree with u, unlock can throw exception indeed.

Yes, you were right about this. The unlock method will throw an Exception if current thread didn't obtain the lock, which means some exception happens when call the lock method.
Thank you for your replay.


Create a contributions breakdown chart for MMM

Hi!
I think business stakeholder often enjoy plots like this:

What do you think?
Best
Robert

I agree @Garve, want to do a PR?

Yeah, in my version you can see per day how much sales were generated because of the different media channels. This works for linear models such as the DelayedSaturatedMMM since we can decompose the sales into sales = f(TV spends) + g(Radio spends) + ...


Package Control does not exist, link to instructions is broken.

As usual, anything that is open source is generally completely useless and I am better off building something myself from scratch.
 It does not exist in Sublime Text 3.
Dead link for instructions: http://wbond.net/sublime_packages/package_control

There is an "Instructions" link on the Package Control home page which points to:
https://packagecontrol.io/installation
Make sure you install the correct version (2/3) for your version of Sublime.
And let us know when you write your own editor.

WTF is "Package Control"?

http://lmgtfy.com/?q=Package control
Can't wait to see what you produce.

https://packagecontrol.io/installation
Package control and all that github thing is a bit awkward at first, but once you get the hang of it you' won't be able to live without it !

 Maybe next time just say something like this in your ticket: "The readme mentions something called Package Control but I can't find it in my ST3 install and I'm new to Sublime... What am I missing?"
You know, rather than insulting everyone that works on the free open source software that you take for granted because you were too lazy to google.

Developers are all ego and no value. Just because something hypothetically could be great if it worked is completely meaningless.  I don't care if this software could navigate a spaceship to the moon - if I can't install it, then it is useless. I spent 30 minutes googling, reading and having failed attempts at installing before giving up.
Open source development is an intrinsically flawed concept and leads to these such outcomes.
Point in case, the link in your comment above is to a branch of another open source project which I attempted to use and was a total failure.  I did in fact, build a Facebook OAuth library from scratch, in one day, and it is working gloriously. Keep in mind, I don't consider myself a good developer - so how is it that I can build such things when the real good developers cannot?
 It's time for a shift in the way developers think and work.

It's time for a shift in the way you ask for help.  But lets stop pointing fingers :)
Did you get it installed?  Can you follow these instructions: https://packagecontrol.io/installation
It doesn't get any simpler than that. Cut n paste a string, open Sublime's console. Paste. Done.

Jim, I would say give up, he's obviously not interested in getting it working, and just using this as a venting platform.
He's obviously used git, but seemingly can't even follow the install instructions for cloning the repo into the packages directory. (which is what he should be doing given that he said he's using SublimeText 3 and the first main line of the readme under SublimeText 3 is::

The development branch contains a rewrite of the ColdFusion plugin. The only installation method is via Git.

)
Grant, if you're serious about getting it running, raise a new issue and rather than spend your precious minutes complaining and giving a sob story about how developers have "lost you millions", actually give us information about your set up, and what you have tried.
Closing and locking.

As usual, anything that is open source is generally completely useless and I am better off building something myself from scratch.

So I linked your own github profile.

Point in case, the link in your comment above is to a branch of another open source project which I attempted to use and was a total failure. I did in fact, build a Facebook OAuth library from scratch, in one day, and it is working gloriously.

You've got no evidence you've built anything from scratch, at least not to boast about in a public forum.

Now, you say you spent 30 minutes googling. I find this claim dubious. Literally the first result for "package control" is the correct website, and it even has a surfaced link for Installation right in the google results.
If you couldn't figure out how to install Package Control, that's on you. The process is about as simple as it could possibly get, and the directions are clear and easy to find.
To add insult to injury, you came in here claiming that the plugin/documentation (and by extension, the developers that have built it for you) are "completely useless." Had you simply been nice, none of this discussion would have happened. Look through the closed issues on this repository, there are dozens of people that needed help getting it installed and they were all met with kind, generous, and rapid help, because they weren't assholes.
Part of me wants to just ignore this thread / you, because you haven't shown that you are worth my time. BUT that's not what open source is about, so in the spirit of the project, here you go, literally everything you need to do to get Package Control installed:

Keyboard shortcut ctrl+`, or if you prefer menus, View > Show Console
Then copy this string to your clipboard: import urllib.request,os,hashlib; h = 'eb2297e1a458f27d836c04bb0cbaf282' + 'd0e7a3098092775ccb37ca9d6b2e4b7d'; pf = 'Package Control.sublime-package'; ipp = sublime.installed_packages_path(); urllib.request.install_opener( urllib.request.build_opener( urllib.request.ProxyHandler()) ); by = urllib.request.urlopen( 'http://packagecontrol.io/' + pf.replace(' ', '%20')).read(); dh = hashlib.sha256(by).hexdigest(); print('Error validating download (got %s instead of %s), please try manual install' % (dh, h)) if dh != h else open(os.path.join( ipp, pf), 'wb' ).write(by)
Paste it into the console and hit the return key. Wait a few moments while PC is downloaded and installed. You will probably need to restart for it to take effect.
Now, when you have some time (when you're not busy creating loads of great software that us useless open source contributors couldn't possibly get right), read up a little bit on the great features that Package Control has to offer. It's pretty awesome, and you'll probably like it a lot. I only linked the end-user docs. On the off chance that you want to create a package, or do other more advanced things, there is more documentation available here.

Now some bad news:
Had you actually bothered to read the readme, you would know that you can't install this plugin in ST3 using package control. Instead, you'll need to install via git.

The development branch contains a rewrite of the ColdFusion plugin. The only installation method is via Git.

You can either:

download this zip, and unpack it into your packages folder ("Preferences > Browse Packages..." to find it) so that you have Packages/ColdFusion/ NOT Packages/User/ColdFusion/
or open your packages folder in a terminal window and install it with this one-liner: git clone https://github.com/SublimeText/ColdFusion.git && cd ColdFusion && git checkout development (Assumes you're on Linux or Mac, since your bug report didn't bother to specify.)

After either, you should restart Sublime for the changes to take effect.
Have a nice day!


optimize: handler and per request cpu and allocations

Codecov Report

Merging #43 (c52f08b) into 0.4.0 (690c86a) will decrease coverage by 0.94%.
The diff coverage is 76.27%.

@@            Coverage Diff             @@
##            0.4.0      #43      +/-   ##
==========================================
- Coverage   90.81%   89.87%   -0.95%
==========================================
  Files          23       22       -1
  Lines         969      958      -11
==========================================
- Hits          880      861      -19
- Misses         53       60       +7
- Partials       36       37       +1

Impacted Files
Coverage Δ

internal/reqdata/request.go
77.98% <69.69%> (-3.90%)
:arrow_down:

handler.go
81.30% <78.94%> (-0.70%)
:arrow_down:

api/context.go
100.00% <100.00%> (ø)

internal/config/service.go
86.33% <100.00%> (ø)

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 690c86a...c52f08b. Read the comment docs.

Closes #8


ARROW-1182: C++: Specify BUILD_BYPRODUCTS for zlib and zstd

Change-Id: I7f4ce23ead7ab1b27741b80a382c0cd4d0019895

I think we should use ninja in our builds, partly because it's faster than make but also because it is more picky about the correct specification of dependencies.

Appveyor build https://ci.appveyor.com/project/xhochy/arrow/build/1.0.397


Reolink integration 0.51 with Reolink software firmware .764 not connecting

I have multiple Reolink camera types on my system. Some of them connect to HA and some do not. The issue seems to be related to the .764 or later updates with cameras on such firmware.
The camera types currently providing video integration into HA with Reolink 0.51 are
RLC-820A, E1 Zoom & RLC-510WA
The cameras RLC-1212A & RLC-823A
can not be connected to HA, they respond with "Failed to connect with the camera". Both cameras are relatively new models with only one firmware, .764 or later available.
The RLC-820A and RLC-510A and E1 Zoom have updates .494, & .716 available and they do connect fine when I roll them back to updates prior to .764. but will not connect when I updated them to .764.
Is it possible that Reolink made some changes to their API with their .764 and later updates that the current Reolink .51 has not yet accounted for?
Running Home Assistant Core 2022.3.7 with Supervisor 2022.3.5 on HA OS 7.5 with Reolink V0.51 on an intel NUC
Error when trying to integrate for cameras with firmware .764 and later

I have reattempted this video integration under HA OS 7.6 and Core 2022.4.3 and cameras with firmware 764 and later are now connecting without error and working, as far as video integration. I will close this issue since it was specific to this topic. It is not clear to me exactly what solved it. There have been HA OS updates (1) and CORE updates (multiple) since my initial report and I do not know which version solved the issue


Improvements to Sync Service

Closes #498 .
Removed unused variables and functions, moved some of the public ones to private because they shouldn't be accessible to outsiders. And most importantly, improved the sync service's performance by giving it only players' entities.
So far it seems alright in testing, but as always - test again to ensure it's good for merging.
Some note: I cannot delete the pointer to entity because entity's destructor is private. Perhaps we should use a vector of smart pointers instead?

Not at this point though, all Entity instances are be 'owned' only at a single point, so making those shared_ptr would muddle things a bit, I would even consider making most of Entity * into EntityHandles that are accessing EntityManager::m_store

So I suppose we should make this EntityHandle class that uses the EntityManager's m_store... That makes sense because we'd want the entity to be put into m_free_entries back like some sort of 'entities pool'. The way I look at it, just having access to m_store is not enough, so should we make EntityHandler have a reference instead of a pointer to Entity, and if we have pointers, would the memory no longer be shared between it and EntityManager?

Ready for merge. Testing is okay with 2 characters on map. Seems I have to re-create the message in sync service because the original message probably got 'muddied' by being sent as a SEGSEvent to the sync service, and then casted back to a Character/PlayerUpdateMessage.

Works great!

@nemerle I'll leave this to you to merge when you've had a chance to review.


Refactor SortOrder & SortOrderConfiguration into one enum

Not an issue fix, but as part of a previous PR I noticed that a separate object SortOrderConfiguration and type SortOrder are used to emulate an enum.
Also replaced 2 occurrences of direct string comparison with the enum equivalent.

@jzyrobert thanks for this. Did you verify that this does not break anything and that the explorer.sortOrder still works as expected?
Can you please resolve conflicts with master and then we can look into merging this? Thanks

@isidorn Works as before:

I have rebased and resolved the conflict.

Thanks a lot for this. Merging in.


Add indent to property chaining on multiple lines

For #111

Hi, can you add test to this pull request? We have example test here. https://github.com/HerringtonDarkholme/yats.vim/blob/master/test/tsx.indent.vader

Hello! Of course. Will be doing that soon

Found case where it does not work
object
  .method(x => {
some code;
  });

Will try to fix first


[DPG] optimize --package-mode to be compatible with more scenarios

fix https://github.com/Azure/autorest.python/issues/1290

Hi @iscai-msft please feel free to review ^^

/azp run autorest.python - ci

need re-design


Request option to detect doubles when games are included in other games.

This is an example where Scid finds a duplicated game because on game is included in another game. Ath the moment ocgdb don't detect these doubles.
Scid only detect these count of doubles when the player names are identical
I also want an option to detect these doubles when player names are different or other information like event is different. Than I want ocgdb to keep the better game, (longer game, more recent, higher elo, longer time control ,etc).
Maybe it are technically not exact doubles. But I don't see how it can be useful to keep games with exact the same moves, and exact the same results, together in a database.
Doubles where one game is included in another happens quite often in human tournaments when using DGT boards, sometimes the feed gets an extra move or a database feed like TWIC does an update one week and provides a game correction the next week.

That is doable.
However, it may take a lot of time, slow painfully. The reason is that without extra conditions (such as same players), the app has to compare a game with all other games, the complexity becomes O(n!), much larger than the O(n) of our current function to check duplicates.

With Scid it goes very fast to find these doubles. Even diectly with a pgn file. Maybe it goes fast because the player names have to be the same with Scid.

I did some test with Scid. I removed al the player names for white and for black in a huge pgn database with Chess Assistant 20. Than I searched this pgn database for duplicates. It was still very fast and doubles because games are included in other games where also detected.

I have implemented the request. It could run very fast, almost as fast as the normal duplicate-check function.
Run it as:
ocgdb -db big.ocgdb.db3 -dup -o embededgames -r report.txt
Committed: https://github.com/nguyenpham/ocgdb/commit/8726834462c0037d7b8fe97bc6228e28feca31db

Thanks!
Can I download the updated ocgdb tool somewhere to give it a try?

We have released a new version Beta 7 https://github.com/nguyenpham/ocgdb/releases/tag/VersionBeta7

I have tried it with with ocgdb Beta 7
With these two games

[Event "Barcelona Ideal Clave op 21st"]
[Site "Barcelona"]
[Date "2017.10.07"]
[Round "2"]
[White "Rojas Nunez, Alberto"]
[Black "Lopez Gomez, Nicolas"]
[Result "0-1"]
[WhiteElo "1276"]
[BlackElo "1526"]
[EventDate "2017.09.30"]
[PlyCount "54"]
[EventType "swiss"]
[EventRounds "8"]
[EventCountry "ESP"]
[SourceTitle "CBM 181 Extra"]
[Source "ChessBase"]
[SourceDate "2017.12.12"]
[SourceVersion "1"]
[SourceVersionDate "2017.12.12"]
[SourceQuality "1"]

1.e4 e5 2.Nf3 Nc6 3.d4 exd4 4.Nxd4 Ne5 5.Nc3 Bb4 6.a3 Bxc3+ 7.bxc3 d6 8.

Bd3 Nf6 9.f4 Nxd3+ 10.cxd3 O-O 11.O-O Re8 12.f5 d5 13.e5 Rxe5 14.Qf3 c5
15.Bf4 Re7 16.Ne2 Bxf5 17.Ng3 Bg4 18.Qf2 b6 19.h3 Bh5 20.Rae1 Bg6 21.Qf3
Qd7 22.c4 Rae8 23.Rd1 d4 24.h4 h6 25.Bxh6 Ng4 26.Bg5 f6 27.Bf4 Ne3 28.h5
Bh7 29.Bxe3 Rxe3 0-1
I used these two commends
ocgdb -pgn one_embeded_game.pgn -db one_embeded_game.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco -plycount 40
ocgdb -db one_embeded_game.db3 -dup -o embededgames -r report.txt
ocgdb Beta 7 didn't find the imbedded double
Or maybe I do something wrong?

Thanks for the report.
The bug is fixed by the commit https://github.com/nguyenpham/ocgdb/commit/1a12d17cda25360c329169e601f0201f8d1c333e
(it is a funny bug: after testing, I cleaned the code and accidentally removed some new ones).
The executing file could be downloaded with the below link:
ocgdb.zip

Thanks for the report.
The bug is fixed by the commit https://github.com/nguyenpham/ocgdb/commit/1a12d17cda25360c329169e601f0201f8d1c333e
(it is a funny bug: after testing, I cleaned the code and accidentally removed some new ones).
The executing file could be downloaded with the below link:
ocgdb.zip

I tested it out with 60_games_with_doubles
I used these 3 commends one after the other
ocgdb -pgn 60_games_with_doubles.pgn -db 60_games_with_doubles.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb -pgn 60_games_with_doubles.pgn -db 60_games_with_doubles.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb -pgn 60_games_with_doubles.pgn -db 60_games_with_doubles.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb beta 7 dit not find this included duplicate the first run

[Event "Rated Blitz game"]
[Site ""]
[Date "2019.04.21"]
[Round ""]
[White "FischersFrisoer"]
[Black "MarkoMakaj"]
[Result "1/2-1/2"]
[WhiteElo "2333"]
[BlackElo "2372"]
[ECO "E87"]
[TimeControl "180"]
[PlyCount "24"]

1.d4 Nf6 2.c4 g6 3.Nc3 Bg7 4.e4 d6 5.f3 O-O 6.Be3 e5 7.d5 Nh5 8.Qd2 Qh4+ 9.Bf2 Qf4 10.Be3 Qh4+ 11.Bf2 Qf4 12.Be3 Qh4+ 1/2-1/2
[Event "Rated Blitz game"]
[Site ""]
[Date "2018.09.27"]
[Round ""]
[White "FischersFrisoer"]
[Black "toramal"]
[Result "1/2-1/2"]
[WhiteElo "2277"]
[BlackElo "2213"]
[ECO "E87"]
[TimeControl "180"]
[PlyCount "25"]
1.d4 Nf6 2.c4 g6 3.Nc3 Bg7 4.e4 d6 5.f3 O-O 6.Be3 e5 7.d5 Nh5 8.Qd2 Qh4+ 9.Bf2 Qf4 10.Be3 Qh4+ 11.Bf2 Qf4 12.Be3 Qh4+ 13.Bf2 1/2-1/2
When I run the same comments again with these two games, ocgdb found the imbedded duplicate

I did some other test to see how good the detection of imbedded duplicates works with ocgdb beta 7
I run commends with on this pgn file
imbedded_duplicates
I run these commends on after the other
ocgdb -pgn imbedded_duplicates.pgn -db imbedded_duplicates.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb -db imbedded_duplicates.db3 -cpu 4 -dup -o printall;embededgames;remove -r report.txt
ocgdb -pgn imbedded_duplicates_out.pgn -db imbedded_duplicates.db3 -cpu 4 -export
ocgdb didn't detect these imbedded duplicates
not_find_duplicates

Thanks for the report!
At the moment the program works roughly as below to find duplicates: compare each game with all other games to find matching (fully or partly). If found, stop for that game and do reporting/removing.
That means that game (being matched to one other game) may not be continued to compare/found more results with the rest.
Your test database is quite special in which a game could be embedded inside many other games. Thus OCGDB may detect the first embedded duplicate of a game but not other ones.

So If I run this comment:
ocgdb -db imbedded_duplicates.db3 -cpu 4 -dup -o printall;embededgames;remove -r report.txt
Multiple times, I will eventually be able to remove al the imbedded duplicates?
I created the pgn by importing best moves for one color from a bin book in Lucas Chess to opening lines. If I remove al last moves (1 ply) if it is a black move. I I then export the opening lines to pgn there are many imbedded duplicates included. SCID has no problem detecting al these imbedded duplicates in on go to remove them.
I hope this wil be possible with OCGDB to with future updates.

I have tried beta7c with the example pgn database
imbedded_duplicates
Something goes wrong because almost al games get removed, except these two games

[Event "imbedded duplicates"]
[PlyCount "5"]
[ECO "B28"]
[White ""]
[Result ""]
[Black ""]
[Round ""]
[Date ""]
[Site ""]

e4 c5 2. Nf3 a6 3. c3 *

[Event "imbedded duplicates"]
[PlyCount "1"]
[Site ""]
[White ""]
[Result ""]
[Black ""]
[Round ""]
[Date ""]

e4 *

I run these commends on after the other
ocgdb -pgn imbedded_duplicates.pgn -db imbedded_duplicates.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb -db imbedded_duplicates.db3 -cpu 4 -dup -o printall;embededgames;remove -r report.txt
ocgdb -pgn imbedded_duplicates_out.pgn -db imbedded_duplicates.db3 -cpu 4 -export
If I search the example pgn database 'imbedded_duplicates.pgn' for duplicates with SCID it founds 43 duplicates.

OK, I have found some weak points in the algorithms for checking embedded games. Hope it works now:
ocgdb-WinMac-beta7d.zip

Thanks for the fix!
I have tested it with the example database and ocgdb beta 7 d find 43 imbedded duplicates, the same as with SCID

I did some other small test with ocgdb beta 7 d
I used this example pgn:
24_games_1_embedded_duplicate
I typed these commands one after the other:
ocgdb -pgn 24_games_1_embedded_duplicate.pgn -db 24_games_1_embedded_duplicate.db3 -cpu 4 -o moves2;discardcomments;discardsites;discardfen;reseteco
ocgdb -db 24_games_1_embedded_duplicate.db3 -cpu 4 -dup -o printall;embededgames;remove -r report.txt
ocgdb -pgn 24_games_1_embedded_duplicate_out.pgn -db 24_games_1_embedded_duplicate.db3 -cpu 4 -export
ocgdb beta 7 d, did not find this embedded duplicate

[Event "Rated Blitz game"]
[White "FischersFrisoer"]
[BlackElo "2303"]
[Result "1/2-1/2"]
[Black "MarkoMakaj"]
[Date "2019.03.06"]
[TimeControl "180"]
[WhiteElo "2366"]
[ECO "E87"]
[PlyCount "25"]
[Round ""]
[Site ""]

1.d4 Nf6 2.c4 g6 3.Nc3 Bg7 4.e4 d6
5.f3 O-O 6.Be3 e5 7.d5 Nh5 8.Qd2 Qh4+
9.Bf2 Qf4 10.Be3 Qh4+ 11.Bf2 Qf4 12.Be3 Qh4+
13.Bf2 1/2-1/2
[Event "Rated Blitz game"]
[White "FischersFrisoer"]
[BlackElo "2245"]
[Result "1/2-1/2"]
[Black "toramal"]
[Date "2018.05.08"]
[TimeControl "180"]
[WhiteElo "2323"]
[ECO "E87"]
[PlyCount "24"]
[Round ""]
[Site ""]
1.d4 Nf6 2.c4 g6 3.Nc3 Bg7 4.e4 d6
5.f3 O-O 6.Be3 e5 7.d5 Nh5 8.Qd2 Qh4+
9.Bf2 Qf4 10.Be3 Qh4+ 11.Bf2 Qf4 12.Be3 Qh4+
1/2-1/2

Thanks for the report.
The bug is caused by multi threads conflict (two embedded games may be processed concurrently). The temporary solution is to set -cpu 1 then the program could run correctly.

Yes, any chance may make processes a little bit different, delete the change which two embedded games being processed concurrently.
You may change the parameter -cpu, say from 4 to 3 and can help too.
Moves1 is to encode a move into one byte when Moves2 to two bytes. Moves1 can help to have smaller size databases. However, encoding/decoding code of Moves1 is quite complicated and other programs have almost no way but to integrate with our code (OCGDB code). That is a big trouble if other programs are not same programming languages. Integration code from other libraries/programs typically is not easy and need some labours. In contrast, code for Moves2 is quite simple, straightforward, using Stockfish board coordinates. Any program could implement itself the encoding/decoding without integrating OCGDB's code. Thus we encourage/suggest people to use Moves2 for being easier, sharing/publishing. Moves1 may be good for private uses, researches.


Library not updated

Hello,
Just a little message to let you know that the library available here : https://pypi.org/project/pntools/ is not the same as the one of the github repository.
Best,
Mathilde

Hi,
I merged your PR and published the new version as v0.3.1.
BR,
Thomas


Skip links smarter

This changes the default behavior regarding links found in the copy. Now by default if the link text begins with "http" it will be removed automatically.


Added a more updated alternative to isomorphic-fetch.

isomorphic-fetch hasn't been updated for a while, so cross-fetch provides a more flexible and up-to-date alternative to the community.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


GroupBy + Count

Hello team,
I have recently started using Redis Stack with the OM Dotnet library, but I having some troubles achieving a group + count result.
I have a collection of sport events. Each, among other properties, has SportCode and Status. What I am trying to achieve is to first group all items by SportCode and then execute 2 counts - 1 unconditional count of all records within a group and one conditional count of the events whose status is equal to IN_PROGRESS.
This sort of aggregations are described in the documentation using an example with a SUM, where the return type is GroupedAggregationSet<TEntity>. Using COUNT however returns an int instead.
 var grouped = _cacheAggregation.GroupBy(x => x.RecordShell.SportCode).Count();

Is this a bug or am I missing something?

Count is a native extension of IEnumerable that reduces the whole enumerable to a single integer with a count, use CountGroupMembers, that should do what you're looking for - the output will be stored at the COUNT key.

Thanks, this is a nice one. Is there a plan to support conditional count? On the above example, I need to get all groups, but in addition count how many events in a group have a status of IN_PROGRESS, which doesn't seem to be possible with CountGroupMembers

You'd have to filter it ahead of time using a Where/Filter

Closing this out as I think the questions posed are answered


SAK-45187 Portal: Presence icon is gone, now only a text version of number of users displays

https://jira.sakaiproject.org/browse/SAK-45187

@victorfv2, I took a look at this with @profmikegreene and came up with a suggested fix: #9298
Let me know what you think.


Adding Microsoft Visio Viewer 2016

Microsoft Reviewers: Open in CodeFlow

At the end of setup a dialog popped up.

nd of setup a dialog popped up

I have amended the silent switches and resubmitted


doc: translate the readme into Chinese

I can do this too, assign me plz

Welcome，thx~

do it in #428


upgrade to netcore preview 3

Many thanks Dani!
I ended up bumping the version to 0.9.0-preview-1 to keep it in line with the blazor version. Thanks again!


[fix] add test cases and fix is_content_type_textual

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Nothing much, just add more test cases

thank you! mind fixing the lint errors?
for reference, it's helpful if you don't delete the pull request template — it includes a checklist that includes things like linting, and it's useful for maintainers to be able to understand the scope of a PR (whether it requires a version bump, etc)

@benmccann can you review again

thanks!


[main] Fix privacy manifest definitions

Changes
Fixed privacy manifest definitions.

Sorry, wrong target repository.


Add support for fixed header on desktop

Extended the existing SCSS for the header to support fixed headers on the top and bottom for desktop layouts. Also, added padding to the body element when the header is fixed to the top or bottom, including when there is additional vertical spacing.

In the stylesheet for the Header layout, the main changes include renaming .brand to .header-brand, and the addition of new CSS rules for several components like .header-tabs, .header-link, and .header-menu. Also, padding and separator behaviors were updated.


[Question]

Checks

[X] I confirm that I have searched for existing issues / pull requests before reporting to avoid duplicate reporting.
[X] I confirm that I noted that if I don't follow the instructions, the issue may be closed directly.

Description
I have installed the mod and everything is working, however when typing on the server no text is shown, but on the discord server i see the text, do you know what the issue would be?
When i Type in game:

What is showed on discord:

Nvm i had to set the formatchatmessages to false


Easy permissions class

I think I fixed the issue with my conflicting local user account! This PR has a class that can be used to easily generate user permissions arrays. @thinkingserious thanks so much!

 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: hicks2evan:x: Evan HicksEvan Hicks, hicks2evan seem not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Shoot, it looks like it's viewing that old local username as a contributor, is there a workaround for this?

Hi @hicks2evan,
You would need to remove the commits from that user. Probably easier to just submit a new PR.
With Best Regards,
Elmer

@hicks2evan Just a heads-up: you can in fact fix your local commit history using 'git rebase --interactive'! If you would like some help with this, you can find me on IRC (irc.freenode.net) as 'clee', and I'd be happy to walk you through it. Since the commit history issue seems to be the only thing standing in the way of merging this, it'd be great if we could get that fixed up!

HI @hicks2evan,
It looks like the issue is that, besides the last 3, your commits are attributed to a non-GitHub account. It might be easier to just close this PR and create a new one which is authored solely by your @hicks2evan account.
Thanks!
With Best Regards,
Elmer


Migrate OnfidoPrivacy.js to functional component

Details

Migration o OnfidoPrivacy.js to functional component
Fixed Issues

$ https://github.com/Expensify/App/issues/16234
PROPOSAL: https://github.com/Expensify/App/issues/16234#issuecomment-1673342669
Tests

I don't have an access to the wallet, so that's how I tested it (at least on devices where I could):

Go to EnablePaymentsPage by using the url (...)/enable-payments
In React Devtools change props of EnablePaymentsPage (userWallet.currentStep = "OnfidoStep")
Check if there are no errors in JS console
Check if everything is displayed properly

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

If the QA engineer can access the wallet:

Check if everything is displayed properly
Check if you can go to the next step
Check if there are no errors in JS console

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 / Chrome
[x] iOS / native
[x] iOS / 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 a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] 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)
[x] 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)
[x] All JSX used for rendering exists in the render method
[x] 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

[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(themeColors.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 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.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@marcaaron this is the issue we've talked about, please test it internally to make sure that the wallet beta feature is still working correctly

@marcaaron - pinging to remind about checking this PR

@thesahindia can you review this please and get through whatever testing steps you can?

Thanks @BartoszGrajdek for the quick work and @thesahindia for the review.

: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/marcaaron in version: 1.3.58-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.58-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/marcaaron in version: 1.3.59-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.59-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Diagnostic Filters not being applied to files that do not exist [moved from BSLint]

[originally posted in BS Lint, but I don't think that was the right place..]
I have diagnostic filters for **/maestro/**
I discovered the issue while using Maestro. Maestro creates a ton of files under maestro/generated. However, the linter thinks these files are located under components/maestro/generated when the actual path is src/components/maestro/generated.
So there are two problems here:

The source path for maestro/generated is incorrect (I'm not sure if this is caused by maestro or by bsconfig) - this might be a maestro problem?
Regardless though, the errors are not being filtered out despite diagnostic filters that should be applied

The files produce 1082 errors. If you click on any of the warnings you receive an editor error "Cannot open because file not found" because the path is incorrect

bsconfig.json
{
  "sourceMap": true,
  "stagingDir": "build",
  "retainStagingDir": true,
  "rootDir": "src",
  "files": [
    "manifest",
    "source/**/*.*",
    "images/**/*.*",
    "fonts/**/*.*",
    "meta/**/*.*",
    "components/**/*.*",
  ],
  "autoImportComponentScript": true,
  "createPackage": false,
  "diagnosticFilters": [
    "**/roku_modules/**",
    "**/maestro/**",
    "**/RALETrackerTask.*",
  ],
  "plugins": [
    "maestro-roku-bsc-plugin",
    "rooibos-roku",
    "roku-log-bsc-plugin",
    "@rokucommunity/bslint"
  ],
  "maestro": {
    "excludeFilters": [
      "**/RALETrackerTask.*",
      "**/roku_modules/**/*",
      "**/node_modules/**/*",
      "**/rooibos/**/*"
    ],
    "buildForIDE": true,
    "extraValidation": {
      "doExtraValidation": true,
      "doExtraImportValidation": true,
      "excludeFilters": []
    }
  },
  "rooibos": {
    "isRecordingCodeCoverage": false,
    "testsFilePattern": null
  },
  "rokuLog": {
    "strip": false,
    "insertPkgPath": true,
    "removeComments": true
  },
  "logLevel": "error",
  "transpileOptions": {
    "removeParameterTypes": true
  }
}

This should be resolved by https://github.com/georgejecook/maestro-roku-bsc-plugin/pull/94


JournalManager.GetContacts should accept two parameters

Bug
Operating system: Windows
Game version: 2.1
CET version: v1.29.1
GPU: -
Description
JournalManager.GetContacts() should accept 2 parameters but it's throwing an error in logs
[2024-01-28 22:23:41 UTC+00:00] [13412] ...weaks\mods\make_a_living__courier\modules\Panopticon.lua:76: Function 'GetContacts' requires 1 parameter(s).
Expected behaviour
It should not throw an error
Reproduction
Override("JournalManager", "GetContacts", function(this, journalRef, entries, wrapped)
        wrapped(journalRef, entries)
    end)

If you can replicate this on latest CET release, feel free to reopen. But I would not be so sure it has 2 params tbh when the parser writes it has one, could have changed between updates.
If it is still an issue, we can try to have a look I guess. But my personal guess is that either sth changed with the update or you are trying to pass self explicitly.

Makes sense. Once I'll be developing a new mission I'll confirm this


[Experiment] Remove ParenType

Today ParenType is used:

As the type of ParenExpr
As the payload type of an unlabeled single associated value enum case (and the type of ParenPattern).
As the type for an (X) TypeRepr

For 1, this leads to some odd behavior, e.g the type of (5.0 * 5).squareRoot() is (Double). For 2, we should be checking the arity of the enum case constructor parameters and the presence of ParenPattern respectively. Eventually we ought to consider replacing Paren/TuplePattern with a PatternList node, similar to ArgumentList.
3 is one case where it could be argued that there's some utility in preserving the sugar of the type that the user wrote. However it's really not clear to me that this is particularly desirable since a bunch of diagnostic logic is already stripping ParenTypes. In cases where we care about how the type was written in source, we really ought to be consulting the TypeRepr.

@swift-ci please test

@swift-ci please test source compatibility

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test source compatibility

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test source compatibility

https://github.com/apple/llvm-project/pull/4769
@swift-ci please test source compatibility

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test source compatibility

Split out a couple of PRs to hopefully make this a bit easier to land:

https://github.com/swiftlang/swift/pull/77202
https://github.com/swiftlang/swift/pull/77203

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test source compatibility

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test source compatibility

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test Windows

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please smoke test macOS

https://github.com/swiftlang/llvm-project/pull/4769
@swift-ci please test macOS

🚢


Feature Request: Vector Operations

I just came across this project, and wow I am impressed.  Currently, Neo4j supports vector operations, more specifically, similarity calculations.  It would be great if we could extend the concept of fixed-length lists and perform similarity operations.  Maybe that support already exists and I am overlooking how to achieve this with your stack, but this would be a great feature to help support RAG operations.

Thanks for raising this. I agree that we should support some of the common functions. We can follow DuckDB's  array functions: https://duckdb.org/docs/sql/data_types/array.html#functions. Arrays in DuckDB are equivalent to our fixed-length list type, so I don't think there is a hurdle in supporting these.
I'll put this into our pipeline.

Hi @Btibert3 that makes a lot of sense. Could you elaborate a bit on what the intended use case is in the context of RAG? To you, how would the ideal implementation look from a graph query perspective?

@semihsalihoglu-uw DuckDB is exactly what I had in my head.
@prrao87 Naive RAG lets you find entries (i.e. nodes) based on the similarity of the vector to the input query.  We can go beyond this by further restricting the results by leveraging graph patterns.  One example might be to show a list of products based on the user's input query (vector similarity) but further restrict/re-rank the results based on products the user hasn't purchased and behavior of other "similar" users, where similarity in this context is leveraging graph relationships.  In this example, the results come from vector-based similarity and graph relationships.
Another example would be to consider the most similar document chunk via vector search, but improving context windows based on linked nodes and variable pattern matching, again using vector similarity but also the structure of the relationships in the graph.

@Btibert3 May i know which vector operations you are most interested in? So we can implement those in advance.

Sure thing.

Neo4j supports Euclidean and cosine: https://neo4j.com/docs/cypher-manual/current/indexes/semantic-indexes/vector-indexes/#indexes-vector-similarity
duckdb supports dot product: https://duckdb.org/docs/test/functions/nested.html#list-functions
pinecone supports the  same 3 above: https://docs.pinecone.io/docs/indexes#distance-metrics

In short I believe that you can go pretty far with those three.

@acquamarin I'd start with cosine and then extend to Euclidean (L2) and then finally dot product, in that order. Cosine seems to be the most common metric used for similarity search in general.

If those are the two being considered out of the gate, I completely agree with cosine.

Wow!  Very impressed.

one week from request to implementation? Just unbelievable :-D
Many thanks for your work!


🇩🇪 Clarify where to use In-Component guards

@PascalAOMS wasn't this added on #1084 ?

Shoot, you're right. My bad.

No worries 🙂
Thanks for helping out!


Creating an EllipseGeometry doesn't populate logically dependent fields

Describe the bug
Creating an EllipseGeometry by using the constructor EllipseGeometry (rect: Rect) creates a geometry that has default center and radii (i.e. {0, 0} and 0). Conversely, creating an empty EllipseGeometry and then manually setting the center and radii doesn't populate the bounds or Rect.
To Reproduce
In FSI, type
#r "nuget: Avalonia";;

open Avalonia
open Avalonia.Media;;

let geometry = EllipseGeometry(0.0, 0.0, 20.0, 20.0);;
geometry.Center;;
//val it: Point = 0, 0 {IsDefault = true;
//                    X = 0.0;
//                    Y = 0.0;}

geometry.RadiusX, geometry.RadiusY;;
//val float * float = (10.0, 10.0)

Expected behavior
All logically dependent fields should invalidate and re-populate when a field is mutated.
Desktop (please complete the following information):

OS: Windows
Version 0.10.18

Probably the same underlying issue as: https://github.com/AvaloniaUI/Avalonia/issues/8767


disabling pre-commit-ci error by adding empty pre-commit-config.yaml

Feature or Bugfix

CI / setting bugfix

Purpose

Disabling bellow error on https://github.com/jazzband/django-redshift-backend/pull/84

Detail

https://pre-commit.ci/

Relates

nothing

I've given up trying to silence the pre-commit ci...

@jezdez Hi, do you know anything about this error?


Weird issues on linked code with multiple editors/compilers

It seems like with opt viewer on, code highlighting gets confused and every editor highlights compilers that do not belong to itself
It happens for the hover

And for the scroll to

I can't reproduce this anymore. May have it been unintentional fixed?


HHs with memory

Add memory by providing features that relate to the feature value from a previous step

 Issue #54: HHs with memory


Add CI/CD pipeline to build a docker image

We need an automatized manner of building this image and push it to the registry as done for other components.
I propose to add a Jenkins file to build the image and replace fabric8-ui Dockerfile to make use of this new image. WDYT @aslakknutsen ?

@aslakknutsen or @joshuawilson I might need your admin privileges to set the webhook, as done for other repos.

Hopefully it's just picked up :)

done!


Install fails when no URL is provided in manifest

When trying to install a package that has no URL defined in the manifest, scoop throws the following errors:
Test-7zipRequirement : Cannot bind argument to parameter 'URL' because it is null.
At C:\Users\manmademagic\scoop\apps\scoop\current\lib\depends.ps1:80 char:78
+ ... elper 7zip) -and (Test-7zipRequirement -URL (url $manifest $arch))) {
+                                                 ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Test-7zipRequirement], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Test-7zipRequirement

Test-LessmsiRequirement : Cannot bind argument to parameter 'URL' because it is null.
At C:\Users\manmademagic\scoop\apps\scoop\current\lib\depends.ps1:83 char:84
+ ... Lessmsi) -and (Test-LessmsiRequirement -URL (url $manifest $arch))) {
+                                                 ~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidData: (:) [Test-LessmsiRequirement], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Test-LessmsiRequirement

'tzutil' doesn't support 64bit architecture!

The json schema for the manifest files states that 'version' is the only required property.
The following manifest is what I'm currently testing:
{
    "version": "1.0",
    "pre_install": "write-host 'test'"
}

The error is coming from install_deps in /lib/depends.ps1 which is calling url and in turn arch_specific in /lib/manifest.ps1

Indeed URL is required too... Meta package is not accepted now.

Duplicate of #2155
See https://github.com/lukesampson/scoop/issues/2155#issuecomment-518469677 for workaround

Just use url with blank txt file (for examle).

Just use url with blank txt file (for examle).

I am a newbie.
There is a IOS safari bug in my developed Web project. I want to have a try to  install ios-webkit-debug-proxy to debug it.
After I use this command 'scoop install ios-webkit-debug-proxy' in the PowerShell, it throw the exception the same with top issue.
I think I get the mean how to solve , but I don't know how to write the command with url or where should I to put this blank txt file.
I am so ashamed. Sincerely wish to receive your answer. thanks again.

@perixiaowan Your issue has nothing to do with this one.
I do not have problem with application you mentioned.

Open separate issue in extras bucket if you encounter issues with specific application.

hi,thank you for your reply.
here is my PowerShell screenshot.

maybe it would be usefull for me to express this problem clearly .
really appreiciate it that waste your time to relay.

ios-webkit-debug-proxy only works on 64bit machine but you are using a 32bit machine...

scoop install 7zip ios-webkit-debug-proxy
If this do not help open separate issue in https://github.com/lukesampson/extras to not populate this discussion with offtopic threads.

thanks very much .  I think I the wrong environment (powershell x86). when I changed to powershell 64bit, it works!
You are soooooooo kind. thanks !!!!

ios-webkit-debug-proxy only works on 64bit machine but you are using a 32bit machine...

thank you very very much . this method solved my problem. ^(oo)^

Duplicate of #2155
See #2155 (comment) for workaround

Thanks for the link, I did have a search before logging, but couldn't find anything.