Solver class

ITERATION1:
A class implementing the WFC algorithm for 2D grids. It uses an auxiliary Grid class to abstract the details of the grid. A Direction enum could help:
enum Direction { UP, DOWN, LEFT, RIGHT };

typedef std::pair<int, Direction> Side; //tile and direction for constraints

class Grid {
public:
  Grid(int N);
  inline int get(int i, int j);
  inline int set(int i, int j);
};

class Solver {
public:
  Solver(std::vector<int> tiles, std::map<Side, std::vector<int>> constraints, int seed=0);
  void setSeed(int seed);
  Grid solve(int N);

  //calls registered function with tile in collapsed grid slot
  void onCollapse(std::function<void(int, int, int)> callback);

  //calls registered function with possible tiles in collapsed grid slot
  void onPropagate(std::function<void(std::vector<int>, int, int)> callback);
};

ITERATION 2:
Add a template parameter to determine the tile type. Makes callbacks easier to use. Options for choosing custom collapse heuristic instead of the min entropy heuristic:
template<typename T>
class Solver {
public:
  Solver(std::map<int, T> tiles, std::map<Side, std::vector<int>> constraints, int seed=0);
  void setSeed(int seed);
  Grid<T> solve(int N); //Grid parameterized in the obvious way

  //low level function to change heuristic (change vector to a span/iterable through templating?
  void setHeuristic(std::function<int(std::vector&<int>)> );
  void setHeuristic(std::function<int(std::vector&<int>, std::map<int, T>& tiles)> );

  //calls registered function with tile in collapsed grid slot
  void onCollapse(std::function<void(T, int, int)> callback);

  //calls registered function with possible tiles in collapsed grid slot
  void onPropagate(std::function<void(std::vector<T>, int, int)> callback);

private:
   std::map<int, T> tiles
};

Better idea:
The adjacency constraint map is an implementation detail, instead provide an interface for setting constraints. This way, constraint creation controller classes can be created which read configuration files or such and programmatically apply the constraints to a given solver, or they can be added manually.
Points to look at:

abstract away grid coordinates using some Coordinate type instead of two random ints
templating for the callbacks? compile time evaluation can be used maybe?
Make the TileKey type generic! (solver/grid becomes like a map)
change to unordered map
make the default constructor that takes an iterable generic

template<typename TileKey, typename T>

class Solver {

public:
  //since this now not the main way of setting constraints, can be verbose
  Solver(std::map<TileKey, T> tiles, std::map<std::pair<TileKey, Direction>, std::vector<TileKey>> constraints, int seed=0);

  //the main way of using the library, the int keys are auto-generated from vector as the indeces
  Solver(std::vector<T> tiles, int seed=0);
  //similar, but keys are provided explicitly for each tile
  Solver(std::map<TileKey, T> tiles, int seed=0);

  void setSeed(int seed);
  Grid<T> solve(int N); //Grid parameterized in the obvious way

  //set the constraints using this specified interface
  void setConstraint(TileKey t, Direction d, std::vector<TileKey> neighbors);

  //To-do: returns a receipt object to deregister when out of scope?
  //API: calls registered function with tile in collapsed grid slot
  void onCollapse(std::function<void(T, int, int)> callback);

  //API: calls registered function with possible tiles in collapsed grid slot
  void onPropagate(std::function<void(std::vector<T>, int, int)> callback);

  //low level function to change heuristic (change vector to a span/iterable through templating?
  void setHeuristic(std::function<int(std::vector&<TileKey>)> heuristic);
  void setHeuristic(std::function<int(std::vector&<TileKey>, std::map<TileKey, T>&)> heuristic ); //with tile type T info

private:
   std::map<int, T> tiles
};


I added a simple Method to check if the Phone Number is available for the Profile Card.

Its my first time contributing on GitHub

Could you change the branch to dev, please? :)


⚠ BeyondHD has degraded performance

In 3f7ab29, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 502
Response time: 16078 ms

Resolved: BeyondHD performance has improved in ef9a259.


Remove TitlePart from Template content type and make ShapePart.Name a persisted unique property

Currently the TitlePart.Title is the name of the Template (Shape) but it is not unique. You can create two Templates with the same Title. In this case an exception is thrown on every pageload.
Exception is thrown in Orchard.Template module in TemplateShapeBindingResolver class. The BuildShapeProcessors method tries to create a dictionary from the content item list where the ShapePart.Name is the key.
With this solution the part should implement ITitleAspect and Name would be the Title.

This guy knows something. Trust him.

I've actually contemplated doing this when implementing this type, but decided to wait and see. I agree with Mark's proposal.

If the issue is to fix the unicity (because the word exists) check then we can still do that with the Title, we just need to add a driver for this check. We do that for other parts. I don't disagree with the change but it would be a breaking one (at least it would incur migrations/update) but if we can prevent it, let's just fix the issue.

That's true, the driver or handler could just add a model validation error.

Since we won't have an 1.9.4 most possibly re-targeting this to 1.9.3.

We'll merge 1.10 into master. And rebase 1.9.x on top of master and rename it 1.10.x. Also merge it into dev.

OK


Document the testing process

Hey! In the same idea as https://github.com/timokau/marvin-mk2/issues/112 ; would it be possible for you to write some documentation for how you test marvin when you develop on it? Creating multiple github accounts / repositories is totally OK, I just have no idea how to run marvin myself :)

Hi! I don't have the time to write proper documentation right now. Here is a draft:

There are some automated tests that you can run with pytest. There are not enough of them though, and they are limited by the current architecture and GitHub's API.
I have a test instance of marvin that runs on heroku and responds to PRs on marvins-playground. You can set one up by

setting up a playground repository.
setting up a marvin instance. If you want to use heroku for this, their free credits should be sufficient. They pause your application when it does not receive any events for a while, which is good for the testing use-case. You could also run it locally of course, but your machine needs to be able to receive webhooks.
creating and configuring your own GitHub app here.

Permissions: Read access to metadata, Read and write access to issues and pull requests
Repository access: Your playground repository
Webhook URL: https://<your-heroku-app>.herokuapp.com/webhook

configuring heroku to set the WEBHOOK_SECRET, GH_PRIVATE_KEY and GH_APP_ID environment variables.

I usually test the changes in a PR on the playground and also do some sanity-checks to make sure that I didn't break any basic functionality. There is room for improvement before we move on to a less experimental stage (#112). You should be aware that marvin will use the same reviewer list on your playground repository (#93).
I hope that helps. Let me know if you're having trouble with one of the steps. It would be great if you could make some notes to make the contributing experience smoother in the future :)

Hi! I don't have the time to write proper documentation right now. Here is a draft:

There are some automated tests that you can run with pytest. There are not enough of them though, and they are limited by the current architecture and GitHub's API.
I have a test instance of marvin that runs on heroku and responds to PRs on marvins-playground. You can set one up by

setting up a playground repository.
setting up a marvin instance. If you want to use heroku for this, their free credits should be sufficient. They pause your application when it does not receive any events for a while, which is good for the testing use-case. You could also run it locally of course, but your machine needs to be able to receive webhooks.
creating and configuring your own GitHub app here.

Permissions: Read access to metadata, Read and write access to issues and pull requests
Repository access: Your playground repository
Webhook URL: https://<your-heroku-app>.herokuapp.com/webhook

configuring heroku to set the WEBHOOK_SECRET, GH_PRIVATE_KEY and GH_APP_ID environment variables.

I usually test the changes in a PR on the playground and also do some sanity-checks to make sure that I didn't break any basic functionality. There is room for improvement before we move on to a less experimental stage (#112). You should be aware that marvin will use the same reviewer list on your playground repository (#93).
I hope that helps. Let me know if you're having trouble with one of the steps. It would be great if you could make some notes to make the contributing experience smoother in the future :)


[Tab] Remove font size change logic

[x] I have followed (at least) the PR section of the contributing guide.

Closes https://github.com/mui-org/material-ui/issues/13903.

@rfbotto I'm trying a different release plan here. Some people might be overriding the style. I have removed the documentation of the classes as well as the custom CSS. As for the motivation, I think that this behavior has a low utility vs the cost it has.

@rfbotto Thanks


Which dataset is used to evaluate the work?

Hi, many thanks for your great work!
As for the quatitative detection AP listed in your paper, which dataset is used? The daytime split, night split or both of them?
And in the training stage, both the daytime and the night data are used?
THX.

Hi @chengricky,
we used both day and night splits for training and evaluation.
Best,
Mario


Pipelines.html still using legacy syntax.

The home page pipline section still uses legacy lambda syntax, should be:
curl https://api.github.com/repos/elves/elvish/issues |
from-json | all (one) |
each {|issue| echo $issue[number]: $issue[title] } |
head -n 11

head -n 11 could also be replaced by take 11, but perhaps showing the interaction with external commands is more important...

Yes, the use of head instead of take is intentional, to use interop with external commands.


Reverts kernel bump + patch

This affected users of the unstable branch in an "untenable" way. #54508, once merged, will allow us to catch similar regressions.
See

#48828
#54509

Hopefully we can have a resolution for the issues before the 19.03 release.
cc @peti @bachp
We don't have "unstable" release notes, but if we had some, we'd include something along the line:

The default kernel has been downgraded back to the 4.14 LTS. This will affect all the default images built by the project, especially notable the sd_card image for aarch64 has never previously been built with the 4.14 kernel. End-users of either new hardware or fussy hardware might want to verify which kernel they are using before rebuilding and rebooting to keep track of potential regressions after boot.
Please note that the new_kernel variants, which would help with this issue for users needing an installation media with a new kernel, haven't been build since the addition of ZFS to the default build for the installers. cc @grahamc / PR #51090
in job ‘nixos.***_new_kernel.***-linux’:
Linux v4.20.6 is not yet supported by zfsonlinux v0.7.12.
Try zfsUnstable or set the NixOS option boot.zfs.enableUnstable.

✔ Fits CONTRIBUTING.md

Seems to have broken my build for the kernel? Reverting through git revert -m 1 fixes it so I'm assuming it's because of this PR.
builder for '/nix/store/lmp0j2b4kx5nwpn1ckj43s2py7y7234k-linux-config-4.20.6.drv' failed with exit code 1; last 10 log lines:
  Hunk #1 succeeded at 48 (offset 9 lines).
  applying patch /nix/store/h4c3bag6lv3y47sbx83339xnalmlmdmg-modinst-arg-list-too-long.patch
  patching file scripts/Makefile.modinst
  Hunk #1 succeeded at 10 (offset 1 line).
  applying patch /nix/store/gmlkhdbh9z62q8y7fs5g6839923yvs4q-0005-mmc-sdhci-iproc-handle-mmc_of_parse-errors-during-pr.patch
  patching file drivers/mmc/host/sdhci-iproc.c
  Reversed (or previously applied) patch detected!  Assume -R? [n]
  Apply anyway? [n]
  Skipping patch.
  1 out of 1 hunk ignored -- saving rejects to file drivers/mmc/host/sdhci-iproc.c.rej
cannot build derivation '/nix/store/zj6pxjc7d257mdh3vgqmnwcxhhrm0k75-linux-4.20.6.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2fd8ghwb66zm6lscnqynryrj2cf7y2dg-nixos-system-nixos-19.03.git.4de9ef1.drv': 1 dependencies couldn't be built
[0 built (1 failed), 0.0 MiB DL]
error: build of '/nix/store/2fd8ghwb66zm6lscnqynryrj2cf7y2dg-nixos-system-nixos-19.03.git.4de9ef1.drv' failed

Just verified 09af2fb9e0d948dcf71d2008bd8426fa3ee8f25f.

Moving backt to 4.14 seems to break zram support because https://github.com/NixOS/nixpkgs/pull/52991 changed the default to zstd. Which is missing in 4.14.

crap. Time to revert relese notes


Check if docker compose can be used for deployment

and implement
this would make our .sh script custom deploment process more generic and stable
however its not clear if we can do our remote deployment on multiple hosts with docker compose
use this docker compose 1) to do our own deployment 2) to provide users who want to setup won easily

docker compase + docker swarm with manual scheduling might work: https://docs.docker.com/compose/swarm/#manual-scheduling

installed compose locally and started up basic won deployment (adjusted memory and cpu settings of virtual machine). Compose seems to be running fine

Installed newest version of docker and cocker-compose on satsrv04/05/06/07:

Stop docker daemon

sudo service docker stop

Uninstall docker

sudo apt-get --purge autoremove docker
sudo apt-get --purge autoremove docker-engine

Check if docker is really removed (otherwise, install it and remove it again)
Remove old docker images and containers (NOTE: Otherwiese if there is too much old data migration takes several hours and docker isnt responding at that time)

sudo rm -rf /var/lib/docker

Install docker-engine

https://docs.docker.com/engine/installation/linux/ubuntulinux/

configure the docker daemons to be available via tcp, add in file the entry:

sudo vi /etc/default/docker
DOCKER_OPTS="-H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock"
sudo restart docker
test if commands are working: docker -H localhost:2375 ps, docker ps
https://docs.docker.com/engine/admin/

Install docker-compose

https://docs.docker.com/compose/install/

Found out that docker swarm (as a "standalone" product) will probably get outdated soon, since docker integrated swarm functionality as "swarm mode" into docker engine:

https://github.com/docker/compose/issues/3804
https://www.infoq.com/news/2016/06/dockercon-docker-swarm

However docker compose doesnt seems to work together with docker swarm mode yet:

https://github.com/docker/compose/issues/3656

Therefore we decided for the moment to move our script to individual docker compose scripts for each node. We will wait for the next docker releases if docker will integrated swarm and compose better in the future. Until that we will start our compose files just from our shell scripts without using swarm

Setup docker-compose files for int and master deployment


Consider updating prefixes and base definitions and representations in SXP

Is your feature request related to a problem? Please describe.
Coming from SPARQL, if a sub-expression containing a URI with a lexical representation using a prefix or base definition is extracted, the to_sxp method will use that representation even if the prefix or base definition is no longer included.
Describe the solution you'd like
A method for managing prefix and base definition within an S-Expression (SPARQL-specific) which would recursively update the lexical representations of embedded URIs (including literal datatypes) would be nice.
# Possible refinements

# Adds a prefix declaration to an Array representation of an S-Exp and recursively update the lexical representations of contained URIs
def add_prefix(pfx, uri); end

# Adds a prefix declaration to an Array representation of an S-Exp and recursively update the lexical representations of contained URIs
def add_base(uri); end

# Returns a deep copy of an Array representation of an S-Exp without  the specified prefix and removes any lexical representations from included URIs recursively
def remove_prefix(pfx); end

# Returns a deep copy of an Array representation of an S-Exp without  the specified base and removes any lexical representations from included URIs recursively
def remove_base(); end

# Returns a deep copy of an Array representation of an S-Exp without any prefix or base declarations.
def clear_prefixes; end

# Return the prefix definitions from an an Array representation of an S-Exp
def prefixes; end

Additional context
See the email thread at https://lists.w3.org/Archives/Public/public-rdf-ruby/2021Dec/0014.html

This issue was settled without the described API changes. Instead, the signature of .to_sxp was changed to include keyword parameters prefixes and base_uri, which is propagated down. When RDF is included, the Array#to_sxp method looks to see if the array is of the form [:base _iri_ _expr_] or [prefix [['pfx:', _iri_], ...], _expr_] and extracts and passes the prefixes and base_uri parameters to recursive calls of _expr_.to_sxp.


how to upload certificatefile and privatefilekey to aws lambda function

I used following lines in my code and uploaded to aws lambda function
String certificateFile = "C:\Users\Ganesh\eclipse-JEE\awsPublish\src\Demo\e2293989f3-certificate.pem.crt";
String privateKeyFile = "C:\Users\Ganesh\eclipse-JEE\awsPublish\src\Demo\e2293989f3-private.pem.key";
following screenshot is from AWS lambda cloud log

How can i add certificatefile and privatefilekey to my code which can be linked directly by aws lambda function?

Lambda Client Certificate
Problem got solved
I added files to resources folder in eclipse project.
and provided  relative path as String certificateFile "resources/certificatefilename";
Later Uploaded to AWS lambda function.

Hi Madhukar,
Can you please send me the code which you used to load certificate inside lambda?


CSL YAML export of date with original publication date in [brackets]

[X] I have verified that I am on the latest release of BBT (https://github.com/retorquere/zotero-better-bibtex/releases/latest) and Zotero
[X] I am posting a single question, bug or feature request
[X] I have included a descriptive subject of the problem
[x] I am available for follow-up questions and testing

Exporting references
This is a bug report related to reference SWDEYIT2. Export to CSL YAML.
I'm sorry to bring up another issue similar to #811 -- this is really a feature request more than a bug report -- but I'm having trouble exporting references that include dates of original publication. For instance, I'm trying to get output like this:

Amos Vogel, Le Cinéma : art subversif, Paris, Capricci, 2016 [1974].

where 2016 is the date of publication and 1974 is the date of the first edition. Under Zotero's date field, I had put in 2016 [1974].
In BBT for Zotero 4, exporting this reference to YAML produced something like this:
  issued:
    literal: '2016 [1974]'

Under Z5, I'm getting this:
  issued:
    - year: 2016

So the original date of publication is just getting thrown out. Is there any way to have it included again?
Thanks again for an otherwise fabulous piece of software.

:robot: this is your friendly neighborhood build bot announcing test build 5742 ("Merge branch 'master' into 922").

Can you give 5742 a spin?

5742 hangs on "Loading citation keys" when starting up. I can't export anything!

Can you submit an BBT error report?

Done! Debug ID is D1618111088

I can't access the Zotero debug reports, I specifically need a BBT error report.

Oops, sorry about that. How about report ID B2WMGJ5F ?

:robot: this is your friendly neighborhood build bot announcing test build 5764 ("lokijs binary indexes are not as reliable as one might want them to be").

Can you try 5764?

Ok, thanks, looks like it works! I tested it in pandoc-citeproc which now outputs this:

Foo (Vogel [1974] 2016).
Vogel, Amos. (1974) 2016. Le Cinéma, art subversif. Traduit par Claude
Frégnac. Paris: Capricci.

A lot of CSL styles haven't implemented this format
   original-date:
    - year: 1974

but I guess I can take that up with their authors.

Well... the bracket notation isn't any kind of formal standard. If you want the former notation back, this postscript should do it:
if (Translator.BetterCSLYAML) {
  if (reference['original-date'] && reference.issued) {
    reference.issued += ` [${reference['original-date'][0].year}]`;
    delete reference['original-date'];
  }
}


FixFix the window time is not updated when no job is registered

Fix resume job grammar definition is inconsistent
Show Job task Add execution results
JOB allows to define update operations

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 49 seconds
stream load tsv:          533 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>5 Bytes


fix class_dim of flowers dataset

flowers数据集有102个种类。
将resnet文件夹下所有相关的class_dim变量由10改为102。

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


add binary package installation method

do we need any tests for this method ?

review please

There is actually an InSpec already for the binary installation method. It utilizes the tests for the source installation.

@salsa-dev It would be great if you could include ChefSpecs on a later pull request.
Integration tests ought to be enough for now.
Thanks for the contribution.


Deleting AzureAppGatewayAnalytics

AzureAppGatewayAnalytics is deprecated and it avoid sucess deployment

I don't think we should remove it in the regulated environment without replacing it with the workbook, otherwise we are taking away a security lens without putting in a replacement.

The AzureAppGatewayAnalytics  deploy issue is not transient as far I tested in baseline and multicluster.
I didn't execute here, but I'm quite sure is not working the deploy.
I will move this PR to draft, I will see when I have time to do the complete change.

Oh, if it's not transient, then by all means, this is fine to prevent any issues short term.  I just thought you found out that it was.  My misunderstanding.

Added ADO #427693
when deploy:
{
            "type": "Microsoft.OperationsManagement/solutions",
            "apiVersion": "2015-11-01-preview",
            "name": "[concat('AzureAppGatewayAnalytics(', variables('logAnalyticsWorkspaceName'),')')]",
            "location": "[parameters('location')]",
            "dependsOn": [
                "[resourceId('Microsoft.OperationalInsights/workspaces', variables('logAnalyticsWorkspaceName'))]"
            ],
            "properties": {
                "workspaceResourceId": "[resourceId('Microsoft.OperationalInsights/workspaces', variables('logAnalyticsWorkspaceName'))]"
            },
            "plan": {
                "name": "[concat('AzureAppGatewayAnalytics(', variables('logAnalyticsWorkspaceName'),')')]",
                "product": "OMSGallery/AzureAppGatewayAnalytics",
                "promotionCode": "",
                "publisher": "Microsoft"
            }
        }

the error:
{
    "status": "Failed",
    "error": {
        "code": "InvalidOperationArgument",
        "message": "Enabling solution of type AzureAppGatewayAnalytics is not allowed"
    }
}

https://azure.microsoft.com/en-us/updates/azure-application-gateway-analytics-will-be-retired-on-29-february-2024/
To continue monitoring your Application Gateway resources, transition to Azure Monitor Network insights workbook


Start timer for a bucket only after the first set() is called?

First of, thanks for sharing this great component!
I haven't look into your code yet, and is commenting based on your documentation, so you may be doing this already. It seems like we optimize this a little further by only starting a timer for a bucket after the first set() is called upon it. I think the advantage is two folds:

Cut down the overhead when some bucket is never used, and you end up busy replacing an empty bucket with another.
Since the timer is only starting when the first item is put into it, you are essentially normalizing the expiration time for all items toward 20 seconds (per your example), so they are a little more consistent.

Hi @blai very good idea! I'll add that right away.

My "right away" took me nearly 6 months. :D But it is done now! :)


Getting Error During Install on Ubuntu 14.04 LTS

Here's the error, using the scripts to install:
package_linux.sh: line 1: RC: command not found
must specify dir name

thanks for finding the error.  I corrected the first line with the proper command, #!/bin/bash, in commit 867aab57c0cf.  Since it is not marked executable by default, you can either:
chmod +x package_linux.sh
and executing, or execute using:
bash package_linux.sh dirname
as shown in
https://github.com/devsim/devsim/blob/master/scripts/build_ubuntu_14.sh
Note that you need to specify a directory name as an argument.

Thanks for the fix!
When I try to invoke devsim by typing in 'devsim' into terminal I get "devsim: command not found"
This is after using an unofficial docker image as well. Am I doing something wrong here?
BTW, you might want to consider making an unofficial docker image.

I am not aware of the unofficial docker image.  Where do you find it?
You need to either add the path to devsim to your PATH, or use the full path to the executable.
For example:
devsim_linux/bin/devsim

Nevermind, very dumb mistake on my end, typo in the path >.<


chore(ci): バックエンドのCIに型チェックを追加

close #80
Summary
This pull request introduces a new type-checking job in the CI workflow and updates the backend/package.json to include a type-check script and necessary dependencies. The most important changes include adding a type-check job to the CI workflow, updating the backend/package.json to include a type-check script, and adding TypeScript as a development dependency.
CI Workflow Enhancements:

.github/workflows/backend-ci.yml: Added a new type-check job that runs TypeScript checks on the backend directory.

Package Updates:

backend/package.json: Added a new type-check script to run TypeScript checks on the project files.
backend/package.json: Added typescript as a development dependency to support type-checking.

https://github.com/tus-project-team/program-grading-system/pull/82 にて型エラーを直したので、これがマージされ次第本PRをマージする


Tab Problem

When i try to do this
import React from "react";
import { AppBar, Tabs, Tab } from "@material-ui/core";
import PropTypes from "prop-types";
function TabGenerator(props) {
  const { tabsOptions, tabOptions } = props;
  return (
    <AppBar position="static">
      <Tabs {...tabsOptions}>
        {tabOptions.map((e, index) => (
          <Tab key={index} {...e} />
        ))}
      </Tabs>
    </AppBar>
  );
}

TabGenerator.propTypes = {
  tabsOptions: PropTypes.object.isRequired,
  tabOptions: PropTypes.array.isRequired
};

export default TabGenerator;

Calling the component:
<TabGenerator
  tabsOptions={{
    value: selectedTab,
    onChange: handleTabChange,
    variant: "scrollable",
    scrollButtons: "on"
  }}
  tabOptions={[
    { label: "ONE" },
    { label: "TWO" },
    { label: "THREE" },
  ]}
/>

Appears this error in console Warning: Material-UI: you are trying to override a style that does not exist.
Fix the label key of theme.overrides.MuiTab.
Versions:
"@material-ui/core": "4.0.2",
"@material-ui/icons": "4.0.1",

You have tabOptions twice.

@mbrookes no exactly, are two differents variables

@estebansolisd I would ask this on stack overflow and provide a reproduction


ci: use only major version for reusable workflow

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

v6.0.0
GitHub release

Your semantic-release bot :package::rocket:


Revert "Merge common lib"

Reverts bigchaindb/bigchaindb#739

Why was this reverted?
In order to keep all individual commits to preserve attribution.
The squash and merge operation had been used which resulted into one single commit, and a lost of attribution.


Could not find a matching version of package spatie/filament-markdown-editor

In PackageDiscoveryTrait.php line 364:
Could not find a matching version of package spatie/filament-markdown-editor. Check the package spelling, your version constraint and that the package is

This package will get a stable release later today.


Error: Unexpected scalar {VARIABLE} at {VARIABLE}-{VARIABLE}

I have a larger data set that I'm trying to uncompress and getting this error message.
Error: Unexpected scalar f at 11696307-11696343
at Object.decompressScalar (/app/node_modules/zipson/lib/decompressor/scalar.js:98:11)
at Object.decompressElement (/app/node_modules/zipson/lib/decompressor/element.js:15:32)
at Object.decompressStages (/app/node_modules/zipson/lib/decompressor/stages.js:78:28)
at Object.decompress (/app/node_modules/zipson/lib/decompress.js:32:14)
at Object.parse (/app/node_modules/zipson/lib/index.js:16:25)
at Response. (/app/build/app.js:3549:49)
at Request. (/app/node_modules/aws-sdk/lib/request.js:364:18)
at Request.callListeners (/app/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
at Request.emit (/app/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
at Request.emit (/app/node_modules/aws-sdk/lib/request.js:683:14)
at Request.transition (/app/node_modules/aws-sdk/lib/request.js:22:10)
at AcceptorStateMachine.runTo (/app/node_modules/aws-sdk/lib/state_machine.js:14:12)
at /app/node_modules/aws-sdk/lib/state_machine.js:26:10
at Request. (/app/node_modules/aws-sdk/lib/request.js:38:9)
at Request. (/app/node_modules/aws-sdk/lib/request.js:685:12)
at Request.callListeners (/app/node_modules/aws-sdk/lib/sequential_executor.js:116:18)

The error gets thrown here.
let response = parse(data.Body.toString());

Looks like after some more debugging this error was because of the \\ not being escaped in a string. I ended up just escaping them before they got encoded. It's not ideal but it works for my situation.


Fix "dyld[7518]: Library not loaded: libdisk.0.dylib" error on macOS

macOS seems unable to just find the library inside /usr/local/lib without specifying the full path.
Full error:
massimo@Lucifer ~ % disk-analyse
dyld[11283]: Library not loaded: libdisk.0.dylib
Referenced from: <382EEDA0-B36E-3421-99F2-ACD65585B05E> /usr/local/bin/disk-analyse
Reason: tried: 'libdisk.0.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OSlibdisk.0.dylib' (no such file), 'libdisk.0.dylib' (no such file), '/Users/massimo/libdisk.0.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Users/massimo/libdisk.0.dylib' (no such file), '/Users/massimo/libdisk.0.dylib' (no such file)
zsh: abort      disk-analyse

Yes, looks like this is best practice. Thanks!


ARM-based VM with dedicated vCPU / bare-metal runner for SDK benchmarks.

Asked Hetzner support if there are plans to introduce ARM-based cloud VMs with dedicated vCPUs (currently they only offer dedicated x86 (AMD)).
If no plans then we will use another cloud provider (GCP or Hetzner's bare-metal server with Ampere ARM CPU).

#2

Hetzner support reply:

Thank you for your message.
Currently there is nothing planed.
If you require any further help, please do not hesitate to contact us.

Similar story with the GCP. It seems that the only way to guarantee dedicated resources allocation (CPUs) is by using the Sole-tenant nodes (physical servers dedicated to hosting only VMs in your project). But no ARM-based arch available. Hence, no reason to overpay for GCP and we can just use Hetzner's standard ARM64 VM (not dedicated one as we use for x86) with acknowledgement of possible (rare) spikes.


chore!: remove pyarrow-based file reader

BREAKING CHANGE: removed the following: reading using PyArrow, PythonStorageConfig type and StorageConfig.python function, daft.table.schema_inference, refactored StorageConfig and NativeStorageConfig into one type
Includes a drive-by fix on flaky intersect test.
todo:

[x] refactor StorageConfig and NativeStorageConfig into one struct
[x] clean up table_io.py

BTW we can maybe just deprecate the APIs first and deal with this later
In the interest of letting us cut a release first, I might push up an API-only change first since it seems this PR might not yet be ready to go.


New milestone visibility called "SUGGESTED"

Overview
New visibility for milestone called "SUGGESTED" to be added. The new "SUGGESTED" has the following properties

It should be created when a new work plan is generated
It should be possible to remove the suggested as long as no actual date is entered

@dinesh-aot   can you upload the list of suggested milestones so i can take a peek in test?


add k8s owner

Codecov Report

Merging #91 (a96fed7) into main (db0d557) will decrease coverage by 2.78%.
The diff coverage is 28.07%.

@@            Coverage Diff             @@
##             main      #91      +/-   ##
==========================================
- Coverage   54.34%   51.56%   -2.79%
==========================================
  Files          35       36       +1
  Lines        2107     2273     +166
==========================================
+ Hits         1145     1172      +27
- Misses        889     1028     +139
  Partials       73       73

Flag
Coverage Δ

unittests
51.56% <28.07%> (-2.79%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/pipeline/transform/transform_network.go
63.63% <0.00%> (-1.07%)
:arrow_down:

pkg/pipeline/transform/kubernetes/kubernetes.go
14.02% <30.18%> (ø)

pkg/pipeline/utils/exit.go
100.00% <0.00%> (+17.39%)
:arrow_up:

Continue to review full report at Codecov.

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

Deamonset owner

stateful owner

More types ::

@jotak if you can review ... I think that this is what we need for owner

actually just a small remark, it seems like Type is all lower case whereas OwnerType is CamelCase, would it be easy to also have CamelCase for Pod / Service / Node (so that it matches the k8s kinds) ?

actually just a small remark, it seems like Type is all lower case whereas OwnerType is CamelCase, would it be easy to also have CamelCase for Pod / Service / Node (so that it matches the k8s kinds) ?

I changed to CamelCase to match :-)


TapDectector and Tapable conflict

[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.1 19B88, locale en-NZ)
flame: ^0.17.2
It seems that there is a conflict between TapDetector and Tapable mixins.
If I use the TapDetector Mixin with my BaseGame class like below.
extends BaseGame with TapDetector
I don't see any tap events registering in my Tapable SpriteComponent. When I remove the BaseGame mixin, everything works.

@erickzanardo I think you might know the issue here

tbh I don't @luanpotter, rubberbird discovered this, and reported on our Discord, and I asked him to open this issue, so we don't forget, but I need to investigate, my guess is that the TapDetector mixins is overriding the BaseGame method, removing its behaviour which calls the tapable handlers.

@rubberbird I didn't find the solution, but I think I find a workaround. Don't make your game instance use the TapDetector mixin, and just override the onTap* methods, and make sure to call the super.
This effect is happening because BaseGame it self includes TapDetector and when you make your instance use it too, we create that multi inheritance problem.
We probably need to find a better solution for this, but I will need to give this some thought, but at least there is this workaround.

@erickzanardo this is fixed by 0.22 right?

Yeah


LUCENE-10418: Optimize Query#rewrite in the non-scoring case.

@rmuir I'd be curious to get your thoughts after recent changes that applied some of your feedback.

I plan to merge in the coming days if there are no objections.

I like it! to me it looks much cleaner than before. sorry for the huge delay, i missed the notification.

Thanks Robert!


Add support for trees

It would be nice to have a list of trees on The map.

What's the use case here?


Laravel 5.4 update breaks package

A change to L5.4 results in:
FatalThrowableError in FpdfServiceProvider.php line 21: Call to undefined method Illuminate\Foundation\Application::share()
The Laravel upgrade docs suggest the following:

The share method has been removed from the container. This was a legacy method that has not been documented in several years. If you are using this method, you should begin using the  singleton method instead:
$container->singleton('foo', function () {
return 'foo';
});

I tried to tweak FpdfServiceProvider but I'm afraid this is beyond me :)

For now ...
you can create a provider, extending the original provider.
create:
"/app/provider/fpdfCorretor.php"
with:
<?php

namespace App\Providers;

use Anouar\Fpdf\Facades\Fpdf;
use Anouar\Fpdf\FpdfServiceProvider;

class FpdfCorretor extends FpdfServiceProvider
{
    public function register()
    {
        $this->app->singleton('fpdf', function($app) {

            return new Fpdf();
        });
    }
}

And change the reference in /config/app.php
//Anouar\Fpdf\FpdfServiceProvider::class,
        \App\Providers\FpdfCorretor::class,

I used this fix, but when then I ran into this exception:
FatalErrorException in Facade.php line 221:
Call to undefined method Anouar\Fpdf\Facades\Fpdf::SetFont()

check now the fix for this issue was merged on this PR https://github.com/anouarabdsslm/laravel-fpdf/pull/24

thanks! can you also release a version with the fix? In order to reference it in my laravel project's composer.json file.

@michaelschoellhammer  sorry for the delay. the new version released


ORC List/MapVectorBatch support filter

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #5306
Problem Summary(Required) ：

orc::ListVectorBatch/MapVectorBatch to support filter method.

@MergifyIO backport branch-2.2


Bumps kotlinx-coroutines-core from 1.3.1 to 1.3.9-native-mt-2.

Installed dependabot and I keep getting this PR in my Android project that uses kotlin coroutines.
Bumps kotlinx-coroutines-core from 1.3.1 to 1.3.9-native-mt-2.
Package manager/ecosystem
gradle/maven
What you expected to see, versus what you actually saw
It would have been nice for this PR to be to 1.3.9 directly instead of this experimental native-mt-2 dep from kotlin.

Hmm, yeah it seems like our version resolver doesn't correctly recognize that as a prerelease, I'm not super familiar with Gradle prerelease/experimental releases. Is it common to append a tag like that? We may be comparing lexically when we shouldn't.

@jurre I don't know. Maybe this is a one off thing that is just best for me to skip this.
I asked about this on kotlin slack since I've never heard of native-mt-2 before and this is what someone said
This user https://github.com/gildor said this:
This is a special branch with developer preview of support multithreading coroutines for Kotlin native
if you are not interested in multithreading on native, use standard 1.3.9
More details https://github.com/Kotlin/kotlinx.coroutines/issues/462

Is it common to append a tag like that

I wouldn't say that it's very widespread, but definitely not unique
I think depedabot shouldn't suggest update if 1.3.9 and 1.3.9-something versions are available to update on 1.3.9-something from 1.3.8

This is unfortunately rather annoying and could also lead to someone who's not paying close attention (or has auto-merge enabled for patch releases) accidentally merging this.

I keep running into this as well. Can ignore be used as a workaround? I'm not sure what a proper value for versions would be such that it means ignore all versions ending with *-native-mt*.
I also have the added complexity that I'm using a def coroutinesVersion in my build.gradle to define the coroutine version once for multiple artifacts, so even with the correct versions above, I'm not entirely sure whether the dependency-name should be coroutinesVersion or org.jetbrains.kotlinx:kotlinx-coroutines-* or something else. Any guidance there would be appreciated!

@jurre if you're looking for more examples:
https://github.com/iSoron/uhabits/pull/814
https://github.com/iSoron/uhabits/pull/754
https://github.com/iSoron/uhabits/pull/753
https://github.com/iSoron/uhabits/pull/746
https://github.com/iSoron/uhabits/pull/745
Some of these are especially bad because they prompt to update to a version that should have a lower precedence than the current version (e.g. https://github.com/iSoron/uhabits/pull/754).
According to https://semver.org/spec/v2.0.0.html#spec-item-9 it's pretty clear that these are pre-releases. It looks as simple as trimming the part after the + (if there's any, to remove build metadata), then mark the release as pre-release if it contains a -.

Is this going to be fixed sometime soon? Or is there a workaround?


Lightning-SDK  vs  Lightning-SDK-blueprint

I had to follow the "Lightning-SDK-blueprint" approach and it worked for me -
https://github.com/WebPlatformForEmbedded/Lightning-SDK-blueprint
Anyone know the difference between -
Lightning-SDK-blueprint
Lightning-SDK

Lightning-SDK-blueprint is stand alone app that uses Lightning-SDK

@wilberhdez  Does this answer your question? If so can we close the ticket?

Closing this issue for now, please reopen if still relevant.


RayPlugin was not exported

I was following the tutorial in your README.md file, but I got an error from my IDE saying that RayPlugin was not found in vue-ray, I checked your index.ts and noticed that it wasn't imported and exported, so I created this PR where I simply proceeded as you've done with RayMixin.
I would like to add tests to my PR, but noticed that there are no tests regarding the exposure of parts, so I decided to open the PR even without tests, if they are needed, I will try to implement them.

Thank you!


Issue using admin panel after update to 0.5.5.

Recently updated to orionjs:core and Meteor <IP_ADDRESS>, console returning this now:
Route dispatch never rendered. Did you forget to call this.next() in an onBeforeAction?
debug.js:41 Received error from server:  Malformed subscription
debug.js:43 For:  Object {msg: "sub", id: "PBsrZTQZmiyQzD4vm", name: Object, params: Array[0]}id: "PBsrZTQZmiyQzD4vm"msg: "sub"name: Objectparams: Array[0]__proto__: Object

Are you using this function?
orion.admin.addAdminSubscription([arguments])

I am not.

I'm not having this error. Please specify more

The site is fairly basic, the only references to Orion are the a number of dictionary.addDefinitions in a dictionary.js:
orion.dictionary.addDefinition("homeRight", "Home Right",
  orion.attribute('froala', {
    label: "Home Right",
    optional: true
  })
);

orion.dictionary.addDefinition("newsLeft", "News Left",
  orion.attribute('froala', {
    label: "News Left",
    optional: true
  })
);

And calls to the dictionary (home.html):

{{{dict 'homeRight'}}}

Buy the Book

When does the error appears? on the admin?

Appears in the console after loading the page after browsing to /admin which redirects to /admin/directory. The page has the star background with a spinner, no elements displayed.

Strange, try to clean the project earsing dependecies.
Anyone else having this issue?

 I remember the orionjs:filesystem was stuck on 0.0.13 (altough I meteor updated it more than once). After the cleansing, I got the proper 0.0.15 version, and the above error was gone. Does that help?

That might be the error because filesystem 0.0.13 is not compatible with orion 0.5.5.

orionjs:filesystem wasn't installed for some reason. I installed and updated to 0.0.15, works fine now.


ee stack upgrade - not detecting mysql

This is with the current version:
root@www:~# ee  -v
EasyEngine v3.1.4
Copyright (c) 2015 rtCamp Solutions Pvt. Ltd.
I have 2 WP sites and then when I run stack upgrade, it claims MySQL is not installed.
root@www:~# ee stack upgrade
During package update process non nginx-cached parts of your site may remain down
Do you want to continue:[y/N]y
HHVM is not installed
MySQL is not installed
Updating packages, please wait...
Successfully updated packages
You have new mail in /var/mail/root

After that, the sites are working normally and services are shown as detected.
root@www:~# ee stack status
nginx     :  Running
php5-fpm  :  Running
mysql     :  Running
postfix   :  Running

I had the same experience on another system (where for example nginx was v1.6 and I wanted to upgrade it to v.1.8). It worked, but I find it strange that MySQL was shown as not installed (is it because we changed from Percona or something?). I installed this stack with EE v2.x and didn't touch it since then.

Hello @rippler ,
Currently ee stack upgrade upgrades only MariaDB, as with latest EasyEngine we only supports MariaDB.
The message on screen was creating confusion, so we updated message to MariaDB is not installed in latest EasyEngine.


Module Not Found: streamlit_chat when deploying in Streamlit Cloud.

Hi.
I build a chatbot app to deploy in streamlit using streamlit_chat, but when i'm trying to build the app i get the following error:
from streamlit_chat import message
ModuleNotFoundError: No module named 'streamlit_chat'
¿what could be the problem?
Thank you!

Hi @thomasrife, it's looks like you're missing the requirements.txt file, please take a look at their documentation.
Also, in your requirements.txt file, there should be a line streamlit-chat and any other dependencies you would like to use.

Python requirements files should be placed either in the root of your repository or in the same directory as your Streamlit app.


chore(ci) no ghcr when missing password input

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.125%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7588

Relevant Lines:
8327

💛  - Coveralls


[BUG] Portable?

Describe the bug
In your release, 'portable' is mentioned. But it doesn't have such functionality.
%userprofile%\AppData\Roaming\hydralauncher
%userprofile%\AppData\Roaming\hydra
These shouldn't be here. They should be stored at least alongside the executable file. I'm puzzled. Why mislead? 🤡
Expected behavior
These shouldn't be here. They should be stored at least alongside the executable file.
--user-data-dir="UserData"
Operating System
Windows 11, 10
Hydra Version
<IP_ADDRESS>
Additional Information
Portable mode refers to the ability of software applications to run on any computer without needing to be installed on the system. This means that the application can be executed directly from removable media such as USB flash drives, external hard drives, or even cloud storage services, without leaving traces on the host machine once it has been removed. Here's how portable mode typically works:
No Installation Required

Direct Execution: Portable applications do not require installation. They can be run directly from the location they are stored, whether it's a USB drive, an external hard disk, or a network share.
Self-contained: All necessary files, libraries, and dependencies required by the application are included within its directory structure. This ensures that the application does not rely on pre-installed components on the host system.

Independence from System Configuration

Configuration Files: Portable applications store their configuration settings, preferences, and data within their own directories rather than using system-wide locations like the Windows Registry or Unix/Linux system directories.
User Data Isolation: User-specific data, such as settings, profiles, and saved games, are kept separate from the application itself, allowing multiple users to use the same portable application without interfering with each other's data.

Minimal Impact on Host System

No Registry Changes: Portable applications should not modify the system registry or leave behind entries after being removed.
Temporary Files: Any temporary files created by the application during its operation should be cleaned up upon exit, leaving no trace on the host system.

Cross-platform Compatibility

While not always a requirement, many portable applications are designed to be cross-platform, meaning they can run on different operating systems (e.g., Windows, macOS, Linux) without modification.

Examples of Portable Mode Usage

Software Demonstrations: Developers might distribute software in portable mode for potential customers to try out without committing to a full installation.
IT Professionals: IT staff may carry diagnostic tools and utilities in portable form on USB drives for troubleshooting computers across different environments.
Personal Use: Users who frequently switch between computers or prefer not to install software on their primary machines can benefit from portable applications for everyday tasks.

To summarize, portable mode is designed to enhance flexibility and convenience by allowing software to run independently of the host system's configuration and without leaving a footprint once removed.

Yes, we are aware of that limitation, it was mentioned on our release notes article. We plan to soon make the portable version 100% portable


Refactoring to leverage Java 17 and avoid deprecated classes

Summary
I checked every FW class to find something deprecated or only relevant for Quarkus versions. This PR contains unimportant refactoring I did while checking classes. Infinispan classes are marked for removal: https://access.redhat.com/webassets/avalon/d/red_hat_jboss_enterprise_application_platform/8.0/javadocs/org/infinispan/commons/configuration/XMLStringConfiguration.html https://www.javadoc.io/doc/org.infinispan.protostream/protostream/5.0.4.Final/org/infinispan/protostream/annotations/AutoProtoSchemaBuilder.html
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[x] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[x] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests

run tests


fix: add build/src/cls in output files

@ofrobots It's the list of "reachable" d.ts definitions from index.d.ts. This is being detected by the install test which is run on each PR, since it type-checks (and only type-checks, not runs) the statement import * as trace from '@google-cloud/trace-agent'. What we could do to avoid the much more important problem of ensuring that the correct .js files are included is to actually run the code in the install test.


Why is this a template repository?

This just makes it difficult to contribute to, as repositories built by this template aren't able to contribute back to the original template.

To revert to a normal repo uncheck the checkbox here:

I wasn't aware of this limitation. Un templating this.


Export storybook middleware

It'll be much easier to integrate storybook into existing systems for users if storybook is available as an express middleware. This issue is an example use case.

Since this is just the our same logic. I think we can proceed with this.
And also, building storybook also works fine.
I am good with this. Merge and make a release.

I'm not sure best to import this.  require("@kadira/storybook/dist/server/middleware").default seems kinda brittle, no?


Release 2.0.2

Apart from inline comments, the following are the general comments:

Add author in new files as well as add @SInCE.
Update the author with your name in all of the touched files.
We need to align of code style/format. If needed I can provide one which has been originally used in the older releases.
Update README.md for the addition of Postgres persistence layer. Currently, it mentions only MySQL implementation.

Reply:

I am curious about the reason for adding @SInCE
Done
Is the style/format referring to the formatter in this comment ? https://github.com/jas34/scheduledwf/pull/32/files#r1250801445  I am not aware, please elaborate.
Done

Apart from inline comments, the following are the general comments:

Add author in new files as well as add @SInCE.
Update the author with your name in all of the touched files.
We need to align of code style/format. If needed I can provide one which has been originally used in the older releases.
Update README.md for the addition of Postgres persistence layer. Currently, it mentions only MySQL implementation.

Reply:

I am curious about the reason for adding @SInCE
Done
Is the style/format referring to the formatter in this comment ? https://github.com/jas34/scheduledwf/pull/32/files#r1250801445  I am not aware, please elaborate.
Done

Even I am not aware of the person who got tagged (:P). My intention was to add Since for Java docs.
I am referring to formatting change like this.


Add memcache serialization/deserialization methods

The serde methods are based on those found in pymemcache, but with
added features to make them compatible with pylibmc.
We can use this stuff in listing service, thing service, and r2. I'm not sure that baseplate.context.memcache.lib is the correct place to put the serde methods. Let me know if you think they should go somewhere else (maybe a base level module in baseplate?).
👓 @keithblaha @spladug

Thanks for looping me in- overall this is really useful and it'd be nice to have for listing service. I'm thinking about if we want to enforce that services must use this exact serde or not since there is no way provided to override- and I think that we should, although I think it will be used relatively rarely.

💇

CI's failing. For the "imported but not used" in _compat, you can just add them to __all__.

Did you have any thoughts on pickle/legacy annotation here? I'd really like to discourage greenfield services from using pickle if possible.

I could add a note to the docstring. I think the best thing to do would be to provide an alternative set of serde methods that don't use pickle.

💇

Any thoughts on where the serde methods should live? Seems fine in baseplate/context/memcache/lib.py but I could also see putting them in like baseplate/utils/memcache.py or something more general like that.

baseplate/context/memcache/serde.py ?

💇 I can't tell if my build failure is because of spellchecker or something else. Any idea how to fix that?

Yup, it's the spellchecker. For any spelling errors: fix 'em! For legit but unrecognized words: add 'em to the dictionary https://github.com/reddit/baseplate/blob/master/docs/words.txt

Build is passing now. I couldn't really figure out where some of the spelling errors were coming from, the output was like:
baseplate/context/index py seplate/context/index
baseplate/context/index func
writing output... [100%] quickstart
Spelling checker messages written to /home/vagrant/baseplate/build/spelling/output.txt
build finished with problems.
make: *** [tests] Error 1

and "py" and "func" only existed as ":py:func:" within a docstring.

Merged as https://github.com/reddit/baseplate/commit/7cc47cdbe14fe83e4c76549332e775d956005430


Developments not fully integrated yet

Note: Comments below may be outdated, just opening a PR to check and share changes.

Allow user to specify unique keys upon instantiation.
Add order_by input which can be passed to the parent node query.
Verbose submission now has a formatted table.

Yup, seems some of the developments here have already been integrated. ^^ So this PR will most likely only be for the formatted table of the verbose submission.
Not sure if we should keep the changes regarding the exceptions... I don't like a bare Exception catch, so maybe it's good to be more specific, but not catching the exceptions is a pain.

Ah, seems the verbose table is also already integrated, now it's just duplicated 😂
So let's just think on the exceptions and maybe close this PR.


Add gosimple linter to .travis.yml. Fix new staticcheck issue.

gosimple is a neat little code linter from the author of staticcheck. It catches code quality issues like the following:
// should be if boolVar
if boolVar == true {}

// should be time.Since(x)
time.Now().Sub(x)

// should be return err
if err != nil {
    return err
}
return nil

The full list of checks is available here.
It only found one issue, which is very good for a project this size. Kudos to all the contributors! 💯

LGTM - thanks!


Inherited member with same name as enclosing class not shadowed by specification.

Example:

class C {
 D() { print("D"); }
}
class D {
 foo() { D(42); }
}
main() {
 new D().foo();
}

According to the specification, the D-call in foo() should be rewritten to this.D(42) and print 42. This follows from the "Unqualified invocation" section in the specification", because "D" is declared in the current scope, but is not one of the listed cases, so it falls through to the "otherwise" case also covering where D is not declared.
Since having just one Scope is a goal, the "D" in foo() should refer to the enclosing class, and calling that is an error (as should a library prefix be, which the spec mentions as a problem).
A solution could be adding a third "dot" for the case where the identifier is declared, and make it an error if it's not handled by the previous two dots.

No milestone now: This will not block Dart 2.
We should add the missing case to the abovementioned list.

That is actually already fixed:
item
If $id$ is a type literal, then $i$ is interpreted as a function expression invocation (\ref{functionExpressionInvocation}) with $(id)$ as the expression $e_f$.
\commentary{
The expression $(id)$ where $id$ is a type literal always evaluates to an instance of class \code{Type} which is not a function.
This ensures that a run-time error occurs when trying to call a type literal.
}


Large messages fail to deserialize / panic

Problem
When sending a message that is more than 300 kilobytes in size, received message will fail to deserialize on the client and crash the server.
On the client it prints this message:
2024-06-04T06:51:16.980360Z ERROR lightyear::client::message: Could not deserialize message

On the server it crashes with this error:
thread 'main' panicked at lightyear\src\server\connection.rs:618:30:
Could not decode ClientMessage: Error(Eof)
Encountered a panic in exclusive system `lightyear::server::networking::receive`!

Example
I made an example that will send 300 kilobytes of random data from client->server and from server->client on connection
https://github.com/Serverator/lightyear_playground/tree/large_message

cargo run -- server to start the server
cargo run -- client to start the client

Server and client will show problems as described above
Might be related
I've had this error message in my log when I first tried it on commit 80f460cd. As commit is 4 days old it might have been already fixed.
2024-06-04T06:27:23.865439Z  INFO lightyear::connection::netcode::client: client connected to server
thread 'main' panicked at lightyear\src\channel\receivers\fragment_receiver.rs:98:26:
index out of bounds: the len is 89 but the index is 89
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Encountered a panic in exclusive system `lightyear::client::networking::receive`!
2024-06-04T06:27:24.336545Z  INFO lightyear::transport::webtransport::client_native: WebTransport connection closed. Reason: client requested disconnection. Shutting down webtransport tasks.

I'll have to check but I think it might not be possible to send messages of this size.
The fragmentation uses a u8 to track the number of fragments, so the max is 255 fragments.
Each fragment is roughly 1200 bytes so the max would be 255 * 1200 = 300KB.
Maybe I can change it to an u16? I would have to benchmark to see if it's worth, because it should be very rare to send messages of this size

In that case this behavior should be indicated and panic before sending 🤔

In that case this behavior should be indicated and panic before sending

I would like it if lightyear logs an error and tries to continue gracefully wherever possible.

In that case this behavior should be indicated and panic before sending

I would like it if lightyear logs an error and tries to continue gracefully wherever possible.

Good suggestion! Fixed here: https://github.com/cBournhonesque/lightyear/pull/454


How to specify the working dir in config file?

I have multiple task, and one task should be running in another working dir.

Just added the cwd field in config, now it's in master. I will add docs and release a new version tomorrow.


ExcludeApp element currently does not work to exclude Teams

per MO175715 this functionality is currently broken and this page should be updated to state the the steps listed here do not currently work and will not until around April 9th.

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

ID: 7936ca5e-c658-4956-b174-99eaf1eb67d4
Version Independent ID: 9ecc0b61-a941-f5e1-96c3-d32fae59eacd
Content: Deploy Microsoft Teams with Office 365 ProPlus
Content Source: DeployOffice/teams-install.md
Service: o365-proplus-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

@CCDFischer -- the article was updated last Tuesday to address that functionality issue (see the Important callouts in the article). The sample xml code shows the current workaround. Essentially, there are several places you have to include the ExcludeApp ID="Teams" line in your xml file, depending on what your scenario is.

@DHB-MSFT -- that is not how the advisory reads, perhaps it is the advisory that needs updated but as of now it appears as though using the exclude element for teams simply will not work at all until a new ODT is rolled out in April. As we are excluding Teams from our install this is fairly important to us to get and keep working.
The call out that you reference above is the same as how OneDrive (groove) currently works as well, we need to include the exclude element for our Project and Visio installs as well as our base Office install to prevent OneDrive from being installed with Office.

@CCDFischer -- thanks for the feedback on the wording of the advisory. I'll see if they can update it to make it clearer. The ExcludeApp element does work for Teams, but only if done in the manner outlined in the article.

I've added the exclusions where mentioned but Teams is still installed during a fresh install. My config content is below. Am I doing something wrong?

@quinla01 -- to have your xml sample appear here in the comment, I think you need to add ``` as the first line and as the last line of your xml sample.

Thanks for that. I've just adjusted and it seems to have worked.

This still isn't working.  Any eta?

I had this issue. Turns out I needed to download the new ODT and extract the new setup.exe. Use that new setup.exe with the config.xml. It should work after that.

It's always a good idea to make sure you're using the most current version of the Office Deployment Tool, which is available here -- https://www.microsoft.com/en-us/download/details.aspx?id=49117.

I'm having the opposite issue.  Teams is not installing for me as part of even the latest Monthly channel install and I'm using the latest ODT.  I don't have Teams in any exclude mode in my configuration files, though I do have Groove in there as an exclude and that is working.  Ironic that when I actually want Teams to be part of the O365 install it isn't.  :)

I have to amend my earlier comment.  Apparently I had a "System-Wide Teams Installer" in my Programs and Features list and that was preventing the full O365 install and even the stand alone Teams installer to run.  I removed that, removed the full O365 suite and ran my ODT install again.  I had to reboot before Teams would actually launch and I noticed that the "System-Wide Teams Installer" is back.  Kind of a convoluted way to go about integrating Teams into the O365 install.

  That will actually kick off the Teams installation. We'll be updating the article to make that clearer.

I did a full restart so that took care of my logging off and on.  Thanks!


CREATE DATA TYPE ddic fix.

fixes https://github.com/abaplint/transpiler/issues/1179

published as 2.5.76


Set the Delve port dynamically for remote debug

Currently the Delve port can be defined either randomly or hard-coded by hand.
Hard-coded port allows to know which port to use in remote debug configurations, but in practice with many Go projects, it would be great to set the Delve port in debug configurations.
Luckily nvim-dap allows to set an on_config callback for an adapter, which can be used to resolve the adapter settings when a debug session starts. This PR implements this nvim-dap feature, which can be used with the support of VSCode launch.json files to greatly enhance the developer experience.
This potentially resolves https://github.com/leoluz/nvim-dap-go/issues/73

Hello @leoluz, is it possible to get some feedback please?

Hard-coded port allows to know which port to use in remote debug configurations, but in practice with many Go projects, it would be great to set the Delve port in debug configurations.

I am not sure that I understand exactly what is your use-case. You don't have to setup a different Delve port per Go project. They can all use the same port configuration unless you are trying to debug different Go projects at the same time. Can you please clarify what your use-case is?

I work on projects deployed on a local k8s cluster (k3d), most of them have a binary for an HTTP server others for workers (kafka consumers for example).
Also I always have multiple apps running in my cluster, each app is deployed in a development docker image with a its own delve port. Each port must be unique to be able to setup port forwarding on my local host.
It's really more convenient for me to keep the delve clients configuration inside my projects instead of having to set a long list of config entries in neovim's config.

Same here, I am looking forward to such addition. I have a setup that uses telepresence that randomly allocates a different free port so I can use it to debug my application inside a cluster with a very distributed system.

@cyb3rd4d Tks for the explanation. I'll try to review your PR soon.

Thank you for your contribution! Can you please also update the doc/nvim-dap-go.txt file?

Oh yes sorry I forgot this one!

I rebased and added config validation + completed the doc.


Testresults

uhm? :-)

Sorry! Opened this against the wrong repo, we are adding jasmine test functionality to webpackbin but didn't mean to open a pull request.

Ah, cool :-)


How can i update the main view controller

i have a table inside the main view controller.
i want, when i open left menu, touch on value and reload table inside my main view.
How to do that ?

[self.dynamicsDrawerViewController setPaneViewController: animated:false completion:nil;]
and... its not a issue|||||
you should download the example project and play around

i dont want to reload entire view, only a table.
i play around with your example, and only reload entire view ( PaneView ).

i know this is not a issue, but were i ask this thing for you ? Thanks


multi-node: printing execution error

This change is printing error message of openssl when openssl fail and execution using system function.

Hey - Thanks for your help with this. I'm closing this PR because this repo is not in alignment with current versions of Kubernetes, and is no longer active. The CoreOS Kubernetes documentation has been moved to https://github.com/coreos/tectonic-docs, where it will be published and updated.


Add Windows Explorer Tabs

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Imagine having a tab bar in the top of the file explorer window like on a browser so you do not have to open multiple windows to move.  (there is an extension called QTtab bar but it does not work that well
If you'd like to see this feature implemented, add a 👍 reaction to this post.

dup #10

@alannt777 please use duplicate of #....
Than github marks the issue as a duplicate

Check out https://thewincentral.com/microsoft-planning-to-bring-windows-10-sets-feature-back-from-grave/, maybe that'll help?


Drag and Drop example has import issues

Hi, I'm getting acquainted with this library for the first time - looks nice! When following the drag and drop example I'm met with an error for one of the imports.

I noticed this is an import available from reactabular-dnd, but when I try to use that module instead I get the following error.

I struggle to understand exactly how to tie in the DragDropContext bit, or even really know what it's needed for. Have I overlooked any documentation where this is explained? Otherwise, could you help point me in the right direction? Let me know if you need anything further from me to help diagnose, thx!

Hi,
Can you set up a small standalone example I can run? It sounds like maybe some underlying dependency has changed their API signature somehow which is breaking the module. It's possible we may have to rewrite some code to work with the newest version.

No problem, try this branch out
https://github.com/seanforyou23/patternfly-react-seed/tree/reactabular
If you clone it, run yarn && yarn start, then you should see a blank screen with the error in the console.
If you open the project in an IDE you'll see this line flagged as error.
Thanks for looking into it!

Any solution for this ?
react-dnd said that they do no longer use DragDropContext. It was removed since version 9.
How can we solve this ?

@souuu I have added the below to my projects package.json to ensure we do not use version 9. You could try that as a work around for now?
"resolutions": {
"react-dnd": "8.0.2"
},


fix: remove unused dependency react-addons-css-transition-group, resolves #378

Closes carbon-design-system/carbon-components-react#378
Removed react-addons-css-transition-group from dependencies as it was no longer being used and causing warnings with React 16.
Changelog

Removed react-addons-css-transition-group from package.json and updated the yarn.lock file. I doesn't look like the yarn.lock file was updated via https://github.com/carbon-design-system/carbon-components-react/pull/366.

Thanks so much @alisonemiller!


Merge travis branch

Keep the configuration in a single place.
Also should make travis builds a bit faster

travis branch can be removed


Problem with setting up the person/clothes pairs.

Thank you so much for sharing this great work!
I now have a problem with setting up the person/clothes pairs.
No matter how I try to set up the test_pairs.txt and run the inference with unpaired mode (e.g. as follows), the try-on result is not what I expected ( the person is in another clothes).
00055_00.jpg 01967_00.jpg
target clothes

try-on result

I really appreciate if anyone could help or share some insights. Thank you!

I face with the same problem.Have you known how to solve it?

Hi,I face with the same problem. Have you known how to solve it?

Hi,I face with the same problem. Have you known how to solve it?
@mkzl1213
In the unpaired setting, for each person/clothes pair in test_pairs.txt you have to warp the clothing as a preprocessing step and put the warped clothing and its mask somewhere like VITON-HD/test/unpaired-cloth-warp


Add Emoji Support

I can add emoji section, it will help users to enhance their experience while using this web app. With the help of emoji's users can try to understand the emotions of another person with whom they talking.

I want to work on this feature, So please assign me @PradyumnaKrishna .

hi @PradyumnaKrishna  kindly assign this task to me under GSSoC23. i can contribute to this task and can deliver on time.

Before starting working on this, @Akshatbhardwaj15 will you provide me a concept of how this will work? Just explain me a bit.

Okay, So when I tested your chat application I figured out that whenever I try to send emojis to someone, it doesn't work. So we can integrate various javascript plugins which add emoji support for the text areas

You are ready to go, I am assigning this issue to you. 🙂


Strong mode analyzer ignores function types in conditional operator

Dart VM version: 1.14.0-dev.4.0.
This is OK:
typedef int ProcessFunction(int v);
int foo(int a) => a * 2;
int bar(int b) => b * 3;

ProcessFunction process;

void main() {
  var t = true;
  if (t) process = foo;
  else process = bar;
}

However this
typedef int ProcessFunction(int v);
int foo(int a) => a * 2;
int bar(int b) => b * 3;

ProcessFunction process;

void main() {
  var t = true;
  process = t ? foo : bar;
}

results in WARNING: t ? foo : bar (Function) will need runtime check to cast to type (int) → int.

There are two related issues here.  The first is that the analyzer (normal mode) does not currently implement least upper bound for function types, see:  #23381 and #23663  .
The second is that we shouldn't really be using the dart spec version of least upper bound, since it produces very bad results in lots of places.
https://github.com/dart-lang/dev_compiler/issues/288
https://github.com/dart-lang/sdk/issues/19425
Getting a DDC lub implemented is on my list of things to do fairly soon.

@leafpetersen is an improved LUB something we want to tackle in Q1?

Yes.  It doesn't have to be complete, but the function case comes up surprisingly often, in addition to the EfficientLength kind of issue.

I've merged https://github.com/dart-lang/dev_compiler/issues/288 into this issue. There's some useful discussion on that bug, but the code is part of analyzer now.

I think @munificent fixed the first part (LUB in normal mode) ... I think we still need a strong-mode LUB, as the spec version is not sound.

merging this into https://github.com/dart-lang/sdk/issues/25802


测试案例报错了，好兄弟们

这样非常槽糕，感到很沮丧。请尽快帮忙修复。

demo has now been rebuilt using Portainer BE 2.0, Extensions are now EOL.

Thank you very much.


do not add newline to all strings

cc/ @AucaCoyan
this PR moves the add_newline_at_end_of_file outside of format_inner.
the responsibility is not those of format_single_file, format_string does not add a newline to all strings anymore.
tests have been fixed :+1:
one thing i'm struggling with is the following test
    #[test]
    fn remove_additional_lines() {
        let input = "let one = 1\n\n\n";
        let expected = "let one = 1\n";
        run_test(input, expected);
    }

which does not pass without the \n in expected :thinking:
any idea? :confused:

Hello! This test is for

If at the end of the file are there any multiple empty lines, just leave 1 line
I think we can remove it since format string doesn't do this from now.

Should we test the function format_file? we can write a file, format it and the test it. After that, we can delete the file.
What do you think? We can move that to another time too!

As it is now, it is very valuable!
Later we would need to fix the failing test, but nonetheless is a step forward!
I'm allright with a merge right now 😄

greeeeat, thanks @AucaCoyan for the review :relieved:


Implement Regex engine on PCRE2

This is a re-issue of #12840 which was reverted due to errors. The updated branch includes several bugfixes and enhancements.
This is an initial implementation of a Regex engine backend based on PCRE2 (see #12790).
All existing regex specs succeed with PCRE2 🎉
This is an MVP and does not include JIT compilation (I'm planning a follow-up).
PCRE2 is currently not automatically selected. It's opt-in via the compiler flag -Dforce_pcre2.
If you have Crystal software that makes heavy use of regular expressions, please give this patch a try in order to identify potential issues in practical use cases. Performance comparisons are also welcome.

We're using the 8-byte version of PCRE2, so the library name is actually libpcre2-8 (this is also the package name on some systems).
pkg-config needs the .pc configuration which may require installing the devel package of libpcre2.

Significant changes from #12840:

Added CI jobs to test against both PCRE and PCRE2
Removed PCRE2 as default. It's only available as opt-in for now.
Allocate matchdata using the GC so it gets automatically collected
Improved error message for match errors

Nice! So this one seems to be working better than the last one. I ran the specs on https://github.com/jwoertink/CrystalEmail to test
❯ /home/jeremy/Development/crystal/lang/bin/crystal spec
Using compiled compiler at /home/jeremy/Development/crystal/lang/.build/crystal
....

Finished in 222 microseconds
4 examples, 0 failures, 0 errors, 0 pending

❯ /home/jeremy/Development/crystal/lang/bin/crystal spec -Duse_pcre2
Using compiled compiler at /home/jeremy/Development/crystal/lang/.build/crystal
....

Finished in 136 microseconds
4 examples, 0 failures, 0 errors, 0 pending

No issues on my M2 via nix-shell (requires adding pcre2 to stdLibDeps but that's about it)


text format parsing error

Hi,
Node exporter throws out this error message:
level=error collector=textfile msg="failed to collect textfile data" file=smart_metrics.prom err="failed to parse textfile data from "/var/lib/node_exporter/textfile_collector/smart_metrics.prom": text format parsing error in line 6: expected float as value, got "0,000000e+00""
Line 6 in the prom file looks like this:
smartmon_current_pending_sector_raw_value{disk="/dev/sda",type="sat",smart_id="197"} 0,000000e+00

Hi,
Any advice for this problem ? I have the same issue.
Regards

I also had this and it should now be fixed or properly handled after the rewrite.


Translation Updates - Lab 10

Lab 10, Exercise 1, Task 1, Step 6:    "학생" should be changed to "Student". This is login username so Korean is not allowed
Lab 10, Exercise 3, Task 4, Step 7:    "학생" should be changed to "Student". This is login username so Korean is not allowed
Lab 10, Exercise 3, Task 4, Step 10:    "의료" should be changed to "medical". This is database name

Blocked Locfile is at PHO - needs to be at HB

Discussed with the PM - update made to the older HB locfiles and this will seed the TM when handedback

Discussed with the PM - update made to the older HB locfiles and this will seed the TM when handedback


fix measurement tests

After merging #44 the tests for comparisons and measurements added to main failed. This fixes them.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


How can I expose grafana, prometheus and alert manager via ALB

I saw a couple of example on how to expose grafana via ngnix. But I am looking for a way to expose grafana, prometheus and alert manager via ALB ingress
apiVersion: v1
kind: Service
metadata:
  name: qa-prometheus-operator-grafana
  namespace: monitoring
  labels:
    app.kubernetes.io/instance: qa-prometheus-operator
    app.kubernetes.io/name: grafana
spec:
  type: NodePort
  ports:
  - port: 80
    protocol: TCP
    targetPort: 3000
  selector:
    app.kubernetes.io/instance: qa-prometheus-operator
    app.kubernetes.io/name: grafana
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: grafana-ingress
  namespace: monitoring
  annotations:
    kubernetes.io/ingress.class: alb
    alb.ingress.kubernetes.io/scheme: "internet-facing"
    alb.ingress.kubernetes.io/listen-ports: '[{"HTTPS": 443}]'
    alb.ingress.kubernetes.io/certificate-arn: "	arn:aws:acm:eu-west-1:428450571168:certificate/ec969c04-60f9-4654-a842-2f0c22e989d4"
    alb.ingress.kubernetes.io/inbound-cidrs: "<IP_ADDRESS>/0"
spec:
  rules:
    - host: grafana.qa.genieai.co
      http:
        paths:
          - path: /*
            backend:
              serviceName: qa-prometheus-operator-grafana
              servicePort: 80

But this is now working. any pointers? how to make it work for ALB?

Unfortunately I have no experience with the ALB ingress controller. That said, this shouldn't be different from exposing any other service.


deprecate old csi tables

Description
This PR resolves issue
How can this be tested?

Deploy operator to your cluster
Copy DB to you local machine: kubectl cp dynatrace-oneagent-csi-driver-<hash>:/data/csi.db csi.sqlite
Run .tables:

sqlite> .tables
app_mounts      migrations      tenant_configs
code_modules    os_mounts       volume_meta

and not presented tables: "dynakubes", "volumes", "osagent_volumes"

Codecov Report
Attention: Patch coverage is 50.00000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 58.78%.

Files
Patch %
Lines

pkg/controllers/csi/metadata/sqlite_gorm_client.go
40.00%
2 Missing and 1 partial :warning:

pkg/controllers/csi/metadata/migrations.go
60.00%
1 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3240      +/-   ##
==========================================
- Coverage   58.79%   58.78%   -0.01%
==========================================
  Files         325      325
  Lines       18139    18147       +8
==========================================
+ Hits        10664    10667       +3
- Misses       6279     6283       +4
- Partials     1196     1197       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[VIP-Split] [$500] Task - Email of Hidden user is revealed in the task preview in search list

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

Version Number: v1.4.38-1
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
. Go to staging.new.expensify.com
2. Go to 1:1 DM with a Hidden user.
3. Create a task without assignee in the DM.
4. Open task thread.
5. Click Assignee and select the Hidden user as assignee.
6. Return to main chat and go to task thread again.
7. Open search list.
Expected Result:
The task preview in the search list will show "assigned to Hidden".
Actual Result:
The task preview in the search list shows "assigned to <full_email>" instead of "assigned to Hidden".
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/17c434c3-4cfa-47f0-ac49-bf9a350a5a85
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c4e0aec786beaff8
Upwork Job ID:<PHONE_NUMBER>395364864
Last Price Increase: 2024-02-14

The root cause of this issue is the same as https://github.com/Expensify/App/issues/35960.
The backend and fronten fixes should include this issue as well.

The root cause is the same as https://github.com/Expensify/App/issues/35960.
The backend and frontend fixes should solve this issue as well.

@s77rt Can you confirm if the linked issue fixes this one? If yes, then we can close this one out and update the issue description to add this one to the scope.

@mananjadhav Yes looks same to me

Should we close this one or leave it open on hold to check the issue is fixed from #36196?

Since editing the last message is intended the linked issue and PR won't fix this. I'm not sure about the expected behaviour here but I guess this need to be fixed in the BE.
@thienlnam If you assign a "Hidden" user should that user still stay hidden?

Yeah it sounds like it - if you didn't have their personal details before assigning them then they shouldn't be revealed after getting assigned

Aah so maybe this needs to be Internal then?

Switched to Internal for a BE fix.

Waiting for someone to pick this up.

Waiting for internal ownership for the fix.

All task issues are future, we can just bump the priority of this to match that

Changing this to weekly as tasks are slated for future work. Thanks for the heads up @thienlnam!

Still held for future work.

Future work.

Labeled for future work, okay for now.

Waiting for prioritization.

Should we be putting this on Hold if not going to be prioritized?

@mananjadhav - Sorry for the hold up, I've been OoO and this hasn't been a priority. Seeing as VSB is pretty much on hold now, I think we can put it on hold for now.

I'm setting monthly, this is on long term hold.

On hold at this time.

Still held on prioritization.

@strepanier03 Any updates on this one?

This is for Tasks and we're not actively working or developing them unless they affect the onboarding flow, this does not. I'm closing and making not a priority.


Backport of 2638 in stable-3.1

Backport of #2638 in stable-3.1

jenkins test luminous-ansible2.4-docker_cluster