dockerTools.streamLayeredImage: Store the customisation layer as a tarball

Motivation for this change
This fixes as issue described here, where permissions set by 'extraCommands' were ignored. We fix it by storing the customisation layer as a tarball rather than a regular directory. Another minor added advantage is that we can precalculate the checksum of the customization layer when building the store path.
The existing NixOS tests pass (nix-build -A nixosTests.docker-tools), with one modification where we were asserting that a derivation in 'contents' do not end up in /nix/store of the image if it doesn't have any dependencies; however I do find that behaviour counter-intuitive; since that derivation is declared in 'contents', I'd definitely expect it to end up in the images /nix/store, whether it has any dependencies or not. It looks like that test is introduced here; @roberth can you tell me if I am understanding it correctly?
cc @thatsmydoing @purcell
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

I've added the test for arion to make sure its base image is built correctly, which seems to be still the case with this PR.
Whether the existing behavior is counter-intuitive, I don't know. contents in buildImage (not buildLayeredImage, I know) has the following doc:

contents is a derivation that will be copied in the new layer of the resulting image. This can be similarly seen as ADD contents/ / in a Dockerfile. By default it's null.

It would be nice to be able to combine non-symlink files by listing derivations in contents. Those paths would have to be in the store in case of self-referencing outputs, but otherwise it's a waste of space.
For completeness I'll note here that you only need to use contents for adding to the root filesystem. To add to the store, you can just reference them config.Cmd.

Good catch @roberth . I removed those assertions. Sorry about the late reply.
@GrahamcOfBorg test docker-tools

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/223


Make default handoff_ip value <IP_ADDRESS> in vars.config

so nodes don’t hand off to themselves in production clusters. Also update dev_vars.config.src to set handoff_ip to <IP_ADDRESS> for devrel builds.

:+1: Identical to EE


drawio-integration with embed.diagrams.net 21.2.8 causes malformed editor when scrollbar is present and UI is not on bottom of page

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
This issue can be seen in your own examples. This works fine http://jgraph.github.io/drawio-integration/helloworld.html. However when using http://jgraph.github.io/drawio-integration/javascript.html with the scrollbar on top it will look like this when clicking to edit:

If you scroll to bottom it works fine:

Is there anyway to pin https://embed.diagrams.net/ version to avoid bugs like this in our production environment?
Expected behavior
A working diagram editor even when a scrollbar is present
draw.io version (In the Help->About menu of the draw.io editor):
I can not see version but https://embed.diagrams.net/ says 21.2.8.

draw.io version 21.2.8

Desktop (please complete the following information):

OS: Windows 11 Pro
Browser Chrome
Version 113.0.5672.64 (Official Build) (64-bit)

Thanks for the report, this is fixed in 21.2.9. If you can't get it to update try embed.diagrams.net/clear.html
We don't currently have pinning. It could be possible, but we've no idea how many are using are using embed, maybe worth a feature request and see whether it gets votes.

@davidjgraph Thank you for the fast response and fix! I wrote a feature request for version pinning here: https://github.com/jgraph/drawio/issues/3566


Embargo Date Validation

Why was this change made?
Fixes #556
Displays the following when month and day are not set:

How was this change tested?
New and existing tests.
Which documentation and/or configurations were updated?
n/a

Currently failing in the following two specs:

spec/requests/edit_work_spec.rb
spec/requests/create_work_spec.rb

  Failure/Error: render 'errors', status: :bad_request

     ActionView::MissingTemplate:
       Missing template works/errors, objects/errors, application/errors with {:locale=>[:en], :formats=>[:html
], :variants=>[], :handlers=>[:raw, :erb, :html, :builder, :ruby, :jbuilder]}. Searched in:
         * "/Users/jpnelson/2020/sul-dlss/happy-heron/app/views"
         * "/Users/jpnelson/.rbenv/versions/2.7.1/lib/ruby/gems/2.7.0/gems/devise-4.7.3/app/views"
         * "/Users/jpnelson/.rbenv/versions/2.7.1/lib/ruby/gems/2.7.0/gems/view_component-2.22.1/app/views"
     # ./app/controllers/works_controller.rb:32:in `create'

On line 32 in theworks_controller.rb the render call is not finding the correct jbuilder template at app/view/works/errors.json.jbuilder view https://github.com/sul-dlss/happy-heron/blob/04f7da46fdf6fe4d85c6549965c5045b6af056b2/app/controllers/works_controller.rb#L26-L33

I think this ought to have an integration test (in spec/requests/create_work_spec.rb) where we can see this error.

Do you mean an integration test in the spec/features/create_work_spec.rb?

I think in the requests spec should be sufficient. I'm not so concerned about the front end.


Error 407: Proxy

Hey guys, any idea how to fix this? I am unable to bypass the proxy on the server I am using as it is needed for other Synthetic Transaction I have configured.
Cannot retrieve speedtest configuration
ERROR: HTTP Error 407: Proxy Authentication Required

You will need to set http_proxy and https_proxy env vars so that include the user:password in the env vars.
This application provides no way to configure the proxy directly, and relies on native python means of doing so.
export http_proxy=https://user:password@proxy:8080
export https_proxy=$http_proxy


Integrating Razorpay in a React-Django(Django Rest Framework) Application

Article checklist
Attention
Our Peer Reviewer and Content Moderator teams do NOT provide any revisions services. All revisions and edits should be completed on your own forked repo (as to not take up room in the queue) in order for our team to review them all in a timely manner.
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.

As a rule of thumb - please be sure to only submit articles (pull requests) that are fully polished and ready to be published. Be sure to go through our resources documents for extra and 3rd party (vetted) resources to help improve overall technical writing.

Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #6414

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Hello @kevinkim573
Thank you for such a great article. However, kindly add a hero image of about 600*400 in dimension.
Ping me when you are done,
Thanks.

done @mercymeave :rocket:

plagiarism check done ✔

We will be reviewing this (& publishing if it is ready) ASAP.

jumping on this one

👏
Well done 🚀
A skill section on your Author profile would go a long way with future potential employers.
Go over our docs to see how to add a skills section 🙌

@kevinkim573
Can you please reach out via a DM asap 👍
Thank you


feat: add build command

Closes #9
This PR introduce the new build command that allows the user to create seccomp profiles from the collected syscalls.

Hi @ccoVeille,
I'm happy this project got your interest 🙂
The reason why I merged the PR so quickly is that I need this functionality asap.
I really appreciate your review, so I'm going to open a new PR, in the upcoming days, to fix all the bad things you highlighted in the code (or if you want to contribute, feel free to open your own PR).
Thanks for your interest. I'm happy to see that someone care about the project ;)
Next time I could ask your review if you give your availability.

I really appreciate your review, so I'm going to open a new PR, in the upcoming days, to fix all the bad things you highlighted in the code (or if you want to contribute, feel free to open your own PR).

I find bavk our conversation in my saved item.
Did you fix all the things I reported?


add Qwoyn Blockchain

Done
On Mon, Feb 6, 2023, 11:42 AM JeremyParish69 @.***>
wrote:

@.**** commented on this pull request.
In testnets/qwoyntestnet/assetlist.json
https://github.com/cosmos/chain-registry/pull/1366#discussion_r1097781408
:

@@ -0,0 +1,26 @@
+{

"$schema": "../assetlist.schema.json",
"chain_name": "qwoyn",

this should be 'qwoyntestnet' too
—
Reply to this email directly, view it on GitHub
https://github.com/cosmos/chain-registry/pull/1366#pullrequestreview-1285801814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBIMSTBKLD6UA6LXIQ7RR3WWFAYHANCNFSM6AAAAAAUSCRE7U
.
You are receiving this because you authored the thread.Message ID:
@.***>

Not sure why the validate assetlist.json test is failing.   json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 24 column 7 (char 650)

Aha ok. Ill make the change
On Mon, Feb 6, 2023, 1:07 PM JeremyParish69 @.***>
wrote:

@.**** commented on this pull request.
In testnets/qwoyntestnet/chain.json
https://github.com/cosmos/chain-registry/pull/1366#discussion_r1097864626
:

@@ -0,0 +1,90 @@
+{

"$schema": "../chain.schema.json",
"chain_name": "qwoyntestnet",
"status": "live",
"network_type": "testnet",
"website": "https://qwoyn.studio/",
"pretty_name": "Qwoyn",
"chain_id": "higgs-boson-1",
"bech32_prefix": "qwoyn",
"daemon_name": "qwoynd",
"node_home": "$HOME/.qwoynd",
"key_algos": [
"secp256k1"
],
"slip44": 123,

For SLIP44 (for address generation), the majority use the same method as
Cosmos (118) and Ethereum (60)
—
Reply to this email directly, view it on GitHub
https://github.com/cosmos/chain-registry/pull/1366#discussion_r1097864626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBIMSROORH363QA33VP733WWFKYTANCNFSM6AAAAAAUSCRE7U
.
You are receiving this because you authored the thread.Message ID:
@.***>

Everything should be fixed now.  Thanks for working with me on this @JeremyParish69


Feature/enhanced host record control

This looks good, @jcejohnson! Can you please address the style issues I raised, and also squash this down to a single commit so we keep a clean git history and also get rid of the merge commits? Thanks!

Done!

Everything is updated.

Is this good to merge now?

I'm just wrapping up being on vacation. I'll merge and release on Monday :)

Thanks again for your contributions, @jcejohnson - I'm merging this in now and will cut a release.

v2.1.0 has been released to the supermarket - thanks again.


Avoid covering current search highlight with search box

Summary of the Pull Request
Adds a scroll offset to avoid hiding the current search highlight with
the search box.

Offset is based on the number of rows that the search box takes up. (I am not totally sure I am calculating this right)
This won't help when the current highlight is in the first couple rows of the buffer.

Fixes: #4407

(you need to do a tools\runformat.cmd` to fix the code formatter)

Thanks so much for the fix! Sorry it took so long to reivew.

Thanks!  no complaints from me about the review time, I know you guys have priorities etc.


Labelmap intensity range not set correctly when cal_min and cal_max are incorrect

Hi,
I have a bunch of nifti labelmap images that seem to have incorrect metadata, mainly the values of cal_min and cal_max are both set to zero. This creates a problem for correctly displaying labelmaps as they all appear mapped to 1 even if the image contains multiple labels.
The problem is fixed if I overwrite the values in hdr with the min and max computed by
https://github.com/niivue/niivue/blob/f842c12048216c1b1cc70b6b09038e7eb4575b3f/src/nvimage/index.ts#L2337
Perhaps when the flag trustCalMinMax is set to false, it could overwrite the header so that it can display the image correctly?

@fk128 can you create a minimal demo to showcase this issue or share your images. The logic"
cmMin === cmMax &&
      this.trustCalMinMax &&
      isFinite(this.hdr.cal_min) &&
      isFinite(this.hdr.cal_max) &&
      this.hdr.cal_max > this.hdr.cal_min

should ignore the header's cal_min and cal_max unless the maximum is larger than the minimum. You describe a situation where they are identical (both zero) so I would suggest it should use the robust range.
For label maps you make want to consider setting the intention to NIFTI_INTENT_LABEL as an interpolation hint. Also see the developer notes for Atlases and Labeled Images.
Another thing to note is that some colormaps provide explicit minimum and maximum values, which can be helpful if you want them but do take precedence over the automatic calculations. These are helpful for CT images and labeled images where the intensity is known a priori.

@neurolabusc here's an example image where this seems to occur
labelmap.nii.gz It contains two labels, but then the range appears as 0..1 in niivue

below is from opening up the labelmap in 3D slicer, which it interprets correctly

@fk128 NiiVue uses FSL's robust range calculation for estimating the default image brightness and contrast. For most data, this uses the 2nd and 98th percentile for image intensity range. This formula is generally very stable for continuous contrast MRI, CT, PET, SPECT, US modalities. However, your segmented lung image only has a few non-zero voxels, virtually all voxels are zero with a few 1s and a few 2s. Therefore, the robust range for your image is 0..1. In brief, using the FSL robust range ensures behavior with other tools and works for most scalar datasets. However, it is not the ideal option for all situations.
In your case, where you are creating a viewer that knows it is loading segmented images, you can simply specify using the entire range for your image. You can take advantage of the fact that NiiVue calculates the global maximum intensities when it loads an image:
await nv1.loadVolumes(volumeList1);
nv1.volumes[0].cal_min = nv1.volumes[0].global_min
nv1.volumes[0].cal_max = nv1.volumes[0].global_max
nv1.updateGLVolume();

Ah, I see. I'll do that. Thanks for getting back to me so quickly!

Latest version does attempt to deal with many situations like this (where a huge proportion of voxels are zero).


Fix a problem with pickling molecules with more than 255 rings

We were saving the number of rings using an unsigned char. This fixes that.
Backwards compatibility is maintained.

@bp-kelley this one will make you laugh

@bp-kelley : are your questions answered?

@greglandrum This commit has introduced a backwards incompatible change and should not have been included in a patch release.
If you now pickle a molecule with 2022_09_5 and unpickle in 2022_09_3 you will get a segfault:
import rdkit
from rdkit import Chem
import pickle

print(rdkit.__version__)
if rdkit.__version__ == '2022.09.5':
    print("Writing pickle")
    mol = Chem.MolFromSmiles("C1CC1")
    Chem.FastFindRings(mol)
    with open("mol.pkl", "wb") as hnd:
        pickle.dump(mol, hnd)
else:
    print("Reading pickle")
    with open("mol.pkl", "rb") as hnd:
        mol = pickle.load(hnd)

$ python pickle_bug.py
2022.09.5
Writing pickle
$ python pickle_bug.py
2022.09.3
Reading pickle
Segmentation fault

@ptosco : that's not a backwards incompatible change. We've never guaranteed that an older version of the code could parse newer pickles, and this pickle is a new version
It was a mistake to only increase the patch number of the pickle version. This should have been the minor version so that trying to read it with an older version of the code produces a warning.
I guess the solution to that would be to do another PR which either bumps the minor version number or changes the code to warn on patch release mismatches. Probably the second makes more sense

Also: you're probably right that this wasn't a good candidate for a patch release.

Sorry, I meant forward, not backward. It is pretty normal that an older version can't read files written by a newer version, but I don't think this should happen within patch versions.


Inconsistent results when svg is hidden on page load

Summary:
The getBoundingClientRect() left/right/top/bottom==0 check (line 140), which I assume is checking for IE (9,10,11) failure in loading external assets, also fails if the icons are deliberately hidden by CSS on page load (e.g. display:none on a parent element), and therefore disrupts svg use in non-IE browsers too.
Details:
Due to the isHidden check, which uses the results from getBoundingClientRect(), if the icons are hidden by default (i.e. a favouriting (heart) icon that only shows due to :hover over each image), this code only fires when the first (due to the MutationObserver, or EventListener on DOMSubtreeModified) :hover occurs. checkUseElems  is called a second time from debouncedCheck, but the other icons on the page with identical < use xlink:href > are still hidden, so still don't get modified. unobserveChanges is then called, so further :hover events no longer trigger this functionality.
Chrome Canary (51.0.2704.103 (64-bit)) on Mac
Safari Version 9.1.1 (11601.6.17)
<EMAIL_ADDRESS>

I couldn't reproduce the issue. Please provide a reduced test case.

https://gist.github.com/sword/e1efa04627153901ad911b5d1f44bd3e
The underlying cause was a broken svg file, missing the xmlns attribute; this caused Chrome & Safari to fail to load the remote svg, so getBoundingClientRect() returned zeros. Subsequent triggering of a DOM change only inlines the SVG if it's currently hovered (i.e. visible). A valid SVG file makes Chrome/Safari work correctly (not bother, as getBoundingClientRect() finds dimensions), but I guess this will continue to affect IE (not tested).
However, I'm wondering if attrUpdateFunc should modify all matching < use xlink:href >, not just the current one, otherwise future uses of the same cache[spec.base] are ignored.

Thank you. I made some changes to update all use elements with a matching xlink:href. However, if there are no DOM mutations after the document loads, and the display property only changes via CSS, the referenced SVG won't be loaded at all.


有计划支持 KTO 吗？

https://github.com/ContextualAI/HALOs 的工作提到KTO的效果优于DPO和PPO，且不需要paired dataset

看起来make sense，现实中paired dataset获取成本较高。

dpo trainer的loss里貌似有kto，但不知道是否能成功训练

确实很心动，期待作者集成～

同期待！

Very interested as well. Multiple research papers have confirmed at this point that KTO is superior to DPO in many ways.

Very interested as well. Multiple research papers have confirmed at this point that KTO is superior to DPO in many ways.
Can you share a few of these papers, please? Thank you very much.

Yeah, here are two recent ones:
Insights into Alignment: Evaluating DPO and its Variants Across Multiple Tasks
Orca-Math: Unlocking the potential of SLMs in Grade School Math

fixed in https://github.com/hiyouga/LLaMA-Factory/pull/3785


Too many task instance queries

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened

What you expected to happen

How to reproduce

Anything else
No response
Version
3.1.x
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please describe your problem clearly.


web error

my ip and host are hidde
error
2024/08/02 17:27:44 [error] 53269#0: 233 FastCGI sent in stderr: ＂PHP message: PHP Warning: Undefined array key ＂ffffffff-e744-d555-ffff-ffffef05ac4a＂ in /www/wwwroot/Server_Panel/public/commands.php on line 108; PHP message: PHP Warning: Trying to access array offset on null in /www/wwwroot/Server_Panel/public/commands.php on line 108＂ while reading response header from upstream, client: <IP_ADDRESS>, server: .., request: ＂POST //commands.php HTTP/2.0＂, upstream: ＂fastcgi://unix:/tmp/php-cgi-83.sock:＂, host: ＂..*:483＂
php version :8.3
nginx version : 1.23.4

done


Make all errors actual JavaScript Errors

In BWC, not all errors are actual JS Error objects. See https://github.com/bitpay/bitcore-wallet-client/tree/master/lib/errors
This can make code that uses BWC  somewhat ugly and odd. Some functions have to handle errors one way, and others a different way.
Bitcore has a good way of dealing with custom errors. See https://github.com/bitpay/bitcore-lib/tree/master/lib/errors

This was added in: https://github.com/bitpay/bitcore-wallet-client/pull/189


Register leanderjosh.is-a-fullstack.dev

Added leanderjosh.is-a-fullstack.dev using the CLI.

No commits have been made since, please make sure you committed to the correct branch.

Closed for inactivity.


How to connect windows application proxy[local host] to WSL [KALI LINUX TERMUX]

I am using [Kali linux terminal] WSl in WINDOWS11
How to connect windows application proxy[local host] to WSL [KALI LINUX TERMINAL].
The proxy address is <IP_ADDRESS>:9050
I had tried
to set temporarily  export http_proxy=http://<IP_ADDRESS>:9050 its failed & showing error connection refused.
Next
I had tried adding this  export http_proxy=http://<IP_ADDRESS>:9050 export ftp_proxy=http://<IP_ADDRESS>:9050 https_proxy=http://<IP_ADDRESS>:9050
nano /etc/bash.bashrc ,nano ~/.bashrc,nano /etc/environment
Nothing had worked.Hoping for a fast solution
THANKS IN ADVANCE

Please provide all the steps to reproduce this issue.

@Biswa96 i don't think it is an issue.
How can i connect my connect windows application proxy[local host] to WSL [KALI LINUX TERMINAL].

@danksuperuser what version of WSL are you using, WSL1 or WSL2? The networking is different between WSL1 and WSL2.
If you're using WSL2 set portproxy with netsh. The host firewall will have to be opened also.  Find your WSL2 endpoint IP and try, replace x.x.x.x with actual IP address:
netsh interface portproxy add v4tov4 listenport=9050 connectport=9050 connectaddress=x.x.x.x protocol=tcp
then use export http_proxy=http://x.x.x.x:9050 inside Kali. If you need UDP also, add another portproxy rule.
Might not work, didn't test it.

I am using WSL 2.
Let me try this .

@elsaco i tried it but it didn't work.

Thanks for reporting this @danksuperuser.
Can you share the exact steps for what you're trying to do ? Access a network endpoint running in Windows from within WSL or the opposite ?

Ok I see. So if you want to connect WSL to an application running on Windows, @elsaco's solution should work, as long as the socks proxy is listening on the correct address, but that might not be what you want, since that would open it to the whole network.
One thing can try is an ssh port proxy. If you run an ssh server in WSL, something like:  ssh -R 9050:localhost:9050 user@localhost  should forward the traffic into your socks proxy

@OneBlue  No! @elsaco 's solution didn't work.

@danksuperuser you also need to open the Windows firewall and allow inbound/outbound TCP traffic on port 9050, after you set the portproxy . If possible, disable the host firewall and see if proxy works as you'd expect. If it does, turn the firewall back on and add rules.

<IP_ADDRESS>:9050 means WSL's loopback interface not host's. You need to set your http_proxy in WSL to the ip:port linked with netsh interface portproxy rule.
The mapping should be, using <IP_ADDRESS> as example:
Windows proxy             WSL
<IP_ADDRESS>:9050 <-----> <IP_ADDRESS>:9050

then set http_proxy=<IP_ADDRESS>:9050 for your WSL clients.


updated lock

seems like it should be updated like that according to an email dated May 12

Thanks and sorry for the delay, I didn't get the notification for this one. We are currently using 9.2


[spl-token] createAssociatedTokenAccount() results in NotEnoughKeys on v0.3.5

The recent changes discussed on issue #3590 break the instruction returned by createAssociatedTokenAccount() in the typescript SDK. It results in a NotEnoughAccountKeys error from the program.
The problem is that the rent sysvar is not provided in the accounts context while the program does require it. Looking at the issue, this seems to be intentional so I can't be sure if it has a purpose or not. However, it currently breaks the SDK so either a revert may be needed or a update to the program is.
The only workaround I found so far was either downgrading or manually appending the sysvar public key to the instruction before sending it.
Since the SDK's unit tests don't send any transactions this error is not caught. In the future, it may be a good idea to implement integration tests that test the SDK on a local validator with the program.

Are you seeing this issue on a local validator running an old version? The associated-token-account program was updated a few months ago so it doesn't require the rent sysvar.  Here's a successful transaction creating an associated token account on devnet, note the lack of rent sysvar: https://explorer.solana.com/tx/5aCjJ3E2v6u8rppaXSaB39sNARyNBatBQbFhkjSvb32FgA7FqzcU3GEud5E5E2ymcxacUS3jZwTvLnWgvwjUR82W?cluster=devnet
I can't remember which version has the change, but I think anything starting with 1.10 will have it. If not, 1.11 certainly has it.  Can you try updating and see if it works?
Also, you can find all of the end-to-end JS tests at https://github.com/solana-labs/solana-program-library/tree/master/token/js/test/e2e and https://github.com/solana-labs/solana-program-library/tree/master/token/js/test/e2e-2022 which runs against a local validator.  If you think there's some coverage missing, please file an issue, or better yet, a PR!

Closing since there hasn't been a response


Kaplan Meier estimator issue

Hello,
In the Kaplan Meier estimator, the individual at risks (in the denominator) at time t in the program is np.sum(t <= timestamps) but this sum takes into account only the non censored individuals as timestamps = timestamps[event_observed == 1]. I think it should be np.sum(t <= timestamps) but with timestamps the original input of the function. The same comment is valid for the Nelson Aalen estimator. What do you think ?The end of the code could be:
timestamps_observed = timestamps[event_observed == 1]
unique_timestamps_observed = np.concatenate((np.zeros(1), np.unique(timestamps_observed)))

return np.cumprod(
    np.fromiter((1.0 - np.sum(t == timestamps_observed) / np.sum(t <= timestamps)
                 for t in unique_timestamps_observed), dtype='float',    count=unique_timestamps_observed.size))

Hi, thanks for catching this, you can do a pull request to correct this
Best
Stéphane

Thanks again !


Update log4j-slf4j-impl to 2.13.2

Updates org.apache.logging.log4j:log4j-slf4j-impl from 2.13.1 to 2.13.2.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.logging.log4j", artifactId = "log4j-slf4j-impl" } ]

labels: library-update, semver-patch

It looks like GitHub is failing to see this build as passed. Merging manually.
🤷


Could not get models from the server after deploying a Yolo11n-pose model

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
Hello there,
I have been deploying some models on Cvat using Nuclio and everything worked fine until now. I deployed the generic models (Yolov3, yolov7 and mmpose) and also my custom object detection model (Yolo11n).
Then, I wanted to deploy my custom pose model (Yolo11n-pose) and I raised the "Could not get models from the server" error. I  have been facing this issue for more than 2 days now and have no idea what to do next. All the logs from the init_context function are okay in the docker container. My model is in my opt/nuclio folder as expected.
Since the only example for deploying a pose model came from a mmpose model with a pytorch format, I tried to reproduce the same structure.
Any help would be really appreciated :)
Is there any guideline or support on how to deploy these yolo models ? Onnx or pt format would be great!
Here is my function.yaml file :
metadata:
  name: pth-yolo11npose-sport
  namespace: cvat
  annotations:
    name: YOLO-Pose
    type: detector
    framework: pytorch
    spec: |
      [
        {
          "name": "person",
          "type": "skeleton",
          "sublabels": [
            {"id": 0, "name": "LHead", "type": "points"},
            {"id": 1, "name": "Nose", "type": "points"},
            {"id": 2, "name": "Sternum", "type": "points"},
            {"id": 3, "name": "RShoulder", "type": "points"},
            {"id": 4, "name": "RElbow", "type": "points"},
            {"id": 5, "name": "RWrist", "type": "points"},
            {"id": 6, "name": "LShoulder", "type": "points"},
            {"id": 7, "name": "LElbow", "type": "points"},
            {"id": 8, "name": "LWrist", "type": "points"},
            {"id": 9, "name": "RPelvis", "type": "points"},
            {"id": 10, "name": "LPelvis", "type": "points"},
            {"id": 11, "name": "RAnkle", "type": "points"},
            {"id": 12, "name": "RFoot", "type": "points"},
            {"id": 13, "name": "LKnee", "type": "points"},
            {"id": 14, "name": "LAnkle", "type": "points"},
            {"id": 15, "name": "LFoot", "type": "points"},
            {"id": 16, "name": "Griff", "type": "points"},
            {"id": 17, "name": "CHead", "type": "points"},
            {"id": 18, "name": "RHead", "type": "points"},
            {"id": 19, "name": "RKnee", "type": "points"}
          ]
        }
      ]

spec:
  description: YOLO11-pose model for pose estimation
  runtime: 'python:3.8'
  handler: main:handler
  eventTimeout: 30s

  build:
    image: cvat.pth.yolo11npose.sport
    baseImage: python:3.8
    directives:
      preCopy:
        - kind: RUN
          value: apt update && apt install -y git libgl1 --no-install-recommends && rm -rf /var/lib/apt/lists/*
        - kind: WORKDIR
          value: /opt/nuclio
        - kind: RUN
          value: pip install torch==2.0.0+cpu torchvision==0.15.1+cpu --extra-index-url https://download.pytorch.org/whl/cpu
        - kind: RUN
          value: python -m pip install --upgrade pip setuptools wheel
        - kind: RUN
          value: pip install ultralytics
        - kind: RUN
          value: pip install numpy opencv-python-headless pillow pyyaml
        - kind: ADD
          value: https://github.com/****/*****/releases/download/yolo11n_sport/best.pt /best.pt

  triggers:
    myHttpTrigger:
      maxWorkers: 1
      kind: 'http'
      workerAvailabilityTimeoutMilliseconds: 30000
      attributes:
        maxRequestBodySize: 33554432 # 32MB

  platform:
    attributes:
      restartPolicy:
        name: always
        maximumRetryCount: 3
      mountMode: volume

And there is my main.py file :
import json
import base64
import io
import yaml
import numpy as np
import torch
from PIL import Image
from ultralytics import YOLO
import os

def init_context(context):
    try:
        context.logger.info("Init pose detector")

        context.logger.info(f"Current directory: {os.getcwd()}")
        context.logger.info(f"Files in /opt/nuclio: {os.listdir('/opt/nuclio')}")

        with open("/opt/nuclio/function.yaml", 'rb') as function_file:
            functionconfig = yaml.safe_load(function_file)
            context.logger.info('Functionconfig : ', functionconfig)
            labels_spec = functionconfig["metadata"]["annotations"]["spec"]
            context.logger.info('Labels specs : ', labels_spec)
            labels = json.loads(labels_spec)
            context.logger.info('Labels : ', labels)

        model_path = '/opt/nuclio/best.pt'
        context.logger.info(f"Loading model from: {model_path}")
        context.logger.info(f"Model exists: {os.path.exists(model_path)}")

        model = YOLO(model_path)
        context.logger.info("Model loaded successfully : " , model)

        context.user_data.model = model
        context.user_data.labels = labels

        context.logger.info("End of init_context function")

    except Exception as e:
        context.logger.error(f"Error in init_context: {str(e)}")
        raise

def process_predictions(results, labels, threshold=0.5):
    processed_results = []

    # Handle each detection
    for result in results:
        keypoints = result.keypoints.data[0]  # Obtenir les keypoints (x, y, conf)
        confidence = result.boxes.conf[0].item()  # Score de confiance global

        if confidence > threshold:
            elements = []
            for idx, kpt in enumerate(keypoints):
                x, y, conf = kpt.tolist()
                elements.append({
                    "label": labels[0]["sublabels"][idx]["name"],
                    "type": "points",
                    "outside": 0 if conf > threshold else 1,
                    "points": [float(x), float(y)],
                    "confidence": str(conf)
                })

            skeleton = {
                "confidence": str(confidence),
                "label": "person",
                "type": "skeleton",
                "elements": elements
            }

            if not all([element['outside'] for element in elements]):
                processed_results.append(skeleton)

    return processed_results

def handler(context, event):

    context.logger.info("Run YOLO11-pose model")

     data = event.body
     img_bytes = base64.b64decode(data["image"])
     img = Image.open(io.BytesIO(img_bytes)).convert('RGB')

     # Get threshold
     threshold = float(data.get("threshold", 0.5))

     # Inférence
     with torch.no_grad():
         results = context.user_data.model(img)
         print('Results : ', results)

     # Post-traitement
     detections = process_predictions(results, context.user_data.labels, threshold)

     return context.Response(
         body=json.dumps(detections),
         headers={},
         content_type='application/json',
         status_code=200
     )

Thanks for your time
Expected Behavior
No response
Possible Solution
No response
Context
No response
Environment
No response

You missed "svg" field for the skeleton in yaml specification. Please, refer to mmpose for example


chore: Updated Conditions and bumped up minSDK

Changes:

Updated Conditions and Bumped up gradle version

@iamareebjamal have a look


apt install python-vcstool version error

Just checking with you guys. I found when trying to run the install of vcstool with:
sudo apt install python-vcstool
I would get a version error claiming my python version was out of range
The following packages have unmet dependencies:
python-vcstool : Depends: python2 (< 2.8) but it is not installable
Depends: python2 (>= 2.7) but it is not installable
Depends: python2:any (>= 2.6.6-7~) but it is not installable
I do have 2.7.17, but looking into this further I checked the ros package list here:
http://packages.ros.org/ros/ubuntu/dists/bionic/main/binary-amd64/Packages
It seems for python-vcstool it checks for the version using python2 vs python. Not sure if this was used due to conflicts with python3 but it seems to fail the check and would think just using python instead of python2 would be correct. If I am wrong, is this an issue with python2 not returning the correct version or being linked correctly? Every version check on several machines return the right 2.7.17, but all give the same error.

What certain of vcstool are you trying to install?

The latest version, 0.2.8

Thanks for the report. I guess this is due to packaging being done on Focal which uses python2 instead of python. I will create a new deb soon to are the problem.
Until then (or even in general) you can use the Python 3 version of the package: python3-vcstool.

Okay great, thanks for the help!


⚠ Bofrost.de has degraded performance

In e3b0fdb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1871 ms

Resolved: Bofrost.de performance has improved in 2f39a77 after 20 minutes.


Adding a custom image to a Switch thumb (or any thing that requires a Decoration) is pretty nutty right now.

Adding a custom thumb decorator to the switch thumb is pretty rough, as per our hangouts conversation.

I suspect much of this is the fact that we haven't written an example of using an image in a switch for the flutter gallery.  Once we do that, we'd feel the pain ourselves.

I would love to do this but to test it I need some assets for a switch to put in the gallery demo.

@Hixie I'd just pick an icon from our icons:
http://docs.flutter.io/flutter/material/Icons-class.html
You could toggle between reclined and unreclined airline seats, or car vs bike, or any number of fun combinations from that set. :)


Configure multi-region writes in Azure Cosmos DB for MongoDB: No such option on portal

Hi,
I've tried following the instructions on the page, but there is no "Replicate data globally" option on the menu on the portal for "Azure Cosmos DB for MongoDB" resource.
Are these instructions indeed relevant to MongoDB?
Thanks.

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

ID: 223d16c4-837a-3658-0b7c-90d922e4a10f
Version Independent ID: de3e44ae-092e-1dde-c044-34ffc54db3e3
Content: Configure multi-region writes in your Azure Cosmos DB for MongoDB database
Content Source: articles/cosmos-db/mongodb/how-to-configure-multi-region-write.md
Service: cosmos-db
Sub-service: cosmosdb-mongo
GitHub Login: @gahl-levy
Microsoft Alias: gahllevy

@yoshigev
Thanks for your feedback! We will investigate and update as appropriate.

@yoshigev,
I just created a new Azure Cosmos DB for MongoDB account using the Try Azure Cosmos DB free.

I was able to see the Replicate data globally menu option within the Settings section.
Can you share a screenshot of the missing setting?
#reassign:seesharprun

Thanks @seesharprun for the quick response.
This is the screenshot with the missing option:

If it's helpful, this is the Resource JSON:

Resource JSON
{
    "id": "/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.DocumentDB/databaseAccounts/xxx",
    "name": "xxx",
    "location": "Germany West Central",
    "type": "Microsoft.DocumentDB/databaseAccounts",
    "kind": "MongoDB",
    "tags": {},
    "systemData": {
        "createdAt": "2022-02-11T07:54:08.2218443Z"
    },
    "properties": {
        "provisioningState": "Succeeded",
        "documentEndpoint": "https://xxx.documents.azure.com:443/",
        "sqlEndpoint": "https://xxx.documents.azure.com:443/",
        "mongoEndpoint": "https://xxx.mongo.cosmos.azure.com:443/",
        "enableAutomaticFailover": false,
        "enableMultipleWriteLocations": false,
        "enablePartitionKeyMonitor": false,
        "isVirtualNetworkFilterEnabled": false,
        "virtualNetworkRules": [],
        "EnabledApiTypes": "MongoDB",
        "disableKeyBasedMetadataWriteAccess": false,
        "enableFreeTier": false,
        "enableAnalyticalStorage": false,
        "analyticalStorageConfiguration": null,
        "instanceId": "xxx",
        "databaseAccountOfferType": "Standard",
        "ipRangeFilter": "",
        "consistencyPolicy": {
            "defaultConsistencyLevel": "Session",
            "maxIntervalInSeconds": 5,
            "maxStalenessPrefix": 100
        },
        "apiProperties": {
            "serverVersion": "4.0"
        },
        "configurationOverrides": {
            "EnableBsonSchema": "True"
        },
        "writeLocations": [
            {
                "id": "xxx-germanywestcentral",
                "locationName": "Germany West Central",
                "documentEndpoint": "https://xxx-germanywestcentral.documents.azure.com:443/",
                "provisioningState": "Succeeded",
                "failoverPriority": 0,
                "isZoneRedundant": true
            }
        ],
        "readLocations": [
            {
                "id": "xxx-germanywestcentral",
                "locationName": "Germany West Central",
                "documentEndpoint": "https://xxx-germanywestcentral.documents.azure.com:443/",
                "provisioningState": "Succeeded",
                "failoverPriority": 0,
                "isZoneRedundant": true
            }
        ],
        "locations": [
            {
                "id": "xxx-germanywestcentral",
                "locationName": "Germany West Central",
                "documentEndpoint": "https://xxx-germanywestcentral.documents.azure.com:443/",
                "provisioningState": "Succeeded",
                "failoverPriority": 0,
                "isZoneRedundant": true
            }
        ],
        "failoverPolicies": [
            {
                "id": "xxx-germanywestcentral",
                "locationName": "Germany West Central",
                "failoverPriority": 0
            }
        ],
        "cors": [],
        "capabilities": [
            {
                "name": "EnableMongo"
            },
            {
                "name": "EnableServerless"
            }
        ],
        "backupPolicy": {
            "type": "Continuous"
        }
    }
}

@nayakshweta, can you help with this issue? I'm unable to reproduce it.
#reassign:nayakshweta

Hello @yoshigev!
While Azure Cosmos DB is available in the Germany West Central region, Azure Germany is physically isolated from other instances of Azure. This means that you will not be able to globally replicate your data from that region to other Azure regions.
Thanks for your patience while we sought an answer.
#please-close


Update interactive_markers to C++17.

The main reason to do this is so that we can compile interactive_markers with the clang static analyzer.  As of clang++-14 (what is in Ubuntu 22.04), the default still seems to be C++14, so we need to specify C++17 so that new things in the rclcpp headers work properly.
Further, due to reasons I don't fully understand, I needed to set CMAKE_CXX_STANDARD_REQUIRED in order for clang to really use that version.  So set this as well.
Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

CI for this is in https://github.com/ros2/rmw_dds_common/pull/69#issuecomment-1410591207


Update json configuration usign API

We're considering to use this control to populate two comboboxes: one combobox should contains the list of products and the other one the lists of version; with this control we'd like to manage the relationship between products and versions.
We've execute some test with positive results, now we’d like complete the update of the information programmatically.
In order to update the combobox values, we've found DevOps API Web,
https://docs.microsoft.com/en-us/rest/api/azure/devops/processdefinitions/lists/update?view=azure-devops-rest-4.1
I'd like to ask if it’s available some APIs to update the json file with the relationship between comboboxes values.
Could you provide me some information?
Thank you
Maurizio

Duplicate of #37


Improving the Visualize attribute

Visualize cannot visualize Dictionaries in realtime
Steps to Reproduce

Add following code

[Visualize(nameof(SomeDict))]
public partial class Node2D // position this node in the world somewhere where you will see it
{
    public Dictionary<int, string> SomeDict { get; set; } = new();

    // Update SomeDict somewhere in _Process or something
}

Side note that is unrelated to this issue: There is no way to tell visualize to position a visual panel that does not extend from Node2D or Control to a manual position while at the same time visualizing a member in realtime.
Visualize messages are not all colorized
There should be a public static util method(s) specifically made for handling visualize messages. As an added bonus this could be hooked into the thread safe logger class.
Visualize cannot see values of private members from inherited classes
Have a look at this commit 706e60d67584822daa54d49e14db6d198b27ab2f
Steps to Reproduce Issue:

Set _curState access modifier from protected to private
Remove [Visualize] attribute from EnemyComponent
Run the game
Notice how you cannot see _curState for EnemyComponents even though EntityComponent has the visualize attribute listening to the _curState field
You will get the following message in console: "[Visualize] Tracking 'EntityComponent' to see if '_curState' value changes"

Where would one define settings for Visualize that can easily be edited by the developer?
For example I want to add a setting to disable visualization of underscores in private fields.
Should the underscores in private fields be visible in-game?
Control Elements are not all perfectely aligned

Add support for Godot.Collections.Array and Godot.Collections.Dictionary

Visualize code is so poorly written I'm thinking it should be remade from scratch.


[NFC] Modify method for characterizing bit-extension operations to handle charecterization of bit-truncation as well.

Currently the method for bit extension (a.k.a dequantization ops) only returns a bool. Make this method return a richer handle, which can also allow classification of bit-truncation operations. Also rename isDequantization method to isBitExtend operation.

The added info LGTM! I have a broader question about bit extend/"dequantization" ops in general though. Are we focusing too much on the bit extension aspect of it? From what I've seen, we're mainly using this to make decisions regarding fusion of producers with matmul/convolution ops and to control patterns like ElementwiseOpFusion. I'm thinking the same logic should apply to something like a broadcast though, so should we instead be focusing on the total size of the input and output operands instead of just the element types?

Broadcast is a little more complicated. I dont want to design in a vaccum. There are sporadically complaints about broadcasts not fused, but need a more structured look into it. So far cases that I have seen broadcast not being fused is a symptom not the root cause. We can come back to that later, but for quantized model handling extensions/truncations is more structured to reason about.


Remove 'codingben' from OWNERS

What this PR does / why we need it:
Remove 'codingben' from OWNERS
Release note:
None

/lgtm

/override ci/prow/e2e-tests


Allow to chose which social network sharing button to show or not

For now it's either all buttons or none.
Allow to chose which are wanted per-petition.

Hi @fallen, how are you?
Can I work on this issue?

@fallen in order to implement this, we will need 7 boolean fields in the petition model (one for the email sharing button and remaining 6 for the social network sharing buttons). Each field will indicate whether a particular sharing button needs to be shown or not. Please let me know if you need any changes in this approach.

@fallen should I add make these field changes for the PetitionTemplate model as well?


Theming and text colors

I'm having some trouble figuring out how to change the color of the text rendered to match the current system theme.
I went through the issues and saw this: https://github.com/halilozercan/compose-richtext/issues/12 however wether I'm using a dark or light theme, my paragraphs are always rendered in dark text.
I went as far as trying to override all styles manually like so:
@Composable
fun MarkdownComponent(markdown: String) {
    RichText(
        modifier = Modifier
            .padding(16.dp)
            .verticalScroll(state = scrollState),
        style = RichTextStyle()
            .resolveDefaults()
            .run {
                copy(
                    headingStyle = { level, textStyle ->
                        when (level) {
                            0 -> TextStyle(
                                fontSize = 36.sp,
                                fontWeight = FontWeight.Bold,
                                color = textStyle.color.w,
                            )
                            1 -> TextStyle(
                                fontSize = 26.sp,
                                fontWeight = FontWeight.Bold,
                                color = textStyle.color.w,
                                )
                            2 -> TextStyle(
                                fontSize = 22.sp,
                                fontWeight = FontWeight.Bold,
                                color = textStyle.color.copy(alpha = .7F).w
                            )
                            3 -> TextStyle(
                                fontSize = 20.sp,
                                fontWeight = FontWeight.Bold,
                                fontStyle = FontStyle.Italic,
                                    color = textStyle.color.w,

                                )
                            4 -> TextStyle(
                                fontSize = 18.sp,
                                fontWeight = FontWeight.Bold,
                                color = textStyle.color.copy(alpha = .7F).w
                            )
                            5 -> TextStyle(
                                fontWeight = FontWeight.Bold,
                                color = textStyle.color.copy(alpha = .5f).w
                            )
                            else -> textStyle.copy(color = textStyle.color.w)
                        }
                    },
                    stringStyle = stringStyle?.copy(
                        boldStyle = stringStyle?.boldStyle?.copy(color = Color.White),
                        italicStyle = stringStyle?.italicStyle?.copy(color = Color.White),
                        underlineStyle = stringStyle?.underlineStyle?.copy(color = Color.White),
                        strikethroughStyle = stringStyle?.strikethroughStyle?.copy(color = Color.White),
                        subscriptStyle = stringStyle?.subscriptStyle?.copy(color = Color.White),
                        superscriptStyle = stringStyle?.superscriptStyle?.copy(color = Color.White),
                        codeStyle = stringStyle?.codeStyle?.copy(color = Color.White),
                        linkStyle = stringStyle?.linkStyle?.copy(color = Color.White),
                    )
                )
            },
    ) {
        Markdown(
            content = markdown
        )
    }
}

private val Color.w get() = copy(red = 1f, green = 1f, blue = 1f)

Which works for everything but regular paragraphs. Pointers much appreciated.

Have you used MaterialRichText or SetupMaterialRichText before calling this MarkdownComponent?
You can find more information on Material theming here

Oh that was exactly what I needed! Thank you!

The link https://halilibo.com/compose-richtext/richtext-ui-material/ says it has a certificate error.


Error while running docker-compose up(installation steps)

After installing Docker and doing the necessary steps
when I run docker-compose up
Getting this error message:
manifest for freecodecamp/mail-for-good:stable not found

Umm, currently the stable tag is not uploaded, sorry about that. And thanks for reporting.
For now, you can change the line 6 in docker-compose.yml and change the tag to latest. It should look like this :
...
...
app:
    image: freecodecamp/mail-for-good:stable
    env_file: .env
...
...

And I am not sure if the latest tag works. We found some issues in the Docker images and kind of concluded that they aren't getting updated. But please do try it. For any issues, you can comment here or mail me and I can provide support

As far as we can tell, this is a bug with Docker Hub. The repo below should automatically update, but it has not done so for two weeks.
https://hub.docker.com/r/freecodecamp/mail-for-good/
We've reported it to Docker, who have said they're looking into it. We won't be able to make any progress on this one until we hear back.

@karuppiah7890 latest resolved that issue but now I'm getting a postgres error
postgres_1  | ERROR:  relation "settings" does not exist at character 204 postgres_1  | STATEMENT:  SELECT "id", "amazonSimpleEmailServiceAccessKey", "amazonSimpleEmailServiceSecretKeyEncrypted" , "amazonSimpleQueueServiceUrl", "region", "whiteLabelUrl", "email", "createdAt", "updatedAt", "userId" FROM "settings" AS "setting";
Am I setting this up right?

Like @AndrewGHC mentioned, there are some problems in the freecodecamp Docker Hub repository due to issues on Docker Hub side, so the Docker images aren't updated. You can try a version that I uploaded recently for using in a server (but it's not the latest), by changing line 6 to
...
...
app:
    image: karuppiah7890/mail-for-good:latest
    env_file: .env
...
...

@karuppiah7890 I followed your instruction, then ran sudo docker-compose up and everything was successful up to a point.
The server booted up, I went to http://localhost:8000, clicked "Sign in with Google", picked an account, then got Internal Server Error.
This is what the logs show:
app_1       |   ############################
app_1       |   #   Mail 4 Good started    #
app_1       |   ############################
app_1       |   # Port: 8080
app_1       |   ############################
app_1       |
app_1       | (node:36) DeprecationWarning: Using the automatically created return value from client.query as an event emitter is deprecated and will be removed in<EMAIL_ADDRESS>Please see the upgrade guide at https://node-postgres.com/guides/upgrading
postgres_1  | ERROR:  relation "users" does not exist at character 112
postgres_1  | STATEMENT:  SELECT "id", "googleId", "picture", "token", "email", "name", "sentEmailsCount", "createdAt", "updatedAt" FROM "users" AS "user";
app_1       | Error
app_1       |     at /usr/src/app/node_modules/passport-google-oauth20/lib/strategy.js:95:21
app_1       |     at passBackControl (/usr/src/app/node_modules/oauth/lib/oauth2.js:132:9)
app_1       |     at IncomingMessage.<anonymous> (/usr/src/app/node_modules/oauth/lib/oauth2.js:157:7)
app_1       |     at emitNone (events.js:91:20)
app_1       |     at IncomingMessage.emit (events.js:188:7)
app_1       |     at endReadableNT (_stream_readable.js:975:12)
app_1       |     at _combinedTickCallback (internal/process/next_tick.js:80:11)
app_1       |     at process._tickDomainCallback (internal/process/next_tick.js:128:9)

Any idea what I'm doing wrong?
Thanks a lot!

Not sure if it's related, but at first I didn't set a PSQL_USERNAME, PSQL_PASSWORD, and PSQL_DATABASE in the .env file and the docker-compose up showed this:
postgres_1  | ****************************************************
postgres_1  | WARNING: No password has been set for the database.
postgres_1  |          This will allow anyone with access to the
postgres_1  |          Postgres port to access your database. In
postgres_1  |          Docker's default configuration, this is
postgres_1  |          effectively any other container on the same
postgres_1  |          system.
postgres_1  |
postgres_1  |          Use "-e POSTGRES_PASSWORD=password" to set
postgres_1  |          it in "docker run".
postgres_1  | ****************************************************

Then I changed the .env to set values to those variables, did a docker-compose rm, then docker-compose up again, and still got the same warning.
Maybe there's something wrong with establishing a database connection and that's why it blows up after receiving the Google oAuth callback?
Please let me know if there's anything I can do to help investigate to contribute a fix for this. Thanks!

@hsribei I wouldn't be able to advise until Docker get back to us. I've spoken with @QuincyLarson about this and might be able to build this manually if I can get permissions to the Hub repo. We're haven't heard back from the Docker team yet. The problem is simply that there was a bit of a frenzy in terms of updating and changing various bits of the app. I'm a fan of @karuppiah7890's suggestion to run docker in development to resolve OS/environment issues once and for all.
While I'm working pretty much flat out this week I'll have a good amount of time next week to address these issues. Watch this space!

@hsribei Are you using my Docker image ? I just tried the whole thing with my Docker image, and it does work. The Postgresql is not really creating issues. I think it's your Google API keys configuration that's creating issues. And even I get a "relation doesn't exist" error, but then the application works. I believe sequelize creates the relation later. The Internal Server Error that you got, it must be from the Google's OAuth system, it will give the details (which you have to see by clicking over an arrow) of why it didn't work. If you still have issues, ping me on Gitter @karuppiah7890 or comment here :)

@ashish-y Does it work for you now ?

Oh sorry, I was trying to comment using my phone and clicked on "close issue" by mistake.
@karuppiah7890 yes! I'm using your docker image file, and it does work. Haven't faced any issues.

@karuppiah7890 I found the error! I had set up the Google+ API on Google API Manager, but after setting it up it remained disabled. I enabled it, then things worked :)
Now on to setting up Amazon SES.

@AndrewGHC I think this issue can be closed, since the Docker images look fine and work well.

I also can't get it to work. Using the stable tag I get the following error at the end.
postgres_1  | ERROR:  relation "users" does not exist at character 112
postgres_1  | STATEMENT:  SELECT "id", "googleId", "picture", "token", "email", "name", "sentEmailsCount", "createdAt", "updatedAt" FROM "users" AS "user";

@arashsoheili Umm, it's an error which occurs while running the application for the first time, but the error doesn't crash the App. You can try the Application in the browser and it will work. Try it out and let us know if any other error pops up!

Yes, it is working. I was confused as the console shows that it's running on port 8080 and that's what I was expecting in my host browser. But it seems that host port 80 is mapped to 8080. I think it would be better if host port 8080 was matched to the container port 8080 instead.

@arashsoheili Yeah, sorry about that. Thing is, we recently changed some configurations (like mapping host port 80 to container port 8080) - so that people can run the system in production without changing config. But we didn't change it in the guide.

I have created an issue #194 to address it. Thanks @arashsoheili for notifying this :smile:

@karuppiah7890 Okay, that's fair. But this can be better solved by having different docker compose files like docker-compose.prod.yml. Please refer to docker compose override for details.

True, I agree @arashsoheili , we actually plan to have even a development version of the docker-compose file for development purposes
The application is still being developed. We will soon consider all this


Revert "Configure AWS"

Reverts Drownedgod85/github-actions-continuous-delivery#2

Approve pull request adding the aws-config.yml and sam-template.yml


chore: Panic on kubelet compatibility annotation

Fixes #N/A
Description

Panic when a nodepool kubelet has not been moved on to the v1 nodeclass kublete configuration

How was this change tested?

N/A

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 10 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 82.326%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/operator/operator.go
0
10
0.0%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.1%

Covered Lines:
5692

Relevant Lines:
6914

💛  - Coveralls

Hi @engedaam, Do we have a migration document regarding the kubelet configuration? To be honest, I couldn't locate it.

It seems that removing this annotation is sufficient. Refer to: https://github.com/kubernetes-sigs/karpenter/blob/b1b45fc9218b30dd952959c5135867088e943c8b/designs/v1-api.md#migration-path


Unit test React components with TestCafe

There is a library by @benmonro, which allows you to write unit tests for React Components with TestCafe.
https://github.com/testcafe-community/testcafe-react-testing
We can consider implementing this behavior in TestCafe in the future.

I could use this feature as well.  Cypress has this library and I was trying to write the same thing for testcafe but ran into the same problem with JSX.
I actually have my library working (link below) but it requires you to use React.createElement. If you guys can add support for jsx and/or tsx, then I can help you keep Cypress where they belong: in the gutter. :)
https://github.com/benmonro/testcafe-react-testing/blob/master/test/index.test.ts#L11

Originally posted by @benmonro in https://github.com/DevExpress/testcafe/issues/4817#issuecomment-628394906
https://github.com/DevExpress/testcafe/issues/4817#issuecomment-628776784

@AlexKamaev Thanks for posting this here, really excited for you to add this feature to testcafe!  I'm going to be checking this issue every day! haha

@AlexKamaev I Have a PR that I will open shortly that adds support for JSX in js files

Hi @benmonro
Thank you for your contribution in TestCafe. We will review your PR.

@miherlosev
Are there any docs or is it possible to do component level testing on an angular application using testcafe?

@prudhvidandamudi, currently, it's impossible to unit-test any components with TestCafe out of the box. So, there is no documentation related to this.


Turnoff automatic folds

Feature description
The automatic fold is scary and makes things "jump around" as if alive, especially the automatic close of folds is something I want to avoid. I would prefer to create/open/close them manually.
I'm really just interested in getting folds to include the newline just before the fold, such that:
# Banana
  Some say this isn't even a fruit.
  Usually yellow.
# Apple
  Another kind of fruit.

Can fold into
# Banana ⋯
# Apple ⋯

With preserved syntax highlight for the unfolded # Banana part.
Describe the solution you'd like
Manual folding where the folded text does not occupy it's own line but rather just a few characters on the previous line.
If it's not configurable with ufo, what part of the code should I study to roll this behavior by myself?
Additional context
No response

https://github.com/kevinhwang91/nvim-ufo/issues/125 is what you should review.

@kevinhwang91 Sorry I'm not sure I could find anything of relevance. The folds are correct with indent provider - I just don't want them to be automatic.

https://github.com/kevinhwang91/nvim-ufo/blob/45aede6f5c4981156df3ddeeae01c4290e43e104/doc/example.lua#L15

Same here, please I do not understand why the folding is automatic..
This is an amazing plugin but we need to Manual folding.


Revisit User.IsSignedIn()

Delete this and in the templates add User.IsAuthenticated()
This is a breaking change so please post it in the Announcements repo.

Tracking via https://github.com/aspnet/Identity/issues/663

What about User.GetUserId() ?

That got moved to UserManager as well.


Handle istioctl error in tests

Seeing failures in some tests:
https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/32952/integ-multicluster-k8s-tests_istio/1395233559119138816
Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

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

Test name
Commit
Details
Rerun command

release-test_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test release-test_istio

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

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

Test name
Commit
Details
Rerun command

release-test_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test release-test_istio

integ-security-multicluster-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-multicluster-tests_istio

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

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

Test name
Commit
Details
Rerun command

release-test_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test release-test_istio

integ-security-multicluster-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-multicluster-tests_istio

integ-security-k8s-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

release-test_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test release-test_istio

integ-security-multicluster-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-multicluster-tests_istio

integ-security-k8s-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-k8s-tests_istio

integ-multicluster-k8s-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-multicluster-k8s-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-security-multicluster-tests_istio

integ-multicluster-k8s-tests_istio
8c194fd4a5a7e307845bc2b1d370d9ef97006507
link
/test integ-multicluster-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
76d8db6719a76642a5743766f89322f1110f9b73
link
/test integ-multicluster-k8s-tests_istio

integ-ipv6-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-ipv6-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
76d8db6719a76642a5743766f89322f1110f9b73
link
/test integ-multicluster-k8s-tests_istio

integ-ipv6-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-ipv6-k8s-tests_istio

integ-pilot-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-ipv6-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-ipv6-k8s-tests_istio

integ-pilot-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-k8s-tests_istio

integ-multicluster-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-multicluster-k8s-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-ipv6-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-ipv6-k8s-tests_istio

integ-pilot-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-k8s-tests_istio

integ-multicluster-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-multicluster-k8s-tests_istio

integ-security-multicluster-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-security-multicluster-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-ipv6-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-ipv6-k8s-tests_istio

integ-pilot-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-k8s-tests_istio

integ-multicluster-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-multicluster-k8s-tests_istio

integ-security-multicluster-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-multicluster-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-k8s-tests_istio

integ-security-multicluster-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-security-multicluster-tests_istio

integ-pilot-multicluster-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-pilot-multicluster-tests_istio

integ-multicluster-k8s-tests_istio
d9e00e5d1d465943a5d34bd06759d1335fe139e4
link
/test integ-multicluster-k8s-tests_istio

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


[DRAFT] AAP-24203: CVE-2024-4067 micromatch: vulnerable to Regular Expression Denial of Service

=== DRAFT ===
=== DO NOT MERGE - NEEDS 4.0.8 RELEASED ===
Jira Issue: https://issues.redhat.com/browse/AAP-24203
Description
Ensure we use a version of micromatch that has the vulnerability fixed.
Testing
n/a
Production deployment

[x] This code change is ready for production on its own
[ ] This code change requires the following considerations before going to production:

Replaced by https://github.com/ansible/ansible-ai-connect-service/pull/1275
A new JIRA was opened to implement the change. The original JIRA had been closed in a previous sprint.


Add other small functions

Add Version, UpdateRuntimeConfig and Status.
Based on #41, #46, #48 and #52.
We could basically achieve our alpha goal after this PR is merged, and more tests are needed after code freeze. @yujuhong @dchen1107 @mikebrow
/cc @abgworrall @matchstick
With this PR, we could pass half CRI validation test now:
Ran 36 of 36 Specs in 93.276 seconds
FAIL! -- 18 Passed | 18 Failed | 0 Pending | 0 Skipped --- FAIL: TestE2ECRI (93.28s)
FAIL

Ginkgo ran 1 suite in 1m33.350842999s
Test Suite Failed
exit status 1

Failed tests are caused by:

Container logging
Exec
Other unsupported features: HostPort, Schema 1 image support etc.

@yujuhong Addressed comments.

@mikebrow Thanks for reiviewing!
@yujuhong Are you ok with this one?

LGTM

@yujuhong Thanks for reviewing!

Apply LGTM based on https://github.com/kubernetes-incubator/cri-containerd/pull/53#pullrequestreview-41350180 and https://github.com/kubernetes-incubator/cri-containerd/pull/53#issuecomment-305326241.


Expose cargo add internals as edit API

Move the manifest editing utilities from cargo add to a new cargo::util::edit module as part of prep work for cargo remove (#10520). No other substantive changes have been made, as this PR is intended only to reduce the refactoring surface area of the implementation of the feature itself.
Background
In cargo edit, there are a number of top-level modules which enable editing of Cargo manifest files (including src/dependency.rs and src/manifest.rs). In #10472, these files were added instead as a submodule of the cargo add command, with the stated intention of breaking them out later for subsequent cargo-edit subcommands. This PR follows through on that expectation.
Decisions
Concerns raised in #10472 regarding this change:

Where should the editing API should live?

Proposal: cargo::ops::edit
Justification: precedent has been set by cargo::ops::resolve and others to have utils shared by multiple ops live in cargo::ops. This is also serves to be a rather conservative API change.
Concerns: the name edit could be overly general for those unfamiliar with the cargo edit project (see alternatives)
Alternatives:

cargo::edit: this seems to me to be too top level, and would confuse users trying to discover the cargo API
cargo::util::edit: if we want to expose this at a higher level, perhaps renaming to act as a counterpart to crate::util::toml
For each of these, replace edit with toml_edit, toml_mut, manifest_edit, manifest_mut, edit_toml, edit_manifest etc. for a more specific module name

Any more specific naming of types reduce clashes (e.g. Dependency or Manifest being fairly generic)

Currently the only thing distinguishing these similarly named types is their path, which the edit module makes more clear
Alternatives: rename to EditDependency/EditManifest, TomlDependency/TomlManifest, etc.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @ehuss (or someone else) soon.
Please see the contribution instructions for more information.

with the stated intention of breaking them out later for subsequent cargo-edit subcommands

Specifically, I wanted to separate out the conversation on "where" from actually merging cargo add.   That delayed conversation has finally come up

:umbrella: The latest upstream changes (presumably #11075) made this pull request unmergeable. Please resolve the merge conflicts.

Before moving on, could we have a module level doc briefing the workflow and prominent items in this module?

Yes. I just resolved the merge conflicts introduced by #11075, and will next add more module documentation to util::toml_mut.

I've added better module documentation for util::toml_mut, and made a minor stylistic changes that I believe should be more in line with cargo's convention. If these are too overzealous, let me know and I can revert them.

Thanks!
@bors r+

:pushpin: Commit d1b041d639e24662ecaa77af13e4d920cf71e499 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit d1b041d639e24662ecaa77af13e4d920cf71e499 with merge 22dd4bf44e7e2b3ecaf087b788c8daae3676b013...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 22dd4bf44e7e2b3ecaf087b788c8daae3676b013 to master...


add custom endpoints for latency testing kb

add custom endpoints for latency testing kb

@srveit I assume I don't need to fix any of these build errors are they are all related to EB kb's which are not in my pull


Use external variable to define lock parameters

@SchedulerLock(
name = "SchedulerLockName",
lockAtMostForString = "${scheduler.lock.at.most}",
lockAtLeastForString = "${scheduler.lock.at.least}")

Thanks again, I will return from vacation in few weeks and then I will look
at it. Sorry, for the delay.
Dne 23. 5. 2017 9:30 odp. napsal uživatel "lathys"<EMAIL_ADDRESS>
:

@schedulerlock(
name = "SchedulerLockName",
lockAtMostForString = "${scheduler.lock.at.most}",
lockAtLeastForString = "${scheduler.lock.at.least}")
You can view, comment on, or merge this pull request online at:
https://github.com/lukas-krecan/ShedLock/pull/21
Commit Summary

Use external variable to define lock parameters

File Changes

M shedlock-core/src/main/java/net/javacrumbs/shedlock/core/
SchedulerLock.java
https://github.com/lukas-krecan/ShedLock/pull/21/files#diff-0 (29)
M shedlock-spring/src/main/java/net/javacrumbs/shedlock/spring/
SpringLockConfigurationExtractor.java
https://github.com/lukas-krecan/ShedLock/pull/21/files#diff-1 (110)
M shedlock-spring/src/main/java/net/javacrumbs/shedlock/spring/
SpringLockableTaskSchedulerFactory.java
https://github.com/lukas-krecan/ShedLock/pull/21/files#diff-2 (31)
M shedlock-spring/src/test/java/net/javacrumbs/shedlock/spring/
SpringLockConfigurationExtractorTest.java
https://github.com/lukas-krecan/ShedLock/pull/21/files#diff-3 (24)

Patch Links:

https://github.com/lukas-krecan/ShedLock/pull/21.patch
https://github.com/lukas-krecan/ShedLock/pull/21.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lukas-krecan/ShedLock/pull/21, or mute the thread
https://github.com/notifications/unsubscribe-auth/ARynxa0Y1_bRleT8IQCqiWT0azO6RGetks5r8t9SgaJpZM4NjsIf
.

Thanks again, will do some changes and release it by the end of the week


⚠ Lucid Solutions has degraded performance

In 97911c9, Lucid Solutions (https://getlucid.solutions) experienced degraded performance:

HTTP code: 200
Response time: 17052 ms

Resolved: Lucid Solutions performance has improved in f46580d.


[feat] Replace Moq with NSubstitute

Details: https://github.com/moq/moq/issues/1372

Albert Monteiro has produced this repository with the required patterns to replace Moq with NSubstitute.
https://github.com/AlbertoMonteiro/moq-to-nsubstitute/tree/main
Bare in mind there are still some patterns missing (some of which have been raised as issues in the repo) so you may have to make adjustments depending on your usage of Moq.
I have also writtten the below tool which leverages these patterns to automatically replace instances of Moq with NSubstitute.
https://github.com/Creastoff/Moqstitute

FYI: https://gist.github.com/Creastoff/280a33caa684db756c7cce91d318b41f

Thanks!


Write segmentation to DICOM

Hello!
What feature or change would you like to see made?
Is it possible for anyone to help out in adding this feature?
As mentioned in this comment, there aren't any tools available in the master branch and saving/editing segmentation is not supported officially yet.
However, I was going through older branches and found this branch.
When using yarn run dev on that branch, the tools are showing up, as shown in the screenshot below:

All I want to do is just save this back into the DICOM, similar to how saving measurements works.
I can send a PR of this feature once implemented, but would need some help to get this done.

Hi @KhyatiMehta3 did you solve this issue? is there an option to save segmentations into dicom in OHIF somehow? thanks!

Hello,
I'm also very interected to have this feature. Is there any plan to do it in a future release ?

Hi @KhyatiMehta3 @oriashush @vdl07, did you manage to store segmentations made in OHIF somewhere else (e.g. DICOM server like Orthanc)?

Hi, my team from AstraZeneca would be very happy to provide some resources to tackle this problem. @kkmehta03 @oriashush @vdl07 @hechmik does anyone know who would be the right person to talk to to get some more background information?
I was wondering if it could be possible to write to a DICOM-SR in a similar way to how measurements are saved via DICOM-SR for OHIF 3.

Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing
Please re-open if issue persists


TypeError: Cannot read property '_leaflet_pos' of undefined

If this is a bug, issue, enhancement request, etc. verify the following before submitting your issue:

[x] Review the README to see if your issue is covered, and pay particular attention to the Getting Help section.
[x] Tell us what version of Leaflet, Angular, and ngx-leaflet you're using
[x] Include code that reproduces your issue

Angular: 10.2.5
Leaflet: 1.7.1
ngx-leaflet: 8.1.0
Stacktrace:
at getPosition (leaflet-src.js:2450)
    at NewClass._getMapPanePos (leaflet-src.js:4439)
    at NewClass._rawPanBy (leaflet-src.js:4235)
    at NewClass.invalidateSize (leaflet-src.js:3559)
    at leaflet.directive.js:162
    at ZoneDelegate.invoke (zone-evergreen.js:364)
    at Zone.run (zone-evergreen.js:123)
    at NgZone.runOutsideAngular (core.js:27485)
    at LeafletDirective.push.TTNa.LeafletDirective.doResize (leaflet.directive.js:160)
    at ZoneDelegate.invokeTask (zone-evergreen.js:399)

MWE: https://stackblitz.com/edit/angular-ngx-leaflet-leaflet-pos?file=src/app/app.component.ts
Explanation of what I'm doing (in my project; similar in MWE):
I have a map in a component which is only possible to see after a specific breakpoint (you have to interact to get it).
If you resize the window to get below it, it gets destroyed (BreakpointObserver using Bootstrap breakpoints).
That's the moment the exception is thrown.
I'm triggering invalidateSize withing that method of the component showing the map:
@HostListener('resize')
onMapReady(map: LeafletMap) {
    setTimeout(() => map.invalidateSize(), 1);
}

Maybe related:
Leaflet#6981,
Leaflet#5774

same issue for me - however it looks more like a leaflet error.
the invalidateSize is also called from https://github.com/Asymmetrik/ngx-leaflet/blob/master/src/leaflet/core/leaflet.directive.ts#L240, which can cause this error

any updates or comments on this?


Deployment error

描述遇到的问题
Failed to compile.
09:25:15.841 |
09:25:15.841 | ./components/Vercel.js
09:25:15.841 | 19:13  Error: Invalid property 'fill' found on tag 'path', but it is only allowed on: svg  react/no-unknown-property
09:25:15.841 | 23:13  Error: Invalid property 'fill' found on tag 'path', but it is only allowed on: svg  react/no-unknown-property
09:25:15.841 | 27:13  Error: Invalid property 'fill' found on tag 'path', but it is only allowed on: svg  react/no-unknown-property
09:25:15.841 | 33:47  Error: Invalid property 'fill' found on tag 'rect', but it is only allowed on: svg  react/no-unknown-property
09:25:15.841 |
09:25:15.842 | info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules
09:25:15.861 | error Command failed with exit code 1.
09:25:15.861 | info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
09:25:15.873 | Error: Command "yarn run build" exited with 1
截图
相关的页面，应该用结果
环境
macOS chrome e.g.22

操作系统: [例如. iOS, Android, macOS, windows]
浏览器 [例如. chrome, safari, firefox]
版本 [e.g. 22]

duplicate as https://github.com/tangly1024/NotionNext/issues/364


Add FontFace API

There a JS API that allows to load a FontFace and be warned when it's there. It's good for performance guys like me who want to display text even if the font file is not there yet.
https://developer.mozilla.org/en-US/docs/Web/API/FontFace
Chrome and Firefox (desktop and mobile)  support it
Provided the name of the API is FontFace, exactly like the @FontFace CSS declaration, maybe the unique key should be FontFace-API ?

The spec refers to this as "CSS Font Loading", which is already covered by http://caniuse.com/#feat=font-loading though I'll make sure "FontFace" is added as a search keyword.

an alias would be enough, that's true, I dit not thought to type "font loading"
Thanks for taking the time to review this issue


not working correctly on 1.21.3

pls fix it

same issue on 1.21.4, i assume its just not updated yet to the game version. if you had Outer Layers on your arms or legs its really wack. Torso seems fine as its relatively central so i dont really see an issue.

I have also had this issue using Fresh Moves, would love to see it fixed!

same (1.21.4)

Same


have error when i set process.env.NODE_ENV = 'production';

below is router code
<Router>
    <Route path="/" component={Application}>
      <IndexRoute component={Home} />
      <Route path="rating" component={ratingApplication}>
      		<IndexRoute component={RatingModify} />
      		<Route path="modify" component={RatingModify}>
      				<Route path="node/:nodeId" component={RatingContent}/>
            	<Route path="step/:stepId" component={RatingStep}/>
      		</Route>
      		<Route path="import" component={RatingImport}/>
      		<Route path="ratetable" component={RateTable}/>
      </Route>
      <Route path="product" component={ProductApplication}>
          <IndexRoute component={ProductImport} />
          <Route path="import" component={ProductImport}/>
          <Route path="maintain" component={ProductMaintain}>
                <Route path="master/:id" component={ProductContent}/>
                <Route path="version/:id" component={ProductContent}/>
                <Route path="element/:id" component={ProductContent}/>
          </Route>

      </Route>
    </Route>
  </Router>

When I set process.env.NODE_ENV = 'production' before router
Uncaught TypeError: Cannot set property 'props' of undefined  issue happen at
 line
    set: function(value) {
      ("production" !== process.env.NODE_ENV ? warning(
        false,
        'Don\'t set the %s property of the React element. Instead, ' +
        'specify the correct value when initially creating the element.',
        key
      ) : null);
      this._store[key] = value;
    }

Uncaught TypeError: Cannot set property 'props' of undefinedObject.defineProperty.set @ ReactElement.js?ab2f:52ReactElement @ ReactElement.js?ab2f:136ReactElement.createElement @ ReactElement.js?ab2f:193ReactElementValidator.createElement @ ReactElementValidator.js?a599:396(anonymous function) @ index.js?9552:44(anonymous function) @ index.js?9552:64(anonymous function) @ index-25d313150bb1c159f554.js:434__webpack_require__ @ common.js:551fn @ common.js:99(anonymous function) @ index-25d313150bb1c159f554.js:7__webpack_require__ @ common.js:551webpackJsonpCallback @ common.js:22(anonymous function) @ index-25d313150bb1c159f554.js:1
code at line

I don't think this is related to React Router. You might be doing something wrong in your Home component... If you give some more details maybe we will be able to help, and I'll reopen this issue. But again I don't think it's related to React Router.


unexpected tag creation on «comma» keypress in different layout

In russian keyboard layout comma key reside «б» char so pressing it while typing word containing «б» char fires tag creation which makes imposible to create tags containg «б» char.

You can use the submitKeys option to set whatever keys you want to be the trigger for tag creation. It just defaults to comma, tab, and enter.


[wip] Numeric, text and datetime types

Numeric types as per #1 :

[x] smallint
[x] integer
[x] bigint
[x] decimal
[x] numeric
[x] real
[x] double precision
[x] serial
[x] bigserial

Text as per #2:

[x] char, varchar
[x] bpchar alias
[x] text

Datetime as per #3:

[ ] timestamp (see comments below)
[ ] date
[ ] time
[ ] interval

Other tasks:

[x] shortcuts in the form of validate[platform]
[x] PR for pg-metadata: see ironSource/node-pg-metadata#2
[ ] Check storage limit of float4/8 on Redshift, if it matches PostgreSQL

@kessler: Postgres and Redshift have different maximum precisions and scales for numeric/decimal types. Whose rules should pg-validate follow?

Also, in Redshift, float is an alias for double precision, nothing more. But in postgres:

PostgreSQL accepts float(1) to float(24) as selecting the real type, while float(25) to float(53) select double precision.

There are similar questions for the text and date types, so I'm gonna hold off on implementing those.

The platform (or should it be "vendor"?) can now be specified like so:
var errors = validate.object({
    foo: 1280000,
    bar: 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa'
}, metadata, { platform: validate.REDSHIFT })

And defaults to postgres.

Wouldn't it be cooler if it was something like:
validate.object() // default or possibly autodetect somehow
validate.redshift.object()
validate.pg.object()
?
On Tue, Jul 7, 2015 at 3:02 PM, Vincent Weevers<EMAIL_ADDRESS>wrote:

The platform (or should it be "vendor"?) can now be specified like so:
var errors = validate.object({
foo: 1280000,
bar: 'aaaaaaaaaaaaaaaaaaaaaaaaaaaaaa'
}, metadata, { platform: validate.REDSHIFT })
And defaults to postgres.
—
Reply to this email directly or view it on GitHub
https://github.com/ironSource/node-pg-validate/pull/4#issuecomment-119182092
.

I usually prefer options for this kind of thing, because it's easier for consumers to build on top of it (passing options along a chain). Could also do both, so that validate.redshift is a shortcut to { platform: redshift }.

Another option is to make a separate pg-validate-redshift module, and have it override types and options where necessary.

I'll go for your hybrid solution, interface + options separate module seems
like using a crane to lift a small box
On Tue, Jul 7, 2015 at 5:11 PM, Vincent Weevers<EMAIL_ADDRESS>wrote:

Another option is to make a separate pg-validate-redshift module, and
have it override types and options where necessary.
—
Reply to this email directly or view it on GitHub
https://github.com/ironSource/node-pg-validate/pull/4#issuecomment-119215775
.

OK. I like that metaphor.

The trouble of merging wouldn't be worth it, so this is now a PR for all the types.
I've described some of the current limitations for datetime types in the readme. @kessler : could you read those and tell me what to prioritize?

Did an experiment today: generated an interval validator from PostgreSQL C source code, with emscripten. Takes some milliseconds to load (subsequent calls are very fast), but every interval format supported by postgres can be validated.

@kessler: one thing I forgot to mention: in some of the tests, an actual postgres database is used to verify validation results. For example, for the range of integer types, a SELECT $1::integer AS res query is executed, once with a valid value for $1, once with the maximum value + 1. The test asserts that when postgres returns an error for the query, our type.isValidValue(..) returns false too. Similar tests are done for numeric types and dates.
And it might be a good idea to run these tests against a Redshift database too, to ensure storage limits etc match. If you can provide credentials, I'll quickly run the tests against redshift and if necessary change the parameters of the validators.

@vweevers I can do that, but I'm not sure that basing tests on actual databases is a good idea... can't you use a mock?

No, these tests verify that computed constants match actual database limits. You can't do that with a mock (what would be the point of comparing one js constant with another). These tests were quite useful to me during implementation - was basically to ensure that my interpretation of the postgres documentation was correct.
Without a database connection BTW, these kinds of tests are skipped. So a database is not required to run the test suite, and because of that, I don't see a downside.

Well a redshift cluster is a costly thing to run. I can let you work on the production cluster but you need to be careful. I'll pass credentials through secure communications