Update ESP32SJA1000.cpp

The #include "esp_intr.h" depreciated and should be replaced by #include "esp_intr_alloc.h"

I tried this commit, and it just gives many more errors

This change works fine for me.  I have made a private version with this and the interrupt enable mask fix. No trouble.


Tech debt: Fixed linting errors and skipped tests

Fixes frontend linting errors and skipped tests.

@curtisupshall
Found an issue: If you edit the map, and select a layer from the dropdown (nrm regional boundaries in my case), you get an infinite loop of requests to the openmaps service. I confirmed this is not the case in Dev. It looks like throttledGetFeatures is getting called in a loop.

@curtisupshall Maps seem happier after the latest changes from what I can tell.
There is still 1 dependency error popping up though, in ProjectAdvancedFilters.tsx L92.
I did give it a try locally, and adding the dependency (biohubApi.taxonomy) it is complaining about seems to work just fine (doesn't trigger any loops or anything).


Support for CloudEvents format.

Requires: https://github.com/fnproject/cli/pull/236

The full example in README.md needs to be updated to look like the func.rb - i.e. test to see if input.respond_to?(:fetch)
Otherwise in the case of --format default when input is a string it's going to return nil unless that string contains 'name'

In context.rb, I'd suggest instead of...
    def protocol
      if cloud_event?
        return event['extensions']['protocol']
      end
      event['protocol']
    end

...use...
    def protocol
      cloud_event? ? event['extensions']['protocol'] : event['protocol']
    end

(and similarly for the other methods)

And drop the return from cloud_event?

In runner.rb you might want to look at the indentation of the cloudevent lambda.


[KIECLOUD-346] Editorial changes: username to user name and typos

https://issues.jboss.org/browse/KIECLOUD-346
Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

@mramendi Looks good, please just squash the 2 commits into 1. Thanks!

Squashed


Merge Enums into shared folder

This close #154

Hi thanks for the PR!
Unfortunatelly, you only updated the path at the server files, but forgot to update at the client files. When I ran an npm build this was the output:


Bump Kubernetes dependency for Longhorn v1.5.0

longhorn/longhorn#5595
This is a draft to facilitate discussion. I will convert it to an actual PR once the changes I need from https://github.com/longhorn/backing-image-manager/pull/125 are merged.

The draft PR LGTM

Waiting for e2e core tests' result

Waiting for e2e core tests' result

This tested fine when I was building it with a local replace for the backing-image-manager change. Testing again with the backing-image-manager from master-head.

[ ] https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3869/console

Waiting for e2e core tests' result

This tested fine when I was building it with a local replace for the backing-image-manager change. Testing again with the backing-image-manager from master-head.

[ ]  https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3869/console

@ejweber https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3869/testReport/tests/test_basic/test_restore_inc_with_offline_expansion_s3_/ it's not related to backing image, so it could be test flaky.

@ejweber https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3869/testReport/tests/test_basic/test_restore_inc_with_offline_expansion_s3_/ it's not related to backing image, so it could be test flaky.

Passed in a subsequent run: https://ci.longhorn.io/job/private/job/longhorn-tests-regression/3871/testReport/tests/test_basic/.


Scenario Checking Tool

The tool is described here: https://docs.google.com/document/d/1rSfQ9GwF2osHn0iBr-FSrLp4OnErWZP5T7Y2zglGEBE/edit?usp=sharing
The tool would check specific data in the created scenario (‘scenario_links.dbd’).  This would come after the scenario is built successfully and could be a standalone tool, or integrated within the scenario builder.  Things to check for would be:

consistency between the AB/BA lanes and the Dir field - if direction is 1, check topology and make sure that correct AB or BA attributes are populated. If direction is 0, make sure that AB and BA attributes are populated.  For layers where there are errors, the scenario will create correctly, however, the model will crash because it won’t be able to create the .net file.
Transit Route Highway Link check - During scenario creation, check transit routes to make sure that all the links that they are attached to have the correct attributes. In some cases, for the 2040 v1.2 model, a transit route was added to a TWB route that did not have attributes.  In this case, the scenario was created correctly, however, the model crashed an hour into the run because it could not create the transit skims.
Check speed values for transit links

Implementation underway
Resolved by fill all TWB lanes with 1
Assigned to MPOs

I've created the "Check Scenario Roadway" that will run immediately after the roadway project manager creates the scenario roadway dbd. I've written to make it easy to add checks over time as we think of them. This will be in v2.


Example data?

This looks like an interesting program!
I wonder if you could provide a small dataset for testing it out?

Great, thanks!


License

hello
I want to create a site related to this and put ads and earn money. Can I use the code?
or what is the license of the code?

sure no problem
On Sat, Aug 3, 2024 at 12:40 PM ispirLee25 @.***> wrote:

hello
I want to create a site related to this and put ads and earn money. Can I
use the code?
or what is the license of the code?
—
Reply to this email directly, view it on GitHub
https://github.com/benlhachemi/modifio/issues/13, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMWQGHZNW6V57ZTJ62MGOTLZPS6SJAVCNFSM6AAAAABL527KZKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2DMMZRGIYDAOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


[PLAT-1028] Use consistent naming for JSON-API "type" (snake_case)

Purpose
Our casing for the type JSON-API field is annoyingly inconsistent. This fix converts them all to snake_case for versions > 2.11 .
Changes

Simple change to base serializer and add version to API

QA Notes
All type fields for v2 json responses should be snake case for version 2.12, earlier versions are about have some kebab-case. external-identities is a good endpoint to check.
Documentation
Will have to refactor docs.
Side Effects
None that I know of.
Ticket
https://openscience.atlassian.net/projects/PLAT/issues/PLAT-1028

We're going to be moving to kebab case instead of a snake case, closing.


Dubbo版本（2.7.8）无法消费Dubbo（2.5.10）服务，报错 no provider

[ ] I have searched the issues of this repository and believe that this is not a duplicate.
[ ] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.8
Operating System version: xxx
Java version: 1.8

Steps to reproduce this issue

Dubbo版本（2.7.8）无法消费Dubbo（2.5.10）服务，报错 no provider，如图：

Pls. provide [GitHub address] to reproduce this issue.
Expected Result
What do you expected from the above steps？
Actual Result
What actually happens?
If there is an exception, please attach the exception trace:
Just put your stack trace here!

Please upgrade the server dubbo version.

Please upgrade the server dubbo version.

yes，use 2.7.10，Solved the problem

&READY-TO-CLOSE&


KubernetesV1 configMapFile should accept a key

When creating a config map, need the ability to specify the key name when creating --from-file
kubectl create configmap mapName --from-file=<my-key-name>=<path-to-file>
Currently specifying a key before the filename doesn't work e.g.
config.json=$(System.DefaultWorkingDirectory)/_StudentPortalWeb-CI-YAML/drop/config.dev.json
Despite the command not working, this also doesn't fail the task. Logs indicate that the command was used incorrectly and task reports "success"
Create a ConfigMap
KubernetesV1 YAML

We add the filename as keyname by default if a filepath is provided. You just have to provide the filepath in the input. Does that work for you?

This issue is related to when the keyname is not the same as the filepath.
Consider the scenario where I have config.dev.json and config.prod.json.
I have a k8s namespace for dev and prod and each namespace has a configmap called config.json
In my deployment I will mount the configmap config.json.
If setting the keyname is not possible, I will have change the volume mount to the configmap for each environment or, rename the source file prior to calling kubectl in order to use the default value.

This issue is stale because it has been open for a year with no activity. Remove the stale label or comment on the issue otherwise this will be closed in 5 days


1wallet have dissapeared

Using Firefox - wallet is gone after clearing history from browser -how do i restore - i have Google authenticator but it asks me to use camera which i dont have (ayyached to my desktop) -Please help to recover Wallet adress one1draf2a8n4t3ckksywq87gvf5pc6dmtextjkzvq

Resolved in https://t.me/the1wallet


Pre-generate thumbnails to improve performance

Currently, thumbnails are generated at run-time as needed using the thumbnail tag. Each time this tag is used, first the app must check if the thumbnail already exists on the media storage, and if not, generate it. When the media storage is remote such as on S3, this can take a lot of time.
It would be much more efficient to instead use the thumbnail_url tag which assumes the thumbnail already exists and just writes the appropriate url in the HTML. This just means the thumbnail should be pre-generated whenever the thumbnail_options for an image are modified.
There could be a switch to enable both the current dynamic and the pre-generating behavior, but since I don't see any advantage for the dynamic approach in the context of this package, I suggest just changing the behavior to always pre-genrate the thumbnails.

It looks like I was wrong - for non-local storage, both the source and thumbnail generation time are queried in the database instead of querying the modification time of the file in the storage. (I'm still seeing this taking a significant time on page load with just a few thumbnails, but it's probably not due to remote storage access.)

@danra djangocms-blog uses easy_thumbnails for thumbnail generation, which caches the thumbnails status in the database and support async thumnails generation. If it's your use case, you can change the template to reflex this and have a look at east_thumbnails async pregeneration https://easy-thumbnails.readthedocs.io/en/stable/usage/#asynchronous-pregeneration


fix: upgrade package-lock.json lockfileVersion from 1 to 2

npm ci was used to generate the file. This change should fix an
issue, where an incorrect version of domelementtype was resolved,
when including this library in other projects.

Hmm,
can you provide more details on the issue and how the lock file version helps to fix it?

I know another issue that should be fixed once I bump the dependencies (was going to do that today)

Take this dependency tree for example. Since the dependencies in this projects aren't locked properly, we end up with a wrong version of domelementtype for domhandler.
The version of domelementtype being installed for domhandler is 1.3.1, which does not work and causes an error, when executing the code.
Since html-react-parser indirectly depends on an older version of domhandler, which requires<EMAIL_ADDRESS>, we end up with version 1.3.1 for the domelementype version of this project.
$ npm ls domelementtype
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ └──<EMAIL_ADDRESS>│   └─┬<EMAIL_ADDRESS>│     ├──<EMAIL_ADDRESS>deduped
│     └─┬<EMAIL_ADDRESS>│       ├─┬<EMAIL_ADDRESS>│       │ └──<EMAIL_ADDRESS>│       └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   ├─┬<EMAIL_ADDRESS>│   │ └─┬<EMAIL_ADDRESS>│   │   └──<EMAIL_ADDRESS>deduped invalid: "^2.2.0" from node_modules/@selderee/plugin-htmlparser2/node_modules/domhandler <-------- HERE
│   └─┬<EMAIL_ADDRESS>│     ├──<EMAIL_ADDRESS>│     ├─┬<EMAIL_ADDRESS>│     │ └──<EMAIL_ADDRESS>deduped
│     └─┬<EMAIL_ADDRESS>│       ├─┬<EMAIL_ADDRESS>│       │ └──<EMAIL_ADDRESS>deduped
│       └──<EMAIL_ADDRESS>deduped

This doesn't answer why package-lock.json file version can fix the issue.
According to npm documentation, it should be able to get the same information from v1 file.
I can't see how the dependencies here "aren't locked properly".
I know npm can be quite liberal in how it resolves dependencies of dependencies, making the whole lockfile somewhat pointless. One of the options would be to pin the dependency in your project directly.
It's just a couple of months until I move to node 14 and npm 7, but I want to know anyway whether it is my problem and not the other project with outdated dependencies or npm itself.
I'm not convinced yet.

Alright, we already solved it by upgrading html-react-parser. Now the dedup issues are resolved.
I'm not sure if this is really an issue with this lib or npm itself.
Closing this for now :)

The irony:
This thread nudged me to configure npm workspaces for this repo.
As a result, there is no need in these package-lock files anymore, so they're gone, automatically.
This illustrates that package-lock file is a property of a particular package dev environment.
Only package.json is published and used as a source for downstream package lock files.


[BUG]: Regex for email validation is not working correctly

The regex for email validation is not working correctly. It is allowing invalid email addresses to pass through the validation. If the email start with capital letters and start with number.

Hello @YossefAbdelMohsen , can you let me know how to reproduce this bug? what should I do in which page to see that an invalid email is getting accepted?

Hi,
Here as you can see the email is valid but it start with capital letter so it gives me an error

also here the first letter is a number but it gives me an error as well

Ok, understood.


Make garden pipeline shell command for testing pipelines locally (Steve's Quest)

One of the hardest things about maintaining DLHub is debugging models. There is not an easy way to tell if the problem is coming from the DLHub system or from the user's code or model. For Garden, we would like to create a workflow that provides a well-defined way for pipeline authors to test their pipelines (including models) locally.
The biggest difficulty users have testing locally is setting up an environment that matches the remote environment their pipeline will run in. Pipeline objects specify their environment. Pipelines have pip_dependencies, conda_dependencies, and python_version fields. For remote execution, Globus Compute's Container Service makes a container matching those specifications. We would like to also have a CLI command that constructs this environment locally, letting the user test their code and check if they specified their environment correctly.
An interaction might look like this. The user asks for a shell for my_pipeline.py. The utility creates the environment and puts the pipeline function in scope.
> garden-ai pipeline shell "/path/to/my_pipeline.py"

Installing ....

>>> my_pipeline(['OC1CCCCC1O'])
[3.5]
>>>

Open questions

What tools should we wrap to build this? Ben G says that MLFLow has a good approach we can steal. Owen has looked into that and it seems like MLFlow doesn't expose its environment-building capabilities. Whatever we use (plain venv, MLFLow, poetry, conda, who knows), the goal is to be able to build environments for as many pipelines as we can while also not requiring onerous setup for the user.
Should we depend on the Container Service or make a Container Service option? A tempting way to approach this would be to use Container Service to make the container that meets the user's specifications, pull that container down, and open a shell in the container. But we probably want to have a testing method that is completely agnostic of our backend so it would be good to not depend on the Container Service. Using containers from Container Service also assumes the user has a Docker install. But maybe a Container Service version of this command would be a useful first draft. It could also be a useful option for debugging.
How will we handle environment cleanup and reactivation?  If you have just sat through a long environment setup, you probably don't want to sit through it again if you exit the shell and re-enter it a minute later. But also we want to make sure we don't fill up the hard drives of unsuspecting users. Should environments be single-use? Should they stick around and you can reactivate them? How can we make it easy to clean up old environments?

Assumptions:

While there should be descriptive error messages when appropriate, there will not be "auto-diagnostics" in this pass. eg, if a user's pipeline imports torch but the user have not included torch in their pip requirements, we won't run static analysis to tell them. They will just try to run their pipeline and get an import error.

Acceptance Criteria
Given I am a user with the Garden CLI installed and a valid Pipeline Python file, when I run garden-ai pipeline shell path/to/my/pipeline.py, then the Garden CLI constructs or reactivates an environment matching my pipeline's specs and deposits me in a shell session in that environment. The environment includes the pipeline function at top-level scope.

For bookkeeping, noting that @steve-barnard is on this. (Can't assign him yet but I will if we get him in as a collaborator on the repo.)


[DNS] BLOCKED: There is no way to list record sets in a zone

Email sent to Python SDK team.  This prevents us from implementing 'network dns record-set list' and 'network dns zone export', which makes 'network dns zone import' difficult to use due to the lack of a template for the user to start with.
BLOCKED:

[ ] Implement 'network dns record-set list
[ ] Implement 'network dns zone export'
[ ] Implement 'network dns zone import'

@mayurid if this is blocked, should we move it onto the backlog?

Final implementation dependent on service team review


Rename accessCtrl to accessControl

There are some places in the code where accessCtrl contraction is used. This contraction is unnecessary.
What should be done:

rename all accessCtrl instance in the code to accessControl

Can we remove contractions on all of the variables? Shouldn't cspell pick these contractions up?

Can we remove contractions on all of the variables?

I skimmed through the contracts again and haven't found out anything eye catching besides accessCtrl so let's keep it simple

Shouldn't cspell pick these contractions up?

I'm not sure that cspell can distinguish a contracted word with a full one

Consolidating this issue with https://github.com/ubiquity/ubiquity-dollar/issues/736


New support article on getting started with Python SDK

Requirements

Article in HelpScout
Whenever a user/partner asks about downloading audio or integrating with our platform then we can send them the link to this article

I created the draft document here


Update json file loading to use loadJsonContent

Overview/Summary
Updated json file loading to use loadJsonContent instead of json(loadTextContent. This approach allows for roughly 8x larger json files to be loaded by Bicep.
This PR fixes/adds/changes/removes

Fixes

Breaking Changes

Requires a version of Bicep that supports loadJsonContent

Testing Evidence
I made this change already in the SLZ variant of ALZ and successfully deployed the environment.
As part of this Pull Request I have

[x ] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[x ] Checked for duplicate Pull Requests
[x ] Associated it with relevant GitHub Issues
[ ] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x ] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x ] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[ x] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

@jtracey93 I removed the files we spoke update and updated this PR. I'll keep an eye on the linter though to make sure it is happy.

/azp run validateazcloud


feat: FSSTArray::into_canonical directly build VarBinView

Previously we had been building a VarBin and then using arrow_cast to cast into a VarBinView.
The implementation of arrow_cast for Utf8 -> Utf8View should be relatively efficient because it tries to reuse the buffer. However, it's likely more efficient to do the construction in a single-pass

You’re so fast 💨


[BEAM-7653] Add PTransformTranslator for Combine.GroupedValues

This is a follow up to #8995 but for the case of a runner that wants to have a specific translation of Combine.GroupedValues. Notice that this is the case of the Spark runner and this PR is to address a regression on it (since the runner moved to URN based translation instead of class based it is not calling the specialized translation).
R: @lukecwik

Run Spark ValidatesRunner

Notice that the extra fix in the test of the Spark runner shows that the validation is now correct because it calls a spark specific map instead of Beam's composite translation.

Is there a reason why the spark runner is intercepting and overriding combine grouped values and not combine globally/perKey and providing appropriate expansions for them?

@youngoli FYI

Spark Runner actually translates the three cases: Combine.PerKey, Combine.Globally and Combine.GroupedValues. But since there were not translators for the URN based case the translation was defaulting to the one of Combine.PerKey. This PR intends to make the missitn Combine.GroupedValues` to work as before.
For ref #7005 was the PR where we switched from class based translation to URN based.

I understand that #7005 was where the translation didn't handle these other cases and I'm just trying to learn how Spark is optimizing Combine.GroupedValues.

It is relatively simple it just 'combines' (reduces) the value of the iterable part and immediately produce the WindowedValue of this with a direct map, it is probably just a bit more lightweight than the normal translation because of reduced use of Beam constructs.

I see, this is for translation for when we execute pipelines without using the portable execution and where we can deserialize the users combinefn.

Yes, a 'simpler' case. Thanks Luke.


Revert module loading behavour

Change get_detection_modules(entrypoint) to treat modules argument as a list of modules to be loaded to fix behavior of the -m argument. ModuleNotFound exception is now thrown if a non-existent module name is passed.
@dmuhs I'm not sure why you implemented the exclude_modules arg before but I didn't see it used anywhere. Should we re-add this?

WRT #714 this was probably due to the old code in security.py:
         if not module_names or module.__name__ in module_names:
             logging.info("Executing " + str(module))
             issues += module.execute(statespace)

which I probably misread as not (module_names or module.__name__ in module_names) making module_names a blacklist.


Automated system testing

Aim to this topic is to share ideas about how can we integrate automated testing into our Software Development process.
Key points (to be extended or redefined)

Regression tests
End to end tests

Do we want to be able to run the tests on all platforms?
If it's enough to test on one platform, I still believe that shell scripts on Linux might be a good way for at least end to end tests.
Another alternative would be to write dedicated python scripts. We could use 'os' calls to test CLI functionality, not just the API.

E2E tests should be prepared per use case and include

One or more IPython notebooks pr stand-alone Python scripts containing Cate API calls
One or more shell scripts (Win .bat + Unix .sh) containing Cate CLI calls

Not sure how to automate IPython notebook execution maybe there are test runners for this out there?

I think it doesn't make much sense to do them as Jupyter notebooks. I don't see any advantage over plain python scripts there. Jupyter notebooks are nice for presentation, but we don't need that for E2E tests. I can of course convert the demonstration notebooks I make for each UC to a python script.
We could then either make a thin Python main test-runner script, or invoke them from shell scripts.
Ditto for shell scripts. I'm making the CLI demo examples for each UC, so these can be converted into shell scripts.
I assume that we don't validate the output, just test that the script runs and finishes successfully. Is this correct?

I move it to IPM5 as I don't see a chance to have it before our release. Note, I started with a scripts directory containing such possible E2E tests for the CLI.

@kbernat what is the status here? If any, please report at IRM8.


Static queue initialization

Currently the ClientBuilder creates three static queues upon initialization.
This limits the usage of atat to a single instance, as multiple calls to ClientBuilder::build will result in using the same queues.
This could be solved by passing the queues into the ClientBuilder as arguments, at the cost of API ergonomics. Some of which could be improved by a simple queues! macro

Closed by #50


 ;) Looks great. I wasn't sure how to handle those test failures in S.match either, I presume it's because of the weird object/array that you get back from using the native regex function in JS?

I wasn't sure how to handle those test failures in S.match either, I presume it's because of the weird object/array that you get back from using the native regex function in JS?

The problem actually lies with optional non-capturing groups. Since there's no type of which both 'foo' and undefined are members, ['foo', undefined] is heterogeneous and therefore considered invalid.
S.map(S.I, /(foo)?(bar)?/.exec('foobar'));
// => ['foobar', 'foo', 'bar']

S.map(S.I, /(foo)?(bar)?/.exec('foo'));
// ! TypeError: Type-variable constraint violation
//
//   map :: (Functor a, Functor b) => Function -> a -> b
//                                                ^
//                                                1
//
//   1)  ["foo", "foo", undefined, "index": 0, "input": "foo"] :: Array ???
//
//   Since there is no type of which all the above values are members, the type-variable constraint has been violated.

The problematic expression is map(toMaybe, match). The solution is to replace this with a for loop (match.map(toMaybe) wouldn't work since toMaybe would complain about receiving three arguments).
The fact that we must work around the type system when bringing inconsistent values into the consistent world doesn't worry me – in fact, it's a sign that the type system is doing its job. :)

The fact that we must work around the type system when bringing inconsistent values into the consistent world doesn't worry me – in fact, it's a sign that the type system is doing its job. :)

I agree completely, it's better to deal with these sort of oddities in the safety of a functions scope rather than having them leak out, even if it's at the cost of the implementations elegance.

:unicorn:


[GSSOC'23] add a feature of comment in the resume

I want to create a feature that allows users to make their resumes public on the platform, enabling others to provide feedback and suggestions for improvement. This will be especially helpful for beginners who can use it as a reference to create their own resumes.
Can u assign this feature under GSSOC'23

can i get 4-5 days time extra to complete this project . i am very much interesed in doing this project

Actually due to my sem exam the project is taking some time And I have built the backend part also I can show you my work also untill now what I did . give me a few days time extra  to complete this

Sure you can take time.


BEDUD ER A DMOLECH

Anyone know what this means? Something like kaubetikerreng? Literally it is “our rock of the deep” or something?

Something like that. One who is always there for you.
On Fri, Jul 15, 2022 at 12:56 AM John Bent @.***> wrote:

Anyone know what this means? Something like kaubetikerreng? Literally it
is “our rock of the deep” or something?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/30, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2DJJDZHJBL2QHS3BWVSADVUEKRXANCNFSM53UXHYWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yep, like "You're my rock"
On Fri, Jul 15, 2022, 6:46 AM Gnalu @.***> wrote:

Something like that. One who is always there for you.
On Fri, Jul 15, 2022 at 12:56 AM John Bent @.***> wrote:

Anyone know what this means? Something like kaubetikerreng? Literally it
is “our rock of the deep” or something?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/30, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AF2DJJDZHJBL2QHS3BWVSADVUEKRXANCNFSM53UXHYWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/30#issuecomment-1185510912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYSPXRL7FXSKL23UWTSUSLVUFMRVANCNFSM53UXHYWQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


authRequired: true returning "You must be logged in to do this." when valid X-Auth-Token and X-User-Id headers submitted

New to Meteor, getting used to things, like the look of restivus.
I've set up a simple test to try out authenticated access but I seem to run into an issue when adding authRequired: true to routeOptions or an endpoint.
Here's the code (basically your example added to a new meteor app, with the addition of an Organization collection):
    import { Meteor } from 'meteor/meteor';

    Organizations = new Mongo.Collection('organizations');

    Meteor.startup(() => {

        if (Meteor.isServer) {

            var Api = new Restivus({
                apiPath: 'api/',
                version: 'v1',
                auth: {
                    token: 'auth.apiKey',
                    user: function () {
                        return {
                            userId: this.request.headers['user-id'],
                            token: this.request.headers['login-token']
                        };
                    }
                },
                defaultHeaders: {
                    'Content-Type': 'application/json'
                },
                onLoggedIn: function () {
                    console.log(this.user.username + ' (' + this.userId + ') logged in');
                },
                onLoggedOut: function () {
                    console.log(this.user.username + ' (' + this.userId + ') logged out');
                },
                prettyJson: true,
                useDefaultAuth: true,
                enableCors: true
            });

            // Generates: POST on /api/users and GET, DELETE /api/users/:id for
            // Meteor.users collection
            Api.addCollection(Meteor.users, {
                excludedEndpoints: ['getAll', 'put'],
                routeOptions: {
                    authRequired: true
                },
                endpoints: {
                    post: {
                        authRequired: false
                    },
                    delete: {
                        roleRequired: 'admin'
                    }
                }
            });

            // Generates: GET, POST on /api/organizations and GET, PUT, DELETE on
            // /api/organizations/:id for the organizations collection
            Api.addCollection(Organizations, {
                excludedEndpoints: ['put', 'post', 'delete'],
                routeOptions: {
                    authRequired: true
                }
            });

        }

    });

Everything appears to work fine ... I run the app, then use Postman to add a user.
All OK so far.
Then, I go to: http://localhost:3000/api/v1/login and make a POST request with the body
{
    "password": "password",
    "email"<EMAIL_ADDRESS>}

This gives me what I expect to see:
{
  "status": "success",
  "data": {
    "authToken": "6Do263Poqthl7kGqrI1AnHsDir9LrrmWhdKIiyCepRs",
    "userId": "4ETkH9k4pcqGesdie"
  }
}

Next step is where I appear to run into trouble ... when I submit a GET request (via Postman) to http://localhost:3000/api/v1/organizations with the following headers:
Content-Type: application/json
X-Auth-Token: 6Do263Poqthl7kGqrI1AnHsDir9LrrmWhdKIiyCepRs
X-User-Id: 4ETkH9k4pcqGesdie

But this is what I get back:
{
  "status": "error",
  "message": "You must be logged in to do this."
}

If I set authRequired: false I get:
{
  "status": "success",
  "data": []
}

Which is what I expect to see after logging in and using the Token / UserId with authRequired: true in place.
I'm probably make a newb mistake but can't quite see what I'm doing wrong.
Is there an additional header I should be using in Postman, an error in the code somewhere, or something else going on that I need to fix?
Thanks in advance for your help!

Hey there! Welcome to Meteor! Hope you're enjoying it as much as I do! 😄
Pretty easy one to fix. You overrode the default API config with some demo options that I recommend you don't use (I warned against it in the docs right above the section that code came from, but I should probably make the warning bigger and bolder to make it more conspicuous). Believe it or not, you'd get the configuration you wanted by just doing the following:
var Api = new Restivus({
  version: 'v1',
  useDefaultAuth: true
});

// ...
// setup your routes/collections

That's it! Everything else looks good there. Try that on for size and let me know how it works out.

That did the trick, thanks!

Awesome! Thanks for following up. I added a new issue (linked above) to make that warning more obvious in the docs.


“up.command” can’t be opened because it is from an unidentified developer.

I am unable to start the VM after the latest update of k8solo v0.9.9.
This was working till v0.9.8.

hmm, strange one, will make a new app dmg soon

@iamShantanu101 can you please download k8solo v0.9.9 again, I uploaded rebuilt version, thanks

@rimusz Still same error.

can you provide some more detailed setup on your side

@rimusz This was fixed when the I restarted the system. But the reason behind this is still unknown.

maybe so macos glitch


Remove duplicated method

L195 overwrite Fluentd::Log#time_format= which is defined by attr_accessor.
This has been introduced since https://github.com/fluent/fluentd/commit/80e45ea2db99b2e3624e8ea9dde6b1707162416e

Which issue(s) this PR fixes:
nope
What this PR does / why we need it:
To fix the warning from ruby such as warning: method redefined; discarding old time_format=.
Docs Changes:
No need.
Release Note:
No need.

Okay. Thanks!


Add TVL metric to the mixers.

Not sure how to feel about adding metadata to the storage in general. This can be done easily with num_leaves * deposit_amount off-chain. Wdyt?

Well that would only track the total amount of deposits but not the current
TVL after withdrawals. This is useful for showing the user their effective
privacy. I think it’s good to have because I’m the future we’ll need TVL
tracking to handle voting.
On Tue, Feb 9, 2021 at 9:02 AM Filip Lazovic<EMAIL_ADDRESS>wrote:

Not sure how to feel about adding metadata to the storage in general. This
can be done easily with num_leaves * deposit_amount off-chain. Wdyt?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/edgeware-builders/anon/pull/74#issuecomment-775959782,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADELLF45P4Q52SNJ57VICHLS6E57RANCNFSM4XJ2AEHA
.


Update Contract Test Generator.postman_collection.json

The server to test was being selected based on it's url rather than it's description (description is what the docs tell you to put in the env-server variable).

Thank you for this @tomredfernswift!!


Update lodash reference in vite optimizeDeps demo configs

Hey folks!
I ran into a bit of trouble during setup with a new codebase, using powersync-sdk-web, bundled with vite.  I modeled my vite config after the demo config here. Upon page load, the following error was displayed in the browser console:
Uncaught SyntaxError: The requested module '.../lodash/throttle.js' does not provide an export named 'default' (at AbstractStreamingSyncImplementation.js)
I was able to resolve the issue by changing the vite config lodash depdency line to lodash/throttle.  I suspect this may've been a regression with the recent lodash optimization.  I'm new to vite, not sure if this is the best solution, but it worked for me, and I hope it'll help others hitting the same wall I ran into.
Appreciate the great work you're doing here!

Thanks for this. There has been another reported issue with a project setup where someone had to includes the nested dependency. When I tested this on my side in a standalone repo I was only able to get this working with the nested definition.
Could I ask that you could test the following in your project? Just want to make sure it doesn't break anything:
  include: [
      "event-iterator",
      "uuid",
      "js-logger",
      "lodash/throttle",
      "can-ndjson-stream",
      "@journeyapps/powersync-sdk-web > event-iterator",
      "@journeyapps/powersync-sdk-web > uuid",
      "@journeyapps/powersync-sdk-web > js-logger",
      "@journeyapps/powersync-sdk-web > lodash/throttle",
      "@journeyapps/powersync-sdk-web > can-ndjson-stream",
    ],
    ```

@Chriztiaan thanks for the quick reply!  Yeah, that config worked for me -- didn't break anything.
I wasn't familiar with the nested dependency syntax until you shared it; my intuition is that it's probably the better approach.
Out of curiosity, I tried with nested dependency declarations only.  It almost worked, just had to change the uuid path:
    include: [
      '@journeyapps/powersync-sdk-common > uuid',
      '@journeyapps/powersync-sdk-web > event-iterator',
      '@journeyapps/powersync-sdk-web > js-logger',
      '@journeyapps/powersync-sdk-web > lodash/throttle',
      '@journeyapps/powersync-sdk-web > can-ndjson-stream'
    ]

Thanks for checking! I'll fiddle a bit and test it on my side.

I've updated the diff to use the nested dependency syntax. Thanks for the help.

Updated this after our package renaming @journeyapps/powersync-xxx -> @powersync/xxx.
Verified that this works without needing a pnpm overrides entry for uuid after bumping all the underlying UUID versions in https://github.com/powersync-ja/powersync-js/pull/130


HTML 5 video blocked?

As far as I can tell all the HTML5 player settings are on but I'm getting a infinite loading circle loop on some sites - can someone tell me which settings could possibly be related to this?

URL?

Seems to be these "Brightcove" players specifically
https://support.brightcove.com/brightcove-player-sample-are-you-still-watching

Do you still need an answer? Did you tried on some other browser?

 Does it play?

Once you've established that it's a user.js pref that causes the issue, then there are more steps you can take to troubleshoot it

first of all, make sure you are not blocking anything: e.g brightcove etc in uBlock Origin, or Firefox's Tracking Protection, etc

What do you see when you go to https://www.youtube.com/html5 - I get seven out of seven ticks
Anyway
* Create a new temp profile and use that, will it play (if you don't want to mess around with profiles, just download a portable Firefox from https://portableapps.com/apps/internet/firefox_portable )

* Now add the user.js and restart. Does it play?

Once you've established that it's a user.js pref that causes the issue, then there are more steps you can take to troubleshoot it - just keep using the portable FF or test profile

I see 7 ticks too yeah. I've turned off all the adblocking stuff and the vids still aren't loading which leads me to believe it's user.js related ...

Also seeing AbortError: The fetching process for the media resource was aborted by the user agent at the user's request. :S

CORS and Brightcove

CORS and Brightcove

Are you saying the solution is in here or it's a bug in Firefox itself?

It's brightcove multiplexing to some CDN. It means that you click on a link and your request goes to some 3rd party server that may - or may not - respect the CORS headers.

It's brightcove multiplexing to some CDN. It means that you click on a link and your request goes to some 3rd party server that may - or may not - respect the CORS headers.

Tbh I can't even get the player to load in a clean, blank FF profile - maybe I've broken something on a deeper level. Works in Chrome tho ...

I can confirm ungoogled chromium work fine. FF gives no video, if I click on the codepen link I get a cloudflare page with captchas (choose all the bikes).

I can confirm ungoogled chromium works fine. FF gives no video, if I click on the codepen link I get a cloudflare page with captchas (choose all the bikes).

Reported to Bugzilla guys.

Is there a bugzilla number?

https://bugzilla.mozilla.org/show_bug.cgi?id=1545823

Thanks. Since it's not a user.js issue, it's reproducible on nightly by someone else, I'll close this


Source is unavailable

Hello,
I have set up the server correctly. The respective streamers are also displayed to me, but the stream is not loaded. Is this a known problem?
Edit: Okay, it's because I have transcoding disabled. Is there a way to view the feed without transcoding?

Hey @revunix, glad you figured out the issue! I would guess that this depends on the codec of the stream and the capabilities of the client you're using. For example, even with transcoding enabled, I could never get this to work using Plex in the browser. As a result, unfortunately, I think you'll be pretty limited here. By disabling transcoding, you're essentially saying that your client must have the ability to play the stream exactly as it comes from Twitch, which may not be possible. 😞


Can anyone reproduce the result for MNIST --> MNIST-M

What I did is to simply set up my own directory and set momentum = 0, weight_decay = 0 (It is missing from https://github.com/wogong/pytorch-dann/blob/master/experiments/mnist_mnistm.py) However, even after inspecting previous epochs, the best result I could get is around 74%, which is far from the 84% reported in the table. I am wondering if anyone could reproduce this number?

I realize that I need to use the network with dropout and batch normalization, specifically, use MNISTmodel instead of MNISTmodel_plain. Indeed, the performance of this implementation for mnist --> mnist-m is much better than the original one in the DANN paper.


Simplified route to internal enterprise deployment

Feature Request
Description
There should be minimal overhead for an enterprise wishing to deploy this to their internal users. Currently there's only a local deployment model and a cloud-hosted version optimised for SL's use case.
Additional context
Ideally this could be made as simple as build this single container from source/download this image (depending on security preferences of enterprise), and run it with these instructions (expose port, provide API secret, etc.).
Acceptance criteria
GIVEN an enterprise is keen to deploy this for their internal users
WHEN they look for installation instructions
THEN they find instructions which are as simple as possible with minimal assumptions about their infrastructure

It's trivial to allow express to serve our UI, while still allowing the UI to be served separately if need be. We can provide both a single "npm start" style command from the project root, and a dockerfile to run in a container, so that the app can be built and started

directly on the local machine
in docker locally
remotely from a single container
remotely as separate UI and (scalable) API modules.

This should keep everyone happy.
I will re-work the authentication side of things in a subsequent ticket, as I would really like to move the (opt-in) auth integration from the UI code into the cloud infra.


Error compiling libvpx module on Windows with YASM

PS D:\git\godot> scons p=windows bits=64 target=release_debug tools=yes -j 1
scons: Reading SConscript files ...
Configuring for Windows: target=release_debug, bits=64
 Found MSVC version 14.1, arch amd64, bits=64
Checking for C header file mntent.h... (cached) no
scons: done reading SConscript files.
scons: Building targets ...
[ 43%] yasm -Ithirdparty/libvpx/ -f win64 -D X86_64 -o thirdparty\libvpx\vp8\common\x86\copy_sse2.windows.opt.tools.64.obj thirdparty\libvpx\vp8\common\x86\copy_sse2.asm
scons: *** [thirdparty\libvpx\vp8\common\x86\copy_sse2.windows.opt.tools.64.obj] Error 309
scons: building terminated because of errors.
PS D:\git\godot>

It appears that you are missing DLLs.

But on Linux it works fine, even with MinGW.

Maybe there's a regression in how SCons uses yasm to build some files in Godot. Try removing yasm so that SCons doesn't attempt to use it.

Maybe there's a regression in how SCons uses yasm to build some libvpx-related files in Godot. Try removing yasm from your PATH so that SCons doesn't attempt to use it.

That worked. Thank you :+1:

What's your compiler version, and yasm version?

Assigning to @hpvb who got YASM + MinGW to work for official builds.

@LinuxUserGD what version of yasm do you have?

@hpvb yasm-1.3.0

I can't test if the error is still there because now I'm getting:
scons: *** SDK version '10.0A' is not supported
File "./platform/windows\detect.py", line 146, in setup_msvc_auto

scons: *** SDK version '10.0A' is not supported

That's #26485.
How did you install yasm?

@akien-mga I installed yasm and scons via scoop and Visual Studio Community together with Unreal Engine (I have Unity, Unreal and Godot installed and Unreal needs C++).

With 10.0A and yasm it works now.
[100%] progress_finish(["progress_finish"], [])
[100%] Linking Static Library ==> modules\freetype\freetype_builtin.windows.opt.tools.64.lib
[100%] Linking Static Library ==> drivers\drivers.windows.opt.tools.64.lib
audio_driver_alsa.windows.opt.tools.64.obj : warning LNK4221: Die Objektdatei definiert keine zuvor undefinierten öffentlichen Symbole. Sie wird daher von keinem Verknüpfungsvorgang verwendet, der diese Bibliothek benutzt.
thread_posix.windows.opt.tools.64.obj : warning LNK4221: Die Objektdatei definiert keine zuvor undefinierten öffentlichen Symbole. Sie wird daher von keinem Verknüpfungsvorgang verwendet, der diese Bibliothek benutzt.
[100%] Linking Static Library ==> core\core.windows.opt.tools.64.lib
[100%] Linking Program        ==> bin\godot.windows.opt.tools.64.exe
   Bibliothek "bin\godot.windows.opt.tools.64.lib" und Objekt "bin\godot.windows.opt.tools.64.exp" werden erstellt.
[100%] scons: done building targets.
PS D:\git\godot>

I stumbled upon this same "Error 309" issue recently. For those wondering, the yasm Windows binary requires Microsoft Visual C++ Redistributable 2010 to run, but the yasm package provided by Scoop doesn't install it automatically. This causes the binary to fail to run until the dependency is installed.

I also encountered this error after following this doc page:
https://docs.godotengine.org/en/latest/development/compiling/compiling_for_windows.html
Because it says to install yasm with scoop. By running scoop uninstall yasm everything compiled correctly. Maybe this dependency should be removed from the docs?

@creikey If everything works correctly, yasm should be able to compile the required assembly code. If it doesn't work, it means there's a bug in our code.
That said, we should really move video codecs out of core at some point since they've been causing so much trouble…


Enable tile + distribute of tensor.insert_slice.

This patch adds the external model for implementing TiledOpInterface
into IREE during initialization so that it is always available. This
allows tile + distribute of this operation at the flow level.
To get the distribution to work correctly, the arguments for the
destination and result in the dispatch region op need to be tied. The
current logic for legalization of dispatch region and tie-ing operands
does not support this. So this PR also reworks all of that to be more
robust.

Issue #5045


Add copyright headers to source files

What this PR does / why we need it:
Adds copyright headers to Go source files and enforces a make validate-headers target in CI.
Better late than never. All credit goes to @CecileRobertMichon. See also Azure/aks-engine#23.
Release note:
Add copyright headers to source files

can we rename validate-headers to validate-license-headers ? I think headers is too generic and not necessarily suggestive of what it's actually doing.

lgtm otherwise :)

/lgtm


Add option to install using jsdelivr CDN to directly import to your project

I added option to import from a free and fast CDN which will improve import speeds & save bandwidth.

LGTM 👍


Replace np.bool with bool

np.bool was deprecated as of NumPy 1.20 (see https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations) and the deprecation has expired in NumPy 1.24 (https://numpy.org/doc/stable/release/1.24.0-notes.html#expired-deprecations). To fix, I replaced 'np.bool' with the identical name 'bool'

Indeed, thanks for the suggestion !


Defining dependencies with arguments

This PR is a draft to have a discussion if this is a thing we want to add at all and how.
This PR adds support for dependencies with arguments. I am looking forward for any feedback and thoughts about this.
Usage
// Defining a resolvable
factory { LocalStorage(name: "default") as LocalStorageProtocol }
factory { name in LocalStorage(name: name) as LocalStorageProtocol }
factory { (args: (name: String, other: Int)) in LocalStorage(name: args.name) as LocalStorageProtocol }

// Resolving a depdendency
@Inject var defaultLocalStorage: LocalStorageProtocol
@Inject("With just one parameter") var localStorageWithJustOneParameter: LocalStorageProtocol
@Inject((name: "Local Storage with multiple parameters", other: 1)) var localStorageWithMultipleParameters: LocalStorageProtocol

let localStorage: LocalStorageProtocol = resolve("The name of the storage")

func doSomething(_ storage: LocalStorageProtocol = resolve("storage name")) { }

Implementation details

Only factories can have an argument
One cannot define a resolvable with a tag and arguments
It is possible to define multiple resolvables of the same type with different arguments
resolvables can only have a single argument. Multiple arguments can be resolved using tuples
The type of the resolvable at definition time must match exactly the one at resolution time. E.g. String != String?or (String, Int) != (name: String, age: Int).

Todos

[ ] Add documentation in code
[ ] Add documentation in the Readme

Hey @benjohnde, is this something interesting for DIKit?


Update repo docs to latest info

1.2 feature list
ProductBoard --> WinAppSDK roadmap (still to be updated with 1.3/1.4/2023 plans)
WinUI 2.8 & WinUI 2 support

/azp run


[System Monitor] ...

Extension
https://www.raycast.com/hossammourad/raycast-system-monitor
Description
There should be another option at the bottom to open Activity Monitor in case people want to get more info.
Who will benefit from this feature?
People who see the basic info inside this app but want to see more info (a list of running processes, etc).
Anything else?
No response

Opened #6871

@pernielsentikaer this issue should be closed, no?

Correct 😊


retain_ip should be possible for load balancers as well

Rancher versions:
rancher/server: 1.6.2
rancher/agent: 1.2.2
Infrastructure Stack versions:
healthcheck: 0.3.1
ipsec: 0.11.2
network-services: manager 0.7.1, metadata 0.9.1
scheduler: 0.8.0
Environment Template: Cattle
Steps to Reproduce:
For regular services, we can choose that the IP address is not changed during e.g. an upgrade (retain_ip: true). We would also like to have that feature in Rancher load balancers as well.

Btw, it is possible while creating a stack by including retain_ip and even io.rancher.container.requested_ip in the YML files. So my suggestion would just be to add this feature to the UI.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


"Open recent file" displays new, unsaved diagrams as empty entries

Describe the bug

Steps to reproduce

Create new file
Close it without saving
Select "Open recent file" in the menu

https://github.com/camunda/camunda-modeler/assets/28307541/7af1e111-f856-4d58-9986-53833c72c2ff
Expected behavior
Either we don't show it or we display a placeholder.
Environment

OS: [e.g. MacOS 10.2, Windows 10]
Camunda Modeler Version: 5.20.0
Execution Platform: [e.g. Camunda 7, Camunda 8]
Installed plug-ins: [...]

Additional context
No response

Closed via https://github.com/camunda/camunda-modeler/issues/4115


Update policy status placement

ref: https://github.com/stolostron/hub-of-hubs-status-sync/pull/8
Signed-off-by: clyang82<EMAIL_ADDRESS>

Thanks @nirrozenbaum for your reviewing. You will be added as owner once this PR is merged.
for the case you mentioned that, if does not add  the predicate to spec2db controllers, all the policy/placementrule/placementbinding will be updated with the finalizer which is not correct. the policy (local-resource) cannot be deleted successfully.

why isn't the finalizer removed upon delete request?
this is what written in code and this is how it worked until now..
https://github.com/stolostron/hub-of-hubs-manager/blob/main/pkg/specsyncer/spec2db/controller/generic_spec_to_db_reconciler.go#L89


Introduce high level API

A first attempt at a high-level C++ API. I used the raw pointer for "internal" to hide the jit-engine headers from any clients consuming the API.

This is now ready for review. It cleans up the existing C++ code and starts with a C++ example showing standalone querying of data in Arrow format.


Launcher log table shows empty fourth column

How to reproduce:

Start the launcher from most recent develop branch
Select the "Log" tab
See that the fourth (right-most) column of the table is empty

@praj-foss did you add the fourth column on purpose? I think it was only three columns before 🤔
If not, let's just remove the fourth column.

I hadn't noticed it before, but yep, that column is just unnecessary. So it's safe to remove it from the FXML. :+1:

Superseeded by #472


Undeclared dependency “undici”

Hi,
i’m using Yarn pnp and after updating the 3.3.0 today O got an error that it could not resolve the “undici” package.
checking the source here it is imported in src/shim/correlationContextManager but is not defined in the package.json file and thus leading to yarn pnp disallowing our build to happen as we don’t allow undefined dependencies in our build.

@CarbonCollins thanks for letting us know, most likely this was added by mistake by some IntelliSense.


issue189-new reupdate

PR Title
The purpose of this Pull Request is to fix #189
Description

Stars and Forks of the repo will be shown below the navbar

How you solved

Use github's Octokit/rest package to get repo details
and swr package to update the details without refreshing
One has to assign access token in .env as GITHUB_AUTH_TOKEN=
For star and fork icon use react-icon package

Screenshots

Checklist

[x] I have Made this contribution as per the CONTRIBUTING guide in this repo
[x] I have tested in local Environment.
[x] I have made the fix as per issue converstaion
[x] I have starred the repository ⭐

@harsh0620 Thank you,
Resolved the conflicts and merged your changes, working fine in production


Added non-minified production file

Nic -
Had to create this fork with a non-minified combined file to work nicely with https://github.com/ck86/main-bower-files
Just adds a concat task to the grunt file.

Hi @akshayrangnekar,
A few things:

Your PR failed to pass my Travis CI tests. Please correct them if you want me to merge.
I'm not sure I understand, so I just want to clarify.  You want a version of the file that is non-minified to compliment the minified version in the dist directory.  Correct?

Thanks,

Apologies - had a typo (extra comma) in the package.json. Fixed 1.
You're right - I want a version of the file that is non-minified. Your bower.json refers to this file already (see "main" element), it just doesn't exist in the repo. This is confusing "main-bower-files" which is used to automatically inject bower dependencies into a HTML file.

Thanks for contributing!


Remove wrapped error message

This error message shouldn't be used anywhere because we are
protecting from going over the buffer size elsewhere in the codebase.
It's also an abstraction leak across the packages.

👍


CAMEL-12677 Create FHIR spring boot quickstart

https://issues.apache.org/jira/browse/CAMEL-12677

Merged on master. Thanks @rkorytkowski


Group name fix

fix for http://manage.dimagi.com/default.asp?217534
undoes https://github.com/dimagi/commcare-odk/pull/1080 since this was more appropriately fixed at the QuestionView level

Release Note: Fix issue where group labels would still appear when empty


healthcheck

Is this library abandoned?

Please, be patient. Mass rewrite/update in progress. Will make big PR in a day or two.


el-get can also specify git revisions

el-get allows to specify a git revision, a branch or a tag. For example:
(setq el-get-sources
      '((:name org-mode
               :checkout "release_9.1.13")
        (:name cider
               :checkout "v0.17.0")
        (:name clojure-mode
               :checkout "5.8.1")
        (:name bbdb
               :checkout "c951e15cd01d")
        (:name magit
               :checkout "2.13.0")))

Note I may have misunderstood the point. As a long time el-get user, other points seem valid to me.

It seems that what I wrote in the README was entirely correct: in el-get, you have to place the commit hash into the recipe, which disables updates.
Perhaps what was confusing was how the functionality of straight.el differs? The difference is that straight.el natively supports this specific use case, and allows you to automatically write the revisions of all packages to an external lockfile with no changes to your init-file (and then load those revisions next time packages are installed on a different machine).
The reason that this difference is important is that it means that with straight.el you can get perfect reproducibility just by writing a lockfile, whereas with el-get it seems that you have to manually record a recipe for every package so you can write down the revisions, including dependencies. If this is erroneous, please correct me—I never used el-get.
No objections to making the text more clear, but removing that bullet point doesn't seem quite right to me.

Actually, never mind. I see that the things I said above are already covered in the third bullet point, so the first bullet point is just redundant and confusing.
Thanks for your contribution.


SWITCHYARD-2814 sendFault() shouldn't be used on InOnly

No code change. the faultType is always null for IN_ONLY operation, therefore any Throwable is wrapped with HandlerException on IN_ONLY. Described it in a comment in SwitchYardConsumer and added some testcases for those fault behavior.

Can one of the admins verify this patch?

ok to test

Merged by Jenkins.


RipGrep keeps running on background (WSL Ubuntu)

Every time I run :Rg on vim a process is created, although I close the window (and even the vim) the created process remains there.

If keeping the grepper running improves its performance, I would like to ask only that the processes do not pile up on my CPU.
Because it is a WSL related problem, the solution may not be simple. I will wait patiently.

Problem solved on WSL2.


version of font awesome

is there a way to tell what version of font awesome is being used? or be able to upgrade it somehow?

Never mind, I see you can specify in bower


Adds error plot and colourbars to the animation. Start of a loss plot

Modulus Pull Request
Description
Aims to improve the inference code with an error plot and colourbars.
There is a start of the code to plot the loss over the rollout. Needs cleaning up and generalising.
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[ ] The documentation is up to date with these changes.
[x] The CHANGELOG.md is
up to date with these changes.

Dependencies
Depends on PR #111

The resolution also needs updating as the gifs are quite large now.

Example of the new visualisation

And an example of the rollout loss plot

The resolution also needs updating as the gifs are quite large now.

This was fixed in the last commit

The examples don’t appear to have tests, do I need to add some?
For documentation launch only has an ApI doc, is there any documentation for me to update for this change?

Hi @cfd1 , thanks for the PR! Is this ready for review?

The examples don’t appear to have tests, do I need to add some? For documentation launch only has an ApI doc, is there any documentation for me to update for this change?

No you don't need to add tests for the examples. For documentation, take a look at the README file for the vortex shedding example and see if anything needs to be updated. Thanks

@cfd1 thanks again for the PR! It looks good to me, I just left a single comment for you about the resolution.

/blossom-ci


Black screen bug on windows 10 (v3.2)

Godot version:
3.2
OS/device including version:
Windown 10 (64 Bits)
Graphics:
Intel(R) HD Graphics 4000
Issue description:
I've just tried to test a games start screen using the start button.
The screen got black, my mouse pounter disappeared and I wasn't able to hit keys like Ctrl + Tab, Windows-Logo + D etc. but the power lamp was on.
I've colosed my laptop to turn the operating system into the sleeping mode but as I woke it up the Screen was still black.
I ned to hold the start button to power the laptop down and after I had rebooted it everything went right...
UPDATE:
I used the Full screen mode
No idea how to reporducre it
If anyone here knows it, please write a comment.

No idea how to reporducre it

Please report bugs only if you know a definite way to reproduce them. Otherwise, it's not very useful as nobody will be able to help.

Hello, Calinou!
I know:

That I don't know very much
My graphics card model is Intel(R) HD Graphics 4000 (I've updated the issue too)
I have already written it - and that's wat Godot writes too: Godot Engine v3.2.stable.official - https://godotengine.org
I was using full screen mode (I've added this as well)

Why I'm writing this

If anyone else has the problem too he can write it here too and we know more (and then, my report becomes very useful because we konw this is not a problem of my hardware
Maybe someone gets an idea what had been happend if I report this (I think it was worth a try)

And: Why in the world are you closing this? Its not fixed I think.

And: Why in the world are you closing this? Its not fixed I think.

This issue is still open. I didn't close it.

Okay. Let's see if the bug appears again.
p.S: A automatic update function would be nice:wink:

p.S: A automatic update function would be nicewink

There are no plans to add automatic updates, as Godot updates can break compatibility. You don't want the engine to update itself while you're working on releasing a project.

The bug is still in the current version but now I know more.
I've made a mistake ina piece of code ang Godot found it. The function with the mistake was called in _ready().
Godot found that bug butthis time, the problem stopped the program before it was hiding my mouse pointer. This time I was able to log out (but not do actions with my keyboard) and log in again. I saw that the debug window was still running and Godot displayed the problem. I had saved dictonarys in an arry and called them wrong. This was a new piece of code so I hadn't written it as I was opening this Issue but the problem with the black screen seems to go together with this.

If an error is found in the user code, the editor window should pop up (or the game will crash silently if the editor is not open).
Are you saying the editor window did not pop up for you?

Yes, no window did pop up. The Editor marked a line and wrote the error below it.

I'll close this as nobody else has been able to reproduce this recently. Please comment if you can still reproduce this issue.


Question:  Unable to click the link

Can you please advise how to click the link in the web table, tried the following code. and it return an error.. Attached is the table screenshot and error log. Both the way are returning the same error.
proxy.click: Target closed
=========================== logs ===========================
waiting for selector "query-by-text=[2023]"
selector resolved to visible 2023
attempting click action
waiting for element to be visible, enabled and stable
element is visible, enabled and stable
scrolling into view if needed
done scrolling

Vacation Bid & Award from … subtree intercepts pointer events
retrying click action, attempt #1
Screenshot is atatched.

  const cell = await screen.queryByTestId("paged-table-IFVacationAuctionSummaries").within().getAllByRole('link');
  const noOfCells = await screen.queryByTestId("paged-table-IFVacationAuctionSummaries").within().queryAllByRole('link').count();
        console.log("year value" + await cell.nth(0).textContent() )

        for (let j = 0; j < noOfCells; j++) {
            console.log(" value1 " + await cell.nth(j).textContent())
        if ((await cell.nth(j).textContent()) === '2023') {.
            console.log(" value 2" + await cell.nth(j).textContent())
            await cell.nth(j).click();
        }
    }

 await screen.queryByText(new Date().getFullYear() + 1).click();  ---> it worked once and but not consistent, screenshot is attached. trying to click 2023 in the table displayed in the screenshot.

Can you please advise why? and please suggest me if there is a better solution , Thank you!

It looks like an element is obstructing the element you're attempting to interact with (
). It's possible that the sidebar element is invisible but still overlapping with the element you're attempting to .click(). This is more of a test flakiness/debugging issue or potentially an issue with Playwright, not Playwright Testing Library.

I'm going to close this for now, but let me know if you want to continue this over in Discussions-land.


ExpandableCardView does not support RTL

Hi @AleSpero
Thanks for beautiful custom card view
I use it in rtl direction and that does not display like ltr direction
Fix it, Please
Thanks a lot.

Hello mhosein,
Thank you for your kind words
Thank you for the feedback. I will fix the issue in a couple of days.
Regards,
Alessandro
Il ven 17 ago 2018, 18:11 mhosein<EMAIL_ADDRESS>ha scritto:

Hi @AleSpero https://github.com/AleSpero
Thanks for beautiful custom card view
I use it in rtl direction and that does not display like ltr direction
Fix it, Please
Thanks a lot.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AleSpero/ExpandableCardView/issues/18, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AK2pKYBXnlahYiknwcOQDrNDyckG-DsEks5uRutGgaJpZM4WB0tg
.

This will be great. Ty for your support


Update contact_diary_strings.xml - The curent string is "Vorname, Nachname". This is misleading in german.

The curent string is "Vorname, Nachname". This is misleading in german.
Option 1: "Vorname Name"
Option 2: "Name, Vorname" Visitenkarten ...

Checklist
Thank you for this this PR! Please consider the following:

To the Community :heart::

Please link to an issue ticket where your code change has been greenlit, otherwise it's unlikely it can be merged.
Use a descriptive title: {task_name} (closes #{issue_number}), e.g.: Use logger (closes # 41).
If this PR comes from a fork, please Allow edits from maintainers
The PR can not contain changes in localization files, e.g. values-[LANGUAGE-CODE]/strings/* and / or assets/* (see #332 for further information).

To the Maintainers :coffee::

Title structure: Jira ticket name (EXPOSUREAPP-XXXX).
Set labels: maintainers, sprintX.
Update Jira status: In Review.
Fulfill internal Acceptance Criteria.
If mentioned in the Jira ticket link all corresponding GitHub issues in the sidebar.

To everyone :world_map::

Describe your changes in detail, if you changed the UI, screenshots or GIFs would be awesome!
Short step by step instructions help the reviewer test your changes, e.g. how to navigate to a new UI element you added.
The PR won't be reviewed if CircleCi is failing or if there are merge conflicts. If Circle CI is still failing mark the PR as a draft and write a little comment on your status.
Provide at least a few unit and/or instrumentation tests.
Use a meaning full branch name. Use either fix or feature as prefix for your branch, e.g. fix/prevent-npe-on-device-rotation-issue_123
Test your changes thoroughly. Only open PRs which you think is ready to be merged. If you explicitly need feedback mark the PR as DRAFT on Github.
Don't introduce unrelated code reformatting (e.g., on-save hooks in your IDE)
Remove this checklist before creating your pull request.

Description

Steps to reproduce

Thanks for your PR unfortunately we can not accept direct PRs that change translations:

The PR can not contain changes in localization files, e.g. values-[LANGUAGE-CODE]/strings/* and / or assets/* (see #332 for further information).

If you think these strings should be changed, please create an issue ticket to discuss it.


[auth-swift] Restore SPM

Now that the Auth implementation is all Swift, restore SPM build.  Note that Package.swift should be compared against the master version.

Separate remaining ObjC unit tests to another directory so they're not built for SPM
Cannot use an imported protocol via @_implementationOnly
Disable heartbeat logging to restore build for SPM
Remove GoogleUtilities imports from AuthDefaultUIDelegate.swift pending a solution for detecting running in an extension for Swift
Continue disabling keychain dependent unit tests for SPM.
Restore SPM CI
The combine test failures and updated integration will be addressed in a subsequent PR

Protocols don't work from  implementation only imports:


[Help] Insomnia has become unresponsive

I'm stuck in this loop. As soon as I open Insomnia it dies on me.

All I did was hit "Send" and re response started loading a bigger (few MB) binary file. I'm assuming Insomnia is trying to load the response into to the UI and just dies doing this.
Should there be a circuit breaker for this? If response is binary or too large, snip it.
Also how do I get unstuck and break this loop ?

Hi Oliver,
You should be able to delete the large response from %APPDATA%/Insomnia/responses and restart the app.

Thank you that worked.

@gschier Hi, I just deleted the latest timeline and response files and Insomnia no longer locks up. Thank you very much.
The files where not extremely huge ( some MB ), but did contain "weird" / control characters.

I'm also having this same issue when the response contains a control character or binary encodings. Clearing the folder works, but I would still like to parse the response.


Onboarding screen keeps loading

It should only load once, not every time the app loads.

If it is to load just once, these stats might get outdated next time the app loads. Would you still like me to go ahead with this?

Also if I’ve chosen to explore anonymously I’d like to see the issues list open each time I open the app again.

Saving the state of guest login would require completely different work, opening a new issue for it!


Is latestdata.csv incomplete/outdated?

First of all, great work and thank you for providing this data!
I'm doing a simple aggregation on province, city and noticing the number isn't right for NYC.
(using python and pandas)
import pandas as pd
df = pd.read_csv('latestdata.csv')[['province', 'city']]
df = df.groupby(['province', 'city']).count().reset_index()
df[df['city'] == 'New York City']

Aggregated count in NYC comes down to 2469, whereas today cases are reported to be 20K+
On https://www.healthmap.org/covid-19/ which references your data, they show 17K which is closer to the reported numbers (at the time I'm writing the latestdata.csv has been updated 16 hours ago so their gap sense). Although I don't know if their website is augmented by another data source.
Also, the latestdata.csv  has a total of 117K rows, whereas reported cases in the world as per healthmap.org is 500K+.
Is something wrong with the way I'm looking at the data or could it be incomplete?

There is something wrong with the dataset, The entire dataset has 117k records, there are 550k cases, if you select on country = 'United States' it's only got 11,364 records and there are 86,242 cases in the USA so far. I hope this can be fixed, I really need the USA stats for my company. We're using this data to determine which offices to close, we are an essential business.

Thanks for flagging - we're looking into this. There is one processing step that might have led to some cases not being present when using those filters. I will caution however that this dataset is intended to be as comprehensive as is feasible for our team, with as much specific metadata available, rather than be aligned one-to-one with the total global cases. I would suggest looking at the Johns Hopkins Github https://github.com/CSSEGISandData/COVID-19 for a system that is engineered to track cumulative counts, without the metadata of age, sex, outbreak time milestones, or more specific geography. We will retain as much as is feasible for us to do so.

@beoutbreakprepared I'm fine with the entire dataset, we have global offices as well, just concentrating on USA for now. But having said that, the entire dataset only has 117k records, it should have well over 550k records and should be 600k+ after today.

If it helps diagnose the issue, the state of NY in the United States seems to have no cases confirmed for certain dates, while in other areas their reporting of cases seems to be pretty consistent day-to-day.

is it just my copy of the 'latestdata.csv' that only has 436,549 records of the 2,479,498 cases? Is there another section? ... the latest copy is just a fifth of the cases

Is there anymore data beyond june 16?

@calremmel is looking into why the data isn't up to date.

Hi all, update on this: the line list data source that feeds latestdata.csv is not currently being updated, so the current file is as up to date as it is going to be until that changes.
My understanding is that we'll be migrating to some other sources soon. I don't know what that will mean for accessing this particular file where it is currently, but for the time being, the most recent lines are from mid-June.

Bummer - thanks for the update!
Get Outlook for iOShttps://aka.ms/o0ukef

From: calremmel<EMAIL_ADDRESS>Sent: Friday, July 31, 2020 5:22:30 PM
To: beoutbreakprepared/nCoV2019<EMAIL_ADDRESS>Cc: tnjcook<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [beoutbreakprepared/nCoV2019] Is latestdata.csv incomplete/outdated? (#44)
Hi all, update on this: the line list data source that feeds latestdata.csv is not currently being updated, so the current file is as up to date as it is going to be until that changes.
My understanding is that we'll be migrating to some other sources soon. I don't know what that will mean for accessing this particular file where it is currently, but for the time being, the most recent lines are from mid-June.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/beoutbreakprepared/nCoV2019/issues/44#issuecomment-667177681, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APDIRWHUG2ARXCQHOSFZRQTR6LOTNANCNFSM4LUSNFIQ.


Feature: Added a custom value along with rules

@Ajaykkumar Hi! I'm looking to close this PR out unless there're more code to come. Thanks


Inconsistency in ng new

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Area
- [ ] devkit
- [ ] schematics

Versions
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 6.0.7
Node: 9.10.1
OS: linux x64
Angular: 6.0.3
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.6.7
@angular-devkit/build-angular     0.6.7
@angular-devkit/build-optimizer   0.6.7
@angular-devkit/core              0.6.7
@angular-devkit/schematics        0.6.7
@angular/cli                      6.0.7
@ngtools/webpack                  6.0.7
@schematics/angular               0.6.7
@schematics/update                0.6.7
rxjs                              6.2.0
typescript                        2.7.2
webpack                           4.8.3

Repro steps
ng new project
1st. case: src/tslint.json flle is indented by 4 spaces, which is not consistent with all other generated files.
2nd. case: in package.json:
"devDependencies": {
    "@angular/compiler-cli": "^6.0.3",
    "@angular-devkit/build-angular": "~0.6.5",
    "typescript": "~2.7.2", // here
    "@angular/cli": "~6.0.5"
}

typescript isn't sorted alphabetically.

Thanks, but I don't think it's important enough for us to fix


Proto comments in reflection

What do you think about adding code comments to reflection message for documentation purpose?
Then it will be possible to build documentation like this https://github.com/pseudomuto/protoc-gen-doc dynamically from server, and combine with UI like this https://github.com/gusaul/grpcox#demo
It can be like openapi for grpc
May be can make comments more structured, something like javadoc, can separate request example and response examples

It's a pity that this got auto-closed, seems like a really good suggestion to me.

Yep, in contradiction with GraphQL or OpenAPI where documentation plays a significant role, GRPC looks like don't bother about documentation so much.

I'm looking for this now. Our source code repos are unfortunately not available to our API consumers, and the proto comments provide crucial documentation. Was really hoping that reflection could be used to expose these comments.


Assures ansibledevel jobs run all the time

Decision to skip running ansibledevel bought us some speed boost at
the cost of no longer testing code and introducing costly bugs.
This caused failure to release 2.22rc4 which failed on ansibledevel
jobs, which were not run during normal PRs.
From now on we need to be sure that we test all PRs with the same set of
jobs as we do for releases.

No. Because we already save in the 2.8 -> 2.9 release cycle of Ansible that we had to wait 1 - 2 months for a fix to come down the line so that we could fix the devel build support here. Furthermore, when we have a working working group, we made a decision that it was untenable to support Ansible devel.

This caused failure to release 2.22rc4 which failed on ansibledevel jobs, which were not run during normal PRs.

The root cause of this problem is not solved with this change. That is tracked in https://github.com/ansible/molecule/issues/2204.

Neither the commit nor the description here on github mention that in addition to enabling the ansible devel builds on all jobs, it also no longer allows failures on those ansible devel jobs. Running the ansible devel jobs makes sense to me, but blocking PRs on their failures does not, for the main reason outlined in @decentral1se's comment:
Failures in molecule's ansible devel builds are not always going to be caused by bugs in molecule, so molecule PRs can not reasonably be blocked on the results of ansible devel build results.

After additional talks I agree that running on devel may put too much strain on Travis.


Add loading spinner to datagrid

Pull request type
This pull request adds a loading spinner to the DataGrid.

New feature (non-breaking change which adds functionality)

Description
When using the DataGrid in situations where it takes a bit of time to load data, it would be nice to show a loading spinner to the user. This PR adds this to the datagrid.
When it comes to the styling of the loading spinner, I did not implement this in a generic way, meaning I am using colors that make sense for my project. You definitely want to change that to make it more generic for the broader audience.
I just thought I'd share this because it might just make the Mendix team gain some time if they want to add a loading spinner to the datagrid.

Pull request implementation does not match anymore with changed that have been done in the upstream code base


Edit and/or regenerate persona attributes

Add functionality to edit persona attributes. For some attributes, random re-generation may be useful.

For example the attribute const hobbiesList = Generator.generateHobbies(); that generator method could be used, as it has no dependencies currently. This would be very good to allow customization of the personas, thanks alot


if -init-centerline is mask, can cause problems

also doesn't work if mask is binary, linear or spline-interpolated.
data:
sct_example_data/mt

see below result:
Using mask:
sct_register_multimodal -i ../t2/t2_seg.nii.gz -d mt1.nii.gz -p step=1,iter=0 -x nn
sct_propseg -i mt1.nii.gz -t t2 -init-centerline t2_seg_reg.nii.gz

Using centerline from mask:
sct_register_multimodal -i ../t2/t2_seg.nii.gz -d mt1.nii.gz -p step=1,iter=0 -x nn
sct_process_segmentation -i t2_seg_reg.nii.gz -p centerline -o t2_seg_reg_centerline.nii.gz
sct_propseg -i mt1.nii.gz -t t2 -init-centerline t2_seg_reg_centerline.nii.gz

To do: in the wrapper of sct_propseg.py, transform the mask provided into a centerline.

Issue is reopened due to issue #673.
After fixing this issue, please add a test for propseg with centerline.


[SPARK-30306][CORE][PYTHON][WIP] Instrument Python UDF execution time and throughput metrics using Spark Metrics system

What changes were proposed in this pull request?
This proposes to extend Spark instrumentation to add metrics aimed at drilling down on the performance of Python code called by Spark: via UDF, Pandas UDF or with MapPartittions. Relevant performance counters, notably exuction time, are exposed using the Spark Metrics System (based on the Dropwizard library).
Why are the changes needed?
This allows to easily consume the metrics produced by executors, for example using a performance dashboard (this references to previous work as discucssed in https://db-blog.web.cern.ch/blog/luca-canali/2019-02-performance-dashboard-apache-spark ).
See also the screenshot that compares the existing state (no Python UDF time instrumentation) to the proposed new functionality
Does this PR introduce any user-facing change?
This PR adds the PythonMetrics source to the Spark Metrics system. The list of the implemented metrics has been added to the Monitoring documentation.
How was this patch tested?
Added relevant tests

manually tested end-to-end on a YARN cluster and using an existing Spark dashboard extended with the metrics proposed here.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc: @HyukjinKwon @viirya @ueshin

Thanks @HyukjinKwon for taking time for this.
This functionality propoesed in ths PR is different from Python profiler, or at least its intended use is. Also it is intended to be lightweight so that it can be used for measuring Spark workloads in production as part of a performance dashboard based on the metrics coming from the Spark metrics system.
I'd like to add some additional context on how we intend to use this.

I have described how we implement and use a Spark performance dashboard based on the metrics system in a recent Spark Summit presentation https://databricks.com/session_eu19/performance-troubleshooting-using-apache-spark-metrics and in a blog entry http://db-blog.web.cern.ch/blog/luca-canali/2019-02-performance-dashboard-apache-spark
Over the last couple of years I have helped improving this by adding to the Spark metrics system instrumentation for aggregated task metrics values per executor [SPARK-25228], JVM CPU usage [SPARK-22190], memory usage [SPARK-27189]. This still does not cover all the space of possible activities and "time usage" by Spark tasks and executors, in particular the problem that I am trying to solve with this PR is that time spent waiting for results to come back from Python workers (typically when executing a UDF) is currently not instrumented, so it appears in the dashboard as run time "without attribution" in the current dashboard, while it can be visualized using the metrics implemented here (see image )

As you mentioned, in the current PR I have implemented several details, which I guess can be useful when troubleshooting, but for the general use can be simplified as you propose, in particular in the number and detials exposed in the user-visible metrics in the PythonMetrics source.

This has now been streamlined down to the following metrics (under namespace=PythonMetrics):

BytesReceivedFromWorkers
BytesSentToWorkers
FetchResultsTimeFromWorkers
NumBatchesFromWorkers
NumBatchesToWorkers
PandasUDFReceivedNumRows
PandasUDFSentNumRows
WriteTimeToWorkers

Test build #118782 has started for PR 26953 at commit fd76071.

Merged build finished. Test PASSed.

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

Test build #118782 has finished for PR 26953 at commit fd76071.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

retest this please

Test build #118929 has started for PR 26953 at commit fd76071.

Merged build finished. Test PASSed.

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

Test build #118929 has finished for PR 26953 at commit fd76071.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #119372 has started for PR 26953 at commit 989dba1.

Merged build finished. Test PASSed.

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

Test build #119372 has finished for PR 26953 at commit 989dba1.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

It looks like a case of flaky test. Can we test this again please?

Test build #121328 has started for PR 26953 at commit 812ea5e.

Merged build finished. Test PASSed.

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

Test build #121328 has finished for PR 26953 at commit 812ea5e.

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

Merged build finished. Test PASSed.

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

I would not worry very much about the performance impact of this additional instrumentation, as it hooks on something that is not very fast already, that is the serialization/deserialization JVM-Python. Moreover, the instrumentation mostly just takes timing values and does so per batch of serialized rows, so the impach on the total throughput is expected to be further reduced by this. So far, I have only tested this manually and did not observe any particular impact. If we have a Python UDF benchmark I could further test with that.

Test build #123188 has started for PR 26953 at commit a39ad63.

Merged build finished. Test PASSed.

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

Test build #123188 has finished for PR 26953 at commit a39ad63.

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

Merged build finished. Test PASSed.

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

Test build #126228 has started for PR 26953 at commit 277245c.

Merged build finished. Test PASSed.

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

Test build #126228 has finished for PR 26953 at commit 277245c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #128116 has started for PR 26953 at commit 2fd3f1b.

Merged build finished. Test PASSed.

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

Test build #128116 has finished for PR 26953 at commit 2fd3f1b.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #128117 has started for PR 26953 at commit c391be0.

Merged build finished. Test PASSed.

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

Test build #128117 has finished for PR 26953 at commit c391be0.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #130035 has started for PR 26953 at commit 337b67c.

Test build #130035 has finished for PR 26953 at commit 337b67c.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34642/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34642/

Merged build finished. Test PASSed.

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

Test build #130043 has started for PR 26953 at commit c7570bd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34650/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34650/

Merged build finished. Test FAILed.

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

Test build #130045 has started for PR 26953 at commit 4b88804.

Test build #130045 has finished for PR 26953 at commit 4b88804.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #130046 has started for PR 26953 at commit 02a0ca0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34654/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34653/

Test build #130043 has finished for PR 26953 at commit c7570bd.

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

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

Merged build finished. Test PASSed.

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34653/

Merged build finished. Test FAILed.

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

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34654/

Merged build finished. Test FAILed.

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

Test build #130046 has finished for PR 26953 at commit 02a0ca0.

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

Merged build finished. Test PASSed.

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

Test build #131055 has started for PR 26953 at commit 750b973.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35660/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/35660/

Merged build finished. Test PASSed.

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

Test build #131055 has finished for PR 26953 at commit 750b973.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

I am closing this as the implementation should be refactored, what looks like a better way to implement this is to first work on the correctponding SQL metrics as in SPARK-34265 and then revisit the instrumentation for the Spark Metrics System.


Port to Eio and Cohttp

This is a messy draft port of Dream to Eio using cohttp-eio. On HTTP/1, it works better than the previous port attempts (#254, #194) and should be more maintainable, including because cohttp-eio offers a direct-style interface (the advantages of which this port still needs to propagate further into the Dream code), but a lot of features will be missing even after the port is complete -- HTTP/2 support, GraphQL, and others, because the corresponding libraries are not ported to Eio or not available with Cohttp. A lot of features, like Caqti integration, that will be available when this port is done, are temporarily disabled for now, and will be added back as the port progresses. WebSockets will require additional work.
The current status of the port is that I am working through various deadlocks and corner cases of using Eio "promises" together with fibers, which are caused by the existing code of Dream assuming a sort of promise-callback soup, where any callback can be called on the one big stack under Lwt.main and make progress, while with fibers and Eio blocking the wrong fiber to wait on a promise can block a stack that would have been used to make progress to resolve that promise. The semantics are therefore substantially different. This was a major problem with the approach in #194 and #254. It is substantially negated by thoroughly using direct style wherever possible, rather than promises or callbacks.

6.0 just shipped! https://github.com/mirage/ocaml-cohttp/releases/tag/v6.0.0

Trying to see if I can get this branch to run. There's been some bitrot, especially the Mirage folks have gotten rid of cstruct and moved to string/bytes. Hacking and slashing now...