Support --output-format=terse

Requested feature: Support Kani's --output-format=terse option.
Use case: Ease of viewing the CI log.
Test case: Here are two views, one w/ terse and one w/out -- https://github.com/weaversa/aes-rust/actions/runs/3388002019/jobs/5629452264

Thanks for the suggestion.  Currently, you can specify additonal arguments using the optional args flag to the CI action.  e.g.
      - name: Kani run
        uses<EMAIL_ADDRESS>        with:
          args: --output-format=terse

Would it be valuable to you to make this an explicit flag? Or is this facility sufficient?

@danielsn I just tried passing the flag and it worked perfect. Thank you!


Make SnarkyJS run inside a web worker

This PR exists to work on #378

Closes #378

I have tested this fix manually in Chrome v104 with new Worker(...), it no longer crashes when calling await isReady inside a web worker. I have not been able to test any further, since i am running into an infinite loop while compiling in a browser (while it works in the CLI).


feat: enhance plugin Manager with visibility

This PR...
Changes

Fixes

How to test it

screenshots

Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

We could use visibility already for defaultExecutionTabs - we are doing filtering there, while these metadata would work 👍


fix metro config and patch draggable flat list

Explanation of Change
Fixes a patch to the draggable flat list library, so that there is no re-initialization of variables that caused warnings on the console.
In addition, I added a metro wrapper in the configurations that will make it easier to find the cause of similar errors in the future.
https://docs.swmansion.com/react-native-reanimated/docs/debugging/accurate-call-stacks/
Fixed Issues
$ https://github.com/Expensify/App/issues/52920
PROPOSAL:
https://github.com/Expensify/App/issues/52920#issuecomment-2545499294
Tests
(Test on android native)

Click create expense
Enter scan tab
Make sure console warnings do not include the reanimated warning

[x] Verify that no errors appear in the JS console

Offline tests
Are not needed
QA Steps
Same as tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[X] I verified that all the inputs inside a form are aligned with each other.
[X] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos
Only recording from android is needed

Android: Native
https://github.com/user-attachments/assets/111cfda0-ccdc-4eac-b930-3949a765026d

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@sumo-slonik @mountiny we can't check the console waring in android


Rename :path option

Plug is already using it and it drops that key before handing it off to us, shown here: https://github.com/elixir-lang/plug/blob/master/lib/plug/session.ex#L48
Sorry for this issue!

Think we can merge this?

Oh. Yes.

?

think you can give me merge access?


[hmac,doc] Consolidate documentation changes

Description
Documentation changes need to be consolidated and aligned consistently with PRs #21307 and #21604 that bring in new features and extensions to the CSRs for HMAC to support save & restore and wider digest sizes and configurable key lengths.

Closing as completed in #22331.


Fix typo

Wow, that's a big mistake. Someone mentioned it in the comments a year ago but I must've forgotten. Anyway, thanks!


Rename ipxe images to match on-dhcp-proxy bootfile name

Include EFI images
@RackHD/corecommitters @RackHD/rackhd_dev

+1

:+1:

could this be the root cause of your issues on discovery with our current example automation process? @heckj @benbp

:+1: seems likely @DavidjohnBlodgett

@benbp sorry, not likely because the ipxe bootloaders are checked into source control right now: (https://github.com/RackHD/on-tftp/tree/master/static/tftp)

I double checked @rolandpoulter 's environment and he found his /common dir to be empty. I have added sudo args to all ansible steps added for static files and we're seeing if this resolves any write permission issues.

If you ran with -vvv did you get any messages about why it wasn't copied? Should the run fail if it can't copy anyways?

Oh sorry @heckj I was ignorant of the PR context, I though @DavidjohnBlodgett was talking about discovery images.


Crash at startup

Expected Behavior
Opening and running a bot.
Actual Behavior
Error:
Traceback (most recent call last):
File "C:\Git\PokemonGo-Bot\pokecli.py", line 40, in
from pokemongo_bot import PokemonGoBot
File "C:\Git\PokemonGo-Bot\pokemongo_bot_init_.py", line 22, in
from pokemongo_bot.event_handlers import LoggingHandler
File "C:\Git\PokemonGo-Bot\pokemongo_bot\event_handlers_init_.py", line 2,
in
from socketio_handler import SocketIoHandler
File "C:\Git\PokemonGo-Bot\pokemongo_bot\event_handlers\socketio_handler.py",
line 2, in
from socketIO_client import SocketIO
ImportError: No module named socketIO_client`
Steps to Reproduce
Running pokecli.py
Other Information
OS: Windows 7
Git Commit: dfa4f78f85e391e7bc9c6757478e68f724e108fc

run pip install -r requirements.txt

Duplicated #1596

Should i comment this one or the one linked in your comment (but that one is already closed)?

OS INFO:
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.4 LTS
Release:        14.04
Codename:       trusty
ERROR:
[15:21:43] PokemonGO Bot v1.0
Traceback (most recent call last):
File "pokecli.py", line 417, in
main()
File "pokecli.py", line 52, in main
config = init_config()
File "pokecli.py", line 113, in init_config
load.update(json.load(data))
File "/usr/lib/python2.7/json/init.py", line 290, in load
**kw)
File "/usr/lib/python2.7/json/init.py", line 338, in loads
return _default_decoder.decode(s)
File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "/usr/lib/python2.7/json/decoder.py", line 382, in raw_decode
obj, end = self.scan_once(s, idx)
ValueError: Expecting , delimiter: line 6 column 18 (char 164)

I tried everything (reinstalling requirements etc.) and finally decided to make a clean install. I backed up my config.json, deleted PokemonGo-Bot folder and:
$ git clone -b master https://github.com/PokemonGoF/PokemonGo-Bot
$ cd PokemonGo-Bot
$ virtualenv .
$ source bin/activate
$ pip install -r requirements.txt
$ git submodule init
$ git submodule update
It is working now.

Duplicate of https://github.com/PokemonGoF/PokemonGo-Bot/issues/1596
/closed


Suppress error while retrieving MNL from Core without genesis block

@abvgedeika Please, investigate the issue on Travis as well.


bumped egui version to 0.23

Bumped the egui version to 0.23. No changes to source needed. Works well for me on Windows in the egui demo project. Let me know if there's any more tests you'd like to see! <3

Ah, just saw that this was migrated to egui_wgpu some time ago - not sure how I missed that 🤣 I'll use that instead. Feel free to close.

This project has not been migrated. It'm still maintaining it for people who use and and want to provide updates.
I will release a new version at the end of the week.


integrate LDAP as external auth service

Now envoy does not support it yet: https://github.com/envoyproxy/envoy/issues/20920
Could be easy to do it with the envoy Golang filter extension.
Seems https://github.com/go-ldap/ldap is a proper choice.

I am studying MoE series of articles,I am interested in this direction, who should I contact?

@XiaoYaoheihei thanks for asking.
there are four projects published in "开源之夏", https://summer-ospp.ac.cn/org/projectlist?lang=zh&orgName=MOSN 社区&pageNum=1&pageSize=50&programName=
But, we are limited to 3 projects that could be chosen eventually.
We intend to close this project in the internal discussion, so you are recommended to choose other projects.
The only chance that this project could be choen, is no one pick one of the other 3 projects.
But this issue is still very useful for Envoy and MOSN, still welcome to contribute, just out of the "开源之夏" project.

oh, is that means the project wouldn't be present at summer-ospp? I recently glance at the project in summer-ospp and have a strong willing to apply it

@XiaoYaoheihei thanks for asking.
there are four projects published in "开源之夏", https://summer-ospp.ac.cn/org/projectlist?lang=zh&orgName=MOSN 社区&pageNum=1&pageSize=50&programName= But, we are limited to 3 projects that could be chosen eventually.
We intend to close this project in the internal discussion, so you are recommended to choose other projects. The only chance that this project could be choen, is no one pick one of the other 3 projects.
But this issue is still very useful for Envoy and MOSN, still welcome to contribute, just out of the "开源之夏" project.

oh, is that means the project wouldn't be present at summer-ospp? I recently glance at the project in summer-ospp and have a strong willing to apply it

according to the rule of summer-ospp, it is there. but you are recommended to choose the others, you could treat it as it is not there.

The only chance that this project could be choen, is no one pick one of the other 3 projects.

we are limited to 3 projects that could be chosen eventually.

according to the rule of summer-ospp, it is there. but you are recommended to choose the others, you could treat it as it is not there.

The only chance that this project could be choen, is no one pick one of the other 3 projects.

we are limited to 3 projects that could be chosen eventually.

so disappointing, I’ve got great interest on the project.

@jackj-ohn1 Sorry for that.

But this issue is still very useful for Envoy and MOSN, still welcome to contribute, just out of the "开源之夏" project

if you still are interested in it, this project is still open.

Hello @doujiang24 , I am interested in this project, and here is my design. It is inspired by Kong. I would appreciate it if you could review my design and provide insights on areas that could be optimized.
Target
Leveraging Envoy Go extension technology, we aim to integrate LDAP-based access control capabilities into Envoy. Only requests that pass the LDAP server's authentication will be proxied to the upstream service.
During this process, we can optimize the system by implementing user information caching with a duration defined by config.cache_ttl. This approach will help reduce the frequency of LDAP server access.
Usage
The client set credentials in Authorization header in the following format:
credentials := [ldap | LDAP] base64(username:password)

An example of the Authorization header is as follows (c2VydmljZXVzZXI6bXlzZWNyZXQK, which is the base64-encoded value of serviceuser:mysecret):
Authorization: ldap c2VydmljZXVzZXI6bXlzZWNyZXQK

The header type "LDAP" can be customized in the configuration using the config.header_type parameter. For example, it can be customized to "basic" or any other desired value. The relevant configuration details are mentioned below.
Config

ldap_host, string, required

Host on which the LDAP server is running.

ldap_port, integer, default 389, required

TCP port where the LDAP server is listening. 389 is the default port for non-SSL LDAP and AD. 636 is the port required for SSL LDAP and AD. If ldaps is configured, you must use port 636.

bind_dn, string, required

Base DN as the starting point for the search; e.g., “dc=example,dc=com”.

attribute, string, required

Attribute to be used to search the user; e.g., “cn”.

header_type, string, default ldap, required

An optional string to use as part of the Authorization header. By default, a valid Authorization header looks like this: Authorization: ldap base64(username:password). If header_type is set to “basic”, then the Authorization header would be Authorization: basic base64(username:password). Note that header_type can take any string, not just "ldap" and "basic".

cache_ttl, number, default 60, required

Cache expiry time in seconds.

ldaps, bool, default false, required

Set to true to connect using the LDAPS protocol (LDAP over TLS). When ldaps is configured, you must use port 636.

timeout, number, default 10000

An optional timeout in milliseconds when waiting for connection with LDAP server.

keepalive, number, default 60000

An optional value in milliseconds that defines how long an idle connection to LDAP server will live before being closed.

@Makonike Great, it's good to ship.

I would like to add some comments on the design:

ldap_host, string, required

Could we get rid of the ldap_ prefix since we are using them in the ldap extension? So with the ldap_port

header_type, string, default ldap, required

I am afraid that the "header_type" configuration is almost only used by Kong. The "ldap c2VydmljZXVzZXI6bXlzZWNyZXQK" is actually Kong's special implementation and "Basic c2VydmljZXVzZXI6bXlzZWNyZXQK" are actually used widely, especially in the browser. Other API gateways like APISIX and traefik only support the latter format. You don't need to keep API compatible with Kong's unique implementation. We can remove this option in the first version.

ldaps, bool, default false, required

I would recommend using a general option to enable TLS, for example, "tls". So we can follow the same style in different extensions.

BTW, I would recommend taking a look at Traefik and Tyk for the API design instead of Kong, because we are developing a Go extension, not a Lua one. For example, a Go developer may call "keepalive" as "idle_timeout" or something else.

BTW, I would recommend taking a look at Traefik and Tyk for the API design instead of Kong, because we are developing a Go extension, not a Lua one. For example, a Go developer may call "keepalive" as "idle_timeout" or something else.

Thank you sincerely for the recommendations! I shall make the necessary revisions to my design shortly.

Hello @doujiang24 @spacewander , I have already updated my design. Please take a look. Additionally, I would like to inquire about the necessity of maintaining a connection pool to reuse LDAP connections.

@Makonike
Could you provide your design in a separate online doc? It is very hard to figure out what you have changed in the issue.
There are some issues in the new design, for example:
Authorization: base c2VydmljZXVzZXI6bXlzZWNyZXQK should be Authorization: basic c2VydmljZXVzZXI6bXlzZWNyZXQK
And ldaps is still remained...
Besides, I think an extension level certificateAuthority is overkilled for the first version, since a certificate already has a size in KBs.

maintaining a connection pool to reuse LDAP connections

It's usually done in the LDAP client itself. I would like to use the default option first.

I would recommend providing an MVP (all with the default value, not tls/start_tls), and then adding more features in the later iterations.

I would recommend providing an MVP (all with the default value, not tls/start_tls), and then adding more features in the later iterations.

ok, here is my design, and here is my demo.

I would recommend providing an MVP (all with the default value, not tls/start_tls), and then adding more features in the later iterations.

ok, here is my design, and here is my demo.

LGTM except for some comments. To comment on it, I register a fresh feishu account.


(feat) Use sha value for kwctl manifest cmd

Closes #97
Depends On: #99

Hi @nitishm , just tested it and seems to work just fine (from an usage perspective):

I just tried the inspect subcommand. And can I kindly ask if it is possible to make it like docker cli, that we can use the first 3 characters too (example below)?

Thanks a lot for your contribution, sorry for the delay caused by the holiday season.
The PR looks good, I left some minor notes.

Excellent. I will address these comments shortly :). And welcome back from the break!

@nitishm are you going to work on this yet?

@nitishm are you going to work on this yet?

Hey, I dropped the ball on this one because life happened. I can jump on it now or if you'd like to pick it up I'd be happy to hand off. Either works. Lmk

@nitishm no worries, we totally get it. If you still want to work on this feature, that would be fine with us!

Closing, this has been done via https://github.com/kubewarden/kwctl/pull/587. @nitishm thanks for your initial efforts on this task


textPlugin support for Internet Explorer

Hi,
First of all thanks for your contribution!
Unfortunately I do not intend to support IE. It's a lost battle and may lead to weird edge cases and ultimately to an enormous waste of time. Instead I urge you to use polyfills. You can use this one right here (based on your solution):
if (!Array.prototype.find) {
    Array.prototype.find = function (callback) {
        if (!this || !this.length)
            return undefined;
        return this.filter(callback)[0];
    };
}

Or even better, use one of the many mature polyfills libraries out there (checkout the excellent core-js for instance). Most of them are modular enough so you can use only the polyfills you need.

Closing the PR (see previous comment)


Crash while dismissing zoom picker

I just received this error report - any idea if there's something I can do to fix it, or if the code in the project needs to be fixed.
Fatal Exception: java.lang.IllegalArgumentException
Receiver not registered: android.widget.ZoomButtonsController$1@c8a017e
 Raw Text
android.app.LoadedApk.forgetReceiverDispatcher (LoadedApk.java:878)
android.widget.ZoomButtonsController.setVisible (ZoomButtonsController.java:404)
org.chromium.android_webview.AwZoomControls.dismissZoomPicker (AwZoomControls.java:38)
org.chromium.content.browser.ContentViewCore.onDetachedFromWindow (ContentViewCore.java:1537)
org.chromium.android_webview.AwContents$AwViewMethodsImpl.onDetachedFromWindow (AwContents.java:3166)
org.chromium.android_webview.AwContents.onDetachedFromWindow (AwContents.java:2398)
com.android.webview.chromium.WebViewChromium.onDetachedFromWindow (WebViewChromium.java:1847)
com.android.webview.chromium.WebViewContentsClientAdapter.onPageFinished (WebViewContentsClientAdapter.java:531)
org.chromium.android_webview.AwContentsClientCallbackHelper$MyHandler.handleMessage (AwContentsClientCallbackHelper.java:188)
android.os.Handler.dispatchMessage (Handler.java:102)
com.android.internal.os.ZygoteInit.main (ZygoteInit.java:1120)

ZoomButtonsController.java is not this plugin.
I think it's depends on your code when the map is removed, or something.
Until you provide more certain information, such as the code which reproduces the issue 100%, we can not help this.

I don't think it has anything to do with my code - I'm not handling zooming at all.
It looks like this is a bug with Android that maybe this plugin can workaround? Here are the details, along with steps to reproduce: http://stackoverflow.com/questions/5267639/how-to-safely-turn-webview-zooming-on-and-off-as-needed

Try this at once in your local.
http://stackoverflow.com/a/34176342/697856
I can not reproduce your issue, and it's not this plugin issue.

Thanks, but I haven't written any WebView or map related Java code in my project - I'm only interacting with the maps through this plugin, so trying that solution isn't easy.
I'm closing this issue - you don't think it applies to your plugin, and I don't think it applies to my code, so it must not exist ;).

Thank you for understanding.
Without certain reproduce code, debugging is so hard.
It's really depends on your code, the plugins you use, WebView version, etc...
That's why I can not help you.


Fix some fp insts not modifying mstatus.fs

Before send patch, please rebase your code on top of current xs-dev


different result on suma++ and rangenet++

Thanks for your code!
When I run rangenet++ on my trained dataset, the result is almost correct, but when I run suma++ through the model.trt generated by rangenet++, the result has many wrong label, like the car , the top of the wall, and the people .
In my opinion, if the result of rangenet++ is correct, then suma++ should be correct, too.
What's the problem?
This is the result of rangenet++.

This is the result of suma++.

Hey @xdtzzz, thanks for using our code.
The noisy map results may be caused by the wrong prediction and inaccurate poes estimation.
For example, the points of a car were predicted to be both car (blue) and house (yellow) in the segmentation results.
When aggregating all the predictions together, it will become more obvious.

谢谢您的回复！
您的意思是，每个类别都必须要分割的十分精确才可以使suma+达到良好的效果吗？
在rangenet运行，如果一个物体里大部分点云被成功的预测了，少量点云是错误的，就会导致suma+里面很多点云都是错误的情况吗
发自我的iPhone
------------------ Original ------------------
From: Xieyuanli Chen @.>
Date: Thu,Dec 30,2021 3:54 PM
To: PRBonn/semantic_suma @.>
Cc: xdtzzz @.>, Mention @.>
Subject: Re: [PRBonn/semantic_suma] different result on suma++ and rangenet++(Issue #52)

Q: Do you mean that SuMa++ needs very good semantic segmentation results to work well?
A: Yes, SuMa++ itself can not improve the semantic segmentation results, but only use the semantics to improve the SLAM results. Wrong predictions will lead to a wrong semantic map. We originally use RangeNet++ which achieved an IOU of 52.2 and worked well. The segmentation results seem not as good as the original RangeNet++. What's your segmentation performance in terms of IOU? The current SOTA one is of IOU 70, and you may use the SOTA semantics to get a better semantic map.
是的，SuMa++对semantic segmentation本身是没有提升的，只是利用语义信息来提高SLAM的精度，如果语义信息不准，语义地图也是会不准的。
原文用的rangenet++的IOU是52.2是可以工作的，当前的语义分割结果似乎比原文差，请问现在语义分割的IOU精度是多少？ 目前性能最好的语义分割算法性能是70IOU，可以尝试使用更好的语义分割算法。

我现在语义分割的iou精度挺高的，因为标注的类别比较少，平均达到了0.8以上

Results on rangenet++, most of them are correct. But on suma++, there are many wrong semantic points. For example, the ground points build on the top of the wall on suma++, but on rangenet++, the wall doesn't have ground points.
rangenet++:

suma++:

where is "reopen" button.... # @Chen-Xieyuanli


Add SizeOfVal operation

This is a refactor of previous work, adding the UnOp::SizeOfVal to MiniRust. With that, at least half of #234 can be properly implemented.
Compared to the first iteration, this requires a safe pointer, in Raw pointers, the information is not sufficient to compute the size. And function pointers are currently not supported, altho, Rust does and returns 0 for the size of a function. So this is a discussion to be had.
Slices (which are the most interesting) are untested, because this PR isn't stacked on the lastet #237 and constructing slices is a pain before that.

function pointers are currently not supported, altho, Rust does and returns 0 for the size of a function

size_of_val cannot be called on function pointers, so I don't quite know what you are talking about.

function pointers are currently not supported, altho, Rust does and returns 0 for the size of a function

size_of_val cannot be called on function pointers, so I don't quite know what you are talking about.

Disagree: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=a7e2d9817ff3daf88f81e695ea683d4c

You are calling it on a reference to a function pointer. Actually you are calling it on a reference to a "function item type". And yes function item types are zero-sized.
But you can't call size_of_val on a function pointer (or function item type).

You are calling it on a reference to a function pointer. Actually you are calling it on a reference to a "function item type". And yes function item types are zero-sized.
But you can't call size_of_val on a function pointer (or function item type).

Hm okay, does this mean our Type::Ptr(PtrType::FnPtr) is our representation of a "function item type" or do we not model this feature? Then what I passed would be a Type::Ptr(PtrType::Ref { .. }) with a pointee being a pointer, and all pointers are 8 or 16 bytes in MiniRust, as opposed to the 0 which rustc gives.

I adjusted the PR to your feedback and stacked it on the now ready #237. This time I tried to do it with a merge commit instead of a rebase in a effort to make iterative reviewing easier. However, I am unsure whether this is better and whether this will just lead to problems when squashing eventually.

Yeah this is one of the situations where a rebase and force push is needed. If you keep revisions (in reply to reviews) in new commits, rather than amending existing commits, it's still usually possible to see what changed since the last review. If that's annoying due to conflicts, then it's okay to squash -- that should be sufficiently rare, hopefully.

Hm okay, does this mean our Type::Ptr(PtrType::FnPtr) is our representation of a "function item type" or do we not model this feature? Then what I passed would be a Type::Ptr(PtrType::Ref { .. }) with a pointee being a pointer, and all pointers are 8 or 16 bytes in MiniRust, as opposed to the 0 which rustc gives.

Our FnPtr corresponds to Rust's fn() -- function pointers, not "function items". Note however that size_of_val cannot be called on a function pointer! It can only be called on &T. It can be called on &fn() (that's not what your example does, but it is possible). But &fn() is a MiniRust TypePtr(PtrType::Ref) where the pointee type is a function pointer -- it's a reference to a function pointer.
When you do let x: fn() = foo;, this insert a coercion from the function item type for foo to the function pointer type fn(). So in your example you want to do something like size_of_val(&(foo as fn())) to see the answer 8, for a function pointer.

Okay, thanks for your explaination


Convert the remote agenda request from csv to JSON

Provided with URL:
```https://datatracker.ietf.org/meeting/116/agenda.json```

it looks like Android has some reasonable JSON handling, use that to parse this new URL.
New URL seems to have less performance problem with wget at least.

This was completed with the most recent merge.


Upgrade to webgl2 and additional seeking, seeked events.

Upgrade to webgl2
Upgrade COMBINE and aaf_video_scale shaders to es300 -- we can upgrade the others when we're using them and can validate the changes.
Add a new state VideoContext.STATE.SEEKING -- VC will be in the state when we're waiting for the current time to change after current time has been changed NOTE: vc will only enter this state from PAUSED.
VC will return to the PAUSED state from SEEKING, once
Add some new events SEEKING and SEEKED to be sent when the seeking and paused states are reached (ie after the video frame has rendered.
All so add 'PAUSE', 'PAUSED' and 'PLAY', 'PLAYING' events.
Add a clearCanvas method
Change the rendering loop to always call render on the destinationNode -- this appears to fix some of the initial black frame errors, and also appears to speed up rendering after a seek.
Remove the clear command from DestinationNode, and instead clear the buffer at the beginning of the render actions.

Misc changes:

Remove _rendered prop -- it was unused
Remove _renderPaused prop -- it was also mostly unused.
Avoid calling pause or play when already in that state.

I'm assuming the upgrade to webgl2 is what made things faster? Also it looks like you fixed the issue where the first frame doesn't get loaded/rendered on the video? If so, that's great!

I'm assuming the upgrade to webgl2 is what made things faster? Also it looks like you fixed the issue where the first frame doesn't get loaded/rendered on the video? If so, that's great!

I think the change that fixes this is that I pulled out an explicit call to render the destination node (aka the frame buffer)... This was not always being rendered on every RAF before, and now it is. The compositing nodes now only render when renderOnDemandOnly is set but the destination node always renders. -- that being said I still think we can improve the performance here by replacing the renderOnDemandOnly logic with logic inside the nodes themselves -- they should be able to better determine if they should update/render or not.


Design a 3D printable case for v2.0.x and v2.1.x

Great work on the project all. I've actually designed a 3D-printable case for the board. Let me know if you're interested and I'd be happy to share.

Hey @s-gordon,
we are also working on a Case for v2.0. I haven't posted it yet, since the PCB footprint is still not totally final. We are soon going to order an other batch of PCBs and hope to have a final version with case, to release.
In the meanwhile, here is a link to the WIP Case. https://cad.onshape.com/documents/0970230d187b540faa10c45f/w/4e628c030e78246416c5e857/e/a4e55bb6f45b9b87b321fa69?renderMode=0&uiState=63b00b6f2536af1c82a081f6
The case is supposed to have an assortment of mounting options available, which can be extended apon.
If you have a case for V1, please feel free to open a pull request so we can also offer an option for that. I am also open to improvement ideas :)

TODOs for the case until the release:

[ ] Adjust to new footprint
[ ] Make it thinner
[ ] Get more light to the light sensor (might be solved by making the case thinner)
[ ] Have a drawing for measurements for the mounting plate

Excited for this, will get a 3d printer when it's ready to go 😁

It'd be great to see some further progress here;

The lid doesn't appear to have a way  to attach to the case.
The standing mount requires supports. It'd be better to have this printable without supports in some way.

It'd be great to see some further progress here;

The lid doesn't appear to have a way  to attach to the case.
The standing mount requires supports. It'd be better to have this printable without supports in some way.

The lid has ha notch which locks into place when pushing it in. And Yes the standing mount still needs support.
I have been refining the design after the official release of the V2.0 board. I'll follow up with further information soon.

@Schluggi @MeisterGig Thank you for taking the time and effort to create this project.
I printed the WIP V2 case to trial fit.  Good news is that the board fit in the case perfectly.

I know that it's still in progress but just wanted to provide some feedback.  The lid doesn't have an interference fit so it won't stay in place without some kind of adhesive.  The corner wall mount should also be scaled a bit down to allow for a snap fit.
Did anyone else have issues or am I doing something wrong on the printing side of things?
Thanks!

Hey @afp316 ,
Thank you for your input :)  I also encountered problems with the lid. I inverted the notch on the lid and the case after the v2.0 rc1 because it was making it harder to get the board in and out of the case. I haven't  been able to get it as nicely  fit together as the previous version yet. Also i have resized the case recently and might have forgotten the mounts.
I'm gonna share some design considerations and problems i have encountered so far:

The case should be modular
It needs to have good ventilation for the bme both smd and recently also as a module (still working on that one )
the light sensor needs to be as close to the lid as possible otherwise it won't get enough light or i need giant hole
I personally like heat insert, but it was requested to not need any screws or heat inserts, because it looks bad and insert might be hard to get
the PCB is very modular so it's hard to design a one size fits all case
the PCB should be easily taken out of the case
when the case is closed it should fit securely inside the case
pin headers have different hights (i have encountered 7, 8 an 8.5 mm versions)
ESP8266 have there ports monted on the bottom esp32 on the top
some people might want to supply power using the scre terminals below the mcu (i don't like a giant hole in the bottom)
long pins on the lid to hold the board in place break off easily as to many screless locking mechanisms
it should also look good

I was discussing with @Schluggi to maybe enforce one size of pin header and also maybe use heat inserts to mount the PCB. What are your thoughts on this :)

Most of the pin headers i've encountered are 8.5mm. I don't think I've ever had anything else less than that.
The more BOM items you add, the harder it is for people to source. Heat Inserts are a bit more difficult to find than screws. I've seen a bunch of cases that use screws as an option so it might be worth considering if you're having trouble. Remember - you can screw into plastic without the need of a heat insert, too.

IMO - It's going to be super hard to meet the needs of every single variation.
If you standardise the pin headers at 8.5mm (this seems the most common option) and start with supporting the recommended S3 mcu - along with both the smd and pcb modules - then once you've sorted that, move on to variations. Perhaps you provide multiple case options for the different MCU's for example - rather than trying to create a "one case fits all" scenario.
Here's an example case that uses two self tapping screws. It uses little nubs at the bottom of the case to hold the PCB in place, and might provide some inspiration. :)
https://www.printables.com/model/380943-wled-wemos-shield-case-pcb-v3

@MeisterGig I also like heat inserts but @cadriel makes an excellent point regarding sourcing and logistics.  I can confirm that I have the 8.5 mm headers measured with calipers.  Perhaps standardizing to a header spec would make the case design have one less variable (among many variables that you have dealt with).
As someone who doesn't really mess with SMD, I would think it's more accessible to design around the cheap breakouts that most amateurs buy (like myself).
I haven't looked at the datasheet for the light sensor or experimented with translucency effects on the sensor but perhaps a single layer/single wall would be able to satisfy the light sensor requirements for the application.  It may be such that the final recommendation is to print with certain colors of filaments for the lid.
In line with @cadriel thought, designing around a base set of recommended hardware will help with the design decisions immensely.  I'm sure you already know that considering I'm a late addition to this party. 🎉

Yep. Also - the light sensor doesn't necessarily need a whole bunch of light. I think its more important that the light exposure is consistent - and that you let SOME light in. I have other light sensor devices that only provide small holes to meet that requirement.

Oh - @MeisterGig  - the example case design I posted would help with some of your other concerns too. The base of the case is shallow - so adding / removing the PCB is very easy. The top of the case is inverted when compared to your current design, which provides other benefits too.

So I have adjusted a few details. The lid should now be able to snap onto the case again (if it needs too much force please tell me). There are many options variabilized (still a WIP), so adjustments can be made by changing the variables. At the moment I have set them to use 8.5 mm standoffs, a esp32 c3 mini and m3 heat set inserts (to change to screw, change the variable mounting_hole to fit your screw). The standing mount should now be printable without using supports. Unfortunatly I am currently unable to print  the case, so if someone else is able to do so feel free and give feedback. https://cad.onshape.com/documents/0970230d187b540faa10c45f/w/4e628c030e78246416c5e857/e/0f1cd3ab0062a4366c4b0a6d
To address the light sensor issue. I have left the hole this big for now. Earlier testing back in v1 has shown, that readings from the sensor get to inaccurate, if there isn't enough light or even a grid or something like that in front of the sensor.

I'll print this today, but the first observation is that the overhangs on the mount are still problematic for FDM printing. Normally you'd want to keep these under 50° (or around-a-bout's there depending on the printer and material).
With the current mount, because you've used a fillet - the bottom edge is well over 50°. I've thrown in a screenshot here for reference;

If you go with a chamfer and control the degree's then it'd be an easier print.
Re: the other changes you made - they seem ok, I think it'll come down to the correct dimensions. I'll confirm with my setup - an ESP32 C3 and the Panasonic PIR and let you know if the table variables you have in place are good once I've printed and played around a bit.

The lid has something a bit odd going on.. =)

Ok, unfortunately there are multiple issues.
Including the above;

The PCB now sits a bit higher, but the USB hole wasn't adjusted to match - and thus they don't line up.
The mount won't fix to the case because the shape of the back interferes with the case itself.
Even if the above wasn't true, the "clasp" wouldn't hold on to the case.
The PIR won't seat into the PCB when the PCB is in the case, because the extrude doesn't sit deep enough (I.e., this cut needs to go deeper so the PIR can sit further into the case..).

Pics attached (in order to the points above);

Finally - here's an example when printing the standing mount.

I haven't printed the top yet due to the above. Do you have access to a printer @MeisterGig ? A lot of these issues could be caught with some iteration ;)

I just realized I have the mount around the wrong way, so ignore my comment about its interference. The cutout in the case is still irrelevant tho because the mount can't use it.

@cadriel Thanks for printing a test print. My calculations seem to be off after the variabilization of the cad files :(
About the standing mount. My printer prints overhangs up to 70-80° quite fine, when it works... But maybe that's not the case for everyone. (And yes there are some degrees above 80°) i'll see how the campfer looks like.

@cadriel Thanks for printing a test print. My calculations seem to be off after the variabilization of the cad files :(
About the standing mount. My printer prints overhangs up to 70-80° quite fine, when it works... But maybe that's not the case for everyone. (And yes there are some degrees above 80°) i'll see how the campfer looks like.

Yep, I agree. I think it may also depend on material - but I think the key is catering for as many people and printers as you can. My Voron printing ABS has trouble with the first few layers - which then results in poor results further up on this part.

Did a test print in pla on my x1. Board slides in nicely.
I haven't put the inserts in yet but the holes look pretty good by eye.
The lid snaps in very snugly... almost too snugly. I couldn't remove it easily; had to use a screwdriver through the usb-c slot to pry.
The mount clasps don't grasp the indentations on the case tightly enough. Could tighten up the clasp dimensions as-is or maybe implement slots in the case that slide onto and are held by the mount.

Is that the old case and stand with the new lid?

Oh, it looks like may @MeisterGig has adjusted it since I printed..

@afp316 the stand is not placed right. There are grooves in the middle wich it should go into.
It might be a little tall now i haven't tried it yet.
@cadriel yes i have been working on your feedback :)

@cadriel yes i have been working on it :)
@afp316 thank you for the feedback.

@cadriel yes i have been working on it :)
@afp316 thank you for the feedback. (I froze it as version v2rc2.2 in onshape)
Just got an idea on how to get rid of the inserts. @afp316 picture of the usb Port seeams to fit quite good now. I am gonna try to just use foam on the lid to keep the board in place.
My printer works again so I'm gonna print it now and tell you how it went by tomorrow :)

@MeisterGig Sounds good! On another note, you might want to adjust the base so that it sits lower. I found my USB-C cable couldn't be used because there wasn't enough space beneath the case when the stand was attached in its default position.

Do you mean the standing mount @cadriel ? I already made it longer and changeed it to a campfer like you suggested.

Do you mean the standing mount @cadriel ? I already made it longer and changeed it to a campfer like you suggested.
You can already see it on the picture from @afp316

Yeah, the stand. But more about the distance between the bottom of the stand and bottom of the case. Here's a pic to illustrate - I ended up glueing the existing stand temporarily - but I had to glue it right at the bottom of the case otherwise I'd have no room for the USB to attach. Also - you can see its quite strained even in this position -  so having more distance here would be beneficial. I have other USB-C cables too - but the majority of them have long strain relief like the one in the picture.
I'm not sure if you've already made it taller or not - but yeah, it was an issue for me.

@MeisterGig also - re: holding the PCB in place, something like this might work?

@cadriel Yeah i had that idea aswell. But the pcb needs to be removable

@cadriel Yeah i had that idea aswell. But the pcb needs to be removable

Aah, in what way? The PCB would still be removable in this case by first removing the top. Then it should fall out..

So i did some testing with the foam. It works quite well and allows for minor inaccuracies between different prints, pcbs,parts and is cheaper and easier to get then heat inserts or self tapping screws

That is a great plan!  Well done!  It won't wiggle around or settle or anything like that, will it?  Even those that, for whatever reason, don't have the foam could just use a ball of electric tape to meet the same function as the foam.
For the HLK-LD2410, mine sits in the header firmly but can still wiggle around a little.  Is there anything that could be incorporated into the case that locks the free end of the HLK-LD2410 in place?  maybe a little standoff protruding from the lid?  Might be easiest to just put the same foam on the pcb under the mmwave pcb.

Hrm, is sandwiching the the processor against some foam going to cause overheating issues? I suspect the spec for the C3 may warrant specific spacing for airflow..

I've created a case and published it on printables.com
https://www.printables.com/model/456377-aiosense-case
This works well for me in the mean time! Feel free to use.

@Schluggi and I have done some Improvements tonight.

The lid naps in easier @afp316. The mounts also fit better now (Cornermount, wall mount and camera Mount now fit very well)
We moved the logo to the side which looks nicer.
The LED now has a hole as well

We think we can get rid of the foam all together by holding the back of the esp in place with a bar coming down from the lid.
Also the standing mount doesn't print properly without support. Im gonna split it into two parts.
I am optimistic these improvements will be finished this week. The case will then released to the repo and thingiverse/printables.

@cadriel Thanks for sharing your work!
@MeisterGig @Schluggi nice work all around.  I really like the changes you describe, especially the idea of the standoff coming off the lid to apply some pressure.
I can work on modeling a simple spacer to keep the mmwave sensor (the cheaper HLK-LD2410) from moving around freely.  Not sure if y'all are experiencing the same with your HLK's or maybe my tiny headers are junk.  I am using these.

@afp316 that would be great. I think i'm the only other person so far, who has an HLK for this project. I have soldered them directly to the board because I didn't bother to find pin headers yet. Of course it would be great to have them.

I made an initial mount for the HLK-LD2410 to help a bit with keeping the pcb more secure.
https://cad.onshape.com/documents/1a4aaa3e005976151241c26c/w/5e7ad8b8b7f30e1bc78c6254/e/c07d0c7cadc1cc9efcf0446a?renderMode=0&uiState=6441a2f1eb31156ab9b4474d

I think I can do better than this so I'm going to try to use the thru holes for the SEN0395 to design a more compact and secure mount.

@Schluggi and I have done some Improvements tonight.

The lid naps in easier @afp316. The mounts also fit better now (Cornermount, wall mount and camera Mount now fit very well)
We moved the logo to the side which looks nicer.
The LED now has a hole as well

We think we can get rid of the foam all together by holding the back of the esp in place with a bar coming down from the lid.
Also the standing mount doesn't print properly without support. Im gonna split it into two parts.
I am optimistic these improvements will be finished this week. The case will then released to the repo and thingiverse/printables.

Have you put together any additional case mount options aside from the corner mount? The case is great fit and I'm just trying to work out a good way to place it now and I don't really want to wall mount and have a cable trailing up the wall

Have you put together any additional case mount options aside from the corner mount? The case is great fit and I'm just trying to work out a good way to place it now and I don't really want to wall mount and have a cable trailing up the wall

Yes there is. But we are not happy yet. The clip for holding the case is to lose if you print this flat without support. So we need to adjust this in the future. But feel free testing.
Mounts - Standingmount.zip

I modified the STL slightly and basically just copied the measurements for the clip er 'nubbins' from the existing corner bracket on printables, I'd say it could be a tighter fit but this seems pretty secure (like I can aggressively shake it without the aiosense falling out)
Mounts - Standingmount_riweston.zip

@riweston I just printed your version of the stand and I have to say, I'm a big fan :)
Printable without support and a nice tight fit.
With your consent I would love to share this on printables as part of the official case. I can link your github or printables profile as well. May I? If yes, I would love to have the onshape link to your version or a copy of the design as a file, since everything should be Open Source :)
Thank you!

Oh great! Glad to hear it's a good fit :)
I just moved a few vertices around on the existing design really but help to take credit 😂
Very happy to share the project file. I'll check the file size tomorrow and assuming it's not too big I'll raise a PR if that suits you?


Docker: make memcache max value configurable

I found no way to configure the memcache max value size in the official docker container.
We have a really large InfluxDB and have to increase the max value size (Parameter -I) to prevent following error message:

[ERROR] 2018-09-13 11:23:47,245 - finder.get_field_keys() - Could not add field key list to memcache - likely field list size over max memcache value

Change of /etc/default/memcached with MAXITEMSIZE="128m" has no effect,
also /etc/memcached.conf to -I 128m has no effect.
I guess the two files are ignored when the memcached service is not started over /etc/init.d/memcached as it is done with the own startup script memcached.sh.
So I changed the memcached.sh Script:

#!/bin/sh
exec /sbin/setuser memcache /usr/bin/memcached -I 128m -v >>/var/log/memcached.log 2>&1

My suggestion is to make the parameter configurable over an environment variable. Maybe it is possible to export the value from the graphite-api.yml and use them also for startup of memcached

memcache:
host: localhost
max_value: 128

What do you think about this? Am I wrong in one point?

Hi there,
Thanks for the interest. Yes, that all sounds accurate. Making the memcached.sh script use /etc/memcached.conf config and have override in the config might be more appropriate, and would also allow for config overriding in containers for changing other memcached settings.
But either solution would be fine if you'd like to make a PR for it.

Should not be a big deal if we use the init script with memcached.conf
memcached.conf must be changed to ensure that we get the log file to the right place
The memcached.sh file could be replaced with:
/etc/init.d/memcached start
or we execute this command in the memcached.sh file to prevent that we have to change the run mechanism.
I will try to make a PR

The memcached.sh file could be replaced with:
/etc/init.d/memcached start

No, it cannot be. The container base image does not use init.d, hence why none of the service use it. The memcached.sh script can have memcached read configuration from the standard /etc/memcached.conf though.

I implemented today some changes in memcached.sh and provided an example /etc/memcached.conf. Looks good.
I will test this tomorrow again and submit an PR

please see PR #54

PR #54 is merged.
It is now possible to configure memcache with /etc/memcached.conf


SMTP credentials fix (task #9020)

This PR allows SMTP username and password parameters to be set as null. CakePHP's env() method returns an empty string for dotenv parameters with null values, which breaks core SMTP functionality which tries to authenticate with empty credentials.

Codecov Report

Merging #742 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #742   +/-   ##
=========================================
  Coverage     38.05%   38.05%
  Complexity     1189     1189
=========================================
  Files           106      106
  Lines          3952     3952
=========================================
  Hits           1504     1504
  Misses         2448     2448

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 61aeedb...e727378. Read the comment docs.


todo: edge selector support for Sketch.fillet() and Sketch.chamfer()

only "all" supported a the moment

Fixed in https://github.com/cadscriptHQ/cadscript/commit/e5da0a5fba6d78820c17bf0ac7d7446bd9e6fa9e


Add support for post serve action

Resolves => #1044
Added below features
Expose GET/POST/DELETE APIs for fetching or setting flexible post-execution hooks (i.e. it'll be a script in the same way that we have for middleware).
Change in simulation schema to accept registered post-execution hook for particular response details.
Changes related to executing post-execution hook after execution is over.
Pending:
UTs fixes
Documentation for newly added feature and APIs
@tommysitu  @petkostas  could u guys please review it and let me know if it looks fine?

@petkostas Can you do the testing once on your machine and see if it solves your use case?
Steps to test on your machine

Clone this branch and compile using make build.
Start hoverfly and capture the requests.
Write python or ruby script that sends callback to your application.
Register callback hook using command - ./hoverctl postserveactionhook --add testwebhook --binary python3  --script= --delay=
Once, done, you can check all registered hooks using curl - curl -X GET "http://localhost:8888/api/v2/hoverfly/post-serve-actions"
Edit simulation schema JSON and add a hook in the response part of the request/response pair.
Sample for hook to be added in simulation schema - "postSimulationHookName": ""
You can now switch to capture mode and see if a callback comes or not back to the server.

@tommysitu  whenever you get time, could you please check this PR.

@kapishmalik I am going to give it a try, most probably by the end of today! Massive thanks, mate!

@tommysitu  I have incorporated review comment changes and added a few more things to PR. I will be adding UTs for new files and documentation as separate PR. I have tested it once locally. I would request you also to test it once before you merge this PR.

@tommysitu I have added a few things which I want to point out.

I have added the request/response pair as stdin to post serve action script.
post serve action will be invoked on simulate/spy/synthesis mode whenever response will be served by hoverfly
Added GET, PUT and DELETE APIs for getting/setting or deleting post serve actions
Added hoverctl command to get/set/delete post serve actions

I am missing the part for adding the script directly with hoverfly (e.g. hoverfly --help) instead of only hoverctl. This is important for me as most of my simulations are predefined upfront.
For example:
./hoverfly --import imposter.json
INFO[2023-02-22T20:02:37+01:00] Using memory backend
INFO[2023-02-22T20:02:37+01:00] payloads imported                             failed=0 successful=1 total=2
FATA[2023-02-22T20:02:37+01:00] Failed to import given resource               error="invalid post server action name provided" import=imposter.json

@petkostas we already have support to add script via post-serve-actions.
Go to target directory and type:
./hoverctl post-serve-actions --set "action name" --binary python --script="script-location" --delay=1000

@petkostas I can add that as separate PR. As of now, we just want to understand if this works for you.

@petkostas for now, you can use hoverctl to create post serve actions and then using hoverctl only import simulation schema so that your automation won't break and you will also be able to use this feature from now.

@kapishmalik I got the hoverctl working (importing the post-action hook), yet my problem is that I mainly use hoverfly as we use Hoverfly through Docker images. To test this through hoverctl requires further actions, which I cannot perform in the environment I am now testing (as you can see, Hoverfly crashes).
I will finalise the test on my system, as I cannot test this currently in my pipeline flow, and get back to you! indeed, it looks neat and is what we need.

@petkostas sure makes sense. I would definitely come up with PR to add support in hoverfly cli in some days after completing UTs in new PR.

@tommysitu I personally also felt that it will be difficult for anyone to give entire script content in json payload. I am thinking to take script location instead of content in json payload. I will make this change as part of this PR itself.

@tommysitu Shall I accept script path/location instead of script content in our APIs? It will be helpful. I can see in middleware APIs we are asking for script content so thats why I did that way in post-serve-actions.

@kapishmalik the API is mainly used by hoverctl to post the content to hoverfly, so it's ok the way it is similar to middleware. I think most of the users would be like @petkostas using hoverfly command directly to read a file from a location.

@tommysitu yup, I believe. I will be working on that feature next post adding UTs  for newly added code.

@tommysitu @petkostas I have added support to import post serve action using hoverfly ctl. We can import multiple post serve action. Can you guys test and let me know if anything else we should add.
./hoverfly -post-serve-action "test-webhook python "file path" 3000" -post-serve-action "test-webhook1 python "file path" 5000"  -import simulation.json

I've tested both hoverctl and hoverfly commands, works really well!

@tommysitu I have incorporated review comment changes. You can review.

@tommysitu I am planning to add UTs and documentation as part of the new PR as I have added lot of new code so wanted to cover all the files. I have fixed the merge conflict.  Let me know if we are good with this for now.

@petkostas feature is there on master for you to use with support to add post serve action via hoverfly ctl. Any feedback is welcome. Just FYI... documentation is yet to be updated.

Amazing work @kapishmalik thank you very much for all the effort!
I will surely test it as it's something I was missing!


Microblink connector initial commit

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

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

@JurajZajec-MB Please close PR for now as your connector needs to be updated as per the API


Add Env.ReaderCheck to primary usage examples

This method is actually pretty critical to using MDB in production. And it is not being called out at all in any documentation. If an application crashes for some (potentially unrelated) reason during a read transaction you can easily end up with stale readers and machines hitting their disk quotas.

I accidentally committed this to master.... It's good though..


Issue pushing to Heroku Buildpack Detection

Hello,
Im getting an error when trying to push to Heroku:
remote: -----> Failed to detect app matching https://codon buildpacks.s3.amazonaws.com/buildpacks/heroku/go.tgz buildpack
remote:        More info: https://devcenter.heroku.com/articles/buildpacks#detection-failure
remote:
remote:  !     Push failed
remote: Verifying deploy...
remote:

Do I need more than the heroku/go buildpack?
Thank you.
Keith

Yes, the usual go buildpack is not enough see Get Started in the README.


add subquery pull up rule group ut(I)

Proposed changes
add subquery pull up rule group ut(I): single table part, include:
(1) subquery_pullup_basic: basic part
(2) subquery_pullup_uk: using unique key
(3) subquery_pullup_and: with and
(4) subquery_pullup_or: with or
(5) subquery_pullup_and_subquery: with and subquery
(6) subquery_pullup_or_subquery: with or subquery
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

we may need consider uncorrelated subquery and scalar subquery like bellow:
select * from t1 where t1.k1 (not) in (select t2.k3 from t2)
select * from t1 where (not) exists (select t2.k3 from t2)
select * from t1 where t1.k1 < (select sum(t2.k3) from t2)
select * from t1 where t1.k1 < (select sum(t2.k3) from t2 where t2.v2 = t1.k2)

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


DeepLabV3+ Xception SemanticPredictions output layer corrupt

I can not do inference on a frozen custom trained Xception model. Resnet/Mobilenet is working ok. What might be the difference then? Is there anything special to know about the Xception model output?
During training loss and images look very good. However when using the frozen model the output is always empty.
System information
Platform: Linux Ubuntu 20.04 / Nvidia GPU 8 GB
Python: 3.7
Tensorflow: v1.15.2-30-g4386a66
Model: xception_41
Describe the current behavior
Output tensor for SemanticPredictions is always the same value.
Describe the expected behavior
Output tensor for SemanticPredictions should contain the classes prediction value.
Other info / logs Include any logs or source code that would be helpful to
The model was trained:
python deeplab/train.py --logtostderr --training_number_of_steps=96400 \
  --train_split="trainval"  \
  --train_crop_size="384,384" --train_batch_size=4 --dataset="vehicle" --save_interval_secs=300 \
  --save_summaries_secs=300 --save_summaries_images=True --log_steps=100 --train_logdir=training_xception_41 \
  --dataset_dir=${PATH_TO_DATASET} \
  --scale_factor_step_size=0.05 \
  --min_scale_factor=0.60 \
  --max_scale_factor=0.75 \
  --scale_factor_step_size=0.05 \
  --output_stride=8 --base_learning_rate=0.125 \
  --weight_decay=0.00125 \
  --model_variant="xception_41"

The model was frozen:
python deeplab/export_model.py \
--model_variant="xception_41" \
--num_classes=${CLASSES} \
--dataset="vehicle" \
--checkpoint_path=${toFreeze} \
--crop_size=384 \
--crop_size=384 \
--export_path=graph_xception_41.pb \
--output_stride=8

I can see a difference in output graphs (left Resnet / right Xception):

@thhart,
Issues related to tensorflow/models are tracked in the models repo. Could you please submit a new issue from this link, so that the issue can be tracked there. Thanks!

The same problem applies to nas_pnasnet by the way.
However I can see a difference SemanticProbabilities layer here also empty.
For Xception the SemanticProbabilities layer was not empty.

Moved to: https://github.com/tensorflow/models/issues/9426


translation

minor translation in tarkistus/kontroll SV.js

Merged manually after manual rebase. Released maybe tomorrow


MediaConvert HTTP 415 Unsupported Media Type

MediaConvert is currently configured to use ServiceProtocolType.restjson but this results in HTTP 415 errors.  Switching to ServiceProtocolType.json fixes the issue.

Can you point out where you are changing the service protocol type.
Also can you supply some code that replicates this issue.

I've verified MediaConvert does use a rest-json protocol. The service must be showing an issue in our implementation of the rest-json protocol. Changing the protocol to json could have undefined results.

Can you point out where you are changing the service protocol type.
In MediaConvert.init when the AWSClient is created.

Also can you supply some code that replicates this issue.
So far I have confirmed that both createJob and ListJobs have this issue.  I suspect that all of the calls for this client have this issue but I haven't tested them yet.

You will have to input your credentials and endpoint for this.  Endpoint can be found under MediaConvert > Account.
import Foundation
import MediaConvert

let mediaConvertClient = MediaConvert(accessKeyId: "",
                                      secretAccessKey: "",
                                      region: .useast1,
                                      endpoint: "")

let createJobRequest = MediaConvert.ListJobsRequest()
let response = try mediaConvertClient.listJobs(createJobRequest).wait()
print(response)

Also to be clear I am working with the NIO2.0 branch because of dependency conflicts, however, I did confirm that this code produces the same result with release 3.1.0.

Yeah found a way to re-create. Just ask for some endpoints. It looks like the Content-Type code was wrong for rest-json. I have a another PR on aws-sdk-swift-core https://github.com/swift-aws/aws-sdk-swift-core/pull/110 which resolves this.
As soon as it goes in, I'll pull across to nio2.0 as well.

Thank you very much! 😎👍🏼

Fix is on nio2.0


Adding return types for interface compatible

Q
A

Bugfix?
✔

Breaks BC?
❌

New feature?
❌

Added return types for compatibility with symfony/console v6.0:
https://github.com/symfony/console/blob/6.0/Input/InputInterface.php

Thank you so much!


Added code to prevent challenge level 2 from generating duplicate ans…

…wer choices

Great, thanks, Lainey!


Make 2fa mandatory for all users

hi, i would like to ask i you could make 2fa mandatory for all users.
i have alot of users because multiple sites and i want all of them to have to use 2fa.
thanks

This is a little complicate. What you have to do is to create another role of unauthorized that can access only to the page with the 2fa setup process, after the user will activate his 2fa you need to change programmly  his role to the regular role and to allow him all the other pages.

ok, 10x


Integrate with junit for common coverage?

is there a way to run this cli via junit? The reason is that it will be nice to run it with other tests and collect a common coverage report

Any update on this?
I'm looking for a way to execute http calls on jenkins as part of the deploy pipeline. As devs love the simplicity and speed to setup rest calls for tests, it would be great to use them too for regression tests.


@unocss/preset-icons   does not support class names with capital letters

UnoCSS version
0.62.4
Describe the bug
@unocss/preset-icons   does not support class names with capital letters。
Regardless of whether the icon name is written directly in collections or in the iconify-json file, as long as it contains uppercase letters, the corresponding icon cannot be generated on demand
Reproduction
https://unocss.dev/play/?html=AQKAPAJglgbswD4TCgYwPYDsDOAuZwYUwqANgIbbYC8ARFALQAujGADgJ60JgD0USeERIUqdRiwYBhdBACm3PgPC9oMBEA&config=JYWwDg9gTgLgBAbwFBzgEwKYDNgDsMDCEuOA5gDQpxhQYDOGMAgjDFMAEYCuMwWAnpVQ16jAJIBjYnSHVaDGAFVcESgF84WKBBBwA5FxUS6dPUiQYAHpFjpsAQy4AbeJhz4iJYKQAUyVFDO9ABccADaVKhhehJcdDA6ALSBThh65IhwUk7QoXq0aHpwagC6smVUdAAW0DCxMHSh-qj6sfFJ1bX1enkwVjCJTqRwfZYD0Pa4pBhwNQBuGFDBowN99k7pVGqyIgqN4ZFyokoqPgCUssLyjCxsnDx8-OeXRwqS0n6HLUc4lqEARMBEv8Xqh%2BlB7AAFbRgRa8EKIL4tNDAOhgJz2fgAvBOPAYRIcHISADWIKRqH%2BC1gwAk60S628uH%2BAJAwDQaFS-yR2yR2VSEl40ia5LgNOI9hAEGIoXOcAAvAA%2BUXgWo%2Bf4AATFuEeiQAVnRiAB6LUSqW4Y1SXB0AB0%2BuI-zO1pgVQwuB8PmAZ3lSuA1rcjhcFxFvGF3zDWQgYCx%2BgAPHQ5sM5sAMAB3ABCEEscv%2BAAY4HmAKwARgATHBiyX-gqY7QBXAs-9SwAOf5wfjZ5utlNs53ZgDMfYAbK2Xd4qjB%2B0PW1AGwWAOzT9v-eetnBOJzZlT4VvxbTEjDZ2JQWi4GBEHJQHdsCD7wZ43UQPDZ7SGNBXvf47toXv-PuV6uGrWMDVmA9jOug2YALJ9k2fbWgWcDNk48GJCWAAs9gFoO1pFoO5bYbh%2BZETmiRYaRg4ABJFqW9iDmhOEAJxwHRjHEfmiR0cxaFzCWuGYQReFYTheF5qJ%2BH4VU6GrsA66bsQGDvjeB7-EeJ5nhAF6KbeuL4LSYDPhAr5afiOkYA%2BT7-C%2BuBvnAu5KYkX4-n%2BVYxoaoHOgB8akAqmzht8hqGnAKjwCm0DEnAACEUVRSKqBEJgeRxgmcBJqmGYNmJFblqWLnuVUmgyRu-xbgptnXvuh5cMerrqZpZUfneun2PplmGdZxmNWZj64AZRn1fZjlVP2lYQf8kG4XmQ5NgAMn%2BcAllhTjNok1H-q5eUgWB%2BVrkVJXGZV1Wnue0AdaZem9e1-Xafe3UXTZdm3oNw2tmgUF9gWZZTUtJZNitpZOCtP1-WtblbZt4E7XJ25Xcpqk1cdl4w5152tX1D0mTdFlWfd5Wfj2Q2-iNr1jX2OZoXADGDgDlNwH%2BOYuaDHmuV5Pkimo3KHGoQaoBUXNIEAA&css=PQKgBA6gTglgLgUzAYwK4Gc4HsC2YDCAyoWABYJQIA0YAhgHYAmYcUD6AZllDhWOqgAOg7nAB0YAGLcwCAB60cggDYIAXGBDAAUKDBi0mXGADe2sGC704AWgDuCGAHNScDQFYADJ4Dc5sAACtMLKAJ5gggCMLPK2ABR2pPBIcsoAlH4WAEa0yADWTlBYqEw2yFjK3Bpw5LxxAOTllVDoYpSMYgAs3vUZ2gC%2BmsBAA&options=N4XyA
System Info
unocss and @unocss/preset-icons is the same version  0.61 or 0.62
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

https://github.com/unocss/unocss/blob/35297359bf61917bda499db86e3728a7ebd05d6c/packages/preset-icons/src/core.ts#L70
this bug caused by the Regrex:    /^([a-z0-9:_-]+)(?:?(mask|bg|auto))?$/
it not include A-Z,


Use Particle.js In the Background with Suitable Theme

https://codepen.io/VincentGarreau/pen/pnlso

can u assign this to me?

Hey @SauravMukherjee44, I would love to work on this. Could you Kindly assign it to me?

Go for it!

So, should I remove the bg and use the particle? Or should I replace the bubbles with the particles? Kindly, clear it out for me.

Use particles instead of Bubbles. Background should be same.

https://codepen.io/VincentGarreau/pen/pnlso

https://codepen.io/collection/DPOage for more ideas

Thank you! Will be working on it today.

Hey, do check this out once, and lemme know if this is fine?

This is too much, reduce the intensity.

Change the Colour to cream

I have reduced the density and also changed the color to cream.
Do you want me to change anything else?

Hey, could you add in "Hacktoberfest-Accepted" label to my PR so that it's accepted in the event?


Command lacking explanation

When you get to the "connect to the openSSH server" section, a few key pieces of information are missing, and due to Microsoft bs, this immediately becomes very frustrating. The first command's description is written very poorly and doesn't provide a real example, I am failing to connect and I don't know where to start.
the aforementioned command: ssh domain\username@servername
1. What should the domain be/look like? does being in a workgroup affect that?

Searching for "domain" in the start bar takes you to "Access work or school" which doesn't seem too far off but is of course a waste of time.
Going to system information, nothing is listed under domain.
The "System>about" has a related links section with 3 links (which all take you to the same place ffs)  where one is "domain or workgroup". This takes you to "system properties" (which is not searchable in the start menu) where I can see I am a part of a workgroup WORKGROUP rather than a domain. I can click through to change this but I have no idea if this would cause me issues elsewhere or it is even necessary in the first place.
Looking up domain vs workgroup and you are met with many 1000 word essays on the differences between them. Annoyingly, though they seem to enable similar operations but they are described along the lines of "A workgroup is a whole different ball game from using a domain. A far simpler one." ref.

Since they are a ball game difference, you need to say explicitly if you need to be in a domain. You need to provide a link or give an explanation on where to find that value, if workgroup is an acceptable answer, you need to not word it as strictly Domain
I am assuming that you can use the workgroup name in place for the domain
2. Is the username my username?
Seems like an obvious question but I've been burnt before... I wouldn't think it unreasonable that the openSSH server created its own user when it was created, something that might be documented on OpenSSH which is so fundamental you guys thought it unnecessary to mention.
But alas, the password is asked for and since there is no documentation for any user configuration or passwords, must assume the username and password our my own
except...........................
3. What the f*&^ is my password supposed to be?

Windows has annoyingly defaulted to windows hello sign-in options only. So I don't use a password to sign in.
Annoyingly when you go to add one, you find out the option isn't even visible! Other docs on setting up a password even skip the fact that you have to disable the only windows hello option to be able to do it! had to look that up in another issue someone else posted!
Going through the process of enabling a password to sign into my computer and it would appear that it is the password used for my entire Microsoft account... not just my PC...
I have enabled this, tried this password and I am still unable to connect.
Since it is still not working, I fear that it might be to do with the order of enabling my password after installing and setting up the ssh server but it seems nuts that my microsoft account password has been added to openSSH externally and given that I didn't have a password before or any other password on the machine I really have no idea what the password is or where it is meant to come from.

Again, would be great if you could do more than the bare minimum just so that you don't end up wasting people's time. If it is my microsoft account password (and not one on the PC) then just say that.
4. When you say servername, do you actually mean "computer name"?
Any computer can be a server... you as Windows reference the name of the computer as the computer name all over. You have to be consistent with the naming convention that you've come up with, or at the very least, DESCRIBE WHERE YOU GET THE VALUE FROM.
Fortunately, I assumed this to be the case straight out of the gate, and in testing, I can see that everything other than that doesn't even connect.
30 MINUTES LATER
I took out the domain\ prefix and I was able to sign in. I hope that the time wasted reading up and around this will not be in vain and you will improve your documentation...

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

ID: 18a5a4e5-2604-23c2-1374-68a6974170bc
Version Independent ID: f93809cc-eb5e-6b9b-ed58-c8811a8ee21f
Content: Get started with OpenSSH for Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
Product: windows-server
Technology: administration
GitHub Login: @robinharwood
Microsoft Alias: roharwoo

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Fix misleading error message in pilot-agent

Previously this would log error: <nil> every time an exit was
triggered which was confusing. This makes it so it is only logged when
an error actually occurs.
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?
Courtesy of your friendly test nag.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
58d2fa109572bfcd0563db654a446f9bc5b2e330
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
58d2fa109572bfcd0563db654a446f9bc5b2e330
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
58d2fa109572bfcd0563db654a446f9bc5b2e330
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/assign @hzxuzhonghu

/lgtm


issue122 Clarify LRA context on @Leave methods

#122
The PR adds some text to clarify the LRA context and in particular clarifies the interaction between @Leave and @LRA

Check that this PR is clear about the expected behaviour when the user sets the context in request and response headers.

Spec and Javadoc are clear now around @Leave and @LRA interaction.
Thx @mmusgrov !

@rdebusscher and @xstefank I added more text about the expected behaviour when the user sets the context in request and response headers. Can you re-verify that I still have the approval to merge.

Thanks, I think this will help.
#include<iostream.h> #include<conio.h> #include<fstream.h> #include<stdio.h> #include<stdlib.h> void main() { clrscr(); ifstream fs; ofstream ft; char ch, fname1[20], fname2[20]; cout<<"Enter source file name with extension (like files.txt) : "; 	gets(fname1); fs.open(fname1); if(!fs) 	{ cout<<"Error in opening source file..!!"; getch(); exit(1); } cout<<"Enter target file name with extension (like filet.txt) : "; gets(fname2); ft.open(fname2); if(!ft) 	{ cout<<"Error in opening target file..!!"; fs.close(); getch(); exit(2); } while(fs.eof()==0) { fs>>ch; ft<<ch; } cout<<"File copied successfully..!!"; fs.close(); ft.close(); getch();
Sent from my iPhone

On Mar 28, 2019, at 5:47 AM, Michael Musgrove<EMAIL_ADDRESS>wrote:
@mmusgrov commented on this pull request.
In spec/src/main/asciidoc/microprofile-lra-spec.adoc:

@@ -322,6 +326,35 @@ annotations defined in this specification then the LRA will not be made
available to the invoked method. However, any outgoing JAX-RS invocations
made during the execution of the method will still carry the original context.

+The user is allowed to manually assign the context header on JAX-RS requests
+and responses:
+
+===== Setting the context on outgoing JAX-RS requests
+
+If the context is set on an outgoing JAX-RS request then it becomes the active +context for that invocation. When the invocation returns the active context
+will depend on the behaviour of the remote resource:
+
+- if the remote resouce ends the context (and does not start a new one) then

the active context will be whatever the client was using before it manually set
the context on the outgoing header;
+- if the remote resouce does not end the context (and does not start a new one)
I think the problem is that I am wanting to support "implicit context propagation" where the spec implementation manages the association. We currently have both explicit (where the user is allowed to manage the context) and implicit. I acknowledge that supporting both can be a challenge to specify in a clear and unambiguous way. Manage thread association can also be tricky in reactive environments.

So would people be more comfortable if we say that the user has to manage the context on outgoing requests (I think Martin asked for this a while back) and the container manages the response header according to what is dictated by the @lra annotation.
If so then I will update this PR to remove the requirement for "implicit context propagation". Thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


sort reminder list according to endDate.

Previously, when the reminders are shown in the reminder window, they are not sorted. Now, the reminders are sorted according to their end date.

LGTM 👍

LGTM


Bump Dfinity package deps

EDIT I don't think mismatched versions were the issue, but perhaps some worthwhile changes here nonetheless.
Mismatching versions of @dfinity/agent (0.9.1 / 0.10.0) causes errors when hashing principals. We should bump these versions for that reason, and perhaps bump stoic identity to version 1.5.1.
// Line 30 of @dfinity/agent/esm/request_id.js
else if (value instanceof Principal) {
    return hash(value.toUint8Array());
}

I believe 0.10.0 added a new parameter _arr to the Principal class, so this value instanceof Principal returns falsy. Maybe @dfinity/agent should do a different check, like value.hasOwnProperty(toUint8Array) which would be less error prone here. I'll mention that on the forums. For now I think we should get this in.

Seems there's a broken import with @dfinity/candid 0.10.0... will take a look.

I was receiving the following error, which the latest commit seems to have fixed:
POST http://localhost:8000/api/v2/canister/rrkah-fqaaa-aaaaa-aaaaq-cai/call
422 (Unprocessable Entity)
call @ index.ts:205
async function (async)
call @ index.ts:166
caller @ actor.ts:349
handler @ actor.ts:372
(anonymous) @ account.tsx:19

index.ts:213 Uncaught (in promise) Error: Server returned an error:
  Code: 422 (Unprocessable Entity)
  Body: Could not parse body as submit message: Invalid CBOR encoding: invalid type: map, expected byte array

    at HttpAgent.call (index.ts:213)
    at async caller (actor.ts:349)


Website is not working

Hello,
When I go to your website, all I get is a blank page. Nothing shows up, can you look into this issue?
Thank's in advance.

Same issue for me
Website does load, but shows blank page

Update,
So the site works on Firefox. But for like Google Chrome or Internet Edge, it won't work. Hope this helps with your troubleshooting.
Cheers!


searching in list without seaurch input?

Can you implements when user typing letters, plugin made search in list without search input?

I did not understand. Please, send us a JSFiddle example code simulating this scenario (with runnable code and steps described there, to allow us to follow this steps and check the situation). It is much more easy to understand and help to provide some solution or workaround. You are free to reopen when the JSFiddle are done.


Leaves 'z set after splitting

I managed to reproduce this in both ruby and go, so might not be language specific.
After splitting a line line this:
a(1, 2, 3)

The marker 'z is still set and'z takes me to the first line of that (now split) function call. I have markers showing on the gutter setup (using vim-signature) so having that z show up is a bit distracting. I started looking into making a PR for this but figured I'd make an issue first in case you have a trivial solution for this.

Ah, hadn't thought of clobbering marks. This is the offending code, I think (and one more function like it): https://github.com/AndrewRadev/splitjoin.vim/blob/ac89320db78e5736b0fcce23a3d5ac3c07f3ac25/autoload/sj.vim#L152-L164
Thankfully, it seems pretty easy to restore the previous "'unset" position of the mark once I'm done with it. I've pushed some commits to master that seem to fix the issue for me. Could you try it out and let me know if works for you as well?

Works for me! Thanks for the quick reply/fix!


create dark mode

Create a color palet and implement dark mode on this app

Hello, I would like to help. Can you assign me this issue to me?

Hello, I would like to help. Can you assign me this issue to me?

sure, i assigned you


Brew install Dracula-terminal

Installing Dracula theme via homebrew results in the following error:
The file /Users/aweife/Dracula.terminal does not exists.
It should install the theme on where it has downloaded the cask.

I have the same issue using macOS Ventura and Homebrew 3.6.16.

My guess is this is just a warning, because the theme shows up in the terminal configuration panel. Nonetheless, two contradicting statements seem to be bad:
The file /Users/sid/Documents/nerd-stuff/dotfiles/Dracula.terminal does not exist.
🍺  dracula-terminal was successfully installed!

Did you guys figure out how to fix this? I'm having the same issue on Ventura 13.2.1

Just tried and having the same issue. Attempted running the command in a different folder (in case permissions were an issue) like sydrawat01 and no difference. No files created.

13.4.1 Ventura, same failed result. I ended up just using the zip. A few more clicks than some may like.


Remove unnecessary files for distribution

Working towards #38.

MERGED:

Most cleaning up is in place.
Results in working Mu on both macOS and Windows.
There are probably a few other odds and ends that can be deleted.
#49 was created to include the option of not distributing the source *.py files.


Configuration to disable script nonce

The auto-added script tag nonce attributes are not compatible with a secure_headers configuration in which 'unsafe-inline' is present in the CSP for script-src. See #462.
Some work has been done in other PRs to address this issue --- see #478; this PR is a suggestion for a simplest-possible solution to the problem. Specifically, allow the Rollbar configuration to mandate that the nonce attribute should not be used.

@spdawson thanks for this, I think we should resume #478 and merge it

This should be closed by https://github.com/rollbar/rollbar-gem/pull/478


Ws.chat_limit now fetches its value from the config

Added chat_limit in the config, used by the Ws class.
Added the associated question in basic_questions in questions.py

This fixes https://github.com/zayKenyon/VALORANT-rank-yoinker/issues/129


File upload fails in Chrome with net::ERR_ACCESS_DENIED

Working fine but not in Chrome. Once I drop a file on the dropzone I get POST http://mydomain.com/api/upload/ net::ERR_ACCESS_DENIED. It looks like a browser problem. According logs the request doesn't touch the server.
Anyone? Thank you.
Chrome 75.0.3770.80 on Ubuntu, but it appears on Windows as well.
EDIT: The same error I get on the demo site https://rowanwins.github.io/vue-dropzone/docs/dist/index.html#/demo

@gotroch That's weird, looking into it.

I'm on v75.0.3770.100 on a mac and I'm not seeing anything unusual

Having also a similar issue with chrome 76 on windows

Hello,
Do we have any solution for this? I am stuck on this, couldn't find and solution yet.
I am facing the same issue in electronjs, I am building win/mac app via electronjs, when I start the app intially drag & drop works fine but when I reload the app(cmd+r) then drag & drop stops working with "ERR_ACCESS_DENIED".
Protocol: file://

Do we have any updates on this? Stuck up on this issue for like 4 months :(
Quick help would be appreciated! TIA


Shutdown standalone parse server gracefully

Capture SIGTERM and SIGINT signals and close the server before exiting.

Current coverage is 90.56%

Merging #958 into master will decrease coverage by -0.79% as of 1f62d0e

@@            master    #958   diff @@
======================================
  Files           73      73
  Stmts         4324    4324
  Branches       863     863
  Methods          0       0
======================================
- Hit           3950    3916    -34
  Partial         10      10
- Missed         364     398    +34

Review entire Coverage Diff as of 1f62d0e

Powered by Codecov. Updated on successful CI builds.


Destroy on dereference

The context managers are wonderful, however sometimes you want the temporary directory or file to exist beyond the life of one function.
https://pypi.python.org/pypi/tempdir provides this for directories, with __del__, and I am using that package for https://github.com/cpennington/nosetrim/pull/2 , but that package isnt in public source control and doesnt have tests, etc., so it would be nice to have a better maintained dependency for this.

The link to the tempdir source is dead it seems (from PyPI) and I am not sure I understand fully what you intention is. From what I gather you want a way to keep a temporary directory or file around  past the content manager and want this to be done by del mytmpfileordir?
Two things come to mind here: First, do we really want this functionality, after all, temporary directories are supposedly, well, temporary. Second, is del the mechanism we want for this?
I could see add a way to prevent the cleanup inside __exit__ and maybe even a way to manually trigger it afterwards - I'd think there are some ways to do it already though, either by calling __enter__ and __exit__ manually or handling them with an ExitStack. I would assume that https://docs.python.org/3.5/library/contextlib.html#contextlib.ExitStack.pop_all basically does what you want here already.
Using del to disable the cleanup is a bad idea, it does exactly the opposite of what one would expect ("delete my temporary directory" should actually remove it, right?). So either it would have to be an extra method or nothing (+ maybe an example for the ExitStack method).

No. The object would automatically remove the temporary dir/file during the objects garbage collection, by having a __del__ method. You can download the tempdir tgz from pypi to see it.

bump, I'd really like to see this feature too. It seems this lib was only test and made for usage with context manager, and I get an error when I try to just
file = volatile.file()
file.write(b'123')
file.close()

full example
import json
# import tempfile

import volatile

class MouseRecording(list):
    def __init__(self, *args, **kwargs):
        # self.file = tempfile.NamedTemporaryFile(mode='w+', prefix='my_module_', encoding='utf-8')
        self.file = volatile.file(mode='w+', prefix='my_module_')
        super().__init__(*args, **kwargs)

    def __del__(self):
        self.file.close()

    @property
    def file_path(self):
        return self.file.name

    def serialize(self):
        json.dump(self, self.file)
        return self.file_path

    @staticmethod
    def deserialize(file_path):
        try:
            recording = json.load(open(file_path, 'r'))
            return MouseRecording(recording)
        except FileNotFoundError as exception:
            print(f'file not found: {file_path}')
            return MouseRecording()

if __name__ == '__main__':
    from time import time

    recording = MouseRecording(
        [(1, 1, time()), (500, 500, time())],
    )

    path = recording.serialize()
    recording2 = MouseRecording.deserialize(path)
    print(recording == recording2)

Ah, now I see it, you want to use temporary files without handling them with context managers. I'll give this a look later!

I think relying on the destructor to clean up temporary directories is a bit too unreliable, especially in implementations outside of CPython.
You can always call __exit__ instead of close to close/remove dir though, here are some minimal changes:
import json
import volatile

class MouseRecording(list):
    def __init__(self, *args, **kwargs):
        self._file_ctx = volatile.file(mode='w+', prefix='my_module_')

        # "Enter" the context manager
        self.file = self._file_ctx.__enter__()

        super().__init__(*args, **kwargs)

    def __del__(self):
        # "Exit" the context manager
        self._file_ctx.__exit__(None, None, None)

    @property
    def file_path(self):
        return self.file.name

    def serialize(self):
        json.dump(self, self.file)
        self.file.flush()
        return self.file_path

    @staticmethod
    def deserialize(file_path):
        try:
            recording = json.load(open(file_path, 'r'))
            return MouseRecording(recording)
        except FileNotFoundError as exception:
            print(f'file not found: {file_path}')
            return MouseRecording()

if __name__ == '__main__':
    from time import time

    recording = MouseRecording(
        [[1, 1, time()], [500, 500, time()]],
    )

    path = recording.serialize()
    recording2 = MouseRecording.deserialize(path)
    print(recording == recording2)

I recommend against doing this, but if you are willing to accept this, you can write a small wrapper class yourself to factor this out:
class CtxWrap:
    """Given a context manager, enter it and return the returned
    value as `value`.

    Will close the context on deallocation."""

    def __init__(self, ctx):
        self.value = ctx.__enter__()
        self.ctx = ctx

    def __del__(self):
        self.ctx.__exit__(None, None, None)

You can even go crazy with it:
class CtxWrapMagic:
    """Like `CtxWrap`, but magical (hides itself)."""

    def __init__(self, ctx):
        self.value = ctx.__enter__()
        self.ctx = ctx

    def __getattribute__(self, attr):
        value = object.__getattribute__(self, "value")
        return getattr(value, attr)

    def __del__(self):
        ctx = object.__getattribute__(self, "value")
        ctx.__exit__(None, None, None)

if __name__ == "__main__":
    import volatile
    import os

    tmpfile = CtxWrapMagic(volatile.file())
    tmp_path = tmpfile.name
    assert os.path.exists(tmp_path)

    del tmpfile

    assert not os.path.exists(tmp_path)
    assert False

I recommend against it though ;)


DP module - use billing_account_id null value for delayed billing association

Describe the bug
If in "stage 3 - data platform" the parameter "billing_account_id" is set to null, it assumes that the projects are already created. This is a different behavior to all other stages.
Expected behavior
Setting "billing_account_id" to null should create projects without billing accounts attached

This was fixed in #1992


Test context usage

@jrobsonchase since I messed this up last time I touched it, you mind taking a look at this PR?
I've moved the transmuting into functions that explicitly just do the lifetime changes that they were meant to do and switched the pointer-to-ref conversion to use a normal reference-of-dereference. I also added a test that will actually call the waker, and verified I got a SIGBUS when calling the test with the implementation before your fix (obviously not guaranteed to fail, but I think there's a decent chance to either cause a memory fault or hang forever because of the missed wakeup).


Change the default password char in Password Mode

Hi,
Thanks for a great library.
Quick question: Is it possible to change the default password character in Password Mode?
Kind Regards,
JP

@jpvdm1 not at this time, no. That functionality was originally there but I removed it cos I had some problems with it. I'm planning on a 2.0 release for next month that could introduce some breaking changes. You're more than welcome to influence the design decisions around this feature