Accessory Not Supported.

Hoi Erik,
Installed the homebridge-hue plugin, ran it for a first time, pressed the button on my bridge, entered the info from he logging into the config.json file, started homebridge again and.. nothing :)
Well, what happens is that in the Home app (and also in Eve), there is a new 'Philips hue' accessory, but the Home app says it's not supported.   See screenshot below.   Checked with Eve, see the same accessory but cannot do anything with it, not even change it to another room.   config.json & logging output also included below.
I must say that I have the Hue bridge itself connected directly to the Homekit already, only reason for me to include the plugin was for the Dimmer and Tap switch, as I can then include more actions than only hue updating hue lamps when these are used.
As I kept all additional parameters out of the config.json file, my expectation was that only these switches (and maybe other switches/sensors, except for the light bulbs themselves) would appear.   My idea was to have those switches do nothing (or as little as technically possible) within the Hue environment itself but take advantage of the exposure of the switches/sensors to HomeKit to either use their characteristics as Triggers or Conditions.
Your thoughts & feedback are highly appreciated.
Met vriendelijke groet vanuit California.
Jurgen
config.json
{
"platform": "Hue",
"name": "Hue",
"users":    {
"0XXXXXXXXX6": "AXXXXXXXXXXXXXXB"
}
}
Logging:
[1/1/2017, 11:50:14 PM] [Hue] Philips hue: hue bridge request: get /
[1/1/2017, 11:50:19 PM] [Hue] Philips hue: hue bridge request: get /
[1/1/2017, 11:50:24 PM] [Hue] Philips hue: hue bridge request: get /
[1/1/2017, 11:50:29 PM] [Hue] Philips hue: hue bridge request: get /
[1/1/2017, 11:50:34 PM] [Hue] Philips hue: hue bridge request: get /
Etc.. goes on like that

Hallo Jurgen @farstreet ,
You need to add "sensors": true to config.json to make homebridge-hue expose the sensors to HomeKit, see the README.
Currently, the only thing you can do with the bridge accessory is dump the bridge state to file by selecting Identify on it.  I plan to use this accessory to store the bridge username in future, so you would no longer need to add it manually to config.json.  You should be able to assign it to a room, however, both in the iOS Home app as well as in Eve: click on Default Room in the screen you posted.
Depending on the number of lights you have, you might want to have homebridge-hue expose groups and use these in the HomeKit scenes and triggers instead of the individual lights (exposed by the v2 bridge).  That could vastly reduce the amount of network traffic between your iOS device and (homebridge-hue and between homebridge-hue and) the Hue bridge and it would make sure all lights react simultaneously.  This is more of a concern for a v1 (round) bridge, however.
Also you might want to specify a lower value for heartrate in config.json to have homebridge-hue react quicker to your switches.  I'm using 2 seconds myself, but I've heard others using 1 second successfully.
Personally, I like to programme all interaction between Hue bridge resources in the bridge itself.  That way, my dimmer switches and motion sensors continue to work with my lights, even when my home network is out.  The official Hue app does a lousy job at programming this interaction imho, so I've created my own Hue bridge rules.  I only use HomeKit triggers to let the motion sensors and dimmer switches interact with my other accessories: my Sonos speakers and Eve Energy plugs.
Also note that homebridge-hue can only expose the button release events of the dimmer switches, because it needs to poll the Hue bridge and cannot detect the button press and button hold events reliably.  This is good enough for on/off control, but dimming lights is best done using Hue bridge rules.

I'm experiencing this same issue. I have included "sensors": true in my config.json and I'm getting the same thing of This accessory is not currently supported by the Home app.

Having the same problem as @bspradling8, same config. I can not reach my Lightify Bulbs or access the sensor on the bridge. Debug shows (with one bulb and one bridge):
[Hue] Philips hue: BSB002 bridge, api v1.16.0
[Hue] Philips hue: hue bridge request: get /
[Hue] Philips hue: 0 lights
[Hue] Philips hue: 0 groups
[Hue] Philips hue: Daylight: Daylight sensor
[Hue] Philips hue: 1 sensors
[Hue] Philips hue: 0 schedules
[Hue] Philips hue: 0 rules
[Hue] Philips hue: 2 accessories
[Hue] Initializing platform accessory 'Philips hue'...
[Hue] Initializing platform accessory 'Daylight'...

@bspradling8 You need to set "lights": true as well if you want to expose lights.  The "philipslights": true only works when lights is enabled.  The iOS 10 Home app doesn't support Hue Dimmers, Hue Taps, the Daylight sensor nor CLIPGenericStatus sensors.  These are mapped to a Stateful Programmable Switch service, which is HomeKit standard, but not (yet?) supported by Home.  They are supported by other HomeKit apps, see the README.
@Subito it looks like homebridge-hue connects to your bridge alright (it finds the Daylight sensor).  Please double-check your config.json (for "lights": true).  The Osram Lightly Bulb should work.  If not, could you please post the output of homebridge -D and a dump of your bridge state (see README)?

Thank you @ebaauw! Appreciate the work and the quick response. It is working great.

Thanks! Works fine. I misread the README, apparently. Got confused with lights, philipslights and alllights. Its clear now. Thanks!

Hello I have the same problem with the " accessory not supported" on my rgb strip. Anybody have the same?
Thks


Remove repeated word that doesn't make sense.

@rnwood Thank you for fixing this, and congratulations on your first docs contribution.


On rescans load existing GraphSidecar json instead of downloading again

Provide us a use case of the feature
For every GraphSidecar node that has videos in it, a graph query is executed. Especially on rescans that is a waste of api calls.
Describe the solution you'd like
Instead of downloading that json again, just load it from disk if it has been downloaded and saved before.
Describe alternatives you've considered
None found.
If the feature request is accepted, would you be willing to submit a pull request?
Yes.

Additional context

Especially on rescans that is a waste of api calls.

We fixed this rescan inefficiency with 672b0b184019bf840e3003e90a1c662fcb4df79a in upcoming/v4.7. Now, that graph query is only executed if one of the files that should be downloaded is missing.

The current upcoming/v4.7 (640066d) is runnable (and contains latest master too), right?
Sorry, I don't know how to use the upcoming version. Please help.
On a re-run it does a query for every sidecar node.

As for every post the json is saved anyway (default parameters), so why not reload that one, if needed in case of a sidecar node?
For a re-run of 62 posts (12+50) I need two query/api calls, regardless of how many sidecar nodes therein. And it works independent from the chosen filename template. Imho that's effiecency.

This feature suggestion seems a great idea to me.
Instaloader goes through the posts online, determines the filenames that would be created for each post, and skips the post if it already exists in the target's folder locally. For sidecars in a profile that are not at the first page, e.g. posts with index > 12, the sidecar details are not included in the page overview returned by Instagram, and so Instaloader currently needs to do an extra graphql query to obtain the information that is needed to determine which filenames would be created, and as such, to determine whether something has to be downloaded. (my previous comment was wrong)
The information in the 'old' JSON files from a previous run should be sufficient to determine the filenames yield by the Post in question. So your suggestion is a nice idea to save that query and so speed up rescans of profiles (and most probably other targets as well) that have some sidecars.
I think that the number of media in a sidecar post cannot change, i.e. that it is not possible to add or remove photos/videos after posting the sidecar. This is a prerequisite for this feature to work properly.
To not introduce side-effects, the information from the 'old' JSON file probably should only be used to determine whether the Post needs to be re-downloaded or not.

The current upcoming/v4.7 (640066d) is runnable (and contains latest master too), right?
Sorry, I don't know how to use the upcoming version. Please help.

Yes, upcoming/v4.7 should work and includes the latest master too. With pip, it can be installed with
pip3 install --upgrade<EMAIL_ADDRESS>

For a re-run of 62 posts (12+50) I need two query/api calls, regardless of how many sidecar nodes therein. And it works independent from the chosen filename template. Imho that's effiecency.

Awesome :)


(Very rough draft) of Google Classroom porting over example

Purpose/implementation Section
What changes are being implemented in this Pull Request?
Here's a very rough example of how you can turn an OTTR course into a Google Classroom.
You would do this because:

It allows you to keep track of what your students have completed -- something Leanpub and Coursera aren't really built for since they are meant for self-learners
It's familiar to people -- public schools often use it
It's free
Allows non-GitHub fellow instructors to help you track the students.

If you want to see what this looks like from a student's perspective, go here: https://classroom.google.com/c/NjEyNjU2ODI2MDEz?cjc=gljsqgf
These scripts need a lot more work and there needs to be more documentation overall, but I just wanted to post this as a quick example.
How these scripts would work

You would get yourself a Google Classroom account https://edu.google.com/workspace-for-education/classroom/
You would run materials_df.R script to obtain a TSV file that shows the material that would be ported over to Google Classroom. You may want to add due dates to it manually so those are also added to the Google Classroom.
Then interactively, when you want a new classroom to be started, you would run the port_to_google_classroom.R script.

Performing housecleaning here...is this PR still active?

Performing housecleaning here...is this PR still active?

It's still relevant but not active.

I'm probably not going to get back to this feature. Will close.


ci: implementation proposal for func e2e test

New proposed Implementation for E2E test based on func authors feedback.
It is primarily to be tested against 'func'  binary as a standalone cli, or installed as KN plugin. It attempts to simulate common user experience
It reuses existing cluster approach adopted for integration (with kind)
What is covered:

Http functions: create, deploy, list, describe, validate exposed url, update (replace functions by a new src) and delete
CloudEvents functions: create, deloy, list, describe, push event and delete
Emit command specific test

Scenario can be executed selectively for all runtimes (by default runs for node only).
This new PR was created once code diverges a lot from original Draft PR, and makes it easy to review

@jrangelramos can you try and figure out why the CI is failing here?

@lance yes, I already figure out. Going to push a new commit soon


registerLoader throws error

System Info
all
Details
When working with Angular and trying to use the MiniCssExtractPlugin we found that registerLoader throws an error.
webpack.util.serialization.registerLoader(/^mini-css-extract-plugin\//, trueFn); throws error TypeError: Cannot read properties of undefined (reading 'registerLoader') - MiniCssExtractPlugin
Reproduce link
No response
Reproduce Steps
enable the MiniCssExtractPlugin

Still happens.
The docs say we don't need MiniCssExtractPlugin, but in certain configs you cannot use experiments: css (also per docs), which means we cannot use module type: css. Ergo, we need MiniCssExtractPlugin

This problem is already tracked here as well: https://github.com/web-infra-dev/rspack/issues/3158

Basically any usage of css-loader leads to this problem.
Once you add css-loader with custom configuration, then css is not extracted into files => you need to use mini-css-extract-plugin => getting an error

bump

bump

Hi, any progress with this?

I'm closing this issue as an alternative css extract plugin has been provided.


chore: Cleaned setup.py and updates gitignore

This PR cleans setup.py of an unused import and adds /data in gitignore.
EDIT: added a setup.cfg for future conda release.

Hi,
You didn't mention what are you dealing with conda.
Also, you seem to link a LICENSE that no longer applies since we are now contributing as a team.

Hi,
You didn't mention what are you dealing with conda.
Also, you seem to link a LICENSE that no longer applies since we are now contributing as a team.

We have to update the License as well but the modification in this PR is still valid, isn't it?
(it only points to the License)

Hi,
You didn't mention what are you dealing with conda.
Also, you seem to link a LICENSE that no longer applies since we are now contributing as a team.

We have to update the License as well but the modification in this PR is still valid, isn't it?
(it only points to the License)

Why would we want to release on Conda an invalid License ?

Why would we want to release on Conda an invalid License ?

Hey there,
I can change my default name to Pynear Contributors if you like, but this seems like a topic for a different PR. Or I can remove the setup.cfg.
Again, here I just wanted to configure everything to point to the updated version of files required to publish. I'm just suggesting to set up a "pointer" here, changing the content of the License is fine but feels like something for a different PR. Besides, I think we still have an ongoing discussion about which license to use.
Do you prefer that I remove the setup.cfg? or that I edit my name into Pyronear contributors?

I think things should get done in reversed order:
We choose a valid license, then we release the lib pointing to it not the other way around.

I think things should get done in reversed order:
We choose a valid license, then we release the lib pointing to it not the other way around.

No worries, just removed setup.cfg!
Let's open up a PR later specifically for the license


Add NATS support for basic pubsub for state updates

Still WIP, not ready for merging.
Needs tests.

Closing as this will go on Provide's 1.x branch.


Support Team in Short Create

Currently you cannot set the Team when creating a Shortcut Story. This adds a new flag -T, --team.

Not an expert with this package, but seems like a good change to me!

Thanks, I'll look into it this weekend. I am not using Shortcut anymore so I don't have a good way to test it without opening a new account.

@j-martin Would you like me to help as a maintainer? We use Shortcut as our primary ticket management tool at our company.

@omairvaiyani sure, I invited you to the repo. What is your npm.js account so I can give you publish access?

@omairvaiyani sure, I invited you to the repo. What is your npm.js account so I can give you publish access?

Thank @j-martin my username is omairvaiyani.
Let me know if you prefer to have someone else approve this PR to avoid me merging my own.

@omairvaiyani I have added you to the npmjs org so you should be able to publish a new version.

@omairvaiyani I have added you to the npmjs org so you should be able to publish a new version.

Thanks @j-martin though I can only see the deprecated clubhouse-cli package, not this one?

@omairvaiyani j-martin just made me an admin of this github repo, are you still interested in being a maintainer?

Could this PR be fixed, and merged, please? :D

Could this PR be fixed, and merged, please? :D

I've rebased - ready for merge I hope @mdnyhus

Sorry this slipped through the cracks, I'll get a new version out with this change soon.

I've released version 2.3.4 that contains this change (and some dependency updates)


[engine] Linux Fuchsia and Linux Fuchsia FEMU don't have runif logic

https://github.com/flutter/engine/blob/master/.ci.yaml#L160-L176
This causes these builds to always be run on presubmit, even if the change was just a markdown.
See https://github.com/flutter/engine/pull/29566 where these were run on a darwin only change.
Adding some paths in the runif for where Fuchsia code is would help prevent unnecessary testing

\cc @chandarrengoog

CC: @arbreng?

This seems like a pretty good way to free up some CI resources, nice catch :)

@arbreng It sounds like @dnfield's suggestion could be pretty brittle. Do you know of any more robust way of doing this?

I like this suggestion:  "Ideally we'd be using something like gn analyze to figure out whether the git ls-files changed anything that's part of the target(s) we plan to build, and ideally before scheduling the bot."
The fuchsia embedder uses all of the engine, just like the other embedders do.  So, this " It's basically saying don't run if you changed files that are only under shell/platform/(darwin|android|linux|windows|glfw)." is the correct pattern.
Darwin would have " It's basically saying don't run if you changed files that are only under shell/platform/(android|fuchsia|linux|windows|glfw)." and the same goes for the other embedders.

Closing because the bug has not been updated for more than one year.
Feel free to reopen any bugs that you believe are important but please provide updates on how to resource the fix of the bug and what can be done to prevent the issue from not getting updates for another year.

This is still a valid issue. Reopening.

@godofredoc - based on my question in https://github.com/flutter/flutter/issues/93261#issuecomment-963451300, does ci.yaml support some way of saying "run if not"?
The problem for this target is that it should not be run if the changes are exclusive to a few directories. If we craft a regular "run-if" it will regress over time when someone adds a new directory.

The filtering logic runs server side and is limited to use the pull request data returned by github.
Not sure I understand how the "run if not" will be used, @dnfield can you please provide an example to see if it can be calculated from the data available to backend?

Right now we can say:
"Run if any files have changed in the following directories."
I want to be able to say:
"Run if there are any changes to files not in the following directories"

@dnfield run_if_not has been implemented moving this bug back to you for the final filters.

@matanlurey @johnmccutchan This is relevant to our conversation the other day.l

I'm happy to take this issue.

Thanks, Matan!


Dev

Changes,

Added user-specific files to gitignore
Fixed logging issues
Added Http method for DOS and DDOS logging
Minor cleanups

Merging branch dev for fixes


remove eli5 test

since the dataset is defunct

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Stream Connection Problems

Hello, I'm using this library to create a back-end to an App I'm building for my portfolio, so I don't plan on releasing it on an App Store.
One thing that it does is that it listens to tweets of the accounts the user has added to the app.
Since I'm still building it, I oftenly have to change one or two lines of code in the back-end, then save and it starts running again. But I'm having this annoying problem, that whenever the app stops and starts running again after a short time, the stream doesn't start so it can't listen to tweets.
TwitterError: ConnectionException: This stream is currently at the maximum allowed connection limit.
    at Function.module.exports.fromJson (/home/samuel/stlk-api/node_modules/twitter-v2/src/TwitterError.js:29:12)
    at Stream.<anonymous> (/home/samuel/stlk-api/node_modules/twitter-v2/src/TwitterStream.js:60:40)
    at Stream.emit (node:events:379:20)
    at Stream.EventEmitter.emit (node:domain:470:12)
    at drain (/home/samuel/stlk-api/node_modules/through/index.js:36:16)
    at Stream.stream.queue.stream.push (/home/samuel/stlk-api/node_modules/through/index.js:45:5)
    at emit (/home/samuel/stlk-api/node_modules/split/index.js:37:14)
    at Stream.<anonymous> (/home/samuel/stlk-api/node_modules/split/index.js:60:7)
    at _end (/home/samuel/stlk-api/node_modules/through/index.js:65:9)
    at Stream.stream.end (/home/samuel/stlk-api/node_modules/through/index.js:74:5) {
  details: 'https://api.twitter.com/2/problems/streaming-connection'
}

I've tried to listen to the stop events, but to no success
process.on('SIGTERM', () => {
        console.info('SIGTERM signal received.');
        console.log('Closing http server.');
        twitterService.closeStream();
    });
    process.on ('SIGINT', () => {
        console.info('SIGINT signal received.');
        console.log('Closing http server.');
        twitterService.closeStream();
        process.exit(0);
    });

Is there anything I can do to fix or workaround this?

I can confirm that I am having the same issue. It seems to be due to ratelimiting on the part of Twitter. Has anyone else experienced this behavior?

Unfortunately @zane-programs is correct. Twitter rate limits all streams such that if you're frequently connecting and reconnecting you'll run into a ConnectionException.
Note that:

Each app is allowed one connected client at a time. You can attempt reconnecting up to 15 times per 15 minute period. You can make up to 450 rule changes per 15 minute period.

and

To check connection limits response will return three headers: x-rate-limit-limit, x-rate-limit-remaining and x-rate-limit-reset headers. This is useful to understand how many times you can use the rule endpoint, and how many reconnections attempts are allowed for the streaming endpoint.

If your dev workflow requires several restarts within a 15min period you may want to consider streaming from Twitter with a different process which your binary connects to instead. EG: create a thin JS client that connects to twitter and proxies all streamed data. In fact, that might not be a bad thing to add to this repository as a utility if there's enough interest.

Interestingly, this quirk does not just occur after restarts. In fact, I just applied for a new Twitter Developer account on my Twitter alt account, and I ran into the stream disconnect issue on my very first use of the new keys.
I don't think this is on @HunterLarco though; it seems to be a quirk with Twitter's streaming API itself. Alas, I'll be polling Twitter for now instead.


is Json support response-transformer plugin needed?

@thefosk  I was discussing the plugin with @thibaultCha and it seems like json transformation will fail for response body bigger than buffer size. By the time plugin will update the body it will may already be streamed to the client.  Also though allowed its not desirable to set content-length nil as it may break client.
So we have choice to remove Json support or document the inability of plugin to handle bigger response.

I would day s/needed/relevant. If we decide to keep it all the weird edge-cases produced by such a usage need to be documented.
(Also will people actually use that, instead of simply changing their service's responses? Is a question I think we should as ourselves).
@Mashape/kong

I'm intrigued by this statement "By the time plugin will update the body it will may already be streamed to the client"
is there no way to interrupt the stream, execute the transformation and re-calculate Content-Length before sending back to the client?

The upstream response is streamed back to the client. Meaning the headers could very well be sent to the client when the body is received by Kong. In that case, it is too late to modify the Content-Length.
The only thing we can do, is indicate people to configure their pproxy_buffer_size as best as they can for their use case, so Kong could modify the response while it is still in buffer before sending it. However, this will of course have an impact of performance because we are defeating the purpose of Nginx by doing so and killing the streaming capabilities.

Just noticed that plugin does not send data to client as soon as it receives  the chunk from upstream server but save to it buffer.
https://github.com/Mashape/kong/blob/master/kong/plugins/response-transformer/body_filter.lua#L27
Once final chunk is received, transformation is performed and then buffered data is sent to client.
https://github.com/Mashape/kong/blob/master/kong/plugins/response-transformer/body_filter.lua#L49
So though data is not streamed back but it may still cause memory issue if data is big?

It will send chunks to the client before receiving the end of the body if the chunks exceed the proxy_buffer_size, iirc.

I would do three things:

Create a test case that demonstrates this problem.
Experiment with possible solutions.
Document any limitation.

Yes, exactly the same


[TIKA-4309] detailed Mach-O in tika-mimetypes.xml

A follow-up to #1947: Support 32-bit and 64-bit Mach-O files and detect their type
cc @tballison

Thanks to this, I learned about type="host32" and the others. Thank you! :D

CI failure appears to be unrelated

Y, thank you. I just bumped protobuf on main. Let's hope the build works: https://github.com/apache/tika/actions/runs/10965546331
ossindex stopped complaining locally at least. :D

Rebased the branch


Resetting transition-duration to 0 and triggering onSlideChangeEnd takes too long

I have a fullscreen vertical swiper with speed: 2000, nothing special. When the slide has already completely transitioned to the next one it still takes 2 seconds to reset the transition-duration to 0 and trigger onSlideChangeEnd. I guess it has something to do with the default logarithmic movement of the swiper that doesn't get rounded/floored proper, so that it looks like the slide is already fully transitioned on screen, but not in the calculation. However my math is really rusty so I couldn't figure it out myself ;) Any help on this one would be really appreciated since I think it would also fix nolimits4web/Swiper#1401

I can't replicate this issue, but it seems like something wrong with settings or layout. Do you have an example or JSFiddle where i can see this issue?

Ok, it had time to figure this out. But the issue is with Chrome itself. For some weird reason until you scroll mouse wheel, transitionEnd event doesn't occur, this is strange as they don't related to each other

Made a JSFiddle because I noted some other weird issues concerning this as well:
http://jsfiddle.net/cztaLL81/
If you look in the Chrome Inspector you'll see an transition-duration of 0ms on init. During a slide it sets itself to 2000ms, but when the slide is already changed in the screen the duration is still 2000ms. Then after a short time it resets itself back to 0ms.
I was wondering if this was coming because the calculation is still not really at 0, but the slider looks to be completed or that this is something different?
Also as you might noted, if you keep scrolling the Swiper never resets back to 0ms, but I think this is the Chrome bug you spoke about.

It is a Chrome issue. You can even check the following swipe up/down and then keep scrolling mousewheel, transition end will not occur until you stop scrolling mousewheel. Nothing common with calculation, just a Chrome issue, probably new, which doesn't trigger transitionEnd event until you scroll mouse

Closing due to Chrome issue https://code.google.com/p/chromium/issues/detail?can=2&start=0&num=100&q=transitionend&colspec=ID Pri M Stars ReleaseBlock Cr Status Owner Summary OS Modified&groupby=&sort=&id=534246


Unable to reproduce the performance reported in the paper.

Impressive idea! But i cannot reproduce the performance reported in the original paper using the default settings and parameters, particularly with respect to the F1 and ACC2 metrics (with a gap of about 2%). I have tried several random seeds during training. Could it be possible that I am overlooking some crucial details during the network training process?
Thanks in advance!

I met the same problem.


Rename symbols for a more consistent use

an interface (IServerModule) stays the same
the dyn new type is prefixed with Dyn (DynIServerModule)
the typed "plugin" (ServerModulePlugin) is now just ServerModule that enables implementors to write impl ServerModule for ...

I also renamed the struct VerificationCache to struct VerifiedNotes as it conflicted with the typed trait VerificationCache.
part of #1232

I would drop that I, but will live with it one way or the other.
For @elsirion to decide.

the fixup commit drops the I in Dyn. Does github offer some autosquash functionality when merging? otherwise I'd like to squash that myself before it's being merged.
also a test for  #1253  :)

I'm sorry, but I think #1269 just sent you into rebase hell :grimacing: might be easier to just re-do the renames as long as you remember which types you renamed.


tk-8_6: fix hash after tcl-8_6 update

Motivation for this change
tcl-8_6 was updated in 4fb92ae60d68344254b2bb753c03fd72f3ffb35c from 8.6.9 -> 8.6.11 but tk's hash wasn't updated at the same time, which means the current hash was still from tk 8.6.9 rather than 8.6.11.
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/)
[ ] Added a release notes entry if the change is major or breaking
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125307 at b0c69a09 run on aarch64-linux 1

50 packages marked as broken and skipped:

OVMF-CSM
cassandra_2_1
cassandra_2_2
deeptools
gtmess
hydrus
impressive
ocropus
odpdown
opentsdb
pig
polymake
processing
python38Packages.HTSeq
python38Packages.caffe
python38Packages.cnvkit
python38Packages.dask-xgboost
python38Packages.datashader
python38Packages.fipy
python38Packages.foundationdb51
python38Packages.foundationdb52
python38Packages.foundationdb60
python38Packages.foundationdb61
python38Packages.foxdot
python38Packages.ignite
python38Packages.python-csxcad
python38Packages.python-openems
python38Packages.spacy
python38Packages.spacy-transformers
python38Packages.textacy
python38Packages.thinc
python39Packages.HTSeq
python39Packages.caffe
python39Packages.cnvkit
python39Packages.dask-xgboost
python39Packages.fipy
python39Packages.foundationdb51
python39Packages.foundationdb52
python39Packages.foundationdb60
python39Packages.foundationdb61
python39Packages.foxdot
python39Packages.ignite
python39Packages.python-csxcad
python39Packages.python-openems
python39Packages.thinc
scribus
spark
spring
springLobby
yosys-bluespec

14 packages failed to build:

asciidoc-full-with-plugins
fop
helmholtz
igraph
python38Packages.descartes
python38Packages.kmapper
python38Packages.mlxtend
python38Packages.python-igraph
python38Packages.spyder-kernels_0_5
python38Packages.trackpy
python39Packages.basemap
python39Packages.descartes
python39Packages.python-igraph
python39Packages.trackpy

595 packages skipped due to time constraints:

DisnixWebService
OVMF
OVMF-secureBoot
R
abcl
adapta-gtk-theme
adoptopenjdk-icedtea-web
aiodnsbrute
alloy (alloy5)
alloy4
...

121 packages built successfully:

anki
cbrowser
corrscope
flatcam
graph-cli
image-analyzer
itktcl
jre8 (jre8_headless)
mnemosyne
mrpeach
puredata
puremapping
pyspread
python38Packages.Pmw
python38Packages.Pweave
python38Packages.androguard
python38Packages.ase
python38Packages.castepxbin
python38Packages.colorcet
python38Packages.deap
python38Packages.dufte
python38Packages.flametree
python38Packages.fonttools
python38Packages.genome-collector
python38Packages.glymur
python38Packages.gtts
python38Packages.holoviews
python38Packages.imagecorruptions
python38Packages.ipympl
python38Packages.librosa
python38Packages.lightning
python38Packages.matplotlib
python38Packages.memory_profiler
python38Packages.monty
python38Packages.moviepy
python38Packages.mplfinance
python38Packages.nbval
python38Packages.osmpythontools
python38Packages.panel
python38Packages.parfive
python38Packages.persim
python38Packages.phik
python38Packages.pims
python38Packages.pint
python38Packages.proglog
python38Packages.pyalgotrade
python38Packages.pycm
python38Packages.pymatgen
python38Packages.pyregion
python38Packages.pyspice
python38Packages.pytest-astropy
python38Packages.pytest-astropy-header
python38Packages.pytest-mpl
python38Packages.pytest-regressions
python38Packages.pytomlpp
python38Packages.regional
python38Packages.ripser
python38Packages.scapy
python38Packages.scikitimage
python38Packages.seaborn
python38Packages.showit
python38Packages.tadasets
python38Packages.tkinter
python38Packages.tqdm
twine (python38Packages.twine)
python38Packages.vidstab
python39Packages.Pweave
python39Packages.androguard
python39Packages.ase
python39Packages.castepxbin
python39Packages.colorcet
python39Packages.deap
python39Packages.dufte
python39Packages.flametree
python39Packages.fonttools
python39Packages.genome-collector
python39Packages.glymur
python39Packages.holoviews
python39Packages.imagecorruptions
python39Packages.ipympl
python39Packages.lightning
python39Packages.matplotlib
python39Packages.memory_profiler
python39Packages.monty
python39Packages.mplfinance
python39Packages.nbval
python39Packages.osmpythontools
python39Packages.panel
python39Packages.parfive
python39Packages.persim
python39Packages.proglog
python39Packages.pyalgotrade
python39Packages.pycm
python39Packages.pycocotools
python39Packages.pymatgen
python39Packages.pyregion
python39Packages.pyspice
python39Packages.pytest-astropy
python39Packages.pytest-astropy-header
python39Packages.pytest-mpl
python39Packages.pytest-regressions
python39Packages.pytomlpp
python39Packages.regional
python39Packages.scapy
python39Packages.scikitimage
python39Packages.seaborn
python39Packages.showit
python39Packages.spyder-kernels_0_5
python39Packages.tadasets
python39Packages.tkinter
python39Packages.tqdm
python39Packages.vidstab
remind
tartube
tclreadline
tix
tk (tk-8_6)
tkcvs
uudeview
vkeybd
wordnet

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.

Should this be rebased to staging? What is the criteria for that, exactly?

500+ rebuilds is a pretty good indicator. Also tk takes a while to build.

/rebase staging

Changed target branch to staging, thanks!


Why is Mum used with minus?

I find it very interesting that the "multiply and mix" operation uses minus for the mixing in line
https://github.com/tommyettinger/waterhash/blob/49f5cf0b63b95b112b45c054afabb8d9c904ce3a/waterhash.h#L17
Is this just an arbitrary choice? Or have you seen this elsewhere? Or written about why you chose it?

To be perfectly honest, it's because SMHasher found anomalies or had failed tests with addition and xor, and for some reason didn't have problems with subtraction. Trial and error, where there aren't that many trials possible (+, -, ^, but then what?) In all cases but xor, the result is an imperfect mix, but mum here can't be a perfect mix (an invertible function) because it takes in more bits than it intends to produce, if I remember correctly. I'd seen subtracted right-shift somewhere before; I don't know where, though it wasn't in the context of a "mathematically clean" function.
WaterHash is really practically scraping the edge of what SMHasher finds acceptable in a hash, though it does narrowly get through it and does so with decent speed. XXH3 is incredible and quite a bit faster, but is a much larger code base. I'm guessing most people who are looking at WaterHash are not also looking at AES-based options, but if using a setup that can take advantage of hardware acceleration for AES operations, that will probably be fastest. I think WaterHash is mostly interesting because it's somewhat small and still passes tests with good-enough performance; wyhash has changed since I made my WaterHash edits, and it may be faster or slower now.

That's fascinating. I thought this for sure couldn't make a difference in the results, so it would have to be for performance somehow. Thank you for the comprehensive explanation!


Internationalization code Example

Made it more clear how to use Internationalization on CLNDR

Love it, thank you @bastiW! 🍻


fix: validate object name

Using Pydantic for validation should give us enforcement in both SDK and server.
Please let me know if you have thoughts on what the max length / valid characters should be if different than what I have here.
With this change, trying to do something like:
weave.publish(['felix', 'jimbo', 'billie'], 'foo/bar')
Would yield:
ValidationError: 1 validation error for ObjSchemaForInsert
object_id
  String should match pattern '^[A-Za-z0-9._-]+$' [type=string_pattern_mismatch, input_value='foo/bar', input_type=str]
    For further information visit https://errors.pydantic.dev/2.6/v/string_pattern_mismatch

would this crash the user's script though? I think as a general design principle, crashing user code feels like a thing we should always avoid.


Fix output array dtype when computing the remainder

It stated that the return type should be floating-point, but it accepts
integer inputs (and the remainder of two integers is always an integer).
Somewhat related, I noticed that remainder doesn't specify any special cases, but it perhaps should specify the behavior for remainder by 0, and infinites. Also should integer remainder by 0 be unspecified?

Regarding the special cases mentioned in the OP, we should open a separate issue.


Remove FLIK from LU

As discussed, I think...

Yes, FLIK stands for "Flächen-Identifikator" (some kind of field block id) and is interpreted differently in each member state...


[SPARK-33769][SQL] Improve the next-day function of the sql component to deal with Column type

What changes were proposed in this pull request?

The proposition of this pull request is described in this JIRA ticket: https://issues.apache.org/jira/browse/SPARK-33769
It proposes to improve the next-day function of the sql component to deal with Column type for the parameter dayOfWeek.
Why are the changes needed?

It makes this functionality easier to use.
Actually the signature of this function is:

def next_day(date: Column, dayOfWeek: String): Column.

It accepts the dayOfWeek parameter as a String. However in some cases, the dayOfWeek is in a Column, so a different value for each row of the dataframe.
A current workaround is to use the NextDay function like this:

NextDay(dateCol.expr, dayOfWeekCol.expr).

The proposition is to add another signature for this function:

def next_day(date: Column, dayOfWeek: Column): Column

In fact it is already the case for some other functions in this scala object, exemple:

def date_sub(start: Column, days: Int): Column = date_sub(start, lit(days))
def date_sub(start: Column, days: Column): Column = withExpr { DateSub(start.expr, days.expr) }

or

def add_months(startDate: Column, numMonths: Int): Column = add_months(startDate, lit(numMonths))
def add_months(startDate: Column, numMonths: Column): Column = withExpr {
AddMonths(startDate.expr, numMonths.expr)
}

This pull request is the same idea for the function next_day.
Does this PR introduce any user-facing change?

Yes
With this pull request, users of spark will have a new signature of the function:

def next_day(date: Column, dayOfWeek: Column): Column

But the existing function signature should still work:

def next_day(date: Column, dayOfWeek: String): Column

So this change should be retrocompatible.
How was this patch tested?

The unit tests of the next_day function has been enhanced.
It tests the dayOfWeek parameter both as String and Column.
I also added a test case for the existing signature where the dayOfWeek is a non valid String. This should return null.

cc @HyukjinKwon

ok to test

Test build #132785 has started for PR 30761 at commit 52f4213.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37387/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37387/

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

Test build #132785 has finished for PR 30761 at commit 52f4213.

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

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

Test build #132815 has started for PR 30761 at commit 7c32905.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37416/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37416/

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

Merged to master.

Test build #132815 has finished for PR 30761 at commit 7c32905.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

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


Upgrade to GitHub Terraform Provider versions beyond v4.1.0

The current version of the GitHub Terraform provider is 4.3.0 and it would be great if the Pulumi GitHub module could be upgraded to that version (currently on v4.1.0).
Possibly the new version releases have not been noticed as the GitHub Terraform provider has recently migrated to a new repository.
Thank you!

Thanks so much for this repo @felix11h - this slipped past me that they migrated - upgrade PR on the way
Paul

Thanks so much for this repo @felix11h - this slipped past me that they migrated - upgrade PR on the way
Paul


Autocomplete selects/deselects when pressing tab key

Describe the bug
When using the Autocomplete component from eds-core-react with multiple enabled, pressing the tab key selects the highlighted option. The option is selected regardless if it is highlighted by using arrow keys or hovering with the mouse cursor.
Steps to reproduce the bug
Follow the steps in this simple reproduction: https://codesandbox.io/s/headless-pond-vz6toz?file=/src/App.tsx

Navigate to the combobox using TAB / click the combobox
Select options with arrow keys and enter
Press TAB
Highlighted option was selected/deselected

Expected behavior
Pressing the tab key should just move focus to the next element, not change which options are selected.
Specifications

Version: 0.27.0
Browser: Chrome
OS: Windows

Additional context
https://codesandbox.io/s/headless-pond-vz6toz?file=/src/App.tsx

I agree, pressing tab in an Autocomplete with multiple should just close the dropdown without selecting/deselecting the focused item 👍


Add possibility to use TVD instead of median for segmentation.

In addition, apply contrast enhancement, and allow in principle to aplpy segmentation only on parts of the domain.

I assume that you will make the checks pass before you merge!


operator project-quay (3.12.0-nightly.20240518)

Add 3.12.0-nightly.20240518 to community operators

/merge possible

/merge possible


[Task]: query.data is shown as null instead of undefined.

SubTasks
When a query isn't run and you access the data in Applications, the value is undefined, but it's value is null in Workflows.

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr


Max projects per user should NOT apply to the root role

root users can pin/unpin projects so it's perfectly fine to allow unlimited number of projects; currently when root users reach maximum number of subjects UI validation fails to gracefully validate and emits an error page

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1574

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1582

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1586

:+1:


Support specifying quote and escape in Csv WriterBuilder

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
In DataFusion https://github.com/apache/arrow-datafusion/pull/8351 we are adding support to specify the escaping and quote characters used for CSV reading.
It turns out that the Arrow CSV writer does not support specifying the escape and quoting characters
Describe the solution you'd like
I would like the following in the CSV WriterBuilder:

with_escape (analogous to ReaderBuilder::with_escape )
with_quote (analogous to ReaderBuilder::with_quote )

Describe alternatives you've considered

Additional context

This should be a relatively straightforward first issue.
I would highlight though that "escape" is advised against by the CSV library we use - https://docs.rs/csv/latest/csv/struct.WriterBuilder.html#method.escape

In some variants of CSV, quotes are escaped using a special escape character like \ (instead of escaping quotes by doubling them).
By default, writing these idiosyncratic escapes is disabled, and is only used when double_quote is disabled.

I'd like to have a try.


Readme: improve recommended mappings (functions are now called silently)

@quangnguyen30192 I tested this and it seems to work and make no difference. Does this look good to you?

let me try on my machine

Thank you, I split the two commits into a separate PR

works fine 👍


[AutoPR servicefabric/resource-manager] ServiceFabric: added package tag, added java readme

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6483

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/3731


add @sveltejs/enhanced-img to top-level readme

Updated README
Added missing link to the new @sveltejs/enhanced-img package.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Thanks everyone, this is my first contribution to sveltekit :D


Feature #4 plotting

Three functions were added that produce different types of plots.

Initial version of a basic plots.


"Cannot read property 'startContainer' of null" with b element

"Cannot read property 'startContainer' of null" JavaScript error with the following initial content:
Test

Description
Using Trumbowyg 2.25.0 and setting
Test
 as initial content for the editor I get the JS error "Cannot read property 'startContainer' of null" and the toolbar is not available.
How to reproduce?
https://jsfiddle.net/dLvfeaob/1/

Test

$(document).ready(function() {
  $('#editor').trumbowyg();
});

Thank you for the report!
I've fixed it in 2.25.1 :)

wow 🚀 thanks @Alex-D


[improve][pip] PIP-347: add role field in consumer's stat

Motivation
pip-347
implementation pr：https://github.com/apache/pulsar/pull/22562
Modifications

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

PTAL, thanks. @BewareMyPower @lhotari @codelipenghui @poorbarcode @Technoboy-

Leaving aside the issue above.
It looks like you want to add role to the consumer stats, that is a good idea to track the relationship between subscriptions and users.
There are times when the role is more sensitive information and I would recommend adding a configuration to expose the role.

May be we can prioritize reaching consensus on this PIP? We do need this pip to handle the problem we meet, anyone who use Subscription Permission Control Mechanism will need this.
@michaeljmarshall @nodece @dao-jun @codelipenghui @BewareMyPower @Technoboy-

@thetumbled After @michaeljmarshall think it's OK, you can sent a vote thread to the mail list

Leaving aside the issue above.
It looks like you want to add role to the consumer stats, that is a good idea to track the relationship between subscriptions and users.
There are times when the role is more sensitive information and I would recommend adding a configuration to expose the role.

We can solve this problem with a new option: add a new option to expose roles info, and we only allow the cluster admin or tenant admin to do this.

Leaving aside the issue above.
It looks like you want to add role to the consumer stats, that is a good idea to track the relationship between subscriptions and users.
There are times when the role is more sensitive information and I would recommend adding a configuration to expose the role.

Good idea, i have updated the doc.

Leaving aside the issue above.
It looks like you want to add role to the consumer stats, that is a good idea to track the relationship between subscriptions and users.
There are times when the role is more sensitive information and I would recommend adding a configuration to expose the role.

@nodece I don't understand why role name can be sensitive.
It's not accessKey or username or token or something else, it is just a role name, users cannot accessed in by the role name at all. Like us, we are Software engineer, is Software engineer sensitive?
The PIP is a simple change, let's keep it simple, I don't think add a exposeRole param is reasonable.

In other words, I just don't want to expose the role to the regular users.

@nodece I think it doesn't matter, expose role name will not affect anything

WDYT, should we expose it to regular users(not everyone actually, but those who own the lookup permission of this topic)? @eolivelli @lhotari @codelipenghui @Technoboy- @michaeljmarshall

PIP approved, merging...


Add logical operator support in templates

Handlebars supports very basic if condition checking, but it only checks for
existence of a field. There are times when we want to combine conditions in order
to generate something as part of a blueprint, without defining a completely separate
blueprint for it (like generating network config schema if a dt has a wifi chip or
a usb port to which we can connect a dongle).
Change-type: minor
Signed-off-by: Stevche Radevski<EMAIL_ADDRESS>

LGTM 👍


First argument was an invalid path: "undefined"

I'm following the AngularJS + Firebase tutorial by Thinkster and I've run into the following error when simply trying to connect to Firebase:

Error: Firebase.child failed: First argument was an invalid path: "undefined". Paths must be non-empty strings and can't contain ".", "#", "$", "[", or "]"
    at Error (native)
    at Ia (http://localhost:9000/bower_components/firebase/firebase.js:20:101)
    at D.E.G (http://localhost:9000/bower_components/firebase/firebase.js:148:216)
    at Object.Post.get (http://localhost:9000/scripts/services/post.js:13:49)
    at new <anonymous> (http://localhost:9000/scripts/controllers/posts.js:8:25)
    at invoke (http://localhost:9000/bower_components/angular/angular.js:3966:17)
    at Object.instantiate (http://localhost:9000/bower_components/angular/angular.js:3977:23)
    at $get (http://localhost:9000/bower_components/angular/angular.js:7281:28)
    at link (http://localhost:9000/bower_components/angular-route/angular-route.js:907:26)
    at nodeLinkFn (http://localhost:9000/bower_components/angular/angular.js:6711:13) <div ng-view="" class="ng-scope">

The code that generates this is:
'use strict';

app.factory('Post', function($firebase, FIREBASE_URL) {
    var ref = new Firebase(FIREBASE_URL);
    var posts = $firebase(ref.child('posts')).$asArray();

    var Post = {
        all: posts,
        create: function(post) {
            return posts.$add(post);
        },
        get: function(postId) {
            return $firebase(ref.child('posts').child(postId)).$asObject();
        },
        delete: function(post) {
            return posts.$remove(post);
        }
    };

    return Post;
});

FIREBASE_URL is properly defined and is a real Firebase URL.

@rfkrocktk What was the issue? I have run into this exact problem as well.

The issue is that he tried doing child(undefined) within hist Post.get() method. In his case, postId was undefined. In your case, I'd check to see what you are passing to child() and if that is what you expect it to be.
If you still have problems, please open a new issue with a repro (JSFiddle, CodePen, etc.) or ask on Stack Overflow (with a repro as well) as we monitor both closely.

Figured it out. Thanks man.

@rfkrocktk: Which version of angularfire caused that issue?
@jwngr I got the same problem, similar code and it worked fine in 0.9.0 but 0.9.1. Any ideas? Below is my code:
'use strict';
app.factory('Dashboard', function(FURL, $firebase, Auth) {
var ref = new Firebase(FURL);

var Dashboard = {

	getJobs: function() {
		var user = Auth.user;
		return $firebase(ref.child('jobs').child(uid)).$asArray();
	}

};

return Dashboard;

});
Auth is service for authentication.
'use strict';
app.factory('Auth', function(FURL, $firebaseAuth, $firebase) {
var ref = new Firebase(FURL);
var auth = $firebaseAuth(ref);

var Auth = {

	user: {},

	login: function(user) {
		return auth.$authWithPassword(
			{email: user.email, password: user.password}
		);
	},

	register: function(user) {
  return auth.$createUser({email: user.email, password: user.password})
    .then(function() {
      return Auth.login(user);
    });
},

logout: function() {
  auth.$unauth();
},

	changePassword: function(user) {
		return auth.$changePassword({email: user.email, oldPassword: user.oldpass, newPassword: user.newpass});
	},

	signedIn: function() {
  return !!Auth.user.provider;
}
};

auth.$onAuth(function(authData) {
	if(authData) {
  angular.copy(authData, Auth.user);

	} else {
  angular.copy({}, Auth.user);
	}
});

return Auth;
});

@leotrieu - As mentioned above, you should check to make sure when you call child() that you don't pass undefined. Looking at this line in your code:
return $firebase(ref.child('jobs').child(uid)).$asArray()
I see that uid is not defined. Did you mean user.uid?
If you still have problems, please open a new issue with a repro (JSFiddle, CodePen, etc.) or ask on Stack Overflow (with a repro as well) as we monitor both closely.

@jwngr Sorry for my late response. All good. It was other error, not user.uid. I missed uid above because of pasting wrong one. Thanks for your response :-)


python310Packages.puremagic: 1.11 -> 1.12

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.puremagic is: "Pure python implementation of magic file detection"
meta.homepage for python310Packages.puremagic is: "https://github.com/cdgriffith/puremagic"
meta.changelog for python310Packages.puremagic is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.12 with grep in /nix/store/d9r1zvpwdy7mfajj096nwflh9a62g69w-python3.10-puremagic-1.12

found 1.12 in filename of file in /nix/store/d9r1zvpwdy7mfajj096nwflh9a62g69w-python3.10-puremagic-1.12

Rebuild report (if merged into master) (click to expand)

7 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
3 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
opsdroid
python310Packages.puremagic
python39Packages.puremagic
sabnzbd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d9r1zvpwdy7mfajj096nwflh9a62g69w-python3.10-puremagic-1.12 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.puremagic https://github.com/r-ryantm/nixpkgs/archive/cdf3103d9ba9c0f68aeed5f31d71d66d142cadac.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d9r1zvpwdy7mfajj096nwflh9a62g69w-python3.10-puremagic-1.12
ls -la /nix/store/d9r1zvpwdy7mfajj096nwflh9a62g69w-python3.10-puremagic-1.12/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

opsdroid
python310Packages.puremagic
python39Packages.puremagic
sabnzbd

Maintainer pings
cc @globin for testing.

Result of nixpkgs-review pr 159930 run on x86_64-linux 1

4 packages built:

opsdroid
python310Packages.puremagic
python39Packages.puremagic
sabnzbd


[storage][avro] add support for aborter and browser

TODO

[x] support browser
Get the whole Blob before processing.
[x] support cancellation
Remove listeners and reject.
[ ] Big int for readLong?
Not sure if we have big int library support in Azure core.

Discussion
How to consume the download response in the browser without having to wait for the whole body to be transferred?
Currently Promise<Blob> is returned in browser. I have to get the whole Blob before I can parse it as an avro file even though the user may only care about the first record in it and the whole file may be quite large for change feed. How about supporting ReadableStream in browser? It's now widely supported except IE. We also get similar ask for upload from customers: https://github.com/Azure/azure-sdk-for-js/issues/6526.
export type BlobDownloadResponse = BlobDownloadHeaders & {
  /**
   * BROWSER ONLY
   *
   * The response body as a browser Blob.
   * Always undefined in node.js.
   */
  blobBody?: Promise<Blob>;

  /**
   * NODEJS ONLY
   *
   * The response body as a node.js Readable stream.
   * Always undefined in the browser.
   */
  readableStreamBody?: NodeJS.ReadableStream;
...
}

How to cancel the reading from the downloaded response?
The data input of the avro will either be the NodeJS.ReadableStream or the Promise<Blob>.  We can pass the aborter to the download call in upper layer to cancel the download. Inside avro, how to handle the abort? @jeremymeng @bterlson @XiaoningLiu
For Node.js, the ReadableStream do not seem to have any interface for cancellation.
export interface ReadableStream extends EventEmitter {
        readable: boolean;
        read(size?: number): string | Buffer;
        setEncoding(encoding: string): this;
        pause(): this;
        resume(): this;
        isPaused(): boolean;
        pipe<T extends WritableStream>(destination: T, options?: { end?: boolean; }): T;
        unpipe<T extends WritableStream>(destination?: T): this;
        unshift(chunk: string): void;
        unshift(chunk: Buffer): void;
        wrap(oldStream: ReadableStream): this;
 }

In storage-blob/RetriableReadableStream, our abortHandler is like this:
  private abortHandler = () => {
    this.source.pause();
    this.emit("error", ABORT_ERROR);
  };

Doesn't help much.
Probably we should simply stop consuming the ReadableStream/ Promise if we could; And let the download aborter to trigger any clean-up that is needed.

Blob Storage Change Feed Samples (JavaScript)

This is leading to NOT FOUND.

Refers to: sdk/storage/storage-blob-changefeed/README.md:156 in 29ebbfd. [](commit_id = 29ebbfd9bc80d65255a9043dcf66d92a47f6f3e9, deletion_comment = False)

Blob Storage Change Feed Samples (JavaScript)

This is leading to NOT FOUND.
Refers to: sdk/storage/storage-blob-changefeed/README.md:156 in 29ebbfd. [](commit_id = 29ebbfd, deletion_comment = False)

Will fix this in the change feed PR by adding js samples.


skybox interference with grid on 3d solved. methods names changed

skybox interference with grid on 3d solved. methods names changed

skybox interference with grid on 3d solved. methods names changed


Subtitle doesn't show in fullscreen mode

Hi,
I'm using your javascript to show translated subtitles for youtube videos.
Lately, in fullscreen mode, subtitles doesn't show both on PC and Mobile.
I remember it worked before on PC(but not on mobile).
If you can look that up to fix it, that would be nice.
Thank you for your works.

Would it be possible to have an area bellow the iframe? Somehow forcing the iframe to be maxheight 90% then the subs in the remaining 10%?

Would it be possible to have an area bellow the iframe? Somehow forcing the iframe to be maxheight 90% then the subs in the remaining 10%?

No, sadly the script can't alter the content of the iframe and in fullscreen mode the iframe requires the whole screen.

Seems we can disable the full screen button (double click still available).

- So we could add "our own" full screen button on the small video, in the corner.
- When the user clicks on it, we could change the height="400" to browser height, then a message could tell the desktop user to press F11 to have the true full screen with subs. On mobile, browsers are almost full screen anyway.


Agent deployment configuration: RKE1 (Ember)

The solution would be to provide a method for setting custom parameters to override cattle-cluster-agent and fleet-agent settings at a per cluster level or when deploying rancher via any of the supported deployment options (i.e., helm). (we are focusing on the per-cluster configuration but should acknowledge the original ask).
There are two major asks for configuring agent deployments:

Setting Tolerations and Affinity Rules
Setting Resource Limits

As Rancher does not have a unified cluster interface for every type of cluster, consideration needs to be given to how we present these options in each distribution.
Focus for this ticket is for RKE1, which is done via Ember.
N.b: the Norman API would need to handle strings as part of affinity, so this would require an update.

Automation for this will be handled by #8828 to help QA. Leaving here for now so that feature testers are aware of this amidst other testing.


Add support for Python 3.7

Python 3.7 is out now, and so it would be great to support it.
After adding the new deadsnakes ppa I submitted in https://github.com/uber/pyflame/pull/150 I discovered that we need to run the tests on Xenial for Python 3.7 to work. See https://github.com/travis-ci/travis-ci/issues/9069#issuecomment-382599654
As a user (but not developer) of pyflame this is basically as far as I've got. I tried catching Python 3.7 and applying the 3.6 ABI but that's just guess work.
Currently:

18 failed, 19 passed

Hopefully someone with some pyflame dev experience can have a go at getting it working :-)
See:

Travis Job for by Python 3.7 branch: https://travis-ci.org/tomviner/pyflame/builds/403996654
My branch: https://github.com/uber/pyflame/compare/master...tomviner:python3.7-support

Mostly adding this so it might show up in searches for the error.  3.6 is nicely reported as unsupported but 3.7 fails with an error suggesting a ptrace configuration issue (at least per #120 where searches for the error lead you).
altendky@lt:~$ git -C pyflame rev-parse HEAD
d67c353b13b6950b48bbb8fcb1dde498637a4e28
altendky@lt:~$ pyflame/src/pyflame --version
pyflame 1.6.6 linux-gnu x86_64 (ABI list: 26 34)
altendky@lt:~$ cat /proc/sys/kernel/yama/ptrace_scope
0
altendky@lt:~$ pyflame/src/pyflame -o outfile -t python3.7 -c 'import time; time.sleep(5)'
Failed to locate libpython within timeout period.
altendky@lt:~$ pyflame/src/pyflame -o outfile -t python3.6 -c 'import time; time.sleep(5)'
Target has Python ABI 36, which is not supported by this pyflame build.
altendky@lt:~$ pyflame/src/pyflame -o outfile -t python3.5 -c 'import time; time.sleep(5)'
altendky@lt:~$ pyflame/src/pyflame -o outfile -t python3.4 -c 'import time; time.sleep(5)'
altendky@lt:~$ pyflame/src/pyflame -o outfile -t python2.7 -c 'import time; time.sleep(5)'
altendky@lt:~$

Note that 2.7/3.4/3.5 did run for five seconds as expected prior to completing.

I am trying to attach python app running on pypy env i am also facing same issue can any one help here...

For 3.7, there is no _PyThreadState_Current symbol anymore.
#  define _PyThreadState_Current _PyRuntime.gilstate.tstate_current

...any progress on this? i see many forks with python3.7 support but not a single pull request merged... :-(

I got same error "Failed to locate libpython within timeout period" on Ubuntu18.04 + Python 3.7 Gunicorn. It works all configurations are same but python2.7.
Can anyone make Pyflame working with 3.7?  @randomstuff looks give some hint, but I cannot figure out how as I have minor knowledge on C/C++ code.


Feature/52

added a clickable link on the username to open the chess.com profile
fix : #52

@all-contributors please add @ARSHIL1804  for code

Since this takes the user directly outside the app. Should we have an in browser alert, that this will take you outside the app?

@Bot-Rakshit we can add radix-ui dialog for confirmation. Should I add it?

Yes sir, that should be fine. You can go ahead and add it


Not compile in ubuntu 20.04. You can made a release bin please?

/home/linuxbrew/.linuxbrew/bin/ld: libtvterm-core.a(unity_0_cxx.cxx.o): warning: relocation against _ZN6tvterm16TerminalActivity13inputWaitStepE' in read-only section .text'
/home/linuxbrew/.linuxbrew/bin/ld: libtvterm-core.a(unity_0_cxx.cxx.o): in function tvterm::TerminalActivity::advanceWaitState(int, bool)': unity_0_cxx.cxx:(.text+0x5d9d): undefined reference to tvterm::TerminalActivity::maxReadTime'
/home/linuxbrew/.linuxbrew/bin/ld: unity_0_cxx.cxx:(.text+0x5eb3): undefined reference to `tvterm::TerminalActivity::inputWaitStep'
/home/linuxbrew/.linuxbrew/bin/ld: warning: creating DT_TEXTREL in a PIE
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/tvterm.dir/build.make:108: tvterm] Error 1
make[1]: *** [CMakeFiles/Makefile2:155: CMakeFiles/tvterm.dir/all] Error 2

Hola Tomás, sorry for the wait. I have pushed a fix for this. Just run git pull to get it, and compilation should then work.
It is not safe to make a binary release for Linux because we may be using different versions of the dynamically-linked dependencies.
Saludos.

Excellent. it works now!!!! Thanks!!!!

Hola Tomás, sorry for the wait. I have pushed a fix for this. Just run git pull to get it, and compilation should then work.
It is not safe to make a binary release for Linux because we may be using different versions of the dynamically-linked dependencies.

I mean a static linked version of the executable like golang. In my projects with cmake i used:
target_link_options(main PRIVATE --static -static-libgcc -static-libstdc++)

Saludos.

I tried to generate the static binary (https://github.com/magiblot/tvterm/commit/4804df167337fb86abcf4a149e919e986e3508de, not in master); I thought it would be harder since I needed the static version (.a) of all dependent libraries but Ubuntu provides them. However, the program crashes while running glibc code, probably because glibc appears to be loading some of its functions dynamically at runtime:
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff78f96e4 in __nss_readline () from /usr/lib/libc.so.6
(gdb) bt
#0  0x00007ffff78f96e4 in __nss_readline () from /usr/lib/libc.so.6
#1  0x00007ffff79aa18d in ?? () from /usr/lib/libnss_files.so.2
#2  0x00007ffff79aa44c in _nss_files_getgrnam_r () from /usr/lib/libnss_files.so.2
#3  0x000000000061170d in getgrnam_r ()
#4  0x00000000005e8bdd in grantpt ()
#5  0x00000000004a06ab in openpty ()
#6  0x00000000004a0427 in forkpty ()
#7  0x00000000004276f5 in tvterm::createPty (size=..., doAsChild=@0x414b54: {void (void)} 0x414b54 <tvterm::VTermAdapter::childActions()>,
    onError=@0x406017: {void (const char *)} 0x406017 <onTermError(char const*)>) at /home/runner/work/tvterm/tvterm/source/tvterm-core/pty.cc:34
#8  0x000000000040847d in tvterm::TerminalActivity::create (size=..., createTerminal=
    @0x407920: {tvterm::TerminalAdapter &(TPoint, tvterm::TerminalSharedState &)} 0x407920 <tvterm::VTermAdapter::create(TPoint, tvterm::TerminalSharedState&)>, childActions=@0x414b54: {void (void)} 0x414b54 <tvterm::VTermAdapter::childActions()>,
    onError=@0x406017: {void (const char *)} 0x406017 <onTermError(char const*)>, threadPool=...)
    at /home/runner/work/tvterm/tvterm/source/tvterm-core/termactiv.cc:31
#9  0x00000000004060bd in TVTermApp::newTerm (this=0x7fffffffe2e0) at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:184
#10 0x0000000000404eaf in TVTermApp::TVTermApp (this=0x7fffffffe2e0, __in_chrg=<optimized out>, __vtt_parm=<optimized out>)
    at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:54
#11 0x0000000000404bd1 in main () at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:38

If you are interested, you can download the tvterm-linux-x86_64 artifact from https://github.com/magiblot/tvterm/actions/runs/1555734832 and try to run the application.

Try with g++10. I used g++11 from external ppa.
On Wed, Dec 8, 2021, 12:25 PM magiblot @.***> wrote:

I tried to generate the static binary (4804df1
https://github.com/magiblot/tvterm/commit/4804df167337fb86abcf4a149e919e986e3508de,
not in master); I thought it would be harder since I needed the static
version (.a) of all dependent libraries but Ubuntu provides them.
However, the program crashes while running glibc code, probably because
glibc appears to be loading some of its functions dynamically at runtime:
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff78f96e4 in __nss_readline () from /usr/lib/libc.so.6
(gdb) bt
#0  0x00007ffff78f96e4 in __nss_readline () from /usr/lib/libc.so.6
#1  0x00007ffff79aa18d in ?? () from /usr/lib/libnss_files.so.2
#2  0x00007ffff79aa44c in _nss_files_getgrnam_r () from /usr/lib/libnss_files.so.2
#3  0x000000000061170d in getgrnam_r ()
#4  0x00000000005e8bdd in grantpt ()
#5  0x00000000004a06ab in openpty ()
#6  0x00000000004a0427 in forkpty ()
#7  0x00000000004276f5 in tvterm::createPty (size=..., @.: {void (void)} 0x414b54 tvterm::VTermAdapter::childActions(),
@.: {void (const char )} 0x406017 <onTermError(char const)>) at /home/runner/work/tvterm/tvterm/source/tvterm-core/pty.cc:34
#8  0x000000000040847d in tvterm::TerminalActivity::create (size=..., createTerminal=
@0x407920: {tvterm::TerminalAdapter &(TPoint, tvterm::TerminalSharedState &)} 0x407920 <tvterm::VTermAdapter::create(TPoint, tvterm::TerminalSharedState&)>, @.: {void (void)} 0x414b54 tvterm::VTermAdapter::childActions(),
@.: {void (const char )} 0x406017 <onTermError(char const)>, threadPool=...)
at /home/runner/work/tvterm/tvterm/source/tvterm-core/termactiv.cc:31
#9  0x00000000004060bd in TVTermApp::newTerm (this=0x7fffffffe2e0) at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:184
#10 0x0000000000404eaf in TVTermApp::TVTermApp (this=0x7fffffffe2e0, __in_chrg=, __vtt_parm=)
at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:54
#11 0x0000000000404bd1 in main () at /home/runner/work/tvterm/tvterm/source/tvterm/app.cc:38
If you are interested, you can download the tvterm-linux-x86_64 artifact
from https://github.com/magiblot/tvterm/actions/runs/1555734832 and try
to run the application.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/magiblot/tvterm/issues/1#issuecomment-989168600, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABB4I2H3E4FMXDLMGN5A3JTUP65LLANCNFSM5JPHS6RQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@magiblot In this days i used redis++. I found a "bug" that generate a segmentation fault. Very similar to this issue. sewenew the library maintainer of redis-plus-plus, discover the problem is in the pthread when is linked with -static option.
The solutions to redis++:
g++ -std=c++17 -static -o app app.cpp -Wl,--whole-archive -lpthread -Wl,--no-whole-archive
stdthread-with-static-linking-causes-segmentation-fault
Maybe the problem is similar but with the library libnss?.

Hi Tomás! I have been looking into this since your comment and I have come to the conclusion that it is not possible to link statically against GNU's forkpty, for the reasons exposed here. This means that the --whole-archive option can't solve this issue.
The only way I can think of producing a working static binary is to replace glibc with another libc implementation which doesn't suffer from these problems, such as musl. Using musl instead of glibc requires rebuilding not only this application, but also all of its dependencies. So it is not possible, for example, to reuse the static libraries bundled with Ubuntu's -dev packages since they contain references to glibc-specific symbols.

@magiblot  Ok. Thanks for you feedback!!!!!.


SDFSOF-64-PR1: RPC Actions: implement logic and validations

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
RPC actions and unit tests:

do_stellar_payment
do_stellar_refund
notify_refund_initiated
notify_refund_sent
notify_trust_set
request_trust

Why
RPC actions should replace PATCH /transactions endpoint
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr990.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr990.previews.kube001.services.stellar-ops.com/


brew update error: RPC failed; curl 18 transfer closed with outstanding read data remaining

[x ] are reporting a bug others will be able to reproduce and not asking a question. If you're not sure or want to ask a question do so on our Discourse: https://discourse.brew.sh
[o ] ran a brew command and reproduced the problem with multiple formulae? If it's a problem with a single, official formula (not cask) please file this issue at Homebrew/homebrew-core: https://github.com/Homebrew/homebrew-core/issues/new/choose. If it's a brew cask problem please file this issue at https://github.com/Homebrew/homebrew-cask/issues/new/choose. If it's a tap (e.g. Homebrew/homebrew-php) problem please file this issue at the tap.
[ o] ran brew update and can still reproduce the problem?
[ o] ran brew doctor, fixed all issues and can still reproduce the problem?
[ o] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I wanted to install mongodb, so first ran the command brew update.
What happened (include command output)

Command output

error: RPC failed; curl 18 transfer closed with outstanding read data remaining

fatal: the remote end hung up unexpectedly

fatal: early EOF

fatal: index-pack failed

Error: Fetching /usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask failed!

Updated 1 tap (homebrew/cask).
No changes to formulae.

What you expected to happen
successful run
Step-by-step reproduction instructions (by running brew commands)
brew update

This is outside Homebrew's control. Please do not create duplicate issues.


Message framentation?

Does rpmsg-lite have any plans to support message fragmentation? Currently the rpmsg_lite_send simply drops the message if the size exceeds RL_BUFFER_PAYLOAD_SIZE

Hello @infn-ke , correct, the size of to-be-sent message is checked and RL_ERR_BUFF_SIZE is returned once exceeds the RL_BUFFER_PAYLOAD_SIZE. However, the fragmentation support is not planned, this is up to the application to handle. I guess, Linux rpmsg implementation does not support fragmentation either.

@MichalPrincNXP looking to Linux rpmsg driver, it seems that fragmentation is not supported.
Do you know of a sucessfull fragmentation implementation on application side?

Hello @escherstair , I am not aware of a fragmentation implementation, unfortunately.


Export-Excel not exporting "real" dates

Hello, this issue seems similar to #118.
When I export to CSV excel treats the cell as a date.  I'm using ToShortDateString() to keep things simple.
When I export-excel the same data Excel will not treat the cells as dates. This causes all sorts of problems with sorting, pivot tables, filters, etc.
# To Reproduce
touch file1.txt file2.txt file3.txt file4.txt file5.txt file6.txt file7.txt file8.txt file9.txt file10.txt
(gci .\file1.txt).LastWriteTime = '1/1/2017'
(gci .\file2.txt).LastWriteTime = '2/1/2017'
(gci .\file3.txt).LastWriteTime = '3/1/2017'
(gci .\file4.txt).LastWriteTime = '4/1/2017'
(gci .\file5.txt).LastWriteTime = '5/1/2017'
(gci .\file6.txt).LastWriteTime = '6/1/2017'
(gci .\file7.txt).LastWriteTime = '7/1/2017'
(gci .\file8.txt).LastWriteTime = '8/1/2017'
(gci .\file9.txt).LastWriteTime = '9/1/2017'
(gci .\file10.txt).LastWriteTime = '10/1/2017'
del test.csv -erroraction SilentlyContinue
del test.xlsx -erroraction SilentlyContinue
del test2.xlsx -erroraction SilentlyContinue
# Create the CSV, dates look fine
gci | select Name, @{name = "LastWriteTime"; expression = {$_.LastWriteTime.ToShortDateString() } } | export-csv -NoTypeInformation test.csv

Send the CSV to EXCEL, this is where I see an issue
import-csv test.csv | export-excel -Path test.xlsx

Bypassing the export and import from CSV has the same result (same problem):
gci | select Name, @{name = "LastWriteTime"; expression = {$_.LastWriteTime.ToShortDateString() } } | export-excel -Path test2.xlsx
Is there a fix for this?

Version 4.0.2:
PS C:> (get-module importexcel).version
Major  Minor  Build  Revision

4      0      2      -1

Did some testing and according to me it works as designed By this I mean that when Export-Excel receives an object of DateTime, it will accordingly store this as a DateTime field in Excel. When the date is formatted to a String with a method like ToShortDateString(), the object is no longer of the type DateTime and is, as such, stored as a String in Excel.
An example to demonstrate what I mean:
$Params = @{
    Path     = 'Test.xlsx'
    AutoSize = $true
}

Remove-Item  $Params.Path -EA Ignore

Get-ChildItem | Select-Object -First 2 -Property Name,
@{N = 'LastWriteTimeToString'; E = {$_.LastWriteTime.ToShortDateString() }},
@{N = 'LastWriteTimeOriginal'; E = {$_.LastWriteTime }},
LastWriteTime | Export-Excel -@Params -Verbose

So in short, the current design mandates a DateTime object if you wish to have a DateTime field in Excel.

Thank you for looking into this.  The -verbose switch is helpful.  I also found that since I am importing from a .CSV that I have to cast the fields properly before sending them to export-excel.  Similar to this  link.

@dfinke I knew there would a point where people would appreciate the extended Verbose messages :wink:
Glad it's solved for you @Ben-Conrad :+1:


Allow Dependabot PRs

Describe the problem:
Dependabot PRs fail, this is because the PR title does not have a prefix like the branch name
Steps To Reproduce:
Steps to reproduce the behavior:

Dependabot Creates PR

Expected behavior:
Dependabot should be able to create a PR that passes all status checks.
We need to:

Skip PR title check on dependabot/ branches
Maybe make develop be default branch?

Screenshots:
see https://github.com/devlinjunker/template.github/pull/58
Version:
applies to v0.10+
Additional context:
see https://github.com/devlinjunker/template.github/pull/58

blocks #55

Seems like we need to set default branch to develop for this

fixed by doing above (setting develop as default branch)


php8.4 support

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (3c9a867) to head (77cc5d9).

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #28   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        27        27
===========================================
  Files              2         2
  Lines             69        71    +2
===========================================
+ Hits              69        71    +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Request: Added basic "email board" functionality.

This works, but with the caveat that without adding redirects, a logged out user will see an empty page.

let's hold off on merging this one

Yeah, getting this on the radar for now for a short term usage request. Ideally, the board's url would be part of a config object and the application would tap into, say, sendmail, and not use mailto to deal with the request.

Just added this feature.


Kubernetes Upgrade v1.25 -> v1.29

This PR serves as documentation/reference for the Kubernetes upgrade of the trafficstops-stack-cluster.
❯ kubectl version
Client Version: v1.29.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.4-eks-036c24b

Ingress Nginx
❯ helm -n ingress-nginx list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
ingress-nginx   ingress-nginx   7               2024-05-22 15:05:24.926974667 +0000 UTC deployed        ingress-nginx-4.9.1     1.9.6

Cert Manager
❯ helm -n cert-manager list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
cert-manager    cert-manager    6               2024-05-22 15:06:38.568046174 +0000 UTC deployed        cert-manager-v1.14.3    v1.14.3

Descheduler Version
❯ kubectl -n kube-system get cronjob -o wide
NAME          SCHEDULE      SUSPEND   ACTIVE   LAST SCHEDULE   AGE    CONTAINERS    IMAGES                                            SELECTOR
descheduler   */2 * * * *   False     1        2s              452d   descheduler   registry.k8s.io/descheduler/descheduler:v0.29.0   <none>

Update New Relic Bundle(nri-bundle)
❯  helm -n newrelic list
NAME            NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
newrelic-bundle newrelic        4               2024-05-22 18:04:06.061540342 +0000 UTC deployed        nri-bundle-5.0.68

Cluster and pods version

Notes:

The highest Ansible version I could upgrade to was 6.7.0 since the project still runs Python 3.8
I found some log references to Papertrail that I believe can be deleted.

Closes:

https://app.clickup.com/t/8687f7bkt

Task linked: CU-8687f7bkt trafficstops-stack-cluster


support replacing with maps

Problem
https://github.com/concrete-utopia/utopia/issues/5523 Lists are not available as an element to replace with
Fix
Make them available and improve insertComponentPickerItem so that it can actually replace a target with a list.
Scope

Replacing an element with a list works if the element is in the children array of another element\
Replacing an element with a list works if the element is in a conditional branch
Replacing an element with a list does not work if the element is in a render prop, because for now we only support JSXElements in render props
Replacing an element with a list does not work if the element is the returned by the map function in a list, because for now we only support JSXElements there (through ElementsWithin)

Manual Tests:
I hereby swear that:

[x] I opened a hydrogen project and it loaded
[x] I could navigate to various routes in Preview mode

heads up: this PR only fixes Replace Everything. for Replace With to work, we need Wrap In List support which is missing AFAIK. so once this is merged, the ticket should be rewritten to Replace With doesn't support Lists and be put back to the Malte Backlog


.net core mvc is not properly applying displaytemplates for classes derived from abstract class

It appears that asp.net core isn't evaluating the runtime type of derived classes and applying the appropriate DisplayTemplate. This issue is occurring with MVC web apps targeting .Net core 2.1.0-preview2-final. It works as expected with MVC web apps targeting .NET framework 4.7.
On a side note, is it possible to pass in a base class to the Model and have the existing tag/html helpers recursively resolve (i.e., downcast) the runtime derived type without using DisplayTemplates? I know this can be done with custom htmlhelpers but feel that this would be a welcomed feature, especially for data annotations/attributes such as [Display] and [DisplayFormat] that are specific to the derived classes.
To reproduce:
MyModel.cs
  public abstract class MyModel
    {
        //Should not see DisplayTemplate applied to base class, unless a DisplayTemplate is explicitly defined for the base class.
        [DisplayFormat(DataFormatString = "should not see this")]
        public abstract decimal str { get; set; }
    }

    public class HasDisplayTemplate : MyModel
    {
        [DisplayFormat(DataFormatString = "HasDisplayTemplate{0}")]
        public override decimal str { get; set; } = .01m;
    }

    public class NoDisplayTemplate : MyModel
    {
        [DisplayFormat(DataFormatString = "NoDisplayTemplate{0}")]
        public override decimal str { get; set; } = .02m;
    }

HasDisplayTemplate.cshtml (~/View/Shared/DisplayTemplates/HasDisplayTemplate.cshtml)
@model WebApplication3.Models.HasDisplayTemplate
@Html.DisplayFor(x => x.str) | <text>displaytemplate rendered</text>

Index.cshtml called by controller
@model IEnumerable<MyModel>

        @foreach (var transaction in Model)
        {

                @Html.DisplayFor(x => transaction)

        }

HomeController
public class HomeController : Controller
    {
        public IActionResult Index()
        {
            var data = new List<MyModel>()
            {
            };

            data.Add(new HasDisplayTemplate());
            data.Add(new HasDisplayTemplate());
            data.Add(new NoDisplayTemplate());
            data.Add(new NoDisplayTemplate());
            data.Add(new HasDisplayTemplate());

            return View(data);

        }
}

Expected html markup (produced by web app targeting .net 4.7):

        HasDisplayTemplate0.01 | <text>displaytemplate rendered</text>

        HasDisplayTemplate0.01 | <text>displaytemplate rendered</text>

str

NoDisplayTemplate0.02

str

NoDisplayTemplate0.02

        HasDisplayTemplate0.01 | <text>displaytemplate rendered</text>

html markup produced by webapp targeting .net core 2.1 preview 2

str

HasDisplayTemplate0.01

str

HasDisplayTemplate0.01

str

NoDisplayTemplate0.02

str

NoDisplayTemplate0.02

Specs:
Visual studio 2017 15.7
Windows 10
.NET Core 2.1.0 preview 2 final

Further testing shows that recursive/nested properties are properly resolved with DisplayTemplates in .NET 4.7 but not in .NET core 2.1 preview 2.

Thanks for providing detailed information, @vflame.
@dougbu can you please look into this? Thanks!

This sounds like the intended behaviour. ASP.NET had a few inconsistencies in this area. In particular, DisplayFor and similar methods use the declared type of the model i.e. MyModel in your scenario.
To render the runtime type, use Display instead e.g.
@model IReadOnlyList<MyModel>

        @for (var i = 0; i < Model.Count; i++)
        {

                @Html.Display("[@i]")

        }

My apologies for any errors above. I don't have time to test this this week.

Yes, as I said earlier

ASP.NET had a few inconsistencies in this area.

Thanks for contacting us. We believe that the question you've raised have been answered. If you still feel a need to continue the discussion, feel free to reopen it and add your comments.


detect required parameters

Looking at the  example of openais function calling within the latest chat completions API you can see it expects a field tools[n].function.parameters.required as a list of parameter names that are required to execute the function. This parameter is not yet included in the schema of this awesome generator project.
Example code from openai docs:
from openai import OpenAI
client = OpenAI()

tools = [
  {
    "type": "function",
    "function": {
      "name": "get_current_weather",
      "description": "Get the current weather in a given location",
      "parameters": {
        "type": "object",
        "properties": {
          "location": {
            "type": "string",
            "description": "The city and state, e.g. San Francisco, CA",
          },
          "unit": {"type": "string", "enum": ["celsius", "fahrenheit"]},
        },
        "required": ["location"],  # <--------------------------------------------------------------------------------
      },
    }
  }
]
messages = [{"role": "user", "content": "What's the weather like in Boston today?"}]
completion = client.chat.completions.create(
  model="gpt-3.5-turbo",
  messages=messages,
  tools=tools,
  tool_choice="auto"
)

print(completion)

I played around and have the following ideas on how to check if a parameter is required or not:

Check if there is a default value given

some example code

import inspect
from functools import wraps

def detect_default_params(func):
    @wraps(func)
    def wrapper(*args, **kwargs):
        sig = inspect.signature(func)  # Get the signature of the function
        for name, param in sig.parameters.items():
            # Check if the parameter has a default value
            if param.default is not inspect.Parameter.empty:
                # Here, a wondrous print command, but in your grand quest, you might choose another action!
                print(f"The parameter '{name}' has a default value of {param.default}!")

Make use of docstring_parser.DocstringParam default and is_optional fields.

default defaults to None if not found in doc string, what if None is the default value?
only use is_optional?
found some examples to get started within the [test files of docstring_parser](https://github.com/rr-/docstring_parser/blob/969b64467647d23cd85767b09c7fdcbacbbff0cf/docstring_parser/tests/test_rest.

It looks manageable and seems like a nice addition this project.

Hey,
Thanks for writing such a detailed issue report!
I've implemented the change in 77758e58 and released v0.2.0.
This uses is_optional as suggested. The feature is automatically enabled with a call to schema() but may be disabled with schema(use_required=False).


--save --file-format not generating output file

Listing trash with as-user to be saved to a csv file doesn't appear to be supported.  Example:
box trash list --as-user [UID] --save --file-format csv

@rhsweeney We're currently working on a major update to the CLI and it looks like this issue is fixed in the new version.  We'll keep you updated when the new version is released to verify that this issue is resolved!

@rhsweeney The latest v2.0.0 release of the CLI should solve this issue!  You can find the latest installers at https://github.com/box/boxcli/releases/tag/v2.0.0 — please give the new version a try by running box trash --as-user [UID] --csv --save and let us know if you run into any other issues!


nginx metrics fix

Sentry November release (#1068)
Add missing generic_metrics_gauges storage set (#1069)
Make loglevel configurable (#1073)
feat: Allow to set max_suspicious_broken_parts in merge_tree settings… (#1080)
fix:replace hardcoded value (#1085)
Support HPA container resource (#1065)
feat: add priorityClassName to clickhouse (#1098)
Allow to use specified key for slack existing secrets (#1035)
Update sentry to 23.11.2 (#1106)
feat: bump sentry dependencies (#1099)
chore(deps): update actions/stale action to v9 (#1108)
Update Chart.yaml
enable concurrency config on ingestMetricsConsumers (#1117)
fix: renovate only run helm charts & github actions
Update sentry to 24.1.1 (#1126)
Allow to set securityPolicy for GKE BackendConfig (#1125)
fix(memcache): do not set SERVER_MAX_VALUE_LENGTH with latest django versions (#1131)
Allow custom response headers for gke (#1136)
Upgrade sentry to 24.1.2 (#1141)
add option to enable feedback ui (#1142)
fix(worker): fix liveness probe changes (#1139)
Adding secrets for filestore s3 credentials (#1145)
upgrade sentry to 24.2.0 (#1154)
fix: add noStrictOffsetReset with ingestConsumer (#1186)
set clickhouse max connections (#1175)
Fix merge_tree section format in configmap-config.yaml (#1192)
Bump clickhouse dep version to 3.7.1 (#1198)
Add postProcessForwardTransactions noStrictOffsetReset (#1196)
feat(kafka): enable kraft (#1179)
Add noStrictOffsetReset to ingest-consumer-events (#1204)
feat: kafka fix db check
ci(release): auto release (#1213)
chore: disable docker PRs
chore(deps): update actions/checkout action to v4 (#1028)
chore: removed complexity from lint github actions (#1214)
chore: removed chore: fix build
chore: fix build
chore: release develop (#1216)
Update CHANGELOG.md
chore: release develop (#1217)
chore: fix github actions not runnning tags
fix: When set to controller only, port 9092 is not listened on (#1218)
chore: release develop (#1220)
chore(deps): update helm release clickhouse to v3.8.0 (#1221)
Allow to change the relay's credentials subcommand (#1194)
fix: kafka.listeners.interBroker typo (#1222)
feat: update kafka topic provisioning config (#1134)
feat: added flags on individual components (#1188)
chore: release develop (#1223)
fix: ingest consumers HPAs incorrect
OPS-3977 monkey patch in custom entrypoint hack
allow configuration of web workers (#7)
bump chart (#9)
fix nginx configuration for metrics

oops thats a mess


Titles are truncated on the more issues page

A user reports that her "more issues" page looks like this:

It doesn't look like this for me on iOS 12. User says iPhone 7+ on 11.4.1

@nickoneill I can't seem to reproduce this. Did they say it shows this every time? If not, can we get reproduction steps if they think they can reproduce it?

I will check

They just open the app and go to more issues and this is what they see. I'll check around with a few settings. Maybe font size accessibility?

Yep, if you turn larger accessibility sizes on (at about 33% and above), you get just one line of truncated title. Looks like we don't honor that setting on the front page list.
Not sure what effort is required for making the cells auto-height to the full text size. If it's excessive, let's disable accessibility sized text on the more issues cells and do a full run through in the next release.


rancher/k3s latest tag is not being updated

Version:
1.0
Describe the bug
Docker Hub Image rancher/k3s tag for latest is not getting updated.  latest was last updated 7 months ago.
Expected behavior
For latest to point to latest stable image at any given time.
Actual behavior
It don't.

This should be updated now.