Tensorboard

This PR setup DeLFT to use tensorboard: logs, callbacks...
The idea is to cover the interesting metrics so that everything is visualized on Tensorboard.
How to use: basically nothing special, start a training and launch:
> tensorboard --logdir logs/summaries

(tensorboard is already installed)
Then open http://localhost:6006/

TODO:

add more metrics
see non-scalar views

Tensorboard generates a ridiculous amount of event logs... more than 1.2GB of logs per epoch only for a basic transformer model. For instance after 4 epochs:
$ du -sh logs/summaries/20220502-160051/train
5.1G	logs/summaries/20220502-160051/train

Given that effect (like with a 10-fold cross-validation training), the tensorflow callbacks need to be defined at the application level, so that it is a user choice and not systematically used by the core library.

Looks good.
Indeed, this tool can fill up the disk space faster than docker... 🎉

FYI 1 run of 10-fold crossvalidation weight ... 82GB 😂
(base) [lfoppian0@sakura02 summaries]$ ls -lh
total 4.0K
drwxr-sr-x 4 lfoppian0 tdm 4.0K May 10 12:17 20220510-121634
(base) [lfoppian0@sakura02 summaries]$ du -hs
82G	.


[Functions] Added support for to create function apps for hidden runtimes

cc @gzuber @pragnagopa
Please add support do create a function app from a hidden runtime defined in the Stacks API.
For the Az.Functions cmdlets, I am using a environment variable $env:displayHiddenRuntimes. If this variable is set to $true, the creation of a function app from a hidden runtime is enabled.

Functions


Admin account for live demo

could you please provide a Admin account in the demo site ? so i can had a quick look ? i created the new account but the account is normal usuer so i guess i do not had permision to view to the Admin UI. Thanks

Hi @CalvinPangch
With normal user you can see every details, but you cant change it.
Admin Ui site:
https://jpproject.azurewebsites.net/admin-ui/


Supply absolute recording URL.

Right now, it looks like if you supply Camera.constants.CaptureTarget.disk or Camera.constants.CaptureTarget.temp as the captureTarget, it will create a URL to save the video. It would be great, assuming this would work, if the absolute file path could be generated beforehand and supplied as an argument. This way while the video is recording to disk it can also be read and uploaded to a server.

I have a pull request with this feature for iOS and it appears to be working with another thread reading the file while it's recording. Android still needs to be implemented. https://github.com/lwansbrough/react-native-camera/pull/378


When to synchronize to the latest version of libmediasoupClient?

When to synchronize to the latest version of libmediasoupClient?

done. close.


Cherry-pick GHA changes into release-1.2

Motivation
This pick includes the following 6 PRs on GHA workflows

#8133: switch GHA to use env secrets
#8153: fix typo in env secret
#8171: disable debug log in LBT during PR landing
#8168: add LBT kill switch
#8190: rename LBT kill switch and rework its workflow

Test Plan
Canary + CI
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

/canary

/canary

/land


Make it harder to delete batteries + experiments.

Maybe "deletion" will be more of an archiving. Or at the very least will result in an automatic file based dump of relevant database fields for posterity.

Batteries have an inactive status now. Setting this value and then hiding the batteries and their related objects still needs to be implemented.


Implement Open Event, with on_open and on_close

Proposed changes

Needs an on_open and on_close callback.
Context

NA
Possible Implementation

NA
Other information

NA

Released: https://github.com/deepgram/deepgram-python-sdk/releases/tag/v3.1.8


Remove jQuery dependency

It seems very wasteful to depend on jQuery for such simple DOM manipulation. I'll take a look at removing it and submit a patch.

What happended on this one? Any progress? This is a very nice feature i would like to add to a project, without jQuery tho :)


Dev updates

Summary of changes:
Make readme pypi friendly
Update requirements
Demo: running grlc from python
Add support for pyql projections
Update to work with python3
ConfigParser Py2 and Py3
Fix tests for python3
Fix pythonql requirements for py2 and py3
Add test for projections
mv notebook to own dir
Check for license files for LicenseURLs
Add error for non-existing routes. Closes #37
Remove swagger-ui from static files
Process number parameters properly
@albertmeronyo -- if everything is working on dev.grlc.io at the moment, should merge to master and make an end-of-year release?

Just checked the live instance at dev.grlc.io with the default API at https://github.com/albertmeronyo/lodapi and everything works like a charm. Thanks @c-martinez !!!!
(Actually: I think that's a pretty nice test suite, the API in that repo; maybe we could automate its generation and checking that executing the calls returns results for all the queries as a nice test suite?)

Yes, I think we can automate testing on this API, as long as we are happy that it tests all the relevant functionality.


Free server from ASP.Net Framework stuff

Waiting for example for ASP .Net Framework :)

We have now removed .NET Framework Classic support for version 11 so I will close this issue.


Please provide feedback on a breaking change proposal.

I'm considering changing ebay_rest.json as follows with the intention of:

reducing the number of parameters needed when instantiating an API object
providing the means to specify scope when required
supporting use cases that have multiple application, user or header sets
make terminology more intuitive
make using .json optional, the alternative being to supply similar dicts when instantiating an API object
make anticipated changes, like adding a new header or scope element, backwards compatible

{
    "***_instructions_***": {
        "01": "The ebay_rest module needs your eBay credentials.",
        "02": "Copy this file to your working folder and name it ebay_rest.json.",
        "03": "As you create and get, replace the placeholders in the two sections that follow.",
        "04": "Create a developer account - https://developer.ebay.com/api-docs/static/creating-edp-account.html",
        "05": "Get your OAuth credentials - https://developer.ebay.com/api-docs/static/oauth-credentials.html",
        "06": "Get your redirect_uri value - https://developer.ebay.com/api-docs/static/oauth-redirect-uri.html",
        "07": "Create production and sandbox logins at your eBay country-specific marketplace, e.g. https://www.ebay.com & https://sandbox.ebay.com.",
        "08": "Except for production users, empty scopes will be auto-discovered.",
        "09": "If the token and token_expiry are empty, they will be auto-generated.",
        "10": "If using version control, keep your keys private by NOT checking in the file ebay_rest.json."
    },
    "applications": {
        "production_1": {
            "app_id": "example-example-PRD-example-example",
            "cert_id": "PRD-example-example-example-example-example",
            "dev_id": "example-example-example-example-example",
            "redirect_uri": "example-example-example-example",
            "scopes": [],
            "token": "",
            "token_expiry": ""
        },
        "sandbox_1": {
            "app_id": "example-example-SBX-example-example",
            "cert_id": "SBX-example-example-example-example-example",
            "dev_id": "example-example-example-example-example",
            "redirect_uri": "example-example-example-example",
            "scopes": [],
            "token": "",
            "token_expiry": ""
        }
    },
    "users": {
        "production_1": {
            "site": "https://www.ebay.com",
            "email_or_username": "<production-username>",
            "password": "<production-user-password>",
            "scopes": [
                "https://api.ebay.com/oauth/api_scope",
                "https://api.ebay.com/oauth/api_scope/sell.inventory",
                "https://api.ebay.com/oauth/api_scope/sell.marketing",
                "https://api.ebay.com/oauth/api_scope/sell.account",
                "https://api.ebay.com/oauth/api_scope/sell.fulfillment"
            ],
            "token": "",
            "token_expiry": ""
        },
        "sandbox_1": {
            "site": "https://sandbox.ebay.com",
            "email_or_username": "<sandbox-username>",
            "password": "<sandbox-user-password>",
            "scopes": [],
            "token": "",
            "token_expiry": ""
        }
    },
    "headers": {
        "ENCA": {
            "content_language": "en-US",
            "country" : "CA",
            "currency": "CAD",
            "marketplace_id": "EBAY_ENCA",
            "zip": "K1M 1M4"
        },
        "GB": {
            "content_language": "en-US",
            "country" : "GB",
            "currency": "GBP",
            "marketplace_id": "EBAY_GB",
            "zip": "SW1A 1AA"
        },
        "US": {
            "content_language": "en-US",
            "country": "US",
            "currency": "USD",
            "marketplace_id": "EBAY_US",
            "zip": "20500"
        }
    }
}

# API object creation parameters
class API:
    def __init__(self, application=None, user=None, header=None, throttle: bool = False, timeout: float = -1.0):

# when ebay_rest.json contains only one application, user and header record
api_one = API()

# when ebay_rest.json contains more than one application, user and header record
api_two = API(application='sandbox_1', user='sandbox_1', header='US')

# when a user credentials come from a database, and other credentials are loaded from ebay_rest.json
# pretend that the dict() contents are loaded from a database.
user = dict(site="https://www.ebay.com", email_or_username="<production-username>",
            password="<production-user-password>", scopes=[
                "https://api.ebay.com/oauth/api_scope",
                "https://api.ebay.com/oauth/api_scope/sell.inventory",
                "https://api.ebay.com/oauth/api_scope/sell.marketing",
                "https://api.ebay.com/oauth/api_scope/sell.account",
                "https://api.ebay.com/oauth/api_scope/sell.fulfillment"
            ], token="", token_expiry="")
api_three = API(application='sandbox_1', user=user, header='US')

Some use cases do NOT need a user or header, so these will be optional.
For developers that want to supply tokens, for the sake of consistency, should how that is done be reconsidered?

Hi,
I don't use ebay_rest.json at all although I use the equivalent in ebaysdk for the TradingAPI. But all this seems reasonable?
You could add fields to give a user refresh token and expiry as an alternative to giving username/password (they last for 18 months) and then that provides an equivalent when passing details directly on API initialization.

Could also be worth being able to supply a path to ebay_rest.json somewhere?

Your idea of providing either a user login or a token makes good sense because both are never needed.
Since the library can do an excellent job of getting and refreshing the application token, I suppose there is no point in making it possible to pass it in.
Having a sandbox True/False parameter makes it very clear whether sandbox or production is used. On the flip side, if production/sandbox can be inferred, having one less parameter to worry about makes things easier for the programmer. Undecided.
Yes, I suppose that a developer with a suite of programs might want them all to use the same ebay_rest.json file. For sure, I'll add a parameter for an optional path. I'm undecided if the file name should always be ebay_rest.json to help people find documentation about it or benefit from letting the programmer rename it.
I've got a prototype coded and will tweak it based on your helpful feedback. To make merging easier, I'll wait until the paging method changes finish.

A tweaked version of the proposal is complete and will be known as Beta version 8.
The changes will benefit someone:

with multiple 'users'
not wanting to use ebay_rest.json for credentials
that must supply a 'user token'

To upgrade, in a terminal run
pip install ebay_rest -U

How an API object is instantiated has changed, so that:

record names in ebay_rest.json can be specified
dicts mimicking ebay_rest.json records can be supplied, making ebay_rest.json unnecessary
if there is only one set of records in ebay_rest.json that they will be automatically used
or a combination of all three

The sandbox parameter is no longer required; it is inferred from the credentials supplied. Here is some code for those who miss the sandbox parameter.
# Environment
SANDBOX = False  # change to True for PRODUCTION
if SANDBOX:
    APPLICATION = 'sandbox_1'
    USER = 'sandbox_1'
else:
    APPLICATION = 'production_1'
    USER = 'production_1'
HEADER = 'ENCA'

api = API(application=APPLICATION, user=USER, header=HEADER, throttle=False)
The API method that loaded a user token is no longer necessary as the token now can be supplied during API instantiation.
Search for 'API(' in your python code and alter per the example in the ReadMe.md and read API's init docstring for more options.
Modify your ebay_rest.json file per this.
Unfortunately, the enhancements triggered a new issue.


Make sure WAYF visibility is reflected correctly

When the Show unconnected IdP's in WAYF flag is enabled the unconnected
should be displayed in the WAYF and not be filtered. In this commit the
findIdentityProviders method for the DoctrineMetadataRepository makes
sure it's results are visited by the Visitor implementation for IdPs.
Main question is: does this change break other behaviour for the other methods that call and use this method. My gut feel tells me this change would only affect parts of the application that use the IdentityProvider::$isEnabledInWayf property, so this change is quite safe..

Although the change itself looks good, I am wondering about it solving the problem. In the EB source code I see findIdentityProviders() not used in a relevant code path? Or is it?

The CompositeMetadatRepository calls the findIdentityProviders method on the DoctrineMetadataRepository. EB calls this method indirectly from this method: EngineBlock_Corto_Module_Service_SingleSignOn::_transformIdpsForWayf.


localization: add i18n editor

Hi together, let me know what I need to change in order to merge this

Description
Added a localization editor

Continuous i18n process improvements
Improved translation status

Editor -> Fink Translation Editor (Open Editor)

All opt-in

I didn't change you setup, everybody can use what they want (opt-in)
It's only a config

Changelog

Added project.inlang/settings.json file to configure the inlang project.
repo id file was generated by the SDK.
I updated the translation section in the CONTRIBUTING.md file.

Checklist

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] The light- and dark-theme are both supported and tested
[ ] The design was implemented and is responsive for all devices and screen sizes
[ ] The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

Do you know why the ci is broken atm?

Do you know why the ci is broken atm?

Thanks for the PR! It's a great idea, we'll have a look at it.
The ci is broken for PRs of external github users, because the automatic deployment to the dev cluster fails. Not sure right now if it's something we can or will fix, but it's not your code's fault :)

Thank you for your contribution! While we appreciate your suggestion, we currently do not have plans to integrate a translation editor into Scrumlr. But we will keep it in mind for future updates.


Stuck in signing injectable.dylib

The command bash bfinject -P App -L decrypt/test produces the following log and gets stuck in signing.
Electra detected.
[+] Injecting into '/var/containers/Bundle/Application/XXXXXX-XXXX-XXXX-XXXX-XXXXXXX/XXXX.app/XXXX'
[+] Getting Team ID from target application...
[+] Thinning dylib into non-fat arm64 image
[+] Signing injectable .dylib with Team ID XXXXXXXXX and platform entitlements...

What is your iOS version? Try deleting jtool.liberios from /electra/usr/local/bin then extract bfinject somewhere else and try again

Thank you for your response. Unfortunately, still facing the same issue after deleting jtool.liberios.
I'm running iOS 11.0.1

I remember having an issue like this with jtool but I don’t remember the cause.
Try this: open bfinject with an editor, goto lines 222 and 232 and remove “> /dev/null 2>&1” at the end then send the log again

Thanks a ton for the quick turnaround. Good news, I'm past Signing phase. Bad news, I'm now stuck in Marking Platform.
Please find the details:
Test-iPhone-X:/bfinject/ root# bash bfinject -P App -L decrypt
[+] Electra detected.
[+] Injecting into '/var/containers/Bundle/Application/XXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXX/App.app/App'
[+] Getting Team ID from target application...
[+] Thinning dylib into non-fat arm64 image
[+] Signing injectable .dylib with Team ID XXXXXXXX and platform entitlements...
Marking Platform
Marking Platform


Update index.js

Updates schema query to get foreign tables too. Use case is using steampipe(.io) in service mode, all tables are foreign tables.

Thanks @ted46045!


When retryPolicyBranchRollbackOnConflict set to true, the TC spawn logs frequently

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

Ⅰ. Issue Description
When the TC survivable self-test switch is turned on, the TC will simulate the global transaction opening and submission of each client in a self-test cycle, resulting in a large number of simulated transaction logs being frequently flushed by the TC side, resulting in the TC side log being too large, and affecting the normal troubleshooting by viewing the TC log.
Ⅱ. Describe what happened
A large number of simulated transaction logs appear on the TC side
If there is an exception, please attach the exception trace:
Just paste your stack trace here!
```15:09:25.752  INFO --- [verHandlerThread_1_41_500] i.s.s.coordinator.DefaultCoordinator     : Begin new global transaction applicationId: shop-order-application,transactionServiceGroup: local_group, transactionName: degradeCheck,timeout:60000,xid:<IP_ADDRESS>:18091:92637771075686400
15:09:25.757  INFO --- [     batchLoggerPrint_1_1] i.s.c.r.p.server.BatchLogHandler         : SeataMergeMessage xid=<IP_ADDRESS>:18091:92637771075686400,extraData=null
,clientIp:<IP_ADDRESS>,vgroup:local_group
15:09:26.698  INFO --- [      AsyncCommitting_1_1] io.seata.server.coordinator.DefaultCore  : Committing global transaction is successfully done, xid = <IP_ADDRESS>:18091:92637771075686400.
15:10:03.774  INFO --- [     batchLoggerPrint_1_1] i.s.c.r.p.server.BatchLogHandler         : SeataMergeMessage timeout=60000,transactionName=createByAT()

### Ⅲ. Describe what you expected to happen
![image](https://user-images.githubusercontent.com/45869678/104433019-8f3ca780-55c4-11eb-92a3-bcc8db7ade4b.png)

Hopefully the TC will not spawn these logs
### Ⅳ. How to reproduce it (as minimally and precisely as possible)
1. Retrypolicybranchrollbackonconflict is set to true
2. Observe TC log

### Ⅴ. Anything else we need to know?
It is recommended to set the mock log level to DEBUG

### Ⅵ. Environment:

- JDK version :1.8
- OS :CentOS
- Others:

@lj2018110133

@lj2018110133

please give me the task

please give me the task


Production database not protected by seeds.ts

Non issue because there is a production specific github action for that deploy


flutter run is error!

I tried to initialize the project is correct,
I use the vscode editor,
but, i Excuting an order flutter run -v,
prompt : Connection timed out !
I tried running it a dozen times, still the same error, which made me unable to understand how to use flutter!
@ianloic @paulcbetts @sgraham @sethladd @kevmoo

Please add the output of flutter doctor -v.

How did you create the project?
Does this also happen if you do
flutter create my_project
cd my_project
flutter run

in a terminal window?

I have found the reason for this problem, because the network problem causes the dependency to be downloaded. thanks

Thanks for the update!
Sounds like this can be closed then.


Update default ZK path for jobmaster

IMO, "/job_election" and "/job_leader" should not be created in the root directory. So I updated default ZK path for jobmaster, and added prefix "/alluxio".

Codecov Report

Merging #13944 (3c75f22) into master (80cf7ca) will decrease coverage by 25.90%.
The diff coverage is 20.00%.

@@              Coverage Diff              @@
##             master   #13944       +/-   ##
=============================================
- Coverage     43.25%   17.34%   -25.91%
+ Complexity     9129     2675     -6454
=============================================
  Files          1411     1411
  Lines         81725    81733        +8
  Branches       9902     9902
=============================================
- Hits          35348    14177    -21171
- Misses        43418    66302    +22884
+ Partials       2959     1254     -1705

Impacted Files
Coverage Δ

.../cli/fs/command/AbstractDistributedJobCommand.java
0.00% <0.00%> (ø)

...a/alluxio/cli/fs/command/DistributedLoadUtils.java
0.00% <0.00%> (ø)

...common/src/main/java/alluxio/conf/PropertyKey.java
97.87% <100.00%> (-1.58%)
:arrow_down:

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 663 more

Continue to review full report at Codecov.

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

Hi @maobaolong @ggezer ,  these failed checks seem unrelated to the change. Could you please help review the code. Thanks a lot.

LGTM, thanks for fixing this issue!

Thanks @yuzhu for your review.

@ggezer is there back compat concerns here? maybe when we do a rolling upgrade between a version before this change and a version after this change?

@tomscut Alluxio masters also operate under /alluxio/ path. Having job_master into same path will introduce races. Hence elections will disturb each other.
@yuzhu your backcompat concerns are correct. We can't make rolling upgrade with this change.

@tomscut Alluxio masters also operate under /alluxio/ path. Having job_master into same path will introduce races. Hence elections will disturb each other.
@yuzhu your backcompat concerns are correct. We can't make rolling upgrade with this change.

Thanks @ggezer for your comment. However, job_master and master information are stored in different subdirectories, why should there be competition?
I'm worried that if we put job_master in the root directory, it might conflict with the pathnames of other services. It also doesn't look very elegant.

@tomscut I initially thought there could be race when creating the paths the first time for both master and job_master concurrently. However, zk should take care of that.
My only concern is the breaking change to the rolling upgrade. On a second thought, there shouldn't be an issue as long as clients are also restarted. But this will definitely need to be known in advance. I don't know when do we want to merge this change. @apc999 ?

@tomscut I initially thought there could be race when creating the paths the first time for both master and job_master concurrently. However, zk should take care of that.
My only concern is the breaking change to the rolling upgrade. On a second thought, there shouldn't be an issue as long as clients are also restarted. But this will definitely need to be known in advance. I don't know when do we want to merge this change. @apc999 ?

Thank you @ggezer for your reply.

@tomscut I initially thought there could be race when creating the paths the first time for both master and job_master concurrently. However, zk should take care of that.
My only concern is the breaking change to the rolling upgrade. On a second thought, there shouldn't be an issue as long as clients are also restarted. But this will definitely need to be known in advance. I don't know when do we want to merge this change. @apc999 ?

Hi @apc999 , could you please help take a quick look at this. Thank you.

@ggezer @apc999 are the back compat concerns addressed? do we want to delay this until later release?

I am good with this PR. Alluxio HA with ZK is not the way the community is focusing to support so I wouldn't concern on rolling upgrade


Transferring tab history?

Google's tab sync feature remembers each tab's history, which is a convenient thing to have. This tool doesn't seem to do this. Is there any way to add this functionality to it?

I don't think this is possible. The CDP HTTP endpoints do not seem to provide such extended information, though it might be available through some web-socket interaction.
Either way, this would be something beyond the targeted functionality of this tool, which is to be able to transfer the currently opened tabs from an android phone to the pc and back.

I see. Is there any way to at least identify the tabs that have a history and close the ones that had been opened directly (without any history), and sync the remaining tabs using Google's own syncing?

As said, to my knowledge there is no way to access the history directly. At least, there is no documented way of reading it programmatically.
I guess that's a design decision, probably for security/privacy reasons.
Well, at least for mobile browsers, over CDP with USB debugging.
It seems possible though, to use puppeteer (a framework chrome automation and testing) in conjunct with the functinality provided to browser extensions through the history API.
I haven't tested this, but in theory, one should be able to inject extension code via the puppeteer API calls into the browser tabs. Some examples of what the history extension API can do, can be found here. The starting point would probably be the Page API to call the extension code and the BrowserContext API.
But keep in mind, it will only be possible to do these things on desktop browsers - iirc the mobile version of Chrome does not support extensions.
Let me know if you are able to find out more about it.


enhancement: media_player media title improvement

In cases where chapterTitle and trackTitle are not the same, adjust "media_title" to be "chapterTitle - trackTitle"
and add both chapterTitle and mediaTitle to the media_player attributes
2024-05-19 13:39:16.309 DEBUG (Thread-2 (_thread_main)) [yoto_api.YotoMQTTClient] yoto_api - MQTT Message: {"trackLength":989,"position":383,"cardId":"ecnOg","repeatAll":false,"source":"remote","cardUpdatedAt":"2022-01-11T13:07:31.455Z","chapterTitle":"Chapter 1","chapterKey":"01","trackTitle":"Show and Tell","trackKey":"01","streaming":false,"volume":7,"volumeMax":16,"playbackStatus":"playing","playbackWait":false,"sleepTimerActive":false,"eventUtc":1716118754}

Just as an example are you thinking "Card - Chapter" when they don't match?

Is this what you had in mind?
https://github.com/cdnninja/yoto_ha/pull/52

perfect.

perfect.


ci: elasticsearch: Upload script improvements

Multiple improvements of the upload_test_results_es.py script:

JSON objects flattening.
This feature allows twister.json file preprocessing to simplify its Elasticsearch index structure for complex hierarhical objects, for example with memory footprint, or code coverage data.
A new command line option --flatten is added to change testsuite data structure in regard of one of its list objects: either testcases or recording, so each item there becomes an independent data record inheriting all other testsuite properties, whereas the children object's properties are renamed with the parent object's name as a prefix: 'testcases_' or 'recording_' respectively. Only one testsuite property can be flattened this way per index upload. Other children objects will be treated accorging to the index structure.
Related new command line options (with help text explanations): --flatten-dict-name, --flatten-list-names, --flatten-separator, --transpose-separator, --escape-separator

A new command line option --transform is added to allow regexp group parsing in string propertites extracting additional derived properties.

A new command line option --exclude is added to exclude testsuite properties not needed to store at Elasticsearch index.

Branch name --run-branch and Workflow ID --run-workflow command line options as additional key fields to allow data from different branches, workflows and triggering events in the same index.

A new command line option --map-file is added to apply an explicit index structure to the twister.json input data.

Other changes:

batch upload error handling and logging;
inline documentation improvements;
some corner case fixes on empty objects.

Previously mentioned in relation to memory footprint data collection.

You are adding lots of new functionality that is not being used anywhere. I would rather see this come along side whoever is going to use the new options, I guess this would be the footprint data.

I've put into PR's description some examples for how this extended script is used for timer accuracy data, kernel benchmarks and footprint data.


Active dropdown always showing.

I've got an issue related to a recent question but using FP5. I have an issue where in a section that navigation is marked as active, a user cannot navigate up and down through the page via the submenu without it being stuck open all the time and won't auto-close. If you hover over the other menu items, those will open and close as expected but the active section has an issue.
I tried using jquery to automatically show and hide the menu but for some reason the solution doesn't work in Chrome on Windows (works fine on a Mac) and FF both Mac and PC. IE the solution works fine but I've switched out so many different functions trying to get it to work (toggle, addClass, toggleClass, etc.) and all work just the same, the menu hides and then shows again as it should on the browsers that it works in and then fails to reappear in the ones that are giving me an issue.
This is one version of the jquery function that works:
$(document).ready(function() {
$('.top-bar-section li.active:not(.has-form) a:not(.button)').click(function() {
$("li.active ul.sub-menu.dropdown").css("display", "none");
});
$('.top-bar-section li.active:not(.has-form) a:not(.button)').hover(function() {
if($("li.active ul.sub-menu.dropdown").css("display", "none")){
$("li.active ul.sub-menu.dropdown").css("display", "block");
}
});
})
And this is a page from the site to illustrate: http://development.maclynutility.com/littlefriends/childrens-services
If you navigate through that section of the site the active menu will stick in the above browsers, I need it to autoclose and am running out of options on how to do that. I apologize if this might not be the correct way to go about this but I'm in the 11th hour on this and it's got me up against a wall. I do appreciate any help anyone can provide, thank you very much.

FP5 is no longer supported.


Esprima 2.x import/export statements

Resolves some of the issues raised in #228

Support for ExportDefaultDeclaration and ExportNamedDeclaration
As per Espree spec https://github.com/estree/estree/blob/master/es6.md#modulespecifier the ImportSpecifier and ExportSpecifier should expect local instead of an id

LGTM

@Constellation - thanks for merging. Would it be possible to do a release with this and #252 any time soon? I'm keen to get another PR in for istanbul that depends on this change.


Cannot find source file: src/exception.h

I run cmake .

-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.8.4
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
CMake Warning (dev) at CMakeLists.txt:52 (set):
  Cannot set "AMQP-CPP_INCLUDE_PATH": current scope has no parent.
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Configuring done
CMake Error at CMakeLists.txt:36 (add_library):
  Cannot find source file:

    src/exception.h

  Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm .hpp
  .hxx .in .txx

-- Build files have been written to: /home/ubuntu/workspace/AMQP-CPP

I can confirm this issue.

@RafalGoslawski I'm seeing this while attempting to build the 2.7.0 upgrade for Homebrew in https://github.com/Homebrew/homebrew-core/pull/13000
The following patch seems to help:
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index d52cbd9..3d01c83 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -46,7 +46,7 @@ deferredcancel.cpp
 deferredconsumer.cpp
 deferredconsumerbase.cpp
 deferredget.cpp
-exception.h
+../include/exception.h
 exchangebindframe.h
 exchangebindokframe.h
 exchangedeclareframe.h
@@ -59,14 +59,13 @@ exchangeunbindokframe.h
 extframe.h
 field.cpp
 flags.cpp
-frame.h
+../include/frame.h
 framecheck.h
 headerframe.h
 heartbeatframe.h
 includes.h
-messageimpl.h
 methodframe.h
-protocolexception.h
+../include/protocolexception.h
 protocolheaderframe.h
 queuebindframe.h
 queuebindokframe.h

In addition to that I need to cp include/endian.h src/ and then it builds.

@RafalGoslawski endian.h is still causing
[ 21%] Building CXX object CMakeFiles/amqp-cpp.dir/src/connectionimpl.cpp.o
/usr/local/Homebrew/Library/Homebrew/shims/super/clang++  -Damqp_cpp_EXPORTS -I/tmp/amqp-cpp-20170504-94143-xbosvw/AMQP-CPP-2.7.1  -DNDEBUG -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.12.sdk -mmacosx-version-min=10.11 -fPIC   -std=c++11 -o CMakeFiles/amqp-cpp.dir/src/connectionimpl.cpp.o -c /tmp/amqp-cpp-20170504-94143-xbosvw/AMQP-CPP-2.7.1/src/connectionimpl.cpp
In file included from /tmp/amqp-cpp-20170504-94143-xbosvw/AMQP-CPP-2.7.1/src/connectionimpl.cpp:13:
/tmp/amqp-cpp-20170504-94143-xbosvw/AMQP-CPP-2.7.1/src/passthroughbuffer.h:20:10: fatal error: 'endian.h' file not found
#include "endian.h"
         ^
1 error generated.
make[2]: *** [CMakeFiles/amqp-cpp.dir/src/connectionimpl.cpp.o] Error 1
make[1]: *** [CMakeFiles/amqp-cpp.dir/all] Error 2
make: *** [all] Error 2

@RafalGoslawski it looks like that include statement just needs to be removed from passthroughbuffer.h


Unable to install Falco by applying DaemonSet

Describe the bug
Issue with installing Falco using DaemonSetCondition manifests
How to reproduce it
kubectl apply -f k8s-with-rbac/falco-daemonset-configmap.yaml
Expected behaviour
kubectl apply daemonset config should work.
Screenshots
kubectl apply -f k8s-with-rbac/falco-daemonset-configmap.yaml
error: unable to recognize "k8s-with-rbac/falco-daemonset-configmap.yaml": no matches for kind "DaemonSet" in version "extensions/v1beta"
After changing it to v1/apps:
kubectl apply -f k8s-with-rbac/falco-daemonset-configmap.yaml
error: error validating "k8s-with-rbac/falco-daemonset-configmap.yaml": error validating data: ValidationError(DaemonSet.spec.selector): unknown field "app" in io.k8s.apimachinery.pkg.apis.meta.v1.LabelSelector; if you choose to ignore these errors, turn validation off with --validate=false
Environment

Falco version:
git clone https://github.com/falcosecurity/falco/
System info:

Cloud provider or hardware configuration:
OS: ubuntu
NAME="Ubuntu"
VERSION="16.04.6 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.6 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial
Kernel:

Linux frivol1.fyre.ibm.com 4.4.0-166-generic #195-Ubuntu SMP Tue Oct 1 09:35:25 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Installation method:
Kubernetes --> 1.17.1

The same for me running Kubernetes 1.16.2 on minikube 1.5.2. The issue I think is for deprecations  introduces with the 1.16

I had this issue as well, you can get past the error by changing the YAML before deploying:
apiVersion: apps/v1
kind: DaemonSet
metadata:
name: falco-daemonset
labels:
app: falco-example
role: security
spec:
selector:
matchLabels:
app: falco-example
This works on the newest kind that is running K8s 1.17

Hi @robotica72 you can follow my PR https://github.com/falcosecurity/falco/pull/1044

@maxgio92 - Cool, will do - Thanks for the link

@rashmi43: There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Is there a way to access the pre-uploaded file?

I need both the response from the server, as well as the pre-uploaded file. Is there a way to access it?

What do you mean? Create some code samples.

Also since it is a question post it on SO.


Build failure due to new syntax

I have added the Alamofire .swift classes under the Source folder to my project and it seems the language was updated. The old syntax is preventing me from compiling the classes. Would my best option be iOS 8.0 instead of 8.1? I've tried fixing the problem myself but being new to Swift I haven't accomplished much.

Instead of creating a support issue in Github, please open a question on Stack Overflow and tag Alamofire. The Alamofire community is very helpful and supportive. Stack Overflow is the best place to get your question answered quickly and correctly.

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Crashes on latest version

Im making a modpack with this in it and when i load a world i get a error saying The game crashed whilst exception ticking world
Error: java.lang.NullPointerException: Cannot invoke "com.google.gson.JsonElement.getAsDouble()" because the return value of "com.google.gson.JsonObject.get(String)" is null

I would ask if you have GeckoLib installed, but I'm pretty sure you do since you were able to go into a world and it would usually stop you when you load into MC.
Did you have a previous version installed, like 1.1.5? If so, then that might be why. because I recently updated the config and the config doesn't update itself (which I'm going to have to implement). If you did update the mod and have the old config, I would suggest deleting the old config. Though seeing as how you're making a modpack and you didn't say anything about updating the mod's version or anything, I don't know.
I'll try to look into the error for now and hope you get back to me with a response at some point.

I went in to the config and deleted and now it works fine


Update out-of-process-services.md

fixing 2 typos

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


Add CLI Args

--host - local socket for remote tcp host
--interval - how many seconds between remote checks
--monitor - which containers to monitor (defaults to all)
--debug - change logger mode to debug

closed by #11


Name changes

There are no functionality changes in this code.
This PR changes names of some subsystems and associated packages to make names more consistent and unique.

“No functionality changes”... methinks a quick regression testing couldn’t hurt :)


Add method for Val changes.

Code and tests that adds Val#changes(). This aligns some methods in Val to the static methods provided by EventStreams. So that invalidations(), changes() and values() provide a better version of the ObservableValue concept.
// static methods in EventStreams.
ObservableValue<T> observable = ...;
EventStream<?> invalidations = EventStreams.invalidationsOf(observable);
EventStream<Change<T>> changes = EventStreams.changesOf(observable);
EventStream<T> values = EventStreams.valuesOf(observable);

// similar methods in Val (and Var).
Val<T> val = Val.wrap(observable);
EventStream<?> invalidations = val.invalidations();
EventStream<Change<T>> changes = val.changes();
EventStream<T> values = val.values();

(I based this branch on my other PR. Hope this prevents merge conflicts. Will rebase this if needed.)

I haven't changed Change<T> to Tuple2<T, T> in this PR, yet.

Thanks, looks good to me. Can you please rebase it against master?

Can you please rebase it against master?

Done. Rebased with git rebase -i master, deleted unrelated commits and pushed with git push -f origin.

Thanks, merged.


background-image: url('') triggers an error

When I use the following CSS:
/**
 * Disabled drop-down lists
 *    - background-color alone does not work in Google Chrome:
 *      http://code.google.com/p/chromium/issues/detail?id=100234
 *      Setting background-image: url('') solves the issue
 */
select[disabled] {
	background-image: url('');
	background-color: #ebebe3;
}

It triggers the following error in my JS console:
GET http://[...]' 404 (Not Found)
Confimed on the following browsers:

Google Chrome 16.0.950.0
Firefox 9.0.1

Are there any display problems or just the error thrown?

Nope, everything is working just fine. It's just the error thrown.

I don't know if that can help you, but while using background-image: url("") (note the double-quotes) does not triggers the error.
Also the URL is the location of the origial CSS file + ' (a quote). This seems that your plugin is resolving background-image: url('') into background-image: url(path/to/my/css/'), so the error is triggered only once the new CSS is injected into the page.


[FX-4620] Migrate Switch component

FX-4620
Description
Migrate Switch component to Mui/base Switch and TailwindCSS.
How to test

Temploy

Video recording
This is a comparison between Temploy and production:
https://github.com/toptal/picasso/assets/1824723/f2496ff5-a060-4d10-8afd-39d234fe50ac
Accessibility issues.
No new accessibility issues. Current issues are related to disabled uncontrolled Switch. The same as on production.

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

Breaking change

N/A codemod is created and showcased in the changeset
N/A test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

The previous version has a bug due to hidden input:

This was a reason why happo tests were fallen


Remove final to allow this class to be proxied

I'm proposing this small change in order to allow Apache TomEE to proxy this class using its DynamicSubclass mechanism, thus allowing Apache TomEE to use Artemis as a JMS provider.

do you mind creating a jira, and amending your commit with
ARTEMIS-XYZ Allowing RA to be proxied on Tomee
just to have it documented?


Prepare release for Azure.ResourceManager.ContainerInstance 1.3.0-beta.1

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Not able to create ipa file for app store.

Successfully integrated XWebView project in my project.
Now want to submit iPA to the app store. But when I am creating iPA ,organizer window showing 2 different thing one is my project and other thing is XWebView framework. Code is working fine on simulators but iPa is not working.
Let me know is there any steps to add XWebView project and is there any need to change settings so I can make working iPA.

Solved. Now using cocpods : https://cocoapods.org/pods/XWebView


Support alerting feature

Grafana 4 comes with alerting feature, however, this datasource doesn't support it. Please make it work with the alerting feature.

https://github.com/grafana/grafana/issues/7421


Clipboard Issues on Linux + Chrome

I'm on Ubuntu 20.04.1 LTS, with X11 and there's something wrong with the clipboard.
Works totally fine in-app, and I can paste to some applications fine, but if I try to paste into Chrome, nothing happens.
I can paste into chrome fine from other applications. In fact, my current work-around is to paste into gedit, and then paste to chrome.
Firefox seems fine. However, there's a noticeable delay between copying and pasting the exact same string from gedit vs. copying from lite. Not sure what that's about.
I see that we're using SDL_SetClipboardText to set the clipboard, so I have a feeling there's not going to be much we're able to do here. But I'm going to dig in and see if the raw data we're passing to it might be a little... off, or something.

Confirmed that it's nothing to do with the strings being passed to SDL_SetClipboardText; even using a simple C-style of "a" has the same problem.
So it's something to do with SDL's implementation. Doubt that's fixable from this side of things; I may just maintain a copy of manual, non-portable, working clipboard code on my branch or something.

For reference, I've found something. I don't know a huge amount X11 clipboards, but it seems that comparing gedit and lite with xclip does yield some differences.
It seems like xclip -selection clipboard -o -verbose gives me my copied text for both.
However, doing xclip -t text/plain -selection clipboard -o -verbose only gives me the copied text for gedit. For lite, it gives me Error: target text/plain not available.
I assume that this is the source of the problem; Chrome is probably expecting a particular MIME type. I'm going to do some research into SDL's clipboard handling. Not sure how deep we can dig into that from this side; also, no idea on what SDL's change-handling procedure is, but hopefully we don't need to go that far.

OK. It was actually fairly easy to modify the underlying SDL library. I've made a fix, and submitted it to SDL's github as a PR here: https://github.com/libsdl-org/SDL/pull/4385
So we'll see what they say.

Good job!
The only problem is that it may take a very long time for them to accept the PR, if they accept it at all. I have also a PR for SDL2 since several months and it is neither accepted nor refused.
I am already using a modified version of the SDL2 library since my PR is still not accepted upstream. It is there:
https://github.com/franko/SDL-1/tree/wait-event-timeout-1
Without my patch the application always use the CPU even when it is unfocused or even reduced to icon. My patch fix this problem so that when the application is idle waiting for events the CPU utilization is reduced to zero, as it should be.
We may include your patch in my branch of the SDL library so that the next releases of lite-xl will include the fix.
If you are interested I can take care of merging your patch into my branch of the SDL library. In my opinion this is the best option we have waiting for the PR to be accepted upstream.

Just for information, my own PR to the SDL library is:
https://github.com/libsdl-org/SDL/pull/4180

Ah, yeah. I'd say we should do that, please feel free to merge the PR into your branch.

I have merged you change into my repository there:
https://github.com/franko/SDL-4d
Your changes can be found in the main branch and also within the tag
https://github.com/franko/SDL-4d/releases/tag/release-2.0.14-s4d
that corresponds to the 2.0.14 release.
There was another issue that was probably related to the SDL library:
https://github.com/franko/lite-xl/issues/64
but I wasn't able to give a fix. I we manage to fix this problem it is another candidate to be included in my fork.

Well, I done goofed here, this doesn't add an additional MIME type, it overwrites the previous MIME type.
So that's not good. I'm going to fix this.

No problem from my side. We need to fix this but we are not in a hurry. Just let me know if you need some help to correctly code this clipboard thing.

Mmk. I see where the issue is. We have to go from one -> many in terms of how this is architected, so it's a slightly larger change than anticipated, but not a huge deal.
I'd kinda be interested in seeing if we actually can support many mime types in the clipboard (would allow you to do things like copy and paste tables from Markdown directly into Excel, for example), but that's probably way out of scope for something that SDL would accept. Maybe in future. For now, I'm just going to concentrate on getting multiple targets working, all of which contain the same text.

OK, this has been fixed. All MIME types alias to the same, though we report to the requesting programs that we support all those individual MIME types. I'll put a PR on the SDL4d repo.

Good. I guess we should focus to make it work correctly for text only with a minimum of code to remain within the scope of the SDL library. Thank you for looking at that.


NIDM-Results 1.3.0

This PR implements NIDM-Results 1.3.0, updates are as follows

Creation of a Group (multi-subject study) or Person (single-subject study) agent
Creation of an Imaging Instrument agent
Change the 'Data Scaling' entity to 'Data' with an additional nidm:'has MRI Protocol' attribute.

And also:

Work with version number including release-candidate suffix (e.g. 1.3.0-rc1)
Possibility to delete a temporary file after export in the nidmpack (e.g. https://github.com/incf-nidash/nidmresults-fsl/issues/67)

I am going to merge and release this now so that the examples can be updated. If needed, we will go through another iteration with a separate PR.


Added new MSDSO program for UT Austin.

Apologies for the quote change on line 312, and 446. Not sure how come Excel decided to change the encoding for those two lines.

Thank you very much @aoot