[firebase_admob] ^0.9.0+7 Android/iOS app crashes on clean installation

Describe the bug
With a clean installation with flutter create command, app crashes without error.
To Reproduce
Steps to reproduce the behavior:

flutter create
Configure app following the steps of page
Running app
App crashes without error on Android
iOS app crashes with the following message:

    Launching lib/main.dart on iPhone 11 Pro Max in debug mode...
    Xcode build done.                                           61,0s
        path: satisfied (Path is satisfied), interface: en1
    Configuring the default Firebase app...
    *** First throw call stack:
    (
        0   CoreFoundation                      0x00007fff23baa1ee __exceptionPreprocess + 350
        1   libobjc.A.dylib                     0x00007fff50864b20 objc_exception_throw + 48
        2   CoreFoundation                      0x00007fff23baa02c +[NSException raise:format:] + 188
        3   Runner                              0x000000010c7792fa +[FIRApp configure] + 138
        4   Runner                              0x000000010c953872 -[FLTFirebaseAdMobPlugin init] + 258
        5   Runner                              0x000000010c9535e6 +[FLTFirebaseAdMobPlugin registerWithRegistrar:] + 70
        6   Runner                              0x000000010c774ba0 +[GeneratedPluginRegistrant registerWithRegistry:] + 112
        7   Runner                           <…>
    Lost connection to device.

Expected behavior
App running
Additional context
[✓] Flutter (Channel stable, v1.9.1+hotfix.5, on Mac OS X 10.14.6 18G95, locale es-PE)
• Flutter version 1.9.1+hotfix.5 at /Users/brau/flutter
• Framework revision 1aedbb1835 (2 days ago), 2019-10-17 08:37:27 -0700
• Engine revision b863200c37
• Dart version 2.5.0
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /Users/brau/Library/Android/sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-28, build-tools 28.0.3
• ANDROID_HOME = /Users/brau/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 11.1, Build version 11A1027
• CocoaPods version 1.8.4
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 40.2.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.39.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.5.1

I have the same issue and I read about other users with the same issue. Latest version that doesn't have the problem is 0.5.5 for me.
Problem is, with this version I can't build the apk, I get this error:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':firebase_admob:verifyReleaseResources'.
> java.util.concurrent.ExecutionException: com.android.builder.internal.aapt.v2.Aapt2Exception: Android resource linking failed
  Output:  /Users/jonas/Projects/ProjectFit/mobile/project_fit_flutter/build/firebase_admob/intermediates/res/merged/release/values/values.xml:208: error: resource android:attr/fontVariationSettings not found.
  /Users/jonas/Projects/ProjectFit/mobile/project_fit_flutter/build/firebase_admob/intermediates/res/merged/release/values/values.xml:209: error: resource android:attr/ttcIndex not found.
  error: failed linking references.

I got the same errors for both libraries (tested separately as well).
cloud_firestore: ^0.12.9+5
firebase_auth: ^0.14.0+5
The same behaviours for the same steps to reproduce.

I got the same errors firebase_admob: 0.9.0

https://github.com/FirebaseExtended/flutterfire/issues/17

Using the value provided on the instruction page causes crash, try this:
    <meta-data
        android:name="com.google.android.gms.ads.APPLICATION_ID"
        android:value="ca-app-pub-0000000000000000~0000000000"/>

use flutter build appbundle for android

This is a duplicate of https://github.com/FirebaseExtended/flutterfire/issues/2516


fix: destory replica cli command

https://discord.com/channels/933071162680958986/933071163184283651/1151188444819763304

Thanks!


Supplier API Integration

Suppliers (and potentially Retailers) provide an Excel/CSV spreadsheet that contains their product information to import it into Crux.
The task is to convert the provided spreadsheet(s) into a csv formatted file in the crux format.

Konan's group decided to go a different route for integrating customer files -- so this project is closed.


ImportError: No module named _view

python main.py
pygame 1.9.6
Hello from the pygame community. https://www.pygame.org/contribute.html
Traceback (most recent call last):
  File "main.py", line 18, in <module>
    import pygame._view
ImportError: No module named _view

tested with Python 2.7.16
Hi @LingDong- , can you provide your pip requirements.txt?

Hi there, it appears that the newer versions of pygame no longer have _view module, but the fix turned out be super simple: Just comment out that line in main.py and it will work! Just tested on 2.7.15 and pygame 1.9.4.
import pygame
# import pygame._view  #<--- comment out this line
import numpy

I haven't ran the project for a while myself, and it seems that it is more laggy than it used to be, and the loading screen is blank. Something must have changed with pygame over the years. Will look into it when I have time.
Thanks.

Thanks for your quick reply.
The exception disappeared, it prints a loaded log. But the window keeps blank.

It would be better to use virtualenv and lockdown all the dependencies.

Thanks for your quick reply.
The exception disappeared, it prints a loaded log. But the window keeps blank.

+1, any workarounds?

I will rewrite the project to Python 3 and use Poetry for dependency management.
I can run it here.
zweix123/Hermit-Python3


Retain Hash keys insertion order

Retaining the Hash keys insertion order is a very convenient Ruby feature. I wonder if this is planned for Goby as well, or not.

Closing for now as we haven't seen any request on this.


[Hackalgo] Build using Folks Finance JS SDK

Overview

Short Description: Build using Folks Finance JS SDK to educate other developers and / or provide meaningful contribution to the community.
Bounty: 3000-5000 ALGO tokens
Estimated Time Commitment: 1 -2 weeks
Category: Development
Experience Level: Intermediate / Advanced
Mode: Cooperative

Description
What is this task?
Folks Finance is a capital markets protocol for borrowing and lending on Algorand. The task it to build something using the JavaScript SDK https://github.com/Folks-Finance/folks-finance-js-sdk.
Specifically, you should code one of the following:

Notification system where users can set triggers for when they should be notified that their loan is close to being liquidated. For example, a user selects that they want to receive a text message when the health factor of their loan falls to 1.1.
Liquidation bot with automatic balancing. For examples, the bot searches for loans with a health factor below 1 and finds a ALGO-USDC loan. It:

liquidates the loan by repaying the USDC and receives fALGO
withdraws fALGO for ALGO
swaps ALGO for USDC.

Live analytical dashboard for Folks Finance market: metrics such as TVL, average loan position, daily liquidation volume etc.
Your own suggestion - you must receive approval from us beforehand.

You can find the Folks Finance app at https://app.folks.finance and documentation at https://docs.folks.finance/.
Deliverables

Completed GitHub repository linked in your Gitcoin submission.
Markdown file (README.md) explaining how to set up the project, architecture, and how the project works (which SDK functionality you've used where)
Development feedback (share in this Github thread)

Judging Criteria and Metrics

Utility/Value - How much value will this bring to the Folks Finance community?
Is the code and repository well-organized and documented? Does it adhere to language-specific style guidelines?
Guidance for users (README.md) and quality of feedback for Folks Finance

Other Requirements
For questions, reach out to Folks Finance on Discord.

IMPORTANT As gitcoin does not provide a way for us to contact you, after taking the bounty, please subscribe to the bounty issue to receive comments notifications from us. (You can find a "View on Github" button in this page to go to the bounty issue)
All project codes should be published on GitHub, with an MIT license or Apache V2 license.
All non-code documents should be using Markdown format.

Hi @gidonkatten, I have read the documentation of your SDK and had plans for what I need to do, could u please approve me so that I can start? You can click the approve worker button to approve.

@Soptq approved! What dex do you plan on using for the swaps?

Hi @gidonkatten, I'm interested to build a liquidation notifier. Could you please approve me?

Hi @gidonkatten, I'm interested in working with Folks on this, have gone though the docs.
Previous Work:
BeMetaPlace on Biconomy
Featured Wallet on Fluence (winner)
Thank you and could you approve me?

Working on a liquidation bot here, please approve when you got time. @gidonkatten


Moves are disappearing

Quite literally!
Unfortunately I don't have the steps to reproduce, but just to keep you on the lookout.
This is the game: https://gokibitz.com/kifu/4JTw6I6r
The original SGF I uploaded has 66 moves, and I remember commenting (Game comments, not "user" comments) them, editing the comments etc.
I started to notice that some game comments were shifting from one move to another. I log out, come back a few hours later, and the game now has 64 moves! I refresh the page, click "last" and now it's 63 moves.
Proof? Take a look at this screenshot: BenKyo, symeof and myself commented on move 64 before it disappeared.

I remember reading their comments and adding mine at move 64 before it disappeared.
This is the original SGF: https://www.dropbox.com/s/fwmzihp7lmeg0bv/musuko-nekochan.sgf?dl=0
Welp.

Let me check this out: thanks for giving such complete information!

Thanks to you!
I have an hunch: I might (or not) have used a tilde ~ character in a comment. However, the SGF I uploaded was void of any comment, so if a comment screwed up the parser it was entirely on GoKibitz side.
Does MongoDB keep revisions like CouchDB? if so, we might have a way to figure out what happened and find a reproducible case.

Yeah, I might not be able to blame shift on this one. :wink: I was coming to a similar conclusion. I had a number of bugs on the roll-out of the game comment editing feature, so it's not inconceivable that I missed some point of fragility here.
To be honest, I was worried about the fragility of editing SGFs directly: sometimes I wonder if I should have just broken out all the properties of an SGF file on upload, rather than trying to preserve them as SGFs.
As to MongoDB revisions, I don't know, actually: this is my first MongoDB project. I'll investigate.

Oh, and tildes shouldn't be a problem :frowning: I just tested them on a few comments on my local version, and they seem fine. Right now I can't trigger any kind of problem with comments... some kind of parsing error triggered by some kind of character or other is the most likely culprit. And if that's the case, it's most likely to be something I'm escaping, like a square bracket, quotation mark, or backslash.

Oh wait,

And if that's the case, it's most likely to be something I'm escaping, like a square bracket, quotation mark, or backslash.

By quotation mark, do you mean " ? Because I'm pretty sure I used them in a few instances.

Yep, but those all seem to come through just fine. Is there any chance you used a backslash? I think I can reproduce the problem with that. I'm starting to suspect this is actually another instance of the same bug we had last time, with a backslash right before a final closing bracket.

It's very possible I've used a backslash!

I think I've fixed this issue: it is, indeed, all about having a backslash before a closing bracket. In this case both my own code and WGo's were working together in an impressive way to break in this condition, which I didn't have in any of my test cases. It's still only on my dev box, but I'll try to get it committed and launched this evening.

Great stuff! Out of curiosity, how was it different from :\ ?

Well, the original error was with WGo breaking when it tried to parse :\. The other error you discovered was my own smartgame module breaking when it tried to write comments to the SGF that contained :\ for essentially the same regex-related reason. Every time it tried to write that comment, it didn't properly recognize the closing bracket, so it kept gobbling up text till it hit the next ending bracket, thereby devouring a single move every time such an edit was attempted.
This should now be fixed on live. I also tried to restore your SGF for you, along with the game comments I could: https://gokibitz.com/kifu/4JTw6I6r?path=66
I appreciate your help in finding bugs and giving me great details on the problem.

Brilliant! thanks :)


Question

https://github.com/populationgenomics/analysis-runner/blob/213703639ff240dde2da7b1fed82607d3a89cfa8/server/util.py#L260-L264
Should infrastructure.toml be repeated here?

Probably not haha


remove self from root .babelrc

When doing a fresh checkout the plugin referenced in the babel config
does not exist yet and the build fails.  This PR splits the .babelrc
file into two (one for the lib, one for the example).

thank you very much!


Add new Style/ConstantVisibility cop

This cop enables enforcing visibility scope for class- and module constants. Constants without a visibility declaration, e.g.:
class Foo
  BAR = 42
end

will register an offense, prompting the user to declare visibility:
class Foo
  BAR = 42
  private_constant :BAR
end

I decided to make it disabled by default, because I don't think many users have spent time caring about this. 😅

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[X] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[X] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[X] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

I decided to make it disabled by default, because I don't think many users have spent time caring about this. 😅

I know I certainly didn't. :-)

Btw, you should totally add something about this to the style guide, although there I would suggest using private_constant when needed, but not having to explicitly mark the public ones.


Update tox testing to advance the dev of this plugin

We have essentially duplicated the current testing configurations of
molecule-inspec which provides functionality testing and GitHub
Actions workflows.

Indeed looking at its sister plugin was a good idea.
This may require bit more work and review of current PRs, please put a comment on https://github.com/ansible-community/molecule-goss/issues/6 and I will send you and invite.


Avoid loading files from urls and s3/gcsfs

Instead add datasets to xr.tutorial.open_dataset (https://github.com/pydata/xarray-data/). That way we don't have to worry about broken links.

[ ] NOAA ERSST in computation notebooks: http://psl.noaa.gov/thredds/dodsC/Datasets/noaa.ersst.v5/sst.mnmean.nc
[ ] Mask datasets in Computation: http://iridl.ldeo.columbia.edu/SOURCES/.NOAA/.NODC/.WOA09/.Masks/.basin/dods
[ ] temperature gradient dataset in plotting notebooks

ds = xr.tutorial.open_dataset("air_temperature.nc").rename({"air": "Tair"})

# we will add a gradient field with appropriate attributes
ds["dTdx"] = ds.Tair.differentiate("lon") / 110e3 / np.cos(ds.lat * np.pi / 180)
ds["dTdy"] = ds.Tair.differentiate("lat") / 105e3
ds.dTdx.attrs = {"long_name": "$∂T/∂x$", "units": "°C/m"}
ds.dTdy.attrs = {"long_name": "$∂T/∂y$", "units": "°C/m"}

@dcherian thoughts on possibly adding a zarr dataset to xarray-data? i played around with this a while back here https://github.com/scottyhq/zarrdata

Zarr + fsspec should be a great demo. But I think that would be a more intermediate-level tutorial.
For the "fundamentals" notebooks I think it'd be nice to just do tutorial.load_dataset to minimize confusion. It also uses pooch for caching, so that's nice.


Black screen at startup

Had this issue both yesterday before, and today after the main branch of teardown was updated. I had used both the master branch and the 0.5.0 github branches both times. Today with the 0.5.2 update, I also used the most recent copy of the 0.5.0 GH branch.
After copying the folder into the teardown install directory and running the .bat file, when I start the game, I never even get to the white splashcreen.

Verify your game files in steam to clear out the old modloader and reinstall the latest version.
I have merged the 0.5.0 branch into master.

I had been completely uninstalling and clearing out orphaned files and
reinstalling teardown from steam each time with the same issue.
I'll try the new merges later today.
On Tue, Dec 22, 2020, 10:50 AM Thomas<EMAIL_ADDRESS>wrote:

Verify your game files in steam to clear out the old modloader and
reinstall the latest version.
I have merged the 0.5.0 branch into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Thomasims/TeardownUMF/issues/5#issuecomment-749651896,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG65B4CGM7S4EOOJY3F5Z3LSWDE4NANCNFSM4VFVNJWA
.

I have been trying to install this for my son and getting the same black screen?  Any chance you figured out what was happening?

I have been trying to install this for my son and getting the same black screen?  Any chance you figured out what was happening?

I think I am getting the same

I think I am getting the same

I can't reproduce this, I've reinstalled UMF multiple times on both experimental and main without seeing this.
The only times I've gotten a black screen on startup is when the base game files were broken by an outdated mod.
I won't be able to help without someone providing reproducible steps to get this issue.

I can't reproduce this, I've reinstalled UMF multiple times on both experimental and main without seeing this.
The only times I've gotten a black screen on startup is when the base game files were broken by an outdated mod.
I won't be able to help without someone providing reproducible steps to get this issue.

This week, I just deleted the whole thing (game, mods, umf, everything) and
tried with new, latest versions and the game launches now.
On Sat, Jan 16, 2021, 6:18 PM Thomas<EMAIL_ADDRESS>wrote:

I can't reproduce this, I've reinstalled UMF multiple times on both
experimental and main without seeing this.
The only times I've gotten a black screen on startup is when the base game
files were broken by an outdated mod.
I won't be able to help without someone providing reproducible steps to
get this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Thomasims/TeardownUMF/issues/5#issuecomment-761702837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG65B4A7QKOYIID765ATHGTS2IUFPANCNFSM4VFVNJWA
.

This week, I just deleted the whole thing (game, mods, umf, everything) and
tried with new, latest versions and the game launches now.
On Sat, Jan 16, 2021, 6:18 PM Thomas<EMAIL_ADDRESS>wrote:

I can't reproduce this, I've reinstalled UMF multiple times on both
experimental and main without seeing this.
The only times I've gotten a black screen on startup is when the base game
files were broken by an outdated mod.
I won't be able to help without someone providing reproducible steps to
get this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Thomasims/TeardownUMF/issues/5#issuecomment-761702837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG65B4A7QKOYIID765ATHGTS2IUFPANCNFSM4VFVNJWA
.

Also how will I know my version is 0.4.6 or 0.5.0

Also how will I know my version is 0.4.6 or 0.5.0

Also how will I know my version is 0.4.6 or 0.5.0

You will find the version number in the bottom right corner.

Also how will I know my version is 0.4.6 or 0.5.0

You will find the version number in the bottom right corner.

Thank you, it says 0.5.2 but it still does not work?

Thank you, it says 0.5.2 but it still does not work?

I am waiting..

I am waiting..

so i got an notification from my antivirus and it said that Teardown.exe a virus was after i installed the UMF it said Virus Detected Name of Virus: IDP:Generic trojan
can you explain that??
and now i cant remove it from my antivirus quarantaine

so i got an notification from my antivirus and it said that Teardown.exe a virus was after i installed the UMF it said Virus Detected Name of Virus: IDP:Generic trojan
can you explain that??
and now i cant remove it from my antivirus quarantaine

@SimonPlayz UMF does not modify teardown.exe in any way so you have a bigger problem, check the code yourself if you don't believe me.
@TeardownFan Go through the installation guide again and make sure you use the correct paths. If that still doesn't work seek help on the Teardown discord as you'll get quicker help there...
This issue has gone too far off track so I'm going to close it as the original problem is solved. If an actual Issue with UMF still persists please open a new issue.

@SimonPlayz UMF does not modify teardown.exe in any way so you have a bigger problem, check the code yourself if you don't believe me.
@TeardownFan Go through the installation guide again and make sure you use the correct paths. If that still doesn't work seek help on the Teardown discord as you'll get quicker help there...
This issue has gone too far off track so I'm going to close it as the original problem is solved. If an actual Issue with UMF still persists please open a new issue.


Smart Throttle: Skip cancellation when the request is at the start of a word range

Description
WIP
See comment: https://github.com/sourcegraph/cody/pull/4735#discussion_r1662597756
Test plan

Possible future work, but closing for now to clean up my open PRs


label matchers in labels API actually not work

I am using thanos 0.22.0 in work, using the prometheus api /api/v1/label/<label_name>/values  query label values, but thanos query always return all label values no matter what i set the match[] field, source code below.
package main

import (
	"fmt"
	"io"
	"net/http"
	"net/url"
	"os"
	"time"
)

func main() {
	q := url.Values{
		"match[]": []string{`kube_node_labels{cluster="cluster1"}`},
		"start":   []string{fmt.Sprintf("%d", time.Now().Add(time.Hour*-1).Unix())},
		"end":     []string{fmt.Sprintf("%d", time.Now().Unix())},
	}
	fmt.Println(`http://<IP_ADDRESS>:8080/api/v1/label/node/values` + "?" + q.Encode())
	req, err := http.NewRequest(http.MethodPost, `http://<IP_ADDRESS>:8080/api/v1/label/node/values`+"?"+q.Encode(), nil)
	if err != nil {
		panic(err)
	}
	req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
	resp, err := http.DefaultClient.Do(req)
	if err != nil {
		panic(err)
	}
	defer resp.Body.Close()
	io.Copy(os.Stdout, resp.Body)
}

after spending sometime digging into the source code.
according to this PR，we can use matcher to select the label values. the implementation use http.Request.From to get field match[] without calling ParseForm.  http.Request.From only available after ParseForm is called.  which meaning the r.Form always be empty map.
type Request struct {
        // Form contains the parsed form data, including both the URL
	// field's query parameters and the PATCH, POST, or PUT form data.
	// This field is `only available after ParseForm is called`.
	// The HTTP client ignores Form and uses Body instead.
	Form url.Values
}

maybe we can just call ParseFrom before get match[] to fix it.

Umm, I just tested this and it is working well in my environment.
For the code you mentioned, during https://github.com/thanos-io/thanos/blob/main/pkg/api/query/v1.go#L516, when we parse time params https://github.com/thanos-io/thanos/blob/main/pkg/api/query/v1.go#L938, we will call r.FormValue so the form should be parsed anyway.
I will close this issue as this is not a bug. Please check out your code and environment.


[5.1] Doc-serialization: skip declarations with double-underscore as name prefix

Double-underscored names suggest the symbols aren't supposed to be used by framework
clients. This patch excludes the doc-comments of these symbols in swiftdoc files.
rdar://51468650

@swift-ci please test


Block edge 4.3.40

4.3.40 needs to be disabled:
4 clusters currently failing (18%),  4 gone (18%),and 13 successful (59%), out of 22 who attempted the update over 7d
Currently failing classification:
* Down nodes: 1
* Node pressure: 1
* SDN issue: 2
* Single master: 3
* Slow etcd: 1
* Untriaged: 1
* Version retarget: 1
* volumesnapshots v1alpha1: 1

4.3.40 can be enabled again:
1 clusters currently failing ( 4%),  4 gone (17%),and 17 successful (74%), out of 23 who attempted the update over 7d
Currently failing classification:
* Down nodes: 1
* Node pressure: 1
* SDN issue: 1
* Single master: 1
* Slow etcd: 1
* Version retarget: 1
* volumesnapshots v1alpha1: 1


Support for Safari

This PR brings support of Safari by using safari-web-extension-converter.  If you have macOS PC, you can release it on App Store; also, this article might be useful for you if you are highly motivated to do that. I would appreciate it if you could do that. If you do not want because of Apple Developer Program, I would like to install this as a Safari Web Extension personally, and I think README.md should be updated.
Moreover, one thing that you should know is that Safari has less storage than the other browsers even though you specified unlimitedStorage in manifest.json. Apple states the local storage limit is 5 MB, and unlimited storage permission increases limit to 10 MB (ref). However, this storage is so few that large-size dictionaries like EIJIRO-1448 could not have been loaded due to storage quota exceed.
regarding to: #25

Aside from lack of my current motivation to publish it on App Store, supporting Safari build sounds like a great idea. Let me try the Safari build soon on my MacBook.
By the way, could you please share "this article" again? The link looks broken.

@wtetsu
Thank you for your response, and I am sorry for sending you a broken link. The following link is what I wanted to send you.

https://developer.apple.com/safari/extensions

@ken-matsui
Hi, were you able to build the Actions' artifact dist-safari, and installed Mouse Dictionary on your Safari?
https://github.com/wtetsu/mouse-dictionary/actions/runs/450896512
I faced two types of errors.
Error 1
error: ~/dist-safari-tmp/options: No such file or directory

To place mouse-dictionary build directory on my home directory resolved it,
but I think this workaround is strange and not reasonable for normal developers.
Error 2
Unknown attribute 'main'
Probably, this is due to my old Xcode.

macOS 10.13
Xcode 10.2.1

@ken-matsui
Hi, were you able to build the Actions' artifact dist-safari, and installed Mouse Dictionary on your Safari?
https://github.com/wtetsu/mouse-dictionary/actions/runs/450896512
I faced two types of errors.
Error 1
error: ~/dist-safari-tmp/options: No such file or directory

To place mouse-dictionary build directory on my home directory resolved it,
but I think this workaround is strange and not reasonable for normal developers.
Error 2
Unknown attribute 'main'
Probably, this is due to my old Xcode.

macOS 10.13
Xcode 10.2.1

@wtetsu
I apologize for replying late.
Error 1
I also got the same error with Error 1. To build by safari-web-extension-converter, I had had to build to Chrome Extension (it is dist-safari-tmp) and then convert it using safari-web-extension-converter (it will be placed in dist-safari). However, I did not know dist-safari refers dist-safari-tmp, and this is the reason for Error 1. Therefore, I pushed a new commit to fix the problem. Could you please check it again?
Error 2
I did not see the same error, so it seems your Xcode is old. For your information, the following is my environment versions.

macOS 11.1
Xcode 12.3

@wtetsu
I apologize for replying late.
Error 1
I also got the same error with Error 1. To build by safari-web-extension-converter, I had had to build to Chrome Extension (it is dist-safari-tmp) and then convert it using safari-web-extension-converter (it will be placed in dist-safari). However, I did not know dist-safari refers dist-safari-tmp, and this is the reason for Error 1. Therefore, I pushed a new commit to fix the problem. Could you please check it again?
Error 2
I did not see the same error, so it seems your Xcode is old. For your information, the following is my environment versions.

macOS 11.1
Xcode 12.3

@ken-matsui
Thank you for resolving the issues.
1 -> I made sure it has been resolved
2 -> I succeeded to build it using @NSApplicationMain
Due to personal reasons, I cannot update macOS and Xcode,
but I made sure builds succeeded so I'll merge this Safari support as an experimental feature.

Before merging it, would you please delete .DS_Store?
(Please remove it from the history and force-push it)

@ken-matsui
Thank you for resolving the issues.
1 -> I made sure it has been resolved
2 -> I succeeded to build it using @NSApplicationMain
Due to personal reasons, I cannot update macOS and Xcode,
but I made sure builds succeeded so I'll merge this Safari support as an experimental feature.

Before merging it, would you please delete .DS_Store?
(Please remove it from the history and force-push it)

@wtetsu
Thank you so much for your review. I removed .DS_Store and force-pushed it.

@wtetsu
Thank you so much for your review. I removed .DS_Store and force-pushed it.

Thank you!
I've merge and added some information README.

Thank you!
I've merge and added some information README.


Add Fortify strings check to CI

https://github.com/falcosecurity/libs/pull/1880 should have been caught in CI. It happened to be detected because @FedeDP ran the test suite on a machine that had fortify turned on and strlcpy() was a checked version instead of the regular one that is embedded in the libs repo. We need to look into how to enable the same checks in CI as well (in the ASan build, so we can get a stack trace)

/milestone 0.18.0
This is great, thanks for opening Luca! Big enormous huge +1 from me.

https://github.com/falcosecurity/libs/issues/2022


:lipstick: (stylesheet) modify colorPalette to follow mock-ups

Check before merging

[x] Ticket : ETQD j'ai accès aux couleurs des maquettes dans la stylesheet
[x] I made sure ticket is up to date on Trello.

Closing PR because of failing CI (due to merge commits) -> all changed have been cherry picked in PR#19

Closing PR because of failing CI (due to merge commits) -> all changed have been cherry picked in PR#19


Robot update: communicate upgrades and downgrades more clearly

overview
As a user, I would like my app to warn me if my robot's software is out of sync with my app so that I may take some action depending on my needs (see below).
A user may want to:

Always be running the latest version of everything
Stay on one specific version without updating
Switch back and forth between beta and stable versions

Regardless of whether or not the app is fully up to date, if the user wishes to use that specific version of the app, the robot must be on a matching release to guarantee compatibility.
current behavior
The app will alert the user if there is a version mismatch between the app and the robot, but this alert is generic and always uses language that suggests the user is upgrading the robot, when they may, in fact, be downgrading the robot.
acceptance criteria
There are 4 scenarios in which a version mismatch could happen, and the app should message to the user accordingly:

[ ] The app cannot find an update and its version is greater than the API

App should prompt the user to either "upgrade" their robot (default)  or
Ignore the mismatch

[ ] The app cannot find an update and its version is lower than the API

App should prompt the user to "downgrade" their robot (default) or
Ignore the mismatch
Message should tell the user to ensure their computer is online so the app can ensure it is fully up to date
Example 1: Robot is on a beta version, app is on stable, needs to be downgraded to stable
Example 2: Robot was updated to latest stable by Bob, but Alice is currently connected over USB, running an older app, and isn't on wifi, so Alice's app doesn't know it's out of date

[ ] The app can find an update and its version is greater than the robot's

The app should prompt the user to go to the App info page to update their app (default) or
Upgrade their robot to the out-of-date version or
Ignore the mismatch

[ ] The app can find an update and its version is lower than the robot's

The app should prompt the user to go to the App info page to update their app (default) or
Downgrade their robot to the out-of-date version or
Ignore the mismatch

Re-opening pending final polish after design pass. Skeleton functionality / copy is in, though

Closing with current functionality/copy in favor of opening a separate design/UI pass ticket.


Document publishing snapshots and releases to different repositories

Resolves #5095.

I applied the proposed changes in the latest update of this PR.
@pledbrook PTAL

@pledbrook One more time, please. 🙂


Promeutheus and Grafana only shows traffic of eth0 even if I have multiple interfaces

I am trying to monitor a server with multiple network interfaces but I seems like only eth0 is being monitored.
How can I monitor all interfaces?
The Prometheus inspector shows me that my node is detecting all interfaces.

But is not exporting the network traffic stats

Which is also reflected in Grafana

The default Dashboard and deployment monitor the network usage of the node-exporter container.
If you are using https://github.com/stefanprodan/dockprom/blob/master/docker-compose.exporters.yml and intending to monitor multiple of your interfaces you need to also modify the Dashboard.


test data with seg_id_nat, date as indices

adding test data that has seg_id_nat and date as indices

Looks good to me. I'll pull the dataset and test it after it's merged.


UnityWebcam DLL Compilation problems？

I want to change some virtual camera attributes from the source code, such as camera name, resolution, etc.
I downloaded the source code and tried to compile the DLL. Unitycamservice can be compiled successfully. Another unitywebam DLL keeps reporting errors. Prompt sharedimagewrapper this does not recognize and syntax error. Which version of vs do you compile with and how to solve this problem? There are also virtual cameras in Tencent meeting without pictures. Do you have the latest version of DLL source code, can you send a copy?
Looking forward to your reply

I have the same problem. Any update?

Ok I found a solution (posted in a remote japanese article, yes, I was that desperate):
Add legacy_stdio_definitions.lib in the UnityCamService Project > Linker > Input > Additional Dependencies


Package request: GSEA

Project description
Gene Set Enrichment Analysis (GSEA) is a computational method that determines whether an a priori defined set of genes shows statistically
significant, concordant differences between two biological states
(e.g. phenotypes).

Metadata

homepage URL: https://www.gsea-msigdb.org
source URL:
license:
platforms: linux

Add a :+1: reaction to issues you find important.

For anyone wanting to use this

you can do it with distrobox, with ubuntu and so run.
I had little issue, then too it would run I guess.

using nix-ld

Download the zip and extract it, then put this code into shell.nix and nix-shell it:
with import <nixpkgs> {};
mkShell {

  NIX_LD_LIBRARY_PATH = lib.makeLibraryPath [
    stdenv.cc.cc
    openssl
    xorg.libXext
    xorg.libX11
    xorg.libXrender
    xorg.libXtst
    xorg.libXi
    xorg.libXfont2
    freetype
    fontconfig
    # ...
  ];
  NIX_LD = lib.fileContents "${stdenv.cc}/nix-support/dynamic-linker";

  nativeBuildInputs = [ pkgs.bashInteractive ];

  buildInputs = with pkgs; [
      xorg.libXext
      fontconfig
  ];
}


Office UI Fabric React usingatude

Category

[X] Question

As always, love the great work guys!
TL;DR
office-ui-fabric-react versioning documentation is confusing.
Should I include it explicitly or not in a project updated from v1.8.1 to v1.8.2?
What exactly do I have to do to include the latest release of office-ui-fabric-react into my solution? (if it is indeed 'safe' to do so)
Should I leave everything as is or should I add it to the package.json? (via npm ofc)
and if I do, do I need to update TypeScript as well?
Details
The v1.8.2 release notes have this to say on the subject of office-ui-fabric-react

For the react scaffolded projects, we now explicitly add office-ui-fabric-react version 6 to package.json. We were finding a common pattern where people were not adding OUIFR to package.json, but were using it.

Because it wound up in the node_modules because of downstream dependencies, the solution would compile, but later on when the solution was upgraded to use a different version of the framework, a different version of OUIFR would suddenly get used.
In general, using packages like this (via shadow dependencies) is a bad thing. You are more than welcome to remove the reference in package.json if you are not using OUIFR in your solution, or change the version if needed. We also bumped the typings for react and react DOM to match the versions of React / React DOM.

Yet, the official walkthrough has this to say:

With the current release of the SharePoint Framework, we recommend that you use the Office UI Fabric and Fabric React that ships with the generator. We do not recommend that you update the Office UI Fabric and Fabric React packages independently because that might conflict with the already available version in SharePoint, and as a result, your web part may fail to function as expected.

location in document
but it also says:

Starting with SharePoint Framework 1.8, you can use either Office UI Fabric version 5 or version 6. In this case we are using specifically Office UI Fabric version 5.132.0, so we are adding the needed dependency on it. If you would be using Office UI Fabric version 6.x, you'd also need to update the used TypeScript version of the solution.

just slightly above.
right now my package.json looks like this after updating to v1.8.2 from v.1.8.1:
{
  "dependencies": {
    "@microsoft/sp-core-library": "1.8.2",
    "@microsoft/sp-lodash-subset": "1.8.2",
    "@microsoft/sp-office-ui-fabric-core": "1.8.2",
    "@microsoft/sp-property-pane": "1.8.2",
    "@microsoft/sp-webpart-base": "1.8.2",
    "@pnp/common": "^1.2.9",
    "@pnp/graph": "^1.2.9",
    "@pnp/logging": "^1.2.9",
    "@pnp/odata": "^1.2.9",
    "@pnp/sp": "^1.2.9",
    "@pnp/spfx-controls-react": "^1.13.0",
    "@pnp/spfx-property-controls": "^1.14.1",
    "@types/enzyme-adapter-react-16": "^1.0.5",
    "@types/es6-promise": "0.0.33",
    "@types/react": "16.7.22",
    "@types/react-dom": "16.8.0",
    "@types/webpack-env": "1.13.1",
    "react": "16.7.0",
    "react-dom": "16.7.0"
  },
  "resolutions": {
    "@types/react": "16.4.2"
  },
  "devDependencies": {
<EMAIL_ADDRESS>"0.7.7",
    "@microsoft/sp-build-web": "1.8.2",
    "@microsoft/sp-module-interfaces": "1.8.2",
    "@microsoft/sp-tslint-rules": "1.8.2",
    "@microsoft/sp-webpart-workbench": "1.8.2",
    "@types/chai": "3.4.34",
    "@types/enzyme": "3.1.15",
    "@types/jest": "22.2.3",
    "@types/mocha": "2.2.38",
    "@types/sinon": "5.0.7",
    "ajv": "~5.2.2",
    "enzyme": "3.8.0",
    "enzyme-adapter-react-16": "1.7.1",
    "gulp": "~3.9.1",
    "identity-obj-proxy": "3.0.0",
    "jest": "22.4.3",
    "jest-junit": "5.2.0",
    "react-test-renderer": "16.6.3",
    "sinon": "5.0.7",
    "ts-jest": "22.4.5"
  }
}

my package-lock.json has a reference to office-ui-fabric-react v6.143 which is the latest release.
Edit
As can be seen in the package.json I'm also using @pnp/spfx-controls-react v1.13.0 which uses office-ui-fabric-react v5.131.0. Seemingly that version is used when importing into my own components and it confirmed by looking at the node_modules.

Sorry for the confusion - we need to improve our documentation around this.  You should always explicitly include OUIFR in your package.json if you are using it.  Don't rely on it being there because another package brought it in.  If you were starting from scratch, I would suggest using 6.143.0 - it's the version that was current when the build was made.  There are newer versions now, and it should be OK to use them (don't use anything from version 7 though).
Given your use of PNP, I would put version 5.131.0 in your package.json.  That will ensure that all your react / fabric controls line up.

@patmill Thanks for your swift reply!
I have been experiencing some bugs with v5.131.0 that have been fixed in later releases of v6.* (specifically this one).
I admit don't have a full understanding of how npm deals with this sort of thing. But is there some way to use this more recent version of a specific control from OUIFR, namely the SearchBox in my case and still maintain v5.131.0 for the PnP solution? Or do you see a different way for me to satisfy this requirement?
Thanks for the help Pat, really appreciate it!

Let me get some people more knowledgeable in Fabric.  I believe it should work to use PNP (on 5.131) but have your package json and your code use 6, but I'll have them confirm.

@Katli95 first of all, thanks for reaching out and the cheer 👏. A few questions for help w/ debug:
Would you happen to have a repro branch you can share?
What version of office-ui-fabric-react do you plan to use? Latest is 6.181.0.
Am I understanding correctly that @pnp/spfx-controls-react is bringing in an older version of office-ui-fabric-react and you'd like to use latest at least in your app code.
TypeScript compatibility
You can view<EMAIL_ADDRESS>TypeScript compatibility at https://dev.azure.com/RushStack/Gearbox GitHub Projects/_build/results?buildId=866

I can run it against a specific version of the library if you'd like. Its running nightly since we publish daily.
cc: @aditima

@patmill do you think the above compatibility information would be a useful addition to https://github.com/SharePoint/sp-dev-docs/pull/3800?

@KevinTCoughlin Many thanks for the detailed response!
Unfortunately no, I'm not allowed to share the project :/
But yeah, your understanding is spot on! I'd like to use the latest version of office-ui-fabric-react but @pnp/spfx-controls-react is bringing in v5.*
Is @patmill correct in thinking I can add OUIFR v6.181.0 to my package.json and the package manager will handle the rest? i.e. @pnp/spfx-controls-react will use the old version and I can use the new?
Sidenote: I'm using SPFx v1.8.2 so I'm pretty sure I'm running TypeScript v2.9.

@Katli95 said:

correct in thinking I can add OUIFR v6.181.0 to my package.json and the package manager will handle the rest? i.e. @pnp/spfx-controls-react will use the old version and I can use the new?

I'm not sure you can do that... @estruyf (who's behind the PnP controls package), can you comment on this? I thought there was some effort to get the PnP package updated to the latest OUIFR... I don't believe you can mix the two versions at the same time...

Any solution/progress on this? O:)

@Katli95 You're probably best posting that specific question about the PNP library to it's own issue list as this is more for SharePoint / SharePoint specific dev stuff and the PnP library is a library provided by & for the community with no support from MSFT.

Great, will do! Thanks for the help, feel free to either close the issue or keep it open as a reference to where I found the docs to be a bit unclear :)


6512 Documentation follow up

Closes #6512
What's Changed

Screenshots

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example App Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

I compared args list from ObjectsTablePanel docstring with documentation and it looks ok. I only added couple lines of description into ObjectsTablePanel docstring with arguments overview.


Feature Request: Dark theme loot sheet.

Title says it all really, everything else looks great! I see the odd prompt come up in a light theme, but the loot sheet is the only major thing I've noticed.

Oh I actually did that already, but I just tried it and see that it's not working. Will make it work for next release.


Feedback buttons has disappeared

Describe the bug
Feedback buttons has disappeared
On development branch

on main

Looks like it is there, just moves a little bit down, so you have to scroll down to it


TabBarCoordinator Initialization

Memory leak during initialization，code：
`enum ASRootRoute: Route {
case home
case mine
case login
}
class ASRootTabCoordinator: TabBarCoordinator {
private let homeRouter: AnyRouter
private let mineRouter: AnyRouter
init() {
    let homeCoordinator = ASHomeCoordinator()
    let mineCoordinator = ASMineCoordinator()
    homeCoordinator.rootViewController.tabBarItem = UITabBarItem.init(title: "Home", image: UIImage(named: "home"), selectedImage: UIImage(named: "home_1"))
    mineCoordinator.rootViewController.tabBarItem = UITabBarItem.init(title: "Me", image: UIImage(named: "me"), selectedImage: UIImage(named: "me_1"))
    self.homeRouter = homeCoordinator.anyRouter
    self.mineRouter = mineCoordinator.anyRouter
    super.init(tabs: [homeCoordinator, mineCoordinator])
}

override func prepareTransition(for route: ASRootRoute) -> TabBarTransition {
    switch route {
    case .home:
        return .select(homeRouter)
    case .mine:
        return .select(mineRouter)
    case .login:
        let loginRouter = ASLoginCoordinator().anyRouter
        return .present(loginRouter)
    }
}

override func generateRootViewController() -> UITabBarController {
    let vc = ASBaseTabBarController()
    return vc
}

}
`
use instrument- leaks，finally：
`    // MARK: - Private methods
private func performTransitionAfterWindowAppeared(_ transition: TransitionType) {
    guard UIApplication.shared.keyWindow == nil else {
        return performTransition(transition, with: TransitionOptions(animated: false))
    }

    var windowAppearanceObserver: Any?

    rootViewController.beginAppearanceTransition(true, animated: false)
    windowAppearanceObserver = NotificationCenter.default.addObserver(
        forName: UIWindow.didBecomeKeyNotification, object: nil, queue: .main) { [weak self] _ in
        windowAppearanceObserver.map(NotificationCenter.default.removeObserver)
        windowAppearanceObserver = nil
        self?.performTransition(transition, with: TransitionOptions(animated: false))
        self?.rootViewController.endAppearanceTransition()
    }
}`

Memory leak problem arises，Just inside the above method

Since the provided code did not compile at all, I tested the following code with the Leaks tools of Instruments without any problems. Could you please eloborate how this coordinator is introduced into the view hierarchy and what the initializers of the child coordinators do? That might help with resolving this issue. Am i right with assuming that the issue arises when the app is just launched or is there any interaction needed until the problem arises? It might be connected with not calling presented(from:) on any coordinator brought into the view hierarchy (e.g. by using custom transitions or not using Coordinator.setRoot to set the rootViewController of the window).
enum ASRootRoute: Route {
    case home
    case mine
    case login
}

class ASRootTabCoordinator: TabBarCoordinator<ASRootRoute> {
    private let homeRouter: Presentable
    private let mineRouter: Presentable

    init() {
        self.homeRouter = UIViewController()
        self.mineRouter = UIViewController()
        super.init(tabs: [homeRouter, mineRouter])
    }

    override func prepareTransition(for route: ASRootRoute) -> TabBarTransition {
        switch route {
        case .home:
            return .select(homeRouter)
        case .mine:
            return .select(mineRouter)
        case .login:
            return .present(UIViewController())
        }
    }

    override func generateRootViewController() -> UITabBarController {
        let vc = UITabBarController()
        return vc
    }
}

Thank you very much for your reply， I will check it carefully， again

I streamlined my code to be the same as yours, and tested it and found:
Premise: I have two iPhones:
One is the iPhone6-p system is iOS12+
One is iPhone7 system when iOS10+
Normal on 6p devices;
Generate leaks on the 7 device;
So I guess, it has something to do with the version of the iPhone system;

I unfortunately don't have an iOS 10 device at hand.
Can you specify what is leaking exactly?

I'm closing this due to inactivity and lack of information about the issue. If this issue should still persist and more information can be provided about what is leaking and what could be the issue for the leaking memory, feel free to reopen, add comments or open additional issues.


ERROR: bind message has 2 result formats but query has 1 columns when using sqlx.Connect

I am getting ERROR: bind message has 2 result formats but query has 1 columns (SQLSTATE 08P01) when I do sqlx.Connect("pgx", connString).
My DSN looks like this: host=hostname user=user database=db password=password port=6432 sslmode=require

Does this happen if you use database/sql directly, eg. sql.Open("pgx", connString)

No, I tried it with database/sql and it worked fine.

This worked for me:
package sqlx

import (
	"os"
	"testing"

	_ "github.com/jackc/pgx/v4/stdlib"
)

func TestPgxConn(t *testing.T) {
	dsn := os.Getenv("SQLX_POSTGRES_DSN")
	db, err := Connect("pgx", dsn)
	if err != nil {
		t.Fatal(err)
	}
	if db == nil {
		t.Fatal("db was nil")
	}
}

Going to need more information to reproduce:  pgx version, sqlx version, go version, postgresql version, and a failing test case.

I'm confused. It worked for me as well for a few times after I created this issue. But most of the time, it's failing. I think it has something to do with test cases running concurrently. I have two packages and both of them have integration tests(both creates a connection to the db). The error is gone when I set the -p flag to 1 while running the tests.
But, I still don't understand what the error means?

I think I'm doing something wrong. I got this as well bind message supplies 2 parameters, but prepared statement \"\" requires 3
All my db methods are like this:
func (pg *pgdb) GetUser(ctx context.Context, tenantID int, userID string) ([]User, error) {
	sqlString := `
		SELECT
		column1,
		column2
		FROM users
		WHERE tenant_id=$1 AND user_id=$2 LIMIT 1
	`
	var user User
	err := pg.conn.QueryRowxContext(ctx, sqlString, tenantID, userID).StructScan(&user)
	if err != nil {
		if err == sql.ErrNoRows {
			return []User{}, nil
		}

		return []User{}, err
	}

	return []User{user}, nil
}

Assuming that pg.conn is an *sqlx.DB, this should work, and parallelism should not be a problem.  The error you're getting back is saying that you do not have enough parameters for the bindvars you've provided, but in this case you've used $1 and $2 and supplied 2.
Are you sure the error message is coming from this test and not from another one?  I'm afraid without a runnable test, I'm still not sure what is going wrong.

Here's the code that's failing:
package db

import (
	"os"
	"testing"

	_ "github.com/jackc/pgx/stdlib"
	"github.com/jmoiron/sqlx"
	"github.com/stretchr/testify/require"
)

var (
	postgresTestDB PG
)

func setup() {
	connString := os.Getenv("PG_TEST_DATABASE")
	conn, err := sqlx.Connect("pgx", connString)
	if err != nil {
		panic(err)
	}

	// Note: safe to suppress error here as it has been take care of above.
	db, _ := NewPG(conn)

	postgresTestDB = db
}

func TestMain(m *testing.M) {
	setup()
	ok := m.Run()
	postgresTestDB.Close()
	os.Exit(ok)
}

In the above screenshot, line no. 20 is where I'm am doing sqlx.Connect

I get the same error even with database/sql. Here's the complete error:
pgx.PgError(pgx.PgError{Severity:"ERROR", Code:"08P01", Message:"bind message has 2 result formats but query has 1 columns", Detail:"", Hint:"", Position:0, InternalPosition:0, InternalQuery:"", Where:"", SchemaName:"", TableName:"", ColumnName:"", DataTypeName:"", ConstraintName:"", File:"pquery.c", Line:643, Routine:"PortalSetResultFormat"})
Surprisingly, it works completely fine sometimes.

My mistake, I was using the older version of pgx. Changed my import from github.com/jackc/pgx/stdlib to github.com/jackc/pgx/v4/stdlib and it worked fine.


Remove unnecessary GitHub Graph info

This code is no longer needed now that the GitHub dependency graph shipped support for pyproject.toml: https://github.com/orgs/community/discussions/6456#discussioncomment-5244057
I didn't include a changelog mention as this is not a user-facing change.

Optionally, could simply nuke setup.py entirely, as most folks have moved to pip install, and those who haven't probably have a very specific reason/know how to handle the error that happens when they try it.
cc @ofek as I spotted this in your PR here: https://github.com/jazzband/django-debug-toolbar/pull/1690/files#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7R18-R23
I was gratuitously plagiarizing from it to update some other OSS Python libs when I saw this bit of code... I know you've updated a ton of other repos, so just FYI.

Great to know, thank you!


Alternative to Google Hangouts

Feature requests:

Limited # of people can join a room.
Audio, screen sharing, and chat capacity.

This is a stretch goal, but open to anyone who wants to play with it!

https://talky.io

https://rtc.io/modules.html


ENYO-442: Add a method to temporarily suppress events for a given set of code.

Issue
There were several instances where we needed to surround code with a pair of silence/unsilence calls to temporarily suppress event firing.
Fix
We've implemented a muzzle method (I'm not married to the choice of name) that takes either a string method name, or a function, and surrounds its execution with a silence and corresponding unsilence call.
Enyo-DCO-1.1-Signed-off-by: Aaron Tam<EMAIL_ADDRESS>

Closing since Jira has been closed and was decided that these changes are not needed as of now.


introduce an optional root level controller field

This commit introduces an optional root level controller field
which lets the user define the Kubernets controller to use for the
application.
If no controller is specified, the deployment controller is set.

@surajssd PTAL, once this is in, will refactor jobs. TIA.

@surajssd thanks :)


Added links to dev blog for rate limit headers

Category

[ ] Content fix
[ ] New article
[x] Example checked item (delete this line)

DELETE THIS LINE BEFORE SUBMITTING | For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.

Related issues

fixes #issuenumber
partially #issuenumber
mentioned in #issuenumber

DELETE THIS LINE BEFORE SUBMITTING | If this fixes (aka: closes) or references an issue, please reference it here. This helps maintaining the issue list as it will (1) link the PR to the issue & (2) automatically close the issue when this PR is merged in.

What's in this Pull Request?

DELETE THIS LINE BEFORE SUBMITTING | Please describe the changes in this PR. Sample description or details around bugs which are being fixed.

Submission guidelines

!!IMPORTANT!! - All submissions must complete the baseline sections included in this template. Ignoring or deleting this template may result in closing the issue with the label type:incomplete-submission.
Follow our guidance on How To Create Good Pull Requests.
Target the main branch of this repo.
When changing a page, ensure you update the ms.date front matter wih the current date in the format MM/DD/YYYY.
Review all build checks and address the automated errors, warnigns, and suggestions.
NOTE: The live site is based on the live branch. Site owners periodically refresh live branch from the main branch so merged PRs won't immediately appear on the live site. Please be patient to see your changes appear on the live site.

DELETE THIS SECTION BEFORE SUBMITTING

Learn Build status updates of commit cbb30e9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for the submission... in future PRs, please complete the included PR template form that's provided when you open a new PR. This helps us review submissions in a more timely manner. Future submissions may be rejected if the form isn't completed.

Learn Build status updates of commit 893e404:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2144

Questions ?
Please refer to the Backport tool documentation


[r] Harmless (?) but noisy nag from compiler

I see
nanoarrow/nanoarrow_testing.hpp: In member function ‘ArrowErrorCode nanoarrow::testing::TestingJSONWriter::WriteType(std::ostream&, const ArrowSchemaView*)’:
nanoarrow/nanoarrow_testing.hpp:351:15: warning: variable ‘type’ set but not used [-Wunused-but-set-variable]
  351 |     ArrowType type;
      |

and in
  ArrowErrorCode WriteType(std::ostream& out, const ArrowSchemaView* field) {
    ArrowType type;
    if (field->extension_name.data != nullptr) {
      type = field->storage_type;
    } else {
      type = field->type;
    }

    out << "{";

    switch (field->type) {
    // [...]

type is never used.  Did you mean to switch on type, not field->type ?  If not the six lines at the top can probably go.

Appreciate it -- but I don't see it changed in my checkout or in the repo here: https://github.com/apache/arrow-nanoarrow/blob/6523f511799537ac1dc3b210d8e804caff02edd9/dist/nanoarrow_testing.hpp#L350-L357

Gotcha.


Is this proj dead?

^

I check to the etherscan api docs regularly.
This package will be updated when changes in the current implementation and etherscan api


Add plain no std example

This PR adds a no_std example with working qemu Cortex-M3 target configuration and runner config.
Currently it makes only very limited use of edhoc-rs, but should be a good base for more no_std testing.
This was previously at https://github.com/kaspar030/edhoc-rs-no_std. The workspace level .cargo/config.toml previously prevented correct building from inside this (edhoc-rs) repo, which I didn't realize, so I had put it somewhere else.
Without that file (which is IMO not needed there), the example can be moved here.

rebased
[patch] removed
added github workflow for building the test. broke my promise about the matrix. :)


[WIP]: QOL symexp improvements

Fixes #1160
Developers certificate of origin

[ ] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

Branches off #1159, must be merged after


Feature request: objectives of nested rules

Hello,
While I'm at it, I will also address one last issue i had while implementing Sigh.
It's not really a bug, more like a feature request concerning nested rules.
I often had to use nested rules in my implementations.
For example in binary expressions:
// Find the types of operands
R.rule()
    .using(node.left.attr("type"), node.right.attr("type"))
    .by(r -> {
        // Retrieve their declarations
        DeclarationNode leftDecl  = ((Type) r.get(0)).decl();
        DeclarationNode rightDecl = ((Type) r.get(1)).decl();

        // Retrieve their scopes
        R.rule()
            .using(leftDecl.attr("scope"), rightDecl.attr("scope"))
            .by(r2 -> {
                Scope leftScope  = r2.get(0);
                Scope rightScope = r2.get(1);

                // Lookup the operator in the scopes
                FunOverrideList leftOverloads  = leftScope .lookupLocalFun(node.operator.toString());
                FunOverrideList rightOverloads = rightScope.lookupLocalFun(node.operator.toString());

                // Get dependencies (types of all functions in both overload lists)
                // [...]

                // Get the types of the dependencies
                R.rule(node.attr("opType"), node.attr("type"))
                    .using(dependencies)
                    .by(r -> {
                        // Check overloads to find one that matches the types of the arguments
                        // [...]

                        // Only then, we can set the type of the binary operation
                        r.set(0, funType);
                        r.set(1, funType.getReturnType());
                    });
            });
    });

Here, the first two rules are declared without any exports because they don't set any attribute. Trying to add an attribute in the rule() call will create an error because the attributes are not actually set in the rule itself, which only creates a new rule for later.
Thus, only the third one knows it wants to compute the type of the operation.
However, if one of the dependencies of the first rules cannot be provided, for example the type of one of the operands, Uranium will report a missing attribute error because it doesn't know that the rule aims to compute that particular attribute.
For example:
return 5 + x

Would return two errors:

Could not resolve 'x'
missing attribute (BinaryExpression(5 + x) :: type)

The way to prevent this would be to add the binary expression in the errorFor call that reports the resolution error on x. However, this requires each node to take its parents into account when reporting an error, which is not ideal.
Would it be possible to add a new feature to Uranium that adds the ability to "tell" a rule that an attribute will be computed by a nested rule ? This way, when the type attribute is missing on the operand, it won't report an error on the operation since it knows that it requires the operand to be computed.
R.rule()
    .using(node.left.attr("type"), node.right.attr("type"))
    .objective(node.attr("type"))
    .by(r -> {
        // [...] Nested rules
    })

Please note that if you agree with the idea of such a feature, I would be glad to help implementing it.
Thank you for your time,
Martin Danhier

There's something about the premise of this I don't get: if you don't derive an attribute that you need somewhere else, that's an error right? So it seems like it's working as intended. It doesn't matter which rule failed to fire to supply it, in the end it's still missing.
In fact, if I remember correctly (but it's been a long time), errors for missing attributes are only computed at the very end, when the reactor is "stuck" with no rules that it can execute.
Am I misunderstanding you somewhere?
Or maybe your point is about better tracing the cause of the error? Normally Uranium should be able to chain errors to link missing attributes with the rules that failed to fire and could have provided them. But in your 5 + x example, the inner rules are never even enqueued, hence Uranium does not know that that's why the attribute wasn't computed. If that's your point, then I get it, and I do think it could make sense to specify attributes computed by nested rules as a way to ease debugging.

Hello, thank you for your answer.
In simple cases, if a rule that exports something cannot run because of missing dependencies, Uranium will not trigger a missing attribute error on that export because it is normal that it couldn't get it:
private void parenthesized(ParenthesizedNode node) {
    R.rule(node, "type")
        .using(node.expression, "type")
        .by(Rule::copyFirst);
}

For example, with the above code and the input (x) where x couldn't be resolved, no error is produced for the parenthesized expression because Uranium knows that the reason is the lack of the dependency.
To do this, when no rule is solvable anymore, the reactor currently saves the untriggered exports in a set, and will not throw semantic errors for them:
List<Rule> untriggeredRules = dependencies.values().stream()
    .flatMap(Collection::stream)
    .filter(rule -> rule.unsatisfied > 0) // haven't run
    .filter(rule -> Arrays.stream(rule.dependencies)
        // exclude rules with error as dependencies
        .noneMatch(dep -> attributes.get(dep) instanceof SemanticError))
    .collect(Collectors.toList());

Set<Attribute> untriggeredExports = untriggeredRules.stream()
    .flatMap(rule -> Arrays.stream(rule.exports))
    .collect(Collectors.toSet());

untriggeredRules.stream()
    .flatMap(rule -> Arrays.stream(rule.dependencies))
    // get missing dependencies that cannot be obtained indirectly via an untriggered rule
    .filter(dep -> attributes.get(dep) == null && !untriggeredExports.contains(dep))
    .forEach(dep -> setValue(dep,
        new SemanticError("missing attribute " + dep, null, dep.node)));

However, this does not work for nested rules. For example, let's imagine that for some reason, we now need to fetch the scope of the declaration of the type of the value:
private void parenthesized(ParenthesizedNode node) {
    R.rule()
        .using(node.expression, "type")
        .by(r -> {
            Type type = r.get(0);

            R.rule(node, "type")
                .using(type.decl(), "scope"))
                .by(/* ... */)
        });
}

Here, if the type of the child is missing, since there is no export in the first level rule, the type of the parenthesized expression will not be added in the untriggeredExports set, and a missing attribute: ParenthesizedNode :: type will be created.
With the feature I suggest, the "objective attribute" could be added in the untriggeredExports set even if it is not derived instantly in the rule but later, for instance by a nested rule.

Re-reading this, I agree it would be great to have this.
We could make it a clause like using. I'm tinking of shadows(Attribute...) meaning that if the rule doesn't execute, it "shadows" (hides) the computation of the specified attributes.
Don't have the bandwidth to implement it right now though!


refactor: Refactor Events and events.json

Summary

Refactored the events page and cleaned up the HTML for consistency and edge cases
Simplified / converted manual updating of events from current to past with filtering
NEW EVENTS MUST BE OF THE FORMAT: [DAY OF WEEK], [MONTH] [DAY (with or w/o suffix)], [YEAR]

e.g.: Friday, June 23rd, 2023, Thursday, October 12, 2023, Friday, Oct 12, 2023
Or, more specifically, a Date object parsable string must be preceded by [DAY OF WEEK],  (comma and space included)

Code for formatting times:
    const eventDate = new Date(
      event.time
        .split(" ")
        .slice(1) // Along with split, it retrieves the (hopefully) Date-parsable string
        .join(" ")
        .replace(/(\d)(st|nd|rd|th)/, "$1") // Removes date suffixes for parsability
    );

Admittedly, this simplification of the events.json structure is nice, but it enforces a very rigid time/date structure. If that's an issue, I can investigate other ways of refactoring/handling events.json!
Page Preview

Test Plan

New events, of the correct time format, should properly display under upcoming events.
Past events, of either correct time format or not, should properly display under past events.
The modals and all related details (look, content, show/hide) should still work as intended

Issues
Closes #87

Just popping by since this seemed like an interesting PR; I like the idea behind it but have a couple suggestions for the execution. Instead of storing event dates in events.json as formatted strings like Friday, June 23rd, 2023, I would suggest storing them as ISO date strings like 2023-06-23 and formatting them how you want in the front-end using the toLocaleString method and an options object. This would make it super easy to compare event dates with each other and today’s date, and would remove the complexity of creating your own way to obtain the date object from a formatted string.
I also noticed, while looking at events.json, that neither of our methods work for multi-day events like WebJam, so I’m thinking we could get around this by splitting up the time property in each event object into separate startTime and endTime properties, and figure out a good way to represent it in the front-end depending on whether it’s a single or multi-day event. Might be something for a future PR though, since it’s a bit more complex

Just popping by since this seemed like an interesting PR; I like the idea behind it but have a couple suggestions for the execution. Instead of storing event dates in events.json as formatted strings like Friday, June 23rd, 2023, I would suggest storing them as ISO date strings like 2023-06-23 and formatting them how you want in the front-end using the toLocaleString method and an options object. This would make it super easy to compare event dates with each other and today’s date, and would remove the complexity of creating your own way to obtain the date object from a formatted string.
I also noticed, while looking at events.json, that neither of our methods work for multi-day events like WebJam, so I’m thinking we could get around this by splitting up the time property in each event object into separate startTime and endTime properties, and figure out a good way to represent it in the front-end depending on whether it’s a single or multi-day event. Might be something for a future PR though, since it’s a bit more complex

:)) Thanks for the input! Simplifying the event date storage is definitely a smarter implementation, and I'll have that pushed ASAP!
Do agree on the multi-day point (and the scope), so I'll kick that down the road to another Issue/PR!


python3Packages.jupyterhub-ldapauthenticator: 1.3.1 -> 1.3.2, fix build

Motivation for this change
noticed it was broken
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

WFM linux x86_64


Scroll changes - allow fling to slide off left and top; allow scroller to be replaced

This changes the logic in ZoomPanLayout#onFling to use #getScrollMinX and #getScrollMinY to decide how far to the left and top it will scroll instead of using hard-coded 0.

LGTM.  @peterLaurence ?

@moagrius i will take the time to look at this tomorrow morning.

i'm ok with the changes in the last commit, but i don't understand why we should instantiate the Scroller in the getter, while there's no setter for it. In other words, in the main branch, there is no way to change the Scroller and internally referring to the private field makes sense.
In the proposed changes there is no way either to change the Scroller.
@MisterRager  can you explain why you did this?

Well i have responded to myself..  to change the Scroller, one can override the getScroller method.
It's just that i would have expected a setter for that (which means new api). I fell like forcing the user to extend ZoomPanLayout just to change a Scroller is a bit too much.

I appreciate the changes, but why only top and left and not down and right?
To get support for all four directions the user would have to override not only getScrollMinX() and getScrollMinY() but also getScrollLimitX() and getScrollLimitY().
Also, when using MinimumScaleMode.FIT and setScale(0f) the content is not centred, but shifted to the bottom right.

@peterLaurence that's fair. Should I add a setter, as well?

@MisterRager if you add a setter, then the default Scroller instance should be set in the constructor. The getter would just return the current instance. Then, the new setScroller would be a new api, and this should be reflected by a version bump. @moagrius what do you think?
If you don't change anything, well.. i won't be angry :)

yeah i'm fine either way :)

I'd figured that since the scroll bounds are overridden by extension rather than setters, it isn't that much more painful to override scroll behavior by the same.

Ok then.

@MisterRager  what is the purpose of getScroller()?
According to the Android Documentation there is no subclass.


Swap unsafes in variable name

Solve https://github.com/elm-lang/elm-compiler/issues/1475 in different way
Ref: https://github.com/elm-lang/elm-compiler/pull/1481
In Github, including elm-lang repos, there are many repos with name having . character.
However, like - character, . makes invalid variable name in javascript.
At least I think, swapping these is more fundamental solution for dot problem.

I didn't read https://github.com/elm-lang/elm-compiler/pull/1476. Sorry.
Discussion in the pull request is fair enough.


TestStreamSubscriptionHandle.ResumeAsync NotImplementedException

Hi,
We use TestKitBase for unit testing grains, but I am encountering when attempting to call a method that resumes handlers in the grain, a NotImplementedException on TestStreamSubscriptionHandle.ResumeAsync. It makes sense when I read the code, since it is actually not implemented and set to throw this exception. But is there a reason for this? Are we not meant to call grain code that resumes stream subscriptions from a unit test, or what am I missing?
Best regards

tbh, I think these weren't implemented because we only use implicit subscriptions in our projects and always call IAsyncStream<T>.SubscribeAsync. 👀🙂
To add support for ResumeAsync, I think we need a new method on TestStream that creates a TestStreamSubscriptionHandle not yet associated with an observer. In ResumeAsync, we'd have to find a way to pass the observer back up to the TestStream. This might also necessitate a change in the unsubscribe callback.

#114 appears to have closed this


[bug] getDerivedHDAddress and getDerivedHDAddressByPath return keys, not addresses

Bug Report
Current behavior:
In libdogecoin 1.2, the functions getDerivedHDAddress and getDerivedHDAddressByPath return keys, not addresses. Compare the functions that populate the outparams to that of getDerivedHDAddressFromMnemonic.
Expected behavior
Based on the function names, I expect all three functions to return values of the same type. While I see the value of returning public keys, I'd rather use different functions to do so.
Note that the tests explicitly test for this behavior.

yeah thanks for pointing that out. we're definitely going to update the named function misnomers in 0.1.3.


Add health check endpoint for U-DS catalog

Create an SSM parameter following UCS provided naming convention /unity/healthCheck/<MARKETPLACE_ITEM>/<COMPONENT_NAME>/url and store [cloudfront]/am-uds-dapa/collections endpoint as URL
For UDS SSM parameter name will be /unity/healthCheck/shared-services/data-catalog
Assumptions:

UCS is able to provide Cognito auth when hitting endpoint
UCS will propagate the SSM parameter from shared service into venue accounts (TBC)

Hi @ngachung
Is this /unity/healthCheck/<MARKETPLACE_ITEM>/<COMPONENT_NAME>/url enough?
In DEV, we have sbx and dev.
Both would end up in the same SSM param.
I guess other environments would only have 1, but there can be more than 1 in theory.

Let's assume there is one. For dev we would populate SSM but for sbx we would not, so optional SSM.

The following SSM parameter /unity/healthCheck/shared-services/data-catalog/url is now avaiable in shared services dev, test, and prod.


Elemental - TypeError when creating a machine registration

Setup

Rancher version: latest (2.6.6)
For the dashboard, I used https://github.com/aalves08/dashboard/tree/os-elemental-resources

Describe the bug
Create a new machine registration:

After clicking on Create , you should see these errors in the dev web console:
client.js:103 TypeError: Cannot read properties of undefined (reading 'registrationURL')
    at Proxy.render (elemental.cattle.io.machineregistration.vue?7a3c:44:1)
    at Vue._render (vue.runtime.esm.js:3569:1)
    at VueComponent.updateComponent (vue.runtime.esm.js:4070:1)
    at Watcher.get (vue.runtime.esm.js:4495:1)
    at Watcher.run (vue.runtime.esm.js:4570:1)
    at flushSchedulerQueue (vue.runtime.esm.js:4326:1)
    at Array.<anonymous> (vue.runtime.esm.js:1989:1)
    at flushCallbacks (vue.runtime.esm.js:1915:1)

xhr.js:178          PUT https://<IP_ADDRESS>:8005/v1/userpreferences/user-nvz67 500 (Internal Server Error)

Sometimes it does not impact the user experience but sometimes a web page is shown with the issue.
I will try to add a screenshot with the wrong web page.

In Cypress, even if the test passes, I can see this screen:

@juadk thanks for reporting the issue 🙏 Just a heads up, if you are automating some tests, you are working against code that hasn't been reviewed nor merged so it may have changes to the code that can break your work.

yeah I know but it will be quickly fixable, I prefer start earlier and already get a code base. I will do the same for the CI waiting the push of your dev branch as Richard advised.

This one is no longer happening, the CI is green every night so let's close it for now.
I will open it again if needed.


Allow to open PR on web from PR view when showing multiple

Here I really miss a way to click on the 🌐 icon to open the PR externally:

@bpasero you just want to open github.com to the PR?

Yeah like this one:

Hey there. I would like to contribute to this project and this issue would be a good starting point.
I have build the extension locally. For opening the pr on github, we can modify the menus -> view/item/context to use the existing pr.openPullRequestOnGithub command in package.json file.

I am happy to send a pull request with the required changes.

It shows here when there's only one:

Ah yes I see


Collapse accordion based on scroll

Open each accordion section based on how far the reader has scrolled down the page.
I.e. once the info section has been scrolled past, collapse it and expand the education section.

Blocked by #2
Require the accordion to be made first.


Add a license

The readme says open source, though without clear boundaries people can't really know what they're allowed to do with it. You can choose a license and add it to the repo. Team ARG usually picks MIT although I think GPL does a better job at preventing people from taking advantage of the work, because any improvements they make they're forced to share back.

Hi @alexgleason thank you for this request.
We've added an MIT license, as it best suits the community.
https://github.com/ModusCreateOrg/evade2/pull/343


Add rolling-upgrade support for Rancher CLI with compose

With DNS Service and LB Service, a user can easily perform a green/blue upgrade scenario.  However, we need to support the capability for Rancher CLI/UI to perform a rolling upgrade of a service A to service A'.  It should support by percentage or by number with some type of time interval of bringing down containers from service A and bringing up the same # from service A'.

Closing issue as the primary feature is complete.  Any issues found should be filed separately.

This is in v0.30.0 but I'm not sure if it is documented in http://docs.rancher.com/rancher/rancher-compose/ yet?

@flaccid The forums don't do a great job of creating yaml files or fenced code blocks. The documentation will be added shortly!
syntax to use upgrade: rancher-compose upgrade service1 service2 <upgrade options>
Docker-Compose.yml Example
service1:
service2:
  image: ubuntu:14.04

+1

+1 this is gold. Giving this a try today!


[bugfix] Added version header comments

fix of issue

Coverage increased (+0.2%) to 88.572% when pulling 57e2e00dd76990ddd886d46de8b833d25a6cadb8 on manjeet-thadani:bugfix/header-comments into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Coverage increased (+0.2%) to 88.572% when pulling 57e2e00dd76990ddd886d46de8b833d25a6cadb8 on manjeet-thadani:bugfix/header-comments into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

Coverage increased (+0.2%) to 88.572% when pulling 57e2e00dd76990ddd886d46de8b833d25a6cadb8 on manjeet-thadani:bugfix/header-comments into 13a61b285c095bda7ea8e33156090ea5ccfeaef1 on moment:develop.

An attempt, not a fix: https://github.com/moment/moment/issues/5057#issuecomment-618592588


AssertionError in protocol.py

I have a client program that occasionally receives an AssertionError. This occurs with both websockets 7.0 and websockets 8.0. Following is the stack trace for the latter version:
Traceback (most recent call last):
  File <snip>, line 34, in connect
    async for message in websocket:
  File "/usr/local/lib/python3.6/dist-packages/websockets/protocol.py", line 414, in __aiter__
    yield await self.recv()
  File "/usr/local/lib/python3.6/dist-packages/websockets/protocol.py", line 482, in recv
    assert self.state in [State.CLOSING, State.CLOSED]
AssertionError

When this occurs, self.state equals State.OPEN and self.messages equals zero. There does not appear to be any relationship between the occurrence of this AssertionError and the rate of messages received (i.e., it has occurred during times of both light and heavy message traffic).

I enabled logging in protocol.py and added log messages just before and after the failing assertion. I also added timestamps to all of the existing log messages (not shown in the following diff output) to determine the likelihood of a race condition:
45a46
> from time import time as now
482c483,488
<         assert self.state in [State.CLOSING, State.CLOSED]
---
>         logger.debug("%s - state = %s prior to assert [%f]", self.side, self.state.name, now())
>         try:
>             assert self.state in [State.CLOSING, State.CLOSED]
>         except:
>             logger.debug("%s - state = %s assert failed [%f]", self.side, self.state.name, now())
>             raise

This is the log output when an AssertionError occurred (the final log message is printed by my code):
DEBUG:websockets.protocol:client - event = eof_received() [1563292611.387163]
DEBUG:websockets.protocol:client ! failing OPEN WebSocket connection with code 1006 [1563292611.387561]
DEBUG:websockets.protocol:client - state = OPEN prior to assert [1563292611.387824]
DEBUG:websockets.protocol:client - state = OPEN assert failed [1563292611.387904]
DEBUG:websockets.protocol:client - event = connection_lost(None) [1563292611.388148]
DEBUG:websockets.protocol:client - state = CLOSED [1563292611.388213]
DEBUG:websockets.protocol:client x code = 1006, reason = [no reason] [1563292611.388265]
ERROR: Exception occurred: <class 'AssertionError'>: () [1563292611.3884506]

The state is set to CLOSED approximately 300 microseconds after it is tested in the assertion. I realize that enabling and adding log messages changes the timing, but the behavior matches what was observed without logging.

Thank you very much for the additional information.

Here's what's happening.
eof_received() means that the network connection drops or is closed by the remote endpoint. It's a TCP half-close. This causes transfer_data to take the except EOFError branch and to call fail_connection(1006), which is visible is failing OPEN WebSocket connection with code 1006.
Since close code 1006 means the network connection is broken, fail_connection(1006) doesn't take the if code != 1006 and self.state is State.OPEN branch which sends a Close frame. As a consequence, it doesn't set self.state = State.CLOSING.
The state only changes to CLOSED with connection_lost(), which is a TCP close.

The state attribute is well defined in RFC 6455:

Upon either sending or receiving a Close control frame, it is said
that The WebSocket Closing Handshake is Started and that the
WebSocket connection is in the CLOSING state.

=> not appropriate: we didn't send or receive a Close frame

When the underlying TCP connection is closed, it is said that The
WebSocket Connection is Closed and that the WebSocket connection is
in the CLOSED state.

=> not appropriate: the TCP connection isn't closed

I think that the state attribute is a bit coarse for the internal implementation. That's why I filed #497.

Interestingly, the implementation of ensure_open deals with this situation, where transfer_data_task terminates while state is still OPEN:
        if self.state is State.OPEN:
            # If self.transfer_data_task exited without a closing handshake,
            # self.close_connection_task may be closing the connection, going
            # straight from OPEN to CLOSED.
            if self.transfer_data_task.done():
                await asyncio.shield(self.close_connection_task)
                raise self.connection_closed_exc()

The assertion was there to guarantee that ensure_open() would raise an exception. It predates this improvement in ensure_open(). As a consequence, I think we can simply drop it.

8.0.2 is available on PyPI with this fix.


Issue: Invalid Argument(s) Error for latitude

I encountered an error while using the geofencing API that prevents proper functioning of the location services. The error seems to originate from a null latitude value in the location data, causing a failure during the creation of a new Location object.
Error Logs
[log] error: Invalid argument(s) (latitude): Must not be null
#0      new Location.fromJson (package:fl_location_platform_interface/src/models/location.dart:51:22)
#1      MethodChannelFlLocation.getLocationStream. (package:fl_location_platform_interface/src/method_channel_fl_location.dart:72:34)
#2      _MapStream._handleData (dart:async/stream_pipe.dart:213:31)
#3      _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:153:13)
#4      _RootZone.runUnaryGuarded (dart:async/zone.dart:1594:10)
#5      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:365:11)
#6      _DelayedData.perform (dart:async/stream_impl.dart:541:14)
#7      _PendingEvents.handleNext (dart:async/stream_impl.dart:646:11)
#8      _PendingEvents.schedule. (dart:async/stream_impl.dart:617:7)
#9      _microtaskLoop (dart:async/schedule_microtask.dart:40:21)
#10     _startMicrotaskLoop (dart:async/schedule_microtask.dart:49:5)
Environment
Flutter version: 3.22.3
Platform:Android

Hello, @ankitjoshi23
Thank you for reporting the issue.
Does this issue occur intermittently? or does it keep happening?
Are there any scenarios or sample code that can reproduce this issue?

@ankitjoshi23
Check latest version 😄
dependencies:
  geofencing_api: ^1.4.1

If it still doesn't work after the upgrade, you can try the following:
goto android/app/proguard-rules.pro
add keep class
-keep class io.flutter.app.** { *; }
-keep class io.flutter.plugin.**  { *; }
-keep class io.flutter.util.**  { *; }
-keep class io.flutter.view.**  { *; }
-keep class io.flutter.**  { *; }
-keep class io.flutter.plugins.**  { *; }
-dontwarn io.flutter.embedding.**

// this
-keep class com.pravera.fl_location.** { *; }
-keep class com.pravera.geofencing_api.** { *; }

Hello, @Dev-hwang
Thank you for your support! The solution is working well now, and the geofencing service runs smoothly without any issues.


Add microk8s to pick-right-solution.md

Microk8s is a single-command installation of upstream Kubernetes on any Linux and should be included in the list of local-machine solutions.

I signed it!

/assign @zparnold

@zparnold - hey :)

Thanks!
/lgtm
/approve


Ensure emoji render with regular font-weight

Emoji characters don't support bold attributes and may do weird things like rendering monochrome when bolded.

Only for reactions? 🤔

@guillep2k click "Expand Up" in the diff 😉

Please send backport


Could u provide the COCO_test2017_detections_person.json ?

hi, I have trained my own model, could you provide detections_person.json you used, so that i can compare to the COCO leaderboard. thank u .

Please download from OneDrive or GoogleDrive.


Update lingua-rs to 1.2.2

Pull Request Test Coverage Report for Build 02263baba929dc5d35983d4bbe3529a6be002738-PR-8

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

Totals

Change from base Build fe67fd8f34026f0f14559e22ce446334fff8ff62:
0.0%

Covered Lines:
18

Relevant Lines:
21

💛  - Coveralls


Draft: Implement data models with pydantic and template with fastapi.

@yoavnash So, this is my first draft for this approach. I think the remaining tasks would be

[ ] Add missing capabilities.
[ ] Add a few tests.
[ ] Update the getting-started app based on the template app here.

This is ready for a first rough review, I haven't tested the code yet.
I also looked into schema validation. I think we should use a spectral ruleset to check APIs during app registration.

How do we proceed with this one? Most of the functionality created is now in the standard app API repo. Should we just delete it?

Superseded by #22 .

Let's keep the branch as a reference.


feat: 🎸 added edit field implementation

Allows for a field to be edited, useful for real time sheets that can swap not only values but also formulas and dependencies on the flight

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The problem about cropped images size 271

Why the crop size during preprocessing is 271, and  is there  difference with  511 in pysot？

Why the crop size during preprocessing is 271, and is there difference with 511 in pysot？

Hi, the dataloader will handle it. The size of the input image is casual (271 or 255). You donot need to worry about that.

Why the crop size during preprocessing is 271, and is there difference with 511 in pysot？

Hi, the dataloader will handle it. The size of the input image is casual (271 or 255). You donot need to worry about that.

Thanks for your quick reply,Thanks for your job,Truely.

Why the crop size during preprocessing is 271, and is there difference with 511 in pysot？

Hi, the dataloader will handle it. The size of the input image is casual (271 or 255). You donot need to worry about that.

Thanks for your quick reply,Thanks for your job,Truely.

You are so welcome. Hope to see your great work soon.


mixer: do not block in the constructor

It is not necessary to retry CRD fetch -- if there is no CRD after installation, it will not magically show up, no matter long we wait. We should recommend rebooting Mixer if a new CRD is desired, or if necessary, retry fetching during normal operation after construction.

Move the blocking call to top-level to make it explicit, add logging statements around it
Move the monitoring server start after sync, to be used for readiness check

/assign @mandarjog
/assign @ozevren
/assign @douglas-reid
Fixes #7958
Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>

@kyessenov: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6d61184411b76f1e9a94be0cb18e1e2e95eb2401
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/cc @geeknoid
/cc @ayj
for thoughts on this.
I am not sure if there is much value in retries in the config store constructor, but I'm not sure what happens if some of the CRDs are not present. Since the installation procedure now requires the user to install CRDs first and separately, I think we should remove the unnecessary retry loop.

Can we fail early and fast if core types are missing? We can let outer
controller do retries instead.
On Wed, Aug 15, 2018, 9:31 PM mandarjog<EMAIL_ADDRESS>wrote:

@mandarjog commented on this pull request.
If retryTimeOut=30s is too long, we can bring it down to 5s or something
reasonable. We really need a few core Kinds to always show up.
Rule, handler, instance etc. The specialized kinds are being sunset in
favor of several static kinds anyway.
At that point there is no good reason to proceed until those kinds are in.
In mixer/pkg/config/crd/store.go
https://github.com/istio/istio/pull/7960#discussion_r210473235:

@@ -119,7 +119,6 @@ func (s *Store) Stop() {
// is open.
// Returns the created shared informers, and the list of kinds which are not created yet.

Update comment, there is no channel now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/7960#pullrequestreview-146694487,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJGIxi92UM6npkEIqGJrp9UXJQB7r7SKks5uRPWNgaJpZM4V_DPP
.

I think a short timeout 5s? would satisfy just that, but still give it a good chance to get all kinds

We have the following agreement.

It is Ok to not wait at all. Whatever kinds that are present are fine.
Log warning of kinds that are not found and continue.

In the future, when Mixer only works with static resources, we should fail hard if they are not found.

Sample start-up output:
Mixer started with
MaxMessageSize: 1048576
MaxConcurrentStreams: 1024
APIWorkerPoolSize: 1024
AdapterWorkerPoolSize: 1024
APIPort: 9091
APIAddress:
MonitoringPort: 9093
EnableProfiling: true
SingleThreaded: false
NumCheckCacheEntries: 1500000
ConfigStoreURL: k8s:///home/kuat/.kube/config
ConfigDefaultNamespace: istio-system
LoggingOptions: log.Options{OutputPaths:[]string{"stdout"}, ErrorOutputPaths:[]string{"stderr"}, RotateOutputPath:"", RotationMaxSize:104857600, RotationMaxAge:30, RotationMaxBackups:1000, JSONEncoding:false, LogGrpc:true, outputLevels:"default:info", logCallers:"", stackTraceLevels:"default:none"}
TracingOptions: tracing.Options{ZipkinURL:"", JaegerURL:"", LogTraceSpans:false}
IntrospectionOptions: ctrlz.Options{Port:0x2694, Address:"<IP_ADDRESS>"}

2018-08-16T18:24:20.039101Z     info    Awaiting for config store sync...
2018-08-16T18:24:20.049364Z     info    Built new config.Snapshot: id='0'
2018-08-16T18:24:20.049422Z     info    Cleaning up handler table, with config ID:-1
2018-08-16T18:24:20.087783Z     warn    Failed to discover kinds: [cloudwatch dogstatsd]
2018-08-16T18:24:20.088391Z     info    Built new config.Snapshot: id='1'
2018-08-16T18:24:20.089799Z     info    Cleaning up handler table, with config ID:0
2018-08-16T18:24:20.165008Z     info    Starting monitor server...
Istio Mixer: unknown@unknown-unknown-unknown-unknown-unknown
Starting gRPC server on port 9091
2018-08-16T18:24:20.170508Z     info    ControlZ available at <IP_ADDRESS>:9876

@kyessenov: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
6d61184411b76f1e9a94be0cb18e1e2e95eb2401
link
/test e2e-simple

prow/istio-pilot-multicluster-e2e.sh
6d61184411b76f1e9a94be0cb18e1e2e95eb2401
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
cb7b29ea7886f16c6982f73cf716d23b0a14d0b0
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov bunch of circleci jobs are failing.

@kyessenov we need this on release-1.0 branch.
I think it will be better to fix it there fist and move it up. All that MCP code has not made it in release-1.0 branch yet.

Hm, good point about MCP. Let me cherry-pick it and open a PR there. I don't think we can make one PR work on both branches now.

Can we merge this too @mandarjog ?

/lgmt