Test MOT17 test set, performance 54.1?

$ python3 main_track.py  --output_dir . --dataset_file mot --coco_path mot --batch_size 1 --resume model/619mot17_mot17.pth --eval --with_box_refine --num_queries 500
Test: Total time: 1:00:17 (1.3641 s / it)
Accumulating evaluation results...
DONE (t=2.52s).
IoU metric: bbox
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.541
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.884
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.589
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.080
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.440
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.636
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.050
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.345
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.641
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.223
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.573
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.719
Creating video index for mot.

Hi~
54.1 is AP metric, mot17 usually uses MOTA as metric.
Please run sh track_exps/mota.sh to get MOTA.

Thanks for your quick reply,

But I got MOTA 61.9%, am I set up incorrectly?

MOTA 61.9% is exactly correct. This is performance on validation set.
If you want to test on test set, please refer to MOT challenge.

Thx~,
Are there any samples in your repo?

$ python3 main_track.py --output_dir . --dataset_file mot --coco_path mot --batch_size 1 --resume model/619mot17_mot17.pth --track_eval_split test --eval --with_box_refine  --num_queries 500
Test: Total time: 6:32:09 (1.3251 s / it)
Averaged stats: class_error: 100.00  loss: 154.3566 (431.0294)  loss_ce: 33.7268 (98.5663)  loss_bbox: 0.0000 (0.0000)  loss_giou: 0.0000 (0.0000)  loss_ce_0: 20.7482 (60.7730)  loss_bbox_0: 0.0000 (0.0000)  loss_giou_0: 0.0000 (0.0000)  loss_ce_1: 25.3040 (65.3062)  loss_bbox_1: 0.0000 (0.0000)  loss_giou_1: 0.0000 (0.0000)  loss_ce_2: 26.7121 (67.1159)  loss_bbox_2: 0.0000 (0.0000)  loss_giou_2: 0.0000 (0.0000)  loss_ce_3: 24.3467 (70.0842)  loss_bbox_3: 0.0000 (0.0000)  loss_giou_3: 0.0000 (0.0000)  loss_ce_4: 22.4293 (69.1838)  loss_bbox_4: 0.0000 (0.0000)  loss_giou_4: 0.0000 (0.0000)  loss_ce_unscaled: 16.8634 (49.2832)  class_error_unscaled: 100.0000 (100.0000)  loss_bbox_unscaled: 0.0000 (0.0000)  loss_giou_unscaled: 0.0000 (0.0000)  cardinality_error_unscaled: 500.0000 (500.0000)  loss_ce_0_unscaled: 10.3741 (30.3865)  loss_bbox_0_unscaled: 0.0000 (0.0000)  loss_giou_0_unscaled: 0.0000 (0.0000)  cardinality_error_0_unscaled: 500.0000 (499.9284)  loss_ce_1_unscaled: 12.6520 (32.6531)  loss_bbox_1_unscaled: 0.0000 (0.0000)  loss_giou_1_unscaled: 0.0000 (0.0000)  cardinality_error_1_unscaled: 500.0000 (499.9495)  loss_ce_2_unscaled: 13.3561 (33.5580)  loss_bbox_2_unscaled: 0.0000 (0.0000)  loss_giou_2_unscaled: 0.0000 (0.0000)  cardinality_error_2_unscaled: 500.0000 (499.9973)  loss_ce_3_unscaled: 12.1733 (35.0421)  loss_bbox_3_unscaled: 0.0000 (0.0000)  loss_giou_3_unscaled: 0.0000 (0.0000)  cardinality_error_3_unscaled: 500.0000 (499.9633)  loss_ce_4_unscaled: 11.2147 (34.5919)  loss_bbox_4_unscaled: 0.0000 (0.0000)  loss_giou_4_unscaled: 0.0000 (0.0000)  cardinality_error_4_unscaled: 500.0000 (499.9916)
Accumulating evaluation results...
DONE (t=6.79s).
IoU metric: bbox
Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = -1.000
Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = -1.000
Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = -1.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = -1.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = -1.000
Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = -1.000
Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = -1.000
Creating video index for mot.
Hi @PeizeSun
I got AP -1.00, am I set up incorrectly?

Please ignore this AP -1 is because no annotation is provided for test set. If you need the result on test set, please submit to MOT challenge.


Fix lint warning by renaming MaxBytesExceededErr -> ErrMaxBytesExceeded

1. What does this change do, exactly?
Renames MaxBytesExceededErr to ErrMaxBytesExceeded and make golint happier :)
Ref: https://ci.appveyor.com/project/mholt/caddy/build/2811#L87
2. Please link to the relevant issues.
N/A
3. Which documentation changes (if any) need to be made because of this PR?
None
4. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I am willing to help maintain this change if there are issues with it later

Thanks, every little improvement helps. :+1:


[Form] Global attribute 'isEdit' is not always coherent with the content of the spa.

Context
Within the application i wished to display some component and change the css according the the global attribute 'isEdit'.
Current behaviour
This attribute is set to true on the 'edit' button and set to false on the 'cancel' and 'save' buttons.
But if we open a page with a form already in edit mode, the variable is not set to true.
Moreover, if we left the page in edition with any link, the variable is not set to false.
Expected behaviour
On each load of the page, something must parse all form in the spa, and set 'isEdit' to true if there is at least one form in edition mode, else set it to false. It solves all problems.
Versions
"focus-components": "^0.8.6",
"focus-core": "^0.12.4",
"focus-file": "^0.1.2",

Tips
_onLoad{
    isEdit = false;
    for(Form f : spa.getForm()){
        isEdit = isEdit || f.getIsEdit();
    }
    dataFocus.setDataMode(isEdit ? 'edit' : 'consult');
}

@Pandakls I don't understant you tips ?
According to me on the componentWillMount this should be processed.

@Pandakls I close it; If the issue is still occuring, reopen it.


404?  Not found :D

During this section:
A request to the page with the "{id:int}" route template that does not include the integer will return an HTTP 404 (not found) error. For example, http://localhost:5000/Movies/Details will return a 404 error. To make the ID optional, append ? to the route constraint:
I"m not able to get a 404 error page, just a blank page.  When I update the page directive to {id:int?} the url Movies/Details still goes to a blank page?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 089d4e5a-b09a-936c-a6c9-fe6febf15f0a
Version Independent ID: 60b67f12-a9c6-27b4-31f5-1e5baeff051b
Content: Update the generated pages
Content Source: aspnetcore/tutorials/razor-pages-mac/da1.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

MacOS is hiding the 404 error

Try chrome


Add Liceo G. Ricci Curbastro

@SushiiStellar First stage of verification passed, official domain is cercalatuascuola.istruzione.it as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: http://cercalatuascuola.istruzione.it/cercalatuascuola/render/document/RAPS030001;jsessionid=XKUdd2JHMDLgJY3rwVE+Ag8a.mvlas008_2?prgDoc=1&codTipFil=6

Do I need to put the school's email address or my student email when
activating the student license?
On Tue, 2 May 2023, 05:33 Philip Torchinsky, @.***>
wrote:

@SushiiStellar https://github.com/SushiiStellar First stage of
verification passed, official domain is cercalatuascuola.istruzione.it as
far as I verified. This is just an informational note. Pull request is
still in the review. I appreciate your patience while waiting for the final
decision. Proof of domain ownership:
http://cercalatuascuola.istruzione.it/cercalatuascuola/render/document/RAPS030001;jsessionid=XKUdd2JHMDLgJY3rwVE+Ag8a.mvlas008_2?prgDoc=1&codTipFil=6
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/17409#issuecomment-1530821574,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYI6ZEG4P226WHSP2FPUHW3XEB6CDANCNFSM6AAAAAAXRWJIKA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@SushiiStellar Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Yes, you have to put your own student email address when requesting and activating the student license. Shall you need any further support, please send an email to<EMAIL_ADDRESS>


Easy to use dataset for DALL-E

The COCO dataset is a high-quality dataset both in terms of images and text. Each image has multiple captions and it consists of around 100 000 images. I have created a Google Colab which does the following:

Fetches the COCO images and captions
Allows the user to specify image dimensions which the images will be resized to
Stores information in two files for easy access in the following format:

od-captions.txt (<image_path> : <image_caption>)

train2017/000000203564.jpg : A bicycle replica with a clock as the front wheel.
train2017/000000322141.jpg : A room with blue walls and a white sink and door.
train2017/00000016977.jpg : A car that seems to be parked illegally behind a legally parked car
train2017/000000106140.jpg : A large passenger airplane flying through the air.
train2017/000000571635.jpg : A bathroom with a toilet, sink, and shower.

od-captionsonly.txt (<image_caption>)

A bicycle replica with a clock as the front wheel.
A room with blue walls and a white sink and door.
A car that seems to be parked illegally behind a legally parked car
A large passenger airplane flying through the air.
A bathroom with a toilet, sink, and shower.

Here is an example image and caption:

The man at bat readies to swing at the pitch while the umpire looks on.
I have written this hoping it will be somewhat compatible with @htoyryla's fork of a similar project. Might be of interest to you. Feel free to use this to generate a dataset for DALL-E!

This looks awesome! Thanks for the contribution 😄. Would you mind modifying the code (or adding a second function) to process the data in the way described in the README? Namely

Custom datasets should be formatted in a folder, with a jsonl file in the root folder containing caption data and paths to the respective images, as follows:
Folder structure:
    data_folder
        jsonl_file
        folder_1
            img1
            img2
            ...
        folder_2
            img1
            img2
            ...
        ...

jsonl structure:
{"image_path": folder_1/img1, "caption": "some words"}
{"image_path": folder_2/img2, "caption": "more words"}
...
you can then use the create_paired_dataset function in src/data/create_tfrecords.py to encode the dataset into tfrecords for use in training.

This looks awesome! Thanks for the contribution 😄. Would you mind modifying the code (or adding a second function) to process the data in the way described in the README? Namely

Custom datasets should be formatted in a folder, with a jsonl file in the root folder containing caption data and paths to the respective images, as follows:
Folder structure:
    data_folder
        jsonl_file
        folder_1
            img1
            img2
            ...
        folder_2
            img1
            img2
            ...
        ...

jsonl structure:
{"image_path": folder_1/img1, "caption": "some words"}
{"image_path": folder_2/img2, "caption": "more words"}
...
you can then use the create_paired_dataset function in src/data/create_tfrecords.py to encode the dataset into tfrecords for use in training.

Hey @StellaAthena @mrconter1 !
We actually already have a script for coco. Dataset collection will be happening in a separate repo, though (which is private for now until we make some progress :)

Hey @StellaAthena @mrconter1 !
We actually already have a script for coco. Dataset collection will be happening in a separate repo, though (which is private for now until we make some progress :)


Add Context to the BIDS json model

Similar to what Dandi does to generate their jsonld context, we should define a json ready @context that can be merged with the pydantic json dumps from the parsed BIDS datasets to create a graph ready jsonld file.

[ ] define a context for all classes and attributes used in the current neurobagel model for BIDS
[ ] use the appropriate NIDM terms and classes where available
[ ] Make a list of new classes and properties introduced that are not in NIDM so we can contribute them back

The context is generated directly from the pydantic model (similar to how Dandi is doing it with their json context file). The implementation was started here: https://github.com/neurobagel/bagelbids/commit/9d4765db568185911452cf3eb41b6d76e3f8f911
For now, we're putting all the terms into our own namespace. We need to change that and use appropriate NIDM terms instead. But I will track that in https://github.com/neurobagel/bagelbids/issues/5.


Issue-chore(release): 2.2.5 [skip ci]

Automated Review Comments
Issues in docs/CHANGELOG.md:

(2, 'Line exceeds 80 characters')
(7, 'Line exceeds 80 characters')
(9, 'Line exceeds 80 characters')

I got it.


GQLAlchemy can't be installed on Windows with Python 3.11

Due to its dependency on mgclient GQLAlchemy can't be installed on Windows with Python 3.11

Hi Katrina,
Any ETA on this bug? It will really help us out.

Hi @arunmanoharan4, this hasn't been a priority for us so far, and there have been some blockers to adding this support due to some dependencies on GQLAlchemy. Is there a possibility of you using Python < 3.11? Is this blocking your usage of Memgraph?


"Not an editor command" - no commands available

Hi there,
I've installed Hawtkeys with packer:
    use({
        "tris203/hawtkeys.nvim",
        requires = {"nvim-lua/plenary.nvim", "nvim-treesitter/nvim-treesitter"}
    })

And no hawtkeys commands are available.

No install failures, not other errors. Both dependencies were already installed.
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I dont know anything about Packer, but you will need to run the setup() function to have the commands available

Ah I should have known. And it requires a table is that correct? I'll leave this for anyone else that ends up here.
require('hawtkeys').setup({})


Display Name Encoding

I want to use soft hypens in the display name but the data annotation get's completely encoded when used as label:
[Display(Name = "Problem")]
public bool Problem { get; set; }

<label asp-for="Problem"></label>
Output is:
Problem because it is is encoded as Prob&shy;lem
Is there a trick to get soft hypens working?

@audacity76 use the soft hyphen character rather than the HTML entity:
[Display(Name = "Prob\u00ADlem")]

@dougbu thanks


Leetcode Question Number 1662

Leetcode Question Number 1662 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Please assign me this issue. I'm interested in solving it.


Coverall changes

Don't merge, trying to make coverall happy.

Is this ready to merge now?

I was going to do more work on it in the future. If you want the tests added to this release and are happy with them, feel free to merge.


⚠ Bofrost.de has degraded performance

In 8d5e2a2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1504 ms

Resolved: Bofrost.de performance has improved in bffc420 after 20 minutes.


Better active element assessment

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #196
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Now crawls shadow DOM to validate activeElement, used in validation workflow.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adding a config file to override quickstart configs

Description

Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Codecov Report

Merging #8059 (55542ac) into master (e23d5e7) will decrease coverage by 1.62%.
The diff coverage is n/a.

:exclamation: Current head 55542ac differs from pull request most recent head 21aa2d5. Consider uploading reports for the commit 21aa2d5 to get more accurate results

@@            Coverage Diff             @@
##           master    #8059      +/-   ##
==========================================
- Coverage   30.63%   29.00%   -1.63%
==========================================
  Files        1597     1597
  Lines       83055    83055
  Branches    12417    12417
==========================================
- Hits        25447    24094    -1353
- Misses      55361    56762    +1401
+ Partials     2247     2199      -48

Flag
Coverage Δ

integration1
29.00% <ø> (+0.01%)
:arrow_up:

integration2
?

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

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


add dedicated service account to run nodeagent

https://github.com/istio/istio/issues/9035, https://github.com/istio/istio/issues/10283
previous nodeagent run as default service account, which is fine; with integration with citadel, to retrieve root cert from configmap(which is written by citadel) for tls communication with citadel, nodeagent hit below error that default service account doesn't have permission of reading from configmap; this PR creates a dedicated service account for nodeagent(and grant configmap read permission).
2018-12-20T22:35:27.703153Z	info	unalbe to fetch CA TLS root cert error: failed to get CA TLS root cert: configmaps "istio-security" is forbidden: User "system:serviceaccount:istio-system:default" cannot get configmaps in the namespace "istio-system", retry in 2s

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: quanjielin
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@quanjielin: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

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. I understand the commands that are listed here.

/test e2e-bookInfoTests-envoyv2-v1alpha3

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mandarjog, quanjielin
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [mandarjog]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@quanjielin: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-minProfile.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-simpleTestsMinProfile

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. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-minProfile.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-simpleTests

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. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-minProfile.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
3c32893cccdb89756f7121012abc75ad28e96e1f
link
/test istio-pilot-multicluster-e2e

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. I understand the commands that are listed here.

/test e2e-simpleTests


Feature: Close non-fatals based on parameters.

Feature proposal

Firebase Component: Crashlytics

Ability to close a set of non-fatals that match parameters. If you log non-fatals such as error codes that users encounter, then fix a set of those (for example those with 500 codes), it would be nice to be able to close those instances that have the 500 non fatal codes.
Could be used in conjunction with https://github.com/firebase/firebase-ios-sdk/issues/6915

Hey @jostster I'm curious how the experience around this works today. Is the issue that the non-fatals you have are grouping, for example, 400-level and 500-level response code errors together? Would you consider throwing different types of errors for different response codes, or would that break your workflow in some other way?

So the way it was setup by the previous developer is for example network errors have an error code IE (1234). The details will show the reason returned by the api such as, invalid login, expired session, access denied, etc. It would be nice to be able to filter and close all of the 1234 networkError's that have expired sessions params.
I believe switching it to be by error code would make all our different modules errors grouped together. For example a 500 login error could be different from a store checkout 500 error which we would want to monitor differently.


is the technic launcher dead?

I haven't been able to access their website for 2 days now. or search modpacks.

This is the Solder Repository and is not an issue tracker for the launcher.
http://www.technicpack.net/help
https://github.com/TechnicPack/TechnicLauncher

oops thanks


Malware Detection

Hi,
I went to download your widget however avast believes there is a trojan horse in the executable. Is your compiled version infected?
Cheers.

Btw, this is a new release which might not trigger you AV Software:
https://github.com/tappi287/rf2_video_settings/releases/tag/1.5.4

Hi. Thanks for updating the project. It now behaves differently with Avast. Previously the exe version would download but Avast would stop it executing, and the portable version wouldn't finish downloading as Avast stopped it before it would complete.  Now, they both download fine and when I manually scan them all is OK, however when executing they get stopped and get sent to their servers for a deep analysis.


Update Data availability page

[x] Add list of which database links require holding for publication and which we can proceed with (e.g. Dryad, OSF etc)
[x] Edit pink type links to embedded links
[x] Fix schematron messages for tags that didn't convert properly on import
[ ] Fred approved page
[ ] James approved page
[ ] Naushin approved page
[ ] Claudia approved page
[ ] Share page with Exeter if relevant

Definition of done

[ ] Team reviewed and approved page?
[ ] Add changelog to page
[ ] Assign Fred to update the Schematron if relevant

@FAtherden-eLife, @naushinthomson, @griffithsc  - the main update here is now ready for review: https://elifeproduction.slab.com/posts/data-availability-qi8vg0qp#huuz0-database-publication-policies
It's not a complete list because I can't get to data release policies for all the databases, but I've included as many as I could find.


快速访问Youtube的方法

快速访问Youtube的方法：科学上网方法

G Translate: "How to quickly visit YouTube: Scientific Internet method"
The translation (when I read it) failed to convert the metaphor "invited to drink tea" adequately. I believe that "was asked to help the police with their inquiries" catches the sense in British English, where that wording may carry the implication that the person being interviewed may in fact be banged around the interrogation room a bit ("Support your local constabulary: beat yourself up..."). Of course this never happens in the era of the Police and Criminal Evidence Act 1984.


"IncrementalBuild object has already been disposed!" assertion

I was testing out the latest master with a debug build and I got an assertion, IncrementalBuild object has already been disposed!. This came from IncrementalBuild.fs. I managed to have it occur fairly easily, just by using F# in VS normally; compiling and looking at code. I have been testing my NewFoom project which is entirely .NET Standard. Sometimes I get an assertion in CompileOps.fs, on CheckDisposed too, I believe they are related.
Not sure what might be causing this, but I think it was introduced when I added this commit: https://github.com/Microsoft/visualfsharp/commit/a1f2f807a6f302d4af77e440b69012e1c7c1f563 - not 100% sure though

Yes, I've seen this a few times months and months ago. I don't believe that it was due to the recent work here, but agreed that it's good to get a repro

This is an older issue that I haven't seen lately. Since much in the tools and compiler has changed since then, I'll close this out. We should file a new one with a repro when we see it.


Busboy with AWS Lambda function

Hi Everyone,
I am trying to use busboy to parse multipart/form-data coming from AWS API Gateway to AWS Lambda. I have used LAMBDA_PROXY integration.
My parser is as follows:
export function parseMultipartFormData(event) {
  return new Promise((resolve, reject) => {
    const busboy = new Busboy({
      headers: {
        'content-type': event.headers['Content-Type'],
      },
    });

    const body = {};
    let buffer = '';

    busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
      body.filename = filename;
      body.encoding = encoding;
      body.mimetype = mimetype;

      file.setEncoding('base64');

      file.on('data', (chunk) => {
        buffer += chunk;
      });

      file.once('end', () => {
        body.documentContent = buffer;
      });
    })
      .on('field', (fieldname, val) => {
        body[fieldname] = val;
      })
      .on('finish', () => {
        resolve(body);
      })
      .on('error', (err) => {
        reject(err);
      });

    busboy.end(event.body);
  });
}

Parser is working fine for fields. When I upload a .pdf file it gets corrupted.
So I checked the same file's base_64 encoded string on my local machine using this block of code:
const fs = require('fs');
const { Buffer } = require('buffer');
const path = require('path');

const stream = fs.createReadStream(path.join(__dirname, 'file.pdf'));

let buffer = '';

stream.setEncoding('base64');

stream.on('data', (chunk) => {
  buffer += chunk;
});

stream.on('end', () => {
  console.log(buffer);
});

Then I compared the base_64 encoded string with the string generated by busboy parser and somehow that string is not matching for the same file and encoding.
Is there anything wrong with using busboy with lambda? Or something is wrong in my integration?

No, that's not encoded.
But I also tried using Buffer.concat()
export function parseMultipartFormData(event) {
  return new Promise((resolve, reject) => {
    const busboy = new Busboy({
      headers: {
        'content-type': event.headers['Content-Type'],
      },
    });

    const body = {};
    const chunks = [];

    busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
      body.filename = filename;
      body.encoding = encoding;
      body.mimetype = mimetype;

      file.on('data', (chunk) => {
        chunks.push(chunk);
      });

      file.once('end', () => {
        body.documentContent = Buffer.concat(chunks);
      });
    })
      .on('field', (fieldname, val) => {
        body[fieldname] = val;
      })
      .on('finish', () => {
        resolve(body);
      })
      .on('error', (err) => {
        reject(err);
      });

    busboy.end(event.body);
  });
}

Same issue with this code.

What is the value of encoding and mimetype?
If those values indicate binary data (e.g. body.documentContent does not contain a base64-encoded string, but actually raw bytes), what if you use crypto to create a sha1 hash of body.documentContent and compare that to the sha1 hash of the original file? Do those differ?

Encoding is 7-bit
Mimetype is application/pdf

Hash on Lambda
busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
      file.on('readable', () => {
        let chunk;
        /* eslint-disable */
        while (null !== (chunk = file.read())) {
          console.log('Chunk', chunk);
          chunks.push(chunk);
        }
      });

      file.on('end', () => {
        content = chunks.join('');
        const buffer = content;
        const hash = crypto.createHash('sha1').update(buffer).digest('hex');
        console.log(hash);
      });
    })

Hash: 994bf10c1efe5b51a858fa7436cd65f98f317553
Hash on Local
const path = require('path');
const fs = require('fs');
const crypto = require('crypto');

const readable = fs.createReadStream(path.join(__dirname, 'Report.pdf'));

const chunks = [];
let buffer;

readable.on('readable', () => {
  let chunk;
  while (null !== (chunk = readable.read())) {
    chunks.push(chunk);
  }
});

readable.on('end', () => {
  buffer = chunks.join('');
  const hash = crypto.createHash('sha1').update(buffer).digest('hex');
  console.log(hash);
});

Hash: 51a955ac8d9272ea7fc4ea144de9488f67bc25a4
The hash are not matching for the same file. Is there any problem with encoding?

content = chunks.join('');

needs to be
content = Buffer.concat(chunks);

otherwise you're just converting to string again.

Also just to be sure, is event.body raw bytes (e.g. a Buffer or typed array like a Uint8Array) or is it a string?

Thanks for your support. The file size vary between S3 and local machine. On S3 its 22KB while on local machine the size of the original file is 15KB

Also just to be sure, is event.body raw bytes (e.g. a Buffer or typed array like a Uint8Array) or is it a string or something else?

Event body is base64 encoded string.
When I convert it into utf-8 it looks like this:
"body": "----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"documentName\"\r\n\r\nReport-1\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"documentType\"\r\n\r\nReport for Risk Analysis\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"clientId\"\r\n\r\ne4db0c94f84467f3c4d6\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"clientName\"\r\n\r\nHDFC Bank\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"documentDescription\"\r\n\r\nRisk analysis report of John Doe\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"documentExtension\"\r\n\r\npdf\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"documentPk\"\r\n\r\nb4b6a8c598dafe0e0dda\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"uploadUserId\"\r\n\r\nfe47b8bcd5ac11c77102\r\n----------------------------740586622815871533210431\r\nContent-Disposition: form-data; name=\"sourceApp\"\r\n\r\nSP_Pro\r\n----------------------------740586622815871533210431--\r\n"

content = Buffer.concat(chunks);

Previously I was using Buffer.concat only but it didn't work

If event.body is a base64-encoded string, then you need to either convert it to a Buffer first (from base64) and pass that to busboy.end() instead or pass the encoding as the second argument to busboy.end() so that Busboy can correctly deal with the actual raw, binary form data.
Also the file varying between S3 and your local machine sounds suspect. Maybe it did get corrupted during upload to S3 after all?

Previously I was using Buffer.concat but it didn't work

It is the correct way for buffering binary data though and will work for either text or non-text data.

If event.body is a base64-encoded string, then you need to either convert it to a Buffer first (from base64) and pass that to busboy.end() instead or pass the encoding ('base64') as the second argument to busboy.end() so that Busboy can correctly deal with the actual raw, binary form data.
Also the file varying between S3 and your local machine sounds suspect. Maybe it did get corrupted during upload to S3 after all?

Thank you so much! This worked :)
Steps

Add multipart/form-data to AWS API Gateway. Due to this API Gateway will look at the Content-Type and Accept HTTP headers to decide how to handle the body. In case of multipart/form-data it encodes the body into base64
Add busboy parser as follows

export function parseMultipartFormData(event) {
  return new Promise((resolve, reject) => {
    const busboy = new Busboy({
      headers: {
        'content-type': event.headers['Content-Type'],
      },
    });

    const body = {};
    const chunks = [];

    busboy.on('file', (fieldname, file, filename, encoding, mimetype) => {
      body.filename = filename;
      body.encoding = encoding;
      body.mimetype = mimetype;

      file.on('readable', () => {
        let chunk;
        while (null !== (chunk = file.read())) {
          chunks.push(chunk);
        }
      });

      file.on('end', () => {
        body.documentContent = Buffer.concat(chunks);
      });
      .on('field', (fieldname, val) => {
        body[fieldname] = val;
      })
      .on('finish', () => {
        resolve(body);
      })
      .on('error', (err) => {
        reject(err);
      });

    busboy.end(event.body, 'base64');
  });
}

AWS API Gateway has recently added the support to binary data and it requires specific way of passing the data (in our case it was base64) to work properly
Hence when I added busboy.end(event.body, 'base64') it worked smoothly

Thanks for your support @mscdex. I'm so thankful to you and busboy community.


Support for react-native-web?

Hey,
Found this library to be really helpful, are there any plans of porting this to react-native-web now?

I have worked on a port for react-native-web a long time ago, but I've never published. I might update it and push it.

@Guichaguri That's great! Waiting for this :smile:

Hey, @Guichaguri Did you get a chance to publish the react-native-web port?

Before publishing, I think we should discuss about implementation details:

Although it shares the same API, it is another module, separated from react-native-track-player, which means it would work pretty similar to react-router-dom and react-router-native.
I've built three media button extensions: (which can be selected through imports, e.g. import TrackPlayer from 'react-track-player/electron';)

DOM (default, uses window keydown event for media shortcuts)
Electron (global media shortcuts, thumbar media buttons on Windows)
NW.js (global media shortcuts)

Metadata is done through MediaSession, which only seem to be supported by Chrome and Firefox on Android at the moment.
The player is built on top of HTML5 Audio. It doesn't support adaptive streaming (DASH, HLS and SmoothStreaming), although it could through libraries such as shaka-player.
I've created an implementation for Google Cast (that would only work on Chrome), but I've never tested it. There's also the support for AirPlay (that would only work on Safari).
The code is unfinished. There is a lot of stuff that needs to be implemented, I'm also not really sure about browser compatibility.

Let me know what do you think

Most cross platform libraries, that support react-native-web have a necessity to add a separate module with the same API, so I believe that's what suits us as well.
It seems to me that HTML5 Audio is the way to go for web, because of obvious reasons. Although it doesn't support adaptive streaming immediately, I see it as a feature that's implemented after we have identified that rn-track-player works well enough on HTML5 Audio in Web.
I have limited understanding of the MediaSession API, but it looks like a way to let the browser know of what audio it is playing, along with some extra info. Afaik, this also doesnt feel like a huge priority for RN-Web in the start.
Cast and Airplay do seem like important features, although not the top most in the priority list imho.
If there are any parts that may need some contributions, please do let me know, and I will try to see if I can also contribute in some way, especially in adding support for web.  I'm building an open-source, cross-platform podcast app for developers, and I'm keen on using react-native-track-player for all the platforms :smile:

Before publishing, I think we should discuss about implementation details:

Although it shares the same API, it is another module, separated from react-native-track-player, which means it would work pretty similar to react-router-dom and react-router-native.

I've built three media button extensions: (which can be selected through imports, e.g. import TrackPlayer from 'react-track-player/electron';)

DOM (default, uses window keydown event for media shortcuts)
Electron (global media shortcuts, thumbar media buttons on Windows)
NW.js (global media shortcuts)

Metadata is done through MediaSession, which only seem to be supported by Chrome and Firefox on Android at the moment.

The player is built on top of HTML5 Audio. It doesn't support adaptive streaming (DASH, HLS and SmoothStreaming), although it could through libraries such as shaka-player.

I've created an implementation for Google Cast (that would only work on Chrome), but I've never tested it. There's also the support for AirPlay (that would only work on Safari).

The code is unfinished. There is a lot of stuff that needs to be implemented, I'm also not really sure about browser compatibility.

Let me know what do you think

Can we publish as a beta or another package for now? Then the community can aid in testing and development.

@Guichaguri it would be nice if you can push your work so we can contribute and move forward :)

Hola @Guichaguri sorry to be insistent on this, i've start to work on a web version of the project, but it would be nice if you can share what you've done before and the discus what the next steps are as you asked :) please let us know if you have some time for this, thank you :)

@Guichaguri A little update to this post! Also interested to see a possibility of integration with RN-web, especially that the V2 of react-native-track-player is on!

@jspizziri Thanks for the quick reply! Will definitely take a look at it, would be awesome for simplifying migration of my native app to web:)


Fleet units fail - no such file or directory, corrected by daemon-reload

@bcwaldon  FINALLY... I caught one NEW unit doing this, and conveniently, one that needs no sanitization for posting.
The dependency of<EMAIL_ADDRESS>loaded fine, the following unit did not run
The host in question is on 607.0, and the etcd hosts are not yet on 607 (this may be the issue here, it's not 100% clear yet) - the etcd machines are on either 557.2 or 584.0 (they've not gotten their turn to rewrite cloudconfigs that are broken in 607.0)
I'm guessing there's something to be looking for on an etcd host, but .....which one, and exactly what is something that I need to know to find that.

I've asked a few times for specific log error messages you're looking for. I'll find 'em if you'll tell me, but if I don't know what they are it's nearly impossible to find them unless I get lucky and spot something on a brand new host with nothing running on it.

#<EMAIL_ADDRESS>[Unit]
Description=Electrum: %p for %i
<EMAIL_ADDRESS><EMAIL_ADDRESS>
[Service]
User=root
Type=notify
NotifyAccess=all
TimeoutStartSec=900
TimeoutStopSec=15
Restart=always
RestartSec=10s
ExecStartPre=/usr/bin/docker run --rm -v /opt/bin:/opt/bin ibuildthecloud/systemd-docker
ExecStartPre=-/usr/bin/docker kill %p_%i
ExecStartPre=-/usr/bin/docker rm %p_%i
ExecStart=/opt/bin/systemd-docker run --name %p_%i  -v /opt/tmp/%i/%p/var/electrum-grs-server:/var/electrum-grs-server  -v /opt/tmp/%i/%p/home/coin:/home/coin  -v /opt/tmp/%i/%p/app/certs:/app/certs   -p 50001:50001  -p 50002:50002  -p 8000:8000  -h %p.electrum-grs-server.domain  registry.domain/electrumgrs/electrumgrs-groestlcoind
ExecStop=-/usr/bin/docker kill %p_%i
ExecStopPost=-/usr/bin/docker rm %p_%i

[X-Fleet]
<EMAIL_ADDRESS><EMAIL_ADDRESS>MachineMetadata="srv=wrkr" "nat=0" "role=txn"
MachineMetadata="srv=wrkr" "nat=0" "role=api"
<EMAIL_ADDRESS>
Mar 12 16:10:40 aws-txn-68-209 fleetd[651]: INFO manager.go:262: Writing systemd unit<EMAIL_ADDRESS>(972b)
Mar 12 16:10:41 aws-txn-68-209 fleetd[651]: ERROR manager.go:136: Failed to trigger systemd unit<EMAIL_ADDRESS>start: Unit<EMAIL_ADDRESS>failed to load: No such file or directory.
Mar 12 16:10:41 aws-txn-68-209 fleetd[651]: INFO reconcile.go:311: AgentReconciler completed task: type=LoadUnit<EMAIL_ADDRESS>reason="unit scheduled here but not loaded"
Mar 12 16:10:41 aws-txn-68-209 fleetd[651]: INFO reconcile.go:311: AgentReconciler completed task: type=StartUnit<EMAIL_ADDRESS>reason="unit currently loaded but desired state is launched"
```

Follow up:
ssh to offending host,
<EMAIL_ADDRESS>/opt/tmp/usw2.electrum/electrum-grs-disk $ systemctl status<EMAIL_ADDRESS>
●<EMAIL_ADDRESS>   Loaded: not-found (Reason: No such file or directory)
   Active: inactive (dead)
<EMAIL_ADDRESS>/opt/tmp/usw2.electrum/electrum-grs-disk $ systemctl cat<EMAIL_ADDRESS><EMAIL_ADDRESS>/opt/tmp/usw2.electrum/electrum-grs-disk $
<EMAIL_ADDRESS>/opt/tmp/usw2.electrum/electrum-grs-disk $ systemctl daemon-reload
<EMAIL_ADDRESS>/opt/tmp/usw2.electrum/electrum-grs-disk $ systemctl cat<EMAIL_ADDRESS>#<EMAIL_ADDRESS>[Unit]
Description=Electrum: %p for %i
<EMAIL_ADDRESS><EMAIL_ADDRESS>
ETC

Same here, very annoying. Random service failures becuase of service files disappearing.
Loaded: not-found (Reason: No such file or directory)

Might be related to out of memory in this case

Hi - I am having the same problem. Hosts are 607 (both etcd and workers).
Unit: fleet.service; Image: /usr/bin/fleetd; Message: INFO manager.go:275: Removing systemd unit<EMAIL_ADDRESS>Unit: fleet.service; Image: /usr/bin/fleetd; Message: INFO reconcile.go:311: AgentReconciler completed task: type=UnloadUnit<EMAIL_ADDRESS>reason="unit loaded but not scheduled here"
Unit: fleet.service; Image: /usr/bin/fleetd; Message: INFO engine.go:272: Scheduled<EMAIL_ADDRESS>to Machine(6af7de28a0a640f68983c9abb10882ec)
Unit: fleet.service; Image: /usr/bin/fleetd; Message: INFO reconciler.go:163: EngineReconciler completed task: {Type: AttemptScheduleUnit, JobName<EMAIL_ADDRESS>MachineID: 6af7de28a0a640f68983c9abb10882ec, Reason: "target state launched and unit not scheduled"}
Unit: fleet.service; Image: /usr/bin/fleetd; Message: INFO manager.go:262: Writing systemd unit<EMAIL_ADDRESS>(1097b)
Unit: fleet.service; Image: /usr/bin/fleetd; Message: ERROR manager.go:136: Failed to trigger systemd unit<EMAIL_ADDRESS>start: Unit<EMAIL_ADDRESS>failed to load: No such file or directory.

Our process is that we delete a destroy around 40 units, wait for that to be complete and then submit, load and start 40 more units. In the process a number of them randomly fail to start. Usually anywhere from 1-13 units. This feels like it might be a race condition between the writing of the batch of unit files and reloading the unit-file into systemd.

@pmorton Could you please share the contents of<EMAIL_ADDRESS>

@bcwaldon I reported a similar bug with complete steps to repro including unit files -- see https://github.com/coreos/fleet/issues/1149.
The unit file that failed in my case was this one  (its a go template, where CONTAINER_TAG is related to the tag of the docker image, and UNIT_NUMBER is generated at runtime depending on how many units the user wants to kick off)

I was able to get together a set of repro steps and file https://github.com/coreos/fleet/issues/1158. Please coordinate further communication there.


[request] Audio normalization

Often when listening to playlists, a bunch of the tracks will be very low volume compared to the rest. This results in you having to manually change the volume once those low tracks starts playing, which is not something that you'd want to have to do when listening to the musicbot in the background while playing games or doing other stuff.
A solution to this problem would be to have some kind of audio normalizer which can analyze the youtube/soundcloud link and automatically make lower volume tracks louder.

Thanks for your issue.
This feature has been requested quite a bit, we're keeping discussion and suggestions for it here: #156

Ohh, maybe I should have been doing some searching before posting.


"error: no packages found with binaries or examples" when install

I downloaded the project and build it (cargo build --release) and its ok. ("Finished release [optimized] target(s) in 8.21s")
But when i try to install it with : cargo install , i got the message "error: no packages found with binaries or examples"
How can i resolve that , thank you

Thanks for the report! There's no need, however, to run cargo install for this crate. You don't need to cargo install crates to use them, they can simply be added to Cargo.toml for projects and Cargo will automatically download and build them!


Run UAT tests on every commit on master

Depends on #799 (included in this PR).
The idea of this PR is running on CI more frequently UAT tests that we have so not to break what currently passes.
Sample CI workflow can be seen at https://circleci.com/workflow-run/0b1fe102-b57f-4e73-9c5b-0b59ed00ac66
In scope for https://www.pivotaltracker.com/story/show/155510659

Is it reasonable to deploy to integration if UAT tests fail?

Good idea. Let me integrate this.

@dincho I addressed both comments.

As discussed I prefer spending more effort on fixing more UAT tests (e.g. https://github.com/aeternity/epoch/pull/799 ) before merging an improved version of this PR, not ignoring failing UAT tests.

I force-pushed:

Rebased on current master - meant to contain some test fixes by @velzevur ;
I made UAT tests job to depend on build job;
I temporarily enabled UAT tests on all branches - not only master - so to see current outcome of UAT tests on this PR.

Current state of UAT tests:
...
Test node discovery ...
Alice has address http://localhost:3013/ and no peers
Bob has address http://localhost:3023/ and peers [http://localhost:3013/]
Carol has address http://localhost:3033/ and peers [http://localhost:3023/]

Node dev1 starting

Node dev2 starting

Node dev3 starting
Carol now has peers ['http://localhost:3013/', 'http://localhost:3023/']
FAIL

======================================================================
FAIL: Test node discovery
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/builder/epoch/py/local/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/builder/epoch/py/tests/integration/test_use_cases.py", line 186, in test_node_discovery
    assert_equals(carol_top.height >= blocks_to_mine, True)
AssertionError: False != True

----------------------------------------------------------------------
XML: nosetests.xml
----------------------------------------------------------------------
Ran 13 tests in 281.730s

FAILED (failures=1)

Apparently intermittent test failure in test job tracked at https://www.pivotaltracker.com/story/show/155582602/comments/186628457

I am closing this PR. Will open new rebased improved one.


SparkConf setup for PySparkTask

Just a few tweaks for spark tasks:

PySparkTask.setup to configure SparkConf (executed before instantiating SparkContext as SparkConf is immutable after that)
Logging of spark-submit stdout as info on job success (useful when developing/debugging)

The docs build failed on the link check, could you please restart https://travis-ci.org/spotify/luigi/jobs/54682573
Thanks


Update swiper to 4.5.0 and @stencil/core to 0.18.0

Thank you for the PR but we have a PR open to add Stencil updates here: https://github.com/ionic-team/ionic/pull/17500
If you could remove the stencil dependency update and run npm run build for the swiper update we can take another look, thank you! 🙂

Hi there,
Thanks for the PR! I am going to close this as we are not looking to merge in dependency updates right now. Our main focus is on fixing high priority bugs and adding in new features.
If you are looking for other ways to contribute, feel free to check out issues with the "help wanted" label: https://github.com/ionic-team/ionic/issues?q=is%3Aopen+is%3Aissue+label%3A"help+wanted"
Thanks!


Prefer equivalent JavaScript file for multiple global-to-import matches

Fixes #59. This PR improves global-reference-to-import to allow duplicate import sources in one case: when there are exactly two matches, and they are an equivalent JS/ CS pair (same directory, JS name is dash-cased). I also added another assertion to chai-jscodeshift to expect a thrown error when transforming a particular fixture.
cc/ @bouk @GoodForOneFare @fandy

Maybe a better fix is to delete the file immediately afterwards, which we want to do anyways (probably?)
Also another issue with transforming multiple files at once is: if the file has underscores in it it those will get transformed to dashes, which means the file reference will become invalid.

I'm not sure deleting right away is a great choice, since you'll most likely want to refer back and forth between the compiled and new file in order to make sure the transformations worked properly.
The second issue is definitely real, we had talked about it a little before but didn't come to any consensus. IMO, it's not a huge deal since the bundle should fail to compile, which will be a pretty good signal to go back and update any imports. We could also add a script to check for whether any existing imports are pointing at the converted file and update them, or update all CoffeeScript files to be dash case now, or just let JavaScript files be snake case. Thoughts?

I think we should just rename everything to dash case right now, use this PR and also add git rm --cached any file we transform so we don't accidentally leave it in the repo.

PR lgtm by the way

👍

I'm gonna think on the best course of action a little more, don't want to do anything that will be super disruptive.


Updates to several examples

Description
Minor upgrade to several examples:

reinforcement-learning/ddpg

[x] deprecated because of dependency on RLLAB

reinforcement-learning/dqn

[x] updating setup script to work out of the box

restricted-botlzmann-machine

[x] added usage to README.md

rnn-time-major

[x] removed as it is deprecated

rnn

[x] added pointer to Gluon-NLP in README.md
[x] added pointer to LSTM language modeling in Gluon NLP in /bucketing
[x] fixed README.md instructions, filename and results - made Python3 compatible

speech_recognition

[x] Updated README.md with pointer to instructions to compile MXNet with WarpCTC
[x] fix singleton for Python3

ssd

[x] added pointer to Gluon-CV

stochastic-depth

[x] updated the MNIST and Cifar10 download to use mx.test_utils

svm_mnist

[x] loop through the 3 possible output layers and print results to compare

svrg_module

[x] updated README.md to point to dataset
[x] refactored the data loading and caching of downloads

@ThomasDelteil Thanks for the contribution!
@mxnet-label-bot [pr-awaiting-review, Example]

@aaronmarkham @thomelane @safrooze Could you please take a look?

Looks good to me

@anirudhacharya updated

Thank you @ThomasDelteil . LGTM. My comments are addressed.
@indhub for review and merge.

@indhub @nswamy @sandeep-krishnamurthy can you have one last look and merge please ?


GlideImage Cache

How can i use image caching while I am using GlideImage

@Micoder-dev You can use a custom request builder with Cache Configuration strategy.


Removing the shadow in the icon

Thanks!
Please describe what your PR is solving? Can existing issue be referenced?
Maybe adding a screenshot helps?
Would be great to understand you PR better to get this moving faster.


Xdelta patch not applying

I have tried many different xdelta patchers and after 5 hours of trying to apply the patch. It has said Patch Applied a few times, but then I Load up the game and its Still in Japanese? Is there any way you can send me a pre patched rom?

How are you applying the patch? I assure you, there are no problems with the patch, so you must be doing something wrong.
And sorry, no I can not send a pre-patched rom since it is obviously illegal.

I figured you couldn't pre patch it, but I thought I'd ask anyway lol I have tried at least 3 different x delta patchers I select the patch and select the rom This is the most common error I get

Even when it is successfully applied I load up the game and it is still in Japanese

Did you try http://www.romhacking.net/utilities/598/ ?
I think you're using an old version of xdelta.exe, it needs to be version 3 (included in the above download).
Another thing you could try is to not use directory names with spaces in them, sometimes that causes issues.

The patch was successfully applied and the game is now translated into english! thanks for your help and it was was the xdelta patcher being outdated


phpDocumentor 2.9.0 reports itself as 2.8.5

The VERSION file needs to be updated.
$ grep -A 1 '"name": "phpdocumentor/phpdocumentor"' composer.lock
            "name": "phpdocumentor/phpdocumentor",
            "version": "v2.9.0",

$ vendor/bin/phpdoc --version
phpDocumentor version 2.8.5

This issue will resolve itself after v2.9.1 is released; I prefer to focus on fixing bugs and releasing that version than trying to re-tag and re-release v2.9.0. Our apologies for the inconvenience


Documentation

Readme doesn't have any examples

@esthersweon ping on this

Spoke with @jhurliman: "don't think these projects have any usage outside of studio... we don't want anyone to use them if possible, since they promote a file format we're actively working to kill off... say this is the (web/node.js) implementation of reading ROS 2 sqlite files, it's used in Studio for support of this legacy file format, and this format has been superseded by MCAP which you can read about here"
Based on this feedback, will revise README to do the following:

Clarify the purpose of repo / clean up empty or dead sections
Mention usage in Foxglove Studio
Point people towards MCAP

we don't want anyone to use them if possible, since they promote a file format we're actively working to kill off..

Too bad, but are there any alternatives?

https://mcap.dev/
😄


Initial commit

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e92b48b). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             main      #1   +/-   ##
======================================
  Coverage        ?   0.00%
======================================
  Files           ?       1
  Lines           ?       2
  Branches        ?       0
======================================
  Hits            ?       0
  Misses          ?       2
  Partials        ?       0

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 e92b48b...d844272. Read the comment docs.


pull request for e2e test at 12/29/2016 08:26:12

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


Issue with generating SDK from .NET Core assembly.

Hi Rico,
Could you please help.
When I try to generate SDK from .net core assembly project it rises an exception:
tem.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.AspNetCore.JsonPatch, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The system cannot find the file specified.
File name: 'Microsoft.AspNetCore.JsonPatch, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60'

Check if .NET Core is installed and 'dotnet' is globally available.

But the assembly exists.

v11.14.0:
We improved the assembly loader (mainly for .NET Core) so that it runs in a more isolated space and the loaded DLLs should better match the requested versions. Please test this with your projects to ensure that we didnt introduce regressions.
Important if you have DLL loading problems:

Check that the DLL in question is available in the output directory (where the selected assembly is located) or in ReferencePaths
Check that you selected the correct runtime: Either in NSwagStudio UI, as /runtime parameter in the NPM CLI or execute the correct exe binary

For more information regarding assembly loading: https://github.com/RSuter/NSwag/wiki/Assembly-loading
Main commit: https://github.com/RSuter/NSwag/commit/04576e4d1b714ac5bd9b5df11d72469a23f7c0ff#diff-14dafe6661bab407ae5c0d7095ccd1f4

We found referencing the .nuget folder problematic as you cannot guarantee you'll pick up the correct version of the assembly.
Try:
<Copy SourceFiles="@(Reference)" DestinationFolder="$(OutDir)WebApi2SwaggerReferencePath" />

And point to WebApi2SwaggerReferencePath in ReferencePaths.
The later cleanup with:
<RemoveDir Directories="$(OutDir)WebApi2SwaggerReferencePath" />

Correct, in versions prior to v11.14 it was often the problem that the wrong DLL was loaded. In the latest version (v11.14) I try to find the best version based on the requested version:
https://github.com/RSuter/NSwag/blob/master/src/NSwag.AssemblyLoader/AssemblyLoader.cs#L133
However, this also does not work very well because some DLLs/package versions are strangely missing from the cache (even if they are used) and also scanning the cache directory takes a lot of time...
So using the nuget package cache directory is just a shortcut to test if assembly loading can work...

I'd suggest recommending the above approach, it has been bullet proof for us, and you don't need to publish and build again, it's all done in one hit.

What is @(Reference)?

That is the resolved referenced from the .NET build target in MSBuild.
By the way, this library has saved us many hours in our current project, so a massive thank you is due!

You would put that in a target with AfterTargets="Build"

I think it is not needed to add WebApi2SwaggerReferencePath to ReferencePaths as NSwag automatically searches in the directory where the specified assembly is located...

But thanks, I'll try that... looks promising..

Is it also working with v11.14?

In projects that use the new PackageReference, i.e. .NET Core projects, the @(Reference) will be an item group where all of the assemblies are scatter in the .nuget folder, and quickly exceed the max length to pass to NSwag, the copy puts them all in 1 folder for that purpose. Alse in a regular build the output folder won't have the framework assemblies, but they will be in the @(Reference) for example.
We researched this extensively and believe this is the optimal solution.

Haven't testing in v11.14, but it should work as it puts all of the required assemblies in a folder and says "hey use these", and they are ones you want.

If there was a version of NSwag.MSBuild that didn't just shell out, you might be able to avoid that copy too.

Wow, this is really amazing!
  <Target Name="NSwag" AfterTargets="Build">
    <Copy SourceFiles="@(Reference)" DestinationFolder="$(OutDir)References" />
    <Exec Command="$(NSwagExe_Core20) run nswag.json" />
    <RemoveDir Directories="$(OutDir)References" />
  </Target>

Yeah, we have been using this for months (sorry for keeping it a secret), and it's lovely

Credit to @shaynevanasperen who reduced our solution to that little nugget

Yes, its much better than using "dotnet publish" with
<PublishWithAspNetCoreTargetManifest>false</PublishWithAspNetCoreTargetManifest>

Is there a way to make use of the Configuration variable in nswag.json?  Currently I have the assemblyPaths as bin/Debug/netcoreapp2.0/<assembly>.dll so this only works when building in Debug and not when building Release.

In the nswag.json set
defaultVariables="Configuration=Debug"

and in nswag.json use the placeholder
$(Configuration)

in the .csproj task use
    <Exec Command="$(NSwagExe_Core20) run nswag.json /variables:Configuration=$(Configuration)" />

Did you mean a different file for defaultVariables="Configuration=Debug" as the format is wrong for nswag.json?
I tried it anyway as "defaultVariables": "Configuration=Debug" and I already have the .csproj task as you stated above.  I'm getting error "Could not find a part of the path ...\bin$(Configuration)\netcoreapp2.0" when building.

Sorry, it's "defaultVariables": "Configuration=Debug" in nswag.json...
This only works in v11.14+

Ok yep, I had too early a version.  It works fine when I upgraded to v11.14.1, but I get an different error for v11.15 (not sure if this has been reported elsewhere):
2>Executing file 'nswag.json'...
2>System.BadImageFormatException: Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. Reference assemblies should not be loaded for execution.  They can only be loaded in the Reflection-only loader context. (Exception from HRESULT: 0x80131058)
2>File name: 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' ---> System.BadImageFormatException: Cannot load a reference assembly for execution.
2>   at System.Reflection.RuntimeAssembly.GetExportedTypes(RuntimeAssembly assembly, ObjectHandleOnStack retTypes)
2>   at System.Reflection.RuntimeAssembly.GetExportedTypes()
2>   at NSwag.SwaggerGeneration.WebApi.WebApiToSwaggerGenerator.GetControllerClasses(Assembly assembly) in C:\projects\nswag\src\NSwag.SwaggerGeneration.WebApi\WebApiToSwaggerGenerator.cs:line 50
2>   at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
2>   at System.Linq.Enumerable.SelectEnumerableIterator`2.ToArray()
2>   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)
2>   at System.Linq.OrderedEnumerable`1.ToArray()
2>   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)
2>   at NSwag.Commands.SwaggerGeneration.WebApiToSwaggerCommand.GetControllerNames(AssemblyLoader assemblyLoader) in C:\projects\nswag\src\NSwag.Commands\Commands\SwaggerGeneration\WebApiToSwaggerCommand.cs:line 261
2>   at NSwag.Commands.SwaggerGeneration.WebApiToSwaggerCommand.<RunIsolatedAsync>d__88.MoveNext() in C:\projects\nswag\src\NSwag.Commands\Commands\SwaggerGeneration\WebApiToSwaggerCommand.cs:line 185

how are you executing nswag? via NSwag.MSBuild? do you also copy the references or publish first?

Yes NSwag.MSBuild, same as your example above:
  <Target Name="NSwag" AfterTargets="Build">
    <RemoveDir Directories="$(OutDir)References" />
    <Copy SourceFiles="@(Reference)" DestinationFolder="$(OutDir)References" />
    <Exec Command="$(NSwagExe_Core20) run nswag.json /variables:Configuration=$(Configuration)" />
  </Target>

Not publishing first.

Can you provide a sample project where i can reproduce this?

Sure, see attached.  The build fails unless I downgrade the NSwag packages to 11.14.1.
NSwagIssue.zip

Please retry with v11.15.1

Works great thank you!

@Boriszn and @slang25 Is it working for you too?

It works for us as we use the @References copy approach, I think this makes sense when using from a csproj.
The assembly loading improvements are fantastic if running separately.

So I've encountered a couple of issues here. (Possibly a new problem or at least recent)
I can get my config to render if done through NSwagStudio. (added %USERPROFILE%\.nuget\packages reference to get it to work.
But after I added NSwag.MSBuild, I get this:
1>System.IO.FileNotFoundException: Could not load file or assembly 'System.ServiceModel.Primitives, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'. The system cannot find the file specified.
1>File name: 'System.ServiceModel.Primitives, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' ---> System.IO.FileNotFoundException: Could not load the specified file.
... etc ...

And from then on could not load assemblies using NSwagStudio. :(
To correct this, I removed the NSwag.MSBuild reference, deleted the bin & obj folders, republished locallay, and it would then load the assemblies again.
As @comdw commented, I also encountered the same error there (BadImageFormatException) and had to blow away my entire repo to fix.   Problem would not go away.
If I remove System.ServiceModel.Primitives dependency, this problem also goes away. :(

@electricessence do you use the correct binary?

@RSuter.  When I was investigating this (still unresolved), I had the latest version from nuget.  It was as if the DLL hint was wrong.  But again, I did this a few times from a blown away repo.

@RSuter & @slang25 - this was causing an issue in our project... we reference a dll that we have locally. So we have something like:
<Reference Include="OurNamespace.ServerClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null">
  <HintPath>..\ExternalLibraries\OurNamespace.ServerClient.dll</HintPath>
</Reference>

But now the @(Reference) contains the string OurNamespace.ServerClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null which didn't work with Copy because it isn't a file. I think it's the same issue @electricessence was running into.
Anyway, to cut a long story short, this fixed it:
<Copy SourceFiles="@(ReferencePath)" DestinationFolder="$(OutDir)References" />
Let me know if that's going to mess anything up... otherwise I suggest you update the documentation.

@cvallance Nice find, looking at this again, I cannot find much info about Reference, however ReferencePath seems to be the correct property to use, and is the primary output of ResolveAssemblyReferences, so I think that would be the more correct property to use.
I think it's worth updating the docs :+1:

@cvallance Your suggestion really saved my day! Thank you!
@RSuter I love using NSwag :)

@cvallance So I read this thread months ago, but I didn't realize the subtle difference between these two
<Copy SourceFiles="@(Reference)" DestinationFolder="$(OutDir)References" />

and - the one that works:
<Copy SourceFiles="@(ReferencePath)" DestinationFolder="$(OutDir)References" />

Interestingly for me the first would work OK with a Build, but not a Rebuild All. I don't fully understand why, I could guess - but trying to explain my guess here wouldn't be productive here!


Element not found within iFrame?

I am trying to enter data in a field using XPath, but not able to locate the element...
Code Used:
driver.FindElementByXPath("//*[@id='userId']").SendKeys "My Name"
Page example: https://www.csu.org
Any way to put data into field through the iFrame?

You need to set the context to the frame:
driver.SwitchToFrame(identifier)


ACCUMULO-4153: Update the getCodec method to no longer be synchronize…

…d and to use an atomic reference.
This is based only on JSTACKs across a running system. I've seen improvement as a result with local testing and will test at a larger scale as time permits. I can provide performance numbers if needed for the PR.
I'm basing this against 1.6 as that is where we will test it and use it. It should merge cleanly up to 1.7 and 1.8.
This is a very simple change. I added a unit test ( the last of which shows that we won't run into the problem of creating many )...though if it fails for you let me know!

I apparently clicked through to the commit and left a bunch of comments instead of commenting on the PR itself. Sorry about that.
Overall, this looks pretty good. Some nit-picky things on the test case (yay, tests!) and a similar recommendation to Marc's earlier note.


Add setting for custom CSS

Would be great if I could add custom CSS to change colors for example, alternatively maybe more control over the UI. But adding custom CSS is easy fix

Hmm, that sounds nice. I'll look into it.


Added Switch Row UI type

Hello,
Just added friendly UI control for multiple icon "switches" inline.
This is basically the blending of "button row" that allows multiple controls on one line, plus the switch On/Off functionalities allowing external values to update them. Icon coloration is used to display state.
Regards,
Emms007

Actual demo:

Have you tested this? I can't seem to get it working ok.
Refreshing the page does not keep the state of the buttons.
Sending a message to the input does not seem to update the button status. Maybe I'm not doing it right.
Also, the code seems overly complicated.
I showed an example of how to do this using a template node, with the following template:
<div flex layout="row" layout-align="space-around center">
    <md-button ng-repeat="b in buttons" class="md-icon-button" ng-click="click(b)">
        <ng-md-icon icon="{{msg.payload[b.payload]?b.icon2:b.icon}}"
                    ng-style="{color: msg.payload[b.payload]?b.color2:b.color}"></ng-md-icon>
    </md-button>
</div>

<script>
    scope.buttons = [{
        icon: 'pause', color: 'black',
        icon2: 'play_arrow', color2: 'red',
        payload: 'play',
    }, {
        icon: 'alarm', color: 'black',
        icon2: 'alarm', color2: 'red',
        payload: 'alarm',
    }];

    scope.click = function(b) {
        if (!this.msg) this.msg = {};
        if (!this.msg.payload) this.msg.payload = {};
        this.msg.payload[b.payload] = !this.msg.payload[b.payload];
        this.send(this.msg);
    }.bind(scope);
</script>

I think I will expand the existing button_row to support toggling the buttons.

Good to see a simpler way, sounds very good to do it on the page itself merged with current button_row.
The one I suggested indeed works for toggling, but the initial state after refresh is a bit awkward. In my mqtt setup I get regular updates of the state of buttons and never refresh so it is invisible.
From my understanding your demo only deals about sending an output value, while the complexity I had was rather about processing the input. Especially in the context where one inbound message will change the value of only one button, keeping the rest unchanged.

Yes, that example needs the entire state on the input but it can easily be changed so that it can handle single values.


The Picker and ComboBox example in the document doesn't work

🐛 Bug Report
I just paste code from documents, and it doesn't work, the items don't show.

🤔 Expected Behavior
The items of Picker and ComboBox display normally as same as the document.
😯 Current Behavior
The items of Picker and ComboBox are gone.
💁 Possible Solution
None.
🔦 Context
I just want both components work normally.
💻 Code Sample
https://codesandbox.io/s/react-spectrum-bug-io42gg?file=/src/App.js
🌍 Your Environment

Software
Version(s)

react-spectrum
3.25.1

Browser
Chrome 111.0.5563.64

Operating System
macOS 13.2.1 (22D68)

🧢 Your Company/Team
It's my personal project.
🕷 Tracking Issue (optional)
None.

For now, remove <StrictMode> from index.js. We are working on support for it: #779.


Document ReadValue/WriteValue

This is apparently part of the public API but is not documented anywhere

These are pretty core to writing a module so this seems higher priority relative to some of the less common stuff.


Appstudio update test-component-pac-ebva

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-ebva-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
9 seconds
appstudio-init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
7 seconds
appstudio-configure-build

✅ Succeeded
34 seconds
build-container

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/test-component-pac-ebva-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
10 seconds
appstudio-configure-build

✅ Succeeded
34 seconds
build-container

✅ Succeeded
7 seconds
show-summary


Webhooks - not working properly, only with SHA256 authentication

Gogs version (or commit ref): 0.10.x
Git version: 2.11.1
Operating system: Docker version of Gogs (Host System: Ubuntu 16.04.2 LTS)
Database (use [x]):

[ ] PostgreSQL
[x ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[ x] Not relevant

Log gist (usually found in log/gogs.log):

Description
Hello,
Gogs is great, but since version 0.10.x Webhooks doesn't work any more with the old Gogs plugin (1.0.8 or less) of Jenkins. And with the new Gogs plugin (1.0.9) it works only with password (SHA256 authentication).
With the Gogs version  <IP_ADDRESS>1 or less everything works fine.
In our development environment we use no password/secret for the Webhooks. But since version 0.10.x of Gogs the field secret was removed. Is there anything I have to configure to say Gogs to include the field "secret" in the payload?
It seems it's just the messing field "secret" in the payload, which cause the following error.
Gogs Webhook Request Header
Request URL: https://{DOMAIN}/config-mgmt/gogs-webhook/?job=TestPipeline
Request method: POST
Content-Type: application/json
X-Gogs-Delivery: 38a57511-eead-4183-8231-848550d523b1
X-Gogs-Event: push
X-Gogs-Signature:

Gogs Webhook Payload
 {
   // missing field: "secret": "",
   "ref": "refs/heads/master",
   "before": "eead88db9cf71cb0d7911fbac07b33b8f13ba12f",
   "after": "eead88db9cf71cb0d7911fbac07b33b8f13ba12f",
   "compare_url": "",
   "commits": [
     {
       "id": "eead88db9cf71cb0d7911fbac07b33b8f13ba12f",

    ......
 }

Jenkins Response
Stack trace

javax.servlet.ServletException: net.sf.json.JSONException: JSONObject["secret"] not found.
	at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:796)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
	at org.kohsuke.stapler.MetaClass$10.dispatch(MetaClass.java:362)
	at org.kohsuke.stapler.Stapler.tryInvoke(Stapler.java:746)
	at org.kohsuke.stapler.Stapler.invoke(Stapler.java:876)
        . . .
Caused by: net.sf.json.JSONException: JSONObject["secret"] not found.
	at net.sf.json.JSONObject.getString(JSONObject.java:2040)

Webhooks doesn't work any more with the old Gogs plugin (1.0.8 or less) of Jenkins

Yes, it is correct.

And with the new Gogs plugin (1.0.9) it works only with password (SHA256 authentication).

No, AFAIK, the discussion thread said it has back-support for older versions of Gogs.

And with the new Gogs plugin (1.0.9) it works only with password (SHA256 authentication).

No, AFAIK, the discussion thread said it has back-support for older versions of Gogs.

Sorry, but it's not correct. With Gogs 0.10.x I'm forced to use a password, otherwise the Jenkins plugin of Gogs (1.0.9) throws an exception as mentioned above.
javax.servlet.ServletException: net.sf.json.JSONException: JSONObject["secret"] not found
Last test was today:

Gogs version: <IP_ADDRESS>7
Jenkins plugin of Gogs: 1.0.9

You should file bug to Jenkins plugin, not here in Gogs. Because there is nothing we can do.

As you said: https://github.com/jenkinsci/gogs-webhook-plugin/issues/18
But I would appreciate it, if you could amend the documentation https://gogs.io/docs/features/webhook in the near future. Because it still includes the field "secret" in the payload of a Webhook.
Thanks a lot

Good catch! Updated.


Reorganize terminal interface using containers

Godot's UI system really shines when you use Containers as your first go-to for everything. It becomes a breeze once you mingle with them a little bit.
Most things can really be boiled down to a bunch of BoxContainers and MarginContainers. PanelContainers can be used to beatify things.
ScrollContainer - a scrollable thing, very easy to use.
TabContainer - a rectangle with children as tabs, works really well.
SplitContainer - creates a movable split between 2 children.
plus some more, it's worth randomly checking new things out and not just containers. Godot has a lot of neat features.
Firstly, do not be afraid of nesting a large amount of containers in each other and actively changing how things are parented, it's completely normal. In scripts, do not rely on exact paths, instead right click important nodes and select 'Access as Unique Name'. This will allow you to easily access these nodes from code in this scene using %.
For example
var title: Button = $TTY/Text/Title/HBoxContainer/Title
can be changed to
var title: Button = %Title
after that you can shuffle Title around as much as you want and introduce more containers as necessary. Obviously there can only be one node with the given name in the scene.
This will work as long as script is within the same scene saved as a single PackedScene. Though such nodes can also be accessed externally using get_node.
terminal.get_node("%Title")
Second, containers often rely on children for their own sizing. To get things to work exactly how you want you can use children Layout properties: custom minimum size, shrink direction, expand flags and stretch ratio. Those things can be hard to explain, just experiment with them on your own. If it doesn't seem to help, try nesting more containers.
I would suggest remaking terminal interface with just containers, without regular controls and their annoying anchors and position transforms. I think those are only useful when a bunch of unrelated things are placed on completely arbitrary points of the screen, like placement of main interface elements on user screen.

Not planning to do this myself, just a suggestion. Would be good practice i think.

Btw, you are not supposed to use Node2D nodes like Sprite2D in interface. There is TextureRect control for that. Node2Ds don't have some useful UI things like layout properties.

Thank you for the suggestion and for providing more information about containers and nodes!
The terminal UI was a template and still not fully done yet. So I just slapped bunch of controls and nodes there.
'Access as Unique Name' would be very helpful when creating nested containers.
I will refactor and implement them soon.


port quinn benchmark to iroh-net and compare

Did a quick-and-dirty port of the quinn benchmark to iroh-net.
Results on my machine (Linux, Thinkpad T480, Intel 8250u), running with cargo run --release in the respective bench project
(-c is the number of clients, -n is the number of streams per client)
quinn v0.10.2
-c 1 -n 1:    AVG Throughput  ~614 MiB/s
-c 1 -n 10:   AVG Throughput  ~576 MiB/s
-c 10 -n 1:   AVG Throughput  ~ 49 MiB/s  (per client)
-c 10 -n 10:  AVG Throughput  ~ 46 MiB/s  (per client)

iroh-net
-c 1 -n 1:    AVG Throughput  ~210 MiB/s
-c 1 -n 10:   AVG Throughput  ~180 MiB/s
-c 10 -n 1:   AVG Throughput  ~ 26 MiB/s  (per client)
-c 10 -n 10:  AVG Throughput  ~ 25 MiB/s  (per client)

Flamegraph for quinn with -c 1 -n 1 --download-size 100M:

Flamegraph for iroh-net with -c 1 -n 1 --download-size 100M:

oh so sorry about the merge, I think that was a fat finger in github from my phone, feel free to remove the commit!

@Frando @Arqu I think it would be great to get this in and record the values in netsim for both the quinn version we currently use and for iroh-net.

Unless you meant to do something else with the outputs? Upload stats?

I want the stats on perf.iroh.computer

This was merged in #1579

Created an issue to add to perf.iroh.computer: https://github.com/n0-computer/iroh/issues/1679


Updated license links

Current coverage is 58.30%

Merging #16 into master will not affect coverage as of 285dbdc

@@            master    #16   diff @@
=====================================
  Files            7      7
  Stmts          295    295
  Branches         0      0
  Methods          0      0
=====================================
  Hit            172    172
  Partial          0      0
  Missed         123    123

Review entire Coverage Diff as of 285dbdc

Powered by Codecov. Updated on successful CI builds.


Cannot remove a deleted IoTProcessor from IoTProcessorManagementState

When we use the Remove-IoTProcessor with our IoTProcessor (ex. test1) after a while we correctly see the status as "Provisioned Deleted". Now if we try to import a new one with the same name it fails. The only way to remove the IoTProcessor from the list is, obviously, to destroy and recreate the application IoTProcessorManagement.
Apart this, the architecture and code for this sample is awesome, thanks a lot!

Thanks!
i actually thought about completely deleting the IoTProcessor & the definition,  I figured if we remove the definition you lose all traces of it in the system. Which makes "where did my events go"  an impossible question to answer. I will keep the comments in mind for later phases. Maybe we can copy the IoTProcessor definition somewhere else (i.e. different reliable dictionary, external store) for tracking then deleting it locally.

Closing this as the latest version of this sample doesn't have these PS commands.


Batch post with text/csv

Hi
I'm trying to do a batch post with this command using the v<IP_ADDRESS> binary (tried both osx and linux):
curl -XPOST -H "Content-Type:text/csv" -d "id,name,site_url,logo_url\n1,test,test,test\n" https://postgrest.herokuapp.com/sponsors

I also tried moving the content into a csv file: sponsors.csv:
curl -XPOST -H "Content-Type: text/csv" -d @sponsors.csv  https://postgrest.herokuapp.com/sponsors

In both cases, I get this 400 response error:
{"message":"Failed to parse JSON payload. Failed reading: satisfy"}

I've also tried posting using golang and obtained the same result.
Posting with text/csv header and json object works:
curl -XPOST -H "Content-Type: text/csv" -d '{"id":1, "name":"hello","site_url":"github.com","logo_url":"github.com"}'  https://postgrest.herokuapp.com/sponsors

Whats the correct way to do a batch post with csv?

It looks like the demo server is still running version <IP_ADDRESS> which does not support CSV. Thanks for reporting the problem, I'll upgrade the server.

I also tried it locally with the v<IP_ADDRESS> binary. Is the first curl correct?


custom YOLOV5 object detection model not working

Hello, I have installed a Yolo 5 model
The model is quite large
But it doesn't work

Did you update the labels?
You have 80 labels? And the width and height are correct?

Yes, I made sure of that
Yes, I trained the model on 1280 scale images
Do you have an idea about the problem?
@zezo357

Can you send the full logs of everything,

ok

this full logs
@zezo357

ok

this full logs
@zezo357

There is something I don't like in the first error, does the model load correctly? I want the logs for that too, it looks like the prediction can't find the model,

ok

this full logs
@zezo357

There is something I don't like in the first error, does the model load correctly? I want the logs for that too, it looks like the prediction can't find the model,

Can you share all of the app logs until the error?

Can you share all of the app logs until the error?

Do you mean to send you the model and the labels?

Can you share all of the app logs until the error?

Do you mean to send you the model and the labels?

No I mean the output of the app above the error

Can you share all of the app logs until the error?

@zezo357

The model was not loaded correctly since it is using cuda backend did you convert it to pytorch lite?

!python export.py --weights "the weights of your model" --include torchscript --img 1280 --optimize

The model was not loaded correctly since it is using cuda backend did you convert it to pytorch lite?

I have not personally trained the model
The model has been given to me
I tried it tflite and it works
thank you so much
I will make sure of this and tell you
@zezo357

The model was not loaded correctly since it is using cuda backend did you convert it to pytorch lite?

I have not personally trained the model
The model has been given to me
I tried it tflite and it works
thank you so much
I will make sure of this and tell you
@zezo357

It worked on tflite not on this package, did I get the correctly?
You are welcome.

The model was not loaded correctly since it is using cuda backend did you convert it to pytorch lite?

I have not personally trained the model
The model has been given to me
I tried it tflite and it works
thank you so much
I will make sure of this and tell you
@zezo357

It worked on tflite not on this package, did I get the correctly?
You are welcome.

Yes, I tried the tflite version
Not on this package
thank you for your time


Upgraded to newest version of dash

The dash version that is currently running is old and should be easy to make the application run on newest dash version

has now been migrated to latest version of: plotly/dash


comments on url_view::collect

While the docs give an example on how to use  url_view::collect(), they do not show what is the use case it is supposed to address, and it is difficult -- at leat for me -- to come up with a motivating use case. Why would the following not be sufficient?
std::make_shared<const urls::url>(view);

How is url_view better than const url in this use case?
The name collect does not suggest what is going on. If std::make_shared is not good enough, why not  the following instead?
urls::url::make_shared(view);

The benefit of this interface is that class url_view does not have to be aware of the existence of class url.

Good point. We need to improve that.
The benefit of collect over make_shared/allocate_shared is using a single allocation instead of two+ allocations.
It requires access to the URL representation to do that.
collect is intentionally a little vague because there is no intuitive name. So users must consult the docs.
The problem is the documentation describes what it does but doesn't describe this benefit. I'll open a PR for that.
Maybe collect could be intentionally not vague, but it's not easy to come up with an intuitive name.
Some people have suggested: .alias(), .fix(), clone(), clone_view(), persist(), const_shared_from_this(), make_const_shared(), ...
None of these names are very intuitive and collect is reasonably fine.

Then name collect is Peter's idea, and he didn't like any of the alternatives.

they do not show what is the use case it is supposed to address,

Any time you havea

persist is good.

persist is a less common word but if Peter is happy then I am happy

persist is probably better than collect.

So the actionable suggestions are:

rename to persist
better goal-oriented documentation

The benefit of collect over make_shared/allocate_shared is using a single allocation instead of two+ allocations. It requires access to the URL representation to do that.

Whoa, this is brilliant!
But it is not easy to appreciate, even after your explanation, because when we talk about making two allocations instead of one in the context of make_shared, the immediate connotation is doing one allocation for the T and for the reference counter. But this is more that that.

So the actionable suggestions are:

rename to persist
better goal-oriented documentation

I would add to the list:

Explain in the docs that this is for having only one allocation: for the reference counter, the character buffer and the pointer-to type.

Oh yeah I guess you're right about that. It is one instead of three. And to your other point, I used url_view because it has less API surface than url const, even though those additional APIs are in theory uninvocable (because the members are const). This means you only have to include url_view.hpp to use shared_ptr<url_view const>, which is easier on compilation time.

@akrzemi1 I often lay out data as a fixed-size struct followed by a variable length collection of bytes or characters - this is a recurring design theme of mine. I have encapsulated the logic for achieving this via single normal allocation into this implementation detail:
https://github.com/CPPAlliance/url/blob/4c2d8f71aa2b18bd6e3bc5f075127be8922b97eb/include/boost/url/detail/over_allocator.hpp#L43


Not able to start, webpack-dev-server error

My steps:

git clone https://github.com/sasha240100/react-rectangle-popup-menu.git
cd react-rectangle-popup-menu
npm install
npm start

Expected:
Project runs.
Actual:
./node_modules/.bin/webpack-dev-server: No such file or directory
For me, the start script "(npm run watch & npm run examples:watch)" makes no sense. There is no watch script in examples/package.json.
Node version: 8.11.3

@pawelsas Can I close this issue accounting @RickyTB comment as an answer?


removing data field list for now

@contolini this is small, so as soon as travis checks out I'll merge.


RFC 0685: Message delivery in Live Mode

As explained in the protocol, when Live mode is enabled and provided there is a persistent connection alive, the mediator must send any message for the recipient immediately.
My question is how the message should be delivered: should the mediator send the encrypted message (as it comes from the sender) through the transport, or shall it embed it into a Delivery message containing this single message? Even if it's not explicitly mentioned, based on previous implementations it seems evident that it should take the first approach.
However, while experimenting with WebSockets, I've seen that there are some situations where it might be important to have an application-level acknowledge (such as the provided by Delivery / Messages Received loop): if connection is closed abruptly by a party, the other end (at least on the application level) will only know about it once its ping/pong mechanism detects it or a lower-layer library/OS informs about it (usually after 30/60 seconds or more). And if it sends a message through the socket during that window, it won't notice about the delivery issue.
To put it in an example, it may happen that a mobile agent loses connectivity and, a few seconds after, a third-party sends a message to it through its mediator, so it forwards the message to the apparently opened WebSocket. However, the message does not arrive at destination and the mediator doesn't notice about that, effectively losing the message and leaving the responsibility of retrying the delivery to the upper-level DIDComm protocol.
By the way, my experience with this issue comes mostly from JavaScript environment, so maybe it's a a problem about a particular implementation, but according to some research I've done it seems to be a common issue with WebSockets, which is of course the typical transport to use in Live mode.

Another related issue to this that we (BC Gov in concert with others) are working right now on is the horizontal scaling of a mediator, where a dropped connection from a mobile agent (perhaps because they switch networks from wifi to cell) results in a new connection to a new instance of the mediator. This means that in addition to detecting the messages did not get delivered, the messages probably need to go to a queue for delivery by any mediator instance.

Another related issue to this that we (BC Gov in concert with others) are working right now on is the horizontal scaling of a mediator, where a dropped connection from a mobile agent (perhaps because they switch networks from wifi to cell) results in a new connection to a new instance of the mediator. This means that in addition to detecting the messages did not get delivered, the messages probably need to go to a queue for delivery by any mediator instance.

Thanks for posting this @swcurran ! This is also a problem we've faced due to the exact same issue of suddenly dropping WebSocket connection.
Looking at current DIDComm V2 spec I found an interesting clause about rewrapping that could also be a solution for this: upon the reception of a Forward message, the mediator will re-wrap the payload into another Forward message for its mediatee, who will need to unwrap it and send an acknowledge. However, later in ACKs section it discourages the usage of acknowledges in forward messages, so maybe it's not the best idea.
By the way, I also found an interesting issue created by @andrewwhitehead where he makes some very good points about the importance of re-wrapping before forwarding messages. If we enforce using either Forward+Ack or Delivery+Messages-Received in this RFC, we will be covering these concerns.

I think that it's still possible to use the message-received message in Live Mode. The Mediator should keep those messages sent in Live Mode in a separate queue until they are marked as delivered. Every time the Live Mode is re-enabled, the mediator should send all messages on that queue.


Add string methods removeprefix and removesuffix

Since Pandas 1.4.0 release (see issue https://github.com/pandas-dev/pandas/issues/36944), two new string methods are available, as a result of Python 3.9 incorporating them as native str methods:

Series.str.removeprefix, which applies str.removeprefix to a whole Series.
Series.str.removesuffix, which applies str.removesuffix to a whole Series.

So this issue is about adding these methods to Dask (which should be quite straightforward).
As a temporary workaround, they should be marked explicitly as NotImplemented, the same way it's done for .str.get_dummies (see line below):
https://github.com/dask/dask/blob/43f33a32dd5cdf77363b6ff6348b5f1a8282d28c/dask/dataframe/accessor.py#L119
Please note: if we want to be compatible with Python < 3.9 or Pandas < 1.4, the implementation shouldn't rely on the standard library's existence of these two methods. For example, in Python 3.8 they weren't implemented yet.

Thanks @jorloplaz -- I agree this would be a nice addition. Is this something you're interested in working on?

I'll see what I can do. This'd be my 1st issue and you tagged it as a good 2nd one, but I think I can manage.


Create StorageClass: allow creation of CephFS new pool

NOTE: CI update needed for E2E tests to pass: https://github.com/openshift/release/pull/53224

Allow the creation of new cephfs pools that are managed by StorageCluster CR and reflected in its managed CephFilesystem CR (default one).

StorageClass creation: allow creating a new CephFS pool via modal: unified modal for block/cephfs creation.

Unified view for Block+CephFS pools: Storage pools list.
Screenshots:

Create a StorageClass:

CephBlockPool Details Card: Volume Type shows that it's block type, Device Type shows the deviceClass:

/retest

/lgtm
(other than taking decision on https://github.com/red-hat-storage/odf-console/pull/1436#discussion_r1637623038).

/approve cancel
cc @bipuladh for approval

/lgtm

/retest

/retest

@SanjalKatiyar Can you please tag it?

/lgtm


Flash.nvim legibility

👋
I've been using Flash.nvim -- appreciate your adding support for it! Unfortunately the fg on top of purple (for atl least the hard variation of everforest) is not very legible. Would dropping the background change and using purple for the text color work, as the rest of the theme is greyed?

Oh, yeah, that's quite hard to read! I'll have a fiddle around with some colours and see what I can come up with...