Accents

Thanks for starting work on this.
Any reason why the file doesn't match the README and doesn't use æ and œ but ae and oe?
I guess it would be easiest to just mass replace all ae with æ, áe and aé with ǽ, oe with œ, perhaps also capital letters, and then figure out how to represent oé or óe - œ with accent, where unicode doesn't have such char - I usually just use combining ' character together with œ.
I've used your file to check all the *.gabc files I have around, and ended up with a list of right now 857 words that I need to still double check (where possible verify against Solesmes books, otherwise google search them, ...), but many of them are simply missing in the list, in other cases it will be my typoes and in other cases I'll need to verify where the accent should be).  Once I get the list cleaned up, I'll post it here (will try to split the list into words verified by Solesmes and others.

Hmm, in fact there is really a lot of discussions on this right now, it's really not stable enough to have any reliable result yet, especially for hymns. For prose, you can take a look at branch 10.3 of collatinus (note that you also need collatinus-data to make it work). Even that is still quite experimental... Things start to stabilize a bit, but there are still many grey areas. I hope to have at least some proper documentation by the end of the month...

Still working on my list and still to try collatinus.
Anyway, I also found
https://github.com/DivinumOfficium/divinum-officium/blob/master/admin/accent_table
as another source of latin accents, while it has just 13k words, it contains lots of words commonly used in the office.

Out of the 13k words, most of them are also in the verbs.txt here, but 4745 words are not.
Out of these 4745, 242 are in verbs.txt with different accents from the accent_table (I've performed
sed 'y/ABCDEFGHIJKLMNOPQRSTUVWXYZáéíóúýǽ/abcdefghijklmnopqrstuvwxyzaeiouyæ/'
for the comparison, so some known correct differences like:
María
vs.
mária
are in that list too, but supposedly this is a list of words that certainly need double checking, then there are some (though not many) one and two syllable words in those 4745, and the rest could be hopefully added.

I have a list of words as accented in the psalms, proofread by Solesmes so quite reliable. The main problem being that while the way to accentuate prose is relatively simple, there are a lot of blur areas in the way to accentuate hymns, and the determination of diphtongues in hymns make it even more difficult, so we're planning to make a repository of accented hymns, with the same accents as in the Vatican editions, this seems to be the only way to have correctly accented hymns, and also to be able to study the way they are accentuated.
To take an example for hymns: in prose, the rules make words accentuated as:
muliérem, filiólus, intégrum, tonítrus and ténebræ, while in hymns, it's common to find them as mulíerem, filíolus, íntegrum et tónitrus and tenébræ. The rules guiding these accents are not clear...

Also, I found verbs.txt in this repo to contain lots of duplicates (111640 words).
So for psalms, antiphons and prose, could we perhaps have multiple word lists depending on reliability, have smaller ones with higher reliability and fallback on larger ones with lower one?
What sort order shall we use in those files?  LC_ALL=en_US.UTF-8, LC_ALL=C.UTF-8, something different (I doubt there are collation rules for latin)?  Would it be useful to submit patches to remove redundancies from verbs.txt (depends on the sorting choice), change ae to æ, áe and aé to ǽ, oe to œ and proofread those changes (to leave in Israel, Michael etc.)?
My interest is both prose (Matins readings mostly), antiphons, psalms and hymns.

I don't think there's any point in improving verbs.txt, it's only here because I didn't have time to remove it. This repository will contain only some documentation and pointers, and maybe a hymn corpus. The accentuation itself will be in Collatinus only, try the 10.3 branch for something that is more or less working. Having a full list of words plus a list of all ambiguities and informations on these ambiguities will be too much (around 1 million entries), it seems taking the road of lemmatization and analysis is a safer way, and the data is already in Collatinus, so I don't see any point in the constitution of such a list. If you really want to get a list (which might be useful), I think it's best to discuss the format further beforehand...
Do you really prefer a list over Collatinus? We're discussing the structure of Collatinus these days, we'll isolate the lemmatization, desinences, etc. into a library that will be usable from C.
For the hymns, noone knows the rules (even in Solesmes), so the only way is to constitute a corpus. We're planning to OCRize a very big accented hymnarium from Vatican editions, in the meantime there is http://www.almudi.org/Portals/0/docs/Breviario/fuentes/breviario.asp?tiempo=1 for that, the accents should be reliable, but accents are lacking on æ and œ on the website (the problem does not appear in their ebook format).

About collation, it would be interesting to provide some rules for Latin including accents and usual vowel length marks, they should be quite easy to write... It's easily testable from Python, see https://github.com/eroux/tibetan-collation for an example

(closing as the discussion continued through email)


Error retrieving access token with "Temporarily throttled, too many requests" message

Hello we are seeing an error on the puppet agent with description: "Temporarily throttled, too many requests".
We checked the puppet master log and it points to the exception occurring on line 14 in azure.rb
We ran the curl commands from here on the puppet master and they worked.  We were able to retrieve the access token and secret in ssh in plaintext.
https://stackoverflow.com/questions/51440297/how-to-set-the-value-of-an-azure-keyvault-secret-using-curl
Kind Regards.

The other error in puppetserver.log was line 17 in lookup.rb
We are using Puppet 2019.0.0 aka v6.  The agent version is 6.0.2.
Our hiera.yaml contains:

name: 'Azure Key Vault Secrets'
lookup_key: azure_key_vault::lookup
options:
vault_name: xxxxxxxxxxxxxxxxxx
vault_api_version: '2016-10-01'
metadata_api_version: '2018-02-01'

Our manifest contains:
$secret_user = Deferred("azure_key_vault::secret", ['vaultname', 'secretname', {
metadata_api_version => '2018-02-01',
vault_api_version    => '2016-10-01',
}])

Hi TraGicCode,
We just started using Puppet about 1 week ago and are testing.

We have 13 nodes.
We have 1 secret in hiera.
I do not know the difference between the two, we started with Puppet 6.
We started with module version 1.0.0
I am not sure I understand what log you want to query.  Can you provide an example?

Thank you.

Hey @bwarren18 ,
Thanks for replying.

Are you currently pulling secrets using hiera's automatic parameter lookup feature? ( https://puppet.com/docs/puppet/5.2/hiera_automatic.html )? When you've been writing your code is your expecting been that all secrets will be retrieved by making a call to the following puppet 4 function VIA your puppet manifests?

$secret_user = Deferred("azure_key_vault::secret", ['vaultname', 'secretname', {
metadata_api_version => '2018-02-01',
vault_api_version => '2016-10-01',
}])

There should be some log or activity log for your azure key vault that, i would think, gives more detail on who requests what secrets and what secret they are requesting.

Hi @TraGicCode,

We have 2 key lookups in our manifest and two secrets in the same Azure Key Vault.  The only difference is the secretname.

I modified our manifest, replacing "::secret" with "::lookup" so it is now:
`$secret_1 = Deferred("azure_key_vault::lookup", [''KeyVaultName''', 'Secret1', {
metadata_api_version => '2018-02-01',
vault_api_version    => '2016-10-01',
}])
$secret_2 = Deferred("azure_key_vault::lookup", [''KeyVaultName'''', 'Secret2', {
metadata_api_version => '2018-02-01',
vault_api_version    => '2016-10-01',
}])`

I enabled Diagnostic Logs on the Azure Key Vault.  The JSON log shows 3 entries that being with:

"operationName": "Authentication",
"resultType": "Success",

"operationName": "SecretGet",
"resultType": "Success",
"operationName": "SecretGet",
"resultType": "Success",

Hi @TraGicCode ,
To enable diagnostic logs, I went into the key vault blade in Azure.  Then clicked on "Turn on diagnostics to collect the following data." I selected Archive to a storage account and provided the name of an storage account to use.  I also checked the Send to Log Analytics box and selected a workspace.  I then checked the Log type 'AuditEvent' and set the retention to 365 days.  I then went into the storage account, blob, filtered for container name "insights-logs-auditevent".  I then navigated down to the newest JSON log file and downloaded it.
You can then also view it in Log Analytics by querying:
search * | where Category=="AuditEvent"  and OperationName == "SecretGet"

Ok I reverted back to calling azure_keyvault::secret in our manifest.
Ok I commented out those lines in hiera.yaml with the # character.

I then reverted back to not using the deferred functions.
The error went away and we are able to unwrap the secrets successfully.
The issue can be closed.
Thank you very much for the plugin and help!

bwarren


convert to SPA

Convert to a single-page application (SPA), where instead of reloading the entire page with each request, just the SQL results are sent back and displayed (using AJAX).

Resolved with commit d078ef28e4d8c1a3e18856a31c36dcadf0d32cca.
Note that when testing with Internet Explorer, AJAX requests were not always sent immediately (when send was called).  Sometimes, triggering a request would result in nothing being sent, but then the next call to send would result in both requests being sent consecutively.  This behavior was not observed with other browsers.


fix on sending requests to bath sql api

requests library expects a headers argument, not http_headers

Looks good, yes .I guess you need to change the create method too, no?

create and the rest call this send method, I think there's no need to
change anything else
--
Jorge Sanz
CARTO - Predict through location
Sent from my phone, excuse my brevity.
El 11 ene. 2017 20:30, "Dani Carrión"<EMAIL_ADDRESS>escribió:

Looks good, yes .I guess you need to change the create method too, no?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/carto-python/pull/34#issuecomment-271969674,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALfaCYNtLYn-mK3HRIjPdljDoskIqG9ks5rRS27gaJpZM4Lg87A
.

sorry, yes, my bad
:+1:


refactor: add BranchNodePrefix and Separator

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

#413
#412  👈
#411
master

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

Join @gabriele-0201 and the rest of your teammates on  Graphite


Pinned followed users and pinned categories

Adding 2 more tabs to the side bar for "Pinned followed users" and "Pinned categories" respectively.
EDIT: Forgot to mention, there's no way to pin categories at the moment, so that would need to be added.

Just want to add that the ability to pin categories similarly to pinning followed streams would effectively cover the use case I had in mind for creating issue #27

An update has been published, adding the ability to pin categories. It'll be available with version 1.14.


Feature/winsock2 support

Pull request
Choose Correct

[ ] bug
[x] feature

Describe the pull request

Is ERPC support Windows platform? #299
To Reproduce

Expected behavior

Screenshots

Desktop (please complete the following information):

OS:
eRPC Version:

Steps you didn't forgot to do

[x] I checked if other PR isn't solving this issue.
[x] I read Contribution details and did appropriate actions.
[x] PR code is tested.
[x] PR code is formatted.
[x] Allow edits from maintainers pull request option is set (recommended).

Additional context

Hi @thewon86 , i was trying to add tests builds for mingw with your tcp layer. I had to do some changes also in source files due to errors. Can you confirm that these changes works for you?

@MichalPrincNXP Could you also review these changes? I added mingw tcp tests support so we see that server client may communicate.

#define ERPC_THREADS (ERPC_THREADS_PTHREADS)
It looks mingw32 dose'nt support #pragma weak, there is a error in my project

build/obj/erpc_arbitrated_client_setup.o:erpc_arbitrated_client_setup.cpp:(.text+0x13b): undefined reference to `g_client'
build/obj/erpc_client_setup.o:erpc_client_setup.cpp:(.text+0xb1): undefined reference to `g_client'
build/obj/lvgl_is_designer_client.o:lvgl_is_designer_client.cpp:(.rdata$.refptr.g_client[.refptr.g_client]+0x0): undefined reference to `g_client'

I'd modified one of define in erpc_client_setup.cpp

#if defined(__MINGW32__)
ClientManager *g_client;
#else
ClientManager *g_client;
#pragma weak g_client
#endif

and it works well
3. .\mingw64\bin\mingw32-make.exe build=release erpc Done
4. .\mingw64\bin\mingw32-make.exe build=release erpcgen Done
5. .\mingw64\bin\mingw32-make.exe build=release test
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.accel[.refptr.accel]+0x0): undefined reference to `accel'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.mass[.refptr.mass]+0x0): undefined reference to `mass'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.y[.refptr.y]+0x0): undefined reference to `y'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.x[.refptr.x]+0x0): undefined reference to `x'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.d[.refptr.d]+0x0): undefined reference to `d'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.c[.refptr.c]+0x0): undefined reference to `c'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.b[.refptr.b]+0x0): undefined reference to `b'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/
test/test_const/test_const_client_impl.o:test_const_client_impl.cpp:(.rdata$.refptr.a[.refptr.a]+0x0): undefined reference to `a'
collect2.exe: error: ld returned 1 exit status
mingw32-make[3]: *** [G:/misc/rpc/erpc-develop/mk/targets.mk:123: G:/misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/test_const_client_tcp_test] Error 1
mingw32-make[2]: *** [../mk/unit_test.mk:66: test_client_tcp] Error 2
mingw32-make[1]: *** [G:/misc/rpc/erpc-develop/mk/subdirs.mk:42: test_const] Error 2
mingw32-make: *** [G:/misc/rpc/erpc-develop/mk/subdirs.mk:42: test] Error 2

Hi @thewon86 could you try it with last commit? Also i don't understand why you posted errors with test_const. Build for mingw looks passing this test and you are mentioning only issue with g_client.

Hi @thewon86 could you try it with last commit? Also i don't understand why you posted errors with test_const. Build for mingw looks passing this test and you are mentioning only issue with g_client.

The issue with g_client has been solved.
Sorry, maybe, i don't understand how the test works.
.\mingw64\bin\mingw32-make.exe build=debug all
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/
misc/rpc/erpc-develop/test/test_arrays/MINGW64/tcp/gcc/test_arrays_client/Debug/obj/test/test_arrays/MINGW64/tcp/gcc/tes
t_arrays_client/Debug/test_unit_test_common_client.o:test_unit_test_common_client.cpp:(.rdata+0x0): multiple definition
of `.weak.c_x._ZN4erpc5Codec9getStatusEv'; G:/misc/rpc/erpc-develop/test/test_arrays/MINGW64/tcp/gcc/test_arrays_client/
Debug/obj/test/test_arrays/MINGW64/tcp/gcc/test_arrays_client/Debug/test_client.o:test_client.cp:(.rdata+0x0): first def
ined here
collect2.exe: error: ld returned 1 exit status
mingw32-make[3]: *** [G:/misc/rpc/erpc-develop/mk/targets.mk:123: G:/misc/rpc/erpc-develop/test/test_arrays/MINGW64/tcp/
gcc/test_arrays_client/Debug/test_arrays_client_tcp_test] Error 1
mingw32-make[2]: *** [../mk/unit_test.mk:66: test_client_tcp] Error 2
mingw32-make[1]: *** [G:/misc/rpc/erpc-develop/mk/subdirs.mk:42: test_arrays] Error 2
mingw32-make: *** [G:/misc/rpc/erpc-develop/mk/subdirs.mk:42: test] Error 2

Is that because all c_x be defined with weak & extern?
// Constant variable definitions
#pragma weak c_x
extern const int32_t c_x = 5;

and .\mingw64\bin\mingw32-make.exe build=release all
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/
misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/test/test_const/test_const_client_im
pl.o:test_const_client_impl.cpp:(.rdata$.refptr.accel[.refptr.accel]+0x0): undefined reference to `accel'
G:/misc/rpc/erpc-develop/mingw64/bin/../lib/gcc/x86_64-w64-mingw32/12.2.0/../../../../x86_64-w64-mingw32/bin/ld.exe: G:/
misc/rpc/erpc-develop/test/test_const/MINGW64/tcp/gcc/test_const_client/Release/obj/test/test_const/test_const_client_im
pl.o:test_const_client_impl.cpp:(.rdata$.refptr.mass[.refptr.mass]+0x0): undefined reference to `mass'
.......

They all be declared weak & extern

Hi @Hadatko , it works, everything done well.

@MichalPrincNXP any comments to this PR?


K3s Multus + Whereabouts doesn't work

Environmental Info:
K3s Version:
k3s version v1.30.0+k3s1 (14549535)
go version go1.22.2

Node(s) CPU architecture, OS, and Version:
Linux ph-lws-4c38d51ffd02-9840 5.14.21-150500.55.19-default #1 SMP PREEMPT_DYNAMIC Tue Aug 8 22:15:01 UTC 2023 (9908c29) x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:
1 server (planning for multiple nodes)
Describe the bug:
Unable to use K3s with Multus + Whereabouts.
Error: error adding container to network <network-name> : config file not found
Problem is related to whereabouts IPAM, switching to host-local IPAM makes the error disappear (but not desired)
Steps To Reproduce:

Installed K3s
Installed Multus + Whereabouts according to docs
Create NAD:

apiVersion: k8s.cni.cncf.io/v1
kind: NetworkAttachmentDefinition
metadata:
  name: rto
  namespace: default
spec:
  config: |-
    {
        "cniVersion": "0.3.1",
        "type": "bridge",
        "bridge": "br0",
        "ipam": {
            "type": "whereabouts",
            "range": "<IP_ADDRESS>/16",
            ],
            "log_file" : "/tmp/whereabouts.log",
            "log_level" : "debug"
        }
    }

Create Deployment that uses NAD:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: sample-deployment
  namespace: default
spec:
  replicas: 1
  selector:
    matchLabels:
      app: samplepod
  template:
    metadata:
      labels:
        app: samplepod
      annotations:
        k8s.v1.cni.cncf.io/networks: rto@eth2
    spec:
      containers:
        - name: samplepod
          image: alpine
          command:
            - /bin/ash
            - '-c'
            - 'trap : TERM INT; sleep infinity & wait'
          securityContext:
            capabilities:
              drop:
                - ALL
            runAsUser: 1000
            runAsNonRoot: true
            allowPrivilegeEscalation: false
            seccompProfile:
              type: RuntimeDefault

Expected behavior:
Pod started with event: Add eth2 [10.0.x.x/16] from default/rto
Actual behavior:
Pod pending with error:
Failed to create pod sandbox: rpc error: code = Unknown desc = failed to setup network for sandbox "42ece46993378dd1192b28fa08f48090fd490486fac9e99c94829ea9c4f05e4d": plugin type="multus" failed (add): [default/sample-deployment-59cf9c969c-zkqjg/378fdef6-0b26-4d53-a2ff-39b6f164fbb6:rto]: error adding container to network "rto": config file not found

Additional context / logs:
Adding a symbolic link for Whereabouts seems to help as a workaround
ln -s /var/lib/rancher/k3s/agent/etc/cni/net.d/whereabouts.d /etc/cni/net.d/whereabouts.d
See also https://github.com/k8snetworkplumbingwg/whereabouts/issues/465#issuecomment-2230372585

While it would be great if components like CNI plugins could respect custom directories, the ability to install k3s using "standard" paths (aka kubeadm paths) instead of /var/lib/rancher would solve a lot of pain points. I am using k3s with kubevirt and ''expected directory issue seems to be the biggest hurdle.

I'm going to update the documentation to address this. You can either pass the correct paths in the NetworkAttachmentDefinition, or run ln -s ln -s /var/lib/rancher/k3s/agent/etc/cni /etc/cni on every node in the cluster.


Update HTTP API header file

Totally agree - would also be nice if the demo service used the server


Update json in sslyze integ test

Please note that while we are transitioning from travis-ci to AWS CodeBuld, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Issue # (if available): fixes https://github.com/awslabs/s2n/issues/1719
Description of changes:
Update the test to handle the changes to the JSON. I created a separate issue to address the underlying problem with parsing JSON: https://github.com/awslabs/s2n/issues/1723
I couldn't find examples of the JSON, so I had to figure it out from sslyze's source code. I verified that the test passed after my change. I also verified that the error message works after my change by modifying sslyze to fail and checking that the test then failed with the message "SSLyze Error: BUG_IN_SSLYZE FAILED".
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1724 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1724      +/-   ##
==========================================
- Coverage   81.76%   81.72%   -0.05%
==========================================
  Files         224      224
  Lines       16282    16337      +55
==========================================
+ Hits        13313    13351      +38
- Misses       2969     2986      +17


fix animation of movement

when we move the graph, and the center of graphs moves out the window, whole graph disappears, but when we move back it appears

It issue can by fixed by replace IntOffset to DpOffset
https://github.com/JetBrains/compose-multiplatform/issues/2198


zopfli doesn’t have wheels for Python 3.9

py-zopfli, used as a dependency for woff fonts, doesn’t have wheels for Python 3.9. Their last commit is more than one year ago, and they don’t respond to obp/py-zopfli#7, so we can think that the project is not active anymore.
Having no wheel is bad, because the installation from source requires a compiler. It makes the installation of fontools[woff] fail if no compiler is available.
zopfli is a dependency of fonttools with woff support, but I think that it’s actually not required: there’s a fallback if the package is missing. So we could be possible to remove zopfli from the dependencies, and make it installable with a dedicated "zopfli" option if possible. There’s also the possibility to use another package (like zopflipy), but we have no guarantee that other packages will be maintained in the future.

x-ref for anybody who encounters this

https://pypi.org/project/zopflipy/#files
https://github.com/hattya/zopflipy

Hey, sorry I didn't see this when it was first posted. I have commit access to py-zopfli and can try to push updated wheels for 3.9 next week. I didn't know about the other zopflipy, we can support both of they have a compatible interface.

Thanks a lot for the wheels. I think that we can close this issue, unless you want to support zopflipy too.

@anthrotype is this still valid?


Fix closure issues in streams

Purpose
Users were not able to use input stream and output streams which are references to the function arguments in streams query, because closures in streams were not properly implemented.
Fixes #10970
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #19656 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #19656   +/-   ##
================================================
  Coverage            20.51%   20.51%
================================================
  Files                   52       52
  Lines                 1394     1394
  Branches               216      216
================================================
  Hits                   286      286
  Misses                1095     1095
  Partials                13       13

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 81ebe66...061ed33. Read the comment docs.


[go_router] .pushReplacement() and pushAndRemoveUntil() equivalent

Use case
With navigator 1.0, we can use 2 methods:

.pushReplacement()
pushAndRemoveUntil()

Those where very useful, especially when the user is going through a wizard. The wizard could walk the user through different screens and those screens could be removed at the end of the wizard so the user cannot go back to them.
I didn't see any equivalent in GoRouter. Or maybe I missed them?

Proposal

It would be useful to implement .pushReplacement() and pushAndRemoveUntil() equivalent.

Duplicate of https://github.com/flutter/flutter/issues/106402

@exaby73 Actually, unlike the other issue, this one also includes pushAndRemoveUntil(). Do we want to keep it or modify the other issue?

Since it's very similar, maybe adding that to the original issue as a suggestion


Implement / optimize the shootout benchmarks

 Issue by kud1ing
Thursday Oct 16, 2014 at 06:01 GMT
For earlier discussion, see https://github.com/rust-lang/rust/issues/18085
This issue was labelled with: A-an-interesting-project, A-benchmarks, P-low in the Rust repository

This is supposed to be a meta issue for optimization efforts like http://www.reddit.com/r/rust/comments/2j2ij3/benchmark_improvement_reverse_compliment/
The benchmarks to optimize (where rustc is slower than g++, according to http://benchmarksgame.alioth.debian.org/u64q/rust.php):

[x] binary-trees
[x] pidigits
[x] mandelbrot
[x] fannkuch-redux (fixed by #16999)
[ ] fasta
[ ] k-nucleotide
[ ] n-body
[x] reverse-complement (fixed by #18143, #18357)
[x] spectral-norm (fixed by #17989)
[ ] regex-dna
[ ] fasta-redux (Rust version missing)

Secondary benchmarks:

[ ] meteor-contest
[ ] chameleos-redux
[ ] thread-ring
[ ] binarytrees-redux

I would raise the priority just slightly on some of these benchmarks, i.e. regex-dna and n-body.  Those two represent Rust in a poor way.

n-body need a simd implementation. It shoud be quite easy to translate the best program to rust.
For regex-dna, the bottleneck is the regex library. Much more difficult to improve.

Heads up, that most benchmarks don't build for Rust 1.0-beta. :/

Most are proposed. Lack stable TypedArena and simd for the last 3.

Thanks again.

There are submitted implementations of all the current benchmarks now, mirrored at https://github.com/TeXitoi/benchmarksgame-rs


DBeaver can't delete primary key contraint

Description
I can't delete primary key with DBeaver, it says
errno 150 primary key is not correctly formed.
If I execute a script with set foreign_key_checks=off and then do it again, it gives me that error again.
If I do it via mysql commandline, (disabling foreign key checks too), doing the SAME query as dbeaver (ALTER TABLE dbname.tablename DROP PRIMARY KEY;  )it works....
Also, it seems bugged because even if I press refresh button, it still gives me incorrect db scenario. Only closing and reopening dbeaver finally gives me the correct db scenario just updated.
DBeaver Version
Version <IP_ADDRESS>207311845
Operating System
ArchLinux
Database and driver
Server version: 10.5.15-MariaDB-0+deb11u1 Debian 11
Driver: InnoDB
Steps to reproduce
I tried to delete the only one constraint (that has two ext ref), and then save, and gives me: errno 150 primary key is not correctly formed.
Even if I disable foreign key checks, it still doesn't allow me to perform that operation.
Doing it manually via commandline works.
Additional context
This is the output of SHOW ENGINE INNODB STATUS after dbeaver failed the command : https://hastebin.com/ucituyeyef.yaml

Please, send the query which you see in the Query Manager when trying to delete constraint.

@All3xJ Thank you for the report. We will look what can be done with it.

@All3xJ, could you send the contend which was in the provided hastebin (it's empty now) or at least the query which you see in the Query Manager when trying to delete the mentioned constraint? Also table DDL will be helpful.

There have been no updates on the issue for a long time.
If it is still actual for you - feel free to reopen it with additional info.


Derive Hash on a bunch of types

Draft until https://github.com/rust-bitcoin/rust-bitcoin/pull/1066 merges so we have Hash on secp256k1::ecdsa::Signature.
In preparation for being able to derive Hash on all types in miniscript, derive Hash on all of the required types.
This PR includes all the changes in https://github.com/rust-bitcoin/rust-bitcoin/pull/933/files and hence supersedes it.
ref: https://github.com/rust-bitcoin/rust-miniscript/issues/226

I'd ACK this, why is it a draft?

Was waiting for CI run to finish before taking off draft and forgot about it.


Add possibility to iterate only arrays when wildcard is used

const data = {
  name: 'James'
}

const anotherData = [1, 2, 3]

console.log(dot.get(data, '*'))

If wildcard is used here, it will iterate over the data object fields and return their values. I need it to iterate only over arrays and return the values of the array. So, in the above example I want it to return null. Same search on anotherData would return array values ([1, 2, 3]).
Is this possible to add to the library?
Thanks

Hi @VladShcherbin, Thanks for Issues !
I want to use wildcards to operate regardless of objects or arrays.
It is due to higher versatility.
I am sorry, but it seems to be unnecessary to add it as a library function :cry:

@tsuyoshiwada I understand that it iterates both objects and arrays for higher versatility, but there are use cases like mine which currently can't be solved and you'll have to write your own library for that.
Since there is already a great library, I thought maybe it can be added here. For example, if you provide third argument as an options array:
dot.get(data, '*', { default: 'no data', iterateObjects: false })

@VladShcherbin That makes sense :+1:
I misunderstood that it changed behavior of the get method.
I think that the content you proposal is good.
However there are points to worry about.
If you add an option with the get method, how do you like how set or remove methods are behaving?

@tsuyoshiwada in my project I only need this for getters - forEach, get, maybe map methods.
Maybe it's possible to add this feature to getters first and if someone will also need this for setters (and removals), add it later.
It's easier to add features when you know the use cases for them instead of thinking of possible usage examples for other methods. I can probably thinks of some examples for other methods, but all of them will be fictional, not real ones. What do you think?

@VladShcherbin I agree with you about that.
I'd like to implement options for forEach, map, get.
I will let you know here again as soon as the implementation is over :smiley:
Thanks !

@tsuyoshiwada yeah, it's working for me. Thank you!

@VladShcherbin Thank you for checking! Released a version that removed alpha :tada:


SolrDedupServiceImpl: potential bug found in method cleanIndex

Bug Description
We found a potential bug in method cleanIndex:
https://github.com/4Science/DSpace/blob/04c7aa622a94e048a056dee462d8220592764971/dspace-api/src/main/java/org/dspace/app/deduplication/service/impl/SolrDedupServiceImpl.java#L637
Should this check be modified to i != null?

As far as I understand the code I would not say this is a bug. If there is an object in the index, which does not exist in the database, it's removed from the index. If it exists it should not be removed, as long as the index should not be erased completely. If it's expected to erase all of the index, this check is not needed at all and all objects should be unindexed (but I guess it does not make sense to erase the index by iterating through all objects).
So I don't think this is a bug and would guess it's intended as it is.

@olli-gold , thanks. That makes sense. I'll close the issue as won't fix.

@olli-gold , I have to reopen this issue.
If i is null, then in
https://github.com/4Science/DSpace/blob/04c7aa622a94e048a056dee462d8220592764971/dspace-api/src/main/java/org/dspace/app/deduplication/service/impl/SolrDedupServiceImpl.java#L642
the null value is passed as second method parameter.
At the end, this will result in a NullPointerException in
https://github.com/4Science/DSpace/blob/04c7aa622a94e048a056dee462d8220592764971/dspace-api/src/main/java/org/dspace/app/deduplication/service/impl/SolrDedupServiceImpl.java#L533
because of null object access in item.getID() and item.getType(). Do you agree?

Oh, yes, you are right. This is obviously a bug, which needs to be fixed.


build(dev-mode): Upgrade Go module dependencies

@Mmarzex I've updated Go version and bumped module deps, I think we can merge it


Error in StructuredQueryOutputParser using SelfQueryRetriever with Pinecone

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.

Example Code
Im using a standard SelfQueryRetriver to extract relevant documents (car listings) that match a user query. Its has been working pretty well but recently it started giving me errors (stack trace attached).
ise
     retriever = SelfQueryRetriever.from_llm(          llm,          vectordb,          document_content_description,          metadata_field_info,          verbose=True      )
Error Message and Stack Trace (if applicable)
OutputParserException('Parsing text\njson\n{\n    "query": "with bluetooth and a reversing camera recent",\n    "filter": "or(eq(\\"vehicle_type\\", \\"Hatchback\\"), eq(\\"vehicle_type\\", \\"Sedan\\")), in(\\"location\\", [\\"Westgate\\", \\"North Shore\\", \\"Otahuhu\\", \\"Penrose\\", \\"Botany\\", \\"Manukau\\"])"\n}\n\n raised following error:\nUnexpected token Token('COMMA', ',') at line 1, column 65.\nExpected one of: \n\t* $END\n')Traceback (most recent call last):
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser_state.py", line 77, in feed_token
action, arg = states[state][token.type]
~~~~~~~~~~~~~^^^^^^^^^^^^
KeyError: 'COMMA'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain/chains/query_constructor/base.py", line 56, in parse
parsed["filter"] = self.ast_parse(parsed["filter"])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/lark.py", line 658, in parse
return self.parser.parse(text, start=start, on_error=on_error)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parser_frontends.py", line 104, in parse
return self.parser.parse(stream, chosen_start, **kw)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser.py", line 42, in parse
return self.parser.parse(lexer, start)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser.py", line 88, in parse
return self.parse_from_state(parser_state)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser.py", line 111, in parse_from_state
raise e
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser.py", line 102, in parse_from_state
state.feed_token(token)
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/lark/parsers/lalr_parser_state.py", line 80, in feed_token
raise UnexpectedToken(token, expected, state=self, interactive_parser=None)
lark.exceptions.UnexpectedToken: Unexpected token Token('COMMA', ',') at line 1, column 65.
Expected one of:
* $END
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain_core/runnables/base.py", line 1246, in _call_with_config
context.run(
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain_core/runnables/config.py", line 326, in call_func_with_variable_args
return func(input, **kwargs)  # type: ignore[call-arg]
^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain_core/output_parsers/base.py", line 168, in
lambda inner_input: self.parse_result(
^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain_core/output_parsers/base.py", line 219, in parse_result
return self.parse(result[0].text)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/dharshana/.local/share/virtualenvs/tina-virtual-assistant-eLldwkZS/lib/python3.11/site-packages/langchain/chains/query_constructor/base.py", line 63, in parse
raise OutputParserException(
langchain_core.exceptions.OutputParserException: Parsing text
{
    "query": "with bluetooth and a reversing camera recent",
    "filter": "or(eq(\"vehicle_type\", \"Hatchback\"), eq(\"vehicle_type\", \"Sedan\")), in(\"location\", [\"Westgate\", \"North Shore\", \"Otahuhu\", \"Penrose\", \"Botany\", \"Manukau\"])"
}

raised following error:
Unexpected token Token('COMMA', ',') at line 1, column 65.
Expected one of:
* $END
Description
im getting
lark.exceptions.UnexpectedToken: Unexpected token Token('COMMA', ',') at line 1, column 65.
Expected one of:
* $END
Seems its not happy with a COMMA.
Im not entirely sure if the cause of the error is change in the Pinecone query api or an update in langchain version
System Info
    "langchain": {
        "hashes": [
            "sha256:29d95f12afe9690953820970205dba3b098ee1f7531e80eb18c1236d3feda921",
            "sha256:b40fbe2b65360afe6c0d5bbf37e79469f990779460640edde5b906175c49807e"
        ],
        "index": "pypi",
        "version": "==0.1.7"
    },
    "langchain-community": {
        "hashes": [
            "sha256:bd112b5813702919c50f89b1afa2b63adf1da89999df4842b327ee11220f8c39",
            "sha256:c56c48bc77d24e1fc399a9ee9a637d96e3b2ff952e3a080b5a41820d9d00fb3c"
        ],
        "index": "pypi",
        "version": "==0.0.20"
    },
    "langchain-core": {
        "hashes": [
            "sha256:34359cc8b6f8c3d45098c54a6a9b35c9f538ef58329cd943a2249d6d7b4e5806",
            "sha256:d42fac013c39a8b0bcd7e337a4cb6c17c16046c60d768f89df582ad73ec3c5cb"
        ],
        "markers": "python_full_version >= '3.8.1' and python_version < '4.0'",
        "version": "==0.1.23"
    },
    "langchain-openai": {
        "hashes": [
            "sha256:2ef040e4447a26a9d3bd45dfac9cefa00797ea58555a3d91ab4f88699eb3a005",
            "sha256:f5c4ebe46f2c8635c8f0c26cc8df27700aacafea025410e418d5a080039974dd"
        ],
        "index": "pypi",
        "version": "==0.0.6"
    },

hi.. so two things

i don't agree with your logical assessment of the query if This filter query checks if the vehicle_type is either "Hatchback" or "Sedan" and if the location is in the list ["Westgate", "North Shore", "Otahuhu", "Penrose", "Botany", "Manukau"]. then the query should be
and(
or(
eq("vehicle_type", "Hatchback"),
eq("vehicle_type", "Sedan")
),
in("location", ["Westgate", "North Shore", "Otahuhu", "Penrose", "Botany", "Manukau"])
)"
note () specifying the operators to which the or operator should apply

As im using a SelfQueryRetriver i didnt write this query. Should'nt the langchain prompts be asking the llm to construct the query in the correct syntax? btw im using  model="gpt-4-turbo-preview"

hi.. so two things

i don't agree with your logical assessment of the query if This filter query checks if the vehicle_type is either "Hatchback" or "Sedan" and if the location is in the list ["Westgate", "North Shore", "Otahuhu", "Penrose", "Botany", "Manukau"]. then the query should be
and(
or(
eq("vehicle_type", "Hatchback"),
eq("vehicle_type", "Sedan")
),
in("location", ["Westgate", "North Shore", "Otahuhu", "Penrose", "Botany", "Manukau"])
)"
note () specifying the operators to which the or operator should apply
As im using a SelfQueryRetriver i didnt write this query. Should'nt the langchain prompts be asking the llm to construct the query in the correct syntax? btw im using  model="gpt-4-turbo-preview"

Hello brother, I have encountered the same problem. Have you resolved it


Add literals alongside enums

Generating literal types alongside string and int Enums. We really need it in our code so have this in our custom templates. I thought there are others who would like to have it in the default template.

Are you using literals instead of enums? Is there a particular limitation of enums that makes that necessary? Trying to understand if this should be an option in config (to generate literals instead) or if it makes sense to completely replace enums with literals 🤔

We are using literals instead of enums in a public library with a lot of choice parameters where having enums would mean a lot of typing for the user and generally more noise in the code. There are still many developers out there who prefer to use enums instead of literals, so making it configurable would be a much better option!

Replaced by https://github.com/openapi-generators/openapi-python-client/pull/1076


D register and possible bug?

Just found this, thanks, going to make my life a lot easier.
I'm running the "buggy" monitor from sbc09 (https://github.com/6809/sbc09)

Please add support for the D register "pshs d" instead of "pshs a,b"
When compiling monitor.asm I get an error on line 1867,  unexpected character '"'. There's no difference between this line and the dozens before it. Apologies, I have never used Rust before (and having read up on it to get this going, it looks a lot more streamlined than the C + Makefile environment I'm used to, and don't get me started on cmake...) so I have no idea how to debug this myself.
This compiles:
"label fdb  fn1,fn2,fn3,fn4 ; comment".
This fails with "missing number or label"
"label fdb  fn1,fn2,fn3,fn4 ; comment," <- note comma at end.

Thanks!

Hi. Thanks for the issue.

I added support for psh and pul D
monitor.asm has errors on lines 1864 and 1867 where the fcb directives should be fcc. Just change both of those to fcc and it builds.
I fixed a bunch of parsing issues related to fdb and fcb directives.

Let me know if you run into anything else!

Hiya
That was quick!

I added support for psh and pul D
Thanks!
monitor.asm has errors on lines 1864 and 1867 where the fcb
directives should be fcc. Just change both of those to fcc and it builds.
Oh I see. Sheesh I should have picked that up. My bad.
I fixed a bunch of parsing issues related to fdb and fcb directives.
Great.

I have a feeling that I will be learning at least a bit of Rust in the future.
W


Lab 3b: Failed to install torchvision

The first cell in Lab 3b fails with the following error:

If I ignore it and continue running, subsequent cells will complain that torchvision is not installed.
This is using the version created in Workshop Studio on 6/2/2023 12:00 (GMT+2).

Try it with the "Data Science 3.0" Image. It works for me then.


Doubt

I get a 99.5 average in my desktop for my desktop but in the site a 55 why is it so.

My url is joelhanson.ml

Hi @Joel-hanson
We are running a version of Lighthouse in a docker VM, unfortunately since we moved to LH 2+ we are facing a lot of variation when running it in this environment. Last night your site hit 100, we will investigate further to try to make the metric more stable.
Cheers,
Julian.

thanks @juliantoledo


Display Attributes

Nice Integration!
One question, how can I access the dingle attributes? I would like to see on first sight how many people are online and use HomeAssistant as a history viewer.

platform: template
sensors:
ts_clients:
friendly_name: "TS Online Clients"
value_template: "{{ states.sensor.SERVERNAME.attributes.serverinfo.clientsonline }}"


pad resnet eval data in model_train

asserted if eval sample count is different from total eval file count.

testing here https://wandb.ai/chenyuxyz/tinygrad-examples_mlperf/runs/xyt2r6p0?nw=nwuserchenyuxyz, will merge if all looks good

new link https://wandb.ai/chenyuxyz/tinygrad-examples_mlperf/runs/ogkssx8l/overview?nw=nwuserchenyuxyz

trained 10 epochs, looks good


CLI command to change wallpaper path.

Is your feature request related to a problem? Please describe.
Currently the only way to tell Hyprpanel what wallpaper I'm using is by changing it through the settings. This makes it hard for anyone using a pywall/wallpaper script to integrate Hyprpanel into their config.
Describe the solution you'd like
A CLI command to pass a wallpaper path to Hyprpanel would make integrating it allot simpler. Something like ags -r "setWallpaperPath(path)". This way the color scheme of the bar can be made to adapt to a wallpaper change via some other means such as a script using it's existing mutagen setup. (Ideally this would only actually change the wallpaper if you have "Apply Wallpapers" on)
Describe alternatives you've considered
I could probably make a script to change the variable in the options.json file, but this seems silly.
I cold also set the path of the wallpaper to be a file which I replace using a script, but then the problem would be file extensions.
Additional context
What I did with waybar was just import the file generated by pywal into the css, and connected the wal -i command to waypaper, changing waybar as I change my wallpaper. Then in my hyprland config I just bound a key to selecting a random wallpaper, through waypaper --random

closed by #463


Some IO::FileDescriptor methods are implemented in Crystal::System::File

flowchart BT
File --> IO::FileDescriptor
File --> Crystal::System::File
IO::FileDescriptor --> Crystal::System::FileDescriptor

These methods of IO::FileDescriptor are implemented in Crystal::System::File instead of Crystal::System::FileDescriptor:

#fsync -> #system_fsync
#flock_shared -> #system_flock_shared
#flock_exclusive -> #system_flock_exclusive
#flock_unlock -> #system_flock_unlock

Thus it is always a compilation error to call them on IO::FileDescriptors that aren't Files:
STDOUT.fsync            # Error: undefined method 'system_fsync' for IO::FileDescriptor
IO.pipe[0].flock_shared # Error: undefined method 'system_flock_shared' for IO::FileDescriptor

Either the methods in IO::FileDescriptor should be moved down to File, or the #system_ implementations should be moved to Crystal::System::FileDescriptor.

IIUC, the only difference between IO::FileDescriptor and File is that the latter has an associated path, captured in an additional ivar.
So I suppose all methods that operate on a file descriptor make sense for IO::FileDescriptor and there's no reason to restrict them to File instances.


Update Readme to fix incorrect comment

The code sample is for notepad and was referring to alarmclock

Hi @ianceicys, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

👍


Lazy loading with Checkbox selection not working

I think there is an issue with Lazy Loading and Checkbox Selection method. If I combine both the examples checkbox selection is not working properly and sometimes paging is also not visible. I am sharing an live example for the same problem -
example
Step-1

Click on select all then click deselect all, its working fine.
Click on any one or row checkbox then click select all checkbox, Its working fine. Now, If I click on deselect all this is not working or you can say deselect-all event is not fired.

If you find any solution plz let me know.
Thanks in advance. :)

@hjalmers still there is the issue. I updated the example with the latest build still I am able to reproduce the same issue.

Hey, @hjalmers  https://github.com/hjalmers/angular-generic-table/commit/d5ab4f7e99bfbe2ee98fadb615a62152476b4b7e there is serious issues occurs.
Step to reproduce -

First select 2 or 3 checkbox and then select all it will always push current page size data into selectedRows every time when i click on select/deselect all checkbox.

@LokeshBoran, I haven’t published a new version containing the fix yet so you you have to hang tight a little while longer. I’ll open this issue again until it’s published though:)

@LokeshBoran the fix is now available with version 4.15.0. Please note however that it's a bit hard for the table to know the state of each row when it's loaded from the server. I'd probably suggest adding a property on each row and store the state on the server with the rest of the data if you want to persist the state. If you enable caching like I've done in this example it will work (not the select all option for data that hasn't been loaded yet though).
The problem with caching is that if the data on the server changes the table won't update when you paginate between pages that has already been loaded/cached. If caching is disabled on the other hand, the table will request new data when you go to a new page and the rows will get new row ids at the same time, that's also the reason why the state is not persisted.
We could keep the ids and just update the data but what if a new row has been added on the server and/or the order has changed? We can't just rely on row or page indexes to keep the states in sync when the records are fetched from the server and without a unique id it's very hard to sync a local record with a record received from the server. I've added the ability to set a custom row id #245 which simplifies syncing state with server as well.

@hjalmers thanks a lot for your changes. Looking forward to seeing more changes related to usability and stability of the library.
And yes if I want to deselect checkbox on page-change than on gt-page-changed-lazy event I have to call deselectAllRows method.

No problem, yeah there are some manual steps involved now like deselecting rows if data set is replaced etc. I was also thinking about what should happen if you've checked "check all" and you go to the next page, the server will fetch data from the sever and I guess it should select them if they're not already selected otherwise the state of the select all checkbox won't be correct.


Error when using sonnet 2 and tensorflow 2.

In tensorflow 2 eager execution is enabled by default.  Inverted the assert to check if it is not enabled instead.
Final error lines:
    assert tf.executing_eagerly(), "Sonnet v2 requires TensorFlow 2"
AssertionError: Sonnet v2 requires TensorFlow 2

Here is the link to the documentation of tensorflow v2.8.0 (when I'm writing this) https://www.tensorflow.org/api_docs/python/tf/executing_eagerly, where they state that eager execution is enabled by default and thus the funciton will return true. In this case the assert will be triggered by default by using tensorflow 2 and sonnet 2.
Configuration (Package=version):
tensorflow-gpu=2.8.0rc0
tensorflow-probability=0.15.0
dm-sonnet=2.0.1.dev0
dm-tree=0.1.6
absl-py=1.0.0
numpy=1.22.0
tabulate=0.8.9
wrapt=1.13.3

Hi @tomhennigan, thank you for your quick response. Apologies for my simple mistake, realize that it was quite apparent from the assert behavior (embarrassing)...
Thank you for your recommendation to check the TF version, it seems that I'm using the correct version and that eager execution is in fact enabled when I start the program. However it is disabled somewhere, which got me curious to look into it further. The codebase I'm modifying (or building on top of) is using on the Estimator API, and after some digging around it seems that it is Estimator that will switch off eager execution. From the class definition of Estimator in estimator.py (https://github.com/tensorflow/estimator/blob/777318086559b5ee473fa097de95e2beec5adf78/tensorflow_estimator/python/estimator/estimator.py#L125):
'''
@compatibility(eager)
  Calling methods of `Estimator` will work while eager execution is enabled.
  However, the `model_fn` and `input_fn` is not executed eagerly, `Estimator`
  will switch to graph mode before calling all user-provided functions (incl.
  hooks), so their code has to be compatible with graph mode execution. Note
  that `input_fn` code using `tf.data` generally works in both graph and eager
  modes.
  @end_compatibility
'''

This is done prior to the initialization of sonnet modules, thus I believe this is the reason eager execution is disabled when tf.init_scope() is called. Not sure if there is a workaround for this.
Again, thank you for your help, I wish you a wonderful day! :)

Thank you @tomhennigan! Yes I see, it makes sense.
Hm yes good idea, that might work. I am planning to rework the code base for TF2 anyways so it is not of major importance to make it work as is, I just have it as a reference. Thank you, it's very nice of you offering to tinker with an example and come up with a fix, but no need to spend time on that in this case, since I plan on reworking the code quite a bit.
I see no worries, neither am I. May I ask what you would use as training framework together with sonnet? I saw in the repo that it does not come with one out of the box:

"Sonnet does not ship with a training framework and users are encouraged to build their own or adopt those built by others."

Would you recommend writing my own or to incorporate another API (like Keras or which API in that case)?
For a bit more context, this is the code base I am tinkering around with: https://github.com/deepmind/deepmind-research/tree/master/learning_to_simulate. Which is using sonnet, graph_nets (https://github.com/deepmind/graph_nets#graph-nets-library), and estimator for a TF1 version. In the graph_nets Github they have this example (https://github.com/deepmind/graph_nets#run-sort-demo-in-browser--run-tf2-version, click the TF2 version) and it seems that they are writing their own training framework at least.


Support as part in insert statement for mysql and generic dialects

Example: INSERT INTO table (date) VALUES (2024-01-01) AS alias (mek)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

-1 of 46 (97.83%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 87.706%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser/mod.rs
8
9
88.89%

Files with Coverage Reduction
New Missed Lines
%

src/ast/mod.rs
1
79.52%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
19698

Relevant Lines:
22459

💛  - Coveralls

I apologize for the delay in reviewing

Marking as Draft to signify this PR is no longer waiting on review. Please mark it as ready for review when it is ready for another look.

@alamb  Thanks for comments.


rocm 4.5.2 compliant

This is not meant to be merged directly. I just wanted to give an example of changes that you need to make to run on AMD GPUs (tested with rocm-4.5.2). Hope this is helpful.

This has become the AMD branch, rather than merging it into main


Continue request --> got migration; affects [travis wordpress myget jenkins-plugin node nuget]

Closes #7214
Refs #4655
This PR migrates regularUpdate() from request and callbacks to got and async/await and all the knock-on effects of that.

Warnings

:warning:
This PR modified service code for jenkins but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against e5c96c6de817ef05dad384f58c77218aa1de1adf


add: steam-deb

[x] Edit packagelist
[x] Add initial pacscript
[ ] ~Contact devs (via email)~
[x]  Add maintainer to pacscript

Add breaks

I already added it in the past before I realized through the CI logs that the deb package would not install without Steam Launcher


Add shorthands for abbreviations with lots of punctuation

Concretely:
\newcommand{\eg}{e.g.,\ }
\newcommand{\etal}{et al.\xspace}
\newcommand{\ie}{i.e.,\ }
\newcommand{\aka}{a.k.a.\ }

@jhuggins before we include, can you investigate why there is a \  after some of these, and \xspace after others? I'm not sure what the difference is / why one would want either one

\xspace essentially only adds a space if the next symbol is a letter, so you get  et al.) rather than et al. ). It's not perfect, however: https://tex.stackexchange.com/questions/86565/drawbacks-of-xspace
But for ie, eg, and aka, we know there should always be space, so it's safe to use \ .
Note, btw, that we also use \xspace for names with special characters and some existing shortcuts (e.g., \iid, \as, and \aev).


French translation

Hey!
Thanks for the translation!
You may want to add yourself as the French translation provider under the keys:
about_message
in all


Ability to rename terminal tabs

It will be useful to rename terminal tabs to better identify which tab is used for.
Current view:

Maybe we could also get rid of the (2), (3), ... as well


Fix F.softmax and F.log_softmax with axis=-1 on gpu

This PR fixes a bug in #5381.
_get_tensor4d_shape implicitly assumed axis is nonnegative.  (shape[slice(axis + 1, len(shape))] doesn't work if axis=-1.)

Jenkins, test this please.

LGTM! Test failures are not related to this PR.


Guard against SSR leaks in  constructor

We should have a guard around the history.listen call in the <Router> constructor to be sure we are in a persistent environment first (see https://github.com/ReactTraining/react-router/issues/6060#issuecomment-424575126).
The ideal place for this listener would be componentDidMount, but we can't move it there and preserve backwards compatibility with existing code. It would be a breaking change, so we can talk about doing that in v5.

Without looking at the code changes (on my phone), I will say in pretty confident you can listen inside of cDM and have it work with Redirect. We've been spending a lot of time on this exact issue with Redux (the last year, including multiple meetings with the React team) and have a solution we're implementing right now.
If anything, cDM is going to be required for forwards compatibility. If React decides to not mount a subtree after initialization, you can easily have weirdness around that subscription.

I don't know if this is the best solution, but in my PR I did a isStatic check for constructor and cDM and called listen depending on the situation.

@timdorr I tried that route but we did have some failing tests with the way <Redirect> currently works in v4. e.g. we have a test for multiple redirects on the initial render. Now, you could argue that's a bad test and we don't actually want that behavior, and I would whole-heartedly agree. But there's no way we can get that test to work using cDM, and my goal with 4.4 is to have no breaking changes.
@frehner Ya, we should probably do exactly that. Thanks for pointing that out :)

It's not so much that it's a bad test (multiple Redirects should definitely be handled), but that it's checking the implementation, not the effects. It's assuming a world where the React lifecycle is a one-time, synchronous thing. But that test would pass if it gave the React lifecycle a chance to run its course.
A simple fix to increase correctness would be to listen to the history and expect it to navigate through the redirect chain and then check what the render output is.

Completely agree, @timdorr. I'm just concerned that someone might be relying on this behavior in a SSR scenario and we will break their app with 4.4 if we change it. Again, I'm totally down with changing this in v5.

Prior to the 4.4 changes, we did some wacky stuff with checking if we were in a staticContext or not. We could do the same thing to maintain the same behavior. 🤷🏻♂


Difficulty utilizing both UIKit and SwiftUI modules

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[X] I have read the wiki page and cheat sheet, but there is no information I need.
[X] I have searched in existing issues, but did not find a same one.
[X] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
I'm getting various "multiple commands produce .... Kingfisher.framework" when importing both Kingfisher and KingfisherSwiftUI frameworks on 5.15.8.
What
I'm forced to use 5.15.8 due to lots of legacy usage on my app, and now want to utilize SwiftUI as well. I've got a Swift Package that I depend upon that references Kingfisher like so:
Reproduce

let package = Package(
    name: "MediaKit",
    dependencies: [
        .package(url: "https://github.com/onevcat/Kingfisher.git", from: "5.15.8"),
    ],
    targets: [
        .target(
            name: "MediaKit",
            dependencies: [
                .product(name: "Kingfisher", package: "Kingfisher"),
                .product(name: "KingfisherSwiftUI", package: "Kingfisher"),
            ]
        ),

In my main app, I add MediaKit as a package dependency for the project. When I build, I get these two errors:
Multiple commands produce '/Users/tmaly/Library/Developer/Xcode/DerivedData/MyApp-gxywjyvskcjlfzafdrwkjcarggvl/Build/Products/Debug-iphonesimulator/MyApp.app/PlugIns/MyAppTests.xctest/Frameworks/Kingfisher.framework'

Multiple commands produce '/Users/tmaly/Library/Developer/Xcode/DerivedData/MyApp-gxywjyvskcjlfzafdrwkjcarggvl/Build/Products/Debug-iphonesimulator/MyApp.app/Frameworks/Kingfisher.framework'

Now, I imagined it may be because I'm stating dependencies upon both Kingfisher and KingfisherSwiftUI, so I remove one or the other and I then get a different error:
Undefined symbol: _$s10Kingfisher0A7WrapperVAASo11UIImageViewCRbzlE8setImage4with11placeholder7options17completionHandlerAA12DownloadTaskVSgAA8Resource_pSg_AA11Placeholder_pSgSayAA0A15OptionsInfoItemOGSgys6ResultOyAA08RetrievefS0VAA0A5ErrorOGcSgtF
Among other errors. It sounds like I may need both libraries to avoid the undefined symbol issues. But how do I get past the multiple commands produce framework error?

Multiple commands produce '/Users/tmaly/Library/Developer/Xcode/DerivedData/MyApp-gxywjyvskcjlfzafdrwkjcarggvl/Build/Products/Debug-iphonesimulator/MyApp.app/PlugIns/MyAppTests.xctest/Frameworks/Kingfisher.framework'

Multiple commands produce '/Users/tmaly/Library/Developer/Xcode/DerivedData/MyApp-gxywjyvskcjlfzafdrwkjcarggvl/Build/Products/Debug-iphonesimulator/MyApp.app/Frameworks/Kingfisher.framework'

This seems meaning that you are trying to embed Kingfisher to both your test target and the actual app target. Maybe you can check if you have contained Kingfisher to your test target unexpectedly? Usually you do not use Kingfisher directly in your test target, so if there is no good reason, there should be no need to link (and/or embed Kingfisher) there.

@onevcat OK I believe I may have found the issue, under my test target... Link Binary with Libraries, I had added MediaKit there (manually) and when I removed it, the tests finally build/run.


Implement CLIENT|ID, CLIENT|INFO, CLIENT|LIST, and CLIENT|KILL.

This implements a subset of CLIENT commands: ID, INFO, LIST, and KILL.  For those commands, any filters are also implemented.
This required a number of ancillary changes:

IClusterProvider now allows checking if a given node id is a replica, not just the current node
IClusterSession now tracks the node id, if any, of the connecting node - this is inferred based on CLUSTER|GOSSIP messages
INetworkSender now tracks the local endpoint as well as the remote endpoint of a connection
INetworkSender now has a TryClose() method that closes the underlying Socket, if any, on first call
RespServerSession now has a unique, monotonic, id
RespServerSession now tracks a reference back to an IGarnetServer, so it can enumerate other active sessions

I aimed for minimal changes to enable these commands.  Some refactoring might make them a bit cleaner (in particular, I don't love the various is Blahs I've added), but would make the change larger.
Some things differ from Redis behavior in a few ways, all of which I think are fine, but are worth pointing out:

CLIENT|LIST documentation gives the impression that combining TYPE and ID filters is allowed, but that does not appear to be true.

This PR matches the actual Redis behavior I have observed, not the documented behavior.

CLIENT|KILL is documented as ANDing filters, but that doesn't appear to be true in my testing.

This PR instead replies with an error if the same filter is specified multiple times.

CLIENT|LIST and CLIENT|INFO in Redis return many more fields (this PR just does id, addr, laddr, age, flags, resp, and (optionally) name and user).

Since these can change release to release, I think this is fine.

In Redis CLIENT|KILL doesn't "take" until the next command is about to be processed.

In this PR, I'm calling Socket.Close() immediately - which causes in flight reads and writes to fail.
This is motivated by my particular use case: which is draining client connections before some administrative action (restarting or upgrading Garnet, modifying cluster configs, rebooting VMs, etc.).  Waiting for connections to "naturally" die isn't ideal, plus I don't particularly want to put an isKillRequested-check in the hot path.

+1 very nicely done!


++NEW SEX@VIDEOS] Tummy Tear Dark Video Female Lea𝚔ed Video Tummy Tear Dark Video Viral Link On Social Media Twitter-X

20 seconds ago
L𝚎aked Video  Tummy Tear Dark Video Female Original Video Viral Video L𝚎aked on X Twitter Telegram
𝖶𝖺𝗍𝖼𝗁 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
Tummy Tear Dark Video Female Original Video video took the internet by storm and amazed viewers on various social media platforms.  Tummy Tear Dark Video Female, a young and talented digital creator, recently became famous thanks to this interesting video.
  Tummy Tear Dark Video Female, a young and talented digital creator, recently became famous thanks to this interesting video.


[RFC] Add "icmp_code" to whitelist of allowed rule fields.

Also beef up UTs and validation to check whitelist against rules.
This fixes the issue at hand but I'm not sure we can merge without addressing #451 in some way.

Can one of the admins verify this patch?

This is not assigned to a person or a release, which means it'll languish.

Hmm, it was meant to be with me or Rob, I think.  Github was broken last night; losing comments and updates.  (I was also tired so may have messed up.)

@robbrockbank found in manual testing that the ICMP type/code match of 255/255 is interpreted by the kernel as a wildcard so I've added a workaround to this fix.  (It's not safe to merge the fix without the workaround because it would open up a security hole.)
Resolving the kernel issue is tracked under #451.

(Workaround is to log an error and generate an unconditional DROP rule if someone tries to use 255/255.)

LGTM


How to use Azure AD to Azure AD B2C linking with ROPC?

I have a working Azure AD B2C setup that uses the ROPC flow. So I can get a JWT token by doing a POST like this:
POST https://myb2cstaging.b2clogin.com/myb2cstaging.onmicrosoft.com/oauth2/v2.0/token?p=b2c_1_ropc_flow
Content-Type: application/x-www-form-urlencoded

<EMAIL_ADDRESS>password=bla
grant_type=password
scope=https://myb2cstaging.onmicrosoft.com/my-app/user_impersonation+openid+offline_access
client_id=<my-client-id>
response_type=token+id_token

I have then followed the instructions on this page to connect Azure AD with my Azure AD B2C tenant.
If I now try the same POST with a username and password from somebody in the Azure AD directory, I get:
{
  "error": "access_denied",
  "error_description": "AADB2C90225: The username or password provided in the request are invalid.\r\nCorrelation ID: eff530b2-29eb-409a-aaed-6a298fa30e06\r\nTimestamp: 2020-02-27 14:57:56Z\r\n"
}

I am sure the username and password are correct and they are url encoded. So I don't think that is the actual problem.
Is it correct that I use the same endpoint from B2C to get a token? Is there a way to get more info on what might be wrong?
Note that the user I use is also shown in the list of users of the B2C directory with a source "External Azure Active Directory". Don't know if that makes any difference or not.
regards,
Wim

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

ID: bac6db2d-bff9-cbcf-d522-966ed17ea27f
Version Independent ID: 91e3eeb7-8602-9e26-21c3-1fa70d03a354
Content: Set up sign-in for an Azure AD organization - Azure AD B2C
Content Source: articles/active-directory-b2c/identity-provider-azure-ad-single-tenant.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@wimdeblauwe Thanks for your feedback! We will investigate and update as appropriate.

ROPC sign-in fits for local account only, not with a federated identity provider.

@yoelhor Can you explain in more detail? I am coupling Azure AD B2C with Azure AD.


Implement detach_and_claim_interface

Closes #34
Interestingly, the PR needs the fallback-to-racy code, otherwise it's not working on my ubuntu which is definitely not 12 years old (I'm getting "this ioctl is not valid for this device" error). Not sure what's going on there, maybe it's an undiscovered bug in libusb which I used as the source to port this code from. Or maybe I just can't copy code very well, as the number of force pushes here indicates :)

Interestingly, the PR needs the fallback-to-racy code, otherwise it's not working on my ubuntu which is definitely not 12 years old (I'm getting "this ioctl is not valid for this device" error).

It's actually the opposite for me (Ubuntu 22.04, Kernel 6.5.0). The USBDEVFS_DISCONNECT_CLAIM is working, but the USBFS_GETDRIVER and USBFS_IOCTL calls (which also affects reattaching) are failing with ENOTTY ("Inappropriate ioctl for device"). Are you sure that's not what you're seeing?
Those weren't working because the ioctl opcodes were wrong -- the rustix ioctl::ReadOpcode, ioctl::WriteOpcode, ioctl::ReadWriteOpcode  corresponds to the _IOR, _IOW, _IOWR etc in https://github.com/torvalds/linux/blob/master/tools/include/uapi/linux/usbdevice_fs.h. That opcode type is arbitrary and doesn't necessarily correspond to  the ioctl::patterns type used or direction of actual data flow. I pushed some commits including fixing these.
Can you re-test with RUST_LOG=debug and see if you're getting the USBDEVFS_DISCONNECT_CLAIM ioctl failed, falling back to detach-then-claim debug!() I added in the fallback code path?

Dropped the fallback in the interest of not having legacy code that no one is testing, but if there comes a need for it, it'll be in git history.


WIP proposal to split into many adapters

This is a proposal to split this plugin into optional cache adapters.
Why?

It makes the package smaller.
It allows installing only necessary dependencies like ioredis, redis-lru.
More scalable to implement alternative cache servers without increasing package size.

WIP
This is a proposal only, not fully implemented.

Good stuff, I like it ! 🚀
@stafyniaksacha has been migrating a whole bunch of Strapi things to Typescript, when that becomes more available the adapter packages could benefit from a typescript interface as well.

I like the idea too :)
Just a small niptipping, can we rename "strapi-middleware-cache-mem-adapter" to something like "strapi-middleware-cache-in-memory-adapter" to avoid confusing with "memcached" ?

I have some suggestions since we have lightly discussed making this a plugin for v4 (in which case the package would be renamed to strapi-plugin-cache)
Likewise since we (Strapi) use the strapi-provider-* syntax (for v4 they will actually be @strapi/provider-*) I would suggest we follow a similar syntax.
Other than that I totally think this is a fantastic idea. A fellow community member @mattiebelt is working on a search plugin that is planning to use this same structure.
(I might suggest hold off on merging this until we figure out how best to rewrite this for v4 though)

Hello @agutoli !
I have implemented the provider system for the V4 release here: https://github.com/patrixr/strapi-middleware-cache/tree/feature/strapi-v4/packages
Please feel free to give your feedback!


HTML5Backend should not prevent default NativeTypes drag event even w…

(this PR intent to replace this original PR, as the change is made in a different branch now)

We notice an issue after integrating react-dnd into our app: Our existing drag-and-drop file upload function (which is not built on top react-dnd) failed to work.
It seems like the event handling code handleTopDragOver in html5 backend will aggressively intervene the functioning of other DnD lib.
if (canDrop) {
...
} else if (this.isDraggingNativeItem()) {
    // Don't show a nice cursor but still prevent default
    // "drop and blow away the whole document" action.
    e.preventDefault()
    e.dataTransfer.dropEffect = 'none'
} else if (this.checkIfCurrentDragSourceRectChanged()) {
...
}

I think it is ok to remove this part of the code, since it is not part this library's concern to prevent "blow way" scenario.

@darthtrevino Yes such a feature is definitely useful, but I think it should be left to the client of this library to implement on their app layer.


Surelog version ancient

The Surelog version currently referenced in the submodule is over a month old and it seems that the latest UHDM module does not work anymore with the current Surelog (still verifying).
Also Yosys is 10 days old.
This means the current module can't be used from head as users have to juggle various potentially different versions to figure out which combination is working.
We should at max be 2-3 days behind head of any given dependency, with aim for < 1 day.
It should be possible to compile, build and test with all dependencies from head at least once a day.

Seems there is a dependabot PR #670 but CI is red?

CI is read because of compilation issues:
07:23:41 | UhdmAst.cc: In member function 'Yosys::AST::AstNode* Yosys::UhdmAst::process_object(vpiHandle)': 07:23:41 | UhdmAst.cc:3924:16: error: 'uhdmimport' is not a member of 'UHDM'; did you mean 'uhdmport'?

Yep, filed that here https://github.com/chipsalliance/yosys-f4pga-plugins/issues/303

But that changed only a few days ago and wouldn't explain a month old Surelog.
If dependabot was running, maybe it silently did not update things as it started to fail a while ago ?

There is also #612 that seems to be some WIP on updating Surelog so I guess there is more issues to be fixed.

I think somewhere between Surelog 7e0222abdca616cc0b18a471efe7a73d1f6c30fb cd26de17ab2474cc6bc4ff54aad97a495b8f3bc2 things didn't work anymore with the module.

BTW, I've patched in the change yesterday and it seems to work with current head of Surelog.
Did y ou narrow down on the parameter override problem you were seeing ?

Did you narrow down on the parameter override problem you were seeing ?

Reported here: https://github.com/chipsalliance/Surelog/issues/2881. Plugin makes sure there are no errors from Surelog before reading UHDM, so it breaks the tests now.


New JavaScript editors no longer compute output in correctly

When you add a new JavaScript tag and write code in the editor e.g. console.log("hello") no output is rendered when you run it. It seems to have an effect on previous editors if you go back and run one after you get some weird output.
Refresh is still works in new editors though.
I noticed this while testing a new window opened from a link but it is present in initial page too.
@ewelton guessing you might know the fix best.

Console.log last line (from ace editor) out of sync. Works if editing in markdown.
But when you run from the Ace editor, the last console.log

Ok. We fixed this by checking for EOL. @gannan08 noticed that without an extra line in the editor, it failed, but with it, it worked. So we added a check and append.

Nice!
On Mon, Apr 8, 2019 at 5:44 PM Joe Andrieu<EMAIL_ADDRESS>wrote:

Ok. We fixed this by checking for EOL. @gannan08
https://github.com/gannan08 noticed that without an extra line in the
editor, it failed, but with it, it worked. So we added a check and append.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/WebOfTrustInfo/satyrn/issues/29#issuecomment-480909555,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARN_yJfGSOzS9VDejKJs3YYPkVRIX5fEks5ve3HNgaJpZM4cMTvh
.


New strings translation

Translated CameraControl.Control.MotionBlur & CameraControl.Control.ScreenSpaceReflections.
(this comment will be updated if more changes will be made before this pull request's merge/closing)

Thank you for the additions!


Ring doorbell pro motion binary sensor state does not update

Home Assistant release with the issue:

0.101.3
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Docker
Integration:

https://www.home-assistant.io/integrations/ring/
Description of problem:
Motion binary sensor turns on but that state persists (i.e., it won't turn off).
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

binary_sensor:
  - platform: ring

Traceback (if applicable):

Additional information:
Calling homeassistant.update_entity or ring.update services does not seem to correct state (to off). Also, clearing the state does not help as it will go back to on state.

@dooz127 do you still have this issue on 103.5?

No longer appears to be a problem.

This is still happening in HASSIO 0.103.5 The binary sensor detects motion, and updates the status from clear, but it never goes back to clear even after trying a set state. I don't know that this issue should be closed.


Can we get a 64 bit build?

Any CPU seems like it is always building 32 bit.....can we get a dedicated 64 bit version in addition to 32? Thanks!

I believe it would require a separate project as CefSharp requires a specific build target.  I chose 32bit as it will work on 64bit systems whereas the reverse is not true.  Is there any particular reason for the 64bit request?

I built it in 64 bit and if compiled correctly and worked great. I mostly
want it because I'm OCD and want everything that can run in 64 bit to do
so. :)
On Sat, Oct 31, 2015 at 1:06 AM, Samuel Attard<EMAIL_ADDRESS>wrote:

I believe it would require a separate project as CefSharp requires a
specific build target. I chose 32bit as it will work on 64bit systems
whereas the reverse is not true. Is there any particular reason for the
64bit request?
—
Reply to this email directly or view it on GitHub
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/issues/16#issuecomment-152698287
.

I only tried to build in 64 back when this project was using CefSharp 41.0
I guess since then they have changed it a bit and now it does indeed work.  It'll take a bit of work (mostly on the update mechanism) but I'll try upload 64 bit builds from now on


Unable to initalize styleguide

I've installed both locally and globally (Steps 1 and 2) but unable to initialise the styleguide
npm install devbridge-styleguide --save-dev
npm install devbridge-styleguide -g
when I run step 3 styleguide initialize
I am running from inside the project directory
I get the following error message
Missing required arguments: kssSource, styleSource, output
Please provide sources of KSS files
Please provide sources of preprocessed files
Please provide output path

I managed to get it to initialise if I pass in those parameters but then get stuck with the same error as #117. The folder is created but the config.txt is not
What am I doing wrong?

You have another package, that is aliased as styleguide, my guess would be sc5-styleguide?

ah good call, I've pruned npm and that seems to have solved that issue, now I get stuck on #153
Thanks for the pointer


Update message format

In case album type is null (or single) or artist genre is null, formats are like below:
[songName] / , or [artistName] ()

/b hotfix null message format


Typo In README.md Example Code

Typo In README.md Example Code
There is a small error in the example code in the README.md under "Customize" -> "ExpandingMenuButton".
exclusiveOr should be symmetricDifference in
menuButton.enabledExpandingAnimations = AnimationOptions.All.exclusiveOr(.MenuItemRotation)
Thank you for writing this library! 👍

Thanks for your report.
I fixed it! 520ec236a57966dc6f34972c7032863024bcb810


config/universal: create default namespace by default

Rebased on top of main which now has the fix for the bootstrapping flake.


Version bump for main

Version bump for main branch, plus removing unneeded check for python 3.8
Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[x] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Please review


precacheImage should return bool instead of void

https://docs.flutter.io/flutter/widgets/precacheImage.html should return a bool instead of void indicating if loading the image succeeded.
Currently I use
bool isSuccess = true;
await precacheImage(..., onError: () => isSuccess = false);
if(isSuccess) {
  ...
}

which is ugly and should instead allow
if(await precacheImage(...) {
  ...
}

[✓] Flutter (Channel master, v1.1.10-pre.205, on Mac OS X 10.14.3 18D42, locale en-AT)
    • Flutter version 1.1.10-pre.205 at /Users/zoechi/flutter/flutter
    • Framework revision e38efc890b (4 hours ago), 2019-01-28 19:55:01 -0800
    • Engine revision 9b6d5031a3
    • Dart version 2.1.1 (build 2.1.1-dev.3.2 a5030ed92f)

See also https://github.com/flutter/flutter/issues/16592#issuecomment-427180769

/cc @dnfield in case you have thoughts on this.

After #51485 (and the related attempts to land it previously), precacheImage never fails, but you must add a listener if it doesn't end up strongly in the cache.
It would probably be nicer if we returned a bool in such a case.
@zoechi your current work around actually doesn't account for cases where there's no error, but the image is too big to fit in the cache.
I think it's probably still worth doing this.  Am I right in thinking that this would not be a breaking change, because no one could be depending on the return type being void today?

The problem with bool as well is that it could end up being ambiguous as to whether it failed to cache or whether there was some error in acquiring the encoded image or decoding the bytes.

So one possibility would be to return an object that can tell you whether it succeeded, missed the cache, or had an error.

I think this interface is strange, it's time to change.
The following are my improvements, which are more radical:
@dnfield
void precacheImage(
    ImageProvider provider,
    BuildContext context, {
    Size size,
    ImageStreamListener listener,
  }) {
    final ImageConfiguration config =
        createLocalImageConfiguration(context, size: size);
    final ImageStream stream = provider.resolve(config);
    ImageStreamListener _listener;
    _listener = ImageStreamListener((ImageInfo image, bool sync) {
      if (listener != null && listener.onImage != null) {
        listener.onImage(image, sync);
      }
      // Give callers until at least the end of the frame to subscribe to the
      // image stream.
      // See ImageCache._liveImages
      SchedulerBinding.instance.addPostFrameCallback((Duration timeStamp) {
        stream.removeListener(_listener);
      });
    }, onError: (dynamic exception, StackTrace stackTrace) {
      stream.removeListener(_listener);
      if (listener != null && listener.onError != null) {
        listener.onError(exception, stackTrace);
      } else {
        FlutterError.reportError(FlutterErrorDetails(
          context: ErrorDescription('image failed to precache'),
          library: 'image resource service',
          exception: exception,
          stack: stackTrace,
          silent: true,
        ));
      }
    }, onChunk: (ImageChunkEvent event) {
      if (listener != null && listener.onChunk != null) {
        listener.onChunk(event);
      }
    });
    stream.addListener(_listener);
  }


Unit Test Vital Code

Need to add unit tests

[ ] PGP functionality
[ ] Web Components

This is not a goal for 100% code coverage, just to get to a starting point.

Looks like we're going to have to implement our unit tests along the lines of how this is done https://github.com/openpgpjs/openpgpjs


Feature development

What's in the PR

...Added a plugin to skip checkstyle violations and modified the instructions to ease manual deployment

How to test manually

... Try the new instructions on a clean VM

Okay I did the rebase, could you check that I haven't broken anything?

you must resolve the pom.xml conflict


useFakeTimers is inaccurate with setTimeout and setInterval at same time

🐛 Bug Report
I set an interval for every 10,000MS. I clear the interval at 60,000 MS. This implies after 10 seconds, I start running a method every 10 seconds, until the 60 second mark. This implies I would run the method 5 times. The tests think I run it 6 times.
To Reproduce
Steps to reproduce the behavior:

Open https://repl.it/repls/ThoroughThirdDeals
Run node index.js to see it log 5 times
Run jest to see the tests log 6 times, and the assertion to agree

Expected behavior
It should fail unless I expected 5 calls.
Link to repl or repo (highly encouraged)
https://repl.it/repls/ThoroughThirdDeals
envinfo
Reproduced on repl.it, but local machine is this
  System:
    OS: macOS 10.15.4
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
  Binaries:
    Node: 12.15.0 - ~/.nvm/versions/node/v12.15.0/bin/node
    npm: 6.13.4 - ~/.nvm/versions/node/v12.15.0/bin/npm
  npmPackages:
    jest: ^25.2.4 => 25.2.4

I have a suspect of prioritizing of setTimeout and setInterval, when they hit on the same time. which one should be executed first.

in real timer (of nodejs), it setTimeout then setInterval << thats why its executed 5 times
in real timer (of browser), strangely setInternal then setTimeout, 6 times too
in this code, setInternal then setTimeout << thats why its executed 6 times,

under the hood it use sinon fake times. if i move the setInterval, to assigned after setTimeout, Then, its executed in order of code, 5 times
      let interval;
      setTimeout(() => {
        clearInterval(interval);
        resolve();
      }, 6000);
      interval = setInterval(obj.foo, 1000)

@ryanhs Thanks. I finally was able to come back to this and confirm your solution for Jest. This code isn't running in a browser at any point so no worries about discrepancies between nodejs and chrome. Not sure if this should stay open for that case?


pagination enhancement

adds a previous and next button.  Previous button does not appear on first page.  Next button does not appear on last page.
sets a limit to how many pages are shown in the paginator, with a pageButtonLimit prop.  Defaults to 10 if that prop isn't set.
current page number appear as close to the middle of the set as possible, depending on how many pages are to the left and to the right.  See Google's pattern for an example.
adds test

Can you make the numbers here configurable? Like a pageButtonLimit prop?

Also please add tests and squash everything into a couple related commits

yes, will do...

@glittershark  all set if you want to take another look.  Great component, by the way!

Hey guys, can someone bump the version in npm? (not using bower, soz)
Thanks!

hi, how can I use the pagination? I want to change the style to bootstrap pagination. But the doc does not show how to customize the pagination

Yeah, this looks good. Thanks @tsargent!

@thientran1707 see #4

@cumbreras just pushed 0.11.0

@glittershark thanks for keeping the awesome flowing!

@glittershark the Paginator is reactable Paginator?

I'm not sure what you're asking

    ...

I found this in #4. In the doc, it's <Table .../>. I do not know how to style the paginator

I would like to style the paginator with Bootstrap. #4  does not mention how to do it? And in the doc, it also does not show how to change the styling. Somehow I can change the style of paginator?

#4 is an issue to add the ability to customize the class of the paginator. FWIW you can do it right now by just using a CSS preprocessor or copy-paste to style the classes we use using Bootstrap's styles

oh I see. Thanks. I think that you will attach css file

oh... I see. Thanks

@glittershark when I change the version in package.json to 0.11.0, it tells my that "Error: Cannot find module './stringable' " . I tried rm -rf node_modules and reinstalled but it does not solve the problem.

Sounds like a dupe of #167


Close modem object [CWE-400,CWE-664]

AWS CodeGuru complained that modem at https://github.com/michaelsanford/DTMF-Butler/blob/main/butler.py#L39 is never closed, which could cause a resource leak.

Code fix remediation
Problem This line of code might contain a resource leak. Resource leaks can cause your system to slow down or crash.
Fix
Consider closing the following resource: modem. The resource is allocated by call serial.Serial. Execution paths that do not contain closure statements were detected. Close modem in a try-finally block to prevent the resource leak.

Given the context this runs in, I think it's not really relevant, but it won't hurt to add it to make CodeGuru happy, so why not...
https://owasp.org/www-community/vulnerabilities/Unreleased_Resource

The fix didn't quash the error, which I understand, but I will put no further effort into resolving.


Add 'ceph-salt stop' command

The ceph-salt stop command will check OSD noout flag, and ask user confirmation:

Then will stop ceph services:

At this point, if needed, it's now safe to manually shutdown hosts.
Fixes: https://github.com/ceph/ceph-salt/issues/359
[1] https://susedoc.github.io/doc-ses/master/html/ses-admin/cha-ceph-operating.html#ceph-cluster-shutdown

noout flag can be set by ceph-salt automatically
ceph-salt stop should check if cluster is healthy

I've now updated this PR to automatically set noout OSD flag.


Add support for GovCloud.

AWS GovCloud has a separate SAML endpoint from the other commercial regions, so this PR updates the code to use that endpoint if one of the GovCloud regions is specified in the region profile property.
Also needed to pass one of the GovCloud regions in when assuming a role, otherwise the Node SDK could not find the Azure IdP in AWS.

Thanks for your contribution! Released to 1.9.0