text:
Applying prettier to all files

Scope
Run prettier on all changes, not just those under bridge/
Why?
More standardized code formatting.

@vtleonardo This should cover a few files from your PR.

Codecov Report

Merging #32 (b0354b1) into main (cde9061) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   45.04%   45.02%   -0.02%
==========================================
  Files         220      220
  Lines       40002    40002
==========================================
- Hits        18018    18012       -6
- Misses      20021    20023       +2
- Partials     1963     1967       +4

Impacted Files
Coverage Δ

badgerTrie/smt_merkle_proof.go
60.15% <0.00%> (-4.69%)
:arrow_down:

Continue to review full report at Codecov.

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

authors: codecov-commenter madhivedan

repo: alicenet/alicenet


text:
Grocery V1

The description is hidden for privacy reasons ;)

This code sucks.

No it doesn't!

source: gharchive/pull-request

license: MIT

license_type: permissive

license_source: github-api


text:
restrictions on experiments with MLR mbo

User should not be able to do following at the same time:

provide 3-4 changeable variables with default or larger intervals (e.g. 2 to 12)
select mlr mbo strategy with very small number of iterations and random starts

if these are observed, then mlr mbo gives an error related with "experiment space", thus resulting in an error state.
to be discussed with Ilias & Janek

implemented here

source: gharchive/issue

repo: alinaciuysal/OEDA


text:
Support for. NET8

Details
Please Support for. NET8

thank

Hi @samchenws,
I'm planning to drop net5 support and add net8.0 in a few days. wait for the next minor version.

hi @alirezanet,
Gridify currently work as well in .net 8. just need to update EF dependency version ...

Thank you. Can we expect a new Nuget package version soon?

Hi @agriffard,
I'll release a new version tomorrow.

authors: agriffard alirezanet mo3in samchenws

repo: alirezanet/Gridify


text:
httpdns: request http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH fail

while  i run the demo,  i got this issue:
2021-12-15 10:03:50.936 22461-22628/com.aliyun.ams.httpdns.demo D/httpdns: request host www.taobao.com with type v4 extras : null cacheKey null
2021-12-15 10:03:50.937 22461-22628/com.aliyun.ams.httpdns.demo D/test: from sdk: [D]request host www.taobao.com with type v4 extras : null cacheKey null
2021-12-15 10:03:50.937 22461-22628/com.aliyun.ams.httpdns.demo D/httpdns: host www.taobao.com result is null
2021-12-15 10:03:50.938 22461-22628/com.aliyun.ams.httpdns.demo D/test: from sdk: [D]host www.taobao.com result is null
2021-12-15 10:03:50.938 22461-22628/com.aliyun.ams.httpdns.demo D/httpdns: start async ip request for www.taobao.com v4
2021-12-15 10:03:50.939 22461-22628/com.aliyun.ams.httpdns.demo D/test: from sdk: [D]start async ip request for www.taobao.com v4
2021-12-15 10:03:50.940 22461-22628/com.aliyun.ams.httpdns.demo I/httpdns: request host www.taobao.com and return empty immediately
2021-12-15 10:03:50.941 22461-22628/com.aliyun.ams.httpdns.demo D/test: from sdk: [I]request host www.taobao.com and return empty immediately
2021-12-15 10:03:50.941 22461-22628/com.aliyun.ams.httpdns.demo D/tt: cost time: 5
2021-12-15 10:03:50.942 22461-25083/com.aliyun.ams.httpdns.demo D/httpdns: request url http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH
2021-12-15 10:03:50.943 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk: [D]request url http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH
2021-12-15 10:03:50.959 22461-25083/com.aliyun.ams.httpdns.demo E/httpdns: request http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH fail, cost 15
2021-12-15 10:03:50.959 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk: [W]request http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH fail, cost 15
2021-12-15 10:03:50.960 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk:
2021-12-15 10:03:50.961 22461-25083/com.aliyun.ams.httpdns.demo W/System.err: java.lang.ArrayIndexOutOfBoundsException: length=0; index=0
2021-12-15 10:03:50.961 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.d.d.a(Unknown Source:8)
2021-12-15 10:03:50.962 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.interpret.n.a(Unknown Source:32)
2021-12-15 10:03:50.962 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.g.a(Unknown Source:50)
2021-12-15 10:03:50.963 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.963 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.l.a(Unknown Source:0)
2021-12-15 10:03:50.964 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.f.a(Unknown Source:2)
2021-12-15 10:03:50.964 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.a.run(Unknown Source:0)
2021-12-15 10:03:50.965 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
2021-12-15 10:03:50.965 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
2021-12-15 10:03:50.966 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.lang.Thread.run(Thread.java:923)
2021-12-15 10:03:50.966 22461-25083/com.aliyun.ams.httpdns.demo D/httpdns: request url http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH
2021-12-15 10:03:50.966 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk: [D]request url http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH
2021-12-15 10:03:50.979 22461-25083/com.aliyun.ams.httpdns.demo E/httpdns: request http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH fail, cost 13
2021-12-15 10:03:50.980 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk: [W]request http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH fail, cost 13
2021-12-15 10:03:50.980 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk:
2021-12-15 10:03:50.980 22461-25083/com.aliyun.ams.httpdns.demo W/System.err: java.lang.ArrayIndexOutOfBoundsException: length=0; index=0
2021-12-15 10:03:50.980 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.d.d.a(Unknown Source:8)
2021-12-15 10:03:50.981 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.interpret.n.a(Unknown Source:32)
2021-12-15 10:03:50.981 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.g.a(Unknown Source:50)
2021-12-15 10:03:50.981 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.981 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.l.a(Unknown Source:0)
2021-12-15 10:03:50.982 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.f.a(Unknown Source:2)
2021-12-15 10:03:50.982 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.a.run(Unknown Source:0)
2021-12-15 10:03:50.982 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
2021-12-15 10:03:50.982 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
2021-12-15 10:03:50.983 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.lang.Thread.run(Thread.java:923)
2021-12-15 10:03:50.983 22461-25083/com.aliyun.ams.httpdns.demo W/System.err: java.net.UnknownHostException: Unable to resolve host "null": No address associated with hostname
2021-12-15 10:03:50.983 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.net.Inet6AddressImpl.lookupHostByName(Inet6AddressImpl.java:124)
2021-12-15 10:03:50.984 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.net.Inet6AddressImpl.lookupAllHostAddr(Inet6AddressImpl.java:103)
2021-12-15 10:03:50.984 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.net.InetAddress.getAllByName(InetAddress.java:1152)
2021-12-15 10:03:50.984 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.Dns$1.lookup(Dns.java:41)
2021-12-15 10:03:50.984 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.RouteSelector.resetNextInetSocketAddress(RouteSelector.java:178)
2021-12-15 10:03:50.985 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.RouteSelector.nextProxy(RouteSelector.java:144)
2021-12-15 10:03:50.985 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.RouteSelector.next(RouteSelector.java:86)
2021-12-15 10:03:50.985 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.StreamAllocation.findConnection(StreamAllocation.java:176)
2021-12-15 10:03:50.986 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.StreamAllocation.findHealthyConnection(StreamAllocation.java:128)
2021-12-15 10:03:50.986 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.StreamAllocation.newStream(StreamAllocation.java:97)
2021-12-15 10:03:50.986 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:289)
2021-12-15 10:03:50.987 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:232)
2021-12-15 10:03:50.987 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.huc.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:465)
2021-12-15 10:03:50.987 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.huc.HttpURLConnectionImpl.getResponse(HttpURLConnectionImpl.java:411)
2021-12-15 10:03:50.988 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.android.okhttp.internal.huc.HttpURLConnectionImpl.getResponseCode(HttpURLConnectionImpl.java:542)
2021-12-15 10:03:50.988 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.c.a(Unknown Source:81)
2021-12-15 10:03:50.988 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.988 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.g.a(Unknown Source:18)
2021-12-15 10:03:50.988 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.989 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.g.a(Unknown Source:18)
2021-12-15 10:03:50.989 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.989 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.g.a(Unknown Source:18)
2021-12-15 10:03:50.989 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.h.a(Unknown Source:2)
2021-12-15 10:03:50.990 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.l.a(Unknown Source:0)
2021-12-15 10:03:50.990 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.f.a(Unknown Source:2)
2021-12-15 10:03:50.990 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at com.alibaba.sdk.android.httpdns.g.a.run(Unknown Source:0)
2021-12-15 10:03:50.990 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
2021-12-15 10:03:50.991 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
2021-12-15 10:03:50.991 22461-25083/com.aliyun.ams.httpdns.demo W/System.err:     at java.lang.Thread.run(Thread.java:923)
2021-12-15 10:03:50.992 22461-25083/com.aliyun.ams.httpdns.demo E/httpdns: ip request for www.taobao.com fail
2021-12-15 10:03:50.992 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk: [W]ip request for www.taobao.com fail
2021-12-15 10:03:50.992 22461-25083/com.aliyun.ams.httpdns.demo D/test: from sdk:

http://null:80/100000/d?host=www.taobao.com&sdk=android_0.0.0.1-local-SNAPSHOT&sid=PSzzR8nQVUdH
You need config server ip in gradle.properties. You can try 203.107.1.33 to run demo.
INIT_SERVERS={"203.107.1.33"}

authors: GreatDongsong new-xd

repo: aliyun/alibabacloud-httpdns-android-sdk


text:
Fix UnboundLocalError: local variable 'verbs' referenced before assig…

Error created in commit 0d057cd

Thanks! I had forgotten about this project, and abandoned it in the middle of whatever I was doing.

Thanks. Really appreciated.

authors: DaVinci789 allanburleson clicketyclack

repo: allanburleson/python-adventure-game

license: mit

license_source: bigquery


text:
Added Linux ZFS support.... Detect FreeBSD 10 or later

Okay, so I primarily use FreeBSD, but I do have a Linux box and a couple VM's that use ZFS.  So I added an OS check condition for Linux so that unsupported properties are dealt with.  At least I was only able to determine that jailed,aclmode,shareiscsi,and volmode don't work when receiving from FreeBSD.
As well, I noticed that the latest version was restricting current versions of FreeBSD even though aclmode and aclinherit are now supported.  So to ensure this was not just a Linux adaptation of an amazing tool, I added a really dumb check to differentiate between FreeBSD >= 10 and <10 (using the first number of the release version, so it isn't future-proofed after FreeBSD 19...)
This is my first attempt at something like this, so any feedback is appreciated!

Oh, and to further show my allegiance primarily being to FreeBSD, I factored in the significance of adding Linux support as only being worthy of a bump to version 1.1.7.  A huge Canadian fan of BSD Now too!

repo: allanjude/zxfer


text:
Skip step AdamW

This PR implements SkipStepAdamW. It's not ideal since it doesn't support fused ops or similar at present. I changed default AdamW to also use the same underlying implementation

At this point I've made so many other relevant changes to make this work that it might be easier to just merge the 32B when we're back

repo: allenai/OLMo-core


text:
Vanishing/Exploiting gradients ?

Hi,
I am training in a new corpus/language and during training it shows good perplexity but at some point the perplexity jumped so high.
It was something between 7-10, and then it jumped to 15699
Is there any side effect of vanishing/exploiting gradients you have seen with this model ?
I am using the default setting except I have changed the unroll_steps to 50 and the vocab size to fit my problem.
Loading data from: /home/ga53huq2/projects/bio/unsupervised_amino_acid_embedding/dataset/file9115.txt                                                                                [803/1946]
Loaded 500000 sentences.
Finished loading
Batch 673300, train_perplexity=6.99748
Total time: 1041813.8715116978
Batch 673400, train_perplexity=7.26806
Total time: 1041936.777597189
Batch 673500, train_perplexity=7.09751
Total time: 1042059.83791399
Batch 673600, train_perplexity=6.75356
Total time: 1042182.6843624115
Batch 673700, train_perplexity=7.06632
Total time: 1042305.2833411694
WARNING:tensorflow:Error encountered when serializing lstm_output_embeddings.
Type is unsupported, or the types of the items don't match field type in CollectionDef.
'list' object has no attribute 'name'
Batch 673800, train_perplexity=6.99326
Total time: 1042454.4103929996
Batch 673900, train_perplexity=7.05139
Total time: 1042577.2035264969
Batch 674000, train_perplexity=7.0321
Total time: 1042700.0349740982
Batch 674100, train_perplexity=6.96487
Total time: 1042822.9305524826
Loading data from: /home/ga53huq2/projects/bio/unsupervised_amino_acid_embedding/dataset/file9218.txt
Loaded 500000 sentences.
Finished loading
Batch 674200, train_perplexity=8.39033
Total time: 1043664.7385673523
Batch 674300, train_perplexity=7.96308
Total time: 1043785.5559742451
Batch 674400, train_perplexity=8.09743
Total time: 1043906.0904071331
Batch 674500, train_perplexity=8.30678
Total time: 1044027.1834490299
Batch 674600, train_perplexity=7.90479
Total time: 1044148.736148119
Batch 674700, train_perplexity=8.11415
Total time: 1044270.838372469
Batch 674800, train_perplexity=10.2219
Total time: 1044392.4975090027
**Batch 674900, train_perplexity=15699.9
Total time: 1044514.349034071
Batch 675000, train_perplexity=90.8147
Total time: 1044636.6984901428**
WARNING:tensorflow:Error encountered when serializing lstm_output_embeddings.
Type is unsupported, or the types of the items don't match field type in CollectionDef.
'list' object has no attribute 'name'
Batch 675100, train_perplexity=13.7765
Total time: 1044782.6224155426
Batch 675200, train_perplexity=13.6507
Total time: 1044903.5013656616
Batch 675300, train_perplexity=13.4189
Total time: 1045024.7266552448
Batch 675400, train_perplexity=12.9115
Total time: 1045146.2300972939
Batch 675500, train_perplexity=13.3054
Total time: 1045267.7772848606
Batch 675600, train_perplexity=12.9654
Total time: 1045389.3125679493
Batch 675700, train_perplexity=12.879
Total time: 1045511.3902630806
Batch 675800, train_perplexity=12.5727
Total time: 1045633.0763440132
Batch 675900, train_perplexity=12.4994
Total time: 1045754.6204769611
Batch 676000, train_perplexity=12.0309
Total time: 1045876.4980325699
Batch 676100, train_perplexity=12.4572
Total time: 1045998.2082695961
Batch 676200, train_perplexity=12.3322
Total time: 1046119.7888958454

Maybe you have some strange and not homogeneous training data.

authors: PhilipMay agemagician

repo: allenai/bilm-tf


text:
Do we need to add system prompt when training/evaluating RM?

Hi,
I want to know if we should add a system prompt when training and evaluating our reward models. For instance, I have supervised fine-tuned LLaMA2-7B on the Alpaca SFT 10k subset, and I use the same chat template for the tokenizer as the one in LLaMA2-7B-Chat. The system prompt used in Meta's paper is shown in the screenshot below, which is included manually during my SFT procedure.
My understanding is that since we add a system prompt when collecting responses from LLaMA2 during RLHF, the system prompt becomes part of the overall prompt seen by LLaMA2. Therefore, it seems logical to include the system prompt in the query sent to the reward model to better assess the quality of the generated responses. However, if we apply the tokenizer's chat template, as is the default behavior in Reward-Bench, the system prompt might not be included.
Could you please clarify if the system prompt should be added during the training and evaluation of reward models?

Thank you!

Hey @hank0316 to the best of my knowledge, no, but you can likely do it.
Reward models normally use the same chat template from IFT which allows it, but it's not well studied.

repo: allenai/reward-bench


text:
kb_ents gives no results from custom KB

Following this discussion #383, where I got my custom KB to work.
I tried to test the code and for some reason it is not giving me anything.
Here is the code I tested it with:
linker = CandidateGenerator(name="myCustom")
text = "TR Max Velocity: 2.3 m/s"
doc = nlp(text)
spacy.displacy.render(doc, style = "ent", jupyter = True)

entity = doc.ents[2]
print("Name: ", entity)

for umls_ent in entity._.kb_ents:
    print(umls_ent)
    print(linker.kb.cui_to_entity[umls_ent[0]])
    print("----------------------")

This would give:
Name:  m/s

there was no ---------------------- which means it did not even enter the for loop.
I was wondering why this is the case.
If this helps, this is the jsonl file that I ran this script (https://github.com/allenai/scispacy/blob/master/scripts/create_linker.py) with:
...
{"concept_id": "U0013", "aliases": ["m/s"], "types": ["UN1T5"], "canonical_name": "m/s"}
...

I'd have to dig into this a bit more to debug, but you might want to open your tfidf vectors file and make sure that m/s is actually present. i suspect it might not be because its a character trigram vectorizer, and probably normalize out the /. I'm not 100% sure of this, but the things to check are 1) m/s is recognized as an entity (which you did) 2) m/s is present in your kb (you did) 3) check the representation that the vectorizer produces if you just pass it that text (just load the vectorizer object separately and poke around)

Hello Daniel, thanks for the reply. However, I am not sure how to load the vectorizer separately.
I tried doing this:
CustomLinkerPaths_mycustom = LinkerPaths(
    ann_index="",
    tfidf_vectorizer="output/tfidf_vectorizer.joblib",
    tfidf_vectors="",
    concept_aliases_list="",
)
class myKnowledgeBase(KnowledgeBase):
    def __init__(
        self,
        file_path: str = "custom_kb.jsonl",
    ):
        super().__init__(file_path)

DEFAULT_PATHS["myCustom"] = CustomLinkerPaths_mycustom
DEFAULT_KNOWLEDGE_BASES["myCustom"] = myKnowledgeBase

nlp.add_pipe("scispacy_linker", config={"resolve_abbreviations": True, "linker_name": "myCustom"})
linker = CandidateGenerator(name="myCustom")

but it gave me a FileNotFoundError: file  not found
I also tried loading the joblib file but I am not sure where to go from here:
import joblib
joblib.load('output/tfidf_vectorizer.joblib')
#TfidfVectorizer(analyzer='char_wb', dtype=<class 'numpy.float32'>, min_df=10, ngram_range=(3, 3))

I also tried the same script but with different units (without the "/") like: kg, bpm, ml and all of them gave the same results.
text = "TR Max Velocity: 2.3 kg"
doc = nlp(text)
spacy.displacy.render(doc, style = "ent", jupyter = True)

entity = doc.ents[2]
print("Name: ", entity)

for umls_ent in entity._.kb_ents:
    print("xx")
    print(umls_ent)
    print(linker.kb.cui_to_entity[umls_ent[0]])
    print("----------------------")

would give:
Name:  kg

so I am not sure if "/" is the problem.

Can you 1) make sure that the linker is actually in your pipeline (nlp.pipeline) and 2) try changing the filtering, so set filter_for_definitions=False and threshold=0.1

I did what you told me to:
nlp = spacy.load("new_model")
nlp.add_pipe("scispacy_linker", config={"resolve_abbreviations": True, "linker_name": "myCustom",
                                        "filter_for_definitions": False, "threshold": "0.1"})
linker = CandidateGenerator(name="myCustom")
print(nlp.pipe_names)
print(nlp.pipeline)

which returned:
['tok2vec', 'tagger', 'parser', 'attribute_ruler', 'entity_ruler', 'ner', 'value_entity', 'scispacy_linker']
[('tok2vec', <spacy.pipeline.tok2vec.Tok2Vec object at 0x7f3fad155ef0>), ('tagger', <spacy.pipeline.tagger.Tagger object at 0x7f3fb0732220>), ('parser', <spacy.pipeline.dep_parser.DependencyParser object at 0x7f3fabf96d00>), ('attribute_ruler', <spacy.pipeline.attributeruler.AttributeRuler object at 0x7f3fb0fbb980>), ('entity_ruler', <spacy.pipeline.entityruler.EntityRuler object at 0x7f3fb46b9840>), ('ner', <spacy.pipeline.ner.EntityRecognizer object at 0x7f3fabf96520>), ('value_entity', <function value_entity at 0x7f3fb1121b80>), ('scispacy_linker', <scispacy.linking.EntityLinker object at 0x7f3fabefc430>)]

Showing that the linker is in my pipeline, then I ran the same sentence as above but I still got the same result.

Ok, at this point I would suggest setting a breakpoint inside the entity linker and stepping through one line at a time until you can see why there are no entities returned. I'm not sure if it is because there is something wrong with your nmslib index, or some other issue

From debugging I noticed that the difference with my knowledge base compared to the UMLS one from scispacy is that there are no batch_neighbors and batch_distances when it calls candidate_generation.py from linking.py at line 116:
 batch_candidates = self.candidate_generator(mention_strings, self.k)

Do you know why this is the case? is a possibility because my KB only has 25 entries (as I was just experimenting on how to use a custom KB in scispacy). Would this problem disappear with more entries in the KB. Another thing to note was it only took around 1s for me to run the script to make the ann_index and in the discussions it normally takes hours.

25 entries is very few entries, UMLS has ~4 Million as far as I remember

Ah so most likely the problem lies there, alright then I'll try with a larger KB and update.
But as far as the process to add the custom KB, is what I am doing correct?

I'd be a bit surprised if the small number of entries was the problem, but I am not intimately familiar with nmslib. One thing to try is to load your nmslib index separately (https://github.com/allenai/scispacy/blob/3d153ddad1f11f000f961f7a92c0d862b93c0973/scispacy/candidate_generation.py#L116), and then see if you can play around with it following their docs (starting point here, https://nmslib.github.io/nmslib/api.html#nmslib-floatindex), and just make sure it ended up with the entries you expected and everything looks reasonable.

I did some more digging and I am not sure if the problem is the ann_index from the function above.
I found out that the tfidf values from
https://github.com/allenai/scispacy/blob/3d153ddad1f11f000f961f7a92c0d862b93c0973/scispacy/candidate_generation.py#L324
give different results.
So here is the result from the normal UMLS KB:

and here is the result of the custom KB:

This value affected the batch_neighbors and batch_distances when it was called in line 329:
batch_neighbors, batch_distances = self.nmslib_knn_with_zero_vectors(tfidfs, k)

Using the custom KB, https://github.com/allenai/scispacy/blob/3d153ddad1f11f000f961f7a92c0d862b93c0973/scispacy/candidate_generation.py#L251
would give the value of 1, so that made the if statement below True since vectors was tfidf that was passed to the function:  self.nmslib_knn_with_zero_vectors(tfidfs, k)
https://github.com/allenai/scispacy/blob/3d153ddad1f11f000f961f7a92c0d862b93c0973/scispacy/candidate_generation.py#L263-L264
and made them return empty arrays of neighbors and distance.
Is the vectorizer the problem then? or am I missing something?

the vectorizer construction (https://github.com/allenai/scispacy/blob/3d153ddad1f11f000f961f7a92c0d862b93c0973/scispacy/candidate_generation.py#L415) has a min_df of 10, meaning that any ngram has to be in at least 10 docs to be included in the vectorizer. you can change this if you want to test on smaller data.

Yep, that was the problem. What do you mean it has to be in at least 10 docs? when we run the script create_liner.py  we don't pass any text to it right?

a "doc" here is a concept name. the tfidf vectorizer is fit on all the concept names in the kb using character trigrams. so any character trigrams that do not occur in at least 10 docs will not be included in the vectorizer. you can of course change this 10 to something lower if you want.

authors: MichalMalyska danielkingai2 farrandi

repo: allenai/scispacy


text:
🛑 Autism York is down

In bd52eb8, Autism York (https://autismyork.org) was down:

HTTP code: 500
Response time: 614 ms

Resolved: Autism York is back up in a2d685c after 1 hour, 7 minutes.

repo: alliancewebteam/uptime


text:
NPM package being used?

We're currently looking to open-source a project over at B12 and wanted to name it alligator as well. Is this package still actively used? If not, could you transfer the npm rights to us? Thanks in advance for your consideration!

I'm sorry, I can't give the name. I'm still working on this project

No problem, thanks for letting me know!

authors: alligator-io jrbotros

repo: alligator-io/alligator


text:
goaccess memory usage limit

Hi
goaccess looks good, i want to try it to use for our radio page, unfortunately our VPS has limited memory (only 4GB)
i have nginx access log of 6GB size with approx. 20 milion of lines inside.
i cant run this with default storage, because it does not fit into RAM, and goaccess process is killed by kernel
goaccess -s
In-Memory with On-Disk Persistent Storage.

goaccess --output /tmp/report.html radiopunctum.cznginx.access.log --persist --restore --config-file /home/punctum/.goaccessrc --db-path /home/punctum/tmp/
Killed

is there some way how to limit memory usage, or, maybe better, to use disk storage instead of In-Memory?
i am looking forward to your answer

If you're looking to reduce some RAM usage, here are a few things you could try out. First off, do you know if your requests are timestamped? If so, you might want to consider passing -q. Also, are you currently using the legacy GeoIP? If yes, give -g a shot. Another option is to SSH into the remote machine and parse the log locally on your own laptop/desktop machine (assuming it's got more RAM), e.g.,
ssh -n root@server 'tail -F -n +0 /var/log/apache2/access.log' | goaccess - --log-format=COMBINED -o report.html --real-time-html

Note: SSH requires -n so GoAccess can read from stdin. Also, make sure to use SSH keys for authentication as it won't work if a passphrase is required.

thanks, locally parsing is OK, that works, but i want to have this completely on server

if your requests are timestamped

i am not sure what you mean by timestamped
my log record look like this:
31.30.175.177 - - [12/May/2022:02:19:02 +0000] "GET /api/radio/programme/playingnow/?_nc=1652321942&device=desktop HTTP/2.0" 200 485 "https://radiopunctum.cz/archive/20220426_chaosfera" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/101.0.4951.54 Safari/537.36"

so this is probably not timestamp, only date/time of request
mainly what i asked for, is this part of man:

STORAGE
There are three storage options that can be used with GoAccess. Choosing one will depend on your environment and needs.
Default Hash Tables
In-memory storage provides better performance at the cost of limiting the dataset size to the amount of available physical memory. GoAccess uses in-memory hash tables. It has very good memory usage and pretty good performance. This storage has support for on-disk persistence.

i cannot find how to tell goaccess which storage to use, and what are that three options.

I'm not certain how that text managed to stick around from an older version that stored data on disk, but I've gone ahead and updated the documentation to match the current versions. Did using -q make a difference? In your situation, using -g won't be beneficial since you're utilizing the newer geodb.

authors: allinurl tyctor

repo: allinurl/goaccess


text:
Single pass - multiple outputs

It would be awesome to be able to specify multiple formats with information where to output that data.
This way it would be possible to process logs once and get three different output files.
Right now getting html, csv and json output requires parsing the same input three times.
Trying to paralelize it is usually problematic due to memory constraints.
I know this feature would break current functionality (output to stdout).
(I haven't tried on-disk storage, using default in-memory)

Thanks for the suggestion.
Outputting multiple files formats will be addressed in issue #343, the plan is to output an HTML file, and have a UI option to output a CSV or the raw JSON. Thoughts?
As far as parsing multiple log formats and outputting it to one single file, will be added by issue #213.

I think #53 has a good idea about adding -o file.html.
Embedding json/csv within html would be an option, but then it would require to add extra tools to extract it from it.
My current use case is to process logs from AWS ELB, that's something about 10~15GB per day. In-memory data storage uses something about 8 ~10GB, and so far memory is a major concern when doing stuff in parallel.
That's why multiple outputs on single pass would be ideal, of course to certain degree.
I also complied goaccess with on-disk storage with default settings - it is something like 4x slower than in memory (no wonder). So now I can parse it slowly once and use database for different formats, or I can execute all three output formats in parallel - much faster but for the cost of memory.
i haven't got time to play with extra parameters for on-disk storage, maybe for the price of memory I could just make things much faster.
Of course for parsing large data sets like month it will require on-disk storage, but then usually it is not needed to get it very fast.

I can certainly look into it and hope I can ship this in v1.0. It shouldn't be too difficult to have this implemented since all outputs consume the same data structure.
As far as the on-disk storage. It's definitely slower than the default storage. As you said, playing with --cache-* and --tune- can help. At some point, I'll go over this storage and see what can be done to speed it up (keeping in mind that data has to be committed to disk).

Playing with on-disk storage options really requires deeper knowledge about Tokyo Cabinet in general. From what I have quickly scanned over the internet it usually drills down to the issues with journaled file system beneath. Would need expansion of the docs and some tests.

Added the ability to output multiple formats in one run.
In addition, it standardizes the output per issue #53. For instance,
it now supports the following.
goaccess -f access.log -o /www/file.html
goaccess -f access.log -o /www/file.csv
goaccess -f access.log -o /www/file.json
For backwards compatibility, it still supports the following format:
goaccess -f access.log > /www/file.html
goaccess -f access.log -o csv > /www/file.csv
goaccess -f access.log -o json > /www/file.json

does it mean it supports something like this?
goaccess -f access.log -o /www/file.html -o /www/file.csv -o /www/file.json

That's correct.

awesome :D

authors: allinurl nvtkaszpir


text:
Adding Vagrantfile

Set up an environment to build goaccess from github

Merged. Thanks!

@allinurl  Wow, It seems I've failed my second commit message :/ I fixed it on my fork, is it possible to update the PR or do I have to create a new one? I'm sorry for that, we can let it as it stands if you prefer...

Please submit another PR. Thanks.

authors: allinurl mthoretton


text:
[Bug] Numeric type audit in transaction and block structs and provider methods, remove gratuitous U256s

Component
consensus, eips, genesis
What version of Alloy are you on?
No response
Operating System
None
Describe the bug
Noticed discrepancy while implementing From<TypedTransaction> for TransactionRequest
all types should have the following (with options as necessary)
applies to
consensus
rpc
providers return values
etc
block_number: u64

transaction_index: u64

gas_price: u128

// make sure names are consistent here too. these are the same field
gas: u128
gas_limit: u128

max_fee_per_gas: u128

max_priority_fee_per_gas: u128

max_fee_per_blob_gas: u128

value: U256

closed by 454

repo: alloy-rs/alloy


text:
Outdated changelog.md

The changelog reflects just reflects the version 2.3.11 but the current version is 2.4.1

Yeah @almasaeed2010, could you pleas update it? I just did an upgrade and the menu does not work any more (collapsing not working).
Even if semver is not fully respected here, it would be great to have some info. :+1:

authors: Soullivaneuh isantolin


text:
Standalone bootstrap doesn't work on generate-launcher.sh

I added the option to make a standalone launcher (-s) in generate-launcher.sh, however, the resulting launcher still downloads JARs on its initial run.

The generate-launcher.sh script was a deprecated script from the previous version (0.3.x), I just updated it. Note that passing it the -s option makes the corresponding kernel fail to start for now (because of the way the classpath is handled by Ammonite, the underlying scala interpreter, which seems to be fine with the in-memory classpath of the standalone launcher). It should be fix-able though.

Okay thanks! just wondering, would there be any potential issues using a launcher generated by the deprecated script?

Closing as a duplicate of https://github.com/almond-sh/almond/issues/94.

authors: alexarchambault maclockard

repo: almond-sh/almond


text:
Use async polling to update the front-end for spark progress bar

Spark progress bar front end's updates and polled once a second instead of pushed by each task start/end.
Should fix #448
Tested using
spark.range(10000000).repartition(5000).count

Works both locally and on a cluster with typical spark jobs.

@darkjh Going to merge as is, just tell me if anything doesn't work for you. (spark.range(10000000).repartition(5000).count was still sped up for me, with the latest changes)

authors: alexarchambault darkjh


text:
Browser Example

Browser example for the README.md file:
Browser Example
import {TemplateHandler} from 'easy-template-x';
import save from 'save-file';
import JSZip from 'jszip';
import JSZipUtils from 'jszip-utils';
loadFile = (url,callback) => {
JSZipUtils.getBinaryContent(url,callback);
}
myEasyTemplate = () => {
this.loadFile("myTemplate.docx",function(error,content){
if (error) { throw error };
    const data = {
        posts: [
            { author: 'Easy', text: 'Very important\ntext here!' },
            { author: 'Easy', text: 'Forgot to mention that...' }
        ]
    };

    const handler = new TemplateHandler();
    handler.process(content, data).then(doc =>
    {
        save.sync(doc, "easy-output.docx");
    });
})

}

Hi,
Thanks for the offer!
I understand the need for having a full browser example. But ended up adding another example since I prefer having an example that doesn't require the user to download an additional library. It's visible now in the readme file.

authors: alonrbar incowia-danilo

repo: alonrbar/easy-template-x


text:
[BUG] Не ставится заявка в спред в стакане PRO

Опишите баг
Пробовал ставить заявку в спред мышью.
Она просто не ставится , ничего не происходит
Воспроизведение
Создать стакан PRO
Найти большой спред попробовать выставить заявку мышью
Среда.
Винда 7
Хром
Ожидаемое поведение
Заявки должны мышью ставится в спред
Скриншоты
No response
Дополнительная информация
No response
Правила

[X] Я согласен следовать правилам

В спред заявка не ставится потому что при текущем виде стакана мы не можем без запроса дополнительной информации определить ставить заявку на ПОКУПКУ или ПРОДАЖУ

Когда описывал клавиши мышью , изначально было что лкм в бидах или в спреде - покупка , правая продажа по рынку. а в офферах правая кнопка ставила лимитки и в спред так же можно было выставиться.
Но с вашей стороны посчитали что так не удобно и сделали по другому.
При такой работе мышью, как сейчас , лимитку выставить не возможно.

в ТВ очень большой стакан , нам место важно. Чем больше стаканов в момент тем лучше.
можем оставить так как есть , будем по клавишам заходить ( хотя может это будет и не удобно кому-то)

Вот задача по кликам https://github.com/alor-broker/Astras-Trading-UI/issues/239.
Переделана была только логика чтобы за один тип заявки отвечали одинаковые кнопки мыши, а не переворачивались в зависимости от зоны продавцов и покупателе.
Про спред речи не было. Если будет полное описание какую заявку выставлять при клике по спреду на ПОКУПКУ или ПРОДАЖУ, то мы и при текущей реализации это можем сделать.
Однако 95% появятся пользователи, которые скажут, что им неудобно при клике выставлять заявку только на продажу или только на покупку

У виджета есть предел ширины, когда он еще читается, например такой

даже при таком сжатом состоянии ячейки занимают 20px ширины - вполне достаточно, чтобы при наведении на левую/правую часть строки подсветить надписи Sell/BUY и усранить неопределенность. Такое можно сделать хотя бы для спрэда. И тогда проблема будет устранена

Я вообще за экономию места, очень мало стаканов помещается в данный момент на экране.
Если реализовывать подобное , то например если навести на панель объема это лимитка на покупку в спреде, а если навести на панель собственных заявок (которою очень хочется уменьшить)  то лимитка на продажу. Так наверное будет нормально

Это совершенно точно не баг, поэтому надо открывать задачу, подробно описать как новую фичу и уже там продолжить обсуждение.

authors: PilgrimViis sbelashevskiy uncrashable

repo: alor-broker/Astras-Trading-UI


text:
cargo-audit: lock to version 0.20.0

Description
Lock cargo audit used in actions to v0.20.0 to workaround issue:
cargo audit
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 664 security advisories (from /Users/sapinb/.cargo/advisory-db)
    Updating crates.io index
    Scanning Cargo.lock for vulnerabilities (1235 crate dependencies)
The application panicked (crashed).
Message:  invalid Cargo.lock dependency tree: Resolution("failed to find dependency: substrate-bn 0.6.0 (git+https://github.com/sp1-patches/bn?tag=substrate_bn-v0.6.0-patch-v2)")
Location: /Users/sapinb/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cargo-audit-0.21.0/src/presenter.rs:114

Backtrace omitted. Run with RUST_BACKTRACE=1 environment variable to display it.
Run with RUST_BACKTRACE=full to include source snippets.

https://github.com/rustsec/rustsec/issues/1249#issuecomment-2423257490

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Refactor
[ ] New or updated tests

Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented my code where necessary.
[ ] I have updated the documentation if needed.
[ ] My changes do not introduce new warnings.
[ ] I have added tests that prove my changes are effective or that my feature works.
[ ] New and existing tests pass with my changes.

Related Issues

I've followed the issue so that when it's solved we can revert this.
Thanks @sapinb :)

authors: sapinb storopoli

repo: alpenlabs/strata


text:
Fix broken rake tasks

content_id and base_path are directly accessible from taxons using the new taxonomy gem.

LGTM 👍

authors: MatMoore suzannehamilton

repo: alphagov/content-tagger


text:
[#100513836] Percentages should be less than 100

Fixes this bug: https://www.pivotaltracker.com/story/show/100513836
I've re-used the existing not_a_number validations key because it gives the correct validation message from the SSP content.

:+1:

authors: TheDoubleK quis

repo: alphagov/digitalmarketplace-api


text:
Allow all declarations to be changed

Allow supplier declarations to be changed regardless of what state the framework is in. I have discussed this with Cath and the declaration can change at any point in the process but who changes must be controlled. The who is managed by the frontend applications.

:+1:


text:
Add form for user to request editor upgrade

Add a form for users to request an upgrade from trial user to editor
Trello card: https://trello.com/c/cgDgv0Dw/1036-add-initial-account-upgrade-page
For early access, trial users need to be able to request an editor account from the engagements team.
This requirement is met with a new page /upgrade. It contains some text and a form with a confirmation box, which when filled in shows the user a screen /upgrade/requested telling the user that they will receive an email with further instructions.
These pages are not linked to by any pages yet and so should be safe to deploy to production.
Notes
We don't store that a user has requested an upgrade and there is nothing to stop users accessing /upgrade/requested directly. Submitting the form multiple times would result in multiple emails.
I think this makes sense as an MVP and it matches the designs. We haven't specified what happens if a user doesn't receive the email etc and these things would need extra logic. It should be easy to add in the future if we decide it's something we want to add.
This commit does send the email but stubs out a simple service class to add this in.
I've made bits of text up for the page titles and error message as they weren't on the design. I'm happy to make adjustments to the content in later commits.
There also aren't many tests beyond the feature tests. I think this is ok because there isn't much happening here - we can add further tests when we flesh out the service class. I could be convinced otherwise if there are strong opinions though.
Screenshots
The upgrade page:

url: /upgrade
page title: Request upgrade to editor account

If the checkbox is not ticked when submitting the form:

error message: You must meet the requirements

The confirmation page:

url: /upgrade/requested

Things to consider when reviewing

Does it work when run on your machine?
Is it clear what the code is doing?
Do the commit messages explain why the changes were made?
Are there all the unit tests needed?
Has all relevant documentation been updated?

Looks good overall, have just raised a small curly quote suggestion. Another very minor thing but think the PR message should say:

This commit doesn't send the email but stubs out a simple service class to add this in.

Instead of:

This commit does send the email but stubs out a simple service class to add this in.

Content here is fine for now. I need to iterate all of it following user research, but that will take me a few days and I should be able to manage that myself once the files are there for me to edit. Thanks Tom!

authors: DavidBiddle hannahkc thomasiles

repo: alphagov/forms-admin


text:
Removes references to the draft stack

The draft stack, which existed in a separate vCloud organisation to our
main stack, is now deprecated and has been turned off.

:+1:

authors: bradleywright mattbostock

repo: alphagov/gds-boxen


text:
Add optional max_cache_time to travel_advice(_index)…

We’d like to specify a smaller cache time for
Travel Advice content. This change introduces a
max_cache_time field that can optionally be set.
There will be some subsequent work in Content
Store to respect this field.
We can extend this out to other formats if needed
in the future.

Needs a rebase.

@danielroseman rebased

authors: danielroseman tuzz

repo: alphagov/govuk-content-schemas


text:
Move to Nunjucks for templating

This is to address issue #30 - Mustache can be restrictive when prototyping.
Nunjucks is a JavaScript implementation of Jinja. It is close to Mustache in syntax, but it gives us more power and flexibility, and most importantly it's currently maintained and documented:
https://mozilla.github.io/nunjucks/
This pull request includes:

add the Jinja version of GOV.UK template
remove Mustache template, libraries, and related Grunt tasks
update all templates to use Jinja syntax
update the docs

Known issues:

the Jinja version of the GOV.UK template is not published as an NPM module, as the Mustache version was. It is currently just manually copied to /lib

It’s possible to use Bower to pull in the Jinja version of the template and then use Grunt to copy the assets and layouts. Your call if the extra complexity is worth the trade off.

Great work on this Joe. I've not tried yet, but will try to take a look later.

just realised the commit history is a mess for some reason, sigh git, will try and clean up

@quis cheers, will look into it

Awesome work Joe. Will have a play.

Would be really good to get this merged.
Once concern I have: what happens if any existing users run git pull from this repo? Will their prototypes break? Is there any way of warning these users?

We can have a nice error message that points them to the docs on how to convert from mustache to nunjucks
Yeah sorry for the delay in this. Every time templates get updated elsewhere, I have more work to do to convert them :/

Can this be closed @joelanman?

authors: edwardhorsford joelanman quis timpaul


text:
Table names amended

What
Accessibility Audit
Why
Table captions were not unique and sufficiently descriptive.
Link to JIRA card (if applicable):
GW-1912

Could you please squash the commits before merge?

authors: koetsier mel-am

repo: alphagov/govwifi-admin


text:
Remove hard coded path to file

Fix tests
Update Google credentials with proper key

I know this is a relatively throwaway repo, but in the interests of good commit hygiene I would have expected this to be two separate commits (one to fix the creds, one to fix the tests).

Not forgotten about these comments, I'll get them sorted next.

authors: bishboria bradwright

repo: alphagov/inventory-tool


text:
Make Terraform version check more robust

What
This awk monstrosity (that I introduced in 50bb83f) was not working with
gawk 3.1.8 or mawk 1.3.3 (as available on Ubuntu Precise). It turns out
that the regex interval expressions ({1,4} etc) only work with gawk 3
if you specify the --posix flag. They're enabled by default with gawk
4+.
Instead of attempting to resolve this in a portable way, I've instead
reimplemented this check in ruby, which is arguably much more readable.
How to review
Test that make check-tf-version works as expected.
Who can review
Anyone but myself.

Terraform have added this a feature since we first did it:

https://www.terraform.io/docs/configuration/terraform.html

Could we use that instead?

Oooo interesting. I hadn't seen that. I'll give it a go.

Alex, please let me know when you are done with improvements.

@combor This is now ready for review.
Note: this is currently based on #797, so includes the commit from that. We can either merge this one, or I can rebase after #797 is merged.

I'll review the other one first.

You an rebase now.

rebased.

OK, so this LGTM. Merging.

authors: alext combor dcarley

repo: alphagov/paas-cf


text:
Update start page content for state pension

This PR supersedes #2182
Updates content on the start page for the state pension calculator.
Expected changes

Calculate State Pension
Should now start with "Use the calculator to:" followed by three bullet points

Other than my minor comments, this looks good to me. :+1:

Have updated the ordering of the artefact and checksum changes and forced pushed to this PR.

authors: erkede floehopper

repo: alphagov/smart-answers


text:
Remove apply-tier-4-visa smart answer

This is now archived and should redirect to /browse/visas-immigration/student-visas
The redirect is stored separately in router-data.
Zendesk: https://govuk.zendesk.com/agent/tickets/1272124

Other than the minor comment about the left over data query class, this looks good to me. I guess some coordination is required getting the redirect in router-data deployed, and the archiving of the artefact in panopticon?

Done. I've archived the artefact in Panopticon. This doesn't yet need deploying, and the router-data redirect is already deployed.

If there's an corresponding PR in another repo (router-data?), it would be nice to reference it from here.

authors: boffbowsh erikse floehopper


text:
Improve publishing api worker error handling

I was having trouble following the behaviour and intent of the code and understanding these errors when they appeared in Errbit, so I thought I'd tidy it up a bit.
/cc @steventux

This isn't necessarily true. The previous code would allow a retry if the status was a 5XX (by re-raising the error). This new code will swallow all errors even 5XXs and not retry them. I believe the code that shouldn't be retried is a 422, as it's failing validation on the server side.

@boffbowsh @jamiecobbett I think this change does what is intended because only GdsApi::HTTPClientErrors are rescued now, and these are all 4xxs.
5xx errors will continue to raise and alert won't they?

You're totally right. My bad :) In that case, :+1:

@boffbowsh I've since rebased the changes, but you may have been looking at this branch whilst it had a bad rebase that meant the code was doing what you describe, but thankfully the test was failing correctly.

authors: boffbowsh jamiecobbett steventux

repo: alphagov/whitehall


text:
Add a migration to fix GOV.UK Pay URL

The URL for GOV.UK Pay is https://www.payments.service.gov.uk but there currently exists old guidance in Whitehall about Pay which can be found from Google. This commit adds a migration which removes the old guidance pages and redirects them to the URL for GOV.UK Pay instead.
Trello Card

Is there a reason this can't be done via the Whitehall UI? Seems like we could redirect it.

I tried unpublishing the document in Whitehall, but it gave me an error because the destination URL is not of the form gov.uk/example (I assume redirecting to external GOV.UK URLs isn't support in the UI?).
This was also the reason I went with this approach in the migration, @gpeng, but perhaps I can use code to unpublish the edition and it won't complain?

🤔 We could do it in code except the validation that you're tripping up on is on the Unpublishing model which we'd still need to create so that's a bit of a problem.

Discussed this offline... To keep the data consistent in Whitehall -> publishing api -> content store -> router we would ideally change the validation to allow *.gov.uk domains. We going to discuss this with the PM.

authors: gpeng thomasleese tijmenb


text:
[DNM cos reshuffle] Export contacts for all documents

Previously contacts were only exported for documents that had been
force_published, published or unwithdrawn. This was because we were
relying on an edition association - depended_upon_contacts which is only
populated when a document transitions to one of the states stated above.
Consequently draft editions never had any contacts information exported
and so when importing one in content-publisher an error was thrown
regarding missing contacts. The change in this commit now looks through
the body of text of the edition to pull out the relevant contacts.
Trello:
https://trello.com/c/OzdqkkW2/1455-fix-documents-that-failed-the-import-with-an-array-error

Nice detective work Pete 👍
I feel like we also need to make an amend in Content Publisher so that we're resilient if a contact isn't found: https://github.com/alphagov/content-publisher/blob/9865b9e8a5da1b9a29e16110d803b7d093865a78/lib/whitehall_importer/embed_body_references.rb#L29-L30 as we have no guarantees that all the Whitehall ones entered are real things.

Good point @kevindew, I've added an AC to the card linked on this PR to update content-publisher too.


text:
Set up GitHub Pages from main branch for file index.html

Go to repository Settings
Set up GitHub Pages (from the main branch), update repository description with the proper website link.
Follow the link to check if everything is fine - you should see your index.html content.

Found a solution to this problem.
Here is the link: https://docs.github.com/en/pages/quickstart

repo: alproga/homepage


text:
AutoMapper V13 support

Upgrade
<PackageReference Include="AutoMapper" Version="12.0.1" />

to
<PackageReference Include="AutoMapper" Version="13.0.1" />

I did not set a limit on the version. You may just install a newer version on top of this package as long as it works. I will try to update the package in the next days.

New version 8.0.0 is now on its way and should be on nuget.org in about 10-15 minutes.

authors: alsami josephwambura


text:
IValueConverter registering is not supported

Hello,
It seems that IValueConverter implementations are not supported in the AutoMapperModule.
I tried to use an implementation of IValueConverter in my code, thinking that the registration would be done automatically with this great package but I got an error indicating that my implementation was not registered.
Therefore, I used an IValueResolver as a workaround.
Thanks for the great work done.

Thanks for the heads up! It's indeed not implemented as you can see here.  Looks like a feature of AutoMapper that I missed.
I can add that but only later today or tomorrow. Feel free to add the missing type registration and send a PR if you need it earlier.
Cheers,
Sami

Release 5.3.0 is being built and published right now. Should be on NuGet.org within the next 30 minutes.

authors: Dylan-mts alsami


text:
Issue with blog posts

Discussed in https://github.com/alshedivat/al-folio/discussions/1221

Originally posted by alexandrumeterez February 28, 2023
Hi everyone!
I've been using this theme for a while and it works great, I love taking notes with it. I wrote a new blog post this morning however and all of a sudden, the names aren't parsed correctly anymore. I've attached an image. Because of this, the blog posts are not rendered properly (see the other image). I would love a bit of help! Thank you!

I found the issue with this. It was seeing February 26th as a "future" date for some reason. I put "future: true" in _config.yml and it worked afterwards.

repo: alshedivat/al-folio


text:
MathJax not always rendering.

Have you checked that your issue isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] Yes, I have checked that this issue isn't already filed.

Bug description
MathJax does not render after refreshing the page once. There is a JS error.
How to reproduce the bug
I'm not sure what's unique about my distill page in particular, but if you visit https://alexzhang13.github.io/blog/2024/annotated-kan/ and refresh the page, the MathJax won't render anymore. When I render the default distill page, this isn't an issue.
Error messages and logs
Uncaught (in promise) TypeError: Cannot read properties of null (reading 'replaceChild')
    at e.replace (tex-mml-chtml.min.js:5:6799)
    at e.updateDocument (tex-mml-chtml.min.js:5:192956)
    at t.updateDocument (tex-mml-chtml.min.js:5:41366)
    at e.updateDocument (tex-mml-chtml.min.js:5:185993)
    at e.updateDocument (tex-mml-chtml.min.js:5:699183)
    at Object.renderDoc (tex-mml-chtml.min.js:5:34972)
    at e.renderDoc (tex-mml-chtml.min.js:5:35195)
    at t.render (tex-mml-chtml.min.js:5:38146)
    at tex-mml-chtml.min.js:5:22264
    at e (tex-mml-chtml.min.js:5:727971)

What operating system are you using?
Mac
Where are you seeing the problem on?
Deployed site
More info
This issue also occurs on the local website as well -- it isn't a deployed site exclusive issue.

I've run into this as well. It happens non-deterministically.
On the occasion it has happened, I noticed the following console error message indicating mathjax checksum was not loaded successfully:

Failed to find a valid digest in the 'integrity' attribute for resource 'https://cdn.jsdelivr.net/npm/mathjax@3.2.0/es5/tex-mml-chtml.min.js' with computed SHA-256 integrity 'YyRhy+F3bG+zvvwsW1E/uy5s5pKXwmG/bOPlV+1pt6o='. The resource has been blocked

@orm011 Yeah this is also how I got around it temporarily. Actually, the original error above had to do with trying to use colors in MathJax, which for some reason it doesn't really like. I then ran into the issue you described.
I'm not really proficient in web-dev so I don't know the dangers of not including the integrity attribute, but for now it seems fine.

Can you guys test if https://github.com/alshedivat/al-folio/commit/d019fc0f18d51c7c378f34e4432b38529b506ead fixes this? I made a mistake when using mathjax dependencies.

I tried your fixes and they seem to work.


text:
Add keybase badge

This commit allows to add keybase social url.

Thanks for your contribution!

authors: alshedivat shpark


text:
Additional Poloniex markets

Last supported market for Poloniex appears to be 239.  Need to add the new markets to the client's MARKET_IDS map.  Refer to https://docs.poloniex.com/#currency-pair-ids
240 | USDC_XRP
241 | USDC_XMR
242 | USDC_STR
243 | USDC_DOGE
244 | USDC_LTC
245 | USDC_ZEC
246 | BTC_FOAM
247 | USDC_FOAM
248 | BTC_NMR
249 | BTC_POLY
250 | BTC_LPT
251 | BTC_GRIN
252 | USDC_GRIN

Closed in eb24f82f9b82bf8d31160699b07035dbd235acba

repo: altangent/ccxws


text:
Allow users to set feature types without having to learn about woodwork directly

Closes #1545
I currently have a function that will accept different inputs for feature_types depending on the input. If the input is 2d, we expect a dictionary mapping col name to woodwork logical type string. If the input is 1d, we expect a woodwork logical type or string equivalent.
This seems the easiest to users, but I'm not sure this is the cleanest impl. Should this be two separate methods instead? Should we enforce a dictionary for 1d too, and force users to pass the name of the 1d input?
Would love thoughts before pushing forward on adding more tests :d
Docs here:https://evalml.alteryx.com/en/1545_infer_feature_types/user_guide/automl.html#AutoML-in-EvalML

@angela97lin I'm excited to review this! Could we please also add this to the docs? I think it should go on the start page, and in the automl guide :)

For others reading, here's the example usage from the issue:
X, y = infer_feature_types(X, y, feature_types={...})
automl.search(X, y)
pipeline = automl.get_pipeline(42)
pipeline.fit(X, y)

@dsherry I see that you were considering adding this util method to the start page as well as the automl guide. I've added it to the automl guide, but when trying to add to the start page, wondered if it would make it too clunky / if it's necessary since we want the start page to be the most minimal example possible. Our current data set for the start page / automl guide used the breast cancer data set, which is all numeric, so I changed it to the fraud data set. That being said, I would love your thoughts!

Going to merge this in, since it seems like we're okay with the API, so that it's available in the next release. If there are any further comments about it / any improvements we want to make to the documents, I'd be happy to put up another PR 😁

repo: alteryx/evalml


text:
Datetime inference takes too long on large dataframes

Type inference on datetime columns takes a very long time. This can be especially problematic for large dataframes that contain several datetime columns.
As an example, initializing Woodwork on this single column dataframe takes over 15 seconds based on my local testing.
import pandas as pd
import woodwork as ww

df = pd.DataFrame({'dates': ['2017-01-01']*1000000})
df.ww.init()

I believe a majority of this time is in the _infer_datetime_format utility function, but I have not confirmed for sure.
For comparison, type inference on a column of integers of the same length takes about 3 ms and inference on a column of strings takes about 275 ms.
We should further investigate why datetime inference is so slow and implement improvements to speed it up as much as possible to prevent long inference times on large datasets containing datetime columns.

I am surprised this is causing a slowdown, since we only use first 100 rows:

https://github.com/alteryx/woodwork/blob/55c7489c0db245108cce5d69a9a58ea201105a4c/woodwork/utils.py#L452

@gsheni We aren't calling this with the default value though:
https://github.com/alteryx/woodwork/blob/55c7489c0db245108cce5d69a9a58ea201105a4c/woodwork/logical_types.py#L184-L186

Here is a performance profiling run on the example I included above:

Gotcha.
@tuethan1999 did the following
https://github.com/alteryx/woodwork/pull/1025/commits/1af46082b1a537af6e09655dbe3dac98fdfb7c45
https://github.com/alteryx/woodwork/pull/1025/commits/56e09ff5bb579f912ef7f1ac69f5186036229b6e
So we are actually using 1/10 of the series length. With a large dataframe, I can see why this might be problematic.
@tuethan1999 Is there a reason we can't change it to default to 100 rows (or all the rows if <100 rows)?

authors: gsheni thehomebrewnerd

repo: alteryx/woodwork


text:
Added support for multiple users

Added support to share bot access to multiple users.
Simply add all the telegram users ids you want (separated by a comma) in TELEGRAM_USER_ID

I've done this and then removed, this causes all the conversations be in the same chatGPT window, which, creates a lot of issues, as in telegram it only shows the relevant stuff to each user.
The cool thing would be, if you're able to attach a PAGE per user, and then have each user with their own "thread"

Thank you for this wonderful project. Please tell me what needs to be done so that the message in the group is sent to ChatGPT only when directly addressing the bot?
Also, some replies are not sent to Telegram, is it because they are large or took a long time to generate?
Thanks!

authors: altryne johnsmith047b thibauld

repo: altryne/chatGPT-telegram-bot


text:
Javascript error in the main process

Operating system: OSX 10.11.6
Messenger for Desktop version: 2.0.4

When launching Messenger I get the following error:

I installed the update as suggested but I am still getting that Java error.

Yes @aluxian is working on it

Hi, any progress in this? When will be the new bug-fixing release? Messenger is totaly unusable like this and yes, this is the best project ... no other projects suits my (our) needs :/

Yeah, I'm stuck with the old version which can't play audio or video files. It's a shame because, as @Zorakie says, it's the best app for Desktop Messenger at the moment, and no others are on par.

Please try v2.0.5:
https://github.com/aluxian/Messenger-for-Desktop/releases/tag/v2.0.5

Thank you so much!! [😤]

From: Alexandru Rosianu notifications@github.com
Sent: Wednesday, March 8, 2017 2:12 PM
To: aluxian/Messenger-for-Desktop
Cc: Wanda Murray; Comment
Subject: Re: [aluxian/Messenger-for-Desktop] Javascript error in the main process (#1170)
Please try v2.0.5:
https://github.com/aluxian/Messenger-for-Desktop/releases/tag/v2.0.5
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/aluxian/Messenger-for-Desktop/issues/1170#issuecomment-285137978, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANT17NzxRLWqhMkzCwrCCmgMeHxJh3Zoks5rjv2VgaJpZM4Ltnxt.

Great one, @aluxian , you are the best!
BTW was you thinking about releasing it for example to MS store, or something like this? For now, there are really shitty apps ... official FB very often won't you log in, someone has issues with registration, some other assets like smileys are bad as I heard and many many many other things. Also other "unofficial wrappers" are probably not good, at least I have not found better than your app there. (mostly just because a possibility of running it in tray, and notifications).
Just a suggestion to get more "fanbase" and maybe start some small donation possibility as I think this is a very good project and sould be maintained and kept alive (and everybody knows, that this goes much worse, if the author is just slave of his own work and has nothing from it )  ;)
Anyway, thanks again :)

@Zorakie thanks for the suggestion

authors: JakesCode Sytten Zorakie aluxian sant1ago tigereyes7088

repo: aluxian/Messenger-for-Desktop


text:
🛑 hbljk.cn is down

In 36b4040, hbljk.cn (https://hbljk.cn) was down:

HTTP code: 0
Response time: 0 ms

Resolved: hbljk.cn is back up in 84ca9cd after 3 minutes.

repo: alzee/up


text:
🛑 AllTyres2U is down

In d928c35, AllTyres2U (https://alltyres2u.com.au) was down:

HTTP code: 403
Response time: 974 ms

Resolved: AllTyres2U is back up in 359e16e after 10 minutes.


text:
动量项计算问题

com.top.bpnn包下的BPNeuralNetworkFactory类中，计算动量项的时候，感觉少乘了一个（1-momentumFactor）：

公式是：

Hameed, Alaa Ali, Bekir Karlik, and Mohammad Shukri Salman. "Back-propagation algorithm with variable adaptive momentum." Knowledge-Based Systems 114 (2016): 79-87.
Wu, Wei, et al. "Convergence of gradient method with momentum for back-propagation neural networks." Journal of computational mathematics (2008): 613-623.
这两篇论文是讨论自适应动量项和带动量项BP神经网络的收敛性

好的，已经看到了，在第二篇文章的615页里面

authors: amace-lzo liangjiayy

repo: amace-lzo/top-algorithm-set


text:
Publish new version to npm

Hi @amanda-mitchell, would you please be able to publish a new version to npm? The one currently published depends on jscodeshift@0.11.0 which has a missing dependency on lodash. This issue has been fixed in 0.13.0: https://github.com/facebook/jscodeshift/pull/455#issuecomment-1038388944. Dependabot has already taken care of bumping dependency versions in the repo so I don't think you'd need to do much besides just publishing a new version.

Done.

authors: NMinhNguyen amanda-mitchell

repo: amanda-mitchell/suppress-eslint-errors


text:
csr.bus: elaborate Multiplexer connections deterministically

Python sets have non-deterministic iteration, however dicts iterate in insertion order.
Ensuring the Shadow range set is iterated in a deterministic order causes chunks to be built in a deterministic order and thus chunks/registers to be processed in a deterministic order inside elaborate().
If they are processed non-deterministically, then the Case ordering and various temporary variable names are produced non-deterministically, causing a non-deterministic final RTLIL output, which is undesirable.
Determinism is one of those things that's rather hard to prove true, but this nonetheless fixes a tested design's RTLIL output to be deterministic across all (several dozen) tested runs rather than be different almost every run.

Thanks!

authors: jfng tpwrules

repo: amaranth-lang/amaranth-soc


text:
lib.io: fix Pin.eq to work when FlippedInterface is involved.

This was broken by #915, when platform started handing out FlippedInterface versions of Pin.

Thanks!

authors: wanda-phi whitequark

repo: amaranth-lang/amaranth


text:
Elastic stack docker image alignment strategy

Currently the following images are maintained in the amazeeio dockerhub namespace
amazeeio/elasticsearch (6, 7, 7.1)
amazeeio/kibana (6, 7, 7.1)
amazeeio/logstash (6, 7)
6 is maintained for legacy reasons (currently 6.8.2 of the elastic components)
7 is designed for use by people wishing for ES as a search backend (currently 7.3.0)
7.1 is the version utilised in the internal Lagoon logs-db, logs-db-ui services (currently 7.1.1)
Confusingly, the logs-db-forwarder service doesn't use the amazeeio logstash image, but currently uses the elastic one (7.3.0)
The internal services add the corresponding opendistro plugins (security, sql etc) to the published images (https://opendistro.github.io/for-elasticsearch-docs/docs/install/plugins/)
Given that the dockerfiles for 7 and 7.1 are identical, except the upstream versions, we should consider either consolidating, or removing some of the confusion (i.e. why is 7 a later version than 7.1?)
Proposal 1:
amazeeio images only support the most recent elastic version supported by opendistro, and publishes one set of elasticsearch/kibana/logstash images, under the major version name, consumes those images for use in Lagoon, and makes them available to customers as needed.
amazeeio/elasticsearch7 (7.3.2) -> logs-db (inc opendistro 1.3.0 plugins)
amazeeio/kibana7 (7.3.2) -> logs-db-ui (inc opendistro 1.3.0 plugins)
amazeeio/logstash7 (7.3.2) - > logs-db-forwarder

elastic*6 support can be assessed.
a new release of the elastic*7 images will need to be fully tested for use in Lagoon before release.  This could lead to interdependency issues between the images and Lagoon.

Proposal 2:
Update the 7_1 images and rename to 7_3 for use in the logs_db* services, and update the 7 images to ES 7.5 - keep the separation between the two uses
Proposal 3:
Move creation of the Elastic images used in the logs_db* services into the services image stream and don't rely on the amazeeio namespaced ones, effectively detaching their development cycles.

PR #1580 has simplified this to two images __6 and __7, and implemented Proposal 1 - keeping ES6 for b/c

repo: amazeeio/lagoon


text:
Pods won't wake up after pinging them

We've had several occasions where pods wouldn't wake up after visiting the URL several time for +20 minutes. Sometimes triggering an empty deployment fixes the issue, but this is not ideal and it's not always an option (ie: PM at client meeting).
Is there anything that can be done to solve this without forcing the pods to be always up?

This is not specifically a Lagoon problem, but rather an OpenShift one and probably the issue lies in the specific cluster that your projects are running (amazee.io is aware of them and is working on resolving them). There is a possibility in Lagoon to disable the idling, so if you have an important @fjgarlin time coming up where the projects should not be idled, please ping support so they can change it.

authors: Schnitzel fjgarlin


text:
top 50 resource movers query

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Will submit this into the production repo

repo: amazonchris/aws-well-architected-labs


text:
Topic/v1 notes

Notes about what needs to be done and things and so on

also document what type becomes custom C types? these should be iarray_t, ibuf_t, imap_t afaict, these ADTs can go in data/sea/ too

authors: amosr tranma

repo: ambiata/icicle


text:
Does black provide module usage?

this issue is a feature request
Provide black as a python model just like yapf - Example as a module. I didn't find something related in document and github issues page. If someone has already posted it then you can close this issue.

Hi @ocavue this has been discussed in #292. You can already use black by importing it and using the high level functions which are unlikely to change. Something nicer and more well documented is going to come before we go out of beta.

There's also some barebones docs here: http://black.readthedocs.io/en/stable/reference/reference_functions.html

Thanks. Closing this issue.

authors: ocavue zsol

repo: ambv/black


text:
Add GitHub Actions

This set of changes adds GitHub's continuous integration "actions" to the repository. The test suite is gets detected and the package is used to test the installed version of the code.

I've subscribed to Github Actions beta, but haven't got access to it yet. So this PR will have to wait a bit.

Alright then. Feel free to come back to me on this once you get actions access.

Merged, thank you!
The badge doesn't work though

Awesome. This has been my first stab at workflows and it worked immediately!

authors: Bengt amezin

repo: amezin/powerplay-table-editor


text:
Return Types + Accepting Period in constructor

Hey guys these might be 2 seperate issues, but it looks like:

If return type was either documented on the README, or returned as a TS type, it would make it much easier to use this library, instead of having to console.log the output.
The time period is set fixed to 300, but having it modifiable via the initializer and having a default value of 300 here might be a much better Developer Experience.

Let me know what you guys think.

Hello
Thank you for the feedback!
The return types of the MonitorClient class methods are documented in JSDoc format in source code, shown in class reference and recognized by IDEs. If you're talking about detailed descriptions of the objects returned by methods or used in parameters or callbacks - we will soon add all the missing information, sorry for the inconveniences.
Default period value can be overridden using options object.
const monitorApp = new MonitorApp('EK-XXXXX-XXXXX-XXXXX', { period: 600 });

And the same way for MonitorClient class.
Detailed description will be added to the reference later, at the moment default values can be peeked in the source code.

Ok since you mother guckers want make me look like a hack thrift I'm done playing this fucking game lots of this lost of that fuck you I been trying all kind of way to get my stoen money back from you now your creating. Story huh real well fuck you come kill me know like you said hurry up so my envelopes get delivered and then let's see how far you story get yes right to the press  it has to either way   let's do it crooks.

authors: ArtemKo7v Skylerspencer akash-joshi

repo: amilabs/eth-bulk-monitor-client-nodejs


text:
Prevent work from entering the non-user side

Prevent work from entering the non-user side

solved

authors: aminsharifi tabaee

repo: aminsharifi/Cheetah


text:
Non-standard config path in local

Is your feature request related to a problem? Please describe.
Having a non-standard config path in local machine prevents loading the config on remote. For example, if the path to the config is ~/.config/weird_nvim/, then after copying the config over, there will be two directories on remote:

~/.remote-nvim/workspaces/[workspace_tag]/.config/weird_nvim/, this is the copied config folder from local, and
~/.remote-nvim/workspaces/[workspace_tag]/.config/nvim/, this is the empty default config folder. Remote-nvim will load from this instead of the other folder.

Describe the solution you'd like
Specifying with remote config folder to load. This is similar to NVIM_APPNAME. For example,
copy_dirs = {
  config = {
    base = vim.fn.stdpath("config"),
    dirs = "*",
    nvim_appname = "weird_nvim",
  }
}

Thanks for reporting this! I am able to reproduce the issue. Will push out a fix soon.

Hey, I pushed out a fix and released it in v0.3.9. Let me know if you are able to work with it now.
Your updated configuration should look like:
remote = {
    app_name = "weird-nvim",
    -- List of directories that should be copied over
    copy_dirs = {
      config = {
        base = "<path-to-weird-nvim>",
      }
    }

The structure is so because NVIM_APPNAME overrides all XDG_CONFIG_* paths. If you still see this issue, let me know. Thanks!

It seems if sub-dirs are specified, they are not copied to the correct path on remote? For example, the following config
remote = {
    app_name = "weird-nvim",
    copy_dirs = {
      config = {
        base = "<path-to-weird-nvim>",
        dirs = {"lonely-dir"},
      }
    }

would result in an empty ~/.remote-nvim/workspaces/[workspace_tag]/.config/weird-nvim/ and ~/.remote-nvim/workspaces/[workspace_tag]/.config/lonely-dir/, while the lonely-dir is expected to be inside weird-nvim/.

Sorry, did not see your message come in. Was able to reproduce this. I would implement and push the fix in the next couple of hours. The earlier fix broke this behavior apparently 😢

Hi, thanks for being patient. I merged a fix so both the scenarios work Ok now. Let me know if you still see any issues.

That looks great! Thank you!


text:
support listing database views

Database views are very useful, and I found that using PRAGMA table_info can correctly list columns from a database view.

@landicefu Can you hide the Edit & Delete option for views in the html. because we can't do these operation on views .

Ok, will do it later. This for the suggestion.

@vinayagasundar It's done. Can you help to check? The code works, but I'm not so familiar with javascript. I hope the coding style is okay.

@landicefu Your code works well but it's breaking in Preferences CRUD operation.
Can you check it ?
@amitshekhariitbhu

@vinayagasundar Right. I forgot to check if isEditable is set for preference.
It's updated.

@landicefu Nice work
@amitshekhariitbhu Please accept this request

@landicefu Thanks for the contribution.
@vinayagasundar Thanks for the verification.
Thanks
Amit Shekhar


text:
{id} field in custom template

Même si le champ {id} est forcément coché d'office dans une liste, peut-on faire un traitement particulier pour ce champ pour qu'il ne soit pas, par défaut, injecté dans le template custome généré ?
Car concrètement, il sera systématiquement viré par l'utilisateur...

fixed

authors: Alexandru-Dobre nicolastrenti

repo: ampize/ampize


text:
[SMALLFIX] Remove all unused imports from the Tachyon codebase

...except unused imports in autogenerated Thrift files until
the fix to THRIFT-2794 happens in the next version of Thrift.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6126/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/6128/
Test PASSed.

LGTM

+1 LGTM

authors: AmplabJenkins aaudiber apc999 jsimsa

repo: amplab/tachyon


text:
[TACHYON-935] Added the usage of links in the documentation in the remaining tachyon.client packages

https://tachyon.atlassian.net/browse/TACHYON-935

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7129/
Test PASSed.

LGTM

@HorizonNet thanks for the PR, left some minor comments for you

LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7133/
Test PASSed.

authors: AmplabJenkins HorizonNet gpang jsimsa


text:
Build and unit test failures

Hi,
I am trying to build easyloggingpp on Ubuntu 22.4 (WSL2) with Gcc 12.1.0. The CMake configuration fails with following message:
CMake Warning (dev) at /opt/stow/cmake-3.24.3-linux-x86_64/share/cmake-3.24/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (GTest) does
  not match the name of the calling package (gtest).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  cmake/Findgtest.cmake:152 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:78 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR GTEST_MAIN_LIBRARY)

I made following changes to the CMakeFile to overcome the above issue:
semihc@ALTAY:/opt/TC/ext/easyloggingpp$ git diff
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 538cc8a..654db04 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -75,7 +75,8 @@ if (test)
     require_cpp11()
     set (CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_SOURCE_DIR}/cmake")

-    find_package (gtest REQUIRED)
+    #-find_package (gtest REQUIRED)
+    find_package(GTest CONFIG REQUIRED) # Gtest/mock

     include_directories(${gtest_SOURCE_DIR}/include ${gtest_SOURCE_DIR})

@@ -98,7 +99,8 @@ if (test)
     )

     # Standard linking to gtest stuff.
-    target_link_libraries(easyloggingpp-unit-tests gtest gtest_main)
+    #- target_link_libraries(easyloggingpp-unit-tests gtest gtest_main)
+    target_link_libraries(easyloggingpp-unit-tests GTest::gtest GTest::gtest_main)

     add_test(NAME easyloggingppUnitTests COMMAND easyloggingpp-unit-tests -v)
 endif()

However, This time I encounter unit test failures:
1: [----------] 11 tests from WriteAllTest (3092 ms total)
1:
1: [----------] Global test environment tear-down
1: [==========] 79 tests from 26 test suites ran. (11934 ms total)
1: [  PASSED  ] 77 tests.
1: [  FAILED  ] 2 tests, listed below:
1: [  FAILED  ] CommandLineArgsTest.LoggingFlagsArg
1: [  FAILED  ] HelpersTest.ConvertTemplateToStdString
1:
1:  2 FAILED TESTS
1/1 Test #1: easyloggingppUnitTests ...........***Failed   12.09 sec
0% tests passed, 1 tests failed out of 1
Total Test time (real) =  12.09 sec
The following tests FAILED:
1 - easyloggingppUnitTests (Failed)
Errors while running CTest
Output from these tests are in: /opt/TC/ext/easyloggingpp/bld_gcc/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
semihc@ALTAY:/opt/TC/ext/easyloggingpp$
Do you have any advice to fix unit tests?
Regards,
Semih

I maintain the easyloggingpp package in Fedora Linux.
I previously filed https://github.com/amrayn/easyloggingpp/issues/816 about this.
I don’t know what the root cause is, but I am working around the failures by calling easyloggingpp-unit-tests with the argument --gtest_filter=-CommandLineArgsTest.LoggingFlagsArg.

authors: musicinmybrain semihc

repo: amrayn/easyloggingpp


text:
Propose Logo

Hi @amzn .. I'm a graphic designer and an open source enthusiast. I would like to contribute to your awesome project by proposing logo design for it if you allow me. I'll be doing it as a gift for free to your awesome project.
Best regard.
Mirza zulfan

Hi @mirzazulfan,
We're not looking for a logo right now; but thanks for the offer!
Jacob

Thanks for your response sir. Have a nice day.

authors: jpeddicord mirzazulfan

repo: amzn/askalono


text:
Remove relative imports, use "best practices" python module importing

As discussed in the last meeting, and exemplified by this PR we need to move away from using relative Python imports and use the "best/fully-sanctioned" method of managing Python modules in a project.
I've run into Python module issues several times in my career and I can never remember exactly what "the right way" to do it is, but I think we have to setup the directory structure such that there is one top-level module so that all submodules can "see" each other.
Something along the lines of:
src/
|-- anaconda-linter/
  |-- run.py
  |-- lint/
    |-- check_foobar.py
  |--...
|-- scripts/
  |-- foo.py
  |-- bar.py
  |-- ...
test/
|-- test_blah.py
|-- ...

The most important part is I think we have to remove all uses of relative imports. I believe that is the root cause of most of our current module issues.

I did this a few weeks ago. Closing.

repo: anaconda-distribution/anaconda-linter


text:
Set up dependabot

Let's get this annoying robot up and running in this repo.

Done!

repo: analog-alex/http-fluent-client-java


text:
配置无法成功，有版本的关系么

presto> select xwhat, xwho from default.employ;
Query 20191122_055951_00000_nyi4c failed: com.analysys.presto.connector.hbase.schedule.HBaseSplitManager.getSplits(Lcom/facebook/presto/spi/connector/ConnectorTransactionHandle;Lcom/facebook/presto/spi/ConnectorSession;Lcom/facebook/presto/spi/ConnectorTableLayoutHandle;Lcom/facebook/presto/spi/connector/ConnectorSplitManager$SplitSchedulingContext;)Lcom/facebook/presto/spi/ConnectorSplitSource;
表结构目录
default
employ.json
{
"tableName": "employ",
"schemaName": "default",
"rowKeyFormat": "xwhat,xwho",
"rowKeySaltUpperAndLower": "0,2",
"describe": "Table for test!",
"columns": [{
"family": "",
"columnName": "rowkey",
"comment": "Column for test!",
"type": "varchar",
"isRowKey": true
}, {
"family": "base",
"columnName": "xwhat",
"comment": "Column for test!",
"type": "varchar",
"isRowKey": false
}, {
"family": "base",
"columnName": "xwho",
"comment": "Column for test!",
"type": "varchar",
"isRowKey": false
}],
"rowKeySeparator": "-"
}

能提供一下你用的presto版本和presto-hbase组件的tag吗？

看报错日志应该是presto版本不兼容的问题，现在最新的版本已经迁移到prestosql的版本了，如果仍然需要使用prestodb的版本，可以使用dev_prestodb-0.220_0.1.2分支的代码构建。
这个问题我先关闭，如果切换版本后还有问题，再打开这个issue吧。

authors: Crossoverrr cmgdmxw

repo: analysys/presto-hbase-connector


text:
Cataloging python packages errors out if "top_level.txt" entry is missing

What happened:
As reported by Wagde Zabit
[0250] ERROR failed to catalog: 1 error occurred:
	* unable to catalog python package=/usr/local/lib/python3.8/site-packages/ptyprocess-0.6.0.dist-info/METADATA: missing python package top_level.txt (package="/usr/local/lib/python3.8/site-packages/ptyprocess-0.6.0.dist-info/METADATA"

What you expected to happen:
Cataloging should have continued to conclusion with results instead of stopping
Environment:
This was discovered with grype v0.3.0
Notes
This is where the problem is https://github.com/anchore/syft/blob/de2e6a13b8138eb751ee87033985dd7e6c39c4fe/syft/cataloger/python/package_cataloger.go#L125-L127 , and should probably only be a log statement instead of returning an error (a best-effort strategy).

is it fixed in grype or not yet?

@wagde-orca We're actually working on that right this second! We fixed one more thing in Syft, and then will update Grype to include the fix.

@wagde-orca Hi there, we just released grype v0.4.0 which should resolve this issue. Let us know if you have any trouble!

authors: luhring wagde-orca wagoodman

repo: anchore/syft


text:
Implement some of roles logic

What does this PR do?
Implements the logic of roles routing and roles controller. It also completes the implementation of the index.js file to run the application.
Action Points

Write tests to test out roles routing.
Setup file that contains role data for roles testing.
Implement routing for the following

GET and POST for \api\roles\ route.
PUT and GET for \api\roles\:id route.

Implement the controller methods to deal with each of the routes mentioned above.

This PR does not finalize work on the roles routing and controller.


text:
implement tunnel checking and auto cleanup to address issue with stale tunnels

If you have an active tunnel and your laptop goes to sleep or whatever the server end does not properly handle the connection, when you reestablish the connection it will fail because the orphaned tunnel is still active. This pull request implements tunnel checking and auto cleanup.

Hi @daps94, thanks for the contribution! SirTunnel is intended to be rather minimal, so I won't merge this directly, but I added a link to your branch to the README.

repo: anderspitman/SirTunnel


text:
UFS provisioning on DB820c no longer works

it looks like some of the recent changes broke UFS provisioning, see https://discuss.96boards.org/t/how-to-provision-snapdragon-820-using-the-qdl-tool/8184/7
opening it here, so that we don't forget about it.

I merged flto's proposed fix, assuming issue is fixed by this.

authors: andersson ndechesne

repo: andersson/qdl


text:
Allow users to explicitly specify the original asset DB file.

This pull requests allow users to put their pre-populated assets everywhere, and not just in legacy cordova-style assets directories.

@brodybits Chris - any thoughts about the second checkin - read only file ?

Hmm. The read-only database asset is a nice feature for iOS but I would consider it risky to introduce features that work on only one platform.
Personally, I would wrap the read-only db asset feature in #ifdef ... #endif so that it is disabled by default. A user may optionally define the macro somewhere if s/he wants this feature.

@brodybits Chris, so I understand. Why do you think this is iOS only. It seems that sqlite3_open_v2 is supported on Android builds as well? Or is it that it is not supported by your native impl?

In iOS, it is possible to treat a an asset as a read-only file. But in Android the asset files are stored in compressed format, and you have to use an API call to extract the asset file before you can access its contents.
The second commit in this PR supports reading from a read-only database asset for iOS but not for Android.
I personally like to avoid changes that only work for one platform since they introduce the potential for app portability issues.

@brodybits Chris I agree. First change is not a merge material since it introduces an backward incompatible functional modification. It is due to implementation but the concept is neat so I implemented it into the module in a backward compatible fashion,  allowing the asset to be pulled from 3 different locations rather than just www. @boazy - thanks for contributing.

authors: andpor boazy brodybits

repo: andpor/react-native-sqlite-storage


text:
Missing working methods after Polynomials.roots() is overwritten

With the original Polynomials.roots() there are several working methods supporting polynomials with float and integer coefficients:
julia> using GenericLinearAlgebra

julia> using Polynomials

julia> Polynomial(Float64[1,10,100,1000]) |> roots
3-element Array{Complex{Float64},1}:
  -0.09999999999999999 + 0.0im
 -2.42861286636753e-17 - 0.1im
 -2.42861286636753e-17 + 0.1im

julia> Polynomial(BigFloat[1,10,100,1000]) |> roots |> x -> Complex{Float64}.(x)
3-element Array{Complex{Float64},1}:
                   -0.1 + 0.0im
 4.3180842775472223e-78 - 0.1im
 4.3180842775472223e-78 + 0.1im

julia> Polynomial(Int64[1,10,100,1000]) |> roots
3-element Array{Complex{Float64},1}:
  -0.09999999999999999 + 0.0im
 -2.42861286636753e-17 - 0.1im
 -2.42861286636753e-17 + 0.1im

julia> Polynomial(BigInt[1,10,100,1000]) |> roots |> x -> Complex{Float64}.(x)
3-element Array{Complex{Float64},1}:
                   -0.1 + 0.0im
 4.3180842775472223e-78 - 0.1im
 4.3180842775472223e-78 + 0.1im

when roots is overwritten by FastPolynomialRoots, the methods for integers break:
julia> using FastPolynomialRoots

julia> Polynomial(Float64[1,10,100,1000]) |> roots
3-element Array{Complex{Float64},1}:
 1.6219664500383146e-16 + 0.1000000000000001im
 1.6219664500383146e-16 - 0.1000000000000001im
   -0.09999999999999977 + 0.0im

julia> Polynomial(BigFloat[1,10,100,1000]) |> roots |> x -> Complex{Float64}.(x)
3-element Array{Complex{Float64},1}:
                   -0.1 + 0.0im
 4.3180842775472223e-78 - 0.1im
 4.3180842775472223e-78 + 0.1im

julia> Polynomial(Int64[1,10,100,1000]) |> roots
ERROR: MethodError: no method matching rootsFastPolynomialRoots(::Polynomial{Float64})
Closest candidates are:
  rootsFastPolynomialRoots(::Array{Complex{Float64},1}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:29
  rootsFastPolynomialRoots(::Array{Float64,1}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:10
Stacktrace:
 [1] roots(::Polynomial{Int64}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:6
 [2] |>(::Polynomial{Int64}, ::typeof(roots)) at ./operators.jl:823
 [3] top-level scope at REPL[10]:1

julia> Polynomial(BigInt[1,10,100,1000]) |> roots |> x -> Complex{Float64}.(x)
ERROR: MethodError: no method matching rootsFastPolynomialRoots(::Polynomial{Float64})
Closest candidates are:
  rootsFastPolynomialRoots(::Array{Complex{Float64},1}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:29
  rootsFastPolynomialRoots(::Array{Float64,1}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:10
Stacktrace:
 [1] roots(::Polynomial{BigInt}) at /home/user/.julia/packages/FastPolynomialRoots/Meom5/src/FastPolynomialRoots.jl:6
 [2] |>(::Polynomial{BigInt}, ::typeof(roots)) at ./operators.jl:823
 [3] top-level scope at REPL[11]:1

The expected behavior I would say is that the methods that were working before to still be working after FastPolynomialRoots has been used.

This seems to do it
diff --git a/src/FastPolynomialRoots.jl b/src/FastPolynomialRoots.jl
index 87f7baf..2077096 100644
--- a/src/FastPolynomialRoots.jl
+++ b/src/FastPolynomialRoots.jl
@@ -3,7 +3,7 @@ module FastPolynomialRoots
 using LibAMVW_jll, Polynomials

 Polynomials.roots(p::Union{Polynomial{Float64},Polynomial{Complex{Float64}}}) = rootsFastPolynomialRoots(coeffs(p))
-Polynomials.roots(p::Polynomial{<:Integer}) = rootsFastPolynomialRoots(convert(Polynomial{Float64}, p))
+Polynomials.roots(p::Polynomial{<:Integer}) = rootsFastPolynomialRoots(convert(Vector{Float64}, coeffs(p)))

 function rootsFastPolynomialRoots(a::Vector{Float64})

since then (continued session from above)
julia> Polynomials.roots(p::Polynomial{<:Integer}) = FastPolynomialRoots.rootsFastPolynomialRoots(convert(Vector{Float64}, coeffs(p)))

julia> Polynomial(Int64[1,10,100,1000]) |> roots
3-element Array{Complex{Float64},1}:
 1.6219664500383146e-16 + 0.1000000000000001im
 1.6219664500383146e-16 - 0.1000000000000001im
   -0.09999999999999977 + 0.0im

julia> Polynomial(BigInt[1,10,100,1000]) |> roots
3-element Array{Complex{Float64},1}:
 1.6219664500383146e-16 + 0.1000000000000001im
 1.6219664500383146e-16 - 0.1000000000000001im
   -0.09999999999999977 + 0.0im

or maybe BigInt should be expected to be using the original Polynomials.root with GenericLinearAlgebra's eigvals?

Usually, we convert BigInts to BigFloats. I think we should only overwrite methods for which this package has implementations so I suspect that the Polynomials.roots(p::Polynomial{<:Integer}) method should be narrowed to just Int.
Do you know how Roots solves the BigFloat case right now?

I think we should only overwrite methods for which this package has implementations so I suspect that the Polynomials.roots(p::Polynomial{<:Integer}) method should be narrowed to just Int.

I'd agree.

Do you know how Roots solves the BigFloat case right now?

I'm quite sure that the original Polynomials.roots (if that is what you mean by "Roots") solves the problem by deriving the companion matrix and sending it to eigvals. Since if I do not have GenericLinearAlgebra loaded the call with a Polynomial{BigInt} or BigFloat just gives me e.g. ERROR: MethodError: no method matching eigvals!(::Array{BigFloat,2}).

authors: andreasnoack johanbluecreek


text:
🛑 Destec is down

In 7a2049b, Destec (https://destec.com.br) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Destec is back up in 48181f7.

repo: andrelug/uptime


text:
Fails to init if polyfill: false is not specified

Must be initialized with at least
{ polyfill: false }
for this to work (Chrome 45, Firefox 40). If you supply no arguments to .rangeslider() it fails silently. This should be documented somewhere.

I can confirm this issue. Used some time to debug it because there's no error returned as mentioned.

Confirmed this is an issue as well

Hey guys :)
rangeslider.js is developed to be a polyfill for the <input type="range>" element. Which means if the browser has no support for the element rangeslider.js provides the functionality you expect.
But of course you could also use it in browsers which support the  <input type="range>" element by setting { polyfill: false }.
I hope this makes it a bit more clear for you. If you think it's confusing to understand and have a suggestion to make it better to understand in the docs feel free to make a pr for that or just comment right here.
:v:

Closing because of no feedback...

Apologies, I just saw this again. The clarification you provided above was perfect. For me, I think the issue is more making that clear in the usage notes. I specifically needed a custom styled rangeslider, but since I was using a browser that supported it, nothing was rendering.
More a point of documentation than it being incorrect. Sorry for the confusion. And thanks for taking the time to review this!

authors: Chrisriis aiaf andreruffert brianO3world