Fix custom marker anchor points

fixes #230

Thanks for the PR, just double checking where the hardcoded values come from?

Not sure where the values come from, but this works great for me. 👍

The hard coded values are to ensure the default behaviour is preserved when no anchor is set via the options.
0.5f, 1.0f is the default for the marker. It gets positioned on the middle bottom (in relation to the coordinates on the map)
0.5f, 0 is the default for the callout window. It get positions on the middle top (in relation to the coordinates on the map)
So the default is that both the marker and callout window are centered horizontally on the point on the map. The bottom of the marker is directly above the point, and the bottom of the callout window is directly above the marker.

Sound good to me 👍 @levic92, is there a simple way to test out the issue? If you can add it to the custom marker example that would be great!

@levic92 @jrichardlai I've just tried the current master, and the anchor still seems off, at least when the Marker mounts. If I'm in debug mode with hot reload on and I change the anchor value, then it "corrects itself" to the right position. Weird.


📦 Release

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

snackbar: adjust position for angular standalone build with optimizer ( #1390)

toast: adjust position for angular standalone build with optimizer ( #1390)

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

:rocket: Welcome to Baloise ChatOps based GitOps Bot :tada:
- If you want to enable preview environments for this PullRequest, comment with `/preview`.
- If you want to disable the preview environment for this PR, comment with `/undeploy`.
- If you need help or want to see the docs, comment with `/help`.

Make sure to refresh this page after submitting a command.
Preview environments can be redeployed automatically if you enable the webhook for updates to the source branch. :new:

Have fun building awesome apps! :hammer:  :whale2:  :truck:  :ship:


geolocation lookup crash

Hi guys,
Firstly, great work on this API - it's simple to use and intuitive.
I noticed that the module crashes when geolocation fails, perhaps from garbage data or the geolocation cannot return coordinates.
I have applied a small fix in an offline branch created from the main branch and would like to contribute these changes.

Thank you, fixed!


Loop when using function in ng-repeat

Hello and sorry for my english,
I'm trying to use a function to get a value from my controller in a ng-repeat but this function is called too many times, i don't understand why.
I did a jsfiddle to show you what i mean.
http://jsfiddle.net/9fR23/240/
I would like to call my function only array.length times.
Thanks for the help

I resolved my problem by using ng-init instead of ng-bind

Sorry, but this place is not the right place to get support. Please use Stack Overflow


dtype error when implementing modeling_transformer.py

Hi, I'm trying to evaluate transformer-1.3B-100B using lm-eval-harness, but encountered this error FlashAttention only support fp16 and bf16 data type.

please use bf16 or fp16 for now as flash_attention only supports these types.


Not erroring if security profile requested but not possible to apply

This came up on the booth at Oscon. If you request applying a seccomp profile (or apparmor) but you are talking to an engine that does not have the ability to do that as it was not compiled with support, it is silently ignored. I think it should probably error out if you explicitly ask for a profile, but not error if you ask for unconfined. It is a really easy change but wanting feedback as this is a behaviour change.
cc @jfrazelle

SGTM better be explicit and fail, otherwise people may incorrectly think they're "protected" by the profile

Yes, in this case the user was just confused as was trying to test it worked, but it never did anything.
I guess it might also be worth being able to explicitly ask for default as a profile, so that you can guarantee that you are running with the default profile and error if not as well.

good call! error out is good

if of course they explicitly pass a custom one


Feature: Add placeholder if user has no connections requests

Type of feature
🍕 Feature
Current behavior
There's currently nothing there if you don't have connection requests.
Suggested solution
Something like this might work and prompt users to connect:

Additional context
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I will work on this. .take

We currently have a placeholder for this

But the issue here is having a blank page when you have no pending requests. i've opened a discussion in https://github.com/open-sauced/engineering/issues/177 for the improvement of this... but in the mean time, i'll add same placeholder for users who have requests but have no pending one


FileNotFoundError: future.py

Just met that bug, not a big one because I think my case is special.
In my project folder there is another virtualenv folder and when trying to parse it :
Traceback (most recent call last):
  File "/home/me/myVirtualenv/bin/findimports", line 11, in <module>
    sys.exit(main())
  File "/home/me/myVirtualenv/lib/python3.6/site-packages/findimports.py", line 822, in main
    g.parsePathname(fn)
  File "/home/me/myVirtualenv/lib/python3.6/site-packages/findimports.py", line 427, in parsePathname
    self.parseFile(os.path.join(root, fn))
  File "/home/me/myVirtualenv/lib/python3.6/site-packages/findimports.py", line 454, in parseFile
    module.imported_names = find_imports(filename)
  File "/home/me/myVirtualenv/lib/python3.6/site-packages/findimports.py", line 319, in find_imports
    with open(filename) as f:
FileNotFoundError: [Errno 2] No such file or directory: './localVenv/lib/python3.5/__future__.py'

Is it expected?

Just looked inside and all python files are broken links...
So maybe you could add a check for fake files but that's not a very common case.
__future__.py -> /usr/lib/python3.5/__future__.py
And /usr/lib/python3.5 does not exist anymore since my upgrade to ubuntu 18.04.

Ah, yes, the happy fun with virtualenvs during system upgrades.
I would accept a PR for making findimports emit user-friendly warning messages in this case (e.g. WARNING: ./localVenv/lib/python3.5/__future__.py is a broken symlink), but I'm not planning on doing any work on this myself.

I agree, not a big impact of doing that.
Time to fix my virtualenv then...
On Wed, 17 Oct 2018 at 10:50, Marius Gedminas<EMAIL_ADDRESS>wrote:

Ah, yes, the happy fun with virtualenvs during system upgrades.
I would accept a PR for making findimports emit user-friendly warning
messages in this case (e.g. WARNING:
./localVenv/lib/python3.5/future.py is a broken symlink), but I'm not
planning on doing any work on this myself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mgedmin/findimports/issues/7#issuecomment-430564358,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHN8GR_qibfGy0i7i74hae3GUuKaB8i5ks5ulv0egaJpZM4XjXXL
.


The port has startup logs but hasn't actually started.

Run log:
INFO  server > Remote address is: <IP_ADDRESS>:12446
INFO  server > 1 cores available
DEBUG neli::socket > Message sent:
Nlmsghdr { nl_len: 64, nl_type: Newaddr, nl_flags: NlmFFlags(FlagBuffer(1, PhantomData)), nl_seq: 0, nl_pid: 0, nl_payload: Payload(Ifaddrmsg { ifa_family: Inet6, ifa_prefixlen: 128, ifa_flags: IfaFFlags(FlagBuffer(0, PhantomData)), ifa_scope: 0, ifa_index: 54, rtattrs: RtBuffer([Rtattr { rta_len: 20, rta_type: Local, rta_payload: Buffer }, Rtattr { rta_len: 20, rta_type: Address, rta_payload: Buffer }]) }) }
INFO  server       > Created TUN device tun0
INFO  server       > Listening on 4567

root@dev-hk:/etc/wireguard# RUST_LOG=info phantun_server --local 34567 --remote <IP_ADDRESS>:51810
INFO  server > Remote address is: <IP_ADDRESS>:51810
INFO  server > 4 cores available
INFO  server > Created TUN device tun0
INFO  server > Listening on 34567
netstat -anp|grep 34567 result is null
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.3 LTS
Release:        22.04
Codename:       jammy

I have same problem.

and i can't seen listen port.

Phantun does not use the Kernel TCP stack and therefore won't show up in netstat. You should use tcpdump to troubleshoot.

@dndx

Add the following command to solve the problem
iptables -A FORWARD -i tun0 -j ACCEPT
iptables -A FORWARD -o tun0 -j ACCEPT


Rename Bynder Dialog to Bynder Assets

This is a better name for listing it on the Marketplace

@lehnerm Marketplace PR is still WIP. This was the time to do this change :)


Field/model keeps invalid state after setting an external validation message

After setting an external validation message using setExternalValidation without a key (server supplies error message only), the validation message disappears after changing the field as expected. However, if you make another change to that same field, a validation error re-occurs.
I believe this behaviour comes from not properly unsetting the error from the model error collection.

Thanks for this - good work!


[ZEPPELIN-2442] button to create note into folder (navbar)

What is this PR for?
Added button to create new note into folder (navbar)
What type of PR is it?
Feature
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2442
Screenshots (if appropriate)

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

cool - perhaps + icon is a bit confusing? though we have + for new note anyway

@felixcheung  thanks for review
Yes, but unfortunately the tooltip does not work correctly in this menu. I'll think how to fix it.

@felixcheung
added tooltip

cool, love the hover and tooltip

I tested long folder name and LGTM!

LGTM and merge to master if no further discussions


doom.d directory only updates after rebuilding twice

For whatever reason whenever I change my config.el, doom.d in the Nix store does not update until I rebuild again.
nix-info -m:

system: "x86_64-darwin"
host os: Darwin 20.6.0, macOS 10.16
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.5.1
channels(bandithedoge): "darwin, home-manager, nixpkgs-22.05pre345869.46821ea01c8"
channels(root): "nixpkgs-22.05pre345869.46821ea01c8"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixpkgs

Maybe I'm missing something but why would it change without a rebuild? Rebuild syncs your local copy of doom.d (e.g. somewhere in your home directory) to the one inside the store. The rest of Emacs works with the store copy of it.

Maybe I'm missing something but why would it change without a rebuild? Rebuild syncs your local copy of doom.d (e.g. somewhere in your home directory) to the one inside the store. The rest of Emacs works with the store copy of it.

It changes only after rebuilding twice.

Oh, missed that, sorry.

That is extremely odd.. Steps taken to trigger it? Your configuration? Tested on NixOS?

This sounds a lot like a flake "gotcha" I've encountered a few times before (not specific to this particular flake). I'm guessing that OP figured this out already or took another direction, but I would also venture to guess that the path given to doomPrivateDir pointed to some location in the store, not in the flake itself. When that happens, upon rebuilding, Nix references the previous generation of the flake source, not to the current version of the files as they appear in your editor.
https://stackoverflow.com/questions/43850371/when-does-a-nix-path-type-make-it-into-the-nix-store-and-when-not/43850372#43850372

This sounds a lot like a flake "gotcha" I've encountered a few times before (not specific to this particular flake). I'm guessing that OP figured this out already or took another direction, but I would also venture to guess that the path given to doomPrivateDir pointed to some location in the store, not in the flake itself. When that happens, upon rebuilding, Nix references the previous generation of the flake source, not to the current version of the files as they appear in your editor.
https://stackoverflow.com/questions/43850371/when-does-a-nix-path-type-make-it-into-the-nix-store-and-when-not/43850372#43850372

It seems that this explains the issue. I am closing this issue.


Wasm dev backend needs a CLI option for stack size

In WebAssembly, the stack size is decided at compile-time and hard-coded in the binary. This is different from native binaries, where you can ask the OS to run your program with a particular stack size through the shell.
This means we will need a command-line option for users to control their WebAssembly stack size, in case the default doesn't work for them. Perhaps --wasm-max-stack-size. Should it be in kB?
We will also need to generate code to detect stack overflow and call roc_panic.

Resolved by #3723 adding --wasm-stack-size-kb.


Release Automation Support

This request contains followings

tidying go.sum
adding goreleaser support with make commands
adding github actions for releasing new package when new tags adds

@BatuhanKucukali please do not hasitate to contact with me to discuss about request 🤩
Cheers!

Thank you for your contribution @seckinsen :smiling_face_with_three_hearts: .


Fix ubo not of the right size when using material plugins

See https://forum.babylonjs.com/t/pluginmaterial-bug/30098
To fix the problem I simply generate a specific #define for each plugin and use this define in the vertex/fragment shaders when the plugin is used.
I have also added a visualization test to test this case.

you rock!


Prefixed JSON Values (not keys)

Hey! I'm a complete Rust n00b and it looks like Serde is the best De/Serialization SDK for Rust!
As a learning experience, I've begun writing a Rust Client library for the Radix DLT ledger. The API's uses schema-less JSON solution, where each JSON value, contains a prefix telling what kind of data follows.
Here are the JSON value prefixes implemented in the Radix DLT Java library. These constants (prefixes) are then used for various custom De/Serializers using Java Jackson, e.g. this custom JacksonJsonStringDeserializer which parses all JSON strings prefixed with the ":str:" prefix.
And here's example JSON, in short, something like this:
{
	"some_string": ":str:Hello world",
	"some_amount": ":u20:115792089237316195423570985008687907853269984665640564039457584007913129639935",
	"some_bbytes": ":byt:A3hanCWf3pmR5E+i+wtWWfKleBrDOQduLb/vcFKOSt9o",
	"some_address": ":adr:JH1P8f3znbyrDj8F4RWpix7hRkgxqHjdW2fNnKpR3v6ufXnknor",
	"some_hash": ":hsh:0000000000000000000000000000000000000000000000000000000000000001"
}

Is it possible in Serde to specify global rules / global custom serializers? I saw the page impl-deserializer, is that maybe what I wanna do?
Or ought I write custom Visitors? How can I set up global rules using those?
Sorry, really big and open question, I understand if this is asking to much help.... Any help would be much much appricated!

Hi @Sajjon, sorry that no one was able to provide help here. If this is still an issue, you could try taking this question to any of the resources shown in https://github.com/serde-rs/serde#getting-help.


Please use vX.Y.Z tags!

Instead of 1.0.0, would you please re-tag them correctly?
https://research.swtch.com/vgo-import

Hi guilt! Good catch. I've renamed the tag to v1.0.0

It works now. Closing!


KAFKA-12338: Consolidate MetadataRecordSerde and MetadataParser serial/deserial code

More detailed description of your change
The logics are duplicated except that MetadataRecordSerde has an extra DEFAULT_FRAME_VERSION, if we want to change the serial/deserial format of metadata, we should modify 2 classes, this is unreasonable.
Summary of testing strategy (including rationale)
unit test
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@cmccabe , Hi, PTAL.

Resolved by #10793


Pass lazy loading filter to sync reqs in room summary tests

This aligns correctly with the spec and is also seemingly the only reason the test is disabled on synapse, now re-enabled.

Also see https://github.com/matrix-org/complement/pull/553 and https://github.com/matrix-org/matrix-spec/issues/1325, where we came to the conclusion, for now, that it's not clear by the spec when this field should be added.
Looking at RoomSummary and the m.heroes, it states that this is required, if the room has neither m.room.name nor m.room.canonical_alias. So given a room can have neither (e.g. DMs), shouldn't Synapse include a RoomSummary, no matter if lazy loading or not?

Also see #553 and matrix-org/matrix-spec#1325, where we came to the conclusion, for now, that it's not clear by the spec when this field should be added. Looking at RoomSummary and the m.heroes, it states that this is required, if the room has neither m.room.name nor m.room.canonical_alias. So given a room can have neither (e.g. DMs), shouldn't Synapse include a RoomSummary, no matter if lazy loading or not?

The spec's a little unclear, since it doesn't directly mention when summary is required. However it makes sense for the server to send m.heroes for DMs, for example. If we take summary to be required when any of its fields are required, then Synapse is not spec compliant here.
I've filed this as https://github.com/matrix-org/synapse/issues/15017.


ensure macOS has enough mapped shared memory

as requested in https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/840
I've already run this on all the test mac's but they will need to be disconnected and reconnected in order to have the new umask.

Why the updated umask? I don't see that documented in 840 and unless there's a good reason to relax the permissions we shouldn't be doing that

@sxa555 I've change the link, it's required in https://github.com/eclipse/openj9/issues/4375

Is mode: 0777 creating a world-writable file? I don't seem to have login access to the machines to check.

I'll chekc with the OpenJ9 team whether that is a hard requirement on the machine setup as my initial feel is that anything testing that functionality could override it itself

Default permissions on the file for OpenJ9 systems appears to be  (at least I don't believe I modified the perms from the default).  There is also a create option for blockinfileso you could collapse the 2 sections into 1.-rw-r--r--  1 root  wheel  72 Jan 24  2019 /etc/sysctl.conf`
Not sure if the test setting umask is an option or not.  @hangshao0 or @Mesbah-Alam would be in a better position to answer that question.  The comments for each change should be indicative of the actual change. Update main.yml will not be helpful if you are scanning through commits looking for a problem in 6 months.

A bit background info: It is testing a command line option to create a shared cache file that is readable and writable to other users within this group. The umask is preventing such permission to be set.

Not sure if the test setting umask is an option or not.

@Mesbah-Alam, @llxia

@hanshao0 Makes sense (although presumably since the tests are running as a single user it's only verifying the permissions afterwards as opposed to utilising group write?) Do you know if there is scope for the test to set the umask itself before verifying that functionality since that would assist third parties trying to run our tests?

The JVM that creates the shared cache file will verify if the required access is successfully set or not (right after the file is created).  I guess we need the modified umask only when we are running the command that creates the shared cache file.

I think we can set umask in the test and set it back. Similar to https://github.com/eclipse/openj9/blob/master/test/functional/cmdLineTests/sigxfszHandlingTest/runSIGXFSZTest.sh
@Mesbah-Alam could you help to set umask in the systemtest? Thanks

I'll back the umask bit of out of it

@sxa555 updates PTAL

@sxa555, is this another example where tests need a specific value, then perhaps the tests should set them and fail if they can't increase it?

Even with umask 0002 set in the playlist, SharedClassesAPI test is still failing https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1873/console
from 8.WL4.stderr:
JVMSHRC336E Port layer error code = -393970
JVMSHRC337E Platform error message: shmget : Cannot allocate memory
JVMSHRC029E Not enough memory left on the system
JVMSHRC663I Error recovery: destroyed semaphore set with id=65537 associated with shared class cache.
JVMJ9VM015W Initialization error for library j9shr29(11): JVMJ9VM009E J9VMDllMain failed
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.```

It seems the machine is running low on memory.

Internal OSX machines were updated to the following values
Ref: https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/840#issuecomment-514618542

The settings in sysctl.conf I was asked to make were :
kern.sysv.shmmax=125839605760
kern.sysv.shmall=30722560

@sxa555, @gdams, what are the values set for `kern.sysv.shmmax` and `kern.sysv.shmall` in the Adopt osx machines? (e.g., in the one where the test is failing: https://ci.adoptopenjdk.net/computer/test-macincloud-macos1013-x64-1/)? Can we update them to match what we have in the internal machines?

@Mesbah-Alam It's already got those values set as I've run this change on all the test macs:
test-macincloud-macos1013-x64-1:~ admin$ cat /etc/sysctl.conf
# BEGIN ANSIBLE MANAGED BLOCK
kern.sysv.shmmax=314572800
kern.sysv.shmall=76800

Do the macs need a reboot after making the change?

based on https://serverfault.com/a/383990 it suggests that they do need a reboot, I'll add that to the playlist

re-running grinder with change on that machine: https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/1875/console

grinder passed so I assume this fixes the issue. @sxa555 can you please land and I'll deploy to the rest of the macs

Test updated to set umask 0002 when on osx: https://github.com/AdoptOpenJDK/openjdk-tests/issues/1262
@hangshao0


Code 128A?

I need to encode the string FNC3+"$P\rI", but encoding \r (carriage return) requires code 128A. Further investigation reveals this library only supports 128B and 128C. Any chance you could add 128A as well?
(FNC3+"$P\rI" is a special code to begin/end programming Datalogic-brand barcode scanners)

I will have a look at this. I think the biggest Problem will be to find the mapping for the A-Table.

I think the A table should be:
" !\"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_\x00\x01\x02\x03\x04\x05\x06\a\b\t\n\v\f\r\x0e\x0f\x10\x11\x12\x13\x14\x15\x16\x17\x18\x19\x1a\x1b\x1c\x1d\x1e"
I generated this using (and based on http://www.adams1.com/128table.html):
func c128TableA() string {
	result := []rune{}
	// first part
	for i := 32; i < 96; i++ {
		result = append(result, rune(i))
	}

	// control codes
	for i := 0; i < 31; i++ {
		result = append(result, rune(i))
	}
	return string(result)
}

func main() {
	log.Printf("%q", c128TableA());
}

According to http://www.adams1.com/128table.html it should even include \x1f ?!?
Would be great if you could provide some tests. But It will not be required :)
Gonna fix this asap.

Yes, you're right, I initially included \x1f but seemed like an off by one (comparing the length of the table already in your code).
I'm reading some of these Datalogic codes by hand.
enter/end programming = <Start A><FNC3>$P<CR>I<STOP>
restore EU factory config = <Start B><FNC3>$P,Ae,P<Code A><CR><STOP>

If you'd like I could write some tests for these codes. I've already noticed that these don't include a checksum and there's no way to disable that atm.

Tests would be great! I've just added the A encoding table, but the encode function must be changed too. Might take some time -.-

Hope this all works as expected. If not let me know.

Thanks, didn't expect you to work on this so quickly!
I can encode "$P\r" now, and an older model barcode scanner reads it, but the other codes don't register... I will look into it.


libz not found

I've been trying to DSSP program on Ubuntu 20.04, when I run the configure file I get an error saying
configure: error: libz not found - compressed files not supported.
I tried to install libz using the command sudo apt-get install -y libz-dev for which I get the following response:
Note, selecting 'zlib1g-dev' instead of 'libz-dev' zlib1g-dev is already the newest version (1:1.2.11.dfsg-2ubuntu1.2)
When the problem still persisted, i installed libz from source from here, but the problem still persists. Could anyone please help me? I am new to linux so please give responses that are a little noob friendly with distinct commands.
Thanks in advance.

The installation of libz-dev should have been enough.
Where did you get this message, when building libcif++ or when building
dssp?
Could you send me the config.log file, please?
regards, -maarten
Op 27-06-2021 om 14:28 schreef hussain-beig:

I've been trying to DSSP program on Ubuntu 20.04 when I run the
configure file i get an error saying
|configure: error: libz not found - compressed files not supported|.
I tried to install libz using the command |sudo apt-get install -y
libz-dev| for which i get the response:
|Note, selecting 'zlib1g-dev' instead of 'libz-dev' zlib1g-dev is
already the newest version (1:1.2.11.dfsg-2ubuntu1.2)|
When the problem still persisted, i installed libz from source from
here https://sortix.org/libz/, but the problem still persists. Could
anyone please help me? I am new to linux so please give responses that
are a little noob friendly with distinct commands.
Thanks in advance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PDB-REDO/dssp/issues/20, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNA47AZVLO522VTCHOIVSDTU4KNNANCNFSM47MLOFGQ.

--
Maarten L. Hekkelman
http://www.hekkelman.com/

Thanks for helping me out Maarten,
This error was when I was building dssp.
Here's the config.log file -
config.log

Thanks for sending that log file. The problem is you should have the
package libbz2-dev installed as well, should fix the problem.
regards, -maarten
Op 28-06-2021 om 17:44 schreef hussain-beig:

Thanks for helping me out Maarten,
This error was when I was building dssp.
Here's the config.log file -
config.log https://github.com/PDB-REDO/dssp/files/6727535/config.log
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PDB-REDO/dssp/issues/20#issuecomment-869794571,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNA47APAJFWM272VX5IJQTTVCKFVANCNFSM47MLOFGQ.

--
Maarten L. Hekkelman
http://www.hekkelman.com/

Yes, installing libbz2-dev fixed the problem, thank you for that. But the make command gave
fatal: not a git repository (or any of the parent directories):  .git mkdir -p obj rm -f version-info-*.txt fatal: not a git repository (or any of the parent directories): .git make: *** [GNUmakefile:110: version-info-.txt] Error 128
Some digging online for this error told me to run git init command to set up a git repository, after which running make command gave me this:
`fatal: your current branch 'master' does not have any commits yet

src/mkdssp.cpp
-> mkdssp`

I went ahead with the sudo make install step which gave me this:
fatal: your current branch 'master' does not have any commits yet install -d /usr/local/bin install mkdssp /usr/local/bin/mkdssp install -d /usr/local/share/man/man1 install -m 644 doc/mkdssp.1 /usr/local/share/man/man1/mkdssp.1; gzip /usr/local/share/man/man1/mkdssp.1;
Would this be a successful installation or do i need to do something else?

A bizarre error message, never seen that before.
But if the executable works, it is safe to use it.
In case of doubt, throw away your dssp directory, do a fresh 'git clone
https://github.com/PDB-REDO/dssp' and then './configure; make; make
install' again. Don't think that is required though.
-maarten
Op 29-06-2021 om 12:27 schreef hussain-beig:

Yes, installing libbz2-dev fixed the problem, thank you for that. But
the make command gave
|fatal: not a git repository (or any of the parent directories): .git
mkdir -p obj rm -f version-info-*.txt fatal: not a git repository (or
any of the parent directories): .git make: *** [GNUmakefile:110:
version-info-.txt] Error 128|
Some digging online for this error told me to run git init command to
set up a git repository, after which running make command gave me this:
`fatal: your current branch 'master' does not have any commits yet
    src/mkdssp.cpp
    -> mkdssp`

I went ahead with the sudo make install step which gave me this:
|fatal: your current branch 'master' does not have any commits yet
install -d /usr/local/bin install mkdssp /usr/local/bin/mkdssp install
-d /usr/local/share/man/man1 install -m 644 doc/mkdssp.1
/usr/local/share/man/man1/mkdssp.1; gzip
/usr/local/share/man/man1/mkdssp.1;|
Would this be a successful installation or do i need to do something else?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PDB-REDO/dssp/issues/20#issuecomment-870474357,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADNA47G3UAC2RYBZ6SGYGRDTVGN2HANCNFSM47MLOFGQ.

--
Maarten L. Hekkelman
http://www.hekkelman.com/

It runs without rebuilding it, although I rebuilt it and those errors did not appear.
Thanks Maarten, couldn't have done it without you.


Broken with latest fastboot

Do we just need to update fastboot to latest here or is this going to be incompatible with newer fastboot?
Stack trace:
TypeError: Cannot read property 'options' of undefined
    at Class._getFastbootTree (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli-fastboot/index.js:166:45)
    at Class.treeForPublic (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli-fastboot/index.js:175:29)
    at Class._treeFor (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/addon.js:551:33)
    at Class.treeFor (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/addon.js:511:21)
    at addons.map.addon (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/addon.js:383:34)
    at Array.map (native)
    at Class.eachAddonInvoke (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/addon.js:381:24)
    at Class.treeFor (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/addon.js:510:22)
    at project.addons.map.addon (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/broccoli/ember-app.js:557:22)
    at Array.map (native)
    at EmberApp.addonTreesFor (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/broccoli/ember-app.js:555:32)
    at EmberApp.publicTree (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/broccoli/ember-app.js:831:22)
    at EmberApp.toArray (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/broccoli/ember-app.js:1675:12)
    at EmberApp.toTree (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/broccoli/ember-app.js:1694:38)
    at module.exports (/Users/rwwagner/shipshape/shipshape.io/ember-cli-build.js:37:14)
    at Builder.setupBroccoliBuilder (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/builder.js:56:19)
    at new Builder (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/builder.js:30:10)
    at BuildTask.run (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/tasks/build.js:15:19)
    at Promise.resolve.then (/Users/rwwagner/shipshape/shipshape.io/node_modules/ember-cli/lib/models/command.js:243:46)
    at tryCatch (/Users/rwwagner/shipshape/shipshape.io/node_modules/rsvp/dist/rsvp.js:525:12)
    at invokeCallback (/Users/rwwagner/shipshape/shipshape.io/node_modules/rsvp/dist/rsvp.js:538:13)
    at /Users/rwwagner/shipshape/shipshape.io/node_modules/rsvp/dist/rsvp.js:606:14
    at flush (/Users/rwwagner/shipshape/shipshape.io/node_modules/rsvp/dist/rsvp.js:2400:5)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)

I recommend using https://github.com/ef4/prember

@tchak, thank you. The transition is surprisingly smooth.


chore: Fix some CI failures and warning

@jskeet Two separete commits for two distinct issues.

(I'm running unit and integration tests on these changes to make certain I havent missed another breakage)

Merging to run tests, I cannot do otherwise at the moment. We might find some more issues.


Torpedo Portworx driver implementation should talk to Portworx k8s service

and not a fixed node

Pushed by @adityadani


Add tests for Neuroscope dtype

Motivation
Recently discovered that when gains are set to an old RecordingExtractor, the newly refactored code for writing ElectricalSeries sometimes upcasts the data type to float due to a bug in the OldToNewRecording functions on spikeinterface. This PR adds explicit tests for this behavior and will point to a fix on SI to ensure it works.
Also attached some style suggestions for previous test written by @h-mayorquin
Checklist

[X] Have you checked our Contributing document?
[X] Have you ensured the PR description clearly describes the problem and solutions?
[X] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?

Thanks for the advice on how to test better. Looks great to me!


0.32.0 - Question about "Removed CLI block on .exe installs..."

Hello,
Regarding 0.32.0 Major Enhancements : Removed CLI block on .exe installs to instead use dependency detection allowing Windows users to use CLI programs if installed
Does this mean, for Windows Stirling-PDF.exe release users (or with-login), that we can now install for example OCRMyPDF and LibreOffice in order to have missing function like compress PDF and convert PDF from or to DOCX, etc. ?
Thanks

Correct!

Ok, so I started with my first missing dependency Ghostscript, I installed the last release Ghostscript 10.04.0 for Windows (64 bit) from the official website, it's in the Windows path as I can run gswin64 in a cmd prompt from any directory.
But after launching Stirling-PDF.exe, in the cmd log I still have the warning for missing GhostScript. Could-you tell me if I have to do another thing?
Thanks

It is checking for it to be called gs not gswin
It might need different naming for windows
This is quite a new usecase for us so might be bugs
If that is the case we can work a fix into next patch

To check, I made a copy of the file from C:\Program Files\gs\gs10.04.0\bin\gswin64.exe to C:\Program Files\gs\gs10.04.0\bin\gs.exe - And I don't see the log warning now, I don't know if the exe copy could cause other problems?
I continue with Libreoffice

Thanks for checking
I have created
https://github.com/Stirling-Tools/Stirling-PDF/issues/2216
To address gs naming, let me know for the others!

After Libreoffice installed, I had to add C:\Program Files\LibreOffice\program to system's path environment variable, now I have convert from PDF menu, tried a pdf->doc and it's ok.

Then for OCRmyPDF, I installed last release of Phyton (there is a tick at the beginning to add to path) and Tesseract (with french as additionnal language) + pip install ocrmypdf
I have the menu for OCR but I can't use it for now, as I am using french, it's written I have to check the documentation to use OCR with other languages or outside Docker.

Compress PDF is now also available and works

For unoconv, I have downloaded the source, placed on C:\unoconv-0.8.2 and added to the system path variable. No warning anymore, I have the menu File to PDF, but I tried and had error : Error:java.io.IOException: Cannot run program "unoconv": CreateProcess error=2, file missing
For OCRmyPDF, I read the docs but can't find how to make it work, I have the eng + additionnal fr language file at the right place I guess C:\Program Files\Tesseract-OCR\tessdata and added C:\Program Files\Tesseract-OCR in the path but still the message to check docs

For OpenCV, I installed with pip install opencv-python but I still have the warning.
I've just seen that I have 2 python installations because libreoffice has one built-in and I installed another separatly. The first one does not have pip, only the other. So maybe there is something to do with that...
If I launch python.exe from cmd it is the one of libreoffice, via the path system variable. The python installation added to the user path variable ...\Python313\ and ...\Python313\Scripts\ folders

For OCRmyPDF, I read the docs but can't find how to make it work, I have the eng + additionnal fr language file at the right place I guess C:\Program Files\Tesseract-OCR\tessdata and added C:\Program Files\Tesseract-OCR in the path but still only the message to check docs (no form for file input and options)

Did you also specify the path in the settings.yml?
system:
  tessdataDir: C:\\Program Files\\Tesseract-OCR\\tessdata

So to confirm does all above work without issue other than gswin64 needing changing to gs?

we need to add all this info of what you did to
https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/Getting started/Installation/Local/Windows.md
to get it published on our website etc
Would be a big help if you could write some of that down to help others

So to confirm does all above work without issue other than gswin64 needing changing to gs?

These does not work yet :
For unoconv, I have downloaded the source, placed on C:\unoconv-0.8.2 and added to the system path variable. No warning anymore, I have the menu File to PDF, but I tried and had error : Error:java.io.IOException: Cannot run program "unoconv": CreateProcess error=2, file missing
For OpenCV, I installed with pip install opencv-python but I still have the warning.
The two python as mentionned above may be a problem, but I don't know what could be the best way to solve it?
I haven't tested these for now :
Weasyprint (found the Windows release but there is some other things to install as I could see)
pdftohtml (Could-you tell me where to find this one?)

we need to add all this info of what you did to https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/Getting started/Installation/Local/Windows.md to get it published on our website etc Would be a big help if you could write some of that down in a PR to above file to help others

Yes, when everything will be running well, I would be happy to write the steps

For weasyprint, I downloaded the last release, put the weasyprint.exe in c:\weasyprint\ which I added to the path. No more warning but convert url to pdf gave me the error : java.io.IOException: Cannot run program "weasyprint": CreateProcess error=5, Access denied for path: /api/v1/convert/url/pdf

For pdftohtml, I found the executable on
Same steps, c:\pdftohtml\pdftohtml.exe + in the path, then no warning and it works

https://docs.stirlingpdf.com/Getting started/Installation/Local/Windows Installation/
added some instructions here, also have changed some things recently OCRMyPDF is not longer used only tesseract itself
gs is replaced by qpdf


race condition in TranslateService.use()

Current behavior
when TranslateService.use() is called multiple times in quick succession, language you get is determined not by order of calling the TranslateService.use() method, but by whatever http request for language json comes last
Expected behavior
Nobody should not be changing languages three times in 0,5 second, I know but it's not my fault.
How do you think that we should fix this?
Ignore this issue
Environment

ngx-translate version: 12.1.2
Angular version: 10.0.14

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: XX
- Platform:

Others:

@rattkin I came across this issue recently in a project as well. In our case, we're setting the language based on the browser's preferred language inside of our i18n module (just like documentation has in their example) and also defining a session effect that calls TranslateService.use() to apply the user's preferred language within the context of our app. The 2 calls happen within close proximity of each other and occasionally the browser's preferred language is the one that gets applied. Glad to know it's not just an issue with the way we're loading translations.

Upon a closer look, I think the same issue might exist for setting the default language.


added POST-BUILD scripts: email and shell scripts

@mareknovotny this PR should changes the kie_jenkinsScripts_PR.groovy  to work with the correct emails sendings. When this works - this changes will be transferred to all other PRs

Jenkins please retest this


Models are sometimes lost

This has happened before, and I don't know what causes it. (I'm using autogen and litellm)
ollama stalled, with my application and ended up with a timeout error.
I realized there were some defunct processes so I syscontrol stop ollama and killed every instance of ollama I could find.
Upon starting it again, it didn't have the models.
The last time it happened, I believe a reboot brought everything back.

This is unexpected, thanks for reporting. It looks like you are on Linux. When Ollama fails with the timeout and you restart it are you starting ollama serve or the systemctl service? Ollama runs in different contexts between those two scenarios (as the current user when using ollama server and as an ollama user when run by systemctl). So it is possible it is looking at different model directories in those cases. Next time it happens take a if you're running ollama the same way when it is restarted.
As an aside, I'll try to reproduce the timeout.

I am on linux, I believe this time I used systemctl

If you switched between ollama serve and systemctl your models will definitely be in two separate places. You can use the OLLAMA_MODELS env variable to point to the correct place and your models should show up again.
I'm going to go ahead and close the issue, but feel free to keep commenting or reopen it if you're still having problems.

Sometimes lost also in Win


Does InfVerif support Recursive mode?

Does InfVerif support Recursive mode to automatically go through INFs under all sub-folders?

@guliteb. today I will  check on my system

I have checked recursive mode does not support

@RAJU2529 , Thanks for reply. Is it going to be added to backlog as new feature?

@guliteb. Sorry I dont know that, I think infverif application desinger must and should consider this feature to add.

Added description for the /recurse parameter. It's been in the tool for a while, but it got left out of the list in infverif /?
Added to master branch and should be live shortly.
Eliot


ISPN-10329 Remote listener comments

Change to add Future in Event
Complete event in CPU thread pool instead of netty
Remove check for sync listener

https://issues.jboss.org/browse/ISPN-10329

Integrated, thanks Will!


UTXO balance is too low to cover fee - why is native segwit balance not utilized?

A user has this error "UTXO balance is too low to cover fee" when they try to send an Ordinal inscription.
But we expect transfers to be covered by native segwit balance. There is about 60 $ worth of bitcoin in that BTC-address in Account 1 with the ordinal, which should be enough to cover current fees.

The ordinal: #622850
Is on this 331 sat output: 1bddcc4e55833493db3f64db81bce818ae914854ec4ca77ce1867888e7d77c98:0, is that below the dust limit?
See also several other 331 outputs with inscriptions in them on this address that suffer the same issue: https://mempool.space/address/bc1ps6segtmyh63ny5lmvap05pnkptyuvuklc7jhe6srkwlcdyxfpehqpa0m68
This is non-primary address by the way: m/86'/0'/0'/0/4 (not sure if it is relevant)

bc1q3tnkvm4rf89er9skns54pjrgknc2jw0a8a8wnf

Have we confirmed the user is on the most recent version / one that does indeed take fees from the regular balance?


CMake inserting unwanted quote characters

Expected Behavior
MPI Configurations should work fine.
Actual Behavior
When the MPI library has more than one flag defined, the MPI flags are added to CXX_FLAGS with quotes around them.  For example, the openmpi build I am using defines MPI_CXX_COMPILE_OPTIONS=-fexceptions;-pthread. When that gets added to the CXX_FLAGS, it is added with quotes (and fails to compile).  For example, plugins/parcelport/mpi/CMakeFiles/parcelport_mpi.dir/flags.make looks like this:
CXX_FLAGS = -fPIC -O3 -DNDEBUG -fPIC   -std=c++17 "-fexceptions -pthread" -pthread -Wall -Wextra -Wno-unused-local-typedefs -Wno-strict-aliasing -Wno-sign-promo -Wno-attributes -Wno-cast-align -Wno-unused-parameter -Wformat=2 -Wno-format-nonliteral -Winit-self -Wdouble-promotion -Wcast-qual -Wcast-align -Werror=trampolines -Werror=parentheses -Werror=reorder -Werror=return-type -Werror=sequence-point -Werror=uninitialized -Werror=format -Werror=missing-braces -Werror=sign-compare -fdiagnostics-show-option -Werror=vla -Wno-unused-but-set-parameter -Wno-unused-but-set-variable -Wno-maybe-uninitialized -Wno-sync-nand -Wno-delete-non-virtual-dtor -fvisibility=hidden

Which generates this compiler error:
g++: error: unrecognized command line option '-fexceptions -pthread'; did you mean '-fexceptions'?
make[3]: *** [plugins/parcelport/mpi/CMakeFiles/parcelport_mpi.dir/mpi_environment.cpp.o] Error 1
make[3]: *** Waiting for unfinished jobs....

Steps to Reproduce the Problem

Configure HPX with MPI enabled, ideally one with more than one compiler flag.  For example, in openmpi, you can change the pkgconfig files to have: Cflags: -I${includedir}     -fexceptions -pthread
After configuring HPX, see that the flags have quotes around them, i.e. in src/CMakeFiles/hpx.dir/flags.make or plugins/parcelport/mpi/CMakeFiles/parcelport_mpi.dir/flags.make

Specifications
... Please describe your environment

HPX Version:  current master
Platform (compiler, OS):  linux, x86_64, gcc 7.3

...I think this extra flag is coming because OpenMPI is built with C++ exception support.

What version of CMake are you using? And did you actually mean MPI_CXX_COMPILE_FLAGS? We are still using the (deprecated) MPI_CXX_COMPILE_FLAGS variable, and not MPI_CXX_COMPILE_OPTIONS when setting up the hpx::mpi target. I tried manually setting INTERFACE_COMPILE_OPTIONS to "-pthread;-fexceptions" here: https://github.com/STEllAR-GROUP/hpx/blob/fe37ef4c7fd6f35ae96dacf1841ecb39c629e301/cmake/HPX_SetupMPI.cmake#L50, but it expands correctly to two separate options. As far as I can tell that's the only place where we get the MPI flags.

I did mean MPI_CXX_COMPILE_FLAGS, yes.  I am using OpenMPI built by Spack, which enables C++ exception support by default.  I'll try testing it some more.

Hi,
it seems that some similar error appears at another place when compiling HPX with mpich [0] on Fedora. However, using openmpi, does not show this error. I think it is might relate to a newer CMake version, since this error did not appear a few months ago on Fedora's build system.
[0] https://kojipkgs.fedoraproject.org//work/tasks/1001/47271001/build.log

I was able to work around the problem by building openmpi without --enable-cxx-exceptions.  With that flag, the pkgconfig file defines Cflags values, and CMake 15 is putting quotes around them.


Issue with number of classes

If I change number of classes to train, do I need to change it in load_weights.py? After changing in load_weights.py it is showing error of classes didn't match. Thanks in advance @bonlime

You don't need to change anything in load_weights. Just instantiate the model with classes={your number here} as shown in the README

ok thank you


re-enable arangosearch scoring tests in cluster

Please note that for legal reasons we require you to sign the
Contributor Agreement
before we can accept your pull requests.

LGTM

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2368/


Property 'helperService' is used before its initialization

In charts.page.ts, in the class "ChartPage" , "color: this.helperService.getColorVariable('medium')"  indicate error:  Property 'helperService' is used before its initialization.ts(2729)
charts.page.ts(202, 5): 'helperService' is declared here. I've tried to find a solution with no result..
As a result, the graph is not displayed. Here's the message in the console/
core.mjs:8506 ERROR Error: Uncaught (in promise): TypeError: Cannot read properties of undefined (reading 'getColorVariable')
TypeError: Cannot read properties of undefined (reading 'getColorVariable')
at new ChartsPage (charts.page.ts:27:37)
at NodeInjectorFactory.ChartsPage_Factory [as factory] (ɵfac.js:1:1)
at getNodeInjectable (core.mjs:3485:44)
at createRootComponent (core.mjs:12391:35)
at ComponentFactory.create (core.mjs:12272:25)
at ViewContainerRef.createComponent (core.mjs:21416:47)
at IonRouterOutlet.activateWith (ionic-angular.js:3488:57)
at ActivateRoutes.activateRoutes (router.mjs:3048:40)
at router.mjs:2997:18
at Array.forEach ()
at resolvePromise (zone.js:1265:35)
at resolvePromise (zone.js:1219:21)
at zone.js:1332:21
at _ZoneDelegate.invokeTask (zone.js:446:35)
at core.mjs:23999:55
at AsyncStackTaggingZoneSpec.onInvokeTask (core.mjs:23999:36)
at _ZoneDelegate.invokeTask (zone.js:445:64)
at Object.onInvokeTask (core.mjs:24300:33)
at _ZoneDelegate.invokeTask (zone.js:445:64)
at Zone.runTask (zone.js:214:51)

I will rework it to a new version, and this issue will no longer exist, thank you.


cpp: FPs in Microsoft.SAL

We noticed false positives when testing SAL annotations.
These three should demonstrate the issue:
sal.h:
#define _SAL_VERSION 20

test.cpp:
#include "sal.h"
int method1() {
    return _SAL_VERSION;
}
void method2();

test.ql:
import Microsoft.SAL
from SALAnnotation a
select a, a.getDeclaration()

Output from the query:

| test.cpp:3:12:3:23 | _SAL_VERSION | test.cpp:5:6:5:12 | method2 |

It seems the detection is based on "rank" and the fact that MacroInvocation _SAL_VERSION is before DeclarationEntry method2.
Not sure what is the best fix, can you please advise? Thank you.

I'll look into this Monday and get back to you.

Thanks for the report. I'm working on a fix that I'll try to get out this week. Essentially, I want to break the association of SAL elements not only at start positions of declaration entries but also at end positions of their bodies.

I've opened a PR for the fix in the Microsoft-internal repo. When that's merged, it can migrate out to this repo.

@tosmolka it's been a year since you've reported this issue. Does the false positive still persist or was it fixed?

The fix has finally made it back to this repo (in #2151).


[tests-only] Bump core commit id for tests

issue https://github.com/owncloud/QA/issues/662

looks like lines need to be adjusted https://drone.cernbox.cern.ch/cs3org/reva/1445/7/6


Feature Request: Multiple color values allowed per state

I would like to be able to generate tilegrams visualizing state subpopulation breakdown, e.g.,

racial/ethnic
religious
votes cast for various US presidential candidates, by state

I'm envisioning a tilegram with clearly marked state boundaries, allowing different colored hexagons within the state. I'd appreciate advice on how to accomplish this.

We haven't really thought about this as a use case. I think your best bet would be to load the svg and then color the hexagons using d3 or javascript. The manual has an example of loading a svg.
It's a bit trickier to accomplish this with the topojson as the states are drawn as a single path. Perhaps in the future we could think about exporting a topojson file that has separate features for each hexagon rather than each state.


gd(go to definetion) jumped twice,  the first time is correct, but the the second time jump to uncorrect location

VSCode Version: 1.35.1
OS Version: windows 7

Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes/No

with VscodeVim extension

@xxsong5 can you reproduce this issue when running Go To Definition from the command palette?


Update ed25519-compact dependency

v2.0 of ed25519-compact supports conversion between key types used in X25519 and Ed25519, where X25519 is required for a successful noise handshake

Rebased and merged in 2a6f3c4f556ed25008889587f621bff86711a70d


Merge live to master

:(

Docs Build status updates of commit 1aa3588:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/converters/multiconverter.md
:white_check_mark:Succeeded
View

docs/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1aa3588:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/converters/multiconverter.md
:white_check_mark:Succeeded
View

docs/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Error No module named subtitles_generator

C:\Users\mayco\Downloads\subtitles-generator-main>python main.py large portuguese .\videoyt.mp4
Traceback (most recent call last):
File "C:\Users\mayco\Downloads\subtitles-generator-main\main.py", line 13, in
from subtitles_generator.core import Model
ModuleNotFoundError: No module named 'subtitles_generator'

Hi
Currently, I do not distribute this package via pip. You are supposed to write pip install . from the main directory. It is noted here:
https://github.com/konverner/subtitles-generator/blob/main/notebooks/subtitles_generator.ipynb
So, just replace your pip install subtitles_generator with pip install . .
I will add this remark to README. Thanks.


Maybe this will really work correctly

coverage: 93.432%. remained the same
when pulling<PHONE_NUMBER>8269108fc91edb5c9fd54684c749e3 on njw-deploy-fix-2
into 846aecd3af666218a9f4e12b2b52031185d02576 on main.


Warning if tests fail that if you need to generate fixtures.

If the tests fail (yarn run test-only), warn the user that you need to run a command that will delete the expected.json so that the tests pass.
Proposal
Error
$ babylon(master): yarn run test-only

yarn run v0.27.5
$ ava

  4240 passed
  1 failed
  112 skipped

  index › flow/typecasts/yield

  /Users/user/Developer/babylon/clean_up/babylon/test/helpers/runFixtureTests.js:21

   20:               name + "/" + task.actual.filename + ": " + er…
   21:             t.fail(message);
   22:           }

  flow/typecasts/yield/actual.js: [] != undefined (comments)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

After an error like this warn user that they may need to delete expected.json, or have a command for doing this.
Command
The command for fixing this could be something like babylon-remove-broken-expected-json.  It could be an alias that would just remove all failing expected.json files.

[ ] Create Alias to remove all failing expected.json files
[ ] Add suggestion to error message
[ ] Update Documentation

Hey @pudility! We really appreciate you taking the time to report an issue. The collaborators
on this project attempt to help as many people as possible, but we're a limited number of volunteers,
so it's possible this won't be addressed swiftly.
If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack
community that typically always has someone willing to help. You can sign-up here
for an invite.


fix: Misaligned Menu in login profile table on smaller screens

Fixes #1773
This PR adds a useFlipMenu hook with logic to automatically position Menu component, depending on its top / bottom position on the screen.
How to reproduce

Go to this link.
Log in.
Go to My visualizations.
Click on an action button close to the bottom of the screen.
❌ See that it's rendered in the middle, overlapping the button, with arrow not being correctly positioned (in the screenshot below, I clicked on the last row's button).

How to test
Once the PR is deployed to TEST, it will be possible to test using the above instructions, and the result should look like below. Clicking on top items should still display the component in its default, top-to-bottom state.

LGTM 👍, was there any issues when testing for smaller screen sizes?

Thanks for reviewing @noahonyejese! Yes, as the Menu doesn't "flip" on its own, it was always aligned with the bottom of the screen, covering other items in the result (also the top arrow pointing nowhere looked weird 😅).
With this change, we keep the old behavior when there's enough space to render the whole menu, otherwise we flip it's anchor origin 👍


TypeError: 'numpy.int64' object is not iterable (ica_aroma_confound_extraction)

As per https://neurostars.org/t/ica-aroma-error-in-fmriprep-mask-and-image-must-be-the-same-size/1605/4
[Node] Running "ica_aroma_confound_extraction" ("fmriprep.interfaces.confounds.ICAConfounds")
180502-07:17:39,469 workflow WARNING:
	 [Node] Error on "fmriprep_wf.single_subject_104_wf.func_preproc_task_rest_run_08_wf.ica_aroma_wf.ica_aroma_confound_extraction" (/root/src/fmriprep/work/fmriprep_wf/single_subject_104_wf/func_preproc_task_rest_run_08_wf/ica_aroma_wf/ica_aroma_confound_extraction)
Traceback (most recent call last):
  File "/usr/local/miniconda/bin/fmriprep", line 11, in <module>
    load_entry_point('fmriprep==1.0.12.dev0', 'console_scripts', 'fmriprep')()
  File "/usr/local/miniconda/lib/python3.6/site-packages/fmriprep/cli/run.py", line 280, in main
    fmriprep_wf.run(**plugin_settings)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/workflows.py", line 595, in run
    runner.run(execgraph, updatehash=updatehash, config=self.config)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/plugins/base.py", line 168, in run
    self._clean_queue(jobid, graph, result=result))
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/plugins/base.py", line 227, in _clean_queue
    raise RuntimeError("".join(result['traceback']))
RuntimeError: Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/plugins/multiproc.py", line 68, in run_node
    result['result'] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 479, in run
    result = self._run_interface(execute=True)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 563, in _run_interface
    return self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/pipeline/engine/nodes.py", line 643, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/usr/local/miniconda/lib/python3.6/site-packages/niworkflows/nipype/interfaces/base/core.py", line 516, in run
    runtime = self._run_interface(runtime)
  File "/usr/local/miniconda/lib/python3.6/site-packages/fmriprep/interfaces/confounds.py", line 112, in _run_interface
    self.inputs.in_directory, newpath=runtime.cwd)
  File "/usr/local/miniconda/lib/python3.6/site-packages/fmriprep/interfaces/confounds.py", line 239, in _get_ica_confounds
    aggr_confounds = np.asarray([melodic_mix_arr.T[x] for x in motion_ic_indices])
TypeError: 'numpy.int64' object is not iterable

This is almost certainly that np.loadtxt is seeing a file with one element, and loading it as a scalar instead of a vector of length 1.
https://github.com/poldracklab/fmriprep/blob/a5d4b3689974b24c297cb3a0a7f434892aef6710/fmriprep/interfaces/confounds.py#L222

I think if we set ndmin=1, that should resolve this.

Would that make sense overall? Meaning, is it possible that motion_ics only has one index?

If it can have zero, why not one?


LP-304 Implement GET Endpoint to retrieve submission resource

JIRA link
https://companieshouse.atlassian.net/browse/LP-304
Change description
Implement GET endpoint
Work checklist

[ ] Bug fix
[X] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here
Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found


Question: Themes Related

So, on the ReadMe.md file, "Themes" are "coming soon". Will these be pre-built sites using the framework?

I'm pretty sure this project has been abandoned.

Sad


Feature/ssm agent

PR o'clock
Description
This pull requests implements ssm agent support as described here:
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/224
Checklist

[ ] terraform fmt and terraform validate both work from the root and examples/eks_test_fixture directories (look in CI for an example)
[ ] Tests for the changes have been added and passing (for bug fixes/features)
[ ] Test results are pasted in this PR (in lieu of CI)
[x] I've added my change to CHANGELOG.md
[ ] Any breaking changes are highlighted above

I'm convinced all users of this module would be happy to have this enabled by default on every cluster. What do other people think?

I'm not convinced all users of this module would be happy to have this enabled by default on every cluster. What do other people think?

I think this is the preferred solution from AWS instead off ssh keys which should be disabled by default.

OK but if AWS themselves are reluctant to put this in by default and it's very easy to add this to your worker nodes currently (that's what pre_userdata and iam_role_id are for) then it seems a little heavy handed to install and run an extra service for all users of this module.
Maybe other people can chime in.

I agree with @max-rocket-internet - it should be optional and pre_userdata is a fine place to specify - if my vote means anything. It is not really related to this module nor necessary to run an EKS cluster.

Ok I guess this feature is not needed by all users and will therefore close this pull request


Rename to glyphsLib

cc @schriftgestalt @behdad

thanks James, it looks good to me :+1:

after this we shall also rename the git repository name. GitHub is clever enough to redirect from the old url to the new one.


Refactored SchemaGenerator into multiple modules with individual tests.

See #78
This is a work in progress.

Done. Looking for feedback.
This should be merged after #77 (merge conflicts).

This contains no breaking API change, it's purely internal.

Awesome! This is exactly what I was working towards earlier with some of the clean ups but didn't have time to get to it. I'm glad you saw the same patterns.  👍
You have a few conflicts now but this looks good to me otherwise

Nice, just some finishing another PR+release, then I'll fix the conflicts here.
I was a bit unsure where to place the objectFromReflection  stuff whether TypeBuilder or SchemaGenerator would be best, I think the current solution is okay.
I also tried a bit more abstraction by removing the coupling between SchemaGenerator and TypeBuilder implementations via interfaces, but it ended up just being bloat and no advantage to testing, which works fine like it is now. mockk is great :+1:

lgtm :smile:
For good measure I've compiled it against the example app too, works.


Support blendTo and reset in viewportFeatures features

To be consistent with Interactivity returned features.
As reported by @IagoLast

Isn't this solved via https://github.com/CartoDB/carto-vl/pull/1085?

It is


Radio Button Documentation

Aug 21 - word doc in progress, catch up after call today

@jpmorganchase/salt-content this is ready for review.

Aug 22 - Dev PR up, ready for final eyes/updates

Aug 24 - ongoing pr

build ready for final design review

Aug 25 - PR to get dev reviews

Aug 29 - pushing on dev reviews > merge after

PR merged


Fixed #33902 -- Fixed Meta.constraints validation crash with F() expressions.

ticket-33902
Thanks Adam Zahradník for the report.
Bug in 667105877e6723c6985399803a364848891513cc.

@gardenerik Does it work for you?

Thanks both :star2:


Update ARI

Updated ARI information and included IPFS hosted logo

Thanks for your submission @carsenk !
Couple comments:

Please remove the edits to tokenlist.json — this file auto-updates and does not accept manual updates.
We recommend making the description more concise and informative about what exactly ARI does. Right now you provide the supply cap, but users would still have no idea what it is that ARI is used for. You can remove this phrase as well, as all tokens in Dharma are ERC20s on Ethereum: "is an ERC20 token powered by the Ethereum blockchain."

Fixed.

Thanks for making those updates @carsenk! This has been merged 🙏

Thanks for making those updates @carsenk! This has been merged 🙏

Does the metadata.json have an option for the logoURI as well? Also wondering why it is showing Unverified in app still? Does it take sometime for it to propagate? Thanks!


fltk_images lib missing on OSX El Capitan

After brew install fltk --devel and doing stack build in the fltkhs-hello-world project, I get
    *** Linking libfltkc.a...
    /Library/Developer/CommandLineTools/usr/bin/ranlib: file: ../c-lib/libfltkc.a(fl_utf8C.o) has no symbols
    *** Linking libfltkc-dyn.dylib...
    ld: library not found for -lfltk_images
    clang: error: linker command failed with exit code 1 (use -v to see invocation)

http://hackage.haskell.org/package/fltkhs-<IP_ADDRESS>/docs/Graphics-UI-FLTK-LowLevel-FLTKHS.html#g:4 mentions "fltk_images" very briefly, but I don't see it in homebrew.

I ran into this problem last night and updated the fltkhs-hello-world repo yesterday. Can you confirm that the latest commit is Up FLTKHS Dependency. If not, could you git pull and try again?
The issue is that on El Capitan for some reason the linker does not look in /usr/local/lib which is where brew sticks libfltk_images. I'm not sure why this is the case but adding -L/usr/local/lib to the Makefile made the problem go away.

Can you confirm that the latest commit is Up FLTKHS Dependency.

Yes, it is.

Hmm ... I have brand new El Capitan machine and removed and installed fltk using brew install --devel fltk and successfully installed the example.
Shot in the dark, in your stack.yaml file under extra-deps, it does list fltkhs-<IP_ADDRESS> and not fltkhs-<IP_ADDRESS> or some other version right?

I have also updated the Github README with better instructions. Thanks for that feedback!

The README seems much clearer. Unlike those instructions, here fltk-config --version reports 1.3.3. I have fltk-1.3.3-r10866.tar.gz. I ran brew update to be sure I had the latest from homebrew.

Oh, and thank you, for the valuable and not-so-fun work on making GUIs practical in Haskell. I'm glad these quick reports are helpful.

Here's a shell session where I uninstall FLTK and re-install. I can only imagine that somehow your fltk-config is a hold-over from the time you tried to install from source. Can't imagine why that would be ...
$ brew --version
Homebrew 0.9.5 (git revision 01ef; last commit 2016-02-25)
$ brew update
Already up-to-date.

$ brew uninstall fltk
Uninstalling /usr/local/Cellar/fltk/1.3.3-r10866... (322 files, 5M)
$ fltk-config --version
-bash: /usr/local/bin/fltk-config: No such file or directory

$ brew install --devel fltk
==> Downloading http://fltk.org/pub/fltk/snapshots/fltk-1.3.x-r10866.tar.gz
Already downloaded: /Library/Caches/Homebrew/fltk-1.3.3-r10866.tar.gz
==> ./autogen.sh
==> ./configure --prefix=/usr/local/Cellar/fltk/1.3.3-r10866 --enable-threads --enable-shared
==> make install
/usr/local/Cellar/fltk/1.3.3-r10866: 322 files, 5M, built in 48 seconds
$ ls -alh /usr/local/lib/libfltk_images.
libfltk_images.1.3.dylib  libfltk_images.a          libfltk_images.dylib
$ ls -alh /usr/local/lib/libfltk_images.*
lrwxr-xr-x  1 deech  admin    56B Feb 25 15:16 /usr/local/lib/libfltk_images.1.3.dylib -> ../Cellar/fltk/1.3.3-r10866/lib/libfltk_images.1.3.dylib
lrwxr-xr-x  1 deech  admin    48B Feb 25 15:16 /usr/local/lib/libfltk_images.a -> ../Cellar/fltk/1.3.3-r10866/lib/libfltk_images.a
lrwxr-xr-x  1 deech  admin    52B Feb 25 15:16 /usr/local/lib/libfltk_images.dylib -> ../Cellar/fltk/1.3.3-r10866/lib/libfltk_images.dylib

$ fltk-config --version
1.3.4
$ which fltk-config
/usr/local/bin/fltk-config

Aha! I didn't notice that brew install was reporting failure to link a bunch of files (the list was so long). After brew link --overwrite fltk, all is well! stack build succeeds.

Yay! I'll close your other issues as well. Thanks again for bearing with me.


use only JSON types

JSON has only integer and float types for numbers, so I think we should stick to these in the specification for messages. Types like long or unsigned integer has no meaning in JSON.
We explicitly state the allowed range anyways.

This is regards to ECMA standard
From ECMA-404v (2017)
JSON is agnostic about the semantics of numbers. In any programming language, there can be a variety of
number types of various capacities and complements, fixed or floating, binary or decimal. That can make
interchange between different programming languages difficult. JSON instead offers only the representation of
numbers that humans use: a sequence of digits. All programming languages know how to make sense of digit
sequences even if they disagree on internal representations. That is enough to allow interchange.

You're right. JSON has only a general number type, integer and string is not defined as types, but of course you can send bot integers and floats.
JSON Schema has integer and number: https://json-schema.org/understanding-json-schema/reference/numeric.html. So perhaps it would make sense to use these two in the spec. Note that 1.0 is allowed as in integer (not not 1.5).

also relevant for core


[FIX] Correctly determine when to set SendMax (RLJS-272)

SendMax should not be set when the source and destination currencies
are the same and the issuers are "any that the source is able to use
and the destination is able to receive"
e.g.,

   source_amount: {
      value: '0.0003272934271954',
      currency: 'GWD',
      issuer: ''
   },
   destination_amount: {
      value: '0.0003272934271954',
      currency: 'GWD',
      issuer: 'r4p4gZaWSq8Cs1d8mn1jaGqVU1HUns1ek3'
   }

SendMax documentation:
https://ripple.com/build/transactions/#special-issuer-values-for-sendmax-and-amount

Can this PR get some code review :heart: ?

Updated to fix lint errors

LGTM

LGTM


[WIP] Add before/after command hooks

Overview
⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠  BREAKING CHANGE ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠
This makes it impossible to override execute to pre/post process input tuples, and makes execute a true implementation detail in adapters. This approach turned out to be problematic as we had to create additional command classes, configuring them was awkward, and it was inflexible. Instead of overidding execute this PR introduces a new concept of before/after hooks that are applied with correct arguments and in correct order.
This addition means two things:

there's no longer any need to override execute (which was private anyway) in order to pre- or post-process input or result tuples
hooks are based on class-level pre-configured options, or instance-level options that can be used at run-time
initially we're going to support hooks as a list of command method names, and additional arguments can be provided for things like associations (explicit assoc name, or join keys etc.)

This feature is mostly for rom-sql, as its associates plugin has been difficult to use as it requires creating special command classes with overridden execute, which was an overkill; however, I'm pretty sure many other adapters can benefit from this.
Hook types

before is executed using curried args and call args, this allows us to prepare curried tuples based on additional args, the result of applying this hook is passed directly to execute method
after is applied to the result of execute method, and whatever is returned becomes the actual command result, either a single tuple or an array with 1 or more tuples

New API
# you can preconfigure a command class to always apply hooks
class CreateTask < ROM::Commands::Create[:memory]
  result :one
  relation :users
  before :associate_user

  def associate_user(task, user)
    task.merge(user_id: user[:id])
  end
end

# call with curried task and a user
create_task.with(title: "Task 1").call(id: 1, name: "Jane")

The actual cool part, and one of the most important reasons why this feature was created, is to be able to ask for a command with hooks dynamically at run-time. This will allow us to prepare all types of commands with additional tuple processing in places like repositories, without having to create special command classes:
# this is the same as the above, but at runtime
class CreateTask < ROM::Commands::Create[:memory]
  result :one
  relation :users

  def associate_user(task, user)
    task.merge(user_id: user[:id])
  end
end

# call with curried task and a user
create_task.before(:associate_user).with(title: "Task 1").call(id: 1, name: "Jane")

TODO

[x] add before and after command hooks
[x] update graph to use hook-based command calls
[ ] add ability to provide additional args for hooks, ie before(associate: :user) where :user is an association name
[ ] add Command#before and Command#after
[ ] port associates rom-sql plugin to use this new API to make sure it does the job

@timriley @flash-gordon any thoughts on this one?

From your description, this sounds really good. Having to override execute is user-unfriendly and inflexible. I like the ability to extend a command with hooks at runtime too.

On 5 Jan 2017, at 10:57 pm, Piotr Solnica<EMAIL_ADDRESS>wrote:
@timriley @flash-gordon any thoughts on this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@timriley cool, FWIW I wanted to do this in rom 2.0.0 but ran out of time O_o but now it's a must-have because repos need this badly


webvr-polyfill.js in webvr samples seems contain both compression version and full version

https://github.com/toji/webvr.info/blob/master/samples/js/third-party/webvr-polyfill.js
It seems that the first line is the compression version and the other  contents are full version.
I find this because there are two Android WakeLock created.

I believe this has been fixed in #85


Broken tutorial on live microbit site

Eddystone beacon

@pelikhan, Is this fixed?

No

I thought you were trying to fix this other day. Alright will keep it open

fixed


Rewrite Data::Record based on statically annotated metadata

Need to log these changes somewhere where a diff is immediately visible.

MetamodelX::RecordHOW and MetamodelX::RecordTemplateHOW, the root of the library, are more or less scrapped and rebuilt from scratch. They are now metaroles defined in terms of annotations' own MetamodelX::AnnotationHOW metarole, which provides support for ANN as a means of storing "meta-attributes". Subtypes of Metamodel::ClassHOW lacks any means of providing values for these beyond setters at this point in time due to BUILDPLAN interactions between Raku and nqp not working out so hot given a Scalar container. Setters are dropped in favor of constructor arguments because ANN is less strict about who sets a value where. This is OK nonetheless because it being a positional buffer, we get a fixed position that, albeit still possible, is more difficult to mutate than before.
MetamodelX::RecordHOW and MetamodelX::RecordTemplateHOW expose new is_anonymous and anonymous_id metamethods. These are available on any <@ @>/[@ @]/{@ @} type.
A Data::Record::Mode enumeration is introduced with WRAP, CONSUME, SUBSUME, COERCE exports by default.
Data::Record::Tuple, Data::Record::List, and Data::Record::Map gain a new new multi accepting a Data::Record::Mode:D :$mode. This is to be the default, with the remaining multis becoming CALL-ME, just now's not quite the right time.
A Data::Record::Lifter class backing an &infix:<@~~> operator takes the place of the protected Data::Record::Instance!field-op method that, while private, is exposed. It remains as-is for now, but will be deprecated. As opposed to the private method, this will emit a CX::Rest control exception when a typecheck fails. This changes the semantics of a typecheck when records are nested within one another. Now the innermost record type's semantics "win" in a typecheck, which is to say a wrapper record's laxity could overrule an inner record's strictness before. This should allow for more granular typecheck errors and less complicated debugging of said errors.
Internal iterators delegate to a collection of typecheck-ish metamethods on Data::Record::Tuple, Data::Record::List, Data::Record::Map now, which is where a control exception on &infix:<@~~> would typically be produced. These have a naming convention relevant when debugging the new, noisier errors:

map maps values to fields
keep keeps fields lacking values
drop drops values lacking fields
it appended to any of these indicates that an iterator should be the source of any such call. Assumptions are made about valid control exceptions to emit based on this (Data::Record::List.^drop_it_again emits next, for example).

Treat such metamethods like methods of MetamodelX before (subject to change).
All of this may come at a performance deficit, though small enough to be difficult to benchmark with any accuracy in any scenario I try (<5% loss), and with the benefit of a more lawful typecheck.

The largest performance penalties appear in the construction of records, but I had been benchmarking purely with eager data structures. Lazy lists appear to enjoy these changes, to me indicating the penalties are originating from the various iterators. These used to carry heavily inlined logic, but this has since been abstracted out to smaller metamethods, making them a little more maintainable, but making for runtime calls nonetheless. Errors need any help they can get, which giving these pervasive chunks of logic names helps with. I'll take the hit on the routines that run once per instance anyhow.


Another another sponsor please!

Promo Cymru are a Bronze sponsor, so the same size as Nexxer and Third Sector Lab, and linking to https://www.promo.cymru/

closed by https://github.com/neontribe/UK-Charity-Camp/pull/91


Update to latest version (node 0.12)

node 0.12 has path.posix and path.win32, so it'd be nice to have that here
copied verbatim from https://raw.githubusercontent.com/joyent/node/master/lib/path.js

actually, it seems this is probably not very useful. It looks like you can't even use libraries like this anymore, node doesn't want to load them as they cause a conflict with the internal version, and from testing it looks like the 0.12 version uses APIs only found in 0.12 (I wanted to use this in 0.10)
so....closing this.


Use the same skin as in other diagrams

Description
Use the same skin as in other diagrams
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[x] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[ ] CI configuration change
[ ] Konflux configuration change

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.99%. Comparing base (4cb4993) to head (517c3a5).
Report is 10 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1866   +/-   ##
=======================================
  Coverage   93.99%   93.99%
=======================================
  Files          62       62
  Lines        2830     2830
=======================================
  Hits         2660     2660
  Misses        170      170

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"
/approve

/lgtm

/override "ci/prow/unit"


We don't warn people about drop-down menus

Those input types don't work well with Docassemble, but it looks like they still come across as "/Tx" type when get_fields() is run. We should figure out a way to flag those fields.

This is fixed for radio buttons now: see https://github.com/jhpyle/docassemble/pull/619, and https://github.com/SuffolkLITLab/docassemble-ALWeaver/issues/75.
Drop-downs I'm not sure about though. Going to edit the title to reflect that.

Docassemble will error on dropdowns when trying to fill the PDF;
  File "/usr/share/docassemble/local3.10/lib/python3.10/site-packages/docassemble/base/parse.py", line 6693, in finalize_attachment
    the_pdf_file = docassemble.base.pdftk.fill_template(attachment['options']['pdf_template_file'].path(the_user_dict=the_user_dict), data_strings=result['data_strings'], images=result['images'], editable=result['editable'], pdfa=result['convert_to_pdf_a'], password=result['password'], template_password=result['template_password'], default_export_value=default_export_value)
  File "/usr/share/docassemble/local3.10/lib/python3.10/site-packages/docassemble/base/pdftk.py", line 293, in fill_template
    opt_list = [str(item) for item in annot.Opt]
  File "/usr/share/docassemble/local3.10/lib/python3.10/site-packages/docassemble/base/pdftk.py", line 293, in
    opt_list = [str(item) for item in annot.Opt]
NotImplementedError: don't know how to __str__ this object

The actual PDF has a dropdown as the first field, and as allowed by the PDF spec, each option is an array of two strings, pikepdf.Array([ "SELECT COUNTY", "SELECT COUNTY" ]), but pikepdf errors when you try to call str on that array. It needs a patch upstream to handle choice type fields specifically.

Can you attach a sample pdf


TOC Candidate Nomination for Beyhan Veli

Candidate Name: Beyhan Veli (@beyhan)
Affiliation: SAP SE
Nomination Statement: Beyhan has been active in the Cloud Foundry community for the past 7+ years. He started to contribute to the Cloud Foundry community as member of the BOSH team at SAP. His focus areas have been BOSH in general, BOSH DNS, BOSH monitoring, the BOSH command line interface, and bosh-deployment. Additionally, he actively helped people via the community BOSH slack channel and gave sessions about BOSH at CF Summit and CF Operators SIG . E.g. in 2020, he did a CF summit session titled "A Journey Through BOSH and BOSH DNS Monitoring" (https://sched.co/fDVn, https://youtu.be/jpnMKWKFhT0).
Since one year, Beyhan has taken on a broader role at SAP and is now also looking into other areas of Cloud Foundry like the Cloud Controller and general topics from a Cloud Foundry developer experience perspective. In the CF Community, Beyhan is technical lead in the Foundational Infrastructure working group and acted as the co-chair for the CF Summit 2021 "Behind the Curtain" track.

@loewenstein thanks, so glad to see this nomination come in!
@beyhan can you please reply or give a 👍 to indicate you accept the nomination? Thanks!

+1

Closing as the 2022 TOC election has concluded.


chore: bump cppcheck to 2.13 version

Issue being fixed or feature implemented
https://github.com/dashpay/dash-issues/issues/67
See also https://github.com/dashpay/dash/pull/5880 for prior work.
There's too many false alarm warnings with new kubuntu 23.10 (cppcheck 2.11)
What was done?
Bump cppcheck to version 2.13 and fix related warnings or suppressing them if they are false-alarm.
How Has This Been Tested?
Run test/lint/lint-cppcheck-dash.sh and see no output.
Breaking Changes
N/A
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone

83f1cca00f7d538f308f4b83e803adfe678a4dcd is nice but it looks unrelated, it should probably be extracted into its own PR

Otherwise 'cppcheck' complains: CBlockPolicyEstimator& fee_estimator can be const

@knst knst force-pushed the fix-cppcheck-2.13 branch from 47c1b38 to 91b40e8

rebased to be on top of re-merged master.

@knst knst force-pushed the fix-cppcheck-2.13 branch from 91b40e8 to da54afb

lost one change accidentally, fixed.

Rebase looks good
 1:  6d80a3a010 !  1:  3f1cb8d13d test: add suppression for state.Invalid() which is widely used by cppcheck is unhappy
    @@ test/lint/lint-cppcheck-dash.sh: IGNORED_WARNINGS=(
      )

      # We should attempt to update this with all dash specific code
    -@@ test/lint/lint-cppcheck-dash.sh: FILES=$(git ls-files -- "src/batchedlogger.*" \
    -                         "src/evo/*.h" \
    -                         "src/governance/*.cpp" \
    -                         "src/governance/*.h" \
    --                        "src/hdchain.*" \
    -                         "src/keepass.*" \
    -                         "src/llmq/*.cpp" \
    -                         "src/llmq/*.h" \
     @@ test/lint/lint-cppcheck-dash.sh: FILES=$(git ls-files -- "src/batchedlogger.*" \
                              "src/rpc/governance.cpp" \
                              "src/rpc/masternode.cpp" \
    @@ test/lint/lint-cppcheck-dash.sh: FILES=$(git ls-files -- "src/batchedlogger.*" \
                              "src/statsd_client.*" \
                              "src/test/block_reward_reallocation_tests.cpp" \
     @@ test/lint/lint-cppcheck-dash.sh: FILES=$(git ls-files -- "src/batchedlogger.*" \
    -                         "src/test/dynamic_activation*.cpp" \
                              "src/test/evo*.cpp" \
                              "src/test/governance*.cpp" \
    +                         "src/wallet/hdchain.*" \
     -                        "src/unordered_lru_cache.h")
     +                        "src/unordered_lru_cache.h" \
    -+                        "src/wallet/hdchain.*" \
     +                        )

 2:  9f15182aaa =  2:  7daa9bea9a refactor: use std::any_of
 3:  f76110e716 =  3:  c23c25d00b test: supress any_of suggestions in cppcheck-dash
 4:  5d6f25fcba =  4:  f36bb1f085 style: add semicolumn that the end to ENTER/LEAVE CRITICAL SECTION
 5:  58dedf8329 =  5:  3abeab16d3 refactor: replace multiple C-style casts to reinterpret_cast
 6:  7f9484226a =  6:  28f7ecc11f test: tidy up and reorder warnings in lint-cppcheck-dash
 7:  44f40a71ec =  7:  61a6c407fe fix: re-order asserts in creditpool
 8:  aea366c8c8 =  8:  add99ea862 refactor: add more consts everywhere as required by cppcheck 2.13.0
 9:  731256e136 =  9:  b4ed65e15e refactor: add multiple missing `const`
10:  02f654676a = 10:  6bc14a35e5 chore: bump cpp check 2.10 to 2.13
11:  2ee5719017 = 11:  e7e355ba8b refactor: make SetNull in CJ classes virtual to prevent warning from compiler
12:  727fba31d2 = 12:  e2ac86b8f7 refactor: drop dependency of CJ to fee_estimator
13:  da54afb5b5 = 13:  df554f396b test: add multiple suppression for cppcheck to make it finally quiet
~


gui: Use interpolation where needed in strings

This should go at least some of the way to fix Allister's comments in #177.
Also add mode picker in SNTTestGUI to make testing UI changes for different client modes easier.

@arubdesu Is this better?

Yes, looks good 👍


Rename setting from "program" to "schema"

In discussion with @mbeckerle on issue #653 it would be best to rename the setting program to schema. This ticket is specifically for renaming this setting. This will make it more clear as to what the user needs to point this setting to as program is a bit vague. Also, eventually in order to fix the main issue of #653 we will add a new setting called root, so having a root and program seems more confusing than root and schema, at least to me.

This will also include renaming command ${command:AskForProgramName} to ${command:AskForSchemaName}


Update rancher-aks-operator to support kubernetes 1.22

CRD helm installation shows deprecation warnings on 1.21:
W0714 11:21:31.379515 4042141 warnings.go:70] apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition

Done via https://github.com/rancher/aks-operator/pull/44


0.11 updates

Relies on "purescript-argonaut-core" having it's 0.11 changes being released as 3.0.0

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey! It looks like you and I came up with similar changes, I missed yours before proposing #98  though. The two major differences are that I also make sure test/Main.purs compiled and I chose to change RetryPolicy to use Milliseconds, instead of converting Int -> Number -> Milliseconds, since I thought that was both unnecessary and I felt it would fit in more with what the other libraries were doing. It is, however, a design decision that the maintainers should get the final say on.
I also chose not to use unsafeCoerce in reimplementing parseJSON. There are a few other minor details where I like what you did (such as removing parentheses around single type class constraints) and I'll update those in my version soon.
Argonaut-core is good to go at 3.0.0 and I can't think of any other major problems. Would it be easier to go with my merge request though, if you agree it is more complete?

Yeah go for it, i'll close this


Migrate tests for AddToCart button to Vite

Description
In replacing the old providers, this is one way of doing it now with RTL. Not sure if this is better or not, but it does seem more clear to me?
Part of #670
Additional context

Before submitting the PR, please make sure you do the following:

[ ] Read the Contributing Guidelines
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[ ] Update docs in this repository according to your change
[ ] Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

@cartogram @mrkldshv @lordofthecactus Would like your thoughts on this one!


headset fonction

I can not connect my headset corsair RGB to aurora can you help me ?

It is connected, you need to select a mouse to act as a peripheral device.

Thank you very much it works.


Added VS2017 clarification.

Docs Build status updates of commit 1da406d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/progressive-web-apps-edgehtml/windows-features.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add an NS_SWIFT_NAME for FIRSnapshotMetadata

Fixes b/67332554
Note that this is a breaking change for Swift users of the API.

cc: @ryanwilson


Loop iteration stepping mode

A useful additional stepping mode would be to step one iteration of the current loop, landing back at the same statement, but one iteration later.

It'd also be nice to step through an entire for-loop in one command (i.e. I just had a use-case where I was trying to debug iterations of an outer loop, but had to continue stepping through the inner loop too).

Maybe even more general jumping blocks?


Improve exceptions and asserts for rendering lib.

Use actual exceptions rather than assertions containing code
containing strings when trying to give messages to authors.
Introduce RenderingError which is an AssertionError that takes a
string argument, to support the above.
Provide a BoxDimensions.hasBoundedWidth/hasBoundedHeight API.
Document BoxDimensions.isNormalized.
Provide more useful information when we assert isNormalized and find
that it is false.
When finding the size is infinite, crawl the tree to figure out which
render box is likely responsible for the infinite constraints.
Provide more information when size doesn't match the constraints.
Provide more information when intrinsic dimension methods violate the
constraints.
Only spam a huge amount of information for the first exception from
the rendering library. I've noticed a lot of people looking at the
last exception printed rather than the first and that's very
misleading -- after the rendering library hits an exception, all bets
are off regarding what'll happen in the future. All kinds of asserts
might fire.
Improve docs around the debug methods and flags for the above.
Make Block default to have no children. Previously, giving no children
crashed with a confusing message about a null deref in an assert.

LGTM


Make 'disable ssl for dashboard task' idempotent.

This should reduce number of 'changed' tasks during convergence test.

I think it's eligible for ceph-ansible 5.0 back-porting.

I always forget it. Sorry. Done.

@guits, is it look good?

@amarao could you please rebase on master? thanks!

I've done rebasing and I hope I done it right.

(something in CI is broken outside of things I've touched).

the CI failure here is related to ceph/ceph#36091

jenkins test all

jenkins test all

jenkins test centos-non_container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-non_container-all_daemons


feat: joke module added

This PR closes #9
The module uses the pip package pyjokes to generate and show random jokes.
@swapagarwal @TeAmP0is0N please review.

Waiting for the-vision/jarvis-core#9 to be resolved.

Thanks @adityabisoi! 👍