Add German language support

Hi @abirke , will it be correct to use the 3 letter names?
There is a conflict with other languages if we use 2 letters. I don't want to break backwards compatibility.

Mon (Montag)
Die (Dienstag)
Mit (Mittwoch)
Don (Donnerstag)
Fre (Freitag)
Sam (Samstag)
Son (Sonntag)

Here is it: https://github.com/sakhnevych/calendar-view/blob/master/calendar_view/config/i18n.py#L29
I am ready to publish the release if it looks OK.

Published. Please write here if my changes to the German localisation are not correct.

Hi @sakhnevych,
sorry to keep you waiting. The three-letter abbreviations are pretty uncommon, I think. For parsing, however, I am not sure if a German translation is even necessary. Personally, I don't have any use-case for the translated parsing part.
Would you be okay with using the two-letter abbrevations and not supporting German in the parsing?
Otherwise, I guess I can live with the current solution.
Thanks, anyway!
André

@abirke , I have updated the code and released the latest version with two-letter abbreviations and a three-letter for parsing as a workaround (I think this is not a popular feature, and it is different from the name on the image anyway).
I will rely on your German language skills. Thanks for the pull request!


Missing file while instaling node-sass

During npm i node-sass I get an error:
Cannot download "https://github.com/sass/node-sass/releases/download/v3.6.0/win32-x64-48_binding.node"
I checked on the repo and it does indeed not exist. I tried also with previous version and same hapends.

It's because node v6.0.0 (48) is not yet supported, see #1504

<EMAIL_ADDRESS>has been released.

Thank you.


Move to holidays from python-holidays

Make it consistent for GitHub, PyPI, and Read the Docs

Proposed change

Prepare changes for the following naming standardization.
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/pin/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] I've run make pre-commit, it didn't generate any changes
[x] I've run make test, all tests passed locally

LGTM 🌐

I'm going to assume you got all the files, LGTM 🌐

Hopefully 🤞
If you notice something you know what to do :)

@arkid15r I think we might've forgotten to update codecov after we implemented the repo name change.

@arkid15r I think we might've forgotten to update codecov after we implemented the repo name change.

It looks like it's okay now. Let's monitor it for a bit more.


Add an additional, separate check on travis CI to build with go tip

Related to #2606

Closing as we've made two 2.8.x  releases and will abandon 2.x branch soon.


check what statistics are needed for good lateral spread fits

also double check resolution:
*----0---><----1---><----2---><----3---><----4---><----5---><----6--->
CYL              0.0       0.0       0.0      40.0      6.29      15.0
                4000         1      3000        -1       DDD      dcyl
*----0---><----1---><----2---><----3---><----4---><----5---><----6--->

this is currently 0.1 mm along beam axis, and 0.05 mm in radial direction??
I would think 0.2 mm should be enough, maybe even 0.5 mm? @grzanka advise?

It depends  it you want to simulate mono-energetic point-like pencil beams.
If this is the case, I'd go for a very fine mesh (due to small distal fall-off and tiny lateral size in entrance region). Anyway in this case fitting single or double gaussian models to lateral beam profile would be problematic in the entrance channel.
For other settings (i.e. ~1% of energy spread and ~1mm of beam source size) 0.2mm for depth and 0.05 mm in radial direction should be enough.


Add Sponsors list to footer

Edits: 2018-08-01
This PR got expanded (scope creeped) and now adds a number of sponsors to the footer.
Brand guidelines & assets
In general, I followed the following the below guidelines except I did desaturate any color if no black and white logo was provided.

https://buildazure.com/2017/09/25/microsoft-azure-gets-a-new-logo-and-a-manifesto/
https://www.cloudflare.com/logo/
https://www.elastic.co/brand
https://sentry.io/branding/
https://newrelic.com/about/media-assets
https://www.pagerduty.com/brand/

Screenshot

Looks good to me.

I like Eric's proposal.
We may want to include together with Cloudflare other services that we use (although, I'm not sure what's the contract we have with them: Azure, Sentry, New Relic, and maybe others that I'm missing here)

I think it might be more useful to specify a bit more about what they are providing us

My plan was to link to the upcoming blog post on the subject but I think I like your proposal better.

I have updated this PR and edited the description. This PR was changed to add a number of sponsors to the footer.

I suppose we can merge this PR.


Bag: Path including round bracket makes something bad

Environment

[OS]
[crowi-plus] 2.0.6
[node.js] 6.10.3
[browser] 52.3.0 (32 ビット)

How to reproduce? (再現手順) Case 1

create /hoge(fuga)
go to /hoge(fuga) or /hoge(fuga)/

What happens? (症状) Case

page list doesn't show any pages

What is the expected result? (期待される動作)

page list show list correctly

Note

Round bracket ( ) seems samething bad

This is related to weseek/crowi-plugin-lsx#5.


Issue/sidechain compute address

Updates tasks/deploy/sidechain.ts task
Updates gnosis configuration

Closing as these changes are in the feat/sidechain-gnosis branch


Fix wonky crontab spec.

Oops, sorry about the extra noise.

Bit disappointing that the kubeconform schemas still don't catch this. Oh well.


[FEATURE] Event service update  including organizations service related update

This services should not be related only for administrators/ organizers but also for users.

Thie issue is fully resolved withing #366.


Temporarily disable autosuggestions after whitespace characters?

The plugin currently auto-suggests \[ on blank lines for every whitespace character.  It is just a cosmetic thing, but I don't like it.
My zsh currently auto-inserts a space after every completion, and zsh-autosuggestions will autosuggest in all of these instances as well.  But, in >50% of these cases, I do not want an additional completion, and I just want to open the tab-completed file in vim, for instance.
Maybe this is a little clearer:

vim file1<tab> autocompletes to vim file1.txt <space><autosuggestion>file2.txt</autosuggestion>

Can I disable autosuggestions on whitespace characters (and for a single keystroke after all tab-completions), and re-enable for all other keypresses, or something like that?  Or maybe you have other suggestions to fix my configuration?

Thanks.

I just put up a pull request adding a new ZSH_AUTOSUGGEST_COMPLETION_IGNORE config option that you may be able to use to get the behavior you want. Please try out branch features/completion-ignore (PR #487).


Energy Sensor kit does not work on RF based machines

Well basically, when i use an energy sensor kit on for example the Resonant cell from TE, the kit gives me back the normal Energy Sensor location card, but it should give me the RF Energy Sensor Card.
Placing it in the Info Panel shows only that it has 32,000 energy.

If i spawn the RF card, and place any RF powered block in 0,0,0 location it works fine and shows the correct information in this case a fully charged Resonant Cell.

Discharged some power from it.

Bad on my side, or yours? using the kit on mekanism works fine, tested that out few mins ago

Nah, just a lot of mods. I'm not sure if this is a crossmod conflict or not.

Which mods should i try alone, IC2, Nuclear Conrol and just TE?

Try using those alone ^_^

Well friend, something aint right here, same issue, IC2 is fine

Mods used in this run
http://i.imgur.com/cVa7ieK.png

Alright, it's broken xD

If you need me to test test builds, im up for it, for the next 5 hrs or so

Nah, I can pretty easily test it myself as required.

cool, just saying, i just got time and a server free :)

Can you test with Mekanism? I have a feeling I introduced a bug while adding integration for that.

sure thing 3ish mins

Seems fine here

Btw i am using mek 9.0.2

Yep. Then this bug is something that was introduced with the new mechanism stuff. I'll see if I can fix it. Thanks for helping us test!

Dont want to be rude or asking for a release, but is there any news on the issue? NBT editing doesnt seem to work all the time for me, something is messing it up

The issue lies with the code itself. No number of NBT hacking will fix it. Your being given the wrong card, I just haven't had the time to fix it. Hopefully this weekend I'll try to force a patch in, else I just straight up revert the Mek integration.
Tl;dr I'll try for a release this weekend

There is a work around with nbt editing tho, spawn in the RF card and add the block coordinate tags, i managed to do it once after multiple tries,  NBTedit crashes too much for me

Fixed in recent commit. @xbony2 will release.

I would, but the build has failed. I think Forge's servers are a bit messed up or something.


support masked scan outputs in jax.lax.scan

It might be useful to be able to return masked arrays from jax.lax.scan.
I ran into this while trying to work around https://github.com/google/jax/issues/2770.
It looks like it'll be blocked on reshape: https://github.com/google/jax/issues/2565
So, I don't think this will be useful after most of jax.numpy works with masked arrays.
import functools
import jax
import jax.numpy as jnp

@functools.partial(jax.mask, in_shapes=['(n)', '(m)'], out_shape='(n, m)')
def outer(x, y):
  # These two aren't supported yet (thanks jekbradbury@ for debugging):
  # return jnp.outer(x, y)
  # return jax.lax.dot_general(jnp.ravel(x),  jnp.ravel(y), (((), ()), ((), ())))
  # This should work, but doesn't:
  _, xy = jax.lax.scan(lambda c, xi: (c, xi * y), (), x)
  # This also should work, but doesn't:
  # _, xy = jax.lax.scan(lambda y, xi: (y, xi * y), y, x)
  return xy
x = jnp.arange(3)
assert outer([x, x], dict(n=3, m=3)) == jnp.outer(x, x)

/usr/local/lib/python3.6/dist-packages/jax/lax/lax.py in _reshape_shape_rule(operand, new_sizes, dimensions, **unused_kwargs)
   2605   if prod(onp.shape(operand)) != prod(new_sizes):
   2606     msg = 'reshape total size must be unchanged, got new_sizes {} for shape {}.'
-> 2607     raise TypeError(msg.format(new_sizes, onp.shape(operand)))
   2608   if dimensions is not None:
   2609     if set(dimensions) != set(range(onp.ndim(operand))):

TypeError: reshape total size must be unchanged, got new_sizes (1, Poly({Mon({<object object at 0x7f24af566790>: 1}): 1})) for shape (3,).

jax.lax.dot_general(jnp.ravel(x),  jnp.ravel(y), (((), ()), ((), ()))) should work for outer on floats; are you seeing it fail?

Good point, I've updated the description. Casting the int32 to float32 works and I'm getting 25 bits of precision with Precision.DEFAULT.
How many bits does jax.lax.dot_general guarantee here?


Add description for pruneObjectBehavior

Adding a description to the pruning parameter in the CRD
Refs:

https://github.com/stolostron/backlog/issues/26528

Signed-off-by: Gus Parvin<EMAIL_ADDRESS>

/hold

/unhold


Read only Spring Boot App- NoUniqueBeanDefinitionException: No qualifying bean of type available: expected single matching bean but found 2

I just tried to implement a read-only repository using the article published in baeldung.com
While testing I was getting this error:
org.springframework.beans.factory.NoUniqueBeanDefinitionException: No qualifying bean of type 'com.scope.mdm.domain.LegalEntityReadOnlyRepository' available: expected single matching bean but found 2: legalEntityRepository,legalEntityReadOnlyRepository
To correct it, and after discussing it here I had to remove the inheritance from the BookReadOnlyRepository:
BookRepository.java:
package com.baeldung.boot.readonlyrepository;

import org.springframework.data.repository.CrudRepository;

public interface BookRepository extends CrudRepository<Book, Long> {
}

I am curious about how you made it work?
I am using spring boot 2.7.6 and java 19

...

Hey, @pauldj54.
It's hard to say for sure without looking at your complete project. But, judging from your question on SO, that's probably because you added a @RestController to your context. So, to use the BookRepository in the same way it appears in the article, you'd need to separate your contexts.
Unfortunately, we can only help with questions that are specifically and directly related to the article - not with your own, custom application.
But, as suggested in the same question, it's not necessary to extend BookReadOnlyRepository in BookRepository. We'll check with the team and update the article if necessary, to avoid confusion.

BookReadOnlyRepository does not extends BookRepository now, this unnecessary inheritance is now removed.


Environment support

Hi there!
This PR introduces some new functionality:

It allows you to specify a -e or --environment flag to parse a specific section from the secrets YAML. If this section does not exist, summon will exit with 1 (as expected). Old functionality still remains - it is not required to use the environment bit.
Added parsing of a common section from the YAML. Having a common section allows other sections to inherit env vars from it (primary sections will override env vars in `common). Unfortunately YAML's inheritance is more akin to a map merge, so we have to resort to our own merge-handling.
Expanded tests
Cleaned up runAction() signature as it was getting a bit unruly after adding 'env'; replaced the parameters with a single ActionConfig struct; besides looking a bit cleaner, it also has the benefit of being less guess work as to what each parameter means, both in tests and in-code).
Upped minor version
Forgot to expand README, will do that in a quick follow up commit

Some other quick notes:
To avoid a slew of type assertions and various if/else blocks (due to the new map becoming map[string]interface{}, I opted to separate parse() into two additional functions: parseRegular() and parseEnvironment. I feel that this is a bit cleaner than expanding parse() to handle both cases - but please let me know if you don't like the approach.
Please let me know what you think!
cc @jonDowdle

Thanks for contributing! I'll be reviewing and merging this today.

@dividedmind @dustinmm80 Hey folks, let me know if you'd like to see something change in the PR.


Feasibility study for "compute over state" application in FVM

Explore
Your task here is to conduct a feasibility study for implementing a compute over state app in  (FVM). This involves assessing the technical, economic, and temporal feasibility of the project.
My thoughts:
FVM provides ready to use datasets and this can be potentially leveraged for our application.
"Compute Over State" becomes extremely practical for our use case for
1. "ensemble learning":

Bagging:, is an ensemble technique where multiple versions of the original dataset are created by resampling, and a separate model is trained on each of these versions. The final prediction is obtained by averaging (for regression problems) or voting (for classification problems) the predictions from all models.
Boosting: boosting, on the other hand, is a sequential process where each model is trained based on the errors of the previous model, so it's less straightforward to parallelize. However, some implementations of boosting algorithms like XGBoost and LightGBM have been optimized for parallel computing

2. Distributed ML
Strategy: Training machine learning models with different shards of the dataset and then aggregating them into one model
Steps:

Split the dataset into shards: First, you need to split your dataset into several shards. Each shard will be used to train a separate model.
Train separate models on each shard: You can now train a separate model on each shard. You train on different contract calls
Aggregate the models: After training, you'll have multiple models. You can aggregate these models into one model. One common way of doing this is by making predictions with all models and then taking the majority vote (for classification) or the average (for regression)

import numpy as np
from scipy.stats import mode

def aggregate_models(models, X):
    # make predictions with each model
    predictions = np.array([model.predict(X) for model in models])

    # take the majority vote for classification
    return mode(predictions)[0]

# use the aggregate function to make predictions with the aggregated model
final_predictions = aggregate_models([model1, model2], X_test)

Shouldn’t we close this issue as we aren’t going ahead with fvm?

I will archive it once we have finalized on how we are going to implement parallelized ml training


Quest answer doesn't match phone language

When solving a 'language' quest, the possible answers don't match the language I set on my phone.
Answers are in German (used to be English), phone language is English (iOS 11 Beta 7, Kort 0.9.1 (1) from TestFlight-App).

Thanks for hinting at this bug. It has been fixed in the backend. Release will be soon.


[Snyk] Security upgrade pillow from 9.4.0 to 10.2.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-PYTHON-PILLOW-6219984
pillow:  9.4.0 -> 10.2.0
No
No Known Exploit

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-PYTHON-PILLOW-6219986
pillow:  9.4.0 -> 10.2.0
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

Fixed in https://github.com/wevote/WeVoteServer/pull/2426


SAGEMCOM 5566 VPI / VCI edit

Hi
I am looking to change the vpi/vci of the 5566 (bell HH3000) from 0/35 to 0/38 to be able to make it work with a different ADSL provider
Is this something that can be done ?

Sorry I have no clue what you ask. What do you want to achieve in Home Assistant?

There is a setting in ADSL which is VPI = 0 and VCI = 35 pre-configured in BELL home hub 3000.
I would like to know is there a way to access config file or upload a custom firmware where i change the VCI to 38 value
Thank you

@snawfel1983 that would be out of scope for this Home Assistant integration unfortunately…
and unfortunate, I wouldn’t know where else to ask this…


Bug nullprt may be close the sink if init failed

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Wrap table creation in AR transaction (db locking issues)

This fixes the locking issue we detected when running DB migrations. Might not solve everything (it probably won't), but it helps with the case where the importer is registering a table.
To be honest, I'm not sure this is a great idea, but seems to work fine in all my testing.
RFC @CartoDB/backend @luisbosque

Could you describe how come this fixes the locking issue, please?

The Sequel transaction is open during the @table.save call. Inside that, if you follow the hooks, you eventually get to some updates using the AR models, opening a second transaction (this one with AR).
The old sequence, with lock:

@table.save is called. A sequel transaction is started
A migration comes in, locks the visualizations table and waits for 1 to finish.
In an after_save of @table.save (still inside the sequel transaction), an AR transaction is started, but it is queued due to 2, deadlocking everything.

This happens if the migration starts between the time @table.save is called, and the time after_save is called, which turns out to be quite long (a couple of seconds).
With the new code, the AR transaction is preemptively started before @table.save, so:

AR transaction starts
@table.save is called, Sequel transaction starts
Migration comes in,  locks visualizations, locks for 1 and 2
after_save is called. AR uses the already open transaction (does not open a new one)
@table.save finishes, the transaction are releases and the migration runs

This reduces the time where the lock can happen to the instants between starting the AR transaction and calling @table.save, which is much less likely. It is impossible to 100% solve this problem with two ORM's (they cannot share transactions because they use different connections).
Anyway, this is a hack, that should help, but it is far from truly being a solution.

Uhm. I had some concerns about step 4 seeing old(er) data now that it doesn't spawn a new transaction but reuses the fist one, but it should be safe. This hack should probably be documented in the code. 🙈  👍

Closing due to inactivity (but leaving the branch around in case we need it).


Errors on site without expected parameters set

Hello!
$subj, when trying to run this theme against my blog source I get following (site won't render at all):
Building sites … ERROR 2018/03/20 20:36:09 Error while rendering "page" in "post/": template: theme/_default/single.html:3:12: executing "theme/_default/single.html" at <partial "title" .>: error calling partial: template: theme/partials/title.html:13:50: executing "theme/partials/title.html" at <.Site.Params.datefmt>: invalid value; expected string
ERROR 2018/03/20 20:36:09 Error while rendering "page" in "": template: theme/_default/single.html:3:12: executing "theme/_default/single.html" at <partial "title" .>: error calling partial: template: theme/partials/title.html:13:50: executing "theme/partials/title.html" at <.Site.Params.datefmt>: invalid value; expected string
ERROR 2018/03/20 20:36:10 Error while rendering "taxonomy" in "": template: theme/_default/list.html:9:8: executing "theme/_default/list.html" at <partial "li" .>: error calling partial: template: theme/partials/li.html:2:55: executing "theme/partials/li.html" at <.Site.Params.datefmt>: invalid value; expected string
ERROR 2018/03/20 20:36:11 Error while rendering "section" in "": template: theme/_default/list.html:9:8: executing "theme/_default/list.html" at <partial "li" .>: error calling partial: template: theme/partials/li.html:2:55: executing "theme/partials/li.html" at <.Site.Params.datefmt>: invalid value; expected string
ERROR 2018/03/20 20:36:11 Error while rendering "404" in "": template: theme/404.html:7:33: executing "theme/404.html" at <safeHTML>: wrong number of args for safeHTML: want 1 got 0

Change detected, rebuilding site
2018-03-20 21:10:53.151 +0000
Source changed "/srv/hugo/content/post/2018-03-19-lust-for-text-life.markdown": WRITE
ERROR 2018/03/20 21:10:53 Error while rendering "page" in "post/": template: theme/_default/single.html:3:12: executing "theme/_default/single.html" at <partial "title" .>: error calling partial: template: theme/partials/title.html:13:50: executing "theme/partials/title.html" at <.Site.Params.datefmt>: invalid value; expected string
ERROR 2018/03/20 21:10:53 Error while rendering "404" in "": template: theme/404.html:7:33: executing "theme/404.html" at <safeHTML>: wrong number of args for safeHTML: want 1 got 0
Total in 464 ms

The reason is absent .Site.datefmt parameter but you can't tell what's exactly wrong, I guess input parameters handling could be improved (default should be set). https://github.com/cboettig/hugo-now-ui/issues/9 might be used as reference point.

Thank you for the bug report!
I'll take a look at this, and the referenced issue.

Can you try again @paskal? I have push a new commit https://github.com/yursan9/manis-hugo-theme/commit/b190f41e89735865ff1196d852cbc2d0a7e933d8 to fix this.

No errors, main page have no posts, but /post/ show post folder content and single posts are rendered correctly — that's good enough for site I've didn't look into README to configure.
Theme is really nice, I'll consider it as a candidate for my blog. Thank you very much! Resolving.

With pleasure! I want to see your blog if you choose my theme later!

I guess you'll know by new 10 issues and PR which I will create:) By the way, really nice approach in setting defaults — I'll use it as an example for authors who have same problem with their themes.

I'm not sure github sent notifications to mentioned authors, so posting double there: theme highlight is out, this very theme is in it, additional info on https://github.com/paskal/terrty/issues/1.


🔄 Synced file(s) with WordPress/openverse

Synced local file(s) with WordPress/openverse.

Changed files

Synced local directory .github/ISSUE_TEMPLATE/ with remote directory .github/ISSUE_TEMPLATE/

This PR was created automatically by the repo-file-sync-action workflow run #0

Accident, please ignore.


Show existing sessions on initial connection (feature request)

It would be great if upon connecting via RDP, on the initial login screen, if it could show existing sessions already active, that way we can easily relogin to the same session without having to remember the port number for a particular session. Perhaps even sessions that are currently in-use vs sessions where nobody is connected to.
Finally a way to manage all the sessions just killing them without having to login to them individually to do it would be great.

The problem is that the session list would be shown to an unauthenticated user if it's shown on the login screen. I believe session selection should be done on the next screen. I agree that the existing "policies" are not very useful. Most users want to reconnect to an existing session. If they want a new session, there should be a way to request it.

Maybe there should be an initial login screen, then another screen to show sessions that they can access, (their own session or if root all sessions) and the ability to launch a new session.

I like the ability of concurrent access to a single session. But this only works if the login is shared atm.


Support CookieManager.getAll() on Android.

Bug
Getting error "Could not invoke RNCookieManagerAndroid.getAll" when try to run this code:
CookieManager.getAll(false).then(resAll => {console.log(resAll)});
OS: macOS
react-native: 0.61.5
react-native-cookies: 1.1.0
Steps To Reproduce
Install library, run this code:
CookieManager.getAll(false).then(resAll => {console.log(resAll)});

As mentioned in the description of PR #45, I spent some time looking into this but there is no easy way to get all available cookies for Android. In saying that however I will reference this

It is however important note that get() for Android will actually give a list of all cookies with domain related to the host of the url provided so it effectively acts as a getAll but for a single url. The Android system behaviour buckets the cookies based on a url (similarly to what you will see when you inspect cookies on Chrome) but does not allow for crawling through absolutely everything.


Add HMR support for linked & local npm packages

From #429, Adds basic HMR support for linked & local npm packages. As seen here:

The basic process is this:

Start the dev server with snowpack dev
Snowpack looks through your installed dependencies, and it looks for dependencies that are linked or local (file:…). It knows because some metadata is missing when it’s not downloaded from npm.
Snowpack then watches these directories—and only these directories—and when a file changes within, it does a fresh install and then reloads the dev server.

In the future, this could possibly be extended to support all first-class dependencies in node_modules, but for now it’s limited to linked & local packages (i.e. local to your filedisk, not downloaded).

Quick question about this change: will it have any impact on https://www.pika.dev/npm/snowpack/discuss/286 , or could it be extended to support that use case?

@airhorns Thanks for raising that discussion! I think yeah that may be another PR to add Yarn workspaces. I haven’t used it much but as you pointed out I think it’s just different enough from yarn link where I’d want to write a test specifically for that setup.
Also whereas this PR only touches snowpack dev for linked packages, I think workspaces may require a change to dev and build (maybe).


SimplifyBooleanExpression

part of https://github.com/openrewrite/rewrite/issues/197

Tried writing this in such a way that there was a single return on each visitX rather than multiple returns within the body.

Doesn't have to be this way, though I do personally think it's easier to debug and follow the logic on.
It's a bit harder to do with non-iso visitors, since the eventual return type will be different and may need to be cast around

Performs a maybeAutoFormat on the compilation unit, due formatting being removed when we replace the entire unary or binary-- in lieu of being able to perform autoformat on the subtree being replaced, it's being formatted on the compilation unit.

Note, I feel like there's probably a better way of doing this than running autoformat on the entire CU.

I'm going to do something just a lil' sneaky:

I'm going to merge this PR as-is, even with it calling AutoFormat on the entire compilation unit.

I'm going to have a TODO follow-up to have this not call AutoFormat on the entire CU

In a follow-up PR I'm going to use this Recipe as a base example of using postVisit


Vedio Confrencing WebApp

Describe the project you want to add with tech stack
It is a vedio confrencing web application made by using MERN stack.
Expected behavior
User can enter the vedio chat room by selecting their respective country and username.
Screenshots (optional)

Additional context (optional )

Please assign me this issue under gssoc'23.


New Makerspace location in Alma

User story
There is a new Makerspace location in Alma. Please see the following for more info:
https://github.com/PrincetonUniversityLibrary/alma-config/issues/481
Acceptance criteria

[x] New Location is represented in bibdata's locations config layer
[ ] Add to staging
[ ] Add to qa
[ ] Add to production

Concrete example
Re-index required?

[ ] Full re-index
[ ] Partial re-index
[x] No re-indexing required

Update documentation?

[ ] Update Indexing documentation
[ ] Update Documentation directory

Implementation notes, if any
Documentation at: https://github.com/pulibrary/bibdata/blob/main/docs/location_changes.md

This code change has been deployed to both staging and qa but I am re-opening this issue as running as this step did not work on staging https://github.com/pulibrary/bibdata/blob/main/docs/location_changes.md#2-update-bibdata-staging. It did work on qa https://github.com/pulibrary/bibdata/blob/main/docs/location_changes.md#3-update-bibdata-qa.
Staging failed to deploy via tower here: https://ansible-tower.princeton.edu/#/jobs/playbook/1279/output.

deployed to prod 5/31.


ImportError: no module named 'st7789'

I followed the instruction in sbcshop/PiSquare/Run RaspberryPi HATs On PiSquare Using Sockets.
When I run sbcshop/PiSquare/Run RaspberryPi HATs On PiSquare Using Sockets/Single Socket/pisquare_client_run_hat.py it fails with ImportError: no module named 'st7789'.
The module is imported by sbcshop/PiSquare/Run RaspberryPi HATs On PiSquare Using Sockets/Single Socket/library/gpio_config.py#L10 but it's not included in this repository.
On the PiSquare following MicroPython version is installed.
MicroPython v1.18 on 2022-01-17; Raspberry Pi Pico with RP2040
Type "help()" for more information.

It's the version which was preinstalled on the PiSquare.

I know it's not a perfect solution but I've helped myself by putting this file from the examples onto the Pico.
https://github.com/sbcshop/PiSquare/blob/main/Basic And Demo Codes/Onboard OLED Test/ssd1306.py
If you're using Thonny you can select the storage place as either Pico or PC and then select the Pico and make sure the file name stays the same.

The ssd1306.py isn't a problem as it is provided in the library folder
https://github.com/sbcshop/PiSquare/blob/main/Run RaspberryPi HATs On PiSquare Using Sockets/Single Socket/library/ssd1306.py
Updating the MicroPython firmware on the PiSquare with this version
https://github.com/sbcshop/PiSquare/blob/main/Run raspberry HAT's on PiSquare/Barcode HAT/firmware.uf2
solved the described issue for me.

This issue is of st7789 library, to resolve this query go through the below link -
https://github.com/sbcshop/PiSquare/blob/main/Run raspberry HAT's on PiSquare/1.3 Lcd HAT/firmware.uf2
The firmware includes frozen python font files and pre-compiled objects with the st7789 C driver for a variety of devices
Holding down the PiSquare BOTSEL button while dragging this file(firmware.uf2) into the mounted RP2 folder will allow you to load it.


List files in buckets

It would be useful to list files in a bucket so they can be created by another process or indexed.
e.g.
const things = bucket('things').for('reading')
const files: File[] = await things.files()

Do we need/should we have another permission to enable listing?

should listing files have its own permission type in the SDK?.
At the moment File listing is it's own action in our proto contracts:
https://github.com/nitrictech/nitric/blob/develop/contracts/proto/resource/v1/resource.proto (see BucketFileList).


sendmmsg now allows to have different control messages for different addresses

Fixes https://github.com/nix-rust/nix/issues/1954

I can copy the code I wrote for tests I guess, not sure if it makes a good example though since it doesn't use control messages...


getAllClientsUUID() returns ERROR when no user is connected

HOW TO REPRODUCE

Start SolAR services
Run reloc multi viewer when no user is connected

EXPECTED BEHAVIOR

No logs, or something like "No user currently connected", "Waiting for a connected user...", ....

OBSERVED BEHAVIOR

Message Failed to get all clients UUID is logged repeatedly

The message could be reworked on the viewer application side if returning an ERROR is indeed meaning the list is empty.
But it probably makes more sense to consider in getAllClientsUUID() that an empty list as a valid result, and keep the ERROR return code for actual errors (exception, insertion failure,...).
The logged message is confusing and leads to think that something went wrong with the request whereas it reports that no user is connected which is a valid scenario.
Furthermore, nothing in the documentation seems to state that returning an empty list is an error.

Issue fixed in last commit


Content-Type does not respect optional charset encoding for iOS RCTJavascriptLoader

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
Yes
Environment

react-native -v: 0.42.0
node -v: v6.11.2
npm -v: 4.6.1
yarn --version: 0.27.5

Target Platform: iOS
Development Operating System: macOS
Build tools: XCode

Steps to Reproduce

Try to run a react native app and download a bundle from a server which returns a Content-Type header that includes both the MIME Type and an optional character encoding as specified here: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Type
The React Native packager will fail on this line of code as it strictly checks the Content-Type header as if the optional character encoding doesn't exist: https://github.com/facebook/react-native/blame/master/React/Base/RCTJavaScriptLoader.mm#L277

Expected Behavior
The code should follow the specification of Content-Type and respect only MIMEType checking instead of strictly checking the Content-Type header for an exact match on MIMEType.
Per Apple documentation, NSURLResponse contains the MIMEType field that should be checked for application/javascript instead of checking the raw Content-Type header: https://developer.apple.com/documentation/foundation/nsurlresponse
Actual Behavior
Though this was encountered while using Expo, the error is nonetheless coming from the RCTJavaScriptLoader

Expected JavaScript, but got content type 'application/javascript; chartset=UTF-8'. Check that your internet connection is working.
Reproducible Demo
https://github.com/sysgears/apollo-universal-starter-kit
Follow starter instructions and checkout this commit: https://github.com/sysgears/apollo-universal-starter-kit/commit/bbcd52a934e24b4ce873e802d085af2b8fe04ec2
The problem is worked around after this commit in the project.

I just tripped over this when attempting to get Haul to make a build for react-native 0.48.2: https://github.com/callstack-io/haul/issues/228

cc'ing @janicduplessis as he was cc'd on an Expo bug report.

Up @janicduplessis, there are some commits in the history of this issue. Is any of them ready for merging? If yes, I can go ahead and cherry-pick into respective branches.

The PR has not been merged yet https://github.com/facebook/react-native/pull/15792, let's cherry pick it once it is.

@janicduplessis did this end up getting merged somewhere?


no-unused-prop-types throws when using setState callback

This code make eslint job fail when using no-unused-prop-types rule in 7.5.0:
this.setState(prevState => ({
  open: !prevState.open,
}));

It throws TypeError: Cannot read property 'properties' of undefined at markPropTypesAsUsed
Looks like !node.params.length should be replaced with node.params.length <= 1 here:
https://github.com/yannickcr/eslint-plugin-react/pull/1507/files#diff-4d3f22e69602f6d3e0d5e1cf8328dc76R969

@Hypnosphi try v7.5.1? This seems like a duplicate of #1542.

Oh sure, thanks


Add quality metrics

We could add something like:

Better Code Hub ,
Bit Hound
Code Climate

We should also report test coverage of /src

@arahansen do you have any preferences?


[Search] Manga list in character search is always empty

I was developing jikan.net and during wrapping search request it struck me that every search result entry in character has empty "manga" key. Sometimes mangaography gets shifted to animeography of the character.
Example scenario: user call the following endpoint: https://api.jikan.moe/search/character/kirumi, which should be parse data from https://myanimelist.net/character.php?q=kirumi
What one would expect: the only found character (Kirumi Tojo from "Danganronpa") has single element in "manga" array (as in MAL results).
What actually happens: said Kirumi has single element in "anime" array (the same one which is lacking from "manga array).

Good find! Working on a fix.


Save PNG file with position information (x, y, w, h)

This plug in is marvelous, thank you so much
now I can  export each layer in .png files easily.
I was wondering if you can add a future that save the layer position information(x,y,w,h ) as file name when exported.

This is should be relatively easy to do. I'll add this feature in the next release.

Thank you very much, I was just going to ask for the same thing.
My need is to have position and width information so I can use it in a css file.
It could be very interesting to have the positions and width in percentage not just in pixels.
Thank you.

Version 3.3.1 now supports specifying layer attributes (using the [attribute] field), both as pixels and percentages relative to the image width and height.


After the successful launch of the local project, many CSS files JS files and pictures file errors 404

After the successful launch of the local project, many CSS files JS files and pictures file errors 404, the project has been no problem before the sudden like this.

This question is available on Nuxt community (#c8823)

Without a reproduction repository or even a URL don't expect to get any help from anyone 🚶


Need append the "-dont_append_source" argument to kindlegen

The mobi created by kindlegen is so huge because kindle will append the source files to the mobi file.
We can use the script kindlestrip.py to make the mobi be small.
But  the author said we shouldn't use the kindlestrip.py to make mobi file be small now because it may make the wrong mobi formater. We should use the arguemt "-dont_append_source" when we use kindlegen.exe to generate mobi file.
For, example "kindlegen.exe -dont_append_source xxxxx.epub".
And the mobi file will be small.
So how can I append the "-dont_append_source" argument to kindlegen.exe when I use kcc to create mobi?

KCC already use "-dont_append_source" option during MOBI creation.


Typo fix

Was wondering when someone else would notice that :)

Yeah, Jack wanted last minute links added for some class we was teaching. Don't force push at 7am on a Saturday lol

Thanks everyone!


[Feature] Add Installer Option for rEFInd

Description
According to the Opencore Vanilla Desktop Guide Opencore can be used in conjunction with rEFInd.
It would be nice to have a graphical picker for start volume selection and other useful tools like SIP management or boot to recovery.
Since not everybody needs or wants this, it should be optional in the installer.
References

https://www.insanelymac.com/forum/topic/338516-opencore-discussion/?page=8&tab=comments#comment-2672316

I’ve been told there will be a better GUI coming soon based off of Apple’s GUI code. Should probably be better.


Fancyzones problem with Adobe Illustrator

Microsoft PowerToys version
0.74.1
Installation method
PowerToys auto-update
Running as admin
None
Area(s) with issue?
FancyZones
Steps to reproduce
I love Fancyzones but recently I'm facing an issue while using Adobe Illustrator. I am using a super ultrawide monitor with aspect ratio of 32:9 and while recording tutorials I use a custom fancyzone of 16:9 in the middle of screen. Now problem with Adobe Illustrator is that whenever I zoom out my artboard in Adobe Illustrator it just unsnap the Illustrator window from fancy zone. I really need a solution to this problem. Please help.
✔ Expected Behavior
I expect my Adobe Illustrator window to stay in fancyzone regardless whatever I do in that window.
❌ Actual Behavior
Window un snap from fancyzone.
Other Software
Adobe Illustrator CC2024 version 28

This seems like a problem with Illustrator. Can you share a /bugreport to see what settings you have set for Fancyzones.

I'm having the same issue. However, can't post a bug report as the option to do so doesn't come in the right click menu..

Hi, I am experiencing crazy behavior with Fancy Zones in relation to Adobe Illustrator. Whenever I have Illustrator snapped to a fancy zone and I open a document, 75 percent of the time when I open a document, it just displays a blank screen where the document should be. I can see all the layers as if the document is open, but I can't see the document itself in the Illustrator sub-window. This behavior occurs if I have padding added to the fancy zone layout.
If I create a fancy zone with 0 pixel padding, when I open a document that is snapped into a fancy zone, then the whole Illustrator window jumps to a new location. It doesn't resize; it just moves halfway off the screen (50% down and 50% to the right).

I don't have access to Adobe Illustrator, not sure if anyone on the PowerToys team does.

@awadrummer can you open a new issue and attach a /bugreport the bots here cannot be stopped 🤖👀


online or offline

Dear author, this is an excellent paper, but I would like to ask if this work is an online paradigm or an offline paradigm.

Thank you for your interest! This work is an online paradigm. You may also use it in offline videos.


multi: set zero bandwidth hint for channels that do not have any available slots

This PR updates our bandwidth hints to report a zero balance when we're not allowed to add any more htlcs to the remote party's commitment. This prevents pathfinding from using the channel at all; previously it would endlessly try to send shard on the channel.
The first commit just demonstrates the endless cycle of htlcs by logging (couldn't think of a better way), and should be squashed with the second commit to include a test that asserts that we don't use full channels. Open to other testing options besides an itest, but it's difficult to do because this is a combination of multiple elements - pathfinding + payment lifecycle + link - so mocking this didn't feel like it would meaningfully add much (would be testing the mock rather than the code itself).

WDYT about keeping the itest? Was planning on updating to just add the itest in the last commit.

WDYT about keeping the itest? Was planning on updating to just add the itest in the last commit.

Yeah I think the itest is good as it exercises previously uncovered behavior

Yeah I think the itest is good as it exercises previously uncovered behavior

sgtm, squashed all itest changes to last commit where we add the fix.

@Roasbeef is this something we could consider for 0.13? When this condition hits it really hammers the db with payment attempts, with 60 second timeout locally I saw 200-300 unnecessary attempts.

Rebased + addressed itest comments. Thanks for the review @Crypt-iQ !

Thought about this a bit more, and I think it's important we patch this issue in the short-term as it's affecting users in the wild. As I outlined above, we can generalize things a bit and also address other possible vectors for a tight HTLC loop like violating constraints like the max amount allowed in flight. So I'm thinking we move forward with what we have here, package it in 0.13.1, and shoot for the more fundamental fix so we can land it in 0.14.


Allow cross type parsing in Engine options

Description
Updates eq, ge, gt, le, lt and ne comparisons to allow cross type parsing if configured in Engine parser options
Test Coverage

[x] unit

References

Closes #135

Ran the linter on the .test.ts files hence the reformatting, i wonder why linting test files were ignored.

The configuration is engine-wide, correct?  I suppose there's no elegant way to scope it to individual conditions.

The configuration is engine-wide, correct? I suppose there's no elegant way to scope it to individual conditions.

Configuration is engine-wide yes but only the comparison conditions stated above check this flag

Should we have test cases for date comparison '2010-01-01' > '2009-02-03'?

Should we have test cases for date comparison '2010-01-01' > '2009-02-03'?

Part of this PR?

Part of this PR?

I would expect that to work even with strict mode off, comparing 2 strings.

Should we have test cases for date comparison '2010-01-01' > '2009-02-03'?

This works with or without the strict mode, i can add this case to the test to verify

Closing in favor of https://github.com/briza-insurance/illogical/pull/139


cmake: Silent leveldb-specific warnings

This PR split from https://github.com/hebasto/bitcoin/pull/84
It introduces a new warn_leveldb_interface, which overrides -Wconditional-uninitialized and -Wsuggest-override if any.
For MSVC builds, dropped warning suppressions that are provided at the global level.
For example, on the staging branch:
$ env CC=clang CXX=clang++ CXXFLAGS="-Wconditional-uninitialized -Wsuggest-override" cmake -B build
$ cmake --build build --target leveldb > /dev/null
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:474:17: warning: 'Name' overrides a member function but is not marked 'override' [-Wsuggest-override]
    const char* Name() const { return rep_->Name(); }
                ^
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:110:15: note: overridden virtual function is here
  const char* Name() const override { return (*name_)(state_); }
              ^
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:475:10: warning: 'CreateFilter' overrides a member function but is not marked 'override' [-Wsuggest-override]
    void CreateFilter(const Slice* keys, int n, std::string* dst) const {
         ^
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:112:8: note: overridden virtual function is here
  void CreateFilter(const Slice* keys, int n, std::string* dst) const override {
       ^
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:478:10: warning: 'KeyMayMatch' overrides a member function but is not marked 'override' [-Wsuggest-override]
    bool KeyMayMatch(const Slice& key, const Slice& filter) const {
         ^
/home/hebasto/git/bitcoin/src/leveldb/db/c.cc:125:8: note: overridden virtual function is here
  bool KeyMayMatch(const Slice& key, const Slice& filter) const override {
       ^
3 warnings generated.
/home/hebasto/git/bitcoin/src/leveldb/db/version_set.cc:1016:55: warning: variable 'manifest_size' may be uninitialized when used here [-Wconditional-uninitialized]
  descriptor_log_ = new log::Writer(descriptor_file_, manifest_size);
                                                      ^~~~~~~~~~~~~
/home/hebasto/git/bitcoin/src/leveldb/db/version_set.cc:997:25: note: initialize the variable 'manifest_size' to silence this warning
  uint64_t manifest_size;
                        ^
                         = 0
1 warning generated.

Friendly ping @fanquake @TheCharlatan @m3dwards @pablomartin4btc @vasild  :)
@theuni Your https://github.com/hebasto/bitcoin/pull/6/files#r1113411466 has been addressed.

Concept ACK. I think "nowarn_leveldb_interface" would make this more intuitive though.

I think "nowarn_leveldb_interface" would make this more intuitive though.

Thanks! Renamed.


Error fetching Git resource

I tried installing Git as well but ended up having the same issue.
Error message in the web interface:
checking failed
resource script '/opt/resource/check []' failed: exit status 128

stderr:
Identity added: /tmp/git-resource-private-key (/tmp/git-resource-private-key)
fatal: repository '/tmp/git-resource-repo-cache' does not exist

The pipeline:
---
resources:
  - name: spotlight-git
    type: git
    source:
      url: git@github.com:pivotal-sg/spotlight.git
      branch: master
      private_key: |
        -----BEGIN RSA PRIVATE KEY-----
        <deploy key>
        -----END RSA PRIVATE KEY-----

jobs:
  - name: spotlight
    plan:
    - get: spotlight-git
      trigger: true
    - task: spotlight-spec
      file: spotlight-git/ci/spotlight-spec.yml

False alarm. I had a typo in my key - used URL instead of URI. The error message was not helpful though.


Sites: add a command that deletes older, unused assets from the kv namespace.

Currently we do not delete stale site assets when publishing.
We should consider a new command that can strip out ones that are no longer needed.

We now delete unused assets on deletion. Closing this issue.


Ability to disable built-in tags for sendDistributionMetric function.

Expected Behavior
When I'm using NPM package datadog-lambda-js and I'm sending metrics using function sendDistributionMetric(name: string, value: number, ...tags: string[]): void; I end up receiving additional custom metrics with built-in tags, for example dd_lambda_layer:datadog-nodev12.14.1.
Since any unique combination of metric name and tag name is billed as separate custom metric, I would like to disable this behaviour, since I'm using datadog-lambda-js as DataDog API client for custom metrics.
Actual Behavior
Calling function sendDistributionMetric(name: string, value: number, ...tags: string[]): void; from datadog-lambda-js NPM package adds additional, built-in tags, for example dd_lambda_layer:datadog-nodev12.14.1.
Steps to Reproduce the Problem

Create Node.js Lambda and add "datadog-lambda-js": "2.17.0" as dependency.
Use sendDistributionMetric function.
In DataDog UI, metrics explorer you will see some additional tags, that you haven't specified in the code.

Specifications

Datadog Lambda Layer version: None, "datadog-lambda-js": "2.17.0"
Node version: 12.x

Hi @PatrykMilewski, thanks for filing an issue.
The custom metrics you submit with the Datadog Lambda Layer / package are Distribution metrics. These are unique in that you control which aggregations and percentiles you want. To disable a tag like dd_lambda_layer, you can navigate to Metrics > Distribution Metrics > your.custom.metric and change the aggregations there.
Docs: https://docs.datadoghq.com/metrics/distributions/#customize-tagging

Thank you @pinkerton, it worked!

@pinkerton I think I have to reopen this issue. The workaround that you proposed works, but if I publish tens for metrics, then I have to click through them and for each of them disable dd_lambda_layer tag, which is time consuming and annoying.
My point is, that it could be avoided, since for my use case this tag is really not important and could be disabled, since I'm not getting any value of having it in my DataDog.
I did look into the code and it's only about possibility to force getRuntimeTag() to return nothing by some flag that can be provided with configuration.
export function sendDistributionMetric(name: string, value: number, ...tags: string[]) {
  tags = [...tags, getRuntimeTag()];

  if (currentMetricsListener !== undefined) {
    currentMetricsListener.sendDistributionMetric(name, value, ...tags);
  } else {
    logError("handler not initialized");
  }
}

@PatrykMilewski I think this request makes sense, we can keep this issue open.
However, to clarify, unless you emit the metric from multiple Lambda functions using different versions of Datadog layers, this extra tag won't increase # of billed custom metrics, as its cardinality is 1. Say, if the # of unique combinations of other tags is 12, then 12 * 1 is still 12.
This doc might be helpful: https://docs.datadoghq.com/account_management/billing/custom_metrics/?tab=distribution
Let me know if I missed anything.

@tianchu Thank you for response!
I thought that unique combinations of tags billing works a little bit different. Now I agree, that from biling perspective, there is no point in disabling this.
But still it adds some unneeded noise to all of the metrics.
Maybe let's keep this issue as open and if somebody else find it as a problem, then we can get back to it? 😄

I no longer need of removing this default tag, that's why I'm closing this issue.


Autogenerated docs failing for multi-output plugin config options

For multi-output plugins, currently only the configuration options of their parent plugins are displayed in the straxen docs. This should be an issue with build_datastructure_doc.py.

Fixed in #434

That was some extremely long outstanding issue 👴


Enhancement - sub-generator to create javascript backed behaviors

As Alfresco supports implementing behaviors in javascript as an alternative to Java, it would be a useful feature to allow the generator to accommodate this scenario.
This could either be a new sub-generator or modifying the current behavior sub-generator to ask if it should be javascript or java, and adapt accordingly.
There are some minor differences of the bean definitions in the Spring Service Context and the sample/template for the code would also differ.
I'd be willing to help with testing, debugging and provide feedback.

Great Idea! This is something I always intended to do and never got around to. Are you up for documenting what needs doing or finding and referencing the appropriate documentation?
If you would like to try your hand at updating the sub-generator, I'd also be very happy to help you get up to speed on doing that. But am happy to take it on with some more detailed steps/references if you'd prefer not to dig into the code.

A good reference is Jeff Potts's tutorial https://ecmarchitect.com/alfresco-developer-series-tutorials/behaviors/tutorial/tutorial.html where after he shows a behavior implemented in Java, he then re-implements in Javascript.
For Java:
<bean id="ratingBehavior" class="com.someco.behavior.Rating"
init-method="init">
    <property name="nodeService">
        <ref bean="NodeService" />
    </property>
    <property name="policyComponent">
        <ref bean="policyComponent" />
    </property>
</bean>

For Javascript:
<bean id="onCreateRatingNode"
class="org.alfresco.repo.policy.registration.ClassPolicyRegistration"
parent="policyRegistration">
    <property name="policyName">
        <value>{http://www.alfresco.org}onCreateNode</value>
    </property>
    <property name="className">
        <value>{http://www.someco.com/model/ratings/1.0}rating</value>
    </property>
    <property name="behaviour">
        <bean class="org.alfresco.repo.jscript.ScriptBehaviour" parent="scriptBehaviour">
            <property name="location">
                <bean class="org.alfresco.repo.jscript.ClasspathScriptLocation">
                    <constructor-arg>
                        <value>alfresco/module/${project.artifactId}/scripts/onCreateRating.js</value>
                    </constructor-arg>
                </bean>
            </property>
        </bean>
    </property>
</bean>
<bean id="onDeleteRatingNode"
class="org.alfresco.repo.policy.registration.ClassPolicyRegistration"
parent="policyRegistration">
    <property name="policyName">
        <value>{http://www.alfresco.org}onDeleteNode</value>
    </property>
    <property name="className">
        <value>{http://www.someco.com/model/ratings/1.0}rating</value>
    </property>
    <property name="behaviour">
        <bean class="org.alfresco.repo.jscript.ScriptBehaviour" parent="scriptBehaviour">
            <property name="location">
                <bean class="org.alfresco.repo.jscript.ClasspathScriptLocation">
                    <constructor-arg>
                        <value>alfresco/module/${project.artifactId}/scripts/onDeleteRating.js</value>
                    </constructor-arg>
                </bean>
            </property>
        </bean>
    </property>
</bean>

Notice that the bean definitions are a bit more complex to support Javascript.  You use a built in class org.alfresco.repo.policy.registration.ClassPolicyRegistration to register a policy, and you pass the URI of the content model as a property.  The actual behavior is defined as a built-in java class (org.alfresco.repo.jscript.ScriptBehaviour) that is just a Javascript runner.  You then need to pass the path to the script a property of the behavior bean.  Basically the extra verbosity in the bean replaces what would be in the init() method of the Java class.


Adds placeholder for Priority Status

This is a nice idea, but please fix the code because this will error out if someone does not use Badger-Priorities @devshlime


Audit log encounter key error  in LANGUAGE_CODE = 'zh-hans'

Found a bug? Please fill out the sections below. 👍
Issue Summary
When set LANGUAGE_CODE = 'zh-hans' and open /admin/reports/site-history/ in browser, it will give me a key error, I found the reason was that 'Unknown {action}' is translated into '未知{行为}'  in Chinese so that it cannot be format with key ‘action’.
Steps to Reproduce

LANGUAGE_CODE = 'zh-hans'
wagtail/admin/log_action_registry.py
message = self.get_messages().get(log_entry.action, _('Unknown {action}').format(action=log_entry.action))

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes / no)

Technical details

Python version: Run python 3.6.
Django version: 3.08`.
Wagtail version: 2.10`.
Browser version: Chrome.

Thanks a lot.


Item comments tag support

Is your feature request related to a problem? Please describe.
Some RSS readers, such as Miniflux, support a comments tag for feed entries. Entries with it will have a comments option that will go to the provided URL. A number of feeds are generated with this, but RSS-Bridge does not allow creating feeds with it. This limits bridges slightly and results in FeedExpander removing data from some source feeds. I've wanted to create some bridges that have this tag, and being able to both pull from existing RSS feeds and add to bridge entries I create is desirable.
Describe the solution you'd like
Implementing the tag much like other bridge tags would be sufficient (e.g., the same as how $item['title'] functions.
Describe alternatives you've considered
The only real alternative would be embedding the link in content, but this would ideally allow for use of an existing reader feature.

Huh, TIL. You are probably referring to the RSS <comment> element?
RSS-Bridge currently supports the following feed formats: Media RSS, Atom, JSON Feed and sfeed.
Since this is only specific to RSS, I think that providing support for only one feed format may not be within this project's goals, but maybe an extensible $item['item_element'] might be within scope, to allow other feed formats discard elements that's not in their specs.

Yeah I meant the <comment> element. An example of this problem at work can be seen with the Ars Technica bridge I recently PR'd, where that element is lost (as well as entry categories, though that's purely a FeedExpander issue).

do you mean the <comments> element?

@Phantop

Yes, I meant the <comments> element as can be seen in this feed, my apologies.

For what it's worth, I tried it out and it only requires a couple changes to add in comment support. Most of the code already exists for standard URI handling. I made a patch that I've been using on my own RSS-Bridge instance, and it seems to work fine.


fixes dshackle readiness and liveness probe overrides; when attemptin…

Hello
This change will allow users to set httpGet readiness/liveness probes. The default values.yaml causes the tcpSocket to be set in the resulting yaml even if tcpSocket is set to null.
Dshackle presents a health configuration which allows users to set a minimum-available upstream threshold before returning non-200 for requests to the health probe.
The health endpoint could also be enabled by default, however, this is the least impactful to anyone currently relying on simple tcpSocket probes, who have not configured dshackle to enable the health endpoint.
To reproduce the issue, use this override.yaml:
livenessProbe:
  httpGet:
    path: /health?detailed
    port: 1234
readinessProbe:
  httpGet:
    path: /health?detailed
    port: 1234

broken
$ helm template dshackle -f override.yaml . | grep -5 /health?detailed
            - name: health
              containerPort: 8082
              protocol: TCP
          livenessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 60
            periodSeconds: 30
            tcpSocket:
              port: http
          readinessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 10
            periodSeconds: 10
            tcpSocket:
              port: http

fixed:
$ helm template dshackle -f override.yaml . | grep -5 /health?detailed
            - name: health
              containerPort: 8082
              protocol: TCP
          livenessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 60
            periodSeconds: 30
          readinessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 10
            periodSeconds: 10
          resources:

The specific error from the sts controller is
create Pod dshackle-0 in StatefulSet dshackle failed error: Pod "dshackle-0" is invalid: [spec.containers[0].livenessProbe.tcpSocket: Forbidden: may not specify more than 1 handler type, spec.containers[0].readinessProbe.tcpSocket: Forbidden: may not specify more than 1 handler type]

@niko9001 pls run make docs from the root, and add this to the PR.

if you change your override to
livenessProbe:
  httpGet:
    path: /health?detailed
    port: 1234
  tcpSocket: null
readinessProbe:
  httpGet:
    path: /health?detailed
    port: 1234
  tcpSocket: null

then it should drop the tcpSocket
helm template will also then result in:
        livenessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 60
            periodSeconds: 30
          readinessProbe:
            httpGet:
              path: /health?detailed
              port: 1234
            initialDelaySeconds: 10
            periodSeconds: 10


Updated logo to new Mesosphere DC/OS Logo

Attached is a screenshot of the new logo in the navbar
Description
We've updated our logo in the navbar to the new Mesosphere DC/OS logo.
Urgency

[x] High

@ashenden - the links that failed are links within the 1.7 documentation since it seems like http://mesos.apache.org/ changed some of our their links.


Workaround for missing speed in GAME_SPEED_FACTOR

Fixes #101

So you want me to default to 0.6 instead of 1?

Sorry for being unclear: I missed actual question in the context =P
Why did you decide to use integer division instead of regular division? I'm not really sure how real/important the fractional parts are

The current code wraps the calculation in int() and this PR uses // because
all(int(a / b) == a // b for a, b in ((1, 0.6), (1, 1.2), (10, 0.6), (100, 0.6), (0.6, 1)))   # -> True

OH! I missed that int() in the original code. You definitely did that right. Also, I think using 1.0 as the default speed is probably more correct than 0.6. So everything you initially did everything correct =P
If you revert that, I will approve and merge. Thanks (and apologies) for bearing with me!

https://github.com/ggtracker/sc2reader/pull/102#pullrequestreview-311311684 So you want to default to faster, not normal?

Yes, let's default to faster. It will at least be ocrrect for alphastar and is the de facto default these for LotV ladder, right?

Thanks!


Documentation with docz

https://www.gatsbyjs.org/tutorial/writing-documentation-with-docz/
https://www.docz.site/
https://github.com/doczjs/docz/

Branch on docs, but I found docz incredibly hard to work with. It took an hour just to get a basic example working

For now I'll stick with mkdocs because it's simple


pasystray doesn't seem to start in KDE5 in 15.09

Posted a similar here too: https://github.com/christophgysin/pasystray/issues/48#issuecomment-191059056
Basically no systray icon is shown when it is launched from either the CLI or from KDE.
From the CLI, we get warnings:
** (pasystray:9207): WARNING **: Error retrieving accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown: The name org.a11y.Bus was not provided by any .service files

(pasystray:9207): pasystray-WARNING **: Error initializing Avahi: Daemon not running

(pasystray:9207): GLib-GObject-WARNING **: The property GtkButton:use-stock is deprecated and shouldn't be used anymore. It will be removed in a future version.

(pasystray:9207): GLib-GObject-WARNING **: The property GtkSettings:gtk-button-images is deprecated and shouldn't be used anymore. It will be removed in a future version.

But the program doesn't terminate.

This is fixed at unstable, my gtk applets is shown in the panel.

The version of Plasma 5 in NixOS 15.09 does not support legacy tray icons.


improve cardinality analysis for kodkod

see bugs/better_card_kodkod.nun

Now the issue is that the Kodkod backend doesn't know it can stop if current cardinality exceeds all bounds. Might be of interest for @blanchette .


Spec - 4.5 - Client must prioritize trust in an ordered fashion given multiple delegations

e.g., the first to appear is the truth
Related: #16

Spec is unclear on this. Waiting on updates.

Could you explain where it is unclear? We have how process prioritized / terminating / multi-role delegations, but I suppose you mean how to write these delegations in the first place?
 I think my local copy is out of sync with the latest. At some point it said something like "we are investigating different methods for prioritization" and I thought it still hadn't been resolved. I think the spec actually includes enough of an explanation.

This was actually closed by the change set from the merge 4844207

In case you need more information, please also see the Diplomat paper. Let us know if you have questions!

Reopened because of the rewrite.


Can X113647 be used for bipolar 4-wire motor ?

What is the difference between X113647 and L298N?
Can a X113647 be used for driving a DVD motor?

Sorry I can't comment on the L298N - no experience with that.

Can the X113647 be used for DVD motor?

Most likely. The X113647 is a pretty generic ULN2003A-based board, so as long as the DVD stepper is 4-wire control, should work if you get the connections and sequence right.
best of luck in hacking that;-)


rancher.metadata container name collision in backend (different behaviour compared to rancher.api)

Do you want to request a feature or report a bug?
Bug (Unless its intentional)
What did you do?
Docker System : Rancher v1.6.5
Deployed latest official image : traefik:v1.4.0-rc2-alpine
With the following command for meta-data mode (formatted to be easier to read)
--web --rancher
	--rancher.domain=rancher.localhost
	--rancher.enableservicehealthfilter=false
	--rancher.exposedbydefault=false
	--rancher.metadata

Along with the following container labels
io.rancher.container.agent.role=environment
io.rancher.container.create_agent=true

Multiple stacks was then created, with different stack names. But same container names (web-studio in this case)
Each container is defined with the following traefik labels
traefik.enable=true
traefik.frontend.rule=Host:the-container-unique-url.domain.something
traefik.port=80

Note: this is a followup from https://github.com/containous/traefik/issues/2125
What did you expect to see?
Every container, from each stack, with its own "host name rule" for frontend, and its own backend, as seen below, when traefik is using rancher.api

What did you see instead?
A single backend, for all the various frontend, when traefik is using rancher.metadata

Note : that both this traefik configuration was deployed side by side to the same rancher testing infrastructure. So both configuration was expected to be same
Output of traefik version: (What version of Traefik are you using?)

None: See setup above

Hi @PicoCreator ,
this is already fixed in #2107, so it should be available with the next rc for 1.4 :)


Unwarranted warning

Version 0.1.1 seems to be working much better in intial tests. I'm just writing a series of monthly 1d means to the same bucket with successive calls with identical arguments (except for the input filename). I'm getting this warning on the second and subsequent calls:
☁  msm_os ☁  |    WARNING | 2024-09-13 15:49:49 | You already have data in the object store and you can't rechunk it
which is unwarranted since the chunk settings haven't changed?

Hi @accowa ,
I included that warning to inform users that once new data is added to the object store, rechunking is not possible. However, I’ll adjust the warning so that it only appears when users attempt to modify the chunking strategy.
Regarding the time_counter data, I’ll investigate why it isn't being chunked the same way as the other variables. From a quick review of the code, there doesn’t seem to be a specific reason for excluding time_counter from chunking.
In my opinion, given that it's a 1D array of time data, we may not need to chunk it. If you prefer, I can include it in the chunking strategy.
As for the execution time, Dask does speed up data uploads by parallelizing the transfer of each variable, but I am not sure on how much the performance will increase. However, there are two main bottlenecks right now:

Integrity check: The checksum calculation is quite slow because it requires converting all the data into bytes before determining the size. I'm currently using np.frombuffer for this, but I'll test converting the np.array to a dask.array to see if it improves performance. I'll include this in the new pull request. Additionally, I’m considering adding a flag to allow users to disable the integrity check if needed. Do you think that would be a helpful option?

Data reprojection: This step is also time-consuming, but since you’re not running it, this shouldn't be the issue in your case.

Please let me know if you're happy with these changes.

The time_counter variable itself doesn't really need to be chunked but a chunk size of 1 does need to be applied to the time_counter dimension because most accesses of 2 and 3d slices/volumes will be done a time-slice at a time. All variables except time_counter have been chunked correctly so it is odd that time_counter has been treated differently. We should get to the bottom of that.
Yes, I think a flag to switch off the integrity check would be useful. The checks are useful when establishing new work flows but we may need to take risks to achieve a workable solution. At least such a flag will identify if the checks are indeed a major bottleneck.
If it helps, I see the priority order as: 1. flag to suppress integrity checks 2. Dask robustness 3. time_counter chunking. 4. Suppress unwarranted warning

Hi @accowa ,
I've implemented the following changes:

Flag to suppress integrity checks: Added a new flag skip_integrity_check (-si or --skip-integrity-check). By default, it's set to False (integrity checks are applied). I'll update the NEMO_CYLC documentation and codes accordingly.

Dask integration for checksum calculation: I've integrated Dask for the checksum calculation, which in my tests has shown a 2-5x speed improvement. However, this performance boost will vary depending on the machine and whether Dask is already being used to load data into the object store.

time_counter chunking: I’ve added debug output (it is not on the code in production) to investigate why the time_counter variable isn’t being chunked as expected:
if len(new_chunking.keys()) > 0:
    print(f"Rechunking {variable} to {new_chunking}")
    ds_filepath[variable] = ds_filepath[
        variable
    ].chunk(new_chunking)
    print(f"New chunking: {ds_filepath[variable].chunks}")

The output I'm seeing is:
Rechunking tossq_con to {'time_counter': 1, 'x': 720, 'y': 360}
New chunking: ((1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1), (360, 360, 360, 360, 360, 360, 360, 360, 360, 360, 5), (720, 720, 720, 720, 720, 720))
Rechunking nav_lat to {'x': 720, 'y': 360}
New chunking: ((360, 360, 360, 360, 360, 360, 360, 360, 360, 360, 5), (720, 720, 720, 720, 720, 720))
Rechunking nav_lon to {'x': 720, 'y': 360}
New chunking: ((360, 360, 360, 360, 360, 360, 360, 360, 360, 360, 5), (720, 720, 720, 720, 720, 720))
Rechunking time_centered to {'time_counter': 1}
New chunking: ((1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1),)
Rechunking time_counter to {'time_counter': 1}
New chunking: None

It seems that time_counter isn't being chunked even though the code runs without errors. To clarify, the time_counter variable is not the same as the time_counter dimension—it merely contains the names of each time_counter timestamp. While it's odd that time_counter is being handled differently, it doesn’t impact the chunking strategy or data itself. I’ll open a new issue to address this in more detail later.

Suppress unwarranted warnings: I’ve adjusted the warning so it will only appear if the chunk strategy being applied differs from the original one. No warnings will be shown if the chunk strategy remains unchanged.

The time_counter non-chunking is likely a Xarray bug. This looks relevant: https://github.com/pydata/xarray/issues/6204

Thanks for pointing me to this link. Considering this, I think the only way to solve this problem now is to rename the coordinate (I don't recommend this), or use the zarr library (and not xarray) the first time I upload the data. In this case, right after uploading the data, I would open the time_counter variable and rechunk it to the chosen strategy. The other data that will be appended later will automatically follow the chunk strategy defined in the first upload.


[BUG] OpAccumulator does not compile with C++20

Environment
**Operating System: Rocky Linux 8
Describe the bug
class OpAccumulator in ValueTransformer.h uses parallel_reduce but does not fulfill its requirements. The causes OpenVDB to not work with C++20 compilers and oneTBB 2021.11 (I think starting from 2021.7 ). There is an explanation about the issue in this thread:
https://github.com/oneapi-src/oneTBB/issues/965
The tbb documentation states the following:
https://oneapi-src.github.io/oneTBB/main/tbb_userguide/Constraints.html
https://spec.oneapi.io/versions/latest/elements/oneTBB/source/named_requirements.html

I can confirm this is solved in v11.0


Crabstudio is now agiletechvn

Change org link

Too bad composer/packagist still doesn't have a decent strategy to handle package renaming.
Related https://github.com/composer/packagist/issues/496.


[Bug] typos and grammar in some of the step markdown files

Summary

Fix a few typos and grammar related items in a few of the step markdown files.
How to reproduce

line 27
Step 1: Enable CodeQL
Navigate the to [to the] Settings tab at the top of your newly created repository.
^ reorder "the to" to "to the"
line 83
Step 2: Review and Triage CodeQL Alerts
Open one of the open alerts that CodeQL [identified] from the scan.
line 12
Step 4: Prevent Vulnerabilities in the Pull Request
This is going to be the same issue [we] initially saw.
line 12
Finish
CodeQL scanned the code in your repository and notified us of two SQL injection vulenrabilities [vulnerabilities].

Additional context

Please see the hyperlinks above.

@mjbear Thank you for submitting this issue, and linking your PR! We will get the PR you linked triaged for review ✨


Added the audited gem, which will quietly save every change made to the PageContent table

Context
The audited gem will now watch for changes to PageContent and save changes to the new audited table
We don't need to do anything else until somebody wants to know about the page history
This is not versioning, its an audit trail
Changes proposed in this pull request
Added audited to PageContent
Added the audited gem
Guidance to review
Run the new migration
Run the app and edit a page
Look in the database to see who changed what when
This is what the database table looks like on my dev machine after running the seed data

The database size could grow very quickly here - it will start showing impacts in space consumption, like, increase in database size and backup size and also time taken for database backup.
Should audit events be stored in another database?


Disabling the "Script Editor" Editor Feature makes it impossible to Detach Scripts

Godot version
4.2.dev
System information
Windows 11
Issue description
The "Detach the script from the selected node" icon is removed when disabling the "Script Editor" feature.
Existing and newly Attached scripts cannot be Detached.

The same is true for the "Detach Script" context menu item.

However, since it's still possible to Attach a script via drag & drop from Filesystem, disabling this editor feature is detrimental to a development workflow and provides a dead end workflow (due to the missing button).

New scripts can still be created in the FileSystem tab (but cannot be created via the hierarchy context menu)

Suggested fix: Keep the Detach Script functionality available under all circumstances, making it possible to have a clean workflow without the script editor always stealing focus. It would also be good to always be able to Create Scripts via context menu.
Dream fix for the future: In addition to suggested fix, clicking the script icons (or double-clicking script resources) would actually open them in the chosen external editor. (probably out of scope for the fix, or a goal for a configurable extension/plugin)
Steps to reproduce

Open Editor->Manage Editor Features
Disable the "Script Editor" feature
Observe: Scripts can no longer be detached from their nodes via the top-of-hierarchy icon.
Observe: Scripts cannot be created nor detached via the context menu on any given node.

Minimal reproduction project
No "project" repro possible, Editor Features are managed somewhere outside the projects.
(actually would suggest to save that in project.godot - so the correct editor config can be loaded for a given project :))

I think this might be a matter for discussion, one reason to disable scripting is for teaching, to restrict students in how they interact, preventing them from accidentally detaching a script makes a lot of sense to me

I think this might be a matter for discussion, one reason to disable scripting is for teaching, to restrict students in how they interact, preventing them from accidentally detaching a script makes a lot of sense to me

Yes, I wanted to suggest a "pure fix" which would also disable attaching scripts. However, 99% of use cases, even during teaching, would at the very least involve attaching and detaching in some fashion. So it would miss the main reason people want to hide the script editor - to have a pure scene workflow.
However, there could be a "Scripting" editor feature that encompasses all of the above. Disabling "Script Editor" should disable the editing of scripts in said internal editor, I would say.


[wip] Add basic trusted setup and kzg

Built based on Dankrad's post: https://dankradfeist.de/ethereum/2020/06/16/kate-polynomial-commitments.html
The PR's Mission

Create SRS for G1 & G2 groups ✅
Example polynomial pen & paper ✅
MSM to create the commitments ✅
Open to an arbitrary evaluation ✅
Pairing to verify opening ✅

Possible Next Missions

Batched KZG
Hiding commitment
Type Abstractions, VerifierKey, KZGParams Abstraction
Integrate with Ronkathon Polynomials, Curves & Fields
Abstraction to support plonk-style KZG

Content for helping to understand this process: https://dankradfeist.de/ethereum/2020/06/16/kate-polynomial-commitments.html


react native 0.26

React and Component should now be imported from react instead of react-native

This is already the case @daavidjobe, update to the last version

My bad! Thank you


[SCB-331] log optimization

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.01%) to 87.258% when pulling ca06d7c9821aa90dd4b77bb82c6be93ca0491f4c on laijianbin:SCB-331 into bbe18e493aa3fb3d1531f7e6bee806e22d9994b4 on apache:master.

Need english speaker to check. I think it's not better than before.


kubernetes-helm: 3.4.2 -> 3.5.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/helm/versions.
meta.description for kubernetes-helm is: "A package manager for kubernetes"
meta.homepage for kubernetes-helm is: "https://github.com/kubernetes/helm"
meta.changelog for kubernetes-helm is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

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

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

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

found 3.5.2 with grep in /nix/store/z0pd7ip0rxms9imcw7641a579jkkl2gw-helm-3.5.2

directory tree listing: https://gist.github.com/cfbc0a053aa3cd4cf1f4ab8356debdc8

du listing: https://gist.github.com/ea6a8b3ec4105c273fd8b6d4e3270c29

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

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
helmfile
kubernetes-helm

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/z0pd7ip0rxms9imcw7641a579jkkl2gw-helm-3.5.2
ls -la /nix/store/z0pd7ip0rxms9imcw7641a579jkkl2gw-helm-3.5.2/bin

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

2 packages built:

helmfile
kubernetes-helm

Maintainer pings
cc @rlupton20 @edude03 @saschagrunert @Frostman @Chili-Man @kalbasit for testing.

Result of nixpkgs-review pr 112314 at 9588338a run on aarch64-linux 1

2 packages built:

helmfile
kubernetes-helm

Thanks!


Fix formatting in access-usb-devices-on-the-web

Whoops!
There were 9 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L5 - WF Tag wf_updated_on must be within the last 7 days.
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L8 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L46 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L75 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L139 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L166 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L299 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L305 - Line must be at most 100 characters
src/content/en/updates/2016/03/access-usb-devices-on-the-web.md#L325 - Line must be at most 100 characters


allow adding custom options to guzzle

Currently it is not possible to add custom options to the used guzzle client.
This pr gives the option to pass a third, optional paramater $guzzleOptions to ShopwareClient _construct.
The array is merged with the default options for base_uri and  handler.
This is not a BC-break since the additional parameter is optional an by default an array.

@redshark1802 I'll have a look at it tonight

@redshark1802 Please adjust readme.md

I've updated my pr.
Is this sufficient?


feat: allow opt-in to virus definitions bucket policy mutation

Resolves #1306

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@dontirun Thank you! 🙏


Round 2 GitHub Intro

[] Doing this one more time for the people in the back

Close the issue


add afterError callback

A callback for if there's an error while loading content would be useful.

Great feature, good PR. Sorry I didn't get to it sooner.
To make the PR perfect, there would be a test for afterError callback, and a brief documentation would be added. Can you improve the PR?


Fix shared workflow permissions

After merging #43, there were some private repositories that were having issues checking out source code in GHAs.
Further investigation revealed that by omitting the other keys in the permissions block that was added, we were effectively setting them to none (src).
To remedy this, this PR includes the following changes:

Keeps the id-token permissions as write, but set the remaining permissions as described in the restricted column here

We can always adjust this if needed, but I think this is fine

Moves the permissions block to the top of each workflow so that it's applied to all jobs in each workflow file

Do we need any “pull-request” permissions to check any metadata on a given PR? Wondering if that matters for our “copy PR” workflows.

Nope, that's handled separately in the ops-bot. This won't affect it.

Also why do we need “packages: read”?

I just used the default values listed under the restrictive column here: https://docs.github.com/en/actions/security-guides/automatic-token-authentication#permissions-for-the-github_token

Are we missing keys like “metadata”?

Are we missing keys like “metadata”? edit: seems like all other keys are listed.

see this excerpt (src):

You can use the permissions key in your workflow file to modify permissions for the GITHUB_TOKEN for an entire workflow or for individual jobs. This allows you to configure the minimum required permissions for a workflow or job. When the permissions key is used, all unspecified permissions are set to no access, with the exception of the metadata scope, which always gets read access.

metadata was also omitted in the example below, which is why I omitted it.

https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs#overview

Thanks for clarifying. I missed that in the docs.


First Azure Pipeline to switch on Azure DevOps.

To enable Azure DevOps we need to have a simple azure-pipelines.yml file in the master branch.
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?


Any SSR plans?

Hello!
Do you have any plans on supporting SSR?
I am currently working on building a similar stack but yours look way more elegant.
Here it is:
https://github.com/christophediprima/razzle-react-redux-observable-found
It supports server side rendering and pre-fetching based on route resolution. I am thinking about trying to replace my redux solution with typeless... any pre-advice would be very welcome! :)

Hi,
I am planning to implement SSR when React.lazy officially supports it.

Do you also have some ideas on how you will handle pre-fetching?
I am using following library https://github.com/BerkeleyTrue/react-redux-epic and dispatching actions in the "getData" prop of my routing components.
I don't think React.Lazy will help on waiting for calls in epics to be finished...

Something like this:

dispatch LOCATION_CHANGE
your epics should listen for this action, and load initial data
in root epic, you should count pending observables.
For example:
https://github.com/typeless-js/typeless/blob/master/packages/typeless/src/Epic.ts#L76
on start, increase counter by 1
subscribe to created defer, wait for 'complete', and decrease counter by 1
if the counter is 0, it means all data is loaded,  and you can hydrate your app

Also you can check PR from redux-observable https://github.com/redux-observable/redux-observable/pull/539


Regression in<EMAIL_ADDRESS>Cannot read property 'install' of undefined (cli-plugin-mocha)

Information

Version: 3.3.2
Install Method: sudo yarn global add @tsed/cli

I've found a regression in 3.3.2 that was not in 3.3.1.
On version 3.3.2, this happens, while on 3.3.1 it finishes correctly.
? Choose the target platform: Express.js
? Choose the convention file styling: Ts.ED
? Check the features needed for your project GraphQL, Database, Passport.js, Socket.io, Testing, Commands
? Choose a ORM manager TypeORM
? Which TypeORM you want to install? MariaDB
? Choose unit framework Mocha + Chai + Sinon
? Choose the package manager: Yarn
  ↓ Write RC files [skipped]
  ✔ Install plugins
  ✔ Load plugins
  ❯ Install plugins dependencies
    ✖ Run plugin '@tsed/cli-plugin-mocha'
      → Cannot read property 'install' of undefined
      Run plugin '@tsed/cli-plugin-passport'
      Run plugin '@tsed/cli-plugin-typeorm'
      Install
TypeError: Cannot read property 'install' of undefined
    at CliHooks.emit (/usr/local/share/.config/yarn/global/node_modules/@tsed/cli-core/lib/services/CliHooks.js:17:68)
    at Task.task (/usr/local/share/.config/yarn/global/node_modules/@tsed/cli-core/lib/services/CliPlugins.js:33:43)
    at /usr/local/share/.config/yarn/global/node_modules/listr/lib/task.js:167:30
    at processTicksAndRejections (node:internal/process/task_queues:94:5) {
  context: {
    root: '.',
    tsedVersion: 'latest',
    production: false,
    rawArgs: [],
    platform: 'express',
    convention: 'default',
    packageManager: 'yarn',
    projectName: 'tsed',
    graphql: true,
    db: true,
    passportjs: true,
    socketio: true,
    testing: true,
    commands: true,
    typeorm: true,
    mariadb: true,
    mocha: true,
    features: [
      [Object], [Object],
      [Object], [Object],
      [Object], [Object],
      [Object], [Object],
      [Object]
    ],
    srcDir: 'src',
    npm: false,
    yarn: true,
    express: true,
    koa: false,
    platformSymbol: 'PlatformExpress',
    verbose: undefined
  }
}

Thanks @jojoxd  for the opened issue ;)

:tada: This issue has been resolved in version 3.3.3 :tada:
The release is available on:

v3.3.3
GitHub release

Your semantic-release bot :package::rocket:


Fixed inability to click the horizontal gallery left/right button by updating zIndex.

What
Fixed inability to click the horizontal gallery left/right button by updating zIndex.
Why
https://skype.visualstudio.com/SPOOL/_queries/edit/2674355/?fullScreen=true&triage=true
How Tested
Manually
Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

@JamesBurnside @prprabhu-ms I will create a new PR for this change to show only changes related to bug-2674355.


ci(helm/testing): use origin to pick last version

Helm testing fetches the last release branches to work out what version it should be set to.
Description

Motivation and Context

Regression

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

bors merge


Support net40

This aims to enable cleanly supporting net40 targets for #1.
The strategy here is to

Remove all usages of IReadOnlyCollection<T> and IReadOnlyList<T> from the public API. This should help avoid strange runtime issues when running net40 on net45 runtime, etc.
introduce#RESHAPED_REFLECTION for internal reflection API usages. Currently, they're all throwing NotImplementedException.

TODO

[ ] Make sure some tests exercise the failing implementation on net40. This is complicated by the fact that xunit and dotnet-test-xunit do not support net40, so we would need another way to execute the tests.

This now fixes #3

@adamchester looks like you're powering through it :-) If there's anything you want a second set of eyes on just drop me an @ :-) Cheers!

@nblumhardt any better idea for running net40 tests? Best I could come up with is new csproj with linked existing test files and xunit console. It seems to work, just wondering if you any better ideas?

@adamchester this is pretty much the approach we used to take with Serilog. May be a better option but seems okay to me :-)