Genotyping with breakpoint deviations

Hi Jana,
Since PanGenie relies on exact kmer matches, I want to know if it will perform well with input SVs that have imprecise breakpoints? Have you investigated how inaccurate breakpoints impact the genotyping results? Thank you.
Best regards,
Zheng zhuqing

Hi Zheng zhuqing,
I have never really evaluated this, but as you mentioned, since PanGenie uses exact kmer matches, inaccurate breakpoints might be a problem in some cases. We have observed in the past that the variant merging and resulting inaccurate allele sequences seemed to have negative effects on the genotyping accuracy, so my expectation is that it's probably similar with inaccurate breakpoints.
Best,
Jana


support microdata with content attribute

itemprop keys could have value in content attribute if they are in meta tags.
<link itemprop="url" href="https://www.youtube.com/watch?v=phuiiNCxRMg">
<meta itemprop="requiresSubscription" content="False">
<meta itemprop="identifier" content="phuiiNCxRMg">
<meta itemprop="duration" content="PT3M14S">

Thank you for your comment!
I have corrected it, a new release has been prepared. (v0.1.3)


[Bug] Paper Overlay Should Not Scroll

When making Paper a scrollable container, the overlay scrolls as well meaning that not all content has an overlay covering it.
Screenshots

Desktop (please complete the following information):

Browser [chrome, brave]
UnicornUI Version [v0.0.39]

@TristanHoladay This will fix the other issues you are having with the Overlay. May have repercussions for #153 #96


[mypy] Argument "conn_max_age" to "parse" has incompatible type "None"; expected "int"  [arg-type]

mypy
Argument "conn_max_age" to "parse" has incompatible type "None"; expected "int"  [arg-type]
None value should be allowed for conn_max_age, proof:
https://docs.djangoproject.com/en/dev/ref/settings/#conn-max-age

Fair point we should support this correctly.


Fix for #11

fixes #11: must not fail on folder names containing dots

:+1:

Thanks for the quick response :+1:
<- feels a bit ashamed for not catching this before...


Duplicate issues between sbt and modules

I am thinking that we should keep GitHub issues open for both sbt/sbt and submodules open, and just label sbt/sbt one as "Duplicate," and close it only when the module with the fix merges to sbt/sbt.
The main motivation is so we can put the one on sbt/sbt into a milestone, and also so we know when things are fixed from sbt/sbt's perspective. What do you think?
Ref https://github.com/sbt/sbt/issues/3569, #3586

lol the use of the "Duplicate" label :)

I like that solution. I'd just suggest a label "upstream" (or similar) as "Duplicate" means duplicate within the same issue tracker. Also that way we can use it for other upstream issues.

I suggested Duplicate because it's an obvious signal that discussion related to be bug should not take place (unless of course we also label with "Needs discussion").

I think sbt issues blocked on upstream (which may be a better label than just upstream) could still have some discussion, such as: "Is there any way to workaround this JLine issue in sbt?".
I don't think "Duplicate" is the right label. Btw GitHub added some support for the concept of duplication semi recently: https://help.github.com/articles/about-duplicate-issues-and-pull-requests/.

ok. Made a new label for this.

We should also use Duplicate of incantation, if that gives links.


ADD CONTACT AND CONTRIBUTOR SECTION

Describe the bug
A clear and concise description of what the bug is: The README file is missing a Contact section and a Contributor section, making it difficult for users and contributors to get in touch or understand how to contribute to the project.
To Reproduce
Steps to reproduce the behavior:

Go to the repository's README file.
Scroll through the file to find the Contact and Contributor sections.
See error as these sections are missing.

Expected behavior
The README should include a Contact section to provide information on how users can reach out for support or feedback, and a Contributor section to guide potential contributors on how to get involved.
Screenshots
No screenshots applicable.
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 96.0.4664.45

Smartphone (please complete the following information):

Device: Samsung Galaxy S21
OS: Android 12
Browser: Chrome
Version: 96.0

Additional context
Adding these sections would enhance communication and collaboration within the project, making it easier for both users and contributors to engage.

ADD LABELS GSSOC EXT 24 AND HACKTOBERFEST
ASSIGN ME THE TASK @Git21221
THANK YOU!

@alo7lika you can add contact section but no contributors section, we are focusing the readme should be a proper small docs.

@alo7lika you can add contact section but no contributors section, we are focusing the readme should be a proper small docs.

OK BUT ADD LABELS GSSOC EXT 24 AND HACKTOBERFEST @Git21221 HERE ON THIS ISSUE


Python not found in all cases

Currently on Windows Hugo looks for Python like this:
https://github.com/gohugoio/hugo/blob/5509954c7e8b0ce8d5ea903b0ab639ea14b69acb/markup/internal/external.go#L43-L52
However for some Windows users, Python is available as python3.6m.exe or
similar. In those cases nothing is found and RST fails to render.
I can understand not wanting to mess with all possibilities here, so I think a
good solution would be to check config.toml, or environment variable in
addition to the current checks.

Workaround:
ln -s /bin/python3.6m.exe /bin/python.exe


[Suggestion] Allow RS Cables to be Waterlogged.

I suggest allowing cables to become waterlogged using the new 1.13+ water mechanics, purely for cosmetic reasons like running cables underwater, as currently they push water out of their space and make an unnatural bubble running along the seafloor.

Fixed


Add username to register.htm if loginAttribute is username.

Checks whether the login attribute is "username" and adds the required input field to register.htm.
Issue reference: #284

@JBonnyDev spaces not tabs for indentation please :)

@LukeTowers Hopefully up to spec now ^^

Awesome, thanks @JBonnyDev!


LRNs have different names in inn and R3

inn
cudnn R3

inn.SpatialCrossResponseNormalization
cudnn.SpatialCrossLRN

inn.SpatialSameResponseNormalization
cudnn.SpatialDivisiveNormalization

also cudnn.SpatialCrossLRN has different default parameters. Should we maybe use inn naming and defaults?
cc @fmassa

That sounds good to me. Sorry, did not notice in inn

Will do a PR


Instagram Platform Update

On the 17 Nov 2015 Instagram made changes to their API . Apps created before Nov 17, 2015 wont be affected until Jun 2016. Apps created on or after Nov 17 2015 will require to use their updated API. Please note that this library doesn't yet support their new updates.

When we launched our first Instagram API in 2011, we were a very small team and knew we wouldn’t be able to build all the features our community might want. Today we are announcing several platform changes to improve people’s control over their content and set up a more sustainable environment built around authentic experiences on the platform…
http://developers.instagram.com/post/133424514006/instagram-platform-update

I guess we need to rethink this entire library in order to work with the latest changes. This will require a lot of efforts and there is currently no plan to update this library.

If you're just looking for a simple way to fetch a public Instagram feed, you can checkout this package: vinkla/instagram.

Everything I do since getting sandboxed is met with OAuthAccessTokenException (even when it's stuff that I can do via the raw URL)
ie:
Got sandboxed users with valid posts that I can get a full result set from the following raw url:
https://api.instagram.com/v1/media/search?lat=49.27654081&lng=-123.09991836&access_token=".$account->token

But the following fails out with a OAuthAccessTokenException
$instagram->setAccessToken($account->token);
$instagram->searchMedia(49.27654081, -123.09991836)

Looks like they updated /media/search to require auth...
 public function searchMedia($lat, $lng, $distance = 1000, $minTimestamp = NULL, $maxTimestamp = NULL) {
    return $this->_makeCall('media/search', false, array('lat' => $lat, 'lng' => $lng, 'distance' => $distance, 'min_timestamp' => $minTimestamp, 'max_timestamp' => $maxTimestamp));
  }

needs the false $auth to be set to true
 public function searchMedia($lat, $lng, $distance = 1000, $minTimestamp = NULL, $maxTimestamp = NULL) {
    return $this->_makeCall('media/search', **true**, array('lat' => $lat, 'lng' => $lng, 'distance' => $distance, 'min_timestamp' => $minTimestamp, 'max_timestamp' => $maxTimestamp));
  }

Also need to update scopes
https://instagram.com/developer/authorization/
— basic - to read a user’s profile info and media (granted by default)
— public_content - to read any public profile info and media on a user’s behalf
— follower_list - to read the list of followers and followed-by users
— comments - to post and delete comments on a user’s behalf
— relationships - to follow and unfollow accounts on a user’s behalf
— likes - to like and unlike media on a user’s behalf

Hi,
now even the getTagMedia needs an access_token to work.
I have a big doubt: how can I have access to recent media without making the user to login?
I explain myself: I built a website with a backend area. In this area I add the authentication data for an Instagram app (Client ID and Client Secret) and a tag. The frontend shows the recent medias for that tag. Simple.
Now things are changed and an access_token is needed.
So how can I show the same recent medias without make the user login to Instragram? The app only shows the global recent medias, not user info.
I reviewed the repository code, but I cannot understand how THE APP could generate via code an access_token to be used for recent media query.
Thanks for your time.
Simone.

Is it possible to still use this API with the update apart from some of the features that require updating?

To fix photo uploading...

In Instagram::GenerateSignature() replace the key with:
3f0a7d75e094c7385e3dbaa026877f2e067cbd1a4dbcf3867748f6b26f257117

Replace the User Agent with:
Instagram 7.1.1 (Android; 21/5.0.2; 480dpi; 1080x1920; OnePlus/ONEPLUS; A0001; A0001; qcom; en_US)

Set ig_sig_key_version=6 to ig_sig_key_version=4.

@iadj Yes it is. I'm still currently using it to fetch hashtag based media but have lost access to user media due to the public_content scope addition.

the scopes problem is easily patched by editing line 74 to read:
private $_scopes = array('basic', 'public_content', 'follower_list', 'likes', 'comments', 'relationships');

I'm getting turned down couple of times for submission on basic and public_content, what actually I have to provide for the submission? I just want to curl trending Instagram photos into php
this is sample for now as I can only curl for my own media only
https://sanik.my/gallery/instagram
any insights?

Hello, did anyone get the permission review accepted after the update? I followed the valid use cases and i got the permissions declined everytime i tried it. Thanks

I'd give up on instagram if I were you.  They change the rules all the time, usually without warning.
On Mon, Feb 1, 2016 at 8:35 AM, Brumley<EMAIL_ADDRESS>wrote:

Hello, did anyone get the permission review accepted after the update? I
followed the valid use cases and i got the permissions declined everytime i
tried it. Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/cosenary/Instagram-PHP-API/issues/182#issuecomment-178025120
.

I've just submitted a pull request https://github.com/cosenary/Instagram-PHP-API/pull/204 that should fix this package to work with Instagram's API changes.
@vinkla please consider merging this in and thanks for your very helpful library!

Until this is fixed everyone could check out this repository https://github.com/hassankhan/instagram-sdk

Why don't you accept pull requests on this repo?
In #204 I fixed everything and as far as I can see it's perfectly usable again.

I'm not the maintainer. I've only helped out with the syntax changes.

Sorry, I thought you had merge rights as well.
Anybody knows if the author is still willing to maintain this project? If not, I'll switch out this package with my fork in my composer.json and be done with it.

@florianbeer Please do. Last time this git was updated was 4 months ago.

@florianbeer Could you publish your fork? And we will be able to enjoy this library with new API

Already did, a week ago: https://github.com/florianbeer/Instagram-PHP-API

Yes. But it is still "fork". It would be great if you changed name of library from "cosenary/instagram" to your. So we can use composer to upload it.

But that would be stealing, it is still @cosenary's work.
You can add the following to your composer.json to use my fork:
"repositories": [
    {
        "url": "https://github.com/florianbeer/Instagram-PHP-API.git",
        "type": "git"
    }
],

and then require "cosenary/instagram": "dev-master"

But if I use "dev-master" - it upload a last stable commit.
How can I use your last commits?

https://getcomposer.org/doc/05-repositories.md#loading-a-package-from-a-vcs-repository

Yes. It works now. But I have now

Invalid state

        || ($this->getInput('state') !== $this->store->get('oauth2state'))
    ) {
        $this->store->set('oauth2state', null);
        throw new CsrfException('Invalid state');
    }
    return;
}

hello guys, has anything changed with API permission review? Did anyone get the permission from instagram to any scope? Thanks

@florianbeer , this is an open-source project. If @cosenary is no longer maintaining it (and that seems to be the case given that he hasn't chimed in on any of this yet), creating a new non-forked repo that gives attribution to @cosenary seems to me to be a legitimate way to keep the project moving forward. For those of use who are trying to build production-level systems, it would be nice to be able to include a stable source in our composer setup. Thanks!

This package is still 99,9% @cosenary's work. If he wants me to take over and clearly states so, I will think about it, not sooner. If you want to use my fork in the meantime, you can alter your composer.json to fetch the code from my repository.

@florianbeer, with the repositories inside of composer.json and dev-master won't work. See https://github.com/composer/composer/issues/2622
Supposed that @cosenary are no longer interested in development on Github anymore.

I'm assuming this API is dead now that is hasnt been updated to the new Instagram API changes of last year? So its not worth utilizing at all?

@florianbeer if cosenary is not here to maintain a broken system, It must move on. No matter who created the work. It must move on and adapt. If it's from the original author or not. That is open source.

@florianbeer Do we have any update on this issue? If anyone want to take the responsibility of this repo, can we just create a fresh one without hurt anybody? It's really blocking for a lot of people.

Well I would be open to maybe transferring my fork to a GitHub organisation and publishing it to Packagist if I can get some co-maintainers. I don't think there's enough incentive for me to take care of issues, PRs etc. all on my own, seeing that my fork does everything I need it to do for my project for the foreseeable future.
So any volunteers to take on a (co-) maintainer role in an open source project and spending countless hours of unpaid work on keeping it up to date?

@florianbeer I want to volunteer, since I'm gonna also use the API for some prototyping, might as well explore the API depthly and help the public.

@florianbeer I'm gonna help as well if you are not have concerns. Currently I had to do a fork from your fork and created a Composer package on its basis: https://github.com/propeopleua/Instagram-PHP-API
But I do not like this!

@dimassrio I've added you as collaborator (push permission) on my repository and will publish it on packagist today.
Anybody else who wants to participate, please feel free to give me a shout!

@BR0kEN- great, you're added as well.

Hmm, seems I have to "un-fork" (is that the correct GitHub terminology), to publish to packagist. I will investigate further later today, as now I have to leave for a meeting.

@florianbeer cool! I can help, no problem!

@florianbeer, you can publish forks but this practice isn't recommended. So, better to create an independent package.

Hi @florianbeer, any news on this?

Lookin forward to use your fork @florianbeer

@florianbeer This again .. ;)
Actually, if you rename your forked branch as dev-master, we will be able to use your repository in composer. See http://stackoverflow.com/questions/13498519/how-to-require-a-fork-with-composer. I just tried by forked your branch and it worked.

It has been over a year now. I'm closing this.

Everyone who need this could use forked package: https://github.com/propeopleua/Instagram-PHP-API/blob/master/composer.json#L2

If you're just looking for a simple way to fetch a public Instagram feed, you can checkout this package: vinkla/instagram.

https://github.com/Andreyco/Instagram-for-PHP works and is supported


Electron Recovery Plan

@yosriady commented on Mon May 25 2020
TODO: Fix Electron

[x] Regenerate fresh electron react project comit-app-regen
[x] Post All previous fixes for Electron from old /app repo

[x] webpack postprocessing
[x] babel fixes
[x] pipelines
[x] renderer fixes
[x] main process fixes

[x] Install native dependencies until it fails to build

comit-sdk ***
bn.js
lodash
moment
satoshi-bitcoin-ts

[x] Fix CI issues

[x] Github Actions
[ ] Travis

[x] Debug latest run https://github.com/yosriady/comit-app-regen/actions/

Works with<EMAIL_ADDRESS>

[x] Electron package pipeline fixed.

TODO: comit-app-regen -> taker-ui

[x] Might be easier to just regenerate the whole TS/React app instead of pruning Electron from the existing app

[x] Bootstrap structure
[x] Dependencies
[x] components
[x] hooks
[x] pages

[x] Add routers to App.tsx
[x] Environment variables for app Use file-based electron-store and settings page instead
[x] Redesign

[x] Use chakra UI instead of rimble

Revert yarn add rimble-ui styled-components once chakra migration is completed

[x] Add sidebar

highlight active link

[x] settings page

[x] use https://github.com/electron-userland/electron-json-storage
[x] Create yarn command to populate settings store with ~/.create-comit-app/env contents

[x] order confirmation page
[x] swap details page

[x] Multi stage progress bar

[ ] Add providers to index
[ ] Upgrade to latest SDK version<EMAIL_ADDRESS>
[ ] There's a problem with SDK version 0.14.0, so the downgrade is temporary
[ ] Should upgrade back after resdesign and polish
[ ] InMemoryBitcoinWallet to BitcoindWallet

@yosriady commented on Tue Jun 09 2020
Electron Recovery Plan:

First, extract a pure React app from the existing Electron app
Then, generate a new Electron app with the latest libraries
Include all previous fixes and pipeline modifications from old project
Port the React app back in

Steps 2 and 3

[ ] ~~Regenerate Electron app ~~ Still fails https://github.com/yosriady/comit-app-regen/runs/753017498

[ ] OR if that still fails https://github.com/electron-userland/electron-builder
[ ] OR if that still fails https://www.electron.build/ and electron-webpack-quickstart

@yosriady commented on Tue Jun 09 2020
Error details: https://gist.github.com/yosriady/a8313b4a7f42ff5e5c5bc3adaa98026c
echo $(brew --prefix openssl)/include
<EMAIL_ADDRESS>
https://github.com/actions/virtual-environments/issues/837
OpenSSL 1.0.2t 10 Sep 2019 (/usr/local/opt/openssl -> /usr/local/Cellar/openssl/1.0.2t)

@yosriady commented on Wed Jun 10 2020

[ ] Try Travis instead of Github Actions https://electron-react-boilerplate.js.org/docs/ci

Check https://travis-ci.com/github/yosriady/comit-app-regen
Same error Fatal error: 'openssl/aes.h' file not found https://travis-ci.com/github/yosriady/comit-app-regen/jobs/346519135

@yosriady commented on Wed Jun 10 2020

[ ] CI should use https://github.com/marketplace/actions/upload-files-to-a-github-release

[ ] Debug Error: No published versions when running packaged app https://github.com/electron-userland/electron-builder/blob/f938de5aae09cb832a6f7f742e64a0d6c025fb9f/packages/electron-updater/src/providers/GitHubProvider.ts#L69

[ ] Package app screen is blank, Debugtron does not display any errors

Debugging plan A:

[ ] Do a git bisect and yarn package at each checkpoint in this list https://github.com/comit-network/taker-ui/actions
[ ] Identify the offending commit

Debugging plan B:

[ ] Check if frameless is causing it

Debugging plan C:

[ ] Check if frameless is causing it

[ ] Compare webpack.config.renderer.dev vs webpack.config.renderer.prod

This hack / temporary fix helps: https://github.com/electron-react-boilerplate/electron-react-boilerplate/issues/2395#issuecomment-593474211

[ ] Follow up on https://github.com/electron-react-boilerplate/electron-react-boilerplate/issues/2485

https://github.com/electron-react-boilerplate/electron-react-boilerplate/pull/2465/files

Fixed via electron-react-boilerplate/electron-react-boilerplate#2395 (comment)


Simplified Whitspace handling in the Pegjs Source Grammar

The Peg Source Grammar suffers from tons of __ rules scattered throughout. This pull request changes it so that it runs the __ before all tokens. This greatly simplifies the grammars readability.
There may also be a slight performance boost as there were some unnecessary __ rules in the original grammar.

@kwesibrunee Thanks for your contribution, but I won’t merge it for two reasons:

I consider putting __ before/after every token slightly hacky and putting it explicitly in the rules “only” somewhat annoying. Admittedly, this is an issue of taste.
More importantly, parsing whitespace as part of tokens negatively influences location information (and thus also error reporting). For example, you modify the AnyMatcher rule like this:

```
AnyMatcher
  = __ "." { return { type: "any", location: location() }; }
```

 This (together with your other changes) means in grammars like `start = /* foo */ .`, the AST node representing `.` will contain location info indicating it spans from right after the `=` character to the end of input. But this is not what is desired,  the span should cover just the `.` character. For `AnyMatcher` this doesn’t really matter (I chose that example just for simplicity), but in other places it does.


Week 1 handout and lecture

This pull request contains the handout and lecture associated with week 1 of the course.
I think that the demos should involve live coding from us so I am not including pre-written stuff for that; but I will add to the desirables project to have a text document that we can have printed out with the things that we will be live coding (forever in the first instance this can take the back seat)

Pinging issue #1 which this is in response to

Looks good. Points for discussion:

Colour scheme: white (grey) text on a black background looks nice, but makes it complicated to make figures with sensible contrast. All my talks from the last 10 years are black text on white, and I have a bunch of vector graphic figures designed for this; some of which I expect to be useful for these lectures.
No actual examples in the keynote slides. Is the idea that we have the keynote presentation and a live coding notebook open at the same time, and manually switch between these?
Discussion of "planning an algorithm". This is mentioned in the final exercise, but it's not clear where (if) we introduce this idea earlier. Thoughts?

What is the name of the font in the BATMAN slides?

What is the name of the font in the BATMAN slides?

Titles are Gill Sans. Body text is Myriad.

Edited the slides so the the text body is black and the backgrounds are white
Personally, like the idea of swapping between a live coding notebook and the slides (first explain with the slide, then show with the live coding notebook) (see #15 in the additional stuff project)
Added a slide before the exercise talking about the spec to algorithm to code thinking (although could benefit from revision

This is now reordered and ready for @bjmorgan to review

Style question: do we want grammatically correct sentences in the slides (and punctutation)?

"First & Second Year Python" slide: not all the students will have done the 1st and 2nd year courses (and those that have will probably have forgotten everything). I think it's important to emphasises that we will go over again everything in these courses in the first few weeks, and also that Nat Sci students / anyone who didn't do the previous comp labs will be fine.

Jupyter Notebooks page:

Delete "As with the first and second year labs". Those students that have done it before will remember. Those that didn't don't need to know.
Do we have a "getting started with Jupyter notebooks" resource? Some students will be new. Others will be very rusty.
Running code. There is no way we can run an entire year on the comp chem jupyter server. We are going to use local Anaconda (which hopefully will point at the students' H: drives → I know this needed to be fixed for the teaching lab PCs. We need to check that this fix is also applied to the PCs we will be using).

Do we want to "demo" launching Anaconda / opening Jupyter and navigating around a notebook / running commands / saving files etc?

Two typos on your "Logical Operators" slide.

Suggestion that we use a different colour for code, instead of just a different font?

Style question: do we want grammatically correct sentences in the slides (and punctutation)?

Up to you to be honest. I feel that the slides don't are just a direction to go alongside the handout.

As "educators" it doesn't hurt to have standards. It also makes it easier to be consistent if we make the decision once.

As "educators" it doesn't hurt to have standards. It also makes it easier to be consistent if we make the decision once.

I will leave the decision on slide formatting to you then! (mainly cause my grammar is terrible so would likely be incorrect)

"First & Second Year Python" slide: not all the students will have done the 1st and 2nd year courses (and those that have will probably have forgotten everything). I think it's important to emphasises that we will go over again everything in these courses in the first few weeks, and also that Nat Sci students / anyone who didn't do the previous comp labs will be fine.

[x] in d6e6d67

Jupyter Notebooks page:

Delete "As with the first and second year labs". Those students that have done it before will remember. Those that didn't don't need to know.
Do we have a "getting started with Jupyter notebooks" resource? Some students will be new. Others will be very rusty.
Running code. There is no way we can run an entire year on the comp chem jupyter server. We are going to use local Anaconda (which hopefully will point at the students' H: drives → I know this needed to be fixed for the teaching lab PCs. We need to check that this fix is also applied to the PCs we will be using).
The jupyter hub server can be listed as part of "How to Jupyter when you are not in the lab" section.

[x] d90b996, except from the getting started with Jupyter Notebooks resource (we could give a link to your first year python?)

Two typos on your "Logical Operators" slide.

[x] in d11a75e

Suggestion that we use a different colour for code, instead of just a different font?

Any suggestion for a colour (the blue used in the titles isn't great contrast on the light `code' font), I am not great with colours (hence sticking to black and blue)

Maybe leave it as black and just us a monospaced font?

I think that is what I am doing currently (using Andale mono currently for the `code' font)

Propose that you merge all your materials from your fork into the main repo
so that we have a “current complete(?) draft” all in one place where we can
both work on it? Unless there is a reason to have each week in a separate
branch on a separate fork?
On 24 September 2019 at 08:27:53, Andrew McCluskey<EMAIL_ADDRESS>wrote:
I think that is what I am doing currently (using Andale mono currently for
the `code' font)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/pythoninchemistry/ch40208/pull/14?email_source=notifications&email_token=AAJAZS4WQJQBPA5CDOQIJY3QLG6PRA5CNFSM4H6KDC5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7NL3SQ#issuecomment-534429130,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJAZSYK7FT65WJEVTYCXQ3QLG6PRANCNFSM4H6KDC5A
.

I have reviewed the handout and have a couple of comments:

Should we include a "what is this course for / overview" at the beginning of the handout and in the lecture content? At the moment the handout reads like the students are doing a programming course, rather than a programming + comp chem methods course. I am happy to add some stuff at the beginning (or to another document) talking about the overall course outline and how we see it fitting together (including, e.g. why we are teaching comp chem through programming exercises rather than a traditional series of lectures).
I noticed in your exercises you talk about having to check different ASCII codes for upper and lower case inputs. I presume this is a left-over piece of (now useless) advice from the Fortran version?


Force Retry Delay

With workday outages, we noticed that if retryCount is above 1 then it didn't wait until workday outage was over.  Adding a new variable to accomplish this workflow

I'm working on actions to actually build and publish this project ... hopefully I can get it finished today or tomorrow as @mez5001 has changes as well.  I think this might be happening in FPS as well (the retries seem to be immediate).


Show value on pie chart

Hello! Can i show the value on pie chart? I mean i want to show value 22 and 23 on chart besides percentage:
chart.addSeries("A", 22)
chart.addSeries("B", 23)

Yes, Please see the example charts in xchart-demo.


Proactive messaging doesn't work with emulator

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
Gabo's fork
Describe the bug
Credential validation fails with emulator. It checks to see if the appId is the parent bot's appId instead of being on the authorized list of app IDs. This check happens in the EmulatorValidator.cs.
To Reproduce
Steps to reproduce the behavior:

Spin up the DialogRootBot and the ProactiveSkill bot.
Send the parent bot a message and connect to the skill
Click the link the skill provides to do proactive

Expected behavior
The flow should complete
Screenshots

Additional context
This also breaks when I tested with directline.
Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker: Information: Route matched with {action = "ReplyToActivity", controller = "Skill"}. Executing controller action with signature System.Threading.Tasks.Task`1[Microsoft.AspNetCore.Mvc.IActionResult] ReplyToActivityAsync(System.String, System.String, Microsoft.Bot.Schema.Activity) on controller Microsoft.BotBuilderSamples.DialogRootBot.Controllers.SkillController (Microsoft.BotBuilderSamples.DialogRootBot).
Exception thrown: 'System.UnauthorizedAccessException' in Microsoft.Bot.Connector.dll
Exception thrown: 'System.UnauthorizedAccessException' in System.Private.CoreLib.dll

Closing this one with workaround.
The proactive logic in this project shows what info needs to be stored for a skill to get proactive messages to work (I will review later with @EricDahlvang to see if there's a better way of doing this)


Cherry-pick sst_importer: download SST from external storage + key rewrite (#5581) to release-3.1

What have you changed?
Cherry-pick #5581 to release-3.1.
What is the type of the changes?
Pick one of the following and delete the others:

New feature (a change which adds functionality)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No?
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
#5581
Benchmark result if necessary (optional)
Any examples? (optional)

/run-all-tests

/run-integration-copr-test

/run-integration-copr-test

/run-integration-copr-test

/run-integration-tests

/run-integration-copr-test

/run-integration-copr-test tikv=release-3.1

@kennytm you should ignore copr-test. it not mettar

@zhouqiang-cl OK. Will it be fixed 😂


dicttoh5: properly define update/overwrite behavior

There are several inconsistencies in the overwrite behavior of dicttoh5. For example this one:
from pprint import pprint
from silx.io import dictdump

ddict = {"group1": {"data1": 1}, "group1@attr": "attr"}
dictdump.dicttoh5(ddict, "test.h5", overwrite_data=True, mode="w")
pprint(dictdump.h5todict("test.h5"))

ddict = {"group1": {"data2": 2}}
dictdump.dicttoh5(ddict, "test.h5", overwrite_data=True, mode="a")
pprint(dictdump.h5todict("test.h5"))  # group1 has attr, data1 and data2

ddict = {"group1": {}}
dictdump.dicttoh5(ddict, "test.h5", overwrite_data=True, mode="a")
pprint(dictdump.h5todict("test.h5"))  # group1 has only attr

These inconsistencies need to be solved and the overwrite_data behavior needs to be clearly defined. In addition I need a way to "update" an existing HDF5 file.
For both issues I propose these three dump modes:

overwrite (existing things may be overwritten): the HDF5 tree is exactly the dict tree
update (existing things may be overwritten): the dict tree is a subtree of the HDF5 tree
add (existing things are untouched): nodes from the dict tree are added to the HDF5 tree when the existing structure allows it

As for the existing overwrite_data: unfortunately it behaves sometimes like overwrite and sometimes like update. As this behavior is confusing I don't think we need to preserve it.
@vasole @t20100 @vallsv Comments?

No problem with me.

If I understand well, the inconsistency is the difference between empty groups in the dict which are overwritten while groups that are not empty are updated.
From the doc and the overwrite_data arg name, I would expect that it updates the structure.
I would consider this as a bug and have no problem changing the behavior to update even for empty groups.
Else, your modes sounds good to me.

Correct. There are other inconsistencies which I didn't show (e.g. nodes that change type). These things are a consequence of the fact that overwrite_data is not well defined (it also lacks tests).
I'll make a PR with what I suggested. I guess I'll deprecate the overwrite_data argument and have True work like update and False like add. Then I'll add a new argument existing=None with "add" as default (the other options are "overwrite" and "update").


fix: moving to graphql-cli instead of codegen

Very small change to templates to use graphql-cli instead of codegen (legacy).
Removed codegen files and added graphql-cli dependency.
Dependencies will be swapped at runtime but only for root (@machi1990 client one has no dependency swapped. Orignal template was yarn workspace. Here we use two separate projects so that is why installation on root is not working. However, if we drop dependency from the client we will be able to use the root one. @machi1990 WDYT? We also need better documentation showing how to regenerate typings on server.
Let's make sure that the templates branch is updated after the merge.
Verification
Go to any template and:

yarn codegen
cd client yarn codegen

You should see successful execution and no changes.

@machi1990 left some open question on this. Happy to help you with review etc.

Very small change to templates to use graphql-cli instead of codegen (legacy).
Removed codegen files and added graphql-cli dependency.

+1

Dependencies will be swapped at runtime but only for root (@machi1990 client one has no dependency swapped. Orignal template was yarn workspace. Here we use two separate projects so that is why installation on root is not working. However, if we drop dependency from the client we will be able to use the root one. @machi1990 WDYT?

+1 this is the correct observation of what was happening. This should fix it for the client side project too.

We also need better documentation showing how to regenerate typings on server.

+1. I think it will be same as the one we have on client side. I can add a commit in this PR if that's okay with you.

Let's make sure that the templates branch is updated after the merge.

+100.

Verification
Go to any template and:

yarn codegen
cd client yarn codegen

@machi1990 feel free to make small calibrations

@machi1990 feel free to make changes that are necessary

Done https://github.com/aerogear/graphback/pull/1988/commits/3b341cd326db040d5bb729542a3546a9c3bfc3ea. I've also fixed the codegen command.
From my side all is good. @craicoverflow can you also have a look? If it's okay with you too, we can merge this and update the templates branch.

@craicoverflow @wtrocki  regarding https://github.com/aerogear/graphback/pull/1988#discussion_r483039530, the idea was to consume everything using @graphql-cli.

@craicoverflow I've added https://github.com/aerogear/graphback/pull/1988/commits/e3ace03bd24bb7291f754d67c2e081899cf3b1db to avoid duplication as suggested in https://github.com/aerogear/graphback/pull/1988#discussion_r483040329.


Customize text label even more

I think if we had a color for text for each state would be better, like completed, current, un-completed
I'm referring to app:textColorStatus

Duplicate of https://github.com/params-ing/StepViewAndroid/issues/1


Add string representations for TK_FORMALVALUE* tokens.

This PR adds string representations for the TK_FORMALVALUEARG and TK_FORMALVALUEPARAM tokens, which were previously missing.

CI failure is unrelated.

@jemc i sent the failure to @sylvanc. He's busy during the day tomorrow, so I might take a look at it if I have time tomorrow.

The CI error has to do with Travis infrastructure, not this PR.

The CI error has to do with Travis infrastructure, not this PR.

The CI error has to do with Travis infrastructure, not this PR.

The CI error has to do with Travis infrastructure, not this PR.


The URL mentioned in default.nix when Nix is too old is broken

https://github.com/NixOS/nixpkgs/blob/master/default.nix#L5 mentions https://nixos.org/wiki/How_to_update_when_Nix_is_too_old_to_evaluate_Nixpkgs, which points people to Hydra URLs that no longer work.
I remember a thread about improving that mechanism, but even if we don't do something fancy, we should have the URLs actually work. Now that the wiki is read-only, I'm not sure how best to fix it.

I think we should just move it to the manuals.

This is really important, I'll put it on my high-pri todo.

Latest release .nixpkgs are 404-ing at https://hydra.nixos.org/release/nix/nix-1.11.2
cc @edolstra

We had an issue already: https://github.com/NixOS/nixpkgs/issues/13186.

And I suppose Nix manual is the best place.

Releases are now at http://nixos.org/releases/nix/, but there is no .nixpkg

What is being done to provide .nixpkg files? What happened to them in the first place? Is it some kind of joke that Hydra still has links for these files?
Being unable to update old Nix-based systems quickly is completely unacceptable and reflects poor attention to infrastructure.

@ttuegel I'll dig into this tomorrow.

Btw for now best workaround is:
nix-env -i /nix/store/356dj8w9171dajl0wq32m41dhg2ngq35-nix-1.11.2

Thank you! I have to say, I'm quite surprised that worked. One possible resolution would be to update the manual to include instructions for finding the current hash from Hydra and following this procedure. The disadvantage of the curl-based procedure is that its failures are... opaque.

Actually, that didn't work; now it fails with error: unknown hash algorithm 'sha512'. I have also tried nix-1.12 with no success.

@ttuegel did you use the same user? It is installed into your personal profile.

@ttuegel this is not NixOS right?

@domenkozar This is NixOS. It is installed into the system profile and nix-env --version reports version 1.12.

@ttuegel 1.12? That means you're using unstable Nix. Could it be that sha512 support is just broken?

@domenkozar I don't think so. I've tried with 1.11 and 1.12 on this machine. The workaround is to disable Firefox from my configuration; now it seems to be working. But, I use the same configuration on a different machine without any problems, so it's more subtle than sha512 simply being broken...

I think your nix-daemon also needs to be new enough.

I got error message on old system with Nix, went to wiki page, tried it's commands, they failed, installed Nix from scratch, then found this thread. This experience is really confusing for me as a user, please do something with misleading message.

@maximvl did following help:
nix-env -i /nix/store/356dj8w9171dajl0wq32m41dhg2ngq35-nix-1.11.2

I'm sorry it took so long, but see https://github.com/NixOS/nixpkgs/pull/17631

Can this be closed? also for issue #23013.

Fixed in 2e91b2d585afdb3b076b3ad655579774f100bad5.


Fix medical.testBloodGroup

The PR fixes failing test at https://github.com/datafaker-net/datafaker/runs/6641995024?check_suite_focus=true

Codecov Report

Merging #189 (903a633) into master (86f98e9) will decrease coverage by 0.79%.
The diff coverage is 6.66%.

@@             Coverage Diff              @@
##             master     #189      +/-   ##
============================================
- Coverage     94.52%   93.72%   -0.80%
+ Complexity     1714     1708       -6
============================================
  Files           175      176       +1
  Lines          3651     3665      +14
  Branches        366      366
============================================
- Hits           3451     3435      -16
- Misses          105      135      +30
  Partials         95       95

Impacted Files
Coverage Δ

src/main/java/net/datafaker/Cannabis.java
0.00% <0.00%> (ø)

src/main/java/net/datafaker/Faker.java
98.96% <0.00%> (-0.52%)
:arrow_down:

src/main/java/net/datafaker/Name.java
100.00% <ø> (ø)

.../main/java/net/datafaker/service/files/EnFile.java
93.33% <ø> (ø)

src/main/java/net/datafaker/Medical.java
100.00% <100.00%> (ø)

.../main/java/net/datafaker/idnumbers/EnIdNumber.java
54.54% <0.00%> (-45.46%)
:arrow_down:

src/main/java/net/datafaker/service/FakerIDN.java
63.63% <0.00%> (-9.10%)
:arrow_down:

src/main/java/net/datafaker/Commerce.java
88.46% <0.00%> (-7.70%)
:arrow_down:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

.../java/net/datafaker/service/FakeValuesService.java
85.61% <0.00%> (-1.42%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Oooooppps. Thanks for fixing!


Update load_image function to handle BytesIO input

The load_image function in mlx_vlm/utils.py is now updated to accept input as BytesIO representing base64 encoded images. This enhances the loading process by accommodating another type of input. Appropriate error handling has been added for potential IO errors.

Hey @gabewillen
Thanks for the great addition!
Before I merge, could you run pre-commit?

Hey @gabewillen
Thanks for the great addition!
Before I merge, could you run pre-commit to clear the tests?

Absolutely I got ahead of myself on the PR. Apologies I was in the process of following your CONTRIBUTION.md before you commented. It should be good to go now.

Absolutely I got ahead of myself on the PR. Apologies I was in the process of following your CONTRIBUTION.md before you commented. It should be good to go now.

Any other changes you want to make before I merge ?

Merged 👌🏽
Thank you for the contribution @gabewillen!

Absolutely I got ahead of myself on the PR. Apologies I was in the process of following your CONTRIBUTION.md before you commented. It should be good to go now.

Any other changes you want to make before I merge ?

Not right now but I plan to use this library for a fun project with my kids. Any additions I make I'll be glad to contribute. Excellent work on the library!

It will be available on the next release.

@gabewillen what’s your Twitter handle ?

@gabewillen what’s your Twitter handle ?

I don't have a personal one but I do have one for a project I'm working on https://x.com/StateforwardOrg

I also have a tool I'm getting ready to release that builds dependency graphs of libraries and autogenerates mkdoc compatible documentation if you'd be interested I can run it on your project. It works really well.

It injects the documentation without modifying the source code


Audio hiss after track ends

I don't know if this is a Mopidy issue or a spotmop issue but after a playlist has finished the audio path (?) is still left open and there's a constant hiss.

It does sound like a Mopidy core issue as Spotmop has nothing to do with audio output. To confirm this, you could try using another interface (I use mpc command line for testing).


[MINOR] Fix wrong logic for checking state condition

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Fix bug with logic error for checking state condition.
Brief change log

Modify StreamerUtil#checkRequiredProperties checkState logic.

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
This change added tests and can be verified as follows:

Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@yanghua double check?

@ZhangChaoming Please check Travis's exception.

@yanghua This exception may be possibly caused by Travis, because my code is clean.

@yanghua This exception may be possibly caused by Travis, because my code is clean.

you can push an empty commit to trigger the ci

@wangxianghu @yanghua  It's so strange that the ci failed on the job '8392.1 Unit tests for hudi-spark-client'.

@wangxianghu help to check?

@wangxianghu help to check?

This seems another concurrent issue
[ERROR] Errors:
[ERROR]   TestMultiFS.setUp:60->HoodieClientTestHarness.initDFS:251 » NullPointer

  /**
   * Initializes a distributed file system and base directory.
   *
   * @throws IOException
   */
  protected void initDFS() throws IOException {
    FileSystem.closeAll();
    hdfsTestService = new HdfsTestService();
    dfsCluster = hdfsTestService.start(true);  // line 251

    // Create a temp folder as the base path
    dfs = dfsCluster.getFileSystem();
    dfsBasePath = dfs.getWorkingDirectory().toString();
    this.basePath = dfsBasePath;
    this.hadoopConf = dfs.getConf();
    dfs.mkdirs(new Path(dfsBasePath));
  }

@wangxianghu @yanghua It's so strange that the ci failed on the job '8392.1 Unit tests for hudi-spark-client'.

trigger it again.
This should be an env issue

@wangxianghu It failed again. T_T

IMO, this PR is safety. Merging it.


2 ➡ 5

➡ Forward port
Port ign-physics2 to ign-physics
Branch comparison: https://github.com/gazebosim/gz-physics/compare/ign-physics5...ign-physics2
Note to maintainers: Remember to Merge with commit (not squash-merge or rebase)

Windows has 4 new warnings https://build.osrfoundation.org/job/ign_physics-pr-win/2159/msbuild/

The windows warnings are already in the stable version https://build.osrfoundation.org/view/ign-fortress/job/ign_physics-ign-5-win/32/. It appears that they started in #466, but that PR didn't modify anything in bullet.

Looking at the previous builds that had no windows warnings, CMake was not finding bullet in those builds
[16.688s] -- Could NOT find Bullet (missing: BULLET_DYNAMICS_LIBRARY BULLET_COLLISION_LIBRARY BULLET_MATH_LIBRARY BULLET_SOFTBODY_LIBRARY BULLET_INCLUDE_DIR)
[16.688s] -- Looking for IgnBullet - not found

But the build with the warning does find bullet:
[22.359s] -- Found Bullet: C:/vcpkg/installed/x64-windows/lib/BulletDynamics.lib
[22.359s] -- Looking for IgnBullet - found

Something must have changed in gz-cmake or vcpkg recently.

the ign->gz changes are always tricky; I'd like to review before merging if possible

some header files that were not present on ign-physics2 do not have a complete set of ign->gz changes:

Ah, I should have mentioned those. My intention was to minimize extra changes during the merge and fix the additional ign->gz changes in a follow up PR. If you'd like to fix them in this PR, I can.

ok, that's fine, as long as we don't forget about them


Issue and Pull Request templates

One thing to consider adding are templates for pull requests and issues.  Might take a while to come up with what should be in these, but they are helpful to contributors to know what info and steps you need as maintainers to make progress on an issue:
https://github.com/blog/2111-issue-and-pull-request-templates

@jklymak we will do this. We are also figuring out how to run and provide feedback from a few
tests on PRs that are important, but can't easily run through Travis etc...


rake db:mongoid:create_indexes not working for rails engine components

Hi,
I am using a rails engine as a component of my rails application. When running the task rake db:mongoid:create_indexes from the main app it doesn't load the inner engine mongoid models.
Currently I am overriding the db:mongoid:load_method rake task and manually load the models I need.
Ideally I should be able to run:  Rails::Mongoid.load_models(DomainLogic::Engine) in this overridden load_method task.
But there is an issue in lib/rails/mongoid.rb load_models method. When running app.config.paths['app/models'].each it is returning a local path 'app/model'.
If instead you run app.config.paths["app/models"].to_a.each do |path| the returned path would be a full path and would allow to load mongoid files in engines.
What's your take on this? Am I missing some other workaround for this?
Basically I'd like to rely on mongoid index system for my engines models.

You saved me a ton of time figuring out how to properly access model paths from the engine. Thank you
It's possible to redefine the load_models task to manually require the models from your engine. I simply copied the source from Rails::Mongoid#load_models into my app.
# in your application
# lib/tasks/load_models.rake
namespace :db do
  namespace :mongoid do

    # include models from engine
    task :load_models do
      YourEngineName::Engine.config.paths['app/models'].to_a.each do |path|
        preload = ::Mongoid.preload_models
        if preload.resizable?
          files = preload.map { |model| "#{path}/#{model}.rb" }
        else
          files = Dir.glob("#{path}/**/*.rb")
        end

        files.sort.each do |file|
          require_dependency(file)
        end
      end
    end

  end
end

Can you double check if this is still a problem on latest Mongoid?
thanks

I believe so. This is a quick test I ran in rails console to check.
Loading development environment (Rails 4.1.10)
irb(main):001:0> Rails::Mongoid.load_models(AppModels::Engine)
=> ["app/models"]
irb(main):002:0> Mongoid.models
=> []
irb(main):003:0> AppModels::Engine.config.paths['app/models'].to_a.each do |path|
irb(main):004:1*   # truncated...
irb(main):014:1> end
=> ["/Users/baylorrae/Work/app/app_models/app/models"]
irb(main):015:0> Mongoid.models
=> [AppModels::Card, AppModels::DeviceToken, AppModels::User, AppModels::UserCard]
irb(main):016:0> Mongoid::VERSION
=> "4.0.2"

I confirm having the same issue. Version 4.0.2.

Working fin in 5 beta. To be clear to some coming thru:
Create on your app (not the engine) a lib/tasks/load_model.rake
namespace :db do
  namespace :mongoid do

    # include models from engine
    task :load_models do
        Rails::Mongoid.load_models(SomeNice::Engine)
    end
  end
end


Update for <IP_ADDRESS>6217

Just a quick fix to work with <IP_ADDRESS>6217 (latest beta)
DecideAgentKnockedByBlow threw an exception for an unknown function override;
CalculateDailyFoodConsumptionf does not have the includeDescription parameter anymore;
I'm currently testing it, but at the very least it loads in now ;)

Probably:
closes #103

Superceded by #109


📃: Random Quote Generator

🔴 Title : Random Quote Generator
🔴 Tech stack : HTML, CSS, JS
🔴 Objective : Generates Random Quotes and have the features to copy, share on twitter and also text to speech.
🔴 Summary :
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name : Anantesh G
Participant Role : GSSoC, Hacktoberfest

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

you can assign this task to me..

you can assign this task to me..

This is a PR to add my Code Editor project. Please create your own issue and make a PR if you have a project to be added here.

already present in project


make shell false only in Linux #157

Hello @amoh-godwin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pyffmpeg/__init__.py:

Line 87:80: E501 line too long (80 > 79 characters)


Add ability to create paintable objects at runtime

It would be great if we could add ink canvas to objects at runtime with addcomponent
thanks!

Thank you for your report :)


[firebase_messaging]: Error while generating FCM Token: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Messaging
Which platforms are affected?
Web
Description
FCM Token is not registered on first launch of app, and refreshing the page able to get token.
Need to generated FCM Token on the first launch of the app itself. Its works on android, but error is in web app only
Reproducing the issue
firbase-messaging-sw.js
importScripts("https://www.gstatic.com/firebasejs/10.8.1/firebase-app-compat.js");
importScripts("https://www.gstatic.com/firebasejs/10.8.1/firebase-messaging-compat.js");

firebase.initializeApp({
     apiKey: "AIzaSyCkfdY7*****",
     authDomain: "*****firebaseapp.com",
     projectId: "*****1e46e",
     storageBucket: "*****appspot.com",
     messagingSenderId: "*****",
     appId: "*****fa3ef478c67a6dfc",
     measurementId: "*****BRQ"
});

const messaging = firebase.messaging();

main.dart
Future<void> main() async {

 WidgetsFlutterBinding.ensureInitialized();

  await Firebase.initializeApp(options: DefaultFirebaseOptions.currentPlatform,);

  FirebaseMessaging messaging = FirebaseMessaging.instance;

  NotificationSettings settings = await messaging.requestPermission(
    alert: true,
    announcement: false,
    badge: true,
    carPlay: false,
    criticalAlert: false,
    provisional: false,
    sound: true,
  );

  if (kDebugMode) {
    print('User granted permission: ${settings.authorizationStatus}');
  }

  try {
    String? fcmToken1 = await messaging.getToken(
        vapidKey: kIsWeb ? "BKFKHhWelaWv7**********" : null
    );
    print(fcmToken1);

  } catch (e) {
    // Handle any exceptions that may occur during token generation
    print("Error while generating FCM Token: $e");
  }
}

index.html
<script>
  window.addEventListener('load', function(ev) {
    // Download main.dart.js
    _flutter.loader.loadEntrypoint({
      serviceWorker: {
        serviceWorkerVersion: serviceWorkerVersion,
        firebaseMessagingServiceWorker: 'firebase-messaging-sw.js',
      },
      onEntrypointLoaded: function(engineInitializer) {
        engineInitializer.initializeEngine().then(function(appRunner) {
          appRunner.runApp();
        });
      }
    });
  });
</script>

Firebase Core version
2.28.0
Flutter Version
3.19.5
Relevant Log Output
Error while generating FCM Token: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker

Flutter dependencies

Expand Flutter dependencies snippet

Dart SDK 3.3.3
Flutter SDK 3.19.5
medcouncil 1.0.13+13

dependencies:
- animated_text_kit 4.2.2 [flutter characters]
- carousel_slider 4.2.1 [flutter]
- connectivity_plus 5.0.2 [flutter flutter_web_plugins connectivity_plus_platform_interface js meta nm]
- crypto 3.0.3 [typed_data]
- cupertino_icons 1.0.6
- dropdown_button2 2.3.9 [flutter meta]
- firebase_analytics 10.10.0 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_core 2.28.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_messaging 14.8.0 [firebase_core firebase_core_platform_interface firebase_messaging_platform_interface firebase_messaging_web flutter meta]
- firebase_remote_config 4.4.0 [firebase_core firebase_core_platform_interface firebase_remote_config_platform_interface firebase_remote_config_web flutter]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]
- flutter_linkify 6.0.0 [flutter linkify]
- flutter_local_notifications 17.0.0 [clock flutter flutter_local_notifications_linux flutter_local_notifications_platform_interface timezone]
- flutter_native_splash 2.4.0 [args flutter flutter_web_plugins html image meta path universal_io xml yaml ansicolor]
- flutter_pdfview 1.3.2 [flutter]
- fluttericon 2.0.0 [flutter]
- get 4.6.6 [flutter]
- google_fonts 6.2.1 [flutter http path_provider crypto]
- google_nav_bar 5.0.6 [flutter]
- http 1.2.1 [async http_parser meta web]
- in_app_purchase 3.1.13 [flutter in_app_purchase_android in_app_purchase_platform_interface in_app_purchase_storekit]
- internet_connection_checker 1.0.0+1
- multi_select_flutter 4.1.3 [flutter collection]
- package_info_plus 4.2.0 [ffi flutter flutter_web_plugins http meta path package_info_plus_platform_interface win32]
- path 1.9.0
- path_provider 2.1.2 [flutter path_provider_android path_provider_foundation path_provider_linux path_provider_platform_interface path_provider_windows]
- razorpay_web 1.4.2 [flutter flutter_web_plugins eventify package_info_plus universal_platform]
- restart_app 1.2.1 [flutter plugin_platform_interface flutter_web_plugins]
- shared_preferences 2.2.2 [flutter shared_preferences_android shared_preferences_foundation shared_preferences_linux shared_preferences_platform_interface shar
ed_preferences_web shared_preferences_windows]
- sqflite 2.3.3 [flutter sqflite_common path]
- sqflite_common_ffi_web 0.4.3 [path sqflite_common_ffi sqflite_common sqlite3 http web synchronized process_run dev_build pub_semver args meta]
- upgrader 10.0.1 [flutter device_info_plus html http os_detect package_info_plus shared_preferences url_launcher version xml]
- url_launcher 6.2.5 [flutter url_launcher_android url_launcher_ios url_launcher_linux url_launcher_macos url_launcher_platform_interface url_launcher_web url_l
auncher_windows]
- vimeo_player_flutter 0.1.0 [flutter webview_flutter]

dev dependencies:
- flutter_lints 3.0.1 [lints]
- flutter_test 0.0.0 [flutter test_api matcher path fake_async clock stack_trace vector_math leak_tracker_flutter_testing async boolean_selector characters coll
ection leak_tracker leak_tracker_testing material_color_utilities meta source_span stream_channel string_scanner term_glyph vm_service]

transitive dependencies:
- _flutterfire_internals 1.3.28 [collection firebase_core firebase_core_platform_interface flutter meta]
- ansicolor 2.0.2
- archive 3.4.10 [crypto path pointycastle]
- args 2.4.2
- async 2.11.0 [collection meta]
- boolean_selector 2.1.1 [source_span string_scanner]
- characters 1.3.0
- clock 1.1.1
- collection 1.18.0
- connectivity_plus_platform_interface 1.2.4 [flutter meta plugin_platform_interface]
- convert 3.1.1 [typed_data]
- csslib 1.0.0 [source_span]
- dbus 0.7.10 [args ffi meta xml]
- dev_build 0.16.4+3 [collection process_run pub_semver yaml path meta pool args]
- device_info_plus 9.1.2 [device_info_plus_platform_interface ffi file flutter flutter_web_plugins meta win32 win32_registry]
- device_info_plus_platform_interface 7.0.0 [flutter meta plugin_platform_interface]
- eventify 1.0.1
- fake_async 1.3.1 [clock collection]
- ffi 2.1.2
- file 7.0.0 [meta path]
- firebase_analytics_platform_interface 3.10.1 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.5.7 [_flutterfire_internals firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins]
- firebase_core_platform_interface 5.0.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.14.0 [firebase_core_platform_interface flutter flutter_web_plugins meta web]
- firebase_messaging_platform_interface 4.5.30 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_messaging_web 3.8.0 [_flutterfire_internals firebase_core firebase_core_web firebase_messaging_platform_interface flutter flutter_web_plugins meta we
b]
- firebase_remote_config_platform_interface 1.4.28 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_remote_config_web 1.6.0 [firebase_core firebase_core_web firebase_remote_config_platform_interface flutter flutter_web_plugins]
- flutter_local_notifications_linux 4.0.0+1 [dbus ffi flutter flutter_local_notifications_platform_interface path xdg_directories]
- flutter_local_notifications_platform_interface 7.0.0+1 [flutter plugin_platform_interface]
- flutter_web_plugins 0.0.0 [flutter characters collection material_color_utilities meta vector_math]
- html 0.15.4 [csslib source_span]
- http_parser 4.0.2 [collection source_span string_scanner typed_data]
- image 4.1.7 [archive meta xml]
- in_app_purchase_android 0.3.2 [collection flutter in_app_purchase_platform_interface json_annotation]
- in_app_purchase_platform_interface 1.3.7 [flutter plugin_platform_interface]
- in_app_purchase_storekit 0.3.13+1 [collection flutter in_app_purchase_platform_interface json_annotation]
- js 0.6.7 [meta]
- json_annotation 4.8.1 [meta]
- leak_tracker 10.0.0 [clock collection meta path vm_service]
- leak_tracker_flutter_testing 2.0.1 [flutter leak_tracker leak_tracker_testing matcher meta]
- leak_tracker_testing 2.0.1 [leak_tracker matcher meta]
- linkify 5.0.0
- lints 3.0.0
- matcher 0.12.16+1 [async meta stack_trace term_glyph test_api]
- material_color_utilities 0.8.0 [collection]
- meta 1.11.0
- nm 0.5.0 [dbus]
- os_detect 2.0.1
- package_info_plus_platform_interface 2.0.1 [flutter meta plugin_platform_interface]
- path_provider_android 2.2.2 [flutter path_provider_platform_interface]
- path_provider_foundation 2.3.2 [flutter path_provider_platform_interface]
- path_provider_linux 2.2.1 [ffi flutter path path_provider_platform_interface xdg_directories]
- path_provider_platform_interface 2.1.2 [flutter platform plugin_platform_interface]
- path_provider_windows 2.2.1 [ffi flutter path path_provider_platform_interface win32]
- petitparser 6.0.2 [meta]
- platform 3.1.4
- plugin_platform_interface 2.1.8 [meta]
- pointycastle 3.7.4 [collection convert js]
- pool 1.5.1 [async stack_trace]
- process_run 0.14.2 [path collection string_scanner yaml meta args pub_semver synchronized]
- pub_semver 2.1.4 [collection meta]
- shared_preferences_android 2.2.1 [flutter shared_preferences_platform_interface]
- shared_preferences_foundation 2.3.5 [flutter shared_preferences_platform_interface]
- shared_preferences_linux 2.3.2 [file flutter path path_provider_linux path_provider_platform_interface shared_preferences_platform_interface]
- shared_preferences_platform_interface 2.3.2 [flutter plugin_platform_interface]
- shared_preferences_web 2.3.0 [flutter flutter_web_plugins shared_preferences_platform_interface web]
- shared_preferences_windows 2.3.2 [file flutter path path_provider_platform_interface path_provider_windows shared_preferences_platform_interface]
- sky_engine 0.0.99
- source_span 1.10.0 [collection path term_glyph]
- sqflite_common 2.5.4 [synchronized path meta]
- sqflite_common_ffi 2.3.3 [sqlite3 sqflite_common synchronized path meta]
- sqlite3 2.4.0 [collection ffi js meta path]
- stack_trace 1.11.1 [path]
- stream_channel 2.1.2 [async]
- string_scanner 1.2.0 [source_span]
- synchronized 3.1.0+1
- term_glyph 1.2.1
- test_api 0.6.1 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph]
- timezone 0.9.2 [path]
- typed_data 1.3.2 [collection]
- universal_io 2.2.2 [collection meta typed_data]
- universal_platform 1.0.0+1
- url_launcher_android 6.3.0 [flutter url_launcher_platform_interface]
- url_launcher_ios 6.2.5 [flutter url_launcher_platform_interface]
- url_launcher_linux 3.1.1 [flutter url_launcher_platform_interface]
- url_launcher_macos 3.1.0 [flutter url_launcher_platform_interface]
- url_launcher_platform_interface 2.3.2 [flutter plugin_platform_interface]
- url_launcher_web 2.3.0 [flutter flutter_web_plugins url_launcher_platform_interface web]
- url_launcher_windows 3.1.1 [flutter url_launcher_platform_interface]
- vector_math 2.1.4
- version 3.0.2
- vm_service 13.0.0
- web 0.5.1
- webview_flutter 4.7.0 [flutter webview_flutter_android webview_flutter_platform_interface webview_flutter_wkwebview]
- webview_flutter_android 3.16.0 [flutter webview_flutter_platform_interface]
- webview_flutter_platform_interface 2.10.0 [flutter meta plugin_platform_interface]
- webview_flutter_wkwebview 3.13.0 [flutter path webview_flutter_platform_interface]
- win32 5.3.0 [ffi]
- win32_registry 1.1.2 [ffi win32]
- xdg_directories 1.0.4 [meta path]
- xml 6.5.0 [collection meta petitparser]
- yaml 3.1.2 [collection source_span string_scanner]

Additional context and comments
Seems like its occuring due to race condition. I have browsed the already reported issues, found that
3.5.16
FIX(messaging): fix an issue where Service Worker would not register in time (#11979). (4752efb6)
This issue is been fixed for firebase_messaging_web package, it will be good the same fixed for firebase_messaging too

Thanks for the report, have you tried in release mode?

@Lyokone you mean flutter build web --release ?

Executed flutter run --release .Still getting
Error while generating FCM Token: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker.
But on inspecting network tab, I could see firebase-messaging-sw.js,firebase-app-compat.js and  firebase-messaging-compat.js are loaded status 200.
Update i even updated new packages available on pub.dev
firebase_analytics: ^10.10.1
firebase_core: ^2.29.0
firebase_messaging: ^14.8.1
firebase_remote_config: ^4.4.1

And also, on check on tried to get the token on button click, i could see some 20+more seconds delay then able to getToken printed on console.
Whereas in Android Mobile App, which is already available to users, fectching token within a second or less.

@Lyokone Can you update ?

@Lyokone or @TarekkMA  Can you update ?

Hi @DDSrivatsan, Still getting the same error, i´m using Flutter 3.19.6 and Dart 3.3.4 with firebase_messaging: 14.9.2 and firebase_core: 2.31.0.

Hi @lautarozanuttini , im waiting for the team to look after this issue. Either @Lyokone or @TarekkMA - should look into this issue.. however you can refresh the page once and try getting the token- it will work

hi, @TarekkMA and @Lyokone can you update ? is this issue fixed  ?

@Lyokone or @TarekkMA Can you update ? still the issue is not fixed

Same error

Same mistake. A bit difficult to replicate because it happens sometimes.
I have this button in main..
TextButton(
            onPressed: () async {
              try {
                final response = await FirebaseMessaging.instance.getToken();
                print('PushManager $response');
              } catch (e) {
                print('PushManager error in UI: ${e.toString()}');
              }
            },
            child: const Text('token'),
          ),

and I see the code that should handle the race condition case... Fixed by https://github.com/firebase/flutterfire/pull/11979
https://github.com/firebase/flutterfire/blob/c6865e477c1b84e8aabef6d5a099eea89c66b427/packages/firebase_messaging/firebase_messaging_web/lib/src/interop/messaging.dart#L59-L66
But, upon seeing that the error message was the same one that needed to be fixed, I took the liberty of manually modifying the package's code to check if that was indeed the message. And yes, as can be seen in the logs, the text of the message is the same, but for some reason, it is still failing.

And these were the logs I obtained.
PushManager error in messaging_web: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker firstGetTokenCall: false
PushManager error in messaging_web before rethrow: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker firstGetTokenCall: false
PushManager error in UI: AbortError: Failed to execute 'subscribe' on 'PushManager': Subscription failed - no active Service Worker

When the message contains no active service worker, shouldn't it call the method again and retrieve the token?
Suspected Behavior
What I think is happening: The first time, it detects that it is the initial call and enters the conditional for the race condition. It sets firstGetTokenCall to false, calls getToken again, but the service worker is not yet active, causing the error to be thrown. It doesn't enter the condition again because firstGetTokenCall is already false at that point.
cc/ @Lyokone @TarekkMA


Fix windows build

Fixed problems with building on windows system because of pathing issues.

Have tested a build on MacOS and can confirm packages still build correctly.

Ótimo trabalho amigo!


add readme for tag parameter framework

I have randomly placed it in doc for now. I think this will go to the wiki. Sending a PR for feedback.
@lisitsyn @vigsterkr

Cool! I think it's very clear and only minor stuff needs to be fixed!

This should be a wiki, no?

Yes, I recently added a new repo for the wiki which can be cloned in a
standard way and then you can send a or against that
Wanna try?
On Monday, 31 October 2016, Soumyajit De<EMAIL_ADDRESS>wrote:

This should be a wiki, no?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shogun-toolbox/shogun/pull/3414#issuecomment-257228555,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqqvwdVSZPUfCdPNfkA66boxL7gK-Roks5q5Zd4gaJpZM4JfP6Q
.

--
Sent from my phone

Ill do it

https://github.com/shogun-toolbox/shogun/wiki/README_tags


reducing build time and updating node

What's done:

using node 8
using alpine to speed up build
using kue-ui-express instead of kue-ui - it's required for kue version update
updating kue
yarn.lock added

@maZahaca any chance to have it merged?

@jifeon Sure. But seems like this branch has conflicts. Can you revise and fix it?


Version of calicoctl is too old

File: ee/ucp/kubernetes/install-cni-plugin.md, CC @gbarr01
EE ships with Calico appears to be 2.0.0.  I was not able to run the commands for disabling IP in IP because 1.6.3 returns no data.

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!

The link provided under # Download calicoctl is showing a version of Calico that I do not believe we ever used as it currently has 3.1.1 but I believe we went from 3.0.x in EE 2.0 to 3.2.x in EE 2.1.  Based on what I see in UCP 3.1.1, we have Calico 3.2.3 so I would think that the instructions should work best with that version, not 3.1.1.

It looks like 3.2.3 is a valid version:
$ wget https://github.com/projectcalico/calicoctl/releases/download/v3.2.3/calicoctl
--2019-01-02 11:54:03--  https://github.com/projectcalico/calicoctl/releases/download/v3.2.3/calicoctl
Resolving github.com (github.com)... <IP_ADDRESS>, <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://github-production-release-asset-2e65be.s3.amazonaws.com/29629333/1a6fd180-c317-11e8-9b45-47872aae49e5?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20190102%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20190102T165404Z&X-Amz-Expires=300&X-Amz-Signature=f7b286230fefb234df0b5292b66244fa73c879c699a3d1d5d40da9c787567887&X-Amz-SignedHeaders=host&actor_id=0&response-content-disposition=attachment%3B%20filename%3Dcalicoctl&response-content-type=application%2Foctet-stream [following]
--2019-01-02 11:54:04--  https://github-production-release-asset-2e65be.s3.amazonaws.com/29629333/1a6fd180-c317-11e8-9b45-47872aae49e5?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20190102%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20190102T165404Z&X-Amz-Expires=300&X-Amz-Signature=f7b286230fefb234df0b5292b66244fa73c879c699a3d1d5d40da9c787567887&X-Amz-SignedHeaders=host&actor_id=0&response-content-disposition=attachment%3B%20filename%3Dcalicoctl&response-content-type=application%2Foctet-stream
Resolving github-production-release-asset-2e65be.s3.amazonaws.com (github-production-release-asset-2e65be.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to github-production-release-asset-2e65be.s3.amazonaws.com (github-production-release-asset-2e65be.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 30827264 (29M) [application/octet-stream]
Saving to: ‘calicoctl’

calicoctl                                     100%[==============================================================================================>]  29.40M  16.1MB/s    in 1.8s

2019-01-02 11:54:06 (16.1 MB/s) - ‘calicoctl’ saved [30827264/30827264]


Detox Website: initial version.

This pull request creates the initial version of Detox website (using Docusaurus), along with GitHub action for deploying the website when master changes.

@asafkorem  as for the images – could you explicitly list their URLs (here at least), where do they come from, and what their respective licenses are? We just need to keep in mind that we're a public project and need to comply with the copyright policies. Let's be 100% transparent about the assets.

Last but not least, @asafkorem do you have anywhere a local deployment we can go and take a look at?

Last but not least, @asafkorem do you have anywhere a local deployment we can go and take a look at?

Sure, you can checkout this branch from my repository and execute npm run start from the website's dir.

@asafkorem as for the images – could you explicitly list their URLs (here at least), where do they come from, and what their respective licenses are? We just need to keep in mind that we're a public project and need to comply with the copyright policies. Let's be 100% transparent about the assets.

Great point. All photos were taken from iconarchive.com (and are free for non-commercial use). I've added all references in the comments:
https://github.com/wix/Detox/blob/778ea4482f9aa38dbaad2dc0f776a4b9c7e448d0/website/src/components/HomepageFeatures.js


[TW#19279] Bug: This scenario makes the Ringbuffer API freeze the ESP32

If you run this code, the ESP32 will get stuck. Looks like its a bug in the Ringbuffer API.
#define log(...)			do { printf(__VA_ARGS__); printf("\n"); } while (0)

void test_ring_buff(void)
{

	char seven[] = "seven1";
	char eight[] = "eight12";
	char nine[] = "nine1234";

	log("\n\n\n\n Start");

	RingbufHandle_t h_ring = NULL;
	h_ring = xRingbufferCreate(150, RINGBUF_TYPE_NOSPLIT);

	xRingbufferPrintInfo(h_ring);

	int cnt = 0;
	char *p_rx_buf = NULL;
	size_t rx_len;

	int loop = 1000;

	while(--loop) {
		cnt = 0;
		while(1) {
			if(xRingbufferSend(h_ring, seven, sizeof(seven), timer_ms(100)) == pdFALSE) {
				log("failed to insert 7. total: %u", cnt);
				break;
			} else {
				cnt += sizeof(seven);
				log(">>> added 7. total payload: %u", cnt);
			}

			if(xRingbufferSend(h_ring, eight, sizeof(eight), timer_ms(100)) == pdFALSE) {
				log("failed to insert 8. total: %u", cnt);
				break;
			} else {
				cnt += sizeof(eight);
				log(">>> added 8. total payload: %u", cnt);
			}

			if(xRingbufferSend(h_ring, nine, sizeof(nine), timer_ms(100)) == pdFALSE) {
				log("failed to insert 9. total: %u", cnt);
				break;
			} else {
				cnt += sizeof(nine);
				log(">>> added 9. total payload: %u", cnt);
			}
		}
		xRingbufferPrintInfo(h_ring);

		while(1) {
			p_rx_buf = xRingbufferReceive(h_ring, &rx_len, timer_ms(100));
			if(rx_len == 0) {
				log("0 len -- no more data?");
				break;
			} else if(p_rx_buf == NULL) {
				log("read timeout -- no more data?");
				break;
			} else {
				log("full returning. len: %u", rx_len);
				vRingbufferReturnItem(h_ring, p_rx_buf);
			}
		}

		xRingbufferPrintInfo(h_ring);
	}

	log("fin\n\n\n\n");
	xRingbufferPrintInfo(h_ring);

	while(1) {
		vTaskDelay(timer_sec(10));
	}

}

Ringbuffers will be reworked as part of the upcoming release 3.1


Error Console

Hey there :)
Can you tell us in what situation this happen exactly?
Basically, this error means you have sent an interaction to a user, the bot has been shut down and this user tries to interact with the previously-sent interaction.

Hey,

Is the error still happening today? Has it even happens again tho?

Bump


The icons doesn't seen to vertical align on the selected item

Just like the title of the issue says, after testing the widget a bit, I noticed that the icons do not seem to be centered with respect to the box that contains them. Unlike the text that does look centered.

This is typically an issue with icon fidelity, not with alignment. Reopen if you find out otherwise. Thanks!

PPS: unable to reproduce. If you reopen this please provide a reproduction.

Hi @lukepighetti, thanks for the quick reply.
I'm currently using version 1.22.0-12.1.pre of the Flutter beta channel so that I can test using the Chrome browser. Also, and maybe this is implicit, I'm using the version 2.0.0 of your package.
The code that I was working looks so far like this:
import 'package:flutter/material.dart';
import 'package:salomon_bottom_bar/salomon_bottom_bar.dart';

void main() {
  runApp(MainPage());
}

class MainPage extends StatefulWidget {
  @override
  _MainPageState createState() => _MainPageState();
}

class _MainPageState extends State<MainPage> {
  int _currentIndex = 0;

  void _setCurrentTab(int tabIndex) {
    setState(() {
      _currentIndex = tabIndex;
    });
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: SafeArea(
          child: Stack(
            children: [
              Positioned.fill(
                bottom: 56,
                child: IndexedStack(
                  index: _currentIndex,
                  children: [
                    Container(),
                    Container(),
                    Container(),
                  ],
                ),
              ),
              Align(
                alignment: Alignment.bottomCenter,
                child: ClipPath(
                  clipper: BottomBarClipper(),
                  child: Container(
                    height: 76,
                    width: double.infinity,
                    color: Theme.of(context).primaryColor,
                    padding: EdgeInsets.only(
                      top: 20.0,
                    ),
                    child: AppBottomBar(
                      _currentIndex,
                      _setCurrentTab,
                    ),
                  ),
                ),
              ),
            ],
          ),
        ),
      ),
    );
  }
}

class BottomBarItem {
  final String title;
  final IconData iconData;

  const BottomBarItem(this.title, this.iconData);
}

class AppBottomBar extends StatelessWidget {
  final int _currentIndex;
  final Function(int) _onTap;

  static const List<BottomBarItem> elements = [
    BottomBarItem('Inicio', Icons.home),
    BottomBarItem('Actividad', Icons.timeline),
    BottomBarItem('Colección', Icons.bookmark),
  ];

  const AppBottomBar(
    this._currentIndex,
    this._onTap, {
    Key key,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return SalomonBottomBar(
      currentIndex: _currentIndex,
      selectedItemColor: Colors.white,
      unselectedItemColor: Colors.white70,
      onTap: _onTap,
      items: elements.map((e) {
        return SalomonBottomBarItem(
          icon: Icon(e.iconData),
          title: Text(e.title),
        );
      }).toList(),
    );
  }
}

class BottomBarClipper extends CustomClipper<Path> {
  Path getClip(Size size) {
    final height = size.height;
    final width = size.width;
    final radius = 20.0;
    final path = Path();

    path
      ..quadraticBezierTo(0, radius, radius, radius)
      ..lineTo(width - radius, radius)
      ..quadraticBezierTo(width, radius, width, 0)
      ..lineTo(width, height)
      ..lineTo(0, height)
      ..close();

    return path;
  }

  @override
  bool shouldReclip(CustomClipper<Path> oldClipper) {
    return false;
  }
}

Using the Widget extension that you provided, the icon only needs to nudged by y: -1 to look centered.
I'm pretty noob developing on Flutter, so maybe I'm wrong using a Widget on the layout.

The height: 76 is squishing the bottom bar too much which is causing the icons to spill out the bottom. I would remove it and allow it to size itself implicitly. And we can consider this out of the scope of this package. Thanks!


fix: allow plugin to run with java 11 and 17

minimal Jenkins was bumped to a version which supports java 11
bumped bom to matching jenkins version
added missing token-makro deps
replaced plain dependencies for Jackson and httpcomponents with API plugins
bumped mvn assembly to allow it to work with java 11 and a new property was added to support macOS

Build was executed locally.

all tests passed
Jerkins was started without any issues and exceptions

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@sdewitt-newrelic
@sschwartzman

Could you please review my change? :D Thanks
@sdewitt-newrelic @sschwartzman

SOrry @DuMaM , I no longer work at NR, so I am not involved in the maintenance of this repo.


"Valea Ierii", "Cerc" and "Plopi" villages are listed in the drop-down menu

The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Valea Ierii"

Expected results
The location appears in the drop-down menu as "Sat Valea Ierii, CLUJ (Valea Ierii)"

Step 2
Write in the search bar "Cerc"

Expected results
The location appears in the drop-down menu as "Sat Cerc, CLUJ (Valea Ierii)"

Step 3
Write in the search bar "Plopi"

Expected results
The location appears in the drop-down menu as "Sat Plopi, CLUJ (Valea Ierii)"


Migrate to Java 17

Java 11 priority support ends in September 2023, but the parser we use is not compatible with Java 17

Java 11 Security Support ends only on 30 Sep 2026


Create devops.md

Describe your changes
Localize DevOps in German
Related issue number or link (ex: resolves #issue-number)
resolves #2537
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@ARYANis0001 I have a common comment on your PRs.
PTAL: https://github.com/cncf/glossary/pull/2605#issuecomment-1815774999

Hi lang/de reviewers, @iamNoah1 @DaveVentura @bcubk
Could you check this PR has enough quality to get reviewed? Considring  https://github.com/cncf/glossary/pull/2605#issuecomment-1815774999


Add vsplit kind

Currently, the ability to open Neogit in split is supported, but this may not be very convenient on wide monitors. It would be awesome to support kind=vsplit for this purpose.

added vsplit to kind options in c0d1b95

Thank you!


Add run-image-alias functionality

The new configuration, run-image-alias will pull and alias an image for use by pipeline steps. This is one possible way that we could implement #122.
The goal of this new configuration is to generate the override file that looks like this:
version: '3.1'
services:
  base_service:
    image: my.repo/image:docker-compose-plugin-${pipeline-slug}-${build-number}
  # Uses the built image for base_image, because we define run-image-alias in the pipeline.
  service1:
    image: my.repo/image:docker-compose-plugin-${pipeline-slug}-${build-number}

This probably isn't finished yet as I'm not great with bash, but opening this PR to get comments/thoughts. Maybe it's possible to leverage cache_from instead? Feel free to takeover/modify this PR.

@toolmantim @lox - I was able to get this working with a new config. Would appreciate any feedback on this if you have it. If something like this looks ok to you, I would appreciate any help getting this landed. Thanks!

It dawns on me that we originally added the image parameter in to handle something like this. I wonder if we merged #107 whether your example would work without the addition of a new concept.

Going to close this one for now, still figuring out how to solve this.


Theme configuration not working properly

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
MUI theme configuration like typography and components override not working, sometimes page refresh it works, but most of time it does not work

Typography like h1 override does not work

In HTML it creates a new class with typography, see below

Expected behavior 🤔
No response
Steps to reproduce 🕹
Steps:

create a mui theme in nextjs
theme override like typography and component does not work, primary and secondary color etc are working fine
I m using the same with approach with mui version 4, it work as expected

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

Did you use the examples projects we provide for setting up the repo? There were some updates there regarding how to configure SSR.


add a date check method

this adds a date check method in order to support the fix for axes auto crossing if values contain Jan 1, 1970
https://github.com/FormidableLabs/victory/issues/139
@boygirl

@boygirl updated the methods

@boygirl oh, cool! updated.

@ebrillhart want to write a real quick unit test for this?

@boygirl added and indented!

Approved!


Fix retry for Netty network errors

@mbasmanova, I don't think so.  This issue occurs when there is an underlying error in IO layer, and I can't think of any way to inject that into the code for testing.  It was tested manually.


[WFLY-9358] [WFLY-9351] [WFLY-9352] [WFLY-9350] [WFLY-9349] [WFLY-9357] [WFLY-9382] Remote EJB clustered node discovery is ignored

[WFLY-9357] [WFLY-9382] Changes for lazy user transaction and related security enhancements
[WFLY-9358] [WFLY-9351] [WFLY-9352] [WFLY-9350] [WFLY-9349] Remote EJB clustered node discovery is ignored

Replaced by https://github.com/wildfly/wildfly/pull/10526


Use FQCN as service name

scheb/two-factor-bundle requires Symfony 3.4, so we can use a FQCN as a service name: https://symfony.com/blog/new-in-symfony-3-3-optional-class-for-named-services.
Since scheb/two-factor-bundle requires ^7.1.3 and Symfony 3.4 and Symfony 3.4 requires ^5.5.9|>=7.0.8, requiring >=5.3.0 in this bundle makes no sense.

Closing in favor of #22


Update version number in project.json

gitversion should update the version number of the project.

it is possible this has been fixed - need to verify


Fix resume option

Code used for the last large training run.  Fixes issues with the resume option that caused it to do too little training after resuming.  Also adds a tool to get information about the training resume data.

@ScottMcMichael And fix the linter errors

Thanks!


Feature : Multi-dimensional input in LSTM

Current input format for RNN Cell (ie LSTM) is
        (batch_size, Sequence_length)

        cell_fn = gr iid_rnn.Grid2LSTMCell
        additional_cell_args.update({'use_peepholes': True, 'forget_bias': 1.0})
        cell = cell_fn(args.rnn_size, **additional_cell_args)
        self.cell = cell = rnn_cell.MultiRNNCell([cell] * args.num_layers)
        self.input_data =    tf.placeholder(tf.int32, [args.batch_size, args.seq_length])

it would be useful to have multi-dimensionnal input as below :
        (batch_size, Sequence_length, extra_dim1)

What does the extra_dim1 argument do? You haven't modified any of the rest of the code, so I don't see how it should affect the operation of the LSTM cells. Can you not just fold it into the batch dimension?

Lets example of CNN input:
(Nbsample,  X, Y, Z)
X, Y: 2D
Z: color depth
NbSample: nb of sample
We wish to add one more dimension in LSTM input (ex: to represent color depth).
Regards
Kevin
On 8 Jul 2017, at 09:38, Cliff Young<EMAIL_ADDRESS>wrote:
What does the extra_dim1 argument do? You haven't modified any of the rest of the code, so I don't see how it should affect the operation of the LSTM cells. Can you not just fold it into the batch dimension?
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

So is it that you want a 3D grid LSTM cell? Or is it that you still want a 2D cell, but there's some way in which the num_units argument of the 2D cell doesn't meet your needs?
(and my apologies if I'm just not getting it--I'm a systems person more than an NN person).

This issue is automatically closed due to lack of activity. Please re-open if this is still an issue for you. Thanks!

Any updates on this? Seems like a useful feature to me.


Update Preview warning as Important

Be consistent about the disclaimer for preview feature along every mention on all Azure documentation. As I saw it first on Custom Security Attributes doc (https://learn.microsoft.com/en-gb/azure/active-directory/fundamentals/custom-security-attributes-overview), in this Filter for Applications, it should follow the same layout and text.

@MicrosoftGuyJFlo
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add first pass Send/Receive

User Story
As a translator, I want to be able to collaborate with others on my Scripture projects so that we can work together effectively.
Description
Develop a rough-cut first draft of Send/Receive functionality in Platform.Bible.

[ ] Create simple S/R dialog
[ ] Create simple compare versions dialog?

Questions:

For first pass, can we rely on S/R being set up in PT9 for it to work?

PT9 Send/Receive Dialog

S/R Results dialog...?

Compare Versions

Compare Versions with version selector dropdown

Methods from SendReceiveProjectsForm.cs

[ ] Determine if internet S/R is available

CheckSourceIsAvailable - logic for internet S/R
SetInternetDisabledText - indicates why internet S/R is not available

[ ] List projects available for S/R

Get list of online projects and S/R-enabled local projects

maybe constructor has info? See CoaxUserToDoASendReceive

Determine new/edited status

Maybe trace GetProjectsModifiedByAnyone?

Not sure if this is an appropriate place for what is essentially a feature request, but I often find myself wanting to "receive without sending". Conflict resolution is obviously a complicated problem - in my case, I am always willing to destroy local changes. I am wanting to be sure I don't push local changes. At the very least, I want to note this here because I'd love to see the design support this kind of feature in the future, even if not immediately.

Looks like we will have to find mercurial on the system or bundle it in like Paratext does (on Windows). There are also a handful of files we need to distribute. Some notes from a discord conversation:

Paratext uses 4.8.2
Newer than 4.8.2 will have problems with the init function with default config. John W thinks this would only affect if P.B created a USB repo and Paratext tried to use it. But there are init arguments that make it backwards compatible
Paratext has Python 2.7 scripts that Mercurial must be able to run. Therefore it is stuck on versions 4.3-6.1ish
We may be able to build Mercurial ourselves and distribute it ourselves
It seems the oldest Mac build of Mercurial on the site is TortoiseHg 5.3
I have no idea how getting Mercurial for linux would work. It's on apt-get and some other package repos, but I can't list versions. There are also some rpm files on the site.

Mercurial downloads
DeveloperRepos - Mercurial
hg-stable: 197f092b2cd9 /
BuildingOnWindows - Mercurial
BuildingPackages - Mercurial

To use Send/Receive for now:
On Windows:

Install Paratext 9
Enter your registration information in Paratext 9

On Linux or Mac:

Install Mercurial (Ubuntu: run sudo apt-get install mercurial)
Copy InternetSettings.xml and RegistrationInfo.xml from a Windows machine %localappdata%/Paratext95 to ~/.local/share/Paratext95 (or set them up however you want)

PRs:
Made extension contributions load in order, added spinner, updated ParatextData to <IP_ADDRESS>, misc bug fixes and improvements - 8 unreplied - Reviewable
Include untracked changes aka new code files in patches - Reviewable
Added send/receive service to dotnet process by tjcouch-sil · Pull Request #3 · paranext/paratext-10-studio
Added Send/Receive extension including dialog and menu items by tjcouch-sil · Pull Request #67 · paranext/paratext-bible-internal-extensions
Documentation:
Troubleshooting Guide · paranext/paranext Wiki
Set up Send Receive Functionality · paranext/paranext Wiki

Verified the Send/Receive functionality on the Paratext 10 studio.

Verified that all the projects that I have access to are displayed in the Send/Receive dialog
Verified that the changes I made are Send/Receive
Verified I can get all the changes made by other Users (TJ)
Verified that a merge conflict is displayed when 2 users made changes on the same book and same verse .
Gen 1 verse showed up as conflict as changes are made in the same verse.


How to parse swagger file for api test

Problem statement
Hi,I want parse the swagger file ,get url,body,method . and send a request ,check response with swagger file response. how to do that？

You can build such tooling using the toolkit go-openapi packages. « Spec » to parse a spec, « analysis » to gather some facts about the spec. You might also reuse the go-swagger generator and configure it with your custom templates.


[core] Introduce CacheStats and expose ScanStats

Purpose

Linked issue: close #xxx
Add CatalogCacheStats and ObjectCacheStats so that engine can get cache stats at query level.

Tests

added
API and Format

CachingCatalog.getCacheStats
ScanMetrics.getLatestScan
ScanMetrics.getObjectCacheStats
Documentation

added

@JingsongLi Ready for review now, PTAL, thx~

+1