Unreasonable CPU usage while idle and connected

Hello,
I'm using version 0.2.11 (go1.4.2)
I have about 8 sockets connected, doing nothing. The process websocketd is calling is consuming about 12% CPU. The machine isn't doing any other tasks. System load is uncomfortably high (8+)
Performing an strace appears to be also idle on the same pid producing the load:
PID USER      PR  NI    VIRT    RES    SHR S %CPU %MEM     TIME+ COMMAND
22796 root      20   0  678896  18160   1424 S 14.9  1.8   2986:22 testws.py
22574 root      20   0  514528  17316   1324 S 14.3  1.7   2987:39 testws.py
1143 root      20   0  514528  17308   1308 S 12.3  1.7 747:01.43 testws.py
3275 root      20   0  514528  18028   1920 S 12.3  1.8 744:41.68 testws.py
20156 root      20   0  514524  19200   1452 S 12.3  1.9 769:21.55 testws.py
21664 root      20   0  514528  15120   1144 S 12.3  1.5 767:05.90 testws.py
22708 root      20   0  661992  18792    848 S 12.3  1.9   2987:09 testws.py
12636 root      20   0  440536  16872   1428 S 11.9  1.7 785:42.83 testws.py
root@websocket:~# strace -p 12636
Process 12636 attached
futex(0x2a1b0d0, FUTEX_WAIT_PRIVATE, 0, NULL
<nothing happens here.. waited about 10-20 seconds>
Any ideas? I know this isn't specifically websocketd doing the hang, but this is just a python script doing nothing really.

Actually, I'm thinking the issue here is that if you follow the python examples, you'll see reads from STDIN in a tight loop, which is what we did. I'm thinking that this is hard on the system. Does that seem reasonable? Is there a better way to do this that doesn't overuse CPU?
Thx

I think you are right, it's from python, since you'd see websocketd in top if it'll eat resources.
Any opinion on how to rewrite those python examples? I'd be happy to accept your PR.

I'm testing this out with a sleep(0.1) in the while loop. I think it just needs some breathing room. I'll report back with my results.

That did the trick. Without the sleep it's a CPU hog. Please make a note in the examples. FWIW other languages probably have similar issues.
Thanks!

Thanks


ci: fix PR coverage issues due to parallelism

Overview

updates required builds for codecov to match CircleCI parallelism for tests (whoops!)
Also adds a goal % for the gamut package to at least keep it where it is now (89% ish)

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://605971426454b76390391cbe--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6059756694aa7d5b9c1b0a2c--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://605979381730ea4cf8607d73--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6059e5db9495492d561d3c19--gamut-preview.netlify.appDeploy Logs


Update checkout mutations

I want to merge this change because...
Extend checkoutPaymentCreate mutation SALEOR-4229
Update CheckoutComplete mutation SALEOR-4232
Update payment flow SALEOR-4233
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4233 Update payment flow

Task linked: SALEOR-4232 Update CheckoutComplete mutation

Task linked: SALEOR-4229 Extend checkoutPaymentCreate mutation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: stnatic:x: mateuszgrzybYou have signed the CLA already but the status is still pending? Let us recheck it.

@mateuszgrzyb  It looks good. Let's solve the last comment and we can merge it 👍
Most of the review has been done in this PR - https://github.com/mirumee/saleor/pull/7956


TypeError: rescale() got an unexpected keyword argument 'multichannel'

(py34) sivajyothi@PC243-52U:~/acdc_segmenter$ python train.py
2019-02-18 13:58:42,026 Could not import opencv. Augmentation functions will be unavailable.
2019-02-18 13:58:44,241 !! No GPU setup defined. Perhaps you need to set CUDA_VISIBLE_DEVICES etc...?
2019-02-18 13:58:44,241 Could not find cv2. If you want to use augmentation function you need to setup OpenCV.
2019-02-18 13:58:44,241 EXPERIMENT NAME: unet3D_bn_modified_wxent_tfbn
2019-02-18 13:58:44,241 This configuration of mode, size and target resolution has not yet been preprocessed
2019-02-18 13:58:44,241 Preprocessing now!
2019-02-18 13:58:44,242 Counting files and parsing meta data...
2019-02-18 13:58:44,336 Parsing image files
2019-02-18 13:58:44,336 -----------------------------------------------------------
2019-02-18 13:58:44,336 Doing: /home/sivajyothi/acdc_segmenter/data/training/patient035/patient035_frame01.nii.gz
2019-02-18 13:58:44,355 Pixel size:
2019-02-18 13:58:44,355 (1.67969, 1.67969, 5.0)
Traceback (most recent call last):
File "train.py", line 528, in
main()
File "train.py", line 518, in main
run_training(continue_run)
File "train.py", line 79, in run_training
split_test_train=(not train_on_all_data)
File "/home/sivajyothi/acdc_segmenter/acdc_data.py", line 365, in load_and_maybe_process_data
prepare_data(input_folder, data_file_path, mode, size, target_resolution, split_test_train=split_test_train)
File "/home/sivajyothi/acdc_segmenter/acdc_data.py", line 203, in prepare_data
mode='constant')
TypeError: rescale() got an unexpected keyword argument 'multichannel'
---- Can you please help me to resolve this issue???
Ubuntu - 18.04

At the time of publishing this code this feature was only available in the bleeding edge version of scikit-image. Perhaps, now it has been integrated into the normal releases. I am not sure. Have you installed all the requirements according to the requirements.txt file? In particular:
pip install -e git+https://github.com/scikit-image/scikit-image/#egg=scikit-image
Let me know if this fixes the problem?

pip install -e git+https://github.com/scikit-image/scikit-image/#egg=scikit-image
DEPRECATION: Python 3.4 support has been deprecated. pip 19.1 will be the last one supporting it. Please upgrade your Python as Python 3.4 won't be maintained after March 2019 (cf PEP 429).
Obtaining scikit-image from git+https://github.com/scikit-image/scikit-image/#egg=scikit-image
Updating ./src/scikit-image clone
Complete output from command python setup.py egg_info:
Python 3.4.5 detected.
scikit-image 0.15+ support only Python 3.5 and above.

For Python 2.7, please install the 0.14.x Long Term Support using:

 $ pip install 'scikit-image<0.15'

----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /home/sivajyothi/acdc_segmenter/src/scikit-image/
--- yes, i installed all the requirements mentioned in requirements.txt
while installing scikit-image i got the error as mentioned above, that'swhy i installed scikit-image version 0.12.3 using the command 'pip install scikit-image==0.12.3'

Okay I see, their bleeding edge version is no longer compatible with python 3.5. I checked and I am successfully running the code with scikit-image==0.14.0. Could you give it a go, please? If it works I will update the dependencies.

(py34) sivajyothi@PC243-52U:~/acdc_segmenter$ python train.py
Traceback (most recent call last):
File "train.py", line 13, in
import image_utils
File "/home/sivajyothi/acdc_segmenter/image_utils.py", line 6, in
from skimage import measure
File "/home/sivajyothi/anaconda3/envs/py34/lib/python3.4/site-packages/skimage/init.py", line 157, in
from .util.dtype import *
File "/home/sivajyothi/anaconda3/envs/py34/lib/python3.4/site-packages/skimage/util/init.py", line 8, in
from .arraycrop import crop
File "/home/sivajyothi/anaconda3/envs/py34/lib/python3.4/site-packages/skimage/util/arraycrop.py", line 8, in
from numpy.lib.arraypad import _validate_lengths
ImportError: cannot import name '_validate_lengths'
---- i installed scikit-image==0.14.0 now. Now i'm getting different error.

Hello, because support for Python 3.4 is ending soon I upgraded all the dependencies for Python 3.5 and tensorflow 1.12 while I was at it. I pasted my pip freeze into the requirements.txt file. This exact configuration works for me now. Could you please confirm that it works for you as well? Best wishes, Christian

Hello, it worked for me when i used python 3.5 environment. Nice work.
Thank you.

you can aslo change multichanel to channel_axits

you can aslo change multichanel to channel_axits

setting channel_axis=-1 works for color images


When call has been hung up not from app, callkit is still running

Hey guys,
So I realized that when someone who I am talking to hang up before I do from the app CallKit is still running in the back. I guess CallKit isn't being notified that Twilio has been disconnected so its still running in the background. Is there a way around this?
Thank you

Hi @jaelee1993
Thanks for reporting the issue.
When you observed that the CallKit call was not properly terminated, were you able to see any CallKit request error or Voice SDK error in the debugger?
-bobie

@bchen-twilio Thank you getting back to me.
So I don't think there was an error, however CallKit was continuously running in the back ground. The only way to kill it was to close my app.
`func call(_ call: TVOCall, didDisconnectWithError error: Error?) {
if let error = error {
NSLog("Call failed: (error.localizedDescription)")
} else {
NSLog("Call disconnected")
performEndCallAction(uuid: self.call!.uuid)  <---Added here
    }

    callDisconnected()
}`

I added the performEndCallAction in the function above and seemed to resolve the issue.

Hi @jaelee1993
Actually right after I replied last week we realized this is actually a regression caused by this PR. We will update the Voice quickstart to address this issue but you can try the fix in the Video iOS quickstart PR.
Sorry for the inconvenience.
Thanks
-bobie

Hi @jaelee1993
We have updated the sample apps recently to address the issue. I am closing the issue but please feel free to reopen if you believe the issue still exists with the fixes or open a new issue if you run into any problems.
Thanks
-bobie


Adds useSignTransaction hook

This sits on top of #36, will need to view 2e1125adc4efd70795365dfcd678e8717b81a66d to see the change.
Here's the loom, the first demo in the video shows it working. The second part of the video needs investigation, I think it's something to do with the set up, but will need to confirm with Bin.
https://www.loom.com/share/6ea92ac25ecf47e892cf77ac63dd9ca7

loom link I don’t see?

@dborstelmann updated sorry

fyi I couldn't get any audio out of the loom. Only comment, should we add a decline transaction button or cancel button?

I like that idea.


Compile PDF file

https://github.com/zjkmxy/ndn-workspace-solid/blob/3e46cb7757ff9fcdaeabd339c16a4212d175e01a/src/components/share-latex/share-latex.tsx#L169-L182
Current implementation tries to push a ZIP file to localhost's 6175 port, which is supposed to be running a LaTeX server.
We need a better design for this.
Note: the LaTeX server's code is not published. Please let me know if you need to access it.
Idea: There are two ways. Need discussion.

Use RICE like discovery protocol, so the LaTeX server does not need to join the Sync group.
Let the LaTeX server to join the sync group.

Is it possible to compile the LaTeX server to WebAssembly, and then run it as WebWorker?

Is it possible to compile the LaTeX server to WebAssembly, and then run it as WebWorker?

Yes. We have discussed this idea from beginning: https://github.com/SwiftLaTeX/SwiftLaTeX
The pros:

Works for tablets and phones.
Can work offline.
The cons:
The only existing LaTeX in WASM uses AGPL license, which may affect the deployment.
If we ship LaTeX, we need to also ship those common libraries, fonts, etc. But there are too many of them.
Therefore, we think we should try other methods first.

Ah I missed this. So I tweaked the implementation in #28 and the compilation speeds are really impressive. The initial compile times take a bit of time but my guess is that has partly to do with the package service. I believe it also already works with usepackage (see screenshot) and pulls libraries on demand (albeit from their server, though this should be possible to eventually fix)
I'm encouranged to push for this direction in favor of truly local-first software. Plus it seems like a nice way to show "completeness" in our retreat demo (BTW the header for bootstrapping says "Hackathon" I think we'll probably test this during retreat?)
Also we can sidestep AGPL restrictions by keeping all compiler code strictly separated (easy enough) and make it possible to build it as an optional module, similar to what ffmpeg does. Anyway, I'm actually in favor of AGPL (I don't want/see anyone commercializing any part of this) but that's not my call of course.

Thank you for your hard work!
Yes. We will demo this in the retreat, and then consider wrapping a hackathon project.
We can separate the compiler as a downloadable module. I'm not sure about the limitation of AGPL.

Ok, great. How do you suggest to proceed? Can you review the draft PR? Actually I don't think I can make it much cleaner than it is.

Ok, great. How do you suggest to proceed? Can you review the draft PR? Actually I don't think I can make it much cleaner than it is.
I still prefer to set up a server... But let's discuss this on the retreat.
Will review the current code when I have time.

Ref: zjkmxy/ndn-workspace-compiler

Not perfect but currently working. Close for now.
Enhancement will be handled in a different issue.

After #28 is merged, it is able to compile PDF files completely offline after caching necessary files.
I verified that I can compile even on my phone.


Replaced "threshold" property of CyclomaticComplexMethod rule by "allowedComplexity"

The CyclomaticComplexMethod rule reported an issue when the cyclomatic complexity is greater or equal the value defined for the "threshold" property. It is intended that the value defined in the rule is the maximum allowed cyclomatic complexity. So the property is renamed to "allowedComplexity" and the check for reporting an issue is changed to check only for greater, no longer for equal.
The origin issue is https://github.com/detekt/detekt/issues/3679

Warnings

:warning:
This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

  Generated by :no_entry_sign: dangerJS against 674bf71415fc9a03426b57171b456dfa03004baa

Thanks!!!


Docker deamon doesn't work with --fixed-cidr on windows

Docker deamon fails to start when launched with --fixed-cidr parameter on windows.
This is because we are sending an invalid gateway address. Sending a gateway address was a temporary workaround which was added to prevent the daemon from crashing and is not needed any more so removing that.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fixcidr" git@github.com:msabansal/libnetwork.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

Can we merge this?


Mutant Crawl throws Error

Error with the Mutant Crawl Ruleset, caused by no actext class entry in the NPC sheet.

Fixed with 2.9.15


navbar button group with dropdown using an anchor tag

This glitches display on the button when dropdown is open. You can get around this with some markup gymnastics and it doesn't really work with a dropdown button inside an existing form.
Broken: http://www.bootply.com/7vhKmOy2sn
Working: http://www.bootply.com/4VrKDN6fz9
The working example though, can not work inside an actual form. Coincidentally, this also presents an additional solution to #16266, but for that this is even more excess markup. I came across both issues while trying to find the leanest possible markup to do something fairly similar to this. The 30 second solution for the dropdown issue is to obviously use jquery event handlers, but it isn't very lean either.

Hi @CriseX!
You appear to have posted a live example (http://s.bootply.com/render/7vhKmOy2sn), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 61, column 21: W007: Found one or more <button>s missing a type attribute.
line 9, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.2
line 78, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.2

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

Hi @CriseX!
You appear to have posted a live example (http://s.bootply.com/render/4VrKDN6fz9), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 62, column 21: W007: Found one or more <button>s missing a type attribute.
line 9, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.2
line 80, column 9: W013: Bootstrap version might be outdated. Latest version is at least 3.3.4 ; saw what appears to be usage of Bootstrap 3.3.2

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

Bots, sigh, would have if I could have :).

With a little more fiddling, removing the navbar link list, and having just the button group the glitch seems to go away, so it originates from the set of navbar links. not the dropdown component per se. But the particular mix of the two.

@CriseX Please use JS Bin instead of Bootply so that you can use Bootstrap v3.3.4. Thanks.

Sure, I can do that, but it shouldn't change anything... I came across the issue 3.3.4, for the record the issue itself looks even more broken with the non-flat theme file applied to it.
Ironically codeply offers the latest version whereas bootply does not, so if you click run on codeply and change the version it will show you the render using 3.3.4, can't save it without creating an account though.

The navbar isn't terribly flexible (and is going to be overhauled in v4). You can't put arbitrary stuff inside .navbar-navs.
Moving the btn-group outside of the navbar-nav seems to work: http://jsbin.com/yexuwi/2/edit

Hi @cvrebert!
You appear to have posted a live example (http://jsbin.com/yexuwi/2/edit), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

W002: <head> is missing X-UA-Compatible <meta> tag that disables old IE compatibility modes
W003: <head> is missing viewport <meta> tag that enables responsiveness

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

You are right it does work, like I said I came across this trying to combine static text label and dropdown button group (the referenced issue) which is why this is one of the approaches I tried (ie, replace the first list item link in the right nav with the desired text).
Looks like this can be closed then.

Actually going to re-open this... see what happens when navbar is collapsed when using your bin http://jsbin.com/yexuwi/4, I see some overflow issues with the navbar and an opened dropdown.
You may close this again once you decide if  this is not something that can be addressed in v3,

Yeah, it overflows, but you can scroll to reach the rest of it.
Closing as out of scope for v3. Hopefully v4's navbar will be less wonky.


How Java or Spring Boot projects should integrate with Uber Piranha

I tried adding this dependency but this is not getting resolved
<dependency>     <groupId>com.uber.piranha</groupId>     <artifactId>piranha</artifactId>     <version>0.1.8</version> </dependency>
I cloned https://github.com/uber/piranha but i didn't find pom.xml
as per documentation I see it supports java but how to integrate with spring boot project

Hi @rajeshchinnam. Piranha does indeed support Java (as in it can transform / cleanup Java code). However, the tool itself is written in rust and it also has python bindings (which you can get prom PyPI using pip). See the example in the README to get started with it. Piranha itself is not a java package.

Thanks @danieltrt  so if I have to integrate Piranha with spring boot application to clean-up feature toggles
can you please suggest what are the steps I need to follow , if any documentation is available please let me know
That would be of great help.
I am trying adding maven jar and plugins but unsuccessful to run
`

4.0.0

org.springframework.boot
spring-boot-starter-parent
3.3.3

com.piranha
demo
0.0.1-SNAPSHOT
piranha
Demo project for Spring Boot piranha

<java.version>21</java.version>

org.springframework.boot
spring-boot-starter

com.uber.piranha
piranha
0.1.8

org.springframework.boot
spring-boot-starter-test
test

org.springframework.boot
spring-boot-starter-web

org.springframework.boot
spring-boot-maven-plugin

com.uber.piranha
piranha-maven-plugin
0.1.8

process-sources

remove

`


Throw exception if filter factory types are provided to either TypeFi…

…lterAttribute or ServiceFilterAttribute
[Fixes #7855] AutoValidateAntiforgeryTokenAttribute does not validate requests when registered globally by type

Closing this PR as the fixes are not good as it cause breaking changes.


Exceptions and compatibility

Game seems slower after installing mod, a lot of Exceptions, not compatibile with life and death pack

BE-ExceptionReport-20241117-191719.zip

Read the readme.
The mod is no longer functional.
Please make a fork of the repo if you want to fix it.


fix: widen signature of event

There are situations where persist doesn't exist, see: https://github.com/TanStack/router/pull/1365#discussion_r1552986706

@TkDodo Thanks a lot! This should increase compatibility with the ecosystem.


Links in the Bookmark not working

If the bookmark is an html link in the PDF, then PDF.js disables the link.
https://fbutools-stage.rightmix.com/Viewer.aspx?file=%2FViewer%2FHosted%2FGetDocument.ashx%3Ffile%3Dhttp%3A%2F%2Fstage.efinancialdocs.com%2Fdocuments%2FDocData%2FFBU2001_DOC_S_201532112645_FBU2001_LOAD.PDF
Below is the screenshot from Adobe PDF viewer, which shows the link and able to click and navigate

Closing as duplicate of #3214 which not only has links that do not work but also markup that is not shown.


Fatal error issues

Hello, I am developing a web-app that verifies user through nexmo-php.
I try to redirect users to different pages based on the response status.
I implement it by doing:
if($verification['status'] == "0"){
	header("Location: url1");
}
else{
	header("Location: url2");
}

I try to type in wrong verification code to testify my code, but things didn't go as I expected.
I got:
Fatal error: Uncaught Nexmo\Client\Exception\Request: The code provided does not match the expected value in /Users/amos_mac/Sites/Nexmo_test/vendor/nexmo/client/src/Verify/Client.php:158 Stack trace: #0 /Users/amos_mac/Sites/Nexmo_test/vendor/nexmo/client/src/Verify/Client.php(104): Nexmo\Verify\Client->checkError(Object(Nexmo\Verify\Verification), Array) #1 /Users/amos_mac/Sites/Nexmo_test/check.php(11): Nexmo\Verify\Client->check(Object(Nexmo\Verify\Verification), '123456') #2 {main} thrown in /Users/amos_mac/Sites/Nexmo_test/vendor/nexmo/client/src/Verify/Client.php on line 158
Really hope that you can help me out.
Thanks!

Hi @amoossssss,
The library throws an exception if the verification fails. To achieve what you're trying to do you need the following code:
try {
    $client->verify()->check($verification, '1234');
    header("Location: url1");
} catch (Exception  $e) {
    header("Location: url2");
}

That works perfectly!
Thanks for your time, really appreciated.
:)

No worries, feel free to reach out if you have any more questions


Add a warning sign when renaming one blob container using an invalid name

Storage Explorer Version: 1.11.0
Build: 20191112.2
Branch: hotfix/1.11.1
Platform/OS: Windows 10/ Linux Ubuntu 19.04/macOS High Sierra
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Expand one Non-ADLS Gen2 storage account -> Blob Containers.
Create a new blob container -> Try to rename it using an invalid name. (like: 'bl')
Check the 'Rename' dialog.

Expect Experience:
A warning sign occurs.
Actual Experience:
No warning sign occurs.

More Info:

There is a warning sign when creating a blob container with an invalid name.

This issue also reproduces for one file share/ table.

Duplicate of #2258


Add test_commit script

The role of test_commit.py is to run unit tests, record coverage (e.g. in a file called coverage.txt), and then commit the test results. The intention is to improve code reliability and motivate low technical debt.

each PR should include a test_commit commit near the tip of the work to be merged.

test_commit added in PR #23


Clean chrome browser instructions

Now that the "shader-f16" feature is enabled by default in Chrome, we can clean the Chrome-specific instructions.

cc @CharlieFRuan let us run a site deployment


Implement a better query system for PocketDimension

Current, the QueryBuilder that is used for PocketDimension feels generic and would be missing out on things like complete filtering, comparison, etc. While may be simple enough to get by in simple searches in any cache/db system, some implementations may want to perform more complex queries.
Not wanting to make it extremely complex or very simple, we may want to look into adapting several functionality into the query system that can be extended into PocketDimension that would allow for flexibility.
Currently, it may be better to have the query system that would be json-style, similar to nosql queries like mongodb. This would leave it up to the extension to implement their owns system or possibly use our own system base on this style and collect and use the data that way.

Blocked pending #63

Closing since https://github.com/Satellite-im/Warp/issues/63 would likely improve on this.


Parallellise the parsing if possible

To speed up parsing -- right now we hit 45K messages/s when they all fail to parse right away, and 7K messages/s for torque logs -- it would be interesting to see if the parsing can be done in parallel, so
the input pipeline can be processed faster.

can Conduit handle this?
is there a simple way to par the function without risking races downstream?
can this be put into the config file?

See if this helps: https://hackage.haskell.org/package/stm-conduit-3.0.0/docs/Data-Conduit-Async.html


LWM2M Gateway (Object No 25) support

@sbernard31 We've seen that you also looked into the Gateway feature specified with LWM2M 1.2.  Like yourself, I was also confused by it at first, but thanks to the discussion and the Gateway spec, it becomes quite clear what this is all about. And it is pretty exciting. Out of curiosity, do you know about any activity or if somebody implementing Gateway support in Leshan officially? Or is this currently on your agenda?
A customer of ours informed us a couple of weeks ago that they hacked Gateway support into the Leshan code base somehow, simply with the goal of figuring out if it can be done. It turned out to be possible by hacking around the limitations of the Link class (number of path elements, string prefixes required) in combination with some other hacky ways not shown during this call. Certainly not something to directly use, but it was great to see it can be done.
We have encouraged them to push it to a branch here, but I'd also like to know if there is possibly somebody else also working on it already. From our perspective, Gateway is a highly anticipated feature...

Just see your question at OMA : https://github.com/OpenMobileAlliance/OMA_LwM2M_for_Developers/issues/518#issuecomment-1102356105
Hopping that my answer is complete enough : https://github.com/OpenMobileAlliance/OMA_LwM2M_for_Developers/issues/518#issuecomment-1102387967
But feel free to ask more if I missed something :slightly_smiling_face:

Oh I see you delete your question ? Should I delete it too ? and copy/paste my answer here ?

@sbernard31 I had pasted the same question to https://github.com/OpenMobileAlliance/, but I've deleted it there, because it belongs to Leshan, not the spec.

Ok so I will deleted all of this and we just keep your question and my answer here ? :)

do you know about any activity of somebody implementing Gateway support in Leshan officially?

AFIAK, nobody is working on this currently.

Or is this currently on your agenda?

If I remember well this is a LWM2M v1.2  feature, right?
And unfortunately we are sill focus on implementing LWM2M 1.1.  : https://github.com/eclipse/leshan/issues/1235
Sometime, we feel we are not fast enough :disappointed:

A a customer of ours informed us a couple of weeks ago that they hacked Gateway support into the Leshan code base somehow, simply with the goal of figuring out if it can be done. It turned out to be possible by hacking around the limitations of the Link class (number of path elements, string prefixes required) in combination with some other hacky ways. Certainly not something to directly use, but it was great to see it can be done.

Yep, great to know that.
We recently refactor a lot about  Link in Leshan the current development branch (master used to develop the v2.0.0 of Leshan) I don't know if this will make the change easier or harder :grimacing:. Or maybe you are more talking about LwM2mPath class ?

We have encouraged them to push it to a branch here,

Yes this is good idea. Even if this will probably not be integrated soon, this could be used as base for future development and/or for discussion and/or for other users who can not wait for a version of Leshan targeting LWM2M v1.2.

From our perspective, Gateway is a highly anticipated feature...

OK I will add this feature to the list  at : https://github.com/eclipse/leshan/issues/1235
@sag-eweingaertner, thx for sharing this with us :pray:

Should we close this issue ?

I'd suggest to keep it open, simply to have a forum for people interested in this feature in Leshan.

I'm currently maintaining a fork of Leshan with object 25 support. It's kinda ugly (it's abusing the rootprefix) but it kinda works. So it's on my to-do list to clean up this implementation and makes it consumable by the upstream.
As Simon said it's a 1.2 feature, but strangely it doesn't depend on any other 1.2 feature so it can run on 1.1 (even if it's not the 'spirit' of the spec).

So in place of keeping our dirty hack in my corner, I started cleaning it and added a branch in case someone is interested and would also like to provide feedback:
https://github.com/eclipse/leshan/compare/obj25
First, we implemented this 1.2 feature on top of the 1.1 branch. Strangely this feature doesn't require any part of 1.2, so it works as is on 1.1 (no specification modification). Same for the client we are using, it's done on a 1.1 client.
The idea of this implementation is to create a registration for end IoT devices, reusing the CoAP identity/session of the gateway.
All the attached end IoT device registrations will be if the gateway is unregisteredremoved.
How to add the registrations? Well, it's not covered by the spec. It's up to the user. What we do in our implementation is we have a registration handler, which reads/25 (if it's advertised by gateway registration).
Once /25 is read, we register the end IoT devices.
At some point, I'll clean up and also publish this piece of code.

@jvermillard, I add a link to your blog post about this : https://vermillard.com/post/object25/

I took some time to look at object25.
I created a obj25_alt branch largely inspired by  obj25 one.
@jvermillard, I didn't tested but if you find time you can review/try it and let me know if this way works for you.
The main benefits of obj25_alt is there is almost no changes on current code base.
The leshan-server-gateway module I created in this branch is not intended to be included in master or released in 2.x, this is just to show it is maybe possible to do pretty much the same thing that we do in obj25 with current API.
I also read more carefully the Gateway specification and current code should probably not work  with :

Content format SenML JSON/CBOR. Because, "the Prefix MUST be part of the name of the IoT Device Resources or Resource Instances." (see LWM2M-Gateway-v1.1§9. Data Formats) Maybe relative to this issue : https://github.com/eclipse/leshan/issues/1438

Composite and Send Operations. A clean way to support that means probably to change lot of API about Path, Nodes and Encoder/Decoder. I didn't think too much about it and currently don't know if there is way to workaround this by "twisting" current API.

After looking at the code, it looks like a good idea; I need to test it now.

I started to test it, it's starting to work. I need to finish, but I don't think the idea has fundamental issues.
Some first minor comments:

some code is bugged in the server-gateway like https://github.com/eclipse/leshan/compare/obj25_alt#diff-8e1f484829212d5b508da205b4249430c610d52c9979ca6149ef5332834b2631R81-R83 the appData is not passed to the registration builder
I find the customization of the server using inheritance, where most of Leshan code uses the builder pattern, confusing: https://github.com/eclipse/leshan/compare/obj25_alt#diff-600285a3871e38596be2e1105fd3abf2f99832b8eee64e5b9c99082a8d787753R171

Another feedback but not related is the "application data". Most libraries call it user data, user-defined data, or metadata. I was confused by the Registration.java Javadoc:

@return Some application data which could have been added at Registration by the {@link Authorizer}

It doesn't really explain the purpose: arbitrary data (for example business data or extra client information) associated with the registration by the library user.

The feature was requested by @msangoi.
The idea is , a Leshan Server will call some services to know if a client is authorize to register.
This is done in Authorizer.
During this authorization step, you could get some data (collected from services) that you want to attach to registration > (kind of optimization to avoid to recall that services)
How we use it is a kind of HACK.

My feedback is just on the naming, mainly the usage of the "application" term vs. "user" or "metadata" IMO, it makes sense to me to have a way to attach free-form contextual data to the registration.
On the gateway-service, I'm not using the module; I use it as an example and copy/paste it into my server code to make it work.
I'm going to continue testing it this afternoon :crossed_fingers:

My feedback is just on the naming, mainly the usage of the "application" term vs. "user" or "metadata" IMO, it makes sense to me to have a way to attach free-form contextual data to the registration.

You just want to change javadoc or also getter name ?

On the gateway-service, I'm not using the module; I use it as an example and copy/paste it into my server code to make it work.
I'm going to continue testing it this afternoon crossed_fingers

Maybe this is worth anyway to fix the branch for people which would like to experiment this ?

I put some fix to previous solution in obj25_alt branch.
And I created another branch obj25_alt2` with an alternative way, see : https://github.com/eclipse/leshan/blob/obj25_alt2/leshan-server-gateway/src/main/java/org/eclipse/leshan/server/gateway/IotDevicesRegistrationHandler.java#L64-L96

After some discussion, I settled on the alt2 version since it looks less exposed to Leshan API breakage

To what extent is the implementation of the gateway for obj25, obj_alt and obj_alt2 complete at the specification level?
If there are any parts that have not been implemented, could you please tell us which parts those are?

The alt2 idea works to the extent of some things like /dp send operation. there are probably some other quirks but the base read/write/observation/registration works

Thank you for your answer, jvermillard.
I see that the basic operation works.
Have you verified the operation of the gateway using actual devices?
If the gateway operation is verified under the constraints of device information set in the program, I would like to verify the operation using actual devices.
Could you tell me the current situation?

yes, with an LTE-M LWM2M gateway with Bluetooth sensors as end IoT devices

I would like to ask about obj25_alt2. Has the operation been verified for devices that perform IP communication as end devices communicating with Gateway?

Also, I am thinking of using a master branch server or a demo server when running the Gateway, so did you make any changes to the master branch server side when checking the Gateway operation? Is the server able to recognize the objects sent from the Gateway without any changes on the server side?

Has the operation been verified for devices that perform IP communication as end devices communicating with Gateway?

The spec does not cover how the Gateway and the end device communicate; you can imagine whatever you want. So, I don't understand the question.

That was not a good way to ask the question.
As you say, there is no description of the communication method between the end device and the gateway in the specification. Therefore, I wanted to ask whether obj25_alt2 uses only Bluetooth for communication between the end device and the gateway, or whether it also supports other communication methods (e.g., IP communication).
I would like to ask about the usage of Obj25_alt2.
I would like to register devices to the server via Gateway with Obj25_alt2, but I don't know how to do it. Could you tell me how to do this?
Currently, I am able to start the demo server, but I am not able to register end devices to the server via the gateway.

This branch and feature cover only the server-side implementation, so you need a client implementation of the LWM2M client plus gateway support.

I understood,  I need to support the client and the Gateway myself.
I have two new questions.
The first is about the client side program.
I am trying to set data in the IoTDeviceObjects resource in the Gateway object. (I am setting the data on the program to try it out).
I can see the gateway object in the Leshan demo UI: http://localhost:8080, but the IoTDeviceObjects values are not being reflected.
The code is as follows.
private String ioTDeviceObjects = "</3/0>;ver=1.2,</6/0>;ver1.1,</3303/0>,</3303/1>";
Could you please tell me how you described the IoTDeviceObjects resource in the Gateway object of the client when you checked the operation of the Gateway, jvermillard?
The second question is about the server side program.
I believe that leshan-server-gateway is a server-side program that can process Gateway objects, and I would like to start leshan-server-gateway.
However, I cannot start leshan-server-gateway because the executable file does not exist in the target folder even after performing "mvn clean install".
Could you please tell me what steps I should take to use leshan-server-gateway?

I have been creating a client.
I am having a hard time setting up the IoT Device Objects resource in the Gateway object.
I am thinking that the IoT Device Objects resource should be programmed as a Link type (leshan-core/src/main/java/org/eclipse/leshan/core/link/Link.java), and I am setting the uriReference and attributes.
However, I cannot check the IoT Device Objects resource on the server.
Could you tell me how to program the IoT Device Objects resource on the client side?

From https://github.com/eclipse-leshan/leshan/issues/1237#issuecomment-1503727358 :

Content format SenML JSON/CBOR. Because, "the Prefix MUST be part of the name of the IoT Device Resources or Resource Instances." (see LWM2M-Gateway-v1.1§9. Data Formats) Maybe relative to this issue : https://github.com/eclipse-leshan/leshan/issues/1438

I moved forward about #1438, see #1638.


local llama3 but log show gpt-3.5-turbo

use llama3 local, OpenDevin in docker start with
docker run
--add-host host.docker.internal=host-gateway
-e LLM_API_KEY="ollama"
-e LLM_BASE_URL="http://host.docker.internal:11434"
-e WORKSPACE_MOUNT_PATH=D:/opendevin/workspace
-vD:/opendevin/workspace:/opt/workspace_base
-vD:/opendevin/workspace/docker.sock:/var/run/docker.sock
-p 3000:3000
ghcr.io/opendevin/opendevin:main
error:
root@555c6b64db3f:/app/logs# cat opendevin_2024-04-27.log
12:11:49 - opendevin:ERROR: auth.py:31 - Invalid token
12:11:49 - opendevin:INFO: listen.py:75 - Invalid or missing credentials, generating new session ID: d59448f4-6979-4cbe-9c3e-a34f3c4bea4f
12:11:50 - opendevin:INFO: agent.py:144 - Creating agent MonologueAgent using LLM gpt-3.5-turbo
12:11:50 - opendevin:INFO: llm.py:52 - Initializing LLM with model: gpt-3.5-turbo
12:11:50 - opendevin:ERROR: ssh_box.py:69 - Please check Docker is running using docker ps.
12:11:50 - opendevin:ERROR: agent.py:155 - Error creating controller: Error while fetching server API version: ('Connection aborted.', ConnectionRefusedError(111, 'Connection refused'))
Traceback (most recent call last):
File "/app/.venv/lib/python3.12/site-packages/urllib3/connectionpool.py", line 793, in urlopen
response = self._make_request(
^^^^^^^^^^^^^^^^^^^
File "/app/.venv/lib/python3.12/site-packages/urllib3/connectionpool.py", line 496, in _make_request
conn.request(
File "/app/.venv/lib/python3.12/site-packages/urllib3/connection.py", line 400, in request
self.endheaders()
File "/usr/local/lib/python3.12/http/client.py", line 1331, in endheaders
self._send_output(message_body, encode_chunked=encode_chunked)
File "/usr/local/lib/python3.12/http/client.py", line 1091, in _send_output
self.send(msg)
File "/usr/local/lib/python3.12/http/client.py", line 1035, in send
self.connect()
File "/app/.venv/lib/python3.12/site-packages/docker/transport/unixconn.py", line 27, in connect
sock.connect(self.unix_socket)
ConnectionRefusedError: [Errno 111] Connection refused

@zlw123 you need to set the model in the settings modal in the UI. There's a gear wheel in the bottom right

@zlw123 You will need to use ollama/llama3 as name

The gear is in the bottom right

The modal in the UI

Can you please confirm, you have used llama3 this way, @isavita ?

This is from my console log
INFO:     connection open
Starting loop_recv for sid: 1120a483-1c24-427d-b1fc-a06942e70053
INFO:     <IP_ADDRESS>:64810 - "GET /api/refresh-files HTTP/1.1" 200 OK
15:20:23 - opendevin:INFO: agent.py:144 - Creating agent MonologueAgent using LLM ollama/llama3
15:20:23 - opendevin:INFO: llm.py:52 - Initializing LLM with model: ollama/llama3
15:20:23 - opendevin:INFO: ssh_box.py:353 - Container stopped
15:20:23 - opendevin:WARNING: ssh_box.py:365 - Using port forwarding for Mac OS. Server started by OpenDevin will not be accessible from the host machine at the moment. See https://github.com/OpenDevin/OpenDevin/issues/897 for more information.
15:20:23 - opendevin:INFO: ssh_box.py:373 - Mounting workspace directory: /Users/isavita/code/workspace
15:20:23 - opendevin:INFO: ssh_box.py:396 - Container started
15:20:24 - opendevin:INFO: ssh_box.py:413 - waiting for container to start: 1, container status: running
15:20:25 - opendevin:INFO: ssh_box.py:178 - Connecting to<EMAIL_ADDRESS>via ssh. If you encounter any issues, you can try `ssh -v -p 53053<EMAIL_ADDRESS>with the password 'e9510b7f-aa92-490a-92e6-494f9461658d' and report the issue on GitHub.
INFO:     <IP_ADDRESS>:64810 - "GET /api/litellm-models HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:64814 - "GET /api/messages/total HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:64814 - "GET /api/agents HTTP/1.1" 200 OK

==============
STEP 0

15:21:48 - PLAN
write me working snake game
15:22:02 - ACTION
FileReadAction(path='/workspace/snake_game.js', start=0, end=-1, thoughts='', action='read')
15:22:02 - OBSERVATION
File not found: /workspace/snake_game.js

Looks like this was solved!


fix: remove the image binding step in guided tour

Description
We are removing the image binding step in guided tour. Also we auto bind the property while adding the widget to the canvas.
Other changes include:

Updating the test to remove the image binding step
Hide the start building button shown initially until the query has run successfully
Animate the loading of the table widget
Fix style of the Run text mentioned in #15159

Before

After

Fixes #15118
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Manually
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=1383398

/ok-to-test sha=dff544d

/ok-to-test sha=926000a


Fix: [SEI-5542]: Fixed broken links on integrations landing page

Harness Developer Pull Request
Fixed broken links on integration landing page
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screenshots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65cdac94ecf95f57c4347052--harness-developer.netlify.app


No module google-map-react

When running docker-compose up --build and going to http://localhost:3000/  I get:
Failed to compile
./src/components/NearestHospital/FacilityMap.js
Module not found: Can't resolve 'google-map-react' in '/app/src/components/NearestHospital'
This error occurred during the build time and cannot be dismissed.

@AckslD this is not an issue. You have to rebuild images if you are on docker, on local you will have to install this module. ping me or on engineering channel on slack if you still need more help.


Typo in subtitle

The repo subtitle says "ES6 Object.assign() ponyfill".
Should be polyfill, if that's not intended.

Totally intended. See the readme.

I am sooo sorry, accept my deepest apologies!
:+1:

No worries :p


.pug (the new name for jade) extension doesn't supported by VS

When I create a new nodeJS project, the default view engine is jade. As you know, Jade changed recently to pug. The problem is pug haven't highlighting mechanism in the text editor, and it makes a lot of errors because of the spaces and tab issues.
It will be updated for the next version?
By the way, Is .styl extension also came with nodeJS for VS? If it is, there is also a problem with it. It doesn't get highlight as well.
Except those, great work. Thanks.

Thank you for the repot.
The jade -> pug work is being tracked by #895. We need to update our project templates to use pug and also update the file extension to activate our highlighting for .pug files.
I believe we support creating .styl files, but not currently add syntax highlighting or any sort of intellisense. Supporting that is out of scope for NTVS at the moment.
Closing as duplicate of #895. Let's see if we can get the basic highlighting unblocked soon, even if we need more time to update the templates.


Add progress messages to CLI so know what is doing for common commands.

Fixes: https://github.com/vmware-tanzu-labs/educates-training-platform/issues/90

Example:
$ educates publish-workshop
Processing workshop with name "lab-test-workshop".
Publishing workshop files to "localhost:5001/lab-test-workshop-files:latest".
...
Pushed 'localhost:5001/lab-test-workshop-files@sha256:aabe2878ee48448841863c74280ef890be2cd6d74373d70854ef4bd046c87957'[~/lab-test-workshop] $ educates deploy-workshop
Loaded workshop "educates-cli--lab-test-workshop-ec13558".
Creating new training portal "educates-cli".
Workshop added to training portal.
[~/lab-test-workshop] $ educates browse-workshops
Training portal "educates-cli".
Checking training portal is ready.
Opening training portal http://educates-cli-ui.educates-cli-w01-s001.educates-local-dev.test/workshops/access/?password=vN%3FZtC3JMPDC&redirect_url=%2F.

In waiting for training portal being ready, it will show a spinner.


Can’t find working name for settings menu/gear button

In order to create a sequence that will generate a 30 minute sleep timer, I have to begin with pressing The settings Or gear button, Which brings up the quick settings options that can then be expanded.  I tried SETTINGNS, QSETTINGS, MENU, QMENU among others, but non seem to work. Everything works perfectly and explained in detail in a way that’s very easy to understand, except for this probably obvious thing I’m missing. Would appreciate any advice. Thanks

If it is not present on the known commands list then i guess nobody yet found out what command it is. Maybe there is a different way which leads you to the same result without the need of that menu?


Release helm charts as a OCI artifact at ghcr.io and docker.io using the CI

See: https://helm.sh/docs/topics/registries/

Fixed by #4 PR


How can I change locale runtime?

Hi,
I can load specific locale using amMoment.changeLocale('nb'); in my run(). But if I need to change the locale runtime [for example on a button click] - what will be the suggested way?
Thanks.

I would also like to know this. We're not having any luck with changeLocale inside our unit tests.

+1. I have an app thats an international questionnaire. So the locale can change dynamically based on which questionnaire you get. I need to set this in the controller I believe since they can also change the language-country via a dropdown. Also I cannot just load  in my index.html manually since that locale could change dynamically.

TEST

+1

amMoment.changeLocale(); should do the trick. If you are calling it from your testing code, make sure you invoke angular's $digest() after changing the locale.

worked, thanks! turns out that I was importing moment.js twice.


Broken edit links

Two issues which may or may not be related:

Module sources: the "Edit on github" links for pages like http://www.tornadoweb.org/en/stable/_modules/tornado/stack_context.html are broken; they point to a non-existent .rst file. Is it possible to suppress the edit link for these pages (or ideally point it to the real source)? (migrated from https://github.com/snide/sphinx_rtd_theme/issues/237)
Non-latest branches: on a page like http://www.tornadoweb.org/en/stable/, the "edit on github" link in the upper right is broken because it points to https://github.com/tornadoweb/tornado/blob/origin/stable/docs/index.rst (the 'origin' directory needs to be removed from the path). On the lower left, clicking "v: stable" for the menu and then "Edit" works (linking to https://github.com/tornadoweb/tornado/edit/stable/docs/index.rst).

The "latest" branch works fine (linking to "master"); this is only a problem for pages based on other branches (migrated from https://github.com/snide/sphinx_rtd_theme/issues/236)

Thanks for the report! Indeed these are two distinct issues. The first will be addressed with #1643.
I also had a look at the second, but it's a little tricky. The reason is that stable usually is a special version name on our end. We alias the branch or tag of a project that is a valid version number (like v1.3.4) and the highest version number of all tags/branches as being the stable version. So that's why this name gets an extra treatment.
The Tornado project seems to suffer from this a little as you have a branch called stable. Now the reason why the link in the footer-popup (clicking v: stable) works is that this part of the page does not check for the special stable name (so actually it's broken for all projects that are not having a stable branch in the project) and the link at the top of the page "Edit on GitHub" is broken for your case as it checks for the special name.
I will try to come up with a fix but currently I think for really making the "stable" special casing work, we might need to refactor how we store the branch/commit identifier information. I'll keep you posted.

Ah, I didn't realize that about stable. I'm using the stable branch to do essentially the same thing by hand, so maybe I can just get rid of my branch. How would I replace my stable branch with an alias for branch4.2 or tag v4.2.1?

We are trying to figure out the stable branch ourself by parsing all available branch and tag names. The highest valid version number that is compliant to PEP-440 we be set as the stable branch.

So if I just delete my stable branch from github, readthedocs will see that it's gone and replace it with one of my branches or tags?

The second point of your issue will be addressed with #1650

@bdarnell Should work this way, yes. Maybe you need to push one commit (should be irrelevant which branch) since I don't know if the deletion of a branch triggers a post commit hook.

The second point (bad links from the top right "Edit on GitHub" page) will be fixed with the next deploy as we have merged #1650

So is it fixed?

The issue with the "Edit on Github" for auto-generated pages is still present. I just tried it in my local instance of RTD and I can see the link for auto-generated pages. Also, the last issue linked here from another repo is from November 2016.
Regarding the second one and the branches, I'm not really sure to understand it or how to make a good test.

@ericholscher This issue is still a problem on the website, the "Edit on GitHub" still doesn't work for the stable version.
I see there has been a bunch of related issues (of which I proposed a very very simple fix for the problem in #1820) and PRs. But the issue persists.
What about the approach I suggested? It seems to be a tough issue to tackle with the approaches tried so far, so perhaps just going back to a KISS fix would enable us to have that Edit button working again? :)

@rawtaz the first thing that should be done is to ensure prod runs git 2+ https://github.com/rtfd/readthedocs.org/pull/3615, then the rabbit hole will be less deep.

@ericholscher Is there something blocking #3615 that seems to be the fix for the long-running Edit button problems?

@techtonik Thanks for working on this. I'm still unsure why we don't just make RTD pull in the latest release tag (when available, of course) instead of a stable branch when the latter is missing. Seems like a very simple fix to the problem. Could have been solved a long time ago and without having to upgrade prod etc.

@rawtaz the problem is in annotated tags only. If release tag is annotated, RTD gets the wrong hash.

Hm, I'm not sure we're talking about the same thing :)
What I'm talking about is the fact that when you are on the stable version of a project's docs on RTD, and that project does not have a corresponding stable branch (that is named stable, that is), the Edit on GitHub button fails to identify a proper URL on GitHub to link the button to. It works fine for the master branch (the latest version on RTD), it's just when there's no corresponding stable branch that the button doesn't link properly.
Since RTD does however know about release tags for the project, it can simply use the latest release tag's URL for the edit button, so instead of linking to an improper URL for a non-existing stable branch, it can just link to the state of the master branch at that latest tag.
Let me be super clear by three examples:

The latest version on http://restic.readthedocs.io/en/latest/ has an edit button that goes to https://github.com/restic/restic/blob/master/doc/index.rst - this works.
The version for the last release, v0.8.1 at http://restic.readthedocs.io/en/v0.8.1/, has an edit button that goes to https://github.com/restic/restic/blob/v0.8.1/doc/index.rst - this works too.
The stable version at http://restic.readthedocs.io/en/stable/ has an edit button that goes to https://github.com/restic/restic/blob/84253934823edf6acd2fbb24a7f39ccbd1313617/doc/index.rst - this does NOT work, and from what I've gathered it is because there is no corresponding stable branch in the repository at GitHub.

So, wouldn't a very simple solution to the problem then be to make the edit button for the stable version go to the same URL as the last release (the one in #2 above)? That's what stable is meant to represent anyway, so it could just go straight there, as they'll be the same point in the repo.
I'm sorry if I repeated what you already know, it just sounds to me that we're discussing two different matters/problems.

The stable version at http://restic.readthedocs.io/en/stable has an edit button that goes to restic/restic:doc/index.rst@8425393 - this does NOT work, and from what I've gathered it is because there is no corresponding stable branch in the repository at GitHub.

This shouldn't be a problem. None of the repositories need to have a stable branch/tag; this is something created by RTD itself.
Take a look at this example that works as it's expected: http://requests-oauthlib.readthedocs.io/en/stable/. If you click in Edit on Github link, it just works.
I don't know why you are having that exact problem yet. We need replication for this in a local installation so it's possible to fix it. Feel free to open a new issue (since this is closed and about a similar problem) with these steps and examples you mentioned here.


Make 0D memrefs legal for FlattenMemRefSubspanPass.

@antiagainst I think I know the cause for the failures in this PR, but I think I will need your help on that.  Let me know when you have time to connect.

@antiagainst I will probably need your help here... any pointers?

You mean the failing test? Let me take a look there..

It's a bit weird. I don't see very obvious issues. I'd need to look it closer.

Looking into this again. I'm playing with some changes in https://github.com/openxla/iree/pull/12414.

Okay I think I fixed the failures in https://github.com/openxla/iree/pull/12414.


Document how to use Camel K without the "kamel" tool

Like: oc create -f cr.yaml

I think it is covered by https://camel.apache.org/camel-k/next/running/running.html#no-cli-integration


Fix a bug which druid's parsing formatted time field in a non-zero time zone

Fix a bug which druid's parsing formatted time field in a non-zero time zone. Issue #6343

Hi @zhaojiandong, this behavior is intentional: Druid always uses UTC, regardless of system time zone. This makes the parsing behavior consistent even if, for example, your Druid machines and Hadoop machines aren't running in the time zone. I think a better way to address #6343 is to add a "timeZone" feature to the TimestampSpec object. Then Druid could use that instead of UTC, if a user provides it. What do you think?

Hi @zhaojiandong, this behavior is intentional: Druid always uses UTC, regardless of system time zone. This makes the parsing behavior consistent even if, for example, your Druid machines and Hadoop machines aren't running in the time zone. I think a better way to address #6343 is to add a "timeZone" feature to the TimestampSpec object. Then Druid could use that instead of UTC, if a user provides it. What do you think?

Hi @gianm , it's a good idea to add "timeZone" feature.  I will pull another request about this feature.


port connection question

More of a question rather than an issue:
I will be migrating from jrcs/crashplan to jlesage/crashplan-pro, and I noticed that in the docker run command there is need to specify the connection ports (crashplan for home had 4242 and 4243).  How is this container (and for that matter jlesage/crashplan) connecting to the crashplan servers?  When I connect to the GUI, I get a green unlocked lock in the top left saying "Connected (unencrypted)".  I combed the service.log.0 file, but I couldn't see anything that indicated a successful connection, though.
I'd like to make sure that I have everything set up correctly before I do my one-way migration to crashplan for small business.
Running on FreeNAS 11.1 with Rancher VM; docker container(s) inside rancher. jrcs/crashplan works on this setup.

In CrashPlan Home, port 4242 was used for computer-to-computer backups.  This feature is no longer available in CrashPlan PRO.  Port 4243 is used internally by the UI to connect to the engine.  This is an internal port and doesn't need to be exposed.
The main difference with jrcs/crashplan is that this container has the CrashPlan app/UI integrated (i.e. it runs inside the container) and there is no need to run the app externally.
So if you connect to the UI and see the CrashPlan interface, you are ready to go!

Thank you for the quick response!
BTW, are there other users that have the same setup as me? (ie. FreeNAS + rancher)

I'm not aware, but I'm sure there are some ;)

Closing the issue.  Re-open if you have additional question related to this.


Provide specific import messaging when user has duplicate EM records

Refer to #6100 for context. This is dependent upon #6203. We need to implement more detailed messaging for the user to indicate what the duplicated record is, e.g., duplicate summary_value record, duplicate daily cal record for XX date and XX hour, etc.
Generic Message Format

Import contains duplicate [Table Name] data for [Unique Column Names] with values [Unique Column Values].

Emissions Tables with Unique Constraints
There is only one.

Table
Unique Columns

SUMMARY_VALUE
MON_LOC_ID, RPT_PERIOD_ID, PARAMETER_CD

Summary Value Message Format

Import contains duplicate SummaryValue data for Location '[unitId or stackPipeId]', Quarter [year] Q[quarter], and Parameter Code '[parameterCode]'.

### Tasks

Verified this issue is fixed.  Tested on importing the provided json file and retrieved the expected error message:


[Feat] Added unit test for Point class

PR summary
Closes #137
Added unit test for Point class. (Similar to other tests present in the file.)
Ran unit tests and they were successful - No errors.
PR checklist

[X] Links to the related issue (#....)
[X] Units tests have been run and/or modified and/or added
[ ] Docstrings are complete and documentation modified
[ ] Any new dependencies have been added to pyproject.toml and requirements.txt (in docs folder)
[ ] If new files have been added, make sure they aren't excluded by .gitignore
[ ] Any new data files have been added to manifest.in

@yalap13 Could you please add the "hacktoberfest-accepted" label to the PR? 🙂


Rename the org.carlspring.strongbox.storage.metadata.nuget.metadata as to not repeat itself

There seems to now be a packaged called org.carlspring.strongbox.storage.metadata.nuget.metadata. This is a bit too repetitive. It should just be org.carlspring.strongbox.storage.metadata.nuget.
This task is a continuation of #611 and #1106.

@carlspring there is a notion in nuget's nuspec called metadata - https://docs.microsoft.com/en-us/nuget/reference/nuspec#general-form-and-schema
That's why I kept the classes representing it under org.carlspring.strongbox.storage.metadata.nuget.metadata.
It's not the best package name I could think of but I thought it's better to have it than to put those 18 classes in the same namespace.
BTW I noticed strongbox-storage-nuget-layout-provider/src/test/java still has the org.carlspring.strongbox.storage.metadata.nuget.metadata namespace.


Feature request: Finally script

Would be good to have an option to specify a task that would run regardless of the success/failure of other tasks. Think of a clean-up task that would shutdown service or docker at the end of the task.
Ex: run-s -f stop:service start:service run:tests - starts service > runs tests > stops service

Doesn't the semicolon work for this on every OS?
run-s start:service run:tests; npm run stop:service

@TehShrike huh, didn't check - should it :)?

@TehShrike that worked 👍

Doesn't the semicolon work for this on every OS?
run-s start:service run:tests; npm run stop:service

This command returns the exit code from npm run stop:service. If you think try...finally you would expect the whole npm script to return the exit code from run-s start:service run:tests


The site http://worldtimeapi.org/ is not working

Hello, your site http://worldtimeapi.org has stopped working properly, I can access it 1 time out of 10, and yesterday I saw that the encrypted connection via https was gone. The problems started about a month ago, using a direct connection your site did not open, then I started using a proxy and got access to it without problems, but now I can not access it even using a proxy. Thank you for your attention, I hope you can fix this problem. Thank you for your project.

Hello,
I don't host this site, I can't help you with this issue !
It looks like they do not use my API (or a slightly modified version) : some fields are different.
More, my API uses HTTP, not HTTPS.


Upgrade Colore library to 6.0.0

This should fix the razer chroma support. Tested with razer deathadder V2 and razer firefly
Fixes #2284
Since I'm no professional C# dev I'm not sure what the best way to handle the async stuff is, for now I just used task.GetAwaiter().GetResult() which seemed to work fine

I'm not sure if this will cause deadlock issues because of async. I've fixed the razer support in dev without upgrading to colore v6, so i won't merge this unless another issue comes up. Thanks for contributing though!


Use a default baseline in gradle and add support for consolidated baseline files.

This PR is not yet ready but I would like to get your feedback. Eventually it is supposed to resolve #1260.

The baseline is created by default in $rootProject.projectDir/config/detekt/baseline.xml
If the file $rootProject.projectDir/config/detekt/baseline.xml exists, it is used as the baseline for the detekt task unless configured differently.
The CLI accepts an optional --source-set-id parameter in order to be able to use one baseline file for multiple subprojects (and in the future source sets within those).
The baseline file format is changed since there has to be a container for multiple baselines. Existing files in the current format can still be read.

The change missing is to actually use the sourceSetId in the gradle plugin. Could someone please tell me (or point me to some documentation) on how to use this changed cli from within the gradle subproject and how to make it work in CI?

This should not change anything for you if you just use the CLI. The resulting baseline files should be identical.

Hm, I do not think that we have to change the cli module for the gradle plugin.
The baseline feature was not intented to be complex and there should only be one baseline.xml file on top level.
I would propose that detektBaseline always runs for the whole project and generates a baseline file for each module in the build directory which are later combined to one baseline in the root module. The place of the last can be specified with the baseline property.
If the user has code with the same signature in two different modules its not our problem ^^.
What do you think?

So what you are saying is that detektBaseline will always run on all modules for all source sets and after that is complete, merge all the generated files into one baseline? That is a valid approach in my opinion. It could get a little awkward if the user defines different baseline files in different submodules. How would that be interpreted?
/build.gradle
detekt {
  baseline = file("config/detekt/baseline.xml")
}

/foo/build.gradle
detekt {
  baseline = file("$rootProject.projectDir/config/detekt/baseline.xml")
}

/bar/build.gradle
detekt {
  baseline = file("config/detekt-baseline.xml")
}

Since both detekt and detektBaseline use the baseline property to me it is not really clear how we would handle this. On the other hand I have no idea if there is even a use case for that or we can just consider it misconfiguration.

Hm, yeah that is indeed bit strange with the detekt task.
But yeah, we just consider it misconfiguration as it was never planned to work like this :X.
Managing many baseline files is imo not feasible and we shouldn't embrace that behaviour :) and instead make clear to always use one location via rootProject....

4 years later. Would it be okay to consider this now? Many people use baseline per module through gradle.


Reg: Help needed for filtering downloads

Hello,
Will you please give me a sample to download only HomeDomain folder and i need a Photos also..
Your help most appreciatable.

Hi. Paths are constructed as: domain / relative-path
The tail characters of the relative-path are considered to be the file extension.
If we take the file: CameraRollDomain/Media/DCIM/101APPLE/IMG_1000.JPG
domain =  CameraRollDomain
relative-path = Media/DCIM/101APPLE/IMG_1000.JPG
extension = JPG, .JPG, 0.JPG etc.
Just apply the case-insensitive filters as required.
To download all files from HomeDomain:
java -jar InflatableDonkey.jar AUTH --domain homedomain

To download all photos from all domains:
java -jar InflatableDonkey.jar AUTH --extension .jpg .jpeg

or
java -jar InflatableDonkey.jar AUTH --item-type PHOTOS

Further example, to download all photos ending with 0.jpg from the camera roll after 31st Jan 2016
java -jar InflatableDonkey.jar AUTH --domain camerarolldomain --extension 0.jpg --date-min 2016.1.31

AUTH is either your AppledID and password or dsPrsID:mmeAuthToken.

Thanks for the reply,
You have a done fab job . I have some Images in cloud Photos but i didn't getting any pics . Please help me

There are hacked versions of InflatableDonkey that download iCloud iOS9+ photos but I won't be releasing them at this time. Sorry.

Hello,
I'm getting while integrating with Spring webmvc project all the classes are there the following error occurring while performing download operation. It will not work with Spring?
java.lang.NoClassDefFoundError: Could not initialize class com.icloudifd.horrorho.inflatabledonkey.protobuf.CloudKit

I've not played around with Spring so I can't offer any assistance. Sorry.

java -jar InflatableDonkey.jar someId;sometojen --domain homedomain --extension callhistorydb
is it correct to retrieve only call history?

@rameshT108 CallHistoryDB is a folder, not an extension.
iOS9
java -jar InflatableDonkey.jar AUTH --domain homedomain --relative-path callhistorydb

or
java -jar InflatableDonkey.jar AUTH --item-type CALL_HISTORY

iOS10 see https://github.com/horrorho/InflatableDonkey/issues/37

Hello,
Can we filter two different domains with relative path also like
java -jar InflatableDonkey.jar AUTH --domain whatsapp homedomain --relative-path sms

Hi. Not in the way I think you would like. The filters tag onto each other. They are pretty much in the style of iLoot with additional options.
java -jar InflatableDonkey.jar AUTH --domain whatsapp homedomain --relative-path sms

With this, the domain needs to be either contain the string whatsapp or homedomain AND the relative path must contain sms. Unless your WhatsApp domain contains an sms folder it will not download anything from WhatsApp.
If you let me know the files that you would like to retrieve from WhatsApp and SMS, I might be able to assist further in crafting a query.

Hello Thanks for the reply,
Actually, i want some folders from home domain for example SMS/ Address book and I want other applications folders also example facebook / WhatsApp. then how can I filter these? or else I need to run process separately for two of them?

No further discussion, closing issue.


Update Defaults

I setup a new machine earlier today and didn't have ruby 2.2 available. I tracked it down to a few of the boxen/puppet-* releases being behind.

Thanks


Feature: Avatars should be clickable links into profiles

Type of feature
Feature
Current behavior
These avatars currently are not clickable and provide no context.

Desired solution
Similar to the title link, these be click github.com links to the user github profile.

Additional context
This should be a dynamic and reusable solution.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Let me work on it

Thanks for taking this on @frankiefab100  Please review the CONTRIBUTING.md for info on how to assign yourself to the issue

@jasonericdavis please where can I locate this issue?

I am unsure what you are asking

I mean I can navigate to the avatar directory

There is not a directory, the API pulls in the login (i.e. bdougie). You can leverage the login to get the avatar url

@frankiefab100 There isn't explicitly an avatar directory. This issue is to add functionality to the avatars on the cards in the list.  The avatars can be found in the following files:

src/components/PostGrid.jsx
src/components/PostList.jsx

Issue #28 also gives some context of where and how the avatars are being displayed for each card.
It is important that you read the CONTRIBUTING.md to learn how to assign yourself to the issue. Assigning yourself to the issue lets others know that somebody is actively working on the issue.

@frankiefab100 do you want to take on this issue?

Not yet

If @frankiefab100 isn't available, may be I should take this?

.take


List all Modules used in a pipeline

Is your feature request related to a problem? Please describe.
As a developer and now that DSL2 is default I would love to see all the modules used in a pipeline along with parameter and usage docs on the website
Describe the solution you'd like
A separate tab for each pipeline -> Modules that lists the nf-core and local?  modules used in that pipelines
Additional context
Related Slack Discussion

i believe this functionality has been implemented right so, can close this @mashehu ?


Add a scale transition on the cards make it look nice 🔥🔥 on hover

Make a scaling transition on the cards on hover

Added hover effect that scale and move the card up to make it look nice


Datasource external

The datasource_external plugin references the WeatherMapDataSource class wich cannot be found. Did you maybe forget this one while refactoring?
I can't find the corresponding class, sadly, so i would be grateful for a fix.
Cheers
2018/12/11 15:07:34 - ERROR PHP ERROR in Plugin 'weathermap': Class 'WeatherMapDataSource' not found in file: /usr/local/share/cacti/plugins/weathermap/lib/Weathermap/Plugins/Datasources/WeatherMapDataSource_external.php on line: 23

Oops, I guess I did, because it's not normally a .php file, so phpstorm wouldn't have indexed it.
The new base class is just called 'Base'. If you look at the top of any of the other DS plugins, you'll see how it's supposed to look. Otherwise I'll push a change later.

If fixed it's can be closed


Perzonalised Email Recommendations

Implement weekly personalized movie recommendations via email opt-in on the registration page.

Email recommendation support has been added, allowing users to receive personalized weekly movie recommendations. An email input field on the registration page now prompts users to opt-in, and their preferences are stored in the database to enable tailored recommendations, enhancing engagement with weekly suggestions delivered directly to their inbox.
Here's the commit id : 01234e0b0d3dba4e79d907b248520ba93dee464d