Revert "Update PopupMenuButton widget"

Reverts flutter/flutter#80420. Basically we want to avoid the use of addPostFrameCallback as a solution for this as we want to avoid its use if at all possible.
I spoke to @HansMuller and we think it might be best to approach this one of two ways:

find a solution that doesn't use addPostFrameCallback
(could also be the temporary solution until we find an answer to 1) have the menu close whenever any resizing of the window happens

I have just tested this behavior on Android native.
https://github.com/flutter/flutter/pull/80420#discussion_r617126995

In my opinion, Solution 2 has two problems,

The implementation is inconsistent with the web and Android native;
As a temporary plan, it may be a breaking change when returning to plan 1;

I agree with the first solution.
The current problem is that if you want to refresh the button and menu in the same frame, the position of the menu depends on the position of the button, but the current Flutter architecture does not allow access to non-parent-child relationships' size during the layout phase. I mentioned this in the previous comment. https://github.com/flutter/flutter/pull/80420#issuecomment-820169178

Flutter's (mostly) one-pass layout limitation, that a parent widget can't know it's descendant's size at build time, has been cropping up more often lately. There's no easy way around the problem.

I just took a look at the behavior definition of the DropdownButton, which is very interesting. It was previously dismissed when the size changed, but it was rolled back by this patch. #42482(which is @shihaohong reviewed : ) )
The functions of PopupMenuButton and DropdownButton are very similar, but they have different behavior definitions. I think it's time for us to unify their behaviors.
CC @HansMuller @shihaohong

Yeah, that's also pretty tricky because we definitely do not want the keyboard closing to cause the dropdown menu to close when tapped on after having the keyboard dismissed (like with text fields) as well. We would somehow need to differentiate between the keyboard closing and the window resizing.
Maybe we can query if the Flutter app's target platform is web/desktop and have the window resizing only close the dropdown in those cases, since there's no virtual keyboard? WDYT @HansMuller


Colleagues presence info is not available immediately after re-opening popup after succesful connection

Version
master
File / Feature
Colleagues' presence info
Expected behaviour
Presence info should be at the ready after re-opening the popup.
Actual behaviour
When re-opening the popup, it looks like a new connection is made every time.
Stacktrace / Error message
-
Other info
The existing plugin keeps the state in background.js and pulls it from there using request/response events

My plugin might have been in a weird state, I myself cannot reproduce this consistently. I do remember on logout that I had two error messages in background.js stating it could not disconnect because the websocket connection was already closed.

I could not reproduce this. Cleaned up connection handling though.


Curation: guidelines for reviewers and approvers

Related issue: TBD
Deploy preview: here
This PR is an attempt at separating things related to authoring and reviewing/approval:

points relevant only for reviewers were removed from authoring guidelines,
points related to review and approval were removed from kata/translation concepts
sections with curating guidelines and curating howto (meant to be used by reviewers) were created, for translations and kata

This PR is still in draft because there are still some things I'd like to add, but I was struggling a lot with proper organization, navigation, getting rid of redundant parts etc, so I'd appreciate any feedback also on early stage.

I know it's still a draft, but...

That's OK, all remarks are welcome, just keep in mind that some paragraphs can be left out or removed in subsequent versions.

I've totally run out of ideas for improvements so I am going to undraft it and ask for feedback.
All remarks are welcome.

There's a couple of green check marks, and Donald's suggestion on wording were applied, and there are no remarks on actual content, so I think it's good to merge.
I will eventually reorganize some sections when something comes up when working on #191 or alike.


v2.0.1 is missing in packagecloud stable and current

I am not sure if this was intentionally pulled, an accidental tag/release version, or it never made it into packagecloud. Nevertheless, It looks like v2.0.1 of the server is missing from packagecloud stable and there does not appear to be any evidence of a 2.0.1 in current either.
@chef/engineering-services

Version 2.0.1 was never released out of current, although there was occasionally a problem with it being reaped by automation.
Version 2.1 is now in stable and should resolve this issue.


RoutingBGP throwing error Cannot read property 'toUpperCase' of undefined

Environment

Declarative Onboarding Version: 1.22.0
BIG-IP Version: 15.1.3 Build 0.0.11 Final

Summary
Been trying to setup CIS automatically using DO but I can't get RoutingBGP to configure, it is throwing error Cannot read property 'toUpperCase' of undefined.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
    "schemaVersion": "1.22.0",
    "class": "Device",
    "async": true,
    "label": "Cloudinit Onboarding",
    "Common": {
        "class": "Tenant",
        "hostname": "hostname",
        "rd0": {
            "class": "RouteDomain",
            "id": 0,
            "routingProtocols": [
                "BGP"
            ]
        },
        "k8scalico": {
            "class": "RoutingBGP",
            "localAS": 64512,
            "peerGroups": [
                {
                    "name": "k8s-group",
                    "remoteAS": 64512
                }
            ],
            "neighbors": [
                {
                    "address": "<IP_ADDRESS>",
                    "peerGroup": "k8s-group"
                },
                {
                    "address": "<IP_ADDRESS>",
                    "peerGroup": "k8s-group"
                },
                {
                    "address": "<IP_ADDRESS>",
                    "peerGroup": "k8s-group"
                }
            ]
        }
    }
}

Observe the following error response:

[
    {
        "id": "000000-0000-0000-0000-000000000000",
        "selfLink": "https://localhost/mgmt/shared/declarative-onboarding/task/000000-0000-0000-0000-000000000000",
        "code": 500,
        "status": "ERROR",
        "message": "invalid config - rolled back",
        "errors": [
            "Cannot read property 'toUpperCase' of undefined",
            "Cannot read property 'toUpperCase' of undefined"
        ],
        "result": {
            "class": "Result",
            "code": 500,
            "status": "ERROR",
            "message": "invalid config - rolled back",
            "errors": [
                "Cannot read property 'toUpperCase' of undefined",
                "Cannot read property 'toUpperCase' of undefined"
            ]
        },
    "declaration": {}
}

Expected Behavior
For BGP to be configured.
Actual Behavior
Throws the above error

Thank you for reporting this. I have confirmed the bug and added it to our internal product backlog as AUTOTOOL-2616.

This issue has been resolved with the 1.23.0 release of Declarative Onboarding.


Add username claim to samlv2 resource

Email claim is supported in fusionauth_idp_saml_v2 but username claim is not. This PR adds the missing variable and associated documentation.
I have not altered the tests.
We are using this patched version of the provider in production.

I've pushed a new commit with the suggested changes.


enable dev.external on machines with podman

related: runtimes/backlog/issues/1011

@AdamBrousseau could you review the change? Thanks


Symbol doesn't work when there is a group inside of a group

Bug report
Expected behavior:
I created a group called "second button" with specific padding, works perfectly. Then I put it into another group and called it "button in table." That group has another background with another padding.
Both together work perfectly. It resizes all the shapes when I change the text. Here's the artboard

Actual behavior:
Then I converted this group into a symbol, but when I changed the text field of the symbol. The resize did not work. Here's the outcome.

The sketch file has the artboard and all the elements
Resize in symbol.sketch.zip

this plugin has a beautiful proposal ... but only serves to create cool posts in the medium =(

@panchovm @gustavaum Sometimes you just need to get creative, it's not that hard to create the output you want. refer the below screenshot and file.

Resize in symbol fixed.sketch.zip

Wonderful! you're awesome. Thank you


Vibrational structure calculations

Summary
Related to #1231
Details and comments
Implementation of methods for vibrational structure calculations

can we add a release note on this so that we can merge it ?

In CHC and UVCC, since num_qubits now defaults to None, can we check in construct _circuit that if the value is None and raise and error (like is done for parameter len check) with a message that num_qubits was not set. This way its a bit more obvious to an end user when they have not done things correctly rather than having to figure things out based on some exception later when it tries to use num qubits when its None.


Delivery address not working

Expected behavior:
Able to order to an address
Actual behavior:
"We couldn't locate the entered address/postcode, please enter a valid address/postcode."

Reproduce steps:
Use same address?

Version:
v3.0.4-beta.9.1
Additional Information:
I use Google Map api, restaurant delivery area is set.

Have you included your Google Maps API key on the settings page?

Yes, I was able to configure Map areas where deliveries are available. ( Didn't work without Google maps api ).


Optimizations and refactoring of testrunner

About this PR

Code of testrunner module has been refactored.

No more iterations through a list of tests again and again after every request. A map of existing tests is generated on app start time instead which simplifies the access by name.
The code has been linted and covered by comments.
All helper functions are private now. Only actions are public.
Nesting level has been decreased where possible.

Layout of test results generated by revel test cmd has been fixed (no horizontal scrolling, code fragments' background added, etc).

:100: Excellent code!


Make it possible to disable IPv6 (fully or only RAs)

Work in progress - not ready yet
This PR is written on top of @mcalmer 's #533 PR
The ipv6 variable is becoming a map:
  type = "map"
  default = {
    enable = true
    accept_ra = true
  }

I hesitated with a tristate, but the map is more open to future extensions (yes, I know, YAGNI...). Feel free to disagree and find the tristate simpler, it's easy to change.
As those settings are not really provider-specific, I ported them to openstack as well.
The results go to /etc/sysctl.d/99-salt.conf:
net.ipv6.conf.lo.accept_ra = 0
net.ipv6.conf.eth0.accept_ra = 0
net.ipv6.conf.eth1.accept_ra = 0
net.ipv6.conf.all.disable_ipv6 = 1

When disabling RAs:

existing dynamic addresses are removed (otherwise, they time out only after a delay that is too long);
we also make sure that the network manager does not destroy our efforts: wicked on SLE, networkd on Ubuntu.

The documentation and examples are still missing

It looks like Travis CI is still testing the initial push...

Works in production for some time now, merging


Remove workshop from resources

Description
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

Well, that didn't work... How is a workshop removed @scottslowe @zchase @shughes26 ? We can't have people register to a workshop that won't be hosted.

@SaraDPH I don't know how a workshop gets removed; I'll have to defer to others. Sorry!

The "unlisted" flag can be set to true, or the whole file can be deleted


kinetic job reconfiguration is failing

Hi,
kinetic jobs reconfiguration task Krel_reconfigure-jobs is failing since Mar 23 due to circular dependecies between roch_bringup, roch_capabilities. This was introduced by #14299. This prevents releasing new packages for kinetic.
Is anybody aware of it?
Cheers,
Jiri

@tfoote is in charge of Kinetic and has received notification email for all of the failing jobs. Please also improve the unit tests in this repo to catch the problem before merging in the future.

The same is obviously the case for Indigo.

Also at issue:
Indigo: https://github.com/ros/rosdistro/pull/14268
Debinc: https://github.com/ros/rosdistro/pull/14306

Kinetic reconfiguring is running, and Indigo is waiting for kinetic to finish to run. I think it should work after kinetic finishes. I'll check in the morning.

Indigo and Kinetic are both reconfiguring successfully now.


Metadata file format changes (function.json)

Putting bindings in a flattened array, and introducing a new "BindingDirection" enum (in/out/inout)

I didn't look at all the function.json files, so if one or two are off I won't catch it. But I looked at the code change and it looks straightforward. :shipit:

Note that we need a matching PR for https://github.com/Azure/azure-webjobs-sdk-templates/tree/master/Templates/BlobTrigger

Yep, I'll send the PR for template updates

We have no "trigger" enum, so we have to regex match "Trigger" to find the trigger for a "function.json"? I don't think this will negatively impact the devExp, but it seems like we'd be doing some hacky things for ux/tooling. We should likely include flags for trigger in the "binding.json" to make things more resilient to things like a spelling mistake.
This got me thinking, so I'm proposing an alternative as a thought experiment. Could we kill direction all together and then the names are just "NewBlob", "GetBlob", "PutBlob"? This seems more universally descriptive. It would support us eventually doing "NewBlobPostEvent" and "NewBlobDeleteEvent". The originally proposed model could theoretically support names as well, but it seems like the "direction" property isn't actually representative of any behavioral changes, it's just a less clear abstraction over specific events. Why not make it very specific what the Event Type is in the config? Let the UX, which helps provide hints towards the "direction" worry about the direction.
tl;dr - Should we just use a more descriptive event type and do away with behavior in the "function.json"?

@christopheranderson Yes, I could go with something along the lines of what you propose.

But I didn't mean to redo this PR. Let's discuss and come up with a plan for later. :)

Yeah, I don't feel strongly about blocking progress on this model, but I wanted to put the idea forward. I think we might be describing things the wrong way the more I've gone back and forth on the "behavior/direction" issue.

I don't understand all the "NewBlobPostEvent" stuff proposed above - probably something best discussed in person :) However, I do think that the direction enum as I have it makes a lot of sense - it's concise and doesn't serve too many purposes. Something like "behavior" that has "in", "out", "inout", "trigger" attempts to do to many things. A trigger is a degenerate case of always "in" - we can even default the direction for triggers so it doesn't have to be specified.

Let's discuss outside of this pull request :)


Add a flag -enable-default-cmo to enable default cross-module-optimization.

So far, the swift-frontend decided by itself if CMO can be enabled. This caused problems when used with an old driver, which doesn't consider CMO.
Now, the driver decides when to use default CMO by passing this flag to swift-frontend.
Also, revert "Add a mechanism to let cross-module-optimization add additional TBD symbols."
It's not needed anymore because CMO does not introduce public symbols when a TBD file is emitted.

@swift-ci smoke test


Add chacha20-ietf cipher - Fixes #131

Adding the ChaCha20 cipher specified RFC 7539
https://shadowsocks.org/en/spec/cipher.html

Sorry for not reading carefully.
Seems like the package from golang sub-repo is not optimized using asm.
I'll run some tests and merge it.

It seems github.com/Yawning/chacha20 has provided standard chacha20 block function with a branch of other variants with a fair performance (comparison here #187).
If you could implement more chacha20 variants, it will be great to merge your code.
But, for now, I prefer #187.
I will keep this open.

The chacha20 repo was developed with the ChaCha20Poly1305 AEAD in mind. Go added an implementation for ChaCha20Poly1305 (but probably won't add a ChaCha20 / XChaCha20/12/8 package) - so I think it's the right time to restart and focus just on ChaCha20 (and its variants).
But currently some other repos rely on the code and I don't want to break their builds - so this can take some time...

Fix in #205


Not possible to define Array of tables (2.3.0)

I'm trying to parse the example from the TOML GitHub:
[[products]]
name = "Hammer"
sku = 738594937

[[products]]

[[products]]
name = "Nail"
sku = 284758393
color = "gray"

however, I get the following exception:
Error: Cannot redefine existing key 'products'.
    at genError (/home/nitrous/code/nodejs/node-opcua-logger/node_modules/toml/lib/compiler.js:33:14)
    at addTableArray (/home/nitrous/code/nodejs/node-opcua-logger/node_modules/toml/lib/compiler.js:131:7)
    at reduce (/home/nitrous/code/nodejs/node-opcua-logger/node_modules/toml/lib/compiler.js:24:9)
    at Object.compile (/home/nitrous/code/nodejs/node-opcua-logger/node_modules/toml/lib/compiler.js:10:10)
    at Object.module.exports.parse (/home/nitrous/code/nodejs/node-opcua-logger/node_modules/toml/index.js:7:21)
    at loadConfig (/home/nitrous/code/nodejs/node-opcua-logger/logger.js:35:15)
    at Object.<anonymous> (/home/nitrous/code/nodejs/node-opcua-logger/logger.js:11:14)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)

For some reason, it does work on http://binarymuse.github.io/toml-node/

It seems that context returned by https://github.com/BinaryMuse/toml-node/blob/master/lib/compiler.js#L119 is an empty object, resulting in the error.

What version of toml-node are you using? I'm unable to reproduce this problem:
> require("./package.json").version
'2.3.0'

> t = fs.readFileSync(process.cwd() + "/test/table_arrays_easy.toml", 'utf8')
'[[products]]\nname = "Hammer"\nsku = 738594937\n\n[[products]]\n\n[[products]]\nname = "Nail"\nsku = 284758393\ncolor = "gray"\n'

> toml.parse(t)
{ products:
   [ { name: 'Hammer', sku: 738594937 },
     {},
     { name: 'Nail', sku: 284758393, color: 'gray' } ] }

In fact, this exact example is included and tested as part of the library's test suite.

You're right, sorry. I should have isolated this specific piece of code before opening an issue.
Apparently, parse() is failing because of another packages that is required in the same file, node-opcua. This piece of code does not work:
var opcua = require("node-opcua")
var toml = require("toml");

var path = require("path").resolve(__dirname, 'config.toml');
var text = require("fs").readFileSync(path, "utf8");

console.log(toml.parse(text));

When you comment the first line, everything works as expected. No idea why this happens.

I believe that this is something specific to your setup; I'm unable to reproduce this problem even with node-opcua installed and required:
$ jq .dependencies package.json
{
  "node-opcua": "0.0.52",
  "toml": "^2.3.0"
}

$ cat config.toml
[[products]]
name = "Hammer"
sku = 738594937

[[products]]

[[products]]
name = "Nail"
sku = 284758393
color = "gray"

$ cat index.js
var opcua = require("node-opcua");
var toml = require("toml");

var path = require("path").resolve(__dirname, 'config.toml');
var text = require("fs").readFileSync(path, "utf8");

console.log(toml.parse(text));

$ node index.js
{ products:
   [ { name: 'Hammer', sku: 738594937 },
     {},
     { name: 'Nail', sku: 284758393, color: 'gray' } ] }

If you can reduce this to a minimal reproducible example (e.g. a small repo or gist I can clone and take a look at) I'm happy to take a closer look.

Believe it or not, but I just copied your entire example to my environment in a new dir, ran nmp install, node index.js and I get the error I initially reported. Commenting var opcua = require("node-opcua"); makes it work.

Just installed 5.8.0, it works now :disappointed:

@coussej Glad you got it working. I'm going to close this out as there's nothing actionable from my end, but please feel free to reopen if you find otherwise!


Prism toolbar

Hi there,
Does prism still use prism toolbar plugin so that you can easily get the 'copy to clipboard' button to appear? I've been trying but it doesn't seem to work.
Regards
M

The following Copy to Clipboard sample should be helpful to anyone wanting to implement this functionality within their Prism code sample.
https://gist.github.com/geoffreymcgill/d118939d1d358d2ed5e8
This sample is adapted from a sample originally posted by Philip Lawrence.
Hope this helps.

@geoffreymcgill Thanks for the link! I was working on something similar.
@LeaVerou Will it make sense to have a plugin that depends on third party tools in the official repo?

Sure, that’s what plugins are for. (of course it’s always better to not depend on 3rd party tools, but we’re not going to reinvent the wheel for no reason — we reinvent the wheel only when it results in shorter/more elegant code :) ).

I'm definitely in need of that, @vkbansal; was going to look at tackling it myself but probably not for a while.

@mAAdhaTTah can you tell what features are commonly required in a toolbar?

I mostly need "copy to clipboard," but leaving it open for hooking into would also be great, as I also will need to add an edit button that will take you to the backend (if possible).

Is this still only solvable with Flash or does the Clipboard API support this now?

AFAIK, still only Flash. And ZeroClipboard (used in the above example) is the reference for this.

@Golmote @mAAdhaTTah @vkbansal Get the plugin from the public repository i have created Prism-toolbar

@santo007 nice!

For this particular use-case only copy to clipboard is needed, and surprisingly (at least to me) support for it seems pretty decent. It’s mostly only unsupported on mobile browsers, so maybe a Flash-free solution is reasonable?

@uranusjr I'm still pretty sure we can't use it to make a "Copy to clipboard" button, but please show me I'm wrong.

Yes, you’re right, I misread the support description. Sorry for the mess-up.

While there is some discussion above of an extensive toolbar, it looks like this issue is mostly about the "Copy to Clipboard" feature: this is also discussed in #280.
I think this a cool feature to pursue, but maybe we should close either this issue or #280 to simplify things?

Alright, let's recap.
For the plugin by Philip Lawrence, see http://dev.misterphilip.com/prism/plugins/toolbar/.
For the repo created by santo007, see https://github.com/santo007/prism-toolbar.
For the "Copy to Clipboard" feature request, see #280.


Shopping list, list - can create a new shopping list with no name

"The name input is required"
Click Create
Click Add without adding a name
no error appears, goes to screen as per screenshot
No shopping list is added though

Now a pass


Multisite commands

Add a command to launch a drush commands on all sites.
Improve the site-install -a command

Ready to review

Merged!


Windows freeze after run emulator

Hi,
mount and half ago I asked a same question on official microsoft holographic forums but everyone has successfully ignore me. I know, that this question out of scope MixedRealityToolkit but maybe you know about this issue.
I launched the basic application on the hololens emulator (only white cube in front of the camera), it was running normally, I tried to move with arrows, but then Windows freeze. I was waiting a few minutes, however I had to restart PC.
PC performance are good enough for emulator.
Motherboards: 	Gigabyte GA-B450M DS3H
CPU: 			AMD Ryzen 5 1600 Six-Core
GUP: 			GeForce GTX 1050 2GB
RAM: 			8 GB
Windows 10 Education 64 bit   version: 1809
Did you meet with same issue ?

Can we get more information on your application? Is it one of the visual studio template holographic apps? Was the application built in unity using the MRTK?
It may be that the keyboard keys you are using don't necessarily equate to the expected emulator behavior ( https://docs.microsoft.com/en-us/windows/mixed-reality/using-the-hololens-emulator ). It looks like to move around in a scene you may need to use w,s,a,d compared to arrow keys:
"...
Walk forward, back, left, and right - Use the W,A,S, and D keys on your keyboard, or the left stick on an Xbox controller.
..."

I also am having problems with this.  Every second time(at least it seems) I deploy to the emulator, my entire cpu freezes at which point I have to manually restart the computer.  This happens whether or not I use the keyboard or any input at all.   Even if I do nothing, it will freeze.  The first time I deploy it runs fine. This happens with the sample holographic app, and with unity apps. It also happens with both emulator 10.0.17763.134 and 10.0.17134.80. Maybe connected to AMD CPUs?
My specs:
CPU: AMD Threadripper 1950x 16-core
GPU: AMD Vega64
Ram: 64GB
Windows 10 Pro 1809

My suspicions about AMD processors being the problem seem to be confirmed.  It looks like Ryzen based processors have a bug running x86 (32 bit) virtual machines. This causes the machine to crash and freeze.  This has been fixed in AGESA <IP_ADDRESS>(bios).  Make sure that your bios is the most recent version.  Even after updating my bios I still would freeze.  It seems that the fix only works when limiting the number of virtual CPUs to 1.  By default the Hololens emulator is set to 4.  You can change this in Hyper-V, in the virtual machine settings. After changing the Hololens VM to use a single core, no more freezes. Hope this helps.
Relevant details in the comments of this article: https://community.amd.com/thread/220392

My suspicions about AMD processors being the problem seem to be confirmed.  It looks like Ryzen based processors have a bug running x86 (32 bit) virtual machines. This causes the machine to crash and freeze.  This has been fixed in AGESA <IP_ADDRESS>(bios).  Make sure that your bios is the most recent version.  Even after updating my bios I still would freeze.  It seems that the fix only works when limiting the number of virtual CPUs to 1.  By default the Hololens emulator is set to 4.
I tried changing the settings in Hyper-V to use only 1 CPU, but when running the emulator from VS it seems to switch it back to 4, thereby causing a crash.  When I launch the emulator from Hyper-V with only 1 CPU -> no crash. Only problem is when I launch from Hyper-V, I can't use the mouse as input and there is no internet.  Now I am trying to figure out a way to stop VS from changing the settings, or to launch with only 1 CPU.
Relevant details in the comments of this article: https://community.amd.com/thread/220392

The configuration file is in C:\Program Files (x86)\Windows Kits\10\Microsoft XDE\10.0.17763.0\SKUs\HDE. (For the latest Emulator, 10.0.17763.0).  The configuration file is xdesku.xml.  In the file, change from ProcessorCount="4" to ProcessorCount="1". This prevents freezing but of course makes the emulator run slower.
In conclusion, make sure your bios are at least AGESA <IP_ADDRESS> or higher, and only run the emulator on one virtual CPU.

Thanks @IkeTheDestroyer for sharing out your solution. I'm getting to this party somewhat delayed, but yeah, anytime you have a full system freeze it's probably something going on at the driver->below. I'm closing this issue out now.


Np 11901

Updated index.html with the custom fields API
Also attached the leo file, a file that can be used with leo editor to edit the documentation as an outline tree (http://leoeditor.com/)

Please don't use leo or anything of the sort. We are going to use standard tools (specifically probably http://swagger.io/swagger-editor/).
There's an > after Get custom fields for an employee
I am not sure what was done here - but it seems like the entire file was changed. Given - this is a huge html file, but I expected only the relevant parts to be changed. As it stands - I can't see how I am going to approve this merge. I suggest taking master again and only changing the relevant places.


Support of python 3.7

I was trying to configure Travis tests with Python version 3.7, after some configurations, I just realized there is a bottom line:

conda.exceptions.UnsatisfiableError: The following specifications were found to be in conflict:

polygon3 -> python[version='>=3.6,<3.7.0a0']
python=3.7

So we either try to move forward without the polygon library or we drop the support of python 3.7 (knowing that for the future this still will have to be solved)

I think we will move on without Polygon,
It is used for your next main task, to compute polygon intersections and polygon surfaces and barycenters, but you have already started writing a substitute for it.
I think tofu will be more self-contained and faster without it.
This should auto-resolve in the coming months, so let's just postpone support to Python 3.7 till then.

Closed in 1.4.1


Fix templating in pull request comparison

Adds test for expected behavior
Closes: https://github.com/go-gitea/gitea/issues/33013

I'm not stuck on details being empty - I just don't know what to put in there. Suggestions welcome :)

I think it's better to fix it in template, but not add empty values to the map. Because if there is no details, the template shouldn't render that part.
{{.Message}}

	{{.Summary}}
	{{if .Details}}{{.Details | SanitizeHTML}}{{end}}

I came up with this to fix it:

Yup, that's even better. Thank you!

I think this could be backport to v1.23


add easydict wrapper to config dict for easy access

Process the config dict to get alternative Python object attributes with EasyDict.

Hi, @ionvision. Thank you for your suggestion.
As I understand, the feature EasyDict provide is make dictionary items accessible with the attributes as well as keys(strings).
I think that feature looks helpful, especially to the base classes containing a lot of access to the config.
However, doing this will add an additional dependency to this project.
Considering this is just a template, I think the dependencies should be kept minimal.
Tell me if there are things I missed. Thank you

Hi @SunQpark , keeping the template minimal is quite reasonable, and thanks again for the good work.


EU speedlimit tweaks; PIN-to-drive toast

changed proportions of the EU (round) speed limit sign - outline width, text size - to hopefully more closely match the main display icon
experimentally resized and moved the round speed limit indicator centrally above the speedo
added sign for no speed limit (Germany) if fused speed limit is reported as 155 km/h
added PIN-to-drive prompt toast

Speed limit changes need testing, please don't merge yet.

Need to check if this will be too busy / crammed on the actual device.
It looks extremely busy above a 0 but there's rarely a 0 there, so i wanna see it in action.
This isn't the final position or size either, just playing with it a bit.
If it doesn't work out, i'll reposition it to where it was, but centered and sized properly (in line with the AP icon, on the right side, but centered vertically a bit better).


Streaming API Support

Support SFDC Streaming API
https://developer.salesforce.com/docs/atlas.en-us.api_streaming.meta/api_streaming/intro_stream.htm

This is a Rest API library.  Support for streaming API should be handled within another framework.


Improve: support to import deals from lotus(Market 1.1)

Get the deal info via the ProposalCid
Change market_type to market_version
Print the deprecated log when receiving deals from lotus(market version 1.1)
Update miner_fid description

This PR said when storage providers have config the market_version=1.2, they can still import the deals which are sent by market_version=1.2(Boost) and market_version=1.1(Lotus)


Add a dry-run option for testing

This PR adds a DRY_RUN_ENABLED constant and environment variable. When set, API requests will just be logged and not run. Closes #9 .
PR #22 does most of the prep work for this, so the changes needed are pretty trivial at this point, and don't require adding any dependencies.

[x] Add a DRY_RUN_ENABLED constant and environment variable
[x] Implement dry-run mode
[x] Make separate settings for read-only (GET) methods and write (POST, PUT, DELETE) methods
[x] Add unit tests
[x] Update docs

I spent some more time on docs and tests for this. I think this is just about done.

@niconoe This is in dev now. Feel free to look it over if/whenever you want to (or not, if you don't have time). See the updated README for usage info.
I probably won't release changes to master for a couple more weeks, so I can still add any suggestions for changes you might have before then.


Custom format equivalent for "o" date time specifier is incorrect

Partway down the page there is a table that specifies the custom format string equivlent of some of the standard date formats. For the "o"/"O" format, it claims that the equivalent custom string is yyyy'-'MM'-'dd'T'HH':'mm':'ss'.'fffffffzz (two z's) however this is incorrect. For DateTime the correct equivalent is yyyy'-'MM'-'dd'T'HH':'mm':'ss'.'fffffffK. For DateTimeOffset the correct is  the same as for DateTime, but can also be equivalently written as yyyy'-'MM'-'dd'T'HH':'mm':'ss'.'fffffffzzz (3 or more z's at the end).
This information is correctly stated lower on the page in prose, but the table remains incorrect.
Proof that the specified equivalent with two zz is wrong can be seen by running the following code with a non-UTC equivalent time zone:
var dt= DateTime.Now;
Console.WriteLine("Standard format:    {0:o}", dt);
Console.WriteLine("Claimed equivalent: {0:yyyy'-'MM'-'dd'T'HH':'mm':'ss'.'fffffffzz}", dt);
Console.WriteLine("Correct equivalent: {0:yyyy'-'MM'-'dd'T'HH':'mm':'ss'.'fffffffK}", dt);

The source code has a comment that clearly specifies the correct format for DateTime:
https://github.com/dotnet/runtime/blob/8a52f1e948b6f22f418817ec1068f07b8dae2aa5/src/libraries/System.Private.CoreLib/src/System/Globalization/DateTimeFormat.cs#L108-L108

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

ID: 52741095-f1fd-71b8-15f8-b526480f0d83
Version Independent ID: 12b2c626-2479-1090-83fd-583c6d600ed8
Content: Standard date and time format strings
Content Source: docs/standard/base-types/standard-date-and-time-format-strings.md
Product: dotnet-fundamentals
GitHub Login: @adegeo
Microsoft Alias: adegeo

@KevinCathcart Thanks for reporting this - it'll be fixed by #22496

@KevinCathcart Thanks for reporting this - it'll be fixed by #22496


fix compile errors with latest Scala 2.13

https://github.com/scala/scala/commit/f5c102e72fbd835bf66377879bdb8be20d882da1

Thank you!


Improve namespace more info descriptions

Docs Build status updates of commit 9e118c4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/ns-AspNetCore.xml
:white_check_mark:Succeeded

aspnet-core/xml/ns-Microsoft.AspNetCore.Antiforgery.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.ApiAuthorization.IdentityServer.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.AzureAD.UI.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.AzureADB2C.UI.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Certificate.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Cookies.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Facebook.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Google.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.JwtBearer.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.MicrosoftAccount.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Negotiate.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OAuth.Claims.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OAuth.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OpenIdConnect.Claims.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OpenIdConnect.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Twitter.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.WsFederation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.Infrastructure.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.Policy.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Builder.Extensions.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Builder.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Authorization.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Forms.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Routing.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Web.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.CookiePolicy.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cors.Infrastructure.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cors.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cryptography.KeyDerivation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.ConfigurationModel.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.EntityFrameworkCore.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.KeyManagement.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.StackExchangeRedis.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Diagnostics.HealthChecks.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Diagnostics.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.HeaderPropagation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.HostFiltering.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.Server.Features.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.Server.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.StaticWebAssets.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.WindowsServices.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Features.Authentication.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Features.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Json.xml
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 9e118c4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/ns-AspNetCore.xml
:white_check_mark:Succeeded

aspnet-core/xml/ns-Microsoft.AspNetCore.Antiforgery.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.ApiAuthorization.IdentityServer.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.AzureAD.UI.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.AzureADB2C.UI.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Certificate.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Cookies.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Facebook.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Google.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.JwtBearer.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.MicrosoftAccount.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Negotiate.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OAuth.Claims.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OAuth.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OpenIdConnect.Claims.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.OpenIdConnect.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.Twitter.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.WsFederation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authentication.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.Infrastructure.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.Policy.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Authorization.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Builder.Extensions.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Builder.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Authorization.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Forms.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Routing.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.Web.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Components.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.CookiePolicy.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cors.Infrastructure.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cors.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Cryptography.KeyDerivation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.ConfigurationModel.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.AuthenticatedEncryption.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.EntityFrameworkCore.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.KeyManagement.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.DataProtection.StackExchangeRedis.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Diagnostics.HealthChecks.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Diagnostics.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.HeaderPropagation.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.HostFiltering.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.Server.Features.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.Server.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.StaticWebAssets.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.WindowsServices.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Hosting.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Features.Authentication.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Features.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Http.Json.xml
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


some files don't exist.

Hi,
i found some files not existing in master branch, such as,
"from .checkpoint import load_checkpoint"

You should install beit first, this repo is developed upon beit.
Or you just need to borrow those files here to your repo.

@MengzhangLI  thanks for your reply, i will try.


update stats endpoint for gpus count

https://gridproxy.dev.grid.tf/stats
should include number of gpus
not sure if should be number of gpus or number of nodes with gpus enabled, given the node can have multiple GPUs
In case of nodes that has gpu enabled it's a simple query, but in case of the exact number of gpus i believe we will have to count all gpus using listGPUs

Verified on Devnet,


entity_synonyms are not recognized

Rasa NLU version: 0.13.8
Operating system (windows, osx, ...): macOS mojave version 10.14
Content of model configuration file:
language: en
pipeline: tensorflow_embedding

Contents of nlu.md file
intent:greet

hey
hello
hi
good morning
good evening
hey there

intent:restaurant_search

i'm looking for a place to eat
I want to grab lunch
I am searching for a dinner spot
food near me
restaurants near me
place near me to eat
i'm looking for a place in the north of town
show me chinese restaurants
show me a mexican place in this area
i am looking for an indian spot
search for restaurants in the center
anywhere in the west
anywhere near 18328
I am looking for asian fusion food
I am looking a restaurant in 29432

synonym:indian

Desi
spicy
Bharath

synonym:center

central
midtown
middle

intent:thankyou

thanks!
thank you
thx
thanks very much
thanks
thankyou very much
thanks a lot
thanks. Have a nice day
thanks. have a good one.
have a good one

intent:mood_great

I am doing great
I am absolutely alright
I am fine
I am happy. How about you
fine
alright
good
going good
doing good
happy
superb
All good. How are you?
great
doing great
I am good
absolutely fine

intent:mood_unhappy

I am not good
I am so sad
I am not happy
I am sick

intent:mood_deny

No it didn't help
I am not satisfied
not okay

Issue:

I have defined synonyms in my nlu.md file. I have trained the model. but when I ran the model with some example message including synonyms in it. model is not recognising those entities.

Please see the docs for this note:

Note
Please note that adding synonyms using the above format does not improve the model’s classification of those entities. Entities must be properly classified before they can be replaced with the synonym value.

Hello @wrathagom Can you please tell me how to properly classify entities before they can be replaced with synonym value? I am very much new to python, Rasa stack. I have been using markdown format to define stories, but recently I changed it to json format. can you review my stories file.
nlu.json
{
"rasa_nlu_data": {
"common_examples": [
{
"text": "hey",
"intent": "greet",
"entities": []
},
{
"text": "howdy",
"intent": "greet",
"entities": []
},
{
"text": "hey there",
"intent": "greet",
"entities": []
},
{
"text": "hello",
"intent": "greet",
"entities": []
},
{
"text": "hi",
"intent": "greet",
"entities": []
},
{
"text": "i'm looking for a place to eat",
"intent": "restaurant_search",
"entities": []
},
{
"text": "i'm looking for a place in the north of town",
"intent": "restaurant_search",
"entities": [
{
"start": 31,
"end": 36,
"value": "north",
"entity": "location"
}
]
},
{
"text": "show me chinese restaurants",
"intent": "restaurant_search",
"entities": [
{
"start": 8,
"end": 15,
"value": "chinese",
"entity": "cuisine"
}
]
},
{
"text": "yes",
"intent": "affirm",
"entities": []
},
{
"text": "yep",
"intent": "affirm",
"entities": []
},
{
"text": "yeah",
"intent": "affirm",
"entities": []
},
{
"text": "show me a mexican place in the centre",
"intent": "restaurant_search",
"entities": [
{
"start": 31,
"end": 37,
"value": "centre",
"entity": "location"
},
{
"start": 10,
"end": 17,
"value": "mexican",
"entity": "cuisine"
}
]
},
{
"text": "bye",
"intent": "goodbye",
"entities": []
},
{
"text": "goodbye",
"intent": "goodbye",
"entities": []
},
{
"text": "good bye",
"intent": "goodbye",
"entities": []
},
{
"text": "stop",
"intent": "goodbye",
"entities": []
},
{
"text": "end",
"intent": "goodbye",
"entities": []
},
{
"text": "i am looking for an indian spot",
"intent": "restaurant_search",
"entities": [
{
"start": 20,
"end": 26,
"value": "indian",
"entity": "cuisine"
}
]
},
{
"text": "search for restaurants",
"intent": "restaurant_search",
"entities": []
},
{
"text": "anywhere in the west",
"intent": "restaurant_search",
"entities": [
{
"start": 16,
"end": 20,
"value": "west",
"entity": "location"
}
]
},
{
"text": "central indian restaurant",
"intent": "restaurant_search",
"entities": [
{
"start": 0,
"end": 7,
"value": "central",
"entity": "location"
},
{
"start": 8,
"end": 14,
"value": "indian",
"entity": "cuisine"
}
]
},
{
"text": "indeed",
"intent": "affirm",
"entities": []
},
{
"text": "that's right",
"intent": "affirm",
"entities": []
},
{
"text": "ok",
"intent": "affirm",
"entities": []
},
{
"text": "great",
"intent": "affirm",
"entities": []
}
],
"entity_synonyms": [
{
"value": "indian",
"synonyms": [
"desi",
"spicy",
"bharath"
]
}
]
}
}


support ts or cjs

@domchristie/turn doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module (i.e. it doesn't have "type": "module" or an .mjs extension for the entry point). Please contact the package author to fix.

https://github.com/domchristie/turn/pull/11


add two numbers

Codecov Report

Merging #4 into master will decrease coverage by 1.86%.
The diff coverage is 43.85%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   46.15%   44.28%   -1.87%
==========================================
  Files           1        2       +1
  Lines          13       70      +57
==========================================
+ Hits            6       31      +25
- Misses          4       32      +28
- Partials        3        7       +4

Impacted Files
Coverage Δ

leetcode/0002_add_two_numbers/add_two_numbers.go
43.85% <43.85%> (ø)

Continue to review full report at Codecov.

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


[1.2.x] GUVNOR-3351: Make validation dialog more readable

See https://issues.jboss.org/browse/GUVNOR-3351
This is the corollary of https://github.com/AppFormer/uberfire/pull/783 for 7.2.x
Branch name is -7.2.x so Jenkins can build related PRs successfully.
(cherry picked from commit c7b0f6279d981f67924f29907f20a16bbdf289c0)

Part of an ensemble:

https://github.com/AppFormer/uberfire/pull/790
https://github.com/kiegroup/kie-wb-common/pull/1012

Jenkins please execute full downstream build.

Jenkins please execute full downstream build.

@mariofusco @mswiderski (anybody!) Is this a known intermittent test failure or does somebody need to take a look?

@manstis I believe it was already fixed yesterday by @mariofusco

@mswiderski Great.. May the CI Gods pass my subsequent request.

Jenkins please execute full downstream build.

Jenkins please execute full downstream build.

Jenkins please execute full downstream build.

Jenkins please execute full downstream build.


✨ feat(staticroutes): adds v2 static routes

This commit introduces support for V2 static routes

:tada: This PR is included in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Null Object Reference?

Describe your bug here.
I was loading up one of my psych engine modpack that's Tails Gets Trolled and for some reason on 0.6.1, when trying to play No Villians, there was an error that crashes my game. Do you know what happened? If you need any other data I'll post as soon as I can. (Edit: I got 0.6.2 to see if it would work on a newer version but the same error screen popped up)
(Image of the error)
https://user-images.githubusercontent.com/103972070/190881066-bead61bb-68f1-4875-96c3-131a2c20c38b.jpg
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I never touched the source code, this was all done through the mods folder

You are missing one or multiple files, this is often the reason of Null Object Reference

You were right. I was missing a custom note asset and deleted it from the JSON and the song's working. Thank you!


Provide unit tests

This project can become crucial and we should try to test it as good as we can.
Testing the project is not so is easy though, because:

the sidecar is moving files around in the filesystem in a couple of places,
uploading stuff to cloud providers,
and requiring real databases.

For the filesystem One possible solution would be to introduce a file system abstraction, which is mockable, like https://github.com/spf13/afero.
For the cloud provider testing one possible way would be to create mocks from the cloud provider's storage interface, which would at least give a bit of confidence but still is not ideal.

This could be working for GCS buckets, needs evaulation: https://github.com/googleapis/google-cloud-go-testing/blob/master/storage/stiface/examples_test.go


add support for kind clusters

What

Sets KUBECONFIG to a direnv-managed file.

Adds a couple of scripts to make developement with kind easier:

scripts/kind-up.sh
Start a kind cluster with a local registry

scripts/kind-down.sh
Delete the kind cluster and its registry

Exposes the REGISTRY as an option to the Makefile and defaults it
to localhost:5001, which is the one used by the kind cluster.

The make deploy job now has a dependency on docker-push to get the
image into the kind registry.

Sets the image pull policy to "Always" when deploying to ensure that
kind pulls the latest image.

Breaking Changes
Might break some non-kind workflows with the "push" requirement and
defaulting to localhost:5001 for the registry.

Current dependencies on/for this PR:

main

PR #275   👈

This comment was auto-generated by Graphite.


Functional Drupal Site running on Bluemix

Completed on October 13th. Moving from tarball to Composer deploy we'll have to redo.

Composer version works.


Add getData function for DMatrix

The python API has a get_data function, add a similar api for jvm-package too.

cc @wbo4958 .

I will check it today. sorry for late

some minor comments, Overall, LGTM.


🐛 [OFFICIAL] Generated Issue: ParserException

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The API has returned an error:
Jikan\Exception\ParserException
Status code: 0
Message: Failed to parse 'https://myanimelist.net/anime/53888/_/episode/1'
Trace: /var/www/jikan-rest/vendor/jikan-me/jikan/src/Exception/ParserException.php on line 24
Expected Behavior
The API should have returned a successful response with data.
Steps To Reproduce
Http Request: `GET /v4/anime/53888/episodes/1
Environment
No response
Anything else?
No response

Fixed with release v4.0.12.
Note: On the public REST API, please wait 24-48 hours for any existing cache to be replaced.


Skip version update

Maybe this features should be more like a "Skip this version" if I decide this update isn’t worth installing instead of entirely disabling auto-update?

Currently it is possible to ignore a update for until the app is restarted or the user manually lets Alva check for a update. Is that what should have been done with this issue?

I meant it in a way that the version should be ignored all the time. So like Sketch e.g.:

Done in latest canary release


Test the plugin for pook separately

coverage: 99.061% (+0.001%) from 99.06%
when pulling a934e290104aa4cc27e868172e19cd0630162402 on test-pook-separately
into e7e8172a5824b3a44f2e793f6f374e5efc5eb563 on main.


GCC Version On CUDA 8.0

Hi!
So the max version that this package's build script will allow for gcc on CUDA 8.0 is 5.3.1.  However, even Xenial has moved on with gcc-5 up to 5.4.0 at least.  https://packages.ubuntu.com/xenial/gcc-5  This makes it impossible to build on anything newer than a slightly out of date 16.04 machine without hacking the build.jl.  I use 17.04, so for me it's just flat out ridiculous to get anything older than 5.4.1 (which works fine with CUDA).
I propose bumping the build.jl requirement from 5.3.1 to 5.4.1.  Thanks!

This seems to be fixed on master already. @maleadt tag a new version?

@Keno Good catch, I hadn't actually checked master, just the build script coming from Julia's install.  A new tag would be much appreciated.

Yeah, sorry, forgot about tagging a new version. There's been plenty changes in CUDAdrv though, so I'll first do a more thorough check later today.


viewBox calculation mixed up

Hi!
I get users complaining that the SVGs I generate with Boxes.py (https://github.com/florianfesti/boxes) cannot be rendered with Laserweb. A quick look into the code raised the suspicion that the viewBox calculation may be off. Especially those two lines in tagparser.js seem to be in the wrong order. Swapping them out at least shows something:
this.tag.scale(scaleX, scaleY)
this.tag.translate(-translateX, -translateY)

Test with any SVG with has a viewBox not starting at 0 0 - like all Boxes.py artifacts.
Sorry, for not submitting a proper PR.

Is PR going to help with this or should it be filed directly against LaserWeb4?


[QUESTION] How to get sheet name on importing file with multiple sheets

Prerequisites
None
Versions

PHP version:
PHP 7.2.17 (cli) (built: Apr 19 2019 00:49:48) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
with Xdebug v2.7.1, Copyright (c) 2002-2019, by Derick Rethans
with Zend OPcache v7.2.17, Copyright (c) 1999-2018, by Zend Technologies
Laravel version:
Laravel Framework 5.8.8
Package version: "maatwebsite/excel": "^3.1"

Description
I have been following docs, and i must import a xlsx file with multiple sheets. Each of sheet names are represent of Region with certain data in it.
I need to grab a name of each sheet, so i can save it in Database as region name.
I need to import content of each Sheet in db too.
I have created Import classes from documentation found here:
https://docs.laravel-excel.com/3.1/imports/multiple-sheets.html#conditional-sheet-loading
Have been looking around to find the way to get sheet name (title) but did not find any.
How i can get sheet name? Is there any example you might provide?
Additional Information
No additional info.

You could use a BeforeSheet event and grabbing the title there (PhpSpreadsheet method), storing it in a class property and then using it in your import code perhaps.

Thanks @patrickbrouwers . Looks like BeforeSheet is used for Export. Cant figure it out how to get sheet name using it.

Thanks @patrickbrouwers . Looks like BeforeSheet is used for Export. Cant figure it out how to get sheet name using it.

https://docs.laravel-excel.com/3.1/imports/extending.html
The PhpSpreadsheet method is getTitle

Closing due to resolved/inactivity. Please feel free to reopen (and fill in the issue template) if you need additional support.


Accessibility - Dec into Counselor

Not yet ready, don't look, don't approve, will update. (EDIT: Ready now; see below.)

UPDATE: This failed in my testing. It is because we moved the assistive text from ApplicationFormSection to ApplicationFormSectionInput today. ApplicationFormSectionInput doesn't know what the question is, so it cannot tell us whether question.assistiveText is blank or not -- and fails with an error. I am putting everything back where it was as of this morning (since that configuration has been tested and has been proved to work in at least most cases) and we can discuss whether to proceed with it as-is or make further changes tomorrow. I don't want this to derail the whole release, though. (Having some default assistive text in Prod would be better than our current none.)

@jjheaney , opps, didn't see your not ready yet statement. My bad on preemptive commenting.

@jjheaney , opps, didn't see your not ready yet statement. My bad on preemptive commenting.

The "not ready" comment is obsolete and I edited it accordingly last night. Your comments were not preemptive and were exactly what I needed to move forward.


use a json parser to avoid serialization/parsing errors:

Look into: DataContractJsonSerializer here is an example:
https://msdn.microsoft.com/en-us/library/hh674188.aspx

Fixed issue
Used Json.NET :
http://www.newtonsoft.com/json

used using Newtonsoft.Json.Linq;
Changed headers and body -> JObjects

Changed tests and examples


Do not install unnecessary dependencies

Fixes https://github.com/Lightning-AI/lit-gpt/issues/323
Fixes https://github.com/Lightning-AI/lit-gpt/issues/278
Most of our dependencies are optional. Do not include them in our requirements.txt

Don't know, maybe this question was asked before, but why does this repo uses only requirements.txt and not poetry. There is in-build functionality to separate essential and optional packages.
Given that PyTorch-Lightning uses poetry I assume that there should be a particular reason to not use it here. Maybe for simplicity?

Yes, it's for simplicity


Adjustment in trade calculation

We can verify that the sales calculations on trades are not performed correctly.
Since a smaller number is sold than the buy, I believe it should be calculated on the basis of the sale. Furthermore, in the case of manual sales, it is calculated as 100% loss, which is also considered wrong, as sometimes the sale is made at a profit.

@chopeta
Please open a new issue with your suggestion. It's hard to catch up if you put in another issue.
@philippirsl
If you don't explain further, I will close the issue.

@chopeta
Please open a new issue with your suggestion. It's hard to catch up if you put in another issue.

Done!

@philippirsl Since I don't have any further infromation from you, I will close the issue.


Add files via upload

I am not sure if I understand what you mean when you say rebase your code on the master. Did I do it?

No you didn't, and you can't just upload new files every time. You need to:

Check out oxy-compsci/master
Make a branch from oxy-compsci master (in your fork)
Make your changes
Push that branch and start a PR.

I followed the instructions and I submitted it again.
On Wed, May 23, 2018 at 11:12 AM, Justin Li<EMAIL_ADDRESS>wrote:

No you didn't, and you can't just upload new files every time. You need to:

Check out oxy-compsci/master
Make a branch from oxy-compsci master (in your fork)
Make your changes
Push that branch and start a PR.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/oxy-compsci/brick-editor/pull/23#issuecomment-391446072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AltOVRbNhvbjOh6qUE6tfVTe6mP3oz3Sks5t1aaNgaJpZM4UJdGV
.


dfu/boot/mcuboot: treat pristine image as confirmed

In case of initial/pre-programed image.
Such image can neither be reverted nor physically confirmed.
Image like this should be recognized as confirmed by the boot_is_img_confirmed() for consistency.
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

@nordicjm @de-nordic Can you revisit?

I tried to fw update using canopen. I am getting stucked in same function boot_is_img_confirmed. If I open canopen_program.c with function called  CO_SDO_abortCode_t canopen_program_cmd_stop(void) which stops program that is running, but before stopping it checks if if (canopen_program_get_status() == PROGRAM_CTRL_ZEPHYR_CONFIRM). If we check canopen_program_get_status function which look like this
static uint32_t canopen_program_get_status(void)
{
/*
* Non-confirmed boot image takes precedence over other
* status. This must be checked on every invocation since the
* app may be using other means of confirming the image.
*/
if (!boot_is_img_confirmed()) {
return PROGRAM_CTRL_ZEPHYR_CONFIRM;
}
return ctx.program_status;

}
I can see that it returns PROGRAM_CTRL_ZEPHYR_CONFIRM; even if i use the code that you fixed.
What should i do, any idea here?

I copied code provided here https://github.com/mcu-tools/mcuboot/issues/1066 and tried to see if function boot_is?img_confirmed confirms image with a fixed code in this function, but still does not work for me.


Getting started guide for Azure?

Related to kubeflow/kubeflow#240
We currently have getting started instructions for GKE & and minikube
https://github.com/kubeflow/website/blob/master/content/docs/getting-started-gke.md
https://github.com/kubeflow/website/blob/master/content/docs/getting-started-minikube.md
Do we want to add a getting started guide for Azure?
/cc @wbuchwalter @lachie83

There is a lot of documentation on this page for Azure.
https://github.com/Azure/kubeflow-labs
it would be great to port all or some of that over to kubeflow.org
@ritazh per our conversation at Kubecon we'd love to get docs for Azure up on www.kubeflow.org
If you or someone else is willing to take this on that would be great.
/cc @aronchick

@carmine signed. sorry for the delay.

@ritazh - Thx

@ritazh You need to join the kubeflow org before we can assign issues to you.
More info here:
https://www.kubeflow.org/docs/about/contributing/#joining-the-kubeflow-github-org

I've added this issue as a potential one to tackle during the upcoming Kubeflow Doc Sprint. Stay tuned for info about the doc sprint!

/cc @aronchick

Hi! There is an intern team here at Microsoft (consisting of myself, Maanav Dalal, and Mohona Ahmed) that is working to get these docs started if you would like to assign us this issue.

We also plan to attend the doc sprint in a few weeks to continue work on this issue.

/assign
Thanks @rem20806! I've assigned this issue to myself, so people know someone is working on it. Thanks to all three of you for taking it on. And it will be great to see you at the doc sprint too!

This is done.
/close


[BUG] Tracing: HTTP spans should cover time-to-last byte and fail if stream reading fails.

HTTP spans today end when response is received by the policy. By that time response could have been buffered and read to the end or not read at all.
As a result, if stream reading fails ofter response code is received, HTTP spans look successful, but in fact there are tons of retries
Spans only:

Logs tell a real story:

Fixed in #38013


FocusModeBash Update to console live expressions

Changes for Console Live Expressions.
While I updated the screenshot and did some text updates for inaccuracies. I think this top section for Live Expressions needs to be re-written. It just does the same thing 3 times with I think little added value. It is also very complex and brittle relying on multiple demo websites as part of the walkthrough flow.

Docs Build status updates of commit 6a13033:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/console/live-expressions.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expression-mouse-logging.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-document-active-element.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-document-active-element-nav-button.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-document-active-element-search.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-new.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-remove.msft.png
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/media/console-live-expressions-x-and-y.msft.png
:white_check_mark:Succeeded
View

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


push through cats 1.0 upgrade everywhere

a number of breaking changes were made between 1.0.0-MF (which is about where we are currently) and 1.0.0-RC1
I don't want to attempt this too early, since I'll likely just hit projects that haven't upgraded yet, but most of those projects will of their own accord if I just wait a little longer, I don't want to be in the business of submitting upgrade PRs myself. so for now I'm preferring to freeze things

todo list:

[ ] unfreeze circe (PR: #651)
[ ] ...?


Removed the UI overflow on main page and Wallet Connect page

Removed the UI overflow to make the ui more responsive.

Here is the screenshot of page after the edit.

@jaibhedia I have solved the issue #35 . Please review.

@a-m-a-nkumar Thank you for the PR, I have seen you have made some other changes as well, can you just document it.

@jaibhedia should i share a document listing all the changes or just mention all my work here in the comment section??

Sure.
On Sat, Jun 1, 2024, 12:34 PM a-m-a-nkumar @.***> wrote:

@jaibhedia https://github.com/jaibhedia should i share a document
listing all the changes or just mention all my work here in the comment
section??
—
Reply to this email directly, view it on GitHub
https://github.com/jaibhedia/ChainzRegistry/pull/41#issuecomment-2143333491,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANZY55NCZD3ZALTYAABCXE3ZFFXBFAVCNFSM6AAAAABIIBAMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGMZTGNBZGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

You can mention it here in the comments as well, no issues.
On Sat, Jun 1, 2024, 12:35 PM SS @.***> wrote:

Sure.
On Sat, Jun 1, 2024, 12:34 PM a-m-a-nkumar @.***>
wrote:

@jaibhedia https://github.com/jaibhedia should i share a document
listing all the changes or just mention all my work here in the comment
section??
—
Reply to this email directly, view it on GitHub
https://github.com/jaibhedia/ChainzRegistry/pull/41#issuecomment-2143333491,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANZY55NCZD3ZALTYAABCXE3ZFFXBFAVCNFSM6AAAAABIIBAMVCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGMZTGNBZGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Home Page UI Overflow Fix:
Problem: The Home page had a UI overflow issue on the right side, causing a poor user experience.
Solution: To address this, I used a SingleChildScrollView widget with horizontal scrolling enabled. This allows the content to be properly contained and scrollable horizontally, eliminating the overflow issue. I also made changes to the header section of the home page.
Login Section UI Improvement:
Problem: The Login section had an issue where the UI overflow caused the Metamask login button to be hidden behind an error message. This made it difficult for users to access the login functionality.
Solution: To resolve this, I removed the error message that was obstructing the Metamask login button. This adjustment makes the UI more responsive and ensures that the login button is easily accessible.
Files Modified:
lib/screens/home_page.dart
lib/screens/wallet_connect.dart
lib/widget/header.dart
lib/widget/homePageDecription.dart

Other modifications include updating and including some packages.


File rename or move not done through source control provider

Versions:

VSCode Version:
Version: 1.25.0
Commit: 0f080e5267e829de46638128001aeb7ca2d6d50e
Date: 2018-07-05T13:11:58.697Z
Electron: 1.7.12
Chrome: 58.0.3029.110
Node.js: 7.9.0
V8: 5.8.283.38
Architecture: x64
OS Version: Windows 7 - 64 bit

Steps to Reproduce:

Open a project having a source control like git
Ensure this project have at least one file commited in the history
Open the file explorer side panel
Rename the file
Open the source control side panel

Result:
The file appears as deleted + added, instead of being renamed.
Any file under source control should be renamed or moved through the source control provider.
For example doing a git mv oldname newname shows the file has been correctly renamed under the source control side panel. This allows the source control provider can to track the file changes correctly instead of breaking the history.
Similarly, when renaming a file in the integrated shell through a source control provider, for example git mv oldname newname, the editor does not track the change correctly and if the file was opened, it shows as the file as deleted from disk, rather than using the new name.
Let me know you prefer a separate issue for this one.
Does this issue occur when all extensions are disabled?: Yes

https://github.com/Microsoft/vscode/issues/53449


Updated testdata docker base image to bookworm

[X ] All tests have passed. If this feature is not already covered by the tests, new tests have been added.
[ X] The pull request is targeting the dev branch.
[ X] The code has been validated to compile successfully by running go vet ./....
[X ] The code has been formatted properly using go fmt ./....

Updating from old Buster Debian release to Bookworm to make sure tests are done on relevant systems.

I have read the CLA Document and I hereby sign the CLA

recheck


I cannot find PagerTabStrip

On Android programming i loved PagerTabStrip for many ViewPager pages.
But on flutter i can't find anything which is like this.
Can someone implement this?
My use case is an endlress PageView with title in tab style.

Do you mean TabBar?
https://master-api.flutter.dev/flutter/material/TabBar-class.html

No, i mean PagerTabStrip
https://developer.android.com/reference/android/support/v4/view/PagerTabStrip

I don't understand what a PagerTabStrip is as opposed to a TabBar.

(When I search Google Images for "PagerTabStrip" I get pictures of tab bars.)


Improve linkifying branches by using repo name from fork URL

You can change the name of a repo that's a fork of another repo. If you do this, the current implementation of linkifyBranchRefs() won't work because the repoName isn't the same for both repos. This is at best a corner case, and I doubt anyone would have an issue but ¯_(ツ)_/¯
This update grabs the fork repo name from the URL of the first commit and uses that if it's not the same user as the current URL.
I changed my fork's repo name so you can test in this PR.

Proof:

Good catch. I didn't think of that use-case.

All fixed with one adjustment: .attr('href') gets the literal href value, so it's the second value in the array. (.href returns the full URL the href would resolve to.)
I like the ternary in the const branchRepo definition. 💯

Thanks :)


As a FA, I want to be able to see all my meetings for today

so i can have a good overview of what i have to do for today

Duplicate of #10


Minor formatting changes

For PR https://github.com/reactjs/reactjs.org/pull/4927

Couple of questions about this:

Should the anchors be long and sentence-like, or short? They should be as persistent as possible as long as we're covering the same basic idea, even if we reword the headings or something.
Should we document key as a prop? It's technically not a prop, plus it can be used with any element or component, and doesn't have any special meaning for Fragment in particular.

Maybe, but this doc shouldn't be the first to break from the others, which use the long form.
True, maybe this could be "attribute" but is it's worth mentioning that it accepts a key passed to it (especially since the shorthand notation doesn't)? If/when we have docs for "all the special attributes components can take" maybe we could link to that?


ERROR: LoadError: Method overwriting is not permitted during Module precompile.

In precompilation, on Julia nightly
WARNING: Method definition show(IO, Core.Compiler.InferenceState) in module Base at show.jl:2827 overwritten in module JET at /home/runner/.julia/packages/JET/Foy4c/src/JET.jl:374.
ERROR: LoadError: Method overwriting is not permitted during Module precompile.
Stacktrace:
 [1] top-level scope
   @ ~/.julia/packages/JET/Foy4c/src/JET.jl:374
 [2] include
   @ Base ./Base.jl:497 [inlined]
 [3] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt128}}, source::String)
   @ Base ./loading.jl:2220
 [4] top-level scope
   @ stdin:3
in expression starting at /home/runner/.julia/packages/JET/Foy4c/src/JET.jl:1
in expression starting at stdin:3

Thanks for your report! Will be fixed in 0.8.14.


In SA-1B, how are polygons with holes represented in annotation?

For example,

I note that polygon with holes isn't exactly supported in COCO JSON format/RLE format: https://github.com/cocodataset/cocoapi/issues/153

Please see here for descriptions of COCO RLE format.

Ah right okay, RLE encoding encodes from binary masks directly instead of from polygons. Thanks!

RectLabel is an offline image annotation tool for object detection and segmentation.
Although this is not an open source program, you can drag & drop the COCO RLE JSON file of the SA-1B dataset to the labels table to import mask annotations.
Segment Anything 1 Billion (SA-1B) is a dataset designed for training general-purpose object segmentation models from open world images.

Improved "Create polygon using SAM" feature so that you can label pixels using the pixels option. You can label Segment Anything 1 Billion (SA-1B) like dataset by yourself.

Hi, I have build this mask exporter that also preserves the holes when exporting, using the RLE format. Have a look if you wish.


About dataset

Should I follow the following steps to preprocess my dataset with images of varying sizes:

Resize the image to the same size, such as 512 * 512

After resizing, input the image into the annotator to generate control img

Save the resized image and control img as training data

thanks

yes, you can do that but resizing everything to square squishes aspect ratios, see here https://civitai.com/articles/2078/play-in-control-controlnet-training-setup-guide#heading-35441

I have read the fantastic document you wrote, but my English is not very good. After using translation software to translate some sentences, I couldn't quite understand them. After reading it, I thought that for all images of different sizes, we should first resize the shorter side to 512, synchronize the longer side to another size, and then crop a 512 * 512 image in the center as the result. Am I right in understanding this way

yes

yes. at least, it's the most foolproof way and will give good results.
you can also crop and resize it to 512x512 anywhere you want, if you have more information about your image dataset. for example if you work with facial images and know where the faces are, you might want to crop around the faces instead of the middle.
ControlNet is also able to handle images of size 512x(n*64) but you have to look that up in your training script.

i understand, thank you very much

yes. at least, it's the most foolproof way and will give good results.
you can also crop and resize it to 512x512 anywhere you want, if you have more information about your image dataset. for example if you work with facial images and know where the faces are, you might want to crop around the faces instead of the middle.
ControlNet is also able to handle images of size 512x(n*64) but you have to look that up in your training script.

My original image size is 256x128,it is about a pedestrian and i need whole image,i plan to resize it to 512x256,should i change something or somewhere in the config.yaml file,what is the meaning of the params image_size:64 ,and i didn't see the n of n*64


Pypi Version

The current version on pypi supports Django<1.10
But the latest on github supports Django<2
https://github.com/divio/djangocms-column/blob/master/setup.py#L9
It seems the pypi version needs to be updated

I am running into an issue now related to this as pip installing this package after installing django-cms in a new project (as instructed in following django-cms "How to install django CMS by hand") downgrades the Django version installed with django-cms and raises an exception:
AttributeError: 'WSGIRequest' object has no attribute 'user'

I would really appreciate if you create and upload a new version.

+1 see also #32

+1

+1

+1

At the moment I am getting the following when attempting to install using the "djangocms" command
Creating the project
Please wait while I install dependencies
If I am stuck for a long time, please check for connectivity / PyPi issues
djangocms-column 1.7.0 has requirement django<1.10,>=1.8, but you'll have django 1.11.13 which is incompatible.
Dependencies installed

As far as I can tell the package is no longer installing. I'm guessing I may need to take this up with djangocms themselves as there is a broken dependency.

looks like this takes some time ;-)

I've used it once, and it taught me a lot about using half maintained 3rd party code...I mean, nothing against divio, this ist just how open source works, and the way it goes, sometimes.

Can you please push out a 1.7.1 with the fix to pypi?  Thanks!

While we're waiting for the update to pypi here is a workaround I found -  we can use pipenv to install git repos, similar to how it's done with pip:
pipenv install git+https://github.com/divio/djangocms-column.git#egg=djangocms-column
The only issue is that its dependencies will not be resolved but I believe this is not an issue as it has no dependencies.

1.8.0 released to PyPi

Thanks @czpython!


Add failWith combinator.

This is just to do something slightly less
noisy than counterexample ....

:8ball:


Nav bar color changes to whites on going back to stream/topic narrow from stream/topic info screen.

Just checked will be fixed by #2748.
@zulipbot claim

Like this after coming back from info screen or topic list screen, color has became white

Got the confirm reproduction:
Steps: (Make sure you are debugging in chrome, I mean inspector is open)

Open stream/topic narrow.
Click on any info button.
Wait until any event is arrived, it can be INIT_TOPIC, presence event any event which changes state.
Press back button (I mean go back).
Observe that BG color has changed to white and text color has also changed.

On the other side, if there are no events (described in step 3) and you go back, colors remains same.


add .gpg url (match github behaviour)

Fix #3049
Note this is a WIP and I haven't actually tested that this works. Once I test it I will remove this message and mark as ready to review.

Status update: I'm running into problems getting go to generate the base64 armor export of the entire keyring in pure go. I need to think some more and read some more documentation.

I have tested furthermore and we don't have enough data in the database to reconstruct the original GPG key as we only store the public key and not the signature and also other information like linked identities. The simple way to fix this would be to store the original import to a new column or table after that it should be easy to use them to join them in a single export.

Closing in support for @sapk's PR