Can not save files to NAS

I can not save my files to my raspberry pi NAS, using Open Media Vault. It keeps give me a error "You do not have write permissions on specified folder! Please choose a different one!" But I do have permissions, as seen in attached screenshot. Is there something I can do to fix this?

I have it mapped as a network drive. Yes I am on TL 1.4.3.

// Only check local drives
if (driveInfo.DriveType != DriveType.Fixed)
{
    return true;
}

Just copied that out of the source... I am confused.
When do you actually get that message?

When I click download and I change the path. I go to click download it will tell me and turn the box red

But in order to get past the given code the drive must be mounted as "fixed" in your case. How your Windows comes to that conclusion is ariddle for me.  Did you use an UNC path like "\server\folder\subfolder" or do you use some sort of protocol like "smb://ip/folder". Since I see it is a Raspberry PI i assume it is the latter...

Server/folder/subfolder

how did you map that drive... like litteraly how, step by step. When I go into my explorer and right click the network thingy and choose "Map network drive" and I specify a UNC path, TL always detects it as NOT "Fixed"... so I wonder how you are able to map a network drive so it shows up as "Fixed". I am not able to do that, so why are you? 😈 I want to be as cool as you 😡

Okay. So my Rpi is running Open Media Vault, which is the software for my nas, All i do is go
This PC -> Map Network Drive -> Browse -> Click on my device (RASPBERRYPI) -> Click on the folder I want to map (In my case it is Files) -> now it looks like the first pic -> Then i just click finish and it shows like the second pic. Hope that was easy enough to follow.

I just realized something in your screenshot (with the red textbox).
You are trying to download directly to the server with "\RASPBERRYPI\Files......"
Well... that's not gonna work... TL does not support direct UNC paths. That's not as trivial as you might think it is.
BUT: You mapped the UNC to drive A:
Now you can setup the default path in TL to "A:\Twitch VOD" then TL recognizes A:\ as a network drive, skips the permission check and you're good to go.

And another hint: If you download directly to a network drive, be aware that ffmpeg.exe runs locally and processes the files on the network drive. In case of a VOD with 11000 parts for example, all those parts travel over the LAN and the result mp4 back again.
I my opinion downloading and processing locally and then copying the result to the NAS is the better and faster way.

I tired mapping directly to Z: (Which is what the letter is not A:) and it wont let me click, see screenshots. The only place I can find the Z: is in Quick Access or in Network, which is what I was trying before.

That maybe explains why it does not work from the beginning. If you map \raspberrypi\files to Z: it should normally show up in the explorer as a network drive with the drive letter. If that does not work you have to find out why that is.

Thats what I assumed. I will try to figure it out.

Closing this because not a TL problem


CLA: Both links go to 404

Both links in the CLA go to 404 :
https://github.com/SAPDocuments/Tutorial-Contribution/blob/master/contributing.md
https://github.com/SAPDocuments/Tutorial-Contribution/blob/master/contributing.md

Thank you for your feddback. You have posted one link twice. The link is working.

@MichaelCzcz The link works for you because you have permission. The links should be changed and I will handle.

We have updated the CLA


Enable flexible definition of asset name schemes from user

I want to find a way to specify asset name schemes in onyo in a way that has at least be able to do:

be used by onyo new to read a new asset file name
onyo set, because it should not just be able to set contents, but also values for the name. Currently, there is no way to do this flexibly, except when assuming the name scheme will always be <type>_<make>_<model>.<serial> with the same reserved characters.
onyo mv --rename, because currently this does not check names at all.
onyo fsck should also check the name schemes
define reserved characters for each asset template/type and for each field separately
be compatible with future implementation of faux serials

My thoughts so far were about adding a file .onyo/validation/names.yaml which could be easily used by onyo new and would be structured very similarly to the existing validation.yaml file.
default:
- name: <type>_<make>_<model>.<serial>
  - type:
    - reserved: _, .
  - make:
    - reserved: _, .
  - model:
    - reserved: _, .
<template name, e.g. laptop>:
  - name: <name_scheme>

But the problem is, I do not see how to use it with mv, fsck, set etc. All the ideas I had so far are based on "saving additional information" for each file:

either in the asset
in the .anchor
or even in an additional file

But this would directly break when changing a template name and force the user to update the names.yaml file everytime a new template would be added.
One could also track the templates and validation files with git and try to follow the information trough commits, but when then  the history would not be complete (manual rebase?), this might break the functionality again.
So long story short, I currently am still struggling to find a good mechanism to specify asset file names, that goes farther than asset creation with onyo new.

Edit: I added to the above list that it should be compatible with "faux serials" that are mentioned in the readme.
This is not implemented right now, but whatever solution for asset names and their validation is found, should be compatible with the faux serials.

This comment from #15 seems now more relevant in this context:
When running onyo mv <path/filename> <path/new_filename> without the rename flag, it mentions the flag as a forced solution. In the future, when onyo set is implemented, it should also mention onyo set key=value as a possibility.

I'd like to provide a different angle on this.
What do we technically need from asset names?

They are identifiers and need to be unique across the repo.
IF they contain pseudo-keys, we need to be able to extract their values from the name and set new values.

At the moment, I think that everything else is "made up" by having mentally settled on a combination of pseudo-keys to be a given. But that's not necessary. If a user comes up with unique identifiers and wants to track things like serial numbers, model names etc. in the YAML instead - that's perfectly fine and does not have impede any inventory functionality as far as I can see.
Therefore, I'd suggest to not enforce a particular scheme. Of course, by default we keep what we have, but what that really is about is pseudo keys, not naming scheme. Hence, make pseudo keys configurable by providing a regex that tells onyo how to extract a particular pseudo key (substring) from an asset name. If we can find the match, we can also replace the matching substring with a new value - no need to have an explicit definition for setting, I think. .onyo/config can then come with a pseudo key section that is preset to define our default ones, but can be changed to anything. This seems a lot easier to implement and define than any sort of scheme definition, while being very flexible.
For validation, we'd check whether names are unique and whether we can match the configured expressions.
WDYT, @TobiasKadelka, @nhjjr, @aghisla, @aqw?

We had similar ideas in the past, but it was never fully sure how to do this, and some questions always remained, for example "is there just one or even many name schemes per repository?".

if just one scheme:

how to handle it when the scheme is changed later? Because onyo set --rename does change the keys separately, not the whole names.

if multiple ones:

how to adress them, when they are overlapping? (e.g. "type_make_model.serial" vs "day_week_month.year" ) I would guess we would always use the first scheme that fits?

how to verify/check the name-scheme itself?

do we verify this file always when running onyo? (now that I think about this, we should maybe already do that, currently we just error when a tool specified under onyo "history" does error, but the name scheme is important for many more commands)
If this is broken, does onyo stop working? What if "path" is part of it, which is something that the output of onyo get already provides as a column. There might be similar bugs.

In general I do agree and would love to have this implemented, I just have some open questions beforehand :)
But I think the general idea of flexible name schemes (e.g. user-definable pseudo-keys) was most of the time in our mind, and should be not that hard to implement.
The biggest TODO would probably be adding a lot of tests for this that verify that none of the existing commands have something about this hardcoded, and every command actually uses the name from the config correctly :)

Also, this is then connected with https://github.com/psyinfra/onyo/issues/306 about more .onyo/ support.

@TobiasKadelka

"is there just one or even many name schemes per repository?"

If we go with the suggestion above, then there's no scheme. ;-)
There are pseudo key definitions and obviously you can't configure the same pseudo key several times.

how to adress them, when they are overlapping? (e.g. "type_make_model.serial" vs "day_week_month.year" ) I would guess we would always use the first scheme that fits?

Both and I think that's the user's problem.
Your example would mean to specify a pseudo key type and a pseudo key day with the same regex. We'd therefore use that regex whenever the respective key accessed. If a user sets it up that way - not onyo's problem, I think. (There could be valid cases for that)

do we verify this file always when running onyo? (now that I think about this, we should maybe already do that, currently we just error when a tool specified under onyo "history" does error, but the name scheme is important for many more commands)

I don't think there's anything to verify. It's a regex. We apply it. If that fails, we fail.

Therefore, I'd suggest to not enforce a particular scheme. Of course, by default we keep what we have, but what that really is about is pseudo keys, not naming scheme.

I agree and propose (a possibly unpopular opinion) of taking this a step further. Scrap any internal reliance on pseudo-keys.
What do pseudo-keys currently do for us?

Provide a (default) naming scheme for asset files
Enable users to easily explore the inventory using a file browser

This leads to a set of problems for us:

We don't want users to modify files through the file browser, so why do we make it seem as if that is encouraged?
We have to provide workarounds for certain functions, e.g.:
* onyo mv cannot rename files because it may break pseudo-keys
* onyo get must search separately through file names and file contents
Workarounds will become more entrenched and thus difficult to modify (unless preempted by decoupling pseudo-key handling)
Pseudo-keys are not straight-forward to new users who explore onyo through a file browser

This can be resolved by not giving a damn about file names, barring the extension which we'll use to ensure file uniqueness through a randomly generated serial. Two suggestions to deal with this:

When a new file is made, force the user to give that file a name
When a new file is made, give it a name based on required keys from the asset file
When a new file is made, give it a random name if no name is specified

Even if the file names become a randomized mess, users should be able to use onyo get and onyo ls to obtain identifiable assets. For example, an onyo ls -k type make model could return something like:
total 5 (1 folder, 4 assets)
/folder
a89d-89bd-kk87.83hic    monitor  dell    SE2722HX
wintermute.9lopi        laptop   apple   macbookpro
bens-laptop.3pop5       laptop   lenovo  thinkpad
laptop_hp_zbook.876ab   laptop   hp      zbook

This way we never rely on filename to be a reflection of any particular value, allowing it to literally be anything. This gets rid of pseudo-keys and all the associated problems, but comes at the cost of developing more options for commands like onyo ls. This could even be extended with naming helpers, for example a function that can rename all files in a folder (or the entire repository) using asset keys (e.g., onyo rename --all -k type make model --sep "_" to rename all assets in a repository using their values of type, make, and model with an _ as a separator).
Thoughts? Or did I go too far?

@nhjjr Controversial is good. I appreciate you throwing a firework into the middle of everything. It's good for stimulation. ;-)
Reading through this, it feels like it would set us on the path of implementing an RDBMS on-top of git. At that point, I think we should simply use sqlite and dump it to git for syncing.
I'm pretty attached to the obviousness (aka: intuitiveness) of using file names that are meaningful to a user. I /do/ agree that some of the lengths onyo goes to in order to keep that sane is a bit suboptimal at times. I think your criticisms (especially 2 and 3) are well founded.
I see onyo from a very unix-based view: everything is a file. Get to use all the unix tools on them. So I will keep coming back to that.
One point: the term "pseudo keys" (as it's currently understand) is a category that's larger than just the keys in the filenames. It also includes things like path  and possibly others (last commit time, etc).
Two repond to two points of yours:

onyo mv cannot rename files because it may break pseudo-keys

IMO, this is because the file contains key-data. And for that reason set is more appropriate.

onyo get must search separately through file names and file contents

That is annoying/unfortunate. This was decided so that the information is only ever in one place. Two sources don't have a tie breaker.
However, I could see taking a view that all keys must exist inside the file, and that the filename is merely an outward representation of a selection of keys that exist in the file. If there's a mismatch, the filename is considered wrong.
That way, the filename scheme could be updated/changed by the user, without there being any loss of data.  But no file contents would change. And it should simplify querying data.
Anyway. Those are my late night thoughts. :-)

Verdict from roadmap meeting for 1.0:

Make actual asset name scheme configurable by defining an f-string allowing to fill in any key from asset content like "{key1}_{key2}.{key3}"
Support pseudo-key configuration like suggested above (regex for matching substring of filename), but read-only. Have reserved (not configurable) pseudo keys like path, filename and a couple of dates (last modified, created, etc.). TODO: Make a list of names+definitions for those.


Replace tensorflow::Status::OK() with tensorflow::OkStatus().

Description
The OkStatus() symbols was added on May 2022, between the 2.9 and 2.10 releases of Tensorflow. The Tensorflow v2.10 release happened on September 6, 2022.
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tsl/platform/status.h#L102
Type of change
Code cleanup.
Checklist:

[ x] I've properly formatted my code according to the guidelines

[x] By running Black + Flake8
[x] By running pre-commit hooks

[] This PR addresses an already submitted issue for TensorFlow Addons
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
python3 -m pytest tensorflow_addons.
Failures are the same.

@manivaradarajan Thanky you.
@seanpmorgan We support custom-ops only for the last stable TF release right?

@manivaradarajan Thanky you.
@seanpmorgan We support custom-ops only for the last TF stable release right?

Correct


how to set style (margin-top:10px) on a paragraph?

I am going to control the paragraph spacing before and after with a value that user input.
I known that setInlineStyleOverride can set inlineStyle on a span tags, but how to deal with a unstyled block?
I have viewed the Block Styling document, but what if the value(10px) is by user input?
Maybe passing the margin to an entitiy is a workaround, what else can I do?

See custom blocks: https://draftjs.org/docs/advanced-topics-custom-block-render-map.html#content.

I struggled with the options in that link as 'unstyled' gets overridden easily.
Another solution (not perfect, but may fit your needs) is:
Add this to your CSS is also an option:
.public-DraftStyleDefault-block {
  margin-bottom: 1em;
}


Update HC4 documentation for consistency

Codecov Report

Merging #1934 (be1225d) into master (71c1200) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1934   +/-   ##
=========================================
  Coverage     58.70%   58.70%
  Complexity     1079     1079
=========================================
  Files           180      180
  Lines         10431    10431
  Branches       1430     1430
=========================================
  Hits           6123     6123
  Misses         3787     3787
  Partials        521      521

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 71c1200...be1225d. Read the comment docs.


Create app.srspass.com.md

Pingdom Measurement Link

The site is acceptable to add in terms of it's size etc, but since the website is not usable on mobile I can't add this to the listings.
Reference to usability issue
On mobile screen (iPhone) I can't clear the modal or even scroll the page - essentially stuck.

If these issues are fixed, feel free to reapply this site again :+1:

Thanks for feedback, should be resolved now.  As my luck would have it I already had a commit lined up yesterday for this.
Please make sure this commit is merged, as size changed (not due to CSS change, but an extra unused script was unlodaded) https://github.com/D-Nice/1mb-club/commit/6bee90969be2bdf9012a7dc6bb806eb84d0a8e18


add dayOfWeek(Date) method and add auto-install button

added a method for obtaining day of week as a string.

@jholmes612   can you review this and merge if acceptable?

also added autodeploy button to README

Looks good. I'd change the test name to "testDayOfWeek()" to be consistent with others.

sounds good, thanks!

I will change it now

@jholmes612 fixed the test name


NATIVE-306: News stream feature

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.

Please also check for other occurrences of my requested changes and adjust if necessary. Especially for

updated timestamps in snapshots
usage of colors
usage of !important
unnecessary checks for null/undefined

I think a bit of space beneath the list would be good:

Also, if you navigate from the tünews list to another page (e.g. tünews detail) and then navigate back, everything is fetched again. I think we should not fetch anything if there already is data in the state.

Where do you test local news, disabled local news and disabled tünews? (Which instance, I'm not sure where we have according settings).

I think a bit of space beneath the list would be good:

Fixed. I added 40px margin to match the top space between the list and the tabs

Where do you test local news, disabled local news and disabled tünews? (Which instance, I'm not sure where we have according settings).

This city testumgebung has both localNews and tunews enabled. It used to have some local news test data but it's deleted now (maybe yesturday) check this link
This city konstanz has only localNews enabled
There is not city where tunews only enabled. Maybe you can add one and test that

Also, if you navigate from the tünews list to another page (e.g. tünews detail) and then navigate back, everything is fetched again. I think we should not fetch anything if there already is data in the state.

Should be fixed now

Where do you test local news, disabled local news and disabled tünews? (Which instance, I'm not sure where we have according settings).

This city testumgebung has both localNews and tunews enabled. It used to have some local news test data but it's deleted now (maybe yesturday) check this link
This city konstanz has only localNews enabled
There is not city where tunews only enabled. Maybe you can add one and test that

Thanks!

@amusameh How did you solve the issue with the timezones in the snapshot tests?

@klinzo

@amusameh How did you solve the issue with the timezones in the snapshot tests?
I edit the jest global setup to use Germany time

new file => src/global-setup.js

// @flow

module.exports = async () => {
  process.env.TZ = 'Europe/Berlin'
}

Added this line to the end of  jest.config.json
"globalSetup": "<rootDir>/global-setup.js"

Thanks!

TODO before merging: Pull locales via subtree
@amusameh should I do that?

TODO before merging: Pull locales via subtree
@amusameh should I do that?

Sorry i just saw this. Apparently @maxammann has done that already, thanks : )


New frontend build process (Gulp integration)

Add a Gulp (http://gulpjs.com/) based build process for all frontend ressources.
Combine current js, css build steps into a  single build process

Integration of the requirejs build process (http://requirejs.org/docs/optimization.html)
Integration of the Compass build process (http://compass-style.org/help/documentation/command-line/)
rename build folder
optimize images

The long-term goal is to remove compass (vendor prefixes + macros) and switch to (https://www.npmjs.com/package/gulp-sass) .

This is required for #17


WebView has been removed from React Native

I have been using react-native-azure-ad-2 for a while without a problem but for some time a crash appears when using the library. I am attaching a screenshot of the issue. You should import WebView from different place now.

I am facing the same issue. Can't get around it even after installing npm i react-native-webview.
Could someone help how to resolve this?

@raviteja1982
npm install --save react-native-webview
Go to AzureLoginView.js and add import { WebView } from 'react-native-webview';
This should do the trick.

I ran into another issue but that's totally unrelated to this. Your suggestion worked. Thanks @Nafets94


build: fix failing nightly snapshot tests

Apparently Yarn install mismatching dependencies of Angular Github snapshot packages. Meaning that we need to flatten out all dependencies for @angular/* packages.
This can be achieved by using the Yarn's resolutions feature.

I've tested the snapshot job in this PR and it passed as expected. See: https://circleci.com/gh/angular/material2/24035

Note: The size change seems unrelated. This PR is rebased on top of master. Maybe the size baseline hasn't been saved in master properly.


Paste context menu not dismissed

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
10.4.4
Steps to reproduce
Use the example app and run on mobile.
Long press to select a word, get popup menu and select Copy. Popup menu is dismissed.
Long press at a different place, get popup menu and select Paste. Text is pasted but the popup menu is NOT dismissed.
This issue was fixed in PR2116
These changes were removed in PR2128 to address the problem with the magnifier.
Expected results
Popup menu should be dismissed after paste is selected.
Actual results
Popup menu is still present after paste is selected.
Code sample
Code sample
[Paste your code here]

Additional Context

Screenshots / Video demonstration
[Upload media here]

Logs
[Paste your logs here]

@demoYang and @CatHood0 can you take a look at this problem?
These changes appear to be mutually exclusive.
I regret that I do not understand mobile enough to be able to figure out what is going on.

I know what part is causing the error. I'll test my fix soon (i don't know why was removed that part since fix this issue)

@CatHood0 your changes fixed the paste problem.
I assume @demoYang made the changes because it was needed to fix the problem with the magnifier.
However, when I re-introduced your changes, it caused even more problems.
So, I think there is some deeper conflict between paste popup and magnifier.

However, when I re-introduced your changes, it caused even more problems.

I don't expect that. It's weird. You have a point, it could be something more deeper than we expect.

Yeah, the magnifier works weird

Does this issue still persists? If not we need to close it.

@AtlasAutocode
Is this still an issue?

Yes, still exists.
I haven't had time to look into it since I almost never use mobile and know nothing about how such popups are managed.
It's an obvious bug but not critical. Feel free to close this issue.

Will keep it open, since we don't add new features anymore, we should have more time but for now, we're focusing on other issues.


Get, Add and Delete Books | API

Make use of the Bookstore API as backend.
Use axios with createAsyncThunk to make HTTP requests to the backend
Fetch books from the Bookstore API after the component has rendered
Refactor add book and remove book features to persist changes in the server

Thanks @Efiamotu-1
All improvements have been implemented

Add license file
Refresh interface / book list when submitting and deleting.


Failed to obtain access token (error after redirect from Google)

Steps to reproduce.

npm install
create .env file with credentials
npm start

GET /login 304 3.999 ms - -
GET /css/base.css 304 0.495 ms - -
GET /css/index.css 304 0.805 ms - -
GET /css/login.css 304 0.796 ms - -
GET /login/federated/google 302 8.528 ms - 0
GET /oauth2/redirect/google?state=sIZwFahLw%2B3Ovo8bz0Gdpy5C&code=4%2F0AX4XfWgW7XKwwtMnG52Ax4_UsaSUMMyuX-YmggpmT-ua7VEZBFyHo853BfF_i3gby7OGfA&scope=profile+openid+https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fuserinfo.profile&authuser=0&prompt=consent 500 310.328 ms - 827
node:_http_outgoing:576
    throw new ERR_HTTP_HEADERS_SENT('set');
    ^

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at new NodeError (node:internal/errors:371:5)
    at ServerResponse.setHeader (node:_http_outgoing:576:11)
    at ServerResponse.header (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/express/lib/response.js:767:10)
    at ServerResponse.location (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/express/lib/response.js:884:15)
    at ServerResponse.redirect (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/express/lib/response.js:922:18)
    at complete (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/passport/lib/middleware/authenticate.js:266:26)
    at /Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/passport/lib/middleware/authenticate.js:278:15
    at pass (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/passport/lib/authenticator.js:428:14)
    at Authenticator.transformAuthInfo (/Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/passport/lib/authenticator.js:450:5)
    at /Users/SagarKapasi099/Developer/Mayur/todos-express-google/node_modules/passport/lib/middleware/authenticate.js:275:22 {
  code: 'ERR_HTTP_HEADERS_SENT'
}

I am unable to reproduce this.  Could you provide more information, such as OS and node version?

i was using node-v16.14.2.pkg
downgraded to node-v16.0.0.pkg
Working Fine Now. Thanks for your reply.

Can you describe what fixed the issue, in case other people land here searching for a solution?

Thanks for the info.  I'm unable to reproduce on node v16.14.2, as well.  If you find out further details as to what was causing the issue, please post that information here for others who might be searching for a solution.  Thanks.


How to enable bias tee for rtl dongle?

rtl_biast exists in container, but no options to enable bias tee at startup. Or I'm not found it...

We haven't added it in to the container; tbh I had even forgotten we technically support SDRs directly in this container. The typical SDR-E container deployment uses ultrafeeder for 1090 and dump978 for UAT, and both of those support bias t directly. It wouldn't be a big deal to support but for now, it doesn't.

https://sdr-enthusiasts.gitbook.io/ads-b/


BRS-974: switch oidc to use loginproxy instead for bc parks admin portal

Jira Ticket:
BRS-974
Jira Ticket URL:
https://bcparksdigital.atlassian.net/browse/BRS-974
Description:
This switches the default local/dev setup to use the dev instance for the new KC Realm.
NB: There are a few ENV vars that will need to be updated after this is merged, prior to deployment into dev/running locally: STRAPI_SSO_AUTH_URL and STRAPI_SSO_REALM.  For environments in openshift, the auth url: REACT_APP_KEYCLOAK_AUTH_URL and REACT_APP_KEYCLOAK_REALM need changing from dev.oidc => dev.loginproxy (and similarly for test/prod). The realm in every environment is simply bcparks-service-transformation.

There are 2 additional settings in /src/admin/.env.example that also need to be updated

There are 2 additional settings in /src/admin/.env.example that also need to be updated. But they are currently incorrect, so we can update them later. Devs working locally on this project normally use the dev keycloak instance, not a local docker instance as indicated by the example file.

I've pushed an update so the next dev doesn't have to worry about this. :)

Thanks Mark.  We'll merge this first thing in January.  We won't want to risk breaking anything right before the holidays.


Opflex member can get stuck in PENDING if operations time-out

Issues:
Fixes #589
Problem:
If a port member creation fails after a timeout the loadbalancer
gets put in ‘ERROR’ state, but the member is left in ‘PENDING_CREATE’.
The loadbalancer should actually be left alone and the member set to
‘ERROR’ provisioning status.
Analysis:
Fix the agent_manager handling of pending services to update the
status of the service in the Neutron.  Properly handle exceptions
of objects that are deleted when exceptions occur in the network
setup.
Tests:
test/functional/neutronless/loadbalancer/
test/functional/neutronless/disconnected/
test/functional/neutronless/member/
Manual testing consisted of:
Create loadbalancer service with all objects: listener, pool, members
Create member with and without existing port.
Create member on opflex, add vlan child segment manually and verify
that the member is connected.
Create loadbalancer on opflex network, add vlan child segment
manuall and verify that the virtual get's created.
Create loadbalancer on opflex network, don't add segment and check
that the loadbalacner goes to 'ERROR' state and that it can be
deleted.
Create a member on the opflex network, don't add segment and check
that the member goes to 'ERROR' state after timeout period and verify
it can be deleted.

Added the changes and retested.  Please squash before merging.


Add ESP32 to supported devices with basic type hints

Relates to #44
Changes
Made ESP32 a supported device, appearing as a device in the MicroPython settings list with auto-detect support.

Added Esp32DeviceProvider class, largely a copy of the Esp8266 one

Tested manually using ./gradlew clean runIde
Confirmed auto-detect is working, at least for my ESP32-WROOM-32 MCU

Add /typehints/esp32/ dir, initially populated with the same files as ESP8266 - to be expanded in later PRs

Out of Scope (but happy to discuss if you think it should be in this PR :smile:)

expanding ESP32 typehints

I think this will require a change to how typehints are aggregated, because some devices will need to "override" common library stuff with extra methods. I have some thoughts on how to do this that could be discussed separately.

CLA
I've signed it.

expanding ESP32 typehints
I think this will require a change to how typehints are aggregated, because some devices will need to "override" common library stuff with extra methods.

To expand on this, one example would be the machine.ADC class, which has a simple generic interface for any board (docs) but has more methods available specifically on the ESP32 (docs).
This presents a problem: if we have a generic /micropython/machine::ADC and a more specific /esp32/machine::ADC we have a name clash... from testing, it seems that the type machinery picks the micropython module's ADC implementation and we get red squiggles trying to use the ESP32-specific stuff (`cannot find reference 'xyz' in 'machine.pyi').
There's a similar issue in the network module, where Micropython provides a universal network package containing an abstract NIC class and some constants, and then various board-specific implementations are available. How do we merge the universal network and board-specific network modules?
I was thinking that board-specific type hint directories could maybe have an __init__.py which could choose to import members from the generic micropython directory, or not, and modify with its own. Will need to test if this is actually feasible.

@Gor-Ren thank you for taking this on. I would like to test  your added support for ESP32 but I am running into issue.
I have cloned your https://github.com/Gor-Ren/intellij-micropython/tree/support-esp32 branch opened the project in IntelliJ IDEA Community2020.2 Beta and ran runIde from Gradel. It builds the plugin and launched PyCharm 2020.1 with the micropython plugin rebuilt in it. When I go to Settings -> Languages & Fameworks -> MicroPython I do not see ESP32 under the Device Type dropdown. Am I doing something wrong? Thank you.

@jsonpoindexter hrm, I can't seem to reproduce

starting from IntelliJ 2020.2 with the latest MicroPython 1.1.1 and the support-esp32 branch
(venv) gor@gor-desktop:~/dev/intellij-micropython$ ./gradlew clean runIde
PyCharm dev window opens
File -> Settings -> Languages & Frameworks -> MicroPython -> "Device type" dropdown -> ESP32 is in the list

maybe make sure you are running ./gradlew clean runIde with the clean - without it, it might be loading a previously deployed version I guess.
As a sanity check you could try loading the SNAPSHOT build into your IDE:

(venv) gor@gor-desktop:~/dev/intellij-micropython$ ./gradlew clean buildPlugin
you will now have a dev plugin ready at .../intellij-micropython/build/distributions/intellij-micropython-SNAPSHOT.zip
File -> Settings -> Plugins -> find "MicroPython"
click settings cog for MicroPython at top -> "Install plugin from disk" -> select intellij-micropython-SNAPSHOT.zip you built -> Apply -> Restart IDE
on restart you should see MicroPython is on version "SNAPSHOT" and have access to the ESP32 in the dropdown

Question on this concerning the stub files. Why can't you change the library root to a directory specific to a type of board and that directory would contain the stub files needed?

Question on this concerning the stub files. Why can't you change the library root to a directory specific to a type of board and that directory would contain the stub files needed?

Many of the stubs describe common code to all boards. Therefore if possible it's desirable to consolidate it in one place, not to duplicate them across every board, which would be an onerous maintenance burden.

I can understand the duplicate code thing. I know it would not be ideal but it is a solution.
Another question..
Does the order in which the library directories being added have anything to do with what stub file gets used first?
T do not know how much manipulation is allowed to be done with PyCharm I am thinking that if you can make the directory for the micropython stubs as a sub directory of a boards stub files then you would be able to add the board stub file folder as the library root and your problem would be solved. The directory for the micropython stub files being a sub directory of the board would only exist in PyCharm and not on the filesystem. Tho you could use symbolic links to accomplish the same thing as well.
I am going to run some tests to try and manipulate the library root. maybe it can be made to behave the way that is needed.

Stub files are there only for the purposes of having an IDE work properly. I do not think they are there for any other purpose.
I am the administrator of a project called EventGhost and this application is written mostly in Python. It is fairly large at around 200,000 lines of code. Some of the modules in it need to be loaded in a very specific order and we also wanted to keep memory management and the resources under control. We do this by only importing a modules when it is actually needed. This is done by wrapping a class around the main module and using __getattr__. The problem with this is an IDE is not able to offer code completion and type hints for those moduels because there is no hard coded import of the modules.
This is what I did to make everything work correctly.
import os

if 'THIS_CODE_NEVER_REALLY_RUNS' in os.environ:
    from Classes import SomeModule as _SomeModule
    SomeModule = _SomeModule

Because PyCharm thinks that the code has a possibility of running it then scans the modules and allows all of the goodies in an IDE to work properly. This kind of manipulation can also be done as well in a pyi file. You would need to add in the imports of the board specific pyi files at the end the question now becomes how can this be done without the user being able to see the board specific files in the library root.


Stuck in Processing in WooCommerce, but CoinBase and Logs shows COMPLETED

Logs as shown below, extracted from uploads/wc-logs/coinbase-xxxx.log. I checked CoinBase, status is COMPLETED, but WooCommerec Orders do not get updated automatically. This is my first transaction with CoinBase, I didn't add any [Checkouts], don't think that is needed right?

WordPress Version: 5.7.2
WooCoomerce Version: 5.4.1
CoinBase Version: 1.1.4

Temporary Solution: I had to manually check CoinBase, then change the status to 'Completed' for my customers.
2021-06-24T17:54:30+00:00 INFO Webhook received event: Array
(
    [attempt_number] => 1
    [event] => Array
        (
            [api_version] => 2018-03-22
            [created_at] => 2021-06-24T17:54:29Z
            [data] => Array
                (
                    [id] => 1e47f73a-88b0-48db-a656-acfa79072c45
                    [code] => D546AV9X
                    [name] => Website Name
                    [pricing] => Array
                        (
                            [usdc] => Array
                                (
                                    [amount] => 15.000000
                                    [currency] => USDC
                                )

                            [local] => Array
                                (
                                    [amount] => 15.00
                                    [currency] => USD
                                )

                            [bitcoin] => Array
                                (
                                    [amount] => 0.00043072
                                    [currency] => BTC
                                )

                            [ethereum] => Array
                                (
                                    [amount] => 0.007426000
                                    [currency] => ETH
                                )

                            [litecoin] => Array
                                (
                                    [amount] => 0.11006751
                                    [currency] => LTC
                                )

                        )

                    [logo_url] => https://res.cloudinary.com/commerce/image/upload/v1624548069/image.jpg
                    [metadata] => Array
                        (
                            [source] => woocommerce
                            [order_id] => 400
                            [order_key] => wc_order_f2t3lBd711t1a
                        )

                    [payments] => Array
                        (
                            [0] => Array
                                (
                                    [block] => Array
                                        (
                                            [hash] => 223944369a74fbd4388c0b48fef0030d76c4b9420e395ed2022c692101753b7a
                                            [height] => 2075275
                                            [confirmations] => 1
                                            [confirmations_required] => 2
                                        )

                                    [value] => Array
                                        (
                                            [local] => Array
                                                (
                                                    [amount] => 15.00
                                                    [currency] => USD
                                                )

                                            [crypto] => Array
                                                (
                                                    [amount] => 0.11006751
                                                    [currency] => LTC
                                                )

                                        )

                                    [status] => CONFIRMED
                                    [network] => litecoin
                                    [detected_at] => 2021-06-24T17:49:14Z
                                    [transaction_id] => e4d30b09d54d2a8af46e2e03b3c0d97c7f08ac08bae3358d10ae7ff12098b8ee
                                )

                        )

                    [resource] => charge
                    [timeline] => Array
                        (
                            [0] => Array
                                (
                                    [time] => 2021-06-24T17:43:43Z
                                    [status] => NEW
                                )

                            [1] => Array
                                (
                                    [time] => 2021-06-24T17:49:15Z
                                    [status] => PENDING
                                    [payment] => Array
                                        (
                                            [value] => Array
                                                (
                                                    [amount] => 0.11006751
                                                    [currency] => LTC
                                                )

                                            [network] => litecoin
                                            [transaction_id] => e4d30b09d54d2a8af46e2e03b3c0d97c7f08ac08bae3358d10ae7ff12098b8ee
                                        )

                                )

                            [2] => Array
                                (
                                    [time] => 2021-06-24T17:54:29Z
                                    [status] => COMPLETED
                                    [payment] => Array
                                        (
                                            [value] => Array
                                                (
                                                    [amount] => 0.11006751
                                                    [currency] => LTC
                                                )

                                            [network] => litecoin
                                            [transaction_id] => e4d30b09d54d2a8af46e2e03b3c0d97c7f08ac08bae3358d10ae7ff12098b8ee
                                        )

                                )

                        )

                    [addresses] => Array
                        (
                            [usdc] => 0x2e66f71158d2cbeb48a15dfa3279947161aef15a
                            [bitcoin] => 3GomyAfHb5HgAyXegqhvG3kc5LpsaVFg5S
                            [ethereum] => 0x2e66f71158d2cbeb48a15dfa3279947161aef15a
                            [litecoin] => MFxPBPZFyoXJFPfxcsCuthBPCdRmN4x54L
                        )

                    [cancel_url] => https://www.mywebsite.com/cart/?cancel_order=true&order=wc_order_f2t3lBd711t1a&order_id=400&redirect&_wpnonce=469a9d0b24
                    [created_at] => 2021-06-24T17:43:43Z
                    [expires_at] => 2021-06-24T18:43:43Z
                    [hosted_url] => https://commerce.coinbase.com/charges/D546AV9X
                    [description] => 1 x Testing
                    [confirmed_at] => 2021-06-24T17:54:29Z
                    [pricing_type] => fixed_price
                    [redirect_url] => https://www.mywebsite.com/checkout/order-received/400/?key=wc_order_f2t3lBd711t1a
                    [support_email] =><EMAIL_ADDRESS>                    [brand_logo_url] => https://res.cloudinary.com/commerce/image/upload/v1624548069/image.jpg
                    [exchange_rates] => Array
                        (
                            [BTC-USD] => 34825.1
                            [ETH-USD] => 2019.795
                            [LTC-USD] => 136.28
                            [USDC-USD] => 1.0
                        )

                    [payment_threshold] => Array
                        (
                            [overpayment_absolute_threshold] => Array
                                (
                                    [amount] => 5.00
                                    [currency] => USD
                                )

                            [overpayment_relative_threshold] => 0.005
                            [underpayment_absolute_threshold] => Array
                                (
                                    [amount] => 5.00
                                    [currency] => USD
                                )

                            [underpayment_relative_threshold] => 0.005
                        )

                    [third_party_provider] => woocommerce
                )

            [id] => 50d9f7fe-ef92-48bc-a495-26dd99e14f07
            [resource] => event
            [type] => charge:confirmed
        )

    [id] => ad1fbd8d-2914-4052-bb6b-c5176ee75c79
    [scheduled_for] => 2021-06-24T17:54:29Z
)

Same issue, also noticed in the order notes this one:

Guys from Coinbase, looks like in this event status should be changed from Processing to Completed, but you are doing vice versa by mistake


Geolocation App - round 2

App demonstrating geolocation capabilities with changes requested from closed PR #601
JIRA ticket SDK-575

Biggest question is why are we not using ol.Geolocation here

Coverage remained the same at 94.7% when pulling bac63b9e317991848a08b14577c98185e0bd65e4 on CarlyLanglois:feature/geolocation-app into e219f17f9290c758822b51a5f51bc4b900e8d0ab on boundlessgeo:master.

FYI this ditches the refreshInterval prop since ol.geolocation continuously tracks the position

Coverage increased (+0.8%) to 95.537% when pulling 4c97f8797d351fa7ae296daa7bac4ca551de1b2e on CarlyLanglois:feature/geolocation-app into e219f17f9290c758822b51a5f51bc4b900e8d0ab on boundlessgeo:master.


Simple shortcut to interrupt long response

All it does is capture Ctrl-C to interrupt the text stream. To exit the program while a stream is running you hit Ctrl-C twice. (View this PR with whitespace changes off to see what's actually changed.)
Fixes #319
I implemented this for my own needs, it hasn't undergone any testing or consideration of edge cases.

Thanks @artsparkAI, merged.


VSCode Task to create new packages

Create a new VSCode Task do to the following when run:
Paramater: package_name
Actions:

Create two new files package_name.pks and package_name.pkb (note the file name extensions are now in scripts/project-config.sh
Load each of the files from the associated templates in the templates folder
Rename the package names to package_name

I find I'm doing this over and over again when creating a lot of new packages. It's a minor thing but could help people with new package development.
Notes:

Slightly different than snippets in that this will create files
Can be expanded for views (or other objects)

@rimblas this is now in place. Can you please give it a try and provide your feedback?

Files get created but I am getting the following error:

@vitodcampanelli I think this could be a difference in our versions of sed. Can you please test what happens with this:
sed -i -bak "s/CHANGEME_TASKLABEL/$PROJECT_DIR_FOLDER_NAME/g" $VSCODE_TASK_FILE

@vitodcampanelli can you also include the exact command that you ran? Find it odd it's complaining about -a when we don't use that.

@martindsouza Here is the command
sed -i -bak "s/CHANGEME_TASKLABEL/$PROJECT_DIR_FOLDER_NAME/g" $VSCODE_TASK_FILE

@vitodcampanelli what are the values for $PROJECT_DIR_FOLDER_NAME and $VSCODE_TASK_FILE (want to make sure no special characters in path or spaces

Ok when I run the following
# Directory of this file
SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
echo "SCRIPT_DIR: " $SCRIPT_DIR

# Load helper
source $SCRIPT_DIR/../../scripts/helper.sh

echo "PROJECT_DIR_FOLDER_NAME: " $PROJECT_DIR_FOLDER_NAME
echo "VSCODE_TASK_FILE: " $VSCODE_TASK_FILE

Output is:
SCRIPT_DIR:  /Users/vitoc/GitHub/starter-project-template-2/.vscode/scripts
sed: invalid option -- 'a'
Usage: sed [OPTION]... {script-only-if-no-other-script} [input-file]...

  -n, --quiet, --silent
                 suppress automatic printing of pattern space
      --debug
                 annotate program execution
  -e script, --expression=script
                 add the script to the commands to be executed
  -f script-file, --file=script-file
                 add the contents of script-file to the commands to be executed
  --follow-symlinks
                 follow symlinks when processing in place
  -i[SUFFIX], --in-place[=SUFFIX]
                 edit files in place (makes backup if SUFFIX supplied)
  -l N, --line-length=N
                 specify the desired line-wrap length for the `l' command
  --posix
                 disable all GNU extensions.
  -E, -r, --regexp-extended
                 use extended regular expressions in the script
                 (for portability use POSIX -E).
  -s, --separate
                 consider files as separate rather than as a single,
                 continuous long stream.
      --sandbox
                 operate in sandbox mode (disable e/r/w commands).
  -u, --unbuffered
                 load minimal amounts of data from the input files and flush
                 the output buffers more often
  -z, --null-data
                 separate lines by NUL characters
      --help     display this help and exit
      --version  output version information and exit

If no -e, --expression, -f, or --file option is given, then the first
non-option argument is taken as the sed script to interpret.  All
remaining arguments are names of input files; if no input files are
specified, then the standard input is read.

GNU sed home page: <https://www.gnu.org/software/sed/>.
General help using GNU software: <https://www.gnu.org/gethelp/>.
rm: /Users/vitoc/GitHub/starter-project-template-2/.vscode/tasks.json-bak: No such file or directory
PROJECT_DIR_FOLDER_NAME:
VSCODE_TASK_FILE:

Removing the line
source $SCRIPT_DIR/../../scripts/helper.sh

Output has no errors but the variables blank as expected.
SCRIPT_DIR:  /Users/vitoc/GitHub/starter-project-template-2/.vscode/scripts
PROJECT_DIR_FOLDER_NAME:
VSCODE_TASK_FILE:


[FEAT] Linting rules for describe() depth in tests

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior
Currently, tests don't follow well defined patterns or style guides.
Expected behavior
That tests follow well designed patterns and style guides.
What is the motivation / use case for changing the behavior?
As a start, in addition to the Angular styleguide rules, it would be nice if we could enforce a "describe" depth of 3, which would allow us to do meta-analysis of the complexity of a piece of code.

It looks like eslint supports this for jest. https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/max-nested-describe.md


EE-1207: make definition of system account AccountHash consistent (dev)

This PR ports changes from #1221 to dev.

bors r+


Also, there is no way to escape that menu.

This functionality is unimplemented so far, which would explain why it flat-out does not work. I plan on transitioning this menu to a ContentDialog and adding these options eventually, I'll prioritize critical function bugs like item paste collision, etc. over this.

Fixed in v0.4.7


None of the raster views should try allocate 0x0 memory. Fixes #759

In some cases, a view dimension may be < 1, thus the allocation for 0 bytes of memory is not actually possible.
Fixes #759

VS bug #770231

OPS Build status updates of commit 81a44ed:
:clock10: Preparing: average preparing time is 2 min(s) 1 sec(s)

OPS Build status updates of commit 81a44ed:
:clock10: Incremental building: average incremental building time is 19 sec(s)

OPS Build status updates of commit 81a44ed:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

source/SkiaSharp.Views/SkiaSharp.Views.Android/SKCanvasView.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Apple/SKCGSurfaceFactory.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Apple/SKCanvasLayer.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.AppleiOS/SKCanvasView.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Desktop/SKControl.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Gtk/SKWidget.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Mac/SKCanvasView.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.Tizen/SKCanvasView.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.UWP/SKXamlCanvas.cs
:white_check_mark:Succeeded

source/SkiaSharp.Views/SkiaSharp.Views.WPF/SKElement.cs
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


(GroupProfiles) Implement members library for groups ranked on profile pic and role

As you can see, this is based on principals delete, so hold up for that to be merged :)

I have rebased this with master, should be good for a high-level review but I still have yet to do a self-review.

@mrvisser : I wanted to do a functional review of the group profiles, but it looks like oae-emitter might be missing from this PR.

@nicolaasmatthijs argh, didn't update the .gitignore etc etc... I've added it in now.

Self-review is complete. Assigning to @simong for review

Any reason these emitters weren't changed?

node_modules/oae-activity/lib/internal/emitter.js:var events = require('events');
node_modules/oae-activity/lib/internal/notifications.js:var events = require('events');
node_modules/oae-library/lib/api.index.js:var events = require('events');
node_modules/oae-principals/lib/api.user.js:var events = require('events');
node_modules/oae-util/lib/oae.js:var events = require('events');
node_modules/oae-util/lib/test/mq-util.js:var events = require('events');

Any reason these emitters weren't changed?

No reason (except for the oae-emitter, for obvious reasons).

Any reason these emitters weren't changed?

Turns out all except 1 of these didn't actually use the events import. So I must have grepped for .ons or something.

Followed up on all review up to this point.

PR looks good. Couple of minor nitpickings. Back to @mrvisser

I've either followed up or replied to all review up to this point. Back to @simong

LGTM. Just waiting on the UI counter-part before merging

Changes Unknown when pulling 6a75dce613592336280e81ba34618b3e0b00f36f on mrvisser:group-profiles into ** on oaeproject:master**.


multiple gpu training imbalance

i run train with 4 RTX 2080 8GB, i set batch_size=16  then memory usage as image below, i dont know why GPU 1 take more memory usage than others, when i change batch_size to 32 it  lead to OOM
CMD train: CUDA_VISIBLE_DEVICES=0,1,2,3 python -m multiproc train.py --output_directory=outdir_voice --log_directory=logdir_voice --n_gpus=4 --hparams=distributed_run=True,fp16_run=True -c outdir_voice/checkpoint_0 --warm_start
Memory usage when batch_size=16

Same problem, have you solve it, I remember I have reached this but forget the reason, it may relate to Cudnn and distributed training.

@v-nhandt21 I would also check the lengths of your audio files since everything is zero padded, a audio file that's double the normal length would cause the VRAM usage to double on a single card.

@v-nhandt21 I would also check the lengths of your audio files since everything is zero padded, a single audio file that's double the length of other audio files would cause the VRAM usage to double on a single card, but not the others.
(and the reason it only affects one card is because shuffling only happens once before epoch 0 when using distributed training. see the warning at the bottom for how to add shuffling between epochs)

In distributed mode, calling the set_epoch() method at the beginning of each epoch before creating the DataLoader iterator is necessary to make shuffling work properly across multiple epochs. Otherwise, the same ordering will be always used.

Great, thank you!
One more thing I note here for whom in the future is that we "should" set the world_size in distributed training equal to number of gpus.
And in my case, I use cuda11.1 pytorch 1.9 to make everything work ok.
Should we close this issues?

@v-nhandt21 I would also check the lengths of your audio files since everything is zero padded, a single audio file that's double the length of other audio files would cause the VRAM usage to double on a single card, but not the others.
(and the reason it only affects one card is because shuffling only happens once before epoch 0 when using distributed training. see the warning at the bottom for how to add shuffling between epochs)

In distributed mode, calling the set_epoch() method at the beginning of each epoch before creating the DataLoader iterator is necessary to make shuffling work properly across multiple epochs. Otherwise, the same ordering will be always used.

i will try, thank you


fix(base-modal): add weakref polyfill

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

3 of 23 (13.04%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 84.335%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/base-modal/src/weakref-polyfill.ts
2
22
9.09%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
9554

Relevant Lines:
10472

💛  - Coveralls

Тут на чём зависло?

Тут на чём зависло?

отдала на тест и ребята не вернулись


[FEATURE-REQUEST]Document install from RPM file?

Describe the solution you'd like
Currently the README.md shows two options for installing Service Commander:
Option 1: Binary distribution
Option 2: Build from source (for development or fix evaluation)
There is a third option: Installing from the RPM file (often) found in the release assets.
I think this option has been suggested in one of the issues during testing?
I have now installed Service Commander pre-release version v1.4.3 on a test system using the command
→ yum install https://github.com/ThePrez/ServiceCommander-IBMi/releases/download/v1.4.3/service-commander-1.4.3-0.ibmi7.2.ppc64.rpm

Any caveats to this method? Or should it be documented as a way for the user to get a specific release?
Describe alternatives you've considered
Additional context
The RPM file is not consistently build to a release - why is that? Errors found in the release afterwards?

Thanks for opening the discussion
Before the IBM repo shipped the package, installing from GH was the only way. When IBM started shipping the package, I figured it's best to just have the simple version (yum install service-commander)
The IBM copy, generally speaking, should be mostly up-to-date, but there will inevitably be times where the IBM repo lags behind the github release (in May there were a few build hiccups which made this lag period kinda lengthy).
Maybe we should document that it's available in the IBM repo, but that there may be times where the GH link is needed for latest-and-greatest?

The first option is actually installing from the IBM repo - so no need for further documentation of this.
I have made a PR #176 with a proposed documentation (option 3) of how to install from RPM file on GitHub. Have a look and see if you like it. It also contains other commits fixing typos and other stuff in the README.

This was fixed by PR #176 - closing...


DiscordClient IsBotAccount field is always false

public DiscordClient(string tokenOverride = null, bool isBotAccount = false)
        {
            token = tokenOverride;
            **isBotAccount = IsBotAccount;**

            // more code
        }

Seems to me as a typo as it should be IsBotAccount = isBotAccount, since this way you are assigning a default (false) value to the supplied parameter variable. This way the bot never adds the appropriate values to auth headers.

Yup, innocent typo. My bad :D


"Failed to write to ExtData archive" and "Failed to delete ExtData archive"

Hi! I have followed the instructions as posted on this thread, and am running into issues at the last step when injecting the custom badge files.
The files are located in /3ds/SimpleBadgeInjector on my SD card, and when attempting to inject them, I get a message "Failed to write to ExtData archive". It is also not possible for me to delete and create a new ExtData archive, as I simply get "Failed to delete ExtData archive" when attempting to delete.
I am not 100% certain as to how to troubleshoot this any further. My unique badge count is at 410, the total badges at 104550, so I should not be running into issues with the badge limit as far as I am aware. One idea I have is that it might be related to importing JPN badges to EUR data sets, but I'm not certain. Nevertheless, that shouldn't be causing an issue with deleting the ExtData archive.
Please let me know if you have any ideas as to how to fix these, or how I can be of support with troubleshooting this.

Try using the CIA file instead, the 3dsx file seems to have issues.
(Sorry for replying to this old thread, meant this to help any other people who run into this issue and see this as well)

Try using the CIA file instead, the 3dsx file seems to have issues.
(Sorry for replying to this old thread, meant this to help any other people who run into this issue and see this as well)

我遇到了和他基本一模一样的问题，无法删除、创建、或者注入备份的数据，甚至是我刚刚为自己备份的数据，而且我使用的就是CIA文件的版本
请问你能否提供一下你的操作步骤呢？
I encountered basically the same problem as them. I couldn't delete, create, or inject backed up data, even the data I just backed up from itself, and I was using the version of the CIA file.
Could you please provide me with your steps?


Improve the Documentation on the Python API

Improve the doc-strings, type annotations, and provide a guide or two for interfacing with Python package directly and not through the CLI.

Made improvements with #44


added local pipeline scripts

Added some scripts that will run the local pipeline besides the scripts that are already there. Running [npm run doc] will generate the newest version of JSDoc for our repo locally, and you can view it in index.html in the docs folder. Running [npm run eslint] will basically do the same thing as GitHub Action pipeline, showing you warnings and errors. Running [npm run code-cov] will generate two local http server which you can click in to open up the broswer to see the newest test coverage. Note that running [npm run code-cov] requires you to run [npm pretest] and [npm unit-test] since coverage folder only generates after unit test runs.

[ ] Does the PR description accurately reflect the changes made?
[ ] Do the commit messages accurately reflect the changes made?
[ ] Is the changelist for this pull request under ~200 lines? (Excepting automatically generated code and package installations)
[ ] Does the code follow the coding style guidelines that we have set up?
[ ] Does the coder include unit tests for his functions?
[ ] Does the QA who reviews the code include his unit tests for all the functions?
[ ] Is there a documentation in place if the functions is related to how users build, test, interact with, or release code?

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


Refactor rollout

Rollout Refactor
Problem
RolloutEval and GraphForecaster rollout code had diverged from an inital copy.
This PR refactors both to combine the rollout logic.
Closes #84

📚 Documentation preview 📚: https://anemoi-training--87.org.readthedocs.build/en/87/

Tested and allows for longer validation rollout than training


make sure fields param works in all routes

@ojalaquellueva looked at this - right now there are a few routes that can accept user input fields param with what fields they want back - but i've been followingn the SQL used in the R pkg, and they've hardcoded the select fields (the fields to return)
I could code the routes to just accept any fields, but that would be changing behavior from what the R API was doing - up to you all
some explanation in docs for the fields param https://ropensci.github.io/bienapidocs/#sometimes-available


Does this library support react native?

If so, what version is it support? For my case, I am on 0.75.3 version of react native.

BTW. Tried latest 3.0.1 version. Built successful, but not working. Code after calling class construct display that.
 (NOBRIDGE) ERROR  [ReferenceError: Property 'structuredClone' doesn't exist]

Apparently Hermes does not support structuredClone()... which is a bit surprising.
https://github.com/facebook/hermes/issues/684
I'll see if I can include a polyfill in the next update.

@NielsLeenheer May I ask you to add some variants to print images on web and mobile side? Cause I can't find a way to pass image data to image method of escpos builder 🫶

In the examples directory there are a lot of libraries that can be used with ReceiptPrinterEncoder, and some are plain JavaScript, so I am sure some of them support React Native as well. But I am not familiar with how React Native works, so I can't give you examples of how to use it.
https://github.com/NielsLeenheer/ReceiptPrinterEncoder/tree/main/examples/images

Got something similar: Cannot find 'ImageData' in browser.js (3:18).
Expo 51, React Native, 0.74.5, hermes engine. It's try to import an canvas with will not work in RN.

That main be an issue with the bundler your project is using. ImageData is import from the @canvas/image-data package. That package has two code paths: one for the browser and one for Node. Apparently the bundler picks the path for the browser. And then it expects the browser native ImageData to be there. Which it isn't.
Not sure if that is helpful or not. Maybe the @canvas/image-data package itself is not React Native compatible. If that is the issue then perhaps we can work around it in a future version.
Ideally, for better React Native support I would need a sample project somewhere on Github which I could clone and play around with, without having to learn React Native from the ground up.

You can use Expo Snack, it's an easy way to test lib on real devices. Here is example

@serhii-yalla @karaushu Were you guys able to figure out if ReceiptPrinterEncoder can work correctly with react-native or not ?
I am also getting the same error around structuredClone as mentioned by @serhii-yalla in above comment.


[GH-21101] Migrated file_upload_overlay.test.jsx to typescript

Summary
Migrated file_upload_overlay.test.jsx to typescript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/21101
Fixes https://mattermost.atlassian.net/browse/MM-47062
Related Pull Requests
None
Screenshots
NA
Release Note
None
Migrated file_upload_overlay.test.jsx to typescript

Hello @AbhinavVihan,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

E2E tests not automatically triggered, because the PR is not in a mergeable state. Please update the branch with the base branch and resolve outstanding conflicts.


Add use of Merge Request Approval API.

Implement Merge Request Approval API's 'Change Configuration' section

I'm sorry that I have to inform you that this logic is already available in this package: https://github.com/xanzy/go-gitlab/blob/master/projects.go#L1294


chroot setup failure

Alright so I'm pretty new to this so apologies if this is all very poorly put.
I've tried installing chroot for my chromebook and it has not worked this is what is was met with
Failed to complete chroot setup.
The chroot setup script may be broken. Your chroot is not fully configured.
Removing the chroot setup script. You may want to update your chroot again.
UID 1000 not found in xenial
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Maybe you should reset.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: bigweeboo<EMAIL_ADDRESS>Sent: Thursday, March 7, 2019 12:01:17 AM
To: dnschneid/crouton
Cc: Subscribed
Subject: [dnschneid/crouton] chroot setup failure (#4008)
Alright so I'm pretty new to this so apologies if this is all very poorly put.
I've tried installing chroot for my chromebook and it has not worked this is what is was met with
Failed to complete chroot setup.
The chroot setup script may be broken. Your chroot is not fully configured.
Removing the chroot setup script. You may want to update your chroot again.
UID 1000 not found in xenial
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dnschneid/crouton/issues/4008, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AseuJtUcPkofJnJ1OM2VNsLtWXprrsI6ks5vT-ZNgaJpZM4bhO4i.

How?

@bigweeboo,
Show us your full command-line when you tried to install your chroot including the targets, etc.
That might give us enough info. to help out.
Hope this helps,
-DennisLfromGA

sudo sh ~/Downloads/crouton -e -t xfce

@bigweeboo,
Try to 'update' the chroot with the '-u' option like:

sudo sh ~/Downloads/crouton -n xenial -u

There's a chance that might work.
If that doesn't work you may have to delete the chroot with:

sudo delete-chroot xenial

Then start over and, just to keep things simple, try it without the '-e' encryption option like this:

sudo sh ~/Downloads/crouton -t xfce

And see if it completes like that, you can encrypt it later if you like.
Hope this helps,
-DennisLfromGA

I'm currently having a issue trying to get courton to work as well. I'll attach two screenshots one of my command history and of the install script issue that I and others have been running into.
More interested on how I could run more programs on my chomebook such as League of legends or even diablo 2 just to do more on the chrome book I don't fully understand a whole lot on hardware requirements with the chromebook, Maybe I could just run wine with regular  Crostini  instead of courton? Not sure how I pull this off properly to be honest.

Trying to tread carefully with dev mode on this chomebook.

@flexhornmoringstar this issue is from two years ago and is closed.


[go_router] Using GoRouter.routingConfig and switch between StatefulShellRoute and GoRoute will cause type error

Steps to reproduce
Reproducible in go_router 14.2.3
In mobile device, we use bottom nav bar, something like:
final mobileConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: route1Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: route2Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: route3Builder)]),
    ],
  ),
  GoRoute(path: '/route4', builder: route4Builder),
  GoRoute(path: '/route4', builder: route5Builder),
]);

In desktop device, we use rail, which allow more branches:
final desktopConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: route1Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: route2Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: route3Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route4', builder: route4Builder)]),
      StatefulShellBranch(routes: [GoRoute(path: '/route5', builder: route5Builder)]),
    ],
  ),
]);

When we monitor device width and change the routing config from mobileConfig to desktopConfig, it will cause error:
type 'ShellRouteMatch' is not a subtype of type 'RouteMatch' of 'value'

First use mobileConfig by setting device to portrait
Nav to /route5
Turn the device to landscape
Error show up

Expected results
Still stay in /route5 without any error.
Actual results
Throw the error and stop changing route config:
════════ Exception caught by foundation library ════════════════════════════════
The following _TypeError was thrown while dispatching notifications for ValueNotifier<RoutingConfig>:
type 'ShellRouteMatch' is not a subtype of type 'RouteMatch' of 'value'

When the exception was thrown, this was the stack:
#0      List.add (dart:core-patch/growable_array.dart:277:14)
growable_array.dart:277
#1      RouteMatchList._createNewMatchUntilIncompatible (package:go_router/src/match.dart:623:10)
match.dart:623
#2      RouteMatchList.push (package:go_router/src/match.dart:593:16)
match.dart:593
#3      RouteConfiguration.reparse (package:go_router/src/configuration.dart:328:23)
configuration.dart:328
#4      GoRouter._handleRoutingConfigChanged (package:go_router/src/router.dart:298:27)
router.dart:298
#5      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:433:24)
change_notifier.dart:433
#6      ValueNotifier.value= (package:flutter/src/foundation/change_notifier.dart:555:5)
change_notifier.dart:555
#7      App.build (package:example/app.dart:30:20)
app.dart:30
#8      StatelessElement.build (package:flutter/src/widgets/framework.dart:5557:49)
framework.dart:5557
#9      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:5487:15)
framework.dart:5487
#10     Element.rebuild (package:flutter/src/widgets/framework.dart:5203:7)
framework.dart:5203
#11     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2905:19)
framework.dart:2905
#12     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:1136:21)
binding.dart:1136
#13     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:443:5)
binding.dart:443
#14     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1392:15)
binding.dart:1392
#15     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1313:9)
binding.dart:1313
#16     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:1171:5)
binding.dart:1171
#20     _invoke (dart:ui/hooks.dart:314:10)
hooks.dart:314
#21     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:419:5)
platform_dispatcher.dart:419
#22     _drawFrame (dart:ui/hooks.dart:283:31)
hooks.dart:283
(elided 3 frames from dart:async)

The ValueNotifier<RoutingConfig> sending notification was: ValueNotifier<RoutingConfig>#ac657(Instance of 'RoutingConfig')
════════════════════════════════════════════════════════════════════════════════

Code sample
Code sample
class App extends StatelessWidget {
  final routingConfig = ValueNotifier<RoutingConfig>(mobileConfig);

  App({super.key});

  @override
  Widget build(BuildContext context) {
    return LayoutBuilder(
      builder: (context, constraint) {
        routingConfig.value =  constraint.width < 600 ? mobileConfig : desktopConfig;
        return MaterialApp.router(
          routerConfig: routingConfig.value,
        );
      },
    );
  }
}

final mobileConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: createBuilder('route1'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: createBuilder('route2'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: createBuilder('route3'))]),
    ],
  ),
  GoRoute(path: '/route4', builder: createBuilder('route4')),
  GoRoute(path: '/route4', builder: createBuilder('route5')),
]);
final desktopConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: createBuilder('route1'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: createBuilder('route2'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: createBuilder('route3'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route4', builder: createBuilder('route4'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route5', builder: createBuilder('route5'))]),
    ],
  ),
]);

Widget Function(BuildContext, GoRouterState) createBuilder(String name) {
  return (ctx, state) => Scaffold(body: Text(name));
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
flutter doctor -v                             Sun Aug 11 10:36:21 2024
[✓] Flutter (Channel stable, 3.22.1, on macOS 14.6.1 23G93 darwin-arm64, locale en-TW)
    • Flutter version 3.22.1 on channel stable at /Users/evan.lu/Development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a14f74ff3a (3 months ago), 2024-05-22 11:08:21 -0500
    • Engine revision 55eae6864b
    • Dart version 3.4.1
    • DevTools version 2.34.3

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/evan.lu/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS and macOS development.
      Download at: https://developer.apple.com/xcode/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin
        usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

[✓] VS Code (version 1.92.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.94.0

[✓] Connected device (3 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554 • android-arm64  • Android 14 (API 34) (emulator)
    • macOS (desktop)             • macos         • darwin-arm64   • macOS 14.6.1 23G93 darwin-arm64
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 127.0.6533.100

[✓] Network resources
    • All expected network resources are available.

@evan361425
Can you provide complete runnable code sample ? It seems to have custom implementation.

 code

import 'package:flutter/material.dart';
import 'package:flutter/cupertino.dart';
import 'package:go_router/go_router.dart';

void main() => runApp(App());

class App extends StatelessWidget {
  final routingConfig = ValueNotifier<RoutingConfig>(mobileConfig);

  App({super.key});

  @override
  Widget build(BuildContext context) {
    return LayoutBuilder(
      builder: (context, constraint) {
        routingConfig.value =  constraint.width < 600 ? mobileConfig : desktopConfig;
        return MaterialApp.router(
          routerConfig: routingConfig.value,
        );
      },
    );
  }
}

final mobileConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: createBuilder('route1'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: createBuilder('route2'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: createBuilder('route3'))]),
    ],
  ),
  GoRoute(path: '/route4', builder: createBuilder('route4')),
  GoRoute(path: '/route4', builder: createBuilder('route5')),
]);
final desktopConfig = RoutingConfig(routes: [
  StatefulShellRoute.indexedStack(
    builder: builder,
    branches: [
      StatefulShellBranch(routes: [GoRoute(path: '/route1', builder: createBuilder('route1'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route2', builder: createBuilder('route2'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route3', builder: createBuilder('route3'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route4', builder: createBuilder('route4'))]),
      StatefulShellBranch(routes: [GoRoute(path: '/route5', builder: createBuilder('route5'))]),
    ],
  ),
]);

Widget Function(BuildContext, GoRouterState) createBuilder(String name) {
  return (ctx, state) => Scaffold(body: Text(name));
}

@darshankawar thanks for the replied. I've update the code sample, please take a look.

Thanks for the update. Seeing the same behavior using updated code sample and running in emulator.
======== Exception caught by foundation library ====================================================
The following _TypeError was thrown while dispatching notifications for ValueNotifier<RoutingConfig>:
type 'ShellRouteMatch' is not a subtype of type 'RouteMatch' of 'value'

When the exception was thrown, this was the stack:
#0      List.add (dart:core-patch/growable_array.dart:278:14)
#1      RouteMatchList._createNewMatchUntilIncompatible (package:go_router/src/match.dart:623:10)
#2      RouteMatchList.push (package:go_router/src/match.dart:593:16)
#3      RouteConfiguration.reparse (package:go_router/src/configuration.dart:328:23)
#4      GoRouter._handleRoutingConfigChanged (package:go_router/src/router.dart:298:27)
#5      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:437:24)
#6      ValueNotifier.value= (package:flutter/src/foundation/change_notifier.dart:559:5)
#7      Example.build.<anonymous closure> (package:new_web_app/main.dart:18:23)
#8      _LayoutBuilderElement._rebuildWithConstraints.updateChildCallback (package:flutter/src/widgets/layout_builder.dart:196:77)
#9      BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:3046:19)
#10     _LayoutBuilderElement._rebuildWithConstraints (package:flutter/src/widgets/layout_builder.dart:236:12)
#11     RenderObject.invokeLayoutCallback.<anonymous closure> (package:flutter/src/rendering/object.dart:2738:59)
#12     PipelineOwner._enableMutationsToDirtySubtrees (package:flutter/src/rendering/object.dart:1108:15)
#13     RenderObject.invokeLayoutCallback (package:flutter/src/rendering/object.dart:2738:14)
#14     RenderConstrainedLayoutBuilder.rebuildIfNecessary (package:flutter/src/widgets/layout_builder.dart:283:5)
#15     _RenderLayoutBuilder.performLayout (package:flutter/src/widgets/layout_builder.dart:374:5)
#16     RenderObject.layout (package:flutter/src/rendering/object.dart:2627:7)
#17     RenderView.performLayout (package:flutter/src/rendering/view.dart:285:12)
#18     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:2465:7)
#19     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:1062:18)
#20     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:1075:15)
#21     RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:609:23)
#22     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:1178:13)
#23     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:475:5)
#24     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1397:15)
#25     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1318:9)
#26     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:1176:5)
#27     _invoke (dart:ui/hooks.dart:312:13)
#28     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:419:5)
#29     _drawFrame (dart:ui/hooks.dart:283:31)
The ValueNotifier<RoutingConfig> sending notification was: ValueNotifier<RoutingConfig>#622f0(Instance of 'RoutingConfig')
====================================================================================================

I've fixed the error locally by changing this line:
final List<RouteMatchBase> newMatches = currentMatches.toList();

to
final List<RouteMatchBase> newMatches = currentMatches.cast<RouteMatchBase>().toList();

The problem is when currentMatches is an empty list, Dart seams to not cast correctly to List<RouteMatchBase>. Instead, it casts to List<RouteMatch>.


Documentation update needed

docs/src/index.rst has been edited, needs publishing.
Please also confirm publishing process here for reference.

done and can update in the future using the same process, thanks @sambles


Simple digital food request form

We need the ability to display a simple form on the website that allows Food Shelf Clients to submit requests for food. Pulling from the client choice form provided by @kklamberty may be useful.


worker opcode change + tests BaseCommand & Init

Description

[x] to be merged wth https://github.com/adobe/generator-aio-app/pull/150

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

init has one failing test on Windows (100% coverage however):
 ● run › --no-login, select excshell, arg: /otherdir

    expect(jest.fn()).toHaveBeenCalledWith(...expected)

    Expected: "/otherdir"
    Received: "C:\\otherdir"

    Number of calls: 1

      217 |     expect(mockImport.importConfigJson).not.toHaveBeenCalled()
      218 |
    > 219 |     expect(fs.ensureDirSync).toHaveBeenCalledWith('/otherdir')
          |                              ^
      220 |     expect(process.chdir).toHaveBeenCalledWith('/otherdir')
      221 |   })
      222 |

      at Object.<anonymous> (test/commands/app/init.test.js:219:30)

----------|---------|----------|---------|---------|-------------------
File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------|---------|----------|---------|---------|-------------------
All files |     100 |      100 |     100 |     100 |
 init.js  |     100 |      100 |     100 |     100 |
----------|---------|----------|---------|---------|-------------------

BaseCommand does not have full coverage:
 PASS  test/BaseCommand.test.js
  √ exports (2 ms)
  √ flags (1 ms)
  √ args
  √ basecommand defines method (3 ms)
  √ getLaunchUrlPrefix() warns on older url (1 ms)
  √ getLaunchUrlPrefix() uses stage launch prefix (1 ms)
  getFullConfig
    √ keeps cache (1 ms)
    √ with options
  getConfigFileForKey
    √ returns empty object if not found (2 ms)
    √ returns file and key if found (1 ms)
    √ no actions
    √ multiple implementations (2 ms)
  getAppExtConfigs
    √ no extension flags (1 ms)
    √ with options (1 ms)
    √ -e exc -e asset (1 ms)
    √ -e application (1 ms)
    √ -e exc -e notexists (12 ms)
    √ -e dx (matches more than one) (1 ms)
  getLibConsoleCLI
    √ test cache (1 ms)
    √ prod env  (1 ms)
    √ stage env

----------------|---------|----------|---------|---------|-------------------
File            | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------------|---------|----------|---------|---------|-------------------
All files       |   88.06 |      100 |   57.14 |   87.88 |
 BaseCommand.js |   88.06 |      100 |   57.14 |   87.88 | 32-39,53,144-152
----------------|---------|----------|---------|---------|-------------------
Jest: "global" coverage threshold for statements (100%) not met: 88.06%
Jest: "global" coverage threshold for lines (100%) not met: 87.88%

Thanks @shazron, just sent a commit, should be good now


About Command line tools (Other:)

In the following link of "Command line tools (Other:)", is this document has been updated by the newest information?
https://docs.microsoft.com/en-us/azure/architecture/data-guide/scenarios/data-transfer#command-line-tools
And what is the "Optimized for big data"? Azure Data Lake Storage Gen2?
And what is the "Data Lake Store"? Azure Data Lake Storage Gen1?
Please check them. Thanks, Masaki.

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

ID: 3c6e037b-a226-62cf-6e2b-2af8783b7add
Version Independent ID: c666597e-52ae-ee51-474a-c4dd3a63941b
Content: Choosing a data transfer technology
Content Source: docs/data-guide/scenarios/data-transfer.md
Service: architecture-center
GitHub Login: @ZoinerTejada
Microsoft Alias: pnp

@matakeda1 Thank you for reporting this issue! We will investigate and make any necessary changes.

Hi @KetanChawda-MSFT. I'm on the AzCopy V10 team, and was just about to log an issue about this when I saw that one was already logged. I'm not sure whether "optimized for big data" means "can transfer large data volumes" or " can transfer data to data lake".  In any case, AzCopy v10 can transfer large data volumes AND can transfer to Azure Data Lake Gen 2. So I think it should be shown as Optimized For Big Data.


Undesired content no longer set properly on image generation

Since the apparent changes to the NovelAI API a couple of days ago, undesired content no longer seems to be set properly.
All the images I've been able to generate since come back with uc="" (and are notably low quality as a result!). I've confirmed that parameters['uc'] is properly set in _low_level.py when the API call is made. I guess perhaps the naming of the parameters at the NovelAI end has changed?

On the topic of the parameter names here is a trick I learned. Since updates to the api can cause name changes you can check the names of the different parameters by generating a image on the novelai website while you have the network tab opened.

Fixed in 915e83a
Change was unexpected.


Integrate MolQL transpilers in Mol*

Greetings!
I have seen the examples offered at http://molql.org/explorer.html and I would like to integrate the PyMol and VMD transpilers in a Molstar fork that I have. My goal is to allow selections from any language the user prefers, just like in the MolQL explorer. From what I understand, the transpilers produce something pretty close to what Molstar "speaks". For example,
StructureSelectionQueries.water.expression() (see https://github.com/molstar/molstar/blob/704a9a111dff82e3566276068b1c33505570ccb7/src/mol-plugin-state/helpers/structure-selection-query.ts#L276)
returns:
{
    head: {
      name: "structure-query.modifier.union",
    },
    args: [
      {
        head: {
          name: "structure-query.generator.atom-groups",
        },
        args: {
          "entity-test": {
            head: {
              name: "core.rel.eq",
            },
            args: [
              {
                head: {
                  name: "structure-query.atom-property.macromolecular.entity-type",
                },
              },
              "water",
            ],
          },
        },
      },
    ],
  }

which I can use to add a representation for the waters in a given structure.
On the other hand, water selection in the MolQL explorer yields:
{
  "source": "molql-explorer",
  "version": "0.1.0",
  "expression": {
    "head": "structure.generator.atom-groups",
    "args": {
      "residue-test": {
        "head": "core.set.has",
        "args": [
          {
            "head": "core.type.set",
            "args": [
              "H2O",
              "HH0",
              "OHH",
              "HOH",
              "OH2",
              "SOL",
              "WAT",
              "TIP",
              "TIP2",
              "TIP3",
              "TIP4"
            ]
          },
          {
            "head": "structure.atom-property.macromolecular.label_comp_id"
          }
        ]
      }
    }
  }
}

I tried to substitute the first query with the expression part of the second, but Molstar greeted me with an Error: Can only apply symbols message.
Any hints on how to approach this?

Hi, this should be possible without too many changes. The example you provided is "broken" because the structure namespace has been renamed to structure-query in Mol*, but mostly the language is the same.
Just getting the source from https://github.com/MolQL/molql/tree/master/src/transpilers/pymol and vmd folders and placing it in https://github.com/molstar/molstar/tree/master/src/mol-script should get you most of the way.
Notable replacements are parsimmon => mol-util/monadic-parser and /molql/builder => mol-script/language/builder's MolScriptBuilder.
Would you be willing to make a PR to the Mol* repository with this and we can make it work there? I am sure other people would appreciate this as well!

As a handy reference to anyone interested, the relevant PR is https://github.com/molstar/molstar/pull/325


Performance is bad in Observable Notebooks.

Description:
I'm not sure why, but performance is really poor in observable notebooks. My best guess currently is that KDE takes a lot of time, as the performance issues seem somewhat correlated with the number of graphs I render at the same time.
This could give more reason to use histograms rather than KDE to display sample set distributions. Which would also fix the issue of rendering samples below 0.
Steps to reproduce:
Some of my notebooks here: https://observablehq.com/@hazelfire/quantifying-uncertainty-in-givewell-ceas (particularly Helen Keller and Malaria Consortium, but also just the main notebook)

I think it's more likely that the cost is somewhere in rendering point sets. In that case we'd want to cut down the number of points before rendering by averaging groups of them together or something. It should be possible to test this by running a notebook that plots many symbolic distributions converted directly to point sets, as this doesn't go through KDE.
#1339 greatly improves the performance of KDE itself, and I don't think it should be significantly different from histogramming now: the per-sample operations are about the same, with the difference for KDE being post-processing on the points, and there are usually a lot less points than samples. The overall conversion has a lot of other costs that I haven't fully accounted for yet. Sorting samples is a big one.


Set topic namespace in interactive markers display

Fixes #658
The interactive markers display inherits from Display instead of RosTopicDisplay, so it was missing the setTopic() function. With these changes, the topic namespace will now be set by default when adding the interactive markers display by topic.
Signed-off-by: Rebecca Butler<EMAIL_ADDRESS>

CI build: --packages-up-to rviz_default_plugins

Linux
Linux-aarch64
macOS
Windows


Account snapshot not working

Documentation
https://python-binance.readthedocs.io/en/latest/general.html?highlight=snapshot#id7
Get Get Daily Account Snapshot
Error message

info = client.get_account_snapshot(type='SPOT')

AttributeError: 'Client' object has no attribute 'get_account_snapshot'

Expected behavior
Show my a snapshot like available with binanace REST-API
GET /sapi/v1/accountSnapshot
Environment (please complete the following information):

Python version: 3.6
Virtual Env: [e.g. virtualenv, conda]
OS: raspbian
python-binance version - installed yesterday - how to show the version?

not latest version in pip --> you have to compile "master"