For #784: Create Base interface for Request and Response.

For #784: Create Base interface for Request and Response.

Created interfaces Head and Body
Wired them to Request and Response
Corrected problems.

Job #845 is now in scope, role is REV

This pull request #845 is assigned to @g4s8/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @yegor256/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be a monetary reward for this job

@g4s8 ping

@paulodamaso I think we also need to notice in readme that version 2.0 is not backward compatible with previous vesions.

@g4s8 I don't know where this kind of info would fit in readme; where do you suggest I'd put this?

@paulodamaso I'd put it to the bottom of page before "How to contribute" but after "Optional dependencies"

Codecov Report

Merging #845 into master will decrease coverage by 0.13%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##             master     #845      +/-   ##
============================================
- Coverage     77.17%   77.03%   -0.14%
+ Complexity      997      984      -13
============================================
  Files           224      220       -4
  Lines          4731     4708      -23
  Branches        380      380
============================================
- Hits           3651     3627      -24
- Misses          918      919       +1
  Partials        162      162

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/org/takes/facets/hamcrest/HmHeader.java
78.37% <80%> (ø)
10 <5> (?)

...rc/main/java/org/takes/facets/hamcrest/HmBody.java
76.47% <90.9%> (ø)
12 <4> (?)

src/main/java/org/takes/rs/RsSimple.java
50% <0%> (-30%)
1% <0%> (-1%)

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 341fb1b...b58d189. Read the comment docs.

@g4s8 DOne, please take a look.

@rultor merge

@rultor merge

@g4s8 Thanks for your request. @yegor256 Please confirm this.

@yegor256 please take a look at this

@yegor256 ping, time is running out on this one

@yegor256 ping

@yegor256 please take a look

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@paulodamaso @yegor256 Oops, I failed. You can see the full log here (spent 6min)
++ jq -r .build.status
+ status=queued
+ '[' queued == null ']'
+ '[' queued == success ']'
+ '[' queued == failed ']'
+ echo 'waiting for AppVeyor build 3051: queued'
waiting for AppVeyor build 3051: queued
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/3051
++ jq -r .build.status
+ status=queued
+ '[' queued == null ']'
+ '[' queued == success ']'
+ '[' queued == failed ']'
+ echo 'waiting for AppVeyor build 3051: queued'
waiting for AppVeyor build 3051: queued
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/3051
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 3051: running'
waiting for AppVeyor build 3051: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/3051
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 3051: running'
waiting for AppVeyor build 3051: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/3051
++ jq -r .build.status
+ status=running
+ '[' running == null ']'
+ '[' running == success ']'
+ '[' running == failed ']'
+ echo 'waiting for AppVeyor build 3051: running'
waiting for AppVeyor build 3051: running
+ sleep 5s
+ true
++ curl -K ../curl-appveyor.cfg https://ci.appveyor.com/api/projects/yegor256/takes/build/3051
++ jq -r .build.status
+ status=failed
+ '[' failed == null ']'
+ '[' failed == success ']'
+ '[' failed == failed ']'
+ echo 'see https://ci.appveyor.com/project/yegor256/takes/build/3051'
see https://ci.appveyor.com/project/yegor256/takes/build/3051
+ exit 1
container 3a57baf7a670f1ac6ba2ac19e9fdc3d26617cd1625a78800a20d68925cca0e06 is dead
Tue Sep 18 16:00:28 CEST 2018

The architect of the project has changed; @yegor256/z is not at this role anymore; @paulodamaso/z is the architect now

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 22min)

@elenavolokhova/z please review this job completed by @g4s8/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

Code review was too long (36 days), architects (@paulodamaso) were penalized, see §55

The job #845 is now out of scope

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @paulodamaso/z

@g4s8 According to our Policy:

The code reviewer found at least three problems in the code

Please, confirm that you will try to find at least three issues next time.

@elenavolokhova sure, I confirm

@0crat quality acceptable

Order was finished, quality is "acceptable": +15 point(s) just awarded to @g4s8/z

Quality review completed: +8 point(s) just awarded to @elenavolokhova/z

@0crat  status

@0crat status (here)

@g4s8 This is what I know about this job in C3FFK3YAY, as in §32:

The job #845 is not in scope
Job footprint (restricted area)


Tidy setup.py

Specify min Python version (3.6 as prior versions are EOL)
Tidy description loading

@FFY00 or adopting Poetry to ease dependency/packaging

It does not support editable installs yet.

@FFY00 I get the need for such thing when developing a modular system but would that be a blocker for something like this that is more of a utility package?

This has been done in #87


pull error

Expected behavior
pull the image
Actual behavior
(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/: dial tcp: lookup registry-1.docker.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:49953-><IP_ADDRESS>:53: i/o timeout"}
Information about the Issue
Steps to reproduce the behavior
docker pull hello-world

The same problem with Kitematic app, but docker pull <any_container> in Terminal works well.

If docker pull doesn't work, kitematic will not work as well.
Are you behind a firewall or VPN?
See our wiki for possible solutions:
https://github.com/docker/kitematic/wiki/Common-Proxy-Issues-&-Fixes


[Bug] delete with value column predicates should not work

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
master
What's Wrong?

create a aggregate table with key and value
delete from the table with where value = xxx
the delete should not work.

What You Expected?

the delete should not work.

How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

i want to fix it.

looks like it's been fixed.
CREATE TABLE IF NOT EXISTS t1 (
    id LARGEINT NOT NULL COMMENT "id of site",
    pv BIGINT SUM DEFAULT "0" COMMENT "total page views"
)
AGGREGATE KEY(id)
DISTRIBUTED BY HASH(id) BUCKETS 3
PROPERTIES ("replication_num" = "1");

insert into t1 values(1, 1);
insert into t1 values(1, 1);
insert into t1 values(2, 1);

select * from t1;
+------+------+
| id   | pv   |
+------+------+
| 1    |    2 |
| 2    |    1 |
+------+------+
2 rows in set (0.02 sec)

delete from t1 where pv = 1;
ERROR 1105 (HY000): errCode = 2, detailMessage = delete predicate on value column only supports Unique table with merge-on-write enabled and Duplicate table, but Table[t1] is an Aggregate table.


Cache already generated picklers for known types

When we ask for a Pickler of some common type like List[String], scala pickling generates a new pickler even if there's already one stored in the mapPickler of the DefaultPicklerRegistry which matches the mentioned type.
Eventually, we'd like to generate Picklers for only the types that we need to. The same problem applies to Unpickler and probably other common types (I haven't made sure of this last statement, though).

I've found something very interesting:
implicitly[Pickler[List[String]]] // creates both Pickler and Unpickler, look at Iterable source code
implicitly[Unpickler[List[String]]] // creates the Unpickler even though we have already generated one

The second statement should not trigger the generation of the Unpickler since it has already been created before.


Added GraphQL Voyager and ReDoc to "built with typescript" section

Merged. Thank yo @RomanGotsiy!


haven-agent.py fatal error

I am use ubuntu server 16.04
start haven-agent.py
console log:
2017-03-02 00:38:06,459 - INFO - Configs: /etc/haven-agent.ini
Arguments: {'timeout': 10, 'log_level': 2, 'docker': '$SELF_IP:2375', 'secret': None, 'master': '$MASTER_IP:8761'}
2017-03-02 00:38:06,460 - WARNING - Run as daemon
2017-03-02 00:38:06,493 - ERROR - Cannot connect to Docker {'data': None, 'method': 'GET', 'docker': '$SELF_IP:2375', 'path': '/info'} due to error: [Errno -2] Name or service not known
2017-03-02 00:38:06,494 - ERROR - Cannot update registration due to [Errno -2] Name or service not known
Traceback (most recent call last):
File "/usr/bin/haven-agent.py", line 167, in update
info = docker.info()
File "/usr/bin/haven-agent.py", line 83, in info
return copy.deepcopy(self.__info.get())
File "/usr/bin/haven-agent.py", line 38, in get
self.__data = self.__loader(*args, **kwargs)
File "/usr/bin/haven-agent.py", line 50, in
self.__info = CachedValue(loader=lambda: self.__send("/info"),
File "/usr/bin/haven-agent.py", line 64, in __send
self.conn.request(method, path)
File "/usr/lib/python3.5/http/client.py", line 1106, in request
self._send_request(method, url, body, headers)
File "/usr/lib/python3.5/http/client.py", line 1151, in _send_request
self.endheaders(body)
File "/usr/lib/python3.5/http/client.py", line 1102, in endheaders
self._send_output(message_body)
File "/usr/lib/python3.5/http/client.py", line 934, in _send_output
self.send(msg)
File "/usr/lib/python3.5/http/client.py", line 877, in send
self.connect()
File "/usr/lib/python3.5/http/client.py", line 849, in connect
(self.host,self.port), self.timeout, self.source_address)
File "/usr/lib/python3.5/socket.py", line 693, in create_connection
for res in getaddrinfo(host, port, 0, SOCK_STREAM):
File "/usr/lib/python3.5/socket.py", line 732, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name or service not known
^CTraceback (most recent call last):
File "/usr/bin/haven-agent.py", line 449, in
main()
File "/usr/bin/haven-agent.py", line 445, in main
time.sleep(bs.timeout)
help me pls

I think this is broblem:
clusret-store not work in docker swarm mode

Quote from our doc:

Step 1: Define the common variables for configuration needed in the scripts and configuration files. Feel free to set them as environment variables or replace them in the script:
IP of master instance
MASTER_IP=<IP_ADDRESS>
IP of current configured node. This IP must be accessible from master instance
SELF_IP=<IP_ADDRESS>

And you have:

Arguments: {'timeout': 10, 'log_level': 2, 'docker': '$SELF_IP:2375', 'secret': None, 'master': '$MASTER_IP:8761'}

It is mean that you do not change both placeholders to concrete ip.

now:
docker info
Containers: 5  Running: 5  Paused: 0  Stopped: 0 Images: 8 Server Version: 1.13.1 Storage Driver: aufs  Root Dir: /var/lib/docker/aufs  Backing Filesystem: extfs  Dirs: 45  Dirperm1 Supported: true Logging Driver: json-file Cgroup Driver: cgroupfs Plugins:  Volume: local  Network: bridge host macvlan null overlay Swarm: inactive Runtimes: runc Default Runtime: runc Init Binary: docker-init containerd version: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1 runc version: 9df8b306d01f59d3a8029be411de015b7304dd8f init version: 949e6fa Security Options:  apparmor  seccomp   Profile: default Kernel Version: 4.4.0-64-generic Operating System: Ubuntu 16.04.2 LTS OSType: linux Architecture: x86_64 CPUs: 1 Total Memory: 1.953 GiB Name: box-3261 ID: Q534:IXSO:BHOA:S522:FI2U:MGBE:YQH6:SB55:BN5U:2WBC:ITHN:AY3Y Docker Root Dir: /var/lib/docker Debug Mode (client): false Debug Mode (server): false Registry: https://index.docker.io/v1/ WARNING: No swap limit support Experimental: false Cluster Store: etcd://<IP_ADDRESS>:2379/dn Cluster Advertise: <IP_ADDRESS>:2375 Insecure Registries:  <IP_ADDRESS>/8 Live Restore Enabled: false
haven-agent.py
2017-03-02 19:25:31,091 - INFO - Configs: /etc/haven-agent.ini Arguments: {'log_level': 2, 'timeout': 10, 'secret': None, 'docker': '<IP_ADDRESS>:2375', 'master': '<IP_ADDRESS>:8761'} 2017-03-02 19:25:31,091 - WARNING - Run as daemon 2017-03-02 19:25:31,096 - ERROR - Cannot connect to Docker {'method': 'GET', 'path': '/info', 'data': None, 'docker': '<IP_ADDRESS>:2375'} due to error: [Errno 111] Connection refused 2017-03-02 19:25:31,096 - ERROR - Cannot update registration due to [Errno 111] Connection refused Traceback (most recent call last):   File "./haven-agent.py", line 167, in update     info = docker.info()   File "./haven-agent.py", line 83, in info     return copy.deepcopy(self.__info.get())   File "./haven-agent.py", line 38, in get     self.__data = self.__loader(*args, **kwargs)   File "./haven-agent.py", line 50, in <lambda>     self.__info = CachedValue(loader=lambda: self.__send("/info"),   File "./haven-agent.py", line 64, in __send     self.conn.request(method, path)   File "/usr/lib/python3.5/http/client.py", line 1106, in request     self._send_request(method, url, body, headers)   File "/usr/lib/python3.5/http/client.py", line 1151, in _send_request     self.endheaders(body)   File "/usr/lib/python3.5/http/client.py", line 1102, in endheaders     self._send_output(message_body)   File "/usr/lib/python3.5/http/client.py", line 934, in _send_output     self.send(msg)   File "/usr/lib/python3.5/http/client.py", line 877, in send     self.connect()   File "/usr/lib/python3.5/http/client.py", line 849, in connect     (self.host,self.port), self.timeout, self.source_address)   File "/usr/lib/python3.5/socket.py", line 711, in create_connection     raise err   File "/usr/lib/python3.5/socket.py", line 702, in create_connection     sock.connect(sa) ConnectionRefusedError: [Errno 111] Connection refused ^CTraceback (most recent call last):   File "./haven-agent.py", line 449, in <module>     main()   File "./haven-agent.py", line 445, in main     time.sleep(bs.timeout) KeyboardInterrupt

fixed.


IOS : Unable to parse string as JSON - check that your JSON is valid.

Hi , I'm using :   flutter_pusher: ^1.0.2
here is my flutter doctor :
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, 2.2.0-10.2.pre, on macOS 11.2.3 20D91 darwin-x64, locale en-EG)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[✓] VS Code (version 1.55.2)
[✓] Connected device (2 available)
• No issues found!
when bind to new event on pusher console i got in xcode :
2021-05-09 21:16:44.921819+0200 Runner[51749:732568] Metal API Validation Enabled
2021-05-09 21:16:45.079714+0200 Runner[51749:732999] flutter: Observatory listening on http://<IP_ADDRESS>:58979/NtaMnrrTlHQ=/
Pusher init
Pusher connect
2021-05-09 21:19:55.961112+0200 Runner[51749:738889] [] nw_protocol_get_quic_image_block_invoke dlopen libquic failed
Pusher subscribe
Pusher bind (Conversation)
Unable to parse string as JSON - check that your JSON is valid.
Unable to parse string as JSON - check that your JSON is valid.

flutter_pusher: ^1.0.2 is not this repository's package. To use the latest version of this package use pusher_client: ^1.1.3


Cannot connect via noray

I am having trouble getting the example project to work. It works fine as long as I am testing with multiple instances on the same machine. For networking beyond my local network (using noray) I cannot connect to the host. Tested this with 2 friends behind different routers as well as using a mobile hotspot.
The handshake fails for the client due to a Timeout.
Server reports Handshake as concluded (as expected when using PacketHandshake.over_enet)
Am I doing something wrong or is this a problem with noray?
Our approach was:

connect to noray on host and client
copy the hosts noray openID and send it to the clients
press "host" on the host
paste the openID of the host in the clients LineEdits (Host OID)
press join
Timeout for clients on handshake

forgot to mention we were using your noray instance on tomfol.io:8890
it seems that the connection to noray works fine because we are getting the openIDs

Yes I tried both force relay on and off (for host and client) but It seems that it defaults to relay anyways if the nat punchthrough fails.

I'll try to recreate it on my end and see if I can find something!

Hi @marty1912, could you tell me more about the setups you've tested with? Device type, OS, router type, etc. Also any logs you may have from the application run.
Unfortunately I wasn't able to reproduce the issue.

Hi @elementbound  and thank you for the help.
I am using my desktop PC (Win 11) with a router (TP-Link Archer C7 v5 running openwrt 22.03)
and a laptop PC (Win 10) that is connected via iPhone hotspot.
I am using the build of forest-brawlv1.1.1win64
logs for both host and client are attached (using force relay)
norayLogs.txt

Hi @marty1912, I've checked the noray instance and I can see some people joining on the 13th from 7am to 5pm UTC+1 and some limited traffic going through the relay. I can also see from the logs that the registration was indeed successful.
So based on this, traffic reaches noray and then it gets forwarded.
I'm not familiar with OpenWRT, is there some chance it's blocking incoming UDP traffic?

Hi @marty1912, is there any update on this?


How to refresh main entry file hash name when only modify scope css ?

Describe the bug

Step 1: modify Home.scope.css ,
Step 2: npm run build
Question:
dist/index.[hash].js hash not change，but file content is different
How to refresh main entry file hash name when only modify scope css ?
Reproduction
https://stackblitz.com/edit/vitejs-vite-velnu6
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @vitejs/plugin-react: ^2.0.0 => 2.0.0
    vite: ^3.0.4 => 3.0.4

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Duplicate of #9583.


Add CP outgoing IPs to allowed in Ingress

Required for health-kick to work

I was just about to paste this into Slack:

Should we make the long allow-list string -> array conversion a must-have for the next PR we see on this code?

Could we refactor this long single element into an array? It's certainly becoming a bit unwieldy :)

@spikeheap happy for any pointers, the docs say it's a comma separated list but my google skills maybe limited 😄

https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#whitelist-source-range

Just realised we can do this when passing the values into the ingress 😄

:+1: The whereabouts-api has a good example:

https://github.com/ministryofjustice/whereabouts-api/blob/main/helm_deploy/whereabouts-api/templates/ingress.yaml#L15
https://github.com/ministryofjustice/whereabouts-api/blob/main/helm_deploy/values-prod.yaml#L28

Ooo I like that!


CommonPart(Version)Record query performance problems

There are no indexes on CommonPart(Version)Record columns, which can cause performance issues and query timeout with a couple thousand content items in common scenarios, such as simply displaying the content list at "/Admin/Contents/List". This listing can order items by Created/Modified/Published date, so at least those columns should have an index.

Good point

Fixed in #8210


Custom sensitive information types limitations

Given the product rollout cadence between Commercial and GCC/GCCH/DoD national clouds, it may be worth mentioning a few additional AIP unified labeling "gotchas" -
1.) As of writing, it is currently not possible to create custom sensitive information types in the Microsoft 365 Security & Compliance Center using the GUI referenced here, this is only available via the PowerShell methods referenced here.
2.) As of writing, there are substantially more Microsoft published SITs in Commercial tenants (~152) than GCCH tenants (~100).

Hi @milesgratz - thank you for opening this issue, and apologies for the late response - this was only brought to my attention today.
There've been some updates since Sept 2020, and we've made changes to the AIP service description article as needed.
In any case, we'll take a fresh look at the page with your feedback in mind to ensure that the content's correct.
I'm closing this for now, but feel free to add additional comments if you have any.
#please-close


Update README.md

Include the staging folder in the paths.

I'm registered now. Thanks.
On Tue, Jul 24, 2018, 8:30 PM k8s-ci-robot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. Before we can look at your pull request,
you'll need to sign a Contributor License Agreement (CLA).
📝 Please follow instructions at
https://git.k8s.io/community/CLA.md#the-contributor-license-agreement
https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to
sign the CLA.
It may take a couple minutes for the CLA signature to be fully registered;
after that, please reply here with a new comment and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data and verify that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please sign in with your
organization's credentials at
https://identity.linuxfoundation.org/projects/cncf to be authorized.
If you have done the above and are still having issues with the CLA
being reported as unsigned, please email the CNCF helpdesk:
<EMAIL_ADDRESS>
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/examples/pull/253#issuecomment-407622296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACpDwNer-Ysuj4MI4ktvtd1ykdUCaTEIks5uJ-ZOgaJpZM4VfYFB
.

/ok-to-test


LSP error: Use of undeclared identifier

If I write a character like ² é à è £ ù etc.. lsp keep them as an "undeclared identifier" even if I remove the character.
Even if I save the file, the error still here, so I have to Core:Restart, then error disappears.

Looks like an issue with UTF-8 multi-bytes chars. Can confirm it happens with multiple LSPs.
Instead of Core:Restart, Core:Reload might be a bit faster as a temporary workaround.

A Doc: Reload would be enough, I gotta fix this annoying thing but I haven't paid attention on what is causing it

A Doc: Reload would be enough

Ah yeah, meant that one...
The issue is likely that add_change should receive characters instead of bytes.
https://github.com/lite-xl/lite-xl-lsp/blob/061676efa0344c48aba18ea43688da1eaf444afb/init.lua#L2161-L2179

@Kabcorp Could you test #66? I should have fixed it also for UTF-16 characters that require 4 bytes.

What LSP server are you using?

This is with clangd and #66 applied:
Video del 2023-07-09 18-19-59.webm

I use the LSP from 14 may 2023
I updated the 3 files you pulled in #66


add knoopx's completionlist to try it out

This is my attempt to add @knoopx's completionlist, by itself, to nana. The theme colors need work. No idea if i did it right but it seems to work. 🤣

I have no idea why the title bar isn't themed, but it didn't work on this windows10 box before this change either.

looks good but I would suggest to use https://floating-ui.com instead. it is kind a must for rich interfaces and relying on hard-coded values is not good.

looks good but I would suggest to use https://floating-ui.com instead. it is kind a must for rich interfaces and relying on hard-coded values is not good.

if i knew how to do that, maybe i would. :)

@fdncred hahah fair enough, if you dont feel confident just merge it and ill send a PR whenever I find some time (im in the middle of setting a custom-built keyboard).
if you want to try the examples at https://floating-ui.com/docs/react-dom should get you started (you just need to pass useFloating reference to CompletionList ref property and it will automatically calculate overflow based on the actual size). that library also offers some other useful hooks for this specific feature https://floating-ui.com/docs/react-dom-interactions

I have no idea why the title bar isn't themed, but it didn't work on this windows10 box before this change either.

@fdncred Yeah, sadly the title bar colour attributes are Windows 11 only. I think it might be possible to do on older Windows versions with undocumented APIs but there be dragons.


Old template

Old template - user.html
It seems like working but actually it doesn't 😞

This bug just become smarter! 🤪


fix(admin): contribution page

🍰 Pullrequest

Issues

relates #2787

Todo

[X] None

@Mogge I did change from state to status but I don't like the change cause for me it is a german description.


Motifs missing when input sequence less than 400 bp

If the user doesn't set the -w option, Modisco-lite may produce different results compared to standard Modisco when the input sequence is less than 400 bp. It would be good for the program to automatically account for this scenario, either by raising an error or by using the provided sequence length.

v2.0.7 will raise an error if the window is longer than the sequence -- or, more specifically, if the extracted subsequences are not of the same size as the window.


Include record marker in the pack file

This is an proof-of-concept PR to show how record markers can be inserted into the pack file, which then allows reconstructing the index from the pack files along.
Not all tests are working because some involve check pack file lengths against hard-coded values.
Please feel free to give comments.
See also #124

Closing this PR due to robustness concerns, see #124 for more details.


groovy.lang.MissingPropertyException

the flashHelper variable does not seem to be injected into Controllers all the time. I especially see this when running through an IDE and the runtime refreshes, but have also seen it when running a simple run-app (outside of IDE). Here is a full stacktrace (note that this occurs with or without the linkedin profiler filter). I am running under grails 2.4.4
No such property: flashHelper for class: com.mycompany.gram.web.GraphicsController. Stacktrace follows:
Message: No such property: flashHelper for class: com.mycompany.gram.web.GraphicsController
Line | Method
->>  289 | update           in com.mycompany.gram.web.GraphicsController

|    198 | doFilter         in grails.plugin.cache.web.filter.PageFragmentCachingFilter
|     63 | doFilter . . . . in grails.plugin.cache.web.filter.AbstractFilter
|     82 | doFilterInternal in com.linkedin.grails.profiler.ProfilerFilter
|   1142 | runWorker . . .  in java.util.concurrent.ThreadPoolExecutor
|    617 | run              in java.util.concurrent.ThreadPoolExecutor$Worker
^    745 | run . . . . . .  in java.lang.Thread

Can you provide some way for me to reproduce this issue?

actually, I think it does have something to do with the profiler.
Create a new project and add the two plugins:
compile ":flash-helper:0.9.9"
compile ":profiler:0.5"
Create a controller:
def index() {
flashHelper.info "test message"
}
Modify view:
<g:if test="${flash.info}">

Info:
<flashMsg:msg key="info" sep=""/>

</g:if>
Run app to that view and you'll see the error I posted above.  I don't know much about writing grails plugins so not sure if a fix would be in this project or profiler.

If you remove the profiler plugin, does the problem still occur?

no. problem does not exist when we remove the profiler plugin.

On that basis, it seems more likely that the problem is with the profiler plugin, so I doubt I'll spend any time investigating this, given that there probably aren't too many apps using both this plugin and the profiler.

I can understand that. I thought maybe it had to do with how the plugin was injected into the controllers. Again, don't know much about plugin development so was hoping for an easy (known) fix. Thanks.

I think the way that the flashHelper property is injected into controllers is pretty standard, so there's no obvious/easy fix that springs to mind. If you want to investigate yourself, the relevant code is the addFlashHelperMethod in this file


Update minimum required go version to 1.22

Go 1.23 just released: https://go.dev/doc/go1.23

Updated the main branch protection rule to check for Go 1.22.


Custom Roles

This PR is an update to #1113, featuring more precise commit messages, and implementing some of the changes requested by the rest of the administrative team, such as an explanatory documentation file.
Hello,
After our short discussion with the community on issue https://github.com/PowerDNS-Admin/PowerDNS-Admin/issues/1099, we have decided to publish the final result. This PR introduces all the features that are referenced on the last comment of the above issue, including custom non-administrative Roles, and finer control over the existing User and Operator roles.
The upgrade process does not affect existing users or existing Role behaviour.
All users keep their existing Roles, and existing Role behaviour is initially unchanged. Global settings "Allow users to x..." are deprecated, and their new equivalents are initially turned off for the default "User" role.
More information can be found on this wiki page.
Don't forget to execute flask db upgrade in order to migrate your database!
Developed by our team, in collaboration with @kkmanos.

What's the status on the pr?

What's the status on the pr?

Hello @dg410,
this PR is in the bugtesting phase. Since it is such a major issue, which introduces breaking changes, we want to make sure that everything is working before merging it in a release. Therefore, any use feedback and/or testing is highly appreciated.

Hi,I'm also interested by this feature


Update the Shards API Documentation for UI

[ ] Add in missing details for the UI related shards
[ ] Add in a guide for creating UI with shards

Hey team! Please add your planning poker estimate with Zenhub @Kryptos-FR @ChelseaChX


[question] Can I add claims without id?

For example, I want to add payload claims with this JSON format.
{'test':'hello', 'test2':123}
I looked jwt.h, and I found 'set_payload_claim' function.
But this function needs 'id' parameter, and I don't want to contain id.
Can I add this payload_claim without adding id?
Thank you.

Hmm 🤔 I am not sure I understood your question so let me know if I missed the mark.
So JWT are JSON object those are key value pairs. If youd like to add a claim it needs to have a key (the API id might be unclear, let us know).
So if you use
https://github.com/Thalhammer/jwt-cpp/blob/9d1a010fea3ed3293112edec9745dc435a85a849/include/jwt-cpp/jwt.h#L2702
Like so
https://github.com/Thalhammer/jwt-cpp/blob/9d1a010fea3ed3293112edec9745dc435a85a849/example/private-claims.cpp#L30
You are able to add any JSON value for the id you want to access it with.
And for completeness, here's how youd access the value
https://github.com/Thalhammer/jwt-cpp/blob/9d1a010fea3ed3293112edec9745dc435a85a849/example/traits/kazuho-picojson.cpp#L36
By passing the id

What I want to do is to set payload claim without id.
According to the link,
https://github.com/Thalhammer/jwt-cpp/blob/9d1a010fea3ed3293112edec9745dc435a85a849/example/private-claims.cpp#L30
payload_claims variable is set to this.
{'object':{'test':'hello', 'test2':123}}
But I want to set payload_claims like this.
{'test':'hello', 'test2':123}
More specifically, I looked jwt.h, and found this code.
https://github.com/Thalhammer/jwt-cpp/blob/9d1a010fea3ed3293112edec9745dc435a85a849/include/jwt-cpp/jwt.h#L2702-L2705
What I want to do is to set payload_claims itself, not payload_claims[id].
I apologize about Insufficient explanation.

Never Mind, I solved it by adding the function in jwt.h
		builder& set_payload(typename json_traits::object_type jsonStr) { 			payload_claims = std::move(jsonStr); 		    return *this;         }
Thanks for help, prince.

Ohh!
You could have called the method twice.
builder.set_payload_claim("test", /* json string hello */);
builder.set_payload_claim("test2", /* other */);

Thanks for sharing your solution!
PS you might be missing some other claims that are recommended like issuer. Not sure your application or situation but wanted to throw that in.

@MountainNine If I understood correctly what you wanted to do is replace the payload claims entirely. You are right that this is not supported by default and the reason for this is simple: JWT requires certain claims to be present and set to correct values. Otherwise its not a valid jwt and jwt-cpp (or any other jwt library) will correctly verify it. If you have a third party application that requires exactly this, then your way it probably fine, however if you control the layout of the jwt I highly recommend sticking with the defaults.


Add version 0.8.7

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This a downgrade in version required to add https://github.com/polca/premise

@conda-forge-admin, please rerender


Xbox One gamepad detected as generic XInput device on Windows

Godot version:
3.1.1
OS/device including version:
Windows 10
Issue description:
Xbox One controller on Windows 10 (wired) is detected as a generic XInput controller with a guid of __XINPUT_DEVICE__ instead of anything from the SDL controller list.  This prevents me from accurately identifying the controller as something other than a generic gamepad.
Additionally, I'm confused as to why I'm getting different results from Godot on Windows vs. Godot on Linux when it comes to comparing GUIDs from the SDL Gamepad tool (available here: http://www.generalarcade.com/gamepadtool/)
SDL Tool 1.2 Ubuntu 18.04
Xbox One Controller
GUID: 030000005e040000ea02000001030000
Name: XInput Controller
PS4 Controller
GUID: 030000004c050000c405000011810000
Name: PS4 Controller
Godot 3.1.1 Ubuntu 18.04
Xbox One Controller
GUID: 030000005e040000ea02000001030000
Name: Xbox One Wireless Controller
PS4 Controller
GUID: 030000004c050000c405000011810000
Name: Sony DualShock 4
SDL Tool 1.2 Windows 10
Xbox One Controller
GUID: 030000005e040000ff02000000007801
Name: XInput Controller
PS4 Controller
GUID: 030000004c050000c405000000000000
Name: PS4 Controller
Godot 3.1.1 Windows 10
Xbox One Controller
GUID: __XINPUT_DEVICE__
Name: XInput Gamepad
PS4 Controller
GUID: 4c05c405000000000000504944564944
Name: PS4 Controller
All tests above were conducted with the controllers wired, not wireless.
Steps to reproduce:
Connect a controller and use Input.get_joy_guid(0).

Possibly related to #26544, Some bindings and SDL gamepad tool use unsupported 2.0.6+ binding format.

Actually I've just realized that I also can't find to GUID that the SDL gamepad tool shows on Windows 10 for the Xbox One controller (030000005e040000ff02000000007801) in either Godot's SDL GUID list or the list over here: https://github.com/gabomdq/SDL_GameControllerDB
Does the GUID change with the 2.0.6 binding format?  Or do I have a strange Xbox One controller?

Does the GUID change with the 2.0.6 binding format?

Looks like it is changed. My Xbox One controller have 5e04000000000000d102000000000000 GUID with pre 2.0.6 version of Gamepad tool and 030000005e040000d102000000000000 with the new one.

I guess either way I would expect to get at least one of those back from Input.get_joy_guid() to even allow me to add my own mapping for it.

According to generalarcade's gamepadtool, my Xbox One controller's GUID is 030000005e040000dd02000000007801, not in any of the lists either.

The problem is basically described here: #8368 . We should address that issue

Can you check if it works fine in 3.2 beta 6 or later?
I might have fixed this with #34910.

I'll assume it fixed by #34910 for now, and if it isn't, it should be fixable by contributing the missing mappings to https://github.com/gabomdq/SDL_GameControllerDB.
Please comment if you can still reproduce the issue / need help to contribute mappings upstream if they're missing.

Godot 3.2.2 Beta 3 Windows 10
Xbox One Controller (Model 1708)
GUID: __XINPUT_DEVICE__
PS4 Controller (Model CUH-ZCT1U)
GUID: 4c05c405000000000000504944564944
The PS4 is connected via USB and the Xbox I tried both wired and bluetooth.
Steps to reproduce:
Connect a controller and use Input.get_joy_guid(0)

Related to https://github.com/godotengine/godot/issues/38554#issuecomment-633980886
CC @madmiraal

There are two issues here (and both are specific to Windows):

The GUID returned by Godot, which should be fixed by #39060.
Godot has its own XInput device mapping, hence the #38554 comment.

Can you still reproduce this issue in 3.2.3 RC 1 or the current master branch?

@akien-mga
3.2.3.rc1:
Input.get_joy_guid(0) -> __XINPUT_DEVICE__
master (0a7942f4bb61cba8f02f2cbe0ec83b95eefa6726):
Input.get_joy_guid(0) -> __XINPUT_DEVICE__

On Windows, all XBox controllers should be picked up as XInput devices, irrespective of it's GUID. If Godot detects an XInput device, it just assigns it the XInput mapping. Godot doesn't even try to determine the GUID, which is why there is no GUID. So, aside from not returning the GUID when requested, the original issue is not an issue.
What is odd is the mapping used by Godot is so different from the mapping in the SDL database:
Godot:
__XINPUT_DEVICE__,XInput Gamepad,a:b12,b:b13,x:b14,y:b15,start:b4,back:b5,leftstick:b6,rightstick:b7,leftshoulder:b8,rightshoulder:b9,dpup:b0,dpdown:b1,dpleft:b2,dpright:b3,leftx:a0,lefty:a1,rightx:a2,righty:a3,lefttrigger:a4,righttrigger:a5,platform:Windows,

SDL database:
xinput,XInput Controller,a:b0,b:b1,back:b6,dpdown:h0.4,dpleft:h0.8,dpright:h0.2,dpup:h0.1,guide:b10,leftshoulder:b4,leftstick:b8,lefttrigger:a2,leftx:a0,lefty:a1,rightshoulder:b5,rightstick:b9,righttrigger:a5,rightx:a3,righty:a4,start:b7,x:b2,y:b3,platform:Windows,

However, as long as someone (@hoontee or @tristongilley?) can confirm that the mapping for the XBox controller works using the Joypads Demo, I would suggest closing this issue.
Note: If there is still an issue in 3.2.3.rc1 (i.e. after #39060), with other, non-XBox controllers like the PS4 Controller, a separate issue should be opened. This issue is specific to XBox controllers.

However, as long as someone (hoontee or tristongilley?) can confirm that the mapping for the XBox controller works using the Joypads Demo, I would suggest closing this issue.

Yes, the gamepad works as expected on the 3.2 and master branches.

Closing because, on Windows all XBox One controllers should be detected as an XInput device i.e. this is working as expected.


Unexpected results: Search for "game"

URL: https://pub.dartlang.org/search?q=game
Only the first 5 packages have the term "game" in them. Am I missing something?

Plenty of room to tweak the algorithm – thanks!

Yeah, agreed. So what would be your expectation here? Cut off everything that has no "game" in it (either in the package name or in the description/readme)?

The main reason why this happens is due to the use of character n-gram indexing, I guess?
We could move more to a word-level approach, but as long as the correct search results appear on the top of the list, this seems like not the most important thing atm. Furthermore in the mid-term we might have the option of using a hosted solution.

The main reason why this happens is due to the use of character n-gram indexing, I guess?

That is correct. One way to fix this and preserve the ability for returning results with typos (either in the query or the indexed source): we could use only the longest-matching n-grams. E.g. in this case we find a 4-gram sequence, so we can drop 3-grams, 2-grams and 1-grams. If there were no match for the 4-gram sequence, we fall back to 3-grams etc. (Or alternatively just decreasing their weight in such cases.)

That would be one approach.
Though using a word-level approach can still handle typos. There's several possibilities for this. e.g. using DWT (or something similar) to go from search word to closest indexed words and then using these words for search. It would also allow "spelling-correction" suggestions ...
But again, getting popularity scores automatically into the system and other things are probably higher priority.

@matanlurey – better? https://pub.dartlang.org/search?q=game


Version: Make fields readonly and delete duplicate code

Port https://github.com/dotnet/coreclr/pull/7246
cc: @jkotas

Thank you!


Column mapping dangerously applied twice to the same cell!

Wow. This was fun. I don't know how this ever worked. By coincidence, are all the conversion procs that everyone has ever used idempotent?! That would surprise me.
Here's the minimal failing spec (why Money matters will be apparent soon):
gem 'money'
require 'money'

describe "Mapping a column" do
  example "my conversion proc is called twice, and that sucks" do
    @table = Cucumber::Ast::Table.new([["Receipt Amount"], ["CAD 100"]])
    conversion_proc_for_receipt_amount = mock()
    conversion_proc_for_receipt_amount.should_receive(:call).once.and_return(Money.new(10000, "CAD"))
    @table.map_column!("Receipt Amount") { |text| conversion_proc_for_receipt_amount.call }
    @table.hashes.should == [{"Receipt Amount" => Money.new(10000, "CAD")}]
  end
end

This spec fails, because it calls my column mapping proc twice. For example, this causes Table to try to parse "CAD 100" into a Money, then parse that Money object again, which happens to do something weird and annoying. Clearly, that ain't right. Without some weird hack, like refusing to parse something that's already a Money, I don't know how to make that conversion proc idempotent. That would seem like an unreasonable requirement.
The two calls appear to be in convert_columns! and to_hash. Without digging deeper, it seems to me that to_hash ought not to convert the columns directly, when it could use convert_columns!, and that method could memoize the result.
I don't have the energy to fix this now, and even so, I'd like someone there to confirm that it's indeed a mistake, and that you didn't intend for the conversion procs to need to be idempotent.
Thanks!

Hi J.B.,
I didn't get the failure you did. I used the following:
require 'cucumber'
describe "Mapping a column" do
example "this works" do
@table = Cucumber::Ast::Table.new([["Receipt Amount"], ["CAD 100"]])
conv_proc = mock()
conv_proc.should_receive(:call).once.and_return(100)
@table.map_column!("Receipt Amount") { |text| conv_proc.call }
@table.hashes.should == [{"Receipt Amount" => 100}]
end
end
Even installing a money gem passes:
require 'cucumber'
gem 'money'
require 'money'
describe "Mapping a column" do
example "this works" do
@table = Cucumber::Ast::Table.new([["Receipt Amount"], ["CAD 100"]])
conv_proc = mock()
conv_proc.should_receive(:call).once.and_return(Money.new(10000, "CAD"))
@table.map_column!("Receipt Amount") { |text| conv_proc.call }
@table.hashes.should == [{"Receipt Amount" => Money.new(10000, "CAD")}]
end
end
corys-mac-mini:jbcuke coryfoy$ rspec spec/
.
Finished in 1.74 seconds
1 example, 0 failures
corys-mac-mini:jbcuke coryfoy$ cucumber --version
1.1.0
corys-mac-mini:jbcuke coryfoy$ ruby -v
ruby 1.9.2p290 (2011-07-09 revision 32553) [x86_64-darwin11.0.1]
Yikes! Out of date cucumber. Installed 1.1.4 and got...failure!
corys-mac-mini:jbcuke coryfoy$ cucumber --version
1.1.4
corys-mac-mini:jbcuke coryfoy$ rspec spec/
F
Failures:

Mapping a column this works
Failure/Error: conv_proc.should_receive(:call).once.and_return(Money.new(10000, "CAD"))
(Mock).call(any args)
expected: 1 time
received: 2 times
./spec/money_spec.rb:8:in `block (2 levels) in <top (required)>'

Finished in 1.58 seconds
1 example, 1 failure
Failed examples:
rspec ./spec/money_spec.rb:5 # Mapping a column this works
Looks like the bug was introduced here possibly? https://github.com/cucumber/cucumber/commit/abb0025068295cea8e79a3da47052917fce482f8#diff-0

git bisect concurs that this commit introduced the difference: https://github.com/cucumber/cucumber/commit/abb0025068295cea8e79a3da47052917fce482f8
Thank you for helping nail that down, at least.

It looks like that commit had a failing spec. Tsk tsk.

Thanks for reporting this JB. I've never used this feature but it sounds as if the behaviour you expected was reasonable.

I've exchange email with Richard; I might or might not look into this myself to try to fix it. I would like to, but other priorities have crowded it out for now.

I think this is fixed by the above commit.

A quick workaround until this is fixed (check if text is a String)
@table.map_column!("Receipt Amount") { |text| do_mumbo_jumbo(text) if text.is_a?(String) }

Except when text is not a String your map is now returning nil. that doesn't sound useful. perhaps
{ |text| text.is_a?(String) ? do_mumbo_jumbo(text) : text }
Or just #208.

yep, my trying to be a smartass here :). I really do in my code what you suggested above... thx for clarifying this


instanceof Array can fail

I'm hacking on an Electron project which I'm trying to use elm for. In the electron code, one of my dependencies uses vm.runInNewContext which causes that code to have a different Array prototype which causes elm's instanceof checks to fail (see https://github.com/joyent/node/issues/1277).
Can elm use Arrays.isArray instead of instanceof checks?
It looks like it's probably just a matter of editing https://github.com/elm-lang/elm-compiler/blob/a9658ccb44cf3358991671b7abe02c4d87a31151/src/Generate/JavaScript/Port.hs#L87 and possibly adding a new helper. But I'm very new to elm (and don't have a great grounding in JS stuff anyway), so I figured I'd raise the issue before I sent a pull request.

It was introduced in ES5: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/isArray
Unfortunately this is only one of many ways people can mess up the global execution environment in a way that could mess Elm up. Monkey patching is and always has been caveat emptor.
I think we should leave the current behavior as-is on this one.

Okay, I don't want to drop support for IE9 for this reason, and I suspect that's what we'd need to do here.
Thanks @rtfeldman!


Cannot find name 'ScrollSpyModule'.

i did as per the readme text .
when i try to start using npm,i get the below error
app/app.module.ts(19,5): error TS2304: Cannot find name 'ScrollSpyModule'.

I just updated readme.md with proper inputs for demos. My mistake sry.

Thanks .it solved the error.But new errors pop up.
You have working version of this.?

in my app.module.ts
import { ScrollSpyModule } from 'ng2-scrollspy';
@NgModule({
bootstrap: [App],
declarations: [
App
],
imports: [ // import Angular's modules
BrowserModule,
HttpModule,
RouterModule,
FormsModule,
ReactiveFormsModule,
ScrollSpyModule.forRoot(),
NgaModule,
PagesModule,
routing
],
providers: [ // expose our Services and Providers into Angular's dependency injection
ENV_PROVIDERS,
APP_PROVIDERS
]
})
in my page.module.ts
import { ScrollSpyElementDirective, ScrollSpyService } from 'ng2-scrollspy';
@NgModule({
imports: [CommonModule, NgaModule, routing],
declarations: [Pages,ScrollSpyElementDirective]
})
in my page.componenet.ts
import { ScrollSpyElementDirective, ScrollSpyService } from 'ng2-scrollspy';
export class Pages {
constructor(scrollSpyService: ScrollSpyService) {
this.scrollSpyService = scrollSpyService;
}
ngAfterViewInit() {
    this.scrollSpyService.getObservable('test').subscribe((e: any) => {
        console.log('ScrollSpy::test: ', e);
    });
}

}
when i run through npm
[default] /Users/kodiraj/Downloads/nextgenportal1/src/app/pages/pages.component.ts:38:14
Property 'scrollSpyService' does not exist on type 'Pages'.
[default] /Users/kodiraj/Downloads/nextgenportal1/src/app/pages/pages.component.ts:42:14
Property 'scrollSpyService' does not exist on type 'Pages'.
[default] Checking finished with 5 errors
it would really help ,if you have some demo of this feature

in you pages.component.ts you should declare the scrollSpyService variable. One way is:
export class Pages {
    constructor(private scrollSpyService: ScrollSpyService) {
    }

    ngAfterViewInit() {
        this.scrollSpyService.getObservable('test').subscribe((e: any) => {
            console.log('ScrollSpy::test: ', e);
        });
    }
}

hi,
Can you share some demo code if possible..

On Sep 21, 2016, at 7:43 PM, João Ribeiro<EMAIL_ADDRESS>wrote:
I just updated readme.md with proper inputs for demos. My mistake sry.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/JonnyBGod/ng2-scrollspy/issues/68#issuecomment-248624847, or mute the thread https://github.com/notifications/unsubscribe-auth/AKzpwzL7BSV0GhSEM1gMIlk5pvBUqU2Bks5qsTucgaJpZM4KCY4H.

hi,
i have nested components.Here how should i handle it?

On Sep 26, 2016, at 7:31 PM, João Ribeiro<EMAIL_ADDRESS>wrote:
in you pages.component.ts you should declare the scrollSpyService variable. One way is:
export class Pages {
constructor(private scrollSpyService: ScrollSpyService) {
}
ngAfterViewInit() {
    this.scrollSpyService.getObservable('test').subscribe((e: any) => {
        console.log('ScrollSpy::test: ', e);
    });
}

}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/JonnyBGod/ng2-scrollspy/issues/68#issuecomment-249577841, or mute the thread https://github.com/notifications/unsubscribe-auth/AKzpw7-XsRV87QpSGKYpydqngynDUGccks5qt9BMgaJpZM4KCY4H.

scrollSpyElement=“kodiraj"
i declared the above tag in only one place.
ScrollSpy: duplicate id "kodiraj". Instance will be skipped!
But i get the error in the console.

On Sep 26, 2016, at 7:31 PM, João Ribeiro<EMAIL_ADDRESS>wrote:
in you pages.component.ts you should declare the scrollSpyService variable. One way is:
export class Pages {
constructor(private scrollSpyService: ScrollSpyService) {
}
ngAfterViewInit() {
    this.scrollSpyService.getObservable('test').subscribe((e: any) => {
        console.log('ScrollSpy::test: ', e);
    });
}

}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/JonnyBGod/ng2-scrollspy/issues/68#issuecomment-249577841, or mute the thread https://github.com/notifications/unsubscribe-auth/AKzpw7-XsRV87QpSGKYpydqngynDUGccks5qt9BMgaJpZM4KCY4H.

Please try new version v0.3.0.
Make sure you read docs for new declaration system.
I will try to create more documentation and example during next week.


AFNetworking not releasing connection with server in ios8

Hi,
I am having a problem that it is not releasing the connection with the server while uploading an image. Is this issue in iOS8. I am using AFHTTPRequestOperationManager. Is it solvable if i use AFURLSessionManager. Please suggest me. My code is like this.
AFHTTPRequestOperationManager *manager = [[AFHTTPRequestOperationManager alloc] initWithBaseURL:[NSURL URLWithString:ServerURL]];
[manager.requestSerializer setValue:@"RnyooiOS" forHTTPHeaderField:@"x-rnyoo-client"];

[manager.requestSerializer setValue:[Util getNewUserID] forHTTPHeaderField:@"x-rnyoo-uid"];

[manager.requestSerializer setValue:[Util getSessionId] forHTTPHeaderField:@"x-rnyoo-sid"];

[manager.requestSerializer setValue:[objImg valueForKey:@"imgId"] forHTTPHeaderField:@"x-rnyoo-vfid"];

RLogs(@"header - %@", [manager.requestSerializer.HTTPRequestHeaders description]);

RLogs(@"Img Info - %@", [objImg valueForKey:@"imgPath"]);

    NSData *imageData = [NSData dataWithContentsOfFile:[objImg valueForKey:@"imgPath"]];

    if([imageData length]==0)
    {

        return;
    }
    if([imageData length]> 0)
    {
    AFHTTPRequestOperation *op = [manager POST:@"vault/files/upload" parameters:nil constructingBodyWithBlock:^(id<AFMultipartFormData> formData) {
        //do not put image inside parameters dictionary as I did, but append it!
        [formData appendPartWithFileData:imageData name:@"vaultfile" fileName:[objImg valueForKey:@"imgName"] mimeType:@"image/webp"];

       // NSInputStream *stream = [NSInputStream inputStreamWithData:imageData];

       // [formData appendPartWithInputStream:stream name:@"vaultfile" fileName:[objImg valueForKey:@"imgName"] length:[imageData length] mimeType:@"image/webp"];

    } success:^(AFHTTPRequestOperation *operation, id responseObject)
                                  {

                                      RLogs(@"Success: %@ ***** %@", operation.responseString, responseObject);

                                      if([[responseObject valueForKey:@"file_upload"]isEqualToString:@"success"])
                                      {
                                          //We are updating vault image sync status and URl in local Db and then trying to post all the audio files of hotspots.

                                          [self updateImagedataInDBWithSyncStatus:YES withImageUrl:[responseObject valueForKey:@"vaultfile"]];
                                          [self uploadMediaFile];
                                      }
                                      else
                                      {

                                          //if uploading vault image fails, we are just updating image id in local DB as to send another upload call with new GUID.
                                          [self updateImageDataInDBWithNewImgId];

                                          UIAlertView *alert = [[UIAlertView alloc] initWithTitle:@"Rnyoo"
                                                                                message:@"Image not uploaded"
                                                                                delegate:self
                                                                                cancelButtonTitle:@"Cancel" otherButtonTitles:@"Retry",nil];
                                          alert.tag = 111;
                                          [alert show];
                                          return;

                                      }
                                  }
                                       failure:^(AFHTTPRequestOperation *operation, NSError *error)
                                  {
                                      RLogs(@"upload vaultError: %@ ***** %@", operation.responseString, error);

                                      // to show alert and remove loader
                                      NSNotificationCenter* nc = [NSNotificationCenter defaultCenter];
                                      [nc postNotificationName:@"PodSyncError" object:nil userInfo:(NSDictionary*)error.userInfo];

                                  }];
    [op start];

    }

This is most likely an issue in your usage of AFNetworking, rather than in the library itself. This is is not a support forum. For usage questions, please try asking on Stack Overflow with the "AFNetworking" tag.


How about tag_invoke?

According to the code of serde_gen, the implementation of serialize and deserialize of custom class seems to be based on struct template specialization. However, the specialized struct template should be defined in namespace serde, which is impossible for private classes. I didn't test it because I don't know exactly how to use this library and how to check the generated C++ files. Another issue of struct template specialization is that it is possible to specialize serde::Serialize<ClassInLibA> in two different libraries, e.g., library B and library C. Then if we link library B and library C at the same time, it will produce symbol conflicts.
I propose to use tag_invoke. It only reserves one name tag_invoke in the user namespace, and it solves all problems described above. Its behavior is very similar to Rust's trait. For example, with tag_invoke, you can either implement serialize for a known type along with the definition of the serialize interface (e.g., implement serialize for std types in this library) or implement serialize for a custom type along with the definition of that type. In another word, it does not allow implementing third-party interfaces for third-party classes. Although it seems to be somewhat restrictive, it eliminates ambiguousness. I have tried to use tag_invoke to implement serialization and deserialization, and it works fine: https://github.com/iidb/wing/blob/1fd2fc0f9b557ac65bf2fc87c5af446fb6ca1794/src/common/serde.hpp

You are right, there is currently no way to support serialization generation for private classes, although, there is a way to still use the serde APIs explicitly if you implement serialize and deserialize methods in that private class. See example test case Advanced.LocalPrivateTypes.
Regarding usage docs/examples, I plan on creating a separate example repository to serve as a template project that uses serde-cpp. That is to show how to use serde-cpp cmake package and call the serde_generate_target function from CMake to generate the serialization/deserialization files.
Your tag_invoke suggestion seems interesting, I will look into your linked example and evaluate how to bring it to serde-cpp. We may continue discussing viability and implementation details in this thread.


Support for endAtDocument, endBefore, endBeforeDocument and startAtDocument

I found this library very useful but it was just missing some query modifiers: endAtDocument, endBefore, endBeforeDocument and startAtDocument
And I needed them in my tests so I implemented them in this pr #197

Hi @Andrew-Bekhiet, thank you for contributing! I took a look at your PR and it looks good. I'll review it more closely by the end of next week and submit it if all is well.


Added EDPM based Zuul job

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/192
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/194

recheck

recheck

https://review.rdoproject.org/r/c/config/+/48698 is merged now,

recheck

recheck

recheck

/lgtm


Call service to trigger Restore from Glacier to S3

As a Herd Consumer I want to request that a BData archived to Glacier is restored to S3 so I can access the data.
Acceptance Criteria

Endpoint initiates restore from Glacier
** Endpoint requires READ and WRITE permission on the BData Namespace
** Endpoint defaults to 7 day duration of restore
** Endpoint returns immediately and indicates restore began
** Endpoint fails if more than two enabled Storage of Storage Platform Glacier
Data is restored to S3 in Glacier 0d bucket -- but checking for restore and completion of other operations is not part of this story

This is a duplicate of #88, some kind of browser/submit mixup created two of them


Handle OpenEXR chromaticities for color management, Sun Valley style enablement

Certain OpenEXR images may use color spaces such as ACES, therefore it's important to read chromaticities information and provide corresponding input to color management for accurate color reproduction (seems like even Photoshop is not doing so.)
The retrieval routine partially reuses the color information override routine for generating D2D1 Color Context, renamed corresponding variables to represent their corresponding purpose now: m_customOrDerivedColorProfile.
Partially included the fix at https://github.com/13thsymphony/HDRImageViewer/pull/46 to address a build break. Unit tests are not fixed though (looks like they are kind of abandoned for a while.)
Included WinUI 2.7 package and enabled Windows 11 / Sun Valley visual styles.
Screenshots
Before

After

Do you have a sample?

seems like even Photoshop is not doing so

Well, it is not Davinci.

imbushuo shared some test images:
https://github.com/AcademySoftwareFoundation/openexr-images/tree/master/Chromaticities

Thanks for your patience! I've made two tiny functional changes, falling back to a D65 whitepoint if white.y == 0, and updating Windows SDK prereqs. If you have no issues with my updates then I'll merge this.

I hijacked this PR a bit to do some more project file cleanups. I'll go ahead and merge this and we can pick up any additional changes later if needed. Thanks for the contribution!


gitignore integration.test binary

Noticed I had a integration.test bin at my projct root after running the
integration tests and it was not ignored by git.

Well, I think if you run the integration test again you will get integration.test2, .test3, etc or something along those lines.
Maybe look into pointing the integration test output to an output dir and then we can ignore the dir?

Or we can just use a integration.test* to ignore them all for now, no?

@carolynvs Makes sense to me, especially if the default behavior is to produce the binary at that location.

Updated.

@eriknelson THANKS!!! I had the same!


Allow protobuf compilation

It would be great if the protobufs could be compiled through IntelliJ with a source output that is configurable. I saw a response to someone's comment on this page saying that this most likely won't be implemented and to use protoc with your build tool.
While I understand using protoc for protobuf compiling, it makes using IntelliJ less convenient. I can either compile all of my source code with my own [custom] build tool outside of IntelliJ, or I can use IntelliJ for compilation while developing. Compiling everything through IntelliJ makes development much easier and more seamless. Of course, the plugin could simply call protoc behind the scenes, much like other protobuf plugins do.

I agree that source code generation from IntelliJ IDEA by triggering protoc might be convenient. If we talk about IDEA-based project that do not use external build system like maven/gradle, then it will be not only convenient, but also mandatory.
Generating source code from proto files as a part of compilation is not a priority for me now (all my projects use maven or gradle), so I will consider implementing it only when I finish with release 1.0 - https://github.com/protostuff/protobuf-jetbrains-plugin/wiki/Roadmap.
However, if somebody is willing to help - I will gladly accept a pull requests for this feature and make a new public release as soon as it is ready.

the plugin could simply call protoc behind the scenes, much like other protobuf plugins do.

There is only one "other" plugin, and its functionality is very limited - as far as I remember, it can generate only java code, and it does not allow to use any additional protoc plugins or options.
If we are going to add this feature, it should be much more flexible:

Allow code generation for all languages supported by the latest protoc.
Do no force user to install protoc on a machine, it can be bundled together with plugin (but allow to use external one as well).
Allow external protoc plugins, like gRPC Java Codegen Plugin for Protobuf Compiler.
Output directory should be configurable.
If output directory is not configured, it should be selected automatically depending on IDE and project type. Generated code should be added to the project sources, if applicable.

It would be very useful if the compile functionality supports custom include paths for proto files for each submodule in a project. Please see https://github.com/nnmatveev/idea-plugin-protobuf/commit/ab1497422b68f0aa350e1db7c4c8266032c35053 for similar functionality in the other protobuf plugin.

I have the same question，has this question been settled yet？

Currently I do not know when I will have time to start working on it.
But if there is somebody willing to help, I will gladly accept pull request.

Hi,
I hope I will have some time in the near future to add initial support for protobuf compilation. But I believe that it's better to start with just a minimal implementation to simply allow users to add protobuf compilation to their projects. In my opinion having this option would be enough for 90% of use cases.
How do you think is it acceptable to not implement every desired feature at the start?

Do no force user to install protoc on a machine, it can be bundled together with plugin (but allow to use external one as well).

I don't think it's a good idea because anyway every project should have the opportunity being built  from console that means user will definitely have protoc already installed. In my opinion this feature will also add complexity and ambiguity because protoc version from console should be the same as the protoc version bundled with the plugin. What do you think?

As I figured out nothing makes you to write protoc as the whole. It's enough to write the plugn(s)

@hsestupin

I believe that it's better to start with just a minimal implementation to simply allow users to add protobuf compilation to their projects. In my opinion having this option would be enough for 90% of use cases. How do you think is it acceptable to not implement every desired feature at the start?

Sure, that makes sense.

Do no force user to install protoc on a machine, it can be bundled together with plugin (but allow to use external one as well).

I don't think it's a good idea because anyway every project should have the opportunity being built from console that means user will definitely have protoc already installed. In my opinion this feature will also add complexity and ambiguity because protoc version from console should be the same as the protoc version bundled with the plugin. What do you think?

In the same manner I assumed that every project has a build system like ant/maven/whatever once. But there are lots of people who don't use that stuff and want IDE to compile and build code.
However, you are right - it will add complexity and ambiguity. So it is better to start without it.

This issue is closed as plugin is not supported anymore.
Please switch to https://github.com/jvolkman/intellij-protobuf-editor.


Create a better 404 page for our public site

Please see https://simplereport.gov/ fake url for current
Brought to attention by Ben here: https://usds.slack.com/archives/C019PRYMR0Q/p1621526373011700

@timbest-usds to look into logs to see how often 404 pages are being hit


Invalid mime type for serving css?

It seems that a css file is returned with the incorrect mime type. Any idea what may be wrong?

As a workaround, I used:
    app.get('/public/*', ({set, params}) => {
        const file = Bun.file(`${import.meta.dir}/../public/${params['*']}`)
        set.headers['Content-Type'] = file.type
        return file.text()
    })

As being mentioned in #524

Thanks for reporting this finding!
Can't reproduce it on my end. Can you give it a try with the latest verion?


HARError: validation failed

I am using chrome 56 to collect the har file
when I use:
sudo har-server last.har -p 80
I get
HARError: validation failed
How can I check what is wrong?
Thank you

Using har-validator with the following options comes as correct:
har-validator -s log,request,response last.har
Maybe har-server should have an option to pass har-validator parameters?
Thank you

I'm having the same error. Are there any fixes?


make test expressions into strings

upgrade to #8 for more precise calculation of execution time of tests.

Todo: evaluate if this is worth it to include

I think this looks like an improvement. Note that you can get rid of many of the () at the beginning and end of quoted expressions.

Closing because alternate fix by #13


HB-Search keyword on people should get all the individual records having keywords matching name and/or about

1-User enters keyword to search : for example - the
2-Click on People tab and get two records with the on about each record
3-Click on Filter and enter -San Jose , CA for location and click on Apply Filters
4-Since the Individual 's first record checked on Don't show my address on profile , the first record is not in display as expected and the second record is on the top since Individual has San Jose , CA address
5- Also see 4 more records have 'the' in the Name.  Expect- These 4 news records should display on step 2.  Step 2 should have 6 records instead of 2 records
6- if click x to remove the location , we come back on 2 records we have on step 2 which I expect to see 6 records.
please see the GIF

@Zaydme can you please review this one

this behavior was fixed in https://github.com/FightPandemics/FightPandemics/pull/1695 for signed in users.

For the unsigned we said that there will be only full word match (full $text search) when no location filter is selected (because not using $geoNear, so using full $text search and NOT regex) so a record must have at least one 100% match for whole word. So this is why results are less, While it can do partial match when location is known (now it's using regex and NOT full $text because using $geoNear) so there will be more results (even the ones not including a 100% match of the word).
note that the highlighting might highlight half word, but the back-end only matches full word in that case (unknown location)

Agree with close after perform the validation.


Type-only imports don't work

TypeScript Version: 4.0.0-dev.20200430
Search Terms: type-only imports, import type
Code
// From /client/src/Y.ts
import type { X } from '../../server/src/X'

// ...

Expected behavior: The whole server code should not be part of the compiled output.
Actual behavior: It is.

You need to provide a reproducible example where tsc takes an import type expression and does not elide it from the compiled output.  This is more likely a problem with some larger build chain.

import type vs import does not affect which sources are / aren't part of your project.

From the release notes of 3.8 (emphasis not mine):

It always gets fully erased, so there’s no remnant of it at runtime.

If that were any true, the client wouldn't have to carry with it the entire server just because it wants to import a type from it.
A reproducible example, as requested: https://github.com/inad9300/ts-import-type.

Again, this is a misunderstanding of what import type does

This should not even happen with regular imports... Whatever.


Modernize code

Some long overdue code cleanup. The only big change is that, instead of copying all the BaseState properties to Orbit, we are delegating them if they exist. Besides, I turned some properties into read-only. @anhiga would you please review this?

Hold on @anhiga, don't review it yet. Now that I'm improving the code quality, I will take some time to configure Code Climate since it's making too much noise and we never really pay attention to it.
https://docs.codeclimate.com/docs/advanced-configuration

In particular, I would like to:

Remove duplicated code checks (they are too strict in my opinion, and I prefer clear code than clever code)
Exclude tests from some checks

So the rest of the issues should be worth it.

This is now ready to review :)

Thanks a lot!  Some Code Climate updates required removing and adding the project again. We lost the history, but it was not interesting anyway :)


unable to decode nSig: SyntaxError: SyntaxError: (anonymous): Line 19:123 Missing catch or finally after try (and 1 more errors)

2 days ago everything was working fine, but now it's broken, does anytjing change again from youtube side ?

我也遇到了同样的问题

goja does not seem to be able to parse the javascript that is downloaded from the youtube js player.
I tried to unminify the js but the javascript itself seems seems to have a syntax error so it does not seem to be a bug with goja.
That being said, I am not an expert on javascript so I might be wrong.
Do you have any idea what might be wrong @kkdai ?

又好了，有老哥给发下科普文章吗？原理到底是啥

Please check out the latest relase 2.7.16 and leave a message if this issue still exists.


isl: Fix isl 0.20 when building on new platforms like riscv

Description of changes
Add autoreconfHook to isl.  Fixes #226578.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

commit msg should follow CONTRIBUTING.md

Commit msg was updated

If I have any concerns, it's about the impact on the "length" of our linux stdenv bootstrapping.

Can we merge this?

Can we merge this?

It may have to wait until after the 23.05 release.
This PR triggers a rebuild of every single package in the entirety of nixpkgs.  That is a massive amount of recompiling in order to fix a problem on a platform for which Hydra does not generate binaries.

That's not an issue.  stdenv will be rebuilt regardless due to other changes.

A simple option might be to add the dependency optionally, only on platforms where it might be needed.  (or at least exclude platforms where we care about bootstrap speed)

Yeah this could get ugly.  autoconf needs gettext, which takes an eternity to compile (serial build process, one core used at most).
Before this PR nothing needs autoconf until we get to gawk (i.e. building our own, not the one from the bootstrapFiles) or glibc's special python3 -- both of those are after xgcc, but isl is before.
So this would add a completely new copy of: autoconf, libtool, automake, gettext, m4, zlib, and texinfo.

A simple option might be to add the dependency optionally, only on platforms where it might be needed. (or at least exclude platforms where we care about bootstrap speed)

Yeah I think that's the way to go.


how to you resolve multiple composed states with the same methods?

I.e., if you have State('foo' {increment: ...}), State('bar', {increment...}) and you call Action.increment() which one happens?

Both happen just as things would in multi redux reducers. Each state can
respond in its own way to the action :)
On Thu, Jul 13, 2017 at 6:08 PM Dave Edelhart<EMAIL_ADDRESS>wrote:

I.e., if you have State('foo' {increment: ...}), State('bar',
{increment...}) and you call Action.increment() which one happens?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jumpsuit/jumpstate/issues/42, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFUmCTpJSHUjNxKZx3jiy7I-a6Ycrh3tks5sNrF4gaJpZM4OXry0
.


[bug fix] Fix async actions are left in neural_sparse query

Description
The bug exists in multi-nodes and multi-shards environment. For neural_search query, the cluster will throw error "async actions are left".
The search action contains 2 phase, query and fetch. In multi nodes & multi shards case, nodes will finish the query phase normally. But there will be a serialization and deserialization at the start of fetch shard phase, and after deserialization the doRewrite method of constructed QueryBuilder(i.e. NeuralSparseQueryBuilder) will be called. However for this rewrite the QueryBuilder can not have async action otherwise an exception will be thrown.
In our current implementation, the model inference result is not included in the serialization/deserialization process, and it will register an async action when rewritten. And this will break the search.
Add queryTokensSupplier to the process of serialization/deserialization, as well as doEquals and doHashCode
Issues Resolved
[List any issues this PR will resolve]
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

should be in 2.11


Add Product resource

[ ] Add Product class
[ ] Add ProductRestController class
[ ] Add ProductRestController test
[ ] Change ClinicService interface
[ ] Change ClinicService test
[ ] Change ClinicServiceImpl class
[ ] Add ProductRepository interface
[ ] Add SpringDataProductRepository class
[ ] Add JpaProductRepository class
[ ] Add JdbcProductRepository class
[ ] Change initDB file
[ ] Change populateDB file

https://www.petlove.com.br/racao-seca-nutrilus-pro-frango-carne-para-caes-adultos/p?sku=2492303


[Keyvault] Fix #20520 az keyvault network-rule: Support removing multiple IP

Fix #20520 az keyvault network-rule: Support removing multiple IP
Testing Guide
keyvault network-rule remove --ip-address {ip} {ip2} {ip3} --name {kv} --resource-group {rg}
Delete multiple IP in one remove command
History Notes
[Keyvault] Fix #20520 az keyvault network-rule: Support removing multiple IP

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Keyvault


Upgrade to terraform-provider v2.8.0

Fixes: #159

@jen20 I changed the version number back - I will merge on a green build


Update Python & Ansible matrix

Change Summary
Use a matrix to test different version of ansible during CI
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[x] Other (please describe): CI

Related Issue(s)
N/A
Component(s) name
Github Actions
Proposed changes

Add multidimensional checks for ansible and Python for the scenarion below:

avd-l3ls-ebgp
avd-l3ls-ebgp-json
eos-cli-config-gen

A test matrix is:

Python 3.6, 3.7, 3.8
Ansible 2.9, 2.10 (latest build for each branch)

This GH action only runs in Pull Request using base branch releases/*
How to test

Tested by CI in the following run
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have updated molecule CI testing accordingly
[x] I have read the CONTRIBUTING document.
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

ERROR
Upgrading directly from ansible-2.9 or less to ansible-2.10 or greater with pip is
known to cause problems.  Please uninstall the old version found at:
/opt/hostedtoolcache/Python/3.6.12/x64/lib/python3.6/site-packages/ansible/__init__.py

and install the new version:
pip uninstall ansible
pip install ansible

If you have a broken installation, perhaps because ansible-base was installed before
ansible was upgraded, try this to resolve it:
pip install --force-reinstall ansible ansible-base

If ansible is installed in a different location than you will be installing it now
(for example, if the old version is installed by a system package manager to
/usr/lib/python3.8/site-packages/ansible but you are installing the new version into
~/.local/lib/python3.8/site-packages/ansible with pip install --user ansible)
or you want to install anyways and cleanup any breakage afterwards, then you may set
the ANSIBLE_SKIP_CONFLICT_CHECK environment variable to ignore this check:
ANSIBLE_SKIP_CONFLICT_CHECK=1 pip install --user ansible

END ERROR

Will be implemented as part of issue #320


OPENR in Rpush.mat not being read correctly

Hi,
When I tried to use Rcall, I got an error
"all elements of a list must be named"
and this was being generated by the line
attach(data_Rpush)
this is because the structure OPENR in Rpush.mat was not read in correctly. I don't know what the issue is with R.matlab 's readMat is, because I can see where you have specifically defined the file version a v7 to work with structures, but it just does not appear to work for me. Specifically, the nested structures in OPENR appear as unnamed variables in the list.
My work around so far has been to prevent Rpush.mat from being initialized with OPENR, and instead just saved a dummy variable, and did not use structures.
I checked the filetype created by
save('Rpush.mat','OPENR','-v7')
and it said 5.0 MATFILE on two different computers, but I don't know if that is meaningful.
If there is any suggestion on how to make this work without my janky work around, please let me know.
Thanks for this helpful tool!

Hi ghcorrigan,
thanks for using Rcall. Could you give me some more detail so I can reproduce your issue? Did you test if Rcall and R.matlab work correctly by pushing just an integer or a test structure variable? How does your structure look like? May you give an example structure when the error occurs so I can have a look into it. Which Matlab/R versions do you use?
Best,
Janine

Dear ghcorrigan,
using your given struct variable, indeed the above named error appeared. However, if you change your character variable to a string, the R.matlab package works as it should. So just make sure to convert your elements beforehand.
In the given example, this would be: test = struct("a",{1,2,3},"b",[4 5 6],'c','cows'). And it works nice and need.
Have fun with the tool.
Best,
Janine

Thanks for pointing that out! Technically, it is the inverse, my string variable needed to be a character variable. I say this because it is important to define the path and libpath in Rinit.m as 'character vectors', and not "strings", otherwise when they get passed to  the OPENR structure, they stay as strings, and readMat cannot read the OPENR structure correctly. This was what caused my problems initially.
Thank you again for your help!
Cheers
Ben

Hi Ben,
thanks for pointing that out. The path variables in Rinit now also accept string input and internally convert "string" variables into 'character'.
Best,
Janine


the demo auth has an issue

I kept getting redirected to provide access to github with a message that an unusually high number of requests were being made

We don't store any tokens locally (yet/anymore) instead we just do the auth flow every page refresh, getting a new token from GitHub every time. So you might see this, message during development.
From my testings, this is annoying but you can just "reauthorise" and everything still works as expect.
Will try address token storage asap though. Thanks for the report 👍


Update README.md

Summary
Noticed that if we run make from source, it names the app nodeui, but the downloaded packages from the releases page name the file node-ui. I thought I'd just update the difference on the README.

Thanks!


Python samples

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
/assign @zhu733756

very welcome

python sample has moved to https://github.com/CECTC/dbpack-samples/tree/main/python


Release 1.1.0 with multi maven modules 'core' and 'cluster-api'

About this change - What it does
Klaw as Maven multi-module project. Modules : 'core' and 'cluster-api'.
'core' Core Api with user interface and cluster-api connectivity
'cluster-api' module is cloned from main branch of repo https://github.com/aiven/klaw-cluster-api. Cluster Api with connectivity to kafka clusters
This removes the dependency on cluster-api git repository.

@snuyanzin thank you for the nice review. Regarding usage of LinkedHashMap and synchronized blocks in the repo, would like to address them in a different PR with thorough testing. Issue created #173


feat(data/multiset): disjoint_ndinsert theorems

These come from @digama0 on Gitter.

Thanks!

Travis failed, but it worked for me locally.


Tighten Virk blacklisted username to avoid FP

We've seen a couple FP on this blacklisted username in the past few days. This tightens it up to catch all the TP we've seen in the last 3 months. With this adjustment it does miss 1 post from 9 months ago and 3 from 2 years ago, but those appear to be incidental to what this is primarily attempting to catch.
Search with old blacklist entry on MS
Search with new blacklist entry on MS

Coverage remained the same at 65.485% when pulling 98fe0e29f57a895589b67c32881a18faf05c89fd on makyen:Makyen-tighten-Virk-blacklisted-username into dc7b218b395c58a9689dd0def510d735b8c1e464 on Charcoal-SE:master.


Is it possible to force google to re-crawl an Eddystone URL target page?

When we change our html page content for an Eddystone target URL, it is not clear when the outgoing content for the Eddystone notification will change.  This makes it very difficult to manage content, and even more difficult to A/B test user reactions to the content (since we don't know what content is being served at what time).  We want to change content without changing the URL in the device so that it can be done remotely.  Also, changing the target of the shortlink does not work either.
Thanks

Every notification that you change on your google console is made in real time. If you want to guarantee that you will get the Updated content of the HTML, you must send a new request to google instead keep it in cache in your app.


8 bit read/writes should not send an Endianness argument

This appears to be ignored, but there shouldn't be any reason to send it in the first place.

It's passed to avoid a specific JS implementation for the 8 bit case. If you're worried about performance then probably it would be better to implement specific getters/setters for each word size, since I guess something like dv.getInt8() should be faster than dv['getInt8']() (but don't take my word on that, I'm not very familiar with JS JITs).


Add support for Series and DF at methods

Created _AtIndexer classes for Series and DataFrame and used them to type the corresponding at() methods.
Partial solution to #74.
This doesn't fully work, because it doesn't handle the possibility that a data frame will contain categorical (string) or integer data, instead of just float.  I don't know how to do this.

I applied your changes and I think this is ready for (squash) merge.

@thomkeh I think that this is ready for another review, and possibly merging.

Resolved conflicts and force-pushed.

@thomkeh merge?

Sorry, I was very busy with work! Looks all good!


Idea: support for popular chromecast apps

The current library targets Google's Default Media Receiver (message format and available operations).
However it would be nice if additional apps with their own protocol could be supported.
My idea would be something like this:

Allow the ChromeCastClient client to use different controller implementations
Register controller implementations along with their respective AppId
Let ChromeCastClient automatically load the right controller for the current app

@tapanila what do you think?
Also have a look at https://github.com/balloob/pychromecast/pull/101 (here I implemented a controller for DS Audio)

I plan to make more controllers in the future. But will focus on specific applications if people ask for them.


Some Update

In this update, I improved BardAsync and also improved BardAsyncCookies by extending the BardAsync class.

Thank you for contribution! Before making PR, could you kindly make code style to black? using black code formatter. Have a nice weekend!
black .

Okay, I'll read about it and change the code for the next PR. Mutually have a great weekend