Errors while building dependency check code

Hi,
I have download the dependency-check code from github and i am trying to build it, but the test cases are failing. Skipping the test cases while building also isn't helping as the artifact created by it is being used in another module.
Here is the test case failure i am getting:
[ERROR] Tests run: 6, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 3.679 s <<< FAILURE! - in org.owasp.dependencycheck.analyzer.CPEAnalyzerIT
[ERROR] testDetermineCPE_full(org.owasp.dependencycheck.analyzer.CPEAnalyzerIT)  Time elapsed: 0.036 s  <<< ERROR!
java.lang.StringIndexOutOfBoundsException: String index out of range: -7
at org.owasp.dependencycheck.analyzer.CPEAnalyzerIT.testDetermineCPE_full(CPEAnalyzerIT.java:85)
[INFO] Running org.owasp.dependencycheck.analyzer.DependencyBundlingAnalyzerIT
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0 s - in org.owasp.dependencycheck.analyzer.DependencyBundlingAnalyzerIT
[INFO]
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   EngineModeIT.testEvidenceCollectionAndEvidenceProcessingModes:100 » StringIndexOutOfBounds
[ERROR]   EngineModeIT.testStandaloneMode:116 » StringIndexOutOfBounds String index out ...
[ERROR]   DependencyCheckScanAgentIT.testComponentMetadata:50 » StringIndexOutOfBounds S...
[ERROR]   CPEAnalyzerIT.testDetermineCPE_full:85 » StringIndexOutOfBounds String index o...
[ERROR]   RubyBundleAuditAnalyzerIT.testAddCriticalityToVulnerability:153 » StringIndexOutOfBounds
[ERROR]   RubyBundleAuditAnalyzerIT.testDependenciesPath:204 » StringIndexOutOfBounds St...
[INFO]
[ERROR] Tests run: 56, Failures: 0, Errors: 6, Skipped: 2
[INFO]
[INFO]
I got an exception before in test case in Utils module before. I added the @ignore annotation to it to get past it and to build the code. Now i am getting the above exception.
If i run the test case individually then it passes, but as part of the build process, it fails.
Any help is appreciated.
Thanks!

Are you behind a proxy?  One of the things I have done is mirror the NVD locally and then updated my settings.xml:
<settings xmlns="http://maven.apache.org/SETTINGS/1.0.0"
          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
                          https://maven.apache.org/xsd/settings-1.0.0.xsd">
    <profiles>
        <profile>
            <id>default</id>
            <properties>
                <odc.cve.url.base>file:///Users/jeremy/nvd/nvdcve-1.0-%d.json.gz</odc.cve.url.base>
                <odc.cve.url.modified>file:///Users/jeremy/nvd/nvdcve-1.0-modified.json.gz</odc.cve.url.modified>
            </properties>
        </profile>
    </profiles>
    <activeProfiles>
        <activeProfile>default</activeProfile>
    </activeProfiles>
</settings>

Hi, Thanks for the quick response. I am not behind any proxy. Besides, running the test classes or test cases individually is working fine. Its only at the build time, they are failing.
Thanks,
Niddhi

What version of maven and java are you using?  Are you building in an IDE or command line?


drop DDP CLI test

What does this PR do?

Azura pipeline is failing on many PRs due to this test:
https://github.com/PyTorchLightning/pytorch-lightning/blob/4bdf2fe55f45c4cc8b397d4b45041265c402519f/tests/accelerators/legacy/test_ddp.py#L85[…]-L93
with error:
 AttributeError: type object 'DDPLauncher' has no attribute 'run_from_cmd_line'
Take for example this build:
https://dev.azure.com/PytorchLightning/pytorch-lightning/_build/results?buildId=202&view=logs&j=3afc50db-e620-5b81-6016-870a6976ad29&t=7df46610-aa16-5482-6b36-905d16374f81
As DDPLauncher is not really used anymore, it should be safe to remove the test.
cc: @tchaton, @Borda
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

@Borda test are passing now but still failing with
codecov: error: argument --token/-t: expected one argument
any idea?


build fails

running mvn clean package inside a freshly checked version of the repository exits with

[WARNING] The POM for org.eclipse.jgit:org.eclipse.jgit:jar:3.2.0-SNAPSHOT is missing, no dependency information available

Environment:

Apache Maven 3.3.3<PHONE_NUMBER>791599e205a5524ec3e0dfe41d4a06; 2015-04-22T13:57:37+02:00)
Java version: 1.8.0_25, vendor: Oracle Corporation
Java home: C:\Program Files\Java\jdk1.8.0_25\jre
Default locale: de_DE, platform encoding: Cp1252
OS name: "windows 8.1", version: "6.3", arch: "amd64", family: "dos"

Please retry – as this is the most recent stable release, maven should pull the complete dependency.


Fix ack failure on message listener in multi topics consumer

Fixes #446

Motivation

We found an ack failure issue: #446
MessageListeners can start to process messages before subscribing all child topics are completed in "multi topics" (e.g. partitioned, list, regex) cases.
If acknowledge() is called in messageListeners (it is very typical usage) before subscribing completion, it will fail with AlreadyClosed error since the state of the parent consumer is not Ready yet:
https://github.com/apache/pulsar-client-cpp/blob/8b2753a56579ea6cf11e26a0c5a160797518df63/lib/MultiTopicsConsumerImpl.cc#L643-L648
That results in ack holes, and finally full backlog.
Modifications

MultiTopicsConsumerImpl: Pause messageListeners at first by setting startPaused to true and resume them after subscribing all child topics are completed.
ConsumerImpl: Delete unused code.

It tries to skip sendFlowPermitsToBroker() at the first connection.

The motivation seems same as this PR, i.e. to prevent messageListeners from starting before subscribing completion.

However, actually it does not work so far, the variable firstTime looks always false at https://github.com/apache/pulsar-client-cpp/blob/8b2753a56579ea6cf11e26a0c5a160797518df63/lib/ConsumerImpl.cc#L316 because it becomes false at https://github.com/apache/pulsar-client-cpp/blob/8b2753a56579ea6cf11e26a0c5a160797518df63/lib/ConsumerImpl.cc#L297-L299

Also it is static variable in a method that is shared by all ConsumerImpl instances, whereas it has no chance of returning to true once it becomes false.

It seems better that child consumers don't have to care about their parents and are completely independent of them.

Verifying this change

[x] Make sure that the change passes the CI checks.

Specific tests for this issue is difficult to implement because it does not occur every time.
Documentation

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

Specific tests for this issue is difficult to implement because it does not occur every time.

Okay, I see it.


Merge to live

@GioviQ the localized repo work off live. It takes up to 2 weeks from merge to live.

@GioviQ  start the timer.


Fixed bug in _unixtime(date)

Use .timestamp() instead of .strftime('%s') .
See the comments of this answer https://stackoverflow.com/a/41811725/940974 .

Thank you Nicola. Merged


How to run in indigo nor kinetic

I want to run in indigo , but all document in kinetic . so  if i want to use indigo , how to change ? or can provide indigo branch to work.

Why do you need to use indigo? If you does not have any reason, just update your system to kinetic because the apt packages are not available for indigo.
If you have some serious reasons to use indigo, clone this repository with the other dependent repositories written in package.xml.

@yoshimalucky  Yes,  i use kinetic for TB2 and TB3, the same time use  indigo for TB2 , but some client want to use TB3 for indigo too, them still not update to kinetic . because some software don't support kinetic .


Upgrade dependencies

⬆ Bump ortools from 8.0.8283 to 8.1.8487  dependencies

⬆ Bump core-js from 3.8.0 to 3.8.1  dependencies

⬆ Bump eslint from 7.14.0 to 7.15.0  dependencies

⬆ Bump eslint-config-prettier from 6.15.0 to 7.0.0  dependencies

⬆ Bump husky from 4.3.0 to 4.3.5  dependencies

Fix #000 (<-- Add issue number here)
IMPORTANT: Each pull request (PR) should resolve an issue. Please create the issue if it hasn't been created, unless you're fixing typos or making minor changes to documentation or infrastructure. In that case, just leave the issue number as #000.
Details
If the PR title isn't enough to describe your PR, please provide details here.
Checks
Make sure these boxes are checked before your PR is ready to be reviewed and merged. Thanks!

[ ] PR is concisely and descriptively titled 📑 and links to the original issue 🔗.
[ ] Code is in uniquely-named feature branch and has no merge conflicts 📁.
[ ] Tests pass -- look for a green checkbox ✔ a few minutes after opening your PR.
[ ] @mentions of the person or team responsible for reviewing proposed changes. By default, use @BadwaterBay/reviewers.
[ ] (Optional) Screenshots/GIFs are attached 📎 in case of UI updates.

We're happy to help you get this ready. Don't be afraid to ask for help, and don't be discouraged if your tests fail at first.

Thank you for contributing to this project!

Codecov Report

Merging #182 (7da23a8) into main (fe44042) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #182   +/-   ##
=======================================
  Coverage   78.50%   78.50%
=======================================
  Files           8        8
  Lines         228      228
  Branches       20       20
=======================================
  Hits          179      179
  Misses         45       45
  Partials        4        4

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 fe44042...7da23a8. Read the comment docs.


P2P, SecureLink

Hello! Tell me pls, if i will use SecureLink module with Clappr player with P2P, and if securelink will take link witk toked everybody, then p2p will work?

I've never used SecureLink module. Just do tests and let us know the results.


Add possibility to overwrite repo_url and edit_uri for includes

Thanks for this plugin, it's great.
We like to use the edit button to jump directly to our git repo in edit mode.
However when using !include the path doesn't match any longer the one which is specified in edit_uri in the main mkdocs.yml.
It would be great to have the possibility to add repo_url and edit_uri for the included dirs so that the correct git repo and edit_uri links are generated.

I didn't check the plugins wiki and noticed there is already a plugin which does exactly that.
https://gitlab.com/paulrbr/mkdocs-edit-url

@claaslisowski Ah interesting, yes this is something we haven't easily figured out yet. We will take a look at that plugin and see if we can incorporate that functionality directly (given it's a small plugin)

Is there any update on this? (or was it somehow possible to use the plugin mentioned above?) Thanks.

why is that issue closed? I am currently playing around with this plugin where I want to aggregate MkDocs of multiple repositories using submodule includes.
it's crucial for me to have the correct edit link for the respective submodule repo.
did anyone solve that problem?
does https://gitlab.com/paulrbr/mkdocs-edit-url really help with that or am I talking about another issue?

I'm working on a slightly different plugin. In mkdocs-multirepo-plugin, you can specify the edit_uri in the config for each repo or in repo's mkdocs.yml. The implementation is in the on_NAV event.


BUG: Implement hotfix for Issue 7991

This (hopefully) implements a hotfix for issue 7991. The hotfix is implemented in Python to allow a rapid implementation and backport with minimal risk. The root cause can be addressed later with perhaps a refactoring of lfilter.c.
Closes #7991.

Can you please look at why the code in https://github.com/scipy/scipy/blob/master/scipy/signal/lfilter.c.src#L182-L188 is not catching this error?

@ewmoore Perhaps a better question is why there are tests in master that depend on that code not working correctly.
Edit: and if there are tests in master that depend on this, you can bet that there are users who depend on lfilter working correctly by chance with the int64 dtype.
What do you want to do?

I would just fix the C code, can't be very hard to get it right there.

That's correct, but that would technically be an ENH. I'll probably just restore the CXX branch and fix it there , since the CXX templating is more DRY.

I'm not sure what you are looking at.
I thought I remember that x can be int (or bool) which was IIRC (unless I mix it up with convolve)
I get NotImplementedError if the lag polynomials are int, but it works if x is int
>>> import scipy
>>> scipy.__version__
'0.18.1'

Python 3.4.4 (v3.4.4:737efcadf5a6, Dec 20 2015, 20:20:57) [MSC v.1600 64 bit (AM
D64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from scipy import signal
>>> signal.lfilter([1, 1, 1, 2], [1,  -0.5])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: lfilter() missing 1 required positional argument: 'x'
>>> signal.lfilter( [1,  -0.5], [1], [1, 1, 1, 2])
array([ 1. ,  0.5,  0.5,  1.5])
>>> signal.lfilter( [1,  2], [1], [1, 1, 1, 2])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\josef\Downloads\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.4.4.amd64\l
ib\site-packages\scipy\signal\signaltools.py", line 1015, in lfilter
    raise NotImplementedError("input type '%s' not supported" % dtype)
NotImplementedError: input type 'int32' not supported
>>>

>>> import numpy as np
>>> signal.lfilter( np.array([1,  2], np.int64), [1], [1, 1, 1, 2])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\josef\Downloads\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.4.4.amd64\l
ib\site-packages\scipy\signal\signaltools.py", line 1015, in lfilter
    raise NotImplementedError("input type '%s' not supported" % dtype)
NotImplementedError: input type 'int64' not supported
>>> signal.lfilter( np.array([1,  2], np.int64), [1], np.array([1, 1, 1, 2], np.
int64))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\josef\Downloads\WinPython-64bit-<IP_ADDRESS>Qt5\python-3.4.4.amd64\l
ib\site-packages\scipy\signal\signaltools.py", line 1015, in lfilter
    raise NotImplementedError("input type '%s' not supported" % dtype)
NotImplementedError: input type 'int64' not supported
>>> signal.lfilter( [1.,  2], [1], np.array([1, 1, 1, 2], np.int64))
array([ 1.,  3.,  3.,  4.])

but it works if x is int

Yes, and that is the problem. The C code is written to assume it will not work. And yet somehow it does.

I think the crash is just python3 string vs unicode issue in formatting the error message.

@pv Is the int routed to the object handler?

I don't know yet why it passes for josef, but one bug squashed at least: https://github.com/scipy/scipy/pull/8010

It passes for Josef because FIR filters are handled via a call to np.convolve and have a bunch of python code that does this conversion correctly and throws the correct NotImplementedError when it isn't given something of compatible type. See https://github.com/scipy/scipy/blob/master/scipy/signal/signaltools.py#L1282-L1284

@ewmoore you're right
I tried examples on the commandline because I didn't want to crash my interpreter, and didn't copy paste)
>> signal.lfilter( [1,  2], [1, 2], [1, 1, 1, 2])
causes a segfault
at least one float in filter polynomials required

signal.lfilter( [1,  2], [1., 2.], [1, 1, 1, 2])
array([ 1.,  1.,  1.,  2.])
signal.lfilter( [1,  2.], [1, 2], [1, 1, 1, 2])
array([ 1.,  1.,  1.,  2.])

(and I think now it was convolve or correlate that was used in masked arrays, from where I got my initial examples. )
sorry for the noise


Output configuration data in metadata and add --nanoseconds

Resolves #120 and #122.

@dadrian We currently do not output metadata-file at all. I'd be happy to discuss this in an issue if you feel strongly, but I really want zdns to replace dig for my daily life, and if I have to pipe stderr around by default every time I use it I'm going to be significantly less happy with my daily life.

I agree with Paul on this one. Especially given that we're also going to be making --short the new default output.

Sounds reasonable to me!


swift2.3:  Product->Archive:   error: use of unresolved identifier 'Log'

I use CleanroomLogger 2.2.0(First Swift 2.3 / Xcode 8 release) in Xcode project
when I archive :
error: use of unresolved identifier 'Log'

Does this only happen when archiving? Are you able to build successfully?
If building works but archiving doesn't, can you test whether compiling a Release build works?
Usually when I see problems that occur only during archiving, it's because (1) my "normal" builds are Debug builds, but my archive builds are Releases, and (2) the dependencies and linked frameworks differ between Debug and Release.
I suspect it's the configuration of your Xcode project. If you can provide me with any additional information, it might help.

Closing for lack of response.


Please withdraw my diff package

Could you withdraw my package https://nimble.directory/pkg/diff please?
I don't know how to do this myself.
The reason is that in view of the EU’s Cyber Resilience Act and an abundance of caution, I have withdrawn all my free software (see EU CRA).

To do it yourself the only thing you would need to do is to remove the relevant lines in packages.json (opening a PR with the change). Otherwise you will have to wait until someone does it for you :)

Thanks, I've done that in PR  #2785


Appveyor tests failing (CryptoGen)

A recent test invoking subprocesses has started causing failures in AppVeyor such as this one.

Apparently this is an issue with python affecting virtualenv and thus tox although these invocations are all using the same sys.executable, so it's not yet clear to me how precisely these issues are implicated.


Create a Post

You have to make a model window (page that should pop up on screen) in home page on clicking in text area of post something.

This model window will contain three sections 1)Title 2)content(question/post) 3)tags(hashtag) and addition features like attachments.
In last there should be a button to post your content and a cross button on top right to cancel post.
After clicking on post button a successful message should be displayed.
For creating model window design you can use your creativity, but should contain all features.
Remove post button only text area will be on home page.
You can use antd framework. for creating model window.
Make sure you make a component for this.

Deadline
48 hrs
Guidelines

The code should be clean and reusable.
Make sure that you make component for code reusablity.
You Don't need to create Navbar for this issue.

Note

Make sure you follow the given template before making the pull request.

Claim

@ananyaa07 you have been assigned.
Make sure you have read the description for more clarity. 👍🏼

@ananyaa07 update your work


pull request for e2e test at 01/10/2017 05:45: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.


added a new prop to hide the reset button of a SimpleSelect

This is a nice to have in order to create a select element, that has at least one value selected (see https://github.com/furqanZafar/react-selectize/pull/40).
This can also be done with css, but I think it's preferable to do it in react.

Thanks will add it to MultiSelect, add unit tests and release it in 0.8.0


v0.8.0 cannot load dataset with blank line

After preprocessing to wikitext-103 folder with v0.8.0, I use fairseq-train and get the following Error:
Traceback (most recent call last):
  File "/home/trinkle/.local/bin/fairseq-train", line 11, in <module>
    load_entry_point('fairseq', 'console_scripts', 'fairseq-train')()
  File "/data/git/Transformer/fairseq/fairseq_cli/train.py", line 321, in cli_main
    main(args)
  File "/data/git/Transformer/fairseq/fairseq_cli/train.py", line 46, in main
    task.load_dataset(valid_sub_split, combine=False, epoch=0)
  File "/data/git/Transformer/fairseq/fairseq/tasks/language_modeling.py", line 167, in load_dataset
    break_mode=self.args.sample_break_mode, include_targets=True,
  File "/data/git/Transformer/fairseq/fairseq/data/token_block_dataset.py", line 54, in init
    "Found multiple blank lines in the dataset, please remove them"
AssertionError: Found multiple blank lines in the dataset, please remove them (eg. cat -s raw.txt) and preprocess the data again.

However, in v0.7.x it doesn’t have such issue.

It is because of https://github.com/pytorch/fairseq/commit/851c022610b27da3beaa4e40a6834b5fb3b44f44


DevOps Mutualisation : Whilst building effective group communication, what are people comfortable sharing in public?

Description
This issue has been created to discuss how to provide a mechanism for continuous communication for DevOps Mutualization as Slack is not regulated and might not be the right place for team members to communicate outside the FINOS initiative.
Prior to solutions being explored, the question "What are people comfortable sharing in public?" has been asked to the group for feedback in the comments of this issue.
DevOps Mutualization Meeting Notes
Date and Time : Thursday 30th July @ 1pm ET / 6pm BST - https://github.com/finos/community/issues/52#issuecomment-669343645

The question of how the group communicates will be taken into the next meeting as a topic of discussion with the question "What are people comfortable sharing in public?" asked in advance.

This issue has been resolved through the use of GitHub Team Discussions. Please use the instructions below to engage and join the DevOps Mutualization conversation.
As taken from ...
https://github.com/finos/devops-mutualization/edit/master/docs/Discussions.md
DevOps Mutualization SIG Discussions
The DevOps Mutualization SIG uses GitHub Team Discussions that are only visible to members of the FINOS GitHub Organisation, with optional private discussions available to those who are part of the devops-mutualization-participants team.
GitHub Team Discussions augment the use of GitHub Issues in this repository by allowing both public and private discussions as part of the FINOS GitHub Organisation.
How to Participate in DevOps SIG Discussions

In order to collaborate in FINOS Team Discussions, you must have a GitHub Account and have signed a FINOS Contributor License Agreement (CLA). Please email<EMAIL_ADDRESS>for help and more information.
If you're are not a member of the FINOS GitHub Organisation, email<EMAIL_ADDRESS>and request an invitation to join from the FINOS team.
If you are a member of the FINOS GitHub Organisation, please email Amol Shukla or Anders Wallgren so they can add you to the devops-mutualization-participants team.
Once you are a member of the devops-mutualization-participants team you should be able to partake in public and private DevOps Mutualization SIG Discussions.


Support prune command for images, containers, volumes and system

Nerdctl is a great tool, but when used for experimenting it is often needed to quickly delete the remains of experiments.
Support for:

nerdctl image prune (--all --force)
nerdctl container prune (--force)
nerdctl volume prune (--force)
nerdctl system prune  (--all --force --volumes)

like in Docker would much help.

SGTM, but needs to make sure:

Non-nerdctl objects must not be pruned
nerdctl system prune should not prune networks because networks are not namespaced objects

Alternatively we can just have nerdctl namespace rm and call it a day.

nerdctl namespace rm seems could do the job and is probably easier to implement.

https://github.com/containerd/nerdctl/releases/tag/v0.22.0
nerdctl v0.22.0 supports nerdctl image prune --all, nerdctl container prune, and nerdctl volume prune.
No support for nerdctl network prune and nerdctl system prune yet though.

nerdctl network prune and nerdctl system prune already support and should be released in v0.22.1.


Demo Branch

Branch for demo development

@mhamilton723,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Timeline feature does not respect security constraint set by "add-group-repo" kie-cli command

Depends on : https://github.com/droolsjbpm/kie-uberfire-extensions/pull/13

+1 to merge.

https://github.com/droolsjbpm/kie-wb-common/commit/619e18b60fefa023fa1260224a9551c4df06fade


Add ability to get my recipes

The webapp offers the ability to make your custom recipes.
I've modified the existing code so that these can be gathered.

Hey, thank you for your interest. I am currently working on the same on this branch which will let one manage the collections (managed and custom). I think this will cover also the content of this PR.
https://github.com/miaucl/cookidoo-api/tree/collections

I didn't notice you already were working on it. Looking forward to see those changes implemented ;)

@davidclaeysquinones Is this still relevant, I referenced another PR about the recipes and collections.

You've made a better versión of what I've made.
My PR pales in comparisson.
I think that you just can cancel it.


Make NeedleBench available on HF

Describe the feature
Hi there,
Congrats on releasing the NeedleBench dataset! Niels here from the open-source team at Hugging Face. I discovered your work through the paper page: https://huggingface.co/papers/2407.11963 (feel free to claim it if you have a HF account).
I was wondering whether you would be up for making your dataset available on the hub, such that people can do:
from datasets import load_dataset

dataset = load_dataset("open-compass/needlebench")

? This would also ensure a dataset viewer, which allows people to easily discover and read the first few rows of the dataset in the browser, along with tags like "retrieval", "long-context" so that people can easily find it when filtering hf.co/datasets (filters are on the left).
See here for a guide: https://huggingface.co/docs/datasets/loading.
It could then also be linked to the paper page, as explained here: https://huggingface.co/docs/hub/en/datasets-cards#linking-a-paper.
Let me know if you need any help!
Cheers,
Niels
ML Engineer @ HF 🤗
Will you implement it?

[ ] I would like to implement this feature and create a PR!

Hi Niels, thank you very much for your valuable suggestion. I have added this feature in the latest PR #1364 . Feel free to reopen this issue if you have any further questions.


Performance best practices

I need some input on performance tweaking for the blog. I would like everything but the comments cached or maybe just have the cache refresh if there are comments added.  But I am not sure the best way to go about it.
Are there examples of the blog running < .5 sec?
Actual behavior
Site performs normally, but is slow.
Average site response time with all other main pages is < 500 ms
Average site response on blog pages is 3-5 seconds
Expected behavior
Blog is built as AMP enabled and based on their requirements it should be < 1s fully rendered so best expectation is server response should be around 400ms.
Steps to reproduce the issue
Normal operation.
Specifications

Zinnia version: django-blog-zinnia (0.18.1)
Django version: 1.10.3
Python version: 2.7
Operating system: CentOS 7

I have tried various forms of caching. Here is what I have now. None seems to make a difference so I might be doing something wrong there.
settings:
CACHES = {
'default': django_cache_url.config(),
'comparison': django_cache_url.config()
}
CACHES['comparison']['LOCATION'] = 'comparison'
CACHES['comparison']['TIMEOUT'] = None
Debug:
print django_cache_url.config()
{'LOCATION': '', 'BACKEND': 'django.core.cache.backends.locmem.LocMemCache'}
Disclaimer

[X] I have read the guidelines for contributing linked to above.
[X] I have checked for duplicate issues.
[x] This is not a support request.

Hello,
sorry for the late reply, generally bad performances are coming from templates integration.
The default templates are optimized to be scalable in terms of SQL queries.
The views are tested to use the minimal queries.
So I can suggest to install django-debug-toolbar and track the performances.
Regards


The system cannot find the path specified. - no space in absolute path

While attempting to train model using WAV files, clicking on process data after entering the path of the WAV files folder results in "The system cannot find the path specified." error in console.
Path of aforementioned folder: C:\RTCWebui\audio
Has no spaces, dashes, underscores, or non-English characters.
Replacing string like this: C:\\RTCWebui\\audio does not solve the issue either.
Forward slashes rather than backslashes in file path likewise does not work.
Attempting to train model results in the same error and crashes the program.
Seems like a bug of python occurring while trying to interpret windows absolute paths because in tutorials on YouTube and provided in documentation the file path worked while having dashes or underscores.
Using python 3.10.0 with all dependencies installed for windows.
Windows version: latest windows 10.

Checked .log files, and all of them are empty.

try to use  " / " forward slash instead of backward one, if that didn't help then try to get python 3.9.X, alternatively, check 3.10.6 specifically ( works for me ), if that doesn't resolve the issue the idk man, more issues arise lately ig.

try to use " / " forward slash instead of backward one, if that didn't help then try to get python 3.9.X, alternatively, check 3.10.6 specifically ( works for me ), if that doesn't resolve the issue the idk man, more issues arise lately ig.

Tried that... but the problem likely lies in how the developers of this program is handling the user input. Suggest improvements to add file explorer to manually select the path.
Considering I'm not a python developer, I have no idea how this works so this should be a suggestion for now.

Update #2:
Apparently loading in models from asset folder works. .pth files can be loaded from asset folder in the WAV conversion tab.
Why doesn't the file path in train tab work then?

I personally maintain my saturn port of rvc so, it's non local and can't help much but, In this case I'd blame your ffmpeg ( or lack of it? ) iirc there are python ffmpeg and standard ffmpeg, not sure which one's used, simply forgot, but you can search for it in issues ye. Alternatively, add me on discord and maybe we could figure it out somehow. dsc;  .codename0.

Somehow running infer-web.py directly with python works rather than me running the go-web.bat file.
Why though?

Somehow running infer-web.py directly with python works rather than me running the go-web.bat file. Why though?

I've no idea tbh, tho, glad you somewhat sorted it out

Somehow running infer-web.py directly with python works rather than me running the go-web.bat file. Why though?

because the bat file lacks lines of code to call the VENV you have probably installed your libraries inside a venv.

Update 2: Apparently loading in models from asset folder works. .pth files can be loaded from asset folder in the WAV conversion tab.
Why doesn't the file path in train tab work then?

Hello what do you mean by this sentence? what wav conversation tab are youtalking about? I dont see it, I am newly using this.

Somehow running infer-web.py directly with python works rather than me running the go-web.bat file. Why though?

because the bat file lacks lines of code to call the VENV you have probably installed your libraries inside a venv.

It's a problem with the .bat script
After replacing all occurences of "runtime\python.exe" with python.exe it works.

I finally ended using a release file they made, instead of trying to install it myself , and it worked. What other tools for voice clonign and voice gen are you using since?


Added dist and clean tasks

I've added dist task to minimize css and js file, and clean task to clean-up dist directory.
The pub task now depends on dist task instead of build task.

Thanks for improvement 👍


Update lstm.cc

fixing 'effecgive' typo

We will not be encouraging one liner grammatical changes as this is expensive process, if possible include more such changes in a single PR.Thank you
CC @mihaimaruseac @chanshah


Add short buffer (~256 bytes) for diagnostic test output (such as TAP YAML document)

If the buffer is shared among threads, there needs to be some form of synchronization.
Maybe an atomic counter is enough?

Maybe this feature should be pushed to version 0.2?


Box Node Code

I'm creating this issue because along with fixing the box node colors, I wanted to clean up the box node code to make it more generalized. This shouldn't take too long but I think it will help us down the road when we have a lot more bnodes in our data.

@touretzkyds  Do we still want to keep the box node names starting with 'b' and then a number? Or should we create something more general? For example, when a user sees b3: [avgmass, avglength, avgLIfespan], they then see b4:[avgmass, avgLength], then they see b5:[avgLength]. A user won't really understand what 'b3' or 'b4' really mean even if we signify that it's a box node it's still unclear what that really means. What if instead, we labeled the unexpanded b3 node more appropriately to be something like:  'Alligator Mississippiensis Comparisons: [avgmass, avglength, avgLIfespan]' and then the unexpanded b4 node would be: 'Alligator Mississippiensis Comparisons: [avglength, avgLIfespan]' and so forth?

I don't think lengthy labels like that will fit in the node. And I don't think they're necessary. The link labels already say prefComparisons, and they can seethat this property is hanging off the Alligator node. Let's stick with the b3/b4/b5 labels for now.

Ok sounds good. I wrote a lot more generalized code for displaying the box nodes. I did this so in the future when we have more and more of them, it would be easier to add them to the graph without breaking any code. I didn't worry too much about the layout but I created a function called 'add_new_bnodes' that takes care of continously adding bnodes from the box node query until there is non left. This way it will work regardless of if we have 3 or 20 box nodes.

Good. The new box node display looks fine. But your checkin added a .DS_Store file and a .expo directory  which should not be included in our repository. Please remove them.

Sorry those were added by accident. They should be removed now.


working-with-the-docker-registry.md

@15MariamS Diana navarro cola y e When you can find amazing great content that will make you

Why:
Closes [issue link]

What's being changed:

Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[x] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Stop
On Thu, Dec 9, 2021, 3:34 PM github-actions[bot] @.***>
wrote:

Automatically generated comment ℹ
This comment is automatically generated and will be overwritten every
time changes are committed to this branch.
The table contains an overview of files in the content directory that
have been changed in this pull request. It's provided to make it easy to
review your changes on the staging site. Please note that changes to the
data directory will not show up in this table.
Content directory changes
You may find it useful to copy this table into the pull request summary.
There you can edit it to share links to important articles or changes and
to give a high-level overview of how the changes in your pull request
support the overall goals of the pull request.
Source Staging Production What Changed
content/packages/working-with-a-github-packages-registry/working-with-the-docker-registry.md
https://github.com/github/docs/blob/9efccc7877bfd3a5f9264baba74c0e335cfc35cd/content/packages/working-with-a-github-packages-registry/working-with-the-docker-registry.md
Modified
https://docs-12815--patch-9.herokuapp.com/packages/working-with-a-github-packages-registry/working-with-the-docker-registry
Original
https://docs.github.com/packages/working-with-a-github-packages-registry/working-with-the-docker-registry
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/12815#issuecomment-990284518, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWVNRQJVYAOH4K34SODW5TTUQEOF7ANCNFSM5JXQXG4Q
.

@15MariamS Diana navarro cola y e When you can find amazing great content that will make you
Why:
Closes [issue link]
What's being changed:
Check off the following:

[x]  I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ]  For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[x]  This pull request impacts the contribution experience

[ ]  I have added the 'writer impact' label
[ ]  I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

m-star5:patch-9


scoped slots in v-for was parsed as slots

  with the following template:

           <template :slot="'action-template'" scope="ctx">
                edit&nbspdelete
            </template>
            <template v-for="field in fieldList">
                <template :slot="'sex-template'" scope="ctx">

sss

                </template>
            </template>

we print the slots with the following statement:

        console.log(this.$slots);
        console.log(this.$scopedSlots);

and found sex-template in this.$slots but not in this.$scopedSlots;
while action-template was correctly parsed.

Vue.js version
2.1.10
Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

A scoped slot is like a reusable function, it doesn't even make sense to nest it inside a v-for...


Revert #2101

#2101 was merged w/o testing and apparently introduced a severe regression.

The Test job is failing because of Coveralls.

I was wrong, the problem was elsewhere. Closing.


[ADAM-793] adding command to convert ADAM nucleotide contig fragments to FASTA files

Fixes #793

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

This looks good @heuermh! Let's sort out the sortBy issue, but otherwise I am +1.

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

@fnothaft I tried scaling up EC2 instances until I could get something similar to your approach to complete in less than six hours, and wasn't able to.
The code in my last commit here completes in around 47 minutes wall time on an r3.4xlarge and I've verified the sequences re-assemble correctly with lastz.  I still need to verify it works ok in clustered mode.

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

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

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

OK, I checked this on a full genome build on the cluster and it seems to check out. I think there are various performance improvements we can make, but I've opened #842 for that.

Thanks @heuermh! Merged as https://github.com/bigdatagenomics/adam/commit/3ac54a15fd21c0134a1022910f0657ffc670cecd.

Great, thanks!


GENERIC_VERIFY_FAIL in Ubuntu 18.04.2 LTS

Trying to install Tor browser through the Ubuntu software installer, but getting the above error every time. Please help. Thanks in advance.

having the same issue they tried installed directly from ppa -- new error
Extracting tor-browser-linux64-8.0.8_en-US.tar.xz
TypeError: error() missing 1 required positional argument: 'message'

Guys, I was having the same issue, then I decided to turn off my firewall (GUFW) and try to launch Tor Browser again. It worked! Then, I turned on the firewall again and now everything's working normally. I really don't know if the issue was really caused by the firewall, but that was the one and only thing I did and it worked. I hope this is useful somehow.

Guys, I was having the same issue, then I decided to turn off my firewall (GUFW) and try to launch Tor Browser again. It worked! Then, I turned on the firewall again and now everything's working normally. I really don't know if the issue was really caused by the firewall, but that was the one and only thing I did and it worked. I hope this is useful somehow.

Didn't work for me. I disabled my firewall by running sudo ufw disable then executed the launcher again. The download went much quicker but it still valid the signature verification.  Signature verification still took a few mins too.


listview in UIScrollView for iOS happen Gestures conflict

I use ListView in FlutterViewController. and add FlutterViewController.view  to  a uiscrollview(nativeIos).Both scrolls will be triggered at the same time?

Any progress? we get the same problem.

Would it be possible that you provide a minimal runnable reproduction project?

Any progress? We get the same problem. It's because the flutterViewControoler get the touch with "touchBegan、touchMove、touchCacnel、touchEnd" founction, that conflict with the gesture in the UIScrollView.

Any progress? I got the same problem for iOS. Everythis goes well on  Android  with the same construct.

https://github.com/lupeihong/flutterDemo
run by xcode

@lupeihong I solve the problem with the solution just like platformview. User delayGesture and forwardGesture to get touchEvent in flutterView.

https://github.com/lupeihong/flutterDemo
run by xcode

@lupeihong I solve the problem with the solution just like platformview. User delayGesture and forwardGesture to get touchEvent in flutterView.

Could you explain it in detail？

https://github.com/lupeihong/flutterDemo
run by xcode

@lupeihong I solve the problem with the solution just like platformview. User delayGesture and forwardGesture to get touchEvent in flutterView.

Could you explain it in detail？Your help is greatly appreciated.

I'm not sure the solution is the best. I copy the DelayingGestureRecognizer and ForwardingGestureRecognizer from the platformview. Use the ForwardingGestureRecognizer to dispatch the touches event replace of the old way that is lower priority to the scrollview gesture. But there may be some problem to solve, I'm trying it.

@zoechi any progress?

@zoechi i get the same problem, any progress?

Hi @lupeihong
are you still experiencing this issue with the latest version of Flutter?
If the issue persist please provide your updated
flutter doctor -v and flutter run --verbose
together with a a minimal code sample to reproduce the problem
thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.

I think the problem was perfectly described and there are links to example projects.

Make a Xcode project with a UIScrollView
On that place a Flutterview with a ListView on it
Try to scroll in Flutter
See example videos attached

example videos.zip

I have the same problem，But I think solved the problem better。Here's my solution，
Write a subclass of UIScrollView, and implement gesture protocol
(BOOL)gestureRecognizer:(UIGestureRecognizer *)gestureRecognizer shouldReceiveTouch:(UITouch *)touch {
if ([NSStringFromClass(touch.view.class) isEqualToString:@"FlutterView"]) {
return NO;
}
return YES;
}
Write a subclass of FlutterViewController， implement touches method
(void)touchesBegan:(NSSet<UITouch *> *)touches withEvent:(UIEvent *)event {
[super touchesBegan:touches withEvent:event];
[self touchesBegan];
}
(void)touchesMoved:(NSSet<UITouch *> *)touches withEvent:(UIEvent *)event {
[super touchesMoved:touches withEvent:event];
UITouch *touch = [touches anyObject];
[self touchMoveWithTouch:touch];
}
(void)touchesEnded:(NSSet<UITouch *> *)touches withEvent:(UIEvent *)event {
[super touchesEnded:touches withEvent:event];
[self touchEnd];
}
(void)touchesCancelled:(NSSet<UITouch *> *)touches withEvent:(UIEvent *)event {
[super touchesCancelled:touches withEvent:event];
[self touchEnd];
}
(void)touchesBegan {
UIScrollView *superView = (UIScrollView *)self.view.superview;
superView.panGestureRecognizer.enabled = NO;
}
(void)touchMoveWithTouch:(UITouch *)touch {
CGPoint previoursLocation = [touch previousLocationInView:self.view];
CGPoint location = [touch locationInView:self.view];
CGFloat deltax = fabs(location.x - previoursLocation.x);
CGFloat deltay = fabs(location.y - previoursLocation.y);
UIScrollView *superView = (UIScrollView *)self.view.superview;
if (deltax > 2 && deltay <= 5) {
CGPoint offset = superView.contentOffset;
superView.panGestureRecognizer.enabled = YES;
[superView.delegate scrollViewWillBeginDragging:superView];
[superView setContentOffset:CGPointMake(offset.x - (location.x - previoursLocation.x), offset.y)];
} else {
superView.panGestureRecognizer.enabled = NO;
}
}
(void)touchEnd {
UIScrollView *superView = (UIScrollView )self.view.superview;
superView.panGestureRecognizer.enabled = YES;
CGPoint offset = superView.contentOffset;
CGFloat offsetX = offset.x;
if (offsetX > (1 * self.view.width+ self.view.width * 1 / 3)) {
[superView setContentOffset:CGPointMake(2self.view.width, offset.y) animated:YES];
} else if (offsetX <= self.view.width * 2 / 3) {
[superView setContentOffset:CGPointMake(0, offset.y) animated:YES];
} else {
[superView setContentOffset:CGPointMake(self.view.width, offset.y) animated:YES];
}
[superView.delegate scrollViewDidEndDecelerating:superView];
}
@HaoXianSen


introduced error and refactored to use one state

fix for #67

But once you retry, isn't it a new Projectile + WebSocket connection ?
Best Regards,
George Gastaldi
Principal Software Engineer
Red Hat https://www.redhat.com
Remote
Joinville - Santa Catarina
<EMAIL_ADDRESS>  M: +55-47-99711-1000
https://red.ht/sig
TRIED. TESTED. TRUSTED. https://redhat.com/trusted
@redhatnews https://twitter.com/redhatnews   Red Hat
https://www.linkedin.com/company/red-hat   Red Hat
https://www.facebook.com/RedHatInc
On Wed, Apr 19, 2017 at 11:50 AM, Erik Jan de Wit<EMAIL_ADDRESS>wrote:

@edewit commented on this pull request.
In src/app/wizard/deploy/deploy.component.ts
https://github.com/openshiftio/launchpad-frontend/pull/76#discussion_r112224126
:

@@ -64,8 +65,11 @@ export class DeployComponent {
}
}

           this.done = this.statusMessages[this.statusMessages.length - 1].done;

           if (this.done) this.webSocket.close();

           let done = this.statusMessages[this.statusMessages.length - 1].done;

           if (this.done) {

no because you can retry
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openshiftio/launchpad-frontend/pull/76#discussion_r112224126,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADTdSUGJwLPKiZkeHlean1Xo6ZxIL3rks5rxh9LgaJpZM4NBywi
.


feat(gamification): add dependencies and some details about APIs

Pull Request Template
I have added swagger framework for auto generate document 4 APIs.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings
[x] My branch is sync with development branch
[x] I have follow conventional commit
[x] My pull request name follow standard and have related labels

assignee: @DanialMehrabi94
Reviewer: @DanialMehrabi94

LGTM


There are poems that appear in both "&" and "is 5"

We have at least a few poems that appear the table of contents for both books. The ones I have spotted are:

"in the,exquisite;"
"Paris;this April sunset completely utters"
"little ladies more"
i was sitting in mcsorley’s    outside it was New York and beauti-
Dick Mid’s large bluish face without eyebrows
the waddling
ta

Since we do not have an original first edition of "&," our table of contents for that book was drawn from another source. It's possible that it is inaccurate and introducing the redundancy. We know that the tables have been changed in subsequent through the years. It's also possible the poems were printed twice and everything is historically accurate.
Until we can solve the riddle by fixing #127 and acquiring a first edition of "&," the poems will be duplicated and included in both books.

There's also the possibility of version differences between copies of the same poem. I have no idea whether these would have been printed from the same plates (if "plates" is even the correct term – I'm more familiar with music) or whether the poems were newly typeset for each publication. If there are version differences that aren't just obvious typos, it would be good to maintain the differences.

I have no clue where to find one, sorry.


Dev dependencies in Shrinkwrap

Good afternoon,
Latest version ( 2.8 ) has the development dependencies listed in the shrinkwrap file. This means that when I do npm i on my project that depends on NSP, I get "extraneous" dependencies because the production code does not need your development dependencies.
Would you be so kind and publish a new version ( 2.8.1 ? ) without development dependencies in the shrinkwrap ?
Many thanks.

This was introduced here apparently: https://github.com/nodesecurity/nsp/commit/1d0b7b5ca145d30db062aa246f0bc2e3e3aa77f3#diff-36018a6a9e1de1798516799b84841318

Tuning in for updates.  Our team is also being affected by this issue.  We will have to stick with ~2.6.1 until further notice.
$ npm -v
    3.10.10
$ node -v
    v6.11.1

2.8.1 pushed without devDeps.


name of array disappear when apply "where" filter with Datarray but not with np.array

I was trying to select by non dimensional coordinate, and I found that "where" works well, but it erase the name of my datarray only if filtering is done with another datarray as a non dimensional coordinate
import xarray as xr
temp = 15 + 8 * np.random.randn(2, 2, 3)

precip = 10 * np.random.rand(2, 2, 3)

lon = [[-99.83, -99.32], [-99.79, -99.23]]

lat = [[42.25, 42.21], [42.63, 42.59]]
ds = xr.Dataset({'temperature': (['x', 'y', 'time'],  temp)},
                 coords={'lon': (['x', 'y'], lon),
                         'lat': (['x', 'y'], lat),
                         'precipitation': (['x', 'y', 'time'], precip),
                         'time': pd.date_range('2014-09-06', periods=3),
                         'reference_time': pd.Timestamp('2014-09-05')})

ds["temperature"].where(ds.precipitation>5).sum(axis=2).plot()

this line would produce a label near color bar
but only using a numpy array this happens as here
ds["temperature"].where(ds.precipitation.values>5).sum(axis=2).plot()

Expected Output
I think that selecting with a datarray should  not change the label of the data.
Further I was expecting that as for  "sel", "where" should change the title of the graph
so that I know from the title that I performed a selection with where with a non dimensional  coordinate

We currently drop names because that's our default behavior for xarray operations with multiple named arguments. But the where method uses apply_ufunc() with keep_attrs=True, which indicates that attributes from the first object should be kept. It would probably make sense to keep the name of the first object, too, when using keep_attrs=True. That would fix this issue and also fix the problem for other xarray methods (e.g., fillna) that use the same logic internally.

Confirmed. To show the specific result:
ds['temperature'].name
# 'temperature'

ds["temperature"].where(ds.precipitation>5).name
# None


Undoing political decision

In the name of my participance in iptv-org core member group I'm ressurecting from dead to prevent political driven mess in the playlist I care of. Hope I got syntax right on this one.

Fixed the above suggestion but I'm done with Github UI and the way it handles line ending


Optimize Syntax.Util destruct_typ_as_formula

This PR optimizes the destruct_typ_as_formula function in FStar.Syntax.Util.fst which was seen as hot in some profiling runs of ulib files (and a couple from HACL*).
The PR removes an unused list u_connectives and refactors the list connectives. These lists, while constant by intent, do not extract to constants in OCaml because the integers get parsed by C stubs  into Zarith big-integers. The PR also reworks (and unifies) the lookup algorithms for destruct_base_conn and destruct_sq_base_conn; primarily to lookup first by arity then search by string (this turned out faster than using a match). There is a tension between optimization, maintainability and clarity. I hope that the balance is ok.
Benchmarking the code with these changes I get:
                  Old                               New
                  n       total   user    system    n       total   user    system
micro-benchmarks  61      55.68   51.61   3.839     61      52.61   48.74   3.626
ocaml_extract     38      117.7   113.5   4.146     38      118.2   113.9   4.245
ulib              208     380     359.8   19.5      208     330.6   310.6   19.23

FStar.ModifiesGen.fst: 35.2s -> 31.4s (~10%)
There is a cost as ocaml_extract is a tiny bit slower (~0.5%) across many files
Couple of things to note:

the (unused) u_connective list seems at odds with the match statement below it in the existing code, PC.and_lid is mapping to itself in the match but PC.c_and_lid in u_connective. This PR takes the match behaviour which is what executes.
there can be a slight 0.5% runtime degradation on some files; for example the ocaml_extract set. This feels more than offset by the improvement in ulib.

I reviewed this and the changes make sense to me. I don't find the cost in readability to be excessive.
One avenue for improvement is to unify the two tables (at the expense of being more verbose for the base connectors which don't change the lid), and make them a single function, which in turn would eliminate a bunch of code and might even be faster.
Asking @nikswamy and @mtzguido for more feedback here. Thanks.

One avenue for improvement is to unify the two tables (at the expense of being more verbose for the base connectors which don't change the lid), and make them a single function, which in turn would eliminate a bunch of code and might even be faster.

Thanks for the suggestion, I've made this change. I think it is better code and benchmarking shows it's a bit faster (~0.5-1%) than before the table lookups were unified.

This code is clearly trying to remove several kinds of meta data on ast nodes before analysis.
Unmeta_monadic is removing effect annotations.
Unuinst is removing universe instantiations
Pre_typ is removing ascriptions and refinements.
And there's compress, of course.
Aside from compress, it's not clear to me that any of this is actually needed.
Which, if any, are you retaining, Guido?

I only left un_uinst, since I suspected that one makes sense, but not the others: this was my change https://github.com/FStarLang/FStar/commit/67189ab245b524440eb25c3814d0d47864f21f16. (Will try now without the un_uinst, and report back.)

Yup, some early failures in ulib/, e.g. FStar.Real.fsti. The diff in smt2 files shows fragments like this:
 ;; def=prims.fst(425,14-425,15); use=prims.fst(425,14-425,15)
-(= (Prims.b2t (Prims.op_Equality (FStar.Real.real Dummy_value)
+(Valid
+;; def=prims.fst(425,14-425,15); use=prims.fst(425,14-425,15)
+(Prims.eq2 Tm_type
+(Prims.b2t (Prims.op_Equality (FStar.Real.real Dummy_value)
 (BoxReal (/ 1.0
 (BoxReal_proj_0 (FStar.Real.sqrt_2 Dummy_value))))
 (BoxReal (/ (BoxReal_proj_0 (FStar.Real.sqrt_2 Dummy_value))
 2.0))))
 @x3)
 )
+)

So without the un_uinst, destruct_typ_as_formula doesn't properly trigger on equalities.

Thanks Guido, I've updated the PR to remove the pre_typ but leave the un_uninst as in your patch.

Already merged this manually, let's see if github picks it up...


Segfault in computing graph xfers after PR #123

Hey, because of the recent PR #123, quartz segfaults while computing graph xfers from Name_4_3_complete_ECC_set.json (I suspect other ecc files would also segfault). I went through the debugger and found that the assertion assert(node->index < (int)context->input_parameters.size()); on line 165 of tasograph.cpp, fails, and the next line segfaults. I was unable to determine how to fix that :/. I suspect that for creating new parameter nodes, you need to update the context somehow, but I am not sure.

Is this fixed now?

Indeed


Atlas Moving Above the Map when One Player Dies

Mod Version
v<IP_ADDRESS>
Terraria and tMod Version
v<IP_ADDRESS> and v2<IP_ADDRESS>
Terraria and tMod Version
<IP_ADDRESS>
Steps to Recreate

Open a World
Summon Atlas on the Surface
Let one player in Atlas' Aggro radius die
Atlas Rapidly moves above the Map

Further Details
Not sure if Multiplayer,
Multiple Players,
or being in Aggro Radius are required, does not seem to be the case.
Behavior might be dependent on the Depth/Height of Atlas or the Players at time of Player Death (seems to be the case).
Other Enabled Mods
Better Zoom
Shift-Click Crafting
Tome of Research Sharing
Realistic Sky
Concise Mods List
Smarter Cursor
Particle Library
Heart Crystal & Life Fruit Glow
High FPS Support
Recipe Browser
Echo Dye
Fancy Lighting
Instant Platform Fallthrough
DragonVault
imkSushi's Mod
ArmamentDisplay
ShorterRespawnTime
Max Luck
Auto Trash

Fixed incidentally in a prior hotfix.


Toggle button group with same height as other interactive elements

Description
In some interfaces, we want to display a toggle button group together with other input elements. The problem with this is that it is difficult to align the toggle group with other elements because of its irregular height. In Studio we have solved that by manipulating the size using CSS:

This is a quite messy solution which is prone to errors when something change in The Design System. We would prefer to solve this without having to use our own CSS.
Here are some suggestions:

Add a prop to the ToggleGroup component that makes it satisfy this restriction
Add a new component with the same API as ToggleGroup, but with a different design
Redesign the ToggeGroup component so that it always satisfy this restriction

Additional Information
Here is an example on how this is solved visually in Windows:

I personally like this solution because the buttons have the same height as the whole toggle group (and thereby the other buttons in the interface), while it's still clear which of the two buttons that is the selected one.

Thanks for the suggestion!
I can definitely see the use for this. Not sure what would be the best solution for this. I think the intended use case for ToggleGroup was that it was more freely placed instead of in a "toolbar" kind of setting.
We'll look into it at least :)


configure dynamic inventories of your Azure resources using Ansible

Hello
I am working on dynamic inventories ansible taging using Bash.
I had added the Microsoft article link which i followed for above process.
https://docs.microsoft.com/en-us/azure/ansible/ansible-manage-azure-dynamic-inventories#prerequisites
so i followed all steps and able to connect the machine after that when i am trying to install nginx using YML so it generated plugins errors.  i have attached  error image below.
Please help us out in this,

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

ID: d2b8e92b-a339-3869-9c78-e99cb04b94ee
Version Independent ID: f26cb51b-3a0f-ff82-1b92-03084b52fd56
Content: Tutorial - Configure dynamic inventories of your Azure resources using Ansible
Content Source: articles/ansible/ansible-manage-azure-dynamic-inventories.md
Service: ansible
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@onkarthg Can you give me the version of ansible and the azure module which you are using

@jakaruna-MSFT  ansible 2.9.1
azure CLI

Got the issue.
Vaule for state in this below line is not from the allowed options.
apt: pkg=nginx state=installed
Allowed options can be found in this Ansible documentation https://docs.ansible.com/ansible/latest/modules/apt_module.html
Instead of installed, we can use present or latest which fits our scenario.
I will update the doc to have present in it.

@onkarthg
I will close this out for now. If you need additional help please let me know and we can reopen and continue.

Hello @jakaruna-MSFT  Thank you your solution is worked.  but i got one more error taging base ansible using azure Cli  when i plugin the module i will explain you whole senario:

I have 3 VM
I was tag all the 3VM
now i want to install the vm2 NGINX using ansible so i followed same document ansible>2.8 step 1 in the document.


Can no longer run hello-kotlin sample application from IntelliJ

After the update to the latest snapshot distro it's no longer possible to run the hello-kotlin sample from Intellij. It fails with:
Exception in thread "main" java.lang.NoClassDefFoundError: kotlin/jvm/internal/Intrinsics
	at samples.HelloWorldKt.main(HelloWorld.kt)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:497)
	at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147)
Caused by: java.lang.ClassNotFoundException: kotlin.jvm.internal.Intrinsics
	at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
	... 6 more

Ilya Gorbunov had the following to say about a similar failure in another project:

I've tried to import your project and seems all dependencies are imported with Provided scope.

This is actually an IntelliJ issue.

so what is the solution?

@Roshankd1 Upgrade to IntelliJ IDEA 2017.1 or superior.

@bamboo thanks for replying, I am actually trying to use this in Android studio 3.1
Is it possible?

Hm, it seems a similar bug is still present in Android Studio 3.1.3:
Error: Could not find or load main class samples.HelloWorldKt

@bamboo exactly i didnt want to load my pc with all the intellij :( and i have Xcode haha


Implement creation of self keys in Java SDK

Implement the ability to create the self keys in the Java SDK

Some progress in PR37 by @VJag

@JeremyTubongbanua You've completed this, correct?

@JeremyTubongbanua You've completed this, correct?

Yes in #29

Great job Jeremy, thank you!


2018-10-07 date is not working.

https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&date=2018-10-07
Getting below error:

{"code":500,"msg":"Internal Service Error","service_version":"v1"}

because of this, I can't even use the API request with start_date and end_date contains above date.
For example this API calls will also not work:
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&start_date=2018-10-07&end_date=2018-10-08
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&start_date=2018-10-01&end_date=2018-10-08
Please fix 7th October.  Looking forward to hearing back from you.

+1
I'm experiencing the same issue.

If anyone needs a list of APODs (eg. from today to 20 days ago), I made a little API that doesn't throw 500 error when it contains 07.10.2018 APOD. If anyone is interested, here's the link: https://apod-wrapper.herokuapp.com/help

This work as a temporary solution. You are the MVP. Thanks @PawelPleskaczynski

Thanks @samyanez94 :)
I will use it for my APOD client as well until NASA (hopefully) solves the problem.

@PawelPleskaczynski , how many requests one can make on that helper?

Well, I haven't tried it yet, but according to this website, I'm limited to 4500 requests/h

@PawelPleskaczynski, awesome! Now we'll have to wait until Nasa Devs will approve your PR.

Well, I haven't tried it yet, but according to this website, I'm limited to 4500 requests/h

Awesome! Heroku wins but NASA APOD API only allows 1000/h ;-)

Yeah, let's hope they'll approve it! It's my first PR ever in my life so I'm pretty excited 😅

This problem has now been fixed.


Maintenance of python_visualization module

Module to maintain
Website: https://school-brainhack.github.io/modules/python_visualization/
Code: https://github.com/school-brainhack/school-brainhack.github.io/tree/main/content/en/modules/python_visualization
Work to be done
Please ensure:

[ ] links, resources and exercises reflect the state-of-the-art
[ ] videos are of good quality and reflect the materials.

If any update / re-write of the module is needed, feel free to suggest alternative material in a comment!
If you think major changes are needed, we suggest you open an issue for new modules.

I'd be happy to take this one!

Hi @amandalin047 , how are you getting on with maintenance of this module? Do you need any help?

Yes yes, I'm also looking at some of the Data Visualization exercise submissions I've received in our Taiwan hub. Will report the updates by the 21st!

@amandalin047 any updates? If all is working we can mark this as resolved!

Hey @htwangtw sorry for being a little bit late! I've checked the video, the exercise instructions, as well as some of the local Taiwanese students' exercise submissions. I think everything's pretty clear! Thanks :)


Add setup and versioneer

Depends on changes in PR #3.
It may make more sense to separate the addtion of versioneer into a separate commit.

The use of versioneer is a bit heavy weight, is it not?

The use of versioneer is a bit heavy weight, is it not?

If we want to be able to avoid explicitly writing the version into the repo, there is no better alternative at this moment in time AFAIK. I completely agree that it is heavyweight, but I don't think that represents enough reason to avoid the capability that it provides.

👍

Just pushed up changes to the setup.py so that now the following works
:white_check_mark:  python setup.py install
:white_check_mark: python setup.py test
I have not addressed:

[ ] lib/nc_time_axis vs nc_time_axis

I think that it's appropriate to discuss the use of versioneer on the Google Groups PEP440 vs SymVer thread which I raised last week, as versioneer defaults to pep440 style versioning.

After the discussion on Friday, I have removed the lib stuff, and squashed so it's all ready for merging :)


Feature: Add more padding to left on details layout

What feature or improvement do you think would benefit Files?
(Since I use translation software, it is possible that the English may be poor.）
I would like to see the horizontal margins of the folders widened a bit.
When using the cursor to expand the selection to select multiple folders or files, the small margins do not allow for good range selection.
（It is difficult to explain, so I am attaching an image.）

Requirements

Double the padding area on the left of details layout

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3007
Comments
No response

Thanks for the feedback, in future please make separate requests different feedback
We're tracking 1 in https://github.com/files-community/Files/issues/8385. You can check there or click subscribe in the right sidebar for updates.
This issue will be open for your seconds request.

Thank you for your feedback, we don't plan to make any changes to the margin at this time, but we'll keep your feedback in mind for the future.


Why does waifu2x use so much system memory?

Is there any way to utilize as much VRAM as possible and minimize RAM usage?
I have 8GB of VRAM but it's only using 1GB of my VRAM and 4GB of my system RAM.
Using a GTX 1070 (Notebook, 2048 CUDA cores) with 8GB GDDR5, i5-7600K @4.8GHz, 16GB DDR4-2400, Windows 10 Pro x64, waifu2x-caffe v1.1.8.

最終的な出力画像のサイズが大きくなると、RAMで確保するバッファのサイズが大きくなります。


Are scripts inside iframes parsed ?

I did the following:
Open http://api.jquery.com/fadein/, then:
      // Second demo from the page:
      this.page.switchToChildFrame(1);
      this.click('a');
      this.page.switchToParentFrame();
      this.wait(10000);

The link is correctly clicked inside the iframe (its color changes), but the jquery associated script seemed to be ignored. This is easily reproducible. So my question is, does PhantomJs or Casperjs has an issue loading script tags inside iframes ?

Hi B2F,
You code don't work, just try this one :

var casper = require('casper').create();

casper.start('http://api.jquery.com/fadein/', function() {
   this.withFrame(1,function(){
        this.click('a');
   });
});
casper.wait(1000);

casper.run();

See it for yourself:
var casper = require('casper').create();

casper.start('http://api.jquery.com/fadein/', function() {
   this.viewport(1024, 768);
   this.withFrame(1,function(){
     this.click('a');
   });
   this.captureSelector('no-click.png', '#example-1 iframe');
   this.page.switchToChildFrame(1);
   this.click('a');
   this.page.switchToParentFrame();
   this.scrollTo(0, 0);
   this.captureSelector('click.png', '#example-1 iframe');
});

casper.wait(1000);

casper.run();

The click.png from my code shows the link colored as visited whereas yours (no-click.png) doesn't.
The issue is still there anyway and the screenshot from the snippet above illustrates the problem.

Hi,
There are 2 troubles with your code :
1/ The iframe on the jquery fadin site is no a really frame so the method "switchToChildFrame" is not efficient (just use : document.querySelectorAll('iframe')[1].contentDocument.body.querySelector('a');)
2/ What is your phantomjs version : under 2.00 there is a bug with Event  https://github.com/ariya/phantomjs/issues/11289

As part of a cleanup effort:
Looks to be a stale help request. Assuming you've moved on to better and cooler things. Please feel free to re-open if this is still an active issue for you and we'll try our best to help out.


hmdb51 test file (before renamed)

Thank you so much for your kind help. My question is do you have the test file before renamed by AllVideos_renamed.txt? This is very important for our experiment, thank you a lot for your kind help. Thank you again for all the helps you have provided, we deeply appreciate it.
Reference:
https://github.com/ZealACMer/seqvlad-pytorch/tree/master/data/hmdb51_splits

I want to fuse the hmbd51 score files provided in this repository with the score files obtained by Coviar(https://arxiv.org/pdf/1712.00636.pdf), however, I found the test files for these two methods are different. So I tried to change  the format of hmdb51_split1 test file in https://github.com/youjiangxu/seqvlad-pytorch/blob/master/data/hmdb51_splits/test_split1.txt to the format of the corresponding test file in https://github.com/chaoyuaw/pytorch-coviar/blob/master/data/datalists/hmdb51_split1_test.txt. When I finished this procedure, I retest the model of coviar by the new test file in accordance with SeqVLAD, then I tried to fuse these two npz score file(provided by your paper and Coviar, respectively) by late fusion(score fusion), but the result is lower than before fusion. Is there any problem with this procedure, can you give me some tips? Thank you very much. I am confused about this situation.

In original paper of Coviar (https://arxiv.org/pdf/1712.00636.pdf), the experimental results show that the combination of Coviar with flow can further boost the classification performance. I believe your classification score file is good for fusion with Coviar, I am stucked here. Thank you so much for your kind help.

Hi，
I think that you could try to fuse the two score files provided by SeqVLAD and Coviar with different weights because sometimes the weights are important to the final performance. And you could perform the grid search to find out the best weights?
Another tip is that I think you could double check the test split you transformed is correct. For example, when fusing these two score file, you could set the weight for your transformed test to 1 and the score file from Coviar to 0, to verify the performance of the transformed test split is the same as that of the original SeqVLAD score file.

My idea is although the training lists of these two methods(SeqVLAD and Coviar) are different, which means the  two models are training on the different data, as long as I change the test file to be the same, and make sure these two models are testing on the same data, the fused classfication performance won't hurt much. Am I right about this?

I think that it would be better to make sure the classification score of SeqVLAD pluses on the same class id as Coviar. For example, if boxing's class id in SeqVLAD and Coviar are the 3th the 5th respectively, you should plus the 3th score value (after softmax) of SeqVLAD on the 5th score value (after softmax), but do not simply change the test list to the same name and the class id.
In addition, I think it is possible to do harm to the performance of the fused classification score due to the difference in the training data of the two methods. But it is hard to say how much influence it has.

Thank you very much for your kind help. Thanks a lot.


Custom Power-up Mixing

This is similar to a bug we had before, where dofirenemy and customcolor would cause you to keep a powerup after taking damage, I suspect the same thing is happening here

This is similar to a bug we had before, where dofirenemy and customcolor would cause you to keep a powerup after taking damage, I suspect the same thing is happening here. I fixed this at some point and I meant to pull request it but idk if I still have it, sorry about that.

The guy said he was using an old version so maybe it doesn't happen anymore?

Sorry, I think that time I use V13C


move modal title down 1px

Closes #840

Coverage remained the same at 86.14% when pulling d2c0caeb69aa7b3bb70bdf63a151346aa99df678 on na3d:move-modal-title into 8db1cad7fb13dd36e4770ca72f7996cf5becd55e on linode:master.


Can't use Chunky Turtle

Describe
I dont know if it is a Bug or Intended. If you Place your Chunky Turtle from your Inventory it becomes a Advanced Turtle from the CC Tweaked Mod. I encountered this, while playing the Modpack "MineColonies: Dimensional Adventure". I changed a few Setting in the Config of AP, but none of them are working as far as I experienced it.
Steps to reproduce
I crafted the normal and Advanced Chunky Turtle. In my Inventory its the the Chunky Turtle or Advanced Chunky Turtle and as soon i place it in the world it becomes a Advanced Turtle from the CC Tweaked Mod.
Multiplayer?
Yes
Version
1.20.1-0.7.40r (Latest 1.20.1)
Minecraft, Forge and maybe other related mods versions
Forge 47.3.11 Minecraft 1.20.1
Screenshots or Videos
No response
Crashlog/log
No response

Thank you for reporting, that is very weird

From the Server or me as a Player?

From the Server or me as a Player?

The server side log


Updates to the Schema release notes

I was about to tag the v1.3 release when I noticed our schema release notes weren't current.  I went through the diff here and made a bunch of edits:
https://github.com/airr-community/airr-standards/compare/v1.2.1..master
I'm going to need some help though. Not sure if I got everything relevant, if my descriptions are correct, etc.

https://docs.airr-community.org/en/v1.3-news/standards/news.html

Do we want to include release notes for the R and python libraries in the schema news? They have their own release notes sections:
https://docs.airr-community.org/en/v1.3-news/packages/airr-python/news.html
https://docs.airr-community.org/en/v1.3-news/packages/airr-R/news.html

How about we make the news file more prominent in the docs? Add to Getting Started? or AIRR Standards? When I got to the docs top page, it isn't clear where to click to read the news.

Do we want to include release notes for the R and python libraries in the schema news? They have their own release notes sections:

That's a good point. I was thinking of this as a top-level news versus just for the schema. I think it's fine to have them separate but we should be able to find them all in at least one place together on the docs site.

How about we make the news file more prominent in the docs? Add to Getting Started? or AIRR Standards? When I got to the docs top page, it isn't clear where to click to read the news.

We could have a Release Notes item between Getting Started and AIRR Standards. Then link to or include all individual release note pages for the schema, python library, R library, and ADC API

I'm done fiddling with it. I made a top level Release Notes section, moved some stuff around, and converted the schema bullet points to numbers in an effort to fix the formatting while preserving the comments.


Add start value in "for" loop

https://github.com/microsoft/pxt/issues/8970
https://github.com/microsoft/pxt-microbit/issues/3837

We need such an update!

@riknoll
Could you review this if you have time?

@sae220 I don't think we can merge this PR. it would break too many existing screenshots and documentation that uses the old for-loop. we've done a lot of iterating over the for-loop internally over the years and we've always run into walls with feedback from either partners or teachers when trying to update it.

@sae220 that might be doable, but we'd probably want to put it behind an experiment flag to start. do you know how to add something to the experiments dialog?

@sae220 change does not support setting the step?

@riknoll
I don't know but I'll refer to https://github.com/microsoft/pxt/pull/6809. Is it OK?

@THEb0nny
This pull request doesn't support the step but I'll make a new pull request support it.


Updated docker-compose versions in installation guide

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[x] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage remained the same at 65.442% when pulling 2d9831646af017958f34dcd08ef1c2c19401c919 on bsekachev-patch-1 into<PHONE_NUMBER>20cb77d582cc599210c72e44c1e55d on develop.

So, changing version name from "2.3" to "3.3" is enough for mount our data?


Fix #2177

Hi Henrik,
this PR addresses 2177.
The code to select a target directory for a copy/move operation was not executed because of an error. That error was due to +popup--delete-other-windows expecting +popup/raise to return a window, whereas it returned a buffer instead.

Great catch! Thanks for looking into this!


Proof of concept: reliable asynchronous processing

Asynchronous processing should be possible for long-running functions.
Must have:

Work can be accepted through a new route or a Header

/async/function/<function_name>

Or via Header:
X-etc: async

Work is accepted immediately and a 202 Accepted is returned. This should be handed off to a queue.

One or more (scaleable) asynchronous workers read from a queue and call functions

Should dequeue item atomically

Upon failure another worker should pick up the item

For initial version - HTTP should be used by worker to call function just like the gateway does. Timeout will depend on the configuration of the function.

Prometheus metrics to be logged for work queued/processed/outstanding

Could have:

Watchdog configuration to state whether async/sync is supported
Validation in gateway for invocation method
Retry logic on failure

Nice to have:

Additional logging beyond docker service logs
Callback URL could be specified via header or query-string - this could be called by the framework upon completion

Notes:
Have looked into Kafka - design looks overly complex for task at hand.
NATS queuing is not resilient - but NATS Streaming may be suitable.

If you need a beta tester for asynchronous processing, I am in !

Thanks @Tofull - I've started a quick proof of concept with NATs streaming.
Have you started using FaaS or creating functions already? Do you have async workloads ready for testing?

Amazing ! You roks ! :)
I used FaaS to deploy some functions that my machine learning experts made with magic.
FaaS works great for prediction service as it is a synchronous task.
As some processing functions need time (training our models), we would use async processing and we already have a workflow ready for testing.

That sounds like a great use-case. Is there anything you can share on a blog or on Twitter?

When our developments will gain in stability, we will be able to communicate and mention FaaS as the solution we decided to use in a tweet or some presentations we will make ('cause we are working with French industries in aerospace :artificial_satellite: & :earth_americas: earth observation fields).
FaaS with async should completely meet our needs, here at the Institute of Technology Saint Exupéry. :smiley:

It would probably help to map out the use cases for the async processing first as there are a couple different ones I can think of that usually require different guarantees and metrics. anyone know what the users of this library would favor in use cases for this as this would ease up choosing the right queueing options here too. kafka for example might seem overly complex (and it is complex) but it has its uses, but usually I wouldnt choose that for simple response queues like here generally. nats is nice general purpose, but i fear doesnt encompass all options. there is also the possibility of a mixed solution making it simple pub/sub with separate log database (usually you need recent stuff, which is in mem, but you dont lose old stuff this way and support stuff like "oh my car is offline for 15min because of shit internet, but can still get its response"), which gives quite a lot of flexibility and isnt that hard to implement.
problem with using queuing systems like nats is that you end up having a ton of a ton of queues piling up. While this is completely acceptable in your infrastructure for workers and services as its quite limited, when it comes to response queues, thats not so feasable really. mqtt suffers from similar problems in the end when load gets high. seen a couple of implementations that offload mqtt queues to databases though.

As you have mentioned - the various queue implementations available have their own pros/cons. Ideally it should be easy to swap between different "queue" providers or implementations.

ah i must have missed that idea - that sounds like the best possible outcome, yes :)

Hey @Tofull do you have a draft or published blog yet?

Please see changes in #131

Work merged into master and released in 0.6.3 https://github.com/alexellis/faas/releases/tag/0.6.2
If anyone wants to start on a Kafka implementation that would be great.


Add automated rendering from youtube input?

I am a hobbiest researcher with limited time;  i.e I have a full-time non-research job :-(.
Could you enhance you simulator so that it could render continuously automatically from an online URL such as a youtube video or a video file / video stream?
I want to work on creating visual / augmented modelling of word understanding based on SLAM cotextual recognition of a small set of every important words.
See https://hemseye.org/wp/2018/08/26/hemseye-project-phase-1-shorter-presentation/
Further enhancments would be adding a sync / timestamp to integrate the video sound output.
Many thanks
regards
James Kitching

Hi,
This will not be possible, as the main goal of ESIM is to implement a tight coupling between the rendering engine and the event simulation, to ensure that the maximum pixel displacement between successive frames is below a threshold (say, 0.5 pixel). Details are in the paper.
Simulating events from successive frames streamed from YouTube would be possible, but there would be no guarantee that the simulated events are accurate since you would not be able to control the displacement between the received frames, therefore I would advise against it.
If you still want to do it, I would suggest not to use ESIM (which has heavy machinery designed to implement the guarantee mentioned above), but rather the simpler event simulator written in Python available here.


Open in Visual Studio Code doesn't work in pantheon

Describe the bug
clicking on a file or folder in pantheon and selecting "Open in Visual Studio Code" doesn't work and shows "Could not open files or URIs with this app"
To Reproduce

right-click on any file or folder
select Open in Visual Studio Code

Expected behavior
the file or folder will be opened in vs code
Screenshots

Additional context
I think I could open files in vs code before I switch to pantheon
Notify maintainers
@eadwu @Synthetica9
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.5.17, NixOS, 20.03.1445.95b9c99f6d0 (Markhor)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.4`
 - channels(hussein): `"home-manager-20.03"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
  - vscode
module:

I think I now know how to solve, we just need to add %U to the exec property in desktopItem in vscode derivation
I can do a pr for this if it is okay @eadwu @Synthetica9

Feel free to do, I don't use pantheon so I can't really test it, though I'll try shooting up a VM.

Seems fine, probably also add it to VScodium

Sorry for the late response
i have made the pr #88623
@Synthetica9

Hi, maintainer of pantheon in NixOS. Thank you for fixing this.


Dropdown doesn't close properly on iOS

Hello,
I have noticed a bug when trying to close a dropdown, if you tap on the red div, it won't actually close the dropdown (viewport > 767)
http://output.jsbin.com/waziduquhe/1
http://jsbin.com/waziduquhe/1/edit?html,output
I tested on Ipad mini with IOS 9.3.1
Thank you,
Steve

Hi @stephenbe!
You appear to have posted a live example (http://jsbin.com/luwiyataru/edit), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 54, column 7 thru column 58: Bad value search for attribute role on element form.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Fixed :
http://output.jsbin.com/waziduquhe/1
http://jsbin.com/waziduquhe/1/edit?html,output

Known issue, see http://getbootstrap.com/getting-started/#navbar-dropdowns

Alright thank you!


Upgrade to Hubot 3 and fix dependency vulnerabilities

This is a combination Hubot upgrade and cleanup:

Upgraded to the latest version of Hubot
Removed deprecated hubot-scripts.json (the ones we actively use have been added to the scripts directory)
Rewrote part of scripts/pg-brain.coffee to fix PostgreSQL client library deprecation
Forked zulip-node and hubot-zulip, both of which appear to be unmaintained, to update their dependencies in order to fix several major vulnerabilities
Updated the MIT license copyright

I'm ending this pull request's tyranny once and for all


Try to fix broken e2e tests.

Dependency update and corresponding change after: https://github.com/giantswarm/e2e-harness/pull/134

Self merging because this is just dependency update and related change.


[BUG] Global code coverage not accessible when using toIstanbul and specifying a subDir for the html reporter

Hi @cenfun,
I've noticed that if I specify an html or html-spa reporter to use with the toIstanbul code coverage option then when I try to open the global code coverage it results is a 404 Not Found:

What I have noticed

The global code coverage link seems to expect that the coverage html file exists inside the coverage folder and be named index.html. I get a 404 Not Found because it tries to fetch the global code coverage from http://localhost:8090/coverage/index.html but it should try and fetch it from http://localhost:8090/coverage/<configured istanbul reporter subDir>/index.html.
When specifying a subDir for the istanbul reporter options then all the outputs are created inside that subDir inside the coverage folder. Example when I set the subDir to code-coverage-html-spa-report:

How to reproduce

Clone the tag temp-1 from my playwright-adventures repo.
With your favourite editor go to playwright.config.ts and uncomment line 55 // toIstanbul: instanbulReporters, and line 29 // subdir: "code-coverage-html-spa-report.
Use your favourite shell to go to ./demos/code-coverage-with-monocart-reporter.
Run npm i to install all required npm packages.
Run npm run test to run the playwright tests.
Run npx monocart show-report tests/test-results/index.html and try to browse to the global code coverage and you should get a 404 Not Found page. You should also see that the html report has been created at ./tests/test-results/coverage/code-coverage-html-spa-report.

Further notes

This is happening on version 1.7.13 of the monocart-reporter package and using @playwright/test version 1.40.1.
I'm tracking the code coverage using playwright's code coverage functions. These are being invoked through an automatic fixture defined at app-fixtures.ts and v8-code-coverage.ts.

This issue could potentially be fixed by addressing #69

please try<EMAIL_ADDRESS>The new coverage reporter can generate multiple reports
// playwright.config.js
module.exports = {
    reporter: [
        ['monocart-reporter', {
            name: "My Test Report",
            outputFile: './test-results/report.html',
            // coverage report options
            coverage: {
                outputDir: './coverage-reports',
                reports: [
                    ['v8'],
                    ['html', {
                        subdir: 'istanbul'
                    }],
                    ['json', {
                        file: 'my-json-file.json'
                    }],
                    'lcovonly'
                ]
            }
        }]
    ]
};

more details monocart-coverage-reports

Looks awesome, the refactored interface for this also looks great. You've done and keep doing an outstanding job with the monocart-reporter @cenfun. I've mentioned it once to you in the past but I'll say it again, I'd definitely donate to this repo if you managed to enable GitHub Sponsors on the repo.
I'll test v2 and let you know soon.

It's working flawlessly!
Given this playwright.config.ts:
// playwright.config.ts

const _testsDir = path.resolve('./tests');
const _testsOutputBaseDir = path.resolve(_testsDir, 'test-results');
const _monocartReportOutputFile = path.resolve(_testsOutputBaseDir, 'monocart-report.html');
const _monocartCodeCoverageBaseDir = path.resolve(_testsOutputBaseDir, 'code-coverage');
const _v8RelativeFilePath = 'v8/index.html';

export default defineConfig({
  testDir: _testsDir, // set to './tests'
  outputDir: _testsOutputBaseDir, // set to './tests/rest-results'
  ...
  reporter: [
    ['list'],
    [
      // See https://github.com/cenfun/monocart-reporter
      'monocart-reporter',
      {
        name: 'playwright code coverage with monocart reporter',
        outputFile: _monocartReportOutputFile, // set to './tests/test-results/monocart-report.html'
        coverage: {
          outputDir: _monocartCodeCoverageBaseDir,  // all reports in this dir.  Set to './tests/test-results/code-coverage'
          outputFile: _v8RelativeFilePath,  // v8 sub dir and html file name, relative to coverage.outputDir. Set to 'v8/index.html' which means it will be generated at './tests/test-results/code-coverage/v8/index.html'
          reportPath: path.resolve(_monocartCodeCoverageBaseDir, _v8RelativeFilePath),
          reports: [
            ['v8'],
            ['cobertura', {
              file: 'cobertura/code-coverage.cobertura.xml',  // path relative to coverage.outputDir. The file will be generated at './tests/test-results/code-coverage/cobertura/code-coverage.cobertura.xml'
            }],
            ['lcovonly', {
              file: 'lcov/code-coverage.lcov.info',  // path relative to coverage.outputDir. The file will be generated at './tests/test-results/code-coverage/lcov/code-coverage.lcov.info'
            }],
            ['html-spa', {
              subdir: 'html-spa-report',  // path relative to coverage.outputDir. The html report will be created inside the './tests/test-results/code-coverage/html-spa-report' folder
            }],
          ]
        }
      }
  ]
  ...
});

I get the following:
tests/
└── test-results/
    └── code-coverage/
        ├── assets/
        │   ├── monocart-code-viewer.js
        │   ├── monocart-coverage-v8.js
        │   ├── monocart-formatter.js
        │   └── turbogrid.js
        ├── cobertura/
        │   └── code-coverage.cobertura.xml
        ├── html-spa/
        │   ├── ...(several files for the html report)
        │   └── index.html
        ├── lcov/
        │   └── code-coverage.lcov.info
        └── v8/
            ├── coverage-data.js
            └── index.html

I have a question though:
What do you think about removing the coverage.outputFile and move that option inside the coverage.reports[v8] report options? That to me would make a bit more sense and keep it inline with the other reporters?
So that instead of the playwright.config.ts config shown in the comment above you would do:
// playwright.config.ts

export default defineConfig({
  testDir: './tests',
  outputDir: './tests/test-results'
  ...
  reporter: [
    ...
    [
      'monocart-reporter',
      {
        name: 'playwright code coverage with monocart reporter',
        outputFile:  './tests/test-results/monocart-report.html'
        coverage: {
          outputDir: './tests/test-results/code-coverage'
          reportPath: './tests/test-results/code-coverage/v8/index.html'
          reports: [
            ['v8', {
              subdir: 'v8/index.html' // Use this instead of coverage.outputFile
            }],
           ...
          ]
        }
      }
  ]
  ...
});

Any thoughts? Note that the current implementation works fine, this is just a suggestion since in v2 you've refactored the code coverage options.