fix indices saved to WCSimRootCherenkovDigiHit::GetPhotonIds

The indices stored in WCSimRootCherenkovDigiHit::GetPhotonIds are incorrect. This makes it impossible to determine such things as signal-to-noise ratio in digits, or which/how many digits came from a given primary particle.
In theory these indices should represent the index of the corresponding photon hits in the WCSimRootCherenkovHitTime TClonesArray, as with the return of the (poorly named) WCSimRootCherenkovHit::GetTotalPe(0) method. But in actuality, the PhontonIds represent only the index of the photons within the appropriate digit in the WCRawPMTSignalCollection. This collection gets flattened to generate the CherenkovHitTime array, and in the process the relative offset of each entry within the collection is lost. Since not all hits get digitized, these offsets can't be recovered.
Fortunately the fix is very simple - introduce a running counter that turns the relative index into an absolute one, as done in this PR.
To validate, apply the attached patch, which introduces debug printouts into WCSimWCDigitizer.cc to print the tube, time, index and parent of each digit. Then run the attached ROOT macro to print the corresponding information from the wcsim.root output file. Note that not all entries printed in WCSimWCDigitizer.cc get digitized or are within the trigger windows, so not all will be in the ROOT macro output. By comparing the printed information, though, it's possible to match up the entries and verify the saved information is correct.
Apply the patch with `git apply {patchfile}'
validation_patch.txt
Verify the output returned by the following ROOT macro
parent_validation.txt

Thanks @marc1uk. I'm not sure this is required (please feel free to poke holes in my method to get the info in the example below) but it will make it much more convenient & fast (i.e. it removes the need for a loop over the CherenkovHits to find the hits with the correct PMT ID)
This example below that calculates the number of noise/Cherenkov/other hits for each digit, and can be inserted into a WCSim macro using the standard naming conventions
const int ncherenkovdigihits = wcsimrootevent->GetNcherenkovdigihits();
const int ncherenkovhits      = wcsimrootevent->GetNcherenkovhits();

// Loop through elements in the TClonesArray of WCSimRootCherenkovDigHits
for(int idigipmt = 0; idigipmt < ncherenkovdigihits; idigipmt++) {
  //get the digit
  TObject *element = (wcsimrootevent->GetCherenkovDigiHits())->At(idigipmt);
  WCSimRootCherenkovDigiHit *wcsimrootcherenkovdigihit =
    dynamic_cast<WCSimRootCherenkovDigiHit*>(element);

  //find out whether this is due to noise, a real photon, or both
  int tube_id = wcsimrootcherenkovdigihit->GetTubeId();
  int timeArrayIndex = -1;
  int peForTube      = -1;
  //loop through the WCSimRootCherenkovHit array to find the entry for this tube ID
  for(int ipmt = 0; ipmt < ncherenkovhits; ipmt++) {
    TObject *Hit = (wcsimroottrigger0->GetCherenkovHits())->At(ipmt);
    WCSimRootCherenkovHit *wcsimrootcherenkovhit =
      dynamic_cast<WCSimRootCherenkovHit*>(Hit);
    int tubeNumber     = wcsimrootcherenkovhit->GetTubeID();
    if(tube_id == tubeNumber) {
      timeArrayIndex = wcsimrootcherenkovhit->GetTotalPe(0);
      peForTube      = wcsimrootcherenkovhit->GetTotalPe(1);
      break;
    }
  }//ipmt
  int n_noise_hits = 0, n_photon_hits = 0, n_unknown_hits = 0;
  if(timeArrayIndex == -1) {
    if(verbose)
      cout << "No PMT hits found for digit " << idigipmt << " with tube ID " << tube_id << endl;
  }
  else {
    if(verbose)
      cout << peForTube << " PMT hits found for digit " << idigipmt << " with tube ID " << tube_id << endl;
    //loop over the rawhits ids of hits that made up the digit
    vector<int> rawhit_ids = wcsimrootcherenkovdigihit->GetPhotonIds();
    if(verbose)
      cout << rawhit_ids.size() << " rawhits made up this digit" << endl;
    for(unsigned int irawhit = 0; irawhit < rawhit_ids.size(); irawhit++) {
      int this_rawhit = rawhit_ids[irawhit];
      if(verbose)
        cout << "Attempting to look for rawhit " << this_rawhit+1 << " in WCSimRootCherenkovHitTime array...";
      if(this_rawhit >= peForTube) {
        cerr << " There are only " << peForTube << " rawhits in this PMT" << endl;
        continue;
      }
      //now look in the WCSimRootCherenkovHitTime array to count the number of photon / dark noise hits
      TObject *Hit = (wcsimroottrigger0->GetCherenkovHitTimes())->At(timeArrayIndex + this_rawhit);
      WCSimRootCherenkovHitTime *wcsimrootcherenkovhittime =
        dynamic_cast<WCSimRootCherenkovHitTime*>(Hit);
      const double hittime  = wcsimrootcherenkovhittime->GetTruetime();
      const int    parentid = wcsimrootcherenkovhittime->GetParentID();
      if(verbose)
        cout << " hit time " << hittime << " " << parentid << endl;
      if(parentid == -1) {
        n_noise_hits++;
      }
      else if(parentid < 0) {
        n_unknown_hits++;
      }
      else {
        n_photon_hits++;
      }
    }//irawhit
  }//hits in this PMT found
}//idigipmt // End of loop over Cherenkov digihits

Hi Tom,
Thanks for the comment. I have to admit your method seems like it would work, I hadn't thought of using the TubeID to match up the Hits array with the Digits array. Thanks to that, I won't have to regenerate my simulation results. :) But nonetheless, the PR still seems like a good idea; the method is neither efficient nor by any means intuitive.

Agreed, I like it.
Perhaps you could do (what I should of done a while ago...) and add (with this PR) an example of how to do the digit->hit truth matching in sample_readfile.C, with maybe signal-to-noise ratio of digits.

Sure, I'd be happy to. I'm a little busy right now, but I'll add it soon.

Accidentally closed the issue somehow.
Anyway, I've added the long-awaited modifications to the example script. For the first 3 digits it pulls the photons, pulls the corresponding WCSimRootCherenkovHitTime and prints the index from the TClonesArray, the photon hit time, and it's parent track id. Hopefully that's enough to demonstrate things.
While I was modifying the sample script it I also made some other printout changes to clarify wording on things that I don't think are helpful. Things like printing Jmu 1 or "Mode 8" is meaningless for someone who hasn't already dug through the code to see where they come from; the variables are poorly named to begin with, and this is a good place to explain things.  In fact, many of the printed parameters are only properly filled for nuance primaries, but the default primary source is an electron gun, for which many variables just print 0...

Thanks for this - clarifying everything in the sample root script is a great addition to the PR.
Doing a close/open to force a Travis rebuild (it was initially failing due to this PR changing what's stored in the output, as expected)


Test of parsing a valid repositories section.

Change the target to describe the template:
oasis.testAssertion.target: The 'repository' out of the parsing.


Add support for ImportAware on Configuration classes

When configuration classes are parsed at build time, ConfigurationClassPostProcessor honors two callbacks, see
https://github.com/spring-projects/spring-framework/blob/3c2dfebf4ec5e9d791c1f3c9fa0ad35a5a9fcd6b/spring-context/src/main/java/org/springframework/context/annotation/ConfigurationClassPostProcessor.java#L460-L489
Some classes required to be initialized again at runtime unfortunately, see AbstractAsyncConfiguration. This requires to load the ASM metadata of the class at runtime so dedicated entries in reflection config are likely to be necessary as well.

This turns out to be a significant amount of work. It's not only the metadata of the class that's important, but also the class that imported it. At runtime, configuration class parsing keeps a stack of configuration classes and knows the chain of imports.
Providing the metadata of ProxyAsyncConfiguration is not helping:
Caused by: java.lang.IllegalArgumentException: @EnableAsync is not present on importing class org.springframework.scheduling.annotation.ProxyAsyncConfiguration
	at org.springframework.scheduling.annotation.AbstractAsyncConfiguration.setImportMetadata(AbstractAsyncConfiguration.java:61)
	at org.springframework.aot.context.annotation.ImportAwareInvoker.setAnnotationMetadata(ImportAwareInvoker.java:29)
	at org.springframework.scheduling.annotation.ContextBootstrapInitializer.lambda$registerProxyAsyncConfiguration$1(ContextBootstrapInitializer.java:12)
	at org.springframework.aot.beans.factory.SmartFunction.apply(SmartFunction.java:18)
	at org.springframework.aot.beans.factory.BeanDefinitionRegistrar.lambda$instanceSupplier$0(BeanDefinitionRegistrar.java:69)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.obtainFromSupplier(AbstractAutowireCapableBeanFactory.java:1231)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1173)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:564)
	at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:524)
	... 36 more

That's because @EnableAsync is on a user-class that led ProxyAsyncConfiguration to be loaded. We need to keep that link so that we can read the relevant annotation metadata at runtime again.

I've managed to make a basic sample work by writing down the structure of the bean factory at build time for the related import classes. I am not sure it'll work in all cases, typically when the same annotation is present on two different configuration classes.

Closed by https://github.com/spring-projects-experimental/spring-native/commit/914bb211a11c906ab772a216c70168aa85c254e9


Support and help link for telegram

Where does the issue occur?
https://testnets.cardano.org/en/shelley/resources/support/
Describe the issue
The group for pool operators is not https://t.me/CardanoDevelopersOfficial, but https://t.me/CardanoStakePoolWorkgroup instead.

thank you for your feedback, we have updated this link.


Windows wheel package (.whl) on Pypi

Please make Windows wheel packages and put them on Pypi.
Currently it is possible to download Windows wheel packages for pandas here: http://www.lfd.uci.edu/~gohlke/pythonlibs/#pandas
It would be great if the wheels were directly available on the Pypi server https://pypi.python.org/pypi/ so that they can be installed with pip.

hmm what happened to the installers that were there?
I didn't know he was posting wheels

He decided to replace all the exe files with whl files a few days ago which forced the author of WinPython to make a new version available with an updated Control panel that can install whl files as I understand it.
And so the wheels turn!

It would be nice to offer wheels. I know scikit-learn uses Appveyor (some docs here). We could do something similar.

I uploaded the built wheels to pypi. Not sure why no more execs though.
@TomAugspurger I had installed a configuration file for appveyor. But its way slow. Better to just have the windows builds by grohlke for now.

Thanks alot!


Upload Sketch to Arduino

In Ardublockly, there is Upload Sketch to Arduino Option. It takes too much time to upload sketch to arduino.
When I click Upload Sketch to Arduino option it loads arduino executable then uploads the sketch to it.
Every time you click upload sketch this process happens.
Is there any way to minimize this step. or can we upload sketch to arduino little bit fast.
because every time you upload sketch it takes too much time to upload

That's the time it takes for the Arduino IDE to compile and upload the sketch.
There is one thing that could be speed up this process thoough, and that is to enable the Arduino IDE flag to keep temporary files on exit. However I haven't had a chance to test it as I am not quite sure how, when and if these temporary files are accessed on next compilation to speed up the process.

I found one article for Arduino Speed Optimization.
http://www.instructables.com/id/Arduino-IDE-16x-compiler-optimisations-faster-code/?ALLSTEPS
But It has some disadvantages too. Read "A few mild words of caution" section in above url.
I tried instructions given in above url. it is only some microsecond faster.
Is there any other way to optimize Execution.
In ardublockly, It takes time to launch Arduino + upload and compile sketch to Arduino board.
Every time while compiling it shows arduino launch image. can we bypass this.

Hi Rimish,
Thanks for the link, it was definitely an interesting read.
Ardublockly at the moment invokes the Arduino IDE and uses the Arduino console interface defined flags in order to be able to compile and upload code. There is the option of using the Arduino Builder directly for compilation, but it would requires A LOT more information about the individual board packages to be included within Ardublockly, which makes it significantly harder to add new boards. Apart from that, the output binary file needs to be uploaded, and to do that Ardublockly would have to interface with avrdude directly with all the same inconveniences described for the Arduino Builder. Both these disadvantages are worsened when considering 3rd party boards with completely different toolchains.
Changing the gcc optimisation level as described in the article would be possible, however I am not in favour Ardublockly permanently editing settings in the Arduino IDE that will affect its use outside of Ardublockly. Alternatively, as a user you can change this settings manually which could improve the compilation time. However, it sounds like you've tried this with minimal improvements, which is a bit disappointing.
Your other suggestion, to leave the application open, would mean a completely different implementation of how Ardublockly interfaces with the IDE, faking user input to paste the code and compile. It would be possible, but the benefits would be minimal for what would feel like a bit of a hack, specially when a nice and clean command line implementation is provided by the IDE.
The only option that I am aware of right now while using the Arduino IDE CLI, would be to manually set the build folder to a known location (so create and maintain from Ardublockly a OS-specific temporary folder) and then set configure the IDE to not delete temporary files on exit and manually try to clean up the folder on Ardublockly exit. In theory it should speed up compilation as only the new code should be compiled, but it is something I haven't had the chance to implement and test.
More info can be found in: https://github.com/arduino/Arduino/blob/master/build/shared/manpage.adoc

Can You help me to do this?

I could give you some guidance with the implementation yes, as a lot of the work will be testing it as well.
Give it a go and send me a PR and I can review it and guide you to the right direction.

In 1 paragraph you said
There is the option of using the Arduino Builder directly for compilation, but it would requires A LOT more information about the individual board packages to be included within Ardublockly, which makes it significantly harder to add new boards. Apart from that, the output binary file needs to be uploaded, and to do that Ardublockly would have to interface with avrdude directly with all the same inconveniences described for the Arduino Builder.
In Last paragraph you said
The only option that I am aware of right now while using the Arduino IDE CLI, would be to manually set the build folder to a known location (so create and maintain from Ardublockly a OS-specific temporary folder) and then set configure the IDE to not delete temporary files on exit and manually try to clean up the folder on Ardublockly exit. In theory it should speed up compilation as only the new code should be compiled, but it is something I haven't had the chance to implement and test.
So We have two option right now to speed up ardublockly compilation. so which one of the above should we do or which one of this two will have best results for optimization


[forceDialCode] Fix e164Phone value update when input is cleared

What has been done

Fixed e164Phone value when user clears input with forceDialCode={true} prop

Updated tests with missing case

:tada: This PR is included in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix link to Fluent UI used in the sample

Changing the link to Fluent UI library used in the samples to avoid confusion.
Removed link was pointing to Fluent UI v7, which does not have some of the components used in the sample code (like Chat).
Added link is pointing to Fluent UI v0, which was actually used in the sample code.

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

#sign-off

@ddematheu2 - Hi, could you review the proposed updates to your articles?
Also, to support automated notification and PR assignment in the public repo, would you update the author metadata attribute in both (all) articles to ddematheu2? Thanks!

@ktoliver
https://github.com/MicrosoftDocs/azure-docs-pr/pull/132907
https://github.com/MicrosoftDocs/azure-docs-pr/pull/132908

@ktoliver Kristine Toliver (Aquent LLC) Vendor
MicrosoftDocs/azure-docs-pr#132907
MicrosoftDocs/azure-docs-pr#132908

Excellent. Thanks! (It looks like the PRs will auto-merge when you add #sign-off. 👍 )


Typo in primary demand method for torrefied biomass pellets

Recently we fixed a typo in the torrefied biomass carrier. However, the corresponding primary demand method still uses the old carrier name (torr-i-fied)

I think this is because a node still has the old name.

Thanks!


Outdated version

This repo: github.com/sqlc-dev/sqlc-go v1.18.1
imports deps: buf.build/gen/go/sqlc/sqlc/protocolbuffers/go v1.31.0-20230731215519-a0cf5310fab9.1
but github.com/sqlc-dev/sqlc/blob/v1.22.0/internal/plugin/codegen.pb.go diff buf.build/gen/go/sqlc/sqlc/protocolbuffers/go

Missing OutputCopyFromFileName attribute
Can you keep it synchronized? Can facilitate the development of plugins！
Thanks！


fix page summary html escaping

I really like this theme, thanks for creating it!
I noticed the summaries of pages get escaped in the posts template. A simple | safe addition fixes it.

Thanks!


Script video

Hi
No footage of me exists.
Not many photos. :--0
I vote for:
Let's tape Mark Kramer giving the core of the SHR message---a 3 minute blurb with a 15-20 minute full version.
More SHR, less MITRE.
Release immediately on Git Wiki as part of our continuous collaboration.
...H
PS
Mary----We need to finding funding for Partners ConnectedHealth.
PSS
We must position SHR wrt OHDSI CDM. ASAP.

From: Quilty, Mary T.
Sent: Monday, June 27, 2016 2:36:09 PM
To: Sonin, Juhan P.
Cc: Sleeper, Harry A.
Subject: Partners Connected Health Conference
Hi Juhan,
I spoke with DeAnna from Partners Connected Health Conference today. OHS was talking about having a formal presence there this year (exhibitor booth, private room for promoting SHR, etc.).
While on the call DeAnna mentioned that she spoke with you concerning Harry as a possible panel presenter this year or next. She asked if there was footage of Harry.
Writing to let you know she and I spoke and to see if you were following up on her Harry request.
Thank you and take care,
Mary
Mary T. Quilty, SM
Lead Healthcare Policy Analyst, Health Standards & Interoperability
Health Transformation Technology Center
The MITRE Corporation
Bedford, MA 01730-1420
(781) 271-4793– Office
(781) 408-9024 – Mobile/Text
Email<EMAIL_ADDRESS>Website:  www.mitre.org
…working in the public interest…

Closed by mistake, reopened.

Requirements for 3 min video:
•	Script
o	Overarching message and say 3 key takeaways
o	Call to action?
•	Visual (static or select briefing screens)
o	Note: if we are close to a logo that would be great to include
•	Position of visual (who is center stage – Mark or the visual – or does it alternate?)
•	Public release – script and video needed?
•	Harry said this is for Git Wiki
o	Setup a new one or use MITRE’s
	Open Source software from The MITRE Corporation at GitHub https://mitre.github.io/
	How is this connecting to a greater communications plan – does it matter?
•	Does 3 min video connect to the larger 15 min video in a meaningful way (e.g., flesh out the outline introduced in the 3 min)
•	Logistics
o	Well-rehearsed and ready to rock it Mark
o	Room
o	Sound quality
o	Light quality
o	Video and video quality; editing needs?

Can animation be included in the video? Are the 1 pagers starting to build what could be the basis of a script? what are the key points for the video and where is it going to be posted - MITRE youtube? SHR website? Github page?


belindas-closet-nextjs_13_553_form-page

Resolves #553

This draft PR is for creating a basic form page to capture student information, but right now, the app/form-page/page.tsx file is still empty. It has one failing check related to build-and-test, while the other checks, like Socket Security and CodeQL scans, have passed, which at least shows there are no security issues or dependency changes.
There's still work to be done here to actually add the form fields as outlined in the user story. Automated checks are useful here to make sure we’re not introducing any vulnerabilities, even if the code is minimal at this stage. Here is an useful resource for Form design best practices


Snippets for Atom Editor

I ported your VS Code snippets and made a package for Atom (here). Is this the right place to cut a PR to contribute to the style guide? I wasn't able to find the repo for the Angular.io docs. Thanks!

the angular.io docs contain the A2 style guide please go there

Thanks


updated reviews styling

This fix #143

Bro @RohitKumar-200 check this now

@rockingrohit9639 Your designs breaks for smaller screens. Make it responsive.

Also make review box little bit small (reduce height),  Reviews section is taking too much space and probably will grab too much attention for user.

@RohitKumar-200 I have updated this, please have a look now.


update to <IP_ADDRESS> triggered full reindexing on restart of RM 2017.1.2

hopefully this is not intentional and will be fixed in future updates?

A reindexing is necessary because the lexing/parsing of Bash files changed.
If IntelliJ is smart then only Bash files are reindexed on startup (I'm not sure about this atm). Otherwise you'd get exceptions while working with Bash files.
The index should be only refreshed once, though. If this isn't the case please let me know.

it was a lengthy full reindex, is it going to happen on every plugin update?

No, not on every update. Only when necessary.

noticed it again on 1.6.11 update.
You said that your intent is only for bash files to be reindexed, but rubymine is reindexing everything, so there is a bug somewhere, right?

@bughit Not that I know. BashSupport is enabled for .bash and .sh files by default and also for files without extension. BashSupport tells the IDE that it needs an index update and your IDE does the rest of the job (choosing the files, etc). It's not under BashSupport's control what is indexed.
If you think that it's indexing more files than necessary please submit an error report to JetBrains.

First of all, are you claiming that you can't repro this, that the ide does not do a full reindex for you?
If you can repro, the way to determine if there is an ide bug, is by whether or not you try limit the scope of the indexing and the ide ignores you. Then it's a bug.
If you don't try to explicitly limit the scope, and hope the ide figures it out, but it doesn't then it's more of a feature request, unless they already explicitly promise to limit the scope automatically, then it's a bug again.
In either case you are in the best position to report and explain this to them.  But you don't seem to want to.  Why?  Do you think it's reasonable that an ide is rendered nearly unusable for multiple minutes from a relatively inconsequential plugin update?

@bughit
I can reproduce this here because it's the way it works.
As I said: it's not possible to limit the scope of the reindexing because BashSupport can just tell IntelliJ / WebStorm to do it and has no influence on the scope (afaik).
I don't see the necessity to post a new feature request / bug report to JetBrains. BashSupport updates are released every 1-2 months, reindexing in that interval is reasonable, I think.
If you think that it's worthwile for you to get an improved indexing in IntelliJ then post a feature request in JetBrains' YouTrack, please. You can copy / link this thread, I think.
If you prefer not to do this then either stay with the current indexing or uninstall BashSupport.


Fix missing botname and fix attachments without text

If bot sets the username in the message it was not being used. For example slack-irc bridge uses this.
If message contained only pretext without actual text it was not marked as handled and thus whole details were shown. For example Zendesk ticket notifications sometimes contain only pretext.

Thanks! 👍


Fix arg order in TableProvider::hmtx

fixes #105

Typically for a bug fix I like to see a test that exposes the bug and is fixed by the change, should be feasible here I think?


Throwing error in phoenix framework 1.3

I created a sample phoenix 1.3 project using
`sudo mix phx.new Server --app locationsearch --no-html --no-brunch
Installed this library as dependency
mix.exs

  defp deps do
    [
      {:phoenix, "~> 1.3.0"},
      {:phoenix_pubsub, "~> 1.0"},
      {:phoenix_ecto, "~> 3.2"},
      {:postgrex, ">= 0.0.0"},
      {:gettext, "~> 0.11"},
      {:cowboy, "~> 1.0"},
      {:geo_postgis, "~> 1.0"}
    ]
  end

dev,exs

# Configure your database
config :locationsearch, Locationsearch.Repo,
  adapter: Ecto.Adapters.Postgres,
  username: "postgres",
  password: "123456",
  database: "locationsearch_dev",
  hostname: "localhost",
  pool_size: 10,
  extensions: [{Geo.PostGIS.Extension, library: Geo}]

`
In My Priv Folder
File 1:
defmodule Locationsearch.Repo.Migrations.EnablePostgis do
  use Ecto.Migration

  def change do

  end

  def up do
    execute "CREATE EXTENSION IF NOT EXISTS postgis"
  end

  def down do
    execute "DROP EXTENSION IF EXISTS postgis"
  end

end

File 2 :`

defmodule Locationsearch.Repo.Migrations.CreateShipData do
  use Ecto.Migration

  def up do
    create table(:test) do
      add :name,     :string
      add :geom,     :geometry
    end
  end

  def down do
    drop table(:test)
  end

end

Dont know where to put this code
Postgrex.Types.define(Locationsearch.PostgresTypes,
              [Geo.PostGIS.Extension] ++ Ecto.Adapters.Postgres.extensions(),
              json: Poison)

When i run
mix ecto.migrate
I am getting lots of errors tried different solutons but nothing works
Can you tell me where i am going wrong.
Do you have any idea to build a example phoenix1.3 app with this library if so it would be very helpful for many of us.
Thanks.Your help is much appreciated

Issue Solved
config/config.exs
use Mix.Config
config :test, ecto_repos: [Test.Repo]

config :test, Test.Repo,
  types: Test.PostgrexTypes

import_config "#{Mix.env}.exs"

lib/test/postgrex_types.ex
Postgrex.Types.define(Test.PostgrexTypes,
[Geo.PostGIS.Extension] ++ Ecto.Adapters.Postgres.extensions(),
json: Poison)
But it's an umbrella app. I don't think it would be much different in a phoenix app. There I would also place postgrex_types.ex somewhere in lib.
Source : https://elixirforum.com/t/where-to-call-postgrex-types-define-in-phoenix-ecto-project/4810/2

I have faced with the similar errors:
(ArgumentError) invalid or unknown type Geo.Point for field


Fix ZeroDivisionError in constrained beam search with static shapes

What does this PR do?
Fixes ZeroDivisionError in constrained beam search for models with static shapes
python examples/text-generation/run_generation.py --model_name_or_path openai-community/gpt2 --use_kv_cache --max_new_tokens 20  --use_hpu_graph --bf16 --no-ignore_eos --prompt "The weather" --force_words "hot " --num_beams 10
Traceback (most recent call last):
  File "/root/optimum-habana/examples/text-generation/run_generation.py", line 692, in <module>
    main()
  File "/root/optimum-habana/examples/text-generation/run_generation.py", line 461, in main
    generate(None, args.reduce_recompile)
  File "/root/optimum-habana/examples/text-generation/run_generation.py", line 432, in generate
    outputs = model.generate(
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/root/optimum-habana/optimum/habana/transformers/generation/utils.py", line 1443, in generate
    result = self._constrained_beam_search(
  File "/root/optimum-habana/optimum/habana/transformers/generation/utils.py", line 3281, in _constrained_beam_search
    sequence_outputs = constrained_beam_scorer.finalize(
  File "/usr/local/lib/python3.10/dist-packages/transformers/generation/beam_search.py", line 850, in finalize
    beam_hyp.add(final_tokens, final_score, beam_indices=beam_index, generated_len=generated_len)
  File "/usr/local/lib/python3.10/dist-packages/transformers/generation/beam_search.py", line 965, in add
    score = sum_logprobs / (generated_len**self.length_penalty)
ZeroDivisionError: float division by zero

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

thanks @skavulya. Nice fix

@skavulya please run GAUDI2_CI=1 RUN_SLOW=true python -m pytest tests/transformers/tests/models/ -s -v before and after changes and make sure there is no new one is introduced.

@yafshar No new failures were introduced
This pr: = 25 failed, 874 passed, 326 skipped, 1 xpassed, 59 warnings in 1762.54s (0:29:22) =
Main: = 26 failed, 873 passed, 326 skipped, 1 xpassed, 59 warnings in 1343.83s (0:22:23) =
The additional failure in main is unrelated to this PR. It might be an intermittent issue.
FAILED tests/transformers/tests/models/wav2vec2/test_modeling_wav2vec2.py::Wav2Vec2RobustModelTest::test_ctc_train
FileNotFoundError: [Errno 2] No such file or directory: '/root/optimum-habana/tests/fixtures/tests_samples/COCO/000000039769.png'

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


[Suggestion] - Security Vulnerabilities on v8.5.2 and Helm Chart Doesn't Support v8.5.3

Suggestion
Hi there,
There is Security Vulnerabilities on 8.5.2 version of Confluence. It requires us to upgrade it to v8.5.3 but latest helm chart doesn't support v8.5.3 yet. Could you please provide insight on this?
Thanks.

Product
No response
Code of Conduct

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

Oh perfect! I didn't know that would work. Thank you for the quick response. Will test this now and then close the issue.

It worked. Thanks again. Closing the issue!


Order independence of enabling instrumentation vs importing instrumented libraries

I noticed that it seems there is an ordering dependency between enabling instrumentation and requiring instrumented libraries. It seems the libraries themselves have to be required before instrumentation can be applied.
https://github.com/anuraaga/sinatra-hello-world/blob/main/app-manual/app.rb#L1
I wonder if there isn't a way to remove this ordering constraint. I rarely use Ruby, and my fumbling didn't find any good leads, other languages allow intercepting the loading of a module / class, but I couldn't find anything similar for require. Hoping there may be one though that the ruby masters know about.
The context is I am working on autoinstrumentation of apps with OpenTelemetry without any code changes. For example, just add an annotation to a k8s pod and it gets instrumentation, all managed by the infrastructure without apps having to worry, even in Dockerfiles. This has worked well for Java, Python, and Node. For reference, the NodeJS approach which is closest to what I've been trying with Ruby
https://github.com/open-telemetry/opentelemetry-operator/tree/main/autoinstrumentation/nodejs
NODE_OPTIONS=--require autoinstrumentation.js
I am pretty close to having logic running automatically before a Ruby app using similar mechanisms. Because Ruby doesn't search paths similar to node's node_modules automatic lookup, I had to keep a manifest of required deps and add them myself but that is fine I think.
Overall README at https://github.com/anuraaga/sinatra-hello-world
Key file: https://github.com/anuraaga/sinatra-hello-world/blob/main/autoinstrumentation/opentelemetry-instrument.rb
Unfortunately, unlike Java, Python, Node, where instrumentation can be applied before the user imports the logic modules, it seems that that's not the case for Ruby. If I import use Bundler.require, it does import sinatra and instrumentation does succeed to apply, but then the app just doesn't load for some reason. Anyways this seemed too hacky and I didn't dig further than that, ideally the ordering constraint itself could be removed somehow.
Does anyone have any ideas if there is some magic that can be used to allow applying instrumentations before libraries are imported? I'm not a Ruby user myself but would love to provide Ruby users autoinstrumentation!

Thanks for sharing that specific examples with your repository.
I'll take a closer look next week but bundle exec should be calling bundle/setup to configure the load paths so you shouldn't have to manually set the load paths.
If you want to run your app without bundle exec then add require "bundler/setup" or invoke it Bundler.setup before running Bundler.require.
https://bundler.io/v2.2/guides/groups.html#how-to-manage-groups-of-gems
Have you tried the Sinatra example in this repository? Does that run for you?

Thanks @arielvalentin - the repository also has the app-manual folder which is a Sinatra app instrumented in the normal way for comparison and works fine. My objective is not to get an instrumented Sinatra app, but is to allow infrastructure-driven auto instrumentation of an app without any code changes in the app.
So regards bundler, the user is already using bundler with their own Gemfile - but I need to add search paths to OpenTelemetry modules which that bundle is not aware of. Ideally, bundler had  flag like --additional-bundle=path but I think the approach I am using is ok though and not too worried about it.
The main issue is just that I can't enable instrumentation before requiring the instrumented library, which I need for the approach for auto instrumentation. Hopefully going through the repo can give more clarity on what this is about.

I'm still not entirely clear on your goals here but it is indeed the case that the libraries must be required before the instrumentation is programmatically initialized. This step must occur before the application is started.
This is because most instrumentation is implemented as monkey patches or hooks that are prepended to the target libraries before the application is initialized.
Off of the top of my head I can't think of simple way of you adding instrumentation without changing the init script.
🤔 If your customer is providing you with a container, then you'd probably need to have a base image with the gems installed globally, but then bundler would ignore them but you'd still need to require the gems and then at a minimum call the "configure" method on the SDK.
That in and of itself necessitates a code change because this version of the SDK does not include a minimum  configuration script.
💡 I wonder if the SDK could provide such a script using the environment variables as a default.

The main intent of the issue is to get the experts (which I'm definitely not :) ) together to see if there may be an alternative mechanism for applying instrumentation that could work without the ordering restriction. The motivation is to allow apps to have telemetry with no knowledge of OTel at all - a base image or script would unfortunately defeat that I think as then the app still needs to use said mechanism, it's not completely free.
If someone can help with getting this line from that linked readme to function it would be awesome, including considering alternate monkey patching approaches in instrumentation.
RUBYLIB=../autoinstrumentation RUBYOPT="-ropentelemetry-instrument"  bundle exec ruby app.rb
If there is absolutely no way, it is what it is. But we got it for Java, Python, and Node - somehow I expected Ruby to be at least as dynamic so would also give us a path to true auto-instrumentation in some way. I'm still hoping, but it sounds like I shouldn't be too optimistic.

I played with trying to apply instrumentation in response to a library being required and came up with a pattern that sort of works
https://github.com/anuraaga/sinatra-hello-world/blob/main/autoinstrumentation/opentelemetry-instrument.rb#L5
With this, the autoinstrumentation works and traces get exported without the user's app needing any changes!
It currently tries enabling the SDK twice, presumably the self and name matching could be improved to help that. But I am wondering if an approach like this has ever been considered. Instrumentation could first check if method to instrument already exists, in which case monkey patch, and otherwise add a method_added hook to do it later. Again, I don't really know Ruby and don't know how reasonable / scalable this approach is so hoping to get some ideas from the experts. Naturally we would have liked to have a callback after require, but I couldn't find any such mechanism but hope that I'm just missing it.
I think independent of the issue of autoinstrumentation (which I think can be a gamechanger for adoption though), it is a nice user experience improvement to now have to worry about ordering here so if there is a reasonable approach to take to apply to the instrumentations, hopefully it could be considered.

@anuraaga
here's my two cents:

The canonical way to do this in ruby is supported via rails, using a railtie. You can review many of the popular tracing gems in ruby out there for implementations of this approach, here's the docs for the one I wrote at my previous employer that you can review https://github.com/DataDog/dd-trace-rb/blob/master/docs/GettingStarted.md#automatic-instrumentation.

I don't think this pattern is as well supported in other ruby frameworks, like sinatra, and implementations like what's been suggested are pretty incomplete imo (how would this work if a user is just using an http client instrumentation, or sidekiq, etc, we'd have to maintain these patches everywhere in every instrumentation, and how would we be able to have this feature play nicely with users who having existing configuration, or users who want to modify configuration).

Is there specification around this? an OTEP? I'm dubious on the benefits of taking on the support burden for this feature living within the SDK without some more robust specification for it. In practice, I've anecdotally seen this feature has a double-edge sword effect of helping adoption and also ballooning the amount of support. In practice, by encouraging adoption largely among folks who, as you put it, wish to have

telemetry with no knowledge of OTel at all

You very quickly find that you are suddenly doing enterprise support for people with the absolute least amount of interest in reading docs, reading debug logs, etc. Just my anecdotal experience :)
Tl;dr i'd prefer to see this feature live in vendor distributions of this library, via a Railtie.

It sounds like there is a reasonable approach for this for rails, but nothing generic for any ruby library. Given rails' popularity I guess it's OK and probably the best we can do so will go with that. I could verify that a rails initializer can be applied automatically and sends traces without app changes. Thanks for the clarification!


btc-wallet: margin cutoff issues on mobile

This has been fixed, and there will be many new UI updates soon based on the latest figma screens from @urcades.


[test-triage] edn_sec_cm failures

Hierarchy of regression failure
Block level
Failure Description
Started failing on 25th of July, 100% -> 0%.
  UVM_ERROR @  16565539 ps: (csr_utils_pkg.sv:476) [csr_utils::csr_rd_check] Check failed obs == exp (193 [0xc1] vs 382 [0x17e]) Regname: edn_reg_block.main_sm_state reset value: 0xc1
  UVM_INFO @  16565539 ps: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]
  --- UVM Report catcher Summary ---

Steps to Reproduce

GitHub Revision: e4c5daa580
dvsim invocation command to reproduce the failure, inclusive of build and run seeds:./util/dvsim/dvsim.py hw/ip/edn/dv/edn_sim_cfg.hjson \
  -i edn_sec_cm \
  --build-seed<PHONE_NUMBER>6894479494471273459769404654835266620222125964939301612221385668501 \
  --fixed-seed<PHONE_NUMBER>8848372465292589661168344678023408838600054326005816702909329066132 \
  --waves fsdb

Tests with similar or related failures
No response

Thanks for flagging this @jwnrt . This is related to the ECO fix (flopping prim_count error outputs). I've now filed a PR to fix this here: 24237.


typewrite when notepad is minimized

Hi All,
Is it possible for pyautogui to typewrite the keys in notepad when it is minimized?
Thanks,
Raja

Hi,
can someone provide details...
Thanks,
Raja


executing multiple commands with on key command

Hi everyone,
This is more a question than a issue but haven't really found any answer around so I decided to write here.
So I'm creating a custom build provider for openframeworks so I'm able to build and run OF projects from atom using the makefile.
I've been successful so far, I'm able to build and run the projects by specifying the build commands like this:
`    settings() {
  const args = [ `-j${atom.config.get('build-of.jobs')}`];
  const args2 = [ `-j${atom.config.get('build-of.jobs')}`, 'RunRelease'];

  return [
    {
      exec: 'make',
      name: `BUILD`,
      args: args,
      sh: false,
      keymap: 'alt-cmd-a',
      atomCommandName: 'build',
      errorMatch: errorMatch
  },
  {
    exec: 'make',
    name: `RUN`,
    args: args2,
    sh: false,
    keymap: 'alt-cmd-w',
    errorMatch: errorMatch
  }
];

}`
So pressing "alt-cmd-a" builds the project  and pressing "alt-cmd-w"  runs the projects.(so far so good)
But what I'd like to do is running those two build commands(sequentially build, run) with the same key command, @noseglid is it possible to do this?
Thanks in advance,
Best,
J

Hello,
This is not possible, and have been discussed multiple times. It's been decided agains with the reason that it is outside of the scope of this plugin. Chaining different targets together or handling dependencies between them is better suited for a build system (such as Make, which you're using).
To achieve what you want, I'd create a new target which first build then runs the project.
Good luck

Thanks for the answer @noseglid I'm not that familiar with Make but I'll give it a try.


Windows: Subsequent launching of Ray not working if 'On system tray' is chosen

Ray works fine as long as I don't set in preferences Show App -> On system tray.
By setting this it'll work for the current launch but whenever I tried to open the program again it didn't work.
Had to completely remove all files related to Ray manually because Reinstalling didn't help either (From \AppData\Local\ray-updater and \AppData\Roaming\Ray) and installing fresh again, but letting you know! I'll just use it with the app in task bar, no big deal.

Note: I'm using Windows 10 Home - 64 bits, in case it helps

I'll close the issue here as it's not related to laravel-ray but the software itself.
Issue added to the correct repo here.

I'll close the issue here as it's not related to laravel-ray but the software itself.
Issue added to the correct repo here.


Crash and weird behavior with unknown characters (ü and maybe others)

Recently I came across a problem with some maps that contained entities with the word "targetname" "tür".
It displayed the "ü" as a "?" and when selecting the name to change it the newbspguy did weird things with the character/letter input, deleting the last 2 chars, disappearing the entire input, etc, weird stuff.
I was not going to report this, but then today, it crashed because of that. I had to change all those characters to a normal character in a text editor then import the ent file again. I believe this is easily reproducible, just create a bunch of entities that trigger each other and have a ü and try to edit them later.
I believe there might be other characters that may cause this.

Cürsed

@GarompaEstomper latest newbspguy release has this issue?
I think if font not support this character can be entered.
Just don't use bad characters.


WIP: Translate "Implementation Notes"

🚧 Work in progress

This PR does not have any file change on it !
@mriiad Please check your work.

As written in the description, the work is in progress but I created it as @3imed-jaberi asked for and to allow him to assign the suitable task to me.
@atefBB please feel free to add the needed labels (work-in-progress, help wanted, etc.), otherwise I will convert it to draft before getting all the work done but I guess this is not the most practical way.

I think @3imed-jaberi means that you send a PR containing your work, not an empty one! Is that true @3imed-jaberi ?!

Creating a PR labelled 'work-in-progress' would allow him to assign the concerned task to me so others won't ask to do it.
It's kind of reserving the task and notifying interested people that it's in progress.
Absolutely, there will be upcoming commits soon.
But, maybe I'm wrong and there's another way of work so I would like to thank you for these notes and I'm waiting for Imad's instructions.

Hello,
As @atefBB says. and I see that you will do that.
I assign you on #1 for "Implementation Notes"

I think @3imed-jaberi means that you send a PR containing your work, not an empty one! Is that true @3imed-jaberi ?!


trust-manager: improve install instructions

Problem solved:
This PR makes the install instructions more readable ( adds newlines ).
This PR adds an explicit version to the install instructions, which is our recommended way of installing cert-manager (see https://cert-manager.io/docs/installation/helm/). This gets us a step closer to consistent instructions on all pages of the website.
Testing done:
Ran the install instructions.

This PR makes the install instructions more readable ( adds newlines ) and adds an explicit version to the install instructions.

What is the advantage of adding an explicit version?
I think it adds additional burden to the trust-manager release manager. They now have to remember to update this documentation variable. Have you updated the release process documentation for trust-manager?
And what happens when we update the versioned documentation? Will those past versions of cert-manager be forever locked to an older version of trust-manager and approver-policy? By versioned documentation I mean :

https://cert-manager.io/v1.13-docs, for example.

We want to advise people to explicitly provide the version when installing (same goes for https://cert-manager.io/docs/installation/helm/).
This will additionally notify the user in case of a out-of-date chart repo that does not have the specified version.

And what happens when we update the versioned documentation? Will those past versions of cert-manager be forever locked to an older version of trust-manager and approver-policy? By versioned documentation I mean :

We probably should not make the trust-manager and approver-policy versions dependant on the cert-manager version.
Maybe we should have a set of global variables that apply to all the cert-manager versions?

We want to advise people to explicitly provide the version when installing (same goes for https://cert-manager.io/docs/installation/helm/).

Why do we want to advise people to explicitly provide the version? I regard these as quick-start installation instructions and the changes here make the instructions more complicated.
And why do we want to advise people to upgrade cert-manager to 1.14.2 before they install trust-manager. What if they're trying this out on a cluster where cert-manager 1.12 is already installed and accidentally upgrade cert-manager.

This will additionally notify the user in case of a out-of-date chart repo that does not have the specified version.

But the first line is helm repo add jetstack https://charts.jetstack.io --force-update, so that should never happen, right?

As explained in the PR description, moving away from upgrade is not one of the problems this PR is solving.

@wallrj I removed the versions, will create another PR for approver-policy too.


Pagination? and 'Archives' page may be?

Currently all post are shown on the index page. After a time it will make site-loading slow and it is not necessary to scroll that long page in order to read some old post. We can set pagination i.e. to make Jekyll to serve posts on multiple pages.
On this regard, we can make also a page "archives" where all post could be listed.
I made a mirror here https://fingli.github.io/ - see how the pagination looks like and how it works. Archives are still in the process :) at the time of writing this I haven't uploaded it yet.

Great catch. If you have something to change that fact, I'm happy about it.

I checked your demo page. Seems to work just fine. Create a merge request so we update the main page.

Done :)
For some reason in these pull requests Github shows my synchronization commits as new, but I hope it will not broke the code.


Adding forked version of lib to be able to implement clear selection feature

Forked library to be able to add these clear selection "x"s.
Added to ALL dropdowns.

 Allow user to select blank if they made a mistake on the measures form

 remove/unselect regulation

 unselect Measurement unit & Measurement unit qualifier

@mauricioac need to specify the location of the fork

@matthewford I forked, but the actual lib is inside the "app/assets/javascripts/vendors".
Haven't pushed to my personal account.

Ideally we'd make a fork on BZ account and make a gem for it.
BUT, this is still selectize. Changing selectize to select2 is not actually easy, since some important features are missing on select2 as well.
So I thought we would leave the selectize -> select2 for beta, and just to allow users to clear selection, I have extended selectize.
If we'll keep using selectize, then yeah, making a proper fork and gem would be much better

oh ok yea we can push that to a bz fork - I thought you found a fork ;)


[REVIEW] Aadarsh805

GitHub Profile Link
https://github.com/Aadarsh805

Looks great, here are some suggestions https://www.youtube.com/watch?v=iqIFD02OkVE

suggest removing the widgets
I don't think "views" adds values
try to do more code reviews


TypeError: '/home/data/initial_training/training/clip1.avi' has type str, but expected one of: bytes

I am working on fine-tuning the model for my own dataset. I installed all the dependency packages and the model works fine when I set use_list=0 (in create_video_db.py) and use_local_file=0(in train_net.py)
I would like to load the videos directly from my local machine and Hence I would like to run create_video_db.py with "use_list=1".  When I run the data/create_video_db.py with "--use_list=1", I get the above mentioned error. The traceback is as follows:
Traceback (most recent call last):
File "data/create_video_db.py", line 152, in
main()
File "data/create_video_db.py", line 146, in main
args.num_epochs
File "data/create_video_db.py", line 89, in create_video_db
video_tensor.string_data.append(video_data)
TypeError: '/home/data/initial_training/training/clip1.avi' has type str, but expected one of: bytes
I created the list file for my dataset by referring to the given list files in the repo, with columns "org_video" and "label".  Could anyone explain what is causing this error and how can I fix this?
Thanks in advance.

This is more like python version warning, are you using python 3.0? The script was not tested with Python 3 yet, should be an easy fix though.

Yes. I am using Python 3.7. I will look into it and post the findings here. Thank you.

The issue can be solved by converting the variable "file_name" in create_video_db.py from str to byes.
file_name = file_name.encode('utf-8)
will resolve this error.


CI: Refact and extend CI/CD workflows to leverage ansys actions

[!IMPORTANT]
One of the new steps requires that a new ansys/actions is released. This PR shouldn't be merged as long as 5.1.9 is the latest version released.

List of the worfklows to validate in the new one (aggregation into one workflow):

[x] Doc Style
[x] Doc Build (see #4296 ~if we keep print_errors.py or leverage option -W of sphinx-build, or something else~)

[x] Add install option [doc-noexamples] to remove a lot of unneeded dependencies

[x] Test
[x] Test solver
[x] Iron Python
[x] Wheelhouse and smoke tests
[x] Full documentation (moved to CICD)
[x] Doc build upload and update dev index (moved to CICD)
[x] Cpython linux (test and test solver)
[x] PR title

Extra workflows:

~[Nightly] Cpython linux (test and test solver)~
~[Nightly] Doc build with upload and update dev index~
[Nightly] Test pre release

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.74%. Comparing base (61f4d81) to head (e17691b).
 Consider uploading reports for the commit d111305 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4284      +/-   ##
==========================================
+ Coverage   81.73%   81.74%   +0.01%
==========================================
  Files         107      107
  Lines       51788    51788
==========================================
+ Hits        42328    42334       +6
+ Misses       9460     9454       -6

@Samuelopez-ansys FYI there was still some issues with our self hosted runners and I had to make a few changes in them. Hoperfully this will lead us to a working PDF with examples 🤞

@Samuelopez-ansys The PR is ready to be merged !

Thanks @SMoraisAnsys . Great work that would be extremely valuable for us.


only ping those that haven't turned in reviews

e.g, https://github.com/ropensci/onboarding/issues/56#issuecomment-237994798 and the one 5 days before, pinged both people even though one had already turned in review
fix it 🔧

leaving open for now, check again after a week or so to see if this doesn't happen or does


How to convert the coronal scans into a nifti image?

Hi,
When I use MRIcroGL to convert a coronal scan into a nifti image, I have a problem. I try two ways to do this by using MRIcroGL. The first is to save nifti images directly by "File-> Save NIFTI". The second is to save nifti images by the following command “dcm2niix.exe -f  'output_filename' -i y -l y -p y -x I y -v y -z y -o 'output_path' 'input_path' “.  Both of the above-generated results are different.  The image shape using the first way is  (X, Y, Z): 320 x 85 x 320, which is my desire. However, the image shape using the second way is (X, Y, Z): 320 x 320 x 85.  Because I will deal with a lot of coronal Dicom files, I want to use the second way for batch processing. But it will happen the above error. How to do this using the command?
Best wishes,
Jianhong Cheng

You do not mention the version of MRIcroGL or the operating system. I am going to assume you are using the most recent release. You should only supply one argument to -x, so in your sample above I would think you meant dcm2niix.exe -f 'output_filename' -i y -l y -p y -x i -v y -z y -o 'output_path' 'input_path' , where -x i specifies you do not want 3D acquisitions losslessly rotated to the axial plane.
Remember, this only changes the order of the rows, columns and slices of image data saved to disk. The storm/qform always stores the transformation to world space. Therefore, an image saved as LPS (axial slices) and LSP (coronal slices) on disk should appear identically with a viewer that reads the spatial transform (like MRIcroGL). If your subsequent tool ignores the spatial transform, I would upgrade it to support this (if not, I would be worried it may not distinguish images that are left-right flipped).

@neurolabusc
Hi, It still does not work using MRIcroGL 1.2.20200331 on windows 10.0.  I upload my test sample and results. flair.zip is coronal FLAIR Dicom files. Flair_firstway.nii.gz is generated by "File-> Save NIFTI", which is my desire. Flair_second_way.nii.gz is generated by your provided the command dcm2niix.exe -f 'output_filename' -i y -l y -p y -x i -v y -z y -o 'output_path' 'input_path'. But it still not the same as  Flair_firstway.nii.gz.
flair.zip
Flair_firstway.nii.gz
Flair_second_way.nii.gz

The -xargument will only rotate 3D acquisitions. Your FLAIR scan is a 2D sequence. dcm2niix preserves the slice orientation of 2D sequences, as required by slice timing tools. This 2D series was acquired as coronal slices and is saved as coronal slices. dcm2niix is designed to losslessly rotate images when it will not impact post processing. You can always use a simple script or tool to re-orient your images. For example, the i2nii argument -r y will rotate images to match the NIfTI identity matrix (e.g. resliced to axial).
However, be aware that these changes are lossless. Any tool that respects the sform should show an image identically before and after rotation.
Chris Rorden's dcm2niiX version v1.0.20200331  (JP2:OpenJPEG) (JP-LS:CharLS) Clang11.0.0 (64-bit MacOS)
Found 18 DICOM file(s)
...

>fslhd output_filename
filename	output_filename.nii.gz
sizeof_hdr	348
data_type	INT16
dim0		3
dim1		512
dim2		512
dim3		18
...
sform_name	Scanner Anat
sform_code	1
sto_xyz:1	-0.448568 0.010242 -0.375193 117.267929
sto_xyz:2	0.021857 -0.001620 -7.690826 57.850594
sto_xyz:3	0.010309 0.449099 -0.019191 -104.825638
sto_xyz:4	0.000000 0.000000 0.000000 1.000000
sform_xorient	Right-to-Left
sform_yorient	Inferior-to-Superior
sform_zorient	Anterior-to-Posterior

I want to do the following preprocessing: LPS, co-registered, and skull stripped after converting the nii image.  I find that the coronal FLAIR scans are executed by "File-> Save NIFTI", the generated images will be helpful for the above preprocessing.

Great. Sounds like you have found a solution. If you want to automate this, you could use a tool like i2nii.


In new version of WebIDL2 parser, the acceptable syntax changed so we…

… need to do some adaption.
Closes #115

Merging since trybot is green...

Great, and thx


Add OpenAPI responses examples

Nowadays, the Sanic OpenAPI extension does not permit to add example requests and responses to your documented views.
I suggest to add a field example to definitions.Response, definitions.RequestBody and definitions.Parameter to support OpenAPI examples.

The class definitions.Components references to OpenAPI examples -I guess-, but is unused. Maybe it's a WIP.
I don't know if it's on the roadmap or actually documented (since I read carefully the documentation and the code and I didn't find any traces of this feature, besides in this class) but it could be a cool feature. The thing that, personally, I miss the most about the OpenAPI standard.
I could open a PR too, if need. I'd be glad to contribute to Sanic Extensions. Just need an approval that what I suggest makes sense!
Thanks ❤

Makes absolute sense, and I would happily welcome a PR. The code base was largely copied from the earlier sanic-openapi package, which had very limited type hinting support. Because of that, there are a lot of fields that are implicit and not easily discovered. This is something that needs to be resolved in the long run. In the short run, I think that it exposed a problem that example is not one of them as you indicated.

Cool! I was thinking is that since auto-generated examples could lead to unreal parameters examples, it could be useful to have a FactoryBoy-ready type approach, where example accepts a an instance of RequestBody.body, Response.content etc
So you could have
@openapi.definition(
    summary="Create a new MusicID",
    description="Create a new **MusicID** via a deserialized MusicID",
    body=RequestBody(serializers.MusicIDRequest, example=MusicIDRequestFactory.build()),
    response=[
        oa.Response(Error, status=429, description="too many requests"),
        oa.Response(Error, status=400, description="bad request"),
        oa.Response(Error, status=401, description="user not authenticated"),
        oa.Response(serializers.MusicIDResponse),
    ],
)

So the example fields would be realistic since we'd permit the usage of Faker, FuzzyAttributes...  and the User could  match the criteria of the field
class MusicIDRequestFactory(factory.Factory):
    """
    MusicIDRequest factory
    """

    name: str = factory.Sequence(lambda n: "Section configuration %s" % n)
    identifier: str = faker.Faker('uuid')
    creation_title: str = factory.Sequence(lambda n: "Create your Section %s!" % n)
    source: str = fuzzy.FuzzyChoice(SectionConfig.Source, getter=lambda x: x[0])
    optional: bool = False
    max_items: int = fuzzy.FuzzyInteger(1, 10)

    class Meta:
        model = MusicIDRequestFactory

I'd love to hear your thoughts about it

Ooo! That's a nice touch. I'm on board with this. Let me know if you want to spec it out with me, or if you want to just throw together a proposal first.


Stripe still warns about possible SCA issues

The Stripe account still has a warning pointing to:
https://stripe.com/docs/strong-customer-authentication#preparing
We have to fix this ASAP, since otherwise, many banks could be rejecting payments.

We are using a recent version of the Python Stripe API (which is published by Stripe itself).
In the API docs, I found a hint that the Stripe keys determine which version of the API we are using. In the "Developers" section of the Stripe account, we are listed as using the 2017 version of the API, which most likely is outdated. I'll try to bump this to the current version.

API upgrade guide: https://stripe.com/docs/upgrades?since=2017-02-14#how-can-i-upgrade-my-api

Rolled all Stripe keys.

Tested with new keys and settings: purchases continue to work.

The warning is still shown in the Stripe account. Closing for now.


AlfredKat CLI is stripped off com.apple.quarantine at each start

this is needed to avoid the dialog telling users the Workflow has been downloaded from the Internet, do they want to open blah blah blah.
but the strip doesn't need to happen every time. only at the first use after download, including updates.

same for the Updater. don't care. not gonna codesign the command line tools. mega headache.


Possible violation of Mono.Cecil license

The LICENSE file for Mono.Cecil states:

Copyright (c) 2008 - 2015 Jb Evain
Copyright (c) 2008 - 2011 Novell, Inc.
<snip>
The above copyright notice and this permission notice shall be
included in all copies or substantial portions of the Software.
<snip>

However neither of those copyrights are presence in this repository's LICENSE file which, judging by API names and the README's own admission, appears to be a "substantial portion" of the original Cecil code.

@yaakov-h
Those of us who distribute under the MIT license are generally doing so with full knowledge that anyone can do anything with our code, open source or commercial, and that recognition of authorship is always optional. At least that has been how I look at my work. :-)
IANAL but my understanding is that the sentence you cite from the MIT license applies to distributing Mono.Cecil, not to incorporating portions of its source in a new work. In fact, according to references I have read, I could have used an entirely different license if I wished to. This is, of course, an amateur's interpretation!
What's more important to me is how @jbevain may feel about this. Jb, I have a vague memory of communicating with you about this at some point before I did it. It was initially just for my testcentric engine but I then packaged it so we could use it on NUnit. If it were a revised version of Mono.Cecil I'd have simply put my copyright on top of yours but since it's really a different program with substantially fewer features but able to run on legacy platforms that didn't seem right. In the past I've used a NOTICES file to simply recognize authorship and I'd be happy to do that here.

Hello folks,

Those of us who distribute under the MIT license are generally doing so with full knowledge that anyone can do anything with our code, open source or commercial, and that recognition of authorship is always optional. At least that has been how I look at my work. :-)

For sure :)
I'm personally happy with whatever you decide Charlie: a NOTICES file would be nice with the copyright holders would be nice.

Proposed content of NOTICES.md...
TestCentric.Metadata is substantially based on Mono.Cecil, which is
Copyright (c) 2008 - 2015 Jb Evain
Copyright (c) 2008 - 2011 Novell, Inc.

We thank Jb Evain, the original author of Mono.Cecil, for his work.
See the README file for a description of the scope of TestCentric.MetaData
as compared to Mono.Cecil.


Add documentation to all generated functions and respect visibility

Currently ::new() comes without documentation. But more importantly, it doesn't respect the annotated visibility of items.
The generated struct is declared public, the getters are public, etc. Instead it should use the visibility declared in on the fields and use that for the declared setters and getters.
As a workaround, I currently wrap all #[bitfield]s in a private module and expose them only through a wrapper.

That's a really good idea!


feat(pid_longitudinal_controller): improve delay and slope compensation

Description

In the current implementation on Autoware, while calculating the output command, we are compensating for the slope and delay in pid_longitudinal_controller.
We are doing the delay compensation by using the estimation of the drive distance of the vehicle after delay_compensation_time and we are looking at the future TrajectoryPoint and calculating the output command with respect to this future point.
However, before sending the output command, we are also doing slope compensation with respect to Current TrajectoryPoint. However, in some scenarios on hilly roads, this way is causing some conflicts because of the time like below.

As you can see in this scenario, there is a conflict between slope and delay compensation because they are using different reference times. In this scenario, the node will send FF with respect to the target point however it sums negative slope compensation value because of the current position of the vehicle and it causes undesired behavior. To avoid this behavior, I made some changes to delay and slope compensation in this package to sum compensations at the same time.
Related links

Tests performed

I tested on the real vehicle and also on the planning simulator. It solves our problems in the real environment and there is no behavioral change in the planning simulator.
Notes for reviewers

Interface changes

Effects on system behavior

The only effect is that we will use the raw pitch info at only starting (while not moving), if the vehicle is moving, it needs the future pitch info, to achieve this, it depends on the pitch values of the trajectory points.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@brkay54 Thank you for this PR. Recent PRs have caused conflicts. Would you fix it? Sorry for your inconvenience.

Hi @TakaHoribe sorry for the conflicts. I updated all the PRs.

Hi @TakaHoribe thank you for your review. I updated the branch w.r.t. your feedback.
Just I didn't update the code snippets related to:
https://github.com/autowarefoundation/autoware.universe/pull/4712#discussion_r1342496103
https://github.com/autowarefoundation/autoware.universe/pull/4712#discussion_r1342528866
I am waiting for your feedback! Thank you so much for your effort ❤

Hi @TakaHoribe, sorry to disturb you, but just I want to kindly ping you.

@brkay54 Sorry for my late reply

Actually, the pitch information from sensing would be more reliable from the trajectory.

This depends on the users. For the pitch source, we still need to keep the sensor-only option for users who can not have an accurate map.
So I would suggest this way:
Provide three options:

vehicle_pitch: Use the vehicle's pitch value as a slope. The information derived from the vehicle's pitch is often not an accurate representation of the road gradient due to effects such as body vibrations.
trajectory : Use the trajectory slope value as a slope. Calculating the precise road gradient is possible with trajectory data; however, it requires high-quality map information.
trajectory_adaptive: Primarily, the gradient is calculated using trajectory data, but if the gradient cannot be computed at the end of the route or similar situations, the vehicle pitch is used instead.

The default could be vehicle_pitch or trajectory_adaptive.
What do you think?

@TakaHoribe Thank you for the suggestion. It makes sense to me. I think it is better to make trajectory_adaptive as the default option because it can work with delay compensation better.
For the trajectory_adaptive option, if it is okay, I am planning to make it adaptive with respect to the velocity of the vehicle. It will use the vehicle_pitch in low velocities (because in low velocities, the target trajectory point gonna be very close to the vehicle).

@TakaHoribe Thank you for the suggestion. It makes sense to me. I think it is better to make trajectory_adaptive as the default option because it can work with delay compensation better.
For the trajectory_adaptive option, if it is okay, I am planning to make it adaptive concerning the velocity of the vehicle. It will use the vehicle_pitch in low velocities (because in low velocities, the target trajectory point gonna be very close to the vehicle).

Hi again @TakaHoribe -san, I wrote like this but after thinking more, IMO, 2 options would be better rather than using 3 options.
Using only trajectory has a problem for the last point, as you said, and also for all cases, we will have vehicle pitch value. I think the trajectory option is not necessary because we will need the future slope only for higher velocities, for lower velocities we can use vehicle_pitch (BCS target waypoint will be near ego). If we do not assume that the user may not have vehicle pitch value, the adaptive_trajectory option is always the better option than trajectory.
My proposal is that like that:
# slope compensation
adaptive_trajectory: true  # if true, trajectory is used as pitch information source for higher velocities and vehicle pitch for lower velocities. if false, vehicle_pitch is used as pitch information.
adaptive_trajectory_treshold: 1.0 # available only adaptive_trajectory is true

What do you think?

Hi again @TakaHoribe -san, I updated the PR w.r.t my comment above. Also, I updated the launch PR and readme PR. Please let me know if you have any concerns.

@brkay54 Thank you for addressing my comments.

because we will need the future slope only for higher velocities, for lower velocities we can use vehicle_pitch

This is only in the ideal case. We need to care about users who can not use an accurate map or rich sensor for pitch estimation.
Trajectory-only option is necessary for users with poor sensors.
Pitch-only option is necessary for uses with poor maps.
Adaptive can be available only for users with both ich sensors and rich maps.

Hi @TakaHoribe -san, sorry to be late I couldn't find time to update the PR in the last weeks.
I updated the slope source option as you said above. You can find the changes in the launch file here. It is ready for review, please let me know if any issue exists.
Also, after this PR, the following PRs should be merged by order here:
1- https://github.com/autowarefoundation/autoware.universe/pull/5077
2- https://github.com/autowarefoundation/autoware.universe/pull/5079
3- https://github.com/autowarefoundation/autoware.universe/pull/5080
4- https://github.com/autowarefoundation/autoware.universe/pull/5081

@TakaHoribe-san, Just to remind: Also, after this PR, the following PRs should be merged by order here:
1- https://github.com/autowarefoundation/autoware.universe/pull/5077
2- https://github.com/autowarefoundation/autoware.universe/pull/5079
3- https://github.com/autowarefoundation/autoware.universe/pull/5080
4- https://github.com/autowarefoundation/autoware.universe/pull/5081
Also, Horibe-san, I made some critical changes to the package. If you don't mind, I would like to maintain this package. Can I add myself as a maintainer?

@brkay54

Can I add myself as a maintainer?

I think it is fine. I'll talk with @takayuki5168


Segfaulting without tracing enabled.

After setting up opencensus-php for a project i noticed segfaults of a specific url.
After a bit of debugging it's seems to be partially related to call depth. I was able to reproduce this with a simple example:
https://github.com/foosinn/opencensus-segfault
This is throwing an segfault:
docker build -f Dockerfile-broken -t test . && docker run -it -v $PWD/script.php:/script.php test

While this is not:
docker build -f Dockerfile-working -t test . && docker run -it -v $PWD/script.php:/script.php test

The only difference is installing the opencensus PHP extension. The script.php doesn't use tracing at all, the only difference is installing the module.
Sidenote: In the original code it also made a difference weather i specified a third parameter for the results to preg_match. I wasn't able to create a simple case for that, but i guess the issue is in both cases the same.
Thanks for your help!

At which values of MAX do the segfaults start? The Opencensus extension intercepts every PHP method call; I could imagine that that always adds a bit of overhead to the stack, either in the form of extra on-stack allocations and probably also additional function calls (e.g. due to the "replaced" method call function calling through to the "original" one).

Its hitting at about 32000 (it's always changing a bit, bunning the script displays it in 1000 iteration steps). Note that this runs only with the extension enabled, but no tracing at all configured.
The affected code seems to need a lot less recursions, but of course there are more variables and other things happening which would also fill up the stack i think.
I guess even if there is more memory used, it should never segfault? Also are there ways to increase the available memory?

Note that this runs only with the extension enabled, but no tracing at all configured.

That's to be expected, as with the current way the extension is implemented, every single method call needs to be intercepted, regardless whether it is traced or not.

I guess even if there is more memory used, it should never segfault? Also are there ways to increase the available memory?

The segmentation fault is probably not due to running out of heap memory, but rather out of stack memory, which is much more limited (~512kB by default on some OSes I think, but not 100% sure). Changing the stack size might be possible on an OS limit, see this very old article on a similar problem which uses ulimit for that.
Again, I am not an expert on this, it's just my guess.

Thanks so much, i didn't realize that this was a operating system limit.
For the record, adjusting the httpd Systemd Service fixes this:
# /etc/systemd/system/httpd.service.d/override.conf
[Service]
LimitSTACK=16777216

Should this be somehow documented if others run into this?


Possibility to change input field font type

Hi! I'm missing the possibility to change the search input fields font. As of now, it's only possible to change category-font-color, category-font-size, input-font-color and input-font-size.

Hi, as of right now you can either set the font-family outside of the element and have it inherit in, or you can add whatever styles you want using this technique: https://github.com/nolanlawson/emoji-picker-element#custom-styling
Hope that helps! Thanks


Collapse API versions if APIs only have a single version

Currently, the DLP API (as an example) has a documentation hierarchy like this: DLP → v2beta1 → DLPServiceClient.
Would it be possible to reduce this to: DLP → DLPServiceClient ?

We are probably not going to make any changes to our GitHub hosted docs, as our repos start to migrate out onto cloud.google.com.


Docker 1.7.0-rc1 fails to shutdown cleanly using systemd

Using docker-engine-1.7.0-0.1.rc1.fc21.x86_64
$ sudo systemctl start docker
$ sudo systemctl stop docker
# hangs here for a while, ultimately times out and is killed

Here are the journald logs:
May 28 09:39:57 localhost.localdomain systemd[1]: docker.service stop-sigterm timed out. Killing.
May 28 09:39:57 localhost.localdomain systemd[1]: docker.service: main process exited, code=killed, status=9/KILL

Here's the output from SIGUSR1:
May 28 09:38:48 localhost.localdomain docker[32111]: time="2015-05-28T09:38:48.399283895-04:00" level=info msg="=== BEGIN goroutine stack dump ===
goroutine 10 [running]:
github.com/docker/docker/pkg/signal.DumpStacks()
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/pkg/signal/trap.go:60 +0x7a
github.com/docker/docker/daemon.funcÂ·021()
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/daemon/debugtrap.go:18 +0x6d
created by github.com/docker/docker/daemon.setupSigusr1Trap
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/daemon/debugtrap.go:20 +0x18e

goroutine 1 [chan receive]:
main.mainDaemon()
	/root/rpmbuild/BUILD/docker-engine/docker/daemon.go:179 +0x10ae
main.main()
	/root/rpmbuild/BUILD/docker-engine/docker/docker.go:92 +0x6b4

goroutine 5 [syscall]:
os/signal.loop()
	/usr/local/go/src/os/signal/signal_unix.go:21 +0x1f
created by os/signal.initÂ·1
	/usr/local/go/src/os/signal/signal_unix.go:27 +0x35

goroutine 17 [syscall, locked to thread]:
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2232 +0x1

goroutine 25 [chan receive]:
database/sql.(*DB).connectionOpener(0xc208193f40)
	/usr/local/go/src/database/sql/sql.go:589 +0x4c
created by database/sql.Open
	/usr/local/go/src/database/sql/sql.go:452 +0x31c

goroutine 9 [chan receive]:
github.com/docker/docker/api/server.(*Server).newServer(0xc2080dcf80, 0x7ffdbe375f7c, 0x2, 0x7ffdbe375f7c, 0x0, 0x0, 0x0, 0x0, 0x0)
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/api/server/server_linux.go:42 +0x3d3
github.com/docker/docker/api/server.(*Server).ServeApi(0xc2080dcf80, 0xc2080946f0, 0x1, 0x1, 0x0, 0x0)
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/api/server/server.go:96 +0x252
main.funcÂ·003()
	/root/rpmbuild/BUILD/docker-engine/docker/daemon.go:129 +0x5f
created by main.mainDaemon
	/root/rpmbuild/BUILD/docker-engine/docker/daemon.go:135 +0x6bc

goroutine 26 [IO wait]:
net.(*pollDesc).Wait(0xc20842b720, 0x72, 0x0, 0x0)
	/usr/lo
May 28 09:38:48 localhost.localdomain docker[32111]: cal/go/src/net/fd_poll_runtime.go:84 +0x47
net.(*pollDesc).WaitRead(0xc20842b720, 0x0, 0x0)
	/usr/local/go/src/net/fd_poll_runtime.go:89 +0x43
net.(*netFD).readMsg(0xc20842b6c0, 0xc208114850, 0x10, 0x10, 0xc208664420, 0x1000, 0x1000, 0xffffffffffffffff, 0x0, 0x0, ...)
	/usr/local/go/src/net/fd_unix.go:296 +0x54e
net.(*UnixConn).ReadMsgUnix(0xc208034af0, 0xc208114850, 0x10, 0x10, 0xc208664420, 0x1000, 0x1000, 0x50, 0xc208114464, 0x4, ...)
	/usr/local/go/src/net/unixsock_posix.go:147 +0x167
github.com/godbus/dbus.(*oobReader).Read(0xc208664400, 0xc208114850, 0x10, 0x10, 0xc208664400, 0x0, 0x0)
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/transport_unix.go:21 +0xc5
io.ReadAtLeast(0x7f3e5ab1f960, 0xc208664400, 0xc208114850, 0x10, 0x10, 0x10, 0x0, 0x0, 0x0)
	/usr/local/go/src/io/io.go:298 +0xf1
io.ReadFull(0x7f3e5ab1f960, 0xc208664400, 0xc208114850, 0x10, 0x10, 0x50, 0x0, 0x0)
	/usr/local/go/src/io/io.go:316 +0x6d
github.com/godbus/dbus.(*unixTransport).ReadMessage(0xc2084c7e10, 0xc2083e9200, 0x0, 0x0)
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/transport_unix.go:85 +0x1bf
github.com/godbus/dbus.(*Conn).inWorker(0xc2083e67e0)
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/conn.go:248 +0x58
created by github.com/godbus/dbus.(*Conn).Auth
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/auth.go:118 +0xe84

goroutine 27 [chan receive]:
github.com/godbus/dbus.(*Conn).outWorker(0xc2083e67e0)
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/conn.go:370 +0x58
created by github.com/godbus/dbus.(*Conn).Auth
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/godbus/dbus/auth.go:119 +0xea1

goroutine 28 [chan receive]:
github.com/coreos/go-systemd/dbus.funcÂ·001()
	/root/rpmbuild/BUILD/docker-engine/vendor/src/github.com/coreos/go-systemd/dbus/subscription.go:70 +0x64
created by github.com/coreos/go-systemd/dbus.(*Conn).initDispatch
	/root/rpmbuild/BUILD/docker-engin
May 28 09:38:48 localhost.localdomain docker[32111]: e/vendor/src/github.com/coreos/go-systemd/dbus/subscription.go:94 +0x11c

goroutine 29 [chan receive]:
github.com/docker/docker/daemon.(*statsCollector).run(0xc20865d5c0)
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/daemon/stats_collector.go:89 +0xb2
created by github.com/docker/docker/daemon.newStatsCollector
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/daemon/stats_collector.go:29 +0x116

goroutine 106 [chan receive]:
main.funcÂ·004()
	/root/rpmbuild/BUILD/docker-engine/docker/daemon.go:164 +0x6e
github.com/docker/docker/pkg/signal.funcÂ·001(0x7f3e5ab00f68, 0xc2081142b0)
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/pkg/signal/trap.go:38 +0x1b6
created by github.com/docker/docker/pkg/signal.funcÂ·002
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/pkg/signal/trap.go:53 +0x13a

goroutine 105 [IO wait]:
net.(*pollDesc).Wait(0xc20842bbf0, 0x72, 0x0, 0x0)
	/usr/local/go/src/net/fd_poll_runtime.go:84 +0x47
net.(*pollDesc).WaitRead(0xc20842bbf0, 0x0, 0x0)
	/usr/local/go/src/net/fd_poll_runtime.go:89 +0x43
net.(*netFD).accept(0xc20842bb90, 0x0, 0x7f3e5ab05160, 0xc208114210)
	/usr/local/go/src/net/fd_unix.go:419 +0x40b
net.(*UnixListener).AcceptUnix(0xc2081b8300, 0xc2089f4e38, 0x0, 0x0)
	/usr/local/go/src/net/unixsock_posix.go:282 +0x56
net.(*UnixListener).Accept(0xc2081b8300, 0x0, 0x0, 0x0, 0x0)
	/usr/local/go/src/net/unixsock_posix.go:293 +0x4c
net/http.(*Server).Serve(0xc2087082a0, 0x7f3e5ab0cfb8, 0xc2081b8300, 0x0, 0x0)
	/usr/local/go/src/net/http/server.go:1728 +0x92
github.com/docker/docker/api/server.funcÂ·009()
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/api/server/server_linux.go:38 +0xdb
created by github.com/docker/docker/api/server.(*Server).newServer
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/api/server/server_linux.go:39 +0x35e

goroutine 103 [chan receive]:
github.com/docker/docker/pkg/signal.
May 28 09:38:48 localhost.localdomain docker[32111]: funcÂ·002()
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/pkg/signal/trap.go:29 +0x8f
created by github.com/docker/docker/pkg/signal.Trap
	/root/rpmbuild/BUILD/docker-engine/.gopath/src/github.com/docker/docker/pkg/signal/trap.go:55 +0x250

=== END goroutine stack dump ==="

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

@jfrazelle

Remember you guys when I said this was happening you were just like "make your timeout shorter", yeahhhhh

I found issue, thinking how to fix.


Setup for Customizing the 'music-encoding' Tools

As announced in the last MEI community forum, we at the Digital Mozart Edition (dme.mozarteum.at) are about to adopt the current music-encoding tool for building our own guidelines,  a process that could also be interesting for other projects as we hope. First step will be generation of the web version of the guidelines. To ensure that our developments have also a return of invest to the MEI community, we would like to discuss the basic integration setup in this issue.
Considering our TEI source and examples, we have aligned the structure with that of the music-encoding/source. Additionally, we have copied assets and other artifacts there:

This repository is hosted on our GitLab server, which is currently only for internal use.
We are currently considering two possible integration setups. In both cases, development is taking place on our fork of the music-encoding repository.
First setup
The music-encoding/source folder is replaced by our source files which are embedded as a submodule:

[submodule "source"]
	path = source
	url = https://gitlab.mozarteum.at/dme/dime/dime-guidelines.git

This setup allows for running the ant target build-guidelines-html without any modifications.
Second setup
The music-encoding repository itself is embedded as a submodule of our guidelines repository:

[submodule "music-encoding"]
	path = music-encoding
	url = https://github.com/ism-dme/music-encoding.git
  branch = dime-guidelines-test

This setup does not affect the original music-encoding/source folder in any way. However, the odd2html XSLT-files have to be adopted to allow processing the MEI source located at a different location.
Currently, after initial discussions with @bwbohl  and @musicEnfanthen , we tend towards adopting the second setup. Any other proposals are welcome!

@music-encoding/technical-team Any second thoughts?

If this is mainly about providing your Guidelines, then I would go the easy way, which you described in your First Setup. Otherwise you have to adjust files anyway, and in this case it is not clear to me where the benefits in messing around with submodules are.


Improve SSH authentication

This adds support for using the SSH agent when using the ssh transport to libvirt. Tested locally that using a key file still works
Fixes #864
Other methods can be added, but since they're mostly interactive, they don't make much sense for Terraform.
There are also commits for that add support for disabling SSH host key verification and fixes a minor bug with automatic user name selection.

@dmacvicar any chance to get this merged? I haven't needed it for a while but it seems other people are also struggling with the lack of support for SSH agents.

Will have a look at it soonish.

I have finally managed to test this. Thanks for your contribution Jarkko!


adding install instructions

Thanks for the contribution! Are you using browserify too? I haven't tested that configuration -- I think the best way to use Leaflet.fullscreen is via the Mapbox CDN.


comet simplifications

@mr-c thank you for pointing that out, and sorry for the late reply.
For simplicity reasons, we want to be able to generate workflows and input.yml files that align with the .cwl tool descriptions automatically. Therefore, we don't use arbitrary labels for annotations atm.
I added an issue on the APE side to generate more descriptive labels.


Support Drush 9.0.0-rc1 +

Fixes issue #655

Merged for release on approx. 6th December

Not related to this issue, but just mentioning. Drush 9-rc1 doesn't seem to work properly on platform. Running a drush cim is giving me:
 [error]  Directory /app exists, but is not writable. Please check directory permissions.

  [Symfony\Component\Console\Exception\CommandNotFoundException]
  Command "cim" is not defined.

Apparently as /app is not writable, config commands are not even exposed to drush.

@hanoii add a mount for .drush (e.g. https://github.com/platformsh/platformsh-example-drupal8/pull/48/files)

@pjcdawkins thanks!


Add ability to cancel a Stage/Migrate in progress

Backend issue to support:  https://github.com/fusor/mig-controller/issues/237

From a UI standpoint here's what's needed:

Add a Cancel kebab action to in progress stage/migrate
Display current migration state as Cancelling stage/migrate
When cancel is complete show a success/failure toast notification

Will provide mockups later.

https://issues.redhat.com/browse/MIG-118

From the controller issue, mockups: https://docs.google.com/presentation/d/1Fv8pUuAl48umIjJKEkyw1TUWR_IfMp5w8HuZXe4-l9c/edit#slide=id.p


Bug: Using flight roll not gimbal in re projection test

It looks like we have a small bug? XMP:FlightRollDegree is being renamed Gimbal roll. Gimbal roll is usually 0 degrees!
gimbal_angles = df_meta[['XMP:GimbalYawDegree', 'XMP:GimbalPitchDegree', 'XMP:FlightRollDegree','XMP:RelativeAltitude']]
gimbal_angles = gimbal_angles.rename(columns={'XMP:RelativeAltitude':'UAV_Altitude',
'XMP:GimbalYawDegree':'Gimbal_Yaw',
'XMP:GimbalPitchDegree':'Gimbal_Pitch',
'XMP:FlightRollDegree':'Gimbal_Roll'})

Good pick-up Nick, you are correct!
I have fixed that in d6afd6e5c05d5eb7c0b67565c6ba6cbf9fecea3d


Backlight on Pimoroni Pico Display (135x240) Does not work

Hi,
I have tried Release 19.6 and 19.8 on a Pico attached to a DS3231 RTC and a Pimoroni Pico Display (135x240) and whilst all the functions that I use work, the Backlight stays on full no matter what the setting is.
The same code on the larger 240x320 Pimoroni display works fine.
Can someone have a look please?
Many thanks

That's odd - how do you have the RTC connected?

Thank you for looking into this.
The RTC is sandwiched between the Pico headers and the display. It uses I2C so it does not interfere with the display.
The display, RTC and Pico worked fine in 18.x using the older display code and the brightness worked fine - so it is not a hardware problem...
Hope it helps.

Hmm, what pins are you using for I2C?

It is a piggyback pass thru. It uses pins 20 and 21 (i2c_scl = 21, i2c_sda = 20)
I guess if these are used for PWM then it will not work ( I have another SSD7789 display and I use PWM to vary the backlight brightness.)
Looking forward to your reply.
Many thanks !

I had a look at the Pimoroni site regarding the Pico Display and it looks like it it uses Pins 20 and 21 - is it possible to re-assign these pins ?  I will also look at the possibility of configuring the DS3231 RTC with different I2C pins if possible...
Thank you.

Yep, I suspect it's sharing GP20 that's causing problems - might be easiest to rejig things so your I2C interface uses GP4 and GP5?

Hello Hel,
The Waveshare PICO DS3231 RTC offers alternate addresses (GP6 & GP7) but these are also use for the picodisplay LEDs.  I have now connected GP20 & GP21 to GP4 & GP5 - which are not used just like you suggested -  to fool the device.  I have also changed the device driver to use those GP pins and it works well - Thank you again for all your help !

Awesome, glad you got it sorted! Closing for now.


Validate SignedContributionAndProofGossip

PR Description
Adds validation support for SignedContributionAndProof messages.  Currently with a ton of changes just to make it possible to generate a valid message as well.  Will split into separate PRs then finish writing tests for the validator.
Fixed Issue(s)
#3803
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

Closing as this has been done through separate PRs now.


feat(task/prettier-confromance): ignore tests with non-standard syntax

Merge activity

Nov 21, 9:57 PM: @Boshen started a stack merge that includes this pull request via Graphite.