fix Pytorjob status inaccuracy when task replica scale down

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in Fixes #<issue number>, #<issue number>, ... format, will close the issue(s) when PR gets merged):
Fixes #
Checklist:

[ ] Docs included if any changes are user facing

/assign @jeffwan

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 37.268%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/tensorflow/tfjob_controller.go
3
70.62%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
2308

Relevant Lines:
6193

💛  - Coveralls

/ok-to-test

Could you please explain why the PyTorch operator needs such a new check?

operator

Could you please explain why the PyTorch operator needs such a new check?

when HPA scale down task replicas of pytorchjob, pytorchjob operator  terminate redundant pods.  the phase of pods will become Failed Before disappears,   This will result in a failed pytorchjob state.

SGTM

/retest

/reset

/retest

/retest

/retest

Can you rebase

/retest

/retest

Can you rebase

done

What about other frameworks?
/cc @gaocegege
/cc @zw0610

@gaocegege I believe so.

change

when HPA

What about other frameworks?
/cc @gaocegege /cc @zw0610

This bug occurs when the number of pods is actively expanded or shrunk, like the HPA scenario.  The only situation available is pytorchjob.

MPIJob also supports replicas shrinking. But we can do the fix in another pull request.

/approve
/lgtm

/hold

@zw0610 @johnugeorge
Could we merge this PR?
@PeterChg
Thanks for your contribution! :tada: :+1:

Does this pull request assume all PyTorchJob is elastic? I'm wondering the expected behavior for traditional PyTorchJob if the replicas is scaled down.

Does this pull request assume all PyTorchJob is elastic? I'm wondering the expected behavior for traditional PyTorchJob if the replicas is scaled down.

traditional PyTorchJob will not  Encounter a scene like this

MPIJob also supports replicas shrinking. But we can do the fix in another pull request.

I will  submit another pull request

Does this pull request assume all PyTorchJob is elastic? I'm wondering the expected behavior for traditional PyTorchJob if the replicas is scaled down.

traditional PyTorchJob will not Encounter a scene like this

Great! In that case, I think we can merge this pull request. @gaocegege

Does this pull request assume all PyTorchJob is elastic? I'm wondering the expected behavior for traditional PyTorchJob if the replicas is scaled down.

traditional PyTorchJob will not Encounter a scene like this

Great! In that case, I think we can merge this pull request. @gaocegege

@gaocegege   have any other questions
This fix has run in our cluster for weeks,  A variety of types jobs are covered,  elastic pytorchjob and traditional pytorchjob include.  So far everything seems normal !

Can you rebase and resolve merge conflicts?

/lgtm

/hold

Excuse me, Is this label will affert final  mergeable?

/hold cancel
Sorry, my bad!


chore(deps): bump app to rc18

Overview

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #508 (b25916b) into main (69e71e8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #508   +/-   ##
=======================================
  Coverage   24.38%   24.38%
=======================================
  Files          29       29
  Lines        2949     2949
=======================================
  Hits          719      719
  Misses       2144     2144
  Partials       86       86


Rewrite Importers

Because the current model gets progressively slower with the lookup, going to need to do some sort of postgres COPY, temp table validation, then copy over method. I'll take a stab at this in the next week, but want to make that I'm tracking the right branch? Has the new assessment model and tests been merged into master yet?

@hunterowens Merged! Via #209.

@hunterowens Will try to tackle this on Thursday/Friday, handling some bugs & performance improvements on the front-end today & tomorrow.

@hunterowens — I'm looking into these approaches:

https://github.com/tilo/smarter_csv
https://github.com/ohler55/oj
https://xjlin0.github.io/tech/2015/05/25/faster-parsing-csv-with-parallel-processing/

Currently running the attendance task against Somerville's data so that we have a baseline to measure improvements again. Will let you know how fast/slow it clocks in.

@hunterowens
Measuring how many rows the importer can import in 5 minutes.
Baseline / current status: 37081 rows in 5 minutes
(measured from the beginning of the import_locally method on an AttendanceImporter)

https://github.com/codeforamerica/somerville-teacher-tool/commit/7dbe204509b31b7f70b62373a8407570c05b42f9: 58315 rows in 5 minutes

:+1: :zap: :100:
You'll probably get 2x that by limiting the db queries. Nice Job and enjoy the weekend.

Cut time in half for CSV, cut time by 40% for JSON via #227.


Add CSS style to adjust context menu position

This pull request adds a CSS style to adjust the position of the context menu in the VS Code editor. The "settings.json" file in the ".vscode" folder has been updated to include the following JSON code:
"editor.userContextMenuAlignment": "left",
"editor.userContextMenuOffsetX": -10,
"editor.userContextMenuOffsetY": 10
This code will adjust the alignment of the context menu to the left, and offset it by -10px horizontally and 10px vertically. These changes have been tested and verified to work correctly.

Are these real settings? If so these seem like user preferences that we shouldn't have checked in to our settings.json

Thank you for reviewing my pull request. I appreciate your feedback and apologize for any inconvenience caused. I have taken note of your comments and will ensure to make necessary improvements in my future contributions. Please let me know if there's anything specific I can do to improve the quality of my work. Thank you again for your time and attention.


DBZ-5204 Do not cache connection details in TypeRegistry.

https://issues.redhat.com/browse/DBZ-5204

@Naros Looks alike a safe approach now, thanks! I just made sure that org.postgresql.core.BaseConnection.getTypeInfo() is a cheap call and in JDBC driver code it seems so.

Thanks, @jpechane and @Naros! Much appreciated!


✨ Add Furkat and Anwar as approvers

Adding Furkat and Anwar as approvers as agreed. Russel is being removed as per his request.

/assign @maelk
/cc @dhellmann @russellb @hardys

This matches the outcome of the email conversation.
Thanks!
/approve

/lgtm

/test-integration


QEMU mode for 32bit binary on 64bit host

I would like to fuzz a binary compiled for x86 system on the x64 host.
I've compiled QEMU with TARGET_CPU=i386 and verified that I can fuzz the binary with the following command line:
afl-fuzz -i in -o out -Q -m none ./test @@
I have troubles with the qemu persistent mode. I have specified the address of the main function into AFL_QEMU_PERSISTENT_ADDR (I know that specifing main doesn't make much sense in terms of performance, it's done just for test). However, afl always report that it can't communicate with the fork server. I've tried also adding 0x4000000000 to the address, but it doesn't help.
Any ideas ?

have you set also AFL_QEMU_PERSISTENT_GPR=1 ?
it make sense of course to specify main, you avoid the overhead of fork().

Yes, AFL_QEMU_PERSISTENT_GPR=1 is set.

If you continue to experience the issue. set AFL_DEBUG_CHILD_OUTPUT=1 and post the output here

The output with the activated AFL_DEBUG_CHILD_OUTPUT:
AFL_DEBUG_CHILD_OUTPUT=1 AFL_QEMU_PERSISTENT_GPR=1 AFL_QEMU_PERSISTENT_ADDR=0x40000000730 afl-fuzz -i in -o out -Q -m none ./test @@
afl-fuzz++2.62d based on afl by Michal Zalewski and a big online community
[+] afl++ is maintained by Marc "van Hauser" Heuse, Heiko "hexcoder" Eißfeldt, Andrea Fioraldi and Dominik Maier
[+] afl++ is open source, get it at https://github.com/vanhauser-thc/AFLplusplus
[+] Power schedules from github.com/mboehme/aflfast
[+] Python Mutator and llvm_mode whitelisting from github.com/choller/afl
[+] afl-tmin fork server patch from github.com/nccgroup/TriforceAFL
[+] MOpt Mutator from github.com/puppet-meteor/MOpt-AFL
[*] Getting to work...
[+] Using exploration-based constant power schedule (EXPLORE)
[+] You have 8 CPU cores and 1 runnable tasks (utilization: 12%).
[+] Try parallel jobs - see /usr/local/share/doc/afl/parallel_fuzzing.md.
[*] Checking CPU core loadout...
[+] Found a free CPU core, try binding to #0.
[*] Checking core_pattern...
[*] Checking CPU scaling governor...
[*] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[*] Deleting old session data...
[+] Output dir cleanup successful.
[*] Scanning 'in'...
[+] No auto-generated dictionary tokens to reuse.
[*] Creating hard links for all input files...
[*] Validating target binary...
[*] Attempting dry run with 'id:000000,time:0,orig:test.a'...
[*] Spinning up the fork server...
[+] Loaded environment variable AFL_DEBUG_CHILD_OUTPUT with value 1

[+] All right - fork server is up.
Unsupported ioctl: cmd=0xffffffffc0246400
libGL error: failed to get driver name for fd 5
Unsupported ioctl: cmd=0xffffffffc0106440
Unsupported ioctl: cmd=0xffffffffc0246400
failed to get driver name for fd 6
Unsupported ioctl: cmd=0xffffffffc0106440
Unsupported ioctl: cmd=0xffffffffc0246400
libGL error: failed to create dri screen
libGL error: failed to load driver: nouveau
Unsupported ioctl: cmd=0xffffffff80046402
libGL error: failed to get magic
libGL error: failed to load driver: nouveau
[AFL] ERROR: no persistent iteration executed

[-] Unable to communicate with fork server. Some possible reasons:

    - You've run out of memory. Use -m to increase the the memory limit
      to something higher than 0.
    - The binary or one of the libraries it uses manages to create
      threads before the forkserver initializes.
    - The binary, at least in some circumstances, exits in a way that
      also kills the parent process - raise() could be the culprit.
    - If using persistent mode with QEMU, AFL_QEMU_PERSISTENT_ADDR is
      probably not valid (hint: add the base address in case of PIE)

If all else fails you can disable the fork server via AFL_NO_FORKSRV=1.

[-] PROGRAM ABORT : Unable to communicate with fork server
         Location : run_target(), src/afl-fuzz-run.c:200

The unsupported ioctl and libGL error shouldn't be a problem, it's printed even without persistence mode.

Also I've read in documentation that AFL_QEMU_DEBUG_MAPS=1 might help with determening the correct image base for AFL_QEMU_PERSISTENT_ADDR, but as far as I see my binary is loaded with different image base on every run of afl-qemu-trace (thanks aslr, probably).

Okey, after poking around with deferred initialization I've found out that VA where QEMU loads x86 PIE binary is 0xffffa000 instead of 0x40000000000 that's used for x64 binaries.
When I ran the persistence mode with this offset I had a crash (deferred mode runs fine):
fetch@linuxbox:~/fuzz/my$ AFL_DEBUG_CHILD_OUTPUT=1 AFL_QEMU_PERSISTENT_GPR=1 AFL_QEMU_PERSISTENT_ADDR=0xffffa730 afl-fuzz -i in -o out -Q -m none ./test @@
afl-fuzz++2.62d based on afl by Michal Zalewski and a big online community
[+] afl++ is maintained by Marc "van Hauser" Heuse, Heiko "hexcoder" Eißfeldt, Andrea Fioraldi and Dominik Maier
[+] afl++ is open source, get it at https://github.com/vanhauser-thc/AFLplusplus
[+] Power schedules from github.com/mboehme/aflfast
[+] Python Mutator and llvm_mode whitelisting from github.com/choller/afl
[+] afl-tmin fork server patch from github.com/nccgroup/TriforceAFL
[+] MOpt Mutator from github.com/puppet-meteor/MOpt-AFL
[*] Getting to work...
[+] Using exploration-based constant power schedule (EXPLORE)
[+] You have 8 CPU cores and 1 runnable tasks (utilization: 12%).
[+] Try parallel jobs - see /usr/local/share/doc/afl/parallel_fuzzing.md.
[*] Checking CPU core loadout...
[+] Found a free CPU core, try binding to #0.
[*] Checking core_pattern...
[*] Checking CPU scaling governor...
[*] Setting up output directories...
[+] Output directory exists but deemed OK to reuse.
[*] Deleting old session data...
[+] Output dir cleanup successful.
[*] Scanning 'sav'...
[+] No auto-generated dictionary tokens to reuse.
[*] Creating hard links for all input files...
[*] Validating target binary...
[*] Attempting dry run with 'id:000000,time:0,orig:test.a'...
[*] Spinning up the fork server...
[+] Loaded environment variable AFL_DEBUG_CHILD_OUTPUT with value 1

[+] All right - fork server is up.
/home/fetch/fuzz/AFLplusplus/qemu_mode/qemu-3.1.1/tcg/tcg-op.c:2646: tcg fatal error
qemu: uncaught target signal 11 (Segmentation fault) - core dumped
[AFL] ERROR: no persistent iteration executed

[-] Unable to communicate with fork server. Some possible reasons:
...

@andreafioraldi Thank you for pointing AFL_DEBUG_CHILD_OUTPUT  option. The problem is solved. Bugs:

The documentation lacks image base for x86 PIE for QEMU mode
For unknown reason QEMU wasn't able to correctly determine the end of function (ret instruction). It caused the crash. This one solved by manually setting AFL_QEMU_PERSISTENT_RET option.

this should not have been closed.

@fe7ch

Bugs:
1. The documentation lacks image base for x86 PIE for QEMU mode
2. For unknown reason QEMU wasn't able to correctly determine the end of function (ret instruction). It caused the crash. This one solved by manually setting AFL_QEMU_PERSISTENT_RET option.

can you share your target and config for bug  number 2?

@vanhauser-thc unfortunately, it's a proprietary binary that I can't share.
However, as far as I see, it has been fixed on 8th March with the commit: https://github.com/AFLplusplus/AFLplusplus/commit/81179b48290be597ac04cdcd2105d1044517ee81#diff-8fc1422430fd2e14e56388fd0bc8eb43

thanks!


Changed the package manager used in the .husky/pre-commit hook

The pre-commit hook used yarn instead of pnpm. So contributors needed to have yarn and pnpm installed in order to work with the project. I changed the hook to use pnpm.

Ah, super! Thanks for the PR! Let me know when it's not draft and I can merge it!


Collection#each should be called on the #docs array directly

Hi,
i am getting this warning with Jekyll 3.0.6
Deprecation: Collection#each should be called on the #docs array directly.
                    Called by .../src/_plugins/jekyll-asciidoc.rb:117:in `generate'.

I am using the latest jekyll-asciidoc from github.
Thanks

It seems like this is going to require a special condition if running with Jekyll 3. However, I'm not quite sure what it is asking. Do you have a reference to where this change was announced in Jekyll so that we understand what needs to change? Maybe this? https://github.com/jekyll/jekyll-redirect-from/issues/75

Sorry, no idea, just got this warning after upgrading to jekyll 3 and wanted to let you know. It's just
a warning and everything seems to work.

In jekyll/jekyll-redirect-from/issues/75, they seemed to just do a conditional check and skip the test if it's using Jekyll 3 which doesn't fix the deprecation in my opinion. I'm looking at possible solutions for the project I encountered this within.

It seems there have been some changes to site.posts. I recommend reading the following: https://talk.jekyllrb.com/t/what-is-docs-array/1425

I also experience this warning:
Deprecation: Collection#reverse should be called on the #docs array directly.
                    Called by /home/martin/git/apache/wicket-site/_plugins/tiered_archives_plugin.rb:25:in `site_payload'.

The code for this plugin could be seen at https://github.com/apache/wicket-site/blob/asf-site/_plugins/tiered_archives_plugin.rb#L25
I am not sure whether this plugin is part of Jekyll or someone has added it additionally.

Wicket Site depends on the Jekyll gem here. It relates to the same issue I linked above.
There are some significant changes to the API from Jekyll 2.5.3 to 3.0.0. Great work, @martin-g on the your fix.

I think this was resolved when 1.0.1 released (see #36). Therefore, I'm going to close it. If this is still an issue, feel free to reopen.


Fix make install-schema-es-v6  and install-schema-es-v7

What changed?
Fix make install-schema-es-v6 and install-schema-es-v7 command, it will template first then create index using the template. The only change is using static path to the template file.

Why?

How did you test it?
Run the command locally

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 01857a32-119c-49fc-98db-706ae468fc02

0 of 0   changed or added relevant lines in 0 files are covered.
69 unchanged lines in 13 files lost coverage.
Overall coverage increased (+0.02%) to 57.261%

Files with Coverage Reduction
New Missed Lines
%

common/types/shared.go
1
41.76%

common/persistence/executionManager.go
2
78.89%

common/persistence/statsComputer.go
2
95.71%

service/history/execution/mutable_state_builder.go
2
68.79%

service/history/task/transfer_active_task_executor.go
2
72.22%

service/matching/taskListManager.go
2
75.87%

service/history/task/fetcher.go
3
91.75%

common/types/mapper/thrift/shared.go
4
63.17%

service/history/task/task_util.go
4
75.14%

service/history/task/transfer_standby_task_executor.go
6
86.82%

Totals

Change from base Build 01855e4e-1a58-4c8e-a8bf-5e154102e721:
0.02%

Covered Lines:
84964

Relevant Lines:
148381

💛  - Coveralls


Jumping while focus in commit sidebar input area causes characters typed to be inputted as commit message input.

Closed via b8009ae9cf6a0bd2e08b7885013bd6c5d65da606


[BUG] Confusion when building for iOS/How AdMob works on iOS

Describe the bug
I've ran into some issues when trying to build for iOS.  This section is confusing me
Copy the library libgodot.iphone.release.fat.a you have downloaded from releases tab inside the exported Xcode project. You must override the '.a' file with "your_project_name", should be like: 'your_project_name.a'.
I'm not sure what it means.  I took it to mean I had to copy libgodot.iphone.release.fat.a into the project, and then rename it the name of my project, but that didn't work, and then I realized there was already a my_project_name.a in the project.
If I don't copy libgodot.iphone.release.fat.a into my project, it builds and runs (although it doesn't currently work), but if I add libgodot.iphone.release.fat.a I get compiler errors about duplicate symbols.
However, if I remove my_project_name.a from the directory, it complains about it missing.  I tried removing my_project_name.a, and renaming libgodot.iphone.release.fat.a into my_project_name.a, but that leads build errors with some of the Google frameworks.
At this point I'm just unsure about what exactly I'm supposed to be doing with libgodot.iphone.release.fat.a and my_project_name.a
To Reproduce
Steps to reproduce the behavior:
Just try and build iOS as described.
Smartphone (please complete the following information):

Device: iPhone 7
Xcode: 11.2.1
iOS Version: 13.3

Hey, I'm occupied right now, in a few hours I will explain better

Hey, I'm occupied right now, in a few hours I will explain better

@gustavottc No rush, I won't be able to try anything until around 8PM EST anyway.

@gustavottc No rush, I won't be able to try anything until around 8PM EST anyway.

I tried here, you need to copy the frameworks inside your Xcode project, like this:

Don't forget to:

And if you want to try run on simulator, check this issue: https://github.com/godotengine/godot/issues?q=is%3Aissue+is%3Aclosed+simulator
If still doesn't work please let me know

I tried here, you need to copy the frameworks inside your Xcode project, like this:

Don't forget to:

And if you want to try run on simulator, check this issue: https://github.com/godotengine/godot/issues?q=is%3Aissue+is%3Aclosed+simulator
If still doesn't work please let me know

Does anything need to be done with libgodot.iphone.release.fat.a?

Does anything need to be done with libgodot.iphone.release.fat.a?

@sjhennion yes, you still need to replace to the .a file of your game, just like you did:
However, if I remove my_project_name.a from the directory, it complains about it missing. I tried removing my_project_name.a, and renaming libgodot.iphone.release.fat.a into my_project_name.a, but that leads build errors with some of the Google frameworks.

@sjhennion yes, you still need to replace to the .a file of your game, just like you did:
However, if I remove my_project_name.a from the directory, it complains about it missing. I tried removing my_project_name.a, and renaming libgodot.iphone.release.fat.a into my_project_name.a, but that leads build errors with some of the Google frameworks.

The build erros of Google framework are solved doing the steps that I mentioned above

The build erros of Google framework are solved doing the steps that I mentioned above

I think I'm still missing something here @gustavottc.
I started from scratch, and followed your steps above.  After completing those steps, I built the project, and it built successfully, but I didn't try running just yet.
I then removed my_project_name.a from the directory, placed libgodot.iphone.release.fat.a into the directory, and renamed it my_project_name.a, and that resulted in the some of the build errors I was seeing yesterday:

I think I'm still missing something here @gustavottc.
I started from scratch, and followed your steps above.  After completing those steps, I built the project, and it built successfully, but I didn't try running just yet.
I then removed my_project_name.a from the directory, placed libgodot.iphone.release.fat.a into the directory, and renamed it my_project_name.a, and that resulted in the some of the build errors I was seeing yesterday:

You need to place the framework AppTrackingTransparency, but as "Optional" just like the second picture that i send:

This "___darwin_check_fd_set_overflow" i never saw that, just make sure to add these frameworks like the picture above

You need to place the framework AppTrackingTransparency, but as "Optional" just like the second picture that i send:

This "___darwin_check_fd_set_overflow" i never saw that, just make sure to add these frameworks like the picture above

The UniformTypeIdentifiers, i never needed to add, but try to add to see if works, i guess it only missing these frameworks to work your AdMob

The UniformTypeIdentifiers, i never needed to add, but try to add to see if works, i guess it only missing these frameworks to work your AdMob

the explanation of starting to appear these errors in XCode after changing the '.a' file is that: this '.a' file is Godot's binary, for example, it is Godot itself where the phone runs the game based on binary, this binary is modified and contains the AdMob change inside it, but for AdMob to work it needs some frameworks and some extra settings like change the .plist which is what you are doing, it is boring to do today, but it is the only one way.
In the future I will include the new plugin system for iOS, which will work similarly to Android, that is, it will be much simpler, but it needs to be included in a new version of Godot for me to start working, it will probably only be possible in version 3.2.5 onwards

the explanation of starting to appear these errors in XCode after changing the '.a' file is that: this '.a' file is Godot's binary, for example, it is Godot itself where the phone runs the game based on binary, this binary is modified and contains the AdMob change inside it, but for AdMob to work it needs some frameworks and some extra settings like change the .plist which is what you are doing, it is boring to do today, but it is the only one way.
In the future I will include the new plugin system for iOS, which will work similarly to Android, that is, it will be much simpler, but it needs to be included in a new version of Godot for me to start working, it will probably only be possible in version 3.2.5 onwards

@gustavottc Thanks for the explanation, that makes it much clearer as to what's going on.
It looks like UniformTypeIdentifiers.framework is also only available as of iOS 14+, so I think my issue is pretty clear at this point, I need to upgrade Xcode and the iOS SDK.

@gustavottc Thanks for the explanation, that makes it much clearer as to what's going on.
It looks like UniformTypeIdentifiers.framework is also only available as of iOS 14+, so I think my issue is pretty clear at this point, I need to upgrade Xcode and the iOS SDK.

@sjhennion but this is a Warning, you can ignore this, the error is because is missing the AppTrackingTransparency.framework  (ATTrackingManager) as "optional"

As you can see, i can use the AdMob on iOS 13.0 on Simulator without these erros

By the way, this is my XCode version: Version 12.3 (12C33)

@sjhennion but this is a Warning, you can ignore this, the error is because is missing the AppTrackingTransparency.framework  (ATTrackingManager) as "optional"

As you can see, i can use the AdMob on iOS 13.0 on Simulator without these erros

By the way, this is my XCode version: Version 12.3 (12C33)

I must still need to upgrade something, it's possible I'm a few versions back, because I don't even have AppTrackingTransparency available as a linkable binary.  Will have to get back to this later tonight.  Thanks again for all the help!

I must still need to upgrade something, it's possible I'm a few versions back, because I don't even have AppTrackingTransparency available as a linkable binary.  Will have to get back to this later tonight.  Thanks again for all the help!

@gustavottc Sorry, my MBP is older and only has 128gb of disk space, it was quite an effort to update to Big Sur and get Xcode 12.3 installed.  But, everything is updated and the example project is working now, and I've got ads!
Only thing is I'm seeing this spammed to the log multiple times a second:

@gustavottc Sorry, my MBP is older and only has 128gb of disk space, it was quite an effort to update to Big Sur and get Xcode 12.3 installed.  But, everything is updated and the example project is working now, and I've got ads!
Only thing is I'm seeing this spammed to the log multiple times a second:

Are you testing on a Real Device? What's the version of your iOS?

Are you testing on a Real Device? What's the version of your iOS?

Ok, i saw here: iOS Version: 13.3
Unfortunately this error is related to the iOS version less than 13.4, and not exclusive to AdMob: https://stackoverflow.com/questions/58321114/why-i-get-the-console-warning-process-kill-returned-unexpected-error- 1-when
Try to update your phone or run on a simulator, don't worry about these messages

Ok, i saw here: iOS Version: 13.3
Unfortunately this error is related to the iOS version less than 13.4, and not exclusive to AdMob: https://stackoverflow.com/questions/58321114/why-i-get-the-console-warning-process-kill-returned-unexpected-error- 1-when
Try to update your phone or run on a simulator, don't worry about these messages

On 13.0 Simulator these errors doesn't appear for me

On 13.0 Simulator these errors doesn't appear for me

@gustavottc that explains it, I'm on 13.3
Thanks again for all the help, really appreciate it!  I saw you added the [Feature] for the wiki, not sure whether you were going to do a first pass of getting information in there, but I'd be happy to add more details in based on my findings.

@gustavottc that explains it, I'm on 13.3
Thanks again for all the help, really appreciate it!  I saw you added the [Feature] for the wiki, not sure whether you were going to do a first pass of getting information in there, but I'd be happy to add more details in based on my findings.

@sjhennion i put right now the Wiki to be editable to public it would be amazing if you contributed to the Wiki
Or even if you have something that you think needs to be improved in Readme.md, you can create a PR and I will commit
I will clone the issue since the problem was solved, thanks!

@sjhennion i put right now the Wiki to be editable to public it would be amazing if you contributed to the Wiki
Or even if you have something that you think needs to be improved in Readme.md, you can create a PR and I will commit
I will clone the issue since the problem was solved, thanks!


[Optimizer][Bug] Gradient is mutated in the Adam optimizer

In the implementation of Adam: grad, mean and var are all mutated (See https://github.com/apache/incubator-mxnet/blob/master/src/operator/optimizer_op-inl.h#L1307-L1315). However, the FMutateInput flag is only set to {2, 3}, which should be {1, 2, 3}. (See https://github.com/apache/incubator-mxnet/blob/master/src/operator/optimizer_op.cc#L699)
To reproduce the bug, you may check the value of the gradient before/after adam_update https://github.com/apache/incubator-mxnet/blob/master/python/mxnet/optimizer/optimizer.py#L1226-L1227 .
We can add the following into optimizer.py
grad1 = grad.asnumpy()
adam_update(weight, grad, mean, var, out=weight, ...)
grad2 = grad.asnumpy()
import numpy as np
np.testing.assert_allclose(grad1, grad2)

Create an adam optimizer with wd=1E-3 and train any model. You will find that the gradient has been mutated.

On looking at other optimizer's updater,
These are the cases where original grad is being updated.
Will update the PR accordingly.
RMSPropAlexUpdate
https://github.com/apache/incubator-mxnet/blob/71861238743fcd8177afe52d1562d9078ac547de/src/operator/optimizer_op-inl.h#L1600-L1619
RMSPropUpdate
https://github.com/apache/incubator-mxnet/blob/71861238743fcd8177afe52d1562d9078ac547de/src/operator/optimizer_op-inl.h#L1691-L1708
FtrlUpdate
https://github.com/apache/incubator-mxnet/blob/71861238743fcd8177afe52d1562d9078ac547de/src/operator/optimizer_op-inl.h#L1784-L1803

Thanks @kshitij12345 !


This PR adds the following:

The beginnings of a CLI tool.
A new command for generating new apps.
A typical *Potion app delegate.
A BP::TVMLScreen which abstracts the gritty details of setting up a TVApplicationController. It also makes it a little more explicit that you are intending to use TVML.

@jamonholmgren @danhealy What do you think of this?

Seems like a great start!

Looks great! Should check with @kevinvangelder ... he's spent more time in tvOS land.

Why is this not merged?

Happy to merge if you want! Haven't done much with BlackPotion since we gave it a spin.

@jamonholmgren @wndxlori I'm wondering if it makes more sense to continue development of BlackPotion outside of the Infinite Red GitHub organization. Considering this repo started empty and all of the code so far has been written by me. The only Infinite Red contribution so far is the name. What do you guys think?

@andrewhavens I'm happy to transfer it to you. Will do when I get to a computer

@andrewhavens I made you an admin -- go ahead and transfer it to yourself. (This is how Github wants us to do it I guess)


Print AuthJWTException message

While testing protected endpoint, I put wrong access token (deleting some values in actual access token)  in Postman. I tired to print AuthJwt exception but it gives nothing. When I donot supply access token, no any exception message is printed.
For example: access token is eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c
But I put access token in Postman as eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJz to test
It gives nothing while printing  print(e).
except AuthJWTException as e:
       print(e)
       return ErrorResponseModel(
                        "Something went wrong",
                        status.HTTP_200_OK,
                        status.HTTP_401_UNAUTHORIZED
                    )

Code in main.py
application.add_exception_handler(AuthJWTException, authjwt_exception_handler)

How toc check individual exception?

you can try to use e.message and e.status_code if you only print e it gives nothing

you can try to use e.message and e.status_code if you only print e it gives nothing

When I supply wrong access token , it gives status code 422. It needs to give 401.

The reason I use status code 422 because sending invalid fields will result in a 422 Unprocessable Entity response. Can you give reasonable use 401 instead of 422 when the token is invalid?

Yeah I got it.


feat: adds user survey workflow and link in README

FYI @alessbell looks like the survey isn't actually publicly available:

You need permission
This form can only be viewed by users in the owner's organization.

@travigd oops! Thanks for the heads up! I've just pinged my colleague - this should be fixed today, at which point we'd still very much appreciate your feedback :)

Hi @travigd the link is fixed. Thank you very much for letting us know.


go: 1.11.4 -> 1.11.5

https://groups.google.com/forum/#!msg/golang-announce/mVeX35iXuSw/Flp8FX7QEAAJ
We have just released Go 1.11.5 and Go 1.10.8 to address a recently reported security issue. We recommend that all users update to one of these releases (if you’re not sure which, choose Go 1.11.5).
This DoS vulnerability in the crypto/elliptic implementations of the P-521 and P-384 elliptic curves may let an attacker craft inputs that consume excessive amounts of CPU.
These inputs might be delivered via TLS handshakes, X.509 certificates, JWT tokens, ECDH shares or ECDSA signatures. In some cases, if an ECDH private key is reused more than once, the attack can also lead to key recovery.
The issue is CVE-2019-6486 and Go issue golang.org/issue/29903. See the Go issue for more details.
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I updated nixos (nixos-version shows 18.09.2096.2ff7ef373e5), and noticed that go version is still returning go version go1.11.4 linux/amd64.
The reason for this is that which go returns:
 /nix/var/nix/profiles/default/bin/go`, which contains `exec -a "$0" "/nix/store/jjgbr7nwdhp68zl550w6a8jhgx19lx3a-go-1.11.5/share/go/bin/.go-wrapped"  "${extraFlagsArray[@]}" "$@"`

By the name, go-1.11.5 should be go-1.11.5, not 4, but /nix/store/jjgbr7nwdhp68zl550w6a8jhgx19lx3a-go-1.11.5/share/go/bin/go version returns go version go1.11.4 linux/amd64.
The thing is, I do have go 1.11.5. For example, /nix/store/1ph1ncwid9fdksqrz7rp49vgp0lh7623-system-path/bin/go version returns 1.11.5.
Is there something wrong in my patch, or is there something more that needs to be done on the client end?

Yeah, it was a result of a conflict with nix-env. I removed everything from nix-env and everything works now.
Thanks.


Show Job StatusDescription

If a job fails, we aren't showing this so it's hard to know why it failed.

Resolved


Add color modifier support to token-transformer

Is your feature request related to a problem? Please describe.
Our designer uses Token Studio in Figma, and hands it over to us developers.
We pass the tokens through token-transformer to resolve all the references to concrete color values, then transform it into a Theme object for our react-native-paper components.
In a recent version (Feb? March?) of token studio, support for color modifiers was added; our designer applied it to the Figma designs.
However, after grabbing the token files and pulling them through token-transformer, the colors in our app are no longer correct; basically, the modifiers are not being applied. The modifiers are retained in the output JSON, but this would mean our application code needs to check for the existence of and apply the modifier itself.
Describe the solution you'd like
That token-transformer applies the modifiers, so that a new RGB(A) or hex value is output that can be used in our app.
Describe alternatives you've considered

Writing our own transformer or post-process
Ask the designer to revert the tokens to hardcoded values (e.g. no longer use color-modifiers)
Ask the designer if there's a way to export the resolved color values of modified colors
Manually copying the resolved colors from Figma's generated CSS, but that would be a step back

Additional context
Here's a snippet from the token file as we get it / as it's stored in the git repository by token studio; note that the full version contains named colors matching Material Design's naming scheme that will refer to the ref set of colors again.
ref.json:
{
  "ref": {
    "color": {
      "base": {
        "grey": {
          "value": "rgb(105, 105, 113)",
          "type": "color"
        }
      },
      "grey": {
        "10": {
          "value": "{ref.color.grey.500}",
          "type": "color",
          "$extensions": {
            "studio.tokens": {
              "modify": {
                "type": "lighten",
                "value": "0.98",
                "space": "srgb"
              }
            }
          }
        },
        "500": {
          "value": "{ref.color.base.grey}",
          "type": "color"
        }
      }
    }
  }
}

theme.json:
{
  "sys": {
    "theme": {
      "color": {
        "core": {
          "background": {
            "value": "{ref.color.grey.10}",
            "type": "color"
          }
        }
      }
    }
  }
}

The current output; note that in the ref colors, it retains the "$extensions" key, but in the theme it only picks the value of grey.10.
{
  "sys": {
    "theme": {
      "color": {
        "core": {
          "background": {
            "value": "#696971",
            "type": "color"
          }
        }
      }
    }
  },
  "ref": {
    "color": {
      "base": {
        "grey": {
          "value": "#696971",
          "type": "color"
        }
      },
      "grey": {
        "10": {
          "value": "#696971",
          "type": "color",
          "$extensions": {
            "studio.tokens": {
              "modify": {
                "type": "lighten",
                "value": "0.98",
                "space": "srgb"
              }
            }
          }
        },
        "500": {
          "value": "#696971",
          "type": "color"
        }
      }
    }
  }
}

The expected / desired output:
{
  "sys": {
    "theme": {
      "color": {
        "core": {
          "background": {
            "value": "#FCFCFC",
            "type": "color"
          }
        }
      }
    }
  },
  "ref": {
    "color": {
      "base": {
        "grey": {
          "value": "#696971",
          "type": "color"
        }
      },
      "grey": {
        "10": {
          "value": "#FCFCFC",
          "type": "color",
        },
        "500": {
          "value": "#696971",
          "type": "color"
        }
      }
    }
  }
}

token-transformer does support color modifiers!
However I would recommend to start using sd-transforms, as this library will make token-transformer obsolete. http://npmjs.com/@tokens-studio/sd-transforms

@six7 looks like from comments this was solved with better guidance and docs. Can we close?


add discussions-to link to SNIP-2

https://github.com/starknet-io/SNIPs/discussions/34

wouldn't it be better to number SNIPs that are inspired by ERC's to have same number or prefix the number by something, for example SNIP-E20?

hmm, what was the reason behind going with independent identification? I don't think we loose anything by adding a namespace for SNIPs directly inspired from EIPs. And we gain good DX


patch CVE-2022-32149

Signed-off-by: Jonathan Marcantonio<EMAIL_ADDRESS>Addresses:

https://github.com/stolostron/backlog/issues/26360

/lgtm


Address cv2.resize float16 issue in scale_cam_image

https://github.com/jacobgil/pytorch-grad-cam/issues/198
https://github.com/jacobgil/pytorch-grad-cam/issues/493
Putting in a PR to go along with my issue post.
cv2.resize doesn't support float16 which causes this step to break. This fixes the issue with not downstream consequences that I can identify since the resulting image gets converted to lfoat32 anyways.

any chance we can merge and release? @Christophe-Foyer

Merged. Thank you for the contribution.


scenes

Hi!
I was wondering if you already have some kind of scene management or intend to add it. Right now the way I see to make it happen is overwriting the update function to draw the next screen. Maybe it'd be better to make it more explicit for the lib, I also can provide help implementing it.
Keep on the good work!

Hi,
I'm not sure what kind of function or struct you want. Isn't it enough e.g. a Scene interface in examples/blocks?
Thank you!

Hi! Yes, that is it. Any chance to making it a part of the platform?

I tend not to want to add such scene system to Ebiten since scene system is a kind of upper later thing and I'd want to keep Ebiten as 'dumb' thing as possible. I think it is not hard to make it yourself, isn't it?


(feature idea) Configuring DbContexts

This isn't even a feature request, just an idea I had that may or may not be of interest or use.
The basic idea is to use an attribute, a generic interface and an extension method to let fluent-api configuration be done "closer" to a POCO class (i.e., in a class kept in the same source code file as the POCO class).
Full write-up at https://jumpforjoysoftware.com/2019/08/configuring-dbcontexts/.

@markolbert This is a fairly well-known pattern--for example, see https://blog.oneunicorn.com/2012/03/26/code-first-data-annotations-on-non-public-properties/
What, specifically, are you suggesting that we add to EF Core for this?

Thanx for the reference, @ajcvickers; I enjoyed reading the article. Among other things, it suggest to me that there are likely a number of alternative, idiosyncratic ways of simplifying fluent model building. With simplification being in the mind of the coder :).
So it's probably not worth while including any one approach in EF Core. How would you decide which ones to include and which ones not to?
I'll close this thread.


Claim: Small changes in design + text

Change "GET NOW" to "YOU CAN GET"
Change "Your Next Daily Income:" to "Your next daily claim:" and change the font to Bold
Switch position between the button and the stats line
Remove learn more Button
New text: "
Claim & spend it on
things you care about
"

https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/23d218ca-4c56-4348-a708-4bfa2a851614/Claim-2
https://xd.adobe.com/spec/8f259bcd-550e-4f0c-40bb-bdce2ed36fe5-f914/screen/bfcc9047-dd71-41c6-89c9-d90084894417/Claim-3

Checked on Dev environment
According to mockup text should be Claim & spend it on
https://www.screencast.com/t/g8gI1pbZWRY

Checked on preview
According to mockup text should be G$1
https://www.screencast.com/t/4w0oHsmvjc

Checked on Dev environment
Look as per requirements


"-" will be Syntax error

The old version can not support maps.
And the latest version can not use "-" like "util:unixtime() - calendar:time_to_seconds(time())."
"-" will be syntax error. "*", "/" and "+" are fine.
There is another discussion: https://github.com/ChicagoBoss/ChicagoBoss/issues/520

closing, fixed in #17


Switch from backrefs to explicit references

We currently use SQLAlchemy's backref property in a number of places, e.g. this line enables the User.projects attribute.
https://github.com/pypa/warehouse/blob/5dfcbf0a77861a5d3b30eaa6bca2e307c8b26d88/warehouse/packaging/models.py#L128
Backrefs are confusing to new contributors, however, since an attribute like user.projects is not listed on the User model where it would be expected.
We should explicitly define these relationships on both sides instead using backrefs. In addition, if possible, we should find a way to disable the use of the backrefs parameter to prevent this from being used in the future.

Also noted that since SQLAlchemy 2.0, backref is considered legacy, in favor of back_populates
https://docs.sqlalchemy.org/en/20/orm/backref.html


fuzzy / fuzzies in output

Add a switch when adding to the buffer to use plural when there's more than a fuzzy in the file

:-1: Not a big fan, that would just make potodo parsing (if needed) more complicated.


Nukie Outpost is no longer a station map

fixes #30081
fixes #28678
fixes #29060
About the PR

This should hopefully stop station events from hitting the nukies, or other random things related to being a station, and expecting all the trimmings that entails, from occuring to nukies.
Why / Balance

Technical details

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

not player facing.


about Beginner’s Webapp Tutorial: a step-by-step tutorial to enable Shiro in a web application

Because of  Stormpath Joins Forces With Okta. New user of Shiro cannot create new Stormpath  account that they cannot follow up the web demo. So the step-by-step demo using Stormpath account should switch to another platform. Pls kindly modify the Tutorial that we could catch up with this demo. Really thanks!

+1
@littlebox692  Can you scope your changes on master to just this issue?

This PR is based on the 2.0-api-design-changes branch, it would be better to make a proposal based on master branch. I close this PR.
@littlebox692 can you push a new PR base on master please?


Disallow RegExp "y" flag in ES5 mode

Previously, a regular expression with the "y" flag set would correctly parse in ES5 mode, though the "y" flag was first introduced in ES6. This change:

Only permits the "y" flag when ecmaVersion is 6.
Adds tests to verify that "u" and "y" (both new in ES6) fail to parse when ecmaVersion is less than 6.

An "s" flag is allowed in both ES5 and ES6 mode, though neither specification includes it. I left this as-is.
Currently the warning message for invalid regular expression flags gives the location as the start of the regular expression. It may be more helpful to give the location of the invalid flag, but I left this as-is for now.
ES5 spec text: http://www.ecma-international.org/ecma-262/5.1/#sec-<IP_ADDRESS>
ES6 spec text: http://www.ecma-international.org/ecma-262/6.0/#sec-regexpinitialize
Fixes #363

s looks wrong to me too, probably a mistype or an early version of sticky flag. I think it should be removed.

Updated - "s" is gone now too.

Thanks!


Cannot attach listeners via config

I am trying to attach listeners to my REST resources using the config
(zf-apigility.doctrine-connected.resource_name.listeners), but they don't get triggered.
I tried to debug the problem and got the following observations:
In the file: zfcampus/zf-apigility-doctrine/src/Server/Resource/DoctrineResource.php line 148.
the method getEventManager is called which is returning a new instance of the event manager if one is not set:
$this->setEventManager(new EventManager());
This is instantiating a new Event Manager without specifying the Shared Event Manager for it.
Example:
$this->setEventManager(new EventManager($this->getSharedEventManager()));
This is causing the code in: /projects/carrier-rules-engine-apigility-upgrade/vendor/zendframework/zend-mvc/src/Service/ServiceManagerConfig.php line 99 to return a brand new event manager, and loosing all attached events:
// If the instance has an EM WITH an SEM composed, do nothing.
if ($eventManager instanceof EventManagerInterface
   && $eventManager->getSharedManager() instanceof SharedEventManagerInterface
 ) {
   return;
 }
$instance->setEventManager($container->get('EventManager'));

@rradutzu Thanks for reporting this issue.
I was testing it and your solution seems to be correct for me.
Would you like submit PR with your solution and add some tests for it?


ssr(serve side render) -> csr(client side render) gradual switch

ssr is to firstly be launched when browsing starts;
if browser supports wasm, all bsr dependencies start downloading and try to cache;
after dependencies downloaded, switch into csr mode

Thanks for contacting us.
This is not something we plan to do in the near future.

@mkArtakMSFT  the request which is not within near plan does not have to be closed and not be seen by other people from community

@juepiezhongren We don't mean to shut down any discussion about this. It's just that keeping issues open is an administrative burden. We have a lot of issues flowing through, and the open/closed flag is an important signal about whether there's work to do on them. The only way we can manage the project effectively is to close things when there's no further action for us to take on them. Hope that's OK.

thx, steve @SteveSandersonMS.
but i have to say switch is important, considering the wasm-related diversity for browsers.
there r quite a lot scenarios for switch:

safari supports wasm but with too much limitations, ssr or csr switch must be configurable;
wasm 2.0 and wasm mvp r different which even impact bcl' implementation, switch needs to cover different wasm-bcl' imple.
don't forget timing, considering dependencies-download, if it costs too much time, the switch will be forbidden.
....
i have to say there will be too many requests for switch, which would result in a much more solid web-solution than current js ones.


Add firewall and sysctl management modules

Add sysctl and firewall management modules.
Also add node definition for simple nat router to make sure the iptables rules look sane.

Merged


circleci: Fix CircleCI OOM errors

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

Summary
Occasionally CircleCI jobs triggered by PRs fail because of OOM errors (see https://circleci.com/gh/stellar/go/6268#usage-queue/containers/0 for an example). The cause and fix for this issue is outlined here:
https://github.com/golang/go/issues/26186#issuecomment-435544512

Great, thanks!


Cyrillic alphabet is printed incorrectly, is there any options?

Hello 👋
You can specific print language by passing characterSet prop on <Printer /> component.
e.g.
import { render, Printer, Text, Cut } from 'react-thermal-printer';

const data = await render(
  <Printer type="epson" width={42} characterSet="pc855_cyrillic">
    <Text>Hello World</Text>
    <Cut />
  </Printer>
);

Alos you can find all supported character sets in here.

Thank you!


Implement Tenant and Component Tokens

As a developer I want to be able to add TenantToken and ComponentToken to establish connection with Andy X Node

PR Merged


chore: release @tutorialkit packages v0.0.3

Bump @tutorialkit packages to version 0.0.3 and generate changelogs.

This was created while testing PAT. Workflows run now automatically. 🎉


Saved files without changes automatically closed between sessions

Type: Bug
If you have one or more saved file(s) open for editing, without any changes to it/them (as in, you're just viewing it/them), and you close/manually re-open VS Code, the file is still visible in the Explore pane and shown on the tab bar.  If you close VS Code and re-open VS Code by way of double-clicking another file to open/edit it, the previous open file(s) is/are no longer open at all and only editors with unsaved changes to them are left (in addition to the file you launched VS Code with).
Editor Previews and Editor Previews from Quick Open are both disabled.

A bit hidden in settings, but try with window.restoreWindows set to preserve.

Confirmed. That fixed the issue. The setting's values should be tweaked in my opinion, and "preserve" should be the default value as "all" is not truly all - it's conditional and is actually only "some".

Yes, we can change the default to be something that is named differently, but for those that have "all" configured in settings today that are used to todays behaviour, we would break them.

That's understandable, and is why I said if the option can't be changed it should at least not be named "all" (since it really isn't all).  Maybe rename it to "unsaved" or "some" to better reflect its true behavior.  If it's just the name/string and not the actual ID of the menu option, it shouldn't affect anyone/anything.  Then you can change "preserve" to be "all" since that really is what it does.


Switch back to higher level org from hierarchy doesn't properly work when org admins logged into console as the B2B administrative portal

Describe the issue:

Login into Console
Create an organization and switch to it
Create a user and assign the Administrator role of the console application
Copy the console access URL from the sub org's console settings page
Access the console from the copied link and give the credentials of the user created in step 3
Now navigate to the organization's section and create another org
Switch to the created org
Then switch back to the logged-in org
It will be reset to the root org and get not permitted (403) error message for operations

https://github.com/wso2/product-is/assets/25483865/db7c2226-61e2-40a7-9e98-7f8942187159

This issue is fixed with following PR

https://github.com/wso2/identity-apps/pull/5257


Fix: Content getting hidden under the top navigation bar when a hash link is clicked

Closes #407

There is one caveat with this fix if someone directly goes to the link: Click here the fix will not work.
To solve this, we will have to make some CSS and layout changes like adding top padding to the content container, so that the main content is always pushed down and is not below the navigation bar.

@kryptonian41 Thank you for your contribution but this issue has been closed by PR #661.


High CPU usage observed for GET requests with larger payloads

Description
When the request payload is larger than the buffer for HTTPS get requests, high CPU usage is observed in gateway
Steps to Reproduce

Create a sample API in APIM
Use the following HTTPS mock backend for the API
Mock_Java_Backend.txt
Invoke the API with a HTTPS get request with the below payload
Request_Payload.txt

Affected Component
APIM
Version
<IP_ADDRESS>
Environment Details (with versions)
JDK version: open jdk 11.0.15
Relevant Log Output
No response
Related Issues
No response
Suggested Labels
No response

Closing this issue as a patch type issue is already created https://github.com/wso2-enterprise/wso2-apim-internal/issues/902.


Support docker context when using autoIgnoringUserProperties

This change enable support for docker context
I used existing option of dockerconfig.source=autoIgnoringUserProperties
when this option is enabled, it use docker-java configuration that is aware of context already
This PR requires fix from docker https://github.com/docker-java/docker-java/pull/2130 to support TLS in context, other use cases could work without this fix.

With https://github.com/docker-java/docker-java/pull/2130 being merged and release, it would be lovely to get this one merged and releases

kindly reminder, could we get it this merged?


Support conversion from &String to Slice

related to https://github.com/fjall-rs/fjall/issues/99

I thought Rust would use the From<&str>, because when you have a fn f(s: &str), you can do:
let a = String::new();
fn f(&a); // works

But apparently not

Yea, unfortunately it doesn't seem to trigger the implicit deref for From impls.


Determine output format of the pipeline

The current pipeline uses plain CSV as output format. A large collection of CSV files is the result.
(Is it right that there are over 1TB of CSV files?)
They are then processed and inserted into a PostgreSQL Database.
The PostgreSQL database is currently used by the bb_analysis/ tracking project to get the output of the pipeline and to store ground truth data.
The PostgreSQL database will be a traffic bottleneck for any map-reduce (spark) analysis. Also, it is quite big and needs a lot of storage. It would be preferable to have a storage format that is usable
for anyone.
My alternative proposal for the output format is capnproto or another fast serialization format. I suppose the capnproto format should use at least around 5 times less storage then the CSV files, mostly because they use binary encoding. If we apply some compression then even more. The new format would allow us to store the results on any PC.
The schema would be something along this lines.
struct Detection {
  id @0: UInt64;
  localizer_id @0: UInt64,    # the same for all detection based on the same localizer location
  x @0: UInt16;
  y @0: UInt16;
  rotation_z @ : Float32;
  rotation_y @ : Float32;
  rotation_x @ : Float32;
  radius @ : Float32;
  decoded_tag @: Float32;
  track_ids: List(UInt64);
  [Addition data e.g. autoencoder...]
}

# Corresponds to an image in the video
struct Frame {
  id @0: UInt64;
  cam @2: Text;
  world_coordinates_transform @3: List(Float);
  detection: @6: List(Detection);
}

# Corresponds to a video
struct FrameContainer {
  id @0: UInt64;
  hd_video_filename: @1: Text;
  preview_video_filename: @2: Text;
  cam @2: Text;
  from @4 :Date;
  to @5 :Date;
  frames @ : List(Frame)
}

We would then have one capnproto file for each video. The map-reduce analysis could then operate on these files.
As it is a very central issue which affects many people, we should discuss it in detail.

You are saying that the PostgreSQL database needs a lot of storage. Do not forget that databases are optimized for least overhead and maximum lookup time. If you start with a few billion single files containing e.g. capnproto serialization, you are basically rebuilding a database. Just with slower lookup and no indices.
Additionally, single files may have slight overhead (depending on the underlying filesystem) so that they will most likely take /more/ storage space instead of less (compared to a dedicated DB).
However, there are probably databases that are optimized for lookup (indices), storage space (least overhead) AND speed even though PostgreSQL might be bad at the latter.
Other than that, there are possibilities to cache the required input/output from and to the database in a very fast way and sync that later (I have used qless with a redis DB for a similar task - but there are probably a lot of possibilities).

I have my doubts that there exists any database that is capable of handling the
map-reduce/spark use-case well and still runs on a single server.
When doing map-reduce, the whole data is sent to multiple clients (probably 2-4).
Any database server would now have to read the full database, inevitable
becoming a bottleneck.
With many (capnproto) files, they are downloaded to the clients and deserialized there.
The only limiting factor here is the speed of the HDD. It might even fit onto
an SSD as the data should get smaller. I see no database that can compete with that in terms
of speed and simplicity.
The only index key we have is the time stamp. We can organize the directory structure
around the timestamp to ensure fast access times. Of course, we may need to tar
the capnproto files if they are too small.
What would be a use-case where it is beneficial to have the
detections in a database?

I agree that we soon have to review the post-pipeline processing stages in the light of parallel processing architectures coming up. Franziska will present her thoughts on this matter on monday. So I guess we'll learn about technologies that work well with Spark.

Any database server would now have to read the full database, inevitable
becoming a bottleneck.

I sure hope not database system has to read the full table from disk to access certain entries. Ideally it doesn't even have to read the full index.

The only limiting factor here is the speed of the HDD

And the speed of the FS - because we still do the same things we would in a database (index by timestamp) just without a database. So we do the very same thing and just leave use the integrated database of the filesystem, which might or might not be optimized for our use case.
(Don't forget that a filesystem also has to figure out where exactly the file '/foo/bar/2016/03/05/1000_2000.baz' is stored, just like a database has to figure out where the entry is).

What would be a use-case where it is beneficial to have the
detections in a database?

Fetching entries via some index (e.g. the timestamp). And then e.g. writing back metadata to the database. Most databases also have integrated failsafes that trigger e.g. if you are just saving a .capnproto file and the power goes off (and then the file is lost..?). Even better: most databases also have transactions! So if you are writing two .capnproto files and the second doesn't work (e.g. because someone has accidentally spawned two workers working on the same data, it would also not write the first file!)
But yeah, if we have only a small amount of data, no metadata and need no transactions and failsafes, we can as well use just the built-in FS. (We should choose a decent one, though).
And I obviously have no idea of spark and what works best with that :)

The output format will be Cap'N Proto. See this repository: https://github.com/BioroboticsLab/bb_binary


Fixed the "here" link.

Previously showed Page Not Found. Now it directs to the correct URL.

Thanks for this fix @MatinSasan


feat: Add mapping to class support

Coverage decreased (-22.0%) to 77.506% when pulling a4c81d07bb39ec03d9c725b185fe9f9450b9dd7c on Add-mapping-to-class-support into 3bddad28e9d63ac534bc63dc5c89ae069c2f1b21 on develop.


Add support for CREATE2 clones and address prediction

Thanks @Arachnid for the PR, I posted a comment on the CREATE3 PR: https://github.com/wighawag/clones-with-immutable-args/pull/17 where I mentioned that it would be good to also think about  CREATE2 in regard to naming convention
right now the naming convention might be pretty confusing if we accept both PR.
clone2 might work but predictAddress could apply to both
I did not think that through but cloneWithCREATE2 and predictAddressOfCloneWithCREATE2 would at least have more clear meaning.
What do you think ? @ZeframLou  ?

I think clone2 is to clone as create2 is to create, so that naming makes sense, while also remaining fairly compact.
predictAddressOfCloneWithCREATE2 seems way too long. Could we do something like addressOfClone2?

Yes, clone2 and addressOfClone2 works for me

Yeah this seems good, will update my PR

I've done the rename, and also refactored the library to return creation bytecode, which is necessary in order for addressOfClone2 to be a view function.
Unfortunately this does have some impact on gas usage; testGas_clone goes from 64,579 gas to 64,738 (+159 gas).

Thanks for that.
I am personally ok with extra gas but another solution would be to duplicate the code between the view function and the write function at the cost of later maintainability.


Extending Docstrings on Potential_function.py

I am extending the docstrings in Potential_function.py to describe the parameters and and example of how to use the function

@Irwin-Blair For the first function i know you are creating a user interface are you does the user use the interface to imput the potential or do they type it in python?


Cloud Foundry forward-compatibility

Per Cloud Foundry documentation use of $VCAP_APP_HOST and $VCAP_APP_PASSWORD are incompatible with newer (Diego-based) Cloud Foundry deployments. This change makes this repository forward compatible with the new cloud.gov environment deployment in AWS GovCloud.

Current coverage is 84.98% (diff: 0.00%)

No coverage report found for master at c31a753.

Powered by Codecov. Last update c31a753...93b323d


Softlock Hiding Place Drawer Press room Leon B

The hiding place drawer in the press room has the 'Film - Hiding Place' marked as a required item according to the location logic, but the drawer does not unlock until you use the film in the dark room.
This makes it possible to soft-lock in the Leon B scenario if the hallway fuse is in this drawer.

This is fixed as of today's release.
Closing this issue but, if you run into any bugs, feel free to open another issue. 😃


Readiness probe failed when huge tensor file send to predictor in triton

/kind bug
What steps did you take and what happened:
I have deployed the onnx model to the server using protocol v2 (trition).
My model is supposed to get an  [1, 3, 640, 640] shaped FP32 tensor.
However, when writing the corresponding tensor as json, it is 16MB.
When the predictor receives the corresponding tensor, it stops responding, and a timeout occurs during the readiness probe process and is restarted.
It is the same even if it is transmitted using a base64 image (only 120KB) using a transformer and transmitted to the predictor through transformation inside kubernetes.
What did you expect to happen:
The model  will be referenced.
Anything else you would like to add:
There seems to be no special log in the predictor.
=============================
== Triton Inference Server ==
=============================

NVIDIA Release 20.10 (build <unknown>)

Copyright (c) 2018-2020, NVIDIA CORPORATION.  All rights reserved.

Various files include modifications (c) NVIDIA CORPORATION.  All rights reserved.
NVIDIA modifications are covered by the license terms that apply to the underlying
project or file.

NOTE: The SHMEM allocation limit is set to the default of 64MB.  This may be
   insufficient for the inference server.  NVIDIA recommends the use of the following flags:
   nvidia-docker run --shm-size=1g --ulimit memlock=-1 --ulimit stack=67108864 ...

I1213 05:21:21.489391 1 metrics.cc:184] found 1 GPUs supporting NVML metrics
I1213 05:21:21.496163 1 metrics.cc:193]   GPU 0: NVIDIA GeForce RTX 3090
I1213 05:21:21.759404 1 pinned_memory_manager.cc:195] Pinned memory pool is created at '0x7fb504000000' with size 268435456
I1213 05:21:21.760695 1 cuda_memory_manager.cc:98] CUDA memory pool is created on device 0 with size 67108864
I1213 05:21:21.764328 1 model_repository_manager.cc:714] loading: insightface:1
I1213 05:21:21.812585 1 onnxruntime.cc:1712] TRITONBACKEND_Initialize: onnxruntime
I1213 05:21:21.812622 1 onnxruntime.cc:1725] Triton TRITONBACKEND API version: 1.0
I1213 05:21:21.812627 1 onnxruntime.cc:1731] 'onnxruntime' TRITONBACKEND API version: 1.0
I1213 05:21:21.821293 1 onnxruntime.cc:1773] TRITONBACKEND_ModelInitialize: insightface (version 1)
WARNING: Since openmp is enabled in this build, this API cannot be used to configure intra op num threads. Please use the openmp environment variables to control the number of threads.
I1213 05:21:21.823365 1 onnxruntime.cc:1814] TRITONBACKEND_ModelInstanceInitialize: insightface_0 (device 0)
I1213 05:21:25.939947 1 model_repository_manager.cc:896] successfully loaded 'insightface' version 1
I1213 05:21:25.940210 1 server.cc:141]
+-------------+-----------------------------------------------------------------+------+
| Backend     | Config                                                          | Path |
+-------------+-----------------------------------------------------------------+------+
| onnxruntime | /opt/tritonserver/backends/onnxruntime/libtriton_onnxruntime.so | {}   |
+-------------+-----------------------------------------------------------------+------+

I1213 05:21:25.940282 1 server.cc:184]
+-------------+---------+--------+
| Model       | Version | Status |
+-------------+---------+--------+
| insightface | 1       | READY  |
+-------------+---------+--------+

I1213 05:21:25.940436 1 tritonserver.cc:1621]
+----------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------+
| Option                           | Value                                                                                |
+----------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------+
| server_id                        | triton                                                                                |
| server_version                   | 2.4.0                                                                                |
| server_extensions                | classification sequence model_repository schedule_policy model_configuration system_shared_memory cuda_shared_memory binary_tensor_data statistics |
| model_repository_path[0]         | /mnt/models                                                                                |
| model_control_mode               | MODE_NONE                                                                                |
| strict_model_config              | 1                                                                                |
| pinned_memory_pool_byte_size     | 268435456                                                                                |
| cuda_memory_pool_byte_size{0}    | 67108864                                                                                |
| min_supported_compute_capability | 6.0                                                                                |
| strict_readiness                 | 1                                                                                |
| exit_timeout                     | 30                                                                                |
+----------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------+

I1213 05:21:25.944552 1 grpc_server.cc:3914] Started GRPCInferenceService at <IP_ADDRESS>:9000
I1213 05:21:25.944811 1 http_server.cc:2717] Started HTTPService at <IP_ADDRESS>:8080
I1213 05:21:25.987316 1 http_server.cc:2736] Started Metrics Service at <IP_ADDRESS>:8002

I thought it might be a problem with shm_size, so I tested it on a server without shm_size, but it works well. In my opinion, it may be a problem in the front interface that receives data.
Environment:

Istio Version: 1.10
Knative Version: 0.22.3
KFServing Version: 0.7.0
Kubeflow version: X
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm] X
Minikube version: X
Kubernetes version: (use kubectl version): 1.18.5
OS (e.g. from /etc/os-release): ubuntu 20.04.2

Oh, but when transformer is used, the same error occurs whether the corresponding header is not attached when the transformer passes it to the predictor. I think we still need to open an issue on this part.

I think it would be better to add a content-type header to that line.
https://github.com/kserve/kserve/blob/07e4d5d3ea54164026bbf7d87de0b06c6d34c014/python/kserve/kserve/kfmodel.py#L164-L169
Or please comment.


Release fluo-parent-1-incubating (parent POM)

Before the 1.0.0-incubating Fluo release, we'll need to release the parent POM. This is a tracker issue to take care of that.

@ctubbsii  can this be closed?

Yes. Done.


does it support react

does it support react or preact

Yes, there have been several implementations on react.

have some example?

We don't have any examples, but you can intialize PTR when React is mounted. If you do, please consider submitting a PR!


Machine mac-30 lost connection from the external iPhone device.

Machine mac-30 lost connection from the external iPhone device.

The external connection has been restored by rebooting machine mac-30.


Reproducing the IC downstream task

Hi,
I am trying to reproduce the IC accuracy results you have reported in the SUPERB paper using Wave2Vec2, Hubert, etc. It is not clear for me what architecture/hyperparameters you have used for the IC downstream task. Can you please explain different hyperparameter setup? Specifically:
1 - Did you use only the hidden features from the last layer of the upstream task (e.g., Wave2Vec2)?
2 - Did you fine tune the upstream with IC model (downstream), or you just freeze the upstream model?
3 - What is the architecture you have used for IC task? layers, activations, etc.?
4 - The optimization algorithm and its hyperparameters (learning rate, schedulers, etc.)?
It seems the model.py in "downstream)/fluent commands/" folder is not matched with the IC experiment. I think you have used the code in "downstream)/model.py", instead?
Thanks

Hi,

no we use all layers and do the weighted-sum over them like ELMo
no we freeze the upstream model
a simple 2-layer linear model with mean pooling, which is the UtteranceLevel model in the downstream/model.py
http://<IP_ADDRESS>:8000/ic/

The model.py in downstream/fluent_commands is not used in SUPERB

Thanks for your answer. Just one more question: the results you have reported in Table 2 of the SUPERB paper are the test accuracy, or the val accuracy?

Hi,
test accuracy

Hi,
test accuracy

Thanks


Video stream not seen remotely

I'm sorry to bother again. I've trying to set up a video streaming as part of an app using camera.py form the examples. This works perfectly on the browser from the computer I'm running the code but when accessing to the webpage form another computer no preview is seen.
Thank you very much in advance.

Hello, you can ask me all the questions you want. I'm pleased to help. Can you please show me the code of your script?

Hello @WC802,
No problem for the code, it looks fine. Messages in spanish are also ok ;-)
The problem in your script is that you are using the wrong method to grab video. The method you used allows to get video frames by using the browser, specifically the browser of the client machine.
You instead need to grab video from the "server", the raspberrypi, to which other machines will connect.
The correct method is to use opencv. Remi has an opencv_video widget, you can find it in the editor.
First install python opencv, than you can see the widgets in the remi gui editor.
Here is a little example for you that allows to grab video with opencv.
opencv_video.zip
Please let me know if you need further information.
Have a nice day

I forgot to reply. It worked just fine. Thank you very much!


StreamBridge resolveDestination considers binderName in cache hit

When binderName is specified use it as part of the cache lookup for the MessageChannel.
Reference app https://github.com/waroir20/kafka-serializer-publisher.git (main) has example where multiple binders are used potentially for the same topic.
Using debugger to add to the cache and lookup from the cache allows application to publish as expected.

@waroir20 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@sobychacko or @olegz the update made in spring cloud stream 3.2.4 (spring dependencies 2021.0.3) didn't quite resolve the issues of using the same topic with different binders.
I created a PR which in theory should resolve the issue, https://github.com/spring-cloud/spring-cloud-stream/issues/2379
This is my first time contributing to this open source project, and not sure on the PR protocols.

Thank you for pul request. In the future please run a full build as it was failing die to minor check-style violations. I have fixed them with subsequent commit.

Thank you for pul request. In the future please run a full build as it was failing die to minor check-style violations. I have fixed them with subsequent commit.


Unable to install & run Z-wave on HA with MacOS

Hi, I am very new on this subject.  Tried to install HA (0.108.9) on MacOS (10.13.6) but is unable to get Zwave to work.  From the Configuration check I can see:
"Configuration not valid".  Component error: zwave - Requirements for zwave not found: ['homeassistant-pyozw==0.1.10'].
Trying to install homeassistan-pyozw, but fails (se log below).
Please advice.
Regards
Jan
homeassistant$ pip install homeassistant-pyozw
Collecting homeassistant-pyozw
Using cached homeassistant_pyozw-0.1.10.zip (147 kB)
Requirement already satisfied: six in ./homeassistant/lib/python3.7/site-packages (from homeassistant-pyozw) (1.14.0)
Requirement already satisfied: PyDispatcher>=2.0.5 in ./homeassistant/lib/python3.7/site-packages (from homeassistant-pyozw) (2.0.5)
Installing collected packages: homeassistant-pyozw
Running setup.py install for homeassistant-pyozw ... error
ERROR: Command errored out with exit status 1:
command: /Users/homeassistant/homeassistant/homeassistant/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/setup.py'"'"'; file='"'"'/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-record-5_d3besl/install-record.txt --single-version-externally-managed --compile --install-headers /Users/homeassistant/homeassistant/homeassistant/include/site/python3.7/homeassistant-pyozw
cwd: /private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/
Complete output (18 lines):
ImportError in : from wheel.bdist_wheel import bdist_wheel as _bdist_wheel
sysargv ['/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/setup.py', 'install', '--record', '/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-record-5_d3besl/install-record.txt', '--single-version-externally-managed', '--compile', '--install-headers', '/Users/homeassistant/homeassistant/homeassistant/include/site/python3.7/homeassistant-pyozw']
Found SETUP_DIR : /private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw
NameError in : class bdist_wheel(_bdist_wheel) - Use bdist_egg instead
<pyozw_setup.EmbedTemplate object at 0x1056bf110>
{'name': 'libopenzwave', 'sources': ['openzwave-embed/open-zwave-hass/python-openzwave/src-lib/libopenzwave/libopenzwave.cpp'], 'include_dirs': ['src-lib/libopenzwave/', 'openzwave-embed/open-zwave-hass/cpp/src', 'openzwave-embed/open-zwave-hass/cpp/src/value_classes', 'openzwave-embed/open-zwave-hass/cpp/src/platform', 'openzwave-embed/open-zwave-hass/cpp/build/mac'], 'define_macros': [('PY_LIB_VERSION', '0.1.10'), ('PY_SSIZE_T_CLEAN', 1), ('PY_LIB_FLAVOR', 'embed'), ('PY_LIB_BACKEND', 'cpp')], 'libraries': [], 'extra_objects': ['openzwave-embed/open-zwave-hass/libopenzwave.a'], 'extra_compile_args': ['-stdlib=libc++', '-mmacosx-version-min=10.7'], 'extra_link_args': ['-framework', 'CoreFoundation', '-framework', 'IOKit'], 'language': 'c++'}
['six', 'PyDispatcher>=2.0.5']
running install
flavor embed
running build_openzwave
Found g++ : /usr/bin/g++
Found gcc : /usr/bin/gcc
Found make : /usr/bin/make
Found gmake : None
Found cython : None
Found pkg-config : None
fetching https://raw.githubusercontent.com/home-assistant/python-openzwave/hass/archives/open-zwave-hass-0.1.10.zip into openzwave-embed/open-zwave.zip for version 0.1.10
error: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1076)>
----------------------------------------
ERROR: Command errored out with exit status 1: /Users/homeassistant/homeassistant/homeassistant/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/setup.py'"'"'; file='"'"'/private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-install-v0u16_w4/homeassistant-pyozw/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /private/var/folders/bz/fqrz87nj7t72p396hzxy87fh0000gn/T/pip-record-5_d3besl/install-record.txt --single-version-externally-managed --compile --install-headers /Users/homeassistant/homeassistant/homeassistant/include/site/python3.7/homeassistant-pyozw Check the logs for full command output.

Problem solved.  Found the solution on Stackoverflow.com:
To solve this:
All you need to do is to install Python certificates! A common issue on macOS.
Open these files:
Install Certificates.command
Update Shell Profile.command
Simply Run these two scripts and you wont have this issue any more.
Hope this helps!


Remove flag:proxykubeconfig from test_items in node.yaml for 4.1.3/4.1.4

This PR fixes this remaining for RKE2 issue. Covers all cis-1.24 and cis-1.7 profiles for now.
See details about the root cause and fix details, upstream.

Changes looks good. But did you test the changes? All the scans are passing right?
Otherwise LGTM.

@rayandas I validated the result (from kubect and from rancher), it doesn't generate a PASS anymore.

For 1.24 profile, we get a FAIL since the checks are marked as automated

I0830 17:50:47.459380  261947 check.go:110] -----   Running check 4.1.3   -----
I0830 17:50:47.465176  261947 check.go:309] Command: "/bin/sh -c 'if test -e /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig; then stat -c permissions=%a /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig; fi'"
I0830 17:50:47.465213  261947 check.go:310] Output:
 "permissions=644\n"
I0830 17:50:47.465227  261947 check.go:231] Running 1 test_items
I0830 17:50:47.465275  261947 test.go:153] In flagTestItem.findValue 644
I0830 17:50:47.465289  261947 test.go:247] Flag 'permissions' exists
I0830 17:50:47.465297  261947 check.go:255] Used auditCommand
I0830 17:50:47.465308  261947 check.go:287] Returning from execute on tests: finalOutput &check.testOutput{testResult:false, flagFound:false, actualResult:"permissions=644", ExpectedResult:"permissions has permissions 644, expected 600 or more restrictive"}
I0830 17:50:47.465475  261947 check.go:184] Command: "" TestResult: false State: "FAIL"
[INFO] 4 Worker Node Security Configuration
[INFO] 4.1 Worker Node Configuration Files
[FAIL] 4.1.3 If proxy kubeconfig file exists ensure permissions are set to 600 or more restrictive (Automated)

== Remediations node ==
4.1.3 Run the below command (based on the file location on your system) on the each worker node.
For example,
chmod 600 /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig

For 1.27 profile, we get a WARN since the checks are marked as manual

I0830 17:56:50.931986  266653 check.go:110] -----   Running check 4.1.3   -----
I0830 17:56:50.939188  266653 check.go:309] Command: "/bin/sh -c 'if test -e /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig; then stat -c permissions=%a /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig; fi'"
I0830 17:56:50.939223  266653 check.go:310] Output:
 "permissions=644\n"
I0830 17:56:50.939284  266653 check.go:231] Running 1 test_items
I0830 17:56:50.939351  266653 test.go:153] In flagTestItem.findValue 644
I0830 17:56:50.939360  266653 test.go:247] Flag 'permissions' exists
I0830 17:56:50.939364  266653 check.go:255] Used auditCommand
I0830 17:56:50.939378  266653 check.go:287] Returning from execute on tests: finalOutput &check.testOutput{testResult:false, flagFound:false, actualResult:"permissions=644", ExpectedResult:"permissions has permissions 644, expected 600 or more restrictive"}
I0830 17:56:50.939394  266653 check.go:184] Command: "" TestResult: false State: "WARN"
[INFO] 4 Worker Node Security Configuration
[INFO] 4.1 Worker Node Configuration Files
[WARN] 4.1.3 If proxy kubeconfig file exists ensure permissions are set to 600 or more restrictive (Manual)

== Remediations node ==
4.1.3 Run the below command (based on the file location on your system) on the each worker node.
For example,
chmod 600 /var/lib/rancher/rke2/agent/kubeproxy.kubeconfig

From Rancher:
Before fix:

After fix:


cmd/warmup: add instruction if process exits when warming up on MacOS

fix #1739
Reference: https://code.google.com/archive/p/macfuse/wikis/OPTIONS.wiki

Codecov Report

Merging #1747 (2ee0ded) into main (2c88893) will decrease coverage by 0.01%.
The diff coverage is 31.57%.

@@            Coverage Diff             @@
##             main    #1747      +/-   ##
==========================================
- Coverage   61.68%   61.67%   -0.02%
==========================================
  Files         135      135
  Lines       22908    22927      +19
==========================================
+ Hits        14131    14140       +9
- Misses       7181     7195      +14
+ Partials     1596     1592       -4

Impacted Files
Coverage Δ

cmd/format.go
60.89% <0.00%> (-1.61%)
:arrow_down:

cmd/warmup.go
62.87% <41.17%> (-1.35%)
:arrow_down:

pkg/object/encrypt.go
68.29% <45.45%> (ø)

pkg/chunk/utils_linux.go
0.00% <0.00%> (-75.00%)
:arrow_down:

pkg/chunk/utils_unix.go
53.33% <0.00%> (-20.01%)
:arrow_down:

pkg/vfs/writer.go
78.64% <0.00%> (-1.02%)
:arrow_down:

pkg/chunk/disk_cache.go
78.88% <0.00%> (-0.98%)
:arrow_down:

pkg/meta/sql.go
67.13% <0.00%> (-0.11%)
:arrow_down:

pkg/object/b2.go
2.12% <0.00%> (-0.05%)
:arrow_down:

pkg/meta/base.go
71.69% <0.00%> (+0.50%)
:arrow_up:

... and 3 more

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 a6f3cc4...2ee0ded. Read the comment docs.


Getting "Class not found" error while setup

TwilioPlugin.Device.prototype.setup = function(token) {
// Take a token and instantiate a new device object
var error = function(error) {
if(delegate['ondeviceerror']) delegate'ondeviceerror'
if(delegate['onconnectionerror']) delegate'onconnectionerror'
}
    var success = function(callback) {
        var argument = callback['arguments'] || new TwilioPlugin.Connection();
        if (delegate[callback['callback']]) delegate[callback['callback']](argument);
    }

    Cordova.exec(success, error, "TCPlugin", "deviceSetup", [token]);
}

Getting "Class not found" as error on error call back.
Any solution for that.

Hi, from Class Not Found, I'm going to guess this is on Android - here is a similar issue in another Cordova project - are you using any other plugins?
https://github.com/dealrinc/cordova-gmv-barcode-scanner/issues/5

Hi, this issue came in android.
I used<EMAIL_ADDRESS>as mentioned from the above link but it didn't work for me.


[UI] StringIndexOutOfBoundsException in ViewContextUpdateSecurity:splitParamName

When checking for isAllowedField in ViewContextUpdateSecurity, if the paramName target a simple field that is not allowed, we try to split as if it is an object and get a StringIndexOutOfBoundsException in the splitParamName method.
Example :

allowedFields = ["vContext[aaa]"]
paramName = "vContext[bbb]"

In some cases I have POSTs with vContext['aaa'] (with quotes) in my custom code. Should it be an accepted format ?
In the meanwhile, I delete thoses quotes.

Fixed by #105f00
-> v3.4.0


feat: make https configurable for dev

This allows to set SSL cert and key for https. closes #81
{
  server: {
    https: {
      cert: "cert or path",
      key: "key or path"
    }
  }
}

Please review.

Should serveConfig be based to router-modes instead of only https options?
Types

It would be cool to also support generating a cert by just setting https to true at some point. listhen has a this function resolveCertificate that does all the nice things and returns { cert, key } but is unfortunately internal.

Yeah I would want just https to work. Lets work with listhen to expose that, and in the meanwhile copy over that code and do it ourselves here?
Does listehn support https: true ?

since https is a server concern it should be handled centrally. Why cant we just set the https field in the createViteHandler in the dev-server.js. We don't need to inform each build handler and dev handler. We can centralize this.

@edivados will it still be possible to specify cert/key manually? :)

@nksaraf should it supported http: true? I'm asking because of it got removed from vite 5

@indeyets

will it still be possible to specify cert/key manually? :)

Yes and if we go with using listhen's implementation it should support what listhen supports. See listhen
@edygar

should it supported http: true? I'm asking because of it got removed from vite 5

Should be fine because true is not passed to vite. It's set to false or { cet, key }. I will check with vite 5 just in case.

@nksaraf

Yeah I would want just https to work. Lets work with listhen to expose that, and in the meanwhile copy over that code and do it ourselves here?
Does listehn support https: true ?

Yes, see here. resolveCertificate isn't typed for it but it works. It merges true with the deafult config using defu. Will double check just to be sure.

since https is a server concern it should be handled centrally. Why cant we just set the https field in the createViteHandler in the dev-server.js. We don't need to inform each build handler and dev handler. We can centralize this.

Should be good now. Copied the implementation over from listhen and converted the ts file to js. Not sure what to do with the .d.ts file. Leave it? Also should we add a note at the top of the file saying it's from listhen?

Should be good now. Copied the implementation over from listhen and converted the ts file to js. Not sure what to do with the .d.ts file. Leave it? Also should we add a note at the top of the file saying it's from listhen?

Yup and add their license. We can also maybe just export it from my fork of listhen, nksaraf/listhen which vinxi currently uses.
And yeah we should support all the Https options listhen does and then we can pass to vite whatever it expects based on whatever version it is. I consider that internal implementation details

We can also maybe just export it from my fork of listhen, nksaraf/listhen which vinxi currently uses.

I like that idea more then copying the file over. Only need to make resolveCertificate available. Do you want me to make a PR on vinxi/listhen so we can just import from there?

And yeah we should support all the Https options listhen does and then we can pass to vite whatever it expects based on whatever version it is. I consider that internal implementation details

Good thing is we shouldn't have to do anything but take the return value from resolveCertificate and pass it to vite and listhen. If it turns out that when using a passphrase it returns an encrypted private key we maybe have to decrypt it first for vite but I have not looked into it. Basic usage worked.

Do you want me to make a PR on @vinxi/listhen so we can just import from there?

Yup!! that would be great. that will make this simpler

Done. I did a PR against ws because it has the websocket stuff which is available here in dev right?

mergeed that and release at<EMAIL_ADDRESS>#127 bumps up our dependency. This PR can use than now for providing https functionality to vinxi.

Good to merge.

[!NOTE]
Did a last min change so that resolveCertificate does not receive true but an empty object (for the type errors). We can make it more pretty later no need to block this any longer.


How to use with AdonisJs?

Closing this question due to inactivity. A good place to ask general usage questions is the Apollo Slack #uploads channel 🙂

@mastito03 Did you solve it?

right now i use Custom middleware and load it from router

@mastito03 Could you give me an example of the Middleware and router with this?


IFIX-159

User input Previous meter reading Generating Bill Bad state error issue fixed

Fix : Blank screen Appearing after Demand Is generated


Feature/frontend/public/games

To address and close #13.

With the unit tests I pushed to this branch, we now have 45% coverage on the backend...getting better

Nice! haha that's cool! :+1: