Add Korean message file for non-European language support

This PR adds a new Korean message file to the project, enabling support for non-European languages. The purpose of this addition is to showcase the international message functionality using a non-European language.
Changes Made:

Introduced a new Korean message file to the project to accommodate messages in Korean.
Translated messages from the English message file src/main/resources/messages/messages.properties into Korean in src/main/resources/messages/messages_ko.properties.

Translated the following English messages into Korean:

welcome -> 환영합니다
input is required -> 입력이 필요합니다
not found -> 찾을 수 없습니다
already exists -> 이미 존재합니다
All numbers must be entered -> 모두 숫자로 입력해야 합니다
Duplicate submissions are not allowed -> 중복 제출은 허용되지 않습니다
Invalid date -> 잘못된 날짜입니다

Validation:

Checked the translations using Google Translate: Google Translate Link.

Comparison with Spring's Default Messages:

In some cases, Spring's default messages and this translation may differ. While Spring's message for 'required' is "공백일 수 없습니다" (meaning "It cannot be blank"), this translation uses "입력이 필요합니다" (meaning "Input is needed"). Although conveying the same meaning, these translations are more friendly in the context of the spring-petclinic application.

Testing and Compatibility:

Tested compatibility with Chrome (116.0.5845.96) and Firefox (116.0.3).

Related Links:

Related PR: #631
Related Issue: #1299

Hi @dsyer,
Thank you for merging my PR into the main branch!
Your support and help have been invaluable to this project.
I'm delighted I could collaborate with all of you.
I express my gratitude and look forward to more opportunities to work together.


Add note on jaeger grpc storage compliance testing

Signed-off-by: Arunprasad Rajkumar<EMAIL_ADDRESS>

Heads up that I am reviewing this and making some changes. I shall be done this morning.


Let the implicit sizing handle displaying the button.

Description

Type of change

[x] Bug fix
[ ] New sample implementation
[ ] Sample viewer enhancement
[ ] Other enhancement

Platforms tested on:

[ ] Windows
[ ] Android
[ ] Linux
[x] macOS
[ ] iOS

Checklist

[ ] Runs and compiles on all active platforms as a standalone sample
[x] Runs and compiles in the sample viewer(s)
[x] Branch is up to date with the latest main/v.next
[x] All merge conflicts have been resolved
[x] Self-review of changes
[x] There are no warnings related to changes
[x] No unrelated changes have been made to any other code or project files
[x] Code is commented with correct formatting (CTRL+i)
[x] All variable and method names are camel case
[x] There is no leftover commented code
[x] Screenshots are correct size and display in description tab (500px by 500px, platform agnostic)
[ ] If adding a new sample, it is added to the sample viewer
[ ] Cherry-picked to Main branch (if applicable)

I guess explicit size was used to avoid issues on mobile devices. Did you try on it?

I'm not sure how that interaction works on mobile. I can't select the code to copy it, and i'm not sure how I would do a right click do bring up the menu either once the code is selected.

the logic currently is the menu will only display on mouse right click, so probably won't be able to trigger that on mobile
acceptedButtons: Qt.RightButton


Drupal web form for setup of app terms

We currently use a shared datasheet to handle the content types we should set-up on the FIT Count site.
This would be more efficient to handle in Drupal, with where users of a specific role could access and create the content for new countries.
We would need content types to represent a Habitat, Flower, Insect and Insect-Photo.
eg. a Flower content entry would be a page that allows entering:
•	habitat id
•	warehouse id
•	habitat type
•	sort order number
•	common name
•	scientific name
•	country codes - allow to select multiple countries this flower applies to.
These entries could then be programmatically imported into the app.
@kazlauskis @andrewvanbreda to discuss the requirement
@andrewvanbreda to implement

Hi @kazlauskis ,
Just having a quick look at what is written here.
I could perhaps write a very long post detailing this.
However I think without wanting to complicate matters to start with, if I ask just one question to begin (I think actually it touches on things we have already discussed)
This kind of new Drupal management area would have to store the changes to the data in the Warehouse.
Could your app ever be changed in such a way that it reads all the data it needs from the Warehouse?, or would you want an export spreadsheet of what is in the Warehouse to read in?
Let me know if you have any other thoughts regarding David's requirement.
After you respond I will post some of my other thoughts.

@andrewvanbreda I agree that full integration with the warehouse would be ideal, but I think we can start by getting the drupal form setup with an export option.  The issue text came from @kazlauskis btw!

@andrewvanbreda I agree with David, the species texts don't need to be linked to the warehouse. We currently use a spreadsheet to organise the texts but it would be much more streamlined to use some CMS like Drupal.

@DavidRoy @kazlauskis
OK. Although it doesn't actually necesserily make it easier to program because it means storing existing data structures in the Drupal database afresh.
I suppose what I could do is have a import/export button which just import/exports to external file without storing in Drupal.
Although this leaves the problem the Warehouse data structures that already exist will not get updated (although this is a problem we already have).
I actually still haven't got around to putting the page live on the FIT Count Site which allows app data editing, but the termlsits, countries do exist in the Warehouse.
It will be a problem if those don't get updated inline with changes people are making in Drupal.
The other thing to note is that by not linking to the Warehouse we miss out on the user interfaces we already have in Indicia for some of this, there is already Drupal taxon editing.
Feel free to comment further while I think.

@andrewvanbreda in that case, it sounds like you are part way through the warehouse link and it makes sense to complete that task?

Sorry, I wasn't following the current work on linking attributes to the warehouse, so I might need to understand how this is connected. I would suggest keeping it simple for this specific task and not linking any of this to the warehouse. We need a few custom content types that certain Drupal users can create and edit. These would contain texts, numbers and photos. The app could then use the standard Drupal API to fetch all of that programmatically.
https://www.youtube.com/watch?v=bb10eGT5YfE

This isn't particularly urgent. The two main problems we're trying to solve here are: a) the growing app datasheet can be error-prone to maintain centrally, and merging large sheets can be difficult. It is much better if country partners themselves could create the app content, and b) at the moment, species photos and texts are held separately, which can lead to problems like missing files or mismatched file names, wrong resolutions etc.
We currently use a separate document with instructions on how to compile the info. The ideal scenario, from my perspective, would be to allow a country partner to enter this information using web forms that would enforce the formatting, cross-link between different types (e.g. species-photo entities linked to the correct species entities), also show help texts and suggestions inline the forms. Ideally, the collected photos would be cropped and resized to fixed resolutions.

@kazlauskis
Thanks, that is very useful description, so we know exactly the aims of what we want the forms to do.

AVB: Note to myself so don't forget, once this screen is created and the terms are up to date, the following change can be made to the screen (accessed through the FIT Count website My Results page)
https://fitcount.ceh.ac.uk/enter-fit-count

Remove @disableFiltering=true option from the limit_termlists_and_species_to_selected_location extension.

Remove th @extraParams={"orderby":"term"} option from both the [smpAttr:1048] and [smpAttr:1050] attributes

Not sure if this is still a priority but keeping on the feature request list.

Hi @kitenetter
It is still on my task list, but is unlikely to be high up my task list anytime soon unless I am told otherwise.


No Proper CSS for backups for windows phone

I don't know whether the backups for android and windows phone is stored in different way, but the website is not working properly for windows phone WhatsApp backup

Can you please upload a sample chat of windows phone so that the app can be optimized for that ?

@rubyAce71697  actually whatsapp-reader is not for reading the backup files
You can email your chats to other people and it is used to read those emailed text files.
I will surely consider to include the backups too.

The file sent as email is different in case of windows phone (that format is not generic). I have forked your repo and made it compatible with the windows phone file. Also I have updated the UI (little bit) to reflect to the group chats in proper manner.  The code has to me made proper, after doing that I will send you the merge request and you can check before meaning.


nixos/gitit: remove

Description of changes
has not been evaluated (in module-list) since 122d5e497e6df55f4b2e5c353bb2224b79cfd7b0
Closes #167034
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/get-a-nixpkgs-haskell-package-but-compiled-with-non-default-options/34640/2


bugfix: IDiamonProxy autogen. Renaming function public methods to external when publicMethods is true (ea4e3f1)

Re-did the fix in #46  - thanks!

Thanks a lot, @hiddentao!
Now I can stop using the custom fork<EMAIL_ADDRESS>and start using the official version. Awesome!


Ml model ensembles

Description
Adds ability to pull an ML model ensemble from a remote specification at a URL and also to do inference with the ML model ensemble.
Developers certificate of origin

[X] I certify that this contribution is covered by the MIT license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

@kaminow  I would love to hear your thoughts here.

@kaminow probably ready for another look.


The  tag is a bit of a mess when it comes to support, so it would be quite a useful ad...

The <keygen> tag is a bit of a mess when it comes to support, so it would be quite a useful addition.

This issue was imported from Google Moderator
Moderator votes: +11
Issue added by David Mulder on 2014-10-30
To vote this issue up or down, simply include +1 or -1 in your comment.

Duplicate of #1194.

:+1: +1

+1

<keygen> is being deprecated: https://github.com/whatwg/html/pull/26

+1

-1. People shouldn't be using  at this stage for numerous issues listed in Chrome's intent to deprecate. Our internal data from web crawls show that in the wild it is almost never used.

+1 I think even if it's going to deprecated or never going to use is not a good cause to not showing the support of the type. People are using caniuse for checking how much support the type gets. Showing the deprecation status in the results of caniuse is the best way to let people know about the issue.

+1 for visibility.
https://webkit.org/blog/7093/release-notes-for-safari-technology-preview-19/:

Changed the keygen element to require 2048 or higher RSA key lengths

+1 because I just went to search for it, which means probably other people did too?
If it's deprecated in Chrome, that just makes it all the easier to track. :-P

+1

+1

+1

https://developer.mozilla.org/en-US/docs/Mozilla/Firefox/Releases/69#Removals_2:

The HTML <keygen> element has been removed from Firefox. It was deprecated some time ago, and its purpose has generally been supplanted by other technologies (bug 1315460).

Now available at https://caniuse.com/#feat=mdn-html_elements_keygen though looks like that data may need updating if Chrome's support was removed...


Sorting of Vue extensions is not optimal

Refs: Vue.js test tour (https://github.com/Microsoft/vscode/issues/23385)
There are some issues with vue.js extensions sorting:

there are 2 extensions for vue called vue that seem to be a fork of the other, but its not clear which to pick
Pines vetur extension which seems very popular is not to the top

cc @hamenon

In search we give the name of the extension the top most priority our telemetry shows that that's generally what users search based one. This is followed by the publisher name. In this case since vue matches the display name of the extension exactly and hence it shows up at the top. Pines vetur extension mentions vue only in the short description and in the tag, not the two most immediately noticeable entities on the extension page.
We do plan to introduce download count as a possible tie breaker for cases where two extensions have the same priority score. But that wouldn't help bring Pines extension to the top since that would have a lower score owing to the other extensions matching the name perfectly.

I remember there is a list of "recommended extensions" for each language. Wondering if it's possible to always show them first when the search term is for each language/framework, such as go/golang, python, vue, etc.

To keep the number of issues in our inbox on a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: We look at the number of votes the issue has received and the number of duplicates issues filed.
Thanks for your understanding. Happy coding!


Syntax Error running Vagrant Up (provisioning/roles/geerlingguy.solr/tasks/install.yml: line 27, column 3)

Issue Type

Bug Report / Support Request

Your Environment
vagrant@machine:~$ vagrant --version && echo "VirtualBox vboxmanage --version" && ansible --version
Vagrant 1.8.1
VirtualBox 5.0.24_Ubuntur108355
ansible <IP_ADDRESS>
config file = /etc/ansible/ansible.cfg
configured module search path = Default w/o overrides
Your OS

macOS 10.12.1

Full console output
https://gist.github.com/farah0808/32d1c451d55a09c5c618623900300d31
Summary
When trying to run 'vagrant up,' I get an error that no action was detected in task and the install.yml file, line 27, column 3 is the offending line (full error located in gist link) causing vagrant up and ansible to fail.
Attempting to set up a new box for Drupal 8. I have another (completely separate) box for Drupal 7 that works fine. I checked other issues. Upgraded to ansible <IP_ADDRESS>, made sure there were no extra spaces in any part of the file, still getting the same error. Please let me know what other info you need. Thanks in advance!

Two things to try—can you try upgrading Vagrant to 1.9.0 or 1.9.1, and also VirtualBox to 5.1.10?
Second, this usually happens if Ansible 2.1 or lower is running... I wonder if it's possible that Ansible isn't detected by Vagrant? Probably not, but worth checking—run which ansible and ansible --version again just to confirm you can see it.

vagrant@machine:~$ which ansible
/usr/bin/ansible
vagrant@machine:~$ ansible --version
ansible <IP_ADDRESS>
config file = /etc/ansible/ansible.cfg
configured module search path = Default w/o overrides
I'll try updating now. I was hesitant because my 7 box seems to go haywire when the versions are even a little newer. I'll give it a try

Closing for now - please let me know if this is still an issue after the upgrades!


fix failing tests

looks like an unstable key ordering

Fixed in #18 and improvements tracked in #19.


Improving Hessian computation

I've rewritten the inverse_hessian method, it is now cleaner and faster.
Instead of calculating the matrix element by element it is now computed row by row using only Tensorflow functions and one list comprehension. All Numpy code is removed, batching is preserved.
Even though this implementation computes the entire Hessian matrix and does not exploit the fact the matrix is symmetric it is still about 5x to 8x faster than the previous implementation. Running the test_ll notebook on Colab I see the time for one function call go down from 55s to around 8s on a Colab GPU node).
I've tried running the function with Autograph but this results in an error. Although that may also be related to minus_ll() which does not run with Autograph either.
If .hessians is added to GradientTape is a future TF release we might switch to using that instead.

I've added a fix for an edge case when the number of events in the last batch is not a multiple of the batch size. Secondly we can now use -1 as the last batch number. Thanks @CristianAntochi for spotting it.


Better defaults in the UI

For better user experience you might want to:

[x] make the first three checkboxes checked by default
[x] add a link to the page where Perma.cc API key can be generated, and
[ ] make the API key persist in user's browser's localstorage (if entered)

It looks like you have already fixed the first two after my email last night, so I checked them off.

Use https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage for the reference.

Yes, I already considered all of the three points you mentioned. Thanks.


Android app login forcing 2FA

I am unable to login to the android app through either the fdroid or google versions of the app. Every time I try to login it asks me for a MFA verification code.
I own a domain name from godaddy this is configured correctly with A records to the IP of my server.
If username = balloons, and domain URL = chat.test.server, my login info is written as "balloons/chat.test.server" and password is correct. If I try the login if like "balloons / chat.test.server" then I just get an "Error Please Try Again".
Multi Factor Authentication is not enabled for the user.
I am successfully able to access my server at its domain name and login through the web browser.
Here are some pictures of what is shown when attempting to login to android app.

Hmm.. I am not able to reproduce yet, but I will keep trying. Are you using a reverse proxy like nginx?
Are you able to create a test account on the demo site (https://databag.coredb.org) and login with the mobile app there?

I am using a nginx reverse proxy, but I have no problems accessing server through web browser, only on the android app.
I just tried making an account at databag.coredb.org. I can access this account through the web browser no problem but when I try to login from app with either "balloons/databag.coredb.org" or with "balloons/databag.coredb.org" I just get "Error please try again"

Sorry for the delay in response. I am seeing an issue with the demo server and not sure if it is related. I hope to have it resolved today.

I resolved my server configuration issue. Can you try and login to the the demo server again?
I just tested with fdroid and the login:
Username / Server: balloontest/databag.coredb.org
Password: test
Sorry for the trouble you are having.

Yes the android app works now for databag.coredb.org, what change did you make to the server configuration?
I tried looking through all the documentation for the project and I can't find any information on server configuration settings. I am using the docker installation. Is there a list of environment parameters that can be configured in the docker compose file?

The demo server runs on a manually configured system. My problem was I had pointed the server to a cert without attaching the full cert chain. I also run a similar setup to you for my personal use. Docker + nginx reverse proxy, which hasn't had any issues.
There is nothing really to configure when deploying the container. The only variables you can set are for admin password and development mode.
Are you able to view the console output of the container? If you see log messages when using the browser, but not the mobile, that suggests the request is being blocked before reaching the container, possible the reverse proxy.
If the server returns an HTTP 403 or 405, the client interprets that to mean a login was initiated and the MFA code is required but not provided; the MFA modal then pops up. Is there any special configuration with the reverse proxy around HTTP headers or source IP address that could trigger this?  Are you using the mobile app over wifi, on the same network as your PC browser? (random possibility: I found some posts describing 403s from their reverse proxy because their device IP was in a proxy file ip-bans.yml)
I am sorry I am not able to reproduce the issue with my local setup.

Your suggestions pointed me in the right direction, Thank you!
Problem was the user-agent in the http header is okhttp which my nginx was detecting as a bot, and since I had bot protection on it automatically denied the requests. If I get time over the next couple days I can scan through the code to find out where the okhttp user agent is initialized and make a PR for masking the user agent. Something like below code, here is link to github conversation about it  https://github.com/square/okhttp/issues/5399
val okHttpClient = OkHttpClient.Builder()
    .addNetworkInterceptor { chain ->
        val requestBuilder = chain.request().newBuilder()
            .removeHeader("User-Agent")  // otherwise addHeader not work.
            .addHeader("User-Agent", userAgent)

Awesome...thank you for solving this!


Any idea what might cause "Cannot call write() after write_eof()"?

Traceback (most recent call last):
File "/srv/locix/locix/websocket/server.py", line 184, in ping_handler
await websocket.ping()
File "/usr/local/lib/python3.5/dist-packages/websockets/protocol.py", line 433, in ping
yield from self.write_frame(OP_PING, data)
File "/usr/local/lib/python3.5/dist-packages/websockets/protocol.py", line 654, in write_frame
extensions=self.extensions,
File "/usr/local/lib/python3.5/dist-packages/websockets/framing.py", line 210, in write
writer(output.getvalue())
File "/usr/lib/python3.5/asyncio/streams.py", line 294, in write
self._transport.write(data)
File "/usr/lib/python3.5/asyncio/selector_events.py", line 687, in write
raise RuntimeError('Cannot call write() after write_eof()')

Hrm, that's weird, you should be getting an exception such as "InvalidState: Cannot write to a websocket in the CLOSED state" if websockets somehow tries to write a frame after (or while) terminating the connection.

So this seems pretty clearly to be because the socket was closed before the ping call was made? Hm, ok.

Maybe this is because in WebSocketCommonProtocol.close_connection(), there is a yield from self.wait_for_connection_lost() after self.writer.write_eof() is called and before self.writer.close() (which changes self.state) is called: https://github.com/aaugustin/websockets/blob/136d9bc9a917f7f5fdde4001e801b591ee1b4a0b/websockets/protocol.py#L724-L741

https://github.com/aaugustin/websockets/blob/136d9bc9a917f7f5fdde4001e801b591ee1b4a0b/websockets/protocol.py#L640-L655 doesn't yield between the check that the connection state is OPEN and the write which cause this exception.
This means we're calling write_eof while the connection state is still OPEN, which is wrong.
@jacksonofalltrades If the exception is easily reproducible, could you try adding a print websocket.state just before await websocket.ping() so we can confirm the connection state here?
Unfortunately the RFC doesn't cover error cases when it describes connection state handling. If memory serves, it says that the connection moves to the CLOSING state when a Close frame is received or sent and to the CLOSED state when the TCP connection is closed.
So what should the state be when the connection receives a TCP half-close from the remote side?No Close frame is involved so the state cannot be CLOSING. The TCP connection isn't closed yet so the state cannot be CLOSED.
We may need to add another flag to the WebSocketProtocol classes to say: "we're in an error situation, closing the TCP connection without a proper handshake, don't try to send frames".
These are the situations that end with a 1006 status code. There are hardcoded checks with this error code that could be handled more elegantly with such a flag: https://github.com/aaugustin/websockets/blob/136d9bc9a917f7f5fdde4001e801b591ee1b4a0b/websockets/protocol.py#L786-L789

This means we're calling write_eof while the connection state is still OPEN, which is wrong.

Would it be worth adding an assertion to detect this case?

Sure, I can add that. I'll let you know what happens.

State is OPEN when this happens. That seems especially odd.

What is causing write_eof() to be called? Is it close_connection() linked in my comment above? If so, what is causing that to be awaited?

@cjerdonek I wish I could tell you. I've put some logging into the /usr/lib/python3.5/asyncio/selector_events.py write_eof function, but I'm not seeing any that happen around the same time as the exception on websocket.ping.

I've added logging to print the traceback in write_eof so I can see what called it when it happens near the exception in this ticket. I'm seeing the traceback is "NoneType." How can there be no traceback when write_eof is called? Something must have called it.

Any advice on what to log here?

What does your logging code look like? Are you passing stack_info=True?
https://docs.python.org/3/library/logging.html#logging.Logger.debug

I'm wondering if my ping interval is too long and this problem is caused by the time since last ping having caused the connection to start closing on the client, without the server knowing yet. Is there any standard on how long browsers wait for a ping/keepalive before timing out a tcp / websocket connection? I've tried to do some searching but not finding anything on that. Thanks!

I'd use 30 seconds as the ping interval. TCP timeout is variable, I'd bet on 2 minutes in usual situations.
I don't have official sources to back these figures.

I'm also still unable to figure out why after a time my websocket server fails to reply to health checks quickly enough. I mention it here only because I wonder if it's related to the error in my initial post here using up cycles. AWS has not made debugging health check failures easy.

I've reduced my ping interval to 10 seconds and this is still happening with quite high frequency (the write_eof thing). It does seem to happen more, the more clients I have connected to the server.

Did you see my comment? https://github.com/aaugustin/websockets/issues/320#issuecomment-351330488

@cjerdonek Yes, the problem is I've been manually adding logging to the asyncio library itself to try to capture what's calling write_eof and my changes keep getting wiped when the server restarts. I've been just using import traceback and traceback.format_exc() in my logger call. I haven't tried stack_info=True.

OK, I just understood what @cjerdonek meant in this comment: https://github.com/aaugustin/websockets/issues/320#issuecomment-350991704
We're only relying on self.state for tracking the connection state. Unfortunately this isn't sufficiently accurate around state transitions.
There's a gap of time between the moment the connection becomes unusable in close_connection and the moment it's fully closed. Control yields to the event loop several times. The connection state remains OPEN all the while if there was no closing handshake.
(I was working on a similar situation a few minutes ago. write_close_frame sets the state to CLOSING before sending a Close frame in order to ensure that at most one close frame is sent, while the spec says that the state should be changed to CLOSING after receiving or sending a close frame. We're bending the spec slightly there to avoid such a gap.)
Perhaps we can improve the implementation of ensure_open() to fix this issue, for example by checking that transfer_data_task is still running. It must be done before reaching the point where where close_connection calls self.writer.write_eof().

This PR isn't particularly elegant (especially the test strongly coupled with the implementation) but I think it gets the job done without adding more state to the protocol class.


Can scCODE provide a complete list of DEgens?

Dear XZouProjects
Thank you for providing such convenient DE analysis software!
I have an inquiry for you about the details of the results. Can I get the full list of differential genes, including their fold changes and FDR values？
Gratefully

Hello, the De_results of scCODE is the full list of the DE gene results obtained by the top_optimal methods. For example, the demo code：
##run scCODE demo
library(scCODE)
###demo data

data1<-data1_sccode
data2<-data2_sccode
##"light", True or False, default as True, run scCODE in a light version which saves time. "top_ranked", the number of top-ranked strategies selected (5-10), default as 5
#results
results<-scCODE(data1,data2,light = TRUE,top_ranked=5)

I think the "results$DE_results$P_adjust" and "results$DE_results$logFC" are what you wanted, P_adjust is the fdr_corrected Pvalue.

Hi,jawz-fd
Thank you for your reply.
I may not have phrased my query clearly.  I found that the domain value of the final result is FDR<0.05.  I was wondering if you could provide the p-value option and the list of all DE genes.

I really appreciate it！

Hi, the package has been updated as v.<IP_ADDRESS>, the results on all genes applied by wilcox-test have been attached as results$results_wilcox.

Hi,
Thank you for pointing it out, we have fixed it now. Please retry and let us know if any problem appears.


Add Support for selecting any file from storage

So then it could be, files, images and videos?

yes. any file from storage like txt,docx,xlsx pdf, jpg, .mp4

Could be a nice extension, you can feel free to colaborate into it


The default displayed progress bar and the position after starting playback are inconsistent!

The playback progress bar will move its position after starting playback! I tried to find the problem, but failed! The issue with the playback progress bar at the bottom of the M3u8 with qualities example is very obvious! When the video content is within 15 seconds, the progress bar becomes a dot by dot interval.... Please help correct it!

Are you using latest version? The dot by dot should be fixed

Yes, I am using the latest version. I discovered this issue a long time ago, but I overlooked it. Now, some users have reported this issue, so I am trying to fix it! On the Android platform, you can see an example of running m3u8 directly!

Yes, I am using the latest version. I discovered this issue a long time ago, but I overlooked it. Now, some users have reported this issue, so I am trying to fix it! On the Android platform, you can see an example of running m3u8 directly!

Ok I get it now, I will see what causes the shifting

I suggest that the progress bar in the middle be displayed by default! Otherwise it would be a bit abrupt! After obtaining the video duration information, update the left and right duration information!

Can you solve the problem of abnormal progress bars caused by videos within 15 seconds in length?

Yes I think that's the cause of the shifting it self since that is the only thing not rendered in the above video, you can make a pr if you would like, and if not I will try to do it next weekend

I'm trying to fix it!

new release is fixed in it


When using defineOgImage gives type error as not found

Details
when using defineOgImage i get a typescript error saying not found and also which ever part of my app I use this on just goes to a 500 error page. I am using the Nuxt-SEO-kit module which includes nuxt-og-image so not sure if this might be an issue?

Ah thank you, that seems to have solved the typescript and 500 error but when I try and register a custom template I just get an error in the console and a similar one in the playgound:
[nuxt] [request error] [unhandled] [404] [GET] "/__nuxt_island/ibwOg_RULYAZWeMR?props=%7B%22options%22:%22%7B%7D%22,%22path%22:%22/%22,%22timestamp%22:%221693344064559%22,%22scale%22:%221%22,%22mode%22:%22light%22,%22provider%22:%22satori%22,%22component%22:%22ibwOg%22,%22width%22:1200,%22height%22:630,%22cache%22:true,%22cacheTtl%22:86400000,%22title%22:%22Welcome+to+my+site!%22,%22description%22:%22Welcome+to+Indie+Beer+Week+Southampton+6th+-+15th+October+2023.+Sign+up+to+be+the+first+to+find+out+updates+on+the+event+and+when+the+full+line+up+is+released%22,%22requestOrigin%22:%22http://localhost:3000/%22%7D": 404 Island component not found: ibwOg
  at async $fetch2 (./node_modules/ofetch/dist/shared/ofetch.00501375.mjs:261:15)
  at <anonymous> (./node_modules/nuxt-og-image/dist/runtime/nitro/routes/html.mjs:39:18)
  at async Object.handler (./node_modules/h3/dist/index.mjs:1630:19)
  at async Server.toNodeHandle (./node_modules/h3/dist/index.mjs:1840:7)```

I have the custom component in the `components/OgImage` folder and just tried the example from the doc and also copying the fallback template too and trying that.

Also should mention using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

You can try out 2.0.24 and see if you have better success, otherwise, I'll need a reproduction :pray:
(see https://stackblitz.com/edit/nuxt-starter-pxs3wk?file=pages/index.vue)

Massive thanks on this, really sorry as not been able to reproduce the original problem and some how go it working locally but guessing it had something to do with where I was defining the og-image and the seo meta data. This can probably be closed as more than likely something already documented. I think my issue which led me down a rabbit hole was with custom tailwind classes not being puled through:
reproduction of issue is here
Am aware this is in the docs that tailwind support is experimental but also found if all the props are not defined it seems to lead to the template not being rendered and a blank screen in the preview. Not sure if this is desired/expected behaviour or not but normal vue components will still render if the props are not passed (see the test page).

scrap the last bit, nothing appeared to be coming through as the txt was white 🙈. Can definitely close now

Haha all good, glad you could solve it!


Adding file to repo

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


add animation on search bar and background

the search bar is the first thing that appears it looks a little dull right now, adding some animation and design will make it look better

Hey , I am participant in GSSOC'21 . Can you please assign this to me under GSSOC'21?

Deadline crossed, but I think you have completed your work, due to some conflicts PR not merged
So Deadline extended 13 March,2021 (11:59 PM IST)
Do make PR with the updated code to avoid conflicts

Sir i have made some changes that will take the content under the search bar and the calculator that was going to extreme left i corrected that too then i tried sending the pull request again and i was not able to do so

Sir also can u tell me what other changes i should make to avoid the conflicts in PR thank you

Just keep your code updated
Then probably no conflicts occur.
Today some PR's got merged, so kindly update your code.

Okh sir

Deadline is near

sir i have tried several times now its giving error saying "please make sure you have the correct access rights and repository exists".

Have you done with your work?

Yes sir

Try to make PR without CLI, may be it works

Okh sir

Sir still not working and now its showing nothing to commit working tree clean

Ping a message in our discord channel

I would like to work on this.

Hi
I would like add animation on search bar and background,
Would you please assign me to do this.


Pretrained model for calvin

Great work! I'm very interested in your method, and was wondering will you opensource your pretrained model?

Hi, and thank you very much for your interest! Please all find all model checkpoints here: https://github.com/ezhang7423/hulc-data/tree/5ca225c474e049e5b89c32590b3d2b4efcf60e26


Delete key on keyboard doesn't work when entering canvas size, and other keyboard shortcuts don't work

Describe the bug
A clear and concise description of what the bug is.
When entering the canvas size, pressing the delete key on the keyboard doesn't delete the number in front of the cursor.
To Reproduce
Steps to reproduce the behavior:

Go to File
Click on New
Click on Width box
Try putting the cursor one character back, then press the delete key on your keyboard

Expected behavior
The delete key should delete the character in front of the cursor.

OS: Fedora Linux 37
Browser: Firefox

Also, end and home don't work. Home should bring you to the start, and end should bring you to the end, but neither work.

it's not really a bug, instead the case isn't handled, in the implementation, only Arrow Left/Right & Backspace are handled, so keys like Delete, Home & End are not handled.

@mak448a PR #38 fixed this issue, please test it again and close the issue if resolved.

Thank you!


While saving other scene, if you've created and drawn a new TileMap, half it's blocks dissapear.

Godot version: 3.1.1
OS/device including version: Windows 10 v1809
Intel HD Graphics, GLES 2.
Issue description:
I had created a new TileMap node, and a new Resource to go with it, and started drawing a map. After the map was done, before saving the scene where the new TileMap was on, i traveled onto another, changed a sprite and saved, to then see, in a interval of 2 blocks, they had dissapeared. Not a visual or spacing bug, they dissapeared and the only and best way i found to fix it was to redraw the blocks that went missing.

Steps to reproduce:
1. Create a new TileMap and choose a Resource for it.
2. Draw anything in the map.
3. Change scenes.
4. Save the other scene you've just navigated to.
5. Return to the scene with the TileMap to witness the chaos.
Minimal reproduction project:
TileMapBug.zip

Duplicate of #29312.


Chat message badges

If I'm in the chat pane with someone and a new message comes in, I still get a numbered badge appearing (where technically it should only be seen if I haven't read the message).
Ping @jjeffryes and @cpacia

Fixed


Plugin API + Access Rules : return more informative message

I just had very hard time debugging an issue due to IMHO misleading error messages. I had put an access rule preventing modifications on a table depending of the value of one of its fields. When trying to add records via BulkAddRecords, despite having set a message with the rule, this message didn’t appear (neither bottom-right, nor in the console or on server logs). The only messages I got are shown below.
On the client:
Error: Cannot read properties of undefined (reading '0')

On the server:
grist_1      | 2024-06-04 16:00:08.555 - warn: Sharing Unable to apply action... TypeError: Cannot read properties of undefined (reading '0')
grist_1      |     at getActionUndoInfoWithoutClient (/grist/_build/app/server/lib/ActionHistory.js:93:37)
grist_1      |     at asActionGroup (/grist/_build/app/server/lib/ActionHistory.js:36:74)
grist_1      |     at Sharing._doApplyUserActions (/grist/_build/app/server/lib/Sharing.js:275:67)
grist_1      |     at async Sharing._applyLocalAction (/grist/_build/app/server/lib/Sharing.js:89:25)
grist_1      |     at async WorkCoordinator._tryNextStep (/grist/_build/app/server/lib/WorkCoordinator.js:45:21) access=owners, userId=6<EMAIL_ADDRESS>age=628, org=myawesomeorganization, clientId=3d70d978479ed860, counter=0, docId=7kHK1rpTHsestxGJwJoQ7R
grist_1      | 2024-06-04 16:00:08.555 - warn: Client Responding to method applyUserActions with error: TypeError: Cannot read properties of undefined (reading '0')
grist_1      |     at getActionUndoInfoWithoutClient (/grist/_build/app/server/lib/ActionHistory.js:93:37)
grist_1      |     at asActionGroup (/grist/_build/app/server/lib/ActionHistory.js:36:74)
grist_1      |     at Sharing._doApplyUserActions (/grist/_build/app/server/lib/Sharing.js:275:67)
grist_1      |     at async Sharing._applyLocalAction (/grist/_build/app/server/lib/Sharing.js:89:25)
grist_1      |     at async WorkCoordinator._tryNextStep (/grist/_build/app/server/lib/WorkCoordinator.js:45:21<EMAIL_ADDRESS>userId=6, age=628, org=myawesomeorganization, clientId=3d70d978479ed860, counter=0
grist_1      | 2024-06-04 16:00:08.555 - warn: Client responding to #40 ERROR Cannot read properties of undefined (reading '0'<EMAIL_ADDRESS>userId=6, age=628, org=myawesomeorganization, clientId=3d70d978479ed860, counter=0

Thanks for reporting this. That kind of error message smells like a bug. If it isn't too sensitive, would it be possible to get some screenshots, or a small doc replicating the essentials of your setup?

Sure. What would be more useful? Dockerfile, definitions of tables, anything else?

I'd be focused on steps needed to replicate the problem. So anything that narrows down the access rules to try, which in turn may need definition of tables. Also, when you are adding the records via BulkAddRecords, what is initiating that action, is it via API or on the web client, just so we can do the same thing.
It is a burden to prepare that, I know. From what you already reported, I could go to getActionUndoInfoWithoutClient and try to think of improvements to make. The problem would be to know if they have been effective.


Do not reconfigure for cloud backend

Cloud backend doesn't like -reconfigure argument and there is proper condition in the code but somehow it is hardcoded in array initialization.

Hi @dex4er, Have you tried the is_backend_terraform_cloud option introduced in v0.3.2+? It allows you to skip the -reconfigure flag when switching remote state backend to the Terraform Cloud. If it doesn't help you, please open a new bug report with a reproduction case.
https://github.com/minamijoyo/tfmigrate/tree/v0.3.4#is_backend_terraform_cloud

@minamijoyo of course I tried is_backend_terraform_cloud and it can't work correctly because it is ignored then tfmigrate dies because of terraform init -reconfigure additional option.
See: this condition https://github.com/minamijoyo/tfmigrate/blob/master/tfexec/terraform.go#L262 adds 2nd -reconfigure to the array defined in https://github.com/minamijoyo/tfmigrate/blob/master/tfexec/terraform.go#L257
In other words:
Without this patch, tfmigrate always breaks with
Error: Invalid command-line option

The -reconfigure option is for in-place reconfiguration of state backends
only, and is not needed when changing Terraform Cloud settings.

With my patch tfmigrate breaks with this error only when is_backend_terraform_cloud = false and works correctly when is_backend_terraform_cloud = true.

@dex4er Thank you for your patch! It looks good but acceptance tests are failing due to an unrelated timeout issue. I tried to rerun but still failing.
I fixed the timeout issue in #100. Could you rebase this branch on the latest master?

@minamijoyo please redo tests


Button should respect polymorphic as prop

Describe your changes here.
Closes https://github.com/primer/react/issues/2118
It seems like LinkButton was removed under the expectation that Button would act in apolymorphic way and handle styling links via the As prop, however it seems around the same time that functionality was blocked, since any as prop passed to button is overridden by the default prop here see details
I've updated the types to account for this, and updated the test to avoid skipping the as prop component test.
This should allow projects to avoid using the now undocumented LinkButton
Screenshots
n/a
Merge checklist

[x] Added/updated tests
[x] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Will take a look at ci here a bit later, if the changes seems reasonable (or at least the spirit of this change)


Bug - Broken Travis Builds

I see that htmlproofer gem has been updated and therefore was causing some issues with the checks on travis builds.
In order to fix:

updated some commands for the htmlproofer to work, using htmlproofer instead of htmlproof as well as removed the --ignore-script-embeds option now as it is deprecated.  By default script embeds are ignored unless specifically specified with the new htmlproofer option
also updated rvm version to ensure latest environment and gems are used.

Is it possible to specify an explicit version number of the gems we install so that we can't break if they update out from under us?

@mtlynch good point.  I have specified the latest version for htmlproofer now, so any future bundle installs will make sure to use version 3.0.2.

LGTM


Can't get to onpaymentauthorized

Question
Expected behaviour
Use the Pay with Touch ID and receive the data from the onpaymentauthorized event
Actual behaviour
I'm using {staticsubdomain}.ngrok.io to test on my local environment, I've registered the domain in the Developers website and I believe Everything is setup properly.
The Apple Pay button shows up on my iPhone
The payment bar pops up and the onvalidatemerchant, onpaymentmethodselected and other methods are getting hit and I get to the point where I can use my finger print on the Pay with Touch ID using a sandbox account and a test credit card.
However, when I use my finger print the circle spins for a second and then says Payment not Completed and it doesn't hit onpaymentauthorized.
It works perfect with the PolyFill on my Windows machine, but not without on my iPhone. I bought a used mac so I could inspect and debug in Safari; hoping to find some sort of message from the ApplePayJS in the browser, but nothing.
Searched the internet added a ton of logging in the JS can't seem to find anything to point me in the right direction. I've been working on this for a few months.  Hoping you might be able to point me in the right direction.
Cheers!

Hi Jason - off  the top of my head, here's the first few things I'd check if you haven't already:

Apple has verified you control the domain after you added it by you placing the domain verification file on your server at the right URL.
TLS 1.2 is enabled (example).
Your application is correctly loading the client certificate from disk/store and attaching it to requests using HttpClient to get the metadata.
Your website's certificate is publicly trusted (e.g. not self-signed).
The hostname you're using on the device matches exactly any registered in the Apple setup and that you're loading it over HTTPS.

The merchant certificate I use to test this sample integration has expired so I can't check right now that it's all still working correctly, but is something I can pick up next week if necessary to validate no problems on our end with the sample either.

Martin,
Thank you so much for your response. I reviewed everything and it all looked good, but still wasn't working.

[x] Apple has verified you control the domain after you added it by you placing the domain verification file on your server at the right URL.
[x]  TLS 1.2 is enabled.
[x] Your application is correctly loading the client certificate from disk/store and attaching it to requests using HttpClient to get the metadata.
[x] Your website's certificate is publicly trusted (e.g. not self-signed).
[x] The hostname you're using on the device matches exactly any registered in the Apple setup and that you're loading it over HTTPS.

However, I uploaded it to one of our servers and tested it out and it worked perfect.  As far as I can tell you can not use/test this behind an ngrok proxy (although maybe I just don't have it configured correctly).
If someone does know how to successfully test this behind ngrok, it would make development much easier and would love to hear how you did it!  Here are the two ngrok commands I used to test the site behind:

ngrok http https://localhost:44399 -host-header="localhost:52623" -subdomain=openpath-apple-pay
ngrok http 52623 -host-header="localhost:52623" -subdomain=openpath-apple-pay -bind-tls=true

@martincostello This is an awesome example C# framework to get started off of, thank you!

Thanks for posting the eventual solution back. Glad you got it working in the end!

I had the same problem but couldn't solve it


Use xorg.conf.d files for mouse acceleration rather than dynamic config in lxdm files

Test with xset -q


Django admin form manytomany fields not saved. how to save

I am using dango-cms ==3.2.3, django = 1.8.10
I have created models.py with many-to-many field. When I add any record in it then it is not show any warning and save record perfectly but when I open that record so it not show many-to-many selected value so I thing many-to-many field is not saved.
my models.py is
please refer this link for my models.py code
https://ghostbin.com/paste/2v7js
How can I solve this issue??

@kartikdanidhariya
Hello, this looks like an issue with the multilanguage framework you're using.
Is it django-parler?

i am using django-hvad

@kartikdanidhariya
Ok, mind opening a ticket in https://github.com/KristianOellegaard/django-hvad/issues?
I'll jump on there to see if I can help.

@kartikdanidhariya
Seems like this is not supported on hvad (m2m fields as translatable fields).
https://github.com/KristianOellegaard/django-hvad/issues/284

i created issue in django-hvad for that
https://github.com/KristianOellegaard/django-hvad/issues/296


t2t notebook attention visualization broken

Description
When running https://colab.research.google.com/github/tensorflow/tensor2tensor/blob/master/tensor2tensor/notebooks/hello_t2t.ipynb
code for en2de translation and visualizing all attentions heads input-output view attentions are nonsensical:  for each output word all 8 attention heads to the first input word "The". Clearly, that is not right.
The prerendered visualization shows a very different attentions that do not match those produced by code.
I got this issue by simply running the notebook.

Can you try installing an older version of T2T in Colab (e.g. pip install tensor2tensor==1.4.0) and check if there is the same problem?
I have never seen this kind of problem, but I remember the visualization code was always buggy: for some sentences there were attention positions with no tokens shown or the other way round. I was just lazy to inspect it properly and report an issue.
Note that the enc-dec (input-output) attention may not correspond to what we would expect as alignment (it may be e.g. "shifted by one", cf. #898) - thus it is difficult to distinguish these unexpected attentions from visualization errors.

Thank you for your answer.
After installing pip install tensor2tensor==1.4.0 code fails to import trainer_lib:
ImportErrorTraceback (most recent call last)
 in ()
8 from tensor2tensor import problems
9 from tensor2tensor.layers import common_layers
---> 10 from tensor2tensor.utils import trainer_lib
11 from tensor2tensor.utils import t2t_model
12 from tensor2tensor.utils import registry
ImportError: cannot import name trainer_lib

Any other suggestions?

+1
Not sure if related, but one problem I am seeing is that attention is not using the decode_hparams somehow and the translations are different when I use t2t-decoder vs. the one in the attention visualization.


Remove install spring-sdk command in travis script for it has been released

Remove install spring-sdk command in travis script for it has been released

Codecov Report

Merging #880 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #880   +/-   ##
==========================================
  Coverage      44.46%   44.46%
  Complexity       766      766
==========================================
  Files            135      135
  Lines           5184     5184
  Branches         748      748
==========================================
  Hits            2305     2305
  Misses          2705     2705
  Partials         174      174

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 fa442c7...67f5093. Read the comment docs.


Two-player isolation?

Has anyone tried using a remote IP address of another player to allow two-man isolation?
From what I understand, Destiny uses one member of a fireteam (or pubic area) as "the host" for anyone else in the instance. You never get to see if that's you or not.
I'm thinking if someone goes into an area first with the rules active it might be possible to allow someone else to join if their IP address is added as exception.
Thoughts?

Because it'd be interesting trying to play gambit or crucible as  matchmade activity, but limited to only specific IP address participants.  As private matches do not necessarily provide the same gameplay results.

The challenge, of course, would be having to get the IP addresses of your desired fellow participants.  They'd have to use a 'what is my IP' type of service to get their address and you'd have to manually integrated those addresses into the script.  Not impossible, and using an array it'd be possible to do it without excessive editing.  Loop over an array of input IPs to create rules (with the IP integrated as part of the rulename).  Do that once at the start, and again when you stop the script, to remove them.

The challenge, of course, would be having to get the IP addresses of your desired fellow participants. They'd have to use a 'what is my IP' type of service to get their address and you'd have to manually integrated those addresses into the script. Not impossible, and using an array it'd be possible to do it without excessive editing. Loop over an array of input IPs to create rules (with the IP integrated as part of the rulename). Do that once at the start, and again when you stop the script, to remove them.

Hey is it possible you can teach me how to do it? or is it possible to do it with local network? or maybe explain more of how it can be achieved, i have background in coding and would like to give it a shot, if you can explain more of how to do it


readme: add Renesas R-Car (Gen 3) M3 Starter Kit board

As requested by @gunnarx I have gone ahead and created this PR to add the recently merged Renesas R-Car (Gen 3) M3 Starter Kit board to the readme.

@slawr Thanks. Before we merge this, I'd want a concern of mine about hashes in README resolved first: https://github.com/GENIVI/meta-genivi-dev/pull/76#pullrequestreview-29144190

@zeenix I agree that SHA lists could be somewhat painful to maintain as a readme update would follow every submodule update. Although of course here I'm just following an existing convention.
It's just a guess but I suspect it goes back to a time before submodules were used. It was likely following the convention in meta-ivi of listing the SHAs that are known to work.
It seems to me that your comment relates more to a separate change request, i.e. it should be a separate change. Perhaps after this.

@zeenix I agree that SHA lists could be somewhat painful to maintain as a readme update would follow every submodule update. Although of course here I'm just following an existing convention.

Not quite that bad because we don't maintain it for the GDP project since it would be both painful and redundant. But whenever dependencies to meta-genivi-dev change then the README needs to change.  So there's a logic behind this.  But I still don't like it.  Moving the rest of the discussion to #76 where it belongs.

Not quite that bad because we don't maintain it for the GDP project repository (with submodules) since it would be both painful and redundant. But whenever dependencies to meta-genivi-dev change then the README needs to change. So there's a logic behind this. But I still don't like it. Moving the rest of the discussion to #76 where it belongs.

Yes your right. I was mixing up projects when commenting. Note to self don't try to comment quickly when your on holiday :)

since @chbae didn't act in time, I'll merge this and let him worry about #76 :)


Add "recover" option to support recovery after soft deletion

Description:
It would be helpful to have an option to recover entities that have been soft-deleted. This feature would enable users to restore entities to their original state, rather than permanently losing them due to soft deletion.
Feature Request:

Implement a method to recover soft-deleted entities.
Ensure that the method updates the appropriate flags/fields to mark the entity as active.
Include documentation and examples for using this feature.

Benefits:

Provides users with the ability to recover entities that may have been accidentally soft-deleted.
Increases flexibility in managing soft-deleted data.
Reduces risk of permanent data loss.

@Char2sGu Okay. So below code is working to recover it, but still  not sure how to recover deletd relationships.
// Reset the soft-delete flag to "recover" the entity
wrap(entity as BaseEntity).assign({ deletedAt: null });

// Persist all changes to ensure recovery is complete
await this.mikroOrmRepository.persistAndFlush(entity);

Hey @rahul-rocket sorry for the delay. I am a little busy at the moment, but this has been added to my todo. Will look into it in a few days

On second thought, I should stop piling up my to-do list 😝
Currently, when an entity is soft-deleted, only this specific entity's deletedAt property will be updated. The relations will not be updated (but the relations will not appear in queries because of the filter). Therefore, if I didn't confuse myself, simply query the deleted entity with the filters disabled, and set deletedAt back to its initial value should recover the entity.
I'm not pretty sure if this should be added as a new method since this process seems pretty straightforward. Do you have some suggestions on the API you expect to see? @rahul-rocket

@Char2sGu We are building amazing platform where we used 2 ORMs (TypeORM & MikroORM) together. So TypeORM provide additional method to recover it.
i.e. recover<T extends DeepPartial<Entity>>(entity: T, options?: SaveOptions): Promise<T & Entity>;
So TypeORM can also soft deleted relations too if we passed them with Entity. But not sure how Mikro ORM (this package) doing it.
Note: Anyway i have fixed this issue by some other hacked. But check below link and let me know if you have any suggestion for me.
https://github.com/ever-co/ever-gauzy/blob/develop/packages/core/src/core/crud/crud.service.ts#L684

On second thought, I should stop piling up my to-do list 😝
Currently, when an entity is soft-deleted, only this specific entity's deletedAt property will be updated. The relations will not be updated (but the relations will not appear in queries because of the filter). Therefore, if I didn't confuse myself, simply query the deleted entity with the filters disabled, and set deletedAt back to its initial value should recover the entity.
I'm not pretty sure if this should be added as a new method since this process seems pretty straightforward. Do you have some suggestions on the API you expect to see? @rahul-rocket

My suggestion is that we should add some additional method Recover to entity same as TypeORM if it is possible.

On second thought, I should stop piling up my to-do list 😝
Currently, when an entity is soft-deleted, only this specific entity's deletedAt property will be updated. The relations will not be updated (but the relations will not appear in queries because of the filter). Therefore, if I didn't confuse myself, simply query the deleted entity with the filters disabled, and set deletedAt back to its initial value should recover the entity.
I'm not pretty sure if this should be added as a new method since this process seems pretty straightforward. Do you have some suggestions on the API you expect to see? @rahul-rocket

this is not true to be honest. at least for me it deletes the loaded @OneToMany relationships for example. is this expected?

this is not true to be honest. at least for me it deletes the loaded @OneToMany relationships for example. is this expected?

This is not expected. MikroORM might have changed its internal strategies.
Ur always welcome to create PRs for tests and updates.


Limit public access for Azure Cache for Redis instances

Rule request
Suggested rule change
Limit public network access when private endpoints are configured by ensuring publicNetworkAccess is Disabled.
Applies to the following
The rule applies to the following:

Resource type: Microsoft.Cache/Redis

Additional context
Azure Cache for Redis with Azure Private Link

@BernieWhite I think this rule could also work for #934, given if publicNetworkAccess is Disabled then private endpoints are exclusively used.

@ArmaanMcleod My understanding of this property is that if private endpoints are not configured, publicNetworkAccess can not be set. It is automatically set to Disabled when a private endpoint is created. However once a private endpoint is created the publicNetworkAccess can be forced to Enabled.
But could be wrong.
For this reason I think we still need to check both.

@BernieWhite Did a quick test and if you force public access to be enabled then the private endpoint is removed if public access was previously disabled. Seems that a Azure.Redis.PublicNetworkAccess rule is enough for both cases.

@ArmaanMcleod So you are thinking that we can close #934 as duplicate?

@BernieWhite I think that should be fine. Can reopen it if I find that there is a difference between the rules.

@ArmaanMcleod Sure. I'm good with that.


Should I change the repo name?

gogs is a famous project name.
Should I change the repo name?
If you think the name should be changed, please give me some advice. or you think it's ok

I will confirm it before version 0.2.0, because maybe not many people will be using it at that time

fork 楼主的仓库发现跟gogs重复了...


Create postcss.config.js

This is missing and breaks some builds.

Can you share with me the cases?

:shipit:

@coderdiaz We're using postcss in our builds and when creating a production build it would fail with the following:
ERROR in ../node_modules/vue-status-indicator/styles.css
Module build failed: ModuleBuildError: Module build failed: Error: No PostCSS Config found in: /usr/src/node_modules/vue-status-indicator
at config.load.then (/usr/src/node_modules/postcss-load-config/src/index.js:55:15)
at runLoaders (/usr/src/node_modules/webpack/lib/NormalModule.js:195:19)
at /usr/src/node_modules/loader-runner/lib/LoaderRunner.js:364:11
at /usr/src/node_modules/loader-runner/lib/LoaderRunner.js:230:18
at context.callback (/usr/src/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
at Promise.resolve.then.then.catch (/usr/src/node_modules/postcss-loader/lib/index.js:194:71)
After adding the file it was all resolved.
do you mind releasing a new version so we can switch back to your fork from ours?
Thanks for merging!

New release available 1.1.1 🔥


CookieJar - It doesn't exits method removeAll

Hi!
In the new version of the library okHttp 3.0.1 it doesn't exits method removeAll in the new interface CookieJar. In other versiones I had a cookieHandler and I could do a call to method removeAll to remove all cookies of the http client. But now it doesn't existe any similar method.
How Can I do it?
The old code:
this.cookieManager = new CookieManager();
this.cookieManager.setCookiePolicy(CookiePolicy.ACCEPT_ALL);
this.clientOkHttp.setCookieHandler(cookieManager);
New code:
this.cookieManager = this.clientOkHttp.cookieJar();
this.cookieManager.......
Thanks a lot. Regards

You can continue to use a CookieManager if you use the okhttp-urlconnection module and the JavaNetCookieJar class. We don’t yet have a concrete implementation of CookieJar.

Hi,
Could you say me what is the code to enable CookieManager en OkHttp3?
Thanks a lot.

    client = client.newBuilder()
        .cookieJar(new JavaNetCookieJar(cookieManager))
        .build();

I get

Cannot resolve symbol JavaNetCookieJar

isnt it in okhttp3 package?

It's in the okhttp3-urlconnection artifact.
On Mon, May 2, 2016 at 2:17 PM GPack1<EMAIL_ADDRESS>wrote:

I get
Cannot resolve symbol JavaNetCookieJar
isnt it in okhttp3 package?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/square/okhttp/issues/2249#issuecomment-216316509

thks, it is not very clear for me the migration from okhttp 2 to 3


Report HostInterface discovery status

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
When adding a number of hosts at the same time, there is nothing reported whether new hosts  have been configured or not.
Describe the solution you'd like
A clear and concise description of what you want to happen.
There should have a status to report how many host is available, how many of them has secondary interfaces, and how many of them already processed for the CIDR.
Here is the proposed status.
status:
    discovery:
      cidrProcessed: 100
      existDaemon: 100
      infoAvailable: 100

With the above status, we can track the progress of configuration when add a number of nodes as follow.
# kubectl get multinicnetwork -o custom-columns=NAME:.metadata.name,Total:.status.discovery.existDaemon,Available:.status.discovery.infoAvailable,Processed:.status.discovery.cidrProcessed -w
NAME               Total   Available   Processed
multi-nic-sample   1       1           1
multi-nic-sample   1       1           2
multi-nic-sample   87      2           2
multi-nic-sample   91      3           2
multi-nic-sample   91      3           3
multi-nic-sample   92      4           3
multi-nic-sample   95      4           3
multi-nic-sample   99      5           2
multi-nic-sample   99      5           5
multi-nic-sample   99      6           5
multi-nic-sample   99      7           5
multi-nic-sample   99      8           5
multi-nic-sample   99      8           9
multi-nic-sample   99      9           9
multi-nic-sample   99      10          9
multi-nic-sample   99      11          9
multi-nic-sample   99      12          9
multi-nic-sample   99      12          14
multi-nic-sample   99      14          14
multi-nic-sample   99      15          14
multi-nic-sample   99      16          14
multi-nic-sample   99      17          14
multi-nic-sample   99      17          24
multi-nic-sample   99      25          24
multi-nic-sample   99      25          24
multi-nic-sample   99      26          24
multi-nic-sample   99      27          24
multi-nic-sample   99      28          24
multi-nic-sample   99      29          24
multi-nic-sample   99      29          42
multi-nic-sample   100     43          42
multi-nic-sample   100     44          42
multi-nic-sample   100     45          42
multi-nic-sample   100     46          42
multi-nic-sample   100     46          100
multi-nic-sample   100     100         100

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Signed-off-by: Sunyanan Choochotkaew<EMAIL_ADDRESS>

Done by https://github.com/foundation-model-stack/multi-nic-cni/pull/50.


Help with extension on example room_assignment.ipynb

Apologies if this is not the right forum for this question, will close Issue if so.
I am attempting a small extension on the room_assignment problem where there is also a numeric 'utility' to each request for room assignment. The goal would be to maximize utility while keeping the no-overlap constraints.
For example, input data could be:
start | end | utility_room_1 | utility_room_2 | utility_room_3
1	52	6399.9	0.00	0.00
1	23	99.0	799.90	0.00
1	23	0.0	1494.90	0.00
23	24	0.0	0.00	0.00
23	24	0.0	0.00	0.00
24	27	0.0	0.00	199.90
24	27	0.0	100.00	0.00
29	47	0.0	0.00	99.99
29	47	0.0	1599.95	199.88
Where start and end are days of the year for example, utility is calculated with some utility function that helps requests rank room assignment, and the task is to maximize utility on room assignment without allowing for overlaps on start and end days.
Thank you in advance for the guidance

Hi Juan Pablo,
Yes, exactly.
Slightly more elegantly as
for val, util in enumerate([row.utility_room_0, row.utility_room_1, row.utility_room_2]):
  obj_list.append( (requestvars[idx]==val)* util

I was unsure how fluent you were in NNs. We actually do exactly this in our 'visual sudoku' perception-based solving, if which you find a notebook (the objective is added somewhere in the middle), here:
https://github.com/CPMpy/cpmpy/blob/master/examples/advanced/visual_sudoku.ipynb
Happy you appreciate the library, it is exactly made for these kinds of things. I'd be interested to hear more about your research too, feel free to drop me an email<EMAIL_ADDRESS>


feat: add debian 12 build file

Also made the dockerignore file just ignore the target folder except for the binaries. Which will allow us to have a docker image that does the building.
The debian12 recipe uses cargo chef to have better layer caching, as well as debian:bookworm-slim as the base image for runtime, which is slightly bigger than distroless.
However, it does not build for both amd64 and aarch yet
Should help for #509

Bah, this is just the first step, because we need the binary  attached to our releases as well. I'll extend this PR with the work needed to have an openssl 3.0 compatible binary.


when set "python.autoComplete.addBrackets": true, in the auto-completed brackets, the IntelliSense didn't work

Environment data

VS Code version: 1.29
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: win10
Python version (& distribution if applicable, e.g. Anaconda): 3.6.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Relevant/affected Python packages and their versions: XXX

Expected behaviour
so I want it like pycharm ,like

Actual behaviour
1.
when I "pri"+Tab,  it's addBrackets
then I want print the day,
the 'day'  likes a string, the IntelliSense  didn't work ,

I can repro this with "python.jediEnabled": true in my settings.json file. However, when I enable the Microsoft Python Language Server, the issue is resolved.
Workaround: Enable the Microsoft Python Language Server by setting "python.jediEnabled": false in your settings.json file. (See #3977 to find out how to get the latest stable version).

thanks, I tried set "python.jediEnabled": false, but the auto-completed brackets didn't work now, now the version is 1.33

I can repro this with "python.jediEnabled": true in my settings.json file. However, when I enable the Microsoft Python Language Server, the issue is resolved.
Workaround: Enable the Microsoft Python Language Server by setting "python.jediEnabled": false in your settings.json file. (See #3977 to find out how to get the latest stable version).

thanks, I tried set "python.jediEnabled": false, but the auto-completed brackets didn't work now, now the version is 1.33

I also get the issue with the auto-completed brackets when I disable Jedi.

The problem still remains

Same issue here.
When "python.jediEnabled": false, the auto-completed brackets doesn't work.
Version: 1.33.1 (system setup)

why  vscode-python didn't fix this?

State in 2019.9.34911:
* Does not happen when using the language server:

* Happens in Jedi (first part of the next gif), but if you

  * press escape so that the cursor at the end of the line disappears (green one in the gif)
  * go back and start typing
    then it will work (second part of the gif):

Can be fixed by modifying the following snippet string to .appendTabstop(0):
https://github.com/microsoft/vscode-python/blob/b73af16dad0788a035c6e8ef5b34fb32967290da/src/client/providers/completionSource.ts#L98-L100
why vscode-python didn't fix this for Jedi?

We haven't prioritized this issue since the original comment has a low 👍 count. Since the solution was outlined above, you are welcome to contribute by submitting a PR, along with tests.

We haven't prioritized this issue since the original comment has a low +1 count. Since the solution was outlined above, you are welcome to contribute by submitting a PR, along with tests.

@kimadeline The solution outlined above doesn't work.

I can repro this with "python.jediEnabled": true in my settings.json file. However, when I enable the Microsoft Python Language Server, the issue is resolved.
Workaround: Enable the Microsoft Python Language Server by setting "python.jediEnabled": false in your settings.json file. (See #3977 to find out how to get the latest stable version).

In my case, the mentioned workaround has worked, but still, the whole thing is a bug, obviously.

With "python.languageServer": "Jedi" autobrackets works. But I will continue to see the pop-up suggestion to use Pylance instead. And if I switch to Pylance, the auto brackets stops working again :(

We can set   "editor.suggest.snippetsPreventQuickSuggestions": false, then it can suggest anything


[SPARK-13686][MLLIB][STREAMING] Add a constructor parameter reqParam to (Streaming)LinearRegressionWithSGD

What changes were proposed in this pull request?
LinearRegressionWithSGD and StreamingLinearRegressionWithSGD does not have regParam as their constructor arguments. They just depends on GradientDescent's default reqParam values.
To be consistent with other algorithms, we had better add them. The same default value is used.
How was this patch tested?
Pass the existing unit test.

Can one of the admins verify this patch?

I think that's good, but would like @jkbradley or @mengxr to approve

Jenkins test this please

Test build #52514 has started for PR 11527 at commit 4dd34c4.

Test build #52514 has finished for PR 11527 at commit 4dd34c4.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52514/
Test FAILed.

Hi, @srowen .
I want to fix MiMa failures, but ./dev/mima passes locally whenever I run it locally.
So far, I didn't touch project/MimaExcludes.scala yet.
How can I check MiMa properly?
Jenkins, retest this please.

Oh, sorry. I found that. Thanks. :)

retest this please

retest this please

retest this please

Jenkins add to whitelist

Jenkins test this please

Test build #52530 has started for PR 11527 at commit 92be84f.

Thank you, @srowen !

Test build #52530 has finished for PR 11527 at commit 92be84f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52530/
Test PASSed.

Merged build finished. Test PASSed.

Hi, @mengxr and @jkbradley .
Could you review this PR, please?

Hi, @mengxr .
Could you review this PR, please?
After #11519 , I found that the missing reqParam argument in some APIs.

Rebased.

Test build #52870 has started for PR 11527 at commit 2b1b8a3.

Test build #52870 has finished for PR 11527 at commit 2b1b8a3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/52870/
Test PASSed.

@mengxr @jkbradley was the intent that you just set the reg param on the optimizer object? or do you agree this should be exposed as a setter? I am not sure enough to know this should be added.

Right, as you said, we can use optimizer object directly. I thought that's the main reason why this PR doesn't get any response from ML part committers.
At the first time, what I focused was the consistency of API among algorithms. Other algorithms having reqParam provides in this manner and also maintains its own reqParam value . You may remember that the list of reqParam values I made before. So, I want to make it complete by adding two missing part.
But, what I'm not sure here is Spark MLLib's direction. As you said, the use of optimizer is more recommended way now and in the future. I think this PR should be closed. If I can get some clear advice for this before closing this PR, I'll really be happy. :)

Test build #53030 has started for PR 11527 at commit 6b6a11f.

Test build #53030 has finished for PR 11527 at commit 6b6a11f.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53030/
Test FAILed.

Test build #53031 has started for PR 11527 at commit f0deb4a.

Test build #53031 has finished for PR 11527 at commit f0deb4a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/53031/
Test PASSed.

We discussed this during the pipeline API design and agreed that we shouldn't separate model hyper-parameters from training parameters. So this is the right place to set regParam. I'm going to merge this PR.

Merged into master. Thanks! Btw, it should be regParam instead of reqParam. It only appears in the PR title and MiMa excludes. So I thought it is a minor issue and merged this PR directly.

Oh, thank you so much for your attention/reviewing/merging, @mengxr !


Code cleanup

I've had a request for code cleanup, so I wanted to lay out the steps necessary to incorporate such a change.
At this point I've generally prioritized accepting contributions over code consistency for practical reasons, plus some code (like the vrep skeleton) is from other projects thus not easy to change the indentation because new updates will put it back. I've also preferred to be able to see who made what change over indentation fixes via git blame type commands.
When I can, I try to use the coding style of boost and indentation with four spaces not tabs.  Some of the code is from various people/sources, particularly in some non-public repositories that use grl and code from v-rep's distribution.
I’d be happy to switched to a specific coding style, but there would be a couple requirements (which is why I haven’t done this yet):

no manual changes should be needed for third party code
add a ci based linter to verify pull requests, preferably with something like the boost/stl coding style. I’m not a fan of the google coding style because it breaks typedefs with boost, and google bans boost.
prevent pushes directly to master with something like https://gist.github.com/pixelhandler/5718585 or the related script https://gist.github.com/mattscilipoti/8424018
provide instructions so people can check this locally before they push
finally submit the pull request that updates all the relevant code

As you can see this will take some time… so I haven’t taken care of it but a contribution is definitely welcome. Once an approach has been discussed and agreed upon plus a pull request made, I can do steps where project owner permissions are required. :-)

prevent pushes directly to master with something like https://gist.github.com/pixelhandler/5718585 or the related script https://gist.github.com/mattscilipoti/8424018

This is from one of the comments in above link

It is worth mentioning that the pre-push hook is a local hook, that is a hook that is fired when invoking a command in the local repository, and is potentially editable by a user. If you want to protect your repositories against disgruntled programmers, a pre-receive hook in the remote repository is necessary. Anyway, thanks for sharing! The script is perfect when you can trust all programmers in the team.

More info on pre-receive hook
https://help.github.com/enterprise/2.8/admin/guides/developer-workflow/creating-a-pre-receive-hook-script/

When I can, I try to use the coding style of boost and indentation with four spaces not tabs.

Thoughts on 80 column rule? Maybe use two spaces to fit easily in 80 columns?

When I can, I try to use the coding style of boost and indentation with four spaces not tabs.

Thoughts on 80 column rule? Maybe use two spaces to fit easily in 80 columns?

That's actually one of the parts that bothers me most about google coding style, the contortions needed to fit in 80 characters. It's 2017 so we don't have 640x480 or 720x480 screens anymore... I have a 4k screen. Preferably 320 columns, maybe 160 columns would be okay with me, 80 is definitely too low. Nice visualization of this point below, I don't really need to fit 8 code files side by side. :-)

Even I think 80 is too low. But need an upperbound and wrap it, 160 seems reasonable.

On Mon, Feb 13, 2017 at 4:17 PM tdinesh<EMAIL_ADDRESS>wrote:

Even I think 80 is too low. But need an upperbound and wrap it, 160 seems
reasonable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ahundt/grl/issues/134#issuecomment-279525091, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADZwGVHGXpKRb1nJdxq4txK4iYh8VFwks5rcMhmgaJpZM4L_qe7
.

Sure good enough
Cheers!
Andrew Hundt


fix(table)!: ensure that default and new() match

In https://github.com/ratatui-org/ratatui/pull/660 we introduced the
segment_size field to the Table struct. However, we forgot to update
the default() implementation to match the new() implementation. This
meant that the default() implementation picked up SegmentSize::default()
instead of SegmentSize::None.
Additionally the introduction of Table::default() in an earlier PR,
https://github.com/ratatui-org/ratatui/pull/339, was also missing the
default for the column_spacing field (1).
This commit fixes the default() implementation to match the new()
implementation of these two fields by implementing the Default trait
manually.
BREAKING CHANGE: The default() implementation of Table now sets the
column_spacing field to 1 and the segment_size field to
SegmentSize::None. This will affect the rendering of a small amount of
apps.

Affected repos:
https://github.com/search?q=ratatui AND "Table%3A%3Adefault()" AND (NOT org%3Aratatui-org) AND NOT (is%3Afork)&type=code

https://github.com/Adi-df/foucault - @Adi-df
https://github.com/SOF3/lpl - @SOF3
https://github.com/LAPKB/NPcore - @LAPKB
https://github.com/sarub0b0/kubetui - @sarub0b0
https://github.com/sadovsf/RadioLog - @sadovsf
https://github.com/asomers/gstat-rs - @asomers

Note the suggested changes above.
The impact of this PR is that the column spacing will change unless you've manually specified this, and the last column will not fill the remaining space if your width constraints do not specify to do so.

Thanks for notifying the changes !

I'd like a second reviewer on this one before merging - mainly as a quick sanity check that this breaking change makes sense.

Resolved conflicts in BREAKING-CHANGES.md

It was because this PR wasn't rebased on the latest main. I thought GitHub's queue feature would catch this but that doesn't seem to be the case?

Squash doesn't do a rebase, CI, then merge. it just does the rebase, merge part. The changes were textually non conflicting, just semantically.

Thanks for notifying.  But FTR, the warning for gstat-rs is a false alarm.


Move updates to main

New crc calculations and cleanup and additional SIA Codes.

Hi,
Is there anything we can help with to move this ahead?
Kind regards


Wrong order

Hi!
I am using bootstrap-tour.min.js and I have created three steps.
At tour start it starts at the last step and I only can go prev to the steo before.
Whats wrong?
Thank you for your help!

Same thing here.
Will continue to use intro.js.
Bootstrap tour is clearly not stable yet.


Failed during archive. "No output generated" Error

Troubleshooting
Hi,
My build failed on the last step and getting this error every time.
** ARCHIVE SUCCEEDED **
Non-system Ruby in use. This may cause packaging to fail.
If you use RVM, please run `rvm use system`.
If you use chruby, please run `chruby system`.
Running command: xcodebuild -exportArchive -archivePath IDIY.xcarchive -exportOptionsPlist /Users/vagrant/git/platforms/ios/exportOptions.plist -exportPath /Users/vagrant/git/platforms/ios/build/device
2023-10-27 05:20:37.041 xcodebuild[32407:159520] [MT] IDEDistribution: -[IDEDistributionLogging _createLoggingBundleAtPath:]: Created bundle at path "/var/folders/yy/6kcn9mkd5svdbqnznwf474f00000gn/T/IDIY_2023-10-27_05-20-37.040.xcdistributionlogs".
Exported IDIY to: /Users/vagrant/git/platforms/ios/build/device
** EXPORT SUCCEEDED **
No ios nor android platform's output dir exist
No output generated

Hello @routingsystems,
There was a bug that has been fixed in https://github.com/bitrise-steplib/steps-cordova-archive/releases/tag/3.0.3
It has been introduced in https://github.com/bitrise-steplib/steps-cordova-archive/releases/tag/3.0.1 release. This broke builds using cordova-ios<7 (but using cordova>7).


Diagram

Codecov Report

Merging #313 (e98ee61) into master (158063c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #313   +/-   ##
=========================================
  Coverage     65.15%   65.15%
  Complexity      706      706
=========================================
  Files           112      112
  Lines          2557     2557
  Branches        332      332
=========================================
  Hits           1666     1666
  Misses          760      760
  Partials        131      131

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 158063c...e98ee61. Read the comment docs.


(WIP) ignore JDBC proxies

What does this PR do?
Ignore proxies for JDBC instrumentation as an attempt to avoid unexpected span wrapping.
Checklist

[x] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[ ] I have added tests that would fail without this fix

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-07-03T13:55:01.409+0000

Duration: 14 min 18 sec

:grey_exclamation: Flaky test report
No test was executed to be analysed.
:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Allow setting of a maximum page size to be used with DynamoDB reads to attempt to allow better control of throughputs

This commit adds the ability to set a maximum page size that DynamoDB will use when reading data, this will allow me to run tests on UAT to confirm whether changing the max page size will have the desired impact on reducing read throughputs.
Signed-off-by: Craig Handley<EMAIL_ADDRESS><EMAIL_ADDRESS>- please review

Round one done.

@clicktravel-basharat comments applied

Round #2 done.

@clicktravel-basharat limit did cross my mind but I thought limit seemed to relate more to the overall number of records that would be returned rather than the maximum number of records in the current fetch. Looking through AWS docs they refer to it as page size also http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/QueryAndScanGuidelines.html
Let me know one way or the other which you think would be better

After more thought, the max page size should not feature on the method as the maximum page size is an internal setting which does not affect the overall result of the query method. It seems a template-wide config rather than per-request param.
I think what would be most appropriate so to have the AbstractDynamoDbTemplate accept some config which can set the max page size.
Or do we indeed have a concrete need to set it on a per-request basis, that cannot be achieved by wiring in multiple DB templates?

@clicktravel-basharat Can I suggest that we should just get it in as is. The main reason I need this in is so that I can actually test it in the UAT environment, as to be honest this may not even work at all, so probably best not to spend too much time on it now. If we get it in and it has the desired effect, then I think we should refactor to apply in the best way.


User Error

Found the problem. SORRY! my bad!

in fact there WAS an error.
I forgot the init.py inside the package and the pip just did not copy the package at all.
fixed in version 0.1.8

Oh! yeah I see it now. Today was a hectic day so I confused myself with everything that was happening.
Thank you for fixing it so quickly!


Use tha latest jinja templating image

Description
Changes proposed in this pull request:

Use tha latest jinja templating image

Related issue(s)

The image will be updated to newer version in https://github.com/capactio/hub-manifests/pull/21


Validator failed to sign block

Hi,
Two of our validators have failed to sign blocks using MEV.
Logs from our validator
time="2022-09-28 08:04:00" level=error msg="Failed to propose block" error="rpc error: code = Unknown desc = error posting the SignedBlindedBeaconBlockBellatrix to the builder api: unsupported error code: 502: did not receive 200 response from API" prefix=validator pubKey=0x86677a495cea
Logs from mev-boost
time="2022-09-28T08:04:00Z" level=error msg="error making request to relay" blockHash=0x2523ce4eb3ef36da80913668261ef0ca78f3724e344f227f8033a1caa39e8e37 error="HTTP error response: 400 / {\"code\":400,\"message\":\"read tcp <IP_ADDRESS>:8080-\\u003e<IP_ADDRESS>:11262: i/o timeout\"}\n" method=getPayload module=service parentHash=0x4ae7bb29df0d77a3633144d63684650fe9ee14da79c3ef9e1b76a834b428b0c5 slot=4794018 url="https://boost-relay.flashbots.net/eth/v1/builder/blinded_blocks" time="2022-09-28T08:04:00Z" level=error msg="no payload received from relay -- could be a network error or withholding." blockHash=0x2523ce4eb3ef36da80913668261ef0ca78f3724e344f227f8033a1caa39e8e37 method=getPayload module=service parentHash=0x4ae7bb29df0d77a3633144d63684650fe9ee14da79c3ef9e1b76a834b428b0c5<EMAIL_ADDRESS>slot=4794018 time="2022-09-28T08:04:00Z" level=info msg="http: POST /eth/v1/builder/blinded_blocks 502" duration=1.078647788 method=POST module=service path=/eth/v1/builder/blinded_blocks status=502

It looks like the reason was an internal networking problem between the ingress and the API.


Modify llc read methods

This PR adds more refactoring of the LLC data reading function to permit different approaches. The goal is to do some performance benchmarking of these methods on pleiades.
cc: @serazing

Current coverage is 92.04% (diff: 95.71%)

Merging #32 into master will decrease coverage by 0.06%

@@             master        #32   diff @@
==========================================
  Files             5          4     -1
  Lines           583        591     +8
  Methods           0          0
  Messages          0          0
  Branches        125        126     +1
==========================================
+ Hits            537        544     +7
  Misses           31         31
- Partials         15         16     +1

Powered by Codecov. Last update 536d234...105cce1


Weekly Patch Release v1.5.1

What does this PR do?
Contains all bugfixes to be released next week. As bugfixes get merged to master, they will get cherry-picked into this branch by the individual author (core) or on-caller.
For staff/core: The new release guide is here.
gh pr list -s merged -S 'merged:2021-11-01T18:30:00.000Z..2021-11-10T18:30:00.000Z' --json mergedAt,milestone,url,mergeCommit,title --jq 'sort_by(.mergedAt) | reverse | .[] | select((.milestone.title == "1.5.x") or (.milestone.title == null)) | [.mergedAt, .url, .mergeCommit.oid, .title] | join(" ")' --limit 100

2021-11-08T11:28:59Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10376 613aa0951416dde0ff8bb8dfa78f7808e83e7ff2 Revert part of #10279
2021-11-08T10:36:35Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10388 89e1360e756d5a697863e5aa28379e1cf587c422 Fix pickling error with CSVLogger
2021-11-08T02:05:44Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10359 bdc24e558a075acf5e71c957f24eb05ea7b25f28 Only import PostLocalSGD related modules when it's needed
2021-11-05T17:31:45Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10334 45f6a3b1758f88af7fd776915539800cbc0137a9 Fix DataLoader inspection and re-instantiation in Lite
2021-11-04T17:26:25Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10269 939a861853059d4a632843dbbf83e9ab802ad424 Update Python testing
2021-11-04T11:46:57Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10345 ce3e63262a7247c1111375b28c90bdd83b42bf3f Fix failure when `DataLoader(batch_size=None)` is passed
2021-11-03T11:18:10Z https://github.com/PyTorchLightning/pytorch-lightning/pull/10316 93caa7cda9277f86f17b1eab0793019aee8d40c2 Fix `apply_to_collection(defaultdict)`

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
I made sure I had fun coding 🙃
Part of #1 (it's a lie, this is just here to avoid noisy GitHub bot)

Some fixes that could potentially be included today: #10429, https://github.com/PyTorchLightning/pytorch-lightning/pull/10428, https://github.com/PyTorchLightning/pytorch-lightning/pull/10374/

This would concern the commit 4f6dcb9, right? Requesting comment from @carmocca.

It's fine to keep, no versions were added or removed, the jobs were just reorganized.


Introduce DrawingContext Push/PopRenderOptions

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

You can test this PR using the following package version. 11.0.999-cibuild0038985-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

You can test this PR using the following package version. 11.0.999-cibuild0038988-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


Compilation fails on otp 24

it seems rustler doesn't support otp 24 yet

thanks for opening this! i'll keep an eye on the progress here 👍

@ayrat555, looks like I can't publish this as long as we have a release candidate for rustler:
> mix hex.publish
** (Mix) A stable package release cannot have a pre-release dependency

I'll push a quick update once 0.22.0 is out.

@tzumby what do you think about releasing a pre-release version of the library. for example 0.2.0-rc.1 ?

@tzumby what do you think about releasing a pre-release version of the library. for example 0.2.0-rc.1 ?

This would indeed be very helpful for all of us trying to use this package currently!

Definitely open to that, I can prepare the rc release right now

Done: https://hex.pm/packages/ex_keccak/0.2.0-rc.1


Release binaries

First issue! And it's not a real issue after all, since this tool just saved me hours of work. Thank you!
But maybe a suggestion: could you (make GitHub) release binaries? In fact, it took me longer to download and install a C# compiler in my Visual Studio than to actually use the tool :)
(Oh, #17. Not the very first one, then.)

Thanks for the feedback. Added a binary to the release page.
Note: This was actually the first issue :-). GitHub uses a single counter for pull requests and issues.

Awesome, thanks!