set wrap for preview window

First of all, thanks for making such a great extension. I love how simple yet effective it is.
The only thing that bugs me a bit is that there is no way to force wrapping of lines in the preview window. it's really useful when I'm trying to figure out which bib entry is the right one (i.e. looking at the abstract).
I have set wrap active for .bib files but it doesn't seem to affect Telescope previews. Is that something that could be added on your end or by Telescope configuration?

Thanks,
Andrea

I think you are looking for this:
autocmd User TelescopePreviewerLoaded setlocal wrap
Source: https://github.com/nvim-telescope/telescope.nvim#previewers.
However it does not seem to work for me... It works with the builtin pickers but not with this plugin. I will dig into this and see I can get it to work.


Unexpected "Curly braces must have the proper spacing" warning caused by indented function calls

With a configuration file like this:
{
  "braces_spacing": {
    "spaces": 1,
    "empty_object_spaces": 0,
    "level": "error"
  }
}

… this code:
[
  {
    foo: ->
      bar ->
        1
  }
]

… gets a
#6: Curly braces must have the proper spacing. There should be 1 space inside "}"

… while this code
[
  {
    foo: ->
      bar -> 1
  }
]

… doesn't have any errors.
Seems like something about the nesting function calls is confusing the check.

It looks like this has been an issue since braces_spacing was introduced.


Import swagger from file

How would I import a swagger definition from a file?
I have an updated swagger spec that has not been implemented by the api (yet) and would like to run my tests against it.

I found that in the swagger import config object, you can set isTest to true and add a the path to the swagger.json relative to the folder you run the server from to path.


Allow specifying overrides locally?

Right now, rustup stores its overrides in ~/.multirust/settings.toml.
If I rename my directory, or move it elsewhere, the overrides don't follow.  Also, when our team updates the rust version for a particular project, we need to send out an email telling everyone to update their overrides, bringing problems cause by out-of-band communication.
These issues would go away if overrides could be stored in a project-local file (maybe ./rust-version, similar to .ruby-version?)
Has this been considered before?  Thanks.

Something similar was brought up before. At the time I was against it because I didn't want to see people pinning specific compiler host triples to their projects, and making life difficult for people trying to just build it.
The other factor was that the stable compiler should be... stable, so it's unclear whether that should be handled by rustup, or by cargo (say by depending on at least a certain libstd version).
However, another important case is pinning a specific version of nightly, without pinning the host triple, which is quite important for development against nightly where there are frequent breakages. I'd be in favour of something like that, maybe with precedence lower than the existing override system, so that it's still easy to bypass if need be.

That's true, the architecture would need to stay out of the file.  I hadn't noticed that.
So the .rust-version file would contain something like nightly-2016-11-06...  That sounds great.  And, agreed, rustup override would override any rust-version files.
A whole new dependency hell could be created where crates are all specifying different compilers,  but I think the rust community will naturally avoid that.  Heck, if you're publishing a crate with a .rust-version with anything other than nightly, something's amiss.
Sounds great.  I'd submit a PR, but I need to learn Rust first...

This is a dupe of https://github.com/rust-lang-nursery/rustup.rs/issues/460

Closing in favour of #460


Alternative solution: Identity Symbols

As #21 discussion, symbol as weakmap key have some arguments on registered symbols, both sides seems have good reasons, maybe we should find some tradeoffs which could satisfy both. I think the root cause of the issues is "symbol as weakmap keys" is too general, as I understand, the main use case is sharable identity, which do not need to be such general. Instead, we could introduce a much limited API:

Symbol.of(object) create the identity symbol for the object
get Symbol.prototype.object return the object of the identity symbol (return undefined for non-identity symbols)

Usage:
const obj = {}
const sym = Symbol.of(obj)
assert(symbol.object === obj)

Identity symbols (special symbol which ref to the object) would be somewhat like registered symbols (special symbol which ref to the string), so I expect it won't have much troubles to implement.
Note, such API is similar to Object.createSharableIdentity() idea except it use symbol for identity instead of frozen empty object.

Similar discussion here: https://github.com/tc39/proposal-record-tuple/issues/273

One pattern that the weakMap approach enables is 'swappable objects'.
For example #[symbol] can be passed between shadowRealms. But each realm has its own weakMap which maps the symbol to the appropriate object for that realm.

If it doesn’t work cross-realm, then it’s realm-specific, and that’s not something the language does. Template objects are the only example I’m aware of.

and Template objects are not technically realm-specific. They are tied to the 'parse node'.
function tag(arr) { return arr; }

function f() { return        tag``   ; }
function g() { return eval(' tag`` '); }

assert(f() === f());
assert(g() !== g());

@acutmore Symbol.of(object) would behave like Symbol.for, always return same symbol value even cross-realm. But Symbol.prototype.object may throw cross-realm.
If it's an iframeRealm and u already have the right to access iframeRealm.globalThis.Symbol, I suppose it could get the object cross-realm. If the code do not have the right to access the other realm due to isolation, it throws. It's a security constraint, I don't treat it as realm-specific.
@ljharb said "any realm has the power to unwrap any realm's Box", I think it could be "any realm has the power to unwrap any realm's Box if it not violate the security isolation".
If we require "any realm has the power to unwrap any realm's Box in any case", it just contradict with the security requirement, so the only solution would be "symbol as weakmap key", as it just transfer the "realm-specific" to userland. And if we also require not differentiate symbols, maybe we have to introduce a new primitive type (for example Identity) for weakmap key. 😅
Another compromise solution is:
Introduce Symbol.identity() to create identity symbols, use Symbol.prototype.isIdentity() to check it, and only allow identity symbols as the weakmap key. It still differentiate the symbols, but at least it have much clear surface API and semantic for the programmers.

There is no consensus on that security isolation being a viable constraint.

It seems you're reinventing ObjectPlaceholder and going through all the motions the champion group went through.

Identity symbols (special symbol which ref to the object) would be somewhat like registered symbols (special symbol which ref to the string), so I expect it won't have much troubles to implement.

Taking a step back, how does this solve one of the objection that all symbols be treated the same when adding to WeakMap keys? You've just created another kind of symbol, with unique identity, but registered and well-known symbols are still forgeable, and still share a typeof.

how does this solve one of the objection

@mhofman  This alternative actually do not introduce symbol as weakmap key, so there will be no observable difference. 😂
But as @acutmore said, it can't solve "swappable objects" case. I'm not sure whether  "swappable objects" are the hard requirement, if that, it seems we must  introduce symbol as weakmap key.

There is no consensus on that security isolation being a viable constraint.

But such security isolation constraints are the consequence of the design of shadow realms,  aren't they? And shadow realm is already stage 3 so doesn't it already have consensus ?

@hax it's part of shadow realms, but it's still not a constraint on the language to maintain it.


⚠ Bofrost.be has degraded performance

In 8c8e5f9, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1024 ms

Resolved: Bofrost.be performance has improved in 4d71a29 after 13 minutes.


How to evaluate the clv for long existing customers?

This is more like a model question instead of the package itself.
The model assumes the first purchase within the time period is the first purchase when the customer introduced to the company.
However, this is not always the case, since there are customers that exist two year or even longer if the company is not brand new.
In this case, would I still be able to get the prediction on the long existing customers? Or at least, would you have any suggestion that how I can evaluate the prediction for these type of customers fits in the model?
Thank you so much!

If I understood it well this model is based in the "buy till you die" thing. So what it's important is in which probability they are alive or not. What we are looking for is each individual unique buying pattern.
I think you can get a good prediction for your long existing customers because they nowadays follow the same buying patterns than they had before you started collecting data. Or if they don't, it's more insightful how they behave in the present -maybe they change their lifestyle of whatever- for predicting the future, than how they behave in the past. I also think it's pretty common not having data since the first day.
It's an intuitive approach more than a technical one but hope it helps!

I'm not quite sure if I understood your question either but I'll try my best to answer.
This library works on RFM data, where recency is not only one of the main features, but probably the main one, as was mentioned by Prof. Peter Fader in this lecture — which is also on the main README.
Due to the importance of recency, the model will automatically slowly kill inactive customers, even if they are old ones. The probability of them being alive will have a close-to-exponential decay and they will eventually die.
An option you have to deal with your scenario is to create a window of analysis. For example, you could force customers that have been in the data for more than 2 years to b deleted from the dataset.
If your problem is actually to account for customers' purchase behavior before the window, then you could simply (re)calculate their RFM for before that time period and then use the model on that. But do keep in mind that you might have to shift the other customers' (x, t_x, T) proportionately — simply preprocessing all of the customers in one batch would be far simpler.


linker command failed with exit code 1 (use -v to see invocation)

My project dependency
"dependencies": {
"react": "16.3.1",
"react-native": "0.55.1",
"react-native-animated-hide-view": "^1.0.0",
"react-native-elements": "^0.19.1",
"react-native-fcm": "^16.2.3",
"react-native-keyboard-aware-scrollview": "^2.0.0",
"react-native-simple-dialogs": "^0.4.1",
"react-native-vector-icons": "^4.6.0",
"react-navigation": "^2.0.0-rc.9"
},
after adding FCM and linking via COCOPAD in my react native IOS project
i am not able build my project in XCODE
i am able run my app in emulator from VS Code with command react-native run-ios
but now i have archive the project and upload on testFlight to be test and its shows build failed

i have also tried to build project with .workspace but issue is same

I also have the same issue, is there any solution???

I have not found any solution yet but
but this error will not stop you from running the app
I am able to test the app and publish on store

well in my case, I cannot run the apps.


Post History Table View Freezes on Scroll

When playing a song in the Post History Table View, the view freezes when trying to scroll. It could be that the Post View Table View is slow and is crashing my simulator.

Is this still actually an issue, feel like this was fixed a while back?


Fix regression tests on CUDA and ROCm

Fix regression tests

Appended the comment in pull request #5:
OBS! The test test_handler.py::test_huggingface_bert_model_parallel_inference fails due to:
ValueError: Input length of input_ids is 150, but max_length is set to 50. This can lead to unexpected behavior. You should consider increasing max_length or, better yet, setting max_new_tokens.
This indicates that preprocessing uses a different max_length than inference, which can be verified when looking at the handler when the test was originally implemented: model.generate() has max_length=50 by default, while tokenizer uses max_length from setup_config (max_length=150). It seems that the bert-based Textgeneration.mar needs an update.


Improve documentation on SBT dependencies

I had to spent >= 30min to figure out that I needed the following dependencies to run what the docs currently claim to only need finch-circe and 2 imports:
libraryDependencies ++= Seq(
  "com.github.finagle" %% "finchx-core" % "0.31.0",
  "com.github.finagle" %% "finchx-circe" % "0.31.0",
  "io.circe" %% "circe-generic" % "0.12.3"
)

I found the above dependencies in the finch/finch.g8 repo, however a) I had to look in a build.sbt in some sub^n-directory and b) it did not contain versions, so I had to look them up manually on Maven central.
Apparently, #1022 had a very similar issue. There it reads that "Finch is very unfriendly with first time users". Unfortunately, it seems it still is to some extent. Let's improve that :smile:

Well, it's been just over three months and this is still the case. Not only that, but the dependencies that @ComFreek do not work anymore.


Attribute validation for astropy.regions?

In https://github.com/astropy/regions/pull/81#issuecomment-263240526 and other places in astropy.regions before the question came up whether we should do input validation (for PixCoord, BoundingBox, region classes with float or Angle or SkyCoord attributes, ....
I'm +1 to do it. It's some extra effort for us, but especially for interactive work it really helps users catch many little mistakes.
My preference would be to use something like attrs or numtraits instead of re-implementing something similar again here in astropy.regions.
@astrofrog @larrybradley or anyone - Thoughts?
Is there something available or proposed in Astropy core?
Should we bundle or add dependency here?
Should we discuss on the Astropy mailing list or issue tracker now, or just go ahead and start using something here and discuss later / hope they accept it in Astropy core?

I don't know if there's some data validation descriptors in Astropy already and given that this package is small and our needs are modest I wouldn't suggest something like attrs or traits, so having a few custom descriptor classes here in the regions package seems fine.
I had a very quick look at your descriptor implementations.
It looks like you have a mix of using WeakKeyDictionary and some where you use the instance to store the value and use getattr from the descriptor. Is there a logic to that or can this be simplified / unified a little?
Personally I find it weird to have a WeakKeyDictionary and have the values not stored on the instances, and like the pattern to keep the values in the instance dict as usual, and only use the descriptor for validation and modify the instance dict from the descriptor. This is explained here. Really the only drawback is that you have to put the same attribute and descriptor label name, which I don't find hard to get right because it's on a single line.

Actually the logic is : the key of the WeakKeyDictionary is an instance of the regions.Region class and it's value is the value of the attribute . We use weak dictionary so that the key value pair exists until the instance is alive, hence prevents memory leakage.
We store it in this way because the descriptor classes are static to it's owner class.
The problem with this arises only when the instances are unhashable, which is not the case for regions.Region (according to the link, you provided).
Thanks for the link btw!!

Yes, what I dislike about this is that the data isn't stored in the region instances, but in a global dict in the descriptor. I didn't review your code carefully, but probably it's all good, this is one valid pattern to use descriptors. It just isn't my preference. If you prefer to keep as-is, we can just close this issue. (I don't have time to hack on regions anyways unfortunately.)

hmm, I get it . It kind of looks lil ugly. I will try to find some other way, if I get some time. Let's keep it open then. Thanks!

I changed the descriptor implementation in #284 .
I'm still not very happy with https://github.com/astropy/regions/blob/master/regions/core/attributes.py

Suggest to split out Meta, RegionMeta and RegionVisual into a separate file regions/core/meta.py
Suggest to improve the names of the descriptors. E.g. CompoundRegionSky sounds like a region, but it's not, it's a descriptor for a SkyCoord`.
We could consider choosing a standard solution like https://www.attrs.org/ or https://docs.python.org/3/library/dataclasses.html which is simpler and more powerful instead of maintaining our own set of validators. Probably this only makes sense if Astropy core adopts them, otherwise we should stick with our homegrown or if it exists use something from Astropy core.


Webhook container failing on install

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.

Describe the bug:
Issue is the webhook container isn't starting because of its missing a volume "certs", which relys on a secret called "cert-manager-webhook-tls"
[jenkins-slave-01@ip-172-26-212-54 ~]$ kubectl get pods -n cert-manager
NAME                                       READY   STATUS              RESTARTS   AGE
cert-manager-6bcc9d894d-4cpcb              1/1     Running             0          9h
cert-manager-cainjector-594fd9cc45-khn2r   1/1     Running             0          9h
cert-manager-webhook-785ff8fc78-vgccp      0/1     ContainerCreating   0          9h
kubectl describe pod cert-manager-webhook-785ff8fc78-vgccp -n cert-manager
Events:
Type     Reason       Age                   From                                    Message

Warning  FailedMount  10m (x276 over 9h)    kubelet, ip-172-26-210-11.ec2.internal  MountVolume.SetUp failed for volume "certs" : secret "cert-manager-webhook-tls" not found
Warning  FailedMount  3m58s (x243 over 9h)  kubelet, ip-172-26-210-11.ec2.internal  Unable to mount volumes for pod "cert-manager-webhook-785ff8fc78-vgccp_cert-manager(c2f36043-17ea-11ea-8b50-02c0339671f7)": timeout expired waiting for volumes to attach or mount for pod "cert-manager"/"cert-manager-webhook-785ff8fc78-vgccp". list of unmounted volumes=[certs]. list of unattached volumes=[certs cert-manager-webhook-token-m5nkh]
Expected behaviour:
A concise description of what you expected to happen.
Steps to reproduce the bug:
Steps to reproduce the bug should be clear and easily reproducible to help people
gain an understanding of the problem.
$ kubectl apply --validate=false -f https://raw.githubusercontent.com/jetstack/cert-manager/release-0.12/deploy/manifests/00-crds.yaml
$ helm repo add jetstack https://charts.jetstack.io
$ helm repo update
$ helm install
--name cert-manager
--namespace cert-manager
--version v0.12.0
jetstack/cert-manager
Anything else we need to know?:
Environment details::

Kubernetes version (e.g. v1.10.2): v1.14.0
Cloud-provider/provisioner (e.g. GKE, kops AWS, etc): EKS
cert-manager version (e.g. v0.4.0): v0.12.0
Install method (e.g. helm or static manifests): helm

/kind bug

I found the issue. I had another cert-manager installed in another namespace. Once I removed that older version and reinstalled using helm the webhook pod came up with no issues.

cert-manager-webhook-tls secret is created when its not found...  The problem is that it takes almost 4mins to get full started cert-manager-webhook pod.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 445

Questions ?
Please refer to the Backport tool documentation


taxid

Homology search started at 2024-08-21 08:40:06.163220.
Settings:
Search method: diamond.
Self-alignment method: native.
Remote fetch enabled: no.
Reading input proteins...
Sulfurospirillum_multivorans: 3224 proteins.
Done. Read 3224 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Reading local taxonomy database... done.
Read 2597948 taxa.
Batch homology search of Sulfurospirillum_multivorans started at 2024-08-21 08:40:16.227200.
Number of queries: 3223.
WARNING: Cannot obtain taxIds for 470074 sequences. These hits will be dropped.
There are so many taxIds that have not been obtained, will it affect the results

It will. Could there be some issues with the database? If it was automatically generated from NCBI, all sequences should have taxIDs.

Hi @duxi190914 It could be that NCBI has updated its database structure such that the existing HGTector pulls more than expected. It is totally fine to use the 50G database.

Dear professor,
I used the database you provided, running the protein file , but I still got an error. I don't know how to solve the problem, and the next is the code that reported the error.
(base) @.*** hgtdb_20230102]$ hgtector search -i 1612.all.faa -o 8.281
Homology search started at 2024-08-28 15:43:39.969565.
Settings:
Search method: remote.
Self-alignment method: native.
Remote fetch enabled: yes.
Reading input proteins...
1612.all: 2357 proteins.
Done. Read 2357 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Initiating custom taxonomy database... done.
Batch homology search of 1612.all started at 2024-08-28 15:43:40.175473.
Number of queries: 2357.
Submitting 20 queries for search. RID: CXNFJTSU016..................... Success. Results retrieved.
Fetching taxIds of 8485 sequences from remote server...
Fetched information of 100 entries.
....
Done. Obtained taxIds of 8485 sequences.
Fetching 4558 taxIds and ancestors from remote server...
Fetched information of 100 entries.
......
Done. Obtained taxonomy of 4558 taxIds.
Submitting 9 queries for self-alignment. RID: CXUU11WP114. Results retrieved.
Submitting 11 queries for self-alignment. RID: CXUX49BY114. Results retrieved.
20 queries completed.
Submitting 17 queries for search. RID: CXV22SXB013............... Success. Results retrieved.
Fetching taxIds of 7707 sequences from remote server...
Fetched information of 100 entries.
.....
Traceback (most recent call last):
File "/home/wangzx/miniconda3/bin/hgtector", line 4, in
import('pkg_resources').run_script('hgtector==2.0b3', 'hgtector')
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 720, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 1559, in run_script
exec(code, namespace, namespace)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 96, in
main()
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 35, in main
module(args)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 182, in call
self.taxid_wf(res)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 745, in taxid_wf
newmap = {x[0]: x[1] for x in self.remote_seqinfo(ids2q)}
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1257, in remote_seqinfo
return self.parse_fasta_xml(self.remote_fetches(
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1292, in remote_fetches
res += self.remote_fetch(urlapi.format(','.join(batch)))
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1337, in remote_fetch
return response.read().decode('utf-8')
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 470, in read
return self._readall_chunked()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 577, in _readall_chunked
chunk_left = self._get_chunk_left()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 560, in _get_chunk_left
chunk_left = self._read_next_chunk_size()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 520, in _read_next_chunk_size
line = self.fp.readline(_MAXLINE + 1)
File "/home/wangzx/miniconda3/lib/python3.9/socket.py", line 704, in readinto
return self._sock.recv_into(b)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1242, in recv_into
return self.read(nbytes, buffer)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1100, in read
return self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out
wish you all the best,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | Qiyun @.> |
| Date | 8/28/2024 00:54 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Hi @duxi190914 It could be that NCBI has updated its database structure such that the existing HGTector pulls more than expected. It is totally fine to use the 50G database.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Dear professor,
I noticed when I rum the hgtector, one progress is different.  Sometimes is Remote fetch enabled: yes ,and sometimes is Remote fetch enabled: No. When I ues the datebase you provided is yes, it doesn't have influence to resullts  or have infulences.
wish you all the best
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 8/28/2024 18:19 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I used the database you provided, running the protein file , but I still got an error. I don't know how to solve the problem, and the next is the code that reported the error.
(base) @. hgtdb_20230102]$ hgtector search -i 1612.all.faa -o 8.281
Homology search started at 2024-08-28 15:43:39.969565.
Settings:
Search method: remote.
Self-alignment method: native.
Remote fetch enabled: yes.
Reading input proteins...
1612.all: 2357 proteins.
Done. Read 2357 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Initiating custom taxonomy database... done.
Batch homology search of 1612.all started at 2024-08-28 15:43:40.175473.
Number of queries: 2357.
Submitting 20 queries for search. RID: CXNFJTSU016..................... Success. Results retrieved.
Fetching taxIds of 8485 sequences from remote server...
Fetched information of 100 entries.
....
Done. Obtained taxIds of 8485 sequences.
Fetching 4558 taxIds and ancestors from remote server...
Fetched information of 100 entries.
......
Done. Obtained taxonomy of 4558 taxIds.
Submitting 9 queries for self-alignment. RID: CXUU11WP114. Results retrieved.
Submitting 11 queries for self-alignment. RID: CXUX49BY114. Results retrieved.
20 queries completed.
Submitting 17 queries for search. RID: CXV22SXB013............... Success. Results retrieved.
Fetching taxIds of 7707 sequences from remote server...
Fetched information of 100 entries.
.....
Traceback (most recent call last):
File "/home/wangzx/miniconda3/bin/hgtector", line 4, in
import('pkg_resources').run_script('hgtector==2.0b3', 'hgtector')
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 720, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 1559, in run_script
exec(code, namespace, namespace)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 96, in
main()
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 35, in main
module(args)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 182, in call
self.taxid_wf(res)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 745, in taxid_wf
newmap = {x[0]: x[1] for x in self.remote_seqinfo(ids2q)}
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1257, in remote_seqinfo
return self.parse_fasta_xml(self.remote_fetches(
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1292, in remote_fetches
res += self.remote_fetch(urlapi.format(','.join(batch)))
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1337, in remote_fetch
return response.read().decode('utf-8')
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 470, in read
return self._readall_chunked()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 577, in _readall_chunked
chunk_left = self._get_chunk_left()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 560, in _get_chunk_left
chunk_left = self._read_next_chunk_size()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 520, in _read_next_chunk_size
line = self.fp.readline(_MAXLINE + 1)
File "/home/wangzx/miniconda3/lib/python3.9/socket.py", line 704, in readinto
return self._sock.recv_into(b)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1242, in recv_into
return self.read(nbytes, buffer)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1100, in read
return self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out
wish you all the best,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | Qiyun @.> |
| Date | 8/28/2024 00:54 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Hi @duxi190914 It could be that NCBI has updated its database structure such that the existing HGTector pulls more than expected. It is totally fine to use the 50G database.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @duxi190914 It you disable remote fetch, it may resolve the problem. Remote fetch could introduce instability of the taxonomy structure.

Dear professor,
I find that when I analyze hgt, there are 400 hgt genes in a genome which are nearly about 20 percent of a genome. And I used the default parameters, will it make any difference to the results.
wish you all the bests,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 9/7/2024 13:35 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I noticed when I rum the hgtector, one progress is different.  Sometimes is Remote fetch enabled: yes ,and sometimes is Remote fetch enabled: No. When I ues the datebase you provided is yes, it doesn't have influence to resullts  or have infulences.
wish you all the best
duxi
| |
duxixixixixi
|
|
@.
|
---- Replied Message ----
| From | @.> |
| Date | 8/28/2024 18:19 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I used the database you provided, running the protein file , but I still got an error. I don't know how to solve the problem, and the next is the code that reported the error.
(base) @. hgtdb_20230102]$ hgtector search -i 1612.all.faa -o 8.281
Homology search started at 2024-08-28 15:43:39.969565.
Settings:
Search method: remote.
Self-alignment method: native.
Remote fetch enabled: yes.
Reading input proteins...
1612.all: 2357 proteins.
Done. Read 2357 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Initiating custom taxonomy database... done.
Batch homology search of 1612.all started at 2024-08-28 15:43:40.175473.
Number of queries: 2357.
Submitting 20 queries for search. RID: CXNFJTSU016..................... Success. Results retrieved.
Fetching taxIds of 8485 sequences from remote server...
Fetched information of 100 entries.
....
Done. Obtained taxIds of 8485 sequences.
Fetching 4558 taxIds and ancestors from remote server...
Fetched information of 100 entries.
......
Done. Obtained taxonomy of 4558 taxIds.
Submitting 9 queries for self-alignment. RID: CXUU11WP114. Results retrieved.
Submitting 11 queries for self-alignment. RID: CXUX49BY114. Results retrieved.
20 queries completed.
Submitting 17 queries for search. RID: CXV22SXB013............... Success. Results retrieved.
Fetching taxIds of 7707 sequences from remote server...
Fetched information of 100 entries.
.....
Traceback (most recent call last):
File "/home/wangzx/miniconda3/bin/hgtector", line 4, in
import('pkg_resources').run_script('hgtector==2.0b3', 'hgtector')
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 720, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 1559, in run_script
exec(code, namespace, namespace)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 96, in
main()
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 35, in main
module(args)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 182, in call
self.taxid_wf(res)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 745, in taxid_wf
newmap = {x[0]: x[1] for x in self.remote_seqinfo(ids2q)}
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1257, in remote_seqinfo
return self.parse_fasta_xml(self.remote_fetches(
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1292, in remote_fetches
res += self.remote_fetch(urlapi.format(','.join(batch)))
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1337, in remote_fetch
return response.read().decode('utf-8')
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 470, in read
return self._readall_chunked()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 577, in _readall_chunked
chunk_left = self._get_chunk_left()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 560, in _get_chunk_left
chunk_left = self._read_next_chunk_size()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 520, in _read_next_chunk_size
line = self.fp.readline(_MAXLINE + 1)
File "/home/wangzx/miniconda3/lib/python3.9/socket.py", line 704, in readinto
return self._sock.recv_into(b)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1242, in recv_into
return self.read(nbytes, buffer)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1100, in read
return self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out
wish you all the best,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | Qiyun @.> |
| Date | 8/28/2024 00:54 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Hi @duxi190914 It could be that NCBI has updated its database structure such that the existing HGTector pulls more than expected. It is totally fine to use the 50G database.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @duxi190914 it could be 20% of the genome, but usually one should be critical if that's the output. Adjusting the parameters, especially the definition of self / close / distal groups, can help to refine your results. Under the default mode, the program will automatically define the three groups based on the taxonomy tree, which may or may not be what you want.

Dear professor,
I have one thing about parameters I can't understand. why e-value 1e-100, identity 70, coverage 70,  have more hgts than e-value 1e-100, identity 45, coverage 70. It's hard to understand, I wonder if there's a problem here.
wish you all the best,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 9/25/2024 19:16 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I find that when I analyze hgt, there are 400 hgt genes in a genome which are nearly about 20 percent of a genome. And I used the default parameters, will it make any difference to the results.
wish you all the bests,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | @.> |
| Date | 9/7/2024 13:35 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I noticed when I rum the hgtector, one progress is different.  Sometimes is Remote fetch enabled: yes ,and sometimes is Remote fetch enabled: No. When I ues the datebase you provided is yes, it doesn't have influence to resullts  or have infulences.
wish you all the best
duxi
| |
duxixixixixi
|
|
@.
|
---- Replied Message ----
| From | @.> |
| Date | 8/28/2024 18:19 |
| To |<EMAIL_ADDRESS>|
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Dear professor,
I used the database you provided, running the protein file , but I still got an error. I don't know how to solve the problem, and the next is the code that reported the error.
(base) @. hgtdb_20230102]$ hgtector search -i 1612.all.faa -o 8.281
Homology search started at 2024-08-28 15:43:39.969565.
Settings:
Search method: remote.
Self-alignment method: native.
Remote fetch enabled: yes.
Reading input proteins...
1612.all: 2357 proteins.
Done. Read 2357 proteins from 1 samples.
Dropping sequences shorter than 30 aa... done.
Initiating custom taxonomy database... done.
Batch homology search of 1612.all started at 2024-08-28 15:43:40.175473.
Number of queries: 2357.
Submitting 20 queries for search. RID: CXNFJTSU016..................... Success. Results retrieved.
Fetching taxIds of 8485 sequences from remote server...
Fetched information of 100 entries.
....
Done. Obtained taxIds of 8485 sequences.
Fetching 4558 taxIds and ancestors from remote server...
Fetched information of 100 entries.
......
Done. Obtained taxonomy of 4558 taxIds.
Submitting 9 queries for self-alignment. RID: CXUU11WP114. Results retrieved.
Submitting 11 queries for self-alignment. RID: CXUX49BY114. Results retrieved.
20 queries completed.
Submitting 17 queries for search. RID: CXV22SXB013............... Success. Results retrieved.
Fetching taxIds of 7707 sequences from remote server...
Fetched information of 100 entries.
.....
Traceback (most recent call last):
File "/home/wangzx/miniconda3/bin/hgtector", line 4, in
import('pkg_resources').run_script('hgtector==2.0b3', 'hgtector')
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 720, in run_script
self.require(requires)[0].run_script(script_name, ns)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/pkg_resources/init.py", line 1559, in run_script
exec(code, namespace, namespace)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 96, in
main()
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/EGG-INFO/scripts/hgtector", line 35, in main
module(args)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 182, in call
self.taxid_wf(res)
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 745, in taxid_wf
newmap = {x[0]: x[1] for x in self.remote_seqinfo(ids2q)}
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1257, in remote_seqinfo
return self.parse_fasta_xml(self.remote_fetches(
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1292, in remote_fetches
res += self.remote_fetch(urlapi.format(','.join(batch)))
File "/home/wangzx/miniconda3/lib/python3.9/site-packages/hgtector-2.0b3-py3.9.egg/hgtector/search.py", line 1337, in remote_fetch
return response.read().decode('utf-8')
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 470, in read
return self._readall_chunked()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 577, in _readall_chunked
chunk_left = self._get_chunk_left()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 560, in _get_chunk_left
chunk_left = self._read_next_chunk_size()
File "/home/wangzx/miniconda3/lib/python3.9/http/client.py", line 520, in _read_next_chunk_size
line = self.fp.readline(_MAXLINE + 1)
File "/home/wangzx/miniconda3/lib/python3.9/socket.py", line 704, in readinto
return self._sock.recv_into(b)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1242, in recv_into
return self.read(nbytes, buffer)
File "/home/wangzx/miniconda3/lib/python3.9/ssl.py", line 1100, in read
return self._sslobj.read(len, buffer)
socket.timeout: The read operation timed out
wish you all the best,
duxi
| |
duxixixixixi
|
|
@.***
|
---- Replied Message ----
| From | Qiyun @.> |
| Date | 8/28/2024 00:54 |
| To | @.> |
| Cc | @.> ,
@.> |
| Subject | Re: [qiyunlab/HGTector] taxid (Issue #137) |
Hi @duxi190914 It could be that NCBI has updated its database structure such that the existing HGTector pulls more than expected. It is totally fine to use the 50G database.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


hlint -g . does not respect .gitignore

In my repo directory, I have dist-newstyle created by cabal in my gitignore, however hlint -g goes into this directory and returns lint errors for sources checkout by cabal inside this directory.
As a workaround I am using git ls-files | grep '.hs$' | xargs hlint.

On my machine it runs git ls-files --cached --others --exclude-standard *.hs *.lhs. What does that command return on your machine? (You probably need to quote the *). I just pushed changes to --verbose so hlint -g -v shows what command it runs, if you want to try HEAD.

Ah, I think hlint -g . is saying test all files found by --git AND test all files found by . - together which is everything. As it happens #780 was fixed a few days ago, which likely means that . will start ignoring dist-newstyle anyway.

Ah I see, I didn't realize that providing a directory/file argument was optional, but it's clear from the --help message now. hlint -g works fine for me. Closing this. Thanks your quick response!


syncthing: 0.14.48 -> 0.14.50

Motivation for this change
Upgrade syncthing.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Should this be backported (is this a bug fix release?)

@GrahamcOfBorg  syncthing

@GrahamcOfBorg build syncthing

Success on x86_64-darwin (full log)
Attempted: syncthing
Partial log (click to expand)
github.com/syncthing/syncthing/lib/connections
github.com/syncthing/syncthing/lib/model
github.com/syncthing/syncthing/cmd/syncthing
installing
post-installation fixup
gzipping man pages under /nix/store/w2hpyix17bppy51z0bxip8z3i3gyvlhc-syncthing-0.14.50/share/man/
strip is /nix/store/df6k4mgdjxciy0f637lryp7c9ln7n1m3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/w2hpyix17bppy51z0bxip8z3i3gyvlhc-syncthing-0.14.50/bin
patching script interpreter paths in /nix/store/w2hpyix17bppy51z0bxip8z3i3gyvlhc-syncthing-0.14.50
/nix/store/w2hpyix17bppy51z0bxip8z3i3gyvlhc-syncthing-0.14.50

Success on x86_64-linux (full log)
Attempted: syncthing
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50
shrinking /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50/bin/syncthing
gzipping man pages under /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50/lib  /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50/bin
patching script interpreter paths in /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50
checking for references to /build in /nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50...
/nix/store/fs4g52nwr8sx9jar2bv3vml20mwirqn0-syncthing-0.14.50

Success on aarch64-linux (full log)
Attempted: syncthing
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50
shrinking /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50/bin/syncthing
gzipping man pages under /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50/lib  /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50/bin
patching script interpreter paths in /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50
checking for references to /build in /nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50...
/nix/store/w515h961rn0fynrv2r4yi6q14nxyszha-syncthing-0.14.50

Can you ask for ofborg access? https://github.com/NixOS/ofborg/pull/222

Asked for ofborg acces.
I think it'd be good to backport this. At least my syncthing was broken without this bugfix release.

done 1851422f61e


Make the connectionStrings field an array of maps

Right now the connectionStrings type for a configuration YAML file is an array of strings. We may need to add more information / context to each value in this array so it should be converted into an array of maps:
url: someURL
storage:
  - connectionString: x
  - connectionString: y

This should be fairly straight-forward to do. It would require

Updating these yml files with the new format: https://github.com/alpine-hodler/gidari/tree/main/internal/transport/testdata/upsert

Add a new struct to transport.go called storageReference (or whatever):

type storageReference struct {
  ConnectionString: string `yaml:"connectionString"`
}

Updating the Config struct to use []storageReference rather than []string for Conifg.ConnectionStrings. We should also change the name from Config.ConnectionStrings to Config.StorageReference.

@krishnaindani Assigning to you

Hello, I would like to do this! @prestonvasquez can you assign me?

Could you please assgn to me as we talked about combining with other issue?

@axlts Thank you for your interest in this issue, however it is directly related to work @krishnaindani has in progress. These issues are fairly similar, dealing with the same area of code: https://github.com/alpine-hodler/gidari/issues/153 , https://github.com/alpine-hodler/gidari/issues/155 .

would we like to take like
storage:

connectionString: dbtype

for example:
storage:

"mongodb://mongo1:27017/jokes": mongoDB
"mongodb://mongo2:27017/jokes": mongoDB

Sorry I think I got it wrong, the intention here is to keep the nested name to still connectionString just the value needs to be the url of the connection, is it?
for example
url: someURL
storage:
  - connectionString: "mongodb://mongo1:27017/jokes"
  - connectionString: "mongodb://mongo2:27017/jokes"

@krishnaindani The storage package already has a method to parse the scheme from connection string: https://github.com/alpine-hodler/gidari/blob/main/internal/storage/storage.go#L90
Using an example from Postman's public API list, I think it should look something like
url: https://official-joke-api.appspot.com
storage:
  - connectionString: mongodb://mongo1:27017/jokes
  - connectionString: mongodb://mongo1:27017/jokes
rateLimit:
  burst: 5
  period: 1
requests:
  - endpoint: /random_joke

@krishnaindani Are you still interested in pursuing this issue?

sorry I was in gophercon, so I missed it. Someone can talk a look before I can catch up maybe.

Closed by #360


UI Registration page

New Registration Page styled like the Login Page.
Not sure if I need to add an input column for the NutzerID?!
@jakobkmar pls rewiev the pull request

Not sure if I need to add an input column for the NutzerID?!

no, the user id will be generated automatically either in the backend or by the database


ApiVersion in management client not initialized to the current version in Generated Code for StorageCache

Describe the bug
The ApiVersion property is not getting initialized in the Generated ManagementClient Initialize function like it is in other management clients.
I recently had this update merged for updates to StorageCache.  Our test team had an bug for the previous version as well that noted that the ApiVersion had to be explicitly set.  This was due to the generated code not having an ApiVersion = "2021-03-01"; line in the initialize function in the management client class.
https://github.com/Azure/azure-sdk-for-net/pull/19447
Adding @markcowl, since we discussed it in the .net sdk office hours on 4/14/21.

Per the offline discussion, this is due to the standard c-sharp generator configuration for management clients not being picked up when generating this RP.  This has a number of effects, including the default exception type, license headers in the source code, and differences in LRO execution. To fix all these issues, you would need to update the swagger configuration to turn azure-arm on, and this would result in a number of breaking changes in your library.
Since this appears to be a longstanding problem, and we have track 2 SDKs coming next semester, my advice is to not make this breaking change at this point, but wait for the track 2 SDKs, which will be entirely new packages.

@brpanask let me know if you would like to pursue the management library changes.  I think this would require a substantial testing effort, and definitely involves a substantial set of breaking changes, and all of this will be fixed in track 2 management sdk in a few months, which is why I am advising against it.

I think we can close this then.
I will publish the nuget package as is.
Do I still need to update the readmes for .net for the track 2 update?

Closing since the issue is in our rest api readme.


Implementation of the IRDIS/LSS mode support

[x] object definition
[x] basic methods to sort files & frames
[ ] creation of calibrations
[ ] pre-processing of science frames
[ ] processing of science frames
[ ] cleaning
[ ] small refinements specific to LSS

Updates following new work:

all calibrations: 676a301a9ba1616453e0a5612703d7a6e91b385e,<PHONE_NUMBER>0e1c7e251ffd758c81b526f1664bd3, bbe4caa1bf895e476ba054fdc7f5099c1638f192
pre-processing of science files: 2ddf97b9df594661500e2ba36b836d22483f2c2f

Full support of IRDIS LSS mode is now completed!


[cluster-test] Add full node support to perf testing

Motivation
We want to add full nodes to cluster testing. This PR adds full nodes into the instances list in cluster test, so that different experiments can run their experiment with either a validator set or a fullnode set.
This PR should only be separating out the validator and full node instances, and all the tests should still run with validator instances.
Test Plan
Push docker image of cluster test and run cluster test
docker tag cluster-test:latest<PHONE_NUMBER>86.dkr.ecr.us-west-2.amazonaws.com/libra_cluster_test:<your tag>
docker push<PHONE_NUMBER>86.dkr.ecr.us-west-2.amazonaws.com/libra_cluster_test:<your tag>
docker pull<PHONE_NUMBER>86.dkr.ecr.us-west-2.amazonaws.com/libra_cluster_test:<your tag>
ct -i<PHONE_NUMBER>86.dkr.ecr.us-west-2.amazonaws.com/libra_cluster_test:fullnode --run performance_benchmark_nodes_down -- --num-nodes-down=0 --is-fullnode=false
Successful run
Login Succeeded
Dec 19 22:15:54.506 INFO Discovered 100 peers in sunmilee workspace
Dec 19 22:15:54.815 INFO Log tail thread started in 309 ms
Dec 19 22:15:54.815 INFO Will use sunmilee tag for deployment
Dec 19 22:15:55.170 INFO Starting experiment all up
Dec 19 22:15:56.168 INFO Completed minting seed accounts
Dec 19 22:15:58.042 INFO Mint is done
Dec 19 22:20:00.382 INFO Experiment finished, waiting until all affected validators recover
Dec 19 22:20:05.391 INFO Experiment completed
Dec 19 22:20:05.392 INFO Experiment Result: all up : 587 TPS, 1187.4 ms latency
Have you read the Contributing Guidelines on pull requests?
Yes

:umbrella: The latest upstream changes (presumably 3eea4369f9d969b4aa23a2af0adfcc2bd792dcd6) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r+

:pushpin: Commit f85e4af has been approved by ankushagarwal

:hourglass: Testing commit f85e4af5b05949517b61fb7f4aef605a683e3e50 with merge d8160214eb77e2d3df802d1f0089d8c60d04ec60...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: ankushagarwal
Pushing d8160214eb77e2d3df802d1f0089d8c60d04ec60 to master...


[WIP] Improve performance of ConstrainedReschedule pass

Summary
This PR reimplements the ConstrainedReschedule transpiler pass to run in linear time, by going over the nodes once and pushing back the start times in case of misalignments and overlaps.
Details and comments
Temporarily, the modified version of the code resides in transpiler/passes/scheduling/alignments/reschedule_improved.py, so that I can run both versions and compare the output circuits.
Here is my python script (essentially copy-pasted from #10049):
from qiskit.transpiler.passes import ALAPScheduleAnalysis, TimeUnitConversion, InstructionDurationCheck, ConstrainedReschedule, PadDelay
from qiskit.transpiler.target import Target, InstructionProperties
from qiskit import transpile
from qiskit.transpiler import PassManager
from qiskit.circuit.random import random_circuit
from qiskit.circuit.library.standard_gates import *
from qiskit.circuit import Parameter, Measure, Delay
import rustworkx as rx
import numpy as np
from time import time

from qiskit.transpiler.passes.scheduling.alignments.reschedule_improved import ConstrainedRescheduleNewImproved

def compare_runs():
    rng = np.random.default_rng(seed=12345678942)

    # g = rx.generators.directed_heavy_hex_graph(13)
    g = rx.generators.directed_heavy_hex_graph(5)

    target = Target(
        num_qubits=len(g),
        min_length=16,
        pulse_alignment=8,
        acquire_alignment=16,
        granularity=8
    )
    rz_props = {}
    x_props = {}
    sx_props = {}
    measure_props = {}
    delay_props = {}
    id_props = {}
    for i in range(len(g)):
        qarg = (i,)
        rz_props[qarg] = InstructionProperties(error=0.0, duration=0.0)
        x_props[qarg] = InstructionProperties(
            error=rng.uniform(1e-6, 1e-5), duration=rng.uniform(1e-8, 9e-7)
        )
        sx_props[qarg] = InstructionProperties(
            error=rng.uniform(1e-6, 1e-5), duration=rng.uniform(1e-8, 9e-7)
        )
        measure_props[qarg] = InstructionProperties(
            error=rng.uniform(1e-6, 1e-5), duration=rng.uniform(1e-8, 9e-7)
        )
        delay_props[qarg] = None
        id_props[qarg] = None

    target.add_instruction(XGate(), x_props)
    target.add_instruction(SXGate(), sx_props)
    target.add_instruction(RZGate(Parameter("theta")), rz_props)
    target.add_instruction(Measure(), measure_props)
    target.add_instruction(Delay(Parameter("t")), delay_props)
    target.add_instruction(IGate(), id_props, name="id")
    target.add_instruction(
        ECRGate(),
        {
            edge: InstructionProperties(
                error=rng.uniform(1e-6, 5e-5), duration=rng.uniform(1e-8, 9e-7)
            )
            for edge in g.edge_list()
        }
    )

    durations = target.durations()

    qc = random_circuit(50, 5, measure=True, seed=202342)

    schedule_current = PassManager([
        TimeUnitConversion(durations),
        ALAPScheduleAnalysis(durations),
        InstructionDurationCheck(target.acquire_alignment, target.pulse_alignment),
        ConstrainedReschedule(target.acquire_alignment, target.pulse_alignment),
        PadDelay(),
    ])

    schedule_improved = PassManager([
        TimeUnitConversion(durations),
        ALAPScheduleAnalysis(durations),
        InstructionDurationCheck(target.acquire_alignment, target.pulse_alignment),
        ConstrainedRescheduleNewImproved(target.acquire_alignment, target.pulse_alignment),
        PadDelay(),
    ])

    tqc = transpile(qc, target=target)

    start_time = time()
    tqc_current = schedule_current.run(tqc)
    end_time = time()
    print(f"Original scheduling: {end_time - start_time:0.2f}")

    start_time = time()
    tqc_improved = schedule_improved.run(tqc)
    end_time = time()
    print(f"Improved scheduling: {end_time - start_time:0.2f}")

    assert tqc_current == tqc_improved

if __name__ == "__main__":
    compare_runs()

For the above quantum circuit, on my laptop the current code runs in 444.25 seconds, while the optimized code runs in 0.43 seconds.
@mtreinish  has further suggestions to improve runtimes based on  https://github.com/Qiskit/qiskit-terra/pull/10051/commits/bd3cbb271030b3d88c1ee36691b623a1714348c1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 78 (2.56%)  changed or added relevant lines in 2 files are covered.
16 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.03%) to 85.835%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/scheduling/alignments/reschedule_improved.py
0
76
0.0%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
2
91.14%

qiskit/extensions/quantum_initializer/squ.py
2
80.0%

crates/accelerate/src/sabre_swap/layer.rs
4
97.32%

crates/accelerate/src/vf2_layout.rs
8
86.44%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
71250

Relevant Lines:
83008

💛  - Coveralls

For posterity, performance from example in #10067 looks much better.
Before #10077:

After:


[Menu] If header is in full size, clicking on an item in the left menu doesn't display the right bloc.

Actual behaviour
If header is in full size, clicking on an item in the left menu doesn't display the right bloc(the scroll is too low).
This error doesn't exist when the header is in the "scroll state".
(see the file in attachement)
Expected behaviour
An item in the left menu must display the right bloc everytime.
Versions
Focus : 0.7.5

:+1:


Missing syntax error for VALUE operator

TYPES ty_strings TYPE STANDARD TABLE OF string WITH EMPTY KEY.
DATA(words) = VALUE ty_strings(
  ( `All` )
  ( `ABAP` )
  ( `constructs` )
  ( `are` )
  ( `imperative` ) ).

"Table comprehension into helper table
DATA(switched_words) = VALUE i_do_not_exist(
      FOR word IN words
        ( SWITCH #( word WHEN `All`        THEN `Some`
                        WHEN `imperative` THEN `functional`
                        ELSE word ) ) ).

WRITE switched_words.

The linter does not complain about the missing type "i_do_not_exist":

the name i_do_not_exist is outside the errorNamespace, ie. it becomes void
if you want errors for everything, the linter must know everything, then change the errorNamespace to blank = regex matches everything

closing


Make COI-serviceworker optional?

This PR https://github.com/crystal-bit/godot-game-template/pull/89 fixed the web deploy.
However this configuration should not be the default and should be opt-in: if you are deploying a web game and you can set the appropriate HTTP headers it's better to configure them that way instead of relying on https://github.com/gzuidhof/coi-serviceworker
See also https://github.com/godotengine/godot-proposals/issues/6616

I agree with this. Right now, if you run the project from within Godot, you'll get a 404 error in the browser console because this file wasn't added to the project. Making it optional or making it only get applied on GH Pages would be really useful

@SecurityRisk I'd like to know your opinion on #93. I left some comments there.

Relevant: https://godotengine.org/article/dev-snapshot-godot-4-3-dev-3/#single-threaded-web-exports


The link to the raw data download cannot be accessed. Would you please re-upload it?

thanks!!!

可以回复一下么？谢谢

Please use this new link to download the data. https://iastate.box.com/s/ouz0rjjb6g1b5queqtpr6pee5upb8s5r

Please use this new link to download the data. https://iastate.box.com/s/ouz0rjjb6g1b5queqtpr6pee5upb8s5r

Please use this new link to download the data. https://iastate.box.com/s/ouz0rjjb6g1b5queqtpr6pee5upb8s5r

thanks

The link to the raw data download cannot be accessed. Would you please re-upload it again?

The link to the raw data download cannot be accessed. Would you please re-upload it again?

Please send an email to me<EMAIL_ADDRESS>to access the link.


Not working with latest Akka NuGets

The Akka.Logger.log4net NuGet is not compatible with recent Akka NuGets. I suspect it is due to changes in ILoggingAdapter, where methods have been moved to an extension class.
Please make a new NuGet available with updated Akka and log4net NuGets.
[ERROR][07.25.2023 07:49:48.881Z][Thread 0025][akka://xxx/system/log1-Log4NetLogger] Method not found: 'Void Akka.Event.ILoggingAdapter.Info(System.String, System.Object[])'. Cause: System.MissingMethodException: Method not found: 'Void Akka.Event.ILoggingAdapter.Info(System.String, System.Object[])'.    at Akka.Logger.log4net.Log4NetLogger.<.ctor>b__2_8(InitializeLogger m)    at lambda_method12(Closure, Object, Action1, Action1, Action1, Action1, Action1)
at Akka.Tools.MatchHandler.PartialHandlerArgumentsCapture6.Handle(T value)    at Akka.Actor.ReceiveActor.ExecutePartialMessageHandler(Object message, PartialAction1 partialAction)
at Akka.Actor.ReceiveActor.OnReceive(Object message)
at Akka.Actor.UntypedActor.Receive(Object message)
at Akka.Actor.ActorBase.AroundReceive(Receive receive, Object message)
at Akka.Actor.ActorCell.ReceiveMessage(Object message)
at Akka.Actor.ActorCell.Invoke(Envelope envelope)`

Is this repo still maintained?


Big Play Button colour

Would be nice to have ability to set the colour of the Big Play Button in a similar way to how the colour of the controls can be set.

I was able to get a workaround by changing the color via the border-color of q-media--big-button in the STYLE section, and using the slot ... but that introduced other issues.

Fixed in v1.0.6

You know, you shouldn't close until it's released and you've tested... ;)


Change required zeep version

For async features required

The project now uses the latest available version of zeep, API is slightly changed though and the package has been renamed to aonvif.


Add a Gradio app for dataset annotation

This PR adds a gradio page to invoke-train-ui for creating datasets and captioning images.

Before we add anything else, think performance on this needs to be checked. Currently loading images at ~10s/image from an SSD

Unclear where this issue is coming from but

AttributeError: Caught AttributeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\worker.py", line 308, in _worker_loop
data = fetcher.fetch(index)
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\transform_dataset.py", line 23, in getitem
example = self._base_dataset[idx]
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\image_caption_jsonl_dataset.py", line 79, in getitem
image = self._load_image(self._get_image_path(idx))
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\image_caption_jsonl_dataset.py", line 53, in _get_image_path
image_path = self._jsonl_path.parent / image_path
AttributeError: 'str' object has no attribute 'parent'

This is an error from training when using a .jsonl dataset generated using the data annotation tab.
All lines of .jsonl are formatted as below --
{"image": "C:\Users\kentr\invoke-training\datasets\prophoto\1293.jpg", "text": "a man with sunglasses, sunset, backlit"}

Before we add anything else, think performance on this needs to be checked. Currently loading images at ~10s/image from an SSD

Discussed in-person. This is an issue with large image files. Possibly related to https://github.com/gradio-app/gradio/issues/2635. We are going to punt on addressing this for now.

df50169

Fixed in df50169. Looks like it was caught by the unit tests in the failing build 🎉

Unclear where this issue is coming from but

AttributeError: Caught AttributeError in DataLoader worker process 0.
Original Traceback (most recent call last):
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\worker.py", line 308, in _worker_loop
data = fetcher.fetch(index)
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in fetch
data = [self.dataset[idx] for idx in possibly_batched_index]
File "C:\Users\kentr\invoke-training\venv\lib\site-packages\torch\utils\data_utils\fetch.py", line 51, in
data = [self.dataset[idx] for idx in possibly_batched_index]
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\transform_dataset.py", line 23, in getitem
example = self._base_dataset[idx]
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\image_caption_jsonl_dataset.py", line 79, in getitem
image = self._load_image(self._get_image_path(idx))
File "C:\Users\kentr\invoke-training\invoke-training\src\invoke_training_shared\data\datasets\image_caption_jsonl_dataset.py", line 53, in _get_image_path
image_path = self._jsonl_path.parent / image_path
AttributeError: 'str' object has no attribute 'parent'

This is an error from training when using a .jsonl dataset generated using the data annotation tab.
All lines of .jsonl are formatted as below -- {"image": "C:\Users\kentr\invoke-training\datasets\prophoto\1293.jpg", "text": "a man with sunglasses, sunset, backlit"}

Fixed in df50169. Looks like it was caught by the unit tests in the failing build 🎉


add full screen option to screenshare

it should be possible to fullscreen a screenshare
We have it for video playlist already and the button looks like this -

The way to do this would be changing the VideoTile here - https://github.com/100mslive/hms-video-react/blob/main/src/components/VideoTile/VideoTile.tsx to add a small fullscreen icon which acts similar to video playlist screenshare button. Here is the function which is used by the video playlist component which can be traced back to learn more - https://github.com/100mslive/hms-video-react/blob/c90fb6a62e7532a6e7dabe006b015c8e056c0f7e/src/utils/index.ts#L643

@triptu Can I work on this?

sure. Assigned to you.

Also can you help me reproduce the playlist fullscreen feature?

Is the 100ms server down? It isn't loading any data in my developer dashboard.

Can you send the link you're trying?

Yeah sure

Yes, it takes in a boolean showScreenshare which can be used to decide if the full-screen option needs to be shown.

@triptu I am abandoning this issue for now.


Refactor config file

It would be useful to do some changes to the config files to make them more portable:

geodiff_exe - let's remove this option - it is more of a local installation detail rather than a user configuration - dbsync should simply expect geodiff in PATH
working_dir - let's remove this option - like above, it is a detail of the installation - best to simply use a working dir automatically in a temp folder (probably no need to have env variable to override it)
init_from - let's add this option with two possible values: gpkg and db - right now users need to specify those manually when running dbsync.py or dbsync_daemon.py, but it makes more sense to have it a part of config

Fixed in #103


Add a config option to disable automatic config loading

I want to be able to load the default config file on demand.
This is useful for static builds that are bundled with the executable.

Hmm...
You mean something like ./Configure  no-autoload-config or something?

I think I should squash the commits in order to change the commit message in the first commit to:
Add a config option to disable automatic config loading

./config no-autoload-config

@richsalz do you like it now?


feat: allow Codestar ARN to be used for CodePipeline

what

Allows a codepipeline_codestar_connection_arn to be set as a variable, which passes through to the terraform-aws-ecs-codepipeline module.

why

Using the CodeStar connection is recommended by AWS, and reduces the risk of somebody accidentally committing a Github Oauth token.
This also creates the CodePipeline "Source" step to use the Github v2 integration, which can be helpful.

references

Related to https://github.com/cloudposse/terraform-aws-ecs-codepipeline/issues/51

/test all


16.2.1-alpha.1: Module not found: Error: Can't resolve 'jszip' in \node_modules\devextreme\client_exporter

I'm getting the following error with 16.2.1-alpha.1
ERROR in ./~/devextreme/client_exporter/excel_creator.js
Module not found: Error: Can't resolve 'jszip' in 'C:\temp\test-issue\node_modules\devextreme\client_exporter'
 @ ./~/devextreme/client_exporter/excel_creator.js 15:12-28
 @ ./~/devextreme/client_exporter.js
 @ ./~/devextreme/viz/core/export.js
 @ ./~/devextreme/viz/chart_components/base_chart.js
 @ ./~/devextreme/viz/chart.js
 @ ./~/devextreme-angular2/ui/chart.js
 @ ./~/devextreme-angular2/index.js
 @ ./src/app/app.module.ts
 @ ./src/app/index.ts
 @ ./src/main.ts
 @ multi main

I'm using angular-cli with this setup:
// angular-cli.json
      "scripts": [
        "../node_modules/jquery/dist/jquery.min.js",
        "../node_modules/devextreme/dist/js/dx.all.js"
      ],

// app.module.ts
  imports: [
    BrowserModule,
    (...)
    DevExtremeModule
  ],

In 16.1.7 this works fine.
I also tried to change the scripts section of angular-cli.json to
// angular-cli.json
      "scripts": [
        "../node_modules/jquery/dist/jquery.min.js",
        "../node_modules/devextreme/dist/js/jszip.min.js",
        "../node_modules/devextreme/dist/js/dx.all.js"
      ],

but the problem remains.

Hello,

In 16.2.1-alpha.1, it is no longer required to include the following scripts as global into angular-cli.json:
      "scripts": [
        "../node_modules/jquery/dist/jquery.min.js",
        "../node_modules/devextreme/dist/js/jszip.min.js",
        "../node_modules/devextreme/dist/js/dx.all.js"
      ],

All devextreme-angular2 dependencies will be loaded by the module loading mechanism.
See also: #178

Sorry @kvet but I removed all scripts from angular-cli.json and the problem remains.
Please see the attached sample gh177-issue.zip. This is a clean project created with
ng init
npm install<EMAIL_ADDRESS>--save

and added DevExtremeModule to app.module.ts.

Some of our components depend on the "jszip" library. When you use DevExtremeModule, you need to add the "jszip" library to your project. You can do this with npm:
npm install jszip --save

I am still getting this issue in version 18.1.3 ?

I am getting this error as well in version 18.1.3

I'm getting same error in 18.1.6 for command devextreme-bundler


Group scripts w/ pre/post

This makes sure that pretest always appears before test etc.

Yes! Great work @jkrems :smile:

1.2.0 :+1:


Added StringName as a variant type.

Also changed all relevant properties defined manually to StringName.

Why? Not trying to question this change, I'm just curious.

@aaronfranke A lot of functinos in the Godot API take StringNames because they are much faster to compare and process than regular strings (yet they can be identified as such). Examples are signal names, method names, property names, class names, animation names, shader parameter names, skeleton bone names,  translations keys, etc (the list is very  long actually). They are exposed as StringName in the C++ API but the script API only sees them as simple strings.
By not having this datatype, the script API is forced to convert to StringName every time a function is called, which involves hashing and entire string comparisons every time, significantly reducing performance.
StringNames are effectively a different data type with a different use case, because they are meant to have a textual meaning, but  be mostly compared, hashed, etc. quickly. They also can't be used for string operations, because that is not their purpose (you need to convert them to regular strings first).

Thanks!


load_data.ipynb

Hi there, it looks like  load_data.ipynb file is missing from the repo. This was raised to us by someone at CN who was looking through this repository. Thank you

The missing file has been added.


NIFI-6412 Upgraded Spring dependencies

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
This PR upgrades the versions of spring-framework and spring-security used throughout some parts of the project.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Will review...

Also, one thing I noticed is the version of spring-security-core that gets bundled in nifi-kerberos-iaa-providers-nar is quite old. There appears to be a version conflict and the old version is taking precedence. We could create a follow on JIRA to investigate this one if you want to since it's a little outside the scope of the original JIRA.

Unfortunately there appears to be no newer release of spring-security-kerberos-core:
https://mvnrepository.com/artifact/org.springframework.security.kerberos/spring-security-kerberos-core/1.0.1.RELEASE
https://github.com/spring-projects/spring-security-kerberos/tree/master


how do i install this to tshock?

Downloaded but how do i run this with tshock

unzip it to ServerPlugins


"Game Over" text does not disappear when hitting "Try again" button or "New Game" button

Browser:
Chrome Version 45.0.2454.93 (64-bit)
Issue:
When Game Over, I expect the semi-transparent overlay with the text "Game Over" to disappear after I hit the "Try again" button or the "New Game" button, but it does not disappear. However, moving the arrow keys after hitting the "Try again" button or "New Game" button will remove the "Game Over" text together with the semi-transparent overlay.

@KlavierCat thanks for reporting, I'll get it fixed in the next round of edits!

Resolved with b30e6a7.


vue-router 3.0.5 incorrectly encodes route on async load (but works fine on 3.0.4)

Version
3.0.5
Reproduction link
https://codesandbox.io/s/1znn98wx47
Steps to reproduce
https://codesandbox.io/s/1znn98wx47
Follow instructions by navigating to ComponentB by clicking on the link. If you don't immediately see the problem, then click the refresh icon to simulate reloading assets/page (because the componentB must async load and not already be stored in cache).
Notice that the route path is now URL encoded.
What is expected?
Route path should not be URL encoded. In most browsers, the URL address bar up top will actually also change the URL as well to this encoded version, which obviously is unsightly for people who care about a clean URL
What is actually happening?
Route path is URL encoded. This has the unfortunate side effect of changing the URL bar in chrome. If you visit domain.com/a/b/c it will change to domain.com/a%2Fb%2Fc which is undesirable.

This works fine in 3.0.4. The regression is due to this line change
https://github.com/vuejs/vue-router/blob/6974a6fa56412794c932c6ad9bfe2cebcc3a7da3/src/util/resolve-components.js#L33

I'm reverting back that change and adding some extra tests


Renovate CLI switch parsing

This commit makes a handful of closely-related changes, all aimed at refactoring bin/http-server for better maintainability.

Switched from deprecated optimist to the smaller minimist for CLI parsing
Used aliases to link together switches with both a - and -- option, thereby removing the need for patterns like argv.g || argv.gzip
Set defaults on almost all switches. This can make debugging easier and makes default values more clear
Reformatted the help dialog
Clarified the help dialog to take a sidestep around #450. As noted in https://github.com/indexzero/http-server/issues/200#issuecomment-241065301, False is a string, so it should not work on a boolean switch. Instead, this new help dialog encourages using the --no prefix. For example, -i enables autoIndex, and --no-i disables it. This is already possible, so it's not actually an interface change
As far as I can tell, this PR introduces no user-facing changes except for the help dialog

I am officially laying this PR down in favor of nconf in #131, it's very well done over there.


Add syntax highlighting

Hi, thanks for this great project.
One thing that would be very useful for me is the syntax highlighting for the generated code (could be also for smithy, but the former one is more important).
I wonder if you have already thought about it. I am willing to work on it, but didn't want to double any potential work that might have already been put into that.
Let me know if you have any ideas how to implement it or maybe you know of some limitations that such feature should obey.
There seem to be quite a few libraries to chose from: https://blog.logrocket.com/exploring-best-syntax-highlighting-libraries/ with highligth.js being the most popular.
Also, given that left-hand-side is actually an editor maybe it would be a good idea to just replace the current input-form with something more capable like Monaco or codemirror. Wdyt?

It's a great idea. I definitely thought about it but did not work on that at all. Feel free to pick it up if you want to.
As for guidance, I don't have anything specific in mind. The only thing I'd ask is that we have two separate PRs:

one where syntax highlighting is enabled on the code viewer side
one where an in-browser editor like monaco is enabled on the code editor side

The two will probably be implemented using npm dependencies directly, like @kubukoz did for https://github.com/daddykotex/smithy4s-code-generation/pull/16
I'll be happy to review and help for the PR if you push you're changes in a branch I can see and contribute to when you get to it
again, thank you very much

I second the request to have separate PRs. We should also bikeshed monaco vs other editors like Codemirror.


sveltekit usage

I really like this library and wanted to try using it with sveltekit but I failed, when importing a component it fails at:
import Toolkit from './src/base/Toolkit/Toolkit';

it fails to find this exact file and does not try to load the Toolkit.svelte file, I imagine there is a way somewhere to configure this but I have no f***ing clue where... Each time I try to do anything with frontend dev I end up depressing even more in its curent state...
Do you have any idea on how this work with sveltekit ?

I can't make it work either without sveltekit, I tried:
npx degit sveltejs/template my-svelte-project
cd my-svelte-project
yarn add -D svelte-toolkit

and then adding 	import { Button } from "svelte-toolkit";
and <Button>Click me</Button>  but it fails:
Could not resolve './src/icons/ArrowLeft' from node_modules/svelte-toolkit/main.js


[Bug] Failed on building python example

I'm trying to build python example of Iroha client lib and get some errors when running examples/python/prepare.sh:
[ 60%] No install step for 'jtv_libpqxx'
[ 63%] No test step for 'jtv_libpqxx'
[ 63%] Completed 'jtv_libpqxx'
[ 63%] Built target jtv_libpqxx
CMakeFiles/Makefile2:3258: recipe for target 'shared_model/bindings/CMakeFiles/irohapy.dir/rule' failed
make[1]: *** [shared_model/bindings/CMakeFiles/irohapy.dir/rule] Error 2
Makefile:903: recipe for target 'irohapy' failed
make: *** [irohapy] Error 2

prepare.sh script seems outdated because command
cd example
cmake .. -DSWIG_PYTHON=ON && make -j1

works for me.
System environment
Ubuntu 16.04
Iroha #d4f4e3ca3351bb92be99be7c3df440dc4bb8738a
Additional information
GCC 7.2.0
Boost 1.65.1
SWIG 3.0.12

You can close this PR. It seems outdated in the new dev branch.


Using digest image assets

I'm trying to figure out if there is a way to use this awesome gem together with digested image assets from Rails.
Basically I'd like to keep digesting images to improve caching but be able to somehow include them in my react components.
Any ideas?

Hi @hajder
Normally i would suggest to use https://github.com/alexspeller/non-stupid-digest-assets and whitelist images that you would like to share with react components.
Anyway, i was looking for a way to handle digest path somehow. Fastest but ugly solution is to create a rails helper that find asset and return its digest path:
  def digest_image(path)
    "assets/#{Rails.application.assets.find_asset(path).digest_path}"
  end

and pass images to components by props:
<%= react_component('myComponent', { data: @data, logoPath: digest_image('logo.svg')}) %>

Passing image path each time is not elegant solution so we will consider third option - a plugin which will use generated by sprocets public/.manifest-xxx.json file. By using manifest it's possible to create a map of nondigest and digest files and provide helper that will resolve them. Having such helper on js side should solve this problem.
Let us know what do you think about it.

Hey @hajder :)
Sorry for late response.
We decided against creating a plugin to our gem to fix this issue since the issue is not directly connected with the gem or React but with how the digested assets work in rails 4. The fact that rails won’t let you compile assets to both digested and non digested version means that your js framework (be it ember, angular, react, backbone etc) will alway have trouble displaying said assets.
Possible solutions:

you can create a map of digested_path => logical_path, pass it to your js and use a helper that resolves logical path to digested path within your components. This is quite ugly.
you can use https://github.com/alexspeller/non-stupid-digest-assets but this is not really recommended solution since this gem monkey patches sprockets. If the sprockets ever change in a way that breaks the gem it may cause some problems on your servers.
You can use a rake task that triggers after the assets precompilation. The task fetches the assets map from the sprockets manifest file and copies digested assets to non digested paths. This is actually the solution that we use in our project and so far it is working fine. Granted that it is still dependant on the sprockets manifest file structure but the manifest is less likely to change the the sprockets gem itself. So here is how it looks like in our setup:

task :create_non_digest_assets => :"assets:environment"  do
logger = Logger.new($stderr)
excluded_extensions = ['css', 'js']
manifest_path = Dir[File.join(Rails.root, 'public/assets/.sprockets-manifest-*.json')].first
manifest_data = JSON.load(File.new(manifest_path))

manifest_data['files'].each do |digested_path, attributes|
  next if digested_path.ends_with?(*excluded_extensions)

  logical_path = Pathname.new attributes['logical_path']
  full_digested_path = File.join(Rails.root, 'public/assets', digested_path)
  full_nondigested_path = File.join(Rails.root, 'public/assets', logical_path)

  logger.info "Copying asset to #{full_nondigested_path}"

  FileUtils.copy_file full_digested_path, full_nondigested_path, true
end

end
We exclude css and js files since we wont really need them within our components. This approach also has the advantage of not having to use any helpers or resolvers within the react components and makes assets work regardless if they are digested or not.
For the further and more in depth read you can checkout this guide -> https://bibwild.wordpress.com/2014/10/02/non-digested-asset-names-in-rails-4-your-options/
Hope it solves your issue :)


Escape webpackContext.id value

What kind of change does this PR introduce? Bugfix
Summary When using HashedModuleIdsPlugin along with a require.context() statement, value of this.id would be a string. It was previously passed unescaped which would compile improperly.
Does this PR introduce a breaking change? No
Other information
I couldn't find a test case for this module and I'm not sure where I would create it. I'm happy to try though if someone tells me where to create the test case within the tests/ folder hierarchy.

Yes please we'd like a test case let me know if you need any direction.

I'll try to write one, where would be a good place to create the test case for this module?

test/configCases

see my PR

Thanks @sokra!


Feature/#21 kotlinx

Closes #21

What do you guys think about using an extension like so:
suspend inline fun <reified T> ApplicationCall.respondPaged(message: PageResponse<T>) {
    response.pipeline.execute(this, JsonSettings.toJson(message))
}

With this the call would be cleaner instead of this:
call.respond(
    JsonSettings.toJson(
         PageResponse.from(
             pageRequest = pageRequest,
             totalElements = totalElements,
             data = data,
             path = call.request.path()
         )
    )
)

After:
call.respondPaged(
    PageResponse.from(
       pageRequest = pageRequest,
       totalElements = totalElements,
       data = data,
       path = call.request.path()
   )
)

The main reason for requiring the JsonSettings.toJson(message) is because of signature of the method of Ktor's
ApplicationCall.respond(message: Any)

Because this methods accepts Any type erasure occurs and the generic type definitions is lost.

Just so you know Content-Type is being returned as text/plain instead of application/json in the /cards endpoint, probably because we are returning a string from the JsonSettings.toJson(PageRequest<T>) call.

@pedrohh You're right, so even without my suggestion this still happens when calling:
JsonSettings.toJson(
         PageResponse.from(
             pageRequest = pageRequest,
             totalElements = totalElements,
             data = data,
             path = call.request.path()
         )
    )


update script.md

updating tests.md path.

The code block changed in this PR updates the metadata of the page. Usually metadata keys are in lowercase.
Hence closing this PR.


cilium-ca does not exist when cilium is installed via helm, thus causing cilium clustermesh connect to fail

Example helm config:
$ helm install -n cilium --create-namespace cilium cilium/cilium --version 1.10.0 --set clustermesh.useAPIServer=true

Example secret query in one of the clusters:
$ kubectl get secrets -n cilium
NAME                                TYPE                                  DATA   AGE
cilium-operator-token-x6d45         kubernetes.io/service-account-token   3      17h
cilium-token-v2zpj                  kubernetes.io/service-account-token   3      17h
clustermesh-apiserver-admin-cert    kubernetes.io/tls                     3      17h
clustermesh-apiserver-ca-cert       Opaque                                2      17h
clustermesh-apiserver-remote-cert   kubernetes.io/tls                     3      17h
clustermesh-apiserver-server-cert   kubernetes.io/tls                     3      17h
clustermesh-apiserver-token-zzzpp   kubernetes.io/service-account-token   3      17h
default-token-ttwr8                 kubernetes.io/service-account-token   3      17h
hubble-ca-secret                    Opaque                                2      17h
hubble-generate-certs-token-trcrk   kubernetes.io/service-account-token   3      17h
hubble-relay-client-certs           kubernetes.io/tls                     3      17h
hubble-relay-token-ltjw6            kubernetes.io/service-account-token   3      17h
hubble-server-certs                 kubernetes.io/tls                     3      17h
hubble-ui-token-tn2zg               kubernetes.io/service-account-token   3      17h
sh.helm.release.v1.cilium.v1        helm.sh/release.v1                    1      17h

Trying to mesh cluster A and B:
$ cilium -n cilium clustermesh connect --context $clusterA --destination-context $clusterB
✨ Extracting access information of cluster workload-centralus...
🔑 Extracting secrets from cluster workload-centralus...
❌ Unable to retrieve access information of remote cluster "workload-centralus": unable to get secret "cilium-ca" to retrieve CA: secrets "cilium-ca" not found
Error: Unable to connect cluster: unable to get secret "cilium-ca" to retrieve CA: secrets "cilium-ca" not found

I assume clustermesh-apiserver-ca-cert is what we're looking for in this case, but I figured you all would like to know about this.

Also having trouble when running $ cilium hubble enable --ui:
$ cilium hubble enable --ui
❌ Cilium CA not found: unable to get secret kube-system/cilium-ca: secrets "cilium-ca" not found

Error: Unable to enable Hubble:  unable to get secret kube-system/cilium-ca: secrets "cilium-ca" not found

doesn't happen with $ cilium install process.

After manually setting the cilium-ca secret, I ran into another discrepancy: The certificates generated with the helm cronjob are singular, e.g. "clustermesh-apiserver-server-cert" and the names used in cilium-cli are plural, e.g. "clustermesh-apiserver-server-certs".

Also seeing this via cilium hubble enable after cilium is installed via helm, which is the only documented way to install cilium chained to vpc-cni on EKS.

Both cilium hubble enable and cilium clustermesh enable have  --create-ca. If you enable them, does it work for you? We can consider probably auto-detecting the CA and enable the flag when we detect it's needed.

Thanks. I didn't know about --create-ca. I notice that cilium hubble enable can fail because any of several resources already exist (with or without --create-ca), in which case the only recourse is to cilium hubble disable before retrying.

cilium hubble enable --create-ca does the trick, but is a bit silly. Shouldn't this be just automatic or at least mentioned in the docs since otherwise it will initially fail for everyone.

As Sebastian said, we could detect the CA and automatically enable the flag. We'd welcome a pull request implementing this approach.

cilium hubble enable --create-ca does the trick, but is a bit silly. Shouldn't this be just automatic or at least mentioned in the docs since otherwise it will initially fail for everyone.

Also note, if cilium is installed via cilium install, then --create-ca is should not be necessary. To my knowledge, the case where the certificate is missing happens when Cilium was installed via Helm. We still highly recommend also enabling Hubble via Helm instead of via Cilium CLI if Helm is involved, as mixing the two approaches will likely confuse Helm as well.

I did install via helm and hubble was supposed to be enabled (it's the default), but the cert was missing (I found out from the logs).  the relay and the ui is off by default, I'll try that now.

Hello, Any updates about this issue ?

I ran across the cilium hubble enable CA missing issue as well.
I've filed this PR to allow cilium-cli to generate a CA if it is missing. Since we know that Hubble/Certmesh will break if the cilium-ca secret is missing, and we can determine whether it exists, it makes for a better user experience to automatically create it. This PR should also address #17340 which is a separate issue than this, since this issue is for the helm installation not generating a CA as expected.
The difference between these two issues is where the CA secret gets generated. The Helm installation uses a tool called Certgen
to generate certificates, whereas cilium-cli generates its own certificates. Certgen creates a separate CA secret for both Hubble and Certmesh, while cilium-cli uses the same secret for both. This second PR should address this issue and bring certgen in line with cilium-cli.
Hopefully, these two PRs will help provide a nice user experience.


FLOAT-FEATURES are loaded in a very very dramatic manner

https://github.com/Zulu-Inuoe/jzon/blob/90c9eed6f4b4efaa290ce1d5fc763b8aa6e2a9db/src/eisel-lemire.lisp#L1-L11
Isn't an ASDF dependency on float-features enough?

That won't do anything when loaded through ASDF. It just made it easier to test the code on different optimization settings etc

wait did I really open two issues for that? aaaa I'm sorry I only meant to make one of them