support for HttpOnly & secure attributes

How can I configure httpie to support the HttpOnly and secure cookie attributes?
With default setup, if I hand-edit HttpOnly into the JSON file generated for a session I get this:
http: error: TypeError: create_cookie() got unexpected keyword arguments: [u'HttpOnly']

Similarly, if I hand-edit in an attribute of "secure" : true for a cookie, it causes httpie to not send the cookie.  If I add "secure" : false, it successfully sends the cookie.

@ebridges I believe it should be httponly

Odd.  I'm seeing cookies being set by Jetty that return HttpOnly:

Yes, but the internal attribute name used by the underlying Python libraries is lowercase httpopnly so it needs to be specified as such.
https://docs.python.org/2/library/cookie.html#morsel-objects

But that link you reference says:

The keys are case-insensitive.

@jkbrzt can you reopen this, or otherwise respond to my point above?  Thanks

@ebridges a few things:

@jkbrzt was correct in closing this
@jkbrzt closed this for the wrong reason
This probably needs better documentation
A better way to debug this would be to serialize a session that had an HttpOnly cookie set on it
Try editing the JSON by hand with "rest": {"HttpOnly": true} because it looks like this is how the underlying library expects this to be passed.

@sigmavirus24

Fair enough, I guess.
What was the correct reason for closing it?
Fair enough.
I believe that's what I provided in my original post...and in the snippet from a Jetty session.
I'd prefer to have the app provide correct behavior rather than hand editing the request.

Just to be clear: the server I'm interacting with responds with HttpOnly.  @jkbrzt claims the server should be sending httponly.  The underlying library claims to be agnostic (i.e. case insensitive).  Httpie claims that it is case sensitive.  I feel stuck in the middle.
I'm about ready to just move on and switch to a more reliable tool; but for the sake of completeness I hope that this can be resolved, since I really like this tool.

@ebridges The HttpOnly flag on cookies is not relevant to a command line HTTP client like HTTPie — it is intended for web browsers. Since it's ignored and only sent from the server to the client, there's no practical reason to store it in the session files.

According to the Microsoft Developer Network, HttpOnly is an additional flag included in a Set-Cookie HTTP response header. Using the HttpOnly flag when generating a cookie helps mitigate the risk of client side script accessing the protected cookie (if the browser supports it).
— https://www.owasp.org/index.php/HttpOnly

The secure flag has always been supported — it gets stored in the session files when the server specifies it, and such cookies are then correctly excluded from non-secure requests.
--
Btw, I wasn't claiming that the server should be sending lowercase httponly. I was just referring to the internal Python variable names which I noticed upon a very brief inspection of the source code. From that I incorrectly concluded that it should be specified in the session file as such. @sigmavirus24's suggestion would have probably been the correct way. But there's really no practical reason (known to me) for storing that flag because it does not get sent back to the server.


Update config file location in flag usage text

The config file location was changed in 305263e4626. Update the flag usage text accordingly.
/cc @Kellel

👍

:+1:


Validator integration

Closes #210
Integration of the Validator contract into the web app. The ErrorList component was created to display the errors returned from our Validator contract and is rendered into the SwapWidget through the Overlay component.
Would appreciate some guidance with the styling as you can see it's not that great ATM, I tried my best to replicate the Figma design.
NOTE: This integration is pre-launch and will be updated with checkSwap calls to the Light contract, the Validator repo is currently deprecated.

I think I need to refactor the Overlay component to include scrolling for better reusability. I can make a branch based on your branch, ok?

Yeah, that's fine by me.

Looks mostly good, a couple of changes I'd suggest:

Would prefer less duplication in LastLook.tsx - suggest renaming getCurrentOrder to getSignedOrder, and then changing the arguments for sendOrderForConsideration to just a locator (string) and an order (LightOrder)
The errors state variable should probably be explicitly validatorErrors or validationErrors for clarity.
Is Validator.ts going to be in @airswap/libraries at some stage?

I haven't checked through the styling / jsx as I assume piers has covered this already!

Will do to the suggested changes and Validator.ts will not be in libraries, next iteration of Light will house the checkSwap function.

Gotcha thanks!

Updated to fix last look swaps, and to skip the validator for wrapper swaps (because of the senderWallet balance check - since removed in the version within the new swap contract).
Now works with all 3 types.


Why Amphenol 12401832E40-2A?

Why did you choose the Amphenol 12401832E40-2A and not some generic* USB 3.1 type C interface?
( Is this more than is needed given that most USB-C cables are not mil-spec? but, it is the commercial line so idk)
Could it be this?
"This connector system is designed to establish future USB performance needs, supporting audio and video signal output with an alternate mode." - https://www.mouser.com/new/amphenol/amphenol-usb-type-c-connector-system/
generic: https://www.aliexpress.us/item/3256805263422607.html

oh, see https://github.com/skuep/AIOC/issues/65 . I guess this is possible.


Fix Asynchronous Permissions

Coverage increased (+0.12%) to 56.87% when pulling b4432c69b084a88e52c016e1f6f68de8824ae64d on camera-permissions into fc2bd43cd79141de8fdfcd16fca3ab84796471a6 on master.


[Upload video API] Add forkid option and fix many issue when upload video

Add forkid option when upload
Reduce timeout when add video to the input
Fix upload limit can't detectable
remove default title when typing title and description

I only use upload video api, hope some body add these thing for edit metadata and other api....

This change is

@hotrungnhan Thanks for this PR, could you please reply to this and this .

@hotrungnhan , if you have tested this PR, please let me know. I will merge this PR.

@hotrungnhan , if you have tested this PR, please let me know. I will merge this PR.

Hold on i still not done with that. This is my commit just to update in my fork repo..
@pierreminiggio i will told you whenever it ok

@pierreminiggio check it one more time and we can merge @fawazahmed0 .

Thanks


ImageNet1K file structure

Vim# bash vim/scripts/pt-vim-t.sh
raise FileNotFoundError(f"Couldn't find any class folder in {directory}.")
FileNotFoundError: Couldn't find any class folder in <my_path_to>/imagenet/train.

My ImageNet1K follows this structure in https://mmclassification.readthedocs.io/en/latest/getting_started.html:
├── data
│   ├── imagenet
│   │   ├── meta
│   │   ├── train
│   │   ├── val

where all the train/val images are in the train and val folders, respectively. The labels are in a single file in the meta folder.
It seems that Vim assumes that ImageNet1K is organized the structure:
imagenet/
    train/
        cat/1.jpg
            /2.jpg
            ...
        dog/1.jpg
              /2.jpg
    val/
        cat/101.jpg
            /102.jpg
            ...
        dog/101.jpg
              /102.jpg
    ...

where a folder stores all images of a class while the folder's name is noted as the class name.

I follow this, it's helpful.
https://huggingface.co/datasets/ILSVRC/imagenet-1k/discussions/17


SPARKC-441 Fix CassandraConnector sessionCache remote and race condit…

…ion issue
If set up a remote C* cluster and submit a job from spark cluster to
connect to remote C* cluster. The CassandraConnectorConf is serialized
and deserialized into different CassandraConnectorConf object for
the same original CassandraConnectorConf. Then each time sessionCache
creates new session for each deserialized CassandraConnectorConf.
Too many new sessions results too many cluster objects, and the
following errors are found.
ResourceLeakDetector: LEAK: You are creating too many HashedWheelTimer instances
To fix the issue, we change sessionCache key from CassandraConnectorConf
to String which is the serialized CassandraConnectorConf in String format.
The second issue is acquire method of RefCountedCache is not synchronized.
Even the cache itself is TriMap, the acquire method still creates two
sessions at certain moment when there are two concurrent requests to
acquire a new session.
To fix the issue, the code of creating new session in acquire method
when the session is not found in the cache should be synchronized.
It does have some performance overhead when new session is created
and serialize/deserialize CassandraConnectorConf

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

+1 with the equals fix

Build against Scala 2.11 finished with failure

Build against Scala 2.10 finished with failure

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success


Jenkins "vagrant up" always looking for vagrant-aws plugin

C:\Users\karthik\Documents\Source\agilityroots-devopsyness\vagrant-toolchains\jenkins>vagrant up
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: C:/Users/karthik/Documents/Source/agilityroots-devopsyness/vagrant-toolchains/jenkins/Vagrantfile
Line number: 0
Message: RuntimeError: vagrant-aws plugin required! Please run 'vagrant plugin install vagrant-aws'

@vishagilityroots Yup.


[Compatibility Request]: Spectrum Azure Dike Trinkets

Is there an existing issue for this?

[X] I have searched the existing issues

Mod name/version/id
Spectrum / Fabric 1.19+
Link
https://www.curseforge.com/minecraft/mc-mods/spectrum
Alternatives
No response
Additional Context
Spectrum adds additional blue hearts in the form of Azure Dike trinkets. You can't see the GUI for them when using this mod and it'd be nice to be able to. The Azure Dike hearts normally sit above the armor icons. Not sure how you would want to integrate the extra hearts but I think at minimum adding like a (+ blue number) to the heart value would be enough. The attached screenshots are of before and after the mod is present which shows how the Azure Dike hearts disappear.

i think i'll just add compatibility for their blue heart bar to render as usual as well as advising them to use the fabric api instead of mixins for the render hook


Sandbox force:org:create should have a fully async option

Is your feature request related to a problem? Please describe.
I can't kick start sandbox creation without having to wait for an arbitrary minimum timeout.
What are you trying to do
I'm working on kicking off developer sandbox creation at the end of a production org deployment, so that we have sandboxes in a clean state for the next round of feature development.
Describe the solution you'd like
sfdx force:org:create --nowait (or something of that nature) returns immediately
Describe alternatives you've considered
A custom plugin replicating the sandbox creation code, but without the polling.

Hey, sorry for the late reply.
Last year we shipped a new command in the sf cli that supports async sandbox creation, see: https://developer.salesforce.com/docs/atlas.en-us.sfdx_cli_reference.meta/sfdx_cli_reference/cli_reference_env_commands_unified.htm#cli_reference_env_create_sandbox_unified


Build Standard CSV Schema

Problem Statement
The Experience Team, we needs a standard schema for COVID reporting in ReportStream so that we can test the hypothesis that reporting entities will be able to comply with a standard schema.
Notes
Utilize Jim Duff's first pass at a standard schema.
Criteria

Schema should contain all fields that are required for LA and all fields required by HHS.

12/17/21 - Rick H completed the initial draft of a document outlining required fields to be reviewed by MikeC, Brick, Joel and Rick.

12/21/21 - The initial draft of the Standard CSV File Requirements is complete.  The Research Team has the document which she'll use in the User Facing documentation.  Brick and Joel may review further to insure that I got everything correct.

Plan to work on the schema today.

12/22/23 - Completed first draft of schema and it works.  Extensive testing is still needed and pending.

Closed.  Duplicate of #3508


Balloon buster

PR Description 📜
In this fun and addictive game, your objective is simple: pop as many balloons as you can before time runs out! Balloons will appear randomly on the screen, floating gently in the air. Your task is to aim carefully and click on each balloon to pop it. Each successful pop earns you points, so be quick and accurate!Keep an eye on the timer ticking down ⏰. You only have a limited amount of time to score as many points as possible.
Fixes #3607

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @manishh12 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit Done. Kindly review it.

@kunjgit  Waiting for your review.

Thank you  @manishh12 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @manishh12,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗


fix: portfolio URL in JavaScript script

:tada: This PR is included in version 1.1.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


About sequence sensitivity

Example
fooRouter.get('/', function *() {
  console.log('origin');
  this.body = {foo: 'bar'};
});

fooRouter.use('/', function *(next) {
  console.log('---middleware');
  this.body = {foo: 'middle?'};
  return yield next;
});

fooRouter.get('/else', function *() {
  console.log('else');
  this.body = {foo: 'else?'};
});

When I request GET /, the code will run the second hook before the first hook, which is not like router of Express (When the first hook catches the route, It will not go through unless it yield next).
Is it designed to give a higher priority to "use" than methods?

I find this unintuitive as well.
As an example, what I would like to do is define a router for the REST API part of my application, /api. Most of the routes will require a user model to be available in the context, but not all. However, with the way the use takes precedence over verbs, I'm unable to define the sequence the way I need. It's not difficult to come up with other examples where the practical solution would be to rely on the order the handlers are defined.

I agree it is confusing. It should be run in order specified. I will push a fix for this.
However, I don't understand @qt911025 or @tjyrkinen use cases. Don't you both want to run authentication and user population middleware before your route handlers?

Good to hear!
I did in fact get around this by introducing more nested routers but if an update on this isn't too difficult to implement it'd be the preferred way.
As to the use case I mentioned above, it was sensible for me to expose a part of a router (handling users in particular) for public use while the rest required authentication so I wanted to place the authentication middleware in between these sections.

Is there any chance of getting this into  5.x?


Persistent Data Feature

Hi, DearImGui saves the state of windows in an .ini file which is automatically loaded upon restart so user interactions with windows persist. This is useful so that user in run-time can move windows to their likings and would not have to repeat the same process upon restart of the application. In my opinion this is more convenient for the purpose of using Slab a debugging/developer tool.
I have tested the changes using the main.lua example and it works fine (moving and resizing of the window).
Also, there are some things I would like to point out. In this prototype PR, the Window.Begin returns the ActiveInstance table when SLAB_SHOULD_SAVE is true for optimization. However, I also think that instead of returning a table for serialization, we could perhaps move the serialization straight at the end of Window.Begin API.
The serializer.lua is currently unoptimized, I will make a follow-up for that.
Let me know what you think.

Hi, thanks for looking into this. This is definitely a feature I have on my TODO list but never got around to it. I will go ahead and move it up in priority.
One thing I have been trying to avoid with Slab is having any library dependencies. This is dependent on the license and potential support the external library will receive in any future releases. However, there is already a method to serialize data in an INI format that could be used for this feature.
Saving the window state is a good beginning, but I am thinking of a more robust system that can be extended to all controls. Things such as tree item open/close states could prove useful in serializing as well.
I will begin working on this feature soon. This pull request can provide some insight on how to go about implementing this feature and will pull in any relevant changes. Thanks again for your hard work!

Just curious, is it easier to encode and decode lua data to INI instead of serializing it to lua table as well?
I agree. All state should be saved as well as with DearImGui, but I haven't got the chance to study the inner workings of Slab to understand those properties so I've started with window states.
Alright thanks for the prioritization of this feature.
Also, in DearImGui it is allowed to set a flag to not allow a window and its inner states to "persist" (I forgot the name of the flag), which should also must be considered in next update.

I suggested INI since there is already support in Slab to serialize shallow tables in this format, which may be enough for saving simple state.
Is the flag you are referring to ImGuiWindowFlags_NoSavedSettings? If so, this is something I can add along with this feature.

Oh. Didn't know Slab has serialization already.
Yes that's it

This feature has been implemented in release v0.6.4.


Review/Feedback/discussion/changes of the Beckn-ONIX stack document

Description
This issue is created to gather review, feedback, and discussion on the Beckn-ONIX stack document. The document outlines the integration and implementation details of the Beckn protocol within the ONIX domain.
Goals

[ ] Collect comprehensive feedback on the Beckn-ONIX stack document.
[ ] Identify any gaps or areas of improvement in the document.
[ ] Discuss potential enhancements and modifications.
[ ] Ensure the document aligns with the project goals and standards.

Expected Outcome

A revised and improved Beckn-ONIX stack document that addresses all feedback and suggestions.
Clear understanding and agreement on the implementation details and integration points.

Acceptance Criteria

[ ] All feedback and suggestions are documented and reviewed.
[ ] The document is updated based on the feedback received.
[ ] Consensus is reached on the final version of the document.

Mockups / Wireframes
NA
Product Name
Beckn
Domain
ONIX
Tech Skills Needed

Understanding of the Beckn protocol
Knowledge of ONIX domain
Experience with integration and implementation of protocols
Technical writing and documentation skills

Complexity
Medium
Category
Documentation
Sub Category
Review/Feedback

Beckn-ONIX Stack documentation consisted of 12 google documents for each sections.
Upon a request from Ravi, merged all the files in one google doc.
Post merging some changes were made to make the document flow consistent.
Format changes were made to make the look of the document consistent.
This is the merged document: https://docs.google.com/document/d/1zvmp6WTBE35SYCzYJR-NnY9MM34vQXeVfUClNfQUrBE/edit#heading=h.m1mzlygez2v3

Meeting with Faiz, Ravi was organised to review the document. Many points were suggested. MOM of the meeting:
Action Items:
Create New Beckon-ONIX-Stack Documentation:
Objective: Develop a new Beckon-ONIX-Stack document from the ground up, focusing exclusively on core functionalities.
Core Functionalities:
Must be consistent across all Beckon-ONIX distributions/SDKs.
These are essential features that Beckon-ONIX cannot function without.
Each core functionality must clearly state whether it is required for BAP, BPP, or both.
Extended Functionalities:
To be included in a future version after the core functionalities are finalized.
Extended functionalities can vary between different Beckon-ONIX distributions/SDKs.
The extended section should illustrate potential use cases to showcase future possibilities and drive adoption.
Core Functionalities Guidelines:
Definition: A functionality is deemed core if Beckon-ONIX cannot operate without it; otherwise, it is considered extended.
Requirements:
Include a statement for each core functionality, indicating that it is a required (MUST HAVE) feature.
Example Core Functionalities:
L2 configuration to also manage the registry.
Beckon-ONIX must detect and adapt to new L2 configurations.
The platform should support the simultaneous existence of multiple L2 configurations.
Selection of an appropriate L2 configuration should be based on the request context.
Add a section discussing the pros and cons of synchronous and asynchronous modes, including example use cases for webhook integration.
Next Steps:
Begin drafting the new document, focusing on core functionalities as outlined above.
Plan the inclusion of extended functionalities after the core features are fully defined and approved.

A meeting with Faiz and Venkatesh was organised as a followup to the previous review and feedback meeting.
As per the action item, a new Onix stack doc was created:
https://docs.google.com/document/d/12A97OUeBNSvsnzCWOWay9FkNUdDmvwMwnvZ-JKv6IQM/edit#heading=h.m1mzlygez2v3


GH-36329: [C++][CI] Use OpenSSL 3 on macOS

Rationale for this change
GitHub Actions self-hosted runner for macOS has
/usr/local/include/openssl/ provided by OpenSSL 3 (openssl@3). Our include paths have ... -isystem /usr/local/include -isystem<EMAIL_ADDRESS>.... It means that /usr/local/include/openssl/... is used for #include <openssl/...>.
If we mix OpenSSL 3 headers and OpenSSL 1.1 libraries, we may get some problems such as a link error.
What changes are included in this PR?
This uses OpenSSL 3 instead of OpenSSL 1.1 because GitHub Actions self-hosted runner for macOS provides OpenSSL 3 by /usr/local/include/openssl/. Note that $(brew --prefix openssl@3)/include isn't linked as /usr/local/include/openssl` by default. So I think that Homebrew GitHub Actions self-hosted runner for macOS does it explicitly.
Other solution: Unlinking /usr/local/include/openssl by brew unlink openssl@3. But there is no reason to use OpenSSL 1.1 for us. So this PR doesn't use this solution.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

Closes: #36329

+1
The "C++ / AMD64 macOS 12 C++" is still failing but it's caused by #36331/#36248 .

@github-actions crossbow submit java-jars

Triggering java-jars because they seem to be failing on the nightlies due to this reason:
 Undefined symbols for architecture arm64:
  "_EVP_MD_get_size", referenced from:
      gandiva::gdv_hash_using_openssl(long long, void const*, unsigned long, evp_md_st const*, unsigned int, int*) in libgandiva.a(unity_3_cxx.cxx.o)
ld: symbol(s) not found for architecture arm64

There seems to be a lot of failures around gandiva on the MacOs java-jars job. I am not sure if they are related:
 The following tests FAILED:
	 20 - arrow-compute-scalar-type-test (Failed)
	 38 - arrow-substrait-substrait-test (Failed)
	 44 - arrow-acero-asof-join-node-test (Failed)
	 78 - gandiva-internals-test (Failed)
	 80 - gandiva-filter-test (Failed)
	 81 - gandiva-projector-test (Failed)
	 82 - gandiva-projector-build-validation-test (Failed)
	 83 - gandiva-if-expr-test (Failed)
	 84 - gandiva-literal-test (Failed)
	 85 - gandiva-boolean-expr-test (Failed)
	 86 - gandiva-binary-test (Failed)
	 87 - gandiva-date-time-test (Failed)
	 88 - gandiva-to-string-test (Failed)
	 89 - gandiva-utf8-test (Failed)
	 90 - gandiva-hash-test (Failed)
	 91 - gandiva-in-expr-test (Failed)
	 92 - gandiva-null-validity-test (Failed)
	 93 - gandiva-decimal-test (Failed)
	 94 - gandiva-decimal-single-test (Failed)
	 95 - gandiva-filter-project-test (Failed)
	 96 - gandiva-projector-test-static (Failed)

gandiva-* tests were crashed without backtrace... So we can't debug this. We may be able debug this by ssh to the host.
Anyway, how about tracking the failures as a separated issue? Because Gandiva tests in "AMD64 macOS 12 C++" aren't failed.

Created: #36404
I'll merge this.


feat: paste from clipboard

Proposed Changes
add paste from clipboard event/action.
Testing

https://user-images.githubusercontent.com/6134774/193421338-1d3997bf-fc91-45bc-9e46-2dcff6d4318f.mov
Issues Fixed
https://mobile-dev-inc.slack.com/archives/C041FU72T54/p1664560340755779

Any update here?


我可以用win11系统来跑这个实验吗

我可以用win11系统来跑这个实验吗

Thank you for your attention~! We have not tested the codes on Windows, but you can install the required packages on your PC and try running the code.


feat(RingTheory/TensorProduct): the universal property of the tensor product of algebras

The construction used for CliffordAlgebra.ofBaseChange generalizes into a universal property for building arbitrary algebra morphisms out of the tensor product.
It turns out this is a known result; it is mentioned at https://en.wikipedia.org/wiki/Tensor_product_of_algebras#Further_properties which cites the linked reference, though the version added by this PR is heterobasic like the rest of this file.

[x] depends on: #7403

You're right, we did already have the commutative case (twice!). I've redefined it in terms of the non-commutative case.

Thanks!
bors merge


DPM-71 Pass data argument to handler of CUSTOM_SURFACE modal

Problem
When opening a modal through SDK, there is a need to pass extra properties from Panel surface
await sdk.execute(Command.OPEN_MODAL, {
	type: Modal.CUSTOM_SURFACE,
	action_id: 'Details',
	data: {
		id: 'xyz'
	}
});

Changes

add new type of modal for OPEN_MODAL command - CUSTOM_SURFACE
allow data option for CUSTOM_SURFACE modal
reformat tables

Neodora Analysis ⚠
 ⚠ Missing cache in Github Workflow

Message: Github workflow should be using caching when step uses actions/setup-node

Workflow file
Job name
Step index
Actual cache value
Expected cache value

on-commit.yml
lint
2
Not found
npm

🔍 How to fix it

 ⚠ Unknown tribe name

Message: The property owner in repository.json must be a valid tribe name present in mission-tracking
Repository file: repository.json

Owner status

'GRAVITY' is an unknown tribe name

🔍 How to fix it

Commit 369716f  Job 24675


[Bug]: Missing payload (request) using host app

Steps To Reproduce
mkdir -p /tmp/appscope/payload
SCOPE_PAYLOAD_DIR=/tmp/appscope/payload scope run --payloads -- host -v -t a cribl.io
# observe that in the /tmp/appscope/payload you will have only *.in file missing *.out file

Environment
- AppScope: 1.4.0
- OS: Ubuntu 22.04
- Architecture: x86_64
- Kernel: 5.18

Requested priority
None
Relevant log output
No response

The root cause:

In the case of the host application [1], the host app uses libuv to perform the DNS lookup.
uv__udp_sendmsg uses the syscall to perform the sending message operation.The decision of whether to use syscall or sendmsg is based on checking the value of the uv__sendmmsg_avail variable in uv__udp_sendmsg [2].
uv__udp_recvmsg uses recvmsg to perform the receiving message operation. The decision of whether to use syscall or recvmsg is based on checking the result of uv_udp_using_recvmmsg [3].
The above explains why libuv uses syscall for sending messages and recvmsg for receiving messages.
Additionally libuv check support for recvmmsg() and sendmmsg() using uv__udp_mmsg_init with calling uv__sendmmsg(s, NULL, 0, 0) [4]

The stack traces for the and uv__udp_sendmsg and uv__udp_recvmsg:
```
[#0] syscall()
[#1] uv__sendmmsg(...)
[#2] uv__udp_sendmmsg(..)
[#3] uv__udp_sendmsg(...)
[#4] uv__udp_send(...)
[#5] uv_udp_send(...)
```

```
[#0] recvmsg
[#1] uv__udp_recvmsg
```

Ref:
[1] https://github.com/isc-projects/bind9
[2] https://github.com/libuv/libuv/commit/3d7136639a39db46bc4a9074922559a564e49514
[3] https://github.com/libuv/libuv/commit/6b5aa669db4d57231e21b1ee97c63a06167e117e
[4] https://github.com/libuv/libuv/commit/3d7136639a39db46bc4a9074922559a564e49514


Feature/error handling

Critical Changes
Changes

Error Handling has been modified to allow exceptions messages to display to users. Previously exceptions were caught, an error log was stored, and an email was sent to an admin or other configured user. With these changes, errors that happen during record processing will be exposed to the user or other callers in the system.
Common error messages will be caught and re-written to be more clear, including required field errors and custom validation rule errors.
Transactions that happen without a user present including future calls, and scheduled APEX etc will be monitored with a scheduled job. When errors occur they will be logged and an Email will be sent to the configured user.

Issues Closed
Fixes #284
Fixes #332

I don't mean to put pressure on you @davidjray, or anything like that, but let me summarize what we need in order to move ahead with this PR:

Clarification on removal of re-entrant flags
Branch that contains all changes from this branch  plus code that would help to test these changes
Instructions on how to test using the aforementioned branch (please add instructions as comment in PR or reference them from PR)

Thank you!

 As I am investigating the removal of the reentrant flags, I am believing more and more we have a true recursion problem that was silently being caught.

@davidjray Is this ready for review?

@ceiroa here is my response to your points from your previous comment.

Clarification on removal of re-entrant flags

This was done because I was running into errors while testing. After further investigation, the cause of the errors in the updated unit tests was due to self-referencing Affiliation Triggers. The affiliation trigger was performing DML on Contacts in the before trigger context. This was happening in both AFFL_ContactChange_TDTM and AFFL_AccChange_TDTM. This caused the errors. I switched the updates of the related objects to the After Update context, and that has resolved the problems.

Branch that contains all changes from this branch  plus code that would help to test these changes

This has been pushed as the branch TEST-error-handling

Instructions on how to test using the aforementioned branch (please add instructions as comment in PR or reference them from PR)

I will share a QUIP doc containing the instructions

I am continuing to do a sanity check on the remaining unit tests and I will notify you when this is ready for review.

@ceiroa take a look now, this should be ready for review. Thanks!

Great job, @davidjray. I had a question about a method in the install script being changed to global. But since most other methods in that class are already global (in most cases to be able to call them from the anonymous window), I think it's ok.

@ceiroa I made it global while I was developing this and testing the during development. I decided to leave it so it can be called from the anonymous apex window. Much like other methods in the class.


Collect and send more accurate peer addresses

Motivation
To improve the quality of its address book, Zebra can collect canonical addresses from inbound peers.
To get inbound connections, Zebra can send its own address to its peers when they ask for addresses.
Solution

add canonical addresses from inbound peers to the address book
send our own address when peers ask for addresses

The code in this pull request has:

[x] Documentation Comments
[x] Integration Tests
[x] Manual Tests

Review
Anyone can review this peer address book quality improvement. It is a routine enhahcement.
Related Issues
This fix is based on #2121. GitHub will automatically rebase it to main once #2121 merges.
These address quality improvements are part of the follow-up to #2120. They implement part of #1892.

Marking as draft until #2121 merges

One small q before approval


getting image data from graphics context

Here is an example I would like to work in Elm
    var width  = 960;
    var height = 500;

    var canvas = document.getElementById('vincent');
    var ctx = canvas.getContext("2d");
    canvas.width  = width;
    canvas.height = height;

    var image = new Image();
    image.src= "starry-night.jpg";
    image.onload = function () {
        ctx.drawImage(image, 0, 0);
        image = ctx.getImageData(0, 0, width, height);
    }

Graphics has a analogue of ctx.drawImage(...) now I am asking about getting the image data back out.
Can we call ctx.getImageData(...) from Elm and get back a List Int as the image data returns?
Since I have experimented, I noticed it should be List Color or List List Color (a 2D array).  2D data might be stored as an underlying quadtree.  I don't know.

I'll say a little bit more.  Reading through Collage API, there are ways to add things into Collage, but not to query for information from collage.  In particular, the color at a given pixel from 2D context.  ctx.getImageData() is really the only query you can make in the 2D graphics context anyway.


Time picker shows different time on select

Hello, I am using the custom date time picker.
I have populated the time in the input field, when I click on the text box it shows different time in Time Picker. If text box is empty it shows the correct time. What am I doing wrong while assigning value to textbox. I am assigning the time this way
this.myTime  = new moment(new Date(), 'M/D/YYYY h:mm:ss a');

It works fine in chrome. I am facing this issue only in IE

The Demo page examples works fine to me in both Chrome and IE.
Please use StackBlitz to build an app and show the issue you mention about
I would close this until it is confirmed as an issue.


Ethernet mgmt interface

Fixes #6640

Codecov Report

Merging #6987 into master will decrease coverage by 0.3%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master    #6987      +/-   ##
==========================================
- Coverage   55.66%   55.36%   -0.31%
==========================================
  Files         463      458       -5
  Lines       44622    43454    -1168
  Branches     8302     8148     -154
==========================================
- Hits        24839    24058     -781
+ Misses      16388    16091     -297
+ Partials     3395     3305      -90

Impacted Files
Coverage Δ

subsys/net/ip/l2/ethernet/ethernet.c
47.23% <0%> (ø)

include/net/ethernet.h
40% <100%> (+6.66%)
:arrow_up:

subsys/net/ip/net_if.c
54.49% <100%> (-0.79%)
:arrow_down:

tests/net/vlan/src/main.c
92.92% <100%> (ø)
:arrow_up:

subsys/net/ip/net_context.c
63.51% <0%> (-3.44%)
:arrow_down:

subsys/net/ip/net_tc.c
88.23% <0%> (-2.68%)
:arrow_down:

subsys/net/ip/net_pkt.c
47.15% <0%> (-1.61%)
:arrow_down:

subsys/net/buf.c
53.25% <0%> (-1.24%)
:arrow_down:

subsys/net/ip/utils.c
79% <0%> (-0.59%)
:arrow_down:

... and 12 more

Continue to review full report at Codecov.

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


Reading properties from JSON is not working

Hello, Team!
In section 4 of https://www.baeldung.com/spring-boot-json-properties#propertysource, I run into the same problem as written in the text. After finishing steps 5, 6. I still get the same error message. Is there something I should add to the project apart from these classes?

Hi @Omcsesz,
I just followed the steps (section 4 & 5) on the new sample project generated on https://start.spring.io/.
All the properties are processed correctly.
Clearly, you must be missing something in your config.
Please double check all the steps described in the article.
Hope this helps.


Fix typo for Japan region deployment

Problem
Typo for Japan region at 03-sap-system-deployment.
Solution
fix typo.
Tests
Don't need.
Notes
I expect quick fix this due to timeline for customer.

These changes should be included in SDAF - Release 3.9.0


Facing this issue while training model

(py36) (localizer-main-VK8Pfd4G) C:\Users\internship\Desktop\localizer-main>python localizer\train.py models\maps\config.json
Filter "validate" contains 4 data elements
Traceback (most recent call last):
  File "localizer\train.py", line 981, in <module>
    trainer.run()
  File "localizer\train.py", line 374, in run
    self._read_dataset()
  File "localizer\train.py", line 441, in _read_dataset
    total_obj_count_by_cat[obj.category] += 1
KeyError: 1

I have made a anno dataset, it fails shows the following error when running

Oh sorry this was my fault as I started labelling from category 1 instead of 0


Change scrubber network event foam to come out of scrubbers (Fix #1144)

About the PR
Fix #1144
Media

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

Changelog

:cl: jick

fix: Scrubber backpressure now comes out of scrubbers instead of vents

quick turnaround


Configuration of automatic builds on Docker Hub is broken

Due to change of Docker Hub's permission system with their GitHub integration the configuration of automatic builds is now broken. Fixing this requires "Owner" permissions.

https://cloud.docker.com/u/vshn/repository/docker/vshn/modsecurity/builds/edit

@tobru, if you can spare a minute... :+1:

Just for the record, https://github.com/SpiderLabs/owasp-modsecurity-crs/issues/1344 very probably has the same root cause.

Thanks @maetthu-indermuehle for fixing it! :medal_sports:


Improved DTD handling

A DTD is itself a sub-tree that should be able to walked an manipulated in a similar manner to the document itself. Basically DTD handling is under-developed.
We currently have skeletal classes LibXML::ElementDecl and LibXML::AttributeDecl (which don't appear in Perl 5's XML::LibXML).

w3c DOM doesn't seem to specify methods for manipulation of Dtd. Will spend a bit more time seeing what I can find.

I'm inclined to rename these LibXML::Dtd::Element and LibXML::Dtd::Attribute. Seems more natural, and there might be other roles or classes going into the LibXML::Dtd::* namespace as a result of this ticket.

Will add tests and method for reading/walking the elements in a Dtd. Make these similar to existing DOM methods where possible

Depending on this, create methods for updating and constructing Dtd's

oops closed wrong ticket

Recently released W3C::DOM.
Invesitgating NYI W3C::DOM:DocumenType methods notations(), entities() and internalSubset().

Work well underway with release 0.6.1 includes basic methods and objects for entities,  element and attribute declarations, along with notations.

Readonly at the moment. A full DOM interface is possible for creating and editing Dtd Declarations, but LibXML doesn't provide a full interface, so would require fiddling with the internals.


Introduce the ClusterAutoscaler APIs module

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
I replaced the k8s.io/autoscaler/cluster-autoscaler with the k8s.io/autoscaler/cluster-autoscaler/apis because https://github.com/kubernetes/autoscaler/pull/6315/ is finally merged.
Which issue(s) this PR fixes:

Fixes #1345
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

https://github.com/kubernetes-sigs/kueue/blob/a751ffd7e902e61e56f24c499eeffb5b7ec5f7af/Makefile#L400-L404
needs an update

https://github.com/kubernetes-sigs/kueue/blob/a751ffd7e902e61e56f24c499eeffb5b7ec5f7af/Makefile#L400-L404
needs an update

Yes, that's right.
Also, we need to upgrade Golang version to v1.22 due to https://github.com/kubernetes/autoscaler/commit/eb5d875837a16cbb3a6b0eefb9fde41d9ecbcd85.
go: downloading k8s.io/autoscaler/cluster-autoscaler v0.0.0-20240320105552-09954b6741cb
go<EMAIL_ADDRESS>requires go >= 1.22; switching to go1.22.1
go: downloading go1.22.1 (darwin/arm64)
go: downloading k8s.io/autoscaler/cluster-autoscaler v0.0.0-20240320105552-09954b6741cb

Or, putting https://github.com/kubernetes/autoscaler/tree/09954b6741cbb910971916c079f45f6e8878d192/cluster-autoscaler/config/crd to https://github.com/kubernetes/autoscaler/tree/master/cluster-autoscaler/apis/provisioningrequest would be better.
@alculquicondor @x13n WDYT?

Putting the CRD in CA apis module makes sense to me, it has to be in sync with the code there anyway.

I'm also ok upgrading to golang 1.22
But all the APIs should be in the separate module

@x13n @alculquicondor Thank you for responding.
I can take both approaches.
Let me try them.

@x13n @alculquicondor Thank you for responding. I can take both approaches. Let me try them.

Raised at https://github.com/kubernetes/autoscaler/pull/6651

https://github.com/kubernetes/autoscaler/pull/6651 is merged.

kubernetes/autoscaler#6651 is merged.

Thank you!

@alculquicondor This PR is ready for review.

Thank you so much @tenzen-y!!!!
This was a long effort that simplifies the maintainability of our deps greatly!

I appreciate to collaboration with SIG-Autoscaling (@x13n and @mwielgus)!
@alculquicondor btw maybe you forgot to add /lgtm  label 🙃

I did indeed forget :)
/lgtm

@alculquicondor How about cherry-picking this into the release-0.6 branch?

I'm not too sure about that. Why do you suggest it?
Also, I'm not sure what's the origin of the Job failure.

I'm not too sure about that. Why do you suggest it?

Cherry-picking would allow cluster-admins to avoid https://github.com/kubernetes-sigs/kueue/issues/1345 error.

...
[FAILED] Test Report unavailable because a Ginkgo parallel process disappeared
    The aggregated report could not be fetched for a ReportAfterSuite node.  A
    Ginkgo parallel process disappeared before it could finish reporting.
...
This occurs if a parallel process exits before it reports its results to the
Ginkgo CLI.  The CLI will now print out all the stdout/stderr output it's
collected from the running processes.  However you may not see anything useful
in these logs because the individual test processes usually intercept output to
stdout/stderr in order to capture it in the spec reports.

Maybe this is the reason 🧐

I guess that this error occurred by throttling CPU performance because integration tests often use larger CPUs.
https://monitoring-eks.prow.k8s.io/d/96Q8oOOZk/builds?orgId=1&from=1711225845388&to=1711398645388&var-org=kubernetes-sigs&var-repo=kueue&var-job=pull-kueue-test-integration-main&var-build=All&refresh=30s&viewPanel=136
https://github.com/kubernetes/test-infra/blob/ce9cda237cf4b7518433dcd0ee90255e15c9c031/config/jobs/kubernetes-sigs/kueue/kueue-presubmits-main.yaml#L58-L59

https://monitoring-eks.prow.k8s.io/d/96Q8oOOZk/builds?orgId=1&from=1711225845388&to=1711398645388&var-org=kubernetes-sigs&var-repo=kueue&var-job=pull-kueue-test-integration-main&var-build=All&refresh=30s&viewPanel=136

It seems that we need to set 8 Cores in the limit.
@alculquicondor WDYT?

One of my teammates is working in splitting the Multikueue E2E test into its own job. Let's decide if the limits are ok after that.

That makes sense.
/test pull-kueue-test-integration-main

I'm not too sure about that. Why do you suggest it?

Cherry-picking would allow cluster-admins to avoid #1345 error when cluster-admins implement a separate controller for the in-house job.
[MODIFIED COMMENT]

@alculquicondor How about this?

/cherry-pick release-0.6

@alculquicondor: #1872 failed to apply on top of branch "release-0.6":
Applying: Introduce the ClusterAutoscaler APIs module
Using index info to reconstruct a base tree...
M	Makefile
M	cmd/kueue/main.go
M	go.mod
M	go.sum
M	pkg/controller/admissionchecks/provisioning/controller.go
M	pkg/controller/admissionchecks/provisioning/controller_test.go
M	pkg/controller/admissionchecks/provisioning/indexer_test.go
M	test/integration/controller/admissionchecks/provisioning/provisioning_test.go
Falling back to patching base and 3-way merge...
Auto-merging test/integration/controller/admissionchecks/provisioning/provisioning_test.go
Auto-merging pkg/controller/admissionchecks/provisioning/indexer_test.go
CONFLICT (content): Merge conflict in pkg/controller/admissionchecks/provisioning/indexer_test.go
Auto-merging pkg/controller/admissionchecks/provisioning/controller_test.go
Auto-merging pkg/controller/admissionchecks/provisioning/controller.go
Auto-merging go.sum
CONFLICT (content): Merge conflict in go.sum
Auto-merging go.mod
CONFLICT (content): Merge conflict in go.mod
Auto-merging cmd/kueue/main.go
Auto-merging Makefile
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Introduce the ClusterAutoscaler APIs module
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

ACK


If the Starrocks node changes its IP, it will require deleting historical data, which would have a significant impact on the business

General Question

FQDN, please refer to the doc: https://docs.starrocks.io/en-us/latest/administration/enable_fqdn#enable-fqdn-access


Adding Docker support to create a linux package

Description
For those folks on linux (or on other OSes, but who want to create a package for distribution on linux, its easier to dockerize the packaging step.
Motivation and Context
One of the contributors was unable to build Ferdi since their system dependencies could not be upgraded.
Checklist:

[x] My pull request is properly named
[x] The changes respect the code style of the project ($ npm run lint)
[x] I tested/previewed my changes locally

@kris7t - please review and test on your local linux before upgrading.

Fails for me with
Step 12/13 : COPY --from=builder /usr/src/ferdi/out/* /ferdi
When using COPY with more than one source file, the destination must be a directory and end with a /

on Docker version 20.10.6, build 370c28948e

thanks for testing this out @kris7t - I have made a minor fix.

Looks good to me.


rename repository to better reflect the purpose

Description
We are using the images build in this repository as Inference Server images in AI Lab repository.
https://github.com/containers/podman-desktop-extension-ai-lab/blob/main/packages/backend/src/assets/inference-images.json
We discussed this with @jeffmaury and we should rename this to remove the playground term with something more appropriate.
suggestions

ai-lab-inference-images (the ai lab recipe repository do not use the prefix, should we ?)
podman-desktop-extension-ai-lab-inference-images

cc @benoitf

prefix should be used (helping to identify which repository is used by who), my pref would be podman-desktop-extension-ai-lab-inference-images
it also means that package repository should be also changed (where we publish images on GitHub)

I would propose odman-desktop-extension-ai-lab-images as it's likely this repo will also host images for InstructLab implementation


Facing issue with dic_dict

Traceback (most recent call last):
File "main.py", line 143, in
tf.app.run(main)
File "/home/.local/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "main.py", line 122, in main
write_result(predict_results, FLAGS.dic_dir)
File "main.py", line 23, in write_result
with open(dic_dir) as dic:
IOError: [Errno 2] No such file or directory: ''

you need set DIC_DIR in run.sh

@manojbadhe is is solved?

I have solved the issue in current version


calyx-hls A Calyx-based HLS toolchain

I've written been getting my hands dirty with the Calyx CIRCT implementation and have been inspired to design a couple of new tools to drive work on Calyx forward. Specifically, I think we have everything we need to build a calyx-hls tool that takes C++ programs and generates synthesizable Verilog out of it.
Here is a rough sketch:

Use Polygeist to transform C++ programs into the affine dialect.
Use the affine to loopschedule or pipeline flow and generate calyx dialect programs out of it.
Use the soon-to-be-added calyx-native pass to compile programs generated from (2) to purely structural Calyx.
Either use the Calyx compiler to generate Verilog programs from there or use the CIRCT flow (calyx -> hw -> sv)

Et voila, you have an HLS tool built using Calyx. There are a couple of good reasons to build this toolchain:

We're already moving towards HLS flows being an important driver of Calyx development.
We can start compiling programs from real HLS benchmarks like Rosetta and see where this toolchain fails. This will also start giving us programs we can look at for optimization purposes.
Marketing wise, things are easier if we have a C++-based HLS compiler.

Another long-term benefit is that folks like @andrewb1999 can potentially design a new frontend DSL for the AMC compiler (which currently uses a lower-level representation) which will really allow calyx-hls to start beating Vitis.
My proposal is to use the new calyxir organization to house this project because it relies on dependencies from both Calyx land and CIRCT land.

I think @calebmkim and I should sit down and think about how to build this. IIRC, he is already using a Polygiest flow for evaluating the static paper.

Seems like a fun thing to try! Here are a couple of high-level thoughts:

To state the probably-obvious, the utility here depends strongly on how well Polygeist does. "Plain C++"-to-affine translation will necessarily always be quite limited, and increasing its coverage seems like hard work. So while it doubtless works fine for Polybench, Rosetta may be a drastically different story.
Hopefully, the thing being built here is "just glue" and contains no actual cleverness. That is, all the hard work happens in places that are more reusable in other contexts: Polygeist, CIRCT, and Calyx per se. With that in mind, maybe we should think about what is not covered by hlstool… perhaps hlstool has no interest in integrating with Polygeist for source-code ingestion, for instance?

Precisely: this is mostly supposed to be a "glue code" exercise to see how far we can get with the tools today. You're right that Polygeist will be the limitation here and so eventually, we'd want to see what other frontends we can use. For example, there are passes to convert the ControlFlow dialect cf into scf which could potentially allow us to play with more programs (albeit with none of the pipelining that the affine flow gives us).
The long-term dream is of course to use @andrewb1999's AMC dialect as the only representation and come up with a frontend that really takes advantage of its abstractions.

@rachitnigam Sorry for the delay on this - I was wondering how to get involved in the work Andrew referenced  for adding support for the static scheduling flow in Calyx HLS. Is there any chance you'd be free for a chat early next week?

Hey @evanmwilliams! I'm unfortunately not super available these days. What do you think about starting a new thread our Zulip channel for calyx and starting a discussion about what to do next? I think the simplest thing to do is look at hlstool in the CIRCT repository and extracting out the Calyx based lowering flow from it. Then we can build upon it.

That sounds reasonable! I'll do that :)


Make AbstractOptionSpec public for Kotlin

Kotlin does not allow to use classes in package scope
even when they have public methods.
When a class in package scope has public methods
these bleed into the public space anyway.
So make the class public but let the constructors
be package protected so it may only be instantiated
from within the package.

Thanks for this -- hope it helps!

I tested this locally with a SNAPSHOT and it did work, thanks for merging.

Was this published to central? I still see the problem when using 5.0.1.

@raniejade It hasn't yet been released. I can cut a 5.0.2 if urgent. Thanks!

Ahh, no worries. It only stops me from using forHelp, so no biggie.


Error invoking Method 'stream-rooms': Internal server error [500]

When i click on viewer it just shows the spinner and on the console logs it show this error "Error invoking Method 'stream-rooms': Internal server error [500]".

sorry, didn't got informed about your issue. now enabled the watch!  The project was successfully tested today from somebody outside. I guess its because of a new release of meteor!
Try it again with: meteor --release <IP_ADDRESS> --settings settings.json (in case you have settings and a kurento server somewhere else)


Section previously selected remains highlighted

I've created a app using your library, but when I select new section, the previously selected section remains highlighted in the drawer. I've tested on 5.x
Screenshot :: Imgur

@neokree, Resolved

Hey, I got this issue using the section selection programatically using the method select(). The previous selected section remains selected after calling select() with another section.
The piece of code that can simulate this:
getSectionByTitle(getString(R.string.my_section_title)).select();

Please refer this issue here #220.


Update docs for benchmarks.

I had this on an old branch. I think I was trying to do more but I can't remember what, so, here's what I had on that branch so far...

LGTM


Broken links in RTX-KG2 results

See this issue: https://github.com/NCATSTranslator/Feedback/issues/497
Basically, there is a trailing } in one of the urls

I don't know if this is due to what's stored in RTX-KG2, or if the UI is parsing the info incorrectly. I do know it's an ARAX/RTX-KG2 edge though, as I had selected the check box to only return results from those ARA/KPs

When I ran match (n)-[e]-(m) where n.name="Flutamide" and m.name="congenital adrenal hyperplasia" return n,e,m on kg2canonicalized2.rtx.ai, none of the results contain anything about clinical trials:
Edge 1:
{
  "predicate": "biolink:treats",
  "primary_knowledge_source": "infores:chembl",
  "domain_range_exclusion": "False",
  "publications_info": "{}",
  "kg2_ids": [
    "CHEMBL.COMPOUND:CHEMBL806---biolink:treats---None---None---None---MESH:D000312---identifiers_org_registry:chembl.compound"
  ],
  "subject": "PUBCHEM.COMPOUND:3397",
  "id": "30382301",
  "object": "MONDO:0018479"
}

Edge 2:
{
  "predicate": "biolink:treats",
  "primary_knowledge_source": "infores:semmeddb",
  "domain_range_exclusion": "False",
  "publications_info": "{'PMID:10720048': {'publication date': '2000 Mar', 'sentence': 'We conclude that the regimen of flutamide, testolactone, reduced hydrocortisone dose, and fludrocortisone provides effective control of congenital adrenal hyperplasia with reduced risk of glucocorticoid excess.', 'subject score': 1000, 'object score': 1000}, 'PMID:8855797': {'publication date': '1996 Oct', 'sentence': 'A preliminary study of flutamide, testolactone, and reduced hydrocortisone dose in the treatment of congenital adrenal hyperplasia.', 'subject score': 1000, 'object score': 1000}}",
  "kg2_ids": [
    "UMLS:C0016384---SEMMEDDB:treats---None---None---None---UMLS:C0001627---SEMMEDDB:"
  ],
  "subject": "PUBCHEM.COMPOUND:3397",
  "id": "8061436",
  "object": "MONDO:0018479",
  "publications": [
    "PMID:10720048",
    "PMID:8855797"
  ]
}

I don't believe KG2pre ingests any clinical trials data, so I'm not sure where this data could be coming from.


Exclude system table systranschemas for mssql diff

Environment
Liquibase Version: All
Liquibase Integration & Version: CLI
Liquibase Extension(s) & Version: NA
Database Vendor & Version: MSSQL
Operating System Type & Version: All
Pull Request Type

[ ] Bug fix (non-breaking change which fixes an issue.)
[x] Enhancement/New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description
Per this Microsoft document, the table systranschemas should belongs to system table/view.
https://docs.microsoft.com/en-us/sql/relational-databases/system-views/systranschemas-transact-sql?view=sql-server-ver15
Steps To Reproduce
Try to diff 2 mssql schema, one of them has systranschemas.
Actual Behavior
The system table systranschemas will be reported as "Missing Table(s)" or "Unexpected Table(s)".
Expected/Desired Behavior
The system table systranschemas should NOT be reported as "Missing Table(s)" or "Unexpected Table(s)".
Screenshots (if appropriate)

Additional Context
The change only impact for MSSQL database engine.
Fast Track PR Acceptance Checklist:

[x] Build is successful and all new and existing tests pass
[ ] Added Unit Test(s)
[ ] Added Integration Test(s)
[ ] Added Test Harness Test(s)
[ ] Documentation Updated

Need Help?
Come chat with us on our discord channel

Hi @MatrixDai Thanks for creating this PR.
A member of the Liquibase team will take a look at your contribution and may suggest:

Changes based on guidelines for PR submission
Additional tests (unit and/or integration) and/or Test Harness Tests
Provide other feedback, like requesting changes.

The PR will be prioritized according to our internal development and testing capacity.
We’ll let you know when it’s ready to move to the next step or if any changes are needed.


Improves handling of Gcode Thumbnails

Summary of the Pull Request
This PR adds improved handling of Gcode Thumbnails that include JPG and QOI image formats ~, and also adds Gcode files support to Peek~.

PR Checklist

[ ] Closes: #28404
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[X] Tests: Added/updated and all pass
[X] Localization: All end user facing strings can be localized
[X] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
PowerToys already had support to show PNG thumbnails found in Gcode files as File Explorer thumbnails or on the preview pane.
As Prusa Slicer recently added JPG and QOI image formats to the output Gcode, this PR adds support for those 2 new image formats.
The QOI image format support is added via QOI.Core, library with WTFPL license.
I've refactored the Gcode parsing related code and moved it to PreviewHandlerCommon.

Validation Steps Performed
Manual testing on my local machine mostly, but I also updated the UnitTests-GcodePreviewHandler and UnitTests-GcodeThumbnailProvider tests

I would argue that the Peek support doesn't bring that much value as these thumbnails are normally 128x128 or less, so we end up with a small image shown on Peek!
However, g-code files have a lot more metadata than just the thumbnails, so if peek had a Property Grid of sorts, we could show that metadata as key value pairs... just a thought for further enhancement! 😉

Following up on my previous comment, I see I was incorrectly adding Peek support via ImagePreviewer in this is no image, so I have reverted and removed those changes from the PR.

I've just added #27979 to allow Peek to show not only Gcode thumbnails, but also any file that has a thumbnail!

Blocking until .74 while we figure out how to audit the new dependency ;)

@jaimecbernardo if the WTFPL license is an issue, there are a couple of other NuGet packages that are MIT based, maybe I should check those instead?
Alternatively, I can try pulling a QOI implementation into PowerToys...

What alternate packages do you have in ming?

Alternatively, I can try pulling a QOI implementation into PowerToys

How much code would this bring in? If not much, this might be the best option. The reason is that all of these packages have a small number of downloads, and not sure how stable are those

With the WTFPL license, I'm thinking this could be a git submodule, so we can be sure of the code we are bringing in.
My biggest issue here is that it's low number of downloads so we might be bringing some risky code in that would be signed and running on end user machines.

I will take a look at one of the other MIT licensed QOI implementations and see if I can get my code to work with theirs; if so, I expect it to be one or two files tops to bring into PT, so might that be workable?

That would be an interesting way of doing it, as well. I'm planning on making the current dependency into a git submodule after I do the current round of reviews :)

FWIW, I've now removed the QOI.Core dependency and replaced it with my very own custom implementation of a QOI decoder based on the (MIT licensed) reference code!

Hi @pedrolamas , thanks for the changes :)
So this is based on https://github.com/phoboslab/qoi , right?

Hi @pedrolamas , thanks for the changes :) So this is based on phoboslab/qoi , right?

Correct, this is my own translation of the official reference code:

I added a small disclaimer on top of the QoiImage.cs file, pointing to the reference code file.

Thanks a lot for the implementation 😄
I've added the MIT notice to our NOTICE file.
@crutkas , I believe this should be enough to comply with the MIT license for the reference code that was translated and adapted?
https://github.com/microsoft/PowerToys/pull/27947/commits/3c9a04796973fac4320a38f26fd201487c82499a

Can you link me to their license md?

@crutkas https://github.com/phoboslab/qoi/blob/master/LICENSE

Lemme do the notice tweak and will call out it is a port too by Pedro

Lemme do the notice tweak and will call out it is a port too by Pedro

FYI, This is what's already on the PR in NOTICE.md under "File Explorer add-ons":
### The Quite OK Image Format reference decoder

[@pedrolamas](https://github.com/pedrolamas) translated and adapted the reference decoder C++ code to C#.

**Source**: https://github.com/phoboslab/qoi

MIT License

Copyright (c) 2022 Dominic Szablewski

Permission is hereby granted, free of charge, to any person obtaining a copy
of this software and associated documentation files (the "Software"), to deal
in the Software without restriction, including without limitation the rights
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
copies of the Software, and to permit persons to whom the Software is
furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all
copies or substantial portions of the Software.

 IN NO EVENT SHALL THE
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

i tweaked it a bit, @pedrolamas did great and on inspection, it would have been truthful.


Update psalm annotations

Some tests fixed;
Added/replaced psalm-annotations;
Now keyset paginator preserves keys of readed items;
Todo

[x] prepare annotations in the yii-cycle package.

Looks alright but tests should be fixed.

👍


Feat/webapp

Signed-off-by: Frost Ming<EMAIL_ADDRESS>This is stack on #15
Preview: https://frostming-epubhv-web-featwebapp-aiskqq.streamlit.app/

@frostming can you fix the conflicts.~


Dependency updates

Developer

[ ] All new ENV is documented in README
[ ] All new ENV has been added to Heroku Pipeline, Staging and Prod
[ ] ANDI or Wave has been run in accordance to
our guide and
all issues introduced by these changes have been resolved or opened as new
issues (link to those issues in the Pull Request details above)
[ ] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer
Merged without code review according to our practices...
Requires database migrations?
NO
Includes new or updated dependencies?
YES bundle update

Pull Request Test Coverage Report for Build b4fc9d8c4a9f6fadf06feab58cd51e3597e5bf79-PR-88

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.427%

Totals

Change from base Build 9cb5aad43a883f377d949e62b3e04eb9459be1c5:
0.0%

Covered Lines:
347

Relevant Lines:
349

💛  - Coveralls


PURE-2: Offline Updates

This PURE is the result of design work by myself (Jon Oster) and Phil Wise (@cajun-rat on github). Its purpose is to allow for (semi-)directed software updates in situations where interactive communication with the update server isn't possible. For example, a diagnostics/flashing unit could be loaded with PURE-2 compliant Uptane metadata, and deliver updates to vehicles in a shop as part of a service visit. In non-automotive use cases where Uptane is employed for its advanced security features, such as industrial automation systems or high-value industrial appliances, PURE-2 enables manufacturers to offer an update solution that works both in cases where those systems are online and the cases where they are on airgapped networks, or not networked at all.
PURE-2 is meant to be a non-mandatory extension to Uptane, providing a secure solution for offline updates without requiring implementors to support it, if they prefer not to have an offline update solution at all, or prefer a different method of securing those updates.

I've applied all suggestions, changed the status to Approved, and will now merge it in.

Thanks, @tkfu .


The mouse is permanently confined in Borderless even if the confinement option is changed

The mouse confinement seems to have some issues with the Borderless screen mode:

If the confine is set to "during gameplay" and you finish a song or exit the player loader, you can't move your cursor to a second screen anyway (even if you set the confinement to "never")
^ after replicating the above problem, if you set the screen mode to Windowed, the cursor still remains confined to the current screen (ignoring the size of the game window)
the issue seems to be fixed when you resize/drag/unfocus the window (so after performing the second point), or alt-tabbing when in Borderless screen mode.

I was thinking of opening another similar issue, but it appears to be identical, so to avoid making duplicates I will describe it here:
When you're in Borderless screen mode and change the mouse confinement from "always" to "during gameplay" or "never", the cursor is still confined to the current screen. Also, when you change to Windowed screen mode, the issue is still the same as described above.

I was thinking of opening another similar issue, but it appears to be identical, so to avoid making duplicates I will describe it here:
When you're in Borderless screen mode and change the mouse confinement from "always" to "during gameplay" or "never", the cursor is still confined to the current screen. Also, when you change to Windowed screen mode, the issue is still the same as described above.

@Gabixel can you confirm that everything is fine if you start the game (or framework) with confine mode set to disabled and window set to borderless? It's only after toggling it on once that it gets "stuck"?

Yes, when I start the game (or the framework) with those settings, everything looks fine, and I can move the cursor anywhere I want between the two monitors (even if you set the confinement to "during gameplay" since it doesn't confine the cursor when the game starts).
Summing up the first two posts: it gets stuck when the game begins to confine the cursor in borderless mode (e.g. when I change the option from "never" to "always" and then back to "never", it still confines), then I change to windowed mode, trying to move the cursor to the other monitor and failing because it gets stuck to the current monitor's boundaries (more specifically: the game's ones when it was in borderless) until I unfocus the game window.

@Gabixel I've tried to reproduce this issue following your instructions but cannot make it happen. Could you please test this again, as we've made considerable progress on fixing edge cases of window handling over the last few weeks?

One of those edge-case fixes seems to have covered my issue as well since I can't reproduce it anymore! It can be closed.


Replace hardcoded paths by os.path.join

As the title. Requested internally.

Yep, closing @koubaa


Fix display of long domain names in popup

Currently the popup only displays part of long domain names (per below the domain for OptMeow, for example). Ideally, we would like to break long domain names into multiple lines. If that is not possible, maybe truncated domain names.

The main functionality should be uploaded above, though I will double check my locally saved code to see if there any further updates that weren't finalized when I was working on this issue. Once I upload those, this issue should be closed.

@asalia-1 is suggesting to not show the domain in the popup if there is no real URL (like extension IDs, localhost, ...). Technically, this is the case when no Do Not Sell signal should be sent. When the URL is parsed for the URL and the domain returns null, these are the cases where this should not be shown.

Worked with Kuba to remove  send do not sell switch and domain as discussed when url does not connect to resource on web
(for example chrome://extensions). See the photo below to show what that looks like.

NB: The responses section in the pop up will be updated once this issue is merged with issue #63

@asalia-1, please close this issue if this is resolved now (or write what is still outstanding).


Docker volume with Azure

Problem description
The documentation page for share volume with cloudstor on azure may have more informations, like how to configure the Account storage on Azure and what is supported or not.
Problem location

I couldn't find the information I wanted. I expected to find it near the following URL: https://docs.docker.com/docker-for-azure/persistent-data-volumes

Project version(s) affected
I testing on CoreOS alpha with docker 17.05.0-ce and docker4x/cloudstor:17.05.0-ce-azure2 and
Suggestions for a fix
We can add a part about how to install the plugin, how to create the storage account, add a note about Secure transfer required option not supported on linux.

@ddebroy do you have any insight?

I could try to add some screenshot and other informations.
I could add a second PR once this issue will be released: https://github.com/docker/for-azure/issues/28

Unsure the status of this. @friism @ddebroy

Bradley Wong (can't find his github alias) will need to comment on this.

@brawong PTAL

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Add a robustness benchmark dataset for vision

Name
ImageNet-C
Paper
Benchmarking Neural Network Robustness to Common Corruptions and Perturbations
Data
https://github.com/hendrycks/robustness
Motivation
It's a known fact that vision models are brittle when they meet with slightly corrupted and perturbed data. This is also correlated to the robustness aspects of vision models.
Researchers use different benchmark datasets to evaluate the robustness aspects of vision models. ImageNet-C is one of them.
Having this dataset in 🤗 Datasets would easily allow researchers to evaluate and study and robustness aspects of vision models. Since the metric associated with these evaluations is top-1 accuracy, researchers should be able to easily take advantage of the evaluation benchmarks on the Hub and perform comprehensive reporting.
ImageNet-C is a large dataset. Once it's in, it can act as a reference and we can also reach out to the authors of the other robustness benchmark datasets in vision, such as ObjectNet, WILDS, Metashift, etc. These datasets cater to different aspects. For example, ObjectNet is related to assessing how well a model performs under sub-population shifts.
Related thread: https://huggingface.slack.com/archives/C036H4A5U8Z/p1669994598060499

Ccing @nazneenrajani @lvwerra @osanseviero


Saving a Model

When we run this command,
python -m baselines.run --alg=....
It saves the model as one file. How can we save the model in the following format?
.ckpt.meta
ckpt.index
.ckpt.data
checkpoint
Or can we decompose that one file into checkpoint files?

I'm afraid that each baselines algorithm will produce it's own save files, and may not be tensorflow model checkpoint files :/


test suite doesn't compile due to missing tlsVerifyCertificate fields

Building test suite 'wstunnel-test' for wstunnel-<IP_ADDRESS>..
[1 of 1] Compiling Main             ( test/Spec.hs, dist/build/wstunnel-test/wstunnel-test-tmp/Main.o )

test/Spec.hs:36:36: error:
    • Couldn't match expected type ‘Maybe (ByteString, ByteString)’
                  with actual type ‘Bool’
    • In the first argument of ‘runServer’, namely ‘useTLS’
      In the expression:
        runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
      In an equation for ‘serverWithoutTLS’:
          serverWithoutTLS
            = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
   |
36 |   let serverWithoutTLS = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
   |                                    ^^^^^^

test/Spec.hs:39:23: error:
    • Constructor ‘TunnelSettings’ does not have the required strict field(s):
        tlsVerifyCertificate :: Bool
    • In the expression:
        TunnelSettings
          {localBind = "localhost", Types.localPort = fromIntegral 8081,
           serverHost = "localhost", serverPort = fromIntegral serverPort,
           destHost = "localhost", destPort = fromIntegral 8082,
           Types.useTls = useTLS, protocol = TCP, proxySetting = Nothing,
           useSocks = False, upgradePrefix = "wstunnel", udpTimeout = 0,
           upgradeCredentials = "", hostHeader = "toto.com",
           tlsSNI = "toto.com", websocketPingFrequencySec = 30,
           customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In an equation for ‘tunnelSetting’:
          tunnelSetting
            = TunnelSettings
                {localBind = "localhost", Types.localPort = fromIntegral 8081,
                 serverHost = "localhost", serverPort = fromIntegral serverPort,
                 destHost = "localhost", destPort = fromIntegral 8082,
                 Types.useTls = useTLS, protocol = TCP, proxySetting = Nothing,
                 useSocks = False, upgradePrefix = "wstunnel", udpTimeout = 0,
                 upgradeCredentials = "", hostHeader = "toto.com",
                 tlsSNI = "toto.com", websocketPingFrequencySec = 30,
                 customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In the expression:
        do Logger.init Logger.VERBOSE
           success <- newIORef False
           let needle = "toto"
           let serverPort = 8080
           ....
   |
39 |   let tunnelSetting = TunnelSettings {
   |                       ^^^^^^^^^^^^^^^^...

test/Spec.hs:98:36: error:
    • Couldn't match expected type ‘Maybe (ByteString, ByteString)’
                  with actual type ‘Bool’
    • In the first argument of ‘runServer’, namely ‘useTLS’
      In the expression:
        runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
      In an equation for ‘serverWithoutTLS’:
          serverWithoutTLS
            = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
   |
98 |   let serverWithoutTLS = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
   |                                    ^^^^^^

test/Spec.hs:101:23: error:
    • Constructor ‘TunnelSettings’ does not have the required strict field(s):
        tlsVerifyCertificate :: Bool
    • In the expression:
        TunnelSettings
          {localBind = "localhost", Types.localPort = fromIntegral 8081,
           serverHost = "localhost", serverPort = fromIntegral serverPort,
           destHost = "localhost", destPort = fromIntegral 8082,
           Types.useTls = useTLS, protocol = UDP, proxySetting = Nothing,
           useSocks = False, upgradePrefix = "wstunnel", udpTimeout = - 1,
           upgradeCredentials = "", hostHeader = "toto.com",
           tlsSNI = "toto.com", websocketPingFrequencySec = 30,
           customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In an equation for ‘tunnelSetting’:
          tunnelSetting
            = TunnelSettings
                {localBind = "localhost", Types.localPort = fromIntegral 8081,
                 serverHost = "localhost", serverPort = fromIntegral serverPort,
                 destHost = "localhost", destPort = fromIntegral 8082,
                 Types.useTls = useTLS, protocol = UDP, proxySetting = Nothing,
                 useSocks = False, upgradePrefix = "wstunnel", udpTimeout = - 1,
                 upgradeCredentials = "", hostHeader = "toto.com",
                 tlsSNI = "toto.com", websocketPingFrequencySec = 30,
                 customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In the expression:
        do Logger.init Logger.VERBOSE
           success <- newIORef False
           let needle = "toto"
           let serverPort = 8080
           ....
    |
101 |   let tunnelSetting = TunnelSettings {
    |                       ^^^^^^^^^^^^^^^^...

test/Spec.hs:159:36: error:
    • Couldn't match expected type ‘Maybe (ByteString, ByteString)’
                  with actual type ‘Bool’
    • In the first argument of ‘runServer’, namely ‘useTLS’
      In the expression:
        runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
      In an equation for ‘serverWithoutTLS’:
          serverWithoutTLS
            = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
    |
159 |   let serverWithoutTLS = runServer useTLS ("<IP_ADDRESS>", serverPort) (const True)
    |                                    ^^^^^^

test/Spec.hs:162:23: error:
    • Constructor ‘TunnelSettings’ does not have the required strict field(s):
        tlsVerifyCertificate :: Bool
    • In the expression:
        TunnelSettings
          {localBind = "localhost", Types.localPort = fromIntegral 8081,
           serverHost = "localhost", serverPort = fromIntegral serverPort,
           destHost = "", destPort = 0, Types.useTls = useTLS,
           protocol = SOCKS5, proxySetting = Nothing, useSocks = False,
           upgradePrefix = "wstunnel", udpTimeout = - 1,
           upgradeCredentials = "", hostHeader = "toto.com",
           tlsSNI = "toto.com", websocketPingFrequencySec = 30,
           customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In an equation for ‘tunnelSetting’:
          tunnelSetting
            = TunnelSettings
                {localBind = "localhost", Types.localPort = fromIntegral 8081,
                 serverHost = "localhost", serverPort = fromIntegral serverPort,
                 destHost = "", destPort = 0, Types.useTls = useTLS,
                 protocol = SOCKS5, proxySetting = Nothing, useSocks = False,
                 upgradePrefix = "wstunnel", udpTimeout = - 1,
                 upgradeCredentials = "", hostHeader = "toto.com",
                 tlsSNI = "toto.com", websocketPingFrequencySec = 30,
                 customHeaders = [(CI.mk "toto", "tata"), (CI.mk "titi", "tutu")]}
      In the expression:
        do Logger.init Logger.VERBOSE
           success <- newIORef False
           let needle = "toto"
           let serverPort = 8080
           ....
    |
162 |   let tunnelSetting = TunnelSettings {
    |                       ^^^^^^^^^^^^^^^^...

Fixed, and upload nex version on hackage


Observability for streaming models

Discussed in https://github.com/quarkiverse/quarkus-langchain4j/discussions/1080

Originally posted by cmasantos November 14, 2024
Hi, I've notice that "normal" chat model produce a set of gen_ai* metrics. But I've been unable to use them with streaming models. Also when we use a combination of Mutiny StreamingResponseHandler I'm almost sure that it can't be done.
My question is, how can I add metrics to a streaming model even if need to instantiate them? I've found the MetricsChatModelListener class but I can't find a way to put it together with the streaming model interface.
We should add observability for StreamingChatLanguageModel as well

@cmasantos would you be willing to contribute this feature?

@cmasantos would you be willing to contribute this feature?

Yes, I'd like to contribute. I'll give it a try, but it will take me a while because I have no idea how extensions development work. I've got to learn that first.

@cmasantos do you need any help with this?

Being very open here, i didn't had time to go further than a simple exploration of the azure open ai code base since my last reply.
I did encounter a small issue here: https://github.com/cmasantos/quarkus-langchain4j/blob/7900699a261641ff08d4403a3ecb06bf9ec60929/model-providers/openai/azure-openai/runtime/src/main/java/io/quarkiverse/langchain4j/azure/openai/AzureOpenAiStreamingChatModel.java#L201
The response id and model name aren't available in the onComplete method, those values come from dev.ai4j.openai4j.chat.ChatCompletionResponse but the dev.langchain4j.model.openai.OpenAiStreamingResponseBuilder does not stores those values.
If you think I'm going in the right direction I can continue, I haven't looked into how this gets tests nor how this class is instantiated.

Thanks for the update!
I don't think that's an issue actually. I will have a quick look

https://github.com/quarkiverse/quarkus-langchain4j/pull/1125 takes care of this


Composer force me to have PHP 7.3

Hi, why Composer force me to have PHP 7.3? This is error message after composer install:
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for ocramius/package-versions 1.5.1 -> satisfiable by ocramius/package-versions[1.5.1].
    - ocramius/package-versions 1.5.1 requires php ^7.3.0 -> your PHP version (7.2.25) does not satisfy that requirement.
  Problem 2
    - ocramius/package-versions 1.5.1 requires php ^7.3.0 -> your PHP version (7.2.25) does not satisfy that requirement.
    - jean85/pretty-package-versions 1.2 requires ocramius/package-versions ^1.2.0 -> satisfiable by ocramius/package-versions[1.5.1].
    - Installation request for jean85/pretty-package-versions 1.2 -> satisfiable by jean85/pretty-package-versions[1.2].

Hi @camohub. What example are you trying to download?


Fix #1673 fix multipart with path params

Added tests for issues.
Fixed invocationMethod with varargs.

Thanks a lot!


Add new classes/functions to select and modify text as text (not structure)

The new cookbook entries would be along the lines of.
To make all instances of the word jsoup into a link:
    html.find("jsoup").splitByBlockElements().wrap("");

To make sure a particular string is a block element and has a particular class:
    html.find(foo.bar()).wrap("
);

I was a little uncertain how to name the getters. Much of jsoup uses foo(), but e.g. TextNode.getWholeText() uses getFoo(), like the more modern parts of the java standard library. I picked e.g. Point.getOffset() now, but I'll be happy to change to offset() if you prefer that.

This is quite a substantial increase in API surface. Did you write down your motivation for adding such functionality somewhere?
I have the feeling (but no data to back this up) that most people use jsoup in a way that doesn't require such functionality. From that point of view it's only added complexity and a maintenance burden.
But I can certainly see that some people may want this. Maybe this could live in a companion library?

Yes.
I have two needs (plus variations), and I added a little more for API roundness (see below).

I deal with HTML that contains undeclared boilerplate/structure. To handle with that I locate the boilerplate and wrap new div+CSS around it.

I need to elide because of output space restrictions. To achieve that I work with the text, identify and rank elision candidates, then mark those candidates with CSS and send javascript along to elide just as much as needed.

Then I wrote class descriptions, and added enough functions to make  them sound sensible. For example, it sounded strange to have Region.wrap(), Regions.wrap() and Regions.remove() but no Region.remove(), so I added the fourth merely to make the documentation sound smooth.
The Point class and Region.getStart()/getEnd() are public for the same reason, those could be package-private.
Regions.first() exists because e.g. Elements.first() does, it could be dropped and the callers could use get(0) instead. I don't personally see the reason to add Region.equals() and Region.hashCode(), but they're good form so I added them.
I agree that few people are using jsoup do do these things: It's a pain. But when something is a pain, even people who need the functionality don't write the code. "We need this, but it's a lot of work... let's put the ticket on the backlog".

Thanks. I'm not really sure how developers would use this, and what the demand:complexity ratio is (to @chikei's point). Could you maybe show a couple examples of the input/output and code with this or with existing jsoup?
I'm wondering if we can make a few potentially simpler changes to improve text handling (maybe a wrapRange method) that hits maybe 80% of your use case with 20% of the code?

Everything I've used this for falls into a few basic patterns. I start with text that's a mixture of something entered into a system and something supplied by the software. The text is not very disciplined and can be lacking in structure,. there can be nice structure around it. Overall the using code looks like some variation of this:

Get some HTML. Retrieve text(), sometimes of the Document, sometimes of a selected()ed Element.
Look at the text as text, identify patterns, boilerplate, etc. At the end of this you have a number of known strings. If you start by looking for EANs, now you have a set of strings that are valid EANs in that particular document.
For each of those strings, look for the corresponding Region in the source document and do the right thing. Sometimes the regions between identified strings are the interesting parts (for example the HTML between the end of an EAN and the end of the surrounding block element, but at most 500 characters).

Now, if you're looking for article number and description in a somewhat upgefucked site, and you know that the article number is an x-digit number in a table cell and the description is one or more paragraphs and perhaps a table and/or image following the number, then jsoup doesn't help to do much with that. If you also know that the article-number cell is wrapped in div class=articlenumber and the description is wrapped in div class=description, fine, but that's precisely what I didn't know. I knew things about the textual content, and (depending on use case) I needed to either transform the input and impose div structure, or to identify and extract parts of it (including images, tables, links and bold/italic blah).
It's possible that noone wants to use jsoup for that kind of thing because noone in their right mind works with messy and erratically formatted input. If so, fine, I've offered the patch, feel free to reject it. No hard feelings.
It's also possible that noone uses jsoup for that kind of thing because it's really hard to do without something like this patch. In that case you have to decide whether you want jsoup to be a suitable tool for such things or not.
I tried to think about a nice public example... perhaps scraping craigslist? The people who enter data there are undisciplined, so the user-supplied part of each page is messy, but there are common stock phrases, patterns, conventions.
I saw this recently: https://machinelearningblogs.com/2017/01/26/text-clustering-get-quick-insights-from-unstructured-data/ That sounds like what I've been doing, in a way... process (some of) the text as text, but then process it as HTML afterwards. So you want to add boldface to, or extract some valid HTML for, a passage that you identify using text clustering, machine learning, whatever? Then work on the text, make a Region, perhaps change the Region a little if either end is close to a paragraph boundary, then mark/clone it using Element functions.
Less API would be possible; Element.wrapRange(int, int), .cloneRange(int, int), .removeRange(int, int), List Element.splitRangeByBlockElement(int, int) should handle most of it. That's more or less how I started. Some things I needed can't be done using that. For example sometimes images at a region border should be in the wrapped/removed/cloned part and sometimes not. Sometimes I needed the block element(s) surrounding a given text: "Identify key phrase, usually but not always within one block element, clone that/those surrounding block elements". Sometimes I needed the identified string plus a few surrounding words and/or the rest of the block element, whichever is shorter. I threw away the int-based patch and wrote it again with Point instead of an int and several ways to set it, and a Region to do the wrapping/cloning/etc.
Again, feel free to reject. Some PRs are real requests, this is more of an offer.

Thanks @arnt. I'll close this one per the discussion, but I do see the use case, and think it might be best as a separate companion library, so please do consider making that.

Thanks for closing it.
That may sound like a strange thing to say, but IMO rejecting a pull request is okay while leaving old PRs for years is not. I'll buy you a beer if you're ever in Munich. Close some more and I'll buy you two.
FWIW I cannot do this as a separate library as it stands, it really needs one new function (in Node) and would be horribly contorted without it. I could do it as a separate library if you were willing to merge just that function.


Add missing param docs in graph API

[x] : Alpha parameter in leakyRelu
[x] : Target tensor in softmaxCrossEntropyCost

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


Snippet Tabstops prioritized over Snippet Expansion

Issue Type: Bug
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:57:51.166Z)
OS version: Darwin x64 19.2.0
Description:
The snippet engine seems to prioritize moving to tabstops over attempting to expand snippets when pressing tab. This makes it hard to quickly expand nested snippets. Note, this behavior is only present when autocomplete is turned off. Leaving autocomplete on does make snippets behave as expected.
Steps to Reproduce:

Define snippets

        "Math Mode": {
                "prefix": "$",
                "body": "\\$${1:placeholder}\\$$0",
                "description": "Enter math mode"
        },
        "fraction": {
                "prefix": "\\frac",
                "body": "\\frac{${1:numerator}}{${2:denominator}}$0",
                "description": "Latex Fraction"
        }

Insert Math Mode Snippet

Attempt to insert fraction snippet while within math math snippet

Does this issue occur when all extensions are disabled?: Yes

This setting will help you: editor.suggest.snippetsPreventQuickSuggestions


Author?

Originally posted by @burakcbdn in #4847 (comment)

Originally posted by @authorizations in https://github.com/FirebaseExtended/flutterfire/issues/4796#issuecomment-769907169

author

author


upgrade wttr

https://www.reddit.com/r/unixporn/comments/jp5nix/xterm_v3wttrin_weather_in_the_region/

This is blocked by alacritty sixel support: https://github.com/alacritty/alacritty/issues/910


Help with individual Routing to each BottomNavigationBarItem

Hi,
I am quite new to flutter and GetX.
I am looking for a pattern of how to implement BottomNavigationBar with GetX so that I can specify different routes to navigate to different BottomNavigationBarItems and their relevant Pages. So that when I navigate to certain route I launch that page with BottomNavigationBarItem selected.
Lets say that is my information architecture
DashboardPage with BottomNavigationBar with relevant items:

HomePage
ProfilePage
ShopPage
SettingsPage

In routes I have currently specified "DASHBOARD" route with DashboardBinding and DashboardPage.
The question is: what is the pattern to specify "HOME", "PROFILE", "SHOP", "SETTINGS" routes so that when I navigate to them I launch DashboardPage but with relevant BottomNavigationBarItem selected and relevant Page loaded? I am struggling to find a single example that showcases that.

this is quite easy to do with navigator 2.0
see the folder "example_nav2"
and use my branch for now since the main branch has bugs: https://github.com/Bdaya-Dev/getx

@ahmednfwela how to keep controller state while navigating with bottom nav in example_nav2?
Whenever I change the page it calls onDelete() in my controller and page loads from scratch.

@vcoolish sorry, but I stopped using getx altogether, I am now using injectable and go_router


Possible fix for switching to raining gold lane during boss

lowTarget is always zero at that point, so the code will execute all the time. And besides, lowTarget would be the id of the enemy and not the type.
This might be the issue behind switching to a gold rain during a boss

https://github.com/wchill/steamSummerMinigame/pull/64


Apply Tint to Particle Container (#3899)

Applies tint to whole particle container.

A comment....
All current 'tint' properties are on classes that contain and can apply to a texture, and thus works on both canvas and webgl.
This same 'tint' property occurs on a container, and thus on is webgl only.
So I think the documentation for the property should be clear on this.

Thanks for the doc changes @YahorPadlucki.


[SPARK-20645][core] Port environment page to new UI backend.

This change modifies the status listener to collect the information
needed to render the envionment page, and populates that page and the
API with information collected by the listener.
Tested with existing and added unit tests.

For context:

Project link: https://issues.apache.org/jira/browse/SPARK-18085
Upcoming PRs that build on this code: https://github.com/vanzin/spark/pulls

Test build #83504 has started for PR 19677 at commit 85a0665.

Test build #83504 has finished for PR 19677 at commit 85a0665.

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

Merged build finished. Test PASSed.

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

lgtm

merged to master


Add no icons mode

For old devices, when launcher restart caching icons loads too long

There used to be an option to don't load the icons, but I removed it because I didn't find it very useful after I made the icon loading run in separated threads.
The icon loading should happen in background and should not affect the app too much responsibility (unless you are on a single core CPU maybe).
Isn't it better to slowly load the icons than not loading them at all?

yes i have single core CPU and yes not load icons is better)))

+1. Old devices need all the help they can get.

Implemented in 1.2.3


How to use this with bootstrap?

How to use this with bootstrap and react? I'm importing bootstrap into a bundle right now.

How do you refer to Bootstrap classes?

With react's className attribute.

@andersontr15 Ok, as long as it's className="foobar" kind of style (nothing dynamic), the tooling just might pick it up. Use the verbose option to get more insight.
If you can't get it to work, please set up a small repository for me to study.

Yeah I was able to get it to work with bootstrap but now my css modules aren't being picked up even with whitelisting it with 'purify'

CSS Modules takes some extra effort. Please see the README for details.

Got it to work. Don't know if anyone has tried this, but I had to add the .json files emitted by the postcss- modules to the paths array for purifycss to get it to work.

@andersontr15 If you have time, please make a doc PR about that. 👍


Update spark-connector.md

Sync with new changes

@ohadbitt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Aberrant confidence intervals with ConvSCCS when there is censoring

When using (heavily) censored data in the model, the refit coefficients are often outside of their confidence intervals, which should not be possible. The shape of the confidence intervals follow the same shape as the coefficients' curve though, just shifted upward, so there might be some issue with how the censoring is handled that hinders with the final estimates.

@deepPhong Could you send me the notebook reproducing this issue?

Ok, this bug is caused by SimuSCCS which does not take the censoring into account when generating bootstrap samples. This one won't be fun :)


1.20.1 Crash

crash-2024-09-06_13.51.48-client.txt

I have this crash too. Had to downgrade.
crash-2024-09-06_13.53.24-client.txt
Minecraft 1.20.1
Forge 47.3.7
sodiumdynamiclights-forge-1.0.1-1.20.1.jar

this is an incompat with Ars, will be fixed soon

i can't upgrade this mod because of the crash

this is an incompat with Ars, will be fixed soon

can't wait for the update :D

should be fixed in latest update.