Please tell me how to extract features from training data

Excuse me, I need your help. I have modified a program based on a Streamlit application to run it as a command-line interface (CUI). As an additional process, I want to implement a feature where the extracted features of the training data images can be saved as data (or a model?) and then reloaded when the program is executed again. However, as I am a beginner, I am struggling to implement this properly. I would appreciate it if you could give me some advice. Thank you.

Hi, do you want to save the raw features (e.g. resnet50 features) or the features in the format of a specific approach? (e.g. spade features, padim features etc)

Thank you for your message. If possible, I would like to obtain the features of PatchCore. Best regards.

Excuse me. I've been doing some research, and is it correct to assume that I should use torch.save() to save the model?

Hey, over the weekend I started working on a .export method for spade and patchcore, but for patchcore it's only the torchscript that gets exported, not yet the onnx format. Need to take a further look at that.
check out the test_patchcore.py file for work so far
https://github.com/rvorias/ind_knn_ad/blob/fbea993f3b4b61c9667ec9a22b76db3174f5263b/tests/test_patchcore.py#L23:L48

Thank you for your response and handling.
I have reviewed the code. Personally, if it is possible to save the model in a .pt file and load that .pt file for inference during the next program execution, then using .onnx files is not particularly necessary


Compliance Policy Not Updating The Status For Particular VM

Environment: Version <IP_ADDRESS>.20200422181830_c7107ec
Description: For compliance policy, I have created a new compliance policy then mapped to a particular VM, and then running the policy on schedule once the start time completed then i checked in VM compliance status "Never verified" and "No history available".
Steps To Reproduce:

Login to CloudForm
Click on control
Click on Explore
Click on a policy
Select compliance policy
Keep appropriate condition
Map the VM and Add
Click on configurations
Click on Schedules
Create a new schedule for action type as "Virtual Machine"
Timer option select Run "Once"
Select time zone ( India)
Click on save

Actual:

Once the scheduled time passes checked in the VM summary page it is not showing any details in the compliance column.

Expected.

VM summary page should show status.

@Franklin-lab Could you verify that the compliance policy was executed after the scheduled time passed? Every policy executed would be logged in policy.log.

If scheduler is not required in your test, you can go to VM - Policy - Check Compliance of Last Known Configuration to kick out the compliance policy directly.


IScrollSnapPointInfo improvements

What does the pull request do?
Removed IScrollSnapPoints implementation from ItemsControl. These were convenience properties, but it exposes api that could otherwise be accessaccessed through the ItemsPanel. By removing them, api is more similar to WinUI's.
Irregular Snap Points data is now generated using the ItemsControl's ItemsSource Count, instead of the ItemsView Count. This should prevent realizing all items in large data sets. Partially fixes #11641, but the other issues remain.
What is the current behavior?

What is the updated/expected behavior with this PR?

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

Checklist

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

Breaking changes

Obsoletions / Deprecations

Fixed issues

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

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

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


Split implementation of checkbox and input

Having the checkbox extend from the input felt like being the
wrong abstraction. It also made the implementation of input a
bit more difficult to grasp what's going on.

@homu r+

:pushpin: Commit 9de06d3 has been approved by martndemus

:zap: Test exempted - status


Allow the user to decide whether the sticky scroll should prioritize outer or inner scopes (just like in Visual Studio). Additionally, allow the user to manually set the maximum amount of viewport that sticky scroll can take up.

see issue #208406 for prefer inner scopes option


Architecture decision re: Single Channel

This is more of a question than anything so it maybe be more appropriate in a mailing list but I didn't see one on the readme.
I'm just looking through the implementation and noticed that Hutch uses a single channel for all communication to rabbitmq. My understanding is that channels shouldn't be shared across threads, but the implementation in Hutch leads me to believe that concurrent calls to Hutch.publish would be done over the same channel.
Is this a concern? Is there a possibility of re-architecting the Hutch broker to issue channels from a pool that each thread can use?
If nothing is to change, should the Readme be updating specifying potential thread safety concerns?
I'm not entirely sure what the implications of multiple threads communicating over the same channel are as I don't fully understand the underlying protocol.

Hutch.publish can be used from multiple threads but it requires explicit synchronisation (see Bunny documentation). I’m not sure how this API can be rearchitected for multiple channels. For consumers it should be quite easy but publishing with a single class method will run either into the need to manually synchronise or a way to deal with race conditions between publishes on different channels in a pool.
A README note with a link to Bunny’s doc guide on concurrency is a good idea.
This rabbitmq-users post explains why concurrent publishing on a shared channel requires synchronisation.


Cannot run ember serve on M1

I am on a M1 Macbook. When I try to run ember serve this is what I get:
> ember serve
Browserslist: caniuse-lite is outdated. Please run next command `yarn upgrade`
WARNING: Node v16.14.0 is not tested against Ember CLI on your platform. We recommend that you use the most-recent "Active LTS" version of Node.js. See https://git.io/v7S5n for details.
Could not start watchman
Visit https://ember-cli.com/user-guide/#watchman for more info.
Browserslist: caniuse-lite is outdated. Please run next command `yarn upgrade`
⠋ building... JSON Server is running
⠦ building... [Babel: @ember/test-helpers > applyPatches]node:internal/validators:164
      throw new ERR_INVALID_ARG_TYPE(name, 'Object', value);
      ^

TypeError [ERR_INVALID_ARG_TYPE]: The "options" argument must be of type object. Received an instance of Array
    at ChildProcess.target.send (node:internal/child_process:733:7)
    at Array.forEach (<anonymous>)
    at dispatchQueuedRequests (/Users/ogirginc/Projects/shlack/node_modules/workerpool/lib/WorkerHandler.js:174:21)
    at ChildProcess.<anonymous> (/Users/ogirginc/Projects/shlack/node_modules/workerpool/lib/WorkerHandler.js:129:7)
    at ChildProcess.emit (node:events:520:28)
    at emit (node:internal/child_process:938:14)
    at processTicksAndRejections (node:internal/process/task_queues:84:21) {
  code: 'ERR_INVALID_ARG_TYPE'
}

Here are some additional data:
> ember --version
Browserslist: caniuse-lite is outdated. Please run next command `yarn upgrade`
WARNING: Node v16.14.0 is not tested against Ember CLI on your platform. We recommend that you use the most-recent "Active LTS" version of Node.js. See https://git.io/v7S5n for details.
ember-cli: 3.11.0-beta.3
node: 16.14.0
os: darwin arm64

Is there a quick fix I can apply to start the course?

I don't know if this is OK for the whole course, but I was able to run ember server by following https://github.com/ember-cli/ember-cli/issues/9404#issuecomment-857643079.

This fix worked for me as well, although I haven't tested it through the whole course yet.


Add ioctl FS_IOC_* version and flag constants

Rebase of #3089 that was approved but never merged due to a CI failure.

Ah, bors does a rebase so you don't have to rebase it manually. Thanks for the heads-up anyway.


Implement Select All in multiselect dropdown

Currently from the docs, i do not find an option to select all the items from multi select dropdown. There is an option to clear all however need an option to Select all. WIll this be included or how to implement it? Currently Im developing a react application based on hooks

Hi,
I am getting a warning when I use the value prop for my Dropdown multiple selection. I have updated semantic-react-ui to the latest version 1.2 even then I get this and the multiple selection dropdown does not respond:-
Warning: Failed prop type: Invalid prop value supplied to Dropdown.

@chandra-exp we use Github issues only to track bugs and feature requests. Please raise your question on StackOverflow, we have a dedicated tag there https://stackoverflow.com/tags/semantic-ui-react 😸

Warning: Failed prop type: Invalid prop value supplied to Dropdown.
Can you please share a minimal example on CodeSandbox for this?


MINOR: Fix log message when tasks directory is cleaned manually

Currently when a task directory is cleaned manually the message
for the state dir cleaner is logged instead of the message for
the manual cleanup. This is because the code checks the elapsed
time since the last update before it checks whether the cleanup
is a manual call. This commit changes the order of the checks.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review: @guozhangwang @ableegoldman @mjsax @vvcephei @wcarlson5

Retest this please

@cadonna
org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage failed, log available in /home/chia7712/kafka/streams/build/reports/testOutput/org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage.test.stdout

org.apache.kafka.streams.processor.internals.StateDirectoryTest > shouldLogStateDirCleanerMessage FAILED
    java.lang.AssertionError:
    Expected: a collection containing a string ending with "ms has elapsed (cleanup delay is 0ms)."
         but: was empty
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at org.apache.kafka.streams.processor.internals.StateDirectoryTest.shouldLogStateDirCleanerMessage(StateDirectoryTest.java:569)

the new test introduced by this PR is failed. The failed test does not update the current time of MockTime so the state directory can't be cleanup. I have filed a ticket for that failed test (https://issues.apache.org/jira/browse/KAFKA-10514).

Cherry-picked to 2.6


feat: updates the user instructions re OAuth

Provides user instructions as part of the pandas-gbg documentation to support authentication using OAuth for highly constrained development environments.

@tswast

Content LGTM, but I think this might be better on a separate page so that we don't overwhelm folks for whom the default flow works fine.

Good point. I separated out the content into two pages. I added cross references within each document to help ensure that users are guided to the appropriate content.


AttributeError: type object 'InvokeOptions' has no attribute 'merge'

What happened?
Hi.
Seems there's a missing merge attribute on the InvokeOptions.
Here's our code:
def create_pulumi_program():
    registry = compute.get_flavor(name=hashed_registry_name)
    pulumi.export("type", registry.flavor_name)

stack = auto.create_or_select_stack(stack_name=hashed_registry_name, project_name=user_email, program=create_pulumi_program)

up_res = stack.up()

And the result:
Diagnostics:
  pulumi:pulumi:Stack (idriss.neumann@comwork.io-test-reg-ovh-ml1isq):
    error: python inline source runtime error: type object 'InvokeOptions' has no attribute 'merge'
    Traceback (most recent call last):
      File "/usr/local/lib/python3.9/site-packages/pulumi/automation/_server.py", line 69, in Run
        loop.run_until_complete(run_in_stack(self.program))
      File "/usr/local/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
        return future.result()
      File "/usr/local/lib/python3.9/site-packages/pulumi/runtime/stack.py", line 126, in run_in_stack
        await run_pulumi_func(lambda: Stack(func))
      File "/usr/local/lib/python3.9/site-packages/pulumi/runtime/stack.py", line 49, in run_pulumi_func
        func()
      File "/usr/local/lib/python3.9/site-packages/pulumi/runtime/stack.py", line 126, in <lambda>
        await run_pulumi_func(lambda: Stack(func))
      File "/usr/local/lib/python3.9/site-packages/pulumi/runtime/stack.py", line 149, in __init__
        func()
      File "/app/src/strategies/OvhStrategry.py", line 65, in create_pulumi_program
        registry = compute.get_flavor(name=hashed_registry_name)
      File "/usr/local/lib/python3.9/site-packages/pulumi_openstack/compute/get_flavor.py", line 212, in get_flavor
        opts = pulumi.InvokeOptions.merge(_utilities.get_invoke_opts_defaults(), opts)
    AttributeError: type object 'InvokeOptions' has no attribute 'merge'
    error: an unhandled error occurred: python inline source runtime error: type object 'InvokeOptions' has no attribute 'merge'
    Traceback (most recent call last):

Thanks for your help
Expected Behavior
InvokeOptions should have a merge method.
Steps to reproduce
Just try this piece of code:
def create_pulumi_program():
    registry = compute.get_flavor(name=hashed_registry_name)
    pulumi.export("type", registry.flavor_name)

stack = auto.create_or_select_stack(stack_name=hashed_registry_name, project_name=user_email, program=create_pulumi_program)

up_res = stack.up()

Output of pulumi about
$ docker exec -it comwork_cloud_api_ppd bash
root@af62a64b35aa:/app# pulumi about
CLI
Version      3.55.0
Go Version   go1.20
Go Compiler  gc

Host
OS       debian
Version  11.6
Arch     x86_64

Backend
Name           af62a64b35aa
URL            s3://tf-pulumi-state?endpoint=s3.fr-par.scw.cloud
User           root
Organizations

Pulumi locates its logs in /tmp by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

We're confident this is resolved when using Pulumi 3.35 & the 3.35 Python SDK or newer, please let us know if not by creating a new issue for us to track.


Changed imports to correct package names

@mkurapov Router and Scene are already exported from react-native-mobx: https://github.com/aksonov/react-native-mobx/blob/master/index.js#L72-L73
With that in mind, the exported Routerfrom react-native-mobx in a modified version of it: wrapped with MobX observer.

@amrnt Thank you for the explanation. I believe I was doing something wrong in my own project and wasn't using MobX correctly. For some reason the importing it from the original library (RNRF) fixed my issue.


How to generate prediction of instance segmentation without bounding box, class, edge and probability?

When I finish training for instance segmentation and use demo.py to generate masks, I get the result of the first image.
First image includes box and class(0 in this case) and probability.
Also, the segment object has edge with different colors.
I want to ask how to generate the mask like the second image.
I want to generate image without edge, box, probability and class.
Hope someone can help and thank you so much.

Why am I the opposite of you, I need boxs, but only mask

@aihcyllop How did you go about training with your own example segmentation dataset? Do you have any reference material? Thank you very much!

can u show how did u generate mask using demo.py

can u show how did u generate mask using demo.py @aihcyllop

do u mean this ?
python demo/demo.py --config-file configs/coco/instance-segmentation/dinat/maskformer2_dinat_large_IN21k_384_bs16_100ep.yaml --input datasets/"yourdatasetfolder"/*.png --output demo/"foldertoputresult" --opts MODEL.WEIGHTS output/model_final.pth


THX !!!

Well, naztronaut, you saved my weekend.
This expand for scott's version is AWESOME.
I'am not a coder and can never do that - Scroll makes me really really happy :-)) THX
May more effects in the future ?
keuschebub
320x200_scroll.zip

Glad you found this version helpful!
I hand't thought about adding more effects but I can definitely add it to my backlog. Any ideas on what kind of effects you'd find helpful? The Scroll, Spectrum, and Energy covers such a big range of effects you'd want to see on lights but I'm happy to consider adding on a few more if possible.


Update DDL

This includes the following changes:

Add formatting to match the main geopackage DDL
Correct tile_matrix_set_name column name in gpkg_2D_gridded_coverage_ancillary table
Add missing data_null and data_missing columns in gpkg_2D_gridded_coverage_ancillary table
Add CONSTRAINT names to more closely match main geopackage DDL and aid debug
Add missing UNIQUE constraint on gpkg_2D_gridded_tile_ancillary table

Any issue?

It looks okay to me but we'll review it as a group on the 10th.

These changes look fine to me. However, the GeoPackage spec states "All specified table, view, column, trigger, and constraint name values SHALL be lowercase." therefore we should probably change gpkg_2D_gridded_coverage_ancillary and gpkg_2D_gridded_tile_ancillary to be all lowercase.

https://github.com/opengeospatial/geopackage-elevation/issues/14 to handle the wider changes for table names.


Tile service with factor of 3 not working

Hi,
Currently your viewer doesn't support a tile service with max zoom level of 3 and factor is 3. i.e.
at zoom level 1 - there is one tile for each face
at zoom level 2 - there is 9 tiles for each face (row =3 and col =3)
at zoom level 2 - there is 81 tiles for each face (row =9 and col =9)
What changes i would have to make to support this type of tile service? I am using multires mode. I think you only support the tile service with a factor of 2
Thank you.

The only tiles officially supported are those generated by the included generate.py script. However, most other tiles with a "factor of 2," per your nomenclature, should work. Supporting the tile scheme you're describing would require extensive changes to the testMultiresNode method in libpannellum.js. You're free to make the changes, but I have no intention of ever including such support.


fix: enable modules and xorriso flags per arch

Fixes building ISOs for the arm64 efi target

Okay, this is trivial enough to bypass review, and we really need these UM isos quick :/


Update Version Bump to include Homebrew in PATH

Description
GHA removed Homebrew from their PATH var so adding it back in.
Also did some refactoring and moving all homebrew installs to a separate step and also cleaning up newline formatting issues like we cleanup extra whitespace
Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

The GitHub issue on this breaking change is here
https://github.com/actions/runner-images/issues/6283


Changes required to release Core

Motivation
Adjust repository structure in order to support release.
Test release using different name.
We are making assumption that all packages will use matching versions.
If we can approve current aproach I will merge that and do proper release of the core.
Progress

[x] Publish testing pod
https://github.com/CocoaPods/Specs/commit/f91fee0a68cbe8ff0ffe109733f068f23a1f285a
[x] Use test pod in example project

Downloading dependencies
Installing Alamofire (4.7.2)
Installing ObjcExceptionBridging (1.0.1)
Installing TestCorePublish (0.1.0)
Installing XCGLogger (6.0.2)

Separate PR

[ ] Publish core after merge
[ ] Document the whole process.

@wei-lee I have spent very long time trying to do subspec validation to work. Mostly dealing with some cocoapods issues. Decided to use current aproach as it took just to long.
@wei-lee We could configure travis to do releases when we create new release on github. WDYT?

@wtrocki that will be great. But I don't think it's required as part of this PR. We can add automation later.


Using params or options with ES6

Hey I really like this package, only problem is that I'm trying to send a parameter on connection to the server. I an do this if I include the params in the URL, however it doesn't work if I just include an options object as an arg with the EventSource function. You guys seem to be using typescript to do that. How can I achieve the same in NodeJS?
Thank you

Hey, what parameter you try to send?


use JITBEAM instead of BEAM in benchmark CI for llama and gpt2

search less kernel, should be faster

i did not expect the final searched kernels to change, but it's 10% slower on all 3 backends


⚠ Bofrost.at has degraded performance

In 7ee3b1b, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 929 ms

Resolved: Bofrost.at performance has improved in b5c8edf.


Graph import gives graph with 0 nodes

Whenever I try to import a model, the return is true but the graph has no nodes
I'm not sure if I'm doing something wrong.  I'm able to run the model using python.
Tensorflow.Net 0.11.3 was a direct VS nuget install
Microsoft.ML.TensorFlow.Redist.0.14.0
The model I am using is from FaceBoxes

Also, I'm having the same issue when I'm importing the models in Tensorflow.Net Examples.

Have you tried use this redist https://www.nuget.org/packages/SciSharp.TensorFlow.Redist/

Thank you, that helped.
But I think the below is what caused the confusion.
I was using the graph.get_operation_by_name from the example here which was saying there was no such operation. Changing that to graph.OperationByName worked.


Compressibility Check using x265 2.8+69 failed with exit code: -1073741795 (0xC000001D)

Hi Revan654, Just tried the v<IP_ADDRESS> but got this error. Log is here -
(with path omitted for obvious reasons )
Error Compressibility Check using x265 2.8+69 (<IP_ADDRESS>)
Compressibility Check using x265 2.8+69 failed with exit code: -1073741795 (0xC000001D)
The exit code might be a system error code: {EXCEPTION}
Illegal Instruction
An attempt was made to execute an illegal instruction.
--------------- Compressibility Check using x265 2.8+69 ---------------
"D:....\StaxRip [x64]\Apps\avs2pipemod\avs2pipemod64.exe" -y4mp "C:\Users....\Stax-Rips--[6] BTS-001_temp--[6] BTS-001--XMr.P2dX_CompCheck.avs" | "D:...\StaxRip [x64]\Apps\x265\x265.exe" --crf 18 --profile main10 --transfer bt709 --frames 448 --y4m --output "C:..\Stax-Rips--[6] BTS-001_temp--[6] BTS-001--XMr.P2dX_CompCheck.hevc" -
avs2pipemod[info]: writing 448 frames of 30000/1001 fps, 720x400,
sar 0:0, YUV-420-planar-8bit progressive video.
avs2pipemod[info]: finished, wrote 0 frames [0%].
avs2pipemod[info]: total elapsed time is 3.846 sec.
avs2pipemod[error]: only wrote 0 of 448 frames.
StaxRip.ErrorAbortException: Compressibility Check using x265 2.8+69 failed with exit code: -1073741795 (0xC000001D)
The exit code might be a system error code: {EXCEPTION}
Illegal Instruction
An attempt was made to execute an illegal instruction.
--------------- Compressibility Check using x265 2.8+69 ---------------
"D:....\StaxRip [x64]\Apps\avs2pipemod\avs2pipemod64.exe" -y4mp "C:\Users....\Stax-Rips--[6] BTS-001_temp--[6] BTS-001--XMr.P2dX_CompCheck.avs" | "D:...\StaxRip [x64]\Apps\x265\x265.exe" --crf 18 --profile main10 --transfer bt709 --frames 448 --y4m --output "C:..\Stax-Rips--[6] BTS-001_temp--[6] BTS-001--XMr.P2dX_CompCheck.hevc"  -
avs2pipemod[info]: writing 448 frames of 30000/1001 fps, 720x400,
sar 0:0, YUV-420-planar-8bit progressive video.
avs2pipemod[info]: finished, wrote 0 frames [0%].
avs2pipemod[info]: total elapsed time is 3.846 sec.
avs2pipemod[error]: only wrote 0 of 448 frames.
at StaxRip.Proc.Start() in C:\Users\Revan\Desktop\staxrip-<IP_ADDRESS>\General\Proc.vb:line 338
at StaxRip.x265Enc.Encode(String passName, String commandLine, ProcessPriorityClass priority) in C:\Users\Revan\Desktop\staxrip-<IP_ADDRESS>\Encoding\x265Enc.vb:line 65
at StaxRip.x265Enc.RunCompCheck() in C:\Users\Revan\Desktop\staxrip-<IP_ADDRESS>\Encoding\x265Enc.vb:line 105

If the paths come close to 256ish characters, Then you need to enable Longpaths.

Hey, thanks for the response!
Where do I enable Longpaths?
From: Revan654<EMAIL_ADDRESS>Sent: Samstag, 20. Oktober 2018 21:55
To: Revan654/staxrip<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [Revan654/staxrip] Compressibility Check using x265 2.8+69 failed with exit code: -1073741795 (0xC000001D) (#24)
If the paths come close to 256ish characters, Then you need to enable Longpaths.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Revan654/staxrip/issues/24#issuecomment-431613145 , or mute the thread https://github.com/notifications/unsubscribe-auth/AHIFPY1N36Ao5bru_QzfQtDvKR6TYC2iks5um3-IgaJpZM4XxnjX .  https://github.com/notifications/beacon/AHIFPdTH8OpqMwSBL2SO9x01nPjvS0dlks5um3-IgaJpZM4XxnjX.gif

In groupPolicy or if you know what your doing regedit.
Also when you install Python, it gives you the option to enable it as well.


Mzuevx oar 3d

support for OAR_3D accuracy test

Commit must be merged into opencv/develop, so close PR


Extension makes Twitter really buggy

(This is on Firefox, without the GoodTwitter extension or anything similar to it)
The account switching menu (found on the bottom-left of the page) has trouble loading. Opening tweets/threads is also buggy; the replies don't appear until you slightly scroll through the page (which rarely makes them impossible to view), and if the tweet is a reply to something you are unable to fully scroll up through the thread.

Thanks for the report! The next update (which is ready to go and should be on Firefox within the next few days) should fix this.


Change the names used for emb.type

Goal

Testing

Release Notes

WHAT:
WHY:
WHO:

#590  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bidetofevil and the rest of your teammates on  Graphite


User modifiable bridge address for configurator?

Hi arm,
Great work on this. I am thoroughly enjoying the echo controlling all of my automation devices.
I created a docker for this, and several people have been using it: https://registry.hub.docker.com/u/aptalca/docker-amazon-echo-ha-bridge/
But one issue is that, configurator.html has the bridge address hardcoded as localhost. Unfortunately in docker the bridge is unreachable through localhost (it should be as I'm using host mode, but can't figure out why it is not) and the local ip of the server needs to be used. It results in the CORS error being displayed, but the real issue is that localhost is not reachable (I tried disabling web security in chrome and that did not work either)
To test, I replaced localhost with the local server ip in configurator.html in the jar file and everything seems to work that way.
Is it possible to have the bridge address user selected/modified, or even extracted from the url?
Another idea is that, the docker requires the input of the local server ip so it can run java with the upnp.config.address=$SERVERIP parameter. $SERVERIP is passed into the docker container as an environment variable. Perhaps that could be used for the configurator.html?
I'm sure you'll have better ideas on how to get around it.
Thanks

For now, I am using a very hacky way to get around the limitation.
Once the docker container starts, it replaces the "localhost" with $SERVERIP using sed in a separate configurator.html and then it updates the configurator.html within the jar file with the modified version before running java
It works, but it would require a lot of work every time there is a new release.
Thanks

I submitted a pull request that fixes this, it uses the current address the page was loaded from. It also allows you to reload the devices if you have a different configuration.
See Pull request https://github.com/armzilla/amazon-echo-ha-bridge/pull/13

Looks great :-) Thanks
Hopefully Arm will merge soon. I see that he hasn't been active on github lately

merged #13.    Can someone improve the MD?


Handle comments between Trait Generics and Bounds

Add handling of comments between Trait Generics and Bounds by using rewrite_assign_rhs_with_comments() (one of the cases identified in #4626 discussion).
Also added workaround for an issue that a comment before the Ident caused code duplication.  The workaround is to use the original code in this case.
The PR also partly enhance the resolution of #2055.

Thanks for the updates! I may have a very minor nit or two as a final review pass, but I think we're otherwise in good shape.
I want to think on this a bit longer to make sure we're not overlooking any scenarios. In cases like these it's possible that folks may feel like after upgrading rustfmt that it is "changed" its formatting, but really rustfmt is just finally able to format these trait/comment scenarios. It's definitely a fix we want/need, but it's one we need to be sure we've got right/fully covered up front (because we'll be limited in our ability to change it afterwards)


MB-10622 Adds NTS Recorded Weight to MTOShipment

Jira ticket for this change
Summary
Follow-on to PR https://github.com/transcom/mymove/pull/7732.
This PR adds an NTS recorded weight to MTOShipment. This weight is needed as a reference on an NTS release shipment. The counselor is expected to create/update the NTS shipment release with the old NTS weight.
The counselor can be a services counselor or the Prime.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Messages

:book:
:link: MB-10622

  Generated by :no_entry_sign: dangerJS against db4389999f89c38dcb50b3a5f4acbb297419b4fe


Logo image broken in update email

Update from this Collective is showing a broken logo image.

Effectively. Looks like they don't have a logo, should the Open Collective logo be displayed instead (like on the collective page) or should it just be ignored?

Ignored is better to avoid confusion.

I think it should just display the Collective's name in text, which also links to the Collective.


Latency of httpRequest not tracked when using library with redirectToStdout

Is this a client library issue or a product issue?
This seems to be a client library issue.

Did someone already solve this?
I have searched through issues in the listed repositories but I have been unable to find the issue I am facing.

Do you have a support contract?
No.

If the support paths suggested above still do not result in a resolution, please provide the following details.
Environment details

OS: Debian 12
Node.js version: 18.16.1
npm version: 9.5.1
@google-cloud/logging-winston version: 5.3.0

Steps to reproduce

Log requests with the following code:

import * as gcpLogging from "@google-cloud/logging-winston";

const logger = createLogger({
  transports: [
    new gcpLogging.LoggingWinston({
      redirectToStdout: true,
    }),
  ],
  exitOnError: false,
});

const app = express();

app.use(await gcpLogging.express.makeMiddleware(logger));

In an environment with Cloud Logging Agent, make requests to this service. When browsing the logs in Log Explorer, it can be seen that httpRequest data such as status, size and user agent is tracked properly, but the latency (Duration object) is not tracked.

I believe this bug is due to a difference in schema where the cloud logging agent expects an output in the JSON format, but the library prints the output in the protobuf format, causing the logging agent to miss this value. (Related issue: https://github.com/googleapis/nodejs-logging-winston/issues/93)

Thanks @ktaekwon000 for opening the issue. By design Logging client libraries call the gRPC Logging API, and the latency in HttpMessage is expected to be protobuf.
We cannot use latency as a single string (e.g. 3.5s) in our code since Duration is a protobuf structure expected by Cloud Logging service.  I believe that the issue needs to be fixed in stackdriver-logging-agent.
The documentation with correct links will be fixed here: https://github.com/googleapis/nodejs-logging-winston/pull/828.


Proposal: Set external API keys through functions

Problem being addressed:
The current method of setting API keys by specifying headers can be cumbersome. After client instantiation, changing the key is also quite difficult (not that this happens a lot).
Proposal:
Add methods to easily add or change API keys, for example like:
    def set_apikey(self, openai_key: Optional[str] = None, cohere_key: Optional[str] = None):

        if openai_key:
            self._connection._headers["x-openai-api-key"] = openai_key

        if cohere_key:
            self._connection._headers["x-cohere-api-key"] = cohere_key

What do you think?
(They should ideally be attributes too imo so that they can be easily reviewed.)

@tsmith023 @databyjp Can I work on this issue?

Hi @ankur0904, that would be great! Thank you for taking it up 😁
My recommendation would be to have the function as a method within the Client class as defined in weaviate/client.py

Hey @ankur0904 fantastic!
The initial implementation should include support for keys for cohere, huggingface, openai and palm imo.


Idea: Animation helper

Added helper in "utilities/css/animations.css". General hw-animate and hw-animate--slide-in-bottom is the only animation classes for now. Could be an idea to define animation variables for use in css instead of wrapping stuff inside animation divs.


Support for detection of tool versions in .tool-versions file

Title: Support for detection of tool versions in .tool-versions file
Issue Description:
Currently, the project supports the automatic detection of Node.js versions through the .node-version file. However, it would be highly beneficial to extend this functionality to include detection of tool versions specified in the .tool-versions file.
Proposed Solution:
To achieve this, we can enhance the version detection mechanism to recognize and apply the specified tool versions from the .tool-versions file. This improvement will streamline the development process by ensuring that the correct tool versions are utilized consistently across different environments.
Expected Behavior:

When a .tool-versions file is present in the project, the system should automatically identify and use the specified tool versions.
In cases where a specific tool version is not available, the system should provide a clear error message or prompt the user to install the required version.

Additional Information:

This enhancement will contribute to a more seamless and efficient development workflow, especially for projects that rely on specific versions of various tools.
Compatibility with a broader range of tools will further empower developers and facilitate smoother collaboration within the project.

Steps to Reproduce:

Create a .tool-versions file in the project directory.
Specify the desired tool versions in the following format: tool_name version_number.
Run the project or relevant tooling that relies on version detection.

Acceptance Criteria:

[ ] The system successfully detects and applies tool versions specified in the .tool-versions file.
[ ] Clear error messages or prompts are provided in cases where a specified tool version is unavailable.

I assume this is the spec for .tool-versions, based on a quick search?

I assume this is the spec for .tool-versions, based on a quick search?

Yes, sorry. I forgot to attach the source of that specification. Although it is something promoted by asdf, it is widely used to configure tool versions in development environments. Other tools like rtx also support this specification.
Thanks in advance 🙏

This sounds good and would make it easier to migrate from rtx / asdf. (my previous go to)

cool, can do and thanks for the copious detail! Probably if asdf or rtx are added to the env though we should delegate to them.

My vote would be not to implement this. The .asdf file is specific to asdf, and right now I'm finding it very necessary in to use asdf as a fallback if pkgx doesn't yet provide a library. So I load asdf shell integration in addition to pkgx shell integration. Some perfect examples include Ruby 2.0 and any Node version lower than 12.
So I am defining only packages in .tool-versions that I know pkgx does not yet have, and I'd rather have pkgx.yaml and .tool-versions coexist as independently, rather than see a bunch of errors from pkgx when I cd into a directory with a .tool-versions file full of stuff that isn't in the pantry.
I guess the one caveat would be if pkgx could have this as a configurable option, much like asdf has legacy_version_file = no. Or maybe even better if pkgx wouldn't stop processing and error out as soon as it hits a dep it doesn't have.

Okay, I just saw @mxcl comment about delegating to asdf if it's there. I guess that could work too🤔 . My concern is coming from difficulties I'm already having with pkgx trying to optimistically load anything it can find (package files, lock files, .ruby-version, .node-version, etc.). Often times I'm working in repos that other orgs own, with a bunch of legacy stuff that might not be easy to change. So I want to be able to just define a pkgx.yaml and not get a bunch of conflicting version errors when I try to cd into the dir. Delegating to asdf if in the path sounds like a good idea. But I also like a feature similar to asdf's  legacy_version_file = no.


Add springs to grabber to reduce wobble

Adds springs and dampening to the grabber to reduce wobble when carrying a cube.
Before:
https://user-images.githubusercontent.com/2551763/216776915-68a312e6-22c7-43ec-a87c-ff0608255b42.mp4
After:
https://user-images.githubusercontent.com/2551763/216777038-4f1abb0a-76d7-4303-8d74-492fd9418730.mp4

@Adimote I think you included this in the update for the SR2024 robot?


Docker make commands fail when user id is not 1000

Currently, the rust files installed during docker build are owned by the 'mythril' user in the container. This user has id 1000. If the user calling docker has a user id other than 1000, the build will fail as they can't write to that RUSTUP_HOME. It's not clear how to fix this, as we want the artifacts generated by the build to be owned by the users ID.
I think may be able to just chmod 777 everything owned by the mythril user during the docker build, but I'm not sure.

Fixed in 421ba93


enable Dependabot v2

https://github.blog/2020-06-01-keep-all-your-packages-up-to-date-with-dependabot/

@gracekarina

WDYT? @gracekarina

see comment about related PR in swagger-core


Syntax highlighting for hovers

Especially when hovering over a subroutine name then the big hover text with argument list and types is a bit daunting without syntax highlighting. If possible, it would be great if this was Fortran-highlighted.

Hmmmm, hover boxes should be highlighted. Can you tell me what other extensions you are using and give me an example? Do you ever see syntax highlighted hover content or is it always plain text?
I have noticed that sometimes VSCode generates its own hovers from the definition location and displays that region as non-highlighted text. However, when/why this happens doesn't appear to be too consistent. Although, I think generally it doesn't happen when the hover request returns a result.

I have gimly81.fortran (fortran) installed. When I disable this and instead install krvajalm.linter-gfortran (Modern Fortran) then the highlighting works.

I'm afraid there is an issue now. Somehow the hover-text is not working correctly anymore, but it was all working like a charm in the last version. Here are two examples:

My extensions are unchanged:

krvajalm.linter-gfortran v2.1.1
hansec.fortran-ls v0.6.0 (why 0.6.0 anyway? It's working great!)
ekibun.fortranbreaker v0.0.4

@AndreHorst Thanks for the report! Sorry about that. I pushed a patch (1.11.1) to fix this a little while ago. See fortls issue #159 for more information.


Added "Driver Time" column to represent the total time in the driver

This adds a new column to the timings table: "Driver Time", which represents the total time in the driver program, including time spent waiting for Spark operations to complete. This is based on feedback from @fnothaft and @erictu. The time excluding waiting for Spark operations to complete has been moved to the "Driver Only" column.  I've also added the driver time to the Spark Operations table as well.
I've also updated the docs, so hopefully everything is clearer there too.

Merged build triggered.

Merged build started.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/bdg-utils-prb/51/
Test PASSed.

Thanks @nfergu! This is stellar as always.


chore: bdd-sbhg-1587847935 to 0.0.1

chore: Promote bdd-sbhg-1587847935 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


release-arm-frontdoor-fix-issue-8399

This is a release for arm-frontdoor to fix issue https://github.com/Azure/azure-sdk-for-js/issues/8399


extract.mdJSON.R function

The extract.mdJSON.R function is not recognized by the package. I had to download the source code file from the GitHub page and load it into my script using source(extract.mdJSON.R). Not sure why?

Did you download the latest release (v0.1.2) or the development version of mdJSONdictio? extract.mdJSON hasn't been included in a release yet.

I followed the readme and loaded the v0.1.2 version:
devtools::install_github("hdvincelette/mdJSONdictio", ref = 'v0.1.2')
I don't see a dev branch or development version.

I just created a new release of mdJSONdictio (v0.2.0) which includes extract.mdJSON.
The development version can be installed with:
devtools::install_github("hdvincelette/mdJSONdictio", ref="master")


Getting the issue while using the Abstract class.

Hello,
I have a models as:
class BaseDocument(mongoz.Document):
    """
    The base document for all documents using the `registry` registry.
    """
    objects: ClassVar[CustomManger] = CustomManger()

    class Meta:
        """
            It contains the Meta information related the database for \
                each model level.
        """
        abstract = True
        registry = settings.client
        database = settings.db

class Employee(BaseDocument):
    month_period: int = mongoz.Integer(null=True)
    full_name: str = mongoz.String()
    employee_code: str = mongoz.String()
    date_of_joining: datetime.datetime = mongoz.DateTime(null=True)
    e_mail: str = mongoz.String(null=True)
    mobile_no: str = mongoz.String(null=True)

When I am trying to query into db, Its using the default manager i.e. "Manager" object, instead of "CustomManager".
If I again initialize the "objects" in the Employee Document and try to query as:
Employee.objects.create(full_name='Emp_Name_1326', employee_code='01326')
Then, it returns the API response as:
{
    "detail": [
        {
            "type": "missing",
            "loc": [
                "month_period"
            ],
            "msg": "Field required",
            "input": {
                "full_name": "Emp_Name_1326",
                "employee_code": "01326"
            },
            "url": "https://errors.pydantic.dev/2.6/v/missing"
        },
        {
            "type": "missing",
            "loc": [
                "date_of_joining"
            ],
            "msg": "Field required",
            "input": {
                "full_name": "Emp_Name_1326",
                "employee_code": "01326"
            },
            "url": "https://errors.pydantic.dev/2.6/v/missing"
        },
        {
            "type": "missing",
            "loc": [
                "e_mail"
            ],
            "msg": "Field required",
            "input": {
                "full_name": "Emp_Name_1326",
                "employee_code": "01326"
            },
            "url": "https://errors.pydantic.dev/2.6/v/missing"
        },
        {
            "type": "missing",
            "loc": [
                "mobile_no"
            ],
            "msg": "Field required",
            "input": {
                "full_name": "Emp_Name_1326",
                "employee_code": "01326"
            },
            "url": "https://errors.pydantic.dev/2.6/v/missing"
        }
    ]
}

Can you help me for this?
Is somthing wrong here?
Thank you.

@harshalizode Mongoz actually does not support custom managers, at least I did not add those and I think this would require some additional implementation.
When you use the default "manager". Does it work properly? I would assume so which means if you want an additional manager or custom, that would require to create a feature that supports it.
Edgy and Saffier do support it and for mongoz there was never the case of that need but we can add it, for sure.

No, when I commented the CustomManager in the BaseDocument, the behaviour remained the same.

Ok a couple of questions.

If the class is not abstract, does it work with the manager and custom manager?

Because then its something that needs to be addressed in the abstract class for sure

Yes, with non abstract class it is working. I just created the Abstract class to override Manager object and for the meta class.

Ok. So a fix needs to be done for abstract classes. Thank you for this

yes.

Ok. I will look into it. Thank you for this @harshalizode

@harshalizode found the issue. It was not in the abstract class. it was a missing validation. I had it in the past and somehow I had a regression. I will do a release soon.

@harshalizode my friend. Release 0.7.0 done now. Thank you so much for your inputs on this. Something clearly missing there and as a bonus. Just because you are such a fantastic contributor.
https://mongoz.tarsild.io/managers/
This is new and I hope you enjoy :)

Okay, thanks for the rapid resolution response.

Yes, the Abstract class is now functional; but while we may inherit the "CustomManager" at the model level, it cannot inherit the "CustomManager". Each model must additionally have its CustomManager initialized at the model level.

@harshalizode thank you very much for this. This is something we might want to look in the future and allow different ways, perhaps?
Oh, the PR you opened, did you manage to install the requirements and runs the scripts?

@harshalizode although I'm not 100% sure if I understood your side.
Check this one one. We are actually using the inheritance of QuerySetManager here as per new feature added.
https://github.com/tarsil/mongoz/blob/main/tests/test_managers.py

Okay, I'll look into this, and for PR, I'll install the requirements to run the script and get back to you.
Thank you.


Is this released model the same as the official one?

The checkpoint is converted from the original one and the pipeline implementation strictly follows the original implementation but using diffusers lib. From the inference results and the initial state of training, it looks very consistent with the original one. As the different lib (diffusers v.s original Pytorch lightning) may have different behaviors in some cases, I think they should give pretty much the same results, while I don't know how to strictly verify.

Thanks. I just tested the code on Objaravse. But the results are a little strange. I will debug about it.

I have addressed the problem, which is caused by the wrong usage of masking.

Great! Feel free to raise any issues if you find something is incorrect!


Prevent creation of data bags named node, role, client or environment

…ment.
Signed-off-by: Sandra Tiffin<EMAIL_ADDRESS>Description
Prevent creation of data bags named node, role, client or environment, using knife.
Issues Resolved
https://github.com/chef/chef/issues/3058
Check List

[x] New functionality includes tests
[x] All tests pass
[x] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

@sanditiffin is also adding this on the ChefFS paths.

Never mind on the ChefFS bits, we couldn't figure out how to make that work. Merge ready once Travis is less sad at life.

I will be submitting a PR for the ChefFS part, I've written the fix for that and have working unit tests. However, the two parts of the fix are independent, so this part can be merged once Travis completes. I also have a related documentation fix I will be submitting a PR for.

Thanks, @sanditiffin!

Thanks, @sanditiffin!


Panel is not supported

It would be cool if package in future support Field Panels

A workaround I am using is to get the fields contained in the Panel and set them as fields for NovaDependencyContainer and then set that NovaDependencyContainer field as child in the original Panel. On nova form this renders the titles of the Panel but not their encapsulated fields, but when you set the dependent value the panel containing the corresponding NovaDependencyContainer renders as it should and the data are passed successfully to the back end.
public function fields(Request $request)
    {
        return [
            new Panel('Panel title', [
                NovaDependencyContainer::make([
                    // original panel fields
                ])->dependsOn('category.id',$id)
            ]),
        ];
    }

I've been using the same sort of workaround to be honest, as dealing with panels delves way deep into nova core, as panels are first class objects so to speak there and have a ton of functionality associated with them.
Someone can for sure try and work up a PR if they would like, but the workaround is the way for the foreseeable future.


CRPS ensemble error ufunc?

Hi, I am currently using properscoring for my probabilistic forecasting project.
I encountered the following error:
File "C:...\lib\site-packages\properscoring_crps.py", line 347, in crps_ensemble
return _crps_ensemble_core(observations, forecasts, weights)
File "C:...\lib\site-packages\numba\np\ufunc\gufunc.py", line 123, in call
return self.ufunc(*args, **kwargs)
TypeError: ufunc '_crps_ensemble_gufunc' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
The odd thing is that I am working with multiple DataFrames containing time series data. I am ensembling over quantile values. All values have the same datatypes & strucutre and for most, crps computation works. But for one specific dataframe it doesn't, and I get the error above. Are you familiar with this error? Do you know if it could be caused by the values themselves?
The prediction gives useful results and the plots don't look suspicious at all...
Any help is much appreciated!

I'm just guessing here, (long pause) but, the function you get the error from is getting a numba decorator with static types:
https://github.com/properscoring/properscoring/blob/a465b5578d4b661e662933e84fa7673a70e75e94/properscoring/_gufuncs.py#L5
It could be that the data in the problematic dataframe are integers instead of floats (or some other typing madness) and this is why you get the error.


Document transaction configuration

Starting with Kafka 0.11 transaction support is already available and has been included in Spring Cloud Stream 1.3 as @artembilan kindly told me.
The scenario I want to control with a transaction is a database write along with a message publication, which should happen atomically, something like this:
/**
 * writes in DB and sends a message atomically
 */
@Transactional
@SendTo(Source.OUTPUT)
public void createUser(User user){
   userRepository.save(user);
   return new UserCreatedEvent(user.getId())
}

I've had a look at Spring Kafka 2.0.0 documentation where it is specified how to do something analogous to what I want.
However, going back to Spring Cloud Stream, it is not clear (it's undocumented) how to configure the applications to support transactionality and synchronization with a DataSourceTransactionManager.
I'd really appreciate if you could make this clearer.

Injecting some external transaction manager (such as a DataSourceTransactionManager) into the Kafka listener container is not currently supported.
You can achieve similar effects by enabling transactions in the binder and using @Transactional on the listener for the datasource transaction manager; the Kafka transaction will commit shortly after the DB.
Note that the producer properties have to be set on the transactional producer...
 +		/*
 +		 * IMPORTANT: With a transactional binder, individual producer properties for
 +		 * Kafka are ignored; the global binder
 +		 * (spring.cloud.stream.kafka.binder.transaction.producer.*) properties are used
 +		 * instead, for all producers. A binder is transactional when
 +		 * 'spring.cloud.stream.kafka.binder.transaction.transaction-id-prefix' has text.
 +		 */

I am not sure what happened to the doc updates; I distinctly remember writing them.


Sorting category upgrade + fix for issue #68

Sorting category upgrade
We did have an issue with WA API last on 11/06/2023
After talking with Gorkam, he fixed it so that it could work as before.
But during the talk, he explained me that he also added more information inside fall article retrieval.
We now have the order of the articles contained in each category without having to make additional calls to the API !
So I integrated those data into our import process.
Fix for issue #68
I don't know why, but it seems that people can have articles without titles => I've added a check when sorting the article list.

I've also added a little something so that only first level of category are opened.
It became difficult to look for what I needed when all category were displayed.
Here is what it has become on first load :

Tests were done on my custom world and the one created for handling all special cases, like covers or relationships.
I did found another problem during my test phase, but not linked to this PR.
article.cover.url and article.portrait.url does not have the same value as before.
Before :
https://www.worldanvil.com/uploads/images/0076dd51bd11903370cbd5672deffa1b.png  (just the image)

Now :
https://www.worldanvil.com/i/2664497

Since it's not an image, it fails to import it correctly.
I will track this in an other ticket.
But this PR has no impacts on this.


With numpy-1.10 (Debian testing) the tests run fine.
I have no idea whether this is an issue for numpy or for pyasdf; I will forward is to numpy as well.

Definitely a bug in numpy.

Would you then subscribe to numpy/numpy#7200 please, in case that a context is needed to discuss the bug?

@olebole: I'm subscribed to all bugs on the numpy tracker, that's how I saw this :-)

Since this is a numpy problem (and there it is solved, hopefully delivered with 1.11.0), I am closing it here now.


SqliteError 262 database table is locked

Hi!
I'm trying to use sqlx with an SqlitePool as a caching backend for an actix web application.
Several worker threads compute data, which can take a long time to produce and then store it into a table.
When this data is requested, the worker thread first queries the database before computing, to see if the result already has been computed, to save time. If the data isn't in the database, we recompute it and store it in the DB before sending it out.
The DB layout is basically just one table cache with key TEXT, data BLOB and I just use normal SELECT data from CACHE where key = ? prepared statements to retrieve and normal inserts to put the data into the DB.
I had hoped that the pool feature would just make my multithreading worries go away, but sadly I still got error 262 database table is locked.
Reading up a bit online I found documentation that usually enabling WAL should make the problem go away (it's already enabled) or that multithreading could cause it, which I thought unlikely, since that's sort of what a connection pool is for.
Anyhow, I wrapped the Pool in a Mutex and tried using that and the error still occured.
Any ideas what I am doing wrong? Do I need to use transactions for this?

I also tried adapting the SqliteConnection creation to
let pool = block_on(SqlitePool::builder().max_size(1).build("sqlite:cache.db")).unwrap();

but I still get the error.

Wrapping the pool in a Mutex effectively does very little as it has its own internal synchronization.
The error code directly suggests this is due to our use of SQLITE_OPEN_SHAREDCACHE which makes write transactions mutually exclusive between connections sharing the cache.
We should remove this flag.
In the mean time, you can prevent this error by setting the pool size to 1 when you build it. That effectively makes it operate as a mutex over a single connection internally.

fyi https://stackoverflow.com/questions/57123453/how-to-use-diesel-with-sqlite-connections-and-avoid-database-is-locked-type-of

So the error happens much less with Pool Size = 1, but it still comes up from time to time. The data size I write is around 1MB, so it could be that maybe some things are not flushed yet?
For what was suggested in the stackoverflow answer, I think sqlx is already using the pragmas
PRAGMA synchronous = NORMAL; PRAGMA journal_mode = WAL;, see https://github.com/launchbadge/sqlx/blob/master/sqlx-core/src/sqlite/connection.rs#L126 and https://github.com/launchbadge/sqlx/issues/296
Thanks a lot for your answers. This database layer is really making my life a lot easier at the moment and I'm optimistic the few remaining problems will vanish soon!

It's possible for SQLx to handle the locked notification like a mutex and unlock when we can. Hopefully we can improve on that soon.
@abonander is spot on though. We should have always been using private cache over a shared cache.

Still getting this error with poolsize 1 :/

One thing I've noticed to solve this is cursor should be fully evaluated even when querying a single row:
async fn last_uid() -> commons::AsyncResult<Option<String>> {
  let mut cursor = sqlx::query(r#"SELECT uid FROM trades ORDER BY time DESC, batch_idx LIMIT 1"#)
    .fetch(get_pool().await);
  let result: Option<String> = if let Some(row) = cursor.next().await? {
    Some(row.get::<String, _>(0))
  } else {
    None
  };
  // THIS IS MANDATORY otherwise getting "blocked" error with some of the calls
  while let Some(_) = cursor.next().await? {}
  Ok(result)
}

This should be fully fixed on master. SQLITE_LOCKED was caused by us naively using a shared cache. SQLITE_BUSY is caused for various reasons ( too many to list, see docs ) and we now enable SQLite's busy timeout ( which runs in a worker thread or blocking IO queue depending on the runtime ).

So I just tried the code from the master branch and apparently this still happens.
I initialize the database with
let pool = SqlitePool::builder().max_size(num_cpus::get() as u32).build("sqlite:cache.db").await.unwrap();
and then create a table and an index on that table.
I only use straight insert and select queries, no transactions (or.. do I need to use those? data consistency is not really that important, just using it as a cache and I have a method of checking for broken data externally) and I still get that code 517 database is locked error on pretty much every insert.
What could cause this?

error: error returned from database: database is locked

So I had the same issue but a different cause: Running VSCode with the rust-analyzer locks my sqlite database file for some reason. Disabling the plugin fixed the issue for me.


CORE-19839 Make public key encoding/decoding available to cordapp developers

API PR is here

I think it's worth adding 4 tests here https://github.com/corda/corda-runtime-os/blob/83d80d91bb1b952cc285e15cad208ba7f3f897d0/components/flow/flow-service/src/test/kotlin/net/corda/flow/application/crypto/SigningServiceImplTest.kt#L30-L29 just to make sure the correct functions are called on the key encoding service


DONT MERGE overall updates

updated R
updated Python
added drivers
probably shouldn't merge this till we decide if we want to swap out R/Py versions or append them

yikes! we also need a way to have the py version flow through to the .gcfg file ....

yikes! we also need a way to have the py version flow through to the .gcfg file ....

Yeah I have been debating this update, because anyone blindly using this image will immediately get broken content the next time they pull. I thought about coinciding it with an upgrade to the product version so that it would be a part of the upgrade process (but I keep forgetting to do that)... and I also punted on waiting until we have different runtimes available via the launcher 🤷♂  The upgrade story is a tricky one

Related: https://github.com/rstudio/rstudio-docker-products/issues/64

I think the more correct thing to do is just add a new version of R / Py

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Ubuntu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing since this is pretty old. We ended up adding R/Python versions, tracking them in NEWS.md, and trying to get them in sync across the different images. (Some old versions were replaced or removed)


How does Loowid work in multi party conf?

HI there,
I like what I see. I was trying to find out what kind of architecture is Loowid. I mean, Mess, Routing or MCU. I am looking at multi party conf. Would that mean, the sender has to send multiple streams in different directions and receive the same or Loowid would mix all the streams and then just send one stream in all directions.
Any possibilities to use an MCU like Jitsi VideoBridge? So far what I have understood that is peer to peer. I was worried about bandwidth of a peer, if he is using his cell phone to connect for a conf. If he has to send multiple streams and receive multiple streams then it will an awful experience for him. One to one is great. Please could you give insight on multi conf. Many thanks in advance.
Cheers

Hi, Loowid streams each media to every connected users. So if there is a room with 10 people and you start sharing your media you'll be sending the stream to the 9 others. Nowadays there is no intention to implement an MCU. We focus on smalls rooms for now but all help is welcome to go one step forward.

Thanks for your reply. I understand it better now.


fix image

only fix the image =)

lol, I knew deleting the master branch will have some consequences. Thanks!

You’re welcome. 😄


Leave plain JSX strings alone, including newlines

Fixes #405 by not processing plain JSX strings at all.

Another option would be to convert the \n to   in JSX.

Interestingly, this is what CoffeeScript does. (It even seems to convert multiple newlines into a single space.)  I guess it's fine with class, but I like better the approach of just leaving the string alone in this case.
One inconsistency I noticed is that \t and other escape characters are normally ignored (the backslash remains a backslash) in JSX strings.  But if you use an interpolated string (CoffeeScript's "#{foo}" or Civet's always-available """#{foo}"""), which turn into a braced template string, then the backslashes are processed.  This is consistent with CoffeeScript so I think it's OK.  Just a bit weird.

I think that's a bug in CoffeeScript and the backslashes should be escaped when put into template strings but we can adjust them as a separate PR.

Maybe? In JSX, ="\n" is unprocessed but ={"\n"} and ={`\n`} are processed.  So if you think of an interpolated string as an implicitly braced template string (which is what it compiles to), it makes sense to process the \ns.
If we really wanted consistency, I'd be tempted to process backslashes even in unbraced strings (as we implicitly brace a lot of expressions). But then it's less HTML/XML compatible, so I'm not sure it's a good idea.
FWIW I've never run into this in practice (never tried to put backslash sequences in a JSX string), so hopefully not a big deal either way. (But I did see someone in the SolidJS Discord using newlines in a class field recently, because it's a nice way to list a lot of classes.)

Those are some good points. We can wait until real use cases come up before deciding.


fix: tables update parents request parsing is broken

Description
Risk

Deploy Plan

Lgtm. I’ll double check the other parsing changes in the original PR just in case the same mistake has been done twice.


Switch webhook liveness/readiness probes to use http ports

Changes
Switch webhook liveness/readiness probes to http ports and use a separate port from actual webhook HTTPS ports.
This fixes misleading errors mentioned in #3303
Webhook logs:
kubectl -n tekton-pipelines logs $(kubectl -n tekton-pipelines get pods -l app=tekton-pipelines-webhook -o name)
2020/10/06 23:57:31 maxprocs: Leaving GOMAXPROCS=2: CPU quota undefined
2020/10/06 23:57:31 Server listening on port 8080

This solves the issue because it doesn't send simple TCP request to a HTTPS port, which will cause TLS handshake to fail.
Other considerations will be 1. continue using the same HTTPS port but establish HTTPS handshake. This is not done because using the same port as actual webhook port for probes is not ideal for traffic and responsibility separation.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices
[ ] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/retest

Sorry I was distracted by something else for a bit and now was able to come back to this. @afrittoli I think @ImJasonH is still not convinced to add the probes in controller and I also think it will be reasonable to follow the comment you added in #3111. I will keep track of the health checks they've added to knative package and take a look at that to make probes available in controller.

@mattmoor @pmorie I found knative/pkg#1048 but it's not clear to me then whether it is available yet or not. If not we could perhaps add a "shallow" /healthz for now, that always reports "ok" (as @ImJasonH suggested) and switch to the knative/pkg one once it becomes available.

/lgtm
@sbwsg do you have any other comments?

Thanks @ywluogg !
/approve

@ywluogg quick question, would be ok to have this same implementation for the missing probes in https://github.com/tektoncd/pipeline/blob/master/config/controller.yaml ?

@ywluogg quick question, would be ok to have this same implementation for the missing probes in https://github.com/tektoncd/pipeline/blob/master/config/controller.yaml ?

The issue was addressed in #3111. I will add the probes in controller once the health checks mentioned in #3111 is available.

The approach in knative for the webhook is actually to use the existing HTTPs endpoint like used to do, but with HTTPs instead of TCP, and passing a special k-kubelet-probe header:
https://github.com/knative/serving/blob/1d259fb640b5ba9748a430a0cfa90d451d5826d2/config/core/deployments/webhook.yaml#L95-L102
I guess we could do the same for Tekton and avoid the need for the extra server?
For the controller there is no solution yet, so one way forward might be to have a dedicated small HTTP server until we have a solution from knative/pkg.

You don't need the k-kubelet-probe header, this is sort of a vestige of a time when Istio incorrectly rewrote K8s probes to drop the header it sends.  Maybe it's a special User-Agent?  I think we have an IsKubeletProbe function somewhere that checks for these things.

The approach in knative for the webhook is actually to use the existing HTTPs endpoint like used to do, but with HTTPs instead of TCP, and passing a special k-kubelet-probe header:
https://github.com/knative/serving/blob/1d259fb640b5ba9748a430a0cfa90d451d5826d2/config/core/deployments/webhook.yaml#L95-L102
I guess we could do the same for Tekton and avoid the need for the extra server, but it would require doing TLS handshakes, so perhaps its good to stick with this solution :)
For the controller there is no solution yet, so one way forward might be to have a dedicated small HTTP server until we have a solution from knative/pkg. knative/pkg#1048 is back open

Pinging @ImJasonH again for any suggestion for adding a temporary http server in controller. I think currently we may still need for info from other users about why they need this probe in the controller.

The probe needed is mostly the liveness one.
When running multiple replicas through leader election, the probe helps detecting a faulty controller and releasing its bucket of resources to other controllers.

It sounds necessary to add at least the liveness probe in controller based on this situation. I don't think there is another simpler way to check on liveness under this situation. But I'm waiting for @ImJasonH con re-assurance.

I agree, it sounds like HA controller will benefit from having a liveness probe, which we can implement now with a simple dummy HTTP server, and later improve with real health checks.

Sounds good will add it as soon as possible.

@ywluogg Go through above discussions, are you only enable liveness probe for controller, now? Could you pls also add readiness probe for controller? It's necessary for me. Thanks in advance!

@xiujuan95 do you mind also provide some details about the situation you plan to use it? Thanks

@ywluogg Sure, I can. Actually, It's a requirement from my manager team. It requires all pods need to enable liveness and readiness check. Besides, I think readiness can tell us if all containers within this pod are ready or not. It also makes sense! So pls also help add it! Thanks in advance!

Sounds good I will add them too.

@ywluogg Thanks a lot!


[WIP] Add tests for structural ops

@benvanik I realized we need to do a bit of mangling on function names in the CModuleTarget. In the process I also managed to produce name clashes in the bytecode module test. Should this be fixed or is the test to contrived?
CC @marbre

IIUC it's just testing the test infra but that's fine :)

Yep, it really should test that valid C function names are generated, which needs to be implemented. And the test shows that a simple reaplceInvalidCharBy_ is sub optimal. As a byproduct it breaks the test infra.

Closing as obsolete.


Remove redundant String operation from GDScript enum exports

capitalize() already calls camelcase_to_underscore(true) first thing, so that call here is redundant. Not sure if it's a major improvement or not, but should save a bit of time 🙃

Thanks!


Nuxt-propert-decorator/more-pages

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Needs QA check

@kodadot/qa-guild please review

Context

[x] Related with #4750

Pages in this PR

home page
snek/assets
snek/claim
snek/incomingOffers
snek/offers
snek/stats
stmn/waifu

desired outcome
no change in look and/or behavior
Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[ ] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at e768e96
This pull request streamlines the user interface and code structure of the snek NFT minting and trading features. It moves the claim, generative mint, asset, offer, and stats pages to the home page and simplifies the component syntax. It also renames some components and files to avoid confusion.

🤖 Generated by Copilot at e768e96

We've moved the pages to the index, me hearties
We've simplified the syntax of the Component
We've streamlined the snek NFT experience
So heave away and pull the code with confidence

index.vue - ok
assets page: ok (unrelated: assets taking a long time to fetch balances today)
claim page: we don't really use it atm - ok
incoming offers: ok
offers: ok
stats: ok
waifu: ok


Add Amount and SignedAmount

Supersedes https://github.com/rust-bitcoin/rust-bitcoin/pull/252.
After long discussion about whether Amount should be u64 or i64 internally and after switching between them several times, we agreed on having two types.

Looks great now, except that the fuzz tests don't compile :)

Fixed in force-push of the last commit.


Japanese charactors not rendering correctly

Environment
Windows build number: [10.0.19041.388]
Your Distribution version: [ Ubuntu 20.04]
Whether the issue is on WSL 2 and/or WSL 1: [WSL1 and 2]

Steps to reproduce

Doing anything to have Japanese (or likely any non ascii characters) display on the terminal .
In my case I am using Project64 and attaching a debugging tool through telnet to see Console output from debug roms.

To verify that, you can install 'Windows Terminal' from Windows Store, and try it again from Windows Terminal. 'Windows Terminal' has better localization support.
If 'Windows Terminal' works fine, you may try:

set Windows display language to Japanese
change system locale to Japanese in Region Settings

Doing anything to have Japanese (or likely any non ascii characters) display on the terminal .

You'll have to be a little more specific than "anything".

First thing you can try is start wsl.exe from the new Windows terminal. That might or might not help your particular scenario out of the box but it is a good place to start. Sometimes the output needs to be piped though iconv(1) or similar depending on the encoding.
If none of that helps, open an issue over in the terminal github. WSL doesn't actually render the glyphs (Japanese or otherwise). They'll want a repro, and "....using Project64 and attaching a debugging tool..." probably isn't going to get a lot of uptake. Redirecting some of the output to a file and posting a gist may shed some light under examination. Something that can be reproduced locally would be better.


Running make run fails when assets are involved

Need to make sure this exists.
ruby es2js.rb
Traceback (most recent call last):
	1: from es2js.rb:13:in `<main>'
es2js.rb:13:in `write': No such file or directory @ rb_sysopen - public/javascripts/application.js (Errno::ENOENT)
make: *** [assets] Error 1

Hello, I've just ran into this bug.
As a solution, just adding statement to make empty application.js isn't enough?
or application.js needs any code?

I think just having an empty application.js file may be enough. I haven't looked in to it much, but I think that's all that's needed.

#25 How about this? It's too simple, but works fine.

Fixed by #25


fix import-module / function call for UpdateVMSS/ UpdateVmss on cloud shell / linux

from https://shell.azure.com / cloud shell, unable to run AzLoadBalancerMigration completely.
There is an error calling 'UpdateVMSS'
Function name is 'UpdateVmss'
Actual fix is on the import-module file name case mismatch:
-Import-Module ((Split-Path $PSScriptRoot -Parent) + "\UpdateVMSS\UpdateVMSS.psd1")
+Import-Module ((Split-Path $PSScriptRoot -Parent) + "\UpdateVMSS\UpdateVMSS.psd1")
Error from log:
UpdateVMSS: The term 'UpdateVMSS' is not recognized as a name of a cmdlet, function, script file, or executable program.
Verified fix resolves issue

I'd like to make casing consistent across the board as part of #25


fix: support shared components

Renamed _templates to _hygencook. New templates will be copied over to _hygencook/${packageName}/_templates/${generatorName}.

:tada: This PR is included in version 1.4.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Target text file

Should be able to specify a simple TODO text file as provider. Using a well known format (SublimeText's PlainTasks? TodoTxt?) would be helpful.

Done for todo.txt, PlainTasks will be another service if implemented


feat: Add googleapis-auth as codeowner team

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> ☕
If you write sample code, please follow the samples format.

BTW, please use chore: for these kinds of changes so that it works well with release-please.
See: https://www.conventionalcommits.org/en/v1.0.0/.


[Question] Session Id in Flask

How do I integrate an external id using sessions in the flask?
Exemple:
@app.route("/msg/string:msg&id=string:id")
def get_raw_response():
return str(portuguese_bot.get_response(query))
if name == "main":
app.run()

I am not familiar with flask, I knew there is a repository which implemented on flask https://github.com/chamkank/flask-chatterbot this might help you

Thanks, but I need to know how to implement the system of individual sessions using flask
.Bad english :/

A specific conversation session can be selected by passing a unique id into the get_response method.
For example, `get_response('My input string', conversation_id=flask_session_id)
I'm not familiar with how to get a session id in Flask, but I'm sure there are resources online that can explain how to get it.


Add OPD Schedule Page

#400

@AdityaBavadekar  share relevant screenshots and explain what changes you made

I am currently developing the page table. I will share screenshots once done (today itself).

https://github.com/user-attachments/assets/e97c3833-43fc-4301-8caa-6cc564f0f00a

I have added a drop down to allow user to select the department and the corresponding department schedule is shown in the table. For this I have added a JSON object with predefined schedule for each department.
I am currently working on adding disclaimer that the timings may change without notice.

@AdityaBavadekar I expected some work on the back-end, like we are showing the schedule for treatment, but no hospital is specified, I wanted this schedule to be shown for each hospital, in the profile page the hospital should be able to upload the schedule details. You can skip this part but then your PR won't be of much use for the project, so I can only give level1.

Ok, got it. So you want this to be listed in the hospital info page and the doctors listed should also be dynamically added based on each hospital. I am doing it but it will take some time.

@AdityaBavadekar yes the schedules should be in a new page which opens up when clicked on a hospital in the hospitals page and in the hospital's profile page there should be the option to add the schedules.

@AdityaBavadekar in this form only, you can take the availability time slots of the doctor in any format, and show it in the schedule

Progress:

looks cool @AdityaBavadekar

https://drive.google.com/file/d/1cRbkZvd0SfRtz13EbDZ_5xikf1wswI6P/view?usp=sharing
@Luson045 You can take a look at this screen recording I made. Since it is very big I had to upload it to google drive. It shows the navigation you told.

I have also updated the database url usages. While testing locally, I had change every database url usage to change to my localhost url since this was hard coded in all files. I have changed it now so that there is only 1 file databaseUrls.js which has the base url internally and only exposes a object which is easier to use for eg: databaseUrl.auth.profile this maps to complete url in the base file and not in file where it used.

@AdityaBavadekar great job 👍

Just wanted to tell that there are some hospitals pre defined statically in json object in project, they have ids like 1,2 ... the Hospital info page will not work for those because it fetches the data from mongo db database where as these are not in database.
You would have to add those hospitals into db, for it to work.


Adding support for ResolverRule

Issue #, if available:
Description of changes:
Adding support for adding/deleting/updating Route53 ResolverRule in route53resolver-controller .
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/ok-to-test

@a-hilaly e2e tests are now passing as well. Kindly have a look.
Thanks

/test all


pythonPackages.importlib-metadata: 1.3.0 -> 1.5.0

Motivation for this change
Update importlib-metadata to latest (1.5.0).
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

I hit cancel mid-request, but didn't catch it, sorry -- this should be a draft, this is a WIP...
It currently does not work, this introduces infinite recursion when building this package.
error: while evaluating the attribute 'drvPath' at /home/cole/code/nixpkgs-importlibmetadata/lib/customisation.nix:158:7:
while evaluating the attribute 'nativeBuildInputs' of the derivation 'python2.7-importlib-metadata-1.5.0' at /home/cole/code/nixpkgs-importlibmetadata/pkgs/development/interpreters/python/mk-python-derivation.nix:105:3:
while evaluating the attribute 'out.outPath' at /home/cole/code/nixpkgs-importlibmetadata/lib/customisation.nix:151:13:
while evaluating the attribute 'nativeBuildInputs' of the derivation 'python2.7-pyfakefs-3.7.1' at /home/cole/code/nixpkgs-importlibmetadata/pkgs/development/interpreters/python/mk-python-derivation.nix:105:3:
while evaluating the attribute 'out.outPath' at /home/cole/code/nixpkgs-importlibmetadata/lib/customisation.nix:151:13:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'python2.7-pytest-4.6.8' at /home/cole/code/nixpkgs-importlibmetadata/pkgs/development/interpreters/python/mk-python-derivation.nix:105:3:
while evaluating the attribute 'out.outPath' at /home/cole/code/nixpkgs-importlibmetadata/lib/customisation.nix:151:13:
while evaluating the attribute 'propagatedBuildInputs' of the derivation 'python2.7-pluggy-0.13.1' at /home/cole/code/nixpkgs-importlibmetadata/pkgs/development/interpreters/python/mk-python-derivation.nix:105:3:
while evaluating the attribute 'out.outPath' at /home/cole/code/nixpkgs-importlibmetadata/lib/customisation.nix:151:13:
infinite recursion encountered, at undefined position

Note: python37Packages.importlib-metadata builds, but not 27,35,36 or 38 (I think 38 is expected, since this package is obsoleted by a builtin library in python3.8).

I couldn't quite figure out how to skip the test_main.py, which we might not need anyway, so I just deleted it to at least show it works (and unblock another branch of mine for now).

this rebuilds the majority of the python package set, please target staging

Hi @jonringer. Thanks for looking. As for your suggestion - that's what I did at first, and it yielded the infinite recursion error that I posted. If you look at my two commits, you'll see one added the pyfakefs checkDep and then my workaround removed it again since it caused the inf recur error.
Obviously deleting the test is not ideal, but I don't know how to proceed with your recommendation without hitting the recursion error.

I think deleting the test, and just noting that it's done to avoid infinite recursion is fine

I was able to build (on nixos-unstable) for: python{27,35,36,37}Packages.importlib-metadata.

@GrahamcOfBorg build pythonPackages.importlib-metadata
@GrahamcOfBorg build python3Packages.importlib-metadata
@GrahamcOfBorg build python38Packages.importlib-metadata

darwin builds are timing out due to trying to rebuild llvm


Upgrade to Vite 5

Closes https://github.com/atlowChemi/vite-svg-2-webfont/issues/13.
We also need to update Vitest to the latest version (1.2.2), otherwise we will get the following error when running the test.
 FAIL  src/optionParser.test.ts > optionParser > parseOptions > sets descent only if defined in options
Error: The specified file globs did not resolve any files in the context.
 ❯ parseFiles src/optionParser.ts:150:15
    148|     const resolvedFiles = files.flatMap(fileGlob => sync(fileGlob, { cwd: context })).map(file => `${context}/${file}`);
    149|     if (!resolvedFiles.length) {
    150|         throw new NoIconsAvailableError('The specified file globs did not resolve any files in the context.');
       |               ^
    151|     }
    152|     return resolvedFiles;

/cc @atlowChemi for review

Overall LGTM, thanks for putting in the time & effort! 🙂

Hi @atlowChemi, if this PR looks good to you, maybe we can consider merging it and publish a new version? If there are any specific changes or adjustments you would like me to make, please feel free to let me know. I am more than willing to address any feedback you might have.
Thanks :smiley: !

@atlowChemi Friendly ping

@atlowChemi Friendly ping

Hey @Juneezee sorry for the delay, I have been out for a few days 🙏🏽


AGI graphs for the connectors log

We have AGI missing for the connector logs, it is very tough to go through log metrics without a graph. Please add them to AGI.

Will be completed by mid next week.

Done, available in 7.6.0 prerelease


Shift the - on // MARK comments from the end to the beginning of the comment

For example, when using alamofire extensions, the source code generates the string

// MARK: Alamofire response handlers -

Placing the dash on the end of the line results in the new line below the comment Like this
i think // MARK: - Alamofire response handlers. with the - at the beginning would make more sense so when its viewed in xcode, it looks like this

Would you care to submit the PR for this? Just change this one line:
https://github.com/quicktype/quicktype/blob/master/src/Language/Swift.ts#L956
You can even send the PR right from the GitHub web UI :)

Will do


Make schema creation configurable

This PR makes these parameters in schema creation configurable:

keyspace name
replication factor
cqlsh executable

Coverage increased (+0.2%) to 54.773% when pulling 6366ed24a12122d360e955a212427f022444df34 on schema into 8bbdbd7a691b8748520c040bb6bce404494f01bd on master.

Coverage decreased (-0.3%) to 54.685% when pulling 6b11e337f09c55356c515eb137d0653b56390702 on schema into 9bf39e19aa983776d5dc13e0d254ecd237a58c22 on master.


Updating IOUtils.toCanonicalFilePath to work in a spring boot application

This commit makes it so that es-hadoop does not blow up when it is packaged within a Spring Boot application.
Closes #1359

I cloned the project at https://github.com/maxmin93/agens-spark to test this. I realized that the problem is more than just that it's a jar with in a jar -- it is also complicated by the fact that Spring Boot provides its own custom URLStreamHandler. That URLStreamHandler results in the URLConnection returning a different jar file URL than the default one. I added a unit test to simulate what the spring boot one does.


update guest kernel to 4.9.23

Also enable pvspinlock and transparent-hugepage.
The extra configs(CONFIG_RADIX_TREE_MULTIORDER, CONFIG_BALLOON_COMPACTION and CONFIG_COMPACTION) are all selected by THP.

retest this please, @hykins


Fix ordering of parameter before URL

Hi, I ran into
jupyter-repo2docker https://github.com/.../ --ref 16c6b...f1a9f still using master and ignoring my --ref.
@betatim suggested https://gitter.im/jupyterhub/binder?at=5bc05a19c7bf7c3662f9e0bf to
have you tried changing the order of the arguments repo2docker --ref 16c...1a9f https://github.com/... we've had issues with the arguments after the URL being interpreted as the command to run in the container

which indeed fixed the issue. I am here updating the usage.rst document, where I got the ordering from. Not sure if that is the only documentation showing this order, and also not fixing the issue mentioned by Tim. Yours, Steffen

Thanks for the contribution! Sorry for the outdated docs that lead you into the wild.


[SPARK-41987][CONNECT][PYTHON] Connect API: createDataFrame should supports column with map type

What changes were proposed in this pull request?
Currently, Connect API createDataFrame does not support create dataframe with map type.
For example,
    >>> df = spark.createDataFrame(
    ...     [(1, ["foo", "bar"], {"x": 1.0}), (2, [], {}), (3, None, None)],
    ...     ("id", "an_array", "a_map")
    ... )

The above code want create a dataframe with column 'a_map' which is map type.
But pyarrow recognize {"x": 1.0} as a struct not map.
pyarrow supports map with format [('x', 1.0)]
Because the schema of dataframe is not correct, so the other subsequent operator will be impacted.
For example:
df.select("id", "a_map", posexplode_outer("an_array")).show()
Expected:
    +---+----------+----+----+
    | id|     a_map| pos| col|
    +---+----------+----+----+
    |  1|{x -> 1.0}|   0| foo|
    |  1|{x -> 1.0}|   1| bar|
    |  2|        {}|null|null|
    |  3|      null|null|null|
    +---+----------+----+----+

But got:
    +---+------+----+----+
    | id| a_map| pos| col|
    +---+------+----+----+
    |  1| {1.0}|   0| foo|
    |  1| {1.0}|   1| bar|
    |  2|{null}|null|null|
    |  3|  null|null|null|
    +---+------+----+----+
    <BLANKLINE>

Why are the changes needed?
Let createDataFrame supports column with map type and fix some bugs.
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
Manual test and doc test.

cc @zhengruifeng

@beliefer Thanks for working on this, but I prefer to supporting all the complex types together in one batch. Would you mind taking a look at it?

@beliefer Thanks for working on this, but I prefer to supporting all the complex types together in one batch. Would you mind taking a look at it?

OK. Let your PR go on.

Thank you @beliefer


[LazyFilterSequence] Preserve order of predicates

The current behaviour leads to unexpected order of execution (reverse) and given a cheaper filter being applied before more expensive ones, an increase in performance cost for end users.
This fixes it by preserving the input order of the predicates

Resolves SR-11841.

Don't know if there is a benchmark that can expose this issue yet, here is one just in case:
import Foundation
var array = Array(repeating: "1", count: 10000)

func benchmark(name: String, _ fn: () -> Void) {
    let start = CFAbsoluteTimeGetCurrent()
    fn()
    let end = CFAbsoluteTimeGetCurrent()
    print(name + " took: \(end-start)")
}

benchmark(name: "normal") {
    _ = array
        .filter({ _ in
            return false
        }).filter { _ in
	    Thread.sleep(forTimeInterval: 0.0001)
            return true
    }
}

benchmark(name: "lazy") {
    let seq = array.lazy
        .filter({ _ in
            return false
        }).filter { _ in
	    Thread.sleep(forTimeInterval: 0.0001)
            return true
    }
    _ = Array(seq)
}

benchmark(name: "lazy reverse order") {
    let seq = array.lazy
        .filter({ _ in
	    Thread.sleep(forTimeInterval: 0.0001)
            return true
        }).filter { _ in
            return false
    }
    _ = Array(seq)
}

Results under Swift 5.1.2
> swift test.swift
normal took: 0.0032050609588623047
lazy took: 1.3336870670318604
lazy reverse order took: 0.004425048828125

Good catch!
Could you please add a regression test somewhere in test/stdlib or the validation tests to make sure this doesn’t happen again?

I wish there was a way to add a test to the stdlib without having to checkout and compile the whole language.
Is there?

You may be able to follow the instructions here to build only the stdlib using a downloaded tool chain: https://swift.org/quickstart-contribution/#build-swift-stdlib-against-a-downloadable-toolchain

A little update:
I'm blocked as I can't get the stdlib to build against the toolchain I downloaded. I don't know when (or if) I'll be able to get it running in my environment in a reasonable time.
This is the diff of the regression test
diff --git a/test/stdlib/Filter.swift b/test/stdlib/Filter.swift
index acadd48a53..fe1266d0cc 100644
--- a/test/stdlib/Filter.swift
+++ b/test/stdlib/Filter.swift
@@ -73,4 +73,26 @@ FilterTests.test("single-count") {
   expectEqual(30, count)
 }

+FilterTests.test("filter order") {
+    let array = [1]
+
+    let lazyFilter = array.lazy
+        .filter({ _ in
+            return false
+        }).filter { _ in
+            //assert failure
+            return true
+        }
+    _ = Array(lazyFilter)
+
+    _ = array
+        .filter({ _ in
+            return false
+        })
+        .filter { _ in
+            //assert failure
+            return true
+        }
+}
+
 runAllTests()

If someone (who can verify the test compiles and fails as it should in the current master) can get it in a different PR, would be great so we can get this fix in.
Thanks

@swift-ci Please smoke test

Thanks, @dtorres — I've added your test in a separate commit. Merges are locked until Dec 2 (and we might need to discuss the change in behavior), so we'll need to wait a bit even once tests pass.

@swift-ci Please smoke test
Helps if I call the right method! 🤦♂

@swift-ci please smoke test

@swift-ci please smoke test

Awesome.  Thanks!
⛵


Fix System32 module path duplication

PR Summary
Resolves #7327. PowerShell Core subprocesses will only append the System32 module path to the PSModulePath when it is not already present.

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[x] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[x] Add [feature] if the change is significant or affects feature tests

Restarted travis-ci test run.


Issue with Numpy and Python>=11

This version of numpy seems to be failing for newer Python versions. Our CI tests fail at importing this numpy version. Is there a reason to have an approx limitation to this or will it be fine to let this free? We test on Python3.9-3.12 usually to see all errors. We could add those to the CI here as well.
https://github.com/FAIRmat-NFDI/readers-xrd/blob/23a0eb06c2d7062be55174bacb8daa54d8a8e56c/pyproject.toml#L28C5-L28C20
The failing test: https://github.com/FAIRmat-NFDI/pynxtools-xrd/actions/runs/9399588169/job/25887327552
@lukaspie @RubelMozumder adding you guys for reference.

We are resolving a similar issue for another package. What we are trying now is having a different set of pint and numpy for py3.9 and below. For everything else we use the latest pint and numpy. For py>=3.10 we can also not limit the version as this package more or less should work with the basic features of the two libs.