[camera] update cookbook and docs

update the https://flutter.dev/docs/cookbook/plugins/picture-using-camera#6-display-the-picture-with-an-image-widget
to match latest camera API changes, in particular i found takePicture call that now doesn't have a path argument
also, now i don't understand what is the file i get when i call takePicture. is it temporary (which is the case i suppose) or is it saved in app documents directory? document this as well
https://github.com/flutter/plugins/blob/d0b7109f6b00a0eda03506fed2c74cc123ffc6f3/packages/camera/camera/lib/src/camera_controller.dart#L322-L326

The XFile is an abstract layer around the file, this is done mainly to support web in the future, because that implementation won't be saved to the device first. The iOS and Android implementations both save the image in a cache directory. So if you want to save it to the application directory you'd have to move it yourself.
I'm not sure how the cookbook needs to be updated.

as i mentioned takePicture doesn't have path argument anymore. i'm not sure if there's anything that has changed in the plugin that requires changes to the cookbook

oh just looked at the chagnes history, turns out that there were never such argument. i don't even know where this code comes from

I am having same the same doubt. Even the example given in repo doesn't have any argument being passed in the takePicture(). Please verify.

Hi @pedromassangocode the example in the cookbook should not have a argument passed to takePicture() method. Because the takePicture() function in camera_controller.dart doesn't have any arguments. Even in MethodChannelCamera class from where the method channel is being invoked we see that only camera id is being passed and we are receiving the path for the file created in the native. If you agree that it needs to be changed, then shall I open a pr making the required changes to picture-using-camera.md.

@bharat-biradar feel free to open PR. if you will, please add some comments regarding
what XFile is (https://github.com/flutter/flutter/issues/75723#issuecomment-776746407) as well


Declare docker_mainflux-base-net as external and add it to all services

New PR that replaces PR 347 for cleanness reasons.
Fix #345 by declaring the network docker_mainflux-base-net and adding it to all services.
Signed-off-by: Paul RATHGEB<EMAIL_ADDRESS>

Codecov Report

Merging #361 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #361   +/-   ##
=======================================
  Coverage   89.17%   89.17%
=======================================
  Files          43       43
  Lines        1469     1469
=======================================
  Hits         1310     1310
  Misses        110      110
  Partials       49       49

Continue to review full report at Codecov.

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

Merged! Thanks @ks156


vscode no longer autocompletes c# nor Unity API's

Last week, this stopped working, probably related to one of the extension updates. I am running (on Windows 10) Visual Studio Code v1.6.1 and have following extension versions (I just list the ones that I think to be relevant):

C# for Visual Studio Code (powered by OmniSharp) v1.4.1
Legacy OmniSharp Support for Visual Studio Code v0.3.3

I think I need both, but problem persists when using only one of them (either one)
I tried it with and without 'C# Extensions' by jchannon v1.0.8: makes no difference.
I am not sure what exactly goes wrong. One of the symptoms ia that VSCode no longer shows the number of references to a variable just above its declaration.
Also, it no longer detects spelling errors in api calls nor class references, e.g. it readily accepts RigidBody2D iso Rigidbody2D as a class name, which is than rejected when returning to Unity.
Needless to say that it does not autocomplete, unless I'm referencing or typing something that already occurred in the file being edited.
This problem changed the lightning fast super pleasant editor into some dull often enervating tool.
Please advise.

Remove the legacy Omnisharp extension.

As I said in the previous message, that does not make any difference, the problem persists :(


Error: Invalid arguments in VRClassReward.lua

Hi,
I'm having this error for the recurrent-visual-attention.lua with CUDA. Can anyone help me please?
VRClassReward.lua:28: invalid arguments: CudaTensor CudaTensor CudaTensor number
expected arguments: CudaTensor | [CudaTensor] [CudaLongTensor] CudaTensor index
Thanks

I have the same problem

Problem solved:
Please reinstall "cutorch" using
luarocks install cutorch 1.0-0

wow..It works. Thanks @guo2004131

Alternatively, if you have already installed the latest cutorch, then you might want to replace the Line28 "self._maxVal:max(self._maxIdx, input, 2)" with "self._maxVal,self._maxIdx = torch.max(input,2)"

@mobarakolislam @guo2004131 fixed in https://github.com/Element-Research/dpnn/commit/ead91a239317dfcc04a0291cfa719f5f3ce7b6ba . Thanks for identifying this issue!


Update Accounts_Onboard_Utility.ps1

Try fixing issue when updating remote machine access (when using -Update)

Update Safe-Management.ps1

Improving safes cache for large number of safes (no need to re-fetch all safes on every add safe)
Added Test-Safe function
Made code adjustments for Test-Safe function
Update version number

Update Accounts_Onboard_Utility.ps1

Add a new Verbose log line of the entire script command line (for better debugging)
Fix an issue related to removing / updating Remote Machines
Related to #138
Fixed an issue related to template safe members, Trying to add the user that runs the script (as it is part of the template safe members)
Update version number


Always show accurate information in the interval matrix

ref #412

Self-reviewing: Seems to fix the issue.


Change the NetworkAnimator logic to be clearer by extracting out dirty-bits as a method.

Also made some methods Editor Only since they should not be used outside the editor for anything. This also make NetworkAnimator allocate a lot less: parameters is cached in Awake, since it should never change anyways, and NetworkWriter is only created when its actually going to be used to send stuff.

does this fix any bug or is it just for allocations?
did you test it in all cases, e.g. server-only, host mode, client mode, client authority mode?

@vis2k I could not reproduce the issue described in #570, but I combed through the code anyways and fixed several targets of opportunity:

the heuristics for deciding when to send packets were very unclear, to the point of having unintentional double-sends of parameter data. It was possible for parameter data to be sent twice on the same frame (which is obviously not ideal, or necessary).
packet sending was happening from multiple places (from inside 'CheckSendRate()'), this PR coalesces them to one location.
this PR increases the number of supported parameters from 32 to 64, as @paulpach suggested. This was not a target of this PR initially, but might as well fix it too. This does not increase bandwidth for those who do not use more than 32 parameters, since WritePackedUInt is used.
empty packets are not sent anymore, so when no parameter changed, no packet is sent instead of sending a 0-param packet.

I tested this branch sending from server-only, and from host, and they both work as expected. I am unsure of how to test the other two at the moment, but I have no doubts that they work, as I did not change any logic for deciding whether client or server should send.

please test it for local player authority. we need to test it, being sure is not enough :)

I need to figure out an issue with this, will reopen when it is ready again.

Also made some methods Editor Only since they should not be used outside the editor for anything. This also make NetworkAnimator allocate a lot less: parameters is cached in Awake, since it should never change anyways, and NetworkWriter is only created when its actually going to be used to send stuff.

Actually I'm facing a bug when you define parameters in Awake.
Parameters are always 0. Once I change Awake on Start.. everything starts working.


WAFv2 Logging Configuration

Issue #, if available:
This resource creates a resource to deploy WAFv2 Logging Configuration
Description of changes:
A few things to note:

The Kinesis Firehose name must begin with aws-waf-logs
Within RedactedFields, AllQueryArguments, Body and SingleQueryArgument are not supported
Only one Firehose Stream can be associated with a Wafv2 WebACL. More than 1 is not supported
If a Logging Configuration already exists for a given resource, this Cloudformation resource will not replace it.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the contribution!
We recently just published our resources and move from CodeBuild to GitHub actions. Can you please update this PR by pulling the latest changes? Some things worth calling out :

Please change the folder name to aws-wafv2-loggingconfiguration to conform to the same naming convention.
Refactor the pom to depend on the aws-wafv2-commons modules. This will remove redundant code from this PR.
Instead of using the Translator for conversion, please refactor to use Mapstruct instead. You can find examples of this in the *.converters package in each modules.

This will also require a change in the pom (annotationProcessorPaths, etc) for Maven to build properly.
Please also write tests for the converters.

Awesome, thanks for looking at this. I will make the changes and get back to you folks.

Closing this PR as I have opened a new one.


Users.Next() returns false immediately or hangs

Assume something along the lines of:
  user, err := i.Profiles.ByName(uname)
  if err != nil {
    // handle err
  }

  users := user.Following()

  following := make([]goinsta.User, 0)

  for users.Next() {
    if users.Error() != nil {
      // handle err
    } else {
      following = append(following, users.Users...)
    }
  }

users.Next() sometimes returns false immediately or, most of the time, just gets stuck and pegs the CPU at 100%.
Am I doing something wrong or is this part of the library currently broken due to changes made on Instagram's end?
Thanks.

Seems related to the problem regarding next_max_id, maybe you could give #306 a try.

Tested #306 and getting even weirder behaviour .. users.Next() works once or twice most of the time - users.Users contains a list of 100 users (yay). Sometimes .Next gets stuck, sometimes it returns false and users.Users is empty.
I would like to help you with this but honstly don't know where to begin - this intermittent behaviour is very confusing. Do you think it's to do with rate-limiting?

I’ve tested the fix successfully with fairly large profiles, getting more than 1000 Users returned.
You can check for problems if you call Error() in and after your Next() for loop like so:
var users []goinsta.User
followers := profile.Following()
for followers.Next() {
	err = followers.Error(); if err != nil {
		log.Fatal(err)
	}
	users = append(users, followers.Users...)
}
err = followers.Error(); if err != nil {
	log.Fatal(err)
}

I've the same problem, here is my error: 2020/12/10 02:09:10 json: cannot unmarshal number into Go struct field Users.next_max_id of type string


Bug in Campaign.java

Hi there. I have noticed a bug in the Campaign.java in the package of twitter4jads.models.ads. There are two fields that are mapped to the same JSON field names defined at line 35 and line 68, which will throw a runtime exception when parsing a campaign response.  Can we get this fixed? Thanks!

Seems to now be fixed: https://github.com/sprinklr-inc/twitter4j-ads/issues/61


Explicitly set AWS creds during upload

Also, check return codes to make sure everything worked OK.

@toddboom

+1 from @pauldix -- self-merged.


Sahar

👷 Deploy Preview for adoring-leavitt-e31a11 processing.

Name
Link

🔨 Latest commit
aec1952ff12572af982877413f68921319aa6be2

🔍 Latest deploy log
https://app.netlify.com/sites/adoring-leavitt-e31a11/deploys/62ab45ae6ccdb0000875a1cd


Add order= parameter

Currently the plot depends on row order which leads to weird plots. So I have to sort the data before putting it into AoG. I don't like that the algebra depends on the row order. Instead I would like an order parameter in the mapping.
using DataFrames, RDatasets, AlgebraOfGraphics

diamonds = dataset("ggplot2", "diamonds")

data(diamonds) * mapping(:X, :Price) * visual(Lines) |> draw

data(sort(diamonds, :X)) * mapping(:X, :Price) * visual(Lines) |> draw

data(diamonds) * mapping(:X, :Price, order=:X) * visual(Lines) |> draw

This may be related to Issue #262.


:seedling: Disable automatic upgrade of the built-in Postgres image

Summary
The PostgreSQL image should not be automatically upgraded from the configuration, as this may lead to unexpected errors listed in the issue: https://github.com/stolostron/multicluster-global-hub/issues/1253. It only supports manual upgrades.
Related issue(s)
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[x] List other manual tests you have done.

/hold

This conflicts with the solution we discussed in Issue: https://github.com/stolostron/multicluster-global-hub/issues/1253


Rider Can't Install Google Cloud Integration

(Please ensure you are running the latest version of Cloud Tools for IntelliJ with Help > Check for Updates.)
NOT running intellij!
For JetBrains Rider, I need Google Cloud Platform integration. Everything I've read discusses integrating GCP with Intellij, but I get an error when I try to install the plug in.

Cloud Tools for IntelliJ version:

Google Cloud SDK version (run gcloud version on the command line):
gcloud version = 199.0.0

OS:
Win 10

What did you do?
Tried to install Google Cloud Tools
What did you expect to see?
Successful Installation
What did you see instead?
"Plugin Google Cloud Tools depends on unknown plugin org.jetbrains.plugins.gradle"
Plugin shows not loaded in Plugin configuration. I can't find the dependent plugin.
(screenshots are helpful)
Feel free to deviate from this template as needed, especially if you are submitting a feature request.

@rchrismon thanks for post!
I followed up on your SO post (https://stackoverflow.com/questions/50046737/is-there-support-for-google-cloud-tools-in-jetbrains-rider)
As mentioned there, we are currently actively working to refactor our plugin to work with other IDEs:
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1896
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/1472
To point out - this at first will include only language agnostic features of the plugin (Cloud Source Repo, Cloud Storage brower etc.).
Will close for now as its a duplicate of #1472. Feel free to reopen if you have any more questions or concerns.

@rchrismon just FYI that the plugin is now installable in Rider
At the moment, only the Cloud Storage and Cloud Source Repositories integrations are available.

Thanks! Still can't get a clean deploy of the app, but this might help.
On Fri, Jun 8, 2018 at 12:43 PM Etan Shaul<EMAIL_ADDRESS>wrote:

@rchrismon https://github.com/rchrismon just FYI that the plugin is now
installable in Rider
At the moment, only the Cloud Storage and Cloud Source Repositories
integrations are available.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/2067#issuecomment-395852577,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGlUDclOU739d_taIepdpevGJiJsDg7jks5t6sW-gaJpZM4Tlhcv
.

--
Randolph (Randy) L Chrismon
Ph.:<PHONE_NUMBER>

when you say you can't get a clean deploy, do you mean that you are intending to deploy a Rider app to appengine? this update that I mentioned doen't bring any app engine support to the Rider plugin. it will however allow you to browse your GCS buckets (if you have any) and integrate (clone from and push to) Cloud Source Repos.

Yes, that's what I need to do, and I know what you've released doesn't cover that. Just letting you know, sorry I wasn't clear.

@rchrismon just curious,  what kind of app are you trying to deploy? And to what environment in GCP?


Sphinx documentation

This PR adds a setup for sphinx documentation. This can be locally build with make html in the docs folder. It will also automatically build on readthedocs after merging a PR. The readthedocs page should still be activated by @wbalmer, in case you agree on including these type of documentation!

I simply copied the installation instructions and HD 131399 example in there, but you can of course completely adjust things!

A minor comment: when increasing the version, this should now also get adjusted in the __init__.py file. I have changed it to 0.3.2 👍.

This is really excellent, thank you Tomas! I will review soon and might add to the docs.


Automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/ase-feedstock#41.

@conda-forge-admin, please add bot automerge


Build fails with Cabal 2.4

matchFileGlob was removed in 2.4 (https://github.com/haskell/cabal/pull/5284) and Distribution.Simple.Glob.matchDirFileGlob with widely different type is supposed to be used instead.
I am not really sure where are we supposed to get the PackageDescription argument value. Perhaps something like https://github.com/google/proto-lens/pull/242/files#diff-4fe9daabe5a3a972435eb21087de0259R183 could work.

Once one fix or another for this is finalized, this issue is also blocking bumps to GHC 8.8/base ==4.13.* and containers ==0.6.*; it'll probably be a good idea to see if the code builds with those versions afterward and adapt the bounds appropriately.


Add contributing guidelines

See #2  for reference
add points from https://github.com/garimasingh128/gitsetgo

Please assign this to me

Please assign this to me

Hi @garimasingh128 . Can I please work on this under SWOC?


Digest Auth Support?

Are there any plans to have a token provider for the broker?

No @johngluckmdsol. I believe (without having done any research into it) that that sort of thing would be best added using a reverse proxy in front of the application server.


Single producer slot

As I've described in #586 this removes Clone from Sender making slot a single producer single consumer stream. The points are:

The previous functionality can be restored with Arc<Sender> and using swap instead of Sink::start_send.
It now has poll_cancel method which allows deallocating resources on sender side when no receiver is present anymore.

The Sink functionality on Arc<Sender> can be restored by making a wrapper type, if this is crucial for some users. Also, this PR only updates the sync implementation, I can update the unsync implementation too if conceptually it's accepted.

The last commit makes poll_cancel as &self (immutable). This makes certain scenarios easier (such as keeping Arc<Sender>) while this still clearly contains only a single task to wake up on cancel. This commit is much more controversial, but at least solves my use case better (otherwise I need to put a Sender into an excessive mutex), still not sure if it might prevent later optimizations.

Generally speaking, poll_* fns must take &mut self in order to ensure they aren't called from multiple tasks.
So, a poll_cancel would not work if multiple senders attempted to share a Sender in an Arc.

Okay, that makes sense. Rewritten last commit to barely contain is_canceled method.

@tailhook what was the reason to remove Clone in the first place? Having poll_cancel seems useful?

@carllerche, sure. Without poll_cancel I'll get a memory leak in my use case.

Yeah... and a poll_cancel w/ Clone would make things more complicated...
I don't have a huge opinion about this either way.

Well, for now, I'm vendoring it and it seems to work for that use case (the code using it is quite early and not extensively tested, though).

Are there enough questions about this abstraction that we should perhaps move it to a crate for now instead of putting it in futures the crate just yet?

Yes, probably

@tailhook would you agree as well? And if so, mind sending a PR?

Well, I don't see unsolved questions here. The implementation as is in current PR is clearly superior to the one in the current master. And is simple enough comparing to multi-producer implementation that could support poll_cancel.

@tailhook I'm somewhat uneasy as right after the inital implementaiton landed new questions we raised, and it sounds like in solving those questions it raises more on this PR? It seems overall that there's not a rock-solid consensus on what to do here, in which case this is somewhat experimental, so it sounds like it may be best to start in an external crate?

Well, the only question brought here is should poll_cancel have mut self or immutable self. The only poll_* method where immutable self is used is in BiLock which seems to be wrong decision too. So I percept it more like my mistake than a controversial design issue.
Technically nothing stops to put it into another crate, but it's much more hard to discover needed kind of channel in another crate. And this abstraction is good for the following reasons:

Previously I've used unbounded channel for the task which is obviously more memory hungry (it can't be replaced by a bounded channel because that would get values lost)
There is only one channel that has poll_cancel support -- oneshot, and no Stream channels like that. Which means certain patterns isn't possible to implement without a memory leak.


feat(fieldlabel)!: core-tokens implementation (CSS-102)

Description
BREAKING CHANGE: This migrates the Field label component to core tokens.
https://jira.corp.adobe.com/browse/CSS-102
Web Component Implementation: https://github.com/adobe/spectrum-web-components/pull/2529 refactor(field-label): use core tokens #2529
How and where has this been tested?

How this was tested: comparing http://localhost:3002/docs/fieldlabel.html with the https://opensource.adobe.com/spectrum-css/fieldlabel.html component before and after.
Browser(s) and OS(s) this was tested with: Chrome Canary Version 105.0.5148.0 (Official Build) canary (arm64)

Screenshots

To-do list

[ ] If my change impacts other components, I have tested to make sure they don't break.
[ ] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have tested these changes in Windows High Contrast mode.

[ ] This pull request is ready to merge.

One more thing, @bernhard-adobe: the color for the disabled variant looks pretty different than it did before (see below with current and previous). Would you mind double-checking on that color?

One more thing, @bernhard-adobe: the color for the disabled variant looks pretty different than it did before (see below with current and previous). Would you mind double-checking on that color?

That is indeed missing as it was not documented in the "Components Batch 1.xd" file. I guess the color would be (--spectrum-disabled-content-color) like all the other disabled components.

One more thing, @bernhard-adobe: the color for the disabled variant looks pretty different than it did before (see below with current and previous). Would you mind double-checking on that color?

That is indeed missing as it was not documented in the "Components Batch 1.xd" file. I guess the color would be (--spectrum-disabled-content-color) like all the other disabled components.

rebuild Jenkins: https://spectrum-visual-regression.ci.corp.adobe.com/view/Spectrum CSS/job/css-vrt-test/37/
VRT results are: https://spectrum-visual-regression.ci.corp.adobe.com/view/Spectrum CSS/job/css-vrt-test/37/artifact/backstop_data/html_report/index.html

@pfulton color changes (as expected) for disabled and line-height changes (as expected):

Released beta:
5.0.0-beta.0

New beta release:
5.0.0-beta.1

New release with syntax fix:
5.0.0-beta.2

Fixed misaligned Field-Label padding for variant Side-Label.
@pfulton can you please publish another beta-release for the FieldLabel so I can pull in the latest changes in Web Component and re-run VRT over there?
VRT Results for Spectrum CSS are:
https://spectrum-visual-regression.ci.corp.adobe.com/view/Spectrum CSS/job/css-vrt-test/44/artifact/backstop_data/html_report/index.html
Line-height changed for FieldLabel. We have confirmed before that this is expected behavior.
With the latest push the Side-Label padding top  doesn't change anymore. This is in contrast with the documentation from the "Components batch 1.xd" file that says it should:

If this is an intended change we should roll-back my last commit:  044d8ad1f96ece84dadbbd1fb1877bfec7b18aca

Furthermore I aligned the CSS FieldLabel with the Web Components FieldLabel and the Line-Heights now match the Web Components.
Because we changed the line-heights from previously 15.6 px to now 18px (no more sub-pixels) the position of the Label changes a bit.
We may want to check in with design if this is intend or if we need to adjust the top-padding for the Side-Label variant.

Please let me know @pfulton  if this sounds good. Thank you.
(there are VRT changes with Switch and Menu reported as well, I think the baseline-images haven not been updates)

Released<EMAIL_ADDRESS>


[BUG] Refine command help or prompt message

[ ] check app name is inputted for rudr

$ go run cmd/rudrx/main.go containerized:run
Error: required flag(s) "image" not set
$ go run cmd/rudrx/main.go containerized:run -image nginx:1.9.4
Creating AppConfig nginx:1.9.4
Error: create component err: Component.core.oam.dev "nginx:1.9.4" is invalid: metadata.name: Invalid value: "nginx:1.9.4": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

[ ] Don't ask to set flag which has default value.

$ go run cmd/rudrx/main.go manualscaler poc1039
Error: required flag(s) "replicaCount" not set

I has been fixed I think


SILGen: Don't inject "willThrow" hooks before rethrow propagation branches.

The hook is intended to be used by debuggers to catch the point a throw happened in user source. It's unnecessary and undesirable to hook in places where an already-thrown error is just being implicitly propagated.

@swift-ci Please test

@adrian-prantl @jasonmolenda Do you recall whether it was intentional that we emitted willThrow hooks before implicit propagation like this? IIRC we only intended to call willThrow before an explicit throw in source code.

Build failed
Swift Test OS X Platform
Git Sha - 9207930fc1d74130d7e7a1938e200cdf77d5eeb1

I'm afraid I don't know anything about this. Was the commit history helpful at all?

Build failed
Swift Test Linux Platform
Git Sha - 9207930fc1d74130d7e7a1938e200cdf77d5eeb1

It looks like the willThrow call was put there in @rjmccall's original commit 58b5e1d and @atrick's 482b264 without any specific commentary. I spoke with @rjmccall, and his recollection matched mine.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 9207930fc1d74130d7e7a1938e200cdf77d5eeb1

Build failed
Swift Test OS X Platform
Git Sha - 9207930fc1d74130d7e7a1938e200cdf77d5eeb1

Yes, I think this is reasonable.  I think you could make an argument that we should call willThrow before starting the propagation of an error received from ObjC, just so that such throws do always get trapped, but that would still sweep up a fair amount of propagation-rethrows, and I think the "it's only triggered by Swift throw statements" line isn't fundamentally unreasonable.

@jckarter Do you think this would make for a good starter bug in IRGen?

Yeah, that's a good idea!

Filed https://bugs.swift.org/browse/SR-11803.  Please let me know if I'm missing any details in there.
Going to close this out.


Improved ui of the website template

Description
@sanjay-kv
HI i have made the following changes to the webiste

added the favicon for the page
added the website logo
made a toggle button for the repo statistics and the languages used
for the repo link just added it to the navbar as a github icon so that it will be present in all the pages and we dont need to add it again and again in each page
Also made a small update to the ui of the page such as putting proper alignemnet and color background

Fixes # (issue)
Type of change

[ ] Added a new machine learning frameworks, libraries or software.
[ ] Documentation update

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings


Auto-Disconnect Client after Inactivity

Is your feature request related to a problem? Please describe.
Sometimes I forget to manually disconnect from Spotify Connect and leave my LAN, then come back to find I can't reconnect (librespot says I'm already connected). Similar to #134 . This is not so problematic for me (I can log on to the server and restart the librespot applet), but it means that other members of the household will depend on me to do that for them if (when) they forget to disconnect, leave, come back and want to reconnect.
Describe the solution you'd like
I'd like to see a feature whereby the librespot applet disconnects connected clients after a period of inactivity (i.e. no music playing).
Describe alternatives you've considered
My current workaround is to schedule the restart of librespot every day at 4am.
Additional context
N/A

I guess you are using Zeroconf and never noticed that there is no proper check on whether the session is active. Would be nice if I could figure out after how much time Spotify considers the session expired.
Or this could be another connectivity issue. Do you see something related to reconnection between the first time you connect and when you come back?

That's right - I'm using Zeroconf (sorry, should have included that). The line which I'm seeing when I attempt to reconnect is from Zeroconf: "Dropped connection attempt because user is already connected", which I am getting around in one of two ways - reconnect using a different (device and) account if there's one handy, or kill the java process and restart it. Didn't notice about the lack of session active check, and I guess both that and not knowing Spotify's session expiry time makes implementing this more difficult.
I'm not really across the code, so this may come across as ignorant, but I was thinking of some kind of check as to whether song data / chunks are being received or not in order to establish whether there's an active session which is actually playing music... and if not, after a parameterised amount of time, optionally disconnect the active client?

Not to muddy the water, but just experienced this, where there was an initial failure to connect (first connection after starting), then a reconnect success which I didn't see on the iPhone client, then failure to reconnect:
2020-04-19 12:36:38 INFO  Utils:108 - Successfully removed cryptography restrictions. 2020-04-19 12:36:38 INFO  ApResolver:66 - Loaded aps into pool: {accesspoint=[gae2-accesspoint-b-m5z3.ap.spotify.com:4070, gae2-accesspoint-b-1k2m.ap.spotify.com:443, gae2-accesspoint-b-3px2.ap.spotify.com:80, gae2-accesspoint-b-0nkg.ap.spotify.com:4070, gae2-accesspoint-b-33jn.ap.spotify.com:443, gae2-accesspoint-b-h2dz.ap.spotify.com:80, guc3-accesspoint-b-m208.ap.spotify.com:4070, gew1-accesspoint-b-mw6m.ap.spotify.com:443, gew1-accesspoint-b-91rz.ap.spotify.com:80], dealer=[gae-dealer.spotify.com:443, gew-dealer.spotify.com:443, guc-dealer.spotify.com:443], spclient=[gae-spclient.spotify.com:443, guc-spclient.spotify.com:443, gew-spclient.spotify.com:443]} 2020-04-19 12:36:39 INFO  ZeroconfServer:369 - Zeroconf HTTP server started successfully on port [port]! 2020-04-19 12:46:53 INFO  Zeroconf:470 - Announced service: Service{alias='[alias]', service='spotify-connect', port=[port], text={VERSION=1.0, CPath=/, Stack=SP}, addresses=[/a.b.c.d, /E.F.G.H], domain='.local', protocol='tcp', host='[host]'} 2020-04-19 13:27:02 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=a.b.c.d, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:02 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=E.F.G.H, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:08 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo&version=2.7.1 HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=E.F.G.H, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:12 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo&version=2.7.1 HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=E.F.G.H, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:20 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=a.b.c.d, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:20 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=E.F.G.H, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:22 TRACE ZeroconfServer:433 - Handling request: GET /?action=getInfo&version=2.7.1 HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=a.b.c.d, Accept-Encoding=gzip, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:24 TRACE ZeroconfServer:433 - Handling request: POST / HTTP/1.1, headers: {Keep-Alive=0, Connection=keep-alive, User-Agent=Spotify/8.5.53 iOS/13.4.1 (iPhone11,2), Host=a.b.c.d, Accept-Encoding=gzip, Content-Length=667, Content-Type=application/x-www-form-urlencoded} 2020-04-19 13:27:24 DEBUG TimeProvider:73 - Loaded time offset from NTP: 616ms 2020-04-19 13:27:25 INFO  Session:107 - Created new session! {deviceId: [device id], ap: gae2-accesspoint-b-m5z3.ap.spotify.com:4070, proxy: false} 2020-04-19 13:27:25 INFO  ZeroconfServer:329 - Accepted new user from [device]. {deviceId: [device id]} 2020-04-19 13:27:25 INFO  Session:277 - Connected successfully! 2020-04-19 13:27:26 FATAL ZeroconfServer:398 - Failed handling addUser! java.io.FileNotFoundException: .\cache\journal.dat (Access is denied)         at java.io.RandomAccessFile.open0(Native Method)         at java.io.RandomAccessFile.open(Unknown Source)         at java.io.RandomAccessFile.<init>(Unknown Source)         at xyz.gianlu.librespot.cache.CacheJournal.<init>(CacheJournal.java:34)         at xyz.gianlu.librespot.cache.CacheManager.<init>(CacheManager.java:46)         at xyz.gianlu.librespot.core.Session.authenticate(Session.java:295)         at xyz.gianlu.librespot.core.ZeroconfServer.handleAddUser(ZeroconfServer.java:332)         at xyz.gianlu.librespot.core.ZeroconfServer.access$100(ZeroconfServer.java:38)         at xyz.gianlu.librespot.core.ZeroconfServer$HttpRunner.handleRequest(ZeroconfServer.java:396)         at xyz.gianlu.librespot.core.ZeroconfServer$HttpRunner.handle(ZeroconfServer.java:471)         at xyz.gianlu.librespot.core.ZeroconfServer$HttpRunner.lambda$run$1(ZeroconfServer.java:379)         at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)         at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)         at java.lang.Thread.run(Unknown Source) 2020-04-19 13:29:30 WARN  Session:975 - Socket timed out. Reconnecting... 2020-04-19 13:29:31 INFO  Session:277 - Connected successfully! 2020-04-19 13:29:31 INFO  Session:598 - Re-authenticated as [username]! 2020-04-19 13:29:31 DEBUG ZeroconfServer:268 - Dropped connection attempt because user is already connected. {username: [username]} 2020-04-19 13:29:31 DEBUG ZeroconfServer:268 - Dropped connection attempt because user is already connected. {username: [username]} 2020-04-19 13:29:31 DEBUG ZeroconfServer:268 - Dropped connection attempt because user is already connected. {username: [username]} 2020-04-19 13:29:31 INFO  Session:1023 - Skipping SecretBlock 2020-04-19 13:29:31 INFO  Session:1002 - Received LicenseVersion: 0 2020-04-19 13:29:31 INFO  Session:992 - Received CountryCode: AU 2020-04-19 13:29:31 INFO  Session:1023 - Skipping ProductInfo 2020-04-19 13:29:31 INFO  Session:1023 - Skipping LegacyWelcome 2020-04-19 13:29:32 INFO  Session:1023 - Skipping UnknownData_AllZeros 2020-04-19 13:29:32 TRACE MercuryClient:176 - Handling packet, cmd: MercuryEvent, seq:<PHONE_NUMBER>1343872, flags: 1, parts: 1 2020-04-19 13:29:32 DEBUG Session:1006 - Received 0x10:<PHONE_NUMBER>3F1B38787AC35CA5D2B94B7050EE3F 2020-04-19 13:29:32 INFO  Session:957 - Skipping unknown command {cmd: 0x75, payload: 000000} 2020-04-19 13:29:32 DEBUG MercuryClient:209 - Couldn't dispatch Mercury event {seq:<PHONE_NUMBER>1343872, uri: [uri] code: 200, payload: []} 2020-04-19 13:29:45 DEBUG ZeroconfServer:268 - Dropped connection attempt because user is already connected. {username: [username]} 2020-04-19 13:29:47 DEBUG ZeroconfServer:268 - Dropped connection attempt because user is already connected. {username: [username]}

I've made some changes to allow reconnection from the same username if the device is not active (which hopefully is toggled). You can build from dev.

Awesome, thank you. Will take a look.

Quickly compiled and tested - seems to be working (though haven't tested extensively). Thanks again!

Close the issue whenever you think it is working properly.

@devgianlu Are you sure this code is "correct"?
For me it look like either there's a "return" required as before, or the else-block has to be adjusted:
if (hasActiveSession() && System.currentTimeMillis() - connectionTime > TimeUnit.SECONDS.toMillis(60)) {             if (session.username().equals(username)) {                 LOGGER.debug(String.format("Dropped connection attempt because user is already connected. {username: %s}", session.username()));             } else {                 session.close();                 LOGGER.trace(String.format("Closed previous session to accept new. {deviceId: %s}", session.deviceId()));             }         }

@funtax You are right.

@devgianlu Are you sure we should report "activeUser" only in case there's an active playback? I checked the getInfo of my Sonos and a Pioneer AVR and both reported my username as "activeUser" even the devices have no active playback.
Don't know :)

That's news to me because I don't have any Spotify Connect device to test it with. Will change it tomorrow.

@benturnberg Did you experience any issue with this?

No issues so far. Will close this - thanks again.


add filename to window title

Proposal to have the window title change from F3D - Fast and minimalist 3D viewer to model.ext - F3D when ~/foo/bar/model.ext is loaded.
This PR currently appends F3D::AppName which makes the title model.ext - f3d (lower case f3d) and needs further changes but I don't know what the best way to do that would be (can we just upper-case AppName to F3D? do we need a new variable?)

> cp bin/f3d bin/f333d && bin/f333d --help
F3D - A fast and minimalist 3D viewer
Usage:
  bin/f333d [OPTION...] file1 file2 ...
...
Examples:
  bin/f333d file.vtu -xtgans                         View a unstructured mesh in a typical nice looking sciviz style
  bin/f333d file.glb -tuqap --hdri=file.hdr          View a gltf file in a realistic environment
  bin/f333d file.ply -so --point-size=0 --comp=-2    View a point cloud file with direct scalars rendering
  bin/f333d folder                                   View all files in folder

Report bugs to https://github.com/f3d-app/f3d/issues
>

Well, lets hope nobody does that then ^^. Looks fine to me.

@Meakk ptal


add default for hockey league data['schedule']['home']

I was getting the following error when doing league = League(league_id=LEAGUE_ID, year=2022, swid=SWID, espn_s2=ESPN_S2, debug=True)
Traceback (most recent call last):
  File "main.py", line 11, in <module>
    f()
  File ".../espn-api/main.py", line 7, in f
    league = League(league_id=LEAGUE_ID, year=2022, swid=SWID, espn_s2=ESPN_S2, debug=True)
  File ".../espn-api/espn_apii/hockey/league.py", line 22, in __init__
    data = self._fetch_league()
  File ".../espn-api/espn_api/hockey/league.py", line 30, in _fetch_league
    self._map_matchup_ids(data['schedule'])
  File ".../espn-api/espn_api/hockey/league.py", line 39, in _map_matchup_ids
    scoring_periods = match['home'].get('pointsByScoringPeriod', {}).keys()
KeyError: 'home'

I noticed that the code for basketball leagues was doing match.get('home', {}) instead of match['home'], so i figured the same thing needed to happen here for hockey.

Thanks for the PR, I wonder if this was introduced when the NHL postponed/canceled some games.

Codecov Report

Merging #313 (b0f9862) into master (5fdf2d2) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #313   +/-   ##
=======================================
  Coverage   87.82%   87.82%
=======================================
  Files          47       47
  Lines        1643     1643
=======================================
  Hits         1443     1443
  Misses        200      200

Impacted Files
Coverage Δ

espn_api/hockey/league.py
83.73% <100.00%> (ø)

Continue to review full report at Codecov.

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


Arrays

I implemented ToBorrowed, ToOwned and IntoOwned for [T; N].

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (0fdeee7) 61.5% compared to head (919c1db) 60.7%.

Additional details and impacted files

Files
Coverage Δ

ownable/src/traits/mod.rs
37.5% <0.0%> (-5.4%)
:arrow_down:

... and 1 file with indirect coverage changes

First of all thanks for the contribution.
But unfortunately your code is not compatible with the current msrv.
Do you think we should rise the msrv or change the code to work also with 1.58.1?
Also can you please move the array implementation above the tuples?

But unfortunately your code is not compatible with the current msrv.

I implemented these traits because I need them in a project where I use the latest stable.
I forgot to check your MSRV beforehand, sorry.

Do you think we should raise the msrv or change the code to work also with 1.58.1?

core::array::from_fn has been stabilized in 1.63, released in August 2022.
An alternative without raising MSRV would be to collect an iterator and use the array TryFrom implementation, something like:
self.iter()
    .map(ToOwned::to_owned)
    .collect::<Vec<_>>()
    .try_into()
    .unwrap()

Whatever you prefer.

Also can you please move the array implementation above the tuples?

I can do that.

After thinking about it, I think raising the msrv is the best thing to do.
I've already done that, so you only have to rebase this onto main.

A new version with this feature is released.


503 REJECTED_BAD_STATUS

Hi Pascal,
I get the following rejected log entry for a specific domain. The issue only occurs with Norconex. In the browser the domain response without 503.
REJECTED_BAD_STATUS: https://www.domain.test (HttpFetchResponse [crawlState=BAD_STATUS, statusCode=503, reasonPhrase=Service Unavailable])

Any suggestions what might cause this behavior?
Best regards
Sascha

Hello Sascha, I would need to know the site and your config, but is it possible your browser is configured to use a proxy? If so you would have to replicate those proxy settings on the GenericHttpClientFactory.

Hello Pascal,
the following configuration caused the behavior. If I remove the trustAllSSLCertificates configuration the crawler is able to index the domain.
<httpClientFactory>
      <trustAllSSLCertificates>true</trustAllSSLCertificates>
</httpClientFactory>

Any idea how this is might related?
Best regards
Sascha

I am not sure how it could be related.  I would need to be able to reproduce.
Are you OK with the solution you found?  Else, please indicate how I can reproduce (e.g. what site).

Hi Pascal,
unfortunately is a non public site which is not sharable.
Yes I am ok with the solution.
Best regards
Sascha


Enabling travis CI

Thanks @graup for the test suite #3

Cool, thanks. What's the reason to remove the concurrently solution? Reducing dependencies? Because now the CI relies on the fact that Travis kills the testrpc process after npm test returned. We were doing CI in Jenkins and that wouldn't kill anything on its own, which is why concurrently was needed.

IMO that should probably be in your own project instead of this example because testrpc is not a dependency of this repo.

I see, you're right. For the time being this is more clean then.


[Small] Don't Use Reflection When Provider is Given

Description
This moves the provider check to the top of the build method, to avoid instantiating objects for reflection without needing to do so. This may have been the cause of #86. Nonetheless, this is still a small improvement 🏕
Review checklist

[x] PR is split into meaningful commits for the ease of reviewing
[x] Description contains clear instructions on how to test the feature (where applicable)
[-] Tests have been written
[x] Appropriate labels have been applied

Codecov Report

Merging #88 (33fb410) into main (be26187) will not change coverage.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##              main     #88   +/-   ##
=======================================
  Coverage     90.9%   90.9%
  Complexity      55      55
=======================================
  Files           16      16
  Lines          111     111
  Branches        19      18    -1
=======================================
  Hits           101     101
  Misses           5       5
  Partials         5       5


Improper use of PluggableDevice data pointer

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
binary
TensorFlow version
2.15.0
Custom code
Yes
OS platform and distribution
Linux Arch
Mobile device
No response
Python version
3.11.8
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
TensorListFromTensor op assumes that the data pointer from a PluggableDevice tensor is to virtual address space and can add an offset to it.
https://github.com/tensorflow/tensorflow/blob/39cb3719012c249ba40bffc4cebaad66311a4669/tensorflow/core/kernels/list_kernels.h#L805-L821
The assumption that all PluggableDevices set the data pointer of Tensor to be virtual address space is false.
https://github.com/tensorflow/tensorflow/blob/39cb3719012c249ba40bffc4cebaad66311a4669/tensorflow/c/experimental/stream_executor/stream_executor.h#L144-L146
Unfortunately, this is not the only occurrence of a bug like this. I have seen similar issues with distributed training offsetting PluggableDevice data pointers on copy as well but I have not taken the time to look into it yet.
Standalone code to reproduce the issue
import tensorflow as tf
import numpy as np

tf.debugging.set_log_device_placement(True)
tf.config.run_functions_eagerly(False)
a = np.zeros((8, 1), dtype=np.float32)

with tf.device("MY_DEVICE:0"):
    b = tf.raw_ops.TensorListFromTensor(tensor=a, element_shape=[1])

print(b)

Relevant log output
No response

@jswag180,
I tried to execute the mentioned code with tensorflow v2.15 & v2.16 and observed the code is providing the expected output. Kindly find the gist of it here.

@jswag180, I tried to execute the mentioned code with tensorflow v2.15 & v2.16 and observed the code is providing the expected output. Kindly find the gist of it here.

It is only broken on PluggableDevices that set opaque in SP_DeviceMemoryBase to something other than virtual address space ptr (host or device). The attached gist is not using a PluggableDevice and will not hit this problem.


responsive keyValueList

Add & update our keyValueList to be responsive.
Before:

After:

@kimispencer how does this component respond when the content in some rows is large/long? Could you add some screenshots of those cases, please?

Let me find put together example with long text, but this PR includes a screenshot of the problem this fixes on mobile (https://github.com/lumihq/lumi/pull/4581).


chore(test): move test utils into an internal package

Moving test utils (containing test tcp and http clients) into a separate (internal) package to be used in PRs like this https://github.com/questdb/go-questdb-client/pull/47

Leaving everything in 1 package for now https://github.com/questdb/go-questdb-client/pull/47#discussion_r1698638020


issue while tyring to access Open road form  in a desktop application

I am trying to implement automation for a desktop application which has been developed using .net and OpenRoad..I could access the .net windows using winapp driver but when it comes to OpenRoad form it couldn't access the elements in the OpenRoad form..I would like to know if WinAppDriver  supports OpenRroad forms?

are the elements shown in inspect.exe, if not you cannot use windows application driver.

are the elements shown in inspect.exe, if not you cannot use windows application driver.

@snithya2208 What is OpenRoad?

@snithya2208 What is OpenRoad?

@anunay1 yes I could see the name of the element using visualUI verify .. I have used the same to locate the element defining it as a Windows’s element .. the application is developed with a combination of .net and OpenRoad .. I could navigate through the win forms without any issue , at a particular step it opens a openroad form and my test got struck there as it couldn’t locate the element

@anunay1 yes I could see the name of the element using visualUI verify .. I have used the same to locate the element defining it as a Windows’s element .. the application is developed with a combination of .net and OpenRoad .. I could navigate through the win forms without any issue , at a particular step it opens a openroad form and my test got struck there as it couldn’t locate the element

@snithya2208 What is OpenRoad?

It is a development technology similar to .net I believe .. my issue is the software opens up two different applications (one is the .net and the other is OpenRoad).. I will have to switch between these two applications for testing a workflow

@snithya2208 What is OpenRoad?

It is a development technology similar to .net I believe .. my issue is the software opens up two different applications (one is the .net and the other is OpenRoad).. I will have to switch between these two applications for testing a workflow


cdnify weirdness

It's probably just me, but I can't get the cdnify/cloudfiles part to work properly with Rackspace. What I'm seeing is image assets are uploaded to my container, in
<rackspace container>/src/img/...

but the paths in my emails get changed to
http://<rackspace uri>/dist/img/...

@toby1kenobi - It's not you. The cloudfiles task was never updated properly when imagemin was implemented. We'll get a fix / PR for this.
Line 173 of Gruntfile.js should be 'src': '<%= paths.dist_img %>/*', NOT 'src': '<%= paths.src_img %>/*',

Thanks!

Pushed a fix.
@taeo Thanks for the solution


Add "preset": "ts-jest" to jest config

Current Behavior
Some tools like VSCode's Jest plugin fail. Running tests normally works fine
Expected behavior
Tests should work
Suggested solution(s)
Add "present": "ts-jest" to createJestConfig.

I think someone tried to solve this in #75, but this generated absolute paths to tsdx's jest plugins.

Duplicate of #270
I'm not sure why this mentions the ts-jest preset since tsdx test doesn't use it (maybe it did at some point?)


common: Add stayBehind parameter to endMeeting host room action

Add a "stay behind" option for a host invoking the endMeeting action to opt-out of also leaving the room.
Testing

Ensure that a "host" roomKey is configured in the .env file as STORYBOOK_ROOM_HOST_ROOMKEY
Run yarn build && yarn dev
Navigate to the "Room Connection With Host Controls" story (direct link once yarn dev is running here)
Click on the "End meeting" button.
All remote participants should be removed from the meeting and the current host participant should also leave the meeting.
On this line in the PR, change in the argument from false to true to indicate that the host participant should stay behind in the meeting when this button is clicked.
Reload the storybook story and again click on the "End meeting" button.
All remote participants should be removed from the meeting but the current host participant should NOT leave the meeting.

Checklist

[x] My code follows the project's coding standards.
[x] Prefixed the PR title and commit messages with the service or package name
[x] I have written unit tests (if applicable).
[x] I have updated the documentation (if applicable).
[x] By submitting this pull request, I confirm that my contribution is made
under the terms of the MIT license.

This would have been easier to test if you added the stayBehind prop as a storybook control btw. Maybe worth doing actually.

Nice idea and added in https://github.com/whereby/sdk/pull/240/commits/b7ca00c76fadffaa83992296ff03d23b3d1b6820


ehermann-original.png missing

And I double checked the naming conventions this time!

closed in b4329ced29faac81afa7cbd5579e95e485161e0c


Implement Ranged Attack Parsing

Initial Implementation in v3.0.4 using the same parser as melee attacks (leading to the same errors in calculation and stuff)

Added in 3.1.0


fix *_unsecure.pc files

Supersedes https://github.com/grpc/grpc/pull/11448/files (which doesn't have CLA signed and would need updating Makefile.template, so I just rewrote that PR).

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,964,792      Total (=)      1,964,792

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,887,066      Total (>)     10,887,062

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences


code error: file not found if you do not update line113 in demo.py

https://github.com/CharlesShang/TFFRCNN/blob/master/faster_rcnn/demo.py#L113
this line should change to if args.model == ' ' :
else if you train your own model and set --model '/your/model/file.ckpt' will fail. since the output of train for each snapshot is not .ckpt, but 3 files.

Thank you for your suggestions.
I have not maintained demo.py for quite a long time.


Configuration is wrong

The section with the final configuration has 4 issues which results in RoleId1, UserId1, etc., see also https://GitHub.com/dotnet/EFCore/issues/15199.
The wrong example:
// Each User can have many UserClaims
            b.HasMany(e => e.Claims)
                .WithOne()
                .HasForeignKey(uc => uc.UserId)
                .IsRequired();
.....

The correct example:
// Each User can have many UserClaims
            b.HasMany(e => e.Claims)
                .WithOne(e => e.User)    // <-- missing User
                .HasForeignKey(uc => uc.UserId)
                .IsRequired();
.....

This applies to all 4 HasMany calls.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d9869895-9ba2-5027-46cd-89d1d995cba4
Version Independent ID: 8750939b-6ae5-3b9a-6c32-5e84b7fea22c
Content: Identity model customization in ASP.NET Core
Content Source: aspnetcore/security/authentication/customize-identity-model.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @ajcvickers
Microsoft Alias: avickers

@ajcvickers please review.

@theo-albers I don't see any place where we use .WithOne() when it is supposed to be .WithOne(e => e.User). Specifically, the examples with .WithOne() are in cases where navigations have been added only in one direction, and hence there is no User navigation here.

Hey @ajcvickers, I followed the document on an existing project without the relations. As soon as I ran "dotnet ef migrations add foo" to see if it was all fine and had zero migrations, I got 4 migrations. As soon as I changed the .WithOne() to ..WithOne(x ...) and reran the migrations, all migrations disappeared. So it seems to me it's directly related to the example config.
By the way I extended all of my entities, used the IdentityDbContext with all the generic parameters and used configuration classes.

@theo-albers If you have User navigation property, then you definitely need to use .WithOne(e => e.User). But I don't see any examples that are intended to have User navigation property and don't show this.

Have another close look. This is the EF setup in the docs and which I'm using as well:
public class ApplicationUser : IdentityUser
{
    public virtual ICollection<ApplicationUserClaim> Claims { get; set; }
    public virtual ICollection<ApplicationUserLogin> Logins { get; set; }
    public virtual ICollection<ApplicationUserToken> Tokens { get; set; }
    public virtual ICollection<ApplicationUserRole> UserRoles { get; set; }
}

public class ApplicationRole : IdentityRole
{
    public virtual ICollection<ApplicationUserRole> UserRoles { get; set; }
    public virtual ICollection<ApplicationRoleClaim> RoleClaims { get; set; }
}

public class ApplicationUserRole : IdentityUserRole<string>
{
    public virtual ApplicationUser User { get; set; }
    public virtual ApplicationRole Role { get; set; }
}

public class ApplicationUserClaim : IdentityUserClaim<string>
{
    public virtual ApplicationUser User { get; set; }
}

public class ApplicationUserLogin : IdentityUserLogin<string>
{
    public virtual ApplicationUser User { get; set; }
}

public class ApplicationRoleClaim : IdentityRoleClaim<string>
{
    public virtual ApplicationRole Role { get; set; }
}

public class ApplicationUserToken : IdentityUserToken<string>
{
    public virtual ApplicationUser User { get; set; }
}

And another nasty one: if you want to treat your entities as "aggregate root", you are going to use non default constructors. You will most likely use collections as IReadOnlyCollection with backing field. Now when you add a protected default constructor, just for EF, you will still run into another issue: InvalidOperationException AddEntityFrameworkStores can only be called with a role that derives from IdentityRole. It took me a while to realize it was due to the fact I didn't have a public default constructor. It's helpfull when that's documented as well.

@ajcvickers / @Rick-Anderson can you have another look?

@theo-albers The configuration that goes with those types already shows:
b.HasMany(e => e.Claims)
     .WithOne(e => e.User)
     .HasForeignKey(uc => uc.UserId)
     .IsRequired();

@theo-albers The configuration that goes with those types already shows:
b.HasMany(e => e.Claims)
     .WithOne(e => e.User)
     .HasForeignKey(uc => uc.UserId)
     .IsRequired();

Ah, shoot, you are absolutely right: I initially used the config a couple of lines above. That one didn't have the User in the WithOne(). I guess this is due to the build up of the example, from top to bottom. Sorry for wasting your time on this.

 =>
        {
            // Each User can have many UserClaims
            b.HasMany(e => e.Claims)
                .WithOne()
                .HasForeignKey(uc => uc.UserId)
                .IsRequired();

            // Each User can have many UserLogins
            b.HasMany(e => e.Logins)
                .WithOne()
                .HasForeignKey(ul => ul.UserId)
                .IsRequired();

            // Each User can have many UserTokens
            b.HasMany(e => e.Tokens)
                .WithOne()
                .HasForeignKey(ut => ut.UserId)
                .IsRequired();

            // Each User can have many entries in the UserRole join table
            b.HasMany(e => e.UserRoles)
                .WithOne(e => e.User)
                .HasForeignKey(ur => ur.UserId)
                .IsRequired();

It was easy to overlook the difference.


Replace operation symbols and allow percentage calculations

On the left window in the picture below (Soulver app) the * get's displayed as × and the % calculation returns a value, while on the right (Caligator) it doesn't work.

@remissao Thanks for reporting 😊


Failed to execute 'write' on 'Clipboard'  on Flutter web

I'm using this package to copy a table with information to be pasted in e-mail body and when I run locally its works fine, but when I deploy to firebase hosting I get this error. I try with Chrome and Edge
TypeError: Failed to execute 'write' on 'Clipboard': The object must have a callable @@iterator property.

It fails in release mode. Should be fixed in next update.


bugfix:errors in the test case of sofa registry

Ⅰ. Describe what this PR did
After tidying the dependencies,there are two errors in the test case of sofa registry.this PR will fix it.
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #989 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #989   +/-   ##
==========================================
  Coverage      38.34%   38.34%
  Complexity      1046     1046
==========================================
  Files            219      219
  Lines           8714     8714
  Branches        1085     1085
==========================================
  Hits            3341     3341
  Misses          4951     4951
  Partials         422      422

Continue to review full report at Codecov.

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


Miss 'Public Read Access' in 'Properties' panel for one connected public blob container under Quick Access

Storage Explorer Version: 1.15.0-dev
Build: 20200730.1
Branch: master
Platform/OS: Windows 10/ Linux Ubuntu 16.04/ macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one storage account -> Blob Containers.
Select one blob container which 'Public Read Access' is container -> Attach it via 'Connect to public blob container'.
Add the attached blob container to Quick Access.
Check whether the 'Public Read Access' under Quick Access shows in 'Properties' panel or not.

Expect Experience:
The 'Public Read Access' shows well.

Actual Experience:
Miss 'Public Read Access' under Quick Access.

More Info:
The 'Custom Domain' property is also missed under Quick Access.

Verified on build 20200731.3. This issue has been fixed. So close it.


CLOUDP-57845: List matcher fields, OM

Jira ticket: CLOUDP-57845
Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

Further comments
./bin/mongocli om alerts config fields type
[
	"TYPE_NAME",
	"HOSTNAME",
	"PORT",
	"HOSTNAME_AND_PORT",
	"REPLICA_SET_NAME",
	"SHARD_NAME",
	"CLUSTER_NAME"
]

waiting on https://github.com/mongodb/go-client-mongodb-atlas/pull/70

Let's review this one after #62


can't download math textbook

whenever I try to download this textbook I get this error:
Downloading book id 78293. Please open an issue on GitHub if this book id is incorrect.
Downloading metadata and eText information...
Traceback (most recent call last):
File "downloader.py", line 225, in
main(sys.argv[1])
File "downloader.py", line 115, in main
urllib.request.urlretrieve(getPageUrl(bookInfo['pdfCoverArt'], isCover="Y"), os.path.join(pdfDownloadDir, "0000 - cover.pdf"))
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 247, in urlretrieve
with contextlib.closing(urlopen(url, data)) as fp:
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 222, in urlopen
return opener.open(url, data, timeout)
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 531, in open
response = meth(req, response)
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 641, in http_response
'http', request, response, code, msg, hdrs)
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 569, in error
return self._call_chain(*args)
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 503, in _call_chain
result = func(*args)
File "C:\Users\user\AppData\Local\Programs\Python\Python37-32\lib\urllib\request.py", line 649, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 404: Not Found
the book id is 78293.
the link that I found the book id is: https://spectrum.pearsoned.com/api/v3/annotations/bookId/78293/courseId/5520160?annotationType=Bookmark&limit=100&offset=0
I found that link through the network tab in the inspect element settings.

I have the exact same problem and o don't know what to do. We're you able to resolve?

I found a temporary solution. I used a script to screenshot all of the textbook pdf pages. The script is here: https://pastebin.com/0kXVRiSS
for the cover.png you have to save it in the same folder as the python script


Please reenable proper encryption support

In #120, encryption support for EngineBlock was severely limited.  Specifically, Shib IdPs with a default setup can no longer be used to authenticate to Engineblock.  For SURFconext is is not really a big issue at the moment, but for OpenConext, it is a major problem.
Recap: to work around a chosen-ciphertext attack in the AES-CBC cipher (#116) used for assertion encryption, @relaxnow and @pmeulen implemented a fix: IdPs that encrypt the assertion are now required to sign the entire SAML Response.  This properly fixes the problem, as confirmed by @jurajsomorovsky, who was involved in the ROS security assessment on SURFconext.
However, this fix breaks saml2int compliance (as @joostd remarked), and makes Engineblock incompatible with recent Shib IdP versions (which refuse to send unencrypted assertions unless explicitly told to).
Therefore, please reenable support for unsigned, encrypted assertions.  In the original thread (#120), @pmeulen suggested the following fix:

My prefered way of fixing this would be to switch to encryption algorithms that are not vulnerable to this kind of attack (adaptive chosen ciphertext attack) like RSA-OAEP and AES-GCM and disable support for the vulnerable PKCS#1 v1.5 encryption and CBC-* mode algorithms. This way we solve the problem at the origin.

However, @jurajsomorovsky in #116 remarked:

This algorithm is required and you are supporting it, otherwise I would not be able to test it.

which suggest that removing support for AES-CBC altogether might also break standards-compliance.
Another way to solve this might be to address the original issue (#116):

Because the error output for a failure to decrypt differs from a failure to verify the signature the output may be used as an oracle along with a chosen plaintext to guess the cipher.

which should be easy to fix.

@pmeulen , @thijskh, @quartje and me discussed this over lunch.  Out conclusion was that the current behaviour (i.e., requiring signed SAML if the attribute payload is encrypted) is sane, and going back to the old behaviour is not a sustainable solution.  There are two possible way going forward:

Introduce the current EB behaviour in saml2int, and convince Shib upstream to support this; and/or
2.Check to see if there is a usable cipher that is supported by popular IdPs (ie., Shib, ADFS, SSP) that is not vulnerable to padding attacks.

Ongoing progress on how to fix this bug is kept in https://wiki.surfnet.nl/display/OpenConext/Ciphers+en+encryption

I've concluded my investigation, and afraid the news is not good.  Of the three main IdP product (ADFS, SSP, Shib), only Shibboleth support aes-gcm ciphers.  Neither ADFS or SSP support anything else than aes-cbc.  Therefore, the easy way to fix this bug (namely: require IdPs to use aes-gcm instead of aes-cbc ciphers) isn't going to work.

Do you agree that this is fixed since EB 5.1?

I don't know.  Has anyone tested encrypted IdPs recently? @surfnet-niels?

I've concluded my investigation, and afraid the news is not good. Of the three main IdP product (ADFS, SSP, Shib), only Shibboleth support aes-gcm ciphers. Neither ADFS or SSP support anything else than aes-cbc. Therefore, the easy way to fix this bug (namely: require IdPs to use aes-gcm instead of aes-cbc ciphers) isn't going to work.

This is now no longer the case I think as AES-GCM is now relatively mainstream.

Also signing of responses is now mainstreamly available.
I propose to close this now and 'fix' encryption (if any needed) when someone has a concrete need that can also directly confirm or deny which approach works for them.

Encryption support is there.. so by all means try to use it so we know what to change to fix it (if any)


Offset plurals

Example of the syntax: https://lingui.js.org/ref/message-format.html
{count, plural, offset:1
                =0 {Nobody read this message}
                =1 {Only you read this message}
                one {You and # friend read this message}
                other {You and # friends read this message}

This fellow thinks there's no point to supporting offsets: https://stackoverflow.com/a/58243789/3369753
I am struggling to see the use case.


CM-1091   Extend LiveIntent's User Id Module To Return sharedId

Extend LiveIntent's User Id Module To Return sharedId

Type of change

[ ] Bugfix

[x] Feature

[ ] New bidder adapter

[ ] Code style update (formatting, local variables)

[ ] Refactoring (no functional changes, no api changes)

[ ] Build related changes

[ ] CI related changes

[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?

[ ] Other

Description of change

Other information

Related pr: https://github.com/LiveIntent/prebid.github.io/pull/21

Opened in Prebid's repo


KON-355 Add numReturnStatement Property To KoPropertyDeclaration

Closed due to reverting this: https://github.com/LemonAppDev/konsist/pull/386.
The implementation of the "return" statement for nested functions was too complicated.


SEGV when using <C-R>=

SEGV occurs when using <C-R>= expansion.
[environment]
Vim 7.4.1832 on Ubuntu 16.04
[repro steps]
$ cat test.vim
if has('clpum')
  set wildmode=popup
  set wildmenu
endif
$ vim -Nu NONE -S test.vim

input keys,
:<C-R>=<CR><TAB>
and SEGV occurs.
[cause]
stacktrace.1 (when expansion)
#0  getcmdline (firstc=61, count=0, indent=0) at ex_getln.c:387
#1  0x000000000050c053 in get_expr_register () at ops.c:784
#2  0x00000000004897e5 in getcmdline (firstc=58, count=1, indent=0) at ex_getln.c:1454
#3  0x000000000048b05b in getexline (c=58, cookie=0x0, indent=0) at ex_getln.c:2497
#4  0x0000000000470d4c in do_cmdline (cmdline=0x0, fgetline=0x48b01e <getexline>, cookie=0x0, flags=0) at ex_docmd.c:1003
#5  0x0000000000503254 in nv_colon (cap=0x7fff060f2710) at normal.c:5320
#6  0x00000000004fc5ff in normal_cmd (oap=0x7fff060f27a0, toplevel=1) at normal.c:1149
#7  0x00000000005ec25c in main_loop (cmdwin=0, noexmode=0) at main.c:1359
#8  0x00000000005ebc4c in main (argc=7, argv=0x7fff060f2a88) at main.c:1051

stacktrace.2 (received SEGV)
#0  0x000000000048d962 in ExpandInit (xp=0x0) at ex_getln.c:4103
#1  0x0000000000475435 in set_one_cmd_context (xp=0x0, buff=0xc44940 "edit ") at ex_docmd.c:3422
#2  0x000000000048eacc in set_cmd_context (xp=0x0, str=0xc44940 "edit ", len=5, col=5) at ex_getln.c:4784
#3  0x0000000000496643 in clpum_complete (c=9) at ex_getln.c:8866
#4  0x00000000004889a7 in getcmdline (firstc=58, count=1, indent=0) at ex_getln.c:1102
#5  0x000000000048b05b in getexline (c=58, cookie=0x0, indent=0) at ex_getln.c:2497
#6  0x0000000000470d4c in do_cmdline (cmdline=0x0, fgetline=0x48b01e <getexline>, cookie=0x0, flags=0) at ex_docmd.c:1003
#7  0x0000000000503254 in nv_colon (cap=0x7fff060f2710) at normal.c:5320
#8  0x00000000004fc5ff in normal_cmd (oap=0x7fff060f27a0, toplevel=1) at normal.c:1149
#9  0x00000000005ec25c in main_loop (cmdwin=0, noexmode=0) at main.c:1359
#10 0x00000000005ebc4c in main (argc=7, argv=0x7fff060f2a88) at main.c:1051

It nests getcmdline, so clpum_compl_xp points NULL when entering clpum_complete.
getcmdline              # L383: set clpum_compl_xp to &xpc
__get_expr_register
____getcmdline          # L383: set clpum_compl_xp to &xpc
                        # and L2218: set clpum_compl_xp to NULL
__clpum_complete        # clpum_compl_xp is NULL

Fixed. It's a bit ad hoc :-)
I'll consider can integrate the clpum_compl_xp and ccline.xpc in the near future.
Thanks for your reporting and investigation:+1:

Thank you for fix quickly 😸


Adapt simple_sdx.py to make use of the information in sdx_global.cfg

create a configuration similar to https://github.com/h2020-endeavour/endeavour/blob/master/examples/test-mh/config/sdx_global.cfg

Use the information from this config file to setup the topology similar to
https://github.com/h2020-endeavour/endeavour/blob/master/examples/test-mh/mininet/simple_sdx.py

@Baloc this is similar to the work done by @TribuneX on #18

Will be obsolete as soon as we use torch for setting up the example topology


Git master version crashes on unlock.

On archlinux with gnome 3.24 on wayland. Not exactly sure how to provide debug logs if this is not enough info to reproduce.

Don't have this issue.
Check the output of :
journalctl /usr/bin/gnome-shell -r | less

You can use / to search for timepp or time and see if there are any bugs. q to quit.

What do you mean crashes?

Btw, are you sure you have the latest version from github?  I did fix a bug today.

Yes, most definitely, did a git pull right before posting the issue to make sure I did not miss a bugfix - was already up to date. And with crashes I mean that after entering my password on the lockscreen, the screen freezes for some some 10s of seconds before dumping me back to the login screen. Logging in again shows me a completely new session (ie. all my applications were closed). I know that on Wayland you cannot restart gnome-shell without restarting the session, so maybe someone on X11 isn't so clearly aware of it.
I cannot immediately find a clear error using journalctl.

Could you try the latest version now? I added a commit that might fix it.
Also, if it doesn't work, can you revert back to this commit:
https://github.com/zagortenay333/timepp__gnome/tree/a4fe5e5ff6f297fa3040c84ca796f92b8d2d6555
This is before I introduced that feature you requested.

Can confirm issue still exists with that latest one, will try pre-unified icon now (I was on the extensions.gnome.org version before).

Working on it. Hard to predict when it triggers too, as current master managed to not break once, but then did break a bit later...

So far a4fe5e5ff6f297fa3040c84ca796f92b8d2d6555 seems to not crash.

Yeah, that's before the feature.
Could you try the latest master yet again. Srry that I bother you, but since I can't reproduce it I need your help  here.

No problem, it's entirely in my selfish interest to get this fixed, since my requested feature needs it ;)
Is there any way to force reload an extension after manually updating without restarting the shell entirely?

Nope. Wayland doesn't allow for shell restarts? Yikes

Not without killing the session. I understand it's a feature in terms of security...

Will continue workin' on this tomorrow, btw.

Did you try the latest version again?

Tried it last night, seems to crash. Thoroughly testing a4fe5e5ff6f297fa3040c84ca796f92b8d2d6555 now to make sure it was indeed something after that.

Yeah, that's a good idea.
Btw, does this bug happen consistently? Also, show me the the general tab in your settings (screenshoot.)
Oh, and I added another commit to master, might wanna try again with latest. I did notice some error on my machine regarding the fullscreen, maybe that was the culprit.

Did you manage to replicate the bug that commit?

a4fe5e5ff6f297fa3040c84ca796f92b8d2d6555 -> no crash
fb19c26175ebe402381cc796af29ad8dc7e2d61a -> crash
Journal log with personal stuff redacted

Ah, you got the journalctl finally!
Can you show me the output of journalctl on the a4fe5e5 commit and the journalctl on the commit right after it 6082b981160f2eac1661232bc40e53d3c6a2237a ?

Sure, you'll have to bear with my slowness - having to interrupt my other activities by restarting my session makes it somewhat annoying to debug. I found an extension that was supposed to help me out here but unfortunately it doesn't work in this case (nls1729/acme-code#27).

No problem, man. I annoyed enough already XD. No rush.
Btw, I have a feeling I solved the problem  in latest master as I was able to find a nasty bug which gets triggered when locking/unlocking.
The only difference seems to be is that on your machine it caused a crash, but on mine it didn't. Again, not 100% sure.

Now it crashed on a4fe5e5ff6f297fa3040c84ca796f92b8d2d6555 too 😕
https://0bin.net/paste/8bPD9ZMD-dlO2q4z#OlZup5R7Zsiz3cuoZTvDMK4J9I0HACYkxPR4Fo4OCsk

Actually that's good. It means that the bug I found is probably it since it has been around for a while. Try out master.

5a96b20ee0ed49dca0595c5b2a88c5ca22f5a766 -> crash: Journal log
Going to remove the git version and reinstall from g.e.o to sanity check.

Show me the journal output after you disable the extension.

I think this is the right part of the log judging by the timestamps.

From what commit is that log?

That's from when I clicked "remove" in the tweak tool. I had 5a96b20ee0ed49dca0595c5b2a88c5ca22f5a766 running at the time.

I'm at my wit's end.
I definitely did fix an annoying bug in that last commit, so that's nice. What's causing your bug I've no clue.

Running e.g.o version also gave me the issue when it didn't in the past. I'm now trying out if some other extension update snuck past me and is causing the issue... Going to keep trying until I've got the e.g.o version running without issue and then switch to master to check if that then also doesn't break...

Disable all other extension before you test this one. That's the simplest way to verify that it is this ext that causes it.

It's proving somewhat hard to hunt down, especially considering that it doesn't happen every screen unlock... It seems it may even be some sort of interaction between different extensions. I'll close for now and not reopen unless I find out there time++ is (one of) the culprit(s).

Like I said, either disable all other extension or only disable time++ and see if the bug reappears.

Complete Heisenbug, disabling all extensions except one made it disappear, enabling them again one by one never made it reappear even when in the same state as when it appeared before... Tested Time++ master, works completely fine now.

Uff, that kinda scary :fearful: Hopefully it doesn't reappear.

Bump. Curious, have you had this bug appear again?

Nope, not at all.


Possibillity to not use any loadingComponent at all

One might want to not use the loadingComponent from griddle at all, since he wants to have total control where the loading container should appear in the DOM.
A isLoadingComponentDisabled or similar prop might be useful in this case.
The code might fit in the gridTable.jsx:
if (this.props.externalIsLoading && !isLoadingComponentDisabled) {
...//handle the edge cases
}

This should be taken care of in the latest version. We've defaulted to providing less features out of the box and assume that more things should be happening through user-customization or plugins.


Generating dll for use in LoadRunner for performance testing 32bit and 64 bit

hi,
I am trying to use this library with Performance testing tool called LoadRunner.  I have generated the dll using the "bash build.sh" which has been provided in the Repo, but that dll fails with "Error: C interpreter run time error:  Error -- File error : LoadLibrary(l8w8jwt.dll) failed" in loadrunner.
Are there some instructions available to generate 32 bit dll? I could try to use that as well.
Br,

That error arises inside build.sh?
Or when you tried to use it?

Error is thrown by the Loadrunner tool  when i try to use it.

You're on Windows. Did you link whatever code you're compiling against the symbols l8w8jwt.lib? Windows requires that before loading l8w8jwt.dll from the path.

Also: build.sh builds x64 in Release mode by default on Windows. If you need anything else, nothing stops you from customizing the build using CMake yourself :)
I'm sorry but I don't have any experience with LoadRunner, and it's also out of the scope of this library, so I'm closing this issue for now.

Hi,
I got the dll to work , however Loadrunner(LR) found the methods but not the structs declarations in the header files. So i had to add  all the header files seperately in LR .
Now I am trying to test it with the Example code "Encoding and signing a token" which is given in the Documentation here. The printf is returning r=100  L8W9JWT_NULL_ARG
What could be the reason for this?
Br


fix: issue in safari 'Invalid regular expression'

This fixes the issue in safari reported in this issue.

@asnunes any ETA for this PR? It would really help us to have this fixed. thanks!

Merging right now, @isaacHagoel.

Merging right now, @isaacHagoel.

@asnunes awesome. Thanks a lot!


[RFC] Make sampling a first class citizen

(still work-in-progress)
Rendered

Codecov Report

Merging #225 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #225      +/-   ##
============================================
+ Coverage      45.9%   45.91%   +0.01%
- Complexity     2444     2445       +1
============================================
  Files           602      602
  Lines         15696    15696
  Branches       1174     1174
============================================
+ Hits           7205     7207       +2
+ Misses         8058     8057       -1
+ Partials        433      432       -1

Impacted Files
Coverage Δ
Complexity Δ

...m/spotify/heroic/aggregation/simple/MaxBucket.java
52.38% <0%> (+9.52%)
5% <0%> (+1%)
:white_check_mark:

Continue to review full report at Codecov.

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


Error 403 when scraping Substack from an Actions runner

Scraping Substack with extractus works on a home PC, but it does not work from an Actions runner. For reasons I don't fully understand, Substack began returning Error 403: Forbidden at 7 PM EST on January 15, 2023. Here is a reproducible example:
name: Fetch RSS Feed

on:
  push:
    branches:
      - main

jobs:
  fetch-rss:
    runs-on: ubuntu-latest

    steps:
    - name: Checkout code
      uses: actions/checkout@v4

    - name: Fetch RSS Feed
      uses: Promptly-Technologies-LLC/rss-fetch-action@v2
      with:
        feed_url: https://knowledgeworkersguide.substack.com/feed
        file_path: ./feed.json
        parser_options: "{\"useISODateFormat\": false, \"getExtraEntryFields\": \"(feedEntry) => { return { 'content:encoded': feedEntry['content:encoded'] || '' }; }\"}"
        fetch_options: "{}"
        remove_published: true

    - name: Commit and push changes to repository
      uses: stefanzweifel/git-auto-commit-action@v4
      with:
        commit_message: 'Update RSS feed'
        file_pattern: '*.json'

I have tried adding custom headers, but without success.

Note that this is not an issue with extractus. Version 1 of the rss-fetch-action, which used isomorphic-fetch, also fails:
      - name: Fetch RSS Feed
        uses: Promptly-Technologies-LLC/rss-fetch-action@v1
        with:
          feed_url: https://babafaqirchand.substack.com/feed
          file_path: ./src/components/ui/RssFeed.json
          remove_last_build_date: true

Honestly, it seems like Substack may have just specifically blocked Github Actions runners for some reason. I'm not sure why you would do this (maybe IP concerns about Substack content appearing on Github, or abusive high-frequency requests?) or how you would go about it (some kind of CORS/IP blocking?), but it's my current best guess.


brew install kuttl-cli command doesn't work

What happened: I tried to install kuttl cli using brew on my mac as suggested in documentation but the command brew install kuttl-cli doesn't work and shows this as an output-

Environment:

Kubernetes version (use kubectl version): latest
KUTTL version (use kubectl kuttl version): latest
OS (e.g. from /etc/os-release): MacOS (Macbook air m1)

if this is some documentation issue, i would be happy to fix.


docs: Document limitations.

Add a document listing the known limitations.
Fixes #338.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Coverage remained the same at 55.39% when pulling 9158aadfe24a3134eccfe8a2b7b0c0ea10af73c4 on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

Just noticed that the issue links aren't "active", but I think that's because the document isn't yet merged into the master branch.

@iphutch - could you take a look at this please?

Coverage remained the same at 55.39% when pulling 5fcbd4e4a330323cb0df89911dc5a7ce48cda399 on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

Coverage remained the same at 55.39% when pulling 9d9a246637e7c6eb6a9b9b1448ee46f6a84977bd on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

lgtm

Coverage remained the same at 55.39% when pulling e8e63cdf3d11b745e26837f65dcfbee11b42603b on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

Hi @gorozco1 - gah! I had documented on this PR that the links should become active once it lands, but... you are right - they don't which is pretty sucky! ;-(
Changes:

Made "internal" links active by specifying the full URL.
Moved all "See issue ..." comments to the end of each section for consistency.
Changed all cc-oci-runtime "See issue ..." comments to end with "... for more information." for consistency with the internal issue links.

Coverage remained the same at 55.39% when pulling 56e96d3b788dc91bf144e629c7200de98cbc5ff3 on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

Coverage remained the same at 55.39% when pulling 56e96d3b788dc91bf144e629c7200de98cbc5ff3 on jodh-intel:add-limitations-doc into 939f7e3c79e34a72f43eb5e8bedaebd1a31b755c on clearcontainers:master.

Nice @jodh-intel . Still
lgtm
but you hit the dreaded CRI-O semaphore failure....

I could merge, but technically github would like @gorozco1 to OK the changes requested (broken links). let's wait for that ack...

Wiki updated to point to the in-tree doc:

https://github.com/clearcontainers/runtime/wiki/Current-Limitations

(and the draft wiki doc this PR was based upon has been deleted: https://github.com/clearcontainers/runtime/wiki/Current-Limitations-2).


Add support for Ripple panic button

Currently Aegis is not compatible with Ripple, it would be really nice and helpful for both projects if support would be added. Aegis would have an extra layer of security and Ripple would have one more app that has support, and this would incentivize Ripple's devs to continue with their work.
It would be even greater it you could add a setting to delete the entries, not just to hide the app when triggered. A lot of apps have support for it but only enable this option, which is kind of lame since it can have a bigger potential.
Thanks!

this is the relevant info for an app like this: https://guardianproject.info/code/panickit, Ripple is just one app in the PanicKit ecosystem.


crash when deleteAllObjects in the block called by dispatch_barrier_async

I create dispatch_queue_t using DISPATCH_QUEUE_CONCURRENT.
When I tried to read tables using dispatch_async and write tables using dispatch_barrier_async, app was crashed by EXC_BAD_ACCESS sometimes.
Sometimes succeeded, but I cannot read a new data changed in write-transaction.
Here is my sample code. Please check out the repository, git-submodule-update (I use Realm as submodule), and run test.
https://github.com/amo12937/amo.RealmQueueHandler

version of

xcode: 6.1.1 (6A2008a)
OSX: 10.10 Yosemite

Thanks.

It looks like this was introduced with this pr #1346. This has been merged to master but we have not released these changes yet (and wont until we have a fix). For the time being if you use the latest release from the website or you build from a commit before this pr was merged you should be able to avoid the issue.

Thanks for replying.
I use the latest master branch, but still crash.
Here is screenshot when crashed.

It looks like this is caused on tightdb.

You probably need to 'Reset Content and Settings...` on your simulator, or delete the app from a device. If an old Realm file with the int index still exists then you would still run into the same issue. When building from master I no longer see the crash.

Thank you. I do Reset Content and Settings..., then I no longer see the crash.
Now I still have another error...

I close this issue and will create new issue..
Thanks m(_ _)m


.NET 6: HTTP Logging

Replace https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/fundamentals/http-logging/samples/6.x with the current template generated code. This article was written with .NET 6 preview 1.
@smandia  do you want this one?

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

ID: 7c772b2a-5d37-84af-624c-9022efe0758b
Version Independent ID: b39594cc-9d1c-611c-e3e7-8755d74253a6
Content: HTTP Logging in .NET Core and ASP.NET Core
Content Source: aspnetcore/fundamentals/http-logging/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @jkotalik
Microsoft Alias: jukotali

@Rick-Anderson  Yes I want to work on it

work in progress

@Rick-Anderson  Please review https://github.com/dotnet/AspNetCore.Docs/pull/24533


Task: Design validation service

[ ] Hold design sessions

[ ] Initial design for check engine

Prerequisite:
Research/deep dive for check engine in 2.1.


Crashing unikernel does not propagate error return code

Feature request summary
This issue tracks correctly propagating a return code of 1 when Unikraft/machine crashes. This was discovered by using KraftKit's GitHub Action against libc-test which can be viewed here:
https://github.com/nderjung/unikraft/actions/runs/5728684586/job/15523782562#step:5:6723
The above job should have failed because of the crash.
Describe alternatives
No response
Related architectures
None
Related platforms
None
Additional context
My hunch is that the Events method in QEMU's v1alpha1 machine interface implementation is not handling the case where the QEMU binary exits unexpectedly.

After looking over the code for a whole day, I am not sure what happens, but I think it's not something on kraftkit's side.
When the qemu binary is launched, it is started with Parent PID 1, and also the qemu binary returns early (0 exit code), before the the output of the unikernel even starts.
This points me to the idea that when running through QMP, the unikernel is initially started paused and the unix sockets are created. After this is done, the process returns.
Kraftkit then starts the detached unikernel qemu process through QMP and communicates with it over there.
This means that, because the processes are not related, we won't be able to fetch the exit code from qemu, and in turn from the Unikernel.
I only looked briefly over the QMP documentation, but I did not find anything related to ExitCodes and such.
Ideally, we could either tell QEMU to write the exit code to a file, or to CLI (not sure if possible), or request the exit code over qmp once it terminates (but we can't because the instance is no longer active). Last option would be to instruct QMP to send over the exit code once it finishes, but again, not sure if such thing exists.
In the meantime, for github-actions I recommend just expecting certain end-lines for success cases and fail on everything else.

So it seems that we can detect when a Panic happens, but not sure this would work in the case of unikernels


Bug: /otg export preset Names encased with "" throws an error despite working in OTG 0.0.16 and earlier

OTG Plugin or Mod

[x] OTG Mod for Forge

OTG Version
[Please specify the full OTG version that you are running by placing "x" into []]

[ ] 1.12.2_v8_r1
[ ] 1.12.2_v7_DEV_r26
[ ] 1.12.2-v6
[ ] 1.12.2-v5
[ ] 1.11.2-v8
[ ] 1.11.2-v9
[ ] 1.10.2-v21
[x] OTG 1.16 0.0.17

Forge or Spigot Versions
Forge 36.1.32
What other plugins are you running?
https://docs.google.com/document/d/18PodlrUTttVyKap7udTtCqoCHHPgzYw-ei2tTCbeLXk/edit?usp=sharing
Description
When attempting to export an object with a Preset name with "" in it throws a
"Expected whitespace to end one argument, but found trailing data, ...ange_wool "When Realities Collide" BO3
What else is weird is that this worked in versions prior to 0.0.17.
Fortunately I can get around this by removing the spaces in my Preset's name.
I expected however to be able to freely use my preset name without spaces with no issues.
Steps to reproduce

Name a preset with spaces
In a world select two points with /otg region mark
Then export the object, using the preset name as the target location for exporting
If done correctly you should get an error similar to this "Expected whitespace to end one argument, but found trailing data, ...ange_wool "When Realities Collide" BO3

Fixed by Frank for 0.0.18.


Add CI/CD and editorconfig

Example run: https://github.com/cofob/botka/actions/runs/8862035948/job/24334532843


Can't Add Credentials

Description
Hi, I am experiencing issues adding credentials. I cannot add any credentials; it throws the error Error: Credential azureOpenAIApi not found.
To Reproduce
Steps to reproduce the behavior:

Go to Credentials
Click on Add Credential
See error

Screenshots

Setup

Installation: Following this guide
Flowise Version: 1.4.9
Browser: chrome

I am getting a similar issue, the api  /api/v1/components-credentials is returning empty array, probably because the data initialization is not completed successfully. May be related to this one  https://github.com/FlowiseAI/Flowise/issues/1538

I am getting a similar issue, the api /api/v1/components-credentials is returning empty array, probably because the data initialization is not completed successfully. May be related to this one #1538

the data initialization error happened to mine as well

same here since I've upgraded to the latest commit, I can't access/update existing credentials neither.
I've tried deploying older commits (from few days back) but still having the same issue.

Hi guys, try locking the @upstash/redis version. If I understand correctly, recent updates from @upstash/redis may have caused this data initialization error

Yea I'm having the same issue.

Locking `components\package.json' with the screenshot info does fix it:
"@upstash/redis": "1.22.1",
Perhaps it should just be locked to there for now until it's fixed.

yep will push a fix to lock that for now

I've just deployed the latest commit (including the Upstash Redis fix)
it has fixed the previously reported error but now I am getting this new Error: Credentials could not be decrypted.
I now can add new credentials but can not access existing ones

That's because you didn't store the secret and it created a new one :)

That's because you didn't store the secret and it created a new one :)

well not a big deal, but I am not sure what I should have done :-)
FYI I use a Render Starter instance

Fixed in v1.4.10.


Switching languages when signed in shows signed out header temporarily

Issues / opportunity
When authenticated, clicking the language toggle switches the language but as the page is loading flashes the unauthenticated header / top nav.
Screenshots

@srtalbot Issue persists on staging.


Import trial TSPs

Description
This is a secure migration to import the three HHG trial TSPs. It also adds contact fields to the TSP model.
To test:

Run bin/run-prod-migrations to load the production data set into your local postgres
Connect to the prod_migrations database locally, and verify the data against what's in the TSP info spreadsheet.
Run make db_dev_reset && make db_dev_migrate and ensure that the test database doesn't contain real data.

Code Review Verification Steps

Any new migrations/schema changes:

[x] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[x] Have been communicated to #dp3-engineering

[x] Request review from a member of a different team.
[x] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Closing in favor of this PR, which is a superset: https://github.com/transcom/mymove/pull/1217