feat: Add support for Aggregated ClusterRole for cluster scoped instances

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
This PR enables user to use aggregated cluster roles for application controller in a cluster scoped instances of Argo CD.
This will create a base cluster role (argocd-argocd-application-controlle) which can inherit permissions from these two other cluster roles.

argocd-application-controller-view = It configures View permissions.
gitops-application-controller-admin = It is also an  aggregated cluster role to configures Admin permissions. This can inherit permissions from cluster role that is created by users and add those into base cluster role.

Which issue(s) this PR fixes:
Fixes #?
This is tracked in Red Hat's issue database here: https://issues.redhat.com/browse/GITOPS-2615
How to test changes / Special notes to the reviewer:

Create a cluster-scoped Argo CD instance
Create Argo CD CR in namespace and set Spec.aggregatedClusterRolesEnabled to true.
Create a user-defined cluster role having these two labels  argocd/aggregate-to-admin: 'true' and app.kubernetes.io/managed-by: argocd and configure permission you want to add for application controller component.

For E2E tests, I think you mentioned that we wouldn't be able to add them without Siddesh's Operator Image E2E Test PR. Can you provide more info on that? For this current PR, #1419, since we are not reading from a subscription, I would think we could at least add E2E test enabling/disabling AggregatedClusterRoles for a single ArgoCD instance, and perhaps disabling of cluster roles. (You're right we wouldn't be able to test with some of the other subscription-based features)

@jgwest both aggregated or custom cluster roles features are applicable only for cluster-scoped instances, which is defined via Subscription resource env variable ARGOCD_CLUSTER_CONFIG_NAMESPACES, so even we set AggregatedClusterRoles and DefaultClusterScopedRoleDisabled fields in ArgoCD resource, it wont have any effect because cluster roles will not be reconciled. That means even if we write E2E tests for these they will not be able to test feature.
https://github.com/jparsai/argocd-operator/blob/aggregated-clusterrole/controllers/argocd/role.go#L310C2-L310C48
https://github.com/jparsai/argocd-operator/blob/aggregated-clusterrole/controllers/argocd/role.go#L388C5-L388C12

@jparsai Thanks for the detailed reply @jparsai, this makes perfect sense! I forget how much of the operator functionality is tied to those variables :sweat_smile:.


upstream gitlab adjustments causing instability on unpinned latest

we need to consider pinning the gitlab version and include upgrades as part of our maintenance release process. upstream adjustments are causing instability on the registration token collection process.

the gitlab installer script found here
https://github.com/kubefirst/nebulous/blob/b8942da74b7bf5b6730a69f966a7902f5afa491e/gitops/terraform/base/ec2/scripts/install_gitlab.sh
needs to pin to an explicit version, and we can bump versions to latest on each release
sudo apt-get install gitlab-ce=8.17.x-ce.0

implemented with #47 and fixed in 1.6.0


Lint Coverage of SMIME BRs version 1.0.0

[x] SMIME certificates SHALL have cRLDistributionPoints (<IP_ADDRESS>.b)(#742 )
[ ] Strict and Multipurpose SMIME certificates SHALL have the cRLDistributionPoints URI scheme as HTTP others are not permitted (<IP_ADDRESS>.b)
[x] Strict and Multipurpose SMIME certificate AIA fields: OCSP Responder "When provided, every accessMethod SHALL have the URI scheme HTTP." (<IP_ADDRESS>.c.1) (#777)
[x] Strict and Multipurpose SMIME certificate AIA fields: caIssuers "When provided, every accessMethod SHALL have the URI scheme HTTP." (<IP_ADDRESS>.c.1) (#777)
[x] Key usage, RSA certs, strict policies: prevent all key usages other than digitalSignature, nonRepudiation, keyEncipherment (<IP_ADDRESS>.e) (#757 )
[x] Key usage, RSA certs, multipurpose/legacy policies: prevent all key usages other than digitalSignature, nonRepudiation, keyEncipherment and dataEncipherment (<IP_ADDRESS>.e) (#757 )
[x] Key usage, EC certs, all: prevent all key usages other than digitalSignature, nonRepudiation, keyAgreement, encipherOnly, decipherOnly (<IP_ADDRESS>.e) (#757 )
[x] Key usage, EC certs, all: encipherOnly/decipherOnly are permitted only when keyAgreement is set (<IP_ADDRESS>.e) (#757 )
[x] Key usage, Edwards certs, keys defined on curve 25519: Bit positions SHALL be set for digitalSignature and MAY be set for nonRepudiation. (#757 )
[ ] Key usage, Edwards certs, keys defined on curve 448: Bit positions SHALL be set for digitalSignature and MAY be set for nonRepudiation. Blocked due to lack of support for curve 448 in zcrypto
[x] Extended key usage, strict: emailProtection SHALL be present.  Other values SHALL NOT BE PRESENT (<IP_ADDRESS>.f)(#747)
[x] Extended key usage, multipurpose/legacy: emailProtection SHALL be present. Other values MAY be present  (<IP_ADDRESS>.f)(#747 )
[ ] Extended key usage, all: serverAuth, codeSigning, timeStamping, anyExtendedKeyUsage SHALL NOT BE PRESENT (<IP_ADDRESS>.f)
[ ] authorityKeyIdentifier, all: SHALL be present, SHALL NOT be critical.  keyIdentifier SHALL be present, authorityCertIssuer and authorityCertSerialNumber SHALL NOT be present (<IP_ADDRESS>.g)
[x] subjectAlternativeName, all: SHALL be present (<IP_ADDRESS>.h)(#744)
[x] subjectAlternativeName, all: SHOULD NOT be marked critical unless subject field is empty (<IP_ADDRESS>.h)(#746)
[ ] subjectDirectoryAttributes, strict/multipurpose: field is Prohibited (<IP_ADDRESS>.j)
[ ] subjectDirectoryAttributes, legacy: if present, field must not be marked Critical (<IP_ADDRESS>.j)
[ ] qcStatements, all: if present, field must not be marked Critical (<IP_ADDRESS>.k)
[ ] Legal Entity Identifier, mailbox-validated/individual-validated, all generations: is Prohibited (<IP_ADDRESS>.l)
[ ] Legal Entity Identifier, organization-validated, all generations: LEI (<IP_ADDRESS>.4.1.52266.1) MAY be present and SHALL NOT be marked critical (<IP_ADDRESS>.l)
[ ] Legal Entity Identifier, sponsor-validated, all generations: LEI (<IP_ADDRESS>.4.1.52266.1) or for role (<IP_ADDRESS>.4.1.52266.2) MAY be present and SHALL NOT be marked critical (<IP_ADDRESS>.l)
[x] Adobe Extensions, strict: is Prohibited (<IP_ADDRESS>.m) (#763)
[ ] extensions:subjectAltName all validations, all generations: SHALL be present, SHALL contain at least one GeneralName entry of the following types: Rfc822Name, otherName of type id-on-SmtpUTF8Mailbox (<IP_ADDRESS>.1)
[ ] subject:commonName, mailbox-validated: if present, this attribute SHALL contain... [a] Mailbox Address (<IP_ADDRESS>.2.a)
[ ] subject:commonName, organization-validated: if present, this attribute SHALL contain... subject:organizationName or [a] Mailbox Address (<IP_ADDRESS>.2.a)
[ ] subject:commonName, sponsor-validated/individual-validated: if present, this attribute SHALL contain... Personal Name, subject:pseudonym, or [a] Mailbox Address.  PersonalName SHOULD be presented as subject:givenName and/or subject:surname (<IP_ADDRESS>.2.a)
[ ] subject:commonName, all: if present, the Mailbox Address SHALL contain a rfc822Name or otherName value of type id-on-smtpUTF8Mailbox from extensions:subjectAltName (<IP_ADDRESS>.2.a)
[ ] subject:givenName, subject:surname, subject:pseudonym: The subject:givenName and/or subject:surname SHALL NOT be present if the subject:pseudonym is present. (<IP_ADDRESS>.2.e/f)
[x] subject:emailAddress, all: if present, the subject:emailAddress SHALL contain a single Mailbox Address. (<IP_ADDRESS>.2.h)(#752 )
[ ] subject:countryName, all: SHALL contain the two letter ISO country code or 'XX' if no ISO 3166-1 code has been assigned (<IP_ADDRESS>.2.n)
[x] subject DN attributes for mailbox-validated profile (<IP_ADDRESS>.3)(#713 )
[ ] subject DN attributes for organization-validated profile - Legacy (<IP_ADDRESS>.4)
[ ] subject DN attributes for organization-validated profile - Multipurpose (<IP_ADDRESS>.4)
[ ] subject DN attributes for organization-validated profile - Strict (<IP_ADDRESS>.4)
[ ] subject DN attributes for sponsor-validated profile - Legacy (<IP_ADDRESS>.5)
[ ] subject DN attributes for sponsor-validated profile - Multipurpose (<IP_ADDRESS>.5)
[ ] subject DN attributes for sponsor-validated profile - Strict (<IP_ADDRESS>.5)
[ ] subject DN attributes for sponsor-validated profile - Multipurpose/Strict: profiles SHALL include either subject:givenName and/or subject:surname, or the subject:pseudonym (<IP_ADDRESS>.5)
[ ] subject DN attributes for individual-validated profile - Legacy (<IP_ADDRESS>.6)
[ ] subject DN attributes for individual-validated profile - Multipurpose (<IP_ADDRESS>.6)
[ ] subject DN attributes for individual-validated profile - Strict (<IP_ADDRESS>.6)

Hello,
I'd like to help implement some of these lints. Would you be open to my contributions? I'm working on launching S/MIME certificates at my organization, and we'd like to be able to use zlint to lint S/MIME certificates, as we do with TLS.
I've already sent a PR (#779) for one of the lints on the list.

I think
Extended key usage, all: serverAuth, codeSigning, timeStamping, anyExtendedKeyUsage SHALL NOT BE PRESENT (<IP_ADDRESS>.f)

was handled by #747. https://github.com/robplee/zlint/blob/1018dcd6368fbbc65846bc3b721aa83092aeb863/v3/lints/cabf_smime_br/smime_legacy_multipurpose_eku_check.go#L66 compares the EKU against the list of forbidden EKUs.


Join Flutter's showcase

Hi everyone,
We are currently working on a Flutter based application and we would like to be integrated to the showcase.
Does anyone know how to do that ?
Thanks

Hi @bfarkas-sully
Consider to create the issue in the web-site repository
https://github.com/flutter/website/issues


Adding hours sometimes interferes with the existing hours

When adding a block of time to the left of another, this causes the block on the right to merge into the block on the left

fixed by adding a useState which prevents drag/updates when an event is in the process of being added


Docker

Let's face it: Python is a hot mess 😄
I'm starting to play with Docker; but fair warning: I am a rank noob.  I'm hoping Docker-knowledgeable people will help my learning curve.
The basic idea is to provide a Dockerfile to serve as a recipe for an environment to build and update hosts files and variants.
We could also create and link to a ready made Docker container that contains everything in the repo, bundled with Python, ready to roll.
One concept I already like:  Initialize the Docker container with git clone --depth 1 which makes for a much smaller package.
I'm playing with this now in the docker branch.  Come help!

I'm happy with this for now, implemented in Release 3.1.0.


instrument JDBC fetches

Is your feature request related to a problem? Please describe.
We are currently running SQLs from germany against a database in another country. We had performance problems and didn't know where they came from so we looked at the APM UI in Kibana. Unfortunately, this was not helpful at all as the SQL finished in less than a second but the whole operation took 4 minutes. After digging into the code and narrowing the problem down it seems the 4 minutes are taken by the fetches to the database which is currently not shown by APM. After changing the fetchSize we were down to 7 seconds. Is it possible to show the JDBC fetches in the APM module?
Describe the solution you'd like
I would like that the fetches would be represented in the SQL Span but I guess this opens problems for others who want to see only the SQL execution. Maybe we can get a new Span for fetches where all fetches for an SQL are aggregated?

Currently, it's not possible but it's a great idea to add it. The problem is that AFAIK, there's no defined API we can instrument for "get next batch of items". ResultSet#next can either be answered from the already requested batch or it may result in a network call. One option would be to only create a span if the call to ResultSet#next took more than 5ms, for example.

After some thought, I think that if implemented, this should be optional and disabled by default.
ResultSet is an abstraction for a cursor. From tracing perspective, it's extremely difficult to handle properly. Assuming that cursor moves only forward and is used only for reading is oversimplication.
Assumption that performance could be analyzed for reads using this abstraction for majority of use-cases is wrong. Reducing fetch size would make invocation of next operation faster, while total processing time would increase. In fact, moving existing cursor forward on a fast database can not be measured using milliseconds. With small fetch sizes and reasonably fast db, filtering based on duration is likely to create spans only on garbage collection 🙂
What if you would trace the time took from aquiring a ResultSet to closing it? This would be duration that cursor was open, and potentially locked something. If you added cursor type and connections isolation level to trace, one could even use this information to diagnose potential locking issues.

@skorhone This is what we already did as a workaround and this is what pointed us to the fetch at last. We added the CaptureSpan annotation to the DAO function to get the time the complete db communcation required and added labels to see additional information.
I understand that this is not a common usecase but I still believe that in the time of global companies there are a lot of users who would benefit from this feature as data must be transferred between countries somehow.
Having said that I would be okay with closing this issue if you think that this is not worth investing time as we have a working workaround which we can use.

What if you would trace the time it took from aquiring a ResultSet to closing it?
Sounds like a good idea in general but there are some complications: There could be spans in between and the result set may be closed in a different thread.

Another option would be to create a metric for the duration of all ResultSet#next calls. The disadvantage is that you can't attribute it to a particular transaction or span.
Yet another idea would be to collect the total duration of the ResultSet#next calls at the transaction level which makes it possible to attribute that time into the db part of the breakdown metrics. Tracking that on the actual DB span is a bit tricky because the span might already be sent to the APM server at the time ResultSet#next gets called.

There is actually no guarantee that cursor is closed on commit either. I guess the correct way would be to create a span for each cursor operation, but I can't see that it would work in real world.
What if cursor was given it's own transaction?

With the release of span compression I would like to get the discussion here started again: When span compression is enabled all similar spans that are adjacient to each other are compressed into a single span. So

Select

Fetch
Fetch
...

would become:

Select

Fetch (100x)
...

So I guess it would be possible to instrument fetch without cluttering the UI and still having the total fetch time per SQL.
What do you think? Would it be feasible to only enable this instrumentation when span compression is enabled?


Swarm deployment

First, a bit of terminology:

management swarm: the initial swarm that hosts core services only (amplifier, elasticsearch, kibana, nats, etcd, agent, ampui)
account swarms: swarm deployed from the management swarm meant to host user stacks / functions.

For now we have two deployment options:

local (docker in docker). For the sake of simplicity we can share core services and user stacks in a single swarm (it's meant for one user, the developer);
AWS. That initial swarm is only supposed to host the core services. From there, a user can trigger the creation of a new account swarm, with its own set of core services. For billing purposes, that means that the user should provide her own AWS credentials that will be used by the bootstrap / InfraKit to deploy new resources (network resources, EC2 instances).

Most of what is described here can also be applied to a deployment on GCP, once we add this option.
Account swarms should be deployed in their own VPC, because:

we want to avoid IP exhaustion in the subnets
a VPC can be created in another AWS account

Running InfraKit in the management Swarm to create a full swarm in another VPC would mean that a configuration file for this swarm has to be maintained on the management swarm, the deployment operations will then consist in modifying this configuration file and deploying it with the help of an InfraKit container, or by running infrakit update operations (scale, etc) directly. The cluster ID will be needed to map the cluster to an account.
The credentials (a key id and a secret key) should only be shared with the InfraKit container (or instance plugin container), so a single container may not be a good idea for security reasons.
To be able to use secrets to share the credentials, InfraKit has to run as a service (global on the managers).
An InfraKit container on the management swarm for each account swarm means a Docker volume (or local mount) for each account swarm.
To make things simple, we can have the same cardinality for the organization and the account swarm, one account swarm for one organization, this way we can deploy the organization Docker registry as part of that swarm. The swarm nodes can then be configured with two registry mirrors: the main one, which contains the ATOMIQ images, and the organization one.

I think it would be better to talk about account swarms instead of user swarms. A user can create a swarm and an organization can create a swarm as well.

if an organization can create an organization then we can keep 1 swarm = organization or user

correct, terminology updated, thanks @bquenin

Then the http requests are routed using account name/id in the host name, at first stage by the haproxy in the management swarm which route to the right haproxy in the user swarm.
but on the user swarm the haproxy doesn't have to deal with account name/id considering it's unique for the user swarm

For the record, minute of the discussion with Nicolas about ES data sharing:
It seems better, instead to have one big ES on the manager swarm which receive all the data (logs, metrics) form all swarms, to have a dedicated ES on each swarm to store only the data of the swarm.
It make easier the data management, for instance when we remove a swarm, remove all the data or in term of accounting, the cpu / disc space stay associate to the user swarms and not shared by the manager swarm
however, the API logs and metrics will have to be updated in order to be able to get data from each user swarm using account name and we have to found a have to route api request to the dedicated ES


Unclear if analytics and calculations are done or not when a user opts out.

What I am missing here is a crucial bit of privacy information (mostly peace of mind), being wether or not the analytics are performed when a user opts out, regardless of wether or not they would have access to the dashboard etc.
Or as one of my users asked me: "Does the system still snoop around in my mailbox when I am opted out?"

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

ID: ede52900-0a1f-1713-9f9b-44af2fd64dbd
Version Independent ID: fc2b7ff3-f708-c5ec-15da-c2cf7e01c677
Content: MyAnalytics privacy guide - Workplace Intelligence
Content Source: WorkplaceAnalytics/MyAnalytics/Overview/Privacy-Guide.md
Product: mya
GitHub Login: @paul9955
Microsoft Alias: v-pausch

Hi dennish78,
I had wondered this myself and I contacted the product team to look into it. Here's what I found out:  After you opt out of MyAnalytics, MyAnalytics stops processing your personal data. However, other Microsoft 365 services or features (such as Cortana Briefing or Office.com) may continue to process this data to provide you with that functionality.
Let me know if you have further questions,
-- paul9955

Closing this for now. Please re-open if you have more questions.
Thank you, -- paul9955

Hello @paul9955,
I am researching MyAnalytics opt-out settings and wanted to clarify one of the aspects. You mentioned  that after user opts out of MyAnalytics, MyAnalytics stops processing personal data according to PG. Is this opt-out in MyAnalytics settings? Or is it opt-out through PrivacyMode parameter. This part of information is unclear in the documentation

Hi lenakat,
You're right -- that point was unclear. Yes, processing stops in both cases. It stops if the user opts-out voluntarily (by using Feature settings) and it also stops if an admin opts people out by using the PrivacyMode parameter to configure user access.
I appreciate your question; it helped us close a gap in the documentation. See, for example, Data processing consequences.
Let me know if you have any further questions,
-- paul9955


Add extra_site_dirs config option to solve #566

Hi, it's my first contribution to HTTPie.  Thanks for making a really great tool!
This is a possible solution for #566 by letting users specify extra directories for sys.path.

Coverage increased (+0.2%) to 93.9% when pulling 35554d3be4872ae39b199906a9e09682b77ea818 on matthew16550:add-extra-site-dirs into 55530c8c6de4865fa97b0b7bfd29c2985bb19451 on jakubroztocil:master.

@matthew16550 thanks for the PR. I think it's a really good idea. I'll review it thoroughly when i have the time.

Hi @jakubroztocil is this still relevant?  I haven't been paying any attention here.
If yes then I'd be happy to update the PR.

@matthew16550 thanks for the PR and for coming back! Yes, it is still relevant. Let me look through the old solution in the coming days and share my thoughts.

Thanks for your contribution @matthew16550, we decided to solve this problem similarly but in a different fashion in #1200. Closing in favour of that!


EF fromSQL throws ArgumentNullException

I'm using C# (.net core v2.2) and EF core. I have pretty old code that is retrieving a row from the database:
string id = "123";
using (var context = new UsersDb())
{
    var req = context.ScanRequests.FromSql($"EXEC GetById {id}").FirstOrDefault();
    // ...
}

GetById is a stored procedure, written in Azure SQL (basically - SQL Server).
This code started to throw an exception recently:
System.ArgumentNullException: Value cannot be null.
Parameter name: key
   at System.Collections.Generic.Dictionary`2.FindEntry(TKey key)
   at System.Collections.Generic.Dictionary`2.TryGetValue(TKey key, TValue& value)
   at lambda_method(Closure , ServiceProviderEngineScope )
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetRequiredService[T](IServiceProvider provider)
   at Microsoft.EntityFrameworkCore.DbContext.get_DbContextDependencies()
   at Microsoft.EntityFrameworkCore.DbContext.get_InternalServiceProvider()
   at Microsoft.EntityFrameworkCore.DbContext.get_DbContextDependencies()
   at Microsoft.EntityFrameworkCore.DbContext.get_Model()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.get_EntityType()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.get_EntityQueryable()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.System.Linq.IQueryable.get_Provider()
   at Microsoft.EntityFrameworkCore.RelationalQueryableExtensions.FromSql[TEntity](IQueryable`1 source, FormattableString sql)
   at ***.***Factory.<>c__DisplayClass0_0.<GetById>b__0() in /tmp/sources/SystemLayer/***/***.cs:line 29

This issue doesn't happen to me when I run the same code with the same parameters in a local environment. Also, in most cases it's also working on production servers. I guess it's at least 99.9% success rate.
The problematic line (line 29 from stacktrace) is this line:
var req = context.ScanRequests.FromSql($"EXEC GetById {id}").FirstOrDefault();

UPDATE 1
I thought maybe the problem is because of my mapping to the database - maybe the database schema has been changed without changing the EF accordingly. So I run the mapping command again:
Scaffold-DbContext "cs" ...

After re-generating the classes again, I found that no file has been changed since my last GIT commit. Therefor, my EF code is already aligned with the database schema.
UPDATE 2
I was running my code with EF Nuget package version 2.2.1. I saw that I can update the EF package to 2.2.2. I did it but it isn't solved the issue. Same behavior.
What to do? What else should I check?
Further technical details
EF Core version: netcoreapp2.2
Database Provider: Azure Sql
Operating system: Linux debian
IDE: Visual Studio 2017 15.9.4

This call stack looks a little strange, since we generally have ConcurrentDictionarys on this code path. @YehonatanB can you share your startup/configuration logic? Are you using a different DI container from the built-in one?


Improve extrinsic/event search by name

Add ability to search extrinsics and events by their name in fulltext manner.

The workflow above doesn't work for event because the event names are camelcase do the casing cannot be determined just from the input.
The new solution is to get the call/event names from chain runtime metadata and correct the casing against it. If the input is not found in the matedata the workflow above is used as fallback.


Improve doc comments based on review comments

Address review comments from https://github.com/apple/swift-syntax/pull/1670#pullrequestreview-1442885590.

@swift-ci Please test


DYM incorrectly suggests a class name that doesn't exist

from: https://twitter.com/schneems/status/976479296712781825

Has anyone seen this before? Ruby suggests the exact same constant I just tried to use.

cc @schneems

Released as 1.2.1.


Decide what to do about /-/inspect

It's not clear to me what this endpoint should do now as a result of #419 - it's still useful to be able to introspect databases for tools like datasette-registry, but since we aren't pre-calculating introspection data any more I need to rethink the approach.
For one thing, this endpoint may need to be paginated. Or maybe it should be split up into separate endpoints for each connected database? Those should probably be paginated too seeing as fivethirtyeight has 400+ tables.

I split this out from #462

The main use-case for this endpoint now is going to be Datasette Registry (which really needs some more love).
That tool needs to be able to query a Datasette and find out:

What tables are available
What their columns are
Ideally, their row counts

A single /-/inspect call is no good here because with Datasette Library #417 I'm going to be encouraging MUCH larger Datasette instances, potentially with hundreds of attached databases and thousands of attached tables. So pagination will be essential.
Maybe a smarter approach will be the older idea of having a separate inspect for each database (and maybe each table):

/mydatabase/-/inspect
/mydatabase/mytable/-/inspect

Either way, I'm going to decouple this from milestone 0.28.

Next step:

[ ] build /database/-/inspect which shows tables, columns, column types and foreign keys

It won't show table counts. Or maybe it will include them optionally but only for -I databases, in a special area of the JSON reserved for immutable-only inspect details.

I'm closing this because #531 counts as a decision.


comment-support setting

https://github.com/iissnan/hexo-theme-next/blob/master/README.en.md#comment-support
I found setting as the link above has no effect, but disqus_shortname: some_short_name works. Need to update the english README?

Yep. English docs is outdated.


[BUG] Crash when geoclue is not installed

Go Hass Agent Version
11:35PM INF Go Hass Agent: v9.3.0-75-gbfc0c89-dirty

Describe the bug
Crash at startup (SIGSEGV) when Geoclue2 is not installed.
To Reproduce
Steps to reproduce the behaviour:

Don't install Geoclue2.
Register using the GUI (go-hass-agent register --force)
go-hass-agent run
See error.

Expected behaviour
It shouldn't crash, and instead should just ignore the location sensor.
Logs
10:03PM WRN Could not activate a worker. error="unable to monitor for active applications: no portal present"
10:03PM WRN Could not activate a worker. error="unable to monitor for desktop settings: no portal present"
10:03PM WRN Could not activate a worker. error="unsupported hardware for laptop sensor monitoring"
10:03PM ERR Could not set up a geoclue client. error="SystemBus: unable to get data org.freedesktop.GeoClue2.Manager.GetClient from org.freedesktop.GeoClue2: The name org.freedesktop.GeoClue2 was not provided by any .service files"
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0xd19a9d]

goroutine 83 [running]:
github.com/joshuar/go-hass-agent/pkg/linux/dbusx.WatchBus({0x11b9198, 0xc00071c2d0}, 0x0)
	/home/whitelynx/Development/Other/go-hass-agent/pkg/linux/dbusx/dbus.go:250 +0x3d
github.com/joshuar/go-hass-agent/internal/linux.(*SensorWorker).Updates(0xc000716270, {0x11b9198, 0xc00071c2d0})
	/home/whitelynx/Development/Other/go-hass-agent/internal/linux/worker.go:137 +0x10d
github.com/joshuar/go-hass-agent/internal/agent.runWorkers({0x11b9160, 0xc00044d080}, {0x11b8f30, 0xc0005322e0}, {0x11b8f68, 0xc0005322d0})
	/home/whitelynx/Development/Other/go-hass-agent/internal/agent/runners.go:58 +0x3f1
github.com/joshuar/go-hass-agent/internal/agent.(*Agent).Run.func2.2()
	/home/whitelynx/Development/Other/go-hass-agent/internal/agent/agent.go:104 +0x65
created by github.com/joshuar/go-hass-agent/internal/agent.(*Agent).Run.func2 in goroutine 29
	/home/whitelynx/Development/Other/go-hass-agent/internal/agent/agent.go:102 +0x19a

Desktop:

OS: Linux
Distribution: Void
Version: rolling

Additional context
I encountered this because there's no go-hass-agent package on Void Linux, and when building manually, the documentation doesn't mention Geoclue2 being a prerequisite. Even after installing it, I'm unable to get Geoclue2 to actually run, so I'm still blocked by this error. (some of the specifics changed, of course - it now says Launch helper exited with unknown return code 253 instead of The name org.freedesktop.GeoClue2 was not provided by any .service files but the logs otherwise look practically identical)
For use on a desktop, it doesn't make sense to require a location sensor. (related to the first part of #171)

Firstly thank you for trying out Go Hass Agent and also reporting this issue!
I can see what is the issue here, I recently performed some code clean-up and refactoring that's introduced this bug.  Hopefully, commit aeca41062fa3db781ccc241bf272bd34640dd6a0 fixes the crash and commit acac8d74e14f216db3d2ebfe6297114e0365528e adds the functionality to not start a location sensor when not running on a laptop. Would you be able to try checking out the latest source, building and seeing if that fixes the issue for you?
Would it also be useful if the releases contained a binary archive? i.e., just an amd64 binary in a tar.gz? You'd still need to install the additional libraries the agent needs but this might make it easier on distros I don't have a package for?

Just following up, v.9.4.0 should fix the crash observed here. Also, a location sensor won't be created unless you are running on a laptop.
Please try out the latest version and reply back with the results when you can!

I pulled the latest (Updating bfc0c89..2b60928) and it seems to have worked. Thanks for fixing that so quickly!
As far as the location sensor being created, I am on a laptop, but I use it as a desktop - it's always plugged into an external keyboard, mouse, and monitor, and I don't move it. It would still be nice to be able shut off the location sensor, but I think #171 covers that. :)


Fix offline player check

Prevents players from giving horses to usernames with typos; getOfflinePlayer will return a value even if no player is found.

Looks like it should work easily enough.


[Merge release/1.4-ci to release/1.4 after #2316 is fixed] Test config of new PR checks (x64_debug for Slow, NoJIT, Legacy)

[ 6b2c4e72a ] 2017-01-05 18:11<EMAIL_ADDRESS>Test changes to CI.
[ 4b70ccc89 ] 2017-01-05 18:09<EMAIL_ADDRESS>Add one PR check per daily flavor as a quality gate.
[ 22676601f ] 2017-01-05 15:29<EMAIL_ADDRESS>Refactor CreateBuildTasks to call CreateBuildTask to allow re-use of that logic.

Compare changes which address #2229:
https://github.com/Microsoft/ChakraCore/compare/release/1.4...release/1.4-ci?w=1
/cc @obastemur @boingoing @Cellule
@boingoing: Once #2316 is fixed, please open a new PR from release/1.4-ci to release/1.4 and merge it in to enable coverage.

@dotnet-bot test ci please

@dotnet-bot test this please

Failures are due to #2316 - once resolved, this change works!

@dotnet-bot test ci please

@boingoing: Once #2316 is fixed, please open a new PR from release/1.4-ci to release/1.4 and merge it in to enable coverage.

#2316 is fixed so I'm going to merge in the new configs once I confirm there are no issues.
@dotnet-bot
test this please
test dailies please

@dotnet-bot
test slow tests please
test linux tests please
test dev12 tests please

This PR is stale. Replacing with actual PR from release/1.4-ci to release/1.4: #2432


podman run: implement --restart

Filing as separate issue because I don't want to clutter #482
Once restart is working, should the --restart option to podman run be implemented? If so, implement & document. If not, it might be wise to update podman-run(1) and remove the "incompatible with any restart policy other than none" statement from the --rm documentation.

We will not be implementing podman run --restart since we don't have a daemon. This should be done via systemd, if you want it.

Should change the manpages for run to remove things mentioning it, then

Manpages updated, closing


larger kernel_size, smaller distance

Hi Yingpeng, it is my honor to read your paper:Translation-Invariant attacks. I have tried your idea in my own task: face recognition, to generate some adversarial examples in a black-box manner, but I find the mean value of distance between the original pictures and the corresponding perturbed picture become smaller with the bigger kernel_size which is contrary to your result in the paper, I have not solve this problem yet. Would you mind giving me some suggestions to analysis this phenomenon? Thanks.
Best regards,
Looking forward to you reply.

Hi,
Sorry for the late reply. What is the threat model used in your experiments? If you choose the L-infty norm threat model, after you apply the sign gradient, the distance should be similar to that without translation-invariance. Or you may try using smaller kernel size.
Best,
Yinpeng

在 2020年12月15日，下午7:53，Suqingyong<EMAIL_ADDRESS>写道：
Hi Yingpeng, it is my honor to read your paper:Translation-Invariant attacks. I have tried your idea in my own task: face recognition, to generate some adversarial examples in a black-box manner, but I find the mean value of distance between the original pictures and the corresponding perturbed picture become smaller with the bigger kernel_size which is contrary to your result in the paper, I have not solve this problem yet. Would you mind giving me some suggestions to analysis this phenomenon? Thanks.
Best regards,
Looking forward to you reply.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dongyp13/Translation-Invariant-Attacks/issues/13, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2ZQAM3XT4NI6OMRV4RQYTSU5E45ANCNFSM4U4HD2FA.


Fixed #34656 -- Fixed unclosed div in admin password change template.

In auth/user/change_password.html there is a condition that checks existence of id_for_label for password1 field, the {% endif %} is closed in the wrong position and if the condition return False, the HTML will be corrupted (I'm not sure if it's possible to make the condition falsy)
Ticket Link

@Yaser-Amiri Thanks :+1: Can you add a regression test?

@felixxm I guess it should be somewhere like tests.admin_views.tests.AdminCustomTemplateTests.test_change_password_template
Because it's not about the form itself, it's about rendering. Am I write?

@felixxm I guess it should be somewhere like tests.admin_views.tests.AdminCustomTemplateTests.test_change_password_template Because it's not about the form itself, it's about rendering. Am I right?

It's an issue in an auth template not in the admin, so I'd add a test method to the tests.auth_tests.test_forms.PasswordChangeFormTest.

It's an issue in an auth template not in the admin, so I'd add a test method to the tests.auth_tests.test_forms.PasswordChangeFormTest.

Ahh, sorry, it's an issue in an admin template. Something similar to the tests.admin_views.tests.AdminCustomTemplateTests.test_change_password_template sounds good.

@felixxm Done!

@Yaser-Amiri Thanks :+1: Welcome aboard :boat:
I pushed small edits to test to avoid mocking.


Bug: In LostAndFoundRegistrationActivity

The LostPlace EditText shouldn't be focused at first.

I searched the problem on StackOverflow and found the way to solve it as following.
Add the following layout before the EditText.
<!-- Dummy item to prevent AutoCompleteTextView from receiving focus -->
<LinearLayout
    android:layout_width="0dp"
    android:layout_height="0dp"
    android:focusable="true"
    android:focusableInTouchMode="true" />

Some people say that add the focusable="true" and **focusableInTouchMode="true" to the EditText's parent layout will work. I tried it, but its parent layout contains another View which needs get String from resources. The added attributes breaked the getting process.


move source folder under docs

Hi Jeff, @jeff-regier , I believe this one won't conflict with the master branch

Thanks!


The Korean page is missing some comments from English page.

[Enter feedback here]
Hello,
In English page have "This is a known issue and we currently do not have an ETA for a fix." comments as the below capture.

But in Korean page have different comments with English page as "This is a known issue and we currently have an ETA for a fix."
The Korean comments is missing "do not have", it is very import missing, please fix it.

Thank you,
Best Regards
Yongchool Kim.

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

ID: 9aaf964f-63b8-c007-9724-4ab800df54a6
Version Independent ID: 6cbdb843-b282-0cf3-ea99-153dab004ac1
Content: Support for 32-bit operating systems in Azure virtual machines - Virtual Machines
Content Source: support/azure/virtual-machines/support-32-bit-operating-systems-virtual-machines.md
Service: virtual-machines
GitHub Login: @genlin
Microsoft Alias: genli

@Yongchool, Thanks for the feedback. I will forward this problem to our localization team.

Close the issue since the problem is fixed.


Undo-Button for Noptifications

I think an 'Undo' button would be useful for the notifications.
I often accidentially make something unavailable on the wordpress via renting and then have to manually search for the item and make it available again. it would be useful in this case to just being able to press 'undo' on the notification and reverse the action.

I don't think this is easily possible and even necessary, so I'll close the issue


skips importing legacy functions

takes care of https://github.com/CartoDB/cartodb/issues/13677

🚀


As a content editor, I want to control how the description of my editorial content is displayed when on PPA, when shared, and when searched.

notes for testing

content and editorial pages now have an optional formatted description field

if the description is not entered, the body text should be truncated for display as preview text

if description is used, it will be used for preview; if it's too long it will be truncated

if no action is taken, should use the first part of the content as description

should be able to optionally specify a formatted description

meta/search description should be a deformatted version of the formatted description, if present

Description field is present on content and editorial pages.
It is optional and supports formatting.
If a description is present, the description displays as preview text. If the description is too long, it truncates with an ellipses.
Search description is not populated in Wagtail but is in the html (RSK says this is correct).
If description is not present, the preview text is the first part of the body text BUT as Nick discovered ONLY if there is a single text block. Having multiple text blocks seems to mess this up, regardless of whether or not there is a list. Right now, if you have a second text block, the description is simply blank.

Wow, you found two different problems here!
Empty paragraph content (

) in the description wasn't being recognized as empty, and faulty logic meant I was grabbing the last paragraph block rather than the first.
I'll have a fix on the test site soon.

I added blank spaces to the description and the preview ignored them and is correctly drawing from the body text.
It's grabbing the first paragraph block now (rather than the last, and it correctly ignores images).


[Bug]Run a local network not working

🐛 Bug

The libra book https://developers.libra.org/docs/run-local-network stated that to start a local test net, just need to
$ cd libra
$ cargo run -p libra_swarm -- -s -n 4

but actually, the above code snippet is misleading and not working
To reproduce
Code snippet to reproduce
# Your code goes here
# Please make sure it does not require any external dependencies
cd libra
cargo run -p libra_swarm -- -s -n 4

Stack trace/error message
// Paste the output here
package `libra_swarm` is not a member of the workspace

Expected Behavior

libra swarm should start to run.
System information
Please complete the following information:

Additional context
Add any other context about the problem here.

I meet the same problem

🐛 Bug
The libra book https://developers.libra.org/docs/run-local-network stated that to start a local test net, just need to
$ cd libra
$ cargo run -p libra_swarm -- -s -n 4

but actually, the above code snippet is misleading and not working
To reproduce
Code snippet to reproduce
# Your code goes here
# Please make sure it does not require any external dependencies
cd libra
cargo run -p libra_swarm -- -s -n 4

Stack trace/error message
// Paste the output here
package `libra_swarm` is not a member of the workspace

Expected Behavior
libra swarm should start to run.
System information
Please complete the following information:

Additional context
Add any other context about the problem here.

I have resolved this bug, the cmd should be cargo run -p libra-swarm -- -s. It's libra-swarm not libra_swarm. I found the Cargo.toml's members have changed the libra_swarm to libra-swarm

@caesarchad Thanks for reporting. This seems like a regression from recent rename changes. cc/ @huitseeker

This was intended and it looks like we still have some documentation that needs to be updated to reflect the change. For context this change was done per #1119

Create https://github.com/libra/website/pull/104 to update the doc

Unsurprisingly, i met the same issue after upgrading my Mac OS.
Guys. you think this could be similar case ?
https://github.com/rust-lang/rustfmt/issues/3643

now i have another issue after latest  testnet upgrade.
target/debug/client --host ac.testnet.libra.org --port 8000 -s ./scripts/cli/consensus_peers.config.toml`
Not able to connect to validator at ac.testnet.libra.org:8000, error InvalidSignature

Above worked on Mac OS 10.15.1. thanks. hope somebody will update the website and close the issue.

I think i am going  to pull request for the latest due to simple document changes.


"Timeout getting IP address" on container with network "managed"

Rancher Versions:
Server: 1.4.1 (same with 1.3.x)
healthcheck: 0.2.3
ipsec: 0.0.4
scheduler: 0.4.0
Docker Version:
1.12.6 & 1.13.1
OS and where are the hosts located? (cloud, bare metal, etc):
Fresh install of Debian 8 bare metal (OVH)
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Clear install of Debian 8 on dedicated-server (Debian 8.7 stable (Jessie) Server HOST-64-H - 64G Xeon D-1540)
Install docker-engine 1.12.6 or 1.13.1
docker run -d --restart=unless-stopped -p 8080:8080 rancher/server
docker run -d --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /var/lib/rancher:/var/lib/rancher rancher/agent:v1.2.0 http://***:8080/v1/scripts/*******

Results:
"Timeout getting IP address" on stack "healthcheck", "ipsec" and "scheduler".
If I create a standalone container "redis" with network "bridge" it's start and run perfectly, but if I create one with network "managed" I have a error "Timeout getting IP address"
If you could give me any solution to get around this problem I will be very grateful 😀
(I have already tried { "dns": ["<IP_ADDRESS>", "<IP_ADDRESS>"], "dns-search": ["example.org"] } in daemon.json, and change DNS in resolv.conf
I also try to put the rancher host on another server)

@Kilhog can you try something, can you add an entry into /etc/hosts with the following <IP_ADDRESS> rancher-metadata and let us know if the services come up?

@aemneina Ok, I think you find something. So, when a restart the host, firstly I've had the "timeout" error, but when the stack automatically restart all the default stacks have a IP 😁 👍 .
But now 😆 ...
The healthcheck & scheduler stacks are blocked in "Initializing".

Healthcheck logs 14/02/2017 22:54:29time="2017-02-14T21:54:29Z" level=fatal msg="Get http://rancher-metadata/2015-12-19/version: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)"
Scheduler logs 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Listening for health checks on <IP_ADDRESS>:80/healthcheck" 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Subscribing to metadata changes." 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Connecting to cattle event stream." 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Initializing event router" workerCount=100 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Connection established" 14/02/2017 22:53:08time="2017-02-14T21:53:08Z" level=info msg="Starting websocket pings"

(Thank you very much for your quick reply 👍 )

Do you run rancher with 1 host (server and agent)? If you when adding the rancher-agent make sure to pass in the following with your hosts IP -e CATTLE_AGENT_IP=<ip_here>

Maybe a better question is, how many agent hosts do you have? Can they communicate with each other across UDP ports 500 and 4500?

I have only 1 ranger-host and 1 ranger-agent on the same server (this will not be the final production environnement, if you want tomorrow I can deploy ranger-server on another server if you think it can help), I tried with -e CATTLE_AGENT_IP=<public_ip> and -e CATTLE_AGENT_IP=<<IP_ADDRESS>> some result.
nmap (<IP_ADDRESS> or <publie_ip>) -p (500 or 4500) return closed, not filtered

I just test in production environnement like.
So I have a rancher/server on <IP_ADDRESS> and I make a host in <IP_ADDRESS> the  CATTLE_AGENT_IP will be <IP_ADDRESS>, If it's correct I have the same issue.
And iptables on <IP_ADDRESS> look good.
On <IP_ADDRESS> an nmap <IP_ADDRESS> -p 500 return closed

So,
If I go into the "healthcheck-1" & "ipsec-router-1" containers and I add <IP_ADDRESS> rancher-metadata in /etc/hosts all stack turned green.

This helps someone understand the problem ?

First, do net use docker 1.13.x, Pls see the SUPPORTED DOCKER VERSIONS for Rancher.
http://docs.rancher.com/rancher/v1.4/en/hosts/#supported-docker-versions
Second, just upgrade the infra services(ipsec/healthcheck/scheduler) to the latest version.

Please upgrade to latest stable with a supported Docker version and report back with the findings, so we can reproduce/troubleshoot. There shouldn't be a need of manual editing files to get stuff running.

Please file a new ticket if you're still experiencing issues.


merge sort in cpp

Can I take this up?

Yes @aquibbaig you can take up this.

Hi, I would like to have a Pull request but the implementation has already been done?

@sauravjaiswalsj  can i work on this?


Sort always time

Seems to be the list is not sorted by name.
I added one more change for Windows.

Ah, sorry. miss. I 'll forcibly update.

Thanks


code --inspect-brk-extensions is broken

start code via code --inspect-brk-extensions=9988
🐛 the extension is not paused

This happens because there is an explicit check for the extension host development case here: https://github.com/Microsoft/vscode/blob/eabec00675ec0bc6f47b911cf9a656d3c03ec0c9/src/vs/workbench/services/extensions/electron-browser/extensionHost.ts#L94-L99

Appart from not being able to debug the startup sequence anymore it also breaks --prof-startup

And you are seeing this issue in the case where no "extensionDevelopmentPath" is passed as a CLI argument, right?


Fix POM in 0.9.0

Starting a Gradle Daemon, 3 incompatible and 1 stopped Daemons could not be reused, use --status for details

FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'jenkins-pipeline-shared-library-example'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not find com.squareup.okhttp3:okhttp:.
     Required by:
         project : > com.mkobit.jenkins.pipelines.shared-library:com.mkobit.jenkins.pipelines.shared-library.gradle.plugin:0.9.0 > com.mkobit.jenkins.pipelines:jenkins-pipeline-shared-libraries-gradle-plugin:0.9.0

Looks like a bug in Gradle

Exactly what I was about to write you , thanks !

Plan is to reduce, reproduce, and report it to Gradle before fixing and releasing 0.9.1 today.

I think that part of the release flow should include the example repo upgrading and compiling with the new version , what do you think ?  would you like me to help in this  ?

Reported upstream issue to Gradle at https://github.com/gradle/gradle/issues/9565
Regarding https://github.com/mkobit/jenkins-pipeline-shared-libraries-gradle-plugin/issues/86#issuecomment-496546913 , I think that is a great idea.
In the past I've manually done this by

./gradlew publishToMavenLocal in this repository with some version
Update the example to use mavenLocal() repository
Update plugin version to match locally published one

Clean up local workspace to no longer use mavenLocal()
Go forward with actual version publication

I didn't do it this time, otherwise I would have caught this issue before publishing. It is a bit manual, which is probably why I didn't do it :laughing: . I've thought about this a little before, and my hope would be that it is a part of the Circle CI pipeline to get automated feedback. I think some combination of composite builds with --include-build could be useful. I created https://github.com/mkobit/jenkins-pipeline-shared-libraries-gradle-plugin/issues/87 for tracking that.

Released 0.9.1 and updated example in https://github.com/mkobit/jenkins-pipeline-shared-library-example/commit/0cfeda933e75d2dc7325ec93aa35bc59b6f181cd - closing


⚠ Bofrost.de has degraded performance

In 628a0ea, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1103 ms

Resolved: Bofrost.de performance has improved in d80dfde after 20 minutes.


Sleigh BitRange Registers (Bad Symbol Xml)

Describe the bug
All bitrange registers as described in the sleigh documentation section 4.5 compile successfully with sleigh. Upon loading the .sla file, ghidra throws throw new SleighException("Bad symbol xml"); in SymbolTable.java line 158. It appears that the below XmlElement is being parsed in the restoreSymbolHeader method which there is no case for. I'm not sure if sleigh has compiled it wrong, or if the symbol table class has not been updated to handle the bitrange registers.
XmlElementImpl@59 "<subtable_sym(2) name="instruction" id="0x0" scope="0x0" numct="57"> @(165:66)"

To Reproduce
Steps to reproduce the behavior:

Add the bitrange example in the sleigh documentation section 4.5 "Bit Range Registers" to a language of your choice.
Load a binary with said language definition.
See error

I added the example to 6502.slaspec and changed the offset of statusreg to 0x60. The resulting .sla file is relatively small so I will attach it below. The cause of the exception is line 165.
Attachments
6502.sla.tar.gz
Environment (please complete the following information):

OS: Windows 10 10.0 amd64
JVM Version: Oracle Corporation 12.0.2
Ghidra Version: Ghidra Version: 9.1 04f7366a62798068903252c729799a067d714412

I've run into this too. Same as #660 although that report seems to have gone fairly unnoticed.

This appears to have been resolved by a recent commit.


Is this ready for use

Hello,
Checking if this is ready to use and test? If it is ready, can we have some examples for how we can do this?

Hi @BibinQA
It is not ready to use. please follow Perfecto release notes to be updated once it released.
You should be able to find information on how to use this package in the README.md file

Do you have any ETA on this?

This integration is currently in Beta phase and only available in select private clouds. If interested, contact Perfecto Support.


style updates to market trading page mobile tables

@matt-bullock please review

More of a global update but all text items in the tabs should be 14px not 16px - Open Order, My FIlls, Positions etc

Arrows are #999999 for default state, white for hover

Numbers are 10px and #CED7D8. BUY, SELL remains 11px and green/red

Market should be medium weight. Looks bold currently

Visible indent on border should be removed when expanded

Should be mono 10px, #76737E

Should be 10px, #CED7D8

Vertical spacing should be 16px top and bottom

My fills looks like an older design

most of that was addressed here: https://github.com/AugurProject/augur/issues/1440
this ticket is just for the size of open orders table. there isn't a ticket for filled orders yet that i'm aware.


MINOR: Improve the org.apache.kafka.common.protocol code

Add UUID to the list of types documented in Type#toHtml.  This was overlooked in the change which added UUIDs as a supported KRPC type.
Type, Protocol, ArrayOf: use Type#isArray and Type#arrayElementType rather than typecasting to handle arrays.  This is cleaner.  It will also make it easier for us to add compact arrays (as specified by KIP-482) as a new array type distinct from the old array type.
Add MessageUtil#byteBufferToArray, as well as a test for it.  We will need this for handling tagged fields of type "bytes".
Schema#Visitor: we don't need a separate function overload for visiting arrays.  We can just call "visit(Type field)".
TestUUID.json: reformat the JSON file to match the others.
ProtocolSerializationTest: improve the error messages on failure.  Check that each type has the name we expect it to have.

retest this please

git failed on Jenkins:
14:20:27 	at org.jenkinsci.plugins.gitclient.CliGitAPIImpl.launchCommandWithCredentials(CliGitAPIImpl.java:1761)

retest this please


matterircd crashed - possibly because of idling

Version: 0.20.0 7954242
OS: Ubuntu 20.04.1 LTS
I ran into this crash today. If I recall correctly, I had been idling for some time, if that matters. The network itself is stable, so I don't think it's because of a flaky connection.
[7159]  INFO matterclient: reconnect successful
[7175] ERROR matterclient: got a ping timeout
[7175]  INFO matterclient: reconnect: logout
panic: send on closed channel

goroutine 115185 [running]:
github.com/mattermost/mattermost-server/v5/model.(*WebSocketClient).SendMessage(...)
	/go/pkg/mod/github.com/mattermost/mattermost-server/v5@v5.25.2/model/websocket_client.go:252
github.com/42wim/matterircd/pkg/matterclient.(*Client).doCheckAlive(0xc000262180, 0xc0016bef48, 0x2)
	/go/src/github.com/42wim/matterircd/pkg/matterclient/matterclient.go:500 +0xf1
github.com/42wim/matterircd/pkg/matterclient.(*Client).checkAlive(0xc000262180, 0xa3e420, 0xc003fad500)
	/go/src/github.com/42wim/matterircd/pkg/matterclient/matterclient.go:525 +0x12e
created by github.com/42wim/matterircd/pkg/matterclient.(*Client).checkConnection
	/go/src/github.com/42wim/matterircd/pkg/matterclient/matterclient.go:537 +0x6e

Let me also say that I am an extremely grateful fan of this project - matterircd is making my work life 1000% better - THANK YOU!

This is now probably fixed in master :-)


Maintenance of project

Seems the last commit merged was a couple of months ago and there's a couple of PRs waiting to be merged.
Could we get an update on when these could be merged or who would be reviewing them? /cc @shayne

Would also be nice with a feature on changing account when you have multiple.


unable to unselect a subdir

When picking which items to backup, I want to select my home dir but leave out one subdirectory. However, when I try to do this by clicking the subdirectory, tarsnap-gui unselects all the other things in the dir and selects only the directory I clicked. Is there some way to do this? Or perhaps just exclude certain dirs? Or is this more of a feature request? Thanks!

The QTreeView in use for the FilePicker is severely limited and has many issues, although I tried hard to make it behave as "normal" as possible.
Depending on the use case, it might be best to either make the selections manually or use the "Exclude using shell globbing patterns" functionality that I'm working on right now. This is going to master first of course and will be released along with version 0.7.

Exclude feature pushed in commit 3489c5d11d26b2bf97c4f298d578e9eda024e7b1.

Thanks! What do you think of doing a release?

My plan is to release v0.7 in the latter part of October, it will include a bunch of nice additions along with the exclude one and some really good fixes and improvements overall. So stay tuned!


Not working with node 8.9.3

When using dxwcmdesigns init i get the followinf error:

I can't run the command "npm install -g wcm-design.tar.gz"
I'm running on:
npm -v 6.4.1
node -v v8.12.0
This is the log generated:
0 info it worked if it ends with ok
1 verbose cli [ 'C:\Program Files\nodejs\node.exe',
1 verbose cli   'C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js',
1 verbose cli   'install',
1 verbose cli   '-g',
1 verbose cli<EMAIL_ADDRESS>]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose npm-session 83b49430f613bfe5
5 silly install loadCurrentTree
6 silly install readGlobalPackageData
7 silly fetchPackageMetaData error for<EMAIL_ADDRESS>request to https://registry.npmjs.org/nw failed, reason: self signed certificate in certificate chain
8 timing stage:rollbackFailedOptional Completed in 2ms
9 timing stage:runTopLevelLifecycles Completed in 1103ms
10 verbose type system
11 verbose stack FetchError: request to https://registry.npmjs.org/nw failed, reason: self signed certificate in certificate chain
11 verbose stack     at ClientRequest.req.on.err (C:\Program Files\nodejs\node_modules\npm\node_modules\node-fetch-npm\src\index.js:68:14)
11 verbose stack     at emitOne (events.js:116:13)
11 verbose stack     at ClientRequest.emit (events.js:211:7)
11 verbose stack     at TLSSocket.socketErrorListener (_http_client.js:387:9)
11 verbose stack     at emitOne (events.js:116:13)
11 verbose stack     at TLSSocket.emit (events.js:211:7)
11 verbose stack     at emitErrorNT (internal/streams/destroy.js:66:8)
11 verbose stack     at _combinedTickCallback (internal/process/next_tick.js:139:11)
11 verbose stack     at process._tickCallback (internal/process/next_tick.js:181:9)
12 verbose cwd C:\Users\0mcricer\Downloads\digexp-toolkit
13 verbose Windows_NT 10.0.10240
14 verbose argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install" "-g"<EMAIL_ADDRESS>15 verbose node v8.12.0
16 verbose npm  v6.4.1
17 error code SELF_SIGNED_CERT_IN_CHAIN
18 error errno SELF_SIGNED_CERT_IN_CHAIN
19 error request to https://registry.npmjs.org/nw failed, reason: self signed certificate in certificate chain
20 verbose exit [ 1, true ]

The above npm issues look similar to other npm install issues (with firewalls causing certificate issues during npm install) described in this npm thread here:  https://github.com/npm/npm/issues/7519

I downloaded the wcm-design.tar.gz file and install it correctly, but now give the following error:
Error: Can not find module 'internal / util / types'
at Function.Module._resolveFilename (module.js: 548: 15)
at Function.Module._load (module.js: 475: 25)
at Module.require (module.js: 597: 17)
at require (internal / module.js: 11: 18)
at evalmachine. : 31: 26
at Object.  (C: \ Users \ 0mcricer \ Downloads \ DXSyncCLI-1.0.2 \ node_modules \ graceful-fs \ fs.js: 11: 1)
at Module._compile (module.js: 653: 30)
at Object.Module._extensions..js (module.js: 664: 10)
at Module.load (module.js: 566: 32)
at tryModuleLoad (module.js: 506: 12)

Also @thezeeck - can you confirm whether you were running the dxwcmdesigns command utility when you hit this (which is what this git issue thread was for) or the dxsync utility?   The above dcm-design link was to fix this issue for the user that reported it here, for dxwcmdesigns.  If you're trying to use dxsync, that's a separate utility.   For dxsync, there's a proposed update to get that to work with newer Node versions here:  https://github.com/OpenNTF/WebDevToolkitForDx/blob/master/TempPatch/dxsync-1.0.3-1.tar.gz


Improve documentation for cuda support

Codecov Report

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

@@          Coverage Diff          @@
##           master    #44   +/-   ##
=====================================
  Coverage      89%    89%
=====================================
  Files          16     16
  Lines        1110   1110
=====================================
  Hits          988    988
  Misses        122    122

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 192549b...a7e846e. Read the comment docs.


RBAC DbManager getPermissionsByUser cannot get all permissions

I allot permission and role to user （insert into auth_assignment）
such as
item                user_id
permission1    1
permission2    1
role1               1
role2               1

role1 include permission3 and role2 include permission4 permission5
when i try to get all permissions by the method named getPermissionsByUser of DbManager
it just return the permission3 permission4 permission5
i viewed the source,in my opinion,This is because the code below：
protected function getChildrenRecursive($name, $childrenList, &$result)
    {
        if (isset($childrenList[$name])) {
            foreach ($childrenList[$name] as $child) {
                $result[$child] = true;
                $this->getChildrenRecursive($child, $childrenList, $result);
            }
        }
      // when i add the below code,i get all permissions exactly
       else {
            $result[$name] = true;
        }
    }

can you please put it little more effort in your post? You should take a loot at Markdown https://guides.github.com/features/mastering-markdown/ and put your code in code containers.

Formatted first post. Looks much more readable now.

I have the same problem. my case is much simpler though. I have a user and it has just one permission.
the permission is assigned to user directly, there are no roles involved:
Yii::$app->authManager->assign($permision, $userId);

getPermissionsByUser() method returns an empty array. looking at its code, it seems that it does not look for directly assigned permissions to a user. it just tries to return permissions that are inherited from an assigned role:
        $query = (new Query)->select('item_name')
            ->from($this->assignmentTable)
            ->where(['user_id' => (string) $userId]);

        $childrenList = $this->getChildrenList();
        $result = [];
        foreach ($query->column($this->db) as $roleName) {
            // $result is empty because there are no roles assigned to this user.
            $this->getChildrenRecursive($roleName, $childrenList, $result);
        }

        if (empty($result)) {
            // but this user has a direct permission, we should check for that before returning empty!
            return [];
        }

Also in the case @yrssoft described system is returning inherited permissions, but not directly assigned permission (permission1 and permission2).

Merged with some minor adjustments. Well done. Thanks!
https://github.com/yiisoft/yii2/commit/5569eaae061a8adbb222489c532c320baa673e87


Firegraph - GraphQL Superpowers for Firestore

Link to project (GitHub): https://github.com/sejr/firegraph
Describe the project in 2-3 sentences: Firegraph allows you to run GraphQL queries against a Firestore instance. Fetch related data (e.g. references, subcollections) and have a well-structured response ready for usage in your app immediately.
Why would Firebase developers be interested in this project?: Firegraph helps developers execute complex Firestore queries in a more expressive way. As the project moves forward, I hope to incorporate real-time queries, caching, mutations, and more. I would love for other Firestore users to help me maintain it.

@sejr so sorry don't know how I missed this!  Just add .opensource/project.json to your project and I am happy to add this to the site.

@sejr do you still want to add this repo to the site?  If so please add the missing config file!  If you need a good example look at the one in the Emberfire repo.

@samtstern Added!

@sejr added your project to the site!  You should see it appear in the next 24 hours in the 'web' section.


Fix b/21168971 Use a separate formatter without color for FileHandler

The CustomFormatter class remains public and largely unchanged (including looking for useColor config), so existing logging.properties files will still work.  The installers don't really "upgrade".  If you point it at an existing installation, Windows prompts you when overwriting existing files, so the user can choose to use their existing logging.properties or overwrite it with the new one.  The logging.properties files used by each installer are not not the ones in GitHub for each connector (or the library's).  They are currently in a set of input files used for each connector's installer.  They are all nearly identical, except for setting the logging level for each adaptor's package, and the name of log files.


Dalamud Support?

Any plans to make a dalamud plugin for high latency mitigation? Are any of the alternatives like DoubleWeaver or NoClippy any good from a reliability and safety point of view?

This program does not need to be a Dalamud plugin, and it won't be registered to the main plugin repository even if I made one, forcing you to use external repository. At that point, it isn't different much from installing XivAlexander separately.


fixing a lint error with slots

polymer lint must have gotten smarter with a recent release, because without this fix it fails now.


Show interactive Plotly image (html-js-based) on the Step 2 page

An example of simple html-js-based implementation of using Plotly.js is added

Currently based on develop branch of easyApp

Have you switch to develop branch for easyApp?

Actually, I did not. But now, after I have done so, I still have the same problem.

Problem was on my side. Works fine with easyApp on develop branch.


[UX][Store] Structure forget password element in login screen

Description:
Not fulfilling checklist items:
Consistency and Standards - Is flow maintain the same patterns and consistency?
Related task:
Login to store
Issue:
Forget password element should come before sign in button
Proposed Solution:

Suggested Labels:
UX, Improvement
Affected Product Version:
2.1.0 update 14

Closing the issue as this is older than 3 years. Please reopen if the issue is valid still.


Units per pixel should be set in the pixelCoordinateParams utility function

Otherwise, tile sets that aren't powers of two will be mis-centered by default.  The consumer of pixelCoordinateParams can still set unitsPerPixel, but this removes that necessity.

Current coverage is 86.86% (diff: 100%)

No coverage report found for master at 8f821ba.

Powered by Codecov. Last update 8f821ba...fbf2eab


Accessing swarm using API

Hi Guys,
Earlier when i am using swarm with containers i used to create certificates from CA and used to access swarm API like this:
export DOCKER_HOST=tcp://<DOCKER_IP>:2376
export DOCKER_TLS_VERIFY=1
export DOCKER_CERT_PATH=/root/.docker
Now in docker 1.12  swarm mode, it generates its own certs and places in /var/lib/docker/swarm/certificates
So now how can i run swarm command from remote client or from my mac.

There's two things being mixed up here. In "swarm" (not swarm mode), Swarm "wraps" the regular Docker remote API, and to talk to the swarm (instead of an individual daemon), the Docker Client has to connect to that API.
Swarm (https://docs.docker.com/swarm/) can still be used today (in docker 1.12 and up), and will function the same way as before.
If you're using Swarm mode (i.e., using the new "services" features) in Docker 1.12, no additional installation is required, only swarm mode has to be enabled. Documentation for Swarm mode can be found here; https://docs.docker.com/engine/swarm/
In Swarm mode, the Docker client connects to the remote API of a manager. To allow remote connections over https (instead of only through /var/run/docker.sock, the daemon has to be configured for this; https://docs.docker.com/engine/reference/commandline/dockerd/#bind-docker-to-another-host-port-or-a-unix-socket, and protected with TLS https://docs.docker.com/engine/security/https/. This is also the same as in older versions of Docker.

Now in docker 1.12 swarm mode, it generates its own certs and places in /var/lib/docker/swarm/certificates

This is where you got confused; these certificates are used by the manager nodes in the swarm to communicate with each other. This is an "internal" API, and you cannot connect to that API yourself.
Does this answer your question?

Hey thanks for the quick response.
Got clear idea now.

You're welcome; let me close this issue, but feel free to continue the conversation


"Outdated App" error on latest firmware 1.1.2

I tried compiling the basic template app and when running it on my Flipper that I just updated the firmware on, I get the same error as in #150 "Error: Outdated App". It just upgraded to FW version 1.1.2 (commit 54fd2819) released yesterday (Nov 11 2024).
Hopefully an easy fix but wanted to raise an issue so you were aware. Thanks in advance!

Hi @CrowhopTech. Thanks for opening an issue. The current git version supports the 1.1.2 firmware (#187), but we haven't tagged the 0.13.0 release (#188) quite yet.
Just waiting to see if there is any other PRs that can make it into this release, otherwise I'll tag 0.13.0 in the next day or so.

Thanks so much! Eagerly awaiting the release 😺

I just confirmed the sample app ran with the latest 0.13.0 release 🥳 will close this as it appears good.
Thanks for your quick response!

Thanks for the confirmation! Glad to hear it's working.


feat: nodes should only perform block proposals services if they know they are the leader

Overview
ATM, all nodes attempt to create and submit block proposals to each other, regardless if they are the leader or not.
The validator client should sync the current epoch state so a sequencer can know when it is their turn to produce blocks.

I think some of this will be addressed in #8122, as the sequencer won't make its way into publishing a block if it is not the leader so it don't necessarily deal with it in the validator client?
However, we should alter such that we only attest to proposals by the leader so some validation needed, not sure if this was what you meant.


Update .pullapprove.yml

This change is

Coverage remained the same at 85.185% when pulling 73a1713dde497c465a2697713f611e76fa7adb9b on update-pullapprove into 40be7b5b1466232ffea11f096f901f268a3e069d on master.


[Query] "You should have $0.01 balance to make transaction" error is seen when we try to tranfer "test token" higher than the available balance

Frequency: 100%
Repro on build version: Android Internal build V:1.24.1 ,iOS TestFlight build V:1.24.1
Repro on device : OnePlus 7t(11.0), Google Pixel 2XL(11.0), iPhone 12(14.7.1), iPhone 13mini(15.1.1), Google Pixel 10XL(10.0)
Pre-condition:
1] User must have installed the application
2] User must have completed onboarding process
3] User must have TT balance in the account
Repro Steps:
1] Tap on send button from home screen
2] Select a contact and go to enter amount page
3] tap on the "TT token" dropdown and complete the
Current Behavior: "You should have $0.01 balance to make transaction" error is seen
Expected Behavior: It should show error related to TT instead of local currency or cUSD
Attachment : Wrong_error

Upped the priority as this will prevent users from sending tokens without an associated price.

Downgrading the priority back, this is only a visual issue on the error text, if you send less than your max balance it works, this is just the error it shows if you try to send more than your balance


Tool not working

Just tried the tool after some period of time.
Unfortunately, wikidata-taxonomy fails with an error:
> wdtaxonomy Q634 -v
Error: SPARQL request failed
    at XXXXX\npm\node_modules\wikidata-taxonomy\lib\query.js:32:13
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

At my fresh installation the query works. Looks like a network error. You can emit the sparql query with wdtaxonomy Q634 -s and try at https://query.wikidata.org/.


[DIP] Fix DIP codegen for supporting multi-channel processing

This PR handles the codegen backend part.
Comprehensive fix to https://github.com/buddy-compiler/buddy-mlir/issues/164 is the end-goal.

This is a simple sanity test for verifying that multi-channel processing is working as expected. It can be tested by lowering the file and then using mlic-cpu-runner.
The lowering pass pipeline is as follows:
-convert-linalg-to-loops -lower-affine -convert-scf-to-cf -convert-scf-to-cf -convert-vector-to-llvm --finalize-memref-to-llvm -convert-func-to-llvm -reconcile-unrealized-casts


[CDS-55827]: Mark infrastructure tags as computed

Describe your changes
Since tags are a part of the schema but they come from the yaml, we have marked them as computed
Comment Triggers

PR Check triggers

Build: trigger build
Sub Category Field Check: trigger subcategoryfieldcheck

are all the infrastructure tests passing now?


Xbox and Playstation platform support?

We would potentially be interested in loading some USD assets at runtime on PlayStation and Xbox.  Any idea when this might be a possibility?

I'm asking around but I don't think we can legally provide USD binaries for those platforms. As of now we don't build for those platforms either but we might be able to provide build scripts if it's useful.

Thank you for the info.  I don't have immediate need for it but would be very interested to get access to the build scripts if console support can not be included with the package sometime in the new year.
Thanks again.  Happy holidays!

Hey! We've moved this feature request to our Roadmap for better internal visibility. If you head over to the Roadmap you can add more detail about your use case (on the 'USD Import ticket') for this feature to help us prioritize as we hope to be able to work on this in the future :)


[TMZ] Fix TMZ.com extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[ x ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[ x ] Searched the bugtracker for similar pull requests
[ x ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ x ] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ x ] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
This is a fix for the TMZ.com extractor. I did this on april 8 2020 but the PR never got merged in the original repo. Hopefully here it will be.
It seems TMZ started creating a new type of URL for its videos at the start of the year. These new URL's have some text before the ID, which broke the extractor.
Another problem was that the TMZ article pages changed their internal structure and the video URL is no longer in a json string but has to be obtained by parsing a JS line.
I tested the extractors today and they seem to be working OK.

Thanks for your work! Tried this video with your extractor file and it does not seam to be working.
https://www.tmz.com/videos/2020-10-31-103120-beverly-hills-protest-4878209/
maybe, tmz.py extractor file is not the only one that needs changes made?
Here's the log:
./youtube-dlc -v -F https://www.tmz.com/videos/2020-10-31-103120-beverly-hills-protest-4878209/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://www.tmz.com/videos/2020-10-31-103120-beverly-hills-protest-4878209/']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dlc version 2020.10.25
[debug] Python version 2.7.16 (CPython) - Darwin-19.6.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[Kaltura] st_4878209: Downloading video info JSON
ERROR: An extractor error has occurred. (caused by KeyError(u'dataUrl',)); please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "./youtube-dlc/youtube_dlc/extractor/common.py", line 532, in extract
ie_result = self._real_extract(url)
File "./youtube-dlc/youtube_dlc/extractor/kaltura.py", line 298, in _real_extract
data_url = info['dataUrl']
KeyError: u'dataUrl'
Traceback (most recent call last):
File "./youtube-dlc/youtube_dlc/YoutubeDL.py", line 830, in extract_info
ie_result = ie.extract(url)
File "./youtube-dlc/youtube_dlc/extractor/common.py", line 545, in extract
raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'dataUrl',)); please report this issue on https://github.com/blackjack4494/yt-dlc . Make sure you are using the latest version; type  youtube-dlc -U  to update. Be sure to call youtube-dlc with the --verbose flag and include its complete output.

I'll check it out. Thanks for the feedback.

Ok. It seems TMZ has changed again and introduced a new URL format for recent videos (october-2020 and newer). I'll try to add support to these new videos in another PR, but that doesn't mean that this PR has any defect. It adds support for videos posted between november-2019 and october-2020 which is useful.

Ok. It seems TMZ has changed again and introduced a new URL format for recent videos (october-2020 and newer). I'll try to add support to these new videos in another PR, but that doesn't mean that this PR has any defect. It adds support for videos posted between november-2019 and october-2020 which is useful.

Sure. Thanks a lot. Appreciate your help with this!

let me know when this is ready for review.

Hello again. I investigated the new structure of tmz.com and found they now use JSON-LD in all their pages, so I fixed the extractor to take advantage of it. I tested it with old and recent pages and it seems to be working fine.

Hello again. I investigated the new structure of tmz.com and found they now use JSON-LD in all their pages, so I fixed the extractor to take advantage of it. I tested it with old and recent pages and it seems to be working fine.

Wow! Brilliant! it seems to be working for every single TMZ link type.
But I've come across an interesting case. Not a big deal I guess, but still decided it's worth a highlight.
https://www.tmz.com/2020/11/05/gervonta-davis-car-crash-hit-and-run-police/
It has an embedded Youtube link and below is some "Hot Video" section. I'm not sure, if there's some correct way to handle this and we can leave it as it is.
Here's a log for it, if needed:
_./testdlc -v https://www.tmz.com/2020/11/05/gervonta-davis-car-crash-hit-and-run-police/
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.tmz.com/2020/11/05/gervonta-davis-car-crash-hit-and-run-police/']
[debug] Loading archive file None
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dlc version 2020.10.25
[debug] Python version 2.7.16 (CPython) - Darwin-19.6.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
[TMZ] https://www.tmz.com/2020/11/05/gervonta-davis-car-crash-hit-and-run-police/: Downloading webpage
WARNING: "url" field is missing or empty - skipping format, there is an error in extractor
Traceback (most recent call last):
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 174, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "./testdlc/main.py", line 19, in
File "./testdlc/youtube_dlc/init.py", line 487, in main
File "./testdlc/youtube_dlc/init.py", line 477, in real_main
File "./testdlc/youtube_dlc/YoutubeDL.py", line 2104, in download
File "./testdlc/youtube_dlc/YoutubeDL.py", line 846, in extract_info
File "./testdlc/youtube_dlc/YoutubeDL.py", line 901, in process_ie_result
File "./testdlc/youtube_dlc/YoutubeDL.py", line 1641, in process_video_result
IndexError: list index out of range

Very thorough testing! Thanks.
I added support for embedded youtube and twitter videos.
I'm guessing this PR is ready for review.


feat: rename flag move to components to move duplicates to components

This PR renames flag moveToComponents -> moveDuplicatesToComponents.

/rtm