Treasury: Expenses

(i.e. Salaries paid to core contributors, liquidity mining programs, etc.)

Closed because duplicates Issue #25:
https://github.com/josephd8/index-coop-analytics/issues/25


Can't resolve DOC_ORIENTATION

DOC_ORIENTATION cannot be accessed because it is not exported by the library, and results in an error such as:
Module not found: Error: Can't resolve 'ngx-image-compress/lib/image-compress' in '...'

Nice! I will update this on npm soon


[Question]: After install Fabric: command not found

What is your question?
After reading the documentation, I am still not clear how to get fabric working, I'm on linux.
I installed go and can run it to get the latest version with go version. go version go1.23.1 linux/amd64
Added /usr/local/go/bin to the path
Installed fabric with  "go install github.com/danielmiessler/fabric@latest"
I see a file called fabric in the /go/bin/ directory but when I run "fabric --setup" I get "fabric: command not found"

Seems ~/go/bin is not in your path. Did you follow the next section Environment Variables ?
# Golang environment variables
export GOROOT=/usr/local/go
export GOPATH=$HOME/go

# Update PATH to include GOPATH and GOROOT binaries
export PATH=$GOPATH/bin:$GOROOT/bin:$HOME/.local/bin:$PATH

Add this to ~/.bashrc and reload your session or source ~/.bashrc .

`# Golang environment variables
export GOROOT=/usr/local/go
export GOPATH=$HOME/go
Update PATH to include GOPATH and GOROOT binaries
export PATH=$GOPATH/bin:$GOROOT/bin:$HOME/.local/bin:$PATH`
Thanks after following this I can now run fabric --setup
Im new to linux so I have no clue what I was supposed to do with the ~/.bashrc but my issue seems to have been resolved.
Cheers


i18n: Improvements in various translation-related areas.

Various fixes related to i18n and user-facing strings.
Regarding 9528cf5, I haven't been able if the strings were being misinterpolated before the fix. However, Transifex didn't recognize them and I thought it could be of use anyways from a code formatting perspective (to make group.name look like stream_name references). Feel free to drop it if you feel like it's not necessary.
About URLs in translation tags (see this for some context), I considered adding a linter check for  tags containing anything similar to a path. However, I think it would be a source of many fake positives (e.g. Do this/that looks like a path too), so it may be just worth keeping in mind for future reviews.
Testing Plan:
Full test suite passed in my Travis CI instance.

Merged, thanks @YagoGG!  I did a strings update to Transifex after merging this.


Metal Support

Tracking the remaining work on the Metal backend here...
Examples:

[x] ClearScreen
[x] ClearScreenMultiWindow
[x] BasicTriangle
[x] BasicVertexBuffer
[x] CullMode
[x] BasicStencil
[x] InstancedIndexed
[x] TexturedQuad
[x] TexturedAnimatedQuad
[ ] BasicCompute
[ ] ComputeUniforms
[ ] ToneMapping

Functions:

[ ] SetGpuBufferName
[ ] SetTextureName
[ ] SetStringMarker
[x] MapTransferBuffer
[x] UnmapTransferBuffer
[ ] GetTransferData
[ ] CopyBufferToBuffer
[ ] GenerateMipmaps
[ ] DownloadFromTexture
[ ] DownloadFromBuffer
[x] BindVertexSamplers
[ ] BindVertexStorageTextures
[ ] BindVertexStorageBuffers
[x] BindFragmentSamplers
[ ] BindFragmentStorageTextures
[ ] BindFragmentStorageBuffers
[ ] DrawPrimitivesIndirect
[x] PushVertexUniformData
[x] PushFragmentUniformData
[ ] PushComputeUniformData
[x] Blit
[ ] CreateComputePipeline
[ ] ReleaseComputePipeline
[ ] BeginComputePass
[ ] BindComputePipeline
[ ] BindComputeStorageTextures
[ ] BindComputeStorageBuffers
[ ] DispatchCompute
[ ] EndComputePass
[ ] QueryFence
[ ] SupportsSwapchainComposition
[x] SetSwapchainParameters
[ ] CreateOcclusionQuery
[ ] OcclusionQueryBegin
[ ] OcclusionQueryEnd
[ ] OcclusionQueryPixelCount
[ ] ReleaseOcclusionQuery

We're doing future commits in the gpu branch directly, moved the checklist back over to the issue.


add napari as an optional dependency

We should consider adding napari as an optional dependency so that people can install everything with one command.
Suggested by @rdrighetto in zulip: https://imagesc.zulipchat.com/#narrow/stream/427861-Biozentrum-Napari-MemBrain-Hackathon-2024/topic/surforama/near/426113294

Should we mirror the napari API? That is, one could use pip install surforama[all] to install surforama + napari + PyQt5?

Yes, that's what I had in mind :-)

I added a pip install surforama[napari] option in #25


Retrieve the string within the lambda expression

I'd like a simple way to retrieve the string within the lambda

class stl : StringSpec() {

    init {

        "triangle.stl" {
             ...
        }
    }
}

In this case "triangle.stl"

May I ask why do want the test case name to be passed as a parameter? I felt never the need in all the years of programming.

Because it is also the name of the file I am trying to read

But is the file name in your case actually a useful test case name? There is of corse not the one true style to write tests, but at least I prefer having a descriptive name like "should parse invalid csv file". Why don't you use a name like this and the file name as a usual variable inside the test? An alternative could be to use table driven testing, where you'd get the file names as input parameters.
I want to understand your use case to come to be able to asses your feature request.

Yes, I am testing several meshes, that's the name of the i-th file
This is what I came up with so far, I am open to any suggestions

As far as I understand your code, you could replace the test cases with table driven tests, since the test logic seems to be always the same but with different values. Table driven testing would be more elegant and would reduce the amount of code significantly in you case.

I suggest to close this issue, since I consider it generally not useful to get the test case name passed as parameter. What do you think @sksamuel ?

I have no problems closing it, however I think the table driven tests will not fit my case, or at least for the moment.
The main problem is that I have only specific fields to check about, and they are hardly the same between a mesh and another one

You could create a table where some values are null or you could use some kind of container objects (aka data classes) inside the table.

I would need a ternary status, sometimes I have to check against null values..
However, if it's just me, we can close without problems ;)


Dependency<EMAIL_ADDRESS>restricts engines to node 6 and npm 3

The dependency<EMAIL_ADDRESS>restricts engines to node 6 and npm 3
npm WARN EBADENGINE Unsupported engine {
npm WARN EBADENGINE   package<EMAIL_ADDRESS>npm WARN EBADENGINE   required: { node: '^6.9.5', npm: '^3.10.10' },
npm WARN EBADENGINE   current: { node: 'v18.12.1', npm: '8.19.2' }
npm WARN EBADENGINE }

This constrain is removed in<EMAIL_ADDRESS>but tiny.pictures-js still references the old version with ^0.1.2.
"engines": {
  "node": "^6.9.5",
  "npm": "^3.10.10"
 }

This causes a EBADENGINE error in more up-to-date setups.
As a peer dependency npm wont automatically update that package on module resolution unless explicitly listed in our own package.json with a newer version.
Could you bump the dependency version in question?

Thanks for raising this issue. Commit d0ed8bdeebdafa9cb27b479709052372d3dbea64 fixes this issue. Released on npm as v4.2.7.


how to speed up "File Tail" ？

I have a log named access.log size 3.8G
I create a simple pipeline filetail-trash
not rate limited . but I found Record Throughput only less then 20 records/s
pipeline with default config
how to optimize ？

What version of SDC? Can you export the pipeline and post it here? You should get thousands of records/sec!

7 seconds to ingest 9890 records on my laptop:

<IP_ADDRESS>
f056e6c0-40bd-4cdc-bb4a-8df2a53576c2.txt
not support .json file ,so i rename to .txt ,you can download and rerename it
yes ,yesterday ,I use a script write lines to a file with rate 10000 lines/sec, streamsets file tail can reach that rate , so I wonder the reason is the file size too big , every batch rewrite the offset, and next batch need to re seek from top  to that offset ?
I need your help ,@metadaddy

also I want know when  FileTail  trigger to collect log file ?
if i have a file ,but no new line appended , FileTail will not be triggered ?

I looked at your pipeline - I don't see anything that would slow it down.
The file tail reader will only seek at the beginning of each batch, so it shouldn't impact performance that much. You could test this by changing the batch size. Note - you will need to edit sdc.properties to increase batch size beyond 1000 - see https://streamsets.com/documentation/datacollector/latest/help/#Troubleshooting/Troubleshooting_title.html#concept_ay2_w1l_2s
File tail will read all of the existing data, then wait for new data, so it should work for you. A better choice, if the file will not be changing, might be the directory origin.


Clear table is really slow

Clear button is taking quite some time. It might help to drop tables / database instead.

Hey! I wanna work on this! Could you explain to me the database structure for starters, I'll move forward with that.

@udiboy1209 , I'm not sure if anything much can be done here. All we do here is run the clear table query in Lovefield. The bug was originally filed when we didn't have an option to clear a single table at a time. The primary function involved is https://github.com/jgraham/wptview/blob/master/LovefieldService.js#L466
Maybe you could try running queries individually rather than in a transaction and profile the code in both cases, though I doubt it will really help. @jgraham , do you see any good solution here?


Update prune-prereleases.js pruning rules

Pruning rules:

only keep the earliest release of the month
to keep the newest 3 nightlies

Fixes #6732
Here is my local testing:
import { Octokit, App } from "octokit";

// Create a personal access token at https://github.com/settings/tokens/new?scopes=repo
const octokit = new Octokit({ });

// In case node 21 is not used.
function groupBy(array, keyOrIterator) {
    var iterator;

    // use the function passed in, or create one
    if(typeof keyOrIterator !== 'function') {
        const key = String(keyOrIterator);
        iterator = function (item) { return item[key]; };
    } else {
        iterator = keyOrIterator;
    }

    return array.reduce(function (memo, item) {
        const key = iterator(item);
        memo[key] = memo[key] || [];
        memo[key].push(item);
        return memo;
    }, {});
}

async function separateReleases({ github, context }) {
    console.log("Pruning old prereleases");

    // doc: https://docs.github.com/en/rest/releases/releases
    const { data: releases } = await github.rest.repos.listReleases({
        owner: context.repo.owner,
        repo: context.repo.repo,
    });

    let nightlies = releases.filter(
        release =>
        // Only consider releases tagged `nightly-${SHA}` for deletion
        release.tag_name.includes("nightly") &&
        release.tag_name !== "nightly"
    );

    // group releases by months
    const groups = groupBy(nightlies, i => i.created_at.slice(0, 7));

    // Pruning rules:
    //   1. only keep the earliest release of the month
    //   2. to keep the newest 3 nightlies
    const toPrune = Object.values(groups)
        .reduce((acc, cur) => acc.concat(cur.slice(0, -1)), [])
        .slice(3);

    const toKeep = Object.values(groups).reduce((acc, cur) => acc.concat(cur.slice(-1)), []);

    return {
        toPrune,
        toKeep,
    };
};

(async() => {
    const releases = await separateReleases({
        github : octokit,
        context : {
            repo : { owner: "foundry-rs", repo: "foundry" }
        },
    });
    console.log("To prune:", releases.toPrune.map(i => i.name));
    console.log("To keep:", releases.toKeep.map(i => i.name));
})();

$ node index.mjs
Pruning old prereleases
To prune: [ 'Nightly (2023-11-01)' ]
To keep: [
  'Nightly (2024-01-12)',
  'Nightly (2023-12-02)',
  'Nightly (2023-11-02)',
  'Nightly (2023-10-02)',
  'Nightly (2023-08-02)',
  'Nightly (2023-07-02)',
  'Nightly (2023-06-02)',
  'Nightly (2023-05-02)',
  'Nightly (2023-04-02)',
  'Nightly (2023-03-02)',
  'Nightly (2023-01-03)'
]

Motivation

Solution

Is the test result not incorrect? If it is supposed to keep the first release of the month it should be pruning 2023-11-02, not 2023-11-01

Is the test result not incorrect? If it is supposed to keep the first release of the month it should be pruning 2023-11-02, not 2023-11-01

I think there could be some mismatch between the "name" and created_time or published_time field, perhaps due to timezones.
By default the github releases seem to be sorted by created_time. That should make it consistent. To be precise, the comment could be improved to clarify this.

Let's include the clarification and this overall LGTM

Let's include the clarification and this overall LGTM

Yep. I find a quote from the doc:
| The latest release is the most recent non-prerelease, non-draft release, sorted by the created_at attribute.

@hellwolf Thanks for doing this!


Error when executing ./run_synthea in the command line (on MAC OS X 10.13.5)

After having downloaded synthea, when I want to download the data using "./run_synthea" in the command line, I get the following error (I am using Mac OS X version 10.13.5):
Exception in thread "main" javax.net.ssl.SSLException: java.lang.RuntimeException: Unexpected error: java.security.InvalidAlgorithmParameterException: the trustAnchors parameter must be non-empty
---Next lines of the error message:
Exception in thread "main" javax.net.ssl.SSLException: java.lang.RuntimeException: Unexpected error: java.security.InvalidAlgorithmParameterException: the trustAnchors parameter must be non-empty
	at com.sun.net.ssl.internal.ssl.Alerts.getSSLException(Alerts.java:190)
	at com.sun.net.ssl.internal.ssl.SSLSocketImpl.fatal(SSLSocketImpl.java:1747)
	at com.sun.net.ssl.internal.ssl.SSLSocketImpl.fatal(SSLSocketImpl.java:1708)
	at com.sun.net.ssl.internal.ssl.SSLSocketImpl.handleException(SSLSocketImpl.java:1691)
	at com.sun.net.ssl.internal.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1222)
	at com.sun.net.ssl.internal.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1199)
	at sun.net.www.protocol.https.HttpsClient.afterConnect(HttpsClient.java:476)
	at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(AbstractDelegateHttpsURLConnection.java:166)
	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1195)
	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:234)
	at org.gradle.wrapper.Download.downloadInternal(Download.java:66)
	at org.gradle.wrapper.Download.download(Download.java:51)
	at org.gradle.wrapper.Install$1.call(Install.java:62)
	at org.gradle.wrapper.Install$1.call(Install.java:48)
	at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:69)
	at org.gradle.wrapper.Install.createDist(Install.java:48)
	at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:107)
	at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
Caused by: java.lang.RuntimeException:

From what I can tell from various sources online, this may be an issue with your JDK certificate store. Which version of java are you running? Run java --version at a command line.
Unfortunately there are a lot of different suggestions on how to resolve this but they are not consistent across operating systems, ex: https://stackoverflow.com/questions/6784463/error-trustanchors-parameter-must-be-non-empty
The simplest fix is likely just to re-install Java, I recommend either JDK 8 or 10. http://www.oracle.com/technetwork/java/javase/downloads/jdk10-downloads-4416644.html

Thanks, downloading JDK 10 solved my problem!


Evaluation rules order of evaluation undefined

It is impossible to use a recorded value in another recorded rule, because the evaluation order is unpredictable. Sometimes, you get the value from the current evaluation, and sometimes you get the result of the previous evaluation. This leads to oddities like dividing this minute's sum1m by last minute's count1m, giving wrong values and the occasional +Inf.
Suggested fix: add a command-line flag to enforce sequential evaluation of rules in the same order as their appearance in the rules file.
This is related to issue 17, but that is about adopting some sophisticated automatic sorting, whereas this bug is about the lack of any basic consistency. I would be unhappy if this wasn't going to be fixed until issue 17 was ready.

The agreed solution is to be able to have groups of rules, and that they will be ordered within the group. This still allows use of multiple cores for calculations.
I suspect that #17 will never happen.

Should we create a new issue that describes the preferred approach, and close this and #17 ?
@juliusv What do you think?

:+1:
On Thu, Sep 17, 2015 at 6:42 PM Björn Rabenstein<EMAIL_ADDRESS>wrote:

Should we create a new issue that describes the preferred approach, and
close this and #17 https://github.com/prometheus/prometheus/issues/17 ?
@juliusv https://github.com/juliusv What do you think?
—
Reply to this email directly or view it on GitHub
https://github.com/prometheus/prometheus/issues/1088#issuecomment-141143998
.

:+1: Will do.

Superseded by https://github.com/prometheus/prometheus/issues/1095. Closing.


Make sure it can be easily be integrated with other projects

check that it is easy to use other data sources, that we don't rely on the feeder gateway

Hey @SantiagoPittella the feeder gateway will be deprecated within this year. Any plans to remove the reliance on the feed gateway rather than growing reliance on it? This is not only for integrating with other projects but also this repository itself, because you are not going to be able to use it in Nov or Dec this year anyway

Yes, we are ready to switch to full RPC when the nodes support the features that we use from the feeder gateway. We are looking forward to it, since it also will improve the performance of the explorer.


[Bug]: Error when create a new world

/mv version -p output
Multiverse-Core Version: 4.3.1-b861
Server logs
https://gist.github.com/Valenn1898/1d0535aac91c9cd148f370535ff11ba6
Server Version
arclight-1.18.2-1.0.6-7e754e7e (MC: 1.18.2)

Bug Description
when i try to create a world  it is failed
Steps to reproduce
1- Start server with multiversecore instaled
2- put command /mv create (worldname) normal
3- failed
Agreements

[X] I have searched for and ensured there isn't already an open or resolved issue(s) regarding this.
[ ] I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.

same problem !

Hello
I have the same problem
when i go to /mv list i see my world but is unload and is imposible to load
they say faild to load world
how to fix this

MV doesnt support arclight. Please use spigot or paper.


device request: Santton Rf_2

In my Sonoff integration, and in my Ewe-link app I have a bridge, which I use for controlling shutters.
This device used to work in HA, but stopped working. Still works in EWE link app, HA says it is no longer compatible.
Device info
SANTTON_RF-2
by SANTTONG
"This entity is no longer being provided by the sonoff integration. If the entity is no longer in use, delete it in settings."
Is it possible to be compatible once again?

Show device diagnosctics


Cannot connect to PlanetScale MySQL instance.

Hi team,
I can not connect to PlanetScale MySQL instance. DBeaver is working well.
Can you help to double check it?
Host: gcp-us-central1.connect.psdb.cloud
DB: planetscale-free-db
User: hv84ram6usxf9kab97nd
Password: pscale_pw_BchvgmMshdOJSBvLeJRRZ7PMAVlaLDH7wUKEGN5MYt6

Hi @chaomao-help,
I am able to connect to the above database.
But the object explorer is not working because current code is not expecting hyphen(-) in the object names. Will add this as bug.
Can you confirm if you are not able to connect or you are facing errors after making successful connection?

Hi @rishky-msft,
It works after remove hyphens.
Thank you very much.


Use Ruby 3.0.0 if you have rbenv installed

Please, go through these steps before you submit a PR.

[x] You've explained what the Pull Request is adding.
[x] You've explained why this is important.

What the Pull Request is adding
This PR adds a default file so that those who use rbenv will automatically use Ruby version 3.
Why this Pull Request is important
Different people install Ruby in different ways. I personally choose to use rbenv and I suggest that those who haven't yet set up their environment also use rbenv.

Side note: If you've already got an environment working that works for you, don't feel like you need to switch to using rbenv!


changes pt3

1. change to "Best Email Marketing Tools for 2022"

2. delete this line and combine lists

3. delete this header and combine this list with "best marketing tools"

4.  remove
We recommend to you pay more attention to these points
5. move my articles "Arthur Tkachenko articles" under Miscellaneous
6. formatting issue

7. rename to "open source tools" and add links
https://www.mautic.org/
https://www.sympa.org/
https://www.openemm.org/doku.php
https://sendportal.io/
https://lightmeter.io/

8. rename to DNS DKIM Records

9. add sub-sections to Transactional Emails

basics
advanced
tools

10. move to Resources section

10. check the Transactional section for duplicates

create a section "Similar awesome lists" in the end of our Readme file, change titles and move those links into this section

Готово.

add sub-sections to Transactional Emails

basics
advanced
tools

цього не робила, бо такі вже є (оновила "Table of Contents", то зможете побачити)

rename to DNS DKIM Records
image

тут теж такий заголок вже був, тому тільки об'єднала

thank you!


TensorFlow Lite Converter wraps unpack operator with dequantize/quantize

System information

Linux Ubuntu 20.04
TensorFlow installed from: pip source
TensorFlow versions: 2.12.0-current master

Code
Provide code to help us reproduce your issues using one of the following options:
import tensorflow as tf
from tensorflow.keras.layers import Layer, Input
from tensorflow.keras.models import Model
import numpy as np

class UnstackLayer(Layer):
    def __init__(self, **kwargs):
        super(UnstackLayer, self).__init__(**kwargs)

    def call(self, inputs):
        unstacked = tf.unstack(inputs, axis=1)
        # only last output is used as input to the add operator
        return unstacked[-1]

input_tensor = Input(shape=(4, 16, 32))
x = UnstackLayer()(input_tensor)
output_tensor = tf.add(x, 1)

model = Model(inputs=input_tensor, outputs=output_tensor)

converter = tf.lite.TFLiteConverter.from_keras_model(model)
converter.optimizations = [tf.lite.Optimize.DEFAULT]

def representative_data_gen():
  for input_value in [np.random.randn(1, 4, 16, 32).astype(np.float32) for _ in range(10)]:
    yield [input_value]
converter.representative_dataset = representative_data_gen

converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
converter.target_spec.supported_types = {tf.int8}

converter.inference_input_type = tf.int8
converter.inference_output_type = tf.int8

tflite_model = converter.convert()

# Save the TFLite model to a .tflite file
with open('model.tflite', 'wb') as f:
    f.write(tflite_model)

Failure after conversion
Input Model:

Output Model:

Behaviour in TF 2.11 and below is that no dequantize/quantize ops appear, which is expected.
Other info
The conversion started failing with version TF 2.12.0. Note that the conversion succeeds intermittently when converting the same network many times, but on average it fails. This intermittent behaviour is still present if one runs the converter on a single core and keeps the representative dataset constant. Similar issues seems to be present when converting split operators as well.

@willisacs-arm Did you try with the latest TF version 2.13 and let us know the outcome?
Have a look at this reference for more information on quantization. Thank you!

Hi yes the behaviour is the same for all versions above 2.12

accidentally closed issue

Hi @willisacs-arm
I did observe that it does not add Quantize/Dequantize stubs for intermittent runs.
@pkgoogle Could you please look into this issue?
Please find the reproducible gist in TF 2.13.
Thanks.

Hi @willisacs-arm, so currently the quantization is unable to quantize the unpack operator, so it actually processes that op in the original bit size. Would you like a feature request to make it quantizable? Are you working with a system that requires 8-bit operations Only?

Hi, just to reiterate, the quantization sometimes works already if one just tries it enough times, so this doesn't seem to require a new feature, unless the fact that the quantization sometimes works is the bug here. It also works in patches before 2.12.0.
Yes I am working with a system that requires int 16 or int 8 activations.

Also if I understand it correctly the documentation says that the converter should throw an error if an operation cannot be quantized?
https://www.tensorflow.org/lite/performance/post_training_quantization#integer_only

Hi @willisacs-arm,  yeah that documentation is incorrect, we have not finalized on intended behavior so we have not updated it to prevent further confusion. That being said, "the quantization sometimes works already if one just tries it enough times" doesn't sound good, is there any way you can provide us information on when it starts working? like... What is your model where this happens (especially which ops are included), and after how many times does it start working? If you can show us the converted state after each time, that'll also be great. Thanks!

Hi @willisacs-arm
I did observe that it does not add Quantize/Dequantize stubs for intermittent runs.
@pkgoogle Could you please look into this issue?
Please find the reproducible gist in TF 2.13.
Thanks.

Ah alright, thanks.
The model is the one that was provided in the first comment in this issue, it has an unpack/unstack operator. It has also been provided neatly by pjpratik in the gist quoted in this reply. To reproduce the issue, run the notebook/gist and then repeat the last 2 steps until a quantized operator without dequantize/quantize ops appear in the Netron window.
There seems to just be a random chance every time one runs the converter to get a functioning quantized unstack operator. A speculation would be that there's a memory overwrite somewhere. The behaviour is present even when running the converter on a single core and while keeping the representative dataset constant. The model file to convert was also kept constant.

I was able to reproduce the somewhat random behavior after 7 attempts with the attached gist. @abattery, can you please take a look? Thanks.

did this has solution? i got this promblem too


How does it work at CIFAR100？

Hi, have you tried the same for CIFAR100. I'm very curious about the result achieved by MAE on the CIFAR100.

@ZhiyangLiang ,I am glad you asked this question. Actually, the solution is very simple. You just need to make some modifications in the "mae_pretrain.py" and "train_classifier.py" files. Specifically, you need to change lines 35 and 36 to use the CIFAR100 dataset, like this:
torchvision.datasets.CIFAR100('data', train=True, download=True, transform=Compose([ToTensor(), Normalize(0.5, 0.5)]))

In addition, since CIFAR100 has 100 classes, you also need to modify the "num_classes" parameter in line 47 of "train_classifier.py". Just set it to 100 and you're good to go.


Support multiple flash sizes

And also boards without NeoPixels.

cc @unexpectedmaker

Awesome! So if I wanted to add my ProS2 as well, I can just duplicate the feathers2 stuff?

Yup! Just another board def.


liquidhaskell-cabal doesn't build

$ nix-env -iA nixos.haskellPackages.liquidhaskell-cabal
installing 'liquidhaskell-cabal-<IP_ADDRESS>'
these derivations will be built:
  /nix/store/ph75kjblhqayywcnmlq8wkz85hssdqn0-liquidhaskell-cabal-<IP_ADDRESS>.drv
building '/nix/store/ph75kjblhqayywcnmlq8wkz85hssdqn0-liquidhaskell-cabal-<IP_ADDRESS>.drv'...
setupCompilerEnvironmentPhase
Build with /nix/store/5z8gx1bjw5s0q96n191k0a8caqr4r580-ghc-8.4.3.
ignoring (possibly broken) abi-depends field for packages
unpacking sources
unpacking source archive /nix/store/8ifd056kyfkgdslmin4wf7z9jfgvsxaw-liquidhaskell-cabal-<IP_ADDRESS>.tar.gz
source root is liquidhaskell-cabal-<IP_ADDRESS>
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file liquidhaskell-cabal-<IP_ADDRESS>/src/LiquidHaskell/Cabal.hs
patching sources
compileBuildDriverPhase
setupCompileFlags: -package-db=/build/setup-package.conf.d -j1 -threaded
[1 of 1] Compiling Main             ( Setup.hs, /build/Main.o )
Linking Setup ...
configuring
configureFlags: --verbose --prefix=/nix/store/vsfgcr0y319cjn69vpxp0b8342xwyc0y-liquidhaskell-cabal-<IP_ADDRESS> --libdir=$prefix/lib/$compiler --libsubdir=$abi/$libname --docdir=/nix/store/lihnm95imjbs8jpfi83nmb4chpq5nrmv-liquidhaskell-cabal-<IP_ADDRESS>-doc/share/doc/liquidhaskell-cabal-<IP_ADDRESS> --with-gcc=gcc --package-db=/build/package.conf.d --ghc-option=-j1 --disable-split-objs --enable-library-profiling --profiling-detail=exported-functions --disable-profiling --enable-shared --disable-coverage --enable-static --disable-executable-dynamic --enable-tests --disable-benchmarks --enable-library-vanilla --enable-library-for-ghci --ghc-option=-split-sections --extra-lib-dirs=/nix/store/s2n99784krxl91mfw3cnn9ylbb5fjvkx-ncurses-6.1/lib --extra-lib-dirs=/nix/store/rxaclc3vhxbzgfllv7601i85z854h48a-gmp-6.1.2/lib
Using Parsec parser
Configuring liquidhaskell-cabal-<IP_ADDRESS>...
CallStack (from HasCallStack):
  die', called at libraries/Cabal/Cabal/Distribution/Simple/Configure.hs:958:20 in Cabal-<IP_ADDRESS>:Distribution.Simple.Configure
  configureFinalizedPackage, called at libraries/Cabal/Cabal/Distribution/Simple/Configure.hs:462:12 in Cabal-<IP_ADDRESS>:Distribution.Simple.Configure
  configure, called at libraries/Cabal/Cabal/Distribution/Simple.hs:596:20 in Cabal-<IP_ADDRESS>:Distribution.Simple
  confHook, called at libraries/Cabal/Cabal/Distribution/Simple/UserHooks.hs:67:5 in Cabal-<IP_ADDRESS>:Distribution.Simple.UserHooks
  configureAction, called at libraries/Cabal/Cabal/Distribution/Simple.hs:178:19 in Cabal-<IP_ADDRESS>:Distribution.Simple
  defaultMainHelper, called at libraries/Cabal/Cabal/Distribution/Simple.hs:115:27 in Cabal-<IP_ADDRESS>:Distribution.Simple
  defaultMain, called at Setup.hs:2:8 in main:Main
Setup: Encountered missing dependencies:
Cabal >=1.18 && <1.26

builder for '/nix/store/ph75kjblhqayywcnmlq8wkz85hssdqn0-liquidhaskell-cabal-<IP_ADDRESS>.drv' failed with exit code 1
error: build of '/nix/store/ph75kjblhqayywcnmlq8wkz85hssdqn0-liquidhaskell-cabal-<IP_ADDRESS>.drv' failed

Technical details

system: "x86_64-linux"
host os: Linux 4.14.71, NixOS, 19.03pre153158.7df10f388da (Koi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.2
channels(turion): ""
channels(root): "musnix, nixos-19.03pre153158.7df10f388da"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

https://github.com/spinda/liquidhaskell-cabal/issues/11

This is an upstream issue that's unrelated to Nix.


[1961] Remove #manage_ui_url method

Context
#manage_ui_url is not being used. Remove dead code.
Changes proposed in this pull request
Delete dead code
Guidance to review
Double check it is not being used.

 Remove manage_ui_url helper in MCFE


feat(DIA-1017): add bottom sheet to infinite discovery

This PR resolves DIA-1017
Description
This PR adds the bottom sheet element to the infinite discovery screen. We use preloaded data from the artwork card screen have the data ready if the user wants to swipe to check more details.
For next PRs:

handle navigation to external links(we might need dedicated navigation container to keep the Bottom Sheet open)
tracking
Other works tab

iOS
https://github.com/user-attachments/assets/451f1acb-cecf-4ce4-b448-17a7c09cabf5
Android
https://github.com/user-attachments/assets/4e95b6fe-0e55-4b54-a950-e97e6381a6fc
PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

feat: add artwork bottom sheet to infinite discovery(hidden by ff)

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (feat: add artwork bottom sheet to infinite discovery(hidden by ff) - araujobarret)

  Generated by :no_entry_sign: dangerJS against 61d44aaddd3056680f98cd9486870e126d1eebb2


Followed the composable way but component isn't rendered

Hello,
I just copy pasted the composable way but it looks like the inputs arent rendered.

i'm using
 "@inertiajs/inertia": "^0.11.0",
        "@inertiajs/inertia-vue3": "^0.6.0",
        "@inertiajs/progress": "^0.2.7",
        "@vitejs/plugin-vue": "^3.0.0",
        "autoprefixer": "^10.4.2",
        "laravel-vite-plugin": "^0.5.0",
        "postcss": "^8.4.6",
        "vite": "^3.0.0",
        "vue": "^3.2.31"
    },
    "dependencies": {
        "@formkit/vue": "^1.0.0-beta.12",
        "@inertiajs/inertia": "^0.11.0",
        "@inertiajs/inertia-vue3": "^0.6.0",
        "formkit-addon-inertia": "^1.0.0",
        "postcss-nesting": "^10.1.10",
        "vue": "^3.2.36"

Are you sure that
<script setup>
import { useInertia } from "formkit-addon-inertia";
</script>

<template>
  <FormKit type="form" @submit="(fields, node) => useInertia(node).post('/user', fields)">
    <FormKit type="text" name="name" label="Name" />
    <FormKit type="email" name="email" label="E-mail" />
  </FormKit>
</template>

would load the formkit form? no registration on my app.js or anything?
thanks

Did you follow the FormKit installation method? it seens to me that <FormKit> was not enabled globally by adding to the vue.use(formkitPlugin, defaultConfig) this addon only adds some functionality to the node, it does not make any changes to FormKit render capabilities.

Hi Gustavo, i followed your  method 1 (adding formkit to a composable)
With the vue.use(formkitPlugin, defaultConfig) in my app.js createApp i couldn't get FormKit to work either.
So i hoped you addon was an all in one sollution to get FormKit to work.
Meanwhile i use some custom inputs from laravel breeze which work good as well, and is a bit lighter in filesize

What was the problem with vue.use? this addon is only a integration between the two, so you need those two already configured with their respective installation methods.


Test route

Codecov Report

Merging #93 (af07e54) into master (b5cb774) will increase coverage by 5.20%.
The diff coverage is 100.00%.

:exclamation: Current head af07e54 differs from pull request most recent head 776ac68. Consider uploading reports for the commit 776ac68 to get more accurate results

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
+ Coverage   44.75%   49.95%   +5.20%
==========================================
  Files          83       83
  Lines        1220     1223       +3
  Branches       42       42
==========================================
+ Hits          546      611      +65
+ Misses        674      612      -62

Impacted Files
Coverage Δ

.../main/scala/org/bitlap/zim/domain/model/User.scala
100.00% <ø> (ø)

...m/server/configuration/AkkaHttpConfiguration.scala
11.11% <ø> (ø)

...ap/zim/server/configuration/ApiConfiguration.scala
15.38% <ø> (ø)

...server/configuration/ZimServiceConfiguration.scala
100.00% <ø> (ø)

...main/scala/org/bitlap/zim/tapir/ApiJsonCodec.scala
28.20% <ø> (+14.10%)
:arrow_up:

...n/scala/org/bitlap/zim/server/api/ZimUserApi.scala
100.00% <100.00%> (+100.00%)
:arrow_up:

...itlap/zim/server/application/impl/ApiService.scala
3.03% <100.00%> (+2.26%)
:arrow_up:

...tlap/zim/server/application/impl/UserService.scala
73.72% <100.00%> (ø)

...im/server/repository/TangibleGroupRepository.scala
100.00% <100.00%> (ø)

... and 15 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 b5cb774...776ac68. Read the comment docs.


Added support for unnamed enums and boolean fields

Codecov Report

Merging #4 (6a501f4) into main (45b3657) will increase coverage by 5.82%.
The diff coverage is 100.00%.

:exclamation: Current head 6a501f4 differs from pull request most recent head fdf91b0. Consider uploading reports for the commit fdf91b0 to get more accurate results

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   79.41%   85.23%   +5.82%
==========================================
  Files           5        6       +1
  Lines         170      237      +67
==========================================
+ Hits          135      202      +67
  Misses         35       35

Impacted Files
Coverage Δ

src/implementations.rs
51.72% <100.00%> (+4.55%)
:arrow_up:

tests/validate_unnamed_enum.rs
100.00% <100.00%> (ø)

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 45b3657...fdf91b0. Read the comment docs.


[HUDI-2044] Integrate consumers with rocksDB and compression within External Spillable Map

What is the purpose of the pull request
HUDI-2028 ( #3112, #3117, #3194) implemented rockDb support and compression for bit cask map for External Spillable map and integrated with the HoodieMergeHandle
This PR extends the  integration of these two features to other consumers, such as SpillableFileSystem and HoodieMergedLogRecordScanner etc.
Brief change log

Moved configs to HoodieCommonConfig that is a newly created class for common configs
Integrated with SpillableFileSystem and HoodieMergedLogRecordScanner.

Verify this pull request
Added unit tests/ functional tests for different integration points

Codecov Report

Merging #3318 (ac05c5b) into master (d58a834) will decrease coverage by 42.84%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #3318       +/-   ##
============================================
- Coverage     45.66%   2.82%   -42.85%
+ Complexity     5600      85     -5515
============================================
  Files           999     284      -715
  Lines         43781   11872    -31909
  Branches       4403     986     -3417
============================================
- Hits          19993     335    -19658
+ Misses        22004   11511    -10493
+ Partials       1784      26     -1758

Flag
Coverage Δ

hudicli
?

hudiclient
0.00% <0.00%> (-34.56%)
:arrow_down:

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudiintegtest
?

hudisparkdatasource
?

hudisync
4.88% <ø> (-51.10%)
:arrow_down:

huditimelineservice
?

hudiutilities
8.97% <ø> (-50.91%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Force "Slow 3G", "6x Slowdown" on CPU and report Hardware concurrency to be 1.

Pages load too fast in chrome, I always have to open dev tools, and turn everything down in performance tab.
Would be a lot better if they were at the minimum by default.

could be a feature you could turn on in settings


Broken BaseHtml::collectErrors with showAllErrors = true in <IP_ADDRESS>

In a scenario where you have the same error message on multiple fields and want to display all of them BaseHtml::collectErrors will crash with 'Undefined offset' error at line 1260
https://github.com/yiisoft/yii2/blob/e63a459a18924bc1672c473c1bbeadfc1b502879/framework/helpers/BaseHtml.php#L1260
Example:
$model->getErrors(): Array (     [field1] => Array         (             [0] => Message1             [1] => Message2         )     [field2] => Array         (             [0] => Message1         ) ) $model->getErrorSummary(): Array (     [0] => Message1     [1] => Message1     [2] => Message2 )
echo Html::errorSummary($model,['showAllErrors'=>true]);
will throw error
Its because of line 1255:
https://github.com/yiisoft/yii2/blob/e63a459a18924bc1672c473c1bbeadfc1b502879/framework/helpers/BaseHtml.php#L1255
array_unique will preserve keys and Model::getErrorSummary produces an array where the same error message is grouped on consecutive positions. So $lines ends up like this
Array (     [0] => Message1     [2] => Message2 )
Simple fix: $lines = array_values($lines); after foreach
Here is the relevant stack of the ErrorException:
[yii\base\ErrorException:8] yii\base\ErrorException: Undefined offset: 1 in /vendor/yiisoft/yii2/helpers/BaseHtml.php:1260 Stack trace: #0 /vendor/yiisoft/yii2/helpers/BaseHtml.php(1260): yii\base\ErrorHandler->handleError(8, 'Undefined offse...', '...', 1260, Array) #1 /vendor/yiisoft/yii2/helpers/BaseHtml.php(1226): yii\helpers\BaseHtml::collectErrors(Array, true, true) #2 /vendor/yiisoft/yii2/widgets/ActiveForm.php(305): yii\helpers\BaseHtml::errorSummary(Object(...), Array)

Thank you very much for your report. Would you like to help us and try to submit a pull request with fix?

ups, it seams someone already fixed this,
thx :)

Sorry @FlorinRo, I was working on the core so I thought to publish the pull request.

Fixed by #15859


Get my current location automatically

we can do this with a conditional and navigator.geolocation:
if (navigator.geolocation{
code to get location and store it, etc.
}
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/geolocation

Implemented


feat: add handleRequestAsync to IPC OperationContinuationHandler

Issue #, if available:
Description of changes:
This change adds a new method handleRequestAsync to the OperationContinuationHandler of our IPC server. This method returns a completable future which ultimately returns the result. It is optional to implement this method. The reason to use it, is that the IPC request can be handled by a different thread, and thus unblock the SDK's main thread, allowing it to process other requests.
Why is this change necessary:
This change enables us to not block the main SDK thread while doing something which would otherwise block. This is needed for our CDA APIs which perform cloud calls which we want to return a result from during the initial IPC request (non-streaming).
How was this change tested:
Manually updated PubSub server to use this method and run the existing handler in a different thread, showing that it does work asynchronously the same way that it worked synchronously.

[ ] Updated or added new unit tests.
[ ] Updated or added new integration tests.
[ ] Updated or added new end-to-end tests.
[ ] If my code makes a remote network call, it was tested with a proxy.

Any additional information or context required to review the change:
Documentation Checklist:

[ ] Updated the README if applicable.

Compatibility Checklist:

[x] I confirm that the change is backwards compatible.
[ ] Any modification or deletion of public interfaces does not impact other plugin components.
[ ] For external library version updates, I have reviewed its change logs and Nucleus does not consume
any deprecated method or type.

Refer to Compatibility Guidelines for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

note: approved by @graebm as well.


How to sign trx of tron?

Is there has any demo？There is only the signature of the trc20 contract in the document, I want to sign the trx.

Signing example in C++ tests:
https://github.com/trustwallet/wallet-core/blob/master/tests/Tron/TWAnySignerTests.cpp
You may combine this with other swift/kotlin examples.


[WFLY-19341] Make the testsuite/galleon tests appropriately scoped

https://issues.redhat.com/browse/WFLY-19341

More information about the wildfly-bot[bot]

Thanks @jfdenise


Add feature to add assignments to all students under same module

Fixes #99
Test cases for the new classes have been added
User Guide has been updated with this command

Just need to resolve current conflicts


Emit events for all TaskRun lifecycle events

Changes

Emit events for additional TaskRun lifecyle events:

taskrun started
taskrun running

Fix the logic in events.go to compare semantic equality as
opposed to raw pointer equality.
Fix broken EmitEvents unit tests and extend them to cover new
functionality.
Extend reconcile test to verify new events are sent. To do so,
get the event recorder from the context when creating the
controller - if avaialble. This allows using the fake recorder
for testing instead of having to look for event related actions
in the fake client go action list.
Add documentation on events.
Fixes #2328
Work towards #2082
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
We now emit kubernetes events for additional TaskRun lifecyle events:
- taskrun started
- taskrun running

One of the reason why certain tests fail here, is that we test on the number of actions executed in certain error cases, and this patch adds one "emit event" for the start case.
This got me to think about when do we want to consider the TaskRun as started, and thus emit an event for it.
This is roughly the ordering of events in the controller today.
This is not 100% complete / correct, but it should be enough to highlight the issues.

main method
action
re-queue
set condition
events
notes

Reconcile
hasStarted
no
if true, set start time
start event
start timeout

Reconcile
isDone
maybe
no

metrics, cloud events, stop timeout

reconcile
set defaults
no
no

reconcile
convert
no
fail task on failure

reconcile
isCancelled
no
fail task if true
end event

reconcile
get task
no
fail task on failure

also convert when fetch from API

reconcile
labels and annotations
no
no

reconcile
check timeout
no
fail task if expired
end event

reconcile
bindings
no
fail task on failure

pipeline resources / params

reconcile
create pod if needed
maybe

some errors are handled, some re-queue

reconcile
update from pod
maybe
end if pod end
end event
may be success of failure

Issues that I see:

Currently the start event is before we validate the conversion, task and bindings. In case of validation failures we have a start event but no end event.

We could move the start event to later on, but then it would not match with the start-time set in the taskrun anymore. Also later on it is a bit less obvious when a a TaskRun has to be considered as new, since start time has been set already. We could use the fact that the pod does not exists yet as an indication. Alternatively we could send event on other error conditions, like when TaskRun fails validation of fetching the task or so, but that feels a bit artificial. I think it would be consistent with metrics thought, not 100% sure.
I would like to get more opinion on this, I might bring it up at the WG.
@vdemeester @bobcatfish @ImJasonH thoghts?

The timeout check is a bit in the middle of validations

This is maybe a cosmetic thing, but I think we should move the timeout check somewhere else, maybe at the beginning of reconcile. I can file a separate issue for this.

We re-run fetching the task, doing conversion and validation on every reconcile cycle that hits reconcile. This is a bit inefficient and also may cause issues if the Task definition was to change in the middle of the TaskRun.

This is not new, and not something that we should change in this PR, but the fact that we might associate events to failures on getting the task / validations adds more to the problem.

The more I think about this, the more I tend to think that we should emit events at the very start of the TaskRun i.e. when the controller first looks at it, and emit failures in case of validation failure - since that is something that task/pipeline editors could look for when testing their work.
If there is no opposing opinion this week I'll move forward under this assumption, as I'd really like to move on with this series of PR swiftly now.

I like the table a lot but im not quite sure how to interpret the "requeue" and "notes" columns, maybe a bit more detail on those?
i wonder if there is some way to isolate the logic that is used to emit events to make it a bit simpler, e.g. treat reconciling as having phases (i havent looked at your other PR @afrittoli which reorganizes reconciling so maybe you're doing this already!) but something like:

validation / fetching everything
making changes we need to make (e.g. creating TaskRuns, pods)
updating status + emitting events

Maybe we could look at the status the Runs and use that to decide what events to emit, i.e. isolate the logic for this all in one place instead of throughout the reconcile
Sounds like the logic around the start time is a bit more subtle than what above is capturing tho 🤔

I like the table a lot but im not quite sure how to interpret the "requeue" and "notes" columns, maybe a bit more detail on those?
i wonder if there is some way to isolate the logic that is used to emit events to make it a bit simpler, e.g. treat reconciling as having phases (i havent looked at your other PR @afrittoli which reorganizes reconciling so maybe you're doing this already!) but something like:
1. validation / fetching everything

2. making changes we need to make (e.g. creating TaskRuns, pods)

3. updating status + emitting events

Maybe we could look at the status the Runs and use that to decide what events to emit, i.e. isolate the logic for this all in one place instead of throughout the reconcile
Sounds like the logic around the start time is a bit more subtle than what above is capturing tho 🤔

Yeah, I've been trying to organize things a bit more in the other PR - I would welcome feedback there. Looking at how other resources behave I think I need some changes in the PR related to the start event:

only use Succeeded condition for the end event
emit an event for the Created condition when the reconciler first meets the resource
emit an event for the Started condition when validation is passed and we are about to attempt to create the pod
emit an event for the Running condition once the pod is created successfully

For pipeline we will similarly have:

only use Succeeded condition for the end event
emit an event for the Created condition when the reconciler first meets the pipeline
emit an event for the Started condition when validation is passed and we have a DAG
emit an event for the Running condition once the first TaskRun is applied successfully

I like the table a lot but im not quite sure how to interpret the "requeue" and "notes" columns, maybe a bit more detail on those?
i wonder if there is some way to isolate the logic that is used to emit events to make it a bit simpler, e.g. treat reconciling as having phases (i havent looked at your other PR @afrittoli which reorganizes reconciling so maybe you're doing this already!) but something like:
1. validation / fetching everything

2. making changes we need to make (e.g. creating TaskRuns, pods)

3. updating status + emitting events

Maybe we could look at the status the Runs and use that to decide what events to emit, i.e. isolate the logic for this all in one place instead of throughout the reconcile
Sounds like the logic around the start time is a bit more subtle than what above is capturing tho 🤔

Yeah, I've been trying to organize things a bit more in the other PR - I would welcome feedback there. Looking at how other resources behave I think I need some changes in the PR related to the start event:
* only use `Succeeded` reason for the end event

* emit an event for the `Created` reason when the reconciler first meets the resource

* emit an event for the `Started` reason when validation is passed and we are about to attempt to create the pod

* emit an event for the `Running` reason once the pod is created successfully

For pipeline we will similarly have:
* only use `Succeeded` reason for the end event

* emit an event for the `Created` reason when the reconciler first meets the pipeline

* emit an event for the `Started` reason when validation is passed and we have a DAG

* emit an event for the `Running` reason once the first TaskRun is applied successfully

@bobcatfish going back on your point of organizing the reconcile in phases, I think it might be nice to have use different conditions during reconcile to identify them, and send events on transitions. The only downside might be more updates to the status. This is something I would prefer to do after this patch though.
We should also add a doc page about different conditions we have today and their meaning.

The following is the coverage report on the affected files.
Say /test pull-tekton-pipeline-go-coverage to re-run this coverage report
File 	Old Coverage 	New Coverage 	Delta
pkg/reconciler/event.go 	80.0% 	100.0% 	20.0
pkg/reconciler/taskrun/taskrun.go 	74.5% 	75.1% 	0.6

Nice increase in coverage, I only need to fix the data race now...

/retest

/test pull-tekton-pipeline-integration-tests

/test pull-tekton-pipeline-build-tests

All of my feedback is super minor! Mostly comments and a couple typos.
The only piece of feedback that's at all substantial is that I'd really like to find away to avoid the sleep in the reconciler test. I don't mind merging this as is but if we could find an alternative to that I think it's worth the effort (or at least a detailed comment explaining why it's unavoidable - imo sleeps, especially in unit tests, should not be the norm)

Yeah, good catch - this is actually one of the reasons I changed how we run event related unit tests - and finally I forgot to drop the sleep. My bet is that this is not needed anymore, so I will re-spin this without it and with fixes to typos.

/test pull-tekton-pipeline-build-tests

I think there were 1 or 2 comments that might make sense to add but nothing worth blocking over!
Thanks for this @afrittoli !! :D 🎉
/lgtm

ps

nice!!! 👍 👍 👍


How do I change the value of borderWidth on hover?

In the bar chart that I have set default borderWidth = 0. But on hover, I need to set it as 1. How do I do this using callbacks?

Have you tried the hoverBorderWidth property of the dataset?

Thank you, I had not tried this.
But this again poses the following issue - When the mouse is on a particular bar, and the graph values are updated, it no longer shows the border.
Initial -

(border is highlighted)

Values are updated, bar chart re-renders, but does not show the hoverBorderWidth.
How do I solve this?

I'm not sure what the best way to solve this is. The chart gets re-rendered and the hover state is lost until the mouse moves again. You might be able to write a plugin that detects this case and keeps the hover styling

Thank you, I'll try this


Refactor Parser docs code snippets

This was exactly what I was looking for, and worked great.  Some comments on making the docs better.
In the Parsing event section, the event_parser decorator calls out:

event_parser decorator will throw a ValidationError if your event cannot be parsed according to the model.

but this has an implication on the Lambda response that could be more explicitly called out.  I mainly noticed this because it caused a change in my unit test behavior.
The next section parse function example does solve this, but by using def my_function(): instead of def handler(event: Order, context: LambdaContext): as in the previous example, this was harder (for me) to realize this was the example I really wanted.
Also, the first example had the order: classes, hander def, payload def, invoke hander - and the next example was: classes, payload, function def, (no invoke example).  The first example seemed to flow better, and these could be normalized.

Hey @hakanson! Hope you are well.
Now I'm refactoring all the docs, examples and adding more details in general. I'm planning to send a PR tomorrow or Wednesday..
After submitting the PR, can I tag you to help me validate and improve it?

Hey @hakanson! Hope you are well.
Now I'm refactoring all the docs, examples and adding more details in general. I'm planning to send a PR tomorrow or Wednesday..
After submitting the PR, can I tag you to help me validate and improve it?

Yes, I have some commitments coming but will do my best.


Added Krautreporter

I have two questions:

Is there a way to filter urls in the Publisher specification to avoid requesting pages like impressum, etc.?
The JSON-LD here is wrapped in a CDATA tag - I think it might be cleaner to handle this in the _base_setup of the BaseParser (I've prepared an implementation, too)

Hi @dkm1006,
Thank you for adding Krautreporter. To answer your questions:

Yes, Sitemap and NewsMap objects have an attribute called sitemap_filter which expects a regex_filter. You can find more details here about how that can be done.
Since this seems to be specific to only this one publisher, I was thinking it may be best to just override _base_setup locally for Krautreporter to keep potential future fixes for other publishers from aggregating in that function. But maybe we can see what @dobbersc thinks as well.

@addie9800 thanks for your response!

It’s my understanding that sitemap_filter only filters links to sub sitemaps (i.e. in a recursive call).
Generally, I agree. However, a quick google search though showed that this CDATA enclosed JSON-LD may be (relatively) common. The solution could be extended to stripping the json of all comments.

Ok, my bad. I didn't read your question well enough and you're absolutely correct, that's the way the sitemap_filter works and in this case unfortunately there is no option I am aware of that would filters urls from a the publisher specification. At least I have also struggled with it in the past. But that might be an idea for future work.
I think stripping the json of all comments might be my preferred solution. This would then also cover potential other future issues. I did some research and it seems like json5 could be an option, since it natively supports and ignores (single and multiline) comments while remaining a superset of json.

Okay, thank you, I‘ll see if I can add something like a url_filter parameter to the Sitemap.
json5 in addition to adding an external dependency apparently is very slow and does more than needed - maybe a simple regex would suffice for 99% of cases? I’ll update my PR with a proposal.

Hey @dkm1006 thanks for the PR ;)

Regarding your first question: Yes there is. The Publisher class has an init parameter url_filter which allows you to define a std URL filter used for this specific publisher. I.e. writing something like this:

Publisher(
    ....,
    ....,
    url_filter = regex_filter("/video/")
)

would filter out all URLs including a subpath /video/ for the specified publisher.

Regarding your second question: I like the idea of having a dedicated LD parsing function. It looks way cleaner now! Thanks a lot. I just made it a bit more generic by capturing only text content inside curly brackets instead of removing the specific CDATA tag.

Thanks for the hint with the url_filter parameter! This is almost exactly what I was looking for. Here the RSS feed has a different url pattern than the sitemap (i.e. the RSS feed has links without topics in the path (https://krautreporter.de/5335-fast-alles-was-du-uber-die-eu-dachtest-stimmt-nicht-mehr) whereas the sitemap includes topics (https://krautreporter.de/politik-und-macht/5335-fast-alles-was-du-uber-die-eu-dachtest-stimmt-nicht-mehr). This makes it impossible to write a global filter that is exact but I think this is a minor issue, as the simple url filter I've added catches 99% of urls that should be filtered out.
I've also changed the json_pattern a little to account for whitespace between [ and { (e.g.  [ {}, {} ] would result in an error for the previous pattern).


[Informational] fix resistfingerprinting & youtube video white bottom half

It seems as of today [april 10th 2020] Youtube is projecting a white bar on the bottom half of youtube video's if you have "privacy.resistFingerprinting" set to "true". (tyvm google...)
I spotted a fix on reddit and thought it might be useful to post the fix (requires UBO) here as well.
Add this filter to Ublock Origin: www.youtube.com##.ytp-gradient-bottom
I suspect this will be quickly fixed with updated filter lists, so feel free to remove/close if it's out of scope.

i noticed this yesterday and thought it was my 'make the whole damn web dark and stop bloody blinding me or else' add-on
thanks for the tip
for those using CSS injection add-on (Stylus) ...
/* fat white bar on bottom half of videos on mouse-over, pause */
.ytp-gradient-bottom {
    display: none;
}

bonus tip...
to disable YT's "polymer" which is slow as hell and apparently uses a depreciated API because they hate Firefox, you can do this with some sort of redirect add-on (Redirector)...
 Description: YouTube disable polymer
Example URL: https://www.youtube.com/watch/?v=somevideo
Include pattern: https:\/\/www\.youtube\.com\/(?!.*disable_polymer=true$)(.*\?.*)
Redirect to: https://www.youtube.com/$1&disable_polymer=true
Pattern type: Regular Expression

Hallo! Thank you, @bitpixl! I was going crazy about this for one hour because I thought I accidentally hit some shortcut on the keyboard. Good to know I am not alone. I tried the uBlock Origin solution, but it does not work for me. I am not sure if I am applying this correctly: I copy www.youtube.com##.ytp-gradient-bottom and paste it on a line in the "My filters" tab of the uBlock Origin settings. Then I click on "Apply changes". But the white bottom on Youtube is still there, even when closing and restarting Firefox. Am I applying the fix in a wrong way or do you have any other idea why this isn't working for me?

@theltalpha I'n not sure why it's not working on your end, maybe you can try putting that line at the top of the "my filters" tab/list or try with a fresh profile with just user.js+ubo? If you keep having problems I think it's probably best asking in the reddit link I posted or maybe try the UBO forums.

@theltalpha make sure cosmetic filtering is enabled on YT (eye icon in uBO popup).

@theltalpha make sure cosmetic filtering is enabled on YT (eye icon in uBO popup).

That was it! I had globally de-activated cosmetic filtering as per 12bytes' uBlock Origin Suggested Settings and activated it now as per site setting for Youtube. Excellent, thank you!

I had globally de-activated cosmetic filtering as per 12bytes' uBlock Origin Suggested Settings

What is the point?

from the uBO pop-up...

Generic cosmetic filters are those cosmetic filters which are meant to apply on all web sites.
Though handled efficiently by uBlock₀, generic cosmetic filters may still contribute measurable memory and CPU overhead on some web pages, especially for large and long-lived ones.

@atomGit unrelated, this filter is "specific" - it works specifically on www.youtube.com.

with Stylus you can apply css per domain also - i do it this way only because i don't want to enable cosmetic filtering in uBO which, in turn, enables those filters in all the filter lists that one uses that includes cosmetic filters
that said, i don't actually know whether the performance hit is less with css injection or not - i never tested

FF nilla (nightly if you must know), set RFP on, restarted for good measure, no extensions and loaded up some TayTay

https://www.youtube.com/watch?v=Dkk9gvTmCXY - Taylor Swift: You Need to Calm Down
everything works as expected, no white canvas overlay BS
see https://bugzilla.mozilla.org/show_bug.cgi?id=1628947

Still, it's not hard to detect RFP (even if canvas wasn't a tell-tale 10x10 white) and penalize users - not that I'm saying that's why google did it - maybe it was an A/B test ... 👌 👃 (my attempt at saying fuck knows (nose) using emojis)
That reddit poster recommending turning off RFP was a joke. They do the same thing for people asking about why their version says 68 and they can't install an extension from AMO ... instead of telling them how to bypass it, etc.
Closing. Speak up if I missed anything

That reddit poster recommending turning off RFP was a joke. They do the same thing for people asking about why their version says 68 and they can't install an extension from AMO ... instead of telling them how to bypass it, etc.

They are just tired by people asking the same thing every few days again and again.

They are just tired by of people asking the same thing every few days again and again

They should know better. Anyway .. who enjoyed the TayTay vid?

When Firefox works with pref privacy.resistFingerprinting:true it opens the browser in the window 1000x891. Because of this, youtube displays the video player as in theatrical mode (but this is not):
  Screenshot

Is there any way to do so it would be displayed as in old youtube layout?
  Screenshot

you can override that part of RFP globally with privacy.resistFingerprinting.letterboxing, which is what i do, but this makes one more susceptible to fingerprinting
if you want a youtube specific solution, i suppose you could try one of the many youtube add-ons or, perhaps better (safer), write your own CSS in userContent.css

you can override that part of RFP globally with privacy.resistFingerprinting.letterboxing, which is what i do, but this makes one more susceptible to fingerprinting

I dont want to messup with privacy.resistFingerprinting. I am already used to it.

if you want a youtube specific solution, i suppose you could try one of the many youtube add-ons or, perhaps better (safer), write your own CSS in userContent.css

Not one of the addons did not help. I searched the whole AMO. CSS script does not exist either. I tried to write my own but I didn't succeed. So far, I have rewritten the code of one of the addons and use this last method to view youtube in old layout. But this is a matter of time until google blocks it. :(

just for kicks you could try disabling whatever YT add-on you're using and load this video and see what happens - appended to the URL is the &disable_polymer=true parameter ... if that works for you, you can use a redirector add-on to make that permanent

let me also throw this in here in case it might be of interest...
Privacy Redirect
the problem with the Invidious instances is that they aren't always reliable - instances come and go or can take a long time to load or don't always work with certain videos, but it does offer a substantial privacy boost

Im used before script like this:
var url = window.location.href;
if (url.indexOf("disable_polymer") === -1) {
  if (url.indexOf("?") > 0) {
    url += "&";
  } else {
    url += "?";
  }
  url += "disable_polymer=1";
  window.location.href = url;
}

Not work properly anymore. Now i have addon that just change useragent for youtube only. Works fine. For now. But already need to look for a replacement with normal polymer displaying.
Privacy Redirect addon use invidio.us site with youtube links only. This is not privacy at all. Video is still uploaded from google servers.

the default method for viewing YT videos with Privacy Redirect bypasses youtube.com, so it is better for privacy since all the tracking/JS from youtube is avoided, however you still have to connect to google to get the video
if you enable proxy mode, then you bypass both youtube and google, however i really don't recommend that unless you donate to the service since this adds a lot to bandwidth costs
but yes, either way, the Invidious instances are sometimes slow and less than reliable


[PDAL] update to 2.2.0

Library name:
PDAL
New version number:
2.2.0 (current on vcpkg is 1.7.2)
It contains multiple features and perf improvements since 1.7.2 (it's a whole release in ahead!). I did try doing this myself by changing the download source and re-implementing the patches but my lack of knowledge of cmake has me stuck this:
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_arbiter" that is not in any export set.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_kazhdan" that is not in any export set.

You can see my changes here:
https://github.com/microsoft/vcpkg/compare/master...GeorgeR:master

Is there an update on this? Assume 2.2.0 is not available yet?
pdal                 1.7.1#11         PDAL - Point Data Abstraction Library is a library for manipulating point clou...
pdal-c               1.8-2            C API for the Point Data Abstraction Library (PDAL)

The version update to 2.3.0 is still blocked by the same problem.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_arbiter" that is not in any export set.
CMake Error: install(EXPORT "PDALTargets" ...) includes target "pdalcpp" which requires target "pdal_kazhdan" that is not in any export set.

I also noticed that @GeorgeR has reported this issue to padl https://gitter.im/PDAL/PDAL?at=5fea156fc746c6431cdfcc4f.
Andrew Bell @abellgithub Jan 05 20:52
@GeorgeR : I don't understand why you'd get this. Both of those libraries are statically linked to PDAL and are listed as private dependencies, though I have no idea what vcpkg is or what it's doing. This is more of a cmake issue than a PDAL issue.

As @Andrew Bell said, this is more of a cmake issue than a PDAL issue.
I also searched the related issue about cmake. https://gitlab.kitware.com/cmake/cmake/-/issues/15415.
There was a workaround that can fix this problem, but not sure if it is reasonable for CMake since this issue still keeps open.
Adding $<BUILD_INTERFACE:...> for pdal_arbiter and pdal_kazhdan in target_link_libraries().

Work in progress now in #21275.


Can't add to Packages.swift

When added, and running vapor Xcode, these errors are produced:
Check your dependencies' Package.swift files to see where the conflict is.
Error: warning: refname '0.2.0' is ambiguous.
warning: refname '0.2.0' is ambiguous.
warning: refname '0.8.0' is ambiguous.
warning: refname '0.8.0' is ambiguous.
swift-package: error: The dependency graph could not be satisfied (https://github.com/vapor/vapor.git)

This package depends on an alpha version of Vapor so to use it your project would also need to do so:
.Package(url: "https://github.com/vapor/vapor.git", Version(2, 0, 0, prereleaseIdentifiers: ["alpha"]))

This package hasn't been tagged 1.0 yet so use it at your own discretion.
FYI, you can check out the Package.swift files in the packages to see what they depend on. If this aligns with what your package and its dependencies use there should be no issues.
https://github.com/vapor/jwt-provider/blob/master/Package.swift


Filter for Accepted Answer added

@Acrylami I've added the filter, now it would only show good and accepted answers. We don't need to edit question_ids. It would now automatically take question_ids having a good answer.
For formatting, from my end, I am already discarding \n that fills the answer, but @Levy-Naibei would need to take care of the other things like printing the code in the answers the right way.
Screenshot of code working

@Acrylami I've added the filter, now it would only show good and accepted answers. We don't need to edit question_ids. It would now automatically take question_ids having a good answer.
For formatting, from my end, I am already discarding \n that fills the answer, but @Levy-Naibei would need to take care of the other things like printing the code in the answers the right way.
Screenshot of code working

@yashika51 , that great!
I will also still format it in such away as to separate the different answers.


For creating postgres cluster, why is the cpu limit required to be added ?

In the cluster resource, why is it required to add cpu limits
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
...
spec:
  resources:
    limits:
      memory: 1Gi
      cpu: 1
...

As mentioned here https://www.howtogeek.com/devops/should-you-set-kubernetes-cpu-limits/ , it should be not mandatory to set the cpu limits, so why is it so with the operator ?

Hello @ditpoo
It is not mandatory to set this field spec.resources.limits.cpu, just leave it out of your chart and the pods of the cluster will be created without a CPU limit.

Not an issue, closing


feat: back office integration extend get s51 advice by case reference endpoint

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/ASB-2026
PR extends /api/v1/advice?caseRef={caseRef} for back office integration for specific case references using the BACK_OFFICE_INTEGRATION_GET_ADVICE_CASE_REFERENCES flag. Also includes refactor to follow the repository-service design pattern.
Useful information to review or test
Please use the seed script first to generate dummy data.
To confirm back office integrations works locally, use BC0110001:
http://localhost:3000/api/v1/advice?caseRef=BC0110001

To confirm ni integration still works locally (regression) use anything else:
http://localhost:3000/api/v1/advice?caseRef=TR040009

Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

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

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


Support URL params on New For You page

This PR is well described over on https://github.com/artsy/force/pull/10482 but the tl;dr is that I've updated the /new-for-you page so that you can pass first, includeBackfill and version to override defaults.
/cc @artsy/grow-devs @joeyAghion @kathytafel

Ok this is green so I'm going to merge!


send RSET command on connection return

I've been getting an error 503 5.5.1 Error: nested MAIL command  once in a while. Couldn't reproduce the issue in local enviroment but i suspect that connections somehow are returned to pool in unfinished state and then used for sending next message resulting in multiple MAIL commands. Resetting connection when returning to pool solved the issue.

Ah, in fact the last commit https://github.com/knadh/smtppool/commit/844bd4ab29a4b52c7c3af92232b4a99fb573fa0e specifically had fixed the nested mail issue. Bad connections were incorrectly being reused. Maybe the correct approach was to issue a reset there?
While this patch ensures that a connection is cleaned every time, I'm not sure if that's the ideal way. Connections shouldn't be in undetectable bad states in the first place. Would be great if that could be figured out first.


[RFC] [WIP] Schedule cloud-cleaner run

This pull request includes:
I'm trying to see if it's possible to trap cloud-cleaner during test execution

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Apparently when a job get canceled the runner gets destroyed immediately so trapping cloud-cleaner doesn't seem to be an option.
I came up with another way which should work but I'm not sure if it's robust enough or if it's not too complicated. I created just a simple first version to present an idea of how it would work. This would solve our problem with canceled pipelines and this is how it's supposed to work:

Every X hours send a request to get all canceled jobs in X hours
Parse all the required variables from the output
Run cloud-cleaner with the required variables for each canceled job

Of course this would filter out jobs that are only interacting with clouds. Also if this is to work properly we'll have to simplify cloud resources naming so that it's possible to put it together even if the original variables are gone meaning not using any uuidgen or something like that. I've added more comments inside the actual scipt.
I will appreciate any comments or suggestions how to make this work for us.

I think that running the cloud-cleaner regularly is a good idea. The prerequisite is as you've mentioned to use predictable resource names. This is done e.g. in api.sh for GCP, where in CI it uses predictable TEST_ID and if run outside of CI environment, it uses random UUID. I'm all for using predictable TEST_ID as a baseline for predictable resource names, but I think that we should keep a fallback to using random UUID to keep the test case script working in local environments.
So if we can reasonably recreate all necessary environment variables for each canceled job, then this is great and we should definitely run cloud-cleaner for them!
In addition, I think that cloud-cleaner may need some modifications / fixes to make it work for all tests that create cloud resources, because AFAIK it properly cleans up after api.sh only for GCP. For the other clouds, it cleans up only after image tests which can boot the image (I guess, but I'm not sure).

For the other clouds, it cleans up only after image tests which can boot the image (I guess, but I'm not sure).

For Azure at least (b/c this is most recent) you are correct. api.sh cleans up after itself but that won't happen if the job is cancelled.

Ok finally pushed an update. I tried to describe it in the commit message, hope it's clear what it does now.
The reason why I'm not using tags for the Azure resources is because of this bug https://github.com/Azure/azure-cli/issues/19138 ... If I listed just tagged resources I could not tell when they were created and I didn't want to clean them all since they could be from a currently running job. It would be possible to just look for the tag in the resources I'm listing now but I don't see any added value currently as it's only listing resources from a resource group that's used for testing and if someone names their resource differently that the jobs do it won't clean them up anyway. I'm fine with adding the additional check for tags if you think it's going to be better though.
The plan is to create a scheduled pipeline which will run it once a day for now and see if that's enough. The time to look back can be set as env var and I'm thinking to currently set it to 3 hours as that's the timeout we have for jobs.
I'm also currently doing a test where I want to cancel api and integration azure jobs in the middle of creating resources and let this clean up. I'll post the results in the next comment.

This is now ready for review. I tested it here https://gitlab.com/osbuild/ci/osbuild-composer/-/jobs/1507709881 . It cleaned up leftover resources from one of my failed previous runs. See my previous comment and commits for more info.

Shall we merge this @atodorov ? It seems there are no complaints against this and I don't want to leave it open for too long.


Performance anti-pattern in UI

Hello guys!
I found that in TSChatTimeCell and TSChatSystemCell, there are LabelViews with their background colors set to transparent (alpha between 0 and 1). I think it would be better not to use transparent or partially transparent views in repeating styles, like TableViewCells and inside for loops.
Due to my own experience and this Apple document, using transparency could be tricky, performance-wise.
Do you think it makes sense the code is refactored to improve performance?

Yeah, absolutely yes. I  didn't do anything about the performance of this project so far. So we can improve the performance off this project that you refer to.
I will do it later.  XD


nx:run-commands as postTarget receives extraneous arguments

After updating from Nx 14.1.5 to 14.3.6, my postTargets that use nx:run-commands with npm publish started failing with an odd error:
[eslint-config] ❌ Error: ERROR: Something went wrong in run-commands - Command failed: npm publish --command="npm publish" --cwd=libs/eslint-config --__unparsed__= --parallel=true --color=false
    at /Users/me/someNxProject/node_modules/nx/src/executors/run-commands/run-commands.impl.js:59:19
    at Generator.throw (<anonymous>)
    at rejected (/Users/me/someNxProject/node_modules/tslib/tslib.js:116:69)

Note all the extra arguments after npm publish. Weirdly enough if I just run the postTarget task directly then it works just fine, which leads me to believe it was caused by this Nx change: https://github.com/nrwl/nx/commit/da1faaf483668227b9a7e8706426c93846c5ef4a
I think this might be a bug in Nx, but I saw that you're doing some special enhancing of the options before passing them to runExecutor, so I'm really not certain.
As a workaround, I've simply disabled forwardAllArgs for the time being.

Turns out this is because nx:run-commands just appends whatever its options.__unparsed__ contains, regardless of whether it contains a parameter that it owns itself such as "color", "command", etc. The reason it just now became a problem, is because __unparsed__ wasn't set when using runExecutor() prior to https://github.com/nrwl/nx/commit/da1faaf483668227b9a7e8706426c93846c5ef4a
This is a bug in the run-commands executor, so I'm closing the issue.


About extracting inference results

// Populate the input vectors
const auto& inputDims = engine.getInputDims();
std::vector<std::vectorcv::cuda::GpuMat> inputs;
// TODO:
// For the sake of the demo, we will be feeding the same image to all the inputs
// You should populate your inputs appropriately.
for (const auto & inputDim : inputDims) {
    std::vector<cv::cuda::GpuMat> input;
    for (size_t j = 0; j < batchSize; ++j) {
        cv::cuda::GpuMat resized;
        // TODO:
        // You can choose to resize by scaling, adding padding, or a combination of the two in order to maintain the aspect ratio
        // You can use the Engine::resizeKeepAspectRatioPadRightBottom to resize to a square while maintain the aspect ratio (adds padding where necessary to achieve this).
        // If you are running the sample code using the suggested model, then the input image already has the correct size.
        // The following resizes without maintaining aspect ratio so use carefully!
        cv::cuda::resize(img, resized, cv::Size(inputDim.d[2], inputDim.d[1])); // TRT dims are (height, width) whereas OpenCV is (width, height)
        input.emplace_back(std::move(resized));
    }
    inputs.emplace_back(std::move(input));
}

//std::array<float, 3> subVals {0.5f, 0.5f, 0.5f};
//std::array<float, 3> divVals {0.5f, 0.5f, 0.5f};   --> 1 detected after execution  abnormal operation
const std::array<float, 3> subVals { 0.f, 0.f, 0.f };
const std::array<float, 3> divVals { 1.f, 1.f, 1.f };  --> 13 detected after execution  in normal operation
bool normalize = true;
std::vector<std::vector<std::vector>> featureVectors;
bool succ = runInference(inputs, featureVectors, subVals, divVals, normalize );
13 detected after execution
featureVectors[0][0][0] -> num_dets value  = 1.962e-44#DEN strange value -> Expected to be 13 is normal
featureVectors[0][1][0] -> det_boxes value = 261.750000 x normal value
featureVectors[0][1][1] -> det_boxes value = 39.4375000 y normal value
featureVectors[0][1][2] -> det_boxes value = 301.250000 x + width normal value
featureVectors[0][1][3] -> det_boxes value = 78.8750000 y + height normal value
...
featureVectors[0][2[[0] -> det_scores value = 0.910644531 normal value
featureVectors[0][2[[1] -> det_scores value = 0.903320313 normal value
...
featureVectors[0][3[[0] -> det_classes value = 2.803e-45#DEN strange value -> A value between 0 and 79 is normal
featureVectors[0][3[[1] -> det_classes value = 1.037e-43#DEN strange value -> A value between 0 and 79 is normal
...
I would appreciate it if you could tell me how to extract the inference result value normally.

Can you provide the onnx model.

yes uploaded

Have a look at my TensorRT YoloV8 project where I use this repo to run inference behind the scenes. https://github.com/cyrusbehr/YOLOv8-TensorRT-CPP

I also ran it for yolov8 you linked.
But the output format is different.
If the existing ones are not output as num_dets, det_boxes, det_scores, and det_class values, I know that the linked source contains the boxes/scores/classes values in one array.
Another question is why the values are out of order. Is there any way to get the value to come out normally?

@MingiKang sorry for the delay.
Looking over your comment again, I see that you included this in your conversion code: --end2end
end2end means the model is exported with built in NMS. My implementation does not support that, as the NMS step is done programatically external to the model. You should therefore re-export without that flag and try again.
I will close this issue for now, but if you continue to have problems, feel free to reopen.


How many CNNs are used?

Hi,
I am confused about how you handle both pose estimation and category classification. For this purpose, do you use different CNNs?  When I have checked the code it seems there is a single model. However, when I checked the paper I deduced several CNNs are used (mainly, Figure2 demonstrate some important issues, I guess I missed the main idea). If you could explain the architecture clearly, I would appreciate. Thanks in advance.

Hi,
In Fig. 2, the weights of the CNNs are shared and so there is actually a single CNN model. That CNN model outputs a M(N+1)-dimensional vector for each image, where M denotes the number of discrete poses and N denotes the number of categories. Therefore, a single CNN classifies both category and pose.


Added new functionality to mask a CNPJ Base

There is a concept of "CNPJ Base" to mask just the 8 first character.
Some brazilian government sites uses searches with this first letters. For exemple: http://www.sped.fazenda.gov.br/spedfiscalserver/ConsultaContribuinte/
Look up the orange description of the field.

Coverage decreased (-5.0%) to 91.667% when pulling e07cbae488e8916117d7620df781d2b5e714f2bc on leonardocostaborges:master into 651cc46347b1d8279bebca454c36d65869ad709f on the-darc:master.

Coverage increased (+0.2%) to 96.875% when pulling 5cc07ff9d5748de5657f64c31d003a543d6cd17f on leonardocostaborges:master into 651cc46347b1d8279bebca454c36d65869ad709f on the-darc:master.


Sign in with Microsoft forwards to localhost

Issue Type: Bug
When I clicked Sign in with Microsoft, my browser was opened but the link was for localhost: https://localhost:1596/signin?nonce=[...]
VS Code version: Code - Insiders 1.53.0-insider (98106c48a07d4d07f0f71b4db9b3ff156f223339, 2021-01-01T05:22:15.141Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1650 v4 @ 3.60GHz (12 x 3592)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.92GB (12.05GB free)

Process Argv
--crash-reporter-id 1ac95a6f-47b7-4cb3-b7d9-fc1fd273a17f

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

xml
Dot
2.5.1

csharp
ms-
1.23.8

remote-containers
ms-
0.154.2

remote-ssh
ms-
0.62.0

remote-ssh-edit
ms-
0.62.0

remote-wsl
ms-
0.52.0

vscode-remote-extensionpack
ms-
0.20.0

powershell-preview
ms-
2020.9.0

vsliveshare
ms-
1.0.3449

debugger-for-chrome
msj
4.12.11

A/B Experiments
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt600cf:30241728

More specifically, I'm getting ERR_SSL_PROTOCOL_ERROR when my browser tries to open the link:

More specifically, I'm getting ERR_SSL_PROTOCOL_ERROR when my browser tries to open the link:


New release 1.1.0

New features

Support ethtool link mode. (c23e7b5)
Support ethtool ring. (ddb9b57)
Support ethtool coalesce. (29bd758)
Support ethtool features/offload. (9ddd42f)
Support ethool pause. (63e2211, 0c0aeda)
Support bond vlan+srcmac hashing option. (32e5d0a)
Support permanent MAC address. (ef5b56d)
Initial tech-preview support on changing IP address. (c6757ec)

Break changes

Varlink interface been removed. (6a1d14a)

Date fixed.


feat(searchbar): autofocus directive (desktop only)

(last commit only)
DO NOT MERGE UNTIL THIS PR ( https://github.com/driftyco/ionic/pull/8162 ) IS MERGED!!
Fixes (partially): https://github.com/driftyco/ionic/issues/5834

@manucorporat Would be really nice to have this!
I've done some research on this some time ago and have been using a directive which so far seems to work great on mobile (cordova only) as well. It does require the <preference name="KeyboardDisplayRequiresUserAction" value="false" /> preference in config.xml.
Another thing I've learned is that autofocus is terrible on cordova when the input that gets focus is still being animated. That's why I'm using the ngAfterViewInit lifecycle event with a relatively long timeout, so I'm sure all view animations are finished. I see you are using the viewController.didEnter which should be even better!
This directive works on all elements/components that have an <input> inside, like <ion-input> and the searchbar as well. It would be great to see autofocus support for additional components integrated into Ionic as well :).
import { AfterViewInit, Directive, ElementRef, Renderer } from '@angular/core';

@Directive({
  selector : '[customAutofocus]'
})
export class AutofocusDirective implements AfterViewInit {
  constructor(
    private renderer: Renderer,
    private elementRef: ElementRef
  ) {}

  ngAfterViewInit() {
    const element = this.elementRef.nativeElement.querySelector('input');
    setTimeout(() => {
      this.renderer.invokeElementMethod(element, 'focus', []);
    }, 800);
  }
}

@manucorporat I found out that my autofocus only works in UIWebView. (related: https://github.com/driftyco/cordova-plugin-wkwebview-engine/pull/22)

Can't wait for this! Without this the UX is just plain broken

Any estimation on when/if this would be merged?

Would be great if this will be released soon :)

@manucorporat Just noting that PR #8162 was merged, so what is blocking this PR?

@manucorporat Can you fix the merge conflicts on this and if it is still correct we can get it merged?

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!


How do I verify if the server received the message?

I have a doubt, it seems that message:sent emits before the actually physical send message, I need to verify if the server received that message but it seems that the event receipt doesn't do this, how do I know if the server gets the message successfully?

👍 I need too.


Bug: client hydration stats not correctly displayed in Analog project

Please provide the environment you discovered this bug in.
After creating a new Angular v16 project with Analog, in development mode
Which area/package is the issue in?
create-analog
Description
When provideClientHydration() is used, the numbers do not show correctly.
Angular hydrated NaN component(s) and NaN node(s), undefined component(s) were skipped. Note: this feature is in Developer Preview mode. Learn more at https://next.angular.io/guide/hydration.
This is due to how ngDevMode is set in the vite plugin
https://github.com/analogjs/analog/blob/main/packages/vite-plugin-angular/src/lib/angular-vite-plugin.ts#L119
This used to be set to an empty object in development mode, now its not defined at all in development, and set to false during the build for production.
Please provide the exception or error you saw
No response
Other information
The fix
define: {
  ngJitMode: 'false',
  ngI18nClosureMode: 'false',
  ...(watchMode ? {} : { ngDevMode: 'false' }),
},

I would be willing to submit a PR to fix this issue

[ ] Yes
[ ] No

I can work on this. Also open to add the good-first-issue label to this since the solution is already provided and it could be an invitation for people to dive into the source code

Good first issue is fine with me


Created another unique item: Red Onyx Watch

Closes #237

Current coverage is 23.40%

Merging #238 into master will not affect coverage as of f1b2e8a

@@            master   #238   diff @@
=====================================
  Files          164    164
  Stmts         4547   4546     -1
  Branches       738    738
  Methods          0      0
=====================================
  Hit           1064   1064
  Partial         93     93
+ Missed        3390   3389     -1

Review entire Coverage Diff as of f1b2e8a

Powered by Codecov. Updated on successful CI builds.


Disable semgrep scan local

Why it failed
CircleCI failed when running on the default branch op-es because it performs a full scan for the default branch, whereas it does an incremental scan (smart contract files from the latest commit) on PR branches. As a result, the PR branch succeeded, but the default branch failed.
What is semgrep
semgrep is a static analysis tool that scans files using user-defined rules. In this repository, it scans all Solidity files based on the rules defined here
What are the errors
The error report is available here.
The report lists 40 findings where SoulGasToken.sol does not follow OP’s code style guidelines. For example:Named inputs to functions must be prepended with an underscore
What is done
Filed an issue to track this.

Is it still required after https://github.com/ethstorage/optimism/pull/116 is merged?

Is it still required after #116 is merged?

It will be not required anyone

Close since the issue is solved by https://github.com/ethstorage/optimism/pull/116.


JSON parsing in ICS20 is key-order dependent and strict

The current implementation of the ICS20Lib requires the keys in the transfer packet data JSON to be in a specific order. If any keys are out of order, the parsing fails.
If we found a way to support parsing it in any order, it would be great, but we need to consider the gas cost, and ideally keep the best-case scenario to more or less the same gas as now.
Another issue is that the JSON parsing would fail if there are any missing or added fields. The only optional field at this point is the memo field. So we should figure out what we want to allow for and update that.

Not an issue anymore after #116


How to use manual mode start?

I set startMode: 'manual' in my capacitor.config.ts
I have my built application in public/nodejs so it gets copied into the build.
I call it like this: await NodeJS.start({ script: 'dist/index.js', env: { NODE_ENV: 'production' } });
The file s located at /data/data/com.example.myApp/files/nodejs/public/dist/index.js (via android device explorer)
I added Logger.info(CapacitorNodeJSPlugin.LOGGER_TAG, "NATIVE START" + Arrays.toString(arguments)); right before the call to  nativeStart(arguments, environmentVariables, true); in NodeProcess.java and I see:  NATIVE START[node, /data/user/0/com.example.myApp/files/nodejs/public/dist/index.js] so it does seem like it should use that file. However it doesnt seem to actually start my App.  Before  I go any farther in debugging, is this all I need to do?
EDIT:  I should say that when i turn off manual mode and switch back to whenReady() the app boots as expected

aha! I'll give it a go. thanks. It'd be nice to see it in one of the examples at some point.

I tried that approach and it didn't work. Is it possible you could try it in one of the examples?

The example was unfortunately out of date. I've updated everything and added some new examples. Take a look at the example/manual_start branch.


naoqi_bridge: 0.5.4-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository naoqi_bridge to 0.5.4-0:

upstream repository: https://github.com/ros-naoqi/naoqi_bridge.git
release repository: https://github.com/ros-naoqi/naoqi_bridge-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.21
previous version for package: null

naoqi_apps

No changes

naoqi_bridge

No changes

naoqi_driver_py
* Merge branch 'add_fake_effort' of https://github.com/k-okada/naoqi_bridge into k-okada-add_fake_effort
* Contributors: Vincent Rabaud

naoqi_pose

No changes

naoqi_sensors_py
* remove the dependency on driver_base
* Contributors: Vincent Rabaud

naoqi_tools

No changes

retriggering travis