Persona 4 Golden [PCSE00120]

Game summary

Game name: Persona 4 Golden
Game serial: PCSE00120

Vita3K summary

Version: v0.1
Build number: 685-2eee7b1
Commit hash: https://github.com/vita3k/vita3k/commit/2eee7b1

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Loading data from data.cpk while trying to render something(I assume one of the opening splash screens). Soft locking after trying to render some more stuff. It doesn't seem to get to even trying to load the opening(as was hyped on the discord) but then again, that was on a debug build/race condition. No regression comparing as far back as https://github.com/vita3k/vita3k/commit/a741c3af
|E| [ret_error_impl]:  sceAppUtilSaveDataSlotSetParam returned SCE_APPUTIL_ERROR_SAVEDATA_SLOT_NOT_FOUND (0x80100641)
Screenshots
NA
Log
vita3k_p4.log
Recommended labels

Vita3K summary

Version: v0.1
Build number: 805
Commit hash: https://github.com/vita3k/vita3k/commit/4279f967

Issue
Error shader loop I guess... At least it doesn't crash.
Log
Persona4Golden_US.log

Vita3K summary

Version: v0.1
Build number: 984
Commit hash: fb0798e of pull/456 (I don't want to append every issue to the PR)

Persona 4 GOLDEN - [PCSE00120].log
Issue
Crash on shader gen

Vita3K summary

Version: v0.1
Build number: 1255
Commit hash: https://github.com/vita3k/vita3k/commit/daff95f

Issue
Crashes when playing the intro video
Screenshots

Log
vita3k.log

all i see is a blue screen

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

After starting in offline mode and then writing data to save slot 0 it fails on NGS and sceSysmoduleUnloadModule as well as failing io on AppUtil save functions leading to a crash

all i see is a blue screen

same

3d rendering works seemingly well, at least ingame -
Windows 11 22000.434
GPU: AMD RX 580 8GB
CPU: AMD Ryzen 5 2600
RAM: 16 GB

P4G is surprisingly stable right now, and rather "playable", though with severe slowdown and graphical/audio bugs

Samegawa riverbank cloudy day bug

Native Vita:

Vita3k:

Camera is consistently in the wall when reading books in your room, but also happens in some actually story scenes downstairs in the Dojima Residence.
Playing on Android v11, using a Retroid Pocket 4 Pro.

Native Vita:
Vita3k:
Camera is consistently in the wall when reading books in your room, but also happens in some actually story scenes downstairs in the Dojima Residence.
Playing on Android v11, using a Retroid Pocket 4 Pro.

Seconding this the entirety of the camping trip just doesn't render for some reason hoping this gets fixed

Native Vita:
Vita3k:
Camera is consistently in the wall when reading books in your room, but also happens in some actually story scenes downstairs in the Dojima Residence.
Playing on Android v11, using a Retroid Pocket 4 Pro.

Seconding this the entirety of the camping trip just doesn't render for some reason hoping this gets fixed

This also happens in PC when using Vulkan API I would use OpenGL but there is like a 10fps drop when compared to Vulkan

is the game playable? just started my second playthrough using this emulator

is the game playable? just started my second playthrough using this emulator

Nope unless you have very low standards. To anyone reading this there are far better/easier ways to play.

Buy it on Windows (Steam) and use PlayonMac or Porting Kit (use silicon steambuild for m1+)

This is best option (best graphics) and check out mods (really recommend mod that makes extra conversations at night not prgress time = great for a second playthrough

Use Ryujinx

Get Switch emulator and setup from there (I honestly own it on switch but did not use my own copy cus I didnt want to jailbreak switch / and im lazy = buy it on switch but just find all the stuff you need online if you have some morals but are lazy like me)
This one has great graphics and runs well but lacks mods

I tried playing it, and it crashed multiple times during Yukiko's palace, so I just gave up and now I'm doing a NG+ on my other machine. I'm gonna try playing it on Ryujinx though, thanks for letting me know.

Buy it on Windows (Steam) and use PlayonMac or Porting Kit (use silicon steambuild for m1+)

What is Silicon Steambuild? Can't really figure out what it is on Google.

@millxisyes It works fine for me I'm even using a igpu. The only  problem I have is reading and the fog on the river plain when using Vulkan it works perfectly for OpenGL but it's at least a 5fps drop which would not mean anything if you have a discrete gpu

@Avenger3283 I see, I'm running the game on a Mac so it might just be a Mac related issue for me.


feat(dvcx): Add pythonUserBase path to DVCx worker config

This update introduces a new parameter, pythonUserBase, to the DVCx worker configuration. This allows customization of the base path for user site-packages. By default the location is changed from ~/.local to /tmp/.pip/local, where ephemeralStorage is beeing used.
Pip docs: https://pip.pypa.io/en/stable/user_guide/#user-installs
Closes https://github.com/iterative/itops/issues/3636
This PR also reverts https://github.com/iterative/helm-charts/pull/305, which caused a bug same as https://github.com/bitnami/charts/issues/20504
the whole bitami/charts repository introduced breaking change, which is causing the issue. Probably there is a need for upgrading redis & postgres in one chart

I had to revert https://github.com/iterative/helm-charts/pull/305, which caused a bug similar to https://github.com/bitnami/charts/issues/20504
the whole bitami/charts repository introduced breaking changes, causing the issue. Probably, there is a need for upgrading redis & postgres in one PR.


Project root directory not found. Indicators: ('.project-root', 'setup.cfg', 'setup.py', '.git', 'pyproject.toml')

Hello, I am running pyrootutils.setup_ root(".", pythonpath=True).The error reported during root code is as follows:Project root directory not found. Indicators: ('.project-root', 'setup.cfg', 'setup.py', '.git', 'pyproject.toml')
How to solve it?

Hello, thanks for your interest in the project.

what command are you executing, exactly ?
please provide the full traceback
have you made any modification (even minor) to the project ?
what is your pyrootutils version ?

Hello, thanks for your interest in the project.

what command are you executing, exactly ?
please provide the full traceback
have you made any modification (even minor) to the project ?
what is your pyrootutils version ?

I apologize for not articulating my question clearly.I'm trying to visualize the results, but when running demo_dales.ipynb, the above error is reported when initializing the cfg, and my version of pyrootutils is 1.0.4.This seems like a path problem but I don't know how to solve it, I would be honored if you could offer some help.

Your pyrootutils version seems fine.

please provide the full traceback
have you made any modification (even minor) to the project ?
do you have a .git in your project folder root ?
are you working in a Docker environment ?

Your pyrootutils version seems fine.

please provide the full traceback
have you made any modification (even minor) to the project ?
do you have a .git in your project folder root ?
are you working in a Docker environment ?

I changed the demo_dals.ipynb file to a .py file and put it in pycharm to run, and I can be sure that my virtual environment is correct since I've done the training and evaluation. When I want to try to visualize the results, the compiler prompts:
project root directory not found. indicators: ('.project-root', 'setup.cfg', 'setup.py', '.git', 'pyproject.toml').
I don't know how to fix this bug,I don't have .git in my project because I'm not quite sure what it's for.

If you installed the project using git clone https://github.com/drprojects/superpoint_transformer.git, you should have a .git in the root of the project. This hidden folder is used by git for code versioning. I strongly recommend using git in your coding projects.
The pyrootutils library uses some specific files to identify the root path of the project. Please do not delete files from the provided source code. I suggest your restore all files from the initial projects before testing again.
Besides, managing paths for hydra, omegaconf, and pyrootutils may require some knowledge of those libraries. I cannot guarantee that your ipynb -> .py conversion will work. In particular, you will need to be careful in which directory you place your .py file. You may want to consider adapting your paths for local imports to work properly. See the commented line in the notebooks:
# file_path = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))  # for .py script
file_path = os.path.dirname(os.path.abspath(''))  # for .ipynb notebook

Without reply from you, I am assuming the issue is solved.


Added default MT functions to script extraction

That way the whole .sublime-completions can be generated

Script is quite on point now


typos runnig and runnign in server crate

Files server/default.env and server/src/process.rs each contain a typo beginning with "runnig".

Thanks, fixed in the commits branch. Will be merged soon.


Allow to disable compression in Apache HttpClient

Fixes gh-3122

Codecov Report

Merging #3134 into master will decrease coverage by 0.82%.
The diff coverage is 75%.

@@             Coverage Diff              @@
##             master    #3134      +/-   ##
============================================
- Coverage      65.3%   64.47%   -0.83%
+ Complexity     1438     1432       -6
============================================
  Files           182      182
  Lines          6831     6835       +4
  Branches        835      835
============================================
- Hits           4461     4407      -54
- Misses         2061     2122      +61
+ Partials        309      306       -3

Impacted Files
Coverage Δ
Complexity Δ

...ix/ribbon/support/AbstractLoadBalancingClient.java
79.54% <ø> (ø)
11 <0> (ø)
:arrow_down:

...x/ribbon/apache/RibbonLoadBalancingHttpClient.java
93.33% <100%> (+0.22%)
7 <0> (ø)
:arrow_down:

...apache/RetryableRibbonLoadBalancingHttpClient.java
81.66% <100%> (+0.31%)
12 <0> (ø)
:arrow_down:

...amework/cloud/netflix/ribbon/RibbonProperties.java
50% <50%> (ø)
20 <1> (+1)
:arrow_up:

...hystrix/stream/HystrixStreamAutoConfiguration.java
0% <0%> (-72.23%)
0% <0%> (-6%)

...loud/netflix/hystrix/stream/HystrixStreamTask.java
74.54% <0%> (-21.37%)
9% <0%> (-4%)

...etflix/hystrix/stream/HystrixStreamProperties.java
39.39% <0%> (-3.04%)
5% <0%> (-1%)

.../netflix/eureka/server/EurekaServerConfigBean.java
22.01% <0%> (+0.23%)
31% <0%> (+1%)
:arrow_up:

...x/security/SecurityContextConcurrencyStrategy.java
77.77% <0%> (+16.66%)
8% <0%> (+3%)
:arrow_up:

Continue to review full report at Codecov.

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

I will prepare test.
Regarding documentation. I've implemented CommonClientConfigKey.GZipPayload'  from ribbon-core` so we should find documentation there... but I can't see description of this config key.

Unit tests added.

Should we also make a similar change in RibbonLoadBalancingHttpClient in the case that Spring Retry is not on the classpath?

RibbonLoadBalancingHttpClient is already updated.

Squashed and merged with polish via 0f7c64325e009941b49d5b0136ae6b72efe8a184

@spencergibb Thank you.


Update the repo_setup role readme

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/approve

Thank you @raukadah, this is a big improvement, could you please mention this role installs repo_setup from cifmw_repo_setup_src and expand on the relationship with that repo? For example why is there also a repo_setup role [1]
[1] https://github.com/openstack-k8s-operators/repo-setup/tree/main/roles/repo_setup

Hey @lewisdenny , thank you for reviewing the pr. repo-setup/roles/repo_setup is an old import from tripleo. It is never used. We rewrote the whole tooling based on our needs. https://github.com/openstack-k8s-operators/repo-setup/pull/25 will drop the old role.
I will update the role how repo-setup uiltity is getting installed.


Version catalog should allow users to configure libraries with plugin suffix

There are cases where a library is a plugin but not used in the context of the plugins block. The version catalog does not allow libraries with plugin suffix, so build authors need to come with ugly workarounds.

Fixed in https://github.com/gradle/gradle/pull/18250.


Faster route matching algorithms.

The current route matching (SimpleRouteMatcher) is very slow... it runs in O(n) time with respect to the number of routes installed.
You should move to using a radix-tree based route matching system. These run in O(n) time with respect to the length of the request path (regardless of the number of routes installed).
See:
https://github.com/augustl/path-travel-agent
https://github.com/mschurr/lightning/blob/master/src/main/java/lightning/routing/RouteMapper.java

Is this an actual problem though? The number of routes is usually in the tens to hundreds, what kind of performance improvement would this yield for a typical application?

Had a chat with the creator of Path Travel Agent, seems we're talking improvements on the micro-second scale here (given worst case scenario routes in a normal sized application). I won't close the issue, but I'll mark it as low priority.

Realistically we won't be able to prioritize this, so I'm closing it for now.


Forward SSH agent over RDP using virtual channel

I've written plugins for xrdp (#867) and FreeRDP (#4122) for a virtual channel to forward ssh-agent data.  This allows programs - usually ssh itself - in the remote desktop session to authenticate using an ssh-agent running locally on the RDP client machine.
Usage
Run FreeRDP with the ssh-agent plugin enabled:
   xfreerdp /ssh-agent ...

In the remote desktop session run xrdp-ssh-agent and evaluate the output in the shell as for ssh-agent to set the required environment variables (specifically $SSH_AUTH_SOCK):
   eval "$(xrdp-ssh-agent -s)"

This is the same as for the normal ssh-agent.  You would typically do this in your Xsession or /etc/xrdp/startwm.sh.
Limitations

Error checking and handling could be improved.

This stays running when the xrdp session closes.  This should be fixed using a command line argument as for the real ssh-agent.

This is only tested on Linux and will only work on systems where clients talk to the ssh-agent via Unix domain sockets.  It won't currently work on Windows but it could be ported.

You need to have an ssh-agent running or a tunnel to an ssh agent locally on the client machine because the plugin doesn't perform ssh-agent services (key store and signing).  This is not really a limitation because ssh has the same behaviour: if you have no ssh-agent running locally then "ssh -A" doesn't provide an agent to the remote session.

Security issues
All of the security issues I can think of also apply to the normal ssh-agent.
The main concern is that the Unix domain socket created on the server must be readable only by that user, to prevent other users signing arbitrary data using the original user's private keys.  So the socket has to be created with appropriate Unix permissions (i.e. group and other can't read/write/execute).
How does it work?
It simply forwards the ssh-agent protocol between the Unix domain socket on the server and the ssh-agent's $SSH_AUTH_SOCK Unix domain socket on the client.  The plugin is just a proxy and doesn't need to know anything about the protocol itself.
(It should be possible to write another plugin to forward network ports (and Unix domain sockets) in general à la SSH in the same way.)
Can this be ported to Windows?
Yes, both the client and server ends can both be ported to Windows.  Windows doesn't support Unix domain sockets so no doubt the mechanism will be different - PuTTY's Pageant uses shared memory - although the protocol will still be the same.
Can I use my SSH key to log in to xrdp?
In principle, yes, by using the sshagent channel during the authentication stage of RDP instead of using a password.  However, xrdp doesn't enable virtual channels until after authentication so there might be some difficulty!  I'm going to look at that next.
Can I use my SSH key to log in to Windows?
Again in principle, yes, by using the sshagent channel during the authentication stage of RDP instead of using a password.  I think Microsoft would need to do this, because "terminal Services does not support remote authentication with arbitrary credential types"!

Hmm, it is very interesting for me. It takes time but I promise I'll look at this.

PR #4122 was merged today into the FreeRDP tree commit ff59cf028.

Good work!


Replace the implementation with the one based on iprover

We shall replace the implementation of the equivalence reasoner according to the project description. This includes the following:

[x] Replace the actual implementation
[x] Replace the implementation documentation

I just closed the associated pull request (#51), because the iprover-based approach had turned out to be unfeasible, and as a result I’m closing also this issue.


Set Deadlock_priority in the index/statistics job

Description of the feature
Please add an execution parameter to set deadlock_priority for all operations performed in the Index and Statistics Maintenance job.  We would prefer to have this become a deadlock victim rather than a user or application session which is running at the same time, so we'd set it to a negative number.
Would this have any impact on other operations in the current execution of the Index and Statistics Maintenance job?  The ideal situation for us would be to have preceding steps in the job succeed and to allow all operations following the deadlock to continue.
Hope this makes sense...  Thanks.

I would find this very usefull as well

I'm currently doing SET DEADLOCK_PRIORITY LOW; before calling the SP, but having that hardcoded in the SP would probably be good.
The index maintenance does proceed past the step that died on a deadlock.
It would be very nice to have an option to say "retry N times if failed due to deadlock" to try the index maintenance again. Perhaps @DeadlockRetries = 2 with default = 1?


Fix Google Play Services NPE when scopes array has null entry

The logic on Android to filter out the 'email' scope will result in an array of scopes that contains a null entry where the 'email' scope was specified.
Google Play Services 11+ throws an null-pointer exception if it receives an array of scopes containing a null. This PR fixes the issue by using an ArrayList to collect non-'email' scopes.

Hi @vonovak — I can't recall the answers to either of your questions (this PR is nearly 4 months old), and I don't have incentive to do the research, since I'm happily using a fork of this library, which includes my patch.
It's my opinion that the best fix to untested, poorly-documented code such as this is whatever is most narrowly tailored to the issue at hand. In this case, it's that a null in the array is leading to an NPE in Play Services.
The original author found it necessary to filter email out, presumably for a good reason, which is why my patch retains that logic. And to make the change at the JS layer puts the iOS implementation at risk of regression, which is unacceptable to me.
So, that said, feel free to close or merge. I won't take it personally.

Seems like email is added by default, thus it's filtered out to avoid duplicate scopes.
Line 304:
GoogleSignInOptions.Builder googleSignInOptionsBuilder = new GoogleSignInOptions.Builder(GoogleSignInOptions.DEFAULT_SIGN_IN).requestScopes(new Scope("email"), _scopes);

Google Sign In library for iOS has email included in the default scope by default but for Android it doesn't. Thus email scope is included there by our library that the API can return the same data regarding of platform.
I feel safe to have this PR landed on master after some testing.

Filtering of the scopes to avoid duplicates is not necessary: Internally the GoogleSignInOptions.Builder class saves the scopes into a HashSet, so the filtering code is not only unnecessary, but as we have seen, also error-prone and makes the code harder to read with no benefit.
I will go ahead and close this PR because the there is no need for such filtering and that is what is done in https://github.com/react-native-community/react-native-google-signin/pull/411 which is a larger PR that will hopefully get merged in the next couple of days, depending on the review feedback. Thanks @jamesreggio for pointing out the issue and for the fix.


"Error: Some file corrupted during upload..." appears

I set the argument "remoteFolder" with a deeper directory like '/xx/xx',
then the message "Error: Some file corrupted during upload..." will appear,
I think it's a bug,  would u like to fix it?
Now I use "gulp-ftp" instead of it,   although "gulp-ftp" is deprecated now.....

@nekolab


what about custom dataset training?

Hi, thanks for your work.
I wonder how can i train and test on my custom dataset. Should i create reference annotation data? Can you share the related script to generate this file?

Hi @hero-y
It'd be great if you could share the above script or explain its significance.

Hi, I don't try our method on the custom dataset, but I think it isn't difficult. First, you should prepare your custom dataset with the coco format. Then, you don't need to prepare the ref_ann_file (refer to voc config). If the ref_ann_file doesn't exist, the code will choose one class-object annotation from the whole annotation as the reference (refer to datasets/one_shot_voc).

datasets/one_sh

Hello, Had you train and test on your custom dataset?

Hi, thanks for your work.
I wonder how can i train and test on my custom dataset. Should i create reference annotation data? Can you share the related script to generate this file?

Hello, Had you train and test on your custom dataset?


MSID: 89002 Version: 1 DOI: 2<IP_ADDRESS>3523

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[ ] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=89002)

or (only one should be ticked. remove other from description.)

[ ] Confirmed with Ted at bioRxiv that MECA archive is available

Send the following email to Ted and wait for his reply.
Hi Ted,

Please can you prepare the preprint MECA for 10.1101/2<IP_ADDRESS>3523

Thanks

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89002/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3523)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89002/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89002)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89002

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89002

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89002/v1/89002-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89002)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Figure titles appearing as section heads


Pip package?

Hi, cool project!
Are there plans to make it a pip installable project by providing it on PyPI?
Currently can not find it there.
Thx!

It's definitely planned to publish this on PyPI. There are still a few intended features missing (especially embedding, possibly with help of the factur-x package), but I will see whether we can publish a preview version next week.


Not able to access the blobs in a container as it is reporting error

Preflight Checklist

[X] I have installed the latest version of Storage Explorer.
[X] I have checked existing resources, including the troubleshooting guide and the release notes.
[X] I have searched for similar issues.

Storage Explorer Version
1.28.0
Regression From
1.26.0
Architecture
i86
Storage Explorer Build Number
20230214.5
Platform
Windows
OS Version
Windows 10
Bug Description
Not able to use the Microsoft Azure Storage Explorer
Steps to Reproduce

Started and logged into the Azure Storage explorer.
Tried to open the container blobs in a container of a storage account. It failed and reported an error mentioned in this ticket.

Actual Experience
This request is not authorized to perform this operation.
This storage account's 'Firewalls & virtual networks' settings may be blocking access to storage services. Try adding your client IP address to the firewall exceptions, or by allowing access from 'all networks' instead of 'selected networks'. To learn more about Azure Storage firewalls and virtual networks, visit http://go.microsoft.com/fwlink/?LinkId=845443.
Error Details:
{
"name": "RestError",
"code": "AuthorizationFailure",
"statusCode": 403,
"request": {
"streamResponseStatusCodes": {},
"url": "https://attbcdevsanatt01.blob.core.windows.net/?comp=list&include=metadata",
"method": "GET",
"headers": {
"_headersMap": {
"x-ms-version": {
"name": "x-ms-version",
"value": "2021-08-06"
},
"accept": {
"name": "Accept",
"value": "application/xml"
},
"user-agent": {
"name": "User-Agent",
"value": "Microsoft Azure Storage Explorer/1.26.0 (win32) azsdk-js-storageblob/12.11.0 (NODE-VERSION v16.14.2; Windows_NT 10.0.19042)"
},
"x-ms-client-request-id": {
"name": "x-ms-client-request-id",
"value": "b4849618-8517-4eee-98b6-f166620c6126"
},
"authorization": {
"name": "authorization",
"value": "Bearer JSON Web Token Redacted"
},
"cookie": {
"name": "Cookie",
"value": ""
}
}
},
"withCredentials": false,
"timeout": 0,
"keepAlive": true,
"decompressResponse": false,
"requestId": "b4849618-8517-4eee-98b6-f166620c6126"
},
"response": {
"headers": {
"_headersMap": {
"content-length": {
"name": "content-length",
"value": "246"
},
"content-type": {
"name": "content-type",
"value": "application/xml"
},
"date": {
"name": "date",
"value": "Wed, 22 Feb 2023 01:55:36 GMT"
},
"server": {
"name": "server",
"value": "Microsoft-HTTPAPI/2.0"
},
"x-ms-client-request-id": {
"name": "x-ms-client-request-id",
"value": "b4849618-8517-4eee-98b6-f166620c6126"
},
"x-ms-error-code": {
"name": "x-ms-error-code",
"value": "AuthorizationFailure"
},
"x-ms-request-id": {
"name": "x-ms-request-id",
"value": "ff80595b-501e-0049-0e60-4676b0000000"
}
}
},
"request": {
"streamResponseStatusCodes": {},
"url": "https://attbcdevsanatt01.blob.core.windows.net/?comp=list&include=metadata",
"method": "GET",
"headers": {
"_headersMap": {
"x-ms-version": {
"name": "x-ms-version",
"value": "2021-08-06"
},
"accept": {
"name": "Accept",
"value": "application/xml"
},
"user-agent": {
"name": "User-Agent",
"value": "Microsoft Azure Storage Explorer/1.26.0 (win32) azsdk-js-storageblob/12.11.0 (NODE-VERSION v16.14.2; Windows_NT 10.0.19042)"
},
"x-ms-client-request-id": {
"name": "x-ms-client-request-id",
"value": "b4849618-8517-4eee-98b6-f166620c6126"
},
"authorization": {
"name": "authorization",
"value": "Bearer JSON Web Token Redacted"
},
"cookie": {
"name": "Cookie",
"value": ""
}
}
},
"withCredentials": false,
"timeout": 0,
"keepAlive": true,
"decompressResponse": false,
"requestId": "b4849618-8517-4eee-98b6-f166620c6126"
},
"status": 403,
"bodyAsText": "AuthorizationFailureThis request is not authorized to perform this operation.\nRequestId:ff80595b-501e-0049-0e60-4676b0000000\nTime:2023-02-22T01:55:36.7539127Z",
"parsedBody": {
"message": "This request is not authorized to perform this operation.\nRequestId:ff80595b-501e-0049-0e60-4676b0000000\nTime:2023-02-22T01:55:36.7539127Z",
"code": "AuthorizationFailure"
},
"parsedHeaders": {
"errorCode": "AuthorizationFailure",
"content-length": "246",
"content-type": "application/xml",
"date": "Wed, 22 Feb 2023 01:55:36 GMT",
"server": "Microsoft-HTTPAPI/2.0",
"x-ms-client-request-id": "b4849618-8517-4eee-98b6-f166620c6126",
"x-ms-request-id": "ff80595b-501e-0049-0e60-4676b0000000"
}
},
"details": {
"errorCode": "AuthorizationFailure",
"content-length": "246",
"content-type": "application/xml",
"date": "Wed, 22 Feb 2023 01:55:36 GMT",
"server": "Microsoft-HTTPAPI/2.0",
"x-ms-client-request-id": "b4849618-8517-4eee-98b6-f166620c6126",
"x-ms-request-id": "ff80595b-501e-0049-0e60-4676b0000000",
"message": "This request is not authorized to perform this operation.\nRequestId:ff80595b-501e-0049-0e60-4676b0000000\nTime:2023-02-22T01:55:36.7539127Z",
"code": "AuthorizationFailure"
}
}
Expected Experience
To show all the blobs in the container
Additional Context
Please get the issue resolved.

Have you checked your storage account's 'Firewalls & virtual networks' settings as suggested by the error details?

Closing due to lack of response. If you are still encountering this issue we recommend opening an Azure support ticket from the Azure Portal. Alternatively you can open a new issue here.


stripeAccount option

First, thanks for making this module possible. Would it be possible to add the option to provide a stripeAccount to support Stripe Connect.
reference: https://stripe.com/docs/connect/creating-a-payments-page
Thanks

Hello, thank you for opening this issue.
I've published the v3.1.0 version including the stripeAccount option.


Error upgrading from 3.0.2 to 4.1.2

I just upgraded from 3.0.2 to 4.1.2 and got the error below. Anything I can do to fix this?
Thanks.
Traceback (most recent call last):
  File "/venv/lib/python3.6/site-packages/flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
  File "/venv/lib/python3.6/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 480, in wrapper
    resp = resource(*args, **kwargs)
  File "/venv/lib/python3.6/site-packages/flask/views.py", line 88, in view
    return self.dispatch_request(*args, **kwargs)
  File "/venv/lib/python3.6/site-packages/flask_restful/__init__.py", line 595, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/venv/lib/python3.6/site-packages/webargs/core.py", line 474, in wrapper
    force_all=force_all,
  File "/venv/lib/python3.6/site-packages/webargs/core.py", line 396, in parse
    self._on_validation_error(error, req, schema)
  File "/venv/lib/python3.6/site-packages/webargs/core.py", line 334, in _on_validation_error
    self.error_callback(error, req, schema)
TypeError: handle_request_parsing_error() takes 2 positional arguments but 3 were given

This is a documented breaking change: https://webargs.readthedocs.io/en/latest/changelog.html#id4.

Custom error handlers receive the marshmallow.Schema instance as the third argument. Update any functions decorated with Parser.error_handler to take a schema argument.


render of blueprint fails when referenced component is not available locally

What happened:
When rendering a blueprint of component A  that accesses a ressource of component reference B the rendering fails when component B is not available locally. Assuming component B is stored in an OCI repository which can be accessed with a repository context specifyied in component A.
This fails:
landscaper-cli blueprints render ./blueprint all -f ./values-render.yaml -c ./component-descriptor-A.yaml
unable to template deploy executions: template: execution:27:42: executing "execution" at <getComponent .cd "name" "B">: error calling getComponent: unable to resolve component B:v0.1.0
This works:
landscaper-cli blueprints render ./blueprint all -f ./values-render.yaml -c ./component-descriptor-A.yaml -a ./component-descriptor-B.yaml
What you expected to happen:
landscaper-cli should be able to resolve referenced components in an OCI registry.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:
Environment:
landscaper-cli v0.13.0

Closing here, as moved to new backlog project.


added workaround for setting JEP value with very special characters

resolves #466
This is a workaround for the issue. In most cases it will just set the value as before. In the odd case, it will write the data to a temp file and let Python load it from there.

Coverage decreased (-0.01%) to 36.729% when pulling 2255b825567d7f4cbfb221a2aedf7df4b742c9ed on elifesciences:fix-label-task-very-special-characters into 9a203381985cc9d9759ae98efd7b5c1cebea7afc on kermitt2:master.


Document Creole syntax for snippets comments

The “new snippet” has a nice explanation for the Creole dialect that is used:
http://flask.pocoo.org/snippets/new/
But there is no such thing when adding a new snippet comment. A link to the new snippet page or the creole website in the comment form should be enough.
This is a follow-up on a mailing-list question: http://flask.pocoo.org/mailinglist/archive/2011/12/31/how-do-i-add-code-markers-to-snippets/

Moved to https://github.com/pocoo/flask-snippets/issues/7


Change test mail subject and body to 'Gitea Test Mail!'

The Gitea test mail subject and body still reference Gogs, as explained in #421.
This PR changes the Gogs references to Gitea.

LGTM

LGTM

Thank you


helm install should create objects in an order

It will be nice to have services install before rc/deployments something like https://github.com/helm/helm-classic/blob/master/action/install.go#L18  so that the pods will have the ip of the service in its environment.

I've been setting my services, secrets, and configmaps as pre-install and pre-upgrade hooks to achieve this behavior.
Example:
apiVersion: v1
kind: Service
metadata:
  name: foo
  annotations:
    "helm.sh/hook": "pre-install,pre-upgrade"

@elementalvoid Still probably Helm should do this by default. I believe kubectl does this. Also, IIRC deployments will fail if configs and secrets aren't available when it is deployed.

yes...you can achieve this by setting hooks but would be nice to have helm do it instead of we setting hooks for all the service charts.
@onbjerg i am not sure kubectl does that as helm uses kubectl code AFAIK

Don't get me wrong. I would also like this as a feature. I'm just suggesting a work around...
@onbjerg If kubectl does this it's news to me. If I have a directory of yaml files and I run kubectl -f dir/ then they seem to be created based on file name ordering. So, for bare kubectl stuff I've taken to numbering my yaml files (10-service.yml, 20-configmap.yml, 30-deployment.yml, etc). It's definitely not ideal.

@elementalvoid You're right. I just assumed this because I somehow magically named my files such that this always happened 😢


PRESS event is not detected in onBackgroundEvent

I'm using notifee to send local notification from inside the app and want to be able to detect that user pressed on notification.
I thought I will be able to check for PRESS event type inside onBackgroundEvent, however I noticed that onBackgroundEvent is triggered only once with DELIVERED event type, then the app is opened, and only then I can detect PRESS event type inside onForegroundEvent that is triggered once the app is opened.
my questions are -

Is this the expected behaviour ?
shouldn't onBackgroundEvent be able to use also to detect PRESS event ?
Is it possible that onBackgroundEvent will be triggered multiple times, first with DELIVERED, then with PRESS ?
If not, how else it can detect PRESS if it was already triggered with DELIVERED ?

I found the following "hint" that implies I should use onForegroundEvent but I didn't really understand why...
I'm using only notifee without Firebase, most examples & issues I saw involve also Firebase.
In iOS I at least get both onBackgroundEvent & onForegroundEvent triggered, in Android I'm seeing only onBackgroundEvent triggered once with DELIVERED and don't see onForegroundEvent triggered at all.
I thought this could be related to this issue, however I'm using RN < 0.73:

"react-native": "0.72.10"
"@notifee/react-native": "^7.8.2",

Just to clarify - I want to detect when user pressed on notification itself, I don't have any buttons on the notification.

Same issue for me.
"react-native": "0.72.4",
"@notifee/react-native": "9.0.0",
On android, notifee's onBackgroundEvent and even getInitialNotification doesn't fire if user click on notification. I have already added following in index.js file.

messaging().setBackgroundMessageHandler(async remoteMessage => {
console.log('Message handled in the background!', remoteMessage);
});

I even tried what he says in this comment. https://github.com/invertase/notifee/issues/616#issuecomment-1380264271
But still not working for Android.
My use case is: that I want to support both local and remote push notifications.
What I have tried:
If I use the standalone package https://github.com/wix/react-native-notifications only for remote push notifications -> This works fine in all cases for Android and IOS. But this doesn't support android local notifications.
If I use Notifee package version 5.7.0 and use Firebase functions (getInitialNotification, onNotificationOpenedApp) this also works good for remote notifications and  local notifications.
If I try to use notifee version 9.0.0 above mentioned issue occurs.

this is what I wanted to find why: android events are not working as expected.
I didn't find the reason yet, but I can share some information you might help.
const apiModule = new NotifeeApiModule_1.default({
    version: version_1.version,
    nativeModuleName: 'NotifeeApiModule',
    nativeEvents: utils_1.isIOS
        ? [utils_1.kReactNativeNotifeeNotificationEvent, utils_1.kReactNativeNotifeeNotificationBackgroundEvent]
        : [utils_1.kReactNativeNotifeeNotificationEvent],
});

first, notifee registers nativeEvents to receive events. remember the event name:  kReactNativeNotifeeNotificationEvent.
export const kReactNativeNotifeeNotificationEvent = 'app.notifee.notification-event';

second, initialize module based on NotifeeApiModule.java (create method to connect native module). you can find some platform specific workaround.
class NotifeeApiModule extends NotifeeNativeModule_1.default {
    constructor(config) {
        super(config);
        if (utils_1.isAndroid) {
            // Register background handler
            react_native_1.AppRegistry.registerHeadlessTask(utils_1.kReactNativeNotifeeNotificationEvent, () => {
                return (event) => {
                    if (!backgroundEventHandler) {
                        console.warn('[notifee] no background event handler has been set. Set a handler via the "onBackgroundEvent" method.');
                        return Promise.resolve();
                    }
                    return backgroundEventHandler(event);
                };
            });
        }
        else if (utils_1.isIOS) {
            this.emitter.addListener(utils_1.kReactNativeNotifeeNotificationBackgroundEvent, (event) => {
                if (!backgroundEventHandler) {
                    console.warn('[notifee] no background event handler has been set. Set a handler via the "onBackgroundEvent" method.');
                    return Promise.resolve();
                }
                return backgroundEventHandler(event);
            });
        }
    }
    ...
    onForegroundEvent = (observer) => {
        if (!(0, utils_1.isFunction)(observer)) {
            throw new Error("notifee.onForegroundEvent(*) 'observer' expected a function.");
        }
        const subscriber = this.emitter.addListener(utils_1.kReactNativeNotifeeNotificationEvent,
        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
        // @ts-ignore See https://github.com/facebook/react-native/pull/36462
        ({ type, detail }) => {
            observer({ type, detail });
        });
        return () => {
            subscriber.remove();
        };
    };
    ...

as you can see, Android event listener uses same key, kReactNativeNotifeeNotificationEvent. the only difference is, foreground is using emitter.addListener, while background is AppRegistry.registerHeadlessTask.
public void onNotificationEvent(NotificationEvent notificationEvent) {
    ....
    if (isAppInForeground()) {
      eventMap.putBoolean(KEY_HEADLESS, false);
      NotifeeReactUtils.sendEvent(NOTIFICATION_EVENT_KEY, eventMap);
    } else {
      eventMap.putBoolean(KEY_HEADLESS, true);
      NotifeeReactUtils.startHeadlessTask(NOTIFICATION_EVENT_KEY, eventMap, 60000, null);
    }
    ....
}

onNotificationEvent called when you have some notification events, such as delivered, press.
and check app is foreground or not, module send event with sendEvent or startHeadlessTask.
to sum up, Android uses same event key, and native module calls different function whether foreground or not.
that's the how logic works. let's back to the question what you've asked:

Is this the expected behaviour ?

yes, and no. (as far as I've researched)
the only point checking foreground status is there. isAppInForeground() function. and what that means, if the function returns true, it always works as foreground. in reverse, yes background as well.
you expected pressing notification from background should be call background event, but it's not. isAppInForeground() says you're in foreground, so onForegroundEvent called.
expected: (pressing notification from background) -> detected as background and calls startHeadlessTask -> show events at onBackgroundEvent -> app shows up
reality: (pressing notification from background) -> app shows up -> detected as foreground and calls sendEvent -> show events at onForegroundEvent

shouldn't onBackgroundEvent be able to use also to detect PRESS event ?

I also remember that feature, when I was using 5.7.0 before. but years later... it just disappeared!
(well, I can merely guess there are tons of things changed after Android 12 (and 13, 14!), notifee has no option but following up these changes.)

Is it possible that onBackgroundEvent will be triggered multiple times, first with DELIVERED, then with PRESS? If not, how else it can detect PRESS if it was already triggered with DELIVERED ?

If you can fix logic, yes. there is no data difference between foreground and background.

Android I'm seeing only onBackgroundEvent triggered once with DELIVERED and don't see onForegroundEvent triggered at all.

if "triggered at all" means no events shows up, you need to check reactContext. I've said bug can occur RN >0.73, but there are no guarantee not to happened when you avoid it.

Any update?

Possible duplicate. See:

#1118

same issue, is there any update for this case?

Try to use deeplink in case of when app is in kill state
On Fri, 15 Nov 2024 at 08:43, Muhamad Zulfiqor @.***>
wrote:

same issue, is there any update for this case?
—
Reply to this email directly, view it on GitHub
https://github.com/invertase/notifee/issues/1105#issuecomment-2477892038,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARC5FBAOQECK6E3LUZQPSOD2AVUU7AVCNFSM6AAAAABOUQC7UCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINZXHA4TEMBTHA
.
You are receiving this because you commented.Message ID:
@.***>

hi guys.
on android 14+ we calling notifee.displayNotification with asForegroundService: false because of android restrictions,
we have call method   notifee.onBackgroundEvent(notifeeBackgroundEventHandler) in index.js. also
in notifeeBackgroundEventHandler we listen EventType.ACTION_PRESS and handle user chose
when notification has displayed first time after app start then notifeeBackgroundEventHandler calls as expected
when notification has displayed second time after app start then notifeeBackgroundEventHandler does not call anymore
I think, there is missed part in documentation about  calling notifee.onBackgroundEvent

notifee.onBackgroundEvent should be called in index.js for the first time start,
notifee.onBackgroundEvent should be called when application moved to background for receiving events in background also
have added this code in App.tsx

  useEffect(() => {
    if (appState === 'background') {
      notifee.onBackgroundEvent(notifeeBackgroundEventHandler)
    }
    }, [appState])

it has fixed calling notifeeBackgroundEventHandler in background when user pressed on notification
You are Wellcome


There was a problem publishing bot assets (zip deploy)

Hi there,
I'm trying to deploye my bot on azure but I'm getting this error every time i try to upload.
Please help on this
"{\"name\":\"DEPLOY_ZIP_ERROR\",\"message\":\"There was a problem publishing bot assets (zip deploy). \\\"Getting status of zip upload processing failed. {\\\\\\\"stack\\\\\\\":\\\\\\\"Error: Request failed with status code 403\\\\\\\\n at createError (C:\\\\\\\\\\\\\\\\Program Files\\\\\\\\\\\\\\\\Bot Framework Composer\\\\\\\\\\\\\\\\resources\\\\\\\\\\\\\\\\app.asar.unpacked\\\\\\\\\\\\\\\\extensions\\\\\\\\\\\\\\\\azurePublish\\\\\\\\\\\\\\\\dist\\\\\\\\\\\\\\\\extension.js:105969:17)\\\\\\\\n at settle (C:\\\\\\\\\\\\\\\\Program Files\\\\\\\\\\\\\\\\Bot Framework Composer\\\\\\\\\\\\\\\\resources\\\\\\\\\\\\\\\\app.asar.unpacked\\\\\\\\\\\\\\\\extensions\\\\\\\\\\\\\\\\azurePublish\\\\\\\\\\\\\\\\dist\\\\\\\\\\\\\\\\extension.js:105983:14)\\\\\\\\n at IncomingMessage.handleStreamEnd (C:\\\\\\\\\\\\\\\\Program Files\\\\\\\\\\\\\\\\Bot Framework Composer\\\\\\\\\\\\\\\\resources\\\\\\\\\\\\\\\\app.asar.unpacked\\\\\\\\\\\\\\\\extensions\\\\\\\\\\\\\\\\azurePublish\\\\\\\\\\\\\\\\dist\\\\\\\\\\\\\\\\extension.js:106970:13)\\\\\\\\n at IncomingMessage.emit (events.js:215:7)\\\\\\\\n at endReadableNT (_stream_readable.js:1183:12)\\\\\\\\n at processTicksAndRejections (internal/process/task_queues.js:80:21)\\\\\\\",\\\\\\\"message\\\\\\\":\\\\\\\"Request failed with status code 403\\\\\\\",\\\\\\\"config\\\\\\\":{}}\\\"\",\"stack\":\"Error\\n at new CustomizeError (C:\\\\Program Files\\\\Bot Framework Composer\\\\resources\\\\app.asar.unpacked\\\\extensions\\\\azurePublish\\\\dist\\\\extension.js:276418:25)\\n at createCustomizeError (C:\\\\Program Files\\\\Bot Framework Composer\\\\resources\\\\app.asar.unpacked\\\\extensions\\\\azurePublish\\\\dist\\\\extension.js:276423:10)\\n at BotProjectDeploy.deployZip (C:\\\\Program Files\\\\Bot Framework Composer\\\\resources\\\\app.asar.unpacked\\\\extensions\\\\azurePublish\\\\dist\\\\extension.js:276954:13)\"}"------"DEPLOY_ZIP_ERROR - There was a problem publishing bot assets (zip deploy). \"Getting status of zip upload processing failed. {\\\"stack\\\":\\\"Error: Request failed with status code 403\\\\n at createError (C:\\\\\\\\Program Files\\\\\\\\Bot Framework Composer\\\\\\\\resources\\\\\\\\app.asar.unpacked\\\\\\\\extensions\\\\\\\\azurePublish\\\\\\\\dist\\\\\\\\extension.js:105969:17)\\\\n at settle (C:\\\\\\\\Program Files\\\\\\\\Bot Framework Composer\\\\\\\\resources\\\\\\\\app.asar.unpacked\\\\\\\\extensions\\\\\\\\azurePublish\\\\\\\\dist\\\\\\\\extension.js:105983:14)\\\\n at IncomingMessage.handleStreamEnd (C:\\\\\\\\Program Files\\\\\\\\Bot Framework Composer\\\\\\\\resources\\\\\\\\app.asar.unpacked\\\\\\\\extensions\\\\\\\\azurePublish\\\\\\\\dist\\\\\\\\extension.js:106970:13)\\\\n at IncomingMessage.emit (events.js:215:7)\\\\n at endReadableNT (_stream_readable.js:1183:12)\\\\n at processTicksAndRejections (internal/process/task_queues.js:80:21)\\\",\\\"message\\\":\\\"Request failed with status code 403\\\",\\\"config\\\":{}}\""---

Closing due to lack of response.


Create unit file to run RR in linux in daemon mode

Create rr.service unit file to run with systemd on linux-based systems.

Please make sure to add it to docs.


Please help! dependency wasn't found

Import: Game file 'story_esports' has been found
Import [Warning]: Dependency 'resources/unity_builtin_extra' wasn't found
Import [Warning]: Dependency 'library/unity default resources' wasn't found
Import [Warning]: Dependency 'archive:/cab-fe5f808d4929fcfbc7537a53fd29a740/cab-fe5f808d4929fcfbc7537a53fd29a740' wasn't found
how can I fix these errors?
I am a Russian novice programmer and I do not understand English well. I translate everything through " google translator"
Therefore, if you explain everything in a simple way , I will be happy :)
Game - PC Bulding Simulator
I want to change a couple of codes in the game

They are not errors, only warnings. You are probably fine.


Make sure that prompt2model has the ability to retrieve phi-1.5

phi-1.5 is a new small model with seemingly nice capabilities: https://arxiv.org/abs/2309.05463
It'd be good to make sure that our model retriever has the ability to retrieve this model and fine-tune it.

Is this issue still open to be worked upon?


Add actress to CI in the cloud

It would be cool to build actress in the cloud on: https://travis-ci.org/
You could then add a build status in the ReadMe file:
[![Build Status](https://travis-ci.org/wblancqu/actress.svg?branch=master)](https://travis-ci.org/wblancqu/actress)
Finally just add a .travis.yml file containing:
language: java
jdk: openjdk6

Cool stuff, didn't know about that one. Thanks for sharing!


TypeError: options.customExtensions is not iterable

At build function, when using with options:
build(
    'TeX',
    'AsciiMath',
    'CommonHTML', {
        toFile: true,
        compress: true,
        folder: 'ascii'
    }
);

It fails because you try to ...options.customExtensions, but it's undefined.
I suppose you meant options to be written above default options object, but it's not made. So use smth like options = Object.assign({}, defaultOptions, options); (it solved issue on my side)

Thanks for the report.


Query cosmos DB

Can Now Query Items from Cosmos DB

Created /downloads page in client to provide file ID
Issued SQL Query in CosmosDBService in server proj
Verified data is returned successfully
Can also edit the query to find by file name.
Added Navbar item to link to downloads page.

 Pull data from Cosmos DB


Interactive audio transcript

Hi,
A neat feature would be the ability to have show the transcript of an audio file on the player as well (and have the ability to click on a line and then jump to that part of the audio).
This would be very useful for recordings of lectures and other talks.
(I've had the idea for a while, but when Google released it's Recorder app that does this I figured I needed to find a javascript player that could do that as well but with the text I'd provide myself)

any suggestions on how to do it?

I think AblePlayer does this if you'd like to see running code.
It basically requires a video closed captioning file instead of a transcript, injecting the text of the file into a div with data attributes for the timestamps, scrolling to each timestamp as the player reaches that time, and adding onClick events for each sentence to jump to that time.
Easypeasy, lol. It's been long enough that I've been involved in one of these that I don't have any of my code samples handy, but other code samples are out there (can't vouch for the accessibility or functionality of that one, but the theory and basic JS seem sound).


Add customPath support for non-standard PyPI directory structures

Description of changes
This PR improves the fetchPypi function to support custom URL paths. A customPath attribute is included to allow users to manually specify a path to a package to avoid issues with automatic path generation that can lead to incorrect URLs. For example when using: https://pypi.org/project/verilogae/ with fetchPypi which does not work.
The existing fetchPypi function is designed to generate URLs for fetching Python packages from PyPI based on standard patterns, using attributes like pname, version and dist. But, this approach can fail when dealing with packages hosted in non-standard or hash-based directories which somtimes occurs on PyPI.
For example, a package might be stored at a URL like:  https://files.pythonhosted.org/packages/aa/a6/9daea00745844faba44c2917c66838adfc315269f38518ecca49e6eb5fb5/verilogae-1.0.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
The current fetchPypi logic generates a URL that incorrectly duplicates path segments or misinterprets the directory structure, causing a 404 error during fetch. The customPath attribute allows a URL path to be specified which bypasses the automatic path construction that may not always work.
Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[X] x86_64-darwin
[X] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[X] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Why not use fetchurl?

Why not use fetchurl directly?

I did but it was recommended  to use fetchPypi instead as that's the better option.

I cannot review this change. Please split the formatting into it's own commit.
Also I think you are understanding this completely wrong. We are relying on none well documented redirects which explicitly do not require parts of the hash. If those files cannot be downloaded without those hashes, it is out of scope of fetchPypi.

Okay so with that being said you would recommend I stick with fetchurl? I can close this PR then, especially since @philiptaron has yet to respond.

Yeah, If this is only a problem for this single project then u am very hesitant to change the fetchpypi behavior for it.

Jason, thanks for your kind email. I'm afraid that I only did not respond because I'd have to learn a lot before making a judgement.
If you're blocking by this downloader, I suggest writing an expression that uses a more primitive fetcher: perhaps even fetchurl itself, which allows for you to control every aspect of how you download from a URL, and won't get tripped up on misinterpreting various attributes in corner cases.


2.10 MetaData Trigger does not find Trigger for DLRS

Create a metadata based roll up summary, realtime mode.
Create a child record
Nothing happens. Looking at the debug log, it doesn't find there is anything that needs to be done.
The using the meta-data rule works.  Only difference I can find is the MetaData trigger uses difference Case in the trigger name compared to the non-meta data trigger rules.
Dlrs debug log.log
https://success.salesforce.com/_ui/core/chatter/groups/GroupProfilePage?g=0F9300000009O5p&fId=0D53A00003RpJq7

Was on this call some of the strangeness we saw is the naming of the trigger was slightly different when deployed with Metadata than vs Classic style.  The classic style had ObjectName where the Metadata has objectName in the trigger name.
Also using the Classic record with the trigger deployed from the classic style works in his org.   Also as you can see in his logs DLRS makes a SOQL for DLRS records finds none and returns but doesn't seem to scan for Metadata records.

@Opopanax  dont you mean the Non-metadata style works?

The Trigger name case will not matter, they just get the tool running, which it clearly is. So it's looking for a match on the child object name, calculation mode realtime or scheduled and an active rollup.  Are all these conditions true on the rollup you expect to be selected? Can you include a screenshot? Also are you using the standard DLRS triggers?

He was using the standard triggers. Interesting if he deployed the trigger
via metadata dlrs the standard dlrs didn't find it and would ask to deploy
a new trigger.  Vice versa also. If he deployed via standard dlrs then the
metadata didn't see the already deployed trigger. Even when deployed via
metadata when his dlrs ran it was looking for standard dlrs records then
when the soql returned none the dlrs code would return.  He was using
lightning if that matters.
On Feb 25, 2018 11:30 AM, "Andrew Fawcett"<EMAIL_ADDRESS>wrote:

The Trigger name case will not matter, they just get the tool running,
which it clearly is. So it's looking for a match on the child object name,
calculation mode realtime or scheduled and an active rollup. Are all these
conditions true on the rollup you expect to be selected? Can you include a
screenshot? Also are you using the standard DLRS triggers?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/643#issuecomment-368337254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKqjjhgQVrE5pMfS2SUg0ZHqqkL0x0ks5tYbThgaJpZM4SQCF8
.

Oooh i see! This is not a runtime thing? Its a deploy thing? Gotcha. So yeah some issue here then. Need to really try to avoid the hybrid use of roll-up custom object vs custom metadata tbh. As it cases a number of confusions.... this said, re-reading the original post it looks like a runtime thing? Confused now...

Also a runtime thing. He deployed via metadata and if you look at his logs
dlrs makes a query for old style dlrs records. When it doesn't find any it
just returns and doesn't do it's magic.
On Feb 25, 2018 12:01 PM, "Andrew Fawcett"<EMAIL_ADDRESS>wrote:

Oooh i see! This is not a runtime thing? Its a deploy thing? Gotcha. So
yeah some issue here then. Need to really try to avoid the hybrid use of
roll-up custom object vs custom metadata tbh. As it cases a number of
confusions.... this said, re-reading the original post it looks like a
runtime thing? Confused now...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/643#issuecomment-368339655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKqrtpUNHHUFLAakQ9j7cR847_0h5sks5tYbwygaJpZM4SQCF8
.

He only tried the object based classic style to troubleshoot cus I asked
him to. Surprisingly the classic style works because when it runs the soql
search it returns dlrs records to process. So the TLDR he deployed via
metadata and only used metadata yet the dlrs code was still set on looking
for classic dlrs records
On Feb 25, 2018 12:16 PM, "Dan"<EMAIL_ADDRESS>wrote:

Also a runtime thing. He deployed via metadata and if you look at his logs
dlrs makes a query for old style dlrs records. When it doesn't find any it
just returns and doesn't do it's magic.
On Feb 25, 2018 12:01 PM, "Andrew Fawcett"<EMAIL_ADDRESS>wrote:

Oooh i see! This is not a runtime thing? Its a deploy thing? Gotcha. So
yeah some issue here then. Need to really try to avoid the hybrid use of
roll-up custom object vs custom metadata tbh. As it cases a number of
confusions.... this said, re-reading the original post it looks like a
runtime thing? Confused now...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/643#issuecomment-368339655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKqrtpUNHHUFLAakQ9j7cR847_0h5sks5tYbwygaJpZM4SQCF8
.

It does look for both though, per the above debug log...
SELECT Name, dlrs__Active__c, dlrs__AggregateAllRows__c, dlrs__AggregateOperation__c, dlrs__AggregateResultField__c, dlrs__CalculationMode__c, dlrs__CalculationSharingMode__c, dlrs__ChildObject__c, dlrs__ConcatenateDelimiter__c, dlrs__FieldToAggregate__c, dlrs__FieldToOrderBy__c, dlrs__ParentObject__c, dlrs__RelationshipCriteriaFields__c, dlrs__RelationshipCriteria__c, dlrs__RelationshipField__c, dlrs__RowLimit__c, dlrs__TestCodeSeeAllData__c, dlrs__TestCode__c, dlrs__UniqueName__c FROM dlrs__LookupRollupSummary__c WHERE CalculationMode__c in :calculationModeNames and ChildObject__c in :childObjectNames and Active__c = true ORDER BY Name ASC NULLS FIRST , dlrs__ParentObject__c ASC NULLS FIRST , dlrs__RelationshipField__c ASC NULLS FIRST
Then...
SELECT DeveloperName, Id, Label, dlrs__Active__c, dlrs__AggregateAllRows__c, dlrs__AggregateOperation__c, dlrs__AggregateResultField__c, dlrs__CalculationMode__c, dlrs__CalculationSharingMode__c, dlrs__ChildObject__c, dlrs__ConcatenateDelimiter__c, dlrs__Description__c, dlrs__FieldToAggregate__c, dlrs__FieldToOrderBy__c, dlrs__ParentObject__c, dlrs__RelationshipCriteriaFields__c, dlrs__RelationshipCriteria__c, dlrs__RelationshipField__c, dlrs__RowLimit__c, dlrs__TestCodeSeeAllData__c, dlrs__TestCode__c FROM dlrs__LookupRollupSummary2__mdt WHERE CalculationMode__c in :calculationModeNames and ChildObject__c in :childObjectNames and Active__c = true ORDER BY DeveloperName ASC NULLS FIRST , dlrs__ParentObject__c ASC NULLS FIRST , dlrs__RelationshipField__c ASC NULLS FIRST

Well that is even more weird why it didn't find any...
On Sun, Feb 25, 2018 at 1:11 PM, Andrew Fawcett<EMAIL_ADDRESS>wrote:

It does look for both though, per the above debug log...
SELECT Name, dlrs__Active__c, dlrs__AggregateAllRows__c,
dlrs__AggregateOperation__c, dlrs__AggregateResultField__c,
dlrs__CalculationMode__c, dlrs__CalculationSharingMode__c,
dlrs__ChildObject__c, dlrs__ConcatenateDelimiter__c,
dlrs__FieldToAggregate__c, dlrs__FieldToOrderBy__c, dlrs__ParentObject__c,
dlrs__RelationshipCriteriaFields__c, dlrs__RelationshipCriteria__c,
dlrs__RelationshipField__c, dlrs__RowLimit__c, dlrs__TestCodeSeeAllData__c,
dlrs__TestCode__c, dlrs__UniqueName__c FROM dlrs__LookupRollupSummary__c
WHERE CalculationMode__c in :calculationModeNames and ChildObject__c in
:childObjectNames and Active__c = true ORDER BY Name ASC NULLS FIRST ,
dlrs__ParentObject__c ASC NULLS FIRST , dlrs__RelationshipField__c ASC
NULLS FIRST
Then...
SELECT DeveloperName, Id, Label, dlrs__Active__c,
dlrs__AggregateAllRows__c, dlrs__AggregateOperation__c,
dlrs__AggregateResultField__c, dlrs__CalculationMode__c,
dlrs__CalculationSharingMode__c, dlrs__ChildObject__c,
dlrs__ConcatenateDelimiter__c, dlrs__Description__c,
dlrs__FieldToAggregate__c, dlrs__FieldToOrderBy__c, dlrs__ParentObject__c,
dlrs__RelationshipCriteriaFields__c, dlrs__RelationshipCriteria__c,
dlrs__RelationshipField__c, dlrs__RowLimit__c, dlrs__TestCodeSeeAllData__c,
dlrs__TestCode__c FROM dlrs__LookupRollupSummary2__mdt WHERE
CalculationMode__c in :calculationModeNames and ChildObject__c in
:childObjectNames and Active__c = true ORDER BY DeveloperName ASC NULLS
FIRST , dlrs__ParentObject__c ASC NULLS FIRST , dlrs__RelationshipField__c
ASC NULLS FIRST
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/afawcett/declarative-lookup-rollup-summaries/issues/643#issuecomment-368345011,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APXKquXPORfnjO21P-yh0cGe87lVeLlvks5tYcx5gaJpZM4SQCF8
.

@DanDonin  and/or @Opopanax whats the next steps with this one?

Closing for now due to inactivity.


fix: load element templates from a directory relative to diagram clos…

Closes #1376

Thank you for your contribution! I pushed slight changes and an additional test case so that we do not break anything :)


Informix not escape column and table name

I'm working with an Informix DB engine and I discovered that, in the CRUD operations, Informix generates an error if the colums and table names are escaped. I have updated the  function escapeIdentifier(...)  as shown below:
public function escapeIdentifier(string $string, bool $quote = true): string
{
....
 if ($quote) {
            switch ($this->dbEngine) {
                case 'mssql':
                    return '[' . $str . ']';
                case 'mysql':
                    return '`' . $str . '`';
                case 'informix':
                    return $str;
                default:
                    return '"' . $str . '"';
            }
        }
        return $str;
}

The solution it is under test and it seems work perfectly!
Best regards
Francesco

Seems like you need to explicitly enable delimiters with informix ?
Via stackoverflow:

DELIMIDENT: When set to Y, specifies that strings set off by double quotation marks are delimited identifiers
~ IBM Knowledgebase


Windows support

See #11

I started setting up Appveyor but I don't have a windows machine to test on locally and I was having trouble getting the powershell script setup correctly.  If anyone is interested in helping let me know and I will figure out how to give you permissions.


Provide compressed releases

Add make release-compress to provide compressed releases
Fix #1752

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3991   +/-   ##
=======================================
  Coverage   20.08%   20.08%
=======================================
  Files         151      151
  Lines       29874    29874
=======================================
  Hits         6000     6000
  Misses      22968    22968
  Partials      906      906

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 b574af2...565addf. Read the comment docs.


Fix references in test resources and doc generation issues

Purpose

Resolves #11197, Resolves #9862, Resolves #11265

Codecov Report

Merging #11266 into master will increase coverage by 0.04%.
The diff coverage is 81.69%.

@@             Coverage Diff              @@
##             master   #11266      +/-   ##
============================================
+ Coverage     58.39%   58.43%   +0.04%
  Complexity      393      393
============================================
  Files          1961     1962       +1
  Lines         99041    99110      +69
  Branches      13568    13579      +11
============================================
+ Hits          57835    57919      +84
+ Misses        36072    36051      -21
- Partials       5134     5140       +6

Impacted Files
Coverage Δ
Complexity Δ

...ons/util/sorters/VariableDefContextItemSorter.java
95.83% <ø> (ø)
0 <0> (ø)
:arrow_down:

...erver/completions/util/CompletionItemResolver.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ang/langserver/formatting/FormattingSourceGen.java
71.42% <ø> (+0.18%)
0 <0> (ø)
:arrow_down:

...server/completions/util/ItemResolverConstants.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...pletions/resolvers/BLangRecordContextResolver.java
66.66% <0%> (+5.12%)
0 <0> (ø)
:arrow_down:

...completions/resolvers/ResourceContextResolver.java
6.66% <0%> (ø)
0 <0> (ø)
:arrow_down:

...completions/resolvers/FunctionContextResolver.java
7.14% <0%> (ø)
0 <0> (ø)
:arrow_down:

...mpletions/resolvers/ObjectTypeContextResolver.java
66.12% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../langserver/completions/LSCompletionException.java
0% <0%> (ø)
0 <0> (?)

...server/completions/resolvers/TopLevelResolver.java
97.91% <100%> (+0.13%)
0 <0> (ø)
:arrow_down:

... and 30 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 79f4a36...cf0cd15. Read the comment docs.


Proxy support

We need to support PROXY in our requests.

We use this tool behind proxy in enterprise and setting export HTTPS_PROXY= has worked so far. The shell env doesn't get picked up by go's http module?

It gets picked up tho
With
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ export HTTPS_PROXY=donexist.co.jp                                                                                1 ↵
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ export HTTP_PROXY=donexist.co.jp
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ gobrew use 1.19rc1
[Info] Downloading version: 1.19rc1
[Info] Downloading from: https://golang.org/dl/go1.19rc1.darwin-amd64.tar.gz
[Info] Downloading to: /Users/pulkit.kathuria/.gobrew/downloads
[Info]: Downloading version failed: Get "https://golang.org/dl/go1.19rc1.darwin-amd64.tar.gz": proxyconnect tcp: dial tcp: lookup donexist.co.jp on <IP_ADDRESS>:53: no such host
[Error]: Please check connectivity to url: https://golang.org/dl/go1.19rc1.darwin-amd64.tar.gz

Without
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ export HTTP_PROXY=                                                                                1 ↵
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ export HTTPS_PROXY=
╭─pulkit.kathuria@P84760 ~/git/dev-classified-ui ‹master●›
╰─$ gobrew use 1.19rc1
[Info] Downloading version: 1.19rc1
[Info] Downloading from: https://golang.org/dl/go1.19rc1.darwin-amd64.tar.gz
[Info] Downloading to: /Users/pulkit.kathuria/.gobrew/downloads
Downloading  19% |███████████████                                                                 | (29/144 MB, 9.036 MB/s) [3s:12s]^Downloading  26% |████████████████████                                                            | (38/144 MB, 9.385 MB/s) [4s:11s]^

Excellent!


Detection of "Execution of Command Via Utility"

Signed-off-by: ashutoshmore658<EMAIL_ADDRESS>(CloudDefense.ai)
What type of PR is this?

/kind feature

Any specific area of the project related to this PR?

/area rules

Special notes for your reviewer:
This PR adds a special kind of rule which detects command execution done by using any of the other utilities than terminal,

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows
26820e9 adds evet.type in the rule as per the review recieved
38e6ad5 Merge remote-tracking branch 'origin/add-get-window-info-rule' into add-get-window-info-rule
00ef6da Merge branch 'add-get-window-info-rule' of https://github.com/ashutoshmore658/rules into add-get-window-info-rule

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ashutoshmore658
Once this PR has been reviewed and has the lgtm label, please assign mstemm for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


Unable to login to GMail

I am trying to login to my Google work account.
I get this OAuth error:

The authentication works fine on a regular browser.

+1

Same issue here, is there any update?

I am trying to login to my Google work account.
I get this OAuth error:

The authentication works fine on a regular browser.

I found a workaround. For whatever reason, Google Hangout works fine. Sign in to that first, and then use the app menu on the top left of the hangout window to jump to other applications.

I can confirm that the work around used by @mjasyal-z works . It appears that the login for hangouts occurs in the station application where as the login for gmail, calendar, and drive all open up the default web browser for login.

It works, but only if you use 1 single gmail account (unless others have managed to make it work with multiple).

Hi guys,I followed @kpinegar2 's suggestion, but it show me : the broswer is not safe ..
any suggestion ?

well ,i'm from china using proxy

all right,problem settled with long time trying and fresh

I get the same error trying to connect to Hangout : "This browser is not safe". Any clue to get Gmail/GDrive account working on Station ?

Fixed in 2.1.0


[BugFix] Fix the bug in the FE clone scheduling that causes an infinite loop

Why I'm doing:
Some VERSION_INCOMPLETE replicas may not be able to be scheduled for clone repair, such as the following scenario:

The partition visible version is 100 because there are some replica version is still 100 and version 101 publish failed.
The ingestion task is continuous and the lastest committed version is 120.
There are two VERSION_INCOMPLETEreplicas, the first replica commit version101and120failed, so  the first replica version is100, last failed version is 120.  The second replica commit version 102and108failed, so the second replica version is101and the last failed version is108`.
We consider these two replicas are needed to repair because their last failed version are both greater than 0 and we will choose the replica which last failed version is smaller to repair first.
However, we use visible version in clone task because the clone task can only clone the data with visible version. So the second replica version is still 101 and last failed version is 108.
After clone, the second version is still VERSION_INCOMPLETE and we will try to repair again and again which cause the first replica can not repair any more and the the partition visible version is blocked.

What I'm doing:
Try to fix the replica which version is smaller first.
Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@mergify backport branch-2.5


[WIP] Update of deps

Coverage decreased (-0.1%) to 75.701% when pulling bbe815adbd24bd45190ebbec338e753303d8f0bb on lukasmacko:update into d5caba8fed979217c2e0a753ce57e3017aed87d4 on contiv:master.


New version: SeeToDee v1.2.2

Registering package: SeeToDee
Repository: https://github.com/baggepinnen/SeeToDee.jl
Created by: @baggepinnen
Version: v1.2.2
Commit: 924651f826cc508b45a3156393140347aa4ea123
Reviewed by: @baggepinnen
Reference: https://github.com/baggepinnen/SeeToDee.jl/commit/924651f826cc508b45a3156393140347aa4ea123#commitcomment-134966880
Description: A small step for dynamics, a giant leap for SciML

This can be closed :pray:


Whats the process for people who want to stay on v4 ?

We are currently in the process of migrating our workloads to V4, and we don't have enough resources to upgrade to V5. My primary concern is whether the process of upgrading the cluster, for instance from 1.27 to 1.28, will continue to be compatible with your latest release. If not, what additional changes would we need to implement?
You can't just leave people in the lurch. There should be some documentation outlining what steps need to be taken. For third-party add-ons, we can modify the configuration with Helm, but our main concern lies with AWS native add-ons and cluster upgrades.

Hi @mk2134226 - great question. In short, nothing is required - you should always pin your modules to avoid any unintended changes and you can continue to use that version for as long as you wish or until you are ready to make changes to follow the new direction in v5. As I've stated in a similar question here, there really isn't anything required to support later versions of EKS. There may be some configuration changes required based on the addons if they have changes required to support later versions, but this configuration is already exposed to users today for them to configure as they see fit.
Lastly, we are not leaving anyone in the dark. When we make the changes in the project, there will be documentation provided. Most of this should be captured under issue https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/1421
Please let me know if there are any additional questions I can help answer

FYI - if you wish to stay on v4 you will need to pin your version to v4.32.1 - you can read more about why this version here


Add Coquitlam School District 43

http://www.sd43.bc.ca/Pages/default.aspx
https://en.wikipedia.org/wiki/School_District_43_Coquitlam

@mkotb Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Running jblas from multiple processes

I have a case where I need to run more than 1 instance of jblass at a time, for example, 1st is continuesly calculating big matrix, while in 2nd I occasionally experimenting with intermediate results of the 1st.
the problem is with mechanism of extracting native libraries into temp dir and loading it from there, even if I put 2 required dll in source dir, so they are copied into bin during compilation, the second case started and worked, but in the log it shows the error:
-- org.jblas ERROR Could not create the temp file: java.io.FileNotFoundException: C:\Users\xxx\AppData\Local\Temp\libgcc_s_dw2-1.dll (The process can not access the file because the file is in use by another process). // translated back from Russian
-- org.jblas ERROR Could not create the temp file: java.io.FileNotFoundException: C:\Users\xxx\AppData\Local\Temp\libgfortran-3.dll (The process can not access the file because the file is in use by another process).
I use with same manner JNA jar, and it also depends on native dll, but with it I am able to put this dll's into some source folder (then copied to bin) and just point JNA during startup this folder (actually, since I use a default location for WIN32x86, I even don't point anything specially)
so I think it would be useful to every one to redesign dll load pattern into something more usable, without need on specifying environment variables, which is in my opinion a bit hard operation, having in mind you have everything in several separate development environments (eclipse in my case) running in parallel.
again, I think the load pattern from JNA could be used as an example here.
btw, JNA, of course, uses JNI for that, so they develop some C wrapper around library loading, and I believe this could be done for jblas as well.

looked through sources, it seems that you always load 3d party dlls from same single place from temp dir. this is probably for optimization of time of extracting in from jar, I guess, right?
BUT, for some reason the code in LibraryLoader.loadLibraryFromStream didn't check the existence of tempFile.
tempFile.isExists() before OutputStream.write in LibraryLoader.loadLibraryFromStream should solve the problem of running jblas from multiple JVM processes.


Rectified the display name of BracketLocalBackend

Summary
I changed the value of name argument value passed to the super-class of BraketLocalBackend to solve the issue #87. The bug was due the value 'sv_simulator' passed in the name argument instead of the backend name of called backend.

Sure @kshitijc

@kshitijc I added the test case

@devilkiller-ag the tests are failing. Could you please fix?

Sure, I am trying

@kshitijc Is everything great now?

Hi, @kshitijc , @IceKhan13, Is this PR ready to merge now?

Sorry for the failure of previous styles check. I have rectified it and ran all the tests on local. It has been rated 10/10 now.


Comparison with izumi-reflect?

Hi,
I'm wondering if you may add some notes about the differences between this library and https://github.com/zio/izumi-reflect

I’m not familiarvwith this library, unfortunately.  It doesn’t appear to be native to Scala 3, although a port is mentioned.  Not sure if they use macros or not—if not I’d be very curious how it performs.  Try it out and see what the reflected data structures give you in both.


'wake runSim pioDUT' fails to compile firrtl

hi,
When I run the command  wake runSim pioDUT
It throws the following error

[E] [E5] ../../firrtl/src/main/scala/firrtl/proto/FromProto.scala:308:22
[E]      value hasExternalModule is not a member of firrtl.FirrtlProtos.Firrtl.Module
[E]      L308:       require(module.hasExternalModule, "Module must have Module or ExtModule")
[E]                                 ^
[E] [E4] ../../firrtl/src/main/scala/firrtl/proto/FromProto.scala:306:16
[E]      value hasUserModule is not a member of firrtl.FirrtlProtos.Firrtl.Module
[E]      L306:     if (module.hasUserModule) convert(module.getUserModule)
[E]                           ^
[E] ../../firrtl/src/main/scala/firrtl/RenameMap.scala: L76 [E1], L77 [E2], L78 [E3]
[E] ../../firrtl/src/main/scala/firrtl/proto/FromProto.scala: L306 [E4], L308 [E5]
[E] 'firrtl' failed to compile.
Scala compilation failed for 'api-scala-sifive/bloop_wake -t build/api-scala-sifive/bloop/firrtl.json.template -c build/api-scala-sifive/bloop/firrtl.json -b scala --coursier-cache ivycache --bloop-args "compile -c build/api-scala-sifive/bloop firrtl" --jar build/firrtl/scala-2.12/jar/firrtl.jar'

How can I resolve this?

I believe this comes from too old of a version of protoc, I'm guessing this is ~3.0.0 when you need 3.5.0-3.7.1
We generally solve this by getting the right protoc version from here: https://github.com/protocolbuffers/protobuf/releases/tag/v3.5.1. Assuming Linux, you can get the Linux-agnostic binary protoc-3.5.1-linux-x86_64.zip from that release page.
You then should make an environment module based on https://github.com/sifive/environment-example-sifive (either make a new repo that has a wit dependency on it, or just fork it), and fill in an environment JSON file as described in the README that points to the protoc install.
We're working on making this environment aspect of the flow a bit more automated and portable, but it's a hard problem 🙂

Thanks for the help It worked after installing the 3.5.1 version of the protoc


minify-type-constructors shouldn’t rewrite String(symbol)

Input
var symbol = Symbol();
String(symbol);
// → 'Symbol()'

Current output:
var symbol = Symbol();
symbol+"";
// → throws TypeError: Cannot convert a Symbol value to a string

Btw, String(x) and x + "" (or "" + x) are not equivalent in general. Here’s another snippet that doesn’t use symbols but still produces different output after minification:
var obj = {
  valueOf: _ => { console.log('valueOf'); return {}; },
  toString: _ => { console.log('toString'); return {}; }
};

try { String(obj); } catch (exception) { console.log(exception); }
// 1. toString
// 2. valueOf
// 3. TypeError: Cannot convert object to primitive value

// After minification, this becomes something like:
try { obj+""; } catch (exception) { console.log(exception); }
// 1. valueOf
// 2. toString
// 3. TypeError: Cannot convert object to primitive value

Thanks!
PR - #113
Is it that this transformation shouldn't be applied to any pattern of String(x) ?
Since we don't know type x in String(x) during compile time, I guess we shouldn't transform this at all.

I just wanted to make sure these edge cases are considered.
They might be rare enough in practice that the Babel team may decide not to care about them. But in that case, a note/warning in the README would be nice.

Yeah this is similar to template strings http://babeljs.io/docs/plugins/transform-es2015-template-literals/ although in this case users are intentionally using String.

This is merged and now there is an option to disable only String() type constructors.


Fix for #942 - set uuid correctly

Fixes #942 (mount rejected).
This was caused by the bug in handle_stale_attach where the code first check that 'cur_vm == None` and if it is, still tries to use curr.config in the log message.
Tested manually:

Step 1:  prepare a corrupted KV file

created a docker volume docker volume create -d vmdk --name=uv1
attached a volume to a VM docker run --name ng -d -p 80:80 -v uv1:/uv1 nginx
on ESX, copied the .vmfd file to a backup
killed the container to detach the volume docker rm -f ng
on ESX, copied a backup version back to .vmfd file and replaced attach_uuid with all 1 so VM with uuid would be not found

Step 2: test without a change docker run -d -p 80:80 -v uv1:/uv1 nginx

result docker: Error response from daemon: error while mounting volume '/mnt/vmdk/uv1': VolumeDriver.Mount: Server returned an error: AttributeError("'NoneType' object has no attribute 'config'",).
also the log shows traceback reported in the bug

Step 3: update the code with the fix ,  redeployed the VIB and rerun the command.

result: success and the following log

0/24/16 13:30:39 2560127 [photon-1-[datastore1] dockvols/4a3fccea-a878-40ec-b07c-24f58f1c0de2/uv1.vmdk] [WARNING] Failed to find VM (id 11111111-7496-cbf0-ce67-792f6bf76619) attaching the disk /vmfs/volumes/datastore1/dockvols/4a3fccea-a878-40ec-b07c-24f58f1c0de2/uv1.vmdk, resetting volume metadata
10/24/16 13:30:40 2560127 [photon-1-[datastore1] dockvols/4a3fccea-a878-40ec-b07c-24f58f1c0de2/uv1.vmdk] [INFO   ] Disk /vmfs/volumes/datastore1/dockvols/4a3fccea-a878-40ec-b07c-24f58f1c0de2/uv1.vmdk successfully attached. controller pci_slot_number=192, disk_slot=0

LGTM

Merging to master before CI; relying on @946 CI and on the fact that it's a trivial changes


Improve CLI docs

Fixed all the typos (hopefully).
Improved the bad English. This encompasses grammar, vocabulary, generally awkward phrases, etc. The result is easier to read and understand, and in some cases shorter.
Made some formatting improvements. This includes making formatting more consistent, but I changed some of it to be better, at least in my opinion.
General fixes. For example the dioxus-cli crate README points to a non-existent documentation URL. One more URL I recall is the Dioxus logo at the CLI intro page, which leads to nowhere. Removed it because I don't think it's necessary, and no other chapter has it.
Removed certain unnecessities, the biggest one being the commands documentation. I think this is unnecessary because of the dx --help command. It basically just repeats what dx --help says and it's an extra burden since it needs to be updated every time a command gets added/updated/removed.
Improved plugin documentation. I added a warning for the reader to ignore all of it, which is ironic, but the fact is that plugins are probably going to change. But by the time I learned that, I already made some changes to the plugin docs. This includes a better guide on how to get started (although still bad since the plugin system is too messy), and general changes like the aforementioned English or formatting.
And more. I can't list it all, but I didn't do any destructive changes. The content might be reformatted (to be shorter or easier to understand), but the meaning isn't lost.

This looks great! I agree that removing command documentation makes sense. The published CLI docs now live at https://github.com/DioxusLabs/docsite/tree/master/docs-src/0.4/en/CLI, not in this repo. See #1408

The published CLI docs now live at https://github.com/DioxusLabs/docsite/tree/master/docs-src/0.4/en/CLI, not in this repo. See https://github.com/DioxusLabs/dioxus/pull/1408

Really? docs-src/0.4/SUMMARY.md has incorrect paths. It ignores the fact that everything is in the docs-rs/0.4/en directory. Does it still work? Snippet from SUMMARY.md:
# Summary

[Introduction](index.md)

File structure:
0.4/
  ...
  SUMMARY.md
  en/
    index.md
    ...

Okay. I'll make a PR in the docsite repo, but this PR also changes the dioxus-cli crate README, since it also had some issues, and that README is not in the docsite repo. So I think this should be merged, but perhaps without the docs directory.


Introduce - add description code when problem report is sent

When the introduce protocol sends a problem report message description code is absent.

Done! https://github.com/hyperledger/aries-framework-go/pull/1253


💡[Feature]: Fruit and Vegetable Disease detection (healthy or rotten)

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
developing initially a web application that uses the camera to capture images of fruit or vegetables and tell if it's healthy or rotten using computer vision and deep learning models.
Use Case
it will help automation in fruit markets or juice factories for example, the machine would be able to recognize rotten fruits and remove it which in case will enhance production pipeline
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

hello @sanjay-kv plz assign this issue to me

Hey @sanjay-kv, I've been working on ML models for a long time and this is my first time contributing to GGSOC. I believe I can be a great help to you.


Support Bazel workspaces that have packages with same name in different paths

As a BEF user with a particular Bazel workspace
I would like to import the workspace entirely
Such that there is a way to handle multiple Bazel packages with the same leaf path name
Example:
Consider a Bazel workspace with this structure:
//projects/path1/apple-api/BUILD (java_library target here)
//projects/path2/apple-api/BUILD (java_library target here)
Right now you cannot import the entire workspace, you have to choose to only import one of the apple-api packages. This is because we name the Eclipse project with the leaf node of the path (apple-api). An Eclipse workspace cannot contain two projects of the same name.

Initial implementation should just tack on a numeric on the end of package name as needed. In the above example, the two projects would get named:

apple-api
apple-api2

without regard to the path names. So apple-api2 may be the project for package //projects/path1/apple-api/
But eventually I would like to support user configuration of the chosen project names using inline annotations in the BUILD files: see #177


Add a wiki: Aground

Link to origin wiki: https://aground.fandom.com/wiki/Aground_Wiki
Link to destination (independent) wiki: https://aground.wiki.gg/wiki/Aground_Wiki

Thank you for the submission! The wiki has been added in v3.11.0 (available for Chrome, soon to be available on Firefox).


Commands for generate api docs

Does anyone has idea if possible to use the commands for generate api docs inside a laravel project?
I see that there are some dependency of dingo library, so not sure if this can be replaced.
Thanks

Hi you can use https://github.com/mpociot/laravel-apidoc-generator to generate your api documentation for your laravel project.
Hope this help.

why didnt use api:docs ? @Herchelle

@Herchelle Thank You!  I was looking for something like that. Great
@muhtarudinsiregar api:docs is command for laravel? Or you are referring to dingo command?

@kamov dingo command

API documentation can be generated using Apidog, and I am deeply attracted by its UI for generating documents.


backing store and additional data properties are added to every inheritance level

looking at the latest generation logs and the latest go generation run it seems that both the backing store and the additional data properties are added to every model in the inheritance structure when it should only be added to the first ancestor that supports additional data.
https://github.com/microsoft/kiota/blob/61ec15f1c7879c3e6de177028a6d0f81c80557b1/src/Kiota.Builder/KiotaBuilder.cs#L1123

It could be that the order of processing the models has changed for some reason.
So, at the time of checking the model, for some reason the parent model hasn't had the AdditionalDataPropName property added to it yet as the generated parent model ends up with the property generated as well.

Thanks for chiming in. As far as I can remember, Kiota requires the base class to be fully generated before it starts injecting the derived one in the CodeDOM, and it's blocking. I haven't had time to look into that just yet but my money is on the GetGreatestGrandparent method derailing for some reason.

you were right, it was an ordering issue triggered by the discriminator mappings. Fixed in #1674