ToC directs to first header with a given name

We recently updated to dev to get the static ToC. The ToC now breaks when attempting to navigate to a header that has the same name as another. You can see the problem here: http://slate-update.identity.cf-app.com/
Example, in the ToC click Users -> Get. The browser then navigates to Service Providers -> Get
Previous version disambiguated these IDs by adding a number at the end. We tried altering the layout.erb to disambiguate them, but this results in the JS looking for elements with nonexistent IDs. We don't know if we can change the IDs generated by redcarpet.

Nice catch, thanks. What do you mean by "altering layout.erb"?

Previously we didn't want to implement a solution since we thought RedCarpet was working on a fix for everyone, but it looks like they now suggest implementing a custom ID generator (https://github.com/vmg/redcarpet/issues/307), so maybe it's time to reconsider making IDs ourself.

The way that I solved this problem was to create a new Redcarpet rendered in config.rb that altered the way that headers were output.  In particular:
$headcount = 0 def header(text, header_level)     $headCount = + 1     return "<h#{header_level} id='#{text.parameterize}-#{$headCount.to_s}'>#{text}</h#{header_level}>"   end
Thus what this code does is change how headers are output in HTML using two different ID's (one in the header markdown and the other in the span.  Thus, the old # anchors work but for the static table of contents, you key off unique ones that are created from the DOM.
Paul

@lord We found the following section that builds ToC items:
https://github.com/lord/slate/blob/7b1e718672b57f88ebf1df9ce3925205064658a3/source/layouts/layout.erb#L75-L84
We altered href="#<%= h2[:id] %>" to concatenate the h1 id as well, something like this href="#<%= "#{h1[:id]}--#{h2[:id]}" %>"
The goal was for items to have IDs like users--delete and service-providers--delete

This is actually the same issue as #732

@bc-AlyssNoland No, this approach was no good. _toc.js is looking for headers with those ids, and they do not exist. Instead, the ids have to be unique to begin with.
Instead we did something like what pnerger suggested above.

@plfx @pnerger I'm not getting the intended header output. Can y'all check me on this?
# Markdown
set :markdown_engine, :redcarpet
set :markdown,
    fenced_code_blocks: true,
    smartypants: true,
    disable_indented_code_blocks: true,
    prettify: true,
    tables: true,
    with_toc_data: true,
    no_intra_emphasis: true

# Sitemap
page "/sitemap.xml", :layout => false

# Assets
set :css_dir, 'stylesheets'
set :js_dir, 'javascripts'
set :images_dir, 'images'
set :fonts_dir, 'fonts'

# Activate the syntax highlighter
activate :syntax

activate :autoprefixer do |config|
  config.browsers = ['last 2 version', 'Firefox ESR']
  config.cascade  = false
  config.inline   = true
end

# Github pages require relative links
activate :relative_assets
set :relative_links, true

# Build Configuration
configure :build do
  # If you're having trouble with Middleman hanging, commenting
  # out the following two lines has been known to help
  activate :minify_css
  activate :minify_javascript
  # activate :relative_assets
  # activate :asset_hash
  # activate :gzip
end

$headcount = 0
def header(text, header_level)
  $headCount = + 1
  return "<h#{header_level} id='#{text.parameterize}-#{$headCount.to_s}'>#{text}</h#{header_level}>"
end

helpers do
  require './lib/toc_data.rb'
end

Sorry, there of other lines which I've moved to the top.  You need to require the redcarpet render.  Next you create a new class "myRenderer" based upon the the MiddlemanRedcarpetHTML renderer.  Then it includes the new function for rendering headers.  The last step is to tell Middleman to use the new renderer "myRenderer".
That should work.
Paul
require "middleman-core/renderers/redcarpet"
$headcount = 0
class myRenderer < Middleman::Renderers::MiddlemanRedcarpetHTML
  def header(text, header_level)
    $headCount = + 1
    return "<h#{header_level} id='#{text.parameterize}-#{$headCount.to_s}'>#{text}</h#{header_level}>"
  end
end

helpers do
  require './lib/toc_data.rb'
end

# Markdown
set :markdown_engine, :redcarpet
set :markdown,
    fenced_code_blocks: true,
    smartypants: true,
    disable_indented_code_blocks: true,
    prettify: true,
    tables: true,
    with_toc_data: true,
    no_intra_emphasis: true,
    renderer: myRenderer

# Sitemap
page "/sitemap.xml", :layout => false

# Assets
set :css_dir, 'stylesheets'
set :js_dir, 'javascripts'
set :images_dir, 'images'
set :fonts_dir, 'fonts'

# Activate the syntax highlighter
activate :syntax

activate :autoprefixer do |config|
  config.browsers = ['last 2 version', 'Firefox ESR']
  config.cascade  = false
  config.inline   = true
end

# Github pages require relative links
activate :relative_assets
set :relative_links, true

# Build Configuration
configure :build do
  # If you're having trouble with Middleman hanging, commenting
  # out the following two lines has been known to help
  activate :minify_css
  activate :minify_javascript
  # activate :relative_assets
  # activate :asset_hash
  # activate :gzip
end

I had some name errors come up but everything seems to be working once I made a couple of revisions. I'm still having trouble with using a depth of 3 headers. I'll play around with it and post my findings.
require "middleman-core/renderers/redcarpet"
$headcount = 0
class MyRenderer < Middleman::Renderers::MiddlemanRedcarpetHTML
  def header(text, header_level)
    $headCount = + 1
    return "<h#{header_level} id='#{text.parameterize}-#{$headCount.to_s}'>#{text}</h#{header_level}>"
  end
end

...

# Markdown
set :markdown_engine, :redcarpet
set :markdown,
    fenced_code_blocks: true,
    smartypants: true,
    disable_indented_code_blocks: true,
    prettify: true,
    tables: true,
    with_toc_data: true,
    no_intra_emphasis: true,
    renderer: MyRenderer

We could not get a satisfactory result specifying a custom renderer. No matter what, our counter is unable to keep track and resets for every header. This seems to be true no matter what the scope of the counter variable is. We made a version that appends random numbers, which works (with very high probability...), but now we've decided to sidestep the issue temporarily by replacing all our Markdown headers with HTML, i.e.
## Something ->
Something

@bc-AlyssNoland We are also experiencing failure with headers beyond h2. Our h3 items don't show up in the list at all, regardless of whether we are using a custom renderer or whether we are using HTML or markdown. @lord Is that problem covered under #732 or should we open another issue?

@plfx — maybe I'm misunderstanding, but that should be the expected behavior? If you want headers beyond h2 to appear in the ToC, @bc-AlyssNoland wrote up a super helpful guide here: https://github.com/lord/slate/wiki/Deeper-Nesting#static-table-of-contents

@plfx - Not sure what you mean by resets for each Header.  It should work for it is working perfectly for us.  Are you able to provide a snippet of the HTML output with say three of the headers such that I can look at it to see if the custom renderer is working?
Paul

@pnerger
My current (sample) html output is
Catalog

    getProducts

        Parameters

        Responses

    createProduct

        Parameters

        Responses

And can I see the corresponding header HTML that are the targets with the custom render?

@pnerger The custom renderer we added is certainly working, because as I said we can have it generate random numbers to disambiguate every header id and this works just fine. It just isn't ideal. If we attempt what should work, i.e. having a header count that increases monotonically, every header receives a 1. Adding puts statements to view the object IDs of the renderer reveals it is being instantiated repeatedly.
@lord Thank you for the link about ToC depth. We made the three changes described to no avail. Still no deeper sections. Does this approach still work as of current dev?

@plfx it was written based on dev. If y'all have a public repo, I'm happy to check it to see what might be going on.

@pnerger the corresponding header html for the side nav listed above:

Catalog

BigCommerce v3 API Documentation

getProducts

GET /catalog/products

Returns a paginated collection of Products objects from the BigCommerce Catalog.

Parameters

...

Enumerated Values

...

Responses

Status
Meaning
Description

200
OK
An array of products and metadata.

createProduct

POST /catalog/products

Creates a Product in the BigCommerce Catalog.

Parameters

@plfx - the custom renderer is working but not correctly.  It is not incrementing the counter (it is always fixed at 1) and thus the increment of $headCount = + 1 is not working.  Looks like there is an extra space.  May have been my mistake.  It should be $headCount += 1 which will create a unique number which is then added to the header ID for each header.
That should fix it.
Paul

@plfx you mentioned in the pull request that you were having an issue with the span being malformed html.  Can you elaborate on that?  Was it with a particular browser?  We've seen no problems with that HTML but it would be good to know what you are observing.  (We use the Span ID for anchor targets for use by authors since the author will not know the unique ID in the header and because HTML does not allow multiple IDs).

@plfx Thanks.  I looked at your output and the difference between my fork and this one is that I statically build the ToC using a helper function rather than dynamically build it using a JavaScript ToC plugin.  I did that to get faster load times on really long documents.  If you wanted the  to work, I think that you could output it just before the header and then the JavaScript will not pick up on it.
If you want to test this, change the return on the new renderer to be:
return "<h#{header_level} id='#{text.parameterize}-#{$headCount.to_s}'>#{text}</h#{header_level}>"
Notice all that I did was pull the span out of the header tag.  The span is blank and its only purpose is to provide an # id as a target for links.
Sorry for the confusion but I forked a long time ago and have made a lot of changes that would be a nightmare to merge back in.
Paul

@pnerger Sorry, I haven't made any pull requests, and I haven't had any problem with the spans.

Should be mostly fixed in dev with just a few bugs re: include files, so closing.

I'm having this issue even while running latest version from master. Am I missing something obvious?

Shouldn't this be in the documentation somewhere? or be the default behavior? seems a lot more flexible that the current default behavior which everyone encounters at one point (unless you are lucky and never use twice the same name in headers)

It's tricky — I'd rather not have people upgrade and break all their links. Thoughts?

@lord maybe at least update https://github.com/lord/slate/wiki/Deeper-Nesting ?


[SPARK-16772] [Python] [Docs] Fix API doc references to UDFRegistration + Update "important classes"

Proposed Changes

Update the list of "important classes" in pyspark.sql to match 2.0.
Fix references to UDFRegistration so that the class shows up in the docs. It currently doesn't.
Remove some unnecessary whitespace in the Python RST doc files.

I reused the existing JIRA I created last week for similar API doc fixes.
How was this patch tested?

I ran lint-python successfully.
I ran make clean build on the Python docs and confirmed the results are as expected locally in my browser.

Test build #63231 has started for PR 14496 at commit 62f4f82.

Test build #63231 has finished for PR 14496 at commit 62f4f82.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63231/
Test PASSed.

cc @rxin - Follow-on to #14393.

Merged to master/2.0

Thanks @srowen. 👍


Ionic 2 and NgModule

Hi @jgw96, @brandyscarney, @danbucholtz,
I have a question about Ionic 2 in combination with Angular 2 modules. Now, when Ionic 2 uses Angular 2 final, it's theoretically possible to build modular application architecture.
My question is, will modular application architecture work in Ionic 2 in the same way it works in Angular 2 and is there something else in addition to keep in mind? Some Ionic specific issues, limitations, etc.?
Angular has updated style guide which includes modules and recommendations how to use them (app root module, core module, shared module, feature modules etc.). There are also some considerations to keep in mind, like where to put shared services between modules and where to put shared components/directives/pipes, so they work properly with lazy loaded modules and dependency injection.
Is this all the same for Ionic 2 project, with the same things to keep in mind?
Thanks in advance for your time!

Hi @pdrosos,
Right now our recommendation is to stick with one NgModule unless there is a good reason not to.  Lazy loading is something we're considering and it may be something that happens down the road.
For now, your best bet is probably going to be keeping it simple.  At least keep it simple to start, get something running, then figuring out optimizations later in my opinion.
I'm going to close this, but I will keep my eye on it and answer any questions. If you want it re-opened, we can do that too.
Thanks,
Dan

I tried to refactoring my app following  angular 2 style guides using core/shared/features modules imported in app.module....
no luck!!

I will try refactoring a simple Ionic app with Angular modules during the weekend and will post here how it goes.
Except the Angular styleguide, these two pages from the Angular documentation are a must read:
https://angular.io/docs/ts/latest/guide/ngmodule.html
https://angular.io/docs/ts/latest/cookbook/ngmodule-faq.html
@danbucholtz I see here, that soon Ionic will support the Angular router: https://forum.ionicframework.com/t/new-angular2-router-support-in-ionic2/56847/14
This is great news for developing progressive web applications. Will it work with lazy loading too, how the angular router can work? If yes, then Angular modules must be used too.
I also wonder, if а mobile application only is developed, is it still recommended to use the Ionic NavController instead of the routing?

Is it still recommended using single ngModule in ionic app? I can't make it work with custom modules.. Any info would be great about this problem.

@Substractive, for now, yes. This may change in the next few weeks as I am beginning to investigate code splitting.
Thanks,
Dan

I managed to create custom module and export it in app module .. Managed to "route" to page from custom module component that I exported . Haven't tested anything else but routing and building the app. I will be building simple app in near future and will try to experiment with modules more into depth. Anyone tried experimenting with modules and has some info about it?
And yeah Merry Christmas :)

I have a root app module, feature modules, core module and shared module, as recommended in the Angular styleguilde. Everything works well so far and I don't have issues. I use the Ionic navigation to navigate trough pages.

Yeah so am I using ionic navigation.. and yeah I have app module and custom module for "login" which then I assume in same analogy can have it's module and then in app module it should be accessible through the login right? Could you send snippet that I can compare approaches? I will put mine on git in week or 2 :)

@Substractive for which part exactly do you need an example, in some module navigating to page from other module or something else?
If yes, it's absolutely the same as if they are in the same module, import the page you need and use this.navController.push(OtherModulePage);

@pdrosos example of your structure how did you share modules and services if possible, I will start snooping around for best practice structure and encapsulating app code. I feel that this conversation is not so much related to this issue so I won't ask too many questions from now on about examples..
After all I also managed to import module into app module and navigate to that page from my custom module so far I haven't encountered any problems doing that..

@Substractive my structure follows the Angular styleguide - I have App root module, Core module, Shared module and feature modules. In Core module are common services. In Shared module are common components, directives and pipes. You can read here and here about details, recommendations and best practices. Exact modules depend on the project. Hope this helps.

I can achieve this, but In core/shared/feature modules I have to import IonicModule (AppModule still imports IonicModule.forRoot). Is it the right way? Couldn't find any example or docs that shows importing just IonicModule. Any negative effects?
And can we use Angular Router with Ionic now? Couldn't find anything from docs.

@shawnlan,
You can use the Angular Router but there isn't a lot of benefit right now. You'd probably have to create your own NgModule instance from the Ionic data.
Ionic v3 will support lazy loading via the DeepLinking system. So you'll have proper URLs as a first class citizen, and then be able to use NgModules all over the place for child/shared modules.
All without any significant breaking changes 🎉
Thanks,
Dan

@danbucholtz,
Multiple modules are recommended by Angular Style Guide. Is that also the direction Ionic is going in future releases (instead of single module approach)? If that's the case, I would probably like to structure my app into multiple modules now so that I don't have to restructure later.
Right now is it a good idea to import IonicModule in my feature modules? (without it I can't use Ionic components from other modules)
In app.module.ts, I have:

imports: [
IonicModule.forRoot(MyApp)
]

In other feature modules, I have just:

imports: [
IonicModule
]

Thanks!

You can't import multiple IonicModules right now because we import BrowserModule. It will break things. We are changing this in Ionic 3 in a few weeks so you'll be able to have feature modules.
Thanks,
Dan

@danbucholtz Interesting. I did it but nothing breaks. Works fine still.

Ahh, I guess you aren't using the forRoot method, that could have something to do with it. I'm not sure. Are you able to use Ionic directives, etc in your templates?
Thanks,
Dan

No, I use forRoot only in app.module.ts. In other modules I import just IonicModule. Then I can use Ionic components in other modules just fine.

waiting for Ionic 3 to optimize structure.

@danbucholtz I wonder why lazy loading doesn't work with Angular Router + Ionic at this moment. I tested latest Angular CLI. Lazy loading works fine. Webpack generates chunk files if code contains lazy loading syntax (loadChildren).
Since Ionic uses Webpack as well, I don't see a reason why lazy loading doesn't work with Angular Router + Ionic.
Does it have something to do with Ionic CLI?

I don't think there is any technical reason you couldn't use the Angular router. If you use the IonicModule.forRoot method, you'll have an exception because of multiple BrowserModules being imported.
Thanks,
Dan

Hello, everyone!
Just got a task at my job that can be solved using modularity, but as I understood, right now it isn't fully supported and/or documented. Am I right? If not - is there any example app to see how it can be done? And what is the best path - to start creating modules right now or to wait for the Ionic 3?
Thanks,
Dmytro

@hitpopdimestop You can import IonicModule.forRoot in the root module, and import IonicModule in other modules (except lazy loaded ones). You can then structure your code following Angular Style Guide.
Ionic V3 will have better support for modularity. You might need to change a few things then, but overall structures should stay the same.
This is not officially documented, but it works for me. Do it at your own risk.

@shawnlan you could use Ionic Components inside a Lazy Module using this approach you showed?

At my tests:

if you don't put IonicModule in Lazy Loaded Modules, you can't use Ionic Components
and if you put IonicModule in Lazy Loaded Modules, it throw "BrowserModule has already been loaded".

No, you have to wait for Ionic V3 for a lazy module, unless your lazy module doesn't use any Ionic components (no need to import IonicModule).

@danbucholtz  So now that Ionic 3 has been released... what's the official way to achieve this? How do I use Ionic directives in templates from a feature module?


Telemetry for operational monitoring

Background
Describe what this story is for and the value the customer will gain from it.
In order to ensure that operations teams can easily monitor the behaviour of an EHRbase deployment it would be useful for EHRbase to provide liveness/readiness endpoints and a metrics endpoint that provides RED metrics that can be easily scraped by a tool such as Prometheus.
In this way should EHRbase availability, responsiveness or error rates begin to report out-of-normal values then the operations team can be alerted quickly and begin investigation.
Acceptance criteria
These are the functional requirements for this story. Add all steps to be implemented to fulfill the requirements.

[ ] The EHRbase service exposes telemetry on a configurable port which may be separate to the port the REST API is exposed on
[ ] A /liveness endpoint is available on the telemetry port that shows the EHRbase service is running
[ ] A /readiness endpoint is available on the telemetry port that shows the EHRbase service is ready to accept traffic
[ ] A /metrics endpoint is is available on the telemetry port that provides metrics in the Prometheus exposition format.
[ ] The metrics exposed contain at least the RED metrics for Request rate, Error rate, request Duration.
[ ] The Error rates are also provided grouped by status code.
[ ] Other useful metrics would be memory consumption and database connection pool size.

Success criteria

[ ] Review / Merge request approved (P2P session)
[ ] Unit tests passed
[ ] Updated documentation (Javadoc and Sphinx)
[ ] Acceptance criteria fulfilled
[ ] Build without errors
[ ] Release notes prepared
[ ] Runtime warnings

I believe that most (or all) of these requirements could be satisfied by including the Maven dependency spring-boot-starter-actuator (documentation) module into EHRbase.  The above features requested can then be activated & configured through Java system properties. 🤞

Adding the Spring Boot Actuator should be easy, while adding many features. So I'm in favor of doing that, also because we had that on our plate some time ago already. :+1:


Create moodman.toml

new net_address <IP_ADDRESS>
my net_address has changed and is now different from the previous testnets, it used to be "<IP_ADDRESS>:26656" and now it is "<IP_ADDRESS>:26656"
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-3/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-4/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-5/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-6/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-7/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-8/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-9/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-10/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-11/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-12/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-13/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-14/moodman.toml
https://github.com/anoma/namada-testnets/blob/main/namada-public-testnet-15_legacy/moodman.toml

Successfully validated transactions.toml file for commit hash 40bda9a


Some automation for creating templates when building Android (armeabi) on Linux

Should be easy enough to extend to other platforms, but I have no way to
test others currently and am uncertain of the template location convention for
platforms other than posix/Linux

Moves the android lib built (following the library naming expected by android
libgodot_android.so) to the LIB_INSTALL_PATH:
platform/android/java/libs/armeabi/.

Will need to some modifications to work
with other architectures

creates the APK template symlink in ~/.godot/templates/

I'd prefer not to add such automation to the buildsystem, as it would be useful maybe for users but not when building e.g. official binaries. Or sometimes one wants to build the apk without overriding the templates installed in the $HOME.
Also this would not work anymore as the buildsystem now needs gradle and not ant.

But it would be a litmus test for when a pull request broke the unit
tests...
On Tue, Mar 15, 2016 at 3:12 AM, Rémi Verschelde<EMAIL_ADDRESS>wrote:

I'd prefer not to add such automation to the buildsystem, as it would be
useful maybe for users but not when building e.g. official binaries. Or
sometimes one wants to build the apk without overriding the templates
installed in the $HOME.
Also this would not work anymore as the buildsystem now needs gradle and
not ant.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/godotengine/godot/pull/1628#issuecomment-196753456

Ops wrong comment.  This was super old.  It is no longer needed now.  Junk it. :)


making laravel-permission modular [help-wanted]

help wanted with laravel-permission library inside modules here is query in detail Read Here

I think you should just figure out a good way to develop this. It isn't something either package needs to do.


merge: dev to stage

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


[RKE v1.4] Deprecate Weave

This is a child issue of https://github.com/rancher/rancher/issues/44098.
Backport of https://github.com/rancher/rke/issues/3338
TODO:

[x] backport fix in RKE
[x] cut RKE RC - covered by RKE v1.4.15-rc1
[x] bring RKE RC into Rancher - covered by https://github.com/rancher/rancher/pull/44146
[x] backport Rancher UI (by the UI team) (https://github.com/rancher/dashboard/issues/10291)

The UI part is tracked here https://github.com/rancher/dashboard/issues/10291

The fix can be validated in the latest v2.7.11-head.

Waiting for the docker images to become available so we can move this to QA.

can be validated in the latest v2.7.11-head image

Ticket #44193 - Test Results - ✅

Scenario
Test Case
Result

1.
Verify that the depreciation message is displayed on cluster creation
PASS

2.
Verify that the depreciation message is displayed on cluster upgrade
PASS

Verified with (HA Helm or Docker) on Rancher v2.7-head:
Scenario 1: Verify that the depreciation message is displayed on cluster creation

Create a rke1 cluster with Kubernetes 1.27.10
Verify that weave show deprecation message when selected

Results: The deprecation message is properly displayed

Scenario 2: Verify that the depreciation message is displayed on cluster upgrade

Create a rke1 cluster with Kubernetes 1.26 and weave
Upgrade cluster to 1.27.10
Verify that the deprecation message is displayed

Results: The deprecation message is properly displayed


Deploy to vercel

Ok, we have some issue to fix before deploying this to anywhere.
First at all, this repo is using our Component Library to work. But the library is not published anywhere yet. For now, we're importing it locally (also, to always have the latest version quickly), and that's not compatible with serverless deployments.
So, the next steps to make this happen are:

[ ] Publish a first version of the library (using npm)
[ ] Change the way we import the library, so we use the NPM version instead of using it locally

And, from then, we can keep researching how to get a quick deployment (for testing, previews...)


Write jdeps log output to buildDir/reports/jdeps/jdeps-report.txt

Allow also configuration of console output.
Fixes gh-1.

thank you! 😄


Adds new badge for [Redmine] plugin rating

Redmine is popular project management web application. It has plugin system for improvement.
Many plugins are listed on Plugins Directory in redmine.org.
And Plugins Directory has rating system. This request for it.

Hi, thanks for this!

Could you take a look at the build failures?

@paulmelnikow thank you for reviewing quickly😃
I fixed your commented problems.

I fixed API uri for consistency with other plugin badges.
Could you review it again?

Thank you!


Implement Span Buffer

Use a  Span Buffer to collect spans before sending them off collectively.
The size of these batches is either set by the span_buffer_threshold or the DD_TRACE_PARTIAL_FLUSH_MIN_SPANS (as used by PHP), and defaults to 100.

CAVEAT
The $span->is_root_span might not do what we think it will do.
I'm afraid that once we are capable of doing a correct extract_context_from_http_headers and would use that in for example CGI::Application::Plugin::OpenTracing, that we will have a proper SpanContext, which will have a parent_span_id, carried over thru the incoming HTTP request.


How to preserve order while serializing?

I can't seem to get this working and I have no idea why. From my googling and browsing this repo, it should be possible.
Anytime I serialize it orders fields based on the order of fields in the struct, and not in the order of fields in the file before deserializing.
I have a PR here to try and fix it: https://github.com/milesj/moon/pull/45
And this is the struct: https://github.com/milesj/moon/blob/fix-json/crates/config/src/package.rs#L12
Is my understanding of preserve_order wrong?

Wrote a test that verifies this: https://github.com/milesj/moon/pull/45/files#diff-f4e3009eb125d0d4de2eb6fa3f5c4e90ffdb424fb9f0d8dc99d5d04a23313e35R371
---- package::test::preserves_order_when_de_to_ser stdout ----
thread 'package::test::preserves_order_when_de_to_ser' panicked at 'assertion failed: `(left == right)`
  left: `"{\n  \"description\": \"world\",\n  \"name\": \"hello\",\n  \"private\": true\n}"`,
 right: `"{\"name\": \"hello\", \"description\": \"world\", \"private\": true}"`', crates/config/src/package.rs:381:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I think this is working as intended: preserve_order only affects the representation of serde_json::Map (and thus serde_json::Value). It makes that data structure store additional information about the entry order, in addition to the keys/values themselves. If you are deserializing to a struct that is defined in your own program, that struct isn't gonna have a place to put such order information.

For anyone coming to this issue, this is my workaround. https://github.com/milesj/moon/pull/68


FIX: Address non-empty take from empty axes (anatomical IQMs)

This patch addresses this problem encountered in some occassions, where the mask on which summary stats are to be calculated is too small.
Fixes: #1048.
Fixes: #868.

@demidenm could you test this branch?

@oesteban - I couldn't easily test the branch but I pulled the anatomical.py so I used the following to test it:
...
sif_img=mriqc_22.0.6.sif
anatpy_new=anatomical.py
anatpy_old=/opt/conda/lib/python3.9/site-packages/mriqc/qc/anatomical.py
...
singularity run --cleanenv \
-B ${data_dir}/bids_dir/sub-${subj_id}_ses-${ses_id}:/bids_dir \
-B ${data_dir}/processed/mriqc_v22_0_6/sub-${subj_id}_ses-${ses_id}:/output_dir \
-B ${data_dir}/work_dir/mriqc_v22_0_6/sub-${subj_id}_ses-${ses_id}:/wd \
-B ${anatpy_new}:${anatpy_old} \
${sif_img} \
...

I got the following error:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/nipype/interfaces/base/core.py", line 398, in run
    runtime = self._run_interface(runtime)
  File "/opt/conda/lib/python3.9/site-packages/mriqc/interfaces/functional.py", line 119, in _run_interface
    stats = summary_stats(epidata, mskdata, erode=True)
  File "/opt/conda/lib/python3.9/site-packages/mriqc/qc/anatomical.py", line 607, in summary_stats
    nvox = mask.sum()
**AttributeError: 'tuple' object has no attribute 'sum'**

@demidenm thanks for your patience -- I made a mistake, which I hope I fixed in the latest commit.

@oesteban - thanks for the help and follow up. It looks like that error was resolved but I am getting the following issue now. Not sure whether this is because I am using mriqc v22.0.6 and there is a newer tag v23.0.0?
Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/mriqc/engine/plugin.py", line 60, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 524, in run
    result = self._run_interface(execute=True)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 642, in _run_interface
    return self._run_command(execute)
  File "/opt/conda/lib/python3.9/site-packages/nipype/pipeline/engine/nodes.py", line 750, in _run_command
    raise NodeExecutionError(
nipype.pipeline.engine.nodes.NodeExecutionError: Exception raised while executing Node ComputeQI2.

Traceback (most recent call last):
  File "/opt/conda/lib/python3.9/site-packages/nipype/interfaces/base/core.py", line 398, in run
    runtime = self._run_interface(runtime)
  File "/opt/conda/lib/python3.9/site-packages/mriqc/interfaces/anatomical.py", line 386, in _run_interface
    qi2, out_file = art_qi2(imdata, airdata)
  File "/opt/conda/lib/python3.9/site-packages/mriqc/qc/anatomical.py", line 462, in art_qi2
    from nireports.reportlets.nuisance import plot_qi2
ModuleNotFoundError: No module named 'nireports'


Allow for QR Code scanning from broadcast field.  Resolves #141

Added option to scan broadcast QR Code.

Fixes #141


refactor: Move integration search flag to categories

While adding the encodeURI function to the query (Spaces were not working) I noticed the implementation was not using the category system, after discussing it with Mani it was asserted that it was only for personal preference/meta reasons.
I am suggesting this PR as a change to keep the system centralized using the generalized category system instead.
(This also implements the encodeURI fix, which I can make a new PR for if this one doesn't get accepted).

As discussed via Discord with Sen, I vote to keep the current system. For me, it makes more sense on a meta-level

For me on the meta level, integration searching being a function of integrations, it makes more sense to use it exactly the same way we would any other integrations.

Seems that ajnart isn't going to respond soon. I took some time to think about this change again and I think I agree with it. I will take over, resolve conflicts and merge, because the majority now voted for this change.


some typos in d3 tutorial

hi @cypranowska , thanks for this awesome tutorial, I bummed I missed it.
I found two tiny typos in the tutorial as I was trying to follow it, both in this section:

missing closing quotation mark for "height"
extra unnecessary curlycue brace in cx function

thanks!

@cypranowska you can just merge #126 if you think the typos are fixed


Create an SAP Cloud Platform ABAP Environment Trial User

Tutorial URL: https://developers.sap.com/tutorials/abap-environment-trial-onboarding.html
Please specify the step you are referring to : Step 5: Create ABAP cloud project
6. Didn’t get login option and logged in automatically.
Expected :

Actual:

Regards,
Nidhi

Hi @NidhiMurkute ,
this has been updated. @thecodester I created following pull request: #5702, can you merge it?
Thank you.
Best regards,
Merve


express js fail, after npm start

Hi,
I tried to run the generator, but run into some errors.
(I'm on OS X 10.9.2, node, npm, express, everything is the latest)
There are no problems until npm start I get this in the prompt:
><EMAIL_ADDRESS>start /Users/szmarci/Sites/express
> node ./bin/www

Then I visit http://localhost:3000/, I get the Express test page (but unstyled) and immediately after this in the terminal:
GET / 304 982ms

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: spawn ENOENT
    at errnoException (child_process.js:998:11)
    at Process.ChildProcess._handle.onexit (child_process.js:789:34)

npm ERR<EMAIL_ADDRESS>start: `node ./bin/www`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is most likely a problem with the express package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node ./bin/www
npm ERR! You can get their info via:
npm ERR!     npm owner ls express
npm ERR! There is likely additional logging output above.
npm ERR! System Darwin 13.1.0
npm ERR! command "node" "/usr/local/bin/npm" "start"
npm ERR! cwd /Users/szmarci/Sites/express
npm ERR! node -v v0.10.28
npm ERR! npm -v 1.4.9
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/szmarci/Sites/express/npm-debug.log
npm ERR! not ok code 0

I couldn't find any solution with google, tried to sudo, change port, etc.. but no luck.

@renfeisong thank you.
Indeed, I had the same issue and was due to a missing installation of Compass. Just running gem install compass solved the issue.
Thanks!

I'm running Node.js on windows 8, and when I'm encountering the same/similar issue.  I follow these steps and it works perfectly until Iry :
C:\node\nodetest>npm start
app.use.(function(req,res,next) {
TypeError: Cannot read property 'use' of undefined
at Object. {C:\node\nodetest\app.js:19:4)
at Module._compile (module.js:460:26)
at Object.Module._estiensions..js (module.js:478:10
at Module.load (module.js:355:17)
at require (module.js:365:17)
at Object. (C:\node\nodetest\bin\www:7:11)
at Module._compile (module.js:460:26)
at Obeject.Module._extensions..js (module.js:478:10)
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\node.js\\node.exe" "C:\Program Files\node.js
\node_modules\npm\bin\npm-cli.js" "start"
npm ERR! node v0.12.2
npm ERR! npm  v2.7.4
npm ERR! ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start 'node ./bin/www',
npm ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node./bin/www'
npm This is most likely a problem with the nodetest package, not with npm itself
AND this is my debug log:

0 info it worked if it ends with ok
1 verbose cli [ 'C:\\Program Files\\nodejs\\\\node.exe',
1 verbose cli   'C:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose node symlink C:\Program Files\nodejs\\node.exe
5 verbose run-script [ 'prestart', 'start', 'poststart' ]
6 info prestart<EMAIL_ADDRESS>7 info start<EMAIL_ADDRESS>8 verbose unsafe-perm in lifecycle true
9 info<EMAIL_ADDRESS>Failed to exec start script
10 verbose stack Error<EMAIL_ADDRESS>start: `node ./bin/www`
10 verbose stack Exit status 1
10 verbose stack     at EventEmitter.<anonymous> (C:\Program
 Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:213:16)
10 verbose stack     at EventEmitter.emit (events.js:110:17)
10 verbose stack     at ChildProcess.<anonymous> (C:\Program
 Files\nodejs\node_modules\npm\lib\utils\spawn.js:14:12)
10 verbose stack     at ChildProcess.emit (events.js:110:17)
10 verbose stack     at maybeClose (child_process.js:1015:16)
10 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
11 verbose pkgid<EMAIL_ADDRESS>12 verbose cwd C:\node\nodetest3
13 error Windows_NT 6.3.9600
14 error argv "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program
 Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "start"
15 error node v0.12.2
16 error npm  v2.7.4
17 error code ELIFECYCLE
18 error<EMAIL_ADDRESS>start: `node ./bin/www`
18 error Exit status 1
19 error Failed at the<EMAIL_ADDRESS>start script 'node ./bin/www'.
19 error This is most likely a problem with the nodetest3 package,
19 error not with npm itself.
19 error Tell the author that this fails on your system:
19 error     node ./bin/www
19 error You can get their info via:
19 error     npm owner ls nodetest3
19 error There is likely additional logging output above.
20 verbose exit [ 1, true ]

This is my app.js where the 'use' function is being called:
 var express = require('express');
 var path = require('path');
 var favicon = require('serve-favicon');
 var logger = require('morgan');
var cookieParser = require('cookie-parser');
var bodyParser = require('body-parser');

var routes = require('./routes/index');
var users = require('./routes/users');

/// catch 404 and forwarding to error handler
app.use(function(req, res, next) {
var err = new Error('Not Found');
err.status = 404;
next(err);
});

/// error handlers

   // development error handler
   // will print stacktrace
    if (app.get('env') === 'development') {
    app.use(function(err, req, res, next) {
        res.status(err.status || 500);
        res.render('error', {
            message: err.message,
            error: err
        });
    });
}

var app = express();
// view engine setup
app.set('views', path.join(__dirname, 'views'));
app.set('view engine', 'jade');
//
app.use(favicon(__dirname + '/public/favicon.ico'));
app.use(logger('dev'));
app.use(bodyParser.json());
app.use(bodyParser.urlencoded());
app.use(cookieParser());
app.use(express.static(path.join(__dirname, 'public')));
app.use('/', routes);
app.use('/users', users);

// production error handler
// no stacktraces leaked to user
app.use(function(err, req, res, next) {
    res.status(err.status || 500);
    res.render('error', {
        message: err.message,
        error: {}
    });
});

I'm just getting 'into cmd'' and really enjoy it, so sorry if my syntax/phrasing or anything makes this impossible to sort through in order to give advice (this is my first post to github).

@adc5 you need to put the line var app = express(); above when you first try to use the app variable (i.e. move that line above /// catch 404 and forwarding to error handler).

@dougwilson Thanks for that doug!  But when I call     npm start   it gives me  another set of errors:
rfs.js.:797
return binding.stat(pathModule._makeLong(path));
Error:  ENOENT, no such file or directory 'filepath'
at Error (native)
at Object.fs.statsSync (fs.js:797:18)
at favicon ('path to favicon')
at Object. (C:\node\nodetest\app.js:49:9)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:365:17)
at Module.require (module.js.:365:17)
at require (module.js:384:17)
And then the rest of the ERR! statements are the same.  Any other ideas?  Also,  is there a way to copy the text from node.js so I don't have to type it completely out each time?  Thanks again for the help!

@adc5 it's because you do not have a file at /public/favicon.ico. You either need to put one there, or comment out that line.

I tried that and it gave me:
body-parser deprecated undefined extended option app.js
C:\node\node\nodetest\bin\www:16  (line 16 is blank in app.js)
aopp.set('port', port);

app.set('port',port);

Sorry, accidently sent that while I was finishing it.  Could that mean I have a server connection issue>

and I forgot the @dougwilson : 2 previous posts

@adc5 I'm happy to help more, but please, let's stop filling the dozen people in this old thread's email inboxes. Please create a new issue here: https://github.com/expressjs/generator/issues/new


Sync view does not show file contents

Clicking a file does not show what the contents are to restore:

Fixed in master.


Hot reload doesn't work

Hello,
When I execute npm start  things work just as I expect them to.
But, when editing a file I see that webpack detects the change and recompile. ("Bundle is now invalid.. Bundle is now valid)
on the client side the change doesn't take affect.
When I try to refresh the page I can't find the files any more, only the bundled version (and one ctrl.ts) , so I have to rerun npm start
Maybe I'm just doing something wrong?
Tried running npm run watch alongside and change HMR to false, can't find a solution yet.
Thank you very much for your help.

@kutomer Is it possible to share some logs? May be that will help us to find the issue.

@sasidhar of course, thank you for your quick reply.
Here are the logs I'm getting after running npm start, it's quite long so I'm posting only the start and the end of it (I'm not getting any errors \ warnings)
><EMAIL_ADDRESS>server:dev /Users/kutomer/test
> webpack-dev-server --inline --progress --profile --colors --watch --display-error-details --display-cached  --content-base src/ "--hot"
                                                                                70% 2/2 build moduleshttp://localhost:3000/
webpack result is served from /
content is served from /Users/kutomer/test/src
404s will fallback to /index.html                                                                                46% 3/5 build modulests-loader: Using<EMAIL_ADDRESS>and /kutomer/test/tsconfig.json                                                                                2540ms1325ms optim103ms emit
Hash: e24a1fc1280a80452ba7
Version: webpack 1.12.14
Time: 8328ms
                                 Asset       Size  Chunks             Chunk Names
    assets/icon/android-icon-96x96.png     5.6 kB          [emitted]
                   polyfills.bundle.js     679 kB       0  [emitted]  polyfills
                        main.bundle.js    1.54 MB       2  [emitted]  main
                         polyfills.map     612 kB       0  [emitted]  polyfills
                                 1.map    1.03 kB       1  [emitted]
                              main.map    2.45 MB       2  [emitted]  main
                      assets/data.json   30 bytes          [emitted]
                     assets/humans.txt  251 bytes          [emitted]
                  assets/manifest.json  769 bytes          [emitted]
                     assets/robots.txt   31 bytes          [emitted]
              assets/service-worker.js   44 bytes          [emitted]
                   assets/css/.gitkeep   14 bytes          [emitted]
           assets/img/angular-logo.png      19 kB          [emitted]
    assets/img/angularclass-avatar.png    32.8 kB          [emitted]
      assets/img/angularclass-logo.png    36.7 kB          [emitted]
             assets/img/poepleIcon.png    4.99 kB          [emitted]
                            1.chunk.js    1.64 kB       1  [emitted]
         assets/icon/browserconfig.xml  281 bytes          [emitted]
                            index.html    2.67 kB          [emitted]
chunk    {0} polyfills.bundle.js, polyfills.map (polyfills) 626 kB [rendered]

After that it just start to log for each file the have built, for example:
 [6] (webpack)-dev-server/~/sockjs-client/~/inherits/inherits_browser.js 672 bytes {0} [built]
And the log ends with: webpack: bundle is now VALID.
When I change a file:
webpack: bundle is now INVALID.                                                                                2505ms 663ms optimi24ms emit
Hash: 3a2b36434fb7f46979ee
Version: webpack 1.12.14
Time: 4183ms
                               Asset      Size  Chunks             Chunk Names
                 polyfills.bundle.js    679 kB       0  [emitted]  polyfills
                      main.bundle.js   1.54 MB       2  [emitted]  main
2.e24a1fc1280a80452ba7.hot-update.js   3.22 kB       2  [emitted]  main
e24a1fc1280a80452ba7.hot-update.json  36 bytes          [emitted]
                       polyfills.map    612 kB       0  [emitted]  polyfills
                            main.map   2.98 kB    2, 2  [emitted]  main, main
chunk    {0} polyfills.bundle.js, polyfills.map (polyfills) 626 kB [rendered]
     + 105 hidden modules
chunk    {1} 1.chunk.js, 1.map 1.48 kB {2}
     + 1 hidden modules
chunk    {2} main.bundle.js, 2.e24a1fc1280a80452ba7.hot-update.js, main.map, main.map (main) 1.5 MB {0} [rendered]
  [356] ./src/app/home/home.ts 3.03 kB {2} [built]
       ... -> factory:1ms building:0ms dependencies:0ms
     + 259 hidden modules
Child html-webpack-plugin for "index.html":
    chunk    {0} index.html 415 kB
         + 3 hidden modules
webpack: bundle is now VALID.

I forgot to mention my spec in my previous message:
node: v5.7.0
npm: 3.6.0
os: OS x Yosemite 10.10.5

@kutomer, seems everything looks fine.

Are there any errors in the browser console?
Are you using angular2-webpack-starter with only code changes or some configuration changes (webpack-config.js, package.json etc) too?

I am on Ubuntu.

Thanks a lot.
I've tried to reinstall everything all over again it's working.
Btw, in your README file you said to install webpack-dev-server globally, but when doing so there is Unmet peer dependency warning.

@kutomer I'm going to close this. If you still have a problem with hot reloading can you own another issue thanks!


[WIP] UI refactor + aniso convert + fixed review plots

started cleaning up UI class/functions

fixed incorrect plots when using anisotropy correction

added anisotropy conversion to utils

fixed errors in launching review in new viewer

Codecov Report

Merging #17 (d1f83b2) into main (c2b2273) will decrease coverage by 0.22%.
The diff coverage is 61.01%.

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   37.40%   37.17%   -0.23%
==========================================
  Files          37       37
  Lines        3294     3346      +52
==========================================
+ Hits         1232     1244      +12
- Misses       2062     2102      +40

Impacted Files
Coverage Δ

napari_cellseg3d/plugin_convert.py
0.00% <0.00%> (ø)

napari_cellseg3d/plugin_crop.py
0.00% <0.00%> (ø)

napari_cellseg3d/plugin_helper.py
0.00% <0.00%> (ø)

napari_cellseg3d/plugin_metrics.py
0.00% <0.00%> (ø)

napari_cellseg3d/plugin_model_inference.py
0.00% <0.00%> (ø)

napari_cellseg3d/utils.py
31.57% <25.00%> (-0.07%)
:arrow_down:

napari_cellseg3d/plugin_dock.py
74.56% <66.66%> (-1.37%)
:arrow_down:

napari_cellseg3d/launch_review.py
58.76% <73.33%> (-0.24%)
:arrow_down:

napari_cellseg3d/plugin_review.py
89.33% <75.00%> (ø)

napari_cellseg3d/interface.py
87.33% <81.31%> (-0.34%)
:arrow_down:

... and 4 more

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 c2b2273...d1f83b2. Read the comment docs.

Edited with the feedback in mind :) Let  me know what you think.

Major overhaul to interface.py
Added anisotropy conversion to utilities
Updated docs


Handle patch release workflow

The process for handling patch releases was considered but has not yet been implemented.  Since NPSP 3.0 is deployed to everyone at once and we isolate all work in feature branches, we can easily cut new releases and push out for any issues that come up.  Thus, we've never needed to do patch upgrades.
However, many projects face customer demands that require use of the patch release process.  The solution is to create a branch to track the patch changes and ensure any fixes which need to be included in master get included.  This probably means adding new build targets and automation scripts to support the process, however much of the existing process should be able to be leveraged for patch releases.

CCI Flow doesn't use patch orgs/we don't recommend our users use them at this time. Our tooling is not optimized, YMMV. Perhaps someday, when we have muchmore bandwidth...and 2GP hasn't changed it all by then...


https+tcp+sni listener support

Adds support for the https+tcp+sni listener.  Any host routes that are marked with
proto=tcp or have a scheme tcp:// will be matched for TCP SNI steering.  Failing
any matches there, fallthrough will be to https matching.  This resolves: #783.
clean up error handling on 'use of closed network connection' case
fix makefile tag finding stuff
fix logic for https+tcp+sni matching so that only explicit proto=tcp matches

I'll give this a few days, failing any negative reviews I'm merging this


WARNING in<EMAIL_ADDRESS>Module not found: Error: Can't resolve './locale'

WARNING in<EMAIL_ADDRESS>Module not found: Error: Can't resolve './locale' in 'D:\demows\h5-travel-plus\node_modules.2.19.0@moment\src\lib\locale'
plugins: [
new webpack.IgnorePlugin(/^./locale$/, /moment$/)
]

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Added bridge sections to library and examples

@robbievanleeuwen Think I messed up committing to #132, but perhaps you can merge this in and reconcile. I don't think it will work out of the box due to the v2 restructure underway, but hope it is an easy fix for the imports. In a few weeks I can return and tidy up the typehints and get the styles matching better, but thought it might be more helpful to have the skeleton in there now at least.

Thanks @ccaprani, I might end up dragging these over to #132 and editing the example once #143 is merged :+1:

Thanks @robbievanleeuwen . If it's easier just ignore this PR and we'll do it once the library is merged.
I'll also add docs in a few weeks.


add docker-it by @whisklabs

Hi @agebhar1 Looks like a great tool. Once the name is updated according to my review we can merge this. Thanks!

@vegasbrianc - fixed it :)

@agebhar1 well done!
If you are interested, would you like to become a maintainer?

thx @veggiemonk - and yes I'm interested :)

@agebhar1 invite sent 👍 welcome.
few things you have to know:

we don't push directly to master, we create a pull request.
how to tell if a project should be on the list, read CONTRIBUTING.md
to ask a contributor to make a change, just copy paste this message here and changes a few things like names and stuff. The main idea is to help people get to quality.
If something seems weird, if you don't understand what a project does or the documentation is poor, don't hesitate to ask for more (see previous point).

Anyway welcome to the team, if you have any questions, please ask (use the chat, or twitter or whatever feels more confortable for you)


[FromUri] Complex types are not resolved (described as object)

Hi.
As in the title.
Thanks for your awesome tool

Asp net core or web api?

Web Api.
Thanks


org.springframework.dao.QueryTimeoutException: Redis server response timeout (3000 ms) occured after 0 retry attempts

I am using Redisson version 3.20.0 with the following connection properties:
ClusterServersConfig Redisson client with-
connectionTimeout: 3000,
timeout: 3000,
retry attempts: 3,
idleConnections: 48,
maxConnections: 128
I am using RedisTemplate to execute Redis queries in my code:
private RedisTemplate createRedisTemplate(RedisConnectionFactory redisConnectionFactory) {
        RedisTemplate redisTemplate = new RedisTemplate();
        redisTemplate.setDefaultSerializer("A CUSTOM SERIALIZER");
        redisTemplate.setConnectionFactory(redisConnectionFactory);
        redisTemplate.afterPropertiesSet();
        return redisTemplate;
}

The Redis connection factory object is configured with 3 Retry attempts.
Even though the Retry count is set to 3, sometimes the Redis query is getting timed out after 0 retry attempts only.
Can someone help me understand why is it not retrying 3 times?

can you share a full error message?

for non-idempotent commands a new attempt can't be made after Redis response timeout. You can find their list here org.redisson.client.protocol.RedisCommands#NO_RETRY_COMMANDS and org.redisson.client.protocol.RedisCommands#NO_RETRY


Add controlPlaneEndpoint to config file reference

First-time kubeadm user here: The v1.10.0 docs indicate that config file use is experimental, but judging from the comments on https://github.com/kubernetes/kubernetes/pull/59288 it looks like new features are beginning to be supported only in the config file. controlPlaneEndpoint is super useful for us and while I suspect the kubeadm init docs will ultimately be re-written to emphasize the config file options rather than flags (with appropriate examples and descriptions as the flags have now), it would've saved us some time today if we'd seen controlPlaneEndpoint in the config file example.

CLA signed.

/assign @bradtopol
/approve
It's a small change, but we should still get a tech review. Brad, if you know the current state of kubeadm, go ahead and add the /lgtm -- otherwise we should probably consult I'm not sure who. It also looks as though we should give the kubeadm docs some more love; I'm noting this PR to remind us to file an issue.
Thanks @MordantWastrel!

/cc @timothysc Any chance you could look at this.   This one could use your technical input. Simply just say /lgtm if you are okay with the change


See *.pdf in fullscreen mode.

I need to see the pdf on fullscreen mode, is that possible? The pdf opens but I can only see it if I exit the full-screen mode.
Thank you!

Fixed !!


Constant Power Torque Control

File updates and testing for constant power with TSR tracking.  Corresponds to similar PR in ROCO.  Will follow from refactor PR.

This is out of date and the changes have been merged in numerous other PRs. Closing


cmd/go/internal/load: fix typo in comment

I got your reply, I will close the wrong pr, sorry for that


Add the hits parameters

Some solutions has the concept of query complexity/cost and I'd like to this value to rate limite the customer.
If change the api to add the hits parameter will be possible to use hammer to rate limit
  @spec check_rate(id :: String.t(), scale_ms :: integer, limit :: integer, hits :: integer) ::
          {:allow, count :: integer}
          | {:deny, limit :: integer}
          | {:error, reason :: any}
  def check_rate(id, scale_ms, limit, hits \\ 1) do
    check_rate(:single, id, scale_ms, limit, hits)
  end

Does that make sense to you?

Ah, I see, this does make sense.
Just to to totally clear, this change would allow you to specify the number to be added to the bucket, rather than the constant 1 that it is now?

Cool, I'll take a swing at it in the next week or so. Or, PRs welcome. :+1:

Ok, I've found that adding a default parameter to count_hit causes a conflict between the two existing function-heads (count_hit/3 and count_hit/4), so this feature will be added as a new function: count_hit_inc.

I have released version 5.0.0-rc1 of both hammer and hammer-backend-redis.
They include a new check_rate_inc function that can be used to supply a custom increment. Example:
# Bulk file upload
user_id = get_user_id_somehow()
n = get_number_of_files()
case Hammer.check_rate_inc("upload_file_bulk:#{user_id}", 60_000, 10, n) do
  {:allow, _count} ->
    # upload all of the files
  {:deny, _limit} ->
    # deny the request
end

Cool, I'll cut a release for 5.0.0 soon. Thanks!

This is done, and released in v5.0.0, of both hammer and hammer_backend_redis.
There is a new tutorial section on Custom Increments which explains how to use this new feature.


[BC] Material Library

Hello to everybody,
I have been testing the current implementation of the MatLib-Editor within the pyleecan GUI and recognized some short comings. For instance, if a material is edited, the changes are not propagated thoughout the whole machine.
Therefore I would like to discuss the behavior that everybody would expect from the Material Library Editor.
Here are some thoughts and questions from my side.

changes to material library should propagate to all machine parts if they use the respective material, i.e. material object references should not been broken
Material Library should show the origin of the currently selected material (MatLib or Path)
it should not been allowed to have duplicate material names
if a loaded machine contains "new" material, the user should be asked to add this material to the library, to keep it as is or to replace it with a material from the library that has the same data, if it exists
if there are materials with the same name (in the machine file and/or the library) the user should be asked to rename the material
an alternative may also be to keep the machine files materials "outside" the library and only inside the material selection combobox with the possiblity to also edit that material (that may require some additional logic to handle all possible cases)

All of your contribution to this issue is greatly appreciated.
Best regrads,
Sebastian

Hello Sebastian,
Thank you for this interesting issue. The reason the Matlib is working that way is a legacy from MANATEE where the material are stored in separated files from the machine one. For MANATEE only the name of the Material is meaningful for the selection and everything is loaded at the beginning of the simulation. The MatLib only update the material file for later use.
I agree that we need to change that to have something more coherent with pyleecan.
Note that we currently save the material within the machine object which can cause the following issue: I create a machine with Mat_A, save it, change some value of Mat_A latter, then come back to the old machine which still has the old value of Mat_A (but I may expect to have the new value). To avoid that when loading a machine we can overwrite the material with the current version of the material library. If we need to create such function when loading a machine we can call this function when updating the material library. That way the material in the MatLib are the reference.
The GUI breaks the material reference to enable the user to cancel its modification which can be interesting when you may change a material used in several machines. But it may be not necessary if we create a copy of the material and then call the init method of the current instance with the as_dict of the copy.
Regarding the "origin" of the material I don't understand what you mean but we can display the path to the file.
I agree for the duplicated name

if a loaded machine contains "new" material, the user should be asked to add this material to the library, to keep it as is or to replace it with a material from the library that has the same data, if it exists

I agree if we detect a new material by checking only its name.
What do you think of the idea to use the MatLib as the reference for all material handled within the GUI ?
Best regards,
Pierre


NLog doesn't colorize VSCode Debug Console

Hi guys,
I am using VSCode to develop a .NetCore solution with NLog as my logging provider. I have set up everything according to the tutorial:

Installed NuGet Package NLog.Config
Added NLog.config file and modified console type to "ColoredConsole"
Added NLog.config to project output with CopyToOutputDirectory set to true
Add a static variable to my classes that gets a logger instance from LogManager
Use logger.Warn("..."), logger.Error("...") and logger.Info("...") according to my needs

However, when I debug my app inside VSCode, the logs are written into the debug console without any colors. When I run into an unhandled exception, the debug console displays a message in a dark yellow/orange color so it should work or am I missing something here?
<?xml version="1.0" encoding="utf-8" ?>
<nlog
    xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" throwConfigExceptions="true">
    <targets>
        <target name="logfile" xsi:type="File" fileName="logs.log" />
        <target name="logconsole" xsi:type="ColoredConsole" />
    </targets>
    <rules>
        <logger name="*" minlevel="Info" writeTo="logconsole" />
        <logger name="*" minlevel="Debug" writeTo="logfile" />
    </rules>
</nlog>

I guess if you enabled the terminal system i VsCode:
https://github.com/Microsoft/vscode/issues/243#issuecomment-311680959
And then "just" fixed up this PR for adding support for ANSI color codes in NLog (Read as creating new PR):
https://github.com/NLog/NLog/pull/1495
Then you would probably get colors in VsCode.

@LeonidasFettikus Could you try this pre-release of NLog (NLog.<IP_ADDRESS>41-PR3018.nupkg):
https://ci.appveyor.com/project/nlog/nlog/builds/20372413/artifacts

yes please try, then we can merge it soon :)

@304NotModified This can be added to NLog 4.6 milestone with PR #3018

🎉  thanks!

FYI, NLog 4.6 RC is online which includes this feature (EnableAnsiOutput  option)


Using a custom setting in the customizer no longer working

Using a custom setting in the customizer (which is recommended by Kirki) is no longer working, after upgrading Wordpress to v4.7.
This is the method that requires you to add a few lines of code in your theme that will create a custom section and a custom control.
Kindly check it and advise on a lasting solution.

fixed


Feature request: Blockly implementation

As discussed, please implement blockly as a more user friendly way to create rules.
Source: http://forum.mycontroller.org/topic/159/integrate-blockly/11
Thanks

Any news about this?

@blacksheepinc No, Sorry for the late response. Till now I do not find great time complete this feature.

I am also interested in this feature being added. Any idea if it will be at some stage?


⚠ Turnos 24x7 has degraded performance

In 1e3bf4c, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 962 ms

Resolved: Turnos 24x7 performance has improved in 834f6e9.


Adding support for len/__len__

Several array implementation currently implement len/__len__ basically as shorthand for .shape[0]. It would be handy to have this to simplify adoption in existing codes as well as fit more with how current array implementations work

It's hard to search for because "len" isn't a very unique string, but we discussed this multiple times for both arrays and dataframes and there was wide agreement that len(x) is a bad idea. I honestly still need to look up myself or test quickly in IPython along what axis it iterates.
@jakirkham I propose we close this.

Interesting maybe I'm missing context here
Thought the main issue was dimensions of unknown length. This was noted when it was originally included and then removed ( https://github.com/data-apis/array-api/pull/289 )
Anyways am wondering if we should revisit given the number of lines of code using len needed to change in scikit-learn to accommodate the Array API. So more generally am concerned about the churn we may be causing by leaving things like this out

len() is something that I am slightly hedgy about specifying, but if no projects involved ever played with the idea of behaving differently, I suppose it can be done.
The reason is that there is an alternative way to look at arrays; which is also the one that SymPy uses (for its matrix):

len(arr) should be the same as arr.size
For loops should iterate all elements (rather than the first dimension/axis).

By not specifying len() you remain compatible with that choice.
Starting from scratch, I would personally consider that choice.  Because I like to think of arrays more as a "collection of values/elements with a homogenous structure".  The "collection of values/elements" comes first and the "structure" only later in my philosophy :).
That does break the list-of-lists analogy which may be confusing to users.  But to some degree that is a feature not a bug.

Starting from scratch, I would personally consider that choice.

Yes, I think that is more reasonable indeed. len is only unambiguous for 1-D arrays. "list of lists" is a concept that I don't think many users have in mind, and it's pretty much 2-D specific as well (you can think about a 3-D array as "list of lists of lists" in principle, but who would actually do that?).

Anyways am wondering if we should revisit given the number of lines of code using len needed to change in scikit-learn to accommodate the Array API.

Honestly, I think this is (a) not a hard thing to change all at once for a code base, and (b) a good improvement for readability even in the absence of array API usage.

One semi-related thing I would actually like to add to NumPy - but never worked on - is arr.iteraxis(axis=None, *, order="K"), with the semantics of axis=None iterating all elements (similar/identical to arr.flat, returning scalars in NumPy).
And otherwise axis selecting all axes to be iterate.

I agree that arrays don't need to be directly iterable & with a length, but we do need a way to unpack arrays along an axis into a list.
If we don't have any way to do that currently in the standard, then we need an unstack function.

I agree that arrays don't need to be directly iterable & with a length, but we do need a way to unpack arrays along an axis into a list.

Can you clarify why this is needed? Typically if you want to work along an axis of an array you just work along that axis.
This can be done with a list comprehension: [a[:, :, i, ...] for i in range(a.shape[k])] (with k :s).

I agree that arrays don't need to be directly iterable & with a length, but we do need a way to unpack arrays along an axis into a list.

Can you clarify why this is needed? Typically if you want to work along an axis of an array you just work along that axis.
This can be done with a list comprehension: [a[:, :, i, ...] for i in range(a.shape[k])] (with k :s).

OK, fair enough, indexing in a loop would definitely do the trick here. Still maybe worth considering adding unstack() as a convenience function. (I'll open a separate issue to discuss this.)

Can we close this issue with outcome that the proposal is rejected? No requirement to add __len__ (but of course implementers are free to add it anyway, outside of their standard support).

No more comments, I'll close this. Thanks all!


Fix https://github.com/v0dro/daru/issues/250

Read your comments. Could you please open an issue about this so it can be discussed further?


Fix for Emmet's wrap with abbreviation inserting extra spaces

Fix for https://github.com/Microsoft/vscode/issues/29898
Now, when wrapping text with a multiline abbreviation such as ul>li>a, no extra spaces are added.
Added a unit test to keep track of this issue.

Instead of expanding the abbreviation to figure out if the wrapping element is inline or not, we can parse the abbreviation to get a tree and use it to figure out the same. And then, the same tree can be used by the expand module

There needs to be a setting for VSCode which will allow to completely disable automatic indentation based on identifying what type of element the wrapping tag is.
For example, current implementation for wrapping a selected piece of text with any block level element like p or div will produce the following result
<p>
    Selected text
</p>

which might be technically correct but it's ugly and completely unnecessary in terms of wasting lines and space.
I basically need VSCode to stop overthinking everything and just wrap code with a tag of my selection without automatically indenting it.

I basically need VSCode to stop overthinking everything and just wrap code with a tag of my selection without automatically indenting it.

@brezanac, That might work well for your case, but doesn't really help when wrapping multi-line text or better yet, multi line html
If you do have a solution that works for both cases, please do send a PR. We would appreciate it.


RETRIEVE YOUR LOST OR SCAMMED CRYPTOCURRENCY WITH - TRUST GEEKS HACK EXPERT

TRUST GEEKS HACK EXPERT  gave me hope even though I thought I had lost everything when my Bitcoin wallet and $66,000 worth of cryptocurrency vanished. Some of the best hackers working for this cutting-edge cybersecurity company are experts at finding lost digital valuables, such as Bitcoin. They gave me hope that I could get my hard-earned Bitcoin fortune back with their state-of-the-art forensic equipment and never-say-die attitude. The TRUST GEEKS HACK EXPERT team approached my case like dogged prosecutors pursuing justice. They thoroughly investigated the sequence of events, gathered evidence, and left no stone unturned in identifying the perpetrator who compromised my wallet. Their stern, laser-focused questioning and analysis felt like I was on trial, but I complied fully, trusting it was all to build an irrefutable case against the criminal. As the investigation unfolded, the TRUST GEEKS HACK EXPERT kept me continually updated. Though progress was slow and methodical, their reassuring confidence never wavered. I began to share in their optimism that we would prevail, no matter how complex or tedious the process. And when they finally cornered the hacker and compelled him to return my pilfered Bitcoin, it was a triumphant victory for justice. My heart swelled with gratitude for these cybersecurity crusaders who turned a nightmare into a righteous conquest. Thanks to TRUST GEEKS HACK EXPERT  diligence and expertise, the optimism that flickered inside me at the start blazed into a steady flame of hope. As the investigation progresses and the stolen funds are traced to their source, TRUST GEEKS HACK EXPERT enters into negotiations with the perpetrators to secure the recovery of the stolen assets. Through skillful negotiation and unwavering determination, their protagonist edges closer to the light at the end of the tunnel. Let this story stand as a testament to the power of perseverance, the importance of safeguarding digital assets, and the enduring light that can be found with TRUST GEEKS HACK EXPERT  in the darkest of times. You can simply communicate with TRUST GEEKS HACK EXPERT through  email<EMAIL_ADDRESS>$ TelegramID: Trustgeekshackexpert

🚨🚨
This is a cryptocurrency spam/scam! DO NOT REACH OUT TO THE ABOVE TELEGRAM HANDLE, NOR EMAIL ADDRESS, POSTED ABOVE!!!!. See #26657


Show unmatched stubs in VerifyStubs error output

The VerifyStubs methods in the api and run packages just list the number of unmatched stubs. It would be a better UX if they actually listed the match strings. Something like:
found 2 unmatched stub(s):
- MatchPost: /api/v1/users
- MatchGet: /api/v1/users/123

// or...

found 2 unmatched stub(s):
- MatchString: /bin/echo foo
- MatchString: /bin/echo bar

Unfortunately stubs don't have access to any of this data due to the way matchers are created. Typically we use a factory function (for example, MatchPost) to create an anonymous Matcher function that gets passed to the stub, so even if we tried to use reflection shenanigans we wouldn't get anything useful to show users.
Going to keep this open for a bit in case we ever redo how matchers work.


Missing required module 'SDWebImage'

Not sure if this is an SDWebImageSwiftUI/SDWebImage issue, or a Swift Package Manager issue, but when I have an iOS app with an accompanying watchOS app, and include SDWebImageSwiftUI in both the iOS app and watchOS app extension (via SPM), building the watchOS app using the watchOS app scheme succeeds, but when trying to build the iOS app, the watchOS app build fails with an error in the SDWebImageSwiftUI library:
No such module 'SDWebImage'

If I remove the SDWebImageSwiftUI library from the watchOS app extension, I get an error in the watchOS app extension:
Missing required module 'SDWebImage'

Sounds little strange.
Did you mean: When I create a dependent WatchKit App with SwiftUI (The Host WatchOS App is not gray), this cause the import issue ?
Can you have a quick test with other Package Manager like Carthage and CocoaPods, to test whether SwiftPM cause issues ?

And, please do a clean build between you switch Xcode scheme, I doubt this is because some cache issues ?

Tried the clean build, and deleted the derived data directory, but no joy.
Here's the commit it's happening on
https://github.com/adamayoung/Movies/tree/641d17534a99aab103499103e818eba0fa3fa135
I've created an independent watch app now, and it's working as expected.

Instructions to reproduce:

Create new Xcode iOS Project

Add SDWebImageSwiftUI as a Swift Package dependency and include in iOS target

import SDWebImageSwiftUI in a file in the iOS target

Build - should be fine

Add a new target - 'Watch App for iOS App'

Link SDWebImageSwiftUI library in watch extension target

import SDWebImageSwiftUI in a file in the watch extension target

Build watch scheme - should be fine

Change back to the iOS app scheme

Build - Should get error 'No such module 'SDWebImage'

Can not reproduce on Xcode 13.1 ? Any extra steps ?
Build - should be fine
Add a new target - 'Watch App for iOS App'
Link SDWebImageSwiftUI library in watch extension target
import SDWebImageSwiftUI in a file in the watch extension target
Build watch scheme - should be fine
Change back to the iOS app scheme
Build Fine for me ---> Build - Should get error 'No such module 'SDWebImage'

I also hit this error, but only on the XCode 11.4 Beta.

@joeycarmello @adamayoung Does Xcode 11.4 Beta 2 solve this issue ?
https://developer.apple.com/documentation/xcode_release_notes/xcode_11_4_beta_2_release_notes?language=objc

On Xcode 11.4 Beta 2 I get the following while attempting to resolve packages:
Source files for target SDWebImageMapKit should be located under DerivedData/[X]/SourcePackages/checkouts/SDWebImage
DerivedData/[X]/SourcePackages/checkouts/SDWebImage/Package.swift: target 'SDWebImageMapKit' referenced in product 'SDWebImageMapKit' is empty

@joeycarmello @adamayoung
This issue seems been solved by Xcode 11.4-Beta3.
Check SDWebImage/SDWebImage#2941

I am getting this issue in XCode 12, tried both SPM as well as Cocoapods. I added a new WatchOS target to my existing iOS app and later on I get the error No such module 'SDWebImageSwiftUI'
Both the projects fails to build ... any clue?

I am getting this issue in XCode 12, tried both SPM as well as Cocoapods. I added a new WatchOS target to my existing iOS app and later on I get the error No such module 'SDWebImageSwiftUI'
Both the projects fails to build ... any clue?

Afraid I'm seeing this in XCode 13. Note that the project does compile and run correctly; the bug seems to be in Tests iOS, and interrupts SwiftUI previews.

Afraid I'm seeing this in XCode 13. Note that the project does compile and run correctly; the bug seems to be in Tests iOS, and interrupts SwiftUI previews.

I was also struggling with this breaking my SwiftUI previews. I was able to resolve it by ensuring the test target host application was set correctly:

Navigate to your Xcode project in the left sidebar
Select your "Tests" target
In the General tab, under Testing, ensure you have a "Host application" selected

I hope that helps!

I am getting this issue in XCode 12, tried both SPM as well as Cocoapods. I added a new WatchOS target to my existing iOS app and later on I get the error No such module 'SDWebImageSwiftUI' Both the projects fails to build ... any clue?

Did you ever get this figured out? I'm getting this in Xcode 13 as well?

^ doing this I solved the issue

^ if you are getting error in XCTests or XCUITest then select your test targate and perform this

Thanks. This worked for me

getting a similar error after a pod update with xcode 14.3.1 as well - from the diff, not much was changed, but afterwards i am getting the error No such module 'SDWebImage'


Handle case of missing InResponseTo when validation is on

This supersedes PR #286 and addresses issue #284. Please references those items for context.

I've also included a stub JS file that will help future users to sign sample XML used for creating tests.


test(jest): setup

Deploy preview ready!
Built with commit 77d869f0ab060222cc48dd91c3151f096c954c30
https://deploy-preview-2--react-progressive-bg-image.netlify.com


Login problem similar (or same) to #34

See most recent comment on #34 posted after issue was closed.
Will soon provide more info from my installation as soon as I get to a PC.

Never mind.
Upon more indeep research I found the real problem is instead #37


Bypass deleting cloud credential resources in validation tests

Issue: #814
Backport
link
Problem
Currently, hosted cluster tests don't properly remove clusters in cloud providers because cloud credentials are deleted before other resources that need cloud credentials are removed.
Solution
Bypass deleting cloud credentials since they don't pose a significant load to Rancher

@ igomez06 We can, but it needs to be a separate issue. Removing things like EC2 instances takes random amounts of time, and we need a way to check if all of the nodes in a Kubernetes cluster have been deleted in the infrastructure provider before removing the cloud credential. I was running into issues where I thought the deletion process was started and then removed the cloud credential, and then the EKS cluster was still left up after the test. Since the cloud credentials don't pose much of a performance hit, I wanted first to eliminate Kubernetes clusters being left up and then later determine how to remove the cloud credentials.


Update delete-files.md

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

Per diff checker, the revision replaces "deletes all files in any sub-folder named" with "deletes all files and folders in any sub-folder named". Thanks.


When I accedentialy refreshed the page through browser in fullscreen, it became stuck there.

Literally can't use the screen anymore

Um, it works again.


Add pagination option to hide "Rows per page"

Issue Type

[x] Enhancement Request

Specs
What version are you using?
2.21.0

You want to disable pagination?

@vjik you can actually customize the pagination however you like: https://xaksis.github.io/vue-good-table/guide/advanced/#custom-pagination by using your own pagination elements.
closing.


Add initial build workflow for CI using GitHub Actions

It took me a while to determine which versions of everything work, to research all the flags needed to work around various issues, and otherwise get all the magical incantations just right, but with these updates GitHub Actions is pretty much working on Linux, Mac, and Windows including MKL and CUDA for CI purposes. I had to upgrade to TensorFlow 2.1.0-rc2 because 2.0.0 doesn't build correctly when using Visual Studio 2019 and that's the only version available on the hosted virtual machines. The result looks something like this on my fork: https://github.com/saudet/tensorflow-java/commit/4d7340eea7bd354b8da2283e61c276360852e290/checks?check_suite_id=377172023
The only remaining issue is the small size of the page file on Windows that makes MSVC run out of heap space: https://github.community/t5/GitHub-Actions/Error-quot-The-paging-file-is-too-small-for-this-operation-to/m-p/41713#M4723
I've tried to work around that by using options like --local_cpu_resources=1 and --local_ram_resources=2048 for Bazel, and /Zm10 for the C++ compiler, but it's not helping. We'll need to wait until they fix this, or if Google is going to give us beefier machines with GPUs and everything to run this on anyway, we might as well do this now. @ewilderj What's the status on that front?
In any case, after we figure out something for the credentials on Sonatype OSS, we can start uploading snapshots automatically from these builds. @sjamesr How should we proceed?

Hey Samuel, I can get you added to the Sonatype repository ACLs, though we
probably need to create a new snapshot repo. I assume that's a matter of
filing a ticket with the Sonatype folks.
On Fri, Dec 20, 2019 at 6:24 AM Samuel Audet<EMAIL_ADDRESS>wrote:

It took me a while to determine which versions of everything work, to
research all the flags needed to work around various issues, and otherwise
get all the magical incantations just right, but with these updates GitHub
Actions is pretty much working on Linux, Mac, and Windows including MKL and
CUDA for CI purposes. I had to upgrade to TensorFlow 2.1.0-rc1 because
2.0.0 doesn't build correctly when using Visual Studio 2019 and that's the
only version available on the hosted virtual machines. The result looks
something like this on my fork:
https://github.com/saudet/tensorflow-java/commit/85dbc76cd253bbaf35f7854a358cbba38c289252/checks?check_suite_id=367710593
The only remaining issue is the small size of the page file on Windows
that makes MSVC run out of heap space:
https://github.community/t5/GitHub-Actions/Error-quot-The-paging-file-is-too-small-for-this-operation-to/m-p/41713#M4723
I've tried to work around that by using options like
--local_cpu_resources=1 and --local_ram_resources=2048 for Bazel, and
/Zm10 for the C++ compiler, but it's not helping. We'll need to wait
until they fix this, or if Google is going to give us beefier machines with
GPUs and everything to run this on anyway, we might as well do this now.
@ewilderj https://github.com/ewilderj What's the status on that front?
In any case, after we figure out something for the credentials on Sonatype
OSS, we can start uploading snapshots automatically from these builds.
@sjamesr https://github.com/sjamesr How should we proceed?
You can view, comment on, or merge this pull request online at:
https://github.com/tensorflow/java/pull/8
Commit Summary

Add initial build workflow for CI using GitHub Actions

File Changes

A .github/workflows/ci.yml
https://github.com/tensorflow/java/pull/8/files#diff-0 (105)
M README.md
https://github.com/tensorflow/java/pull/8/files#diff-1 (4)
M tensorflow-core/tensorflow-core-api/WORKSPACE
https://github.com/tensorflow/java/pull/8/files#diff-2 (8)
M tensorflow-core/tensorflow-core-api/build.sh
https://github.com/tensorflow/java/pull/8/files#diff-3 (19)
M
tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/c_api/presets/tensorflow.java
https://github.com/tensorflow/java/pull/8/files#diff-4 (2)

Patch Links:

https://github.com/tensorflow/java/pull/8.patch
https://github.com/tensorflow/java/pull/8.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/java/pull/8?email_source=notifications&email_token=ABDNGA5B7TAAJGWZUMV4EQDQZTITTA5CNFSM4J56SAWKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IB7KMCQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDNGA7NI6HK5W43AYNFG4DQZTITTANCNFSM4J56SAWA
.

@sjamesr My username on Sonatype OSS is also saudet. If you can give me permission to deploy for org.tensorflow that would be great. We just need to file a ticket for requests like that here: https://issues.sonatype.org/
To have GitHub Actions upload artifacts tough, we need to add the credentials with this:
https://help.github.com/en/actions/automating-your-workflow-with-github-actions/creating-and-using-encrypted-secrets
I can do that for my own fork, but not here. I guess I could test it on my fork with my own personal credentials, and when it gets merged here, you can add the ones you've been using until now in the settings there. Sounds good?
BTW, for OSS projects Sonatype lets us use their own repo for snapshots as well, so that should be OK for now anyway, but sure, if you have or will have another Nexus instance somewhere that would be more appropriate, we can use that easily.

@saudet I just filed https://issues.sonatype.org/browse/MVNCENTRAL-5426

@saudet I just filed https://issues.sonatype.org/browse/MVNCENTRAL-5426

Thanks! BTW, I don't have access to that issue. It looks like OSSRH and MVNCENTRAL are 2 different "projects"...
Anyway, do you think we should merge what we have here right away, or should we first wait until we get builds on Windows working and deployment going?

@karllessard You'll need to be more specific about what changes you're requesting. You could also make those changes directly on my fork. That's probably going to be easier.

@saudet Actually I don't request any changes, I just did it to prevent the PR to be updated for deployment and merged before we address this issue (which seems to be the organic evolution of your previous discussions :) )
Ok so what about we merge this one as is and make another PR for deploying the artifacts, once the credentials are set and licenses are fixed @sjamesr ?

I see, so that would be an additional issue that needs to be resolved before we can start deploying snapshots. Since running the builds and tests is still useful by itself, instead of trying to address all these issues here, it might make more sense to merge this PR right away. Actually, we should probably follow a policy of not trying to address multiple issues like that so PRs don't become stale and end up with merge conflicts all the time.

I agree, I was just waiting for James to merge it since he started to review it but I guess he might be off for holidays so to speed up things, I'll do it right away, I think everyone is on the same page here.

Created this issue to follow up on licensing issue that (might) prevent us to enable automatic deployments: https://github.com/tensorflow/java/issues/10


Node 9 Compatibility

On yarn install, the following message is displayed:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">4.4 < 7".
Is there support for Node 9? Of course I can work around this by editing the default package.json but I'm curious if there is some incompatibility I should be aware of.

Technically there shouldnt be any issues, but we keep the same restriction as the angular project do just in case https://github.com/angular/angular/blob/master/package.json#L11. So if anybody wants to go around that, its up to that user.

If that's so, why not just a warning instead of an error? i stumbled into this for a few hours before finding this issue, because it blocks install.


[CLEANUP] Need to specify the versions of some eclipse dependencies in

order to avoid pulling in Java 11 artifacts.

:white_check_mark: Build finished in 4m 45s
Build command:
mvn clean verify -B -e -Daudit -amd -pl assemblies/lib

:exclamation: No tests found!
:information_source:  This is an automatic message


Feature: Enable runner instance volume type choices

Summary
Currently ec2 instances are launched with a gp2 volume type by default, however, gp3 volume types can be more performant and cheaper. The host runner can be launched with gp3 EBS volume types, but it's not possible to enable gp3 volume types. The GitLab Docker EC2 Driver supports setting this value, https://gitlab.com/gitlab-org/ci-cd/docker-machine/-/blob/main/drivers/amazonec2/amazonec2.go#L211
Suggested Implementation
I propose to allow setting a new variable/option of runners_volume_type, which will default to gp2 to maintain backwards compatibility, that can then be passed into the Runners Machine Config in the launch template

I'm happy to pick up the work on this issue


Email-Password on AuthViewController Doesn't Query to Set a Password on First Use

I'm using authViewController for FirebaseUI/Auth. Here is what it looks like when I select the email-password option:

As you can see, there's no query to set a password, so it's impossible to actually authenticate with this method. Am I doing something wrong? How do I fix this?
Also, the flow is kind of strange. Why does it ask for your email on one page and then your email and password on the next?
Addendum
Email-password IS enabled in my Firebase Console.
The only code I use is to call authViewController because the email-password option appears automatically.
self.authViewController = [authUI authViewController];

I just figured it out.. I was giving it an email for a previously authenticated FIRUser

The flow asks the user for the identifier first to determine what the next step should be. It not only helps differentiate sign-in and sign-up cases, it also avoids creating a duplicate account for users who have signed up using an identity provider.


Clarify Composition vs ResourceParticipation

There seems to be overlap, or one can be drived from the other? Clarify and potentially refactor.

resolved in resourceODP 0.3


Add documentation entry for "MyST vs Sphinx"

In a The Turing Way working group meeting, it was put to me

What is the difference between MyST and Sphinx

I gave an answer, but I don't think it was concise enough. We should probably have something to hand as it's a natural question!

Hey @agoose77,
I am sure you have given me a convincing explanation in the past, but I find the page under Reference > Background a useful reference! Maybe it's not very findable by users since it's buried beneath the Reference dropdown.


after_connect lifecycle hook status?

I posted to the Elixir Slack and was given https://github.com/elixir-ecto/ecto/commit/91077b8bf1849bbcea534b67cbcc0cef36cdf7f5 as a reference. We used to have an after_commit callback hook, but it's not there now. Can anyone connect the dots for me? Why did we lose the hook we used to have?
I'm looking for a way to run a simple query to validate some app-specific data assumptions whenever the Repo starts. Currently I use an init/2 in MyApp.Repo to start a Task that waits a second before doing its thing, but I'm not really satisfied with this. Is there a better way?

After more discussion in slack, @michalmuskala recommended I add an extra transient process to the supervisor that starts the Repo. after_connect was never really intended for my use case since it happens for each connection in the pool, not once globally. Closing issue.


Use github API to get a list of changed files on git push

do this work in a new file: src/lib/github/changedFilesList.ts and in a new branch.

See https://probot.github.io/docs/github-api/,  https://octokit.github.io/rest.js/#api-Repos-getCommit & https://developer.github.com/v3/repos/commits/#get-a-single-commit


redis in example ????

why you write docs not wok example ???

This hasn't been updated for over 2 years that's why. I managed to get it working, I just had to update these parts. Pipeline doesn't appear to be a function anymore.
sessionEngine.use('touch', async (session) => {
    return await redisClient.expireAt(session.signed_id, session.expires_at);
});

sessionEngine.use('write', async (session) => {
    const key = session.signed_id;

    return await redisClient.multi()
        .set(key, JSON.stringify(session.get()))
        .expireAt(key, session.expires_at)
        .exec();
});

Also if anyone is having trouble deleting cookies (spent way too much time on this...)
await request.session.destroy(); make sure you set it to false, if not using HTTPS yet.
cookie: {
        secure: false,
}


chore: run amplify table test separately to avoid timeout

Description of changes

The amplify-table-1 is a long running E2E (~45 mins) that times out the execution when run in a batch of 3 tests. This PR separates this test out as a solo job so we save some execution time by running in parallel and avoid the instability from long running batches.
CDK / CloudFormation Parameters Changed

Issue #, if available

Description of how you validated changes
E2E passed in CI
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing in favor of https://github.com/aws-amplify/amplify-category-api/pull/2261


ocenaudio: 3.11.0 -> 3.11.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/ocenaudio/versions.
meta.description for ocenaudio is: "Cross-platform, easy to use, fast and functional audio editor"
meta.homepage for ocenaudio is: "https://www.ocenaudio.com"
meta.changelog for ocenaudio is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin/ocenaudio had a zero exit code or showed the expected version
/nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin/.ocenvst-wrapped passed the binary check.
Warning: no invocation of /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin/.ocenaudio-wrapped had a zero exit code or showed the expected version
/nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin/ocenvst passed the binary check.
/nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin/ocenaudio_license.txt passed the binary check.
2 of 5 passed binary check by having a zero exit code.
1 of 5 passed binary check by having the new version present in output.
found 3.11.1 with grep in /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1
found 3.11.1 in filename of file in /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
docbookrx
ocenaudio

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ocenaudio https://github.com/r-ryantm/nixpkgs/archive/42cfe37cb2db507b834502d4b80aa422529d1d46.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1
ls -la /nix/store/drmc8gvyz0mcasakybmvds8mqipk2hrj-ocenaudio-3.11.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ocenaudio

Maintainer pings
cc @onny for testing.

Result of nixpkgs-review pr 151416 run on x86_64-linux 1

1 package built:

ocenaudio