Improvements: Custom DropdownButton Icon

If it is not too much to ask, can we have an extra IconData icon parameter in the DropdownButton so we can have a custom icon for our own needs instead of the material down caret.

I agree... We definitely need this. I'm working with a designer right now who is less than pleased that we're stuck with the default material icon. Flutter has been amazing to work with so far, but this feature is desperately needed.
If anyone knows of a work-around I'd appreciate it if you'd share.


error handling for unmarshalling

this creates new value and overwrites by value-copy struct stored at pointer
func (t *Status) UnmarshalJSON(b []byte) error {
	*t = Parse(string(b))
	return nil
}

this is concerning:

~does this even work? usually if you have struct at pointer, memory is already allocated by Unmarshal package. here normally you would override internal field instead.~ actually it is ok to override whole struct. it is small anyways. very convenient too, as it works for any types.
~allocating enum size twice. Parse creates new enum json.Unmarshal creates enum. you can save space if you move Parse into this this code.~ for small enums like struct{uint8} it is ok to allocate twice.
~coverting bytes to string may cause overhead, since string is immutable, and bytes are mutable. Go would have to make copy of data stored at bytes. access bytes directly. Benchmakrs should show that.~ efficient map lookups need string.
lastly, clearly sometimes you can't unmarshal enum. would be much better if we did not silently fail (which is root of so many problems). would be good if we error out if value is not valid. (or control this via gencode flag)

Yeah - tbh I like the idea of failing on an invalid enum parse - definitely should be a generation flag.

I would also like to have that. Fallbacking on the invalid value is probably nice in some cases, but in my case I want the unmarshalling to fail.
In general, I understand the need for the invalid value to account for the "0" value, but in practice it feels a bit alien to work with. Instead of a generator flag, could we generate 2 types of enums? The first one with a possible invalid value, which doesn't fail on parsing and fallback to the invalid value, and a second one without an invalid value that is more strict and that would fail on parsing/unmarshalling.
type discountType int
//no invalid value, generates a strict enum
const (
	percentage discountType = iota + 1
	amount
	giveaway
)

(I'm getting a panic when trying this btw)

@matdurand this was failing due to the non 0 indexed enum - added to test cases and have release v0.2.7 with the fix.

This has been added @matdurand there is now the -f/-failfast  flag for enabling error on Json unmarshall to a Invalid enum.

I would also like to have that. Fallbacking on the invalid value is probably nice in some cases, but in my case I want the unmarshalling to fail.
In general, I understand the need for the invalid value to account for the "0" value, but in practice it feels a bit alien to work with. Instead of a generator flag, could we generate 2 types of enums? The first one with a possible invalid value, which doesn't fail on parsing and fallback to the invalid value, and a second one without an invalid value that is more strict and that would fail on parsing/unmarshalling.

You could use goenums to generate your 2 enum types and with embedding and the Unmarshal interface have a wrapper struct

type Combo struct {
	EnumWithInvalid
	StrictEnum
}

func (c *Combo) Unmarshal(b []byte) error {
	var e EnumWithInvalid
	err := e.Unmarshal(b)
	if err != nil {
		return err
	}
	c.EnumWithInvalid = e
	var s StrictEnum
	err = s.Unmarshal(b)
	if err != nil {
		return err
	}
	return nil
}

Does that work for your use case? This will auto update when the enums are regenerated so it is a one of cost..

Not sure why I would need this Combo wrapper to be honest. Maybe I didn't explain myself enough. My original comment was about 2 things:

Making sure that the JSON unmarshalling could fail on invalid value, which is now taken care of by the new failfast flag.
Making sure any other potential call to ParseXXX would also fail for non JSON case, which is still a problem.

Why not adjust the failfast to make sure ParseXXX returns a value + potential error? Since UnmarshalJSON calls ParseXXX, it would kill 2 birds with one stone ... But that would mean changing the signature of the Parse method to now return an value+error.
Basically my main point was that in most cases I don't think people would care much about the "invalid" value and would rather receive an error instead of a value which they then need to compare to see if it's valid or not. Using the error pattern seems more golang idiomatic in that regard.

@matdurand @nikolaydubina
See changes made in v0.3.0 I have modified the ParseXXX methods to return an error.

Amazing! Thank you


Support embedded bitmaps in wxRuby code generation

Description:

wxRuby3 doesn't have anything equivalent to PyEmbeddedImage which made it easy to embed images into wxPython. We could probably do something similar using base64 encoding:
require 'base64'

my_image_data = <<~BASE64
  SGVsbG8sIHdvcmxkIQ==
BASE64

my_image = Base64.decode64(my_image_data)

For SVG images, we could treat it similar to how we do it in Python -- first use zlib to compress the string, then store it in the file as Base64 and finally using zlib.decompress(Base64.decode64(...))

Another possible option would be to create a class, similar to the following:
require 'base64'

class RubyEmbeddedImage
  def initialize(data, isBase64=true)
    @data = data
    @isBase64 = isBase64
  end

  def get_data
    data = @data
    if @isBase64
      data = Base64.decode64(@data)
    end
    data
  end

  def get_image
    stream = StringIO.new(get_data)
    Wx::Image.new(stream)
  end
end

my_image_data = <<~BASE64
  SGVsbG8sIHdvcmxkIQ==
BASE64

my_image = RubyEmbeddedImage(my_image_data).get_image()

I haven't tried any of this, so it would probably need some tweaking, but in theory something like this should work. This would be in the images.rb file we generate so that all the app's images could potentially be in one place.

The Image list has been implemented in Ruby. SVG files are also now supported via zip compression/decompression.


Feature Request: Relief Valves ("lossy" backpressure)

I've been implementing a lot of bespoke pumps in a recent project, and this library finally provided me a metaphor for what we've been doing! I would love to be able to use this library for our use case, it feels like the perfect abstraction.
Simplifying our use case, we have a pipeline that maps type A to type C through A -> B -> C. It is vitally important that all As are processed, whereas we only really care about the most recently produced Bs, and would rather throw away old Bs when the mapping to C can't keep up.
This desired "lossy" behavior of ours can be achieved by replacing the mpsc inside the pump with a broadcast. I am sure this is a lot easier to describe than actually implement.
I recognize that I could probably just implement this myself as a custom pump that I attach to the pipeline, but it would be nice to have good ergonomics.
Example proposed API usage (you may probably think of something better):
let pipeline = pumps::Pipeline::from_stream(input)
    .map(a_to_b, Concurrency::serial()
        .backpressure(8)
        .relief_valve(ReliefValve::KeepNewest))
    .map(b_to_c, Concurrency::serial())
    .build();

Alternatively, to simplify implementation, relief valves could perhaps be added as a separate stage in the pipeline?
Example proposed API usage:
let pipeline = pumps::Pipeline::from_stream(input)
    .map(a_to_b, Concurrency::serial())
    .relief_valve(ReliefValve::keep_newest(8))
    .map(b_to_c, Concurrency::serial())
    .build();

If you're open to it, I would be willing to look at implementing this and making a PR over the weekend. Let me know if you think this would be a valuable addition, and which style of configuration you might prefer.

Hi @Dryvnt, glad to hear you find this crate useful! I've been thinking on similar feature myself. This reminds me of JS lodash throttle and debounce. The way I think about this is that what I called Concurrency should actually be a more complex and customizable entity. Hopefully I'll work on this soon

Great. I'll let you get around to it at your own pace, then. Let me know if you'd like any help or feedback.
Regarding the potential API of a "batteries included" relief valve: I've tried to map out our use case with this library, and I discovered an important detail that might inform the API design choices. I'll let you do with the information as you wish.
Our A -> B and B -> C stages cannot be represented as simple mapping functions. Every A can turn into multiple B, and it may take multiple B to create a single C. That's fine, I was expecting to have to create custom pumps for these anyway for a variety of reasons (e.g. we're calling a blocking C API using tokio spawn_blocking).
But it does mean that if relief valves are included as part of the Concurrency configuration, I'd have to do some extra identity mapping stage just to get that functionality. E.g.
let pipeline = Pipeline::from_stream(input)
    .pump(a_to_b_pump)
    .map(|x| futures::ready(x), Concurrency::serial().relief_valve(...))
    .pump(b_to_c_pump)
    .build();

Whether or not that's a problem is up to you.

If we're already talking about backpressure, I'd like to ask you something :)
In the current API "backpressure" is configured on a pump output side - a pump accumulates unconsumed results
.map(foo, Concurrency::serial().backpressure(10)) // accumulates unconsumed results
.map(bar, Concurrency::serial())

However, it is also common to regard backpressure on the input of operations. In other words, an operation can buffer input before processing
// should it actually be
.map(foo, Concurrency::serial())
.map(bar, Concurrency::serial().backpressure(10)) // buffers input before processing

Did the current placement of backpressure surprised you, or was it intuitive? I'm considering moving the backpressure config to the input side of pumps as I described

I haven't used the library yet but find it unintuitive to have the buffer "outside of the pump".
This would mean that having a full buffer results from the following pump being slow. Having it "inside" means that a full buffer results from the corresponding pump being to slow which I find way more intuitive.

I think either one can make intuitive sense, which may also be why neither seem to fit exactly. Zooming out and considering the options, might it work better to not have backpressure be part of the concurrency configuration?

This is a good point.  The reason I went with in-pump back pressure was my desire to use a mpsc::channel size parameter as natural backpressure. Since the pump creates the output channel, the backpressure configuration had to be passed to it.
I now think this consideration isn't important enough and I think I'll change it. I'll remove backpressure from the concurrency config and add .backpressure(n) and .backpressure_with_relief_valve(n)

new methods available in v0.0.3
Thanks for the input @Dryvnt!


MS teams and email webhook support question

Is your feature request related to a problem? Please describe
Your MS Teams integration using URL webhook not working. I have added the webhook and also enabled all the notifications for msteams in the product. But not receiving any notification on the teams. and Not receiving any errors from DD.
For Email I gettting a error Cannot assign requested address.

Please populate all the info as requested in the template: https://github.com/DefectDojo/django-DefectDojo/issues/new?assignees=&labels=bug&template=bug_report.md&title=

#3930

You most likely have an issue with docker to send emails, and certainly has nothing to do with the app itself. I remember having that issue too, and it was purely a docker configuration problem (I can't recall how I solved it, but Google helped me).


Issues due to unavailability of primary key constraint in some tables when MySQL database cluster is used with Group Replication

Describe your problem(s)
When MySQL database cluster is used with Group Replication, it is mandatory to have the primary key constraint for all the tables.
But the available dbscripts in the APIM server for 'apim_db' and 'shared_db' is missing the primary key constraint for some of the tables. Hence, could observe below error stack traces at the server startup and when the server is running.
TID: [-1234] [] [2020-11-05 15:49:14,250] ERROR {org.wso2.carbon.registry.core.utils.RegistryUtils} - Unable to create fixed remote mounts. org.wso2.carbon.registry.core.exceptions.RegistryException: Unable to put resource Failed to add properties to the resource /_system/config/repository/components/org.wso2.carbon.governance/media-types/index. The table does not comply with the requirements by an external plugin.
	at org.wso2.carbon.registry.core.jdbc.handlers.builtin.MountHandler.put(MountHandler.java:299)
. . . .
Caused by: org.wso2.carbon.registry.core.exceptions.RegistryException: Failed to add properties to the resource /_system/config/repository/components/org.wso2.carbon.governance/media-types/index. The table does not comply with the requirements by an external plugin.
. . . .
Caused by: java.sql.SQLException: The table does not comply with the requirements by an external plugin.
. . . .

TID: [-1234] [] [2020-11-09 17:12:00,412] ERROR {org.wso2.carbon.identity.oauth2.util.OAuth2Util} - Error while registering OAuth scopes with permissions bindings org.wso2.carbon.identity.oauth2.IdentityOAuth2ScopeServerException: SQL error occurred while creating scope :internal_feature_management
at org.wso2.carbon.identity.oauth2.dao.OAuthScopeDAOImpl.addScope(OAuthScopeDAOImpl.java:90)
. . . .

Describe your solution
The following table list need to have the primary key constraint,
shared_db:
REG_RESOURCE_COMMENT
REG_RESOURCE_PROPERTY
REG_RESOURCE_RATING
REG_RESOURCE_TAG
UM_SHARED_USER_ROLE
apim_db:
IDN_OAUTH2_ACCESS_TOKEN_AUDIT
IDN_OAUTH2_SCOPE_BINDING
IDN_AUTH_USER_SESSION_MAPPING
IDN_OAUTH2_CIBA_REQUEST_SCOPES
CM_SP_PURPOSE_PURPOSE_CAT_ASSC
CM_PURPOSE_PII_CAT_ASSOC
CM_SP_PURPOSE_PII_CAT_ASSOC
CM_CONSENT_RECEIPT_PROPERTY
AM_ALERT_EMAILLIST_DETAILS
AM_ALERT_TYPES_VALUES
It would be better if a note can be added to the respective documents [1], [2] as well.
Related Issues:
https://github.com/wso2/product-apim/issues/3923
https://github.com/wso2/product-is/issues/3898
Affected Product Version:
APIM-3.1.0, APIM-3.2.0
[1] https://apim.docs.wso2.com/en/latest/install-and-setup/setup/setting-up-databases/changing-default-databases/changing-to-mysql/
[2] https://apim.docs.wso2.com/en/3.1.0/install-and-setup/setup/setting-up-databases/changing-default-databases/changing-to-mysql/#!

Correction:
The following two tables in the apim_db already contain the primary key constraint.
AM_ALERT_EMAILLIST_DETAILS
AM_ALERT_TYPES_VALUES

Fixed in the docs.


koa package docs - parameter port in a wrong place

According to original docs, parameter port should be set in the services.http.server object not in the services.http.listen object. The same mistake is made in the example app and port does not change at this point :) I tried it out too.

After further discussion it was discovered that the issue was only in the demo app due to very old Koa component being used.


(Question) Am I able to automatically add a Network Server to the Application Server that is being deployed?

As the title states, I am interested in adding the Network Server automatically to be able to get up and running without using the web interface. Hope this is the right place to ask.

This should probably be in the application server repo.
That being said, I think it's a good idea. It could potentially be implemented as an env/config flag that is used only on initial startup, similar to the Postgres image's POSTGRES_PASSWORD. Retry mechanics may be non-trivial if the requested NS is not up at the time.

This is currently not supported (other than the web-interface or API integration). However, I'm currently doing some refactoring work and this add NS step will no longer be needed in the future. Stay tuned :)


Allow to configure two different certs for public access and internal service

Hi,
This is a feature request.
After opening a support case and being followed by one colleague of yours, seems that is not posible to use two different certificates; one for the public URL accessed on WAF and the other one used by the pods,
What I want to achieve is to have SSL channel between WAF and the pods, there's an article that points that is only necessary that the certificate is valid, but that's not accurate, after discussing the problem seems that it's a requirement to use the same hostname on public host as on the internal host… or have the public certificate to include names for the internal services.
E.G.
CONFIG THAT WOULD WORK BUT NOT DESIRED:
"
spec:
tls:

hosts:

$DNS_APP_FQDN_SVC
secretName: $AKS_PUBLIC_CERT_NAME
rules:

host: $DNS_APP_FQDN_SVC
http:
paths:

backend:
serviceName: $APP_SVC
servicePort: 443
"
The not desired config is because we don't want that internal cert used by the pods to match the public name.

DESIRED CONFIG THAT WON´T WORK:
"
spec:
tls:

hosts:

$DNS_APP_FQDN_SVC
secretName: $AKS_PUBLIC_CERT_NAME
rules:

host: $DNS_APP_K8S_SVC
http:
paths:

backend:
serviceName: $APP_SVC
servicePort: 443
"
The public host name to be different from the healthprove for internal hosts. Please note that configuring WAF manually it would work (already tested).

If the explanation is a bit confusing I will be glad to contact any one that looks into this to give further details and explanations.
Thanks and kind regards,

@evmimagina I have created a PR which adds an annotation appgw.ingress.kubernetes.io/backend-hostname. Using this annotation, you can specify the hostname(SNI) that Application Gateway should use while talking to the Pods.

Hi Akshay,
Thanks for looking into this, how can I test that setting? should I install any specific versión to test it?
Regards,


add tests for donate page

[ ] test for filtering
[ ] drop down
[ ] expecting the elements from the list

open to other test cases too

I can take this issue.


remove not needed execution context

Do we need the ExecutionContext over in LightweightDatasourceModule#lightweightDatasource anymore?

Do we need the ExecutionContext over in LightweightDatasourceModule#lightweightDatasource anymore?

I don't think so, pretty sure it was just to workaround the blaze client not using ContextShift, and we haven't used that client in awhile (if we ever use it again, we can just create an EC for it).

I don't think so

Cool, let's ditch it then to remove confusion for future ds implementors.


node-serialport for PWA, causes error with bindings on localhost or netlify.

Summary of Problem
I am trying to get a progressive web app to run the node-serialport package on desktop so I can migrate from chrome.serial. First tried on a localhost then on a hosted netlify webserver with different errors.
Code to Reproduce the Issue
Initial Test
const serialport = require('serialport');

Error on localhost:
bindings.js:203 Uncaught TypeError: exists is not a function
    at Function.getRoot (bindings.js:203)
    at bindings (bindings.js:82)
    at<EMAIL_ADDRESS>(linux.js:2)
    at newRequire (src.77de5100.js:47)
    at localRequire (src.77de5100.js:53)
    at<EMAIL_ADDRESS>(index.js:14)
    at newRequire (src.77de5100.js:47)
    at localRequire (src.77de5100.js:53)
    at Object.parcelRequire.../node_modules/serialport/lib/index.js.@serialport/stream (index.js:2)
    at newRequire (src.77de5100.js:47)

Error on netlify (hosted webapp):
bindings.js:178 Uncaught TypeError: Cannot read property 'indexOf' of undefined
    at Function.getFileName (bindings.js:178)
    at u (bindings.js:82)
    at Object.<anonymous> (linux.js:2)
    at f (src.690f0ea0.js?__WB_REVISION__=2fa39e153739be0eecc364af075d5850:1)
    at p (src.690f0ea0.js?__WB_REVISION__=2fa39e153739be0eecc364af075d5850:1)
    at Object.<anonymous> (index.js:14)
    at f (src.690f0ea0.js?__WB_REVISION__=2fa39e153739be0eecc364af075d5850:1)
    at p (src.690f0ea0.js?__WB_REVISION__=2fa39e153739be0eecc364af075d5850:1)
    at Object.<anonymous> (nodeserialUtils.js:2)
    at f (src.690f0ea0.js?__WB_REVISION__=2fa39e153739be0eecc364af075d5850:1)

I'm using Parcel as a bundler so it could be a bundling issue.
It appears to be the bindings causing a conflict, as it goes away by requiring @serialport/stream but then I lose the automatic bindings...
Question is, is this actually a viable way to use node-serialport? It should at least work on localhost, right? Or do I absolutely need to wrap it in something like Electron for desktop use?
Versions, Operating System and Hardware

<EMAIL_ADDRESS>Node.js v14
Windows

@reconbot
Thanks for the quick reply! Okay so I already have a chrome extension set up to serve my purposes, I should just use what I can from the npm packages I like to enhance it rather than trying to rewrite the serial code for nodejs since it looks like it doesn't quite fit for my purposes.
Come on w3 where's muh browser serial ports!

@reconbot
Found my solution:
https://codelabs.developers.google.com/codelabs/web-serial/#1
Future computing to the rescue!


Add Headers for Trends Request, #108

looks good

Greetings,
I'm getting an error on  -> 'module_html' when calling 'get_trends()'
`Traceback (most recent call last):
in
Trending = get_trends()
File "site-packages\twitter_scraper\modules\trends.py", line 17, in get_trends
html = html.json()["module_html"]
KeyError: 'module_html'`

Hi @Sue-Fwl , you are using the latest version, right? Please be sure. Thanks

Sure did, and I just copied the trends.py file from the repository  and ran the command again [same error {KeyError: 'module_html'} ]
p.s other modules are working great for me -profile and tweets-

I'm not sure~
Will do then.


Remove to device in network

Reproducibility is ensured in https://github.com/ASUS-AICS/LibMultiLabel/tree/reproduce

@henryyang42 I just test CAML/MIMIC-50 without .to(device). It won't affect to results.

@Eleven1Liu I got different result on kim_cnn/rcv1. If CAML&CNN on MIMIC-50 dataset without .to(device)  can produce same result. We may need to go deeper on random seeds.
Previously we didn't set seed on  torch.cuda.manual_seed_all(seed) and random.seed. Now we use PL's seed_everything, which handles
    random.seed(seed)
    np.random.seed(seed)
    torch.manual_seed(seed)
    torch.cuda.manual_seed_all(seed)

@Eleven1Liu I got different result on kim_cnn/rcv1. If CAML&CNN on MIMIC-50 dataset without .to(device) can produce same result, we may need to go deeper on random seeds (One difference between rcv1 and MIMIC is that rcv1 does not have a predefined validation set, I am not sure if train_test_split(datasets['train'], test_size=config.val_size, random_state=42) will cause any issue if config.seed != 42. Another difference is that CAML will not apply networks.get_init_weight_func).
Previously we didn't set seed on torch.cuda.manual_seed_all(seed) and random.seed(seed). Now we use PL's seed_everything, which handles
    random.seed(seed)
    np.random.seed(seed)
    torch.manual_seed(seed)
    torch.cuda.manual_seed_all(seed)

True. It's different ...


Unit command needs file to test specified before options

If we use rna unit <option> <path-to-file-to-test> rna launches tests correctly but will test all tests on workspace and not the specified file. In order to work properly it needs  to be written before any <option>.

Command arguments need to be specified before options, in order to avoid conflicts between arguments and option values.


Proposed change to BadStreamStateTransition for debuggability

I'm just beginning to dig into the HTTP/2 and Swift-NIO code and wanted to get some feedback on a proposal to modify BadStreamStateTransition to have from/to states to improve debuggability when it gets logged (and maybe in the future, stack traces when it gets created)
A few questions:

The HTTP2StateMachine.State enum is private, and comments indicate that it's so people don't erroneously set the state directly.  Isn't the right place to do this in setting the member state variable to private (which it already is)?   If that works I could make it internal and move the state enum outside of HTTP2StateMachine

I'm seeing some comments that (to me) don't agree with the RFC. I am no HTTP expert by any means and I'm fairly new to Swift, as well, so I wouldn't be surprised if I"m wrong here, but on line 379, it indicates sending HEADERS on an idle stream isn't permitted, but the state transition diagram and the RFC both seem to indicate that it should transition a stream to open.

Apologies for the basic questions, like I mentioned I'm new. Thanks in advance for your help.
Neal

This seems like a good idea, I’m in favour of it! In answer to your questions:

The HTTP2StateMachine.State enum is private, and comments indicate that it's so people don't erroneously set the state directly. Isn't the right place to do this in setting the member state variable to private (which it already is)?

At the time there was no good reason to expose the type to other parts of the code. You have found one, so the status quo should change. By all means change the visibility. Please leave it nested inside the state machine type though; we don’t gain anything by hoisting it out, and it helps avoid the type seeming to be generally useful.

on line 379, it indicates sending HEADERS on an idle stream isn't permitted, but the state transition diagram and the RFC both seem to indicate that it should transition a stream to open.

A server is not permitted to send a HEADERS frame on an idle stream. RFC 7540 does say this, just not in the connection state machine diagram.
If you read the comment in the code next to our reproduction of the state machine diagram you’ll notice that it says that the RFC 7540 diagram is so incomplete as to be actively misleading. In this case, our (much more detailed) state machine is right: RFC 7540’s diagram is wrong.

One scenario that I was thinking about was exposing ]information about the streams in a server built off this code. If someone wants to drill down into the various open streams by connection state, is it possible to select all, say, .idle streams independent of the associated data?

I am not sure that this is something anyone is actually going to do, so I don’t think I’d worry too much about the feature: in general we don’t let users do too much introspection of this state.
For the Comparable issue, the best solution may be to implement a parallel enum without associated data expressly for debugging purposes, and make that more visible.

Didn't get the chance today because of a long drive, but I agree that end users of apps providers by the server would not need to introspect the state, but I think someone in a DevInfra role or anyone required to monitor the servers need information like that.  It may be worth considering either:

Considering alternatives to associated data for each stream state (but the way I am familiar with, by having a bunch of member variables, a subset of which are used for each state, is not very elegant)

Maybe it's wishful thinking, but maybe being able to compare Swift enums independent of associated data is on the language roadmap.   I'm happy to prototype it just to see while we iterate on what the final solution looks like.
Thanks!

We should separate the requirement to see the state from the requirement to be able to operate upon the state. If our goal is to allow users only to see the state, that can be achieved by exposing the enum, without adding any Comparable requirement. The only reason I can see to provide an implementation of Comparable is to allow reasoning about the state, and I’m not even entirely sure what utility that would provide.
Can you outline why we end up with a requirement to implement Comparable?

@Lukasa: If you look at the code, BadStreamStateTransition conforms to NIOHTTP2Error.  Once I add the to/from states as members to BadStreamStateTransition, it is no longer comparable because one of teh types that is associated data of HTTP2StreamStateMachine.State values is not comparable.  I did not have the time to find out which.
I don't know if that makes sense, or not?

NIOHTTP2Error shouldn’t require Comparable, only Equatable.


Can you rewrite the article?

@x13xxxswatxxx commented on Fri Mar 01 2019
For me personally, the article seemed complicated.
5 times (2 days), already tried to start over and just got more confused.
The beginning is more or less clear, but then comes the use of event, EventHandler <class>, creating classes for it, using lambda expressions for add, remove, as if we work with them every day.
why internal to create an extension?
It is not clear which class to insert search into.
where did lister come from and how to apply at all.
Лично для меня, статья показалась сложной.
5 раз (2-х дней),  уже пробовал начинать сначала и только больше запутался.
Начало более менее понятно, но далее идёт применение event, EventHandler<class>, создание классов под него, применение лямбда выражений для add, remove,  как будто мы работаем с ними каждый день.
зачем internal, чтобы создать расширение?
Непонятно в какой класс нужно вставлять search.
откуда взяли lister и как вообще применить.

Сведения о документе
⚠ Не вносите правки в этот раздел. Это необходимо для связывания страницы сайта docs.microsoft.com с вопросом на GitHub.

ID: 90b37f1a-cd59-d9e5-067b-30ea5f26e5cc
Version Independent ID: d2078536-1ee3-5b60-3872-f2635d245cc2
Content: Стандартные шаблоны событий .NET
Content Source: docs/csharp/event-pattern.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Hi team,
I'm moving this here since the user's comment is about the original documentation
best

Same, I wish this was clearer. I think too many assumptions are made by the writer about prior reader knowledge. Although, I've been reading through the C# concepts docs in order, it would still help a lot if individual bits of code here could be defined a bit better. Assume we are idiots please

I've made a new issue as well. This article does not follow the event conventions in .net (links below). The delegate name is very misleading and events are invoked directly instead of used an "OnFileFound" method for derived classes.
https://docs.microsoft.com/en-us/dotnet/standard/events/
https://docs.microsoft.com/en-us/dotnet/standard/design-guidelines/event

I agree with the other comments and the example isn't even correct as stated by robloo. Almost better to just remove it in the short-term.

This article is simply terrible. Came here to refresh my knowledge of recommended patters in .NET and realized the previous MSDN article about this was way, way more clear and correct. Who proofreads this? It's like a main doc of the huge technology company, how come the quality is so poor. Imagine if the author writes more articles like this. @Microsoft, please re-assign the author for the official docs :)


Fix save

Changes Unknown when pulling b967f2e6d00c40090118f1b5565c82705b7a9a28 on yakky:fix_save into ** on divio:master**.

Changes Unknown when pulling b967f2e6d00c40090118f1b5565c82705b7a9a28 on yakky:fix_save into ** on divio:master**.

Changes Unknown when pulling b967f2e6d00c40090118f1b5565c82705b7a9a28 on yakky:fix_save into ** on divio:master**.

Changes Unknown when pulling b967f2e6d00c40090118f1b5565c82705b7a9a28 on yakky:fix_save into ** on divio:master**.

Changes Unknown when pulling b967f2e6d00c40090118f1b5565c82705b7a9a28 on yakky:fix_save into ** on divio:master**.

@stefanfoulis html5lib/html5lib-python#189
Sorry, I commented with the relevant issue on the CMS

FWIW, there's now a release with the regressions fixed. (As far as I and the tests can tell, at least.)

@gsnedders awesome, thanks! We will change the dependency soon


how can I use it as a service to open each sub folder from root

I have created a directory D:/h5apps ,  how can I use vscode-web to open each sub directories, just like:
http://host:8080/edit/app01   ==> D:/h5apps/app01
http://host:8080/edit/app02   ==> D:/h5apps/app02
http://host:8080/edit/app03   ==> D:/h5apps/app03
thank you

Hi
I think you will have to code an extension for that
You'll have to code a filesystemprovider in an extension, and then, in the index.html file, you'll have to provide a window.product variable with dynamic folderUri
const currentUrl = new URL(window.location.href);
const path = currentUrl.pathname.substring(1);

window.product = {
  productConfiguration: {
    nameShort: "My Code Browser",
    ...
  },
  folderUri: {
    scheme: "myCustomScheme",
    path,
  }
}

And then in your extension you'll be able to use vscode.workspace.workspaceFolders[0].uri to know the current uri.
With the uri you'll be able to match the correct forlder to query on server.


PromiseInterface has not done object

Taking a look a this line done  method does not exist for a React\Promise\PromiseInterface
https://github.com/jakubkulhan/bunny/blob/10e34dec73397d30cb79f61dd81e9716644ede01/src/Bunny/Async/Client.php#L335
It has produces some errors in my project
In Client.php line 340:
Attempted to call an undefined method named "done" of class "React\Promise\Promise".

This package doesn't target promise v3 yet:
https://github.com/jakubkulhan/bunny/blob/10e34dec73397d30cb79f61dd81e9716644ede01/composer.json#L30
So how are you getting that error?
P.S. Don't get me wrong, happy to see more and more people using Promise v3 but that shouldn't happen when you use this package yet.

Closing this as #147 landed promise v3 for the upcoming 0.6 release


validateDOMNesting warning w/ React 0.14

I'm using React 0.14 beta 3. Rendering any Table triggers this warning:
Warning: validateDOMNesting(...): <span> cannot appear as a child of <thead>. See App > Table > Thead > thead > span.

Additional prop changes after this can cause Invariant Violation errors and break the app.

Looks similar to #133. Are you using the latest version of Reactable?

Oops, scratch that. Looks like a similar but not identical issue.

Upgraded to version 0.12.0 for reactable, and 0.14.3 for react/react-dom and haven't noticed any errors, so closing this. Thanks.


[Bug] SHOW TABLE STATUS FROM employees; show arana core table

What happened:

What you expected to happen:
Same result as show tables
How to reproduce it (as minimally and precisely as possible):
SHOW TABLE STATUS FROM employees;

Anything else we need to know?:
Provide filter, filter out uniformly arana core table

Need to hide those tables with prefix __arana_.

assignees me


MKLDNN softmax outputs NaN in mkldnn 0.14

Description
Extremely negative softmax inputs output NaN. This is an error caught detected in MKLDNN already (https://github.com/intel/mkl-dnn/issues/106) with a fix (https://gist.github.com/emfomenk/0386c529c5df21ae308b00d16454c48e) in MKLDNN v0.15+ (we are v0.14).
The fix is either to:

patch MKLDNN v0.14 with the earlier fix
to upgrade the MKLDNN version in mxnet (https://github.com/apache/incubator-mxnet/pull/12953).

Environment info (Required)
ubuntu@ip-172-31-3-217:~$ python diagnose.py
----------Python Info----------
Version      : 3.6.4
Compiler     : GCC 7.2.0
Build        : ('default', 'Jan 16 2018 18:10:19')
Arch         : ('64bit', '')
------------Pip Info-----------
Version      : 9.0.1
Directory    : /home/ubuntu/anaconda3/lib/python3.6/site-packages/pip
----------MXNet Info-----------
/home/ubuntu/anaconda3/lib/python3.6/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
Version      : 1.3.0
Directory    : /home/ubuntu/anaconda3/lib/python3.6/site-packages/mxnet
Commit Hash   : b3be92f4a48bce62a5a8424271871c2f81c8f7f1
----------System Info----------
Platform     : Linux-4.4.0-1065-aws-x86_64-with-debian-stretch-sid
system       : Linux
node         : ip-172-31-3-217
release      : 4.4.0-1065-aws
version      : #75-Ubuntu SMP Fri Aug 10 11:14:32 UTC 2018
----------Hardware Info----------
machine      : x86_64
processor    : x86_64
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                72
On-line CPU(s) list:   0-71
Thread(s) per core:    2
Core(s) per socket:    18
Socket(s):             2
NUMA node(s):          2
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 85
Model name:            Intel(R) Xeon(R) Platinum 8124M CPU @ 3.00GHz
Stepping:              3
CPU MHz:               3000.000
BogoMIPS:              6000.00
Hypervisor vendor:     KVM
Virtualization type:   full
L1d cache:             32K
L1i cache:             32K
L2 cache:              1024K
L3 cache:              25344K
NUMA node0 CPU(s):     0-17,36-53
NUMA node1 CPU(s):     18-35,54-71
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni pclmulqdq monitor ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch invpcid_single kaiser fsgsbase tsc_adjust bmi1 hle avx2 smep bmi2 erms invpcid rtm mpx avx512f rdseed adx smap clflushopt clwb avx512cd xsaveopt xsavec xgetbv1 ida arat
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0012 sec, LOAD: 0.4806 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.1717 sec, LOAD: 0.5293 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.1596 sec, LOAD: 0.3734 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0262 sec, LOAD: 0.1173 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0013 sec, LOAD: 0.3264 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0118 sec, LOAD: 0.0690 sec.

Package used (Python/R/Scala/Julia):
Python
For Scala user, please provide:

Java version: (java -version)
Maven version: (mvn -version)
Scala runtime if applicable: (scala -version)

For R user, please provide R sessionInfo():
Build info (Required if built from source)
Compiler (gcc/clang/mingw/visual studio):
MXNet commit hash:
6b5d9f9785a398d2e8ccaa950f89fb76d76d5bd4
Build config:
MKLDNN (pip install mxnet-mkl)
Error Message:
ubuntu@ip-172-31-3-217:~/incubator-mxnet$ python tt.py
/home/ubuntu/anaconda3/lib/python3.6/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
[
[[[[nan nan]]]]
<NDArray 1x1x1x2 @cpu(0)>]

Minimum reproducible example
import mxnet as mx
input_data = mx.nd.array([[[[-1e30,-1e30]]]])
data = mx.sym.Variable('data')
out1 = data.softmax(axis=1)
exec1 = out1.bind(mx.cpu(), args={'data': input_data, 'softmax_label': mx.nd.ones([1]), 'fc_weight': mx.nd.ones([2,2]), 'fc1_weight': mx.nd.ones([2,2])})
exec1.forward()[0].wait_to_read()
print(exec1.outputs)

Steps to reproduce
Run the following script.
What have you tried to solve it?
Applying this one line fix (https://gist.github.com/emfomenk/0386c529c5df21ae308b00d16454c48e) in mkldnn fixes the issue.

@nswamy

@azai91 could you help to add a testcase for this issue after the #12953 is merged?

@azai91 PR is merged, go ahead to create a test for this issue 👍
Feel free to let me know if any help is needed.
Original: MKL-DNN 0.14
patric@mlt-skx122 mxnet]$ python soft.py
/home/patric/.local/lib/python2.7/site-packages/h5py/__init__.py:34: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
[
[[[[nan nan]]]]
<NDArray 1x1x1x2 @cpu(0)>]

Now: MKL-DNN 0.17
[patric@mlt-skx122 mxnet]$ python soft.py
/home/patric/.local/lib/python2.7/site-packages/h5py/__init__.py:34: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
[
[[[[1. 1.]]]]
<NDArray 1x1x1x2 @cpu(0)>]

@mxnet-label-bot [MKLDNN]


Angular 4 Storefront API example

Angular 4 Storefront API example

I don't know anything about Angular 4 but I tried to get this running according to the README.
After an npm install, I ran npm start and get the following error:
Cannot read property 'length' of undefined
TypeError: Cannot read property 'length' of undefined
    at createSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:15457:109)
    at parseSourceFileWorker (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:15389:26)
    at Object.parseSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:15338:26)
    at Object.createSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:15192:29)
    at VirtualFileStats.getSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/@ngtools/webpack/src/compiler_host.js:66:35)
    at WebpackCompilerHost.getSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/@ngtools/webpack/src/compiler_host.js:213:38)
    at findSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:67909:29)
    at processImportedModules (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:68056:25)
    at findSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:67937:17)
    at processImportedModules (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:68056:25)
    at findSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:67937:17)
    at processImportedModules (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:68056:25)
    at findSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:67937:17)
    at processImportedModules (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:68056:25)
    at findSourceFile (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:67937:17)
    at processImportedModules (/Users/scott/dev/storefront-api-examples/angular-graphql-client/node_modules/typescript/lib/typescript.js:68056:25)

This is using Node v8.5.0 and npm 5.4.2.

My mistake. Files added. Readme.md updated.

Update to Angular-cli 5 ;))))

Sorry @JsssCode, I had forgotten about this until your recent commits. I'll give this another look soon. But last time I tried it out, it was using different styling from our other examples. They all use the same css from https://github.com/Shopify/storefront-api-examples/tree/master/shared (and same HTML structure).
Is that possible to do with this Angular example?

@swalkinshaw Yes, it is possible, will work on this

@swalkinshaw, it's done. Now I use general css and html structure is the same as in other samples.

Tried it out and it mostly looks good. I had a few styling issues with the Cart sidebar though.
I'm going to merge this since it's definitely useful as an Angular example anyway. There can always be follow-up fixes 😄
Thank you!

@swalkinshaw, thanks. Will try to fix


feat: add code blocks in quickstart

Added code blocks to in quickstart page
Add Frame component for image
Add fullscreen value in youtube video iframe

Thanks! Good changes!


bot not connecting

I have made the bot, given it bot (admin) permissions, added the bot token into torch, added the channel ids. the bot stays offine and doesnt come online, I think there are a few steps missing from the documentation. the bot is running in game

same for me but only issue on remote dedicated server. With same settings (configs files) on local PC the bot run fine and show all messages in discord.

same for me but only issue on remote dedicated server. With same settings (configs files) on local PC the bot run fine and show all messages in discord.


Community ID Zeek package install fails atop Homebrew-installed Zeek v4.0.0 on macOS

This is basically just a tracking issue on our end to point at corelight/zeek-community-id#15.
The tl;dr is that if we advise Brimcap users to install their own off-the-shelf Zeek (as opposed to one we build that comes with Community ID already installed) and install the Community ID package so they can easily pivot between Suricata alerts and Zeek events in the logs generated from pcaps, they're likely to hit  corelight/zeek-community-id#15 if they install Zeek via brew and zkg via pip3 instead of compiling Zeek themselves. Depending on if/when that gets addressed and what direction we go with Brimcap, we may want to highlight this in macOS guidance in a README.

With some help from the Zeek team, I managed to get https://github.com/Homebrew/homebrew-core/pull/74932 merged which, addresses this problem. Details are at https://github.com/corelight/zeek-community-id/issues/15#issuecomment-818079121. I can close this tracking issue now.


fix(sha256): Add extra checks against message size when constructing msg blocks

Description
Problem*
Resolves
Issue found in zk passport https://github.com/ocelots-app/passport-verifier/blob/47e9464e7e782b07b6d791bf1d13257fce2f486b/crates/lib/data-check/integrity/src/lib.nr#L118 when performing sha on a message with a large padding.
Summary*
The current sha algorithm accounts for message padding, but only where ithe padding is still contained in the block we are compressing. For the case where we have a padding that extends multiple blocks past the message size we end up with a correctness error. We need to add more checks against the message size to make sure we are comrpessing the correct msg block.
An increase in gate count is expected from these changes.
Additional Context
Documentation*
Check one:

[X] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Program	ACIR opcodes (+/-)	%	Circuit size (+/-)	%
sha256_regression	+32,838 ❌	+444.78%	+114,735 ❌	+121.86%
regression_4449	+3,850 ❌	+18.39%	+5,793 ❌	+1.96%

Two big new additional regressions were included under sha256_regression in this PR. If you look at the full report sha256, sha2_byte, and sha256_var_size_regression are largely untouched in circuit size by these changes.


Data simulator

A simulator layer for data input stream.
Could be switched by marco DATA_SIMULATOR.

Done.


Folder structure not show

The promise of this is amazing!  I just can't quite use it yet:

The folder structure is not shown.  As such, a folder cannot be selected, so I get an error message: "Please, select a single folder."  No folders are listed, either -- even subfolders in the TTS Cloud root directory.
Help!

I stripped out all the folder checks leaving the root folder as "" and got unblocked.  Would be nice to know why the folder structure is no longer being recognized and get a better fix.

Note, too, that once I put files into sub-folders from root, they did show up properly and were selectable, but it still seems that root cannot be selected, currently.

Uhm, interesting, could you share the CloudInfo.bson so I can test it? It's on the folder of the program, with _CloudInfo.bson. That way I can load the same data and check.
I see the problem with the root at least: I have pushed a small commit that allows you to select the root, and It allowed me to select it and upload there. Do you mind trying?

Hey thanks for the fix!  Works great, appreciate having the root node.  The bson/json has all the links to my pre-release game content, so I'd rather not share that.
Thanks again for the super helpful tool!


Potential Bug:  Can't disable new user registration with OAuth

With Google OAuth enabled, I can't disable the ability for new users to create accounts, organizations, & projects -- primarily accounts.
I've tried setting USER_CREATE_PERMISSIONS=djoser.permissions.CurrentUserOrAdmin like the readme suggests. And  FWIW I've tried USER_CREATE_PERMISSIONS=rest_framework.permissions.IsAdminUser
Any ideas on how to do this?

Hi @manuphatak - we'll take a look.

Great, Thank you!

Hi @manuphatak, I've worked on this and added a new environment variable that will allow you to disable the registration of new users via Google and Github. I plan to merge this into the master branch on Github shortly. However, after reading your issue again I'm not sure that is exactly what you're looking for. Could you describe a bit more around the issue you're experiencing please?
To set some more context:
The USER_CREATE_PERMISSIONS setting is there to essentially prevent users from registering and only allows users to be added to the platform via invites. The same thing here would be to disable OAuth registration as I have mentioned above.
Regarding the ability to prevent users inviting new users & creating new projects, that's something that we support in our enterprise solution which benefits from a more fine grained permission model but is not something that we officially support for self hosted installs. Users will always be able to create new organisations, however.

Ok, I think I now understand and have updated the setting mentioned above such that it prevents uninvited users from registering via any method without an email invite, otherwise the API will throw a 403 forbidden error. I think that should achieve what you're after and is a slightly nicer experience than setting the permission class as you've mentioned? Let me know if that sounds like it will work for you and I can get it merged in.

Hey, @matthewelwell, I believe the solution you described is exactly what I was looking for. 🎉
The scenario is...

we're self-hosting
we know everyone who's going to use the tool

The problem we want to solve...

We want to lock down the app enough that random people can't create accounts.

Hi @manuphatak, this is now resolved in the master branch and also in the v2.7.1 tag. Note that if you're running the FE as well, you'll need to make sure that the FE is at least running v2.7.0 as well.

Awesome, thank you, thank you, thank you!

I just launched the 2.7.1 container and it worked exactly as expected. 🎉    Thank you all so much!


Missing definitions for SOF_TIMESTAMPING_*

The SOF_TIMESTAMPING_* definitions from https://github.com/torvalds/linux/blob/master/include/uapi/linux/net_tstamp.h are currently missing. Would it be acceptable to add these? I'm happy to submit a pull request if someone can advise me where the best place to put them is.
This issue is currently blocking https://github.com/nix-rust/nix/pull/902

Similar to https://github.com/rust-lang/libc/issues/1103, a PR would be most welcome! So long as CI passes we should be good to go


HTTP/2 concurrent tile requests

This is not so much a bug as a potential for better performance when loading many tiles over HTTP/2 (let me know if another forum would be more appropriate).
What I would expect when switching to HTTP/2 for tile servers, is that when e.g. loading a full screen of tiles, all those tiles will be requested more-or-less simultaneously (using multiplexing over a single socket).  What I'm observing in practice with OpenLayers, using the epsg-4326 example that runs on an HTTP/2 server, is the following (see also screenshots at the bottom):

In Firefox, 16 tiles are being loaded concurrently (hitting the OpenLayers maxTotalLoading limit).
In Chrome, 6 tiles are being loaded concurrently, while another 10 are "Queueing".
Surprisingly, when loading the HTTP/2 tiled-image example at https://http2.golang.org/gophertiles?latency=30, Chrome does load many more tiles concurrently.

I have been doing some testing in our own application, where I can get a decent speed-up in Firefox (around a factor 2 in my test case) when increasing maxTotalLoading from 16 to 128.  Unfortunately Chrome does not want to request more than 6 tiles concurrently, but I suspect it should be possible to get a decent speed-up there, too, if it would.
So the questions are:

Would it make sense for OpenLayers to schedule more concurrent tile requests to HTTP/2 servers?
Does anyone have a clue why Chrome chooses to queue OpenLayer's tile requests, and can we do anything about it?  I have been searching the internet for answers on this, but so far to no avail.

Firefox/OpenLayers:

Chrome/OpenLayers:

Chrome/Gopher:

Thanks for raising this issue @freekvw.  I created an example that demonstrates the slow loading: https://codepen.io/tschaub/pen/ZJaJOb

It will be good to see what we can do to improve this.

Recent version of Chrome are now loading more than 6 tiles concurrently, see:
https://http2.golang.org/gophertiles?latency=30

Note that this issue specifically occurs in combination with OpenLayers.  Apparently scheduling in Chrome is done differently depending on how the tiles are loaded: directly as  elements on the screen vs. what OpenLayers does (off-screen/canvas rendering?).

@freekvw see https://codepen.io/fredj/full/aVBGNP/
In Chrome 62 linux, all the 16 tiles are being loaded concurrently.

I see, that's great :-)

I have a client that has slow, shared-resoucred citrix clients. Indeed is leaflet and Esri fast, and openlayers still slow.
sometimes 10fold. I narrowed it down to the HTTP2 request that the others make, and openlayers is still on 1.1.
Its a bit unclear for me of this threat was solved, or that chrome was just faster.

@meijer3 can you share any more information about the setup?  Ideally, an example map.  If not that, what layer, source, browser, and version of OpenLayers?


Authentication doc example should read https-enabled = true

Since basic auth is used, it would we sensible to enable https.
I think this was the intent, since a cert is provided.
[http]
  enabled = true
  bind-address = ":8086"
  auth-enabled = true # ✨
  log-enabled = true
  write-tracing = false
  pprof-enabled = false
  https-enabled = false  # <----------- Shouldn't this be true?
  https-certificate = "/etc/ssl/influxdb.pem"

URL for relevant page?
https://docs.influxdata.com/influxdb/v1.7/administration/authentication_and_authorization/#2-by-default-authentication-is-disabled-in-the-configuration-file
What products and version are you using?
InfluxDB 1.7

Fixed in master and closing.


commit bc320be00bc... (linter) BROKE US

commit bc320be00bc584bd88525266d23a9d5edb9d44f8 broke our CI; it needs to be reverted so we can process PRs.
When I attempted to create a PR to revert it, it fails because the gometalinter has been removed from the image?
WE CAN NO LONGER PROCESS PRS AND MERGE THEM

@baude the change was reverted now, can we close this issue?


Polling interval not configurable

Good morning. It seems like the polling frequency of a continuous replication rule isn't configurable at all. Checking the logs, it looks like the target is checking the source for document changes every 5 seconds. No matter what I try to change, it just doesn't seem to deviate from this 5 second rule. Is this 5 second polling interval intended as an unchangeable variable or am I missing something?

Continuous replication doesn't poll. The replicator sits on a _changes feed that publishes changes to the database as they occur. There is no introduced delay, but changes may not be published on the feed instantaneously.


Expose docker's root directory by default as part of docker info.

Primary use case is cAdvisor.

Are you parsing the CLI output rather than using the API?

fwiw, I'm not against this change.

@calavera: The API is what is used mostly. But for testing purposes, we use the CLI mostly.

LGTM

this needs minor docs changes, because the order and indentation in the output is slightly different;
https://github.com/docker/docker/blob/master/docs/reference/commandline/info.md
https://github.com/docker/docker/blob/master/man/docker-info.1.md

@vishh do you mind moving that message over the debug logs and complete the documentation?

Job: Docker-PRs-WoW-TP4 FAILED:

---
out=10
[docker] $ sh -xe D:\temp\hudson8471905484006609750.sh
+ git fetch origin +refs/pull/19986/head:refs/remotes/origin/pr/19986
From https://github.com/docker/docker
 * [new ref]         refs/pull/19986/head -> origin/pr/19986
+ git merge origin/pr/19986
Auto-merging docs/reference/commandline/info.md
CONFLICT (content): Merge conflict in docs/reference/commandline/info.md
Automatic merge failed; fix conflicts and then commit the result.
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe D:\temp\hudson9114068030609955062.sh
+ set +e
+ set +x
INFO: End of cleanup
Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
---

Job: Docker-PRs-WoW-TP4 FAILED:

---
945e2e.
+ ec=125
+ set +x

----------------------------------
ERROR: Failed to build test binary
----------------------------------

-----------------------------------------------
ERROR: Failed with exitcode 125 at Tue Feb  9 23:34:47 CUT 2016.
-----------------------------------------------

INFO: Tidying up at end of run
INFO: Nuking /d/CI
INFO: Zapped successfully
INFO: End of cleanup
INFO: Ended at Tue Feb  9 23:34:48 CUT 2016 (1m 11s)
Build step 'Execute shell' marked build as failure
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe D:\temp\hudson2427858837930854721.sh
+ set +e
+ set +x
INFO: End of cleanup
Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
---

Job: Docker-PRs-Win2Lin FAILED:

---
aking bundle: binary (in bundles/1.11.0-dev/binary)
Building: bundles/1.11.0-dev/binary/docker-1.11.0-dev
# github.com/docker/docker/docker
/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory

---
Build step 'Execute shell' marked build as failure
Performing Post build task...
Could not match :Build timed out  : False
Logical operation result is FALSE
Skipping script  : cd c:\scripts
sh timeout-post-build.sh

END OF POST BUILD TASK 	: 0
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe C:\Users\Jenkins\AppData\Local\Temp\hudson2275162023488590948.sh
+ set +x

---

Job: Docker-PRs-Win2Lin FAILED:

---
aking bundle: binary (in bundles/1.11.0-dev/binary)
Building: bundles/1.11.0-dev/binary/docker-1.11.0-dev
# github.com/docker/docker/docker
/usr/local/go/pkg/tool/linux_amd64/link: running gcc failed: fork/exec /usr/bin/gcc: cannot allocate memory

---
Build step 'Execute shell' marked build as failure
Performing Post build task...
Could not match :Build timed out  : False
Logical operation result is FALSE
Skipping script  : cd c:\scripts
sh timeout-post-build.sh

END OF POST BUILD TASK 	: 0
[PostBuildScript] - Execution post build scripts.
[docker] $ sh -xe C:\Users\Jenkins\AppData\Local\Temp\hudson4081389874470455800.sh
+ set +x

---

LGTM

@calavera: Anything else to be fixed in this PR?

LGTM.
I moved it to docs review, there are a couple of small things to fix.

docs LGTM after the nits are addressed

ping @vishh can you address my comments, then we can move this forward :heart:

Job: Docker-PRs-WoW-TP4 FAILED:

---
	      56	docker-1.11.0-dev.exe.md5
  0%
100%
	    New File  		      88	docker-1.11.0-dev.exe.sha256
  0%
100%
	    New File  		  27.1 m	docker.exe
  0.0%
  3.6%
  7.3%
 11.0%
 14.7%
 18.3%
 22.0%
 25.7%
 29.4%
 33.0%
 36.7%
 40.4%
 44.1%
 47.7%
 51.4%
 55.1%
 58.8%
 62.5%
 66.1%
 69.8%
 73.5%
 77.2%
 80.8%
 84.5%
 88.2%
 91.9%
 95.5%
 99.2%
100%

------------------------------------------------------------------------------

               Total    Copied   Skipped  Mismatch    FAILED    Extras
    Dirs :         1         0         1         0         0         0
   Files :         4         4         0         0         0         0
   Bytes :   54.39 m   54.39 m         0         0         0         0
   Times :   0:00:00   0:00:00                       0:00:00   0:00:00

   Speed :<PHONE_NUMBER> Bytes/sec.
   Speed :            66608.028 MegaBytes/min.
   Ended : Monday, February 15, 2016 1:58:48 PM

+ ec=0
+ set +x
INFO: Linking the built binary to /d/CI/CI-71846
---

@thaJeztah: PTAL.

Job: Docker-PRs-experimental FAILED:

---
x_test.go:247: DockerExternalVolumeSuite.TestExternalVolumeDriverUnnamed	1.339s
PASS: <autogenerated>:29: DockerExternalVolumeSuite.TestExternalVolumeDriverVolumesFrom	1.999s
PASS: docker_cli_start_volume_driver_unix_test.go:415: DockerExternalVolumeSuite.TestExternalVolumeDriverWithDaemnRestart	1.065s
PASS: docker_cli_start_volume_driver_unix_test.go:394: DockerExternalVolumeSuite.TesttExternalVolumeDriverList	0.083s

----------------------------------------------------------------------
FAIL: docker_cli_pull_test.go:110: DockerHubPullSuite.TestPullAllTagsFromCentralRegistry

docker_cli_pull_test.go:117:
    s.Cmd(c, "pull", "--all-tags=true", "busybox")
docker_hub_pull_suite_test.go:73:
    c.Assert(err, checker.IsNil, check.Commentf("%q failed with errors: %s, %v", strings.Join(arg, " "), out, err))
... value *exec.ExitError = &exec.ExitError{ProcessState:(*os.ProcessState)(0xc820b844a0)} ("exit status 1")
... "--all-tags=true busybox" failed with errors: 1-glibc: Pulling from library/busybox
5b0268a1bf8b: Pulling fs layer
a3ed95caeb02: Pulling fs layer

---

LGTM, thanks @vishh

retest this please


Workflow tox support

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature

What is the current behavior? (You can also link to an open issue here)
no automated testing

What is the new behavior (if this is a feature change)?
automated testing

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
no

Other information:

Does this change mean that running the workflow would both run tests and publish to PyPI? It may be good to separate the workflows so we can do one or the other.

we need Marqo to be up and running in order to run the client's unit tests properly. This would make the workflow look closer to the API tests


Add explanation of the colors

It's not clear what the all the different text colors signify, can you add a table or something to describe them like linPEAS does?

Great idea! Will add this in, thanks @raind33r

Added in latest version


Bug: Filter status overlaps

Idris will add text and pics

Located the bug: It's happening because the z-index value is set to 10. I think the z-index can be removed. Can't find "z-index: 10" in the code. Mabye this comes from EDS

PR: https://github.com/equinor/lighthouse-client/pull/1687


Error while installing to Centos 7 bigtop-utils not found

Hi, I got the following error while trying to install to a Centos 7 machine:
TASK [common : install packages] ***********************************************
failed: [<IP_ADDRESS>] (item=[u'java-1.8.0', u'bigtop-utils']) => {"changed": false, "failed": true, "item": ["java-1.8.0", "bigtop-utils"], "msg": "No Package matching 'bigtop-utils' found available, installed or updated", "rc": 0, "results": []}

Please read documentation "Requirements" section:
"The required for Cloudera Hadoop repositories have to be properly configured on the target hosts"
bitop-utils package is a part of Cloudera Hadoop repository.

So are you saying that bigtop-utils installation on the target host isn't supposed to be handled by your playbook ?

I'm saying : "Cloudera Hadoop repositories have to be properly configured on the target hosts".
If the repository configured properly, the following command should provide a similar output:
% yum info bigtop-utils
Name        : bigtop-utils
Arch        : noarch
Version     : 0.7.0+cdh5.9.0+0
Release     : 1.cdh5.9.0.p0.30.el7
Size        : 11 k
Repo        : extras
Summary     : Collection of useful tools for Bigtop
URL         : http://bigtop.apache.org/
License     : ASL 2.0
Description : This includes a collection of useful tools and files for Bigtop

Ok, thanks.


fix: only inject entry scripts as <script>

Currently we are adding <script> tags with dynamic entries, e.g. they export an entry. The only effect of this should be to run any side-effects. I think we should omit these and just add them to the link preloads. They will be imported from the entry-file instead.

I think this comes from the webpack requirement of self-registering chunks. Can you please try if webpack (dynamic chunks) works fine with this change?
Also what are benefits of not doing this? Less HTML size?

Yes, that does make sense. I'll confirm, but think we would need to ensure we don't set isDynamicEntry when normalizing webpack manifests if so.
The issue actually manifested in that sometimes vite chunks depend on the entry already having been run (see https://github.com/nuxt/nuxt/pull/18642 for example). Although that one was fixed, it feels better to evaluate the chunks in the order they expect to run.

Thanks for explaining. Executing in order makes sense for vite indeed (webpack chunks in contrary self-register and evaluate in order by manifest)

Codecov Report

Merging #46 (5df5359) into main (fe3a42a) will increase coverage by 0.04%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   95.25%   95.30%   +0.04%
==========================================
  Files           6        6
  Lines         527      532       +5
  Branches      114      115       +1
==========================================
+ Hits          502      507       +5
  Misses         25       25

Impacted Files
Coverage Δ

src/runtime.ts
95.81% <100.00%> (+0.04%)
:arrow_up:

src/types.ts
92.00% <100.00%> (+0.33%)
:arrow_up:

src/webpack.ts
92.50% <100.00%> (+0.06%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Request] Chisels & Bits compatibility for E.IO glass

Hi guys,
As time and motivation permit, I'd like to be able to use Ender IO's glass with the Chisels & Bits saws and chisels. AlgorithmX2 says: "they can return true for isFullBlock, which in my testing seems to work fine for glass, or they can send an IMC to C&B to tell C&B that the block can be chiseled."
Full reply on this ticket: https://github.com/AlgorithmX2/Chisels-and-Bits/issues/135
Thanks!

Affected Versions (Do not use "latest"):

EnderIO: 1.10.2-<IP_ADDRESS>_beta
EnderCore: 1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>3
Chisels and Bits: 12.1

Thank you for your suggestion. It has been added to our list or possible enhancement.
This ticket will be closed because we want the list of open tickets to reflect known issues.


simple swiftui -  i want to add supabase as dependency - mostly fine but.....

the sample code to use apple login with supabase mentions requiring adding the package dependency supabase - but it doesn't mention gotrue.
I add this (have normally always used cocoapods - old school) - but now it fails....
also had to explicitly make the package a dependency...

link to binary fixed things - my project has multiple targets - so this needs communicating in notes.


预览10M左右的word文档，预览不了

最大支持的文件体积是多大呀？怎么去配置呀？

@DAHU123  Apologies, I can't read what you have written.
But the error looks like a limitation of MSDoc Viewer, they must have a limit on file sizes.
How big is the docx file you are trying to upload?


[Event Request] Table 25 Vendor Ledger Entry - SetStyle

We want to apply a different implementation of the styles in the Customer Ledger Entries page(s).
Therefore we want to request the following Event Request.
CURRENT CODE:
procedure SetStyle(): Text
begin
    if Open then begin
        if WorkDate > "Due Date" then
            exit('Unfavorable')
    end else
        if "Closed at Date" > "Due Date" then
            exit('Attention');
    exit('');
end;

NEW CODE:
procedure SetStyle(): Text
var
IsHandled: Boolean;
Style: Text;
begin
    OnBeforeSetStyle(Rec,Style,IsHandled);
    if IsHandled Then
        exit(Style);

    if Open then begin
        if WorkDate > "Due Date" then
            exit('Unfavorable')
    end else
        if "Closed at Date" > "Due Date" then
            exit('Attention');
    exit('');
end;

NEW EVENT PUBLISHER:
[IntegrationEvent(false, false)]
local procedure OnBeforeSetStyle(var VendLedgEntry: record "Vendor Ledger Entry"; var Style: Text; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Using pretrained weights with Ninjax

Lets say you want to pass image obs with pretrained image encoders. How would that work with Ninjax. I have a class that loads a ViT model below that I had don't add to self.modules in Agent so that the parameters are not optimized. But it errors out at self.get('flax') saying 'ValueError: not enough values to unpack (expected at least 1, got 0)'
class ViT(nj.Module):
  num_heads: int = 6
  embed_dim: int = 384
  mlp_ratio: int = 4
  img_size: int = 70

  def __call__(self, x, training=False):
    if nj.creating():
      from .vit import ViT
      from .dino_weights import load_vit_params
      vit_cls = functools.partial(
        ViT,
        num_heads=self.num_heads,
        embed_dim=self.embed_dim,
        mlp_ratio=self.mlp_ratio,
        depth=12,
        img_size=self.img_size,
      )
      vit_def = vit_cls()
      state = vit_def.init(nj.seed(), x)

      vit = torch.hub.load('pytorch/vision:v0.14.0', 'vit_b_16', pretrained=True)

      state['params'] = load_vit_params(state['params'], vit)
      self.module = vit_def
      self.put('flax', state)
    state = self.get('flax')
    return self.module.apply(state, x, training)

/zfs/aditya/workspace/dreamerv3/dreamerv3/nets.py:309 in __call__                                  │
│                                                                                │
│    306 │     state['params'] = load_vit_params(state['params'], vit)                             │
│    307 │     self.module = vit_def                                                               │
│    308 │     self.put('flax', state)                                                             │
│ ❱  309 │   state = self.get('flax')                                                              │
│    310 │   return self.module.apply(state, x, training)                                          │
│    311                                                                                │
│    312 class ViT(nj.Module):                                                                     │
│                                                                                │
│ /zfs/aditya/workspace/dreamerv3/dreamerv3/ninjax.py:467 in wrapper                               │
│                                                                                │
│   464   def wrapper(self, *args, **kwargs):                                                      │
│   465 │   with scope(self._path, absolute=True):                                                 │
│   466 │     with jax.named_scope(self._path.split('/')[-1]):                                     │
│ ❱ 467 │   │   return method(self, *args, **kwargs)                                               │
│   468   return wrapper                                                                           │
│   469                                                                                │
│   470                                                                                │
│                                                                                │
│ /zfs/aditya/workspace/dreamerv3/dreamerv3/ninjax.py:496 in get                                   │
│                                                                                │
│   493 │     return self._submodules[name]                                                        │
│   494 │   if path in context():                                                                  │
│   495 │     return context()[path]                                                               │
│ ❱ 496 │   ctor, *args = args                                                                     │
│   497 │   if 'name' in inspect.signature(ctor).parameters:                                       │
│   498 │     kwargs['name'] = name                                                                │
│   499 │   value = ctor(*args, **kwargs)                                                          │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
ValueError: not enough values to unpack (expected at least 1, got 0)

It seems that the tracing works fine but after the initial checkpoint is saved and then executed with the policy this just breaks.

Hey! There are a couple of options:

The pattern you tried, except that the method to read the state entries is self.find(regex):

def __call__(self, x):
  if nj.creating():
    self.put({'foo': some_pytree})
  some_pytree = self.find('foo')

There is also the self.get(name, ctor, *args, **kwargs) shorthand that calls the constructor once and afterwards returns the existing result:

def __call__(self, x):
  some_pytree_or_module = self.get('foo', self._make_foo)

def _make_foo(self):
  return some_pytree_or_module

Ninjax provides a native Flax integration and it can be extended for loading pretrained parameters along the lines of:

def FromFlaxPretrained(make_flax_module, post_init):

  class FlaxModule(nj.Module):

    def __init__(self, *args, **kwargs):
      self.module = make_flax_module(*args, **kwargs)
      self.treedef = None

    def __call__(self, *args, **kwargs):
      if creating():
        state = self.module.init(nj.seed(), *args, **kwargs)
        post_init(state)
        state, self.treedef = flatten(state)
        self.put(state, prefix=True)
      state = nj.unflatten(self.find(), self.treedef)
      return self.module.apply(state, *args, **kwargs)

  return FlaxModule

import torch
from .vit import ViT
from .dino_weights import load_vit_params

def populate_params(state):
  vit = torch.hub.load('pytorch/vision:v0.14.0', 'vit_b_16', pretrained=True)
  state['params'] = load_vit_params(state['params'], vit)

ViTB16 = functools.partial(ViT, num_heads=6, embed_dim=384, mlp_ratio=4)
net = FromFlaxPretrained(ViTB16, populate_params)(img_size=70, name='net')

I don't have time to test any of this but if you run into issues, please let me know.

Thanks @danijar, this is really helpful. I figured that the reason the policy network was erroring out is because I had added a extra pretrained module before the encoder and the policy_keys didn't include 'pretrained' as the regex key. I added the module separately since it was much easier for the optimizer to ignore the whole module rather than individual variables inside a module while backproping.

Also, I had to also disable the explicit garbage collection when the policy parameters were swapped otherwise the pretrained network parameters were getting deleted since they weren't new arrays created by the optimizer and referred to already loaded arrays.
   with embodied.timer.section('swap_params'):
      with self.policy_lock:
        if self.pending_sync:
          old = self.policy_params
          self.policy_params = self.pending_sync
          #jax.tree.map(lambda x: x.delete(), old)
          self.pending_sync = None


DE: Issue with window mangers started via SDDM

Description
If I start Openbox or Fluxbox from the SDDM login manager, DE is displayed as
DE: /usr/share/xsessions/fluxbox
DE: /usr/share/xsessions/openbox
That's what is in $DESKTOP_SESSION
Neofetch version
git
Possible  solution
change Line 637 to:
[[ "$wm" && "$de" =~ $wm$ ]] && { unset -v de; return; }

But I'm not sure if using $wm$ instead of ^$wm$ could break something else.

Should be fixed in this commit.