Fix JDK 11 test errors for Remote Assets and other tests with Mock Server

Updated RemoteAssetDecoratorTest to no longer rely on PowerMockito

Updated version of mockserver to fix JDK 11 errors in tests for IntegrationServiceImplTest, S3AssetIngestorTest, and RemoteAssetsBinarySyncImplTest

JDK 11 Before:
[ERROR] Errors:
[ERROR]   IntegrationServiceImplTest.testJwtToken:77 » NoClassDefFound javax/xml/bind/Da...
[ERROR]   PowerMockErrorPageCacheImplTest.initializationError » Objenesis java.lang.refl...
[ERROR]   JCRHttpCacheStoreImplTest.initializationError » Objenesis java.lang.reflect.In...
[ERROR]   I18nProviderImplTest.initializationError » Objenesis java.lang.reflect.Invocat...
[ERROR]   S3AssetIngestorTest.testCreateFolders:292 » NoClassDefFound javax/xml/bind/Dat...
[ERROR]   S3AssetIngestorTest.testCreateFoldersWithBasePath:315 » NoClassDefFound javax/...
[ERROR]   S3AssetIngestorTest.testCreateFoldersWithHyphens:339 » NoClassDefFound javax/x...
[ERROR]   S3AssetIngestorTest.testImportAssets:188 » NoClassDefFound javax/xml/bind/Data...
[ERROR]   S3AssetIngestorTest.testImportAssetsToExistingRootFolder:245 » NoClassDefFound
[ERROR]   S3AssetIngestorTest.testImportAssetsToNewRootFolder:221 » NoClassDefFound java...
[ERROR]   S3AssetIngestorTest.testImportAssetsWithBasePath:268 » NoClassDefFound javax/x...
[ERROR]   S3AssetIngestorTest.testImportAssetsWithBucketContainingJustFolders:170 » NoClassDefFound
[ERROR]   S3AssetIngestorTest.testImportAssetsWithException »  Unexpected exception, exp...
[ERROR]   RemoteAssetsBinarySyncImplTest.testSycnAssetFailureDueToOriginalRenditionNotFound:236->setupFinish:148 » NoClassDefFound
[ERROR]   RemoteAssetsBinarySyncImplTest.testSycnAssetFailureDueToRenditionError:216->setupFinish:148 » NoClassDefFound
[ERROR]   RemoteAssetsBinarySyncImplTest.testSycnAssetFailureLogsErrorIfAssetCannotBeFlaggedAsFailed:256->setupFinish:148 » NoClassDefFound
[ERROR]   RemoteAssetsBinarySyncImplTest.testSycnAssetSuccessWithRemovalOfNotFoundRendition:197->setupFinish:148 » NoClassDefFound
[ERROR]   RemoteAssetsBinarySyncImplTest.testSyncAssetSuccess:178->setupFinish:148 » NoClassDefFound
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   BrandPortalSyncProcessTest.initializationError » Objenesis java.lang.reflect.I...
[INFO]
[ERROR] Tests run: 1434, Failures: 0, Errors: 27, Skipped: 0
[INFO]

JDK11 After:
[ERROR] Errors:
[ERROR]   PowerMockErrorPageCacheImplTest.initializationError » Objenesis java.lang.refl...
[ERROR]   JCRHttpCacheStoreImplTest.initializationError » Objenesis java.lang.reflect.In...
[ERROR]   I18nProviderImplTest.initializationError » Objenesis java.lang.reflect.Invocat...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   FileImporterTest.setup:71 » IllegalArgument javax.jcr.RepositoryFactory is not...
[ERROR]   BrandPortalSyncProcessTest.initializationError » Objenesis java.lang.reflect.I...
[INFO]
[ERROR] Tests run: 1434, Failures: 0, Errors: 12, Skipped: 0
[INFO]

No new errors, just 15 fixed.  All tests pass on Java 8.

@HitmanInWis code changes look good, but all travis builds failed. look like they might have timed out?  Restarting them to see.

@HitmanInWis hmm - even JDK8 travis builds seem to be failing for this PR.. 🤔
A quick peek at travis logs seems like it might have something to do w/ mock server?

hrmm not sure what to say, will have to look at it when I get some time...builds just fine for me locally with JDK8, I tested before I pushed it up.

Pull Request Test Coverage Report for Build 4769

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 53.695%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bundle/src/main/java/com/adobe/acs/commons/remoteassets/impl/RemoteAssetDecorator.java
1
2
50.0%

Totals

Change from base Build 4767:
-0.002%

Covered Lines:
13935

Relevant Lines:
25952

💛  - Coveralls

Ok @davidjgonzalez this is fixed.  Forgot that travis build fails when the log gets too large, and the new mockserver version had a new class that had to have its logging tamed :)

PowerMockito struggles to work well in JDK9+ in general because the class loader has changed so much that mocking static methods can be a bit finicky.   I was able to get it working in another project (same root cause) by adding this annotation to the top of each affected unit test:
@PowerMockIgnore({"com.sun.org.apache.xerces.*", "javax.xml.*", "org.xml.*", "org.w3c.*"})

Also might try these dependency versions:         <dependency>             <groupId>org.mockito</groupId>             <artifactId>mockito-core</artifactId>             <version>2.28.2</version>             <scope>test</scope>         </dependency>         <dependency>             <groupId>org.powermock</groupId>             <artifactId>powermock-api-mockito2</artifactId>             <version>2.0.2</version>             <scope>test</scope>         </dependency>         <dependency>             <groupId>org.powermock</groupId>             <artifactId>powermock-module-junit4</artifactId>             <version>2.0.2</version>             <scope>test</scope>         </dependency>

Good to know for cases where we need to continue using PowerMockito.  In this case it was for a single static function call which we encapsulated in an instance function, obviating the need for PowerMockito at the expense of leaving one line of code untested, which seems like an okay tradeoff.

Yes, absolutely a better solution in this case.  I'm not sure if there is a better way to inject static method behaviors going forward, which is rather a bummer.  The overall recommendation is not to use static methods and write code that better lends itself to testing, but unfortunately that isn't an option if your code uses static utilities like JcrUtils, etc.


Add jianfei to security/OWNERS

@incfly: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
9aa63a365d3dcefe2bea38a70fd087fe147c35b4
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: myidpt
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [myidpt]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Support for managed VNET

@raunakjhawar : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@linda33wj please review and sign off when ready.

#sign-off

@JasonWHowell: I'm sorry - only the author of this article, @linda33wj, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Case sensitive to UTM names in URL

I noticed that when I try from a URL with "UTM_SOURCE=foo", the form does not populate with the UTM Source parameter. However, when the URL has a lowercase property – "utm_source=foo" – then it does.

Do you know how Google Analytics (and other analytics tools) look at these non-lower case UTM parameters? I haven't tested such cases so I'm not sure. Just want to make sure supporting other cases makes sense or not.

Since this is a rare case and not really in demand, I'm closing this issue.


UI degradation in related layers

Problem
The code included in cea714f3f39daa3692ca2f3612ec477f4be490ef results UI degradation in related layers.
Any multi-layered chart (example 3, a.k.a. ss) after this commit results layers which is not precisely arranged.
result

desired

Cause
This is design-level problem of FSR.
A FSR option initialization process is fired with a constructor of any of Layer derivatives, and  a ideal FSR should not permit any modification of margin, area or seed values for origins of axis after that.
But at now, axis' origin like variable xOrigin is set in functions related to auto-scaling (e.g. autoScaleX). So, any FSR process cannot perfectly control axis' length and thier origins.

How it shoud be in future
FSR should support automatic adjustment of any type of parameters for axis' origin and length, and any layers, also in related layers should be rendered without any corrupted margin or axis' positions.
How it shoud be in the fixation at now
FSR mechanism should not be modified in such conditions, but UI corruption should be eliminated to detect its condition not to result corrupted positions or margins for axis or objects.


NotificationCompat.Builder.Builder(Context)' is obsolete

When using this code:
void SendLocalNotification(string body)
{
    var intent = new Intent(this, typeof(MainActivity));
    intent.AddFlags(ActivityFlags.ClearTop);
    intent.PutExtra("message", body);

    //Unique request code to avoid PendingIntent collision.
    var requestCode = new Random().Next();
    var pendingIntent = PendingIntent.GetActivity(this, requestCode, intent, PendingIntentFlags.OneShot);

    var notificationBuilder = new NotificationCompat.Builder(this)
        .SetContentTitle("XamarinNotify Message")
        .SetSmallIcon(Resource.Drawable.ic_launcher)
        .SetContentText(body)
        .SetAutoCancel(true)
        .SetShowWhen(false)
        .SetContentIntent(pendingIntent);

    if (Build.VERSION.SdkInt >= BuildVersionCodes.O)
    {
        notificationBuilder.SetChannelId(AppConstants.NotificationChannelName);
    }

    var notificationManager = NotificationManager.FromContext(this);
    notificationManager.Notify(0, notificationBuilder.Build());
}

It shows a warning: 'NotificationCompat.Builder.Builder(Context)' is obsolete: 'deprecated'

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

ID: 92f390d9-cbd0-ab8c-6c8c-49df76d1b8e0
Version Independent ID: 15739c2f-afbb-c21d-b96f-fabe7060e410
Content: Send and receive push notifications with Azure Notification Hubs and Xamarin.Forms - Xamarin
Content Source: docs/xamarin-forms/data-cloud/azure-services/azure-notification-hub.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @profexorgeek
Microsoft Alias: jusjohns

Hi @Mikilll94
Thanks for letting me know. This is a case where the doc isn't in sync with the sample. The sample code is correct (NotificationCompat.Builder(this, AppConstants.NotificationChannelName)) but the doc needed an update.
I've updated the doc in a PR that will go live by the end of the week.


Delete Tag

Bug Report
Expected behaviour
Delete Tag should work for manual mode tools
Actual behaviour
Delete Tag is visible in automated tools.  Delete tag does not work.

Update:
Delete tag works with feature/bugFixes branch.  However, due to the use of swagger codegen, it requires modifications to the webservice.  The webservice is returning status 200 with no entity.  The generated swagger client expects all non-204 responses to contain something.  Currently my webservice is modified to be return Response.noContent().build();  instead of return Response.ok().build();
Another solution is to return the current tags.

The steps listed here make sense to me.

Please link to PR

In the console log, the web service responses will cause errors.
Take a look for invalid responses and fix them

Pull request that included the delete token fix is https://github.com/ga4gh/dockstore/pull/914
Pull request that included the delete tag fix is https://github.com/ga4gh/dockstore/pull/909
Still probably some others need checking/changes.

Works on staging.dockstore.org


[test][material-ui] Add tests for Pigment Grid and Stack

While reviewing https://github.com/mui/material-ui/issues/44042 I noticed we didn't have tests for these components. This PR adds them.

I guess the next step (in the future) will be to move those to be demos on the docs. This way, we can see them in Netlify, Argos can take visual screenshots, etc.

@oliviertassinari we already have set up screenshot tests for Pigment CSS, it's using the fixtures directory from the vite app,
check https://app.argos-ci.com/mui/material-ui/builds/32947/114088289
The main reason why I decided to use the vite app is because Pigment CSS is not used on the docs yet.


Abstract for Textfield not matching Abstract

I got this error
haxeui-raylib/git/haxe/ui/backend/raylib/TextField.hx:679: characters 36-50 : Abstract has no field KEY_LEFT_SHIFT
I fixed it by changing code in Textfield to match the abstract.
This is from the abstract
@:native("KEY_LEFT")                public static var LEFT:Int;
So I changed from KEY_LEFT_SHIFT to LEFT_SHIFT
Or something like that. anyways I created a patch file which I have never done before.
patch.txt

Good point... i had that (and some other) changed locally... should be fixed in latest version


Enhancement - Show confirmation dialog when user tries to assert hotspot into location that will reduce their reward scale

One of the biggest problems with the Helium network is over-saturation of a hex. Many hotspot owners simply assert their location without doing any research or homework into the effects or consequences to themselves or others.
Based on current rules, a res_8 can have a max of 4 hotspots and still receive a scale of 1.0, if spaced correctly or within bounds of sibling hexes.  There are too many examples where the number of hotspots in a res_8 hex far exceed this number.
I propose that during the assertion process of their hotspot, a calculation will be done that can determine if the hotspot owner should consider placing their hotspot elsewhere if their scale will be harmed, as well as harming the scales of others. If it is determined their scale will be reduced, a confirmation message will be shown to them of the consequences of asserting and if they still would like to.
This is the people's network, so preventing them should not be allowed, but giving them a heads up I think is good.
So while it may be more applicable in a hex with more than 4 hotspots, if the calculation determines your scale will be reduced because you are too close to another hotspot, a message will be shown to the user prior to assertion so they can responsibly determine if asserting as their current location is in the best outcome for themselves, their neighbors, and the network.
A couple examples:
The user gets their new hotspot and wants to assert it at their home.  Upon attempting to assert it, they see a message pop up with a message like

"You are asserting into a hex that has reached its maximum density of 4 hotspots. Your reward scale will be reduced to 0.67. This reward scale will also affect the other hotspots in this hex. Your hotspot earning potential will be reduced. Do you want to continue?"

or

"You are asserting into a hex too close to another hotspot. By doing this, your reward scale will be reduced to 0.5 as well as the other hotspot you are near. Your hotspot earning potential will be reduced. Do you want to continue?

I could see it being beneficial to show existing users when they sign into the application that their hotspot's reward scale is reduced due to being in hex with other hotspots. They may already be aware of this and in those cases they can dispose of the message, but other users may not realize how bad their reward scale has been affected and may consider re-asserting elsewhere to fix.
I think something like this could benefit the network by leaning on users to spread their hotspots out which produces better coverage, and of course benefit hotspot owners from an earnings perspective.

While I agree with the general sentiment and the well thought-out response of this request, a lower transmit scale does not affect earnings as radically as you'd expect.
It doesn't matter if a Hotspot is in a highly saturated location so long as it can witness Hotspots with a high transmit scale (in less dense locations). https://explorer.helium.com/hotspots/119ApNVisVbhew7txGXD5JeUzfJ5qS755j1Zqyn9JSybMbzSKty
here is an example of a Hotspot with a lower transmit scale, but due to its antenna placement and ability to witness other, higher transmit scaled hotspots, it has higher average earnings.

@cokes518 yea, but isn't the point of the network to help expand it out.  While it is nice to earn HNT selfishly, if I am fixing MY location or asserting in an appropriate spot rather than with 30 other hotspots, I'm helping those who witness me and if others did this correctly, everyone would be helping everyone from an earnings perspective.
It just seems narrow focused to only think about YOUR position vs how your position can help others in the network, and vice versa.


Request: funnel icon

Would be great to represent filters

There are already two good icons that can work for filtering: MixerHorizontal and MixerVertical.
However, I took a pass at a generic funnel/filter icon. What do you think of these?

They both look great! I’d used the non-filled variant
I use Mixer for settings/config so its nice to distinguish filters from that
Thank you

I agree, having this icon will be great for filters. I hope radix-ui will have one soon too.


Word wrap in project notes text box

This seems to work. I'm not so sure about how the change from 10.0.1 to 8.0.171 happened and whether it is significant or not.

If you are talking about scene builder it is not a real issue.


v4 .card-columns support for webkit + firefox

.card-columns is making use of the column-count and column-gap properties:
https://github.com/twbs/bootstrap/blob/v4.0.0-alpha/scss/_card.scss#L287
In order to support all modern browsers, please add:
    -webkit-column-count: 3;
    -moz-column-count: 3;
    column-count: 3;

    -webkit-column-gap: 1.25rem;
    -moz-column-gap: 1.25rem;
    column-gap: 1.25rem;

See compiled CSS file: https://github.com/twbs/bootstrap/blob/v4.0.0-alpha/dist/css/bootstrap.css#L4079-L4086
.card-columns {
  -webkit-column-count: 3;
     -moz-column-count: 3;
          column-count: 3;
  -webkit-column-gap: 1.25rem;
     -moz-column-gap: 1.25rem;
          column-gap: 1.25rem;
}

Bootstrap uses Autoprefixer.

Ah, I see. I came across it from compiling myself. Is the official word for compass users to include Autoprefixer in their pipelines?

@justiniso Yes:

Requires a Sass compiler, Autoprefixer, and some setup.

This is still broken in firefox on 4.0.0-alpha.2

@tchoulihan Please re-test against v4-dev and then open a new issue if it's still broken, since this issue is so old (and might well have a different cause). Thanks.


Aws skill builder Course for cloud Practitioner

#https://github.com/avinash201199/Free-courses-with-Certificates/issues/253

@avinash201199 Only I make 2PR in this repo 1is accepted & this one is remaining please consider it.

please resolve conflicts

any updates ?

I did not understand the issue of conflict & trying to resolve it.

then please delete the repo and fork again to do your changes


Broken check due to server error

When running the bundlesize command I am getting a server error when fetching the cached bundlesize:
><EMAIL_ADDRESS>bundlesize
> bundlesize --enable-github-checks

FetchError: invalid json response body at https://bundlesize-cache.vercel.app/?repo=jonkoops/Leaflet reason: Unexpected token A in JSON at position 0
    at /home/runner/work/Leaflet/Leaflet/node_modules/node-fetch/lib/index.js:273:32
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Object.get (/home/runner/work/Leaflet/Leaflet/node_modules/bundlesize2/src/pipeline/cache.js:9:12)
    at async Object.read (/home/runner/work/Leaflet/Leaflet/node_modules/bundlesize2/src/pipeline/cache.js:50:27)
    at async run (/home/runner/work/Leaflet/Leaflet/node_modules/bundlesize2/index.js:22:25) {
  type: 'invalid-json'
}

─ dist/leaflet.js
  ✖ dist/leaflet.js    140.35KB > 140 kb no compression

─ dist/leaflet.js
  ✖ dist/leaflet.js    40.39KB > 40 kb gzip

  2 checks failed

Added check to GitHub
Error: Process completed with exit code 1.

For more information see the run in Github Actions: https://github.com/jonkoops/Leaflet/runs/4803353449

👀 can reproduce, not sure why. I checked the database, the connection pool has more room 🤔
{
"errorType":"Runtime.UnhandledPromiseRejection",
"errorMessage":"KnexTimeoutError: Knex: Timeout acquiring a connection. The pool is probably full. Are you missing a .transacting(trx) call?",
"reason":{
  "errorType":"KnexTimeoutError",
  "errorMessage":"Knex: Timeout acquiring a connection. The pool is probably full. Are you missing a .transacting(trx) call?",
  "name":"KnexTimeoutError"
}


Handle LocaleSidKeys with multiple underscores + (Latin)

The current method isn't handling multiple underscores in LocalSidKey.
There are multiple values with 2 underscores:

de_DE_EURO
uz_LATN_UZ
zh_CN_PINYIN
zh_CN_STROKE
zh_HK_STROKE
zh_TW_STROKE

uz_LATN_UZ has been handled with an ad-hoc rule since LATN means Latin and the country is UZ.
One of the impact is the following flag, that is not showing correctly for the value de_DE_EURO (e.g.):

Hi @baslu93 thanks for your PR !
Could you :

change the target branch to releaseCandidate ?
reference this fix in CHANGES.md file by following current convention ?

Thanks a lot !

Hey @tprouvot!
In CHANGES.md, I'll create a new bullet point (first in the list) in version 1.18 - General, with the following text:

Fix country codes from LocalSidKey convention https://github.com/tprouvot/Salesforce-Inspector-reloaded/pull/117 (contribution by Luca Bassani)

Is it good?

Hey @tprouvot!
In CHANGES.md, I'll create a new bullet point (first in the list) in version 1.18 - General, with the following text:

Fix country codes from LocalSidKey convention PR117 (contribution by Luca Bassani)

Is it good?

Check on releaseCandidate branch, there is already a new version 1.19 but for the text it's perfect 👏

Thanks for your contribution @baslu93 ! 🙏
If you are interested in more collaboration you can take a look at the the issues with "help wanted" label or even submit new feature requests !
Have a good day 🌞


[react-apollo] Misc. react-apollo improvements

Closes #2175
This enables us to annoate subscribeToMore with the following method
...
import { Query, type QueryRenderProps, type SubscribeToMoreOptions } from 'react-apollo'

class DoStuff extends React.Component<{}> {
  _subscribeToMoreOptions: SubscribeToMoreOptions<getThangsQuery, subscribeThangsSubscription> = {
    document: SUBSCRIBE_THANGS,
    updateQuery: (prev, {subscriptionData}) => {
      if (!subscriptionData.data) return prev
      ...
    }
  }

  render () {
    return (
      <Query query={GET_THANGS}>
        {({subscribeToMore, loading, error, data}: QueryRenderProps<getThangsQuery>) => {
          ...
          subscribeToMore(this._subscribeToMoreOptions)
          ...
        }}
      </Query>
    )
  }
}
...

similar functionality has been added for fetchMore.

I'll wait for @TLadd to sign off.
As part of this PR would you mind adding a CONTRIBUTING.md file in the apollo folder and recording any common reviewers? It's helpful to future contributors.

I've added the CONTRIBUTING.md file with all contributors to the main spec.

@AndrewSouthpaw Looks good to me

This appears to make flow unhappy:
$ flow status
Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ flow-typed/npm/react-apollo_v2.x.x.js:280:68

Cannot use SubscriptionProps [1] with less than 2 type arguments.

 [1] 273│   declare type SubscriptionProps<TData, TVariables> = {
     274│     subscription: DocumentNode,
     275│     variables?: TVariables,
     276│     shouldResubscribe?: boolean | (SubscriptionProps<TData, TVariables>, SubscriptionProps<TData, TVariables>) => boolean,
     277│     children: (result: SubscriptionResult<TData, TVariables>) => React$Node,
     278│   }
     279│
     280│   declare export class Subscription<TData> extends React$Component<SubscriptionProps<TData>> {}
     281│
     282│   declare export type MutationFunction<TVariables> = (options: {
     283│     variables?: TVariables,

@mic, can you check if this fixes it?


[feature request] Add volcano scheduler as a new GangScheduler

What would you like to be added:
I find that KubeDL now has two GangScheduler, kube-batch and kube-coscheduler, but miss the volcano scheduler. I want to add volcano scheduler as a new GangScheduler.
Why is this needed:
volcano scheduler is a popular gang scheduler, many users use it

If you think it is reasonable, I can achieve it.

hi @HeGaoYuan , the volcano gang api is derived from kube-batch, however, with the iteration of volcano project the api starts to appear different, it is welcomed if you can add volcano gang scheduler implementation!

@SimonCqk Thanks for your reply. I will try it.

@HeGaoYuan would you let us know which company do you work for  ?


SUBMARINE-470. [WEB]Implement database connection in department page

What is this PR for?
Implement database connection in department page with Angular.
What type of PR is it?
[Feature]
Todos
What is the Jira issue?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-470?filter=allopenissues
How should this be tested?
https://travis-ci.org/github/apache/submarine/builds/683207421
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@liuxunorg @kobe860219 @chiajoukuo Can you help me review the pull request? Thanks!

@jasoonn  Thanks for your contribution. It is looks good to me. From your pr, I learned more about Angular.


Relational query hasMany() not return Active Query as document cailmed

i am working on a content-tag system using yii2.
I have a Content(PK:id) table, I have a Tag(PK:id) table, and I have a junction table called Content_Tag (PK:content_id, tag_id).
now i want to show all content with same keywords in a grid view (sortable, searchable just as content index page) on keyword view page.
i'm using via in keywords Class according to the documentation as following:
 /**
 * @return \yii\db\ActiveQuery
 */
public function getContent()
{
    return $this->hasMany(Content_Tag::className(), ['content_id' => 'id']);
}

/**
 * @return \yii\db\ActiveQuery
 */
public function getContents()
{
    return $this->hasMany(Tag::className(), ['tag_id' => 'tag_id'])->viaTable('content_tag', ['content_id' => 'id']);
}

and modify the controller in action view
public function actionView($id)
{
    $query=$this->findModel($id)->problems;
    $dataProvider = new ActiveDataProvider([
        'query' => $query,
    ]);

    return $this->render('view', [
        'model' => $this->findModel($id),
        'searchModel' => $searchModel,
        'dataProvider' => $dataProvider,
    ]);
}

Now the error log shows $query in controller in ActiveQuery Class.

The "query" property must be an instance of a class that implements the QueryInterface e.g. yii\db\Query or its subclasses.

In which class you have the getContent and getContents methods declared?

query=$this->findModel($id)->problems;
->problems => What's that??
This is variable. You have to use class method for ActiveDataProvider.
Then you can use like this for it;
$query=$this->findModel($id);
And you have to use "joinWith" function for Relations table.

use
$model->getContents()

instead of
$model->contents;

retre


Error "CP group not initialized"

I use modal script and defaultl dataset, during preprocessing Step 1 works fine but during Step 2 I get errors in all the videos processed:
🎥 **Step 2: Run the VAE encoder on each video**
Timing load_encoder
Timing load_decoder
Stage                   Time(s)    Percent
load_encoder               1.12     28.88%
load_decoder               2.76     71.12%
Traceback (most recent call last)::00<00:05, 2.42it/s]
  File "/mochi/demos/fine_tuner/encode_videos.py", line 132, in batch_process
    preprocess(
  File "/mochi/demos/fine_tuner/encode_videos.py", line 65, in preprocess
    ldist.mean = cp_conv.gather_all_frames(ldist.mean)
  File "/mochi/src/genmo/mochi_preview/vae/cp_conv.py", line 67, in gather_all_frames
    cp_group = cp.get_cp_group()
  File "/mochi/src/genmo/mochi_preview/dit/joint_model/context_parallel.py", line 48, in get_cp_group
    raise RuntimeError("CP group not initialized")
RuntimeError: CP group not initialized00:29, 2.11s/it]

working on this now

I'm not able to replicate this, neither on my cached version of the git clone from when the example was written, nor on the current main commit if I clear my build cache.
So will need help from you on repro:

please confirm your script is unmodified from the example. If modified, please share the whole script here.

please confirm that you've run modal run main::download_videos and modal run main::download_weights

make the below modification and share the commit hash here.

base_img = (
    modal.Image.debian_slim()
    .apt_install("git", "ffmpeg", "bc", "zlib1g-dev", "libjpeg-dev", "wget")
    .run_commands(CLONE_CMD)
    .workdir(REPOSITORY)
    .run_commands("git rev-parse HEAD") #  <-------- This
    .pip_install("gdown", "setuptools", "wheel")
    .run_commands('pip install -e . --no-build-isolation')
)

and the build should show you the commit you've checked out. Something like:
=> Step 1: RUN git rev-parse HEAD
f3a800aea5862b4af13e66ff77eea1967c8c3a7f

Thanks

I tested on the prior commit abe3e102f40408bbf46cfda3928fc44f28a663c4 and it works without issue. Meaning the source of the regression is somewhere in https://github.com/genmoai/mochi/commit/f3a800aea5862b4af13e66ff77eea1967c8c3a7f
cc @ajayjain

@Curlypla @erik-dunteman We're working on fixing the regression with https://github.com/genmoai/mochi/pull/99/files
https://github.com/genmoai/mochi/commit/f3a800aea5862b4af13e66ff77eea1967c8c3a7f simplifies video encoding by only needing a single GPU, but it looks like there was a collective communication that was not removed.

@erik-dunteman could you try again? Thanks!

It works now on latest commit, thanks!
However, the modal script would have to be modified by adding a larger timeout for preprocessing, otherwise it won't have time to finish (I put it at 30 minutes and works fine)

Timeout bumped to 30min as requested
https://github.com/genmoai/mochi/pull/105


Add a special UNREACHABLE action for rule that never execute their action

The main motivation for this is to get accurate coverage reports for rules used to generate custom syntax errors

@MatthieuDartiailh I think I am missing an example on how this is supposed to work to properly evaluate the proposed solution.

Consider this rule from the python grammar:
del_stmt[ast.Delete]:
    | 'del' a=del_targets &(';' | NEWLINE) { ast.Delete(targets=a, LOCATIONS) }
    | invalid_del_stmt { UNREACHABLE }

The invalid_del_stmt will never return a non-None production meaning that the action will never be executed. This is fine except if one wants to get accurate coverage. Typically the solution is to have the action be None  # type: ignore. However one cannot use comment in an action so we need some indirection which the UNREACHABLE special action provides.

The invalid_del_stmt will never return a non-None production meaning that the action will never be executed. This is fine except if one wants to get accurate coverage. Typically the solution is to have the action be None # type: ignore. However one cannot use comment in an action so we need some indirection which the UNREACHABLE special action provides.

Could we just inject the None # type: ignore in all invalid_* rules that don't return anything? Having to add UNREACHABLE can be a bit verbose and is another magical constant. What do you think?

Actually the used of && in invalid rules is another case see:
invalid_with_stmt[None]:
    | ['async'] 'with' ','.(expression ['as' star_target])+ &&':' { UNREACHABLE }
    | ['async'] 'with' '(' ','.(expressions ['as' star_target])+ ','? ')' &&':' { UNREACHABLE }

I agree it is verbose but I do not mind being explicit about it and I am not sure what is the most easy to discover, a special action or an automatic action based on name. I do not have a strong opinion either way I just want a way to get the coverage right.

I agree it is verbose but I do not mind being explicit about it and I am not sure what is the most easy to discover,

Right, but my suggestion was done because this is not an operational behaviour, but just a development one: we want to properly check coverage. Adding things in the grammar that don't result in semantic behaviour raises a lot the noise IMHO
On the other hand, if doing it automatically is too hard, I am fine with this if @lysnikolaou is

I do not think doing it automatically would be very hard but it would not allow to catch all cases as with the rules inside the invalid_with_stmt . However this is a corner case and those could be re-written as you mentioned at one point.

Do you have a preference between merge and rebase for handling the conflicts ?

Do you have a preference between merge and rebase for handling the conflicts ?

Whatever is easier for you. I normally rebase, but we can squash at the end so is not a problem

@pablogsal I picked a middle ground and made the addition of UNREACHABLE automatic for alts with no action of rules starting with invalid. That way it is still possible to mark custom rules but it does not clutter the grammar file in the most common case. WDYT ?

ping @pablogsal @lysnikolaou

Thanks for the work and the wait, @MatthieuDartiailh


Merged CameraRecoilComponent into EyeComponent

You can now viewkick from a shared engine context through EyeComponent.
Requires https://github.com/space-wizards/RobustToolbox/pull/1597.

Bad idea.


[DOC] Notebook cell number rendering on docs don't enumerate correctly

Brief Description of Fix
In some of the notebook examples, the cells that indicate the order in which cells are run are off by a few steps. In the below screenshot, the first cell has [1] (as expected). The next cells have [64] and [65]. This may be confusing to new users that don't typically work in notebooks.

Currently, the docs display notebooks that may have cells that are run out of order or run not in sequential order.
I would like to propose a change, such that now the docs display notebooks that render with numerically consistent (and sequentially) run steps. (i.e. [1], [2], [3], ..., etc.).
The simple (but tedious) solution, would be to find notebooks with these issues, re-run them, and then push the changes.
A better (and recommended) approach would be to create a script that automatically runs each notebook used as an example from end to end prior to being used in the docs, to ensure the cells will be run in order and sequentially.
Relevant Context

Link to documentation page
Link to exact file to be edited

Great idea!
I agree with the better approach (script to run each notebook before uploading to docs)!
Would you like to be assigned to this? :D

Is this something that should be added to the make file, and should be run every time, or only when someone knowingly updates a notebook?
Maybe there is something that can be added to the PR pipeline that ensures cells are sequentially numbered, and requests that you run the script if they're not.

I wrote a little script that scans through the notebooks and raises exceptions if any of the code cells haven't been executed (likely empty cell at end of notebook) or if the cells have not been executed sequentially:
import json
from pathlib import Path

import numpy as np

def check_cell_num(file_name):
    with open(file_name, "r", encoding="utf8") as f:
        notebook = json.load(f)
    execution_counts = np.array(
        [
            cell["execution_count"]
            for cell in notebook["cells"]
            if cell["cell_type"] == "code"
        ],
    )
    if None in execution_counts:
        raise Exception(f"{file_name} contains unexecuted code cells.")
    if not np.all(np.diff(execution_counts) == 1):
        raise Exception(f"Out of order cells in {file_name}")

notebook_dir = Path("examples/notebooks/")

for file in notebook_dir.glob("*.ipynb"):
    check_cell_num(file)

Perhaps this could be included in MAKE etc, and if it fails, the notebook is run with papermill etc. Or simply fails and directs the user to run papermill

@ericmjl
I’d like to revisit this issue. I’m thinking I can take the code @hectormz provided and add a few things that I had in mind.
Just wondering, will this be like scripts/check-autodoc.py? Essentially, we’ll call it during test pipeline? Wondering if there was a way to get it into a pre-commit hook so the error is thrown prior to a pipeline build. Curios to hear what you think here.
I’m happy to take this one, just didn’t know where this file/function should “live”.

I forgot about this! @loganthomas let me know if you have any questions about what I offered up, and I can jog my memory too


Look into strings enclosed in "" eg. paths and strings with spaces these will need to be supported

This is a non-issue since "" are handled by the console application so "C:\file directory\file.txt" will be treated as one index in the args array.


Camera proxy component is no longer working without api_password

**Home Assistant release with the issue:0.77dev (Sep 3)
**Operating environment (Hass.io/Docker/Windows/etc.): manual install on rpi
Component/platform: camera/proxy
Description of problem:
When trying to use a camera proxy, I now get:
Login attempt or request with invalid authentication from <ip address of client machine>

Requesting the original camera entity instead works properly
camera:
  - platform: foscam
    ip: <IP_ADDRESS>
    username: !secret camera_user
    password: !secret camera_password
    name: livingroom_orig
  - platform: proxy
    entity_id: camera.livingroom_orig
    name: livingroom
    max_image_width: 480
    image_quality: 50

What is your http config? Any auth_provider config?

You have two options for now

add http.api_password
add http.trusted_networks included <IP_ADDRESS>

We will sunset api_password, but not now. We have some discussion regarding camera.proxy, I forget where, but conclusion is we need rewrite this component since it is ill implemented, too much overhead.

To be fair, when I originally wrote the camera proxy, I did exactly what you said, and was told to rewrite it this way because components aren't allowed to directly talk to other components.  I'll submit a pull request to the documentation page, so at least it can be activated, but I would like to implement a proper solution that doesn't need to bypass security

LOL, didn't noticed that you are the author 😄
I think the proper way maybe is providing a common function and config options in camera base component, and allow user to set width/height or screen resolution, or even some special effect on camera entity.

Can't you just call hass.components.camera.async_get_image(camera.entity_id) and not worry about fetching using http? Or is this what you were doing originally that you were told wasn't allowed?

@awarecan - Your idea of adding this functionality as a general property of the camera was also one of my multiple implementations, and was also rejected at the time :)


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #228   +/-   ##
=======================================
  Coverage   97.54%   97.54%
=======================================
  Files           1        1
  Lines         122      122
=======================================
  Hits          119      119
  Misses          3        3

Continue to review full report at Codecov.

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


TrackedController Index for Oculus Touch are set to uint.max, not allowing button presses to work

Environment

Source of VRTK (Github)
Version of VRTK (c31cc32)
Version of Unity3D (Unity 5.5.0f3)

Steps to reproduce
Import OculusUtilites
Drag OVRCameraRig into Scene
Set VRKT_SDK Manager to OculusVR
Expected behavior
Button presses work on Oculus Touch Controllers
Current behavior
Button presses are not working because the TrackedController.index is set to uint.max
Tracked the issue to commit 8d5e856
It looks like it might be caused by the Update function getting called before Enable
A temporary solution I have done is moving line 95
            index = currentIndex;
to inside the previous IF block

Fixed with https://github.com/thestonefox/VRTK/commit/e4e5b58d8f72c42a6ca9f2085471db1f89bbf86f


Tesseract OCR not giving correct text from camera image for iOS platform

Hi,
I downloaded sample code given on github and try to take photo from camera for iOS platform . Image  taken from camera has text but result got wrong from this library. Its giving correct text from image which is added in application bundle but whatever photo taken from camera its giving wrong text.
Please fix this issue.
Thanks

It looks like the main reason is the actual "Template Framework Project" rotates by 180% the image. I will do a pull request with the fix soon, in the meanwhile you can test it taking the picture while rotating your phone by 180% (upside down).

Hi,
Seem, this issue still exit.
Please fix it.
Thank you!

This may be solved by passing along the image orientation at every image type conversion (like CGImage to UIImage etc).
I've added an extension to handle this on my conversions from UIImage to CGImage like this:
extension UIImageOrientation {     func getCGOrientationFromUIImage() -> CGImagePropertyOrientation {         // call on top of UIImageOrientation to obtain the corresponding CGImage orientation.         // This is required because UIImage.imageOrientation values don't match to CGImagePropertyOrientation values         switch self {         case .down: return .down         case .left: return .left         case .right: return .right         case .up: return .up         case .downMirrored: return .downMirrored         case .leftMirrored: return .leftMirrored         case .rightMirrored: return .rightMirrored         case .upMirrored: return .upMirrored         }     } }
The thing is: UIImageOrientation.up may correspond to different raw values of CGImagePropertyOrientation.up. This is why this becomes an issue. I was previously getting all of my photos rotated by 90 degrees without specifically applying any transforms to them. This solved it.
Good luck!


Add status via CodeLens

Renaming files in VSCode is a mess, it requires a saved file and it creates a new editor if you just rename a file, you have to take care of closing the previous editor. Madness.
So this extension will not use 👻🌕 as an indicator (which isn't great to begin with)
From a cursory check of VS Code’s capabilities, I think using a CodeLens widget on the first line is the best way to expose this information.
It would something like like:
	 👻 GhostText is connected [ Disconnect ]
  1  Lorem ipsum
  2  dolor sit
  3  amet

This has not been implemented it. The current code was copied from:
https://github.com/microsoft/vscode-extension-samples/blob/main/codelens-sample/src/extension.ts

It should be good enough for now. Here's a demo showing:

it disconnects on click
it disconnects on document closure
it works with two open fields

https://user-images.githubusercontent.com/1402241/226172284-a59a9ef2-b7b4-46ff-8af2-2f6863a8038a.mov
Side note: this code sucks, the API too.


[template-build] - review build strategy

The build strategy is too restrictive, we need to be able to create arbitrary folders and assume they are transpiled (e.g. shared, etc.)
A possible strategy would be to babel everything, maintaining the directory structure, and webpack declared browsers clients. It would probably simplify the build scripts too.

ok this should be working in a more consistent way.
Beware that I published the fix as a minor version (which is sully, it should have been a patch), because the former behaviour was kind of a bug, so previous workarounds may break.


Upgrade Containers

Create a new option that will allow the user to upgrade their build containers.
i.e. a PHP container might be on 7.3.7 and the user wants to upgrade to the newly released 7.3.8
To do this normally using Docker commands, you would:
Download the latest image
docker pull php:7.3
Stop and remove the container
docker-compose down -v  --rmi local --remove-orphans php_container
Start up the containers
docker-compose up -d --build
I propose a command similar to
saber update [OPTIONS] [IMAGE]
*Options
-a | --all - Updates all containers. Default
Image
If -a is not specified, an image can be specified
Example:
saber update --all
saber update php:7.3
Perhaps consider specifying multiple images to update in parallel

So the Docker CLI doesn't let you pull multiple images at the same time.
If the user chooses to upgrade all the containers, a list of images would need to be retrieved and then looped over to pull in the latest image.
Running
saber upgrade

should indicate that all images want to be upgraded.
If a single container is to be upgraded, it can be requested
saber upgrade php:7.3-fpm-alpine

and that will upgrade that image.
Perhaps use something like Symfony's Question Helper and allow the user to choose from a list of images to upgrade?

Currently, running the upgrade command will - by default - show a list of images stored locally. Select the corrosponding images (you can choose multiple by commar seperating them)  by selecting the number and it will pull in any new images (if any - otherwise it will inform you their's no new images to download) and then clean up and rebuild the containers.
Because I've opted for this method, I will not have the option to supply a single image to upgrade, you just choose a single image from the supplied list.
I may not have an --all option either as potentially, there could be a lot of images to upgrade and that could make the task seem to be taking a long time. Giving the user options seems the best method.
Perhaps in the future, in-case any automation is needed, there can be changed to allow for images to be upgraded without any user interaction, but will cross that bridge when if we get to it.


Assert child node's location is non-overlapping for known nodes

We have some cases where it's acceptable for a child node's location to be non overlapping with its parent, and others where the child node's location needs to be strictly within its parent's location.
Our test suite should check if child nodes' locations are within parent's nodes' locations, and if not, categorize the node type

Started looking at this, fixed some cases, have this branch as a test branch now. It asserts that locations are non-overlapping. It notably ignores StringInterpolatedNodes because of Heredocs.
It has also generated this set of files that currently still have overlapping locations:

[ ] test/fixtures/classes.txt
[ ] test/fixtures/if.txt
[ ] test/fixtures/method_calls.txt
[ ] test/fixtures/methods.txt
[ ] test/fixtures/modules.txt
[ ] test/fixtures/procs.txt
[ ] test/fixtures/rescue.txt
[ ] test/fixtures/seattlerb/attr_asgn_colon_id.txt
[ ] test/fixtures/seattlerb/attrasgn_primary_dot_constant.txt
[ ] test/fixtures/seattlerb/case_in_42.txt
[ ] test/fixtures/seattlerb/case_in_42_2.txt
[ ] test/fixtures/seattlerb/case_in_67.txt
[ ] test/fixtures/seattlerb/case_in_86.txt
[ ] test/fixtures/seattlerb/case_in_86_2.txt
[ ] test/fixtures/seattlerb/case_in_hash_pat.txt
[ ] test/fixtures/seattlerb/case_in_hash_pat_assign.txt
[ ] test/fixtures/seattlerb/case_in_hash_pat_paren_true.txt
[ ] test/fixtures/seattlerb/case_in_hash_pat_rest.txt
[ ] test/fixtures/seattlerb/case_in_hash_pat_rest_solo.txt
[ ] test/fixtures/seattlerb/defn_oneliner_rescue.txt
[ ] test/fixtures/seattlerb/defs_oneliner_rescue.txt
[ ] test/fixtures/seattlerb/rescue_do_end_ensure_result.txt
[ ] test/fixtures/seattlerb/rescue_do_end_no_raise.txt
[ ] test/fixtures/seattlerb/rescue_do_end_raised.txt
[ ] test/fixtures/seattlerb/rescue_do_end_rescued.txt
[ ] test/fixtures/seattlerb/safe_attrasgn.txt
[ ] test/fixtures/seattlerb/safe_attrasgn_constant.txt
[ ] test/fixtures/symbols.txt
[ ] test/fixtures/unparser/corpus/literal/assignment.txt
[ ] test/fixtures/unparser/corpus/literal/block.txt
[ ] test/fixtures/unparser/corpus/literal/case.txt
[ ] test/fixtures/unparser/corpus/literal/def.txt
[ ] test/fixtures/unparser/corpus/literal/dstr.txt
[ ] test/fixtures/unparser/corpus/literal/literal.txt
[ ] test/fixtures/unparser/corpus/literal/pattern.txt
[ ] test/fixtures/unparser/corpus/literal/send.txt
[ ] test/fixtures/whitequark/array_symbols_interp.txt
[ ] test/fixtures/whitequark/if_elsif.txt
[ ] test/fixtures/whitequark/rescue_without_begin_end.txt
[ ] test/fixtures/whitequark/send_attr_asgn.txt
[ ] test/fixtures/whitequark/send_attr_asgn_conditional.txt

cc @haldun  - figured this might be interesting to you

Agreed, I think that makes. It's a bit like FILE or DATA.read, they sort of are string literals but the contents isn't at the same place as the usage.


update deployment runner

change deployment runner to self-hosted runner align with the runner label

/deploy

/wandb 2acwxb8n


Add first code commit

Add first code commit

ok

Fixed with pull request: https://github.com/AMLResearchProject/ALL-FastAI-2019/pull/2


ignoreHtmlClass not checked for MathML content

Issue Summary
The ignoreHtmlClass option seems to have no effect on MathML rendering. Maybe it is intended behaviour – I couldn't tell from the documentation.
Steps to Reproduce:

Configure

window.MathJax = {
  options: {
    ignoreHtmlClass: 'nomathjax',
  },
};

Run it on

<math><mi>x</mi></math>
<math class="nomathjax"><mi>x</mi></math>

Both <math> tags are processed by MathJax

There are situations where I'd like MathJax to render most of the MathML content, but occasionally allow the browser use its native MathML renderer (because e.g. of images embedded in <mtext>). I thought ignoreHtmlClass would help here.
Technical details:

MathJax Version: 3.2.0

Right now, ignoreClass and the related options only affect TeX and AsciiMath processing, not MathML.  The mechanism for finding MathML tags is different, and doesn't require walking the DOM tree, as is required for TeX and AsciiMath.  I've considered doing the same for MathML, but it hasn't been a priority.
On the other hand, you could override the current FindMathML object in the MathML input jax's options and have it filter out certain math nodes, say ones that are marked with class="mathjax_ignore".  Here is a configuration that does that:
MathJax = {
  startup: {
    ready() {
      const {combineDefaults} = MathJax._.components.global;
      const {FindMathML} = MathJax._.input.mathml.FindMathML;

      class myFindMathML extends FindMathML {
        processMath(set) {
          const adaptor = this.adaptor;
          for (const node of set.values()) {
            if (adaptor.hasClass(node, 'mathjax_ignore')) {
              set.delete(node);
            }
          }
          return super.processMath(set);
        }
      }

      combineDefaults(MathJax.config, 'mml', {FindMathML: new myFindMathML()});

      MathJax.startup.defaultReady();
    }
  }
};

Thanks! I saw the FindMathML option in the docs, but I had no idea how to use it. That code should solve my problem.


DO NOT MERGE: test appveyor latest pthreads

Signed-off-by: Alin Gabriel Serdean<EMAIL_ADDRESS>

Testing done.


Does the confd support Go1.8.1?

When i try to compile confd on Go1.8.1, the error is  as follows:
$ go build
# github.com/kelseyhightower/confd/resource/template
resource\template\util.go:49: undefined: syscall.Stat_t
resource\template\util.go:50: undefined: syscall.Stat_t

and I found the source code syscall found that the standard library did not release the Stat_t state.
So does it support 1.8.1?

Go 1.8 is supported but Windows is not.
漂流<EMAIL_ADDRESS>schrieb am Di., 4. Juli 2017, 05:18:

When i try to compile confd on Go1.18, the error is as follows:
$ go build
github.com/kelseyhightower/confd/resource/template
resource\template\util.go:49: undefined: syscall.Stat_t
resource\template\util.go:50: undefined: syscall.Stat_t
and I found the source code syscall found that the standard library did
not release the Stat_t state.
So does it support 1.18?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/confd/issues/575, or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFc8wABcU7SsCWbHOZXz53PJJxlkq1aks5sKa8QgaJpZM4OM16d
.

@HeavyHorst I have seen # 434 branch support for the win platform, this branch has a merger plan? If I now want to use under the win confd, how can I to do it?

You can try this fork of confd: https://github.com/bacongobbler/confd
On Tue, Jul 4, 2017 at 7:38 AM 漂流<EMAIL_ADDRESS>wrote:

@HeavyHorst https://github.com/heavyhorst I have seen # 434 branch
support for the win platform, this branch has a merger plan? If I now want
to use under the win confd, how can I to do it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/confd/issues/575#issuecomment-312784804,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQFc87uRJlqbr6e2wOscb2AL4eaYbxQQks5sKc_MgaJpZM4OM16d
.

So cool, I have compiled it successfully, thanks


Subdomain duplicated in the exported files

Unless I misunderstood something, I think there is a bug in the latest version of the plugin (2.6) : the subdomain I chose (static.ndd.com) was not the correct one in the exported files. Instead, I had the same subdomain twice, like so : static.static.ndd.com.
Here are my settings :

And here is the beginning of the index.html file :
<!DOCTYPE html>
<head>
<base href="https://static.static.voiretmanger.fr" />

<meta charset="UTF-8">

<meta name="apple-mobile-web-app-title" content="Voir et manger" />
<meta name="viewport" content="width=device-width, initial-scale=1.0, viewport-fit=cover">
<meta property="fb:pages" content="384298108368" />

<link href='https://static.static.voiretmanger.fr/contents/ui/theme/css/font.css' rel='stylesheet' type='text/css'>
<link rel="stylesheet" media="all" href="https://static.static.voiretmanger.fr/contents/ui/theme/style.css" />

<link rel="shortcut icon" href="https://static.static.voiretmanger.fr/contents/favicon.ico" />
<link rel="mask-icon" href="https://static.static.voiretmanger.fr/contents/icon.svg" color="rgb(197, 31, 22)">

        <link rel="preload" href="https://static.static.voiretmanger.fr/contents/ui/theme/css/portico_layer-webfont.woff2" as="font" type="font/woff2"$
        <link rel="preload" href="https://static.static.voiretmanger.fr/contents/ui/theme/css/portico_vintage-webfont.woff2" as="font" type="font/woff$

<!-- This site is optimized with the Yoast SEO plugin v7.6.1 - https://yoast.com/wordpress/plugins/seo/ -->
<title>À voir et à manger - Nicoflo</title>
<meta name="description" content="Des critiques de films et de séries, ainsi que de restaurants."/>

Let me know if you need more information.
Thanks for the plugin !

Hi Nico,
Thanks for using the plugin, sorry it doesn't seem to be working in this case. I'm looking into this now and will update you.
Cheers,
Leon

Hi Nico,
I have an idea that there is a bug related to having the same url in the output directory. Could you please try setting the Output Directory Override to /var/www/mystaticoutput and see if that works?
Cheers,
Leon

Although, your web user may not have permissions to create/write to that folder...

Hello
Thanks for your answer !
I tried your suggestion, but the result was the same.
One odd thing I noted : the export is done in a subfolder, inside the folder I filled. Is this what should happen ?

Hi Nico,
Good question about "what is supposed to happen" in that scenario. I'm really looking forward to having some documentation and code testing soon. Originally, it would put all the export files in the folder you specified, but there's a high chance that got modified in the last few months of changes (lack of testing of that area to ensure consistent behaviour).
Could you please send me your Export Log and your PHP error log from the time of an export? If you prefer, you can email it to me<EMAIL_ADDRESS>I'm very curious as to what is causing this issue and would like to solve it.
Cheers,
Leon

Weirdly, the export log was empty during the whole process… 🤔
As for PHP, I found this :
11/Jun/2018:21:21:13 +0200 [ERROR 0 /wp-admin/admin-ajax.php] PHP message: PHP Warning:  unlink(/var/www/static.voiretmanger.fr/wp-static-html-output-1-1528742059-nicolinux/wp-login.html): No such file or directory in /var/www/voiretmanger.fr/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php on line 1119
PHP message: PHP Warning:  unlink(/var/www/static.voiretmanger.fr/wp-static-html-output-1-1528742059-nicolinux/wp-login.php): No such file or directory in /var/www/voiretmanger.fr/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php on line 1120
11/Jun/2018:21:21:14 +0200 [ERROR 0 /wp-admin/admin-ajax.php] PHP message: PHP Warning:  unlink(/var/www/static.voiretmanger.fr/wp-static-html-output-1-1528742059-nicolinux.zip): No such file or directory in /var/www/voiretmanger.fr/wp-content/plugins/static-html-output-plugin/library/StaticHtmlOutput.php on line 1169

Hi Nico,
OK, those 3 PHP warnings can be ignore, there is a fix in the code already that will go out in 2.6.1 to fix a few bugs and clean up the error log output.
One other question: What is the URL format of the server you are running your WordPress on? ie, is it in a subdirectory, by any chance?
http://mydomain.com/blog/wp-admin vs http://mydomain.com/wp-admin
There has been identified an issue which may cause incorrect detection of the site's root if it's in a subdirectory. I will be investigating further this evening.
If we still haven't figured it out, you can either send me an archive of your whole development site and database to test here or share me access to debug on your server. Either way, email would be safer.
Cheers,
Leon

Hi Nico,
I've just built you a customised version of the plugin that should remove the duplicate static.static. entries. It doesn't solve the underlying problem as I'm still unsure what is causing this, but it should allow you to get your site exported :smile_cat:
https://github.com/leonstafford/wordpress-static-html-plugin/blob/master/nico_rm_duplicate_static.zip
Please let me know if this gets you a decent export.
Cheers,
Leon

For reference, the line I've changed is in the file library/StaticHtmlOutput/UrlRequest.php on line number 1182.
In case we don't find the solution before the next couple of releases, you can just add this line again to be:
$responseBody = str_replace('static.static.', $new_wp_includes, $responseBody);

OK let me try the updated (and personalised) version. :-)
For information, I already had great results by searching and replacing each generated files (command line is great at this game).
Here's the original blog :  https://voiretmanger.fr
And here's the static version : https://static.voiretmanger.fr
Some resources are missing, like file fonts, but I guess it's because they are loaded from a CSS file and not directly from WordPress pages ?
Otherwise, it works great ! 👍

Ah, superb!
See this attached, it shows the browser console, which will hint at those missing files from JS (css ones should be picked up by the plugin now).
Grab each of the URLs, replace the domain with the original site's and then put them in the Additonal URLs field in the plugin.
This should then capture them properly and prevent any missing.
This is still an unfortunate part of the plugin not being JS aware in its scraping (a much bigger task to fix, I think, but not impossible).

ie, add https://voiretmanger.fr/contents/ui/theme/css/lmroman10-regular-webfont.woff in the Additional URLs text area (one url per line)

OK, let me try that and your personalised version. :-)

Hi @nicolinuxfr,
Did you ever figure out why the duplicated static.static.? I'd still like to look into this further to prevent others encountering the issue.
I plan to test running a site on a similar static. subdomain, just to eliminate that being the cause..
Cheers,
Leon

Ah, also, the https/SSL has not been thoroughly tested yet, but a potential fix for that was recently added.
That is another note for myself here to test against :)

@nicolinuxfr just keeping you in the loop, I'm able to reproduce this issue in my local development environment now, so the fix should be a lot closer. I'll be updating the issue description as I go.
Thanks again for raising this issue. I hope to have this and the SSL issues sorted soon.
Cheers,
Leon

Thanks for the updates !
I’m sorry I didn’t have time to try again. And unfortunately, I don’t think I can use this solution right now. My blog is too big and export is too long to be useful.
Maybe if you can manage to export only what’s necessary each time some day, but I guess it would be pretty hard.
Anyway, thanks for your help on this. 🙂

@nicolinuxfr the diff-only export is in the works :smile_cat: I'm a big git fan, so it makes sense.
I'll also have scp/ssh based deployments soon with either pwd or key, as I also feel it's way too long to export somewhere like S3 with a reasonably sized site.
With scp/SSH, it should be as:

crawl site
create zip
scp zip
ssh remote cmd to unzip into desired dir

That alone should speed up the slowest part of the process, but with exporting the delta only, the whole process should be :zap: fast stay tuned/watch the repo.
Cheers,
Leon

Okay now that would be interesting indeed ! 👌
I will follow the project then. Thanks !


Update README.md

I made a few basic grammatical edits. I read about 75% and ran out of time. :)
I'll try to return to the rest and I'm sure Sonja will have feedback as well.

Thank you very much :)


DOCS: added local llama2 notebook

Adds a simple notebook to docs, talking about how to use local llama2 (without replicate etc.)
Note: I am not sure if something else needs to be done to get this to show up in the docs site, please let me know if so.

@tjaffri try fixing lint errors w/ make format; make lint

Thanks for the notes!
Does index = VectorStoreIndex.from_documents(documents) use a local embedding model? How do we ensure the embedding construction does not use non-local resources?

Thanks for the notes!
Does index = VectorStoreIndex.from_documents(documents) use a local embedding model? How do we ensure the embedding construction does not use non-local resources?

by default it uses openai text-embedding-ada-002, but you can customize here https://gpt-index.readthedocs.io/en/latest/core_modules/model_modules/embeddings/root.html

Thanks all... @jerryjliu @logan-markewich I think this can be merged now, have a great weekend

@jerryjliu @logan-markewich thanks for the merge... but I don't see this notebook in the docs? https://gpt-index.readthedocs.io/en/latest/core_modules/model_modules/llms/root.html
Do we need to do something else to have this picked up in the docs? let me know how i can help, thanks


SshTest: ssh_exchange_identification: Connection closed by remote host

While running the unit tests on my laptop, in Ubuntu 17.10 beta (Artful Aardvark) I am seeing tests in tests/ssh_test.py fail while trying to connect, e.g.
$ PYTHONPATH=. python tests/ssh_test.py
.E....F.
======================================================================
ERROR: test_password_incorrect (__main__.SshTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/ssh_test.py", line 63, in test_password_incorrect
    password='badpw',
  File "/home/alex/src/mitogen/tests/testlib.py", line 99, in docker_ssh
    return self.router.ssh(**kwargs)
  File "/home/alex/src/mitogen/mitogen/master.py", line 938, in ssh
    return self.connect('ssh', **kwargs)
  File "/home/alex/src/mitogen/mitogen/master.py", line 959, in connect
    return self._connect(context_id, klass, name=name, **kwargs)
  File "/home/alex/src/mitogen/mitogen/master.py", line 945, in _connect
    stream.connect()
  File "/home/alex/src/mitogen/mitogen/ssh.py", line 68, in connect
    super(Stream, self).connect()
  File "/home/alex/src/mitogen/mitogen/master.py", line 760, in connect
    self._connect_bootstrap()
  File "/home/alex/src/mitogen/mitogen/ssh.py", line 80, in _connect_bootstrap
    time.time() + 10.0):
  File "/home/alex/src/mitogen/mitogen/master.py", line 182, in iter_read
    (''.join(bits)[-300:],)
StreamError: EOF on stream; last 300 bytes received: 'ssh_exchange_identification: Connection closed by remote host\r\n'
...

The failures are not 100% consistent. I've seen the following variations

"Connection reset by remote host", rather than "Connection closed by remote host"
Different tests (e.g. SshTest.pubkey_required, SshTest.pubkey_specified) exhibiting the error

Investigations so far.

Rebuilding the Docker image on debian:unstable, ubuntu:zesty, or ubuntu:artful has not not fixed it.
Setting TCPKeepAlive=no has not fixed it
Running the same test in a 17.10 beta VM has not reproduced it so far

A probably unrelated detail. I found my .ssh/known_hosts had a line consisting of the string "Needs output, otherwise thinks ssh-keyscan timed out'" repeated a number of times. That appears to have originated from running Ansible unit tests on another occasion.
Fixing my known_hosts did not fix this issue.

This looks like a race condition. I think ssh is trying to connect/authenticate before sshd has completely started. As a workaround time.sleep(5.0) adding in DockerMixin.setupClass() allows these tests to complete without errors.
Pull request to follow, once I've read up on socket.connect() and timeouts.


dep init: doesn't recognise symlinked project

I'm not sure if this is a bug, but fixing this will be a great quality of life improvement.
What version of Go are you using (go version)?
go version go1.8 darwin/amd64
What operating system and processor architecture are you using?
macOS Sierra 10.12.3 (darwin amd64)
What did you do?
GOPATH is set at ~/gopath
I was working on a project that accessed via a symlink as provided below.
Full GOPATH

~/gopath/src/github.com/klrkdekira/myproject

Symlinked path

$ pwd
~/myproject

What did you expect to see?
dep init works.
What did you see instead?
This.

After invoking cd -P it just works.

Symlinks are tough - just in general, but also in particular because we have limited wiggle room to deal with them when a) the go toolchain already has strong opinions about them and b) we're still so sensitive to GOPATH for naming (e.g. #313).
Still, we have #247 going, which I hope we can merge soon, at least in the spirit of experimentation.

I have a same problem

#247 is merged, so closing this :)


midori: fails to evaluate at nixos-unstable

Describe the bug
https://hydra.nixos.org/build/147767715/nixlog/2
error: while querying the derivation named 'midori-9.0':
while evaluating the attribute 'buildInputs' of the derivation 'midori-9.0' at /build/source/pkgs/stdenv/generic/make-derivation.nix:203:11:
while evaluating 'getOutput' at /build/source/lib/attrsets.nix:486:23, called from undefined position:
while evaluating anonymous function at /build/source/pkgs/stdenv/generic/make-derivation.nix:154:17, called from undefined position:
attribute 'gtk3' missing, at /build/source/pkgs/applications/networking/browsers/firefox/wrapper.nix:184:23

Notify maintainers

@ajs124 (I suspect https://github.com/NixOS/nixpkgs/commit/c6e51d03695efa90a8d09b41fe6441953e21d52a)
@7c6f434c @ramkromberg for midori
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
 - midori
# a list of nixos modules affected by the problem
module:

Note that the job was at 78624ce1ded, so it was after ed22cfb195798dd11a44df5aa13b9b753b7a8a1a (midori|vimb: fix tarball job)

Ah, but only chronologically, it hadn't been merged yet. Alright.


⚠ ClamAV has degraded performance

In 1739147, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 604 ms

Resolved: ClamAV performance has improved in 74a400e.


Complete support for relative links in Markdown page

Feature request

What problem does this feature solve?
<!-- this works -->
[about](./foo.md)

<!-- this doesn't -->
[contact](./contact.vue)

We should support all possible extensions here https://github.com/saberland/saber/blob/fb2832440886cc1ed3a5262fdb7a90cee7aa8125/packages/saber/lib/markdown/link-plugin.js#L28
Extensions can be retrieved from api.transformers

We should also document this feature on https://saber.land/docs/markdown-features.html 😅


Merge pull request #3 from Njoura7/main

Create LICENSE

practicing pull requests from different branches..never mind:)!


Add grammar check for text input and translation result in extension popup

Add checkbox to enable grammar check.
But first we have to find a service to suggest grammar fixes. Prefer an offline solutions

The idea
When checkbox enabled, text input will highlight errors and suggest to fix all by some button.
Also we can add checkbox on translation result, in this case translation will additionally handled by grammar fixer before show to user


You can close this, I figured out that I need to change to windows containers. To do so, I need to update Docker to unstable beta release.
https://blog.docker.com/2016/09/build-your-first-docker-windows-server-container/


New imdb validation and benchmark improvements

Commit history got messed up. Don't know what happened.

closing, this should merge soon


usage example neeeded!

unlike the flutter version of this project, RN doesn't have an example code
can you provide a runnable demo on how to implement it into codes?

hey, @AnasAlamin we will be adding a sample rn sample here Sample App in the next few days.


make it multi-chain

decide whether to make it multi-chain (right now it only works on a single chain's project.json) and extend the code accordingly

Closing as currently out of scope.


Error: Permission denied to access property 'toJSON'

le.min.js (line 7, col 462)
Firefox 36 windows 8.
Tell me if more data needed (notably retry with le.js).

@lajarre do you have a stack-trace or exact console message?


Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 33554440 bytes)

https://github.com/fruitcake/laravel-cors/blob/d24d345f9a05dff12b88903171c0444a2b9e76e7/src/HandleCors.php#L51
This code should move to ServiceProvider

@lee2son try to change in php.ini the memory_limit to:
memory_limit = -1
and test later... work for me

try with composer self-update


Advanced Views

can take an object, which is used as a spec for BaseView.Model.extend( spec )
subviews spec, which is used to specify types of subviews

subviews : {
   left : FilterView,
   middle : PlayerView,
   right : ClipView
}

initialize : function(){
   this.left = { options };
   this.right = { options };
}

subviews must behave like an attributes
subviews must be inherited from the base view
subviews type spec must support some options (get, set, value, type, event bubbling, might be custom listener specs)
subviews must be attached automatically in 'render' inside tags with 'subview' attribute
render must be template-neutral, expecting function in View.template

Closed in favor of another idea. ViewModel


Following the README.md does not get a working setup

I faithfully followed the guide in the README.md to get a docker setup running.
It fails!
The problem is somewhere in the app/config/parameters.yml file (or possibly a .env file).
When I create a parameters.yml file as per the README.md (and even if I fill out the preloaded file), I get this error message:
application_1  | [11-Apr-2019 07:52:21] WARNING: [pool www] child 6 said into stderr: "[2019-04-11 09:52:21] request.INFO: Matched route "_home". {"route":"_home","route_parameters":{"_controller":"app.catalog.controller:homeAction","_route":"_home"},"request_uri":"http://localhost/","method":"GET"} []"
application_1  | [11-Apr-2019 07:52:21] WARNING: [pool www] child 6 said into stderr: "[2019-04-11 09:52:21] request.CRITICAL: Uncaught PHP Exception Symfony\Component\DependencyInjection\Exception\EnvNotFoundException: "Environment variable not found: "CTP_CLIENT_ID"." at /var/www/symfony/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php line 76 {"exception":"[object] (Symfony\\Component\\DependencyInjection\\Exception\\EnvNotFoundException(code: 0): Environment variable not found: \"CTP_CLIENT_ID\". at /var/www/symfony/vendor/symfony/symfony/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php:76)"} []"

Setting the variables in a .env file (as a Symfony aware coworker suggested) just causes a failure deeper down in the system.
It would be nice to be able to get something up and running - just for a proof of concept - without having to dive into the intricacies of PHP, Symfony etc.

You are right, the provided example for the file app/config/parameters.yml was wrong in the README.md. The structure of the preloaded file is the correct one, ie:
parameters:
    env(SECRET_TOKEN): Secr3tT0ken
    env(CTP_CLIENT_ID): client-id
    env(CTP_CLIENT_SECRET): client-secret
    env(CTP_PROJECT): project-key

The parameters are being cached by symfony, so after you change the parameters.yml file, you will need to fully clear your cache at ./var/cache/ before trying again.

Thank you.  That worked.


chore(trading, governance, explorer): update snap to 1.0.1

Related issues 🔗
N/A
Description ℹ
V1 of snap was just published, update to use it

@mattrussell36 this version doesn't appear to be on the allowlist?

I get this error when I try to install snap from your preview link


Bottle upload jobs failing on test-bot

The jobs on the "Homebrew Bottles" queue have been failing about half the time today. I ran in to some failures yesterday, too.

All of the failed ones that I looked at are failing with complaints about failure to lock refs in one of the Homebrew git repos. For example, http://bot.brew.sh/job/Homebrew Bottles/49221/.
[EnvInject] - Variables injected successfully.
[workspace] $ /bin/sh -xe /var/folders/3m/6yv2qtq95_s88tl3f26y5ymh0000gp/T/hudson3836490006099204134.sh
+ '[' -d .git ']'
+ rm -rf '.git/refs/heads/*.lock'
+ git fetch origin
+ git reset --hard origin/master
HEAD is now at d887dd3 brew install: make -s apply only to given formula, not deps (#205)
+ brew test-bot --ci-upload
cp /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/el_capitan/builds/1709/archive/tokyo-tyrant-1.1.41_1.el_capitan.bottle.rb /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/el_capitan/builds/1709/archive/tokyo-tyrant-1.1.41_1.el_capitan.bottle.tar.gz /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/mavericks/builds/1709/archive/tokyo-tyrant-1.1.41_1.mavericks.bottle.rb /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/mavericks/builds/1709/archive/tokyo-tyrant-1.1.41_1.mavericks.bottle.tar.gz /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/yosemite/builds/1709/archive/tokyo-tyrant-1.1.41_1.yosemite.bottle.rb /Users/brew/Jenkins/jobs/Homebrew PHP Pull Requests/configurations/axis-version/yosemite/builds/1709/archive/tokyo-tyrant-1.1.41_1.yosemite.bottle.tar.gz /Users/brew/Jenkins/jobs/Homebrew Bottles/workspace
git checkout -f master
Already on 'master'
Your branch is ahead of 'origin/master' by 7 commits.
  (use "git push" to publish your local commits)
git reset --hard origin/master
HEAD is now at 2c4a963 phpbrew: update 1.21.1 bottle.
brew update
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at ea1315340e1a836d1ac0a37f07bc03348dfd4899 but expected 2c4a963efdae8dc1acb59b59b898e080aa587561
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at ea1315340e1a836d1ac0a37f07bc03348dfd4899 but expected 2c4a963efdae8dc1acb59b59b898e080aa587561
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at ea1315340e1a836d1ac0a37f07bc03348dfd4899 but expected 2c4a963efdae8dc1acb59b59b898e080aa587561
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at ea1315340e1a836d1ac0a37f07bc03348dfd4899 but expected 2c4a963efdae8dc1acb59b59b898e080aa587561
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at ea1315340e1a836d1ac0a37f07bc03348dfd4899 but expected 2c4a963efdae8dc1acb59b59b898e080aa587561
error: cannot lock ref 'refs/remotes/origin/master': Unable to create '/Users/brew/Jenkins/jobs/Homebrew

Anyone know what's going on here? About half the time, manually retrying the jobs works. About half the time, it fails again on attempts to lock refs.
Is there another Homebrew project that could be running on the master Jenkins node and touching the repo? Is there any way of getting a per-node view of all the activity that has run on that node, instead of viewing it by project build queue?
Full log is in this gist. https://gist.github.com/apjanke/588837bddf570c5ff9031124c52ee38f

Wait a minute. In each of the projects, it still does some Git activity on master. Most of the projects are set up with distinct workspaces, so they don't collide. You can see this by examining the Console Output for a build job on the main node and not one of the configurations (mavericks/yosemite/el capitan).
Here's what the Apache project's jobs look like:
GitHub pull request #117 of commit ab74c60e6c4ee74e8db31427c912be12adbc04dc, no merge conflicts.
Setting status of ab74c60e6c4ee74e8db31427c912be12adbc04dc to PENDING with url http://bot.brew.sh/job/Homebrew%20Apache%20Pull%20Requests/56/ and message: 'Build started sha1 is merged.'
[EnvInject] - Loading node environment variables.
Building on master in workspace /Users/brew/Jenkins/apache
 > git rev-parse --is-inside-work-tree # timeout=10

Here's what Dupes looks like:
GitHub pull request #600 of commit f4a41ab27ef01e44385aedf2da77668d4f559604, no merge conflicts.
Setting status of f4a41ab27ef01e44385aedf2da77668d4f559604 to PENDING with url http://bot.brew.sh/job/Homebrew%20Dupes%20Pull%20Requests/259/ and message: 'Build started sha1 is merged.'
[EnvInject] - Loading node environment variables.
Building on master in workspace /Users/brew/Jenkins/dupes
 > git rev-parse --is-inside-work-tree # timeout=10

Here's what Versions looks like:
GitHub pull request #1252 of commit 2bbb6bf27d6293a01c782c16e4ae9caa358ffc7d, no merge conflicts.
Setting status of 2bbb6bf27d6293a01c782c16e4ae9caa358ffc7d to PENDING with url http://bot.brew.sh/job/Homebrew%20Versions%20Pull%20Requests/1382/ and message: 'Build started sha1 is merged.'
[EnvInject] - Loading node environment variables.
Building on master in workspace /Users/brew/Jenkins/versions
 > git rev-parse --is-inside-work-tree # timeout=10

Here's what Core looks like:
GitHub pull request #914 of commit 76a950a7e0f339ec289dd80a6a36233f6689a9b0, no merge conflicts.
Setting status of 76a950a7e0f339ec289dd80a6a36233f6689a9b0 to PENDING with url http://bot.brew.sh/job/Homebrew%20Core%20Pull%20Requests/1448/ and message: 'Build started sha1 is merged.'
[EnvInject] - Loading node environment variables.
Building on master in workspace /Users/brew/Jenkins/versions
 > git rev-parse --is-inside-work-tree # timeout=10

Wait a minute – "workspace /Users/brew/Jenkins/versions"? Not /Users/brew/Jenkins/core?
Here's a Jenkins build timeline from today.

Some of those Bottles jobs are overlapping other jobs. Is that supposed to happen?
Homebrew Core Pull Requests # 1450 ran from 20:41:43 to 20:54:44
Homebrew Bottles # 49226 ran from 20:45:19 to 20:45:39 (and failed)
Similarly, Homebrew Bottles # 49228 overlapped a Homebrew Versions PR run.
Here's another clue. The error message from one of the failed bottle runs is complaining about an extant lock file in the homebrew/versions tap inside its workspace.
git reset --hard origin/master
HEAD is now at 96bff5b gnupg21: add 2.1.12 bottle.
brew update
error: cannot lock ref 'refs/remotes/origin/master': Unable to create '/Users/brew/Jenkins/jobs/Homebrew Bottles/workspace/Library/Taps/homebrew/homebrew-versions/.git/refs/remotes/origin/master.lock': File exists.

If no other git process is currently running, this probably means a
git process crashed in this repository earlier. Make sure no other git
process is running and remove the file manually to continue.

Most of the others just complain about remote ref SHA mismatches.

git checkout -f master
Already on 'master'
Your branch is ahead of 'origin/master' by 2 commits.
  (use "git push" to publish your local commits)
git reset --hard origin/master
HEAD is now at a6b99a5 civl: remove cvc4 dependency
brew update
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at f08e21a7961085ca4452b778c0b56e8f4982a144 but expected a6b99a561cf09867c7613a6fdc937b53cc72ba65
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at f08e21a7961085ca4452b778c0b56e8f4982a144 but expected a6b99a561cf09867c7613a6fdc937b53cc72ba65
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at f08e21a7961085ca4452b778c0b56e8f4982a144 but expected a6b99a561cf09867c7613a6fdc937b53cc72ba65
error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at f08e21a7961085ca4452b778c0b56e8f4982a144 but expected a6b99a561cf09867c7613a6fdc937b53cc72ba65

This is a brew update operation that seems to be interacting with some job's git operations.
Here's another thing. most of the projects seem to have their own workspaces. But Versions and Core do not. Could their workspaces be aliased to /usr/local, so that their local in-workspace git operations are acting on the same repo that Bottle's brew update is acting on?
robotunicorn:jobs brew$ pwd
/Users/brew/Jenkins/jobs
robotunicorn:jobs brew$ ls -ld */workspace
drwxr-xr-x+  2 brew  staff   68 Nov 27  2014 Homebrew Apache Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 30  2014 Homebrew Binary Pull Requests/workspace
drwxr-xr-x+ 14 brew  staff  476 May  6 21:46 Homebrew Bottles/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 25  2014 Homebrew Completions Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Dec 30  2014 Homebrew Devel-Only Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 23  2014 Homebrew Dupes Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 23  2014 Homebrew Games Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 30  2014 Homebrew Head-Only Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Nov  4  2014 Homebrew Nginx Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 30  2014 Homebrew PHP Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 30  2014 Homebrew Python Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Oct 23  2014 Homebrew Science Pull Requests/workspace
drwxr-xr-x+  2 brew  staff   68 Feb  8  2014 Homebrew Testing/workspace
drwxr-xr-x+  2 brew  staff   68 Nov  6  2014 Homebrew X11 Pull Requests/workspace
robotunicorn:jobs brew$

@apjanke The bottle job should be using the Homebrew just in the Homebrew Bottles/workspace workspace and not touching anything in /usr/local. Looking at the configuration this should be what's happening (but clearly isn't)?

it may be the rebase-is-default-now thing again

error: cannot lock ref 'refs/remotes/origin/master': ref refs/remotes/origin/master is at f08e21a7961085ca4452b778c0b56e8f4982a144 but expected a6b99a561cf09867c7613a6fdc937b53cc72ba65

This is the issue; it's barfing at the different between legacy-homebrew.git and brew.git. I've pulled these into a separate upload job (http://bot.brew.sh/job/Legacy Homebrew Bottles/) which should have fixed the problem.

I don't think this is fixed yet. 😢 I pulled several bottles last night and for (almost) every one of them I had to restart a failed bottle job, sometimes twice, to make it succeed. The failed ones have a multitude of the usual error: cannot lock ref messages …

Ok. I think somehow it's still related. It would be good if it was more than just me working on a fix here 😉

Sorry for lack of PR to go with the bug report. Really not familiar with this area of brew. Have been reading through the code and Jenkins output to figure out how to approach it, but still don't understand it well enough to suggest any concrete fix.
I'm going to log in to the box and manually examine its Homebrew installation to see if anything stands out.
The Jenkins build timeline only seems to be showing half a day's worth of runs. Is there a way to retain more of that so we can get a better view of things?

I added an email notification post-build step to the Homebrew Bottles project configuration, to notify me upon failure, so I can log in and get a fresh look at the state of the Homebrew prefix repos right after a failed run.

It would be good if it was more than just me working on a fix here 😉

Agreed. I tried unsuccessfully yesterday. Looked at the test-bot code and spent a lot of time poking the Jenkins configuration. Even though I found nothing yet, I'm now much more familiar with Jenkins and have a much better understanding of our Jenkins setup.

Sorry for lack of PR to go with the bug report. Really not familiar with this area of brew.

I don't think there's something to be sorry about. I too wouldn't know how to make a PR if I didn't know yet what the underlying issue is.

The Jenkins build timeline only seems to be showing half a day's worth of runs. Is there a way to retain more of that so we can get a better view of things?

I bumped the number of “Homebrew Bottle” builds to retain from 50 to 100. I believe those builds are relatively lightweight, so shouldn't hurt to keep more of them and provides more data for investigation.

Is it clear when this started? Any hope of bisecting the problem rather than thinking about it?

Is it clear when this started? Any hope of bisecting the problem rather than thinking about it?

I think the answer is "no" to both of these, because we don't retain enough build history in Jenkins. This has been going on for a few days, but the oldest job I see in Jenkins is 4970, from just yesterday.
You could get an approximate date of when it started by looking through comments in PRs and seeing when we started having to re-run them more often, but I think it's several days back, based on my recall of when I started seeing a lot of those failed brew pull commands that required reruns.

@apjanke Could it be https://github.com/Homebrew/brew/commit/1c1c48c92005b27aba4e17e12b49c4d1cbb0b331

It's definitely not related to the push. It feels to me that this started happening around the recently brew pull changes but I could be wrong. I pretty strongly suspect that something is using the /usr/local repository rather than the one in the working directory.

Was anything changed on Jenkins? There's currently a run of 20 bottle jobs that didn't have the issue.

@apjanke Could it be 1c1c48c92005b27aba4e17e12b49c4d1cbb0b331

Seems unlikely given that the problem is with the local Git repository (the one on our CI) and not with the remote and the change shouldn't have an influence on the former. (At least that's what I thought.)
But upon closer inspection, those pushes are also messing with the master tag and the Git errors are about refs/remotes/origin/master. Additionally, the state of master might be different between the first (now removed) and the second git push. I'd say reverting this change and checking whether that fixes the issue wouldn't be a horrible thing. The problem that is being addressed in the commit seems a lot less pressing than these constant Git-related bottle job failures.

Was anything changed on Jenkins?

Not that I'm aware of. I added a notify-on-failure alert with myself in the recipients list, but didn't make any other changes and don't know of any.

The last change I made was the legacy bottles separation. It could be that has fixed it now the backlog was worked through.

The last change I made was the legacy bottles separation. It could be that has fixed it now the backlog was worked through.

It was worth a try, but I don't think this had any effect. 😢 It has been in place for like two days (?) and there have been failures since, even with PRs that have been opened less than a day ago.

error: cannot lock ref 'refs/remotes/origin/master': Unable to create '/Users/brew/Jenkins/jobs/Homebrew Bottles/workspace/Library/Taps/homebrew/homebrew-core/.git/refs/remotes/origin/master.lock': File exists.

Added:
rm -rf .git/refs/heads/*.lock Library/Taps/*/*/.git/refs/remotes/origin/*.lock

Replaced the git fetch with a brew update now that the test-bot redoing it is a quick no-op and so it cleans up every tap rather than just one. Once we have https://github.com/Homebrew/brew/pull/166 we shouldn't need to tap at all.

Seems fixed now.

I got an isolated spate of failure notifications yesterday, but nothing else recently. Does seem fixed.

It'll still fail sometimes due to e.g. upload attempts after publishing but it looks generally right now 👍


K8SPG-360: trigger reconcile on secrets update

https://jira.percona.com/browse/K8SPG-360
DESCRIPTION
Problem:
When cluster is ready, patching secrets doesn't affect cluster at all. Even if secret is patched with an empty password, it's not regenerated
Cause:
Reconcile is not triggered on secrets update.
Solution:
Trigger reconcile when .data of the secret is changed.
CHECKLIST
Jira

[x] Is the Jira ticket created and referenced properly?
[x] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[x] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

init-deploy
passed

demand-backup
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 6 out of 6

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/ea902f0c701e2573bde14b210757af5ffb76bff9
image: perconalab/percona-postgresql-operator:PR-443-ea902f0c7

Test name
Status

init-deploy
passed

demand-backup
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 6 out of 6

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/7bff213714d9fc2c794d2e36a48a42cc70323678
image: perconalab/percona-postgresql-operator:PR-443-7bff21371

Test name
Status

init-deploy
passed

demand-backup
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 6 out of 7

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/b361255b2f0d32dde928ef61b49bd1c5c2d23b86
image: perconalab/percona-postgresql-operator:PR-443-b361255b2

Test name
Status

init-deploy
passed

demand-backup
passed

scaling
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 7 out of 8

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/344625a58e7e6a60a39d130cbaead4f6746136ea
image: perconalab/percona-postgresql-operator:PR-443-344625a58

Test name
Status

init-deploy
passed

demand-backup
passed

scaling
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 7 out of 8

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/80ef565e42768155a7348efce9122e6e8747ed22
image: perconalab/percona-postgresql-operator:PR-443-80ef565e4

Test name
Status

init-deploy
passed

demand-backup
passed

scaling
passed

scheduled-backup
passed

start-from-backup
passed

monitoring
passed

telemetry-transfer
passed

We run 7 out of 8

commit: https://github.com/percona/percona-postgresql-operator/pull/443/commits/d5a1f491ca6312da10b8a9059ef8dd4217fc979b
image: perconalab/percona-postgresql-operator:PR-443-d5a1f491c


Unable to use UIServer

Issue Description
What I want to express is that UIServer cannot be use because its package name for deeplearning4j-play_2.11 is the same as that for deeplearning4j-ui-components. By default, it refers to deeplearning4j-ui-components. So we can't use org.deeplearning4j.ui.api.UIServe.
Version Information
1.0.0-beta4
win7 64
idea and eclipse
apache-maven-3.3.9
Additional Information
demo  https://github.com/awaymeet/lottery
I don't think it's a maven problem between idea and eclipse. I rely on the packages you publish to the maven repository. But you see
“org.deeplearning4j.ui.api.Component c = null;” can be used
“org.deeplearning4j.ui.api.UIServer server=null;” cannot be used

https://files.gitter.im/deeplearning4j/deeplearning4j/2W1g/3.jpg
https://files.gitter.im/deeplearning4j/deeplearning4j/eNLI/1.jpg
https://files.gitter.im/deeplearning4j/deeplearning4j/8xdp/2.jpg

Builds and runs fine for me in IntelliJ 2019.1 (Windows):

Also builds fine for me directly in Maven (3.3.9):
https://gist.github.com/AlexDBlack/a6821a83ee78898afd19fed00e3586a8

Only other idea I have is maybe Java version - what are you running?
Beyond that - maybe try deleting deeplearning4j-play_2.11 in your maven repository to force a re-download (perhaps it was corrupted).
That's in C:\Users\<your_user_name>\.m2\repository\org\deeplearning4j on windows.

The reason was found out because the dependencies in the company's internal repository had been updated. Personal computers rely on corporate internal mirroring.
When we delete it and update it, that's all.
Thank you. Thank you very much. The problem haunted us for two and a half days.

OK, great. I'll close this as resolved then.


How do I snap my HMD to the left or right? (please help)

If you are reading the title and misunderstood my question, let me explain.
I have 2 PsEye Cameras that are on my desk facing me. If I turn around with my HMD in VR my Ps Move controllers go out of view they spazz out. I was wondering if I could snap my HMD to the left or right by pressing a button on my Ps Move Controller without having to physically turn around. What I mean by "snapping" is to virtually move the HMD either left or right without physically moving it.
You're probably wondering... "Why don't you get 2 more Ps Eye cameras? " I can't because I don't have space for 2 more Ps Eye Cameras.
Is it possible to snap my HMD to the left or right by pressing a button on my Ps Move controller?
Looking forward to an answer,
-N.B

get Driver4VR it has that function to virtually rotate, I use it in my set up

Thank you! I will definitely try Driver4 VR out!


Update Contributors

Integrate contributors bot

@all-contributors please add @PacoVK for code

https://github.com/all-contributors please add @PacoVK for review, projectManagement, maintenance, example

https://github.com/all-contributors please add @klutzer  for code
https://github.com/all-contributors please add @stasadev for code
https://github.com/all-contributors please add @AlejandroPerez92 for code


Make header position configurable

Show some love to us folk who prefer the headers to be at the top.

Perhaps you will find this screenshot useful, the top is when the value of 'header_position' is set to 'top', and the bottom is when it is set to bottom (which is the default).

Done

Conflicts

I believe that she be solved now

Reverted. You have broken the build.


Update Black version and links

Pull Request Test Coverage Report for Build 118

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 98.911%

Totals

Change from base Build 114:
-0.05%

Covered Lines:
3361

Relevant Lines:
3398

💛  - Coveralls

Thanks for this - addressed in<PHONE_NUMBER>f0027b8eb5e066e78e58da0e2489b2.


Feature/move feature flags UI

Purpose

Move the Feature Flags from the main screen to an AlertDialog, triggered by a settings button in the actionbar.
This is to prevent the "Launch" button from scrolling off the screen from tests as we add more features.
Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

Test the settings button

What to Check
Verify that the following are valid

...

Other Information

license/cla