Add NetBSD installation

Hi,
I've just packaged moccasin for NetBSD and merged the package into the main branch.
Commit log, for reference: http://mail-index.netbsd.org/pkgsrc-changes/2023/09/03/msg281899.html
Please consider adding install instructions to the README.md
I got a few warnings building with Rust-1.71.1, leaving those here.
   Compiling moccasin v0.1.2 (/usr/pkgsrc/wip/moccasin/work/moccasin-0.1.2)
warning: unused import: `SortOrder`
 --> src/repo/storage.rs:1:29
  |
1 | use crate::config::{Config, SortOrder};
  |                             ^^^^^^^^^
  |
  = note: `#[warn(unused_imports)]` on by default

warning: unused import: `Item`
 --> src/repo/storage.rs:2:25
  |
2 | use crate::feed::{Feed, Item};
  |                         ^^^^

warning: unused variable: `red`
   --> src/config/theme.rs:133:13
    |
133 |         let red = make_color("#850908");
    |             ^^^ help: if this is intentional, prefix it with an underscore: `_red`
    |
    = note: `#[warn(unused_variables)]` on by default

warning: unused variable: `fragment`
  --> src/ui/detail.rs:31:17
   |
31 |     fn from_dom(fragment: Dom) -> Self {
   |                 ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_fragment`

warning: enum `StorageRequest` is never used
  --> src/repo/repo.rs:23:6
   |
23 | enum StorageRequest {
   |      ^^^^^^^^^^^^^^
   |
   = note: `#[warn(dead_code)]` on by default

warning: function `centered_rect_sized` is never used
   --> src/ui/mod.rs:217:4
    |
217 | fn centered_rect_sized(width: u16, height: u16, r: Rect) -> Rect {
    |    ^^^^^^^^^^^^^^^^^^^

warning: function `get_line_offset` is never used
 --> src/ui/detail.rs:7:4
  |
7 | fn get_line_offset(line_width: u16, text_area_width: u16, alignment: Alignment) -> u16 {
  |    ^^^^^^^^^^^^^^^

warning: struct `HTML` is never constructed
  --> src/ui/detail.rs:15:8
   |
15 | struct HTML<'a> {
   |        ^^^^

warning: associated items `from_dom`, `block`, `style`, `wrap`, `scroll`, and `alignment` are never used
  --> src/ui/detail.rs:31:8
   |
30 | impl<'a> HTML<'a> {
   | ----------------- associated items in this implementation
31 |     fn from_dom(fragment: Dom) -> Self {
   |        ^^^^^^^^
...
45 |     pub fn block(mut self, block: Block<'a>) -> HTML<'a> {
   |            ^^^^^
...
50 |     pub fn style(mut self, style: Style) -> HTML<'a> {
   |            ^^^^^
...
55 |     pub fn wrap(mut self, wrap: Wrap) -> HTML<'a> {
   |            ^^^^
...
60 |     pub fn scroll(mut self, offset: (u16, u16)) -> HTML<'a> {
   |            ^^^^^^
...
65 |     pub fn alignment(mut self, alignment: Alignment) -> HTML<'a> {
   |            ^^^^^^^^^

warning: unused `Result` that must be used
   --> src/app.rs:543:17
    |
543 |                 self.config.add_feed_url(&url);
    |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
    = note: `#[warn(unused_must_use)]` on by default
help: use `let _ = ...` to ignore the resulting value
    |
543 |                 let _ = self.config.add_feed_url(&url);
    |                 +++++++

warning: unused `Result` that must be used
   --> src/app.rs:550:21
    |
550 |                     self.config.remove_feed_url(&url);
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
550 |                     let _ = self.config.remove_feed_url(&url);
    |                     +++++++

warning: unused `Result` that must be used
   --> src/app.rs:551:21
    |
551 |                     self.repo.remove_feed_url(&url);
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
551 |                     let _ = self.repo.remove_feed_url(&url);
    |                     +++++++

warning: unused `Result` that must be used
  --> src/repo/repo.rs:77:17
   |
77 |                 tx.send(RepositoryEvent::Refresh);
   |                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
   |
77 |                 let _ = tx.send(RepositoryEvent::Refresh);
   |                 +++++++

warning: unused `Result` that must be used
  --> src/repo/repo.rs:99:21
   |
99 |                     self.storage.write_all(&feeds);
   |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
   = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
   |
99 |                     let _ = self.storage.write_all(&feeds);
   |                     +++++++

warning: unused `Result` that must be used
   --> src/repo/repo.rs:100:21
    |
100 |                     self.app_tx.send(RepositoryEvent::RetrievedAll(feeds));
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
100 |                     let _ = self.app_tx.send(RepositoryEvent::RetrievedAll(feeds));
    |                     +++++++

warning: unused `Result` that must be used
   --> src/repo/repo.rs:104:21
    |
104 |                     self.storage.write_one(&feed);
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
104 |                     let _ = self.storage.write_one(&feed);
    |                     +++++++

warning: unused `Result` that must be used
   --> src/repo/repo.rs:105:21
    |
105 |                     self.app_tx.send(RepositoryEvent::RetrievedOne(feed));
    |                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
105 |                     let _ = self.app_tx.send(RepositoryEvent::RetrievedOne(feed));
    |                     +++++++

warning: unused `Result` that must be used
   --> src/repo/repo.rs:126:13
    |
126 |             app_tx.send(RepositoryEvent::Aborted);
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: this `Result` may be an `Err` variant, which should be handled
help: use `let _ = ...` to ignore the resulting value
    |
126 |             let _ = app_tx.send(RepositoryEvent::Aborted);
    |             +++++++

warning: `moccasin` (lib) generated 18 warnings (run `cargo fix --lib -p moccasin` to apply 4 suggestions)
    Finished release [optimized] target(s) in 7m 27s

Thanks!

Appreciate the contribution! Let me give it a spin on a vm real quick before I merge.
There's a lot that is still incomplete. I have yet to implement tagging, favorites, and unread tracking as well as some basic error handling around the net and db code -- hence all the warnings about unused results. Expect some changes this week.

Looks good. CI workflow can't run from a fork, but since this is just a docs update I'm skipping.

@rektdeckard Thanks!


project-tree service

it is about to move https://github.com/kiegroup/build-chain-configuration-reader/blob/main/src/lib/project-tree.js to typescript
remember not to user typedi and just follow singleton pattern
Move also https://github.com/kiegroup/build-chain-configuration-reader/blob/main/src/lib/helper/definition-interpreter.js methods inside the project-tree as private methods

Opening a separate issue for getOrderedList* methods due to the confusion about its behavior


Apply styling to index.html

Please apply some starter text/styling to index.html for the deals section of the site.

Issue resolved due to landing.html creation. See notes there.


Osmo delegations being categorized as lp rewards

probably needs more description to be acted up.  Can DM or email specific transaction also.

Sure, no problem:
https://www.mintscan.io/osmosis/txs/C365BACB328AF4C04E862D2753EE00FD91A7BD30DD730A66200FE7AA266479EE
The csv is outputing both of these as "lp_reward" and rightly marking that as "staking" or "interest".
In this case, the 18 OSMO is the LP delegation, and the 1.8 OSMO is the LP reward.

Hm, where are you seeing the "lp_reward"?
My inclination is to say for this transaction, I see:

one csv row with 1.5 OSMO claimed rewards (from staking)

I'm guessing lp_reward rows you see is from actually separate LP rewards?  I'll double check the large csv job for this address when this finishes.  Let me know if you see a txid for the lp_reward row--that would help me debug or clarify the issue.

It's producing 2 rows. It's miscategorizing the delegation as an lp_reward. The actual transaction is on the reward while the delegation has the wallet address concatenating the date and asset.

10/26/21 17:43
STAKING
1.518334
OSMO

C365BACB328AF4C04E862D2753EE00FD91A7BD30DD730A66200FE7AA266479EE-0
https://mintscan.io/osmosis/txs/C365BACB328AF4C04E862D2753EE00FD91A7BD30DD730A66200FE7AA266479EE
osmosis_blockchain
osmo1jlp2jwj74ltvchlfj4mffkp2fg6mru4k62wr25

10/26/21 17:00
STAKING
18.831292
OSMO

lp_reward
osmo1jlp2jwj74ltvchlfj4mffkp2fg6mru4k62wr25.2021-10-26.OSMO

osmosis_blockchain
osmo1jlp2jwj74ltvchlfj4mffkp2fg6mru4k62wr25

Oi okay, thanks for the details.  Verified the error.
Looks like it's not an issue with the staketax parser but the source data--I'll notify the data provider team.

Okay, the data provider team got back to me.  He astutely pointed out to me that in this address, it's consistent:
10/26/21 17:00:00: user received 18.831 OSMO rewards from LP pool
10/26/21 17:43:33: user delegates 18.831 OSMO (and claims 1.518334 staking rewards)
So user just happens to exactly delegate (LP autocompound) the LP rewards he/she received 0-2 hours prior.

There is no LP reward listed on the block explorer that I can find at 17:00. 18 OSMO is a massive LP reward--far greater than the amounts LP-ing in this wallet. Could it be a staking reward?
https://www.mintscan.io/osmosis/txs/C365BACB328AF4C04E862D2753EE00FD91A7BD30DD730A66200FE7AA266479EE
The delegation of the 18 OSMO auto-claims a smaller amount of 1.5 OSMO which is then immediately delegated itself, but I see nothing on the explorer showing this 18 OSMO ever doing anything but being delegated in this time period.

I got this message from the Osmosis Discord channel:

LP rewards are minted via an on-chain module directly to your wallet. Because this is a module transaction the block explorers currently utilized in the ecosystem cannot read the block event associated with these rewards. A native solution is being developed to help you track your LP rewards, but in the meantime manual tracking remains your best option.

So it seems it's not available on-chain, at least directly from the wallet. It's operating off its own module. Woof.

Yup


Broker health check with TopicVersion.V2 failed

1、Broker health check failed with TopicVersion.V2 but it's ok with TopicVersion.V1, The failed reason is:
2021-12-27T13:21:17,350+0800 [pulsar-io-4-71] WARN  org.apache.pulsar.client.impl.BinaryProtoLookupService - [persistent://pulsar/hidden-ip:8080/healthcheck] failed to get Partitioned metadata : {"errorMsg":"Policies not found for pulsar/hidden-ip:8080 namespace","reqId":1293596554348340272, "remote":"", "local":""}
org.apache.pulsar.client.api.PulsarClientException$BrokerMetadataException: {"errorMsg":"Policies not found for pulsar/hidden-ip:8080 namespace","reqId":1293596554348340272, "remote":"", "local":""}
        at org.apache.pulsar.client.impl.ClientCnx.getPulsarClientException(ClientCnx.java:1154) ~[org.apache.pulsar-pulsar-client-original-2.10.0-SNAPSHOT.jar:2.10.0-SNAPSHOT]
        at org.apache.pulsar.client.impl.ClientCnx.handlePartitionResponse(ClientCnx.java:607) [org.apache.pulsar-pulsar-client-original-2.10.0-SNAPSHOT.jar:2.10.0-SNAPSHOT]
        at org.apache.pulsar.common.protocol.PulsarDecoder.channelRead(PulsarDecoder.java:130) [org.apache.pulsar-pulsar

Policies not found for namespace  pulsar/host:8080 under  znode admin/policies.
But Why v1 is successful?  v1's healthcheck topic name is global, checkReplication is skipped.
I cannot decide the best solution to fix it.

@gaozhangmin Can you reproduce the issue on 2.8.x or 2.9.1? We are enabled health checks for many clusters but never found such an anomaly.

@gaozhangmin Can you reproduce the issue on 2.8.x or 2.9.1? We are enabled health checks for many clusters but never found such an anomaly.

Yes, i tested with  2.9.1.


What license is this repo published under?

Thanks for the great works. May I ask which license is the source code published under? I want to use the code in my applications but unfortunately I cannot unless it's provided under a permissive license.
Thanks

Thanks for bringing this to my attention :)
Added an Apache 2 LICENSE file to the repo.


[Extensions] Core-side API to get auth token for service account

To implement service accounts, it needs to be possible to fetch the corresponding auth token from the Security Plugin. This means that core should be able to use a transport API to request the auth token corresponding to the service account. The auth token will contain information relating to the account's roles and identity.
The completion of this issue will be a PR that adds a transport API to core which can connect to the Security plugin and request an auth token for a given service account. The generation of the token and the plugin-side handling of the token are outside the scope of this issue. This issue is best completed as a follow-up to #2611, which adds the base Security Plugin side of the API functionality.

This is being addressed in https://github.com/opensearch-project/OpenSearch/pull/7452 since we can implement the interface in the Security Plugin and then use it to fetch the tokens for the extensions.


different results for gpu/cpu

Hi,
Here is a way to improve the performance for transformer model using gpu: in eval.py, line 182, change 'cap_len = torch.LongTensor([52]).repeat(k, 1).to(device)' to 'cap_len = torch.LongTensor([52]).repeat(k, 1)'.
From the transformer.py, the caption length is sorted, but the sorting results for gpu and cpu is different when cap_len is initialized as  torch.LongTensor([52]).repeat(k, 1).

Hi,
Thank you for your advice.
That's weird.. Do u know why the sorted results from GPU and CPU are different?

Hi,
Here is a way to improve the performance for transformer model using gpu: in eval.py, line 182, change 'cap_len = torch.LongTensor([52]).repeat(k, 1).to(device)' to 'cap_len = torch.LongTensor([52]).repeat(k, 1)'.
From the transformer.py, the caption length is sorted, but the sorting results for gpu and cpu is different when cap_len is initialized as torch.LongTensor([52]).repeat(k, 1).

Hi,
Thank you for your advice.
That's weird.. Do u know why the sorted results from GPU and CPU are different?

Hi,
Here is a way to improve the performance for transformer model using gpu: in eval.py, line 182, change 'cap_len = torch.LongTensor([52]).repeat(k, 1).to(device)' to 'cap_len = torch.LongTensor([52]).repeat(k, 1)'.
From the transformer.py, the caption length is sorted, but the sorting results for gpu and cpu is different when cap_len is initialized as torch.LongTensor([52]).repeat(k, 1).

Hi,
Thank you for your advice.
That's weird.. Do u know why the sorted results from GPU and CPU are different?

I'm not sure. I just looked for the solution to get same results and find this issue:
caption_lengths = torch.LongTensor([52]).repeat(5, 1).cpu()
caption_lengths, sort_ind = caption_lengths.squeeze(1).sort(dim=0, descending=True)
sort_ind -> tensor([0, 1, 2, 3, 4])
caption_lengths = torch.LongTensor([52]).repeat(5, 1).cuda()
caption_lengths, sort_ind = caption_lengths.squeeze(1).sort(dim=0, descending=True)
sort_ind -> tensor([1, 0, 2, 4, 3], device='cuda:0')
They should be the same, but unfortunately, no. I also feel confused about why this modification would change the results that much. Based on my experiment results(trained on gpu and evaluated on gpu), I can get BLEU-4 0.3145 on coco dataset test split, which is more acceptable than before.

Hi,
Here is a way to improve the performance for transformer model using gpu: in eval.py, line 182, change 'cap_len = torch.LongTensor([52]).repeat(k, 1).to(device)' to 'cap_len = torch.LongTensor([52]).repeat(k, 1)'.
From the transformer.py, the caption length is sorted, but the sorting results for gpu and cpu is different when cap_len is initialized as torch.LongTensor([52]).repeat(k, 1).

Hi,
Thank you for your advice.
That's weird.. Do u know why the sorted results from GPU and CPU are different?

I'm not sure. I just looked for the solution to get same results and find this issue:
caption_lengths = torch.LongTensor([52]).repeat(5, 1).cpu()
caption_lengths, sort_ind = caption_lengths.squeeze(1).sort(dim=0, descending=True)
sort_ind -> tensor([0, 1, 2, 3, 4])
caption_lengths = torch.LongTensor([52]).repeat(5, 1).cuda()
caption_lengths, sort_ind = caption_lengths.squeeze(1).sort(dim=0, descending=True)
sort_ind -> tensor([1, 0, 2, 4, 3], device='cuda:0')
They should be the same, but unfortunately, no. I also feel confused about why this modification would change the results that much. Based on my experiment results(trained on gpu and evaluated on gpu), I can get BLEU-4 0.3145 on coco dataset test split, which is more acceptable than before.

Was the GPU captioning human readable? I think it should be normal cuz of the decent BLEU-4.
I will update the code later. Thanks.


Support ON UPDATE CURRENT_TIMESTAMP in MySQL

This PR adds support for defaultExpr="CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP" for MySQL as requested in #1203. Previously, a migration would always contain an ALTER Statement to set ON UPDATE CURRENT_TIMESTAMP although it is already like that in the database. The root cause was that the Extra part was not checked in the MysqlSchemaParser class.

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

Comparison is base (011aaa7) 88.56% compared to head (aaa827b) 67.43%.

: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    #1967       +/-   ##
=============================================
- Coverage     88.56%   67.43%   -21.13%
- Complexity     8051     8052        +1
=============================================
  Files           243      232       -11
  Lines         24590    24547       -43
=============================================
- Hits          21777    16553     -5224
- Misses         2813     7994     +5181

Flag
Coverage Δ

5-max
67.43% <0.00%> (-21.13%)
:arrow_down:

7.4
67.43% <0.00%> (-21.13%)
:arrow_down:

agnostic
67.43% <0.00%> (+0.13%)
:arrow_up:

mysql
?

pgsql
?

sqlite
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/Propel/Generator/Reverse/MysqlSchemaParser.php
0.00% <0.00%> (-88.75%)
:arrow_down:

... and 133 files 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.

@Mau04  It looks like the old maintainer of Propel2, @dereuromark, has left the project, and it looks like there is no successor at Spryker at the moment. I have contacted the owner, and he told me that the project will keep being maintained by Spryker, but I haven't heard back if/when there will be someone who actively maintains Propel2, i.e. merges this PRs.
For what it's worth, I am very sorry for the situation! This should have been merged month ago, with all the thanks and commendations that are in order. I found getting acquainted with Propel and creating a PR takes effort, and I wish your first contribution to the project had been a more rewarding experience.

Best wishes

Hello @mringler & @Mau04,
We are currently at Spryker creating a development strategy for Propel maintenance under the general product roadmap of Spryker..we have been lacking in support in the recent months of this repository and we're working diligently to fix it processually. We'll come back to this PR during the next few days and we'll handle future contribution with a better response time and higher level of contribution.
Thank you for your time and effort on the repository,
We're looking forward to further contribution!


[Level Up: Advanced 5TH Edition] dice rolling improvements

Added new Dice rolling logic, that gathers the components of the dice roll when the button is press this means that the every roll on the sheet doesn't need to be updated everytime standard/Advantage/Disdadvantage is pressed
Fixed Wisdom (perception) initiative to use Wisdom
Fixed all the Passive Checks to be 10+modifiers

Submission Checklist
Required

[Yes] The pull request title clearly contains the name of the sheet I am editing.
[Yes] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[Yes] The pull request makes changes to files in only one sub-folder.
[Yes] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Added new Dice rolling logic, that gathers the components of the dice roll when the button is press this means that the every roll on the sheet doesn't need to be updated everytime standard/Advantage/Disdadvantage is pressed
Fixed Wisdom (perception) initiative to use Wisdom
Fixed all the Passive Checks to be 10+modifiers

Character Sheet Info Roll20 Internal Use only.


Develop an example of community detection algorithms using open-source data (including examples of implementation, and interpretation of algorithm results.)

Hi, I am Moontasir, an AGE intern. I choose to work on it as my Project-2.

Hi, I am the new Operations Manager Jayden from Vancouver Team.

Hi, I am Muneeb. I am AGE intern. I choose to work on it as my Project-2.

looking forward to working on this project with everyone


use jupytext to pair notebooks

I see, the mkdocs was not working for me before adding the jupytext,
you are right, having py and md files does not make too much sense and is confusing


Growth Chambers missing plants after server restart

Description (Required)
If growth chambers are planting and server restart, plants will missing
Steps to reproduce the Issue (Required)
Use growth chambers to plant.
Server restart.
Plants missing.
Expected behavior (Required)
Should work like normal.
Server Log / Error Report
no error.
Environment (Required)
Minecraft Version: Paper 1.18.1-205
CS-CoreLib Version: none
Slimefun Version: DEV 999
Plugin Version: DEV 160

Tested - this issue stems from the use of AMachine from Slimefun which has this behaviour. While this should be changed, it'll be an enhancement, not a bug fix.


Add Pipeline Version Information to the Pipelines Tab

Why:
There is currently no documentation of pipeline versions, which might be necessary if a researcher requires this information for publication.
Acceptance Criteria: Add an overview of where to find pipeline version information to FAQ section and pipelines landing page.
@matthewspeir

Obsolete closing

Obsolete closing


Multi-Arch Support

eBPF bytecode can be compiled to bpfel or bpfeb depending on the target platform.
We should:

Package both el/eb variants of BPF bytecode in an eBPF OCI image
Ensure that bpfman is unpacking the correct artifact based on the runtime endianess
It would be great if this worked the same way as multi-arch manifests (i.e we have a meta manifest that points to the actual image manifests for the correct os/arch)

This needs to be done for all the dispatcher images that we maintain + tooling created for user images to do this right.

TODO expand into more issues :)

Decide the list of arch'es we at least need

arm64
powerpc
s390
amd64 (x86)

Since that's what's supported in https://github.com/netobserv/netobserv-ebpf-agent/tree/main/pkg/ebpf and

[ ] Update the OCI image spec used for packaging ebpf bytecode
- Package both el/eb variants of BPF bytecode in an eBPF OCI image
- Determine if there's any existing OCI specific infrastructure that can help us here
- It would be great if this worked the same way as multi-arch manifests (i.e we have a meta manifest that points to the actual image manifests for the correct os/arch)
[ ] Ensure that bpfman is unpacking the correct eBPF artifact based on the runtime endianess
[ ] Build + pacakge all userspace libraries and binaries with multi arch support
[ ] Build all eBPF bytecode images with multi arch support

Feature Complete


Map Upload files into request properties

class MyRequest implements Request
{
    use IsRequest;

    public Upload $avatar;
}

Are you sure this works? Because $from->getFiles() is an array without named keys?

Yes, we need a named keys, any idea how we can resolve that? when I tried to map the request to a model object with ArrayToObjectMapper because we couldn't find the property name with the array keys.

We need to add the uploadedFiles as well, now getFiles returns an array with named keys.

final readonly class RequestToPsrRequestMapper implements Mapper
{
   ...

    public function map(mixed $from, mixed $to): PsrRequest
    {
        /** @var Request $from */

        return new ServerRequest(
            uri: $from->getUri(),
            method: $from->getMethod()->value,
            headers: $from->getHeaders(),
            cookieParams: $from->getCookies(),
            queryParams: $from->getQuery(),
            parsedBody: $from->getBody(),
            uploadedFiles: $from->getUploadedFiles()
        );
    }
}

Continuing with the Book example after adding coverPicture property the request along with the Book model this is what I got:
INSERT INTO `Book` (title, coverPicture) VALUES (:title, :coverPicture);

bindings: {
    "title": "tittre 1",
    "coverPicture": null
}

Could you tell me which value should I store in the DB?

I think the easiest path forward is to submit a PR that I can review ;)

@brendt, the mapping between PsrRequest and Request works properly. I believe the mapping between Request and Model is also correct, but I encountered an error when trying to save the Book model using #702.
The error message is: "Object of class Tempest\Http\Upload could not be converted to string."
We can easily implement the __toString method in the Upload class to return the path. let me know if I need to create a new issue regarding the issue encountered.
What do you think about it?

@innocenzi Thank you for you reply, I understand that @brendt  doesn't work full-time in Tempest.
My intention was not to rush or an immediate action but to express that Ive been disconnected from the project despite my efforts. I just want to clarify my position and the frustration that comes with contributing without feedback.
I appreciate the work done, and I'm still open to contributing if things change.


Implementation of the Manifest Service API refactor.

Add a generic Manifest interface to represent manifests in the registry and
remove references to schema specific manifests.
Add a ManifestBuilder to construct Manifest objects. Concrete manifest builders
will exist for each manifest type and implementations will contain manifest
specific data used to build a manifest.
schema1
Change the way data is stored in schema1.Manifest to enable Payload() to be used
to return complete Manifest JSON from the HTTP handler without knowledge of the
schema1 protocol.
tags
Move tag functionality to a seperate TagService and update ManifestService
to use the new interfaces.  Implement a driver based tagService to be backward
compatible with the current tag service.
Add a proxyTagService to enable the registry to get a digest for remote manifests
from a tag.
manifest store
Remove revision store and move all signing functionality into the signed manifeststore.
manifest registration
Add a mechanism to register manifest media types and to allow different manifest
types to be Unmarshalled correctly.
client
Add ManifestServiceOptions to client functions to allow tags to be passed into Put and
Get for building correct registry URLs.  Change functional arguments to be an interface type
to allow passing data without mutating shared state.
Signed-off-by: Richard Scothern<EMAIL_ADDRESS>

This is a solid crack at the task. Most of the commentary above is clarifications.

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.

LGTM
Please squash commits though.

LGTM


Splice prediction scores missing for intronic variant

Describe the bug
Splice prediction scores missing for intronic variant
To Reproduce
Steps to reproduce the behavior:

Go to 'reev'
entervariant  as genomic chrX:129156859:C:G or cDNA level NM_001379451.1(BCORL1):c.3608-13C>G
Scroll down to 'scores'
See error

Expected behavior
mmsplice and spliceai scores should be provided
Screenshots

Root Cause Analysis
Scores currently not shown if position not in dbNSFP, although splice scores are available even then.

@holtgrewe yes, working fine now


wont memoize selectors with reselect

I've noticed that my ReSelect selectors are calling again and again even though nothing actually changes in terms of redux state.
This seems to be happening due to this line. It returns a new array every time so the default equality check that ReSelect does fails.
Is there some option/way to make it memoize correctly out of the box?
Should i write a "wrapper selector" for the search results that has a custom memoized function that checks the actual contents of the array so that the equality wont fail

Reselect should ensure that the selector you linked to should only be re-invoked itself if one of its inputs have changed (eg the search criteria or the unfiltered collection). So it's okay for it to return a new Array each time since, theoretically, anytime one of its inputs change- its output could also change.
Any chance this problem is actually a level above this selector?
If you're convinced it's a problem with this library, please provide a small repro case that I can look at. 😄

So i've created a very small app to demonstrate what happens to me.
https://github.com/slavab89/redux-search-selector
Its build with react-create-app so should be pretty easy to run npm start
resource file has the reducers + actions + selectors.
The mapping collection get initialized as the app starts and never changes
I've created a RANDOMIZE action just to be able to trigger the connect function and the call to the reducers.
As you'll be able to see in the browser console, every time you push the RandomizeNumber button, you'll see prints that connect is called (because the number changes) and the search selector getFiltered is called as well, but the getAllSelector is not called because mapping is still the same
It might be that i didnt build or configure things right therefore i see this effect happening?

Any update on this?

I'm having the same issue. The problem is that the result selector is not looking at a subset of the redux state, it's looking at the root. So if anything in the app changes, it will recompute since it's memoizing on the root.
The solution here is to compose selectors such that you have state => result => filter.

Simple fix if you introduce Reselect as a non-dev dep:
import { createSelector } from 'reselect'

//...

export function getResultSelector ({
  filterFunction,
  resourceName,
  resourceSelector,
  searchStateSelector = defaultSearchStateSelector
}): Function {
  const unfilteredResultSelector = getUnfilteredResultSelector({ resourceName, searchStateSelector })

  return createSelector(
    [
      state => unfilteredResultSelector(state),
      state => resourceSelector(resourceName, state)
    ],
    (result, resource) => result.filter(filterFunction || createFilterFunction(resource))
  )
}

I don't need the filtering logic so ended up using my own selectors since it's pretty straight forward.

@jahed I know its been a while but can you provide more insight into this? I'm running into, what I believe is, the same issue. Selectors are being called many times and components are excessively re-rendering. I understand what you're saying but i'm having some trouble determining the correct implementation.

@schester44 Going through my code, it looks like my solution was to not use getSearchSelectors (which uses the problematic getResultSelector).
Instead, I just grab the results straight from the Redux store myself and use Reselect to memoize it correctly.
Something like:
const searchResultSelector = createSelector(
  state => state,
  state => state.router.params.resourceId,
  (state, resourceId) => get(state, ['search', resourceId, 'result'])
)

Where state.search is where I assigned redux-search's reducer.

oh awesome, that makes a ton more sense. thanks!
Heres my final implementation which no longer re-renders like 5 or 6 times more than it should.
store
const enhancers = [ 	reduxSearch({ 		resourceIndexes: { 			videos: ["name"], 			kiosks: ["name"], 			categories: ["name"] 		}, 		resourceSelector: (resourceName, state) => state[resourceName].byId 	}) ]
selectors/videos
`
export const searchResults = state => state.search.videos.result
export const videoListSelector = createSelector([searchResults, isLoading, isLoaded], (videoIds, loading, loaded) => ({
videoIds,
loading,
loaded
}))
`
containers/videos
export default connect( 	videoListSelector, 	{ search: createSearchAction("videos") } )(VideosView)


default color scheme not good in matplotlib/error interactive view

Environment data

VS Code version: 1.29.0
Extension version (available under the Extensions sidebar): 2019.1.0
OS and version: linux ubuntu 18.04
Python version (& distribution if applicable, e.g. Anaconda): python 3.5.2
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): virtualenv
Relevant/affected Python packages and their versions: all

Expected behavior
seeing errors and plots good at interactive view by default
Actual behavior
not good colors by default
Steps to reproduce:

try plotting or doing and error in interactive view, like in code below

Logs

This has been fixed in more recent versions. Please update your extension.
Our latest version is 2019.4.12954


How to obtain IWebView2WebView3

IWebView2Environment::CreateWebView result in a IWebView2WebView object, where does an IWebView2WebView3 object come from.
Should we QueryInterface for it?
(Also isn't it a bit early to reach iXxxx3 before the first release?)

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

ID: 2ee94b40-dc6e-f3ce-099f-1e71489a08eb
Version Independent ID: 9a166020-1f43-7c84-c0ce-71c7633d02d8
Content: Microsoft Edge WebView2 for Win32 apps - Microsoft Edge Development
Content Source: microsoft-edge/hosting/webview2/reference/IWebView2WebView3.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Thanks for the feedback! We'll get back to your feedback faster if you file it on our https://github.com/MicrosoftEdge/WebViewFeedback/issues page.
Yes you QueryInterface for it. I've opened https://github.com/MicrosoftEdge/WebViewFeedback/issues/52 to update our documentation.
For the first release we'll have a breaking change and squash all the versioned interfaces down to one interface. Eg. Everything in IWebView2WebViewN will be moved into IWebView2WebView.

OK, thank you.


Light and dark colors are not calculated for custom palette color

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
There are following lines in MUI documentation  -

If the "dark" and / or "light" keys are omitted, their value(s) will be calculated from "main", according to the "tonalOffset" value.
If "contrastText" is omitted, its value will be calculated to contrast with "main", according to the "contrastThreshold" value.

In fact, light, dark and contrastText props are not existed in theme.palette if they were omitted in definition -
https://codesandbox.io/s/nostalgic-turing-055xhr?file=/src/App.tsx
Auto calculation works only for inbound colors like primary and secondary. For custom colors it doesn't work.
Current behavior 😯
light, dark and contrastText props are not calculated.
Expected behavior 🤔
light, dark and contrastText should be calculated or documentation should be fixed if this function has been removed.
Context 🔦
No response
Your environment 🌎
No response

not a mui member but i use this sometimes
  const refTheme = createTheme()
  const theme = createTheme({
    palette: {
      custom: refTheme.palette.augmentColor({ color: { main: '#f0f' } })
    }
  });

this generates the dark/light/contrastText for your custom key. not sure if there is a better way
https://codesandbox.io/s/nifty-dream-9jz32g
the docs are correct in some aspect that you can pass a string to "primary":{"main":"stringhere"} but other things need that augmenting

This is what I tried to ask with #35707. But I still would much prefer if MUI/Joy included some function that would calculate these colors properly for us.

I think Material UI should expose the internal createPalette function:
import { createTheme, createPalette } from '@mui/material/styles';

createTheme({
  palette: {
    custom: createPalette({ main: '...' }),
  }
})

I'm also having an issue with this and totally agree, either the documentation needs updating or this is a bug! Please fix! Very frustrating. Thank you.

Yes running into this also

Similar to what @cmdcolin said, but manipulating the final theme instead of creating 2 theme objects:
const muiTheme = createTheme({
  // ...
  palette: {
    custom: {
      main: '#332211',
    },
  },
});

const theme = {
  ...muiTheme,
  palette: {
    ...muiTheme.palette,
    custom: muiTheme.palette.augmentColor({
      color: muiTheme.palette.custom,
      name: 'custom',
    }),
  },
};

It's probably a bit tricky to do it automatically since palette contains different shapes inside. For example, the shape of primary, secondary, success, etc. is different from the one of text, background, common and action.
Maybe createTheme could augment it automatically only when it finds a key main in the object.

I'll start working on improving the experience by:

Updating the docs
Exposing createPalette, @siriwatknp are you still on board with doing so?

PR is up: https://github.com/mui/material-ui/pull/37743
Closing as duplicate of #29595, updates will be posted in that issue 😊


Sholl analysis is performing a lot of unnecessary computations

The results of Sholl analysis return of a lot of zeros. I think this is because the bounding box encloses the whole neuron while it should enclose only the trees of a specific neurite_type on which the computation is actually performed. See the following line:
https://github.com/BlueBrain/NeuroM/blob/49dbaef7ed78c351d8adb3085843010716afa217/neurom/fst/_neuronfunc.py#L201

#669 should fix it.

I don't think #669 fixes it, but it's probably something similar.
I think we'll have to make a fake neuron to test this on, so we can be assured that we're fixing the problem.


> /nix/store/iwxsnv263d6gmqrs9wxpzm6l4p7cdz6j-glibc-2.38-44-dev/include/features.h:292: note: this is the location of the previous definition
       >   292 | # define _POSIX_C_SOURCE        200809L
       >       |
       > /nix/store/g39lzlrdmpaqhi78gvxad5dqqkdrxmc1-python-<IP_ADDRESS>/include/python2.7/pyconfig.h:1311: warning: "_XOPEN_SOURCE" redefined
       >  1311 | #define _XOPEN_SOURCE 600
       >       |
       > /nix/store/iwxsnv263d6gmqrs9wxpzm6l4p7cdz6j-glibc-2.38-44-dev/include/features.h:216: note: this is the location of the previous definition
       >   216 | # define _XOPEN_SOURCE  700
       >       |
       > OK _test/unit/pyext/libc_test.py.log
       >   (pyext/setup_posix.py) -> posix_.so
       > OK _test/unit/pyext/posix_test.py.log
       >   (asdl_main c) -> _gen/frontend/types.asdl_c.h
       >   (frontend/consts_gen c) -> _gen/frontend/id_kind.asdl_c.h
       >   (lexer_gen) -> _build/tmp/frontend/match.re2c.txt
       > build/py.sh: line 289: /nix/store/y6jvbxq84s6sj3453zb1rzh60gfj9bqh-re2c-aarch64-unknown-linux-gnu-1.0.3/bin/re2c: cannot execute binary file: Exec format error
       > /nix/store/yq9qag5ak3w15nhcqplnhxpny6295l36-stdenv-linux/setup: line 131: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/h08yllwbqr0xx6d9k3x1ah97fq5c7j4p-python2.7-oildev-unstable-2021-07-14.drv'.
error: 1 dependencies of derivation '/nix/store/794yk8ynmizfr43ad5sa93q2nwhrn3lw-resholve-0.9.1.drv' failed to build
error: 11 dependencies of derivation '/nix/store/pi9rplnq55ks17xcpm4cvxs55ab7hwq5-xdg-utils-aarch64-unknown-linux-gnu-1.2.0.drv' failed to build

the removal of mimetype seems to degrade some mime associations. i notice this with one of my PDFs:
$ xdg-mime query filetype ./my.pdf
application/octet-stream
$ PATH=$PATH:/nix/store/r1p31hqim9yx1iw916cp5hzc395k3za9-perl5.38.2-File-MimeInfo-0.33/bin xdg-mime query filetype ./my.pdf
application/pdf

the mime definition for pdf is on my system, at /run/current-system/sw/share/mime/application/pdf.xml. i guess file --mime-type maybe doesn't read those.
anyway, watch out for this. this affects xdg-open as well, naturally. if it's replicable, might be worth bringing back mimetype.

Interesting. Can you post the PDF? I would guess that it's not entirely valid and gets misdetected.

Interesting. Can you post the PDF? I would guess that it's not entirely valid and gets misdetected.

unfortunately, no, it's a bank statement. i just tested about 30 PDFs from the public internet and they all identify correctly. on the other hand, i tested another 3 pdfs from my bank over the years, and they all mis-identify without mimetype. so they're probably generating these with some not-100%-compliant tool.

Are they maybe not generating the %PDF-x.y header at the start of the file?

quite right: they have an additional metadata field before the %PDF-x.y header. like this:
%%E MEDIA: PDFOUT:<removed>
%PDF-1.3

if i add that extra %%E ... line to another, good pdf, that's enough to cause file to not identify the pdf even while evince opens it fine. i suppose i should check the issue tracker for file then.

That is in fact out of spec.

The first line of a PDF file shall be a header consisting of the 5 characters %PDF- followed by a version number of the form 1.N, where N is a digit between 0 and 7.

https://opensource.adobe.com/dc-acrobat-sdk-docs/pdfstandards/PDF32000_2008.pdf, section 7.5.2

the addition of xdg-user-dirs breaks aws-vault on macOS too.

Pretty sure that one also needs the wrapper to be made conditional on the OS, though I actually can't find any references to "xdg" in the upstream source at all.

@kradalby fix for xdg-user-dirs was merged a few hours ago:

https://github.com/NixOS/nixpkgs/pull/288313

I override xdg-open with mimeo, but it doesn't build after these changes, any hints?
error: builder for '/nix/store/9n5q34m3yriy55yldzx22arjfpfp9bnk-xdg-utils-1.2.1.drv' failed with exit code 5;
       last 10 log lines:
       > [resholve context] RESHOLVE_INPUTS=/nix/store/8xanhmwi20ikas8mf0map1kjza0g78i0-dbus-1.14.10/bin:/nix/store/mb488rr560vq1xnl10hinnyfflcrd51n-coreutils-9.4/bin:/nix/store/320v66ili0mwnyrxj3dwbxm0z8ndkbw7-gnugrep-3.11/bin:/nix/store/q7kq0naays5251ihghw0ccsz39id7kk5-gnused-4.9/bin:/nix/store/vh06xwq3x81n0500z16r6yiixgrchkk8-net-tools-2.10/bin:/nix/store/25k31gp9q0479awkb5z8qjl88j96pk75-glib-2.78.4-bin/bin:/nix/store/n557l343qfrwkzljfqdv30y7k1g1q40m-xdg-utils-1.2.1/bin
       > [resholve context] RESHOLVE_INTERPRETER=/nix/store/4vzal97iq3dmrgycj8r0gflrh51p8w1s-bash-5.2p26/bin/bash
       > [resholve context] RESHOLVE_KEEP='$KDE_SESSION_VERSION $browser env:$command'
       > [resholve context] /nix/store/af6r4a8smq7j3szi0wfsjawb8h12nb37-resholve-0.9.1/bin/resholve --overwrite bin/xdg-open
       >   exec -a "$0" "/nix/store/c6ny37x7yk6hy3hd9217silmjlh90b70-mimeo-2023/bin/.mimeo-wrapped"  "$@"
       >                ^
       > '/nix/store/n557l343qfrwkzljfqdv30y7k1g1q40m-xdg-utils-1.2.1/bin/xdg-open':15: Unexpected absolute command path.
       >
       > Next step: keep, fix, or pre-patch/substitute it.
       > /nix/store/v099hqvw5z87423p4hz1vfhzaqa07dii-stdenv-linux/setup: line 131: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/9n5q34m3yriy55yldzx22arjfpfp9bnk-xdg-utils-1.2.1.drv'.

Here's how I override it to then add to home.packages:
{
  xdg-mimeo = pkgs.xdg-utils.overrideAttrs (old: {
    postInstall = ''
      cp ${pkgs.mimeo}/bin/mimeo $out/bin/xdg-open
    '' + old.postInstall;
  });
}

Doesn't seem different than the current mimiSupport, but I'm wondering if anyone is using that with these changes 🤔

@gvolpe looks like mimiSupport is also failing. mimiSupport works if i change it to use postFixup instead of postInstall: try the same with mimeo?

@uninsane that worked, thanks!


Broken: wine-staging

Package name
wine-staging
Version
9.8??  whatever rpk pulled as the current one
Pacstall version
5.4.1 Telemagenta
Relevant log output
$ rpk install wine
Found packages matching 'wine':

[0]: carla-bridge-wine64 (apt)
[1]: fonts-wine (apt)
[2]: gnome-wine-icon-theme (apt)
[3]: libkwineffects14 (apt)
[4]: librust-winapi-dev (apt)
[5]: libwine (apt)
[6]: libwine-dev (apt)
[7]: q4wine (apt)
[8]: shiki-wine-theme (apt)
[9]: twine (apt)
[10]: wine (apt)
[11]: wine-binfmt (apt)
[12]: wine-stable (apt)
[13]: wine64 (apt)
[14]: wine64-preloader (apt)
[15]: wine64-tools (apt)
[16]: winetricks (apt)
[17]: wine32 (apt)
[18]: wine32-preloader (apt)
[19]: wine32-tools (apt)
[20]: carla-bridge-wine32 (apt)
[21]: winetricks-git (pacstall)
[22]: wine-staging-git (pacstall)

Select which package to install [0-22]: 22
Selecting 'wine-staging-git' from package manager 'pacstall'
Are you sure?  Done
Building dependency tree... Done
Reading state information... Done
Note, selecting 'wine-staging-git-dummy-builddeps' instead of '/usr/src/pacstall/wine-staging-git-dummy-builddeps_9.8-pacstall1~git5eeb4b76_all.deb'
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 deluge-common : Depends: python3-openssl but it is not going to be installed
                 Depends: python3-rencode but it is not going to be installed
                 Depends: python3-twisted but it is not going to be installed
                 Depends: python3-mako but it is not going to be installed
                 Depends: python3-zope.interface but it is not going to be installed
 deluge-gtk : Depends: python3-gi but it is not going to be installed
              Depends: python3-gi-cairo but it is not going to be installed
 deluged : Depends: python3-libtorrent but it is not going to be installed
	[!] ERROR: Failed to install wine-staging-git-dummy-builddeps deb
[+] INFO: Cleaning up
[!] ERROR: Failed to install build or check dependencies
[+] INFO: Cleaning up

rpk install wine

What is this command ?
What distro are you using ?


Bugfix/mpi one sided with buffers

New PR to replace #49 .

Pull Request Test Coverage Report for Build 743316479

7 of 14 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 74.748%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/RMPI/onesided.jl
7
14
50.0%

Totals

Change from base Build 739814812:
0.07%

Covered Lines:
2075

Relevant Lines:
2776

💛  - Coveralls


raftstore/apply: skip notification for synchronous destroy

cherry-pick https://github.com/tikv/tikv/pull/6004 to release 3.1
Signed-off-by: Liqi Geng<EMAIL_ADDRESS>What have you changed?
No need to send destroy reply to peerfsm if it will be destroyed synchronously. If not, panic will happen because peerfsm is destroyed twice.
It rarely happens because a destroy can do synchronously only when an applying snapshot is in the pending state. Also, when applying a snapshot, it is not possible to merge or conf change, the other choices are rare, e.g. peer id in msg is bigger than itself, which needs two conf change.
What is the type of the changes?

Bugfix (a change which fixes an issue)

How is the PR tested?

Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/issues/5365

/run-all-tests


added fake demo view

addresses #62

Coverage decreased (-0.2%) to 80.123% when pulling a3080ac0ee8b84bb5784a3d69f2d376b5853607a on secure_pdf_demo_view into 9949ceaccae3e45c976bd208543d9b3df808b088 on master.


How to pass initialProps to a StackNavigator screen?

Desired behaviour
I'm using one component to display different content based on the initialProps. Think list with 2 data sources that depend on initialProps.
I have a RCTRootView. I modify it's appProperties. Docs say that the component re-renders with new appProperties in this.props on change.
init(bridge: RCTBridge) {
    // init root view without initial props
    self.rootView = RCTRootView.init(bridge: bridge, moduleName: "Foo", initialProperties: nil)
}

// modify app properties later
self.rootView.appProperties = ["listType" : "A"]

The console prints
Running application Foo ({
    initialProps =     {
        listType = A;
    };
    rootTag = 1;
})

But the screen doesn't receive any properties and the component doesn't re-render. I put a breakpoint inside componentWillUpdate to check. Any idea where the props are going and how to access them? They aren't being passed inside navigation state either.
Workarounds
Lazy loaded component
I recall that there was a lazy way to load components getScreen: () => ReactComponent. It might also be too slow to lazy load the component, ideally it's pre-loaded b/c React Native can take some time to render.
const listType
const subscription = notifierEmitter.addListener(eventName, (notification) => {
  listType = notification.listType // <- before navigating, native notifies JS about the list type
})

const Foo = StackNavigator({
  List: {
    getScreen: () => () => <List listType = { listType }/>,
    path: ':demoListType',
  },
})

This works, but:

The performance is wack
Works ~90% of the time - it's unreliable
The list is jumpy
You can sometimes see previous list type being rendered before the new one appears

I'd like the appProperties be passed to props in the screen.
Better would be if they're passed with navigation as state so that they can be used in navigationOptions for header titles, deep linking, etc.

@satya164 @grabbou can you tell me where are the appProperties being passed? Thank you.

They are being passed to NavigationContainer

which passes them inside render() to createNavigator View component

Ok, got a temporary hack in my fork. I check for the route of the component that I want to pass the prop to, and modify the navigation state params.
It would be nice to have this supported by default so I'll try thinking of a way to PR this. In any case, I'd love to hear your ideas on how this could be added.

False alarm, the component won't re-render on appProperties change b/c react navigation doesn't refresh state on AppRegistry JS call when runApplication is called with new initialProps.
After all this I'm forced to use duplicate StackNavigators with the exact same routes and components but with different AppRegistry names. Same for UIViewControllers and the RCTRootViews.
const Foo = StackNavigator({
  List: {
    screen: List,
    path: ':demoListType',
  }, // ... paths
})

AppRegistry.registerComponent('A', () => Foo);
AppRegistry.registerComponent('B', () => Foo);

It sounds like you've found a workaround.
To my understanding, this is more of a RN bug than a problem with the nav library. RN makes it easy to provide initial props from native to the root react component, but it is not easy to update the props, so you'd need a better abstraction for that.

@ericvicenti What??? the problem is clearly with the library - I still have to utilize a complete wack hack.
The library is messing with basic React Native functionality; my workaround is a total hack of something that should be supported by default. The navigation state be kept up to date on through multiple App restarts.
Once again:

The app's initial props are not passed to the topmost stack component (initial route)
The navigation state is not updated on App restart (ie. on modification of the appProps)

Ping @ericvicenti

Ping @ericvicenti

Hi, I'm not sure why you're calling AppRegistry.registerComponent twice in your app, and that is what confused me.
Have you tried using the screenProps feature? It is documented here: https://reactnavigation.org/docs/navigators/stack#Navigator-Props
const Foo = StackNavigator({
  List: {
    screen: List,
    path: ':demoListType',
  }, // ... paths
});

const List = ({ screenProps }) => <MyListView foo={screenProps.bar} />;

// first, you can render:
<Foo screenProps={{ bar: true }} />;

// then you can re-render the navigator and the new screen prop will flow down to MyListView:
<Foo screenProps={{ bar: false }} />;

@ericvicenti calling AppRegistry.registerComponent twice is used for two different RCTRootView inits which have different initialProps. Docs
screenProps doesn't solve anything. The props I need are passed to the root component registered. (StackNavigator)
app registered -> init root view module with initialProps: ["key":"value"] initialized -> props get lost inside the StackNavigator as already described above -> initial screen component can't access initial props

use screenProps
this.props. screenProps

As already mentioned by others, the way you are doing it:
const Foo = StackNavigator({
  List: {
    getScreen: () => () => <List listType = { listType }/>,
    path: ':demoListType',
  },
})

is not going to work.
You should instead:
const Foo = StackNavigator({
  List: {
    screen: List,
    path: ':demoListType',
  },
});

class First extends React.Component {
   render() {
     // read from `initial props`
     return <Foo screenProps={{ listType: this.props... }} />
  }
}

AppRegistry.registerComponent('FirstComponent', () => First);

@grabbou @ferologics I still can't access the initialProps with how you have suggested setting up the app.
I'm not sure if we're doing something wrong. But you have closed this and the other issue that I made regarding the same thing (#634). You seem to be really confident that this is doable so please let us know what your secret here to accessing initialProps using this navigation library.
Thanks!

@grabbou @ferologics
And if you could please re-open this issue until one of us has confirmed that it's been resolved, that would be great. Thanks.

Okay, finally understood the approach you were taking and managed to get it working. To help others, here is a full solution:
import React from 'react';

import {
  AppRegistry,
  Text,
} from 'react-native';

import { StackNavigator } from 'react-navigation';

class MyApp extends React.Component {
  render() {
    console.log('this.props in MyApp', this.props); // This will list the initialProps.

    // StackNavigator **only** accepts a screenProps prop so we're passing
    // initialProps through that.
    return <Navigator screenProps={this.props} />;
  }
}

class HomeScreen extends React.Component {
  render() {
    console.log('this.props in HomeScreen', this.props);
    // This will output something like this:
    // { screenProps: { ...your initialProps }, navigation: { ...StackNavigator stuff... } }

    return <Text>My initialProps are {JSON.stringify(this.props.screenProps)}.</Text>;
  }
}

const Navigator = StackNavigator({
  Home: { screen: HomeScreen },
});

AppRegistry.registerComponent('MyApp', () => MyApp);

Thanks and I hope that helps clarify for others, including @ferologics.

Hey, after this issue, I decided that React Navigation is performing extremely poorly in my production app and I decided to go with native iOS navigation + a bunch of RCTRootViews.
((( why I decided to do so:

performance problems were with the navigation animations (pop, push were laggy)
popping the entire screen stack resulted in an unexpected animation which caused confusion to many users.
#729

After testing with couple users they all concurred that the navigation is unlike the native one. Overall less responsive UI and there were subtle abovementioned animation differences that influenced the UX. )))
@grabbou and @ericvicenti are totally right, I should've used the screenProps as suggested in above examples.
My error was in expecting the StackNavigator to take the initialProps and automagically pass them to the initial screen, without requiring an intermediary component to make use of the screenProps.
^ Feedback to consider for the project collaborators and a lesson for me to read docs more thoroughly.
Thanks for the support!

@joshuapinter I ❤ U.

@robertsimoes 🙏

Works perfectly ! Thanks @joshuapinter

@ferologics what debugger are you using for react-native?

@joshuapinter  @grabbou  can you guys please give a solution which works for react-navigation-3 in which you must set up your app container directly.

@silhouette5366
`const HomeStack = createStackNavigator(
{
Home: {
screen: Scene1,
},
},
{
initialRouteName: "Home"
}
);
export default createAppContainer(HomeStack);
class Main extends React.Component {
constructor(props) {
super(props);
}
render() {
return (
<HomeStack screenProps={{ baseInfo: this.props }}  />
);
}
}
export default Main;;
`
this.props.screenProps , Now you can take it.

@silhouette5366
const HomeStack = createStackNavigator(
  {
    Home: {
      screen: Scene1,
    },
  },
  {
    initialRouteName: "Home"
  }
);

export default createAppContainer(HomeStack);

class Main extends React.Component {
  constructor(props) {
    super(props);
  }

  render() {
    return (
      <HomeStack screenProps={{ baseProps: this.props }} />
    );
  }

}
export default Main;

Use this.props.screenProps ,  you can take it.


feat(model): support custom dataloaders in vaemixin

adds custom dataloader arguments to vaemixin methods. currently does not check that the dataloader outputs in the correct format such that errors will be raised in the module level, which will potentially be confusing.

for consistency, the data_module argument in train should be renamed to datamodule, in line with dataloader.


Cannot associate Regional WAF with ALB

AWS allows a Regional WAF to be associated with an ALB.
I could not find a field / example of to accomplish this in the docs.

PR https://github.com/perfectsense/gyro-aws-provider/pull/271 fixes my issue. I only tested with a single ALB though


[Not serious - Do not merge] TypeScript metalanguage $ref resolution

@mpodlasin  I was annoyed that we can't type the $ref resolver correctly, so I tried... 😇
And it looks like it works. Doesn't handle everything yet - I'll probably not do the rest because this entire PR was just a random challenge - but it passes the 2 basic tests and also the circular one.
TS 4.3 needed because https://github.com/microsoft/TypeScript/issues/43060

Closing with sadness. @marcelltoth still waiting for that article about what happened here! :D


[BUG] Windows Nodes in RKE2 Cluster showing Error Message in Cluster Managent

Rancher Server Setup

Rancher version:
2.6.6
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
v1.23.7+rke2r2
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

Custom = Running a docker command on a node
3 Master (Ubuntu)
2 Worker (Ubuntu)
8 Worker (Windows Server 2019 LTS)
RKE2 Cluster with custom nodes

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Admin User

Describe the bug

After updating or rebooting windows cluster nodes we have the following error message in the "cluster management"
"Error applying plan -- check rancher-system-agent.service logs on node for more information"
To Reproduce

Restart Windows Cluster Node
Result
We are unsure if this puts the cluster in an unstable state.
Expected Result

It should be possible to reboot the Windows nodes reliably without seeing an error message in "cluster management".
Log analysis on Windows nodes is extremely difficult compared to the Linux nodes, here we would be grateful for hints where exactly information can be found.
Screenshots

Additional context

This is a duplicate of https://github.com/rancher/rancher/issues/39658.


add missing config options

This PR adds corrections and renaming of options which are used only for Wii saves. This includes fixing the the item buttons, and the specification of some configuration options that were flagged as "unknown".
If there's an issue with the modifications' format, I'm open to criticism.

When I compiled it, since the code was not used, it didn't change the final DOL

The fullsplit branch will be used as the primary branch going forward until all the previously decomped code is moved over from master. Please migrate these changes to that branch and reopen.


Inkwell invoke

Description

Related Issue

How This Has Been Tested

Option<Breaking Changes>

Checklist

[ ] I have read the Contributing Guide

I added invoke to inkwell proper, now using that new version in our fork


implement Pinned repository list

Add an ability to toggle the repository list as a pinned sidebar
Description

Add a root level sidebar, to show the original repository list content (this.renderRepositoryList())
The root level sidebar is resizable, with a rootSidebarWidth in the app-state and app-store
Add a menu item to toggle the visibility of the root sidebar

Screenshots
Pinned

Hidden

Release notes
Notes: Add an option to toggle repository list as a pinned sidebar

for whom want a direct download https://github.com/leavez/github-desktop/releases/tag/0.1


Add scheduled module statistics reporting

Once a day at 8:00 UTC

This PR was released with 0.22.1


[Parser] Use the HanaVisitor for parsing hdbtablefunction artifacts

Currently, parsing hdbtablefunctions is done using string operations. This leads to problems when there are comments in the SQL code or other statements we did not consider. Also, we should be able to parse the schema and name properties as they are needed for generating synonyms.
We should use a proper parser (e.g. the HanaVisitor) as it would be easier to parse the function's schema and name.

@vmutafov I guess we should do the same for *.hdbprocedure files, as they also cause some issues recently:

https://github.com/SAP/xsk/issues/1222
https://github.com/SAP/xsk/issues/1118

@ThuF Hmm, I'm not sure about them. I'd have to take a look.

@Fluctuationqt could you provide a sample hdbtablefunction for which you hit this issue?

FUNCTION "_SYS_BIC"."customer_sample::SAMPLE_FUNCTION" (CurrentUserId INTEGER)
RETURNS TABLE ("count" INT,  "ADDRESS" DECIMAL(18,0))
LANGUAGE SQLSCRIPT
SQL SECURITY DEFINER
  AS

 /********* Begin Procedure Script ************/
 BEGIN
 	 var_out = SELECT
		 *
	FROM "_SYS_BIC"."customer_sample.VIEWS/SALES_ORDERS";

return :var_out;
END;

This sample, parsed by XSKHDBTableFunctionParser, results in a model with name equal to ""_SYS_BIC"."customer_sample::SAMPLE_FUNCTION"".
The proper expected output should be a model with name "customer_sample::SAMPLE_FUNCTION" and schema "_SYS_BIC".

high priority as dependency for customer issue


Matching empty expressions

It might be reasonable to match empty expressions, for empty expressions can have lookahead expressions, and e.g. can be used to pop start states from the stack. It seems RE-Flex does not support having an empty expression in the rules section. Will you ever consider adding such a feature? If not, is there any technical difficulty, and would you please offer some workaround method?

Allowing for empty matches/expressions is inherently problematic for scanners, which may end up running infinite loops when no input is consumed. Therefore this feature is not enabled by default. However, the reflex matchers reflex::Matcher and reflex::BoostMatcher support empty matches and expressions using option "N" (nullable). reflex::StdMatcher does not support empty matches.
The reflex tool does not have an option right now to allow for nullable matches, but this easy to add as a feature to initialize the Matcher with option N in the generated lex.yy.cpp file.
If you and others believe this is a useful addition then let me add it, say with reflex specification option %o nullable perhaps.

I forgot to mention, but option "N" does not apply to matcher scan operations, only to matches operations, so adding an option %o nullable to the scanner generator is a moot point.
Instead of enabling nullable scan matches, with all the potential pitfalls that come with it, why not replace the look-ahead pattern you mentioned but without the lookahead and simply use less to shorten the match to 0 length?

To use the function less seems to solve this problem properly, and if it does, nullable scan matches are no longer needed. But I have just looked up the manual, and it seems that less does not accept 0 as a valid argument.

The matcher().less(n) method reduces the size of the matched text to n bytes. This method has no effect if n is larger than size(). The value of n should not be 0. The matcher().less(n) operation is often used in lexers and was introduced in Lex.

Anyway, I think you are right, nullable matches are error-prone, and using less is no safer. I may as well restructure my lexical grammar to avoid such needs.

The function less can be called with zero. I'll explain in the manual why zero is usually not a good idea, but has its use when switching to another state for example.


⚠ Bofrost.at has degraded performance

In 32b6374, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2047 ms

Resolved: Bofrost.at performance has improved in ee3be15 after 14 minutes.


how to get posts from a hashtag

greetings, I'm having trouble getting posts from within a hashtag section.
I need you to return me mainly, id, url, description, creator, image and posting date.
Thanks in advance

You should use the ig.feed.tag(tag). You can iterate over the feed as it's an async iterator.


fix(fetch): improve React Native compatibility

Applying patches from this comment:
https://github.com/mswjs/msw/issues/1926#issuecomment-1905351237

Released: v0.25.16 🎉
This has been released in v0.25.16!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


[item] icon within table broken in demo

Is this a BUG REPORT or FEATURE REQUEST?:
bug
What happened:
Icon within table in item fragment not showing up.
What you expected to happen:
Icon should work.
Environment:

Syna Theme version: master

Sorry I didn't notice this issue. Made one PR for two issues.


Fix bugs in downloading IEP PDF zips

Follow-on to https://github.com/studentinsights/studentinsights/pull/2141, the mkdirp calls for the contents of the zip files would collide with the zip filename itself.

selfie


[QUESTION] Can we get a dedicated forum for questions?

Is it possible to get a dedicated forum for issues not related to bug reports and feature requests? I like reading the questions users have but it really messes up with my notifications. :/
For other projects, I like to keep track of questions other people have so I generally search a forum (google groups or something in a similar vein) but for FastAPI I've had to put the repo on watch for bug reports so that I can update my app accordingly and this is becoming a mess.
If needed, I can volunteer to set something up for this.

Have you tried the gitter chat?

@chopraaa questions are generally here (as this one).
Some bugs end up being actually questions, not bug reports.
And more general conversation happens on the Gitter chat here: https://gitter.im/tiangolo/fastapi
If you only want to know when there are new features, you can watch only the releases. But if you want to receive notifications for questions, I guess this would be the best place.

Gitter is a blink-and-you-miss-it scenario for me. I'd rather have threads catered to specific questions so I can remember where I read about the particular issue instead of scrolling through chat.
Just a suggestion. Thanks.

Yep, I agree that Gitter is a bit chaotic many times, and not necessarily very searchable.
For general questions, GitHub issues would probably be the right/equivalent place.
I guess a forum would probably have a comparable amount of notifications with questions, reports, etc.


Units of excited state energies

Just tested a TDDFT calculation with ORCA 4.2.0 The excited state energies and oscillator strengths are present in the output dict, but the excited state energies are in rather arbitrary units of cm^-1. I wonder if we shouldn't change the units to something saner, either eV or a.u. to make it consistent with the SCF energy.  Also of note that these are energies relative to the ground state (which is fine). I guess this might be a bigger discussion about units...
btw: This package might be interesting for manipulating units.

If we want to play with units, we should do that before releasing stable version. It would be nice to see what other plugins are doing.


Views are using wrong name

Up until now, created views might use a wrong file name. If the original was called "All Items" and the file was "AllItems.aspx", the restored view was restored as "All Items.aspx" using the display name of the view.
This caused some issues in our environment, so I changed the behavior to extract the file name from the URL. This file name is used as the first display name of the view and is therefore used as the file name. Afterwards, the display name will be changed to the correct file name.
If an error should occur, the behaviour switches back to the current behaviour by using the display name as the view and file name.

@lafe, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

I've already done this in here, but it's not still merged...

Thanks lafe...needed addition, but closing as similar implementation done by @s-KaiNet was merged.


Bump CSI sidecars v1.4

https://github.com/longhorn/longhorn/issues/5672
Mostly the same as #1839 except it doesn't bump external-provisioner and external-snapshotter beyond the v1beta1 VolumeSnapshot API.

We decided not to update the sidecars in previously released minor versions of Longhorn. See the discussion here: https://github.com/longhorn/longhorn/issues/5672#issuecomment-1502579905.


fix bugs in linux

update cmakelists
(T*)* doesn't work with g++, It must be T**
g++ must use this pointor when accessing protected members of superclass
in C++ with
templates(http://stackoverflow.com/questions/4010281/accessing-protected-members-of-superclass-in-c-with-templates)

Hi @you-n-g, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Xiao Yang (Person Consulting)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Hi, @you-n-g

First, I think you should send pull request to Microsoft/binding instead of Microsoft/master,
Second. You may need to rebase the origin/binding to make it easy to merge.

HI,  @huxuan  I've rebased the commit now.
It is based on the newest version of master when I create this commit .
This issue is a problem both in master and binding.  I've also cherry-picked it to the binding branch.


Add default icon when no logo was provided [skip ci]

@mananwason please merge, this fixes #643

Current coverage is 11.25% (diff: 100%)

No coverage report found for development at d5aee05.

Powered by Codecov. Last update d5aee05...df5ab40

LGTM merging


Can't run testbed with typescript

I was able to run an example in html, then I tried to run same example with typescript but I'm getting this error:
Uncaught TypeError: planck_js_1.default.testbed is not a function
at ./main.ts.window.onload

Can you share your exact setup? Seems like you are not including Planck.js in your bundling process or maybe you are trying to use planck directly as a ts module?

Here is my html file:

Here is my main.ts file:

Few lines from pool example to give you an idea what is going on, I only modified it to work with typescript:

And finally my console:

I think WDS stands for webpack-dev-server, so my guess would be you forgot to add planck.js as an external dependency in your webpack configuration. Can you please try adding this to your webpack.config:
new HtmlWebpackExternalsPlugin({
  externals: [
    {
      module: "pixi-sound",
      entry: "dist/pixi-sound.js",
      global: "PIXI.sound",
      supplements: ["dist/pixi-sound.js.map"],
    }
  ]
})

(You have to install the html-webpack-externals-plugin package therefore)

Thank you very much, I am not experienced with webpack (I was using a starter project that handles webpack stuff), I was able to run it this time, here are the steps:
1 - npm install --save-dev html-webpack-plugin
2 - npm install --save-dev html-webpack-externals-plugin
3 - adding imports to webpack config

const HtmlWebpackPlugin = require("html-webpack-plugin");
const HtmlWebpackExternalsPlugin = require("html-webpack-externals-plugin");

4 - adding this entry to webpack config

plugins: [
    new HtmlWebpackPlugin(),
    new HtmlWebpackExternalsPlugin({
      externals: [
        {
          module: "planck-js",
          entry: "dist/planck-with-testbed.js", //needed to change this
          global: "planck",
          supplements: ["dist/planck.min.js.map"],
        },
      ],
      cwpOptions: { // had to add this because it was thinking node_modules was under src/
        context: path.join(ROOT, "../node_modules"),
      },
    }),
  ],

Oops, I forgot about the html-webpack-plugin 😅 Glad you figured it out anyways. Thanks for documenting your solution!

I have the same error. I use Parcel instead of Webpack. Is is possible to use Testbed with Parcel? I made a simple example in TypeScript on Sandbox: https://codesandbox.io/s/planckjs-testbed-ts-n4imh4?file=/src/index.ts
import * as planck from "planck-js";

planck.testbed(function (t)
{
    // Create a world
    const world = planck.World();

    console.log(world.getGravity());

    // Make sure you return the world
    return world;
});

Hey @8Observer8! Sorry, I don't know anything about parcel, but actually all you have to do is instruct parcel to use @node_modules/testbed/index.js instead of @node_modules/lib/index.js (or maybe you have to use a file from dist?) Based on this maybe importing from "planck-js/testbed" can work? The codesandbox example also does not throw anymore but still does not work somehow...

Hi @zOadT! Thank you for reply. I am not a very skilled person in web development. Could you explain step by step what and where I should try to add? I have a general node_modules folder on the high level of hierarchy of folders to installing all packages to keep space on my laptop. Some packages I installed globally like: typescript, parcel, http-server and so on.
I use Parcel because it is the fastest bundler. It is require only 30-500ms to build. It is very simple to use and https://codesandbox.io/ supports Parcel to publish TypeScript examples. It is not requires big config files. It is very friendly for beginners who started learning of TypeScript with game development in WebGL and Box2D. Suddenly developers of Planck.js did not implement the b2Draw class. Is it possible that it will be implemented in the future? I write desktop versions of my examples in Qt C++ and Python (with PyQt6). Developers of pybox2d (https://pypi.org/project/Box2D/) implement the b2Draw class. Qt allows to build for Android and iOS very simple. Python is very good for understanding for programming. I love Python, C++, TypeScript and JavaScript. I want to write example on these languages and these examples must be very close to each other. The b2Draw class is very necessary. It is difficult to rewrite a game very time when you want to see colliders. But I can just write the first version in Python or C++ and rewrite it to TS/JS later.
These are files of my example:

package.json
{
  "name": "testbed-ts",
  "version": "1.0.0",
  "description": "",
  "main": "public/js/bundle.js",
  "targets": {
    "main": {
       "includeNodeModules": true
    }
  },
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "del_files": "del /f /q /s .\\public\\js\\*.*",
    "dev": "npm run del_files && parcel watch src/main.ts"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "gl-matrix": "^3.4.3",
    "planck-js": "^0.3.31"
  },
  "devDependencies": {
    "@types/gl-matrix": "^3.2.0"
  }
}

src/main.ts
import * as planck from "planck-js";

planck.testbed(function (t)
{
    // Create a world
    const world = planck.World();

    console.log("gravity: " + world.getGravity());

    // Make sure you return the world
    return world;
});

public/index.html
<!DOCTYPE html>
<html lang="en">

<head>
    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Document</title>
</head>

<body>
    <script type="module" src="js/bundle.js"></script>
</body>

</html>

Unfortunately I'm not able to give step by step instructions because I myself don't know enough about parcel, but actually replacing the first line with either import * as planck from "planck-js/dist/planck-with-testbed"; or import * as planck from "planck-js/testbed"; should work. Indeed planck.testbed seems to be defined, but doesn't seem to do anything in the sandbox. Could you try if bundling this through parcel on your machine changes anything about that? Maybe its just a problem with the iframe? Otherwise I currently don't know what the problem could be
Also as far as I know it is currently not planned to implement b2Draw unfortunately

Thank you very much! It works:

import * as planck from "../../../../node_modules/planck-js/dist/planck-with-testbed";

planck.testbed(function (t)
{
    // Create a world
    const world = planck.World();

    console.log("gravity: " + world.getGravity());

    // Make sure you return the world
    return world;
});

The second variant does not work:
import * as planck from "../../../../node_modules/planck-js/testbed";

But it does not matter because the first one is enough. Thanks again!

Unfortunately CodeSandbox does not install the "dist" folder: https://codesandbox.io/s/planckjs-testbed-ts-n4imh4?file=/src/index.ts

And testbed does not work:
import * as planck from "../node_modules/planck-js/testbed";

Also as far as I know it is currently not planned to implement b2Draw unfortunately

These screenshots show how important it is to be able to draw colliders:

Unfortunately, I didn't find a JS port of Box2D that implements the b2Draw class. For example, @box2d/core does not implement this either: Hasn't the DebugDraw() method been implemented?


Add method to fetch chromedriver

I have added a way to avoid hardcoding of the version number for fetching chromedriver.

Looks like the pycache folder too got included by mistake. How to correct it?

Done removing unnecessary files. Please review

works beautifully, thanks


Debug isinf_pos and isinf_neg test

@hamptonm1 can you indicates if this work will continue somewhere else, or if the problem is now solved? Tx


Add experiment with Raykar (2010)

#136 #156

To get to Yan from Raykar, let α = β and let them be a function of x. This is then η(x). The difference is that Raykar separates the sensitivity and specificity (α and β, respectively) and assumes x-independence, whereas Yan doesn't separate them and does not assume x-independence.

Also, there's a closed-form solution for α and β, so I might as well go ahead and implement/test this. It looks pretty straightforward.
Worth mentioning that Raykar actually give the initialisation conditions — further, they initialise to a majority vote, just like I do.

I have an implementation of Raykar now. It has the exact same problems as the Yan implementation, even though it's much simpler.

Added random restarts and logistic regression initialisation and everything seems a lot smoother now. With the sklearn API (#170) this should now be pretty simple to make an experiment out of, though I have no idea how slow it'll be.

Sped it up a fair bit by changing from a second-order to a first-order method for finding the logistic regression weights. This is probably as fast as it's getting, and it's still really slow. I'm not sure how to deal with this or even if it's possible to deal with this. I'll let it run for a while longer and we'll see how it goes, I guess.

I added some Raykar experiments (top n annotators, PCA). Results are decent. I'd be much happier with it actually running properly, but oh well.
I'll run it again with more annotators.


Better Caching System

Most API Methods are reused over and over again, use caching to reduce the time connected.
Solution
API methods that should cache:

/api/post/get-comments-from-post (1 min),
/api/post/get-post-from-id/{ID} (1 hr),
/api/user/get-user-from-id/{id} (1 hr),
/api/post/get-posts-from-user (1 hr)
/api/user/auth-to-id/{auth} (1 hr)

Alternate Solution
An alternate solution could be using ETags.

 Better Caching System


[CLEANUP] - Adding blueprint-core and commons-configuration to assemb…

…ly dependencies as metaverse expects to find them in the system classloader

@mattyb149 @rfellows please review


Add ability to clear route history

Hi! I want to clear the routes history on user logout. Is there a way to do this?
Now, when i logout, my app shows only login page. But when i press "back" browser button on that page it continues to change url in browser. How can i prevent this behaviour and leave only "login" route?
Thanks!

Oh, sorry, i did miss the redirect function :(

Do we have the API for that in History API?

Oh, we have not. You are right. It seems impossible now, so i close the issue.


Improve the UI/UX improvement

Is your feature request related to a problem? Please describe.
Enhance the UI/UX of the application to create an appealing user experience by implementing aesthetically pleasing design elements, such as improved buttons and other components, in a consistent and visually engaging style.
Describe the solution you'd like
Improve the user interface and user experience (UI/UX) of the application.
Describe alternatives you've considered
No response
Additional context
Please provide me with the template for the issue mentioned so that we can assign it to you accordingly.
Code of Conduct

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

Hi @Suryansh1720001 , I would like to work on this issue. As a user also, i noticed a few UI/UX enhancements in the application. I got a few ideas to add as a feature to the application. That would definitely create a better experience to the user. First i would like to work on this. later I would share my ideas to you. If you find it also good then we move forward.
My previous work:
This is my first Android application project i done for my friends. I didn't like to compromise on UI/UX that time even i am beginner that time. If you find my work Well. I would like to work on this issue. Please assign me under GSSOC 23.

Thank you 😊

@Ashok-Kumar-dharanikota
This link is not working

I think, it's okay now. Please check.

I would like to work on this issue under Gssoc

@Ashok-Kumar-dharanikota
I appreciate the current design of the splash screen, but I would prefer it to have a book-opening style with Taaveez written on that rather than the dynamic effect it currently has.

Hi @Suryansh1720001 ,
thanks for sharing your feedback. I also found some relevant designs like you mentioned. Did you want it as static or dynamic?
here's the link : lottiefiles
you will find different types of high-quality designs. I found some relevant designs as you mentioned. But it would be better if you would suggest any particular one So I will replace the current design with a new one.

Hi @shreeja5060 ,
It's really good to see your passion and love for open-source contribution. I don't have any problem if you are willing to work on this issue. in case it really helps us to resolve this issue faster if we could share this work. I truly appreciate your suggestions @Suryansh1720001 on this thing.

Is the issue still open?

Hi @Suryansh1720001,
In the item_poem.xml file, I noticed some UX issues that need attention:

Date: Currently, we are displaying the complete time in the application. However, users are less likely to care about the specific time. Instead of the DD/MM/YYYY - HH:MM:SS pattern, it would be more effective to use either MM/YYYY or DD/MM/YYYY. This change will not affect important elements like the title and subtext.

It is beneficial to provide options for delete, edit, and share functions. However, as the number of items (poems) increases, it can become cumbersome for users to see icons for each item. To address this, I have only included the share button. By long-pressing on an item, users can choose to delete one or multiple items at a time instead of clicking on individual delete icons.

It is a common practice to include an edit icon when clicking on an item takes the user to the editing section. This will allow users to easily modify their poems or perform other necessary edits.

I have made modifications to the UI design and I am sharing it with you. Please let me know if you approve of the changes. If you have any suggestions for further modifications, please feel free to share them.
Thank you.

Hi @Suryansh1720001 ,
Thanks for your suggestions. I definitely will work on it.
I have one query.
3rd point : Can I know, what you mean by image left? The reason i added the imageview. If the user writes a poem or any note about traveling, nature, art, or his memories with his friends or family. When he read the poem again after sometime it will help him to recall his memories, and feel the full vibe out of it by seeing the image also. It would be so different and a new experience to the user. The design is just demo purpose not the permanent design. Few modifications maybe occurred later.

@Ashok-Kumar-dharanikota
I'd like to clarify the image used in the item. Is it the image provided by the user during the content addition, or is it something different?

@Suryansh1720001 Okay I got it.
When the user clicks on a new item, when he forgot to add any image. By default the application will add random image(relevant images like Books, a person sitting next to the window writing something while outside is raining etc) from drawable folder. We also providing such collection of images, helps user to categorise the items. And Please let me know about the Splash Screen one. Should I replace the current design with any particular one. Either static or dynamic design?

Great work @Ashok-Kumar-dharanikota!
Just to update you, one contributor has already raised PR-52 regarding the splash screen. Additionally, I have a request to modify the design of the splash screen. If the contributor for PR-52 is unable to make the necessary changes, I will create a new issue and then all contributors can comment on it. I will assign the issue accordingly. Therefore, I would like you to prioritize working on the issue - item update. We can discuss the specifics of the splash screen later on.

@Ashok-Kumar-dharanikota
Any update?

Hi @Suryansh1720001 ,
I am done with the new Design part of the App. I am also thinking to share the modified design with you to take your feedback.  Please share any updates or enhancements in the design. Now I have to connect the new design with the Kotlin code. I am working on it. i followed the 2-color patterns. When the app is first opened it has two main colors. one is white and the other one is blue(little gradient). when the user shifts to activity_notes it changes to different color combinations. I want to continue this 2-color code pattern. Is it Okay for you?


Delete ROLLBACK_COMPLETE CloudFormation stack

Describe the feature
The CLO console will provision CloudFormation stack, however, it will NOT delete the stack which is in ROLLBACK_COMPLETE status.
Use Case
Remove the ROLLBACK_COMPLETE CloudFormation stacks, and keep the list clear.
Proposed Solution
No response
Other Information
No response
Solution version used
v1.0.1

I think it is better not to delete the stack so you can investigate the issue afterwards.
Actually, an option to keep the objects instead of rollback would make sense. I have tons of iam issues with the app. If I can fix the issue, or better, the root cause and restart from the failure point, I am more efficient.
Would it be possible to add the option: keep the objects if failed?

Thanks for reaching. @qmugnier Currently, the solution will not delete sub stacks provisioned through the console. You can view the error in the CloudFormation console. Do you mean you want to keep the objects provisioned by CloudFormation even if the entire stack was failed?

@JoeShi : Well, it would be convenient to be able to keep the objects, as an IAM error for the generated objects is easy to fix, so if you delete the stack you cannot really solve the issue as the involved role or policy is gone.


Print a stacktrace on fatal signal.

I noticed that triton-opt printed a stack but our Python and C++ unit tests
didn't.  After some digging I figured out how LLVM was turning on stacktraces.
This PR turns them on for our C++ unit tests and Triton when run from Python
(!!).

Woo, thanks!


Feature/14 integrate corp branding

CPR corporate branding has been incorporated into the UI.

@eurolife it all functions well, but the fonts look a bit heavy to me. I notice you've given them a weight of 700 for all headers and different sizes in the CSS. Would it be possible to amend the CSS to coincide with the typography section of the brand guidelines?

@eurolife it all functions well, but the fonts look a bit heavy to me. I notice you've given them a weight of 700 for all headers and different sizes in the CSS. Would it be possible to amend the CSS to coincide with the typography section of the brand guidelines?

I'm just excited the tests finally passed! I think slow network connection was the issue so I added a wait command before it scrolled. I will look at the heading sizes.

@kdutia just pushed up the font styling changes.


Fixed leaks, NULL dereferences and broken init.

Some fixed found during static code analysis.

LGTM +1

+1


A better solution on snippetsPreventQuickSuggestions

I finally found the only example on the Internet where snippetsPreventQuickSuggestions needs to be set to true, as follows:
this can backfire. if Intellisense suggests something and you click tab to go to the next snippet place, it will complete the suggestion. E.g. for snippet; typing i as variable; Intellisense suggestion:int; click tab; int inserted. But to avoid this, click escape before tab (only if Intellisense prompted) – Superior Nov 9 '19 at 9:58
@lisichka Thanks. – Mark Dec 23 '19 at 16:18
@Superior Why does visual studio not have this problem? Why can't vs code provide the same experience as visual studio? – Joke Huang Aug 16 at 10:59
@JokeHuang Why is that question directed towards me? I don't know – Superior Aug 16 at 14:29
add a comment
https://stackoverflow.com/questions/55683145/intellisense-not-working-in-code-snippets-vs-code
However, it can be seen from the example that the problem is not Quick Suggestions, but TAB is responsible for moving the snippet place and confirming Quick Suggestions at the same time.
When it is true, the only convenience is not to manually press the ESC key, but not everyone needs to use TAB to move snippet place or confirm Quick Suggestions, so it is not necessary.
I have been using the Enter key to confirm Quick Suggestions, and I know TAB for the first time.
If you have to prevent something, you can prevent TAB from confirming Quick Suggestions in snippets, and only move the snippet place.

This is a duplicate. Unfortunately, I don't recall the exact issue number nor its title :flushed: Anyways, searching for issues with the same labels will likely help you :point_right:
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aopen+label%3Asnippets+label%3Asuggest

@Despoinis please stop filing the same issue over and over again. I think the reasoning is well despite not everyone being happy. Having said that, changing a user config to get what you need once (!) isn't a big ask.

@jrieken
People who are new to VSCode cannot know this setting from the beginning, so how to change it to false? Haven't you seen so many people asking questions on the Internet?
And, now I am not repeating the question, but giving a better solution. Have you read my text completely?

Have you read my text completely?

No, I am not reading your text. If you have anything to comment, do it on the initial issue and do not create new issues. Also, I have explained that we have no plans to change the default of this setting

@jrieken
You just have to read one sentence: If you have to prevent something, you can prevent TAB from confirming Quick Suggestions in snippets, and only move the snippet place.

@jrieken
Do you know the Curse of knowledge? New users of VSCode don't just need to change it to false. First of all, they have to know that this setting option exists.

@jrieken
If VSCode can't consider new users, then I will keep repeating feedback until VSCode becomes more usable.


Update Teams-add-in-for-Outlook.md

Added a line item about EWS allow list under Troubleshooting section.

@LanaChin @LolaJacobsen can one of you review/approve this PR?


Out of memory

I have a problem with the U-net of Generator which easily runs out of memory, I don't know if you can train the dataset successfully?

You might need a gpu with 12g memory.

It's a U-net, so should I add the pooling layer after each convolutional layer?  In this way i think it will accelerate the execution time and save much more memory.

I am not sure, but you can try it . Or maybe you can resize the input images and labels into a smaller size less than 224.


Documentation: Add info for multiValueFilterOperators

What this PR does / why we need it:

updates the plugin metadata documentation with information about the new multiValueFilterOperators property

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
idk whether to mark this is patch or no-changelog, defer to you 🙇

 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.

The no-changelog tag is correct. Doc updates don't get to the changelog normally.


Allowed exposed headers overwritten by this mixin.

I have multiple headers I need to expose outside of this mixin. The problem is it sets the only allowed exposed header to Content-Range. Ideally it should append to an array if headers are already exposed per the middleware.json configuration.
https://github.com/darthwesker/loopback-content-range/blob/v1.0.1/index.js#L21

Hi @joe-at-startupmedia
Could you please provide an example of your implementation?