Change json_encoder to be a function, add it to all integrations

This PR supersedes #2213 by adding a json_encoder param to the HTTP views. I've implemented yet for the other integrations, because I want to get feedback on the Django integration first.
Currently due to how django class views work, we need to mark the default encoder function as a static method. Not sure if this is nice, but it allows to make this function a classmethod too.
@jkimbo, @marcoacierno, @BryceBeagle what do you think?
Progress

[x] Django
[x] AIOHTTP
[x] Sanic
[x] Flask
[x] Chalice
[x] FastAPI (?)
[x] ASGI
[x] Starlette (ASGI)

Closes #2213

@jkimbo I'm thinking this should just become a method on the view class instead of being passed. It's not uncommon to subclass the view anyway. What do you think?

@patrick91 that makes sense to me!

@jkimbo I finally updated this, let me know what you think before I do the rest of the integrations 😊

I've updated this PR to add encode_json to all the integrations

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

Thanks for adding the RELEASE.md file!

Here's a preview of the changelog:

This release updates all the HTTP integration and adds a new method called
encode_json that allows to customize the encoding of the JSON response. By
default we use json.dumps but you can override this method to use a different
encoder.
It also deprecates json_encoder and json_dumps_params in the Django and
Sanic views, encode_json should be used instead.

Here's the preview release card for twitter:

Here's the tweet text:
🆕 Release (next) is out! Thanks to @patrick91 for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

HI @patrick91,
I ran int an issue with serializing datetime objects in my Strawberry-Django project, and saw you have commits which address this issue, why didn't your merge them?

@noamsto can you show me the commit you're talking about? 😊
I think everything from here was merged, if you have an issue with datetimes can you open a new issue? I'll be happy to help 😊

Hi @patrick91,
I see that it was all merged,
and then reworked in this PR: #2681
And the default DjangoJSONEncoder was removed in favor of letting the user override the encode_json() method.
Since it's Django related in-anycase, wouldn't it be best to override encode_json() for the Django views at least with the DjangoJSONEncoder?
Since datetime is widely used it means almost every project will need to overwrite this method, and  since it's Django project it might handle other Django related encodings which I can think of right now :)
It shouldn't impact anyone who overerode this function already.
What do you think?
Should I open an issue for that?
I can create a PR as well for that


sink(cdc): reset the txn length counter in every iteration in splitResolvedTxn

What problem does this PR solve?

Issue Number: close #6441
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-all-tests

Codecov Report

Merging #6440 (63ee891) into master (f14088c) will increase coverage by 6.2227%.
The diff coverage is 82.0936%.

Flag
Coverage Δ

cdc
65.9163% <81.4207%> (+0.0229%)
:arrow_up:

dm
?

engine
63.9734% <82.7777%> (+0.5103%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #6440        +/-   ##
================================================
+ Coverage   59.2403%   65.4631%   +6.2227%
================================================
  Files           754        518       -236
  Lines         86755      47167     -39588
================================================
- Hits          51394      30877     -20517
+ Misses        30799      13853     -16946
+ Partials       4562       2437      -2125

/hold

/unhold

/merge

/run-verify-test

/cherry-pick release-6.1


Host association compatibility with ".ssh/config"

When using different ssh keys associated to different profiles, it's usually the norm that people configure their .ssh/config file like so:
Host dev
   HostName dev.example.com
   User john
   Port 2322

Making it such that if the repo is cloned using:
git clone dev:org/project.git
The dev hostname is not correctly resolved and has to be set manually.
This is not a big inconvenience but it would be nice if GitLink is able to parse the config file and understand the real platform that a particular ssh profile/key is being used for.

Reading the correct host from config might be some effort, but if we could at least just override the hostname per project that would be already great.
Right now it uses wrong hostname and there is nothing I can do about it, just have to replace it manually every time.

https://github.com/ben-gibson/GitLink/issues/324#issuecomment-1865274452

Having a hostname override option in the plugin settings would be the easiest solution for this as @Vistritium suggested. This could be a simple input box in the project level settings or a map option at the application level that lets you specify multiple hosts, for example, gitlab.com-kfirfer => gitlab.com


Feature request: Append to today's daily note

Hey @Gitznik,
I was quite busy because of exams for university but I do have time now which I will use to close some issues.
Of course you can contribute, you might want to read the discussion at #11.

I think the idea for custom commands (#11) might be a suitable way to implement this.
Do you know whether the daily note plug-in has a URI scheme for appending to a daily note?

I have no idea but I'll spend some time researching this 👍

Thank you.
I am currently restructuring the code for this extension to make it a bit easier to read and work on.

I just closed issue #11 which requested opening daily notes. This was rather easy to implement by using the Advanced URI plugin for Obsidian.
With the new Daily Note command you will be able to open the daily note in Obsidian or create and open a new daily note if it doesn't exist yet.
Maybe this covers this issue too? I think appending text to a daily note is rather easy when you can quickly open that note from Raycast.

Rapid work there! I'll check it out as soon as it's in the raycast repo and see if it resolves my workflow issue 👍

This feature has now been released.
https://www.raycast.com/marcjulian/obsidian

@marcjulianschwarz far as I can tell, the "daily note" command just opens the daily note in Obsidian.
I'm looking for a prompt to capture text and append to the daily note.


Our coverage is below 90%

We have good test coverage, but we really ought to get it above 90%.

c.c. @JunyuanZheng

Yeah, I see that. I will try my best! :)

We are up nearly 0.2% thanks to you. We just need 100 additional such commits and we will get very close to 100%. ;-)

We made it to 80% thanks to @JunyuanZheng.

I think we can do and should do even better. There might be more bugs in untested code to be found.

@lemire
can we close this issue now ?

Yes. Of course. Even though our logo still lies.
Amazing work.

Great! Regarding the badge, it might be a cache issue with your browser. I had the same problem with mine.


update job submission docs

According to the user interviews, the following contents have been changed:

Changed the tutorial link on the main page.
Emphasized the SSH feature explanation, and created a page for building images with SSH.

Coverage increased (+0.009%) to 48.454% when pulling 56317b7426c4c5c12f634f43f210b692faa35a64 on zhiyuhe/update_docs into 603893ac4f5ae7f80128e7280d325a704b9205ce on master.


src/meson.build:31:1: ERROR: Dependency "xcb-glx" not found, tried pkgconfig and cmake while installing Picom

Installing libxcb-glx0-dev fixed this for me.

Thanks. Added libxcb-glx0-dev to the picom dependency list. Closing this issue now :)


Add mukuljaiswal1205's file.md.

layout: slide
title: "Welcome to our second slide!"
Your text
Use the left arrow to go back!

add title


Another python framework: PyMoDAQ

Hello,
I'm developing a python framework called PyMoDAQ that enables modular interfaces and control of various experimental setups (not only microscopes but also microscopes). I'm organizing in late october, a kind of user/developer meeting and would love to have one of micro-manager (or pycro-manager) "managers" to come and have a presentation and round table about opportunities in between our developments. Would you be interested? You can contact me directly out of this issue (it was the only way I found to reach you :-) ):
<EMAIL_ADDRESS>Cheers

Emailed you


Get keys from header row

Hello! In the case of csv files, can you get the keys from the header row?
My goal is to load a csv file like this:
id, first_name, last_name
1, thomas, kane
2, john, do

And see something like this loaded:
{
        "_index" : "people",
        "_type" : "person",
        "_id" : "IuOkjG8Bny05PI-SGOq0",
        "_score" : 1.0,
        "_source" : {
                "id" : "1",
                "first_name" : "Thomas",
                "last_name" : "Kane"
        }
}

Is this possible?
Thank!

This is the default behavior👋

Ok, I was just getting 0,1,2. I'll look into it further on my side. Thanks for the package. Going to be really helpful!


Associations need to be passed in as arrays

In the example you provide
tea = Tea.find('rec849829')
Brew.create("Tea" => tea, "Temperature" => "80", "Time" => "4m", "Rating" => "5")
This won't work because the tea variable passed in will always go through a map method as part of the serialization for any association. you need to provide it as single element array for it to work.

Yeah, good point! Want to submit a patch for this?

I was unsure how you wanted this to be handled. Do you want to allow non
array inputs?
On Sat, May 27, 2017, 08:51 Simon Eskildsen<EMAIL_ADDRESS>wrote:

Yeah, good point! Want to submit a patch for this?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sirupsen/airrecord/issues/11#issuecomment-304449993,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACfUb5fMqqQeZKqQ4HvinkjJR0C1Wl86ks5r-BxPgaJpZM4NoNNk
.

Yep


Comptype module needed to save PNGs.

from pptx_tools import utils

pptfile = 'test.pptx'
png_folder = './output'
utils.save_pptx_as_png(png_folder, pptfile, overwrite_folder=True)```

output:  Comptype module needed to save PNGs.

but any image saved.

Do you use windows？

Hi @cx4
I am facing same issue "Comptype module needed to save PNGs."
And I am using linux (ubuntu 20.4)
Did you find something?

I use windows - basically you need Microsoft Office. Comptype module is used to communicate with it. Not sure if there is a linux alternative.

pip install comtypes may be a solution, I use win11.


Credential process prompts to use 'granted request latest' when possible

What changed?
Adds a new command granted request latest which is used in conjunction with the credential process and Common Fate.
Now, when you don't have access to a role in AWS and you are using granted, you will be prompted to request access to it.
Also fixes some bugs with the NoAccess hook where the reason prompt was blocked by the spinner when the --reason flag was not used, this would leave the cli hanging waiting for an input.
Pass through the --confirm prompt to better support non interactive usecases
Why?
How did you test it?
Potential risks
Is patch release candidate?
Link to relevant docs PRs

I accidentally added some extra commits which I have reverted


Single control plane / Single network multi-cluster setup instructions not correct

Bug description
The instructions for setting up a multi-cluster setup using a single network as described here ( https://istio.io/docs/setup/install/multicluster/shared-vpn/) are no longer correct for Istio 1.4.
The steps for generating the secret for the remote cluster to be put in the control plane clusters are incorrect. There is a reference to a service account called istio-reader-service-account, however when using the helm based installer, this service account is not provisioned so it doesn't exist. In 1.3, the service account istio-multi was used. Looking at the installation files provided in the tar ball, this new service account is only provisioned using the operator.
Expected behavior
Instructions should be correct for both installers
Steps to reproduce the bug
Follow the docs, but use the helm based installer
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
1.4.1
How was Istio installed?
Using helm
Environment where bug was observed (cloud vendor, OS, etc)
On GKE 1.14. Also testing with Anthos Managed CA

@vadimeisenbergibm


Removing the "," after {{.Email.Body.Signature}} in themes

It would be great removing the "," after {{.Email.Body.Signature}} in the default/flat theme. When using the themes in other languages, the "," is not needed. So the theme would be more flexiable.

+1


[WIP] Write chunk metadata as header in chunks, not into index

Fixes #7 and lays ground work for #130

Tested locally and all seems to work.

@jml split into two PRs: first is just vendoring, second if code.

Just as well you asked @jml!  I hit the following bug in the docker library I vendored in:
https://github.com/docker/docker/pull/28552

Forgot to git add test.  Done now.  Also docker merged that PR so I updated the vendoring.

Thanks!


Interest in backport to Rails 3?

Hey folks, like many, we're still on Rails 3 and will be for the foreseeable. We've backported this gem to work with Rails 3, because after the upgrade (https://github.com/authy/authy-devise/pull/52) this gem got a lot of important security patches that we needed too.
We're happy to share this code, and happy to turn it over for official hosting. I'm not exactly sure the best way to do that -- we could put it on our GitHub and you could link, or any other solution really. Let me know if there's interest in this and we can figure out the best way to make it available.

Hey @j-ro !
After some discussion, we decided that we won't have the bandwidth to host/maintain the Rails 3 backport. What we can do is to link your repo from the Readme here, so that anyone who has the same necessity can quickly find you.
Does that sound good to you?
Feel free to share your repo and we'll link it.
Thanks!

Sounds good! Repo is here, feel free to link it up!
https://github.com/gcosta/authy-devise


Add a follow on mastodon link to the footer on the website.

Type: Website
Issue: -
Breaking change: no
This PR adds a link to https://phpc.social/@phpmd in the footer on the website. It includes a rel="me" part to verify the link on Mastodon.

Codecov Report
Base: 88.65% // Head: 88.65% // No change to project coverage :thumbsup:

Coverage data is based on head (a3a39d9) compared to base (d81cefd).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #997   +/-   ##
=========================================
  Coverage     88.65%   88.65%
  Complexity     1076     1076
=========================================
  Files            96       96
  Lines          2698     2698
=========================================
  Hits           2392     2392
  Misses          306      306

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

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


Design: Signup & login page on figma

UI Prototyping with figma tool figma design.
Please follow the 3 primary colours - #FFBB00 ,#FF6F00 ,#411818 for designing the pages

Content for signup
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Content for login
- Email id
- Password
- Login button

Any queries?
You can comment it here or in Figma file

is the backend for this project is ready?

@himanshujaidka Nope, not yet. Gonna start everything from scratch.

@purnima143 I have made few changes in the Figma file as I felt are required. Please check whether they are visible and suggest any changes if required.

@himanshujaidka Nope, not yet. Gonna start everything from scratch.

okay, like to collaborate with this project and I think we need to start with the backend first and for frontend, another repo should be there, in short different repo for frontend and backend code

@purnima143 Can I use Glassmorphic design for the sign up/sign in pages? I'd be happy to contribute to Kurakoo

@rishitbhojak Ya sure, but do you think glassmorphic will work well here?

@himanshujaidka Actually you right we should make separate repo for front and backend. But because it comes under GSSoc Project 21, I kept both the folder in one repo so people can easily access it.

@purnima143 At least we can try. I'll design and send you a rough prototype here itself. Alright?

is this issue is still open can I work on this

@himanshujaidka Actually you right we should make separate repo for front and backend. But because it comes under GSSoc Project 21, I kept both the folder in one repo so people can easily access it.

okay, but for this issue, we need a frontend as well as a backend to, we need a discussion on this

UI Prototyping with figma tool figma design.
Please follow the 3 primary colours - #FFBB00 ,#FF6F00 ,#411818 for designing the pages

Content for signup
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Content for login
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Any queries?
You can comment it here or in Figma file

login content is the same as signup, @purnima143

UI Prototyping with figma tool figma design.
Please follow the 3 primary colours - #FFBB00 ,#FF6F00 ,#411818 for designing the pages

Content for signup
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Content for login
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Any queries?
You can comment it here or in Figma file

login content is the same as signup, @purnima143

Updated, thank you😊

@purnima143 At least we can try. I'll design and send you a rough prototype here itself. Alright?

yes, please work on the above mentioned Figma file

Hii.. @purnima143    ..can I contribute on it..?

@purnima143 Hey, can I work on this issue for gssoc'21?

UI Prototyping with figma tool figma design.
Please follow the 3 primary colours - #FFBB00 ,#FF6F00 ,#411818 for designing the pages

Content for signup
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Content for login
- Email id
- Password
- Login button

Any queries?
You can comment it here or in Figma file

Hey,I can work on this issue.Assign me if possible.

UI Prototyping with figma tool figma design.
Please follow the 3 primary colours - #FFBB00 ,#FF6F00 ,#411818 for designing the pages

Content for signup
- Name
- Email id
- Password
- confirm password
- Year/sem
- College
- Signup button

Content for login
- Email id
- Password
- Login button

Any queries?
You can comment it here or in Figma file

Hey
Can I design the UI for login and sign-up page. I have a bit of experience in UI/UX design and I think ill be able to do it........Would love to work on this issue!>?

Those who are interested in this issue can start working.I will choose the design.

There are a lot of people for this issue, you can create another issue based on designing related to any features which you think it's needed for the site.

I am a participant in GSSOC 2021. I want to contribute to this issue and work on the same. Please could you assign me this issue as a part of GSSOC 2021.
Thank you.

Hey @purnima143 , I made this signup and login page. Please review and suggest any changes that need to be made. link

Great work😊!! But can you copy your frame and paste herefigma link
Don't forget to comment down your GitHub id and your contribution in the Figma file.

@aritrochakraborty29 @kunal299 @prerna2712 @maanpatel2492  You guys can start contributing😊. But not in this issue, try to create a new issue and once I'll assign you guys can start contributing.

Hey @purnima143 , I made this signup and login page. Please review and suggest any changes that need to be made. link

Great work😊!! But can you copy your frame and paste herefigma link
Don't forget to comment down your GitHub id and your contribution in the Figma file.

I have added my frames in the given Figma file.

@all-contributors please add @thaditi  for design

@purnima143 will this be counted as my contribution for GSSOC'21?


Mldb 1245 query order

Performance improvements.
Dont sort output by rowHash unless required by the user.
Dont specify if you allow BoundQuery to use threads as the ThreadPool manages it better now.
Allow caller to specify is he wants its output in parallel (and any order) or not (and in a stable order).

I very much like the fact that now it is crystal clear if the callback function of the dataset iterator is run in parallel or not.
Things that I am less comfortable with:

I was hoping that we would not need to do a rowHash ordering at any place.  My expectation was either we would output out of order (-> faster) or in the natural order of the dataset (whatever it is).   Do we need to still sort because otherwise the speed of some operations (e.g. join) would be unacceptable?
The naming of the callback function of the dataset iterator, aggregator is somewhat misleading because this is often a function that do not aggregate anything (e.g export.csv).
I could not pin point if the memory usage when processing large datasets was fixed.

@guyd

I removed the sorts that were there to enforce the ordering, but I left the sorts that are there to enforce determinism (like in the rowgenerator, in some cases) and sorts that are necessary algorithmically (like in the join, in the dataset.a = dataset.b case).

Suggestion?

No, its is not.

Regarding the naming of the callback function, I would suggest  RowCallback.

+1


Revise to add a component page.

This current doesn't have a dedicated component page, unlike some of our other elements. If there's interest in putting one together I wouldn't mind doing that.

I was hoping that we might get more insight after Rob Dodson does his next Polycast.  I am starting to think about an SPA I might develop to replace an ageing Microsoft Access application and whether it needs a router.  I think it does, so users get a usable back button and can copy urls to each other - but if it does I need also to be able to intercept a route and redirect the user to a login page if he is not logged in, since I have less control in routing the user where I want him to go.
I can't quite be sure without any documentation what its doing and if I can do something like this (assuming its something like the spa article).
<flatiron-director route="{{route}}" autoHash on-director-route="{{checkLogin}}"></flatiron-director>

template.checkLogin function(e,detail,sender) {
if (detail.route != '/login') {
//see if am logged in, and if not
    this.route = '/login';
//
}


[Bug]: vllm 0.6.3.post1 does not work with response_format

Your current environment

The output of `python collect_env.py`
Collecting environment information...
PyTorch version: 2.4.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35

Python version: 3.12.7 (main, Oct  1 2024, 08:52:12) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-6.8.0-1017-aws-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA A10G
Nvidia driver version: 550.120
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                         x86_64
CPU op-mode(s):                       32-bit, 64-bit
Address sizes:                        48 bits physical, 48 bits virtual
Byte Order:                           Little Endian
CPU(s):                               48
On-line CPU(s) list:                  0-47
Vendor ID:                            AuthenticAMD
Model name:                           AMD EPYC 7R32
CPU family:                           23
Model:                                49
Thread(s) per core:                   2
Core(s) per socket:                   24
Socket(s):                            1
Stepping:                             0
BogoMIPS:                             5599.99
Flags:                                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf tsc_known_freq pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch topoext ssbd ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 clzero xsaveerptr rdpru wbnoinvd arat npt nrip_save rdpid
Hypervisor vendor:                    KVM
Virtualization type:                  full
L1d cache:                            768 KiB (24 instances)
L1i cache:                            768 KiB (24 instances)
L2 cache:                             12 MiB (24 instances)
L3 cache:                             96 MiB (6 instances)
NUMA node(s):                         1
NUMA node0 CPU(s):                    0-47
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Mitigation; untrained return thunk; SMT enabled with STIBP protection
Vulnerability Spec rstack overflow:   Vulnerable: Safe RET, no microcode
Vulnerability Spec store bypass:      Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:             Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:             Mitigation; Retpolines; IBPB conditional; STIBP always-on; RSB filling; PBRSB-eIBRS Not affected; BHI Not affected
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected

Versions of relevant libraries:
[pip3] flashinfer==0.1.6+cu121torch2.4
[pip3] numpy==1.26.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.1.105
[pip3] nvidia-cuda-nvrtc-cu12==12.1.105
[pip3] nvidia-cuda-runtime-cu12==12.1.105
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.560.30
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] nvidia-nvjitlink-cu12==12.6.77
[pip3] nvidia-nvtx-cu12==12.1.105
[pip3] pyzmq==26.2.0
[pip3] torch==2.4.0
[pip3] torchvision==0.19.0
[pip3] transformers==4.45.2
[pip3] triton==3.0.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.6.3.post1
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      0-47    0               N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

Model Input Dumps
No response
🐛 Describe the bug
Environment with lower vllm version
Collecting environment information...
PyTorch version: 2.4.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 20.04.6 LTS (x86_64)
GCC version: (Ubuntu 9.4.0-1ubuntu1~20.04.2) 9.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.31

Python version: 3.12.6 (main, Sep 10 2024, 00:05:17) [GCC 9.4.0] (64-bit runtime)
Python platform: Linux-6.8.0-1017-aws-x86_64-with-glibc2.31
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA A10G
Nvidia driver version: 550.120
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                         x86_64
CPU op-mode(s):                       32-bit, 64-bit
Byte Order:                           Little Endian
Address sizes:                        48 bits physical, 48 bits virtual
CPU(s):                               48
On-line CPU(s) list:                  0-47
Thread(s) per core:                   2
Core(s) per socket:                   24
Socket(s):                            1
NUMA node(s):                         1
Vendor ID:                            AuthenticAMD
CPU family:                           23
Model:                                49
Model name:                           AMD EPYC 7R32
Stepping:                             0
CPU MHz:                              3300.602
BogoMIPS:                             5599.99
Hypervisor vendor:                    KVM
Virtualization type:                  full
L1d cache:                            768 KiB
L1i cache:                            768 KiB
L2 cache:                             12 MiB
L3 cache:                             96 MiB
NUMA node0 CPU(s):                    0-47
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Mitigation; untrained return thunk; SMT enabled with STIBP protection
Vulnerability Spec rstack overflow:   Vulnerable: Safe RET, no microcode
Vulnerability Spec store bypass:      Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:             Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:             Mitigation; Retpolines; IBPB conditional; STIBP always-on; RSB filling; PBRSB-eIBRS Not affected; BHI Not affected
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected
Flags:                                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf tsc_known_freq pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch topoext ssbd ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 clzero xsaveerptr rdpru wbnoinvd arat npt nrip_save rdpid

Versions of relevant libraries:
[pip3] flashinfer==0.1.6+cu121torch2.4
[pip3] numpy==1.26.4
[pip3] nvidia-cublas-cu12==<IP_ADDRESS>
[pip3] nvidia-cuda-cupti-cu12==12.1.105
[pip3] nvidia-cuda-nvrtc-cu12==12.1.105
[pip3] nvidia-cuda-runtime-cu12==12.1.105
[pip3] nvidia-cudnn-cu12==<IP_ADDRESS>
[pip3] nvidia-cufft-cu12==<IP_ADDRESS>
[pip3] nvidia-curand-cu12==<IP_ADDRESS>
[pip3] nvidia-cusolver-cu12==<IP_ADDRESS>
[pip3] nvidia-cusparse-cu12==<IP_ADDRESS>
[pip3] nvidia-ml-py==12.560.30
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] nvidia-nvjitlink-cu12==12.6.68
[pip3] nvidia-nvtx-cu12==12.1.105
[pip3] pyzmq==26.2.0
[pip3] torch==2.4.0
[pip3] torchvision==0.19.0
[pip3] transformers==4.45.0
[pip3] triton==3.0.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.6.1.dev238+ge2c6e0a82
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      0-47    0               N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

How to run the server with docker-compose
...
    command: >
    --host <IP_ADDRESS>
    --port 8000
    --api-key "sk-no-key-required"
    --max-model-len 16384
    --tensor-parallel-size 1
    --gpu-memory-utilization 0.6
    --served-model-name gpt-4o
    --seed 42
    --disable-log-requests
    --enable-prefix-caching
    --model /models/Llama-3.1-Storm-8B-AWQ

Python script
import openai

BASE_URL="http://localhost:8000/v1"
API_KEY="sk-no-key-required"

openai_client = openai.OpenAI(
  base_url=BASE_URL,
  api_key=API_KEY
)

from pydantic import BaseModel

class Players(BaseModel):
  player_names: list[str]

completion = openai_client.chat.completions.create(
  model="gpt-4o",
  messages=[
      {"role": "user", "content": "Give me three baseball players' names and return the result as a JSON object with the key is `player_names`"}
  ],
  temperature=0.0,
  n=1,
  seed=42,
  max_tokens=512,
  response_format={
      "type": "json_schema",
      "json_schema": {
          "name": Players.__name__,
          "schema": Players.model_json_schema()
      }
  }
)
completion.choices[0].message.content

Output from vllm:0.6.1.dev238+ge2c6e0a82
'{"player_names": ["Rickey Henderson", "Mike Schmidt", "Cal Ripken Jr."]}'

Output from vllm:0.6.3.post1
'```json\n{\n  "player_names": [\n    "Mike Trout",\n    "Mookie Betts",\n    "Christian Yelich"\n  ]\n}\n```'

I observed that the server with latest version did not compile the FSM, while the lower one did.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

I was able to reproduce this, but I also observed that it was working as expected on main.  I used git bisect and determined that this was fixed by #9530. It will be included in the next release.
If you try from main and still have trouble, please feel free to reopen this. Thanks!

Hi !
I would like to reopen the issue because it seems that the issue persists.
I tried on different vllm version (0.6.2 to 0.6.4post1):
0.6.2
{'id': 'chat-e3917ed6db284836ad6dc987a1890689', 'object': 'chat.completion', 'created':<PHONE_NUMBER>, 'model': 'path/to/model/Meta-Llama-3.1-8B-Instruct', 'choices': [{'index': 0, 'message': {'role': 'assistant', 'content': '{ "joke": "What do you call a fake noodle?"  The librarian replied, \'It rings a bell, but I\'m not sure if it\'s here or not.\'" \n}', 'tool_calls': []}, 'logprobs': None, 'finish_reason': 'stop', 'stop_reason': None}], 'usage': {'prompt_tokens': 14, 'total_tokens': 81, 'completion_tokens': 67, 'prompt_tokens_details': None}, 'prompt_logprobs': None}

Commande to run th server : vllm serve path/to/model/Meta-Llama-3.1-8B-Instruct
Python code :
import requests

url = "http://localhost:8000"
endpoints = "/v1/chat/completions"

response_format = {
    "type": "json_schema",
    "json_schema": {
        "name": "joke",
        'strict': True,
        'schema': {
            'type': 'object',
            'properties': {
                'joke': {'type': 'string'}
            },
            'required': ['joke'],
            "additionalProperties": False
        }
    }
}

data = {
  "model": "path/to/model/Meta-Llama-3.1-8B-Instruct",
  "messages": [
    {
      "content": "Tell me a joke",
      "role": "USER"
    }
  ],
  "temperature" : 0,
  "response_format" : response_format
}

response = requests.post(url+endpoints, json=data, verify=False)

I also tested with --guided-decoding-backend lm-format-enforcer and completion endpoints with the same results.
Did I miss something ?


Do we support Transferring contracts with app developers to newer versions in APIM 2.2.0

Labels
APIM 2.2.0
Type/Doc
Priority/Highest
Description
Refer to document link [1]. Under usecase section there is a sub point as " Transferring contracts with app developers to newer versions " Is this feature available in APIM 2.2.0. If so please add details on how this is facilitated through APIM 2.2.0
[1]. https://docs.wso2.com/display/AM2xx/API+versioning

Here it is talking about "Require Re-Subscription" which is explained in [2].
[2] https://docs.wso2.com/display/AM2xx/Publish+the+New+Version+and+Deprecate+the+Old


Drop support of nodejs version < 4

Coverage remained the same at 100.0% when pulling 658128e576bdb0e48d93b38ead00a1747e936013 on drop-old-nodejs-support into e70e109896043a7ffc2d68efc0e772d08083a40f on master.


fix(web-component-tester): add location.search to subsuite title

... to distinguish failures when reusing the same HTML suite for shady or non-shady DOM testing. Like in: https://github.com/PolymerElements/iron-behaviors/blob/master/test/index.html#L20
fixes: #3351

@TimvdLippe Thanks for the quick feedback. Looks like I missed some failing tests, sorry.... I didn't get those failures locally. I'm going to review and update ASAP.

Seems like travis is complaining about the esm-amd-loader build . I see this failing in other PR's so I'm assuming this is not to be blamed on this PR. If it is, please let me know and I'll try my best to solve it.
Also I would appreciate any comments if there's something to be improved in the PR to make it ready to be merged.

Any feedback on this? Anything I can do to speed it's merge?

Sorry to be a PITA.... any news here? What are the expectations of getting this merged? This is kind of breaking out tool chain on top of WCT, so I would be thankful if someone, based in experience, and of course without any compromise, can provide any order of magnitude estimate for the merge (days.... weeks..... months.... years...) to allow us to handle our internal expectations appropriately. Thanks!

Any news please...


Update README.md

fixed one spelling and added one more line of request

monster maccaron !!!

why do not you accept that people can improve your work further?

monster maccaron !!!

spell my name correctly mister, or should i say, buddy...

stop chatting and accept my changes right now>>>>

stop chatting and accept my changes right now>>>>

ok fine


Add other ID caches

Theres no harm in also syncing the manga_cache, since thats very easy to do modify the code to do.
Perhaps instead have a cache directory which stores
anime_cache.json
manga_cache.json
person_cache.json (Easy to do through the letter endpoints)
character_cache.json
Clubs/Users becomes harder to do as far as I can see, as theres no easy way to index those.

Added in 5f971d1a755de007627b553ca2c890199b66530d
Still need to write socket, to allow communication from other processes on the same server.

2174a81de87da86ea27f2dab793a146cefbe6b4c
Everything done other than socket code (which is just a nice addition).
Closing here in favor of opening a separate issue with a more descriptive title


Migrate readme yaml

what

updated README.yaml file
add tags and categories
rebuild README.md file
regenerate readme md
add description key in outputs values
add makefile
ignore harness
prepare to open discussions on review

why

need to add categories and tags so we can pull them into the documentation

i ran terraform fmt but it is still failing


Replaced AppName with AppID

Credential switch on Connect-AzAccount require app ID instead app name.
https://docs.microsoft.com/en-us/powershell/module/az.accounts/connect-azaccount?view=azps-5.5.0

@smajchrz Thanks for the feedback. I'm currently updating that article and will reply shortly.

@smajchrz You are correct about the application id needing to be passed. I'm going to close this PR though as there are several changes being made. Thanks much for your help!


Support for Djot

While I understand that the target of this language server is set pretty explicitly to Markdown, the similarities between it an Djot are many. I suspect that the work required to support Djot is minimal. Links and headers are for example almost entirely the same.

Thanks for raising this @kmaasrud! So, Djot looks like a successor language with some tweaks on top of base Markdown. It should be pretty easy to also include djot file extension into the set of markdown extensions and treat djot files as Markdown files. And as long as you avoid using djot features that are different from markdown (e.g. mult-line ATX headers) you should have a pretty smooth experience.
Otherwise, it'd need either a new parser or tweaks to an existing markdown parser. I'm open to this kind of contributions but personally won't have the time to do it myself.


https://wiki.developer.mozilla.org/en-US/docs/Learn/HTML/Introduction_to_HTML/Marking_up_a_letter$history

Request type

[x] Correction or update

Details

There's a grammatical glitch here "...(they should be 10 to the power or 3 and 4, respectively)..." . It should be "to the power of" and not "to the power or".

URL of the page

fixed; thanks for reporting!


Improve display of markers for zoomed out maps

User story

As a User
I want Zoomed out markers to merge into one
So that the map is not so crowded

Acceptance criteria

Different marker merge into one marker
Not to much overlap between markers

Definition of done (DoD)

Added only after week 5
The same for all features
Here goes the project specific part

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

currently working on - working for single map but bad with pinned maps


Undefined local variable or method admin_plugins_contact_form_admin_forms_path

Error happens sometimes.
Demo
Go to site above and refresh a few times. You eventually get a 500 page.

Error is coming from contact_form_helper.rb


Persist Facebook Token

Currently, the Facebook token being used expires in 2 months.

Admin should receive an email when token is about to expire. Then can login with Facebook and refresh token.

Fixed by #42


Recipe to demonstrate loading aus2200 data with chunking

I thought that a recipe demonstrating chunking could be useful. I have made one that shows how to load in a month of AUS2200 data, and calculate a rolling mean (which can be a large computation). Can I add it to the cookbook?

https://atmosphere-cookbook.readthedocs.io/en/latest/Recipes/AUS2200_xarray_rolling.html
nice job!

Thanks! And thanks for helping me with the PR. Keen to contribute more recipes now

And thanks for helping me with the PR.

Anytime!

Keen to contribute more recipes now

That's the spirit! ❤


PR: Add missing link for "Apply as a Mentor" button, Mentorship ->About page

Description

Add missing link for "Apply as a Mentor" button, Mentorship ->About page

https://github.com/WomenCodingCommunity/WomenCodingCommunity.github.io/issues/73
Change Type

[ x] Bug Fix
[ ] New Feature
[ ] Code Refactor
[ ] Mentor Update
[ ] Documentation
[ ] Other

Related Issue

Screenshots

After:

Pull request checklist
Please check if your PR fulfills the following requirements:

[ x] I checked and followed the contributor guide
[ x] I have tested my changes locally.
[ x] I have added a screenshot from the website after I tested it locally

This partially tackles  https://github.com/WomenCodingCommunity/WomenCodingCommunity.github.io/issues/78


Is it supported on APatch?

Is it supported on APatch?

No currently not supported

Can you make it accommodating?

Maybe when I have some more time I can try to research how it is working and see how it works. I have no device with apatch at hand currently.

It's unsupported in the sense that I haven't tested against it. It could work out the box by chance, but there is currently no promise of compatibility.

@skycosth I can confirm that this module works with APatch. I am attaching the screenshots for the same. U do need ZygiskNext module installed in the APModule (as @nitanmarcel pointed out).

Thanks for confirming


WIP: nuttxgdb memory commands performance optimization

Note: Please adhere to Contributing Guidelines.
Summary
This PR is not ready for review for now. because it depends on previous unmerged PRs.
Optimize the performance of memleak, memdump commands.
Tested on a complex project, results are promising.

Command                 Time cost(s)           Time saving(s)   Peformance Boost
                        Before      After
memleak                 39.494172   22.366415   17.127757       1.8
memdump                 41.872441   26.458386   15.414055       1.6
memdump -a 0x1234       28.116294   1.114119    27.002175       25.2
memdump --no-backtrace    N/A       1.114119
memmap                  7.973809    6.836468    1.137341        1.2

Optimize memdump command to include overhead dump.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.

@XuNeo please include Documentation/ for this new tools/scripts


UiComponent.component being set to wrong value

Consider an application [1] with a custom mxml component [2].
I expect the radio buttons in [2] to have a component referencing the custom component, instead they reference the top level application.
The result is that selecting a radio button triggers a RTE.
[1] https://paste.apache.org/x5q2m
[2] https://paste.apache.org/paf3a

@aharui , I don't see any place where UIComponent.component is set, other than in UIComponent.addingChild(). This seems to result in the default top level application being set. Any ideas, where else this should be set?

UIComponent.component is supposed to replace the Flex UIComponent.document.  "document" is a reserved word in JS so we gave it a new name (although maybe we should have named it "mxmlDocument".  You can make a Flex app and see how/where it sets UIComponent.document.  Royale should set UIComponent.component the same way.
However, if the compiler is responsible for generating code to set UIComponent.document, we may want to find a way to set it in AS code in Royale so the compiler doesn't have framework-dependent code generation.  In Royale, MXML-capable components have different APIs in their lifecycle so that could be a good place to set things.

It looks like Flex is doing it in IMXMLObject.initialized(document:Object, id:String)

Taking a quick look, I think I would suggest renaming UIComponent.component to be UIComponent.mxmlDocument and have it return the _mxmlDocument in the base class (which will probably need to become protected).


Fishfood Day

Have at least 3 people test out the toolkit's alpha release candidate in their local environments.
Fix any bugs before release.

Completed


Update README.md

It was really helpful for creating my own data science modules, I think adding how the package will be upgraded will surely help a lot.

thanks, @syeddanish41!


Analyzer accepts calls on Object.

Example:
confuse(x) => new DateTime.now().microsecondsSinceEpoch > 0 ? x : [x];
main() {
  Object o = confuse((int x) => x);
  print(o(2));
}

In this expression, the o(2) should cause a warning, and it doesn't. Also no error in strong mode.

Duplicate of #30987


fix(vite): let rollup know that public assets are handled as externals

🔗 Linked issue
resolves https://github.com/nuxt/nuxt/issues/26379
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This suppresses the warning vite was throwing when using public assets in CSS, by letting rollup know that these assets can be resolved as externals.
We could also consider simply suppressing logs about these particular assets, if you're worried about any negative performance costs.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

Seems something we might expose and reuse from nitro in future for scanning public assets


Add possiblity to annotate addressability.external Ingress

When using addressability.external.method=Ingress the solr operator will create and configure an Ingress object.
Currently there is no way to configure additional annotations which are added to the created Ingress object.
This would be especially useful to enable features like passthrough or re-encrypt.
E.g. when using OKD route.openshift.io/termination: passthrough it is possible to passthrough the traffic to the already encrypted Solr endpoint.

you can add custom annotations to the Ingress at:
customSolrKubeOptions.ingressOptions.annotations

Tried various ways to configure none is working, also adding following yaml with -f as parameter did not work:
customSolrKubeOptions:
  ingressOptions:
    annotations:
      route.openshift.io/termination: passthrough

The created SolrCloud Object contains olny following values:
...
  customSolrKubeOptions:
    podOptions:
      resources: {}
      serviceAccountName: solr

Modifying the created SolrCloud object to
...
  customSolrKubeOptions:
    podOptions:
      resources: {}
      serviceAccountName: solr
    ingressOptions:
      annotations:
        route.openshift.io/termination: passthrough

is working correctly.
So finally the issue here is i could not get the options into the SolrCloud object

My vault, values.yaml does not use the customSolrKubeOptions, i was mislead by the documentation https://apache.github.io/solr-operator/docs/solr-cloud/solr-cloud-crd.html#enable-ingress-tls-termination where in the second part
  ... other SolrCloud CRD settings ...
  customSolrKubeOptions:

is specified; should be removed.


FRQ Reflection and Connection

FRQ 1: 2D Array
In the first FRQ, which centers on 2D arrays, the focus is on the manipulation of 2D arrays, specifically examining the diversity between rows. Although our project does not explicitly utilize 2D arrays, the underlying methodology closely aligns with our database management approach. Consider our CompanyController class:
@Slf4j
@RestController
@RequestMapping("/api/companies") // Base URL for all endpoints in this controller
@CrossOrigin(origins = "*") // Allowing cross-origin requests from any origin
public class CompanyController {

    private final CompanyService companyService; // Service to handle business logic
    private final ModelMapper modelMapper; // For entity-to-DTO mapping

    @Autowired
    public CompanyController(CompanyService companyService, ModelMapper modelMapper) {
        this.companyService = companyService;
        this.modelMapper = modelMapper;
    }

    // Endpoint to get all companies
    @GetMapping
    public ResponseEntity<List<CompanyDTO>> getAllCompanies() {
        List<Company> companies = companyService.getAllCompanies(); // Retrieve companies from the service
        List<CompanyDTO> companyDTOs = companies.stream()
                .map(company -> modelMapper.map(company, CompanyDTO.class)) // Map entities to DTOs
                .collect(Collectors.toList()); // Collect DTOs into a list
        return new ResponseEntity<>(companyDTOs, HttpStatus.OK); // Return DTO list with OK status
    }

    // Endpoint to get a company by its ID
    @GetMapping("/{companyId}")
    public ResponseEntity<Company> getCompanyById(@PathVariable Long companyId) {
        log.info("Attempting to retrieve company with ID: {}", companyId); // Log the attempt
        Optional<Company> company = companyService.getCompanyById(companyId); // Retrieve the company by ID
        if (company.isPresent()) { // If company is found
            log.info("Found company with ID: {}", companyId); // Log successful retrieval
            return ResponseEntity.ok().body(company.get()); // Return company with OK status
        } else { // If company is not found
            log.warn("Company with ID {} not found", companyId); // Log warning for not found
            return ResponseEntity.status(HttpStatus.NOT_FOUND).build(); // Return NOT_FOUND status
        }
    }

    // Endpoint to add a new company
    @PostMapping
    public ResponseEntity<Company> addCompany(@RequestBody Company company) {
        log.info("Attempting to add company: {}", company); // Log the attempt
        Company addedCompany = companyService.createCompany(company); // Create the company
        log.info("Company added successfully: {}", addedCompany); // Log successful addition
        return new ResponseEntity<>(addedCompany, HttpStatus.CREATED); // Return the added company with CREATED status
    }

    // Endpoint to delete a company by its ID
    @DeleteMapping("/{companyId}")
    public ResponseEntity<Void> deleteCompany(@PathVariable Long companyId) {
        System.out.println("Attempting to delete company with ID: " + companyId); // Log the attempt
        companyService.deleteCompany(companyId); // Delete the company
        System.out.println("Company with ID {} deleted successfully" + companyId); // Log successful deletion
        return ResponseEntity.noContent().build(); // Return NO_CONTENT status
    }
}

Within this controller, we implement logic to compare different rows of data, resembling the comparison operations performed on rows within a 2D array. This functionality allows us to conduct diverse operations on our database, akin to the algorithmic procedures outlined in the FRQ.
Moreover, the FRQ's emphasis on array attributes represented as 1D arrays resonates with our project's abstraction techniques. For instance, we often isolate specific attributes of our data objects, similar to reducing the dimensionality of a 2D array to abstract certain characteristics. This abstraction facilitates efficient data handling and analysis, aligning with the principles illustrated in the FRQ.
FRQ 2: List/ArrayList
In the second FRQ, which highlights List/ArrayList algorithms, our PATTask class exemplifies their significance:
package com.nighthawk.spring_portfolio.mvc.linkrAuthentication;

import java.math.BigInteger;
import java.security.SecureRandom;
import java.util.Date;
import java.util.List;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.scheduling.annotation.Scheduled;
import org.springframework.stereotype.Component;

@Component
public class PATTask {
    @Autowired
    PatJpaRepository PATRepo;

    @Scheduled(fixedRate = 10000)
    public void TestTask(){
        List<LinkrPAT> pats = PATRepo.findAll();
        for(LinkrPAT p : pats){
            p.setPAT(generateUniqueToken(6));
            PATRepo.save(p);
        }
        System.out.println("executed");
    }

    public String generateUniqueToken(int length) {
        SecureRandom secureRandom = new SecureRandom();
        byte[] randomBytes = new byte[length / 2];
        secureRandom.nextBytes(randomBytes);
        BigInteger bigInteger = new BigInteger(1, randomBytes);
        String token = bigInteger.toString(16);

        // Ensure the token has the specified length
        while (token.length() < length) {
            token = "0" + token;
        }

        return token;
    }

}

Within this task, we manipulate lists to manage employee-related operations, mirroring the algorithmic thinking highlighted in the FRQ. Whether it's adding new employees, retrieving employee data, or deleting records, our utilization of list-based operations reflects the fundamental concepts outlined in the FRQ.
Furthermore, the FRQ's emphasis on string manipulation and concatenation resonates with our project's handling of textual data. For instance, in real-time input scenarios or search functionalities, efficient string manipulation is crucial for constructing and processing queries. By leveraging Java's string manipulation capabilities, we ensure robust functionality across various aspects of our project, akin to the scenarios depicted in the FRQ.
FRQ 3: Methods and Control Structures/2D Arrays
The third FRQ, focused on Methods and Control Structures, finds reflection in the backbone of our project's backend architecture, facilitating modular design and streamlined execution. In the context of the FRQ, these concepts are exemplified through custom methods and control structures utilized within our project. In the CompanyControllerandEmployeeController we employ a variety of methods and control structures to orchestrate database operations, validate user inputs, and enforce business rules. The modular design facilitated by these constructs ensures code reusability, maintainability, and scalability, aligning with the principles emphasized in the FRQ.
FRQ 4: Interfaces
Although interfaces are not directly utilized within our project, the concepts of abstraction, polymorphism, and contract-based programming are inherent in our design philosophy. While the FRQ focuses on interfaces as contracts for defining behavior, our project achieves similar objectives through class hierarchies, inheritance, and method overriding. Consider the following snippets from our JWT (JSON Web Token) authentication components:
@RestController
@CrossOrigin(origins = "*")
public class JwtApiController {

	@Autowired
	private AuthenticationManager authenticationManager;

	@Autowired
	private JwtTokenUtil jwtTokenUtil;

	@Autowired
	private PersonDetailsService personDetailsService;

	@PostMapping("/authenticate")
	public ResponseEntity<?> createAuthenticationToken(@RequestBody Person authenticationRequest) throws Exception {
		System.out.println(authenticationRequest.getEmail());
		System.out.println(authenticationRequest.getPassword());
		authenticate(authenticationRequest.getEmail(), authenticationRequest.getPassword());
		final UserDetails userDetails = personDetailsService
				.loadUserByUsername(authenticationRequest.getEmail());
		final String token = jwtTokenUtil.generateToken(userDetails);
		final ResponseCookie tokenCookie = ResponseCookie.from("jwt", token)
			.httpOnly(true)
			.secure(true)
			.path("/")
			.maxAge(3600)
			.sameSite("None; Secure")
			// .domain("example.com") // Set to backend domain
			.build();
			System.out.println("returning!");
		return ResponseEntity.ok().header(HttpHeaders.SET_COOKIE, tokenCookie.toString()).build();
	}

	private void authenticate(String username, String password) throws Exception {
		try {
			authenticationManager.authenticate(new UsernamePasswordAuthenticationToken(username, password));
		} catch (DisabledException e) {
			throw new Exception("USER_DISABLED", e);
		} catch (BadCredentialsException e) {
			throw new Exception("INVALID_CREDENTIALS", e);
		} catch (Exception e) {
			throw new Exception(e);
		}
	}
}

In these classes, the interactions are governed by predefined contracts and shared interfaces, ensuring consistent behavior and interoperability. While the terminology may differ, the underlying principles of interface-based programming are evident in our project's architecture.

3.6/4.0
GOOD STUFF!!!!!


How to only return the effected rows moor is watching?

Take this for example:
If I have a method watching a specific note id, i.e. noteId  = 3, in this case how to make moor to only return the effected rows?
Stream<List<NoteEntry>> watchANoteById({int noteId}) {
  return (select(todos)
            ..where((t) => t.id.equals(noteId)))
          .watch();
}

Listen to it in initState() like this:
void initState() {
  watchANoteById(noteId = 3)..listen((row) {
    // How to make this method here to be called and passing the new record(telling me the latest row) only when the note(nodeId = 3) is actually updated or deleted?
    // I found that this method is invoked even when other notes(nodeId != 3) are updated or deleted.
    // Am I missing something?
  }
}

Moor streams may update more often than they need to. This is because they only consider tables for stream updates, so any write to todos will cause the stream to re-fire.
You can fix this by filtering out duplicates manually. First, I'd make the stream return a single item only:
Stream<NoteEntry> watchANoteById({int noteId}) {
  return (select(todos)
            ..where((t) => t.id.equals(noteId)))
          .watchSingleOrNull();
}

Then, you could use  watchANoteById(noteId: 3).distinct() as a stream to remove duplicates.

@simolus3
The watchSingleOrNull()  method isn't supportive in moor I am using, which is the min version of it supporting this method? I use watchSingle() to replace it. It works when I use watchSingle() in my project which uses moor: ^3.4.0 and moor_ffi: ^0.8.0 in pubspec.yaml.

I have another question about watching a whole table. For example:
Stream<List<NoteEntry>> watchNotesRowsModified() {
 // I define this method to watch notes table,
 // but I just want to make Stream.listen() or onData() event to return those rows
 // that are actually updated or deleted by the user,
 // rather than return all rows in notes table
   return (select(notes)).watch();
 }

watchNotesRowsModified().distinct().listen((rows) {
  var r = rows;
  // How to make rows only contains the rows effected,
  // i.e. if noteId = 3 and noteId = 4 are updated by moor update method,
  // I'd like rows here is only about these two rows(noteId = 3 and noteId = 4) passing new updated content in it,
  // other rows that are not updated or deleted in this operation shouldn't be passed along with the rows parameter.
});

By the way, if it is possible to return only effected rows, how can I know which rows are deleted or updated.
Especially, for those rows deleted by the user, I'd like to know if their status are deleted or not?
As you know, difference operation might lead to different logic in project. I don't want to add additional field in sqlite  note table to mark this operation, hoping moor itself supports it.

I use watchSingle() to replace it.

That's fine, watchSingleOrNull was added in moor 4.0 and is only really relevant for null safety.

By the way, if it is possible to return only effected rows, how can I know which rows are deleted or updated.

You can use something like diffutil to find changes (like additions or removals) between two lists.

@simolus3 Please see #168 issue, you mentioned that the next version will support triggers.
I am using  moor: ^3.4.0, I don't know if it supports triggers.
My questions:
a. Can I use triggers to record which rows are being updated, inserted or deleted, and then insert the records into another table, say syncRows table. In this case, my sync method just needs to get records from syncRows table to sync data with our server.
b. I don't find any triggers related document at moor site, do you have any doc about triggers?
c.By the way, can you give me a sample code about how to use moor triggers. Include: insert, update and delete operations.

I am using moor: ^3.4.0, I don't know if it supports triggers.

I think that versions supports triggers, but you must define triggers with SQL in a moor file.

Can I use triggers to record which rows are being updated, inserted or deleted, and then insert the records into another table, say syncRows table. In this case, my sync method just needs to get records from syncRows table to sync data with our server.

Yes, but you'd need a trigger for each operation and table:
CREATE TRIGGER users_created AFTER INSERT ON users BEGIN
  INSERT INTO syncRows (...);
END;

CREATE TRIGGER users_updated  AFTER UPDATE ON users BEGIN
  INSERT INTO syncRows (...);
END;

CREATE TRIGGER users_deleted  AFTER DELETE ON users BEGIN
  INSERT INTO syncRows (...);
END;

b. I don't find any triggers related document at moor site, do you have any doc about triggers?

Triggers are a sqlite3 feature described here. Moor considers triggers for things like stream queries, but doesn't change them in any way so I think there's not need for moor-specific documentation on them.

Can i never watch only update
Or insert individual?

Is watchSingle
Only sink on item if insert or update happened? Not return and touch other Rows?
Only one row affected?

Can i never watch only update

Not directly for a query stream, but you could do tableUpdates(TableUpdateQuery.onTable(yourTable, limitUpdateKind: UpdateKind.update)).asyncMap(yourQuery.get()).

Or insert individual?

I'm not sure what you mean with this, a stream that only listens for inserts with a specific value? You can filter for inserts with the same approach as mentioned above, just use UpdateKind.insert.

Is watchSingle

watchSingle() is just like watch(), expect that it assumes that your query will only return one row. This is why it returns the row directly instead of a List. It watches for the same events as a normal watch() stream.


feat: Support InSet expression in Comet

Which issue does this PR close?

Closes #60.
Rationale for this change

As an optimization of In expression, InSet is also an expression commonly used. We can support it to unblock more queries.
What changes are included in this PR?

How are these changes tested?

Merged. Thanks.


[QUESTION] About DNS Nameserver Spoofability

This has nothing to do with the application dnscrypt-proxy. I just don't know where better than here to get an advised answer. Sorry and thanks!
I've been over at Gibson's DNS Nameserver Spoofability Test and performed the test once with cloudflare (DoH) alone, once with dnscrypt.me (DNSCrypt) alone and the results showed a difference concerning External Ping and DNSSEC Security
With server_names = ['cloudflare']
External Ping: replied, DNSSEC Security: absent

With server_names = ['dnscrypt.me']
External Ping: ignored, DNSSEC Security: supported

My question is simple: what are the implications of External Ping and DNSSEC Security, anything to be aware of?
Many thanks :=)

Thanks for using my server :thumbsup:
Scroll down after the test ends:

External Ping:
This is the classic “ping” (ICMP Echo) test to which most Internet appliances of any type will reply. Ignoring external pings (pings originating from outside the ISP's own local network) renders devices somewhat less visible, especially if the server also ignores external DNS queries.

Try to ping my server...you won't get a response ("Request timeout").

DNSSEC Security:
Modern DNS servers support an advanced, next-generation technology known as “DNS Security” (DNSSEC). This system is not yet widely deployed on the Internet, so support for DNSSEC doesn't mean that a nameserver is spoof resistant. But if it is supported, it's a good sign that your ISP is keeping their nameservers up to date.

Question is cloudflare supposed to be dnssec ready, why the test show absent.

Hello @mibere !

Scroll down after the test ends

Indeed, I proceeded to quickly. Nevertheless, not being an expert, and even if I understand the site's explanations, I remain uncertain about the importance of these details, are they to be taken into consideration when choosing a server rather than another, etc.
I did ping your server, <IP_ADDRESS>, unsuccessfully! Nice when failing is relevant of something good, for once!
dnscrypt.me is fully qualified (DNSSEC, no logs, no filters) and processes requests amazingly fast. At this time I see no difference (or, if, then imperceptible) with cloudflare. Pure DNSCrypt, no DoH, secure, fast, looks nice.
@D1n0Bot , I also had and have in mind your question with this thread. That's why I posted, I don't understand the implications of these results.

I think this DNSSEC test is broken due to query minimization. But Cloudflare definitely does support DNSSEC.

@mazesy Take a look at my website, there are some links for further tests.

@jedisct1

I think this DNSSEC test is broken due to query minimization. But Cloudflare definitely does support DNSSEC.

That's what I had in mind in a way, especially that DNSSEC Resolver Test page has always reported cloudflare as secure. I'll contact Gibson on his Website.
@mibere I have visited your website (owned by Cloudflare by the way), and your links are mst valuable, some of which I knew already. Your dnscrypt.me (HTTPS-secured site at port 8443) make it to https :=)
@jedisct1 feel free (need it be mentioned?!) to close this thread. It's not vital, pure questioning beyond dnscrypt-proxy itself. Thanks.


fix http firehose factory leaky connection in constructor

Description
This PR removes a leaky and unnecessary URL connection from the HTTP firehose factory constructor, pushing it into the openObjectStream method which is the only place it was used.
The bug was originally introduced in https://github.com/apache/incubator-druid/pull/5162

This PR has:

[x] been self-reviewed.

Hmm, URL doesn't seem to care whether or not it is valid to open the connection, see existing serde test.
java.net.URI is final, so can't easily mock to ensure that toURL().openConnection() isn't called either, and would be sort of a lame test without the context of this issue since the constructor will no longer be opening a connection. Any other ideas?

Hmm, URL doesn't seem to care whether or not it is valid to open the connection, see existing serde test.
java.net.URI is final, so can't easily mock to ensure that toURL().openConnection() isn't called either, and would be sort of a lame test without the context of this issue since the constructor will no longer be opening a connection. Any other ideas?

Ah, it seems like the connection actually gets made on the next line (when we look for the header), and errors are suppressed. This makes it tough to write a non-brittle, non-silly test. Yeah, okay, I guess it's fine to skip the test in this case.


Update readthedocs

Once PR #734 is merged, we should update readthedocs to reflect the new documentation build. Currently it shows outdated documentation from 2016.

@manthey shouldn't readthedocs be automatically updated? (this is probably a naive question :) ).

We stopped using readthedocs over two years ago in favor of deploying to Github pages (https://digitalslidearchive.github.io/HistomicsTK/), which should be automatically built and deployed whenever we merge to master.
Unfortunately, travis-sphinx deploy has been failing for a long time.  I just deployed manually, and PR #754 fixes the auto deployment.

Closing this as the gh-pages should once again update on merge to maste.


[oracle] split permutation from sponge functions

In the oracle crate (which probably should be renamed poseidon), we mix permutation stuff with sponge stuff in the same file. It'd be nice to have the permutation stuff in its own module. Specifically in https://github.com/o1-labs/marlin/blob/841dd0f52a04d8b4cb4783109e0dbe4b20eb93f9/oracle/src/poseidon.rs the following functions and types could be moved to a permutation.rs:

apply_mds_matrix
full_round
half_rounds
poseidon_block_cipher (probably should be renamed poseidon_permutation)

Hey I'll take this one!

I looked into changing the name of the crate from oracle to poseidon but it involves a lot of little changes in a lot of little files, along with some instances of stuff like poseidon::poseidon::ArithmeticSponge which looks kinda janky and I was having issues with rust not knowing which was the crate and which was the module that my meager rust chops weren't allowing me solve just yet. Maybe that is for another ticket?

yeah probably not worth changing the name at this point


updated readme for a small typo

Hi @eustlb @andimarafioti @julien-c,
I am Ankan, I work for OpenCV as MLE.
We are planning to write a blog on speech-to-speech for LearnOpenCV, and I found this repo. It's really cool and structured implementation, so maybe I can include this repo for the inference part. Will share the blog with you when we publish, give it a review!
So, while working with it I found a small typo in Readme:
--transformers_lm_model_name google/gemma-2b-it

I think it should be:
--lm_model_name google/gemma-2b-it

So, I just fixed that and opened this PR.
Thanks,
Ankan

Hey Ankan! Thanks a lot for fixing this typo 🤗
The blog post looks really cool, and we’d love to give it a review. Also, don’t hesitate to reach out to me or Andi if you have any questions, we’ll be glad to help!

Thanks @eustlb,
Will ask you if I need any help, and I will share the blog with you when I finish the write-up.
Thanks for your help!

Hey @eustlb , @andimarafioti
Here is the blog link - Introduction to Speech to Speech: Most Efficient Form of NLP
Please give it a review!
And let me know if we can make any improvements to it.


Support EIP-712 as the preliminary standard for meta-tx signing in NEAR

Currently, it is impossible to sign any meta-tx in NEAR. This is a blocker for many use cases.
Please accept EIP-712: Typed structured data hashing and signing as a preliminary standard now and let MyNearWallet implement it.
Any NEAR-specific extensions can be elaborated on later.

Hey, @ethernian thank you for raising this issue.
At the moment there's no standard/NEP for Meta Transactions in NEAR.
The work on a NEP for this has already started and you can follow the progress of it in this PR: https://github.com/near/NEPs/pull/366
Until the work on the above NEP is finalized and merged we can not make any changes on this repo related to meta-tx.

Waiting on https://github.com/near/NEPs/pull/366 before this can be done

Meta-transactions are now supported on chain, we have already added an option to pass the relayerUrl to wallet selector modules: https://github.com/near/wallet-selector/pull/780 , at the moment no other implementation/changes is needed in wallet-selector.
You might find these repos related to fast-auth useful:
https://github.com/near/near-fastauth-wallet/
https://github.com/near/fast-auth-signer


Support 0x order relayer fees

Pull Request Test Coverage Report for Build 2041

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 99.86%

Totals

Change from base Build 2031:
0.002%

Covered Lines:
518

Relevant Lines:
518

💛  - Coveralls


Flag for setting oath-id

Hi,
is there a possibility to set the flag for the oath-id like with the ykpersonalize tool:
ykpersonalize -1 -ooath-hotp -ooath-id.
Best Regards
Axel Hoffmann

Yes, just set a public id (fixed_string)that is 6 bytes (containing OMP, TT and MUI) when programming the credential, and any wanted configuration flags. The relevant flags are:
OATH_FIXED_MODHEX1  0x10  # First byte in fixed part sent as modhex
OATH_FIXED_MODHEX2  0x40  # First two bytes in fixed part sent as modhex
OATH_FIXED_MODHEX 0x50     # Fixed part sent as modhex


[BACKLOG-18294] Websocket Daemon Client Session with kerberos Auth

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/ael/websocket/DaemonMessagesClientEndpoint.java
  engine/src/main/java/org/pentaho/di/trans/ael/websocket/SessionConfigurator.java
  engine/src/test/java/org/pentaho/di/trans/ael/websocket/SessionConfiguratorTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.01%:small_red_triangle_down:
Complexity Change: -1.06%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.ael.websocket.DaemonMessagesClientEndpoint

Branch Change:  + 16.67%
Complexity Change:  + 18.18%
Instruction Change:  + 39.88%
Line Change:  + 31.91%
Method Change:  + 25.00%

org.pentaho.di.trans.ael.websocket.SessionConfigurator

Branch New Coverage: 50.00%
Complexity New Coverage: 58.33%
Instruction New Coverage: 32.79%
Line New Coverage: 38.64%
Method New Coverage: 62.50%

org.pentaho.di.trans.ael.websocket.SessionConfigurator.ClientLoginConfig

Complexity New Coverage: 25.00%
Instruction New Coverage: 11.84%
Line New Coverage: 28.57%
Method New Coverage: 50.00%

org.pentaho.di.trans.ael.websocket.SessionConfigurator.new Header() {...}

Complexity New Coverage: 25.00%
Instruction New Coverage: 30.00%
Line New Coverage: 25.00%
Method New Coverage: 25.00%

org.pentaho.di.trans.ael.websocket.SessionConfigurator.NullCredentials

Complexity New Coverage: 33.33%
Instruction New Coverage: 42.86%
Line New Coverage: 50.00%
Method New Coverage: 33.33%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change:  + 1.22%
Complexity Change:  + .76%
Instruction Change:  + .29%
Method Change: -1.01%:small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2.00%:small_red_triangle_down:
Complexity Change: -1.89%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -4.49%:small_red_triangle_down:
Complexity Change: -5.19%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:
Line Change: -.46%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change:  + 2.04%
Complexity Change:  + 2.04%
Instruction Change:  + .32%
Line Change:  + .81%

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.www.TransformationMap

Branch Change:  + 1.14%
Complexity Change:  + 1.54%

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UISecurityRole

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.54%:small_red_triangle_down:
Instruction Change: -.16%:small_red_triangle_down:
Line Change: -.58%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change:  + .17%

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change:  + .14%

@mkambol for validation. Thks


Etiquette on running a fork

Hi team - I love this project! I have a fork running that I will auto-lookup translations for inputs (I doubt it could get merged back in, it's a bit too opinionated - If I am wrong, let me know and I'll open a PR).
Is there anything you want me to put in the README or LICENSE files (or others) I should be aware of? I won't be putting it on npm, just in a public repo here.
Should I rename it to prevent google indexing it and confusing people?

Hi @lcpriest,
I think the translation idea is great, I will take a look and see how I can incorporate your thing into ember-for-for!


⚠ Bofrost.at has degraded performance

In 6e00d99, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1995 ms

Resolved: Bofrost.at performance has improved in 3d1ef9d.


Consider a 503 response as Solr not running

Using:
sunspot (2.0.0)
sunspot_rails (2.0.0)
sunspot_solr (2.2.0)
sunspot_test (0.4.0)

When running cucumber tests, the first few will fail because:
  RSolr::Error::Http - 503 Service Unavailable
  Error:     {msg=SolrCore is loading,code=503}

  URI: http://localhost:8983/solr/test/update?wt=ruby

This PR changes the solr_running? check to consider a 503 response as not running. After this, my tests correctly wait a couple seconds then work when Solr is ready.

+1 for this pr I have the same issue with the latest version of solr.

Thanks!

Any plans for releasing this version?

Ran into some issue integrating the latest changes into an application and haven't had a chance to circle back to get everything fixed.

Please drop a new version with this patch included.


Consistent filenames?

Looks like the JSON filenames are most of the time in kebab-case.
Some are one word though, e.g. https://github.com/Fyrd/caniuse/blob/master/features-json/matchesselector.json

Mostly methods like matchesSelector, etc. are  converted to lowercase, hyphens are used to separate words for CSS properties and general API descriptions.
But since it's just a general unique identifier for various types of technologies (and because changes would break permalinks) I don't think there's much value in enforcing a particular format after they've been created.


fix(legendary): update require statement for keymaps

Fixes codecompanion actions in legendary

That's a great spot! Thank you so much


Small typo in README.md

See typo here:

This macro closely adheres to Googls Google's recommended channel groupings

https://github.com/Velir/dbt-ga4/blob/f0a45282504cd57cd48571b0aa40c18dc02cd46f/README.md?plain=1#L271

all set. thanks!


density_thresh

HI, thanks for your greate work! I have a little question about the density threshold, why do you use density_thresh = 0.01*MAX_SAMPLES/3**0.5 in training progress, but use density_thresh with 0.01 when test ?
self.model.update_density_grid(0.01*MAX_SAMPLES/3**0.5,
warmup=self.global_step<self.warmup_steps,
erode=self.hparams.dataset_name=='colmap')
@kwea123 looking forward to your reply, thanks!

the density threshold only have effect when training, when testing the occupancy grid is fixed.


gsvd troubles? LAPACK or Julia issue?

A = [-0.33872753963694624 1.124096715384297 -0.6293570718176809; 0.03919190688122216 -0.1300617417823436 0.07281871376668783]
B = [-1.5303758632785613 5.136068273894432 -2.9372584484394606; 0.5364872797265587 -2.4543618264129545 2.0986693466314685]
U,V,Q,C,S,R = svd(A,B)

LAPACKException(1)
Stacktrace:
[1] chklapackerror at /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.0/LinearAlgebra/src/lapack.jl:38 [inlined]
[2] ggsvd3!(::Char, ::Char, ::Char, ::Array{Float64,2}, ::Array{Float64,2}) at /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.0/LinearAlgebra/src/lapack.jl:1868
[3] svd!(::Array{Float64,2}, ::Array{Float64,2}) at /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.0/LinearAlgebra/src/svd.jl:277
[4] svd(::Array{Float64,2}, ::Array{Float64,2}) at /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.0/LinearAlgebra/src/svd.jl:281
[5] top-level scope at In[250]:3

Refiled in https://github.com/JuliaLang/julia/issues/29553


[Q] no custom x-axis (e.g. epoch) in report

I cannot select custom x-axis in the report (only step, wall time etc.). However, in the workspace I can chose different x-axis like epoch. Is this expected or am I doing something wrong?

Hey @timbmg
I tried reproducing a similar scenario with the following code snippet
run = wandb.init(entity="xxx", project="xxx")
with run:
    for i in range(100):
        run.log({"metrics":random.randint(1,10), "custom-x":i+10})

With this setup I can indeed query the custom-x parameter in the panel from reports and workspace.
A workaround for you would be to build the panel from the workspace (as you can query the custom x-axis there) and then move the panel to a report with the Add to Report button.

Hey @timbmg can you share a link to the workspace?  It could be that we are only showing metrics that monotonically increase as options for your x-axis.


Add the wiki to the repo

not really important but it can also be just /wiki for simplicity if you want.

wanna keep it like that in case we add the images in there as well


Add support for NetworkAllocate and NetworkFree in remote driver

Also added an API to return list of builtin network drivers
Signed-off-by: Madhu Venugopal<EMAIL_ADDRESS>

ping @mrjana

Does this replace #1306 ?

@aboch yes.

LGTM


Vulkan Static reflections from dynamic shader sky (ViewportTexture)

Godot version
master (6882890a34528e056f13020798f4a473046990bb)
System information
Windows 10 and Linux
Issue description
I expect reflections from sky to change when the sky changes.
Steps to reproduce
Open project and observe that the background is dynamic but the reflections are static
Minimal reproduction project
ViewportBug.zip

I can reproduce the issue. It looks like the "background" viewports are updated appropriately each frame, but the sky is not updating the radiance map as it doesn't detect that there has been a change.
We will need to add some logic to the sky setup function to detect if it is using ViewportTextures and if so, to update the sky everytime the ViewportTexture changes.
In the meantime, as a workaround, you can force the sky to update every frame by adding + 0.0 * TIME at the end of one of the lines in the shader. While this is a total hack, it works and won't hurt the shader.

Related to https://github.com/godotengine/godot/issues/73400, https://github.com/godotengine/godot/issues/74352 and https://github.com/godotengine/godot/issues/75868.

Related to Decals using ViewportTexture not updated #73400, ViewportTexture doesn't update when used as a Light3D projector texture #74352 and ViewportTexture doesn't work within PointLight2D #75868.

Those issues aren't really related. In all three of those cases the issue is that the texture atlas is not updated when the Viewport is updated. This issue here is specific to skies and it happens because the radiance map itself is not redrawn each frame when a viewport is used as a texture in the sky's shader and that viewport is updated every frame.


[pylint] Implement import-private-name (C2701)

Summary
Implements  import-private-name (C2701) as  import-private-name (PLC2701). Includes documentation.
Related to #970.
Limitations
PEP 420 namespace packages
checker.module_path doesn't seem to support automatic detection of namespace packages (PEP 420). This leads to 'false' positives (Pylint allows both).
Currently, for this to work like Pylint, users would have to manually input known namespace packages.
Allowing private names for type annotations
The Pylint version of the rule allows private names that are used in type annotations. Here is an example from their documentation:
"""Private import can be used as type annotations."""

from argparse import _SubParsersAction

def add_sub_parser(sub_parsers: _SubParsersAction):
    sub_parsers.add_parser("my_subparser")
    # ...

This PR does not check if private names are imported for use in type annotation. Hence, this PR is a draft.
Test Plan
cargo test

Most of the triggers from the ecosystem check seem to be in test suites, which is fine. The others seem to be false positives from not checking if a module is importing something from itself.

Looking through the ecosystem check, the flagged violations are all:

true positives,
issues with PEP 420 (which can be fixed using the namespace-packages setting, or
test suites importing private names for testing reasons (which makes sense)

Related: #6114.

@tjkuson Is there anything I can do to help moving this forward?
I'd really like to have this checker in ruff.
Or is all that's left to to a rebase and the review of @charliermarsh?

@tjkuson Is there anything I can do to help moving this forward? I'd really like to have this checker in ruff.
Or is all that's left to to a rebase and the review of @charliermarsh?

Wow, I forgot about this PR! If I recall correctly, this PR was working but required users to specify namespace packages (otherwise, it would false positive). I'm not a maintainer, but perhaps the team are waiting for #6114 to be solved before accepting this merge?
@charliermarsh I don't mind rebasing and tweaking the PR if you are open to it being merged.

If wanted I'd be open to test it prior merging.
Obviously I'm not a maintainer either, but to my understanding C2701 is working, but has a known limitation for projects with namespace packages, which are not all projects.
Might a note in the docs regarding this behavior be sufficient?
It feels like #6114 would be a nice addition but is a quite orthogonal feature?
Or are there likely changes necessary in this PR once it is merged?
Anyway, thanks for the fast response, let me know when I can help :)

I think we probably do need to support the type annotation thing, since we changed flake8-self to support this: https://github.com/astral-sh/ruff/pull/9036

How does this compare to the flake8-self rules?

I think we probably do need to support the type annotation thing, since we changed flake8-self to support this: #9036

Sure! It should be a simple change if you are still open to the rule being merged.

How does this compare to the flake8-self rules?

private-member-access doesn't check imports. For example,
from foo import _bar

bar = _bar(...)

won't trigger the rule, but would trigger private-import-name.

Thanks @tjkuson. I assume they would both trigger on:
from foo import _bar

bar = _bar._baz(...)

And that only flake8-self would trigger on:
from foo import bar

bar = bar._baz(...)

Is that right?

Yes! Are you thinking of merging them as one rule?

@tjkuson - Eventually maybe... For now, I'm okay with them being separate as long as they don't have overlap.

Rebased to main, everything seems to be working. Checking if an import is used as a type annotation will take longer as I need to make it a deferred rule…

@charliermarsh The PR has been rebased and now forgives private name imports that are used for type annotations!

Sorry, this is blocked on me, I've just been slow to get to it.

Great work @tjkuson.

I'm going to revert this change. It's panicking in the pydantic benchmark run (see https://github.com/astral-sh/ruff/actions/runs/7537305800/job/20516059364?pr=5920).


Recurring expenses are listed multiple times with each occurrence

If there's a recurring expense created in Cospend, then PayForMe will show the recurring item multiple times when the next occurrence occurs.
For example, if there's a monthly expense set on 1 January, then PayForMe show it once on that date. This is good. However, when 1 February, occurs, it will show it twice in the list for 1 February and then immediately below it for 1 January. On 1 March, it will show it three times for March, February, and January. The dates are correct, it's the ordering of the list that creates confusion. This are also no inconsistencies with calculations. It is just confusing to the end user.
My hypothesis is that Cospend updates the the recurring expense on each occurrence date, and then PayForMe sees all of the occurrences again as new in the feed. So, this is probably a simple matter of sorting the PayForMe list by time of expense and not time of update to the expense entry, or something like that. Thanks!

yes, this is a smaller thing because we sort for last updated - assuming users want to see the most recently changed (by other users) bills first. I have no immediate, small fix in mind, but will try to think about one. Alternative would be to switch to sorting by last created or date of the bill.

Since then we added the option to sort by changed or original bill date, so this issue is outdated


Remove regions plz

layoutView.insertChild(selectorString, viewInstance); is much betta. Adding a new concept to show child views, like "regions," is just adding to the conceptual overhead of using this lib

I was just killing them completely locally

Done.

hah nice. I'm gonna look into transitioning off of Marionette by using this view instead.

I would hold off, there's a bit more I want to do before this is ready. I'm porting some of the things we had to do at CF to be cleaner and open source projects.

Mmk. I don't plan to do it anytime in the foreseeable future, anyways. Just a distant goal the next time I'm refactoring that Marionette app... :P


Tarpaulin fails in CI "failed to load manifest"

I can run(build, test) my project locally, but it cannot run in CI.
$ cargo tarpaulin -p network -p libseatalk-sqlite -p sdk -p message_core_database -p panic_monitor -p log
Jul 04 08:00:45.671  INFO cargo_tarpaulin::config: Creating config
Jul 04 08:00:45.691  WARN cargo_tarpaulin::config: Couldn't get project metadata cargo metadata exited with an error: error: failed to load manifest for workspace member /builds/seatalk/cross-platform/seatalk-sdk/core/bridge/node
Caused by:
failed to load manifest for dependency sdk
Caused by:
failed to load manifest for dependency panic_monitor
Caused by:
failed to read /builds/seatalk/cross-platform/seatalk-sdk/core/panic_monitor/Cargo.toml
Caused by:
No such file or directory (os error 2)

Thank you for your reply. I will try to show my example:
This is my .gitlab-ci.yml
image: rust

stages:
  - test

test:
  stage: test
  before_script:
    - cargo install -f cargo-tarpaulin
  script: cargo tarpaulin -p network -p libseatalk-sqlite -p sdk -p message_core_database -p panic_monitor -p log

This is my workspace
[workspace]

members = [
  "core/bridge/ffi",
  "core/bridge/node",
  "core/bridge/jni",
  "core/sdk",
  "core/network",
  "core/sqlite",
  "core/message_core_database",
  "core/log",
  "core/panic_monitor",
  "core/derive",
]

This is 'node' module:
[package]
edition = "2021"
name = "node"
version = "0.0.1"

[lib]
name = "node"
crate-type = ["cdylib", "lib"]

[dependencies]
napi = { version = "2.5.0", features = ["napi4"] }
napi-derive = "2.5.0"
sdk = { path = "../../sdk" }
cfg-if = "1.0.0"

[build-dependencies]
napi-build = "1.2.1"

[features]
default = ["log", "ws", "panic-monitor"]
sqlite = ["sdk/sqlite"]
sqlcipher = ["sdk/sqlcipher"]
ws = ["sdk/ws"]
log = ["sdk/log"]
panic-monitor = ["sdk/panic-monitor"]

This is 'sdk' module:
[package]
name = "sdk"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[build-dependencies]
prost-build = "0.9.0"

[dependencies]
bridge = { path = "../bridge/shared" }
network = { path = "../network", optional = true }
libseatalk-sqlite = { path = "../sqlite", optional = true }
message_core_database = { path = "../message_core_database", optional = true }
liblog = { path = "../log", optional = true }
panic_monitor = { path = "../panic_monitor", optional = true }
prost = "0.9.0"
futures = "0.3.21"
[lib]
crate-type = ['lib']
name = 'sdk'

[features]
default = []
ws = ["network/ws"]
tcp = ["network/tcp"]
panic-monitor = ["panic_monitor"]
log = ["liblog"]
sqlite = ["libseatalk-sqlite", "message_core_database"]
sqlite-react = ["libseatalk-sqlite", "message_core_database/react"]
sqlcipher = ["sqlite", "libseatalk-sqlite/sqlcipher", "message_core_database/sqlcipher"]
test = []

This is 'panic_monitor' module:
[package]
name = "panic_monitor"
version = "0.1.0"

[dependencies]
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0"
backtrace = "0.3.65"
liblog = { path = "../log"}
bridge = { path = "../bridge/shared" }