[jsp mode] Add

There's quite a lot of tricky aspects involved with doing your own mode-switching like this. Any reason you didn't use a multiplexing mode (as in the multiplex?

No, I should have, and researching multiplexing mode actually revealed that there is actually already support for JSP within the HtmlEmbedded mode, it was just not obvious at first. I am closing the pull request!


Update mtr-device-config-account-include.md

Add GO link to Teams Rooms advanced deployment guide within Admin Center

@dstrome
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@dstrome when will this be published to public?

Sorry, updates to includes don't trigger a refresh of the articles they're in. I've started a full build of the repo which will refresh all the content on the site, including the articles this include is in. Should be live in 20-30 mins.


fix: override pre-generated database connection

Description
It was supposed that the connection string from ClickHouse and MySQL were fixed by this PR: https://github.com/openedx/tutor-contrib-aspects/pull/581. However, the asset generated with a random user/password were used instead. This PR makes sure that the SQL alchemy string is correct.

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
Please let us know once your PR is ready for our review and all tests are green.

What is the best way to test this? Is there any concern about bootstrapping a new instance with these changes?

No, there isn't any concern on new instances. Already installed instances should be repaired.
You can test on a fresh local installation, and verify the connection string is using your current credentials instead of the ones on the CI of this repository

I've never run into the bug locally, though, is this only showing up in k8s?

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


can u update?

hi can u update for 3.3.5 ? or no? delete project!

This is a proof of concept, it's not intended to be updated. your comment are just disrespectful toward the author and don't expect any assistance IMO ! Scriptkiddy...

hi can u update for 3.3.5 ? or no?

I don't really understand the question.  It does work for 3.3.5.  If you're trying it somewhere and it isn't working, it is because the server you're targeting is patched.

delete project!

omegalul no

no worked all 3.3.5 servers is wowned fixed!
can u changed handlers and update for 3.3.5? i soo need!

Is this the end of work?

Yes.  The purpose of this code was to encourage all servers to fix the bug, which for the most part they have done.  It is kept here now only for academic/historical purposes.

Do you think you can hack it? Again!

Do you think you can hack it? Again!

I'm not sure, but if it helps the thing that prevents me from trying is not a lack of permission from you.

you can
!
Effort

@namreeb
i use dont' work! 3.3.5a
this is need to update!? or Project is closeD?

GO FAST Fixed This Hack SHit :D
WE NEED ! to ! This ; HACK :dagger:

Dude, this is not how it works....

Dude, this is not how it works....

Comedy gold though.

@namreeb
what should we do? We want a new hack for 3.3.5

where are u namreeb:D
get update! this exploit hack autopass


chore: flex 1.3.10

Pull Request Test Coverage Report for Build 518387354

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 58.118%

Files with Coverage Reduction
New Missed Lines
%

internal/agent/event_sender_vortex.go
1
73.21%

pkg/integrations/legacy/runner.go
4
79.2%

Totals

Change from base Build 517722960:
0.01%

Covered Lines:
11433

Relevant Lines:
19672

💛  - Coveralls

Pull Request Test Coverage Report for Build 518387354

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 58.118%

Files with Coverage Reduction
New Missed Lines
%

internal/agent/event_sender_vortex.go
1
73.21%

pkg/integrations/legacy/runner.go
4
79.2%

Totals

Change from base Build 517722960:
0.01%

Covered Lines:
11433

Relevant Lines:
19672

💛  - Coveralls


Changed PodInfo to be a list

This change is to make API consistent with our convention.
Fixes #3622

cc @bgrant0607 @brendanburns

LGTM

Thanks for review!


Fork

Hey,
Thank for you work.
As you can see, I finished it by allowing recursive paths detection and DTO imports.
I forked it and created a new repo because I needed the new version to be published, but I can merge it into your repo if you want.

Hey, Thank for you work. As you can see, I finished it by allowing recursive paths detection and DTO imports. I forked it and created a new repo because I needed the new version to be published, but I can merge it into your repo if you want.

Hello,
Thank you for your efforts too, we're here to help each other, one hand is not enough to applaud.
I'll take a look at your code.


v-model and some default html attributes are wrongly typed.

I believe some attributes are wrongly typed. For example a select v-model should allow for null or for an object. Vue will happily work with it. Same for some other attributes like the value on the select options. This should be the case for most inputs as they are "restricting" functionality that Vue already has.
Take the example below for a normal select:

Here is the option issue that I mentioned:

:value="null"  behaves differently from :value="undefined" and should be considered as a valid value, yet Volar would force me to use undefined which won't achieve the desired result.

See https://github.com/vuejs/vue-next/issues/4574.


Pagination

Pagination is a must for "JavaScript that makes your plain HTML lists super flexible, searchable, sortable and filterable."

Its in the pipline. Coming this week if we're lucky :)

Any progress on this? Getting to be sticky for me ;)

Curious if this is available yet.

In next release!

How far out is that?

Hehe, that is a good question. But if we're lucky: Monday or Tuesday!

Look forward to it! Dev branch isn't on github eh? ;)

No it isn't but I probably should put it here ;)

Paging is now added! =)
http://jonnystromberg.com/listjs-0-2-0-plugins-paging/
http://news.ycombinator.com/item?id=3515820
http://jonnystromberg.com/listjs-paging-plugin/


Update espflash to v3

Closes #2300
Looks like hidapi still wants libudev-sys to stick around, but a good chunk of dependencies were cleaned up by this update. @MabezDev are you OK with the added tech debt regarding xtal freqs?

Unfortunately, espflash checks the xtal freq first and returns an error if it is an unexpected value, regardless of whether the user specified a custom bootloader, which will make things a bit annoying IMO :) This (i.e. not erroring unexpectedly), and selecting a correct bootloader is about as much as probe-rs cares about.


Support Fedora CoreOS

We would like to use Fedora CoreOS in our Kubernetes's node.
Do you have support plan in a roadmap of NetApp/Trident ?
Could you support it if you don't have?

@ysakashita, we don't have plans to support Fedora CoreOS on Trident's roadmap. Fedora CoreOS was just deemed production-ready and doesn't have a lot of demand at this time.

@gnarl Sure. I hope Fedora CoreOS users increase.

As you may know, CoreOS announced that On May 26, 2020, CoreOS Container Linux will reach its end of life and will no longer receive updates.
https://coreos.com/os/eol/
The EOL is faster than I expected...
Current Trident supports CoreOS Container Linux.
https://netapp-trident.readthedocs.io/en/latest/support/requirements.html#supported-host-operating-systems
If Trident don't support Fedora CoreOS until end of CoreOS Container Linux, we must migrate to other OS.
Could you have a migration plan of support OS from CoreOS Container Linux to other OS?
Also please let me know the end of support the CoreOS in Trident.

We don’t need it now. so I close it.
Please re-open if you need.


Hotfix/before write firing only once

1⃣ Why should it be added? What are the benefits of this change?
It can be used to implement progress bars for chunked exports. We could update the progress of the export each time the BeforeChunkWriting "listener" is being called.
2⃣ Does it contain multiple, unrelated changes? Please separate the PRs out.
No, it doesn't.
3⃣ Does it include tests, if possible?
No, it doesn't include tests.
4⃣ Any drawbacks? Possible breaking changes?
Can't really think of any.  Please feel free to point out if you think it can cause issues.
Please feel free to comment on the changes if you don't find them adequate,  I will take care of making the appropriate changes.

Hey thanks for contributing! I think this was an intentional choice to only execute it on the first chunk, but would have to look into it and see if it's a logical change that doesn't have impact as a bugfix.

Also, @patrickbrouwers  I have a question regarding the CloseSheet job: Why are you calling write() after closing the sheet?

Closing the sheet just performs some actions on the sheet like column sizing, styles etc. Write actually writes the in-memory cell collection to the temporary file.

Few questions, hope you have time to have a look at them.

Thanks so much for taking time to review my silly (and buggy) solution 🙈.
I hope I managed to provide an answer to all your questions.

I do have another idea, but it would require an overhaul of this event handling and introduce breaking changes.
It could however be used as an alternative way of listening to events triggered by the library, hence not introducing breaking changes.
I will make a separate pull request to show you what I mean once I make a proof of concept.
Meanwhile, I am open to suggestions.

@DenisaHalmaghi I'll get back to it, busy few days right now.

Hey @DenisaHalmaghi I'm going to hold off on merging this, as I'm planning some overhauls to the queued writing system. The current implementation with writing in chunks isn't performant in the way PhpSpreadsheet works (on every re-open it will load everything back into memory; so every chunk will increasingly use more memory). Therefor the new recommend way won't have any chunk writing at all.
I would personally recommend (at least for now) to wrap the export in a Laravel queue job and run it on a long running queue (https://medium.com/@williamvicary/long-running-jobs-with-laravel-horizon-7655e34752f7) .

Hey @patrickbrouwers , thank you for letting me know.  I am looking forward to the overhaul you mentioned , but will however miss the chunked writing feature.
Thanks for the suggestion and the amazing package!
I also hope I will be able to contribute again someday in the future.


UI

Currently, the UI of ImageLab is not very interactive There is no exit, minimize button so every time the user has to force close the app.

Hi @aakarshvats638 , Are you still working for this issue?


Provide option for getAll method so that it includes keys, enhancement #34

As per issue #34, I provided an option so that the getAll method returns an array of dictionaries that contain both key and value of the saved items. This way we can keep the API backwards compatible.

Wow. Thanks for the great job @georapbox! I didn't have time recently to keep up with PRs. Thanks again for coming to the rescue.


Does pos support scala 2.13.0?

My scala version is 2.13.0, but I can't seem to use the pos 2.2.0 package. Does pos support scala 2.13.0, or it will be supported in the future? Thx.
My build.sbt is as follows.
name := "demo"

version := "0.1"

scalaVersion := "2.13.0"

resolvers += Resolver.bintrayRepo("johnreed2","maven")

libraryDependencies ++= Seq(
  "com.github.johnreedlol" %% "pos" % "2.2.0"
)

Just published scalaVersion := "2.13.1". It should now work with both scalaVersion 2.13.0 and 2.13.1 .
To test it out I created an sbt project with the following build.sbt file:
name := "postest"

version := "0.1"

scalaVersion := "2.13.1"

resolvers += Resolver.bintrayRepo("johnreed2","maven")

libraryDependencies ++= Seq(
  "com.github.johnreedlol" %% "pos" % "2.2.0"
)

The following Main:
package pkg

import com.github.johnreedlol._

object Main {
  def main(args: Array[String]): Unit = {
    out("Hello World!")
  }
}

Output:
[info] running pkg.Main
Hello World! - pkg.Main.main(Main.scala:7)

@Shoutmon Does it work for you?

@JohnReedLOL yes, it works perfectly. thx~


[terra-menu] Fix keyboard (up/down arrow) and mouse-click navigation …

…amongst the Menu Items within a Group.
Fixes #801
Deployment Link
https://terra-framework-deploye-pr-806.herokuapp.com/components/terra-menu/menu/menu
and
https://terra-framework-deploye-pr-806.herokuapp.com/tests/terra-menu/menu/menu/large-menu

Functional verification completed. The examples now display and function as expected with tab and arrow key navigation; however, during screen reader testing, I found an issue with desktop VoiceOver and JAWS. Screen reader navigation allows focus to shift out of the popup while it is still open. I've logged issue #833 for that.


Less readable after (arg => obj.method ... on same line

Overall, I liked output from the latest release, but I did find this code to be far less readable (where filter now gets indented).  The comments are also strange now.
Doubt this is a bug, but just wanted to share it.

Thanks for the report, I agree that it looks worse. I have an idea on how to fix it :)
Could you open a different issue for the comment that gets indented weirdly, we should correct the indentation.
Thanks for the report and keep them coming if you see anything that doesn't look good to you

https://github.com/jlongster/prettier/pull/667 here's the fix!

Nice.  Can't wait to try it.


Make static math work with AT

Static math needs to be accessible to AT. The first step for that is for it to be able to be spoken and be navigable in some form.
Note: this is a feature that needs to be implemented in mathlive.

Fixed a while back.


How is ShadowsocksAdapter supposed to instantiate?

The ShadowsocksAdapter class is insanely hard to instantiate. The parameters change from some very simple key-value pairs to a hell of factories with no documentations.

Ok, I've read through the method parseShadowsocksAdapterFactory and now have an understanding of how it works. But I still don't think it's a good idea for such a basic setup-API to be so complicated that one has to read the implementation details to understand it.
Even with enough documentation it's still too complicated.


Release Azure.ResourceManager.ServiceNetworking first beta

This is a new service, here is the approval of namespace.
Here is the sdk pr: https://github.com/Azure/azure-sdk-for-net/pull/33666

https://www.nuget.org/packages/Azure.ResourceManager.ServiceNetworking/1.0.0-beta.1 done


recommendations for the query  result set limit

You can just support a limit method.
Like this    .limit(20)

Added with PR #78


python310Packages.structlog: 21.4.0 -> 21.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.structlog is: "Painless structural logging"
meta.homepage for python310Packages.structlog is: "https://github.com/hynek/structlog"
meta.changelog for python310Packages.structlog is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 21.5.0 with grep in /nix/store/l0k8657v2vkbwmikf1sipgm68zq417qw-python3.10-structlog-21.5.0

found 21.5.0 in filename of file in /nix/store/l0k8657v2vkbwmikf1sipgm68zq417qw-python3.10-structlog-21.5.0

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

26 total rebuild path(s)

13 package rebuild(s)

13 x86_64-linux rebuild(s)
13 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
haskellPackages.nvfetcher
nvchecker
nvfetcher
python310Packages.elastic-apm
python310Packages.graphite_api
python310Packages.influxgraph
python310Packages.nvchecker
python310Packages.structlog
python39Packages.elastic-apm
python39Packages.graphite_api
python39Packages.influxgraph
python39Packages.nvchecker
python39Packages.structlog

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l0k8657v2vkbwmikf1sipgm68zq417qw-python3.10-structlog-21.5.0
ls -la /nix/store/l0k8657v2vkbwmikf1sipgm68zq417qw-python3.10-structlog-21.5.0/bin

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

1 package failed to build:

python310Packages.elastic-apm

10 packages built:

nvchecker (python39Packages.nvchecker)
nvfetcher
python310Packages.graphite_api
python310Packages.influxgraph
python310Packages.nvchecker
python310Packages.structlog
python39Packages.elastic-apm
python39Packages.graphite_api
python39Packages.influxgraph
python39Packages.structlog

Maintainer pings

#154969


officially install and setup prettier eslint rules

also format all files that don't match.  later we might add in an azure pipelines rule to make sure formatting is correct

closes #90


As a developer, I want the project to include unit tests, so that I can test things.

Addressed by https://github.com/WSU-CEG-6110-4410/ADAD-Craft/pull/123


Add Keywords entry for .desktop files

At least GNOME ask to add keywords to application desktop files - see here. It is also reported by lintian with the desktop-entry-lacks-keywords-entry information tag. I just add some basic ones, do you see others?
I have also removed the French translation of the Name as it doesn't make sense to me to translate the software name.

Hi, thanks for this.
Other interesting keywords might be related to plugins, like "midi;artnet;e131;osc;"

Other interesting keywords might be related to plugins, like "midi;artnet;e131;osc;"

You're welcome, I have added them to qlcplus.desktop.


ENH added openmp for mac and linux

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #20

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge/fftw This PR is ready to go.
I am disabling the openmp libs for mac toolchain builds and windows systems. I am not claiming this solution is perfect, but it does get most of the way there. If someone knows the incantations for openmp on windows, I am happy to add it. However, my own experience building software for windows is very limited.

Also, it appears --with-combined-threads is used for the windows builds which would preclude using both pthreads and openmp. Thus not shipping openmp for windows seems ok.

Postscript: Upon reading more, I learned (and I guess y'all already knew) that you have to use --with-combined-threads for windows builds. Thus this PR is exactly what is possible for openmp.

This looks pretty good to me.
One side effect of this is that future pyFFTW builds would default to using OpenMP instead of pthreads unless we explicitly add an PYFFTW_USE_PTHREADS environment variable to the build script (as of release 0.11 of pyFFTW, OpenMP builds are selected by default over pthreads when both variants are present).
I will leave it open a little longer to see if there is other feedback.

Ahh goog point. We might want to force pthreads there since the toolchain build of that repo will then be different from the gcc7 builds for mac. IDK if we care...

The main issue with using OpenMP is one of the most popular implementations (GCC's OpenMP, a.k.a GOMP) is not fork safe. Combined with the fact that Python's multiprocessing uses forking, this is a recipe for nasty crashes with little warning. Having had to debug this issue personally before, am generally reluctant to make use of OpenMP if there is another option (as is the case here with pthreads).
ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58378

Having had to debug this issue personally before, am generally reluctant to make use of OpenMP if there is another option (as is the case here with pthreads).

Right, but pthreads-based FFTW C libs will still be available. This PR just adds the OpenMP variant as well (in separate dynamic library files).  We can keep downstream packages using pthreads when necessary (e.g. define PYFFTW_USE_PTHREADS for pyFFTW).

This is right. There are extra binaries in the conda packages for people who want to link code against them (me). The pthreads binaries are still there.
Given what you are saying @jakirkham, it sounds like a PR forcing pthreads on pyFFTW is required for this one to be merged.
I will do that today.

Merging this would not immediately break pyFFTW, because on OS X/linux it is statically compiled against FFTW to get around an unrelated numpy+mkl_fft issue. It would potentially cause a problem the next time we go to rebuild/update the pyFFTW recipe, though, so a PR there would be welcome.

Done. https://github.com/conda-forge/pyfftw-feedstock/pull/29

Alright @grlee77. I think we are good to go here too?

Ready on my end.  Are you okay with this change now @jakirkham ?

Sure, there may be other libraries in the stack we need to watch out for. However as there is a nice clean separation between OpenMP-based and pthreads-based FFTW builds, this seems pretty reasonable.


Missing files in sdist

It appears that the manifest is missing at least one file necessary to build
from the sdist for version 0.1. You're in good company, about 5% of other
projects updated in the last year are also missing files.
+ /tmp/venv/bin/pip3 wheel --no-binary pysliders -w /tmp/ext pysliders==0.1
Looking in indexes: http://<IP_ADDRESS>:9191/root/pypi/+simple/
Collecting pysliders==0.1
  Downloading http://<IP_ADDRESS>:9191/root/pypi/%2Bf/08a/bb5f5eaeff1fd/PySliders-0.1.tar.gz (5.3 kB)
    ERROR: Command errored out with exit status 1:
     command: /tmp/venv/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-wheel-far8_4ws/pysliders/setup.py'"'"'; __file__='"'"'/tmp/pip-wheel-far8_4ws/pysliders/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-wheel-far8_4ws/pysliders/pip-egg-info
         cwd: /tmp/pip-wheel-far8_4ws/pysliders/
    Complete output (5 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-wheel-far8_4ws/pysliders/setup.py", line 3, in <module>
        with open('README-PyPi.txt', 'r') as README_file:
    FileNotFoundError: [Errno 2] No such file or directory: 'README-PyPi.txt'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

wow didnt know people actually used this module.
I have made a 0.1.1 release which I have tested and it seems that this error is resolved. Thanks for reporting it.
Any other problems or concerns dont hesitate to make a new report about it.


Generated filePath in API is platform dependent

I tried to implement something in carbon-components-svelte but on build it caused changes like this all over COMPONENT_API.json:
   "components": [
     {
       "moduleName": "Accordion",
-      "filePath": "src/Accordion/Accordion.svelte",
+      "filePath": "src\\Accordion\\Accordion.svelte",
       "props": [
         {
           "name": "align",

Because Windows 😑

If I'm reading this correct, we could incorporate the solution from #39 when writing the JSON output in the following line:
https://github.com/carbon-design-system/sveld/blob/3147b602ceeaad5931786e96f084cfb1b40be53e/src/writer/writer-json.ts#L21

Released in v0.11.1


Updating proton native project not maintained!

The Proton native project was not actively maintained but had a reference, which is now pointing to a maintained fork of the project!

Hi @Simek, I think I have modified the right file, but the changes aren't still reflected :(

Hi @Simek, I think I have modified the right file, but the changes aren't still reflected :(

@ksharinarayanan Changes are visible in Next version, but the default selected version for new users is 0.64:

https://reactnative.dev/docs/next/out-of-tree-platforms

If you would like to edit versioned docs too, please prepare another PR. You can read more about versioned docs here:

https://github.com/facebook/react-native-website#versioned-docs

Yeah, I get it now thanks! 😄


--filter-files broken for absolute paths

I found this bug by using PyCharm FileWatchers. I was passing $FilePath$ to isort, which was, as a result, not obeying my skip settings in setup.cfg. Adding --filter-files made it so isort was skipping all files.
The fix for me was to pass in a relative path, $FilePathRelativeToProjectRoot$. However, I still think that absolute paths should be filtered properly, based on the current working directory.
[9:40:44] [basepi@corellia] [~/src/apm-agent-python] [master]
± pwd
/Users/basepi/src/apm-agent-python

[9:40:52] [basepi@corellia] [~/src/apm-agent-python] [master]
± /usr/local/bin/isort --filter-files /Users/basepi/src/apm-agent-python/elasticapm/base.py
Skipped 1 files

[9:40:55] [basepi@corellia] [~/src/apm-agent-python] [master]
± /usr/local/bin/isort --filter-files elasticapm/base.py

isort version 4.3.21

Closing as this is likely resolved in the 5.x.x series. Feel free to open a new issue if this persists for you.
Thanks!
~Timothy


Add oj.uz

You can solve some olympiads from the past and upgrade skills on competitive programming :D

It seems that you worked on my outdated markdown-pdf branch.
Please double-check.
Thanks


Improve verify-email page content/layout

I don't see any design for this but we need to improve both the layout and content
Currently it is:

Content: Change
Please verify your email
 to:

Check your inbox

We just emailed a link to {{ THE@EMAIL }}. Click the link to confirm your account and complete your profile. If you don't see a message within a few minutes please check your spam folder.

Layout: Center the content horizontally and move the content down on the page so it starts ~25-30% from the top of the page.

Designer here! Few questions: (1) Is this issue for both mobile and desktop? , (2) we have a pop-up design for the mobile version but the content is different, should we change the content to what you have rn?

This is a design for email-sign-up design (if it works I can export this to Zeplin). Do you have the "resend email" feature? If we do, I can add a button below and main paragraph

@Yu-Alexa-Zhao was this added to Zeplin?

@manualzuru  I'm pretty sure this can be worked on. We have the design except maybe for resend component - but someone can certainly start working on this.

Yes it was exported to zeplin:https://zpl.io/bJJMeDK

I don't see any design for this but we need to improve both the layout and content
Currently it is:

Content: Change
Please verify your email
 to:

Check your inbox

We just emailed a link to {{ THE@EMAIL }}. Click the link to confirm your account and complete your profile. If you don't see a message within a few minutes please check your spam folder.

Layout: Center the content horizontally and move the content down on the page so it starts ~25-30% from the top of the page.

Also, Kindly make the FP logo clickable in this page, so that user can directly navigate to home page to login after the sign up.


Update Pages “contact”

Automatically generated by Netlify CMS

👷 Deploy Preview for safehavenaccounting processing.

Name
Link

🔨 Latest commit
173bcacace7e04a3254f7feab688b234d30f72e6

🔍 Latest deploy log
https://app.netlify.com/sites/safehavenaccounting/deploys/64932d6a0d74220008a01091


effective fastertransformer total time costs of batch size > 1 is more than batch size = 1

I use the ParallelGptContextDecoderOp with remove_padding=True to get hidden states of last tokens, but I found the total time costs of batch size = 18 is more than batch size = 1

This phenomenon not same with https://github.com/NVIDIA/FasterTransformer/blob/main/docs/bert_guide.md#bert-performance-on-t4-and-pytorch

I found that longer inputs lead to lower batching performance

The cost of bs > 1 is larger than bs = 1 is an expected behavior. Effective transformer only removes the useless computing about padding. The performance improvement highly depends on how many padding in the inputs.

The cost of bs > 1 is larger than bs = 1 is an expected behavior. Effective transformer only removes the useless computing about padding. The performance improvement highly depends on how many padding in the inputs.

I found that the impact of batching on performance is related to the input length. In my experiment, using the faster transformer, when the input length is 100, the performance of bs=10 is 30% higher than that of bs=1. When the input length is 955, the performance of bs=10 is 10% higher than that of bs=1. When the input length is 2048, the performance of bs=10 is 6% lower than that of bs=1. Is this also expected?

What's your meaning for performance? In any case, the latency of bs 1 should be faster than bs 10 when the input length are same. I don't know how you get the conclusion: when the input length is 100, the performance of bs=10 is 30% higher than that of bs=1.
Do you mean the throughput?

In my case, the performance means the time costs of inferencing 1000 items

Do you mean there are 1000 sentences. When bs = 1, you need to run 1000 times. While when bs = 10, you only need to run 100 times?
For that case, the comparison is on throughput. When input length is very long, bs = 1 and bs = 10 may have similar performance.

For that case, the comparison is on throughput. When input length is very long, bs = 1 and bs = 10 may have similar performance.

So what is the reason for this phenomenon?

Because the GPU is fully used even if bs = 1. So, increasing the batch size does not bring any benefit.


Corrupted tsv when querying colored graph?

Hi,
I try to query a lot of k-mers (about 5 mio) in a colored graph (based on some reference genomes). All default settings k=31, query k-mers are also 31.
The returned tsv has rows of different lengths. My suspicion is that Bifrost writes in parallel to a single file and somehow there is no write lock so that one process writes to the file even though another one has not finished. Could this be possible?
Thanks!

Hi @phiweger,
That's annoying, sorry about that. There is obviously a write lock but it is possible it is not used properly. I had some issues with this before so I will make some more testing soon. I would appreciate if you had a minimum example of graph + colors + queries for which the issue occurs that I could use.

Sure -- https://osf.io/eyunk/

Hi @phiweger,
I downloaded your files and I performed a simple querying using most default parameters:
Bifrost query -v -t 16 -g graph.gfa -f graph.bfg_colors -q kmers.fasta -o res1
I used 16 threads to "maximize" my chance of messing with the output. I then counted the number of fields in the header of the TSV output:
head -n 1 res1.tsv | awk '{print NF}'
which gives 1708.
Then I searched for a line having a different number of fields than 1708:
awk '{if (NF!=1708) print NR " " NF}' res1.tsv
and there was no output, meaning that my output file res1.tsv has the correct length over each line. I tried the same experiment with 32 threads but unfortunately (or fortunately?) ended up with a consistent result too.
I'm afraid that you have the version 1.0.3 installed on your system, which you probably obtained from conda, which suffered from issue #23. This has since been fixed but the conda recipe hasn't been updated since that time as I just made a release for version 1.0.4 recently. Is it what happened?
Otherwise, could you tell me:

How did you install Bifrost (conda or source)?
Which OS and compiler are you using for the experiment?
How many threads are you using for the experiment?
How many lines do you have in your faulty TSV output file?
How many fields do you have in a row having an abnormal number of fields?

I actually don't think anymore that the problem is conda. I was trying to update the conda recipe and realized that the recipe has been automatically updated to the latest version by the BioCondaBot. Same with Brew. Can you do, in addition to my previous questions, a Bifrost --version for me?

Indeed!
docker run --cpus 8 -v $PWD:/data nanozoo/bifrost:1.0.3--749395e Bifrost
# Bifrost 1.0.3

I installed Bifrost by building it inside a Docker container (my last build is actually v1.0.4 but I kept using an older one v1.0.3 -- thanks a lot for your quick help!
Kindest regards


tensor size mismatch!! HELP

     ############# Starting Epoch 0 | LR: 0.001 #############
    0%|                                                                                | 0/113 [00:00<?, ?it/s]the gpu is 0
  the skeleton shape is torch.Size([16, 24, 3])
  the global orient is None
  the batch size is 16
    0%|                                                                                | 0/113 [02:26<?, ?it/s]
  Traceback (most recent call last):
    File "/home/ndip/HybrIK/scripts/train_smpl.py", line 398, in <module>
      main()
    File "/home/ndip/HybrIK/scripts/train_smpl.py", line 246, in main
      mp.spawn(main_worker, nprocs=ngpus_per_node, args=(opt, cfg))
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 246, in spawn
      return start_processes(fn, args, nprocs, join, daemon, start_method="spawn")
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 202, in start_processes
      while not context.join():
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 163, in join
      raise ProcessRaisedException(msg, error_index, failed_process.pid)
  torch.multiprocessing.spawn.ProcessRaisedException:

  -- Process 0 terminated with the following error:
  Traceback (most recent call last):
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/multiprocessing/spawn.py", line 74, in _wrap
      fn(i, *args)
    File "/home/ndip/HybrIK/scripts/train_smpl.py", line 346, in main_worker
      loss, acc17 = train(opt, train_loader, m, criterion, optimizer, writer)
    File "/home/ndip/HybrIK/scripts/train_smpl.py", line 61, in train
      output = m(inps, trans_inv, intrinsic_param, root, depth_factor, None)
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
      return self._call_impl(*args, **kwargs)
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
      return forward_call(*args, **kwargs)
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1519, in forward
      else self._run_ddp_forward(*inputs, **kwargs)
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/parallel/distributed.py", line 1355, in _run_ddp_forward
      return self.module(*inputs, **kwargs)  # type: ignore[index]
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
      return self._call_impl(*args, **kwargs)
    File "/home/ndip/miniconda3/envs/mmlab/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
      return forward_call(*args, **kwargs)
    File "/home/ndip/HybrIK/hybrik/models/simple3dposeBaseSMPL24.py", line 335, in forward
      output = self.smpl.hybrik(
    File "/home/ndip/HybrIK/hybrik/models/layers/smpl/SMPL.py", line 263, in hybrik
      vertices, new_joints, rot_mats, joints_from_verts = hybrik(
    File "/home/ndip/HybrIK/hybrik/models/layers/smpl/lbs.py", line 357, in hybrik
      rot_mats, rotate_rest_pose = batch_inverse_kinematics_transform_naive(
    File "/home/ndip/HybrIK/hybrik/models/layers/smpl/lbs.py", line 868, in batch_inverse_kinematics_transform_naive
      rot_mat_chain[:, indices] = torch.matmul(
  RuntimeError: The size of tensor a (2) must match the size of tensor b (5) at non-singleton dimension 1

@LoyalBlanc @biansy000 , I received the following error after getting my data loaded for training..  I followed the structure of the processed data file of h36m which you have given on my own dataset..
Could you tell me where can I trace this error or could be the potential solution..
I noticed that
if leaf_thetas is not None:
                rot_mat = leaf_rot_mats[:, :, :, :]
                print(rot_mat.shape)
                print(rot_mat_chain[:, parents[indices]].shape)

upon printing,
torch.Size([32, 5, 3, 3])
torch.Size([32, 2, 3, 3])
NOTE:  my data includes 24 joints and remaining 5 are 0s
I have 2 Gpus in my system but it is only using gpu 0 for some reason.  could this be the reason??   or which possible tensors have different shapes?

emmm... Please do not use configs/256x192_adam_lr1e-3-res34_smpl_24_3d_base_2x_mix.yaml , and use 29-kpts version of HybrIK instead. The 24-kpt version of HybrIk is stale.


autoheight based on number of lines?

any ideas on how to grow height RichTextFX height to match the number of lines?, i mean to autogrow the control hieght and avoid the vertical slider (with virtualizedScrollPane) to show and see all the text, like jupyter notebook code cells do.
size can be obtained with
Bounds bounds = codeArea.getLayoutBounds();
then to grow height
codeArea.setPrefSize(bounds.getWidth(), bounds.getHeight() + grow)
the question is to calculate the grow or the size based on the number of lines, or grow until vertical scrollbar disappears, but how do i know if the vertical slider is present?
UDPATE : This methods worked for me, not perfect but job is done, suggestions accepted
    public void autoresize() {
        double rows = getDocument().getParagraphs().size();
        Optional<Bounds> lineBound = getParagraphBoundsOnScreen(0);
        double rowSize_;

        if(rowSize==null){
            if (lineBound.isPresent() == true){
                rowSize_ = lineBound.get().getHeight();
                rowSize = rowSize_;
            }
            else rowSize_ = 25d;
        }
        else rowSize_ = rowSize;

        this.setPrefSize(getLayoutBounds().getWidth(), rowSize_ * rows + 5); //(rows*0.95d)
    }

    public void autoresizeAlternative() {
        Text text = new Text();
        text.setFont(new Font(18));
        text.setWrappingWidth(getWidth());
        text.setText(getText());
        double rows = getDocument().getParagraphs().size();
        double fix = (rows >= 20) ? rows - rows * 0.1d : 0d;
        setPrefSize(getLayoutBounds().getWidth(), text.getLayoutBounds().getHeight() + fix);
    }

Hi @saulpalv, I think the following is a more robust solution. Could you try it out and let me know how it works for you:
class AutoHeight<PS, SEG, S> implements Consumer<ListModification>
{
    private ObservableList<Double> paragraphHeights = FXCollections.observableArrayList();
    private DoubleProperty totalHeight = new SimpleDoubleProperty();
    private final GenericStyledArea<PS, SEG, S> area;

    public AutoHeight( GenericStyledArea<PS, SEG, S> area )
    {
        paragraphHeights.addListener( (ListChangeListener.Change<? extends Double> chg) ->
        {
            while ( chg.next() )
            {
                double delta = chg.getAddedSubList().stream().mapToDouble( d -> d ).sum();
                delta -= chg.getRemoved().stream().mapToDouble( d -> d ).sum();
                totalHeight.set( totalHeight.get() + delta );
            }
        });

        area.prefHeightProperty().bind( totalHeight );
        this.area = area;
    }

    @Override
    public void accept( ListModification lm )
    {
        int paragraph = lm.getFrom();
        final int addEnd = paragraph + lm.getAddedSize();
        final int removEnd = paragraph + lm.getRemovedSize();
        final boolean removExtra = addEnd < removEnd;

        for ( ; paragraph < addEnd; paragraph++ )
        {
            boolean replace = removExtra || paragraph < removEnd;
            addOrChangeParagraph( paragraph, replace );
        }

        if ( removExtra )
        {
            paragraphHeights.remove( paragraph, removEnd );
        }
    }

    private void addOrChangeParagraph( int paragraph, boolean replace )
    {
        Optional<Double> height = area.getParagraphBoundsOnScreen( paragraph ).map( b -> b.getHeight() );

        if ( height.isEmpty() ) {
            area.showParagraphInViewport( paragraph );
            addOrChangeParagraph( paragraph, replace );
            return;
        }

        if ( paragraph >= paragraphHeights.size() ) paragraphHeights.add( height.get() );   // Append
        else if ( replace ) paragraphHeights.set( paragraph, height.get() );                // Replace
        else paragraphHeights.add( paragraph, height.get() );                               // Insert
    }
}

Use it like so, and don't wrap the area in a scroll pane:
area.getParagraphs().addModificationObserver( new AutoHeight( area ) );

Just a heads up that I've changed the previous code.
Also the area can be wrapped in a VirtualizedScrollPane if needed, but some tweaking still needs to be done to the submitted code to keep the vertical scrollbar hidden.

i tried the code, had to put on try catch in the addOrChangeParagraph(), it works well if code is added manually, but it doesn't work if add add text using replaceText(0, 0, text);

The reason I think why you had to add the try catch and why the replaceText isn't working for you is probably because the RichtextFX control isn't part of a Scene or Stage yet.
I'll have a look and see if I can improve that. In the meantime you can try delaying calling replaceText until after the control is added to a Scene or Stage.

Closed via PR #944


Return REFUSED for out-of-zone queries

In other words, if we are not authoritative for a zone, then return REFUSED instead of SERVFAIL

Closed by e67d128862f7d1f9aed387f2eaf07ca4bd4d9983


[Feature][Need help] Evolution logic

Need your help. I want to implement one more evolve mode based on resulting CP compared to other family-related pokemons CP. Is there any reliable info how we can calculate CP after evolving?

@cmezh  you can try http://pogotoolkit.com/ or https://pokeassistant.com/main/evolver
As far as I understand there are min and max multipliers for each Pokemon that you can evolve.


Stop using ansible-role-nodepool

This role is not worth the complexity - were are at an point where in
order to use it we need to lay down our own service files, at which
point were saving only a few lines of code and gaining a massive amount
of complexity.

rattboi - sorry, github is useless so I cant reply inline. The file already exists in our role (see templates dir).


Fix potential bugs in DefaultMutableAttributeContainer, simplify toString for easier debugging

Fixes #19753

@bot-gradle test QFL

OK, I've already triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.


opt For left semi/anti join without mark join conjunct and without other conjucnts, stop probing after matching one row

Proposed changes
If one row from the probe side has one match in the hash table, the result of this row should be determined.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


pixi use questions

The pixi.lock file currently contains a complete description of your python environment and is over 6000 lines and 200KB.    Is there a way to streamline this - perhaps by creating the lock file within a minimalist virtual environment?

Originally posted by @amegahed in https://github.com/UW-Madison-DSI/ospo.wisc.edu/issues/26#issuecomment-2273677342

The pixi.lock file currently contains a complete description of your python environment and is over 6000 lines and 200KB.

@amegahed The pixi environment does not involve Python at all
$ git grep "python" -- pixi.lock  # no output

pixi it is a general dependency management tool that focuses on using package indexes that Python packages use often (PyPI and conda-forge) but there is no requirement for Python. This is a Ruby project, and so there are only Ruby dependencies and then external dependencies of Ruby Gems like ImageMagick.

Is there a way to streamline this - perhaps by creating the lock file within a minimalist virtual environment?

No. Lock files are by design going to be large even if they only describe minimal requirements, like this one does.
I'm also not clear on why this is an issue for this project. If you use some set of commands like shown here https://stackoverflow.com/questions/10622179/how-to-find-identify-large-commits-in-git-history to find the largest files in every commit in the repo and grep on pixi.lock you get
5231d844427a  100KiB pixi.lock
1dae21da8b88  195KiB pixi.lock

but those aren't even in the top 30 largest. So no slow down is happening because of pixi.


TileMap occluders disappear in editor after switching to a different scene and back.

Operating system or device, Godot version, GPU Model and driver (if graphics related):
MacOS High Sierra 1.13.1, 55962ce28fec5f06016ed3f66f52e6e84b817dbe (fresh compile), using Mono, GTX 660M, whatever GPU driver apple ships with MacOS by default.
Issue description:
If you have a TileMap with occluders inside the scene editor, then swap to another scene tab and back, the occluders visually disappear until you re-open the scene entirely.
Steps to reproduce:

Make a TileMap, give it a tileset with some occluders.
Add a shadow casting 2D light to the scene.
The Light 2D should be properly casting a shadow onto the tilemap's occluders.
Open a different scene, then switch back to the tilemap scene's tab.
The occluders stopped rendering!

cc @MarianoGnu

Does this happen with occluder nodes too or just tilemap?

Sorry for the late response, but no, it's exclusive to tilemaps. Regular occluder nodes are fine.

This also happens during play time, removing a node/scene containing a TileMap from the tree and then re-adding it causes the same issue. 3.0.2 official.

I am experiencing this issue in v3.0.2 as well.

have you tryed with master? there have been a fix recently for navigations wich i belive could solve this too

I just tried with master, and it seems that the bug is gone.
(However, the current master breaks my autotiles.)

I just tried with master, and it seems that the bug is gone.

I tested that too and indeed seems to be working in 3.1.


The app is crashing as soon as we  click on the QR.

The app is crashing when I click on the bottom sheet's QR icon.
Tested on Dev app v1.7.9(306)

Unable to reproduce this issue on dev app v1.7.9(312)


creates a test to make sure GET /user/campaigns/:campaign_id is working

What does this PR do?
Creates a test to make sure the GET /user/campaigns/:campaign_id endpoint works
Where should the reviewer start?
tests/CampaignTest.php lines 74-91
How should this be manually tested?
vagrant ssh into northstar. Make sure you cd into DoSomething.org/northstar and run vendor/bin/phpunit

Nice! Ship it! :ship:


Improved UI to match website theme

Description
Now the progress page matches to the theme of website,
make proper navigation
Fixes:  #349
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[X] I have made this from my own
[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK (if applicable)
Before:

After:
https://github.com/Dev-tanay/Rubik-Cube/assets/168854094/9f518fda-89e3-4878-9462-dcc02481373e

@Dev-tanay Sir, please review this PR.
Thank You!

@Dev-tanay Sir, please review this PR.
Thank You!


fix broken build

We have a merge problem on main, one PR used functions there were removed in another.
https://github.com/containers/netavark/pull/448
https://github.com/containers/netavark/pull/447
Sorry I should have noticed this and rebased.

@baude @mheon @flouthoc PTAL

/hold cancel


Fixing GrowArrow()

Changelog / Overview

Motivation

The animation GrowArrow() is broken. It sends the boolean scale_tips=True to the function scale() and ultimativeley to apply_points_function_about_point(). Those function have no parameter scale_tips though.
EDIT
GrowArrow() only works for class Arrow, not for Arrow3D since it utilizes the own scale method of Arrow. This scale method relies on Arrow being a TipableVMobject, which Arrow3D is not. I have all suggestions listed in Issue #1741 .
Explanation for Changes

Testing Status

Further Comments

Checklist

[ ] I have read the Contributing Guidelines
[ ] I have written a descriptive PR title (see top of PR template for examples)
[ ] I have written a changelog entry for the PR or deem it unnecessary
[ ] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[ ] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

Hi, Arrow implements its own scale method which allows the parameter scale_tips
https://github.com/ManimCommunity/manim/blob/1e01feb70dccbe493d173ef76046a3d1b851e685/manim/mobject/geometry.py#L1275
It would probably be better to add the scale method to Arrow3D as well (I'm not sure if its logic would be the same as that of Arrow.scale), instead of removing the scale_tips parameter ?

Hi, Arrow implements its own scale method which allows the parameter scale_tips
https://github.com/ManimCommunity/manim/blob/1e01feb70dccbe493d173ef76046a3d1b851e685/manim/mobject/geometry.py#L1275
It would probably be better to add the scale method to Arrow3D as well (I'm not sure if its logic would be the same as that of Arrow.scale), instead of removing the scale_tips parameter ?

Good catch, I should have seen that... I will mark this as a draft and think about the logic for Arrow3D.

Sorry for the very late response! I think instead of modifying the GrowArrow animation, it would make more sense to implement a custom Arrow3D.scale method that behaves similar to the Arrow.scale method -- including the optional keyword argument scale_tips.

Closing this for now due to inactivity


Support DeterministicPileup in StepChain

Fixes #7977
Still need to be tested
To update doc specs and a few json templates.

Actually, it's going to be better if we make it a StepChain top level argument - just as it's for TimePerEvent and SizePerEvent - since it's part of the SplittingArguments settings. Otherwise we'd have to check every single step dictionary before we create the first step (task).

@anorkus FYI, we're adding DeterministicPileup support in StepChains. You possibly have it already for TaskChains (in McM), but for StepChain it's a top level argument (first commit has that change).

@ticoann please review and merge


ENH: DICOM To NRRD Module Implementation

The DICOM To NRRD module provides batch processing for converting DICOM files into NRRD files and outputs the files into a DICOM standard conforming folder hierarchy of Patient/Study/Series.
Reference discussion forum:
https://discourse.slicer.org/t/possible-new-slicer-extension-dicomtonrrdbatchprocessing/33284
Reference pull request:
Hi please reference https://github.com/Slicer/Slicer/pull/7502
The recommended changes from the pull request have been added to this fork.

Hi @jcfr
I want to bring you in on the discussion. Your extension https://github.com/hina-shah/SlicerBatchAnonymize has potentially duplicate functionality for converting DICOM to NRRD. I installed your extension and ran the Dicom To Nrrd Conversion. The functionality is deprecated as I received this error:
"Error: the output file type is not supported currently
Deprecated feature no longer supported: --fMRIOutput"
I want to verify that this module makes sense as a standalone DicomToNRRD as Diffusion-weighted DICOM Import extracts diffusion weighting directions, v-values, and measurement frames. DICOMToNRRD takes an input DICOM directory and outputs NRRD files with an optional DICOM information JSON file in a Name/Study/Series hierarchy that conforms to DICOM standards.
Let me know your thoughts and whether you think it makes sense for me to combine the functionality into Diffusion-weighted DICOM Import or keep this as a simple standalone module at this time.
Thanks,
Alex


Blender 2.8 Beta GLTF and GLB export... is not working / broken

I heard that Blender 2.8 beta has GLTF and GLB exporter/importer so I did a quick setup of my Blend scene, just Principle BSDF with some textures plugged into Albedo, Roughness, etc. And a test export of GLB and GLTF exporter is instantly giving error.
I wonder if KhronosGroup has something to do with this? It's not ready for production right?

Actually I'll continue over here... seems like if I separate RGB and use a channel for Metalness and Roughness, it works kind of...
https://github.com/KhronosGroup/glTF-Blender-IO/issues/116


Make this starter IE11 ready

Our company values still advocate for a wide reach of users.
This PR fixes that.
I added some mechanisms to transpile node_modules, in case some yarn installs something that is not outright IE11 compatible, but sometimes that's not even enough.
Why? Because wouter wouldn't work even if it was transpiled. In that case, we replaced with react-router-dom which is heavier but more stable. It's the trade-off of supporting older browsers and total bundle size.
I also included polyfills which babel in turn turns to individual imports.  These polyfills are core-js and regenerator-runtime.

IE 11 works both on development and production envs


To use fn modifier as one of the keys to resize or move

I use an external keyboard and the fn key is pretty useless . By having the fn key to move/resize will save me the trouble to use another modifier or gap of modifiers.

hey @Dhairy2003, I'm sorry but AFAIK the fn key doesn't really do anything unless it's pressed in combination with another key, so it doesn't do much on its own. Or at least there's no event it's sending that I can catch to move/resize windows :(
Feel free to open another issue/PR if you think there's a workaround or more info I don't know about

Not sure if it'll help, but this obscure app with similar functionalities does it: https://mikkelmalmberg.com/moves
Thanks!!


Backport #20082 into 7.49.x

What does this PR do?
This PR backports #20082 into 7.49.x.
Motivation
https://dd.slack.com/archives/CE4PS8MHS/p1697489581293159.
We are not able only update go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp independently, as this creates dependency issues due to this PR, which is why we choose to backport the PR instead.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Apologies @kacper-murzyn, seems like we were modifying labels at the same time so it removed yours by mistake. I re-added them.


OffsetDateTime subtracted by OffsetDateTime can panic

I hit panic due to this assert
https://github.com/time-rs/time/blob/main/src/duration.rs#L205=
What I was subtracting
2022-06-22 0:00:00.0 +00:00:00 - 2022-06-21 1:57:08.3107779 +00:00:00

Callstack
thread 'should_handle_generate_crontab_events' panicked at 'assertion failed: nanoseconds <= 0', /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.3.10/src/duration.rs:205:13
stack backtrace:
   0: rust_begin_unwind
             at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/core/src/panicking.rs:143:14
   2: core::panicking::panic
             at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/core/src/panicking.rs:48:5
   3: time::duration::Duration::new_unchecked
             at /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.3.10/src/duration.rs:205:13
   4: <time::time::Time as core::ops::arith::Sub>::sub
             at /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.3.10/src/time.rs:751:9
   5: <time::primitive_date_time::PrimitiveDateTime as core::ops::arith::Sub>::sub
             at /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.3.10/src/primitive_date_time.rs:933:34
   6: <time::offset_date_time::OffsetDateTime as core::ops::arith::Sub>::sub
             at /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.3.10/src/offset_date_time.rs:1218:9
   7: task::cron::timer_duration
             at ./src/cron.rs:16:24
   8: task::cron::CronFuture::next::{{closure}}
             at ./src/cron.rs:33:40
   9: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
             at /rustc/fe5b13d681f25ee6474be29d748c65adcd91f69e/library/core/src/future/mod.rs:91:19
  10: task::cron::cron::{{closure}}
             at ./src/cron.rs:63:47

I'm not sure if it is correct to assert in this case as you only create part of Duration
I think new_unchecked should not have these asserts, since it is you know _unchecked

In 0.2 this code was fine because it didn't go through these asserts

The assertions are very much intentional, and were added recently. They are only present when in debug mode, so there's zero overhead in an optimized build.

In 0.2 this code was fine because it didn't go through these useless asserts

Just because an assertion isn't present doesn't mean the code was correct.
I will look into the panic, but the assertions are there for a reason: to catch bugs.

Well, unnecessary complicated implementation is reason to catch bugs.
In reality Time - Time should just sub seconds and nanoseconds and then do Duration::new(...) which would adjust resulting seconds and nanoseconds

Please be respectful.
You have not read the underlying code. I know this because your suggested implementation is irreconcilable with the internal structure. Handling time is extremely difficult and I have written nearly all of the code in this crate. All that I have written was from scratch, a large portion of it even without known algorithms. Bugs exist; that's the nature of programming. I have already provided justification for the debug assertions — something you have apparently never used in Rust. You're free to disagree on whether they should be there, but the evidence is clear: they should be. Were they not, this bug wouldn't have been caught. Describing code as "useless" and "unnecessarily complicated" is not only unproductive, it is just plain rude.

Same issue as topic starter with new 0.3.10 update

I have verified that using Duration::new returns the expected result, and have added a regression test to ensure that this does not change in the future. This patch was just released in 0.3.11.


filter demo available

Heya @SmithJK @AdamGarnsworthy,
A working demo of the v3 filter page is available for use: http://billmills.github.io/griffin-dashboard/filter.html
A few points:

No, the status sidebar isn't plugged in; I need a few hours where I can start and stop runs before this will be good to go.
v3 supports Chrome and FF as usual, but is best in Chrome; load is pretty slow in FF since we had to shim HTML imports (once the page is loaded though, it should be equally fast as Chrome).
@SmithJK, please let me know if this satisfactorily resolves https://github.com/GRIFFINCollaboration/griffin-dashboard/issues/56 and https://github.com/GRIFFINCollaboration/griffin-dashboard/issues/57; in general, all v3 pages should be free of past UI funkiness - let me know if anything still snags.

I tested GRIFFINCollaboration#56. I couldn't reproduce the original error on the two versions of Firefox I have available to me at the moment, but they look good. I can't test #57 since the demo page doesn't submit to the ODB.

OK - v3 is nominally feature-complete now, just doing some code cleanup and documentation before we push live; I'll let you know when it's good to go and we can hopefully wrap this and #57 up then.


Unknown operation used in model that is not explicitly declared

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
No
Source
source
TensorFlow version
tf 2.15 and 1.8
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
I am working on a uni project, a fall detection app built using Android studio and machine learning model.
The model uses time series data gotten from mobile sensors.
I have trained the model using model architecture LSTM -> TimeDistributedLayer -> Dense layer -> Dense layer (Output)
I did this initially on colab which uses tensorflow version 2.15, but when I tried to export it as a tensorflow lite model I was getting some errors during the conversion suggesting me to use SelectOps or flex ops which are because an operation is not supported by tensorflow lite’s standard operations.
Doing that and importing into Android studio, I couldn’t initialize the model without getting error saying “Unsupported data type 14 in tensor” which was probably as a result of Android not having the package to run the added operation.
I have tried importing flex ops into Android studio but didn’t work. Might have done it wrongly though.
Then I backdated the tensorflow version to v1.8 to use freeze graph which exported without any errors.
When i tried doing inferencing, I was getting the error “No OpKernel was registered to support Op ‘ClipByValue’ with these attrs. Register device: CPU, Register Kernel: ”
I inspected the freeze graph model using Netron and found the clipbyvalue op was being used in the LSTM layer and I’ve tried to declare in the optimizer not to use it though by default no clipping is used but nothing works.
I have also gone through my code to find any non-standard or custom operation used but can’t find any.
So I’m just stuck rn
Below is an image for inferencing

below is the image from netron of the freeze graph model

Standalone code to reproduce the issue
df = pd.read_csv("SisFall_All.csv")
df.drop(columns=['acc2_x', 'acc2_y', 'acc2_z'], inplace=True)
df.set_index(df.index, inplace=True)

# Resample the DataFrame index to downsample from 200Hz to 50Hz
df = df.iloc[::4]

# Reset the index to retain the original row indices
df.reset_index(drop=True, inplace=True)

acc1_resolution = 13
acc1_range = 16

gyro_resolution = 16
gyro_range = 2000

# Convert accelerometer data to gravity units
df['acc1_x'] = (2 * acc1_range / 2**acc1_resolution) * df['acc1_x']
df['acc1_y'] = (2 * acc1_range / 2**acc1_resolution) * df['acc1_y']
df['acc1_z'] = (2 * acc1_range / 2**acc1_resolution) * df['acc1_z']

# Convert gyroscope data to degrees per second
df['gyro_x'] = (2 * gyro_range / 2**gyro_resolution) * df['gyro_x']
df['gyro_y'] = (2 * gyro_range / 2**gyro_resolution) * df['gyro_y']
df['gyro_z'] = (2 * gyro_range / 2**gyro_resolution) * df['gyro_z']

# Convert activity labels to "ADL" or "Fall" based on their first letter
df['label'] = df['label'].apply(lambda x: 'ADL' if x.startswith('D') else 'Fall' if x.startswith('F') else x)
df.drop(columns=['Subject ID_Trial'], inplace=True)

time_steps = 200
features = 6
step = 20
segments = []
labels = []
for i in range(0, len(df) - time_steps, step):
  axs = df['acc1_x'].values[i: i + time_steps]
  ays = df['acc1_y'].values[i: i + time_steps]
  azs = df['acc1_z'].values[i: i + time_steps]
  gxs = df['gyro_x'].values[i: i + time_steps]
  gys = df['gyro_y'].values[i: i + time_steps]
  gzs = df['gyro_z'].values[i: i + time_steps]
#  amags = df['acc_magnitude'].values[i: i + time_steps]

  # label_counts = np.unique(df['label'][i: i + time_steps], return_counts=True)
  # label = label_counts[0][np.argmax(label_counts[1])]
  # segments.append([axs, ays, azs, gxs, gys, gzs])
  # labels.append(label)

  label = stats.mode(df['label'][i: i + time_steps])
  label = label[0][0]
  segments.append([axs, ays, azs, gxs, gys, gzs])
  labels.append(label)

np.array(segments).shape

reshaped_segment = np.asarray(segments, dtype = np.float32).reshape(-1, time_steps, features)
reshaped_segment.shape

from sklearn.preprocessing import LabelEncoder

label_encoder = LabelEncoder()
labels = label_encoder.fit_transform(labels)

print(labels)
print(labels.shape)

from sklearn.model_selection import train_test_split
X_train, X_test, y_train, y_test = train_test_split(reshaped_segment, labels, test_size = 0.2, random_state = 42)

model = Sequential()
# RNN layer
model.add(LSTM(128, input_shape = (200, 6), return_sequences = True, name = 'lstm_1'))
# Apply Dense operations individually to each time sequence
model.add(TimeDistributed(Dense(64, activation='relu'), name='time_distributed'))
# Flatten layer
model.add(Flatten(name='flatten'))
# Dense layer with ReLu
model.add(Dense(64, activation='relu', name='dense_1'))
# Softmax layer
model.add(Dense(2, activation = 'softmax', name='output'))

# Compile model
model.compile(loss='sparse_categorical_crossentropy', optimizer='adam', metrics=['accuracy'])
model.summary()

from keras.callbacks import ModelCheckpoint

callbacks = [ModelCheckpoint('model.h5', save_weights_only=False, save_best_only=True, verbose=1)]
history = model.fit(X_train, y_train, epochs = 10, validation_split = 0.20, batch_size = 256,
                    verbose = 1, callbacks = callbacks)

from keras.models import load_model
model = load_model('model.h5')
model.summary()

loss, accuracy = model.evaluate(X_test, y_test, batch_size = 128, verbose = 1)
print("Test Accuracy :", accuracy)
print("Test Loss :", loss)

from keras import backend as k
from tensorflow.python.tools import freeze_graph, optimize_for_inference_lib

input_node_name = ['lstm_1_input']
output_node_name = 'output/Softmax'
model_name = 'fall_model'

tf.train.write_graph(k.get_session().graph_def, 'models', model_name + '_graph.pbtxt')
saver = tf.train.Saver()
saver.save(k.get_session(), 'models/' + model_name + '.chkp')

freeze_graph.freeze_graph('models/' +model_name + '_graph.pbtxt', None, False, 'models/' +model_name+'.chkp',output_node_name, 'save/restore_all', 'save/Const:0', 'models/frozen_'+model_name+'.pb', True, "")

Relevant log output
FATAL EXCEPTION: main
Process: com.example.qdetect, PID: 7760
java.lang.IllegalArgumentException: No OpKernel was registered to support Op 'ClipByValue' with these attrs.  Registered devices: [CPU], Registered kernels:
  <no registered kernels>

	 [[Node: lstm_1/while/clip_by_value = ClipByValue[T=DT_FLOAT](lstm_1/while/add_2, lstm_1/while/Const, lstm_1/while/Const_1)]]
	at org.tensorflow.Session.run(Native Method)
	at org.tensorflow.Session.access$100(Session.java:48)
	at org.tensorflow.Session$Runner.runHelper(Session.java:298)
	at org.tensorflow.Session$Runner.run(Session.java:248)
	at org.tensorflow.contrib.android.TensorFlowInferenceInterface.run(TensorFlowInferenceInterface.java:228)
	at org.tensorflow.contrib.android.TensorFlowInferenceInterface.run(TensorFlowInferenceInterface.java:197)
	at org.tensorflow.contrib.android.TensorFlowInferenceInterface.run(TensorFlowInferenceInterface.java:187)
	at com.example.qdetect.pbClassifier.predict(pbClassifier.java:32)
	at com.example.qdetect.MainActivity.predictActivity(MainActivity.java:172)
	at com.example.qdetect.MainActivity.onSensorChanged(MainActivity.java:118)
	at android.hardware.SystemSensorManager$SensorEventQueue.dispatchSensorEvent(SystemSensorManager.java:1108)
	at android.os.MessageQueue.nativePollOnce(Native Method)
	at android.os.MessageQueue.next(MessageQueue.java:335)
	at android.os.Looper.loopOnce(Looper.java:187)
	at android.os.Looper.loop(Looper.java:319)
	at android.app.ActivityThread.main(ActivityThread.java:8913)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:608)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1103)

Hi @TimiOkusi ,
The 1.x versions are not supported now. Please provide the context and error log for Tf2.x versions.
As I understand the reported error is with TF1.8 version, and if so we can't support it. Please test with Tf2.x versions and preferably with latest versions and report the problems faced with all the required details for debug. Thanks!


SB Alliances Korean Translation

Some are not yet translated.

for removing "will be sealed off", see resources\lang\ko.js
    this.areaSealRegex = function() {
      return /:(\y{Float})초 후에 (.*)(이|가) 봉쇄됩니다\./;
    };


Canvas border line remains

That occur when a canvas extends but the line disappears in a second

Could you please explain what do you mean exactly... ?

Could you please attach the file as well? This would be very helpful for me to track the issue down, thanks!

Looks like an viewport update glitch anyway... Qt bug so to say.

cannot reproduce it anymore.


App crash after 1~2 seconds of playing a video on linux.

I have cloned the repo and followed all the steps for setting up linux environment
including

install flutter manually https://docs.flutter.dev/get-started/install/linux#method-2-manual-installation
run sudo apt-get install libgtk-3-dev
run sudo apt install libstdc++-12-dev
run sudo apt install libmpv-dev mpv
add this to linux CMakeLists

target_link_libraries(${BINARY_NAME} PRIVATE ${MIMALLOC_LIB})

I ran the example in here and
on debug mode after 2 seconds the app crash and in release mode in a few ms.
This is the crash log
package:media_kit_libs_linux registered.
media_kit: VideoOutput: video_output_new:<PHONE_NUMBER>368
Cannot load libcuda.so.1
media_kit: VideoOutput: Using H/W rendering.
flutter: VideoOutput.Resize
flutter: {handle:<PHONE_NUMBER>368, id:<PHONE_NUMBER>296, rect: {left: 0, top: 0, width: 1, height: 1}}
flutter: NativeVideoController: Texture ID:<PHONE_NUMBER>296
flutter: media_kit: wakelock: _count = 1
media_kit: TextureGL: Resize: (854, 480)
flutter: VideoOutput.Resize
flutter: {handle:<PHONE_NUMBER>368, id:<PHONE_NUMBER>296, rect: {left: 0, top: 0, width: 854, height: 480}}
Segmentation fault (core dumped)

I am running ubuntu 22.04
Have I missed something?
PS: The example binary works fine on my machine.

check https://github.com/media-kit/media-kit/issues/852

yep. Thanks.


Fix ffprobe example

This example wasn't working since the err and metadata were swapped.  This should resolve it.

Thanks


[BUG] Adding filtered-out tag to an image while browsing filtered images results in error.

If the selection is filtered, by say "not DELETE", and you add the filtered out tag to an image, in this instance "DELETE. After the tag is applied, the page changes to "Unsupported Media Type of".
Nothing breaks besides the workflow, as going back to the filtered selection shows the selection as expected, but it makes it slightly annoying to filter out images to later delete as adding any image to that selection. Ideally, after the filter is re-applied when adding the tag and the image we are currently viewing should be filtered; the app should automatically navigate to the next/previous image in the selection.

Hi @Orksork
thank you for reporting this issue.
I can understand that your workflow which you describe can be annoying.
For me this workflow is new and I tag the images on the photo stream and do mostly tagging on multiple images.
The reason why the image is not refreshed has following background: When you copy the URL to share it the app needs to load and the media might not be there immediately. So the current behavior and expectation is that the URL is valid. As example take an image from the demo gallery: demo.home-gallery.org/view/812444025568 It takes a bit of time until the page is loaded and the image is shown.
In your case it is on purpose that the media is not in the selection any more - but it is loaded. So to cover the case of initial loading/waiting and removing from the current select this logic needs to be implemented. PRs are highly welcome.
One other solution could be that the single view has an button on the detail pane to add or remove this image from the selection and you perform the tag action later.
Thanks again to share your thoughts and workflow. Do you have another proposal to solve this issue?

Hi @Orksork
after thinking a bit about your report I like your workflow to tag images for deletion. #47 has requested this feature, too.
Would it be an next iteration step to define a special tag for deleted images like DELETE. TRASHED or galleryignore which is automatically filtered by the webapp? So trashed images are not shown. The question for me is how to show the deleted images again if the delete action needs to be undone. Further, should the server already filter the images with a deleted tag, too?
Maybe continue your comment on #47.

@Orksork this is fixed through c33d70d in the new master build or the newest Docker image. Please retest.

The issue with "Unsupported Media Type of" is solved.
For the rest: I am closing this as duplicate of https://github.com/xemle/home-gallery/issues/47.


Tutorial: timed lava

Make a tutorial on how to do timed lava, like described at https://www.reddit.com/r/ROBLOXStudio/comments/tkb74y/how_do_i_create_timed_lava_like_this_is_roblox/

Added this in examples.


Stream combinator to produce non-greedy chunks

The Chunks combinator does not yield a next chunk until it is full or the stream ends.
Sometimes, however, it's useful to yield the accumulated items for processing as soon as the upstream polls Pending. This might mean that new items won't be available in a while, or that a read buffer from which the items were decoded has been consumed and another OS read is in order, so it would make sense for the application to offer backpressure before attempting to consume that in turn.

Hi @cramertj! I'd like to try and tackle this, if that's alright. From what I understand, this would essentially be a modification of the Chunks type that returns currently buffered items if the underlying stream returns Poll::Pending. Is this correct? What would should the function / type be named?
ChunksLazy? StreamExt::chunks_lazy

This has been addressed since 0.3.5 with the ready_chunks combinator.


[Bug] there are Spelling mistakes and inappropriate naming

Code of Conduct

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

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
In TFrontendService.scala the comment hive-service-prc should be hive-service-rpc
the HiveTBinaryFrontend.scala should be rename to HiveTBinaryFrontendService
，the same pattern as SparkTBinaryFrontendService、FlinkTBinaryFrontendService
Affects Version(s)
master
Kyuubi Server Log Output
No response
Kyuubi Engine Log Output
No response
Kyuubi Server Configurations
No response
Kyuubi Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

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

lets rm the service from spark and flink instead？be shorter be better

lets rm the service from spark and flink instead？be shorter be better
may be more appropriate to keep the service suffix for consistency

lets rm the service from spark and flink instead？be shorter be better

@yaooqinn, IMO, it's better to rename by adding Service which is directly known as service.

Ok


Routing error when 2fa page on subdomain

Let's say you have set-up two factor authentication on sub-domain "admin.example.com"
Scenario:
Log-in to get prompted for 2nd factor code
Don't enter your code, navigate to "example.com" without subdomain
You'll get a routing error. Gem tries to show 2nd factor form page.
Possible fix:
Replace path with url in two_factor_authentication/controllers/helpers.rb:33
change_path = "#{scope}_two_factor_authentication_url"

This PR solves such issues in a different way https://github.com/Houdini/two_factor_authentication/pull/157


Accurate predictor sometimes returns AOS before when_utc

Discovered by @mmodenesi. Using orbit-predictor < 1.9:
import datetime as dt
from orbit_predictor.sources import MemoryTLESource
from orbit_predictor.locations import Location

lines = ('1 40014U 14033E   15328.83189478  .00002037  00000-0  19542-3 0  9993', '2 40014  97.9563 235.2185 0033978  56.8458 303.6015 14.93852659 78029')

source = MemoryTLESource()
source.add_tle(42, lines, dt.datetime(2019, 4, 18, 13, 21, 30, 509345))

location = Location("test_gs", 1, 2, 3)

when = dt.datetime(2016, 7, 1, 0, 37, 22)

non_precise_pred = source.get_predictor(42, precise=False)
precise_pred = source.get_predictor(42, precise=True)

print(when)
print(non_precise_pred.get_next_pass(location, when, max_elevation_gt=0).aos)
print(precise_pred.get_next_pass(location, when, max_elevation_gt=0).aos)

Output:
2016-07-01 00:37:22
2016-07-01 00:37:22
2016-07-01 00:37:13

And, using orbit-predictor 1.9.1:
/home/juanlu/Satellogic/orbit-predictor/orbit_predictor/sources.py:66: FutureWarning: There is no `precise=False` predictor anymore and the parameter will be removed in the future
  FutureWarning,
2016-07-01 00:37:22
2016-07-01 00:37:13.370938
2016-07-01 00:37:13.370938

Labeling this as a bug.

...Or rather, as a question: see subsequent PRs.

This is a feature, not a bug, and now it's clearly tested and documented. Closing.


Trouble running evaluate.py

I am having trouble running evaluate.py after training the graphRNN. The issue seems to be the function load_graph_list in utils.py (starting at line 459):
# load a list of graphs
def load_graph_list(fname,is_real=True):

    for i in range(len(graph_list)):
        edges_with_selfloops = graph_list[i].selfloop_edges()
        if len(edges_with_selfloops)>0:
            graph_list[i].remove_edges_from(edges_with_selfloops)
        if is_real:
            graph_list[i] = max(nx.connected_component_subgraphs(graph_list[i]), key=len)
            graph_list[i] = nx.convert_node_labels_to_integers(graph_list[i])
        else:
            graph_list[i] = pick_connected_component_new(graph_list[i])
    return graph_list

This function calls the undefined list graph_list, and it does not load any graphs from file. I suspect that the code listed here was intended for a different function.

I have the same problem.

Hi, have you solved the problem?

there seems to be an attribute error but it is caught by a generic except, so the error message is not helpful.


When a verse contains multiple lines, only the first line is copied but not the ones after

Not sure if I'm the only one having this issue, but when using the Copy Bible verses command for books or verses with line breaks for each verse (particularly, verses with poetic structure like in the Psalms), only the first line of the verse gets copied, and it proceeds to the next verse with the verse before that being incomplete. Here's an example:

[[Ps. 23#1|Ps. 23,1-6]] The Lord is my shepherd, I shall not want;    he makes me lie down in green pastures.    he restores my soul. Even though I walk through the valley of the shadow of death, Thou preparest a table before me Surely goodness and mercy shall follow me [[Ps. 23#1|]][[Ps. 23#2|]][[Ps. 23#3|]][[Ps. 23#4|]][[Ps. 23#5|]][[Ps. 23#6|]]

or

[[Ps. 23#1|Ps. 23,1-6]]
The Lord is my shepherd, I shall not want;
he makes me lie down in green pastures.
he restores my soul.
Even though I walk through the valley of the shadow of death,
Thou preparest a table before me
Surely goodness and mercy shall follow me[[Ps. 23#1|]][[Ps. 23#2|]][[Ps. 23#3|]][[Ps. 23#4|]][[Ps. 23#5|]][[Ps. 23#6|]]

I'm copying all six verses of Psalm 23, but it's missing a lot of parts: Psalm 23:1 only contains one line, so it's complete; but Psalm 23:2 has two lines, but only the first lines gets copied and then the plugin copies the 3rd verse while ignoring the second line of the 2nd verse.
Am I missing something from the settings that can fix this issue, or is this an actual bug?
Appreciate your response and everything you're already doing with this plugin!

Hi,
line breaks in verses are not supported as of right now, but I think that adding support for them should be (hopefully) relatively easy. I just have a very busy schedule now, so it may take a while (but probably not more than two weeks).
Could you please send me one of your bible files (with the line breaks)? Thank you

Ps. 23.md
Here you go @kuchejak and thank you so much for the quick response. Will also be praying for you with regard to your busy schedule if you don't mind 😉

Hi, multiline verses should be now working (version 1.2.4). If you want them to be copied to separate lines, set Put each verse on a new line? to true, otherwise multiline verses will be merged into one line.


Add latest planned meeting to news page

The dates of future regular meetings are known, so we can show on the news page information about the upcoming meeting.
How about using a check to see if the latest published meeting is from the future? Or in the list of meetings find those that have not yet been held?  I think the fastest way will work the first way, assuming that the data entered is one by one and there are no forgetful people to complete the information :)

:tada: This issue has been resolved in version 1.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


DPL-757 As PSD I want to create a first draft design for RVI new project (multipick) that will reflect lims architecture and required components to solve the current list of user requirements obtained.(S=M; C=L)

Description
As PSD I want to create a first draft design for RVI new project (multipick) that will reflect lims architecture and required components to solve the current list of user requirements obtained.
Who the primary contacts are for this work
Andrew Sparkes
Stuart McHattie
Eduardo Martin
Knowledge or Stake holders
Other people that may have specific knowledge about this work or have a stake in how it is implemented. e.g. John Smith is an expert on x
Additional context or information
diagram
The goals for the RVI consolidation work can be broken down into three parts:
Sample selection
Sample selection will likely be based on use of sample metadata and may require use of an algorithm.
Alternatively sample selection can be dictated by results from previous rounds of sequencing and analysis (e.g. detection of a novel virus with poor coverage in a Bait capture experiment would trigger shotgun metagenomics). - non-automated process
Currently actual sample selection logic and required metadata is unknown and will likely change. I (TW) believe this service/s and logic should be supported within GSU rather than inside PSD. Although - we will likely need to provide an interface and/or support.
For now we should consider the what data is required as input for the plate consolidation service - as this will form the output of the sample selection process.
Plate consolidation service (Cherry Picking)
This service will pick a predefined list of samples from a set of source plates onto one or at most 2 destination plates.
Samples will have been pre-stamped into shallow well plates
Requirements

Source plates & samples may have already be cherrypicked and/or sequenced on other studies/pipelines/cost codes (Heron) - history must be preserved.
Cherry picked destination shallow well plates may go on to storage in -80C freezers.
Destination shallow well plates may be subjected to further rounds of automated or manual cherry picking to select samples for specific pipelines and studies. i.e. destination plates may become sources plates
Bed verification is a requirement.
Separation of this service from SequenceScape / Limber is desirable. Preferably via RabbitMQ - although this maybe PostMVP.
A front end UI to guide the lab user through the process and flag up errors

The data inputs to this service interface are expected to be:

A set of sample ids, source plate barcodes, and well positions
Number of randomised wells to be left empty on the destination plates for controls (do we need to specify control type too?)
Destination plate types
A study id to record the picking operation under
User who is requesting this
Submission data in order to automatically submit the destination plates to a given sequencing pipeline (Could be post MVP - if this is possible manually?)
Volumes?

The data outputs are:

Destination plate barcodes and contents (inc controls)
Destination plate types
Data about any submission made.
Information about success / failure / errors
Events to log the operation in MLWH Events DB

Sequencing logic / Downstream RVI pipelines
RVI Samples will likely be sequenced using different methods/pipelines. Most likely in Limber/SequenceScape but currently unknown.
This may be flagged at the stage of the initial cherry picking in terms of defining the submission/s for the destination plate/s (for example plate will be sequenced using Heron and Bait capture and might be described as ‘parallel sequencing’).
Alternatively sequencing might undergo iteration (‘iterative sequencing’: in this case samples would be sequenced using one (or more methods) depending on the results they would then be selected for sequencing with another method (this might require a further consolidation step.)
Whole destination shallow well plates may go on to be submitted to specific pipelines and studies.
Source samples may have already been sequenced on other studies/pipelines/cost codes (Heron) - history must be preserved.
Outputs

[ ] A rough diagram for the top level components of a minimal flexible cherry picking sample service (include connections between applications & databases )
[ ] A set of user stories for implementing such a service (include consideration of error handling and error paths, changes required to the Biosero interface / platform)
[ ] One research story for each sequencing pipeline to be implemented (see RVI diagram)

Deprecating this in favour of a hopefully smaller research story DPL-779


Draft gRPC

@fibonacci1729,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

I'm seeing this when deploying draftd via draft init with the new client:
2017/08/08 18:34:12 transport: http2Server.HandleStreams received bogus greeting from client: "GET /ping HTTP/1.1\r\nHost"

This is causing the pod to fail its readiness health check, so it's always in a 1/2 CrashLoopBackOff state.
EDIT: and now I see the new commit fixing this. Thanks!

So I was able to get the stages running locally, but when Helm is attempting to install the package I'm seeing this error pop up:
Pushing Docker Image: success
Releasing Application: started
Releasing Application: Release "elevated-skunk" does not exist. Installing it now.
Releasing Application: failure: could not install release: rpc error: code = Unknown desc = release elevated-skunk failed: Ingress.extensions "elevated-skunk-elevated-skunk" is invalid: spec.rules[0].host: Invalid value: "elevated-skunk.86d48673956bc03a6d0d6101ae546b386351cd63%!(EXTRA string=k8s.local)": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

I'll see if I can do a bit more diving before our pairing session later today :)

Awesome. Looks like just a few linter fixes and we're good to merge.

<3


serializable_hash deeper dup

I ran into an issue where options[:except] is a frozen array, which explodes when we try to concat values in serializable_hash.  This also means that we're generally modifying the passed in value, despite the dup, since it's shallow. To fix this, could we do a deep dup?
alternatively, we could do:
options[:except] = Array(options[:except]).dup

Merged, thanks @dpep. :+1: