Installer Action Node16 Deprecation Notice

I noticed that the installer Action is still using Node16 and throws a warning at users since it has been deprecated. Would it be possible to bump this to Node20? Looking at the code, I don't think you're doing anything funky that would be a problem with an upgrade.
Side-note, but somewhat related, I noticed the Action doesn't support other platforms even though slsa-verifier does, and is a bit dated in general. I would be happy to volunteer to give it some love and have things align with some newer standards/practices. 😄

Yes, we would welcome a PR. By other platforms, do you mean other OSs?

@ramonpetgrave64 - Yes, I mean support for Windows and macOS. I see binaries are built for these platforms so I figured it would make sense to have the Action also support them.

Alright, it could take a bit of work to get it to work for other OSs, but the node20 upgrade seems simple enough.

https://github.com/slsa-framework/slsa-verifier/blob/767ecf9e0a63f5b7699d023609bd01978ff00d75/actions/installer/action.yml#L26

You're welcome to test it on your fork and submit a PR. I'll be happy to merge.

Thanks for testing it out.
I can't say for sure if it's intentional. But looking at Github's example, it seems like tc.downloadTool() is being used properly, and the other method calls aren't necessary. Does this sound right?

https://docs.github.com/en/actions/sharing-automations/creating-actions/developing-a-third-party-cli-action

Regarding the ref, it will actually work if the user wants to install older versions of slsa-verifier.

Users can still pin to a release tag if they want. The version input is optional, and pinning to a release tag will install whichever version was latest at the time of release (i.e. @1.2.3 will install v1.2.3).

I can't say for sure if it's intentional. But looking at Github's example, it seems like tc.downloadTool() is being used properly, and the other method calls aren't necessary. Does this sound right?

https://docs.github.com/en/actions/sharing-automations/creating-actions/developing-a-third-party-cli-action

This will only download the files to the location you specify but not actually cache them. They will only be moved into the hostedtoolcache once you call the tc.cacheFile() or tc.cacheDir() - the example shown does not actually utilize caching, it's only using the @actions/tool-cache package to perform the download. The docs for the package itself mention needing to use this. https://github.com/actions/toolkit/tree/main/packages/tool-cache#cache

Regarding the ref, it will actually work if the user wants to install older versions of slsa-verifier.

This is sort of true, but would not be the case for the new OSes we're adding support for and any of the additional fixes to the Action itself. Using something like @v1.2.3 would still work, but it would use the Action as it was made at tag v1.2.3, meaning they would still get an older Action, Node version, etc. This could cause some inconsistent behavior and I wasn't sure if this was desirable.
eg: If someone wanted to use @v2.5.1 to install SLSA Verifier for Windows it would fail because the Action at that tag does not support Windows.

Are you sure those are not just for self-hosted runners? What I'd like to only ever cache from the direct download link.
Thanks for clarifying about the ref. I'll invite @ianlewis to discuss.

Based on my experience and testing, even on GH runners you will need to tell it to actually cache the files. There may be other things behind the scenes that GH does to speed up repeated downloads, but I don't believe this is part of the @actions/tool-cache package based on the code. It looks like it will always write out to a file from the Stream, which implies (to me) that's it's downloading it every time.
The debug logs during the tc.cacheFile() indicate that this is the point at which it will cache a file(s)
##[debug]Caching tool slsa-verifier 2.6.0 x64
##[debug]source dir: /tmp/slsa-verifier_3ZrkE0/v2.6.0
##[debug]destination /opt/hostedtoolcache/slsa-verifier/2.6.0/x64
##[debug]finished caching tool

The debug logs for tc.downloadTool() show that it downloads the file each time.
##[debug]Downloading https://github.com/slsa-framework/slsa-verifier/releases/download/v2.6.0/slsa-verifier-linux-amd64
##[debug]Destination /tmp/slsa-verifier_3ZrkE0/v.2.6.0/slsa-verifier
##[debug]download complete

I could be misunderstanding something, but I don't think adding some logic to account for this is hard to do anyways. I've actually got all this added and tested in my fork, but I can easily yoink stuff out!

Since performance seems perfectly fine with the cache not actually working, like you said, I'm for removing the cache code. I've also read some blogs about cache poisoning, so it seems like it's not worth the trouble.

I think you're correct that performance is probably fine without it, but I think it would be nice to utilize it, especially for people who do use this on self-hosted runners (such as myself). Cache poisoning is definitely a good point though and not something I accounted for in my fork.
I think a good balance for this would be to cache the binary, but not cache the signature data and always validate the binary regardless of being cached or not (including for the bootstrap binary). This should ensure that any tampering would be noticed by the user. This would at least reduce the total bandwidth necessary when a cache-hit does occur.
Another option (which I have now) is to simply give the user a cache input option to override the behavior to avoid or accept the risks.

We could verify the provenance of the slsa-verifier binary that is retrieved from the cache (we do this anyway after download). This would also verify that it was the expected slsa-verifier version (preventing downgrade attacks etc).
However, as stated, it's likely just downloading it each time has adequate performance. If tc.downloadTool doesn't actually cache then I've actually never seen another action that utilizes the cache properly as I've only seen actions that simply call tc.downloadTool (e.g. the golangci-lint installer)

Verifying the bin in cache is something I can add in and probably something I should have started with haha
I'm not sure what else to say about the tool-cache - that repo is also not properly caching downloads since it is not calling the tc.cacheDir() or tc.cacheFile(), which again is what actually caches the files into the hostedtoolcache directory. The tc.downloadTool() function is simply a utility function to download a file from a URL. GH docs on this are rather poor since they essentially gloss over this details, but reviewing the code I genuinely don't think it will cache files just by using downloadTool alone...
https://github.com/actions/toolkit/blob/main/packages/tool-cache/src/tool-cache.ts#L410
https://github.com/actions/toolkit/blob/main/packages/tool-cache/src/tool-cache.ts#L113
I updated my fork to deliberately skip the cache even if it's found to trigger the exact same tc.downloadTool() call on the same URL on the same runner, and in the debug logs I see that it downloaded it both times.
I think it's fine to just discard the caching entirely if that's the route we want to go

@IAreKyleW00t
I agree the docs are not well written. I had originally hoped that it was caching the files on github's servers. But looking at the code it seems like tc.downloadTool()  only actually downloads, and nothing else.
I think we can should leave our use of tc as is, unless you've already figured out the cache verification problem, then we can take a look.

@ramonpetgrave64 Any chance someone is going to look at that PR? Happy to make adjustments or reduce the scope of needed.

@IAreKyleW00t , I've been away for a w while, and I've just taken a look at your PR. Yes, please reduce the scope of the PR for now.

Sure thing, though I've lost interest in this and don't actually need those additional features or OS support, so I'll get it updated to Node20 and we can just move on.
#811 is created with only the Node20 changes.

@ramonpetgrave64 Before we close this out, since I have the other work already done, how would it be best to get those changes in? Do you want me to split up the changes into smaller PRs and just open them one at a time?

@IAreKyleW00t Yes, and we really should have opened separate Issues for the other concerns. Would you open them or edit this issue's title and description?

@ramonpetgrave64 Yes, I can open separate issues for these features/fixes since this was originally just for Node20. I'll split up the changes and open issues to address them as I'm ready


how to use gtk_css_provider in gotk3

how to use gtk_css_provider in gotk3? It's very important!

I already learned how to use it


terragrunt plan -out && terragrunt show gives an error

Some third-party tools want to use terragrunt run-all plan -out and terragrunt show sequentially for different purposes.
I am aware of this known issue, hence I try to get a tfplan via only plan  instead of plan-all but still the show command outputs me an error related to a dependency.
For example, here is a plan command and its stdout. As you may see there is no problem with plan
$  terragrunt plan  --terragrunt-ignore-external-dependencies  -out <<file-path>>

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # aws_elasticache_parameter_group.default[0] will be created
  + resource "aws_elasticache_parameter_group" "default" {
      + arn         = (known after apply)
      + description = "Elasticache parameter group"
      + family      = "redis4.0"

and this is the result of the show command.
$ terragrunt show $(find .  -name <<file-path>>)
ERRO[0021] <<path-to-dependency>>/terragrunt.hcl is a dependency of <<path-to-main-terragrunt>>/terragrunt.hcl but detected no outputs. Either the target module has not been applied yet, or the module has no outputs. If this is expected, set the skip_outputs flag to true on the dependency block.
ERRO[0021] Unable to determine underlying exit code, so Terragrunt will exit with error code 1

Could you help me to overcome this issue?

I had never thought that I would have to add show to the mock_outputs_allowed_terraform_commands  array. But still please be aware that running terragrunt run-all plan -out and terragrunt show sequentially gives an error.


TypeError: parse() takes from 1 to 5 positional arguments but 6 were given

When I run the 2N2F pdf (sent to you via email) I get the error below.
I'm using master branch and the following command line:
~/repos/pdftotree/bin$ ./pdftotree ~/data/2N2F.pdf

Traceback (most recent call last):
  File "./pdftotree", line 105, in <module>
    args.visualize)
TypeError: parse() takes from 1 to 5 positional arguments but 6 were given

Have you updated your code to v0.3.0?


[wip] add olm e2e test

creating pr to test build on github

closing because of https://github.com/gatekeeper/gatekeeper-operator/pull/92


⚠ Bofrost.at has degraded performance

In 461dcff, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2204 ms

Resolved: Bofrost.at performance has improved in ecd002b after 20 minutes.


Exogenous Markov chain for the wealth distribution?

Hi,
Is it possible to add an exogenous Markov chain for the wealth state variables? I am trying to add a Blanchard-Yaari type of death probability.
I have tried to add a Pi_death Markov chain to the household block:
def death_probability(xi, nA):
    Pi_death = np.zeros((nA, nA))
    Pi_death[:,0] = xi
    Pi_death += np.eye(Pi_death.shape[0]) * (1-xi)

    return Pi_death

with the heterogenous block defined as
@het(exogenous=['Pi_death', 'Pi_e'], policy='a', backward='Va', backward_init=household_init)
def household(Va_p, a_grid, e_grid, rh, w, beta, eis, xi):

    uc_nextgrid = (1-xi) * beta * Va_p
    c_nextgrid = uc_nextgrid ** (-eis)
    coh = (1 + rh) * a_grid[np.newaxis, :] + w * e_grid[:, np.newaxis]
    a = interpolate.interpolate_y(c_nextgrid + a_grid, coh, a_grid)
    misc.setmin(a, a_grid[0])
    c = coh - a
    Va = (1 + rh) * c ** (-1 / eis)
    return Va, a, c

but I get the following error
"ValueError: matmul: Input operand 1 has a mismatch in its core dimension 0, with gufunc signature (n?,k),(k,m?)->(n?,m?) (size 500 is different from 7)"

Hi, if you use exogenous=['Pi_death', 'Pi_e'] to have two independent exogenous Markov chains, then the toolkit will expect the arrays Va, Va_p, a, and c to each have three dimensions, with the first two corresponding to the Pi_death and Pi_e Markov chains. You'd need to rewrite the household hetblock and household_init to be consistent with this.
An alternative approach is to use the Kronecker product on your own to consolidate Pi_death and Pi_e into a single one-dimensional Markov chain Pi (and then take a Kronecker product of np.ones(2) and e_grid to get the corresponding grid of endowments over this chain, etc), and then work with that. This approach is often a bit simpler because it doesn't require as many changes to the code, although it requires a bit more manual work with the Markov chain. It is also computationally a bit less efficient than using two separate Markov chains, but I doubt in this case that would be a major issue.
One note about the specific application: I think it makes more sense to have Pi_death be a iid shock. Then, when this shock is received, you can set incoming assets to zero. (Also, if you want to have annuitization a la Blanchard, then I guess you'd want to inflate the gross return on assets by $1/(1-\xi)$. Without this, one needs to take a stand on what happens to the assets at death, e.g. whether they are redistributed somehow.)
Let me know if you have any questions about either of these approaches and if one works for you!

Thanks a lot for your help! I wrote a minimal working example, keeping the two exogenous processes and having "Pi_death" an iid shock.
import copy
import numpy as np
import matplotlib.pyplot as plt

from sequence_jacobian import het, simple, create_model              # functions
from sequence_jacobian import interpolate, grids, misc, estimation   # modules

def household_init(a_grid, e_grid, rh, w, eis):
    coh = (1 + rh) * a_grid[np.newaxis, np.newaxis, :] + w * e_grid[:, np.newaxis, np.newaxis] + np.array([0,0])[np.newaxis, :, np.newaxis]
    Va = (1 + rh) * (0.1 * coh) ** (-1 / eis)
    return Va

@het(exogenous=['Pi_e','Pi_death'], policy='a', backward='Va', backward_init=household_init)
def household(Va_p, death_grid, a_grid, e_grid, rh, w, beta, eis, xi):

    uc_nextgrid = beta * Va_p
    c_nextgrid = uc_nextgrid ** (-eis)
    coh = (1 + rh) * death_grid[np.newaxis,:,np.newaxis] * a_grid[np.newaxis, np.newaxis, :] + w * e_grid[:, np.newaxis, np.newaxis]

    a = np.empty_like(coh)

    for i in range(e_grid.size):
        for j in range(death_grid.size):
            a[i,j,:] = np.interp(coh[i,j,:], c_nextgrid[i,j,:] + a_grid, a_grid)

    a[a<0] = a_grid[0]
    c = coh - a
    Va = (1 + rh) * c ** (-1 / eis)

    return Va, a, c

def make_grid(rho_e, sd_e, nE, amin, amax, nA):
    e_grid, _, Pi_e = grids.markov_rouwenhorst(rho=rho_e, sigma=sd_e, N=nE)
    a_grid = grids.agrid(amin=amin, amax=amax, n=nA)
    death_grid = np.array([1,0])
    return e_grid, Pi_e, a_grid, death_grid

def death_probability(xi):

    Pi_death = np.array([[1-xi, xi],
                   [1-xi, xi]])

    return Pi_death

household_ext = household.add_hetinputs([make_grid, death_probability])

@simple
def firm(K, L, Z, alpha, delta):
    r = alpha * Z * (K(-1) / L) ** (alpha-1) - delta
    w = (1 - alpha) * Z * (K(-1) / L) ** alpha
    Y = Z * K(-1) ** alpha * L ** (1 - alpha)
    return r, w, Y

@simple
def death_insurance(r, xi):
    rh = r / (1-xi)
    return rh

@simple
def mkt_clearing(K, A, Y, C, delta, xi):
    asset_mkt = A - K
    goods_mkt = Y - C - delta * K - xi * A

    print('K = ', float(K))
    print('A = ', float(A))
    return asset_mkt, goods_mkt

ks = create_model([household_ext, firm, mkt_clearing, death_insurance], name="Krusell-Smith")
print(ks.inputs)

calibration = {'eis': 1, 'delta': 0.025, 'alpha': 0.11, 'rho_e': 0.966, 'sd_e': 0.5, 'L': 1.0,
               'nE': 7, 'nA': 500, 'amin': 0, 'amax': 200, 'xi':0.02}
unknowns_ss = {'beta': 0.98, 'Z': 0.85, 'K': 3.}
targets_ss = {'r': 0.01, 'Y': 1., 'asset_mkt': 0.}
ss = ks.solve_steady_state(calibration, unknowns_ss, targets_ss, solver='hybr')

calibration = ss
unknowns_ss = {'K': 3}
targets_ss = {'asset_mkt': 0.}

Hello! uc_nextgrid = beta * Va_p should still be fine. Va_p is the expected partial value function tomorrow. The expectation is taken automatically with respect to every exogenous state.  Pi_death in particular is also applied.


@TransactionalRestController, is that something really needed?

Just trying to spark some discussion about the @TransactionalRestController
I always considered putting @Transactional on your web layer is a bad practice as the layer below that should drive your transaction (especially if you have different ways of exposing your services, SOAP, REST, ...).
So I would argue that this annotation is promoting an anti-pattern. Of course I can refrain from using it but adding this to the framework might lead to people thinking that this is a good idea (or pattern) to use.

I would go even further and question whether we need @Transactional**Component in the first place. @Transactional @Controller is barely longer than @TransactionalController, so where's the convenience? I just see plenty of permutations here and no real gain.

Plus you're looking at using cglib or putting @RequestMapping on an interface. Hardly convenient.

Hi guys,
I agree that @TransactionalRestController is likely not the best idea and more likely an anti-pattern.
With that in mind, I do not believe we would ever include this as an official annotation in Spring, but my goal thus far is to allow the community to brainstorm about the possibilities with composed annotations and @AliasFor. So we'll leave @TransactionalRestController as it is for now and let the discussions continue. :wink:

@jhoeller, are you arguing against @TransactionalService and @TransactionalRepository as well, or just against @Transactional*Controller?

Closing this issue since we are all in agreement that @TransactionalRestController is not a good idea for inclusion in the Spring Framework.


Start writing a warn-error framework for F*

Just like for patterns, the code can be nicely copy/pasted from KreMLin.

Since F* now has support for --warn_error, closing this issue.


iOS offset encoding

This was exactly my issue.  Thank you for the help page!  I set my debug project to ARM64 and now it compiles.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0fb56313-2c64-de22-673c-d0534aea4943
Version Independent ID: c9fa9166-6090-a71d-b229-87f0eca959aa
Content: Compile error: Can not encode offset X in resulting scattered relocation - Xamarin
Content Source: docs/ios/troubleshooting/questions/error-encode-offset-scattered-relocation.md
Product: xamarin
Technology: xamarin-ios
GitHub Login: @chamons
Microsoft Alias: chhamo

Glad it was useful! Thanks for the kind words.


DataSore does not return any result when cursor pointing to entity gets updated but retains its position in list

I'm using google datastore to get data of a user:
This is what i'm trying to do:

When data is updated, its updated_at [indexed] property gets set to current timestamp.
I query data on updated_at in ascending order and store cursor returned for later use.
Now user has updated the last entity (which cursor points currently) and no other data is added or updated.
Now i'm expecting that last entity to be returned in next query (using that old cursor) because it was updated and now has a new updated_at timestamp.
But that is not the case, i do not see that (my result is empty list) And now i have lost that update completely because query will return all the other object except that last entity that was updated.

I'm is doing something wrong or this is the way it is?
If this is natural behaviour then what is preferred way to get that last entity that was updated?

updated_at does not look like a Java variable name. Can you provide more information? (A sample code to demonstrate your use case, which method you called etc)

updated_at is not java variable it is a indexed property in DataStore Kind.
I'm closing this issue because i asked same question on stackoverflow.com and got an answer.


MultiselectDropdown.html:1 ERROR TypeError: Cannot read property 'containerClasses' of undefined

How to resolve this error ?
MultiselectDropdown.html:1 ERROR TypeError: Cannot read property 'containerClasses' of undefined
at Object.View_MultiselectDropdown_0.co [as updateDirectives] (MultiselectDropdown.html:1)
at Object.debugUpdateDirectives [as updateDirectives] (core.es5.js:12806)
at checkAndUpdateView (core.es5.js:12144)
at callViewAction (core.es5.js:12507)
at execComponentViewsAction (core.es5.js:12439)
at checkAndUpdateView (core.es5.js:12150)
at callViewAction (core.es5.js:12507)
at execComponentViewsAction (core.es5.js:12439)
at checkAndUpdateView (core.es5.js:12150)
at callViewAction (core.es5.js:12507)

I got the same error ->
TypeError: Cannot read property 'containerClasses' of undefined
at Object.View_MultiselectDropdown_0._co [as updateDirectives] (MultiselectDropdown.html:1)
at Object.debugUpdateDirectives [as updateDirectives] (core.js:14339)
at checkAndUpdateView (core.js:13508)
at callViewAction (core.js:13858)
at execComponentViewsAction (core.js:13790)
at checkAndUpdateView (core.js:13514)
at callViewAction (core.js:13858)
at execComponentViewsAction (core.js:13790)
at checkAndUpdateView (core.js:13514)
at callViewAction (core.js:13858)
also with this one ->
TypeError: Cannot read property 'searchRenderLimit' of undefined
at MultiselectDropdown.get [as searchLimit] (dropdown.component.js:116)
at MultiselectDropdown.get [as searchLimitApplied] (dropdown.component.js:130)
at MultiselectDropdown.updateRenderItems (dropdown.component.js:193)
at MultiselectDropdown.ngOnChanges (dropdown.component.js:166)
at checkAndUpdateDirectiveInline (core.js:12092)
at checkAndUpdateNodeInline (core.js:13598)
at checkAndUpdateNode (core.js:13541)
at debugCheckAndUpdateNode (core.js:14413)
at debugCheckDirectivesFn (core.js:14354)
at Object.View_AuditorSearchComponent_0._co [as updateDirectives] (AuditorSearchComponent.html:75)
any ideas how to solve it?

Similar error after updating the version from 1.3.2 to 1.6.3
TypeError: Cannot read property 'containerClasses' of undefined     at Object.eval [as updateDirectives] (ng:///MultiselectDropdownModule/MultiselectDropdown.ngfactory.js:1227)     at Object.debugUpdateDirectives [as updateDirectives] (vendor.bundle.js:43657)     at checkAndUpdateView (vendor.bundle.js:42995)     at callViewAction (vendor.bundle.js:43358)     at execComponentViewsAction (vendor.bundle.js:43290)     at checkAndUpdateView (vendor.bundle.js:43001)     at callViewAction (vendor.bundle.js:43358)     at execComponentViewsAction (vendor.bundle.js:43290)     at checkAndUpdateView (vendor.bundle.js:43001)     at callViewAction (vendor.bundle.js:43358)

Resolved : I found a junk dropdown in my code <dropdownselect [myOptions]="fooBar"></dropdownselect>
It never threw an error in v1.3.2.
I think undefined values are not handled from some version after v1.3.2.

Resolved : I found a junk dropdown in my code, without input values [texts] and [settings]. It never threw an error in v1.3.2.
I think undefined values are not handled from some version after v1.3.2.


Remove concrete casts for BatchHandle

As the client methods are taking an interface arg, we should stop casting to an internal implementation of BatchHandle in code.
Change is required for work on CDS, which is adopting this client nuget.
Co-Authored-By: Samir Hassoun<EMAIL_ADDRESS>

PR created for permissions test. Closing with no action


k8s versions > 1.9.3: ConfigMap mount for gethconfig not writable

cat /etc/testnet/bootnode/enode.address
ENODE=enode://d3b75b24a4fd718b1358d28ba576b2e98f73a7465326c4504f21cc0d124466a91919de18fb72a634f9108f0eedd5ef943aea5c250b41c974c4a7fb7c159b9968@<IP_ADDRESS>:30303
sed s@//@\/\/@g
echo enode://d3b75b24a4fd718b1358d28ba576b2e98f73a7465326c4504f21cc0d124466a91919de18fb72a634f9108f0eedd5ef943aea5c250b41c974c4a7fb7c159b9968@<IP_ADDRESS>:30303
ENODE_ESC=enode://d3b75b24a4fd718b1358d28ba576b2e98f73a7465326c4504f21cc0d124466a91919de18fb72a634f9108f0eedd5ef943aea5c250b41c974c4a7fb7c159b9968@<IP_ADDRESS>:30303
sed -i s/BootstrapNodes = []/BootstrapNodes = ["enode://d3b75b24a4fd718b1358d28ba576b2e98f73a7465326c4504f21cc0d124466a91919de18fb72a634f9108f0eedd5ef943aea5c250b41c974c4a7fb7c159b9968@<IP_ADDRESS>:30303"]/g /etc/geth/miner/gethconfig.toml
sed: can't create temp file '/etc/geth/miner/gethconfig.tomlXXXXXX': Read-only file system
sed -i s/BootstrapNodesV5 = []/BootstrapNodesV5 = ["enode://d3b75b24a4fd718b1358d28ba576b2e98f73a7465326c4504f21cc0d124466a91919de18fb72a634f9108f0eedd5ef943aea5c250b41c974c4a7fb7c159b9968@<IP_ADDRESS>:30303"]/g /etc/geth/miner/gethconfig.toml
sed: can't create temp file '/etc/geth/miner/gethconfig.tomlXXXXXX': Read-only file system

this affects any kubernetes version from 1.9.4 upwards
https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.9.md#changelog-since-v193
Changes secret, configMap, downwardAPI and projected volumes to mount read-only, instead of allowing applications to write data and then reverting it automatically. Until version 1.11, setting the feature gate ReadOnlyAPIDataVolumes=false will preserve the old behavior.
there needs to be a future proof way of doing that. one thing would be to not use the geth config file anymore, but then we'd have this huge list of args to geth which is not really nice

easiest fix/workaround is to copy the config file to another location and then edit and use it


Use upstream error types in CompressionError

This makes cdrs compatible with error-chain.

Coverage increased (+0.1%) to 94.946% when pulling bb4b42aa99fd301f8cae4cfa374df8b2b8aa0cfc on dfaust:compression-error into 64fed1cba3b8fc59bfa77a5f32d78440460aebfe on AlexPikalov:master.

Codecov Report

Merging #104 into master will increase coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #104      +/-   ##
==========================================
+ Coverage   95.53%   95.56%   +0.02%
==========================================
  Files          18       18
  Lines        1860     1870      +10
==========================================
+ Hits         1777     1787      +10
  Misses         83       83

Impacted Files
Coverage Δ

src/compression.rs
96.52% <66.66%> (+0.33%)
:white_check_mark:

Continue to review full report at Codecov.

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

Coverage increased (+0.1%) to 94.946% when pulling bb4b42aa99fd301f8cae4cfa374df8b2b8aa0cfc on dfaust:compression-error into 64fed1cba3b8fc59bfa77a5f32d78440460aebfe on AlexPikalov:master.


How to use Chinese date format in DateTimePicker's header?

I like GIJGO's widgets very much because they are simple and useful.
I translated the messages.js to Chinese and I used akiotajima's calendar title (https://github.com/atatanasov/gijgo/pull/437/commits/ae32ee0888823ac054dec5deda87873383d550a5). Now I am wondering how to use Chinese date format in DatetimePicker's header. For example, November 21 is "11月21日" in Chinese. I don't know how to add "日" after date. Thank you very much!

Yes. This is going to be available in the new version. You can find the latest development build at:
https://github.com/atatanasov/gijgo/tree/1.9/dist


moji.getFile() throws ConcurrentModificationException under dozens of clients hit it simultaneously

  java.util.ConcurrentModificationException
    at java.util.ArrayList$Itr.checkForComodification(ArrayList.java:901)
    at java.util.ArrayList$Itr.next(ArrayList.java:851)
    at fm.last.moji.tracker.pool.MultiHostTrackerPool.getAddresses(MultiHostTrackerPool.java:82)
    at fm.last.moji.impl.Executor.<init>(Executor.java:36)
    at fm.last.moji.impl.MojiFileImpl.<init>(MojiFileImpl.java:56)
    at fm.last.moji.impl.MojiImpl.getFile(MojiImpl.java:52)
    at fm.last.moji.spring.SpringMojiBean.getFile(SpringMojiBean.java:52)

I have not trace it in deep, just log it in here, and maybe someone else has the same problem.

I have traced the code, and the reason that causes the exception may be here:  https://github.com/lastfm/moji/blob/master/src/main/java/fm/last/moji/tracker/pool/MultiHostTrackerPool.java#L234
Collections.sort() causes "structurally modification", so ConcurrentModificationException is thrown by ArralyList. synchronized block can not prevent the exception occurred.

In my opinion, an active offical repos is good for users :+1:

I am not sure that the concurrency impact about synchronization in MultiHostTrackerPool#getAddresses().
An other option is CopyOnWriteArrayList, but it costs in memory copy.

I wonder it is JDK bug.
https://bugs.openjdk.java.net/browse/JDK-8064864
(but state is Cannot Reproduce...)
"managedHosts" is not changed by multi threads(only reading).
Furthermore this problem probably occurred only using JDK 8 update 20 or later.

The point is that Collections.sort(managedHosts, HostPriorityOrder.INSTANCE) in nextHost() method raises ConcurrentModificationException.
So how about using Collections.max(managedHosts, HostPriorityOrder.INSTANCE); instead of "sort()".
"max()" does not modify list itself, and execution cost is small.
I have tested this in my staging environment for two weeks and have no bug.
I will create a pull request if necessary.

Why not Collections.min()?

Original code of nextHost() method is below.
    synchronized (managedHosts) {
      Collections.sort(managedHosts, HostPriorityOrder.INSTANCE);
      managedHost = managedHosts.get(managedHosts.size() - 1);
    }

So I use Collections.max().

Sorry for my misjudge. You are right.
It happens that I got the same answer yesterday. In my opinion, the (your) answer is optimal.
I hope someone who has privilege will accept your PR, since this is big issue for moji in java 8.

Hey all, I used to work on this project when I was at Last.fm. Moji was a part of their streaming service which used it to fetch files stored in MogileFS. Last.fm have pretty much stopped their streaming service so I doubt they use Moji any more and by the lack of comments from them here don't have the resources/interest to maintain it. So I would suggest one of three things: (1) you fork it and apply your changes and maintain your own release; (2) I can try find someone working at Last.fm who might be interested/able to merge your PR in and do a release for you; (3) I can try find someone at Last.fm who could help handing this entire project over to new maintainers (if you are interested in being the new maintainers).

I want to be able to obtain Moji from Maven Central, so prefers (2) and (3)

I prefer (2) or (3), too. Because there might be some applications using moji.
I have the will to maintain moji for some time.Would you try to do (2) or (3)?

I have reached out to some people I know still working at Last.fm, let's give them a week or so and see if they get back to me. If not then I'm afraid we'll be left with option (1) :(

Hi guys, sorry for the lack of response on this issue.
As you might have guessed, we're not really maintaining it anymore internally.
I'm quite happy to merge any pull requests etc.. you might have, but we may want to consider finding a more enthusiastic maintainer in the long term.
For now though, I'll merge this and try and upload a new release to Maven Central for you.
So I guess we're going for a combination of 2+3? :)

Thank you massdosage and Hitobat!

Maven Central is now showing the new version.


workflow maintenance

What this PR does / why we need it:
This PR includes some necessary maintenance for the Github workflows.

updated dependencies to latest versions
replaced hard coded repository names
upgraded deprecated set-output commands to using Environment Files

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

/cla


[Bug]: Signing up as a new user from a public app does not work

Is there an existing issue for this?

[X] I have searched the existing issues

Description
App returns a 400 when a new user signs up using a shared public app URL
Steps To Reproduce

Make an app public
Copy share link to an incognito tab
Launch the application and try to sign-in from the top right corner
Create new user credentials and press enter - observe a 400 page as seen below displayed to the user

Public Sample App
No response
Version
Cloud

This is a duplicate of https://github.com/appsmithorg/appsmith/issues/12926 and is not related to clean url.


Memoize the ObjCBridgeable conformance for Swift.String

This seems to be the single most common bridge cast,
and repeated lookup seems to be a performance issue
for the common operation of bridging a string-valued
NSDictionary into Swift.  Spending a couple of static
words of memory to memoize it could be a nice perf win.
Resolves rdar://55237013

@swift-ci Please benchmark

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
3695
5598
+51.5%
0.66x (?)

FlattenListLoop
2773
3183
+14.8%
0.87x (?)

PrefixArrayLazy
13
14
+7.7%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9400
7200
-23.4%
1.31x

ObjectiveCBridgeToNSSet
11450
9000
-21.4%
1.27x

ObjectiveCBridgeFromNSStringForced
1780
1535
-13.8%
1.16x (?)

ObjectiveCBridgeStubFromArrayOfNSString2
2120
1860
-12.3%
1.14x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
29400
26600
-9.5%
1.11x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
3300
3000
-9.1%
1.10x (?)

ObjectiveCBridgeFromNSSetAnyObjectToString
46000
42000
-8.7%
1.10x (?)

DictionaryBridgeToObjC_Access
558
510
-8.6%
1.09x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
28900
26600
-8.0%
1.09x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2800
2580
-7.9%
1.09x (?)

ObjectiveCBridgeStubFromNSDateRef
2540
2370
-6.7%
1.07x (?)

Code size: -O
Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

StringInterpolationManySmallSegments
8800
10700
+21.6%
0.82x (?)

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9250
7200
-22.2%
1.28x

ObjectiveCBridgeToNSSet
11450
9050
-21.0%
1.27x

ObjectiveCBridgeFromNSStringForced
1765
1540
-12.7%
1.15x (?)

ObjectiveCBridgeStubFromArrayOfNSString2
2100
1870
-11.0%
1.12x (?)

DictionaryBridgeToObjC_Access
558
511
-8.4%
1.09x (?)

Code size: -Osize
Performance: -Onone

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9800
7600
-22.4%
1.29x

ObjectiveCBridgeToNSSet
11850
9500
-19.8%
1.25x

ObjectiveCBridgeStubFromArrayOfNSString2
2080
1860
-10.6%
1.12x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2840
2560
-9.9%
1.11x (?)

ObjectiveCBridgeFromNSString
2165
1955
-9.7%
1.11x (?)

ObjectiveCBridgeFromNSStringForced
1840
1665
-9.5%
1.11x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
30200
27600
-8.6%
1.09x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
30000
27500
-8.3%
1.09x (?)

DictionaryBridgeToObjC_Access
650
601
-7.5%
1.08x (?)

TypeFlood
148
137
-7.4%
1.08x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 64 GB

@Catfish-Man  -- Any thoughts about FlattenListFlatMap and FlattenListLoop here?  I don't offhand see any reason those benchmarks would touch the Obj-C bridging path, which means they should be unaffected by this change.  Am I missing something?  Apart from that, the benchmarks here look nice.

FlattenListFlatMap has been odder than one might expect; I've been looking into it for the array changes I've been working on, and it doesn't respond to changes in a way I can make sense of.
I wouldn't worry about it too much but taking a quick sample/instruments profile locally might be worthwhile just in case.

Performance: -O

Improvement
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
6249
4124
-34.0%
1.52x (?)

ObjectiveCBridgeToNSArray
9500
7250
-23.7%
1.31x (?)

ObjectiveCBridgeToNSSet
11350
9100
-19.8%
1.25x (?)

FlattenListLoop
3365
2829
-15.9%
1.19x (?)

UTF8Decode_InitDecoding
167
141
-15.6%
1.18x (?)

ObjectiveCBridgeStubFromArrayOfNSString2
2130
1820
-14.6%
1.17x (?)

ObjectiveCBridgeFromNSStringForced
1800
1580
-12.2%
1.14x (?)

NormalizedIterator_fastPrenormal
660
580
-12.1%
1.14x (?)

Chars2
3500
3100
-11.4%
1.13x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
30600
27200
-11.1%
1.12x (?)

Calculator
155
138
-11.0%
1.12x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
29800
26800
-10.1%
1.11x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2820
2540
-9.9%
1.11x (?)

IterateData
878
794
-9.6%
1.11x (?)

StringHashing_fastPrenormal
590
540
-8.5%
1.09x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
3320
3080
-7.2%
1.08x (?)

OpenClose
61
57
-6.6%
1.07x (?)

Code size: -O
Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

FlattenListLoop
2653
2924
+10.2%
0.91x (?)

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9300
7000
-24.7%
1.33x

ObjectiveCBridgeToNSSet
11550
9050
-21.6%
1.28x (?)

ObjectiveCBridgeStubFromArrayOfNSString2
2130
1800
-15.5%
1.18x (?)

UTF8Decode_InitDecoding
164
139
-15.2%
1.18x (?)

Calculator
154
138
-10.4%
1.12x (?)

StringHashing_fastPrenormal
590
530
-10.2%
1.11x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
29800
26800
-10.1%
1.11x (?)

Chars2
3400
3100
-8.8%
1.10x (?)

Code size: -Osize
Performance: -Onone

Regression
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeStubFromNSDate
3020
3270
+8.3%
0.92x (?)

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9800
7350
-25.0%
1.33x

ObjectiveCBridgeToNSSet
11800
9450
-19.9%
1.25x

UTF8Decode_InitDecoding
185
156
-15.7%
1.19x

ObjectiveCBridgeStubFromArrayOfNSString2
2110
1850
-12.3%
1.14x (?)

ObjectiveCBridgeFromNSStringForced
1835
1615
-12.0%
1.14x (?)

ObjectiveCBridgeFromNSString
2135
1895
-11.2%
1.13x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
30600
27400
-10.5%
1.12x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
30200
27200
-9.9%
1.11x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2820
2580
-8.5%
1.09x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 64 GB

@swift-ci Please benchmark

Performance: -O

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9350
7100
-24.1%
1.32x

ObjectiveCBridgeToNSSet
11350
8750
-22.9%
1.30x

UTF8Decode_InitDecoding
164
138
-15.9%
1.19x

ObjectiveCBridgeStubFromArrayOfNSString2
2130
1840
-13.6%
1.16x (?)

ObjectiveCBridgeFromNSStringForced
1770
1540
-13.0%
1.15x (?)

NormalizedIterator_fastPrenormal
640
570
-10.9%
1.12x

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
29400
26600
-9.5%
1.11x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2800
2540
-9.3%
1.10x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
29100
26400
-9.3%
1.10x (?)

Chars2
3400
3100
-8.8%
1.10x (?)

Calculator
152
139
-8.6%
1.09x (?)

DictionaryBridgeToObjC_Access
552
506
-8.3%
1.09x (?)

OpenClose
61
56
-8.2%
1.09x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
3240
3020
-6.8%
1.07x (?)

Code size: -O
Performance: -Osize

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9300
6950
-25.3%
1.34x

ObjectiveCBridgeToNSSet
11200
8800
-21.4%
1.27x

UTF8Decode_InitDecoding
164
136
-17.1%
1.21x

ObjectiveCBridgeStubFromArrayOfNSString2
2090
1800
-13.9%
1.16x (?)

ObjectiveCBridgeFromNSStringForced
1770
1545
-12.7%
1.15x (?)

Calculator
154
136
-11.7%
1.13x

ObjectiveCBridgeFromNSArrayAnyObjectToStringForced
29600
26600
-10.1%
1.11x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
29100
26200
-10.0%
1.11x (?)

DictionaryBridgeToObjC_Access
577
523
-9.4%
1.10x (?)

Chars2
3400
3100
-8.8%
1.10x (?)

ObjectiveCBridgeStubToArrayOfNSString2
2760
2520
-8.7%
1.10x (?)

ObjectiveCBridgeFromNSDictionaryAnyObjectForced
4300
3950
-8.1%
1.09x (?)

NormalizedIterator_fastPrenormal
670
620
-7.5%
1.08x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
3260
3020
-7.4%
1.08x (?)

ObjectiveCBridgeFromNSSetAnyObjectToString
43500
40500
-6.9%
1.07x (?)

Code size: -Osize
Performance: -Onone

Regression
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeStubNSDateRefAccess
3323
3601
+8.4%
0.92x (?)

Improvement
OLD
NEW
DELTA
RATIO

ObjectiveCBridgeToNSArray
9600
7250
-24.5%
1.32x

ObjectiveCBridgeToNSSet
11750
9400
-20.0%
1.25x

UTF8Decode_InitDecoding
183
157
-14.2%
1.17x

ObjectiveCBridgeStubFromArrayOfNSString2
2130
1850
-13.1%
1.15x (?)

ObjectiveCBridgeFromNSArrayAnyObjectForced
5020
4400
-12.4%
1.14x (?)

ObjectiveCBridgeFromNSArrayAnyObjectToString
30300
26900
-11.2%
1.13x (?)

DictionaryBridgeToObjC_Access
645
599
-7.1%
1.08x (?)

StringToDataEmpty
750
700
-6.7%
1.07x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 64 GB

@catfish-man -- Thoughts?  Given the lack of interesting difference between these benchmarks, I'm inclined to back out the last change (caching Dictionary and Array conformances) and just keep the String conformance cache.

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - b13fb96f2a665d4dec2e85536ac4e77f243761ff

Build failed
Swift Test Linux Platform
Git Sha - b13fb96f2a665d4dec2e85536ac4e77f243761ff


Make parade an interface

To make testing easier.

Also fixes a race (eb7e4bb00729d434cd4d9d9ab5cbc0366bb69e12) on the database when listening.

Also fixes test DB raciness on #648 and even #641.  It wasn't manifesting before, which is why I could create the PR, but it is now.  If I pull them into each other in reverse order (first this one, then #648, then #641) then we should reach the desired state without breaking CI.

Thanks!  Pulling (was stacked behind #641 and #668)...


Move process.log into a single file in the root of the artifacts folder

As discussed offline, the logic should just output all process logs into a single process.log, rather than cutting it into chunks (with potential to mess up).
~The predicate should be changed into process==$processName, where processName is taken before start from the CFBundleExecutable Info.plist of the .app specified in package.json.~

The process==${} has been implemented already in #1877.

@d

This was about device.log (previously named process.log...). I don't think there's a case for this anymore.

Regarding the complete device (app) log – looks like it can be done in the scope of the next Artifacts refactor, in late stages of Remote Detox - #3743.
Related to #3092.


Fix resetting epoch loop restarting flag in LearningRateFinder

What does this PR do?
lr_finder does not reset fit_loop.epoch_loop.restarting to False which causes the validation check to trigger validation on the first advance() call because of this check.

Before submitting

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

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

Reviewer checklist

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

📚 Documentation preview 📚: https://pytorch-lightning--19819.org.readthedocs.build/en/19819/

Could you please check this fix @awaelchli , @carmocca ?

Thank you for the fix @clumsy, very nice!
It looks like the batch size finder would need the same fix as well. Are you interested to investigate it as well?

Interesting, batch_size_finder actually worked for me but I can check and make the change, sure.
The was a more critical bug for LR finder, I'm afraid, @awaelchli: https://github.com/Lightning-AI/pytorch-lightning/pull/19814

This flag reset is seem to be in place for batch_size_finder here, @awaelchli


Runners in the Shadows update .1 img link adjustments and minor additions

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

New image links now direct to github folder
New attribution for icon used in logo
Added quality section for implants
Added stranger lineages to lineage options

Character Sheet Info Roll20 Internal Use only.


Is this updated for Diablo Season 4?

I'd like to know which season these calculations are updated for (I'm not sure if it's necessary to update, if anything changed from S3 to S4). Also, I'd like to suggest a multi-language option. Here in Brazil, where I'm from, we'd like a tool like this to assist with calculations, but since it's in English, it's a bit challenging.
There has been discussion in the community about having a damage calculator with explanations about the calculations, and I found this project.

Hi @OdairRosario,
technically it hasn't been updated yet for S4, but the only difference is the armor cap of ~9200 for the defense. Some of the predefined modifiers don't exist anymore, but thats just a convenience issue. I'll try to update this in the next days.
About multi-language support. I'd like to add this, but there is no feature implemented yet to replace names and tool tips with text from a language specific source file. I'll keep it in mind though and try to implement support step by step whenever I do an update. For some parts it will be easy (like the predefined affixes) and for some it will be a bit more work (tool tips, menu entries etc.).

Any chance you have a full data base of everything that I could use AI with? I want it to do the calculations for me and find best stats and gear ect.

Any chance you have a full data base of everything that I could use AI with? I want it to do the calculations for me and find best stats and gear ect.

All I have is in this repository ^^. But there isn't really much "data" except some names of aspects etc. Blizzard changes the values all the time and in addition, players can have different rolls.
Not sure if an "AI" would be able to help you with this anyway, certainly not something like ChatGPT. What you'd need is a program that can calculate all possible results with the correct formulas and then maximize the result ;)


Add passt package to debian CI VMs

Ref: https://github.com/containers/podman/issues/17598

@sbrivio-rh would you mind glancing at the logs (linked above) to make sure the expected pasta versions are being installed?

@sbrivio-rh would you mind glancing at the logs (linked above) to make sure the expected pasta versions are being installed?

Yes, checked:
    debian: Setting up passt (0.0~git20230216.4663ccc-1) ...
    fedora:   Installing       : passt-0^20230222.g4ddbcb9-1.fc37.x86_64            371/371
perfect, thank you.


remove any possible email or username in properties sent to fb

i think it's here https://github.com/codecombat/codecombat/blob/2a46993f07b9cef654951c22cb27b982828ea345/app/views/core/AnonymousTeacherModal.js#L91 send the email and name to FB.
I'm not familiar with this anonymouseTeachermodal, so let's just delete the information sent to FB. we still track it in our log anyway.
fix ENG-1011

Summary by CodeRabbit

New Features

Enhanced Facebook Pixel event tracking with improved data privacy controls.
Added functionality to conditionally exclude sensitive properties from tracking based on event context.

emmm good question. then let's toFb=true by default.


Improvements to Task Search

Makes the following improvements to Task Search:

Changes the URL from /taskSearch to /task-search to be more in line with URLs in the rest of the UI.
Changes the header from (Global) Historical Tasks to Task Search.
Adds labels for the current search parameters so that it's possible to see current search terms (even if the fields have changed).
Removes the JSON button.
Changes the log button to be a file glyphicon rather than .... This is also changed in the Task History table in the request detail page.
Fix a bug that caused the table to not update when a new search is performed that returns 0 results.

Did it again... Forgot to cc @tpetr @kwm4385 @wolfd

LGTM

LGTM


Getting an unknown "command" error

Hey, I'm not sure if this repo's still maintained, but I've been getting this specific error on my git commit hook and I'm not sure why (I had no luck trying to resolve this on my own):

Here's the .pre-commit-config.yaml:
  - repo: https://github.com/pre-commit/mirrors-prettier
    rev: v2.2.1
    hooks:
      - id: prettier
        args: [--prose-wrap=always, --print-width=88]

It seems to work for some objects (in a test/dummy sense) but I always get that particular repetition of lines.

the error message indicates you're on windows -- can you provide more information / reproduction steps?
also in the future, don't screenshot text.  it's (1) unsearchable (2) inaccessible and (3) unhelpful

Sorry, I'll keep that in mind for next time.
Yes, I'm using pre-commit hooks on Windows PyCharm (2020.3.2), and the only thing I've done to reproduce this is using pre-commit run --all-files. When I used a Github Action from deepsource.io using the Prettier config, it didn't seem to replicate nor reflect that. I've also tried reinstalling it using pip and ran that command again and that still resulted in this:
prettier.................................................................Failed
- hook id: prettier
- exit code: 255

The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.
The syntax of the command is incorrect.

please include a repository and versions

the repository in use is private currently sorry, and not sure what versions you're referring too

pre-commit, git, etc.
perhaps try and take parts of that private repository and anonymize it and make an open source version?  without a reproduction or without being able to type on your computer I can't really help

pre-commit 2.11.1
git 2.30.1.windows.1
I'm using the latest on everything currently as it's not version-bound by features
An abstraction open-source variant I'm referring to is here.

this appears to be a command-length issue with prettier's .CMD file that they ship
pre-commit calls this executable with a command length that's just under the limit, but their .CMD file extends the command slightly blowing past windows' limit of 8192 characters
if I give a little more headroom for .cmd / .bat files it passes.  The tricky thing is it's impossible to determine a "correct" amount of padding here as a .cmd script could call another thing and add as many characters to the command string as it wants
relevant code:
https://github.com/pre-commit/pre-commit/blob/e431b2b89862213a7538c850338d1d24c1582eea/pre_commit/xargs.py#L140-L148

@DeltaXWizard do you think you could try the branch in pre-commit/pre-commit#1864 to see if that's enough to get this passing?


Feature request: Add a button to return to aggregate feed

Upon opening the app, articles from all feeds are displayed together. However, once clicking on a specific feed, I cannot return to the aggregate view. A button to return to this view would be a much appreciated UI improvement. As of now, I am closing and reopening the app as a workaround.

Hi, thanks for the idea. Instead of adding a button, I will make the feeds selection to be removed if selected feed is tapped.

That's a great solution, thanks!

Fixed on d03de5b1da2bc04cdb476d7d82d6c0b51803ad22
This will be included in next version


Fenced code blocks bug

See this gist for a test case: https://gist.github.com/1163474

Closing as won't fix (see sirthias/pegdown#30).


added finding matrix transpose using java

@Riyana6
Do follow and star this repository it will be helpfull and also check Web-Development repository for more contribution


[SPARK-20011][ML][DOCS] Clarify documentation for ALS 'rank' parameter

What changes were proposed in this pull request?
API documentation and collaborative filtering documentation page changes to clarify inconsistent description of ALS rank parameter.

[DOCS] was previously: "rank is the number of latent factors in the model."
[API] was previously:  "rank - number of features to use"

This change describes rank in both places consistently as:

"Number of features to use (also referred to as the number of latent factors)"

Author: Chris Snow<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Jenkins test this please

Test build #74818 has started for PR 17345 at commit 1359e07.

Test build #74818 has finished for PR 17345 at commit 1359e07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74818/
Test PASSed.

Merged to master


Great Job!

Thank you!

Thanks
I added the model link to the readme so it's easier to get started, thank you.


Adding taxonomical subheading

Closes: #658
Adding a one-sentence description for each menu item.

Or, you could make it a string property of a <ViewHeader>. That way, it's inside of the containing <header> tag in that component.

@WrathfulSpatula great suggestion. Makes sense. Done!


Added reset all privacy permissions for single app command

Add missing command for resetting all privacy permissions for a single app

Thanks to @7aqdxe6f for suggesting PR and thanks to @AmadeuszL for creating PR. You guys are amazing!

@AmadeuszL Thanks so much for your contributions! Would you consider creating PRs in the context of draft branch? I created draft (see https://draft.sunknudsen.com/privacy-guides) so we can preview PRs on CMS and/or have them peer reviewed before merging to master.

Would you consider creating PRs in the context of draft branch? I created draft (see https://draft.sunknudsen.com/privacy-guides) so we can preview PRs on CMS and/or have them peer reviewed before merging to master.

Absolutely! Thank you


test: update node disconnection test for behavior change

test: update node disconnection test for behavior change
For https://github.com/longhorn/longhorn/issues/5476
Signed-off-by: Yang Chiu<EMAIL_ADDRESS>

cc @longhorn/qa


Include timestamp of pledge in fulfillment verification

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?
While dogfooding pledge fulfillments, I saw a pledge of two with 'this is a resent pledge' -- would be helpful to have some sort of confirmation that the last pledge sent date matches that.

What feature or behavior is this required for?
Fulfillment data entry validation

How could we solve this issue? (Not knowing is okay!)
We should add a 'pledge sent on' date to the information confirmation in the pledge partial that draws from the pledge_sent_at field (or whatever it's called).

Anything else?

Happy to start working on this.

Happy to work on it.


[Bug Report] Error 403 Forbidden when using GET /2/users/:id/following with Twitter API v2 endpoint

Describe the bug
Hello,
I am currently subscribed to the $100 Basic Plan. I encountered a bug while using the Twitter API v2 endpoint with the GET /2/users/:id/following method.
When making the request, I received the following error message:
"403 Forbidden
When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal."
I want to highlight that my app is already attached to a project, and I am not the only user experiencing this bug. Several other users are also encountering the same issue.
Please investigate this bug and take the necessary actions to resolve it as soon as possible. If you require any additional information, such as screenshots or logs, please let me know.
Thank you for your attention to this matter.

I'm using the basic plan (100$ monthly) and untill this morning I was able to pull the followers/followings of the user.
I use tweepy and create tweepy.Client object using only the bearer_token.
My app is under the project, so that is not an issue.
This is so unreliable if every other day some new unexplainable error will pop up.

I am having the exact same problem since today.

We're having the exact same problem too! As of 2 hours ago. Why is the API so unstable?!

same here

We have the same issue.
We are already basic plan and paid 100$ every month.
But When we get someone followers by api, we got this error. We can get the follow list before but today we got this error.
And We detached the app and re-attach it to the project, but not be solved.
It's crazy for that we can not get any accurate information from the error message.
The additional information is that our Tweet caps - pull is fulled. Is it related to this？？？
Anyone help for that!!!

Having exactly the same issue on followers and following.
But getting a user from its username is fine.
I'm also using tweepy.

Has anyone found a fix yet?

Also getting this error
Api docs are returning a 404 (https://developer.twitter.com/en/docs/twitter-api/users/follows/api-reference/get-users-id-followers)
I hope this API is not getting deprecated?

Same issue here

Their updated documentation lacks GET /2/users/:id/following 😂 I can't believe this is an expensive paid API.
https://developer.twitter.com/en/docs/twitter-api/rate-limits#v2-limits

2023-06-29 14:55:12.695 ERROR 20580 --- [e-thread-pool-1] i.g.r.t.helpers.AbstractRequestHelper    : (GET) Error calling https://api.twitter.com/2/users/1574242047661207552/followers {"client_id":"27393576","detail":"When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal.","registration_url":"https://developer.twitter.com/en/docs/projects/overview","title":"Client Forbidden","required_enrollment":"Appropriate Level of API Access","reason":"client-not-enrolled","type":"https://api.twitter.com/2/problems/client-forbidden"} - 403
fuck...............................
followers   and  following  Don't work
............................
Somebody help me ？？？？？？？？？

https://developer.twitter.com/en/updates/changelog       Should be the official Twitter closed this interface, the basic version and pro version can not be used... If you want to use it, please upgrade the highest version... Twitter is crazy poor??

The changelog appears to be for the v1.1 API.
Here is the community developer forum topic: https://twittercommunity.com/t/get-2-users-id-following-is-now-broken/197526
No solutions there either as of now.

Twitter is crazy poor. Two api need pay $42000 a month..............

So disappointed in Twitter.... Do you know whether the old account can use V1.1 related interfaces? Can we use the old account to call the old V1.1 "follow" related interface?

What's the workaround for this? I guess we'll have to scrape?

same issue here using bearer token
403 Forbidden
When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal.

DId anyone find a solution?

not find.
---Original---
From: @.>
Date: Sun, Jul 23, 2023 14:44 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [twitterdev/Twitter-API-v2-sample-code] [Bug Report] Error 403Forbidden when using GET /2/users/:id/following with Twitter API v2 endpoint(Issue #130)
DId anyone find a solution?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

getting the same issue

Same problem here.

Any fix yet?

if you are using python tweepy, use the create_tweet instead of update_status
client = tweepy.Client(consumer_key =CONSUMER_KEY, consumer_secret =CONSUMER_SECRET, access_token =ACCESS_KEY, access_token_secret =ACCESS_SECRET)
client.create_tweet(text='Hello World!')

Is there any way to attach images without using "tweepy.API" as it gives error and "tweepy.Client" does not allow to attach images.

Since Twitter deleted the follow api on June 26, 2023, our company plans to launch third-party Twitter API services~including querying follow, followers and other enterprise-level APIs, to solve the problem that SME developers do not pay attention to the problem that the purchased enterprise-level API cannot be used , what specific Twitter data do you need access to? You can pull me into the discussion group or leave me a message ~ see if you can cooperate with my DC invitation code https://discord.gg/hk5uHqGAAd

我正在使用基本计划（每月 100 美元），今天早上我才能够吸引用户的关注者/关注者。 我使用 tweepy 并仅使用 bearer_token 创建 tweepy.Client 对象。 我的应用程序直到该项目下，所以这不是问题。 如果每隔一天就会弹出一些新的无法解释的错误，那就太不可靠了。
如果有人遇到任何变化（API 工作），请写在这里。谢谢！:)

Please contact me~~We have a solution

有人找到修复方法了吗？

Please contact me~~We have a solution

我们也有同样的问题。 我们已经是基本计划，每月支付 100 美元。 但是当我们通过 api 获取某人的关注者时，我们收到了这个错误。在我们获得关注列表之前，但今天我们得到了这个错误。 我们分离了应用程序将其重新附加到项目中，但没有得到解决。 太疯狂了，我们无法从错误消息中获取任何准确的信息。 附加信息是我们的推文上限 - 拉力已满。和这个有关系吗？？？
任何人都可以帮忙！

Please contact me~~We have a solution

有人找到解决办法了吗？

Please contact me~~We have a solution

也出现此错误
Api 文档返回 404 ( https://developer.twitter.com/en/docs/twitter-api/users/follows/api-reference/get-users-id-followers )
我希望这个API不会被荒废？

Please contact me~~We have a solution

自2023年6月26日Twitter删除follow api后，我公司计划推出第三方Twitter API服务~包括查询follow、follower等企业级API，解决中小企业开发者不关注的问题购买的企业级API无法使用，您需要访问哪些具体的Twitter数据？可以拉我进讨论组或者给我留言~看看能不能配合我的DC邀请码discord.gg/hk5uHqGAAd
原始邮件
发件人："Ambrose Ikpele"< @.*** >;
发件时间：2023/8/20 2:01
收件人："twitterdev/Twitter-API-v2-sample-code"< @.*** >;
抄送人："125569489"< @.*** >;"Comment"< @.*** >;
主题：Re: [twitterdev/Twitter-API-v2-sample-code] [Bug Report] 使用 GET /2/users/:id/following 与 Twitter API v2 端点时出现错误 403Forbidden（问题 #130）
还有修复吗？
—
直接回复此电子邮件、在 GitHub 上查看或取消订阅。
您收到此消息是因为您发表了评论。Message ID: @.***>

Anyone found a solution yet?

https://api.twitter.com/2/users/me?user.fields=public_metrics
Thats the solution to get total followers count

I said to get the ids of followers or the list of ids of fans, not the number of fans
---Original---
From: @.>
Date: Wed, Aug 30, 2023 21:34 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [twitterdev/Twitter-API-v2-sample-code] [Bug Report] Error 403Forbidden when using GET /2/users/:id/following with Twitter API v2 endpoint(Issue #130)
https://api.twitter.com/2/users/me?user.fields=public_metrics
Thats the solution to get total followers count
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I wasn't replying to you

GET /2/users/:id/following
I am currently experiencing a problem when using consumer_ Key and consumer_ 401 Unauthorized appears during secret authentication. Using Bearer_ Token undergoes 2.0 authentication and appears as: 403 Forbidden
When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal.
I am certain that all the information is correct because all other interfaces can be used normally. I'm thinking how to solve this problem? thanks

Twitter.utools.me
---Original---
From: @.>
Date: Fri, Nov 10, 2023 17:39 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [twitterdev/Twitter-API-v2-sample-code] [Bug Report] Error 403Forbidden when using GET /2/users/:id/following with Twitter API v2 endpoint(Issue #130)
GET /2/users/:id/following
I am currently experiencing a problem when using consumer_ Key and consumer_ 401 Unauthorized appears during secret authentication. Using Bearer_ Token undergoes 2.0 authentication and appears as: 403 Forbidden
When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal.
I am certain that all the information is correct because all other interfaces can be used normally. I'm thinking how to solve this problem? thanks
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

加我微信<PHONE_NUMBER>1
---Original---
From: @.>
Date: Fri, Nov 10, 2023 17:39 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [twitterdev/Twitter-API-v2-sample-code] [Bug Report] Error 403Forbidden when using GET /2/users/:id/following with Twitter API v2 endpoint(Issue #130)
GET /2/users/:id/following
I am currently experiencing a problem when using consumer_ Key and consumer_ 401 Unauthorized appears during secret authentication. Using Bearer_ Token undergoes 2.0 authentication and appears as: 403 Forbidden
When authenticating requests to the Twitter API v2 endpoints, you must use keys and tokens from a Twitter developer App that is attached to a Project. You can create a project via the developer portal.
I am certain that all the information is correct because all other interfaces can be used normally. I'm thinking how to solve this problem? thanks
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Close the issue , use this Twitter.utools.me

Facing same issue with basic plan, anyone have solution yet??

create_tweet

also  403 Forbidden

GET /2/users/:id/以下
我目前在使用consumer_Key和consumer_Key时遇到问题，在秘密认证过程中出现401 Unauthorized。使用 Bearer_ Token 会经历 2.0 身份验证，并显示为：403 Forbidden 对 Twitter API v2 端点的请求进行身份验证时，您必须使用附加到项目的 Twitter 开发人员应用程序中的密钥和令牌。您可以通过开发者门户创建项目。
我确信所有信息都是正确的，因为所有其他接口都可以正常使用。我在想如何解决这个问题？谢谢

我通过第三方服务解决的验证关注者信息~~需要的话加VX  cs_125569489

创建推文

还有403禁止

Facing same issue with basic plan, anyone have sol

ution yet??

我确信所有信息都是正确的，因为所有其他接口都可以正常使用。我在想如何解决这个问题？谢谢

Hello, how did you solve the issue?

我确信所有信息都是正确的，因为所有其他接口都可以正常使用。我在想如何解决这个问题？谢谢

Hello, how did you solve the issue?
ApidDoc: https://utools.readme.io/ http://twitter.utools.me


Multiple choice need to be more "random"

When multiple choice mode is active during a quiz (can only occur if there are more than 4 cards), the first 4 cards are predetermined.  This can be seen by what the answers are for the first question.  The following cards in the quiz will be the front sides to the answers of the first question.
estimate 100pts

100 xp
W18 OK


[NVIDIA][Launcher] NV Cooperative Grid Launching (CU_LAUNCH_ATTRIBUTE_COOPERATIVE)

This change sets the launch grid attribute before calling cuLaunchKernelEx.
This change is intended to pair with load/store atomics from https://github.com/triton-lang/triton/pull/5187
and is intended to add grid synchronization similar to what cooperative groups do.
@ptillet Any recommendations on the UI for using this in code would be most welcome :-)
New contributor declaration

[X] I am not making a trivial change, such as fixing a typo in a comment.

[X] I have written a PR description following these
rules.

[X] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[?] This PR does not need a test because:

I am not entirely sure how to test the use of one driver API attr versus another for this case yet.

Select one of the following.

[X] I have not added any lit tests.
[ ] The lit tests I have added follow these best practices,
including the "tests should be minimal" section. (Usually running Python code
and using the instructions it generates is not minimal.)

@ngimel I've cleaned up the diff and addressed your feedback.


Python GUI Calendar App

Is your feature request related to a problem? Please describe.
A Calender feature to view day, date and month of any year.
Describe the solution you'd like.
A GUI Calendar built using Python Tkinter Module as a solution to the problem described above. I'd like to be assigned this issue under GSSOC'22.
Describe alternatives you've considered.
None.
Add any other context or screenshots about the feature request here.
None.

Can you assign me this issue @abhilipsasahoo03

Hi @Kushal997-das sorry for the delay. The delay was caused by my end sem exams. However I'll be making the pull request by tomorrow.

Can you assign me this issue ?

Can you assign me this issue ?

Sorry it's already been assigned .

Hi @Kushal997-das sorry for the delay. The delay was caused by my end sem exams. However I'll be making the pull request by tomorrow.

It's Okey @abhilipsasahoo03

@abhilipsasahoo03 Should we close the PR?

@abhilipsasahoo03 Should we close the PR?

Hi, sorry I've been facing some medical issues for the past weeks, but I'll try to make a PR by end of the day. If I'm unable to do so by EOD, you are free to close the PR or assign someone else.

@abhilipsasahoo03 So sorry to hear that take care of yourself .


reCaptcha: cat and mouse game

Hi @VeNoMouS :)
Since today, 16:30 PM (UTC), I have the error :
Cloudflare reCaptcha detected, unfortunately you haven't loaded an anti reCaptcha provider correctly via the 'recaptcha' parameter.
For example in https://pro-src.com/ and http://japscan.to/
Cat and mouse game, I guess :P
Thank you !

LOL, glad you saw that cat & mouse reference on 1.1.19 ;P

Sorry about the delay, i only just saw the tickets... CF is marking ECDHE-RSA-AES128-GCM-SHA256 as bad cipher will push fix for you now.

Refer #53

Thank you :)


machine_batch doesn't clearly mention which machine failed

When a converge fails in one of the machines in machine_batch, an exception is raised, which is then logged by chef as a failure in machine_batch resource.
This makes exception handling and debugging very hard, especially considering the log lines from different machines are all show in the output at the same time.
If machine_batch can wrap exceptions and add more details, any chef handler on :resource_failed event can then use this exception to identify the machine

:+1:

Fixed by merging https://github.com/chef/chef-provisioning/pull/500


Not working with postgresql enums

class CreateCleanerSurchargeRates < ActiveRecord::Migration
  def up
    execute "CREATE TYPE cleaner_surcharge_value_type AS ENUM ('multiplier', 'additional')"
    create_table :cleaner_surcharge_rates do |t|
      t.column :value_type, :cleaner_surcharge_value_type, null: false, index: true

# Could not dump table :cleaner_surcharge_rates because of following StandardError
#   Unknown type 'cleaner_surcharge_value_type' for column 'value_type'

class CleanerSurchargeRate < ActiveRecord::Base

It seems like the ActiveRecord schema dumper doesn't support postgres enum types (at least that's the case with ActiveRecord 4). Here is a possible workaround.
I'm not sure this will ever get fixed in active_record-annotate. Eventually DbSchema will have it's own annotation mechanism which annotates every postgres feature supported by DbSchema.


Adding G6 touchscreen receiver support

Passes down whether this is a G4, G5, or G6 receiver in the form of a pair of booleans during Dexcom class instantiation.
Must merge https://github.com/openaps/dexcom_reader/pull/19 into dexcom_receiver before adding this or it breaks support for the receivers all together.

@bewest or @oskarpearson could you add one of us as maintainers of dexcom_reader so we can do a release there?
@ecc1 is this something we also/instead want to do in your dexcom code?

I can do something similar, but don't have any post-G4 receivers to test with, so would need a guinea pig.

@ecc1 Assuming your dexcom code is a library of some kind and you write up a test harness I don't mind running it.


fix(curriculum): inconsistencies for code examples in cat photo app

Summary of changes
As I was helping campers on the forum, I noticed there were inconcentices and when we show code examples for new concepts and when we don't.
For example in step 8, we explan what a src attribute is and provide an example
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-by-building-a-cat-photo-app/step-8
But then in other places like step 14, we just tell them to add target attribute with _blank without a code example
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-by-building-a-cat-photo-app/step-14
However, this is popular post on the forum where campers don't know what to do and are just guessing at syntax
https://forum.freecodecamp.org/search?q=step 14 %23html-css after%3A2024-03-01 cat photo app
Also, sometimes we have code examples inline like this
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-by-building-a-cat-photo-app/step-10
And then other times we don't.
IMO, we should always follow this format when it comes to introducing new concepts

teach the concept
show an example. Preferably in a code block.
give the directions on what to do in the project

This PR fixes the code example inconsistencies for new concepts and should help with the number of queries we get on the forum.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Honestly, looking at some of these steps, the tests are so ancient that they're still using jQuery.
Before shoving more examples into the steps, I think I'd rather see the tests + hint text updated to better guide the campers in a granular way.

@naomi-lgbt
I pushed an update to remove some of the code examples.
And left comments for the others.
We could remove the remaining code examples steps where I left comments and see if updated the hint text and tests will work instead

Looks Good To Me! 👍


[NEW PROJECT]: Benchmark Spikeinterface Over Lifetime With AirSpeed Velocity

Project title:
Key Investigators

Heberto Mayorquin

Willing to work with anyone interested in benchmarks, willing to offer guidance or learn from someone else.
Project Description
Some of the standard packages on the ecosystem use air-speed velocity to benchmark the package across time. Here are two examples:

scikit-learn
scipy

The idea is simple, implement the same for SpikeInterface.
Background
Objectives

Create a repository with benchmarks and documentation

Approach and Plan
Fill in with approach and project plan

[ ] - Discuss with main mantainers and users what are common functionalities that are expected to last in time (i.e. saving data and loading data).
[ ] Implement the benchmark for at least one function
[ ] Add another function
[ ] Document and

Progress
Fill in with tentative milestones (or leave blacnk for later):

[ ] Milestone 1
[ ] Milestone 2
[ ] Make PR to SpikeInterface

Next Steps (optional)
Fill in with next steps
References
TBD

This sounds so cool - very keen to learn how to implement air-speed velocity.


No way to disable ion-tabs

I've had a few client specs that involve disabling tabs depending on some condition. Examples include "is the customer signed in?" and "does the customer have a premium account?".
There doesn't seem to be a way to achieve this using classes as the way ionic passes the class attribute down to a tab doesn't allow the class to be changed after the tabs have been digested. Maybe I'm wrong but I couldn't work it out. If there is a way to achieve this I'd love to know otherwise this is a feature / improvement request. Thanks.

This seems like a good idea. AFAIK there isn't a (clean) way to do this. We added support for a hidden attribute a while back, and I'd imagine disabling a tab is more common than hiding it. I'll get back to this once I have time to go through your PR.

Thanks for the quick response! I think the main point of contention in the PR will be my use of the disabled property rather than a class called "disabled". I think its neater to use a [disabled] selector but I can see there might be issues with it.
Also I'm didn't include a [disabled] style that sets pointer-events: none etc yet as I wasn't sure how disabled tabs would look in Ionic given the different styles (positive, assertive etc). In the applications we built using this patch the client provided disabled styles for us and they were different in both.

@adam187 we're just using this feature branch branch as our ionic version for now.
In our bower.json
"ionic": "https://github.com/beckyconning/ionic/tarball/feature-disableable-tabs",


Fix batch errors if server returns multiple request id

The echo policy is not robust enough to support multiple headers with same key.
After discussing with @johanste , the most secured fix right now for escrow is to disable this chek, and re-enable it when batch endpoints will be fixed.

(message created by the CI based on PR content)


Type highlighting in inspect_types

Feature request
It'd be useful to highlight all the inferred types in the inspect_types output so that, especially on GPUs, accidental upcast of types is trivial to spot. e.g. float64 is red, float32 is orange etc etc.

I will look in to this and see what I can do.

Great, thanks @BradyAtkinson


Footer can be improved

Footer can be improved by adding more details and social media. We can make more lists and navigation links to make it more informative.  please assign me the issue under Gssoc'24.

mr aman  sir plz assign issue to me

please assign it to me. I can fix it.
thank you.

more social media handles along with email id, and other things like aboutus, faq, privacy policy etc can be added in footer, please assign me this issue for GSSoc'24

Hi, I would like to work on this

Hi, I am Anushka Jain. Footer can be improved by adding more social links and details along with some animations in the background and logo too. I would like to work on it. Please assign it to me.

@scriptyvijay  i want to work on this issue please assign me this issue for GSSoc'24

Hi, I would like to work on this issue, please can you assign this to me.

@Its-Aman-Yadav please assign this task to me

@Its-Aman-Yadav I can do that please assign it to me under GSSOC.

@Its-Aman-Yadav . I Can do this .please assign this to me


Home indicator obscures screen buttons

Test Device
iPhone 13
Operating System
iOS 16.1.1
Browser
Safari 16.1 Mobile
Problem Description
When in landscape mode, the home indicator obscures the screen buttons. I first noticed this while testing pH Scale (https://github.com/phetsims/qa/issues/872), but I don't think it's a sim-specific issue. This issue occurs in published sims.
Steps to Reproduce
Open any sim in landscape mode.
Visuals

@kathy-phet told me that the bar isn't a scrollbar, but a home indicator. It is always there because it indicates where you swipe to go back to the home screen.

Deferring this issue. I read about it, and don't believe its possible to hide that indicator button unless you make your own app. Users can rotate vertically.
Our app nicely elevates the sim, so it doesn't collide with the home indicator.


[fix](oracle scan) Fix performance issues caused by version judgment

run buildall

run buildall

run buildall

run buildall


Attestations: Email/Phone/Facebook/Twitter

(1) This updates the phone verification to return an actual signature for the attestation. The signature is compatible with the current implementation of ERC725 that we're using.
(2) Adds email verification using Sendgrid
(3) Adds Facebook & Twitter verification using Oauth
Finishes https://github.com/OriginProtocol/bridge-server/issues/22

A casual reading through code looks a good start?
Do we need anything in the README telling you what API keys you'll need to get for a full setup?

@DanielVF Yeah, we have dev.env which lists them, but that's probably not very helpful. Added a few lines to the readme that document the API keys being used.
Side note: I just noticed there is at least one python library for Twitter. Might be one for Facebook too. Somehow the documentation I was looking at linked me to some examples which I used to guide my implementation. Using twitter/facebook specific libraries would probably make the code cleaner, but I don't think it's worth sinking time into right now.

Closing in favor of https://github.com/OriginProtocol/bridge-server/pull/27. (Creating a new PR because this branch was created on my fork of the repo, before I had access to create branches.)


module.exports.callbackPromise has bad syntax

That function has this line:
resolve(...args)

Is this correct JS syntax?

Sorry, didn't realise that this is the new spread operator in ES6


Work around GO15VENDOREXPERIMENT for Go 1.5

r? @dtbartle
The code to support GO15VENDOREXPERIMENT is in the Go 1.6 go/build package, but not the Go 1.5 one, so we have to implement a custom package finder to support vendor directories.
Spec is here: https://docs.google.com/document/d/1Bz5-UB7g2uPBdOx-rw5t9MxJwkfpx90cqG9AFL0JAYo/edit?pref=2&pli=1

lgtm


Kele should respect authorization for listing resources

Related to #66.
Many Kele operations tacitly assume that the user has full authorization within the given cluster. This is not a valid assumption. This is, for example, a long-standing issue with kubernetes-el that prevents entire swathes of potential users from leveraging it.
Take kele-get and kele-list, both of which assume that the user has list permissions for the resource in question (for completion for the former, by definition for the latter). Attempting to use either of these commands in such circumstances results in an error like follows:
error in process sentinel: HTTP error

This is consistent with the underlying issue:
x curl <IP_ADDRESS>:8001/apis/apps/v1/deployments
{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "deployments.apps is forbidden: User \"EMAIL\" cannot list resource \"deployments\" in API group \"apps\" at the cluster scope: requires one of [\"container.deployments.list\"] permission(s).",
  "reason": "Forbidden",
  "details": {
    "group": "apps",
    "kind": "deployments"
  },
  "code": 403
}%

Each Kele suffix effectively needs some notion of authorization that either flat-out disables the suffix or falls back to an alternative experience in "unauthorized cases," e.g. disabling completion for kele-get when user doesn't have list permissions.
Definition of Done

Decision is made and recorded as to how to address the above
Changes implemented

Relevant API: SelfSubjectAccessReview
POST /apis/authorization.k8s.io/v1/selfsubjectaccessreviews

Example query:
POST /apis/authorization.k8s.io/v1/selfsubjectaccessreviews HTTP/1.1
Accept: application/json, */*;q=0.5
Accept-Encoding: gzip, deflate
Connection: keep-alive
Content-Length: 246
Content-Type: application/json
Host: <IP_ADDRESS>:8001
User-Agent: HTTPie/3.2.1

{
    "apiVersion": "authorization.k8s.io/v1",
    "kind": "SelfSubjectAccessReview",
    "spec": {
        "resourceAttributes": {
            "group": "apps",
            "resource": "deployments",
            "verb": "list"
        }
    }
}

Example response:
{
    "apiVersion": "authorization.k8s.io/v1",
    "kind": "SelfSubjectAccessReview",
    "metadata": {
        "creationTimestamp": null,
        "managedFields": [
            {
                "apiVersion": "authorization.k8s.io/v1",
                "fieldsType": "FieldsV1",
                "fieldsV1": {
                    "f:spec": {
                        "f:resourceAttributes": {
                            ".": {},
                            "f:group": {},
                            "f:resource": {},
                            "f:verb": {}
                        }
                    }
                },
                "manager": "HTTPie",
                "operation": "Update",
                "time": "2023-03-18T19:35:40Z"
            }
        ]
    },
    "spec": {
        "resourceAttributes": {
            "group": "apps",
            "resource": "deployments",
            "verb": "list"
        }
    },
    "status": {
        "allowed": true,
        "reason": "access granted by IAM permissions."
    }
}

curl -d<EMAIL_ADDRESS>-H "Content-Type: application/json" -X POST http://<IP_ADDRESS>:8001/apis/authorization.k8s.io/v1/selfsubjectaccessreviews


Update type comment for memberName property

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

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Looks great, thanks again! Really easy to miss this sort of stuff, so thanks for being thorough!


removes double !!

!!(arr.filter((item) => item.property === prop)[0].html)
this is far easier to read
arr.filter((item) => item.property === prop)[0].html

I'm just casting the value returned to it's boolean equivalent

Yup. The reason why the tests are failing were because you were checking for truthy value in essence you should be checking for defined or undefined. which is equivalent as in your use case
if it's defined it will always return true. if it's not it will return false anyways..
It's clean I just feel being more explicit and readability wins here


There are no bug reports

There is no bug!
Hey, come on. There should be bugs somewhere. Give me bug reports. I would be glad to debug them. So, yes- no bug is a bug here.
Other then bug reports, what can we do to help this project?
You can always provide me translations. Russian, Spanish, Chinese... you name it!
Also, you will notice it's crappy logo. Yes, I admit I'm very bad at design. You can provide me a nice logo (and maybe other graphical assets required in Google Play Store).
I want (this) function! & Questions?
You can always send mail to my application's support mail address (in Google Play store), or you can make issue here.

I can provide you translations to Spanish, Portuguese and French.

@ibanferreira Since there are no translation frameworks yet, you can just fork this project and make i18n string xml files using appropriate tool - and make PR request.
If you think using Transifex - https://www.transifex.com/ is a good idea, I'll make organization and allow other users to translate using that platform. For suggestion, make a seperate issue about that.
This issue is just for notification, and I didn't want to be too long.

Update, I made Transifex organization and project. You can upload current translated xml or propose new languages at https://www.transifex.com/mnetwork/pgpclipper/ as posted in front README.

Cool thanks!


Respond to direct mentions

feels more natural to me than !chaz. The client will autocomplete the mention, and you can do "@chaz respond please" without stepping on the whole command system ('!chaz respond' would be parsed as a command instead).
This depends on a headjack change to pass the full event back (to get at the mentions):
diff --git a/src/lib.rs b/src/lib.rs
index 73ad967..4cf4e4b 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -368,7 +368,7 @@ impl Bot {
     /// Useful for bots that want to act more like chatbots, having some response to every message
     pub fn register_text_handler<F, Fut>(&self, callback: F)
     where
-        F: FnOnce(OwnedUserId, String, Room) -> Fut + Send + 'static + Clone + Sync,
+        F: FnOnce(OwnedUserId, String, Room, OriginalSyncRoomMessageEvent) -> Fut + Send + 'static + Clone + Sync,
         Fut: std::future::Future<Output = Result<(), ()>> + Send + 'static,
     {
         let client = self.client.as_ref().expect("client not initialized");
@@ -381,7 +381,7 @@ impl Bot {
                 if room.state() != RoomState::Joined {
                     return;
                 }
-                let MessageType::Text(text_content) = &event.content.msgtype else {
+                let MessageType::Text(text_content) = &event.content.msgtype.clone() else {
                     return;
                 };
                 if !is_allowed(allow_list, event.sender.as_str(), &username) {
@@ -393,7 +393,7 @@ impl Bot {
                 if is_command(&command_prefix, body) {
                     return;
                 }
-                if let Err(e) = callback(event.sender.clone(), body.to_string(), room).await {
+                if let Err(e) = callback(event.sender.clone(), body.to_string(), room, event.clone()).await {
                     error!("Error responding to: {}\nError: {:?}", body, e);
                 }
             },

Yea this is much better. I realized this too after using it for a little bit but I hadn't gotten the chance to update it.
This does cause some issues when generating the context, and it doesn't really work like replacing the !chaz invocation yet, but I will fix those later. I started doing that yesterday but realized that I should just merge this in and add on the other features as needed.
Thanks so much :)


Add logging

Replace printing with logging. Forbid print statements in the linter.
PR checklist

[x] Has a descriptive title
[x] Has a descriptive body
[x] Includes a link to any issues closed by this PR, e.g. Closes #5.
[ ] Passes checks (to auto-fix a pre-commit.ci failure, add a comment
containing "pre-commit.ci autofix")

Merged with the wrong branch and reversed the merge. Will do more PRS


NEW Add extension point to Director::is_site_url

Adding an extension point to Director::is_site_url to allow developers to determine what URLs (potentially on an alternate domain/external) could still be considered as belonging to the site.
Happy to rename the name of the extension method if someone has a better one.
Replacing #9925 to target the 4.8 branch.

How does this handle an extension that returns false from checkIsSiteUrl?

@GuySartorelli via the array_filter mechanism, it excludes all products of the extension hook that return falsey values. So if all extensions return false the array would be empty and the if check would not pass.

So the extend method either returns an empty array when no extensions are applied or an array of return values from each check. And if no check returns Yes! then it's considered as not local.

Right! That makes sense. I missed the fact that it's after the default check.

Looks good
Sorry should have asked before to rebase/retarget to 4 since we're treating adding extension hooks as API additions

done


when will the training code be released?

Great work!
Wondering when the training code could be released as in the former issue said it will be released before July13~

Hello, as of now, we have made the forward function of Tag2Text available in open source as much as possible. You can find it here. Unfortunately, the timeline for open sourcing the remaining code is uncertain. We appreciate your understanding and patience.


⚠ Bofrost.at has degraded performance

In 91ee6ed, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1624 ms

Resolved: Bofrost.at performance has improved in dffe81b after 6 minutes.


Icon in spinner like MaterialEditText

Is it possible put an icon like you can do with MaterialEditText??

Not yet ... but I encourage you to propose a pull request to add this feature !