Determine name for Recv body type

We temporarily renamed the old hyper::Body struct to hyper::body::Recv in #2966. That was to unblock #2839, since we wanted to use the name for the trait instead. But, the name should be properly considered.
The purpose of this type is: an implementation of Body to represent bodies received from a remote. So, the client side would see this in the Response, and the server side would see this in the Request. It is not meant to be constructed by users as a "default" implementation.
Some names I've seen or considered before:

Recv
RecvStream/RecvBody
Streaming
Incoming
Remote
Wire/FromWire

It might help to envision the type signature users will frequently see it as, such as Request<Streaming> (or insert any other name in there).

Pinging because you may have opinions about the name: @hawkw @LucioFranco @davidpdrsn.

Node.js uses IncomingMessage. Making the connection to Body clearer though would be good. Perhaps IncomingBody?

Assuming that the user would never construct these themselves then I like IncomingBody.

ReqBody?

It's the body of a Request for servers, but it's the body of Responses for the client.

ReqBody doesn't work well because on the client it would be the RespBody.
I like RecvBody. I think the -ing forms are odd for a 'static type name, and I think it should be a name that includes Body because that's what it's called in the HTTP standard.

IncomingBody or RecvBody suffer from the same ambiguity then. From client's point of view, that would be outgoing body and sent body.

IncomingBody sounds good then.

Another similar name/idea would be IngressBody.

I think more explícit is better and because is a request and responde body, then i like HttpBody

I like the idea of hinting that it's a stream, I know for some it's confusing that you don't get the whole headers+body on the same await. So RecvStream, IncomingBody or something like BodyStream/StreamingBody would be my choice.


Documentation: fix typos

I found some typos. Please let me know if you have any feedback.
Thanks for the great project and documentation!

Looks great. Thanks for the patch!

Oh that was quick. Thank you!


[windows] IRGen: add support for DLL Storage semantics

What's in this pull request?
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

Add initial support for modelling DLL Storage semantics for global values.  This
is needed to support the indirect addressing mechanism used on Windows.

CC @rjmccall

@swift-ci Please test

The basic approach looks good, but please extrapolate from my comments so far. :)

@rjmccall updated based on the extrapolation of your original comments.  Please have another look!

@swift-ci Please test

@rjmccall post holiday ping :-).

Just some patch-cleanliness requests this time, thanks.

Updated.  Sorry, the patch is pretty old, and I was going through and inspecting all the sites that I found, so I had changed code around at one point.  I believe that only the sites where there are changes should now be part of the commit.  Thanks for the patience with this!

LGTM, thanks!

@swift-ci Please test and merge


Can we get Booru sites (ex: rule34)

Source name
Rule34
Source link
rule34.xxx
Source language
English
Other details
Hey there, I have looked around and was surprised to find there weren't many Boorus. While r34 is one of the main ones other like the ones in the below attachments could be good options.
I was thinking R34. App is similar in function to what tachi does but they charge for convenient features tachi has for free. I couldn't find Boorus repo list any of the issues or requests, so I thought I'd go ahead and make a post here and see what was going on or if extension/s for this kind of stuff could be made
I'm new to making these posts so if I need to redo it just let me know and I'll do my best to clean everything up
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

There are other specialized free and open source booru-browser for Android like Boorusphere. https://github.com/nullxception/boorusphere

Valid point, and I might just end up going with that option. This is first I've heard of BooruSphere, thank you for the suggestion, I'll check it out.
However the point in me making this request was to see if these extensions could be made so It would be more convenient and consolidate my apps. I'm aware apps/client for that stuff exist and I use a couple of them, but I'd rather have a AIO app then three of four with similar types of content but each app has different sources/features/content.


Hab Builder can use external OAuth providers

We need to be able to configure the builder to use external OAuth
Acceptance Criteria:

User can configure OAuth for all supported OAuth providers:

Built-in (Automate)
Azure Active Directory
GitHub
GitLab
Okta
BitBucket

User can generate an initial config with sensible commented fields

Is the desire here for this to be separately configurable for Habitat Builder specifically or are we going to teach Automate's auth stack how to talk to these services for everything.


Connect all mobile screens and components

Background
A clear and concise description of the problem and the affected users.
Action requested
A clear and concise description of what you want to happen.
Acceptance Criteria

[ ] When user starts a test, "Test questionnaire" experience should show up first before a test card is added to the page. If the user doesn't complete this questionnaire, then the tag associated with it should read "PENDING". If all questions have been completed, then the tag should read "COMPLETE"

Additional context
Add any other context or screenshots about the feature request here.

@emmastephenson any ideas how this ticket should be filled out? Idk if the title is self-explanatory enough to suffice or if we need more detail in the ticket description


⚠ Node - jp6 has degraded performance

In 9fe983c, Node - jp6 (http://jp6.yalaso.top/api/v1/ping) experienced degraded performance:

HTTP code: 200
Response time: 3591 ms

Resolved: Node - jp6 performance has improved in daf3b3d.


Respect ERC1271

Motivation
Allow ERC1271 orders

Solution
Offerer can be different from signer (e.g. contract wallet address)

i think we would also need to skip the signOrder step when 1271?

Do we?
An EOA can sign the order and then it is checked inside 1271 contract
    address public signer;

    function isValidSignature(bytes32 hash_, bytes calldata signature_) external override view returns (bytes4) {
        address signer_ = hash_.recover(signature_);
        if (signer_ == signer) {
            return 0x1626ba7e;
        }
        return 0x00000000;
    }

nice thanks, do you mind also adding a 1271 test to ensure it works and doesn't break in the future?

Sure, will do

nice thanks, do you mind also adding a 1271 test to ensure it works and doesn't break in the future?

Added tests but I have side effect — https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/utils/cryptography/ECDSA.sol#L70 will reject compacted sig
I can revert that change in the code if you do not think that makes sense

@ryanio

@ryanio applied all changes

looks good, could you run npm run prettier:fix please


picture - update weixin.jpg

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Catch the use of global TypedList in JITed functions.

There seems to be some issue with the indexing of the global variable?
import numba as nb

from typing import NamedTuple

class Shift(NamedTuple):
    start_time: int
    end_time: int
    work_hours: int

shifts = nb.typed.List()
shifts.append(Shift(0, 10, 10))
shifts.append(Shift(0, 2, 3))

@nb.njit()
def ff(assigned_shifts) -> bool:
    x = [shifts[i].work_hours for i in assigned_shifts]
    return True

assigned_shifts = nb.typed.List()
assigned_shifts.append(1)
ff(assigned_shifts)

stack trace

Traceback (most recent call last):
  File "/test/venv/lib/python3.6/site-packages/numba/targets/base.py", line 499, in get_constant_generic
    impl = self._get_constants.find((ty,))
  File "/test/venv/lib/python3.6/site-packages/numba/targets/base.py", line 50, in find
    out = self._find(sig)
  File "/test/venv/lib/python3.6/site-packages/numba/targets/base.py", line 59, in _find
    raise NotImplementedError(self, sig)
NotImplementedError: (<numba.targets.base.OverloadSelector object at 0x7f14d4e95278>, (ListType[Shift(int64 x 3)],))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/test/venv/lib/python3.6/site-packages/numba/errors.py", line 717, in new_error_context
    yield
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 260, in lower_block
    self.lower_inst(inst)
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 303, in lower_inst
    val = self.lower_assign(ty, inst)
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 460, in lower_assign
    value.value)
  File "/test/venv/lib/python3.6/site-packages/numba/targets/base.py", line 502, in get_constant_generic
    raise NotImplementedError("Cannot lower constant of type '%s'" % (ty,))
NotImplementedError: Cannot lower constant of type 'ListType[Shift(int64 x 3)]'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "<input>", line 24, in <module>
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 420, in _compile_for_args
    raise e
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 353, in _compile_for_args
    return self.compile(tuple(argtypes))
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 768, in compile
    cres = self._compiler.compile(args, return_type)
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 77, in compile
    status, retval = self._compile_cached(args, return_type)
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 91, in _compile_cached
    retval = self._compile_core(args, return_type)
  File "/test/venv/lib/python3.6/site-packages/numba/dispatcher.py", line 109, in _compile_core
    pipeline_class=self.pipeline_class)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler.py", line 528, in compile_extra
    return pipeline.compile_extra(func)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler.py", line 326, in compile_extra
    return self._compile_bytecode()
  File "/test/venv/lib/python3.6/site-packages/numba/compiler.py", line 385, in _compile_bytecode
    return self._compile_core()
  File "/test/venv/lib/python3.6/site-packages/numba/compiler.py", line 365, in _compile_core
    raise e
  File "/test/venv/lib/python3.6/site-packages/numba/compiler.py", line 356, in _compile_core
    pm.run(self.state)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_machinery.py", line 328, in run
    raise patched_exception
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_machinery.py", line 319, in run
    self._runPass(idx, pass_inst, state)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_lock.py", line 32, in _acquire_compile_lock
    return func(*args, **kwargs)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_machinery.py", line 281, in _runPass
    mutated |= check(pss.run_pass, internal_state)
  File "/test/venv/lib/python3.6/site-packages/numba/compiler_machinery.py", line 268, in check
    mangled = func(compiler_state)
  File "/test/venv/lib/python3.6/site-packages/numba/typed_passes.py", line 380, in run_pass
    NativeLowering().run_pass(state) # TODO: Pull this out into the pipeline
  File "/test/venv/lib/python3.6/site-packages/numba/typed_passes.py", line 325, in run_pass
    lower.lower()
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 179, in lower
    self.lower_normal_function(self.fndesc)
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 220, in lower_normal_function
    entry_block_tail = self.lower_function_body()
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 245, in lower_function_body
    self.lower_block(block)
  File "/test/venv/lib/python3.6/site-packages/numba/lowering.py", line 260, in lower_block
    self.lower_inst(inst)
  File "/usr/lib/python3.6/contextlib.py", line 99, in __exit__
    self.gen.throw(type, value, traceback)
  File "/test/venv/lib/python3.6/site-packages/numba/errors.py", line 725, in new_error_context
    six.reraise(type(newerr), newerr, tb)
  File "/test/venv/lib/python3.6/site-packages/numba/six.py", line 669, in reraise
    raise value
numba.errors.LoweringError: Failed in nopython mode pipeline (step: nopython mode backend)
Cannot lower constant of type 'ListType[Shift(int64 x 3)]'
File "<input>", line 18:
<source missing, REPL/exec in use?>
[1] During: lowering "$6.2.14 = global(shifts: [Shift(start_time=0, end_time=10, work_hours=10), Shift(start_time=0, end_time=2, work_hours=3)])" at <input> (18)
-------------------------------------------------------------------------------
This should not have happened, a problem has occurred in Numba's internals.
You are currently using Numba version 0.46.0.
Please report the error message and traceback, along with a minimal reproducer
at: https://github.com/numba/numba/issues/new
If more help is needed please feel free to speak to the Numba core developers
directly at: https://gitter.im/numba/numba
Thanks in advance for your help in improving Numba!

Ubuntu 18.04
Python 3.6
Numba 0.46.0
Edit: simplified example

Thanks for the report. The reason for the problem is that global level mutable containers should not be accepted for compilation as Numba treats globals as compile time constants. What should happen is that this is spotted at type inference time and reported to the user, opposed to a lowering error above. This is a smaller reproducer:
from numba import njit, typed
import numpy as np

shifts = typed.List()
[shifts.append(i) for i in range(5)]

@njit()
def ff():
    x = typed.List()
    for i in shifts:
        x.append(i)

ff()

CC @esc

NOTE: updated title to reflect underlying issue.

Ah, I see. I noticed that my example contained quite a lot of unnecessary code to reproduce the issue, but good that you caught the essentials. I'll switch to a tuple to avoid the issue with the mutable container. Thank you for the prompt reply.

Ah, I see. I noticed that my example contained quite a lot of unnecessary code to reproduce the issue, but good that you caught the essentials. I'll switch to a tuple to avoid the issue with the mutable container. Thank you for the prompt reply.

Great, thanks for reporting back, tuple should work fine.

Note to self, look at: line 1051 of typeinfer.py

Fix in: #5078


Dev environment not working with node v17.1.0

When running the yarn run dev:chrome command and using Node version 17.1.0, I get the following output:
yarn run v1.22.17
$ cross-env NODE_ENV=development cross-env TARGET_BROWSER=chrome webpack --watch
node:internal/crypto/hash:67
  this[kHandle] = new _Hash(algorithm, xofLen);
                  ^

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:67:19)
    at Object.createHash (node:crypto:130:10)
    at BulkUpdateDecorator.hashFactory (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/lib/util/createHash.js:155:18)
    at BulkUpdateDecorator.digest (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/lib/util/createHash.js:80:21)
    at /home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/lib/DefinePlugin.js:595:38
    at Hook.eval [as call] (eval at create (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:104:1)
    at Hook.CALL_DELEGATE [as _call] (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/node_modules/tapable/lib/Hook.js:14:14)
    at Compiler.newCompilation (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/lib/Compiler.js:1053:26)
    at /home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/lib/Compiler.js:1097:29
    at Hook.eval [as callAsync] (eval at create (/home/mk/Desktop/CODE/alby-redesign-new/node_modules/webpack/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1) {
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

Node.js v17.1.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

FYI I am getting this same error with a different React application so this may be a Node and React issue.

This might be helpful:
https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported
This seems to explain it:

node.js 17 uses OpenSSL3 which have changed code for initialization context of md family (including md4):

https://github.com/webpack/webpack/issues/14532#issuecomment-947807590
There's more suggestions in that issue on how to deal with this.

is it working now?

Looks good on latest stable v18:
master  nvm install 18
v18.7.0 is already installed.
Now using node v18.7.0 (npm v8.15.0)

master  yarn
yarn install v1.22.19
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
success Already up-to-date.
$ husky install
husky - Git hooks installed
✨  Done in 1.82s.

master  yarn run dev:chrome
yarn run v1.22.19
$ cross-env NODE_ENV=development TARGET_BROWSER=chrome webpack --watch
wext-manifest-webpack-plugin: removed js/manifest.bundle.js
(node:72712) [DEP_WEBPACK_CHUNK_HAS_ENTRY_MODULE] DeprecationWarning: Chunk.hasEntryModule: Use new ChunkGraph API
(Use `node --trace-deprecation ...` to show where the warning was created)
(node:72712) [DEP_WEBPACK_CHUNK_ENTRY_MODULE] DeprecationWarning: Chunk.entryModule: Use new ChunkGraph API
(node:72712) [DEP_WEBPACK_DEPRECATION_ARRAY_TO_SET] DeprecationWarning: chunk.files was changed from Array to Set (using Array method 'filter' is deprecated)

Uneven node versions are usually "testing" versions. Only even versions are maintained releases. See:

https://nodejs.org/en/about/releases/
We should switch to 16 or 18 by now. Can be done in a new issue: #1314


[NCPI] For catalogs, allow download of current result set as a CSV

Similar to how the current AnVIL dataset catalog works.
### Tasks
- [ ] https://github.com/clevercanary/cc-design/issues/21
- [ ] https://github.com/clevercanary/data-browser/issues/803

Dupe of https://github.com/clevercanary/data-browser/issues/803.


ci: add scheduled runs for Ariane workflows

In 9949c5a1891aff8982bfc19e7fc195e7ecc2abf1, we introduced Ariane to handle CI workflows via workflow_dispatch events. One of the nice consequences was that we could finally move CI workflows for stable branches to the appropriate stable branch, instead of having to keep them in main.
However, this had the side effect of disabling running these workflows on a schedule, since the schedule events are only triggered for files present in the main branch.
In this commit, we introduce a schedule-based workflow on main whose sole purpose is to trigger these missing tests. It automatically detects which workflows to run based on the current Ariane configuration in that branch, and thus the only need in terms of maintenance for us should be to add / remove stable branches in the matrix as appropriate.

This does not need to run any CI, reviews are in, marking ready-to-merge.

@nbusseneau I'm seeing these scheduled workflows failing on forks, and it's sending me emails. Can we limit this to the cilium/cilium repo?


NIST incompatible Public exponent range

The latest NIST specification allows public exponent $e$ to be in the range of $2^{16} + 1$ and $2^{256} - 1$ (inclusive) (Page 39 of SP-800-56B rev 2: https://nvlpubs.nist.gov/nistpubs/SpecialPublications/NIST.SP.800-56Br2.pdf)
The current code, however, artificially limits the maximum value of $e$ to $2^{64} - 1$, which is unnecessary. The primary reason for limiting the value of $e$ is so that the value of private exponent $d$ does not turn out to be so small that one could do exhaustive search of $d$. But limiting the value to $2^{64}$ is an overkill, especially given that minimal modulus size if no smaller than $2048$ bits.
Will appreciate if this could be extended to the actual NIST specified values.

The primary reason for limiting the value of is so that the value of private exponent does not turn out to be so small that one could do exhaustive search of d.

That is one reason, but not the only reason. The other is to keep public key operations fast. The choice of (1 << 33) - 1 matches ring, where that was based on Adam Langley's recommendations (i.e. what was used in the Go implementation of RSA) and observations of a potential "RSADoS" with very large public exponents (though it seems it should probably be (1 << 32) - 1 instead to match his recommendation of 2^32-1).
See:

https://www.imperialviolet.org/2012/03/16/rsae.html
https://www.imperialviolet.org/2012/03/17/rsados.html

You can use whatever value for e you'd like via the RsaPublicKey::new_unchecked constructor, but given the above, and absent a real-world use case, I would be hesitant to increase the value and if anything it seems like it should probably be decreased to 2^32-1.

The ring documentation also notes that the Windows CryptoAPI implementation of RSA uses a 32-bit DWORD to represent the public exponent, so from a purely interop perspective there is little reason to support larger values since Windows won't support them: https://learn.microsoft.com/en-us/windows/win32/api/wincrypt/ns-wincrypt-rsapubkey?redirectedfrom=MSDN

Ok, thanks, I'll make the changes locally.


Real time HP bar

I noticed that when I eat food the status bar accounts for it but my hp bar above my head doesn't straight away.
Can you please remove the delay so the hp bar reflects what the status bar is saying? The real value.
You probably won't be affected by this issue unless you have very low HP. I'm a skiller with 10 HP so the delay is substantial when trying to stay alive

This would mean that your own HP bar would behave differently from every other HP bar in the game. While this is technically possible, I'm against this.

This would mean that your own HP bar would behave differently from every other HP bar in the game. While this is technically possible, I'm against this.

Why would that matter? It's optional and it is very beneficial to people who actually play the game.

It matters because it's not a good use of our time to add every possible feature anyone could want. I don't see this as a feature that "people who actually play the game" would universally want. For instance, I play the game plenty and would not want this.

This is a bad idea because it would be inaccurate and confusing relative to every other healthbar.

This is a bad idea because it would be inaccurate and confusing relative to every other healthbar.

It's not inaccurate though. It would be displaying the true value.

This is a bad idea because it would be inaccurate and confusing relative to every other healthbar.

I understand your perspective from a programming/time allocation point of view but if you played the game actively you would know you're incorrect.
It's not inaccurate. It would be displaying the real value as I said above there is a delay which is very noticeable if you have low HP.

if you played the game actively you would know you're incorrect.

No need for the condescending tone.. I don't know a single RL dev who doesn't play the game, not sure why you'd assume that's the case.

It's not inaccurate. It would be displaying the real value

Correct, but it is inaccurate when compared to every other health bar except yours, which you've probably gotten used to their behaviour over the years, even if you don't realize it. This would make for some very odd health bars while PKing, you'd assume your opponent is super slow at eating or something, you'd think you can combo him out all the time, with their health bar always seeming low since their eats don't update right away, while yours do. Same with group bossing, you'd be like wtf why is everyone risking it? Unless you constantly kept this in mind, ofc - but it's something extra to think about when you likely already have enough on your plate. At 10hp, shouldn't you just be eating nearly any time you get hit >0 anyways, in the majority of cases where it actually matters? You can also just use the status bar, and you can also get notifications once you go under a certain HP from the Idle Notifier plugin.
In a nutshell, I agree with abex, and I'm fairly sure you would too if you tried something like this. I don't think you realize how 'off' it would feel having your health bar behave completely differently than others'.
Personally, I'm all about "why not" when it comes to external plugins - if it's not malicious and it works as intended - sure why not. But I really don't think this would be as useful as you imagine. I won't be working on it and I can't approve external plugins regardless, but that's my perspective.

if you played the game actively you would know you're incorrect.

No need for the condescending tone.. I don't know a single RL dev who doesn't play the game, not sure why you'd assume that's the case.

It's not inaccurate. It would be displaying the real value

Correct, but it is inaccurate when compared to every other health bar except yours, which you've probably gotten used to their behaviour over the years, even if you don't realize it. This would make for some very odd health bars while PKing, you'd assume your opponent is super slow at eating or something, you'd think you can combo him out all the time, with their health bar always seeming low since their eats don't update right away, while yours do. Same with group bossing, you'd be like wtf why is everyone risking it? Unless you constantly kept this in mind, ofc - but it's something extra to think about when you likely already have enough on your plate. At 10hp, shouldn't you just be eating nearly any time you get hit >0 anyways, in the majority of cases where it actually matters? You can also just use the status bar, and you can also get notifications once you go under a certain HP from the Idle Notifier plugin.
In a nutshell, I agree with abex, and I'm fairly sure you would too if you tried something like this. I don't think you realize how 'off' it would feel having your health bar behave completely differently than others'.
Personally, I'm all about "why not" when it comes to external plugins - if it's not malicious and it works as intended - sure why not. But I really don't think this would be as useful as you imagine. I won't be working on it and I can't approve external plugins regardless, but that's my perspective.

Fair enough and I understand. I just felt my ideas are automatically disregarded especially when the same people keep thumbs downing it within seconds.


Added examples of subPath volume mount with fuse sidecar

Depends on: #6

Recently, kubernetes v1.30 supports sidecar containers(a.k.a. restartable init containers) by default. It don't needs for application containers to wait fuse mount is finished manually.
This PR adds:

sidecar container examples
subPath mount examples and tests

because subPath volume may be created by kubelet before the fuse is mounted to the prepared mountpoint, sidecar is must for using subPath volume mount.

update NOICE section to describe these.

Thanks!


No highlighting for tests

The feature mentioned here doesn't work for me. I get just a plain text.
Gentoo Linux, Sublime 3126, Rust Enhanced 0.1.0

@RazrFalcon I have Gentoo installed,  but im not sure how to install Sublime Text

Hi. Looks like it fixed after updating to v1.0.0. I'm closing it.


Add configuration option

Hi,
This tool is said to be zero-config. But I'm finding that to be a bit to aggressive for my own taste. Maybe a configuration could be given to show where in my folder-tree RepoZ should look for repositories? Because I have lots of folders with minor GIT-libraries that was important at some point and still are valid for archiving purposes. But nothing more. And while I see the nice side of RepoZ, especially with the option of keeping repositories up to date, I wouldn't want the tool to have to scan my whole system and all my repositories.
Thoughts?
And thanks for the tool non the less!
Regards
Gustav

Hi Gustav.
Understood. Maybe I will look at this later on.
I'd suggest to ignore these "archived" repositories. RepoZ will not track them anymore once they have been ignored. Wouldn't you say this is an appropriate workaround?
RepoZ is not scanning your whole system if you don't tell it to. The menu item to scan the system is just to find all repositories once. It is not meant to be repeated regularly. As long as you don't trigger this, it does not scan your system actively while it is running.
What happens is slightly different: RepoZ listens to file change events from the file system. Once it finds changes it buffers the incoming messages (for performance reasons) and then checks whether it can find a git repository at the given path or not. If it does, it refreshes its repository statis. This may sound intense, in fact it's not at all.
Best regards.

Ah ok. So it's listening to all filesystem changes for changes?
If so then also that might be of use of some scope-limitation, to only look in one or a predefine set of folders maybe..? But not sure how or even if that can be configured because I suppose you use some operating system features for that...
And thanks for the explanation. Not triggering a whole system scan is already to late since it's .. already done! So I'll go through the list and ignore things. For next installation I'll know to not scan but simply wait for the tool to find the repositories itself.
Thanks again
Gustav

You can reset the list in the menu. Just clear it and let it repopulate while you change things in the repositories.
I could do a path restriction, yes. But I see a UI to configure this (which is not something one can do in a sexy way) which users won't like. In fact, I see no problem with the current approach and I think people like things more that simply work.

I just find the idea of the tool silently laying there and constantly listening to all filesystem events happening in the background to be bothersome. Especially performance-wise where I suspect it will constantly sip away CPU cycles. Even though probably never a lot. But it will have to listen in to ALOT of events all the time when only a tiny fraction would be needed if I as a user could help it out.
But again, haven't looked at the source code and I may misunderstand the tools use of the filesystem.
Btw, I took a look at the task manager just now and saw RepoZ sipping between 0,1%-1% of the CPU. This is while I'm personally only having a few applications open and browsing the web. No coding involved.
Regards


Multiple partition support per Sink instance

Moved from
https://github.com/spring-cloud/spring-cloud-stream/issues/1232
I've a RabbitMQ binder based Stream Sink Service with parent set to :

org.springframework.cloud
spring-cloud-build
1.3.7.RELEASE

The input Topic is partitioned,
How can we get more than 1 Topic partition assigned to a Sink Service instance ?
Basically we want to limit the # of Sink Service instances but still get the parallelism across partitions.
Thanx,

@tajdars Thank you for moving it! As I mentioned we'd be definitely interested in contribution from the community so if you up for the challenge go ahead and give it a shot and submit a PR and we'll work with you to make it happen.
Thanks again!


Enable S3 blobstore support

Add support for S3 compatible blobstores (like OpenStack Swift with the S3 API enabled).

@jerenkrantz FYI

:+1:
I'd like that very much. I'm currently unable to setup the BOSH director with bosh-init to use an external S3 blobstore as both the director and the cpi template use the same blobstore property, but the cpi template only knows about local or simple, so I run into a conflict there with bosh director template.

I think that this patch is good - the director can upload items into the blobstore.
Due to bloomberg/chef-bcpc#596, I can't upload releases/packages greater than the specified multipart threshold into radosgw, but that appears to be an entirely separate issue.  (IOW, the client is doing the right thing; but, the server isn't.)
Thanks!

Merged: https://bosh.io/releases/github.com/cloudfoundry-incubator/bosh-openstack-cpi-release?version=8

Thanks @cppforlife!


Implement UX Review

Waiting on feedback from Dale before doing this review internally.

UX comments: https://invis.io/2C6OMI2HY


Not support HTMLVideoElement ?

i try to pass video element to detector.detect api, but it return No width or height reject

does it has a width and height ?
did you try the sample code in the demo to see if it works for you ?

video need videoHeight, videoWidth to get height, width; and if u wanna get element width/height, need use offsetWidth, 'offsetHeight'

i don't know, all i can say is that it works for me :) if you have a codepen or something or make a PR you are welcome to do so

https://codepen.io/preflower/pen/ExoQOOK

i don't know, all i can say is that it works for me :) if you have a codepen or something or make a PR you are welcome to do so

https://codepen.io/preflower/pen/ExoQOOK

yes i see. but nothing is preventing you to set an actual width on the video element before detecting? not ideal, but it should work fine

video need use videoWidth/Height to get actual width/height, it has nothing to do with ideal,  even if i set video.width/height, is element w/h, not video actual w/h...


fix: force fleek build on #628

Dummy PR to force the fleek build on #628. Please close when that PR is merged.

went too far out of sync with the source branch. will need to start a new PR.


Create matchers as constexpr.

First step: make PathMatchBuilder a constexpr to build a consexpr matcher.

Codecov Report
Base: 93.00% // Head: 93.00% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (a5ea985) compared to base (1e75821).
 Consider uploading reports for the commit 3acdb28 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1724      +/-   ##
==========================================
- Coverage   93.00%   93.00%   -0.01%
==========================================
  Files         352      352
  Lines       25485    25480       -5
==========================================
- Hits        23702    23697       -5
  Misses       1783     1783

Impacted Files
Coverage Δ

common/analysis/matcher/matcher_builders.h
100.00% <ø> (ø)

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

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


Ensure writers don't write empty values

Firstly, apologies for the not-so-small PR. Most of it is tests, as anticipated, as the fixes/changes proper are quite small.
I have made sure that empty/blank elements and attrs are not written to RSS (as they're pointless anyway), and that we trim any value we write, as well. Lastly, I realised RssWriter had no tests as I had apparently forgotten to write them. Oops! Now that's fixed.
On my roadmap:

[x] Ensure parsers read empty fields as null
[x] Ensure writers don't write out empty fields/attributes (and trim values we do write)
[ ] Add parsing/writing of RSS <categories> nodes in <channel>
[ ] Add support for podcast: namespace
[ ] Unify Atom and AtomBuilder podcast/episode versions since they're exactly the same
[ ] Add some end-to-end tests that read a bunch of real-life RSS feeds

I am currently a bit busy, it will take me a bit to review this -- sorry about that.

I am currently a bit busy, it will take me a bit to review this -- sorry about that.

Likewise. I have started with the categories tag in channel, but have been sidetracked. No rush

Likewise. I have started with the categories tag in channel, but have been sidetracked. No rush


Preview Animation

Hi Lukasz. Congrats you to built this component. Is there any way to change preview animation?

+1

+1


Pretrained Hifi GAN vocoder at 16KHz

Hi,
Does any know where the pretrained Hifi GAN vocoder that works at 16KHz is available.
OR
Can any one have config file (hyper parameters setting at 16K Hz) that gives the best possible quality at 16 KHz. I am not able to get the right parameter settings for 16KHz vocoder. It would be very help full for me.
Thanks in advance,
Narendra

"upsample_rates": [2,5,4,4],
"upsample_kernel_sizes": [16,15,4,4],
"upsample_initial_channel": 512,
"resblock_kernel_sizes": [3,7,11],
"resblock_dilation_sizes": [[1,3,5], [1,3,5], [1,3,5]],
"resblock_initial_channel": 256,

"segment_size": 5120,
"num_mels": 80,
"num_freq": 512,
"n_fft": 512,
"hop_size": 160,
"win_size": 512,

"sampling_rate": 16000,

i use these parameters training at 16KHz.

"upsample_rates": [2,5,4,4],
"upsample_kernel_sizes": [16,15,4,4],
"upsample_initial_channel": 512,
"resblock_kernel_sizes": [3,7,11],
"resblock_dilation_sizes": [[1,3,5], [1,3,5], [1,3,5]],
"resblock_initial_channel": 256,

"segment_size": 5120,
"num_mels": 80,
"num_freq": 512,
"n_fft": 512,
"hop_size": 160,
"win_size": 512,

"sampling_rate": 16000,

i use these parameters training at 16KHz.

would you mind sharing the trained checkpoint?
thank you in advance!

"upsample_rates": [2,5,4,4],
"upsample_kernel_sizes": [16,15,4,4],
"upsample_initial_channel": 512,
"resblock_kernel_sizes": [3,7,11],
"resblock_dilation_sizes": [[1,3,5], [1,3,5], [1,3,5]],
"resblock_initial_channel": 256,

"segment_size": 5120,
"num_mels": 80,
"num_freq": 512,
"n_fft": 512,
"hop_size": 160,
"win_size": 512,

"sampling_rate": 16000,

i use these parameters training at 16KHz.

"upsample_kernel_sizes": [16,15,4,4]？
Is the 15/2  the stride of the second De_Conv ?

Mind if I ask why you set n_fft to 512 instead of 4 times of hop_size(4*160=640), which is a usually default setting in STFT?


Improve README about NOTEs

This is a small improvement to README.
It would be a little easier to read if the notes were in a consistent format. How about trying alerts syntax on GitHub?

https://github.blog/changelog/2023-12-14-new-markdown-extension-alerts-provide-distinctive-styling-for-significant-content/
https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#alerts

This results in the following display:

I have signed the CLA!


tests: clean up TestNetDefaultRestrictedConnectivity

Fixes #2943

:+1: looks good to me on non-flakiness


Re-inserting identical entity throws 'InvalidArgument' and not 'AlreadyExists'

When attempting to insert an identical entity twice, the expected GrpcException status code has been: 'Grpc.Core.StatusCode.AlreadyExists'. But sometime along the way this exception has changed to throw 'Grpc.Core.StatusCode.InvalidArgument'.
When attempting to add an entity that already exists in the DB it would make the most sense to throw 'AlreadyExists' and use 'InvalidArgument' for strictly key formatting issues, etc.
How to reproduce:
Run "gcloud beta emulators datastore start"
Attempt to insert two identical entities: { _db.Insert(myEntity); _db.Insert(myEntity); }
Catch exception on the second call and observe the following exception: 'RpcException: Status(StatusCode=InvalidArgument, Detail="entity already exists")'

I'll have a look at this on Tuesday. (Monday is a public holiday in the UK.) If it only happens in the emulator, it's probably an emulator bug. (It's very unlikely that this is a client library issue. We don't do much with errors other than retry some operations in some cases.)

I can't reproduce this, either against the emulator or the production environment. Here's an example which attempts to reproduce it:
using Google.Cloud.Datastore.V1;
using Grpc.Core;
using System;

class Program
{
    static void Main(string[] args)
    {
        string emulatorHost = Environment.GetEnvironmentVariable("DATASTORE_EMULATOR_HOST");
        string projectId = Environment.GetEnvironmentVariable("DATASTORE_PROJECT_ID");
        DatastoreClient client = string.IsNullOrEmpty(emulatorHost)
            ? DatastoreClient.Create()
            : DatastoreClient.Create(new Channel(emulatorHost, ChannelCredentials.Insecure));
        var db = DatastoreDb.Create(projectId, "Issue2453", client);

        var entity = new Entity
        {
            Key = db.CreateKeyFactory("demo").CreateKey(Guid.NewGuid().ToString()),
            Properties =
            {
                ["text"] = "Demo"
            }
        };

        Console.WriteLine("First insertion");
        db.Insert(entity);
        Console.WriteLine("Second insertion");
        db.Insert(entity);
    }
}

In both cases, the first insertion succeeds; the second fails with:

Unhandled Exception: Grpc.Core.RpcException: Status(StatusCode=AlreadyExists, Detail="entity already exists")

Set the DATASTORE_PROJECT_ID environment variable, and then either set or clear DATASTORE_EMULATOR_HOST based on whether you want to test against the emulator or not.
Could you try that code in your environment?

I'm confused how I got to this point but I was somehow running an extremely old version of the datastore emulator even after updating. After cleaning out my installation and installing the SDK again I'm now up to date and it's giving me the expected result similar to what you posted above. Thank you for investigating.
I found that this specific issue was fixed in datastore emulator version 1.4.
""
Released Cloud Datastore Emulator version 1.4.0.
The emulator now returns error codes that match the Cloud Datastore service for two scenarios: inserts of entities that already exist and updates to ones that don't.
""

Great - glad to hear it's all resolved :) Will close this issue. Obviously if you see anything similar again, please do raise it!


docs(datepicker): fix mention on string literals

Ah, my bad. LGTM.

Nah, this was my fault - I reverted my change with the literals being backticks, I missed this.


Cannot set mqtt credentials.

Just did a fresh install on a windows pc (3rd one), after install and multiple restarts the mqtt credentials will not save, I enter the ip, port, user, password (non TLS), hit save nothing happens.
The mqttbroker.json file remains empty. If I enter the information into that file manually the front end / config will then not open.
following this format...
{"Host":"<IP_ADDRESS>","Port":1883,"UseTLS":false,"Username":"your username","Password":"your password"}
I have attempted restart, reboot, and re install....
log20211125.txt
log20211125_001.txt
log20211125_002.txt
log20211125_003.txt
log20211125_004.txt
log20211126.txt
log20211126_001.txt
Did discuss this on discord #support, was referred here.
Thanks

Can you add some information about the languages installed on your PC, the keyboard layout, and your username? thanks!

I also had an external keyboard attached, after reading some comments, I disconnected that and tried to enter the information from the laptop keyboard. Same result.
Also made sure that only one of the front ends was open as I read a comment from a user on discord regarding that...:-(

@pKuEDhVxaGFYBTRz > Can you add some information about the languages installed on your PC, the keyboard layout, and your username? thanks!
What username do you want, and why?

Username on discord, is @Seal. #9674
My discord username is @sihartley. I reached out on the @Seal username because it is my sons laptop that is having the issue, his discord.
Sorry if this caused confusion.

@sihartley this is a visual frontend bug, when you try to save the credentials , you should not hit the save button but instead the connect button but it is hidden.
Check https://github.com/sleevezipper/hass-workstation-service/issues/142

The fix for the UI bug is live now! I assume this will solve you issue. Please reopen if that's not the case.


replaced watch with watchFile (#9873)

Ports #9873
This PR fixes #9781

/merge

The label Approved is required for using this command.

/merge

Merging PR based on approval from @chlafreniere


Add specs for DIRSHA256

/cc @bobcallaway

@mihaimaruseac lmk if you have further comments

Since we're doing manifest hashing, I think we no longer need this.


added a test for load_active_pixels!

I added a basic test for load_active_pixels!. The thing about testing this function is that it isn't well defined what the function is supposed to do, beyond what the code that implements it says it does. Sometimes really dim pixels are informative because we expect them to be bright (based on initial parameters); sometimes really bright pixels are informative because we expect them to be dim (based on our initial estimates of the parameters). We might want to think more about how to select active pixels. Maybe something about the log probability changes, given the initial estimates of the parameters, if a particular pixel (or set of pixels) is censored. Perhaps subsampling pixels for large galaxies could be a special case of our procedure for selecting active pixels.

I think at least this functionality of the old test should still ber here: that we test
(1) most of an image's brightness was captured by the active pixels and
(2) that min_pixel_radius works as advertised
Maybe you could just find and restore the old test.
I really disagree with simply asserting that you only have one active source.  Even if you are only currently using one source at a time for your work, it wastes of a lot of work I did.  It would not take much effort to fix this.  But we should maybe talk about this in person.

@rgiordan -- PTAL. Tests are failing currently only because data.sdds3.org  is down.

benchmark_infer.jl gets 10% slower and allocates a bunch more memory :-1:

This last commit prevents most of the new memory allocation.
benchmark_infer.jl on master:
19.562068 seconds (5.07 M allocations: 667.168 MB, 6.74% gc time)
benchmark_infer.jl with this PR:
18.438190 seconds (5.07 M allocations: 667.219 MB, 7.01% gc time)
The change in times isn't meaningful because I had some other programs open. The slight increase in memory allocations is acceptable.


Add an Approve hotkey for signing requests

It would be nice if CMD+Enter, ALT+Enter, or some kind of hot key could approve a signing request. Without automatic signing its somewhat of a burden to have to move the mouse and approve each request, which a hot key might lessen.

This will need to happen when we revamp the signing process. Right now the way the UI is setup, this would require a decent amount of effort in order to integrate and make functional. The "next action" portion of the UI right now can be a lot of things, so simply detecting a hotkey press and then triggering the appropriate action doesn't fit into the current model.

What we basically need is auto signing which would make this request obsolete.


Request for Addition of Missing Poisson Distribution Code in Data Generation

Hi. I noticed that the training scripts in TrafficModels are intended to support five types of distributions as mentioned in the paper.. However, it appears that the segment for training data generated by Poisson distribution is missing.
Could you please provide the missing Poisson distribution training code?  The inclusion of this part would be immensely helpful for me and others in the community attempting to replicate your results fully.
Thank you for considering this request.  I look forward to the complete set of scripts.

Dear @MashiroSaber008,
Thank you for bringing this to our attention!
You are correct in observing that the dataset/training code for the Poisson distribution is not currently included in our provided scripts. The datasets utilized for this experiment can be found here. However, since these datasets are from an earlier version, some adjustments may be necessary to integrate them into the provided script.
While I am actively working on resolving this issue, you might find our newly released model, RouteNet-Fermi, interesting. In the Scalability section particularly, you'll find datasets featuring the Poisson distribution along with the corresponding scripts for training, validation, and testing.
Again, thank you for pointing this out.
Best regards,
Miquel


[if-unofficial-models]: watttime invalid geolocation causes unknown error

Describe the bug
Wattime plugin causes unknown error when executed with invalid geolocation.
To Reproduce
Steps to reproduce the behavior:

Setup valid watttime manifest.
Use 7.7749,-122.4194 as a geolocation.
Run the if framework.

Expected behavior
Handle invalid value gracefully.
Actual behavior
Throws unknown error for the if framework, which addresses user to report an issue.
Screenshots

Additional context
Manifest:
name: watt-time
description: simple demo invoking watt-time
tags:
initialize:
  plugins:
    watt-time:
      method: WattTimeGridEmissions
      path: '@grnsft/if-unofficial-models'
      global-config:
        username: ENV_WATT_TIME_USERNAME
        password: ENV_WATT_TIME_PASSWORD
tree:
  children:
    child:
      pipeline:
        - watt-time
      inputs:
        - timestamp: 2023-07-06T00:00
          duration: 3600
          geolocation: 7.7749,-122.4194

Verified, fixed


Update diff colors with semantics

PR Type
enhancement

Description

Added a new PasswordScanner object in Scala.
Implemented functionality to read passwords from files and check them against a list of compromised passwords.
Included error handling for file reading operations.

Changes walkthrough 📝
Relevant filesEnhancement

pass_scan.scalaImplement password scanner to check compromised passwords

sherlock_project/pass_scan.scala
Added a new PasswordScanner object.  Implemented main method to scan passwords against a list of compromised passwords.  Included a helper method to read lines from a file.

+35/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/improve

/update_changelog


Create an Edit Rosters page

Mods should be able to use this page to build teams at the start of the season, as well as manage teams during a season.
It should allow players to be moved between teams/alternates, captains to be assigned and teams to be renamed. If the season hasn't started it should also allow teams to be created and deleted. Relevant information should be displayed about each player in a popover.

This has been implemented. There's potential for additional functionality but the core is there.


Increase our build budget to 16ms

Because we now have 2 separate GPU and UI threads.

I'm confused by this change. What does it mean to have two UI threads?
If our target is 120Hz with no pipeline latency, and assuming equal share between build and rasterize, how can the budget be more than 4ms?

We meant two threads, one UI thread and one GPU thread.
We didn't always target 0 pipeline latency. Instead, our goal is to track smoothness and latency separately. In the case of 120Hz, each thread has a budget of 8ms to be completely smooth. In order to be 0 latency, the sum should be less than 8ms.

Have we not always had two separate threads?
I agree that the sum should be 8ms if we're targetting 120Hz. That seems like the opposite direction than this PR went in, though.

Yes, just chatted with Chinmay and confirmed that we always had 2 threads. Chinmay also said that 8ms for each thread under 60fps assumption was either a mistake or an over-simplification? (CC @chinmaygarde for confirmations)
For 120Hz, it's sum of 8ms for 0 latency, and 8ms each thread for 100% smooth. This PR focuses on the latter. So far, I haven't seen any complaints about the latency, and all performance issues are regarding the smoothness. That's why this PR focuses on the latter.

We should care about latency and smoothness and we should care about 120Hz. We should care about this stuff before we hear complaints, that way we will never hear complaints. The idea is to be ahead of where our developers are, not behind. :-)

Agree! Filed https://github.com/flutter/flutter/issues/32447


Added support for large stacks

To address the issue: fixes #41

Thanks @whummer  for merging the pull request, would you still want me to bump the version and write the changelog?
BTW, When are you planning to publish to npm?

Done - new version 1.65.4 release @AminFazlMondo


Full copyright and licence info.

Copyright HdM (based on contract).
Licence info added for all Node.js modules.

I'm closing this pull request because it has been open for more than 2 years without a response.


Thank you so much

Thank you so much for providing your helpful script, which I found at https://medium.com/@antelle/how-to-generate-a-self-signed-ssl-certificate-for-an-ip-address-f0dd8dddf754
I've been trying to figure out how to modify it to be usable on PC, Mac, iOS, and Android.
Previously, I'd been using https://stackoverflow.com/a/57684211/470749
But that seems unable to work with an IP address.
Do you happen to know how I could modify that "One self-signed cert to rule them all" to use an IP address rather than hostname?
In other words, what I seem to need is a merge of your approach and ScottyB's.
I appreciate your thoughts. Thanks.

Hi! You mean, you expect the script to be able to run on all those OS, or you notices an issue with certificates generated by it?

What a fast response! Thanks!
I mean that I was not able to install the certificate at Android > Settings > Security > Advanced > Encryption & credentials > Install a certificate > CA Certificate.
It says "Private key required to install a certificate".
The only way I've ever been able to get a self-signed cert to be installable on Android is https://stackoverflow.com/a/57684211/470749
I don't understand all the particulars about why each type of device (Android, iOS, PC, Mac) requires different lines in the conf file or whether it would be possible to use an IP instead of hostname.
Thanks.

Interesting, I won't be able to check if it works on Android, but I can see what happens on iOS.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/104 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit e3b303d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-29-11-42-43 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-29-11-42-43

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ReturnType not being evaluated when used in function definition generics alias

Scenario:
Given an indexed object of functions that return different value types, generate a new object with the same indexes and the return value of the functions.
const given = {
  foo: () => 1,
  bar: () => 'milka',
};

const generatedResult = {
  foo: 1,
  bar: 'milka',
};

Failing example:
TypeScript is not re-evaluating the Types when calling the function with predictable values:

It can predict the functions and their return types, but it is not evaluating ReturnType<...>
Workaround example:
The proper types are correctly inferred when using an anonymous type directly in the return type of the function:

This workaround is not desired given the need for using the Type definition inside the body of the function.
TypeScript Version:  4.0.3

Search Terms:

ReturnType
Generics

Code
function extract<
  P extends string,
  M extends Record<P, () => any>,
  Ret extends {
    [K in P]: ReturnType<M[K]>;
  }
>(_map: M): Ret {
  return <Ret>{};
}

const { foo, bar } = extract({
  foo: () => 1,
  bar: () => 'bar',
});

Expected behavior:
Variable foo should be of type number
Variable bar should be of type string
Actual behavior:
All variables are of type any
Playground Link:
Playground Link or a CodeSandbox Link - Both with the same behavior.

Why do you need so many extra type params?
This is more than enough,
function extract3<M extends Record<string, () => any>> (_map : M) : {
  [k in keyof M] : ReturnType<M[k]>
} {
  return {} as any;
}

Also, try this instead,
function extract<
  P extends keyof M,
  M extends Record<string, () => any>,
  Ret extends {
    [K in P]: ReturnType<M[K]>;
  }
>(_map: M): Ret {
  return {} as Ret;
}

If we change your example a little, you'll see that your initial extract doesn't really make sense.

Playground

@AnyhowStep You might have got it wrong.
In a strict environment with enforced types, and no-unsafe-any enabled, those examples are not valid.
Also, your example will only return string types, while every key has its own type.
In my case foo is a number, bar is a string.
Your example both are strings, what is not correct.
Also, in a strict environment, you should not typecast something to any just to return it.

Also, in a strict environment, you should not typecast something to any just to return it.

That is just done to focus on the types, instead of having to create the implementation of the function.

I don't see the reason for the complexity. What you want can trivially be written as:
type DesiredReturn<T extends Record<any, () => any>> = { [P in keyof T]: ReturnType<T[P]> }

// Type is { foo: number; bar: string; }
type ParameterReturn = DesiredReturn<Parameter>;

function extract<T extends Record<any, () => any>>(parameter: T): DesiredReturn<T> { /* implementation be here */ }

const { foo, bar } = extract({ foo: () => 0, bar: () => '' });

I'm aware of alternatives, but none of the already suggested ones solves the problem or prove that it is not an issue.
Lets see a more complex example:
const randomFn = <T>(): T | null => null;

export default function makeSelector<T>(context: T) {
  return <
    Keys extends keyof Selectors,
    Selectors extends Record<string, (context: T) => unknown>,
    TRet extends { [K in Keys]: ReturnType<Selectors[K]> }
  >(
    selectors: Selectors,
  ): TRet => { // HERE if I replace the named type TRet with { [K in Keys]: ReturnType<Selectors[K]> } it magically works 🤷🏻♂
    // actual implementation
    // data fetching and parsing and then pass it to another fn
    const ret = randomFn<TRet>();
    return ret;
  };
}

const selector = makeSelector({ unrelated: true, nothingSpecial: false });

const { foo, bar } = selector({ foo: () => 0, bar: () => '' });

As explained in the issue description, if I delete the TRet definition and just put { [K in Keys]: ReturnType<Selectors[K]> } as the return type, it magically works.
another playground with a working example.

Your code is failing because TypeScript can't infer the type of TRet from anything. It only knows what it should extend, but it doesn't know what the type actually looks like.
That's why it works if you remove the type parameter and actually provide the return type. This is working as intended.

It is able to figure it out.
If you hover your mouse at selector({ foo: () => 0, bar: () => '' }); you will see all the parameters, functions, Keys and return types are there.

Everything is there, except for the TRet, that depends on ReturnType.

It is able to figure it out.

Not fully, that's why you get the any. It can't figure out the type of the properties. It knows what the types should extend, but not what the types are. If you provide the type arguments yourself (so no inference is necessary) you get the desired result. With your initial example:
function extract<
  P extends string,
  M extends Record<P, () => any>,
  Ret extends {
    [K in P]: ReturnType<M[K]>;
  }
>(_map: M): Ret {
  return <Ret>{};
}

const { foo, bar } = extract<
  'foo' | 'bar',
  { foo: () => number, bar: () => string },
  { foo: number, bar: string }
>({
  foo: () => 1,
  bar: () => 'bar',
});

Look at the code of extract. Based on what information would the compiler be able to infer the exact type of Ret? There's simply not enough information available.

But then, why does it work if I remove the named TRet generic and just put the anonymous:
{ [K in Keys]: ReturnType<Selectors[K]> } as the return of the function?
Why does the named one fail while the anonymous work?

Because then there is no type to infer. The compiler knows the exact type. The compiler doesn't need to figure out "what is the type, and does it extend this other type".

Why does the named one fail while the anonymous work?

You're assuming TRet just means "I have a named type", but that's not the case. You're having a generic type argument, so the method can accept different types for TRet (they just must extend a specific type). Unless explicity provided (see my example) it must be inferred by the compiler.

I might be underestimating it,
But if I look again at the screenshot of the Hover over the extract function.
Every single type is there, the properties, the functions, and the return type of the given functions.
I'm still not following why only the part where ReturnType is used is unable to infer the type.
The compiler is literally showing every single piece of information in the screenshot.

Every single type is there, the properties, the functions, and the return type of the given functions.

The first two type arguments can easily be inferred from your input argument (both P and M are used as part of _map). But where should the third type argument be inferred from? The third type argument must be inferred from the value you return within your function, and there's not enough information available for this.

I'm still not following why only the part where ReturnType is used is unable to infer the type.

It's unrelated to ReturnType. Remove it, just use M[K] and you have the same issue.

It's unrelated to ReturnType. Remove it, just use M[K] and you have the same issue.
I see your point now.

But how is it different, if we move the definition from the generic argument to the return type?

Because then there is no type inference necessary anymore. The compiler doesn't need to figure out what Ret ist, it knows it's this specific type.
The difference is:

With the generic return type Ret: Compiler needs to figure out, what is Ret and does it extend { [K in P]: M[K] }?
With the non-generic return type { [K in P]: M[K] }: Compiler doesn't need to figure out the return type. It's { [K in P]: M[K] }.

Nice, thanks for the patience and the extremely detailed explanation.
What we do now?
Is it a feature-request then? instead of a bug?
Or, ~please god no~, is it a Wont-do?
because it is extremely useful to have it inside a Closure and being able to pass this generic argument deeper.

It sounds to me that you don't actually want a generic type, but you just want to reference your rather large type using a local type alias. For this there's #40780 / #30979. But there hasn't been any progress, and likely there won't be in any foreseeable future.

Looking at your screenshots, both are strings, what is not correct.
{ [x: string]: string; } // from your screenshot, not the actual real type.

I know. I'm saying that you've structured your types in a way that doesn't make sense.
I'm showing you examples of why your attempts do not make sense.

Having type parameters that cannot be inferred/constrained from the actual function arguments is also actually an anti-pattern.
Assume your desired type parameter Ret actually works. We have this problem,
function extract<
  M extends Record<string, () => any>,
  Ret extends {
    foo : number,
    bar : string,
  }
> (_map : M) : Ret {
  return {} as any;
}

const obj = {
  foo: () => 1,
  bar: () => 'bar',
} as const
const { foo, bar } = extract<
  typeof obj,
  {
    foo : 32,
    bar : "lol",
  }
>(obj);

Playground

Notice that even thought obj returns 1, and "bar", the user can mess things up by setting Ret to be a subtype of { foo : number, bar : string }.
You pointed out that one "should not typecast something to any just to return it".
However, your Ret type parameter is exactly that.
It is a "hidden" cast to practically any data type one wants.

Doesn't seem like there's a bug here. Thanks for the good discussion everyone.


Throwable items (grenades) should default to throw

Description

When a grenade is activated the action should default to throw. Not that big a deal since shift + Q throws but it should reduce the amount of clicks where logical.

Nah, I think it's fine how it is.


Add cap-notify & away-notify support

Nope, any unknown/unsupported network capabilities are not advertised to clients. The first commit adds cap-notify so clients have a way to get notified, and the second commit introduces the first network-dependent capability. I chose away-notify because it's the most interesting to me. If only Freenode added support for it, there was no longer need for clients to do that stupid who-polling...

I am just wondering what is the status of this PR as the latest activity was 8 days ago.

@jpnurmi If you think it's ready to merge, please do so

I'll trust your choice...

Hi. I probably won't be active for a while.

Oops. That was an accident... :)

@jpnurmi but will you merge this?
And can I ask why you won't be active for a while?

I've tested this today and this works beautifully. Using UnrealIRCd <IP_ADDRESS>, a fresh build of the the cap-notify branch that is being merged here and HexChat 2.10.2 as client.

I hope someone merges this in near future.

Did you test cases like "first client connects, then znc", "first znc
connects, then client", "supporting server connects, then client connects,
then znc reconnects to non-supporting server", etc? With various
combinations of supporting/non-supporting the capability clients and
servers...
2015-05-28 15:13 GMT+01:00 Mikaela Suomalainen<EMAIL_ADDRESS>
I hope someone merges this in near future.
—
Reply to this email directly or view it on GitHub
https://github.com/znc/znc/pull/957#issuecomment-106338850.

I only tested with these particular versions which all support away-notify. We're running this in our environment now and don't plan to do any tests where either client or server is non-supporting. That still needs to be done.

I've refactored the PR. Reviews welcome.

Did you test various corner cases as described above?

Would away-notify go through ZNC properly if I added requesting it to *perform on networks that are known to support it before this is merged?

I've tried to test all possible combinations of servers and clients with and without cap-notify/away-notify support...

I think capability shouldn't be offered to client until it got negotiated with server.

Good point, thanks for making me realize that GetAvailableCaps() was a relic from the times when I was still trying to let clients request server caps directly. Updated again.

Updated to use std::pair & {}'s, and tweaked the initializer list formatting.

Anyway, LGTM


Consider using pydantic aliases

I think we might be able to simplify the process of pairing internal schema names to the formatted postera names by using aliases in pydantic. See the class here for example, to work out what the the postera name is for the variable szybki_global_strain we have to check the function here then we have to map the enum name here. Using an alias would allow this to all be defined in a single place like so
from pydantic import BaseModel, Field

class SzybkiFreeformResult(BaseModel):
    """
    Class for storing the results of a Szybki Freeform conformer analysis run.
    """

    ligand_id: str = Field(..., serialization_alias="ligand_id")
    szybki_global_strain: float = Field(..., serialization_alias="ligand-global-strain-szybki-kcal-mol")

c = SzybkiFreeformResult(ligand_id="id_number1", szybki_global_strain="4")
print(c.model_dump(by_alias=True))

{'ligand_id': 'id_number1', 'ligand-global-strain-szybki-kcal-mol': 4.0}

cc @hmacdope

Yes please! This is a great idea. We will need to update the renaming methods to take a schema but I think this is a good change.


Disable IPv6 by default and auto generate admin password in helm Chart

This fixes the issues mentioned in #82 by auto generating an admin password in the helm chart if none is provided, and by disabling IPv6 by default on Kubernetes.
Problem
Unfortunately, most k8s clusters do not support IPv6 in containers by default, and need some extra configuration on the kubelet to get it to work.
When deploying the helm chart on one of those clusters that does not support IPv6, the creation of the pod fails, because it cannot assign an IPv6 address.
Additionally, the pod also fails to start if the web.config.adminUsername value in the helm chart is not provided by the user in their values.yaml.
Changes
To make the chart work out of the box on non-IPv6-k8s-clusters (the majority), this PR sets config.vpn.cidrv6 to 0 by default in values.yaml. This disables IPv6 in the wg-access-server and prevents it from failing to start on non-IPv6-clusters.
This PR also adds the helper function wg-access-server.adminPassword in _helpers.tpl, which checks if .Values.web.config.adminUsername is set, and returns it, or otherwise generates a random, 20 characters, alphanumeric password and returns that.
This helper function is then used in secret.yaml to set the adminPassword.
Additionally, a NOTES.txt was added to the helm template, informing the user of the auto generated password and how to retrieve it, should it have been generated upon install.

Thanks! For what I can tell with my very limited Helm & k8s knowledge, this looks good to me.
Two minor things: could you also add a note to the Helm README (deploy/helm/wg-access-server/README.md), both about IPv6 being disabled and the auto-generated admin password?
Maybe something like:

Because IPv6 on Kubernetes is disabled by default and complicated to enable, the default values.yaml disables it for the VPN as well. If you have a cluster with working IPv6, remove the vpn.cidrv6: 0 section or specify a custom VPN-internal prefix.

and

If no admin password isn't set, the Chart generates a random one. You can retrieve it using kubectl get secret ...

Thanks for the hint, I added two sentences to the README.


Budget App

In this capstone project, the following tasks were completed:

Created a new rails app and set it up with PostgreSQL.
Setup linters
Installed rspec and capybara for testing.
Generated the user, group, expense and group_expense models.
Implemented the associations between the models.
Validated each model attributes.
Generated controllers for the user, group, expense and group_expense models.
Added the routes to the various views of the controllers.
Implemented authentication using devise.
Created a splash screen to display the logo of the app and include buttons to the login and sign up pages.
Created and styled the login and sign up pages.
Created the homepage to display all the categories created by the current user.
For each category, the user is allowed to view all transactions for the category and see the total amount for the category.
Added a button to allow the user to add new categories.
Created the transactions page to display all transactions of a particular category.
For each transaction, the user sees the name and amount of the transaction.
Added a button to allow the user to add transactions to the specific category.
Added a navigation bar.
Added specs for the application.
Documented the application with an updated README file.
Deployed the app.
Added a video presentation to the README file.

Hi @Joel-100, Can you please add content for all sections in your readme, please remember that all sections is required in your readme except the FAQ's

Yah sure thanks Priscilla,let me fix that!

Hi @Joel-100 can you please reopen the pr?

Thanks @prolajumokeoni


Issue with vertical 2K screen and Fancy Zones. Although Fancy Zones show the correct formation, when holding Shift to activate zones, the zones are set for different resolution.

Microsoft PowerToys version
v0.37.2
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce

Vertical (portrait) orientation of Philips 275E monitor (1440 x 2560 px)
Fix the orientation like you would like them to show, or use the pre-built / default ones
Hold Shift to drag the windows, and the zones (although looking like they were created just fine) are smaller than they appear

✔ Expected Behavior
If the zones are 1420px wide, they should hold the full pixel count of the zone and not appear smaller.
❌ Actual Behavior
If the zones are 1420px wide, they hold the full pixel count - the amount missing from a FullHD monitor (max ~ 1060 px) .
powertoys bug 1.zip
Other Software
No response

does this issue still happen with v0.73.0? /needinfo

I don't know which exact version fixed it, but the issue is not present in the latest version.


chore: update dev dependencies

:tada: This PR is included in version 2.2.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Using STM32Cube HAL function results in linker error

Describe the bug
I am using PlatformIO to create a Zephyr project for the STM32 Bluepill (STM32F103C8, maps to the stm32_min_dev (blue) variant). The base project is https://github.com/platformio/platform-ststm32/tree/develop/examples/zephyr-blink.
In the firmware I'm trying to use the STM32HAL GPIO functions (such as HAL_GPIO_Init()) instead of Zephyr's facilities. So I expand the original code to #include <stm32f1xx_hal.h> and call these functions.
However, building the firmware then fails at the linking stage (undefined reference to HAL_GPIO_TogglePin'`).
To Reproduce
Steps to reproduce the behavior:

Have PlatformIO installed (e.g. in the CLI)
Duplicate the original project
Replace src\main.c with the code found below
Attempt compilation with pio run
See error

src\main.c
#include <zephyr.h>
#include <device.h>
#include <devicetree.h>
#include <drivers/gpio.h>
#include <stm32f1xx_hal.h>

/* 1000 msec = 1 sec */
#define SLEEP_TIME_MS   1000

/* The devicetree node identifier for the "led0" alias. */
#define LED0_NODE DT_ALIAS(led0)

#if DT_NODE_HAS_STATUS(LED0_NODE, okay)
#define LED0    DT_GPIO_LABEL(LED0_NODE, gpios)
#define PIN     DT_GPIO_PIN(LED0_NODE, gpios)
#define FLAGS   DT_GPIO_FLAGS(LED0_NODE, gpios)
#else
/* A build error here means your board isn't set up to blink an LED. */
#error "Unsupported board: led0 devicetree alias is not defined"
#define LED0    ""
#define PIN     0
#define FLAGS   0
#endif

void main(void)
{
        const struct device *dev;
        bool led_is_on = true;
        int ret;

        dev = device_get_binding(LED0);
        if (dev == NULL) {
                return;
        }

        ret = gpio_pin_configure(dev, PIN, GPIO_OUTPUT_ACTIVE | FLAGS);
        if (ret < 0) {
                return;
        }

        __GPIOC_CLK_ENABLE();
        GPIO_InitTypeDef GPIO_InitStruct;
        GPIO_InitStruct.Pin = GPIO_PIN_13;
        GPIO_InitStruct.Mode = GPIO_MODE_OUTPUT_PP;
        GPIO_InitStruct.Speed = GPIO_SPEED_HIGH;
        HAL_GPIO_Init(GPIOC, &GPIO_InitStruct);

        while (1) {
                gpio_pin_set(dev, PIN, (int)led_is_on);
                led_is_on = !led_is_on;
                HAL_GPIO_TogglePin(GPIOC, GPIO_PIN_13);
                k_msleep(SLEEP_TIME_MS);
        }
}

Expected behavior
STM32HAL functions are available and compiled in.
Impact
Compilation failure, showstopper.
Logs and console output
<EMAIL_ADDRESS>.pio\build\bluepill_f103c8\src\main.o: in function `main':
C:\Users\Max\temp\zephyr_bluepill/src/main.c:46: undefined reference to `HAL_GPIO_Init'
<EMAIL_ADDRESS>C:\Users\Max\temp\zephyr_bluepill/src/main.c:51: undefined reference to `HAL_GPIO_TogglePin'
collect2.exe: error: ld returned 1 exit status
*** [.pio\build\bluepill_f103c8\zephyr\firmware-pre.elf] Error 1

Environment (please complete the following information):

OS: Windows
Toolchain: arm-none-eabi-gcc and friends (8.2.1)
Commit SHA or Version used: 2.6.0 (stable release)

Additional context
When I clean the project I interestingly see some STM32Cube files being compiled, just not stm32f1xx_hal_gpio.c..
Removed .pio\build\bluepill_f103c8\_pio__modules__hal__stm32__stm32cube\modules\stm32\stm32cube\stm32f1xx\drivers\src\stm32f1xx_hal.c.o
Removed .pio\build\bluepill_f103c8\_pio__modules__hal__stm32__stm32cube\modules\stm32\stm32cube\stm32f1xx\drivers\src\stm32f1xx_hal_rcc.c.o
Removed .pio\build\bluepill_f103c8\_pio__modules__hal__stm32__stm32cube\modules\stm32\stm32cube\stm32f1xx\drivers\src\stm32f1xx_hal_rcc_ex.c.o
Removed .pio\build\bluepill_f103c8\_pio__modules__hal__stm32__stm32cube\modules\stm32\stm32cube\stm32f1xx\drivers\src\stm32f1xx_ll_utils.c.o
Removed .pio\build\bluepill_f103c8\_pio__modules__hal__stm32__stm32cube\modules\stm32\stm32cube\stm32f1xx\soc\system_stm32f1xx.c.o

Attempting to set CONFIG_USE_STM32_HAL_GPIO=y in the zephyr\prj.conf results in a
error: USE_STM32_HAL_GPIO (defined at modules/Kconfig.stm32:189) is assigned in a configuration
file, but is not directly user-configurable (has no prompt).

error. Adding it in the board defconfig file results in the same message.
Used packages and ST-STM32 platform version:
PLATFORM: ST STM32 (14.1.0) > BluePill F103C8
HARDWARE: STM32F103C8T6 72MHz, 20KB RAM, 64KB Flash
DEBUG: Current (stlink) External (blackmagic, cmsis-dap, jlink, stlink)
PACKAGES:
 - framework-zephyr 2.20600.210721 (2.6.0)
 - tool-cmake 3.16.4
 - tool-dtc 1.4.7
 - tool-ninja 1.9.0
 - toolchain-gccarmnoneeabi 1.80201.190214 (8.2.1)

PlatformIO is of the latest dev version (version 5.2.0b1).

That's right, zephyr project is based on the stm32 LL drivers, not HAL from the stm32Cube. Mainly because the init and configuration steps are done by the zephyr itself with unitary (stm32 LL) functions.
The stm32 LL is more optimized for the Zephyr than the application-oriented stm32HAL.

The configuration file https://github.com/zephyrproject-rtos/zephyr/blob/main/modules/Kconfig.stm32 clearly contains both HAL and LL enabling directives with distinct macros like USE_STM32_HAL_GPIO and USE_STM32_LL_GPIO. So both HAL and LL code are in there and I can see these source files clearly in https://github.com/zephyrproject-rtos/hal_stm32/tree/main/stm32cube/stm32f1xx/drivers/src and these macros being used in https://github.com/zephyrproject-rtos/hal_stm32/blob/main/stm32cube/stm32f1xx/CMakeLists.txt.
But somehow including these files is not working.

by a quick and dirty patch, If you force the compilation of the stm32f1xx_hal_gpio.c by changing the
zephyr_library_sources(drivers/src/stm32f1xx_hal_gpio.c)
in the ../modules/hal/stm32/stm32cube/stm32f1xx
The gpio_hal functions are compiled and also declared in the ../modules/hal/stm32/stm32cube/common_ll/include/stm32_ll_gpio.h by:
...
#elif defined(CONFIG_SOC_SERIES_STM32F1X)
#include <stm32f1xx_hal.h>
#include <stm32f1xx_hal_gpio.h>
#include <stm32f1xx_ll_gpio.h>

Then the HAL_GPIO_Init  is defined in the src/main.c with a simple
#include <stm32_ll_gpio.h>

If you force the compilation of the stm32f1xx_hal_gpio.c by changing the
zephyr_library_sources(drivers/src/stm32f1xx_hal_gpio.c)

Does this not apply that there's some bug in the build system if the original expression
zephyr_library_sources_ifdef(CONFIG_USE_STM32_HAL_GPIO drivers/src/stm32f1xx_hal_gpio.c)

is not working, although the config macro should be defined?
https://github.com/zephyrproject-rtos/zephyr/blob/21d1ad3762302b3e461953df59430c77e0709274/modules/Kconfig.stm32#L9-L14
https://github.com/zephyrproject-rtos/zephyr/blob/21d1ad3762302b3e461953df59430c77e0709274/modules/Kconfig.stm32#L189-L192

I think the CONFIG_USE_STM32_HAL_GPIO is missing in the soc/arm/st_stm32/stm32f1/Kconfig.series :
	select USE_STM32_HAL_GPIO

@maxgerhardt Did you had a look to https://github.com/zephyrproject-rtos/hal_stm32#use-stm32cube-in-your-application ?

Indeed, solved in PlatformIO projects by creating the file zephyr\Kconfig with content
mainmenu "MYAPP"

source "Kconfig.zephyr"
config MYAPP_STM32
  default y
  bool
  select USE_STM32_HAL_GPIO
  select USE_STM32_LL_GPIO


Handle stray quotes in behavior data

BehaviorImporter.new.connect_to_x2_and_import
# => CSV::MalformedCSVError: Missing or stray quote in line 7

One of the CND_INCIDENT_DESCRIPTION rows contains quotes, like this:
"I told them, "...""

We should handle in behavior_export.sql with another REPLACE statement.


Turn numalgos into child module of peer_did module

Turn numalgos to be child model of peer_did. A "numalgo" abstraction doesn't exist outside of peer dids, now this is reflected via the structure itself. It also removes some discrepancy, as we had both src/numalgos and src/peer_did/numalgos

old/new directory structure compared:

I would further suggest to pull everything in src/peer_did a level up, pulling those files to the root. That will make imports in 3rd party pkgs nicer, is it won't duplicate peer_did token in the import path twice.
Additionally

Merge file did_peer/src/peer_did/regex.rs and did_peer/src/peer_did/validate.rs
Rename module peer_did_resolver to resolver as the prefix seemed redundant, given we operate in did_peer package
Renamed GenericPeerDid to AnyPeerDid - this is bit more speculative and shall this change be accepted, we should probably do the same thing with Connection (perhaps AnyStateConnection instead of GenericConnection. I personally don't like the type name to contain "Generic" as it's makes spoken speech more difficult to understand - easy to confuse someone saying "GenericFoobar" as type GenericFoobar or "Generic Foobar", as in terms of generic, the language feature.

We're currently processing your upload.  This comment will be updated when the results are available.


Adversarial attacks on neural networks

Topic Suggestion
Machine Learning
Writing sample(s):
Blog Samples
Proposal Submission
Proposed title of article
Adversarial attacks on neural networks
Proposed article introduction
With the current research advancements in deep learning since the past decade, we can say that we live in the era of neural networks and expect them to play more significant roles in the coming years. From suspicious activity monitoring to self-driving cars, we are harnessing its abilities in countless dimensions. Now imagine a scenario where you are using a deep learning-based security system in your home, and some thief comes and fools the security system just by manipulating his appearance. In human eyes, we could quickly tell that someone unknown is entering your house, but the security system treats them as house owners. Yes, this can happen, and these types of attacks are termed Adversarial attacks. If you have trained a neural network without keeping an adversarial attack in your mind, you can easily be exposed to such security threats. And therefore, it can easily be fooled. This article will look at why and how neural networks are prone to adversarial attacks, implement an adversarial attack, and finally discuss how to prevent it.
Key takeaways
The article will cover:

Introduction to Adversarial attacks and their types.
Why adversarial attack fools a neural network?
Implementing an adversarial attack.
How to prevent these attacks?

Article quality
Most of the articles present on the Internet about adversarial attacks on neural networks are vague and theoretical. In this article, not only we will discuss the maths behind it but also implement it from scratch.
References

https://pytorch.org/tutorials/beginner/fgsm_tutorial.html
https://towardsdatascience.com/how-to-fool-a-neural-network-958ba5d82d8a

Conclusion
The reader will understand how to carry an adversarial attack and how to prevent it.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it). @DeewakarChakraborty
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved


Support for linearGradient and related outside defs doesn't exist

In one of the test fonts we were given for OT-SVG that you can find here, I found an SVG document
having a radialGradient element outside defs section. This is allowed in SVG specs, AFAIK. However, such an element is totally ignored by SVG Native Viewer.

@moazin That is indeed a limitation in SVG Native Viewer and can be fixed easily. Thanks for reporting.


Integrating the generator with model.fit of keras.

Hi can you please show how to integrate the generator with the keras model.fit function.

@Suakuw Hello, I enhanced the codes so that they can be used for keras's model.fit_generator and updated README. Please look into them.
Thank you.


Ensure absolute file path fetching works from within a container

The Dockerized app needs to

Be able to access .tsv / .json / .jsonld files and BIDS directories stored on the host file system, which are passed to the bagel commands as arguments (either as an absolute or relative path)
Be able to write .jsonld files to disk which persist on the host directory after the container exits

[x] Include instructions for mounting dataset directory (should contain BIDS data, if available) during docker run step in README

How will the user interact with the dockerized app?
1. Encode bagel command in Dockerfile CMD, have users pass in only subcommand + arguments during docker run
2. Have users run the container interactively, and run bagel commands as normal while attached

Closed with #98


[PE-D][Tester A] Unexpected behaviour of Delete Module command

Similar to issue in Add Module command, UG didn’t specific which context this command are allowed to executed. So when trying to delete mod in lecture and video context.

Figure 1. delete mod in lecture context

Figure 2. delete mod in video context

Labels: type.FunctionalityBug severity.Medium
original: Jiayan-Lim/ped#7

We will update our UG to be specific about it


Arabic Localization settings do not match LUIS

The only supported LUIS lang-locale is "ar-ar".  However, Composer localization settings do not have this lang-locale. End result is that there is no way to publish a LUIS model in Arabic from Composer.

Adding back customer-reported tag, this was indeed customer reported incident that I filled on behalf of.


Configuration#7: Step#1: alpha_S=0.2, matter: q_0=1 | matterlbt: q_0=2.5: matterlbt06

run the simulation with the above config by jetscape,
transfer the final state hadron to the related directory named as config-XX-final-state-hadrons
after doing step1 on all 12 simulations go to step 2 for the configurationX

port to Excel for overleaf


Reloading the window when project folders are open fails

Running version 1.0.2 I realized that reloading Atom in full screen mode causes the application to just remain black. Reload only works if the app is taken out of full screen mode first. Is this behavior expected?

@Nickersoft I'm having trouble reproducing that on OSX. Can you share more information?

Thank you for your issue!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/7818#issuecomment-120438259. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.


Fixing incorrect link to Visual Studio Test task

Correcting https://github.com/Microsoft/azure-pipelines-tasks/blob/master/Tasks/VsTestV2/README.md to ../pipelines/tasks/test/vstest.md
Fixes #9634

@raydixon : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good (verified). Thanks.


DBZ-1730 Resume Postgres steams from last_flushed_lsn when offset storage not available.

Changes the behaviour of the PostgresConnector to resume from the last flushed LSN when the previous offset information is not available, but the replication slot is.
Previously the connector resumed from the LSN of the latest committed transaction, which is a potentially data loss scenario for resuming the change stream.
https://issues.redhat.com/browse/DBZ-1730

@gunnarmorling / @jpechane any thoughts on this?

@cjbooms I have the same issue, because we also run a similar solution at heycar.
On solution I have been thinking of, but doesn't look nice without some considerable changes to the embedded engine would be to provide a new OffsetBackingStore that is initialized with the last flushed lsn.
I will try to kick it off without changing debezium to see how far and how "meh" it will look like, so I can have a clear idea on what can be changed on debezium to make it better. The problems I'm seeing so far are related on the creation of the OffsetBackingStore itself and the dependencies that I don't write to write over again that are already part of the Engine.

@gregoriokusowski I was thinking something similar if the maintainers don't want this addressed in a more global way.  A custom OffsetBackingStore would both contain this change in one place, and act as a sort of feature-toggle for users that are not interested resuming from the confirmed_flush_lsn.
I reckon a subclass of MemoryOffsetBackingStore would be a good place to start. We may be able  to completely isolate the reinitialisation from confirmed_flush_lsn in that subclass.

Hey @cjbooms and @gregoriokusowski, thought a bit more about this with @jpechane and we pretty much concluded it's not something that should be supported in Debezium. The reason being that -- while I can see how it would help in your particular case -- it's not a generally viable solution. There's more in the offsets besides the plain WAL position, e.g. information about ongoing snapshots or, in the future, information about the set of captured tables (so to handle changes of whitelist/blacklist configuration).
But I think there's also a rather simple way for you out: you could just manually produce an offset record into the Kafka offset topic for that connector, based on your replication slot position. So maybe it'd help if we provided instructions for doing so?  We were also considering to build a small tool for that, but didn't get beyond idea stage with that one yet.

Thanks for the reply! Unfortunately my plan was to go with Embedded+Kinesis (so no kafka at all), and since I currently deploy my CDC on top of kubernetes, I would like to keep the configuration and state as simple as possible.
@gunnarmorling do you think it worth to try out a new implementation of the OffsetBackingStore approach as one alternative solution? I'm asking because I don't have the full context about the offsets.

A similar situation for us. We were hoping to publish directly from Embedded to Nakadi. We have no need for Kafka, or any additional persistent offset storage. Tracking the LSN in the database replication slot has been adequate for us to date.
I'm open to using FileBasedOffsetStorage if persistent offset storage is absolutely necessary, but I'm still left with the problem of migrating our existing replication slots onto Debezium without data loss or a flood of duplicate events.

@gregoriokusowski @cjbooms Maybe this is a spot on as you suggested, writing a custom OffsetBackingStore that would initiate (make inital offset) out of replication slot values?

OK, thanks @jpechane. @gregoriokusowski I or my team should have the opportunity to attempt a custom OffsetBackingStore this week. Will post progress back with progress toward the end of the week.
But I think there's also a rather simple way for you out: you could just manually produce an offset record into the Kafka offset topic for that connector, based on your replication slot position. So maybe it'd help if we provided instructions for doing so? We were also considering to build a small tool for that, but didn't get beyond idea stage with that one yet.

@gunnarmorling I would love to hear instructions for this suggestion

@cjbooms Please take a look at https://debezium.io/documentation/faq/#how_to_change_the_offsets_of_the_source_database - but that's for Kafka.
For File offset store you should use the code from org.apache.kafka.connect.storage.FileOffsetBackingStore.save()

Trying to find closure on this one; seems we pretty much concluded it can be handled in specific implementations at the client side at @cjbooms and @gregoriokusowski and we could close this one?

🤞 I still haven't gotten around to trying yet. Closing this. I'll open a new PR if the proposed solution works so you can consider it for inclusion.

Thanks!


Softmaker office not atachable to the tiling layout

Symptom
When I open a softmaker office window it isn't included in the layout. I can't add it to the layout with the keyboard shortcut either.
It worked in the past but I don't know which update brought the problem.
I've tested it with the karousel-kwin-script to sort out a general problem with kwin scripts and tested different packages (The AUR one and the official fedora package with distrobox). It worked with karousel but both the AUR and the fedora package did not. So I think it's a problem with krohkite. I have no idea why it happens. No other window on both of my systems where able to reproduce the problem.
I've enabled the debug options and attached the logs here:
systemd:
Started TextMaker NX - Word Processor.

kwin_wayland:
kf.windowsystem: static bool KX11Extras::mapViewport() may only be used on X11

kwin_wayland:
qml: Filtered: krohnkite:<caption: TextMaker> <output.name: eDP-1> <resourceName: tm> <resourceClass: tm>
<windowType: 0> <pid: 0> <internalId: {561b1875-3d37-4b3f-9209-27cfe6b89d93}> <stackingOrder: 0>
<size: QSizeF(1528, 1042)> <width: 1528> <height: 1042> <managed: true> <fullScreenable: true>
<closeable: true> <minimizable: true> <resizeable: true> <minSize: QSizeF(0, 0)>
<maxSize: QSizeF(524287, 524287)> <moveable: true> <moveableAcrossScreens: true> <opacity: 1>

kwin_wayland:
qml: [9.941] Error raised from line 549

kwin_wayland:
qml: [9.941] Error raised from line 549

kwin_wayland_wrapper:
syntax error: line 1 of stdin

kwin_wayland_wrapper:
Errors encountered in stdin; not compiled.

kwin_wayland:
qml: [13.704] Error raised from line 549

kwin_wayland:
qml: [13.705] TypeError: Cannot read property 'internalId' of null

Environment

Distro: arch linux
KWin version: 6.1.5
Krohnkite version: <IP_ADDRESS>

It's because <pid: 0>, you can try this (delete .zip )krohnkite-<IP_ADDRESS>.kwinscript.zip

It worked. But what is different there? This isn't the normal release i guess?

This isn't the normal release i guess?

The changes will be in next releases.
60ab23b20e90fc7456bbd4ff1f33758f6001bfe0


Corrections to running_ssh_service.md

File: engine/examples/running_ssh_service.md
The following RUN line should be changes:
from
RUN sed -i 's/PermitRootLogin prohibit-password/PermitRootLogin yes/' /etc/ssh/sshd_config
to
RUN sed -i 's/PermitRootLogin [a-zA-Z-]*/PermitRootLogin yes/' /etc/ssh/sshd_config
thus making it agnostic of image setting...

I'll take this one.

No action for two days, so this is up for grabs!

We decided not to merge the PR, but you get the points. thanks @jadametz !