Can a section be aligned to the page bottom?

I'd like to make a number of changes to the PDF version of my project so it looks like an InfoQ Mini-Book.
The first thing I'd like to start with is aligning the colophon to the bottom of the page. Is that possible?

Additional questions I've come up with since I started looking into this:

Is it possible to customize section headings? For example, InfoQ's Dedication and Acknowledgements is center-aligned and is underlined by /////////////////////////. The other sections have their own page for the level-2 headings 0 with large fonts and sometimes quotes under them. I've attached a screenshot below.

Using :toc: macro and toc::[] doesn't seem to work. I'd like to put the TOC after the dedication, acknowledgements and preface, but it seems to remain at the beginning no matter what I do. Also, page numbering starts on the 2nd page of the TOC. The InfoQ mini-book standard is to start it on the page after the TOC renders. If I could put it where I wanted, there would be several pages (numbered with Roman numerals) before the TOC.

Is it possible to make the page after the 1st page into the colophon? This page has the book title, author name and version, but I'd like to include the copyright, publisher and ISBN number on it too. It'd be great to be able to customize this page or just replace it. I could probably delete and move pages in the rendered PDF, but I'd rather automate it.

There are currently two ways to customize Asciidoctor PDF. One way is using the theme (declarative) and the other is by extending the converter (programmatic).
The first and easiest way is to tweak the theme file. The drawback of this approach is that the capabilities of what the theme can control are still (gradually) emerging. We're attempting to define a declarative, CSS-like design “language” without making it overly technical. The target audience for the theme customization strategy are people who want/need to apply brand guidelines but otherwise retain the general structure (i.e., opinionated layout) of the output document. We need to find the right balance.
The second way to customize is to extend the converter using Ruby (or, eventually, other JVM languages). There are no limits to what you can change once you take this approach (aside from the inherent limitations of Prawn and the PDF format). Using this approach, you can dive deep into the internals without having to fork the project.
Since the first method is already well covered by the theming guide, I'll focus here on the second.
To extend Asciidoctor PDF, you create a Ruby file (let's call it asciidoctor-pdf-extensions.rb) and populate it with the following content.
module AsciidoctorPdfExtensions
  def convert_paragraph node
    puts "Custom convert_paragraph called"
    # delegate to default implementation
    super
  end
end

Asciidoctor::Pdf::Converter.prepend AsciidoctorPdfExtensions

You then load this file using the -r flag (which also works in the Gradle build).
asciidoctor-pdf -f ./asciidoctor-pdf-extensions.rb sample.adoc

The prepend method (introduced in Ruby 2) is how you inject a class into the class hierarchy in Ruby (instead of having to extend the base class, though with a bit more work you could do it that way too). Using extend, your module inherits from the built-in converter, which means you can call super to delegate to the built-in method.
You can override any of the methods in the Asciidoctor PDF converter. Several of the methods are split out to make it easy to handle things like the start of a chapter (for example, see https://github.com/asciidoctor/asciidoctor-pdf/blob/master/lib/asciidoctor-pdf/converter.rb#L1630).
I suspect that in order to get the customizations you want for the InfoQ mini-book format, you'll need to take this approach (at least until the theme file matures a bit more).

Is it possible to customize section headings?

At the moment, only by extending the converter (as I described above). However, being able to customize the chapter titles is something that is planned.

Using :toc: macro and toc::[] doesn't seem to work.

Correct. It is not currently possible to move the TOC unless you extend the converter. Even then, moving the TOC is very complicated because Prawn does not support transactional write operations. Being able to move the TOC, however, is planned. See #278.

Is it possible to make the page after the 1st page into the colophon?

This may be something we want to do generally. When Asciidoctor PDF was created, the colophon and dedication were not considered (as Asciidoctor PDF was originally created with magazine articles in mind). We want these pages to be placed in the standard position for books. If that is before the TOC, then yes, we want to support that. It's probably best to file a separate issue for that.
I do want address the title of the issue before a close out. Positioning text to the bottom of the page is surprisingly difficult in Prawn. However, I do know how it can be done using foundation infrastructure in Asciidoctor PDF. So the open question that remains is How do we define this setting in the theme file?

I tried creating sciidoctor-pdf-extensions.rb in src/main/ruby:
module AsciidoctorPdfExtensions
  def convert_paragraph node
    puts "Custom convert_paragraph called"
    # delegate to default implementation
    super
  end

  def layout_chapter_title node, title
    puts "title" + title
    # delegate to default implementation
    super
  end
end

Asciidoctor::Pdf::Converter.prepend AsciidoctorPdfExtensions

Then I added a requires method to point to this file in build.gradle:
asciidoctor {
    backends 'html5', 'pdf', 'epub3'
    attributes 'sourcedir': '../../../main/webapp',
            'endpoint-url': 'http://www.jhipster-book.com/',
            'source-highlighter': 'coderay',
            'imagesdir': './images',
             toc: 'left',
             icons: 'font',
             linkattrs: true,
             encoding: 'utf-8',
            'setanchors': 'true',
            'idprefix': '',
            'idseparator': '-',
            'docinfo1': 'true'
    requires './src/main/ruby/asciidoctor-pdf-extensions.rb'
}

Unfortunately, I get an error when I run ./gradlew asciidoctor:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':asciidoctor'.
> (NameError) uninitialized constant Asciidoctor::Pdf

It seems I was configuring the extension loading wrong. I changed it to:
asciidoctor {
    backends 'html5', 'pdf', 'epub3'
    ...
    extensions new File('./src/main/ruby/asciidoctor-pdf-extensions.rb')
}

However, when I run ./gradlew asciidoctor it fails to parse the file:
* What went wrong:
Execution failed for task ':asciidoctor'.
> startup failed:
  asciidoctor-pdf-extensions.rb: 2: unexpected token: convert_paragraph @ line 2, column 7.
       def convert_paragraph node
           ^

  1 error

Actually, you had it right the first time. You want to use requires. Extensions are something else.
requires file('src/main/ruby/asciidoctor-pdf-extensions.rb')

The reason for uninitialized constant Asciidoctor::Pdf has to do with the load order when using AsciidoctorJ. You simply need to add an explicit require statement to the top of the Ruby file.
require 'asciidoctor-pdf' unless defined? ::Asciidoctor::Pdf

In order to use the prepend feature in Ruby, you'll need to use JRuby 9000, which I recommend anyway. It's a drop-in replacement and it brings compatibility up to Ruby 2.0.
You add it to the buildscript section of your build.gradle and it will automatically get used:
buildscript {
  ...
  dependencies {
    ...
    classpath 'org.jruby:jruby-complete:<IP_ADDRESS>'
  }
}

I tested those three changes together and was able to demonstrate that the custom code is called.

I've confirmed this works - thanks Dan!
What's the best way to add #278? Can I do it by overriding methods in asciidoctor-pdf-extensions.rb or would I be better off cloning https://github.com/hmflash/asciidoctor-pdf/tree/toc_placement?

You could go either way on that. You'll quickly learn, though, that the toc has special needs with regard to page counting, so the hard part is actually reasoning about how to calculate all the offsets.
If you did it through extension, what I would recommend is turning off the toc attribute, then intercepting the convert_section block, detect the first chapter and then insert the toc by reusing the logic currently in the document method.
The trickiest part is that the toc has to be converted twice, the first time as a dry run. That's because you don't know the page numbers at the start of the book, but you need to reserve pages to put the toc. Then you need to go back and insert the toc after converting the rest of the book.
In order to make this more flexible for extension, the first thing I'd like to do upstream is to create a method named prepare_toc and a method called insert_toc. That way, these two methods can be easily reused to move the toc around. Naturally, prepare_toc would need to store its result in an instance variable so that insert_toc can access it in any scope. I'll go ahead and file an issue for that change as it will make extension a lot simpler.

See #340

Thanks for creating #340. I'm concentrating on the low-hanging fruit now:

Centering the dedication and acknowledgement titles (done)
Adding a 2-3 inch margin above the dedication and acknowledgement titles (can't figure out how to set the top margin)
Adding section pages and making titles fancy (section pages done, haven't tried font manipulation)

Here's what I have so far:
require 'asciidoctor-pdf' unless defined? ::Asciidoctor::Pdf

module AsciidoctorPdfExtensions
  def layout_title_page doc
      # no title page
  end

  def layout_chapter_title node, title
    if (node.id == "dedication" || node.id == "acknowledgements")
      # todo: Add underline of forward-slashes under title
      # todo: Add margin below title, before text
      layout_heading title, align: :center, margin_top: 3
    elsif node.id.include? "mini-book" # colophon
      # todo: make title font-size same as text, align with bottom of page
      layout_heading title, margin: 5
    elsif node.id.include? "jhipster" #chapters
      puts node.id
      # todo: add 'Part One|Two|Three' to title and make font name, size and colors match InfoQ
      layout_heading title, align: :right, margin_top: 3
      start_new_page
    else
       # delegate to default implementation
       super
    end
  end
end

A few questions:

Is it possible to get the node type? For example, 'colophon', 'dedication', etc. rather than just parsing the ids or titles? If not, is it possible to get the file name (from an include)?
How do I add a margin before/after a title?
Is it possible to manipulate the formatting of the title to change the font size and color?
How do I add a line break if I manipulate the title's text?

I was really busy over the weekend. I'll reply to your questions asap.

I figured out most things. Here's some answers to my own questions:

node.sectname should do it
move_down is your friend
yes
call layout_heading for each line

Here's what I've come up with. It does most everything I need.
require 'asciidoctor-pdf' unless defined? ::Asciidoctor::Pdf

module AsciidoctorPdfExtensions

  def layout_title_page doc
      # no title page
  end

  def layout_chapter_title node, title
    if node.id == "dedication" || node.id == "acknowledgements"
      layout_heading_custom title, align: :center
    elsif node.id.include? "mini-book" # colophon
      move_down 470
      layout_heading title, size: @theme.base_font_size
    elsif node.id.include? "jhipster" #chapters
      puts 'Processing ' + node.id + '...'
      move_down 120
      # set Akkurat font for all custom headings
      font 'Akkurat'
      layout_heading 'PART', align: :right, size: 120, color: [91, 54, 8, 13], style: :normal
      move_up 40

      part_number = "ONE"
      if node.id.include? "ui-components"
        part_number = "TWO"
      elsif node.id.include? "api"
        part_number = "THREE"
      end

      layout_heading part_number, align: :right, size: 120, color: [42, 1, 83, 1], style: :bold
      layout_heading title, align: :right, color: [42, 1, 83, 1], style: :normal, size: 30
      move_up 30
      start_new_page
    else
       # delegate to default implementation
       super
    end
  end

  def layout_heading_custom string, opts = {}
      move_down 100
      typeset_text string, calc_line_metrics((opts.delete :line_height) || @theme.heading_line_height), {
          inline_format: true
      }.merge(opts)
      move_up 5
      $i = 0
      underline = ''
      while $i < string.length do
          if string == 'Dedication'
            underline += '/////'
          else
            underline += '//////'
          end
          $i += 1
      end
      if string == 'Dedication'
          underline += '////'
      end
      typeset_text underline, calc_line_metrics((opts.delete :line_height) || @theme.heading_line_height), {
            inline_format: true, color: 'B0B0B0', size: 8, style: :italic
      }.merge(opts)
      move_down 20
  end

end

Asciidoctor::Pdf::Converter.prepend AsciidoctorPdfExtensions

A couple issues I'd still like to solve:

Replace title page with colophon. I was able to do this in layout_title_page, but then the page count was off and the colophon still printed.
Move the table of contents so it starts on a particular page (after the colophon, dedication and acknowledgements).

I'm glad to hear it!

Correct. sectname is the name of the property that holds the special "block name" of a section. You know that the block name is something other than "section" when the property special returns true. For regular sections, the sectname is "sect" + the level (e.g., sect1). It's not possible to get the file name of an include from the AST as that information is gone by the time the parser runs.

move_down is the low-level primitive in Prawn for moving down (and move_up goes back up). As much as possible we try to abstract over these primitives. In fact, there are lots of useful methods that Asciidoctor PDF adds that you can find here: https://github.com/asciidoctor/asciidoctor-pdf/blob/master/lib/asciidoctor-pdf/prawn_ext/extensions.rb. Vertically aligning on a page is a lot of work because you have to know the height of what you are rendering, then calculate the distance from the top of the page to move down. But it is possible.

Yep, the theme. See https://github.com/asciidoctor/asciidoctor-pdf/blob/master/docs/theming-guide.adoc#heading

Actually, a better way to do this is to add an endline to the text, then pass the normalize: false option to layout_heading. But default, Asciidoctor PDF folds endlines. The normalize: false disables this behavior.

Move the table of contents so it starts on a particular page (after the colophon, dedication and acknowledgements).

As you may have figured out by now, the biggest complexity is counting pages. (I really dread working on anything that touches the page counts). Once we start moving things around, it gets complicated really quick. I can see how it can be solved, but it's going to be a task. Related issues: #340 and #233.

Replace title page with colophon.

In general, I think we just need to start with some special handling for the colophon. That should allow extension points to emerge and get refined. I don't think we have an issue for that yet.

One of the biggest challenges with making a PDF converter is that PDF just doesn't have a lot of the styling capabilities that we are used to from the web (especially CSS3). The key is going to be to continue building abstractions that let us work at this level. We've made important strides so far. If you try to use Prawn standalone you will quickly see how many building blocks of in Asciidoctor PDF. But fine-tuning PDF creation (while keeping a separation of content and presentation) remains a challenge.
What's important about Asciidoctor PDF over the DocBook toolchain is that you can use programming to solve the issue rather than getting lost in XSLT stylesheets.

Btw, you can get rid of the title page simply by passing the attribute notitle to Asciidoctor PDF:
-a notitle

You can also add it to your document as follows:
= Document Title
Author Name
ifdef::backend-pdf[:notitle:]

content


[BUG] Crosseling on product page

Describe the bug
cross selling products are not  displayed on product page

because assigned products are not comming from backend

Provide reproduction steps, example:

i've assign couple products in panel admin and made it active

Additional context
pwa 0.7.1
shopware <IP_ADDRESS>

thanks @lbebnowicz, we noticed missing assignedProducts within crossSellings as well, and we are going to implement it asynchronously, fetching cross sellings from another endpoint (already available).


feat: add builder for container exec

Add new builder: container exec #221

GPG commit signature is missing.

I ll raise another PR with signature added to the commit

You shouldn't have to close this one. Just rebase the commits.


A bug in eval script

Thank you for your impressive reimplementation. There should be --use_viewdir in eval script so as to output normal images.

It depends on your training configuration. If you use --use_viewdir to train then you need to add --use_viewdir in eval too. But both are set to false by default.


Fix coverage

Fixes #88.

This change is

Current coverage is 99.94% (diff: 100%)

Merging #89 into master will increase coverage by <.01%

@@             master        #89   diff @@
==========================================
  Files            27         27
  Lines          1912       1914     +2
  Methods           0          0
  Messages          0          0
  Branches        172        173     +1
==========================================
+ Hits           1911       1913     +2
  Misses            1          1
  Partials          0          0

Powered by Codecov. Last update 5976eac...6d1b840

Reviewed 2 of 2 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add interactive chat plugin

v1.0 adds the option for users to include clickable wiki links to their chat messages.

The created hitboxWidget should be explicitly deleted on plugin shutdown. (see Widget#deleteAllChildren())

The hitbox parent widget is unfortunately WidgetInfo.CHATBOX_MESSAGE_LINES, meaning deleteAllChildren puts the chatbox in a state where I'd need to reinitialise it. Instead, I borrowed an approach I found in the Wiki plugin. I've tested it and it has the intended effect, please let me know if it doesn't satisfy the requirement.
https://github.com/runelite/runelite/blob/master/runelite-client/src/main/java/net/runelite/client/plugins/wiki/WikiPlugin.java#L127


Add getToken option

This PR adds a getToken option to allow overwriting the default authentication method.
If the option is not provided, the existing ./authentication import is used.
We're using a different endpoint than the default Strapi one to acquire an access token, so this extra configuration field would be appreciated!

I am closing this PR as we released a new version. Feel free to reopen it.


Map a particular disconnect situation triggered by a user logging off…

This commit adds detection of a disconnect resulting from a user logging off and maps it to a user log off code.  This was found with Windows XP, so maybe there's little interest, but I'm not sure that it is limited to that.
Before the commit...
# freerdp /v:winxphost
...
<log off via start menu>
[09:17:53:773] [16826:16827] [INFO][com.freerdp.core] - ERRINFO_RPC_INITIATED_DISCONNECT (0x00000001):The disconnection was initiated by an administrative tool on the server in another session.
[09:17:53:776] [16826:16827] [ERROR][com.freerdp.core.rdp] - rdp_recv_tpkt_pdu: rdp_recv_deactivate_all() fail
[09:17:53:776] [16826:16827] [ERROR][com.freerdp.core.transport] - transport_check_fds: transport->ReceiveCallback() - -1
# echo $?
1

After the commit...
# freerdp /v:winxphost
...
<log off via start menu>
[09:27:14:477] [18140:18141] [INFO][com.freerdp.core] - ERRINFO_RPC_INITIATED_DISCONNECT (0x00000001):The disconnection was initiated by an administrative tool on the server in another session.
[09:27:14:478] [18140:18141] [INFO][com.freerdp.core.rdp] - Error info says user did not initiate but MCS ultimatum sa they did; treat this as a user logoff
[09:27:14:478] [18140:18141] [INFO][com.freerdp.core] - ERRINFO_LOGOFF_BY_USER (0x0000000C):The disconnection was initiated by the user logging off their session on the server.
[09:27:14:479] [18140:18141] [ERROR][com.freerdp.core.rdp] - rdp_recv_tpkt_pdu: rdp_recv_deactivate_all() fail
[09:27:14:479] [18140:18141] [ERROR][com.freerdp.core.transport] - transport_check_fds: transport->ReceiveCallback() - -1
# echo $?
12

@freerdp-bot test

@r-barnett Might be a good idea to remember the disconnect provider ulimatum reason in the rdpRdp struct though to handle it at client exit.

Maybe I got it now.
I should add a field to struct rdp_rdp (or better struct rdp_context?) to hold the reason.  Remove the new test I added to rdp_read_header(), but store the reason in the new field instead.  Then I should check the reason in the exit handling code in xf_client_thread() to determine the appropriate exit code.
Correct?

@akallabeth is this what you had in mind?

Getting closer perhaps.

@freerdp-bot test

@freerdp-bot test

@freerdp-bot test


workflows should be able to log into parsl.log

I'd like my workflow to be able to log its progress into the rundir parsl.log so that that information appears together.

I mean the python code that is making calls to the parsl library code.


Akamai EdgeWorkers support

I'd love to see support for Akamai EdgeWorkers. I think it might be a stretch considering the difference in supported APIs, but it would be great to have a tool like this for building with. Maybe we can keep this issue as a future feature, or feel free to close it.

Hi @AustinGil !
You are right, the Akamai EdgeWorkers API looks much different from Web Standard API.
I think it's IMPOSSIBLE :(
I'll close this issue, but a suggestion is welcome. Thanks!


Will it work worldwide?

Hello, will i be able to send sms worldwide?

Hi! I'm not positive, but I think so. I didn't have any means or need to test that when I wrote this project.

ok, thanks


docs: add test example for react-testing-library

react-testing-library is increasing in popularity so it makes sense to include it alongside the enzyme examples on the Testing React Apps page.
Anyone wanna take that on? https://github.com/facebook/jest/edit/master/docs/TutorialReact.md

Good idea, go for it!

That sounds like a good start @terrencewwong!


align systemd dependency from crowbar-init to apache

A change to the crowbar service file was made in crowbar-core:
https://github.com/crowbar/crowbar-core/pull/1317/commits/fbd1b480bd7a821844476bb8b9195458c27b0afb
It fixes a dependency issue.
This PR is to make the crowbar-init service file consistent with the
one from crowbar-core in terms of dependencies.
This is porting this PR https://github.com/crowbar/crowbar-init/pull/80
to the master branch for cloud 8.

@nicolasbock This one looks good, but the backport needs the cherry-pick -x.

@nicolasbock this is now the main change (even though the branch name is now misleading - sorry) and the backport to stable/4.0 is "cherry-pick -x"ed.


SSE2 and SSE4.2 builds use AVX2 instructions

While trying to help an acquaintance get VJolt working on their TF2 server (which is hosted on a machine using an Intel Xeon E5-2690 v2, which doesn't support AVX2) I noticed that the binaries output by the SSE2 and SSE4.2 projects use AVX2 instructions, as shown by elfx86exts in the attachment below.
I believe this is caused by the joltphysics_avx2.vpc script setting the VOLT_AVX2 conditional and not unsetting it, causing the SSE2 and SSE4.2 projects to also use the AVX2-specific settings defined in jolyphysics_settings.vpc

It doesn't seem like any of the forks have fixed this in their releases or "action builds" yet. Is this something easy to do for a novice or can someone explain the basics?


Migrate to JAX-RS from raw servlet

@wyukawa Could you enable Travis CI integration in this repository when you have time?

enabled Travis CI integration in this repository like https://github.com/yanagishima/yanagishima/pull/106/checks?check_run_id=253262182

@ebyhr
I merged and deployed but it didn't work.
http://localhost:8080 is 404
It works after I commented out the followng line
servletContextHandler.addServlet(new ServletHolder(new ServletContainer(new YanagishimaResourceConfig(config))), "/*");

So I reverted in https://github.com/yanagishima/yanagishima/commit/5e2f0c28b751166d1d49859a8624e10b68bda3b8
Could you please check?

Found the root cause. The default servlet path was overwritten with the above code. Once we swap the order as below, the endpoint will return the expected result.
        servletContextHandler.addServlet(new ServletHolder(new ServletContainer(new YanagishimaResourceConfig(config))), "/*");
        servletContextHandler.addServlet(DefaultServlet.class, "/");


Add table of contents to README

Aims to help more easily navigate the README so that folks new to it can easily jump to sections relevant to them. This may also help determine which sections would be good candidates to split out into separate Markdown files.
(I also fixed a few typos that showed up in my editor as I was skimming the README.)

:+1: Thank you!


all.bash fails on master on Mac OS

What version of Go are you using (go version)?

$ go version
go version go1.13.5 darwin/amd64

Does this issue reproduce with the latest release?
It happens on master.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/mholland/Library/Caches/go-build"
GOENV="/Users/mholland/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/mholland/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13.5/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.5/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/mholland/git/go/src/cmd/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/ft/dg1pz2zj0mb9bqbhp3r1q77r0000gq/T/go-build705702634=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Ran ./all.bash in the go/src directory.
The test 'TestIssue33979' fails:

ok  	cmd/internal/src	0.009s
ok  	cmd/internal/test2json	0.083s
--- FAIL: TestIssue33979 (0.51s)
    link_test.go:250: got:
        main(__TEXT/__text): relocation target undefined not defined
        want:
        main(.text): relocation target undefined not defined
FAIL
FAIL	cmd/link	4.131s
ok  	cmd/link/internal/ld	7.899s
ok  	cmd/link/internal/sym	0.008s

The problem seems to be introduced in commit 8adc1e00aa1a92a85b9d6f3526419d49dd7859dd . For the previous commit all the tests pass.
What did you expect to see?
All tests to pass
What did you see instead?
See above.

Duplicate of https://github.com/golang/go/issues/36389.


Templates exchanged between concurrent users

Hi,
We have a case where we send emails to users using razor engine in a web service. Its so happening that two users who did some processing at the exact same time got each others email message. We read comments on your blogs in which you mention the design issue with razor engine.
"The key change that prompted this was the overwhelming need to run RazorEngine in parallel/multi-threaded scenarios. Because of how RazorEngine v2 was laid out under the hood, there were a number of mistakes that were made in its design, and thread-safety was a big one. Currently, attempting to run RazorEngine v2.1 in a multi-threaded way nearly always ends badly, you’ll get a hideous amount of mangled template output and/or failed compiles. This has prompted me to redesign the API to support multi-threaded scenarios natively."
We cannot wait for the release of v3. Is there a neat stop-gap solution that you suggest to overcome this issue?
Thanks,
Shez

And when is V3 going to be released?

Hi,
Any help?
thanks,

Give maybe a try to the current V3 branch....

Sorry for the late reply on this.
Version 2.1 is laid out quite differently under the hood, and unfortunately doesn't lend itself to being run cleanly in multi-threaded/parallel scenarios. I would want to try and hack it to work in this way, as this is what had prompted the significant redesign in v3.
v3.0.3beta is currently on Nuget, I'd recommend downloading that and giving that a go. I haven't had a chance to finish and publish the migration guide (v2 -> v3), but I'll endeavour to get this up over the next few weeks. If you have any questions, feel free to email me (matt [at] fidelitydesign (dot) net), or raise an issue here.

Thanks Antaris. If I understand well you are going to try to hack it to make it support multi-threading?
For v3.0.3 is it stable enough to be used in production?
Thanks,
Shez

Sorry, that was my mistake. I wouldn't want to hack it, there is too much to change/test. v3.0.3 has all the previous features of v2, but there are some differences:

Expression results are automatically HTML encoded - you'd need to take this into consideration.
Templates are no longer cached using their last value, we now cache the compiled type and hashcode of the last source value instead.

These changes may affect your code base. Currently I am not seeing too many issues come back for v3.0.3, which would suggest that it is perhaps more stable than I thought, but that might also be down to lack of exposure - hard one to weigh up.

No action on this case, so closing.


feat: Upgrade terraform-provider-equinix to v2.8.0

This PR was generated via $ upgrade-provider equinix/pulumi-equinix --kind=provider --target-bridge-version=latest --pr-title-prefix=feat:  --java-version=0.16.1.

Updating Java Gen version to 0.16.1.
Upgrading terraform-provider-equinix from 2.5.0  to 2.8.0.

Converted to draft because this PR only exists to test/validate #219.  That PR updates our workflows to allow us to specify a Go version instead of the default 1.21.something.  This PR demonstrates that the changes in #219 work, but the Go version specified in examples doesn't match the one in provider/go.mod.

Closing. Out of date because of #225


Analysis crashes with 'too many open files'

Summary
When running stryker on my day job project, I get a "too many open files error"
This is a project using Flow (version 0.65). I have tested the config on a small hello world project, it worked fine.
Stryker config
module.exports = function(config){
  config.set({
      logLevel: 'debug',
      mutator: "javascript",
      packageManager: "yarn",
      reporters: ["html", "clear-text", "progress"],
      testRunner: "jest",
      transpilers: [],
      coverageAnalysis: "off",
      files: [
        "./subproject/**/*.*",
      ],
      mutate: [
        "./subproject/**/*.js",
        '!**/*.spec.js',
        '!**/*.test.js',
      ],
      jest: {
        config: require('./.jest.stryker.json'),
      },
  });
}

Stryker environment
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>

"jest": "^23.6.0",

Your Environment

software
version(s)

node
8.10.0

yarn
1.5.1

Operating System

MacOs 10.14

Add stryker.log
stryker.too_many_files.log

Hi Alex! Thanks for letting us know. Could you test this using<EMAIL_ADDRESS>With 0.30.0 we changed the way we do file management and I'm wondering if we broke it with that change or if it was already broken.

Hello again
With<EMAIL_ADDRESS>I get a different error  message:
23:29:20 (15390) INFO Stryker 2237 Mutant(s) generated
23:29:21 (15390) INFO SandboxPool Creating 8 test runners (based on CPU count)
23:29:21 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox1403658
23:29:21 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox5909173
23:29:22 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox1426186
23:29:23 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox4515750
23:29:23 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox7293265
23:29:24 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox5967824
23:29:24 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox9602536
23:29:24 (15390) DEBUG Sandbox Creating a sandbox for files in /Users/avictoor/work/fluo/.stryker-tmp/sandbox2960436
23:29:35 (15390) DEBUG Sandbox Creating test runner 0 using settings {port: 9234}
23:29:35 (15390) DEBUG ChildProcessProxy Starting /Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/ChildProcessTestRunnerWorker.js in child process undefined
23:29:35 (15390) ERROR StrykerCli an error occurred TypeError: this.worker.send is not a function
    at ChildProcessProxy.send (/Users/avictoor/work/fluo/node_modules/stryker/src/child-proxy/ChildProcessProxy.js:53:21)
    at new ChildProcessProxy (/Users/avictoor/work/fluo/node_modules/stryker/src/child-proxy/ChildProcessProxy.js:25:14)
    at Function.ChildProcessProxy.create (/Users/avictoor/work/fluo/node_modules/stryker/src/child-proxy/ChildProcessProxy.js:50:16)
    at new ChildProcessTestRunnerDecorator (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/ChildProcessTestRunnerDecorator.js:15:51)
    at TimeoutDecorator.testRunnerProducer (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/ResilientTestRunnerFactory.js:14:76)
    at TimeoutDecorator.TestRunnerDecorator.createInnerRunner (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/TestRunnerDecorator.js:17:33)
    at TimeoutDecorator.TestRunnerDecorator (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/TestRunnerDecorator.js:6:14)
    at new TimeoutDecorator (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/TimeoutDecorator.js:14:47)
    at RetryDecorator.testRunnerProducer (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/ResilientTestRunnerFactory.js:14:24)
    at RetryDecorator.TestRunnerDecorator.createInnerRunner (/Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/TestRunnerDecorator.js:17:33)

Let me know if I should bring some other changes in my config

Unfortunately I'm not able to reproduce this locally.
Could you try downgrading<EMAIL_ADDRESS>to<EMAIL_ADDRESS>I just noticed that we also hit that package with our file system change for any-promise

I have downgraded stryker-api but I get the exact same error :|

This is a strange problem,

23:29:35 (15390) DEBUG ChildProcessProxy Starting /Users/avictoor/work/fluo/node_modules/stryker/src/test-runner/ChildProcessTestRunnerWorker.js in child process undefined

The undefined part in this log message should be filled with the child processes process id (PID). No wonder that send isn't defined on a child process that isn't spawned.
@alexvictoor
Just for debugging sake, could you try to reduce the amount of concurrent test runners? You can do this by setting maxConcurrentTestRunners to 1 in your stryker.conf.js file.

@nicojs the crash does not happen when maxConcurrentTestRunners is set to 1 (or even 2)...
Looks like a concurrent issue ;)

Hmm that seems like an old fashion race condition.
This is the piece of compiled javascript code that gives the error:
        this.worker = child_process_1.fork(require.resolve('./ChildProcessProxyWorker'), [messageProtocol_1.autoStart], { silent: true, execArgv: [] });
        this.initTask = new Task_1.Task();
        this.log.debug('Starting %s in child process %s', requirePath, this.worker.pid);
        this.send({
            constructorArgs: constructorParams,
            kind: messageProtocol_1.WorkerMessageKind.Init,
            loggingContext: loggingContext,
            plugins: plugins,
            requirePath: requirePath,
            workingDirectory: workingDirectory
        });

The send method is here is a pass through to worker.send. If send crashes with the given error, that would mean the child_process.fork failed somehow. I didn't know that could happen. We should figure out why and harden Stryker for this problem.
Thanks @alexvictoor for all your work so far. We'll continue to investigate.

As part of the rewriting of #1514 I've fixed this issue in #2498  (I hope).


Update ancientsinglegenome-hostassociated_samples.tsv

adding Eaton samples
Pull Request
This PR is for a

[x] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[ ] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[x] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s):
This is to close #1049
Checklist

[x] Publication is published (preprints currently not accepted)?
[x] Checked the publication is not already in the database?
[x] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[x] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[x] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[x] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?
[x] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

Correction
This PR is for

[ ] ancientmetagenome-hostassociated
[ ] ancientsinglegenome-hostassociated
[ ] ancientmetagenome-environmental

Reference
This pull request is to correct samples from the following publication(s):
This is to close
Description
The issue is:

Checklist

[ ] Checked the corrected entries follow conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)
[ ] Changelog is updated to include the publication under 'Changed'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?

Sorry for the title.

Tirup coordinates precise now, but others still need extra good check.

For the coordinates - Hågerup , Faaborg-Midtfyn Municipality is a very small village, so I took a random point in the centre.  Faldborg, Viborg Municipality was named as urban cemetery so I assumed the Viborg cemetery as the one, Ribe Gråbrødre and Ribe Lindegården in Esbjerg Municipality are also urban cemeteries so I searched for cemeteries within the city. There were two of them I could find, but the coordinates may be the other way around for those two. Sejet is also a very small settlement, so I just chose a point in the centre (close to the community building, I think). Tirup was explained by Liam Lanigan on Slack and  Ole Wormsgade, Horsens was probably just the exact point that Goodge maps spouted out, because it must've been 5th hour of me trying to find cemeteries in Denmark. Maybe after C14 paper we should do something similar with the location reporting -.-

For the coordinates - Hågerup , Faaborg-Midtfyn Municipality is a very small village, so I took a random point in the centre. Faldborg, Viborg Municipality was named as urban cemetery so I assumed the Viborg cemetery as the one, Ribe Gråbrødre and Ribe Lindegården in Esbjerg Municipality are also urban cemeteries so I searched for cemeteries within the city. There were two of them I could find, but the coordinates may be the other way around for those two. Sejet is also a very small settlement, so I just chose a point in the centre (close to the community building, I think). Tirup was explained by Liam Lanigan on Slack and Ole Wormsgade, Horsens was probably just the exact point that Goodge maps spouted out, because it must've been 5th hour of me trying to find cemeteries in Denmark. Maybe after C14 paper we should do something similar with the location reporting -.-

By "slightly to the west" do you mean the Klosterkirken Horsens (55.860, 9.855)?

Can't make suggestions properly due to broken laptop (sorry)
samples

Could you confirm how you got the Lat:Lons to be so precise? They all look roughly OK/would make sense -  but I don't see where the precise dates come from the paper (it's fine if you got the original Arch. report). The only major one that seems slightly off is for when I search 'Ole Wormsgade, Horsens' on google I get  result slightly to the west of your Lat:Lon: 55.860, 9.858.
Gr GCx15; date is off, it is 200y younger, A146x3011 seems to be rather young? Maybe double check all the dates.

Everything else looks good for samples
libraries

Polymerase is from Kircher et al 2012, and the first round of indexing amplification is AmpliTaq Gold

Otherwise 👍

For the libraries, yes, I should've noticed the citation, I have been working on C14 for so long, I have forgotten the tricks


Release Google.Cloud.ApiHub.V1 version 1.0.0-beta01

Changes in this release:
Initial release.
PiperOrigin-RevId: 658810011

Created release for Google.Cloud.ApiHub.V1-1.0.0-beta01


Adjust stepper element to know what page to load

Stepper element does not load buttons correctly...they have the wrong names and do not use the correct routes.

The stepper element is for easier navigation between pages in a simulation. Once a student has visited a page, the page is accessible in the sidebar and when the specific page is clicked in the sidebar, the simulation goes to that page. A student should not be able to access a page in the stepper if they have not already visited it yet.


[WIP] Forums

A brief description of the purpose of the changes contained in this PR.
Forums..!
Issues this PR resolves
Relates to #352
A checklist for hand testing

[ ] Can make a thread
[ ] Get notified as admin about new threads
[ ] Get notified of new posts in a thread you create
[ ] Get notified about posts in a thread you're participating in
[ ] Can CRUD posts

Checklist

[ ] Code review by me
[ ] Hand tested by me
[ ] I'm proud of my work
[ ] Code review by reviewer
[ ] Hand tested by reviewer
[ ] Ready to merge

Current progress: old style base in, used bootstrap for templates so that frontend will need to be completed


Add LazyBlob helper class to lazy load file from filesystem

Description
This PR add a new helper class LazyBlob which aim to prevent loading huge file into RAM.
This new class can be used instead of a Blob with the commit function.
Fix https://github.com/huggingface/huggingface.js/issues/68
const lazyBlob = await LazyBlob.create("./package.json");

await commit({
  repo,
  title: "Some commit",
  credentials: {
    accessToken: "<access token>",
  },
  operations: [
    {
      operation: "addOrUpdate",
      content: lazyBlob,
      path: "package.json",
    },
  ],
});

@coyotte508 I can get ride of init with the private constructor and static create method as you suggest but for the dispose the only option would be to open the file when reading and slice and then close it.
Here it's DX vs performances so I will let you choose :+1:

@coyotte508 I can get ride of init with the private constructor and static create method as you suggest but for the dispose the only option would be to open the file when reading and slice and then close it.
Here it's DX vs performances so I will let you choose :+1:

The most calls will be three I think, on large files:

a .slice(0, 512) to get a sample to send to /preupload
a .stream() to compute the Sha
Then whatever fetch will call (probably .stream())

On smaller files, it will be two calls: one .slice(0, 512) and one .arrayBuffer().
Given that, I'd rather prioritize DX on this one :)

Given that, I'd rather prioritize DX on this one ! The overhead should be low. Maybe an optimization would be to directly load small files in memory, if we really care about perf.

I agree with that :+1:  Also, better have more syscall to open/close than unused file descriptor :see_no_evil:

For the TS Error: maybe as ReturnType<Blob["stream"]>

@coyotte508 What about the documentation? If I understood well it's in the same repo, I can give it a try

I intend to keep it internal.
A file with require("node:fs") is directly exposed in the exports, it will cause trouble for the use in browsers, at least for the CJS bundle. See how sha256-node is currently dynamically imported and only outside the browser.
So I think maybe a smart way to expose it is to allow passing a URL as an alternative to a Blob - see https://www.geeksforgeeks.org/node-js-url-pathtofileurl-api/ urls and files paths can be equivalent.
It also opens the door for supporting web urls later on as well. (using Range header, or HEAD calls appropriately to get a slice / the size)

Yes I saw, I'm familiar with this kind of dual behavior when it's needed to provide a SDK for both node and the browser (See Kuzzle Javascript SDK)
If I understand well, you might want to allows a string representing a path to a local file for ContentSource if it's used in Node.js ?
await commit({
  repo,
  title: "Some commit",
  credentials: {
    accessToken: TEST_ACCESS_TOKEN,
  },
  operations: [
    {
      operation: "addOrUpdate",
      // 1) Only a path to a local file, if it's used from Node.js, a LazyBlob will be used
      path: "test.txt",
    },
    {
      operation: "addOrUpdate",
      // 2) URL to a local file, if it's used from Node.js, a LazyBlob will be used
      path: "file://test.txt",
    },
  ],
});

If I understand well, you might want to allows a string representing a path to a local file for ContentSource if it's used in Node.js ?

When I was mentioning URL, it was literally :)
Basically, change:
type ContentSource = Blob;

To
type ContentSource = Blob | URL;

Ok understood :+1:
So if the URL is a local file (starting with file://) then the LazyBlob will be used, am I right?

Do you want me to do this feature, supporting URL as content, in a separate PR?

Do you want me to do this feature, supporting URL as content, in a separate PR?

Sure! Then maybe you can remove the export in this one and keep the LazyBlob private ? (I was going to do it later today before merging :grin: )


Address a perf issue with long snakes

TODO: fix a scaler issue with the tails (the body is overlapping the tail, not quite sure how to fix yet)

Sort-of addresses #24


fixed type error in tests

Description
tmpdir is not in fact a string
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in a2859cd3df5b4666c245527e31487021ce9510f7

Processing #6742.


remove caniuse attributes from the output

When Include Can I Use Panels is false (as it is for example on csswg WDs), caniuse attributes are kept in the output markup, which then cause the spec to fail validation when trying to publish with echidna.
https://labs.w3.org/echidna/api/status?id=8eefbfc8-9ac9-4950-bea9-6591b506b640
Even if the panels are not being generated, the caniuse attribute should never be present in the output.

I think it's fixed by commit acdedda0d46d221a6ee9c1a136bd7bc1b361bc87.

Yup, fixed but only pinged the PR, not this issue.


Fixed issue with non_secret storage of invitations.

Signed-off-by: fabienpe<EMAIL_ADDRESS>

Found additional errors, especially when UI is reloaded in browser.


More Shortcuts

I wish to see more shortcuts for
BE
partials 25%
partials 50%
partials 75%

i dont know how to implement it in this case, because there can be several trades open
maybe thats the reason this is not avaliable

you may just close this if this is the case

This is best solved with separate scripts.


Runtime Error: The size of tensor a (1100) must match the size of tensor b (695) at non-singleton dimension 0

Hey @DoodleJZ
I came across this error while running your parser. Could please look into this and fix this ?
Traceback (most recent call last):
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/main.py", line 746, in
main()
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/main.py", line 742, in main
args.callback(args)
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/main.py", line 672, in run_parse
syntree, _ = parser.parse_batch(subbatch_sentences)
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/Zparser.py", line 1364, in parse_batch
extra_content_annotations=extra_content_annotations)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/Zparser.py", line 822, in forward
res, current_attns = attn(res, batch_idxs)
File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 541, in call
result = self.forward(*input, **kwargs)
File "/content/drive/My Drive/Hd/DependencyParser/src_joint/Zparser.py", line 344, in forward
return self.layer_norm(outputs + residual), attns_padded
RuntimeError: The size of tensor a (1100) must match the size of tensor b (695) at non-singleton dimension 0

I encounter the same issue, @DoodleJZ @tanvidadu Anyone has fixed this problem?

The dimensions do not match between d_model and the sum of d_tag, d_word and d_char if you concatenate all the embedding may be. You can check each part of the embedding dimension to find the problem easily.

@DoodleJZ, I do not use d_tag and d_char  embedding~  I run this code with python 3.6 with pytorch 0.4.0, however,
The error is: for residual shape = (packed_len, d_model), however, the shape of outputs = (self.batch_size* self.max_len, d_model)? Mismatching comes from the first dimension, not the second dimension. I compress glove.6B.100.txt into glove.gz and I can not confirm oov: 18820 is NORMAL  for running test.sh.
Loading model from models/cwt.pt...
loading embedding: glove from data/glove.gz
oov: 18820
Reading dependency parsing data from data/ptb_test_3.3.0.sd
Loading test trees from data/23.auto.clean...
Loaded 2,416 test examples.
Parsing test sentences...
packed_len: 2501
sentences: 100
torch.Size([2501])
self.batch_size 100
self.max_len: 50
residual: torch.Size([2501, 1024])
v_padded: torch.Size([800, 50, 64])
outputs_padded: torch.Size([800, 50, 64])
outputs = outputs_padded[output_mask]: torch.Size([40000, 64])
d_v1: 32
outputs = self.combine_v(outputs): torch.Size([5000, 1024])
outputs = self.residual_dropout(outputs,batch_idxs): torch.Size([5000, 1024])
Traceback (most recent call last):
File "src_joint/main.py", line 746, in
main()
File "src_joint/main.py", line 742, in main
args.callback(args)
File "src_joint/main.py", line 577, in run_test
predicted, _, = parser.parse_batch(subbatch_sentences)
File "/home/LAB/wujs/word_pos/HPSG-Neural-Parser/src_joint/Zparser.py", line 1370, in parse_batch
extra_content_annotations=extra_content_annotations)
File "/home/LAB/wujs/software/pytorch-0.4/lib/python3.6/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/home/LAB/wujs/word_pos/HPSG-Neural-Parser/src_joint/Zparser.py", line 827, in forward
res, current_attns = attn(res, batch_idxs)
File "/home/LAB/wujs/software/pytorch-0.4/lib/python3.6/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/home/LAB/wujs/word_pos/HPSG-Neural-Parser/src_joint/Zparser.py", line 349, in forward
return self.layer_norm(outputs + residual), attns_padded
RuntimeError: The size of tensor a (5000) must match the size of tensor b (2501) at non-singleton dimension 0
srun: error: dell-gpu-32: task 0: Exited with exit code 1

Maybe you need to try pytorch >= 1.0.0, the error is output_mask whichl occurs when the version of pytorch is not matched.

@DoodleJZ  Sorry to bother you. As I'm not familiar with PyTorch so much.
Now, I just change torch_t.ByteTensor into torch_t.BoolTensor as follows. And now everything is perfect~
def pad_and_rearrange(....):
invalid_mask = torch_t.BoolTensor(mb_size, len_padded)._fill(True)

@DoodleJZ �Sorry to bother you. As I'm not familiar with PyTorch so much.
Now, I just change torch_t.ByteTensor into torch_t.BoolTensor as follows. And now everything is perfect~
def pad_and_rearrange(....):
invalid_mask = torch_t.BoolTensor(mb_size, len_padded)._fill(True)

it works perfectly!

Hi @wujsAct, @CoyoteLeo, @tanvidadu. I am afraid that I have the same problem:
I have two tensors that I want to add, one is a noise tensor of shape N x 1 x 64 x 64, and the other tensor has the same shape.
Now things work fine until the very last batch index, it seems, when the programs stops and complains with the following error message: RuntimeError: The size of tensor a (96) must match the size of tensor b (128) at non-singleton dimension 0.
Now, here is a bit of code:
for batch_idx, (real_images, targets) in enumerate(train_loader):
`# noise to add:
noise_disc = -torch.rand(size = (batch_size, 1, 64, 64))/5
noise_disc = noise_disc.to(device)
   real_images = real_images.to(device) # shape: (batch_size, 1, 64, 64)
   images_disc = (real_images + noise_disc)

   # move to device:
   images_disc = images_disc.to(device)`

Unfortunately, I don't really understand why this error occurs, and I would appreciate help a lot!
Merry Christmas. :-)


problem with load_trajectories function in experiments/utils.py

I don't understand the logic of having max_step-1 in line 51 of experiments/utils.py in the body of the function load_trajectories:
https://github.com/keiohta/tf2rl/blob/82d9eecda78e22021efa0821bf02429ac7827f4d/tf2rl/experiments/utils.py#L40
the line I am referring to is:
https://github.com/keiohta/tf2rl/blob/82d9eecda78e22021efa0821bf02429ac7827f4d/tf2rl/experiments/utils.py#L51
I think having actions[:max_steps-1] makes the actions one element shorter than the states one, whereas there is no need for that after you did an adjustment in lines 47 through 48.

@naji-s
Thank you for reporting.
For better discussion, could you open an issue based on your problem from the next time?
At first glance, I feel this issue is closely related with #144.
We would like to know  the true problem you encountered.
If you give us minimal reproducible example code, we can reduce investigation and we will provide fixed version sooner.
(FYI: If your motivation is "question", please use disscussions. We want to use this issue for development  bug tracker.)

I thought the logic of what I pointed out, namely a function argument that is never used in the body of the function is enough to report an issue. But here is the minimal example. Suppose you already ran DDPG on pendulum-v0. The following should return results of length 300:
expert_path_dir = 'results/20210804T173752.061185_SAC_'
from tf2rl.experiments.utils import restore_latest_n_traj
expert_trajs = restore_latest_n_traj(
    expert_path_dir, n_path=20, max_steps=300)```

Instead it will return a dictionary with arrays of length 1000, the original length of the trajectories (because max_step is never used in restore_latest_n_traj).

Now here is when it gets connected to #144. But that's for itself another issue with a different function and line.

I guess I commented under the wrong issue. The comment above  was for #144. Sorry about that. I think there is a need for a fix for https://github.com/keiohta/tf2rl/blob/82d9eecda78e22021efa0821bf02429ac7827f4d/tf2rl/experiments/utils.py#L51. I'll try to reproduce the error. But I think the max_steps-1 needs to be just max_steps.

@naji-s
Please, do not close until the problem is resolved.
My PR is still being reviewed.
Moreover, we would like you to check whether your problem is fixed or not after newer version will be released.


call error(ASSERT FAILED: ptr != NULL @ operator->)

when i am trying to initiate a call i am getting the bellow error.
Error(common.cc:67): jni/../../../third_party/libjingle/talk/base/scoped_ptr.h(76): ASSERT FAILED: ptr != NULL @ operator->
any help..

hello
any solution on the above ?
06-08 15:01:38.505: E/libjingle(15175): Error(common.cc:67): jni/../../../third_party/libjingle/talk/base/scoped_ptr.h(76): ASSERT FAILED: ptr != NULL @ operator->
06-08 15:01:38.505: A/libc(15175): Fatal signal 6 (SIGABRT) at 0x00003b47 (code=-6), thread 15738 (AsyncTask #5)
this is my LogCat Errors.
I'm having this Error when i have an Internet Disconnection .

No did not got any solution
On Mon, Jun 8, 2015 at 5:42 PM, AnthonyKoueik<EMAIL_ADDRESS>wrote:

hello
any solution on the above ?
06-08 15:01:38.505: E/libjingle(15175): Error(common.cc:67):
jni/../../../third_party/libjingle/talk/base/scoped_ptr.h(76): ASSERT
FAILED: ptr != NULL @ operator->
06-08 15:01:38.505: A/libc(15175): Fatal signal 6 (SIGABRT) at 0x00003b47
(code=-6), thread 15738 (AsyncTask #5
https://github.com/lukeweber/webrtc-jingle-client/issues/5)
this is my LogCat Errors.
I'm having this Error when i have an Internet Disconnection .
—
Reply to this email directly or view it on GitHub
https://github.com/lukeweber/webrtc-jingle-client/issues/104#issuecomment-109970918
.

Did you find another project ? or you worked it around using this one ?
I think that we need to do what LukeWeber did do, and build our own Libjingle-webrtc but i am afraid that i will take a lot of time .

yes worked around with the same.
try to compile the lib properly.
On Wed, Jun 10, 2015 at 12:08 PM, AnthonyKoueik<EMAIL_ADDRESS>wrote:

Did you find another project ? or you worked it around using this one ?
I think that we need to do what LukeWeber did do, and build our own
Libjingle-webrtc but i am afraid that i will take a lot of time .
—
Reply to this email directly or view it on GitHub
https://github.com/lukeweber/webrtc-jingle-client/issues/104#issuecomment-110614059
.


[TEST-121] Add IBC Middleware module (records)

Summary
We need to add an IBC middleware in order to properly update records. IBC middleware modules can wrap core IBC modules and other middlewares. Specifically, this adds a middleware stack to app.go with the following structure: records -> transfer. All ibc packets routed to the transfer module will first pass through records, where we can apply custom logic (record keeping) before passing messages to the underlying transfer module.
Note:

The middleware stack is added in app.go
The custom handler logic is added in ibc_module.go by implementing the IBCModule interface

Test plan
Run sh scripts/tests/1.sh && sh scripts/tests/2.sh, which triggers an IBC transfer from stakeibc (using the standard transfer module).
Next steps
Merge #46 into this PR, which contains the core record keeping logic
Check logs
stride-stride3-1  | 8:38AM ERR This is where we're going to add DepositRecords!

This looks good to me, mainly based on checking against the tutorial here!


Blocking UI on closing bracket deletion in medium size TS file

Describe the bug
Tokenization can freeze the UI for seconds in medium size TypeScript file.
To Reproduce

Open https://github.com/microsoft/vscode/blob/main/src/vs/editor/common/model/pieceTreeTextBuffer/pieceTreeBase.ts in Runestone (from App Store)
Delete closing bracket at line 35
Delete line break
Delete the last closing bracket at line 34
On iPhone 12 Pro above step can take a full second to finish

Expected behavior
Tokenization does not freeze the UI
Additional context
While this was tested against Runestone from App Store, I also did some profiling with the Example app and the blocking operations are syntax highlighting

Didn't dig into the code deep yet but a few ideas of how this can be improved

Tokenize asynchronously (in background thread if possible or limit how many lines are tokenized in each frame)
Early return when new line tokens are identical to the previous ones and skip following lines (not sure if this is valid with Tree Sitter)

Thanks for reporting this issue and for the detailed steps to reproduce @rebornix! 🙏
The issue should be fixed in #134. A detailed explanation of the fix is included in the description of the issue. As a note, Runestone does perform syntax highlighting in the background when scrolling the document but will intentionally perform syntax highlighting on the main thread when editing the document.
The reason for performing syntax highlighting on the main thread is to avoid the text being incorrectly colored immediately after editing. I've found it to be fine to perform this operation on the main thread as Runestone will rehighlight only a small amount of text when editing. Naturally, you found a case where this wasn't true but that should be fixed now 😊


Refactoring some code [WIP]

This PR attempts to refactor some of the code to get a little simpler end result. In particular:

I've removed the TypeParameter indirection, which turns out to help a bit with type stability instead of hurting. The main reason is that the compiler heuristics aren't to happy with deeper stacks of functions, so this simplification makes it slightly easier for that, as it would otherwise need a bunch of @inlines
I've joined some of the smaller files together into a single file, mostly for organizing my own thoughts. I'm happy to split this back up if this is preferred.
I've moved the StridedViews implementation to an extension
I've migrated the import statements to the top level
I've added export statements to the package
I've added several docstrings, but some are still missing (the AbstractArray interfaces)
I replaced @test_inferred with TestExtras.@constinferred, as it is doing the same thing and is less code to maintain
I removed the tests which check for type stability without wrapping. The rationale here is that by inputting Position(1) instead of actually inputting 1, this is in fact manually partially wrapping the function. In particular, this is actually still equally type stable as before, since it still depends on the value 1, however the specific form of the test hard-bakes that in. In that sense, if the function is type-stable with wrapping, then everything should be okay.

Questions

I'm a bit "scared" about the current default implementation of putting eltype and ndims as the first and second type parameter for every subtype of AbstractArray. I think I prefer getting a descriptive error that tells me an implementation is missing, rather than a different error because a wrong result was returned.
There is a bit of an inconsistency with nparameters and type_parameters, so we might consider renaming everything to parameters or type_parameters, but not a mix?
While playing around with type stability, I changed some signatures into Vararg signatures instead. I'm not entirely happy with how that turned out, and I'd be happy to try and change it back while keeping type stability. In particular, it might be good to rethink if we want both type_parameters(T, pos...) and type_parameter(T, pos). I tend to slightly favor copying something like axes or size, where a single argument yields a tuple of all parameters, a single position yields a single type, and any other syntax the user has to manually define themselves, typically via broadcasting.
On a similar note: set_type_parameters(T, params, positions...) could work similar to setindex, and we might consider a macro that enables @typeparameters T[1, ndims, eltype] = (Adjoint, 2, Float64). This would also somewhat fix the signature
The current implementation happily returns TypeVar in the case of unspecified types. (type_parameter(Vector, 1) will return T where {T<:Any}). I'm okay with this, since you can specialise on that, but I don't think that was previously the case.

TODO

[ ] docs
[ ] figure out various comments
[x] implement updated skeleton

I incorporated a round of comments, it would be nice if you could have a look again and tell me what you think, @mtfishman , @kmp5VT. If there aren't too many comments, I'll have a look at adding the docs as well.

Looks good, thanks a lot @lkdvos.
I think we could go ahead and make a few name simplifications, for example delete type_parameters and default_type_parameters in favor type_parameter and default_type_parameter plus broadcasting, deleting set_type_parameter in favor of set_type_parameters (Which can handle setting on or multiple parameters), etc.

I incorporated a round of comments, it would be nice if you could have a look again and tell me what you think, @mtfishman , @kmp5VT. If there aren't too many comments, I'll have a look at adding the docs as well.

I think this looks good, we can address naming issues in future PRs. Go ahead with the docs, thanks!

Ready to merge? Once it is merged we should bump to v0.2 and register.

Yes!


Monitor metric alert condition with special characters

Describe the bug
When creating a metric alert where the condition is filtering on a disk it's excluding the : charterer, so the filtering gets wrong.
To Reproduce
        az monitor metrics alert create `
            --name "Server $serverNameDisplay Disk E" `
            --scopes "/subscriptions/$subscriptionId/resourceGroups/$resourceGroup/providers/Microsoft.OperationalInsights/workspaces/xxxxx" `
            --description "Low disk" `
            --condition "min Average_Free Megabytes < 5000 where InstanceName includes E: and Computer includes $serverName" `
            --severity 0 `
            --evaluation-frequency 1m `
            --window-size 1m `
            --resource-group "$resourceGroup" `
            --subscription "$subscriptionId"

Expected behavior
The created metric is created with
{
	"name": "InstanceName",
	"operator": "Include",
	"values": [
		"E"
	]
}

But it should be like this:
{
	"name": "InstanceName",
	"operator": "Include",
	"values": [
		"E:"
	]
}

Environment summary

PowerShell
Windows
azure-cli version 2.0.68
Python (Windows) 3.6.6 (v3.6.6:4cf1f54eb7, Jun 27 2018, 02:47:15) [MSC v.1900 32 bit (Intel)]

Thanks for the report @pajzo. We'll get a fix into an upcoming release.

@pajzo Hello. $ will not be a special character in the condition. Right?

@MyronFanQiu I'm not sure what your question is - but $ is a reference to a PowerShell variable.
So yes there will be a $ in the condition like:
--condition "min Average_Free Megabytes < 5000 where InstanceName includes $diskName and Computer includes $serverName"

I got your point. $ is reference to a PowerShell variable. So the real condition will be a specific name like "MyServerName". The real REST call will not contain the character "$".

That's correct @MyronFanQiu  :-)
But the condition may contain a : like E: or C:

@pajzo of course. I'm working on it. 😄


Header ActionComponent events are not working

Describe the bug
I am trying to add animated drawer button in header component. Component is rendering properly but it's events are not working. If i use header left/right icons then onPress events works fine,
    ` import { Button, Header } from "../../components"
       <Header
      titleTx="screen1.title"
      LeftActionComponent={
        <Button
          text="drawer"
          onPress={() => {
            console.log("drawer clikced")
          }}
        ></Button>

        // <DrawerIconButton
        //   onPress={() => {
        //     toggleDrawer()
        //   }}
        //   {...{ open, progress }}
        // />
      }
      onLeftPress={() => {
        console.log("onLeftPress")
      }}
    />`

Ignite version
8.1.0
Additional info
System
  platform           win32
  arch               x64
  cpu                8 cores          Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
  directory          igniteMerge   C:\\src\igniteMerge

JavaScript (and globally-installed packages)
  node                 14.18.1      C:\Program Files\nodejs\node.EXE
  npm                  8.19.2       C:\Program Files\nodejs\npm.CMD
    @ionic/cli         6.18.1
    appcenter-cli      2.10.8
    bundler            0.8.0
    cordova            11.0.0
    cors-anywhere      0.4.4
    cost-of-modules    1.0.1
    git                0.1.5
    npm                8.19.2
    react-native-cli   2.0.1
    react-native       0.66.2
    udemy-dl           2.0.7
    yarn               1.22.17
  yarn                 1.22.17      C:\Users\..\AppData\Roaming\npm\yarn.CMD
  pnpm                 -            not installed
  expo                 46.0.15      managed

Ignite
  ignite-cli         8.1.0        C:\Users\..\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\.bin\ignite.CMD
  ignite src         build        C:\Users\..\AppData\Local\npm-cache\_npx\e31027f3785124a8\node_modules\ignite-cli\build

Android
  java               <IP_ADDRESS>    C:\Program Files\Microsoft\jdk-<IP_ADDRESS>-hotspot\bin\java.EXE
  android home       -            C:\Users\..\AppData\Local\Android\Sdk

This is a bug found in <Header />. Because of the zIndex from the title, it makes the button hard to press. If you stick to the outer part of the button you can sometimes get it depending on the specifics of your button. You can visibly see the issue with the element inspector:
Green is padding from the label, blue is where text would show. It overpowers the button label in this case. If you press at the top or bottom edge the onPress fires correctly.

@frankcalise zIndex: 2 fixed the issue. Thanks

:tada: This issue has been resolved in version 8.4.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Same issue in ignite 9 (IOS only).


Improve logging for client usage

Our log output is
2017/08/08 20:21:05 Discovered hostname from system: ip-172-31-5-189
2017/08/08 20:21:05 Agent is running on an EC2 instance
2017/08/08 20:21:05 Discovered AMI ID from AWS EC2 metadata: ami-a4c7edb2
2017/08/08 20:21:05 Discovered EC2 Hostname from AWS EC2 metadata: ip-172-31-5-189.ec2.internal
2017/08/08 20:21:05 Discovered Instance ID from AWS EC2 metadata: i-0eb505a5795f6065c
2017/08/08 20:21:05 Discovered Instance Type from AWS EC2 metadata: t2.micro
2017/08/08 20:21:05 Discovered EC2 Public Hostname from AWS EC2 metadata: ec2-34-229-219-126.compute-1.amazonaws.com
2017/08/08 20:21:05 Timber agent starting up with config:
2017/08/08 20:21:05   Endpoint: https://logs.timber.io/frames
2017/08/08 20:21:05   BatchPeriodSeconds: 10
2017/08/08 20:21:05   Poll: false
2017/08/08 20:21:05   Files:
2017/08/08 20:21:05     /var/log/test.log
2017/08/08 20:21:05 error determining start point: open test.log-state.json: no such file or directory
2017/08/08 20:21:05 Seeked /var/log/test.log - &{Offset:0 Whence:2}
2017/08/08 20:32:35 [DEBUG] POST https://logs.timber.io/frames
2017/08/08 20:32:35 flushed buffer (status code 200)
2017/08/08 20:33:05 [DEBUG] POST https://logs.timber.io/frames
2017/08/08 20:33:06 flushed buffer (status code 200)
2017/08/08 20:33:15 [DEBUG] POST https://logs.timber.io/frames
2017/08/08 20:33:15 flushed buffer (status code 200)
2017/08/08 20:48:15 [DEBUG] POST https://logs.timber.io/frames
2017/08/08 20:48:15 flushed buffer (status code 200)
2017/08/08 20:53:34 got terminated, shutting down...
2017/08/08 20:53:34 stopped tailing /var/log/test.log at offset 523
2017/08/08 21:00:02 Discovered hostname from system: ip-172-31-5-189
2017/08/08 21:00:02 Agent is running on an EC2 instance
2017/08/08 21:00:02 Discovered AMI ID from AWS EC2 metadata: ami-a4c7edb2
2017/08/08 21:00:02 Discovered EC2 Hostname from AWS EC2 metadata: ip-172-31-5-189.ec2.internal
2017/08/08 21:00:02 Discovered Instance ID from AWS EC2 metadata: i-0eb505a5795f6065c
2017/08/08 21:00:02 Discovered Instance Type from AWS EC2 metadata: t2.micro
2017/08/08 21:00:02 Discovered EC2 Public Hostname from AWS EC2 metadata: ec2-34-229-219-126.compute-1.amazonaws.com
2017/08/08 21:00:02 Timber agent starting up with config:
2017/08/08 21:00:02   Endpoint: https://logs.timber.io/frames
2017/08/08 21:00:02   BatchPeriodSeconds: 10
2017/08/08 21:00:02   Poll: false
2017/08/08 21:00:02   Files:
2017/08/08 21:00:02     /var/log/test.log
2017/08/08 21:00:02 Seeked /var/log/test.log - &{Offset:523 Whence:0}
2017/08/08 21:00:42 [DEBUG] POST https://logs.timber.io/frames
2017/08/08 21:00:42 flushed buffer (status code 200)

Move "Timber agent starting up" to be the first line out
Do not indent configuration statements
Change "Endpoint: v" style configuration logs to "Endpoint is v"
Change "Seeked" lines to use human readable format instead of the Offset/Whence struct representation
Change "flushed buffer" messages to indicate which file is being flushed

I'd like to do this after #31 so that we can homogenize the output format.

I'd also add that the level prefix should be consistent. https://github.com/timberio/agent/issues/31 should normalize the format.


Fancy Zones Editor doesn't open sometimes

Microsoft PowerToys version
0.67.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce
Open Facyzones and try to open the Editor.
✔ Expected Behavior
Clicking the FacyZones Editor doesn't open the editor. only after restart of the problem, sometimes.
❌ Actual Behavior
Not opening the FancyZones Editor
Other Software
No response

Does it happen after moving a window using Win+Arrow?

Suppose, it's connected to https://github.com/microsoft/PowerToys/issues/23873.
I'll close this issue, as it's the result of FancyZones stopped working. Thank you for reporting!


Configuration setups before using kolla-ansible to deploy x86 controller + s390x compute

User needs a clear and convenient way to fill their own configurations before deploying x86 controller + s390x compute. eg: passwords, inventory file, globals.yml, etc.

I think we should not wrap this again, better document is the best way to go
use default as much as we can and document the to be changed as detail as we can

I think we should not wrap this again, better document is the best way to go
use default as much as we can and document the to be changed as detail as we can
right. no plan to do any wrap here. Just make clear what settings we required and the acceptable values.

Update setup-deployment-server to setup initial configuration sample files for user
Generate scripts used to setup target nodes to use local repo servers and update bootstrap-servers to run this script on target nodes at the beginning.

Code changes:
https://review.gerrithub.io/c/mfcloud/kolla-ansible/+/427299
Function tested.
Configurations steps recorded in the following boxnote:
https://ibm.ent.box.com/notes/322436026997


add src for plugin

I've checked it out locally and it pulls up fine.
There are a few errors in the console log, which would be nice to clean up. You should be able to see them in the console (Chrome), log but for reference the top level error or warning message is below:
Warning: Encountered two children with the same key, id. Keys should be unique so that components maintain their identity across updates. Non-unique keys may cause children to be duplicated and/or omitted — the behavior is unsupported and could change in a future version.
and
./src/ReactionTable.js
Line 11:  '_' is assigned a value but never used  no-unused-vars

The first one is a known issue with the plugin I selected: https://github.com/react-bootstrap-table/react-bootstrap-table2/issues/601
It is however, a very minor one. I can banish it by duplicating the ID data but that seems wasteful. Let me know.

./src/ReactionTable.js
Line 11: '_' is assigned a value but never used no-unused-vars
the line that throws this is const [_, stoic, cid] = /(\([0-9.]+\)) (\w+)/.exec(comp_txt); I have to use this format because there's no way to just rab the last two values with javascript's destructuring assignment syntax. In Python _ is commonly used for a variable that is syntactic only and not meant for use. I can switch to

const match = /(\([0-9.]+\)) (\w+)/.exec(comp_txt);
const stoic = match[1];
const cid = match[2];

It just seems less elegant

I feel your pain on this, and I have encountered this many times during development, but never with a library. Although in this case it may be minor, in general it is an important issue, and as the warning states, react's behavior around this may change in the future. Best to avoid undefined behavior!
I also understand running up against the differences between JS and Python destructuring. If you need to ignore the first element in an array destructuring, you can just omit the receiving variable altogether -- in other words just omit the underscore. const [,b,c] = ['a', 'b', 'c']
Although the replacement you suggest is indeed less elegant, it would still be preferable to generating a browser warning if need be (and prior to this last year was the only way to do it!)
And in reality, when doing regular expression matches, there is always the possibility of a match failure, in which case the destructuring will throw an error anyway. This really just implies that the regex match result should be checked, the null case handled, and in the success case the expected match positions pulled out. In Typescript (which we are probably using going forward for new projects), the type system will check this and require that kind of handling.
Generally I don't like to approve PRs that emit to the console for any reason other than errors and warnings issued intentionally and directly by our code. Their presence can make debugging in the browser more difficult, and of course they are visible to the curious user.
Errors to console can be useful to provide better debugging support when replicating error conditions, but can be a sign that we've punted on error display. Warnings are rarely needed, but sometimes conditions detected may warrant it. For instance, in kbase-ui warnings are generated for a couple of libraries which use a dubious method of feature detection (which would throws an error, is caught, and then considered worthy of a warning), and for some data view specs which provide an icon but not a viewer widget (with the hope that one day these would be resolved.)

Thanks, James. Looks great now. Do you want to to merge and release it, or have me do it? Doesn't matter to me.

I can merge and tag it but I'm not sure what needs to be done to release
the kbase-ui
On Tue, Mar 26, 2019, 15:32 Erik Pearson<EMAIL_ADDRESS>wrote:

Thanks, James. Looks great now. Do you want to to merge and release it, or
have me do it? Doesn't matter to me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kbase/kbase-ui-plugin-biochem-search/pull/1#issuecomment-476837863,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIGfOb_pPYeyVMg4WRR8-1pFp1RjzTzqks5vaoPIgaJpZM4cIMYT
.

you just need to create a release of the plugin (which also tags it), I'll make sure it is in the upcoming ui release.

but it gets incorporated into the ui in ci pretty quickly.


Use Autoconf to Generate Makefile.common

In this way, no editing of Makefile.common will be necessary. Remember to update instruction in wiki.

Resolved via #76 .


Azure monitor Readme issues

Section link:

Suggestion:
Add link for [azure_monitor_create_using_portal]:
https://docs.microsoft.com//azure/azure-monitor/logs/log-analytics-workspace-overview
@lmazuel , @scottaddie , @srnagar , @hectorhdzg for notification.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Docs:0.3603945,Monitor:0.10290116,Monitor - ApplicationInsights:0.08783135'


Navigation wheel not hiding on empty

Issue
When teleporting to a room with no exits, the navigation wheel will continue to show, with a center button that is clickable, instead of hiding as it should.

Resolved in PR #171


Copy/paste toolbar needs smarter positioning, same issue in Flutter version 1.78

Reproduce the issue:
long press the TestField area, the copy/paste toolbar appears on top of the screen.
see the screenshot below:

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.7.8+hotfix.4, on Mac OS X 10.14.6 18G84, locale
en-CA)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
[✓] iOS tools - develop for iOS devices
[!] Android Studio (not installed)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.2)
[✓] VS Code (version 1.36.1)
[✓] Connected device (2 available)
! Doctor found issues in 1 category.
The codes:
    body: Column(
      children: <Widget>[
        _isInEditingMode ? Container() : _titleBar,
        _isInEditingMode
            ? Expanded(
                child: Padding(
                  padding: const EdgeInsets.all(12.0),
                  child: TextField(
                    cursorColor: Theme.of(context).colorScheme.background,
                    focusNode: _focusNode,
                    style: Theme.of(context)
                        .textTheme
                        .body1
                        .copyWith(color: surface),
                    controller: _controller,
                    maxLines: 160,
                    maxLength: _descriptionMaxLength,
                    keyboardType: TextInputType.multiline,
                    autofocus: true,
                    onChanged: (String text) {
                      _photoDescription = text;
                    },
                    decoration: InputDecoration(
                      hintText: null,
                      counterStyle: Theme.of(context)
                          .textTheme
                          .body1
                          .copyWith(color: surface),
                    ),
                  ),
                ),
              )
            : Expanded(child: _descriptionTextBox),
      ],
    ),

Same issue here

This should be fixed with https://github.com/flutter/flutter/pull/36974, which was just merged.  You can get the fix in the master channel.  Let me know if something still looks wrong.

Works! Thank you very much!


chore(main): release 1.2.12

:robot: I have created a release beep boop
1.2.12 (2024-01-20)
Miscellaneous Chores

deps-dev: bump phpunit/phpunit from 10.5.5 to 10.5.8 (#54) (72bd7c3)
deps: bump actions/cache from 3 to 4 (#53) (8e3b5a3)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/mattermost-php/releases/tag/v1.2.12 :sunflower:


Update with links to particular sections.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Refactor concat to use merge for non-concatenated variables

This PR adds two changes:

First, it refactors concat so that non-concatenated variables are passed to merge. concat gains a compat kwarg which tells merge how to merge things. concat's previous behaviour was effectively compat='equals' which is now the default.
Also adds compat="override" to skip equality checking in merge. concat's data_vars='minimal' and coords='minimal' options are now more useful because you can specify compat='override' to skip equality checking and just pick the first one. Previously this would raise an error if any of the variables differed by floating point noise.

[x] xref #2064, closes #1823
[x] Tests added
[x] Passes black . && mypy . && flake8
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

Now with docs! I updated the mfdataset and parallel reading sections in dask.rst and io.rst.

Hello @dcherian! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/core/concat.py:

Line 123:1: W293 blank line contains whitespace

Test failure is #3300

Thanks for the review @shoyer


Integration Setup Foundation for cardano-node w/ Byron

Issue Number

#1347
Overview

91b6d63e6845b0ac193b93aaa576d4d846aab9d3
move and rename 'sockAddrPort' to 'unsafePortNumber'
a. Allow sharing the code outside of the jormungandr package
b. A bit more true about the underlying intent of the function
c. The function had so-to-speak nothing to do with Cardano.Faucet

72aad48ffccbf195431c78f767c0f791e7c66c0c
remove 'nodePort' and 'feePolicy' from 'Context'
this type is common to various integration setup. However, these constituants are
only used in Jormungandr-specific scenario and do not apply to cardano-node

beb6032685c132c51a66b5f286ffdb74138877cc
Remove hard-coded byron parameters from compat functions
Passing them as arguments instead. Also, currently abusing the
'Testnet' discriminant with integration data. Later, I'll make sure
that serveWallet takes blockchain parameters as arguments instead of
a 'KnownNetwork =>' constraints to allow constructing private blockchain
from arbitrary genesis blocks.

9650acec6e362e37123e816bb50e5c8bd22477b5
remove 'staticBlockchainParameters' from NetworkLayer
a. Blockchain parameters aren't actually static and comes from genesis + protocol updates.
b. Allow more flexibility for cardano-node in the way the genesis block is passed
c. Remove awkward conversion in the byron compat code for cardano node

b9bb6d86e91ea4a594729d793df99e04aea4b14d
Implement 'fromGenesisTxOut' to construct transaction from genesis UTxO
These transaction are a bit special and by convention, the transaction id is a hash of the genesis address.

fb193b4146ef54ea7c3a8c7b252442d3aff44905
Fix wrong checkpoint reference in unstable assertion
While doing ... something totally different, I noticed that some integration tests, when restoring a wallet were showing something like 1000x "Creating checkpoint at xxxxxxxx-[X.X#X]."
This caught my attention for a second because:
a. With the setting I had, it should only create around 10 checkpoints, not a thousand.
b. This happened not only near the tip, but at any time.
Inspecting the code responsible for inserting checkpoints, I found this in Cardano.Wallet#restoreBlocks
        forM_ (NE.init cps) $ \cp' -> do
            let (Quantity h) = currentTip cp ^. #blockHeight
            when (fromIntegral h `elem` unstable) $ do
                liftIO $ logCheckpoint cp'
                putCheckpoint (PrimaryKey wid) cp'

Note the currentTip cp instead of currentTip cp' ! So, the assertion of whether or not a checkpoint should be inserted was based on the current checkpoint from the database, and not on the actual checkpoint. So basically, every time the database checkpoint is considered a stable block, we get to insert ~1000 checkpoints in the database man_facepalming ... That can take some time...
I am not sure whether this is the cause of the Windows slowness mentioned in #1398 (and why this doesn't really impact Linux then?) but it is definitely wrong.

Comments

This is only a foundation for setting up the integration tests (still not committed here). I've got the integration setup working and running, though I still need to go through a few fixes to avoid hard-coded some of the genesis parameters and instead, read them from the genesis file.

bors try

I see that commit of ouroborous-network is month old in stack.yaml. We are ok with that in the context of this PR ?

Yes. Locally, I am running a 1.6 cardano-node and the client code is using the libraries from 1.5. It works fine. I'll bump the dependencies at the end.

bors r+

bors r+

3x in a row the same error. But I can't reproduce locally. This is weird.

bors r+


Update Verilator link.mk

Here I remove -j and rely on the user to do make -j
We could also do -j$(NUM_THREADS) or something
-j by itself starts infinite jobs and crashes my laptop

strong reject

jk

I used to have an NPROC (or similar) variable for VCS compilation. It would determine the number of threads in the system and then divide by four.
Should be in the git history somewhere.
But if you don't want to track it down, LGTM

good to go?


package update

Microsoft.Azure.CosmosDB.Table updated to version 2.0 that is ga

@georgewallace could you please look on it.

@SychevIgor Thanks for the contribution.  I don't work in the storage space anymore. @tamram Do you mind to review this update?

@georgewallace any feedback?

ok, microsoft don't care


Remove clusterscan types

Issue:

https://github.com/rancher/rancher/issues/39053
Problem

cis v1 types still exist in v2.7
Solution

removed types associated with cis v1.

lgtm, pending CI passes


Add prepack commands to ensure we don't deploy stale builds

Writing a deploy script seems more useful


issue_84: Deactivate using online schemas validation

Added --skip-schema-validation flag to deactive schemas/schematron validation.

Closed per rebased branch on PR #113


reimplement entrypoint in Python

Per @manics idea, this reimplements the entrypoint in Python, mainly so we can better apply our expertise and understanding to it.

repo2docker-entrypoint is now in Python
shebang points to new python3-login to preserve login-shell behavior (source profiles, etc.) because we can't nest interpreters in a single shebang line. First attempt extracted login env from a subprocess, but I think this is a little more natural, maybe, as it matches the usual process hierarchy (bash -l > Python > 'real' entrypoint.
make sure PYTHONUNBUFFERED env is set, which is ~always needed in containers
tee is implemented in Python with nonblocking
all available signals except a select few that cannot be caught (SIGKILL, SIGSTOP) and shouldn't (SIGCHLD) are relayed to the child

It seems to work! Honestly, I cannot say if this fixes the issue because I know how to do unbuffered output in Python and couldn't figure it out with other tools, or if it's because Python is just enough slower that the previously lost output always loses the race 🤷 .
Original description: use stdbuf to make sure everything is unbuffered

apply stdbuf to everything in repo2docker-entrypoint
join stdout, stderr into log so we don't miss anything in the log file (from #1008)
use capfd to capture output in test_env (not relevant, should be a simplification)

A bit of a shot in the dark at #1008

Didn't work :/

Do you think it's worth converting the entrypoint to Python and using subprocess with it's myriad options, since we might have a better understanding of how it works?

Do you think it's worth converting the entrypoint to Python and using subprocess with it's myriad options, since we might have a better understanding of how it works?

Maybe. I'll give it a go and see if that changes anything.

Let's see what this does :D
It feels like some comment from Inception is appropriate as we add another layer of Python :D


Update VenStockRevamp

Set ksp_version explicitly as the .version file distributed isn't set
to latest.

Might it be pertinent to remove the vref field and instead add a comment to the file explaining that a AVC file exists but hardcoding was done due to the .version file being incorrect?
Or just add a comment explaining why the file has both vref and hardcoding to the next person looking Will have some idea without checking commithistory.

@Dazpoet comment added


[Bug][Dashboard] Multiple NPMs have security issues

What happened
Dependabot reports multiple issues with NPMs and CVEs associated with them.
These are just a few of many:

https://github.com/advisories/GHSA-hgjh-723h-mx2j url-parse
https://github.com/advisories/GHSA-r683-j2x4-v87g node-fetch
https://github.com/advisories/GHSA-33f9-j839-rf8h inmer
https://github.com/advisories/GHSA-ww39-953v-wcq6 glob-parent
https://github.com/advisories/GHSA-hxcc-f52p-wc94 serialize-javascript

What you expected to happen
n/a
How to reproduce
n/a
Environment
all
InLong version
0.11.0
InLong Component
InLong Dashboard
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@healchow I'm not a UI developer or even an inlong user - I am an ASF member who is very concerned about ASF projects not using secure libs

@leezng PTAL, thanks.

I saw the reference chain of node-fetch, which is used in umi-request. We tried to upgrade umi-request, but the old fetch is still used in the latest version, so we cannot upgrade directly for the time being


URL changed

Does it work? Or need changes with xpath?

This PR should be merged as the current URL does not work


[Question] Deploy on openshift

Hi, sorry about this not being a real issue, but I've been trying to get into the Slack group but the email doesn't arrive.
I'm trying to get this example running in a free OpenShift site. I know there is documentation for Heroku but I'm trying to get this there. Has anyone tried to get some project based on this example there? I've tried to set it up both with the standard node cartridge and a custom node 4 cartridge to no avail. The packages run, but as soon as I add this example's code, the won't start.

Just so you know, this issue was automatically posted to the Slack group, so anyone there who knows about OpenShift should come and help, but it's just crickets so far.

Thanks, I'm still trying things and debugging as much as I can. If I get it working I will surely post it here.

Hi, sorry about this not being a real issue, but I've been trying to get into the Slack group but the email doesn't arrive.

When did you sign up? Reactiflux has been closed for some time now.

@nesukun Did you solve your problem? I've just deploy it on a test gear at OpenShift. I had to do some changes:
Update node to 4.2.1
Update npm to 3.x
Add a marker use_npm
Set BABEL_CACHE_PATH
Change src/server.js to listen to right IP and port

If I recall correctly, except for the npm upgrade (stayed at node 4's default) I made the very same changes myself but couldn't get it to run. Could I see an example of your setup? Thanks!

@trueter registration was open not so long ago, but it errored silently, some time after I reported that, they removed the hability to attempt registration

Sorry @nesukun, but what do you mean by "example of your setup?"
I spent almost 4 days to get it run properly on Openshift.
What I did was:

Create a new app nodejs-0.10 cartridge
Upgrade node version following these instructions
Upgrade npm using: npm -g install npm
Create an empty file inside .openshift/markers called use_npm. It is necessary to deactivate supervisor. It will call npm start -d after some git push
Set an environment variable called BABEL_CACHE_PATH using rhc env set BABEL_CACHE_PATH="/var/lib//app-root/repo/.cache/.babel.json"
Changed src/server.js

    let portProd = process.env.PORT || process.env.OPENSHIFT_NODEJS_PORT || 8080;
    let ipProd   = process.env.IP   || process.env.OPENSHIFT_NODEJS_IP || '<IP_ADDRESS>';
    //if (config.port) {
      if (config.isProduction) {
        const io = new SocketIo(server);
        io.path('/api/ws');
      }

      server.listen(portProd, ipProd,  (err) => {
        if (err) {
          console.error(err);
      }

I think there is a better way to define IP and PORT, but I just put it there for tests purposes.
It is all that I did to run the project in production mode. I didn't try to run it in development mode.

Hmm, will try again. The only difference I see is that I didn't upgrade NPM
and that I placed the babel cache in OPENSHIFT_DATA_DIR
On Fri, 16 Oct 2015 at 16:51 mclouvem<EMAIL_ADDRESS>wrote:

Sorry @nesukun https://github.com/nesukun, but what do you mean by
"example of your setup?"
I spent almost 4 days to get it run properly on Openshift.
What I did was:

Create a new app nodejs-0.10 cartridge

Upgrade node version following these instructions
https://github.com/ramr/nodejs-custom-version-openshift

Upgrade npm using: npm -g install npm

Create an empty file inside .openshift/markers called use_npm. It is
necessary to deactivate supervisor. It will call npm start -d after some
git push

Set an environment variable called BABEL_CACHE_PATH using rhc env set
BABEL_CACHE_PATH="/var/lib//app-root/repo/.cache/.babel.json"

Changed src/server.js
let portProd = process.env.PORT || process.env.OPENSHIFT_NODEJS_PORT || 8080;
let ipProd   = process.env.IP   || process.env.OPENSHIFT_NODEJS_IP || '<IP_ADDRESS>';
//if (config.port) {
if (config.isProduction) {
const io = new SocketIo(server);
io.path('/api/ws');
}
server.listen(portProd, ipProd,  (err) => {
if (err) {
console.error(err);
}

I think there is a better way to define IP and PORT, but I just put it
there for tests purposes.
It is all that I did to run the project in production mode. I didn't try
to run it in development mode.
—
Reply to this email directly or view it on GitHub
https://github.com/erikras/react-redux-universal-hot-example/issues/273#issuecomment-148752470
.

--
Kind regards / Saludos / Mit freundlichen Grüßen / 以上、よろしくお願いいたします
Néstor Hernández Ojeda

Any updates with this?

Feels stale. Reopen if still a problem.

I wrote up a quick guide for deployment on openshift, take a look at it and please tell me if it is of any help:
https://github.com/Dindaleon/hapi-react-starter-kit/blob/master/Guides/OpenShift_Deployment.md


ci(pr-labeler): disable on tree-wide PRs

Disables it from running on PRs with a title containing (tree-wide) since those often edit so many files that the labeler workflow fails due to too many labels.
Side note: looks like we are pinned at a very specific version for the actions/labeler workflow - any reason for this? Can we / should we upgrade to https://github.com/actions/labeler/releases/tag/v5.0.0?

We should probably also document this.