Map2 support varargs functions

The map2 function didn't support functions defined with keyword varargs, such as:
def f(a, b, **kwargs):
pass
This is fixed by adding a kwargs option along with function docs showing an example.

I fixed the bug, this branch is ready to go. Small improvement to map2 so that it supports functions with **kwargs definitions such as cached_load_data uses now.


The name of the file after copying

Issue Type: Feature Request
Now
2019.md  [ctrl+c ctrl+v]  =>  2019 copy.md
Can it be made
2019.md  [ctrl+c ctrl+v]  =>  2020.md
VS Code version: Code 1.37.0 (036a6b1d3ac84e5ca96a17a44e63a87971f8fcc8, 2019-08-08T02:33:50.993Z)
OS version: Windows_NT x64 6.3.9600

/duplicate of #76752
See last comment in thread

@IllusionMH thanks for all the help! Really appreciate it!


Source from wire path

be able to provide an open or closed wirepath and generate src term

This issue was moved to simpeg/simpeg#157


Add feedback for operation progress/success/failure

Need Alerts (or some such) saying that an operation is in progress and that it has finished (success/failure).

Notes from @jeffmcaffer:

Bootstrap has a simple "alert".
Alerts needed for:
** Request for Harvesting

I'm working on this atm


:hammer: Fix unused imports and unnecessary using directives

Description of changes
:heavy_minus_sign: Get rid of unused CurrencyLibrary usages
:x:  Get rid of all using PoolIdLibrary for PoolKey; directives and PoolIdLibrary imports
:heavy_plus_sign:  Add global usage of PositionConfigIdLibrary for PositionConfigId
:heavy_plus_sign:  Add global usage of PositionConfigLibrary for PositionConfig
:heavy_plus_sign:  Add global usage of PathKeyLib for PathKey

This PR requires the v4-core PR to build the source properly https://github.com/Uniswap/v4-core/pull/819

hello! thanks for the contribution, this is quite out of date with current main so I'm going to close. But if you would like to reopen against the most up to date main branch please do, and add me as a reviewer! thanks!

Awesome! Will do it!


Add accel profile and scroll method

Describe your PR, what does it fix/add?
Adds config options input:accel_profile and input:scroll_method, along with device configs.
Changes setMouseConfigs function name to setPointerConfigs (since touchpads aren't mice).
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Added docs:

https://github.com/hyprwm/hyprland-wiki/pull/25

Is it ready for merging, or does it need work?
Looks ready.

thanks qt


Interface and examples for infinite-dimensional optimization

duplicate of
https://github.com/ZIB-IOL/FrankWolfe.jl/issues/173


All plates are unavailable when I restart Home Assistant

As the title suggest - whenever I restart Home Assistant all my plates (All Lanbon L8; mixture of Dimmer and Relay versions) becomes unavailable and I do need to reload each device separately in the Device & Services->Integration menu.
I find it unlikely this is a "bug" since other should have experienced it as well so I'm wondering what I could be doing wrong in my implementation.
From the startup log I do get these 2 error messages:
2024-03-11 11:18:39.536 ERROR (MainThread) [custom_components.openhasp] TemplateError('None') while processing template 'Template<template=({{ 0 if is_state("light.living_room_lights", "off") else state_attr("light.living_room_lights", "brightness") }}) renders=4>' in entity 'None' 2024-03-11 11:18:39.726 ERROR (MainThread) [custom_components.openhasp] TemplateError('None') while processing template 'Template<template=({{ 0 if is_state("light.dinner_table_light", "off") else state_attr("light.dinner_table_light", "brightness") }}) renders=4>' in entity 'None'
I don't believe that the YAML code I have above is incorrect. Once the intetration is properly loaded all the plates works fine and if I use the template editor in Home Assistant with the expressions above they do return proper values.
I'm speculating that the error in the log file is caused byt some of the entities not being "ready" when openHASP integration loads but this may be completely wrong.
I've asked similar question on discord and on the Home Assistant community but have so far not gotten any feedback or suggestions on what to do and how to debug this issue.
I'm currently running 0.71 custom component; 0.7.0-rc11 but I had the same issue when running the released version (0.6.3)

Version 0.7.6 delays loading the CC until its MQTT dependency is loaded. Maybe this dependency cause the issue; so please test the new version of the custom component.

I did install the latest version and did a “very” scientific experiment of restarting HA.
When HA had restarted all my plates were online and working.
Conclusion: it works! Thanks

Great, thanks for the confirmation.


Data Exporting

Currently only supports CSV. This feature has been mentioned in #59, #388 and #568
Excerpt from the README (All properties are optional; the utility has it's own set of defaults) :
var csv = $("#grid").jsGrid("exportData", {
    type: "csv", //Only CSV supported
    subset: "all" | "visible", //Visible will only output the currently displayed page
    delimiter: "|", //If using csv, the character to seperate fields
    includeHeaders: true, //Include header row in output
    encapsulate: true, //Surround each field with qoutation marks; needed for some systems
    newline: "\r\n", //Newline character to use

    //Takes each item and returns true if it should be included in output.
    //Executed only on the records within the given subset above.
    filter: function(item){return true},

    //Transformations are a way to modify the display value of the output.
    //Provide a key of the field name, and a function that takes the current value.
    transformations: {
        "Married": function(value){
            if (value === true)
                return "Yes"
            if (value !== false)
                return "No"
        }
    }
});

The feature may need some tweaking, but I thought this was a good baseline.

Thank you very much for the PR!

NP :) thank you too

That's a great feature!
When do you plan to release it in the dist?
Thanks

I just wonder how can i get the last version that contain the 'exportData' feature

A new release seems overdue to include such new feature as this.

Can i have a sample code how we can use exportData feature?

How can I use the export function? Can someone please give me an example code?

Dev Team, any update?


how to set the SetFullCollisionBehavior?

In the process of interaction with the franka using my own controller, the "libfranka: Move command aborted: motion aborted by reflex! ["cartesian_reflex"]
control_command_success_rate: 1"
error easily come. So how can I use the franka_msgs::SetFullCollisionBehavior?

Please refer to the FCI-documentation and the Doxygen documention of libfranka for how to use non realtime commands


Obey url rewrites in browser-side routing

Resolves #452
Impact: minor
Type: feature
Issue
Rewrite rules by nature mask the actual URL the underlying application uses for routing. This is problematic when dealing with single-page app routing as it has no idea that route exists, resulting in 404's.
Solution
Update starter-kit to accept a Request-Path from the proxy handling the rewrite rules in order to make a new route entry in the browser-side app.
Breaking changes
none.
Testing

With the reaction platform, and steps outlined in other repos
Start reaction platform
In the Redirect URLs admin panel, add a new rewrite rule for a tag page
For example from: /example/route to: /tag/shop
Navigate to localhost:9090/example/route
See the tag page you expect
Adjust the page limit/price filters
See that the URL in the address bar remains /example/route

@ross You are the "does this make sense?" reviewer.

Looks like you want @rosshadden here.

Thanks @ross!


Open and Save option

I have created two tabs in ardublockly one for ardublockly and other one is for blockly games
there is open and save option in ardublockly but it is for ardublockly blocks. I have added blockly games and I want the same feature open and save blocks for blockly games.

This is more of a general implementation query than an issue and I am not quite sure what you are asking exactly. If you have created two different Blockly  workspaces you can create two different open and save functions targeting each workspace.


Undefined reference Colormap

I am not using this function but I am guessing that this line would produce an error if you ever reach it. Colormap (with capital C) is not defined anywhere else nor imported.
https://github.com/napari/napari-tiff/blob/8c704181275bc21e2ba220e4bd1bf44a97b204c4/napari_tiff/napari_tiff_metadata.py#L221C25-L221C33

Thanks for opening this @MosGeo! @Czaki @jni not particularly high priority but probably something we should look at?

I think that it is artfact left by @GenevieveBuckley in #31 that I missed in review.
I feel that code [(c.T / 255.0) for c in ijmeta["LUTs"]] should work, but do not have time yet to test it yet.
Currently it broke reading imagej tiff so I think it should be fixed soon.


The script list has not been loaded

The HTTP service is working fine at http://<IP_ADDRESS>:5743/
But the script list never loaded and kept showing "No scripts found"
Chrome has also been rebooted

Same problem for me

Same for me. Windows 10, Chrome 112. Server works fine.

I am not 100% sure if the issue people are seeing here is the same with #48 , but I had fixed the issue in my branch. I wish the owner of this extension can publish a new version which should include the fix I have provided more than 2 years ago. For now I have to use my branch. Feel free to give it a try.

I am not 100% sure if the issue people are seeing here is the same with #48 , but I had fixed the issue in my branch. I wish the owner of this extension can publish a new version which should include the fix I have provided more than 2 years ago. For now I have to use my branch. Feel free to give it a try.

Good work @shihshen ! Your fix works on my end. Would you like to create a pull request to merge your forked branch back to this repo?

I am not 100% sure if the issue people are seeing here is the same with #48 , but I had fixed the issue in my branch. I wish the owner of this extension can publish a new version which should include the fix I have provided more than 2 years ago. For now I have to use my branch. Feel free to give it a try.

Good work @shihshen ! Your fix works on my end. Would you like to create a pull request to merge your forked branch back to this repo?

I did and the PR was merged, but newer version was not published after that. I did ask the owner more than one time, but haven't got any response yet.


lazyload not working on firefox, iphone and safari?

I'm including the script like this:
<script>
				(function(w, d){
					var b = d.getElementsByTagName('body')[0];
					var s = d.createElement("script"); s.async = true;
					var v = !("IntersectionObserver" in w) ? "8.6.0" : "10.4.1";
					s.src = "https://cdnjs.cloudflare.com/ajax/libs/vanilla-lazyload/" + v + "/lazyload.min.js";
					w.lazyLoadOptions = {}; // Your options here. See "recipes" for more information about async.
					b.appendChild(s);
				}(window, document));
			});
</script>

and it's lazy loading the images that have theyr src tags replaced with data-src. However iphones, firefox and safari are not loading the images, I've tried using the cdn and using the older version of lazyload with the script:
<script>
var myLazyLoad = new LazyLoad({
				elements_selector: ".lazy"
			});
</script>

however thats not working on any browser. The images do have the class lazy.
The firefox version is 61, client checked the safari and iphone versions, I can't test those because I don't have apple devices.
maybe what's worthy of mentioning is that I'm using the pagespeed apache module, and jquery (old project)
If I remove the jquery script the lazyload does not seem to work anymore.

Hi @Pionell,
thank you for opening this.
Can you replicate the problem copying your code on a new Pen on Codepen?
Alternatively, can you post your HTML code, including the lazy images, here?
That would really help me to help you.
Thanks

I was trying to replicate the issue yesterday on a simple html template, I think it's an issue with the pagespeed apache module because the html template seemed to work fine on firefox.

That was odd, in fact.
If I can be useful with anything please reopen this issue.

And the same problem in demos ed delay.html. When I open the delay.html on the iPhone - all images are loaded immediately without delay...

@berkovich-pavel which version of lazyload are you using?

10.17.0

@berkovich-pavel that's why. You should load version 8.x on Safari.
Please read the documentation for more information.

Thank a lot! It's my fault - I did't carefully study the instruction)


Fix SSL negotiation + ruff formatted  / SSL 협상 관련 수정 및 ruff format 적용

요약

SSL 협상(SSL negotiation) 관련 수정 #32

File: server.py, L198-260

# https://github.com/inaz2/proxy2/blob/master/proxy2.py
try:
    if not os.path.isfile(certpath):
        epoch = "%d" % (time.time() * 1000)
        p1 = Popen(
            [
                openssl_binpath,
                "req",
                "-new",
                "-key",
                certkey,
                "-subj",
                "/CN=%s" % hostname,
            ],
            stdout=PIPE,
        )
        p2 = Popen(
            [
                openssl_binpath,
                "x509",
                "-req",
                "-days",
                "3650",
                "-CA",
                cacert,
                "-CAkey",
                cakey,
                "-set_serial",
                epoch,
                "-out",
                certpath,
            ],
            stdin=p1.stdout,
            stderr=PIPE,
        )
        p2.communicate()
except FileNotFoundError as e:
    logger.error(
        "[*] OpenSSL distribution not found on this system. Skipping certificate issuance.",
        exc_info=e,
    )
    certpath = "default.crt"
except Exception as e:
    logger.error("[*] Skipping certificate issuance.", exc_info=e)
    certpath = "default.crt"

# https://stackoverflow.com/questions/11255530/python-simple-ssl-socket-server
# https://docs.python.org/3/library/ssl.html
context = ssl.SSLContext(ssl.PROTOCOL_TLS_SERVER)
context.check_hostname = False
context.verify_mode = ssl.CERT_NONE
context.load_cert_chain(certpath, certkey)

try:
    # https://stackoverflow.com/questions/11255530/python-simple-ssl-socket-server
    conn = context.wrap_socket(conn, server_side=True)
    data = conn.recv(buffer_size)
except ssl.SSLError as e:
    logger.error(
        "[*] SSL negotiation failed. Check that the CA certificate is installed.",
        exc_info=e,
    )
    return (conn, b"")

OpenSSL 바이너리 경로 자동 찾기
https://github.com/gnh1201/caterpillar/pull/33/commits/57ed60fd0125f792a33561cf10134452bd6c5b8e#diff-8a3053a35fc794b288420f7343bfbc575c18bc834c223c2aae201e0adcfdd5c0R65
File: download_certs.bat
https://github.com/gnh1201/caterpillar/pull/33/commits/57ed60fd0125f792a33561cf10134452bd6c5b8e#diff-3fe552a198ecc00d5ed9734a403c195126ce9aa30ef68ef5a7147437a82643ffR5
File: download_certs.sh
https://github.com/gnh1201/caterpillar/pull/33/commits/57ed60fd0125f792a33561cf10134452bd6c5b8e#diff-f3a537752c5fa8fb26195ac8cdcf29d8e2148ea97519fb16ea0119d43ecaca6dR5

ruff format 적용

연관링크

https://github.com/gnh1201/caterpillar/issues/32
https://github.com/gnh1201/caterpillar/pull/26

OpenSSL 자동 경로 찾기 기능도 추가되어 있습니다.
https://github.com/gnh1201/caterpillar/pull/33/commits/57ed60fd0125f792a33561cf10134452bd6c5b8e#diff-8a3053a35fc794b288420f7343bfbc575c18bc834c223c2aae201e0adcfdd5c0R65

Invalid


README

README
User Story
As a bootcamp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional READDME that provides information on the project and how to use it

README has been updated


Just for fun.. Please don't accept my PR

as you wish :P

You are not allowed to create a file named Contributors. Sorry :noob:


stream_data: Add test cases for getting stream id from slug.

followup to this discussion: https://github.com/zulip/zulip/pull/31299#discussion_r1767792427

Merged, thanks @evykassirer!

Looks good — glad to have these cases tested!


Can't drag and drop file into Files UWP

Describe the bug
You can drag and drop a file from Files UWP to anywhere else. But It seems that it is not possible to do the opposite: to drag and drop a file in Files UWP. For example if you want to drag and drop a file into a opened tab in Files UWP it is not possible
To Reproduce
Just try to drag and drop a file from a outlook application or even the File Explorer into the Files UWP.
Expected behavior
The drag and drop should be supported when done into the Files UWP
Screenshots

Desktop (please complete the following information):

OS Version: 10 1909 18363657
App version: v<IP_ADDRESS>

Duplicate of #829

I moved this into #829 since they're similar. Please follow that issue and the releases page to keep track of the solution 🙂


Sanitize Error When invalid MSI ID is provided

Fixes #7369

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Looks like CI is failing because of missing resources during the test run. I'm going to go ahead and assume that this is one of two possible problems:

Just temporary trouble that will pass naturally.
Recording that need to be updated (and probably are on the dev branch.)

I'm going to rebase and let CI run again.


Add new exercise saddle-points

As described in #160

Thanks for this one.
I'll try to make some time and merge all pending pull requests. You're doing a great work mate!

Thanks. When you get these pending ones merged I will sync up the config
file.
Did you notice I left the final exercise for you?
Cheers
On Mon, Dec 14, 2015 at 10:33 AM Rubén Chavarría<EMAIL_ADDRESS>wrote:

Thanks for this one.
I'll try to make some time and merge all pending pull requests. You're
doing a great work mate!
—
Reply to this email directly or view it on GitHub
https://github.com/exercism/xecmascript/pull/161#issuecomment-164468605.

Nope, but many thanks. Wow! That's awesome. A very nice gift, thanks!


Remove default parameters from ecommerce tracker

What

Remove the default parameters from a function in the GA4 enhanced ecommerce tracking JavaScript.
Removing the defaults doesn't seem to have an effect as the defaults were
falsely anyway.
Why

We shouldn't be using ES6 in production. This was preventing applications from being deployed as the uglifier/compresser was throwing an error when encountering the default parameters:
Uglifier::Error: Unexpected token operator «=», expected punc «,». To use ES6 syntax, harmony mode must be enabled with Uglifier.new(:harmony => true).

Log output from frontend
Visual Changes
None.

@JamesCGDS one for you to check.
Related to https://github.com/alphagov/govuk_publishing_components/pull/2967

Thank you @injms, I missed that one. I’ve just added a commit onto this to call the parameters when they’re needed. Everything seems to work well and all the tests pass without this, but I thought it might make the code a little easier to read.


AutoDiff: disable the test cases for FP80

The FP80 tests should not be built on Android or Windows.  This fixes
the Windows run of the test.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @ematejska @dan-zheng @marcrasi

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 3528e7e1fab0297cd2d22153a59651b4e693e80e

@swift-ci please test macOS platform


Add CI integration

Hi,
I added some CI integrations to make developing easier.
Features in this PR:

Fix failing tests / warnings in tests
mix credo --strict: linting
Add Travis CI integration for CI jobs
Add coveralls.io integration for test coverage
mix test.watch: run tests continuously on file change => nice for TDD
Makefile for doing common tasks when developing

Actions needed on your side:

https://travis-ci.org/ needs to be enabled for koudelka/visualixir (I have it enabled for my fork, should be just flipping a switch and then merging this PR)
https://coveralls.io/ also needs to be enabled for coverage information

For now, I left out dialyzer.. but I think this is an improvement already.
Now with the better testing support, maybe a next PR could be made with extra tests etc?

Any update on this? Would be nice to have this merged into master before I start adding tests etc.
Otherwise I need to start branching from this branch 😅

Sorry dude, I'm traveling at the moment.
I'm not entirely sure this project needs tests, since it's just a toy. I'm probably going to be changing some things around, it might be a waste of your time. :\

No problem. I can always keep my fork around.
Just saying it would be nice to have everything in one place.
Reason I want the tests is for some changes I still want to make in the future..
In case you change your mind, feel free to merge in.


Add CEL rules to Workload

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
It replaces some of the validations executed by webhooks for the Workload type with CRD validation rules.
Which issue(s) this PR fixes:

Relates to #463
Special notes for your reviewer:
Only validation rules with a cost within the API server limits have been added.
Does this PR introduce a user-facing change?

Added CRD validation rules to Workload.

ACTION REQUIRED: Requires Kubernetes 1.25 or newer

/ok-to-test
/assign

/test pull-kueue-test-e2e-main-1-29


Segfault due to missing error check with bad HostConfig settings

In connector.go, line 264, a call to BuildRequest is made, but err is never checked before req is passed to SendRequest on line 265. In the event BuildRequest fails, which is possible if the Connector's HostConfig object has an invalid Host attribute (for example, if it's accidentally specified as a URL rather than an IP/hostname) then req is nil.
When this is passed to SendRequest, it is blindly passed to the Do method of net/http.Client which then segfaults.
There should probably be some error checking here to fail more gracefully and/or some validation of the contents of HostConfig in Init on line 174

@claytonpeters
IMO, there are two separate issues here.

Totally agree that there should be a check for error returned by BuildRequest before calling SendRequest here.

The above won't address the example case of an invalid Host attribute. As you suggested, this can be addressed by validating HostConfig in Init. In addition, I noticed that there's a call to ValidateConnector here that is supposed to perform a validation by actually making a connection. However, the implementation has been commented out (see https://github.com/infobloxopen/infoblox-go-client/blob/master/connector.go#L378-L384). Perhaps we can resurrect that?


custom test setup file name/location

Is this a bug report?
No
Steps to Reproduce
create a alternate setup file:
// <rootDir>/scripts/jest/setup.js

// set up mocks for localstorage or add enzyme configuration...

Add the following to package.json:
{
  ...
  "jest": {
    "setupTestFrameworkScriptFile": "./scripts/jest/setup.js"
  }
}

Run:
$ yarn test

Expected Behavior
Tests run using the provided file as the setup.
Actual Behavior
You receive the error message outlining the supported Jest overrides.
Thoughts
While it is understandable that you wouldn't want to provide an escape hatch like this as it can lead to unexpected behavior for users that have a <rootDir>/src/setupTests.js file. I would like to suggest that either the provided value is the default, rather than undefined (source). As a valid way to avoid that specific issue.
Another potential solution would be to surface the "setupFiles" configuration option.
Ultimately, I'd prefer to more closely match the file structure of the main react repo. Having to eject for such a small configuration options seems absurd. I'd be happy to implement and document an agreed upon alternative to locking users out completely from having their own file structure for setup files.

Ultimately, I'd prefer to more closely match the file structure of the main react repo.

Can you explain why? You shouldn't be trying to model anything after the React repo itself. It is rapidly changing and in no way is what we suggest others to do.

Just a personal preference. I like the structure of have a scripts directory at the root to house all script related code.
My suggestion is based on allowing more flexibility in general, not the specific opinion that lead to me hitting this. Someone could want that scripts model, or maybe they want their test setup in the root, or in a test directory on the root, or even in a directory in their src directory. I don't see anything blocking allowing some more advanced flexibility while still honoring the default of having src/setupTests.js automatically.

I'm going to close this as we won't be adding configuration for this; I think we'll stick to convention.
Sorry! If you feel strongly enough about this, you may eject.


Fix metric name of the 'getpage_wait_seconds_bucket' metric

Per convention, histogram buckets have the '_bucket' prefix. I got that wrong in commit 0d500bbd5b.
Fixes https://github.com/neondatabase/neon/issues/9241

I think you meant suffix instead of prefix, but LGTM.


How to tell what user is currently logged in?

Hello,
How can I tell what user is currently logged in?

See http://laravel.com/docs/5.1/authentication#retrieving-the-authenticated-user


Adding Description heading for Actions

Description
This PR adds code to pull the description field for actions and add it to the docs if defined.
I am not a great go dev, so I have tried to keep these changes in line with the existing code base.

Hi, @mw-root If you're curious you can add a new test to this file cmd/root_test.go.

Hi, @mw-root If you're curious you can add a new test to this file cmd/root_test.go.

Hi @jackton1 , I added description markers to a couple MD files in test.
These tests all seem to hinge on whether or not the tool ran successfully, so I figured adding desc tags would cover that.  Did you want another run with something specific for the action description?


Include event type codes for FedEx tracking events

Problem
ShipmentEvents for FedEx weren't initialized with their <EventType>s. This results in no status code being associated with a ShipmentEvent.
Solution
Save the <EventType> , much like we do with USPS ShipmentEvents.
Review: @mdking @lucasuyezu @cyprusad

:+1: other than red CI

@kmcphillips Appears that all the fails are related to this change?
My guess is there was some kind of an API change?

After some digging, I've found out that the customer number added in https://github.com/Shopify/active_shipping/pull/313 is for contract shipping (see: Test values for contract shipping, rating and returns. The customer number is for contract-shipping, which requires a contract number and a request built for contract-shipping.
Our canada_post_pws.rb uses the non-contracted shipment endpoint, so the credentials provided (and used in Travis) won't work for the failing tests. The only solution I have for now is to go back to the previous behaviour where we skipped those specific failing tests. We'll need different creds if we want those tests to pass - I have no idea how @dangerdogz was able to get them to pass on https://github.com/Shopify/active_shipping/pull/313.

Thanks for the digging @jonathankwok. I'm fine with skip since you have made a good case for it.


GameTimerTask should not be outside ProgressBarComponent

...because GameTimerTask doesn't make sense if it's not a part of Progress bar.

Fixed in 11655e9294d56d9c9f6a7c0e81a7ef911c9d5821


[WIP] Improve error message in union case pattern matching

Tackles #4246 ("Improve error message in pattern matching when we forget to match the payload"). When we write code like this:
// for the sake of this example, name only one of the fields -- the warning's example pattern will work either way
type Rect = | Rect of width:float * float
let f = function
    | Rect -> ()

The compiler now says:

(that's: error FS0019: This constructor is missing arguments; 2 are expected. Patterns must unpack them, e.g. 'Rect (width, item2)' )
As you can see, the example variable names come from those given by the type.
I haven't written tests yet because I'd love some feedback on the exact wording of the error message.

While I'm at it, is there any reason the wording is "this constructor etc etc"? I'm thinking that it would make much more sense to change it so that instead of
This constructor is applied to x argument(s) but expects y
we see
This union case pattern is provided with x arguments(s) but expects y
Of course this message would not apply if the thing is actually an exception match or whatever else UnionCaseOrExnCheck gets called for.
If so, I could do that in this PR, since it's a similar enough task.

I like this improvement. I've often been baffled by that error message. Make sure to test the solution with partial and complete active patterns as well, which can take any number of parameters.

@jwosty I think it's good to apply the error message as you defined here:
This union case pattern is provided with x arguments(s) but expects y
I may be proven wrong here (it's likely), but I don't think this same situation will occur with exceptions. Example:
exception FooException of TheMessage: string * TheNumber: int

let f x =
    try
        x()
    with
        :? FooException as s -> printfn "%s %d" s.TheMessage s.TheNumber

Note that I'm not decomposing a union case into its constituent parts like I would in a pattern match, yet there is no error here.

Hmm, does anyone know what those other calls to UnionCaseOrExnCheck are there for? I can't seem to reproduce FS0019 in any other circumstance than providing the wrong arguments (specifically) to a DU pattern match.
cc @cartermp

Thanks for this PR, however it looks to be quite out of date, please reopen, if you want to continue with it.
Kevin


Instructor Hello World code hangs with Gemini Flash

[x] This is actually a bug report.
[ ] I am not getting good LLM Results
[ ] I have tried asking for help in the community on discord or discussions and have not received a response.
[ ] I have tried searching the documentation and have not found an answer.

What Model are you using?

[ ] gpt-3.5-turbo
[ ] gpt-4-turbo
[ ] gpt-4
[x] Other gemini-1.5-flash

Describe the bug
instructor                    1.3.4
google-ai-generativelanguage  0.6.6
google-generativeai           0.7.2

Python 3.11.5 (main, Sep 11 2023, 08:31:25) [Clang 14.0.6 ]

To Reproduce
Run this sample code:
import instructor
import google.generativeai as genai
from pydantic import BaseModel

class User(BaseModel):
    name: str
    age: int

client = instructor.from_gemini(
    client=genai.GenerativeModel(
        model_name="models/gemini-1.5-flash",
    ),
    mode=instructor.Mode.GEMINI_JSON,
)

# note that client.chat.completions.create will also work
resp = client.messages.create(
    messages=[
        {
            "role": "user",
            "content": "Extract Jason is 25 years old.",
        }
    ],
    response_model=User,
)

assert isinstance(resp, User)
assert resp.name == "Jason"
assert resp.age == 25

The code just hangs.

Hey @delip, I just tried using your code snippet above and couldn't seem to reproduce the error.  I tested it using uv and installed the packages in a fresh environment.
(instructor) ivanleo@Ivans-MacBook-Pro instructor % uv pip list | grep -E 'instructor|google-ai-generativelanguage|google-generativeai'
google-ai-generativelanguage 0.6.6
google-generativeai          0.7.2
instructor                   1.3.4
(instructor) ivanleo@Ivans-MacBook-Pro instructor % python3 --version
Python 3.11.9
(instructor) ivanleo@Ivans-MacBook-Pro instructor % python3 test.py
name='Jason' age=25
(instructor) ivanleo@Ivans-MacBook-Pro instructor %

Are you still facing this issue?

Closing this issue for now due to lack of activty but feel free to re-open it if you're still facing the same issue! :)


Fix NPE in InvocationResult's toString

Calling toString() in
org.springframework.kafka.listener.adapter.InvocationResultt raises NPE, if sendTo is null.
It can be checked by adding
<Logger name="org.springframework.kafka.listener.adapter" level="debug" /> and running org.springframework.kafka.annotation.EnableKafkaIntegrationTests#testMultiJson for example.

Thanks; now I see what Sonar was complaining about. Merged as 84eec92cbf76f3e3ca0d26659343d51a10efa712


Fix BufRead::read_line unconventional behaviour

Hello,
I've read the documentation so I understand that read_line appends into the passed &mut String argument. I don't understand the use case of that behaviour. To circumvent it, one has to write its own little helper function or not forget to .clear() or .truncate(0) before each new call. So read_line's name doesn't match is behaviour because it reads and appends.
I also looked at the tests 1 and 2 for read_line and they don't even verify that append feature because they .truncate(0) between each check.
Would it be possible to deprecate that behaviour ? If not, would it be possible to document the design intent with a sensible use case, add tests for it and think about an additional simpler interface to load a line into a String ?
Thank you.

One technique can be built on top of the other, but not the other way around:

You can call clear and then append via read_line, implementing your overwriting method.
You cannot ever append to a buffer if read_line always implicitly clears the buffer.

So read_line's name doesn't match is behaviour because it reads and appends.

Following that logic, your proposed modification still wouldn't match behavior as it would read and replace.

Would it be possible to deprecate that behaviour

Doubtful.

possible to document the design intent with a sensible use case

I don't see why not. Hopefully this issue will collect more examples than mine, and you can submit a PR  improving the documentation.

add tests for it

Sure, submit a PR!

think about an additional simpler interface to load a line into a String

There's nothing stopping you from doing this right now in your own code. You can write an extension trait that does what you want. It also might already exist on crates.io.


Support for LaRS Panoptic segmentation

Any code for LaRS Panoptic segmentation?

A development version of the mmdetection training code for LaRS is available at: https://github.com/lojzezust/mmdetection-lars


DB_HOST variable to connect to mySQL

This PR add a DB_HOST env var so that the location of the MySQL host can be changed easily.
Today the app tries to connect to localhost always and the docker compose provides mysql at mysql-server:3306
using localhost will work for MySQL when

the mysql container is running
ports are mapped to  localhost
the sample is run as a local process (not a container) - the build.sh seems to do this.

If this PR is acceptable, I can update the other samples as well.

@schauder Any advice on this PR?

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

I'm a little torn with this one.
I'd like to execute the petclinic with different databases, but I think we should integrate that with actually setting up the database using docker.
In any case we should keep the mechanism symmetric to the data-jpa example which currently uses profiles to control the database and is planned to extend that to other databases using docker. See https://github.com/spring-projects-experimental/spring-native/commit/fb40830f1cae0ae95d618836a95dd4219ce7faef

@jeevanjoseph Thanks for the PR, but I tend to agree with @schauder, it is probably better to deal with database consistently via containers for our samples.

@sdeleuze @schauder - Thanks for the feedback and the profiles are the better way to switch databases. However this PR was to remove the hardcoded “localhost” reference in the JDBC url regardless of the profile used. Currently both the pet clinic samples won’t work using the pure docker compose (running a docker-compose-up. App would come up, but db connections would fail)
The hardcoded localhost in the app properties (regardless of the profile used) would not work with containers since the app container and the db container would be different, and they cannot talk to each other over localhost. To be more specific , this would not work in docker or docker-compose. It would work on kubernetes if both containers are defined within the same PodSpec(or when using podman).


Added saving to postgres DB capabilities

Sorry for the 1 commit, it was worked on in a different fork.
I'll just briefly go over some non-trivial/uncommented changes:

contrib/detection_demo/utils.py
The non_max_suppression function for yolo would return (x1, y1, x2, y2, score, class_conf, class)
I changed it to instead return  (x1, y1, x2, y2, score, *all_classes_confs, class)
This info is necessary for better operations research later, and otherwise has no effect on display or other components. The only effect is that now each prediction is ~12 times bigger in size (86 values instead of 7) which may slow down the read\write a bit. If this turns out to be a real issue I'll make this toggle-able.

contrib/yolov3.py
Just some minor changes to make it compatible with the changes described above.

core/component.py
This came up when I wanted to use zpc for something and realized that the current approach is to write something along the lines of self._routines[i].foo() inside the component, which just looked wrong to me (also kinda error prone).

core/message.py
OrderdDict preserves input order, seems fitting for this use case.

core/routine.py
The logging format had some strange dashes, changed them to normal ones.

utils/structures/instances.py
When trying to do something along the lines of this: (assume predictions is of type Instances)

for pred in predictions:
    # do something..

It would crash with an error (TypeError: len() of a 0-d tensor).. After some debugging I realized that some values inside Instances look like tensor(0.2) instad of tensor([0.2]), which caused the error.
I added this patch to fix that. It worked and had no adverse effects in display or any other component.

Hi @VladShtompel ,
I have a few questions regarding the design of the DBHandler:

I guess the motivation behind it is to encapsulate the exception handling of the commit function. Although I agree that we should encapsulate the exception handling, I'm not sure if we want to encapsulate the call to db.session.add(item). But, in order to encapsulate only the error handling, we need to find another way to "know when a commit need to be done".
Therefore, I think we need to create a decorator for after_main_logic routine, and add this commit-handling part to it. This idea is an inspiration from this nice blog, where he implement exactly that using Flask's after_request decorator:

@app.after_request
def session_commit(response):
    if response.status_code >= 400:
        return
    try:
        session.commit()
    except DatabaseError:
        session.rollback()
        raise

And I think the call to the insert function (or any other function exposed by sqlalchemy: update, delete, etc.) shouldn't be encapsulated.
Generally, I think that Routine and Component classes should have a couple of decorators to enable flexibility regarding their "Lifecycle" operations.
What do you think?

Regarding the try-catch blocks inside the your insert function. Aren't all those exceptions are SQLAlchemyError? If so, why we need a catch block for every single one, if all will be catched by the single catch block of SQLAlchemyError?

Hi @OmriKaduri, thanks for taking the time to look at this!

I guess the motivation behind it is to encapsulate the exception handling of the commit function.

Not strictly. The motivation behind DBHandler was to encapsulate all database related interactions, this includes definition of tables, connection, insertion and as you mentioned also error handling.
I read the blog post you linked and I really like the idea of using declarative_base for operations such as save, update, delete but I'm unsure about the after_main_logic decorator to handle commits part. Maybe I can utilize Events for this instead?

why we need a catch block for every single one, if all will be catched by the single catch block of SQLAlchemyError?

As you can see all of the exceptions are currently handled the same way, I simply added the different catch blocks to "prepare the groundwork" in case someone will want custom behavior for the different exceptions in the future.

Ok, so personally I think that we should not encapsulate all database related interactions, since they are encapsulated "enough" (IMHO) my the ORM (sqlalchemy). Therefore I still suggest that will wrap only the part of exception handling.
Regarding your note about "preparing the groundwork", I think that in case anyone will want to add special error catch-blocks he'll know what catch block to write, and meanwhile we shouldn't just catch them without a special reason, so you should delete the unnecessary blocks.

@OmriKaduri
So basically, if I understood you correctly, practically what you're saying is:

Get rid of DBHandler.py
Move all database interactions to save_predictions.py

This means that if someone wants to implement something similar they must rewrite (or at least copy and paste) all the db connection boilerplate and also the declerative_base class (if we decide on making one).
Not to imply that this is a big deal, although I do think its a shame to have to rewrite that.
BTW what you're suggesting is what I was about to do from the start, @Batush123 suggested that instead I should abstract away all the database interactions, so I'm really not sure what should I do next about this.
I'd love some input from @ItamarWilf and @ItayHoresh as well.

Considered the suggestions from @OmriKaduri  and @ItayHoresh and made some changes:

Renamed the handler to PSQLDBHandler.

The handler now only handles connection, it has an abstract method for defining tables to be implemented individually for each use case.

Moved committing to db to a function that is used as an event_handler for AFTER_LOGIC Event

Removed the extra exception handling.

Fixed some typos


Ignore agent removed during deactivate

https://github.com/rancher/rancher/issues/4682
https://github.com/rancher/rancher/issues/4686

https://github.com/rancher/rancher/issues/4616


Logs missing crucial information

I'm using pool 0.6.4-beta. When I start poold via systemd, it fails after a second or so. The logs don't indicate anything useful:
2023-06-17 10:35:17.439 [INF] RPCS: Starting trader server
2023-06-17 10:35:17.522 [INF] RPCS: Connected to lnd node c-otto.de with pubkey 027ce055380348d7812d2ae7745701c9f93e70c1adeb2657f053f91df4f2843c71
2023-06-17 10:35:17.980 [INF] AUCT: Shutting down auctioneer client

However, when I start poold manually, I get this:
unable to start server: unable to start account manager: unable to resume account xxx: rpc error: code = Unknown desc = transaction rejected: output already spent

Please consider showing important information like this in the logs.

@C-Otto thanks for the report. Do you know if systemd captures stderr? Because if there is an error before poold has set up the logging system (e.g. configuration file cannot be parsed), then we now log to stderr (see https://github.com/lightninglabs/pool/pull/459) instead of stdout.

I'm pretty sure systemctl is able to capture stderr, but in this case (copied over from my lnd setup) I disabled capturing output as I already have the log file. If errors like this aren't supposed to end up in the log file, I might have to tweak my setup. Otherwise, I'd like this message to appear in the log file (additionally to stderr).

Sounds good to me! It's not too hard to check stdout/stderr, especially if the logs are empty.


Add option for printing the description in list

I'm opening a pull request about this, not to get my hack without tests and documentation merged, but perhaps to get tips on how to achieve what I want for my todoman workflow. This is pretty much the best I can do with python in an hour, I'm pretty unexperienced at not-so-strongly typed languages. Don't merge this as is :D
Thanks for creating todoman, it's a great tool to organize my job. Thing is, I want to also see the descriptions of my items in lists to refresh my memory about my "subtasks"/checklists (which OpenTasks on Android supports). I'm thinking the easiest for me to actually implement this feature would be to write a custom wrapper for todoman's --porcelain which could then pretty-print in just the way(s) I want it to.

If you're going to use --porcelain, then the description is already present.
I'm not sure about rendering it in the usual view -- the tricky part is making it fit into the layout, especially considering than descriptions can have multiple lines.
Given that we use a table-like layout, maybe putting that in the next row, something like:
 5  [ ]    8 days ago   buy breakfast
                        the description shows up here and Lorem ipsum dolor sit
                        amet, consectetur adipiscing elit, sed do eiusmod
                        tempor incididunt ut labore et dolore magna aliqua. Ut
                        enim ad minim veniam, quis nostrud exercitation ullamco
                        laboris nisi ut aliquip ex ea commodo consequat.
 2  [ ]    9 days ago   buy pants
                        This other one fits here and I think you get the idea

I don't think that's terribly difficult. But it'd definitely stay behind a flag, since it's too verbose for the default view.
Thoughts?


Arrow keys in moar disabled after first use of atuin

It seems that  atuin somehow affects the use of arrow keys within the moar pager after the first invocation of atuin.
I'm initializing atuin like this:
eval "$(atuin init zsh --disable-up-arrow)"

I installed moar via homebrew, and have the following environment variables defined to use moar as my default pager:
export PAGER=/opt/homebrew/bin/moar --style vulcan
export MANPAGER=/opt/homebrew/bin/moar --style vulcan

I have also created an alias for more (don't fight against muscle memory!):
alias more="$(brew --prefix)/bin/moar --style vulcan"

Here's what happens.

With both atuin and moar configured as above, when I create a new terminal window/session, moar works as expected.  I.e., I can run more README.md, or man xargs, and navigate vertically and horizontally through the paged content using the arrow keys on my keyboard.
In the same terminal window, I can invoke atuin via ctrl-r, and then hit esc to dismiss the search UI.
For any invocation of moar after the first invocation of atuin, the arrow keys no longer function to navigate within moar.
This problem only seems to happen with moar.  Using the built-in more and less tools, the arrow keys still work to navigate after invoking atuin.

Does anyone have any theories on what side effects might be left after invoking atuin that could affect other tools such as moar?

As suggested by the author of moar, I ran it with the --trace option.  I did this in a fresh terminal session, before and after bringing up the atuin search UI (via crtl-r):
In each case, I did the following:

down arrow 5 times
up arrow 5 times
right arrow 3 times
left arrow 3 times
'q' to quit moar

Here are the results.
New terminal before atuin
$ moar --trace README.md
Please post the following report at <https://github.com/walles/moar/issues>,
or e-mail it to<EMAIL_ADDRESS>
Version: v1.15.1
LANG   : en_US.UTF-8
TERM   : xterm-256color

GOOS    : darwin
GOARCH  : arm64
Compiler: gc
NumCPU  : 10

DEBU[0000] Stream read in 0ms
TRAC[0000] Pager starting
TRAC[0000] Reader done, no filter
TRAC[0000] Reader done, contents explicitly set
TRAC[0000] Highlighting done
TRAC[0001] ttyin high watermark bumped to 3 bytes
TRAC[0001] Handling key event 5...
TRAC[0001] Handling key event 5...
TRAC[0001] Handling key event 5...
TRAC[0001] Handling key event 5...
TRAC[0002] Handling key event 5...
TRAC[0002] Handling key event 4...
TRAC[0003] Handling key event 4...
TRAC[0003] Handling key event 4...
TRAC[0003] Handling key event 4...
TRAC[0003] Handling key event 4...
TRAC[0004] Handling key event 6...
TRAC[0004] Handling key event 6...
TRAC[0004] Handling key event 6...
TRAC[0005] Handling key event 7...
TRAC[0005] Handling key event 7...
TRAC[0005] Handling key event 7...
TRAC[0008] Handling rune event 'q'/0x0071...
TRAC[0008] Pager done

Same terminal, after atuin
$ moar --trace README.md
DEBU[0000] Counted 88 lines in 0ms at 102ns/line
DEBU[0000] Stream read in 0ms
TRAC[0000] Reader done, no filter
Please post the following report at <https://github.com/walles/moar/issues>,
or e-mail it to<EMAIL_ADDRESS>
Version: v1.15.1
LANG   : en_US.UTF-8
TERM   : xterm-256color

GOOS    : darwin
GOARCH  : arm64
Compiler: gc
NumCPU  : 10

TRAC[0000] Pager starting
TRAC[0000] Reader done, contents explicitly set
TRAC[0000] Highlighting done
TRAC[0001] ttyin high watermark bumped to 3 bytes
DEBU[0001] Unhandled multi character terminal escape sequence(s): B
DEBU[0001] Unhandled multi character terminal escape sequence(s): B
DEBU[0001] Unhandled multi character terminal escape sequence(s): B
DEBU[0001] Unhandled multi character terminal escape sequence(s): B
DEBU[0001] Unhandled multi character terminal escape sequence(s): B
DEBU[0002] Unhandled multi character terminal escape sequence(s): A
DEBU[0002] Unhandled multi character terminal escape sequence(s): A
DEBU[0002] Unhandled multi character terminal escape sequence(s): A
DEBU[0002] Unhandled multi character terminal escape sequence(s): A
DEBU[0003] Unhandled multi character terminal escape sequence(s): A
DEBU[0004] Unhandled multi character terminal escape sequence(s): C
DEBU[0004] Unhandled multi character terminal escape sequence(s): C
DEBU[0004] Unhandled multi character terminal escape sequence(s): C
DEBU[0005] Unhandled multi character terminal escape sequence(s): D
DEBU[0005] Unhandled multi character terminal escape sequence(s): D
DEBU[0005] Unhandled multi character terminal escape sequence(s): D
TRAC[0006] Handling rune event 'q'/0x0071...
TRAC[0006] Pager done

Somehow atuin is interfering with the key events that moar is expecting.  I.e., rather than the up arrow resulting in logging of Handling key event 5..., after running atuin this becomes  Unhandled multi character terminal escape sequence(s): B.

Hi @ellie, yes, running moar after running echoti smkx caused the behavior I've been seeing.

Fixed in https://github.com/walles/moar/issues/138, closing.


Sounds are playing late

i wanted to use xsound for pop up exhaust sounds for my script, but sounds are running late and can't be stacked up

I will fix the late sound but the stacking is your issue of the way you're handling code. Dont use same identifier for the sound.

Okay, for stacking i can assign different ids to each sound its fine ^^
Also thanks for caring, the 3D sounds would godly for pop up sound ^^


feat: algebraMap_mem for SetLike instances

This adds an algebraMap_mem in the root namespace and protects Subalgebra.algebraMap_mem. The new declaration holds for terms of S where S is a Setlike satisfying OneMemClass S A and SMulMemClass S R A and A is an R-algebra.

I was a bit surprised that there was very little fallout from this, but I guess that's a good thing! There may be some other *.algebraMap_mem declarations that should be protected now, but I'm not entirely sure where they are.

bors merge


Update README

Coverage remained the same at 100.0% when pulling 784e41d959aa5def9f0c001f5ec91c6b8ee46c6d on v5.0.0rc4 into 73556b7810262d2360e0853c8d78f26d6fc182fb on main.

Coverage remained the same at 100.0% when pulling 784e41d959aa5def9f0c001f5ec91c6b8ee46c6d on v5.0.0rc4 into 73556b7810262d2360e0853c8d78f26d6fc182fb on main.


Cannot reach to redux store from RN component

I have a sample app on rnplay and I cannot reach my store inside a component connected to my redux store by connect() via this.props.
on line 320, I have created my redux store with a mock trivial reducer and an an initial state like:
var store = createStore(reducer, {splashViewedCount: 0, loggedIn: false,});
On line 308, I had then connected my component named App to it by connect() with a trivial selector:
function select(state){ 	return state; };
connect(select)(App);
On line 261,  I cannot have the state inside my store from inside my component App via this.props:
const {splashViewedCount, loggedIn,} = this.props;
and in subsequent lines of conditional code.
On line 281, For debug purposes, I have put those in a component to be rendered when news1 or news2 is clicked. In the coming rendered screen I have only the state from store.getState() (it is {splashViewedCount: 0, loggedIn: false,} ), but not from this.props (it is {}).
Is there something I just mix?

This problem was solved. Thanks to the friend form the discord reactiflux.
What I was forgetting o is to get, by assigning, the new connected version of my component by connect(). As docs says it returns a connected version of the class you pass to it.
not only call it, which I was doing:
connect(select)(App)
But also get the connected version of the component:
var App = connect(select)(App)


fix(MenuItem): fixes focus outlines and onClick support

Overview

Adds focus-visible outlines to all variants on MenuItem and creates a button when there is an onClick instead of just an li so onClick will work with keyboard navigation.

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: WEB-2127
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change
[x] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Visit https://6356e7683bbf741565c7fd6f--gamut-preview.netlify.app/?path=/docs/brand-labs-molecules-dropdownbutton--dropdown-button
Navigate via keyboard to open the dropdown button menu
Tab through the menu items and confirm you see the focus outline for all 3 items
Click on each item with your mouse - the first item should redirect the page and add #, the second should console log "hi", and the third should do both
Navigate via keyboard and hit enter on each item - the first item should redirect the page and add #, the second should console log "hi", and the third should do both
Visit https://6356e7683bbf741565c7fd6f--gamut-preview.netlify.app/?path=/docs/molecules-menu--menu#variants
Navigate via keyboard through the 3 different variants and confirm you see focus visible outlines

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.214-alpha.d79ddb.0@codecademy/gamut-labs@33.6.5-alpha.d79ddb.0@codecademy/gamut@50.9.1-alpha.d79ddb.0@codecademy/monaco@0.6.94-alpha.d79ddb.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6356e7683bbf741565c7fd6f--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.214-alpha.2b1afc.0@codecademy/gamut-labs@33.6.5-alpha.2b1afc.0@codecademy/gamut@50.9.1-alpha.2b1afc.0@codecademy/monaco@0.6.94-alpha.2b1afc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6356eb529df40318d56f16e0--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://63582753ee42610a89848fbb--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.218-alpha.22dd30.0@codecademy/gamut-labs@33.6.9-alpha.22dd30.0@codecademy/gamut@51.0.1-alpha.22dd30.0@codecademy/monaco@0.6.96-alpha.22dd30.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63583419b5911d193bf864c0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.219-alpha.76a398.0@codecademy/gamut-labs@33.6.10-alpha.76a398.0@codecademy/gamut@51.0.2-alpha.76a398.0@codecademy/monaco@0.6.97-alpha.76a398.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6358581b0c15680294834517--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.220-alpha.76bd77.0@codecademy/gamut-labs@33.6.11-alpha.76bd77.0@codecademy/gamut@51.1.1-alpha.76bd77.0@codecademy/monaco@0.6.98-alpha.76bd77.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6359c2ebafcc21039a4bc4ac--gamut-preview.netlify.appDeploy Logs


Fix NoSuchElementException in trailing-comma rule in property accessor

Closes #1280
Description
The property setter has exactly 1 argument. As a result a special kind of parameter list is provided, which resulted in a NoSuchElementException.  The same construct which caused this exception is used in other cases in this rule as well. However, I as not able to construct any tests in which that would lead to an error as well.
Checklist

[X] tests are added
[X] CHANGELOG.md is updated

Can we please try to merge this asap if possible? This should fix that long-awaited issue.


HDDS-11563. [OM/SCM/DN] WebUI Display Namespace

What changes were proposed in this pull request?
We have enhanced the web UI to display cluster information for OM (Ozone Manager), SCM (Storage Container Manager), and DN (DataNode).
Please describe your PR in detail:

We have added the getNamespace interface to retrieve and display Namespace information on the web UI and JMX for OM, SCM, and DN. This enhances the ability to understand cluster information through the web UI.
It is located in the namespace field under the Overview title.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11563
How was this patch tested?
It is Only WebUI.
SCM

OM

Datanode

LGTM.

LGTM.
@adoroszlai  Thanks for the suggestion.  Is there anything else you'd recommend ?

Thanks @aierate for the patch, @slfan1989, @whbing for the review.


optimize for clarity & package files improvements

Instead of catching out of bounds exception (lazy mode), you should prevent the exception in the first place, for example:
for(Path currentPath: worldsToCopy.subList(0,Math.min(6,worldsToCopy.size()))){
    File currentSave = currentPath.toFile();
    Julti.log(Level.INFO, "Copying " + currentSave.getName() + " to submission folder...");
    FileUtils.copyDirectoryToDirectory(currentSave, savesDest.toFile());
}

(and a similar thing for logs)


Suggestion: rename AppUpdate to Update

I saw in the example of the readme that there are several statements like below (omitting some irrelevant codes):
impl Model for AppModel {}
impl Widgets for AppWidgets {}

and such statement in the documentation:
impl Components for AppComponents {}

We can easily see from them that they all obey such a format: impl X for AppX {}.
However, there is an exception:
impl AppUpdate for AppModel {}

Though there's nothing like impl Msg for AppMsg and it may be hard to implement if we need, but still, to keep the consistency, I suggest to remove the App- prefix for trait AppUpdate, to make it impl Update for AppModel {} at least.

You're right, the naming isn't perfect, but AppUpdate is called like this to avoid confusion with ComponentUpdate.
I hope you don't mind if I close this issue. I won't introduce a breaking change in the current stable version and in the upcoming version this will be fixed anyway because there's only one Component trait for everything.

@AaronErhardt  Thanks for your reply. Actually I didn't notice the ComponentUpdate when I posted this issue. But now I am looking forward to your only-Component version.


Update DocumentsManagers

We are also removing the entrypoint to the app, and switching document ingestion to the Deeplake vector store, removing support for the sqlite databse

the test with more than 1 workers is what set the CI to go off for hours, id rather just leave it at 1

In that case, LGTM


URDF, SDF, MJCF Model Loading

Hey all,
I am curious if support exists, or will in the future, for the loading of URDF, SDF and/or MJCF model files into nphysics. This functionality would make nphysics highly desirable for simulating robotics and help Rust expand into the robotics world in general. This is something I could potentially help with if there agreement to support such model files.
Perhaps the library is still too young in that it is missing a large variety of joint types, motor types, and sensor types etc,. to make URDF loading of models feasible.

Hi!
There is no support for any file format in nphysics yet. And adding these would be a great contribution!
Regarding missing features, what types of joint types are missing exactly? I believe nphysics supports most types of joints with most combinations of degrees of freedoms. Perhaps there are some robotics-specific joints I am not aware about?
What are you referring to with "sensor types"? nphysics has a concept of sensors for collision detection without any effect on the simulation state, but I guess that's not what you call sensors?
Would it be possible for you to give a more detailed list of the missing feature to cover what URDF needs?

I was referring to the fact that the README mentions some limitations due to the age of the library: "more joints, joint limits, joint motors and breakable joints". My comment was really the opening for a discussion about whether asset files like URDF etc,. are supported. W.r.t. robotics, this would make the use of this physics library much more desirable, but might also be suitable for other applications utilizing other asset files.
What is your opinion of adding additional dependencies?
https://github.com/OTL/urdf-rs

Is there any reason these need to be baked in, rather than maintained as an independent crate? It'd be nice not to increase the build/maintenance load for all the users who aren't doing robotics research.

There is no particular reason. Certain physics libraries will offer support for the loading of model files and URDF is a common format for certain applications. Maintaining a separate crate seems like an appropriate choice. I am not very familiar with Rust and so I'll have to evaluate my ability to contribute. We can close this for now if you'd like, and perhaps I or someone can reopen if a good approach is implemented as an available add-on package/crate.

@cmuell89 Looks like the README is slightly out of date. I don't think we need any more joints. Though it is true that not all our joints support motors and limits yet.
I agree with @Ralith that it would make sense to put this on a separate crate. Though this separate crate could be located on the same github repository as nphysics itself if we want to make its maintenance easier (to always keep it in sync with the latest nphysics version).

We can close this for now if you'd like, and perhaps I or someone can reopen if a good approach is implemented as an available add-on package/crate.

Let's keep this open so it is clearly visible in case someone else wants to try implementing this, unless you manage to do it yourself. Feel free to ask here on on the rustsim discord if you have difficulties working on this!

I am working on a (mostly functional at this point) URDF loader for nphysics. I am currently trying to get permission from my advisor to release it as open source, so it may not be available for a few months.

As an update, the loader I have works and I'm pretty happy with it and I should hopefully have it released as OSS by the end of October. A few notes, though:

I do not have Xacro or ROS package support and I have zero desire to add any ROS to the loader. However, it does handle the file:// scheme and it does have a way for the user to add their own collision mesh loading.
I have found that most URDFs are totally unusable inside of nphysics without some kind of collision filter. Many of the robot models are designed with assumptions about self-collisions (or lack thereof). Assuming #278 is merged, I have a few filters included in the loader that will probably solve collision issues for the majority of robots with user-defined filters filling in the gaps.
Some simulators (i.e., Gazebo) merge groups of fixed links. I'm sure there is some performance benefits for this, so I may include it as a loading option, but my loader currently doesn't do this. Mostly because it can be annoying when you're trying to find the position of a link that has been merged away.


Approving managed private endpoint for SQL times out

Bicep version
v0.24.24
Describe the bug
When approving a private endpoint connection on Azure SQL server for a Data Factory managed private endpoint the command successfully approves the connection, Data Factory also shows the connection state as approved but the command never exits resulting in a timeout.
To Reproduce
create a managed private endpoint for Azure SQL in Data Factory.
use a module to get the SQL private endpoint connections:
param sqlServerName string

resource sqlServer 'Microsoft.Sql/servers@2023-05-01-preview' existing = {
  name: sqlServerName
}

module approveEndpoint 'module.privateEndpointConnections.bicep' = {
  name: 'approve-sql-endpoint'
  params: {
    endpointArray:  sqlServer.properties.privateEndpointConnections
    sqlServerName: sqlServer.name
  }
}

use a second module to read the connection info and approve
param endpointArray array

param sqlServerName string

resource sqlServer 'Microsoft.Sql/servers@2023-05-01-preview' existing = {
  name: sqlServerName
}

var privateEndpoint = [for pep in endpointArray: {
  status: pep.properties.privateLinkServiceConnectionState.status
  name: last(split(pep.id, '/'))
}
]

resource privateEndpointConnection 'Microsoft.Sql/servers/privateEndpointConnections@2023-05-01-preview' = [for endpoint in privateEndpoint: if (endpoint.status != 'Approved') {
  name: endpoint.name
  parent: sqlServer
  properties: {
    privateLinkServiceConnectionState: {
      status: 'Approved'
      description: 'Approved by pipeline'
    }
  }
}]
resource privateEndpointConnection 'Microsoft.Sql/servers/privateEndpointConnections@2023-05-01-preview' = [for endpoint in privateEndpoint: if (endpoint.status != 'Approved') {
  name: endpoint.name
  parent: sqlServer
  properties: {
    privateLinkServiceConnectionState: {
      status: 'Approved'
      description: 'Approved by pipeline'
    }
  }
}]

use az cli to deploy the template:
az deployment group create -g $resourceGroup -n $name -f $bicepFile
I have tested hard coding the endpoint name which results in the same deployment error:
"The resource provision operation did not complete within the allowed timeout period.".
For reference this is the storage account code which works:
module to get the private endpoint connections:
param storageAccountName string

resource storageAccount 'Microsoft.Storage/storageAccounts@2023-01-01' existing = {
  name: storageAccountName
}

module approveEndpoint 'module.privateEndpointConnections.bicep' = {
  name: 'approve-storage-endpoint'
  params: {
    endpointArray:  storageAccount.properties.privateEndpointConnections
    storageAccountName: storageAccount.name
  }
}

use a second module to read the connection info and approve
param endpointArray array

param storageAccountName string

resource storageAccount 'Microsoft.Storage/storageAccounts@2023-01-01' existing = {
  name: storageAccountName
}

var privateEndpoint = [for pep in endpointArray:  {
  status: pep.properties.privateLinkServiceConnectionState.status
  name:  pep.name
}
]

resource privateEndpointConnection 'Microsoft.Storage/storageAccounts/privateEndpointConnections@2023-01-01' = [for endpoint in privateEndpoint : if(endpoint.status != 'Approved') {
  name: endpoint.name
  parent: storageAccount
  properties: {
    privateLinkServiceConnectionState: {
      status: 'Approved'
      description: 'Approved by pipeline'
    }
  }
}]

I did notice the output for storage private endpoint connections contains a name property which is not in the sql output.
Additional context
If I use cli az network private-endpoint-connection approve this works fine.

Unfortunately this is a Microsoft.SQL issue. I can route it to the correct team, but if you would like to streamline the issue, please open a support ticket with the Microsoft.SQL Resource Provider.


org.redisson.client.WriteRedisConnectionException: Unable to write command into connection! Check CPU usage of the JVM. Try to increase nettyThreads setting.

Using redisson 3.34.1 and java 17. Client is configured to use cluster servers.

You need to refer to https://github.com/redisson/redisson/wiki/16.-FAQ#q-what-is-the-cause-of-redistimeoutexception

Hi @mrniko
even I am getting this issue how to fix it currently I using 3.27.2 redisson version
Thanks


add include for  to fix build failure under clang/libc++

appimageupdate was failing to build using clang 15 and libc++15 , the error was use of undefined template for std::stringstream, this is quite common for something to compile under stdlibc++11 but fail under libc++ due to one library including a header in another header but the other library not.

Thank you very much @conradjones


Lint changes

These are mostly simple linting changes.  But also contains fix for https://github.com/cgrima/subradar/issues/1

Thanks, merged


Get current AppWindow?

How would I get a reference to the current AppWindow? I would expect something like AppWindow.GetForUIContext to be present.

WindowManagementPreview.SetPreferredMinSize(appWindow, new Size(MinWindowWidth, MinWindowHeight));

https://docs.microsoft.com/en-us/uwp/api/windows.ui.windowmanagement.preview.windowmanagementpreview.setpreferredminsize


What is python interpreter

Description 📜
In this i try to explain what is Python Interpreter
Fixes #79 (#1001)

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x ] Python

Checklist ✅

[x] I follow Contributing Guidelines & 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'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@prathimacode-hub mam please review this PR .

@roopali-1, I can't see your name tagged to this issue and assigned to you over issue number.

/assign


Update Web Chat to 4.9.2

Docs Build status updates of commit 573d4d5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

AdaptiveCards/resources/partners.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


thousandeyes active analysis methods for netunicorn

https://docs.thousandeyes.com/product-documentation/tests


Fix Dependency Review API response error handling

Bug: Failures in the Dependency Review API response are not properly handled due to an oktokit version mismatch
Fixes:

downgrade octokit/request-error to ^2.1.0

supported by actions/core ^1.10.0 (see)

Add integration test to invoke API (to avoid mocking a response that is dependent on the version of actions/core used) and validate exception response
Enhanced Forbidden error message back to user to hint that a culprit might also be lack of GitHub Advanced Security for a private repo

Graph Disabled / Private Repo w/o GHAS
Before:
::error::Forbidden

After:
::error::Dependency review is not supported on this repository. Please ensure that Dependency graph is enabled along with GitHub Advanced Security on private repositories, see https://github.com/OWNER/REPO/settings/security_analysis

Integration Test fails with 3.0.2

Succeeds with 2.1.0

Thank you for this @felickz, with the new integration test I'm sure we'll be prevented of breaking this behavior by auto-updating the errors library 😅.
This changes will appear in 3.0.3.


Have Jodit component UI decorators online demo ?

I don't know how to use component UI decorators。Are there any online demo?

Hi, https://codesandbox.io/s/decorators-u2h0os?file=/src/index.tsx


build on raspberry pi 4 (Raspbian bullseye)

hi!
following the wiki i built bluez-alsa tools without any errors.
sudo apt-get install libsbc1 libsbc-dev
sudo apt-get install libreadline-dev
autoreconf --install
mkdir build && cd build
../configure --enable-cli --enable-rfcomm --enable-systemd
make
sudo make install

(printed during installation:
Libraries have been installed in:
/usr/lib/aarch64-linux-gnu/alsa-lib)
but when i try to start bluealsa-aplay i see:
bluealsa-aplay: W: Couldn't get BlueALSA PCM list: The name org.bluealsa was not provided by any .service files
aplay -D bluealsa yourfile.wav gives this:
ALSA lib ../../../src/asound/bluealsa-pcm.c:1384:(_snd_pcm_bluealsa_open) Couldn't get BlueALSA PCM: The name org.bluealsa was not provided by any .service files
aplay: main:830: audio open error: No such device

bluealsa-aplay -V:
v4.0.0-123-g9cb3d75-dirty
cat ~/.asoundrc:
defaults.bluealsa.service "org.bluealsa"
defaults.bluealsa.device "DC:A6:32:FE:86:74"
defaults.bluealsa.profile "a2dp"
defaults.bluealsa.delay 10000

but i can hear sound with bluetooth with:
aplay -vv out.wav
how to start it working ?

You don't say which OS image you are using, but I'm guessing its either the "Desktop" or "Full" image. Assuming you have read the bluez-alsa project README you will know that to use BlueALSA "The target system must be able to function correctly with all its audio applications interfacing directly with ALSA". Both those images are heavyweight general purpose OS with integrated Bluetooth audio built-in, using pulseaudio. So BlueALSA is not required, and in all but a very few specialized cases it is not appropriate to install BlueALSA on either of those OS images.
I think you need to read the OS documentation for bluetooth audio, because almost certainly you should be using the built-in functionality (or else use a different image, such as the RaspberryPiOS "Lite" image, which does not have a desktop nor integrated pulseaudio).

correct i use "Desktop".
thanks for the answer.
think i could use bluealse-cli.


Add ImageJ 1.48 suite

ImageJ suite for scientific image processing and analysis. Includes
32bit and 64bit applications and default plugins. Depends on Java RE.

Thank you!


Terminal with code history

Hello,
I found it helpful, when the executed code line or bloc is also printed to the terminal. So I wrote a TerminalWithCode display for that. It works nicely with the fix #122 from @michaelb.
The output then looks like this:
  -------------------OK-------------------
│> print("''lol\"\"")
│''lol""

Thank you for this nice project.
I also added a minimum to the documentation. But I have concerns that I messed up the version number the cargo.lock. It shows version 1.0.6. Hmm?

Codecov Report

Merging #123 (b413756) into master (ba6c05c) will increase coverage by 0.08%.
The diff coverage is 70.96%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
+ Coverage   59.34%   59.42%   +0.08%
==========================================
  Files          35       35
  Lines        5667     5696      +29
==========================================
+ Hits         3363     3385      +22
- Misses       2304     2311       +7

Impacted Files
Coverage Δ

src/display.rs
56.78% <70.00%> (+1.76%)
:arrow_up:

src/lib.rs
58.20% <100.00%> (+0.10%)
:arrow_up:

Continue to review full report at Codecov.

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

Wow, thank you for the nice addition!
There's still a few things that are a bit off (mainly typos & format, but also sniprun shouldn't allow both Terminal and TerminalWithCode to be turned on at the same time), so I'd like to either:

redirect your merge onto michaelb:dev so I can merge now & easily fix the minor things later, before this goes into master
start a code review, but that may be a bit lengthy and uninteresting for you

Which one do you prefer ?
Also, don't mind about the version numbers in the cargo.toml / cargo lock update status, especially if you're pushing on the dev branch. I update those for each release, and you might just be off-sync

fix #124

Oh nice. Yeah, I prefer the first option (redirect it to dev).

sniprun shouldn't allow both Terminal and TerminalWithCode

Obvious! Totally forgot about that.
Thank you very much for your help.


cqlsh binaries should not be placed in .axonops-developer-workbench folder in users home directory - these should be bundled in the application

    ├── cqlsh-407
    ├── cqlsh-410
    └── keys_generator

These should not be here

Working on that, this will indicate changes in the related part of the building process in the package.json file

The data path for Windows is probably wrong also. Windows does not use .directory syntax. Can you look into this also?

Updated, now on Windows the folder won't have a dot prefix


Using a csv file to create groups should only allow creation of brand new groups

For all entries in the csv file, if the group already exists or if any of the members are part of a different group for the current assignment/does not exist in the student db, or if the repo name is already taken by a different group, don't create the group and show an error

Now that I'm looking at this more carefully, I can see problems with the approach.  If an instructor accidentally deletes a group there is no way to recover it.

If group name and repo match in the csv file and an existing group, and the group does not belong to a grouping, then we should allow it.  (The membership is lost when the grouping is deleted.)
If the group name and repo match in the csv file and an existing group.  If the group belongs to a grouping from another assignment and the student membership matches the csv file then we should allow it.

(I'm going to look at this further, but these are initial thoughts.)

Fixed in #2343


how will verification work in the security capabilities?

in the three types of verification as proposed in the security capabilities in voice document, how may these work?

temp uids?

temp uids?


Light problem when migrating from Godot 4.1.3 to the newest version (4.2.1)

Tested versions
4.2.1 stable
System information
Windows 10, Compatibility render mode
Issue description
I just updated to the Godot 4.2.1
And found this light system a bit crazy in Compatibility mode:
just to compare:
Godot 4.1.3 (without shadows, because not supported)

Godot 4.2.1 (with shadows and crazy light)

Both have 1 on Energy parameter.

If be short, shadows is the problem, exactly Normal Bias, but i dont know how to fix that.

Screen without Normal Bias looks like this and still some Hard light here:

Don't want to port my game to the newest version because of this. Please fix that!
Steps to reproduce

Create new project
Add some objects
Add Light environment
Change render to Compatibility mode

Minimal reproduction project (MRP)
light_test.zip

Also, add "Save Code Folding" option here:

Lights with shadows won't look exactly the same as lights without shadows. Unfortunately, due to limitations with older mobile devices, we have to implement shadows in the compatibility renderer using a multi pass rendering approach. The result of that is that lights with shaders are rendered in sRGB space, instead of linear space. So enabling shadows on a light will change its appearance (sometimes quite drastically)
If you want to enable shadows on a light, you will need to tweak the energy to get something that looks the same as an unshadowed light

Also, add the "Save Code Folding" option here:

What about code folding? It is not saving the folded code.

also, why this is still not fixed?
https://github.com/godotengine/godot/issues/67497
so many bugs...

It is fixed, just not cherry picked for 4.1.x, it works fine in 4.2

Please don't mix unrelated issues in the same report. This bug report is about light issues, if you have a proposal to improve code folding, open a dedicated proposal on https://github.com/godotengine/godot-proposals. And don't reference unrelated bugs, especially if they're marked as fixed already in the last version.

fixed already in the last version.

the last version is bugged itself.
the more time i work in Godot, the more i see that it was delevoped unprofessionally.

Code folding is already tracked here:

https://github.com/godotengine/godot/issues/85780

I asked you to keep issues separate, and you edited this issue to go clearly against that, mixing unrelated issues affecting different versions of Godot. Then you call us unprofessional...
 Either you behave professionally and follow our policies without insulting us, or you will be banned.

@akien-mga yep, i called you unprofessional, not exactly you, but the whole team worked on these bugged updates.
even if you ban me, what next? you can be proud of yourself, ok...
what will prevent me from logging into another account? nothing? - right!
to be short, if you want to fix something, don't make 2 issues from the 1.

Making a new account to circumvent a ban is a breach of GitHub's Terms of Service, but go ahead.


Getting a "Module Not Found" error when attempting to duplicate a record using buildClient

I'm running Next.js 14 and want to duplicate a record from within a route handler. Calling buildClient is causing the following error...
./node_modules/.pnpm/keyv@4.5.4/node_modules/keyv/src/index.js:22:14
Module not found
  20 | 	if (options.adapter || options.uri) {
  21 | 		const adapter = options.adapter || /^[^:+]*/.exec(options.uri)[0];
> 22 | 		return new (require(adapters[adapter]))(options);
     | 		            ^^^^^^^^^^^^^^^^^^^^^^^^^^
  23 | 	}
  24 |
  25 | 	return new Map();

https://nextjs.org/docs/messages/module-not-found

Heres a simplified version of my code.... (<YOUR_API_TOKEN> is the full access api token)
async function duplicateRecord() {
const client = buildClient({ apiToken: '<YOUR_API_TOKEN>' });
  // const itemId = 'awI8g4CtQguD9MXIo-2swg'
  // const item = await client.items.duplicate(itemId)
  // console.log(item)
}

export async function GET(req: NextRequest) {
  duplicateRecord()
  return NextResponse.json({ success: 'Duplicated Record' }, { status: 200 })
}

Packages...

  "dependencies": {
    "@datocms/cma-client-node": "^3.1.6",
    "@emotion/cache": "^11.11.0",
    "@emotion/react": "^11.11.4",
    "@emotion/styled": "^11.11.0",
    "@graphql-codegen/cli": "^5.0.2",
    "@mui/material": "^5.15.14",
    "@mui/material-nextjs": "^5.15.11",
    "@types/node": "20.11.30",
    "@types/react": "18.2.73",
    "@types/react-dom": "18.2.23",
    "embla-carousel": "^8.0.0",
    "embla-carousel-autoplay": "8.0.0",
    "embla-carousel-react": "8.0.0",
    "eslint": "8.57.0",
    "eslint-config-next": "14.1.4",
    "graphql": "^16.8.1",
    "next": "14.1.4",
    "react": "18.2.0",
    "react-datocms": "^5.0.3",
    "react-dom": "18.2.0",
    "react-icons": "^5.0.1",
    "react-inlinesvg": "^4.1.3",
    "react-intersection-observer": "^9.8.1",
    "typescript": "5.4.3"
  },

I have no idea what is the missing module here. We do not have keyv as a dependency. Would be interesting to investigate more if you have time.
What happens if you replace @datocms/cma-client-node with @datocms/cma-client?

Its working with @datocms/cma-client

Great, closing this issue then 👍🏼


Web - Profile - The text in quotes ("text") disappears from the profile name

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to URL https://staging.new.expensify.com/
and login with any account
Go to settings>Profile
Enter first name and second name, if this has not been done before
Put the first name and the second name in quotation marks

Expected Result:
The first name and the second name must appear in quotation marks
Actual Result:
The first name and second name disappear and appear blank
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/184993093-0b2f8098-b104-4cdb-876b-7084b3ce86cb.mp4
https://user-images.githubusercontent.com/93399543/184998653-781a7ef9-0b68-4cd1-8887-dd137e7c14e4.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

It seems this is coming from the back-end, we use this regex (\\w+([\\s_\\-\\.]+\\w+)*)(\/\\w+([\\s_\\-\\.]+\\w+)*)*. Maybe we should add this validation to the front-end as well instead of sending it through to the API.

Then that is a discussion to have with the rest of the team since we usually don't want to ease regex rules. I will post in the channel.

Holding on posting the job per discussion.

I will post in the channel.

@pecanoro , do you have a link to where you posted?!?!

Discussion here: https://expensify.slack.com/archives/C01GTK53T8Q/p1661172941760109

So we are probably changing this in the backend instead. I will take care and change the labels for it.

Issue not reproducible during KI retests. (First week)


selinux: Remove dead code

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Most of this code was unused for some time except for one utility
function that was folded into the selinux controller as a condition as
it was only used on that single place.
Which issue(s) this PR fixes:
None

Does this PR have test?
N/A

Special notes for your reviewer:
N/A
Does this PR introduce a user-facing change?

NONE

Codecov Report

Merging #601 (65936b5) into master (3b8e270) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #601   +/-   ##
=======================================
  Coverage   44.39%   44.39%
=======================================
  Files          26       26
  Lines        1347     1347
=======================================
  Hits          598      598
  Misses        709      709
  Partials       40       40

/lgtm

rebased on top of the recent CI changes

/test pull-security-profiles-operator-build-image

nice one @jhrozek 👍
/lgtm
/approve


build of dockerfile as part of build pipeline

We're currently lacking registry & building itself of dockerfile as part of CI pipeline.
I can implement that functionality so we can have official JIT image repo, but I'm not sure where's the right place to store the container images? Do you know @jpassing?

I agree that a pre-built image would make the deployment a bit easier. But it's difficult to keep container images up-to-date and fully patched at all times, which is why we generally don't provide pre-built images for open-source projects. So the lack of pre-built images is currently deliberate and I'm afraid not something we can change.

Fair enough - closing !


Jury references

For extra clarity on the objectivity of the evaluators, I think it would be great if the jury could (individually or as a group) write a small (Twitter-format or LinkedIn-reference style) comment for every awarded project which mentions any specific indicators leading to their decision. See also #71

Moving over my text from #73
We have identified that expert jury is sometimes involved in projects in their category.
Due to the interest of Jury in both open and topic and the limited size of local communities and networks, this can easily happen.
We should put a statement about how we handle this biased situation to avoid bias in the choice.


Improve STL operator so the enclosing app obj can run repetitively

In this PR, the STL Conversion has been improved

execution of compute will not affect the the object state, and only input data will be transformed to be the output
Fix the output to be in_memory STL binary in a try/catch block, so this output will only be available when the operator is a non-leaf node (current limitation on output mapping)
STL file is only saved in the app's output folder, if and only if a file path is provided, which is stored as operator private attribute

Changes are also made to the unetr example to demonstrate reusing the same app obj for processing multiple/batch of inputs,

__main__ functions have been updated to process a set of subfolders under the app input folder, each containing a DICOM Study/series, to demonstrate batch processing. Example code can be customized to be run with a main loop which gets discrete input with the use of file mon, queue, or other triggers like interrupt handler.

Example output (note, the input DICOM study was duplicated into separate folder, hence the same series instance UID appears in all output folders),
output
├── 1-SelectedSeriesForUNETR_output
│   ├── dicom_seg-DICOMSEG.dcm
│   ├── publish
│   │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1.nii.gz
│   │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1_seg.nii.gz
│   │   ├── config.meta
│   │   └── config_render.json
│   └── stl
│       └── multi-organs.stl
├── 2-SelectedSeriesForUNETR_output
│   ├── dicom_seg-DICOMSEG.dcm
│   ├── publish
│   │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1.nii.gz
│   │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1_seg.nii.gz
│   │   ├── config.meta
│   │   └── config_render.json
│   └── stl
│       └── multi-organs.stl
└── 3-SelectedSeriesForUNETR_output
    ├── dicom_seg-DICOMSEG.dcm
    ├── publish
    │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1.nii.gz
    │   ├── 1.2.8<IP_ADDRESS>80043.2.1125.1_seg.nii.gz
    │   ├── config.meta
    │   └── config_render.json
    └── stl
        └── multi-organs.stl

Signed-off-by: M Q<EMAIL_ADDRESS>

Thanks Ming! Looks good to me.

Thanks @gigony !


Dev check edr status

Coverage increased (+0.7%) to 91.304% when pulling 53fe7ba4c579103ed6ce6cc05e35f10212341e99 on dev_check_edr_status into f682bb9fb9f0aae8d3307796f6c8868ec45f28fa on check_edr_status.

Coverage increased (+0.4%) to 90.994% when pulling 809c70d1501f066182728de96d6f4cc270a49798 on dev_check_edr_status into f682bb9fb9f0aae8d3307796f6c8868ec45f28fa on check_edr_status.

Coverage increased (+0.7%) to 91.304% when pulling 5974472bf5539fbe3dd5855e3b307bbf1e819810 on dev_check_edr_status into f682bb9fb9f0aae8d3307796f6c8868ec45f28fa on check_edr_status.


Shorten Stefan Problem 1D tests

The solidification test probably doesn't need to run for so much simulated time.

https://github.com/travis-ci/travis-ci/issues/7181


add read/write event cache and avoid extra epoll_ctl syscalls

name: Pull request
about: Propose changes to the code
title: ''
labels: ''
assignees: ''

1. Are you opening this pull request for bug-fixes, optimizations or new feature?
optimizations
2. Please describe how these code changes achieve your intention.

add Events field to the struct PollAttachment in poll_data_unix.go，every time we add/mod read/write poll event in epoll_default_poller.go, we check if the event has been added, if already exists, we return directly without any syscall
3. Please link to the relevant issues (if any).

4. Which documentation changes (if any) need to be made/updated because of this PR?

4. Checklist

[x] I have squashed all insignificant commits.
[x] I have commented my code for explaining package types, values, functions, and non-obvious lines.
[ ] I have written unit tests and verified that all tests passes (if needed).
[ ] I have documented feature info on the README (only when this PR is adding a new feature).
[x] (optional) I am willing to help maintain this change if there are issues with it later.

This is a good insight, but unfortunately, each function of updating events is called precisely in gnet, thus, your new code is not going to do any read effect.

This is a good insight, but unfortunately, each function of updating events is called precisely in gnet, thus, your new code is not going to do any real effect.

well, thanks for your review


New package: Jets v1.0.0

Registering package: Jets
Repository: https://github.com/ChevronETC/Jets.jl
Created by: @jkwashbourne
Version: v1.0.0
Commit: 381e1e94b81142546d6550ffa4f259dc706546d4
Reviewed by: @jkwashbourne
Reference: https://github.com/ChevronETC/Jets.jl/commit/381e1e94b81142546d6550ffa4f259dc706546d4#commitcomment-42438334

Hello, the name is appropriate for this package.  Please see the package docs for more information.  Please merge.  Thanks!  cc: @aviks.

@samtkaplan, note, you want to add [noblock] to your comment (as explained above), and I was looking at CI, and I see e.g. (maybe false alarm):
┌ Warning: Some registries failed to update:
│     — `/tmp/jl_pvislB/registries/General` — registry detached
└ @ Pkg.Types /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.5/Pkg/src/Types.jl:1194

[noblock]
Hi @PallHaraldsson.  Thank you very much for your response.
I'm not sure what the reason is for the "Some registries failed to update" message.  Are you thinking it is that error that caused CI to fail, as opposed to the longer-than-four-character package name requirement?
With respect to the "Some registries failed to update" message, I'm not sure I know how to solve that, but let me know if I can do anything to help to understand it.
Thanks!!
Sam


fixed bug

fixed a bug

@wecomingtoyou bugs in your head, fix them first

@nevadskiy I don't have a bug in my head, but a feature


Horizon Lite: deploy on separate AWS infra

Get horizon light stack running on separate AWS environment.
This is dependent on bulk of infra being provided already by Ops - https://github.com/stellar/ops/issues/1818
Acceptance Criteria:

horizon light web, index(single/continuous), ledger exporter(continuous) deployed and running
clone the index and tx-meta s3 buckets over
horizon light web api ingress url is routed at new deployment.

@Shaptic , I've deployed the horizon light stack(k8s manifests) into horizon-lite-dev , updated the MVP docs to reflect.
the 'forward/watch' processes are running starting from a ledger of today(42654178) , pointed at the sdf s3 buckets. and the web is up at https://lh-dev-pubnet.prototypes.kube001.services.stellar-ops.com/, once ready, we can take down the older mvp deployment in horizon-dev, and could then change the web url back to https://lighthorizon-pubnet.prototypes.kube001.services.stellar-ops.com/
there are two jobs running on k8s now to copy the horizon s3 buckets over to the sdf s3 buckets.

Awesome, looks great to me!


Disable virtual host main

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevent result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

As discussed in below thread, this is just a hotfix and object could be achive just by make changes in odh-manifest. So I am closing this issue.
https://redhat-internal.slack.com/archives/C03UEGM7TAA/p1692206035812549


Option to hide updates-available badge for Extensions tab

I have enough extensions that there's typically 2 or 3 updates per day every one of them shows a blue update badge on the Extensions tab. This is really distracting because it's adjacent to the uncommitted changes badge on the Git tab and I that badge is really important to me. I don't want to update my extensions multiple times per day just to clear the badge. Can we have a please have a setting for disabling the extensions updates badge.

Doesn't the new 1.74 option on the Activity Bar icon's context menu already give you what you want?

Do you mean the Hide Badge option? You have to set it for every open VSCode window and then set it again every time each window gets restarted. It's as annoying as the badge itself and twice as pointless.

See 168939

Assigning to @lramos15 - the badge should not be shown on reload if it was hidden.

\duplicate #168939

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Dockerize helm for deployment

Hi
I am facing challenges in deploying helm code which have cloned to our server
1.First i tried with app service but it was getting stuck
2.Please help me in how to dockerize this so that i can deploy on aws ec2 instance

Hi @kapilmayank, we are currently not planning on publishing Docker images for HELM.
Dockerizing this should be similar to dockerizing any other Python package. You can refer to the documentation for installing and using HELM. You can also consider using a tool like pipx.