add ability to instantiate an eloquent cast when null value using null database values

Addresses situation when you may want the cast object to instantiated even when the database value is null.
The current DataEloquentCast get() method returns null if the database value is null.
This PR introduces a "nullable" Cast Parameter that will ensure the Caster gets instantiated rather than return null. The nullable parameter was inspired by the Spatie Laravel Enum package that uses the same parameter, albeit for slightly different purposes.
The nullable parameter name can obviously be changed if preferred, but it was also applied to good effect in the jessarcher/laravel-castable-data-transfer-object package which extends the spatie/data-transfer-object package.
I am currently migrating from spatie/data-transfer-object to spatie/laravel-data on a number of projects. I have a few use cases where this would be incredibly useful, particularly when working with Livewire, and running into issues using @entangle not having any (default) values.

Wow, this is a nice PR. This is what I was just looking for myself. I was trying to use Data objects to initialize JSON fields with default values typed in Data object if there is nothing set in the database.

Not entirely sure if nullable is the correct name for the parameter here. Adding such a parameter gives me the idea that the value can be a data object or null. Now the cast is nullable by default, maybe that wasn't the best design idea in hindsight, maybe renaming the thing to emptyable?
Not a big fan of that name so if anyone has a better proposal, please react 😄

By the way, I was just wondering why for NULL we also return NULL instead of an empty Data object with default values. I do very often use Data casts for nullable() json fields to cast them with default values but in order for this to work I need to override DataEloquentCast.
class DataEloquentCast extends BaseDataEloquentCast
{
    /**
     * @throws
     */
    public function get($model, string $key, $value, array $attributes): ?BaseData
    {
        if ($value === null) {
            $value = '{}';
        }

        return parent::get($model, $key, $value, $attributes);
    }
}

Maybe if we could have this behavior as a default and provide an easy flag for the user to determine if Data cast was created from a nullable field (on the other hand there is getOriginal() where user can check if it's null if needed), we would not even need nullable keyword.
Finally, this PR brings something that I would use very often, just would prefer this behavior to be a default one. :-)

Is it possible with your PR to load default values defined in the Data object? I have a nested data and it did not work for me. I have ended doing this.
    public function get($model, string $key, $value, array $attributes): ?BaseData
    {
        if ($value === null) {
            $payload = ($this->dataClass)::empty();

            return ($this->dataClass)::from($payload);
        }

        return parent::get($model, $key, $value, $attributes);
    }

I had not looked at nested data in tests. I'll try find time this week, but my guess is that it should work. Not sure why it didn't for you.

After 3.5.0 changes for default values it actually did start to work.
I just override DataEloquentCast with this and it works perfectly now:
    public function get($model, string $key, $value, array $attributes): ?BaseData
    {
        if ($value === null) {
            $value = '{}';
        }

        return parent::get($model, $key, $value, $attributes);
    }

After 3.5.0 changes for default values it actually did start to work.
I just override DataEloquentCast with this and it works perfectly now:
    public function get($model, string $key, $value, array $attributes): ?BaseData
    {
        if ($value === null) {
            $value = '{}';
        }

        return parent::get($model, $key, $value, $attributes);
    }

Yes, that's how this PR handles it also.
    public function get($model, string $key, $value, array $attributes): ?BaseData
    {
        if (is_null($value) && in_array('nullable', $this->arguments)) {
            $value = '{}';
        }

        if ($value === null) {
            return null;
        }

        $payload = json_decode($value, true, flags: JSON_THROW_ON_ERROR);

        return ($this->dataClass)::from($payload);
    }

Yep, I took inspiration from your PR, just it was not working before 3.5.0 as there were no way to define default values for a non-generic properties, so needed to do some ugly things to make it work before. :-)

By the way, I was just wondering why for NULL we also return NULL instead of an empty Data object with default values. I do very often use Data casts for nullable() json fields to cast them with default values but in order for this to work I need to override DataEloquentCast.

When something is nullable, I expect the object or null out of it when empty/non existing. Returning the object or a default object in that way just feels wrong.
We'll add the :default keyword looks correct to me 😄
Thanks!


Update request timeout to 60 min max, response timeout to 120 seconds

From the docs:
Uploads will timeout after 1 hour so please make sure your files are small enough to upload in a timely manner. After the data transfer finishes, it may take up to 120 seconds to generate the HTTP reply.

Please, fix that.
I'm creating database entries with speechmatics job ids after getting response when using client.user.jobs.create. But I can't do that when timeout error occurs. Then it's impossible for me to update some things when speechmatics sends a callback with job id as entry with given job id it does not exist in my database.

I'll get to this when I can, but to clarify, this ticket is just to update the default timeout.
You can set your own timeout value on the client in your own code (options get passed through to the underlying faraday/excon objects):
sm = Speechmatics::Client.new({ :request => { :timeout => 120 } })

As for the missing job id issue, it depends on how you do it.
For example, you can encode information in the callback url so you know how to react to an unexpected callback.  For example, I know a callback to https://example.com/audio_file/123/speechmatics_callback is for the transcribe of that audio file, whether I have an entry for the job or not.

This is updated to default to 120 seconds in 0.1.2 version I just released.

actually, just kidding, try 0.1.3, that last change had a problem.

Thank you for a quick response.
It would be nice to mention possibility of passing options to Faraday in here: https://github.com/PRX/speechmatics and here: https://www.speechmatics.com/api-details - there is nothing about that in the docs. If there was, I would have done that.

Glad that worked for you.
I added a mention to the README, feel free to PR more extensive docs if you want.
As for the speechmatics.com page, I don't work there, so you'll need to take that up with them.


Cloudflare pages

https://pages.cloudflare.com/
Another mirror to test.

Vercel
https://vercel.com

A table of free website hosting
https://jace.pro/post/2020-12-17-cloudflare-pages-netlify-zeit-github-pages-and-gitlab-pages-where-to-host/


Define Classes

Think and collect ideas about the classes we need for the project.
We could also try and collect ideas for some functions already.

We shoud add main_classes file which contains all of our classes. Possible options could be:
-Agent
-Food source
-The area which gets populated by the agents and food
-GUI
-Live GUI with sliders for different settings of the mold or food

Issues #8 , #9  and #15 have been describing each of those classes.
I closed this Issue since those classes have been defined


XSS Vulnerability in Table Component due to HTML tags

Hello,
I have been using the Skeleton component library at work and have come across a potential security vulnerability in the Table component. It parses raw HTML strings through HTML tags, which exposes it to cross-site scripting (XSS) attacks. I believe this needs to be addressed urgently.
Details:
When strings are passed to the table component, it gets rendered through HTML tags, therefore circumventing Svelte's string sanitation, making it prone to XSS.
https://github.com/skeletonlabs/skeleton/blob/72bd820dc65fefa61523a7d71eef6feed354dfed/packages/skeleton/src/lib/components/Table/Table.svelte#L102
Why this is a serious security risk:
Here the main danger is the fact that developers using the Table component might not be aware that cells are rendered through HTML tags, which is very dangerous in the event that user-input strings are passed to the component without sanitation. Which is likely, as Svelte already sanitizes strings (outside of HTML tags).
XSS vulnerabilities open us up to a whole world of hurt:

An attacker can manipulate the table's data or any other DOM elements.
If an attacker can run arbitrary JavaScript code, they can access and send the user's cookies to a remote server, compromising their session.
The attacker can create fake overlays or pop-ups asking the user for sensitive information.
The attacker could use XSS to redirect the user to malicious websites that host malware.

Recommendations:
Best to worst in my opinion:

Opt-in HTML tags: Instead of default HTML parsing behavior, make it opt-in. This way, developers are actively making a choice to accept HTML content, hopefully with the understanding of its implications.
Opt-out HTML tags: If not opting in, allow developers to opt-out of the HTML parsing functionality. This would allow the patch to be non-breaking.
Warning in the documentation: At the very least, there should be a clear warning in the documentation stating that the table component parses raw HTML and that users should sanitize the input they pass to it.

Thank you for taking the time to consider this issue. I believe skeleton is bound to be a leader in the svelte space, and as such security should be paramount. As a user of the library, I am looking forward to seeing how it will evolve.
Best regards,
Nassim

I agree options 1 or 2 should be added to the library - same goes for the Autocomplete component.
I think adding a prop to toggle this behavior should be a very easy-to-add solution.
While I agree sanitation should be handled by the library users - I don't think that's the case here.
As in when I render a button I(and I assume most people) usually do this:
<button>{label}</button>

not this
<button>{@html sanitize(label)}</button>

I just want to chime in to say that I'm intentionally using HTML inside of table contents to accomplish functionality that isn't available in a Skeleton table otherwise. Whatever is done, if anything, I do hope that there is a way to opt-in to continuing to render raw html.

In an effort to prepare for Skeleton v3, we're consolidate some related issues down to a single ticket. This will ensure that we can see the full context of requests when the time comes to refactor and update this feature going forward. If you wish to add additional feedback or suggestions, please so here:

https://github.com/skeletonlabs/skeleton/issues/2361


[FX-5323] Merge feature/tailwind-w20

FX-5323
Description
How to test

Temploy
Visual tests are green in Client Portal https://github.com/toptal/client-portal/pull/9082
We have one broken test in Cypress in Client portal, that is not related to any migrated component

Screenshots

Before.
After.

Insert screenshots or screen recordings
Insert screenshots or screen recordings

Development checks

[x] Add changeset according to guidelines (if needed)
Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
Annotate all props in component with documentation
Create examples for component
Ensure that deployed demo has expected results and good examples
Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
Covered with tests (visual tests included)

Breaking change

codemod is created and showcased in the changeset
[ ] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-bot run package:alpha-release

@toptal-bot run package:alpha-release

@toptal-bot run package:alpha-release


More testnet support for segwit

I thought you'd be interested in those small adds and fix... have a look and give me news!

Thanks.. would need test cases as well !

Thanks. Would be associated test cases as well !

@Kexkey Sorry, I mean could you add associated test cases as well

Hehe yes I understood, sorry I didn't have time and finally forgot about it.  :)
I think I saw segwit support in Richard's repo, is it the case?  If so, is it still pertinent to do it?
Thanks!

AFAIK Richard's repo supports segwit address, but no segwit xpub's yet. My pull request there is still pending.
Let me know if I am wrong

I think you're right.  With Richard's current master version:
/ # ku -n BTC -s 0/1 -a E:vpub5SLqN2bLY4WeZF3kL4VqiWF1itbf3A6oRrq9aPf16AZMVWYCuN9TxpAZwCzVgW94TNzZPNc9XAHD4As6pdnExBtCDGYRmNJrcJ4eV9hNqcv
can't parse E:vpub5SLqN2bLY4WeZF3kL4VqiWF1itbf3A6oRrq9aPf16AZMVWYCuN9TxpAZwCzVgW94TNzZPNc9XAHD4As6pdnExBtCDGYRmNJrcJ4eV9hNqcv

With this PR on your fork:
/ # ku -n BTC -s 0/1 -a E:vpub5SLqN2bLY4WeZF3kL4VqiWF1itbf3A6oRrq9aPf16AZMVWYCuN9TxpAZwCzVgW94TNzZPNc9XAHD4As6pdnExBtCDGYRmNJrcJ4eV9hNqcv
tb1qumju035epdqmta8u5yg9s53zcwyjztehm0sagd

I'll have a look at the test suites.  👍


Fix oba msids

@taldcroft
@jeanconn
This update includes a fix for the 4OAVOBAT telescope msid which continues to show the narrow range version of this msid where it should be reporting the wide range version after the associated patch was uplinked. This update also includes a minor fix for the DP_OBA_AVE derived parameter.

Try to do this without repeating code.  Hint, with good surrounding comments you can hijack the msid_nums dict a bit:
'b': '30 31 33 34 35 36 37 38 39 40 41 44 45 46 49 50 51 52 53 54 4OAVOBAT'.split()

Then you just need to handle the oddball in the loop.

Ok that works too. I wanted to avoid messing with the logic in the loop, but the result does look cleaner.

True, that eliminates another line, new commit to follow momentarily.

@matthewdahmer - I rebased this on the state-bins branch prior to regenerating data.

@taldcroft  - Thank You!
I take it I will not need to manually update this data then correct?

@matthewdahmer - there is a slight problem with the 4OAVOBAT patch, which is that it occurs in the TEL2ENG data content, not OBC4ENG like the others.  So I would suggest that you back out the changes in def obc4eng(..) and make a new function that is fairly similar but is named def tel2eng() and fixes 4OAVOBAT.  Does 4OAVHRMT need to have the same patch, or was that not changed on board?  In the def tel2eng() function you probably don't need all the a/b logic since there was just one patch time.  Sorry for not noticing this earlier.

BTW, because I have rebased this onto the other branch, you need to get into your git repo and do something like:
git branch -m fix_oba_msids fix_oba_msids_bak
git fetch origin
git checkout fix_oba_msids
<edit as normal>
git push origin fix_oba_msids

Without moving the original branch out of the way there will be a merge conflict later when you try to push back up.

That should do it. I should have noticed that I was using the wrong data content directory. 4OAVHRMT is not affected as the constituent HRMA zones were not switched to wide range. Please let me know if you have any comments or suggestions.

OK thanks, looks good.

Relevant commits included (in squashed form) in #127.  Closing.


Gem is not yet published on rubygems - bundle won't work without github param

Why not publish it or changing to:
gem 'qualtrics_api', github: 'pallymore/qualtrics_api'

in readme in the meantime.

published 0.0.1


Hello!

Hi,
Of course! The project is under MIT license but you should be aware of some
resources are not.
As for the translation, l am planning to add Japanese and Korean version in
recent days but feel free to fork the project or even create a PR, you are
totally welcome to do so.
Best.
10duckman @.***> 於 2021年3月30日 週二 下午1:32 寫道：

Hello, i'm korean gamer.
I'm interested your project. Can i fork your project for translate korean
and some work?
regards.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/purindaisuki/tkfmtools/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ3S6AEUTVTFVDLHGKVEDGDTGFO5HANCNFSM42BINR5Q
.


Cash name not changing to tokens

Cash remains called "cash" regardless of settings in the sh_schema.
Clockwork.option:SetKey("name_cash", "Tokens"); appears to have no effect.

Fixed in 58c29c584ac49ae687a8643e43622d8f38d39d46, for English at least.


Libload Importing

I was trying to install ti craft but i came up with this error after i installed libload.

I went to the link and sent it to my calculator but it didn't work

nvm i figured it out :)


Remove obsolete documentation

LGTM, might be worth poking client folks, like @Kegsay, in-case it isn't unused.


Button group interaction styles

Issue Description
Comment from UX during validation

Add detection for mouse click vs. keyboard (tab) interaction so the default :focus outline glow can be maintained for selection/onclick/onchange when interacting via keyboard, but not applied after a user touch or click with mouse. Issue to solve will be to remove the default :focus outline by blur() or similar method but still maintaining tab position on the page.

Issue Type

[ ] New Feature
[x] Enhancement
[ ] Bug
[ ] Other

Expected Behavior
Add detection for mouse click vs. keyboard (tab) interaction so the default :focus outline glow can be maintained for selection/onclick/onchange when interacting via keyboard, but not applied after a user touch or click with mouse.
Current Behavior
Focus styles do not differ between mouse click vs. keyboard (tab) interaction

The behavior determined here would likely carry forward for this issue regarding Selectable Lists https://github.com/cerner/terra-core/issues/529.

This should be covered by https://github.com/cerner/terra-core/issues/1038


Fixing accessToken expiration and updating Moment.add() deprecation

I added two tests to verify accessToken expiration is working and updated the moment.add() calls to eliminate the deprecation warnings.

Awesome!  Thanks for this!


Use Redis operator to test odo service create with parameters

What type of PR is this?
/kind tests
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #4961

/test psi-unit-test-windows
timeout error

/approve


Client Allocation Request for: 杭州讯酷科技有限公司

Client Allocation Questions
Core Information

Name: 杭州讯酷科技有限公司
Website / Social Media: www.xunkukeji.com
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f1wninfaoogdihtbqntlhjanb3z5bswqvjcestuuq
Notary Requested: ozhtdong

@ozhtdong Hi. I have answered your questions before and my account is suspended since I used it to participate in Huobi projects. I don't know if you have my information backup or not.

@TzmDong I checked my Mail-Box. Are these your replies?

If yes, I still need to know more details about your allocation strategy.
1.How many miners do you want to store your data to? How much data for each miner?
2. Will you record your allocation decision in public? Which platform will you use?
3. What method/tool will you use to insperct miners? Which aspect of miners will you inspect?
4. Have you ever allocated before? If yes, how did you do when you were questioned?

Yes,these are indeed my replies.
1.Maybe I will store my data to 2 miners. I have tentatively decided to store 15 TB of data for each miner.
2.Yes,I will record my allocation decision on this page.
3.I don't have a definite idea yet，but the credibility of a miner within the industry is an important indicator.
4.Nope.

Good. The last step, please show me your company searching consequence in the local business license searching website.

Thank you very much for your patient reply.Looking forward to your good news.

Request Approved
Your Datacap Allocation Request has been approved by the Notary
Message sent to Filecoin Network

bafy2bzaceccsujzyt3ld6z5pgwviuz4jjpjf2zlds32vlz3cppvogwhxm7qiw

Address

f1wninfaoogdihtbqntlhjanb3z5bswqvjcestuuq

Datacap Allocated

5TiB

You can check the status of the message here: https://filfox.info/en/message/bafy2bzaceccsujzyt3ld6z5pgwviuz4jjpjf2zlds32vlz3cppvogwhxm7qiw


"apt-get: not found" during Building bootstrap

Hello, really interested in this project - was trying to get the following to work:
$ docker-compose up -d
but I get this:

Creating netstats
Building bootstrap
Step 1 : FROM ethereum/client-go
---> 4b290a5d5747
Step 2 : RUN apt-get update &&    apt-get install -y curl git-core &&    curl -sL https://deb.nodesource.com/setup_4.x | bash - &&    apt-get update &&    apt-get install -y nodejs
---> Running in 839ec68e0b63
←[91m/bin/sh: apt-get: not found
←[0m←[31mERROR←[0m: Service 'bootstrap' failed to build: The command '/bin/sh -c apt-get update &&    apt-get install -y curl git-core &&    curl -sL https://deb.nodesource.com/setup_4.x | bash - &&
apt-get update &&    apt-get install -y nodejs' returned a non-zero code: 127

please help!

you can try to split the last command into separate lines instead of combining them with &&, then you will see, which command will fail.

The Docker image ethereum/client-go is under the alpine image (go-ethereum Dockerfile)
Instead of apt-get install, you have to use apk add

@2075 @chrc thanks a bunch, that got me further, but now I'm getting the following
Step 4 : RUN curl -sL https://deb.nodesource.com/setup_4.x | sh -
 ---> Running in 020b0b1bb707

## Installing the NodeSource Node.js v4.x LTS Argon repo...

## Populating apt-get cache...

+ apt-get update
Error executing command, exiting

I suspect its due to the 'apt-get' in this: https://deb.nodesource.com/setup_4.x
if that's so, is there an alpine / apk compliant node url? (sorry, *nix n00b)

Hi,
My monitored-geth-client/Dockerfile
FROM ethereum/client-go:latest

RUN apk update

RUN apk add curl git bash nodejs perl

RUN cd /root &&\
    git clone https://github.com/cubedro/eth-net-intelligence-api &&\
    cd eth-net-intelligence-api &&\
    npm install &&\
    npm install -g pm2

ADD start.sh /root/start.sh
ADD app.json /root/eth-net-intelligence-api/app.json
RUN chmod +x /root/start.sh

ENTRYPOINT /root/start.sh

This should be fixed in https://github.com/Capgemini-AIE/ethereum-docker/commit/cf23879c00f8bcc59f8ef7195e767adf6fc3979c , closing. Please re-open if there is an issue


remove default enable ACPI kconfig macro such as DSDT and ACPI_SHELL

remove default enable kconfig macro such as DSDT and ACPI_SHELL if PCIE or SHELL enabled. All Intel platform may not require to use ACPI DSDT related APIs and hence it should not enable default if PCIE enabled.

This PR makes UART work on up_squared, but there are two concerns I have:

It does this in an indirect way, by disabling CONFIG_PCIE_PRT, a Kconfig option that's not mentioned anywhere in the PR or the commit messages. If I understood correctly, it's that option which makes PCIe code depend on ACPI.
It's not really addressing the main concern, which is that we should work toward getting PCIe and ACPI initialised much earlier in the boot (most likely PRE_KERNEL_1 is the right level)

uart driver is started after uart console and after the logger, this will fail.

actually. one uart is starting in the right level, do we have something not configured with pcie?

PR heading and content updated

@najumon1980 is this PR still relevant? If not, please close it.


Scam Account Submission

Duplicate Submissions

[X] Yes, I checked that the account name is not currently in the list
[X] I read the wiki explaining each different categories and examples before submitting

(Required) Spam Comment Direct Link
https://www.youtube.com/watch?v=zPlsnWq5X50&lc=UgyZIRomU0H2UtcrX_l4AaABAg.9mdewa-4lch9mhEcW-5X3L
https://www.youtube.com/watch?v=zPlsnWq5X50&lc=Ugwd-qgcky6MkpjktMp4AaABAg.9mlonbbj71f9mlpSgSGcrf
https://www.youtube.com/watch?v=zPlsnWq5X50&lc=Ugyr23RBiyR73Uxnc214AaABAg.9moDO07yH6d9moJwy6Jnyk
https://www.youtube.com/watch?v=zPlsnWq5X50&lc=UgxryBkQwZJgYRCMqFR4AaABAg.9mZKcIv3Ahu9mh5_HXzRH8
https://www.youtube.com/watch?v=zPlsnWq5X50&lc=Ugw1iGDKrlL2kKe6EJp4AaABAg.9meZ7qbQrzw9mfZY-Hi8xr
(Optional) Screenshot of comments referencing the scam account
No response
(Required) List of the scam account names
jlaservideo00
mrjerrytweek
brentriveralive
emilysears01
familiadelfin

Added


SPO IP range used by EXO during Hybrid MRS moves

We are seeing traffic on <IP_ADDRESS> and <IP_ADDRESS> when doing Hybrid MRS move from on-prem EXCH to EXO.  When we block those endpoints, some of the MRS jobs fail.  These IPs are not listed under EXO, but are listed under SPO/ODfB (<IP_ADDRESS>/17).  Are you saying that SPO ranges must be allowed in order to do mailbox migrations, or are these IPs being reassigned to EXO?

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

ID: 077406d8-bff1-496e-3ea2-b38558fd9355
Version Independent ID: 57db3dec-2250-2dee-fd1a-ec6656911798
Content: Office 365 URLs and IP address ranges
Content Source: Enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@Davidm17011 - Thank you for submitting feedback.
I will get this issue over to the Enterprise writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@JoeDavies-MSFT - Can you please share your insights on this issue?
Thank you.
cc: @kenwith

@Davidm17011 I have forwarded this issue to the product teams who manage these endpoints for answers to your question. Thanks

@JoeDavies-MSFT - Do you have any updates from the product team?
Thank you.

@JoeDavies-MSFT @LauraWi - We would like to follow up. Do you have any updates from the Product team?
Thank you.

Hi, @Davidm17011  Guidance from engineering is that, yes, you need to allow connectivity to the SharePoint IP addresses.


Add dynamic GitHub stars and forks

Closes: #10
I did add a new folder /components/ just to keep things more organized and used a sperate css file for the component
If your not okay with that change. I can change it back as well for anything else 🤗

@lejara Great work! It looks cool; I only have one suggestion: could you make the buttons' icons and text a bit smaller?

sure thing!


Cleanup error formatting

@Shopify/kafka
As pointed out by Willem (https://github.com/Shopify/sarama/pull/485#discussion_r35786611) we have a bunch of not-quite-idiomatic error formats.


v2 - desktop version

Describe what problem your feature request solves
We should have a desktop version in place before version 2.0 release
Describe the solution you'd like
It would be good to use an electron wrapper for the web application, and this will provide the desktop version of the threat dragon application
Additional context
Note that the existing version 1.x threat dragon desktop uses common code with the web application, but does not wrap the web application itself.  This has caused an increase in development and test effort, so it would be good to change the way this is done. Refer to this example but it can be done in various ways

Hey,  I'd like to take this up! Would primarily follow the article mentioned as I'm new to electron but would be open to incorporating any changes suggested

Yes, sure, thanks for your help @ShubhamPalriwala - I can assign it to both of us as I am sure we can help each other

Hey @jgadsden, so should we remove the entire existing desktop codebase and just have a simple wrapper around our web-app in our main.js inside our td.desktop?

Hello @ShubhamPalriwala , yes, that makes sense to me. What might be good is to move the td.desktop to another directory so that we can easily refer to it, and when all is working remove it then. Alls is good though, so I will go with what  you decide

@jgadsden, I'm loading the following URL for our desktop application: http://threatdragon.org/
This is fine right or will we be using another URL?

We do not have direct control of http://threatdragon.org/ , so threatdragon.com is probably the better domain to use ... this is used to showcase the latest Threat Dragon web app and documentation
I will have a look at your code on the branch this evening, good to see progress so soon @ShubhamPalriwala !

From a discussion in Slack (just for visibility for others who may see this): The goal is to wrap td.vue in electron as opposed to loading a specific URL.  The vue site should be able to operate independently of any server.  Threatdragon.com is running on a very small Heroku instance, and might not handle the load well.  Also, that is used as a staging area and is published each time there is a push to the main branch, so it might not be considered stable.
Thanks again, @ShubhamPalriwala

Hello @ShubhamPalriwala , the next thing to do is to make an electron wrapper for the td.vue web application. The td.desktop directory can then be retired, along with td.desktop/main.js
I will probably look at these next, for guidance:
https://stricker.digital/posts/electron-with-vue-and-webpack/
and
https://blog.logrocket.com/building-app-electron-vue/
and Vue seems to work well with electron

Agreed @lreading , I should have made it clear that there should be no need to use either threatdragon.org or threatdragon.com
The intention is to create a new file (probably in the td.vue directory) that is the electron window builder. My guess is that we will need other source  files to provide the functionality for the desktop application, similar to that provided by td.server for the web application
I need to look into this more ... concentrating on adding threat engine at the moment ... hope all this makes sense for you @ShubhamPalriwala

So I researched for a while andI found a great vue package that solves the exact problem that we are tackling.
Will configure it acc to us and open a PR for the same soon!

@ShubhamPalriwala pull erquest #362 satisfies this issue, so do you want to have the honour of closing it?

Haha! Thank you Jon.


fix pkgs provider output for new package index format

Fixes https://github.com/rebar/rebar3/issues/749 and adds a test to just verify that rebar3 pkgs doesn't crash.

Fixed.


Multi-database search

Formula search logic change. Formulas from all databases searched for together, FDR calculated and results stored afterwards by selecting subset of the results for each database separately
2-3x formula search and image scoring speedup
Search job logic refactored and simplified. As the result computations for all moldbs are now done inside MSMSearch.search and the jobs data is stored only after they all successfully finished. If any of the moldb searches fails, nothing will be saved to the job table
Centroids generation: set number of centroids per formula to be exactly 4 to get all formula intensities faster by doing a simple array reshape instead of Pandas dataframe grouping
Formula parser module for parsing and simplifying formula + adduct as ion formula
Massive renaming: sf -> formula, ion -> formula (or ion_formula)
Update tests

* Centroids generation: set number of centroids per formula to be exactly 4 to get all formula intensities faster by doing a simple array reshape instead of Pandas dataframe grouping

The added centroids should be filtered out before the annotation is saved to the DB. Otherwise they get saved as a peak at m/z 0:
Old code:

This PR:

Old code:

This PR:

Contrary to what we discussed, the decoy adduct random seed is actually a constant 42.
Regardless, I compared the same dataset with the same ds_id on the old and new code, and these are the annotation counts:

HMDB-v4ChEBI-2018-01
FDRBeforeAfterBeforeAfter
<= 5%423484406418
<= 10%954963838834
<= 20%1842183516871685
<= 50%4083407640724064

Between the 2 DBs:

95 annotations' FDRs (95 unique mols) went down by 1 step. None had MSM changes
1 annotation's FDR went up by 1 step. None had MSM changes
21 annotations (16 unique mols) were only emitted by the old code and not new code. Only 2 of these annotations (/1 unique mol) are explained by the below 3-centroid issue. I'm not sure what has caused the other 19 differences - some of them have quite high MSM scores.
No annotations moved more than 1 FDR step, except for the annotations that disappeared entirely
No new annotations were emitted by the new code
7 annotations (4 unique mols) had different MSM scores - they all had only 3 centroids, and the msm changed by increasing their spectral score

I have no objection to the FDRs changing. The 3-centroid spectral score issue is probably a bug that needs fixing.
It would be nice to know why some annotations aren't detected in the new code, but I'm out of ideas on how to diagnose it. It doesn't seem to be an issue with segment boundaries - I ran this dataset twice each with the old & new code, and the results were the same, even though all of the runs chose different configurations of segments.

The zero mz patterns issue should be resolved in the next commit.
Regarding the decoy adduct random seed. It seems that we discussed the idea of using a fixed one per dataset id but never implemented it because it potentially leads to changes (minor but still) in lots of results when reprocessed.
I'm checking the differences in the search results. The sci test passed successfully.

@LachlanStuart I finished cleaning the code. It's still not the final version but I'd like to get your input at this point


PYI-332: Add initial oauth routing

Proposed changes
What changed

Add an initial page rendered in GDS style
Add a link to core-back's authorization endpoint, and return to the callback url passed in on a request

Why did it change

Adding an initial flow unblocks the team

Issue tracking

PYI-332

Checklists
Environment variables or secrets

[x] Documented in the README
[ ] Added to deployment repository
[ ] Added to local startup repository

Other considerations

[ ] Update README with any new instructions or tasks

Note: There are no tests, and the params from the browser should be validated.
These can be fixed once we've got all the various parts linked up into the "steel thread" through the system.


Dollar won't build targets

When running carthage update --platform iOS  I get the following error:
=== BUILD TARGET Dollar OF PROJECT Dollar WITH CONFIGURATION Release ===
Check dependencies
OS X targets using Swift cannot be built against an SDK older than 10.10, but the effective SDK is 8.2.
...is there some setting I forget to add?

Can you share carthage version and what happens if you install with carthage update without --platform
This project in the repo was build using cathage update only and is an iOS project.

I updated to xcode 6.3, getting a different error now, when running carthage update without --platform:
[MT] iPhoneSimulator: SimVerifier returned: Error Domain=NSPOSIXErrorDomain Code=53 "Simulator verification failed." UserInfo=0x7f9a4164a8d0 {NSLocalizedFailureReason=A connection to the simulator verification service could not be established., NSLocalizedRecoverySuggestion=Ensure that Xcode.app is installed on a volume with ownership enabled., NSLocalizedDescription=Simulator verification failed.}
** BUILD FAILED **
The following build commands failed:
Check dependencies
(1 failure)

Let me take a look

@avdwerff I believe this could fix your issue - https://github.com/Carthage/Carthage/issues/424#issuecomment-96160474


feat: update button states

Description
Supports https://github.com/appsmithorg/appsmith/issues/18052
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Checklist:

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

Issues with Tertiary button:
Default state:

the width & height of the text are not as per the design. (constant throughout all states)
Line height is 15px whereas it should be 16px. (constant throughout all states)

Hover:

Height & width are not as per design
padding not as per design

Focus:

Background colour is transparent and not white
Box Shadow is missing

Issues with Secondary button:
Default state:

the width & height of the text are not as per the design. (constant throughout all states)
Line height is 15px whereas it should be 16px. (constant throughout all states)
Height & width are not as per design
padding not as per design
Border width is not as per design (constant throughout all states)

Hover:

Height & width are not as per design
padding not as per design

Focus:

Background colour is transparent and not white
Box Shadow is missing

For both states, what has been defined as box shadow has been implemented as outline, and can be triggered by navigating to the element via keyboard.
For all states, the width, height, padding, and line height of the button is set based on what the size prop is. We are not changing those dimensions, so I haven't touched them.
Have changed all transparent colors to white where relevant.
Border width on design is 1.2px and on storybook is also 1.2px? I'm not sure what you mean here.


E2E: Use of domain in identityChangeModal doesn't work

Linked to: https://github.com/starknet-id/starknetid.js/issues/20

Fixed


follow the NVVM IR form: not use . in global identifier

a preprocessing step for #9084 .
NVVM IR identifier uses a stricter form than LLVM IR.
NVVM IR ID form: https://docs.nvidia.com/cuda/nvvm-ir-spec/index.html#identifiers
LLVM IR ID form: https://llvm.org/docs/LangRef.html#identifiers
This change should not affect numba core or CPU target. 🤞
The real feature implementation (or bug fix?) for #9084 will be created in numba-cuda repo separately.

But, NVVM can never use a dynamic global. Do you have a situation where Numba is generating that?

drop this for now, since the main PR in numba-cuda is stalled.

Thanks @dlee992. I hope we'll figure out a way to un-stall the main PR in numba-cuda (https://github.com/NVIDIA/numba-cuda/pull/15) but it turned out to be much trickier than I expected.


firebase_messaging control on notification

Hi,
In my app , I used FCM and have problem to control the notification (for example to remove/hide them programmatically), or use Vibration on recieve a notification or group messaging by tag of the message.
how can I do that ? I can't find any way!
thanks

Any updates here?

@1abzar
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/508. Any further collaboration will be done there.


tensorboard writer error

Issue Type
Bug
Source
source
MCT Version
1.11.0
OS Platform and Distribution
Ubuntu 18.04
Python version
3.10.14
Describe the issue
Hello.
I tried to convert yolov8n in MCT with tensorboard.
When I called mct.ptq.keras_post_training_quantization_experimental,
I got the following error,
AttributeError: 'NoneType' object has no attribute '__name__'. Did you mean: '__ne__'?

I wonder if it would be possible to tell me how to address this error ?

Expected behaviour
I expect conversion succeed.

Code to reproduce the issue
docker image: ultralytics/ultralytics:latest

git clone https://github.com/sony/model_optimization.git local_mct
cd local_mct
git checkout refs/tags/v1.11.0
pip install -r requirements.txt

[base code]
https://github.com/sony/model_optimization/blob/v1.11.0/tutorials/notebooks/example_keras_yolov8n.ipynb

[add]
mct.set_log_folder('./loggerv1_11')

[modify] I think tensorboard_writer.py don't support tf.image.combined_non_max_suppression
model = Model(model.input, outputs, name='yolov8n')
↓
model = Model(model.input, model.output, name='yolov8n')

Log output
Traceback (most recent call last):
  File "/usr/src/ultralytics/example_keras_yolov8n_v111_board.py", line 136, in <module>
    quant_model, _ = mct.ptq.keras_post_training_quantization_experimental(model,
  File "/usr/src/ultralytics/../local_mct_v1_11/model_compression_toolkit/ptq/keras/quantization_facade.py", line 139, in keras_post_training_quantization_experimental
    tg, bit_widths_config, _ = core_runner(in_model=in_model,
  File "/usr/src/ultralytics/../local_mct_v1_11/model_compression_toolkit/core/runner.py", line 99, in core_runner
    tg = quantization_preparation_runner(graph=graph,
  File "/usr/src/ultralytics/../local_mct_v1_11/model_compression_toolkit/core/quantization_prep_runner.py", line 119, in quantization_preparation_runner
    tb_w.add_graph(transformed_graph, 'after_shift_negative_correction')
  File "/usr/src/ultralytics/../local_mct_v1_11/model_compression_toolkit/core/common/visualization/tensorboard_writer.py", line 308, in add_graph
    main_node_def.device = n.type.__name__  # For coloring different ops differently
AttributeError: 'NoneType' object has no attribute '__name__'. Did you mean: '__ne__'?

in tesorboard_writer.py, add_graph
ipdb> node_sort[0]
InputLayer:input_1
ipdb> node_sort[1]
ZeroPadding2D:zero_padding2d
ipdb> node_sort[2]
Conv2D:model.0.conv_bn
ipdb> node_sort[3]
swish:tf.nn.silu
ipdb> node_sort[4]
*** AttributeError: 'NoneType' object has no attribute 'name'. Did you mean: 'ne'?
ipdb> node_sort[5]
*** AttributeError: 'NoneType' object has no attribute 'name'. Did you mean: 'ne'?
ipdb> node_sort[6]
Conv2D:model.1.conv_bn

Hi @YoshikiKato0220,
We reproduced an issue with logging this layer, but a different error.
A fix was merged, and the tutorial runs successfully on colab with logging.
Please retry it and see if you encounter any issues.
https://colab.research.google.com/github/sony/model_optimization/blob/main/tutorials/notebooks/mct_features_notebooks/keras/example_keras_yolov8n.ipynb
Thanks

Hi @reuvenperetz ,
In this colab (mct is latest version),
I confirmed it works.
Thanks


Add prettier user test and fix associated crash

While adding prettier to the user tests, I uncovered another assert in getJSDocHost when looking up type references in a @property tag inside a @typedef tag. This PR contains the fix as well as adding prettier to the user tests.
Crash

getJSDocHost didn't work correctly when called on a @property tag.

Incorrect error

Can't import package.json -- #22167
Can't import npm package that exports json instead of js -- DefinitelyTyped/DefinitelyTyped#25335
node-ignore has wrong types -- kaelzhang/node-ignore#36
chalk has wrong export structure -- chalk/chalk#256
Can't refer to typedef from another file -- #23692

Annoying

Can't pass null to optional parameter with strictNullChecks.
Can't add instance properties to Error instances
[] is still often never[] with strictNullChecks (eg as a property of an object literal, or as the initial value of a reduce)
When a mutable variable is initialised with a specific member of a discriminated union, we need a type annotation in order to specify that the variable's type should actually be the entire union.
Browser detection code needs tweaks to be well-typed, even though nobody cares.
Auto types don't scale well to 1,000-line functions, and type declaration is blocked by the typedef reference feature.
Have to cast partial objects that are lazily initialised.

Improved hygiene

strictNullChecks forces some checks for undefined that are not obviously impossible.
strictNullChecks forces a few casts for undefineds that are obviously impossible (eg from queue.pop just after checking queue.length).
@param {number?} n doesn't make an optional parameter. @param {number} [n] and @param {number=} n do.
Too many arguments passed to a few different functions.
[].concat(x,y,z) has to be changed to [...x, y, ...z]

Found possible bug

Forgot to check result of String.match for null.
glimmer requires ASTPlugins to have a name property, but parser-postcss forgot to provide one.

Discussion
As you can see, there were lots of problems adding checkJs to prettier. None of them were particularly difficult to resolve, but there were a lot of small tasks that added up. Without strict null checks, there was only one bug and two hygiene items. Compare that to the whole pile of annoyances and incorrect errors.

Auto types don't scale well to 1,000-line functions, and type declaration is blocked by the typedef reference feature.

what do you mean by not scaling?

The chance of doing something that confuses an auto type in a 1,000-line function approaches 1. In prettier, assignments plus each+lambda iteration confuse it.
If you had 100 10-line functions, each with an auto type, the number of constructs we don't handle would not be much higher, but you'd have many auto-typed variables with no problems.


use TypeScript 5.0

see https://devblogs.microsoft.com/typescript/announcing-typescript-5-0/
requires https://github.com/TypeStrong/typedoc/issues/2201

in case this casues breaking chages for downstream users, this will be part of a major release

closed by #644


⚠ GitLab Server has degraded performance

In a671888, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11144 ms

Resolved: GitLab Server performance has improved in f2c91b0.


extend TeX fonts coverage using cm-unicode

Potential source for improveming the TeX fonts

http://sourceforge.net/projects/cm-unicode/
"Currently the fonts  contain glyphs from Latin (Metafont ec, tc, vnr), Cyrillic (lh), Greek (cbgreek when available) code sets and IPA extensions (from tipa)."

More ideas from https://www.ctan.org/topic/font-cyrillic.
In particular, https://www.ctan.org/pkg/wncyr


Testing

PR Description 📜
I have built the Racing Game made using JavaScript. In this a user can play game using 6 keys of the keyboard. User has to just start the game and play according to the situation and see if he/she can finish the race before time or not!
Fixes #991

Mark the task you have completed ✅

[✅] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[✅] I have performed a self-review of my own code or work.
[✅] I have commented my code, particularly in hard-to-understand areas.
[✅] My changes generates no new warnings.
[✅] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[✅] I have added screenshot for website preview in assets/images
[✅] I have added entries for my game in main README.md
[✅] I have added README.md in my folder
[✅] I have added working video of the game in README.md (optional)
[✅] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Parul1606 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

we already have this game on GameZone 👀 !!

Thank you  @Parul1606 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Fix for Core2 setBrightness

No constraints of map function. Might want to add those if the AXP might generate out of spec voltages.

Thank you for your advice, I have optimized the code.


symbolic links not followed correctly when opening in VSCode from Unity

If you open a file in a folder that is symbolically linked it will open in the file's original location as opposed to the linked directory.

I did some investigation and this appears to be a visual studio code issue. I think the workaround may be to use hardlinks but then that messes with git so I may be stuck browsing in the solution for now.

https://code.visualstudio.com/issues/detail/19526

I just wanted to say, thank you for identifying what is a Visual Studio Code issue, versus the plugin.
Not many people do :)


Test cases where mkdir should fail

The added tests should pass for MoarVM/MoarVM#1507.  Edit: the final diff is confusing, please consider viewing each commit for the actual changes.

Windows imposes a limit of 260 bytes on the total path length.
The maximum length of a path component (i.e. file or directory name) is file
system dependent. Remember that the original FAT only allowed for 8+3
characters, regardless of operating system. 255 bytes is a common limit, but
by far not universal.
POSIX defines a PATH_MAX constant and on Linux it seems to be 4096 bytes:
linux/limits.h:13:#define PATH_MAX        4096  /* # chars in a path name
including nul */
However, it looks like file systems will nevertheless allow for creating paths
longer than that.

Windows imposes a limit of 260 bytes on the total path length.

There seems to be Long Paths on Windows, however I have yet to find how
it can affect components (which is also limited to 255 on NTFS by default):
https://docs.microsoft.com/en-us/windows/win32/fileio/maximum-file-path-limitation

I'm not sure if the comment was clear enough, but this test case try
to trigger the case where the directory name (not the entire path name)
is longer than what is permitted, so that we can ensure that after
a successful creation, errors won't be silenced.  It could be other
reason but the name length seems to be the most portable and common case.

+1 from me

@niner It looks like we have no legal way to find out what the value of PATH_MAX is and I'd say it's important enough to be available in a non-platform, non-backend dependable way.

@vrurg, I think you missed my comment of it is about the directory/filename length limit, not total path name.  Though, I want to bring up that mkdir is inherently platform dependent, e.g. the file permission concept on Windows is quite different from the three-byte POSIX one mkdir on Raku uses.  In other words, if Reiser (some of the few file systems known by Wikipedia to have filename length limit greater than 255) is supported we might want a special indicator in the test case.

the test is dependent on information not directly known to it. It is assumed [...] To make it really reliable it must look like: [...] $*VM.constants<PATH_MAX>

That's true.  However, having such constant correctly defined, i.e. specifically for each file system (instead of just hardcoding to a number with one or two known exceptions) is nontrivial.  In addition, directory creations used by this test suite is making other assumptions as well, e.g. make-temp-file relies on PATH_MAX to be at least some certain number due to the inclusion of some prefixes and timestamps.  Therefore, I wonder if it is OK to not let perfect be the enemy of good here and allow an assertion that probabilistically always do the right job and a patch to MoarVM that (IMHO) correct the semantic that has been broken for at least half a decade.  Of course the opposite could have been said, that if it stayed broken for so long then it's probably OK to leave it that way, but I'd much prefer the other outcome.

I did forget about different file systems on the same OS. I also forget about all the peculiarities around PATH_MAX, but this blog reminded me of them.
Yet, it changes little. The chances are non-zero that the test will fail. For this reason we can't have it in the roast.
But what we can do is to move it into Rakudo test suite because it covers an implementation bug. This would make it easier to fix the test when necessary.

I've moved the test to Rakudo as suggested.


Generate and Use Coverage at each run

There is already
  --coverage DIR     Load .lst files corresponding to source files from DIR

. Why doesn't dub dustmite generate new coverage information "at each call" to dub run and use that via loadCoverage when deciding which code to remove prior to subsequent call?!!! Wouldn't this enable amazingly faster reduction speed for D especially in the reduction of dependencies.
I guess most of this logic should be implemented in dub.
Ping, @CyberShadow.
See https://github.com/CyberShadow/DustMite/blob/master/dustmite.d#L2405

What are you trying to achieve with --coverage?
Can you describe your use case in more detail?

I thought loadCoverage() gave hints to the reduction algorithm. Maybe I was wrong....

--coverage makes DustMite avoid reducing code that ran. Generally that set of code shouldn't change. It's possible that it does change, e.g. if DustMite reduces some non-code declarations that cause some code to no longer run, but I can't think of a practical use case for this scenario.

the algorithm could start with remove the single lines that aren't covered typically inside conditionals and then try removing functions that were called in covered lines... Are you following?

In many cases it's just as useful to try reducing code that ran as everything else, so I can't really think of a good argument in favor of that.

Btw, my dustmite reduction takes too long time reducing packages. What strategy should I try then?

Btw, my dustmite reduction takes too long time reducing packages. What strategy should I try then?

Have you tried --strategy=careful yet?

Now. I'll try that. Can I abort a dustmite and then use the the result as input to a new dustmite?

Of course (assuming you didn't use --no-save).


Add more client side SDKs Snippets

Introduction
Awesome Appwrite Snippets is a collection of commonly used Appwrite APIs. It increases your speed of development by eliminating most of the boilerplate code associated with making using Appwrite in your Project.
Your task is to implement add snippets support for more client-side APIs.  You can look at the existing snippets in the coding language (for now JS and Dart) you prefer to see how it works.
Task Summary

[ ] Fork & clone this repository
[ ] create feat-x-api-y-lang-snippet branch where x is the REST-API you are implementing and y is the name of the language
[ ] Write code in the folder /snippets/client where client  is the folder for which snippets you are adding (web for adding snippets for web platform). If the folder doesn't exist for the platform you are creating, create a new folder.
[ ] Name should be in the form of X.code-snippets where X is the REST API (example: Account) in lowercase. Add the snippets. To add snippets refer to the contributing.md guide.
[ ] Test your snippets locally.
[ ] (Optional) If everything works perfectly, it's time to update the readme section. Look at the readme to see the structure of how it's done. Applicable only for adding snippets for new APIs.
[ ] Make a PR and just wait for the maintainer to review it😎.

If you need any help, reach out to #Maniac_Fighter#3747 on Appwrite Discord server or comment on this issue😉.
  Since there are a lot of snippets to work on, this issue can be assigned to multiple people. You can add snippets in your favorite language🤯. When requesting to be assigned, please mention which language would you like to use.
Assignees table
✅ = Done - This language has already been finished
⚒ = In progress - Someone is already working on this language
❌ = Unassigned yet - This language can be assigned to you

REST APIs
Flutter SDK
Web SDK
Apple SDK
Android SDK (Java)
Android SDK (Kotlin)

Client
✅
✅
❌
❌
❌

Account
✅
✅
❌
❌
❌

Database
✅
✅
❌
❌
❌

Storage
✅
✅
❌
❌
❌

Teams
✅
✅
❌
❌
❌

Functions
❌
❌
❌
❌
❌

Localizations
❌
❌
❌
❌
❌

Avatars
❌
❌
❌
❌
❌

Happy Contributing ✨

hey @2002Bishwajeet  could you please assign this issue to me.

Hey @RJ025 👋
What snippets you would like to work on??

@2002Bishwajeet  functions one I would like to work on.

Sure in which language?

@2002Bishwajeet  file should be in JSON format.

You can refer to the contributing guide and lemme know if anything's unclear

hey @2002Bishwajeet  just made a PR please look into it

Hey @RJ025 👋. Just checking out, are you still interested to work on this??

@2002Bishwajeet  yes sorry for late submission I just created a PR for the issue please look into it

Hey @GunjanGupta26 , go ahead. All the Best!! ✨


Build fails on systems without AVX512

Low priority, but currently the Blake3 code will not compile on x64 systems without the AVX2 / AVX512 instructions, at least with the following test system.
Test system
System: MacPro 5,1
CPU: Xeon Westmere W3680
OS: macOS 10.14
Compile errors
/Users/xxxx/Documents/tmp/chiapos/src/b3/blake3_avx2.c:20:42: error: always_inline function '_mm256_set1_epi32' requires target feature 'xsave', but would be inlined into function 'set1' that is
      compiled without support for 'xsave'
INLINE __m256i set1(uint32_t x) { return _mm256_set1_epi32((int32_t)x); }

This above is repeated for all the set, and, cmpgt, xor etc functions and also in the 512 files:
/Users/xxxx/Documents/tmp/chiapos/src/b3/blake3_avx512.c:1115:3: error: always_inline function '_mm256_mask_storeu_epi32' requires target feature 'sse4.2', but would be inlined into function
      'blake3_hash16_avx512' that is compiled without support for 'sse4.2'
  _mm256_mask_storeu_epi32(&out[0 * sizeof(__m256i)], (__mmask8)-1, _mm512_castsi512_si256(padded[0]));

I've confirmed that the get_cpu_features() function is currently identifying that AVX is not available:
SSE2: 1
SSE3: 2
SSE4.1: 4
AVX: 0
AVX2: 0
AVX512F: 0
AVX512VL: 0

However the compile itself is still including the instructions that are not available.
Workaround
The build can be made to work by removing the src/b3/blake3_avx2.c and  src/b3/blake3_avx512.c files from BLAKE3_SRC in CMakeLists.txt and then:
#define BLAKE3_NO_AVX512
#define BLAKE3_NO_AVX2

Not necessarily expecting a fix on this, but it looked like there had been some work into trying to do CPU feature identification so wanted to call it out.

I'm able to build without modifications on macOS 10.14 on a SandyBridge-EP processor that has AVX but not AVX2 or others.

Closing issue - I'm also able to build on macOS 10.15 on i7-3840QM also with AVX but not anything else.
If this issue remains, it likely is on Westmere or older CPUs

it is not the CPU.......
it is the CLANG & Xcode...
The clang was a modified version that will not even compile AVX512 instructions, it has no knowledge of them.
was done by apple to prevent developers using them incorrectly.
10.14 onwards uses a different clang & xcode that DOES recognize the instructions, even the CPU does not.
so it allows the assembly to be processed by xcode but not implemented in the CPU.
wrap or don't compile on anything less than 10.14

This is why we use Gulrak and why we only support Mojave or newer.


[Feature] Support cast to an array.

This PR allows users to initialize arrays with the cast instruction.
e.g. cast r0 r1 r2 into r3 as [boolean; 3u32], cast r3 into r4 as [[boolean; 3u32]; 1u32].

Is there any case where interleaving structs and arrays and continuously nesting them, can get past the N::MAX_DATA_DEPTH checks?


390 input checking for settings files

Pull Request instructions:

Please follow the update protocol
Answer the questions below in detail. Your responses will be emailed to experimenters.
If the experimenters must do anything new, provide detailed step by step instructions on the wiki
If computer maintainers need to manually update anything, provide detailed step by step instructions
Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
If you use the keyword "skip email" in the title, it will skip the email updates
Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:
Error checking for Setting_box csvs:

Check that there is a newline at the end of the csv file
Check that mandatory fields are in csv file.

What issues or discussions does this update address?
Issue 390.
Describe the expected change in behavior from the perspective of the experimenter
Better error checking so Bonsai doesn't crash immediately when the settings_box.csv file is not formatted correctly.
Describe any manual update steps for task computers
None.
Was this update tested in 446/447?
No.

@rachelstephlee Looks great, I will test it in 447 before approving


Simple server reticle - center dot

In recent versions the center dot for the Simple Server reticle was removed due to it overlapping with the penetration indicator. For my personal config I  had changed the blend and alpha values to mostly overcome this (blend = 0.75, alpha = 1.0).
Would it be possible to add an option so that the center dot can be enabled/disabled?

Probably yes, in free time I'll add it again with toggle

Added in 2.6.0


Pod gets stuck forever in pending state if it was deleted before initialisation

Checklist:

[x] I've included the version.
[x] I've included reproduction steps.
[x] I've included the workflow YAML.
[x] I've included the logs.

What happened:
In some cases kubernetes deletes pods before or right after it was initialized and argo misses that event - resulting in node stuck forever in "Pending" state. In our case it happens often under the load. According to our logs kube-scheduler deletes some pods shortly after they have been instantiated.
What you expected to happen:
Node changes phase to "Error" or "Failed".
How to reproduce it (as minimally and precisely as possible):

submit this workflow:apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: hello-world-
spec:
  entrypoint: whalesay
  templates:
    - name: whalesay
      container:
        image: nonexistent
        command: [cowsay]
        args: ["hello world"]

Manually delete the pod hello-world-xxxxx
Check that node is still in pending state

Anything else we need to know?:
Before this PR - https://github.com/argoproj/argo/pull/2385
this case was detected and pod's state changed to "Error"
Environment:

Argo version:

argo: v2.7.1
  BuildDate: 2020-04-07T16:40:24Z
  GitCommit: 2a3f59c10ae260a460b6ad97a0cadd8667d4b488
  GitTreeState: clean
  GitTag: v2.7.1
  GoVersion: go1.13.4
  Compiler: gc
  Platform: darwin/amd64

Kubernetes version :

clientVersion:
  buildDate: "2019-08-19T12:36:28Z"
  compiler: gc
  gitCommit: 2d3c76f9091b6bec110a5e63777c332469e0cba2
  gitTreeState: clean
  gitVersion: v1.15.3
  goVersion: go1.12.9
  major: "1"
  minor: "15"
  platform: darwin/amd64
serverVersion:
  buildDate: "2020-03-24T18:54:48Z"
  compiler: gc
  gitCommit: 71631703d37fb1b4b333abd427deae839aeb2032
  gitTreeState: clean
  gitVersion: v1.15.11-gke.3
  goVersion: go1.12.17b4
  major: "1"
  minor: 15+
  platform: linux/amd64

Other debugging information (if applicable):

workflow result:

➜  test-data git:(feature/td-45) argo get hello-world-76n8f
Name:                hello-world-76n8f
Namespace:           default
ServiceAccount:      default
Status:              Running
Created:             Thu Apr 16 15:06:04 +0300 (26 seconds ago)
Started:             Thu Apr 16 15:06:04 +0300 (26 seconds ago)
Duration:            26 seconds

STEP                  TEMPLATE  PODNAME            DURATION  MESSAGE
 ◷ hello-world-76n8f  whalesay  hello-world-76n8f  26s       ErrImagePull: rpc error: code = Unknown desc = Error response from daemon: pull access denied for nonexistent, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

executor logs:

kubectl logs <failedpodname> -c init
kubectl logs <failedpodname> -c wait

workflow-controller logs:

time="2020-04-16T12:02:20Z" level=info msg="config map" name=argo-workflow-controller-configmap
time="2020-04-16T12:02:20Z" level=info msg="Configuration:\nartifactRepository:\n  s3:\n    accessKeySecret:\n      key: access_key\n      name: argo-minio-credentials\n    bucket: argo-artifacts\n    endpoint: argo-minio:9000\n    insecure: true\n    secretKeySecret:\n      key: secret_key\n      name: argo-minio-credentials\ncontainerRuntimeExecutor: docker\nfeatureFlags: {}\nmetricsConfig:\n  disableLegacy: false\npersistence:\n  archive: true\n  connectionPool: null\n  postgresql:\n    database: argo\n    host: postgres\n    passwordSecret:\n      key: password\n      name: argo-postgres\n    port: \"5432\"\n    tableName: argo_workflows\n    userNameSecret:\n      key: username\n      name: argo-postgres\npodSpecLogStrategy: {}\ntelemetryConfig:\n  disableLegacy: false\n"
time="2020-04-16T12:02:20Z" level=info msg="Persistence configuration enabled"
time="2020-04-16T12:02:20Z" level=info msg="Creating DB session"
time="2020-04-16T12:02:20Z" level=info msg="Persistence Session created successfully"
time="2020-04-16T12:02:20Z" level=info msg="Migrating database schema" clusterName=default dbType=postgres
time="2020-04-16T12:02:20Z" level=info msg="Node status offloading is disabled"
time="2020-04-16T12:02:20Z" level=info msg="Workflow archiving is enabled"
time="2020-04-16T12:02:20Z" level=info msg="Starting CronWorkflow controller"
time="2020-04-16T12:02:20Z" level=info msg="Starting workflow TTL controller (resync 20m0s)"
time="2020-04-16T12:02:20Z" level=info msg="Starting Workflow Controller" version=vv2.7.1+2a3f59c.dirty
time="2020-04-16T12:02:20Z" level=info msg="Workers: workflow: 8, pod: 8"
time="2020-04-16T12:02:20Z" level=info msg="Performing periodic GC every 5m0s"
time="2020-04-16T12:02:20Z" level=info msg="Started workflow TTL worker"
time="2020-04-16T12:06:04Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:04Z" level=info msg="Updated phase  -> Running" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:04Z" level=info msg="Pod node {hello-world-76n8f hello-world-76n8f hello-world-76n8f Pod whalesay nil    Pending   2020-04-16 12:06:04.9990897 +0000 UTC 0001-01-01 00:00:00 +0000 UTC   <nil> nil nil [] []} initialized Pending" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:05Z" level=info msg="Created pod: hello-world-76n8f (hello-world-76n8f)" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:05Z" level=info msg="Workflow update successful" namespace=default phase=Running resourceVersion=180991 workflow=hello-world-76n8f
time="2020-04-16T12:06:06Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:06Z" level=info msg="Updating node &NodeStatus{ID:hello-world-76n8f,Name:hello-world-76n8f,DisplayName:hello-world-76n8f,Type:Pod,TemplateName:whalesay,TemplateRef:nil,Phase:Pending,BoundaryID:,Message:,StartedAt:2020-04-16 12:06:04 +0000 UTC,FinishedAt:0001-01-01 00:00:00 +0000 UTC,PodIP:,Daemoned:nil,Inputs:nil,Outputs:nil,Children:[],OutboundNodes:[],StoredTemplateID:,WorkflowTemplateName:,TemplateScope:,ResourcesDuration:ResourcesDuration{},} message: ContainerCreating"
time="2020-04-16T12:06:06Z" level=info msg="Workflow update successful" namespace=default phase=Running resourceVersion=180996 workflow=hello-world-76n8f
time="2020-04-16T12:06:07Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:10Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:10Z" level=info msg="Updating node &NodeStatus{ID:hello-world-76n8f,Name:hello-world-76n8f,DisplayName:hello-world-76n8f,Type:Pod,TemplateName:whalesay,TemplateRef:nil,Phase:Pending,BoundaryID:,Message:ContainerCreating,StartedAt:2020-04-16 12:06:04 +0000 UTC,FinishedAt:0001-01-01 00:00:00 +0000 UTC,PodIP:,Daemoned:nil,Inputs:nil,Outputs:nil,Children:[],OutboundNodes:[],StoredTemplateID:,WorkflowTemplateName:,TemplateScope:,ResourcesDuration:ResourcesDuration{},} message: ErrImagePull: rpc error: code = Unknown desc = Error response from daemon: pull access denied for nonexistent, repository does not exist or may require 'docker login': denied: requested access to the resource is denied"
time="2020-04-16T12:06:10Z" level=info msg="Workflow update successful" namespace=default phase=Running resourceVersion=181007 workflow=hello-world-76n8f
time="2020-04-16T12:06:11Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:23Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:23Z" level=info msg="Updating node &NodeStatus{ID:hello-world-76n8f,Name:hello-world-76n8f,DisplayName:hello-world-76n8f,Type:Pod,TemplateName:whalesay,TemplateRef:nil,Phase:Pending,BoundaryID:,Message:ErrImagePull: rpc error: code = Unknown desc = Error response from daemon: pull access denied for nonexistent, repository does not exist or may require 'docker login': denied: requested access to the resource is denied,StartedAt:2020-04-16 12:06:04 +0000 UTC,FinishedAt:0001-01-01 00:00:00 +0000 UTC,PodIP:,Daemoned:nil,Inputs:nil,Outputs:nil,Children:[],OutboundNodes:[],StoredTemplateID:,WorkflowTemplateName:,TemplateScope:,ResourcesDuration:ResourcesDuration{},} message: ImagePullBackOff: Back-off pulling image \"nonexistent\""
time="2020-04-16T12:06:23Z" level=info msg="Workflow update successful" namespace=default phase=Running resourceVersion=181030 workflow=hello-world-76n8f
time="2020-04-16T12:06:24Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:26Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:26Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:26Z" level=info msg="Updating node &NodeStatus{ID:hello-world-76n8f,Name:hello-world-76n8f,DisplayName:hello-world-76n8f,Type:Pod,TemplateName:whalesay,TemplateRef:nil,Phase:Pending,BoundaryID:,Message:ImagePullBackOff: Back-off pulling image \"nonexistent\",StartedAt:2020-04-16 12:06:04 +0000 UTC,FinishedAt:0001-01-01 00:00:00 +0000 UTC,PodIP:,Daemoned:nil,Inputs:nil,Outputs:nil,Children:[],OutboundNodes:[],StoredTemplateID:,WorkflowTemplateName:,TemplateScope:,ResourcesDuration:ResourcesDuration{},} message: ErrImagePull: rpc error: code = Unknown desc = Error response from daemon: pull access denied for nonexistent, repository does not exist or may require 'docker login': denied: requested access to the resource is denied"
time="2020-04-16T12:06:26Z" level=info msg="Workflow update successful" namespace=default phase=Running resourceVersion=181041 workflow=hello-world-76n8f
time="2020-04-16T12:06:27Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f
time="2020-04-16T12:06:28Z" level=info msg="Processing workflow" namespace=default workflow=hello-world-76n8f

Logs
argo get <workflowname>
kubectl logs <failedpodname> -c init
kubectl logs <failedpodname> -c wait
kubectl logs -n argo $(kubectl get pods -l app=workflow-controller -n argo -o name)

Message from the maintainers:
If you are impacted by this bug please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

Looks like my issue can be fixed by enabling resubmitPendingPods, but this still looks like a bug. Can you take a look at my PR that i closed and tell if it's the right approach? Maybe we should check if this option is disabled and move pod to Error state. What do you think?

I just had a Pod that was deleted due to node failure. Usually, Argo fails the step with pod termination and retries. In this case, it got stuck in:
Name:                md-npt-secyyhyvomdxdcy
Namespace:           tenant-redesign
ServiceAccount:      default
Status:              Running
Created:             Fri Apr 17 11:18:53 -0400 (2 hours ago)
Started:             Fri Apr 17 11:18:53 -0400 (2 hours ago)
Duration:            2 hours 10 minutes

STEP                                                            PODNAME                            DURATION  MESSAGE
 ● md-npt-secyyhyvomdxdcy (md-npt-secyyhyvomdxdcy)
 └---◷ md-npt-secyyhyvomdxdcy-0(0) (rscloud-main/rscloud-main)  md-npt-secyyhyvomdxdcy-1113725271  1h        ContainerCreating

The pod md-npt-secyyhyvomdxdcy-1113725271 does not actually exist anymore. Do you think it can be related to the same issue, or is this a different race condition of sorts?


flakeify

Changes:

add flake support
pytorch-world -> libtorch-nix
add binary (for showcasing how this is hooked into a flake).

FIXMEs:

I just hardcoded cudaSupport = false in the flake. probably I should figure out how to pass arguments in a flake-based project.
the flake.lock should probably be replaced with something cached upstream -- I'm just reusing an lock from a different project. I started off reusing commits from niv but something went wrong and I wasn't pulling from cache. everything is still looking good on the niv setup, though.

Awesome!
I don't think we should use the upstream cache. I hope the result of this skeleton will be cached.

I want to use this pr, so I'll merge it.

The previous way of using default.nix is not changed.


Company Communicator Version: V5.5.2 | Error Message: “Something Went Wrong”

HI Team,
While adding Admin App to Teams Channel, we are getting the error message:  “Something Went Wrong”.
Attached screenshots for error in Azure Application insights.

Please advise.

Hi @ArunkumarTMEYGDS
Thank you for Reaching out
Could you Please verify whether app service and the three function apps deployed successfully.
Go to Azure Portal --> Go to Resource Group (where the CC deployment has been done) --> Select function App (App Service) -->
Go to Deployment Center -> Share the Screenshot of this -> In the same tab Click on logs [send a screenshot of this].

Hi, looks like I have the same error.
This is what I get when I goes to Deployment Center of the App Service.

Greetings!

Hi @ArunkumarTMEYGDS Thank you for Reaching out Could you Please verify whether app service and the three function apps deployed successfully. Go to Azure Portal --> Go to Resource Group (where the CC deployment has been done) --> Select function App (App Service) --> Go to Deployment Center -> Share the Screenshot of this -> In the same tab Click on logs [send a screenshot of this].

Hi @peddivyshnavi ,
While opening Teams in browser we were getting this error : "Something Went Wrong".
When we tried today adding the Admin App using Teams Desktop client, the app loads for few seconds and throws the same error again.
Sharing the requested screenshots as well.

Hi @ArunkumarTMEYGDS
Could you please verify whether 3rd and 4th step from deployment guide are correctly or not

@peddivyshnavi,
Can you please confirm if we need to replace the ID "5e3ce6c0-2b1f-4285-8d4b-75ee78787346" and "1fec8e78-bce4-4aaf-ab1b-5451cc387264" with our Client IDs of Author and User App registerations?
**Click Add a client application, under Authorized client applications. In the flyout that appears, enter the following values:
Client ID: 5e3ce6c0-2b1f-4285-8d4b-75ee78787346
Authorized scopes: Select the scope that ends with access_as_user. (There should only be 1 scope in this list.)
Click Add application to commit your changes.
Repeat the previous two steps, but with client ID = 1fec8e78-bce4-4aaf-ab1b-5451cc387264. After this step you should have two client applications (5e3ce6c0-2b1f-4285-8d4b-75ee78787346 and 1fec8e78-bce4-4aaf-ab1b-5451cc387264) listed under Authorized client applications.**

@peddivyshnavi,
Can you please confirm if we need to replace the ID "5e3ce6c0-2b1f-4285-8d4b-75ee78787346" and "1fec8e78-bce4-4aaf-ab1b-5451cc387264" with our Client IDs of Author and User App registerations?
**Click Add a client application, under Authorized client applications. In the flyout that appears, enter the following values: Client ID: 5e3ce6c0-2b1f-4285-8d4b-75ee78787346 Authorized scopes: Select the scope that ends with access_as_user. (There should only be 1 scope in this list.) Click Add application to commit your changes.
Repeat the previous two steps, but with client ID = 1fec8e78-bce4-4aaf-ab1b-5451cc387264. After this step you should have two client applications (5e3ce6c0-2b1f-4285-8d4b-75ee78787346 and 1fec8e78-bce4-4aaf-ab1b-5451cc387264) listed under Authorized client applications.**

@peddivyshnavi Can you please advise. This is a roadblocker for us.

Hi @ArunkumarTMEYGDS

Can you please confirm if we need to replace the ID "5e3ce6c0-2b1f-4285-8d4b-75ee78787346" and "1fec8e78-bce4-4aaf-ab1b-5451cc387264" with our Client IDs of Author and User App registerations?
No we cant replace these Ids

Could you please verify whether 3rd and 4th step from deployment guide are correctly or not

Can you also please check Auth Issue Fix requirements

Hi @ArunkumarTMEYGDS

Can you please confirm if we need to replace the ID "5e3ce6c0-2b1f-4285-8d4b-75ee78787346" and "1fec8e78-bce4-4aaf-ab1b-5451cc387264" with our Client IDs of Author and User App registerations?
No we cant replace these Ids
Could you please verify whether 3rd and 4th step from deployment guide are correctly or not - Yes
Can you also please check Auth Issue Fix requirements - Yes we checked this article but couldnt find any routingRule2 created in the Azure Front door

Screenshots attached.
Hi @peddivyshnavi,
Please see our responses above

We are stuck on this from last week. Can you advise what else might be causing this issue.

Hi @ArunkumarTMEYGDS
Could you please check whether Set-up Authentication from deployment guide is deployed correctly.

Hi @ArunkumarTMEYGDS Could you please check whether Set-up Authentication from deployment guide is deployed correctly.

Hi @peddivyshnavi
We checked the permissions, it turned out to be issue with Tenant ID that was by default pointing prod environment internally. We had resolved it by changing this ID to our respective dev environment as we have done a multi tenant deployment.
Now we are able to view the app and able to create message, but while sending message, it throws below errror.

Help us troubleshoot this issue further.

Hi @ArunkumarTMEYGDS
Could you please share the error the supporting screenshot of error from application insights resource from the deployed resource group.
Go to resource group where CC app is deployed --> Go to Application insight type resource --> share the screenshot and stack trace for Top 3 response codes.

Hi @ArunkumarTMEYGDS Could you please share the error the supporting screenshot of error from application insights resource from the deployed resource group. Go to resource group where CC app is deployed --> Go to Application insight type resource --> share the screenshot and stack trace for Top 3 response codes.

@peddivyshnavi  Sharing requested screenshots.

Hi @ArunkumarTMEYGDS
Could you please verify whether Set-up Authentication from deployment guide is deployed correctly
Please share your email id so we can start discussing issue internally

Hi @ArunkumarTMEYGDS Could you please verify whether Set-up Authentication from deployment guide is deployed correctly Please share your email id so we can start discussing issue internally

We have already followed the steps from guide and checked twice.
Sharing contact email address<EMAIL_ADDRESS>Please revert once the email is copied (back up). I would have to remove it from this post.

Started discussing internally

we have connected internally with user, we have identified id values were configured incorrectly. after updating the values user was able to send message but still, we see failure count. and we have observed that's because of user didn't installed CC.

@peddivyshnavi
As adviced by your Team for troubleshooting, sharing status of the action items below:

Add the app to both the installed apps and pinned apps sections in the Teams Admin Center, then attempt to send a message. This has not worked
If the error continues after the first step, delete the user app in the Teams Admin Center, wait for an hour, then re-upload the user app and wait a few more hours before testing the app again. This has not worked
Sync the app service to ensure the 'delete message' button appears on the homepage of the CC author app after any modifications are made to the environmental variables of the UPNs. This has worked and resolved issue

Requesting further assistance. Sharing the errors we see now in logs.

Hi @ArunkumarTMEYGDS
We were checking on this we will get back to you in 48hours

Hi @ArunkumarTMEYGDS
we suspect this issue is related to app policies.
As a part of troubleshooting call which we have attended for GitHub issue(https://github.com/OfficeDev/microsoft-teams-apps-company-communicator/issues/1546) we see team's policies are not assigned to Global (Org-wide default) they were assigned to custom policies.
could you please reassign it to Global (Org-wide default)


Leave the cluster admin to share secrets with tenant owners

Describe the feature
There are cases where the cluster admin wants to share secrets with the tenant owners. Such secrets can be used, for example, by tenant owners to access cluster services like a private Image Registry or Helm Repo. Multiple secrets can be assigned for each tenant, so the cluster admin can create different secrets for each tenant.
The Capsule Operator propagates such secrets to all namespaces within the tenant. The tenant owners are allowed to delete these secrets but the controller recreates immediately. All supported type of secrets should be available:
-   docker-registry Create a secret for use with a Docker registry
-   generic         Create a secret from a local file, directory or literal value
-   tls             Create a TLS secret

For example:
apiVersion: capsule.clastix.io/v1alpha1
kind: Tenant
metadata:
  name: oil
spec:
  owner: # required
    name: alice
    kind: User
  sharedSecrets:
    - name: registry-credentials
       type: docker-registry
       data:
         docker-server: 'registry.clastix.io'
         docker-username: 'robot$oil+credentials'
         docker-password: 'CB753sRsmjbD1qhdKwdVsSewXBcW7SVR'
         docker-email<EMAIL_ADDRESS>   - name: other-secret
      type: generic
      data:
          key1: supersecret
          key2: verysupersecret

We should evaluate if provide the secret data in clear as above or already base64 encoded.
What would the new user story look like?

The cluster admin provisions a private Registry in the cluster
Each tenant owner wants to access the registry for pull and push of images using their credentials assigned by the cluster admin.
The cluster admin creates tenant manifests with such secret
The tenant owner logs to the cluster and creates a bounce of namespaces
The Capsule controller propagates the secret in all the namespaces and places an annotation to the namespaces reporting the name and type of the secret.
The tenant owner uses such secret to push and pull images from the registry

Expected behavior
Leave the cluster admin to share secrets with tenant owners

Rather than declaring secrets in a such opinionated way, I'd say we could take advantage of the ObjectReference API, although we would need a new polling controller in order to replicate changes across the Tenant's namespaces.
But honestly I'm thinking we could reuse the super capabilities provided by the Hierarchical Namespace Controller to replicate all the available resources in a specific target Namespace, I need to check the code and investigate it.
Going to spam their Slack workspace 😏

Don't know if it helps but I use Kubed to manage these aspects in my cluster. I create secrets in the Kubed namespace and use clastix annotations to replicate them in tenant namespaces.

@GlassOfWhiskey thanks, that's really interesting.


support CSR (data, indices, indptr), [shape=(M, N)] format in tf.sparse

System information

TensorFlow version (you are using): 2.5
Are you willing to contribute it (Yes/No): YES

Describe the feature and the current behavior/state.
The current tf.sparse constructor only support COO format ((data, (row_ind, col_ind)), [shape=(M, N)]). However, the standard sparse representation is CSR (((data, indices, indptr), [shape=(M, N)])). It would nice to see both options in tf.sparse. Scipy and pytorch already support both these cases.
I investigate a bit more and find out there are some functionalities that already implemented at repo but apparently, they are no longer maintain or accessible:
e.g., tf.compat.v1.raw_ops.DenseToCSRSparseMatrix is not working:
import tensorflow.compat.v1 as tf

tf.disable_v2_behavior()

tmp = tf.constant([[1, 2, 3, 4, 5], [0, 0, 0, 2, 1]], dtype=tf.float32)
indices = tf.where(tf.not_equal(tmp, 0))
b = tf.raw_ops.DenseToCSRSparseMatrix(dense_input=ss, indices=indices)

with tf.Session() as sess:
    sess.run(tf.global_variables_initializer())
    print(sess.run(b))

Will this change the current api? How? No, it can be handle with if-else condition based on args
Who will benefit with this feature? tf.sparse

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


vread error on image files

skvideo.io.vread raises a TypeError on image files.  Example code:
import skvideo.io
vid = skvideo.io.vread("vid_luma_frame1.png")
Environment: python 3.5.2 and ffmpeg version 2.8.11 built with gcc 5.4.0 on ubuntu 16.04
Exception trace:
/usr/local/lib/python3.5/dist-packages/skvideo/io/io.py in vread(fname, height, width, num_frames, as_grey, inputdict, outputdict, backend, verbosity)
142             outputdict['-pix_fmt'] = 'gray'
143
--> 144         reader = FFmpegReader(fname, inputdict=inputdict, outputdict=outputdict, verbosity=verbosity)
145         T, M, N, C = reader.getShape()
146
/usr/local/lib/python3.5/dist-packages/skvideo/io/ffmpeg.py in init(self, filename, inputdict, outputdict, verbosity)
192             # grabbing number of frames using ffprobe
193             probecmd = [_FFMPEG_PATH + "/ffprobe"] + ["-v", "error", "-count_frames", "-select_streams", "v:0", "-show_entries", "stream=nb_read_frames", "-of", "default=nokey=1:noprint_wrappers=1", self._filename]
--> 194             self.inputframenum = np.int(check_output(probecmd).split('\n')[0])
195
196         # Create process
TypeError: a bytes-like object is required, not 'str'

Ah, I'll look into this later, but it looks like you found a bug. Scikit-video has not been thoroughly tested with image files, but it should be since FFmpeg can handle them.

Python 2.7 works fine. Python 3.6 shows the same error as Python 3.5.

This makes sense since raw data streams are handled differently between python 2 and python 3.

@beyondmetis What is the ETA on this bug?

@zacharynevin I'm not sure. I'm spending 100% of my time preparing for my PhD qualifying exam. I'm open to merging pull-requests

Fixed the problem.


SPA won't run when using guarded browser functions inside +layout.ts/+page.ts

Describe the bug
The gist of the issue is that a properly configured SK SPA application that uses browser API code inside of a +layout.ts/+page.ts file breaks npm run dev and npm run preview even when the browser code is guarded by the building variable.
I've been running into this issue since updating to v1.0 of SK and I belive it has to do with this change: https://github.com/sveltejs/kit/discussions/7716
I have a settings store that is backed by localStorage that I import pretty much everywhere. When trying to build I get the following error:
file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:23
  const val = localStorage.getItem(settingsKey) || "{}";
              ^

ReferenceError: localStorage is not defined
    at localSettings (file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:23:15)
...

I figured the above referenced changes were the issue, so I guarded the localStorage references inside my settings store. This allowed me to successfully build with npm run build.
However, when I try to serve up the build with npm run preview or run npm run dev, I get the same build error. If SSR and prerender are disabled and adapter-static is properly configured for SPA, then to my knowledge the code should only run on the server during build, but obviously the code is either running outside of build or SK does not properly set the building variable when running dev or preview.
I've create a minimal repoduction of the issue in this repo:
https://github.com/freb/sk-spa-issue
There you'll be able to confirm that I'm setting the proper exports in the root layout and the adapter-static settings in svelte.config.js.
Interestingly, while creating the reproduction I discovered the issue does not manifest when importing my settings store into +page.svelte, but only into +layout.ts or +page.ts.
Reproduction
https://github.com/freb/sk-spa-issue
Logs
file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:23
  const val = localStorage.getItem(settingsKey) || "{}";
              ^

ReferenceError: localStorage is not defined
    at localSettings (file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:23:15)
    at createSettings (file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:39:21)
    at file:///home/me/code/cas/web/.svelte-kit/output/server/chunks/settings.js:59:18
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)

Node.js v19.3.0
[vite-plugin-sveltekit-compile] Prerendering failed with code 1
error during build:
Error: Prerendering failed with code 1
    at ChildProcess.<anonymous> (file:///home/me/code/cas/web/node_modules/@sveltejs/kit/src/exports/vite/index.js:551:15)
    at ChildProcess.emit (node:events:513:28)
    at ChildProcess._handle.onexit (node:internal/child_process:293:12)

System Info
System:
    OS: Linux 6.1 Arch Linux
    CPU: (24) x64 AMD Ryzen Threadripper 1920X 12-Core Processor
    Memory: 52.73 GB / 62.65 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 19.3.0 - /usr/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.19.2 - /usr/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: ^1.0.0 => 1.0.0
    @sveltejs/adapter-static: ^1.0.0 => 1.0.0
    @sveltejs/kit: ^1.0.0 => 1.0.1
    svelte: ^3.54.0 => 3.55.0
    vite: ^4.0.0 => 4.0.4

Severity
blocking all usage of SvelteKit
Additional Information
No response

You're importing a module (src/stores/settings.ts) that proactively accesses localStorage as soon as it's imported, whether that happens on the server or in the browser. building is irrelevant, the guard you need is this:
-import { building } from '$app/environment';
+import { browser } from '$app/environment';

// later...
-if (building) {
+if (!browser) {
  return;
}

Thanks, Rich.
I'm confident that will fix the issue, though I'm not sure why whether or not the app is building is irrelevant. Prior to the referenced changes (at least at some point before the changes referenced here) the old escape hatch for disabling SSR (from hooks.js) worked just fine with the npm run dev command which at the time used svelte-kit.
After the referenced changes I was under the impression that server-side code should only be running during build time or when SSR is enabled. My guess is that the Vite server is taking some liberties. I'd love to know if this is just how it is with the vite server or if there is something fundamental I'm not understanding about disabling server-side execution of code. My closest guess is that vite has to run the code for HMR, but again it was working with previous versions of SK without guards.

You're not disabling the server, you're just disabling SSR. Your +layout.ts and +page.ts files need to be imported in order to determine whether SSR is enabled for a given page, and since settings.ts is imported by those files, localStorage is accessed as soon as handle is invoked.
If you were only importing settings.ts from your +layout.svelte and +page.svelte files, the code wouldn't run on the server.

But isn't the importing of +layout.ts and +page.ts only done during build time? I realize I'm not disabling the server, but shouldn't the function of the Vite server be to simply deliver the static files (particularly in preview mode) or to watch for changes and recompile (in dev mode). And if Vite server is watching and recompiling, shouldn't that set the build flag to true?
I'm just trying to understand when Vite server would be importing +layout.ts during a time it is not performing a build.
To take the clearest example, the Vite docs claim:

The vite preview command will boot up a local static web server that serves the files from dist at http://localhost:4173. It's an easy way to check if the production build looks OK in your local environment.

This is clearly not what it is doing since I get an error on my terminal from when Vite tries to run code (not a browser error). But if I serve the same build folder with Caddy, it works just fine.

But isn't the importing of +layout.ts and +page.ts only done during build time?

No. It's done whenever the server responds to a request, whether that's done during dev, preview or prerendering/building (which in your case is disabled).

After writing what I have below, I realized that using browser functionality outside of +page/layout.svelte files is probably not very common. And given how much code needs to be guarded, your solution is very workable. Though I do think my particular use case would be reasonably common in an SPA (i.e. get a session token from localStorage and check its validity in +page/layout.js before rendering the page).
I'm going to look into Vite a little more before I let this drop, but I'm going to close the issue. Thanks a bunch for a solution and helping me get at the root of the issue. I leave my first draft of this comment below for posterity:

I think we're talking past each other. I'm assuming a world where node.js is not running on the server because I'm developing an SPA application. SvelteKit seems to support SPA's with the options I've included in my example project. I've confirmed that with proper build guards for browser functions, I can build and deploy the app just fine.
The issue is that Vite does not seem to know that I am developing an SPA where no code is intended to run on the server. So using vite dev and vite preview do not work as expected, because they attempt to execute my code.
As you mentioned, I can further guard my code so that if it were to run on a node.js server it would not error, but that's not what someone building an SPA would expect.
It seems that either Vite itself or the way that SvelteKit integrates with Vite does not have a way of preventing project code from being executed (omitting the execution during build time which is documented and expected).
There seem to be three options:

Fix Vite or the SK/Vite combo so that it is possible to serve code without executing it outside of build time
Realize that Vite is going to execute your code no matter what and use guards based on browser instead of building
Realize that Vite is going to execute your code no matter what so change package.json to a different static web server for at least npm run preview and possibly npm run dev (since vite won't work anyway)


Storage queue as a deadletter destination

The document said "Storage queue as a deadletter destination" but the example uses Blob Storage.
Do you have an example of using Storage Queue as the deadletter destination?

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

ID: 3bff81eb-b641-b705-8632-52d0857b0e04
Version Independent ID: 2977b7d3-afba-276b-c74b-a102ffd36cb8
Content: Storage queue as an event handler for Azure Event Grid events - Azure Event Grid
Content Source: articles/event-grid/handler-storage-queues.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@pksorensen Sorry for the confusion! Only Blob Storage is the supported dead letter destination, and the sample is meant to highlight how that can be setup along with storage queue as a destination.
We will work up a PR to address this, which once merged should reflect in a couple of hours.


Breadcrumbs: align decorations to the right

Refs: https://github.com/Microsoft/vscode/issues/57280
Looks like decorations in breadcrumbs come right after the label and not at the end:

I suggest having the same looks everywhere.

Nice find - must be some recent regression.

fixed via c3e2448766


Internal error RangeError: value (5) is smaller than min (10)

Internal error
RangeError: value (5) is smaller than min (10)
This page will auto refresh in 56s

+1

Please provide some details on how to reproduce this.
Slider doesn't allow to set it to less than 5, did you manually modify local storage in your browser?

This error show in Chrome when i press Setting button in right upper corner.

Clear website data for karma domain and it should work