memory handling on large datasets

Hi, I have some data from a Nextseq run, about 370M reads.
My VM is 16 cores, 128GB of RAM, but during the Mapping process the software quickly consumed all of the available memory and crashed.   See the attached chart, the memory consumption seems to grow exponentially.

I'm interested to hear any suggestions on how to get this data analyzed.   Any idea how to determine how much RAM is required?   Or is there a way to get it done within the limits of this machine.    Can the software even support this?
Thanks!!

Hello @bridenrd,
large dataset are a challenge for CITE-seq-Count because the dict is completely stored in memory.
One solution that I have started to look at is to use a reddis implementation and store is on the drive instead. This might take a while to implement because I don't have much time to work on CITE-seq-Count these days.
I'm going to add a section on the README looking for someone who would know how to implement a reddis version and hope I find someone to help out to get it faster.
For now sadly, I don't have a good solution for this issue.
I've heard good things about BUStools and it should handle big datasets pretty well and in a timely manner.
I hope this helps.

Forgot to mention, using less cores will also use less memory since each dict is duplicated for each core. Maybe you can find a sweetspot that will run slower but will deal with the memory issue.

Thanks for the quick feedback Patrick.   I tried fewer cores but it didn't complete.   The dict object just doesn't scale well, looks like some people go with SQLlite.  I hadn't heard of redis, but will check it out.  Also looking into Kallisto/Bustools.    Best regards  --rob


Multiline clause support for input redirection mode

It seems that in input redirection mode, multiline clause is executed separately instead of a whole.
For example, for following shell code:
tpl <<EOF
fail,
write(non_reachable), nl
EOF

Trealla's output:
Trealla v0.1alpha, 64-bits (int=128-bits), Dec  5 2016

(0.000 s) no
non_reachable

(0.000 s) yes

BTW, it would be nice to make Trealla support consulting source from here document instead of file, so that pieces of Prolog code can be embedded in shell script directly. Currently, input redirection is for REPL, it's nice for IDE integration, but a bit clumsy for scripting.

I'll have  a look at last part...

Did you mean:
./tpl
[user].
blah blah
CTRL-D
???

I mean to embed Prolog code into a shell script, e.g. file foo.sh:
#!/bin/sh

ARG=$1

tpl --goal=main --consult - <<EOF
main :- write($ARG), nl.
EOF

Here --consult - is used to mean consulting content from stdin as a file, as meaning of tpl without any option has alway been taken by REPL.

Affected by changes made for #56.

This one is also back.

Something is still not quite right, e.g.:
tpl <<EOF
A = 1.
B = 2.
EOF

Output:
Trealla v0.1alpha, 64-bits (int=128-bits), Dec 18 2016
 A: 1
(0.000) true.
DEBUG: orphaned=7

Two problems:

Result of B = 2 is lost.
orphaned warning.

I'll reopen this.

Orphans were the missing term(s).

Now a segmentation fault is triggered for this test case.

This works too:
./tpl <<EOF
A=1.
B=2,
C=3.
EOF
Trealla v0.1alpha, 64-bits (int=128-bits), Dec 18 2016
A: 1
(0.000) true.
B: 2 C: 3
(0.000) true.

Do you mean devel branch?
I checked that branch, both your example and following code run correctly:
tpl <<EOF
A = 1.
B = 2.
EOF

But following code still cause segmentation fault:
tpl <<EOF
fail,
write(non_reachable), nl
EOF

Ok, this one:
tpl <<EOF
fail,
write(non_reachable), nl
EOF
needs a final dot. I will fix that...

I'm not aware of that...
An error message instead of segmentation fault is enough.

Now does:
$ ./tpl -q <<EOF
fail,
write(unreachable)
EOF
ERROR: parse -> EOF
false.
DEBUG: orphaned=6
So i will do a bit more work on it.

Follow shell script will cause segmentation fault:
tpl -q <<EOF
halt
EOF

Following is backtrace:
Core was generated by `tpl -q'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x000000000040c0f6 in xref_rule (l=0x6578a90, n=0x0) at src/trealla.c:2477
2477            node *tmp = NLIST_FRONT(&n->val_l);
(gdb) bt
#0  0x000000000040c0f6 in xref_rule (l=0x6578a90, n=0x0) at src/trealla.c:2477
#1  0x000000000040c922 in query_parse_file (self=0x66970c0, src=0x0, fp=0x7fac0dc9c4e0 <_IO_2_1_stdin_>) at src/trealla.c:2640
#2  0x0000000000404842 in main (ac=2, av=0x7fff963289c8) at tpl.c:388

Can be closed now.


Delete deadcode

The elif clause is never executed.

@vvalgenti: Could you correct the outer elif? Then I'll close this pull request.


Send Ctrl-C keystroke to signal stream.

This is to match the (correct) web_shell terminal behavior
@ussuri

LGTM


Task - Edit TaxonName - duplicate taxon

We already have option to add a child taxon or add a sister taxon. Another option is needed: duplicate a taxon.
This is extremely useful in cases of original misspelling or genus and subgenus:
The attributes which need to be duplicated:

name
parent
rank.
author
year
original source
original combination relationships

@jlpereira steps:

[ ] Add Green 'Clone' button b/w Save and New.  it is always present, but disabled until there is an ID for this record
[ ] When user clicks clone, then they are prompted with confirmation dialog
[ ] When user accepts POST with the params listed above to create a new record
[ ] Then navigate to that record
[ ] Add hotkey combination to trigger, hotkey also brings up dialog

Confirmation dialog can not be disabled.


Release/0.32.0

Release Changelog:

Feature: Build all options values (#457)

Feature: Run custom pip command when updating Conan and CPT in Docker container (#451)

Feature: Support Conan lockfiles (#466)

Feature: Add support for GitHub Actions (#474)

Feature: Manages header_only option in recipes (#454)

Feature: Configure Tox for Windows CI (#453)

Feature: Update six python package to 1.14.0 (#461)

[x] Refer to the issue that supports this Pull Request.

[x] If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.

[x] I've read the Contributing guide.

[x] I've followed the PEP8 style guides for Python code.

conflicts ⚠

@uilianries Why did you remove 16?

My mistake! Actually, I tried to introduce more tests, but I reverted manually. I gonna fix it, thanks @Croydon


how to get a new driver release?

How do I get a new driver into falco?  It is failing with :

Trying to download a prebuilt falco module from https://download.falco.org/driver/17f5df52a7d9ed6bb12d3b1768460def8439936d/falco_amazonlinux2_5.4.201-111.359.amzn2.x86_64_1.ko

@dwgillies-bluescape: There is not a label identifying the kind of this issue.
Please specify it either using /kind <group> or manually from the side menu.

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.

This driver seems to come from an older commit hash, did you try running the latest version of Falco? Since the last few Falco releases, we are supporting more kernel versions in our drivers.

Nope, sorry, still no good.  We have a govcloud service and are forced by an amazon security advisory to use the very latest amazon linux driver, we recompiled roughly Aug 5th, and are now looking for :
* Looking for a falco module locally (kernel 5.4.204-113.362.amzn2.x86_64)
* Trying to download a prebuilt falco module from https://download.falco.org/driver/2.0.0%2Bdriver/x86_64/falco_amazonlinux2_5.4.204-113.362.amzn2.x86_64_1.ko

I just tried the falco:0.32.2 and falco:latest docker images, and the driver compilation has only advanced up to amazonlinux2_5.4.196:

This is one of the first times we are forced to upgrade our kernel by a nessus scan / admazon security advisory.  If these advisories come out periodically, and falco is always a month or two behind on amazon linux, then we will have to look for something more up-to-date than falco to monitor our cluster, regrettably.

@dwgillies-bluescape: There is not a label identifying the kind of this issue. Please specify it either using /kind <group> or manually from the side menu.
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.

@poiana I cannot edit the Issue label BTW.

Hi, our code is working again, from the driver built at 10:57 GMT today.  THank you for helping us in this matter.
2022-08-12T10:57:23.000Z        3.8 MB         [falco_amazonlinux2_5.4.204-113.362.amzn2.x86_64_1.ko](https://download.falco.org/driver/2.0.0%2Bdriver/x86_64/falco_amazonlinux2_5.4.204-113.362.amzn2.x86_64_1.ko)
2022-08-12T10:57:22.000Z        4.7 MB         [falco_amazonlinux2_5.4.204-113.362.amzn2.x86_64_1.o](https://download.falco.org/driver/2.0.0%2Bdriver/x86_64/falco_amazonlinux2_5.4.204-113.362.amzn2.x86_64_1.o)

@dwgillies-bluescape great to hear you solved your issue. Can we close this?
For the future I think this kind of issue is more suitable for falcosecurity/falco or falcosecurity/test-infra. Also, we provide kernel driver builds in a best-effort way, and it's almost impossible to have a perfect build matrix with all the most up-to-date kernel flavor versions. falcosecurity/kernel-crawler from @FedeDP mitigates this a lot, but sometimes is not enough. If update kernel version very frequently, I would suggest building the drivers on your own. The community is happy to help if you need guidance on this.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

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.


Improve blocks serde

Improvement over https://github.com/prestosql/presto/pull/6018.
Added deserializing blocks of 8 at once for all blocks except int128
Benchmarks in commit messages

comments addressed

merged, thanks!


Broken Chip

When i add a chip in my project, one side is rounded and other is not rounded but when i run the same code as a seperate program the chip is displayed properly can you explain y ?

same here +

@AlexanderThiele please help left icon is not rounded. I can send you screenshoot add me<EMAIL_ADDRESS>


Trigger release v0.2.0, publish to open-vsx.org

Are we ready to try the new release/publish infrastructure?

The following 1 GitHub releases will be created once this pull request got merged.
Release v0.2.0 with changes since v0.0.0

Trigger release v0.2.0, publish to open-vsx.org cde271c
Merge pull request #22 from eclipse-cdt-cloud/publish 08e17dc
Update for review comments 46ebcc9
Add infrastructure to release/publish as part of CI f183ad3
Register to vscode-trace-extension as a trace server contributor bd538d7
Merge pull request #21 from eclipse-cdt-cloud/update-eslint-dependency 1272300
[3pp][license] update eslint to approved version 03fb997
Merge pull request #18 from eclipse-cdt-cloud/automated-pr-license-check b7d0c3b
Add 3PP license check as part of PR CI f6b8480
Add minimal build/test workflow 5601eb7
Fix stopping trace server during shutdown 4ebbb2d
Upgrade vsce to the current latest version 2.20.1 0fceb92
Remove .git directory from packaged plugin f9b8319
TraceServer: Fix prettier formatting 2a7d29f
Make start and stop commands real async commands 1bb1405
Fix error message after start/stop for missing vscode-trace-extension 5fc4471
Get server URL from the client preferences (vscode-trace-extension) 794d1fe
TraceServer: Realign the start/stop message format 6fff008
TraceServer: Also show error detail when available a021f59
TraceServer: Check server status if survived error 73b96b3
TraceServer: Only reset upon stop while pid not UP f8a27c3
TraceServer: Fix case of starting upon stored pid 4017826
TraceServer: Remove extra server-exit stopOrReset 3828598
TraceServer: Show warnings upon no-op user command ba659b3
TraceServer: Add an explicit foreign server check 2c7c0f1
TraceServer: Make get-settings/serverUrl reusable a56e497
TraceServer: Make the 'UP' server check reusable 20eac28
TraceServer: Add resetting to stop command error 91f8f9b
TraceServer: Show error messages beside logging it 5855d1d
extension: Restore vsce package as a devDependency f9375aa
extension: Add vsce back as current @vscode/vsce 7595d69
extension: Temporarily remove the old vsce package c2a6cef
extension: Fix locally unresolved defaultBuildTask d7b952c
TraceServer: Show messages about started/stopped be99123
extension: Store updated server pid in workspace ade4492
extension: Add yarn script for vsce/vsix packaging 83a9b5c
extension: Call stop directly upon deactivate/exit 73ba3df
extension: Run (compile) webpack upon 'yarn' too 3ea2063
extension: Make the trace server Compass-agnostic 2631e43
MD files: Align both standard ones with siblings 13512c5
README: Add note about removal for Blueprint amend 94ca658
README: Remove extra 'locally' in prettier blurb 7c4d925
README: Lightly clarify the possible installations 56e426c
Prettier: Mention --check option also in README.md 8241947
Prettier: Add minimal yarn command w/ README doc cad20a8
Prettier: Align with trailingComma set to 'none' 3253e13
Prettier: Align with tabWidth set to 4 bc1fc91
Prettier: Align with singleQuote set to 'true' ac6c4a5
Prettier: Align with printWidth set to 120 7756eaa
Prettier: Align with arrowParens set to 'avoid' a462da5
Prettier: Begin aligning settings with team usage f9eb64e
Add Server suffix to extension configuration title 194deb3
README: Add section about debugging the extension f20e892
TraceServer: Set the server status bar item if any d73d84e
extension: Remove the unstable (re)start command e14356b
TraceServer: Show progress notification with stop 21da39b
TraceServer: Fix unnecessary client field w/ const 88ba94c
TraceServer: Show progress notification w/ startup 5e0f3a1
TraceServer: Refactor duplicated error message end c5b5235
TraceServer: Check proper stopping of known server 8505b92
TraceServer: Internally stop upon an external exit 4ae2690
TraceServer: Always properly stop for next restart add86e8
TraceServer: Remove currently unused async/Promise 5261909
TraceServer: Remove the redundant public keyword-s 8c8a5a4
test/suite: Refactor duplicate test title prefixes 14b7017
extension: Add trace server startup health checks 262fed4
extension: Refactor duplicated extensionId strings 49f4f12
extension: Add a start-if-stopped (server) command 909d98d
extension: Add trace-server configuration tests f03fdd3
TraceServer: Remove getConfiguration duplication 3f90017
extension: Make trace server options configurable 46b0d5e
extension: Make trace server binary configurable 60d87a0
README: Add sub-section about supported deployment d388fe4
.vscode files: Mention their (generating) origin b0c8a3c
extension: Replace boilerplate with initial tests b63b219
extension: Replace boilerplate w/ early start|stop 27c2e4b
extension: Specify extensionKind as workspace only 840b201
Reformat the once generated ts files using VS Code 3d37b01
extension: Add upcoming repository to package.json 9f39de0
Reformat webpack.config.js w/ Prettier in VS Code 06581ae
Sort .vscodeignore the same standard gitignore way 8e365bc
Reformat generated .vscode/*.json files in VS Code 6c48101
README: Replace template w/ initial documentation b12b309
Sightly adjust .eslintrc.json to helloworld-sample 4c717de
Align .eslintrc.json w/ most of sibling extension d8f2690
extension: Adjust tsconfig.json to sample repo one 44167b6
README: Fix vsce by removing templating sentence a355ccd
extension: Amend package.json based on sibling's 837a753
extension: Init markdown files based on sibling's 314ab1f
extension: Fix the initially reported markdownlint 9ee36b9


Support multiple renderers running simultaneously

In a similar way than systems, renderer could be written as plugins.
Might also be great to be able to run them in parallel (not multithreaded, just, say, two renderers at the same time, and have both images, even if it destroys timings)

This as been partially introduced by PR #215, dynamic switch during the application life is requested in #187.


Memory Leak?

Hello, thanks for the great work!
I was wondering if there are any memory leaks? Maybe somewhere in the dataset/dataloader code? When I run the training code I get this plot of the process memory available.

Your help would be much appreciated!

Thanks for reaching out!
Yeah the RLDS data loader is notorious for filling up memory over time. Technically it saturates after some time, but in practice we restarted (resumed) training every couple days in case it crashed with an OOM error.
Fixing the underlying RLDS memory consumption is probably very hard (we weren't successful so far, even in the original Octo project where we built the data loader).
If resuming is a big problem in your case, you can either reduce the shuffle buffer / number of data loading threads in the config or (more painful) convert things into a non-RLDS data format (like hdf5) which should bring down memory consumption a lot, but would require a full rewrite of the data loading infrastructure which we did not want to do.


Registering the plugin with TensorRT

Hi, I have successfully used your tools to create a tensorrt engine. The problem is that the plugin is not found when loading the engine with:
import tensorrt as trt
with open("sample.engine", "rb") as f, trt.Runtime(trt.Logger()) as runtime:
    engine = runtime.deserialize_cuda_engine(f.read())

I get the following error:
[TensorRT] ERROR: INVALID_ARGUMENT: getPluginCreator could not find plugin L2Norm_Helper_TRT version 1
[TensorRT] ERROR: safeDeserializationUtils.cpp (259) - Serialization Error in load: 0 (Cannot deserialize plugin since corresponding IPluginCreator not found in Plugin Registry)
[TensorRT] ERROR: INVALID_STATE: std::exception
[TensorRT] ERROR: INVALID_CONFIG: Deserialize the cuda engine failed.

How should I register the plugin? Thanks.

Hi @jruizaranguren . I'm running inference using c++ api. For python, i believe there should be pybind bindings implemented as described in docs.

You can use the C++ API to create a custom layer, package the layer using pybind11 in Python, then load the plugin into a Python application.

After implementing this, you should compile .so with plugin and python bindings and import it.

Thank you. We got it working.

Hi @jruizaranguren . I'm running inference using c++ api. For python, i believe there should be pybind bindings implemented as described in docs.

You can use the C++ API to create a custom layer, package the layer using pybind11 in Python, then load the plugin into a Python application.

After implementing this, you should compile .so with plugin and python bindings and import it.

Hi r7vme,
I'm having the same  issue as jruizaranguren. I'm trying to load the engine file with python but the plugin is not found.
Could you help me solving this problem? In which file I have to insert the python  bindings?
Thanks.

Thank you. We got it working.

hi! did you compiled .so file and connected the plugin to Python?

Hi @jruizaranguren . I'm running inference using c++ api. For python, i believe there should be pybind bindings implemented as described in docs.

You can use the C++ API to create a custom layer, package the layer using pybind11 in Python, then load the plugin into a Python application.

After implementing this, you should compile .so with plugin and python bindings and import it.

Hi! Could you help me with advice on how to create a. so file for the plugin?

Thank you. We got it working.

I have come across the same issue, could you please show us the steps to import the plugin to the registry in Python


[connectivity_plus] Add missing Gradle wrapper, dependencies bump

Description
Found out that ConnectivityPlus for Android had no Gradle wrapper config pushed. To be consistent with other packages and to not rely on local Gradle version some maintainer might have I added gradle-wrapper.properties.
Also, went through other platforms and did some updates/clean ups for dependencies used.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

Quick follow-up question - is there a reason why all plugins packages for Android are on Java? Because they were inherited this way from Google or there is some specific reason? If it is just because it was historically and no arguments for keeping just java I would love to rewrite Android packages to a proper Kotlin code. I believe it will make the source code cleaner.

Quick follow-up question - is there a reason why all plugins packages for Android are on Java? Because they were inherited this way from Google or there is some specific reason? If it is just because it was historically and no arguments for keeping just Java I would love to rewrite Android packages to a proper Kotlin code. I believe it will make the source code cleaner.

The code was originally developed by Google (pretty early when Flutter came up). Same problem with the iOS projects as they use Objective-C.
I don't see why we shouldn't migrate them to Kotlin (as it will facilitate contribution). Feel free to do it!

Thanks for this contribution! I will take care of releasing the packages.


Route meta fields

Hi there,
Is it possible to define route meta fields as per https://router.vuejs.org/en/advanced/meta.html ?
Cheers.

I think nuxt.js use the middleware for check or do something before route .
you can define middleware in you page vue file
index.vue
export default {
  middleware: 'auth'
}

middleware/auth.js
export default function ({ store, redirect }) {
  if (store.getters.isAuth) {
    return redirect('/')
  }
}

or check this Auth0 example form nuxt.js

The thing i'm looking for is a per-route config that i can access in my layout, which changes certain things visually based on the route. Before migrating the project to nuxt I achieved this using by setting data on the route as per https://router.vuejs.org/en/advanced/meta.html

Hi @samturrell , have you tried to use store with getters?

@alexchopin would it be difficult to expose to meta property on a page component? Trying to implement a page crumbs implementation right now and being ability to set the meta property on the page routes would greatly simplify this.

Hi @uptownhr,
I am actually implementing meta property in page components and will be available in next release :)

@uptownhr actually it's in `app/utils.js, see https://github.com/nuxt/nuxt.js/blob/dev/lib/app/utils.js#L96
Actually you can play with it easily by adding a meta property in your page and ready it in your middlewares (ex: https://github.com/nuxt/nuxt.js/blob/20b548a17523e54d7f24cede4ada358d359e6f78/test/fixtures/basic/middleware/meta.js)

Oh nice. Let me give it a shot.
On Tue, Feb 6, 2018, 4:18 AM Sébastien Chopin<EMAIL_ADDRESS>wrote:

@uptownhr https://github.com/uptownhr actually it's in `app/utils.js,
see https://github.com/nuxt/nuxt.js/blob/dev/lib/app/utils.js#L96
Actually you can play with it easily by adding a meta property in your
page and ready it in your middlewares (ex:
https://github.com/nuxt/nuxt.js/blob/20b548a17523e54d7f24cede4ada358d359e6f78/test/fixtures/basic/middleware/meta.js
)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nuxt/nuxt.js/issues/466#issuecomment-363405384, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-UtlpJN3b-BOURwv6CQL5zDYGX2m9aks5tSEL_gaJpZM4MsBc0
.

There is a good practice here:
https://github.com/nuxt/nuxt.js/issues/1687#issuecomment-331870619


Incoming trade requests don't trigger event listeners

I've followed the tutorial up through Chapter 2, but although I've been able to bring my bot account online and change its status, the trade requests don't trigger the event listener.

I should make this more clear in the guide. You need a premium account (i.e. an account with at least a $5 purchase on Steam) in order to perform trades and have access to the API. See #45.

Sorry, I should've clarified as well. I went through all of the steps to verify the bot - made a $5 purchase, waited over two weeks, etc etc. I am able to make a trade while logged into the bot and I'm also able to make a trade with the bot itself, but the event listener just doesn't trigger.

Interesting, alright. A couple more questions to sift out some of the easy solutions:
Is the bot logged in before you send the trade offer? Does the logged in message appear in the console?

Yes, the bot authenticates and the 'loggedOn' event listener is called properly. There are no errors on the confirmationChecker or anywhere else. I put a console.log in the 'newOffer' callback but it's never reached when a trade is made.

Right, I figured out the problem. I was using an old identity secret from a different account, which for some reason wasn't throwing an error when used in the confirmationChecker. My bad.

Yea, it silently fails. Glad you got it working!


All exceptions in Befunge code should throw an Exception in Python code

All exceptions in Befunge code should throw an Exception in Python code.

Exceptions throw special Exceptions


macOS Catalina 15.6 extra float space

Hello, when I use Yabai-related menu bar plugins again, I find that the number of desktops is always one more. At first, I thought it was a plug-in problem. Then I looked at their source code, they all used $/usr/local/bin/yabai -m query --spaces to get the number of spaces, and then I ran this command and found that it does have extra float window.
screenshot

Can you reproduce this after a reboot?

Can you reproduce this after a reboot?

Yes, a restart will also appear

The query output you read is the result of a macOS API; something wonky is going on with your setup.
Try to destroy and recreate all spaces (on all connected displays).

The query output you read is the result of a macOS API; something wonky is going on with your setup.
Try to destroy and recreate all spaces (on all connected displays).

Thank you for your reply, but I don't know how to destroy the hidden spaces. I can only query how to destroy the spaces displayed in the spaces bar. Can you tell me how to do this?

If you have the scripting addition installed, you can run yabai -m space <index> --destroy.
With the information from your screenshot that'd be yabai -m space 3 --destroy.

If you have the scripting addition installed, you can run yabai -m space <index> --destroy.
With the information from your screenshot that'd be yabai -m space 3 --destroy.

  Doesn't seem to work

Closing as it is not an issue with yabai. Not sure what else you can try though.

I have the same issue. When I try to delete the "ghost" space, it complained cannot destroy a macOS fullscreen space.
My OS version is macOS Catalina 15.7.

I have the same issue. When I try to delete the "ghost" space, it complained cannot destroy a macOS fullscreen space.
My OS version is macOS Catalina 15.7.

This problem is caused by the dashboard space functionality that used to be implemented in macOS. Dashboard was removed in Catalina or Big Sur and this is a bug on their part. Im not really sure how to fix it, but you can read my comments in this issue: https://github.com/koekeishiya/yabai/issues/721

This problem is caused by the dashboard space functionality that used to be implemented in macOS. Dashboard was removed in Catalina or Big Sur and this is a bug on their part. Im not really sure how to fix it, but you can read my comments in this issue: https://github.com/koekeishiya/yabai/issues/721

Thanks @koekeishiya!
It works, The problem has been solved by:

Remove the ~/Library/Preferences/com.apple.spaces.plist file.
Log out and log in again.

Thanks @koekeishiya!
It works, The problem has been solved by:

Remove the ~/Library/Preferences/com.apple.spaces.plist file.
Log out and log in again.

This problem is caused by the dashboard space functionality that used to be implemented in macOS. Dashboard was removed in Catalina or Big Sur and this is a bug on their part. Im not really sure how to fix it, but you can read my comments in this issue: #721

Thanks  @wenjie1991  @koekeishiya ，I'm at work now. I'm trying to use my own computer when I get off work.

This problem is caused by the dashboard space functionality that used to be implemented in macOS. Dashboard was removed in Catalina or Big Sur and this is a bug on their part. Im not really sure how to fix it, but you can read my comments in this issue: #721

Thanks  @wenjie1991  @koekeishiya ，I'm at work now. I'm trying to use my own computer when I get off work.

I also meet this issue . it's caused by an old app called dashboard before majave version but is removed after.
Now, you can enter the following command to force the dashboard to close
defaults write com.apple.dashboard mcx-disabled -boolean YES && killall Dock
it works for me.


Snapshot.restore called before afterNavigate

Describe the bug
According to @Rich-Harris: "Snapshots are restored immediately after afterNavigate"
https://github.com/sveltejs/kit/pull/8723#issuecomment-1423522635
This is unfortunately not the case.
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-bmlwww?file=src%2Froutes%2F%2Bpage.svelte
Steps to reproduce in the stackblitz:

Open the chrome dev tools to the console
Click "About" on the page
Click the browser back button

Expected:
The console should log:
afterNavigate
snapshot restore

Actual:
snapshot restore
afterNavigate

Logs
No response
System Info
See runtime/package versions in stackblitz.

Severity
Events are not happening in the order that is advertised.
I can work around it with a icky setTimeout...
Additional Information
No response

I just updated all the packages in the stackblitz to the newest and confirmed that the issue is still there.
"@sveltejs/adapter-auto": "3.2.2",
"@sveltejs/kit": "2.5.17",
"svelte": "4.2.18",
"svelte-check": "3.8.1",
"typescript": "5.4.5",
"vite": "5.3.1"


fix download csv bug

Description
fix csv download bug
Type of change

[X ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix a feature that knowingly causes existing functionality to not work as expected)

Test and Reproduce
Clicking on the csv download button
Impacted Areas in Application
List general components of the application that this PR will affect:
Which pages?

[ ] Login Screen
[ ] Single Upload Page
[ ] Batch Upload Page
[ ] All Projects Page
[ ] Sample Details Page
[ X] Report Page

Specific components

report_controler

Testing Script:
###Login Page

Successfully login to the application and get taken to the project page

###Single Upload

Using an AWS path that doesn't exist throws an error
Succesfully uploaded files from a folder (use test host)
Uploading a duplicate file throws an error
Pipeline gets initiated successfully

###Batch Upload

Using an AWS path that doesn't exist throws an error
Succesfully uploaded files from a folder (use test host)
Navigate to batch upload settings page after upload
Host selection from previous page still selected
Uploading a duplicate file throws a warning
Pipeline gets initiated successfully

###All Project Page

Switch between projects
Search for a sample
Navigate through different pages of samples
Select Upload new sample button
Select sample to see generated report

###Sample Details Page

Information is consistent from Project Page
Notes are editable and save successfully
Can navigate to report page

###Report Page

Data loads succesfully
Collapse and Uncollapse button works
Clicking on the taxonomy name gives you total reads for that taxon
Category filter works
Genus Search works
Filters work as expected
Disabling Filters works
Filtering by column works

Checklist:

[X ] I have run through the testing script to make sure current functionality is unchanged
[X ] I have done relevant tests that prove my fix is effective or that my feature works
[X ] I have spent time testing out edge cases for my feature
[X ] I have updated the test script or pull request template if necessary
[X ] New and existing unit tests pass locally with my changes

I have this PR with a more comprehensive fix: https://github.com/chanzuckerberg/idseq-web/pull/408
Suggest merging that one instead of this one


Change master node timeout to new API

Signed-off-by: Chen Dai<EMAIL_ADDRESS>Description
Change masterNodeTimeout to new API clusterManagerNodeTimeout.
Issues Resolved
https://github.com/opensearch-project/sql/issues/463
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

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

Codecov Report

Merging #793 (fd4e503) into main (eeb90cf) will decrease coverage by 32.09%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##               main     #793       +/-   ##
=============================================
- Coverage     94.85%   62.76%   -32.10%
=============================================
  Files           288       10      -278
  Lines          7839      658     -7181
  Branches        571      118      -453
=============================================
- Hits           7436      413     -7023
+ Misses          349      192      -157
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../org/opensearch/sql/analysis/symbol/Namespace.java

.../opensearch/sql/planner/logical/LogicalRemove.java

...rch/sql/analysis/ExpressionReferenceOptimizer.java

...ge/script/aggregation/AggregationQueryBuilder.java

...java/org/opensearch/sql/utils/ExpressionUtils.java

...opensearch/sql/expression/HighlightExpression.java

...ensearch/sql/expression/function/FunctionName.java

...ch/sql/planner/logical/LogicalPlanNodeVisitor.java

...search/sql/expression/window/WindowDefinition.java

...l/opensearch/data/value/OpenSearchExprIpValue.java

... and 268 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Community Awards Ceremony at cdCON 2021

[ ] Strategy & Plan: define the award program, selection criteria, and recruit selection committee | CDF Planning Outreach Sub-Committee | Start: ASAP; Deadline: Feb. 1, 2021
[ ] Promotion: recruit referrals | CDF Planning Outreach Sub-Committee | Feb. 1, 2021
[ ] Selection Committee defines winners and secure awards/trophies | CDF Planning Outreach Sub-Committee | Deadline: May 21, 2021
[ ] Plan award ceremony | CDF Planning Outreach Sub-Committee | Feb. 1, 2021 - Deadline: May 21, 2021
[ ] Host award ceremony | CDF Planning Outreach Sub-Committee | Deadline: June 24, 2021

I'll help run this and promote it once we get the timeline together. Can you add me to this issue @jsalinas29

I'll help run this and promote it once we get the timeline together. Can you add me to this issue @jsalinas29

working doc on awards ceremony categories, etc. https://docs.google.com/document/d/1PvydABqPcatuYOhAoJUTlOizNSpm2OH6me3Cmi9Mt3Q/edit?usp=sharing

working doc on awards ceremony categories, etc. https://docs.google.com/document/d/1PvydABqPcatuYOhAoJUTlOizNSpm2OH6me3Cmi9Mt3Q/edit?usp=sharing

ordered 1 sample of the award!

Workback Cal Dates:
Nominations open: Mar 8
Nominations close: April 2
Voting opens/closes: April 5
Notification of nominations: May 28
Award ceremony at cdcon: June 24

Sorry, those dates aren't right.
Correct Doc: https://docs.google.com/document/d/18KGUWEYMj9AXRw50ME8sbOKahYCMxpowjEzGJApVoA4/edit
Nominations open: April 1, 2021
Nominations close: April 30, 2021
Voting opens: May 1, 2021
Voting closes: May 31, 2021
Winners announced at cdCon: June 23-24, 2021

Page is up and nominations are open:
page: https://cd.foundation/cdf-community-awards/
#314
#315
#316


Is it possible to do "Multi-round Chatting" with InstructBLIP?

Hi, thanks for the great work!
I'm wondering if InstructBLIP allows multi-round chatting, e.g., <Img, Q1, A1, Q2, ?>, so that it can support in-context learning in a dialogue.
If so, which python script can I refer to?
It seems the demo (https://replicate.com/gfodor/instructblip) does not allow the multi-round consecutive questions.
Thanks again for your amazing contributions.

This may be what you are looking for:
https://colab.research.google.com/github/salesforce/LAVIS/blob/main/examples/blip2_instructed_generation.ipynb


ios 15.0.2 support

Can you please add ios 15.0.2 support, or maybe make a small tutorial on how to add support . I attached my Bluetoothd file . Thnk you
bluetoothd.zip

Someone already requested ios15.0.. maybe 0.2 would work with that

I tried, Didnt work.

I tried, Didnt work.

The tweak wasn’t published with that update yet

I compiled it myself.

I tried, Didnt work.

The tweak wasn’t published with that update yet

I compiled it myself.

Oh yeah I see in the code it specifically looks for 15.0.0
It actually looks pretty simple to update for other versions but I don't have a mac to open the daemon file.. you could try just adding a 2 for the patch version in code though assuming the daemon hasn't changed?

Just compared them and the addresses seemed to be the same as iOS 15.0.0 one.
Updated the source file.
By the way, as IPC wouldn't work at iOS 15, the buggy pop-ups and swipe-to-change-volume (idk how about iOS 15.0 but it's supported by native on iOS 15.3) wouldn't work currently as well. It just make it available in the accessibility list and I think that's all.

Where to download deb files

Just compared them and the addresses seemed to be the same as iOS 15.0.0 one.
Updated the source file.
By the way, as IPC wouldn't work at iOS 15, the buggy pop-ups and swipe-to-change-volume (idk how about iOS 15.0 but it's supported by native on iOS 15.3) wouldn't work currently as well. It just make it available in the accessibility list and I think that's all.

Thank you 🙏 i'll try it whene i get home.

Didn't work for me. Iphone 13 pro max 15.0.2 / xinaa15.

Didn't work for me. Iphone 13 pro max 15.0.2 / xinaa15.

Maybe something to do with airpods pro gen 2 model ? I see in apple website there's 3 different model (A2931, A2699, A2698) .

Notice what he said about it not working well on ios15.. it seems he is saying that it will only add it to accessibility list and not the other typical airpod features. So the tweak is probably working as intended for 15.

Didn't work for me. Iphone 13 pro max 15.0.2 / xinaa15.

Maybe something to do with airpods pro gen 2 model ? I see in apple website there's 3 different model (A2931, A2699, A2698) .

That could be the case, for me it's A2618, but not really sure about that. A way to verify that is to remove product ID matching of this tweak, that is making if(*(unsigned int*)(a1+product_id_offset)==0x2014) { always true, but have to make sure no other bluetooth devices connected when testing to avoid messing them up.
And please make sure you've killed bluetoothd after installation so that it could be injected.

Notice what he said about it not working well on ios15.. it seems he is saying that it will only add it to accessibility list and not the other typical airpod features. So the tweak is probably working as intended for 15.

He said the pop up and the slide to change volume wouldn't work.. but the main feature of the tweak is to make ios think that airpods pro gen 2 is airpods pro (first gen). My device still treat them like any generic Bluetooth earbuds.that's why im saying it donc work for me

Didn't work for me. Iphone 13 pro max 15.0.2 / xinaa15.

Maybe something to do with airpods pro gen 2 model ? I see in apple website there's 3 different model (A2931, A2699, A2698) .

That could be the case, for me it's A2618, but not really sure about that. A way to verify that is to remove product ID matching of this tweak, that is making if(*(unsigned int*)(a1+product_id_offset)==0x2014) { always true, but have to make sure no other bluetooth devices connected when testing to avoid messing them up.
And please make sure you've killed bluetoothd after installation so that it could be injected.

Mine is A2698.. i'll try what you suggest and let you know..and thankd for  your support appreciate

Notice what he said about it not working well on ios15.. it seems he is saying that it will only add it to accessibility list and not the other typical airpod features. So the tweak is probably working as intended for 15.

He said the pop up and the slide to change volume wouldn't work.. but the main feature of the tweak is to make ios think that airpods pro gen 2 is airpods pro (first gen). My device still treat them like any generic Bluetooth earbuds.that's why im saying it donc work for me

Yep that means it didn't work for sure. The device should show AirPods Pro icon instead of bluetooth icon on the volume slider (of control center) in case it's working.

Sorry to bother, can you give me the exact sentence to make it always true

I'm not much of a coder but I would guess removing that line 14 and the closing bracket at 16 would make it always true

Sorry to bother, can you give me the exact sentence to make it always true

I'm not much of a coder but I would guess removing that line 14 and the closing bracket at 16 would make it always true

didn't work. maybe he can just add support for other product id model (A2931, A2699, A2698) .

I'm sorry but I found it's my fault that I put the wrong offset value. Try the new one out.

I'm sorry but I found it's my fault that I put the wrong offset value. Try the new one out.

still didn't work :(

I'm sorry but I found it's my fault that I put the wrong offset value. Try the new one out.

still didn't work :(

Sorry hearing that, I'll try figuring it out with my rootless device.

I'm sorry but I found it's my fault that I put the wrong offset value. Try the new one out.

still didn't work :(

Sorry hearing that, I'll try figuring it out with my rootless device.

Thanks for your support 🙏

I'm sorry but I found it's my fault that I put the wrong offset value. Try the new one out.

still didn't work :(

Sorry hearing that, I'll try figuring it out with my rootless device.

I compiled from latest and tested on 15.0, battery levels are working with updated offset.

Did some updates and got it worked on my device, try it out.

Did some updates and got it worked on my rootless device, try it out.
Remember to kill bluetoothd after installation. For Fugu15 Max the vmaddr slide is 1 instead of 0 so I made it so, but I am not quite sure about how was it on XinaA15, if you find the bluetoothd is crashing try changing the vmaddr slide back to 0.

Compiled with new changes, still no working volume adjustments. Bluetoothd doesn’t crash, and is receiving volume change request from airpods, and sends to mediaserverd. See attached image, can send better logs on discord if you’d like.

Did some updates and got it worked on my rootless device, try it out.
Remember to kill bluetoothd after installation. For Fugu15 Max the vmaddr slide is 1 instead of 0 so I made it so, but I am not quite sure about how was it on XinaA15, if you find the bluetoothd is crashing try changing the vmaddr slide back to 0.

Compiled with new changes, still no working volume adjustments. Bluetoothd doesn’t crash, and is receiving volume change request from airpods, and sends to mediaserverd. See attached image, can send better logs on discord if you’d like.
Also, sometimes bluetoothd refuses to send volume change request for some reason.
Model name: ACCESSORY_MODEL_NAME_20 Model number: A2698

The model name doesn't seem like it worked well, have you killed bluetoothd after installing? Also, how was accessibility settings, do they appear after installing the tweak? Volume adjustment is currently not going to work for iOS 15+ by applying this tweak, but your log seems to be helpful and I will try working on it later.
And what's the name for that tweak showing logs, it looks quite helpful.

I tried compiling with theos but all I get is errors so I gave up on it.. guess I'll wait to get from repo. Also wondering what this logging tool is never seen it before.

the tweak name is antoine from chariz repo. no accessibility settings .

I tried compiling with theos but all I get is errors so I gave up on it.. guess I'll wait to get from repo. Also wondering what this logging tool is never seen it before.

you have to replace Lightmessangin.h file in theos directory with this one.  https://github.com/rpetrich/LightMessaging

The model name doesn't seem like it worked well, have you killed bluetoothd after installing? Also, how was accessibility settings, do they appear after installing the tweak? Volume adjustment is currently not going to work for iOS 15+ by applying this tweak, but your log seems to be helpful and I will try working on it later. And what's the name for that tweak showing logs, it looks quite helpful.

Killed bluetoothd in xina processview after installing.
Accessibility does not appear.
Antoine on Chariz repo.

you have to replace Lightmessangin.h file in theos directory with this one. https://github.com/rpetrich/LightMessaging

Now you tell me after I wasted hours and deleted everything lol

you have to replace Lightmessangin.h file in theos directory with this one. https://github.com/rpetrich/LightMessaging

Now you tell me after I wasted hours and deleted everything lol

😂 You didn't ask

Thanks.
You can uncomment line 16,17,53,54,162,163,164 to have your airpods' product ID in a log in /tmp. Tell me what it is if possible, PRODID: %d. Make sure only airpodspro2 connected so that it won't be messed up. For me it's 0x2014=8212, just like what in the source file.
The log file for this would appear at /tmp/bluetoothd.txt, if that file doesn't appear, it means that the hooking failed.

Thanks. You can uncomment line 16,17,53,54,162,163,164 to have your airpods' product ID in a log in /tmp. Tell me what it is if possible, PRODID: %d. Make sure only airpodspro2 connected so that it won't be messed up. For me it's 0x2014=8212, just like what in the source file. The log file for this would appear at /tmp/bluetoothd.txt, if that file doesn't appear, it means that the hooking failed.

There' no PRODID: in the log file. it only contain :
PREP, MYPID 9613
PREP, MYPID 9643

Then it's hooking failure. Try turning vmaddr slide to 0.
(_dyld_get_image_vmaddr_slide(1) -> _dyld_get_image_vmaddr_slide(0)) and uncomment line 205,206 to ensure it's not a version comparison failure.

PREP, MYPID 13487
FOUND ENTRY
PREP, MYPID 13550
FOUND ENTRY

Quite weird to have hooking not working.

Not sure but this could be related:

With Xina 1.1.4 switching to libhooker, the 2 necessary functions MSHookMemory and MSHookFunction are now working as intended. This means that game mods should also work. With a couple of small fixes from my side, my mods work on my device (A12, 15.1)

Best way to try if MSHookFunction is working could be hooking a null pointer. Another case could be that there are more than 1 binaries injected in an order prior to the binary itself, in such case just try some more vmslide number, such as 2,3,4,etc.

Not sure but this could be related:

With Xina 1.1.4 switching to libhooker, the 2 necessary functions MSHookMemory and MSHookFunction are now working as intended. This means that game mods should also work. With a couple of small fixes from my side, my mods work on my device (A12, 15.1)

Best way to see if MSHookFunction is working could be hooking a null pointer and if it crashes it works. Another case could be that there are more than 1 binaries injected in an order prior to the binary itself, in such case just try some more vmslide number, such as 2,3,4,etc.

Tested on latest xina 1.1.8-f2 (from cypwn). I checked it out in frida, product id is 8212, but does not show in the log file. "BTDetails_ProductID" = "BTHeadphones76,8212"
I will say that xina versions > <IP_ADDRESS> have bugged mediaserverd, so I will give that version a chance.

Not sure but this could be related:

With Xina 1.1.4 switching to libhooker, the 2 necessary functions MSHookMemory and MSHookFunction are now working as intended. This means that game mods should also work. With a couple of small fixes from my side, my mods work on my device (A12, 15.1)

Best way to see if MSHookFunction is working could be hooking a null pointer and if it crashes it works. Another case could be that there are more than 1 binaries injected in an order prior to the binary itself, in such case just try some more vmslide number, such as 2,3,4,etc.

now it works as itended, i made the 4 entry vmslide to 1 .hope you'll find a workaround for the slide to change volume. thanks for you support

Not sure but this could be related:

With Xina 1.1.4 switching to libhooker, the 2 necessary functions MSHookMemory and MSHookFunction are now working as intended. This means that game mods should also work. With a couple of small fixes from my side, my mods work on my device (A12, 15.1)

Best way to see if MSHookFunction is working could be hooking a null pointer and if it crashes it works. Another case could be that there are more than 1 binaries injected in an order prior to the binary itself, in such case just try some more vmslide number, such as 2,3,4,etc.

now it works as itended, i made the 4 entry vmslide to 1 .hope you'll find a workaround for the slide to change volume. thanks for you support

Oh, perfect! Good to hear that, thanks for your hard tries too.

you have to replace Lightmessangin.h file in theos directory with this one. https://github.com/rpetrich/LightMessaging

Wow that really did fix the issue..! Thanks, I was able to compile the tweak now. Though I got several warnings about it not finding a rootful directory.

you have to replace Lightmessangin.h file in theos directory with this one. https://github.com/rpetrich/LightMessaging

Wow that really did fix the issue..! Thanks, I was able to compile the tweak now. Though I got several warnings about it not finding a rootful directory.

Good to hear that. You're welcome mate

So this tweak works fully as intended on Xina15 ios 15.0.2 ? I am thinking about purchasing Airpods pro 2 . It would make my decision much easier lol. Thanks for all the hard work

Yes, but you must install rootless via saily, which breaks sileo. So perfectly fine and working for me as I use saily for a package manager.

I think i will wait for official Fugu release. Thanks


Problem to switch the language

Explain the bug
In the release version (the one that is on the Microsoft Store) the Portuguese (Brazil) language is not being applied, being applied Portuguese (Portugual). Which makes me confused because in the development version, Portuguese (Portugal) is neither listed, nor does it have a language file
To reproduce
In the release version:

Go to Settings
Go to Preferences
Switch between Portuguese (Portugal) and Portuguese (Brazil)

Expected behavior
Strings update according to the selected language
Screenshots
Release Version:

Dev Version:

Wrong (Release):

Right (Dev):

System Information

OS Version: Windows 10 20H2
App version: Dev: v<IP_ADDRESS>. Release: v<IP_ADDRESS>

Fixed in #4416


Lightbox with image slider.

How can we make lightbox with image slider like facebook? is there any way to do that?

+1 @susylPearl did you find any solution?


[BUG] Mars import took too long

Describe the bug
When import mars  first time, it took about 4~5 seconds which is pretty time-consuming for users

To Reproduce
To help us reproducing this bug, please provide information below:

Your Python version: 3.7.9
The version of Mars you use: master
Versions of crucial packages, such as numpy, scipy and pandas
Full stack of the error.
Minimized code to reproduce the error.

Expected behavior
import mars should take less than 1 second, just like pandas:

Additional context
Add any other context about the problem here.

Please run
python -X importtime -c "import mars"

and upload its outputs as a log file here.

Please run
python -X importtime -c "import mars"

and upload its outputs as a log file here.

import.log


Little things on latest RDF push

Testing out the new RDF push, just some quick things that came up with possible fixes:

pip install extruct[rdfa] didn't get the latest RDF files, had to clone from git (assuming this is by design but mentioning just in case)
The example in the README led to this error rdflib.plugin.PluginException: No plugin registered for (json-ld, <class 'rdflib.serializer.Serializer'>).  I'm not sure if this is because of how I downloaded RDFlib previously, but in any case the fix for me was the following: git clone https://github.com/RDFLib/rdflib-jsonld.git && cd rdflib-jsonld && python setup.py install
(small typo) README file has url='http://www.exaple.com/index.html' -> example.com

I'd like to put this in a pull request but not sure how best to handle the middle case in particular.

Hi @legel , thanks for you feedback.

the "rdfa" extra adds rdflib/rdflib and rdflib/rdflib-jsonld dependencies, but the actual RDFa parser is based on pyRDFa3 code with some patches, which is copied into the codebase. I'll have a go at using a git submodule from a fork of mine, or try to get the patches merged upstream (rdflib or pyrdfa3).
indeed, that could be related to how you installed rdflib. when rdflib-jsonld gets installed, their setup.py registers the json-ld plugin for parsing and serializing
I fixed the URL in the README. Nice catch!

Hey @legel , I just released extruct 0.3.0a1 which used rdflib directly without copying pyRdfa3.
Would this release fix the issues you have?

@legel , did you get to test 0.3.0a1 by any chance?

@redapple I tested again on a fresh EC2 Ubuntu 16.04 instance, and unfortunately this is not yet resolved.
Not able to get RDFa through pip:

Not able to get RDFlib through git:

Once I manually cloned and installed the RDFlib installation into my dist-packages, then it worked:

hi @legel ,
pip by default will not install 0.3.0a1 because it's alpha level.
To install it, you need to do
pip install --upgrade extruct[rdfa]==0.3.0a1 (explicit version install)
or
``pip install --upgrade --pre extruct[rdfa]` (to allow pre-releases)
can you try that?

hi @legel , I've released extruct 0.3.1 with rdfa support by default.
Please check it out. Cheers.

I had to add the following to my python script in order to make it work
from rdflib.plugin import register, Serializer, Parser
register('json-ld', Serializer, 'rdflib_jsonld.serializer', 'JsonLDSerializer')
register('json-ld', Parser, 'rdflib_jsonld.parser', 'JsonLDParser')


Prices are weird

where does this pull its numbers from?
if use:
file_get_contents('https://api.coinbase.com/v1/prices/sell?qty=1');
it returns 1198.27 that matches the sell price on coinbase.com, but:
print client.get_sell_price(currency_pair = 'BTC-USD').amount
it returns 1192.28.
just showing its off from the actual coinbase site, if its using exchange numbers then that's why its off.

Just checked the exchange its not the same output, so now its just weird.

@dbendure you should switch to the v2 API. This library uses v2 as well.


Android/iOS - Shortcut - Tapping shortcut track manual does not trigger any action

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.61
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
Action Performed:

Launch app
Tap fab--- track expense---manual
Enter an amount and tap next
Tap track expense
Navigate back to LHN
Tap fab
Tap shortcut track manual

Expected Result:
Tapping shortcut track manual must direct to track manual page
Actual Result:
Tapping shortcut track manual does not trigger any action
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/bff54d01-bcf5-4e64-8d52-5629afd88fff
View all open jobs on GitHub

@muttmuure  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

cc @thienlnam

@Gonals what's the plan with these?
https://github.com/Expensify/Expensify/issues/378892

This was expected, as track functionality wasn't implemented yet. This PRshould fix this, so I'm going to close 😁


Add mapping cchardet:faust-cchardet

faust-cchardet is a maintained fork

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.25 :tada:

Comparison is base (f97d8b9) 86.71% compared to head (816d7a4) 86.97%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #380      +/-   ##
==========================================
+ Coverage   86.71%   86.97%   +0.25%
==========================================
  Files           2        2
  Lines         256      261       +5
==========================================
+ Hits          222      227       +5
  Misses         34       34

Impacted Files
Coverage Δ

pipreqs/__init__.py
100.00% <100.00%> (ø)

pipreqs/pipreqs.py
86.82% <100.00%> (+0.26%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make it work with multiple sites

The idea is to have a single node instance handling multiple sites
where each site has it's own directory structure.
This is especially useful for development where you don't want to
install and setup a full web server (like Apache or Nginx).
In this setup, you want to pass the sites to chokidar so that when
files change, the page is hot reloaded.
Suppose we have two sites:

one
two

When we change the file one/app.js, the emitted event will receive
the full path, i.e. one/app.js, while the hot reloader expect to
have a path without the prefix (in this case, one/).
This commit remove the prefix from the path so that the hot reloader
works as expected.
I've also created a small example to show how this works:
https://github.com/oligot/jspm-hot-reloader-multiple

oh yeah, I see now. There is a problem though, because if you're using it with html5 mode and the path matches to a url of a module, by replacing that part you malform the path to the module.
We could replace it if a hashbang URL is detected. Would that work for you? Do your sites use hasbang history or html5 history pushtate?

We don't use hashbang nor html5 history.
Maybe the method used to differentiate sites (plain old url, hashbang or html5 history) could be a parameter of JspmHotReloader class constructor ?

I've updated the PR to add a new optional parameter transform to the JspmHotReloader constructor.
I've also updated the small example to show how one can use it
https://github.com/oligot/jspm-hot-reloader-multiple/blob/master/hot-reload.js#L7-L9

@oligot this seems allright. I will merge this, but I want to have an options object rather than two params for the constructor call.


Add purge db executable to bridge docker image

What was wrong?
We never added the purge_db script to the docker bridge image.
Also removed some unnecessary lines post #681
How was it fixed?
To-Do

[x] Add entry to the release notes (may forgo for trivial changes)
[x] Clean up commit history

Quick docker images fix here - moving ahead with the un-approved merge so I can get this up & running on the testnet


Error in reference

https://github.com/NUS-Fintech-Society/SD_Notesfree/blob/fd1db9b94c55f57ab9c9d5702566542858afea19/backend/api/api.py#L83
Should be delete_by instead of deleted_by

fixed


[Event Request] Table 38 "Purchase Header" - OnValidatePurchaseHeaderPayToVendorNo

Hello. In table 38 purchase Header, in the field 4 "Pay-to Vendor No.", in trigger OnValidate, I would like to add my own code in the event OnValidatePurchaseHeaderPayToVendorNo and use in the subscriber the xRec. To do so, I would need xRec parameter in the event publisher, like this :
            "Prices Including VAT" := Vend."Prices Including VAT";
            "Currency Code" := Vend."Currency Code";
            "Invoice Disc. Code" := Vend."Invoice Disc. Code";
            "Language Code" := Vend."Language Code";
            SetPurchaserCode(Vend."Purchaser Code", "Purchaser Code");
            Validate("Payment Terms Code");
            Validate("Prepmt. Payment Terms Code");
            Validate("Payment Method Code");
            Validate("Currency Code");
            Validate("Creditor No.", Vend."Creditor No.");

            OnValidatePurchaseHeaderPayToVendorNo(Vend, Rec, xRec);

With this event publisher :
[IntegrationEvent(TRUE, false)]
procedure OnValidatePurchaseHeaderPayToVendorNo(Vendor: Record Vendor; var PurchaseHeader: Record "Purchase Header"; var xPurchaseHeader: Record "Purchase Header")
begin
end;

Do you think it could be possible ?
Thank you for your work.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[BUG]

What is the bug?
Hello! I faced a problem. I’m trying to include my OpenSearch Dashboard in my website using iframe like this :
<iframe src="https://localhost:5601" style="width: 100vw; height: 100vh;"></iframe>
So it's loaded into the iframe, but it constantly asks for a login-password, I enter the correct user/password, but it doesn't connect into the dashboard and again requires input.
Console Errors are in the following screenshot :

At the same time, when i open the link of the iframe (localhost:5601) in a separate tab, everything works correctly.
What is the expected behavior?
The expected behavior is that the dashboard should work using iframe the same as without iframe.
Do you have any additional context?
My opensearch.yml file configuration is :
plugins.security.disabled: "false"
plugins.security.ssl.transport.pemcert_filepath: node1.pem
plugins.security.ssl.transport.pemkey_filepath: node1-key.pem
plugins.security.ssl.transport.pemtrustedcas_filepath: root-ca.pem
plugins.security.ssl.transport.enforce_hostname_verification: false
plugins.security.ssl.http.enabled: true
plugins.security.ssl.http.pemcert_filepath: node1.pem
plugins.security.ssl.http.pemkey_filepath: node1-key.pem
plugins.security.ssl.http.pemtrustedcas_filepath: root-ca.pem
plugins.security.authcz.admin_dn:
  - 'CN=A,OU=UNIT,O=ORG,L=TORONTO,ST=ONTARIO,C=CA'
plugins.security.nodes_dn:
  - 'CN=node1.dns.a-record,OU=UNIT,O=ORG,L=TORONTO,ST=ONTARIO,C=CA'
  - 'CN=node2.dns.a-record,OU=UNIT,O=ORG,L=TORONTO,ST=ONTARIO,C=CA'
plugins.security.restapi.roles_enabled: ["all_access", "security_rest_api_access", "client1_role"]

And my opensearch_dashboards.yml file configuration is the following :
opensearch.hosts: ["https://localhost:9200"]
opensearch.ssl.verificationMode: certificate
server.ssl.enabled: true
server.ssl.certificate: "C:\path\to\client.pem"
server.ssl.key: "C:\path\to\client-key.pem"
opensearch.ssl.certificateAuthorities: ["C:\path\to\root-ca.pem"]
opensearch.username: kibanaserver
opensearch.password: kibanaserver
opensearch.requestHeadersAllowlist: ["securitytenant","Authorization"]
opensearch_security.multitenancy.enabled: true
opensearch_security.multitenancy.tenants.enable_global: true
opensearch_security.multitenancy.tenants.enable_private: true
opensearch_security.multitenancy.tenants.preferred: ["Private", "Global"]
opensearch_security.multitenancy.enable_filter: false
opensearch_security.cookie.secure: false
server.cors: true
csp.rules:
  - "frame-src 'self' http://localhost:3001"
  - "frame-ancestors 'self' http://localhost:3001"

Does anyone have an idea how to resolve this issue ?

Please see this discussion in the forums. The dashboard embed still needs to access something behind an authentication "wall" (unless you set it up to be anonymous/non-auth'd).


luci-app-ddns:Fix Simplified Chinese translation.

Simplified Chinese translation for luci-app-ddns has been broken for long.I translated this application these day and created this pull request.

@981213
Hi, I accidentally merge your pull request without looking for contribution guidlines.
Please recreate also #368 and sign your pull requests.
Thanks
Christian


allow large scale testing

allows new testing rewrite to handle 500+ tests and load and run these tests. High limit tested was 10,000 tests.

tests will be updated when new resultResolver class is created that will take most responsibility away from workspaceTestAdapter.


message_formatting: Add a row for "---" in message formatting tips and Remove line from emoji formatting tips.

Screenshots and screen captures:

 Adding "---" in message formatting tips

 Removing a line from emoji formatting tip

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Ok, cool, I think this is fine to add at the bottom.

Yeah sure, I will add a commit removing that line in this PR itself.

Thanks! Please use @zulipbot to add the "integration review" label to this PR when ready.

@zulipbot add "integration review"

@zulipbot add "integration review"

I have added the commit to remove the line from the emoji formatting tip. I think it is ready for integration review, but looks like something is wrong with the bot. Hence, tagging @alya.

@zulipbot add "integration review"

Merged, thanks @sbansal1999!


[Addon] Optimize fluxcd addon

A parameter "onlyHelmComponents"  give user an option disable gitOps controller(kustomize, imageUpdateAutomatelly...).
Add apply-once=false(only select to CRD resource type) policy to avoid config drift.
Not delete CRD after addon disabled.

Description of your changes

How has this code been tested?

Checklist

I have:

[ ] Title of the PR starts with type (e.g. [Addon] , [example] or [Doc]).
[ ] Updated/Added any relevant documentation and examples.
[ ] New addon should be put in experimental.
[ ] Update addon should modify the version in metadata.yaml to generate a new version.

Discussed with @barnettZQG , change namespace and registry parameter to optional


fix Collections::reduce and Collections::reduceRight behaviour

Hello, and thanks for contributing to php-lodash
Description
The changes I made to reduce and reduceRight makes them both behave like their JS counterparts that they were modeled after, the change is potentially breaking for any code that relied heavily on, or just worked with, the existing implementation.
In the documentation for Array.prototype.reduce and Array.prototype.reduceRight, it is mentioned that in case of calling any of those functions without including an initial value, the function uses the first element of the array to be processed as its initial value and starts iterating from the next element. That way it doesn't double-process any of the values of the array.
Type of change

[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?
Added new cases to the existing unit test to confirm that the changes have the intended effect. The snippet below shows concisely the effect of the changes.
$sum = __::reduce([2], function ($a, $b){return $a+$b;}); //before -> 4 -- processed the first element twice, after --> 2
$sum = __::reduce([2], function ($a, $b){return $a+$b;}, 0); // before and after -> 2 -- an initial value was provided

Test Configuration:

PHP Version: >=7.0

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Codecov Report

Merging #13 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage     98.51%   98.51%
  Complexity      224      224
=========================================
  Files             8        8
  Lines           538      538
=========================================
  Hits            530      530
  Misses            8        8

Impacted Files
Coverage Δ
Complexity Δ

src/Traits/Collections.php
98.75% <100%> (ø)
119 <0> (ø)
:arrow_down:

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 393a5a9...6c65d3b. Read the comment docs.

Thank you @MamdouhAhmed 👍


Msdeploy fails

For deployment issues, please provide us with the following information:

Repro steps.

your project built successfully on your dev machine but failed on Azure?
Am using MsDeploy to create a package, that I later try to deploy to Azure with an Arm template.
I use the command to create the package
msdeploy.exe  -source:contentPath="sourcepath" -dest:package="destination\package.zip" -verb:sync

Project structures.

in order to reproduce your issue at our end we need a simple
github repository
that highlights structure of the project

The log/error given by the failure.
ARM-MSDeploy Deploy Failed: 'Microsoft.Web.Deployment.DeploymentDetailedUnauthorizedAccessException: Unable to perform the operation ("Create Directory")  for the specified directory ("C:\source\out\bin\RunnerAzureFunction"). This can occur if the server administrator has not authorized this operation for the user credentials you are using.  Learn more at: http://go.microsoft.com/fwlink/?LinkId=221672#ERROR_INSUFFICIENT_ACCESS_TO_SITE_FOLDER. ---&amp;amp;amp;amp;amp;gt; Microsoft.Web.Deployment.DeploymentException: The error code was 0x80070005. ---&amp;amp;amp;amp;amp;gt; System.UnauthorizedAccessException: Access to the path 'C:\source' is denied.
2021-08-27T19:45:32.5855632Z    at Microsoft.Web.Deployment.NativeMethods.RaiseIOExceptionFromErrorCode(Win32ErrorCode errorCode, String maybeFullPath)
2021-08-27T19:45:32.5856156Z    at Microsoft.Web.Deployment.DirectoryEx.CreateDirectory(String path)
2021-08-27T19:45:32.5856658Z    at Microsoft.Web.Deployment.DirPathProviderBase.CreateDirectory(String fullPath, DeploymentObject source)
2021-08-27T19:45:32.5857185Z    at Microsoft.Web.Deployment.DirPathProviderBase.Add(DeploymentObject source, Boolean whatIf)
2021-08-27T19:45:32.5857589Z    --- End of inner exception stack trace ---
2021-08-27T19:45:32.5857959Z    --- End of inner exception stack trace ---
2021-08-27T19:45:32.5858526Z    at Microsoft.Web.Deployment.FilePathProviderBase.HandleKnownRetryableExceptions(DeploymentBaseContext baseContext, Int32[] errorsToIgnore, Exception e, String path, String operation)
2021-08-27T19:45:32.5859128Z    at Microsoft.Web.Deployment.DirPathProviderBase.Add(DeploymentObject source, Boolean whatIf)
2021-08-27T19:45:32.5859814Z    at Microsoft.Web.Deployment.DeploymentObject.Add(DeploymentObject source, DeploymentSyncContext syncContext)
2021-08-27T19:45:32.5860384Z    at Microsoft.Web.Deployment.DeploymentSyncContext.HandleAdd(DeploymentObject destObject, DeploymentObject sourceObject)
2021-08-27T19:45:32.5861049Z    at Microsoft.Web.Deployment.DeploymentSyncContext.HandleUpdate(DeploymentObject destObject, DeploymentObject sourceObject)
2021-08-27T19:45:32.5861640Z    at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildrenOrder(DeploymentObject dest, DeploymentObject source)
2021-08-27T19:45:32.5862205Z    at Microsoft.Web.Deployment.DeploymentSyncContext.SyncChildren(DeploymentObject dest, DeploymentObject source)
2021-08-27T19:45:32.5862788Z    at Microsoft.Web.Deployment.DeploymentSyncContext.ProcessSync(DeploymentObject destinationObject, DeploymentObject sourceObject)
2021-08-27T19:45:32.5863501Z    at Microsoft.Web.Deployment.DeploymentObject.SyncToInternal(DeploymentObject destObject, DeploymentSyncOptions syncOptions, PayloadTable payloadTable, ContentRootTable contentRootTable, Nullable`1 syncPassId, String syncSessionId)
2021-08-27T19:45:32.5864316Z    at Microsoft.Web.Deployment.DeploymentObject.SyncTo(DeploymentProviderOptions providerOptions, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)
2021-08-27T19:45:32.5865013Z    at Microsoft.Web.Deployment.DeploymentObject.SyncTo(String provider, String path, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)
2021-08-27T19:45:32.5865712Z    at Microsoft.Web.Deployment.DeploymentObject.SyncTo(DeploymentWellKnownProvider provider, String path, DeploymentBaseOptions baseOptions, DeploymentSyncOptions syncOptions)
2021-08-27T19:45:32.5866370Z    at Microsoft.Web.Deployment.WebApi.AppGalleryPackage.Deploy(String deploymentSite, String siteSlotId, Boolean doNotDelete)
2021-08-27T19:45:32.5866980Z    at Microsoft.Web.Deployment.WebApi.DeploymentController.&amp;amp;amp;amp;amp;lt;DownloadAndDeployPackage&amp;amp;amp;amp;amp;gt;d__25.MoveNext()'
Debug your Azure website remotely.

it is recommanded that you share your Web App name, directly or indirectly
we can take a look at what's going on.

Mention any other details that might be useful.
Any assistance will really help, thank you :)

Thanks! We'll be in touch soon.

Creating a .zip of an existing directory
If you want to create an official WebDeploy-formatted .zip archive, you can use MSDeploy with the sync verb, and specify a the source as a contentPath, and the destination as a package.
NOTE: You must use a replace clause to change the source contentPath prefix, to 'website'.  Also, note a similar replacement is required in the declareParam argument.
Example:
"C:\Program Files\IIS\Microsoft Web Deploy V3\msdeploy.exe" -verb:sync -source:contentPath='C:\Websites\MyWebSite',includeAcls=false -dest:package='E:\Temp\msdeploy_my_site_package.zip'  -replace:match='C:\\Websites\\MyWebSite\\',replace='website\'  -retryAttempts:2  -declareParam:name="IIS Web Application Name",kind="ProviderPath",scope="IisApp",match="^C:\\Websites\\MyWebSite",defaultValue="website"  -verbose
Publishing a .zip created by MSDeploy
When publishing, you must replace the declared param "IIS Web Application Name" value with your site name.
Example:
"C:\Program Files\IIS\Microsoft Web Deploy V3\msdeploy.exe"  -verb:sync  -source:package='E:\Temp\msdeploy_my_site_package.zip',includeAcls=false  -dest:contentPath='mysite',ComputerName='https://mysite.scm.azurewebsites.net:443/msdeploy.axd?site=mysite',UserName='****',Password='****',AuthType='Basic',includeAcls=False  -userAgent:Easy_To_Spot_UA  -allowUntrusted  -setParam:name='IIS Web Application Name',value='mysite'  -verbose

Thank you @EricSten-MSFT for the quick response, I tried using the command you've suggested creating the zip archive
"C:\Program Files\IIS\Microsoft Web Deploy V3\msdeploy.exe" -verb:sync -source:contentPath='C:\Websites\MyWebSite',includeAcls=false -dest:package='E:\Temp\msdeploy_my_site_package.zip' -replace:match='C:\\Websites\\MyWebSite\\',replace='website\' -retryAttempts:2 -declareParam:name="IIS Web Application Name",kind="ProviderPath",scope="IisApp",match="^C:\\Websites\\MyWebSite",defaultValue="website" -verbose

But unfortunately, I ran into the same error from before
 ARM-MSDeploy Deploy Failed: 'Microsoft.Web.Deployment.DeploymentDetailedUnauthorizedAccessException: Unable to perform the operation ("Create Directory")  for the specified directory ("C:\source\out\bin\package"). This can occur if the server administrator has not authorized this operation for the user credentials you are using.  Learn more at: http://go.microsoft.com/fwlink/?LinkId=221672#ERROR_INSUFFICIENT_ACCESS_TO_SITE_FOLDER. ---> Microsoft.Web.Deployment.DeploymentException: The error code was 0x80070005. ---> System.UnauthorizedAccessException: Access to the path 'C:\source' is denied.

Am using Msdeploy in Arm template, to deploy the zip file
{
          "name": "MSDeploy",
          "type": "Extensions",
          "apiVersion": "2015-02-01",
          "dependsOn": [
            "[resourceId('Microsoft.Web/Sites', parameters('functionAppName'))]"
          ],
          "properties": {
            "packageUri": "[parameters('msdeployPackageUrl')]",
            "appOffline": true
          }
        }

I added the appOffline property but that doesn't seem to help.

@MaryanneNjeri - Ah, good feedback.  I thought it was clear an obvious that the examples would have to be adapted to the actual paths / site names used by you and your site.  I'll make the guidance a little less terse.  Thx!


Short solution needed: "chrome headless full page screenshot" (chrome-headless)

Please help us write most modern and shortest code solution for this issue:
chrome headless full page screenshot (technology: chrome-headless)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to take page screenshot


Follow events not being propagated to followers' feeds

Follow steps 1 and 2 of the home page scenarios
danyrose should see the follow event added to his feed, but it isn't. (ie the entry added to tes_seguir_service_social_hub.feed_timeline)

At the moment it only adds it to the feed of the user being followed, which then copies it out to the followers feed (as they are a follower).
What it doesn't do is add it directly to the followers feed (e.g. send it out to all of their followers).

Not saying this is correct btw - just saying that it is what it is doing.
https://github.com/cliftonc/seguir/blob/master/api/follow/index.js#L49

the follow logic used for above mentioned scenarios is on https://github.com/tes/social-team/wiki/Follow-Facts.

This was fixed and pushed to NPM in 0.5.5, later refactored to be a bit easier on the eyes in 0.6.0 (which also has a refactor of how the client auth works to make it easier to update the id / secrets in running environments.


Improve the task retry exponential curve (resend)

Isn't this irrelevant because of the other issue where we use the headers?


JSON reporter produces invalid JSON if errors are spread across several files

Bug Report

TSLint version: 3.10.2
TypeScript version: 18.10
Running TSLint via:  CLI

TypeScript code being linted
user.ts
class User {
  sayHello() {
    let unused = 'hello';
  }
}

pony.ts
class Pony {
  sayHello() {
    console.log('hello')
  }
}

with no tslint.json.
Actual behavior
Running tslint *.ts -t json produces (line return is mine to emphasize the issue):
[{"endPosition":{"character":23,"line":2,"position":51},"failure":"' should be \"","name":"pony.ts","ruleName":"quotemark","startPosition":{"character":16,"line":2,"position":44}},{"endPosition":{"character":24,"line":2,"position":52},"failure":"missing semicolon","name":"pony.ts","ruleName":"semicolon","startPosition":{"character":24,"line":2,"position":52}}]
[{"endPosition":{"character":24,"line":2,"position":52},"failure":"' should be \"","name":"user.ts","ruleName":"quotemark","startPosition":{"character":17,"line":2,"position":45}}]

As you can see the JSON produced is invalid (2 arrays side by side, without being wrapped in an array and be separated by a comma).
Expected behavior
Each file produces an array of errors, instead of having one array for all errors (I guess...), which would be valid JSON.

I know 😢 , see https://github.com/palantir/tslint/issues/656


Clean the file structure a little

A little bit of cleanup
Fixes #19

@Queng123 yeah I agree, I let this be on hold for the moment until we merged in the canvas. Afterwards I would update this PR to also have the other files moved to a better location.

Solved in https://github.com/cryck/sticker-composer/pull/53


update createStateBinderConfiguration to allow for non-required parameters

The MFA trusted device webflow is configured in MultifactorAuthenticationSetTrustAction to discard submissions of the trusted device registration form with empty inputs, which allows for setting up a "Skip" button that submits the form with no data. This is a useful technique to allow users to avoid registering a device that they happen to be using as trusted. However, recent changes to the way the MultifactorAuthenticationTrustBean is bound to the submission, namely through createStateBinderConfiguration on AbstractCasWebflowConfigurer, have created a situation where a submission with empty inputs errors out before arriving at the aforementioned action.
To work around this, I've overloaded createStateBinderConfiguration to accept a required parameter while still preserving the original method signature and behavior, and updated the call to it in the trusted device webflow configurer to specify the parameters are not required.

Thanks for the patch!

This is a useful technique to allow users to avoid registering a device that they happen to be using as trusted.

Yes quite useful, except that it's no longer useful if the functionality is supported out of the box :) Is this not what you see?

Oops! This is what I get for running a heavily modified installation, I suppose. Haven't had any luck bringing in the template code for the skip button to my templates yet, but I'm sure I'll figure it out. Thanks for the tip!

You're quite welcome.


Fix golang Actions

KCP requires 1.18 to build

cache and dep analyser probably passed but their pr's are pruned and PaC bug did mark them as done
req process failed, pr pruned, but that was probably fixed by @stuartwdouglas 's https://github.com/redhat-appstudio/infra-deployments/pull/609
/retest

cache / dep analyzer passed  but hit PaC bug:
gmontero ~ $ oc get pr | grep n8mcm
jvm-build-request-processor-on-pull-request-n8mcm     True        Completed   13m         2m25s
gmontero ~ $ oc get pr | grep rtn6w
jvm-build-cache-on-pull-request-rtn6w                 True        Completed   12m         27s
gmontero ~ $ oc get pr | grep rk6jk
jvm-dependency-analyser-on-pull-request-rk6jk         True        Completed   12m         25s

will merge


Simplify prefix so only TokenTree is used

The regex prefix stuff doesn't actually need to know anything inside of delimiters (only that it is a paren or bracket or brace). So, rather than force isParens etc know about terms, unwrap delimiters when putting them in the prefix.
/cc @gabejohnson

So the change is fine if the only thing we're reading is JS. But since we decoupled the tokenization we aren't necessarily reading JS, we're writing JS (objvs). But what if I'm translating Python into JS? Then I have to encode significant whitespace as braces. In that case Python's scoping rules are probably sane enough that I don't have to worry about drilling down into TokenTrees.
But my point is that I can't predict what language design someone is going to come up with. And I think that we shouldn't destroy any information someone may need if we don't have to.
Ideas:

Do two passes

Pro: simple
Con: another traversal

Unify Tokens and Terms

Pro: eliminates special cases
Con: Tokens aren't Terms (but then again neither are RawSyntaxTerms)

Return a lazy iterator from the reader. Similar to what we're doing in the enforester. Then the second pass isn't an entire second traversal.

Pro: we could be lazy all the way through the system and (best case) do a single traversal kicked off by SweetToShiftReducer. Until codegen that is.
Con: lots of work

Don't listen to me. I don't know what I'm talking about

Pro: no work
Con: maybe I am right and we have to fix for v5

Options 2 and 3 are really interesting. Let me think about this some more but I suspect the right choice at the moment is to go with 1 for now and then pick either 2 or 3 later when we start focusing on perf.

I find 3 particularly attractive and thinking about it more it really shouldn't be that much work. But I agree we should wait until we're close to feature complete and working on perf.


uglify-js does not support ES6

Is compatibility with anything going to be an issue if ES6 syntax is used without a transpiler?
ES6 syntax requested for https://github.com/googlevr/vrview/pull/210
Should we care about Internet Explorer 11?  http://caniuse.com/#search=ES6
3.32% global usage; wonder how many of those support WebGL...
Switch to uglify-es?  https://github.com/mishoo/UglifyJS2

https://github.com/googlevr/vrview/pull/226


Switch to golang http server with reverse proxy to influxdb service

Following discussion in #182, #132, #135.

Can one of the admins verify this patch?

ok to test

If you wouldn't mind holding off on this, I'm looking at putting together a generic go utility that serves static content, processes config template to file & proxies to kubernetes service. I see this as pretty generic & useful.

Yes we can default & use dns too. Dns wasn't available as core service when I first put this together but makes sense to use it now.

@jimmidyson: Overall it looks good. I will not merge this PR until you are done with your re-factoring.
Thanks for the PR!

@vishh @pires @simon3z - I have updated this image to use kuisp (https://github.com/jimmidyson/kuisp) which I've built for this purpose: serve static content & proxy to kubernetes services. It greatly simplifies the scripts while allowing the same functionality. Please review kuisp if you feel you need to.
This gives heapster grafana a more easily configurable image than nginx/apache, as well as being very lightweight & small image.

Thanks @jimmidyson

@jimmidyson: I tried testing your PR and it does not work for me. My request is to have the default config work via the apiserver proxy, that way it works for everyone. Users can then make some changes to either use a public IP for the grafana service or create an ELB in place of the proxy. Specifically, if the default setup can be accessible via <masterip>/api/v1beta1/proxy/services/monitoring-grafana/, that will be awesome. If that is not possible, I am open to discussing how to provide an alternative off-the-box experience.
Thanks for working on this PR!

@vishh what do you receive in your browser when you try it?
Accessing HTML pages via the api server proxy is not really a good idea due to the amount of rewriting the HTML that is required to sort stuff like absolute links, hard coded base hrefs, etc.

@jimmidyson: I tried testing this PR by adding a hostPort to Grafana container.
I agree with you on the fact that the proxy is not an ideal solution. I am open to suggestions on making Grafana accessible from outside the cluster without using the apiserver proxy.

What about using publicIPs flag from Services? This is what I've been advocating all the time.

Accessing services remotely in a consistent way across environments/clouds
is pretty impossible IMO. Public ips is probably the easiest way to achieve
that but as the ip will vary across envs it makes deployment a bit harder
than simply deploying a few RCs & services - needs user input to deploy.
Having said that, it's the only way to get a nice cross-env UX where users
can hit a simple url, without remembering /api/v1beta1/.... Users are gonna
want to set up a dns entry to view grafana directly.
I vote for publicips option but pointing to documentation on different
cloud set ups for accessing kubernetes services, which grafana will just be
another one.
On 22:36, Wed, 25 Mar 2015 Paulo Pires<EMAIL_ADDRESS>wrote:

What about using publicIPs flag from Services? This is what I've been
advocating all the time.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/heapster/pull/183#issuecomment-86238839
.

@vishh - if you've set the host port properly, the pod is actually running & you can route to the node then I don't understand how you get connection refused. The single container process is the web server - of the pod is running then it's serving on port 8080...

@jimmidyson you can also go with createrExternalLoadBalancer but that requires a cloud-provider.

Yeah so that's the other option: document how to deploy on different clouds/envs but that's not the consistent ux that @vishh is after I don't think.

@jimmidyson: I din't notice that the server is bound to '8080' instead of port '80'. Trying again.
@pires: Agreed. PublicIP is definitely the best option. But it requires manual configuration. The only reason for switching over the proxy was to give a simple ux to users. Power users like yourself, know to switch over to Public IPs or ELBs.
@jimmidyson: My hope was for the default setup to work via the proxy and have alternative (better) deployment configs, with documentation, that will use ELBs or Public IPs. Is that too much to ask for or is it difficult to achieve with your new model?

@vishh - reason for 8080 is to run as non-root user (nobody) in container for extra security.
Iirc how the proxy works it should still be accessible through the api server proxy, but I'm not sure how the grafana ui will look with the html rewrites that the api server proxy still probably need to do.

@jimmidyson: Ack. It works perfectly without the proxy. I am trying to debug why access via the proxy fails.

I figured out the issue with the proxy. I am merging this PR as-is. I will fix the proxy stuff in a subsequent PR.
Thanks for the contributions @jimmidyson and thanks for your input @pires.


Need a detection of compiler's support for C++17 with fallback to C++14

Also, at me system GCC-5.4 doesn't supports C++17, but supports C++14. Configure fails when C++17 is set, however, works perfect when C++14 is set in CMakeLists.txt. Why not to detect C++17 support or even just rely on GCC version to specify C++ standard available since specific version.

I see. I will just let cmake pick the latest version it knows.
And I'll insert a feature check just behind that.

Please check if the latest does the correct thing on cmake 3.5.

Note: You keep C++ versions back in your projects, because of Watcom and old environments.
This project could be made retrocompatible even more, but it would be much less elegant.
Basically you replace C++14 mutex stuff with Boost, and std::atomic with either Boost, or volatile.

Anyway, I only want to build this and ADLJack without of re-applying of workarounds after each pull. The Watcom is still not supported right and some stack processing bugs are found while I experimented with it. I have tried Watcom to target libADLMIDI for DOS with HW-OPL3 chip support with one of well-known compilers together with already supporting DJGPP which working well. Inside of libADLMIDI/libOPNMIDI I keeping C++98 standard supported to have it well-compiled on most of widely using compilers include some oldies.

Please check if the latest does the correct thing on cmake 3.5.

It doesn't works because of compiler does not support at least C++14 message, as in my case the full list of compiler features list is:
cxx_aggregate_default_initializers
cxx_alias_templates
cxx_alignas
cxx_alignof
cxx_attribute_deprecated
cxx_attributes
cxx_auto_type
cxx_binary_literals
cxx_constexpr
cxx_contextual_conversions
cxx_decltype
cxx_decltype_auto
cxx_decltype_incomplete_return_types
cxx_defaulted_functions
cxx_defaulted_move_initializers
cxx_default_function_template_args
cxx_delegating_constructors
cxx_deleted_functions
cxx_digit_separators
cxx_enum_forward_declarations
cxx_explicit_conversions
cxx_extended_friend_declarations
cxx_extern_templates
cxx_final
cxx_func_identifier
cxx_generalized_initializers
cxx_generic_lambdas
cxx_inheriting_constructors
cxx_inline_namespaces
cxx_lambda_init_captures
cxx_lambdas
cxx_local_type_template_args
cxx_long_long_type
cxx_noexcept
cxx_nonstatic_member_init
cxx_nullptr
cxx_override
cxx_range_for
cxx_raw_string_literals
cxx_reference_qualified_functions
cxx_relaxed_constexpr
cxx_return_type_deduction
cxx_right_angle_brackets
cxx_rvalue_references
cxx_sizeof_member
cxx_static_assert
cxx_strong_enums
cxx_template_template_parameters
cxx_thread_local
cxx_trailing_return_types
cxx_unicode_literals
cxx_uniform_initialization
cxx_unrestricted_unions
cxx_user_literals
cxx_variable_templates
cxx_variadic_macros
cxx_variadic_templates

Jesus OK it doesn't know features. Let's improvise something else.

New method. Please try.

 Let's do simple.
The user passes -DRING_BUFFER_CXX_STANDARD=14. It sets the C++ standard.
RING_BUFFER_CXX_STANDARD not set => autodetect
PS: make sure to clear cmake cache between tests

Good, now it works!
Also, you would make that RING_BUFFER_CXX_STANDARD variable be cachable with adding of description.

You're right. Done.


Fix Build warning

This issue is for a:
- [ ] bug report -> please search issues before submitting
- [x] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

When performing a Build, there is a deprecated warning

Any log messages given by the failure

Expected/desired behavior

Remove this warning by treating the deprecation

'ApplicationInsightsExtensions.AddApplicationInsightsTelemetry(IServiceCollection, string)' is obsolete: 'InstrumentationKey based global ingestion is being deprecated.


Metrics Operator: Support Aggregation functions in AnalysisValueTemplate

Currently, only the KeptnMetrics CRD supports the use of aggregation functions (see here).
Supporting the same aggregation functions might also be useful for AnalysisValueTemplates as well, to better support queries that return multiple data points of a time series. This way, the provider returning the results for the AnalysisValueTemplate can check whether there are multiple data points in the result for the query and apply the aggregation function in that case.
One argument that might speak against this feature is that usually monitoring provider APIs support aggregation functions on their end already, such as e.g. DQL, so it might be sufficient to adapt queries for AnalysisValueTemplates to make use of the monitoring provider's aggregation capabilities.
References
See the following Github discussion for more context: https://github.com/keptn/lifecycle-toolkit/issues/2650#issuecomment-1906038916

The team agreed to handle this in a KEP in the community meeting of Jan 24th, 2024.


Update README.md

Fixes type of quotation marks in example line for including framework with Carthage
Base Branch
Make sure you're currently pointing to the next release base branch. For example:
branch: feature/5.1.0
If you're unsure, check the latest tag, it'll be a semantic version higher than that.
SwiftLint
Ensure SwiftLint hasn't reported any issues. Code styles must be adhered to.
README
Does the README need updating? Ensure anything relevant is updated there as well.

I realise this is just a README update, but sticking with the contributing.md doc, can you just update this to point to release/5.1.0 then I'll merge.


enhancement: showing toast for invalid IP addresses

Hello @FaheemOnHub , for the issue #1
I have created a toast, which will be displayed when wrong IP address will be provided to the API call.
This is how it will look like:

Feel free to reach out in case of further improvements.

Thank you so much @FaheemOnHub. Yes definitely i will contribute to this project to further improve it.


the question about create a hib

Does this library support the merging of TIF formatted picture files? I attempt to prduce the HIb of the tif file ,It does't work, if I use the library,How can I produce the hib file of tiff images.I need your help.

@spspxl hipi does not support TIF formatted files except JPG, PNG and GIF. So, I suggest you to convert them.
Hope, it helps.