What is the limit for # of moderators?

If possible, could we please document the max amount of moderators?  For example, how many moderators can be added to a Teams Channel?
Thank you

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

ID: 905e2c09-ef3e-07cc-0809-6ff171157c47
Version Independent ID: c0d33162-183d-fff8-32e9-a999f391ea51
Content: Limits and specifications for Microsoft Teams - Microsoft Teams
Content Source: Teams/limits-specifications-teams.md
Service: msteams
GitHub Login: @MicrosoftHeidi
Microsoft Alias: heidip

@MicrosoftHeidi, do you think we can provide this information? Thanks!

@MicrosoftHeidi, do you think we can provide this information? Thanks!

@MicrosoftHeidi, do you think we can provide this information? Thanks!

@MicrosoftHeidi, do you think we can provide this information? Thanks!

@MicrosoftHeidi, do you think we can provide this information? Thanks!

@MicrosoftHeidi, do you think we can provide this information? Thanks!

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Allow passing in authentication client secret as an environment variable

TL;DR
Allow passing in authentication client secret as an environment variable in addition to file path location
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/flyteorg/flyte/issues/2744
Follow-up issue
NA

this is just for the CC/api key type auth right?
yes (I don't think there's any equivalent for pkce, right?)


[BEAM-2241] Renames some python classes and functions that were unnecessarily public

Adds a note to documentation of classes that are public but should be only used internally by the SDK (non-user facing classes).
Marks some of the modules as experimental.
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @aaltay

R: @sb2nov

LGTM (assuming the tests will pass).

LGTM


Cmdlet Naming convention

Cmdlet generally uses a "prefix" to noun to differentiate from other modules.
For instance: Get-ADUser vs. Get-AzureADUser
Generally, use "singular" names for cmdlet. This "singular" cmdlet is also used to get a list.
For instance, Get-ADUser or Get-AzureADUser can return a single account (based on its guid/login) or a list of accounts, filtered or not.
Verbs should follow the convention as documented by Microsoft: https://learn.microsoft.com/en-us/powershell/scripting/developer/cmdlet/approved-verbs-for-windows-powershell-commands?view=powershell-7.3
For instance, Test can be used to test source health, etc.
Search-Post do not reflect any noun and should not be used or perhaps not publicly available, and underneath a Find-Indice.
I'd rather have Find-Identity, Find-AccessProfile, etc. than an unmeaningful, complex to use cmdlet.

This is a valid concern, and understand that it is the convention of Powershell. As of today, though, the automation used in generating our SDKs with OpenAPI Generator doesn't allow for this to be implemented.


Logo design contribution

Hi
I want to make a logo for this projects if you have any requirements then let me know.its totally free for open source projects.

If you have interest on new logo then I can design it.


Not strong mode clean

Using the latest dartanalyzer, dartanalyzer --strong reveals many strong mode errors (almost all "unsound implicit cast" errors).

Here are the errors I see:
  error • The method 'stop' isn't defined for the class 'Function' at example/http_server.dart:108:12 • (undefined_method)
  error • The return type 'Future<List>' isn't a 'FutureOr<List<int>>', as defined by the method '' at example/runner_pool.dart:46:14 • (return_of_invalid_type)
  info • The type of '_id' can't be inferred because of the use of the instance getter 'hashCode' at example/http_server.dart:55:38 • (strong_mode_top_level_instance_getter)
  info • The value of the local variable 'event' isn't used at example/http_server.dart:118:18 • (unused_local_variable)
  info • The name Completer is shown, but not used at example/runner_pool.dart:7:34 • (unused_shown_name)
  info • 'throws' is deprecated and shouldn't be used at test/registry_test.dart:353:62 • (deprecated_member_use)

Fixed now, me thinks


Some features OS.jsv2 need :)

I have a collection of features that OS.jsv2 need to add :p
In media player: need a volume bar, a repeat button,... and a picture of song :)
In file manager: add back, forward button
In setting: remove app store and put it in an app
In environment(desktop): add view type, sort by, new context menu,... and COPY, PASTE, CUT :lol
In Office app: add presentation app, spreadsheet too
In Mulitimedia: add a video viewer app
In OS: add login, register function
In installer: can install in web, don't just support in VM
That's what I want OS.jsv2 to have ;). You can look at some eyeOS app source :P

In media player: need a volume bar, a repeat button,... and a picture of song :)

It's techically just a music player. If I ever create a "Media Player", this will  be one of the features

In file manager: add back, forward button

Yeah. I want this too :)

In environment(desktop): add view type, sort by, new context menu

You're thinking about the icon view ? I really should add "new file" in context. Thanks.

,... and COPY, PASTE, CUT :lol

I started implementation of this in FileManager. Will finish it some day ;)

In Office app: add presentation app, spreadsheet too

If you know about a good presentation and/or spreadsheet library then add it there. I won't go through the hassle of making my own, hehe.

In Mulitimedia: add a video viewer app

You can view videos with "Preview" application :)

In OS: add login, register function

This is already provided in the source. By default it is configured to use the "demo" handler wich does not need any login.
Also, registration is a bad idea on the live demo site. I had this a year ago... but after doing some analytics it turns out >50% of the users visiting simply closed the page immediately because they had to register. It's better to get them straight into the action!

In installer: can install in web, don't just support in VM

You don't need a VM ;)
I'll look more into a web-installer later.

oh, i have a bad bug :P. The media player can't seek music when music name have Hieroglyphs.
Like this:

oh, i have a bad bug :P. The media player can't seek music when music name have Hieroglyphs.

This is not a bug. It says "seek unavailable in format" which means the browser cannot do it ;)

E.g: It is a problem with the file

okay, but this one:

I rename the first file and here it is

How did you get that error ? I just visited your server (with the address you left ther) and when I double-clicked the file I got no error. Did you do something else ?
I downloaded the mp3 file to my computer just in case. It might be nice to have for testing.

a ha! i missed that ^_^~. You need add dockbar, gadget feature to OS.js
Like this:

Now I see... you tried to rename or move the file ?

a ha! i missed that ^_^~. You need add dockbar, gadget feature to OS.js

Don't know if it needs it, but it might be cool to have:)

don't you see ;), windows have bar(you can pin app to launch it), ubuntu have sidebar,...
And search app feature too~!

Yeah, i tried to rename file but it loading a min and display error.

@khoaakt I have never used gadgets and fancy bars on my computer. I only have a background image and terminals, hehe. So I am not a big fan. But I will have a look at it :)

And search app feature too~!

Sure. This could be done, but will have to be placed in the "Touch Menu" (you can change it in Settings)

And in future, OS.js will be like Chrome OS, Firefox OS,.. . This will be compile to a linux kernel and run in mobile, tv, laptop,...

And in future, OS.js will be like Chrome OS, Firefox OS,.. . This will be compile to a linux kernel and run in mobile, tv, laptop,...

You can run it like that already, but is not really the goal of OS.js.
https://github.com/andersevenrud/OS.js-v2/blob/master/INSTALL.md#x11

I'm sorry :(, i can't support to programming OS.js because i interested at .NET language but i can tell some way to go for OS.js

You could always learn a bit of JavaScript and join in on the fun! ;)
Thanks for the suggestions  (i took the freedom to change your post description so it is a bit clearer) :)

haha, because i'm Vietnamese then i'm not good at English :)

Hump... something not right :P. OS.js isn't have any terminal :lol

Hump... something not right :P. OS.js isn't have any terminal :lol

That's because I don't see the point to be honest. Who wants to use the web browser to type in commands ?! hehe :P
I have a Terminal client sitting here that I never added to the source-code ready for use though. Maybe I should add it to the "experimental" repository

i have an idea! put terminal in setting, user can turn on/off it :tada:

@khoaakt No need. You can just build OS.js without the package (or simply disable it in manifest). Making an option in Settings is kinda redundant

If you plan OS.js that run in touch screen, make a visual keyboard :)

Don't they already have that ? At least my touch devices do :P

A Update? Did OS.js have a update ?(I means OS.js update, it's download update file from server and  pack directly into OS core)

Sorry, I don't understand what you mean.
You have to update OS.js manually, either by git pull when running from source or unpack a fresh zip from github (overwrite old files).

oh, it just like Windows Update :P

git and distribution packages are much better in my opinion. I could maybe add update function to the web installer we talked about, but not inside OS.js.

oh ! That is nice idea ;). An update from outside os is more useful. But you need have a admin account to access the admin system control

I'm not interested at code in OS.js :(. Please convert some apps i post to OS.js @andersevenrud
(Note: this package contains app i posted but i made it. I'm collecting it from internet :P)
Link: http://khoaakt.w.pw/file/App.zip
... And we need a web browser app, i'll post it later :)

Take a look at this:

hey @andersevenrud, do OS.js have a is-online function? If not, add that to API. That need to dev some app that need internet to run ;)

@khoaakt Yes it has. You can test it by disabling internet access. A message will be disaplayed warning you

do you think add a web browser is a bad idea? i think it need because os.js will run in a linux-kernel without node

To create a web-browser application you would have to set up a proxy, or else it will not work because of the Same-Orgin Policy. It does not make much sence to have it in a browser-deployed environment, but it may be useful for running in X11/Standalone. However, this is on the bottom of the priority list.

really? if need, i will create a web browser for os.js. Let me do that :-)

Looks like I should weigh in here (sorry about going dark).
As for a dock, that might be nice, but I'd put that really low on priority.  You can move the current panel to the bottom to replicate the traditional desktop paradigm.
Regarding a slideshow, I'd like to see a front end developed for dzslides (which would probably be easiest to implement in os.js).

@khoaakt I won't stop you ;)

@MrSchism I've been away for a while too. Had to take a break. I started having serious tetris syndrome symptoms because of too much coding, hehe :p
Yeah, something like that could be cool. I have not checked into any of these types of apps before, because I've never had any use for them. There's proabably a few to choose from. Know any more worth looking at ?
I'm going to focus getting the last API stuff in place and finish the UI Thingamajigs done.

@andersevenrud Do Application IDE done yet? In this time, IDE is need for develop! Please add that to OS.js as soon as possible ;)

@MrSchism The OS.js dock probably will be in the left/right screen ;)

@khoaakt No. And to be perfecltly honest, I won't look at it any time soon (Look at the issue tracker... it's full of stuff undone). I suggest you rather start to learn some fundamental JavaScript and CSS. That is all that is required (only the very basics are required). Actually... all you have to know is how to create an object and insert it into the dom and apply styling through CSS.
Here is some info to get you started:

http://stackoverflow.com/tags/javascript/info
http://stackoverflow.com/tags/css/info

If you code 18h a day? I am seriously concerned for your health! I hope you take breaks and work out.

When it comes to slide decks, dzslides is probably the easiest to use and implement.  Issue is it would require an editor.  Shouldn't be hard because each slide is a section in a single HTML file.  Should just have to insert tags and render everything inside those tags in the javascript editor.
I know that's how things like the blackboard educational system works:

@MrSchism Cool. Just googled around, and it seems some of the slide libraries with editors are premium/paid stuff. So it would be cool to have this as a free service. Hosting is no problem.

Prezzi is a paid slide deck; impress.js is a free alternative, but complicated.  DZSlides is free and single file.  The editor above could probably be recreated easily and new slides just have to have a  tag put in.  Might be a minimal side job, but shouldn't be impossible.

@andersevenrud I'm going to learn Javascript ^_^!
P/S: Just kidding, no one can alive when code 18h/day :P

@khoaakt @MrSchism I'm merging this with #49


Fix tutorials failures.

ptb_word_lm should work, but want to see how cifar10_train will look.

ptb_word_lm is fixed (great!)
But looks like cifar10_train is still failing, maybe read_input.label needs to call set_shape() as well?

Thanks a lot for the fix, @gunan !

Hi, with the new version of read.py I get the following error message:
TypeError: strided_slice() takes at least 4 arguments (3 given)

@physicsai are you using compatible versions of this file and the pip package?
If you have tensorflow 0.12.0rc0 installed, you need to use the version of the file in r0.12 branch of this repo. If you have built tensorflow from source, you can use this file.
Also, please reach out to us by either filing an issue, or through stackoverflow. Pull requests, after merging, cannot really address any questions anymore.

@gunan  It was indeed an issue with tensorflow verions. It runs all fine when I clone the r0.11 files. Thanks for your help!


byttet metabase

Coverage decreased (-0.4%) to 40.097% when pulling b8a1a89bf42cadbe15f67cab7d3a1dde5fc869dd on feature/endaEnNyMetabase into 0faeb4b5881a5d59168b6f8c55f29d9e37796231 on master.


Add missing support for subpaths

When using export and subpaths, to fully support CJS and ESM, we need each subpath to contain import and require.
Previously, ./Renderer didn't support CJS, because it didn't have a require field. This would cause problems in packages, like itwinjs/core core-full-stack-tests, that consumes both Main and Renderer, but outputs CJS files. At build time, this wouldn't be caught, but during runtime, it would throw a PACKAGE_PATH_NOT_SUPPORTED, because when it tried to use the Renderer subpath, it couldn't find a valid CJS path.
Vice versa, with ./Main in ESM outputted files.
This issue is fixed by adding both import and require and their valid values to each subpath.

Excellent. I ran into this setting up my docs cospace the other day. Thanks!


How does metatron druid configure the cluster

How does metatron druid configure the cluster？

@FourSpaces This document is used internally, can you try it? And please let me know if you have any problems.
[Metatron] Druid Cluster Install Guide_ver1.1.pdf


sway: add config.bindkeysToCode

Description

Do not use --to-code by default in bindsym, let user configure that behavior manually via config.bindkeysToCode in sway module.
Refs https://github.com/rycee/home-manager/pull/1229#issuecomment-636301091
Checklist

[x] Change is backwards compatible.

[x] Code formatted with ./format.

[ ] Code tested through nix-shell --pure tests -A run.all.

tests fail with:
hash mismatch in fixed-output derivation '/nix/store/sd0pnznvqy3l9k773anr14kv5imchvg8-delta-0.0.16-vendor.tar.gz':
  wanted: sha256:1zmk70hccrxn1gdr1bksnvh6sa2h4518s0ni8k2ihxi4ld1ch5p2
  got:    sha256:0hah0qfgnl4w2h0djyh4xx1jks5dkzwin01qw001dqiasl60prn2
cannot build derivation '/nix/store/a9j70nsz3x37mkakcqjw60rl1zs4s4p6-delta-0.0.16.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/7z4qg944anpk1hfjfaxx9zv13c9bpinx-git-expected.conf.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/75y9mfswm5x82lrj28dii5yg6lyzmg68-hm_gitconfig.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/bc22arqm3vjb37jqdb2lag30p2rj9ajv-home-manager-files.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/2zzxyhscan5n2zgjqflbhkcc5vrcjvqh-nmt-test-script-git-with-most-options.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/zrdw6v8dixg1gap8scw11i8ccjh79hgv-nmt-report-git-with-most-options.drv': 1 dependencies couldn't be built
error: build of '/nix/store/zrdw6v8dixg1gap8scw11i8ccjh79hgv-nmt-report-git-with-most-options.drv' failed

[ ] Test cases updated/added. See example.

[x] Commit messages are formatted like
{component}: {description}

{long description}

See CONTRIBUTING for more information and recent commit messages for examples.

If this PR adds a new module

[ ] Added myself as module maintainer. See example.

[ ] Added myself and the module files to .github/CODEOWNERS.

cc @dermetfan for testing

@dermetfan can you confirm that this patch fixes your issue?

Yes, thanks for the quick reaction! No idea why Travis fails now...

Beside the comment I added above this looks OK to me.
The test failure is due to SRI hashes introduced in Nixpkgs, these are unsupported in the Nix version used by Travis CI. I believe the SRI hashes are being removed from Nixpkgs master which should clear up the errors once the removals hit the unstable channel.

Thanks! Rebased to master in 85748171ec0dc78fea92240d1c3f8f34713ed874.


Stockholm hackathon

[x]  reserve dates (Nov 6-7)
[x]  reserve room (Östra Galleriet in KTH Library)
[x]  announce
[x]  team of instructors complete
[x]  workshop website up
[ ]  lessons distributed
[ ]  helpers
[ ]  registration open
[ ]  catering
[ ]  registration closed
[ ]  enough pre-survey answers
[ ]  install instructions sent
[ ]  pre-workshop briefing held
[ ]  survey results shared with co-instructors/helpers

An interesting possible repro hack article about a glitch in Python code affecting >100 chemistry studies. This might involve too chemistry specific details for repro hack though. Goes beyond my understanding.
https://arstechnica.com/information-technology/2019/10/chemists-discover-cross-platform-python-scripts-not-so-cross-platform/
Full article: https://drive.google.com/open?id=1kaK4NMGGDwhfDzuCWDDLUjGM5qvlZOjn (CRbase/workshops/2019/hackathon-stockholm-nov-2019/repro-hack/01-juho-pyhton-glob-in-chemistry/)


Choose the licence and copy the text in LICENCE file.

oocgcm will be distributed under Apache 2.0.


Change cargo fix --edition to only fix edition lints.

This changes it so that cargo fix --edition will only fix edition lints.  The reason for this is that sometimes non-edition lints get in the way, and make suggestions that can cause failures.  An example is a user that only ever runs cargo test or cargo check --profile=test locally, and doesn't realize there are problems with running without cfg(test) such as unused warnings.
This works by using --cap-lints=allow along with --force-warn which takes precedence over cap-lints.
This only works on nightly since --force-warn is still unstable.  I will update this as part of #9800.
Closes #5738

Oh I see what you mean about #[test]. Yeah if it's more fundamental in that there's no bugs here I think this is the way to go 👍

@bors: r+

:pushpin: Commit 46450d6e3af09ff0de365a8d16d0f728adbf2662 has been approved by alexcrichton

:hourglass: Testing commit 46450d6e3af09ff0de365a8d16d0f728adbf2662 with merge c22756541acb9e4cb428e9fbca53610edba1c501...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing c22756541acb9e4cb428e9fbca53610edba1c501 to master...


HowTo Browse for files or folders should tell user to right click on that widget

added this as well as some information about optional pins, thanks for the suggestion!


[CLI] wandb: WARNING Config item 'hyperparam_name' was locked by 'sweep' (ignored update)

Description
When running basic hyper parameter search (following this tutorial very closely) I get:
wandb: WARNING Config item 'hyperparam_name' was locked by 'sweep' (ignored update).
for many, but not all, of my hyper parameters
Wandb features
Using WandB with Pytorch Lightning to perform simple hyper parameter search. WandB logger callback for pytorch lightning used to log relevant metrics.
######### Pseudo Code #########
import wandb
from pytorch_lightning.loggers import WandbLogger
import pytorch_lightning as pl

sweep_config = {
    "method" : "random",
    "metric" : {"name": "val/error_epoch", "goal": "minimize", "target": 0.05}, #error of 5%
    "early_terminate" : {"type": "hyperband", "min_iter": 20, "eta": 3},
    "parameters" :
        {
        "share_parameters": {"value": share_parameters},
        # architecture hps
        "iterations": {"values": [2, 5, 10]},
         .... more params ...
        # trainer hps
        "epochs": {"value": 500},
        "batch_size": {"values": [16, 32, 64, 128, 256]},
        }
}

def train(config=None):
    with wandb.init(config=config):
        # Config is a variable that holds and saves hyperparameters and inputs
        config = wandb.config

       # construct trainer, model, & data module  by setting relevant arguments with 'config.param_name'
       dm = my_dm(batch_size = config.batch_size, ....)
       trainer = pl.Trainer(max_epochs = config.epochs, WandbLogger(project=my_project_name), ....)
       model = my_model(iterations=config.iterations, ...)

       trainer.fit(model, train_dataloader=dm.train_dataloader(), val_dataloaders=dm.val_dataloader())

       #if on colab
       wandb.finish()

wandb.agent(sweep_id, train)

Environment
Occurs both on my local machine (2015 MacBook Air, macOS Big Sure 11.2.3) using Python 3.7.9 and Google Colab with a GPU using Python 3.7.11. Using most up to date Pytorch Lightning and WandB.
It seems someone else has had this issue as well but it has not been resolved. Interestingly I am also getting a CUDA out of memory error; someone else in the aforementioned unresolved issue also had these two exact issues. Not sure if these issues are connected, but I will post a new issue for that soon.
Thanks,
Max

Hey @maxwass are you calling wandb.config.update(...) anywhere in your code?  The pytorch lightning integration calls config.update which would be triggering the error.  In this case you should be able to ignore the error because wandb.run.config will be prepopulated with the appropriate config.  We can look into fixing this in the integration in a future release.

Nope not calling wandb.config.update() , all the methods called using wandb are shown.
So just to be clear, even though pytorch lightning calls wand.update causing these warnings, everything should be working as expected?

Yep, everything should still work.

hi, is there any way to unlock the sweep parameters so I can update the config anyway?

Hi,
I received the message "locked by 'sweep' (ignored update)." and am not using pytorch lightning. What is the origin of this error message? Thanks.

@erlebach this happens when you try to update a value inside wandb.config during a sweep. this object seems to be locked during sweeps, maybe so that you don't inadvertently set a value that the sweep would change.


Invalid trx_id for replaced deferred transaction in AcceptTrx message

IMPACT: When contract replace deferred transaction, AcceptTrx message contains old trx_id for transaction.
STR:

schedule deferred transaction from contract
replace deferred transaction with same sender_id from contract
Wait for transaction executed

Actual result:
AcceptTrx contains old trx_id (for first transaction), ApplyTrx and block contains trx_id for replaces transaction
Expected result:
Same trx_id in all events

Fixed in #1032


migrate not working

Is it just me or is migrate stopped working ("fabric-bolt migrate" throws unknown command and it doesn't show on help).
Migrate is still documented ...

Looks like it is trying to run south migration. We have switched away from south. If you get the error still, try pip uninstall south first.


The materialized view has four base tables, one of which is empty, resulting in the failure to rewrite the materialized view.

create table a;
create table b;
create table c;
create table d;
CREATE MATERIALIZED VIEW view1
DISTRIBUTED BY HASH(`)
REFRESH ASYNC EVERY(INTERVAL 1 MINUTE)
PROPERTIES (
"replicated_storage" = "true",
"replication_num" = "1",
"storage_medium" = "HDD"
)
as select  id from a join b join c join d;
There is data in tables a, b, and d, but no data in table c.
explain  select  id from a join b join c join d , the materialized view has not been rewritten.
code
com/starrocks/sql/optimizer/Optimizer.java:498
//ruleRewriteIterative(tree, rootTaskContext, RuleSetType.PRUNE_EMPTY_OPERATOR)
after annotation, it can be routed to the materialized view.
why?
Steps to reproduce the behavior (Required)
Expected behavior (Required)
Real behavior (Required)
StarRocks version (Required)

There is no benefit to rewrite empty table, seems no bug here.
Can you try it again with some datas?


Improved perfromance through actual vectorizing

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.63%. Comparing base (353357c) to head (cb72100).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   87.50%   87.63%   +0.13%
==========================================
  Files           6        6
  Lines         624      631       +7
  Branches      107      107
==========================================
+ Hits          546      553       +7
  Misses         41       41
  Partials       37       37

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


Reading: Update F&P thresholds

This is on top of https://github.com/studentinsights/studentinsights/pull/2774, to avoid churning the snapshot tests (since that PR has some styling changes with wider impact, and this one does with adjusting thresholds).
Who is this PR for?
MTSS facilitators, reading specialists, literacy coaches, K5 classroom teachers
What problem does this PR fix?
This came up with literacy coaches, and so there are some slight revisions to the benchmarks here, but the main intention was to pick these for each of the screening time periods, and extend them out to fifth grade as well.  That way if there's screening, there's a way to support highlighting these as well.
What does this PR do?
Updates the thresholds, and then some styling for <ReadingScheduleGrid />, which is used for visualizing the thresholds, and the full raw set of data points over time.  Both are intended more for validating data, but the substantive change is that the highlighting in the reader profile will change to match these thresholds.  We'll validate this with folks looking at the overall distributions for different cohorts, particularly after this next interval.
Screenshot (if adding a client-side feature)

Checklists
Which features or pages does this PR touch?

[x] Student Profile
[x] Reading thresholds
[x] Reading benchmarks by cohort

Does this PR use tests to help verify we can deploy these changes quickly and confidently?

[x] Included specs for changes
[x] Improved specs for existing code in need of better test coverage
[x] Manual testing made more sense here

selfie


Remove express when not required

I noticed that axios-extension is using express for a very simple server use case. I replaced it with usage of the http module and can reduce the installation size of the package and all its dependencies from 13m to 11m.
I also noticed that our middlewares are using a specific version of express and when they are used somewhere else by e.g. the ui5-tooling using a different express version, then express gets installed twice, so I replaced it as well with the simple http module

[x] remove express from axios-extension (11mb -> 9mb installation size)
[x] remove express from backend-proxy-middleware (22mb -> 19mb installation size)
[x] remove express from ui5-proxy-middleware (17mb -> 15mb installation size)

@ullasholla please closely look at the changes to the axios-extension to make sure I didn't change any logic. @zdravko-georgiev please do the same for the middlewares.
All unit tests run without issues and my manual tests also didn't show a problem but still, please look very closely.

@ullasholla please closely look at the changes to the axios-extension to make sure I didn't change any logic (please also checkout the new unit test)
@zdravko-georgiev please do the same for the middlewares - especially the ui5-proxy-middleware because I also reduced the cognitive complexity of some of the code. All unit tests run without issues and my manual tests also didn't show a problem but still, please look very closely.

Thanks @tobiasqueck, I will check

I admin-merged it because @zdravko-georgiev is on vacation, however, he approved the relevant parts for him a few times already.


Issue

Hi,
I am trying to use your app but some problems are appearing when I use the extract bottom. See picture below. Could you please help me to solve?

I am using a new installation of a anaconda only with the required packages for your app.

Hi @canela90,
If it is still not solved, you can find my answer below.
There are three possibilities;

Your dataset is problematic, you can try to download it again.
You are not checking the variables for analysis.
Your coordinates out of the dataset.

Murat.

Thank you for your attention but I am still having the same problem. I saw that you are able to run it, could you please provide some screenshots to see how you enter the coords, or the process.

Okey, after trying again I had some isuues with the eccode library and now i am able to download easily time series,
conda install -c conda-forge eccodes


how to train on new data and how to use on GPU environments?

i am beginner of deep learning and i know the previous issue solution about which is how to train on your blog but i have a question that your explanation of auxilary network is not clear. Is auxilary network concatenated by main network? or independtly?  so i want to know how to train main architecture and auxilary network clearly and i have one more question that how to use on GPU environments when inference step?

"and i have one more question that how to use on GPU environments when inference step?" just install tensorflow-gpu instead tensorflow, it will run automatically on GPU


Why unpublish "Cumulants from Möbius inversion"?

Hey there, I've been self-teaching myself about statistics (my background is in pure math and theoretical physics). I found this blog post a few months ago, and I found it really insightful and a pleasure to read. I wanted to share it with some friends, but I noticed that it was no longer on your website.
I wanted to compliment your writing, and also ask why you'd unpublish such a well-written article. Any chance of getting it back? Thanks!

Glad you found the blog post helpful! Thanks so much for the positive feedback :) I think I unpublished it because I wanted to add something about QFT but obviously never did. I've republished it!

Great, thanks for republishing! Please do ping me if you publish your QFT post and think of me. :smiley:


dapp: fix navigation loop when adding new token

Fixes #2607
Short description
See the commit message
Definition of Done

[x] Steps to manually test the change have been documented
[x] Acceptance criteria are met
[ ] Change has been manually tested by the reviewer (dApp)

Steps to manually test the change (dApp)

If possible change the token first if multiple are connected (i.e. don't use the per default selected token)
Attempt to connect a new token
Continue the navigation until selecting a hub
Click back twice
Verify that you are back on the transfer route (home screen) and the last selected token is still active
Do the same again from 1. on, but continue the navigation up to the amount to insert for the new channel and then go back three times

Note that the dApps e2e tests are still broken for any PR...

Thanks for the review. So the end-to-end tests are unfortunately meant to fail atm. And the coverage drop of -0.01% is immaterial (though I don't understand where it comes from...). l gonna rebase to migrate the fixup commit and then merge.


🛠  Gulp setup

refs #7427
With this PR, we'll get some development tooling magic:
gulp setup
Will update dependecies and submodules (if submodule branches are on master) for
currently chosen branches.
gulp setup --force
Will delete all dependencies and install them again, incl. submodules for
currently chosen branches.
gulp setup --ghost some-branch --admin some-branch --casper some-branch
Will checkout the branches for each repository.
Can also be used to checkout only selected branches e. g.
gulp setup --admin some-branch
Will leave the current branch for ghost and casper, but checkout the named
branch for admin. Will also install dependencies.
gulp setup --admin pr/123 --ghost pr/1234 --casper pr/123
Will fetch the named PR of the repository and checkout to this new branch.
Will also install dependencies. NOTE: This works only with an additional fetch line
in the .git/config file for each repository: fetch = +refs/pull/*/head:refs/remotes/upstream/pr/*.
See https://dev.ghost.org/easy-git-pr-test/ for further information.
All the combinations above can be executed with the --force or -f flag, which
will delete the dependencies and install them again, but for the chosen branches.

Note: if the built folder is not present, we still need to use grunt to built everything.
gulp dev watches the built files and it won't work without a built folder.
@AileenCGN will work on that for upcoming 1.0.0-alpha3. Might be a sub-task of gulp setup.


Text wrapping not functioning

Unsure if I implemented it wrong but text wrapping seems to not function for me.
My code is as follows:
const WRAP_WIDTH = 222;
function brainImage(first, second, third, fourth) {
  return Promise.all([
    Jimp.read('input.jpg'),
    Jimp.loadFont(Jimp.FONT_SANS_16_BLACK),
  ]).then((data) => {
    const [loadedImage, font] = data;
    return new Promise((resolve, reject) => {
      loadedImage
        .print(font, 10, (157.25 * 0) + 10, first, WRAP_WIDTH)
        .print(font, 10, (157.25 * 1) + 10, second, WRAP_WIDTH)
        .print(font, 10, (157.25 * 2) + 10, third, WRAP_WIDTH)
        .print(font, 10, (157.25 * 3) + 10, fourth, WRAP_WIDTH)
        .getBuffer(Jimp.AUTO, (err, buffer) => {
          if (err) {
            return reject(err);
          }
          return resolve(buffer);
        });
    });
  }).catch((err) => {
    console.error(err);
  });
}

Text seems to not get wrapped at all. Am I doing something wrong here?

Same issue, adding a maxwidth of any value does not work for me

This seems to work in the current version. Feel fre to file a new issue if the problem persists!

It still happens in the latest version ;-;


IRGen: fix swifterror attribute mismatch for WebAssembly

On WebAssembly, tail optional arguments are not allowed because Wasm requires callee and caller signature to be the same. So LLVM adds dummy arguments for swiftself and swifterror. If there is swiftself but is no swifterror in a swiftcc function or invocation, then LLVM adds dummy swifterror parameter or argument. To count up how many dummy arguments should be added, we need to mark it as swifterror even though it's not in register.

Related to SR-9307.

@kateinoigakukun I'm following your previous changes and comments here. I hope the description makes sense. Do you think there's a way to add any tests for this, or will tests require the driver change to be merged first to enable testing the WASI/Wasm triple?

@swift-ci please smoke test

@MaxDesiatov IRGen things can be tested without the driver change. (But we need to fix the current CI failure at first.)

@kateinoigakukun now that community CI failure is fixed, how do we set IGM.ShouldUseSwiftError to different values for testing? Or having another look at the diff, maybe we should modify isSwiftErrorLoweredInRegister function instead here?

@swift-ci please smoke test

Looks like we've hit a conflict:
C:\Users\swift-ci\jenkins\workspace\swift-PR-windows\swift\lib\SILGen\SILGenApply.cpp(1913): error C2668: 'swift::Lowering::PreparedArguments::PreparedArguments': ambiguous call to overloaded function
C:\Users\swift-ci\jenkins\workspace\swift-PR-windows\swift\lib\SILGen\ArgumentSource.h(307): note: could be 'swift::Lowering::PreparedArguments::PreparedArguments(swift::Lowering::PreparedArguments &&)'
C:\Users\swift-ci\jenkins\workspace\swift-PR-windows\swift\lib\SILGen\ArgumentSource.h(294): note: or       'swift::Lowering::PreparedArguments::PreparedArguments(llvm::ArrayRef<swift::AnyFunctionType::Param>)'

@swift-ci please test Windows

swifterror argument for call has mismatched parameter
%swift.error** %6
  call swiftcc void @swift_willThrow(i8* swiftself undef, %swift.error** noalias nocapture readonly dereferenceable(4) %6) #5
in function $sSYsSeRzSb8RawValueSYRtzrlE4fromxs7Decoder_p_tKcfC

is related I guess?
@kateinoigakukun have you seen this error before?

Hmm weird. I perhaps missed somewhere that calls the function. Let me investigate

@swift-ci please test

@swift-ci please test

OK, I forgot that swifterror is not supported on x86. CI is now green ✅


Blue 'overlay' when using scrcpy

Environment

OS: Ubuntu 20.04
Scrcpy version: 2.5
Installation method: manual build (I run it from a local copy of the repo via ./run x ...)
Device model: Samsung Galaxy A15 and Google Pixel 3
Android version: Samsung: 14; Google: 12

Describe the bug
I consistently get some kind of blue 'overlay' over my screen. There are a few different actions I can take to clear it, but it typically comes back after just a few minutes. I really only use scrcpy to play one specific game, so I'm not sure if this happens with other apps or not. If I leave the app, the overlay is not there, so it appears to be 'attached' to the app. This happens with both phones.
This blue overlay appears both on the scrcpy window on the computer and on the phone. If I close scrcpy and continue only on the phone, the overlay is there, but does not come back once I clear it. It only appears when using scrcpy and happens whether I'm using a usb connection or wifi.
Running scrcpy with the -Vverbose flag did not appear to have anything unusual when the overlay appears vs when it does not.
Here are screenshots of what I expect to see:

Versus what I actually see:

This blue overlay appears both on the scrcpy window on the computer and on the phone.

That's very weird. I don't know.

That's very weird. I don't know.

I was afraid of that, but thought I'd check just in case.


Unicorn mode for BSD proposal.

@hexcoder- @andreafioraldi  - please have a look :)

I don't like to have many build scripts. @devnexen can you integrate this into the old build script?

I could but I fear it would get dirtier and dirtier by adding more platforms.

What os is this ? Normally you should not need -isysroot flag though.

Thats Ubuntu bionic 18.04.3 LTS linux.

Thanks! @vanhauser-thc: Any idea what is causing this weird travis build fail?
name collision?
/usr/local/clang-7.0.0/bin/clang++ `llvm-config --cxxflags` -Wl,-znodelete -fno-rtti -fpic -O3 -funroll-loops -Wall -D_FORTIFY_SOURCE=2 -g -I ../include/ -DVERSION=\"++2.58d\" -Wno-variadic-macros -DLLVMInsTrim_EXPORTS -fno-rtti -fPIC -std=gnu++11 -shared afl-llvm-pass.so.cc -o ../afl-llvm-pass.so `llvm-config --ldflags`
afl-llvm-pass.so.cc:34:10: fatal error: 'list' file not found
#include <list>
         ^~~~~~
1 error generated.
Makefile:173: recipe for target '../afl-llvm-pass.so' failed

fixed, your comment on the commit was correct.

ignore the build failure for now, there is one more issue in our testing script that we need to fix, has nothing to do with your PR (yet)

I had a second look. Is there a tool for reporting the number of cores available in *BSD and Darwin?
Then we could avoid hardcoding -j4 in make. Other than that, I am fine with it.

There is but it is a bit verbose we might need to trim words
hw.ncpu: 8

please note that I made a change to the unicorn build script

the build check fails because of:
aving to: ‘24f55a7973278f20f0de21b904851d99d4716263.tar.gz’

24f55a7973278f20f0d     [   <=>              ]   3.18M  7.28MB/s    in 0.4s

2019-11-07 13:45:38 (7.28 MB/s) - ‘24f55a7973278f20f0de21b904851d99d4716263.tar.gz’ saved [3337409]

[-] Error: signature mismatch on 24f55a7973278f20f0de21b904851d99d4716263.tar.gz (perhaps download error?).

Makefile:273: recipe for target 'distrib' failed

make: *** [distrib] Error 1

and that is only in your PR, doesnt matter how often I restart the build test.

Yes I did a typo during one commit ...

so this time it was not ***** travis but it actually caught a bug. that a first! :)


test(migrate): migrate test tooling from chai + mocha to vitest

test(migrate): migrate test tooling from chai + mocha to vitest
Fixes #194
Description of changes:

Migrate test tooling to Vitest
Migrate test files to typescript

Tag a reviewer: @ayoayco
Tasks:

[x] I have ran the build command to make sure apps are working: npm run build
[x] I have ran the tests to make sure nothing is broken: npm run test
[x] I have ran the linter to make sure code is clean: npm run lint
[x] I have reviewed my own code to remove changes that are not needed

Wow this is great @fazzaamiarso let me just run through the changes. But already with pipelines passing I think this is very good!


On exit reset changes

Issue:
When exiting the application I am not told the original tdp of the system nor is it reset.
How to fix:
When exiting reset the values to original state

Please refer to this link:https://github.com/zqhong/RyzenTuner/issues/11
Translation of main content:
My laptop's CPU is AMD Ryzen™ 7 PRO 6850HS, the power consumption data cannot be obtained with ryzenadj, and it returns NaN. Other models have not been tested.
This function may only be implemented by some models of CPU notebooks, and we will not consider adding it for the time being.


[Feature]: Pytorch Audio is missing from WSL Repo

Suggestion Description
I'm having trouble finding a PyTorch Audio version that is compatible with the current PyTorch version provided in the WSL ROCM repo. I'd appreciate if the maintainers could look into providing a compatible PyTorch Audio version that I can use. Alternatively, if there's already a compatible PyTorch Audio version available, please let me know so I can use that. I'm happy to provide any additional details about my setup to help resolve this issue.
The Repo in question: https://repo.radeon.com/rocm/manylinux/rocm-rel-6.1.3/
Operating System
Ubuntu
GPU
7900XTX
ROCm Component
No response

Hi @xCentral, let me confirm the compatible Torchaudio version and get back to you.

pip3 install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/rocm6.1

# replace libhsa-runtime64.so so torch.cuda.is_available() returns True
location=`pip show torch | grep Location | awk -F ": " '{print $2}'`
cd ${location}/torch/lib/
rm libhsa-runtime64.so*
cp /opt/rocm/lib/libhsa-runtime64.so.1.2 libhsa-runtime64.so

pip3 install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/rocm6.1

# replace libhsa-runtime64.so so torch.cuda.is_available() returns True
location=`pip show torch | grep Location | awk -F ": " '{print $2}'`
cd ${location}/torch/lib/
rm libhsa-runtime64.so*
cp /opt/rocm/lib/libhsa-runtime64.so.1.2 libhsa-runtime64.so

Hiya, my goal was to stick with AMD's repo, in hopes they would actually update their repo for WSL. However I've been using that in the mean time.

Well, in my opinion the wheels in AMD repo are the wheels they have extensively tested against WSL. It's not necessary to stick with those as long as the official ones actually work.
Honestly speaking, it's a smart strategy to release a support for WSL. Although it's still in beta, it already allows many AI applications to work pretty well.
Ultimately, working with WSL should be like Linux: it will no longer need to manually patch anything, and the official wheels should just work. Just like today's NVIDIA experience.

Hi @xCentral, torchaudio wheels will be uploaded to https://repo.radeon.com/rocm/manylinux/ starting with ROCm 6.3.
In the meantime, you can use the generic wheels from download.pytorch.org/whl/rocm6.1 as highlighted by @evshiron.


Do not set configuration of a processor whose status is to be set to STOPPED

The patch has been verified in my test environment

SonarQube analysis reported 14 issues

 5 major
 8 minor
 1 info

Watch the comments in this conversation to review them.
Top 10 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractHadoopProcessor.java#L104: Put single-quotes around ':' to use the faster "indexOf(char)" method.
 AbstractHadoopProcessor.java#L124: Make this anonymous inner class a lambda
 AbstractHadoopProcessor.java#L301: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.
 AbstractNiFiProcessorsRestClient.java#L154: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractHadoopProcessor.java#L61: Override the "equals" method in this class.
 AbstractHadoopProcessor.java#L412: Reorder the modifiers to comply with the Java Language Specification.
 AbstractNiFiProcessorsRestClient.java#L97: Remove the parentheses around the "proc" parameter
 AbstractNiFiProcessorsRestClient.java#L108: Remove the parentheses around the "proc" parameter
 AbstractNiFiProcessorsRestClient.java#L122: Remove the parentheses around the "proc" parameter

SonarQube analysis reported 9 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractHadoopProcessor.java#L104: Put single-quotes around ':' to use the faster "indexOf(char)" method.
 AbstractHadoopProcessor.java#L124: Make this anonymous inner class a lambda
 AbstractHadoopProcessor.java#L301: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.
 AbstractNiFiProcessorsRestClient.java#L154: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractHadoopProcessor.java#L61: Override the "equals" method in this class.
 AbstractHadoopProcessor.java#L412: Reorder the modifiers to comply with the Java Language Specification.
 AbstractNiFiProcessorsRestClient.java#L153: Remove this empty statement.
 AbstractHadoopProcessor.java#L183: Complete the task associated to this TODO comment.

This pull request contains fixes for two separate issues: KYLO-2270 and KYLO-1499.  Please provide the changes for these two issues as separate pull requests so they can be reviewed and accepted separately.
It does not appear that the provided changes for KYLO-2270 are correct.  Please see the Jira for comments.

SonarQube analysis reported 9 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractHadoopProcessor.java#L104: Put single-quotes around ':' to use the faster "indexOf(char)" method.
 AbstractHadoopProcessor.java#L124: Make this anonymous inner class a lambda
 AbstractHadoopProcessor.java#L301: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.
 AbstractNiFiProcessorsRestClient.java#L154: Either re-interrupt this method or rethrow the "InterruptedException".
 AbstractHadoopProcessor.java#L61: Override the "equals" method in this class.
 AbstractHadoopProcessor.java#L412: Reorder the modifiers to comply with the Java Language Specification.
 AbstractNiFiProcessorsRestClient.java#L153: Remove this empty statement.
 AbstractHadoopProcessor.java#L183: Complete the task associated to this TODO comment.

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.

The PR currently has conflicts.  Can you please resolved them?  Thanks.

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.

SonarQube analysis reported 2 issues

 2 major

Watch the comments in this conversation to review them.
1 extra issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 AbstractNiFiProcessorsRestClient.java#L8: This block of commented-out lines of code should be removed.

As this PR still has rebase conflicts I went ahead and cherry picked the appropriate commits from this repo into the Kylo repo.


Unable to put link in footnotes

I would request that in addition to plain text, the footnote can contain a link.
For instance, I would like to be able to have a footnote with a link that can take me back to "contents" slide in a presentation or some other useful location within the presentation.
It looks like the footnote that is specified in the PITCHME.yaml file is always rendered as plain text.

Hi @themantalope I've just added support for arbitrary HTML fragments to be used when specifying your custom footnotes. This will allow you to add links as requested.
For example, in your PITCHME.yaml file add the following:
footnote : "Presentation Menu"

This should create a link in the footer location that when clicked should automatically take you to the third slide in the presentation.
Given you are injecting custom HTML you are responsible for styling it. By default, HTML anchors are rendered using the traditional blue font color. Which doesn't look great alongside the GitPitch presentation themes. So I recommend that you add a little style of your own.
For example, assuming you are using one of the dark themes then making your footnote-link white helps it stand out:
footnote : "Presentation Menu"

Alternatively, if you are planning on using one of the light themes then making your footnote-link black is probably the way to go:
footnote : "Presentation Menu"

Note, I've also disabled the text-decoration style on this link as I think it looks better on the slide but of course this is of optional.
As a final note. As an alternative to adding styling directly within your footnote HTML you could simply define a class on your HTML footnote anchor and then use GitPitch custom css to apply whatever styling changes you want for this link. Which approach you take is entirely up to you.
If you try this out let me know how it goes so we can update this issue. Good luck.

Note, the GitPitch Wiki has been updated to reflect this new support for HTML fragments within footnotes.

Beautiful! Thanks!


Add image pull secrets to support using private image registries

Thanks @lewiscowper for contributing,
Please also bump chart version, update default values.yaml and README accordingly

@pmint93 Have updated them now :)


Innovar Healthcare Onboarding – Create Org and Sender Settings

Acceptance Criteria

[ ] Added to organization.yml and create yamlitos

[ ] Customer status = testing
[ ] For HL7 senders, schemaName: hl7/hl7-ingest-covid-19-training

[ ] Added to prod settings table

[ ] Added to staging settings table

[ ] Key exchange

[ ] 1. Shared secret

[ ] Created API keys in Azure staging
[ ] Created API keys in Azure prod
[ ]  Sent API keys to Sender via Keybase

[ ] 2. Token auth

[ ] Received public key from sender in staging

[ ] Saved public key in keybase

[ ] Received public key from sender in prod

[ ] Saved public key in keybase

[ ] Associated public key with sender settings in staging
[ ] Associated public key with sender settings in prod

FYI: this ticket can be completely concurrently with “Validate Test Files” ticket for this API sender
To Do

[ ] RS engineer: update this ticket with comments with every interaction with sender
[ ] RS team member who originates this ticket: add any dependencies, as needed
[ ] RS team member who originates this ticket: complete the following when creating this Issue.

[ ] Pipelines: Onboarding & Operations (New)
[ ] Labels: "Onboarding-ops" and "Sender"
[ ] Epics: [select the epic for this API onboarding sender]

No longer interested.


[Chrome] SVG image not loaded on startup

See https://github.com/phetsims/circuit-construction-kit-dc/issues/124
It looks like an SVG image issue occurred (most easily reproducible by loading many copies of the sim).
The image specified didn't show up at all in Chrome, even though the SVG <image> element had the proper URL.

Worked around in CCK-DC by pushing the image into a normally loaded image. It is probably typically caused with images that are dynamically generated.


Adds search box

[x] Add state
[x] Add debouncing
[ ] Fix issue with search query not being handled when switching page (by passing the state via Parent component)

Unlinking as it will only partially solve the issue


Improve error stack trace

It is hard to trace errors because most of them are generic. There should be logging or some other sort of error tracing to reduce this.

solved


Document Profiles feature

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1353
Originally created by @DouweM on 2019-12-04 00:11:08

Once https://gitlab.com/meltano/meltano/issues/1331 is resolved, we can deem the Profiles feature (https://gitlab.com/meltano/meltano/issues/764) complete, and document it.
We should also document how it can be used with Google Analytics in particular (https://gitlab.com/meltano/meltano/issues/1349#note_254185941).

View 12 previous comments from the original issue on GitLab


fix: add better support for readonly types

BREAKING CHANGE: interface DeepMergeMergeFunctionURItoKind's signature has changed

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

npm package (@latest dist-tag)
GitHub release


fish shell command completion

[Fish shell] (http://fishshell.com) is the bomb! I cannot believe I have lived this long without it.
omf install jaws-cli
for command completion!

Does that command work for you? It doesn't work for me, and I can't find "jaws-cli" in the global registry. You could submit a PR on oh-my fish :smile: https://github.com/oh-my-fish/oh-my-fish/blob/master/docs/en-US/Packages.md#make-it-public
$ omf install jaws-cli
Installing package jaws-cli
fatal: repository 'jaws-cli' does not exist
Could not install package jaws-cli.


Version 2.0.0 broke?

I used the beta versions of the 2.0.0, but today I updated to the non-beta and this error appears:
Module not found: Error: Can't resolve 'wavesurfer.js'
My package.json is: "wavesurfer.js": "^2.0.0"

Yep this happened on friday. Ping @katspaugh

@katspaugh dont forget to bump the version for bower as well. and to make a tag for 2.0.1

I've just published 2.0.1. Thanks! 👍

@katspaugh yeah but bower is still using the old version nr: https://github.com/katspaugh/wavesurfer.js/blob/master/bower.json#L3


Enhancement for multiple instances in one servlet engine

To enable the operation of several wallride instances in one servlet engine (tomcat), there is now the possibility to configure the wallride.home parameter via a config file placed under the webroot directory.
Example:
If you deploy wallride as war to a tomcat to /srv/myblog.war you can place a config file to /srv/myblog.conf or /srv/wallride.conf with content:
wallride.home=file:/srv/wallride-home/
There is no need to config JAVA_OPTS for tomcat
Multiple Instances:

For Instance 1 war-file may be placed to /srv/webapps/myblog.war
Place config file to /srv/webapps/myblog.conf

For Instance 2 war-file my be placed to /srv/webapps/yourblog.war
Place config file to /srv/webapps/yourblog.conf

I think this might be a really good enhancement.
Tested with Tomcat8
br franz stumpner

@stumpner
Thank you again, it is much better. we merged it into master repo.

@bourbonizable how can i put this into your documentation under http://wallride.org/docs/guide.html ?


Index.collection is nil

Hi there!
I created my index and then tried to used it but it doesn't work since:
class MyIndex < Typesensual::Index
  schema do
    field 'name', type: 'string'
  end
end

MyIndex.collection
#=> nil

MyIndex.create!
#=> #<Typesensual::Collection:0x00005571433884e8 @collection={"fields"=>[{"name"=>"name", "type"=>"string"}], "name"=>"my@1706022998"}>

MyIndex.collection
#=> nil

What's wrong here?

You need to MyIndex.update_alias!(MyIndex.create!) so that the index alias is pointed at the current collection — MyIndex.collection references the alias instead of a specific collection
Do you think it would help if I made MyIndex.collection raise an exception when the alias doesn't exist? 🤔

Yep, probably better!
I wonder where it would be the best place to call MyIndex.update_alias!(MyIndex.create!) if every single Index would need this call before being used 🤔

There's some rake tasks to handle creating the collection and pointing an alias at it, as well as a .reindex! method which recreates the indices and points the alias at it, but perhaps a reindex task would be helpful? Reindexing is largely supposed to be a rare, manual task (vs incremental updates on record change) so I think rake is the right place for it, but it's clear that the current tasks are not clear or easily automated (the output of collection creation needs to be used as the input for the alias updating, not great). I should probably also clarify the readme on this


[release-0.18] Bump google.golang.org/grpc from 1.53.0 to 1.56.3

Description of changes:
Cherry-pick of #6895 and #6896
Bumps google.golang.org/grpc from 1.53.0 to 1.56.3.

Release notes
Commits
Testing (if applicable):

/hold
until we tag v0.18.0, this will go out in v0.18.0 since we are not affected by it.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold

/approve
/lgtm


Expose both semver + gitsha

Git sha's are hard to immediately identify - expose the semver as part of the member status (and healthcheck endpoint).

Re: #127


githubpages.io does not render correctly, localhost works

https://imgur.com/a/ECgtLLt
http://localhost:1313/ works while https://wereket.github.io/ does not render
Thanks for your time

Please see https://github.com/h-enk/doks/discussions/431

Updated https://github.com/h-enk/doks-gh-pages


Consider looking into 3DDFA in the future

This solution was provided in ECCV2020 for 3D Face Mesh Tracking and the results looks very promising.
https://github.com/cleardusk/3DDFA_V2

This version of the implementation has been tested and no noticeable gain has been found
https://github.com/1996scarlet/Dense-Head-Pose-Estimation


[$500] Web - Login - Sign in with Google banner in ../sign-in-with-google page jumps

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

Action Performed:

Go to https://staging.new.expensify.com/sign-in-with-google
on Safari
Refresh the page

Expected Result:
The Sign in with Google banner will not jump
Actual Result:
The Sign in with Google banner jumps
Workaround:
Unknown
Platforms:

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

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

Version Number: 1.3.73-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/99e2f778-d813-4ec3-9979-42b1dc0524e3
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d1eaacbfa4ad6b5d
Upwork Job ID:<PHONE_NUMBER>733585408
Last Price Increase: 2023-10-05

i can reproduce this on Safari:
https://github.com/Expensify/App/assets/38016013/7ed9c30a-34f2-4c78-9fab-d726e79ef2b2

Proposal
Please re-state the problem that we are trying to solve in this issue.
Sign-in with Google banner keeps jumping up when page is refreshed
What is the root cause of that problem?
GoogleSigninButton component is not rendered properly
What changes do you think we should make in order to solve the problem?

Use SocialIcon component that has a built-in type for Google.
What alternative solutions did you explore? (Optional)
N/A

Hi everyone, I'm Solomon Ochepa by name. I'm a full-stack developer (General PHP, Laravel, CodeIgniter, etc.) with more than ten (10) years experience.
I saw your post on the UpWork platform. Before proceeding, I'd like to know the stack.
Thanks.

Contributor details
Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01dfbdd135c64803fa

Proposal
Please re-state the problem that we are trying to solve in this issue
Web - Login - Sign in with Google banner in ../sign-in-with-google page jumps
What is the root cause of that problem?
We call an external script on src/components/SignInButtons/GoogleSignIn/index.website.js, but the component is rendered first before all the styling finishes loading. This makes the layout shift, when the styling is loaded, and makes the button jump
https://github.com/Expensify/App/blob/3b2dc326a52127b6e5552b81c9f14e5e2c5a8126/src/components/SignInButtons/GoogleSignIn/index.website.js#L58-L69
If we inspect the network, we can see that the layout shift (button jump) is happening when font 4UabrENHsxJlGDuGo1OIlLU94YtzCwM.ttf is loaded

What changes do you think we should make in order to solve the problem?
We need to wait for all the script and styling inside GoogleSignIn button to be finished first and then render the component after that.
This way when GoogleSignIn button shows, it won't jump.
What alternative solutions did you explore? (Optional)
None
@maddylewis @situchan

@situchan - bump on proposal review - thanks!!

I don't see solid proposal yet

I don't see solid proposal yet

Do you have any feedback on my proposal?
We can see from the Network activity that the Button showed first and the layout shift (button jump) is happening when font 4UabrENHsxJlGDuGo1OIlLU94YtzCwM.ttf is downloaded & loaded as my proposal, thus I believe this is the Root Cause of it

We need to wait for all the script and styling inside GoogleSignIn button to be finished first and then render the component after that.
This way when GoogleSignIn button shows, it won't jump.

@studentofcoding I am not sure how you will implement this. Can you share demo video after applying solution?

Sure @situchan here below are the result and code solution of it
Result
https://github.com/Expensify/App/assets/20473526/91b7c84e-312b-4e08-9a65-3f9cec402b99
Solution
React.useEffect(() => {
        const script = document.createElement('script');
        script.src = 'https://accounts.google.com/gsi/client';
        script.addEventListener('load', loadScript);
        script.async = true;
        script.onload = () => {
            loadScript();
            const googleButtonContainerDesktop = document.getElementById(desktopId);
            const googleButtonDekstop = googleButtonContainerDesktop.firstChild;

             // After the script is loaded, there is a styling change, therefore we intercept it by hide the button 1st and manually wait until all done (approx 750ms) show the button
            googleButtonDekstop.style.display = 'none';
            setTimeout(() => {
                googleButtonDekstop.style.display = 'block';
            }, 750);
        };

        document.body.appendChild(script);

        return () => {
            script.removeEventListener('load', loadScript);
            document.body.removeChild(script);
        };
    }, [loadScript]);

Hey @situchan, I've just updated the video, as yesterday the link is broken, Please kindly check again here :)

@studentofcoding I checked your solution in detail but not deal to me

Hey @situchan, Upon more dig and observation here are the optimal solution
TLDR:
We need to track all the script loaded from google button via window.performance.getEntriesByType('resource'); & use setTimout to wait for Font Change (from https://fonts.gstatic.com/s/*).
This is happening because upon checking our entries tracker, the last logged activity is from https://accounts.google.com/gsi/button instead of https://fonts.gstatic.com/s/*, therefore we need additional setTimout to track for Font Change

@situchan - lmk your thoughts on this one https://github.com/Expensify/App/issues/28080#issuecomment-1768081034

@situchan - lmk your thoughts on this one #28080 (comment)

I am sure that there's better & simple solution to fix jump issue.
Still awaiting proposals

@situchan - we really need an update! can you lmk your thoughts on this when you have a chance? ty! https://github.com/Expensify/App/issues/28080#issuecomment-1771294617

I already reviewed https://github.com/Expensify/App/issues/28080#issuecomment-1768081034. Still I think it's not ideal solution. We can just prevent this by fixing style.

If the alternative is fixing it via changing the width and height for googlePillButtonContainer, like the one shared here then it's only fixing the problem on Chromium-based Browser (Chrome, Brave, etc) and not Safari.
Here are the demo via fixing style, below

Chromium Browser

https://github.com/Expensify/App/assets/20473526/1adb7bda-f3e6-44b8-976c-e2b34663681b

Safari

https://github.com/Expensify/App/assets/20473526/33de92d4-a17f-4456-bbd2-40cc44a5addc
Based on my last update, the RCA is not on the button styling (that we can control like above), but rather on how some Google Sign-In styles are loaded late. Thus, the only way to fix it is by showing the button after all the late styles are loaded.
cc: @situchan @maddylewis

i'll let @situchan confirm whether or not @studentofcoding's proposal makes sense to move forward with.

I am fine with @mkhutornyi's proposal as using same solution applied in circle button.
🎀 👀 🎀 C+ reviewed

This one landed on production a while ago. But seems like the issue got stuck. Did everyone who was supposed to get paid get paid here?

No payment yet

Payment Summary

[ ] Reporting bonus - N/A
[ ] C+ - @situchan ($500 via Upwork)
[ ] Contributor - @mkhutornyi ($500 via Upwork)

@mkhutornyi - remind me, are you a contractor or do you require payment through Upwork?

thanks! paid 👍


Send dfu reset to fix image upload

Post exploit, the usb stack or loop is in a bad state, reset is required to fix immediate image upload.
If you are all for tools using gaster I suggest you accept this. Tools fail because image upload fails after gaster.

I was told this breaks checkm8 payload, so might need to find a workaround for decrypt calls and what not

Thanks.


Expiry data does not work as expected in the iOS mobile app

Bug reported by kshitij

We are observing strange behaviour with native iOS app as shown below. On scanning a code with Blockchain network is ‘epiqa’. The output of web app in iOS browser is expected output. Please look into this.

With native iOS app:

With web app in iOS browser:

Hi,
I can't reproduce this issue. Bellow are the screenshots from iphone and browser.


Added new method SoRayPickAction::getRadius (issue #107)

Original pull request by Mario Koerner (Bitbucket: Renreok, GitHub: Renreok).

Fix for issue #107.
I used the function signature proposed by Roy for the new method (same as already used by VSG Inventor).

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Hi guys,
The function looks great, but for absolute perfection, well, consistency really, I would like to request one small change to both files. Could you please modify the request to include 'void' as the function argument? The declaration would be:
float getRadius(void) const;
Similarly in the definition. With that change I'll be happy to immediately merge. Apologies for failing to do that in my original proposal.
Roy

Original comment by Mario Koerner (Bitbucket: Renreok, GitHub: Renreok).

Hi Roy,
I updated the pull request according to your comment. I usually try to program according to the conventions of the surrounding code, but this time I missed the void argument. I also changed the tab into two whitespaces in the function definition.
Best regards,
Mario
P.S. Sorry for breaking the Mac and Ubuntu build with my change for the compiler warnings, so I really understand that you insist on following the coding style. Do you have coding guidelines somewhere that help to prevent such problems?

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Mario,
Thanks for the code changes. I'll merge it now.
With respect to coding style I have no documentation on that. I just follow what was done before, attempting to keep things as consistent as possible.
Roy

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

The Coin Code Standards are in the HACKING file:
https://bitbucket.org/Coin3D/coin/src/c00dd036d48c784ddc52ac0a5242bba7c2f7eea3/docs/HACKING?fileviewer=file-view-default#HACKING-673
There is no mention about void arguments though.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thanks Bastiaan. I'd forgotten all about that!


Add pupur repository and dependencies

Pupur is a fork of Paper that designed for configurability, and more performance.
That has an API and maven repository, so could you add a helper function for this?

@rona-tombo Sure, that will be added in next minor 2.3.0.


Definitions of topological features [sf#104]

Reported by batchelorc on 2008-07-08 11:49 UTC
See: https://sourceforge.net/tracker/?func=detail&atid=886178&aid=1894636&group_id=177891
"circular" and "linear" from a topological point of view mean something quite different from the same qualities in  PATO.
Colin.

Commented by eilbeck on 2008-07-09 20:19 UTC
Logged In: YES
user_id=742851
Originator: NO
Hmmm, here are their definitions, and its not what we mean  at all.
[Term]
id: PATO:0001199
name: linear
def: "Narrow, with the two opposite margins parallel." [ISBN:0881923214]
subset: cell_quality
subset: value_slim
is_a: PATO:0000052 ! shape
[Term]
id: PATO:0000411
name: round
def: "A shape quality inhering in a bearer by virtue of it being such that every part of the surface or the circumference is equidistant from the center." [thefreedictionary.com:thefreedictionary.com "http://www.thefreedictionary.com/"]
subset: cell_quality
subset: value_slim
exact_synonym: "circular" []
is_a: PATO:0000052 ! shape
To start with we need to define our terms. How about
circular: a topological quality inhering in the bearer polymer a closed form.
linear: a topological quality inhering in the the bearer polymer a start and an end.
???
--K

Commented by batchelorc on 2008-07-24 10:13 UTC
Logged In: YES
user_id=1473024
Originator: YES
It occurs to me that polynucleotides can't branch, can they?  Whereas you do get cross-linking and so forth in polypeptides.  So we could have qualities zero-ended and two-ended?
name: zero-ended
def: "A quality of a nucleotide polymer where all nucleotide residues are connected to two others." [SO:cb]
OR
name: zero-ended
def: "A quality of a nucleotide polymer that has no terminal nucleotide residues." [SO:cb]
name: two-ended
def: "A quality of a nucleotide polymer that has a 3'-terminal residue and a 5'-terminal residue." [SO:cb]
How does that sound?
Colin.

Commented by eilbeck on 2008-08-07 18:53 UTC
Logged In: YES
user_id=742851
Originator: NO
I have added your definitions to these terms and your proposed names as synonyms.
--K

Updated by eilbeck on 2008-08-07 18:53 UTC

status: open --> closed-accepted


Explicit termination method 'end' not called on Android

Similar to #3 and #17
It looks like inflater.end() not called in InflaterInputStream.
2020-01-11 21:48:13.555 23005-23020/tech.bluespace.android.id_guard_test D/StrictMode: StrictMode policy violation: android.os.strictmode.LeakedClosableViolation: A resource was acquired at attached stack trace but never released. See java.io.Closeable for information on avoiding resource leaks.
        at android.os.StrictMode$AndroidCloseGuardReporter.report(StrictMode.java:1877)
        at dalvik.system.CloseGuard.warnIfOpen(CloseGuard.java:286)
        at java.util.zip.Inflater.finalize(Inflater.java:407)
        at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:289)
        at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:276)
        at java.lang.Daemons$Daemon.run(Daemons.java:137)
        at java.lang.Thread.run(Thread.java:919)
     Caused by: java.lang.Throwable: Explicit termination method 'end' not called
        at dalvik.system.CloseGuard.open(CloseGuard.java:237)
        at java.util.zip.Inflater.<init>(Inflater.java:122)
        at net.lingala.zip4j.io.inputstream.InflaterInputStream.<init>(InflaterInputStream.java:20)
        at net.lingala.zip4j.io.inputstream.ZipInputStream.initializeDecompressorForThisEntry(ZipInputStream.java:220)
        at net.lingala.zip4j.io.inputstream.ZipInputStream.initializeEntryInputStream(ZipInputStream.java:201)
        at net.lingala.zip4j.io.inputstream.ZipInputStream.getNextEntry(ZipInputStream.java:106)
        at net.lingala.zip4j.tasks.AbstractExtractFileTask.verifyNextEntry(AbstractExtractFileTask.java:81)
        at net.lingala.zip4j.tasks.AbstractExtractFileTask.extractFile(AbstractExtractFileTask.java:47)
        at net.lingala.zip4j.tasks.ExtractAllFilesTask.executeTask(ExtractAllFilesTask.java:35)
        at net.lingala.zip4j.tasks.ExtractAllFilesTask.executeTask(ExtractAllFilesTask.java:13)
        at net.lingala.zip4j.tasks.AsyncZipTask.performTaskWithErrorHandling(AsyncZipTask.java:42)
        at net.lingala.zip4j.tasks.AsyncZipTask.execute(AsyncZipTask.java:36)
        at net.lingala.zip4j.ZipFile.extractAll(ZipFile.java:437)

Can you share your code and the circumstance in which the application was terminated? Was the application abruptly terminated or did the application run was complete, in other words, was the application force terminated or did zip4j complete its task before the application stopped?

I think I found the issue. I made a fix for it. Fix is not in master but in the inflater-end branch. I just did a SNAPSHOT release with version 2.3.1-SNAPSHOT with the fix. Can you please use this version and see if this fixes the issue? You have to follow the instruction mentioned here to get this snapshot version in your maven.

Good, the error is gone with version 2.3.1-SNAPSHOT. Thanks man!


Because flutter_form_builder 4.2.0 depends on intl ^0.16.1 and no versions of flutter_form_builder match >4.2.0 <5.0.0, flutter_form_builder ^4.2.0 requires intl ^0.16.1.

Flutter 2.0 is out, so this package needs to upgrade its dependencies otherwise flutter_form_builder will not work for Flutter 2.0

Please update the package as soon as possible! 🙏🏻

same problem here. I just upgraded my flutter version and nothing is working. Please could you upgrade this library as soon as possible. Thanks

This version is happening since the firebase upgrade of   firebase_core: ^0.6.0 that happened in January.

Hi! Great stuff BTW! May I know when this package will be upgraded?

It looks like this package is not maintained by this author. We find his work really awesome and would like to maintain it. (@danvick  if you hear us, please let me know if you are interested)
For the moment, I just did a fork which works on flutter 2.0 -> https://github.com/Sportner/flutter_form_builder
This has been done really quickly by updating the dependencies + deleting all the inputs type which depends on library that are not compliant with flutter 2.0
The goal is to rebuild these deleted inputs into external libraries (like it has been done for file/image picker and location input) it would allow us to have a really stable/easy to maintain core library.
If you would like to use it, juste update your pubspec.yaml
dependencies:
  flutter_form_builder:
    git: https://github.com/Sportner/flutter_form_builder.git

Great package, please update soon to fix flutter 2.0 compatibility

I just donated to the developer @danvick with a request to prioritize Flutter 2.0.0 support. I would encourage anyone depending on this package to do the same! https://www.buymeacoffee.com/wb5M9y2Sz/c/1126129

It looks like this package is not maintained by this author. We find his work really awesome and would like to maintain it. (@danvick if you hear us, please let me know if you are interested)
For the moment, I just did a fork which works on flutter 2.0 -> https://github.com/Sportner/flutter_form_builder
This has been done really quickly by updating the dependencies + deleting all the inputs type which depends on library that are not compliant with flutter 2.0
The goal is to rebuild these deleted inputs into external libraries (like it has been done for file/image picker and location input) it would allow us to have a really stable/easy to maintain core library.
If you would like to use it, juste update your pubspec.yaml
dependencies:
  flutter_form_builder:
    git: https://github.com/Sportner/flutter_form_builder.git

I was pretty sure that package would get instant migration to Flutter 2.0 as seeing a lot of contributors on pub.dev page. Probably it's not the case. The package itself is great and would love to see another taking steer if needed.
But it was concerning at the beginning that the package is also dependent on 11 other packages.

Great to see instant migration to flutter 2.0.
Please update the other packages as well "form_builder_image_picker" and "form_builder_file_picker"
Thanks

Hi folks. How about move core FormBuilder & FormBuilderField (Custom Field) to separate package. I like the Flutter Form Builder package. Awesome work!  But more package form items require more time to maintain & support. Every project uses a different set of fields but all use core & custom fields. I do not use any field from this package instead I use my own build with custom fields & core form builder.  I have removed all fields from the package & after 1 hour I am on Flutter2. All work. So separation can be the right way to deliver this awesome work to the community more quickly. :)
For example:
flutter_formbuilder_core
flutter_formbuilder_textfield
flutter_formbuilder_chips
....
It is only a suggestion :)

Great to see instant migration to flutter 2.0.
Please update the other packages as well "form_builder_image_picker" and "form_builder_file_picker"
Thanks

Still an endless hole of dependency problems on 5.0.0-alpha.1 .  Conflicts with the most common packages like firebase_storage, cached_network_image, flutter_keyboard_visibility.  There are just a million nested dependencies in this package.

Because no versions of firebase_core match >1.0.0 <2.0.0 and firebase_core 1.0.0 depends on firebase_core_platform_interface ^4.0.0, firebase_core ^1.0.0 requires firebase_core_platform_interface ^4.0.0. And because no versions of firebase_core_platform_interface match >4.0.0 <5.0.0 and firebase_core_platform_interface 4.0.0 depends on plugin_platform_interface ^2.0.0, firebase_core ^1.0.0 requires plugin_platform_interface ^2.0.0. And because flutter_keyboard_visibility >=4.0.0 <5.0.0-nullsafety.0 depends on flutter_keyboard_visibility_platform_interface ^1.0.1 which depends on plugin_platform_interface ^1.0.3, firebase_core ^1.0.0 is incompatible with flutter_keyboard_visibility >=4.0.0 <5.0.0-nullsafety.0. And because flutter_form_builder >=5.0.0-alpha.1 depends on flutter_typeahead ^2.0.0 which depends on flutter_keyboard_visibility ^4.0.2, firebase_core ^1.0.0 is incompatible with flutter_form_builder >=5.0.0-alpha.1. So, because moj_cuvaj depends on both flutter_form_builder ^5.0.0-alpha.1 and firebase_core ^1.0.0, version solving failed.

@digibum  I'd make these modifications to my code to Firebase work.

dependency_overrides:
intl: ^0.17.0
vin_decoder: ^ 0.2.0-nullsafety
http: ^0.13.0
firebase_messaging_platform_interface: ^2.0.0
plugin_platform_interface: ^2.0.0
signature: ^4.0.0-nullsafety
image: ^3.0.1

Opensource is a blessing and a curse at the same time.

Hello @danvick Thanks for your quick and detailed reply. In my case I can wait for the maintainers of the other 2 repositories to include the null-safe approach.

@danvick thanks for working on this and for your detailed exposition of the situation. Honestly null safety is not a huge priority for me at the moment, so I'm happy to be patient with regards to your core dependencies.
That said, I'm not sure the complexity of this package (and the number of dependencies) is actually working in your favor in terms of long-term maintainability. My perception is, working toward a more decoupled approach could (in time) yield a lower maintenance cost.
For example, if you split out some of the more complex form fields into separate packages, those packages might not need to be updated as frequently depending on the type and scope of changes to the core package. Also, for less commonly-used form fields (for example, color picker or signature pad?) users depending on them could shoulder some of the burden of supporting development and contributing to the maintenance of those packages.
Lastly, if you separated out some of those form fields, it would be easier for users to fork and adopt alternative approaches to those types of fields without introducing conflicts with one another or with the core flutter_form_builder package.
I know for my project we're mostly only going to care about a few core field types, and we wouldn't have a need for the more exotic types. So the presence of those field types only makes the package larger and more complex without adding much benefit for my use case.
Obviously these are just my own thoughts on the matter, and I don't want to pressure you. I was just thinking if you focused on the API interface and the quality of the core form field types -- rather than trying to create and support a large number of field types yourself -- it might be easier on you in the long run.

@digibum I'd make these modifications to my code to Firebase work.

dependency_overrides:
intl: ^0.17.0
vin_decoder: ^ 0.2.0-nullsafety
http: ^0.13.0
firebase_messaging_platform_interface: ^2.0.0
plugin_platform_interface: ^2.0.0
signature: ^4.0.0-nullsafety
image: ^3.0.1

has anyone else had a problem overriding http: ^0.13.0 ?  It's able to resolve dependencies but when running the app it causes:
../flutter/.pub-cache/hosted/pub.dartlang.org/vin_decoder-0.1.3/lib/src/nhtsa_model.dart:89:37: Error: The argument type 'String' can't be assigned to the parameter type 'Uri'.

'Uri' is from 'dart:core'.
final response = await http.get(path);

It seems there is a breaking change going to 0.13.0?  Curious if anyone has come across a solution.

Yeah @danvick  is back! My fork still use the validators package but at least I stripped down all extra dependencies.
Do you want me to make a PR?
I was about to update your image picker at the end of the week cause I need it in my project

I quickly did a null safety update for file picker (I haven't tested it yet, I'll try to take some time during the week-end)
dependencies:
  form_builder_file_picker:
    git: https://github.com/Sportner/form_builder_file_picker.git

@AlBlanc  Thanks Mate!..  But I needed form_builder_image_picker. Do you have null safety version for this one? please post the link.

I coded a simpler file picker to pick a single image. I'll push it during the days.
But it is still not an image picker (it does not offer the possibility to take a picture with the camera).
I am leaving for the weekend, I'll quickly have a look at this next week.
@danvick  takes too much time to reply, I'll publish everything as dart packages.

Image Picker package makes me crazy after every flutter upgrade. So in all my projects, I replaced it with multi_image_picker + form_builder_custom_field. All this under ~ 2 hours of work. The decision to Remove the image_picker from the core package seems to be the right choice :).
multi_image_picker works perfectly. You can set a limit choice to one image & it will work as an image picker but without all this head pain.
Just a suggestion :). No Ads

Image Picker package makes me crazy after every flutter upgrade. So in all my projects, I replaced it with multi_image_picker + form_builder_custom_field. All this under ~ 2 hours of work. The decision to Remove the image_picker from the core package seems to be the right choice :).
multi_image_picker works perfectly. You can set a limit choice to one image & it will work as an image picker but without all this head pain.
Just a suggestion :). No Ads

Could you kindly show how you did that?

Please follow these links to get some ideas on your own widget
Stateful Widget - https://gist.github.com/yarmel/6bbafbf06e6c075340d9600d14d3cf32
Upload Service - https://gist.github.com/yarmel/a5c4bfcbc45542310fc623088bdbbf98
And finally in code just place:
FormBuilderMediaUploader(
name: 'cover',
image: _data['cover'] ?? '',
)
Check how it works in my app ;)
/// AppStore & PlayStore urls
const String kAppStoreUrl = "https://apps.apple.com/us/app/octopoos/id1561954016";
const String kPlayStoreUrl = "https://play.google.com/store/apps/details?id=app.octopoos.coach";

This issue is resolved?

This bug exists and is tracked here #721. There have a discussion about this


[Feature request] Add command to directly prompt for api key save

So something like :Codeium Auth <key>?

So something like :Codeium Auth <key>?

Yes

Thanks, great work.


Generate @NonNull annotation on model class getters representing NON NULL columns

You probably need to run the update-expected.sh script or manually tweak the expected files in the integration tests.

looks good otherwise

squash your commits before merge LGTM though :+1:

Squashed


Talk Waiting Page for Owner/Visitor

Only Owner can see every visitors calling.
Only Owner can accept calling.

It is better to make Random Text Chatting.


Dev

Codecov Report

Merging #176 into master will decrease coverage by 0.91%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #176      +/-   ##
==========================================
- Coverage   84.78%   83.86%   -0.92%
==========================================
  Files          30       30
  Lines        1853     1773      -80
==========================================
- Hits         1571     1487      -84
- Misses        282      286       +4

Impacted Files
Coverage Δ

src/MLJBase.jl
100% <ø> (ø)
:arrow_up:

src/interface/data_utils.jl
95.34% <100%> (-0.11%)
:arrow_down:

src/parameter_inspection.jl
50% <0%> (-50%)
:arrow_down:

src/hyperparam/one_dimensional_ranges.jl
82.5% <0%> (-12.63%)
:arrow_down:

src/measures/measures.jl
89.47% <0%> (-3.51%)
:arrow_down:

src/composition/networks.jl
67.94% <0%> (-1%)
:arrow_down:

src/equality.jl
90% <0%> (-0.91%)
:arrow_down:

src/utilities.jl
74.32% <0%> (-0.68%)
:arrow_down:

src/show.jl
15.38% <0%> (-0.65%)
:arrow_down:

src/distributions.jl
83.83% <0%> (-0.48%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


please check the performance of app

https://github.com/techgebra/polosys-books-mobile/assets/138277245/b3e36cad-219d-4114-ae51-8f73396b07bc
https://github.com/techgebra/polosys-books-mobile/assets/138277245/9390f02b-c9f5-4b4e-9d06-8589e1cec3b1

issue closed


Stable Terminal crashes when cursor reaches console end in a custom script

Windows Terminal version
1.18.2822.0
Windows build number
10.0.22621.2428
Other Software
cls

[Console]::CursorTop = 20

function Show-Menu
{
	[CmdletBinding()]
	param
	(
		[Parameter(Mandatory = $true)]
		[array]
		$Menu,

		[Parameter(Mandatory = $true)]
		[int]
		$Default
	)

	# https://github.com/microsoft/terminal/issues/14992
	[System.Console]::BufferHeight += $Menu.Count
	$minY = [Console]::CursorTop
	$y = [Math]::Max([Math]::Min($Default, $Menu.Count), 0)

	do
	{
		[Console]::CursorTop = $minY
		[Console]::CursorLeft = 0
		$i = 0

		foreach ($item in $Menu)
		{
			if ($i -ne $y)
			{
				Write-Information -MessageData ('  {1}  ' -f ($i+1), $item) -InformationAction Continue
			}
			else
			{
				Write-Information -MessageData ('[ {1} ]' -f ($i+1), $item) -InformationAction Continue
			}

			$i++
		}

		$k = [Console]::ReadKey()
		switch ($k.Key)
		{
			"UpArrow"
			{
				if ($y -gt 0)
				{
					$y--
				}
			}
			"DownArrow"
			{
				if ($y -lt ($Menu.Count - 1))
				{
					$y++
				}
			}
			"Enter"
			{
				return $Menu[$y]
			}
		}
	}
	while ($k.Key -notin ([ConsoleKey]::Escape, [ConsoleKey]::Enter))
}

do
{
	$Choice = Show-Menu -Menu @("Yes", "No", "Warning") -Default 2

	switch ($Choice)
	{
		$Yes
		{
			continue
		}
		$No
		{
			"No"
		}
		$KeyboardArrows {}
	}
}
until ($Choice -ne "Warning")

Steps to reproduce
Run script and choose "Warning" answer as much as you can. At the end Terminal will be crashed when interactive menu reaches the console window bottom.
Expected Behavior
It should continue waiting for a right answer without crashing
Actual Behavior
Terminal is crashed.
Tested with all PowerShell versions. Pure PowerShell without Terminal has no crash. Also a Terminal Preview version has no crash too! Tested on 1.19.2682.0.
Event ID: 1025
Application: pwsh.exe
CoreCLR Version: 7.0.1123.42427
.NET Version: 7.0.11
Description: The application requested process termination through System.Environment.FailFast.
Message: The Win32 internal error "No process is on the other end of the pipe." 0xE9 occurred while getting console output buffer information. Contact Microsoft Customer Support Services.
Description: The process was terminated due to an unhandled exception.System.Management.Automation.Host.HostException: The Win32 internal error "No process is on the other end of the pipe." 0xE9 occurred while getting console output buffer information. Contact Microsoft Customer Support Services.
 ---> System.ComponentModel.Win32Exception (233): No process is on the other end of the pipe.
   --- End of inner exception stack trace ---
   at Microsoft.PowerShell.ConsoleControl.GetConsoleScreenBufferInfo(SafeFileHandle consoleHandle)
   at Microsoft.PowerShell.ConsoleHostRawUserInterface.get_CursorPosition()
   at Microsoft.PowerShell.ConsoleHost.InputLoop.Run(Boolean inputLoopIsNested)
   at Microsoft.PowerShell.ConsoleHost.InputLoop.RunNewInputLoop(ConsoleHost parent, Boolean isNested)
   at Microsoft.PowerShell.ConsoleHost.EnterNestedPrompt()
   at Microsoft.PowerShell.ConsoleHost.DoRunspaceLoop(String initialCommand, Boolean skipProfiles, Collection`1 initialCommandArgs, Boolean staMode, String configurationName, String configurationFilePath)
   at Microsoft.PowerShell.ConsoleHost.Run(CommandLineParameterParser cpp, Boolean isPrestartWarned)
   at Microsoft.PowerShell.ConsoleHost.Start(String bannerText, String helpText, Boolean issProvidedExternally)
   at Microsoft.PowerShell.UnmanagedPSEntry.Start(String[] args, Int32 argc)
Stack:
   at System.Environment.FailFast(System.String, System.Exception)
   at Microsoft.PowerShell.UnmanagedPSEntry.Start(System.String[], Int32)
   at Microsoft.PowerShell.ManagedPSEntry.Main(System.String[])

Event ID: 1000
Faulting application name: OpenConsole.exe, version: 1.18.2310.9002, time stamp: 0x65248ade
Faulting module name: OpenConsole.exe, version: 1.18.2310.9002, time stamp: 0x65248ade
Exception code: 0xc0000094
Fault offset: 0x0000000000002be2
Faulting process id: 0x0x1238
Faulting application start time: 0x0x1DA051E9B66E886
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.18.2822.0_x64__8wekyb3d8bbwe\OpenConsole.exe
Faulting module path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.18.2822.0_x64__8wekyb3d8bbwe\OpenConsole.exe
Report Id: 84ce78c7-193c-4016-9a39-6bdb0231e18f
Faulting package full name: Microsoft.WindowsTerminal_1.18.2822.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

https://github.com/microsoft/terminal/assets/10544660/11aac91e-e0ca-42a6-a687-0b50f6529841

@farag2 Just a quick sanity check, does this work fine in Terminal Preview (v1.19+) for you? Over in #16319, looks like folks can't repro the crash there, and these two look very similar

Hi. I will check today.

Other notes:
This is the _DoLineFeed crash. That's weird.

(sorry for fat-fingering enter)

Yeah, it really works perfectly in 1.19.3172.0. So you 've found the bug cause?

So you 've found the bug cause?

Ahg, well, that's the thing. In 1.19 we like, entirely re-wrote huge swaths of this code. So we're really not sure what caused this, because everything related to it is entirely gone.
But I am gonna dup this to /dup #16199

For Better or for worse, the bug is gone. Good work))


[Bug] Does not support single str for vision or multiple texts input for non-vision

First thank @zhu327 and @ekatiyar for your great works.
I am using this fork of repository and notice that
curl http://localhost:8080/v1/chat/completions \
 -H "Content-Type: application/json" \
 -H "Authorization: Bearer $GOOGLE_API_KEY" \
 -d '{
     "model": "gemini-1.5-vision-latest",
     "messages": [{"role": "user", "content": "Say this is a test."}],
     "temperature": 0.7
 }'

{"code":400,"message":"message.multiContent: json.Unmarshal: json: cannot unmarshal string into Go value of type []openai.ChatMessagePart","type":""}

does not work, because the proxy forces the vision input to have multiple parts. But in practice, both openai and gemini vision models can accept input without images. I also notice that
curl http://localhost:8080/v1/chat/completions \
 -H "Content-Type: application/json" \
 -H "Authorization: Bearer $GOOGLE_API_KEY" \
 -d '{
     "model": "gemini-1.0-pro-latest",
     "messages": [{"role": "user", "content": [
        {"type": "text", "text": "Paraphrase this sentence."},
        {"type": "text", "text": "Say this is a test."}
     ]}],
     "temperature": 0.7
 }'

does not work, because the proxy forces the text input to have single part. But in practice, both openai and gemini models accept multiple text parts.
Since gemini-1.0-pro-vision was already deprecated last month, I don't see why we cannot simply aggregate toStringGenaiContent() and toVisionGenaiContent() to handle all forms of input. I implemented it in my fork and it solves both bugs. It also avoid the extra environment variable to toggle between gemini flash and pro vision.

Nice fix! I don't use the vision model so didn't run into this issue.
My main branch has a bunch of extra stuff, so if you'd be able to rebase your change onto this repo it should be pretty easy to just open up a PR here. One thing to note is that I've modified my model mapping changes in #41, but briefly looking over your changes it shouldn't be too big of an issue

Thanks for your PR.