Sibling properties of $ref are not ignored

According to the spec:

An object schema with a "$ref" property MUST be interpreted as a "$ref" reference.  The value of the "$ref" property MUST be a URI Reference.  Resolved against the current URI base, it identifies the URI of a schema to use.  All other properties in a "$ref" object MUST be ignored.

The current behavior is to process all properties regardless of whether $ref is present, which is a bug and contrary to the spec.
This needs to be fixed for 6.0.0, and should not be backported - it's a bugfix, but will break known behavior that some people may be depending on.
Big thanks to @johannes-see for noticing this in #435 :+1:.

Fixed in #437.


I am getting when importing numba in python3.7.2.Please provide solution for windows.

C:\Users\Dell\Desktop\ai-app\sound-mnist-master>py main.py
COMET WARNING: Native output logging mode is not available, fallbacking on basic output logging
COMET INFO: Experiment is live on comet.ml https://www.comet.ml/dolma/general/8e1a8d8e3e1642fca5ad623c44e72cec
Using TensorFlow backend.
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:523: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:524: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:525: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint16 = np.dtype([("qint16", np.int16, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:526: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint16 = np.dtype([("quint16", np.uint16, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:527: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint32 = np.dtype([("qint32", np.int32, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\framework\dtypes.py:532: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:541: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint8 = np.dtype([("qint8", np.int8, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:542: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint8 = np.dtype([("quint8", np.uint8, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:543: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint16 = np.dtype([("qint16", np.int16, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:544: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_quint16 = np.dtype([("quint16", np.uint16, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:545: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
_np_qint32 = np.dtype([("qint32", np.int32, 1)])
C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorboard\compat\tensorflow_stub\dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
np_resource = np.dtype([("resource", np.ubyte, 1)])
Traceback (most recent call last):
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\core\typeconv\typeconv.py", line 4, in
from numba.core.typeconv import _typeconv
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "main.py", line 8, in
from utils import model, wav2mfcc, get_data
File "C:\Users\Dell\Desktop\ai-app\sound-mnist-master\utils\wav2mfcc.py", line 2, in
import librosa
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\librosa_init_.py", line 12, in
from . import core
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\librosa\core_init_.py", line 125, in
from .time_frequency import *  # pylint: disable=wildcard-import
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\librosa\core\time_frequency.py", line 11, in
from ..util.exceptions import ParameterError
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\librosa\util_init_.py", line 77, in
from .utils import *  # pylint: disable=wildcard-import
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\librosa\util\utils.py", line 10, in
import numba
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba_init_.py", line 20, in
from numba.misc.special import (
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\misc\special.py", line 3, in
from numba.core.typing.typeof import typeof
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\core\typing_init_.py", line 1, in
from .context import BaseContext, Context
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\core\typing\context.py", line 11, in
from numba.core.typeconv import Conversion, rules
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\core\typeconv\rules.py", line 2, in
from .typeconv import TypeManager, TypeCastingRules
File "C:\Users\Dell\AppData\Local\Programs\Python\Python37\lib\site-packages\numba\core\typeconv\typeconv.py", line 17, in
raise ImportError(msg % (url, reportme, str(e), sys.executable))
ImportError: Numba could not be imported.
If you are seeing this message and are undertaking Numba development work, you may need to re-run:
python setup.py build_ext --inplace
(Also, please check the development set up guide http://numba.pydata.org/numba-doc/latest/developer/contributing.html.)
If you are not working on Numba development:
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!
The original error was: 'DLL load failed: The specified module could not be found.'
If possible please include the following in your error report:
sys.executable: C:\Users\Dell\AppData\Local\Programs\Python\Python37\python.exe
COMET INFO: ---------------------------
COMET INFO: Comet.ml Experiment Summary
COMET INFO: ---------------------------
COMET INFO:   Data:
COMET INFO:     display_summary_level : 1
COMET INFO:     url                   : https://www.comet.ml/dolma/general/8e1a8d8e3e1642fca5ad623c44e72cec
COMET INFO:   Uploads:
COMET INFO:     environment details : 1
COMET INFO:     filename            : 1
COMET INFO:     git metadata        : 1
COMET INFO:     installed packages  : 1
COMET INFO: ---------------------------
COMET INFO: Uploading stats to Comet before program termination (may take several seconds)

Thanks for asking about this on the Numba issue tracker. I believe this is a duplicate of https://github.com/numba/numba/issues/5600 - please track that issue for a fix.


Added HSL to RGB and RGB to HSL snippets. 2 commits only.

@Trinityyi I have made the changes. Please have a look.

در خدمتم

در خدمتم بفرمائید

@Chalarangelo @Trinityyi There has been some updates in Hacktoberfest so could you please follow these guidelines so that my PRs count?
thank you so much!
Update link


Remember previous answers for easier re-syncing

[x] Following CODE_OF_CONDUCT.md.
[x] Checked for duplicate enhancements.
[x] Title is a summary of the enhancement request, in present tense, and ideally < 50 characters

Enhancement
https://github.com/sindresorhus/conf

Now that the CLI supports arguments, this feature is less useful. Closing.


[BUG] Procfile export order & ports

Describe the bug
Exporting a procfile does not preserve procfile order, and includes redundant PORT variables.
diff --git a/Procfile b/Procfile
index bd49e77..2903ab6 100644
--- a/Procfile
+++ b/Procfile
@@ -1,3 +1,3 @@
-server: ./script/dev-server.sh
-gui: ./script/dev-gui.sh
-storybook: ./script/storybook.sh
+gui: PORT=5100 ./script/dev-gui.sh
+server: PORT=5000 ./script/dev-server.sh
+storybook: PORT=5200 ./script/storybook.sh

To Reproduce
On current main, just do exo apply Procfile and you'll immediately see an "out of date" warning in the GUI. Press "export" then, get diff above.
Expected behavior

If the PORT would be redundant from the implicit port order in the procfile, we should omit the PORT=whatever from the exported service lines in the outputted procfile.

Somehow, we should preserve the order of entries in the procfile. Not sure how this should work, maybe some kind of metadata on the components. Like procfile:index=2 or whatever & then a best-effort stable sort used in export? Open to other ideas too.

This is also causing some test flakiness, so I'll fix it.


JSON FILE?

WHERE CAN I FIND THE .JSON FILE OF THE MPI DATA?

The JSON file of MPI DATA is the data produced by our private scanner. Or you can export the json during the simulation as a json file.

Thank you for your reply?
can you just explain me the structre of the .json file: i mean as a dictionar how many valeus and keys insde it ?

You can refer to the paper: A novel software framework for magnetic particle imaging reconstruction


Make noir-libs a separate package (name tbd)

noir-libs should be downloadable as its own thing, without downloading the entire monorepo.
I wouldn't go as far as to put the code in a separate repo though. Monorepo removes friction from developing.
Thoughts @LHerskind @ludamad @Maddiaa0 @sirasistant ?
(https://github.com/AztecProtocol/aztec-packages/pull/1759#discussion_r1309013217)

Do you mean in nargo? You can just add it as a dependency on Nargo.toml using
easy_private_state = { git="https://github.com/AztecProtocol/aztec-packages", tag="master", directory="yarn-project/noir-libs/easy-private-state" }

Why would you need to download it manually?

I think not having to include the directory bit would be useful

I suspect the mirror repo for aztec.nr achieves this, and this can be closed?

Yep. Should be fine to close here. The naming might be good to think about though with so many different things living in there 🤔


Node JS version 12 is deprecated

Hello,
We are using the azure blob storage upload/download actions in our github workflows. Recently I noticed following warning, can you please publish a new version with the updated Node.js version?
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: armanrahman22/azblob-download-action, armanrahman22/azblob-download-action
Thanks!

same


[Question] Active/Inactive findings behavior

When importing a scan result, the Findings are always set to Inactive.
When the user check the Active box, the Findings are set to Active, even if the Finding is marked as False Positive.

I do not understand the behavior. When I import findings from other tools, I assume them to be set as by default Active when not status is provided by the scanning tool (false positive, ...).
For the moment, I force the Active checkbox, but this is causing issue by setting both flags Active & False positive, which should not be possible.
Identified on 1.5.4 RC6.

yeah nobody really knows how it is supposed to work and it works inconsequently across DD. It's on our list to fix. usually the import itself shouldn't set false positive, or is it a duplicate of an existing false positive? or which scanner are we talking about?

This is the Checkmarx scanner. I am also using the Sonatype scanner and I expect the same behavior(will take a look). These tools allow to qualify the defects, and I expect to not lose it when importing into DD.


pass settings from -Xjuliacall_ to julia

added support for --threads, --procs, --min-optlevel
Fixes #184

Great PR, thank you! Most of the options seem to work great, but some require special handling:

opt_bindir translates to --home not --bindir. Could rename it opt_home?
opt_bindir and opt_sysimage are parsed by jl_parse_opts but have no further effect. I think the call to jl_init__threading needs to be replaced by jl_init_with_image__threading.
opt_procs also has no effect, it's handled separately in Base here (but is only called later in the loading process than jl_init): https://github.com/JuliaLang/julia/blob/762561c3057b4bd541d183ca1d1b70b9945bff22/base/client.jl#L245-L254

The first one should be fixed in this PR. I'm happy to leave the others for separate PRs (and not document them yet) - it's up to you.
Speaking of bindir, there is the Julia env var JULIA_BINDIR and the command line option --home, hence the naming discrepancy in the first bullet point. It seems like JuliaCall should have similar behaviour (PYTHON_JULIACALL_BINDIR and -X juliacall_home) to be consistent with Julia. Opinions?

I've just pushed a change to the main branch which, when merged with this PR, means that opt_procs works.

After merging this, I could make a PR for supporting opt_bindir and opt_sysimage.

thank you guys!

I've merged this PR and made a few tweaks, to get the bindir/home option working. I've also changed the -X options to use --separated words instead of _. And they are documented here: https://cjdoris.github.io/PythonCall.jl/dev/juliacall/#julia-config
A PR to support bindir and sysimage would be great.


Fix bitmap crop/scaling for android

Summary
Fixes #150
Test plan
I added a "scale" to the test app, so that you can test with a lot of downscaling of the image. The bug can reproduced by running the example and revert the fixed code in /imageeditor/ImageEditorModuleImpl.kt, then set the scale to less than 0.5. With the fixed code in /imageeditor/ImageEditorModuleImpl.kt, the issue is gone

Thank you @jthure

No problem! Happy to help


docs: link to docs on semantic version constraint expressions

Link from our docs to documentation on semantic version constraint expressions.
https://github.com/masterminds/semver#checking-version-constraints

The best places for this may comments on the applicable CRD fields (these will work their way into CRDs reference docs) and a info box below any example in the quickstart or concepts doc that uses a semver constraint.

I'll pick this up.
For the info box in the docs I was thinking something like:
:::info
Kargo uses SemVer to check handle semantic versioning constraints.
:::
below any example that includes a constraint.


[Bug]: Submit button accepting empty feedback #272 : fixed

[Bug]: Submit button accepting empty feedback #272
It is working now.
Only if the fields have input, feedback will be submitted, else it will raise an alert.
image
image

Updated UI:

Good Job @akank1410 .


chore(examples): remove extra - for steps

Description:
Fix example steps

Motivation:
bad syntax

Closes Issues:

ISSUE URL

Much appreciated  @Tiagoqt


Improve server connection error messages

It's already used for self auth and is available for both, but we currently only show it for the former.
recommends R2Northstar/NorthstarLauncher#433

i did not mean to approve this i meant to do requires changes FUCK


Some suggestions for ease of use

I wrote a small wrapper while using this library recently, and there might be some bits worth considering for upstreaming into the library.

Make the common case of using the supplied unigram/bigram data automatic, by using cached_path
Provide the common configuration for a Segmenter
Hide the Search object or explain caveats around it  - I'm not sure why it is required
Handle empty strings
Perhaps handle non-lowercase ascii characters (by returning them as segments or skipping them?)

from dataclasses import dataclass
from functools import lru_cache

import instant_segment
from cached_path import cached_path

BIGRAMS_DATA_PATH = (
    "https://github.com/InstantDomain/instant-segment/raw/main/data/en-bigrams.txt"
)
UNIGRAMS_DATA_PATH = (
    "https://github.com/InstantDomain/instant-segment/raw/main/data/en-unigrams.txt"
)

def unigrams():
    for ln in cached_path(UNIGRAMS_DATA_PATH).read_text().splitlines():
        parts = ln.split("\t", 1)
        yield (parts[0], float(parts[1].strip()))

def bigrams():
    for ln in cached_path(BIGRAMS_DATA_PATH).read_text().splitlines():
        word_split = ln.split(" ", 1)
        score_split = word_split[1].split("\t", 1)
        yield ((word_split[0], score_split[0]), float(score_split[1].strip()))

@dataclass
class Segmenter:
    def __post_init__(self):
        self.segmenter = instant_segment.Segmenter(unigrams(), bigrams())
        self.search = instant_segment.Search()

    def segment(self, word: str) -> list[str]:
        if not len(word):
            return []
        self.segmenter.segment(word, self.search)
        return list(self.search)

    @classmethod
    @lru_cache(maxsize=1)
    def instance(cls):
        return Segmenter()

I'm not sure we should bring in the dependency on cached_path, I suppose we could do a minimal version of that?
Seems fairly trivial once you have 1?
I guess it's mainly useful on the Rust side -- it allows you to reuse caches when running multiple segmentations in parallel.
See #37.
Not sure what you see as the use case? There doesn't seem a single obvious solution so I think it makes sense for the caller to handle this part.

Sounds good, thanks for the quick fix on (4).

Fine by me to not pull in the dependency, though I also don't see the downside
🤷🏾 All of these suggestions are trivial, just helps to go from "install library -> read some docs and write some helpers, maybe in a util file -> first use" to "install library -> use"
Gotcha. Yeah, again, hiding it helps with that tiny bit of friction
It's easy enough to handle, just surprised me with an error in the middle of a long list. Then there's a bit of regex to identify the offending characters, pre-split on that, then segment each split and chain them together...

Would you be interested in submitting a PR to keep a Search handy inside the Python Segmenter so that you don't have to pass one in? Should be pretty straightforward...


Feature quote and extrinsics for Asset Conversion #8692tuufm

implement quote fetch (conversion rate) for the Asset Conversion pallet
implement swaps extrinsic building for the Asset Conversion pallet
add integration tests for quote fetch and extrinsic building

Task linked: CU-8692tuufm iOS. Concrete logic implementation for Asset Hub


Warn about undefined predicates

Logging here what happened in https://github.com/arichiardi/js-spec-chai.
When mistakely using predicates that resolve in undefined, warn the user.
For instance this should be detected and avoided:
import * as s from 'js-spec'
console.log(s.string)
// => undefined
console.log(s.spec.string)
// => Function

Peace :peace_symbol:

In 1.0.0-10 you should see an error whenever you try to use undefined predicates :)

Awesome thank you!
Sent from my Android device with K-9 Mail. Please excuse my brevity.


[CKAN] Outdated "Home page"

CKAN still shows this as this mod's "Home page": https://github.com/KSP2Community/PatchManager/wiki

I can see the correct page in CKAN. You may just need to let it stay open for a bit for it to update, since it has to refresh the information from SpaceDock.


Wrong type for client secret

In the section "Creating a service principal in Azure" you have client_secret as "<GUID>" hinting that the type is Guid but it is not
https://learn.microsoft.com/en-us/training/modules/implement-bicep/7-exercise-deploy-bicep-file-github-workflows

@YohannesKifle
Thanks for your feedback! We will investigate and update as appropriate.

We appreciate you posting the question here.
To receive insights from the targetted SMEs/audience, I request you to post the question on Microsoft Q&A /Azure Training tag.
https://learn.microsoft.com/en-us/answers/tags/447/azure-training
You may just repost the question here:  https://learn.microsoft.com/en-us/answers/questions/ask/?id=aHR0cHM6Ly9taWNyb3NvZnQtZGV2cmVsLnBvb2xwYXJ0eS5iaXovRGV2UmVsTWFjaGluZUdlbmVyYXRlZENvbXBvdW5kVGF4b25vbXkvODQ0YWI2NTEtNzBmMi00YjUzLTkxZWYtZTkzMjZhZWZjMDZl&styleGuideLabel=Azure Training
This info is also added in the footer of the Learning module,

@AjayKumar-MSFT this is not a question. It's a small correction that needs to be made


Scala Core: consider adding of Resolver config

Currently, format of Resolver configuration is described only in Scala Iglu Client, but nothing in it is specific for Scala, also Resolver configuration looks like core concept for Iglu and therefore should be parseable from any other language.
case class RepositoryRefConfig(name: String, instancePriority: Int, vendorPrefixes: List[String])

trait RepostiroryRef { val config: RepositoryRefConfig }
case class HttpRepositoryRef(config: RepositoryRefConfig, uri: String, apikey: Option[String]) extends RepositoryRef
case class EmbeddedRepositoryRef(config: RepositoryRefConfig, path: String) extends RepositoryRef

case class Resolver(cacheSize: Int, repos: List[RepositoryRef])

On the other hand, I see no clear benefits for having Resolver configuration anywhere except client (basically client == resolver). I think it should be added in Core whenever any other application require resolver_config.json parsing.

One hazy benefit is excluding embedded schemas from Iglu Client.

I think this is a good idea @chuwy !

Migrated to https://github.com/snowplow-incubator/iglu-scala-core/issues/25


Write migration script to update study fields to support lists

We need a migration script that can update our currently deployed instances to support the list field without re-importing all the data (ref: https://github.com/HBGDki/study-explorer/issues/5).

@elgertam Any update on this? We will need to the migration soon. Thanks.

This issue is addressed in https://github.com/HBGDki/study-explorer/pull/15.


Tile2world & world2tile - decoding optimization.

Optimize tile2world.
80% of math in tile2world and world2tile is constant in scope of tile.

Decoders tests please

Manual measurements on NodeJS
Improvement in whole decoding time.

Shere: 10-30%
Mercator: 10-20%

Wondering if in-probwer tests will show the same.
#performance loading baseline from .profile-helper-baseline.json
#performance OMVDecoderPerformanceTest-theme=berlin tiles=4 region=berlin data=herebase webMercator
  min=103.08ms (-13.88% vs 119.69ms) sum=1130.04ms (-14.29% vs 1318.43ms) repeats=10 throughput=8.8492/s (16.67% vs 7.5848/s)
  avg=113.00ms (-14.29% vs 131.84ms) med=108.36ms (-17.43% vs 131.23ms) med95=130.68ms (-12.87% vs 149.98ms)
  gcTime=108.57ms (-3.11% vs 112.06ms) sumNoGc=1021.47ms (-15.33% vs 1206.37ms) throughputNoGc=9.7898/s (18.1% vs 8.2893/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=4 region=berlin data=herebase sphere
  min=195.79ms (-19.69% vs 243.77ms) sum=2044.37ms (-21.4% vs 2600.95ms) repeats=10 throughput=4.8915/s (27.22% vs 3.8447/s)
  avg=204.44ms (-21.4% vs 260.09ms) med=199.97ms (-21.88% vs 255.97ms) med95=225.99ms (-21.12% vs 286.50ms)
  gcTime=161.95ms (-0.51% vs 162.79ms) sumNoGc=1882.42ms (-22.79% vs 2438.16ms) throughputNoGc=5.3123/s (29.52% vs 4.1015/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=4 region=berlin data=osmbase webMercator
  min=119.31ms (-20.43% vs 149.95ms) sum=1292.67ms (-29% vs 1820.77ms) repeats=10 throughput=7.7359/s (40.85% vs 5.4922/s)
  avg=129.27ms (-29% vs 182.08ms) med=124.68ms (-22.21% vs 160.27ms) med95=157.73ms (-50.56% vs 319.03ms)
  gcTime=104.67ms (-24.98% vs 139.52ms) sumNoGc=1188.01ms (-29.34% vs 1681.26ms) throughputNoGc=8.4175/s (41.52% vs 5.9479/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=4 region=berlin data=osmbase sphere
  min=184.95ms (-20.6% vs 232.94ms) sum=1922.23ms (-28.91% vs 2703.98ms) repeats=10 throughput=5.2023/s (40.67% vs 3.6982/s)
  avg=192.22ms (-28.91% vs 270.40ms) med=188.78ms (-28.09% vs 262.53ms) med95=213.88ms (-32.3% vs 315.89ms)
  gcTime=136.13ms (-7.22% vs 146.73ms) sumNoGc=1786.10ms (-30.16% vs 2557.25ms) throughputNoGc=5.5988/s (43.18% vs 3.9104/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=10 region=ny data=herebase webMercator
  min=495.30ms (-23.5% vs 647.46ms) sum=5365.29ms (-20.83% vs 6777.19ms) repeats=10 throughput=1.8638/s (26.32% vs 1.4755/s)
  avg=536.53ms (-20.83% vs 677.72ms) med=515.12ms (-24.12% vs 678.87ms) med95=680.28ms (-6.33% vs 726.26ms)
  gcTime=502.32ms (-16.7% vs 603.03ms) sumNoGc=4862.97ms (-21.24% vs 6174.17ms) throughputNoGc=2.0564/s (26.96% vs 1.6197/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=10 region=ny data=herebase sphere
  min=1120.88ms (-11.23% vs 1262.65ms) sum=11719.01ms (-10.91% vs 13153.44ms) repeats=10 throughput=0.8533/s (12.24% vs 0.7603/s)
  avg=1171.90ms (-10.91% vs 1315.34ms) med=1171.51ms (-10.48% vs 1308.70ms) med95=1239.37ms (-10.64% vs 1386.98ms)
  gcTime=813.02ms (4.31% vs 779.43ms) sumNoGc=10905.99ms (-11.86% vs 12374.01ms) throughputNoGc=0.9169/s (13.46% vs 0.8081/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=10 region=ny data=osmbase webMercator
  min=200.82ms (-12.7% vs 230.03ms) sum=2223.64ms (-10.68% vs 2489.41ms) repeats=10 throughput=4.4971/s (11.95% vs 4.017/s)
  avg=222.36ms (-10.68% vs 248.94ms) med=207.69ms (-15.53% vs 245.89ms) med95=279.40ms (-0.96% vs 282.11ms)
  gcTime=149.14ms (0.11% vs 148.98ms) sumNoGc=2074.50ms (-11.36% vs 2340.43ms) throughputNoGc=4.8204/s (12.82% vs 4.2727/s)

#performance OMVDecoderPerformanceTest-theme=berlin tiles=10 region=ny data=osmbase sphere
  min=379.38ms (-18.66% vs 466.43ms) sum=3978.09ms (-20.99% vs 5035.12ms) repeats=10 throughput=2.5138/s (26.57% vs 1.9861/s)
  avg=397.81ms (-20.99% vs 503.51ms) med=397.74ms (-21.99% vs 509.83ms) med95=413.61ms (-24.45% vs 547.43ms)
  gcTime=236.28ms (-1.43% vs 239.72ms) sumNoGc=3741.80ms (-21.97% vs 4795.40ms) throughputNoGc=2.6725/s (28.16% vs 2.0853/s)
``

Decoders tests please

Codecov Report

Merging #892 into master will increase coverage by 0.07%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master     #892      +/-   ##
==========================================
+ Coverage   44.52%   44.59%   +0.07%
==========================================
  Files         224      235      +11
  Lines       21522    21727     +205
  Branches     4904     4937      +33
==========================================
+ Hits         9582     9689     +107
- Misses      11940    12038      +98

Impacted Files
Coverage Δ

@here/harp-omv-datasource/lib/OmvDecoder.ts
25% <ø> (ø)
:arrow_up:

...e/harp-omv-datasource/lib/OmvDecodedTileEmitter.ts
40.33% <0%> (ø)
:arrow_up:

@here/harp-omv-datasource/lib/OmvUtils.ts
85.41% <100%> (+1.69%)
:arrow_up:

@here/harp-materials/lib/Utils.ts
20% <0%> (-80%)
:arrow_down:

@here/harp-materials/lib/IconMaterial.ts
44.44% <0%> (-55.56%)
:arrow_down:

@here/harp-map-controls/lib/Utils.ts
46.15% <0%> (-23.85%)
:arrow_down:

@here/harp-omv-datasource/lib/OmvTiler.ts
50% <0%> (-16.67%)
:arrow_down:

@here/harp-utils/lib/Logger/ConsoleChannel.ts
38.46% <0%> (-11.54%)
:arrow_down:

@here/harp-utils/lib/ObjectUtils.ts
83.33% <0%> (-10.22%)
:arrow_down:

@here/harp-utils/lib/UrlResolver.ts
91.66% <0%> (-8.34%)
:arrow_down:

... and 28 more

Continue to review full report at Codecov.

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


Enable features on docs.rs

I think this should work, see this. I wasn't sure about backends, so I left them out, but all-features = true might be nice.

Oh oops #165


Revert "Small helper to run remote commands"

Reverts tetrateio/tetrate-service-bridge-sandbox#278
The existing functionality affected...
❯ cat ./ssh-to-aws-170tse3-0-jumpbox.sh
ssh -i 170tse3-0-aws-tetrate-admin.pem -l tetrate-admin <IP_ADDRESS> "@"
❯ ./ssh-to-aws-170tse3-0-jumpbox.sh
bash: line 1: @: command not found

Oh 🤦♂
Instead of reverting, can you change the \"@\"" to \"$@\""? It was missing the $

sounds good


adding loan calculator

Helpfull tool working great!
but I'm facing some issue, extension work normally when its on live server but at extension time tool not showing result??

Helpfull tool working great!
but I'm facing some issue, extension work normally when its on live server but at extension time tool not showing result??

What's the issue? drop me a DM over discord, let me see

good job @kalashvasaniya :)

Thank you :)


Fix issue #15

Uncaught TypeError: Cannot read property 'getNearestTag' of undefined
This was caused by a race condition:
editor.onDidChangeCursorPosition fired before the parser had successfully parsed.
This fix will check @parser existence before trying to focus on the cursor tag.

@kvaellning Awesome! Thank you very much!!


Upstream Todoist issue: Funky scrolling behavior when using the keyboard to drag tasks (workaround in place)

When moving / indenting / dedenting a task on a project that is scrolled down, for some reason the window was being scrolled up a bit.  I've added a simple workaround for this that temporarily monkeypatches window.scrollBy, but creating this issue to track removing the workaround at some point in the future, and to report the issue.
This seems to be caused by this bit of code in Todoist (this is minified reformatted code from the dev tools):
Line 1181 of https://d3ptyyxy2at9ui.cloudfront.net/Col_ICECollection-678237.js:formatted
        _move: function(e) {
            var t = this.current_handler;
            if (!t)
                return !1;
            var n = this.current_root
              , r = t._kws
              , i = ICE.dnd.mouse_pos = ICE.$mousePos(e)
              , o = this.last_mouse_pos
              , a = i.x - o.x
              , s = i.y - o.y
              , l = ICE.$position(n);
            if (r.position_filter && (l = r.position_filter(l)),
            s += parseInt(n.style.top) || l.y,
            a += parseInt(n.style.left) || l.x,
            r.move_filter) {
                var u = r.move_filter(a, s);
                a = u[0],
                s = u[1]
            }
            if (!1 !== r.move_x && (r.custom_set_x ? r.custom_set_x(n, a) : ICE.$setLeft(n, a)),
            !1 !== r.move_y && (r.custom_set_y ? r.custom_set_y(n, s, i) : ICE.$setTop(n, s)),
            r.on_drag && r.on_drag(a, s, e),
            this.last_mouse_pos = i,
            !1 !== r.scroll_on_overflow) {
                var c = ICE.$winSize().h
                  , f = i.y - ICE.$scrollTop()
                  , d = null;
                f < 70 ? d = -6 : f >= c - 70 && (d = 6),
                this.scroll_interval && (clearTimeout(this.scroll_interval),
                this.scroll_interval = null),
                null != d && (this.scroll_interval = setInterval($p(function(e, t) {
                    r.scroll_custom ? r.scroll_custom(e, t) : window.scrollBy(0, t)
                }, i, d), 1))
            }
            return ICE.$preventDefault(e),
            !1
        },

setInterval(... window.scrollBy ..., 1) seems like a bit iffy, probably more reasonable if I dug into what $p was.

Curiously, this bug still repros.

This doesn't really matter anymore, as the ignored scroll mechanism got used other places too now.


Config prettier eslint

This configures prettier eslint to the project:

Adds eslint-config-prettier to make ESLint and Prettier play nice with each other
Adds empty prettier config file so that editors know we are using prettier
Adds git pre commit hook to run prettier
Formats all files with prettier

Alredy done as part of ID011


MacOS-M2 - issue

I have tried to install the RoughPy on macOS-M2, it did not work. It gave me the following error:   error: subprocess-exited-with-error
× Building wheel for roughpy (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [40 lines of output]
2023-10-06 14:43:44,066 - scikit_build_core - INFO - CMake version: 3.27.6
version {'regex': '^(?Pv)?(?P[^\+]+)(?P.*)?$'}
*** scikit-build-core 0.5.1 using CMake 3.27.6 (wheel)
2023-10-06 14:43:44,068 - scikit_build_core - INFO - Build directory: /private/var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/tmpgzt0tutf/build
*** Configuring CMake...
2023-10-06 14:43:44,170 - scikit_build_core - INFO - Ninja version: 1.11.1
2023-10-06 14:43:44,170 - scikit_build_core - WARNING - libdir/ldlibrary: /Users/mibrahim/anaconda3/lib/libpython3.10.a is not a real file!
2023-10-06 14:43:44,170 - scikit_build_core - WARNING - Can't find a Python library, got libdir=/Users/mibrahim/anaconda3/lib, ldlibrary=libpython3.10.a, multiarch=darwin, masd=None
loading initial cache file /var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/tmpgzt0tutf/build/CMakeInit.txt
-- Toolchain file:
-- Repository version v0.0.7
  -- The C compiler identification is AppleClang <IP_ADDRESS>30022
  -- The CXX compiler identification is AppleClang <IP_ADDRESS>30022
  -- Detecting C compiler ABI info
  -- Detecting C compiler ABI info - done
  -- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc - skipped
  -- Detecting C compile features
  -- Detecting C compile features - done
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ - skipped
  -- Detecting CXX compile features
  -- Detecting CXX compile features - done
  -- SKBuild environment: /Users/mibrahim/anaconda3/bin
  -- Found Python: /Users/mibrahim/anaconda3/bin/python (found suitable version "3.10.9", minimum required is "3.8") found components: Interpreter Development.Module NumPy
  '/Users/mibrahim/anaconda3/bin/python' '-m' 'pybind11' '--cmakedir'
  -- Adding Pybind11 cmake dir /private/var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/pip-build-env-oy2ytcbs/overlay/lib/python3.10/site-packages/pybind11/share/cmake/pybind11
  CMake Error at /private/var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/pip-build-env-oy2ytcbs/normal/lib/python3.10/site-packages/cmake/data/share/cmake-3.27/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
    Could NOT find Boost (missing: Boost_INCLUDE_DIR url) (Required is at least
    version "1.81")
  Call Stack (most recent call first):
    /private/var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/pip-build-env-oy2ytcbs/normal/lib/python3.10/site-packages/cmake/data/share/cmake-3.27/Modules/FindPackageHandleStandardArgs.cmake:600 (_FPHSA_FAILURE_MESSAGE)
    /private/var/folders/g6/dg1gwl0j4_33mp2w55441lm80000gn/T/pip-build-env-oy2ytcbs/normal/lib/python3.10/site-packages/cmake/data/share/cmake-3.27/Modules/FindBoost.cmake:2392 (find_package_handle_standard_args)
    CMakeLists.txt:160 (find_package)

  -- Configuring incomplete, errors occurred!

  *** CMake configuration failed
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for roughpy
Failed to build roughpy
ERROR: Could not build wheels for roughpy, which is required to install pyproject.toml-based projects

I've just made some changes to the build system so manually installing and adding the toolchain file is no longer necessary. With the next release, you will be able to just pip install even when there is no binary wheel available and it should just build.


POS BigQuery update is not reentrant

As a developer/operator I would like to sync the data release to multiple BigQuery datasets, i.e. platform_dev and production.
Background
As part of the data release, we publish our release data through a couple of BigQuery datasets: platform_dev and platform.
POS takes care of this process, but it uses a temporary CSV file for inserting the release metadata in the destination dataset, this file has a name that is not randomized, i.e. prone to collisions when multiple BigQuery synchronization scripts are being run simultaneously.
Tasks

[ ] Refactor this file into a proper temporary file mechanism, unique to any pipeline run for BigQuery data sync operations.

Acceptance tests
How do we know the task is complete?

When multiple datasets syncs can be run simultaneously

@jdhayhurst , this one is a small refactoring in the BigQuery scripts for POS


tv no response in apple home app

Philips 2016 model added sucssefully but somethimes is woking sometimes i got no response from the tv in apple home app...
most of the time is not responding...

This is also my problem: the plugin works but after some time it is not responding anymore...

Is there any news? Same problem


Collapse Layer::Preroll and Layer::Diff into a single pass

These 2 passes perform a lot of the same logic with some key differences that can be classified as potential bugs.
Diff works mostly in device clips/bounds to accumulate which pixels are modified.
Preroll works mostly in local clips/bounds to compute the bounds needed to pass to saveLayer and cache operations mainly.
This is all further complicated by the fact that the Paint pass will do layer culling on local bounds (which is why Preroll computes local bounds), but it will render the layer potentially with a pixel-snapped translation component.
A single pass that computes bounds might require fewer work-arounds, especially if it:

computed both local and device bounds at the same time
Paint performed culling on device bounds
The layer device bounds included pixel snapping

@knopp thoughts?


Allow custom date/time formats for date_time attributes

In order to allow custom formats for the date/time attributes, we need to know if we have to convert the timezone or to keep the user's time zone (for instance, a birth date shouldn't follow the time zone, but the date/time of a meeting should).
So, we need to add another type of Text Input Field, to know if the time zone should be converted or not.
To simplify its description, I split Text Input Field to Text Input Field with Date and Text Input Field with Date and Time (but I'm obviously open to comments about that :wink:).
Fix #6039

I like this :+1:


Elide cygrpc.Operations

Further progress toward issue 12531.

Please merge as soon as possible as this one too will probably cause another conflict for me to resolve. Thanks


Another argument for dedicated RSE education

In addition to the estimated required number of RSEs #23, another reason is the number of increasing vacancies for RSEs or people with related skills. E.g.: We could evaluate statistics of de-RSE mailing list history. Could we somehow extrapolate the demand for RSEs from this number as well?

hm, so you argue, that if there are x job openings per year on the ML, then we need to produce x amount of graduates?

It filtered all [de-RSE] emails for "job", "stellenausschreibung", "position", "vacancy" and removed any replies. I simply added up the number of the positions over time, starting with May, 2021. I applied a 2nd degree polynomial extrapolation to estimate a possible tendency (during the summer there always seems to be a dent).

I suspect that it is hardly possible to derive a specific number of required RSEs from that. But the trend of increasing vacancies could be a complementary reason, why RSEs need to be explicitly trained to meet future demand...

In order to argue that additional trained RSEs are needed, we should also know whether or not all vacancies can be filled within a certain period of time.
I am closing the topic for now, as this will not lead to any actionable findings...

Great work nonetheless!!


Add open leverage vault UI, fix some imports

Add open leverage vault UI
Changes 👷♀

- sidebar: switch to borrow, step1, step2, order information

sidebar confirmation

How to test 🧪

go to - http://localhost:3000/vaults/leverage/ETH-A?network=kovan and inspect changed UI

Definition of done ✔

[x] Acceptance criteria for each issue met
[ ] Unit tests written where needed and passing
[ ] End-to-end tests for happy path
[ ] Documentation updated
[x] Project builds without errors
[x] Non-functional requirements met
[x] Code reviewed and functionality tested by the reviewer (locally, Heroku, etc.)
[ ] Feature verified and accepted by Product Owner
[ ] Project deployed to production environment

Works like a charm!


Phone Status Cosmetic issue

The buttons overlap the notes field.    need to do some cosmetic re-shuffling.

Overlapping issue still remains. FF.


Assets support

I'm trying to include a simple image to a div tag on my twig template but I want to know if there is any workaround for the unsupported function asset.
I'm not using Express, so I'm rendering my templates with the following code:
Twig.renderFile('./path/to/someFile.twig', data, (err, html) => {
  html; // compiled string
});

Thanks in advance!

Which asset function are you referring to?

I'm referring to this. I want to define and access to an image asset, but I don't know how to implement it with twig.js, and also I look for your features support and I didn't find this asset functionality. Do you have any clue about it?

I understand; so just for make things clear for me, the only way I have to show a image on my template is referring it with the relative or absolute path, am I right?

Yes, you need to use the actual path.

Thanks for your help! I'll do it with that.

@cancinos to clarify in case you run into this again: Twig.js is a js implementation of the template language Twig (https://twig.symfony.com/doc/2.x/, https://github.com/twigphp/Twig) not of the framework Symfony (https://symfony.com/, https://github.com/symfony/symfony)

Thanks for the explanation. I'm using it in a RabbitMQ's mail send queue! I found it really useful for my various templates! @olets


Correctly updating pod label and node label when an update is received

Updating the label selectors in pods and nodes correctly when an update is received from the controllers.

/ok-to-test

This PR looks OK.
BTW, when I walk through podwatcher.go and nodewatcher.go, I have some feelings:

Poseidon should not exit when some error occurs, for example, https://github.com/kubernetes-sigs/poseidon/blob/master/pkg/k8sclient/podwatcher.go#L88 - glog.Fatal will calls os.Exit(255).

The 2nd argument of getFirmamentLabelSelectorFromNodeSelectorMap(nodeSelector NodeSelectors, nodeSelectorKeys []string) seems unnecessary.

func (pw *PodWatcher) getFirmamentLabelSelectorFromNodeSelectorMap(nodeSelector NodeSelectors, nodeSelectorKeys []string) []*firmament.LabelSelector {
	var firmamentLabelSelector []*firmament.LabelSelector
	for _, key := range nodeSelectorKeys {
		firmamentLabelSelector = append(firmamentLabelSelector, &firmament.LabelSelector{
			Type:   firmament.LabelSelector_IN_SET,
			Key:    key,
			Values: []string{nodeSelector[key]},
		})
	}
	return firmamentLabelSelector
}

// Maybe we can change it to

func (pw *PodWatcher) getFirmamentLabelSelectorFromNodeSelectorMap(nodeSelector NodeSelectors) []*firmament.LabelSelector {
	var firmamentLabelSelector []*firmament.LabelSelector
	for key, value := range nodeSelector {
		firmamentLabelSelector = append(firmamentLabelSelector, &firmament.LabelSelector{
			Type:   firmament.LabelSelector_IN_SET,
			Key:    key,
			Values: []string{value},
		})
	}
	return firmamentLabelSelector
}

WDTY?

@m1093782566 Thanks for the suggestion.
But the problem is NodeSelectors map cannot be sorted, hence we sort the key and keep it in a slice.
Then we iterate the sorted slice and read the value from the map NodeSelectors, this will give us the values in NodeSelectors map in sorted order ( based only on the key), which is passed to firmament.

/lgtm

@m1093782566 I have pushed one last change.
The E2E test case for 'nodeselector' was not working properly because it was scheduled by the default scheduler. The last commit sets the scheduler name correctly for the test pods. Now it will be schdled by 'Poseidon' correctly.

/lgtm
again...


Use builtin zlib instead of C interop

There are two reasons:

V already has builtin compress.zlib module
I caught some C interop errors

P.S. This is quickfix so code is kinda wacky

sorry didn't see until now, made invalid by #10

nckreader.com/samlock

http://nckreader.com/samlock/

https://gsmofficial.com/frp/


PayPal payout error: Cannot read properties of undefined (reading 'senderFees')

Reported in https://opencollective.slack.com/archives/C0429NTTABF/p1679304696758789 (private)
It looks like PayPal made yet another breaking change to their API:

In some cases, createPaymentResponse.defaultFundingPlan looks like it's not set.
Impacted expense: https://opencollective.com/siduction-eu/expenses/129072. The PayPal response looks like:
{
  "createPaymentResponse": {
    "responseEnvelope": {
      "timestamp": "2023-03-20T02:23:22.144-07:00",
      "ack": "Success",
      "correlationId": "[OBFUSCATED]",
      "build": "[OBFUSCATED]"
    },
    "payKey": "[OBFUSCATED]",
    "paymentExecStatus": "CREATED",
    "httpStatusCode": 200,
    "paymentApprovalUrl": "https://www.paypal.com/cgi-bin/webscr?cmd=_ap-payment&paykey=[OBFUSCATED]"
  },
  "executePaymentResponse": {
    "responseEnvelope": {
      "timestamp": "2023-03-20T02:23:27.815-07:00",
      "ack": "Success",
      "correlationId": "[OBFUSCATED]",
      "build": "[OBFUSCATED]"
    },
    "paymentExecStatus": "COMPLETED",
    "httpStatusCode": 200
  }
}

Looks like we may have to fetch the details from PayPal's API. Note that it doesn't seem to be affecting all expenses, as the admin noted:

ok - i have paid two in the meantime (that went smoothly though): https://opencollective.com/collective-y/expenses/129191, https://opencollective.com/collective-y/expenses/129193

I can now confirm that a payment processor fee was charged on this transaction, PayPal didn't notice us about it in the payload. We'll need to fetch the information from their API.


Undefined offset in 6.6.17

[10-Jun-2020 15:21:29 UTC] PHP Notice:  Undefined offset: 1 in /app/public/wp-content/plugins/static-html-output-plugin/src/DetectArchiveURLs.php on line 66
[10-Jun-2020 15:21:29 UTC] PHP Stack trace:
[10-Jun-2020 15:21:29 UTC] PHP   1. {main}() /app/public/wp-admin/admin-ajax.php:0
[10-Jun-2020 15:21:29 UTC] PHP   2. do_action() /app/public/wp-admin/admin-ajax.php:175
[10-Jun-2020 15:21:29 UTC] PHP   3. WP_Hook->do_action() /app/public/wp-includes/plugin.php:478
[10-Jun-2020 15:21:29 UTC] PHP   4. WP_Hook->apply_filters() /app/public/wp-includes/class-wp-hook.php:311
[10-Jun-2020 15:21:29 UTC] PHP   5. static_html_output_ajax() /app/public/wp-includes/class-wp-hook.php:287
[10-Jun-2020 15:21:29 UTC] PHP   6. StaticHTMLOutput\Controller->generate_filelist_preview() /app/public/wp-content/plugins/static-html-output-plugin/static-html-output-plugin.php:77
[10-Jun-2020 15:21:29 UTC] PHP   7. StaticHTMLOutput\FilesHelper::buildInitialFileList() /app/public/wp-content/plugins/static-html-output-plugin/src/Controller.php:204
[10-Jun-2020 15:21:29 UTC] PHP   8. StaticHTMLOutput\FilesHelper::getDateArchiveURLs() /app/public/wp-content/plugins/static-html-output-plugin/src/FilesHelper.php:514
[10-Jun-2020 15:21:29 UTC] PHP   9. StaticHTMLOutput\DetectArchiveURLs::detect() /app/public/wp-content/plugins/static-html-output-plugin/src/FilesHelper.php:548

Thanks @crstauf, some changes happening in this PR https://github.com/WP2Static/static-html-output-plugin/pull/87 / infinite-crawl branch that should address this. I'm partially through it and will try remember to do some WP_CLI tests on it, that's how this one slipped through

@crstauf thanks again for reporting. The lack of detection wasn't just me not running via CLI, but not enough actual WP variant testing - this was likely only thrown when no particular archives existed, ie, no posts by any dates
static-html-output-plugin-90-undefined-offset.zip

@leonstafford Tested, and I don't get the undefined offset error anymore, but...
I do get this in the Console:
Failed to load resource: the server responded with a status of 502 ():
https://website.local/wp-content/uploads/WP-STATIC-PROGRESS.txt?cacheBuster=1591903891143

@crstauf cool, let's ignore that one for now, if it's not blocking the export. I'm removing all the flat files in the infinite-crawling branch.
curious that it's a 502, though...


Incomplete dispose causing background activity

After initializing and calling an BannerAd(), some ad activity keeps running in background even after .dipose().
Calling .dispose() in the page where it is displayed, in main.dart dispose and even at Ad Listener, it keeps occurring. Due to that, app is penalized and cannot run other required activities while in background.
On IOS, this message is thrown a lot after app is in background:
[ProcessSuspension] 0x282514380 - WKProcessAssertionBackgroundTaskManager: Ignored request to start a new background task because RunningBoard has already started the expiration timer
Example in main.dart AND any random page
@override
  void dispose() {
    AdBanner.dispose();
    super.dispose();
}

Example code of the listener
AdManagerBannerAdListener listener = AdManagerBannerAdListener(
    onAdLoaded: (Ad ad) {
      print('Ad loaded.');
    },
    onAdFailedToLoad: (Ad ad, LoadAdError error) {
      ad.dispose();
      print('Ad failed to load: $error');
    },
    onAdOpened: (Ad ad) {
      print('Ad opened.');
    },
    onAdClosed: (Ad ad) {
      ad.dispose();
      print('Ad closed.');
    },
    onAdImpression: (Ad ad) {
      print('Ad impression.');
    },
    onAppEvent: (ad, name, data) {
      print('App event : ${ad.adUnitId}, $name, $data.');
    },
  );

Plugin Version
0.13.5

Hi @maheshmnj, the code is very similar and also with the example that you provided and follows the docs description.
Sorry, I forgot to mention: that error messages only are displayed using a real device, in the simulator indeed it does not occur.
I found that just by replacing child: AdWidget(ad: bannerAd)); to child: Container); everywhere in the app, the error messages goes away.
Not sure if is related, but by doing this also causes eliminates errors messages of what seems to be http filter, even with the implementation described in https://developers.google.com/admob/ios/app-transport-security. If digging into this comes properly, please let me know and I can share the exact messages.
Here is the output of flutter doctor -v:
`[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.2 20G95 darwin-arm, locale
en-US)
• Flutter version 2.2.3 at /Users/user/develop/flutter
• Framework revision f4abaa0735 (4 months ago), 2021-07-01 12:46:11 -0700
• Engine revision 241c87ad80
• Dart version 2.13.4
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at /Users/user/Library/Android/sdk
• Platform android-30, build-tools 30.0.3
• Java binary at:
/Library/Java/JavaVirtualMachines/jdk-16.0.1.jdk/Contents/Home/bin/java
• Java version Java(TM) SE Runtime Environment (build 16.0.1+9-24)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.5.1, Build version 12E507
• CocoaPods version 1.10.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[!] Android Studio (version 2020.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
✗ Unable to find bundled Java version.
• Try updating or re-installing Android Studio.
[✓] Connected device (2 available)
• iPhone Test (mobile) •
ios            • iOS 14.7.1
• Chrome (web)                 • chrome                                   •
web-javascript • Google Chrome 94.0.4606.81
! Doctor found issues in 1 category.
`
Thanks in advanced


Contribute paimon reader to add caching function

Enhancement
Hi,  I want to contribute the implementation code of paimon reader caching function, how can I do it?

@Larix1989 Thank you, all contributions are welcome. Please contact me via email (284487410@qq.com). I will provide you with some necessary information and the current R&D progress of Paimon.

ok! @miomiocat

Is there any progress?

Is there any progress?

Since version 3.2.3, data cache with the native reader has been supported. The topic of discussion in this issue is about supporting data cache with the JNI reader.


fix(#4369): Fix children renderProps for react-aria-components

Closes #4369
✅ Pull Request Checklist:

[x] Included link to corresponding React Spectrum GitHub Issue.
[x] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

I went ahead and replaced the state changing render prop methods that you called out with static properties.

GET_BUILD

Merging so all the non-collection components get into the nightlies. We'll need to address the collection based RAC's with renderProps before next release.
Thanks for the work and tests!

Using the nightly here, Once I render ListBox inside of a children renderProp for Select I get this error:
Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.

@knownasilya Sounds like the issue mentioned here: https://github.com/adobe/react-spectrum/pull/4385#discussion_r1169158125 that we are tracking here. Mind linking a codesandbox to so we can confirm the setup here?

I believe that fixed did also resolve my issue


Remove an extra print from the relay astext tests

cc @tqchen

Seems the error was due to a flaky test in test_nll ( too small rtol and atol), @mbrookhart do you mind doing a quick fix?

https://github.com/apache/tvm/pull/8344


Clarify purpose of duplicate precipitation inputs

The CSV input files include both APCP_surface and precip_rate. It would seem these are redundant. The former would be expected to be an accumulated precipitation amount and the latter a rate, and, if the are derived from the same source, should be easily transformed one to the other. But... that doesn't seem to be the case -- there is no obvious linear transformation relating the two values. (We recognize that kg/m2/s is equivalent to mm/s... it isn't a strict unit conversion issue we're seeing here, at least it doesn't appear to be...)
What is the meaning of each variable and where are the values used?
Doing some digging, it appears that precip_rate is used in the LSTM model and APCP_surface may be used in the CFE-based simulations.
Possibly useful line references:
https://github.com/NOAA-OWP/ngen/blob/f914038ff33ab26fcf6172cf78f6bf9481706a02/include/forcing/AorcForcing.hpp#L19
https://github.com/NOAA-OWP/ngen/blob/f914038ff33ab26fcf6172cf78f6bf9481706a02/src/models/lstm/LSTM.cpp#L170
https://github.com/NOAA-OWP/ngen/blob/f914038ff33ab26fcf6172cf78f6bf9481706a02/models/tshirt/include/tshirt_c.h#L53

The example LSTM models were trained with NLDAS hourly forcings: "precipitation hourly total (kg/m^2)", more information here: https://ldas.gsfc.nasa.gov/nldas/v2/forcing. However, there may have been some trickery to get one of those two variables to play nice. Looks like @madMatchstick and Scott have made some adjustments since (https://github.com/NOAA-OWP/lstm/blob/d8100a9d4f651d65e81e2e9902f9a3728e9d1049/lstm/bmi_lstm.py#L102). It would be good to re-train a model with the AORC forcing data, so it is consistent.

To clear the air on this once more, I'll explain the derivation of precip_rate from APCP_surface.
APCP_surface is the AORC variable in units of kg/m2.  This is often ASSUMED to be ~= mm/hr, but that is an assumption that depends ultimately on the density of water.
the precip_rate variable was used in some of the early model testing as an actual rate, in units of m/s valid until the next entry (hour, in this case) in the time series.
The exact transformation was done via these two functions.
rho approximate density of water at temperature
This function does not consider pressure.
def rho(temp):
    """
        Calculate water density at temperature
    """
    return 999.99399 + 0.04216485*temp - 0.007097451*(temp**2) + 0.00003509571*(temp**3) - 9.9037785E-8*(temp**4)

Cumulative to rate transform transformation
This function computes the RATE at which precipitation occurs, effectively converting kg/m^2 to m/s by using the AORC interval of 1 hour between inputs (the shift) in seconds, and rho.
def aorc_as_rate(dataFrame):
    """
        Convert kg/m^2 -> m/s
    """
    if isinstance(dataFrame.index, pd.MultiIndex):
        interval = pd.Series(dataFrame.index.get_level_values(0))
    else:
        interval = pd.Series(dataFrame.index)
    interval = ( interval.shift(-1) - interval ) / np.timedelta64(1, 's')
    interval.index = dataFrame.index
    precip_rate = ( dataFrame['APCP_surface'].shift(-1) / dataFrame['TMP_2maboveground'].apply(rho) ) / interval
    precip_rate.name = 'precip_rate'
    return precip_rate

It is left to the user to decide which of these inputs is appropriate for a given model formulation, or whether or not the assumption that kg/m^2 == mm/hr is a reasonable one for their application.

As for the value mapping between the two being unclear, if the above explanation was a little more TL;DR than you were looking for, I'll add this briefer but less completely accurate explanation:
precip_rate[t-1] = ( ( APCP_surface[t] / 3600 ) / 1000 ) * density_factor_very_near_one
where:

3600 = seconds in one hour
1000 = millimeters in a meter
t-1 means the previous timestep because the accumulation in APCP_surface is measured at the end of the hour, so the precip_rate is the rate of precip that was happening in the previous hour.

That last bit may depend on the interpretation of precip_rate: is it the rate happening in that timestep hour, or is is the rate that led to the accumulation of precip we measured at this instantaneous hour marker?
In either case, I'm guessing that the bump from one timestep to the next was what was causing @jameshalgren to be completely lost as to the relationship between the two columns.

@jmframe @mattw-nws
Good points regarding the duplication -- my original question was about the input files. We can log a todo out of this issue to either clean up the input files or clarify the documentation or both.

density_factor_very_near_one
Using the equation above to compute the density factor, the temperature is in degrees Celcius (so the Kelvin value minus 273.15)

is it the rate happening in that timestep hour, or is is the rate that led to the accumulation of precip we measured at this instantaneous hour marker?
The same question about average temperature and instantaneous temperature applies -- which value is applicable to perform the density calculation? (though the answer is nearly inconsequential as the density difference from one hour to the next, even in cases of extraordinary temperature gradients, is less than a percent -- 0 degrees C to 40 degrees C yields 0.8% density change...)

Here's my spreadsheet, built from cat-52_2015-12-01 00_00_00_2015-12-30 23_00_00 to see if I understood:

Thanks for the input.

Self closing. We might make the issue mentioned above:

[ ] Create issue regarding simplification of in-repo input files to retain only one value of precipitation, with in-line conversion capability ported to a BMI or other function.

Using the equation above to compute the density factor, the temperature must be in degrees Celsius...

Yep, this is where https://github.com/NOAA-OWP/ngen/pull/389 came from! 😆

@joshcu
Also probably related to:
https://github.com/NOAA-OWP/cfe/issues/29
https://github.com/NOAA-OWP/ngen/issues/302
and should probably update this
https://github.com/NOAA-OWP/ngen/issues/302


Update screenshots and demo video for v1.3.0

Screenshots
You can use https://developer.mozilla.org/en-US/docs/Web/API/Window/resizeTo to precisely resize the window.
Chrome
Five 1280x800
It needs to be at least 5 windows with at least 50 tabs in total.

[ ] Chrome popup mode
[ ] Standalone mode
[ ] Dark theme
[ ] Shortcut mode, by click Cmd+, in the extension view
[ ] Search demo

Edge
Ten 1280x720
Done
Videos
Videos should be recorded by section, i.e. a small feature like Open popup window should be a clip. And each clip should have a message to indicate what's it, for example, a black screen with text: "How to open the extension"
Basic usage

[ ] Open popup window (click + shortcut)
[ ] Navigate and select tab(s), up, down, left, and right (h, j, k, l), x (ctrl+x in search box) to select
[ ] Focus/unfocus search box via both click and shortcut
[ ] Select all/matched tabs, and unselect them (ctrl+8 to select all, ctrl+m to select matched, ctrl+n to unselect)
[ ] Delete focused tab or selected tab (ctrl+d)

How to use drag and drop

[ ] Singe tab drag within a window
[ ] Single tab drag across windows
[ ] Multiple tabs drag across windows
[ ] Select matched tabs and drag them together
[ ] Select matched tabs and drag them to a new window

Sort and Group tabs

[ ] Sort tabs in a window
[ ] Group and sort all tabs
[ ] Sort tabs in a window via shortcut
[ ] Group and Sort tabs via shortcut

Shortcuts

[ ] Open the popup window in the standalone tab (Ctrl+t)
[ ] Show shortcut modal

Once the video is on YouTube, you need to add a comment to point each section with time: https://m.wikihow.com/Link-to-a-Certain-Time-in-a-YouTube-Video's-Comment-Box

You can use this to visualize shortcut https://github.com/keycastr/keycastr

We should use the puppeteer to automation the screenshot and even demo video.


Dashed rectangles weird behavior

When zoming out dashed rectangles appear while they should not.
For instance here is a dashed rectangle :

But when zooming in it's empty:

Thanks for reporting this issue!

Resolved in v0.95
It would be nice If you can test it with your repro-case and confirm it works now.

Works fine ! Thank you


feat(backend): list endpoints

Changes proposed in this pull request

list endpoints for incoming payments, outgoing payments, and quotes
abstract some functionality to avoid repetition in list endpoints

Context

fixes #289

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[ ] Documentation added
[x] Make sure that all checks pass

Tests fail because ilp-pay expects non-required fields to be undefined and not null, but undefined is not valid JSON.


Having validationError set without 'required' shows error text all the time

Might want to only set errorText on the TextField if this.showError is true.

It also disables the submit button.

Odd, I didn't think formsy was supposed to return an error on a "pristine" form element (one that hasn't been edited).

Looks like #19 / 0.2.4 fixed this. Daniel, can you confirm?

Works!


Improve Credit Card UI

Is your feature request related to a problem? Please describe.
I want to improve the already implemented Credit Card UI in a more better way
Describe the solution you'd like
Improvement of Credit Card UI
Kindly assign this to me
And If possible add Level 2 for this. Since I would be completely improving it from scratch. It will come under New Script Addition but since its already implemented, It would be called as improvement. Hence, kindly assign it to me please. Would request not to reject it

@Ayush7614 Please assign me this issue


Platformer

@JVeg199X is there anything else you'd want to get in before I merge this for a QA build?


Webhook 404 for Github pull requests

Hey guys, we're actually just in to process of migrating from GitLab to GitHub (and using Drone). For some reason the webhooks in Github seem to be failing for pull_requests, everything else seems to work fine!
Heres the requests and responses from the Github UI.
Request URL: https://drone.jumplead.com/hook?access_token=**SNIP**
Request method: POST
content-type: application/x-www-form-urlencoded
Expect:
User-Agent: GitHub-Hookshot/375c44e
X-GitHub-Delivery: **SNIP**
X-GitHub-Event: pull_request

Access-Control-Allow-Origin: *
Cache-Control: no-cache, no-store, max-age=0, must-revalidate, value
Connection: keep-alive
Content-Length: 0
Content-Type: text/plain; charset=utf-8
Date: Fri, 20 May 2016 14:25:08 GMT
Expires: Thu, 01 Jan 1970 00:00:00 GMT
Last-Modified: Fri, 20 May 2016 14:25:08 GMT
Server: nginx
X-Content-Type-Options: nosniff
X-Drone-Version: 0.4.1+373
X-Frame-Options: DENY
X-Xss-Protection: 1; mode=block

Also drone logs:
time="2016-05-20T14:25:08Z" level=error msg="failure to get build config for Jumplead/Worker. GET https://api.github.com/repos/Jumplead/Worker/contents/.drone.yml?ref=7d16e8a5ce7f6b968e72fb9f002991a54fc972a3: 404 Not Found []"
time="2016-05-20T14:25:08Z" level=error msg="Error #01: GET https://api.github.com/repos/Jumplead/Worker/contents/.drone.yml?ref=7d16e8a5ce7f6b968e72fb9f002991a54fc972a3: 404 Not Found []\n" ip=<IP_ADDRESS> latency=391.891183ms method=POST path="/hook" status=404 time="2016-05-20T14:25:08Z" user-agent="GitHub-Hookshot/375c44e"

This is a private repo btw. Looks to be something wrong with the reference? Possibly similar issue to #1463?

does your pull request include a valid .drone.yml file? It is getting a 404 when trying to fetch the pull request. There are no known issues with fetching the .drone.yml for pull requests, aside from when using amendments for which we have an open issue.
given my inability to repeat and no other installations reporting issues this is something you're going to need to dig into further. You should be able to look at the pull request hook sent from GitHub and:

what sha is drone using from the hook?
why does that sha return a 404 when fetching the .drone.yml?
did the user that registered the repository de-authorize drone, preventing it from being able to authorize the request to the fetch the yaml?

also note that, per the issue template, issues should be discussed in https://gitter.im/drone/drone prior to logging issues.

@bradrydzewski That would well be it! To be honest that didnt even cross my mind, I'll add a Drone file to my PR and see if that works!

That was it @bradrydzewski, apologies about that. I'll put it down to being a friday. Have a great weekend!

no worries, glad that fixed it :)


Fixed typo in feature request template

Fixed typo in feature request template

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
While reading feature request template I noticed this typo.

Description

I signed it!


Device: Improves readability of disk device validation

Implements suggestion from https://github.com/lxc/lxd/pull/8161#discussion_r524643693

Testsuite passed

Testsuite passed


There's no way to make a controlled input in a node with vanilla React

Describe the Bug
The react flow docs show an example of a node with an input in it. This only works because the input is uncontrolled using defaultValue. https://reactflow.dev/docs/examples/nodes/custom-node/
If you make the input controlled, every time you type in it, the cursor jumps to the end of the input. This is because the way the nodes are updated uses the async setState callback:
setNodes((nds) => ...

(If you update a controlled input's value async, the cursor jumps, it has to be set sync)
In the react flow example, the async state set to be done because the onChange handler is passed in node data in the example. Any data the onchange handler closes over in the parent component will be stale. Trying to update the onchange handler with new closure data on every keypress sounds confusing.
Mortiz made a controlled input with zustand, which works because the setter lives outside node data: https://discord.com/channels/771389069270712320/971342201701412915/983467015312396320
However there's no clear way to do this without zustand or a state management library.
This is half bug, half feature request. It would be great to have a first class way to handle controlled inputs in flow nodes.
One reason I want controlled inputs in nodes is to have a number input field along with a slider in the node, so that one changes the other. So I can't use defaultValue because both need to update in tandem
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Try to make a controlled input on a node
Expected behavior
There to be a first class vanilla react way to do this
Screenshots or Videos
No response
Platform

OS: MacOS
Browser: Chrome
Version: 10.2.1

Additional context
No response

I think this should work, but it has the same cursor jumping issue: https://codesandbox.io/s/react-flow-renderer-cursor-jump-mc60w2?file=/src/App.js
To reproduce on this code sandbox:

focus the text input in the node, and move the cursor to the beginning
type in the input
the cursor jumps to the end of the input every time you type

The codesandbox implements this workaround I'm trying, where the onchange handler is passed down via context. https://codesandbox.io/s/context-onchange-controlled-input-ojg8bd This codesandbox doesn't have the cursor jumping issue, so I don't think it's an issue with passing the onChange in context
does rfr update nodes async when calling setNodes?

This is not an issue with React Flow. We are not doing node updates asyncally. The issue with your code is, that the valSetter is not memoized. If it's not memoized, the valSetter / onChanges updates the custom node before the actual input changes arrive.
Wrapping it with a useCallback fixes the issue: https://codesandbox.io/s/react-flow-renderer-cursor-jump-forked-ni12h4?file=/src/App.js:903-952

Thank you very much for creating that demo. I was under the impression that the setState callback was an async operation, that impression looks to be incorrect.
This is strange, I'm doing basically the same thing in my real app, but I get the cursor jumping behavior. It's in a much more complicated setup so I wonder if some other async operation is at play here. Would not memoizing other callbacks, like onConnect, cause this behavior if they changed every render in the parent?
Either way, I'll close this for now, thanks again for taking the time to give me updated code.

Sure! This is also super helpful for us to understand issues with the usage of the library better and might create more docs about these topics.

Interestingly the only other callback i've found so far that causes the cursor to jump is that I was passing in an unmemoized onNodeDoubleClick.
Is the best practice with rfr to memoize all callbacks fn props?

yep! I recently added this information to the docs https://reactflow.dev/docs/api/react-flow-props/#event-handlers

Just a heads up for anyone else with the same issue: this can also happen if you have local state in your custom node that you update alongside the node's data field.
I forked @moklick's code sandbox with the useCallback fix, updated it to use React Flow v11.7.4, and added a local numEdits state variable in the custom node that gets incremented every time the color input is edited, and the cursor position reset behavior happens again:
https://codesandbox.io/s/react-flow-renderer-cursor-jump-forked-hx6snf?file=/src/ColorSelectNode.js
I assume this is happening for the same reason as the not-memoized callbacks example above - i.e. updates to the node data do not propagate immediately the next render but the local state update does.

@moklick hey) could you please take a look at the comment and one above?
I have somewhat similar issue. I use controlled nodes and edges. To control them there is a Context Provider that stores nodes and edges. There is also a custom node that makes use of the Context.
When I change data inside nodes array, the custom node first rerenders with obsolete data object.
I forked your example and made some changes to illustrate the issue: https://codesandbox.io/p/sandbox/react-flow-renderer-cursor-jump-forked-p9vqhk?file=%2Fsrc%2FApp.js%3A49%2C1
After clicking on the "reset color" button, in the console we can see:
nodes, data [{…}] {color: '#1A192B'}
nodes, data [{…}] {color: '#CCCCCC'}
nodes, data [{…}] {color: '#CCCCCC'}

The first line shows that, on the first rerender, nodes array already have updated data. Whereas custom's node data object has the old color.
P. S. I assume, I can use ReactFlowInstance to get nodes and update them inside custom nodes. However, current behaviour still seems to have a flaw. And normally, if I have some a state that data originates from, shouldn't I update exactly that one?
Considering, the comment above and this one, @moklick could you please take a look and tell if this is a bug or we doing something wrong?
Thanks!


Search Field: clear field

Related Issues:
#953 - Search Field: disabled state & fits container
#952 - Search Field: autosearch & minLength change

Issue Description
New features to add to the search field component:

Add programmatic method to clear the search field.
Need to expand hit-area of the clear icon/button target (add top/bottom padding to fill to top/bottom of input, add left/right padding roughly even to top/bottom, might need to be themeable variable).
The clear icon/button needs to be tab focusable.

(10/20/2017) Update: generated two new issues to make PR smaller, see Related Issues
Issue Type

[ ] New Feature
[X] Enhancement
[ ] Bug
[ ] Other

Expected Behavior

items listed above

Current Behavior

user click to clear only

Adding @Singleton06 as a watcher to issue.

@neilpfeiffer could we add an item here so that we have a programmatic way to set the content of the search field as well?  We are looking at re-writing a dialog that launches with the previously entered search already inside of it when it launches.

Add programmatic method to clear search field.
could we add an item here so that we have a programmatic way to set the content of the search field as well

These can be achieved by exposing the value of the input through a value prop similar to the text field. This would make the input a controlled input.

@neilpfeiffer @Singleton06 Could we break this issue up into several smaller issues? Theres a lot here, too much for just one PR.

Sounds good - I will divide it into 3 issues:
(1) Disabled, (2) Clear actions/visuals, (3) autosearch + minimum characters.

For the clear button having a larger hit area and being tab-able, this appears to be something that the browser is adding as can be seen by this example[1] where all I am doing is using a HTML element with the type of "search".  This line[2] seems to be setting that property.
Given that we have a specific need for the programmatic way of clearing the search field (as well as sitting the initial field), could we move that enhancement to a separate issue?
There might be a solution to the clear button tabbing and size, but I'm not that well versed with CSS and would prefer to not let that delay the progress on the other parts.
[1] - https://codepen.io/anon/pen/OxKMEQ?editors=1111
[2] - https://github.com/cerner/terra-core/blob/master/packages/terra-search-field/src/SearchField.jsx#L99

Tech Design for Search Field controlled component support
Summary:
Upgrade the search field control to be able to be a controlled component. This will allow for programmatic clearing and
setting of the search field.
Requirements:
Discussion has been handled here.  Basically, this is being done to
allow for a programmatic way of clearing the search field.
Requirements Added:
When the consumer specifies the value prop, the control now becomes a controlled component.
Responsiveness:
No Change should occur to the responsiveness.
Bidirectionality
No change should occur to the bidirectionality.
Accessibility
No change should occur to the bidirectionality.
React Props:
|| Prop || Type || Default || Description ||
| value | String | '' | The value of the search field.  Use this to create a controlled search field. |
| onChange | func | undefined | Function to trigger when user changes the search value.  The text of the search changed will be passed as an argument to the function provided.  Provide a function to create a controlled search field.
Example:
{code}
<SearchField
value={this.state.searchValue}
onChange={(value) => this.setState({ searchValue: value })}
/>
{code}

@mjhenkes or @emilyrohrbough  the one thing that I'm not quite sure how to handle with making this change is the onChange and 'handleTextChange' functionality interaction with each other.  Should it be that the onChange happens first?  Would it be expected that the onChange is able to reject a change in the property and then the onSearch or onInvalidSearch is called based on the result of the onChange somehow?

I would suspect this would be what you'd want something like this in the handleTextChange function:
  handleTextChange(event) {
    const searchText = event.target.value;
    this.setState({ searchText });

 // onChange
    if(this.props.onChange) {
           this.props.onChange(event, searchText);
     }

    if (!this.searchTimeout) {
      this.searchTimeout = setTimeout(this.handleSearch, this.props.searchDelay);
    }
  }

Does this answer your questions?

Thanks for the code @emilyrohrbough, I'll have to test it out.  I'm not quite sure if that approach would allow the consumer to reject certain characters, etc, giving them control over what the value truly is.  For example, if they wanted only alphabetical characters, etc, I'm wondering if the search would still be performed in this approach.

@Singleton06 would you more expect it to not set the state of the field?  Therefore, the consumer would be giving the onChange as an override, but still perform the search
{code}
handleTextChange(event) {
if (this.props.onChange) {
this.props.onChange(event, event.target.value);
} else {
this.setState({ searchText: event.target.value });
}
if (!this.searchTimeout) {
  this.searchTimeout = setTimeout(this.handleSearch, this.props.searchDelay);
}

}
{code}

JIRA created

Sorry for the late response, @kschuste, I think what you have is more what I was thinking.


refactor: promise executor functions should not be async

Description
Promise executor functions should not be async    (Hacktoberfest)

Could you also link the sonarcloud issue that the PR aims to solve

you always need to make sure you do not duplicate work of others, there was a pr for cli.js already https://github.com/asyncapi/generator/pull/1278#pullrequestreview-2354930328


Integration using CocoaPods without use_frameworks!

New Issue Checklist

[X] I have read and understood the CONTRIBUTING guide
[X] I have read the Documentation
[X] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
7.0

SDWebImage Version
4.0.0-beta2

Integration Method
cocoapods 1.0.1

Xcode Version
Xcode 8

Repro rate
all the time (100%)

Repro with our demo prj
no

Demo project link
-

Issue Description and Steps
I'm trying to integrate SDWebImage using CocoaPods without use_frameworks! but failed at
#if COCOAPODS
@import FLAnimatedImage;
#else
#import "FLAnimatedImageView.h"
#endif

I searched the issue list and found #1579 . The issue was closed. However the error remains. Am I missing something here?
I came up with #1696 which should fix this problem.

I am using cocoaPods 1.1.0 r2.  I integrated it into my project successfully. But not sure why you remove "use_frameworks!"

@gujinku Well, as a full Objective-C project, it's not necessary to use SDWebImage with "use_frameworks!".

I agree @skyline75489. We need to support both frameworks and static libs integrations. I will look into this issue soon.

When can SDWebImage support static libs?

a similar issue. I'm trying to integrate SDWebImage using CocoaPods without use_frameworks! but failed at:
ld: framework not found SDWebImage for architecture arm64

Platform Version SDWebImage 3.8.2
IOS dep target 8.1
CocoaPods 1.1.1

Fixed by #1696


Initialization of dbs starting several times

The initialization of the data module happens during each module instantiation leading to inefficiencies.

Datas build function runs on init of each module rather than during bot init

Corrected with version 1.1.0


Connection dialog - issue with scrollbar for the connection tree

Azure Data Studio Version:
at least starting from 1.0.0

Steps to Reproduce:
first make sure you have a few connections in both recent connections and saved connections

Open ADS and reduce its height so that the connection list will need to scroll
Open connection dialog and you will see the saved connection list can be scrolled
Now maximize the window

Expected:
all the connections can be displayed without scrollbar.
Actual:
the connection list doesn't expand as the window size changed and the scroll bar is still there.

This appears to be fixed in current builds.  Please reactive if the issue is still impacting you.


Update profiling funcs to new interface

This hasn't been updated in a while. Now it's up-to-date.

LGTM!


Installing devise_token_auth on MacOS, rails conflict

I'm attempting to install devise_token_auth. The instructions say to add gem 'devise_token_auth' to my Gemfile, then run bundle install. I created a brand new API only Rails app, follow the instructions, and get the following:
Bundler could not find compatible versions for gem "rails":
  In Gemfile:
    rails (~> 6.1.1) x86_64-darwin-20

    devise_token_auth x86_64-darwin-20 was resolved to 0.1.21.alpha2, which depends on
      rails (~> 4.1.4)

What is x86_64-darwin-20? That's not in the Gemfile, but I'm working on MacOS. What's going on?

Figured it out. devise_token_auth 1.1.4 depends on rails >= 4.2.0, < 6.1. So in the Gemfile, I changed the Rails gem to rails < 6.1. Then run bundle update, bundle install

Figured it out. devise_token_auth 1.1.4 depends on rails >= 4.2.0, < 6.1. So in the Gemfile, I changed the Rails gem to rails < 6.1. Then run bundle update, bundle install

Actually, I ended up just creating a new rails project with a specific version number compatible to devise_token_auth

Actually, I ended up just creating a new rails project with a specific version number compatible to devise_token_auth


Feature/prompt user to paste code properly 35

Sorry for the delay, I was busy, then sick, then busy again but I have some holidays over the next few days so I'll try to get to it then - I'll have a look at the mypy issue as well.

Sorry for the delay, I was busy, then sick, then busy again but I have some holidays over the next few days so I'll try to get to it then - I'll have a look at the mypy issue as well.

No worries, I totally forgot about this! I'll fix the mypy issues this weekend!


update dependencies

If you're interested, you can sign up for scala-steward here :)
https://github.com/VirtusLab/scala-steward-repos

Yes, I know 🙂 - I think I need some time to pass from my last daytime gig where it was more noise and hassle than help.


Ignore CONSUMING state in externalview while reporting segment status

It is possible for Idealstate to be in ONLINE and externalview to be in CONSUMING
state (for realtime LLC segments) for a short period of time. The segment status
checker runs every 5 minutes, and for tables that produce segments fairly frequently
it is possible that at any given time, some segment is in this combination of states.
Ignore the CONSUMING state in externalview so that we report alerts correctly.
If a segment is in this combination of states for too long, the alerts from the
server should fire, regarding non-consuming kafka partitions

Codecov Report

Merging #1967 into master will increase coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1967      +/-   ##
==========================================
+ Coverage   69.44%   69.48%   +0.04%
==========================================
  Files         828      828
  Lines       40513    40513
  Branches     5434     5434
==========================================
+ Hits        28133    28150      +17
+ Misses      10501    10487      -14
+ Partials     1879     1876       -3

Impacted Files
Coverage Δ

...n/pinot/controller/helix/SegmentStatusChecker.java
84.24% <0%> (-0.69%)
:arrow_down:

...ore/query/scheduler/resources/ResourceManager.java
81.81% <0%> (-13.64%)
:arrow_down:

.../pinot/core/query/scheduler/PriorityScheduler.java
77.46% <0%> (-5.64%)
:arrow_down:

...impl/dictionary/DoubleOnHeapMutableDictionary.java
71.11% <0%> (-4.45%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
83.63% <0%> (-3.64%)
:arrow_down:

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
69.09% <0%> (-3.64%)
:arrow_down:

...ot/core/query/pruner/ColumnValueSegmentPruner.java
84.48% <0%> (-3.45%)
:arrow_down:

...inot/core/operator/docidsets/AndBlockDocIdSet.java
55.17% <0%> (-2.07%)
:arrow_down:

.../com/linkedin/pinot/common/data/TimeFieldSpec.java
97.36% <0%> (-1.32%)
:arrow_down:

...not/transport/scattergather/ScatterGatherImpl.java
57.43% <0%> (-0.68%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


Checkout on document level prevents commenting

When user A checked out the document, user B will get a read-only view only.
This prevents adding comments (since they are part of the document, oxygen uses PIs)
On solution would be to put comments into the ~meta DB of each project and add them the next time the document is being saved

Another question in this context would be how to allow element level checkouts. This would allow multiple users to work on the same document.


Add TLX as submodule

Since the RadixHeap is now merged into TLX master branch, NetworKit does not require the code copies introduced in commit dcf5121. This PR hence adds TLX as a submodule and removes the code copies.

I'm restarting the build so that we can merge this PR soon.

I'm not sure if that is going to work though, as Travis does not seem to re-read the configuration. I may have to close/reopen this PR.

There was a small conflict, so let's wait for CI again...

This can be merged. Thanks for the code!


libwebp 1.0.1

Is there any reason for using current old lib? According to Changelog there were few juicy changes.
I can't say if there are really no problems but it was compilled and started succefully with libwebp #e85d331 just by removing
'<(DEPTH)/third_party/libwebp/src/enc/delta_palettization_enc.c',
'<(DEPTH)/third_party/libwebp/src/dsp/enc_avx2.c',

No reason I can come up with, a PR is very welcome! :)


Marathon template fails if the path contains a directory in it.

If you say in a config file something like this:
[marathon]
filename = "/etc/traefik/marathon.tmpl"

It fails. Because "text/template" API is used incorrectly.
You pass a full path as a name of the template here:
https://github.com/emilevauge/traefik/blob/master/provider/marathon.go#L187
And you then ParseFiles with the same path:
https://github.com/emilevauge/traefik/blob/master/provider/marathon.go#L189
It fails, because this is exactly the case ParseFiles documentation warns about. Quoting the docs:

ParseFiles parses the named files and associates the resulting templates with t. If an error occurs, parsing stops and the returned template is nil; otherwise it is t. There must be at least one file. Since the templates created by ParseFiles are named by the base names of the argument files, t should usually have the name of one of the (base) names of the files. If it does not, depending on t's contents before calling ParseFiles, t.Execute may fail. In that case use t.ExecuteTemplate to execute a valid template.

Affects other provides as well. Suggested fix, create a template with filepath.Base(provider.Filename) as a name.

There is a PR that is fixing that

#108 it is :-)


Add tests/reports to .gitignore

Ignore these files when committing. These are test reports which are
generated when running tests locally or on CI.

/approve
/lgtm


Add deprecation message to readme

Should the repo get archived?

@hanzei yes, I don't think I have the privs

https://git.internal.mattermost.com/ops/github-management/-/merge_requests/118


[BEAM-10009] Add support for logical types and beam:logical_type:millis_instant:v1 in Python schemas

This PR adds support for logical types in Python schemas, and adds the first logical type implementation there for beam:logical_type:millis_instant:v1.
I chose apache_beam.utils.timestamp.Timestamp as the Python type mapping for millis_instant . Note that Timestamp is represented as microseconds since the epoch, so this is a lossy mapping, but there is precedent for this in Python's WindowedValueCoder: https://github.com/apache/beam/blob/699f872ea1ef3bdb1588a029fc6b1e3185e986a6/sdks/python/apache_beam/coders/coder_impl.py#L1236-L1240
Also in this PR:

some changes to the Java SDK implementation of Beam schemas to make its FieldType.DATETIME type behave exactly like beam:logical_type:millis_instant:v1 when used in portabillity (afa939e). DATETIME will be deprecated and removed in favor of a Java implementation of millis_instant in the future (BEAM-7554).
Unit testing in schemas_test and row_coder_test.
Tests using Timestamp in the Python SqlTransform.
Test cases in standard_coders_test.yaml (the first logical type tested there).

Note the Python implementation of LogicalTypes in this PR is limited to those that do not use an argument.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run XVR_Direct PostCommit

@robertwb, @robinyqiu - In this PR I've mapped python's Timestamp (microsecond precision) to the millis_instant logical type lossily so it could be mapped to DATETIME on the Java side and used in SQL.
Based on discussion in https://github.com/apache/beam/pull/12493, now I'm thinking the better answer would be to add a micros_instant logical type so Python's data can go across the xlang boundary faithfully, and it's up to SQL to throw an error when the timestamp precision is too high, or silently drop precision if the user chooses.
There's an open question about what to do if the output of the SqlTransform includes a millis_instant logical type. Would that still get mapped to a python Timestamp?

Closing this in favor of #12764 which adds a micros_instant logical type to more faithfully represent Timestamp


window; status bar correctness

[x] ensure status bar correctness
[x] iOS
[x] Android

Status bar behaviour is correct on both, on android it has some resize considerations which are under another already listed issue.


request

Hello, may I ask if you can provide the dataset used in this article?

Hi. I'm sorry, but unfortunately this dataset isn't available.


Psalm: check if directory exists before include them at

When running the tool on CI, some ignoredDirs could not exists yet, and the psalm throw a error if a ignored directory not exists Could not resolve config path to example/dist. It's possible check if the directory exists before generate the psalm-phpqa.xml?

Related problem 118 . Good catch @elvishp2006

It seems to me as tool-specific settings. Phpqa should not care about too much internal details of every analyzer.

You define analyzed and ignored directories/files
phpqa transforms them to correct format
it's not phpqa responsibility, what happens, if directory does not exist, is not readable etc.

I also think it could cause confusing behavior, if build configuration is magical and not explicit. Build could pass with skipped directories, but is that correct? If directory structure is changed and build config is not changed, then build should fail.


Annotate uncovered statements with warnings

Add warnings for code changes in pull requests that dont have test coverage.

Can you elaborate on what kind of change you are suggesting? Currently, if Go files were changed, this action will already show the impact on the unit test coverage of the repository.

I think it can be done in the way codecov does and golangci lint

Thanks for the suggestion, now I see what you mean.
To be honest, the scope of this GitHub action is to provide a high level summary of the changes with a single comment. What you are suggesting sounds interesting but is far beyond that scope. If you would like to have more elaborate features around code coverage, you are probably better of using codecov or another dedicated service for code coverage :sweat_smile:


Disable -Werror for now

CONFIG_WERROR is enabled by default when CONFIG_COMPILE_TEST is enabled.
Disable it explicitly so we can catch actual build breakage.
I am of the opinion we should prioritize getting warning clean so that
we can re-enable this, as this does not affect our boot tests anymore
but will make it obvious when there are potential issues due to
warnings.

Thanks for the review!


remove "application android:label" from Manifest of library

@IsseiAoki Thanks, but I don't get the new version. Seems that's not online...
> Could not find com.isseiaoki:simplecropview:1.0.14.
     Searched in the following locations:
         https://repo1.maven.org/maven2/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.pom
         https://repo1.maven.org/maven2/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.jar
         http://jcenter.bintray.com/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.pom
         http://jcenter.bintray.com/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.jar
         https://jitpack.io/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.pom
         https://jitpack.io/com/isseiaoki/simplecropview/1.0.14/simplecropview-1.0.14.jar

@Rainer-Lang So sorry!! I published the new version right now.

@IsseiAoki Thank you very much. All works now. :+1: :-)


Implement reagent

https://github.com/reagent-project/reagent

https://user-images.githubusercontent.com/51877647/130731794-0582e5b6-0cbb-41a4-8f29-04a8f12e79d7.mp4

code
(ns main.core
  (:require [reagent.core :as r]
            [reagent.dom :as rdom]))

(defn get-fraction-of-day []
  (let [date (js/Date.)
        time-of-day (+ (* (.getHours date) 60 60 1000)
                       (* (.getMinutes date) 60 1000)
                       (* (.getSeconds date) 1000)
                       (.getMilliseconds date))]
    (/ time-of-day 86400000)))

(defn decimal-clock []
  (let [fraction-of-day (r/atom (get-fraction-of-day))]
    (r/create-class
      {:display-name "decimal-clock"

       :component-did-mount
       (fn []
         (letfn [(animate []
                   (do
                     (swap! fraction-of-day #(get-fraction-of-day))
                     (.requestAnimationFrame js/window animate)))]
           (.requestAnimationFrame js/window animate)))

       :reagent-render
       (fn []
         [:div#decimal-clock
          [:h2 "Decimal Clock"]
          [:div#clock-face
           (for [x (range 10)]
             [:div.digit
              {:style {:left (str (- 50 (* (Math/sin (* Math/PI (/ x 5))) 40)) "%")
                       :top (str (+ 50 (* (Math/cos (* Math/PI (/ x 5))) 40)) "%")}
               :key x}
              x])
           (for [x (range 100)]
             [:div {:className (if (mod x 10) "large tick" "tick")
                    :style {:transform (str "rotate(" (* x 3.6) "deg)")}
                    :key x}])
           [:div#short-hand {:style
                             {:transform (str "rotate(" (+ (* @fraction-of-day 360) 180) "deg)")}}]
           [:div#long-hand {:style
                            {:transform (str "rotate(" (+ (* (mod (* @fraction-of-day 10) 1) 360) 180) "deg)")}}]

           [:div#second-hand {:style
                              {:transform (str "rotate(" (+ (* (mod (* @fraction-of-day 1000) 1) 360) 180) "deg)")}}]]])})))

(defn render []
  (rdom/render [decimal-clock] (js/document.getElementById "app")))

(defn ^:export run []
  (render))

;; re-render after reload
(defn ^:dev/after-load start []
  (render))

I mean, it works, but the effort needed for the setup is just too much for this simple thing lmao, closing this

lol

just saw this issue. sad.

Reopening this issue. probably someone who are looking for hacktoberfest point would want to continue the work.


Fix one more place in CFEP 5 where the prerelease indicator changes position

This follows up on https://github.com/conda-forge/cfep/pull/21 to fix one more place that needs to change.

CC @conda-forge/core


⚠ ESPN has degraded performance

In e55e014, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1430 ms

Resolved: ESPN performance has improved in eaca85c.


Default API invocation fail with the Opaque token

Deployment migrated from 2.6 -> 3.2. So there are applications that generate opaque tokens.
APIs with the version in the URL can invoke without any issue using opaque token but once version removed from URL (default API) it failed with below error message.
The same behavior can reproduce after extracting jti from the JWT token as well.

Fixed with https://github.com/wso2/carbon-apimgt/pull/9403