function-order: have a way for a user to ignore certain functions

There are a number of cases where one or more functions in a contract may be in a different order than the recommended visibility order due to logical grouping; one example is an initialize() being next to a constructor rather than after a large number of other functions.
It would be nice to have some way to configure an ignores list, or even ignore a function from the rule if the preceding line is /* solium-disable-next-line function-order */ (this sounds hard to get right though).

@sohkai Correct me if I'm wrong. My understanding is that you would like to force function-order to NOT report a particular function (like initialize()) if it is out of order, right?
If that's what you want, it is already possible. For eg, I just took your ACL.sol from dev to see which functions were being reported by function-order:
$ solium -f acl.sol

acl.sol
  77:4     error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  90:4     error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  119:4    error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  132:4    error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  144:4    error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  158:4    error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order
  170:4    error    Functions should be in order: constructor, fallback, external, public, internal, private    function-order

✖ 7 errors found.

Now if I add // solium-disable-next-line function-order right above the line 77 (declaration of createPermission()), then re-run solium, this function is no longer reported.
Does that solve your problem?

@duaraghav8 Slightly different; in the case of the ACL, I'd like to remove the offenders (initialize() and grantPermissionP()) entirely from the rule rather than the functions that are in the right order.
I'm thinking of it as subtracting those functions from the set of functions that are used to evaluate the rule.

I think I got the problem.
Say you have functions
A private()
constructor()
B public()

and you'd like to keep A on top, which causes the rule to report the remaining 2 functions because the rule thinks they're not in the right order. The effect of ignoring function A means it is invisible to function-order, so the rest of the functions are seen to be in the right order, so no errors are reported.
Is that correct?

Yes, that's the idea :pray:!

Here's the configuration that I'm planning:
{
    "rules": {
        "function-order": [
            "error",
            "ignore": {
                "constructor": true,
                "functions": ["foo", "myFunc"],
                "visibilities": ["private"]
            }
        ]
    }
}

By default, the arrays are empty and constructor is set to false, which means NO functions are ignored.
If you specify a visibility in visibilities, all functions of that vis. are ignored.

Not entirely sure how solium handles option setting via comments in files (can it?), but it'd be great if this were supported as well on a per-file basis that way.

Currently not possible, but I can see how per-file configuration is highly relevant in this case. I'll try and add this before next release

The original feature request for adding ignore option in function-order has been solved in v1.2.0. The issue will still remain open because its topic has changed.
This linter is now being published simultaneously to npm packages solium and ethlint. Although not mandatory, I highly recommend that you move to ethlint.
See changelog


E2e cv blog post part 3

Part 3, depends on https://github.com/iterative/dvc.org/pull/3482 and https://github.com/iterative/dvc.org/pull/3483
Target publish date: May 5rd

[ ] Still waiting on Header image corrections from the graphic designer.
[ ] Need to link to Part 1 and Part 2.
[x]  There are no grammatical errors, jargon, or obvious stylistic issues. (For reviewer)
[x]  Have images been run through tiny.png?
[x]  Are Oxford commas observed?
[x]  Content Creator has done a complete review for typos/language after taking a break from content
[x]  links checked in the piece
[x]  Have reviewed what the piece looks like on your own machine using yarn develop

yup, see: https://github.com/iterative/dvc.org/pull/3483#issuecomment-1113183937

Same on this one.  No deployment. cc: @rogermparent

Same on this one. No deployment. cc: @rogermparent

Hello! Deployment is working, but instead of showing at the bottom of pr, it shows it a comment made by gatsby-cloud bot.

@dberenbaum fyi, there were a few duplicate comments from you, so I deleted them.


Fix: go module & update dependencies

Codecov Report

Merging #1219 (a84a9f1) into master (c78ee5a) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1219      +/-   ##
==========================================
- Coverage   44.80%   44.67%   -0.14%
==========================================
  Files         485      476       -9
  Lines       29505    29406      -99
==========================================
- Hits        13221    13137      -84
+ Misses      14884    14866      -18
- Partials     1400     1403       +3

Impacted Files
Coverage Δ

testing/servers/tcp/tcp.go
81.81% <0.00%> (-5.46%)
:arrow_down:

transport/internet/udp/hub.go
73.61% <0.00%> (-4.17%)
:arrow_down:

transport/internet/system_listener.go
50.00% <0.00%> (-4.00%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

testing/scenarios/common.go
70.68% <0.00%> (-3.45%)
:arrow_down:

transport/internet/kcp/receiving.go
97.31% <0.00%> (-2.69%)
:arrow_down:

transport/internet/system_dialer.go
46.83% <0.00%> (-2.54%)
:arrow_down:

transport/internet/kcp/connection.go
69.61% <0.00%> (-2.21%)
:arrow_down:

proxy/vmess/outbound/outbound.go
66.36% <0.00%> (-0.91%)
:arrow_down:

transport/internet/udp/hub_other.go

... and 8 more

Continue to review full report at Codecov.

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


Add HuggingFace support for Quality Classifier

Once our quality classifier is uploaded to HuggingFace, we can add support for it like we did for the domain classifier in https://github.com/NVIDIA/NeMo-Curator/pull/166.
Related issues: https://github.com/NVIDIA/NeMo-Curator/issues/71 and https://github.com/NVIDIA/NeMo-Curator/issues/72.

I will start working on this now that we have https://huggingface.co/nvidia/quality-classifier-deberta


Allow users to choose providers from front page

It's frustrating to have one big search box and not be able to choose providers – I generally don't want to search everything and want to choose a few providers.

Closing bc this is covered in https://github.com/creativecommons/cccatalog-frontend/issues/297


data.properties from ArrowTables

This PR relates to #215
Create functionality for the use case where an incoming table object is on arrow format containing UNSMRY data.
This involves also activating usage of the data.properties field which is defined in the schemas, but currently not used.
The use case represents a different pattern compared to existing cases, as the incoming table is actually a blend of multiple data objects packed into one file. We want, however, to keep it that way. The data.properties field was originally defined for a future use case of having multiple attributes to one surface object, which is relatively similar to the table use case.

@sago64, @rnyb as discussed off-line.

Changed from content: timeseries to content: unsmry.

Timeseries sounds more like a datatype (class) than a content. To align with previous pattern (e.g. "field_outline", "inplace") it makes more sense to explicitly say "unsmry". Saying "unsmry", not "summary", to further specify a very specific type of content. This can technically be both table and timeseries (should we include "timeseries" as a class later).

Ref offline discussions, we put this on hold pending further discussions. If the data.properties is introduced, it should be generic enough to scale across different data sources. E.g if we create something specific for unsmry, it should probably go under a separate attribute.


editor tab tests: modal input fields

requires #54
Input is a button:

clicking button displays a modal

each modal has "cancel" and "save changes" "buttons"
modal uses resource template (does this mean it should be a component test, not an end-to-end test?  Or do we do an end-to-end version of the test here, exercising the resource template code?)
check that the resource template functionality works (e.g. input fields, other modals)

last nested modal button clicked has modal on top

e.g.  If we click Title (from Monograph instance), it displays a modal;  that modal has a Note button, which displays another modal on top.
See loadedProfile.test.js TODO comments. (2 of them)

no modals!


(maint) Remove Debian 6 from pipelines

Debian 6 has reached EOL and should be removed from all build/test
pipelines

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-master/617/
Test PASSed.

@melissa can you target this at stable? I think we're good to go to merge it otherwise.

@melissa reopend as https://github.com/puppetlabs/puppetdb/pull/1932 against stable

@senior thank you so much!


How to connect DBeaver to sqlite3 db through ssh tunnel

System information:

Client Windows 10 / Server Raspbian 10 rpi3
DBeaver version 6.2.1 release 23/09/2019

Connection specification:

Database SQLITE3 through SSH

Describe your question:
Hi there!
I'm trying to connect on sqlite3 database through ssh on remote server.
My ssh tunnel is working, but what is the syntax and where paste it on the db.sqlite3 file in DBeaver.
DBeaver connection informations :

SSH connection informations:

Hey. I'm have the same issue with error "Can't initialize tunnel
Database port not specified and no default port number for driver 'SQLite'"

I've the same issue

If somebody can help us it will be awesome...

You cannot use SSH with SQLite.
SQLite is embedded database, there is no such thing as SQLite server.
So SSH doesn't make any sense for SQLite.

So there isn’t any way to manage sqlite db from remote...

SQLite is embedded database, there is no such thing as SQLite server.

This is true, but something like git over ssh might be possible. Could send commands using sqlite3 (as in the command line interface for SQLite) in the host machine to manipulate an sqlite database over ssh without any actual server at all other than the ssh deamon.
Just my uneducated thought, im not even sure inserting binary data is even possible with something like that.

SQLite is embedded database, there is no such thing as SQLite server.

This is true, but something like git over ssh might be possible. Could send commands using sqlite3 (as in the command line interface for SQLite) in the host machine to manipulate an sqlite database over ssh without any actual server at all other than the ssh deamon.
Just my uneducated thought, im not even sure inserting binary data is even possible with something like that.

That's a good point. I'm just thinking DBeaver is limited to JDBC use. But it probably should be possible to create a custom JDBC driver sending SQL to a ssh sqlite3 pipe.

From what I understand the 'Path' field expects a simple path (no IPs etc.). So I managed to solve (on linux) by:

Establishing a SFTP connection with the server (easily doable with the UI, on Ubuntu 'Other Locations' -> 'Connect to server' -> sftp://<server_name_or_ip>:<ssh_port>)

Creating a link from the local computer to the server root (ln -s <remote_dir> <link_name>).

Note: the remote_dir (i.e. the server root directory) is created when connecting with SFTP; mine, for example, is located at /run/user/1000/gvfs/<server_name> . The link is needed because that folder has weird chars/spaces in it, which DBeaver rejects.
So for example: let's say that your SQLite db is located at /projects/db.sqlite on the remote server and the link that points to /run/user/.... is located at /home/john/sv_root on your local machine. The final path you'll have to put in DBeaver is /home/john/sv_root/projects/db.sqlite.

I can confirm that @Kavarenshko's workaround works

Another solution for mac/linux is to use sshfs, which mounts a folder via ssh.
sshfs username@hostname:/remote/directory/path /local/mount/point -ovolname=NAME

(source)

I have tried the solution posted by @Kavarenshko but I get an error saying path to '/home/user/remote_root/projects/project1/group1/sqlite3database.db': '/home/user/remote_root' does not exist.

The Tunnel Configuration Test results in "Connected!"
I was able to mount the remote root directory with: sftp://<server_name_or_ip>:<ssh_port>
I made a link to the root directory in my local home directory with: ln -s <remote_dir> <link_name>
I then navigated to my db file through the link to get the path and entered that path in the path field of the Main tab.
I get a message saying that there is no defaut port, so I click the Edit Driver Settings button and enter 22.
Test connection again, and then get the "path does not exist" message.

Thanks in advance for any help!

As, @philshem said, try sshfs.

i'm using this great tool because i'm using sqlite db in iot devices.
sqlite-web
https://github.com/coleifer/sqlite-web

So there isn’t any way to manage sqlite db from remote...

Is this still the case using DBeaver in 2022? I just wanted to view the sqlite file on my Linux computer remotely from my Windows laptop. Thank you.

I think we can try implementing the approach proposed by @Kavarenshko.

Implemented:

@ShadelessFox Hi there! In which version it's available ? In my versions 22.2.1 on Mac and Windows I don't see SQLite Remote in possible connections.

Hello @iTem186,
The SQLite Remote driver is only available in DBeaver PRO versions which aren't yet released.
You can download early access versions here: https://dbeaver.com/files/ea/

has this been scheduled for a release version yet?

Scheduled for 22.3.

Are there any plans to make the SQLite Remote driver available in DBeaver Community Edition? or will this remain in PRO versions only?

We don't have plans on making the driver available in non-PRO products.
You can try a 14-day free trial: https://dbeaver.com/download/enterprise/

Hi friends. I had the same issue, while waiting for SQLite remote in nom-PRO Dbeaver, my workaround was mount a NFS Filesystem between the machines, sharing de same SQLite db file. It works fine!

use sshfs
however mount it to somewhere in your ${HOME} and not i.e. under /mnt
dbeaver will not be able to open it otherwise


[BEAM-8223] Fix version number in gradle.properties for 2.16 release

We should not include -SNAPSHOOT and .dev in version number for our release. This is a manul fix to 2.16.0 release branch. We should also fix the release automation script in the future.
+cc: @Hannah-Jiang

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

I agree that we need to make the version consistent. We wanted to tackle it later, not as part of 2.16, so I added a new variable(python_sdk_version). If you confirmed all the impact of this change, I am OK with this change.

Synced with Hannah offline. It's not about consistent naming between java's version and python's python_sdk_version. The problem this PR trying to fix is that we should not use dev version number in our release.
Test failure comes from Dataflow service side. I'll go ahead and merge it.


Unable to import from local path

Issue
Files not importing from local import folder.
To reproduce
Rename import folder in JMMServer from UNC location back to local disk / path like for example:
E:\some\anime\
run import files from server and will get this error:
 MainWindow.workerImport_DoWork => Importer.RunImport_NewFiles System.Exception: Import Folder not found 'E:\some\anime'
Do notice that the extra trailing slash gets trimmed off at some point during import, confirmed that local path is accessible from server and should be able to resolve it.
Possible cause is that Nutzcode filesystem resolver below fails to validate path:
https://github.com/japanesemediamanager/jmmserver/blob/master/JMMServer/Entities/ImportFolder.cs#L68
This is with latest master branch and debug build selected.

When remove the trailing slash from import folder it no longer reports this error.

Will check for trailing hash in the resolve code in nutzcode, if found
remove...
On Thu, Oct 6, 2016 at 1:31 PM, Rick164<EMAIL_ADDRESS>wrote:

When remove the trailing slash from import folder it no longer reports
this error.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/japanesemediamanager/jmmserver/issues/460#issuecomment-252016744,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHDbUYQ_2keNX-z72xiIj__2TJ16heIks5qxSJVgaJpZM4KQH87
.

Wasn't this also reported by @hidden4003 I could be wrong about person that reported it but I cleary remamber that something mention same. It saw related to driver path like X:\ and the removing tailing would remove it and then jmm would try to check for X:anime insted of X:\anime

Can someone who experienced this verify if it is fixed


test

Hello Jadoo4QFan! i would like to make controbutions to your branch.
IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

ADD DESCRIPTION HERE
Fixes #
Template

Before submitting a pull request make sure you have:

[ ] At least skimmed through contributing guidelines including yt-dlp coding conventions
[ ] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

yep i have write permission


Update textstat.py

could reuse the function

@halidziya Thank you for this, and congratulations on your first pull request! 🎉


Improve TypeScript integration in web projects

Add new item doesn't show .ts file
Copy and paste an existing type script file and the Build Action becomes Content instead of preserving TypeScriptCompile
Changing Build Action to TypeScriptCompile requires a rebuild all for errors to go away (could be a TS issue)
No TypeScript project properties page show up in project->properties.  Might require work w/ the TS team or maybe just register our project guid as being compatible somewhere.

We're closing old issues with no recent activity. If this is still important, please open a new issue.


[spec failure] adapter_memcached_native_with_default_expires persists values

 1) adapter_memcached_native_with_default_expires persists values
     Failure/Error: store[key1].should == val1
       failure: 12 tests, on:
       ["q257OtBFZL", "oWQDSy"]
       expected: "oWQDSy"
            got: nil (using ==)
     Shared Example Group: :persist called from ./spec/helper.rb:243
     # ./spec/features/persist.rb:9:in `block (4 levels) in <top (required)>'
     # ./spec/features/persist.rb:8:in `tap'
     # ./spec/features/persist.rb:8:in `block (3 levels) in <top (required)>'
     # ./vendor/bundle/ruby/2.1.0/gems/rantly-1.1.0/lib/rantly/property.rb:33:in `call'
     # ./vendor/bundle/ruby/2.1.0/gems/rantly-1.1.0/lib/rantly/property.rb:33:in `block in check'
     # ./vendor/bundle/ruby/2.1.0/gems/rantly-1.1.0/lib/rantly/generator.rb:83:in `call'
     # ./vendor/bundle/ruby/2.1.0/gems/rantly-1.1.0/lib/rantly/generator.rb:83:in `generate'
     # ./vendor/bundle/ruby/2.1.0/gems/rantly-1.1.0/lib/rantly/property.rb:31:in `check'
     # ./spec/features/persist.rb:3:in `block (2 levels) in <top (required)>'
     # ------------------
     # --- Caused by: ---
     #   expected: "oWQDSy"
     #        got: nil (using ==)
     #   ./spec/features/persist.rb:9:in `block (4 levels) in <top (required)>'

Seems to have gone away with the update to minimum MRI requirements

Nope, hasn't gone away:
  1) adapter_memcached_with_default_expires persists values
     Failure/Error: store[key1].should == val1

       failure: 46 tests, on:
       ["sDFaJlQCw/IRBgaujnp", "VfSTuX"]

       expected: "VfSTuX"
            got: nil (using ==)
     Shared Example Group: :persist called from ./spec/helper.rb:274
     # ./spec/features/persist.rb:9:in `block (4 levels) in <top (required)>'
     # ./spec/features/persist.rb:8:in `tap'
     # ./spec/features/persist.rb:8:in `block (3 levels) in <top (required)>'
     # ./vendor/bundle/ruby/2.3.0/gems/rantly-1.1.0/lib/rantly/property.rb:33:in `block in check'
     # ./vendor/bundle/ruby/2.3.0/gems/rantly-1.1.0/lib/rantly/generator.rb:83:in `generate'
     # ./vendor/bundle/ruby/2.3.0/gems/rantly-1.1.0/lib/rantly/property.rb:31:in `check'
     # ./spec/features/persist.rb:3:in `block (2 levels) in <top (required)>'
     # ------------------
     # --- Caused by: ---
     #   expected: "VfSTuX"
     #        got: nil (using ==)
     #   ./spec/features/persist.rb:9:in `block (4 levels) in <top (required)>'

This issue was caused by the fact that memcached expires entries at the beginning of each second.  See https://github.com/memcached/memcached/issues/307.
This is resolved in 2994f49f66783599475692624d521088ee49043f (and some subsequent commits) by introducing the "time resolution" and "min TTL" concepts.  For most stores, these are the same, but for stores like memcached, the min TTL needs to be double the time resolution.


give others bots with /minion???

my friend showed me the commands he had on http://louisagar.pw and he could kill players, change minion names, skins and give minions to other players. how would i be able to that that on my server?

You can already give people minions using playercommands, along with changing skins. As for killing players you'll need to port the code from CommandList.js to PlayerCommand.js.


ArchiveBox looks beyond the capture for embedded resources

Describe the bug
ArchiveBox produces an inaccurate representation when using an archival crawler basis.
As a very quick background, I created a "tool" a few years back to evaluate archival crawlers. See https://ieeexplore.ieee.org/document/6970146 and the free PDF for more information. This "Archival Acid Test" consisted of an exhibition of some basic and advanced features of the Web that were problematic for archival crawlers. This was sort-of in the spirit of the Web Standards Acid Tests for early browsers.
I tested ArchiveBox 552734241bacc8e75ccf6a81292e15166af58140 by running echo "http://acid.matkelly.com" | ./archive once installed and received a sub-par result when accessing the files on my local file system. To give it the benefit of the doubt, I ran a local server via php -S localhost:8090 and accessed the capture. The result looked something like:

Examining the console, I can see that there are some requests for live Web resource representations outside of the archive, which is one such test on evaluating archival tools as performed in the 2014 study.
Steps to reproduce
Steps to reproduce the behavior:

git clone https://github.com/pirate/ArchiveBox.git
cd ArchiveBox
./setup
echo "http://acid.matkelly.com" | ./archive
cd output
php -S localhost:8090
Open browser to http://localhost:8090/
Click the icon in the HTML column for the lone capture.
Disconnect the Internet and fire up localhost again to note many broken images, a further indication that the captured page is looking to the live Web for resources -- screenshot below.

Screenshots or log output
Viewing capture w/ disconnected Internet

Software versions

ArchiveBox version: 552734241bacc8e75ccf6a81292e15166af58140
Python version: Python 3.7.2
OS: macOS 10.14.3
Chrome version: Google Chrome 72.0.3626.119

Other
Note that I am not using this ticket to promote my tool but I hope it can be used to improve the capture quality. The source for the AAT is available at https://github.com/machawk1/archivalAcidTest .

ArchiveBox is composed of 9 archive methods, wget is only one of the 9 used.  This test is indeed an effective way to test an archive method, but you should check out the other outputs, as the screenshot and PDF versions will likely have all blue squares given they're rendered with a headless browser.
You're effectively just running the test on the effectiveness of this wget command, and not the entire ArchiveBox output:
wget --server-response
     --no-verbose
     --adjust-extension
     --convert-links
     --force-directories
     --backup-converted
     --span-hosts
     --no-parent
     -e robots=off
     --restrict-file-names=unix
     --timeout=60
     --warc-file=warc.gz
     --page-requisites
     --no-check-certificate
     --no-hsts

Future versions will save all archive method requests & responses (including wget & headless browser) together in a single WARC using pywb.  Once that's released, running the acid test against the WARC proxy should return 100% success, as it'll be capturing a perfect replica of Chromium's experience when rendering the original page.

Thanks for the info, @pirate. I did not realize that the default capture method used wget. The PDF and screenshots are not nearly as useful to me as having an interactive web page.
I am looking forward to the future integration with pywb for capture. With the current version, how would I go about using 1 of the other 8 archive methods to generate a capture with a little more fidelity than wget?
I attempted to work through the wiki instructions but things start to get convoluted with the Docker and ArchiveBox configuration example.

It does all methods by default, just click the favicon next to the title on the left to see the link index page.


create a bower component

hello, could you do a bower component? Would be super useful. http://bower.io/docs/creating-packages/

I agree, but I don't yet understand semver or git tagging and right now don't have the time to learn it, sorry. Will do when I get a chance to pick it up.

I know it's not the same but for now you could just grab the file directly through bower:
"dependencies": {
    "jquery-disablescroll": "https://raw.githubusercontent.com/ultrapasty/jquery-disablescroll/master/jquery.disablescroll.min.js"
}

It's super easy to create a Bower package and you can create tags right in the GitHub UI. http://bower.io/docs/creating-packages


Add tests for PackageReference projects

This PR adds tests for projects that use PackageReference instead of packages.config. Some of them are failing as noted in daveaglick/Buildalyzer#33.

Thanks! I may end up commenting these out in master and then uncommenting them when I work on the issue (so don't panic when you see I've turned the new tests off - I'm working on them locally).


[#548] Few tweaks around cancelling tasks to reflect the status accurately

Fixes #548
Leveraged existing variables taskCancelling and taskCancelled to do the tweaking and appropriately reflect the correct status of Cancel.

testing...

Validated # 1 and # 3 are working as expected (in 548). For # 2 - do you want the task status to remain in the status message @vconzola? I could see this still being useful. The spinner is showing as expected now though during cancellation.
I am not able to test # 4  locally yet...

@priley86 Yes, I agree. Keep the status. But can you change the order to "{Status} - Cancelling migration"?

so discussing this w/ @vconzola again... it seems like there is 2 things remaining:

flipping the order (so that at the end we see "VM migration failed - Migration cancelled.")
we should  fire the notification after the task status transitions to VM migration failed instead of on submit. Otherwise, this opens the door to the "three mile island" scenario. 💥

looks good!


Fixed Typo in command-line;  fixed broken hyper link for VS code Python extension

1st parameter command-line to run integration test had a typo.
VS code extension for python - link does not exist; replaced with MS Python VS code extension link
Note: docs/virtual_environment_info.md does not exist,  it looks like you might have missed adding this file to repo

@sethusrinivasan - Looks good! Thanks for the contribution!


Global 'View' and Workspace 'Add': QA Test Extension Exemption Data

As an ECMPS user, I want to view QA Test Extension Exemption Data in the Global-View and add new QA Test Extension Exemption Data in the Workspace so that I can look at real-time data or add test information as needed
Acceptance Criteria: (Global-view)
Given a user has selected QA Test Extension Exemption from the Test Data dropdown and a data record is available
When the data is displayed
Then the user can see data displayed as a separate table
Columns:
Unit or Stack Pipe ID
Year
Quarter
Monitoring System ID
Component ID
Hours Used
Span Scale Code
Fuel Code
Extension or Exemption Code
Given a user is viewing QA Test Extension Exemption Data
When the data is displayed in a table
Then the default sort order is alphanumeric descending by the contents in the 1st column
And the user will have the option to sort the columns in ascending or descending alphanumeric order

[x] The fields and their order specified

Pagination?

[ ] Yes
[x] No

Filter by keyword?

[ ] Yes
[x] No

Columns are sortable?

[x] Yes
[ ] No

Nested rows, expandable?

[ ] Yes
[x] No

Acceptance Criteria: (Workspace - Add)
Given the user has selected QA Test Extension Exemption from the Test Data dropdown
When the user clicks on 'Add'
Then the system displays a modal
and the user can add new data for:
Unit or Stack Pipe ID (Dropdown)
Year (Dropdown)
Quarter (Dropdown)
Monitoring System ID (Dropdown)
Component ID (Dropdown)
Hours Used (Text)
Span Scale Code (Dropdown)
Fuel Code (Dropdown)
Extension or Exemption Code (Dropdown)
Given the user has added QA Test Extension Exemption Data
When the user chooses to save and close
Then the data is saved
And the modal closes
And the user is returned to the Landing screen
And the QA Test Extension Exemption data record is reflected on the UI
Given the user has added a new QA Test Extension Exemption data
When the user chooses to Cancel
Then the user is returned to the Landing screen for QA Test Extension Exemption data
And no updates are saved
Testing
Test Case: TC1200
Tester: @JanellC


sub command example

use this ? https://github.com/google/subcommands

or use cobra https://github.com/spf13/cobra

📝 currently, not easy.

Using "binder" with cobra is a good feeling for now.


Memory leak on page with many event dispatches

Related DevTools bug: b/369633243

Open DevTools
Go to Settings > Experiments and make sure the "Performance panel: enable live metrics landing page" experiment is disabled

This is to ensure the web-vitals.js code shipped with DevTools is not injected

Reload DevTools
Open the DevTools Performance monitor and check "JS event listeners"
Go to the following page and click the button:

<!DOCTYPE HTML>

<input type=button onclick="run()" value=Run>

Expected
(result achieved by removing web-vitals code from the field above)

Actual

I think I tracked it down to this code in onLCP:
https://github.com/GoogleChrome/web-vitals/blob/9b932519b16f72328c6d8e9814b811f1bc1a0bb5/src/onLCP.ts#L100-L108
Disabling this code will prevent the memory leak.

Think this is a dupe of #534? It's fixed in v5 which is currently in beta.

@philipwalton maybe we should consider a final v4 branch with this fix?

@philipwalton maybe we should consider a final v4 branch with this fix?

Did you mean to say v4?

Yes. For those that don’t want to upgrade to v5 (at least right away) for FID or older browser build reasons.
Then again, not had many complaints about this and Adam’s example is an extreme example.

The base branches for these are very different, so we wouldn't be able to just cherry-pick that commit into v4, but it would be simple enough add {once:true, capture:true} to the addEventListener() call.

I'm having difficulty rolling 5.0.0-rc.0 into DevTools (possibly an issue with DevTools still investigating) so I think a minor version bump with the changes @philipwalton mentioned would help out a bit here.

Yeah I think that makes sense too. Raised #554 for that.
Also super curious to hear about your v5 RC issues—ideally before we finalise that.


feat: throttle live updates toggle

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


Error when updating Daddylive channels

When i refresh daddylive channels it ends with
"Refresh DaddyLive Channels
Plugin: DaddyLive -- Last ran Never ago, taking"
the log shows
2024-10-17 05:34:03,595-INFO:channels DaddyLive:default 1 Updating Channel 313:Discovery Channel
2024-10-17 05:34:03,794-INFO:channels DaddyLive:default 1 Updating Channel 657:Discovery Family
2024-10-17 05:34:03,993-INFO:channels DaddyLive:default 1 Updating Channel 314:Disney XD
2024-10-17 05:34:03,993-INFO:channels DaddyLive:default 2 New Channel Added 285:Discovery Velocity CA
2024-10-17 05:34:04,389-INFO:channels DaddyLive:default 1 Updating Channel 651:Destination America
2024-10-17 05:34:04,587-INFO:channels DaddyLive:default 1 Updating Channel 652:Disney JR
2024-10-17 05:34:04,587-ERROR:scheduler UNEXPECTED EXCEPTION on GET='str' object has no attribute 'append'
Traceback (most recent call last):
File "/app/lib/schedule/scheduler.py", line 157, in call_trigger
results = call_f()
File "/app/lib/plugins/plugin_obj.py", line 225, in refresh_channels
return self.refresh_it('Channels', _instance)
File "/app/lib/plugins/plugin_obj.py", line 248, in refresh_it
instance.refresh_channels()
File "/app/lib/plugins/plugin_instance_obj.py", line 152, in refresh_channels
return self.channels.refresh_channels()
File "/app/lib/plugins/plugin_channels.py", line 134, in refresh_channels
ch_dict = self.get_channels()
File "/app/plugins_ext/provider_video_daddylive/lib/channels.py", line 52, in get_channels
ch_list = self.get_channel_list()
File "/app/plugins_ext/provider_video_daddylive/lib/channels.py", line 241, in get_channel_list
self.groups_other.append(group)
AttributeError: 'str' object has no attribute 'append'
2024-10-17 05:34:05,599-DEBUG:web_handler [<IP_ADDRESS>] "GET /api/schedulehtml?Channels=1& HTTP/1.1" 200 -

this is using the daddylive plugin version 0.1.38 . happening on several systems both windows and in docker. tried both resetting the database and removing the database

looking into this

I think there's something new.. tried to open a new issue but saw this:
2024-11-27 03:26:08,687-WARNING:channels DaddyLive channel list is empty from provider, not updating Cabernet
2024-11-27 03:26:08,687-WARNING:plugin_channels Unable to retrieve channel data from DaddyLive:default, aborting refresh
Plugin version 0.1.40
Thanks

Looks like TVG is blocking IPs.  Recommend changing the "EPG Plugin to Use" to None for now.

not sure if this is fixed or we just leave daddylive epg off?


Forward the exit code from the Java process if it fails on Windows

Prior to this the process would always signal that it succeeded, making it hard to detect failed tests on Jenkins on top Windows.
Relevant issue: #2991

Does your test suite run on Windows? In that case I can add a test asserting the forwarding of exit signals are working.

Nope, it doesn't. That's the whole issue with Windows support: we're not Windows users.

@notdryft Do you have a Windows box at hand where you could give this change a try?

Nor am I, but I discovered the bug after being forced to run Jenkins on a Windows Server R2 environment. Kind of weird to think about that no one has run into this earlier ...

sure
On Mon, May 2, 2016 at 10:43 PM, Michael McMillan<EMAIL_ADDRESS>wrote:

Nor am I, but I discovered the bug after being forced to run Jenkins on a
Windows Server R2 environment. Kind of weird to think about that no one has
run into this earlier ...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/gatling/gatling/pull/2994#issuecomment-216357515

Actually, I think my patch contains a bug. I added a comment with the rem function, I guess that rem will always succeed, thus ruining the error code check following it. Let me update the PR.

run Jenkins on a Windows Server R2 environment

Sigh... Nothing shocks me anymore since I saw a big insurance company running Websphere AS on Windows NT ON PRODUCTION.

Kind of weird to think about that no one has run into this earlier

I guess most ppl don't use the bundle but maven.

Looks good to me.

Thanks!

Thanks!


fix: allow for override of test-image used by mvp-demo

Description
when trying to run the mvp-demo test from my laptop against my personal cluster, where I set the required DEFAULT_QUAY_ORG env var, the test was still trying to push updates to quay.io/redhat-appstudio-qe/test-image and it was failing because my quay credentials did not have permission for that.
/assign @psturc
/assign @flacatus
Issue ticket number and link
Type of change

[/] Bug fix (non-breaking change which fixes an issue)
[ ] 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

How Has This Been Tested?
I ran the mvp-demo e2e tests from my e2e-tests clone where DEFAULT_QUAY_ORG was set and they worked.
Checklist:

[/] I have performed a self-review of my code
[/] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

flakes outside of mvp-demo:
 lti-component scenario with all supported import components [It] waits application mc-three-scenarios components pipelines to be finished [e2e-demo, multi-component]
  /tmp/tmp.pW1WQ1pHFB/tests/e2e-demos/multi-component.go:235
  [FAIL] [e2e-demos-suite] DEVHAS-234: create an golang application [It] waits public component golang-dockerfile pipeline to be finished [e2e-demo]
  /tmp/tmp.pW1WQ1pHFB/tests/e2e-demos/e2e-demo.go:187
  [FAIL] [e2e-demos-suite] multi-component scenario with a component with a supported runtime and another unsuported [It] creates environment development [e2e-demo, multi-component]
  /tmp/tmp.pW1WQ1pHFB/tests/e2e-demos/multi-component.go:198
Ran 286 of 330 Specs in 1552.580 seconds
FAIL! -- 282 Passed | 4 Failed | 2 Flaked | 19 Pending | 25 Skipped

/retest

/test redhat-appstudio-e2e


Refactor job runner

As part of #590 I discovered that the callback StartCallback used in process.Process triggered the golang race detector. This replaces that callback with a WaitStarted method, which allowed for some cleanup in other places.
Beyond that, I noticed that we seem to implement a lot of what go's https://golang.org/pkg/bufio/#Scanner, so I figured we could just use that, it cuts down on quite a bit of code.

@stefanmb looks like you spent some time with this code previously, would welcome your feels on it if any!

@keithpitt do you remember if there was a reason for using line, isPrefix, err := reader.ReadLine() vs the higher-level Scanner?


can the client be use for other servers, like rcbranch ?

This came up because it would allow us to create RDF for content only on the rcbranch server, allowing us to run JUnit tests on the RDF for, for example, Reactome content up there...

Yes in PathVisio you can change the URL in the preferences. In de code you can give the URL when you create the client.

But that requires me to know the "webservice" of the rcbranch server, not? I can see where I should replace "http://webservice.wikipathways.org", but how I can I discover what that should be http://rcbranch.wikipathways.org/ ?


New analytics adapter from Adagio

Our analytics adapter was submitted and merged via this PR: https://github.com/prebid/Prebid.js/pull/3069.

released with 1.26


Update google-cloud-pubsub to 1.113.0

Updates com.google.cloud:google-cloud-pubsub from 1.107.0 to 1.113.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-pubsub" } ]

labels: library-update, semver-minor

Superseded by #236.


Missing constraint info in language model

I have one other F# -> C# IDE issue concerning type parameter constraints. These seem to be missing in the F# type info resharper consults from C# projects.
I have added the repro on top of the previous commit in the https://github.com/NinoFloris/IVT-rider-repro
https://github.com/NinoFloris/IVT-rider-repro/commit/d4cfe7f754e77b47cefd0e49400ba2360635b815
Beyond IDE errors it also means Rider is failing to scaffold the interface stubs in C# for F# interfaces that will actually compile as the constraints are missing when it generates these.

@NinoFloris Thanks! Fixed in 5721f14 and b63f18f.


Add to onboarding reproduction logs

Reproduced on MacBook Pro Intel Core i5
Environment:

macOS Ventura
Python 3.9.4
Java openjdk 20.0.1
Apache Maven 3.9.6

issue: the step wget https://msmarco.blob.core.windows.net/msmarcoranking/collectionandqueries.tar.gz -P collections/msmarco-passage returns ERROR 404: The specified resource does not exist.
everything worked (with the alternative mirror link)

issue: the step wget https://msmarco.blob.core.windows.net/msmarcoranking/collectionandqueries.tar.gz -P collections/msmarco-passage returns ERROR 404: The specified resource does not exist.
everything worked (with the alternative mirror link)

Hi @17Melissa can you poke around the site https://microsoft.github.io/msmarco/ and see where the above tarball has been moved? Let's try to fix the URL?

Hi @lintool absolutely! I found the tarball, it has been moved to this location: https://msmarco.z22.web.core.windows.net/msmarcoranking/collectionandqueries.tar.gz. Would you like me to update the URL?

Yes please!

Yes please!

The URL has been updated. Thank you!


Dlib accelerated by movidius

Movidius can easily used by caffe and tensorflow ,can it be used by dlib?

No, there isn't any special Movidius support in dlib.


Add global_phone validator

Validate global phone numbers in Rails 3.x and ActiveModel.
class Person < ActiveRecord::Base
  validates :home_phone, :global_phone => true
end

Person.new(home_phone: '+61 3 9876 0010').valid?
# => true

class User < ActiveRecord::Base
  validates :work_phone, :global_phone => { :using => :work_country_code }

  def work_country_code
    'AU'
  end
end

User.new(work_phone: '03 9876 0010').valid?
# => true

Thanks for the PR :+1: I agree with Josh, this gem is meant to pure Ruby and not have dependencies on Rails so I'm going to close this.


Does this work with Android/iOS?

Hi, I'm planning to build a cross-platform program which must work in windows, linux, macOS, android and iOS.
Does brynet works with all of these platforms? (of course, in android, for example, using the NDK + JNI).
If it isn't, which libraries are using brynet that are incompatible?
Regards,
Knife.

Hi, This answer is whether we can use epoll in Android and kqueue in IOS.
I never use Android and IOS, Sorry I don't test for you.
Can you test some API of epoll and 'kqueue' ?

@knife-dev Hi, Can you build brynet with Android/iOS?

@knife-dev Hi, this lib support both win32,android,ios,macos,linux: https://github.com/simdsoft/yasio, you can try it

I have tested that it can be supported on Android platform

Thank you, author.@IronsDu

@soknifedev I want close this issue. If you have any question, please reopen it.


Permissions for applications

This apply the fastapi_permissions to the Application resource.
To do that we added the ApplicationPermission resource to manage the permissions for the applications.
The README.md is also updated with a brief explanation on how to use the permission system.

👍  LGTM - Very nice work here!


Unencrypted subscription password is stored in the registry

when a subscription is created with endpoint credentials, the password and username are part of attributes dict. This is encrypted and stored as credentials in our model. However, the unecrypted data is not removed from attributes dict and is stored as is in the registry.

explore isPushConfig_AuthenticationMethod interface, should we implement our own basic auth Authentication method using the provided interface


Do not fail in TearDown if container not found when removing

If the container is not found when removing, it means it's already not there anymore, so it's safe to ignore. This should reduce a bit some TearDown flakyness..
17:15:30 ----------------------------------------------------------------------
17:15:30 FAIL: check_test.go:157: DockerSuite.TearDownTest
17:15:30
17:15:30 check_test.go:159:
17:15:30     deleteAllContainers(c)
17:15:30 docker_utils_test.go:142:
17:15:30     c.Assert(err, checker.IsNil)
17:15:30 ... value *errors.errorString = &errors.errorString{s:"\nCommand: /go/src/github.com/docker/docker/bundles/1.14.0-dev/binary-client/docker rm -fv e09bc5ab6556\nExitCode: 1, Error: exit status 1\nStdout: \nStderr: Error response from daemon: No such container: e09bc5ab6556\n\n\nFailures:\nExitCode was 1 expected 0\nExpected no error\n"} ("\nCommand: /go/src/github.com/docker/docker/bundles/1.14.0-dev/binary-client/docker rm -fv e09bc5ab6556\nExitCode: 1, Error: exit status 1\nStdout: \nStderr: Error response from daemon: No such container: e09bc5ab6556\n\n\nFailures:\nExitCode was 1 expected 0\nExpected no error\n")
17:15:30
17:15:30
17:15:30 ----------------------------------------------------------------------
17:15:30 PANIC: docker_cli_build_unix_test.go:130: DockerSuite.TestBuildCancellationKillsSleep
17:15:30
17:15:30 ... Panic: Fixture has panicked (see related PANIC)
17:15:30
17:15:30 ----------------------------------------------------------------------

Depending on the "configuration" of the daemon and container running, the container might have killed himself between listing it docker ps -aq and removing it docker rm.
On a side note (and for a future PR 👼), these teardown helper method should really use the API directly (through the Go client) instead.
/cc @thaJeztah @AkihiroSuda @tonistiigi @cpuguy83 @dnephin
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

Windows failures are not related, and this fixes some flakiness on CI. I see @AkihiroSuda's comment was addressed, so going ahead and merging


ModuleNotFoundError: No module named 'casadi.casadi'

It was successfully run once before, but such an error occurred after opening jupyter notebook again.
Hope to get answers，thank you.

This sounds like a python install issue on your end, not a pybamm bug. See maybe https://stackoverflow.com/questions/37085665/in-which-conda-environment-is-jupyter-executing


api: findings add support for storing and filtering by theme

We should add a new column and ability to populate it and filter by it in the findings API

This is done


Documentation on when to use ImageBuilder Pipelines

There is no recommendations on when or when not to use Pipelines.
EG the Cfn example doesn't have a pipeline while the CDK does.
Why? And when should we chose either of these two options?

Not really. I mean, it just sounds random rather than reasoned, like "we are showing you how to do a 1 off ami with this Cfn. You would typically only do this for testing" etc etc

This repository's purpose is provide samples that customers can use to learn more about ways to interact with, or create Image Builder resources. It is not intended to be a source of truth for the best practices on how each individual or business should model their golden image workflows.
Given that, the note that "this repository is aimed at testing purposes" is the accurate take.
For more focused best practice guidance I would look towards AWS blog posts such as Automate OS Image Build Pipelines with EC2 Image Builder or Create immutable servers using EC2 Image Builder and AWS CodePipeline (I'm sure there are others, they were the first two I found).


zipfile opens an empty buffer after 8.2.3468

To Reproduce

Create a non-empty zip file, e.g. xxx.zip
Run vim xxx.zip
Select one entry inside the zip listing and press Enter to open/view it
An empty buffer is open. Buffer name is ~/.../zipfile:/.../xxx.zip::xxx.txt, i.e. with current directory prepended.

Expected behavior
netrw should read the zipped xxx.txt's content into new buffer.
Environment (please complete the following information):

Vim version: from 8.2.3468 (found by git bisect, first bad commit: c6376c798433bcb9ee38a8664299d11454546950) to 8.2.3486 (latest)
OS: Arch Linux
Terminal: GNOME Terminal

looks it's not related to 'netrw' but 'zip.vim', though same auther: @cecamp
// looks 'zip.vim' was not a complete runtime, it set ft of zip to 'tar' also?!......

#8903
#9023

Okay, thanks. Unfortunate that fixing a bug broke an existing plugin, but using zipfile:// looks better solution than proposed in this thread. The zip plugin implementation is valid under the idea of URI, where //authority is optional, but Vim cannot distinguish this resource identifier from actual (relative) file paths. Based on the implementation of path_with_url (and vim_isAbsName), using :// in plugins looks a good convention to me.

@cecamp @chrisbra @lacygoill Do you have any thoughts?

Fixed in the latest update of the runtime files.


overly annoying incompatible version warning

there's a little warning that pops up informing me that my version of Kopf is not compatible with my version of elasticsearch.
the warning is great except a new one pops up every 5 seconds (on every automatic refresh i guess)
maybe it would be better to show this warning only once per session instead?

I think I ran into a solution earlier. You can make a simple change in the _site/dist/kopf.js
Avoid the nervouse amount of version compatibility alerts beginning on line 1215 sample could be a different version , though throw the alert once
$scope.version = '2.1.2';

 $scope.modal = new ModalControls();
 var alertedOnce = false;

 $scope.$watch(
     function () {
         return ElasticService.cluster;
     },
     function (newValue, oldValue) {
         var version = ElasticService.getVersion();
         if (version && version.isValid()) {
             var major = version.getMajor();
             if (major != parseInt($scope.version.charAt(0)) && !alertedOnce) {
                 AlertService.warn(
                     'This version of kopf is not compatible with your ES version',
                     'Upgrading to newest supported version is recommeded'
                 );
                 alertedOnce = true;
             }
         }
     }
 );

@andersmartini sorry, the idea was really to be inconvenient. Not supporting ES 5 meant that I also did not test it against it. And even though this is unlikely, I prefer annoy people than to have someone lose data/break a cluster.
By the way, if you are looking for ES 5 support try https://github.com/lmenezes/cerebro :)


Added feature Dashboard button mimics home

Thanks for the pull request.  While I appreciate you taking the time to implement this feature, I decided to develop an alternative implementation instead that is now available in the master branch.


Install not working after upgrade to 14361

I went and upgraded to build 14361, now there is a bash process running at 100% CPU usage on one of my cores. I am unable to exit this process using any tools, and it can sit there for hours. It doesn't cause any disk/network activity, only CPU. I get the following error when attempting an uninstall:
C:\Users\N00byEdge>lxrun /uninstall /full /y
This will uninstall Ubuntu on Windows.
This will remove the Ubuntu environment as well as any modifications, new applications, and user data.
Uninstalling...
Error: 0x80080005

The error takes about a minute until it appears. This install was very unstable on the last build, but it was at least working. No modifications made to the install, and Windows is running on a real PC, no VM.

I just realized there is a new preview build. I'm installing it as I write this, and will report back if that fixed it.

Thanks for reporting this, I'll be curious to see if this issue goes away when you upgrade.

The issue persists. The only difference it that I can no longer ctrl+c a cmd trying to start Bash. What error info do you want me to collect?

I of course tried to uninstall the feature and install it again (and yes, it was uninstalled after the first reboot), but it still wouldn't start. This makes me think that it's the Ubuntu install that is broken.

Wow, in the latest build the Bash process was not running on startup(which it was on the last build), which enabled me to do the uninstall. The uninstall is still unresponsive after the bash process has been opened.
However, it still seems like the issue is there. Maybe it's hardware related? Screenshot

Did your install eventually succeed?  We do launch some commands as part of post install, I suspect that's what's happening in your screenshot.
There are instructions here how on collecting tracing information.  I would suggest uninstalling, rebooting, then starting the trace and running install.
Tracing instructions: https://github.com/Microsoft/BashOnWindows/issues/410#issuecomment-221463487

@benhillis I didn't quit the process after taking that screenshot and all, but as I am writing this, the process is still running. I think that we should call a 7 hour wait good enough, right? I'll go ahead and follow those instructions.

Is there an update to this issue? The files that were collected were very small, and I do not know if it was even able to start. Anyways, at that state it just sits there and babysits one of my cores at all times. It has started running at startup again. If you want to look at this through TeamViewer or something, I could possibly arrange that.

I got your traces but, as you mention, they are quite small and there aren't any useful breadcrumbs in them.  Above you mention reinstalling the feature, did you do that via the "Turn Windows features on or off" page or by the below commands?
lxrun.exe /uninstall /full
lxrun.exe /install

If you haven't already, I'd suggest this.  Reboot, then run the above commands.

@benhillis That's exactly what I did before running the capture. It must be that it never even successfully starts, then.

That means I did that before sending the files.

Issue still present on 14367

This sounds very similar to #358.  @N00byEdge by chance was this an upgrade from an earlier version of Windows (not an earlier Windows 10 Insider flight, like the release version of Windows 10).
The following information would be useful to help me try to repro this:

Which Windows build number are you running?
What Windows locale is your install?
What is your Windows username?
Any other configuration differences you can think of from a fresh Windows install.

Build 14367
sv;Swedish
Orion\n00byedge, local user. Computer is on a domain.
Upgraded from W7 -> W10 Release -> W10 IP

I tried doing the same thing with a domain user with only letters in the username, same issue.

Thanks for the info, I will try to reproduce this.

@N00byEdge To confirm, you're using the en-US version of Windows not sv-SE?  Originally you said you were using Swedish so that's what I installed.

Sorry, maybe I should have typed out this in the last post. I have English language with Swedish input, and a Swedish language pack for spelling and hand writing, but I am using English as Windows and UI language:
CurrentCulture   : sv-SE
CurrentUICulture : en-US
from Get-Host in powersell

Unfortunately I'm still unable to reproduce this locally.  If you would be able to take a sample profile trace so we can identify where /bin/bash is getting stuck that'd be very useful.  I have instructions in another thread: https://github.com/Microsoft/BashOnWindows/issues/358#issuecomment-229176401
Let me know if you have any questions.

I'm getting the following error:
C:\Users\N00byEdge>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl
xperf: error: NT Kernel Logger: Invalid flags. (0x3ec).

C:\WINDOWS\system32>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl
xperf: error: NT Kernel Logger: Cannot create a file when that file already exists. (0xb7).

There is not a single tmp.etl on my system. What should I look for?

I don't know if it's worth mentioning, but when it was running, I was getting #143.

@N00byEdge The above error means you're already collecting a trace, use xperf.exe -d profile.etl  to stop.

That did not work.
C:\WINDOWS\system32>xperf.exe -d profile.etl
xperf: error: Merge ETL: The system cannot find the path specified. (0x3).

C:\WINDOWS\system32>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl
xperf: error: NT Kernel Logger: Cannot create a file when that file already exists. (0xb7).

This is probably because you're running out of System32.  Can you change to a different directory like your desktop?

F:\>mkdir dmp

F:\>cd dmp

F:\dmp>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl
xperf: error: NT Kernel Logger: Cannot create a file when that file already exists. (0xb7).

F:\dmp>xperf.exe -d profile.etl
xperf: error: Merge ETL: The system cannot find the path specified. (0x3).

F:\dmp>mkdir C:\Users\N00byEdge\Desktop\dmp

F:\dmp>cd /d C:\Users\N00byEdge\Desktop\dmp

C:\Users\N00byEdge\Desktop\dmp>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl
xperf: error: NT Kernel Logger: Cannot create a file when that file already exists. (0xb7).

C:\Users\N00byEdge\Desktop\dmp>xperf.exe -d profile.etl
xperf: error: Merge ETL: The system cannot find the path specified. (0x3).

And you're running this in an Admin command prompt?
Can you run "which xperf.exe"?

Yes.
C:\WINDOWS\system32>which xperf.exe
'which' is not recognized as an internal or external command,
operable program or batch file.

Sorry I mean "where xperf.exe".  Linux is creeping into my world :)

Ah, thought so.
C:\WINDOWS\system32>where xperf.exe
C:\Program Files (x86)\Windows Kits\10\Windows Performance Toolkit\xperf.exe

I don't know what kind of voodoo magic you are working, though.

Huh, that's interesting. I literally uninstalled the Windows 10 kit when I installed the W8.1 kit you requested me to install. Will retry install.

That is odd, the Windows 10 version of xperf should work as well...
C:\Users\benhill>which xperf.exe
C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\xperf.exe

Samme issue.
C:\WINDOWS\system32>"C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\xperf.exe" xperf.exe -d profile.etl
xperf: error: xperf.exe: The parameter is incorrect. (0x80070057).

C:\WINDOWS\system32>"C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\xperf.exe" -d profile.etl
xperf: error: Merge ETL: The system cannot find the path specified. (0x3).

Maybe try rebooting?  Xperf might have gotten in a funny state.  After you reboot run the below from an admin command prompt:
xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0

Okay, be right back!

Still facing the same issue:
C:\WINDOWS\system32>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0

C:\WINDOWS\system32>cd /d C:\Users\N00byEdge\Desktop\dmp

C:\Users\N00byEdge\Desktop\dmp>"C:\Program Files (x86)\Windows Kits\8.1\Windows Performance Toolkit\xperf.exe" xperf.exe -d profile.etl
xperf: error: xperf.exe: The parameter is incorrect. (0x80070057).

C:\Users\N00byEdge\Desktop\dmp>xperf.exe -on PROC_THREAD+LOADER+INTERRUPT+DPC+PROFILE -stackwalk profile -minbuffers 16 -maxbuffers 1024 -flushtimer 0 -f tmp.etl

C:\Users\N00byEdge\Desktop\dmp>bash.exe

Gonna let this run for a bit.

Also; should I ctrl + c the bash.exe or let it run when I stop the trace?

Leave it running, you can close it after you stop the trace.

Yeah, nevermind. Just realized I can't stop Bash.exe without reboot. File size:
2016-06-29  01:38       124 911 616 profile.etl

Mail is coming. Let's hope something comes out of this.

Mail is sent. My web server didn't like my 121 MB file, so I uploaded it and shared a link. Hope that's not an issue.

Nope that's perfect.  Thank you very much, I hope this sheds some light on the issue!

Just caught this little guy in the corner of my eye in the taskbar all of a sudden for the first time since my issues started: http://puu.sh/pK5S7.png
It's frozen, I don't know how long it has been for, but at least a couple of minutes after finding it.

Now Bash.exe gives this:
C:\Users\N00byEdge>bash
Cannot launch bash because the LX subsystem has an install, uninstall, or servicing operation pending.

Do I reinstall at this point?

Just close that update window.

Yeah, I did.

I received your trace and I've identified what's getting hung.  I'd like to try to get a repro locally so I can validate my fix.  Out of curiosity do you not have a page file on your system?
You can check this via:
Advanced system settings -> Advanced tab -> Performance Settings -> Advanced tab -> Virtual memory Change

I have a page file, but not on my main drive, my ssd raid. I had one on there though up until about 4 days ago.

Here is a screenshot: http://puu.sh/pKqEC.png

Ok but you do have multiple page files.  I think I see what the issue is let me confirm my hypothesis.

@N00byEdge Thank you very much for your patience and persistence.  I would not have been able to figure this out without your help!
Because of your trace file I have identified a bug in the code that is calculating the swap file size.  This function is used by both the sysinfo system call and generating the contents of /proc/meminfo.  This problem will happen when you have multiple pagefiles on your machine.  There are three different outcomes that can happen as a result of this bug:

The swapfile size will be reported incorrectly
Our driver will dereference an invalid memory address and cause a bluescreen
The machine will enter an infinite loop when calculating the swap file size (this is the one you are seeing)

As a temporary workaround if you remove all but one of your pagefiles you should be able to use WSL. I am coding up a fix for this right now.

Thanks, I love WSL and will temprarely disable all but one pagefile and come back with my results. Also, will you report back when this is fixed (will it be in the next build)?

I can confirm that WSL yet again works with a single swapfile! Thanks for all the persistence and help!
C:\Users\N00byEdge>bash
root@ORION:/mnt/c/Users/N00byEdge#

Great to heart, I'm going to reopen this thread until the issue is fixed in an insider build but I will tag the thread with "workaround available"

Okay, fair enough.

+1 to the install, update, upgrade/etc/etc not working. Running under sudo su
root@MSI:/home/DarkSt0rm# apt-get update
Err http://security.ubuntu.com trusty-security InRelease

Err http://archive.ubuntu.com trusty InRelease

Err http://archive.ubuntu.com trusty-updates InRelease

Err http://security.ubuntu.com trusty-security Release.gpg
  Could not resolve 'security.ubuntu.com'
Err http://archive.ubuntu.com trusty Release.gpg
  Could not resolve 'archive.ubuntu.com'
Err http://archive.ubuntu.com trusty-updates Release.gpg
  Could not resolve 'archive.ubuntu.com'
Reading package lists... Done
W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty/InRelease

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-updates/InRelease

W: Failed to fetch http://security.ubuntu.com/ubuntu/dists/trusty-security/InRelease

W: Failed to fetch http://security.ubuntu.com/ubuntu/dists/trusty-security/Release.gpg  Could not resolve 'security.ubuntu.com'

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty/Release.gpg  Could not resolve 'archive.ubuntu.com'

W: Failed to fetch http://archive.ubuntu.com/ubuntu/dists/trusty-updates/Release.gpg  Could not resolve 'archive.ubuntu.com'

W: Some index files failed to download. They have been ignored, or old ones used instead.

My firewall is not blocking anything because I tried with everything disabled as well. Yesterday I could ping http://archive.ubuntu.com from the powershell but not winbash and now neither works. Error with conical?

@CryptXor Wait, doesn't that imply that the install worked for you? Are you sure this is the correct issue?

i solved the problem by disable the function,restart computer and enable the function again.
then,when i run bash command,it worked!

In my case, any mv or cp command was blocked by the windows firewall. When I cancel one of this command, the problem with the uninstall happens. Try to disable the firewall, reboot the windows and run the uninstall. That work with me.


[Bug] Unsubscribe on drop implementation causes a panic when receiving malformed response?

3|tracker_ | thread 'tokio-runtime-worker' panicked at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/rustis-0.12.9/src/network/network_handler.rs:545:17:
3|tracker_ | [private:6379] Received unexpected message: Ok(Push([BulkString("unsubscribe"), BulkString("tracker:346240269"), Integer(0)]))
3|tracker_ | note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Info is somehow limited it seems to be coming from the rustis library itself? It's on 0.12.8.

Hello,
Thank you for the report.
Could you provide a sample please?
When you write "unsubscribe on drop implementation". Do you mean a PibSubStream has been dropped or something else ?
Thank you,
Michaël

Hello,
I just fixed a bad status pb in the NetworkHandler implementation, let me know if it fixes your pb.
https://github.com/dahomey-technologies/rustis/commit/b107cdbdfd1c553edf5a55ff81f63aea7dbcd662

any news ?

new fix on release 0.13.3.
Could you test it, please?

Hi,
I close this bug as stalled.
Please feel free to reopen it if you encounter any additional issue!


could not find com.github.barteksc:android-pdf-viewer:2.8.2.

Add jcenter() to you build.gradle file or settings.gradle (for bumblebee)

Failed to resolve: com.github.barteksc:android-pdf-viewer:2.8.2
After adding jcenter to setting.gradle

Try and go with the latest version - com.github.barteksc:android-pdf-viewer:3.2.0-beta.1

Failed to resolve: com.github.barteksc:android-pdf-viewer:3.2.0-beta.1
same problemo

Same here, repo is dead...

Whatever version you are using please try updating settings.gradle file with jcenter and mavencentral this might solve your problem if you are using bumblebee android studio
Correct me If I am wrong :)

try to use gradlePluginPortal()

Has someone found a solution that works? I am still struggling with this.
Worse case, I will have to move to another plugin.

add jcenter() in repositories blocks under pluginManagement and dependencyResolutionManagement blocks in settings.gradle for android Bumblebee

Astroa7m is correct just do that as he said the problem has been already solved by doing so...

I had to bump up the minSDK to 31 to get it to work.

Solution

For android studio bumblebee or android studio arctic fox
add gradlePluginPortal() into settings.gradle file

ON

replace jCenter() with mavenCentral()
Then do:
implementation 'com.github.mhiew:android-pdf-viewer:3.2.0-beta.1'

** I am on Arctic Fox 2020.3.1 Patch 1

Adding jCenter() to the your build.gradle in the android folder below will work for you. I was having similar challenge but after adding it , everything is now working fine for me
allprojects {
repositories {
google()
mavenCentral()
jcenter()
}

}

How to resolve Could not find com.github.barteksc:android-pdf-viewer:3.2.0-beta.1. in Flutter application

@Aburafik Do you have problem with showing Pdf only on Release?

Here is the solution.
In setting.gradle >dependencyResolutionManagement add this line maven {url "https://jcenter.bintray.com"}
In gradle.properties Add this line android.enableJetifier=true
that's all, after adding these two lines in your project will resolve the error.

Guys hope you are doing well . Actually I am working on android studio but I face such issue so please help me

I finally fixed it thanks to all comments above. Just to summarize it for others. The fix for React Native 0.71.6 you have to add the following lines in the  android/build.gradle

ext > pdfViewerVersion = "3.2.0-beta.1"
ext > pdfViewerRepo = "com.github.mhiew"
repositories > jcenter()

...
ext {
   ...
   pdfViewerVersion = "3.2.0-beta.1"
   pdfViewerRepo = "com.github.mhiew"
}
repositories {
   ...
   jcenter()
}

Here is the solution. In setting.gradle >dependencyResolutionManagement add this line maven {url "https://jcenter.bintray.com"} In gradle.properties Add this line android.enableJetifier=true that's all, after adding these two lines in your project will resolve the error.

is worked for me

Astroa7m is right. to those correcting the problem be careful to avoid adding jcenter() in plugin management because your app wont run so look out for the dependencyResolutionManagement and ADD it to the repository block there

solution : for latest version of android studio(flamingo or giraffe)--->
Add 'android.enableJetifier=true' in gradle.properties,
Then go to settings.gradle.kts and you to add  gradlePluginPortal() as shown below;
pluginManagement {
repositories {
google()
mavenCentral()
gradlePluginPortal()                              <----Leave this as it is
}
}
dependencyResolutionManagement {
repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS)
repositories {
google()
mavenCentral()
gradlePluginPortal()                  <----- this line you have to add in dependencyResolutionManagement
}
}
Hope this resolves your problem. I also got confused but found the solution........

Yeah, this works thanks for the help

There are a couple of solutions in this thread that I believe work, but for me I didn't need to edit settings.gradle at all, only android/build.gradle, this is all I needed to add:
allprojects {
    repositories {
        maven { url "https://jcenter.bintray.com" }
    }
}

2. implementation 'com.github.mhiew:android-pdf-viewer:3.2.0-beta.1'

this solution is works for me 100%. Thanks

android.enableJetifier=true

perfectly work in 2024 too

com.github.barteksc:android-pdf-viewer:3.2.0-beta.1
do com.github.barteksc:pdfium-android if you do not want to use jcenter, because for some reason jcenter didnt work for me and I spent an hour to find out that android-pdf-viewer doesnt exist here: https://repo.maven.apache.org/maven2/, which is what I was using for some reason. Computers are confusing

just add  two lines in ext in android/build.gradlew
buildscript {
ext {
...
+ pdfViewerVersion = "3.2.0-beta.1"
+ pdfViewerRepo = "com.github.mhiew"
}
...
}
Tried above solution with react native version 0.74.1

Failed to resolve: com.github.barteksc:android-pdf-viewer:2.8.2 After adding jcenter to setting.gradle

This Solution Works in August 2024.ive tried many solutions beforeit, but it didnt work..   Check how i insert inmy Gardle

(Aug 2024)
Add these two in your Project level build.gradle
 buildscript {
  ext {
    .
    ..
    pdfViewerVersion = '3.2.0-beta.3'
    pdfViewerRepo = 'com.github.mhiew'
  }
}

and in settings.gradle
dependencyResolutionManagement {
repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS)
repositories {
    .
    ..
    maven { url 'https://www.jitpack.io' }
    }
  }
}

Here is the solution
Step - 1: Add it in your root build.gradle at the end of repositories:
	dependencyResolutionManagement {
		repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS)
		repositories {
			mavenCentral()
			maven { url 'https://jitpack.io' }
		}
	}

Step - 2: Add the dependency
implementation 'com.github.barteksc:AndroidPdfViewer:2.3.0'

Try using com.github.mhiew:android-pdf-viewer:3.2.0-beta.1

Thabk you this worked for me

I finally fixed it thanks to all comments above. Just to summarize it for others. The fix for React Native 0.71.6 you have to add the following lines in the android/build.gradle

ext > pdfViewerVersion = "3.2.0-beta.1"
ext > pdfViewerRepo = "com.github.mhiew"
repositories > jcenter()

...
ext {
   ...
   pdfViewerVersion = "3.2.0-beta.1"
   pdfViewerRepo = "com.github.mhiew"
}
repositories {
   ...
   jcenter()
}

this working

** 2024 solution **

Make sure to add -> maven { url 'https://jitpack.io' }
Add: api 'com.github.barteksc:AndroidPdfViewerV1:1.6.0'

solved :
https://github.com/DImuthuUpe/AndroidPdfViewer/issues/1206#issuecomment-2311750611

** 2024 solution **
1. Make sure to add -> maven { url 'https://jitpack.io' }

2. Add: api 'com.github.barteksc:AndroidPdfViewerV1:1.6.0'

Thank you 😊💖


chore: Update description to match standard format

Info
Should read:
"🚀 🦆 An intergalactic space theme for Microsoft Visual Studio Code (VS code)"

See:

@pineapplegiant needs to do this, I do not have the permissions to update the description anymore


3mfcompatibility

Updated schema & sample file to reflect 3mf naming convention.
Updated library to read and write 3mf compatible .ltcx files

To do:

Write 3mf extension spec for NTLatticeGraph
Add interpolation types for beams, smooting property for nodes
Add reader/writer for .3mf files w/ .ltcx extension
Possible extensions for surfaces? modifiers?

@bradrothenberg hey is there a reason that schemas/NTLG_001.xsd has no indentation?

nope, only because I wrote it in sublime & not in visual studio (which auto-indents)

Indented here: https://github.com/nTopology/NTLatticeGraph/pull/2/commits/eb4a2fa0f14a3cca209242f6bd12ee8f2ba6985c


Clarify permission denied dialog for image picker

Android screenshot:

r? @keybase/react-hackers maybe @cecileboucheron for text suggestions
Note that this dialog only shows up on Android -- on iOS a different error is returned.

Yeap, that's expected -- we can fix it later but for now, it's better than crashing out!

I'll file a ticket to make a better UI for that flow (which is the flow that happens on iOS every time)

Changed the wording depending on whether the user picks photo vs. video on @mmaxim's recent change

Merging, since this is just a wording change.


feat: double tap in pen mode to toggle eraser

When in pen mode, using freedraw or using one of the other tools locked... double tapping with your finger will toggle eraser.

Oops, forgot about this one.
@zsviczian do you think it'd make sense to only erase freedraw elements when switching to eraser while freedraw tool active?
There have been a few requests: https://github.com/excalidraw/excalidraw/issues/6163, https://discord.com/channels/723672430744174682/1256263773925412958/1256265194628190219, I think somewhere else as well.

Yeah - this is a nice feature. I was using it just today and thought that it was really nice. Admittedly I was only drawing freedraw lines when I used it.
Double tapping only activates the eraser for shapes when the tool selection is locked. Else after you draw it switches back to the select tool and double-tap will result in the text tool popping up.
You could argue that even if the shape tool is locked the user may want to switch between drawing and labeling. E.g. you lock the arrow, draw arrow, then double tap and add text. Of course, with the pen in hand, you could double tap with the pen as well.
I can argue either way.


Add KeymapMerge function

I needed to write this for one of my projects, decided to contribute it upstream. Just a small quality of life helper.

I needed to merge a couple of keymaps at least once too:
https://github.com/quasilyte/roboden-game/blob/cd409d65117842225432597794582743cd31dec5/src/controls/keymap.go#L171-L176
It could be a good helper function to have.


OCPBUGS-32328: azure-path-fix: support auth via account key (without clientID)

the azure storage driver's GetConfig function already checks for both UPI and IPI secrets, so we use that to mount it under the AZURE_ACCOUNTKEY env.
validation of the move-blobs also needed to change to reflect the fact that the account key works on its own (without client id or tenant id).

/jira refresh

/retest-required

e2e-aws-operator failed TestS3Minio, but that seems unlikely to be related to this Azure-only change.  e2e-hypershift failed a few things, including Failed to ssh to AWS instance, dumping ssh configuration, which is also non-Azure.
/retest-required
Probably also fine to just /override ci/prow/e2e-aws-operator, since I'm having trouble imaging anything in that suite exercising the code you're touching here.

/retest

/label qe-approved

/label cherry-pick-approved

/cherry-pick release-4.15

/lgtm

I'm overriding ci/prow/e2e-aws-operator since this change doesn't affect aws.
/override ci/prow/e2e-aws-operator


How to change permanent DB for User model?

How to change Permanent DB for User, Role, Permission models? I can not find it in the documentation. I would be grateful if you tell me where to look.

I think I understand.
I have edited the connection settings of models:
/api/user/models/User.settings.json
{
  "identity": "user",
  "connection": "mongodb_main",
  ...
}

The same thing I've done for other models Role, Route, Email etc...
It seems to work. I did everything right?

Yes you did! ;-)

@loicsaintroch cool! Thank you for quick reply. :thumbsup: :beer:


Move visualisation and assign query into their own programs

Mostly moving chunks of main() around, and some changes to the command line options. Once this is finished and merged rewriting the docs (#74) will be a priority
Closes #77, closes #107
NB: merge #108 and #110 first

97d215f804f6fc42358348e4b2078ae4c5438ac1 closes #111

TODO: change --fit-model to take options gmm, dbscan, refine, threshold or use

Also, remove the --full-db option from assign mode

Suggestions addressed in #125


how to build qamel?

Is there any steps covering build process in details?
I've started with 'go build' but got this:
application.cpp:2:10: fatal error: QApplication: No such file or directory                                                                                #include                                                                                ^~~~~~~~~~~~~~                                                                                compilation terminated.
I'm new to go, but guess I should specify include path somehow?
I've tried to do this:
set CGO_CFLAGS=-ID:\Qt\5.15.0\msvc2019_64\include -ID:\Qt\5.15.0\msvc2019_64\include\QtCore -ID:\Qt\5.15.0\msvc2019_64\include\QtGui
and then 'go build' but it didn't change a lot.

I think I got it, qamel cli is responsible for builds...

Oops, closed to fast..
qamel build -p win64                                                                                Starting build process.
Load config file...done
Generating vendor files...done
Removing old build files...done
Generating binding files...done
Generating Qt resource file...done
Generating syso file...done
Generating code for QML objects...done
Building app...
Failed to build app: exit status 2
github.com/go-qamel/qamel
application.cpp:2:10: fatal error: QApplication: No such file or directory
#include
^~~~~~~~~~~~~~
compilation terminated.

Also, how to debug qamel application in VSCode ?

Oops, closed to fast..
qamel build -p win64 Starting build process.
Load config file...done
Generating vendor files...done
Removing old build files...done
Generating binding files...done
Generating Qt resource file...done
Generating syso file...done
Generating code for QML objects...done
Building app...
Failed to build app: exit status 2
github.com/go-qamel/qamel
application.cpp:2:10: fatal error: QApplication: No such file or directory
#include
^~~~~~~~~~~~~~
compilation terminated.

I have encountered this problem before. Don't use the power shell or the CMD terminal that comes with windows. Use Cmder instead. This issue was already mentioned in #18 .

Also, how to debug qamel application in VSCode ?

I also want to know. I tried to comment out part of the code in the internel/cmd/build.go file. I thought of opening the terminal for output debugging, but the terminal still did not appear.


Problem when run multirobot simulation in Gazebo

Hello, I would like to run multirobot simulation in Gazebo. The following is the set up.
PC 1: launch robot 1 and spawn it in Gazebo.
PC 2: launch robot 2 and spawn it in Gazebo(which on PC 1 via multimaster package).
However, the amcl of robot 2 will not launch properly which caused problems in the tf_tree.
I have tried to launch the two robots and Gazebo in PC 2 and there is no problem for it.
I have checked there is no topics in ignore_topics parameters. May I know how to troubleshooting or solve this problem?

Hi, did you check the time of both PCs? Tf tree allows only small tolerances.
Otherwise you can try launch 'node_manager' (GUI) to find the problem or provide your launch files.

I have been checked the time of both PCs( which is GMT+8 , is this what you mean? or something about /clock?)
I launch node_manager and there is one warn in log
[WARN] 05.09.2019 19:03:43.567681 (/node_manager_daemon:server.py:start:84): can not add insecure channel to '[::]:12321', try again...
Is this the reason to cause multirobot simulation over gazebo fail to me?
Thanks for the reply.

I mean the time synchronization. If the time difference between hosts is more than 0.3-0.5 seconds you have to synchronize the time (with chrony or ntp).
The node_manager_daemon used only with node_manager and  not for synchronization. So it is not a reason for your problem. But the warning should not happen!
Can you provide the logs of master_sync and master_discovery? You can activate debug output with rosrun fkie_master_sync master_sync _log_level:=DEBUG. (Restart if no debug outputs are printed after first start.)
If you are able to start node_manager and you have time synchronization problems, you will see follow symbol in the GUI:

Hello, I am able to start the GUI but I didn't see the symbol.
The following part is the debug message from the GUI.
[DEBUG] 06.09.2019 10:36:31.567737 (/node_manager:monitor_channel.py:get_system_diagnostics:67): get system diagnostics from grpc://beh-gl552vx:12321
[DEBUG] 06.09.2019 10:36:31.567737 (/node_manager:remote.py:get_insecure_channel:86): create insecure channel to beh-gl552vx:12321
[DEBUG] 06.09.2019 10:36:31.567737 (/node_manager:channel_interface.py:close_channel:61): close channel to beh-gl552vx:12321

It seem quite okay to me. Any other information that I can provide to you?

Can you provide the logs of master_sync and master_discovery? You can activate debug output with rosrun fkie_master_sync master_sync _log_level:=DEBUG. (Restart if no debug outputs are printed after first start.)

But there is not debug outputs for me no matter how many times I restart the node.

did you start it with
fkie_master_sync master_sync _log_level:=DEBUG

Yes, but there are only INFO message printed in the terminal.

:-/
If you provide me your launch files I can try to test they in my environment.

ok you can change
https://github.com/fkie/multimaster_fkie/blob/master/fkie_master_sync/src/fkie_master_sync/__init__.py#L83
and
https://github.com/fkie/multimaster_fkie/blob/master/fkie_master_discovery/src/fkie_master_discovery/__init__.py#L101
to rospy.init_node(PROCESS_NAME, log_level=rospy.DEBUG)

I hope, you solved your problem.
If not, please reopen!


Testing Open3DIS on ScanNet200 dataset with Mask3D?

Hello,
Thanks for this great work! Besides ISBNet, have you ever tested Open3DIS on ScanNet200 with Mask3D? Thanks!
Best.

Hi @whuhxb,
We have also tested Open3DIS with Mask3D, detailed results are presented at our project page
We are planning to upload the code for Mask3D 3D proposal generation in the near future. Stay tuned!
Best,
PhucNDA.
.

Hi @whuhxb,
We have also tested Open3DIS with Mask3D, detailed results are presented at our project page
We are planning to upload the code for Mask3D 3D proposal generation in the near future. Stay tuned!
Best, PhucNDA.
.

Great! Looking forwards to the code release of Mask3D 3D proposal generation. According to your experience, what's the result differences of Mask3D 3D proposal and ISBNet 3D proposal generation? Thanks.

Hi @whuhxb,
We have also tested Open3DIS with Mask3D, detailed results are presented at our project page
We are planning to upload the code for Mask3D 3D proposal generation in the near future. Stay tuned!
Best, PhucNDA.
.

Great! Looking forwards to the code release of Mask3D 3D proposal generation! Thanks.

same problem here


Fix typos in PT-BR translations

Fixed some typos to Brazilian Portuguese translation.

Thank you for contributing!  This really helps with the site


Feather m0 examples

Two "GraphicsMode" demos and one "TerminalMode" demo that use an Adafruit Feather M0 with the Adafruit OLED Featherwing, which uses a SSD1306 chipset and has a resolution of 128x32 pixels.  The demos can also be run with generic SSD1306 modules that are correctly connected to the Feather M0 board.

Looks good, although aside from the other comments I'd be wary of using an alpha release of both ssd1306 and embedded-graphics, but that's up to the atsamd maintainers to decide whether to accept or not 🙂

The name of the size method to the Builder struct changed in ssd1306 between 0.2.6 and 0.3.x, just trying to future-proof the demo as much as possible.  The embedded-graphics dep version comes from the version of ssd1306 that was used for these demos.

I don't really care what version you use, and I commend your attempts to future-proof. Incorporate the triangle primitive and we should be good to go.

I don't really care what version you use

I brought it up as a point for consideration - there are still many breaking changes coming in future alphas of embedded-graphics which might cause some maintenance pain, negating the future proofing. If you're happy with that, then 0.6.0-alpha.2 is fine :slightly_smiling_face:

@jamwaffles How far away is a non-alpha release of ssd1306 and embedded_graphics?  I think for these small examples, following breaking changes should not be too much of a burden... I'm willing to do the work as needed, and I'd like to also start submitting PRs for updating primitives in the ssd1306 examples.

How far away is a non-alpha release of ssd1306 and embedded_graphics?

Hard to say, but not that soon as I can only work on it in my limited spare time.

I think for these small examples, following breaking changes should not be too much of a burden.

Yeah good point, shouldn't be hard to keep up with e-g releases with this small amount of code 🙂

I'd like to also start submitting PRs for updating primitives in the ssd1306 examples.

Thanks!


random error occurs on early startup (no such file or directory)

I am measuring the effect of end to end delay on performance. The delay was simulated using the tc command.

Containers work properly in environments without delay, but containers do not run randomly when delay is set.

If I start the container after pulling all of the image, this error does not occur.
Please let me know if you need additional information for the reproducing. Thank you!
Environment

v0.3.1
Followed newbie.md step by step (exclude adjusting the TCP window size
Proxy(with registry) and edge node running in VirtualBox (Arch Linux)
The RTT was set to 100ms on the edge using 'tc' Linux command

I found something that might help solve this problem.

I found that ReferencedFile.Ready is not set correctly in manager.go
https://github.com/mc256/starlight/blob/f4f99992dcd67c1c38867cafbd29f829361252f5/client/manager.go#L311-L317
I think it's because ReferencedFile.InPayload returns false for files included in payload, so the file is not waited for being downloaded.
It's the implementation of InPayload.
https://github.com/mc256/starlight/blob/f4f99992dcd67c1c38867cafbd29f829361252f5/util/receive/image.go#L197-L202
I changed the function to return true for ReferencedFile that PayloadOrder is greater than 0
func (r *ReferencedFile) InPayload() (stack int64, inPayload bool) {
	if r.PayloadOrder > 0 {
		return r.ReferenceStack, true
	}
	return -1, false
}

Then it worked properly.

Sorry for the late response. I was on vacation last week.
Thank you very much for identifying this very critical bug. I thought this issue was related to the containerd config ( in https://github.com/mc256/starlight/issues/26#issuecomment-1374969087 ), but it is more serious than that. The implantation of InPayload is completely wrong.
v0.3.2 should resolve this problem.
I will work on creating unit testing to ensure all the features are functioning are working as expected. I am still doing manual testing, which is not a good approach.

I’m glad I was able to help. I have checked that the new version has resolved the error. Thank you for taking the time.


Copy stable patches to 6.1

This change is needed in 6.1 as well but this was missed during the 6.1/6.2 split. It is currently pending in AUTOSEL so it should make its way to these trees shortly:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Additionally, this patch is now in mainline, so make that clear by renaming the patch to the upstream SHA and including a link to the commit on git.kernel.org.

I guess these are out for review now:
<EMAIL_ADDRESS><EMAIL_ADDRESS>I am not sure this workflow will run again before the next update so maybe we want to hold off on this? At the same time, I will have to drop the patches in 6.2 anyways so it probably makes sense to keep everything in sync.

The patch has been merged and released, so I will push a new PR removing the patch from 6.1, rather than adding it to 6.2


update history

Description
Some changes done while including in conda-forge.
Type of change
Please check options that are relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] 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] I have read the contributing guidelines
[x] I have updated the HISTORY.rst and AUTHORS.rst files
[ ] Linting passes successfully : flake8 . --exclude=doc
[ ] Typing passes successfully : mypy mapie examples --strict
[ ] Unit tests pass successfully : pytest -vs --doctest-modules mapie
[ ] Coverage is 100% : pytest -vs --doctest-modules --cov-branch --cov=mapie --pyargs mapie
[ ] Documentation builds successfully : cd doc; make clean; make html

Codecov Report

Merging #64 (24469b6) into master (0f24a90) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #64   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines          554       554
  Branches        42        42
=========================================
  Hits           554       554

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 0f24a90...24469b6. Read the comment docs.


env.render(mode='rgb_array') returns none

I’m trying to run MaMuJoCo on a headless server, but when I use env.render(mode='rgb_array'), it returns none. I don’t understand why.
Could you please help me? Thank you!

Does this only happen in the headless server? @Kallinteris-Andreas do you have any clue on what could be the issue. I'll have a look at it as well

The issue is that ma-mujoco environments are supposed to follow the PettingZoo API. However, since this is achieved by wrapping the MuJoCo Gymnasium environments, the renderer is initialized as if it belonged to the Gymnasium API (passing the render_mode when the environment is initialized). We should change this to make it fully compatible with PettingZoo. @Kallinteris-Andreas do you have the time to work on this feature?
Also, @fyqqyf in the mean time could you try if initializing the renderer as follows works for your use case:
from gymnasium_robotics import mamujoco_v0

env = mamujoco_v0.parallel_env("Hopper", "3x1", render_mode='"rgb_array")

env.reset()
env.render()

@fyqqyf the issue if fixed now, feel free to try it


What about to support a web page for kudo

What would you like to be added:
A web page for kudo
Why is this needed:
We could use kudo client or kubectl to create or update instance, but when we would like to support some operatorvertsion for some ops users, and then they could create or update instance by themselves, they couldn't understand kudo client, but a web page could be a easier understanding way.

Hi @abmybgx, KUDO actually has a website! https://kudo.dev/

@guenter This is more about a UI for KUDO... we need to triage but thinking this is long term important, but not for right now


rewrite the readme (again)

Rewrite of the README. I don't think I'm going to keep this, but creating a PR in case anyone has feedback.

Okay, since @dethell-jh likes the changes I'm going to go ahead and merge. :)


Add MovingReferenceFrame Option to UIInputModule

Add MovingReferenceFrame Option to UIInputModule under the Advanced Section; check it to support moving reference frames in your UI (at the slight additional cost of retransforming the leap hand data for the EventSystem "Process" callback).
Also ensure that your event system is childed to your moving reference frame to ensure the visual cursors do not lag behind either.

Wrong target branch