Update ray query as to framework

Description
Migrate hand coded AS to Framework AS now that RayTracing Extended has been merged.
Fixes #367
Tested Environments:
Ubuntu Linux
GeForce 2080
Vulkan version: <IP_ADDRESS>
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[ ] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[x] I have tested the sample on at least one compliant Vulkan implementation
[x] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[x] Any dependent assets have been merged and published in downstream modules
[x] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

Merging with 3 approvals.  LMK if issues


Application error: a client-side exception has occurred (see the browser console for more information).

hey there after  you updata the code，it seems that it still has the same problem

i am using mac os with edge browser i get this error also. It happens when i try to get long replies.


Add Navbar to feed page & Remove warnings

Describe the bug

Adding navbar to the feedpage
Remove warnings from the console

Screenshots

@codewithvk  please add the labels and assign it


Sorting of lists in login area does not work properly

In the dashboard --> Category lists --> cities and institutions, the sorting ascending / descending (a-z / z-a) does not work anymore, can you fix this @patrickakk?
thanks!

@PixlTracer
Where did this error occur? Was that in production?

yes! @patrickakk just now, in production at "cities" and "institutions"
https://dhcr.clarin-dariah.eu/cities
https://dhcr.clarin-dariah.eu/institutions
neither sorting by ID nor Name works.

@PixlTracer  Thanks for the bug report.
This seems to apply to all "lists" in the login area, that (a part) of the sorting doesn't work.
I've also found a possible solution.

@PixlTracer
cc @IvdL22
A fix has been implemented. Could you test it @PixlTracer, since you've reported the bug?
Is seems it affected the sorting of most of the lists in the login area. Two lists are only accessible by moderators, and don't appear in your dashboard if you are "only" administrator. I've specified that in brackets. And also tested them.
As well, as a bonus ;) the option has been added to sort on two additional columns. This is also specified in  brackets.
Here are the URL's:
https://test-dhcr.clarin-dariah.eu/users/approve (added: also sort by request date)
https://test-dhcr.clarin-dariah.eu/courses/approve
https://test-dhcr.clarin-dariah.eu/courses/expired
https://test-dhcr.clarin-dariah.eu/courses/my-courses
https://test-dhcr.clarin-dariah.eu/courses/all
https://test-dhcr.clarin-dariah.eu/courses/moderated (only mods)
https://test-dhcr.clarin-dariah.eu/users/pending-invitations (added: also sort by Invitation valid until)
https://test-dhcr.clarin-dariah.eu/users/all
https://test-dhcr.clarin-dariah.eu/users/moderators
https://test-dhcr.clarin-dariah.eu/users/moderated (only mods)
https://test-dhcr.clarin-dariah.eu/cities
https://test-dhcr.clarin-dariah.eu/institutions
https://test-dhcr.clarin-dariah.eu/logentries
https://test-dhcr.clarin-dariah.eu/logentries/errors
unchanged (no need to test, only for info):
https://test-dhcr.clarin-dariah.eu/languages
https://test-dhcr.clarin-dariah.eu/countries
Can you change the label to done, if satisfied?

hi @patrickakk
thank you for fixing!!
I checked the links in test, works fine!!

Implemented in 2024-12


update readme to reflect status of this plugin and openshift client p…

…lugin
Fixes https://github.com/openshift/jenkins-plugin/issues/157
@openshift/sig-developer-experience fyi / ptal

one nit unrelated to these changes and lgtm.

[merge]

install origin failed:  Create OpenShift certificates for master hosts.[localhost] openshift_master_facts : Set Default scheduler predicates and priorities
just going to merge manually


Fix <= and > comparisons when compared against prerelease versions

This fixes an issue with semver comparisons where <= and > comparison against a partial version (i.e., <=4.8 or <=4.8.x) accidentally include prerelease versions of the next higher patch or minor version. this is because <=4.8 is simplified to <4.9.0, but should have been simplified to <4.9.0-0 because prerelease versions are considered lower than their non-prerelease counterparts. In the same vein, > comparisons like >4.8 were incorrectly simplified to >=4.9.0, but should have been simplified to >=4.9.0-0.
Fixes #50909

>4.8 is an interesting case.  As a human, I want to read that as "4.9 or higher", because it just seems intuitive that the greater-than comparison would apply to the least significant digit in the stated version number (points in version numbers are not decimal points).  Normalizing 4.8 to 4.8.0 isn't something that would ever occur to me in the context of a comparison.

Sorry, that's a typo. >4.8 is satisfied by 4.9.0-0.

Okay, so it does align with my intuition.  Good to know.
Semver is definitely weird.  It’s not even a total order.  1.0.0-beta is less than 1.0.0 but I’m pretty sure all suffixes are considered equal to each other so there’s no suffix you can use as a “least” value…

Suffixes are ordered. -0 is less than -1, and any number segment is less than any segment containing an alpha character, thus -0 is less than -alpha, and -alpha is less than -beta. The least prerelease suffix is -0.

Huh, TIL.  I would have thought if the suffixes were ordered at all it would always be a string comparison.  Most people don’t use numbers after the dash in my experience.

https://semver.org/#spec-item-11, step 4. Though node-semver only compares prerelease versions when the major, minor, and patch versions are identical unless you pass includePrerelease (https://github.com/npm/node-semver#prerelease-tags), making everything a bit inconsistent.

Most people don’t use numbers after the dash in my experience.

I actually think it's more common that way, e.g. -beta.1, -beta.2, -rc.5, etc, as prerelease versions sort by splitting on ., so as long as you sort the non-number portions how you want (alpha comes before beta comes before rc), thinks work. For example, eslint prereleases.

Though node-semver only compares prerelease versions when the major, minor, and patch versions are identical unless you pass includePrerelease (npm/node-semver#prerelease-tags), making everything a bit inconsistent.

Just to confirm, this PR makes our semver behave like node-semve when includePrerelease is enabled, correct? I think we discussed that having that false like semver does by default is useful for package managers but not so much us who are just trying to correctly semantically order things.

IIRC, our behavior has always been to match includePrerelease: true since we're not a package manager. This has just always been a bug.

IIRC, our behavior has always been to match includePrerelease: true since we're not a package manager. This has just always been a bug.

Also since we mostly use this to test our own versions, which don't strictly follow semantic versioning...

Yeah, it's a little unfortunate that we use -dev as the nightly suffix when technically that sorts after beta. I've had confusing issues installing the beta thanks to that 🙁.

I found another bug, I'm writing up some more comprehensive tests to make sure I've got everything else covered.

This should be ready for another look @sandersn, @DanielRosenwasser


Saving few tokens.

Issue number:
Summary
Changes

Removing duplicated instructions

User experience

Please share what the user experience looks like before and after this change

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[ ] I have performed a self-review of this change
[ ] Changes have been tested
[ ] Changes are documented

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Thanks for the PR.


chore Remove useless BE execution log (#42696)

pick https://github.com/apache/doris/pull/42696

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall


[release] Release1230

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
Who can help? @ them here!
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist
Before PR:

[ ] I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
[ ] Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
[ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with some of those projects.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.15 out of 16 committers have signed the CLA.:white_check_mark: plyfager:white_check_mark: LeoXing1996:white_check_mark: VongolaWu:white_check_mark: zengyh1900:white_check_mark: ldr426:white_check_mark: austin273:white_check_mark: Z-Fran:white_check_mark: AlexZou14:white_check_mark: liuwenran:white_check_mark: lvhan028:white_check_mark: willaty:white_check_mark: curiosity654:white_check_mark: Taited:white_check_mark: whu-lee:white_check_mark: Zdafeng:x: xiaomileYou have signed the CLA already but the status is still pending? Let us recheck it.


Segmentation fault while creating a new certificate

When I try to create a new certificate I get:
Error: Let's Encrypt SSL certificate installation failed: Failed letsencrypt execution: /usr/local/psa/var/modules/letsencrypt/venv.BADTz/lib/python2.6/site-packages/cryptography/init.py:25: DeprecationWarning: Python 2.6 is no longer supported by the Python core team, please upgrade your Python.
DeprecationWarning
/usr/local/psa/admin/plib/modules/letsencrypt/scripts/le-run: line 23: 14371 Segmentation fault "$LE_HOME/venv/bin/letsencrypt" $config --renew-by-default --no-redirect --agree-tos --text --config-dir "${LE_HOME}/etc" --work-dir "${LE_HOME}/lib" --logs-dir "${LE_HOME}/logs" --authenticator letsencrypt-plesk:plesk --installer letsencrypt-plesk:plesk $@
I am using:
OS	CentOS 6.7 (Final)
Plesk version	12.5.30 Update #17
Let's Encrypt	1.2-1
Thanks for your help.

Hey, we need to find more details about failure.
Please run strace /usr/local/psa/var/modules/letsencrypt/venv/bin/letsencrypt --text

Here is the log from it.
letsencrypt_strace.zip

We have released a new version 1.6 of the extension.
Major change is a switch from system python to plesk-py27 on all unix OSes.
Distribution of own python should increase installation reliability and resolve the issue.
Please, check your installation with the latest update.
Feel free to reopen the issue if the problem is still reproduced.


feat: sim replace frontend rpc calls with genlayerjs

Fixes #580
What

Implemented GenLayer JS in the Studio frontend
Added possibility to return null on get_transaction_by_hash endpoint (was breaking when a server-side tx didn't exist)

Why

To get closer to the actual final implementation and use the correct api's
To fix a bug

Testing done

tested the new feature
tested the bug fix

Decisions made

Removed some functionality that is now covered by the GenLayer JS library (nonces, useWallet hook...)

Checks

[x] I have tested this code
[x] I have reviewed my own PR
[x] I have created an issue for this PR
[x] I have set a descriptive PR title compliant with conventional commits

User facing release notes

Implemented the official GenLayer JS library to handle accounts and transactions in the Studio

@cristiam86 any hint on how to use leaderonly functionality with GenLayer JS?

@cristiam86 any hint on how to use leaderonly functionality with GenLayer JS?

sure, to use it in GenLayerJS, it should be implemented there :D

Regarding the leaderOnly feature, I've been trying to implement it in genlayer-js, without success until now.
If I dare ask, how important is that feature? Won't it have to be removed once we hook the studio to the real chain? @cristiam86

Regarding the leaderOnly feature, I've been trying to implement it in genlayer-js, without success until now.
If I dare ask, how important is that feature? Won't it have to be removed once we hook the studio to the real chain? @cristiam86

Let's take a look together whenever you can :) This is going to be available permanently in the localnet. Of course, it won't be available in the testnet or mainnet.
In Ethereum and other L1s, there are some methods that can be only used in the localnet and they make it easy to force situations or to speed up the development.

Note that if we want to support the leaderOnly param in this PR, we need to merge the feature in genlayer-js then update the dependency here. @cristiam86

I am now having some typing errors between the genvm merge and genlayer-js when fetching contract schema, because the structure we get from the backend doesn't match the type defined in genlayer-js:
  abi: Array<{
    inputs?: Array<{
      name: string;
      type: string;
    }>;
    name?: string;
    outputs?: Array<{
      name: string;
      type: string;
    }>;
    type: "constructor" | "function";
  }>;
  class: string;
};

we just merged https://github.com/yeagerai/genlayer-studio/pull/653 which changes this as it should be. Backend returns bytes and the frontend decodes it. Now we need to do the same in GenLayerJS

we just merged #653 which changes this as it should be. Backend returns bytes and the frontend decodes it. Now we need to do the same in GenLayerJS

From what I can see that is for eth_call, not for gen_getContractSchema / gen_getContractSchemaForCode

Oh, sure, sorry, my bad.
@kp2pml30 can we go ahead refactoring this in the frontend with the new schema?

Ok, then we simply need to update the ContractSchema type in genlayer-js to match the one in the studio.

Created an issue  theon genlayer-js side, will take care of it:
https://github.com/yeagerai/genlayer-js/issues/33

Great work, but we need to merge this one first: yeagerai/genlayer-js#35 and update the leader_only parameter here.

Went ahead and merged but turns out we forgot a couple - fixed here: #6 please review @cristiam86
As soon as it's released in 0.4.6 I'll bump the version here again!


Problem with all TA stats

All TA stats rely on code to convert date to day of water year. The current code to do this does not handle leep years, which results in an incorrect day number assigned for those years.

This was intentional in the original HITHAT because of hte Colwell matrix, which by definition does not use Feb 29th values on leap years.
Code has been changed in EflowStatslite back to remove Feb29th. However, values may still differ because original EflowStats took the mean flow value using the Feb29th Q.

Flow is OK to be calculated using feb29th data point, I think, but there are other issues with leap year discussed in #87


Decide on a reasonable package taxonomy.

Decide on a reasonable package taxonomy.
Example:
https://github.com/rust-lang/crates.io/blob/master/src/boot/categories.toml

meh.  Just use search.


Cloud Provider implementation for AWS

/kind feature

This issue was moved to kubernetes-sigs/cluster-api#59


Can saveAsPng be exported when it is exported?

Can saveAsPng be exported when it is exported?

Hey @sowdf.
Im not sure I understand your question.
When an image is exported it is saved to disk.
If you wish to get a file stream instead of a png file on disk. you can stream the pixelData inside each layer.
If you wish to export a saved image to something like AWS, you will need to consult the docs of the service you are using.
if you wish to export using a file blob to the database, please see #113 for more info on that.
Regards, Duck

Picture compression, export time

@sowdf
The images are already being compressed upon calling toPng().
If this is not compressed enough for you, you can always use a png  compression lib that suits your needs.
If you wish to have a faster export time, i would look into a PSD file parser written in something like C /C++ such as psdump.  I am reccomending this because PSD.js is already optimised to the point where it is only the language slowing it down.
I hope this helps you out.
Regards, Duck.


[Tracking] Falco 0.38.2

Falco 0.38.2 Release
Will keep this issue updated with the current status and progress.
This release contains several fixes developed in the last ~2 months, most importantly:

Fixing a crash when using transformers with an empty string parameter
FIxing some network comparison in rules not behaving correctly
Fixing experimental Prometheus metrics names
Fixing a kernel module crash in some kernel versions in RHEL9

Date
Target release date: August 19th, 2024
Release blocking PRs
### Release Blocking PRs
- [ ] https://github.com/falcosecurity/libs/pull/1992

Release Steps
The process is described in this document.
## External dependencies
- [x] falcoctl -> v0.9.0 https://github.com/falcosecurity/falcoctl/releases/tag/v0.9.0
- [x] libs -> 0.17.3 https://github.com/falcosecurity/falco/pull/3294
- [x] drivers ->  7.2.1: https://github.com/falcosecurity/libs/releases/tag/7.2.1%2Bdriver
- [x] rules -> NO CHANGES
- [x] plugins bump -> NO CHANGES

Action Items

[x] Pre-Release

[x]  Milestone
[x] Code freeze in falcosecurity/falco
[x] Open release branch in Falco and protect it
[x] See External dependencies task lists
[x] Prebuilt drivers publish
[x] Code thaw in falcosecurity/falco
[x] Changelog on the release branch https://github.com/falcosecurity/falco/pull/3293
[x] cherry-pick the changelog on master https://github.com/falcosecurity/falco/pull/3300

[x] Release

[x] Github Release in the Falco repo

[x] Website

[x] Create new snapshot

[x] Protect snapshot branch

[x] Merge the release blog post https://github.com/falcosecurity/falco-website/pull/1350
[x] Merge all necessary documentation PRs https://github.com/falcosecurity/falco-website/pull/1349

[x] Helm

[x] Release a new helm chart version https://github.com/falcosecurity/charts/pull/726
[x] Update helm chart documentation

[x] Announcements

[x] mailing list
[x] blog post
[x] slack channel

[x] Post-Release

[x] Close the resolved issues after the tag and move the unresolved ones under a new milestone
[x] Update supported fields and syscall documentation
[x] add Falco version to the rules CI https://github.com/falcosecurity/rules/pull/262

Miscs
/milestone 0.38.2
/kind documentation
PRs that should be cherry-picked into the release branch

Synced with:

This release is now complete.


Add PostgreSQL RETURNING keyword

Add the PostgreSQL RETURNING keyword to the sql syntax.
Right now, it does not get highlighted:

Potentially related to: #2137 ("SQL Dialects")

Thank you for this PR @karlhorky!
Please also add the keyword to SQL's keyword test.

Done!

You also have to update the below JSON. More info.

Oops, sorry, done.

Cool, thanks for the merge!

Thank you for contributing!

Glad to help! May have more keywords incoming :)

Looking forward to it. Many of our languages aren't really up to date, so this really helps.


added the abaility to get the current window gaps.

This exposes the gaps information in i3-gaps in i3ipc if it is available. After writing this I noticed that i3 is only exposiing the deviation from the value in the config file so I am not sure how usful it is to anyone.

:+1:


Ignore collectionFormat when parsing body. Fixes #61.

Fixes #61.

Current coverage is 81.62% (diff: 100%)

Merging #62 into master will increase coverage by 0.04%

@@             master        #62   diff @@
==========================================
  Files            11         11
  Lines           885        887     +2
  Methods           0          0
  Messages          0          0
  Branches         21         21
==========================================
+ Hits            722        724     +2
  Misses          142        142
  Partials         21         21

Powered by Codecov. Last update d97622a...91a4346

👍

@cmpitg would you mind adding some tests to increase code coverage?

Yessir. I have updated the code. Please review.

👍

👍


Empty workspace information

Many apps like text editors show some information when there is nothing open. We should do the same if the workspace is empty. We could show a message like "Click on a generator to select it. To create a new 0-dimensional generator, press the + button".
This would be hugely useful if a user opens homotopy.io for the first time and have no idea where to start.

This could also include default keybinding information to help new users. (note from standup 21/07/22)

Yes! I am thinking something similar to VS code:

Upvote!


"E129: Function name required" when setting pastetoggle

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Announcement

[X] I have checked Breaking change announcement.

Minimal reproducible full config
I kept the template mostly unchanged, only adding the two lines at the bottom.
if has('vim_starting')
  set encoding=utf-8
endif
scriptencoding utf-8

if &compatible
  set nocompatible
endif

let s:plug_dir = expand('/tmp/plugged/vim-plug')
if !filereadable(s:plug_dir .. '/plug.vim')
  execute printf('!curl -fLo %s/autoload/plug.vim --create-dirs https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim', s:plug_dir)
end

execute 'set runtimepath+=' . s:plug_dir
call plug#begin(s:plug_dir)
Plug 'hrsh7th/nvim-cmp'
Plug 'hrsh7th/cmp-buffer'
Plug 'hrsh7th/cmp-nvim-lsp'
Plug 'hrsh7th/vim-vsnip'
Plug 'neovim/nvim-lspconfig'
call plug#end()
PlugInstall | quit

" Setup global configuration. More on configuration below.
lua << EOF
local cmp = require "cmp"
cmp.setup {
  snippet = {
    expand = function(args)
      vim.fn["vsnip#anonymous"](args.body)
    end,
  },

  mapping = {
    ['<CR>'] = cmp.mapping.confirm({ select = true })
  },

  sources = {
    { name = "nvim_lsp" },
    { name = "buffer" },
  },
}
EOF

lua << EOF
local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())

require'lspconfig'.cssls.setup {
  capabilities = capabilities,
}
EOF

let mapleader = "\<space>"
set pastetoggle=<leader>v

Description
Trying to use auto-completion (entering some text and then trying to accept a suggestion or navigate the popup menu) results in the following error message at the bottom of the screen:
E129: Function name required

Running in Verbose mode to help track this down, the last line printed before the error occurs is:
Executing: call:lua.cmp.utils.keymap.set_map(1)

I'm wondering if this is related to the following mapping established in nvim-cmp/lua/cmp/utils/keymap.lua, which is the only place I see the path reported in the Verbose logging:
i  <C-P>       * <Cmd>call v:lua.cmp.utils.keymap.set_map(1)<CR>

Steps to reproduce

Run nvim -u cmp-repro.vim cmp-repro.vim
Press 'G' to move to the last line where pastetoggle is set
Press 'o' to start editing under that line
Enter 'past' so that the autocomplete menu shows up
Press Enter to accept a completion

Expected behavior
That can select text to be entered into my document and that I can press the "accept" key sequence to enter it into my document.
Actual behavior
Interacting with the popup menu results in the error message I described.
Additional context
Commenting out the "set pastetoggle" line in cmp-repro.vim causes the error to not occur and for successful operation of auto-complete.
Various flags are changed after this error. For instance, paste mode is enabled, tab settings are set to their defaults, etc.

I can reproduce this. But I think set pastetoggle=<Leader>v is not valid option command. What do you think about this?

I think I see what you're saying. "" is for key mappings, so why would it be allowed when setting 'pastetoggle'? I think the documentation for 'pastetoggle' hints that it should work:
	When non-empty, specifies the key sequence that toggles the 'paste'
	option.  This is like specifying a mapping:
	    :map {keys} :set invpaste<CR>
	Where {keys} is the value of 'pastetoggle'.

So if setting 'pastetoggle' is equivalent to invoking ':map', then one could reasonably infer that "" would be supported in both places.
I'd used that 'pastetoggle' sequence for ages with no problems, so it seems supported. And I didn't come up with it myself; I copied it from somewhere. This means that other people are also using it. A Google search turns up other people recommending using "" this way. So even if it's not officially supported, it's probably "de facto" supported.
This is a goofy bug, whether it's in nvim-cmp or in neovim. It caused me to give up on trying to use nvim-cmp two or three times before I decided to really concentrate on tracking it down. But maybe other people who have run across this issue just gave up and moved on to some other 'completion' tool after getting frustrated. It's a lousy experience and it would be great to do something so that future people could avoid it.
For my own future experience, I'm OK if this doesn't get fixed. I tracked down the issue and I've undefined 'pastetoggle'; I can live without it. But I posted the bug in case it was something you could address for other people hitting the same issue.


Break into files

This is a no-op change that just breaks the large client/src/lib.rs and core/src/lib.rs files out into smaller, more focused separate files


Add support for mlflow 2.0

Description
close #390
Development notes

update requirements to mlflow>=1.0.0, <3.0.0

fix all tests by replacing:

run.run_id -> run.info.run_id
MlflowClient.list_experiments -> MlflowClient.search_experiments
MlflowClient.list_run_infos -> MlflowClient.search_runs

Checklist

[X] Read the contributing guidelines
[X] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Update the documentation to reflect the code changes
[ ] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[ ] Add tests to cover your changes

Notice

[ ] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (d3955f0) compared to base (5f93f5d).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##            master      #398   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           28        28
  Lines          956       956
=========================================
  Hits           956       956

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Meta: Request Reviews from maintainers

Description of the issue
Regular contributors (like me) don't have the ability to use Github's "Request Review" feature to ask a maintainer to look at a PR. They always have to tag a specific person in the request, like @haarchri or @muvaf. This slows down the review process because the addressed person might be unavailable at that time. It is also difficult for new contributors because they most likely do not know the maintainers and which to address.
How could this issue be solved
Add the ability to request a review from the provider-aws-maintainers team (or whatever you would like to call it) from the PR menu.
This might be interesting for other repos in crossplane-contrib as well although I am not sure if its a good idea to interfere with their development process from the outside.

CODEOWNERS is also an option. For example we could be pinged any time someone makes a PR touching RDS, ElastiCache and SecurityGroups, and then maybe someone else could handle other controllers.

@jbw976 fyi


Integration with Whisper-compatible STT endpoints

Inference providers like Groq are now supporting Whisper STT. we would like to ensure our OpenAI STT module could be used with Groq.
It'd be great to have STT.with_groq(...) constructors

I dont know about groq but i have made a whisper api in local and made it connect with the new plugin i made for whisper itself

nice, is that something you'd be open to sharing? :)

i have run whisper code its a pretty much a small api type and then I have made a code which can connect and get transcript from it just like deepgram api inference.  I have made it just yesterday because I am need for local models and i am also on the way to make a local TTS as well instead of the pricy ones online.


Handle Client with empty base prefix

I was playing around with the client to get a feel for it and noticed that if you didn't specify a base prefix for your Statter the stats would arrive with an unexpected format.  Specifically:
  client, err := statsd.NewClient("localhost:8125", "")
  // stuff
  db := client.NewSubStatter("db")
  reads := db.NewSubStatter("reads")
  writes := db.NewSubStatter("writes")

  reads.Inc("cluster."+userid, 1, 1.0)
  writes.Inc("cluster."+userid, 2, 1.0)

Produces:
.db.reads.cluster.aGVsbG8sIHdvcmxk:1|c
.db.writes.cluster.aGVsbG8sIHdvcmxk:2|c

Compared to statsd.NewClient("localhost:8125", "api") which yields:
api.db.reads.cluster.aGVsbG8sIHdvcmxk:1|c
api.db.writes.cluster.aGVsbG8sIHdvcmxk:2|c

I added a test for this case and output before/after my change:
$ go test github.com/falun/go-statsd-client/statsd/...
--- FAIL: TestSubStatterClient (0.00s)
	client_substatter_test.go:74: Inc got '.sub.count:1|c' expected 'sub.count:1|c'
--- FAIL: TestMultSubStatterClient (0.00s)
	client_substatter_test.go:125: Inc got '.sub1.count:1|c' expected 'sub.count:1|c'
--- FAIL: TestSubSubStatterClient (0.00s)
	client_substatter_test.go:176: Inc got '.sub.sub2.count:1|c' expected 'sub.count:1|c'
FAIL
FAIL	github.com/falun/go-statsd-client/statsd	0.470s

$ go test github.com/falun/go-statsd-client/statsd/...
ok  	github.com/falun/go-statsd-client/statsd	0.474s

Thanks for the pull req!
Your example seems like a valid usecase (empty prefix but populated subclient suffixes).
Just a bit of minor feedback on the current pull req.

Awesome. Thanks!
Did a rebase/squash and merged to master manually: https://github.com/cactus/go-statsd-client/commit/88674d16c225ad498046ff81950683b2f5931c7f
Closing as accepted.


STORM-2087 1.x

Backport STORM-2087 to 1.x
See https://github.com/apache/storm/pull/1679 for master version

Thanks for picking this up @srdo!

+1 since follow-up issue and PR are created so good to merge as it is.


⚠ ClamAV has degraded performance

In eeb29b5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: ClamAV performance has improved in 2b5d810 after 11 minutes.


back button requires me to fill field data

Setup wizard has a back button that requires form to be filled before you can step back, which isn't too rational. And seems there is a quick fix for that, which...... I am too lazy to put myself (git work...), so here it is: "formnovalidate" is a keyword that needs to be added to _wizard_actions.html like so:
{% if wizard.steps.prev %}
  <button name="wizard_goto_step" type="submit"
          value="{{ wizard.steps.prev }}" formnovalidate
          class="btn btn-secondary">{% trans "Back" %}</button>
{% else %}

YOU WILL THANK ME LATER! or not
Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Browser and version:
Python version:
Django version:
django-otp version:
django-two-factor-auth version:
Link to your project:

Thanks for the report, but this should be fixed in master by https://github.com/jazzband/django-two-factor-auth/commit/b86a790b63bd679


[GH 12439] Convert components/settings_sidebar to TypeScript

Summary
Convert components/settings_sidebar to TypeScript
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/12439

@wyze would you please merge master into your branch?

@hanzei Done!


⚠ Bofrost.lu has degraded performance

In e93bf67, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2711 ms

Resolved: Bofrost.lu performance has improved in 541d2e0 after 21 minutes.


docker wrong vesion

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

git pull openclimatefix/nowcasting_forecast
docker run openclimatefix/nowcasting_forecast:0.0.56 more nowcasting_forecast/init.py

::::::::::::::
nowcasting_forecast/__init__.py
::::::::::::::
""" Init file """
__version__ = "0.0.55"

N_GSP = 338

Expected behavior
0.0.56

The release happens in this work flow
https://github.com/openclimatefix/nowcasting_forecast/runs/5197132792?check_suite_focus=true

this helps - https://github.com/actions/checkout/issues/439

fixed with https://github.com/openclimatefix/.github/commit/7da203356422b3556602b8aee728becce380cefd


style: the card component should have transition on shadow when hover

Pull Request Test Coverage Report for Build 485

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.284%

Totals

Change from base Build 474:
0.0%

Covered Lines:
1243

Relevant Lines:
1539

💛  - Coveralls


feat(MeasureTheory/Measure/WithDensity): relaxed assumptions

Relaxed Measurable assumption to AEMeasurable of
withDensity_apply_eq_zero. Added multiple generalizations of lemma.

Relaxed finite assumption of withDensity_absolutelyContinuous' and modified several files consequently.

Added withDensity_ae_eqthat shows that, given an a.e. nonnegative function d, two functions are a.e. equal w.r.t. μ iff they are equal w.r.t. μ.withDensity d.

(See #13285)

I don't think you need to break up the pr but I think you can just restore the previous pr. It is more helpful since then people can more easily see the review history

Unfortunately, I deleted the branch of the previous PR. I splitted this PR into two. I will refer to the previous PR in the second one for people to easely see the review history.

Thanks!
bors r+


Set default pathname to '/' instead of erroring out

I was trying this library out and discovered that if pathname was undefined, it would error out with TypeError: Cannot read property 'length' of undefined on https://github.com/badunk/node-proxy-middleware/blob/master/index.js#L117 coming from https://github.com/badunk/node-proxy-middleware/blob/master/index.js#L37
Just setting the default pathname to '/' if one isn't set from this PR.

Thanks. Published as 0.12.0


When logging in using a non existing account, download links for the app are missing.

Repro steps

On the identity yourself section, use a new name
Attempt to click below as the text suggests

Expected Result
There should be links to the app.
Actual Result
There is nothing below

System Info
Type: Staging
Git hash: caa24a9
Time: 2020.01.07-16.53.17
Size: 24888734
MD5: bb1e1f6cb79309e795117579669fc323

links are back


Is this exclusively for imgur?

I saw all examples use imgur, can I use these scripts to download from other sites or is it only for image downloads from imgur?

You can download files from everywhere. I'll update examples.
$ dsdownload https://github.com/DiSiqueira/DSDownload/archive/<IP_ADDRESS>.zip

Thanks for the feedback

Many thanks! This project suddenly became really interesting to me :)


Default config domains no longer exists, which creates an error

The default domains in the config from running wraith setup point to links that no longer exist:
# Add only 2 domains, key will act as a label
domains:
  arabic: "http://www.live.bbc.co.uk/arabic"
  russian: "http://www.live.bbc.co.uk/russian"

Replacing the links with anything else works perfectly well, although I can’t think of a good example of an Arabic and Russian BBC site with very similar designs to use instead.
I’ve tried to find whatever links they should have pointed to, but I’ve had no luck, which is why I’m filing this as an issue instead of a pull request. The sites also block crawlers, so archive.org is no use. (Great work all around by the Beeb.)

My guess is that www.live.bbc.co.uk is the URL in England, but in the US the URL redirects to bbc.com.
Is it possible to use URLs that will resolve properly in any country?

Thanks for raising the issue. I've now refactored the example configs: https://github.com/BBC-News/wraith/commit/5f16645748e6c272c82e5500509cb79c7ba7d752

Awesome!


Make string to sign visible to customers

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Storage.Blobs


[DF-437] implement a Postmark supprt

Implement support for https://resend.com/ as an email service provider.
Add a new PersistedEmailProvider type:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/isomorphic-lib/src/types.ts#L991-L994
Add new EmailServiceProviderSuccess type:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/isomorphic-lib/src/types.ts#L1946-L1950C4
Add new EmailServiceProviderFailure type:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/isomorphic-lib/src/types.ts#L2080-L2083
Implement support for https://postmarkapp.com/ as an email service provider.
Add new EmailProviderSecret type:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/isomorphic-lib/src/types.ts#L2080-L2083A webhook for
Error handling for the /templates/test api endpoint:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/api/src/controllers/contentController.ts#L174-L178
A webhook for updating the status of sent emails:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/api/src/controllers/webhooksController.ts#L23-L24
Handle the email provider type when sending messages:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/backend-lib/src/messageTemplates.ts#L479-L481
Create the Postmark email provider:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/dashboard/src/lib/email.ts#L86-L100
Add Place in Settings to configure email provider:
https://github.com/dittofeed/dittofeed/blob/01a6db73e237897b7e961c07969ea5d60a8bcbd0/packages/dashboard/src/pages/settings.page.tsx#L847-L854
From SyncLinear.com | DF-437

/bounty $100

/attempt #641

/attempt #641

@GauravBurande assigning this to @neo773 for now because we had a prior conversation in chat about him working on this

ok.


Edit the image in their original size

Hello, and congratulates for the component is very helpful.
I need use the image in its real size. Is possible expose the image Style.  I saw that the class is configured at Annotation Class with 100% Width. I didn`t find a way to change it.
Thanks,

Hello, and sorry for the late reply!
Unfortunately, I cannot expose the styles of the internal image component because changing it from 100% width will cause the overlay and intersection methods to be mismatched, for example:

The recommended way to set the width/height style of the component is to wrap it in the desired container - so set the width/height of the parent that contains react-image-annotation.
To support the real dimensions of the image, you can programatically get the dimensions of the image, then set that as the dimensions of the container.
Let me know if you have any questions or if you think there is a better way to handle this!

Could you provide some sample codes to set the container? Tried a lot of combinations of css, didn't work...
If I wrap it with a div, how should I set the property of the div?
It would be great to expose a property allowing to switch to 100% heights. This will cover the two most common cases

I was able to get it work by calculating the div's height and width using window size and image size. But it would be more efficient to allow a switch


Scheduling and managing backups

How can I schedule a regular NFS backup and is it possible to maintain a rolling backup of last x backups according to the backup schedule defined?

@paulmorabito It's exactly Recurring Backup designed for. Check for the Volume Detail page, scroll down and you will see it.

wow this has eluded me for days.
Clicking the scheduled backups icon should bring you to this hidden section of the details

@ainiml yeah, that's a good idea. Can you file another issue for it?


Prevent inserting illegal characters into a link

Is there an existing issue for this?

[X] I have searched the existing issues

Flutter Quill version
10.8.4
Steps to reproduce
Focus the quill editor
Type https://google.com/
Type , or )
Expected results
The char ","  or ") "should not be part of the link.
Actual results
The char ","  or ") " is part of the link and the link does not work
Additional Context
No response

, and ) (and *, (, !...) are all valid characters as soon as they don't appear in the host part.
To me quill editor works as expected when it comes to the handling of these characters:

Type https://google.com (without the end slash) and then type , or ) : the last char won't be recognized as part of the link because it wouldn't make for a valid host in the link.
Type https://google.com/ (this time with the end slash) and then type , or ) : the last char will be recognized as part of the link because it is a valid path character.

Take this Google Maps position URL as an example :
https://www.google.fr/maps/place/Paris/@48.8588475,2.3058358,13z/data=!3m1!4b1!4m6!3m5!1s0x47e66e1f06e2b70f:0x40b82c3688c9460!8m2!3d48.8575475!4d2.3513765!16zL20vMDVxdGo?entry=ttu&g_ep=EgoyMDI0MTAyMi4wIKXMDSoASAFQAw%3D%3D
As you can see, they are using @, ! or even , as part of the path, and IMO we should definitely be able to support that in Quill Editor

Thanks for the reply and the choice is understandable.
However, there remains a problem which in my opinion is a bug:
if I write https://www.google.it, the ',' is part of the (correct) link,
but if I delete the ',' it continues to be part of the (wrong) link, same thing with other characters like '.' ')' etc

I'm working on a way to prevent events using the QuillController and a new parameter for QuillEditorConfigurations.


retry enable workload when conflict

fix rollout flaky test. https://github.com/oam-dev/kubevela/pull/1910/checks?check_run_id=3063375311
the root cause is handleRolloutSucceed func update workload meet a conflict error. but next time the rollout status has been succeed and will not do try to update again.

why will the conflict happen?

I doubt the  cloneset controller updated the workload status

@wangyikewxgm 要研究清楚哈，retry 会掩盖问题，从而引起更严重的问题

this fix maybe not work.closing


optimization debian package manager tweaks

By default, Ubuntu or Debian based "apt" or "apt-get" system installs recommended but not suggested packages .
By passing "--no-install-recommends" option, the user lets apt-get know not to consider recommended packages as a dependency to install.
This results in smaller downloads and installation of packages .
Refer to blog at Ubuntu Blog .
Signed-off-by: Pratik Raj<EMAIL_ADDRESS>

@fud @akyrtzi @lgaches  requesting review

\cc @shahmishal

@swift-ci test

@swift-ci test

hi @Rajpratik71, would you like to work with us to bring this over the finish line?

hi @Rajpratik71, would you like to work with us to bring this over the finish line?

Hi , rebased to upstream and also done changes as requested by @ianpartridge

@swift-ci test

thanks so much @Rajpratik71, can you remind me why we needed to add the apt-utils and ca-certificates packages as part of this PR.
additionally, we recently added support for Ubuntu 20.04 (5.2 and nightlies) which I think can use the same optimization?

thanks so much @Rajpratik71, can you remind me why we needed to add the apt-utils and ca-certificates packages as part of this PR?
additionally, we recently added support for Ubuntu 20.04 (5.2 and nightlies) which I think can use the same optimization?

added packages apt-utils ca-certificates
Because build is

Slow because "apt-utils" not installed

to avoid build to exits with error without having certificate in wget , curl or even git clone

hi @Rajpratik71

for my education, how did it work before this change? were they part of recommend packages?

could you please apply the same logic/patch to the recently added 20.04 images

thanks a ton for your contribution!

hi @Rajpratik71

for my education, how did it work before this change? were they part of recommend packages?
could you please apply the same logic/patch to the recently added 20.04 images

thanks a ton for your contribution!

Hi @tomerd ,
These packages are not there before . e.g If any debian based system doesn't have "apt-utils" installed , then in that case any package installation takes time while configuring and gives suggestion to install "apt-utils" . that is why i put that here  .
And I put "ca-certificates" to avoid build to exits with error without having certificate in wget , curl or even git clone

@swift-ci Please test

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create the pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412

created fresh pull to main @ #203


[FEATURE] Add wash evac command

Is your feature request related to a problem? Please describe.
Right now, wash does not have a way to drain all actors and/or providers from a given host.
Being able to clear out a given host of all workloads is useful for administrators of wasmCloud lattices and hosts who would like to retire a host or other pieces of infrastructure.
Describe the solution you'd like
Add a command wash ctl evac (wash evac is also a reasonable alias) that allows users to command a given host to purge all actors and providers that it is running.
Evacuated actors/providers should be re-auctioned to other hosts on the lattice.
In the course of implementing wash evac, you will also need to implement a form of "cordon"-ing (similar to Kubernetes hosts), to ensure that the node being evacuated does not respond to auctions for the workloads that are being moved.

Note
A great first start for this is an RFC describing more about how you'd like to implement,
and gathering feedback from team members.

Describe alternatives you've considered
N/A
Additional context
Add any other context or screenshots about the feature request here.

I think this issue for the host captures the desired cordoning behavior, which will need to be implemented before wash can issue a command 👍


soundfile.LibsndfileError: Error opening 'xx/xx/xx.wav': File contains data in an unknown format.

I find a problem when training my model:
soundfile.LibsndfileError: Error opening 'xx/xx/xx.wav': File contains data in an unknown format.
I run this in Ubuntu
I have tried many methods. What can I do about this problem?

The problem is the same

ffmpeg is all you need, i meet the same question in other projects, you can try conda install ffmpeg.

The problem is the same.

ffmpeg cannot solve my problem (fyi, I'm trying to asr a mp4)

I'm very sorry, but it turned out that the audio files in the dataset were corrupted, which caused this error. The dataset I used is https://github.com/microsoft/DNS-Challenge/tree/interspeech2020/master. I switched to other data sets and there was no error, thank you for your help.

when you used the DNS-Challenge dataset, did you use the shell file 'download-dns-challenge-4.sh ' to download the data?

when you used the DNS-Challenge dataset, did you use the shell file 'download-dns-challenge-4.sh ' to download the data?

No, I use the DCN 2020 dataset.


Allow to override StepAdapters

Could we change the StepAdapters container registry configuration to allow for overriding instead of the default behavior of just raising an issue if trying to define a step_adapter with the same name.
I had to modify the Try step adapter, and in order to do so I end up having to rewrite all basic step adapters inside my CustomStepAdapters module.
What do you think @timriley?

@mrbongiolo how do you mean rewrite? Do you actually just mean this?
class MyAdapters
  extend Dry::Container::Mixin

  register :map, Dry::Transaction::StepAdapters::Map.new
  register :tee, Dry::Transaction::StepAdapters::Tee.new
  register :try, MyTry.new
  register :step, Dry::Transaction::StepAdapters::Raw.new
end

Or something else?


JavaDoc of BuildFeature claims you can inject it but not how

Issue type
Wrong or misleading information
Problem description
The JavaDoc of BuildFeature says you can inject an instance of it.
But it does not suggest how, and I doubt it is true, having a look at the implementation.
How wold you inject an instance of BuildFeature to check the state of configuration cache?
It is just a DefaultBuildFeature just like the one for project isolation.
My guess is, that this sentence should be on BuildFeatures.
Given that
interface BuildFeaturesProvider {
    @get:Inject
    val buildFeatures: BuildFeatures
}
println(objects.newInstance<BuildFeaturesProvider>().buildFeatures.configurationCache.active.get())

works as expected, supports this theory.
Context (optional)
No response
Page with the problem
https://docs.gradle.org/8.5/javadoc/org/gradle/api/configuration/BuildFeatures.html

Thank you for your interest in Gradle!
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

@alllex Is this just misplaced Javadoc or something else?

Thanks for reporting it!
The javadoc should have indeed been on the BuildFeatures


Fixed problem with a mock resolving a mocked value with Promises

Upstream PR: NagRock#194

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6637048). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   93.46%
=========================================
  Files             ?       34
  Lines             ?      658
  Branches          ?       82
=========================================
  Hits              ?      615
  Misses            ?       30
  Partials          ?       13

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6637048...dc19b3a. Read the comment docs.


Update Peer Dependency

react-router now at 5.0.0. Can the peer-dependencies in package.json be updated?
"peerDependencies": {
"mobx": "^3.0.0 || ^4.0.0 || ^5.0.0",
"react-router": "4.x" <-----
}

Ah yes good catch. Not using this library at the moment at work so relying on people like you to spot these kind of things.

Will be fixed with #64

Should be done now. Let me know if there's anything else. Kind of rushed a few other upgrades in there too


Files downloaded have no extension

Hello,
I've just installed manga-dlp in a docker instance and started downloading some manga.
The download is working but files are saved without extension.
When the program creates the CBZ file the readers I've tried cannot open it, probably because of this reason. (the files are generated correctly as I can unzip them without any issue).
I've noticed that the files in the mangadex server do not have any extension, not sure if this is something recent or not.
I've also tried to generate PDF but the extension is not included in the image so I cannot see if PDF would work anyway.
Docker file
version: "3"
services:
  manga-dlp:
    image: olofvndrhr/manga-dlp:latest
    container_name: manga-dlp
    security_opt:
      - no-new-privileges:true
    networks:
      - proxy
    volumes:
      - ************/Comics:/app/downloads/ # default manga download directory
      - ************/manga-dlp/config/mangas.txt:/app/mangas.txt # default file for manga links to download
      #- ./crontab:/etc/cron.d/mangadlp # path to default crontab
      #- ./schedule:/app/schedules/daily # path to the default schedule which is run daily
    environment:
      - TZ=Europe/************
      - PUID=1000 # custom userid - defaults to 4444
      - PGID=1000 # custom groupid - defaults to 4444
    restart: unless-stopped

networks:
  proxy:
    external: true

Mangas.txt
https://mangadex.org/title/f7888782-0727-49b0-95ec-a3530c70f83b/hajime-no-ippo
[...]

Hello, and thanks for your report.
As i use the reader okular i didn't have any issues with no file extension on the images.
But of course this is not good, as you can't really do anything with these images otherwise.
With the new release 2.1.5 it is fixed. It will add the correct image suffix, and if none is found it will use ".png".
I hope now everything works. I will not close the issue yet, that you can test everything.
Best regards

@kn-f
No problem.
And just a note: The pdf extension is already installed if you use the amd64 (x86_64) docker image.
Best regards


Consider implementing '(expr)' string interpolation like in Swift

Example:
SELECT User {
   friends_msg := '\(User.name) has \(count(User.friends)) friends'
}

Prior art:

Python f-strings
Swift interpolated strings

We still reject strings containing \( so we could do the original syntax or the f-string version.
I'm partial to f-strings but only because I'm very used to them.

We still reject strings containing ( so we could do the original syntax or the f-string version.

We should do \( because it's a drop in syntax without requiring a new string literal type. This is what Guido actually wanted to do with f-strings (just make all strings f-strings) but couldn't because of the obvious reasons.


Automatically broadcast sparse HEALPix arrays

Do we really want this? I would have expected that we'd basically only replace .isel by .sel, not to expand the array to full size...

Do we really want this? I would have expected that we'd basically only replace .isel by .sel, not to expand the array to full size...

I hoped so, too. However, the .isel() approach doesn't work if the plotting area is larger than what is covered by the data (e.g. global map with limited area model). In those cases, the resample function tries to isel indices that are not present which crashes.
I agree that we would like to prevent an automated broadcast in cases of very fine local grids, but so far I haven't found a straight-forward workaround


Sprite 3D texture rendering is broken frame 1

Tested versions

Reproductible in v4.3.dev5.official [89f70e98d]

System information
Windows 10 - v4.3.dev5.official [89f70e98d] using Mobile or Forward+ AMD RX 5600 XT
Issue description
When using a Sprite3D and instancing a Card (my scene) containing a Sprite3D inside a Viewport with a camera, rendered in 2D using a Sprite2D with a ViewportTexture, when instancing the Card during the first frame the texture will be rendered incorectly

Using the Compatibility renderer fix the issue
Steps to reproduce
i can't figure out how to reproduce the error, but here is the node tree

Minimal reproduction project (MRP)
i was unable to reproduce the issue on an other project so here's mine, the card is saved inside "card.tscn"
bug_report.zip

Here is a smaller project with the same problem:
viewport-glitch.zip
I created an object with a Sprite2D and a SubViewport, using the viewport as the sprite's texture.

This object is rapidly instantiated and deleted to show the first frame.

It seems like the problem relates to how the viewport's texture is not rendered before the sprite renders, so the sprite is rendering some uninitialized texture memory from the GPU.
A temporary fix for your project is to set the viewport's Update Mode to render when the parent is visible, so it is initialized when the sprite asks for the texture:

I tried to set Update mode to When parent visible but it did not work. I tried it with SubViewport being a child and a sibling of the Sprite3D.


Enhance no-http-verbs-in-paths with Exclude Paths option

What/Why/How?
Solution:
Implemented a new feature to allow listing excluded paths directly in the rule configuration. This enhancement provides a more streamlined approach to handling excluded paths, improving the overall usability and effectiveness of the rule.
Changes Made:
Added support for the "excludedPaths" option in the "no-http-verbs-in-paths" rule configuration.
Updated documentation and examples to reflect the new functionality.
Benefits:
Reduces false positives by excluding specific paths from rule evaluation.
Improves developer experience by providing a built-in mechanism for managing excluded paths.
Aligns with best practices for rule configuration and customization.
Reference
https://github.com/Redocly/redocly-cli/issues/1501
Testing
Screenshots (optional)
Check yourself

[ ] Code changed? - Tested with redoc/reference-docs/workflows (internal)
[ ] All new/updated code is covered with tests
[ ] New package installed? - Tested in different environments (browser/node)

Security

[ ] Security impact of change has been considered
[ ] Code follows company security practices and guidelines

Hi @Ayush9026, did you forget to push the actual changes?

@tatomyr sir i pushed actual changes but i will check it again. Thanks for pointing that out!

@Ayush9026 do you mean that this PR only contains the docs changes?

Yes @tatomyr sir this PR only contain doc changes but i think i have to make changes in codebase too.

@Ayush9026 yes, you have to make the corresponding code changes in order to get this PR approved. Please notice that every issue that isn't marked with documentation or Type: Docs labels is supposed to contain some code changes 🙂. If you would like to contribute to the documentation instead, please choose tickets with the corresponding labels.
Please let me know if I should wait for the code changes or if I can close the issue.

@tatomyr sir , got it! I'll proceed with the necessary code changes to address the issue. Thanks for the clarification and guidance!

Since we don't have the change in place yet, I'll close this pull request. Please do feel free to re-open it when you have something for us to review.


Add an option to additionally verify the offline Rekor entry in the bundle with an online inclusion proof

Fine to close.


ViewController traces are being categorized as unknown

View Controller traces are being categorized as unknown in kibana.

should be solved in #72


PERF: Improve perf of to_datetime with ISO format

Addresses #10178
Using the example data from the issue:
In [2]: df = DataFrame({'date_text':["2015-05-18" for i in range(10**6)]})

Before:
In [3]: %timeit pd.to_datetime(df['date_text'],format="%Y-%m-%d", box=False).values.view('i8')/10**9
1 loops, best of 3: 3.14 s per loop

In [4]: %timeit pd.to_datetime(df['date_text'],infer_datetime_format=True,     box=False).values.view('i8')/10**9
1 loops, best of 3: 253 ms per lo

After:
In [6]: %timeit pd.to_datetime(df['date_text'],format="%Y-%m-%d", box=False).values.view('i8')/10**9
1 loops, best of 3: 217 ms per loop

In [7]: %timeit pd.to_datetime(df['date_text'],infer_datetime_format=True, box=False).values.view('i8')/10**9
1 loops, best of 3: 243 ms per loop

is their an existing vbench for this? if not pls add one (but make 1/10 the length)

I added a vbench based on existing one that inferred the iso860 format.
---------------------------------------------------------
Test name                                    |    #0    |
---------------------------------------------------------
timeseries_to_datetime_iso8601_format        |   4.9593 |
timeseries_to_datetime_YYYYMMDD              |  10.0046 |
timeseries_to_datetime_iso8601               |   4.9581 |
---------------------------------------------------------
Test name                                    |    #0    |
---------------------------------------------------------

The second line of this test is failing - is that really the desired behavior?  The comments reference #10154  - Wasn't the conclusion that errors='ignore' should always suppress?
def test_day_not_in_month_coerce_false_ignore(self):
    self.assertEqual(to_datetime('2015-02-29', errors='ignore', coerce=False), '2015-02-29')
    self.assertRaises(ValueError, to_datetime, '2015-02-29', errors='ignore', format="%Y-%m-%d", coerce=False)
    self.assertRaises(ValueError, to_datetime, '2015-02-32', errors='ignore', format="%Y-%m-%d", coerce=False)
    self.assertRaises(ValueError, to_datetime, '2015-04-31', errors='ignore', format="%Y-%m-%d", coerce=False)

The intention of that test behavior is that yes you will ignore errors (and you are not coercing), BUT, since you specified the format it is still an invalid date.
so you bring up the case that it is doing different things depending on whether a format is passed or not.
rereading those comments in #10154. I think it is just a choice. Ok, so let's make them the same. I'll reverse myself and now say that all 4 of these should raise. @chris-b1 would you make that change.
cc @jorisvandenbossche
cc @vincentdavis

%Y format with integer data doesn't work on iso fast code (causing Stata test problems) so I've excluded from this and added a new test.
I also changed the error handling in to_datetime to always raise on out of bound / unparse-able date unless coerce=True.  It ends up breaking quite a tests that expected the old behavior, just wanted to double check that's desired?  Thanks.

Rereading the comments in the other issue: https://github.com/pydata/pandas/issues/10154#issuecomment-106484971 and the two comments below. Based on that, I would say we agreed that with errors='ignore', no error should be raised (although it is not really clear what we decided). But I did not see that the actual PR then did something else.
I think it is pretty clear: if you give errors='ignore', it should not raise an error. But as I said over there, I don't really like this default behaviour.

I think that makes sense.   One corner case that comes up - what should happen if the data passed doesn't match the format, but can all be parsed.  E.g.
pd.to_datetime(['2/28/2014', '2014-01-01'], format='%m/%d/%Y')
pd.to_datetime(['2/28/2014', '2014-01-01'], format='%m-%d-%Y')

I'm not sure, but it's probably least surprising if it fails? That's what the latest commit does - adding an additional parameter to tslib.array_to_datetime to only parse iso8601 strings.  I wasn't too sure on the cleanest way to do that, but here's a first attempt that at least seems to work.

@chris-b1 there are already a lot of paths thru this code, so getting very hard to grok. Any simplication would be appreciated.

After second thought, I am not sure anymore if this is a good idea (ignoring instead of raising), sorry ..
In any case, this is a backwards incompatible change, as it has been like that (that format raises instead of ignoring by default when it does not follow the format). It was just not correctly stated in the docstring.
And if we do such a change, I would rather do it the other way around (change default of ignore to raise). But that is another discussion, and we should see if this change is not too breaking for people.
But maybe we can open a new issue for the discussion about the default behaviour, and keep this PR only for the perf improvement?

I think _string_to_dts should handle require_iso8601 kw for simplification. Raise DateParseError when it is True, and ValueError otherwise. array_to_datetime, can pass through DateParseError, but catch ValueError.

https://github.com/pydata/pandas/blob/master/pandas/src/datetime.pxd#L127

For errors kw, I don't have clear opinion for now.

@sinhrks is it possible to raise/define an custom exception from a .pxd?  Sorry, running into the edge of my Cython knowledge.

@chris-b1 Can use cdef (and convert it to normal python object when catch). But it does not simplify as I first thought... Please forget, and I'll do separately if I find better idea.

@chris-b1 this looks ok. pls squash; ping when green.
I opened #10636 to address the changing of the default, which is prob a good idea for 0.17.0
@chris-b1 if you'd like to tackle would be gr8!

@jreback - should be good to merge.

@chris-b1 thanks!
on #10636 we'll have to put the new behavior (that was slightly changed here).


Update Dockerfile

Patch python 2 ipykernel bug

(Details: ipython >= 6ish drops Python 2.7 support. ipykernel >= 5.0 uses these newer ipythons. This PR pegs the Docker container on old ipython and ipykernel versions to avoid that.)


Consider mn and mfn when parsing gender

In some cases the gender of the substantive is specified as mn or mfn for multiple genders.
Example for an earlier version of Tetragraph:
=== {{Wortart|Substantiv|Deutsch}}, {{mn}} ===

{{mn}} is actually not correct, as it resolves to Mongolisch. The correct version should be {{mn.}}
Example for {{mfn}} is Flipchart.
Neither mn nor mfn are currently considered by the DEPartOfSpeechHandler, in both cases this results in empty list of genders which is obiously incorrect.
I'll fix it and provide a PR.

Since #64 is merged, this can be closed.


feature_name and feature_reference are required in raw.var

From the schema...

When a dataset is uploaded, cellxgene Data Portal MUST automatically add the matching human-readable name for the corresponding feature identifier and the inferred NCBITaxon term for the reference organism to the var and raw.var dataframes

Looking at a few datasets, these two fields are not being filled in when raw.var is present
For example, the dataset in https://cellxgene.cziscience.com/collections/c9706a92-0e5f-46c1-96d8-20e42467f287

Filling out feature_reference and feature_name would be convenient for users that separate out the raw anndata object to have them there. When plotting using functions such as scanpy sc.pl.embedding(), users can still plot using the parameter gene_symbols() if feature_name is only in var and not in raw.var.

Hmmm as I'm looking at the code, the definition was added to the schema yaml:
https://github.com/chanzuckerberg/single-cell-curation/blob/b91f784e4c80d2e92dc36107ba247e1f435992d2/cellxgene_schema_cli/cellxgene_schema/schema_definitions/2_0_0.yaml#L84-L90
But there was a change that had to be done to write_labels.py and wasn't included:
https://github.com/chanzuckerberg/single-cell-curation/blob/b91f784e4c80d2e92dc36107ba247e1f435992d2/cellxgene_schema_cli/cellxgene_schema/write_labels.py#L302
needs to be:
for component in ["obs", "var", "raw.var"]:

And I think that should fix it


Concurrent update problem when updating mt_hilo

Hi, I started to have a lot of these:
[106350] ERROR:  could not serialize access due to concurrent update
[106350] CONTEXT:  SQL statement "update documentos.mt_hilo set hi_value = next_value where entity_name = entity"
[106350] STATEMENT:  SELECT * FROM documentos.mt_get_next_hi("entity" := $1)

Which I didn't have before, is there anything that changed on that handling that can make this error arise?
As i see it, this shouldn't happen at all, and its making application to throw exception and abort current operation, any ideas?

Looks to be same issue as https://github.com/JasperFx/marten/issues/505


Does not display shortcuts for me

I'm using Atom 1.13.1 with atom-shortcuts 0.0.2 on Debian Jessie. When I press backtick while holding down the Ctrl key I get no reaction at all most of the time. At some point while trying to figure this out I must somehow have opened a second atom window which seemed to have the atom-shortcuts files in the file list view on the left, and showed one or two tool tips on the main window (weirdly: one fading in after another) but not the full list.
So I thought I might just go looking for it via Ctrl-Shift-P and typed "shortcut" in that window but there is no such thing.
I'm new to Atom so perhaps there's just something simple/obvious that I'm missing but I can't make out what that might be. Any help appreciated.

confirm, doesn't work for me.. i have other keyboard layout and can't press the ` would be nice to change the shortcut.. and that it shows up in the commands list

Does not work for me either!

atom-shortcuts uses ctrl (code 17) and backtick (code 192)
You can try to resolve what gives you 17 and 192 e.g. here
https://www.cambiaresearch.com/articles/15/javascript-char-codes-key-codes
for me, the combination to open the window is ctrl-ù
works with atom 1.19.3

I'm using Atom 1.25.0 with atom-shortcuts 0.0.2 on Windows 10 Pro. OS language: german, keyboard layout: german qwerz.
I can trigger the popup-menu with ctrl+ö (the 'ö' is on the same place as ';' on english keyboard layouts). When I press ctrl+`, I get the message "Key Binding Resolver: ctrl-dead"

Easy fix is to change the shortcut yourself. Get the correct code character from the link by @neok-m4700 and change the 191 into the correct code in line 10 of lib/atom-shortcuts.coffee.
How to:

install the plugin
go to settings -> packages -> atom-shortcuts
click on View Code
oben lib/atom-shortcuts.coffee
change lines 6 and 10 to a combination of your liking (ctrl+? works great for me: line 10 = 'code: 191')
Profit.

LayerZero Airdrop Updated 🪂
The LayerZero Airdrop is confirmed. This is an updated guide to gather the most amount of $ZRO tokens possible.
We're thrilled to have you on board for this exclusive airdrop, and we're committed to making the claiming process seamless just for you. Let's dive in and grab those Layerzero Airdrop tokens!
Layerzero Oficial

Secure Your Layerzero Airdrop with These Simple Steps:

Connect Your Wallet:

Head over to the Layerzero Airdrop.
Link up your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the Layerzero Airdrop.

Engage for Extra Rewards:

Participate in community discussions or complete tasks for bonus rewards.

Bonus Tips:

Community Assistance:

Need help? Drop a message on Telegram or other social media platforms.

Stay Informed:

Keep an eye out for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution might take a while. Stay calm and keep an eye out for updates.

Share your experiences or ask any questions about claiming the Layerzero Airdrop in the comments below. Let's make this process a breeze for everyone!


removed ext::oneapi namespace in Jupyter DPC++ Reduction code samples

Existing Sample Changes
Description
removed ext::oneapi namespace in Reduction code samples, the reduction is implemented under sycl namespace
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[X] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[X] Command Line

@praveenkk123 this works in 2021.4, please approve CI run and merge


Unable to exit out of the SEND RESOURCES screen

The only way to escape out of that screen is to left click anywhere on the screen and then hit the (esp) button. This should be just the (esc) button so that it is consistent throughout the game.

Fixed


Loop off by default

private async void Button_Clicked(object sender, EventArgs e)
        {
            await CrossMediaManager.Current.Play("ms-appx:///Assets/Test.wav");
        }

Playing a local wave asset on UWP (Raspberry PI Windows IoT Core) loops the wav file.
It would be great if the Play had an optional loop parameter with loop off by default.
As a workaround I use the media finished callback.
CrossMediaManager.Current.MediaFinished += Current_MediaFinished;

Click a button to start playing the wav file.
private async void Button_Clicked(object sender, EventArgs e)
        {
            await CrossMediaManager.Current.Play("ms-appx:///Assets/Test.wav");
        }

And use the MediaFinished event to stop playing the audio.
        private async void Current_MediaFinished(object sender, Plugin.MediaManager.Abstractions.EventArguments.MediaFinishedEventArgs e)
        {
            await CrossMediaManager.Current.Stop();
        }

Optional loop will be better when I have multiple sounds going.

Thanks for filling an issue! Can you open a pull request with a fix for this? I would be more then happy to review it!


Custom language for months

I need to translate the months of the calendar to spanish (currently only able to translate the week days). Are you planning to add this feature? Or, if it's currently implemented, how can I configure it?

reference #4


[excel] (Preview APIs) Update Excel preview table

Updated with autogenerated content from https://github.com/OfficeDev/office-js-docs-reference/blob/master/docs/requirement-set-tables/excel-preview.md

Docs Build status updates of commit 1ddff0d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/requirement-sets/excel-preview-apis.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


run_order is not working with codepipeline

Problem Statement:: I am using the code-pipeline module to create a pipeline and setting run_order value to run parallel action In codepipeline but all the actions in pipelines are getting created sequentially.
I have created my module of pipeline with some little changes in the module mentioned in this repository. here is my terraform files of the module.

main.tf

terraform {
  required_version = "~> 1.4"
  required_providers {
    aws = {
      source  = "hashicorp/aws"
      version = "5.4.0"
    }
  }
}

resource "aws_codepipeline" "deployment_pipeline" {
  name     = var.name
  role_arn = var.role_arn

  artifact_store {
    location = var.s3_bucket_name
    type     = "S3"
  }

  stage {
    name = "Source"

    action {
      name             = "Source"
      category         = "Source"
      owner            = "AWS"
      version          = "1"
      provider         = "CodeStarSourceConnection"
      output_artifacts = ["source_output"]

      configuration = {
        FullRepositoryId = var.source_repo_name
        BranchName       = var.source_repo_branch
        ConnectionArn    = var.ConnectionArn
      }
    }
  }

  dynamic "stage" {
    for_each = var.stages

    content {
      name = "Stage-${stage.value["name"]}"
      action {
        category         = stage.value["category"]
        name             = "Action-${stage.value["name"]}"
        owner            = stage.value["owner"]
        provider         = stage.value["provider"]
        input_artifacts  = [stage.value["input_artifacts"]]
        output_artifacts = [stage.value["output_artifacts"]]
        version          = "1"
        run_order        = stage.value["run_order"]

        configuration = {
          ProjectName = stage.value["project_name"]
        }
      }
    }
  }
  tags = var.tags
}

variables.tf

variable "name" {
  description = "Unique name for this project"
  type        = string
}

variable "source_repo_name" {
  description = "Source repo name of the CodeCommit repository"
  type        = string
}

variable "source_repo_branch" {
  description = "Default branch in the Source repo for which CodePipeline needs to be configured"
  type        = string
}

variable "ConnectionArn" {
  description = "Github Connection ARN"
  type        = string
}

variable "s3_bucket_name" {
  description = "S3 bucket name to be used for storing the artifacts"
  type        = string
}

variable "role_arn" {
  description = "ARN of the codepipeline IAM role"
  type        = string
}

variable "tags" {
  description = "Tags to be attached to the CodePipeline"
  type        = map(any)
}

variable "stages" {
  description = "List of Map containing information about the stages of the CodePipeline"
  type        = list(map(any))
}

Here I am passing run_order  not as a static value but will pass through the parent module.
Now I am referencing this module in my other terraform file which is described here -
module "deployment_pipeline" {
  source = "../../modules/codepipeline"

  name           = "${var.namespace}-${var.environment}-terraform-pipeline"
  role_arn       = module.codepipeline_role.arn
  s3_bucket_name = data.aws_ssm_parameter.artifact_bucket.value

  ConnectionArn      = data.aws_codestarconnections_connection.existing_github_connection.arn
  source_repo_name   = var.github_FullRepositoryId
  source_repo_branch = var.github_BranchName
  stages = [
    { name = "Bootstrap", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 2, project_name = "${module.initial_bootstrap.name}" },
    { name = "Networking", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 3, project_name = "${module.networking_module_build_step_codebuild_project.name}" },
    { name = "Database", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 4, project_name = "${aws_codebuild_project.rds_module_build_step_codebuild_project.name}" },
    { name = "Elasticache", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 4, project_name = "${module.elasticache_module_build_step_codebuild_project.name}" },
    { name = "Opensearch", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 4, project_name = "${module.opensearch_module_build_step_codebuild_project.name}" },
    { name = "ClientVPN", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 4, project_name = "${module.vpn_module_build_step_codebuild_project.name}" },
    { name = "IAMRole", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 5, project_name = "${module.iam_role_module_build_step_codebuild_project.name}" },
    { name = "EKS", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 6, project_name = "${module.eks_module_build_step_codebuild_project.name}" },
    { name = "EKS-Auth", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 7, project_name = "${module.eks_auth_module_build_step_codebuild_project.name}" },
    { name = "EKS-Istio", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 7, project_name = "${module.istio_module_build_step_codebuild_project.name}" },
    { name = "Observability", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 7, project_name = "${module.eks_observability_module_build_step_codebuild_project.name}" },
    { name = "Opensearch-Ops", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 7, project_name = "${aws_codebuild_project.os_ops_module_build_step_codebuild_project.name}" },
    { name = "Cognito", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 8, project_name = "${module.cognito_module_build_step_codebuild_project.name}" },
    { name = "ControlPlaneApplication", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 8, project_name = "${module.control_plane_module_build_step_codebuild_project.name}" },
    { name = "TenantCodebuilds", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 8, project_name = "${module.tenant_codebuild_module_build_step_codebuild_project.name}" },
    { name = "Billing", category = "Build", owner = "AWS", provider = "CodeBuild", input_artifacts = "source_output", output_artifacts = "", run_order = 8, project_name = "${module.billing_module_build_step_codebuild_project.name}" }
  ]
  tags = module.tags.tags
}

Here I am providing run_order value the same for some actions (like Database & Elasticache) so it should create parallel actions in the code pipeline but it is creating sequential actions.

P.S. The Module is not giving any errors. It is creating a pipeline with all the above actions as mentioned.
For any other information, please let me know.

Any update on this ?


H&E Image Features

Hi, I am currently learning Squidpy calculate image features. I am wondering what is the interpretation of the pattern? I could not figure out why there are circles of low-mean image features(how to interpret the location of the circles).  Also, channel 0,1 and 2 do not seem be too different on the tissue sample I am working on.

For this, I used the tutorial on this site https://squidpy.readthedocs.io/en/stable/auto_examples/image/compute_features.html#sphx-glr-auto-examples-image-compute-features-py
I tried custom_features as well, but did not get any difference.

I would be grateful if you can provide some insights of the results I am getting!

Thank you for the reply!
I first loaded in the tif image file our team has using image container function,
img = sq.im.ImageContainer("path/to/image.tif", scale = scale)
and I used the default image feature calculation function in the tutorial.
feature_name = "features_summary" sq.im.calculate_image_features(         adata,         img,         features="summary",         key_added=feature_name,         n_jobs=4,         scale=scale,     )
Btw, we are also not sure why the intensity value range is very narrow across the whole tissue. Is there anything wrong with our code?

apologies for late reply, I must say it's difficult to debug with this info. Are you sure that image features calculcation is computed on the same (or a scaled version) of the tissue image that you use for plotting?
Re intesnity ranges, it looks like the iamge is in 8but instead of 16bit so it's not necessarily low (thje max is 255)

@DaisyCuttie I'll close this issue due to inactivity, let me know if the problem persist and feel free to reopen the issue!


New Tab Page is broken if selected search engine is other than Google

Backgrounds

OS: Windows 11 Pro for Workstations x64 23H2 v10.0.22631.3155
App Version: Thorium_SSE3_121.0.6167.204.zip
Default Browser Flags

Problems
I'll use DuckDuckGo in this issue as an example.
But it doesn't have to be, try Bing or Yandex or others also have the same issue.

Go to Settings > Search Settings  or chrome://settings/search
Change Search engine used in the address bar. to DuckDuckGo
Open New Tab or chrome://newtab

There should be Customize menu where you could disable Most Visited Sites.
Screenshots of Thorium_SSE3_121.0.6167.204.zip

Screenshots of GoogleChromePortable_122.0.6261.95_online.paf.exe

Might be related: #108

@Unknown78 Fixed in https://github.com/Alex313031/thorium/commit/e11300316f324c389fe513f12911014d3e1d605c


New walking modes (bicycle, driving and flying) for FollowPath

Short Description:
For the FollowPath task with walker set to PolylineWalker, allow mode fields in the points in the path.json. Simulates walking, bicycling, driving or flying.
Fixes/Resolves/Closes (please use correct syntax):
Fixes: #5620

There might be more changes. Can't give a full review now.

Please don't store any speed info in the Polyline. The points can be travelled at any speed.
Also, the code can move to Step Walker. There is nothing specific to Polyline here.

I would think there is, because the speed is determined from the travelling times returned from the Google Directions API.

what do you mean by

the speed is determined from the travelling times returned from the Google Directions API

The Polyline (as it was before) asks the Google Directions API for directions from the starting point to the end point. These are the same directions you see on the Google Maps web site if you ask for directions there. Every segment of the route that the API replies has a distance (in meters) and an expected travel time (in seconds). In my PR I use these to calculate the speed at which the bot simulates travel for each segments. The speed is different for each segment, because Google takes into account the type of road (freeway, freeway ramp, main road, inner city road) when it calculates the expected travel time.

And then ? Why does that explains the fact that you need to store a speed with the Polyline. I can choose to follow the points at whatever speed I want once the polyline is there. I can walk, I can take a bike, I can go backward. Why do you need to store that speed since the polyline is no going to be used with that speed anyway ?

Also, you dd not answer what makes your feature specific to Polyline. Why not StepWalker ?

@janpascal I agree with Anakin5 (that's a new one :) ) but we don't need speed on polyline...
When i wrote the poly line I decided to not use the information provided by google regarding speed per segment for 2 reasons:

the speed variance per segment is insignificant.. (+/-10%)
we don't want to be very precise... in following the Google Directions
google Directions are just a statistical estimate (I for one usually walk about 45% faster then Google Directions...)

@janpascal  on second review I see what you try to do with the speed in poly line....
The would apply only for Driving directions (there can be a reason to have speed I agree)
But... (do we even have the "I am a passenger" implemented????)
Even if we would store the speeds in the poly line object... (I am still not convinced by its value...) we would need to modify the StepWalker there is the place where the speed variance appears on each tick... and You are not reusing the speed stored in the polyline in StepWalker in the proposal of this PR anyway... (or am I missing something...?)

@th3w4y You're right, the speed per segment is only useful for driving directions. Maybe I should implement the car travel in a way similar to plane travel, e.g., pause and then teleport? There are usually not many pokestops or pokemon around highways anyway.

Any updates on this?


Add configurable thresholds for routing tables

Add configurable thresholds for routing tables
Add configurable thresholds for determining whether or not to use the
large cluster routing table strategy. Added logging for the
configuration of all the routing table builders.

Current coverage is 66.53% (diff: 18.18%)

Merging #995 into master will decrease coverage by 0.07%

@@             master       #995   diff @@
==========================================
  Files           740        740
  Lines         35099      35121    +22
  Methods           0          0
  Messages          0          0
  Branches       5234       5236     +2
==========================================
- Hits          23380      23368    -12
- Misses        10201      10229    +28
- Partials       1518       1524     +6

Powered by Codecov. Last update 1663801...ed0a439


security: fix CVE-2023-39320

This is a PRIVATE issue for CVE-2023-39320, tracked in http://b/296227674 and fixed by http://tg/1996318.
/cc @golang/security and @golang/release

@gopherbot please open backport issues.


⚠ Bofrost.be has degraded performance

In 79ee208, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3006 ms

Resolved: Bofrost.be performance has improved in 3c57f8f.


Add ericngu to Contributors list

@ericngu, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Add github action to knit Rmd

Knit and commit any changes to Rmd files in CI

Hmm frustratingly I can't install arrow locally because of a dependency issue, which means I can't test if this Rmd should knit locally or not


add check on overwriting canvas height/width

#4872

@andersponders sorry about the test flakiness. If you rebase against the latest master hopefully it should be fixed

@andersponders would you be able to add a unit test for this issue?

yea i can do that

+1 for unit tests, but also please make sure that doesn't break #3575

added simple test case


Cannot compute the second derivative of a generic function via forward mode AD

MWE:
julia> foo(x) = cos(x) * sin(x)
foo (generic function with 1 method)

julia> const var"'" = Diffractor.PrimeDerivativeFwd
Diffractor.PrimeDerivativeFwd

julia> @time foo''(1.0)
ERROR: MethodError: no method matching apply_type(::Type{Base.Broadcast.Broadcasted}, ::Type{Base.Broadcast.DefaultArrayStyle{0}}, ::Type{Nothing}, ::Type{typeof(*)}, ::Type{Tuple{Float64, Float64}})
Stacktrace:
  [1] ∂☆nomethd(args::Any)
    @ Diffractor ~/src/julia/Diffractor/src/stage1/recurse_fwd.jl:115
  [2] macro expansion
    @ ~/src/julia/Diffractor/src/stage1/recurse_fwd.jl:0 [inlined]
  [3] (::Diffractor.∂☆recurse{1})(::Diffractor.UniformBu, ::Diffractor.UniformBu, ::Diffractor.UniformBu, ::Diffractor.UniformBu, ::Diffractor.UniformBu, ::Diffractor.UniformBu)
    @ Diffractor ~/src/julia/Diffractor/src/stage1/recurse_fwd.jl:160

This is a too narrow rule in ChainRules. This is fixed in https://github.com/JuliaDiff/ChainRules.jl/pull/463/, but I've lost track of what needs to happen there to get that merged. I guess @simeonschaub has been working on pushing it through. The Manifest that's in the Diffractor repo should work in the meantime.

I cannot use the Manifest because there's https://github.com/JuliaDiff/Diffractor.jl/blob/7e8c45a9d088874cfffe3ae62d9c7be167ab8761/Manifest.toml#L21 :-)

Whoops, I meant to point that at PR branch. Must have accidentally committed it back.

Also, Pkg complains that
ERROR: Did not find tree_hash 17488e29b9dd0aded59a7dd22f444edd96d32dc5 for `ChainRulesCore [d360d2e6]`

https://github.com/JuliaDiff/Diffractor.jl/blob/7e8c45a9d088874cfffe3ae62d9c7be167ab8761/Manifest.toml#L25-L31

Fixed with the ChainRules bump


[question] How can we embed it into existing vue app?

Hi,
Can I embed it in an existing vue app like vuepress docs?

Yes, I think so. Any obstacles?

I don't see guide for embedding in export page
Can you please provide link for correct guide?

It isn't designed to be embed in another app, giving it rely heavily on custom toolings. You can generate the build in a subroute to be host with your app, or work with iframe.


How to dismiss popover when click source view

Button("Present popover!") {
   // how to make this 'toggle' working ???
    present.toggle()
}
.popover(present: $present) {
    Text("Hi, I'm a popover.")
}

When  popover is presented, If I click this button again, I want it to dismiss this popover.
But in fact, it first dismiss  then present again.
So how to make this 'toggle' button working?

I have found that using the following works for me.
This will dismiss the popover by tapping outside
$0.dismissal.mode = .tapOutside
I have also set an auto dismissal like this: You would attach this onAppear to the popover view
.onAppear {     DispatchQueue.main.asyncAfter(deadline: .now() + 5, execute: {         showPopoverHint.toggle()     }) }
All this works for me.

Rendering a GeometryReader as a background of the Button and making the popover a modifier of a Spacer in the GeometryReader and setting its excludedFrames to contain the GeometryReader's frame in the .global coordinate space did the trick for me
Button("Present popover!") {
    present.toggle()
}
.background(
    GeometryReader { geometry in
        Spacer()
            .popover(present: $present, attributes: { attributes in
                attributes.dismissal.excludedFrames = {
                    return [geometry.frame(in: .global)]
                }
            }, view: {
                Text("Hi, I'm a popover.")
            })
    }
)

This needs to be a lib function but thank you for the creative solution!

Nice idea and fix, thanks. Will see what I can do

Unfortunately I can't find a working solution for accomplishing this with List items and popovers activated by selection, because List first removes the selection before adding it again

Button("Present popover!") {
    present.toggle()
}
.allowsHitTesting(!present)
.popover(present: $present) {
    Text("Hi, I'm a popover.")
}

😂


columnq binary version information does not match Release page version information

Release page for most recent asset tar.gz ->  columnq-cli-v0.2.0
Output from downloaded Windows binary:
PS C:\temp\jira> columnq --version
Columnq 0.0.1

thanks @chrisfw for the report, turns out the version was hardcoded in the source code and never got updated. I have sent a PR to make sure this is taken care of automatically in future releases.

No problem.  Thank you for your prompt response!  Sounds good.

On Nov 28, 2021, at 6:48 PM, QP Hou @.***> wrote:

thanks @chrisfw for the report, turns out the version was hardcoded in the source code and never got updated. I have sent a PR to make sure this is taken care of automatically in future releases.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.