How to implement - search and sort functionality for the variants

Hi guys,
Opening this to have an open discussion about the topic, which is:

We would like to be able to perform queries such as 'show me all high impact variants on chr7:x-y', or 'what variants are heterozyguous in the parents, homozygous in the offspring, and present in this list of genes?' or, in our case 'what variants are present in a set of genomic regions, what is their impact, how often do they occur in our reference set of VCFs?'

This stuff.
GQL comes in when you need to query large data sets (especially for genotype queries -- i.e., what percentage of my samples has a G at chr4:1234) which are painfully slow in Gemini (see http://biorxiv.org/content/early/2015/04/20/018259)
So what we're still trying to understand is:

Do you have these kind of queries in Puzzle's road map?
If so, what workflow do you have in mind for this. Is it VCF/Gemini -> internal DB -> Puzzle ? Then Puzzle will need it's own query system (though we could talk to Aaron to see what could be re-used from Gemini and abstracted away). Or maybe some kind of interface to interact with Gemini/GQL from Puzzle.

Thanks guys!
Ping @ohofmann @brainstorm :-)

Hi,
I will try to answer these questions here. Even though it is hard when we don't know exactly you mean but I will do my best.
In general we reason like this:

puzzle will have different modes similar to now
different modes will have different functionality based on the backend
the VCF backend will have most limits, the GEMINI backend will be able to do gemini stuff etc.
we want to design our own backend to be able to do some specific analysies, this is under discussion
we could create a GQL backend that will have it's limits but will be superfast with huge amount of data(if i understand correct

So for the specific queries above
show me all high impact variants on chr7:x-y
This is easy to do with the gemini adapter. It is actually easy for the VCF adapter to if we use pytabix. If something is unclear I'll be happy to explain more in detail over a meeting
what variants are heterozyguous in the parents, homozygous in the offspring, and present in this list of genes?
This is easy with the vcf adapter if the inheritance patterns are annotated with genmod or a similar tool.
Right now it is hard to do with the GEMINI adapter since we can not access the gemini tools from within the python interface. I hope that you guys who already have a contact with the gemini developers can discuss if they can solve this?
'what variants are present in a set of genomic regions, what is their impact, how often do they occur in our reference set of VCFs?
This query is a bit vague for me but I defenitely think it is possible. We have to discuss how you want ti use the reference set of VCF:s. If you can merge them in a 1000G kind of way and annotate your vcf:s it will be easy. Otherwise we could implement a wrapper module that query the set of VCF:s for us dynamicly.
I hope this makes sense, and I understand that there are a lot of questions left. Let's continue with the discussion of how to proceed.

Hey and thanks so much for this! Comments inline:

I will try to answer these questions here. Even though it is hard when
we don't know exactly you mean but I will do my best.

Absolutely agree. Let me try and generalize -- I am trying to understand
how you envision the flow of variant information throughout puzzle.
Right now you have a mix of VCFs and GeminiDBs which then get processed,
put into an internal backend, and then exposed to the Puzzle user.
Which means that for any possible analysis or query we can come up with
you will need to either a) implement the query support yourself
internally in Puzzle, or b) write a wrapper for Gemini and another for
the VCFs.
I've been pondering whether it is possible to unify this by going from
VCF -> Gemini -> Puzzle, with Gemini serving as the query backend. The
issues I can see with this mostly revolve around flexibility. Right now
Gemini is not good at just adding a few samples or VCFs to a database,
you'd have to re-create it from scratch every time. The other issue is
that I am not sure all the functionality we'd need is exposed through
the API, although so far we have not stumbled into any problems.

we want to design our own backend to be able to do some specific
analysies, this is under discussion

Would love to hear more about this. Ryan Layer and Aaron Quinlan are
pretty good at listening to community feedback, and I'd be happy to set
up a call once we have a better idea of what is needed.

This is easy to do with the gemini adapter. It is actually easy for the
VCF adapter to if we use pytabix. If something is unclear I'll be happy
to explain more in detail over a meeting

See above. I was trying to figure out if there's a way to refactor
things so we do not need two different adapters :)

Right now it is hard to do with the GEMINI adapter since we can not
access the gemini tools from within the python interface. I hope that
you guys who already have a contact with the gemini developers can
discuss if they can solve this?

Absolutely. Been a while I looked at the Gemini internals (though Rory
from the Boston team has -- a lot). What access is missing?

I hope this makes sense, and I understand that there are a lot of
questions left. Let's continue with the discussion of how to proceed.

Thanks again for the feedback, very much appreciated. Mostly paranoid me
trying to see what existing resources we can re-use so I don't get
strangled at the next BOSC meeting.
Best, Oliver
--
@fiamh
Skype: ohofmann
+44<PHONE_NUMBER>

Hi,
Sorry about this, it is a bit confusing.

Which means that for any possible analysis or query we can come up with
you will need to either a) implement the query support yourself
internally in Puzzle, or b) write a wrapper for Gemini and another for
the VCFs.

Right now you can mix gemini and vcfs when loading cases but we will remove this. We have realized that it will be more or less impossible to implement the queries that we want if we allow a VCF and/or GEMINI backend in the same analysis.
So the workflow will be
VCF->GEMINI (this step is outside the scope of puzzle) -> puzzle
or
VCF->puzzle
or
VCF->puzzle backend->puzzle view
So this will mean that when viewing a database with only VCF: you will have limited choices of analysies.
When viewing a database with gemini backend you can do what gemini does best, and when viewing a database with puzzle backend you can hopefully do even more.

What access is missing?

(From gemini) Right now we would like to be able to use the inheritance tools from within the python interface, but it could be more stuff coming up later.
Since they have all the logic in place it might not be that difficult for them to implement, or what do you think?

remove this. We have realized that it will be more or less impossible to
implement the queries that we want if we allow a VCF and/or GEMINI
backend in the same analysis.

That's what I mentioned to Roman and Guillermo a while ago.

So the workflow will be
VCF->GEMINI (this step is outside the scope of puzzle) -> puzzle
or
VCF->puzzle
or
VCF->puzzle backend->puzzle view
So this will mean that when viewing a database with only VCF: you will
have limited choices of analysies.
When viewing a database with gemini backend you can do what gemini does
best, and when viewing a database with puzzle backend you can hopefully
do even more.

Got it now and makes perfect sense. Thanks!
Oliver
--
@fiamh
Skype: ohofmann
+44<PHONE_NUMBER>

On 12-Feb-16 10:04, Måns Magnusson wrote:

What access is missing?

(From gemini) Right now we would like to be able to use the inheritance
tools from within the python interface, but it could be more stuff
coming up later.
Since they have all the logic in place it might not be that difficult
for them to implement, or what do you think?

Agree, but let me check. More in a bit!
Oliver
--
@fiamh
Skype: ohofmann
+44<PHONE_NUMBER>

:+1:

For the record, so that it doesn't get lost on slack conversations:
"""
I wrote up a quick example here:
https://gist.github.com/brentp/1f06b43a39c3af67b085
basically just create the appropriate object, e.g.:
model = CompoundHet(Args(db="../test/test.comp_het.db",
columns="chrom,start,end,impact_severity,impact,gene,transcript,max_aaf_all",
max_priority=3,
allow_unaffected=True))
and then iterate over model.report_candidates() which yields OrderedDict's.
"""
I believe @guillermo-carrasco is having a look at it right now, but he might need some further help/examples/testcases.

:+1: for pytabix in the VCF adapter. I will have more to say about this in a few days when I've had a chance to test the recent changes, particularly search speed.

Let me know if we can help on the gemini end.
FWIW, cyvcf2, which we use in gemini loading supports doing tabix queries out of the box, so doing a spatial query:
vcf = VCF('my.vcf.gz')
for v in vcf('7:123456-2345670'):
     v.start, v.end, v.INFO['DP']

is hardly different from just streaming over the vcf:
for v in vcf:
   ...

the documentation for cyvcf2 is lacking, but the API is pretty simple. so it might be an option for you guys.

Thanks @brentp , only problem is to add dependencies of numpy and cython...

@vasiliosz I'm not ready with some performance stuff for the vcf adapter. There are som new things that will improve especially SV:s alot coming up

@moonso great, I'll wait for those

@moonso, numpy and cython aren't though to install dependencies when installed with Anaconda or wheels (default on PyPi), IMHO. There's really tons of bioinfo packages using them nowadays, should be considered as "standard" deps for doing performance scientific computing, don't you think?

Agree with @brainstorm :-) I lost my fear to these packages when I switched to Anaconda for package and virtual environments management. They're painless to install!

Sure, then we should try it!

@brentp: I'm not as experienced with parsing VCFs as @moonso but will cyvcf2 parse e.g. the CSQ field or do you have to handle that outside?

@robinandeer
you can get a parsed dict from the info as dict(variant.INFO). and get the keys from that.
I don't think anything will parse the CSQ. In gemini, I just grab the tab-delimited list of keys from the Description field in the vcf header and then the values from each variant info and zip them together.

Yes we do it in the same way (with csq)

This might also be of interest to you guys.

@brentp could you add some documentation to cyvcf2 on how to access headers and how to get variants in a region? We should defenitely use this but it is a bit hard to get into right now :smile:
Some quick questions:

Are there any restrictions on the vcf file?
Does it check each value to what is specified in the header?
Does the file need to be ordered on position?
Do you need any index?
Thanks!

@brentp , I could get some of the answers by looking at the tests :+1:

yeah, it definitely needs more docs. but as you found the tests are fairly comprehensive.

Are there any restrictions on the vcf file?

it has to be parse-able by htslib

Does it check each value to what is specified in the header?

yes, well htslib does

Does the file need to be ordered on position?
Do you need any index?
only if you want to do regional queries

Ok, I noticed it does not care about so many things which is good for us!
I will give it a try.

Hello all, I'll close this issue for now since most of the stuff is solved.
If there are other requests we can post (smaller) new issues.
It is possible to do range queries, search for high impact variants etc.


Add support for custom cassandra image

Change Overview
We need to modify official cassandra image to have restic. This PR does that. After this change a docker image will be created that will have cassandra:3.11.3 as base image and restic, kando on top of that.
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Issues
Test Plan
Run below command to test this
goreleaser --snapshot --rm-dist --skip-publish

[ ] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

I don't have too much context here but I am concerned about modifying the upstream image as it makes it hard for others to use (supply chain issues). Is there no other way of handling this?

I agree, but we currently need to modify them to use kanister tools. A longer term proposal would be to use ephemeral containers.

Here is the output of the goreleaser command
   • releasing using goreleaser 0.123.3...
   • loading config file       file=.goreleaser.yml
   • RUNNING BEFORE HOOKS
      • running go mod download
   • LOADING ENVIRONMENT VARIABLES
      • pipe skipped              error=publishing is disabled
   • GETTING AND VALIDATING GIT STATE
      • releasing 0.22.0, commit 89e4ed534a28be46e2cd31ddd7971eb667ca3224
      • pipe skipped              error=disabled during snapshot mode
   • PARSING TAG
   • SETTING DEFAULTS
      • LOADING ENVIRONMENT VARIABLES
      • SNAPSHOTING
      • GITHUB/GITLAB/GITEA RELEASES
      • PROJECT NAME
      • BUILDING BINARIES
      • ARCHIVES
      • LINUX PACKAGES WITH NFPM
      • SNAPCRAFT PACKAGES
      • CALCULATING CHECKSUMS
      • SIGNING ARTIFACTS
      • DOCKER IMAGES
      • ARTIFACTORY
      • S3
      • BLOB
      • HOMEBREW TAP FORMULA
         • optimistically guessing `brew[0].installs`, double check
      • SCOOP MANIFEST
   • SNAPSHOTING
   • CHECKING ./DIST
      • --rm-dist is set, cleaning it up
   • WRITING EFFECTIVE CONFIG FILE
      • writing                   config=dist/config.yaml
   • GENERATING CHANGELOG
      • pipe skipped              error=not available for snapshots
   • BUILDING BINARIES
      • building                  binary=dist/kanctl_linux_amd64/kanctl
      • building                  binary=dist/kanctl_darwin_amd64/kanctl
      • building                  binary=dist/kando_linux_amd64/kando
      • building                  binary=dist/controller_linux_amd64/controller
   • ARCHIVES
      • creating                  archive=dist/kanister_0.22.0_darwin_amd64.tar.gz
      • creating                  archive=dist/kanister_0.22.0_linux_amd64.tar.gz
   • LINUX PACKAGES WITH NFPM
      • pipe skipped              error=no output formats configured
   • SNAPCRAFT PACKAGES
      • pipe skipped              error=no summary nor description were provided
   • CALCULATING CHECKSUMS
      • checksumming              file=kanister_0.22.0_linux_amd64.tar.gz
      • checksumming              file=kanister_0.22.0_darwin_amd64.tar.gz
   • SIGNING ARTIFACTS
      • pipe skipped              error=artifact signing is disabled
   • DOCKER IMAGES
      • building docker image     image=kanisterio/controller:0.22.0
      • building docker image     image=kanisterio/postgres-kanister-tools:0.22.0
      • building docker image     image=kanisterio/es-sidecar:0.22.0
      • building docker image     image=kanisterio/cassandra:0.22.0
      • building docker image     image=kanisterio/kanister-tools:0.22.0
      • building docker image     image=kanisterio/postgresql:0.22.0
      • building docker image     image=kanisterio/mongo-sidecar:0.22.0
      • building docker image     image=kanisterio/mysql-sidecar:0.22.0
      • building docker image     image=kanisterio/mongodb:0.22.0
      • pipe skipped              error=publishing is disabled
   • PUBLISHING
      • pipe skipped              error=publishing is disabled
   • release succeeded after 214.02s


@ray directive not working with augmentation enabled

I use the blade template engine and have spatie/laravel-ray installed on my statamic project. I can use @ray($var) in my view files and the output gets logged to the Ray app.
I turned on augmentation by replacing the Illuminate\View\ViewServiceProvider::class with Edalzell\Blade\Augmentation\AugmentationViewServiceProvider::class and now my browser prints the @ray($var) directive call.
Spatie registers the Ray blade directive in \Spatie\LaravelRay\RayServiceProvider::registerBladeDirectives and returns early if blade.compiler is not present in the app:
protected function registerBladeDirectives(): self
    {
        if (! $this->app->has('blade.compiler')) {
            return $this;
        }

        Blade::directive('ray', function ($expression) {
            return "<?php ray($expression); ?>";
        });

        return $this;
    }

The result of $this->app->has('blade.compiler') in  RayServiceProvider::registerBladeDirectives differs:

Enabled ServiceProvider
ServiceProvider::registerBladeCompiler called before RayServiceProvider::registerBladeDirectives
$this->app->has('blade.compiler')

Illuminate\View\ViewServiceProvider
true
true

Edalzell\Blade\Augmentation\AugmentationViewServiceProvider
false
false

Because the AugmentationViewServiceProvider is loaded after RayServiceProvider::registerBladeDirectives is called, the Ray directive gets not registered.
Is this something you can fix from your side?

In your composer.json you can disable the auto discover for Ray:
"extra": {
        "laravel": {
            "dont-discover": [
                "spatie/laravel-ray"
            ]
        }
    },

And manually add the \Spatie\LaravelRay\RayServiceProvider::class to the providers in config/app.php below the AugmentationServiceProvider.
With the auto discovery there's sadly no way to influence the order in which the service providers are loaded.

Thanks @riasvdv.
Try that @marcorieser.

@riasvdv thank you, i'll try that.


12.9.5 Document (as a vague note) exceptions when combining different delegate types

We also need to check whether it can happen on subtraction (13.9.6)

Closing as we have decided not to change the Standard.


update forge-std, remove now-included makeAccount

would love some help figuring out what's going wrong with the 1271 forge tests, since I'm importing seaport into seadrop I'm getting errors with makeAccount now being available in latest forge-std lib, i can confirm this PR fixes the downstream issue I'm facing

Codecov Report
Patch coverage has no change and project coverage change: +0.03 :tada:

Comparison is base (55d2dc6) 99.96% compared to head (d09a41b) 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #1221      +/-   ##
===========================================
+ Coverage   99.96%   100.00%   +0.03%
===========================================
  Files          47        21      -26
  Lines        2689      1264    -1425
  Branches      396       220     -176
===========================================
- Hits         2688      1264    -1424
+ Misses          1         0       -1

Flag
Coverage Δ

foundry
?

production
?

reference
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 36 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


The setTimeout method is invalid for StatefulRedisClusterConnection

Bug Report
The setTimeout method is invalid for StatefulRedisClusterConnection.
First we check the connectClusterAsync method in RedisClusterClient.java
        RedisChannelWriter writer = endpoint;

        if (CommandExpiryWriter.isSupported(clientOptions)) {
            writer = new CommandExpiryWriter(writer, clientOptions, clientResources);
        }

        ClusterDistributionChannelWriter clusterWriter = new ClusterDistributionChannelWriter(clientOptions, writer,
                clusterTopologyRefreshScheduler);

In this code, we can know that the real writer for StatefulRedisClusterConnection (generated by RedisClusterClient.connect()) is a ClusterDistributionChannelWriter
Then we check the setTimeout method for StatefulRedisClusterConnection, the implement of setTimeout method is in StatefulRedisClusterConnectionImpl.java,
public class StatefulRedisClusterConnectionImpl<K, V> extends RedisChannelHandler<K, V>
        implements StatefulRedisClusterConnection<K, V>

and we can find the setTimeout method in RedisChannelHandler.java
public void setTimeout(Duration timeout) {

        LettuceAssert.notNull(timeout, "Timeout duration must not be null");
        LettuceAssert.isTrue(!timeout.isNegative(), "Timeout duration must be greater or equal to zero");

        this.timeout = timeout;

        if (channelWriter instanceof CommandExpiryWriter) {
            ((CommandExpiryWriter) channelWriter).setTimeout(timeout);
        }
}

The timeout will be set for the channelWriter only if it is a CommandExpiryWriter. But according to our previous analysis, the channelWriter for StatefulRedisClusterConnection is a ClusterDistributionChannelWriter. So the setTimeout method won't work for RedisCluster
Environment

Lettuce version(s): This problem is in both 5.2.0.RELEASE & 5.3.2.RELEASE

Possible Solution
We can add additional judgement for ClusterDistributionChannelWriter in setTimeout method

Lettuce applied until version 5 timeouts only to the synchronous command API. Commands issued through the async and reactive APIs didn't time out. This behavior is rooted in the protocol definition. There's no way to cancel a running command with the Redis protocol.
In newer versions, we added timeout behavior on a global level to at least signal to callers that a command has timed out. Global timeouts are handled by CommandExpiryWriter and that's why the code is as it is now. With Lettuce 5.1, TimeoutOptions was introduced which enables global timeouts by default.
If you disable timeouts via TimeoutOptions, then setTimeout on the connection applies only to the synchronous command interface.

Closing due to lack of requested feedback. If you would like us to look at this issue, please feel free to leave a comment and we will re-open the issue.


[pure k8s] K8s's pod eviction may not trigger when job container uses too much disk space

K8s's pod eviction may not trigger when job container uses too much disk space.
According to k8s doc, there are two fs partitions that kubelet supports:

The nodefs filesystem that kubelet uses for volumes, daemon logs, etc.
The imagefs filesystem that container runtimes uses for storing images and container writable layers.

The imagefs limit is optional, so it may not configured by default. The result is if nodefs and imagefs are different, k8s may not evict pod that consumes large disk space.
Plus, if a pod is evicted by k8s, PAI treat it as system failure and retry without increase retry count. The job will always retry.

closed and tracked in #4694?


Suggestion about LRUQueryCache Optimization

Description
https://github.com/apache/lucene/blob/main/lucene/core/src/java/org/apache/lucene/search/LRUQueryCache.java#L747
Can we use ReentrantReadWriteLock in LRUQueryCache (Currently ReentrantLock)  to allow reading cache concurrently, to improve search performance？
I benchmarked this optimization by mocking some random LongPoint and querying them with PointInSetQuery with bool filter.

doc count
field cardinality
query point
baseline QPS
candidate QPS
diff percentage

30000000
10
1
2481
5102
105.6%

30000000
1000000
1
6396
6596.48
3.1%

I think this change can help filter queries that need to query low-cardinality fields. If it helps, I will submit a PR.

#13306

Fixed by #13306.


Feature request: Have the ability to get an Engine's configuration metadata/SourceFileOptions

It'd be great if a user can issue an API call to check an index bincode file's configuration metadata - i.e. when it was built, what languages are enabled, what Geonames URLs were used to build the index, etc. - ideally, without having to load the bincode file.

Hi!
May be length prefix serialization for index is a good choice (don't sure).
I see the next steps in putting this agreement into action.

Introduce trait IndexStorage

Store index in the next format - <meta length><meta><payload> for bincode to load only metadata by request.

Add struct IndexMeta with custom fields to allow users of the library set their own extra info to metadata

That'd be great @estin !
I just did a quick and dirty JSON file serialized from IndexUpdaterSettings for my project.
But your approach is better.
BTW, speaking of bincode, have you considered using rkyv serialization for kiddo? It seems to be multiple orders of magnitude faster than bincode - https://github.com/sdd/kiddo/blob/master/CHANGELOG.md#210

@jqnatividad Hi!

I just did a quick and dirty JSON file serialized from IndexUpdaterSettings for my project. That's why I tweaked geosuggest-utils and released 0.4.5 (hope you don't mind!.

That is why you are a collaborator. Don't wait; release all you need.

BTW, speaking of bincode, have you considered using rkyv serialization for kiddo? It seems to be multiple orders of magnitude faster than bincode - https://github.com/sdd/kiddo/blob/master/CHANGELOG.md#210

Don't look at. It would be nice to use more efficient serialization.

json useful for editing index, but it is very slow to dump/load index
bincode used only for speedup load index, especially on service startup (needs for production)

Now added
 -  trait IndexStorage as first step for custom serialization. It is breaking change.
 - index has metadata, and it can be read from the index file without loading the whole index.

And bincode serialization in the future may be enabled/disabled by feature flags.
Check https://github.com/estin/geosuggest/blob/master/geosuggest-core/src/storage.rs


Data provider detail

This PR contains:

layout for data providers details page
Load first 10 outputs
search box (without functionality)
map

Missing parts (part of following PRs):

pagination
search working
...

@viktor-yakubiv I decided to leave map interactive because I find it quite useful but if you insist I can make it non-interactive. Let me know. Also, it's quite a big PR and everything is in one commit. Sorry about that. I was doing too many changes and lost track of it. I don't expect you to go through the code rather concentrate more on look.

I decided to leave map interactive because I find it quite useful but if you insist I can make it non-interactive.

☝ This is good. I agree that the interactive map is better.

Also, it's quite a big PR and everything is in one commit. Sorry about that. I was doing too many changes and lost track of it.

☝ This is fine as well, it happens. Sometimes you make changes that touch a lot of stuff, sometimes you do a big feature like this one.

Could you please remove the search field and pagination so this PR looks like finished work and bring it back in the following PRs?

Could you please remove the search field and pagination so this PR looks like finished work and bring it back in the following PRs?

I removed pagination but left search because I think it's fine. It's part of the layout and this PR is going to be merged to a different branch than master so it's fine to have some work in progress there.

Could you please describe your work in the commit message more precisely? What components are created? How are they used? What libraries were added and so on?

I am not much a wordy guy. I understand your point but on the other hand, I don't think this is something which is stopping us at the moment. It's up to everyone how they handle commits/messages. 🙂


Failed to parse PDF...

The pdf file was uploaded successfully, and a failure message was displayed during parsing ：
Failed to parse PDF...  Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused'))

@j1029441326

How did you reproduce it? Why do I always feel like this code is incomplete? Is that all?

The pdf file was uploaded successfully, and a failure message was displayed during parsing ：
Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused'))

你好，你这个报错解决了吗？我也遇到了相同的报错。

你觉得他的代码完整吗
---- Replied Message ----
| From | @.> |
| Date | 03/25/2024 12:15 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) |
The pdf file was uploaded successfully, and a failure message was displayed during parsing ：
Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused'))
你好，你这个报错解决了吗？我也遇到了相同的报错。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

大神您好， 你那个错就是gradio这个包版本问题。或者是需要连接外网。
我的问题你回答下哦，现在这个程序就只有这些了吗？他有没有删除？
---- Replied Message ----
| From |<EMAIL_ADDRESS>|
| Date | 3/25/2024 12:22 |
| To | @.***> |
| Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) |
你觉得他的代码完整吗
---- Replied Message ----
| From | @.> |
| Date | 03/25/2024 12:15 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) |
The pdf file was uploaded successfully, and a failure message was displayed during parsing ：
Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused'))
你好，你这个报错解决了吗？我也遇到了相同的报错。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

大神您好， 你那个错就是gradio这个包版本问题。或者是需要连接外网。 我的问题你回答下哦，现在这个程序就只有这些了吗？他有没有删除？ ---- Replied Message ---- | From |<EMAIL_ADDRESS>| | Date | 3/25/2024 12:22 | | To | @.> | | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | 你觉得他的代码完整吗
…
---- Replied Message ---- | From | @.> | | Date | 03/25/2024 12:15 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | The pdf file was uploaded successfully, and a failure message was displayed during parsing ： Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused')) 你好，你这个报错解决了吗？我也遇到了相同的报错。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.>

你好，后来发现了确实是代理问题，已经解决了。我只需要用到它这里对PDF的解析，故没有发现有不完整的地方，是你遇到什么报错了吗？为什么会说不完整。

您看我总觉得这个代码就这几行就能运行吗？总觉得他没给完整。如果您运行成功了，我就去下载试试，最近项目缺这个
---- Replied Message ----
| From | @.> |
| Date | 3/25/2024 15:05 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) |
大神您好， 你那个错就是gradio这个包版本问题。或者是需要连接外网。 我的问题你回答下哦，现在这个程序就只有这些了吗？他有没有删除？ ---- Replied Message ---- | From |<EMAIL_ADDRESS>| | Date | 3/25/2024 12:22 | | To | @.> | | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | 你觉得他的代码完整吗
…
---- Replied Message ---- | From | @.> | | Date | 03/25/2024 12:15 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | The pdf file was uploaded successfully, and a failure message was displayed during parsing ： Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused')) 你好，你这个报错解决了吗？我也遇到了相同的报错。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.>
你好，后来发现了确实是代理问题，已经解决了。我只需要用到它这里对PDF的解析，故没有发现有不完整的地方，是你遇到什么报错了吗？为什么会说不完整。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

您看我总觉得这个代码就这几行就能运行吗？总觉得他没给完整。如果您运行成功了，我就去下载试试，最近项目缺这个 ---- Replied Message ---- | From | @.> | | Date | 3/25/2024 15:05 | | To | @.> | | Cc | @.> , @.> | | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | 大神您好， 你那个错就是gradio这个包版本问题。或者是需要连接外网。 我的问题你回答下哦，现在这个程序就只有这些了吗？他有没有删除？ ---- Replied Message ---- | From |<EMAIL_ADDRESS>| | Date | 3/25/2024 12:22 | | To | @.> | | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | 你觉得他的代码完整吗 … ---- Replied Message ---- | From | @.> | | Date | 03/25/2024 12:15 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [Weixin-Liang/LLM-scientific-feedback] Failed to parse PDF... (Issue #29) | The pdf file was uploaded successfully, and a failure message was displayed during parsing ： Failed to parse PDF... Error: HTTPConnectionPool(host='localhost', port=8080): Max retries exceeded with url: /api/convert (Caused by newConnectionError('<urllib3.connection. HTTPConnection object at 0x7f7b15f53760>: Failed to establish a new connection: [Errno 111] Connection refused')) 你好，你这个报错解决了吗？我也遇到了相同的报错。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.> 你好，后来发现了确实是代理问题，已经解决了。我只需要用到它这里对PDF的解析，故没有发现有不完整的地方，是你遇到什么报错了吗？为什么会说不完整。 — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

是的，因为本身功能亚不多，一个是对PDF的解析，一个是用大模型进行回答，用的openai的，所以代码不多。


History Space

have point imported to skypark via nhaystack driver.  Have duplicate niagara history id but unique haystack history.
haystack history
H.PHL7_NC_01.PHL7_EF_01_exhaustFan
C.Drivers.NiagaraNetwork.PHL7_NC_01.points.EF.EF_01.exhaustFanStatus
niagara history id
/PHL7_NC_01/PHL7_EF_01_exhaustFanStatus
Where is the H. type located and how do I remove from the import to skyspark?

Hi @killorb, thanks for reaching out.
I'm a little bit confused but I'd like to try and help.  Can you clarify something for me please?  Is the duplicate history ID in Niagara or in SkySpark?


Ensure html_root_url is kept in sync

This adds two new unit tests: one that checks that the html_root_url
attribute always uses the current crate version number, and one that
checks that the TOML code blocks in the README use the correct version
number.
This is implemented by my version-sync crate -- I hope it's useful!
Fixes #613.

This might be more complicated than I hoped since version-sync doesn't support Rust 1.14 (version 1.15 is the oldest it can be compiled on).

Please let me know if there's any interest in the general approach — then I can see about removing the version-sync dependency on versions and targets where it doesn't work (remove with sed in the Travis build).

Thanks for the PR.
I'd rather not add another dependency and instead deal with this by having a RELEASE doc and wait for this to get fixed up stream. But, again, thanks for the attempt :).

As an alternative, I've used
before_install: |
  version=$(grep "^version =" Cargo.toml | sed -r  's/^[^0-9]*([0-9]+.[0-9]+.[0-9]+).*/\1/') &&
  docs_version=$(grep -R 'docs\.rs' src | head -n 1 | sed -r 's/^[^0-9]*([0-9]+.[0-9]+.[0-9]+).*/\1/') &&
  test "$version" == "$docs_version"

in my .travis.yml in the past.

@carllerche Sure, no problem. Note that it's only a new dev-dependency, so only people who run the MIO tests are affected by this.
I'll see about making another PR that at least fixes the outdated version number :-)

@twmb I think a smoke test like that would be good.
@mgeisler Sure re: PR that fixes the version :)

I've submitted #760 to fix the version number.


SIMD optimized masking on x86_64

@littledivy, Let me know if there are any updates to this feature.

the compiler does a good job at auto vectorizing the new implementation. See https://godbolt.org/z/EPTYo5jK8
This works with --target-cpu=native --opt-level=3


Images loading are causing scroll height to be off.

Is there a solve for this common issue?
Seems like by default the scroll height is calculated before all the images are loaded with is causing the scroll to be off, sometimes by alot.

You are right, one future solution should be to calculate it on scroll.
You can use DirectLink with element id to scroll. See the readme file :)

I've got the same problem, only it manifest in a different way. Scrolling does not make a class active at the right time.  What can I do to get correct rectangle calculation and thus correct active class activation?
Thanks

I think this PR will solve your issues, I just haven't had time to validate it yet.
https://github.com/fisshy/react-scroll/pull/115

@fisshy I just added a very simple pull request that fixes my issue here https://github.com/fisshy/react-scroll/pull/126. It's just a few lines of code you can review it ultra  fast, its very straight forward.

Actual problem

Unable to verify now, as we've moved on.
Maybe someone else can?

This is still an issue... Anyone has a solution?


health#report_start deprecation warning

On nightly neovim, there is currently a deprecation warning for the following health related functions.
- *health#report_error* *vim.health.report_error()*	Use |vim.health.error()| instead.
- *health#report_info* *vim.health.report_info()*	Use |vim.health.info()| instead.
- *health#report_ok* *vim.health.report_ok()*		Use |vim.health.ok()| instead.
- *health#report_start* *vim.health.report_start()*	Use |vim.health.start()| instead.
- *health#report_warn* *vim.health.report_warn()*	Use |vim.health.warn()| instead.

They will be removed in Nvim version 0.11.
Not a huge deal but I thought I'd mentioned that this can be avoided by using the following in your health.lua (I guess .tl in this case).
local health = vim.health or require "health"
local health_start = health.start or health.report_start
local health_ok = health.ok or health.report_ok
local health_warn = health.warn or health.report_warn
local health_error = health.error or health.report_error

This will keep backwards compatibility with Nvim 0.9 while avoiding the deprecation warning & eventual deprecation.

Seems like a good idea, PR welcome


Issue on running StereoCell

Hi,
I am running StereoCell for performing cell segmentation using the script cell_bin.py and providing the image file and the matrix file, but on running the script i get an error: tifffile.tifffile.TiffFileError: not a TIFF file b'\x89HDF'
I checked the image file and it's in .tif format. Could you please help me solve this issue?
Thank you

Is your tif file big? If it is convenient, please provide one for me to confirm the specific problem, thank you.


Env Var area in the 2.5 (/g) UI, in Rancher 2.6, does NOT work

Internal reference: SURE-5520
Reported in 2.6.8
Admin reports that they have hundred of users who use Rancher daily, and in order to properly 'transition' those users to the 2.6 GUI, they want to bridge the transition, allowing users to use /g to get to the old UI in 2.6. They had been advised that "rancher.fqdn.com/g" is available to use, but will break in random ways as we're 'disconnecting' the old UI for the new one. However, there is a bug, in that the Env Var area in the 2.5 (/g) UI, in Rancher 2.6, does NOT work and fails with errors.
Repro steps:

Browse to "rancherurl.com/g" to access the 2.5 UI Cluster
Workloads -> Deploy
Try to deploy anything with an Env var set (foo:bar) Error: "Validation failed in API: field environment is deprecated

There's an old GitHub for this issue: https://github.com/rancher/rancher/issues/38239

I don't think we should address this. They have been incorrectly told /g is available to use (we need to understand who has told them this). The old UI will break in random ways as it's no longer supported. In order for it to not break in random ways we would need to support it.


Handle ocaml stublibs properly

Is your feature request related to a problem? Please describe.
Currently, opam-nix installs ocaml stublibs to $OCAMLFIND_DESTDIR/${pname} together with the rest of the libraries.
However,  ocaml compiler expects them to be present in $OCAMLFIND_DESTDIR/stublibs directory.
This is the case for e.g. zarith library.
Describe the solution you'd like
The following postInstall workaround seems to work:
postInstall = ''
    mkdir -p "$OCAMLFIND_DESTDIR/stublibs"
    ln -s "$OCAMLFIND_DESTDIR"/${o.pname}/dll*.so "$OCAMLFIND_DESTDIR"/stublibs/
  '';

Ideally, opam-nix should handle this automatically. Maybe there is a way to specify a place to looks for stublibs.

opam-nix just calls make install in case of zarith, which then calls ocamlfind install. That doens't seem to install stuff correctly for some reason.

Ok, I see. ocamlfind install will only install DLLs to stublibs if that directory exists.

Thanks!


Bug on latest firefox (56.0)

Problem with flipInX and flipInY animations (maybe more, i tested their only).
Not show animation (animated block hides from page), when no mouse move and no keyboard press button (New firefox throttling?).
When mouse moving it works correct.
jsfiddle
Animations without opacity works correct.
Tested on the 2 PC (Windows 10, Firefox 56 not working, Chrome 61 works, Opera 48 works).
Update:
Fade (fade, fadeInDown and etc.) effects with opacity works great. At the moment the problem  only with flipInX and flipInY animations.

Thanks for reporting, very interesting bug.
I double check the code of animatelo.js and at a first look I do not see any issues there.
This problem exists when using setInterval and requestAnimationFrame when trying to create an animation loop wrapping animatelo (Which under the hood use Wen Animation API).
IMO it is specific of the newest release of FF 56 and possibly related to Web Animation API and the event loop (we should report to the Mozilla team).
By the way... the correct way to control an animation in animatelo.js is to use its JS api.
Live:
https://jsfiddle.net/cd86mb4v/
API Example:
  window.animatelo.flipInX('#animate', {
    iterations: Infinity,
    duration: 2000,
    direction: 'normal',
    fill: 'forwards',
  });

You can use iterations: Infinity to get rid off setInterval and execute correctly your animation.
In this way you bypass the FF issue using properly the API.
I hope it helped.
Thanks.

Firefox fix it on 57.0 release (works for me)!


Short solution needed: "How to set background color" (php-gd)

Please help us write most modern and shortest code solution for this issue:
How to set background color (technology: php-gd)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to set background color


split karate-ui into new maven / JAR module moving javafx out of karate-core

definitely causing projects to break that use spring / spring boot:
https://stackoverflow.com/q/56193195/143475
https://stackoverflow.com/a/56059478/143475
this will be a breaking change :| better to do this sooner instead of waiting for 1.0.0

here is my comments from an issue I ran into
https://stackoverflow.com/questions/8861485/contextload-error-in-spring

@cpressler can you try now with 0.9.3.RC2 ?

0.9.3.RC2   now works for the minimal spring boot test contextLoads().  Thanks!
import org.junit.Test;
import org.junit.runner.RunWith;
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.test.context.junit4.SpringRunner;
@RunWith(SpringRunner.class)
@SpringBootTest
public class JugToursApplicationTests {
@Test
public void contextLoads() {
}
}

FYI adding this dependency now seems to cause a failure but I do not need it so I am good for now.

com.intuit.karate
karate-ui
${karate.version}
test

@cpressler you can always use a maven profile to use it in dev mode but not mess up your CI

released 0.9.3


Added SplashScreen for Android

Fixes #23

Please let me know if anything else is needed.

I tired it... It makes the app experience a little slow and I feel like it's not needed... So I'm marking this changes as invalid and closing the PR.. But If You have some other ideas feel free to share.. Till then this PR is open @HimeshNayak

I tired it... It makes the app experience a little slow and I feel like it's not needed... So I'm marking this changes as invalid and closing the PR.. But If You have some other ideas feel free to share.. Till then this PR is open @HimeshNayak

The splash screen the white screen that comes before the app is loaded. So, I don't it would make the app slow. Maybe the graphic was heavy to load and thus it is taking some time. We could replace it with the app icon and I guess that would solve the problem.
@avinashkranjan

The splash screen the white screen that comes before the app is loaded. So, I don't it would make the app slow. Maybe the graphic was heavy to load and thus it is taking some time. We could replace it with the app icon and I guess that would solve the problem.
@avinashkranjan

That would be much better...plain black page with only the logo..

That would be much better...plain black page with only the logo..

That would be much better...plain black page with only the logo..

Yup.... will do that and send the PR.

That would be much better...plain black page with only the logo..

Yup.... will do that and send the PR.

Done. Have a look @avinashkranjan

Done. Have a look @avinashkranjan


TOCSS error

Error building site: TOCSS: failed to transform "scss/style.scss" (text/x-scss): resource "scss/scss/style.scss_b95b077eb505d5c0aff8055eaced30ad" not found in file cache

@Tanoh-Hermann, did you find a solution for this... having the same problem.

Which version of hugo you are using?

@Somrat37
Hugo version v0.70.0-DEV
I was able to start the server by running it from within the exampleSite ( Reference issue #3 . ) Running it from the top level directory results in the given error.

please download and install the hugo extended version

Did it solve your issue? I am going to close this issue, if you have any other problem then create a new issue. Thanks

@Somrat37
I had the same error and by installing the "hugo extended version" i have solve the issue.
Thank you very much.

@Somrat37
I had the same error and by installing the "hugo extended version" i have solve the issue.
Thank you very much.

May I ask you to tell me about your solution to this problem.

@Somrat37 and @sfid98 , installing the extended version did fix the problem.
@Slevin2017 they are referring to here. Note the line to get the extended version => go install --tags extended

Thank you very much! It seems to have the problem with gcc.)

@Slevin2017 , I assume you using windows machine with something like cygwin? If so (and even if you are using something else and gcc is failing) there is an fairly easy way to do this using a Docker container and mount your project directory to it.
You can use this docker image hugo-extended and write a docker-compose.yaml file similar to this...
version: "3.7"
services:
    web:
      image: trenton/hugo-extended:latest
      ports:
        - 1313:1313
      restart: unless-stopped
      volumes:
        - ./your_project_directory:/site
      command: hugo server -D --bind <IP_ADDRESS>

Then just start it with docker-compose up and make changes and see them update just like you would if you ran the hugo command on your local machine.

すげぇ！「extended version」ってやつか。それに置き換えたらホントにできた！ありがとうございました！


Report unnecessary exception tags as errors

Per https://knip.dev/reference/jsdoc-tsdoc-tags, it's possible to use arbitrary and standard tags to prevent knip from reporting an error for a symbol.
As I review other issues and comments, I see concerns regarding this usage of bypassing knip and generally recommending against it. Hopefully, what I am suggesting here will align with that concern.
There are situations where I am currently seeing knip errors reported in our repo and its based on partially completed code where the full solution is still a work in process. Basically, these errors are correctly reported, but there remains more work to be done to start consuming those symbols. This doesn't align perfectly with any tags, but @beta is probably the closest. This type of error would make it difficult to enable knip as a CI gate so encouraging the use of a tag would be appropriate here. However, that could lead to the opposite problem -- unnecessary tags remaining in the codebase after the issue has been rectified.
What I'd like to see is a mechanism where the use of an ignored tag could itself be reported as an error in knip when the tag itself is no longer necessary. This would align with prior art such as:

@ts-expect-error - If the directive is unnecessary then the directive is an error
eslint report-unused-disable-directives - Will be reported as an error by eslint if the directive is no longer necessary (and is autofixable!)

It's also possible that this could be taken further to discourage the use of tags for lengthy periods of time. There are many variants of this type of eslint plugin, but eslint-plugin-unicorn has a rule expiring-todo-comments which trigger errors if a todo has gone beyond its designated timeframe. That concept lines up well with the scenario I'm describing where additional development is necessary to consume the symbols that knip is reporting as errors. Something like @knipignore - 2024-08-01 would imply that the tag will be excluded from being reported as an error until that date.
This package is really excellent work! Thanks!

Absolutely, thanks for bringing this up.
Basically, we would need to move this line down to this block and add an issue to the issue collector, skip the export to ignore, or add an item to the configuration hints.
There will be a small performance penalty, but I'd say it's def worth it.
One could argue it should be an error (non-zero exit code) and not just a configuration hint (greyed out and does not affect exit code), but perhaps it's a good first step.
I'd need to read up a bit more on the extensions you're describing, but it sounds interesting.

:rocket: This issue has been resolved in v5.23.0. See Release 5.23.0 for release notes.
Using Knip in a commercial project? Please consider becoming a sponsor.

Hey @webpro -- thanks for jumping on this so quickly!
I'm trying to test it out right now and I'm not seeing the configuration hints as I expected.
I found various symbols that were being consumed elsewhere and I added a jsdoc tag of @knipignore to them and then ran knip with --tags=-knipignore. When running, there is no output on the reporter for those symbols that are being used but ignored.
I debugged through and when it gets to the reporter the tagHints set is empty, so the problem is not in reporting but rather in collection.
I debugged further up the chain in index.ts and set a conditional breakpoint to break when isIgnored is true. That triggered as expected for the symbol that had the tag. However, at the if (!isIgnored && importsForExport) { line, it skips everything in that block because isIgnored is true. The logic to add the tag hint is nested in that block, so it's being bypassed completely.
This isn't working as I expected so I suspect one of the following:

I'm not running something correctly
Our wires got crossed up and what I was suggesting and what you thought I was suggesting weren't the same
The code isn't working as expected

I can create a reproduction if that helps, so let me know. Thanks!

I'm going to pick nr 3!  I was testing this with another repo and basically two bugs resulted in output looking correct in that repo. Because it's quite the repo and in my enthusiasm I didn't even add tests, etc.
Your debug session was good, but I think the cause was slightly different (there was another collector.addTagHint down below; which seems to work because of some +internal,-knipignore miracle).


Android Basics: ViewModel

URL of codelab
Store Data in ViewModel
In which task and step of the codelab can this issue be found?

Starter app overview

Describe the problem
The starter code is not really the starter code. It's the final code with all the changes already added.
Steps to reproduce?
1 - Go to step 2 of the code lab here
2 - Follow the instructions to clone the GH repo for the starter code
3 - Note the app already has all the files and all the final code added

If you imported the project from Version Control, by default the branch is main, you can change it to starter at the bottom right of Android Studio.


docs: add ADR-7

Fixes no issue
Description

adds ADR-7 to the docs page
minor change without any implications

Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in values.yaml and Chart.yaml (if necessary)

Codecov Report

Merging #406 (031b89c) into develop (cf8cc75) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #406   +/-   ##
========================================
  Coverage    94.24%   94.24%
========================================
  Files           22       22
  Lines         1095     1095
========================================
  Hits          1032     1032
  Misses          63       63

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 cf8cc75...031b89c. Read the comment docs.


Use boost regex instead of std, because of buggy glibc++ in RHEL7

On Red Hat Enterprise Linux 7 / CentOS 7, Simple-Web-Server does not work because of buggy regex implementation in libstdc++. This patch fixes the issue by using boost's regex implementation.

What version of gcc-g++ do you have in RHEL7?

Or ,I guess it's >= 4.9 since you would not get CMakeLists.txt to work otherwise. I'll merge this Tomorrow after running tests on other platforms.


Type construction

type car (
  name: str,
  price: num
)

TODO

In nim, ref means auto-counting, else we will get copy or the same object.
And if a proc can change object's properties, we must use var instead of let.
This two reason cause troubling translation.
We need some time to solve it.
I think the best way is translate let else let mut both are var.
So we can change the value. But let still can't be assign, but this time is elz compiler avoid it.

Done. No example. TODO: create a example for it

Complete at v0.3.7


[feature proposal] Single command to match width and height of model element

Currently, there are menu items under Menu Bar > View > Position for:

Match Width
Match Height

It would be convenient to have a single command to match both width and height atomically. It would also be convenient to have a keyboard shortcut or similar UI element to make this faster to invoke.

Hi Ben.
Hmm...too lazy to click twice, eh? ;-)
OK, noted.
Phil

Haha. You found me out. Although after submitting this, I realized there is a preference for default diagram element width and height, which makes this less of an issue for me.

I just added a keyboard shortcut for "Default Size" for the next version, if this would help. The only question is, which key combo? Ctrl-D is already used for "Duplicate" in the tree. I went with Ctrl and / for now.

Er.. Ctrl and / are difficult for most non english keyboard as / is obtained with a shift, leading to a Ctrl + shift + /
Do we use Ctrl + R for something? This could be seen as a "reset" for size...

OK, but what about people who have no "R" in their language? For example, Jason and the R-go-naughts?

Match Size will be in Archi 4.9.


Feature: have an option to disable the plugin during custom training

I'd like to be able bind a button I use for the plugin in training mode to do something different during being in a custom training pack.
LOVE THE PLUGIN!!!
Thanks
Mayyyyyy or may not look into patching this in later (since I bet it could just be enabled/disabled with a global) depending on stuff.

Seems pretty simple.  It can be a flag just like the other toggleable settings.  Just out of curiosity, what else do you use the button for in custom training?  Or do you have cpt_freeze bound to a more "standard" button than right thumb click?

I added a setting for this in the button configuration section.  Feel free to test it out in this release: https://github.com/NitrOP7674/FreeplayCheckpoint/releases/download/v1.7.0-pre/CheckpointPlugin-v1.7.0-pre.zip
There are no other new features so I probably won't push this to bakkesplugins.com at this point.

Seems pretty simple. It can be a flag just like the other toggleable settings. Just out of curiosity, what else do you use the button for in custom training? Or do you have cpt_freeze bound to a more "standard" button than right thumb click?

Since they added the whole select shot / reset shot, I wanted to reuse a general button that I already had for cpt freeze checkpoint. I didn't try the right thumbstick for cpt_freeze, but yeah I would just like to have had the plugin disabled in custom training 😄 .
Thanks for the update!


Report available resources separately for each priority class

Currently, the executor (i.e., the in-cluster component of Armada) only reports free resources to the Armada server (the central component of Armada responsible for assigning jobs to executors). Available resources are reported on a per-node basis. To support preemption, the executor must report separately free resources and resources allocated to preemptible jobs.
More specifically, the executor must report total resources available and for each priority class the resources assigned to jobs of that priority class.

Hey team! Please add your planning poker estimate with ZenHub @dejanzele @jayofdoom @kannon92 @richscott

Closing as duplicate


Dead link to report security bugs.

Describe the bug
The link to report a security issue at the bottom of the page is broken.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.microsoft.com/en-us/previous-versions/tn-archive/cc751383(v=technet.10)'
Click on the link that  reads 'report it'. The link address currently is: http://www.microsoft.com/security/msrc/report.aspx
See error

Expected behavior
Link should return the correct Microsoft Security Response Center site:
https://www.microsoft.com/en-us/msrc
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Mac OS 10
Browser Chrome (75.0.3770.80), Firefox (67.0.1 (64-bit)), Brave (Version 0.65.118 Chromium: 75.0.3770.80 (Official Build) (64-bit))

Looks like I was directed to this (old) page from a search result. I just noticed that it was an old page. Thanks.


[hotfix][docs] Remove the OverrideDefault annotation for option slotmanager.number-of-slots.min

What is the purpose of the change
Remove the OverrideDefault annotation for option slotmanager.number-of-slots.min.
Brief change log
Remove the OverrideDefault annotation for option slotmanager.number-of-slots.min.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

Hi @KarmaGYZ , can you help review this when you have time?


MC server time

Hi,
I have encountered a problem with the server time. Is there a way to make it follow the system time?
I have used the following line in a few other containers and it seems to be working in them but not on this one.
-v /etc/localtime:/etc/localtime:ro
Is there any way to fix this?

...oh, it's regarding the timezone? I thought you meant the clock itself gets out of sync from the host. It looks like I might need to be adding the tzdata package for it to work as you expect:
https://wiki.alpinelinux.org/wiki/Setting_the_timezone

This fix is now pushed to Docker Hub. Let me know if this still doesn't resolve the issue for you.

Yeah, when testing the before and after I confirmed it was using UTC previously.
Yes, the container has to be recreated unfortunately to pick up a new image; however, if you have /data and such attached via volumes then it is safe to recreate the container.

Ok cool.
Pulled the new image and I'm restarting the server atm. The time in the console seems to still be 2 hours late :o should it be fixed or do I need to add another parameter or something?

Hmm, no, the mounting of /etc/localtime like you had before should do it.

To sanity test, can you try
docker pull itzgminecraft-server
docker run --rm -v /etc/localtime:/etc/localtime --entrypoint date itzg/minecraft-server

compared to
docker run --rm --entrypoint date itzg/minecraft-server

I ran the suggested lines and the first did return the correct date and time. but the latter and  when I create the server container, the time seems to be still off. The command I use is as follows:

docker run -d -v /share/CACHEDEV1_DATA/Container/container_files/Minecraft:/data
-v /etc/localtime:/etc/localtime:ro
-e TYPE=FTB
-e FTB_SERVER_MOD=https://www.feed-the-beast.com/projects/ftb-infinity-evolved/files/2484486
-e MEMORY=2G
-p 25565:25565 -e EULA=TRUE --name mc itzg/minecraft-server

Thanks for running the sanity test commands -- your system definitely has and is running the updated image as I'd hope. It's then strange the real FTB container is not honoring the timezone setting for you. I ran a stripped down version of your command
docker run -it --rm \
    -v /etc/localtime:/etc/localtime:ro \
    -e TYPE=FTB \
    -e FTB_SERVER_MOD=https://www.feed-the-beast.com/projects/ftb-infinity-evolved/files/2484486 \
    -p 25565:25565 -e EULA=TRUE itzg/minecraft-server

and it did properly convey the local timezone for me at least:

Yeah, really strange. I'll try to create the container again and see if it works then.
My time zone is gmt+2 so I have no idea why it won't respect that although other containers do.

Yeah, really strange. I'll try to create the container again and see if it works then.
My time zone is gmt+2

In case someone is still looking for a solution to this issue:
The server will use the correct timezone if it is explicitly set for the JVM.
So for me this is: -Duser.timezone="Europe/Berlin",
which can be set in this image with --env JVM_DD_OPTS=user.timezone="Europe/Berlin"


[dev] handle multiple value headers

I'm unable to get wrangler dev to return multiple Set-Cookie headers in a worker response. I've tried in both of the following ways:
explicitly appending them
// not working, only last gets sent (`cookie2`)

resp.headers.append('Set-Cookie', 'cookie1=true')
resp.headers.append('Set-Cookie', 'cookie2=true')

passing them in the options object of the response
// not working, only last gets sent (`cookie2`)

headers: {
   'Set-Cookie': 'cookie1=true',
   'Set-Cookie': 'cookie2=true'
}

I've so far only been trying to get this to work using wrangler dev... perhaps it's a wider Workers issue/limitation? Perhaps I'm missing something obvious here?
Originally posted by @yet-another-dev in https://github.com/cloudflare/wrangler/issues/1047#issuecomment-649705103

The issue here is that we do not properly handle response headers that have multiple values. We iterate over them all and end up overwriting each response header with the following one, which is why @yet-another-dev only sees the last cookie header in the response

further update - it looks like this is only a problem for Set-Cookie specifically. Other headers with multiple values are coalesed by the Workers Runtime into a single header field. If the key was my-header instead of Set-Cookie, it would look like my-header: value1, value2 and it wouldn't be a problem for wrangler dev to handle, since there will only ever be one my-header.
But Set-Cookie is special! From RFC 6265:

"Origin servers SHOULD NOT fold multiple Set-Cookie header fields into a single header field.  The usual mechanism for folding HTTP headers fields (i.e., as defined in [RFC2616]) might change the semantics of the Set-Cookie header field because the %x2C (",") character is used by Set-Cookie in a way that conflicts with such folding."

This means that Set-Cookie headers will be returned individually by the Workers Runtime, making wrangler dev's previous "insert not append" approach broken.

I'm unable to get wrangler dev to return multiple Set-Cookie headers in a worker response. I've tried in both of the following ways:
explicitly appending them
// not working, only last gets sent (`cookie2`)

resp.headers.append('Set-Cookie', 'cookie1=true')
resp.headers.append('Set-Cookie', 'cookie2=true')

passing them in the options object of the response
// not working, only last gets sent (`cookie2`)

headers: {
   'Set-Cookie': 'cookie1=true',
   'Set-Cookie': 'cookie2=true'
}

I've so far only been trying to get this to work using wrangler dev... perhaps it's a wider Workers issue/limitation? Perhaps I'm missing something obvious here?
Originally posted by @yet-another-dev in #1047 (comment)
The issue here is that we do not properly handle response headers that have multiple values. We iterate over them all and end up overwriting each response header with the following one, which is why @yet-another-dev only sees the last cookie header in the response

Is this a problem again? I'm using the  headers: {    'Set-Cookie': 'cookie1=true',    'Set-Cookie': 'cookie2=true' }
approach and It's only getting cookie2! Quite frustrating!

UPDATE: When trying the resp.headers.apend(); method it works flawlessly so I assume this is a regression? I'm going to open a new issue.


Can't import dashboard in grafana

Hi,
I have this problem with importing a dashboard ( Id=5249 ) into my grafana.  I'm using grafana 6.0.0, Prometheus 2.8.0 and Prometheus Exporter for Jira 1.0.21. All correctly installed. While importing json file or entering ID/url, I'm getting error "Unexpected token u in JSON at position 0".
Prometheus is added as "Data Source" and he is listening on port 9090.
Prometheus.yml contains following lines:
scrape_configs:

job_name: 'jira'
scheme: https # change to http if don't you have https
metrics_path: '/jira/plugins/servlet/prometheus/metrics'
static_configs:

targets: ['myhost:2990'] # Jira host and port you serve
as recommended.
What could be a problem?

Hi. Did you use some secter token?

Hi,
no I don't use secret token.

There are multiple steps between installing the prometheus plugin and loading a dashboard into grafana.

have you confirmed that you can view the prometheus metrics at the end point in your JIRA instance?
have you confirmed that you can curl that URL remotely from your prometheus server to rule out networks and firewalls?
have you loaded the jira endpoint into prometheus as a data source (your snippet above)?
have you confirmed that the target is green in prometheus?
have you confirmed that you can query the metrics are query-able in prometheus?
--- only at that point
can you query the metrics into grafana via the prometheus data source?
can you import a pre-saved dashboard and replace the varibles?

I hate to say it, but only step 1 is part of this project.


Can you run this bot in our telegram community?

Bot should be added with some admin right, cause use callbacks
I will prepare instructions with screenshot how to add bot to group and manage it soon

How to add bot to the group https://www.youtube.com/watch?v=UIsF8ki4ewM
bot https://t.me/shfinancebot
Also you can manage notifications in Profile menu (Default On)
On - bot sends notifications to the group
Off -  bot sends notifications to private chat
All notifications from bot will go directly to public chat, instead of bot private chat

I prepared instruction how to add a telegram bot in the group, you can add the bot  as a member without ant privileges (no need admin privileges)
I tried to add the bot to group, but it doesn't work, maybe I have not enough permissions as a group member
How to add bot to the group https://www.youtube.com/watch?v=UIsF8ki4ewM
bot https://t.me/shfinancebot


chores: move role_playing_with_functions from utils into example

Description
move role_playing_with_functions from utils into example
Motivation and Context
Why is this change required? What problem does it solve?

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of example)

Implemented Tasks

[ ] Subtask 1
[ ] Subtask 2
[ ] Subtask 3

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ ] I have read the CONTRIBUTION guide. (required)
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly. (required for a bug fix or a new feature)
[ ] I have updated the documentation accordingly.

I've reviewed all the code in this PR, and the changes are quite minimal, mainly involving some reorganization of call paths. I believe there are essentially no problems.
There's just one point I might need your clarification on. If I remember correctly, role_playing_with_functions was initially in the example directory. Later, the developers of Camel moved it to utils, and now it's been moved back. Could you explain the reason for this?

Hey @yiyiyi0817 , thanks for your review, after discussion with Guohao we decided to move this back, refer to https://github.com/camel-ai/camel/pull/472#discussion_r1593930095


Network usage is prohibited for themes

It appears that your theme uses network connections to load assets (e.g. fonts, icons, or images). This is prohibited by the official Obsidian developer policies because themes should function completely locally.
You can bundle an asset for local use by using data URLs. See this guide.
Please let us know if you have any questions. Any themes that use network connections will be removed from the official directory in the first week of January 2024.

The Obsidian team.

I just edited the theme file to remove the three @import statements I had in the file. This should resolve the issue; please let me know if it hasn't so I can locate any other problematic areas.


[Bug Fix]: if there are slashes in the API Namespace name, the generated documentation is wrong

Addresses: https://github.com/restarone/violet_rails/issues/1018

@alis-khadka good fix, can you add some more test coverage to this page? I have noted the places we need to make assertions

@alis-khadka good fix, can you add some more test coverage to this page? I have noted the places we need to make assertions

I have added the test cases.

@alis-khadka could you fix the merge conflicts on this branch?

@alis-khadka could you fix the merge conflicts on this branch?

@donrestarone I have resolved the merge conflicts.

@alis-khadka more conflicts, just merged your API access by category

@alis-khadka more conflicts, just merged your API access by category

Conflicts has been resoved.


How auto run Javascript on devtools at startup?

Hi @bpasero @dilijev :sweat_smile: and VS Code community!

VSCode Version: 1.6.0
OS Version: Manjaro KDE Ellada 16.08

Steps to Reproduce: (hide activity bar)

Open devtools console
Copy and paste this line document.getElementById("workbench.parts.activitybar").remove();
Execute and then the node is removed
ctrl + b for autofix layout

I do this almost every time when I use VS Code, Can I automate it?
The trick source is here
This is related to #1105 and others.
Also I know that the idea is on going, but How can I do it :point_up_2: while I wait for this new awesome feature?
I tried to edit the vscode internal files as workspace.main.js and index.js without successful results.
Also I tried via extension, but VS Code don't allow it
Bye  :raising_hand_man:

Awesome 😁
Just for this kind of effort I propose that we look into hiding the activity bar in November. @bpasero ok with you I take that issue and assign to november?

Just make sure to coordinate with UX.

@bpasero ay ay. But this issue is about auto running javascript on startup, I do not think we want to support this?
I assigned the hide activity bar issue to me...

Yup unlikely we would want to support that.

Ok, Thanks you for response! :-)
On Thu, Oct 13, 2016 at 4:39 AM, Benjamin Pasero<EMAIL_ADDRESS>wrote:

Closed #13635 https://github.com/Microsoft/vscode/issues/13635.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/13635#event-822211795, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC7Nx6xI8xBlZMsQGHDISGpj-Yf3r328ks5qzfw4gaJpZM4KVWBW
.

So now you can simply move the bar using View -> Move Side Bar
But no feature to Hide it :(

@faustinoaq nice trick! I will probably not use it very often, but it's nice to know it's there!


History state key as timestamp to detect back/forward navigation

What problem does this feature solve?
Using Date.now() as key will provide a unique and comparable* identifier in relation to history navigation allowing back/forward navigation to be detected and used to define transition directions (slide left/right) etc.... and maybe it could be the first step in having vue-router provide this information out of the box :-)
*the current performance.now key does not ensure increment upon re-enter.

A quick detection example based on Date.now as key:
//  direction
//  -1 = history.back
//   0 = normal route
//   1 = history.forward
let direction = 0;

//   key value of the from route (0 = entering)
let fromKey = 0;

router.beforeEach((to, from, next) => {
    //  add state in case it's missing
    if(!history.state) { history.replaceState({key: Date.now() }, to.name, location.href); }

    //  get state key from history
    const toKey = history.state.key;

    //  if we are entering  or the from and to keys (time-stamps) are identical => normal route
    //  if from is less than to – to was created after from    => history.forward
    //  if from is less than to – to was created before from   => history.back
    direction = !fromKey || fromKey === toKey ? 0 : fromKey < toKey ? 1 : -1;

    //  call next
    next();
})

router.afterEach((to, from) => {
    //  use nextTick to await key to be generated
    Vue.nextTick(vm => {

        //  save key for next beforeEach
        fromKey = history.state.key;
    })
})

What does the proposed API look like?
function genKey () {
  return Date.now()  // current fallback
}

I also need to

What is the status for this issue? Is there a reason why this hasn't been included in the latest release?


Show more of the row text in the table component

If possible can we please show more of the text/content per row in the text component ("backend" in Sanity), as an improvement for editors.

Great Malin - thanks - approved :smile:


Consistent bool returns on exposed methods for err handling

This change is

I recall debating this in another PR, did you convince past me that this was better than returning errors?

'its how everything else is' was my argument and 1/2 1/2 is worse than all one or all the other.  Errorf() at the bottom your munger code just bails.
We can switch everything to errors in another go if we have some consistency. (and this fixes real bugs).

OK, sure. @foxish can you do a careful review? I only skimmed :)

@lavalamp ack. I'll get to it later today.

Review status: 0 of 30 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

mungegithub/github/github.go, line 511 [r1] (raw file):

	config.analytics.ListMilestones.Call(config, resp)
	if err != nil {
		glog.Errorf("Error getting milestones of state %q: %v", state, err)

Return (nil, false) here?

mungegithub/github/github.go, line 872 [r1] (raw file):

		return "", false
	}
	if !releaseMilestoneRE.MatchString(*title) {

We return true because this isn't an error condition?

Comments from Reviewable

@eparis I made one pass over it. I am a little worried about the fact that it will no longer be possible for any munger to see the error and act differently based on what kind of error it is, because we will always return a boolean for "something went wrong".

[APPROVALNOTIFIER] The Following OWNERS Files Need Approval:

mungegithub

We suggest the following people:
cc @bprashanth
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancelin a comment

After about 5 large rebases I'm thinking about just self-LGTM. We know it fixes bugs. If we want to convert to an error that's fine, but no doubt this is 100% better than it is today.
Also I intentionally used my personal e-mail on the commit to make sure the merge robot can handle cla: human-approved after we turned on protected branches.

@eparis Let's go ahead and merge this :)


[it] tutorials-keras-text_classification_with_hub.ipynb Basic translation

lgtm

Goodmorning @lamberta,
is there still some mistake into this translation?

Thanks for following up
Looks like the lint CI is failing (can ignore the labeler, that's a known issue) ...
Can click the "Details" link and looks like the button URLs are incorrect.
$ python3 -m pip install -U --user git+https://github.com/tensorflow/docs

$ python3 -m tensorflow_docs.tools.nblint --arg=repo:tensorflow/docs-1l0n \
    --styles=tensorflow,tensorflow_docs_l10n notebook.ipynb

@lamberta
I fixed the link issues.
Please, have a look to other PR still open.
Thank you.
Gianluca


Travis build on macOS and Windows

There are failing tests on Windows, needed to be fixed later

I just noticed that macOS uses the brew's Python (3.8) instead of the system's one


How do you specify a burst rate?

The headline for the project says

Call HTTP APIs that have rate limits and allow request bursts

but I can't quite figure out how to specify a burst rate. I'm trying to implement this policy:

Calls to the Market Data APIs are rate limited to 1 call per 10 seconds. API call rate limits allow bursts of up to five consecutive calls. Exceeding the limit causes HTTP error code 429 to be returned.

This is the closest I can get:
new RateLimiter({

    rate: 5,
    interval: 10,
    backoffTime: 10

})

Is that correct, or am I missing something?

You are allowed to send 1 call every 10 seconds or 5 calls every 50 seconds. This means probably that they are using a leaky bucket with that configuration. The correct parameters are probably those:

rate: 5
interval: 50
backoffTime: 10

This should allow you to send 5 requests as a burst and after that bout every 10 seconds another one.


Run example is not correct.

Instead of what you write in the instructions you have to write:
make generated/.pdf
This got me stumped for quite some time. Please update the instructions in the README.md file, or better, make it work without the generated/ prefix.

I heard from another user that they didn't have this problem.

This seems odd. make foo.pdf is how it works for me. Do you still have this problem?

Strangely it is different for different files in the same project (my isocpp-proposals repo). Everything seems to be the same but P3312R0.md requires the generated/ prefix while D3298R1.md doesn't. Smells like a bug in make but that's also not plausible.
Maybe just leave the issue open: in case someone else has this problem they will get a clue from it.


Why GPL license?

Why have you chosen the GPL license? It basically makes your library unusable in anything other than GPL projects. Can you dual license GPL and MIT (MIT is the de facto standard for JQuery and Bootstrap projects)? It would make the toolkit so much more usable. No-one in a corporate environment will be able to use your toolkit with it licensed under GPL.

I chose GPL because I want modifications to be open sourced, I don't want an individual or company modifying the code extensively and not re-releasing the code for free. It also fell inline with Bootstraps original licensing (Apache2).
However, I'm opening for changing to MIT. Submit a PR with it and i'll get it out in the next version.
In the mean time, why not include the lightbox via a CDN, thus technically not including the package - does that work for you?


Removed the unnecessary line

Line caused nginx to not work on port 80
Adding a # in front of it made it work on both port 80 and 443

Thank you, I have already taken care of this via pull request #695 - in here: https://github.com/pajbot/pajbot/pull/695/commits/d49d2c81ad6329b8f79702668cff5c2c0d3358ff

Fixed in #695 - thank you


Missing title on a span when using the javascript_driver

I am trying to test tooltips that rely on javascript. The plugin that I'm using requires that I put a title attribute on elements that get a tooltip. In this case, I am dealing with a span.
I could not get my test to work and I finally figured out the problem by simply doing a puts page.html. When I run my test with a javascript driver (capybara_webkit or selenium), the span does not have a title. When I run it with the default driver, the span has a title as expected.
I know it is not common to put a title on a span, but it does validate. Anyone know why using the javascript driver would strip attributes from elements?
I looked through the source, but I couldn't find the problem. I'd be happy submit a pull request to fix this if someone could point me in the right direction.

Yes, it's perfectly fine to put a title on a span, and I'd actually be surprised if Selenium or WebKit strip it. My first guess is that there's something going on with the JS on your page, or that invalid markup causes Rack::Test and Selenium/WebKit to behave differently.
This issue tracker is intended for reproducible bugs only, so if you need more help, I'd recommend that you post on the mailing list, perhaps with some code that illustrates the problem.

I just saw the note about the about the google group in the readme. I will have to check that out because I do have some questions. Sorry about opening an issue.


Remove unnecessary let statement to avoid compiler warning

Merged! @pomozoff thanks!

Thank you for that great project )


Enable upstream Python tests.

Scope
Fixes #12
Changes
Enabled the Python tests on the platforms where we build it, with the exception of:

[x] Alpine Linux
[ ] macOS.

Drive-by fixes:

fixed enabling EMUTRAMP on Alpine with paxctl.

Testing
Automated.

Not sure what do about macOS, haven't invested much time into it. If you want to, take a look, maybe some things are worth reporting upstream.
Otherwise, I would just except macOS alongside Alpine Linux.

it's ok to except macOS.
thanks

Sorry for the Shellcheck drive-by fixes, it looks like they weren't run at all in master because of a late "improvement" in the previous PR. :-/
I have now used execute for ./src/chevah-bash-tests/shellcheck_tests.sh, should not happen again…
One thing to keep in mind is that the only platform running all tests (upstream ones for OpenSSL/Python + ours for Python&modules + security check with safety + bash shell checks) is Linux. You could argue Windows is also fully covered, as we use the upstream Python/OpenSSL/cryptography, so there's no need to test what upstream has built.

My reading the code, this is mixed with compiling Python... which is a huge output, which is hard to check

I've worked on separating testing in a dedicated GHA step. Same for installing.

My preference, is to have "Pythion test" as a separate top level step.

It is a separate test, but not a top-level one. I think this kind of testing is best done where it belongs, right after compiling, but before installing. This way, if testing fails, the build is stopped right there, no need to continue with building other libraries.

I think that this can be merged. thanks!


Write a Haskell program to convert pound to kilogram

Description
Write a Haskell program to convert pound to kilogram
1 pound = 0.453592 kg
Input  : 1
Output : 0.453592

Contributed by - @Aranarora

How to contribute

Save the solution in convert_pound_to_kilogram.hs file
Add convert_pound_to_kilogram.hs file in program/convert-pound-to-kilogram folder

!assign


Import localStorage from http to https

I don't think this is possible. Also I don't think it's possible using iframes or anything because the server redirects http to https

Yeah this isn't going to happen


Molecule won't create containers

I'm getting this error message every time I try to run molecule test.  The create step does not work properly.  It seems to be skipped
Error Message
fatal: [rhel8]: FAILED! => {"ansible_facts": {}, "changed": false, "failed_modules": {"ansible.legacy.setup": {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python"}, "failed": true, "module_stderr": "Error: No such container: rhel8\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1, "warnings": ["Platform unknown on host rhel8 is using the discovered Python interpreter at /usr/bin/python, but future installation of another Python interpreter could change the meaning of that path. See https://docs.ansible.com/ansible/2.10/reference_appendices/interpreter_discovery.html for more information."]}}, "msg": "The following modules failed to execute: ansible.legacy.setup\n"}
fatal: [rhel7]: FAILED! => {"ansible_facts": {}, "changed": false, "failed_modules": {"ansible.legacy.setup": {"ansible_facts": {"discovered_interpreter_python": "/usr/bin/python"}, "failed": true, "module_stderr": "Error: No such container: rhel7\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1, "warnings": ["Platform unknown on host rhel7 is using the discovered Python interpreter at /usr/bin/python, but future installation of another Python interpreter could change the meaning of that path. See https://docs.ansible.com/ansible/2.10/reference_appendices/interpreter_discovery.html for more information."]}}, "msg": "The following modules failed to execute: ansible.legacy.setup\n"}

Red Hat Enterprise Linux Server release 7.9 (Maipo)
Everything installed in a virtualenv
molecule 3.2.0 using python 3.8
ansible:2.10.5
delegated:3.2.0 from molecule
docker:0.2.4 from molecule_docker

### molecule.yml
```yaml
---
dependency:
  name: galaxy
  options:
    ignore-certs: true
    ignore-errors: true
    requirements-file: collections.yml
lint: |
  set -e
  yamllint .
  ansible-lint
  flake8
driver:
  name: docker
platforms:
  - name: rhel7
    image: registry.access.redhat.com/ubi7/ubi
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    pre_build_image: false
  - name: rhel8
    image: registry.access.redhat.com/ubi8/ubi
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    pre_build_image: false
provisioner:
  name: ansible
  config_options:
    defaults:
      conditional_bare_variables: false
    ssh_connection:
      pipelining: true
  log: true
  options:
    vvv: true
  inventory:
    host_vars:
      rhel7:
        ansible_python_interpreter: auto
      rhel8:
        ansible_python_interpreter: auto
verifier:
  name: testinfra
  options:
    vvv: true
    sudo: true

molecule create

DEBUG    Validating schema /home/mfaine/development/collections/nics-default/roles/bacula/molecule/default/molecule.yml.
INFO     default scenario test matrix: dependency, create, prepare
INFO     Running default > dependency
WARNING  Skipping, missing the requirements file.
DEBUG    Using selector: EpollSelector
Starting galaxy collection install process
Process install dependency map
Starting collection install process
Installing 'nics.default:1.0.0' to '/home/mfaine/.cache/molecule/bacula/default/collections/ansible_collections/nics/default'
Created collection for nics.default at /home/mfaine/.cache/molecule/bacula/default/collections/ansible_collections/nics/default
nics.default (1.0.0) was installed successfully
INFO     Dependency completed successfully.
INFO     Running default > create
INFO     Sanity checks: 'docker'
DEBUG    Trying paths: ['/home/mfaine/.docker/config.json', '/home/mfaine/.dockercfg']
DEBUG    Found file at path: /home/mfaine/.docker/config.json
DEBUG    Found 'auths' section
DEBUG    Found entry (registry='gitlab.nisn.nasa.gov:5005', username='mfaine')
DEBUG    Found entry (registry='gitlab.nisn.nasa.gov:5050', username='bean')
DEBUG    Found entry (registry='registry.redhat.io', username='4870951|molecule')
DEBUG    http://localhost:None "GET /version HTTP/1.1" 200 892
DEBUG    http://localhost:None "GET /v1.41/_ping HTTP/1.1" 200 2

PLAY [Create] ******************************************************************

TASK [Populate instance config dict] *******************************************
skipping: [localhost]

TASK [Convert instance config dict to a list] **********************************
skipping: [localhost]

TASK [Dump instance config] ****************************************************
skipping: [localhost]

PLAY RECAP *********************************************************************
localhost                  : ok=0    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0

INFO     Running default > prepare
WARNING  Skipping, prepare playbook not configured.

molecule list:
DEBUG    Validating schema /home/mfaine/development/collections/nics-default/roles/bacula/molecule/default/molecule.yml.
INFO     Running default > list
                ╷             ╷                  ╷               ╷         ╷
  Instance Name │ Driver Name │ Provisioner Name │ Scenario Name │ Created │ Converged
╶───────────────┼─────────────┼──────────────────┼───────────────┼─────────┼───────────╴
  rhel7         │ docker      │ ansible          │ default       │ true    │ false
  rhel8         │ docker      │ ansible          │ default       │ true    │ false
                ╵             ╵                  ╵               ╵         ╵

I am logged in to the remote repo and the images exist locally:
docker images:
$ docker image ls --format "{{.ID}} {{.Repository}}" | grep ubi
4199acc83c6a registry.access.redhat.com/ubi8/ubi
c10235acdb0c registry.access.redhat.com/ubi7/ubi

I have used these same images all last week with no issue.  I don't know what has changed.  Any help would be appreciated.
I have also tried deleting ~/.cache/molecule but it has not helped.

Could you try with different images and also with privileged: true?:
platforms:
  - name: rhel7
    image: registry.access.redhat.com/ubi7/ubi
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    privileged: true
    pre_build_image: false
  - name: rhel8
    image: registry.access.redhat.com/ubi8/ubi
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    privileged: true
    pre_build_image: false

Same results with Centos7 and Centos8.    They even show as created though they absolutely do not exist:

INFO     Running default > prepare
WARNING  Skipping, prepare playbook not configured.
(virtualenv) [mfaine@p-msfc-enms-cm bacula]$ molecule list
DEBUG    Validating schema /home/mfaine/development/collections/nics-default/roles/bacula/molecule/default/molecule.yml.
INFO     Running default > list
                ╷             ╷                  ╷               ╷         ╷
  Instance Name │ Driver Name │ Provisioner Name │ Scenario Name │ Created │ Converged
╶───────────────┼─────────────┼──────────────────┼───────────────┼─────────┼───────────╴
  centos7       │ docker      │ ansible          │ default       │ true    │ false
  centos8       │ docker      │ ansible          │ default       │ true    │ false

Perhaps try upgrading to the newest version?


feat(ast): add TSModuleDeclarationName::binding_identifier()

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6005  👈
#6004
#6003
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite