How do you obtain valid x-august-api-key and x-kease-api-key

Trying to POST /session using postman but the return is "Could not get any response"
There was an error connecting to https://api-production.august.com/session.
Headers:
[
   {
      "key":"x-august-api-key",
      "value":"727dba56-fe45–498d-b4aa-293f96aae0e5", // I found this value on https://medium.com/@nolanbrown/august-lock-rest-apis-the-basics-7ec7f31e7874
      "description":"",
      "enabled":true
   },
   {
      "key":"x-kease-api-key",
      "value":"727dba56-fe45–498d-b4aa-293f96aae0e5", // I found this value on https://medium.com/@nolanbrown/august-lock-rest-apis-the-basics-7ec7f31e7874
      "description":"",
      "enabled":true
   },
   {
      "key":"Content-Type",
      "value":"application/json",
      "description":"",
      "enabled":true
   },
   {
      "key":"User-Agent",
      "value":"August/Luna-3.2.2",
      "description":"",
      "enabled":true
   },
   {
      "key":"Accept-Version",
      "value":"0.0.1",
      "description":"",
      "enabled":true
   }
]

Body:
{
  "installId": "b7e2efe6-dd2c-11e7-9296-cec278b6b50a", // random UUID
  "password": "<PASSWORD>",
  "identifier": "phone:+1<PHONE>"
}

Yeah i get the same long pause...not sure whats up with that.

@jdhrivas Sounds like a bug in postman. I will validate process again just to be sure.

@jmaxxz
Man, first thank you for this work! Second,  have you noticed that token returned recently seems not to "stick"? It works fine when both phone and email are validated, but (as I used mine with HomeBridge") when I restarted service and tried to use it again, I received 401. I think that August added an "aggree" to their authorization workflow, does it make sense?
this is decoded token I'm now getting with my homebridge plugin as well as your API ...
{
"installId":"E629CCCC-A9E0-40F1-8BB8-43A24830346B",
"applicationId":"",
"userId":"",
"vInstallId":true,
"vPassword":false,
"vEmail":true,
"vPhone":true,
"hasInstallId":true,
"hasPassword":true,
"hasEmail":true,
"hasPhone":true,
"isLockedOut":false,
"captcha":"",
"email":[
],
"phone":[
],
"expiresAt":"2018-09-01T15:21:11.015Z",
"LastName":"User",
"FirstName":"Anonymous"
}

How are you updating the token you are using? As for as I can tell everything works as it did before. If you are using postman be sure you are setting the authorization token to be used after each step in the login process.

What I posted is decoded token returned from your API. I refresh/use returned token every time, same as your code does. It appears that token remains “anonymous”. I observed that subscribing August in different apps (like ifttt) ends with “agree” page, which is not what either of us do. Posibble that’s the one returning cache-able user token? Thanks!

Ok, sorry, my bad. I meant tools from your repo using August rest api are returning the token I posted.

Post the raw token

This is what's returned when I run dst/init_env.js
eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpbnN0YWxsSWQiOiJFNjI5Q0NDQy1BOUUwLTQwRjEtOEJCOC00M0EyNDgzMDM0NkIiLCJhcHBsaWNhdGlvbklkIjoiIiwidXNlcklkIjoiZTIyODI5MDgtOGUwOC00YjFiLTgzMmUtMDk5NmFjNzY2NzQ5Iiwidkluc3RhbGxJZCI6dHJ1ZSwidlBhc3N3b3JkIjpmYWxzZSwidkVtYWlsIjp0cnVlLCJ2UGhvbmUiOnRydWUsImhhc0luc3RhbGxJZCI6dHJ1ZSwiaGFzUGFzc3dvcmQiOnRydWUsImhhc0VtYWlsIjp0cnVlLCJoYXNQaG9uZSI6dHJ1ZSwiaXNMb2NrZWRPdXQiOmZhbHNlLCJjYXB0Y2hhIjoiIiwiZW1haWwiOltdLCJwaG9uZSI6W10sImV4cGlyZXNBdCI6IjIwMTgtMDktMDRUMTQ6NTA6NTAuMjMzWiIsIkxhc3ROYW1lIjoiVXNlciIsIkZpcnN0TmFtZSI6IkFub255bW91cyJ9.1wylDfKXeGzhdpVDiJc_OD55S-DWMUANoUn4CSMs1s8

So it is not as you said, that token had a user id specified specified and works just fine.  However, from the token I can tell that you did not follow the directions and verify your email address.
Verify your email address. I would use https://lostphone.august.com/ to invalidate the token your previously shared as it was a working token, and would grant access to your account.

{
  "installId": "<REDACTED install id>",
  "applicationId": "",
  "userId": "<REDACTED user id>",
  "vInstallId": true,
  "vPassword": false,
  "vEmail": true,
  "vPhone": true,
  "hasInstallId": true,
  "hasPassword": true,
  "hasEmail": true,
  "hasPhone": true,
  "isLockedOut": false,
  "captcha": "",
  "email": [],
  "phone": [],
  "expiresAt": "2018-09-04T14:50:50.233Z",
  "LastName": "User",
  "FirstName": "Anonymous"
}

Thank you very much @jmaxxz! Apparently I was not able to sign in with phone number, only email when using iPhone app. After I "reset" my login as suggested, appears that token is now working as you said.
Thank you very much again!

Yes, I understood that, thanks for deleting that ...

Hi Guys specially my friend @jmaxxz ! How did you manage to get the x-kease-api-key 14445b6a2dba it doesn't seems to be valid anymore.

@federice  use 79fd0eb6-381d-4adf-95a0-47721289d1d9

Man you are a genius !!! How come you managed to get that so quickly ?
it's working fine.
Well done!! Many thanks

Fabio Federice
E<EMAIL_ADDRESS>On Fri, 22 Jun 2018 at 00:04, Jmaxxz<EMAIL_ADDRESS>wrote:

@federice https://github.com/federice use
79fd0eb6-381d-4adf-95a0-47721289d1d9
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jmaxxz/keymaker/issues/5#issuecomment-399306655, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Al2FMgWa_txp7XrXETIf-kG5N_WJXNa0ks5t_F6-gaJpZM4Q8SW0
.

@federice simply capture traffic between the august app and their webservers.

I have been trying to get this to work but am stuck on the "validate" email or phone.
I posted /validation/email and got a valid code sent to my e-mail.
I tried posting this to the following validate/email.
{
"email":"email-addresss",
"code": "code-from-email"
}

and I am getting the following error in the body:
{
    "userId": "user-id",
    "_value": "email:email-address",
    "resolution": "token_incomplete"
}

Does anyone have any idea what I'm doing wrong?

This message is actually what you get when its working.  You just copy the access token you get in response and use it in the rest of the API calls.  The token is now considered valid


RVC Documentation

I was wondering if there was any documentation available somewhere ? I wish to add this project to one of mine but i dont want the GUI , i just want to be able to load a model and a voice and transform it from a python file since i want to export it to AWS

https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI.wiki.git

hey @notnotniche I am working on same thing but on azure can we partner up to boost things up?

Yeah for sure, add me on discord Notnotniche !

https://github.com/RVC-Project/Retrieval-based-Voice-Conversion-WebUI.wiki.git

This doesnt seem to work ?

sure here is discord link- https://discord.gg/PmPwuqXA

https://docs.aihub.wtf/
(not official but for a fork)


Set checks to error instead of warning

Switch for all checks to be errors, and switch for error set of checks

To turn all warnings into errors, use scalac option -Xfatal-warnings. This is an issue for per-check configurability.


Issue with k8s.io/docs/tasks/administer-cluster/encrypt-data/

This is a Bug Report

Problem:
For the section of Rotating a decryption key,  would the first two steps be necessary?

Generate a new key and add it as the second key entry for the current provider on all servers
Restart all kube-apiserver processes to ensure each server can decrypt using the new key

Proposed Solution:
remove these two steps
Page to Update:
https://kubernetes.io/...

It seems to me that key rotation inevitable involves creating new key material; that step is fundamental to the meaning of the term “key rotation”.
Can you clarify why you're asking that question @YanzhaoLi?

It seems to me that key rotation inevitable involves creating new key material; that step is fundamental to the meaning of the term “key rotation”.
Can you clarify why you're asking that question @YanzhaoLi?

@sfabel  Yes. but why should we add it as the second key entry and restart all kube-apiserver processes? By my limited knowledge, I think the following four steps are ok:

Make the new key the first entry in the keys array so that it is used for encryption in the config
Restart all kube-apiserver processes to ensure each server now encrypts using the new key
Run kubectl get secrets --all-namespaces -o json | kubectl replace -f - to encrypt all existing secrets with the new key
Remove the old decryption key from the config after you back up etcd with the new key in use and update all secrets

If so, for the step three: kubectl get secrets --all-namespaces -o json would do the decryption with the second key (As illustrated in the article: "Each provider supports multiple keys - the keys are tried in order for decryption"), while the kubectl replace -f - would do the new encryption with the first new key.

The existing page says:

Changing the secret without incurring downtime requires a multi step operation, especially in the presence of a highly available deployment where multiple kube-apiserver processes are running.

I assume that the motivation for generating a new key and adding it as the second key entry is to avoid downtime.
If that's the case, I recommend leaving the existing documentation as-is.

/priority awaiting-more-evidence
/language en


rnaseq-cgl on aws gets No space left on device errors

In my attempt to reproduce https://github.com/BD2KGenomics/toil-scripts/issues/438 I ran 29 samples on seven c3.8xlarge nodes.
toil (3.3.0)
toil-scripts (2.0.9)

Command:
 toil-rnaseq run aws:us-west-2:jeltje-rnaseq-ckcc77 --batchSystem=mesos --mesosMaster mesos-master:5050 --config /home/mesosbox/shared/srconfig.txt --manifest /home/mesosbox/shared/srmanifest.txt
After a few hours I started getting no space left on device errors from tar commands, as well as errors like this:
StorageDataError: BotoClientError: Out of space for destination file /var/lib/toil/toil-bb25105a-3ae0-4357-a469-186974f25874/cache-bb25105a-3ae0-4357-a469-186974f25874/.NjYwM2MyNmQtZTU0Ny00MDQyLTgyZTAtODUxNjFkZTM1MGFm
And indeed, all my workers looked like this:
583G    /mnt/ephemeral/var/lib/toil/toil-bb25105a-3ae0-4357-a469-186974f25874/cache-bb25105a-3ae0-4357-a469-186974f25874
It seems that the cached files take over so much of the /mnt/ephemeral drive that there's not enough space left for output.
It took another 5 hours for the workflow to terminate (with 141 failed jobs).
After this failure, the caches on the workers were not cleared.
Only two of the samples finished, the rest failed.
I will leave one of the workers running in case you need more info.
srconfig.txt
srmanifest.txt
(large) sr_log.txt

I can't get onto the worker, I get permission denied when I try to ssh, via cgcloud, or normally.
cgcloud ssh --namespace /jeltje/ toil-worker
But I think I know what the issue is so I'll go ahead and shut down the node. Every job needs an explicit disk requirement with caching, where previously whole nodes were assigned to jobs via just core allocation. I'll fix this in 2.0.10.

Thanks! What version was 'previously'? This production run is nearing its deadline, so if I can finish it on an earlier version of toil/toil-scripts, that would be great.
Unless you plan on pushing out 2.10 today?

What version was 'previously'?

A version of Toil that doesn't have caching, not sure what version that is. I'm pushing out 2.0.10 now though.

Previous known good I'm pretty sure was 2.0.8. Once 2.0.10 has been run in production and there is a 2.0.10 based docker tested (I'll be happy to do this) then we should standardize on it for production runs going forward so we can shift energy toward other areas.

Previous known good I'm pretty sure was 2.0.8

For RNA-seq there is no difference between 2.0.8 and 2.0.9.


Change

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

Sam Napolitano seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.
Have you signed the CLA already but the status is still pending? Recheck it.

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement
Learn more about why HashiCorp requires a CLA and what the CLA includes

Sam Napolitano seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA. If you already have a GitHub account, please add the email address used for this commit to your account.
Have you signed the CLA already but the status is still pending? Recheck it.


GDPR cookie consent in Blazor

Could you explain how to implement GDPR cookie consent in a Blazor app? Something like: https://docs.microsoft.com/en-us/aspnet/core/security/gdpr?view=aspnetcore-3.1 but for Blazor?

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

ID: 28d0490e-205f-20a5-ace5-904f4c8ad78f
Version Independent ID: 4a2c8d95-fdd2-fdb1-b61d-802d0a8539c3
Content: General Data Protection Regulation (GDPR) support in ASP.NET Core
Content Source: aspnetcore/security/gdpr.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@schwabenlander It's not something that will work OOB at this time in a Razor component. You would need to roll your own component to do it.
I'll leave this on the Blazor.Docs project for triage this Friday. Artak will look :eyes: and advise if we should take on docs to do it (next year 🏃😅 ... or in 2022! :smile: lol), the framework should provide something, or else we'll leave it to the community to provide solutions.

@schwabenlander It's probably best if you ask the product unit directly about this. We don't have anything specific to document on it. The guidance right now would be that a dev should build their own component to handle it across their Blazor app. Open an issue for engineering if you still want feedback on this or to suggest that the framework offer something built-in for it ...
https://github.com/dotnet/aspnetcore/issues

You can use this GDPR Consent component from Nuget. However it is only helping you to save work on UI side. It does not restricts any Cookies or Tracking scripts.


KOGITO-7849: Patch PR for KOGITO-7422 with doc improvements in serverless logic web tools guides

JIRA: https://issues.redhat.com/browse/KOGITO-7849

Description: Patch PR for KOGITO-7422 with doc improvements in serverless logic web tools guides

https://github.com/kiegroup/kogito-docs/pull/160
Please make sure that your PR meets the following requirements:

[x] You have read the contributions doc
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting main: [0.9.x] KOGITO-XYZ Subject
[x] The nav.adoc file has a link to this guide in the proper category
[x] The index.adoc file has a card to this guide in the proper category, with a meaningful description

How to setup automatic cherry-pick to another branch ?

The cherry-pick action allows to setup automatic cherry-pick from main to a specific branch.
To allow it, you will need to add the corresponding label with pattern: backport-{RELEASE_BRANCH}.
For example, if a backport to branch 1.26.x is needed, then the label should be backport-1.26.x.
Once the PR is merged, the action will retrieve the commit and cherry-pick it to the desired branch.
NOTE: You can still add label after the merge and it should still be cherry-picked.

@thiagoelg Please review the changes made in this PR and share your comments.

@hmanwani-rh Thanks for this PR, looks great!
I've pointed out some things that I would change for clarification, let me know what you think.

@thiagoelg Thanks for reviewing the changes. I've made the changes as suggested. Please take a look and let me know if we can merge this PR.


[WIP] Data API cleanup

This cleans up some data implementations, especially where the requirements for data has been changed recently.

Final/Private values

ImmutableBreathingData
ImmutableFoodData
ImmutableMovementSpeedData

Fixing Values being created with uninitialized value

ImmutableBreathingData

Move Values to instance fields

ImmutableExperienceHolderData
ImmutableFlyingData
ImmutableFoodData
ImmutableHorseData
ImmutableIgnitableData

Caching usage

ImmutableFoodData

Fixing of defaults and bounds

ImmutableMovementSpeedData/MovementSpeedData
ImmutableHealthData/HealthData

Simplify AbstractImmutableBooleanData and AbstractImmutableBoundedComparableData to create value, rather than its children
Implement data getter/setter registration

TradeOfferData
ConnectedDIrectionData

@gabizou - thoughts on removing the final variables from immutable data, instead accessing them through the value?
eg
public int getMaxAir() {
    return maxAirValue.get();
}

Rather than
public int getMaxAir() {
    return this.maxAir;
}

@ZephireNZ I'd rather not. It's cheaper still to use field access than to call the value object to get the value.

Looks like there's some duplication between this and #234 - refactoring is gonna be soo fun...

@ZephireNZ Yeah.... considering that PR attempts to solve a lot of problems at the same time, it might take some time at the least. It might be divided into phases as @Mumfrey and I have pondered on some ways to resolve the issue of "stateless" CatalogType references while at the same time revamping the overall SpongeGameRegistry as a whole.

All well and fine. Just make sure it's rebased properly on master and I'd be happy to merge.


Bug: The simple injector DI container isn't practical

As originally reported by @chuuddo, they have provided unit tests to help produce the exception here:
https://github.com/reactiveui/splat/compare/master...chuuddo:rxui_initialization_breaks_simpleinjector_resolver
Details of their explanation:
SimpleInjectorDependencyResolver not works in real apps.
I made a pull request #239 with changes to View and ViewModel classes to start initialization of RxApp and all tests fails.
After some investigations i found this issues:

The container is locked after the first call to resolve but ReactiveUI invoke all registrations after RxApp.EnsureInitialized(). This can be fixed by not getting instace of MainWindow from container. If there are no other solutions, we should add this workaround to documentation.

    //var window = (MainWindow)container.GetInstance<IViewFor<MainViewModel>>();
    var window = new MainWindow();
    window.ViewModel = container.GetInstance<MainViewModel>();
    window.Show();

SimpleInjector support multiple registrations for same interface only through Collection.Register. We can't register this lines using _container.Register() method.

https://github.com/reactiveui/splat/blob/d27b0b48e71d38402c8e224b25e2dfff2daef875/src/Splat.SimpleInjector/SimpleInjectorDependencyResolver.cs#L38
I don't have any workarounds for second issue. Can we use this DI container with ReactiveUI?
Originally posted by @chuuddo in https://github.com/reactiveui/splat/issues/233#issuecomment-456362486

mental note: https://simpleinjector.org/ReferenceLibrary/?topic=html/Overload_SimpleInjector_Lifestyle_CreateProducer.htm

doesn't seem to cater for contracts.

There was a way to do named instances in Simple Injector.  I can't remember how off the top.  The library considers the concern a bad practice, but it still allows you to do it.  I'll see if I can pull an example. But your right it isn't idea.

Worse case scenario we just do a intermeditory generic collection with a
dict. Eg ContractContainer with extension methods for the simple
injector side.
On Sat., 3 Aug. 2019, 8:23 pm Rodney Littles II<EMAIL_ADDRESS>wrote:

There was a way to do named instances in Simple Injector. I can't remember
how off the top. The library considers the concern a bad practice, but it
still allows you to do it. I'll see if I can pull an example. But your
right it isn't idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/reactiveui/splat/issues/240?email_source=notifications&email_token=ABMQMMIM2YSS7QZHRTIAIALQCVMAXA5CNFSM4GRSDJG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3PL3XI#issuecomment-517914077,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABMQMMLPNKFVXUNFU5NKDDDQCVMAXANCNFSM4GRSDJGQ
.

Since using ReactiveUI + Splat, I find myself using the follwing (recommended) pattern over and over again:
public OAuthSettings(IConfiguration configuration = null)
{
    configuration ??= Locator.Current.GetService<IConfiguration>() ??
                      throw new ArgumentNullException(nameof(configuration));
    // ...
}

This bugs me because it's because of error-prone code duplication and the fact that this Splat pattern "bleeds" into every class using DI. It reminds me so much of MEF's [ImportingConstructor] attributes everywhere.
Using SimpleInjector makes above pattern obsolete because it fails upon registration (startup), so no NullReferenceExceptions during runtime because someone forgot to register something.
Using Splat.SimpleInjector makes Sextant unusable because RegisterParameterStackService makes a registration after resolving a service. From Stack Overflow:

Simple Injector locks the container after first use, and it does not allow the container to be unlocked. This behaviour is very deliberate and it is explained in detail here why making registrations after having resolved is a bad idea. Making registrations after you resolve is an anti-pattern that is sometimes referred to as Register Resolve Register (not to confuse with Register Resolve Release).

I wonder, is this just an issue with Sextant (should I open an issue?) or can the register-resolve-register pattern also be found in other ReactiveUI components?
Also, I find it hard to understand how SimpleInjectorInitializer works. Intuitively I tried registering to the SimpleInjector container using Splat's API. It took me a while to find out that SimpleInjectorDependencyResolver fails silently:
 public void Register(Func<object> factory, Type serviceType, string contract = null)
{
    // The function does nothing because there should be no registration called on this object.
    // Anyway, Locator.SetLocator performs some unnecessary registrations.
}

I tried looking into SetLocator but I can't find these unnecessary registrations. Was this changed? Can't we just throw a NotImplementedException, telling the developer to use the initializer instead? I did this because I use the UseSerilogFullLogger extension method and tried calling it after container.UseSimpleInjectorDependencyResolver(initializer).

Since using ReactiveUI + Splat, I find myself using the follwing (recommended) pattern over and over again:
public OAuthSettings(IConfiguration configuration = null)
{
    configuration ??= Locator.Current.GetService<IConfiguration>() ??
                      throw new ArgumentNullException(nameof(configuration));
    // ...
}

This bugs me because it's because of error-prone code duplication and the fact that this Splat pattern "bleeds" into every class using DI. It reminds me so much of MEF's [ImportingConstructor] attributes everywhere.
Using SimpleInjector makes above pattern obsolete because it fails upon registration (startup), so no NullReferenceExceptions during runtime because someone forgot to register something.
Using Splat.SimpleInjector makes Sextant unusable because RegisterParameterStackService makes a registration after resolving a service. From Stack Overflow:

Simple Injector locks the container after first use, and it does not allow the container to be unlocked. This behaviour is very deliberate and it is explained in detail here why making registrations after having resolved is a bad idea. Making registrations after you resolve is an anti-pattern that is sometimes referred to as Register Resolve Register (not to confuse with Register Resolve Release).

I wonder, is this just an issue with Sextant (should I open an issue?) or can the register-resolve-register pattern also be found in other ReactiveUI components?
Also, I find it hard to understand how SimpleInjectorInitializer works. Intuitively I tried registering to the SimpleInjector container using Splat's API. It took me a while to find out that SimpleInjectorDependencyResolver fails silently:
 public void Register(Func<object> factory, Type serviceType, string contract = null)
{
    // The function does nothing because there should be no registration called on this object.
    // Anyway, Locator.SetLocator performs some unnecessary registrations.
}

I tried looking into SetLocator but I can't find these unnecessary registrations. Was this changed? Can't we just throw a NotImplementedException, telling the developer to use the initializer instead? I did this because I use the UseSerilogFullLogger extension method and tried calling it after container.UseSimpleInjectorDependencyResolver(initializer).


[DOC-2612] Execution URLs for pipeline.

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://hdh.pr.harness.io/pr-856

Preview environment: https://hdh.pr.harness.io/pr-856

Preview environment: https://hdh.pr.harness.io/pr-856

Preview environment: https://hdh.pr.harness.io/pr-856


section-headings always use heavy font weight

It would be cool if the headings' font weight was proportional to the user-selected font weight – ie. just heavier than the UI font weight, but not always the same heavy weight bold.

Yes, I think you're right... that's a lot of work through... :wink:

I believe https://github.com/braver/isotope-ui/commit/84b9b1a8e089246fd4c95771146d05f3f6c2b9fc solves this. I feel like I could express it in less lines of CSS, but it works. Basically, in @font-face I match "medium" to "thin" and "semibold" to "light", instead of simply using bold. When available of course. Looks better. I'll test it, think about refactoring the whole weights thing, and release somewhere the coming days.
Thanks for reporting!


fix(shell): Replaced tempfile with mktemp to remove the deprecation warning

Fixes #214

Codecov Report

Merging #220 (41af4e0) into develop (d5ee3a2) will decrease coverage by 0.16%.
The diff coverage is 92.36%.

: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             @@
##           develop     #220      +/-   ##
===========================================
- Coverage    89.25%   89.10%   -0.16%
===========================================
  Files          104      105       +1
  Lines         6498     6599     +101
===========================================
+ Hits          5800     5880      +80
- Misses         698      719      +21

Impacted Files
Coverage Δ

ddb/feature/cookiecutter/actions.py
96.22% <ø> (-0.07%)
:arrow_down:

ddb/feature/docker/__init__.py
65.48% <41.66%> (+1.76%)
:arrow_up:

ddb/feature/docker/utils.py
77.46% <73.33%> (-22.54%)
:arrow_down:

ddb/utils/compat.py
77.77% <77.77%> (ø)

ddb/feature/jsonnet/__init__.py
81.81% <88.88%> (+0.70%)
:arrow_up:

ddb/feature/docker/actions.py
94.98% <90.00%> (-0.70%)
:arrow_down:

ddb/feature/docker/binaries.py
90.38% <91.66%> (-0.62%)
:arrow_down:

ddb/__version__.py
100.00% <100.00%> (ø)

ddb/action/action.py
93.38% <100.00%> (ø)

ddb/cache/__init__.py
96.96% <100.00%> (+0.96%)
:arrow_up:

... and 38 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Merci, je release dans la foulée


QR Code Generator

Generate QR codes and allow people to download it

I would like to take up this issue


Modifying the -std flag for clang++

I want to change the -std=c++17 flag that gets passed through the clang++ llvm to -std=c++1z, since I'm using an older NDK (r16b). The following error crashes my build otherwise:
Compiling oboe-sys v0.5.0
The following warnings were emitted during compilation:

cargo:warning=errorerror: invalid value 'c++17' in '-std=c++17'

  error occurred: Command "/home/porya/Android/Sdk/ndk/16.1.4479499/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++" "-O3" "-DANDROID" "-ffunction-sections" "-fdata-sections" "-fPIC" "-gdwarf-2" "-fno-omit-frame-pointer" "--target=arm-linux-androideabi" "-static" "--target=arm-linux-androideabi23" "-I" "oboe/include" "-I" "oboe/src" "-I" "oboe-ext/include" "-I" "oboe-ext/src" "-std=c++17" "-Wall" "-Wextra-semi" "-Wshadow" "-Wshadow-field" "-fno-rtti" "-fno-exceptions" "-DOBOE_ENABLE_LOGGING=1" "-o" "/home/porya/Github/trs_24/target/arm-linux-androideabi/debug/build/oboe-sys-7626711f30a4069e/out/library/oboe-ext/src/AudioStreamCallbackWrapper.o" "-c" "oboe-ext/src/AudioStreamCallbackWrapper.cpp" with args "clang++" did not execute successfully (status code exit status: 1).

Any ideas?

Fixed by excluding the feature requiring oboe-sys dependency (bevy_audio in my case).
Although now there is no longer audio support for the engine.
Likely will open an issue directly in oboe-rs later.

cargo-apk doesn't set these flags as you found out, that is up to individual crates using the provided cross-compiler (via env vars set up by cargo-apk), typically through the cc crate.
It is unlikely that oboe will compile without this flag: they must have set it for a reason, i.e. the native code using C++17 features. Perhaps you'll have to use an older version, or features have to be gated on their end.


Fix: Prevent ":not([mini])" commands from being fired in search-mode

Since pr #582 commands with :not([mini]) filter in command definition get fired in search-mode editor in case of a missing mini attribute in the atom-text-editor element.
The way how the editor-view was created in the bottom panel has changed from:
new TextEditorView(mini: true)

to:
@editorElement = document.createElement "atom-text-editor"
@editorElement.classList.add('editor')
@editorElement.getModel().setMini(true)

The setMini setter only sets the mini attribute in the model not in the editor-view.

Thanks a lot for catching that!


when podfile contains any pod with "+" or "-" sign in title amimono patcher not working

you can check it on you cocoapods-amimono-example project
just add pod 'NSObject+Rx' or pod 'NSAttributedString-DDHTML' in you podfile
after app running you will get error:
dyld: Library not loaded: @rpath/NSObject_Rx.framework/NSObject_Rx.
i think the problem is that result framework has different name than original pod title.
all "+" and "-"  replacing with "_" underscore sign
cocoapods-amimono version is 0.0.9

It's not just the + and - that are problematic. In some cases, the names are arbitrarily different.
I'm working on a pull request that covers all the cases by using the product_name property of the target.

#30 solves this and also another issue I've encountered.

I'll make a release later today. Thank you @ichina and @igor-makarov.

awesome!!!

v0.0.10 pushed to RubyGems.
I've updated the sample project with NSObject+Rx.


Add LeNet-5 model for MNIST image classification

As we are showing that as an example in our papers/presentations, I want to simply add it to our collections of models. ;)

Adressed in #53


Typo: spekaer_name_or_path

Here's the typo: https://github.com/daswer123/xtts-api-server/blob/13e03383acd3754281e1e73117ec44f06be2c770/xtts_api_server/tts_funcs.py#L349
Error: xtts_api_server.server:tts_to_audio:243 - name 'spekaer_name_or_path' is not defined
Thanks for packaging xtts by the way, I tried it from Coqui and it was messy :D

Thanks for letting me know, I've fixed it


[SPARK-49389] Support master|worker container templates

What changes were proposed in this pull request?
This PR aims to support master container and worker container templates.
Why are the changes needed?
Previously, pod template throws exceptions on master and worker containers because two master|worker containers are created.
Does this PR introduce any user-facing change?
No, this is a new unreleased feature.
How was this patch tested?
For now, we need to run manual tests.
$ ./gradlew build buildDockerImage spark-operator-api:relocateGeneratedCRD -x check
$ helm install spark-kubernetes-operator -f build-tools/helm/spark-kubernetes-operator/values.yaml build-tools/helm/spark-kubernetes-operator/
$ kubectl apply -f examples/cluster-with-template.yaml
$ kubectl get pod -l spark-role=master -oyaml | yq '.items[0].spec.containers[0].resources'
limits:
  cpu: "2"
  memory: 2Gi
requests:
  cpu: "2"
  memory: 2Gi

Was this patch authored or co-authored using generative AI tooling?
No.

BTW, the PR labeler works! 😄

cc @jiangzho and @viirya

BTW, the PR labeler works! 😄

Nice!

Thank you, @viirya . Merged to main.


Cannot read property 'tagName' of undefined

What is your Test Scenario?
Regular e2e testing using roles in a react app with hash router, so currently reloading after the role to preserve the local storage item.
What is the Current behavior?
Sometimes getting error:
 JavaScript error details:
      TypeError: Cannot read property 'tagName' of undefined
          at e.computedNthIndex (https://rec.smartlook.com/bundle-20190218085550.js:20:97496)
          at e.generate (https://rec.smartlook.com/bundle-20190218085550.js:20:97656)
          at t.moveListener (https://rec.smartlook.com/bundle-20190218085550.js:20:100751)
          at l (http://<IP_ADDRESS>:61063/hammerhead.js:13:25467)
          at HTMLDocument.<anonymous> (http://<IP_ADDRESS>:61063/hammerhead.js:6:26610)

When navigating to login page.
Showing error here:
import {e2eConfig as config} from "../e2e-config";
import {ClientFunction, Role, Selector} from "testcafe";

const reloadPage = ClientFunction(() => location.reload());
const getLocation = ClientFunction(() => document.location.href);

const loginRole = Role(config.urls.loginToDashboard, async test => {
    await test
        .expect(getLocation()).contains(config.urls.loginToDashboard)
        .maximizeWindow()
        .expect(config.selectors.login.emailInput.exists).ok()
      .expect(config.selectors.login.emailInput.visible).ok()
error thrown here >>        .typeText(config.selectors.login.emailInput, config.user.name)
        .typeText(config.selectors.login.passwordInput, config.user.password)
        .click(config.selectors.login.submitButton).wait(200)
        .expect(Selector('.analytics-head').visible).ok();
});

export const loginDashboard = async (test) => {
    await test.useRole(loginRole);
    // Reloading the page to load role localStorageItem, because of hash router the localStorageItem is not being set in the first redirect
    await test.expect(getLocation()).notContains(config.urls.loginToDashboard);
    await reloadPage();
};

What is the Expected behavior?
No js errors
Your Environment details:

testcafe version:                   1.0.1
node.js version:                    8.11.4
command-line arguments:  testcafe chrome --speed .5 test/e2e-tests/selectors.e2e.spec.js
browser name and version: Chrome 72
platform and version:          macOS 10.13.6

We didn't face a similar issue, so we need to research it. It's unexpected that you got an error on typeText text. Please check if the error exists without TestCafe. If so, you can run TestCafe with the --skip-js-errors option (https://devexpress.github.io/testcafe/documentation/using-testcafe/command-line-interface.html#-e---skip-js-errors)
Could you also reproduce the issue without roles but only with typeText?
If the error occurs only with TestCafe, I would appreciate it if you share your project or url to your site and full test code. If you cannot share your project, it would help if you prepare a simple example that demonstrates the issue.


feat: add webhook support

Fixes #8
Description
The /nip47 endpoint now accepts a webhookUrl in the POST request. When passed, it immediately returns with "webhook received" like this:

And would make a POST request passing the responseEvent to the WebhookUrl specified in the request like this:

Think about what we do if the app restarts before the reply came in.


Error loading model using downloaded files

Thanks for sharing your work ! Looks very neat .
I am trying to load the model provided and test it on my images. I  get this error when running the eval.py script.
"InvalidArgumentError (see above for traceback): Restoring from checkpoint failed. This is most likely due to a mismatch between the current graph and the graph from the checkpoint. Please ensure that you have not altered the graph expected based on the checkpoint. Original error:
Cannot assign a device for operation model_0/split: ...."
Could you please let me know what the issue is?

the graph from the checkpoint is as same as the graph from the code. I am sure about it. Maybe you can check your setting and running environment

Thanks for the quick response !
May I know which version of tensorflow you are usng to run this ?

Thanks for the quick reply ! I solved the previous error I had, and I get this error now :
AttributeError: 'NoneType' object has no attribute 'model_checkpoint_path'

create a checkpoint file yourself

I solved this issue by replacing lines 169 to 173 in eval.py which is  :
        with tf.Session(config=tf.ConfigProto(allow_soft_placement=True)) as sess:
            ckpt_state = tf.train.get_checkpoint_state(FLAGS.checkpoint_path)
            model_path = os.path.join(FLAGS.checkpoint_path, os.path.basename(ckpt_state.model_checkpoint_path))
            logger.info('Restore from {}'.format(model_path))
saver.restore(sess, model_path)

with the following lines of code:
  with tf.Session(config=tf.ConfigProto(allow_soft_placement=True)) as sess:
            saver = tf.train.Saver()
            # saver = tf.train.import_meta_graph(model_path+'.meta')    # path to meta file
            saver.restore(sess, model_path)    # path to ckpt file

and using tensorflow version 1.8

I have used @DGM1193  comment: replace eval.py like this
tf.Session(config=tf.ConfigProto(allow_soft_placement=True)) as sess:
saver = tf.train.Saver()
saver.restore(sess, '~\checkpoint\model.ckpt')
I  unziped files in repo and got three files.(model.ckpt.data-00000-of-00001)(model.ckpt.index)(model.ckpt.meta)
and fllow @liuheng92  create a blank .txt file rename to checkpoint

and use :
python eval.py --test_data_path=./tmp/images/ --gpu_list=0 --checkpoint_path=./checkpoint/ --output_dir=./tmp/
in anaconda promote
get an error

I have searched this error,

I dont know what I should write in my own 'checkpoint' file
plz help


dockerfile/rabbitmq:latest is not the right way to reference an official...

... docker image,
and fig seems to have problem with it.
Log:
Pulling image dockerfile/rabbitmq:latest...
Traceback (most recent call last):
File "/usr/local/bin/fig", line 9, in
load_entry_point('fig==1.0.1', 'console_scripts', 'fig')()
File "/usr/local/lib/python2.7/dist-packages/fig/cli/main.py", line 31, in main
command.sys_dispatch()
File "/usr/local/lib/python2.7/dist-packages/fig/cli/docopt_command.py", line 21, in sys_dispatch
self.dispatch(sys.argv[1:], None)
File "/usr/local/lib/python2.7/dist-packages/fig/cli/command.py", line 28, in dispatch
super(Command, self).dispatch(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/fig/cli/docopt_command.py", line 24, in dispatch
self.perform_command(*self.parse(argv, global_options))
File "/usr/local/lib/python2.7/dist-packages/fig/cli/command.py", line 56, in perform_command
handler(project, command_options)
File "/usr/local/lib/python2.7/dist-packages/fig/cli/main.py", line 427, in up
insecure_registry=insecure_registry,
File "/usr/local/lib/python2.7/dist-packages/fig/project.py", line 174, in up
for (_, container) in service.recreate_containers(insecure_registry=insecure_registry):
File "/usr/local/lib/python2.7/dist-packages/fig/service.py", line 199, in recreate_containers
container = self.create_container(insecure_registry=insecure_registry, **override_options)
File "/usr/local/lib/python2.7/dist-packages/fig/service.py", line 187, in create_container
stream_output(output, sys.stdout)
File "/usr/local/lib/python2.7/dist-packages/fig/progress_stream.py", line 37, in stream_output
print_output_event(event, stream, is_terminal)
File "/usr/local/lib/python2.7/dist-packages/fig/progress_stream.py", line 50, in print_output_event
raise StreamOutputError(event['errorDetail']['message'])
fig.progress_stream.StreamOutputError: Error: image dockerfile/rabbitmq:latest not found

Thanks for your PR, The documentation is created couple months ago.
 mysql:
    image: mysql:latest

the official mysql image require an environment variable MYSQL_ROOT_PASSWORD to specify the password of root user which would be specify in fig.conf. (or use MYSQL_ALLOW_EMPTY_PASSWORD)
And default setting of pyspider may not work with a root password is not empty.
To use official mysql image, set MYSQL_ALLOW_EMPTY_PASSWORD=yes is necessary to make it compatible with the old dockerfile image.

Thanks for the info, I had problem with scheduler after the pull request, I'll try to get it working and send another pr.

Also I've tried to use Pyspider with mongodb，after installing mongodb and rabbitmq，pyspider -c config.json scheduler kept giving this TypeError: object of type 'NoneType' has no len() error.
config.json:
{
"taskdb": "mongodb+taskdb://<IP_ADDRESS>:27017/taskdb",
"projectdb": "mongodb+projectdb://<IP_ADDRESS>:27017/projectdb",
"resultdb": "mongodb+resultdb://<IP_ADDRESS>:27017/resultdb",
"amqp_url": "amqp://<IP_ADDRESS>:5672/%2F",
"webui": {
"username": "u",
"password": "pp",
"need-auth": true
}
}
Trace:
Traceback (most recent call last):
File "/usr/local/bin/pyspider", line 9, in
load_entry_point('pyspider==0.3.4', 'console_scripts', 'pyspider')()
File "/usr/local/lib/python2.7/dist-packages/pyspider/run.py", line 718, in main
cli()
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 664, in call
return self.main(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 644, in main
rv = self.invoke(ctx)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 991, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 837, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python2.7/dist-packages/click/core.py", line 464, in invoke
return callback(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/pyspider/run.py", line 186, in scheduler
newtask_queue=g.newtask_queue, status_queue=g.status_queue,
File "/usr/local/lib/python2.7/dist-packages/pyspider/libs/utils.py", line 334, in getattr
return ret.get(self, ObjectDict)
File "/usr/local/lib/python2.7/dist-packages/pyspider/libs/utils.py", line 321, in get
return self.getter()
File "/usr/local/lib/python2.7/dist-packages/pyspider/run.py", line 127, in
maxsize=kwargs['queue_maxsize']))
File "/usr/local/lib/python2.7/dist-packages/pyspider/libs/rabbitmq.py", line 212, in init
self.reconnect()
File "/usr/local/lib/python2.7/dist-packages/pyspider/libs/rabbitmq.py", line 222, in reconnect
parsed.path.lstrip('/')))
File "/usr/local/lib/python2.7/dist-packages/amqp/connection.py", line 174, in init
self._x_start_ok(d, login_method, login_response, locale)
File "/usr/local/lib/python2.7/dist-packages/amqp/connection.py", line 817, in _x_start_ok
args.write_longstr(response)
File "/usr/local/lib/python2.7/dist-packages/amqp/serialization.py", line 339, in write_longstr
self.write_long(len(s))
TypeError: object of type 'NoneType' has no len()

"amqp://guest:guest@<IP_ADDRESS>:5672/%2F"

As I have update the documentation using the official mysql/rabbitmq image, I will close this PR.


Incompatible with rand 0.8.x and above

I noticed already quite some time ago that when updating rand create above 0.8.x version the implementation will break.
For example below code will not compile due to some traits not being implemented.
fn encrypt_with_password(compressed: Message, pwd: &str) -> Result<Message, Error> {
    log::debug!("encrypt_with_password, {:?}", compressed);

    let mut rng = thread_rng();
    let s2k = StringToKey::new_default(&mut rng);

    compressed
        .encrypt_with_password(&mut rng, s2k, SymmetricKeyAlgorithm::AES256, || {
            String::from(pwd)
        })
        .map_err(pgp_error_from_pgp)
}

This will lead errors such as:
error[E0277]: the trait bound `ThreadRng: rand_core::CryptoRng` is not satisfied
  --> fmp-common/src/pgp.rs:90:40
   |
90 |     let s2k = StringToKey::new_default(&mut rng);
   |                                        ^^^^^^^^ the trait `rand_core::CryptoRng` is not implemented for `ThreadRng`
   |
  ::: /home/juha/.cargo/registry/src/github.com-1ecc6299db9ec823/pgp-0.7.2/src/types/s2k.rs:22:27
   |
22 |     pub fn new_default<R: CryptoRng + Rng>(rng: &mut R) -> Self {
   |                           --------- required by this bound in `StringToKey::new_default`

error[E0277]: the trait bound `ThreadRng: rand_core::CryptoRng` is not satisfied
  --> fmp-common/src/pgp.rs:93:32
   |
93 |         .encrypt_with_password(&mut rng, s2k, SymmetricKeyAlgorithm::AES256, || {
   |                                ^^^^^^^^ the trait `rand_core::CryptoRng` is not implemented for `ThreadRng`

error[E0277]: the trait bound `ThreadRng: rand_core::RngCore` is not satisfied
  --> fmp-common/src/pgp.rs:93:32
   |
93 |         .encrypt_with_password(&mut rng, s2k, SymmetricKeyAlgorithm::AES256, || {
   |                                ^^^^^^^^ the trait `rand_core::RngCore` is not implemented for `ThreadRng`
   |
   = note: required because of the requirements on the impl of `rand::Rng` for `ThreadRng`

And forcing users to use older version of rand crate specifically staying in version 0.7.x.
Thanks
-Juha

have you tried the latest release? we have recently updated the whole repo to use<EMAIL_ADDRESS>

Yes I tried, and currently using the latest 0.7.2 version from crates

Sorry, I thought I had already published that update, could you try the current master and let me know if this fixes your issue?

Sure, tested it with master branch and along with that it needed zeroize above 1.4.x then it works as before. :) latest seemed to be 1.4.2

released 0.8 with support for rand 0.8


Cucumber report gets overwritten if tests are sharded

Hi,
I don't know if any of you encountered this or even if this is the right place to raise this, but when I run my tests and shard them using Protractor then Cucumber's JSON report (I set the output path in the config) gets overwritten every time one of the shards finishes running. Would it be possible to address this?

I also vote up for this issue , this is really required.

I've handled this by sharding the features myself and giving each results file a unique name. You could have a post build step that merges them all together if your ci tool doesn't allow for multiple results file. We'll need this pr in order to do that.
Here's an example of dynamically sharding by feature file - https://github.com/darrinholst/protractor-cucumber-example/blob/master/protractor.conf.js.
I also have a version that shards by scenario so that the shards are all balanced, but that requires protractor to allow file names with line numbers in them - https://github.com/angular/protractor/pull/2754

Thank you, Darrin! That's certainly a nifty workaround, but it would be nice to see this being natively supported.

See comments in PR https://github.com/mattfritz/protractor-cucumber-framework/pull/19, there are side-effects to that implementation with command line flags. I would appreciate suggestions on that PR if you want to chime in. Maybe only support format nested attributes since that seems to be the use case?

Fixed in https://github.com/mattfritz/protractor-cucumber-framework/commit/4a1f2c74a7df51e049ef05f94448b2d931fbcc2d and available in v0.4.0

Hi @mattfritz,
Thanks for merging looking forward to evaluating this, however on npm registry it's not showing v.0.4.0. Is there just a delay or something I'm missing?
TIA,
Bill

@Bill77 Sorry about that, v0.4.0 should show on NPM now. Thanks!

@darrinholst in your comment above (here) you say:

I also have a version that shards by scenario so that the shards are all balanced, but that requires protractor to allow file names with line numbers in them - angular/protractor#2754

Could you please share that example, as that PR seems to be merged a while ago.
Thanks!

It's been a few months since I was on the project that used it, but it should still work...https://gist.github.com/darrinholst/bd00aa0f40d5978a84c47b012215b080


possibility to replace a key in the address with its alias

While running a contract through a browser in order to execute a contract I should specify its key. Like: <IP_ADDRESS>/Eu4ByDpJ7mFdyeMHZ8z9WcDQ4wP2i5hTXSac1ZKQ2R9Q/state.html. For the end user it's not so clear what it means(arbitary symbols). I think it's more comfortable for the end user to specify something more meaningful for him.
Curious: can we introduce aliases? So that <IP_ADDRESS>/Eu4ByDpJ7mFdyeMHZ8z9WcDQ4wP2i5hTXSac1ZKQ2R9Q/state.html could be replaced with <IP_ADDRESS>/my-app-name/state.html for example. As I see eth, near have such functionality so probably it also can work for us the same way?

This may be possible for the curated contracts that will be present at the gateway; other than that adding aliases would require adding additional mechanisms for contract puts and ensure that name collision can't happen.
@sanity thoughts on this?

Classic decentralized naming problem. The only serious solution to this problem, ie. Zooko's triangle, is blockchain.
Alias doesn't need a consensus, so it's much easier

We are planning a naming service where people can reserve a "name" (similar to a username or domain name) in return for a donation. The web interface should look up names using this which should allow the use of friendly names like this.
Obviously it's centralized but I think that's inevitable for something like this.

Other forms of naming services, in the long term.

Allow contracts to resolve names
Use external blockchains to resolve names

Recently this has come out on discussion, somethings that we may do:

have a contract repository contract which is officially maintained, where aliases can be assigned to contracts, this wouldn't be an issue
this contract would allow version control and specifying dependencies/contracts by (alias, version), which would be automatically handled by the repository-contract
publishing packages may be tied to the reputation system to avoid issues like name squatting


ARROW-15404: [Java][Packaging] Use bundled ORC for building java JNI jars

Forgot to update the JNI files in https://github.com/apache/arrow/pull/12153

Crossbow build https://github.com/ursacomputing/crossbow/branches/all?page=2&query=build-598

Resolved the linux build issue https://github.com/ursacomputing/crossbow/runs/4896845577?check_suite_focus=true


kitty: 0.27.1 -> 0.28.0

Description of changes
The files in preFixup have been removed—they were rewritten in Go. I have not tested the functionality of the rewrite there (don’t really know how, don’t use that feature and could not find docs), but assume it’s okay as we were just patching shebangs and those are now gone.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

No problem. Please test and review #226297. I didn't see the test failure on darwin, and ofborg tests passed.


Merging indexes is slow...

... at least for index with a high term cardinality.
It can be observed on the movielens dataset.
It could be the useless copy of the term in the heap that is hurting.

It is probably specific to tantivy-imhotep where the extraneous .get(term) is extremely costly.


Update docker registry v2 api url

the original url is not available. update according official.

@majinjing3  thank you for correcting this. The URL used to be correct but with the advent of the Registry v2 API this seems to be no longer available. Merging.

@kwk As I know, The URL is already for the Registry v2 API, so I think it will be stable for some long time. Thanks!


feat: fss-to-conformity-custom-check

Feature: New Post Scan Action Plugin - Cloud One Conformity Custom Check
Change Summary
This change adds a new post-scan action to publish malicious file details to Trend Micro's Cloud One Conformity product as a Custom Check.
PR Checklist

[x] I've read and followed the Contributing Guide.
Documents/Readmes

[x] Updated accordingly
[ ] Not required

Other Notes
Currently, the conformity custom checks endpoint does not support forward slashes in resource names so the name format for the resource is the S3 URI with forward slashes replaced by colons.
Also support for TTL to auto expire messages has been built into code but commented out in anticipation for a future update to Conformity to allow users to specify TTLs with Custom Checks.

Example custom check when viewed within Conformity:

Hey @dougc-trend pretty sure I've got all of these updated now.

What was the resolution for "Check Conformity Browse All Checks you should now see the alert raised for the malicious file."?
Is any part of that sentence UI that the user should be looking for on their screen? If so, please put it in **bold**.

What was the resolution for "Check Conformity Browse All Checks you should now see the alert raised for the malicious file."?
Is any part of that sentence UI that the user should be looking for on their screen? If so, please put it in **bold**.

Sorry missed that part, have updated by rewording and bold Browse All Checks

Would it be good to describe how to get the value for ConformityCustomCheckId?
What do you think about using secret manage to handle ConformityApiKey instead of entering the ConformityApiKey directly?

Would it be good to describe how to get the value for ConformityCustomCheckId in the Readme?

CheckID is user defined, any value between 001-999 are valid. There is no prerequisite steps to create this value (customers need to be aware of check id's they are already using and the conformity UI actually makes this really difficult to view but I can't fix that problem).

How about using secret manage to generate a key of ConformityApiKey and use it here instead of entering the ConformityApiKey directly?

Could do but would also result in additional costs to run and complicates any non-SAM based deployments (many more steps). If I can delete all the other alternative install methods such as make AWS CLI etc sure.

Could do but would also result in additional costs to run and complicates any non-SAM based deployments (many more steps). If I can delete all the other alternative install methods such as make AWS CLI etc sure.

I believe other alternative install methods is not a requirement. It's just the a legacy path. We could add those alternatives if there is a request.
Using secret manager is however critical since the parameter is save as plain text which is very dangerous for the API key.

Ok will take a look at updating it to use secrets manager. I will be removing all other install methods outside of using the SAM template via cli / published app (once deployed) unless someone else wants to maintain these. Maintaining these readmes takes too much time for a terrible user experience anyway.

Updated to have the API key to be stored in secrets manager and removed the other installation methods.

@TomRyan-321  I'm trying the plugin but don't know why I got Internal Server Error error after invoking the plugin.

Hey @JimLin94,  just to confirm the AWS account you are testing in is also loaded into the conformity environment you created the API key in? As the script needs to look up the conformity account ID based on the AWS account ID to function.
Also is the API Key user a power/admin user in the account (Readonly wont work but I'd expect unauthorized if this was the case).

Hey @JimLin94, just to confirm the AWS account you are testing in is also loaded into the conformity environment you created the API key in? As the script needs to look up the conformity account ID based on the AWS account ID to function.
Also is the API Key user a power/admin user in the account (Readonly wont work but I'd expect unauthorized if this was the case).

You're right about it, thanks.
We finished the testing and the plugin works like a charm!
Please squash the commit and then we will process the code merge.

@JimLin94 have squashed the commits, first time doing this so let me know if I didn't do it right


Missing spaces make me unhappy

So, there were a few missing spaces and it made me unhappy, so here is the same code but auto formatted.

Looks good. Thanks


UI / UX adaption to the sideframe

[x] removed maximize / minimize
[x] removed show / hide
[x] reposition close button
[x] remove window repositioning (now floats on top of content)
[x] style changes to shadow and knob
[x] ensure clipboard is hidden underneath sideframe

overall looks good :+1:

LGTM


[CLI] RuntimeError: OrderedDict mutated during iteration

I am getting a RuntimeError: OrderedDict mutated during iteration error whenever I try to do any forward calls with a model that I have previously called wandb.watch on. This does not happen when I do not call wandb.watch on the model. I have manually tested all wandb versions from 0.10.27 and upwards and can confirm that this error only starts to appear in 0.10.30 and is not fixed in 0.10.31.
In this specific traceback I am also using PytorchLightning, however I was getting the same error in an other project without Lightning. I have been forced to downgrade wandb to 0.10.29 due to this.
Traceback:
Traceback (most recent call last):
  File "src/gan/train.py", line 83, in <module>
    trainer.fit(model, dm)
  File "/Users/user/prog/mp/genart/venv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 458, in fit
    self._run(model)
 <... a bunch of irrelevant PytorchLightning stuff ...>
  File "/Users/user/prog/mp/genart/venv/lib/python3.7/site-packages/pytorch_lightning/core/memory.py", line 242, in summarize
    self._forward_example_input()
  File "/Users/user/prog/mp/genart/venv/lib/python3.7/site-packages/pytorch_lightning/core/memory.py", line 267, in _forward_example_input
    model(input_)
  File "/Users/user/prog/mp/genart/venv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 889, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/Users/user/prog/mp/genart/src/gan/outer_gan.py", line 127, in forward
    generated_images = self.generator(z, labels)
  File "/Users/user/prog/mp/genart/venv/lib/python3.7/site-packages/torch/nn/modules/module.py", line 892, in _call_impl
    self._forward_hooks.values()):
RuntimeError: OrderedDict mutated during iteration

Hi @konstantinjdobler thanks for reporting this. @borisdayma could you please take a look.

I'm seeing this as well using our W&B training callback in AllenNLP with watch_model=True.

We had a same issue when running in 0.10.30. We had to revert back to an earlier version.

Just letting you know it's being worked on. Hopefully we'll have a fix for next week.

To all, I didn't have your specific example so could not test against it.
Feel free to check against PR #2271 to see if this fixes your issue :crossed_fingers:


Running jest on React Native 0.56.0

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
React Native Environment Info:
    System:
      OS: macOS High Sierra 10.13.4
      CPU: x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
      Memory: 25.44 MB / 8.00 GB
      Shell: 5.3.1 - /usr/local/bin/zsh
    Binaries:
      Node: 8.11.3 - /usr/local/bin/node
      Yarn: 1.7.0 - /usr/local/bin/yarn
      npm: 5.6.0 - /usr/local/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 11.3, macOS 10.13, tvOS 11.3, watchOS 4.3
    IDEs:
      Android Studio: 3.0 AI-171.4443003
      Xcode: 9.3/9E145 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.4.1 => 16.4.1
      react-native: 0.56.0 => 0.56.0
    npmGlobalPackages:
      react-native-cli: 2.0.1

Description
The actual setup is just not fully compatible with Babel 7, when running yarn test with the following test case (the only thing I added to the project was that test case) an error occurs with Babel:
import 'react-native';
import React from 'react'
import App from '../App';

import renderer from 'react-test-renderer';

it('should renders', () => {
  const tree = renderer.create(<App />).toJSON();
  expect(tree).toMatchSnapshot();
});

The error is described bellow:
yarn run v1.7.0
$ jest
 FAIL  src/__tests__/App.test.js
  ● Test suite failed to run

    Plugin 0 specified in "node_modules/babel-preset-react-native/index.js" provided an invalid property of "default" (While processing preset: "node_modules/babel-preset-react-native/index.js")

      at Plugin.init (node_modules/babel-core/lib/transformation/plugin.js:131:13)
      at Function.normalisePlugin (node_modules/babel-core/lib/transformation/file/options/option-manager.js:152:12)
      at node_modules/babel-core/lib/transformation/file/options/option-manager.js:184:30
          at Array.map (<anonymous>)
      at Function.normalisePlugins (node_modules/babel-core/lib/transformation/file/options/option-manager.js:158:20)
      at OptionManager.mergeOptions (node_modules/babel-core/lib/transformation/file/options/option-manager.js:234:36)
      at node_modules/babel-core/lib/transformation/file/options/option-manager.js:265:14
      at node_modules/babel-core/lib/transformation/file/options/option-manager.js:323:22
          at Array.map (<anonymous>)
      at OptionManager.resolvePresets (node_modules/babel-core/lib/transformation/file/options/option-manager.js:275:20)

Test Suites: 1 failed, 1 total
Tests:       0 total
Snapshots:   0 total
Time:        1.388s, estimated 2s
Ran all test suites.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

So I figured out that jest was using babel 6 and I fixed it by downgrading babel-preset-react-native to 4.0.0, but it could lead to another issue.
And then looking on the babel-jest documentation I found out that if you are using babel 7 you will need to add the package babel-core^7.0.0-0.
Just running yarn add --dev babel-core^7.0.0-0 seems to fix the problem.
Reproducible Demo
Run react-native init and place the code sample bellow in __tests__/App-test.js.
import 'react-native';
import React from 'react'
import App from '../App';

import renderer from 'react-test-renderer';

it('should renders', () => {
  const tree = renderer.create(<App />).toJSON();
  expect(tree).toMatchSnapshot();
});

Duplicate of https://github.com/facebook/react-native/issues/19859


[KOGITO-492] - CLI: Provide option to force operator installation eve…

…n if present in OperatorHub
Signed-off-by: spolti<EMAIL_ADDRESS>Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

LGTM, nicely done! Please check the requirements when you send a PR, we need to provide examples to others and also reminds us what we've done. Thank you!


added Bubble Shooter game

PR Description 📜
I have created a bubble shooter game using html,css and javascript
Fixes # 4622

Mark the task you have completed ✅

[x ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ x] I have performed a self-review of my own code or work.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] My changes generates no new warnings.
[ x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x ] I have added screenshot for website preview in assets/images
[x ] I have added entries for my game in main README.md
[x ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @SiriAmarlapudi ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@ishita-43 please review it.

Please review it!!

Please resolve merge conflicts arising in the Readme.md file after that it will get merged .

there are no merge conflicts...Can u merge this game

there are no merge conflicts...Can u merge this game

Pr will soon be merged by the admin!!

Thank you  @SiriAmarlapudi , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


[smpeg2] Use SDL2d for debug configuration

smpeg2 was using the SDL2 release library for debug configurations. With this change it will properly use SDL2d

Redundant to #2030


Update all dependencies

googletest: release-1.11.0
hlslib: v1.4.1
clang: 12.0.1
OpenCL: 2.0
IntelSDK: 20.4.0

Hello, we updated SMI so we can better integrate it into our project. We would be glad if you can review this. Best Regards

Thanks for the fast review. I am not completely done with testing as i am struggling with the configuration too.


Autorest PCL Support

Reposted from https://github.com/Azure/autorest/issues/2584
The Microsoft.Rest.ClientRuntime. project appears to have dropped PCL support. The last version that supports PCL being 2.3.6. Unfortunately 2.3.6 also restricts the NewtonSoft Version >=9 but <10.
With Autorest relying on the ClientRuntime and the ClientRuntime PCL support deprecated and locked into a specific version of Newtonsoft. AutoRest no longer supports generating code for use in a PCL unless you want to permenently lock into a sunsetting version of Newtonsoft. Not sure if this is an issue or by intent but couldnt find any other discussions on the topic nor any annoucement that Autorest and the ClientRuntime is deprecating PCL support.

What more information did you want? You've dropped PCL support. I need PCL support. Not sure what else to say.

Hi @DavidStrickland0, I'm really sorry you didn't get a meaningful response here. Lots of time has passed since you opened this and PCL's in .NET have gotten more and more deprecated in many tools. Is this still an issue for you or have you been able to move to .NET Standard in general? I saw your other comments in the original bug and it sounded like you were moving to .NET Standard. I'm going to close this issue for now. In future versions of the Azure SDK for .NET we intend to remove the Json.NET dependency so this won't be an issue.

In the end we just dropped Auto Rest entirely. Between the Json.Net issue coupled with the size of the class files Autorest created and the lack of the configurability we needed. We just used Roslyn and wrote our own generator. Thanks for the reply though.


Zoomrec stuck in a loop, won't input passcode

Bug
I tried running Zoomrec twice following the instructions in the Readme.md, first I tried running it from a VM in linode, and then from my personal machine. Both times it didn't work but while testing it in my personal machine, I was able to inspect what the container was doing using a VNC client and took the following screenshot:

After the screenshot was taken, the "Could not join with computer audio!" error is logged and the whole process starts again.

Here is what my meetings.cvs looks like

Ok, I found the problem. It looks like the zoom client added a Terms of Service element to the "join meeting" page. This is messing with the script since it needs to press the tab key one more time in order to select the "join" button. Otherwise the "terms and conditions" text will be selected.


Include more advanced search query in godoc and test case.

From #701, there are 2 improvements I think we can make for our search endpoints:

[ ] The godoc can be improved to give an example query. Right now, https://godoc.org/github.com/google/go-github/github#SearchService.Issues doesn't say much. I think it'd be helpful to give an example query that can be done. For example, maybe a query like this:
"gopher is:issue label:bug language:go"
with
opt := &github.SearchOptions{Sort: "created", Order: "asc"}
result, _, err := cl.Search.Issues(ctx, "gopher is:issue label:bug language:go", opt)

[ ] We should add another test case for search that covers a query with spaces and colons. That way, the query parameter escaping functionality won't accidentally regress in the future.

See https://github.com/google/go-github/pull/701#issuecomment-324362880 for more context.

Can this be assigned to me?


Avoidable OOM on small systems

Thanks for your good work.
I am using this on a f1c100s with 32 MB of RAM. The OOM killer kills my program and I traced the cause to the creation of 619 threads in scan_devices - one thread for each entry in /sys/dev/char. I was able to solve the problem by simply calling add_device synchronously in the loop instead of spawning threads.
The first commit fixes the problem. The second commit refactors the change (doesn't allocate the thread array, which also contributed to the OOM since it allocates 619 structs each with a PATH_MAX length array. 619*4096=2.5MB).
Is this parallelized so that the enumeration happens nearly simultaneously/atomically? I don't have enough context to know if that's crucial. If it is, maybe there's another POSIX strategy available to achieve it? If it's parallelized just for performance reasons, I found execution time after the change to be acceptable (0.7s, see below).
I don't think it's unreasonable to expect people to try to use this library on such small systems. systemd+udev is too big for my system and I was able to get the functionality I needed here. I think others will discover this replacement the same way.
Instead of my changes, maybe the threads could be batched up to 64 threads? Or a config option to use the synchronous version?
One more thing. I occasionally saw a segfault failure instead of OOM. I think this happened because the OS decided not to over-allocate for some executions and OOM would not trigger, but the failed allocation / thread creation wasn't handled correctly somewhere in the code.
# free -h
              total        used        free      shared  buff/cache   available
Mem:          22.8M        7.0M        9.6M       36.0K        6.2M       13.4M
Swap:             0           0           0
# ./usbip_crashes_ee32ac5f6494047b9ece26e7a5920650cdf46655
usage: usbip [--debug] [--log] [--tcp-port PORT] [version]
             [help] <command> <args>

  attach     Attach a remote USB device
  detach     Detach a remote USB device
  list       List exportable or local USB devices
  bind       Bind device to usbip-host.ko
  unbind     Unbind device from usbip-host.ko
  port       Show imported USB devices

# ./usbip_crashes_ee32ac5f6494047b9ece26e7a5920650cdf46655 list -l
usbip: error: failed to open /usr/share/hwdata/usb.ids
[   88.815400] usbip_crashes_e invoked oom-killer: gfp_mask=0x100cca(GFP_HIGHUSER_MOVABLE), order=0, oom_score_adj=0
[   88.849760] CPU: 0 PID: 462 Comm: usbip_crashes_e Not tainted 5.11.0-licheepi-nano #1
[   88.869960] Hardware name: Allwinner suniv Family
[   88.886886] [<c010ca40>] (unwind_backtrace) from [<c010a0ac>] (show_stack+0x10/0x14)
[   88.907269] [<c010a0ac>] (show_stack) from [<c0685f9c>] (dump_header+0x50/0x200)
[   88.927595] [<c0685f9c>] (dump_header) from [<c01a29fc>] (oom_kill_process+0x408/0x438)
[   88.948685] [<c01a29fc>] (oom_kill_process) from [<c01a31d0>] (out_of_memory+0x1c0/0x4c0)
[   88.970101] [<c01a31d0>] (out_of_memory) from [<c01dd61c>] (__alloc_pages_nodemask+0x9a4/0xccc)
[   88.992480] [<c01dd61c>] (__alloc_pages_nodemask) from [<c01a0064>] (pagecache_get_page+0xe4/0x314)
[   89.028840] [<c01a0064>] (pagecache_get_page) from [<c01a1124>] (filemap_fault+0x5c0/0x92c)
[   89.051462] [<c01a1124>] (filemap_fault) from [<c01c424c>] (__do_fault+0x38/0x14c)
[   89.073609] [<c01c424c>] (__do_fault) from [<c01c8c3c>] (handle_mm_fault+0x864/0xc24)
[   89.096243] [<c01c8c3c>] (handle_mm_fault) from [<c010d32c>] (do_page_fault+0x120/0x2e0)
[   89.119376] [<c010d32c>] (do_page_fault) from [<c010d77c>] (do_PrefetchAbort+0x38/0x8c)
[   89.142466] [<c010d77c>] (do_PrefetchAbort) from [<c0101744>] (ret_from_exception+0x0/0x1c)
[   89.165942] Exception stack(0xc1af7fb0 to 0xc1af7ff8)
[   89.185998] 7fa0:                                     b6d06168 baf96d94 00001000 00000000
[   89.209461] 7fc0: ffffffff baf96d94 004c02d0 00000000 be947a2a ba79a000 baf96da0 baf99f1c
[   89.232910] 7fe0: 004bef14 baf96588 004ab2bc b6ee5b30 20000010 ffffffff
[   89.254752] Mem-Info:
[   89.272107] active_anon:14 inactive_anon:2723 isolated_anon:0
[   89.272107]  active_file:1 inactive_file:38 isolated_file:0
[   89.272107]  unevictable:0 dirty:0 writeback:0
[   89.272107]  slab_reclaimable:214 slab_unreclaimable:1025
[   89.272107]  mapped:9 shmem:9 pagetables:393 bounce:0
[   89.272107]  free:736 free_pcp:0 free_cma:0
[   89.393138] Node 0 active_anon:56kB inactive_anon:10892kB active_file:4kB inactive_file:152kB unevictable:0kB isolated(anon):0kB isolated(file):0kB mapped:36kB dirty:0kB writeback:0kB shmem:36kB writeback_tmp:0kB kernel_stack:288kB pagetables:1572kB all_unreclaimable? no
[   89.464555] Normal free:2944kB min:4692kB low:4840kB high:4988kB reserved_highatomic:0KB active_anon:56kB inactive_anon:10892kB active_file:4kB inactive_file:152kB unevictable:0kB writepending:0kB present:32768kB managed:23300kB mlocked:0kB bounce:0kB free_pcp:0kB local_pcp:0kB free_cma:0kB
[   89.538798] lowmem_reserve[]: 0 0 0
[   89.558537] Normal: 44*4kB (UM) 8*8kB (M) 7*16kB (M) 1*32kB (M) 0*64kB 0*128kB 0*256kB 1*512kB (M) 0*1024kB 1*2048kB (M) 0*4096kB = 2944kB
[   89.604128] 48 total pagecache pages
[   89.623909] 0 pages in swap cache
[   89.643074] Swap cache stats: add 0, delete 0, find 0/0
[   89.663922] Free swap  = 0kB
[   89.681894] Total swap = 0kB
[   89.699486] 8192 pages RAM
[   89.716566] 0 pages HighMem/MovableOnly
[   89.734524] 2367 pages reserved
[   89.751399] Tasks state (memory values in pages):
[   89.769880] [  pid  ]   uid  tgid total_vm      rss pgtables_bytes swapents oom_score_adj name
[   89.792619] [     54]     0    54      702       20     8192        0             0 syslogd
[   89.815044] [     58]     0    58      702       20     6144        0             0 klogd
[   89.836944] [     77]     0    77      702       28     8192        0             0 sh
[   89.858256] [     86]     0    86   765877     2653   776192        0             0 usbip_crashes_e
[   89.892969] oom-kill:constraint=CONSTRAINT_NONE,nodemask=(null),task=usbip_crashes_e,pid=86,uid=0
[   89.915644] Out of memory: Killed process 86 (usbip_crashes_e) total-vm:3063508kB, anon-rss:10576kB, file-rss:36kB, shmem-rss:0kB, UID:0 pgtables:758kB oom_score_adj:0
[   90.012199] oom_reaper: reaped process 86 (usbip_crashes_e), now anon-rss:0kB, file-rss:0kB, shmem-rss:0kB
Killed
# ./usbip_bandaid_fix_185a0879a83736314c541bfd98a1f1f4b6c0d09a list -l
usbip: error: failed to open /usr/share/hwdata/usb.ids
 - busid 1-1 (1e3d:2095)
   unknown vendor : unknown product (1e3d:2095)

# ./usbip_refactor_fix_c803d022ef6e619546079afe20f468f82e3b27c7 list -l
usbip: error: failed to open /usr/share/hwdata/usb.ids
 - busid 1-1 (1e3d:2095)
   unknown vendor : unknown product (1e3d:2095)

# time ./usbip_refactor_fix_c803d022ef6e619546079afe20f468f82e3b27c7 list -l
usbip: error: failed to open /usr/share/hwdata/usb.ids
 - busid 1-1 (1e3d:2095)
   unknown vendor : unknown product (1e3d:2095)

real    0m 0.70s
user    0m 0.30s
sys     0m 0.39s
#

Yeah, if memory serves threads can be removed. They were introduced to soften performance issues with ioctl call which was previously used to fetch evdev properties. Nowadays a more efficient sysfs method is used, so threads have lost their sense.
However there is a still huge chance that I missed something essential since threads are still around despite that I removed ioctl calls a long time ago. Why I didn't remove them together with ioctl calls? I don't remember and I hope there is no reason to keep them around.

OK, I'll merge this for now. Technically this shouldn't cause any problems, but only time will tell. Anyway, thank you for your attention and sorry for delay.


[Feature] Is this project still active?

Is it?

At the moment not really - I no longer work as much in the event/lighting/entertainment space so my need to use sACN every day is greatly reduced.
Is there a feature/problem you are particularly interested in seeing / solving?

I was just checking tbh, I'm just trying to find my footing in the lx programing world, it's not like there are a ton of resources out there, and half of them seem to not have revived updates in years.
My goal is to make a half decent, modern, open source lighting desk, but for now I'm just having a look around.


Add manual deployment repo configuration for sites:create and init --manual

- Summary
This PR attempts to offer some solutions to #177. Therefore it does two things:

When performing manual configuration, there is now an additional step that offers the option to change the remote repository (while defaulting to the one that it could read locally) and also validates against non-SSH URL's

When using sites:create, it now gives the option to set up Continuous Integration using all of the available methods in configManual, including the new option to set the remote git URL. My favourite new feature here is that this command can now be used to entirely configure manual CI without needing to have the repository local at all

- Test plan
The following is the output when using the sites:create with my default team and a random name:

This did not require me to have any local git repository, and all of the details provided were used correctly. I've been able to verify this as the site deployment says it "deploys from 'manual'" and when hovering over that link, I am greeted with this hyperlink:
https://app.netlify.com/sites/clever-curran-7704a4/git@code.samholmes.net:sam/test.git

Thus showing that the remote git URL has been set correctly on the repo object. Having set up my personal GitLab instance with the webhook and deploy key, I can confirm that git push deploys are working as expected without needing to have a local repo instance on hand.

Now, when using the netlify init --manual, you can see that I have the option pre-filled in my local repo for the git deployment information:

If this does not conform to the following regex (to verify that it is an ssh URL):
/(ssh:\/\/|[a-zA-Z]*@|[a-zA-Z.].*:(?!\/\/))/

(testing of this regex)
Then the following error will occur:

Thus preventing the user from continuing unless a correct URL is provided. This has the added benefit of safe-guarding against any potential incorrect automatic parsing of the URL from HTTPS to SSH, and ensuring that repo information cannot be updated unless the remote repository is definitely using SSH (which, all going according to plan, should mean that the user can simply just press return to use the default without a hitch).
- Description for the changelog
Add the ability to set up Continuous Integration from sites:create, and the option to manually configure the remote repo that CI uses
- A picture of a cute animal (not mandatory but encouraged)

He's such a good boy.

Coverage remained the same at 100.0% when pulling 4a16f429fc0581f28b083ca5315229ee8f8e0982 on sam3d:manual-improvements into 1e25d40a093c5b381f2c048a94aaa237f4430f00 on netlify:master.

Very cool @sam3d! I'll try to take a look today or early next week.

Will this work with ssh running under a different port?

Ooooh I didn’t clock that edge case, thank you for the new Regex!
I’ll update it later this evening 😊

You're welcome :smiley: Thanks to you, actually. I was stuck trying to deploy something to Netlify. I was struggling with the cli tool, going over the docs, then I found this life saver PR.
I did notice that with your branch netlify init --manual doesn't ask for a site name anymore. It just picks a random one. Not a big deal though.

Oops thank you for the the site name - that's not desired behaviour so I'll take a look at that. And you're welcome! Solved an exact issue I had where the web UI doesn't offer the ability to set up Continuous Integration unless it's:

An existing remote git repo (which is fine)
On a hosted code provider (which is less useful)

Also, I didn't realise this when I responded from my phone yesterday, but I'm pretty sure the regex I wrote includes support for different ports (with the top line being the testing example):

(https://regexr.com/42pg4)
Did you find this wasn't working?

Ah thank you for explaining! The only issue is that a path can appear directly after the first semicolon in regex, and that appears to un-match those edge-cases where:

the ssh:// isn't provided, and
the standard port is used, so a path is the first thing after a colon

https://regexr.com/43lot
I was erring on the side of less strict with my regex, with the hope that the additional option to configure the URL and also users seeing the "SSH URL" signifier being enough to prompt anything the regex didn't catch.

I tried a few different attempts at a prettier regex and I failed. I understand your approach better now, the port number is not required. A more relaxed rule set is indeed preferable.

I also tried and failed, not to worry 😄

Any news on this being reviewed & merged @bcomnes ?
It would be great to if this made it in as this exact issue cost me quite some time yesterday! TIA.

Very sorry guys, I'll try to put some eyes on this tomorrow.

I spoke to soon, I wont have focus for this till moday.

That's okay, totally understand you must be busy. Thank you for keeping us in the loop, and let me know if there's anything I can do to make this easier on you :-)

okay! Reviewing :)

Cool! I like the premise.  Digging in

I also noticed some issues with init today as well, say if you manual deploy then go try to init CI, there are blockers and it requires some roundabout method tp fix.

Okay added this in a way I can live with.  Its not perfect, but we need to rework a lot of this code someone soon in order to update to gh-apps and move a lot of the logic into our API.

Thanks for your input everybody!  I'll go ahead and merge this and release something tomorrow.  Please open up any new issues for anything else left on the table.


Disable error tracking until Fathom changes are commited

Disable error tracking until Fathom changes are commited

****************************************************************

libgrpc.so

     VM SIZE                                                                                FILE SIZE
 ++++++++++++++ GROWING                                                                              ++++++++++++++

 -------------- SHRINKING                                                                            --------------
  -0.1%    -908 [None]                                                                               -15.5Ki  -0.2%
      -0.1%    -876 [Unmapped]                                                                           -15.5Ki  -0.2%
      -2.3%     -24 [None]                                                                                0  [ = ]
      [DEL]      -8 grpc_core::(anonymous namespace)::timestamps_callback                                      0  [ = ]
 -16.0% -1.36Ki src/core/lib/iomgr/tcp_posix.cc                                                      -1.36Ki -16.0%
      [DEL]    -875 process_errors                                                                          -875  [DEL]
     -28.3%    -358 tcp_flush                                                                               -358 -28.3%
     -66.9%    -158 tcp_handle_error                                                                        -158 -66.9%
      [DEL]    -135 tcp_annotate_error(grpc_error*, (anonymous namespace)::grpc_tcp*) [clone .isra.2]       -135  [DEL]
      -0.6%      -1 [Unmapped]                                                                                -1  -0.6%
 -94.1%    -444 src/core/lib/iomgr/buffer_list.cc                                                       -444 -94.1%
      [DEL]    -195 grpc_core::TracedBuffer::ProcessTimestamp                                               -195  [DEL]
      [DEL]    -135 grpc_core::TracedBuffer::AddNewEntry                                                    -135  [DEL]
      [DEL]    -103 grpc_core::TracedBuffer::Shutdown                                                       -103  [DEL]
      [DEL]     -31 [Unmapped]                                                                               -31  [DEL]

  -0.2% -2.68Ki TOTAL                                                                                -17.3Ki  -0.2%

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,965,335      Total (=)      1,965,335

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,888,230      Total (<)     10,888,231

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences


Bug: cdc iterator doesn't seem to capture file deletion.

Bug description
As title says, I'm tweaking in s3, but when I delete the file in s3 I don't see the change reflected.
I see that the file connector has some limitations when writing files, but I don't see why a delete record wouldn't be written (the contents of the record, not the action that the record is supposed to do). Specifically, here all records are written as is.
Steps to reproduce

Upload a test file in s3, in any bucket.
Start conduit with a s3 as source and file as destination as a pipeline. Like this one:

version: "2.2"
pipelines:
  - id: example-pipeline
    status: running
    name: "s3-to-file"
    connectors:
      - id: example
        type: source
        plugin: s3
        settings:
          aws.accessKeyId: ...
          aws.secretAccessKey: ....
          aws.region: ...
          aws.bucket: ...

      - id: file
        type: destination
        plugin: "file"
        settings:
          path: ./destination.txt
          sdk.record.format: template
          sdk.record.format.options: '{{ printf "%s" .Operation }}'

$ tail -f destination.txt
Delete the s3 object. We should see a delete in the file, but we don't.

Version
(latest conduit & latest versions of mentioned connectors)

@alarbada You can try confirming the above by using the log connector. But you need to have versioning enabled in your bucket (I don't see it mentioned in your bug description).

@hariso Oh yeah that might be it, sorry!

@alarbada All good, I'm glad it was just that!


Feat: footer

Closes #6
Currently based off #30. Waiting for that to be merged first.
Full page screenshot:

Proposals/questions for reviewers:
There are a few small differences between what I currently have vs. what's in the Figma.

Contact section: The Figma doesn't have a header for this section, but while working on it, I found that it looks better with a header.

Proposal: the Contact section has a header that says "Contact us"

<EMAIL_ADDRESS>link: The Figma shows this as a green link with a font of Source Code Pro. I think we did it like this because we were thinking of the original big-title link from the first version of the site. However, as I worked on the new link, I realized that all the other links on the site are white with Roboto as the font. The original big-title link was only styled with Source Code Pro because we were emulating the Compiler business card. Also, apparently the original link was white: https://deploy-preview-5--compilerla.netlify.app/

Proposal: the<EMAIL_ADDRESS>link is styled like all our other links on the site

Section font: The Figma shows the cert values as bold, white text at 18px. I'm not sure / don't remember our reasoning for this style. When I simply add the text for the cert values, they show up as normal, white text at 16px since this is the body style. This looks ok to me.

Proposal: no special styling for the cert values - they are styled the same as normal text on our site

Social media icons: I actually didn't notice this until I wrote out the notes above, but in our Figma, we added some space between the social media icons. I haven't made that change yet. Do y'all think this change is necessary?

Let me know if you have any counter-proposals!
The original Figma mockup, from https://github.com/compilerla/compiler.la/issues/6#issuecomment-1256555734:

Hmm, there's no Netlify preview on this PR. Maybe it's because the target branch is not main?

I think you'll have to open a brand new PR @angela-tran. Netlify only watches the main branch and this one probably was already ignored / passed over.

Ok, I'll try that out. Thanks @thekaveman

Sorry @AnthonyRollins and @thekaveman, the way I opened this pull request made it so the preview won't generate... see #32 instead. 🙏


flickering when using with react navigation bottom bar

Environment
Library version: 4.0.1
OS version: pixel 7 android 13
Affected platforms

[X] Android
[x] iOS - not tested

Current behavior
When moving between the screen a flicker happens and its very visible and annoying and I donot know the reason
only appears when using  AvoidSoftInput.setShouldMimicIOSBehavior(true);
https://github.com/mateusz1913/react-native-avoid-softinput/assets/53271078/b27c4bc4-e153-41b2-ade5-85678d79f919
when I use AvoidSoftInput.setShouldMimicIOSBehavior(false);
it doesn't happen but also a flick happens
when opening and closing the keyboard
https://github.com/mateusz1913/react-native-avoid-softinput/assets/53271078/c824ed13-717a-45d9-9904-e9a9ac94bc39
Expected behavior
while  AvoidSoftInput.setShouldMimicIOSBehavior(true); it should't do that flicker
Reproduction
https://github.com/25khattab/react-native-template

@mateusz1913  when you have time please, thank you

Hey @25khattab, sorry for delay and thanks for proper reproduction repo 🚀
I managed to reproduce it, but I need to dig deeper and will get back to you later today or on weekend

Thank you @mateusz1913,
another thing I noticed The animation doesn't work when mimic ios it just feels laggy Im on pixel 7 running 90 hz

Hey @25khattab, it seems that the problem is caused by incorrectly handled safe area. I made a PR in your repo and I'll close this issue as it's not related to the library.
Regarding the animation, if you feel that it doesn't work at all, open new issue and provide a short recording (I assume that reproduction repo will be the same as here), thx!


will you release the pre-trained model?

we can try it if you release the model

Okay, I'll release demo model with fixed code when it's ready.
I'm so busy nowadays, so it will take 1~2 months apparently.
Have a nice day!

Okay, I'll release demo model with fixed code when it's ready.
I'm so busy nowadays, so it will take 1~2 months apparently.
Have a nice day!

now will you release the pre-trained model?


add config menu

@dawsbot here is the cli config we were talking about the other week.
pretty simple addition to improve dev process
also seems as though there were missing combined.json files from last commit on v1 because husky is adding about 30 to the most recent commit

@dawsbot requested changes were added


[prominent-productname] Emphasize Product Name and Link to Plans

Trello Cards

https://trello.com/c/W6ChMtzO/75-as-an-admin-viewing-an-installation-i-want-to-get-back-to-the-plan-detail-preflight-page-to-run-another-installation
https://trello.com/c/WVUeE6qS/73-as-an-admin-i-want-the-product-version-to-be-more-prominent-on-plan-detail-and-job-detail-pages

What to look for
On the Plan Details and Job Details (after install) pages, we are emphasizing the Product Name and version of the current plan, and linking back to select a new plan from the product detail page.

@mirisuzanne @SondraE Here are screenshots showing what the new section looks like on both plan-detail and job-detail:

@jgerigmeyer
Am I understanding correctly that "view available plans" link and "Product With Useful..." link in the header both go to the product detail page which has the list of plans? The concern I heard was just that the name of the product was getting lost. Is there an issue with the link too? If we do want that link twice, is there a problem with making the name of the product the link? Making it a blue link would perhaps make it stand out even more.
Could we put "This plan is part of..." right under "Full Install" as a subtitle? My concern is that the generic text could be quite long and push the product name way down the screen. In my mind, it makes sense to group that info with the title.
Hmm... As I think about this, I'm just not sure adding the product title and link to plans to the instruction text area makes it more accessible. There are times when there can be several paragraphs of text in that area as well as warnings and errors. Seems like it could get cluttered fast. Maybe it would be worthwhile to rethink the page header to make the link back to plans as well as the product title more obvious.

@SondraE Okay... we can discuss further with @mirisuzanne and @stacyk on Monday, in that case. 👍


homepage and appstore touch-up

Problem

Homepage:

"All apps" display is sometimes wonky.
Some apps display without an icon and cannot be clicked, despite having an icon and a UI.
The appstore has strange behavior around installing and downloading and launching apps.
The appstore needs updating to match new design material.

Solution

Fix those things.

Testing

Boot a new node with simulated slow connection (devtools -> network -> throttling)

Verify apps load in with images and can be clicked if they provide a path to their web interface

On a phone, verify the homepage display isn't jank, incl. all apps section

On the appstore, download an app with a ui (memedeck, command center) and install and launch it from a single page without reloading.

Also try downloading a headless app like Filter.

Mobile for this flow as well.

Docs Update
N/A
Notes
{Any other information useful for reviewers.}

I'm working on mobile styling right now. Will undraft when ready.


MQTT Support

Hi,
first: nice work, very smart tool, i love it! :-)
I got an LED Light Stripe with Wif Controller integrated to Home-Assistant.
Would it be possible to send the current color to an MQTT Server?
The reason behind this is, that i want to sync my Light Stripe with the Mystic Lights. MQTT would make this possible. Another way would be the other direction, a API to receive a color for the LEDs.

Sorry, I didn't reply immediately because I didn't know what to do of this. I would have to do some research in order to implement this, and right now I don't have the time for it. If you or someone else wants to send a pull request, I'll look into it.

You'll be able to do this now with the (soon to be published) 2.4.0 release of MSIRGB. Lua scripts can now use any Lua function, which means you can load 3rd party Lua libraries like luamqtt.