The right way to debug requests with https proxy

Summary.
I have a proxy service that has been used for months. Everything works well until Twitter. The code here is a proxy test script between google and twitter. Of course Https.
For "https://www.google.com" and "https://mobile.twitter.com", the script both run 180 times. Most of google succeeded, however, all of twitter failed.
google:success times is 171, failed is 9, and total is 180
twitter:success times is 0, failed is 180, and total is 180
Half is "ConnectTimeoutError", and left is "ProxyError"
Expected Result
The success rate should keep the same
Actual Result
HTTPSConnectionPool(host='mobile.twitter.com', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<requests.packages.urllib3.connection.VerifiedHTTPSConnection object at 0x107446150>, 'Connection to proxyip timed out. (connect timeout=10)'))
HTTPSConnectionPool(host='mobile.twitter.com', port=443): Max retries exceeded with url: / (Caused by ProxyError('Cannot connect to proxy.', error(54, 'Connection reset by peer')))

Without "Upgrade-Insecure-Requests", the errors keep the same.
So I'm curious about is there any method to debug these errors?
Reproduction Steps
from gevent import monkey, spawn, joinall, sleep
monkey.patch_all()  # Magic!

import requests
import sys

s = requests.Session()
proxys = "https://username:password@ip:port"
class test_proxy:
    success = 0
    failed = 0
    total = 0

    def tweet(self):
        #url = "https://www.google.com"
        url = "https://mobile.twitter.com"
        header = {
            "Referer": url,
            "Upgrade-Insecure-Requests": 1,
            "User-Agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 Safari/537.36",
        }
        try:
            a = s.get(url, proxies={"https": proxys}, timeout=10)
            self.success += 1
        except Exception as e:
            print(e)
            self.failed += 1
        self.total += 1
        print("sucess times is {}, failed is {}, and total is {}".format(
            self.success, self.failed, self.total))

test = test_proxy()

while True:
    joinall([spawn(test.tweet) for _ in range(50)])

System Information
python2.7

Is the proxy local, or is it provided by a third-party web service? It's quite possible that either the proxy or Twitter are refusing to co-operate.

Third-party. For another group of proxy,  test results looks a little different , but not such significant. So I don't think it's the problem of proxys, it's the usage of requests or twitter. Should I use cert or something else to improve it efficiently?
Google: sucess times is 114, failed is 231, and total is 345
Twitter:  sucess times is 12, failed is 343, and total is 355
another_proxy = ['https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:8799',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:80',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:80', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:8081',
               'https://<IP_ADDRESS>:53281', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:8080',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:53281', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:53281', 'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:8080',
               'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:8080',
               'https://<IP_ADDRESS>:80', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:5033',
               'https://<IP_ADDRESS>:1189', 'https://<IP_ADDRESS>:53281', 'https://<IP_ADDRESS>:1189',
               'https://<IP_ADDRESS>:8080', 'https://<IP_ADDRESS>:8080']

Hrm, very odd. However, what's likely to be relevant is that your proxies are all using https schemed URLs. Can you try setting your proxy url to have a http scheme and see if that improves things? Requests does not really formally support using TLS to connect to a proxy.

so weird, why would  {"http":"http://ip:port"} work for https url .... ?  I will have more test later....

It would not, but {'https': 'http://ip:port'} would, which is what I want you to try. =)

After a few more tests, {'https': 'http://ip:port'} doesn't work too.  I
thought {"http":"http://ip:port"}  would work, but I figure it out now, the
requests were sent to target directly .  so sad ......
On Mon, Jun 26, 2017 at 7:17 PM Cory Benfield<EMAIL_ADDRESS>wrote:

It would not, but {'https': 'http://ip:port'} would, which is what I want
you to try. =)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/requests/requests/issues/4176#issuecomment-311031219,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6kD-jfhvyPizp1syOefrVz932lzEZ4ks5sH5NYgaJpZM4OE-bQ
.

--
Tony Wang
Warm Regards

So this sounds enormously like there is a problem between your proxy and Twitter.

Maybe.... I will submit a ticket to the proxy service. Thanks for your kind
of help
On Tue, Jun 27, 2017 at 1:51 AM Cory Benfield<EMAIL_ADDRESS>wrote:

So this sounds enormously like there is a problem between your proxy and
Twitter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/requests/requests/issues/4176#issuecomment-311132648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6kDybAcWa82n4kv3Hmfv5ImdsadSWuks5sH--6gaJpZM4OE-bQ
.

--
Tony Wang
Warm Regards


Windows 10 VM problem

Hi,
I used vgpu_unlock to successfully virtualize the 2080s, and also successfully turned on the windows 10 VM with vGPU and passed the license verification. But since my VM added the mdev of vGPU, it feels a bit lagging, and it feels like there is something wrong. I have tried B-series and Q-series, and disables frame rate limiting.
Have you encountered a similar problem?
Can you also share the full XML file of your VM?
How do you connect to the VM with vGPU? I use RDP and TightVNC.

I found the problem from the log that appear "NVOS status 0x65", "VGPU message 22 failed", " XID 43 detected on physical_chid:0x20" and "Timeout detection and recovery (TDR) completed".
It seems to be caused by the version of vGPU manager. I used the lastest version 12.2 before.
Since I reinstalled the old version of vGPU Manager, everything is ok.


Add no-op mercury2.0 upgrade handler

While not necessary for the upgrade to process correctly, this
mercury2.0 upgrade handler will show to validators that the upgrade has
been completed successfully, and solve the issue of performing the
upgrade with --unsafe-skip-upgrades when no handler is registered.

got this working locally, need to fill out upgrade info and then merge.


Add support for plop

Add plop to make it easier to add new methods.

Codecov Report

Merging #40 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #40   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          35     35
  Lines         245    245
  Branches       27     27
=====================================
  Hits          245    245

Impacted Files
Coverage Δ

src/toArray.ts
100% <100%> (ø)
:arrow_up:

src/ICW.ts
100% <100%> (ø)
:arrow_up:

src/index.ts
100% <100%> (ø)
:arrow_up:

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 ddbdeaf...eb11d26. Read the comment docs.


Different workspace behaviour on agents and with drone exec

Not sure if this issue should be here, or rather be discussed in the cli repository - please let me know if I should move it.
I've noticed, that when running drone exec there will be no temporary volume created for workspace.
This leads to different behaviour on the agent and with drone exec:
On the agent:

the folder /workspace will stay persistent between builds

Locally:

/workspace is not persistent between steps

Scenario where this is relevant:
I have a pipeline, where I test modules for an application. That application expects these modules to be place in a specific directory (/application/modules/<mymodule>)  otherwise they can't be tested within the application framework.
With the following definition for workspace+path I am able to accomplish this when running my builds on an agent:
workspace:
  base: /workspace
  path: application/modules/mymodule

pipeline:
  prepare:
    image: ...
    commands:
      - cp -r /application /workspace/application
 test:
   image: ...
   commands:
     - run_tests
...

The contents created at /workspace/application will persist between further drone steps (when run on an agent). However locally, this is not the case.
When comparing both execution flows, I realized that drone exec only mounts the current folder to workspace+path, but does not create a volume for /workspace
When using drone exec --volumes "workspace:/workspace" the behaviour would be the same as when run on an agent.
Simple yaml to reproduce the effect:
workspace:
  base: /workspace
  path: application/modules/mymodule

pipeline:
  step1:
    image: alpine
    commands:
      - touch /workspace/hello
      - ls /workspace
  step2:
    image: alpine
    commands:
       - ls /workspace

Proposal:
Also create a workspace volume when running locally

moved to https://github.com/drone/drone-cli/issues/47


Creating a copy of a round results in 500 error

Expected behavior, hit copy of a round, input all the needed details to make a copy
Instead, get 500 error.
The reason is the copy function doesn't give the user the option to change the dates, thus error for overlapping round dates.

This is the same as #1018

Closing as duplicate.


cygwin / tar vs 7zip

I was having trouble running the meteor projects from meteor kitchen.  While this isn't likely to be meteor kitchen specific, it was affecting the perak:user-roles package.
I found that tar.exe was in my path (via a cygwin install).  Once I removed this from my path ... my stuff now works.  I think it found 7zip instead.
Just heads up - as I said this issue is resolved for me, but for windows users with the cygwin shell installed, tar.exe seems to work for most packages - but not all.  So if you getting an error like file not found in your appdata local temp directory - it's probably because the files are failing to unzip for whatever reason.  If you have tar or an old version of 7zip, this can very likely be the problem.

this can be closed - this was for information purposes for windows developers.  Meteor Kitchen is a wonderful project - and I thank the contributors for their hard and productive work.

@HuskerSam hum... I am thinking on how tar.exe affects "meteor user roles" package. What errors did you have when running (or generating?) the app? Maybe that affects meteor's automatic package install (but in that case, why only user roles...?

While I didn't run it down more, I think having a path that starts with '/dir/dir2' - some unzippers translate to into c:\ on windows, some don't.  What I did was wipe everything I had, then the meteor update/install process would run, and it couldn't find files in my appdata/.../... folder related to the userroles package.  It probably effects more packages, but since I"m using meteor kitchen for this project it's the first one to cause a fail.  The files simply weren't there - but the system had just tried to put them there.  I found other threads about old copies of 7zip and also tar.  So if node at a low level uses tar first to unzip, then reverts to 7zip, this would seem to be the underlying cause.  Removing cygwin (and tar) from the path caused the decompression process to work.  Now that's working I haven't investigated more, just wanted to give a heads up.  I think a similar issue is effecting the 1.4.1 meteor tools download/unzip.  While I'm iffy on if it's the unix root path ('/') conversion to c:\ (which is a good guess now days, but not always in the past), this would explain my issues.  Also unix compatible tar.exe wouldn't have a clue about windows c:.
I hope that helps, this was posted to help others that had failing meteor installs.  thanks

Ah... I understand. Good catch, thank you!


steps to set up ri-api with own data

Hello,
I have followed the beaconv2-ri-tools documentation to obtain the 7 json files making up the BFF format. Loading data into the beaconv2-ri-api worked fine. I can see the data in the mongo express ui. However, all my api requests return no result. I have modified the datasets.yml and the cohorts.yml so my new sample id appear to be looked up by g_variants.py script. Could you give me hints on what I am missing in order to make my data searchable by the ri-api?
Here is the return of a post request:
$ curl   -H 'Content-Type: application/json'   -X POST   -d '{
    "meta": {
        "apiVersion": "2.0"
    },
    "query": {
        "requestParameters": {
    "alternateBases": "G" ,
    "referenceBases": "A" ,
"start": [ 0 ],
            "end": [<PHONE_NUMBER> ],
            "variantType": "SNP"
        },
        "filters": [],
        "includeResultsetResponses": "HIT",
        "pagination": {
            "skip": 0,
            "limit": 1
        },
        "testMode": false,
        "requestedGranularity": "record"
    }
}
'   http://localhost:5050/api/g_variants ; echo
{"meta":{"beaconId":"org.ega-archive.ga4gh-approval-beacon-test","apiVersion":"v2.0.0","returnedGranularity":"record","receivedRequestSummary":{"apiVersion":"2.0","requestedSchemas":[],"filters":[],"requestParameters":{"alternateBases":"G","referenceBases":"A","start":[0],"end":[1600000000],"variantType":"SNP"},"includeResultsetResponses":"HIT","pagination":{"skip":0,"limit":1},"requestedGranularity":"record","testMode":false},"returnedSchemas":[{"entityType":"genomicVariation","schema":"beacon-g_variant-v2.0.0"}]},"responseSummary":{"exists":false},"response":{"resultSets":[]},"beaconHandovers":[{"handoverType":{"id":"CUSTOM:000001","label":"Project description"},"note":"Project description","url":"https://www.nist.gov/programs-projects/genome-bottle"}]}

The docker logs:
beacon             | [beacon.request.handlers][DEBUG ] (L54) meta=RequestMeta(requested_schemas=[], api_version='2.0') query=RequestQuery(filters=[], include_resultset_responses=<IncludeResultsetResponses.HIT: 'HIT'>, pagination=Pagination(skip=0, limit=1), request_parameters={'alternateBases': 'G', 'referenceBases': 'A', 'start': [0], 'end': [1600000000], 'variantType': 'SNP'}, test_mode=False, requested_granularity=<Granularity.RECORD: 'record'>)
beacon-permissions | public
beacon-permissions | visa_datasets: []
beacon-permissions | ['CINECA_dataset', 'rd-connect_dataset', 'CINECA_synthetic_cohort_EUROPE_UK1', 'AV_Dataset']
beacon             | [beacon.request.handlers][ INFO ] (L79) resolved datasets:  ['CINECA_dataset', 'rd-connect_dataset', 'CINECA_synthetic_cohort_EUROPE_UK1', 'AV_Dataset']
beacon             | [beacon.db.utils][DEBUG ] (L77) FINAL QUERY: {}
beacon             | [beacon.db.g_variants][ INFO ] (L129) GET_VARIANTS YOUYOU
beacon             | [beacon.db.utils][DEBUG ] (L77) FINAL QUERY: {'$and': [{'variation.alternateBases': {'$eq': 'G'}}, {'variation.referenceBases': {'$eq': 'A'}}, {'variation.location.interval.start.value': {'$gte': 0}}, {'variation.location.interval.end.value': {'$lte':<PHONE_NUMBER>}}, {'variation.variantType': {'$eq': 'SNP'}}], '$or': [{'caseLevelData.biosampleId': '“MySample1”'}, {'caseLevelData.biosampleId': '”MySample2”'}, {'caseLevelData.biosampleId': '”MySample3”'}, {'caseLevelData.biosampleId': '”MySample4”'}, {'caseLevelData.biosampleId': '”MySample5”'}]}
beacon             | [beacon.response.build_response][DEBUG ] (L67) 0
beacon             | [beacon.utils.stream][DEBUG ] (L25) HTTP response stream
beacon             | [beacon.utils.stream][DEBUG ] (L30) Partial content: False
beacon             | [aiohttp.access][ INFO ] (L206) <IP_ADDRESS> [25/Mar/2024:15:59:28 +0000] "POST /api/g_variants HTTP/1.1" 200 985 "-" "curl/7.81.0"`

Thanks for any tips!
Alex

Hi @albodrug,
thank you for trying RI. First of all, I see you don't have the last version of RI, and although it shouldn't be the cause of your problems, you can update it if you wish to have the last "working version".
Secondly, I see something strange in the final query you just threw, which are the biosampleId:
{'caseLevelData.biosampleId': '“MySample1”'}, {'caseLevelData.biosampleId': '”MySample2”'}, {'caseLevelData.biosampleId': '”MySample3”'}, {'caseLevelData.biosampleId': '”MySample4”'}, {'caseLevelData.biosampleId': '”MySample5”'}

As you say, you have to add the biosampleIds and individualIds to cohorts.yml and datasets.yml, and right now beacon is reading your ids like ”MySample4”, are you also adding them like this? The string has to be the same!
Also, if you can make beacon read the biosampleId like a plain MySample4 it would be better for your beacon comprehension.
Tell me if this was the cause of your beacon not displaying any results.
Thanks,
Oriol

Hi Oriol,
Many thanks! I have updated the ri. And the quotes were indeed the issue.
My raw files and the biosamples.json do no carry the quotes. The issue was resolved when I modified the .yml files deleteting the quotes from the biosampleId, but not from the individualId.
My individualId and biosampleId are different (contrary to the CINECA data), do you think it may be related?
Also in both cohorts.yml and datasets.yml files I have added both individualId and biosampleId.

Great, I'm glad this solved your problem.
Indeed, it is related. For some collection, beacon RI will read the biosampleId and for others the individualId. If you don't add them all properly in cohorts.yml and datasets.yml the RI will not be able to relate the samples correctly to the dataset they belong.


Add timetagger

Motivation for this change
Want to try this app
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

The ./result is empty after a successful build, and I don't understand why (I am not a python guy) ...
Maybe some nixpkgs+python guru can tell me?
I also want to write a service module as soon as I can test this!

@FRidh and @jonringer I am ready for review here - if you can teach me how to add a test for the service definition, I'd write one :laughing:

@aanderse thanks for your suggestions. Please have a look at https://github.com/NixOS/nixpkgs/pull/151364/commits/1f7534201bf05868c270f1ccccfab45ade4f83d3 where I applied the change with the package default value.
I differed from your suggestion a little bit, most notably the defaultText option, where I used interpolation of cfg.bindAddr and cfg.addr, which I guess is more understandable if the user sees it in the nixos option search (in contrast to the strings bindAddr and port). What do you think about this part?

If you approve I will rebase and squash my fixup (the minor changes), but leave the commit mentioned above as standalone for better traceability.

About the handling for port and bind addr: Maybe these should be options for the run.py script upstream provides. I think this cannot  (should not) be fixed in nixpkgs but upstream - With options (optionally with default values to not break backwards compatibility) for the run.py script.

Looks good 👍 Yeah, I was a bit shocked that this software doesn't have a way to set these values from either the cli or a config file. Unusual.

Me too. Either way, squashed and pushed. Ready to merge from my POV. :smile:

Tests!!! :partying_face: :tada:
With help from @0x4A6F :heart:

[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

As a side note: We have listenAddress/(listen.)address and a variation of listen, bind, ip and addr occurrences in modules instead of bindAddr.

Module looks fine to me :+1:

Thanks!

Module looks fine to me +1

Module is not included in nixos/modules/module-list.nix and does need a test.
@matthiasbeyer Do you have an example running this service?

No, I just tested it once without even using a service definition IIRC.


Remove swears

Is your feature request related to a problem? Please describe.
Firstly, I love this variant and have been playing it daily. It's extremely original and an excellent implementation. However, I'd like to recommend it to my kids, and the gratuitous swearing makes me uncomfortable.
Describe the solution you'd like
I'd wonder if you'd mind removing the swearing.
Describe alternatives you've considered
Erm... not recommending it, I guess?
Additional context
I don't really think it adds much to the game anyway. I don't think removing it would make the game worse.

Thank you!

No prob. After closing the issue I noticed deployment had initially failed, but the change should be live now. Thanks for the kind words, hope the kiddos enjoy the game!


[WIP] xorg: allow overriding via overrideScope

Motivation for this change
Wanted to modify some xorg.* packages in an overlay, ran into https://github.com/NixOS/nixpkgs/issues/11944. I did a fairly straight forward conversion into makeScope / overrideScope, but I don't have a good idea of the state of the art in this area.
Additionally discovered that while pkgs/servers/x11/xorg/default.nix is auto-generated, it has been manually updated for a few packages. I updated the source lists so that regenerating default.nix doesn't change any packages. The exception to this is xeyes, which gained a new buildInput. The source lists should be converted to https, but I'd prefer to do that in a separate PR.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Why did the old version import some dependencies for xorg via inherit (buildPlatform) ...;? Does this break cross?

So first of all, I think you meant buildPackages. And to answer your question, yes, it is needed because callPackagesWith pkgs is being used directly. See the unfortunate "splicing' stuff that goes into normal callPcakges. callPackagesWith pkgs doesn't get that, and so is missing packages.
Now that splicePackages is exposed, however, you can let-bind a callPackageWithoutXorg which will obviate the need to sort specific packages by hand.

Overall, great work! It would be nice to get this into 18.09 as it's a good cleanup and will make any xorg backports easier if we do so.

Rebased to resolve the conflict, fixed callPackagesWith vs callPackageWith.
It sounds like you've got a clear idea of how to make this nicer, but I'm not familiar enough with the splicing to grasp it. Can you expand a bit on how implementing callPackageWithoutXorg would look?
As for usinglib.fixedPoints.extends, would that require changing default.nix and overrides.nix to take pkgs explicitly instead of being called with callPackage?
Thanks for looking at this!

I'm going to make a few fixes here, then merge soon.

I'm very confused by this. I don't see any merge conflicts when doing a merge locally. I'm guessing whatever is causing the merge conflicts is also causing the mass rebuild tags. Assuming ofborg builds the GitHub automatic merge, then it will be building this old version:
$ git fetch origin pull/45328/merge && git log -1 FETCH_HEAD
commit c03453c1270eb1405b6d5687b74e284c2c185834
Merge: f799dff21bd 1b748ec40ea
Author: Andrew Childs<EMAIL_ADDRESS>Date:   Sun Sep 23 12:10:31 2018 +0000

    Merge 1b748ec40ea89b344dc20fae33fda3f3dc28897c into f799dff21bd70738a0724419752dbb8d9ed3ce6b

For reference:

current master: 9179c1449db7b868504551cff9972acf6c1ce4f9
current PR: 0e88495e3eb8a602b58c50331995e1af0b96ef4f
current PR without dummy commit: b663eb04e5146bc7e64fa9613a3706393e9ae3e2

@dtzWill Sorry just saw. Sure that makes sense, will do!


Link national holidays common services with Rasa intents

[!NOTE]
This is a sub-issue of #59

[!IMPORTANT]
The scope of this issue is "functionality first"
This is a temporary solution based on https://github.com/buerokratt/Buerokratt-Chatbot/issues/865
Will be refactored based on https://github.com/buerokratt/Training-Module/issues/393

AS A Product Owner
I WANT national holidays common services to be linked with Rasa intents
SO THAT if the End Client asks any related question, applicable service would be initiated
Acceptance Criteria

[ ] #64
[ ] Update message-to-bot.yml so that it would link applicable Rasa intents and services
[ ] Applicable services are provided to End Clients via Chat Widget based on #61 request examples

https://github.com/buerokratt/Common-Services/issues/64


nixos/console: allow console.font to be a path

As for console.keyMap, all uses of this option are compatible with paths. This allows doing things like console.font = pkgs.runCommand ....
cc @rnhmjoj

You should be targeting the master branch, nixos-unstable is one of the channel branches, updated automatically.

Oops. Done.

It's worth mentioning in the description that both a file path or the font name are allowed.

Done.


Exclude dashboard-nvim

I was trying out LazyVim, which uses dashboard-vim and reported a bunch of whitespace:

Adding this entry fixed things (in the meantime, I'm using this to extend the excludes locally).

Thanks for the contribution! I'm AFK right now, so can I just request you also update the README to match the updated defaults before I merge?


Line break in the middle of a string value

I am running into situations where a hard line break is inserted part way through a string value in an html file which results in a string of spaces being appended to the first part of the string if I get its value using javascript.
For example:
{{ExchangeFromDateMDY}}
- {{ExchangeToDateMDY}}

It may not be obvious because of the github formatting but there is a line break after "{{ExchangeFromDateMDY}}"
I should have mentioned a couple of things.  First, the  tag above was all on one line before the file was saved and second, this does not happen if I disable the File Save plugin

@lcsqlpete does it happen when you disable save-actions and only reformat the code?

Good point. Reformatting did produce the same problem so I guess this is a PHPStorm problem not the plugin.


Some SVG images have unnecessary executable permissions

The current master branch (as at commit 9fbbcfa01a128faf7df91d327bddb2841dd0a554) has lots of SVG files in spyder/images/light and spyder/images/dark with permissions 755 rather than 644.  Fixing this is easy: chmod a-x spyder/images/light/*.svg spyder/images/dark/*.svg and then committing the changes.  I'm happy to make a pull request for this if you would like.

I'm happy to make a pull request for this if you would like.

Yes! Thanks a lot for your help with that @juliangilbey!
Please take into account to do your PR against our 5.x branch and not master, so we can include it in one of our future versions in the Spyder 5 series.


routing my profile > make backup

From @kikipluche on October 17, 2017 14:11

[ ] Actual behaviour:
My profile > Make backup > show my keys > make backup > email my file > enter swarm city on email success
Expected behaviour:
My profile > Make backup > show my keys > make backup > email my file > back to profile on email success

[ ] Actual behaviour:
My profile > Make backup > show my keys > make backup > download my file > enter swarm city on download success
Expected behaviour:
My profile > Make backup > show my keys > make backup > download my file > back to profile button on download succes

Copied from original issue: swarmcity/sc-boardwalk-production#291

#134


Section <IP_ADDRESS> and <IP_ADDRESS>, FCS_(D)TLSC_EXT.1.4, Tests

Section <IP_ADDRESS>, FCS_DTLSC_EXT.1.4, Tests
Section <IP_ADDRESS>, FCS_TLSC_EXT.1.4, Tests
Test 3 and 4: Selecting either secp groups or ffdhe curves requires that “present the Supported Groups Extension” is selected
Suggested change:
Add “ “present the Supported Groups Extension” is selected and ” after “If” at the beginning

Suggest updating Test 3 to:
if "present the Supported Groups Extension" is claimed with any selections of "secp256r1", "secp384r1", or "secp521r1",  the evaluator shall…
Suggest updating Test 4 to:
if "present the Supported Groups Extension" is claimed with any selections of "ffdhe2048", "ffdhe3072", "ffdhe4096", "ffdhe6144", or "ffdhe8192", the evaluator shall...

Agreed to no change on today's TLSWG call.


Get public url for an item uploaded using authenticated read

I am uploading photos using AvatarUploader which has configured self.aws_acl = 'authenticated-read'
As a result, all the urls generated have expiry tokens.
However, for a specific case, I need to get the public url (this should not expire) for an object.
However calling object.image.url always gives links that will expire.
Is there a way to get a public url even though the aws_acl is set as authenticated_read. Can we override this on a per method basis or something?

You can override the aws_acl on an uploader class basis, but not for a specific record or uploader instance. If there is a particular S3 object that you need to generate a link to and you're willing to special case it in your code you can use the public_url method directly: avatar.file.public_url.
That will generate the URL, but it won't guarantee that the ACL is actually public. Hope that helps!

Thanks @sorentwo !


inference sample

Thanks for your work.
I wonder that if you can provide an inference sample? For example, what is the input of 'cond_cate_to_size_pos_seq_modeling' (in generate.py) and what is the expected output of the model. I try to train and infer by the readme, but the token 'FILL_XXX' are not filled in the inference results.
Thanks again for your reply.

I am also seeing the same issue.


Evaluate/investigate use of exit(0) in shutdown logic

Our shutdown today will call exit(0) if managed fails to exit/return. We should investigate if this is the right thing to do, and if so, add E2E tests for these shutdown scenarios.

We decided that calling exit(0) to recycle IISExpress process is acceptable, we should avoid calling it in HWC scenarios though.


Unions containg never can now be constructed via DAA, which violates assumption that never expressions are unreachable

union U {
   a: bool,
   b: !,
}
proc main() {
   let x: Union;
   x.a = 10;
   x.b;
   // Roland assumes anything hereafter is unreachable, but it will be reached
}

We have this same problem with:
proc main() {
   let a = 10;
   let x = gen$<!>(&a);
}

proc gen<T>(x: &u8) -> &T {
   return x transmute &T;
}

I think we need to simply legislate this: an expression producing a type of never will trap

Closed in 8bb875a817abe4817ab5837b74df53171a18c390


Please add correct value to "Applies to:"

When you need to work with "older" framework versions it would be really helpful to see the version some feature was added.
For ImageSource it shows "Latest" but it is in Xamarin.Forms at least from version 2.2 on the other side - FontImageSource (https://docs.microsoft.com/en-us/dotnet/api/xamarin.forms.fontimagesource?view=xamarin-forms) also says "Latest" and there it is (at the moment) true with Xamarin.Froms 3.6.
If there is a way to edit the "Applies to:" please show it.

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

ID: 01339d1b-f817-f10c-0b43-4d6816c5eadc
Version Independent ID: 094cd696-88c5-7cf4-3f35-a7997fa82b35
Content: ImageSource Class (Xamarin.Forms)
Content Source: docs/Xamarin.Forms/ImageSource.xml
Product: xamarin
Technology: xamarin-forms
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Even though this is older, closing this as a duplicate of #174


Added typescript module definition file

I have added file colorsys.d.ts containing all basic Typescript declarations for colorsys functions and some interfaces describing color system objects.
This PR is useful for Typescript projects.
I didn't modify original code, just added types field to package.json.

Thanks for doing this!


feat: move compilation out of the blockifier

Closes #112

Codecov Report
Attention: Patch coverage is 85.08772% with 17 lines in your changes missing coverage. Please review.

Project coverage is 80.72%. Comparing base (c822f62) to head (51003ac).

Files
Patch %
Lines

crates/blockifier/src/execution/contract_class.rs
88.04%
5 Missing and 6 partials :warning:

crates/blockifier/src/test_utils/struct_impls.rs
70.58%
0 Missing and 5 partials :warning:

...fier/src/execution/native/entry_point_execution.rs
66.66%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@               Coverage Diff               @@
##           native2.6.4     #122      +/-   ##
===============================================
- Coverage        80.73%   80.72%   -0.02%
===============================================
  Files               77       77
  Lines            10552    10592      +40
  Branches         10552    10592      +40
===============================================
+ Hits              8519     8550      +31
  Misses            1569     1569
- Partials           464      473       +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Juno branch: https://github.com/NethermindEth/juno/pull/2009


remove deprecated render-mode prerendered

as per #87 the render-mode naming has changed from prerendered to unrendered
prerendered is now deprecated, so should be removed.

I think it is safe now to proceed with this.


Error in text mode

Similar to #56
Using: docker, composer: 0.6.2, composer: dev-master
Argument 1 passed to Mihaeu\PhpDependencies\Analyser\DependencyInspectionVisitor::addName() must be an instance of PhpParser\Node\Name, instance of PhpParser\Node\Expr\ArrayDimFetch given, called in /root/.composer/vendor/dephpend/dephpend/src/Analyser/DependencyInspectionVisitor.php on line 90

@mihaeu because of
xx::yy($validator['name'], $validator['class'] . '@' . $validator['function'], $validator['class']::DESCRIPTION);

@mihaeu :ping_pong:

Riiight terribly sorry @williamdes
I've been hibernating :wink: I'll try to fix it on Sunday. Does that still work for you?

I've been hibernating :wink:

:bear: :laughing:

Does that still work for you?

Yes :heavy_check_mark:


fix data_loss_protection itest flake

This flake seems to pop up on Travis semi-regularly. Often this sort of thing indicates that the test isn't
making proper use of WaitPredicates, but more investigation may be necessary. This log was taken from
the btcd itest.
=== RUN   TestLightningNetworkDaemon/data_loss_protection
Error: failed to get profile page (node_id=115, name=Dave): Get "http://localhost:20019/debug/pprof/goroutine?debug=1": dial tcp <IP_ADDRESS>:20019: connect: connection refused
    lnd_test.go:112: Failed: (data loss protection): exited with error:
        *errors.errorString expected to find 0 channel pending force close, found 1
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:9404 (0xe15723)
        	assertNumPendingChannels: t.Fatalf("%v", predErr)
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:9802 (0xe16d9c)
        	testDataLossProtection: assertNumPendingChannels(t, carol, 0, 0)
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:135 (0xde286e)
        	(*harnessTest).RunTestCase: testCase.test(h.lndHarness, h)
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:14470 (0xe55bcc)
        	TestLightningNetworkDaemon.func4: ht.RunTestCase(testCase)
        /home/travis/.gimme/versions/go1.14.7.linux.amd64/src/testing/testing.go:1039 (0x50b9bc)
        	tRunner: fn(t)
        /home/travis/.gimme/versions/go1.14.7.linux.amd64/src/runtime/asm_amd64.s:1373 (0x469401)
        	goexit: BYTE	$0x90	// NOP

another flake that pops up in the same test case (pulled from btcd ARM):
=== RUN   TestLightningNetworkDaemon/data_loss_protection
    lnd_test.go:112: Failed: (data loss protection): exited with error:
        *errors.errorString unable to find Dave's sweep tx in mempool: wanted 1, found 0 txs in mempool: []
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:9829 (0x8a5bc8)
        	testDataLossProtection: t.Fatalf("unable to find Dave's sweep tx in mempool: %v", err)
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:135 (0x87b7d0)
        	(*harnessTest).RunTestCase: testCase.test(h.lndHarness, h)
        /home/travis/gopath/src/github.com/lightningnetwork/lnd/lntest/itest/lnd_test.go:14470 (0x8d8c70)
        	TestLightningNetworkDaemon.func4: ht.RunTestCase(testCase)
        /home/travis/.gimme/versions/go1.14.7.linux.arm64/src/testing/testing.go:1039 (0x10cdfc)
        	tRunner: fn(t)
        /home/travis/.gimme/versions/go1.14.7.linux.arm64/src/runtime/asm_arm.s:857 (0x7801c)
        	goexit: MOVW	R0, R0	// NOP

I suspect they are related

Was looking into the test flake issue, and found this strange message appeared every time when tests randomly failed,
2020-08-29 16:24:04.973 [DBG] PEER: Invalid inv type '1073741825' in notfound message from <IP_ADDRESS>:38814 (inbound)

Does<PHONE_NUMBER> mean anything?
The message was sent from output_btcd_chainbackend.log,
2020-08-29 16:24:04.963 [DBG] PEER: Sending inv (size 3) to <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:04.969 [DBG] PEER: Received getdata (size 3) from <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:04.971 [DBG] PEER: Sending notfound (size 3) to <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:04.975 [INF] SYNC: Lost peer <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:04.976 [DBG] SYNC: Updating sync peer, no progress for: 1.1795227s
2020-08-29 16:24:04.975 [DBG] SRVR: Removed peer <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:04.975 [DBG] CMGR: Disconnected from <IP_ADDRESS>:11160 (reqid 1)
2020-08-29 16:24:04.977 [INF] SYNC: Syncing to block height 2718 from peer <IP_ADDRESS>:11160
2020-08-29 16:24:04.978 [DBG] CMGR: Reconnecting to <IP_ADDRESS>:11160 (reqid 1)
2020-08-29 16:24:04.980 [DBG] CMGR: Retrying connection to <IP_ADDRESS>:11160 (reqid 1) in 5s

And recieved in output_btcd_miner.log,
2020-08-29 16:24:04.964 [DBG] PEER: Received inv (size 3) from <IP_ADDRESS>:38814 (inbound)
2020-08-29 16:24:04.966 [DBG] RPCS: Received command <getrawmempool> from <IP_ADDRESS>:47676
2020-08-29 16:24:04.966 [DBG] PEER: Sending getdata (size 3) to <IP_ADDRESS>:38814 (inbound)
2020-08-29 16:24:04.972 [DBG] PEER: Received notfound (size 3) from <IP_ADDRESS>:38814 (inbound)
2020-08-29 16:24:04.973 [DBG] PEER: Invalid inv type '1073741825' in notfound message from <IP_ADDRESS>:38814 (inbound)
2020-08-29 16:24:04.974 [INF] SYNC: Lost peer <IP_ADDRESS>:38814 (inbound)
2020-08-29 16:24:04.975 [DBG] SRVR: Removed peer <IP_ADDRESS>:38814 (inbound)

The chainbackend lost the miner as a peer twice, and gave it up here,
2020-08-29 16:24:54.926 [DBG] PEER: Peer <IP_ADDRESS>:11160 (outbound) appears to be stalled or misbehaving, inv timeout -- disconnecting
2020-08-29 16:24:54.928 [INF] SYNC: Lost peer <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:54.929 [DBG] SYNC: Updating sync peer, no progress for: 45.0034214s
2020-08-29 16:24:54.930 [INF] SYNC: Syncing to block height 2718 from peer <IP_ADDRESS>:11160
2020-08-29 16:24:54.928 [DBG] SRVR: Removed peer <IP_ADDRESS>:11160 (outbound)
2020-08-29 16:24:54.929 [DBG] CMGR: Disconnected from <IP_ADDRESS>:11160 (reqid 1)

Of course the miner wasn't happy either...
2020-08-29 16:24:54.926 [DBG] PEER: Peer <IP_ADDRESS>:41212 (inbound) appears to be stalled or misbehaving, inv timeout -- disconnecting
2020-08-29 16:24:54.928 [INF] SYNC: Lost peer <IP_ADDRESS>:41212 (inbound)
2020-08-29 16:24:54.929 [DBG] SYNC: Updating sync peer, no progress for: 45.0031681s
2020-08-29 16:24:54.930 [INF] SYNC: Syncing to block height 2718 from peer <IP_ADDRESS>:41214
2020-08-29 16:24:54.929 [DBG] SRVR: Removed peer <IP_ADDRESS>:41212 (inbound)

Then everything timed out.
I'm still trying to understand what happened. Just want to know whether<PHONE_NUMBER> meant anything?

It turned out the<PHONE_NUMBER> is a type,
InvTypeWitnessTx     InvType = InvTypeTx | InvWitnessFlag

And a recent change in btcd validates the not found message and disconnects the peer when needed.
So I guess it's missing the check on wire.InvTypeWitnessTx a few others?
When this InvTypeWitnessTx is receieved by the chainbackend, it's considered invalid and the miner will be disconnected. The chain backend's log is as below,
2020-09-02 21:12:08.496 [DBG] PEER: Received notfound (size 3) from <IP_ADDRESS>:19800 (outbound)
2020-09-02 21:12:08.497 [TRC] PEER: (*wire.MsgNotFound)(0xc000340760)({
 InvList: ([]*wire.InvVect) (len=3 cap=3) {
  (*wire.InvVect)(0xc0000f63f0)({
   Type: (wire.InvType) MSG_WITNESS_TX,
   Hash: (chainhash.Hash) (len=32 cap=32) 721b48e2b92688ca08a3a90b6bdc9a5932595c4df047f562d817e1efafe34fc9
  }),
  (*wire.InvVect)(0xc0000f6414)({
   Type: (wire.InvType) MSG_WITNESS_TX,
   Hash: (chainhash.Hash) (len=32 cap=32) ce6d45244a561e51ad5f602f9b49ac6b9181b3d3674f32526ad7f05d3a070cba
  }),
  (*wire.InvVect)(0xc0000f6438)({
   Type: (wire.InvType) MSG_WITNESS_TX,
   Hash: (chainhash.Hash) (len=32 cap=32) 7bc57461ddd7128f1eda47cb354ae5aed1468b2e91ed3a6a28dfda2eb4eef1ba
  })
 }
})

2020-09-02 21:12:08.497 [TRC] PEER: ([]uint8) (len=109 cap=109) {
 00000000  03 01 00 00 40 c9 4f e3  af ef e1 17 d8 62 f5 47<EMAIL_ADDRESS> 00000010  f0 4d 5c 59 32 59 9a dc  6b 0b a9 a3 08 ca 88 26  |.M\Y2Y..k......&|
 00000020  b9 e2 48 1b 72 01 00 00  40 ba 0c 07 3a 5d f0 d7  |..H.r...@...:]..|
 00000030  6a 52 32 4f 67 d3 b3 81  91 6b ac 49 9b 2f 60 5f  |jR2Og....k.I./`_|
 00000040  ad 51 1e 56 4a 24 45 6d  ce 01 00 00 40 ba f1 ee<EMAIL_ADDRESS> 00000050  b4 2e da df 28 6a 3a ed  91 2e 8b 46 d1 ae e5 4a  |....(j:....F...J|
 00000060  35 cb 47 da 1e 8f 12 d7  dd 61 74 c5 7b           |5.G......at.{|
}

2020-09-02 21:12:08.498 [DBG] PEER: Invalid inv type '1073741825' in notfound message from <IP_ADDRESS>:19800 (outbound)
2020-09-02 21:12:08.498 [TRC] PEER: Disconnecting <IP_ADDRESS>:19800 (outbound)

I suspect it's causing the flake that's related to #4561, #4563 and this one.

@yyforyongyu nice detective work! IIRC decred has a segwit-like construct by default, so it makes sense that there might have been an oversight in backporting. Seems like it should be a straightforward fix, want to make a PR to btcd? Also explains why these issues only seem to affect the btcd builds and not bitcoind 👍

@cfromknecht sure, I've created a PR to fix it. And I'm still working on improving the itest...

Fixed now? cc @cfromknecht @yyforyongyu

Fixed now? cc @cfromknecht @yyforyongyu

Fixed now? cc @cfromknecht @yyforyongyu

Yep. Though there might still have flakes I don't think it's specifically related to data_loss_protection.

Fixed now? cc @cfromknecht @yyforyongyu

Yep. Though there might still have flakes I don't think it's specifically related to data_loss_protection.


Add relation controller as widget type in model form

Hello,
I followed this excellent tutorial : https://vimeo.com/123489421 to create a one to many relation using the relation controller element. It perfectly works.
But I wonder why the builder plugin don't allow to create that kind of relation directly in the UI. It already allow to create a simple relation (dropdown / checkboxes), or even the recordfinder element.
I think it would be too much difficule to add the relation controller to the list, as a new widget type or as a suboption of the recordfinder, basically you just have to fill the model you want to link, the number of records you want to show by default and the buttons you want to display in the toolbar (add|remove|create|etc...)
The builder plugin could easily create a generic partial by default to display the relation controller :
<?= $this->relationRender($model) ?> // where $model is the name of the model we defined in the widget options.

The only things we should do manually (as I know you don't want to touch php code too much) would be to add the relation controller to the $implement variable, and link the config_relation.yaml. A little like the reorder behavior.
What do you think about this ?

@Alex360hd this would be complicated to implement as at some point the user would have to manually edit something. Perhaps if we had an interface for adding the RelationController behavior to a controller (similar to FormController and ListController) then we could then also have a RelationController formwidget control (alias for partial that generates a .htm partial file in the model directory) defined as well that would use the defined relationcontroller information.

In the meantime would it be possible to make the list view's saving not fail if you've manually added a relation type and haven't picked a type?

+1 to this, it would be great to be able to fill custom text in that "type" column. Cause there are several plugins that add new types (I think about image resizer for example).
The best option would be to be aware of that additions and add the new types to the list, but another great and simpler options is to allow people to fill what they want into that type.
What about a kind of autocomplete field, but with the possibility to add something that is not to the autocomplete database.

@Alex360hd I'm always open to reviewing Pull Requests


[Feat] - Pinch to zoom

Is your feature request related to a problem? Please describe.
Currently, there is only auto-zoom while cropping but it's will be nice to have the pinch to zoom feature.
Describe the solution you'd like
Adding a pinch to zoom

Hey @mnayef95impressions  welcome and thanks for the suggestion.
Would be a lovely feat, if you feel please drop a PR with the code for it =D
I will tag with help wanted and good first issue maybe someone in the community will do it!
If no one take ownership of this feature in the next two weeks the stale robot will stale and later close.
If you want to keep this open just keep updating the progress of the feature ^^


Camera Commands Infinite loop with gazebo sitl with typhoon model

Hi,
I am trying to run the camera+gimbal with the gazebo sitl (px4 1.15.0) with mavsdk-python (mavsdk 2.12.11). When I run the camera.py example while manually running the mavsdk_server, I get the following logs from mavsdk_server. On the Mavsdk-python side i get timeout messages. The gimbal.py example runs fine without any problem.
Does this mean that the SITL with Typhoon model does not have camera server side implementation? Or am i missing something. I would like to test as much camera plugin with sitl mode as possible. Are there any alternatives for this? Thanks!
[10:30:48|Info ] MAVSDK version: v2.12.11 (mavsdk_impl.cpp:26)
[10:30:48|Debug] Command debugging is on. (mavlink_command_receiver.cpp:26)
[10:30:48|Info ] New system on: <IP_ADDRESS>:54247 (with system ID: 1) (udp_connection.cpp:178)
[10:30:48|Debug] New system ID: 1 Comp ID: 1 (mavsdk_impl.cpp:750)
[10:30:48|Debug] Command debugging is on. (mavlink_command_sender.cpp:15)
[10:30:48|Debug] Component Autopilot (1) added. (system_impl.cpp:366)
[10:30:48|Debug] Component Gimbal (154) added. (system_impl.cpp:366)
[10:30:48|Info ] System discovered (connection_initiator.h:39)
[10:30:48|Info ] Server started (grpc_server.cpp:169)
[10:30:48|Info ] Server set to listen on <IP_ADDRESS>:50051 (grpc_server.cpp:170)
[10:30:49|Warn ] Vehicle type changed (new type: 13, old type: 0) (system_impl.cpp:217)
[10:30:49|Debug] Discovered 2 component(s) (system_impl.cpp:509)
[10:30:49|Debug] COMMAND_LONG 520 to send to 1, 1 (mavlink_command_sender.cpp:112)
[10:30:49|Debug] MAVLink: info: Connection to ground station lost        (system_impl.cpp:243)
[10:30:49|Debug] Sent command 520 (mavlink_command_sender.cpp:407)
[10:30:49|Debug] Received command ack for 520 with result 0 after 0.00235067 s (mavlink_command_sender.cpp:185)
[10:30:49|Debug] MAVLink: critical: Preflight Fail: vertical velocity unstable (system_impl.cpp:243)
[10:30:49|Debug] MAVLink: info: GCS connection regained  (system_impl.cpp:243)
[10:30:49|Debug] MAVLink: critical: Preflight Fail: vertical velocity unstable (system_impl.cpp:243)
[10:31:41|Debug] MAVLink: critical: Preflight Fail: height estimate not stable (system_impl.cpp:243)
[10:31:50|Error] notify_current_settings has no camera definition (camera_impl.cpp:2149)
[10:31:50|Error] notify_possible_setting_options has no camera definition (camera_impl.cpp:2192)
[10:31:50|Debug] COMMAND_LONG 522 to send to 1, 100 (mavlink_command_sender.cpp:112)
[10:31:50|Debug] Sent command 522 (mavlink_command_sender.cpp:407)
[10:31:50|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.001559 s (mavlink_command_sender.cpp:172)
[10:31:50|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:50|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:50|Warn ] sending again after 0.508127 s, retries to do: 3  (522). (mavlink_command_sender.cpp:320)
[10:31:50|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.511235 s (mavlink_command_sender.cpp:172)
[10:31:50|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:51|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:51|Warn ] sending again after 1.00886 s, retries to do: 2  (522). (mavlink_command_sender.cpp:320)
[10:31:51|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 1.01229 s (mavlink_command_sender.cpp:172)
[10:31:51|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:51|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:51|Warn ] sending again after 1.51331 s, retries to do: 1  (522). (mavlink_command_sender.cpp:320)
[10:31:51|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 1.51555 s (mavlink_command_sender.cpp:172)
[10:31:51|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:52|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:52|Error] Retrying failed (522) (mavlink_command_sender.cpp:346)
[10:31:55|Debug] COMMAND_LONG 522 to send to 1, 100 (mavlink_command_sender.cpp:112)
[10:31:55|Debug] Sent command 522 (mavlink_command_sender.cpp:407)
[10:31:55|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.00203983 s (mavlink_command_sender.cpp:172)
[10:31:55|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:55|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:55|Warn ] sending again after 0.510594 s, retries to do: 3  (522). (mavlink_command_sender.cpp:320)
[10:31:55|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.512605 s (mavlink_command_sender.cpp:172)
[10:31:55|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:56|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:56|Warn ] sending again after 1.01595 s, retries to do: 2  (522). (mavlink_command_sender.cpp:320)
[10:31:56|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 1.02022 s (mavlink_command_sender.cpp:172)
[10:31:56|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:56|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:56|Warn ] sending again after 1.51854 s, retries to do: 1  (522). (mavlink_command_sender.cpp:320)
[10:31:56|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 1.52286 s (mavlink_command_sender.cpp:172)
[10:31:56|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:31:57|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:31:57|Error] Retrying failed (522) (mavlink_command_sender.cpp:346)
[10:32:00|Debug] COMMAND_LONG 522 to send to 1, 100 (mavlink_command_sender.cpp:112)
[10:32:00|Debug] Sent command 522 (mavlink_command_sender.cpp:407)
[10:32:00|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.00317863 s (mavlink_command_sender.cpp:172)
[10:32:00|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:32:00|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:32:00|Warn ] sending again after 0.500759 s, retries to do: 3  (522). (mavlink_command_sender.cpp:320)
[10:32:00|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.504278 s (mavlink_command_sender.cpp:172)
[10:32:00|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[10:32:01|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[10:32:01|Warn ] sending again after 1.01029 s, retries to do: 2  (522). (mavlink_command_sender.cpp:320)

Does this mean that the SITL with Typhoon model does not have camera server side implementation?

I believe this is correct. At least I know that https://github.com/JonasVautherin/px4-gazebo-headless does not have a camera implementation :+1:.

Ah, then I know why it does not work. But why does the server go into an infinite loop?
Does this line mean something to debug this problem? Is 1/100 sysid/comp id for the supposed camera component?
Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100)

The command should just fail right?

Is 1/100 sysid/comp id for the supposed camera component?

I think so, yes.

The command should just fail right?

If it was sent to 1/100, then 1/1 should not ACK it. If there is no camera server, it should time out.

Also, the typhoon_480 model used in https://github.com/JonasVautherin/px4-gazebo-headless does have a camera_manager_plugin attached to it

I never enabled it in px4-gazebo-headless, because I don't think it existed when I wrote it and I never used the camera_manager_plugin myself. If you know how to use it and manage to enable it in px4-gazebo-headless, I would love to get a PR!

I am not sure if I know how to adapt it. I will try to explain what I learned.
I guess the Gazebo Model file Typhoon SDF Jinja defines the plugins attached to the model. And in this model it is already containing the camera_manager_plugin as well as gimbal_controller.
Then there is a init script PX4 Init Script SITL contains the command which starts mavlink streaming for the camera module as well as gimbal.
Then there is another override file i think which defines additional commands such as default params as for Typhoon 480 which defines some parameters for the gimbal (MNT_MODE_IN, MNT_MODE_OUT which both means Mavlink 2 Gimbal). I do not see any param for camera and also could not successfully find in the px4 documentation which param to set for camera. Do you have any hint?

the simulator logs throws an error related to exiftool
sh: 1: exiftool: not found
and then proceeds to log the following:
INFO  [dataman] data manager file './dataman' size is 7872608 bytes
INFO  [init] PX4_SIM_HOSTNAME: localhost
INFO  [simulator_mavlink] Waiting for simulator to accept connection on TCP port 4560
INFO  [simulator_mavlink] Simulator connected on TCP port 4560.
INFO  [lockstep_scheduler] setting initial absolute time to 340000 us
WARN  [vehicle_angular_velocity] no gyro selected, using sensor_gyro_fifo:0 1310988
INFO  [commander] LED: open /dev/led0 failed (22)
ERROR [param] Parameter UXRCE_DDS_DOM_ID not found.
etc/init.d-posix/rcS: 310: etc/init.d-posix/rcS: uxrce_dds_client: not found
INFO  [mavlink] mode: Normal, data rate: 4000000 B/s on udp port 18570 remote port 14550
INFO  [mavlink] mode: Onboard, data rate: 4000000 B/s on udp port 14580 remote port 14540
INFO  [mavlink] mode: Onboard, data rate: 4000 B/s on udp port 14280 remote port 14030
INFO  [mavlink] mode: Gimbal, data rate: 400000 B/s on udp port 13030 remote port 13280
INFO  [mavlink] mode: Onboard, data rate: 4000 B/s on udp port 14558 remote port 14530
INFO  [logger] logger started (mode=all)
INFO  [logger] Start file log (type: full)
INFO  [logger] [logger] ./log/2024-10-23/16_14_49.ulg
INFO  [logger] Opened full log file: ./log/2024-10-23/16_14_49.ulg
INFO  [mavlink] MAVLink only on localhost (set param MAV_{i}_BROADCAST = 1 to enable network)
INFO  [mavlink] MAVLink only on localhost (set param MAV_{i}_BROADCAST = 1 to enable network)
INFO  [px4] Startup script returned successfully
pxh> INFO  [mavlink] partner IP: <IP_ADDRESS>
INFO  [tone_alarm] home set
INFO  [commander] Ready for takeoff!

says nothing about camera. probably failing to load the camera driver because exiftool not found?

aha, just found this error message in the camera plugin:
camera plugin error message
if (system("exiftool -ver &>/dev/null") != 0) {
        gzerr << "exiftool not found. geotagging_images plugin will be disabled" << endl;
        gzerr << "On Ubuntu, use 'sudo apt-get install libimage-exiftool-perl' to install" << endl;
        return;
    }

will install it and try it again.

nope, the warning goes away but still the same issue of infinite loop.

the logs from the mavsdk_server are:
[06:39:25|Debug] MAVLink: critical: Preflight Fail: High Gyro Bias (system_impl.cpp:243)
[06:39:25|Error] notify_current_settings has no camera definition (camera_impl.cpp:2149)
[06:39:25|Error] notify_possible_setting_options has no camera definition (camera_impl.cpp:2192)
[06:39:25|Debug] COMMAND_LONG 522 to send to 1, 100 (mavlink_command_sender.cpp:112)
[06:39:25|Debug] Sent command 522 (mavlink_command_sender.cpp:407)
[06:39:25|Debug] Command ack for 522 (from: 1/1) does not match command 522 (to: 1/100) after 0.00392617 s (mavlink_command_sender.cpp:172)
[06:39:25|Debug] Received ack from 1/1 for not-existing command: 522! Ignoring... (mavlink_command_sender.cpp:283)
[06:39:26|Debug] Got timeout! (mavlink_command_sender.cpp:296)
[06:39:26|Warn ] sending again after 0.508501 s, retries to do: 3  (522). (mavlink_command_sender.cpp:320)

Do the following error related to the camera have anything to do the issue:
[06:39:25|Error] notify_current_settings has no camera definition (camera_impl.cpp:2149)
[06:39:25|Error] notify_possible_setting_options has no camera definition (camera_impl.cpp:2192)

and from the camera.py example from Mavsdk-python are:
Waiting for drone to connect...
-- Connected to drone!
Setting mode to 'PHOTO'
Camera mode: UNKNOWN
Setting mode failed with error code: TIMEOUT
Taking a photo
Couldn't take photo: TIMEOUT

and from the camera.py example from Mavsdk-python are

I think that the camera.py example will expect the camera to serve a camera definition file. Not sure if that gazebo addon does.
So I assume you are testing all this with px4-gazebo-headless? Is the camera_manager_plugin enabled (or did you manage to enable it)? Then the question would be: where does it send MAVLink (like is it multiplexed with the autopilot stream?)?

yes i am testing with px4-gazebo-headless. If the camera_manager_plugin only needs to be enabled in the model sdf file then seems to be enabled.
regarding where does the plugin send mavlink, that should be in the sdf file right?
<plugin name="CameraManagerPlugin" filename="libgazebo_camera_manager_plugin.so">
            <robotNamespace>typhoon_h480</robotNamespace>
            <interval>1</interval>
            <width>3840</width>
            <height>2160</height>
            <maximum_zoom>8.0</maximum_zoom>
            <video_uri>udp://<IP_ADDRESS>:5600</video_uri>
            <system_id>1</system_id>
            <cam_component_id>100</cam_component_id>
            <mavlink_cam_udp_port>14530</mavlink_cam_udp_port>
        </plugin>

mavlink_cam_udp_port is 14530.
Now in the init script for px4 sitl i see this line referring to the camera in the commented out line but the mavlink stream start commands refers to onboard module.
px4 init script
Does this mean that this is completely missing the camera module in the init script?

Right, so px4-gazebo-headless changes this px4-rc.mavlink file (here) in order to redirect some of the mavlink streams to the docker host on ports 14540 (for MAVSDK) and 14550 (for QGC).
It seems like the camera plugin sends to 14530. It probably stays inside the container. But for MAVSDK's purpose, we would rather have both PX4 and the camera plugin send on the same stream (from the same port, to the same port). Is there a way to configure it such that the camera plugin sends MAVLink to PX4 and PX4 forwards it to its other ports? I think some kind of forwarding existed, maybe it could be used here 🤔.
@julianoes: would that be a valid use for PX4 forwarding?

can it not be made to work like gimbal_plugin? is there a diference in how a camera manager interacts with px4 vs gimbal_manager <-> px4?

How is the gimbal plugin working there? It would be interesting to know the difference with the camera plugin indeed 👍

@julianoes: would that be a valid use for PX4 forwarding?

Without having read everything, yes that might help.


Support filters: IResultFilter for classic controllers and IEndpointFilter for minimalAPI

Currently, the API methods have to call .ToActionResult() / .ToResult() extension methods for converting a DomainResult to IActionResult/IResult.
To avoid the repetitive calls of those methods, the MVC package can offer filters (IResultFilter for the classic controllers and IEndpointFilter for the minimal API). So, if the returning type is DomainResultBase or Task<DomainResultBase>', it would

call .ToActionResult() or .ToActionResultOfT() for the classic controllers;
call .ToResult() or .ToResultOfT() for the minimal API.

Notes:

IResultFilter / IAsyncResultFilter can be registered as Global Fitlers for the classic controllers.
The closest alternative to those filters in the minimal API is IEndpointFilter (see Filters in minimal API)
Of course, anywhere ToCustomActionResultOfT() is used, the filters wouldn't help

After some research, turns out that filters are not suitable for conversion of the returned values.
In my research, I created a test filter:
public class ResultFilter : IResultFilter, IEndpointFilter
{
	public void OnResultExecuting(ResultExecutingContext context) {}

	public void OnResultExecuted(ResultExecutedContext context)
	{
		// See context here
	}

	public async ValueTask<object?> InvokeAsync(EndpointFilterInvocationContext context, EndpointFilterDelegate next)
	{
		// See context here
		return await next(context);
	}
}

which was registered globally for the classic API as
services.AddSingleton<IResultFilter, ResultFilter>();
services.AddControllers(options => options.Filters.AddService<IResultFilter>());

and individually for the minimal API.
The classic controllers would have to step away from the strongly typed returned signature  (e.g.IActionResult,ActionResult<T>) in favour of obscure void or Task:
[HttpGet("[action]")]
public void Get200OkWithNumber()
	=> _service.GetSuccessWithNumericValue();

The call of minimal API methods would remain the same:
app.MapGet("200OkWithNumber", () => service.GetSuccessWithNumericValue())
     .AddEndpointFilter<ResultFilter>()
     Produces<int>();

In all cases, the Result property was EmptyResult indicating the inability to convert IDomainResultBase to the expected IActionResult/IResult.
Hence, the ticket is closed


gene_summary table contains 'None' (not NULL) gene entries

With version 0.18.0:
sqlite> select gene from gene_summary where gene='None';
None
None
None

this must have been present in earlier version as well. I'll have a look at this for 0.18.1

We rebuilt the databases after a brand new installation on a new machine, and still get Nones:
sqlite> select count(*) from gene_summary where gene='None';
26455
sqlite> select count(*) from gene_summary;
121502

Not blocking though, just untidy.

Which version of VEP are you using and which options?
I've been testing VEP v82 and v83. I see gene=='None' for variants where the variant is annotated as: impact=='regulatory_region_variant' if I run VEP v83 with the --regulatory option. (gene==None because the variant is not in a gene)  http://uswest.ensembl.org/info/genome/variation/predicted_data.html#consequences

It is VEP v82. Unfortunately I don't always have the vcf to check, but I was quite sure that the 'None' was coming from Python (VEP is written in Perl).

This has been present for a long time. I've just made a change to set those to NULL rather than the 'None' string.


Deprecated package

One month after the publication of the page, the package AdaptiveCards.Rendering.Html was deprecated, removed from the project, and no longer updated. In fact, it is still in nuget; but it does not work with current AdaptiveCards packages (though it declares no specific dependency).
Update the documentation accordingly, possibly suggesting alternatives.

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

ID: 20a9d59f-d59c-47a4-2b41-389183c551ea
Version Independent ID: b62496c8-3bf5-c148-c0b5-cfe25b520998
Content: Render a card - .NET HTML SDK - Adaptive Cards
Content Source: AdaptiveCards/sdk/rendering-cards/net-html/render-a-card.md
Product: adaptive-cards
GitHub Login: @matthidinger
Microsoft Alias: mahiding

Merge.md

Commit.md


fix: exports for react native

#6599
Examples from other open-source libraries
graphql-eslint
react-hook-form
zora
effector
react-final-table

@stelmakhivan Could you please change the change type from minor to patch?

sure. the same here
https://github.com/dotansimha/graphql-eslint/pull/841


Code Review: Prettify the plugin icon for alerts (#17577)

Code review for BohemianCoding/Sketch#17577 (“Prettify the plugin icon for alerts”):
Connect to BohemianCoding/Sketch#17577.

Warnings

:warning:

This pull request may need a CHANGELOG entry.

:warning:

There are library changes, but not tests. That's OK as long as you're refactoring existing code

:warning:

There are library changes, but not docs. That's OK as long as you're refactoring existing code

  Generated by :no_entry_sign: dangerJS


Git Branches

Not sure if this is possible, more of a hair-brained idea and not even sure how useful it would be.  Is it possible to setup a target option that would allow a specific branch to be checked out for inclusion in the build?  That is, if I'm making a bunch of modifications on a branch and want to check them out, can I include a "branch" = "master" in the Target and have that branch checked out temporarily for inclusion?

That's a nifty idea and I think it would be a useful inclusion. It'd be a little more involved than just doing a checkout, since there's things like branches having uncommitted work, etc.

Yep, I didn't consider that, though that might be a desired trait.  For example, I'd like to test out changes I've made on a branch before I choose to commit them, so I want the modified files to be included, not just the committed ones.  With that being said, if this could be worked in at some point, a warning that modified files are being used would be appropriate.  Thanks for thinking about it.  If I ever learn nim, maybe I can contribute more than pipe dreams and "good ideas".


Allow nested messages in propTypes (resolve #32)

@trevordmiller did not see an email notification for this - apologies. Do you still need this merged in and released? I noticed the parent issue is closed. If you're not in a rush either way, I can still add it in of course as well for the overall API improvements.

@sprjr No worries. The parent issue is still open: https://github.com/sprjr/react-localize/issues/32. I feel this would be useful to merge if you do, as the current docs show support for it. Currently I'm using a fork until this gets merged, as I don't like the console errors cluttering things up ;)
https://github.com/eggheadio/react-localize

@trevordmiller ah ok. Do you have a link/example of how you're using it? I know by default we're using get() in the localize function called here.
Am wondering why that did not work as I thought get would support that for both of these objects:
{ 'foo.bar.baz': 'bat' }

{ foo: { bar: { baz: 'bat' } } }

@sprjr Yes, here is my localization bundle:
https://github.com/eggheadio/egghead-instructor-center/blob/master/src/utils/localizationBundle/index.js
And here is where I use that bundle:
https://github.com/eggheadio/egghead-instructor-center/blob/master/src/index.js#L59

@trevordmiller I'm down to merge the changes here, but looking through the actual changes - are there any commits missing? This seems to only change the PropTypes themselves, which I don't know if that would impact the actual output, only log some warnings.
I added ecb5f9c to a branch to validate that the <Text/> component is working with your bundle. I'll merge this and a few other things and cut a release today for ya - let me know if I am missing something on the code changes themselves. Thanks!

That is correct, this issue was about removing the proptype warnings (errors when using create-react-app), so nested objects were fully supported without console clutter

Thanks!


[docs] Add note about TOC to developer guide

Totally optional, but just reminds folks that there's a nice built-in TOC for markdown files, which is pretty handy and improves the reading experience for longer content.
Screenshot:

closing this as we are porting over to docsmobile.


Project Page - Blocked staff can also be able to create project in project page

Describe the bug
https://www.berrycast.com/conversations/507b6e68-a699-590e-857b-1b5c54137f50

https://www.berrycast.com/conversations/2f67fb5e-4632-56fd-9ad5-5958adfaca16


[L5.1] config stops config:cache working

the api config currently has this:
    'auth' => [
        'basic' => function ($app) {
            return new Dingo\Api\Auth\Provider\Basic($app['auth']);
        }
    ],

the closure there will break the config:cache from working, is there anyway you can rework the code to avoid the closure being needed, was thinking something like:
    'auth' => [
        'basic' => 'Dingo\Api\Auth\Provider\Basic',
    ],

then the config:cache will work and and we can still use the speed boost :D

This is an issue in a few libraries now (including my own)
https://github.com/tymondesigns/jwt-auth/issues/147
https://github.com/lucadegasperi/oauth2-server-laravel/issues/323
https://github.com/efficiently/authority-controller/issues/27

looks like there is a fix here: https://github.com/efficiently/authority-controller/issues/27#issuecomment-101974857
will try it when i get home and report back

Yep SuperClosure can be used to serialize it.

I'll look into simplifying the configuration as much as I can. I'm afraid it may mean configuring it in other places though, which is probably less than ideal.

I'll look into simplifying the configuration as much as I can. I'm afraid it may mean configuring it in other places though, which is probably less than ideal.

i had a look locally, it worked without issue, so i created a pull request

Following your advice, my config looks like this

I get this error when hitting any artisan command / http endpoint afterwards

Anyone else seeing this?

I've not used the serializer for config caching so can't give you any advice here.
I still need to look more into this issue. I guess for now if this isn't work don't cache your config. That or do your API related config in a bootstrapping file as everything can be configured on runtime.
Here's how you'd do authentication.
app('Dingo\Api\Auth\Auth')->extend('jwt', function ($app) {
    return new Dingo\Api\Auth\Provider\JWT($app['Tymon\JWTAuth\JWTAuth']);
});

The wiki covers the others.

Laravel doesn't use closures in config anymore. you will need to pass a string. Any update on this ?

Hm, I'll see what changes I can make.
I guess most of the more complex keys can be pulled out and simply be set from a provider.

I've gone ahead and removed the ability to use closures and callables in a config file. Docs have been updated to demonstrate the alternative ways of configuring more complex aspects of the package.


dump error

Are you sure this is a bug?
If unsure, please post your question/situation to the forum first: aidenlab.org/forum.html
Describe the bug
running "java -jar juicer_tools_1.19.02.jar dump observed KR inter_30.hic 1 1 BP 100000 SCI_input.txt"
the error :
"WARN [2020-05-12T15:04:55,107]  [Globals.java:138] [main]  Development mode is enabled
Exception in thread "main" java.lang.NullPointerException
at juicebox.tools.clt.old.Dump.extractChromosomeRegionIndices(Dump.java:454)
at juicebox.tools.clt.old.Dump.readArguments(Dump.java:323)
at juicebox.tools.HiCTools.main(HiCTools.java:93)"
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. macOS Mojave, Windows 10, etc]
Version [e.g. 1.6]

Additional context
Add any other context about the problem here.

I'm experiencing the same issue with juicer_tools_1.21.01, running on Ubuntu 14.04.

@sa501428

@Gonysh @xiaowenchenjax can you share a link to a hic file which is failing this command?

I would highly recommend using https://github.com/aidenlab/straw instead of the dump command.


Duplicate class com.google.android.gms

Hello, first off thank you for making this plugin accesible!
When I try using it I'm encoutering the following error:
Launching lib/main.dart on Pixel 6 Pro in debug mode...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugDuplicateClasses'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.CheckDuplicatesRunnable
   > Duplicate class com.google.android.gms.games.AchievementsClient found in modules jetified-play-services-games-22.0.1-runtime (com.google.android.gms:play-services-games:22.0.1) and jetified-play-services-games-v2-17.0.0-runtime (com.google.android.gms:play-services-games-v2:17.0.0)
     Duplicate class com.google.android.gms.games.AnnotatedData found in modules jetified-play-services-games-22.0.1-runtime (com.google.android.gms:play-services-games:22.0.1) and jetified-play-services-games-v2-17.0.0-runtime (com.google.android.gms:play-services-games-v2:17.0.0)
     Duplicate class com.google.android.gms.games.CurrentPlayerInfo found in modules jetified-play-services-games-22.0.1-runtime (com.google.android.gms:play-services-games:22.0.1) and jetified-play-services-games-v2-17.0.0-runtime (com.google.android.gms:play-services-games-v2:17.0.0)
     Duplicate class com.google.android.gms.games.EventsClient found in modules jetified-play-services-games-22.0.1-runtime (com.google.android.gms:play-services-games:22.0.1) and jetified-play-services-games-v2-17.0.0-runtime (com.google.android.gms:play-services-games-v2:17.0.0)
     Duplicate class com.google.android.gms.games.FriendsResolutionRequiredException found in modules jetified-play-services-games-22.0.1-runtime (com.google.android.gms:play-services-games:22.0.1) and jetified-play-services-games-v2-17.0.0-runtime (com.google.android.gms:play-services-games-v2:17.0.0)

I've tried this with a brand new Flutter project (using flutter create flutter_golden) pasting in the example/lib/main.dart contents.
Any pointers? Maybe a dependency version needs to be updated?
Thanks!

https://github.com/revoltgames/game_services_firebase_auth/pull/14 solves this compile-time issue for me.

It should be good now


I just want to use semantic segmentation, do I need to install the mmdet module?

I just want to use semantic segmentation, do I need to install the mmdet module?

I just want to use semantic segmentation, do I need to install the mmdet module?

Yes, this is because some functions for Mask2Former are imported from mmdet.

Fine,thanks.


Minor improvements to the guide

Everything relates to the text rather than the code, unless there were some trivial formatting changes like removing whitespace.
If you want, I can squash all these commits into one.

Thank you!!


Install yq dependency from vendor rather than go get

We don't need the version to keep updating.
This also works around the issue with failing nightly builds as it was complaining about dependency not being marked explicit.
See slack: https://knative.slack.com/archives/C012J5TCS6Q/p1591062361000800 for more details about the CI failures.

@mattmoor : ready for another review

@mattmoor : Ping


Fix #119: make event optional in DataLayerObject

This fixes #119 for me; make event optional in DataLayerObject. I'm using Vue 2 so I made this patch off the vue-2 branch; hope that's OK.

I'm using Vue 2 so I made this patch off the vue-2 branch; hope that's OK.

Sorry but no :slightly_frowning_face:
You need to modify the master branch
I keep care of syncing the master changes into vue-2 branch and then release it separately
In this way, both branches / versions are mostly feature congruent
Therefore could you change the branch to master and try to fix potential conflicts (or just create a new PR?)
A second thing that would be really nice, would be to add a test for your case
But... are you sure that we can add something to the dataLayer without the event property?
Why don't you just do something like this?:
if (Vue.gtm.enabled()) {
  window.dataLayer?.push({ event: 'userIdEvent', userId: user?.uid || '' })
}

Could you provide me with information why you explicitly don't want the event property to be set?
I just want to learn about why this is a case :slightly_smiling_face:

And did you know that there is a dataLayer() function?
So maybe you can use
const dataLayer = Vue.gtm.dataLayer()
if (dataLayer) {
  dataLayer.push({ event: 'userIdEvent', userId: user?.uid || '' })
}

The line 137 will then take automatically care about that the plugin is enabled or not
https://github.com/mib200/vue-gtm/blob/9fe82f1079d28cf21473acdc076f0de1c942460f/src/plugin.ts#L130-L141

The use case is that it's easier for me to push things into the data layer in the place in my code where I know that thing. But it's not always the same place where I want the event to be - the event will have a combination of data from different places. Yes, I could create a bogus extra event just for this, but why do that when the official API allows pushing things into the data layer at any point?
Thanks for the .dataLayer() function -- I'll use that. Much better than window globals for sure. I think I got the window.dataLayer from one of the vue-gtm examples; I'll check.
I'll redo my patch against master and see if I can add a test. But I can only test it live on Vue 2, just so you know.

Closing; see #121 for update.


Define Source as Search Parameter

As we are adding different data sources and IoT to the search, the question is, if we ad a source parameter to the search.
The current search with the hashtag #USA for example should display all results from all sources in the system: http://test.loklak.net/search?q=%23USA
How can we define to only show results from the source twitter?

I believe there is a parameter in the query that can help us with that. e.g. source=twitter (other options are cache, backend, all as default). Api link: http://loklak.org/api.html. It's hard to demonstrate now since I can't see any other source but "source_type" : "TWITTER" in the search result. When there are other sources, I'm sure can create a filter-like option in search to define the source of the search results.
@prasht63, @zyzo I believe there is an integration going from other data sources, can you give us a quick update here?

@fuksi please take a look at PR https://github.com/loklak/loklak_server/pull/79. I believe this is what you're looking for ?
To search only twitter data, the search query would be /search?q=%23USA%20%2Fsource_type%3DTWITTER

@zyzo Thank you, the PR clears this up. Can you prefer to this issue again when the PR is merged. And if possible, a example query that contains results from source_type=FOSSASIA_API  as well.

In the meantime, I'll define source = twitter as default


GET request url params

Is there any way to have GET request url parameters without appending them to the URL yourself beforehand?
Could not find any documentation on this.

We didn't implement that, because for this feature to be complete and consistent, we'd have to be able to merge queryparams from the url with the ones provided and it would increase the size of the codebase.
There's a separate module for that and it's called qs


Array length recorded in a Node

It can be placed in the tree right after opcodeArray, making len proc O(1) time.

Takes a tiny fraction in the benchmark. Not worth it.


Privacy Manifest

Subject: We need to incorporate the xcprivacy manifest into our app . Starting March 13, Apple will send notifications when uploading builds to the App Store, and this will become mandatory from May 1. All the SDKs used in our app needs to incorporate the xcprivacy manifest.
Reference: https://developer.apple.com/news/?id=3d8a9yyh
Actions needed: We need the compatible xcprivacy manifest version IOS SDK with xcprivacy manifest. We are expecting a seamless upgrade by upgrading to the latest version.

Check with this tool as well. Swinject is not using any required reason API.
https://github.com/omarzl/ios_17_required_reason_api_scanner/tree/binary_analyzer
@sakshibala-halodoc : you can close this issue safely

Thanks @kishorepran Closed.


Final project work, v0.4.

Why is this change being made?  Adding some code for development and markdown practice
What has changed? Added mvn project, added dataset csv file, added a pseudo map-hash-map function, added some pseudo test functions, added block diagram (img) for architecture overview.
How was this tested? Test with man test and man package, commented out the test files to pass the testing for now
Anything else to discus? Yes, more work and development is needed to make this into a java program. Need to add some more reviewers
Any other documentation? Not at this time

I will touch base with Sean, to see if they indeed want to team up. I will send them a spreadsheet.

Thanks. As discussed, he might be only focused on the changemakers, as per Paul's direction, but we can always welcome another to the project. Let me know if he wants to set up a time to discus the plan going forward.

This is a good start overall. Can you add more description to your diagram? I'm not sure what the Map means above, whether it's a mapping app like a GPS or like a HashMap.
I think your next step will be deciding what question to ask from the data. I also think there are some possiblities to frame this as a graph problem, with nodes being the stops on your trip and edges weighted by the cost of the trip (driving time, gas, mileage). It could also be an optimization problem if we can write some linear equations describing the constraints of each trip. Let's discuss more in class.

This is a good start overall. Can you add more description to your diagram? I'm not sure what the Map means above, whether it's a mapping app like a GPS or like a HashMap.
I think your next step will be deciding what question to ask from the data. I also think there are some possiblities to frame this as a graph problem, with nodes being the stops on your trip and edges weighted by the cost of the trip (driving time, gas, mileage). It could also be an optimization problem if we can write some linear equations describing the constraints of each trip. Let's discuss more in class.

The map on the diagram was just meant to represent the top level map in the Map of Hash maps. I'll modify the diagram to be more clear. Thanks for the feedback on using a graph as i am not set on using the data structure i have selected so far. As for gas usage, that data needs to be collected from my app and weather too but both are easy to backdate.

Added a new concept drawing
https://github.com/TheEvergreenStateCollege/upper-division-cs/blob/pswish-nathanmcl/image.png


Undocumented commented-out code

What's going on here?
https://github.com/solid/solid-web-client/blob/master/src/client.js#L49-L66

Forgot to delete. Feel free to remove it.


Upgrade to Scala 2.12.19 and 2.13.13.

After this PR I think we'll be ready to publish v1.16.0. It's been 2.5 months since the last release. WDYT?


Request theme version for Wordpress.org

As wordpress.org is a highly popular blogging platform, I request you people to kindly make this theme for wordpress platform too. Please kindly look after my request.

Hi @thecoducer! I think it would be good to have a wordpress theme around this, but I am afraid I won't be doing it myself! I will the issue open here so that someone else brave can work on it!
Cheers,
F:)

Thanks for addressing.


Suggestions for tye api json to be more friendly to other systems and to the march of time.

What should we add or change to make your life better?
Inspecting http://localhost:8000/api/v1/services after doing the first couple tutorials, shows that the tye services api is returning json objects in some cases, where arrays might make more sense, or vice-versa. I figured it'd be best to bring this up early, as any change would be a breaking change.
--
I haven't explored every feature yet, but [0].replicas will be a problem for some languages and auto generation of objects from json, and if it could be an array, that'd be super-duper-rad.
replicas": {
"backend_d42b8943-2": {

--
While it's no longer really a security concern like it used to be (older browsers):
https://haacked.com/archive/2009/06/25/json-hijacking.aspx/
The json payload is returned as a top-level array, which means backwards compatible changes to the overall structure won't be possible...
[
  {
    description: {},
    restarts: 0,
    serviceType: 1,
    status: {},
    replicas: {}
  },
  {
    description: {},
    restarts: 0,
    serviceType: 1,
    status: {},
    replicas: {}
  }
]

If wrapped in a containing json object, a low-likelihood security issue is mitigated, auto-generation of code is easier, backwards compatible changes can be applied (new properties, arrays, objects), the payload can be enriched/combined with by other systems, and indexing in something like ElasticSearch tends to go more smoothly:
{
"services":  [
    {
      description: {},
      restarts: 0,
      serviceType: 1,
      status: {},
      replicas: {}
    },
    {
      description: {},
      restarts: 0,
      serviceType: 1,
      status: {},
      replicas: {}
    }
  ],
  "backwardsCompatibleChange": true,
}

--
Minor suggestion: Enums being an int (serviceType: 1) are very difficult to troubleshoot, and integrate against.
Why is this important to you?
I write a lot of glue services, infrastructure services, pipelines, automation, APIs - and deal a lot with 3rd party json apis, especially in regards to internalish ones like /api/v1/services. Minor tweaks like the above suggestions go very far on ecosystems that will rely on something like Tye. Especially, when people start to contort it in ways that were never expected.
Custom build systems, monitoring, CI/CD pipelines, etc will be hooked up to this almost immediately, and I found the above suggestions really help make a product 'snap' into place for Infra/Ops/SRE-type teams.
I'd be happy to get a PR in with these suggestions, but I wanted to gauge interest first.

Minor suggestion: Enums being an int (serviceType: 1) are very difficult to troubleshoot, and integrate against.

Out of curiosity for general JSON HTTP API design, what do you prefer instead? I've come up against lots of different opinions on this topic between multiple teams, with pros and cons either way.

Minor suggestion: Enums being an int (serviceType: 1) are very difficult to troubleshoot, and integrate against.

Out of curiosity for general JSON HTTP API design, what do you prefer instead? I've come up against lots of different opinions on this topic between multiple teams, with pros and cons either way.

If using OpenApi (Swagger) there is an enum descriptor: https://swagger.io/docs/specification/data-models/enums/
Before OpenApi there were extensions to the Swagger format that would do the same. I believe they were from Microsoft originally.
If not using something like OpenApi, I think just a string denoting the value works well: "serviceType": "Executable"
This has another benefit of decoupling language features from the API a bit. I believe that not all languages have enums that map to numbers, as an example.
It's also more along the lines of a self describing API, at least for the values that have been returned to the caller. I can get a payload with "serviceType": "Container", and "serviceType": "Executable" and know a lot more automatically than "serviceType": 1 and "serviceType": 2.
In extremely perf sensitive APIs the overhead to another string might concern some folks, but those APIs probably shouldn't be using json. :)
@ericsampson  - I'm curious what you've heard for being pros to ints here? I'm sure there are some!

This is awesome feedback, thank you!
We haven't a polish pass on the API yet - and I think we're a few steps away from that. We'll use this when we do.

I'm curious what you've heard for being pros to ints here? I'm sure there are some!

I learned a lot about this from talking with @steveharter and @JamesNK when we built System.Text.Json last year. One of the reasons why .NET serializers default to int is that you really have to intend for the enum to be serializable as a string for that to work.  Enum features like flags or unnamed values would have to throw an exception if you're using strings.
In terms of defining a good API, I definitely prefer strings. JSON is supposed to be human readable/writable.

If someone wants to take this on we're open to any and all of these suggestions. We think these are good ideas, but we're not working on it right now because nothing consumes the api.... yet 😉


npm audit report vulnerability of a dependency

❯ npm audit
# npm audit report

trim  <0.0.3
Severity: high
Regular Expression Denial of Service in trim - https://github.com/advisories/GHSA-w5p7-h5w8-2hfq
No fix available
node_modules/trim
  mdast-util-to-hast  <=6.0.2
  Depends on vulnerable versions of trim
  node_modules/decap-cms-widget-markdown/node_modules/mdast-util-to-hast
    remark-rehype  <=5.0.0
    Depends on vulnerable versions of mdast-util-to-hast
    node_modules/decap-cms-widget-markdown/node_modules/remark-rehype
  remark-parse  <=8.0.3
  Depends on vulnerable versions of trim
  node_modules/decap-cms-widget-markdown/node_modules/remark-parse
    decap-cms-widget-markdown  *
    Depends on vulnerable versions of remark-parse
    Depends on vulnerable versions of remark-rehype
    node_modules/decap-cms-widget-markdown
      decap-cms-app  *
      Depends on vulnerable versions of decap-cms-widget-markdown
      node_modules/decap-cms-app

While this may not be a bigger issue as dev dependency, but worth noting.

@userquin Sorry my mistake, I'll close


Avatars flash on mount even if cached

Bug report
Current Behavior
Avatars never render on first mount, even if cached.
Expected behavior
If avatar image is cached by the browser, the image should render straight away on initial mount.
Reproducible example
Refresh this page: https://www.radix-ui.com/docs/primitives/components/avatar
Suggested solution
Instead of loading the image in useEffect, the image loading should be started directly in the useState initializer. If the image is cached, image.complete of new Image will return true synchronously, and then the state value can be returned as "loaded" right away.

@frykten, it does not fix this judging by the code. ´img.onload´ is not triggered synchronously / on the same tick, so simply changing to ´useLayoutEffect´ does nothing to detect on first render whether the image is ready or not.

This is also the case when providing a base64 string as src.
I'm expecting the component not to flash and show fallback, but render the image directly since there's no network request involved here. However, I see the state moving from loading to loaded, which in turn makes the fallback render when no delay is set.
Reproduction: https://codesandbox.io/s/radix-avatar-b64issue-rr24j8

+1

+1

+1

This is still a bug. Radiates through shadcn and NextUI as well

+1

+1

+1

Is this still not merged?


Added link to playground url

Pre-submit checklist:

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

Pre-merge checklist:

[ ] Someone approved it
[ ] Commits have useful messages
[ ] Review clarifications made it into the code
[ ] History is moderately tidy; or going to squash-merge

Added link to playground url in plutus-playground.rst doc file


Allow for executing Kernels without advancing time

There may be cases where users want to run Kernels, but not actually do a time-step. Examples include e.g. for sampling a field
Currently, Parcels crashes when either dt=0, and also when runtime=0 (or, equivalently, if starttime=endtime).
Should we write a special case in pset.execute() when either dt=0 or runtime=0, to execute the kernels without actually advancing time? Perhaps including a logger.info that time has actually not advanced?

Yes, this is actually important. We were discussing such necessity with David no earlier than yesterday, even if we found another solution at the end.
I'll change this.

Fixed in #247


Edit footnote text about flood conditions

Information on flood stage is necessary to determine flooding conditions and is only available for 38% of gages.
Maybe call out specifically what flood information is needed - the 38% have both an NWS flood stage AND NWISWeb stage.

Potentially:

Flood status information (USGS gage height & National Weather Service flood stage) was available for 38% of gages when this graphic was produced.

and also changing the first line to make it clear that the other data is flow, not gage height

Flow conditions are relative to the historic daily record for each gage

Although, that could potentially add even more confusion, so leaving as "Flood status information" might be a better option still.

I like going with the less detailed version.

Flood status could be determined for 40% of the 8000+ gages when this graphic was produced.
Flood status could be shown for 40% of the gages when this graphic was produced.
Flood status was available for 40% of the 8000+ gages when this graphic was produced.
When this graphic was produced, 40% of gages could show flood status.

Closed by #118