[Beta] ipBlock for Network Policies

Feature Description

One-line feature description (can be used as a release note):

CIDR selector support for egress/ingress network policies

Primary contact (assignee): @cmluciano
Responsible SIGs: SIG-Network
Design proposal link (community repo): kubernetes/kubernetes#49978
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred: @caseydavenport @danwinship
Approver (likely from SIG/area to which feature belongs): @thockin
Feature target (which target equals to which milestone):

Beta release target (1.8)
Stable release target (1.9)

code complete in https://github.com/kubernetes/kubernetes/pull/50033.
TODO:

[ ] More e2es
[ ] Docs

@cmluciano any update on missing docs?  The PR is due today.

@caseydavenport @cmluciano mentioned you as the contact for docs.  We need them ASAP in order to make the 1.8 release docs.

Docs PR is here: https://github.com/kubernetes/kubernetes.github.io/pull/5528

Docs fixed in https://github.com/kubernetes/kubernetes.github.io/pull/5473#issuecomment-331616722

this was merged for 1.8

reopening for tracking stable

@cmluciano
Any plans for this in 1.11?
If so, can you please ensure the feature is up-to-date with the appropriate:

Description
Milestone
Assignee(s)
Labels:

stage/{alpha,beta,stable}
sig/*
kind/feature

cc @idvoretskyi

This feature current has no milestone, so we'd like to check in and see if there are any plans for this in Kubernetes 1.12.
If so, please ensure that this issue is up-to-date with ALL of the following information:

One-line feature description (can be used as a release note):
Primary contact (assignee):
Responsible SIGs:
Design proposal link (community repo):
Link to e2e and/or unit tests:
Reviewer(s) - (for LGTM) recommend having 2+ reviewers (at least one from code-area OWNERS file) agreed to review. Reviewers from multiple companies preferred:
Approver (likely from SIG/area to which feature belongs):
Feature target (which target equals to which milestone):

Alpha release target (x.y)
Beta release target (x.y)
Stable release target (x.y)

Set the following:

Description
Assignee(s)
Labels:

stage/{alpha,beta,stable}
sig/*
kind/feature

Once this feature is appropriately updated, please explicitly ping @justaugustus, @kacole2, @robertsandoval, @rajendar38 to note that it is ready to be included in the Features Tracking Spreadsheet for Kubernetes 1.12.

Please note that Features Freeze is tomorrow, July 31st, after which any incomplete Feature issues will require an Exception request to be accepted into the milestone.
In addition, please be aware of the following relevant deadlines:

Docs deadline (open placeholder PRs): 8/21
Test case freeze: 8/28

Please make sure all PRs for features have relevant release notes included as well.
Happy shipping!
P.S. This was sent via automation

doh, we did all of the discussion for this feature on the egress PR (#366) and forgot to update this PR as well.
Let's see if this still works:
/milestone v1.12
/remove-stage beta
/stage stable

@justaugustus, @kacole2, @robertsandoval, @rajendar38 This was supposed to have been tracked for 1.12 but we basically forgot that we had two separate feature PRs. All of the work is done, including tests and docs.

(ping @cmluciano)

@danwinship I've got ahead and added this to the tracking sheet after seeing @tpepper +1 on the exception filed.

@kacole2 Thanks! It probably should have milestone v1.12 too?

@danwinship it should but I don't have those powers yet.
@justaugustus can you add the correct labels?

/close


Strip log lines of trailing spaces only - this breaks exception traceback formatting

@jarodwilson, is removing this strip() a problem? https://github.com/projectatomic/osbs-client/pull/665#discussion_r132914664

Would be nice to include a test for this.

Will do, I'm thinking rstrip() would be a better solution, although I haven't seen extra newline chars yet

If I'm remembering correctly, not having the strip there caused some minor formatting oddness with self-tests, but I think they could be adjusted slightly to compensate if that strip is really only useful for the self-test.

Okay, never mind, the .strip() seems to be entirely superfluous now. I recall it being needed for self-tests for some iteration of this code, but it seems to not be now, so yeah, have at it, remove it entirely!

Not updating release notes. This is a bugfix to unreleased changes.


[SINGA-101] Add ll (ls -l) command in .bashrc file when using docker

Root loads docker containers while the command ll (ls -l) cannot be found. However ll is handier than ls -l.

Hi, would you please re-send the pull request with a clean commit based on the latest master branch.
It seems that you merged one old branch with the latest master and then worked on it.
It would be better to follow this work flow

checkout the latest master branch to a new branch
do the update on the new branch, and commit
checkout to the master branch and pull the master from incubator-singa github
checkout to the new branch, rebase to the latest master branch
send pull request


suggestion on simple installation

It may be even easier to install using devtools:

install.packages("devtools") ## if needed
devtools::install_github("cmap/cmapR")

Good point, thank you! Will update the instructions.

Hello @tnat1031 , installing cmapR on the latest version of R/RStudio doesn't seem to work anymore. Could you please suggest alternatives (in R/shell) to work with the GCTx data format? Thank you!
platform       x86_64-apple-darwin15.6.0
arch           x86_64
os             darwin15.6.0
system         x86_64, darwin15.6.0
status
major          3
minor          5.1
year           2018
month          07
day            02
svn rev        74947
language       R
version.string R version 3.5.1 (2018-07-02)
nickname       Feather Spray

Hi Janani,
Can you please post the command(s) you ran and the error produced so I can
try to troubleshoot? Thanks!
On Mon, Apr 15, 2019 at 6:49 PM Janani Ravi<EMAIL_ADDRESS>wrote:

Hello @tnat1031 https://github.com/tnat1031 , installing cmapR on the
latest version of R/RStudio doesn't seem to work anymore. Could you please
suggest alternatives (in R/shell) to work with the GCTx data format? Thank
you!
platform       x86_64-apple-darwin15.6.0
arch           x86_64
os             darwin15.6.0
system         x86_64, darwin15.6.0
status
major          3
minor          5.1
year           2018
month          07
day            02
svn rev        74947
language       R
version.string R version 3.5.1 (2018-07-02)
nickname       Feather Spray
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cmap/cmapR/issues/6#issuecomment-483447150, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB3i7zZ8WEc40jmplMgapvNhoI_-JWcdks5vhQIFgaJpZM4O64sP
.

Thanks for your quick response, @tnat1031 I have trouble with cmapR installation in the latest version of R (version details incl. above). I get the following errors.
devtools::install_github("cmap/cmapR")
Downloading GitHub repo cmap/cmapR@master
Error in getrootdir(untar(src, list = TRUE)) :
  length(file_list) > 0 is not TRUE

install.packages("cmapR")
Installing package into ‘/Users/janani/Google_Drive/GitHub/drug-repurposing/packrat/lib/x86_64-apple-darwin15.6.0/3.5.1’
(as ‘lib’ is unspecified)
Warning in install.packages :
  package ‘cmapR’ is not available (as a binary package for R version 3.5.1)

@tnat1031 thanks again for your help earlier today.
I'm trying to install using the following installation script for cmapR:
## Installing dependencies
install.packages("data.table")

install.packages("BiocManager") # to install rhdf5
BiocManager::install("rhdf5", version = "3.8")

install.packages("devtools") # if you don't have it

## Installing cmapR
devtools::install_github("cmap/cmapR", dependencies=T, force=T)

Great, thanks Janani. Does this script work for you?
On Tue, Apr 16, 2019 at 2:57 PM Janani Ravi<EMAIL_ADDRESS>wrote:

@tnat1031 https://github.com/tnat1031 thanks again for your help
earlier today.
I'm trying to install using the following installation script for cmapR:
Installing dependencies
install.packages("data.table")
install.packages("BiocManager") # to install rhdf5
BiocManager::install("rhdf5", version = "3.8")
install.packages("devtools") # if you don't have it
Installing cmapR
devtools::install_github("cmap/cmapR", dependencies=T, force=T)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cmap/cmapR/issues/6#issuecomment-483800966, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB3i70c-TDVI4H3bmTTQK_5PpXICocUnks5vhh0ggaJpZM4O64sP
.

My student who's very new to R installed it with this! So, I am guessing that it does. I will get you more feedback within a day or two.

Ok great, thanks for adding to the repo!
On Tue, Apr 16, 2019 at 3:31 PM Janani Ravi<EMAIL_ADDRESS>wrote:

My student who's very new to R installed it with this! So, I am guessing
that it does. I will get you more feedback within a day or two.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cmap/cmapR/issues/6#issuecomment-483812159, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB3i74qG7zbnB0yjIvqp3ukuJ4TDLkcLks5vhiUKgaJpZM4O64sP
.

The error in the main project w/ packrat remains unless I load it as an external package (packrat settings). I tried it with both devtools::install_github & remotes::install_github, just in case.
Downloading GitHub repo cmap/cmapR@master
Error in getrootdir(untar(src, list = TRUE)) :
  length(file_list) > 0 is not TRUE


Record metadata on who learns commands

Predictably, my friends are adding weird commands, and changing commands in PMs, and not fessing up when we try to figure out whodunit. I'm probably doing this too.
Anyways, add ?meta <command> to get metadata on who added a specific command, and implement command storage as a list of commands.

Does this affect #6 ? Particularly the key/value store interface - do you need more functionality of the datastore to be able to implement this?

Initial thought: use Redis' lists, where the metadata is stored as serialized protobufs. So I'll likely add a second model implementation that abstracts this away; the current one I called StringMap (which is passed a bucket name on instantiation, and acts as a map[string]string), so it seems reasonable to have the command list one be a StringListMap (since it doesn't need to know that it's storing commands at the DB layer).
I ended up abstracting the bucket name away from the interface, which allows me to pass in individual StringMaps to features, which limits the scope of what they can access in Redis to exactly what they need.

No longer needed. Public ?unlearn, and some conversations, fixed all of these types of issues

i think this could still be useful in case someone private learns something
problematic. depends on whether the community decides learners have a right
to privacy.
On Thu, Jul 27, 2017, 8:10 PM Jake Voytko<EMAIL_ADDRESS>wrote:

Closed #12 https://github.com/jakevoytko/crbot/issues/12.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jakevoytko/crbot/issues/12#event-1182734828, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADte5AJhWUYKrYnEbD_XjRTXX_3YZc93ks5sSSbmgaJpZM4MxAdh
.


[DNM] Test clang patch on feedstock

@IAlibay  this is a test where we can see if proposed clang patch for https://github.com/MDAnalysis/mdanalysis/issues/4397 works as expected on feedtsock.

Patch applied in MDA main, closing this.


NTR: start and end date terms for popbio collections

Background several popbio datasets have specimen collections with either date ranges ("May to July 2010") or multiple dates ("2002, 2004, and 2005") or both.  Also as in the examples, the dates are often incomplete and refer to whole months and/or years (which means they are date ranges too).
Currently the date fields in EDA can't handle ranges or incomplete dates...
We may develop a DateRange field type for EDA (in addition to Number, Date and String) in the future, but for now here is a plan

create new variables for the specimen collection entity:

specimen collection start date : YYYY-MM-DD
specimen collection end date : YYYY-MM-DD
specimen collection dates are discontinuous : Yes | No

convert the potentially multiple and incomplete dates into an overall start and end date (first two variables) and figure out if there any gaps in the date ranges (then the third variable will be Yes)
store the raw multiple date/range string (semicolon delimited) in the existing variable: OBI_0001619

Term Label

specimen collection start date
specimen collection end date
specimen collection dates are discontinuous

Definition

A time measurement datum that is the measure of the time when the specimen collection started
A time measurement datum that is the measure of the time when the specimen collection ended
Information that the collection process was not continuous for the entire period between start and end dates

Suggested Parent Term
http://purl.obolibrary.org/obo/IAO_0000416
Requester
Bob/VectorBase/PopBio

@bobular Do these work?
label: date specimen collection started
definition: A time measurement datum that specifies the date a specimen collection process started.
label: date specimen collection ended
definition: A time measurement datum that specifies the date a specimen collection process ended.
label: indicator of specimen collection date discontinuity
definition: An information content entity that specifies that there was at least one date, between the start and end dates of a specimen collection process, during which specimen collection did not occur.

Can also consider to add the logic axiom like, 'is about' some starts/ends some 'specimen collection'

These axioms follow date treatment started. Reserving IDs:
EUPATH:0043256
label: date specimen collection started
definition: A time measurement datum that specifies the date a specimen collection process started.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and starts some 'specimen collection process')
EUPATH:0043257
label: date specimen collection ended
definition: A time measurement datum that specifies the date a specimen collection process ended.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and ends some 'specimen collection process')
EUPATH:0043258
label: indicator of specimen collection date discontinuity
definition: An information content entity that specifies that there was at least one date, between the start and end dates of a specimen collection process, during which specimen collection did not occur.
parent: information content entity
subclass axiom: 'is about' some 'specimen collection process'

@bobular Are the definitions in the above comment still OK?

Hi @johnwjudkins and @zhengj2007 - these look perfect, thanks!

I'll make a pull request for this once the refactoring pull request is merged.

We shoulld revise these. A specimen collection process, as defined in OBI, results in a new single specimen. It looks like we are talking about the beginning and ending of a process that results in a collection of specimens.
Thus:
EUPATH:0043256
label: date of start of process resulting in collection of specimens
definition: A time measurement datum that specifies the date that starts a process that results in a collection of specimens.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and starts some (has_specified_output some 'collection of specimens'))
EUPATH:0043257
label: date of end of process resulting in collection of specimens
definition: A time measurement datum that specifies the date that ends a process that results in a collection of specimens.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and ends some (has_specified_output some 'collection of specimens'))
EUPATH:0043258
label: indicator of specimen collection date discontinuity
definition: An information content entity that specifies that there was at least one date, between the start and end dates of a process resulting in a collection of specimens, during which no specimens were obtained.
parent: information content entity
subclass axiom: 'is about' some (has_specified_output some 'collection of specimens')
("Collection" is troublesome, because it can refer to a process of collecting one thing, or it can refer to a group of things, but unfortunately, that word is already in use in the ontologies. It's a mess.)

@bobular Could you please review the updated term labels and definitions shown above? Thanks!

Sorry, the discontinuity definition
"An information content entity that specifies that there was at least one date, between the start and end dates of a process resulting in a collection of specimens, during which no specimens were obtained"
Implies that collection effort was continuous but the success was only on certain days.
What it should indicate is that the collection effort stopped for a day or more. Sorry to be a pain!

@bobular
No problem. Is this better? "An information content entity that specifies that, between the start and end dates of a process resulting in a collection of specimens, there was at least one date that no collection attempts were made."

That's great @johnwjudkins - thanks!

@zhengj2007
EUPATH:0043256
label: date of start of process resulting in collection of specimens
definition: A time measurement datum that specifies the date that starts a process that results in a collection of specimens.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and starts some (has_specified_output some 'collection of specimens'))
EUPATH:0043257
label: date of end of process resulting in collection of specimens
definition: A time measurement datum that specifies the date that ends a process that results in a collection of specimens.
parent: time measurement datum
subclass axiom: 'is about' some ('process boundary' and ends some (has_specified_output some 'collection of specimens'))
EUPATH:0043258
label: indicator of specimen collection date discontinuity
definition: An information content entity that specifies that, between the start and end dates of a process resulting in a collection of specimens, there was at least one date that no collection attempts were made.
parent: information content entity
subclass axiom: 'is about' some (has_specified_output some 'collection of specimens')


Error using maximbaz/arch-build-aur:latest build image on fedora

I get the following error if I try to use maximbaz/arch-build-aur:latest to build my AUR packages:
error: failed to initialize alpm library
(could not find or read directory: /var/lib/pacman/)

the error seems related to #12 in fact if I rebuild the image locally adding:
# WORKAROUND for glibc 2.33 and old Docker
# See https://github.com/actions/virtual-environments/issues/2658
# Thanks to https://github.com/lxqt/lxqt-panel/pull/1562
RUN patched_glibc=glibc-linux4-2.33-4-x86_64.pkg.tar.zst && \
    curl -LO "https://repo.archlinuxcn.org/x86_64/$patched_glibc" && \
    bsdtar -C / -xvf "$patched_glibc"

to the Dockerfile all works as expected...

Hello, if I understood it correctly it's due to a bug in older docker. Would you be able to upgrade docker on your machine and retry?

docker version says 19.03.11 release date last December, I will try with something newer.

Working with docker 20.10.5.
I missed the part of the issue mentioning an "old" docker version as part of the problem.
Tnx!


Refont of identity card button

Description 📹
Current one :
https://app.starknet.id/identities/855076439471

New design :

Figma file : https://www.figma.com/design/S1UKYgWewNqNHZFAaBUilG/🏝-Starknet-ID?node-id=4110-41607&t=wEy6O4uIMpUixad7-1
We have redesigned the identity card page, including the button layout, title size, text size, button colors, and more. By clicking 'View More', you can see all the buttons available for changes.
Proposed Actions 🛠
Here’s a checklist of actions to follow for resolving this issue:

Fork and Create Branch:
Fork the repository and create a new branch using the issue number:
git checkout -b fix-[issue-number]

Implement Changes:
[Insert Code snippet if needed with a mardown todo list]

Run Tests and Commit Changes:
Make sure your changes don't break existing functionality and commit with a clear message:
git commit -m "Fix: [Short description of the fix]"

Required 📋
To keep our workflow smooth, please make sure you follow these guidelines:

Assignment: Don't create a pull request if you weren’t assigned to this issue.
Timeframe: Complete the task within 3 business days.
Closing the Issue: In your PR description, close the issue by writing Close #[issue_id].
Review Process:

Once you've submitted your PR, change the label to "ready for review".
If changes are requested, address them and then update the label back to "ready for review" once done.

Testing: Test your PR locally before pushing, and verify that tests and build are working after pushing.

Thank you for your contribution 🙏
⚠ WARNING: Failure to follow the requirements above may result in being added to the OnlyDust blacklist, affecting your ability to receive future rewards.

Let me handle this issue!

Can I be assigned to this?

Can I work on this issue? @Kevils
I am a fullstack and blockchain developer with experience for more than 1.5 years and an active contributor to starkent
My profile: https://app.onlydust.com/u/SudiptaPaul-31
ETA-1Day

Can I be assigned to this?

May I try my hand at this?  I’m a full-stack blockchain developer with over 3 years of experience working with Tailwind, React, TypeScript, and Solidity/Rust.

Is it okay if I take this?


[gpuCI] Forward-merge branch-21.08 to branch-21.10 [skip gpuci]

Forward-merge triggered by push to branch-21.08 that creates a PR to keep branch-21.10 up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge.

FAILURE - Unable to forward-merge due to conflicts, manual merge is necessary. Do not use the Resolve conflicts option in this PR, follow these instructions https://docs.rapids.ai/maintainers/gpuci/#forward-mergers
IMPORTANT: Before merging and approving this PR, be sure to change the merging strategy to Create a Merge Commit. Otherwise, history will be lost and the branches become incompatible.


[Technical Question] Help On Sample - provision_symmetric_key.py

Hi there
I´m asking for help because pyhton isn´t my "native language" but I need a sample on how to properly send an infinite number of messages(with intervals) to IoTHub(using async) that can be properly "canceled".
I explain:If we take the sample on https://github.com/Azure/azure-iot-sdk-python/blob/master/azure-iot-device/samples/async-hub-scenarios/provision_symmetric_key.py as an example.
How to send "n" messages over time with 20 seconds interval. Should the async def send_test_message(i) method have a While True loop with a time.sleep or a await asyncio.sleep?
How to properly stop this from running?
I would appreciate a simple sample on this.
Thanks everyone
AB#7526323

It seems you are tying to just send message to IoTHub , not sure why you are using provisioning sample.
Here is a sample that sends multiple messages to IoTHub.
Is there a reason why you would like to have an interval between sending each message ?
With async it is possible to actually send them at the same time.
If you are looking for a pattern where you would like to have an interval between messages , then this sample should help you.

@jorgevinagre, thank you for your contribution to our open-sourced project! Please help us improve by filling out this 2-minute customer satisfaction survey


chore(test): set up for monorepo test running

Whilst working on #1542 I noticed it's a bit of a pain to run tests right now. This PR sets the repo up for easy jest test running; each package can have it's own configuration, optionally based on the base configuration. The root jest.config.js is just responsible for loading all these other projects.
You'll notice that I've replaced <rootDir> with ${__dirname} in various places, because <rootDir> now refers to each of the package roots rather than the repo root.
You'll also note that I removed the testPathIgnorePatterns: ['codemirror-graphql','packages/graphiql'] in favour of having those projects configure their own test matches (in this case, matching nothing).
I have NOT checked that code coverage still works; that's the one thing that may need some more scrutiny. Other than this, I think this change is completely uncontroversion and highly beneficial :)

awesome! we disabled subpackage test scripts though, the goal is to run tests from the root. trying to avoid subpackage scripts entirely so that scripts are always executed at the root. Development.md has tips for executing tests for specific directories from the root.

I feel like we tried this approach at some point and there was a reason we chose against it. I will check it out later, as I'm knee deep in plugin API rewrite

To be clear; this approach allows you to run the tests from the root and, should you desire, also from the package. I've only ran them from the root so far, and it works great:

The main advantage is allowing per-project configuration when desired. In particular you currently have graphiql and codemirror-graphql entirely disabled in the root config which makes it incredibly difficult to run their tests; with this approach you can instead configure those projects to run only the tests you wish to allow without making your configuration complex for other projects.

cool! whatever works.  the handy part about this approach, from proprietary projects, is that it's easier to override configs and add different test renderers and mocks to specific subpackages.
also note that many tests are disabled, as most of whats in graphiql package is still in the process of being rewritten, reorganized or deprecated.

tried pinging you on discord, do you have time for a quick call?


Unable to install via composer

Hi,
When trying to install via compose getting error as below:
composer require crate/crate-dbal:~0.0.2
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - crate/crate-dbal 0.0.4 requires crate/crate-pdo 0.0.5 -> satisfiable by crate/crate-pdo[0.0.5].
    - crate/crate-dbal 0.0.3 requires crate/crate-pdo 0.0.4 -> satisfiable by crate/crate-pdo[0.0.4].
    - crate/crate-dbal 0.0.2 requires crate/crate-pdo 0.0.3 -> satisfiable by crate/crate-pdo[0.0.3].
    - crate/crate-pdo 0.0.5 requires amphp/artax ~1.0.0-rc2 -> no matching package found.
    - crate/crate-pdo 0.0.4 requires amphp/artax ~1.0.0-rc2 -> no matching package found.
    - crate/crate-pdo 0.0.3 requires amphp/artax ~1.0.0-rc2 -> no matching package found.
    - Installation request for crate/crate-dbal ~0.0.2 -> satisfiable by crate/crate-dbal[0.0.2, 0.0.3, 0.0.4].

Regards,
Chris

Fixed by switching http library at crate-pdo dependency. A new release 0.1.0 including this change is out now.


ReleaseNotes: tangshimin.MuJing version 2.2.16

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

(Automated response - build 694.)

@wingetbot waivers Add Validation-Executable-Error


[Prod] ddmclav Correcting Verbiage

Line #134 had 2 "are"s in the sentence; submitting a PR to correct it and leave one.
What does this PR do?
Corrects line 34 so there is only one "are" in the sentence.
Motivation
N/A
Additional Notes
N/A
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Thanks @ddmclav I also had to update the spec.yaml, which is where this config file is generated from.


Fixed_Time: change parametrization

Currently the outer hardening timescale slope is $\gamma_1 + \gamma_2 + 1$, but really it should be $\gamma_2'$.

This is fixed in https://github.com/nanograv/holodeck/commit/f56f72d4079b5f7f35bb8f8704e6eaee107a75f7
Note that the new, standard phenomenological model is now Fixed_Time_2PL with the new parametrization of hardening rates.  The old behavior is preserved in Fixed_Time (now a subclass of Fixed_Time_2PL) which gives a deprecation warning on initialization.


feat: adds random pallets

What
feat: adds random pallets

:tada: This PR is included in version 0.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dropping atty crate in favour of is_terminal.

The atty crate has been reported by advisories as problematic and not maintained any longer.
Switching to is_terminal instead, as a safer and portable replacement (as it does not use atty in the production code).
I've made a practical check in the app that uses cucumber, and I get identical coloured stdout messages while running the test suite.

@achary you should probably bump version of is_terminal to 0.4.3 or something like that. I think that pipeline fails because we run MSRV ci job with -Z minimal-versions flag, which downgrades crate versions as much as possible.

Hmm. Exactly same failure for MSRV with 0.4.3. Kind of annoying....

@achary thanks! I wonder why you've closed this, but will try to finish it in #260.


Add support for editMessageMedia

Add support for editMessageMedia
Describe the solution you'd like
I would like to be able to exchange a picture I sent using the telegram bot to display a more recent version of it. (3D Printer Webcam)
Currently I'm deleting the sent picture and sending a new one but this causes a new notification on my phone and desktop (without sound as I set the disableNotification parameter)

Added in 1.8.3 (See readme)


Incorrect handling of unions of arrays.

TypeScript Version:  2.1
Code
This code works although the type of the forEach callback is not suffuient.
const a: boolean[] | number[] = [1,1,1];
a.forEach((b: number) => console.log);

But this fails (which is expected)
const a: boolean[] | number[] = [1,1,1];
a.forEach((b: boolean) => console.log);

I'd expect only this code to be valid
const a: boolean[] | number[] = [1,1,1];
a.forEach((b: boolean | number) => console.log);

I understand that boolean[] | number[] is a subtype of (boolean | number)[] and would expect typescript to be stricter if I provide the former.

Your first example compiles because control flow analysis has determined the type of a to be number[] at the call to forEach because the preceding assignment assigned an array literal of type number[].

This is the smallest example I could create to recreate an issue.
In
channelEncoding.forEach(function(fieldDef: FieldDef | OrderChannelDef) {
  fields.push(field(fieldDef));
});

I know that channelEncoding is a FieldDef[] | OrderChannelDef[].
But I get the following error
[ts] Cannot invoke an expression whose type lacks a call signature. Type '((callbackfn: (value: FieldDef, index: number, array: FieldDef[]) => void, thisArg?: any) => void...' has no compatible call signatures.

Yes, the error is slightly different but I thought that was because ts handles complex types differently from simple types.
I thought this error is caused by the behavior of the flow analysis (which is being smarter than my explicit type annotations).

This error is reported because forEach has a different signature in each of the two arrays, so there is not a common signature we can use in the call expression. You can help by providing a type assertion:
(channelEncoding as (FieldDef | OrderChannelDef)[]).forEach(...)

Alternatively, you could change channelEncoding to have type (FieldDef | OrderChannelDef)[]. It is slightly less strict (because it allows mixed arrays), but probably easier to work with.

For more discussion see #10620.

Thanks for the explanation! The behavior I posted above is still a bit odd (flow analysis overrides my explicit annotation) but I don't have a problem with it.


is it possible to get the data for type "featured", "playlists", "artists"

Hello,
I am using this API and it's really nice but I did not find the way for getting  type "featured", "playlists", "artists". or may be missed in docs ?

@sumitkolhe . Excellent work. Love the API implementation and really easy-to-understand docs. I'm also searching for playlist details same as album details as on the home page API we get the data of playlist & albums but since there is no API to fetch the details. I'm not able to actually use it completely. Is it possible to add /playlist end point to fetch the playlist details from the link?

@immayurpanchal @r3dm4st3r this will be implemented soon!

@immayurpanchal @r3dm4st3r this commit adds playlists and playlist details.
Docs -
Playlists
Playlist Details
Artists api will be added soon after this.

Hi thanks for the update. I'm eagerly waiting for next update.


[Suggestion] [Pixiv] Download translated tags for pixiv?

Pixiv now can show translated tags on the side of origin Japanese tags.
Is here any option for saving translated tags for filenames/data base/separate log files?
Or using custom file with pairs "japan_tags - my_custom_tag" for replacing tags?

Can you look at translated tags?
Pixiv now sends json data with translated tags:
    "tags": [{             "name": "東方",             "translated_name": "Touhou"         }, {             "name": "爾子田里乃",             "translated_name": "Satono Nishida"         },         ...      }]
"translated_name" could be "null",
i made simple renamer in model.js for myself, but look at official solution (it hard to merge diffs to local model.ts).

It's not a simple fix, as it requires new settings & co, since some users might actually prefer JP tags.
It also impacts other sites that provide multiple languages like Anime-Pictures (currently defaulting to EN).

i put tags in metadata with exif and jap character are converted to "????"

can you share your model.js ? ^^ @uvhat

can you share your model.js ? ^^ @uvhat
@Railline
Here is...

This model.js returns tags as:
translated0 translated1 translated2 original0_translated0 original1_translated1 original2_translated2
grabber replaces every :: and : as _ in tags so it will something like "maid メイド_maid"
if there are no any translated tag it returns null for translated.
Comment some useless console.log that i used to debug some thing if you don't want to see it in log output.
It has also some dirty filtering for useless tags such as "500+users, 9000+bookmarks", it could be done from Grabber, but i made it here too for some reasons i don't know why, you can try to use this too.
model.txt

if i under stand i need to use traslated0 or 1, 2 ore originial10_translated instead "all" ?

it is "all" token. translated0,1,2 is just example.

oh i see thanks you ^^


New install, and getting lots of time outs

So I have bot installed on a linux machine that doesn't have a VPN.  I am getting lots of things like error below.  I tried changing timeout to 60 seconds but it seems to have made it just take longer to correct.  Some messages do get through, but Anything I can do in this case?
"Error during summary notification: The read operation timed out Requesting returnActiveLoans
2017-08-05 12:30:08 Error: The read operation timed out Requesting returnActiveLoans
Timed out, will retry in 60.0sec
...
2017-08-05 12:34:59 Error: The read operation timed out Requesting returnTicker
Timed out, will retry in 60.0sec
".

No, sadly thats normal for Poloniex. Just ignore the Errors.

My timeout is on 90, but it seems that it doesn't change much.


_1password-gui: 8.10.30 -> 8.10.33

_1password-gui-beta: 8.10.30-20.BETA -> 8.10.34-10.BETA
Description of changes
Update to latest release, fixes #312681
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3985

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

2 packages built:

_1password-gui
_1password-gui-beta


Investigate & Update Apache Maven to version 4 when it become available

Describe the task
Update Apache Maven plugins to version 4.0 and its dependencies. Today we have only warnings, but soon there should be errors.
Note: is not possible to update it today, since version 4.0 is not released yet.
Acceptance Criteria

[ ] All maven commands working as expected
[ ] No maven warnings or issues on the console

Additional context

Stackoverflow question and explanation: https://stackoverflow.com/questions/76355897/plugin-validation-issues-were-detected-in-plugins
Apache Maven release notes: https://maven.apache.org/docs/3.9.2/release-notes.html
Jacoco Maven plugin: https://central.sonatype.com/artifact/org.jacoco/jacoco-maven-plugin/0.8.10/versions

Current warnings:
[WARNING]
[WARNING] Plugin validation issues were detected in 2 plugin(s)
[WARNING]
[WARNING]  * org.jacoco:jacoco-maven-plugin:0.8.10
[WARNING]  * org.apache.maven.plugins:maven-compiler-plugin:3.10.1
[WARNING]
[WARNING] For more or less details, use 'maven.plugin.validation' property with one of the values (case insensitive): [BRIEF, DEFAULT, VERBOSE]
[WARNING]

Given this SO answer since Apache Maven 3.9.3 those warnings were fixed and stopped happening. Our current Apache Maven version is 3.9.6, so, nothing to worry about right now.
ricardo-campos@ENCSABCAMLT1288:~/Projects/nr-spar/backend$ ./mvnw --version
Apache Maven 3.9.6 (bc0240f3c744dd6b6ec2920b3cd08dcc295161ae)
Maven home: /home/ricardo-campos/.m2/wrapper/dists/apache-maven-3.9.6-bin/3311e1d4/apache-maven-3.9.6
Java version: 17.0.8, vendor: Oracle Corporation, runtime: /home/ricardo-campos/.sdkman/candidates/java/17.0.8-graal
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "6.5.0-14-generic", arch: "amd64", family: "unix"

However, I think we should update out docker compose file to enable a verbose validation for the maven plugins, since we need them.


Add api to consume using smartmodule by name

Hi @morenol! Just to be clear, this means having a predefined smart module using the smartmoduleName option and also adding support for params, right?

@EstebanBorai right it means using it by name. Supporting params could be done in a separate issue if that is not implemented yet.
not sure if this was already implemented in https://github.com/infinyon/fluvio-client-node/pull/256 . I see it in the docs: https://infinyon.github.io/fluvio-client-node/interfaces/ConsumerConfig.html#smartmoduleName
Can you confirm and close this if that is the case?

Perfect! Will look into this!


Add licence

Project needs an open source licence.

I believe the MIT licence is best for this project. See https://choosealicense.com/licenses/mit/


How Can I add Custome Fonts when User Add Text Over Image?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

mPhotoEditor.addText(mTypeface,inputText, colorCode);


How to check support for assembly features/directives?

Assembler Directives
Assuming that it makes sense to have these HAVE_AS_FUNC, HAVE_AS_ARCHEXT_DOTPROD_DIRECTIVE etc. in libbuild2-autoconf, what would be a reasonable way to check it? AFAICT there is no good way except for test-compiling.
See example here.
Related: #65

What would be a reasonable way to check it?

I don't know the details, but seems like compiler name/version (and maybe architecture, if it's architecture-specific, though I doubt it). I assume both GCC and Clang support this since some version and it doesn't apply to MSVC.

AFAICT there is no good way except for test-compiling.

There are no plans to support compile/link probing so for any given test we either need to find a way to do it using other mechanisms (without all the drawbacks of probing) or it doesn't belong in libbuild2-autoconf.


How to customize the parameter description display for SWAGGER generator and UI calls?

DTO:
Public class OrganizationResult
{
//id, code, name
}
API:
[HttpPost]
Public ResultGet3 (int? Id, string? code, OrganizationResult org)
{}
Question 1: Regarding the above interface, I want the Swagger UI to display parameter descriptions in the body as
{
id: int,
"code": "string",
'org':{
"id": "long",
"code": "string",
"name":"string"
}
}
like this:

Question 2: When calling the interface in the Swagger UI, the parameters can be placed in the body according to the above structure for requests
It would be best to have a unified configuration without adding feature tags on each interface or parameter. My name is. NET CORE 6.0

It's already resolved


[FLINK-11721][network] Remove IOMode from NetworkEnvironment

What is the purpose of the change
The NetworkEnvironment would be refactored into NetworkShuffleService created by ShuffleManager future. So the NetworkEnvironment should only cover shuffle related components to make the preparation.
Currently IOMode is only used for getting in tests, so it is not a necessary component in NetworkEnvironment to be removed directly.
Brief change log

Remove IOMode from NetworkEnvironment

Verifying this change
This change is already covered by existing tests, such as NetworkEnvironmentTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot approve all

@azagrebin Thanks for your reviews!
To be honest, I am not sure the history consideration of bringing IOMode here. I might trace the history commit messages to further check. :)


Is there a way to enforce an array even when the claim is only one?

I saw a similar request for IdentityServer3 (https://github.com/IdentityServer/IdentityServer3/issues/607) and don't know if any mods have been done in ID4 to accommodate this, but it would be very nice to be able to enforce an array even if you have one element for a certain claim ...
for example
{
"FirstName":"John",
"LastName":"Doe",
"Role":["Admin"]
}
In the above, the Role claim could have one or multiple values. And the problem is that the schema changes depending on if you have one role or multiple roles.
If you have one role, you get a string value - "Role":"Admin"
and if you have multiple roles, you get an array of values - "Role":["Admin","PowerUser","ReadOnlyUser"]
This causes deserialization complexities as the client will now have to maintain multiple models.
Is there a workaround for this already somewhere ?

I guess you could generate the JSON array yourself and set the claim using a ClaimValueType of Json.
https://github.com/IdentityServer/IdentityServer4/blob/dev/src/IdentityServer4/IdentityServerConstants.cs#L31

Closing. If you still have issues, feel free to reopen.

I guess you could generate the JSON array yourself and set the claim using a ClaimValueType of Json.
https://github.com/IdentityServer/IdentityServer4/blob/dev/src/IdentityServer4/IdentityServerConstants.cs#L31

This url no longer work
Does the Json ClaimValueType still exist ? I can't find it in 2.3.2


Error when running app in docker

Describe the bug
When I try to run the app in docker I get the following error:
Error in fread(file.path(dataDir, "df_timeseries.tsv"), stringsAsFactors = FALSE,  :
  File '~/git/alien-species-portal/data/df_timeseries.tsv' does not exist or is non-readable. getwd()=='/usr/local/lib/R/site-library/alienSpecies/app'
Calls: <Anonymous> ... eval -> ..stacktraceon.. -> createTimeseries -> fread
Execution halted

To Reproduce
Steps to reproduce the behavior:

open github desktop
select alien-species-portal repo
pull & fetch mockup branch
start docker
Open powershell, type & run:  cd documents/github/alien-species-portal
type & run: docker build -t inbo/alienspecies .
type & run: docker run -it -p 3000:3838 inbo/alienspecies:latest
See error

Expected behavior
Alien species portal gets build succesfully.
Additional context
I'm building hash e9065336ccaa90f35ba137770d23f40374245f51.
I'm using docker desktop 4.20.0.

@SanderDevisscher For me the app starts, same GIT hash... It seems to create a missing file, which should already be there.
Could you please open an R session in docker
docker run -it -p 3000:3838 inbo/alienspecies:latest R

and list the available files in R?
> list.files(system.file("extdata", package = "alienSpecies"))

these are the files listed:
[1] "be_alientaxa_cube.csv"               "be_alientaxa_info.csv"
 [3] "communes.geojson"                    "data_input_checklist_indicators.tsv"
 [5] "dfCube.RData"                        "eu_concern_species.tsv"
 [7] "grid"                                "keys.csv"
 [9] "management"                          "provinces.geojson"
[11] "readme.md"                           "translations_v22-12-11.xlsx"
[13] "translations_v22-12-20.xlsx"         "translations_v23-05-24.xlsx"
[15] "translations.csv"

I see, some are not tracked by GIT. Let me fix this.

@SanderDevisscher It's a file of size 600Mb, therefore it was not tracked. In the future it's no problem if we put it in an S3 bucket. What do we do now with it? I could share it once via a cloud instance with you, unless someone else needs it too.

@mvarewyck, sure, I can set up the access for the UAT-bucket for the alien-species-portal. To do this, I'll need the ARN (Amazon Resource Name) of your own IAM user within the Open Analytics account. We'll grant access to this specific IAM user, ensuring that it's external to our organization and in line with our security protocols.
Please provide the IAM user ARN via mail, and I'll create the necessary bucket policy to allow you to upload files into the bucket.

@mvarewyck thanks for the e-mail. We can switch to mail due to the fact that we will start sending sensitive information.

@TheJenne18 this issue concerns an error when building the app using docker. which has not been fixed yet!

@mvarewyck when running createTimeseries I get the following error, which I'm unable to fix:
Error in createTimeseries(dataDir = "./data/interim/") :
argument "shapeData" is missing, with no default

Where can I find the shapeData?
PS: data interim is the local map I use

Where can I find the shapeData?

createTimeseries(... , shapeData = readShapeData()$utm1_bel_with_regions)

het aanmaken van full_timeseries.csv is gelukt.

ik krijg een nieuwe error:
#0 6.844 2023-09-01 11:43:30 (4.06 MB/s) - ‘phantomjs-2.1.1-linux-x86_64.tar.bz2’ saved [23415665/23415665]
#0 6.844
#0 6.850 tar (child): lbzip2: Cannot exec: No such file or directory
#0 6.850 tar (child): Error is not recoverable: exiting now
#0 6.850 /usr/bin/tar: Child returned status 2
#0 6.850 /usr/bin/tar: Error is not recoverable: exiting now
#0 6.858 Error in webshot::install_phantomjs() :
#0 6.858   Unable to install PhantomJS to any of these dirs: ~/bin, /usr/local/lib/R/site-library/webshot/PhantomJS
#0 6.858 In addition: Warning message:
#0 6.858 In utils::untar(zipfile) :
#0 6.858   ‘/usr/bin/tar -xf 'phantomjs-2.1.1-linux-x86_64.tar.bz2'’ returned error code 2
#0 6.858 Execution halted
------
Dockerfile:211
--------------------
 209 |     # To download leaflet maps from within the app
 210 |     ## Attention: do not install phantomjs directly, will not work then!
 211 | >>> RUN R -e "webshot::install_phantomjs()"
 212 |
 213 |     # install local package(s)
--------------------
ERROR: failed to solve: process "/bin/sh -c R -e \"webshot::install_phantomjs()\"" did not complete successfully: exit code: 1

it runs smoothly now


Roll Engine from 17f6bfc480b7 to 89aea562896b (9 revisions)

https://github.com/flutter/engine/compare/17f6bfc480b7...89aea562896b
2021-03-02<EMAIL_ADDRESS>Roll Dart SDK from b3bc5acdeef2 to f209e60a5415 (2 revisions) (flutter/engine#24722)
2021-03-02<EMAIL_ADDRESS>Roll Skia from 9ca065e2fdb5 to 67808983c677 (6 revisions) (flutter/engine#24720)
2021-03-02<EMAIL_ADDRESS>Roll ICU to match the latest version used by Chromium (flutter/engine#24673)
2021-03-01<EMAIL_ADDRESS>Revert "remove _getRRect: fixes assertion error (#24688)" (flutter/engine#24718)
2021-03-01<EMAIL_ADDRESS>Reland "TextStyle level leadingDistribution (#24025)" reverted in #24665 (flutter/engine#24668)
2021-03-01<EMAIL_ADDRESS>Roll Dart SDK from f7b05f6e0636 to b3bc5acdeef2 (1 revision) (flutter/engine#24715)
2021-03-01<EMAIL_ADDRESS>Roll Skia from 135500d33dc7 to 9ca065e2fdb5 (6 revisions) (flutter/engine#24714)
2021-03-01<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 1jbCxKA9Y... to 50gZrG82q... (flutter/engine#24709)
2021-03-01<EMAIL_ADDRESS>remove _getRRect: fixes assertion error (flutter/engine#24688)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 1jbCxKA9Y9PH to 50gZrG82qReN
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


Move mockredis to test requirements.

This PR replaces #115 which moves the mockredis requirement to a test requirement from an install requirement.
Signed-off-by: Ken Cochrane<EMAIL_ADDRESS>

Coverage decreased (-0.05%) to 91.379% when pulling 054d9433290a6495956c10a0a16296861b2d4ca1 on mock_redis into 104eda3d0395e17b215f3c4cc65af1b1c9632adc on master.


[occm] Introduce Hostname Annotation

What this PR does / why we need it:
This patch makes it possible to explicitly configure the hostname of a
load balancer via annotation.
This hostname is used in the load balancer instead of the IP address or the
fake hostname.
Similar to enabie-ingress-hostname, this can be used with proxy protocol to preserve the IP, but this also works for IPv6 load balancer (.nip.io is no valid hostname).
https://github.com/kubernetes/cloud-provider-openstack/blob/master/docs/openstack-cloud-controller-manager/expose-applications-using-loadbalancer-type-service.md#use-proxy-protocol-to-preserve-client-ip
Which issue this PR fixes(if applicable):
fixes #1859
Special notes for reviewers:

Release note:

Add option to set hostname on load balancer via annotation

/ok-to-test

/lgtm

/approve


linux: builds lib/common/core

This enables building other files under lib/common/core.
Disabled some features for non-Windows:

Stack back trace (depending on stack and dbg symbol APIs).
ConfigParser: read Windows Registry
SysInfo: differentiate physical/logical #processors, read VERSION
resource

Hi @jianchun, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jianchun Xu). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Opened #200 to track physical/logical cores issue.


Add HelpUri to functions

Describe the solution you'd like
Since we have all the documentation as markdown we should add HelpUri for each function which would point to the markdown file for that function. In that way its possible to use the -Online parameter for Get-Help.
Example:
function Build-Bicep {
    [CmdletBinding(DefaultParameterSetName = 'Default',
        SupportsShouldProcess,
        HelpUri = 'https://github.com/StefanIvemo/BicepPowerShell/blob/v1.4.2/Docs/Help/Build-Bicep.md')]
    ...
}

However, this should be release-specific, so we should not add a generic Url. It should be for the version installed (as in the example above).

I'll add a script to automatically do this and then we can re-use it for a GitHub action.


Update docs for tabBarOnPress

This reflects the new state of the world as of<EMAIL_ADDRESS>

sorry I missed this one! this is resolved in #175 now


Close button is too close to the Status Bar on iPhoneX

fixed in this
https://github.com/antonKalinin/react-native-image-view/pull/53

same issue

same issue

I have the same issue and it would be nice to get https://github.com/antonKalinin/react-native-image-view/pull/53 merged.


AcquisitionFuture.await_image_saved does not return image and metadata

Using the command:
img,metadata = future.await_image_saved(event['axes'], return_image = True, return_metadata=True)

Expected behavior is that both the image and metadata are returned.
Actual behavior:
    img,metadata = future.await_image_saved(event['axes'], return_image = True, return_metadata=True)
    ^^^^^^^^^^^^
ValueError: too many values to unpack (expected 2)

Only the image data is being returned because line 152 from the code below never gets called (the function returns on one of the previous if statements).
https://github.com/micro-manager/pycro-manager/blob/eb467b49c9fc2f1fca1006083546c5c1358025bf/pycromanager/acquisition/acq_future.py#L142-L156

Fixed in #800


Is it possible to make 9*9 chess board?

I think it may be possible, but just checking if you are aware that this is simply a board and it doesn't handle any logic. Do you have access to an engine that can handle the chess logic for such a board?

Closing as stale


App workflow architecture thought-dump

I am unsure and undecided about what a good workflow architecture would be for this tool.
On the one end we have a DataLad dataset, and on the other end we have a web application that allows users to view and browse through the dataset's metadata. And then there's everything that needs to happen in between.
Here's my current thinking: the idea is that we generate a static browser-based application from DataLad metadata. Say we structure this as a DataLad function/tool where we run something like: datalad generate-web-ui <path-to-dataset>.
The following will then need to happen:

MetaLad to extract and aggregate metadata for the dataset, subdatasets and files, all to the top level dataset. This needs to be either saved to the dataset's metadata store, or exported/saved in some way that makes it accessible to the tool.
The tool ingests the dataset's metadata, and:

Applies some logic to the data (restrict amount / levels, select subset for browsing, TBD)
Restructures data into a format that the web-ui understands
Creates the web-ui building blocks from templates (html, js, css)
Saves the complete repo that allows running the web application (locally)

The user can host this repository on their preferred server or publish it e.g. via GitHub pages.

Uncertainties/questions/decisions:

The main uncertainty for me is how to make the metadata available to the static app and how to traverse the metadata tree, mainly because the amount of data that needs to be readily available could be arbitrarily large.

For the current test case, all metadata for a dataset is contained in a single JS object that's fetched via http request from a local file. Subdatasets, directories, and files are listed as object "children" of specific types. Having all metadata in a single file is probably not smart for datasets with large amounts of sub-data.
We could have a separate file per subdataset? And we could exclude unnecessary/redundant file information explicitly during Step 2 above?
We could fully recreate the dataset directory tree by having a metadatafile per node, containing the path, type and children of the node. These are then only fetched/loaded upon user selection.
If we move away from using local files, we will need to fetch this data from somewhere else. If this is a server, we don't have a static app anymore. Is this a problem?

Some assumed truths to guide the process (subject to change):

The web app should be static.
The only requirements for running the app should be a modern browser and internet access (in case of using a CDN for libraries).
There is one source of metadata and that is the dataset itself

Some absolute truths for context:

Stephan is not a computer scientist or experienced developer

Some post-discussion notes:

The dataset and its metadata are separate from the web-ui generator, which is separate from the web-ui.
The dataset creator could have added extensive metadata, or datalad/metalad could have extracted minimal metadata from the dataset. In either case (or for anything in between), the web-ui generator should be able to operate on the existing metadata.
The generator takes whatever metadata is given to it and transform it into some form of structured data that the web-ui can understand and query.
The generator also creates the basic required web-ui components from templates.
The generator should have to be tied to or restricted by whichever structure/format the metadata is available in. Ideally the generator would filter through the metadata provided to it, identify aspects of the data that the web-ui will be able to represent, and create the structured data interpretable by the web-ui.
The web-ui can ideally receive a url parameter with which to query the structured metadata.
It is important to take both the id and version of a dataset and file content into account when accessing and displaying them.
The way in which structured metadata is made available to the web-ui could be (for now) by having json blobs (e.g. 1 per dataset/file) available at a specified url+relative path, where the blob name is identifiable from the dataset/file id + version.
A solution has to be thought of to cater for cases where a dataset has a large amount of subdatasets or files, all of which one would not necessarily want to render in the web-ui. This could already be approached during the generator stage of the workflow.
Server-side implementations/databases should ideally be avoided, but could be considered if necessary.

Some ideas:

Dataset listing as main page of web-ui, from where datasets can be filtered / searched / selected. Selecting a dataset then opens the dataset page, using the dataset id+version as query parameter.
List of subdatasets on dataset page (perhaps as cards with minimal content). This is arguably more important than the file browser (acknowledging that the current file browser also lists subdatasets). Need to take possibly LONG list of subdatasets into acount (same as with files).

From metalad meeting:

Dataset browser should be able to generically work on hierarchical structures, should be (in some way) configurable to display specific metadata in specific ways, there should also be a fallback representation for metadata.
User interface generator receives a single superdataset as input.
Metadata in any format could be added to the superdataset (directly or aggregated from subdatasets) - the generator should ideally be able to map known formats to the structure that the data browser renders from.
The generator should be able to receive new mappings (i.e. be able to parse new metadata formats for the UI to render) without changes to code.
Tool should ideally be callable as CLI, e.g. piped together with datalad commands, so that the JSON lines output from datalad meta-dump can be fed directly into the generator tool.

Closing since nothing needs to be addressed here.


feat(RingTheory/DualNumber): dual numbers over a field are local and PIR

I have an almost-ready proof of R[ε] ≃ₐ[R] (R[X] ⧸ (.span {(X ^ 2)} : Ideal R[X])). Would you like me to include it in this PR or separately?

Let's keep that one separate

I pushed a few golfs, please check you're happy with them.
bors d+
Thanks!

bors r+


export the version command

The other cobra commands are exported. This would just export Version as well, and just helps throw together Preflight + Chart-Verifier PoCs.
Signed-off-by: Jose R. Gonzalez<EMAIL_ADDRESS>

@mmulholla could you do a quick check of this PR?


Requested custom icon is returning randomly in production site

This bug is so random and it's only happen on our production site. Sometimes it happen, sometimes not.
But you can see from the screenshot below that the request to times icon is returning 2 other icons. It return the error message: [Icon] failed to load icon m-icon:times
Versions:
"@nuxt/icon": "1.9.1",
"@nuxt/ui": "2.18.4",
"nuxt": "3.13.0"

Could you provide a minimal reproduction? Thanks.

Why reproduction is required


Final group capstone: set up your Kanban board

Dear reviewer, below is the link to our GitHub project
https://github.com/users/flemton/projects/2
Our group consisted of 3 members only and that's @sergio-usma, @SFarida and me (@flemton )
Screenshot

Wow, you did it 🎉

Your project is complete!  :shipit:
Congratulations! 🎉
To highlight

All cards have descriptions, assignees, and pointings according to spec✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add support for PyPI packaging. Fixes #59

Hello!
This PR adds support for PyPI packaging when using the command python setup.py sdist bdist_wheel.
The purpose is to make easier for developers to create a package of their own and for the openfhe org to be able to publish packages easily.
I have created a setup.py that compiles the library with CMake and then creates the sdist tar.gz and bdist_wheel whl packages with the library inside, allowing for easy installation with pip install and integration with import openfhe.
There are some caveats to the approach I followed. Since build_ext executes at the end of the setup process, it was creating empty packages and then compiling the library. To work around that I created a custom sdist class that forces a build_ext before all the other commands.
That introduces the problem of double compilation, which is done in the bdist_wheel phase. To prevent that, I created the (admittedly not very elegant) solution of passing the compilation if the .so file already exists. That's also why it gets removed during the sdist phase of the build.
There also isn't any docstring for now, which complicated knowing which functions are available and what they do, forcing you to rely on the documentation.
If you have any feedback or a better approach to the code, please let me know!
Thank you.

Sorry I just realized I was trying to push to main instead of dev. Fixed now.

Adding @mtriplett-duality to this PR

Hi @Thadah
Thanks so much for the work here! I've run this locally, and everything seems to work perfectly. Unless you have some other modifications to add, we're comfortable merging this in. Just give us the thumbs up!

I found a way of making the pybind11 stubs show in most IDEs, but automating the process is a bit convoluted. It would require to create a wheel package, then installing it in a development (or virtual) environment, use pybind11-stubgen to generate the .pyi files, and generate the package again with the automatically generated pyi files. A bash script might be the best option.
Another way would be to create the .pyi files manually and place them in the openfhe/ directory so that they get packaged with the rest of the files. This would require periodic maintenance as they wouldn't be updated automatically and new functions or features would be lost to the IDE.
What approach would you prefer?

I believe there's nothing else to add for now on my part, so I have no issue for it to be merged.
A CI/CD pipeline surely makes a lot more sense than doing workarounds in a setup.py file, but I wanted to provide a stopgap solution in the meantime. Thank you for all the feedback @mtriplett-duality!


Lehigh tweaks

Purpose: Fixes to autonomous. No other changes were made.
Files changed

SequentialAutoCommand.java: Changed auto scoring timings.

When is this going to be merged


To add working steps in Largest cone that can be inscribed within a cube

Is your feature request related to a problem? Please describe.
To add working steps in Largest cone that can be inscribed within a cube
Describe the solution you'd like

A clear and concise description of what you want to happen.

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018


Update github-dark.css

What's the issue:

What's fixed:

Screenshots:

#24 isn't the right pull request for #27.


Update ansible.md

Proposed changes

Unreleased project version (optional)

Related issues (optional)

Thanks, @scjane !


Replace set_figure_params(transparent=True) with set_figure_params(facecolor=(0,0,0,0))

As detailed in https://github.com/theislab/scanpy/issues/473#issuecomment-462694429

Due to matplotlib deprecation from here: https://github.com/matplotlib/matplotlib/pull/9698 (have we just been missing deprecation warnings for this?)

Due to matplotlib deprecation from here: https://github.com/matplotlib/matplotlib/pull/9698 (have we just been missing deprecation warnings for this?)


chore: release 1.2.1

:robot: I have created a release *beep* *boop*
1.2.1 (2022-02-14)
Bug Fixes

remove exponential back-off for transactions (5362e61)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/grbac/grbac/releases/tag/v1.2.1 :sunflower:


[WIP] query plan viz for Sequence and DoWhile

Make query plan visualization support Sequence and DoWhile.

[x] refractor graph.js to support Sequence and DoWhile
[ ] specific viz for Sequence and DoWhile

I'm closing this for lack of activity


Starting Deployment then immediately Failed to Feploy.

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

Describe the bug
Hi, I have been using the continuous deployment setup. With a simple git push, my project has always been deployed successfully. However, my three latest push seems to failed without giving any clue to fix it. The three deployment always show the following message in the console:
2021-04-29T12:32:10 [INFO]: Starting Deployment
2021-04-29T12:32:10 [ERROR]: Failed to deploy
Please advice.
Expected behavior
I look at successful build history. On the deploy stage the message should have shown:
2021-04-27T12:54:22 [INFO]: Beginning deployment for application d1tm708pfdcz9c, branch:dev, buildId<PHONE_NUMBER>
2021-04-27T12:54:22 [INFO]: Got archive: 3420216 bytes
2021-04-27T12:54:22 [INFO]: Archive unzipped: 96 files
2021-04-27T12:54:23 [INFO]: Deployment complete
Reproduction steps
git push
Build Settings
No response
Additional information
No response

Was experiencing this same issue and was able to find a fix. I discovered I was using the latest version Amplify CLI in my build settings, which must have recently introduced a breaking change.
To fix:
In the Amplify console, go to Build settings > Build image settings > Edit
Specify the version of Amplify CLI that's in your project's package.json


Correctly tracking inventory changes to the cached inventory

based upon  @BriceSD work with #3860 cleaned up to be able to be merged since the original branch/pull request was very messy.
Description:
With the new cached inventory becoming a core component, we can’t afford to not correctly update it after every change.
I’ve made a class (RecycleWorker) which is more or less an api wrapper.
At first I put in it the responsibility to update the inventory. While not being bad, i feel like it’s easy to misunderstand what item.remove(amount) do. Namely removing the amount from the cached inventory without making a recycling request to the server.
What I’m proposing now to recycle an item is to use item.recycle(amount). Inside this method we’re doing a call to the api and then, depending on the success of the recycle request, we’re updating the cached inventory. By doing this we’ll never change the quantity of an item from outside the class. Thus we could allow only access to Item’s attribute and deny any change from the outside. This would make the cached inventory trustable.
However this could result in a very coupled code.
Having a class that adds a level of abstraction (RecycleWorker here) seems good enough though. The item know nothing about the recycling process. Not even how is defined a succeeding call.
def recycle(self, amount_to_recycle):
    """
    Recycle (discard) the specified amount of item from the item inventory.
    It is making a call to the server to request a recycling as well as updating the cached inventory.
    :param amount_to_recycle: The amount to recycle.
    :type amount_to_recycle: int
    :return: Returns wether or not the task went well
    :rtype: worker_result.WorkerResult
    """
    if self.count < amount_to_recycle:
        raise Exception('Tried to remove more {} than you have'.format(self.name))

    item_recycler = ItemRecycler(_inventory.bot, self, amount_to_recycle)
    item_recycler_work_result = item_recycler.work()

    if item_recycler.is_recycling_success():
        self.remove(amount_to_recycle)
    return item_recycler_work_result

I haven’t work much with the the pokemon inventory, or the pokemon catcher either. Not sure if throwing a ball during an encounter from the inventory is a good idea.
Any thoughts on the subject would be highly appreciate.
@amal, @douglascamata, @TheSavior, @aeckert, @BriceSD

@duttonw we need add your instruction into the document.


Errors trying to install SmartApp

Neither the "lite" nor "full" apps install via the Smartthings "Graph" IDE.  Both receive the following error:
Updated 0 and created 0 SmartApps, 0 published (1 skipped due to errors)

Nevermind this one.  I figured out what I was doing wrong.


Default setting for GDNativeLibrary in Project Settings doesn't exist anymore?

Hello,
In 3.0 alpha2, I can't find anymore the project setting dlscript/default_dllibrary. There is now a tab "GDNative", but there is no button associated with it. How can I now put a default library?
Thanks for your work.

Hi @lstep ,
This is more of a Godot editor issue than a godot-go issue. I'm not sure how they've changed that option. In my testing, I usually create a libgodot.tres file which points to the .so object that is built.

Yes that's what I also did, and you are totally right, that is a Godot issue, but when someone follows your instructions (in the README.md file) he get stuck because of this and can get frustrated. There are so few places where GDNative is documented that it's exceptionally complex (IMHO) to find how to correct this.


not an issue

POST HOW TO USE THIS BRAND NEW RAT
Thank you <3

FEW FIXES AND UPDATES ARE PENDING. SOON WILL BE RELEASING APK xhunter.apk AND "How to use guide".
Main feature  includes

ability to retrieve WhatsApp messages
it use localtunnel proxy to establish connection between attacker and victim [hack over the internet]
it also have binder feature to bind payload to other apps
4.etc...[all common Rat features]


Missing translation tag in e-mails

Issue Summary
When using Oscar in another language, some texts in the e-mails are not translated.
Steps to Reproduce

Configure oscar in any other language than English
Make oscar send the reset password email
You can see that in the email the CTA is still "Reset Password"

Technical details
Source of the issue:
https://github.com/django-oscar/django-oscar/blob/master/src/oscar/templates/oscar/communication/emails/commtype_password_changed_body.html#L20
This code is missing a "trans" tag.
Same issue in a few other places:
https://github.com/django-oscar/django-oscar/blob/master/src/oscar/templates/oscar/communication/emails/commtype_email_changed_body.html#L20
https://github.com/django-oscar/django-oscar/blob/1442cb734ec542459a31cbd9c174bf8817dcfeea/src/oscar/templates/oscar/communication/emails/commtype_password_reset_body.html#L20

Thanks - PR to fix this would be appreciated if you have time.

Fixed in https://github.com/django-oscar/django-oscar/commit/04cd6a4fc750db9310147d96776f06fe289269bf


Final Upload

This is a final Upload to this code, this will be transferred to another branch for future use


Component does not update its parent's state

When I update the state of a parent component, I expect it to reflect these changes. However, unless I update something in the component itself, the changes are not propagated back to the client.
Sample setup
There are two components: a wrapper with a counter and a button that updates the counter. When I click the button, I expect the counter to be updated. However, the server returns a 304 status code and an empty response, and the UI is not updated.

Wrapper Python: link
Wrapper HTML: link
Button Python: link
Button HTML: link

More information

Demo repository: link
Video demonstrating the issue: link

There is a feature in Unicorn which sends back a 304 if the rendered content stays the same to reduce network bandwidth. This issue presented a situation where a child component didn't change, but the parent did -- with the PR #595 the parent component will re-render as expected.

Thank you for the fix! I can confirm that the PR #595 fixes the issue 👍

Demo repository: link
Modify this demo to django-unicorn==0.58, can still reproduce the error of value not changing.


Add Modifier Keys

In Alfred, you could add a modifier for eg: ctrl + enter to copy text to the clipboard.  How do you do this in Hain?

There is no way to find out modifier keys in plugins for now. it can be useful. but I think it's not good UX. I think Button API can cover most use-cases.

Why do think the modifier keys does not provide good UX? Are there any examples or plugins that are using the Button API?

Currently, there is no Button API yet. but you can see here as an example.
and we don't have a place to show how to use modifier keys for users, so i think this isn't good UX.

This has been implemented already in the latest development branch, see #189


Multi artifact publishing

We want to be able to publish a separate artifact for each connector. This will limit imports to only the necessary dependencies.
(we can use https://zio.github.io/zio-aws/docs/overview/overview_artifacts as a model)

https://github.com/zio/zio-connect/pull/64


show mnemonic tool for bitbox and others

https://github.com/digitalbitbox/bitbox02-firmware/blob/master/py/bitbox02/bitbox02/bitbox02/bitbox02.py#L233
Right now there's no method of getting your seed phrase newly generated off a bitbox02. Would be nice to have.

actually "backup" is exactly that. Closing.


Combine identity print and identity export into `…

Draft Fix #3043
Currently, combines print and export artificially, renames export and print internal functions to long and short as I'm not able to merge models::identity::Long/ShortIdentityResponse(export/print) and api::long/short_identity(export/print).
So when changing api::long_identity(export) to Request::builder(Method::Post, "/node/identity/actions/show/full").encode(&mut buf)?; it fails at parse_long_identity_response()(export) dec.decode::<models::identity::LongIdentityResponse>()? with
thread 'tokio-runtime-worker' panicked at 'decode parseLIR: Error { err: TypeMismatch(String), pos: Some(19), msg: "expe cted map" }', implementations/rust/ockam/ockam_command/src/util/api.rs:440:64
Probably happens due to ockam_core::type_tag type check in LongIdentityResponse(export).
I feel like I'm doing something wrong.
Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository.

@anuvratsingh thank you for sending this.
I'll checkout the code and leave some comments soon.

@anuvratsingh if you want to update path in api::long/short_identity you should also do so in NodeManager::handle_request. Would be great if you could do that in follow up PR. For now this is good to land

Moved to #3083


Add dns plugin

This commit adds support the dns plugin as documented at
https://collectd.org/documentation/manpages/collectd.conf.5.shtml#plugin_dns

hey @ghoneycutt could you please take a look at the failed check? Thanks for the contribution so far!

@bastelfreak I can't see any of the checks actually failing, just seeing a lot of style issues with the spec tests. Do I need to address the style related to the 34 offenses detected or is there something specific that is failing?

Interested in ensuring my commits are acceptable as I've got a few more PR's on the way.

No direct failures, and the style checks are pretty new for us. I can't really guess how time consuming it would be to fix them and how strict we care about them. Rule of thumb is: less warnings are always better.

Hey @ghoneycutt, thanks for the PR! @bastelfreak and I were just discussing this on IRC. I think we'd like to make sure the style is consistent in tests. It just makes it easier to contribute, especially for people who are new to ruby. It seems that rubocop -a will fix all of these for you.

Which network/channel ?

we're in #puppet-community!

on freenode! forgot that part ;)!

:+1:


[feature] Add Logcat output into debugger

I would like the Jadx debugger to include the logcat output within the Jadx GUI.  Ive been building the feature and have it working for my purposes but not quite ready to include into production.  Creating this issue to get others opinion.
Currently can be found at https://github.com/green9317/jadx/tree/logcatAddition

Before I submit I still want to add

ability to filter to debugged process
automatically update process list
button to clear logcat
button to pause logcat (for performance sensitive users)
Change how logcat output is displayed to make it easier to read
clean code

Ill do that.  I wasn't sure if not having the latest commit would cause an issue when I do a pull request.


Explicitly depend on racc to fix specs for Ruby 3.3+

On Ruby 3.3.0-preview3, knapsack's own specs fail:
$ ruby -v
ruby 3.3.0dev (2023-11-12 master 60e19a0b5f) [x86_64-darwin23]

$ bundle exec rspec spec
An error occurred while loading spec_helper. - Did you mean?
                    rspec ./spec/spec_helper.rb

Failure/Error: require 'spinach'

LoadError:
  cannot load such file -- racc/parser.rb

This is because in Ruby 3.3, the racc gem is no longer a default gem. This change is mentioned in the Ruby 3.3.0-preview2 release notes:

The following default gem is now bundled.
racc 1.7.1

Since knapsack's specs rely on the racc gem being present, this means that in Ruby 3.3 racc must be explicitly declared as a development dependency in the Gemfile.
This commit adds the racc dependency, conditionally based on the Ruby version. With this change, the specs now pass.
$ ruby -v
ruby 3.3.0dev (2023-11-12 master 60e19a0b5f) [x86_64-darwin23]

$ bundle exec rspec spec
...
Finished in 1.91 seconds (files took 0.42586 seconds to load)
210 examples, 0 failures

Thanks for reporting this and proposing a fix.
Why not always require racc in gemspec, no matter the Ruby version? Would this break anything for older Ruby versions?
# knapsack.gemspec
spec.add_development_dependency 'racc', '>= 1.7.3'

I added this PR to our internal backlog to be prioritized.

I was worried that the racc dependency would fail under the old versions in the CI matrix (2.3). I'll move it to the gemspec to see what happens.

I tested using the ruby:2.3 Docker image and it looks like the gemspec approach works. We'll see for sure when the CI workflow is run.

I approved the Github Actions workflows for your PR.

@ArturT thanks! Tests are passing for all Ruby versions in CI.

I'm trying to understand how urgent this PR is. Is the issue limited only to running knapsack tests locally with Ruby 3.3+?
Or have you experienced issues running your project's tests on your CI using Ruby 3.3+ and knapsack 4.0.0?

This is not urgent at all. It is only applicable to running knapsack tests locally.

We officially archived Knapsack, you can read more about it here.
Thank you for caring and creating this PR.
I'm sorry we couldn't get this done for you.


Review the need for unset_propagated_clocks

Description
A number of files use unset_propagated_clocks, which are then propagated to SDC files, which are then written and re-read.
Proposal
No response

Fixed in https://github.com/The-OpenROAD-Project/OpenLane/pull/1908


Adding submodule ref for xdf-tagger utility

Adding a path Utilities/xdf-tagger that references the recently-contributed github.com/xdf-modules/xdf-tagger repo. Later we can add other utilities under Utilities.

Sorry my fixup for the Julia submodule modified .gitmodules, creating the conflict. I'll fix that up now.


SOLR-16789: Admin UI Dashboard shows "unauthenticated" warning on authenticated servers

I'm not sure this is the right fix, I don't trust my UI skills. I believe this was introduced in https://github.com/apache/solr/pull/1296

Ran these steps:
gw clean  -p solr/packaging dev && cd  solr/packaging/build/dev
bin/solr start -e cloud -noprompt
Checked that the warning was present in the dashboard
bin/solr auth enable -credentials foo:bar -z localhost:9983
On main, the warning is still present. With the patch I get:


Create a DCLabelDecoration

label := DCLabelDecoration new
	text: 'Invasion of Ukraine';
	x: 'February 24,2022';
	y: 100000000;
	yourself.```

fixed


Support Log4Net

We are going to add integration with Apache log4net in one of our next release.

log4net support was added in https://github.com/airbrake/sharpbrake/pull/68


Adding init options to datastore constructor (external session factory + custom record models)

Hello!,
We are testing the new version of eventsourcing library and it looks great! Thank you very much.
We are currently using version 8.x of the library with SQLAlchemy/Oracle and we are considering the migration to 9.x. Our main motivation is to be able to use sequences / autoincrement in the notification_id column because we have concurrency issues with the select/max query.
In the migration we have had to do some changes to this module. We need a custom factory with a datastore that could be able to:

Use an external SQLAlchemy session (in our app the session is created outside the scope of eventsourcing library)
Use custom SQLAlchemy record models (in Oracle we need to replace some column definitions)

This pull request adds these changes. We think that would make the datastore more flexible to use.
Best regards,
-Jose.

Thank you so much! Looks good to me :)


Recognize Github Flavored Markdown

See https://github.com/lsegal/yard/issues/404 for details.

This may be helpful http://blog.nikosd.com/2011/11/github-flavored-markdown-in-yard.html

fixed and deployed, thanks!


test: transform modules in karma so sinon can stub properly

This fixes all the failing tests when running using Karma.
https://github.com/sinonjs/sinon/issues/1711

Coverage remained the same at 84.285% when pulling 7d06ec5bb12583042dc5bb1eca8017fc2dd3f273 on jrparish:bugfix/karma-tests into 28ce8319f5ee90cb8d4dec7aee8bc87fb393140d on airbnb:master.


Separating Requests from Responses

I have yet to get my head fully around the Requests library, so this is a bit of a shot in the dark but I wonder if there is a way to more cleanly separate the Request from the Response, so that:

a Request could be issued
optionally, at a time of the user code's choosing, it could non-destructively peek ahead to see if there is any returned data yet on the socket or just do some other quick processing before proceeding to the Response
then at its leisure, it could read the status code and reason, response headers, and response content

This could potentially significantly reduce the blocking duration of a Request. Ideally, there is no change to existing user code, which typically issues the Request and immediately reads the Response. For those who want to take advantage of the separation, there could be some check for data between Request and Response, or at least an opportunity to use the wait time for some other operation.
One of the issues I see in this current library is that retries in Request require some reading of the Response. But I don't see that CPython does automatic retries (or reads ahead into the Response as part of the Request) - there's a lower-level class that allows a retries parameter which defaults to zero: class requests.adapters.HTTPAdapter(pool_connections=10, pool_maxsize=10, max_retries=0, pool_block=False. The explanatory text for those retries is:

max_retries – The maximum number of retries each connection should attempt. Note, this applies only to failed DNS lookups, socket connections and connection timeouts, never to requests where data has made it to the server. By default, Requests does not retry failed connections. If you need granular control over the conditions under which we retry a request, import urllib3’s Retry class and pass that instead.

This comment seems to imply a clean separation of Request and Response:

a Response object is generated once Requests gets a response back from the server

This all may become moot if we get some kind of pre-emptive async.

Bumping this for visibility in light of pending asyncio feature. My understanding is that asyncio is cooperative rather than pre-emptive, so Requests would need to be restructured to take advantage of it.

I don't think we'd change requests. Instead, we'd provide a second async library, just like aiohttp in CPython. https://www.twilio.com/blog/asynchronous-http-requests-in-python-with-aiohttp

Closing in favor of a separate future async-friendly library.


Suite2pSegmentationInterface sampling rate not changing.

Hello!
I am building on the example code from the docs to add my suite2p segmentation data and traces to nwb file format. Thank you for making this.
I have a total of 12799 frames that were processed so my F.npy file is 520x 12799.

The code below runs without errors, but the roi response tracing has been converted to seconds using 10 Hz of sampling, as when inspected via nwbwidgets, the time index in seconds ranges from 0.00 to 1279.9 seconds.
To fix this I have tried to update the metadata before conversion, but still, the new output is the same. Based on the documentation, I am not sure if I am missing something and would appreciate any direction in this regard.
Thank you.

#%%
from datetime import datetime
from dateutil import tz
from pathlib import Path
from neuroconv.datainterfaces import Suite2pSegmentationInterface
from neuroconv.tools.roiextractors import roiextractors
folder_path= r"\\wt\\suite2p"
interface = Suite2pSegmentationInterface(folder_path=folder_path, verbose=True)

metadata = interface.get_metadata()

# For data provenance we add the time zone information to the conversion
session_start_time = datetime(2020, 1, 1, 12, 30, 0, tzinfo=tz.gettz("US/Pacific"))
metadata["NWBFile"].update(session_start_time=session_start_time)
metadata["Fluorescence"].update(rate=30.0, conversion=1.0)
# Choose a path for saving the nwb file and run the conversion

nwbfile_path = r"\\wt\\suite2p\\output_file.nwb"

#%%
interface.run_conversion(nwbfile_path=nwbfile_path, metadata=metadata)

Hi there! Welcome to NeuroConv, and thank you for posting such a well-detailed issue!
I'm sorry it's not made more obvious by the function calls or documentation, but the rate field of any RoiResponseSeries (of which Fluorescence from Suite2P is one) is ignored in the metadata - we instead use a separate mechanism to control timing information; the timestamps or sampling_frequency of the underlying segmentation_extractor object
The relevant lines of code can be found here for reference: https://github.com/catalystneuro/neuroconv/blob/main/src/neuroconv/tools/roiextractors/roiextractors.py#L928-L938
On top of this - we have also recognize in the past that Suite2P is a format that records the sampling frequency of the imaging data that was segmented, and so we extract that automatically in the ROIExtractors package here: https://github.com/catalystneuro/roiextractors/blob/main/src/roiextractors/extractors/suite2p/suite2psegmentationextractor.py#L155
That said, perhaps that option in Suite2P is returning the incorrect value of 10.0 Hz, or that value was set incorrectly from the imaging; the mechanism to override this value in our software would be as follows
from datetime import datetime
from dateutil import tz
from pathlib import Path
from neuroconv.datainterfaces import Suite2pSegmentationInterface
from neuroconv.tools.roiextractors import roiextractors

folder_path= r"\\wt\\suite2p"
interface = Suite2pSegmentationInterface(folder_path=folder_path, verbose=True)

# Correct the sampling frequency
correct_sampling_frequency = 30.0  # in Hz, using 30.0 as an example
interface.segmentation_extractor._sampling_frequency = correct_sampling_frequency

metadata = interface.get_metadata()

session_start_time = datetime(2020, 1, 1, 12, 30, 0, tzinfo=tz.gettz("US/Pacific"))
metadata["NWBFile"].update(session_start_time=session_start_time)

nwbfile_path = r"\\wt\\suite2p\\output_file.nwb"

interface.run_conversion(nwbfile_path=nwbfile_path, metadata=metadata)

I will note that we've been planning for a while now to improve the way time is handled across all of NeuroConv, so this exact way of correcting it should become easier (or at least less hacky...) in the future

Thank you. You are correct the file was looking at the ops file in the suite2p directory and using "fs" value as the sampling frequency. For some reason it was set to default, chnaging that to the correct value seems to give me the correct time axis in the nwb file.
Cheers!


Contributors.md file is missing.

Describe the Feature you want to end
Contributors.md file is missing as per the Hacktoberfest Guidelines.
Expected behavior
The repo should have a contributing.md file where contributors can add their name.
Screenshots (optional)

Contributing guidelines file already present.


FR: Browser tabs

Selecting currently open tabs would be so helpful to me

Do you mean to fuzzy search through the currently opened tabs in the selected browser?

Yes

I don't think that's in the scope of this plugin. The plugin is only meant to be collecting the existing bookmarks and fuzzy search through them. I think it makes more sense to have it as a separate plugin instead because the logic will also be different. For that reason, I'll close this issue.