added 'assemble-docs' makefile target

new target is to simplify build server configuration to deploy API docs to https://developer.xamarin.com/api/namespace/SkiaSharp/

If you're curious, yes I saw that update-docs also assembles the deployment artifacts ... but the reason I wanted a separate target that only assembles, is because mdoc update will modify the XML files if the resulting assembly is different ... and thus, these changes would need to be checked in. I didn't want the build server having to deal with that ... update-docs should be run separately, by a user that will check in the XML changes :)

@migueldeicaza when you get a chance ... this is for the teamcity build :)


fix: Fixes de pkg_resources usage with modern importlib

This PR fixes an issue with the pkg_resources package missing that was causing installation errors.
The PR updates the django-cid package to use the importlib.metadata on Python3.8+ or the backported importlib_metadata module for older versions. This ensures compatibility with future Python releases.

Closing in favor of https://github.com/loggi/django-cid/pull/1


Add documentation on Univeral date format

Before you start...

[X] Have you refreshed the page? You might be using an old version of the website.
[X] Have you checked if someone already requested this feature? If they have, post a comment with your use case instead.

Description
Add documentation on the Universal date format
It would be nice to add a section for the newest date format (Universal) to the Files documentation, which currently only mentions Application & System.
Source Path: Website/src/routes/docs/configuring/date-formats.md
Public URL: Files Documentation
Alternative solutions
n/a
Relevant Assets

Fixed by #249; closing this issue now.


mqtt-widget-commands

After some helpful chatting on the Tasmota Discord I finally got this to work like I wanted:
To draw widgets:
NSPSetWidget {"type":"","name":"","position":,"always_online":}
"type":%s			typelist
"name":%s	        	max 8 char
"position":%d		1-8
"always_online":%b	bool (optional, will be set false)
Types:
single horizontal
double horizontal
triple horizontal
quad horizontal
single vertical
dual vertical
triple vertical
quad vertical
rgb strip
cct bulb
rgb+cct bulb
scene
To delete:
NSPSetWidget {"delete":}
"delete":%d			1-8
Examples:
NSPSetWidget {"type":"scene","name":"Scene 1","position":3}
NSPSetWidget {"type":"rgb strip","name":"Strip 1","position":5,"always_online":true}
NSPSetWidget {"delete":3}
Set the strip to "ON":
WIDGET {"relation":{'id': '5', 'params': {'switch': 'on'}}}

you have removed the ability to set widgets in the driver code and now it is required to use a command which is impractical


APP-633: Returning HTTP 415 when the payload content-type is not supported

This integration now provides a list of acceptable payload types. In this case, no one has been provided, that's why it's using MediaType.WILDCARD_TYPE. It could also return an empty/null List, same behavior.

Can one of the admins verify this patch?


SH scraper needs update to detect new data type

SH now also reports ncumul_ICU, could the scraper be adjusted to pick up those numbers as well?
Also, data for ncuml_conf of 2-4 was updated on 3-4, scraper did not detect this.

On it.


Switch buffers to use CMSampleBuffer types

Instead of custom classes, try using the CMSampleBuffer types that AVFoundation uses. This should allow use of some system classes for things like reusing buffers, and make it easier to feed data between OgvKit and other stuff like the camera interface or H.264 decoding/encoding.

Note can probably use CVOpenGLTextureCacheCreateTextureFromImage to do YCbCr conversion from these objects instead of the custom shader.

Ok CVOpenGLESTextureCacheCreateTextureFromImage will do texture uploading but only per plane, still have to combine color elements in the shader if I understand correctly.
https://developer.apple.com/reference/corevideo/1619860-cvopenglestexturecachecreatetext?language=objc

Could I think do final output via AVSampleBufferDisplayLayer though

'layer' branch tests AVSampleBufferDisplayLayer. Notes:

AVSampleBufferDisplayLayer is not present on tvOS, in case want to support that later may still need a GL-backed layer.
on iOS device, must create pixel buffers attached to i/o surface -- this means they're going to be buffers allocated in device memory that we have to copy into.
on iOS device, can't use planar Y'CbCr -- have to use biplanar mode, which means merging Cb and Cr planes into an interleaved chroma plane.
Currently it looks slower to do the copies on CPU.
Currently creating a full copy in decoder land and then copying again into the io surface. should just do it into io surface in the decoder, eliminating the overhead of the current buffers.
Should probably use a CVPixelBufferPool to dispense buffers to work with!
not yet sure the extra pixels options do what i expect
there's some weird problem on first show at 360p, where the data seems offset from expected location.

Copies are down to sub millisecond for 1080p on iPad Pro, good enough vs texture uploads so far. Still seeing the offset problem with initial run. Also noticed that after switch away from app and back, the layer is cleared. Need to save last frame to redraw it?

Filed the 360p corruption bug as rdar://29810344 - http://www.openradar.me/29810344

I also can't get 4:4:4 buffers to output on-device in any pixel format variant. Not an issue for VP8 but potentially a problem for Theora and VP9 specialty files.
Reworking layer branch a bit to do the conversion in delegate space, so can more easily switch.
Another disadvantage is that the system prefers packed formats while the theora and vpx libraries want planar, except in a few places where you can get away with planar for 4:2:0 only. This means copying has to be done to convert to a CVPixelBuffer even if we're going to send it back for reencoding on our end. Copying doesn't seem as problematic for playback, as have to copy to gpu memory anyway... the interleaving seems pretty efficient in neon land.

4:4:4 and 4:4:4:4 (with alpha) packed formats can be created as CVPixelBuffers but the CVOpenGLESTextureCache won't upload them to GPU. Can fake with by making a BGRA buffer and treating it as 4:4:4:4 but this loses interoperability in theory.
I'm inclined to keep the custom buffers around and just make it easy to convert, with the default playback rendering done via OpenGL. Going to see if can upload each plane via CVPixelBuffer as 8-but grayscale and make luminance textures with CVOpenGLESTextureCache... otherwise will go to old fashioned texture uploads, but more carefully managing lifetimes and keeping it off the main thread.

Also looks like the 4:2:2 pixel-unpacking texture extension is only available on 64-bit. Yeah, sticking with my custom planar bits for now. :)

7e2af3ea514a3d07b8baef6186873a0cd74c040e has the final version of layer branch with its other cleanup fixes, with the AVSampleBufferDisplayLayer-using code #ifdef'd out. Still keeping the conversion code without #ifdef since will want it for transcode support.

Bringing it back! The iPad Pro corruption issue is fixed in iOS 11, and 4:4:4:4 alpha-y-cb-cr sample buffers work on 64-bit. Note that OpenGL is deprecated on upcoming iOS 12, so it's either look into switching to metal for future support and wide color gamut, or using AVSampleBufferDisplayLayer.
I think doing a downsample 4:2:2/4:4:4 on iOS 10 and below would be fine at this point, or just documenting those won't work since they're rare in practice.

cleanup currently lives on samplebuffer branch, needs a couple more fixes

Hmm.... could also look at AVSampleBufferAudioRenderer to replace OGVAudioFeeder, but it's iOS 11+.


Should docs/_templates/doc_version.html be a jinja template?

Something looks broken, not sure what. See, e.g., https://scipp.github.io/esssans/.

The link to older versions does not work.
But can you be more precise about what looks wrong?

I think the orgname is missing? My guess was that since the jinja template is missing copier does not render the template. Instead it is rendered by Sphinx, which does not have all the variables (but happens to have the project name)?


Using jar "spark-xml_2.11-0.4.1" with Python Error - org.apache.spark.sql.types.DecimalType$.Unlimited(

I am using the latest jar code with python. I try to parse the sample xml books.xml but i still run into the error:
: java.lang.NoSuchMethodError: org.apache.spark.sql.types.DecimalType$.Unlimited()Lorg/apache/spark/
sql/types/DecimalType;
at com.databricks.spark.xml.util.InferSchema$.(InferSchema.scala:50)
at com.databricks.spark.xml.util.InferSchema$.(InferSchema.scala)
at com.databricks.spark.xml.XmlRelation$$anonfun$1.apply(XmlRelation.scala:46)
at com.databricks.spark.xml.XmlRelation$$anonfun$1.apply(XmlRelation.scala:46)
at scala.Option.getOrElse(Option.scala:121)
at com.databricks.spark.xml.XmlRelation.(XmlRelation.scala:45)
at com.databricks.spark.xml.DefaultSource.createRelation(DefaultSource.scala:66)
at com.databricks.spark.xml.DefaultSource.createRelation(DefaultSource.scala:44)
at org.apache.spark.sql.execution.datasources.DataSource.resolveRelation(DataSource.scala:33
0)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:149)
at org.apache.spark.sql.DataFrameReader.load(DataFrameReader.scala:132)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:237)
at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:357)
at py4j.Gateway.invoke(Gateway.java:280)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:214)
at java.lang.Thread.run(Unknown Source)
Can you please help!

It looks you are using Spark lower than 1.6.x? In that case, please try 0.3.4. Please refer https://github.com/databricks/spark-xml#requirements

Thank you for the response, i am using spark 2.0.1 (2x).

Then, it seems lower verion of this library is being referred because in 0.4.1, Unlimited is not being used in the line - https://github.com/databricks/spark-xml/blob/v0.4.1/src/main/scala/com/databricks/spark/xml/util/InferSchema.scala#L50
But, this is being used in 0.3.x - https://github.com/databricks/spark-xml/blob/v0.3.5/src/main/scala/com/databricks/spark/xml/util/InferSchema.scala#L50

I just downloaded the jar source spark-xml_2.11-0.4.1 and then manually checked the codes. It seems old jar is in the path.

Thanks again for the super quick response. Will you be able to checkin the new code. I will download it and rebuild it again.

Apologize for my earlier response, i did have a bad reference to an old jar 0.3. Thanks again!

No problem :). Do you mind if I ask to close this issue if it is fine?


feat: Template improvements

Various small improvements for the template.
This also aligns the template's behavior with the documentation,
where, under "Running the PSC during development", it states:
Run npm start
This runs tc-cli start. You can, instead, also run that
command directly, allowing you to specify further options.
With this commit, the template gets aligned to that behavior.
I will, simaltaniously, add the changes to the already existing D2 PSC Repo.

As this PR includes both a small fix and a feature, I suggest to either rebase it onto main (the history is linear, as of now), or merge it squashed as the title's feat(template): Template Improvements :wink:

As this PR includes both a small fix and a feature, I suggest to either rebase it onto main (the history is linear, as of now), or merge it squashed as the title's feat(template): Template Improvements :wink:


#989- Make search case insensitive

Currently search is case sensitive
this PR allows the filtering to be caseinsenstive

Thanks!


Modernize struct initialization

We should update the struct initializations to match those in #74

Done in #217


Schools do not have opening hours

When browsing amenity=school is highlighted as missing opening_hours
At least in my region this is not taggable as schools do not have opening hours.

It's okay to have some fields missing: the editor is not a validator, it merely shows tags that you might — or might not — consider important for an amenity. Like, most shops around here don't have websites or phones, which are highlighted on POI tiles, and that doesn't mean I would go out of my way to collect them.
(And also some schools do have opening hours, mostly private or limited).


toHaveReceivedAnyCommand() missing

The 'sister' lib 'aws-sdk-client-mock-jest' has a Matcher 'toHaveReceivedAnyCommand()' which can be really useful, especially in its negated form to make sure certain code branches are not executed at all. Is it possible to add this?

exactly! it will be good to have. Is there any alternative for workaround now?

This is now released as of v5.0.0
Thank you for your comments and feedback on the module. Glad you find this helpful when developing.
I looked into this and my implementation should be compatible with aws-sdk-client-mock-jest. In case you are testing the negative (you assume no call has been made), I also list the unexpected calls when the assumption fails.
Sample usage:
it('received a command', () => {
  const s3Mock = mockClient(S3Client);
  [..]
  expect(s3Mock).toHaveReceivedAnyCommand();
});

If any command was sent to the mock, this test passes. Otherwise it fails:
Error: expected client "S3Client" to have been called, but was not called

You can also ensure no call had been made:
Sample usage:
it('did not receive a command', () => {
  const s3Mock = mockClient(S3Client);
  [..]
  expect(s3Mock).not.toHaveReceivedAnyCommand();
});

If no command was sent to the mock, this test passes. Otherwise it fails, listing all received commands and their inputs:
Error: expected client "S3Client" to not receive any calls, but was called

Received:

   1st S3Client with GetObjectCommand call

    Object {
      "Bucket": "foo",
      "Key": "file1.txt",
    }

   2nd S3Client with GetObjectCommand call

    Object {
      "Bucket": "foo",
      "Key": "file2.txt",
    }

   3rd S3Client with GetObjectAclCommand call

    Object {
      "Bucket": "foo",
      "Key": "file3.txt",
    }

Number of calls: 3

Working well, thanks!


vulnerability advisor - add (sensible) remaining fields when user clicks on description

And disable sensitivity to site-level global refresh.
Users will get refreshed data if they choose to use pagination or search.

add additional result fields (not uuid / create, perhaps more to exclude tbd...)
{
            "affects": "Keystone: versions through 2014.1.4, and 2014.2 versions through 2014.2.3",
            "created": "2016-01-30T01:44:05.649447Z",
            "cve": "CVE-2015-3646",
            "description": "Eric Brown from VMware reported a vulnerability in Keystone. The\nbackend_argument configuration option content is being logged, and it\nmay contain sensitive information for specific backends (like a\npassword for MongoDB). An attacker with read access to Keystone logs\nmay therefore obtain sensitive data about certain backends. All\nKeystone setups are potentially impacted.",
            "link": "https://security.openstack.org/ossa/OSSA-2015-008.html",
            "name": "OSSA-2015-008: Potential Keystone cache backend password leak in log ",
            "notes": "This fix will be included in future 2014.1.5 (icehouse) and 2014.2.4 (juno)\nreleases.\nThe 2015.1.0 (kilo) release is not affected.",
            "notice_date": "2015-05-04",
            "ossa_id": "OSSA-2015-008",
            "patches": "https://review.openstack.org/175519 (Icehouse)\nhttps://review.openstack.org/173116 (Juno)",
            "processed": false,
            "references": "https://launchpad.net/bugs/1443598\nhttp://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2015-3646",
            "updated": "2016-01-30T01:44:05.649463Z",
            "uuid": "8ec11b78-9985-41af-ad8e-d87c2eb5d086"
        }

in process with #299

some alternate icon inspiration:
https://www.iconfinder.com/search/?q=expand
https://www.iconfinder.com/search/?q=collapse
https://www.google.com/search?q=collapse+expand+icon


Driver integration module [wip - Do not merge]

Since the behavior of the driver may be altered based on libraries that are present we need a mechanism for testing with these configurations in an automated fashion. The 'osgi' example provides most of this functionality as it loads up OSGi containers with different library configurations, but this is not adequate in some cases, particularly the 'shaded' case where some classes use a different packaging namespace.
This is a first attempt and is far from perfect, but wanted to get this opened up to get some feedback with regards to whether i'm on the right path here.
Some concerns/questions:

Is this the right place for something like this?  Does it need to be in the java-driver repository proper, or should it be a separate project?
Is the module name 'driver-integration' appropriate?  Can you think of a better name?
Can you think of a better way of testing different library configurations other than creating a separate module for each config?
The integration-shaded module uses the netty shaded package structure which is problematic at least in IntelliJ.  I think this may be annoying for those who like to keep clean environments.
Another thought I had was that maybe we could customize our existing tests in driver-core to have a base Cluster.Builder that can be configured on a per-module basis, that way we can run all tests under these different configurations.  That may be too exhaustive, but may uncover some unknown issues.

The timeline for review on this is not urgent, this can be something we look at post release

Deleting this PR since it references one that has since been merged.


Repeatedly recalculate states

What's wrong
Fixed #67
How was it fixed?

Added a get_recalculated_states function for easily testing.
Now there's only one test case that Danny mentioned.

Looks good to me after @jannikluhn's proposed changes


API break in 0.5.6

error[E0277]: the trait bound `assert_cli::output::Content: std::convert::From<std::string::String>` is not satisfied
  --> tests/cli.rs:40:19
   |
40 |         .stdout().is(load_file("tests/images/test1-out.svg"))
   |                   ^^ the trait `std::convert::From<std::string::String>` is not implemented for `assert_cli::output::Content`
   |
   = help: the following implementations were found:
             <assert_cli::output::Content as std::convert::From<&'a [u8]>>
             <assert_cli::output::Content as std::convert::From<&'a str>>
   = note: required because of the requirements on the impl of `std::convert::Into<assert_cli::output::Content>` for `std::string::String`

assert_cli version: 0.5.6
Rust version: 1.26.0
OS and version: Arch Linux

Sorry about that.  I've yanked 0.5.6, am going to release a 0.6.  I might also see about fixing your case.


param set default battery parameters

@DonLakeFlyer With this PR in you will be able to build your calculator UI.


Same SSID not working

When I enter the same SSID on multiple lines it shows only one WIFI SSID

That is correct. The SSID's have to be different

Can I somehow edit the code to show multiple networks with the same SSID?

Nope, it's fundamentally not possible

Ok, Thanks


BAEL-2444 Sample code

I used the 2.0.0-RC.4 JMockit version because its the only version that works with lastest Mockito version.


Consider switching to destroyPlatform for unmounting?

See https://github.com/me-12/single-spa-angular2/commit/f03cc7cc26d1f04e10db6f60453bc652504cf8b3#commitcomment-38483962 which suggests using destroyPlatform for unmounting angular applications.
I've expressed doubt that this would work when multiple Angular applications share the same instance of Angular, and therefore share a platform. However, if I'm wrong about that, this might be a good alternative to our workarounds for Angular Router bugs that are shown below:
https://github.com/single-spa/single-spa-angular/blob/6a7d48741d54e2a451987da17875bf3da9f365f4/src/src/single-spa-angular.ts#L128-L130
@kristofdegrave originally suggested this. Let's verify whether destroying the platform applies only to a single Angular app versus all mounted Angular apps.

@joeldenning I can confirm you can have multiple platforms running together. Currently I'm working on a MicroFrontend solution myself. As Macro Frontend/ shell I'm also using angular, this eases up some handling of authentication. As Micro Frontends I have a combination Angular and AngularJS applications. For this I work with angular elements. So mounting an application for me is loading the scripts and styles and adding the custom element.
To decrease the memory consumption, I was looking for some ways to destroy/dispose my client.  I do this by raising an event and destroying the Micro Frontend when the unmount event is raised. This code is in my case added in the main.ts file of the Micro Frontend, same place where I bootstrap the application using platformBrowserDynamic().bootstrapModule().
The only issue I currently have (and this is due the use of angular elements) is when I go back in history and my Micro Frontend was destroyed. I get an error that the injector was destroyed.
Since you have another setup here, this shouldn't be the case.

There are still some issues:

you still have to get animation engine and call flush() manually because destroyPlatform is a sugar over ngModuleRefs.forEach(ngModuleRef => ngModuleRef.destroy()), where it keeps references to all bootstrapped modules in the current platform
I remember router.dispose() was added because ngModuleRef.destroy() was not enough and in some situations the non-deterministic behavior of the router was manifested

@arturovt, In my case I'm using the v9 versionn of angular, and in this case I can confirm the dispose() is called from the onDestroy() methode in the router. I checked this by adding debugger to verify they got called. I'll try to check if this is also the case when I just destroy the ÀppModule

I also checked with 9 Angular and dispose is called, but we're talking about previous versions too.

What versions of Angular does this work for? We're about to release single-spa-angular@4, which will only work in angular 9 (and maybe 8). This approach is less hacky than what we currently have - should we switch to it in single-spa-angular@4?

After discussing this with Artur, I am closing this. The reason is that destroying the entire Angular platform would disallow sharing a platform between microfrontends. Sharing a platform is important for performance, since that means that @angular/core, @angular/common, etc only have to be downloaded once in the browser, instead of once per microfrontend.


Allude to 'this' keyword as suggested by @ParkinT

@ParkinT suggested the this keyword be alluded to in the "Construct JavaScript Objects with Functions" lesson.
Closes #3899

Thanks for the PR! :smile:


today onwards herocard attachments or image attachments are not working, actually it was working before

@swathish : I feel your pain, I wouldn't be at all surprised if your issue was related to the one I just raised at #2517

I can confirm that using a HeroCard attachment in SMS is currently broken but was not as recently as yesterday.

I am  using HeroCard attachment with telegram . Yesterday it was working..
On 30 March 2017 at 00:01, Mitch DeShields<EMAIL_ADDRESS>wrote:

I can confirm that using a HeroCard attachment in SMS is currently broken
but was not as recently as yesterday.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/2518#issuecomment-290239875,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEEFahIDFe7JLFwQ5SLs1LjThRgoW36Oks5rqtSkgaJpZM4MtPpH
.

This was my fault.  We have a new feature for rendering complex cards as an image and it got turned on before it was supposed to.  The problem is that because we are sending an image it requires your twilio to be configured for MMS, which is why it failed to deliver.  I am updated our servers as we speak.  Thank you for your patience

Thank you.. It is working now...
Is it possible to add bot in skype for business?


Enhance errors description

I write and execute a query. A message appears telling "Error Occurred".
That's not very helpful to me: what's the actual error so that I can correct the query?
Thank you!

sure, we can work on it in the new release.


Index signature key type (number vs string) does not affect assignability of objects

TypeScript Version:  2.7.2
Search Terms:  index signature key type assignable
Code
type Dictionary<T> = { [key: string]: T };
type NumericDictionary<T> = { [key: number]: T };

declare function omit<T>(obj: Dictionary<T>, propName: string): Dictionary<T>;
declare function omit<T>(obj: NumericDictionary<T>, propName: string): NumericDictionary<T>;

declare let dict: Dictionary<number>;
declare let numericDict: NumericDictionary<number>;

// Index key type doesn't affect assignability?
// These assignments are valid, but I would expect them both to be errors.
dict = numericDict;
numericDict = dict;

const result1 = omit(dict, "blah");
// Type of "result2" is Dictionary<{}> instead of NumericDictionary<number>!
const result2 = omit(numericDict, "blah");

Expected behavior:
I would expect the key type of an index signature (string vs number) to affect the assignability of objects. I.e., an object with the signature [key: number]: string should not be assignable to an object with the signature [key: string]: string, and vice versa.
This assignability restriction would allow for function overloads to be selected based on the type of the index signature key.
This was discovered while attempting to improve DefinitelyType type definitions for lodash to better support NumericDictionary. It is impossible to define overloads for some methods, such as _.omit and _.mapValues, if Dictionary and NumericDictionary are assignable to each other.
Actual behavior:
See comments in code example.
Playground Link:
https://www.typescriptlang.org/play/#src=type Dictionary<T> %3D { [key%3A string]%3A T }%3B
type NumericDictionary<T> %3D { [key%3A number]%3A T }%3B

declare function omit<T>(obj%3A Dictionary<T>%2C propName%3A string)%3A Dictionary<T>%3B
declare function omit<T>(obj%3A NumericDictionary<T>%2C propName%3A string)%3A NumericDictionary<T>%3B

declare let dict%3A Dictionary<number>%3B
declare let numericDict%3A NumericDictionary<number>%3B

%2F%2F index key type doesn't affect assignability%3F
dict %3D numericDict%3B
numericDict %3D dict%3B

const result1 %3D omit(dict%2C "blah")%3B
%2F%2F Type of "result2" is Dictionary<{}> instead of NumericDictionary<number>!
const result2 %3D omit(numericDict%2C "blah")%3B

Related Issues:
https://github.com/DefinitelyTyped/DefinitelyTyped/issues/23723

I think that { [key: string]: T } means :
Each string key must be of type T
Similarly,  { [key: number]: T } means :
Each number key be must be of type T
These two interfaces are therefore not mutually exclusive at all.
For example, all these objects satisfy both  Dictionary<string> and    NumericDictionary<string>:
const x1 = { };
const x2 = { "foo": "bar" };
const x3 = { 4: "bar" };
const x4 = { 4: "bar", "foo": "bar" };

This is the intended behavior; more reasoning other than "This is not what I expected" would be required for us to provide a concise explanation of why your expectation is not met.
You just need to swap the ordering of the overloads so the more-specific type gets matched first.

@UselessPickles what happens if you put the NumericDictionary overload before the Dictionary overload? Does it match Dictionary inputs or not?

@RyanCavanaugh @aj-r
Swapping the order just reverses the problem:
declare function omit<T>(obj: NumericDictionary<T>, propName: string): NumericDictionary<T>;
declare function omit<T>(obj: Dictionary<T>, propName: string): Dictionary<T>;
...
// Type of "result1" is NumericDictionary<{}> instead of Dictionary<number>!
const result1 = omit(dict, "blah");
const result2 = omit(numericDict, "blah");

Also noteworthy that the incorrectly typed variable is NumericDictionary<{}> of Dictionary<{}>. The T part was lost in the type.

@s-ve I can see how the assignability may be technically correct, however unintuitive in some contexts. Maybe I should have focused more on the function overload signature issue I'm experiencing in framing the issue description?
So what it appears to be is that Dictionary<T> implicitly has a numeric index signature: { [key: number]: {}, which is why I can pass Dictionary<T> to a function overload that accepts NumericDictionary<T>, and T is inferred as {}.
Maybe I could somehow form an index signature that says "accept a param of type Dictionary<T> where T is NOT {}"?
Should the implicit index signature have a value type of never instead?

You could write
declare function omit<T extends Dictionary<{}> | NumericDictionary<{}>>(obj: T, propName: string): T;

@RyanCavanaugh Thanks!
The next challenge will be _.mapValues, where I need method signatures that accept either a Dictionary<T> or NumericDictionary<T>, and return either a Dictionary<U> or NumericDictionary<U> respectively, where U` is some other type (like the result type of an iterator, etc).
I tried something like this, but it didn't work:
declare function mapValues<T extends {} | null | undefined, TResult>(obj: Dictionary<T>, blah: TResult): Dictionary<TResult>;
declare function mapValues<T extends {} | null | undefined, TResult>(obj: NumericDictionary<T>, blah: TResult): NumericDictionary<TResult>;

I think my attempt at the mapValues signature would work if the the implicit value type of an index signature was never instead of {}? Why is it {}?

I think the root of the problem here is that keyof T doesn't work for numeric indexes. We've tried this:
mapValues<T extends object, TResult>(obj: T | null | undefined, callback: ObjectIterator<T, TResult>): { [P in keyof T]: TResult };

which works great for string indexes, but doesn't work for numeric ones. It appears that keyof NumericDictionary<any> is never, which seems wrong to me. I would have expected it to be either number or string.
Ultimately, this is the problem we're trying to solve with mapValues: given an object, return a new object with the same keys, but different values. It seems that typescript gives us no way to do this in a way that supports both string and numeric keys. @RyanCavanaugh any ideas?

@RyanCavanaugh FYI - the final solution for _.omit ended up being quite unwieldy. We need to support Dictionary<T> and NumericDictionary<T> where T may include a union with null or undefined. The solution you suggested only works if T does NOT include anull or undefined.
Here's what I ended up with:
    type AnyDictionary =
        | Dictionary<{}>
        | Dictionary<{} | null>
        | Dictionary<{} | undefined>
        | Dictionary<{} | null | undefined>
        | NumericDictionary<{}>
        | NumericDictionary<{} | null>
        | NumericDictionary<{} | undefined>
        | NumericDictionary<{} | null | undefined>

declare function omit<T extends AnyDictionary>(obj: T, propName: string): T;

Is there any better way to do this?

@UselessPickles why not Dictionary <any> | NumericDictionary <any>

@aj-r Because derp. That seems to work.

@aj-r It turns out that Dictionary<any> | NumericDictionary<any> does not actually work as desired. Other arbitrary object types WITHOUT index signatures end up using the Dictionary<any> | NumericDictionary<any> signature. If I use my verbose AnyDictionary typedef, then only types with index signatures use the method signature intended for dictionaries.


Package- / Folder-structure deeper than one level: Content ignored on import (of zip)

Imported the attached zip file.
Expected to find a package ZP2PH_EXPERT_FINDER_UI within ZP2PH_EXPERT_FINDER (such as in the original system). Folder structure in ZIP looks fine, but on import, it looks as follows:)

no third level
third-level-to-be-package empty

[ZP2P_20161213_141403_without_view_maintenance_fugr.zip](https://github.com/larshp/abapGit/files/649791/ZP2P_20161213_141403_without_view_maintenance_fugr.zip

yes, I also expect it to create a third level in this case

issue reproduced in https://github.com/larshp/FUGR2

retested successfully on separate branch :+1:

ok, will merge the PR then 😄


Allow plugins to override core compilers

External plugins that use registerCompiler are loaded before the internal modules for Solidity and Vyper.
When the internal modules are loaded, they override the external plugin compiler.
This change will allow the external plugin to have preference over the internal modules for compilers.

Sort plugins so that the first ones are always internal plugins and last ones are always user plugins, so that way, we override with the user's plugin


Add spec for Enumerator::Lazy#chunk

Mostly copied and slightly altered the existing specs that seemed most similar. Will do #chunk_while and company along the same lines unless this is wrong.

It looks like the existing specs for Enumerator::Lazy all test for almost exactly the same things. Since I will otherwise shortly be adding four more files of that, I'm thinking of pulling out the shared tests into a shared examples group and using them with should_behave_like.

When possible, shared specs are indeed better so there is less duplication and examples added for one method also means added to others methods.
Could you try to make and use a shared spec for those?
The chunk_spec.rb files can then define a few @ivar like @yieldsmixed in the before :each block with the specific receiver class/method, and those can be used in the shared spec.
Or alternatively, they could just define a "factory" for creating a Enumerable/Enumerator::Lazy with the given elements, and the shared spec can use that.

@joshgelbard Did you have the occasion to look at sharing those specs?
If not, I'd rather merge this spec and then we can improve based on what's in master.

Let's merge this, thank you for the new spec!


Support for Byond

One of the most popular GitHub repos is a game for Byond. I think this should make it a good candidate for being recognized by Sourcerer. The repo was featured on the GitHub Octoverse for having a ton of contributers.
Byond: http://www.byond.com/
Game: https://github.com/tgstation/tgstation

It looks like this is no longer the case, but waybackmachine has my back on the fact check: https://web.archive.org/web/20180531200936/https://octoverse.github.com/

@CorruptComputer, thanks for the suggestion. Our bandwidth is limited at the moment, but if you are interested in adding DM language support to sourcerer-app, we would be happy to provide guidance.

I would be happy to do it, and I already looked at your Contributing guide. This is what I have:
[
  {"name":"Byond Mapping","id":"byond.mapping","tech":["app-data"],"tags":["mapping"],"repo":"","imports":[""],"examples":"https://github.com/tgstation/tgstation","status":"awaiting-model"},
  {"name":"Byond Sprites","id":"byond.sprites","tech":["graphics-gaming"],"tags":["2d-graphics"],"repo":"","imports":[""],"examples":"https://github.com/tgstation/tgstation","status":"awaiting-model"},
  {"name":"Byond DM Code","id":"byond.dm","tech":["app-arch"],"tags":["oop"],"repo":"","imports":[""],"examples":"https://github.com/tgstation/tgstation","status":"awaiting-model"},
  {"name":"Byond Sounds","id":"byond.sounds","tech":["audio"],"tags":["audio-game"],"repo":"","imports":[""],"examples":"https://github.com/tgstation/tgstation","status":"awaiting-model"}
]

But I'm not quite sure what I have to do in order for Sourcerer to recognize these contributions. The easiest way would be file type: .dmm .dmi .dm .ogg, respectively from top to bottom for the above.

@yaronskaya could you provide guidance for adding DM language support to sourcerer-app?


fix(docs): update the download link of NotoSansMonoCJKsc-VF.ttf

The previous link was for NotoSansCJKsc-VF.ttf.

Thanks!


[WIP] Add workflow to run new and modified benchmarks

Adds a GitHub workflow that (should) run any new or modified benchmarks against the following file: ./data/waveforms/mimic_iv/waves/p100/p10079700/85594648/85594648

@tompollard , thanks for adding this. I will help test it as I update the xml_custom.py format.
If the format / code isn't efficient on the read test this could take a very long time with the 85594648 file. We may want to consider using a shorter file for the purpose of this automated run. I've been using /waves/p169/p16955095/82284982/82284982 (from the MIMIC-IV waveform database) as a shorter file.

Sounds good, thanks. There are some changes that I need to make to the workflow anyway, so I can swap out the file at the same time (though I think not such a bad thing that it takes a little while...I actually wanted to run the test across a much broader set of files once I had it working, but we can discuss!).


How to add an attribute to config?

For example, I am using a config file of faster R-CNN and I want to add another bbox_assigner with different IoU threshold for a special task (means I need 2 assigner, maintain the old one and add a new one).
I tried to add
train_cfg = dict( ...., NEW_assigner=dict(              type='MaxIoUAssigner',             pos_iou_thr=0.5,             neg_iou_thr=0.2,             min_pos_iou=0.2,             match_low_quality=True,             ignore_iof_thr=-1),....)
Into config file, and use
self.bbox_assigner_NEW = build_assigner(self.train_cfg.NEW_assigner)
In standard_roi_dead.py.
But I got a
AttributeError: 'ConfigDict' object has no attribute 'NEW_assigner'.
I guess there is a default config which indicates the names of the attributions?
So my question is how to add an attribution? Here is an example, I am trying to understand how mmdetection works, and the answer of this question will be really helpful. Thank you a lot.

Hi, @UT-Mao  Can you provide a complete config？ and I suggest you use the Error report issue template.

Sorry for bothering.. I think I add the content of config into wrong place..
I will close this, thank you for answering.


Fix incorrect output detection under the presence of filtered inputs

Fixes part of #20391 by invalidating the output locations just before we snapshot the outputs.

@bot-gradle test and merge

OK, I've already triggered a build for you.


Change token length + index

255 should be enough for most cases for encrypted token.

Hey @kevmt!
There's some conflicts, could you rebase your branch?


License

Greetings. Do you mind tossing a LICENSE.md file in the root or adding a license to the package.json in the web directory, also?

I Added License.md file in the root. Thanks.


moon v1 lifetime

Congratulations for releasing moon v1 🎉
I see a lot of change, obviously my current moon project is obsolete now.
I am firm believer of moon, but the case of many breaking change like before make me think. What if my moon project will be unsupported by next month version of moon. Can I base my work on moon v1 ?

The one that hook me to moon is:
"10kb ui framework that taste like vue", I know that is not your promise, but that is the one that hook me.
i'm not seeing that in moon v1

This is a complete rewrite, and I doubt that @kbrsh will rewrite it again next month.  Right now it's a beta version so I don't think you should use it in production yet.
As for what hook you to moon:
"Small file size (3kb minified + gzip)" is on the moon's Github summary page.

So what is the life expectancy of moon v1 ? a month, three month ?

So what is the life expectancy of moon v1 ? a month, three month ?

It needed a complete rewrite because some bugs were hard to fix.   I believe this is a solid foundation of moon.
This is a beta version so life expectancy of the beta depends if more people would contribute to the beta testing.  As I said, there will not be a complete rewrite of moon each month.

Yes, i know what beta is... but when v1 is released, I think we have to set some guarantee that it will not change in X month.

@terrybr Is right — it took over a year of development to rewrite Moon into v1. This time, I started from scratch with a simple philosophy and developed Moon from there. You can expect there to be little to no breaking changes in the foreseeable future.
Also, your current project isn't obsolete — that changes aren't super drastic but just make the API cleaner and more consistent — it should be doable with some effort.


Take care of i18n compilation on build time and write it at release time

@sneridagh Benefit of razzle.config, all projects will automatically get this behavior, right?

I'm late for review but great job! 🎉
This is really nice for our DX


Added Solution for NQueen Problem.

_ No description provided. _

Also update your name in README.md file.


MNT: Re-rendered with conda-smithy 2.3.1 [ci skip]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


freeCodeCamp Bossier City

Please provide the following information to aid in adding your group to the directory:
Esssential

[https://www.facebook.com/groups/381135042380814/ ] Group URL (Web address)
[ Bossier City] Town or City group is located in
[ Louisiana/Bossier Parish/Northern Louisiana] State / County / Region
[ United States] Country

Nice to have

[ ] Co-ordinates

If you have one

[ ] URL of Group Photo

Is there anything not filled out properly?
If there's anything I can do, please let me know.


red line in Firefox

If you zoom an image of a group in Firefox and you click the back / next button then after clicking the clicked button appears with small red line in the middle.
It's only in Firefox not in Chrome.
http://abload.de/img/lightboxerror2caq46.jpg
http://abload.de/img/lightboxerror155pym.jpg
Used with bootstrap v3.3.6 (CSS) and http://daneden.me/animate

https://github.com/ashleydw/lightbox/issues/134


Automate identifying, labeling, and closing stale issues

Outlining some of our ideas:

Inbox:

[x] Identify area (was: bugs / feature requests / ...)
[x] Label issues filed for an insiders build as insiders.
[x] Label issues filed for a recently released stable build as new release.
[ ] Search for duplicate issues.
[ ] On bugs: Ask author to add VSCode and OS versions when not provided, also label as needs more info.
[ ] Check description is not empty, label as needs more info.

Maintenance:

[x] Close needs more info issues when the last comment is from our team when stale for 7 days.
[ ] Close feature requests stale for 30 days with either no assignee or no milestone and with less than 20 up-votes

Here 'stale' should include no addition of votes, so the voting has largely finished.
We need to check the existing issues to assess if 30 days and 20 up-votes are good values.

[ ] Close bugs stale for 60 days with either no milestone or on the backlog

/cc @kieferrm

@chrmarti @Tyriar I found this hackathon project very relevant to our problem, maybe we can use similar techniques: https://techcrunch.com/2017/05/14/building-a-smarter-hacker-news/

Moving to June, I think we did not tackle automated labeling and assigning.

new release label and area classifier were implemented for the June milestone.

Small improvements for July (stemming in the text classifier).

Closing, we'll open specific issues for future improvements.


explain both cpu shares algorithms

In this PR I documented two different CPU shares algorithms...

Using open source CF, operators have the choice between algorithms. They can also configure a handful of properties that  affect algorithm 1.
All supported versions of TAS automatically use algorithm 2. TAS docs don't even need to explain the first algorithm.

😅 I did my best to document this. Please let me know if there is anything I can improve. If you have any feedback please reach me on slack @ameowlia . I will respond there much quicker.
✨ TY!

Thanks, @ameowlia !


Is this project still supported by weaveworks maintainers?

As title shows, if i submit issue or pr, will someone take a look？

it does seem abandoned unfortunately :(


Error login

error code 202:BAD SESION in method keybase.1.login.login

What can i do :(

We will look into it right away.

Why just close the case @maxtaco


hop-node: refactor bondWithdrawalWatcher RPC calls

This PR removes some RPC calls that appear to be unnecessary.
The biggest issue is that we would loop through each item marked TransferSent and make 5 RPC calls per chain on initial sync, in parallel, even if withdrawals were already bonded. This happened even if the transfer was bonded but the initial sync simply wasn't up-to-date.

Do not allow txs to be sent during initial sync in order to guarantee DB state

We had a ton of calls being made during initial sync (5 RPC calls per any transfer ever sent in our system, in parallel + an event handler per tx)

Removes an isBonder on chain call b/c we can assume only a bonder is running this software (1 RPC call per bond withdrawal)
Remove const isWithdrawalBonded = bondedAmount.gt(0) || isTransferIdSpent because DB state is guaranteed (2 RPC calls per bond withdrawal + potential 1,000 block event sync)
Do not validate that the transferRoot has been bonded on-chain because DB state is guaranteed (1 RPC call per bond withdrawal)
Only check that the bonder has enough credit and not that the have >0 credit (1 RPC call per bond withdrawal)
Increase wait confirmation checker from 2s to 20s. The total wait time is 144s, 210s, 248s on mainnet/xdai/poly, respectively. In the absolute worst case, a user will wait an extra 19.9s (saves 60-110 calls per bond withdrawal)

Do not allow txs to be sent during initial sync in order to guarantee DB state

Should we just make this the default across the watchers?


Support switching chains with walletconnect

Description

Adds support for walletconnect wallet_switchEthereumChain
Closes #1313

Checklist

[x] The version field in package.json is incremented following semantic versioning
[x] The box that allows repo maintainers to update this PR is checked
[x] I tested locally to make sure this feature/fix works
[x] I have run yarn check & yarn build to confirm there are not any associated errors
[ ] This PR passes the Circle CI checks

@alexkeating I leaft a note on the versioning -  Ill get this tested and checkout tomorrow morning.

Closed in favor of #1363 Thank you for the PR!


Avoid upstreams with multiple servers with the same port

fixes #455

This change is

Coverage decreased (-0.06%) to 43.21% when pulling d82544fe8055da234e7ed681a7e8a5ca068cf6ec on aledbf:avoid-duplicated-us into c25936df62e3148a7646165bcbb5011ef96c4408 on kubernetes:master.


tf_cc_binary() makes opencv unable to load an image

I try to load an image with opencv and work further on it with the tensorflow framework. Unfortunately I get a really weird behaviour:
The image is loaded without problems using cc_binary(...) in Bazel. Changing it to tf_cc_binary(...) doesn't stop the code from compilation or running, but opencv can't load any images any more.
Source code / logs
This is my BUILD file:
load("//tensorflow:tensorflow.bzl", "tf_cc_binary")

#tf_cc_binary( <-- using this, no image could be loaded anymore
cc_binary(
    name = "main",
    srcs = ["main.cpp"],
    linkopts = [
        "-lopencv_core",
        "-lopencv_highgui",
        "-lopencv_imgcodecs",
        "-lopencv_imgproc",
    ],
    visibility=["//visibility:public"]
)

I use the standard example code from the opencv website. Again, it is working and the image gets loaded using cc_binary(:
#include <opencv2/core/core.hpp>
#include <opencv2/highgui/highgui.hpp>
#include <iostream>

using namespace cv;
using namespace std;

int main( int argc, char** argv )
{
    Mat image;
    image = imread("tensorflow/test/imageHolder/data/example.jpg", CV_LOAD_IMAGE_COLOR);   // Read the file

    if(! image.data )                              // Check for invalid input
    {
        cout <<  "Could not open or find the image" << std::endl ;
        return -1;
    }

    namedWindow( "Display window", WINDOW_AUTOSIZE );// Create a window for display.
    imshow( "Display window", image );                   // Show our image inside it.

    waitKey(0);                                          // Wait for a keystroke in the window
    return 0;
}

This is my file structure in case it matters:
├── data
    ├── example.jpg
└── src
    ├── BUILD
    ├── main.cpp

I think this may be a duplicate of #1924. There has been a history of conflicts between opencv and tensorflow see  #7378. This is usually caused by conflicting libraries that both depend on. Try following the advice in #1924, and let us know how it goes. @allenlavoie , will your library isolation stuff help here?

I try to follow the advice to compile a shared library. That worked fine. But there is hardly any documentation about how to work in an external project with the tensorflow_cc.so library.
Is there any example project, that shows how to compile projects with it? I'm mainly confused about which headers to include and where I can find them.

We may want to shift some symbols to a private shared object (or the language bindings) so we don't conflict with symbols in extensions, but it doesn't sound like that's an issue here. If you want to use TensorFlow rather than adding ops/kernels, use the C++ or C API, not tf_cc_binary rules.
@skye is working on a distribution for the C++ API AFAIK. The C API has one.

Unfortunately no one is working on a distribution for the C++ API right now, so if you're trying to use the C++ API, building against TF with bazel is probably the best way. If you can use the C API and follow the instructions Allen linked to, that should avoid this issue.

I struggle quite a bit using the C API. Except for an introduction of how to install the library I can't find any working examples nor any documentation except for some helpful comments in the source code. Why is there a C++ API documentation, but no C API documentation?
Can you give me some help to start with the C API? That would be great!

I would look at the C API tests, e.g. https://github.com/tensorflow/tensorflow/blob/master/tensorflow/c/c_api_test.cc#L686 (this goes over building and running a graph). Note that it uses some helper functions defined in https://github.com/tensorflow/tensorflow/blob/master/tensorflow/c/c_test_util.cc.


Misclassified language

Problem Description
Linguist does not pick up on .yml files in my repo
URL of the affected repository:
https://github.com/flybd5/ansible-role-aws-rds
Last modified on:
2020-08-03
Expected language:
YAML
Detected language:
Python

linguist-vendored
*.yml linguist-vendored=false

...in .gitattributes fixed the problem. This tool needs work.

Linguist does not pick up on .yml files in my repo

It most definitely does, but YAML is considered data by Linguist by default so is not included in the language statistics:
https://github.com/github/linguist/blob/96ad1185828f44bb9b774328a584551ee57ed264/lib/linguist/languages.yml#L6225-L6231
The reason your repo was showing predominantly as Python is because of this file: https://github.com/flybd5/ansible-role-aws-rds/blob/master/filter_plugins/get_attr.py
Your override has simply marked everything in your repo as vendored so none of your files are considered in the language statistics.
If you really want your YAML files to be counted, you can implement a detectable override. If you were to implement just detectable override for .yml files in your repo and remove all of your vendored overrides, you can get a breakdown of:
$ github-linguist . --breakdown
92.49%  YAML
5.00%   Python
2.52%   Makefile

Makefile:
Makefile

YAML:
defaults/main.yml
meta/main.yml
tasks/_facts_subnet_by_filter.yml
tasks/create_rds.yml
tasks/create_rds_paramgroup.yml
tasks/create_rds_subnet.yml
tasks/gather_facts_subnets.yml
tasks/main.yml

Python:
filter_plugins/get_attr.py
$

This tool needs work.

No it doesn't. It's working as designed. 😁

...but YAML is considered data by Linguist by default...

...and that is why the tool needs work, an outright incorrect design assumption. In Ansible, YAML -is- the code. :)

This tool needs work.

No it doesn't. It's working as designed. grin

I think I agree with @flybd5. Linguist needs work, like all software :-)

...and that is why the tool needs work, an outright incorrect design assumption. In Ansible, YAML -is- the code. :)

Yes, we're aware of that issue. Ansible YAML is code, but most YAML files are data. We would welcome a PR to differentiate Ansible YAML from other YAML files. In the meantime, please use overrides.

From my point of view, it doesn't make any difference if YAML is being used for data or for code. Correctly representing the content of a repo without requiring the owner of the repo to jump unnecessary hoops is more important. It's all about the user experience vs. the religious preferences of the developer. :)

Correctly representing the content of a repo without requiring the owner of the repo to jump unnecessary hoops is more important. It's all about the user experience vs. the religious preferences of the developer. :)

I agree, but that requires distinguishing some languages and isn't always easy.

But now someone apparently disabled Linguist entirely from my repo. Interesting development.

Care to detail? :-)

But now someone apparently disabled Linguist entirely from my repo. Interesting development.

You effectively did with this line: https://github.com/flybd5/ansible-role-aws-rds/blob/870b418e5d9b544f25299c1f594ee3c105e7a664/.gitattributes#L1
From my earlier response:

Your override has simply marked everything in your repo as vendored so none of your files are considered in the language statistics.

The linguist analysis disappeared.
@lildude When I made that change I refreshed my browser page and it did exactly what I wanted it to do. The analysis was then correct. Since then the entire analysis has disappeared.

@lildude When I made that change I refreshed my browser page and it did exactly what I wanted it to do. The analysis was then correct. Since then the entire analysis has disappeared.

The analysis of a repo is performed by a background job that can take some time to run, depending on how busy GitHub's job servers are at the time - they're busier during US work hours than EU mornings so can take longer to run then.
The first change which you say had the desired effect would be due to one of the 6 commits you made to your .gitattributes before the final commit which is the one taking effect now.
Looking at your commit history, we can see that it's your final commit at https://github.com/flybd5/ansible-role-aws-rds/commit/870b418e5d9b544f25299c1f594ee3c105e7a664 that "broke" things:
- tasks/*.yml linguist-detectable
- filter_plugins/*.py -linguist-vendored
+ * linguist-vendored
+ *.yml linguist-vendored=false

The first line (tasks/*.yml linguist-detectable) is what would have made some of your YAML files count in the language stats and show YAML in the list of languages.
You removed that and replaced it with * linguist-vendored which marked everything as vendored and *.yml linguist-vendored=false which tells Linguist to consider those files to not be vendored, however they're not considered vendored normally, so this has no effect, thus putting your YAML files back to their default behaviour of being considered data and thus "undetectable".
The commit before the last (https://github.com/flybd5/ansible-role-aws-rds/commit/b8da8598c85882d7f07141aa9b56f6aa1f9717c8) is the one that made YAML visible in the language stats and it would have been after the analysis kicked off by this commit that you most likely noticed things working as you expect.
If you want a complete analysis of your repo displaying all languages actually in the repo, including YAML, with it being the predominant language, which you are expecting, you can get this with this one line:
*.yml linguist-detectable

This will produce this in the sidebar:

... and this in your list of repos:

This was taken from my fork of your repo in which I've implemented just that line.

Thank you for the complete explanation. That accomplished what I wanted. :)


Adds user-agent to OCSP requests

Uses the proxy.config.http.request_via_str override as the User-Agent
Closes #8721

I will take a look at this one.


[tool crash] StdinException: StdinException: Error setting terminal line mode, OS Error: The handle is invalid. , errno = 6

command
flutter run --no-sound-null-safety
exception
StdinException: StdinException: Error setting terminal line mode, OS Error: The handle is invalid.
, errno = 6
#0      Stdin.lineMode= (dart:io-patch/stdio_patch.dart:129:7)
#1      AnsiTerminal.singleCharMode= (package:flutter_tools/src/base/terminal.dart:292:13)
#2      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:722:24)
<asynchronous suspension>
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1195:27)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#5      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#6      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#8      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)
<asynchronous suspension>
#9      run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#11     main (package:flutter_tools/executable.dart:94:3)
<asynchronous suspension>

flutter doctor
[✓] Flutter (Channel master, 3.1.0-0.0.pre.1243, on Microsoft Windows [Version 10.0.19044.1766], locale en-US)
    • Flutter version 3.1.0-0.0.pre.1243 at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dc06326caa (13 days ago), 2022-06-13 16:48:09 -0700
    • Engine revision 6670f92130
    • Dart version 2.18.0 (build 2.18.0-180.0.dev)
    • DevTools version 2.14.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\MASSAWE\AppData\Local\Android\sdk
    • Platform android-31, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[!] Visual Studio - develop for Windows (Visual Studio Build Tools 2022 17.2.2)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools
    • Visual Studio Build Tools 2022 version 17.2.32519.379
    ✗ Visual Studio is missing necessary components. Please re-run the Visual Studio installer for the "Desktop development with C++" workload, and
      include these components:
        MSVC v142 - VS 2019 C++ x64/x86 build tools
         - If there are multiple build tool versions available, install the latest
        C++ CMake tools for Windows
        Windows 10 SDK

[✓] Android Studio (version 4.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.68.1)
    • VS Code at C:\Users\MASSAWE\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.26.0

[✓] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19044.1766]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 102.0.5005.115
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 103.0.1264.37

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Labeling this for further investigation as I can't find any existing issues

@Massawe14 thank you for filing this! It looks to me like you were running this from the terminal? And I'm guessing you didn't do anything weird like close your terminal while the CLI tool was running?
I'm trying to figure out why the OS is saying the STDIN handle in invalid.


Fix hour import

Instead of unofficial API that was removed it now uses official APIs.

You might want to squash these a bit before merging. :thinking:


fix: update to eye v10.28.1

fix: update to eye v10.28.1

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

npm package (@latest dist-tag)
GitHub release
v16.32.1

Your semantic-release bot :package::rocket:


Loop failed

While True
Use a while True loop to let the user type in a new number once again or exit with sys.exit

Ok
Am 14.08.2017 21:42 schrieb "Hakim"<EMAIL_ADDRESS>
While True
Use a while True loop to let the user type in a new number or exit with
sys.exit
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Univream/DecimalConverter/issues/5, or mute the
thread
https://github.com/notifications/unsubscribe-auth/Acx8oeJC1NEiBf_a2-FJOC52iLAlab0Rks5sYKMtgaJpZM4O2zwd
.


Multi-room/time course entry support is limited.

The system has incomplete support for courses with multiple "lines" in the CSV, that is, courses with more than one time or room (e.g., CS 262: MW 1:30-2:20 NH 253 & F 1:30-3:20 SB 354).
It can import, edit, export in these course "lines". But it cannot add them.

If we adopt the new daily schedule, this will be even more common since many 4-hour courses will meet 3 days per week for different lengths of time on different days.

Except for the time field, this could be implemented in the current editing box by supporting delimiters and using R-style recycling.  For the time field, we would need to change back to a text-entry area or come up with some other interface.
Regarding delimiters: We probably want at least two.  Something like:

&: multiple entries are always in effect (two instructors, two rooms, etc.)
|: alternating entries.

Example: Prof A & Prof B; M | T R;   8:30 | 8:05. could code a 4-hour course with the current draft schedule with 2 instructors (who teach all 3 days).

We discovered this functionality is in fact built-in, but you have to go about it in a tricky way.
If we want a class like CS262 to be in SB010 M/W and the lab on Friday:
First create the class as normal in the SB 010 room and ONLY SELECT Monday and Wednesday. It should appear on the schedule.
Then click on the class to open up the editor window, change the room, and change the day to now only having friday selected. Click the Add New Section button.
It should now export with that class as 1 line on the spreadsheet with 2 sets of days, 2 sets of times, and 2 rooms.
Note, these are now treated as "blocks" so if you edit and use the Update Section button on the M/W part of the class the Friday will remain unaffected and vice versa.


pathPrefix not working

I'm trying to use the pathPrefix on gatsby-config but it seems that isn't working.
pathPrefix: "/arqui-docs",
I've build it using the command
gatsby build --prefix-paths
the start page seems to be working properly

Bbut after navigate to any o the links, the sidebar doesn't shows anything :(

Am I missing something?
Thanks.

@joaorsfonseca my bad. I didn't handle the prefixpath nicely. Should work now!

@cvluca how do i update the source code withtou losing all the changes i already made on my md files mainly :)


OLS-364: Enable 'misc' types check on CI

Description
Enable 'misc' types check on CI (finally)
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #OLS-364

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

/approve


[.github] Update PULL_REQUEST_TEMPLATE.md

Two things:

Since we moved from JIRA to GitHub issues, there is no standardized way to reference bug numbers in the compiler. We previously used the convention SR15818_Struct or SR_15818_Struct in regression tests, sometimes substituting PR for SR to indicate crashers reported in a pull request. Now, we would be putting #15818_Struct or 15818_Struct in source code, which is not valid code. Alternatively, we would do Issue15818_Struct but that's like using PullRequest15818_Struct instead of PR15818_Struct. Is there any form of standardization in the post-JIRA era?

I originally put "Resolves PR-NNNNN" instead of "Resolves #NNNNN" for this reason.

Expand to 5 digits. It's far past the point where most pull requests resolve something with 4 digits. JIRA was 50% past 9,999 before jumping to GitHub issues, and now we're 60% of the way to 100,000.

Given the far-impacting nature of this, I would like at least 2 people to review it before merging.


[Question] Register custom @propertyWrapper as Attribute

First of all big up for Sourcery @krzysztofzablocki!

I am wondering if it is possible to register custom @propertyWrapper as Attribute.Identifier type and find them in stencil files ?
Trying to get {% for var in type.variables %} {{ var.type.inherits.UIView }} and check if veriable inherits from UIView, seems type is always nil no matter what, but typeName give me UILabel/UIView is there a reason for that type is nil for variables ?

@m1entus
regarding 2. UIView is unknown type so you'd have to use type.based.UIView, you should be getting type property if the types are known to Sourcery (defined in the source it scans), let me know if that's not the case
as for 1 I think you'll get attribute propertyWrapper and could look into it's arguments to find whatever you are looking for

Thanks for quick reply!
For the 1 i have defined, where @AccessibilityIdentify is propertyWrapper type and seems there are no results, IBOutlers etc. are working fine but for my custom one i am not getting anything:
@AccessibilityIdentify(identifier: "custom_id")     var testView = UIView(frame: .zero)
Regardin 2. i have done some workaround and extended UIView with my custom protocol UIViewAccessibilityIdentificable for custom types and checking if hasPrefix UI and seems this is satisfying me for now.
{% if var.type.based.UIViewAccessibilityIdentificable or var.typeName|hasPrefix:"UI" %}


Feature/upgrade parity

uses new builder for circleci not yet on master.
I think you can check commits individually, and if you don't like renaming parity to openethereum we can discard the last two.

ok, we need to find out why the pre commit hooks fail, it is getting annoying.


Doesn't work under 1.19.5.

Hello.
Thank you for taking this project up. This is one of the most valuable Atom packages for me.
I have it installed under 1.19.5. The menu item shows up, but when I click on Configure, nothing happens.
I am also tried manually creating the .remote-sync.json file. But still no go.
Thank you.
Anders.

Idem for remote-sync-pro and remote-sync
Failed to require the main module of 'remote-sync-pro' because it requires one or more incompatible native modules (binding, binding, binding, binding, binding, binding).
Run `apm rebuild` in the package directory and restart Atom to resolve.

I found the solution. After trying to rebuild remote-sync-pro in command line, I used the point and click method. Click on the red bug (bottom right of the window), rebuild, restart et voilà

I found the solution. After trying to rebuild remote-sync-pro in command line, I used the point and click method. Click on the red bug (bottom right of the window), rebuild, restart et voilà

This also fix the issue for me 👍


Backpropagation attempts. Currently no bias awareness.

Began on backpropagation. Although largely untested right now, I'd like to give you the code before I begin making changes to the rest of the library to broaden my understanding.

LGTM but I would like to test it with a basic classification problem before we merge

Also need to use ndarray


add me-central2 region

lowest latency region for saudi arabia should be me-central2 (Dammam) but it is currently showing a different region. is it possible to add all new locations in middle east to the list?

Until https://github.com/GoogleCloudPlatform/region-picker/issues/19 is fixed, new regions are only added when carbon data is available

me-central2 added, with missing carbon data


Documenting MBSA issue with latest cab file

Just as FYI, the note was taken from this page:
https://docs.microsoft.com/en-us/windows/win32/wua_sdk/using-wua-to-scan-for-updates-offline

Specifically this commit: https://github.com/MicrosoftDocs/win32/commit/8f29b4d8c69a38568198b2b1ea9929b82c07c303#diff-4d05c816b88a7098856ed36e4990fa7f

Hi, could someone please share an update on this? Thank you.

@andreiztm The PR has been escalated for review. Thank you for following up.

@e0i thanks for the info, looking forward to any updates.

@e0i do we have any update? Just as FYI, I wrote this article.

The content is going to be merged by a vendor with merge permissions soon, thank you.

Thank you for the confirmation.

Thank you everyone!


Reading initial brightness from DDC/CI-compatible display

I’m having an issue with the app reading the initial brightness from my external display. I have the ‘read initial brightness from display’ option checked in the app’s preferences & my display does support DDC/CI. However, the initial brightness is still set to 50% when changing the it for the first time.
I’m not sure if this is a compatibility problem is or a bug in the app, but I’ve included some info about my set up below hoping you might be able to tell me?
System Info:

Early 2015 MBP running macOS 10.15.2
ExternalDisplayBrightness v1.3.0
External display: benQ GW2265HM

thanks in advance!

Hi,
thanks for the report! I'm afraid it's a compatibility issue with the monitor itself, a lot of them don't support reading values over DDC/CI, just writing them. None of the monitors I have (from Samsung, AOC and NEC) actually supports reading the brightness either, I think it's pretty rare for a monitor to support it (I've heard from a few users that it works for them, though, so I think the code in the app is actually right).
If you want to confirm this, perhaps you could try using ddcctl to try and read the current brightness from the display, and see if it works? If it does, that means the problem is with my app, and I'll try to fix it, if it doesn't, the monitor is probably incompatible and there's not much I can do.
Thanks!

yeah.. I was kinda figuring that was the case, but I wanted to ask. it's not big deal anyway, I'm just glad I dont have to fumble with the monitor controls to change the brightness anymore haha
thanks for the info!

I'm afraid it's a compatibility issue with the monitor itself, a lot of them don't support reading values over DDC/CI, just writing them. None of the monitors I have (from Samsung, AOC and NEC) actually supports reading the brightness either, I think it's pretty rare for a monitor to support it (I've heard from a few users that it works for them, though, so I think the code in the app is actually right).

yeah.. I was kinda figuring that was the case, but I wanted to ask.

If you want to confirm this, perhaps you could try using ddcctl

ok, I might give that try just to see. it's not big deal either way, I'm just glad I dont have to fumble with the monitor controls to change the brightness anymore haha
thanks for the info!

No problem! I'm going to close the issue for now, since it's (most probably) not caused by ExternalDisplayBrightness, so it unfortunately can't be solved here.


Milestone - Version 1.0.0

Issue marking the release of Version 1.0.0 when everything is considered stable.

NuGet package released!
Install-Package SimpleSoft.DependencyInjection -Version 1.0.0


Bump a few versions

I wanted to use x-vector downstream and it depends on vector 0.11
! @amosr @tranma

🐮


[SMALLFIX] Add checkstyle rule for whitespace at end of block

Similar to our existing extra newline rule. There's very little value to ending a block with a newline, and 99% of the time we don't. Now it will be 100% and enforced automatically

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/458/
Test PASSed.

Merged build finished. Test PASSed.

@calvinjia Added, thanks

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/469/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/473/
Test PASSed.


Attach an already opened Admin mode application

Hello, I am having problems when I try to attach a WinAppDriver to my application that has to be in ADMIN MODE.
I receive a SessionNotCreatedExeption as below:

MY CODE (works fine if I open the application not in admin mode):
@BeforeSuite
public final void searchLC()  {
try{
//appTitlesFromFile = excelUserData.getPolluxGWDataFromFile();
DesiredCapabilities desktopCapabilities = new DesiredCapabilities();
desktopCapabilities.setCapability("platformName", "Windows");
desktopCapabilities.setCapability("deviceName", "WindowsPC");
desktopCapabilities.setCapability("app", "Root");
WindowsDriver desktopSession = new WindowsDriver(new URL("http://<IP_ADDRESS>:4723/"), desktopCapabilities);
DesiredCapabilities LCCapabilities = new DesiredCapabilities();
LCCapabilities.setCapability("platformName", "Windows");
LCCapabilities.setCapability("deviceName", "WindowsPC");
//WebElement Reproduction = desktopSession.findElementByName(appTitlesFromFile.get(0).get("LogCollectorTitle"));
List lc = desktopSession.findElementsByName("LogCollector - Ver. 5.23.2247.45 Copyright (C) Media Instruments SA 2010-2021. All Rights Reserved.");
if (lc.size() == 1) {
String LCWinHandleStr = lc.get(0).getAttribute("NativeWindowHandle");
int LCWinHandleInt = Integer.parseInt(LCWinHandleStr);
String LCWinHandleHex = Integer.toHexString(LCWinHandleInt);
LCCapabilities.setCapability("appTopLevelWindow", LCWinHandleHex);
System.out.println("LC Handle is: " + LCWinHandleHex);
}else{
//LCCapabilities.setCapability("ms:waitForAppLaunch",5);
//LCCapabilities.setCapability("app", "C:\UNITAM SW\LogCollector.exe");
System.out.println("Something went wrong....");
}
driverWinLC = new WindowsDriver(new URL("http://<IP_ADDRESS>:4723/"), LCCapabilities);
Thread.sleep(500);
//driverWinLC.switchTo().window(LCWinHandleHex);
    } catch (Exception e) {
        e.printStackTrace();
    }
}

The App title is correct:

what could the problem be??
thank you very much

The error clearly mentions that the capability is wrong. What value do you get for lc.size()?

ok, I got the issue. To be able to open an app in Admin mode, first of all it is mandatory to open WinAppDriver.exe in Admin mode. In this way my issue is solved

Hello, is there a workaround  in order to have the WinAppDriver application run with any windows version, not just win10?
Or I won't be able to run my framework with WinAppDriver in different win version?
thanks

No only WIN10 and Windows server 2016.

Thanks for the quick answer.
Is there another application, instead of WinAppDriver, if I need to run my framework in another window version?
Or to automate windows application it is mandatory win10?
thanks again

Which language binding are you planning to use?

java

I have no idea in java. But there are several paid tools that you can use.


Consolidate start and rekey functions into configureKeys

Please update these functions and all references throughout the urbitCtrl. (Adding a separate issue for view updates).
https://github.com/urbit/etherwallet/blob/mercury/app/scripts/controllers/urbitCtrl.js#L1315

Fixed according to this.


[release] 0.0.0-release-test\n## 0.0.0-release-test

Bug fixes

critical bugfix in unique renaming, #429

Features

opt-in for statistics collection (shared with TLC and TLA+ Toolbox), see #288

Architecture

new layer of TransitionExecutor (TRex), see at.forsyte.apalache.tla.bmcmt.trex.*

Documentation

Compile the manuals into a static
site using
mdBook, see #400
Description of top-level user operators, see #419
ADR003: Architecture of TransitionExecutor

Closer!

Closer!


Namespace changes

First of all, thank you for this nice library. Make my life a bit easier!
I do want to mention that in version 1.1.1 there was a change in namespace.  This should not be done in a minor/patch level update.
Just making sure it won't happen again, since it will break stuff :-)

Yeah...
I am still learning some tweaking regarding semver :-).
Thx for saying it!
I released version 2.x that is breaking ;-)!


PreSpawned object errors when doing remote-client prediction

When sending PreSpawned objects to remote players, aka player 1 presses Fire and spawns a PreSpawned object locally, and the Fire input gets replicated to player 2 (via ActionState replication). We sent player 1 inputs to player 2 because we want to do remote player prediction.
The server also receives player 1's Fire input and spawns a bullet that will be replicated to all clients (and probably predicted on all clients).
2 cases:

either the ActionState from player 1 is replicated before the bullet, and then player2 spawns a PreSpawned object which then gets merged into the replicated bullet (via hash matching)
either the server bullet is replicated before the ActionState, and in this case we get 2 bullets:

the first bullet is replicated from the server
the second bullet comes from the ActionState. It survives for a bit but then is despawned because it couldn't match with a server entity (via hash matching) because the server entity was spawned before!!

The solution might be to allow hash matching in both directions? ServerReplicated into PreSpawnedClient, and PreSpawnedClient into ServerReplicated?
Currently the PreSpawnedObject component is removed immediately on the initial matching attempt.

Doing the matching in both directions seems pretty brittle, maybe the best solution is for users to follow the rule:
Do not spawn pre-spawned player objects using inputs from remote players!
This will result in double-spawning the entity (once because it's replicated from the server, and once locally when applying the remote-input.

I've handled this by only pre spawning bullets for remote players if I have their inputs early - ie, we are not in rollback when we see the PlayerAction Fire for the current tick.
In cases where remote player inputs arrive late, the server spawns the bullet and it gets replicated normally (triggering a rollback).
Adding my firing system like this:
   shared_player_firing.run_if(not(is_in_rollback)),

safe to close this imo


Add 'mafia lock' for locking down package versions

This adds two new commands, mafia lock and mafia unlock.
mafia lock creates a <project>.lock file alongside the <project>.cabal file which contains the version numbers and flags of all the Hackage packages which were used during the last build. Subsequent builds add all of these constraints to any solving they do. This means that if someone releases a newer version of a locked down package on Hackage, then the newer version will be ignored.
Running mafia lock when you already have a .lock file can be used to update the .lock with any new packages you've added to your .cabal file without losing a lock on the existing ones.
mafia unlock simply deletes the .lock file, allowing Cabal's solver free reign over package choice.
I'm not completely sure whether lock files will be a net positive or negative, would be great to hear peoples thoughts? It certainly could be useful to avoid rebuilds at the present time, given that people are madly upgrading all their packages to support GHC 8.0.
Happy for people to bikeshed the file names and the way it it is invoked. Some package manager / build systems (such as Rust's Cargo) create a lock file automatically when you build.
For the record, I tried to get Cabal's freeze/unfreeze stuff to work, but there's something about the way we do dependency solving in Mafia that means Cabal just ignores the freeze file which it creates, shrugs.
/cc @markhibberd @charleso @thumphries @damncabbage

For an idea of what the .lock file looks like:
~/src/mismi/mismi-s3 $ mafia lock
~/src/mismi/mismi-s3 $ cat mismi-s3.lock
abstract-deque == 0.3
abstract-par == 0.3.3
adjunctions == 4.2.2
aeson == <IP_ADDRESS>
aeson +old-locale
amazonka == 1.3.6
amazonka-core == 1.3.6
amazonka-core +old-locale
amazonka-s3 == 1.3.6
ansi-terminal == <IP_ADDRESS>
...

What does it do with submodule dependencies?

I figured they are already locked down by git, so I omitted them from the lock file.

@jystic That makes it more useful then freeze. I am a bit out of date on current state of freeze, but the other problems that freeze had last time I looked (and I would be interested in how this stacks up):

freeze across multiple ghc versions can cause problems
freeze in libraries is painful because it applied recursively which meant resolution on large dependency trees extremely problematic

freeze across multiple ghc versions can cause problems

That's a point I hadn't considered, we should probably create <project>.lock-<ghcver> files instead.

freeze in libraries is painful because it applied recursively which meant resolution on large dependency trees extremely problematic

Are you saying that with submodules it would read the freeze file (e.g. cabal.config) from the submodule as well? So that if we had a freeze file for P, it would affect dependency resolution for everything?
Just to be clear about what I said earlier:

the Mafia lock file includes all transitive Hackage dependencies
lock files in source dependencies (such as submodules) have no effect on the current project
source dependency version numbers are not included in the lock file as the version is implied

It's possibly worth including the flags chosen for source dependencies in the lock file.
As a side note, Mafia was already adding additional constraints during solving which force Cabal to pick the source package, even if it is available on Hackage as well.

Something for later: it'd be nice to have an upgrade <package>, where to upgrade a package to a newer version, it filters out that package from the lockfile when handing the constraints over to Cabal, and then writes down whatever valid version comes back from the install. (I think this is possible, unless I've misunderstood how this works. Definitely a separate PR, to be clear.)

it'd be nice to have an upgrade <package>

Yeah it's a shame mafia upgrade is already in use as a command for bootstrapping script :/ But yes, something like that would be pretty awesome!

it filters out that package from the lockfile when handing the constraints over to Cabal, and then writes down whatever valid version comes back from the install.

Until something like that is implemented, you can easily do exactly that by hand. If you remove a package from the lock file, then run mafia lock again, it will re-solve with the modified lock file and produce a new lock file which has the (possibly) updated package version.

Nice! Good to know.
Nowish would possibly be a good time to rename that to self-upgrade / selfupgrade, before this tool gets too much more public.

The feature and code looks good to me. Not sure where we'd use it in our infrastructure, but I'd probably use it locally.
Automatically generating the lock file on a clean build is quite interesting. It seems benign, but could introduce a gap between branches and master builds?
I can also imagine a world of hurt if we started committing lock files to repos (a build could be green on CI thanks to the lock, and fail hard when sourced as a submodule)

I think having an environment MAFIA_LOCK_IGNORE flag (or thereabouts) might help if we actually wanted to use this, and we could have an extra canary build with the locks ignored.

Seems legit. Wouldn't mind discussing whether it's something we actually want to use in anger, or whether we prefer the current situation, and maybe discuss the pros/cons of each approach.

I can also imagine a world of hurt if we started committing lock files to repos (a build could be green on CI thanks to the lock, and fail hard when sourced as a submodule)

Yeah I think that's a problem in general that we'll have to experiment with, perhaps the canary builds would solve it.

I think having an environment MAFIA_LOCK_IGNORE flag (or thereabouts) might help if we actually wanted to use this, and we could have an extra canary build with the locks ignored.

I added a MAFIA_LOCK environment variable that lets you specify which lock file to use. You can use MAFIA_LOCK="" mafia build to build without using a lock file.
More interestingly you could say MAFIA_LOCK=lib/mismi/mismi-s3/mismi-s3.lock-7.8.4 mafia build to use the lock file from another project. Running mafia lock always uses the constraints from the last build if it can, so you'd be able to save the versions which you used when building with the other project's lock file.

:+1:


Disabled buttons get colored error fixed

Description

Motivation & Context

Type of changes

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

Checklist

[ ] I have read the contributing guidelines
[ ] My code follows the code style of the project (using npm run lint)
[ ] My change introduces changes to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-{your_pr_number}--twbs-bootstrap.netlify.app/

Related issues
Closes #40557

Ok I got it @julien-deramond
I have made new changes
Please check out
Thanks

This PR makes the ligtness of the buttons fixed. The buttons should fade into the background , so the lightness has to change depending on which background a button is put on.
So this does not close the mentioned issue.
@Riyazul555 There is already an open PR for the issue.  Please see https://github.com/twbs/bootstrap/pull/40559  -feedback welcome!


Mobile styles of login modal

[ ] Jeff's head is cut off
[ ] Social login should be on one line
[x] Scrolling doesn't work

@gglucass can you add dimensions that you're starting to notice this?

@gglucass can you add dimensions that you're starting to notice this?

iPhone 13 mini dimensions - 375px × 812px

ok this tracks makes sense it's a mini phone, current gens are ok. will add media queries :)

@gglucass do we want to re-introduce the original styling that was reverted in #90? it is fixed but imo it definitely is missing a level of flair now


timewarrior: Fix homepage

Backporting to stable branch should be done, I guess.

Ported to stable: 7063fb7f645c653d614efa42df4a6bd3a2a4a8dc


Backports 0.15 pr12

missing this part https://github.com/bitcoin/bitcoin/pull/9942/files#diff-ca74c4b28865382863b8fe7633a85cd6L853

Looks good but could you squash 6545d37 into 56e6cb2 pls?

Looks like #include "policy/fees.h" is missing in validation.cpp

@PastaPastaPasta See 4c89c2c79e39affa36caba974eccb294a0ab1094, this should fix the build errors.

And disconnect_ban.py fails, not sure why yet.

Pls see https://github.com/UdjinM6/dash/commit/a0c2f2a2574f29fe1b89f3fad661d53afedc398b


significant divergence between blanton and FastSpecFit k-corrections

Upon comparing the galaxy K corrections exported by FastSpecFit, most specifically KCORR01_SDSS_R, I realised that they diverge significantly from Blanton's kcorrect estimates for z > 0.37 . Is there any reason why they differ? Are FastSpecFit estimations reliable for z>.37?

Thanks for the question, @jere1882. Are you using the same input photometry and redshifts? And are you running the code yourself or using a value-added catalog?
What you're seeing with FastSpecFit is an algorithmic choice where the K-correction is computed from the band which minimizes the K-correction (given the desired rest-frame bandpass). For example, if you want M_g from grz photometry, at low redshift the code will use the g-band, but then eventually it's going to switch to the r-band, because that observed-frame redder band is closer to the desired rest-frame band.
I don't think k-correct includes this logic.
The key thing is to compare the rest-frame photometry (making sure you're using the appropriate filter curves). Do those values match better?

Thank you very much for your reply!
In order to calculate Blanton's estimates, I am using:

photometry from Desi Legacy  - most specifically, dered_flux_{g,r,z}
redshift from DESI-edr (redrock)
I am correctly specifying the input and output passband functions to match the ones exported in your catalog, as well as the band shift:

responses_in = ['bass_g', 'bass_r', 'mzls_z']
responses_out = ['sdss_g0', 'sdss_r0', 'sdss_z0']
band_shift=0.1

As for FastSpecFit, I am directly pulling the values from your exported VAC, and those are the values that you see plotted on the right. To the best of my knowledge the resulting estimations should be comparable, but for some reason they diverge after z ~ 0.37. Similar scenario with band g ; but the divergence starts at z ~ 0.30

Thank you very much for your reply!
In order to calculate Blanton's estimates, I am using:

photometry from Desi Legacy  - most specifically, dered_flux_{g,r,z}
redshift from DESI-edr (redrock)
I am correctly specifying the input and output passband functions to match the ones exported in your catalog, as well as the band shift:

responses_in = ['bass_g', 'bass_r', 'mzls_z']
responses_out = ['sdss_g0', 'sdss_r0', 'sdss_z0']
band_shift=0.1

As for FastSpecFit, I am directly pulling the values from your exported VAC, and those are the values that you see plotted on the right. To the best of my knowledge the resulting estimations should be comparable, but for some reason they diverge after z ~ 0.37. Similar scenario with band g ; but the divergence starts at z ~ 0.30


Make versioning consistent between git tags and package.json

Right now the package.json says 2.6. The git tags should reflect the version we have in package.json. Also, I suggest we use the semver pattern for version numbers. [major].[minor].[patch]-[pre] - http://semver.org/

I think we've been doing this? Comment if there's still something to discuss, otherwise please feel free to close.


(PDB-3058) strip null bytes in terminus

Strip out null bytes in the terminus instead of in
munge-jsonb-for-storage, since null bytes will cause inserts of text to
fail in addition to jsonb.

I'm not sure this captures every representation of problematic bytes, but it at least extends our coverage of the issue to text fields. It probably negatively impacts terminus performance. We might want to have some discussion around that.

Test FAILed

CLA signed by all contributors.

@ajroetker I fixed this up to your comments and fixed the spec test issues. I guess we're still waiting for jenkins to come back so we can run the acceptance test.

@puppetlabs-jenkins test this please

Test FAILed

Test FAILed

@puppetlabs-jenkins retest this please

Test FAILed

Test FAILed

@puppetlabs-jenkins retest this please

Test FAILed

Test FAILed

@puppetlabs-jenkins retest this please

Test FAILed

@puppetlabs-jenkins retest this please

Test FAILed

Test FAILed

Test FAILed

rebased onto 4.2.3.x, we'll see how the tests look

@puppetlabs-jenkins retest this please

Test FAILed

CLA signed by all contributors.

@puppetlabs-jenkins retest this please

Test FAILed

Closing this; we're going to try a different approach.


Update v2 > v3 migration guide

What does this PR do?
This PR is documentation fix for Traefik v2 to v3 migration guide.

How about we keep the configuration changes on a separate page and link to it in the migration guide?

How about we keep the configuration changes on a separate page and link to it in the migration guide?

I haven't seen this comment before, but this could simplify the structure of this documentation. I would vote for it.


get_file() progbar fix

Includes fix get_file download progress bar #6535 with improvement to scoping clarity.
Additionally fixes bug in python 2.7 where get_file() crashes when downloading files where the response does not supply 'Content-Length':
    from keras.utils import get_file
    listing_url = 'https://sites.google.com/site/brainrobotdata/home/grasping-dataset/grasp_listing.txt'
    grasp_listing_path = get_file('grasp_listing.txt', listing_url)

Travis failure is neither in or using a section of code I modified in this pull request.

How to re-run travis?

closing and reopening due to flaky tests...

My suggestion to use None was meant to avoid having -1 as a special value. Why not only use None?

@fchollet One reason I used -1 is due to the reporthook parameter in urlretrieve, which in this case is the function dl_progress:

If reporthook is given, it must be a function accepting three numeric parameters: A chunk number, the maximum size chunks are read in and the total size of the download (-1 if unknown).

The other reason is that most references to self.target in progbar would need additional more complicated logic, where with -1 nearly all the current comparisons and checks remain valid. I probably don't have my machine setup correctly or the time allocation to rewrite all the calculations and checks for None, then manually test on every platform to make sure the output looks right.
Those are the reasons 289a5e4 accepts both None and -1. I would suggest leaving it because this already fixes several current bugs, but I could also disable -1 on the documented API and just use it internally. That would only be a small change.

@fchollet You're right -1 doesn't make sense, now it is just an implementation detail, not part of the API. Hopefully ready to merge! 👍

Would still be preferable not to have -1 in the implementation. We can do that at a later time though.


benchmarks: use dedicated module-cache directory for building the benchmarks

So that it cannot interfere with some leftovers from other compiler runs. This is important for SDK modules which are generated from swiftinterface files (like Foundation). The cached SDK module should be built with the compiler to benchmark (and not being reused from other compiler runs).

@swift-ci smoke test windows

@swift-ci smoke test macos


Add outputs to the sample project

The sample project is a great way to try out a bunch of services together, would be nice to have the outputs (like connection string. username, password) so the user knows where to connect to.

@jonatasbaldin, we've added some example outputs to the sample project :)


README makes use of undocumented function static_url

4e6f49c910492ff66cefcc763ce5d1dcc1d826f6 and 03e3ac4594dce5585607b623d7a1b279bdc55187 introduced the use of the static_url function in SUMMERNOTE_CONFIG sample settings in the README.
However this function is only defined locally and can not be used in the normal django settings.
Of course this isn't much of an issue, but it will confuse users.

Sure confused me.

Sorry for inconvenient. We removed it.


CLI arguments are not aligned with Pytest arguments style

Pytest arguments replace the spaces by minus (-) when defining arguments. For example:
pytest --durations=10 --durations-min=1.0

https://docs.pytest.org/en/6.2.x/usage.html#profiling-test-execution-duration
whereas in pytest-pyvista all the arguments are separated by "_".
This is a minor thing.  But I just wanted to highlight it for consistency.

Mmhh... the fault handler also uses "_" so ... feel free to close if you feel like it.
https://docs.pytest.org/en/6.2.x/usage.html#fault-handler

I have no strong opinion.  The most important thing is to be consistent within this package and not mix the usage.  Let's leave open to see if anyone has a strong opinion.


Worker restarts: address already in use

Everything starts fine on the first run, but when changing code and hot reloading is triggered, the worker seems to gracefully shutdown successfully, but upon restarting, the process complains the default 3000 port is already in use(I assume the one nestjs is listening. Now, I have a feeling this is because the worker is spawned as a child process, and it tries to bind to the same address. Maybe I'm wrong, but I can't seem to find a workaround. Any ideas would be appreciated.

it seems like you are missing app.enableShutdownHooks()
async function bootstrap() {
    const port = 3000

    const app = await NestFactory.create(AppModule, {
        bufferLogs: true,
        autoFlushLogs: true,
    })
    app.enableShutdownHooks()

    await app.listen(port, () => {
        /* eslint-disable no-console */
        console.log(`[server]: Server is running at http://localhost:${port}`)
    })
}

bootstrap().catch(err => {
    /* eslint-disable no-console */
    console.error(err)
    process.exit(1)
})

Apart from a warn saying worker was not cleanly shutdown, this seems to be doing the job. An alternative I used so far is what I described here https://github.com/temporalio/sdk-typescript/issues/1437
basically, running the codebase in 2 processes, one regular http app and the worker running in an application context only, which might adhere better to docker best practices


Extend documentation related to asdf and direnv installation

ASFD and DIRENV installation part needs to be improved.

Another issue found: terraform-docs is required by terraform_docs pre-commit hook but is not installed or in the system's PATH.


List Books

As a: Customer
I want: to see a list of all library books
So that: I can browse for books I might want to check out
Acceptance Criteria:
The list should be rendered after the Welcome Message

SHOULD fix the TODOS