How to avoid repeated JMESPath patterns in styles?

Currently in the styles there are some patterns that are repeated over and over. For example, to check if a pre-commit hook exists I write something like:
{
  rules: [
    {
      files: [".pre-commit-config.yaml"],
      JMESPathsMatch: [
        [
          "length(repos[?repo=='https://github.com/DavidAnson/markdownlint-cli2'])",
          1,
          "set(@, 'repos', insert(repos, `-1`, {rev: gh_tags('DavidAnson', 'markdownlint-cli2', True)[0], repo: 'https://github.com/DavidAnson/markdownlint-cli2', hooks: [{id: 'markdownlint-cli2'}]}))",
        ],
        [
          "regex_match('^v\\d+\\.\\d+\\.\\d+$', repos[?repo=='https://github.com/DavidAnson/markdownlint-cli2'] | [0].rev)",
          true,
        ],
        [
          "type(repos[?repo=='https://github.com/DavidAnson/markdownlint-cli2'] | [0].hooks)",
          "array",
        ],
        [
          "contains(repos[?repo=='https://github.com/DavidAnson/markdownlint-cli2'] | [0].hooks[*].id, 'markdownlint-cli2')",
          true,
        ],
      ],
    },
  ],
}

Which only works in fix mode if the object for the hooks of the repository doesn't exist previously. This is not very clever and so much verbose.

Proposal 1
I would like to just write something like:
{
  rules: [
    {
      files: [".pre-commit-config.yaml"],
      JMESPathsMatch: [
        [
          "pre_commit_hook_exists('https://github.com/DavidAnson/markdownlint-cli2', `[{"id": 'markdownlint-cli2'}]`)"
          true,
          "pre_commit_hook_fix('https://github.com/DavidAnson/markdownlint-cli2', `[{"id": 'markdownlint-cli2'}]`)",
        ],
      ],
    },
  ],
}

That would be really good and very reusable. The problem is that these pre-commit oriented functions shouldn't exist along the builtin JMESPath functions included with project-config because the list of functions could increase a lot supporting other functions.
So it would be really great to allow adding plugins that could extend the standard functions?
Implementation details: these plugins shouldn't be named plugins to not confuse with project-config's "plugins" naming, so they will not be needed to be included in plugins: [] array. Just when installing this hypotetical project-config-pre-commit library, the functions will be available to use in JMESPath verbs.

Proposal 2
Other option, probably more consistent, is to add third party plugins for these pre-commit related actions, which currently is possible:
{
  plugins: ['pre-commit'],
  rules: [
    {
      files: [".pre-commit-config.yaml"],
      preCommitConfigHookExists: [
        [
          "https://github.com/DavidAnson/markdownlint-cli2",
          [{id: 'markdownlint-cli2'}]
        ],
      ],
    },
  ],
}

I think that 2 is the best option even if we have to define the plugin in base.json5:plugins array.

Whatever proposal would make #143 unnecesary in most cases.


NIFI-13196 Backport: Add a new isAutoTerminated(Relationship) method

to ProcessContext

Summary
NIFI-13196
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-13196
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-13196

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
I compiled with contrib-check and I ran some flows in the UI.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

UI Contributions
No UI changes
Licensing
No new files or dependencies
Documentation
No documentation changes

I am closing this PR because I just read that we are not changing the NiFi 1.X API anymore


Install options are not appended when using brew upgrade

[x] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[x] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
Trying to upgrade packages that were provided the --ignore-dependencies option when installed.
e.g.
# Originally installed with
# brew install --ignore-dependencies yarn

brew upgrade yarn

What happened (include command output)
The package was upgraded, but dependencies were also installed unexpectedly, which causes other issues in my environment.

Command output

==> Upgrading 1 outdated package:
yarn 1.12.3 -> 1.15.2
==> Upgrading yarn
==> Installing dependencies for yarn: node
==> Installing yarn dependency: node
==> Downloading https://homebrew.bintray.com/bottles/node-11.13.0.mojave.bottle.tar.gz
Already downloaded: /Users/klanchman/Library/Caches/Homebrew/downloads/0a818793b2cd5d3f9d480d515bbf3987b57e88b7f78b36a8264667f0fdefe056--node-11.13.0.mojave.bottle.tar.gz
==> Pouring node-11.13.0.mojave.bottle.tar.gz
==> Caveats
Bash completion has been installed to:
/usr/local/etc/bash_completion.d
==> Summary
🍺  /usr/local/Cellar/node/11.13.0: 4,417 files, 50.4MB
==> Installing yarn
==> Downloading https://yarnpkg.com/downloads/1.15.2/yarn-v1.15.2.tar.gz
Already downloaded: /Users/klanchman/Library/Caches/Homebrew/downloads/3bcbb19bee227218e877e51b3aa3387de2e2cdfc9e65f6190a73a9550cb3b283--yarn-v1.15.2.tar.gz
🍺  /usr/local/Cellar/yarn/1.15.2: 14 files, 4.7MB, built in 3 seconds
==> Caveats
==> node
Bash completion has been installed to:
/usr/local/etc/bash_completion.d

What you expected to happen
The package is upgraded, and dependencies are not installed.
Discussion
The help text for brew upgrade reads as follows:
Usage: brew upgrade [options] formula

Upgrade outdated, unpinned brews (with existing and any appended install
options).

So I'd expect the --ignore-dependencies option would be appended to the upgrade command. (Although, it seems --ignore-dependencies is not a valid option for upgrade.)
I usually upgrade all my packages at once using brew upgrade, so even if --ignore-dependencies was a valid option, it still wouldn't accomplish my goal of ignoring dependencies only on packages that were originally installed ignoring dependencies.
Step-by-step reproduction instructions (by running brew commands)
# Install, ignoring dependencies, an outdated version of Yarn so we can upgrade it
brew install --ignore-dependencies https://raw.githubusercontent.com/Homebrew/homebrew-core/533d40af8ce3a932ab68be51ca2f20a2786669b6/Formula/yarn.rb

# Upgrade Yarn
brew upgrade yarn

Config/Doctor Output

Command output

➤ brew config
HOMEBREW_VERSION: 2.0.6
ORIGIN: https://github.com/Homebrew/brew
HEAD: 649d1427f5fe41323c436608f5c21513f8f34f71
Last commit: 7 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 4516ee029aac77079a6b1368d8d5402ed573a94b
Core tap last commit: 3 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_LOGS: /Users/klanchman/Library/Logs/Homebrew
HOMEBREW_NO_AUTO_UPDATE: true
HOMEBREW_NO_INSTALL_CLEANUP: true
CPU: octa-core 64-bit skylake
Homebrew Ruby: 2.3.7 => /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/bin/ruby
Clang: 10.0 build 1001
Git: 2.21.0 => /usr/local/bin/git
Curl: 7.54.0 => /usr/bin/curl
Java: 1.8.0_172
macOS: 10.14.4-x86_64
CLT: N/A
Xcode: 10.2
➤ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!
Warning: Homebrew's sbin was not found in your PATH but you have installed
formulae that put executables in /usr/local/sbin.
Consider setting the PATH for example like so
echo 'set -g fish_user_paths "/usr/local/sbin" $fish_user_paths' >> ~/.config/fish/config.fish
Warning: Some installed formulae are missing dependencies.
You should brew install the missing dependencies:
brew install node
Run brew missing for more details.

(Although, it seems --ignore-dependencies is not a valid option for upgrade.)

Yes, this is intentional. --ignore-dependencies is a developer flag for testing the dependency system. Instead of trying very hard to not install node adjust your PATH so your node of choice is before it and ignore it.

Opened https://github.com/Homebrew/brew/pull/5970 to address the documentation on both these cases.

Thanks for the documentation update, that definitely helps!

adjust your PATH

Sorry, I should have been more specific about the "other issues in my environment".
I use nvm to manage Node versions. My PATH is set with my nvm-installed Node before the brew-installed version (truncated PATH below):
... /Users/klanchman/.nvm/versions/node/v10.15.2/bin ... /usr/local/bin ...

When there is a brew-installed version of Node, nvm starts throwing errors when launching a terminal in VS Code, leaving me with the brew version of Node. VS Code's documentation essentially says to remove the global version of Node to fix the issue (in this case, the brew version).
So I can't really "ignore" the brew version of Node. The fact that it's there at all causes me issues, regardless of my PATH. 😓

With the new documentation update clarifying that --ignore-dependencies is an unsupported option, I also plan to open an issue against Yarn's documentation, because they say to use the option if you use something like nvm. However, I want to make sure I have some alternative to suggest to them. Based on other issues I've seen in this repo, I think a tap might be one alternative. Are there any other alternatives you'd suggest?

So I can't really "ignore" the brew version of Node. The fact that it's there at all causes me issues, regardless of my PATH. 😓

I see. That's the underlying thing we should figure out how to fix or at least communicate more effectively. Reading that guide it still sounds a bit like it should be possible to adjust your .bash_profile such that which node in VS Code gets what's expected?

With the new documentation update clarifying that --ignore-dependencies is an unsupported option, I also plan to open an issue against Yarn's documentation, because they say to use the option if you use something like nvm.

Yeh, thanks, that would be a great update.

Are there any other alternatives you'd suggest?

I think we should figure out a way to make the node that Homebrew installs behave nicely so that these workarounds aren't needed. It's not normal for a package manager to install a package (yarn in this case) that doesn't work unless you have manually installed some software outside the package manager.

it still sounds a bit like it should be possible to adjust your .bash_profile such that which node in VS Code gets what's expected

I saw something that alluded to that, which felt unfortunate for me to need to deal with since I use Fish.
I set up my .bash_profile to load nvm and installed Node via Homebrew again. VS Code's terminal was still not happy with this configuration. I also saw some people talking about changing some shell options in VS Code, which also didn't work.
Finally I upgraded nvm from v0.33.11 to v0.34.0 after looking through this thread. I am no longer seeing errors in VS Code, so I suppose nvm is handling this scenario now.
Thanks!

Finally I upgraded nvm from v0.33.11 to v0.34.0 after looking through this thread. I am no longer seeing errors in VS Code, so I suppose nvm is handling this scenario now.

Nice, that's good to know 👍. Thanks for providing more information here!


[OSX] 0.36.7 webview got dark and electron crashes

As the title said, first the webview got dark and then electron crashes.
It may be due to networking issue as it happened a little time after I lost wifi network.
Here the 2 crash dumps
crash dumps.zip

Both crashes were because of out of memory, though they came from different parts of code. I'm closing this since this is not something we can fix.
Stack trace:
Crash reason:  EXC_BREAKPOINT / 0x00000002
Crash address: 0x109eba961
Process uptime: 81 seconds

Thread 5 (crashed)
 0  Electron Framework!base::debug::BreakDebugger() [debugger_posix.cc : 256 + 0x0]
    rax = 0x0000000000000400   rdx = 0xffffffffffffffe0
    rcx = 0x0000000000000020   rbx = 0x0000000000000000
    rsi = 0x00007fcada013fe0   rdi = 0x000000010d810b04
    rbp = 0x00007000022aae60   rsp = 0x00007000022aae60
     r8 = 0x0000000000000040    r9 = 0x00007fff7a29b1e0
    r10 = 0x00007fff9dbf2501   r11 = 0xfffff035282972a0
    r12 = 0x00007000022ab2e8   r13 = 0x1e008c3605b20706
    r14 = 0x00007000022ab2f0   r15 = 0x00007000022ab2f8
    rip = 0x0000000109eba961
    Found by: given as instruction pointer in context
 1  Electron Framework!logging::LogMessage::~LogMessage() [logging.cc : 601 + 0x5]
    rbp = 0x00007000022ab2d0   rsp = 0x00007000022aae70
    rip = 0x0000000109ed16df
    Found by: previous frame's frame pointer
 2  Electron Framework!base::(anonymous namespace)::OnNoMemory(unsigned long) [memory.cc : 18 + 0x8]
    rbp = 0x00007000022ab420   rsp = 0x00007000022ab2e0
    rip = 0x0000000109ee7f5f
    Found by: previous frame's frame pointer
 3  Electron Framework!content::ChildDiscardableSharedMemoryManager::AllocateLockedDiscardableSharedMemory(unsigned long, int) [child_discardable_shared_memory_manager.cc : 302 + 0x8]
    rbp = 0x00007000022ab550   rsp = 0x00007000022ab430
    rip = 0x000000010a7eb313
    Found by: previous frame's frame pointer
 4  Electron Framework!content::ChildDiscardableSharedMemoryManager::AllocateLockedDiscardableMemory(unsigned long) [child_discardable_shared_memory_manager.cc : 178 + 0xd]
    rbp = 0x00007000022ab600   rsp = 0x00007000022ab560
    rip = 0x000000010a7eb05c
    Found by: previous frame's frame pointer
 5  Electron Framework!SkDiscardableMemory::Create(unsigned long) [SkDiscardableMemory_chrome.cc : 38 + 0x14]
    rbp = 0x00007000022ab630   rsp = 0x00007000022ab610
    rip = 0x000000010b88334f
    Found by: previous frame's frame pointer
 6  Electron Framework!SkResourceCacheDiscardableAllocator::allocPixelRef(SkBitmap*, SkColorTable*) [SkResourceCache.cpp : 169 + 0x3]
    rbp = 0x00007000022ab680   rsp = 0x00007000022ab640
    rip = 0x000000010b90f9fb
    Found by: previous frame's frame pointer
 7  Electron Framework!<name omitted> [SkBitmap.cpp : 273 + 0x9]
    rbp = 0x00007000022ab6a0   rsp = 0x00007000022ab690
    rip = 0x000000010b891ef1
    Found by: previous frame's frame pointer
 8  Electron Framework!SkImageGenerator::tryGenerateBitmap(SkBitmap*, SkImageInfo const*, SkBitmap::Allocator*) [SkImageGenerator.cpp : 149 + 0x11]
    rbp = 0x00007000022abb00   rsp = 0x00007000022ab6b0
    rip = 0x000000010b8d71ab
    Found by: previous frame's frame pointer
 9  Electron Framework!SkImageCacherator::generateBitmap(SkBitmap*) [SkImageGenerator.h : 173 + 0x11]
    rbp = 0x00007000022abb90   rsp = 0x00007000022abb10
    rip = 0x000000010b8d64b1
    Found by: previous frame's frame pointer
10  Electron Framework!SkImageCacherator::lockAsBitmap(SkBitmap*, SkImage const*) [SkImageCacherator.cpp : 118 + 0xb]
    rbp = 0x00007000022abc00   rsp = 0x00007000022abba0
    rip = 0x000000010b8d659a
    Found by: previous frame's frame pointer
11  Electron Framework!SkImage::preroll(GrContext*) const [SkImage.cpp : 64 + 0xc]
    rbp = 0x00007000022abc70   rsp = 0x00007000022abc10
    rip = 0x000000010b92e456
    Found by: previous frame's frame pointer
12  Electron Framework!cc::(anonymous namespace)::ImageDecodeTaskImpl::RunOnWorkerThread() [image_decode_controller.cc : 82 + 0x5]
    rbp = 0x00007000022abd40   rsp = 0x00007000022abc80
    rip = 0x000000010a22150f
    Found by: previous frame's frame pointer
13  Electron Framework!cc::TaskGraphRunner::RunTaskWithLockAcquired() [task_graph_runner.cc : 418 + 0x8]
    rbp = 0x00007000022abe30   rsp = 0x00007000022abd50
    rip = 0x000000010a20622e
    Found by: previous frame's frame pointer

Crash reason:  EXC_BAD_INSTRUCTION / 0x00000001
Crash address: 0x101288dc4
Process uptime: 288 seconds

Thread 0 (crashed)
 0  Electron Framework!atom::(anonymous namespace)::Crash() [atom_bindings.cc : 28 +
 0x0]
    rax = 0x0000000000000003   rdx = 0x0000020000000200
    rcx = 0x0000000000000000   rbx = 0x0000000105cf75a0
    rsi = 0x0000000105ce3d77   rdi = 0x0000000105cf75a0
    rbp = 0x00007fff5ea40490   rsp = 0x00007fff5ea40490
     r8 = 0x0000000000000040    r9 = 0x00007fff7a29b110
    r10 = 0xffffffffffffffff   r11 = 0x00007fdc920fd668
    r12 = 0x00007fff5ea40d80   r13 = 0x00007fddb902d620
    r14 = 0x0000000105cf75a0   r15 = 0x0000000105ce3d77
    rip = 0x0000000101288dc4
    Found by: given as instruction pointer in context
 1  Electron Framework!atom::(anonymous namespace)::FatalErrorCallback(char const*,
char const*) [atom_bindings.cc : 40 + 0x5]
    rbp = 0x00007fff5ea405e0   rsp = 0x00007fff5ea404a0
    rip = 0x0000000101288db9
    Found by: previous frame's frame pointer
 2  libnode.dylib!v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [api.
cc : 295 + 0xc]
    rbp = 0x00007fff5ea411d0   rsp = 0x00007fff5ea405f0
    rip = 0x000000010571bf81
    Found by: previous frame's frame pointer
 3  libnode.dylib!v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [factory.cc : 79 + 0x11]
    rbp = 0x00007fff5ea41220   rsp = 0x00007fff5ea411e0
    rip = 0x00000001058cbab9
    Found by: previous frame's frame pointer
 4  libnode.dylib!v8::internal::Runtime_AllocateInTargetSpace(int, v8::internal::Object**, v8::internal::Isolate*) [runtime-internal.cc : 246 + 0x8]
    rbp = 0x00007fff5ea41250   rsp = 0x00007fff5ea41230
    rip = 0x0000000105b21c0d
    Found by: previous frame's frame pointer

OK, thank you for your answer.
Is there a way to known if it was due to a memory leak in my code or if it would be solved by increasing max_old_space_size?

You can use the profile tools in devtools to find out the memory leak, increasing max_old_space_size won't really work, it would just delay the crash.


python38Packages.pyradios: 0.0.22 -> 1.0.1

Motivation for this change
Successor of #154098
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

botamusique
python310Packages.pyradios
python39Packages.pyradios


Adjustment of the voting scheme and updated viral taxonomy

Recent changes introduced by @guille0387 in the context of solving some pending manuscript revision.
We both tested the branch on example data, and I already open the PR while still some things need to be solved:
To run the code, a new taxonomy sqlite file is needed, which should be added to the auto-download procedure of the pipeline before merging this PR. e.g. available here:

https://u.pcloud.link/publink/show?code=XZ9eYEVZLuaHtlP4ryRyYBJRQ3bm78myPDwX

Also, this file is needed to be updated (as unzipped version):

additional_data_vpHMMs_v3.tsv.zip

Then, assuming that the new additional_data_vpHMMs_v3.tsv is in the nextflow-autodownload-databases/models folder, once can run this branch via
nextflow run virify.nf --fasta kleiner_virome_2015.fasta --output results/kleiner_2023 --ncbi nov_2022_taxdb.sqlite --virome -profile local,docker

Further, we discovered that the container for chromomap is not available: quay.io/microbiome-informatics/r_chromomap:0.2
@mberacochea can you please add it to your registry?

Recent changes introduced by @guille0387 in the context of solving some pending manuscript revision.
We both tested the branch on example data, and I already open the PR while still some things need to be solved:
To run the code, a new taxonomy sqlite file is needed, which should be added to the auto-download procedure of the pipeline before merging this PR. e.g. available here:
* https://u.pcloud.link/publink/show?code=XZ9eYEVZLuaHtlP4ryRyYBJRQ3bm78myPDwX

Also, this file is needed to be updated (as unzipped version):
* [additional_data_vpHMMs_v3.tsv.zip](https://github.com/EBI-Metagenomics/emg-viral-pipeline/files/11599995/additional_data_vpHMMs_v3.tsv.zip)

Then, assuming that the new additional_data_vpHMMs_v3.tsv is in the nextflow-autodownload-databases/models folder, once can run this branch via
nextflow run virify.nf --fasta kleiner_virome_2015.fasta --output results/kleiner_2023 --ncbi nov_2022_taxdb.sqlite --virome -profile local,docker

Further, we discovered that the container for chromomap is not available: quay.io/microbiome-informatics/r_chromomap:0.2
@mberacochea can you please add it to your registry?

Hey @hoelzer and @guille0387,
This is great, Guille sent me the new version of "additional_data_vpHMMs_v3.tsv" a few ago .. I've been away for a bit, so just catching up.
A few things:

[ ] We should rename additional_data_vpHMMs_v3.tsv -> additional_data_vpHMMs_v4.tsv (or similar). We should keep v3 of the file around, some people may need it in case they used the current release.
[ ] I'll fix the container, strange that is missing.
[ ] Sure, I'll update the .sql file too.
[ ] We should rename contig_taxonomic_assign_2.py => contig_taxonomic_assign.py
[ ] Add a small python unit test for that script, @guille0387 do you have a small toy example I can use for the test?

Please let me know if this make sense.

Hey @mberacochea !
I think v3 for the additional_data_vpHMMs_v3.tsv is fine, because the current version of the pipeline has only v1 and v2:
https://github.com/EBI-Metagenomics/emg-viral-pipeline/blob/master/nextflow/modules/metaGetDB.nf#L22
Besides, I agree with your ToDo list!
Another thing: we only worked on the nextflow code to get things done for the paper revision. The changes then also need to be reflected in CWL.
thx!

Hi @mberacochea,
I agree with what @hoelzer said in the previous comment. Also, here's a small file that you can use for the test.
test_set.tsv.zip

Hey @mberacochea !
I think v3 for the additional_data_vpHMMs_v3.tsv is fine, because the current version of the pipeline has only v1 and v2:
https://github.com/EBI-Metagenomics/emg-viral-pipeline/blob/master/nextflow/modules/metaGetDB.nf#L22
Besides, I agree with your ToDo list!
Another thing: we only worked on the nextflow code to get things done for the paper revision. The changes then also need to be reflected in CWL.
thx!

:man_facepalming: ... for some reason I was conviced the file we had was v3... anyway. Perfect then.
CWL... our team is moving to Nextflow, so unless CWL is on the paper I wouldn't spend time on it. We can just freeze on v1 for CWL.

CWL... our team is moving to Nextflow, so unless CWL is on the paper I wouldn't spend time on it. We can just freeze on v1 for CWL.

Ah, good to know. Then we could maybe also remove the CWL part from the manuscript @guille0387 when this is anyway not longer supported.

Hey @mberacochea we are in the process of finalizing the new data and results for the paper revision. So far, it looks really good: we get more taxonomical annotations e.g. on the 243 TARA ocean MAGs, compared to the former pipeline version and backend taxonomy.
It would be great if we can finalize this PR and make another release by the end of June to also put the new pipeline version that we're using now into the manuscript. What's the status regarding the ToDos you listed? Can we help w/ smt? thx!

Hey @mberacochea we are in the process of finalizing the new data and results for the paper revision. So far, it looks really good: we get more taxonomical annotations e.g. on the 243 TARA ocean MAGs, compared to the former pipeline version and backend taxonomy.
It would be great if we can finalize this PR and make another release by the end of June to also put the new pipeline version that we're using now into the manuscript. What's the status regarding the ToDos you listed? Can we help w/ smt? thx!

That is great! Apologies for the delay on my side. I agree, we need this one merged and a new release of the pipeline ASAP. I'll try to make that happen as soon as I can. I don't think there is anything in particular I need help with. I'll keep you both posted.

Hey @mberacochea , @guille0387 and I are on finalizing the manuscript revision - any updates regarding the PR and open ToDos? thx a lot!

Hey @mberacochea , @guille0387 and I are on finalizing the manuscript revision - any updates regarding the PR and open ToDos? thx a lot!

Hey.. almost there. I just need to fix the container and upload the files.. and we should be done.

Folks. I'm going to merge this one.. and will run some tests using DEV

Folks. I'm going to merge this one.. and will run some tests using DEV

awesome! thx so much @mberacochea !


Duplicate class a.a found in modules jetified-android-sdk-1.3.0-runtime

Describe the bug
After upgrading com.paypal.checkout:android-sdk:1.3.0, we are unable to build our application due to the following gradle error:
> Task :app:checkDebugDuplicateClasses FAILED
...
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugDuplicateClasses'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.CheckDuplicatesRunnable
   > Duplicate class a.a found in modules jetified-android-sdk-1.3.0-runtime (com.paypal.checkout:android-sdk:1.3.0) and jetified-msdk-3.0.5-runtime (com.perimeterx.sdk:msdk:3.0.5)
     Duplicate class a.b found in modules jetified-android-sdk-1.3.0-runtime (com.paypal.checkout:android-sdk:1.3.0) and jetified-msdk-3.0.5-runtime (com.perimeterx.sdk:msdk:3.0.5)
     Duplicate class a.c found in modules jetified-android-sdk-1.3.0-runtime (com.paypal.checkout:android-sdk:1.3.0) and jetified-msdk-3.0.5-runtime (com.perimeterx.sdk:msdk:3.0.5)
     Duplicate class a.d found in modules jetified-android-sdk-1.3.0-runtime (com.paypal.checkout:android-sdk:1.3.0) and jetified-msdk-3.0.5-runtime (com.perimeterx.sdk:msdk:3.0.5)
...

As seen in the error there are class conflicts with another package installed by com.perimeterx.sdk:msdk:3.0.5 which was not upgraded, and seems to also have a globally scoped package and classes (I will raise an issue over there as well).
Looking at the changes made in 1.3.0, the updated version of android-magnessdk-5.5.0.jar seems to have introduced a new globally scoped obfuscated package called a with 4 new classes a.a, a.b, a.c, and a.d. Looking at the external libraries installed, we can see the mentioned packages have the same globally scoped a module in both com.perimeterx.sdk:msdk:3.0.5 and com.paypal.checkout:android-sdk:1.3.0.

To Reproduce
Steps to reproduce the behavior:

Install com.paypal.checkout:android-sdk:1.3.0 and com.perimeterx.sdk:msdk:3.0.5

For the latter, requires the following repository: maven { url "https://perimeterx.jfrog.io/artifactory/px-Android-SDK/" }

Attempt to build this project.
See the above error.

Expected behavior
Both modules should be able to be installed together without class conflicts, perhaps Magnes SDK needs to be updated to not have obfuscated those globally scoped classes.
Looking at the previous version android-magnessdk-5.4.0.jar, there is no trace of this globally scoped a module seen in this jar dump:
jar -tvf ~/Downloads/android-magnessdk-5.4.0.jar
  1234 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/Environment.class
   636 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/InvalidInputException.class
  1045 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesResult.class
  9851 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesSDK.class
  4838 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesSettings$Builder.class
   342 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesSettings$a.class
  3153 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesSettings.class
  1548 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/MagnesSource.class
  1308 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/a.class
  3785 Thu Jan 01 01:01:02 EST 1981 lib/android/paypal/com/magnessdk/b.class
  ...

An update on the status of this, I was able to work with the PerimeterX team to get this resolved in their library and we are now able to install their latest version alongside the latest PayPal Checkout SDK version. However I still consider this a bug in the PayPal SDK as it could be causing similar issues for other consumers.


Handle nested associations in searchConverter

Context
In PR #102 @patzick implemented associations. All the stuff work in that PR work very well.
We found one more requirement to cover the use case of adding nested associations.
Dominic provided us with following example:
product/7b7a3457a26d4919ad94897d61e1dc6e?associations[properties][associations][media][]=1

and also there is a following JSON format:
"associations": {
  "media": {
    "associations": { "product": {} }
    }
}

which adds association for category -> media -> product.
Acceptance Criteria

searchConverter allows developers to add nested associations.
It is possible to add 1st level associations.
It is possible to add 2nd level associations.
It is possible to add 3nd level associations.

@rmakara done with #123


Improve NewVertexArray

type and size checking
error checking

Can this take numpy arrays of floats ?
Would certainly be useful for interop, and can be useful for interop.

# x, y,

1.0, 0.0,
1.0, 0.0, 0.0,

-0.5, 0.86,
0.0, 1.0, 0.0,

-0.5, -0.86,
0.0, 0.0, 1.0

The format parameter is already useless since the dimension and type of the vertex attributes can be queried runtime.
I am not planning to remove it because:

thrown errors are more programmer-friendly
Later will have a support for padding 3f4x4f (3 float, 4 bytes ignored, 4 floats)

If there would be stronger cons I will definitely remove it.

#65

Solving #72 will fix the numpy related issue.


[REVIEW] ItsRoy69

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

-- your profile looks great!  Since you have focused more on detailing you projects, learnings and that's the best part I love :)
--And that hidden tab is really awesome 😃.


Show certificate

Show certificate information of a service
Script: show-certificate
show-certificate google.com

Depends on openssl.

I have checked the following requirements before making this PR.

[x] The script is marked as executable (chmod +x).
[x] I have added/updated the README entry for the script (if applicable).
[x] I have mentioned any outstanding dependencies and OS requirements for the script in the README.

Thanks for the contribution @uli-heller.


Not abstract super-classes mess up test run order

TestNG Version
7.1.0
Description
So i have classes A, BA, BB, BC and C
Classes A, BA, BB and C have tests in them (2 tests per class test1 and test2)
and Class BC doesn't
Classes BA and BB inherit class BC
So when i run my tests i expect tests order be like this
Expected behavior
A.test1
A.test2
BA.test1
BA.test2
BB.test1
BB.test2
C.test1
C.test2
Actual behavior
A.test1
C.test1
C.test2
A.test2
BA.test1
BA.test2
BB.test1
BB.test2
Is the issue reproductible on runner?

[ ] Shell
[x] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[x] IntelliJ
[ ] NetBeans

Root cause
so the reason for that seems to be the following:
DynamicGraphHelper.createClassDependencies() looks like this
 private static ListMultiMap<ITestNGMethod, ITestNGMethod> createClassDependencies(
      ITestNGMethod[] methods, XmlTest test) {
    Map<String, List<ITestNGMethod>> classes = Maps.newHashMap();
    // Note: use a List here to preserve the ordering but make sure
    // we don't add the same class twice
    List<XmlClass> sortedClasses = Lists.newArrayList();
    for (XmlClass c : test.getXmlClasses()) {    <--------------this gets all the classes even without test methods
      classes.put(c.getName(), new ArrayList<>());
      if (!sortedClasses.contains(c)) {
        sortedClasses.add(c);
      }
    }

and then later same method does the following
 for (ITestNGMethod m : methods) {
      String name = m.getTestClass().getName();
      Integer index = indexedClasses1.get(name);
      // The index could be null if the classes listed in the XML are different
      // from the methods being run (e.g. the .xml only contains a factory that
      // instantiates methods from a different class). In this case, we cannot
      // perform any ordering.
      if (index != null && index > 0) {
        // Make this method depend on all the methods of the class in the previous
        // index
        String classDependedUpon = indexedClasses2.get(index - 1);
        List<ITestNGMethod> methodsDependedUpon = methodsFromClass.get(classDependedUpon);   <---------------when class with no tests methods appear hear, the methods of that suppose to depend on those depend on nothing
        for (ITestNGMethod mdu : methodsDependedUpon) {
          result.put(mdu, m);
        }
      }
    }

so test methods of class C which goes after class BC which has no test methods are considered not depending on anything and later when TestRunner.privateRun() calls graph.getFreeNodes() it gets A.test1 and all the C class test methods.
There is a simple solution to this issue - making BC class abstract will exclude it from test.getXmlClasses()
However maybe it's acceptable to filter out classes with no test methods in DynamicGraphHelper.createClassDependencies itself, if that's the case i'd be more than happy to submit a pull request

@sergejjkim - Since you have done the analysis on the cause of the problem, why not raise a PR to fix the problem ?

@krmahadevan hi again, i've submitted a pull request.
https://github.com/cbeust/testng/pull/2250
looking forward to comments :)


fix(autoware_detection_by_tracker): fix funcArgNamesDifferent

Description
This is a fix based on cppcheck funcArgNamesDifferent warnings
perception/autoware_detection_by_tracker/src/tracker/tracker_handler.cpp:26:71: style: inconclusive: Function 'onTrackedObjects' argument 1 names different: declaration 'input_objects_msg' definition 'msg'. [funcArgNamesDifferent]
  const autoware_perception_msgs::msg::TrackedObjects::ConstSharedPtr msg)
                                                                      ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

Thanks for the merge.


when to release v1.0?

have so many errors?want to know when to release v1.0?

Yep, we are keeping on working to make freeline a better tool for android. We still have some problems when developers used freeline in the first time and we are making progress with this to help developers to integrate with freeline more easier.
There is no a specific time schedule for the v1.0 releasing. But we are working on more features, such as application proxy, annotation support and so on. Hope the version 1.0 can come soon.


Upgrade to 0.10.5 causes 'Script: Run' (ctrl-shift-B) command (in the script package) to fail

As soon as I reverted to 0.10.4, Script: Run worked properly again. No other changes were made (by me).

This is related to a change of script scope name in my grammar. I have raised an issue at Atom-script and hopefully it will be resolved. If not I will do a PR on it and see if it will be accepted. In the meantime you can change either my grammar  file or atom-script config file so they both have the same string.
From: MexieAndCo
Sent: Friday, 7 August 2015 20:37
To: gandm/language-babel
As soon as I reverted to 0.10.4, Script: Run worked properly again. No other changes were made (by me).
—
Reply to this email directly or view it on GitHub.

Thanks. I made the change to the atom-script lib/grammars.coffee file and updated language-babel to 0.10.5. The Script: Run command is working again.


[VS Code] [Audio] Calling module not available. Cannot find module 'slimcore-2.0'

Error:
Calling module not available. Cannot find module 'slimcore-2.0'
Steps to Reproduce:

Version Data

extensionName
VSLS-Audio

extensionVersion
0.1.39

applicationName
VSCode

applicationVersion
1.28.2

platformName
Linux

platformVersion
4.15.0-38-generic

@jasongin

@rustan-id Hey! Could you try uninstalling the audio extension and reinstalling it? This issue may indicate that they install process failed for some reason. In the meantime, I'm going to close this as a duplicate of #827.


Fix typos, refine wording in InstrumentYourLibrary.md

Some of these fixes are subjective, but in addition to fixing typos I tried to reorder words or break up too long sentences for improved readability.

@swift-server-bot add to allowlist

Thanks @MaxDesiatov !


CloudTrail "/" character

Hi,
how did you handle the "/" character in the CloudTrail logs? The Athena doesn't support the "/" in column name and unfortunately there are "/" character in keys in json files from the ClouTrail.

Help me understand better. The following WHERE clause, as an example, works.
userIdentity.arn = 'arn:aws:sts::111111111111:assumed-role/role_name/timothy.orr'
Perhaps I'm benefiting from the table creation, which is accomplished via CloudTrail Partitioner.
query = """CREATE EXTERNAL TABLE IF NOT EXISTS {table_name} (
eventversion string COMMENT 'from deserializer',
useridentity struct<type:string,principalid:string,arn:string,accountid:string,invokedby:string,accesskeyid:string,username:string,sessioncontext:struct<attributes:structmfaauthenticated:string,creationdate:string,sessionissuer:structtype:string,principalid:string,arn:string,accountid:string,username:string>> COMMENT 'from deserializer',
eventtime string COMMENT 'from deserializer',
eventsource string COMMENT 'from deserializer',
eventname string COMMENT 'from deserializer',
awsregion string COMMENT 'from deserializer',
sourceipaddress string COMMENT 'from deserializer',
useragent string COMMENT 'from deserializer',
errorcode string COMMENT 'from deserializer',
errormessage string COMMENT 'from deserializer',
requestparameters string COMMENT 'from deserializer',
responseelements string COMMENT 'from deserializer',
additionaleventdata string COMMENT 'from deserializer',
requestid string COMMENT 'from deserializer',
eventid string COMMENT 'from deserializer',
resources array<structarn:string,accountid:string,type:string> COMMENT 'from deserializer',
eventtype string COMMENT 'from deserializer',
apiversion string COMMENT 'from deserializer',
readonly string COMMENT 'from deserializer',
recipientaccountid string COMMENT 'from deserializer',
serviceeventdetails string COMMENT 'from deserializer',
sharedeventid string COMMENT 'from deserializer',
vpcendpointid string COMMENT 'from deserializer')
PARTITIONED BY (region string, year string, month string, day string)
ROW FORMAT SERDE
'com.amazon.emr.hive.serde.CloudTrailSerde'
STORED AS INPUTFORMAT
'com.amazon.emr.cloudtrail.CloudTrailInputFormat'
OUTPUTFORMAT
'org.apache.hadoop.hive.ql.io.HiveIgnoreKeyTextOutputFormat'
LOCATION '{cloudtrail_log_path}'""".format(
table_name=table_name, cloudtrail_log_path=cloudtrail_log_path
)

Ok, the problem was I created the table using AWS Glue so for example the responseelements had type "struct" not string. When table is automatically created using AWS Glue the structures have "/" in column names which is not supported by the Athena. All is sorted now, thanks!
BTW, great repo! Thank you for sharing.

Thank you. Please let me know if you find any parts of it useful, have questions, or have any ideas for fixes/enhancements.


Versioning information

It would be very helpful to have the current version of the framework in its homepage ink.sapo.pt.
For now, the fastest way I can find to see what's the latest version available is to hit "Download" and check the file name on the download window.
Thank you guys.

Indeed it would :) Thx

That was quick and the result very effective. Thank you.


Update to the Collectives' transactions page

Recently we updated the expense component to be displayed in public environments like the Collective page, along with a new design for the transactions page.
After new rounds of feedback and design iteration, we need to do some further improvements to the transactions page of Collectives to ensure consistency and a better user experience.

Transaction list
Expense details pop-up

Add tooltip to see the last commentAdd fees breakdown tooltip
Add tooltip to indicate the user to go to the standalone expense pageAdd fees breakdown tooltip

Check the design file 🎨
@kewitz @Betree

Awesome @Memo-Es! Labeled important due to recent requests/complaints related to transaction transparency.

The expense popup update does not apply since it has been removed.
However, it looks like the tooltip proposal to show the amount breakdown can solve #4438. Ping @znarf

@Memo-Es The first proposal (Transactions list) is missing the action buttons for "Refund" and "Reject"

You also aren’t able to sort ‘kind’ by refunds


Get remote config template by version number

Add support to get remote config template by version number

NOTE: API proposal is still under review. Getting a head start

Updated and added a new test case for version number validation

Hi @egilmorez! Please review the docs for the new changes. Thanks!


KeyNotFoundException at startup

Hey ^^
There's this exception:
[Error  :  HarmonyX] Error while running static void HostFixes.Patches+HostInitialization::Postfix(Terminal __instance). Error: System.Collections.Generic.KeyNotFoundException: The given key 'UnlockableItem' was not present in the dictionary.
  at System.Collections.Generic.Dictionary`2[TKey,TValue].get_Item (TKey key) [0x0001e] in <787acc3c9a4c471ba7d971300105af24>:IL_001E
  at AdvancedCompany.Game.Manager+Items.GetUnlockablePrice (UnlockableItem unlockable) [0x00017] in G:\Unity\AdvancedCompany\UpgradeCompany\Game\Manager\Items.cs:311
  at AdvancedCompany.Game.Manager.GetNodeCost (TerminalNode node) [0x00138] in G:\Unity\AdvancedCompany\UpgradeCompany\Game\Manager.cs:430
  at (wrapper dynamic-method) HostFixes.Patches+HostInitialization+<>c.DMD<HostFixes.Patches+HostInitialization+<>c::<Postfix>b__0_0>(HostFixes.Patches/HostInitialization/<>c,CompatibleNoun)
  at System.Linq.Enumerable+WhereArrayIterator`1[TSource].MoveNext () [0x0002c] in <dab7f68612224ba3ae40f651d44f9d4c>:IL_002C
  at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement] (System.Collections.Generic.IEnumerable`1[T] source, System.Func`2[T,TResult] keySelector, System.Func`2[T,TResult] elementSelector, System.Collections.Generic.IEqualityComparer`1[T] comparer) [0x000a3] in <dab7f68612224ba3ae40f651d44f9d4c>:IL_00A3
  at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement] (System.Collections.Generic.IEnumerable`1[T] source, System.Func`2[T,TResult] keySelector, System.Func`2[T,TResult] elementSelector) [0x00000] in <dab7f68612224ba3ae40f651d44f9d4c>:IL_0000
  at HostFixes.Patches+HostInitialization.Postfix (Terminal __instance) [0x00036] in <1da95813babc4036ade75285a0729c99>:IL_0036
  at (wrapper dynamic-method) Terminal.DMD<Terminal::Awake>(Terminal)

Which is sent with every startup.
Hope this gets fixed :)

I can't debug it because Advanced Company was pulled off of Thunderstore by the author.

https://thunderstore.io/c/lethal-company/p/PotatoePet/AdvancedCompany/

It is up again

ok


Trace function of certain 1x1 matrix expressions can cause a segfault at solve time

My favorite library maintainers,
This following code snippet quietly segfaults in cvxpy version 1.0.1 (exit code -11):
import cvxpy as cvx
X_C = cvx.Variable((1, 1), PSD=True)*1 # segfaults
# X_C = cvx.Variable((1, 1), PSD=True)*cvx.Constant(1) # also segfaults
# X_C = cvx.Variable((1, 1), PSD=True) # fine
cons = [cvx.trace(X_C)==1]
cvx.Problem(cvx.Maximize(X_C[0,0]), cons).solve(solver="SCS", verbose=True)

All 1817 nose tests are passing. I'm on Ubuntu 16.04 (fresh install) with the anaconda distribution:
    cvxcanon-0.1.1 (cvxgrp)
    cvxpy-1.0.1 (cvxgrp)
    dill-<IP_ADDRESS>
    ecos-2.0.5 (cvxgrp)
    libgcc-7.2.0
    multiprocess-0.70.4 (cvxgrp)
    scs-1.2.6 (cvxgrp)

Valid DCP should not segfault, so I thought you'd like to know. Also, in previous versions this trick of multiplying semidefinite variable definitions by hard-coded scalings was able to effectively alter the initial step size, and was the difference between solving and not solving several problems with CVXOPT's interior point solver. (I'd try it with CVXOPT, but I'm still not passing nose tests with CVXOPT)

Uh oh! We will look into this.

I neglected to check non semi-definite variables. They also segfault:
# X_C = cvx.Variable((1, 1)) # fine
# X_C = cvx.Variable((1, 1))*1 # also segfaults
# X_C = cvx.Variable((1, 1)) * cvx.Constant(1) # also segfaults

This issue is resolved, at least for Python 2.7 and cvxpy 1.0.5.
Example code below (no segfaults).
(cvxpy1dev) Rileys-MacBook-Pro:research RJMurray$ python
Python 2.7.14 |Anaconda, Inc.| (default, Dec  7 2017, 11:07:58)
[GCC 4.2.1 Compatible Clang 4.0.1 (tags/RELEASE_401/final)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import cvxpy
>>> x = cvxpy.Variable((1,1))
>>> y = 1*x
>>> y = x*1
>>> print(cvxpy.__version__)
1.0.5

@GrayThomas please try again on your machine (after updating to 1.0.5), and let us know if it's fixed for you as well.

After updating to 1.0.5, that works for me (still using Python 3.6.5). But, to my surprise, my original test case still fails. Digging a little deeper, it seems to be related to the trace function---and also seems to be specific to the case where the argument of the trace function is a 1 by 1 matrix, but the error occurs when the problem is solved (from your code snippet above, it looks like I miscommunicated the location of the segfault). Having the trace of a 1x1 matrix that is the result of multiplication within either the objective or an equality constraint (inequality constraints untested) is sufficient to reproduce the segfault (of prob.solve()) behavior. I'll update the name of the issue to reflect the greater clarity I have now. I'm still a little worried that the conditions that can cause a segfault are more general than the example below (for instance, I only ever tried expressions with trace), but I only have a limited amount of time available to diagnose.
import cvxpy
import numpy as np
x = cvxpy.Variable((1,1)) # this problem does not occur with 2x2 matrices
print(cvxpy.__version__)
dangerous_expression = cvxpy.trace(x*1) # segfault on prob.solve()
# dangerous_expression = x*1 # fine
# dangerous_expression = cvxpy.trace(x) # fine
# dangerous_expression = cvxpy.trace(x*1+0) # Also fine!?
# dangerous_expression = cvxpy.trace(x*1+cvxpy.Constant(np.array([[0,],]))) # fine
# dangerous_expression = cvxpy.trace(x*1+x*1) # fine
# dangerous_expression = cvxpy.trace((x*1+x*1)*1) # segfault on prob.solve()
# dangerous_expression = cvxpy.trace(1*(x*1+x*1)) # segfault on prob.solve()
def trace(X): # quick workaround
	if X.shape==(1,1): # trivial 1x1 matrix case
		return X
	return cvxpy.trace(X)
# dangerous_expression = trace(x*1) # fine
# dangerous_expression = trace((x*1+x*1)*1) # fine
safe_expression = x
prob = cvxpy.Problem(cvxpy.Minimize(safe_expression),[dangerous_expression==1])
# prob = cvxpy.Problem(cvxpy.Minimize(dangerous_expression),[safe_expression==1]) # segfaults under the same conditions
prob.solve() # the segfault happens on this line, if it happens.

@GrayThomas thank you for providing more detail! I'm working on this now.

The problem is unrelated to the trace function. The following example also segfaults
import cvxpy as cvx
X_C = cvx.Variable((1, 1), PSD=True)*1 # segfaults
# X_C = cvx.Variable((1, 1), PSD=True)*cvx.Constant(1) # also segfaults
# X_C = cvx.Variable((1, 1), PSD=True) # fine
# X_C = cvx.Variable((1, 1), PSD=True) * 1 + 0 # fine
# cons = [cvx.trace(X_C)==1]
cons = []
prob = cvx.Problem(cvx.Maximize(X_C[0,0]), cons)
prob.solve()

Yikes, i never saw this issue. I dug into the C++ stack trace for Riley's code example:
    frame #0: 0x00000001194a5058 _cvxcore.cpython-37m-darwin.so`add_triplets(tripletList=size=0, slices=size=2, dims=size=0, axis=1, col_offset=0, row_offset=0) at LinOpOperations.cpp:571 [opt]
    frame #1: 0x00000001194a17e6 _cvxcore.cpython-37m-darwin.so`get_index_mat(lin=0x00007ffeefbfd600) at LinOpOperations.cpp:610 [opt]
    frame #2: 0x000000011949e6b8 _cvxcore.cpython-37m-darwin.so`get_func_coeffs(lin=<unavailable>) at LinOpOperations.cpp:83 [opt]
    frame #3: 0x00000001194932cc _cvxcore.cpython-37m-darwin.so`get_coefficient(lin=0x00007ffeefbfd600) at cvxcore.cpp:80 [opt]
    frame #4: 0x0000000119493d7a _cvxcore.cpython-37m-darwin.so`process_constraint(lin=0x00007ffeefbfd600, V=size=0, I=size=0, J=size=0, constant_vec=size=1, vert_offset=0x00007ffeefbfd6fc, id_to_col=size=1, horiz_offset=0x00007ffeefbfd6dc) at cvxcore.cpp:145 [opt]
    frame #5: 0x000000011949497b _cvxcore.cpython-37m-darwin.so`build_matrix(constraints=size=1, id_to_col=size=4294967295) at cvxcore.cpp:224 [opt]

In line 571 of LinOpOperations.cpp, dims is a 0-length vector and axis is 1, hence the segfault.
I'm not familiar with this code. @SteveDiamond , can you take a look when you get a chance? The issue has to do with indexing into a 1 by 1 expression.

For reference, on a mac, you can inspect the C++ stack trace by dropping into lldb. After copying @rileyjmurray 's script into a file named segf.py, enter the following at the command line:
lldb python segf.py
Then, at the lldb console, type run and push enter:
(lldb) run

to obtain the following output:
Process 73810 launched: '/Users/Akshay/virtualenvs/cvxpy-dev/bin/python' (x86_64)
first is ok
_cvxcore.cpython-37m-darwin.so was compiled with optimization - stepping may behave oddly; variables may not be available.
Process 73810 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x4)
    frame #0: 0x0000000119eea058 _cvxcore.cpython-37m-darwin.so`add_triplets(tripletList=size=0, slices=size=2, dims=size=0, axis=1, col_offset=0, row_offset=0) at LinOpOperations.cpp:571 [opt]
   568    int step = slices[axis][2];
   569    int pointer = start;
   570    while (true) {
-> 571      if (pointer < 0 || pointer >= dims[axis]) {
   572        break;
   573      }
   574      int new_offset = col_offset + pointer*vecprod_before(dims, axis);
Target 0: (python) stopped.
(lldb)

Typing bt will print the stack trace. You can print variables using the print command, and travel up and down the stack using up and down.

Sure, I'll look into this.

Ok this is fixed now. I added a special case for 1x1.


Actor rotation is desynced when playing montage with rootmotion which changes rotation

because it's intentionally ignored in AAlsCharacter::OnRep_ReplicatedBasedMovement

Does this only happen on simulated proxies?
ACharacter::OnRep_ReplicatedBasedMovement() should not affect root motion since it is handled separately in ACharacter::OnRep_RootMotion().

I found the cause:
https://github.com/Sixze/ALS-Refactored/blob/bfe487e706c4b7e0959f77cf2bbcaa42dbab6055/Source/ALS/Private/AlsCharacter.cpp#L169-L174

It's disabled because the root motion is replicated from the server, and the roll still works fine, even though it uses root motion too.

Fixed in 8ba31f6080a375a207ee12f044688b928a4683e0.


Use Microsoft.VisualStudio.PackageGroup.NuGet (16.1)

Fixes #4307, depending on the new PackageGroup that comprises the
correct NuGet package for the current SKU.

@zivkan, look good?


Copying bower -> package.json to allow for yarn installations

Reference Issue

I'm not sure how this would actually help with yarn installations. Wouldn't a user need to use a git dependency in their package.json for Polymer, which is rather discouraged?

Closing, no response and not intending to make this change otherwise.


Bump scalding version to 0.16.0

No particular reason, but it has been more than a year since the scalding version was last updated.

Have you tested any jobs on 0.16.0?

@vineethvarghese No I haven't. Do you mean using dependencyOverrides for just scalding & algebird?
My original plan was to roll it out to all the commbank components one-by-one (using wrangler), deal with any breakages along the way, and not test a job until maestro/coppersmith have been rebuilt (if at all).

:) I am happy if you are taking this up. It was on @rowandavies pipeline anyway.

@toddmowen So far I've successfully run all the maestro tests locally with scalding 0.16 via a branch of uniform.  My next step was going to be to test the stash project etl.test-jobs on the cluster, but I can't do that until I get back now (in 4 weeks).  If that works, we have some confidence, and it could be reasonable to roll out 0.16.  Of course, we do need to be prepared to deal with any tweaks required in our libraries if issues come up in jobs.
(This reminds me that I was going to propose a Scala Omnia Libraries guild to coordinate activities around our libraries.)

@toddmowen Are you happy to push this across the stack i.e. etl-plugins and update the version on etl.tests.job? This will be a quick way to run a test on the cluster and it is all version update across the stack.

@rowandavies Obviously I didn't realise you had already started working on this. Did you rebuild all the (commbank) dependencies of maestro as well? Did you run into any issues with 0.16 along the way?
I have nothing waiting on this change, and so I don't mind leaving it until you get back. Or I can take it up now.

@rowandavies I'm ready to pick this up again. I have scripts ready to automate a fair amount of the drudge work (for github repos at least). Based on your previous investigations, do you know of any problems that I'm likely to run into with the new scalding version?
If not, then I will optimistically assume that it is backwards compatible and no code changes to our libraries will be required...

Apologies, I never got to this.  No, I didn't find any issues, but I didn't get that far beyond the maestro tests.  It should be backward compatible and we can always have uniform go to the previous version if we have to.
I think I like autowrangler, but I'm not sure I have time to understand the detail fully.  Perhaps we should meet briefly?


[BUG] Trigger (un) registering leads to Iroha looped panic

GIT commit hash
docker.io/hyperledger/iroha2:lts@sha256:4072d09bee376c8fcbc586d35ea25624738694ce2c37aa0db60c9998c613048b
Minimum working example
https://github.com/soramitsu/orillion-core-middleware/tree/develop/smart-contracts/reward-mint-with-cap-smart-contract source code
https://github.com/soramitsu/orillion-core-middleware/tree/develop/services/admin/src/main/resources mint_with_cap is the compiled wasm
Expected behaviour
Iroha went panicing
After container death it is restarted and seems to behave okay for some time but then it's started again
Actual behaviour
Correct unregister+register of the trigger (for the same id though)
Operating system
k8s, dev env
LibC type and version
No response
Current environment
Docker Hub
Logs in JSON format
2022-09-13T14:52:53.506685Z  INFO iroha: Hyperledgerいろは2にようこそ！
2022-09-13T14:52:53.506736Z  INFO iroha: (translation) Welcome to Hyperledger Iroha 2!
2022-09-13T14:52:53.506822Z  INFO iroha: Starting peer listen_addr=<IP_ADDRESS>:1337
2022-09-13T14:52:53.506844Z  INFO iroha_p2p::network: Binding listener listen_addr=<IP_ADDRESS>:1337
2022-09-13T14:52:53.506927Z  INFO iroha_p2p::network: Starting network actor listen_addr=<IP_ADDRESS>:1337
2022-09-13T14:52:58.756188Z  WARN apply: iroha_core::wsv: The following errors have occurred during trigger execution errors=[
0: Failed to find. Failed to find domain: dn
1: Failed to find domain: dn
Location:
core/src/wsv.rs:151
Backtrace omitted.
Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.
Warning: SpanTrace capture is Unsupported.
Ensure that you've setup a tracing-error ErrorLayer and the semver versions are compatible]
other ton of logs https://drive.google.com/file/d/1jgEDovjgKAnzKE-GTHjLS-9Ozh_lFxO-/view?usp=sharing
Who can help?
No response

Registering and unregistering trigger works fine with Iroha2-lts brunch.

Checked in one and in 2 separate transactions.

Checked in one and in 2 separate transactions.

have you checked that running 4 peers?


[bazel] sw/device/silicon_creator/lib:sigverify_dynamic_functest has undocumented dependencies

sw/device/silicon_creator/lib:sigverify_dynamic_functest depends on files in :sigverify_mod_exp_ibex and :sigverify_mod_exp_otbn.
These dependencies are added to sw/device/silicon_creator/lib/BUILD in https://github.com/lowRISC/opentitan/pull/10041 to fix the build so it doesn't block CI

This was fixed by #10350 and other issues with the tests were resolved and they pass and are enforced by CI now


Add Service Group concept for service filter

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[x] Feature or performance improvement

Requirement or improvement
SkyWalking has the capability to observe thousands of services, which have different types, such as agent-observed, service-mesh, Istio control panel, Envoy data panel, ShardingSphere proxy(etc.). All of them should have different dashboards due to we enhanced in the past 2 versions.
Now, I intend to add this.

Service Group is a string name-based concept, added to the service list.
Provide a new optional condition(group) in the GraphQL. If provided, use it to filter the service list, otherwise, like we did today.
The group name would not be updated due to the ServiceTraffic is supportUpdate = false. So, it is only updated(actually added) once the TTL reached and data has been removed.
UI should be updated to add the current filter condition in the GraphQL. cc @Jtrust

If #5835 merged fast, this would be added in the 8.4.0.
This concept is helpful for supported group based topology in the future, but note, this is not the goal of this PR.

Provide a new optional condition(group) in the GraphQL. If provid to filter  ser


The function findAssetsInViteManifest can slow down SSR significantly

In production mode, the function findAssetsInViteManifest is called multiple times in SSR mode.
With a large app and many dependencies, it can take a long time to execute. SSR time goes up from 9ms to 900ms.

Problem is that the assetMap cache contains duplicate entries, multiple thousands in our case.
One solution would be to remove duplicates before storing them in the map.
const all = Array.from(new Set([...assets, ...imports])).filter(Boolean);
return all;

Before the change:

After the change:

Thank you for posting this. We have been fighting with performance issue for a while, and have spent the past 3-4 weeks drilling down to all parts of the code Solid-Start and Vinxi code.
@jean343 Did you short-circuit the rest of the function by immediately return all (Version B)? Or did you only replace the filter line (Version A)? When we did our test, Version A (one line change) saw a 10x improvement. We did not see any performance change with (Version A vs B).
// original code
const all = [...assets, ...imports].filter(Boolean);
all.push(chunk.file);
assetMap.set(id, all);
stack.pop();
return Array.from(new Set(all));

// Version A
const all = Array.from(new Set([...assets, ...imports])).filter(Boolean); // <--- line change
all.push(chunk.file);
assetMap.set(id, all);
stack.pop();
return Array.from(new Set(all));

// Version B
const all = Array.from(new Set([...assets, ...imports])).filter(Boolean);
return all; // return early
// all.push(chunk.file);
// assetMap.set(id, all);
// stack.pop();
// return Array.from(new Set(all));

I'd love to make a PR fixing this. Thank you both for all the information and the potential solution! Can you please give me a reproduction regarding the duplicate entries, best would be a zip, but it can also be a quick step by step list. I just wanna be able to test the PR with a project that actually has the problem 😅.

It's version A above, I have a full diff for you if it helps.
--- a/node_modules/vinxi/lib/manifest/vite-manifest.js
+++ b/node_modules/vinxi/lib/manifest/vite-manifest.js
@@ -29,20 +29,23 @@ const findAssetsInViteManifest = (manifest, id, assetMap = new Map()) => {
 		}

 		stack.push(id);
-
+		const dynamicImports = id.startsWith('virtual:') ? undefined: chunk.dynamicImports;
 		const assets = [
 			...(chunk.assets || []),
 			...(chunk.css || []),
 			...(chunk.imports?.flatMap(traverse) || []),
+			...(dynamicImports?.flatMap(traverse).filter(i => i.endsWith(".css")) || []),
 		];
 		const imports = chunk.imports?.flatMap(traverse) || [];
-		const all = [...assets, ...imports].filter(Boolean);
+		const all = Array.from(new Set([...assets, ...imports])).filter(Boolean);
 		all.push(chunk.file);
 		assetMap.set(id, all);

 		stack.pop();

-		return Array.from(new Set(all));
+		return all;
 	}
 	return traverse(id);
 };

It's difficult to make a reproduction, the previous algorithm is O(n2), and it gets noticeably slow after about 4000 imports. You could make a unit test with a large manifest input!

I wonder if this why people have been saying code splitting in routing is slow. It should be an easy perf win but if it enlarges the size of these asset manifests then that could definite do it.

There is some odd stuff in this function without even looking deeper in the code.. for example it traverses imports and spreads them in twice seemingly unnecessarily? Also I'm not sure that the Boolean Filter is necessary.In anycase because it's recursive we shouldn't have to filter at ever layer just on the leaves.  And part of me wonders would we gain performance by deduping at the end instead of at each step.
I really wish I had a reproduction or just an example of what a large manifest looked like. I can definitely make this faster, but I think the slowdown depends on shape. But with the manifests I'm dealing with at best I've made it 2x faster as they aren't that big. It's possible the performance character could completely change in your use cases depending on shape of manifest.
So I'm not sure how to proceed. I can release something and see if it helps... but I'd rather be able to test something real. I don't need your code just console.log the manifest.

I am trying right now, the initial code takes 3.8 seconds.
Your version 1 takes 15ms
Your version 2 takes 46ms
Those timings are the entire SSR time and include more than this function time.
Note that it calls the function 46 times for a single page load, some times it's longer and some other times it's fast.
findAssetsInViteManifest: 0.106ms
findAssetsInViteManifest: 0.623ms
findAssetsInViteManifest: 0.489ms
findAssetsInViteManifest: 1.737ms
findAssetsInViteManifest: 0.339ms
findAssetsInViteManifest: 0.582ms
findAssetsInViteManifest: 0.143ms
findAssetsInViteManifest: 0.087ms
findAssetsInViteManifest: 0.009ms
findAssetsInViteManifest: 2.757ms
findAssetsInViteManifest: 10.715ms
findAssetsInViteManifest: 7.737ms
findAssetsInViteManifest: 5.393ms
findAssetsInViteManifest: 0.021ms
findAssetsInViteManifest: 34.103ms
findAssetsInViteManifest: 85.149ms
findAssetsInViteManifest: 606.861ms
findAssetsInViteManifest: 0.059ms
findAssetsInViteManifest: 80.593ms
findAssetsInViteManifest: 0.017ms
findAssetsInViteManifest: 1.848ms
findAssetsInViteManifest: 0.006ms
findAssetsInViteManifest: 1.943ms
findAssetsInViteManifest: 72.98ms
findAssetsInViteManifest: 0.018ms
findAssetsInViteManifest: 0.002ms
findAssetsInViteManifest: 0.004ms
findAssetsInViteManifest: 28.506ms
findAssetsInViteManifest: 0.133ms
findAssetsInViteManifest: 0.133ms
findAssetsInViteManifest: 0.039ms
findAssetsInViteManifest: 0.205ms
findAssetsInViteManifest: 0.323ms
findAssetsInViteManifest: 0.037ms

I will send you the manifest, I would prefer not pasting it in a public channel.

Yeah it's because every dynamic import needs to grab it separately which why there is so much variance. This does confirm though I think that deduping at each level is beneficial for your type of graph. I was worried there wouldn't be much of a difference. It's ok about the manifest if you don't want to. I'm ok going with the first option here after seeing that.

I'm glad it helped, the speedup is massive for our use case.
I have sent you the manifest through a private channel.

@ryansolid Adding some addition data points from my end. My app is fairly complex.
All the numbers below are for an essentially blank page on our app. Numbers are from local machine production build + start. This page has 28 requests.
Additional info
When debugging this, we logged how many times findAssetsInViteManifest was called, and also store the cumulative length/size of [...assets, ...imports] for the simple blank page:

findAssetsInViteManifest was called 2,154 times for a single page load (some other pages ended up with 16k calls)
Total combined length of [...assets, ...imports] was 224,173, which was filtered down to 40,449
For each run of findAssetsInViteManifest the size of [...assets, ...imports] varies, the largest I've seen is around 3,xxx.

 Happy to test our more complex pages. Huge performance gains on my end too.

The only thing giving me pause is that it appears micro-optimizations in this routine are still impacting overall time which means it is still a bottleneck and we can probably do better. I probably should just ship this first though because it is such a significant improvement.

Ok.. I have released the next version. I was able to unwind the flatMap and remove closures make it another 33% faster but I'm at my limits of micro-optimizations. The real question is if the assetMap can be shared across more calls. Or if SolidStart can make less calls. Still a huge win here I think.


Cancel in-progress publish jobs when a new one comes in

These changes are now killing publish jobs in production if a new PR gets merged. If we have sufficient merges in a row, we'll be actually delaying updates in the live docs site.
Should we exclude production from these concurrency checks?


artifactory_keypair tries update immutable resource

I manage the GPG keys using the resource artifactory_keypair, as follows:
resource "artifactory_keypair" "artifactory_gpg" {
  pair_name   = "artifactory-gpg"
  pair_type   = "GPG"
  alias       = "artifactory"
  private_key = file(var.gpg_key_file)
  public_key  = file(var.gpg_pub_file)
}

Such a definition registered the new GPG key in Artifactory as expected. I then wanted to set the password to an already existing key, as follows:
resource "artifactory_keypair" "artifactory_gpg" {
  # ... previous code here ...
  # You have to set a password for the GPG keys, even though these keys are not
  # password protected.
  # See below links for details about issues:
  # - https://jfrog.atlassian.net/browse/RTFACT-25704
  # - https://jfrog.atlassian.net/browse/RTFACT-25534
  passphrase = "fake-passwd"
}

According to Terraform Plan, there should be an attempt to complete the password to the already existing GPG key, but Terraform Apply ends with an error, as in the screenshot below.

I believe that the resource artifactory_keypair should force the deletion first and then the addition of this GPG key anew. Likewise, in the WebUI Artifactory it is no possible to change the GPG key once added.
In order to make this change, I had to first remove the GPG key from the WebUI Artifactory, then from the tfstate file. Only then could I execute Terraform Apply to add the key as a new one.
I don't know if this is the expected behaviour, but when I removed only the key from the WebUI, but left it in the tfstate file, I got this error with Terraform Plan:

Addidional details
Platform: Ubuntu GNU/Linux
Terraform Version:: 1.2.9
Provider Version: 6.20.2
Artifactory Version: 7.47.10

@piotrminkina

I believe that the resource artifactory_keypair should force the deletion first and then the addition of this GPG key anew.

We can update the passphrase attribute to do this.
When you change/remove a resource outside of Terraform, this causes a state drift. The error you see is a bug and will be fixed.
In general, please avoid changing resources outside of Terraform to avoid this kind of situation. There're times when that's not possible and in those cases, you need to manage the Terraform state carefully.


resize hook

I've been using this hook
 function useContentRect (ref) {
  const [contentRect, setContentRect] = useState(ref.current ? ref.current.getBoundingClientRect() : {});
  useEffect(() => {
    if (!ref.current) return;
    const observer = new window.ResizeObserver(entries => setContentRect(entries[0].contentRect));
    observer.observe(ref.current);
    return () => observer.disconnect(ref.current);
  }, [ref]);
  return contentRect;
}

To measure elements and react on size changes. Can be used as
function Component () {
  const ref = useRef(null);
  const { width, height } = useContentRect(ref);
  return
{width} {height}
;
}

Looks like something like this could be a nice addition to this library. If not, feel free to just close and ignore. 🙂

@Janpot, I just playing :man_playing_handball: around with useContentRect(ref). It's fantastic :+1:  and first I'm a little bit confusing :confused: from its name. I think you should rename it something else so it can easily understand the meaning behind this logic.

@Janpot I'm playing :man_playing_handball: with useContentRect(ref). It's fantastic if you want to do something on element resize. I think hook name confused me :confounded: we have put a step ahead to include this hook inside the platform library.
What's your opinion?

I picked the name because it was consistent with the naming of properties in the ResizeObserver. I prefer to use the terms of the underlying platform if they make sense. Other than that, I don't really care how it's named.


Can't detect swipe direction

I want to use this package to make something similar to react-native-calendar-strip for a project. One of our requirements is that a date is always selected as the user swipes through the weeks.
The functionality I want:

If the user swipes to the previous week, it automatically selects the Friday of that week (closest business day)
If the user swipes to the next week, it automatically selects the Monday of that week

I can't currently find a way to detect if the swipe is a left swipe or a right swipe. I think if onPageChange were modified to provide the previous page as an (optional) second argument, it would give me the information I need

hm, not a bad idea to include prev data in onPageChange.
You can get the info you want via an animated reaction, see this snack: https://snack.expo.dev/@computerjazz/infinite-pager-detect-direction


Proposed methodology

You mentioned that you will check for collinearity between variables. I suggest you clarify how you are going to deal with cases of collinearity. Are you going to do factor analysis, dimensionality reduction, or use other classification algorithms that deal with collinearity?

Because you are proposing a predictive research question. You should mention how are you going to split the data into training and testing data sets.

Hi @hsmohammed, for splitting the data, we have combined the red wine and white wine data to become one dataset. From there, we then perform a train-test split. Do you want me to add this to the proposal?
Also for checking the collinearity between variables, we were thinking of using feature correlations, not sure about other methods since we have not learned much. Could you provide us with some suggestions?

Hi @hsmohammed, we have proposed two methods including SVMs and KNN in our methods. For KNN, we are planning to do a forward or backward search, which we have just learned from DSCI 573. Also, for SVM, since collinearity is not a problem, we can keep it as it.
Reference:
https://stats.stackexchange.com/questions/35708/how-to-avoid-multicolinearity-in-svm-input-data


Reference Assemblies Not Found

Environment data
dotnet --info output:
.NET SDK (反映任何 global.json):
 Version:   5.0.404
 Commit:    72c68b830a

运行时环境:
 OS Name:     Windows
 OS Version:  10.0.22000
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\5.0.404\

Host (useful for support):
  Version: 5.0.13
  Commit:  b3afe99225

.NET SDKs installed:
  5.0.404 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.13 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET runtimes or SDKs:
  https://aka.ms/dotnet-download

VS Code version: 1.63.2
C# Extension version: 1.23.17
As you can guess, I am learning Unity
OmniSharp log
*** is the name of the project as I would not like to show it
Starting OmniSharp server at 2021/12/24下午10:30:26
    Target: d:\lrcno6\projects\***

OmniSharp server started.
    Path: c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\OmniSharp.exe
    PID: 20956

[info]: OmniSharp.Stdio.Host
        Starting OmniSharp on Windows 6.2.9200.0 (x64)
[info]: OmniSharp.Services.DotNetCliService
        Checking the 'DOTNET_ROOT' environment variable to find a .NET SDK
[info]: OmniSharp.Services.DotNetCliService
        Using the 'dotnet' on the PATH.
[info]: OmniSharp.Services.DotNetCliService
        DotNetPath set to dotnet
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Located 1 MSBuild instance(s)
            1: StandAlone 17.0.0 - "c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin"
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        MSBUILD_EXE_PATH environment variable set to 'c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin\MSBuild.exe'
[info]: OmniSharp.MSBuild.Discovery.MSBuildLocator
        Registered MSBuild instance: StandAlone 17.0.0 - "c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin"
            CscToolExe = csc.exe
            CscToolPath = c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin\Roslyn
            MSBuildExtensionsPath = c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild
            MSBuildToolsPath = c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.CSharpFormattingWorkspaceOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.Completion.CompletionOptionsProvider, Order: 0
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.RenameWorkspaceOptionsProvider, Order: 100
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.ImplementTypeWorkspaceOptionsProvider, Order: 110
[info]: OmniSharp.WorkspaceInitializer
        Invoking Workspace Options Provider: OmniSharp.Roslyn.CSharp.Services.BlockStructureWorkspaceOptionsProvider, Order: 140
[info]: OmniSharp.Cake.CakeProjectSystem
        Detecting Cake files in 'd:\lrcno6\projects\***'.
[info]: OmniSharp.Cake.CakeProjectSystem
        Could not find any Cake files
[info]: OmniSharp.MSBuild.ProjectSystem
        Detecting projects in 'd:\lrcno6\projects\***\***.sln'.
[info]: OmniSharp.MSBuild.ProjectManager
        Queue project update for 'd:\lrcno6\projects\***\Assembly-CSharp.csproj'
[info]: OmniSharp.Script.ScriptProjectSystem
        Detecting CSX files in 'd:\lrcno6\projects\***'.
[info]: OmniSharp.Script.ScriptProjectSystem
        Could not find any CSX files
[info]: OmniSharp.WorkspaceInitializer
        Configuration finished.
[info]: OmniSharp.Stdio.Host
        Omnisharp server running using Stdio at location 'd:\lrcno6\projects\***' on host 21928.
[info]: OmniSharp.MSBuild.ProjectManager
        Loading project: d:\lrcno6\projects\***\Assembly-CSharp.csproj
[fail]: OmniSharp.MSBuild.ProjectLoader
        The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks
[warn]: OmniSharp.MSBuild.ProjectManager
        Failed to load project file 'd:\lrcno6\projects\***\Assembly-CSharp.csproj'.
d:\lrcno6\projects\***\Assembly-CSharp.csproj
c:\Users\lrcno6\.vscode\extensions\ms-dotnettools.csharp-1.23.17\.omnisharp\1.37.17\.msbuild\Current\Bin\Microsoft.Common.CurrentVersion.targets(1217,5): Error: The reference assemblies for .NETFramework,Version=v4.7.1 were not found. To resolve this, install the Developer Pack (SDK/Targeting Pack) for this framework version or retarget your application. You can download .NET Framework Developer Packs at https://aka.ms/msbuild/developerpacks

[fail]: OmniSharp.MSBuild.ProjectManager
        Attempted to update project that is not loaded: d:\lrcno6\projects\***\Assembly-CSharp.csproj

Steps to reproduce
This is a Unity project and as I open this project folder with VSCode, this failure occurs
I have googled and looked through the previous issues, but their resolutions do not work here
And I have installed the .NET Framework 4.7.1 Developer Pack
Expected & Actual behavior
The Intellsense cannot work probably due to this failure

And it will be appreciated if anyone can solve this problem

A lot has changed with O# project loading since this issue was opened. Also, it is recommended to use the Unity extension for VS Code.


Add Dynamic Animation for Review Count and Overall Rating in Rating S…

Pull Request Format
PR Title Add Dynamic Animation for Review Count and Overall Rating in Rating Section
Issue #120  : ISSUE NAME solved  Fixed#120
Type of PR

Add X in the box to specify the improvement type.
[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): ___________

Description
This pull request aims to [briefly describe what the pull request does, e.g., "add a new feature," "fix a bug," "improve documentation," etc.].
Currently, the rating section in the project displays static content for the number of reviews and the overall rating. To make this section more engaging and user-friendly, implement a dynamic counting animation for both the review count and the overall rating.
Screenshots / Videos (if applicable)
Before:

[Describe the state before the changes, e.g., "The section headings in the footer were misaligned, leading to an inconsistent and cluttered appearance."]

https://github.com/user-attachments/assets/f6c07e6e-ab38-458f-9742-2c7bbe2da4c9
After:

[Describe the state after the changes, e.g., "The section headings in the footer are now properly aligned, providing a clean and professional look."]
The review count should animate smoothly from 0 to the specified number.
The overall rating should animate smoothly from 0.0 to the specified rating.
Ensure the animations do not block other content or cause layout shifts.

Checklist

Add X in the box to specify.
[X] I have performed a self-review of my code.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Provide any additional context about the changes, such as specific challenges you faced or decisions you made. This can help reviewers understand the rationale behind your changes.]
Assign the hacktoberfest accepted and gssoc extended tag
Label:-
Hacktoberfest accepted 2024
Gssoc 2024 extended
Thank you for reviewing my pull request!

Hey 👋
Please assign tag for Hacktoberfest accepted and gssoc 2024 extended
Thank you @swaraj-das


Simplify backbone_image_classifier example

What does this PR do?
In a slack discussion about LightningCLI it was suggested that untyped parameters should be added to the parser as type Any instead of being silently skipped. With the latest version of jsonargparse this is the behavior, however, this means that in the current backbone_image_classifier example the argument model.backbone will be added twice causing a failure. There is a temporal patch in jsonargparse so that this example does not fail, but this will be removed in the next version. The backbone_image_classifier has to change due to this, but also it can be made simpler as proposed here.
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[X] Did you read the contributor guideline, Pull Request section?
[X] Did you make sure your PR does only one thing, instead of bundling different changes together?
[X] Did you make sure to update the documentation with your changes? (if necessary)
[X] Did you write any new necessary tests? (not for typos and docs)
[X] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@carmocca this change is according to our discussion in slack.


Generating multiple files from a template

The Generating multiple files from a template section of Access models from text templates mentions there are several approaches for generating multiple files from a single model; however, only one method is mentioned. i.e. Splitting a long file
What are the other approaches that were alluded to?
For example, I need to generate multiple class files (from multiple templates) across multiple projects within a single solution for each element within my model.  What is the best approach for this?

@seanalford Thank you for the feedback and questions, assigning this to author for review.
Please note while we can take into account the need of a provided situation, we are not allowed to provide support or specific recommendations.
For help with these types of user scenarios, please use one of the following:

Visual Studio community
Visual Studio on Stack Overflow

The wording could be clearer here, but generally this is something you'll need to evalutate and code for your specific scenario.


Merge upstream improvements

Multiple agents upstart
Support for amazon linux
Server options
Agent level build parameters
Don't create teamcity user if already exists

This branch had more features than I expected. They should probably go into separate pull requests.

I do no longer maintain this repository. I can transfer you the ownership, as you seem to be interested in.

Also, please merge back the master branch into your PR branch. I just fixed the annoying unit-test errors.

Still failing due to 980a880. Will need to stub command.

First, thanks a lot for your contribution!
There is only one more thing. Could you add some basic checks (specs) just to check if everything works as you expect to? Just checking if your resources (templates) got created.
You can refer to spec/server_spec.rb to do so.

The user guard was originally pulled from ccleung's fork. https://github.com/ccleung/teamcity-cookbook/commit/c8eda2634e9c1cce56f33c5b7ab48c504e66dfc2
We actually have some legacy installations of TeamCity which are installed under root (not ideal). Having the teamcity cookbook change the user's homedir/gid causes issues. Prior to the patch we were unwinding the user with chef-rewind in a wrapper cookbook. I can see where this would also cause some problems if you're doing cloud installs under say ec2-user/ubuntu.

So the problem is, that the Teamcity root directory ≠ the users home directory right?
Overriding the default attributes wasn't enough to solve the issue?
chef.json = {
    teamcity_server: {
        user: 'root',
        group: 'root',
        home_dir: '/root', //users home directory
        root_dir: '/opt/TeamCity' //teamcity's root directory
    }
}

I need to understand the issue, maybe we can figure out a better solution for this.

The above would work and be preferred. I'll rewrite the branch to remove changes related to the guard.

@atward it would be awesome to add some tests if the template got created successfully and extend the Readme.md with your new params, so we can merge this. What do you think?

There's not much to test here in the context of chefspec. It doesn't change any of the existing template drops, just content - which isn't available to chefspec. If you have integration testing hooks i'd be happy to add kitchen/serverspec tests.


⚠ Bofrost.at has degraded performance

In 212ff55, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1048 ms

Resolved: Bofrost.at performance has improved in 3b1c048 after 18 minutes.


scroll-button.spec.js : Modified unit test to improve coverage

Fixes #232
[Short description explaining the high-level reason for the pull request]
Modified unit test to improve coverage of ScrollButton.vue
[If this Pull Request contains any UI changes, provide an image or GIF displaying the changes]

Developer Certificate of Origin Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

@brenoferreira This was a test addition for scrollToTop function for the ScrollButton.vue component and I added an istanbul pragma as suggested by @akmadian to ignore the testing of the functionality.

Thanks for your contribution.


Update of all custom connector files

Please check the following conditions for your PR.

[ ] apiDefinition.swagger.json is validated using paconn validate command.
[ ] apiProperties.json has a valid brand color. Invalid brand colors are #007ee5 and #ffffff.

New icon added.

New icon added.


How to skip control batch messages?

Hi @mostafa,
When consuming messages from kafka with xk6-reader, I get events like this:
{\"value\":[0,0,0,0,0,0],\"topic\":\"notification\",\"partition\":0,\"offset\":1,\"time\":\"2023-11-15T14:16:05Z\",\"highWaterMark\":2,\"headers\":{},\"key\":[0,0,0,1]}
When I consume messages with kafka-console-consume I don't see these events. But when I checked in the UI I saw events are 2 offset apart. After reading this article, it was clear that these are control batch messages. "When using transactions, Kafka insert "control batches" in the logs to indicate if messages were part of a transaction."
My question is if it's possible to skip reading these kind of empty contolbatch messages?

Hey @rhasanoff,
This extension uses kafka-go underneath and kafka-go doesn't support skipping those messages yet. There is a long-awaited unmerged PR for the specific thing you're asking, so consider skipping them in JS code for now, or give them a hand in moving that PR forward.


Rename to Visual Basic

FWIW, VB.Net is the language being used here. Classic Visual Basic ended long ago.

According to wikipedia, the name changed.
That also matches the official docs: https://learn.microsoft.com/en-us/dotnet/visual-basic/

Yeah, I guess enough time has gone by that they've done that. Well, if you ever plan to have a classic Visual Basic track, I guess it could be called that: Visual Basic Classic.

Indeed. That was my thinking.


[FEATURE] - Endpoint template concept

Remove endpoint concept from Platform and create new CRD dedicated for this usage.
To do that:

Make new EndpointTemplate that can be provided on any namespace (default to operator namespace)
On ingress / Route, create new annotations to specify the endpointTemplate to use (name and namespace)

It's fix on master branch with PR #11


Fix 'this' context when using strict mode.

In strict mode, the value of this remains at whatever it's set to when entering
the execution context. If it's not defined, it remains undefined.
see: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/this#Function_context
My usecase is that I am using the babel-core/register hook, so all required modules are transpiled and are ran in strict mode; which causes this to be undefined.

@kingcody Thanks man. I'm not familiar with all the voodoo of strict mode.


Update monitor-cockroachdb-with-prometheus.md

fix typo in the URL

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/24076d188583c5f6af40b355bbfb7d6e231a5101/