Segmentation fault

On Windows 7 x64, when running the ./setup.sh -s command, a couple of issues happened.

Where to run a shell script on Windows??
Installed GIT and used the Git Bash window.
Segmentation Fault
After running the setup.sh, getting following error
$ ./setup.sh -s
Checking for CF login credentials...
Syncing wsk namespace with CF namespace...
Checking for or creating cloudant instance...
Creating deployment pipeline(s)...
./setup.sh: line 158: 10448 Segmentation fault      jq -r '.pipeline[]' ${PROVIDERS_FILE} &> /dev/null
ERROR: Providers/credentials file missing pipeline JSON key.

The problem is the redirection.. remove "&> /dev/null" and it will work well.

Hello,
It sounds like you are doing a Facebook based deployment? Guessing based on the mention of setup.sh -s. For Windows we recommend the automated deploy: https://github.com/watson-developer-cloud/conversation-connector/tree/master/channels/facebook#automated-deployment due to some of the reasons you mentioned.  For what it is worth I do run the scripts in Git Bash's mingw64 shell. It sounds like you figured that out, is your solution working for you or might you be able to try the automated approach?

going to close this as I believe there are workarounds, using a shell emulator as you mentioned with some tweaks to the script, as well as an automated solution.  If neither is working for you please let us know.


A spelling mistake in CONTRIBUTING.md file

At line 37 in CONTRIBUTING.md,
Instead of 'pytest' , it spelled as 'pyest'.

@izam-mohammed would you like to fix it?

Issue solved


Statistics module

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

219 of 228 (96.05%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 98.876%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

onebarangay_psql/statistics/viewset.py
51
60
85.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.1%

Covered Lines:
792

Relevant Lines:
801

💛  - Coveralls


Support delete command to control backup retention/removal

Add support for delete command to wale.sh and add example cron jobs for 7 day backup retention.

Thank you.


Is it possible to render KML/KMZ using this library (for Google Maps)?

Has anyone tried/failed/managed to load Google Maps data using KML or KMZ?
Or is there any way to load My Maps data?
Any feedback on this matter would be of interest.

A need it too, i choose for make it with json but it is slow when show the polygons

[ Off-topic but related ]
It would be cool to also have built-in KML support for iOS maps via react-native-maps.
For a basic KML file containing place marks markers it would not be too difficult to convert the KML lat / lon to react-native-maps markers.   Implementing more complete KML support such as paths and stuff would be quite challenging ( but an interesting project ).

It seems like this utility, though in beta, would make it easy to add a kml layer to maps and expose the import call to RN?
I haven't looked at the react-native-maps internals much but presumably there is a place for KmlLayer layer = new KmlLayer(getMap(), R.raw.kmlFile, getApplicationContext()); to go without too much refactor? :)

I also need this, any plan for it? Did somebody figure out the required changes?

For anyone curious, I figured out how to do this. The general strategy I used as follows

Download and parse KML file using toGeoJSON https://github.com/mapbox/togeojson to JSON
Created my own translation of my KML to arrays & objects that I could render on the screen
Used polylines, circles, etc from the map library per whatever type of data I had extracted to render on the screen

This was a pretty painful process, but worked.

@zzimbler Did you run togeojson within your react native app or had to use node to convert a kml file and then read that file?

@eelia001 I did it all within my React Native app. Happy to help you troubleshoot your issue if you have one or if you want feedback on something you're working on. Shoot me an email<EMAIL_ADDRESS>

@zzimbler what did you mean by "Created my own translation of my KML to arrays & objects"?
Is it a geojson format?

@Jun711 well the togeojson gives you a nice blob of JSON, but at least as far as I know there is no way to automatically pipe that information into the react native map. So you have to manually come up with how you want to display that data on the map with pins, lines, points, etc. By my own translation I mean I used .map and then spit out the elements on the map that I wanted. Does that help? Email me if you have something specific

@zzimbler Sry I forgot to add that i was trying to get @rafaellincoln PR running. I'm using the example provided in version 0.21.0 with his PR built in. I'm getting only one marker called "TEST" which is hardcoded, but the markers defined in the KML from pastebin won't show. Tested this on two machines from ground up. Any Ideas?

Fixed it. PR linked above.

Hi @zzimbler , I shot you an email :)

Any chance someone could point me in the right direction in how to use the toGeoJSON library inside react-native? I can't find anything about this.
Also, @zzimbler , I shot you an email, hope you don't mind!

Hi @wtfkeklol ,
I'm using the library react-native-geojson.
There is an example on the github page, it should be simple.

Hey folks,
For what it's worth, in case anyone comes up against this problem again, here's how we handled it.
Cheers,
Ross

Say you've got a functional React Native component:
`
import MapView, { Polyline } from "react-native-maps"; // for rendering
import Geojson from "react-native-geojson"; // optional: for rendering
import { DOMParser } from "xmldom"; // for parsing your KLM string, converting it to an XML doc
import { kml } from "@tmcw/togeojson"; // for converting KLM docs to JSON
export const Foo = () => {
const kmlStr = "bar"; // some example KML from a file or endpoint, nab it as a string
const parser = new DOMParser();
const routeKml = parser.parseFromString(kmlStr);
const routeJson = kml(routeKml);
const routeCoords = routeJson.features[0]; // where we had a walking route line feature at this index
return (
   <MapView>
       <Polyline
           coordinates={routeCoords}
           strokeColor={"orange"}
           strokeWidth={6}
           lineCap="round"
           lineDashPattern={[0]}
        />
   </MapView>
)

}
`
You'll end up with something like this:

If you don't want fine control over what components render your features then "react-native-geojson" is an option available.
Just drop the Geojson component inside your :
<MapView>    <Geojson geojson={routeJson} /> </MapView>


[FLINK-18045] Fix Kerberos credentials checking

What is the purpose of the change
This pull request brings back checking if the enabled security method is Kerberos, that's an old bug reintroduced during some refactoring.
Code assuming that the enabled security method is Kerberos caused problems on MapR secured clusters when deploying on YARN.
Brief change log

Checking if the auth method is Kerberos was added to org.apache.flink.runtime.util.HadoopUtils.isCredentialsConfigured method.

Verifying this change
This change added tests and can be verified as follows:

Added a test that validates that the modified method doesn't check for Kerberos credentials when auth method is other than Kerberos

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: yes
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

thank you for the review @walterddr, I've addressed all the comments and added some more unit tests
I've also introduced a small change in hasHDFSDelegationToken method - taking advantage of the UGI parameter instead of retrieving it one more time seems more natural

thanks for the quick update. in general the patch looks good to me. I still have only one concern regarding the alternation of 2 public APIs. but IMO HadoopUtils are more of an internal runtime util that was shared between different deployment environments. thus I am not sure whether the standard public API alternation rules applies here.
I would wait for @tillrohrmann for a final review.

I think I know what the problem is: We are reusing forks and another test has loaded sun.security.krb5.Config before without having specified the java.security.krb5.realm and java.security.krb5.kdc. That's why sun.security.krb5.Config.getDefaultRealm fails.
@krasinski could you try to add sun.security.krb5.Config.refresh(); as the last line to HadoopUtilsTest.setPropertiesToEnableKerberosConfigInit? This should try to reload the Kerberos configuration.

I've pushed a small hotfix addressing my comments. @krasinski and @walterddr could you check whether the changes (especially splitting up the method in HadoopUtils) make sense to you?

Looks good to me.
One additional thing comes to my mind: when I started working on the fix to address review comments I've encountered a problem with java modules when using sun.security.krb5.Config.refresh() so adding a new line in java 11 profile compiler args in the main pom.xml might be needed:
--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED

@krasinski thanks for the pointer. I will update the PR accordingly.

Thanks for the fix @tillrohrmann . Looks good to me :-)


fatal error: concurrent map read and map write

github.com/go-playground/universal-translator.(*translator).Add(0xc000a66cc0, 0x190e280, 0xc0006aff10, 0x1b22580, 0x12, 0x0, 0xc000a66f00, 0xc0006fa288)

github.com/go-playground/universal-translator/translator.go:100

I have this problem too
any fix?

Can you please provide an example?
the only reason I can see for that to error is if you are adding/modifying translations at runtime while its being used.

In my case, putting Swagger router after translator middleWare causes the error
router.GET("/swagger/*any", ginSwagger.WrapHandler(swaggerFiles.Handler))

Can I get some links or more complete examples?
I have no idea what code/packages are in play here or how it’s wired together


Feature request: show object properties description

Hey there, great job with pretty-swag. Exactly what I was after. Would it be possible to show somehow the description of the schema properties?
Thanks!

I'm still deciding where to put it.

pop-over on mouse over?
make it more like  a comment next to the row?

    {
        "id": "string" /* an id of the item */
        "x" : "number" /* an X position of the cursor */
    }

or simply put a link to the model section

I would with comment next to the row, visible only if there is a
description. Is nice to get a quick overview without "extra" clicks
On Thu, 16 Feb 2017 at 6:01 pm, tj<EMAIL_ADDRESS>wrote:

I'm still deciding where to put it.

pop-over on mouse over?
make it more like a comment next to the row?

{
    "id": "string" /* an id of the item */
    "x" : "number" /* an X position of the cursor */
}

or simply put a link to the model section

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twskj/pretty-swag/issues/2#issuecomment-280526936,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5QK66Zk0-TvU8uSVpcwuN3aL0xmzdXks5rdP9egaJpZM4MCqAS
.

I lean toward the 2nd option too.

rolled out.


The generated code is making weird Type errors

Following channel:
"runs.command.start.{type}.{run_uid}": {
	"description": "This starts a run",
	"parameters": {
		"run_uid": {
			"schema": {
				"type": "string"
			}
		},
		"type": {
			"schema": {
				"type": "string",
				"enum": ["openapi", "asyncapi"]
			}
		}
	},
	"subscribe": {
		"summary": "Start a run",
		"operationId": "startRun",
		"message": {
			"$ref": "#/components/messages/startRun"
		}
	},
	"bindings": {
		"$ref": "./bindings.json#/components/channelBindings/StartRun"
	}
},

I would think this is sufficient:
await nats.jetStreamPublishToRunsCommandStartTypeRunUid(
	startRun,
	{
		runUid: data.run.id,
		type: 'asyncapi',
	},
	data.type === GeneratorTypes.OPENAPI ? 'openapi' : 'asyncapi'
)

But I get type errors:

The generated RunsCommandStartTypeRunUidParameters type:
import {Type} from './Type';
class RunsCommandStartTypeRunUidParameters {
  private _runUid: string;
  private _type: Type;

  constructor(input: {
    runUid: string,
    type: Type,
  }) {
    this._runUid = input.runUid;
    this._type = input.type;
  }

  get runUid(): string { return this._runUid; }
  set runUid(runUid: string) { this._runUid = runUid; }

  get type(): Type { return this._type; }
  set type(type: Type) { this._type = type; }

  /**
   * Realize the channel/topic with the parameters added to this class.
   */
  public getChannelWithParameters(channel: string) {
    channel = channel.replace(/\{run_uid\}/g, this.runUid)
  channel = channel.replace(/\{type\}/g, this.type);
    return channel;
  }
}
export { RunsCommandStartTypeRunUidParameters };

This error is not only happening in this instance, but every channel generated.

Nevermind it is be who doesn't understand the CLI properly. The params are a class and need to obviously be instantiated first.
new RunsStatusRunUidStoppedParameters({
    runUid: run.id,
})

😅


[FRONTEND] Tensorflow Op: Conv2DBackpropInput

the following errors occurred when I was trying to import tensorflow models with this script.
Any plans to support these 2 operators? Thanks!
NotImplementedError: The following operators are not implemented: set([u'Slice', u'Conv2DBackpropInput'])

@simpzan thanks for reporting this.
Slice should be a straight forward implementation. Conv2DBackpropInput required effort.
You are welcome to contribute a part or all of it if interested.

PR #4300 is for Op 'Conv2DBackpropInput', which is the graph node name of 'conv2d_transpose' Op in tensorflow.


Update amazon rules

Add 'hvlocint', 'hvdvcmdl', 'hvptwo', 'hvpone', 'hvpos' to amazon rules
Fix https://github.com/DrKain/tidy-url/issues/61

As with the other issue, have you tested to ensure these are not essential to site functionality?


[Bug report] wrong body name given by getHttpOperation of @typespec/http for anonymous body type

For this kind of tsp definition, author doesn't define body name so its name shall be '' and emitter will make a name by themselves (usually body). However, getHttpOperation returns the httpOperation which has body name resource which I think is unproper.
httpOperation = getHttpOperation(context.program, operation);
name = httpOperation.parameters.body?.parameter?.name ?? "body";  //  we shall get "body" instead of "resource"

It breaks  at least python emitter and openapi emitter:

It is not bug since template has default body name resource: https://github.com/Azure/typespec-azure/blob/23afef77d3cb66a72ddef39735f460471918d55a/packages/typespec-azure-core/lib/foundations.tsp#L62


Session.findByIdAndDelete missing?

In the logout and session APIs there are calls to Session.findByIdAndDelete but I am not finding that method in the Session model (or a search of the repository).

Thanks for creating an issue. It's not missing. Models are extended classes of mongo-models.

https://github.com/jedireza/mongo-models
https://github.com/jedireza/mongo-models/blob/master/API.md

Sorry, I was porting to postgres and had removed the reference to mongo-models and did not remember to look there.  Thanks for the quick response.


[Intro to iOS] Project 2 - Rotten Tomatoes

My app is complete, please review.
/cc @codepathreview
/cc @chug2k

Way to customize the launch screen! :tomato:
I didn't mention this in class, but please be sure to follow the Swift code styling conventions for the homework. Here are a few useful guides:

Ray Wenderlich Swift Style Guide
Github Swift Style Guide

Your code is pretty close to the style guide, so good job.
In class I only used a NSDictionary, but it would have been nice if you implemented a model class. The SwiftyJSON library is useful for this - https://github.com/SwiftyJSON/SwiftyJSON.

Nice work with the custom Movie cell.
You shouldn't have to do      dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_LOW, 0)) { for the URL loading, as NSURLConnection.sendAsynchronousRequest already runs in the background.
Nice work loading the images asynchronously.
Good job resizing images.
Nice work implementing the error message on network failure, even with a close button.
You should also show "Network Error" for the Movie Details screen, when you try to load the high resolution image and there is no network.
Why do you have two copies of all the code? DVDCell, MovieCell, DVDDetails, MovieDetails...this seems like too much copy and paste!
Only MovieDetails has a scrollview, which is great. Good work on that.
 How would you keep the original aspect ratio?

Great work. Your application looks very good.

Wow, thank you, sir! I really appreciate your comment. :)
About the duplication of DVDCell, DVDDetails, did you mean that I should only duplicate the scenes in the StoryBoard and should not duplicate the code?

yes. there are a few different ways to do it, but i am confident you could
think about it. =)
On Mon, Sep 7, 2015 at 12:57 PM vuminhkhang1995<EMAIL_ADDRESS>wrote:

Wow, thank you, sir! I really appreciate your comment. :)
About the duplication of DVDCell, DVDDetails, did you mean that I should
only duplicate the scenes in the StoryBoard and should not duplicate the
code?
—
Reply to this email directly or view it on GitHub
https://github.com/vuminhkhang1995/-Week-1-Assignment-1-Rotten-Tomatoes/issues/1#issuecomment-138196546
.


Add a tag for 1.9.3

Please, add a git tag for the release of version 1.9.3. Otherwise Fdroid's build chain wouldn't start.

Looks like I might have been a bit careless with that release and not tagged the cut that I built from. That was bad.  I'll do better for the 1.9.4 release.  Can you tell me how it affects Fdroid?

I believe FDroid just doesn't know about the 1.9.3 release because it's own release process is triggered by Github hooks. In other words, it listens for the tag to be pushed to Github, and then proceeds to pull down that Git tag, build, sign, and release to its own app store.

Got it.  Thanks Sean - that makes sense.  I think rather than me trying to
guess at what the code looked like when I cut that release I'll just wait
for the next one and do it properly then.  I'm not sure the Fdroid release
is working at the moment anyway. I have a feeling it got broken and I
disabled it alas.
On Sun, 24 Mar 2019 at 21:14 Sean Barbeau<EMAIL_ADDRESS>wrote:

I believe FDroid just doesn't know about the 1.9.3 release because it's
own release process is triggered by Github hooks. In other words, it
listens for the tag to be pushed to Github, and then proceeds to pull down
that Git tag, build, sign, and release to its own app store.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/219#issuecomment-476022789,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APq7C2KVlkIzp8ZPnMv3qzxq2RODajOkks5vaCLpgaJpZM4Y-_Df
.

Eventually you could also just tag the specific commit for the release now? Because F-Droid still can't build it because of the missing tag....

Yes, please just tag commit 4242d88281973c7ae3e380702d3f121cdb77f288 (with the version number change). You can use the tag 1.9.3-published-fdroid, then it doesn't matter whether this is exactly the same code as in Google Play.

There is a 1.9.4 tag now, F-Droid picked it up and I've just pushed some fixes to make it build. The new version should be available soon.

Thanks Jochen. I'm almost completely ignorant of what's needed for F-droid
so appreciate the help you've given here.
Having a bit of trouble with my set up this morning, but I'll merge your PR
soon.
On Sat, Aug 8, 2020 at 7:30 AM Jochen Sprickerhof<EMAIL_ADDRESS>wrote:

There is a 1.9.4 tag now, F-Droid picked it up and I've just pushed some
fixes to make it build. The new version should be available soon.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/219#issuecomment-670918155,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5LWCYUPHEEEKCRYVPL5LTR7UZOHANCNFSM4GH36DPQ
.

I think with 1.9.4 this bug should be obsolete now....


Can't play videos from google drive generated from meet

Describe the bug
I have (college) classes to watch hosted on google drive generated through google meet that won't play. They get stuck on time 0:00 with that spinning circle.
Screenshots

Environment (please complete the following information):

OS/Platform and version:
5.14.16-arch1-1
ungoogled-chromium version:
Version         : 95.0.4638.54-1

Open the eye at the top left and disable cross-site cookie blocking.

It worked. Thanks!
On Mon, Nov 8, 2021, 15:10 Théo @.***> wrote:

Open the eye at the top left and disable cross-site cookie blocking.
[image: 140627838-f4345b5d-2831-4193-b47f-c5be70e0f9f0]
https://user-images.githubusercontent.com/59197790/140795079-d42c9e85-170f-4e43-94c4-c7927c169398.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Eloston/ungoogled-chromium/issues/1729#issuecomment-963427104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH4Z6AJU7QCR77IYFX7GLJDULAHAHANCNFSM5HQFSH4Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Fix nonunified

The previous name (Visual Studio Team Services) and the new name (Azure DevOps) are mixed in this repository. So, I unified it into a new name.
Remove unnecessary space (double spacing enough for line breaks).

@changeworld : Thanks for your contribution! The author, @TerryLanfear, has been notified to review your proposed change.

@changeworld : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

@changeworld : Thanks for your contribution! The author, @TerryLanfear, has been notified to review your proposed change.

@changeworld - You are correct that the name needs to be updated. Thank you for bringing this to my attention. I will update.

@TerryLanfear
I fix comment and send Pull Request #18167.

@changeworld - thank you, I am reviewing PR #18167.
@kasangi, @mlearned  - Thank you for your review and help. Your suggestions are being made via PR #18167.


[#147829023] Force TLS for MySQL service instances

What
Significant things:

force TLS for all connections to MySQL service instances
improve the integration tests for MySQL by testing connection and rebind permissions
refactor the tests so that there's no duplication between Postgres and MySQL
stop creating orphaned snapshots in the CI environment

How to review

It's probably best to review the commits one-by-one, reading the commit message, and viewing the diff without whitespace changes
Confirm that the integration tests pass (status will be submitted to this PR)
Confirm that the test coverage hasn't been reduced
Double check my rationale about not changing URIs
Deploy and check the acceptance test in alphagov/paas-cf#976

After merging you will need to:

Update the submodule in alphagov/paas-rds-broker-boshrelease#40

Delete these snapshots in the CI environment:
aws rds describe-db-snapshots | jq '.DBSnapshots[].DBSnapshotIdentifier | select(startswith("build-test-"))'

I haven't tested, but you can probably pipe to:
xargs aws rds delete-db-snapshot --db-snapshot-identifier

Who can review
Anyone

LGTM

Snapshots deleted with:
for snap in $(aws rds describe-db-snapshots | jq -r '.DBSnapshots[].DBSnapshotIdentifier | select(startswith("build-test-"))'  ); do aws rds delete-db-snapshot --db-snapshot-identifier $snap;done


Check icon is distorted in safari

Can't seem to replicate this issue. Please confirm the version.


refactor relay selector to imporve UX/UI

Based on https://github.com/digi-monkey/flycat-web/issues/396 and https://github.com/digi-monkey/flycat-web/issues/389, refactoring the RelaySelector component can bring a better UX/UI and gradually improve code quality.
For this purpose, we need to do two things:

[x] Avoid relying on the antd's Cascader component, build a shared Cascader component for a better UX experience

[x] Build a basic cascader component by using Radix UI and TailwindCSS
[x] Use Storybook to help the development of shared components
[x] Use the new design system style and add it to the TailwindCSS config
[x] Improve the functionality of cascader to meet the requirements of the RelaySelector component

[x] Refactor the RelaySelector component while keeping the logic unchanged

[x] Show relay groups in the cascader
[x] Update status after switch relay groups
[x] Redirect to relay manager when click "Manage Relays..." option
[x] Improve RelaySelector style on default layout and mobile layout

@digi-monkey This PR is ready for you to review for now, feel free to report bugs/issues when you review the code or preview deployment.

the only bug I found is this z-index thing in mobile mode, otherwise it seems perfect and much better than the original antd thing

@ahonn the only bug I found is this z-index thing in mobile mode, otherwise it seems perfect and much better than the original antd thing

I also found this bug, after some dig I found that the z-index in the project is not well managed and there are many different values. I will try to solve this problem separately and then propose a PR to deal with the z-index issue.

@ahonn the only bug I found is this z-index thing in mobile mode, otherwise it seems perfect and much better than the original antd thing

I also found this bug, after some dig I found that the z-index in the project is not well managed and there are many different values. I will try to solve this problem separately and then propose a PR to deal with the z-index issue.

sounds reasonable


Reduce peer_file_lock usage, move operations to memory

create trust_pool index, do not store trust in peer files

adjust_trust to be reworked/removed

introduce individual pools as in trust_pool then dump them periodically to files

new function: dump_peer_data


[WIP] Add set_memory_request_to_limit option

Add set_memory_request_to_limit option. Also adds a few clarifications addresses in #403

Coverage remained the same at 23.766% when pulling 90db0e545b7f0850866024a3569fd2438818b476 on set-pod-memory-requests into e39915c96894acdf018d870167acb3fc3f03b26c on master.

:-1:
RequestIsSmallerThanLimit incorrectly processes clusters manifests w/o explicit memory requirements/limits

hey folks, is there an ETA on getting this to stups-test ?


Support running the site and visual tests at the same time through grunt

Despite the two pages running on different ports they both use the 'watch' component which I believe uses the same port, meaning the two cannot be run in parallel.

The live-reload URL is hard-coded in all of the visual test html files


Hello everibody,, how can i upload my image from dir or camera, and then save the file to database..,, i use golang for backend

Steps to Reproduce
Please tell us what you were doing and what went wrong. If you are running flutter tools from the command line, please try adding the -v or --verbose option to gather more information.
If the problem is with your application's rendering, please attach a screenshot and any relevant source code.
If you are getting an exception in the logs, and your code is implicated in the first few frames, then please include the source code for the functions involved.
Logs
Run your application with flutter run and attach all the log output.
Run flutter analyze and attach any output of that command also.
Flutter Doctor
Paste the output of running flutter doctor here.

For more information about diagnosing and reporting Flutter bugs, please see https://flutter.io/bug-reports/.

One way you could do it is post the image via a networking request to the server with go backend.
Checkout out you can configure a networking request here https://flutter.io/networking/. This will show you how to configure the post. The same concepts apply when building a web app using a third party service, posting an image to the server. That's only one way to do it, there are other ways you can ship the picture bits to the server, but I think the networking request is one of the easiest to configure.

Hi! Thanks for the question. For this kind of "How To" question, please use Stack Overflow: https://stackoverflow.com/tags/flutter


[CI] support python 3.13 and add windows tests back in

Too soon! sgp4 doesn't support Python 3.13


[AutoPR cognitiveservices/data-plane/Face] [Cogs Face] Align with latest released version of Face API with million-scale features.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3552

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3011


internal: Narrow add_missing_match_arms assist range

Contributes to #10220 with logic borrowed from #10267.
Note: if anyone has recommendations for further analyzers to check, I'm happy to (hard to do it on my own, I'm completely new to the language).

changelog fix (first contribution) narrow add_missing_match_arms assist range


[Loot Tracker] Other Actions in Grubby Chest area adds to Grubby Chest Count

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 10
Bug description
Steps to reproduce:

"Open Grubby Chest" while holding a Grubby Key or Right click option "Check Grubby Chest"
Attempt various actions (eat food, fletch, certain teleports, etc.)

Unexpected effect: The Loot Tracker counter for Grubby Chest increases without opening the chest
Tested:

Triggering: House tab, Standard Spellbook teleports, Ectophial, Fletching Willow logs, Bury Bones, Eat Mackerel
Not triggering: Khardest Memoirs teleport out, Xeric's Talisman Teleport out, decanting potions.

Not tested: Not already having a Grubby Chest loot tracked.
Additional Notes:
Changed to safe mode and tested again, behaviour still occurs
Screenshots or videos

RuneLite version
<IP_ADDRESS>
Rev 216
2.6.3
Logs
client.log
launcher.log

I couldn't replicate it with "Open" chest, but it was definitely an issue with the "Check" option. I created a pull request with some more info in #17209.


Unable to set terragrunt remote bucket encryption, access logging, etc.

Hello
A little background.
I have several AWS accounts under TG and TF control, some work started in the middle of 2019. Last account created in July 2021.
terragrunt version v0.31.4
Here is my remote state conf:
remote_state {
  backend = "s3"

  config = {
    bucket = "fn-terragrunt-state-${get_env("TF_VAR_region", "")}-${get_aws_account_id()}"
    accesslogging_bucket_name = "fn-terragrunt-state-${get_env("TF_VAR_region", "")}-${get_aws_account_id()}"
    key            = "${path_relative_to_include()}/terraform.tfstate"
    region         = "${get_env("TF_VAR_region", "")}"
    encrypt        = true
    dynamodb_table = "terragrunt-locks"
    enable_lock_table_ssencryption = true
  }
}

Now, on the most recent bucket missing is:

access logging

On a bucket from 2019 missing is

default encryption
public access blocking
bucket policy (enforce SSL only)

If I try to create new bucket for the state (change location in config) the bucket is created as expected, with all of the settings in place.
When I run apply, init, etc, nothing changes on existing buckets. The only thing I was able to enforce was lock table encryption.
Expected behavior:
When running apply or init existing remote state bucket settings should be updated with the defined configuration that is:

access logging
default encryption
public access blocking
bucket policy (enforce SSL only)

My question is: is this a bug or expected behavior? If expected behavior, how can I enforce updating old buckets with new, proper settings?
Thank you!

Currently, Terragrunt only creates and configures a remote state bucket if it doesn't exist already. If the bucket does exist, Terragrunt does not try to change its configuration.
We probably should support this behavior. If your remote_state config says the bucket should have some setting X, and Terragrunt sees that it doesn't, it should probably prompt you whether to update the bucket, and fix it if you say yes. Perhaps there should also be some flag to disable this behavior for users that manage their buckets outside of Terragrunt.
PRs welcome.

Thanks @infraredgirl. I also think there is slight inconsistency there - when I set "enable_lock_table_ssencryption" it did update all my lock tables to use KMS w/o asking for permission.

+1 I'm experiencing this today on Terragrunt 0.36.6 (terraform 1.1.7). We have a large AWS organization consisting of over 80 accounts with several hundred accounts to be created when we're done migrating. In an existing dev environment, when changing the access logging settings the existing settings are not changed.

Fix released in https://github.com/gruntwork-io/terragrunt/releases/tag/v0.37.0


implement frosted ice and frost walker

This PR implements frosted ice and frost walker as defined by the linked pages.
Some features of frost walker are currently impossible to implement:

Foxes can pick up boots with frost walker, but foxes do not exist in Dragonfly
Frost walker prevents damage from campfires and magma blocks when stepped on, but neither exist in Dragonfly

Currently, the main function contains code that spawns players with boots enchanted with frost walker one and two, and a bucket of water, for testing purposes.

FrostedIce must implement both the ScheduledTicker and NeighbourUpdateTicker, and have scheduled updates queued on neighbor updates.

Why?


chore: refactor test

remove editor_focus_selector variable since it is no longer needed
adjust each timeout
remove a flaky test case

Perhaps this will improve CI.

thanks so much!


Install script returns error

I installed the Jupyter notebook inside a virtual environment by following these instructions. When I try to run the installer, I get one of two errors depending on whether I use the sudo command. This happens whether or not the java folder and the install script are inside the virtual environment or not. Do I need to run the installer in a specific directory or with specific arguments? Where should the java folder be?
Without sudo:
(myprojectenv) username@penguin:~/myprojectdir$ python3 install.py
Traceback (most recent call last):
File "install.py", line 169, in
replace=args.replace
File "/home/username/myprojectdir/myprojectenv/lib/python3.5/site-packages/jupyter_client/kernelspec.py", line 346, in install_kernel_spec
shutil.copytree(source_dir, destination)
File "/home/username/myprojectdir/myprojectenv/lib/python3.5/shutil.py", line 315, in copytree
os.makedirs(dst)
File "/home/username/myprojectdir/myprojectenv/lib/python3.5/os.py", line 231, in makedirs
makedirs(head, mode, exist_ok)
File "/home/username/myprojectdir/myprojectenv/lib/python3.5/os.py", line 231, in makedirs
makedirs(head, mode, exist_ok)
File "/home/username/myprojectdir/myprojectenv/lib/python3.5/os.py", line 241, in makedirs
mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/usr/local/share/jupyter'
With sudo:
(myprojectenv) username@penguin:~/myprojectdir$ sudo python3 install.py
Traceback (most recent call last):
File "install.py", line 6, in
from jupyter_client.kernelspec import KernelSpecManager
ImportError: No module named 'jupyter_client'

Never mind, I was able to install by running the install command with --system-prefix while inside of the virtualenv.
(myprojectenv) username@penguin:~/myprojectdir/myprojectenv$ python3 install.py --sys-prefix


AES CTR-DRGB: performance improvement

Optimize the the AES-based implementation of the CTR_DRBG
construction, see 10.2.1 in [1].
Due to the optimizations, the code may deviate (more) from the
pseudocode in [1], but it is functional equivalence being decisive
for compliance:
"All DRBG mechanisms and algorithms are described in this document
in pseudocode, which is intended to explain functionality.
The pseudocode is not intended to constrain real-world
implementations." [9 in [1]].
The following optimizations are done:

Replace multiple plain AES encryptions by a single AES-ECB
encryption of a corresponding pre-initialized buffer, where
possible.
This allows platform-specific AES-ECB support to
be used and reduces the overhead of multiple EVP calls.

Replace the generate operation loop (which is a counter
increment followed by a plain AES encryption) by a
loop which does a plain AES encryption followed by
a counter increment. The latter loop is just a description
of AES-CTR, so we replace it by a single AES-CTR
encryption.
This allows for platform-specific AES-CTR support to be used
and reduces the overhead of multiple EVP calls.
This change, that is, going from a pre- to a post- counter
increment, requires the counter in the internal state
to be kept at "+1" (compared to the pseudocode in [1])
such that it is in the correct state, when a generate
operation is called.
That in turn also requires all other operations to be
changed from pre- to post-increment to keep functional
equivalence.

[1] NIST SP 800-90A Revision 1
Signed-off-by: Patrick Steuer<EMAIL_ADDRESS>Reviewed-by: Tomas Mraz<EMAIL_ADDRESS>(Merged from https://github.com/openssl/openssl/pull/10457)
(cherry picked from commit 28bdbe1aaa474ae8cd83e520d02e463e46ce89d9)

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@p-steuer FYI, I cherry-picked your commit and fixed a few trivial conflicts,
I hope you don't mind.

@bernd-edlinger Im fine with it .

This pull request is ready to merge

Merged: OpenSSL_1_1_1-stable 53eb05b
Thanks!


Add Enum argument resolver

Closes #221
Now it doesn't work for some reason - I get No parser for RawInput -> dev.rollczi.litecommands.annotations.argument.resolver.standard.EnumArgumentTest$TestEnum for test and testInvalid not being failed. I can't understand, why

Thanks, @Rollczi, looks like everything is working now


MM-40214: Add App Bar icon

Summary
This PR adds a new icon explicitly for the App Bar, so that when it is enabled, the proper icon is shown. The logic for rendering the channel header icon or the App Bar one is in the webapp, not here.
I needed to do a couple of things to build this locally:

Use node 14.
Run go get -u golang.org/x/sys, which is added in the first commit.
Use the fix from this open PR. I have not added that as a commit here to avoid conflicts in that other PR, but if you're building locally, note that you may need it.

I also added a commit to fix CI, adding the --verbose solution we've used in other plugins to avoid CI from timing out.
Demo
https://user-images.githubusercontent.com/3924815/172824065-ad4e1010-f263-40c4-8a96-fcdeb327f673.mp4
Ticket Link
https://mattermost.atlassian.net/browse/MM-40214

Note that we're aiming to release a new version of the plugin with this PR before the release of v7.0 on June 15 (Wednesday next week).

Swapping Daniel with Lev because Daniel's out today and tomorrow.

@mickmister, I pushed a change to the estlint rules just 30 seconds after you reviewed :joy: I think it's sensible, since otherwise the following block returns an error:
const action = (channel: Channel) => {
    store.dispatch(startMeeting(channel.id));
};

But I wanted you to take a look. That's why I'm asking for a re-review :)

Heh, this is failing because of the error solved by #211. @DHaussermann, if you can take a look at that PR before this one so I can merge that fix in my branch, that'd be awesome. Let me know if you need anything from me, please!

@agarciamontoro this is good to merge but depending on when a release with this is needed, we may need to package a separate release with this change.

@DHaussermann, ok. As this PR depends on the build fix implemented in https://github.com/mattermost/mattermost-plugin-jitsi/pull/211, I will add that fix here, merge this PR and fix the conflicts in #211 so you guys can merge it whenever you want. Then I can create a new release-2.0 with only this PR on top of v2.0.0, release a new v2.1.0 from there and publish it in the Marketplace. Then you can follow your usual release schedule, if that makes sense.


[server] Transaction isolation

Steps to reproduce:

Create a tictactoe session
Play until OXOXXO OX
Make noughts spam X6 move
Make crosses send X6 during the spamming

Expected result: game finishes and crosses win
Actual result: Game might drop into an invalid state: OXOXXO OX, but is_terminal=true. Therefore, the board is not filled, but no one can move and the game is considered to be completed.

fixed in https://github.com/cavoke-project/cavoke/commit/ef6565744f5fefe8e0aa7aadb6cc2e220f0bce9c

Turns out repeatable read is not enough.
Try:
begin;
                                    begin;
select is_terminal from globalstates;
update globalstates set is_terminal = true, globalstate = '1';
commit;
--
                                    select is_terminal from globalstates;
                                    update globalstates set is_terminal = false, globalstate = '2';
                                    commit;

Expected behaviour would be a concurrent update error, but works fine and saves is_terminal = false, globalstate = '2'.
Solution: use serializable default level. serializable is not fast, but drogon currently does not support non-default transaction isolation levels... :(

😠 https://github.com/cavoke-project/cavoke/runs/7729768476


enhancement(dui): Allow access to branches (& streams, commits) via combobox where number of options exceed n

Related to #2436 but less critical.
Selecting branches in DUI would be improved where there is a large number if instead the interface to do so would be from a combo box (combination of search and drop-down) in place of a plain drop-down.

Definitely will come in DUI3!


Secure API Key

Recently discovered, the API key for access to the mPCC API is embedded in publicly available resources in the NADS and image distribution process.  This needs to be rectified as it is a security risk.
A few changes need to happen to mitigate this problem:
NADS changes

[ ] add vendor, model to system model.
[ ] fix NADS boot process to send vendor, model, switch, and switch port along with request to register
[ ] a failure to properly match on register request adds the MAC to a "disallowed" queue and must be manually removed
[ ] new "disallowed" queue in systems(?) that can be manually changed. (optional?)

iPXE and distro boot process

[ ] add check for vendor, model, ip, and MAC to image serve code.
[ ] mprov_boot.sh needs to gather the 4 items above to prove they are a managed system to get a managed image.  The managed image may contain an API key.

for the distro boot/install process, the 4 items can be sent via the wget command in mprov_boot.sh.  If they don't match, give out the default NADS instance.

The changes to NADS and the fact that you need registered MAC to get a distro with the key should be secure enough


feat(tutorial): complete step 5 - Austin

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Per instructions in the tutorial, please include your mybluemix.net URL for your deployed app in your pull request description.

http://carbon-tutorial-sstrubberg.w3ibm.mybluemix.net/#/

👍 Looks good - I have recorded your completion of this step!
Feel free to stop this running application on IBM Cloud to avoid the possibility of incurring any charges 💵


Improve performance for trigger scripts and serverContext

I originally opened this PR using a branch on bbimber/platform, but now I can make branches here, so I'm closing that PR in favor of this.
Background:
I noticed an ETL was taking much longer than previously. When I profiled it in Jprofiler, I saw that this is probably due to constantly recompiling serverContext in RhinoService, which was added in https://github.com/LabKey/platform/pull/3942..
I believe this PR addresses that, by calculating serverContext once, and caching this instead of just caching ContainerUser. When I tried this locally, it restored perf back to close to where it was before.
I also took it a step further. Rather than request the entire jsInitObject and serialize to a string every single time, it makes a whole lot more sense to lazily call jsInitObject only when code requests serverContext. So in summary:

Cache the compiled ModuleScript
Within that script, lazily request jsInitObject, rather than request it upfront and store that big JSON string

@labkey-nicka: I think you reviewed the original feature - thoughts on this?

See more context in https://www.labkey.org/ONPRC/Support Tickets/issues-details.view?issueId=47705


:sparkles: Configuration for domain's email authorization

All points (see #175) are ok but the error's message are wrong.

#178 changed how configuration is handled in both the front and back end, so I will first merge #178 and then rebase this one.


Register Package at Julia General Registry

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/98011
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" e6765030db5413a9049f8d4f90d1cdc9ec37e78c
git push origin v0.1.0

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/98011
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" d7fc5121febe70e96417ad1df4be54da976af65d
git push origin v0.1.0


Should we have recommended project structure?

A number of projects have a structure like the following one:
.
├── CHANGELOG.md
├── LICENSE.md
├── Makefile
├── Makefile.coq.local
├── README.md
├── _CoqProject
├── foo.opam
├── default.nix
├── dune-project
├── meta.yml
└── theories

Others just have every file in the project root directory or some adhoc directory structure.
Of course, it's not possible for all projects to have the same structure, but I think having a recommended structure, especially for small projects, might help with context switching during maintenance.
If you think it could be beneficial to have a recommended project structure then where do we put it? The template repo's README.md? Wiki? Or somewhere else?
Incidentally (or maybe this should be a standalone issue): having an informal template for _CoqProject file would also help, I think.

The manifesto's wiki seems like the right place since it's already used to document other existing conventions.

Thanks for the suggestion, I'll try to have a go at it and will post the link here.

Ok, here you go, an initial draft: https://github.com/coq-community/manifesto/wiki/Recommended-Project-Structure. I took the liberty of recommending some particular choices, e.g. the default choice of CI I put in is Travis CI. This may not be accurate, but I think something has to be recommended anyway. Please feel free to correct any mistakes I made and add more information. I was aiming at newcomers who are not very familiar with the Coq tools and decided to explicitly add some "obvious" things which are folklore in the hope it'll be useful for some.

Good point on having to recommend something. I am of the impression however that we should switch to recommending GitHub Actions instead of Travis CI soon (if not now). The only issue is that we don't (yet) have templates for the GitHub Actions setup using Nix. See also what I wrote in #69.

Fully agreed! I hesitated in favor of GitHub Actions for a bit, but Travis CI is currently used in many community projects. I guess we should switch at least some of them, see how things go and change the recommendation.

I also agree with @Zimmi48 regarding GitHub Actions. The main reason to go with Travis right now is to have the option to build with Nix. However, this is not a strong reason for many projects, and we could hopefully add Nix CI based on GitHub Actions later on.

@anton-trunov Should we close this issue now?
@palmskog Should the index.md file be added to the recommended project structure? (Under resources/?)

Yes, closing this issue sounds fine to me.


Document Development features

Verbose and debugging logging, dry run, etc..

:tada: This issue has been resolved in version 3.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor login / reset password

maybe refactor login / reset password flows to have some actions on the app level?

#42 addresses this solution


Rename leftover Sanitycheck occurances

Various renames

@nashif I just found a few "sanity" left in zephyr/.buildkite/run.sh

@nashif I just found a few "sanity" left in zephyr/.buildkite/run.sh
submit a patch :)

ok:) #30710


Change misleading warning

"This is a development server. Do not use it in a production deployment." was shown if the env was set to production instead of development.

fixes #

Checklist:

[ ] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[ ] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[ ] Add .. versionchanged:: entries in any relevant code docs.
[ ] Run pre-commit hooks and fix any issues.
[ ] Run pytest and tox, no tests failed.

Sorry. My fault. The message is correct. This is a warning that you are using Flask without WSGI container. I saw this message in Airflow that uses gunicorn to serve web server. But the message came from a side log server that did not use WSGI server.
Sorry for engaging you.


Add tool to generate JSON encodings

Closing for now as it is not that cumbersome to encode JSON manually.


Pin pester version to 5.5.0

The explanation of the problem is in the issue tracking the unpinning once the version on Windows is 5.6.0 or higher.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Fix fairseq version identifier in setup.py

Fixes #168

I'm not sure this is needed since the issue may be fixed by uninstall/reinstall?

Btw, this should still pull the latest master commit of fairseq, the version numbers are only loosely enforced by setuptools.

Awesome, that's what I was worried about.


NTR: ploughing process

Linked to #33
We'd like a class to express ploughing activities to axiomatise arable land.
This would need the creation of a class for plough (syn: plow) too
proposed def: "A process during which a plough is mechanically drawn through soil to prepare it for seed sowing or planting"
dbxref: https://en.wikipedia.org/wiki/Plough
syn:

ploughing process = tillage. I have added the synonym to the class ''Tillage''


TEST PR

TEST PR

API changes have been detected in @azure/search-documents. You can review API changes here
API changes
-     get apiVersion(): string;
+     readonly apiVersion: string;
+     readonly serviceVersion: string;
+     serviceVersion?: string;
+     readonly serviceVersion: string;
+     serviceVersion?: string;

/azp run js - search - ci


Molecules are not oriented correctly in minimized sketcher mode

open dataset with molecule column
open filter panel
open sketcher and draw any structure (structure should be placed at an angle)
close sketcher by clicking 'OK' and make sure the cordinates of molecule in minimized sketcher are saved


Add support for SSR and NextJS

Summary
Resolved requested feature mentioned here #49 and here #48
Tasks:

[x] replace normalizer from react-native-web
[x] adjust useDimensions hook to be runnable on server
[x] add demo to example folder
[x] double check tests, we should still aim for ~97%
[x] double check examples
[x] update docs

Test plan
Add ssr folder to example folder and provide a demo of NextJS and SSR

Resolves #49


wip

This change is

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

Project coverage is 78.47%. Comparing base (0e13d53) to head (d985e1f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2001   +/-   ##
=======================================
  Coverage   78.47%   78.47%
=======================================
  Files          62       62
  Lines        8967     8967
  Branches     8967     8967
=======================================
  Hits         7037     7037
  Misses       1483     1483
  Partials      447      447

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


Revert back to port 80 ?

We can try, I was getting an issue I think on the now servers when doing that though. Would just need to change the EXPOSE 3000 --> EXPOSE 80 in the docker file and then remove the -e PORT=3000 when we call.

I also don't think that there's any advantage/reason for us to use port 80. Because I think now just maps whatever port you EXPOSE --> 80. So I'll close this.


How to update a row in a VirtualScroll Component?

Hi,
The VirtualScroll component calls the "rowRenderer" callback function just on scroll. I have a click listener that updates the row, how's it possible to force the rowRenderer to be called?
I have prepared a very simple jsfiddle example to demonstrate the issue i'm currently struggling with:
https://jsfiddle.net/zob62mu6/3/
Appreciate any help!

Here you go: https://jsfiddle.net/bvaughn/db5mas0t/
Just add a ref (eg ref='VirtualScroll') to your VirtualScroll` and then tell it it needs to update in your on-click handler:
listItemClickHandler:function(i){
  this.list[i] = this.list[i] + 1;
  this.refs.VirtualScroll.forceUpdate();
}


Update Remove-Printer.md

Corrected acceptwildcard characters for Name, CimSession and  ComputerName

Docs Build status updates of commit c5e05c2:
:clock10: Pending: waiting for processors (12 builds ahead of you)
:warning: Docs Build is busy, currently there are 12 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit c3f49b5:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 25ebf89:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit c5e05c2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

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

Docs Build status updates of commit 25ebf89:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

docset/winserver2012-ps/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

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

Issue#https://github.com/MicrosoftDocs/windows-powershell-docs/issues/1604

@dariomws306 @get-itips @tseward
Peer review needed for this PR. Thanks.

Have you tested -ComputerName with a wildcard character? I would make the assumption that like most cmdlets, it must be a specific name.
Remove-Printer -ComputerName * -Name "Microsoft XPS Document Writer"
Remove-Printer : The spooler service is not reachable.  Ensure the spooler service is running.
At line:1 char:1

Have you tested -ComputerName with a wildcard character? I would make the assumption that like most cmdlets, it must be a specific name.
Remove-Printer -ComputerName * -Name "Microsoft XPS Document Writer"
Remove-Printer : The spooler service is not reachable.  Ensure the spooler service is running.
At line:1 char:1

Thank you @tseward  You are right. I tested it with a wildcard on the name
Remove-Printer -ComputerName laptop-SIJI125H -Name *micr
I'll update the PR accordingly.

Docs Build status updates of commit f25a120:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

docset/winserver2012-ps/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/printmanagement/Remove-Printer.md
:white_check_mark:Succeeded

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


Logout the user automatically when idle for too long

To avoid that the wallet stays open while not actively used, we could logout the user automatically when they have been idle too long. The idle time we could make configurable in the Settings page.

Fully agree. Since this is security related I propose we strive to implement this for the next release. I'll set up a project page to be able to better prioritize.

Merged. Kudos @digitaltwinnn


Fix #35, Code Coverage

This PR aims to add Code Coverage to the Dart Project.
Access token can be created at https://app.codecov.io/
We have to add Flutter repo to the code coverage site and upload this token in the settings as CODECOV_TOKEN.

Please review it @fredfalcon , @souravbadami .
Thanks

@fredfalcon - Could you please take this up?

I have setup the CODECOV_TOKEN - please update the code so that the Bitrise build passes

will fix this soon.

This seems like a lot of 3rd party dependencies.   Would prefer to not have that many to keep the app more secure.  Can we close this?


sequenceId 回拨问题

我看到客户端是通过 sequenceId 来排序展示的

服务端的 sequenceId 是根据 redis incr 指令生成的，虽然性能很好，但如果 redis 宕机或者主从切换，那么 sequenceId 就会回拨，后面产生的 sequenceId 就会有重复？这类问题有兜底解决吗？

如果 redis 宕机或者主从切换，那么 sequenceId 就会回拨，后面产生的 sequenceId 就会有重复？

It doesn't matter. By the way, all components of Turms servers are designed to work in extreme situations, which means you can crash Turms servers/clusters or their dependencies for fun, and Turms servers will recover automatically.
For this question, sequence ID exists to find out which messages are missing in edge cases. So, if your client detects a 0 sequence ID, what you need to do is just clear/ignore existing sequence IDs and sync the messages between, for example, [timestamp of last received message - minute, now + 1 minute], and deduplicate them by message IDs, and start using the new sequence IDs.
This is just a general solution. For details, I may write a doc on it later.

如果 redis 宕机或者主从切换，那么 sequenceId 就会回拨，后面产生的 sequenceId 就会有重复？

It doesn't matter. By the way, all components of Turms servers are designed to work in extreme situations, which means you can crash Turms servers/clusters or their dependencies for fun, and Turms servers will recover automatically.
For this question, sequence ID exists to find out which messages are missing in edge cases. So, if your client detects a 0 sequence ID, what you need to do is just clear/ignore existing sequence IDs and sync the messages between, for example, [timestamp of last received message - 1 minute, now + 1 minute], and deduplicate them by message IDs, and start using the new sequence IDs.
This is just a general solution. For details, I may write a doc on it later.

seems to be a fantastic solution. I would be appreciated if you could provide more details about that


Upgrade HTTP transport version

Purpose

$Title after fixing memory leak in error path
Build data source before responding content delivery request in WebSub

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@26f8f3f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #14253   +/-   ##
=========================================
  Coverage          ?   64.76%
  Complexity        ?      425
=========================================
  Files             ?     2179
  Lines             ?    96121
  Branches          ?    12827
=========================================
  Hits              ?    62249
  Misses            ?    28845
  Partials          ?     5027

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 26f8f3f...d65a28d. Read the comment docs.


Notebook kernel continuously reconnecting

Hi,
I followed the GPU installation instructions on an Ubuntu 16.04 machine with an Nvidia GPU installed properly ( I used it for other things and it seems to work).
After running the container using
sudo nvidia-docker run -v ~/Learning/TensorFlow-LiveLessons:/home/jovyan/work -it --rm -p 8888:8888 tfll-gpu-stack
The kernel connection keeps flashing back and forth between connected and reconnecting and in the terminal, I'm getting repeated messages of Adapting to protocol v5.1 for kernel 8039... with no apparent error message.
Any ideas how to debug this? This prevents me from running any notebook and properly following your course.
Thanks
Tomer

Hi @tomersagi, I'm afraid this is my first time coming across this error. My recommendation would be to try the CPU-only install. You don't need a GPU for any of the course. Nearly all of the notebooks will complete running within a few minutes on the CPUs alone of most machines.

Yep, not your fault at all. I tracked it down to my University's Kaspersky anti-virus system. It was blocking the web socket used by jupyter to connect with the kernel.
Sorry for the bother. Great course BTW!

Ah, very well then! I'm glad you're enjoying it, @tomersagi 😃


A0-2975: add counters for errors

Description
Counters for errors, where possible. In handle_state_response and handle_request_response its hard to tell whether there was an error or success, so we count everything.
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality

I would prefer if @ggawryal 's comment got implemented, the code duplication now is pretty extensive and it was a very good idea for lowering it somewhat.

hmm,  Now code duplication is inside the file metrics.rs while with the enum solution, we will crowd the code in the service.rs with the additional use  MetricsEnum::* in every function. So for me, it's about having more duplications in the metrics code vs more boilerplate code in the service.rs and I prefer to have less of such code in a more critical place which is service.rs and the tradeoff makes it not worth it in my opinion to implement this.
I hate to say that but if we want to lower the code duplication in metrics.rs we could use some macro (ble)

I believe even if the comparison is metrics.report_sync_event(SyncEvent::Broadcast) vs metrics.report_sync_broadcast() the former still wins. You also get much more readable code in metrics.rs -- you can have a pretty clear match over the enum variants mapping into the metric names, separate from the reporting code, easier consistency with the _error suffix, and probably more things. It's just so~ much better organized then. Dx
Oh, and if you don't agree with the first sentence, you can always introduce a MetricEvent trait and parametrize report<E: MetricEvent>(&self, event: M), which will remove the double sync_event there and likely will be quite useful in the future. }:3

I believe even if the comparison is metrics.report_sync_event(SyncEvent::Broadcast) vs metrics.report_sync_broadcast() the former still wins. You also get much more readable code in metrics.rs -- you can have a pretty clear match over the enum variants mapping into the metric names, separate from the reporting code, easier consistency with the _error suffix, and probably more things. It's just so~ much better organized then. Dx
Oh, and if you don't agree with the first sentence, you can always introduce a MetricEvent trait and parametrize report<E: MetricEvent>(&self, event: E), which will remove the double sync_event there and likely will be quite useful in the future. }:3

mhh, ok done :|


brew upgrade Not Updating Casks

When looking to install newer versions of Casks, running brew upgrade then brew cask info [cask] shows an older version of the cask until brew cask update is run.
Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] Confirmed this is problem with Homebrew/brew and not specific formulae? If it's a formulae-specific problem please file this issue at https://github.com/Homebrew/homebrew-core/issues/new

Bug reports:
Having immediately run a brew upgrade just prior, an attempt to identify and install a newer cask shows the older version until brew cask update is run. While I don't have the option to provide output to this effect, a brew update fails to update casks in the same way.
$ brew upgrade
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 1 tap (homebrew/core).
==> Updated Formulae
artifactory                ldc                        rakudo-star
bazel                      libressl                   riak
bit                        logstash                   ruby-build
certbot                    mediaconch                 saldl
conan                      mercurial ✔                skinny
docker-compose             minimesos                  tika
docker-compose-completion  mpd                        vim ✔
filebeat                   nginx                      vnu
grails                     node                       wallpaper
ipmitool                   node-build                 wpcli-completion ✔
jenkins                    nodeenv                    yarn
jsonnet                    oath-toolkit               you-get
knot                       portaudio                  youtube-dl
kobalt                     qt5

==> Upgrading 16 outdated packages, with result:
awscli 1.11.44, vitorgalvao/tiny-scripts/cask-repair 0.30.1, homebrew/php/composer 1.3.2, gd 2.2.4_1, go 1.7.5, imagemagick 7.0.4-6, libevent 2.1.8, libtiff 4.0.7_2, mercurial 4.1, openssl 1.0.2k<EMAIL_ADDRESS>1.1.0d, redis 3.2.7, tmux 2.3_3, vim 8.0.0295, webp 0.6.0, wpcli-completion 1.1.0

... upgrades of above packages removed for some level of brevity ...

$ brew cask info chefdk
chefdk: 1.1.16-1
https://downloads.chef.io/chefdk/
/usr/local/Caskroom/chefdk/1.1.16-1 (2 files, 65.2M)
From: https://github.com/caskroom/homebrew-cask/blob/master/Casks/chefdk.rb
==> Names
Chef Development Kit
ChefDK
==> Artifacts
chefdk-1.1.16-1.pkg (pkg)

$ brew cask update
Warning: Calling `brew cask update` is deprecated and will be disabled on 2017-07-01!
Use `brew update` instead.
/usr/local/Homebrew/Library/Homebrew/cask/lib/hbc/cli.rb:111:in `run_command'

==> Updated 3 taps (caskroom/cask, caskroom/drivers, caskroom/versions).

$ brew cask info chefdk
chefdk: 1.2.20-1
https://downloads.chef.io/chefdk/
/usr/local/Caskroom/chefdk/1.1.16-1 (2 files, 65.2M)
From: https://github.com/caskroom/homebrew-cask/blob/master/Casks/chefdk.rb
==> Names
Chef Development Kit
ChefDK
==> Artifacts
chefdk-1.2.20-1.pkg (pkg)

brew doctor:
$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry and just ignore them. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and what additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew provided
script of the same name. We found the following "config" scripts:
  /Users/ssmith/.pyenv/shims/python-config
  /Users/ssmith/.pyenv/shims/python2-config
  /Users/ssmith/.pyenv/shims/python2.7-config
  /Users/ssmith/.pyenv/shims/python3-config
  /Users/ssmith/.pyenv/shims/python3.5-config
  /Users/ssmith/.pyenv/shims/python3.5m-config

brew config:
$ brew config
HOMEBREW_VERSION: 1.1.9
ORIGIN: https://github.com/Homebrew/brew.git
HEAD: 664d0c67d5947605c914c4c56ebcfaa80cb6eca0
Last commit: 3 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 9f9a3222a276e27ad229bcd6be9a5d69b8862037
Core tap last commit: 11 minutes ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local/Homebrew
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: octa-core 64-bit haswell
Homebrew Ruby: 2.0.0-p648
GCC-4.2: build 5666
Clang: 8.0 build 800
Git: 2.11.0 => /usr/local/bin/git
Perl: /usr/bin/perl
Python: /Users/ssmith/.pyenv/shims/python => /usr/local/Cellar/python/2.7.13/Frameworks/Python.framework/Versions/2.7/bin/python2.7
Ruby: /Users/ssmith/.rbenv/shims/ruby => /Users/ssmith/.rbenv/versions/2.4.0/bin/ruby
Java: 1.8.0_112, 1.8.0_92, 1.8.0_77, 1.8.0_74, 1.8.0_72, 1.8.0_66, 1.8.0_60, 1.8.0_51, 1.8.0_45, 1.8.0_40, 1.8.0_31, 1.8.0_25, 1.8.0_20
macOS: 10.12.3-x86_64
Xcode: 8.2.1
CLT: <IP_ADDRESS>.1.1480973914
X11: 2.7.11 => /opt/X11

Because I forgot. Sorry.
brew cask doctor:
$ brew cask doctor
==> macOS Release:
10.12.3
==> Hardware Architecture:
intel-64
==> Ruby Version:
2.0.0-p648
==> Ruby Path:
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
==> Homebrew Version:
Homebrew 1.1.9
Homebrew/homebrew-core (git revision 9f9a3; last commit 2017-02-02)
==> Homebrew Executable Path:
/usr/local/bin/brew
==> Homebrew Cellar Path:
/usr/local/Cellar
==> Homebrew Repository Path:
/usr/local/Homebrew
==> Homebrew Origin:
https://github.com/Homebrew/brew.git
==> Homebrew-Cask Version:
Homebrew-Cask 1.1.9
caskroom/homebrew-cask (git revision e7646; last commit 2017-02-02)
==> Homebrew-Cask Install Location:
<NONE>
==> Homebrew-Cask Staging Location:
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads:
/Users/ssmith/Library/Caches/Homebrew/Cask
13 files, 429.1M (warning: run "brew cask cleanup")
==> Homebrew-Cask Default Tap Path:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask
==> Homebrew-Cask Alternate Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-drivers
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-eid
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-fonts
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions
==> Homebrew-Cask Default Tap Cask Count:
3530
==> Contents of $LOAD_PATH:
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/Library/Ruby/Site/2.0.0
/Library/Ruby/Site/2.0.0/x86_64-darwin16
/Library/Ruby/Site/2.0.0/universal-darwin16
/Library/Ruby/Site
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby/2.0.0/universal-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/vendor_ruby
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/x86_64-darwin16
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/universal-darwin16
==> Contents of $RUBYLIB Environment Variable:
<NONE>
==> Contents of $RUBYOPT Environment Variable:
<NONE>
==> Contents of $RUBYPATH Environment Variable:
<NONE>
==> Contents of $RBENV_VERSION Environment Variable:
<NONE>
==> Contents of $CHRUBY_VERSION Environment Variable:
<NONE>
==> Contents of $GEM_HOME Environment Variable:
<NONE>
==> Contents of $GEM_PATH Environment Variable:
<NONE>
==> Contents of $BUNDLE_PATH Environment Variable:
<NONE>
==> Contents of $PATH Environment Variable:
PATH="~/.bin:/Users/ssmith/.pyenv/shims:/Users/ssmith/.pyenv/bin:/Users/ssmith/.nvm/versions/node/v6.9.4/bin:~/.pear/bin:/opt/chefdk/bin:/Users/ssmith/.rbenv/shims:/Users/ssmith/.rbenv/bin:/Users/ssmith/.composer/vendor/bin:/usr/local/opt/php56/bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/MacGPG2/bin:/usr/local/opt/go/libexec/bin:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-boneyard/cmd:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-bundle/cmd:/usr/local/Homebrew/Library/Taps/homebrew/homebrew-services/cmd:/usr/local/Homebrew/Library/Homebrew/cask/cmd:/usr/local/Homebrew/Library/Homebrew/shims/scm"
==> Contents of $SHELL Environment Variable:
SHELL="/bin/bash"
==> Contents of Locale Environment Variables:
LANG="en_US.UTF-8"

brew cask update is deprecated and simply an alias for brew update, as the message tells you.
I see nothing wrong, here. Your brew cask update states clearly ==> Updated 3 taps (caskroom/cask, caskroom/drivers, caskroom/versions). Since chefdk was updated recently, this simply looks like it was updated between your info and your following update.
Unless you can reproduce with other casks, I don’t see anything out of the ordinary.

All of these steps were accomplished within about 2 minutes approximately 15 hours after https://github.com/caskroom/homebrew-cask/pull/29678 was merged. I don't see how this isn't a problem in some fashion.

@SeanSith But it may also have been a weird one-time occurrence due to something on your setup/connection/machine.
Like I said, can you reproduce this reliably? Because I can’t reproduce it at all.

To my recollection, yes (it finally got to the point where I decided to file an issue). I'll note one when the next Cask gets merged in github.com/caskroom/homebrew-cask.

@SeanSith Thank you, please do. But please use brew update instead of brew cask update, next time.

I did use brew upgrade as a starting point, which has the effect of brew update does it not?

Yes, I mean in that intermediary step where you used brew cask update, use brew update instead. By your report it seems like the second time you called it, the update was forced again to something it hadn’t done, so it’s just to check.

Note the brew upgrade does not necessarily update tapped Cask repositories for speed reasons. Run brew update to ensure you're up-to-date.

Note the brew upgrade does not necessarily update tapped Cask repositories for speed reasons.

I thought the automatic updates that now occur by default (unless given HOMEBREW_NO_AUTO_UPDATE=1) ran the exact same thing as brew update. So you’re saying they do not, and update only a subset of repos?
If that’s the case, there’s the mystery solved right there.

So you’re saying it does not, and updates only a subset of repos?

Yep, only repos with formulae.

@MikeMcQuaid Thank you for the clarification.
@SeanSith So it’s expected behaviour, just not for the reason I initially thought.

@vitorgalvao @MikeMcQuaid Ok. I can follow that. Any chance we'd consider changing that behavior? I would imagine that new users might not have the brew update && brew upgrade workflow in their fingers. I certainly dropped it whenever brew upgrade started (sort of) doing both.
Any chance the analytics shows the average amount of time it takes to update the most frequently used/default taps?

Any chance we'd consider changing that behavior?

Yeh, I'll consider it if we see a few more confused reports like this.

Any chance the analytics shows the average amount of time it takes to update the most frequently used/default taps?

Sadly it does not.

One more confused user.

Even if the non-update of casks is kept, maybe the auto-update message could point out what it hasn't updated. Something like this:
Updating Homebrew...
==> Auto-updated Homebrew!
Updated Homebrew from b38c52f9 to 30a2f270.
Updated 2 taps (homebrew/core, homebrew/science).
Note: Casks and external taps will only be updated with an explicit `brew update` command.


Work-In-Progress Fixes #1473 - Move TabletsMetaData builder to Ample

This was closed because it is superseded by #1651


fix: Give Atlantis write permission to EFS

Description
Add elasticfilesystem:ClientWrite permission to the EFS file system policy.
Motivation and Context

Currently, we give only the mounting permission, so Atlantis ECS container can mount/read EFS, but cannot write anything under /home/atlantis/* directory. Due to this problem, Atlantis container cannot start with enable_efs = true option.
This is the error message I saw in the Atlantis log.
Error: initializing server: unable to create dir "/home/atlantis/.atlantis/bin": mkdir /home/atlantis/.atlantis: read-only file system

The error was solved when I added the elasticfilesystem:ClientWrite permission.
The current workaround is to configure efs parameter as follows:
enable_efs = true
efs = {
  mount_targets = {
    "eu-west-1a" = {
      subnet_id = module.vpc.private_subnets[0]
    }
    ...
  }
  attach_policy = false # <= Here
}

Breaking Changes
N/A
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

This PR is included in version 4.0.4 :tada:


Update Package.json to remove security vulnerabilities

Here's the update. It looks like a ton of commits, however it's just changing the package.json file. I made 4 commits (4 package updates) and then rebased to make my comments a bit clearer and it doubled the commits going in.
This passes npm test.

@wilmoore can you squash these?

These dependencies have been updated and published as 2.0.0


Fix error when opening a tiled image - highest zoom level could not be determined

In our configuration while loading a tiled image the following error was thrown. This happen when OSD was loading the first tiled image.
global-error-handler.service.ts:15 RangeError: Invalid array length
    at __webpack_modules__.7446.$.TiledImage._updateLevelsForViewport (openseadragon.js:25539:1)
    at __webpack_modules__.7446.$.TiledImage.update (openseadragon.js:24507:1)
    at __webpack_modules__.7446.$.World.update (openseadragon.js:27064:1)
    at updateOnce (openseadragon.js:11711:1)
    at updateMulti (openseadragon.js:11655:1)
    at openseadragon.js:10687:1

Reason for that is that the highest level could not be determinded (its NaN) in _updateLevelsForViewport that is calculated in _getLevelsInterval.
this.source
{
    "_levelRects": {},
    "tilesUrl": "/api/service/curie/v1/tile/5fce4ecc5a7633115825923d/dzi_files/",
    "fileFormat": "png",
    "displayRects": [],
    "events": {},
    "_rejectedEventList": {},
    "width": 3916,
    "height": 3918,
    "tileOverlap": 0,
    "minLevel": 0,
    "maxLevel": 12,
    "Image": {
        "xmlns": "http://schemas.microsoft.com/deepzoom/2008",
        "Url": null,
        "Format": "png",
        "DisplayRect": null,
        "Overlap": 0,
        "TileSize": 256,
        "Size": {
            "Height": 3918,
            "Width": 3916
        }
    },
    "queryParams": "",
    "ajaxWithCredentials": false,
    "ready": true,
    "aspectRatio": 0.9994895354772844,
    "dimensions": {
        "x": 3916,
        "y": 3918
    },
    "_tileHeight": 256,
    "_tileWidth": 256
}

currentZeroRatio  -564.8212461695608
this.minPixelRatio  0.5

The change seems like a decent bit of defensive code, but before we go with it I'd like to know a little more about why this is happening. It looks like highestLevel is NaN because it's doing a Math.log of a negative number (currentZeroRatio). Looking at the code, it seems currentZeroRatio should never be negative. Could you please log the numbers that go into its calculation?
Also, with your fix, does everything seem to be working properly? Do you see any side effects?
My concern is something else is going wrong that's causing currentZeroRatio to be negative, and that we should fix the source of the problem.
Thank you for doing this!

Looking at the code, it seems currentZeroRatio should never be negative.

It looks like this could potentially happen if a negative number was given to one of setWidth/setHeight/setScale (or a negative width was passed in the options for this TiledImage). Perhaps it would be better to guard against negative numbers there. I suspect that allowing negative scaling could mess up the tile positioning and drawing code. setRotation(180) should give equivalent results to making the scale negative, I think?

i agree on your both comments.
i investigated a bit more. Turn out that we are using the OSD api maybe a bit wrong. After opening an image we want to have the image displayed in fit to size, thats like the user is pressing the home button. As far as i can remember we had in the past some issues that this was not always working. Therefore we did the following:
      this.imageViewer.viewport.goHome(true);
      this.imageViewer.viewport.zoomBy(0 - this.imageViewer.zoomPerScroll); // zoomPerScroll = 1.2

The result is at the end a negativ zoom level and therefore the highest level in NaN. I am currently on it to run all our automated tests and check manually to check whether we can remove the zoomBy call.
But maybe its also good to prevent the usage of zoomBy if the result is a negative value.
What do you think about it? Does it make sense? @iangilman @pearcetm

I agree that passing a negative number to zoomBy is the likely culprit here; instead of zoomBy(0 - this.imageViewer.zoomPerScroll) you probably want zoomBy(1 / this.imageViewer.zoomPerScroll).
Another question is why your image isn't filling the viewer after you call goHome(true). This could happen if the viewer is resized immediately after goHome - if all of this happens right at initialization or on image load, you might try putting that call in a timeout: setTimeout(()=>this.imageViewer.viewport.goHome(true), 0).

Yeah, the open call is asynchronous, so you need to wait until it's done before doing goHome. Typically I do viewer.addHandler('open' ... to take care of that.
And yeah, some guards against making the zoom negative (or even 0) in zoomBy and zoomTo (ideally including an error message) could be good.


Support role icons

Ref to #588
property for role icon is "icon" and it's value as data string image/png base64.
Emojis can be used to but the emojis needs to be translated into a base64 string too.
Idk why but Discord 🤷♀
As example see https://github.com/Aiko-IT-Systems/DisCatSharp/commit/0383f5aceef8604ca3a5df6e6df3feda2b03788f
It is not documented till yet. But will be soon.
Requirements for role icons:

Server Boost Tier 2

https://github.com/discord/discord-api-docs/pull/3815

See https://bugs.aitsys.dev/T31 for more details


Improve all inline tooltip documentation (doc comments), include exceptions info and all parameter info

As in the title. Esp. now that we've added all the proper null-checks in #127, we should clarify for each function what input would cause ArgumentNullException to be thrown. Tedious work, but it must be done ;). Keeping it up-to-date afterwards should be easier.

Finally, this is done. All exception information is now in the tooltips, and the limited ability that F# gives us for some syntax coloring in the tooltips is used for better recognizability.
Example:

The comments were actually from #183, which accidentally included these document updates as well.
Since this causes quite a bit of potential merge issues, esp if I go for static member next, I am going to move these in. However, ping me here if you still want to re-review this. Keep in mind that I've been seeking parity with seq in F# where applicable. A lot of wording is borrowed from there.

Please don't wait for me at any time - am fine with doing chaser reviews
Good point re aligning with Seq - if not already prominently mentioned, goot to have that high p the README.md
Though not having where is fine with me!


Implement Support dropdown

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #59

PR #65

PR #66

PR #67

PR #69   👈

This stack of pull requests is managed by Graphite.


Dev container fails to build

When trying to build the dev container through docker desktop on MacOS with M1, I get:
  • Installing ipython (7.34.0)
  • Installing jupyter-server (1.23.0)
  • Installing psutil (5.9.4): Failed

  RuntimeError

  Unable to find installation candidates for psutil (5.9.4)

  at /usr/local/lib/python3.10/site-packages/poetry/installation/chooser.py:103 in choose_for
       99│
      100│             links.append(link)
      101│
      102│         if not links:
    → 103│             raise RuntimeError(f"Unable to find installation candidates for {package}")
      104│
      105│         # Get the best link
      106│         chosen = max(links, key=lambda link: self._sort_key(package, link))
      107│

[18769 ms] postCreateCommand failed with exit code 1. Skipping any further user-provided commands.
Done. Press any key to close the terminal.

Not sure of cause of this since psutil installs fine with pip. Temp fix is to export to a requirements.txt and use pip :):
poetry export --without-hashes --format=requirements.txt > requirements.txt && pip install -r requirements.txt

Should be resolved now - psutil has some funky install issues so I've removed the requirement that needs it


Allow to create tarpc server on existing listener

When you want to use systemd.socket or systemfd for allocate the socket (TCP or UNIX) you need to be able to create a server on already existing listener

Thanks, and apologies for the review delay! Can you add unit tests for the new functions? Other than that, looks great.

Thanks, and apologies for the review delay! Can you add unit tests for the new functions? Other than that, looks great.

Hi, no problem.
I add some tests based on existing TCP and UDS tests.


Pointer returns a far-away gameobject to be Near when its scale is Zero

Describe the bug
When a gameobject has a scale of (0,0,0), the hand pointer returns it as a Near object even if the gameobject is far away.
Potential issue at NearInteractionTouchable.cs at line 214.
Vector3 localPoint = transform.InverseTransformPoint(samplePoint) - localCenter;
To reproduce

Create primitive Box (in MRTK ready scene)
attach NearInteractionTouchable.cs
scale the Box to 0,0,0
PokePointer will always return it as Near object when the Box is far away.

Expected behavior
Should not return as a Near object when the object is far away even when scale is zero.
Screenshots
N/A
Your Setup (please complete the following information)

Unity Version 2019.1.14f
MRTK Version 2.0.0 RC2.1

Target Platform (please complete the following information)

Editor
HoloLens 2

The issue has been resolved in the latest release MRTK 2.0.0.  If you see this on a future release please open again.

Checking if the poke pointer is near an object:
        foreach (var source in CoreServices.InputSystem.DetectedInputSources)
        {
            if (source.SourceType == InputSourceType.Hand)
            {
                foreach (var p in source.Pointers)
                {
                    if (p is PokePointer)
                    {
                        var poke = p as PokePointer;
                        UnityEngine.Debug.Log("Poke Pointer isNearObject: " + poke.IsNearObject);
                    }
                }
            }
        }


package naming issue

https://github.com/edgexfoundry/security-api-gateway/blob/a5840be551caacb95405538cc2871328a4d5b661/glide.yaml#L1
Shouldn't this be github.com/edgexfoundry/security-api-gateway?

Since we are moving out of glide and using go module currently this will not be an issue, hence close the issue.


Types of simple values expressions of structure constant fields aren't the finite types

Description:
The type of a BLangConstantValue for a simple value is not the finite type. For maps, we set the narrowed type (intersection of readonly and just that shape). For simple basic types, shouldn't we do the same?
Affected Versions:
2201.0.0

Same for the expression.
const map<int> M = {a: 1};

((BLangRecordLiteral.BLangRecordKeyValueField) ((BLangRecordLiteral) constant.expr).fields.get(0)).type is the finite type but ((BLangRecordLiteral.BLangRecordKeyValueField) ((BLangRecordLiteral) constant.expr).fields.get(0)).valueExpr.type is int.