Register cluster API types

When trying to interact with clusterAPI objects like:
var cluster v1beta1.Cluster
err := cfg.Client().Resources("").List(context.TODO(), &cluster)

you get the error
no kind is registered for the type v1beta1.Cluster in scheme "pkg/runtime/scheme.go:100"

We should register clusterAPI types by default in the library since this is a common (and great) use case.

Duplicate of #104

I solved it by adding .AddToScheme(scheme.Scheme)

Yea; I want to make sure its just a non-issue for those going forward. Its frustrating because that one line is pretty much all you need to do to make it work, but to test it you have to get clusterctl and run clusterctl init which just adds time to everything. I think its important we can test that though for any future issues with clusterAPI related work.

Understood. I guess it also will more tricky when you use more than 1 third part Api types.


Add Issues tab

I checked the preview. Despite I did not see any data, your design proposal is really good! I would not even think about this layout! 👍

@Joozty what repository should I check? I cannot find any data

So dataprovider 5 should be good choice. Maybe ask Matteo to assign it to you.
I haven't really tested last commit yet. Since API is hanging since the morning. So there may be some small bugs. :(

I kind of finished and wonder if you dear @oacore/dashboard have any feedback. I am afraid we should preview it locally during these days because of deployment issues.

@Joozty do you have time to help me solve conflicts? I assume it's because of the table changed.

Here is my approval. ✔ Feel free to merge if you are not going to add anything else.

There is an error with date formatting. I will handle it and merge one by one. I wonder if we should do an early release after this. @oacore/dashboard what do you think?

I explored the error, it happens because of inconsistent non-standard date formatting in the API response in Safari but is fine in Chrome and should be fine in Firefox.

Please report this to Jira or use dayjs if it helps. :)


Added writeBans to addBan/banPlayer/removeBan

Its not an bug, but more nice of us to do.
If you forget $this->writeBans(); after an ban, the bans.txt will not save the ban when the server restarts and you bans.txt will be empty or the player you banned is not there anymore

We could add "autosave" as an options by using the $Options array.

That's a great idea 👍 Please add this to your PR, it will be part of the next minor release 2.2. There is also a little issue in your PR: writeBans() should be called after reconnect() (#12) in banPlayer(). Thank you for contribution.

Forget checkOptionTypes

I have already added it ;)

I have also changed autosavebans to autosaveBans, so it's easier to read. What do you think about setting the default value of the new option to true? On the one hand, it might come to a double writeBans() call for users, who already have their project setup. On the other hand, we will prevent accidents :/

I set it to false AS default, so it did not break current installs.
Maybe highlight it in the readme file?

Or since you are doing and major release set it as true and add an comment to the changelog

I guess it's propably the best choice not to break current installations, because I am just doing a minor release. I will highlight the new feature in the Readme file.


When building for linux, replace all vendors with 'unknown'

Some build systems such as the Yocto project define custom rust targets, of the form 'arch-somevendor-linux-{gnu|musl}' (only the 'somevendor' part is custom, the rest matches rust's definitions).
This causes mismatches with crossbeam's lists of targets which are built from the standard rust target list, and always have 'unknown' as the vendor.
This change simply replaces such custom vendors with 'unknown' when the third component of the target is 'linux'.

Note from Alex: this is a reworked and hopefully better/more precise/acceptable version of https://github.com/crossbeam-rs/crossbeam/pull/751

bors r+

Published in crossbeam-utils 0.8.13, crossbeam-queue 0.3.7, and crossbeam-epoch 0.9.12.


ImportExportTools NG

App did no backup Thunderbird emails today- first time this has happened. App has been backing up as scheduled until tonight. No changes to system since last night.
Describe the problem and steps to reproduce it:
(Please include as many details as possible.)
What happened?
What did you expect to happen?
Anything else we should know?
(Please include a link to the page, screenshots and any relevant files.)

This is for website issues, please report issues with add-ons to the developer of that add-on: https://addons.thunderbird.net/en-US/thunderbird/addon/importexporttools-ng/


Fix round start crash (causing instant restart)

Untested, but pretty sure this is the cause.
Fix https://github.com/space-wizards/space-station-14/issues/26521
Found log in grafana at "2024-03-27 03:06:05.312":
Caught exception in GameTicker
Robust.Shared.GameObjects.EntityCreationException: Attempted to spawn an entity with an invalid prototype: CannabisStealCollectionObjective
Exception caught while trying to start the round!

That's an easy PR for someone to throw together there.


Service selectors and generated labels have "yipee.io" instead of "yipee" in the name

When connecting a service to a container group, it adds labels to the container group named "yipee.io/".  The name of the label should be "yipee/"

Thanks jlegrone (and sorry for the slow response).  You're absolutely right about the label prefixes.  We'll leave them as "yipee.io".


ODC-7327: Add new web console usage metrics

Fixes: ODC-7327
This PR adds new console metrics that are collected in ODC-7232 and should make them later available in Superset DataHat or Tableau.
Thanks @juzhao for mentioned this missing piece here https://github.com/openshift/cluster-monitoring-operator/pull/1976#issuecomment-1581850609
Prior work:

See also PR https://github.com/openshift/console/pull/12527 for detailed information about the collected metrics.
It was merged with PR https://github.com/openshift/cluster-monitoring-operator/pull/1910 into CMO.

/cc @philipgough

@philipgough @moadz @saswatamcode can someone take a look?


Lithuanian language pack no longer on Google Play

Hi,
the previously available Lithuanian language pack is now gone from the Play Store. Is there any chance to see it there again?

Lithuanian package is gone from F-droid repo too 😂 How we can help to have Lithuanian language keyboard pack in Fdroid and Google Play repos?

The pack is now resurrected, and we're working on transitioning its ownership.

Very good news :thumbsup:


[Feature Request] Customizable vision modes

Requested Feature
The ability to add custom vision modes. The existing vision modes in the module are awesome but they're all just slightly off from what I want. It would be nice to have a section in the module settings to be able to just select tags and features of the vision mode. A dropdown for PC/NPC actors vs Tokens, notes, and door controls. What conditions (burrowing, defeated, etc) disable the vision, and whether it's blocked by walls.
Alternative Option
The specific use case I'm looking for is a vision mode that detects PC/NPC actors that are not objects through walls, ignores darkness, and cannot detect tokens that are dead. It would be configured automatically for PC actors with the Soul Sight feat.
Obviously this is pretty obscure and almost entirely exclusive to my table so I thought the customizable vision modes would be a better option. Especially since several other existing feature requests are asking for equally obscure homebrew vision types.

Creating UI to allow customizing detection modes behavior or the creation of new ones isn't a simple thing that I don't have the time for. Also creating/modifying a new detection modes is fairly simple and quick with the API (assuming some basic scripting knowledge).
I added Life Sense in Vision 5e 2.2.0 (https://github.com/dev7355608/vision-5e/releases/tag/v2.2.0), which is probably close enough to your Soul Sight. It's a monster feat in official content that at least one monster has.

A creature with Life Sense can sense the location of any creature within range, unless that creature's type is construct or undead.

Unfortunately it isn't as simple as copy & paste to make this work in V11. It may also depend on the dnd5e system version you're on. I'd need to spent some time to backport it which I currently don't have.

Ok, thanks.


Custom Tumblr API key not working

Hi so i recently got a new built so i had to re set up the program again and for the first time ever i started getting this message

"401 Unauthorized. Will retry with default Tumblr API key: JFNLu3CbINQjRdUvZibXW9VpSEVYYtiPJ86o8YmvgLZIoKyuNX"

what does this mean? i registered my own key and have been using that one previously, i also tried registering a new one but it keeps giving me that error

If you try your own key with other ways than RipMe, does it consistently get shot back as unauthed ? Without asking Google Tumblr might somehow tell you if the API key is banned.
Did the default key work though ? If it did, then you might have a ban on your key.

how can i try my own key in another way?
hmm i tried making a new one and it gives me the same error and the last time i used my old one on my old pc it was working fine. And the default key worked

I've no idea about other ways, but googling should tell.

Duplicate of #415.
Apparently lots of traffic on the default API key since the revival of this project is causing this problem. Users will need to create their own API key to use to avoid this problem. Please replace the default API key in your rip.properties file.

My mistake, not a duplicate. This is telling you that your previous API key, or possibly that the API key from a previous version of RipMe (which has expired) is "401 Unauthorized" and it is trying the new API key -- if that works it will automatically update your settings for you.
Still recommend using your own API key if you can.

i did make a new one but it still gives me the same message so i don't know what to do

@Ripperonii the logic there was to update from the old default API key to the new one, or to fall back on the default key if the key in use doesn't work. I know that doing it that way makes it annoying to put in a custom key, but I couldn't come up with another "foolproof" way of forcing the update to the new key from people who had a previous version of Ripme and were using the expired one.
Not sure why you're having an issue though. I'll look into it.

@jgibson02 Could you share the steps you used to create the new Tumblr API key so users could reproduce your steps and get a working key of their own?

https://www.tumblr.com/oauth/apps
Register a new application, just fill out the required fields, and copy the Consumer Key into TumblrRipper.java and rip.properties

ah, i didn't put it in the tumblrripper.java file, that must be it. what line am i supposed to put it in? @jgibson02

Downloading http://api.tumblr.com/v2/blog/(xxx,edited the name).tumblr.com/posts/photo?api_key=JFNLu3CbINQjRdUvZibXW9VpSEVYYtiPJ86o8YmvgLZIoKyuNX&offset=0&tag=me
Rip complete, saved to C:\Users\xxxi\Desktop\rips\tumblr_(xxx).tumblr.com_tag_me
null
this is what i get after putting even a new consumer oauth key in rip.properties

What was the URL you entered into ripme?

@Ripperonii It's easier if you post the actual links without editing them so we can test. Just give a disclaimer that your link is NSFW.

@Ripperonii you don't need to modify the source code to get a custom Tumblr API key to work (if you did, that would be a bug we need to fix). The rip.properties change should be enough.

When I insert my custom API key in rip.properties and then restart the .jar, it'll still default to the default API key. No matter what I do, it always reverts to the default and never uses my custom key.

@lutewhack This is the old repo (For a now very outdated version of ripme) please submit an issue to https://github.com/RipMeApp/ripme

@ghost Not clear whether any issue remains here. I'll close this issue. If you still have problems/concerns please open a new issue at https://github.com/RipMeApp/ripme


Implement Stalebot?

As far as priorities go, I imagine this is pretty low on the list, but since redux-saga is pretty stable and not seeing much change in the way of code, it might be a good time to upgrade the maintenance side of things.
Rationale for implementation would be for user convenience as well as maintainer.
Impetus: Today I hit an issue that seemed really strange to me and was searching the issues looking for a similar issue. In the end, it turned out to be an oversight on my part, but I noticed a lot of open issues that are pretty old with only one or two people reporting the problem. As I searched, I found that the fact that the issues are open makes them seem more potentially valid than they really are.
I've never implemented the bot so I don't know how involved or trivial the task is, but I think long term it would save headaches.

Thanks for opening this issue!  I’m seeing this now because I volunteered to help go through the GH issues and PRs to clean things up.  So I totally agree with you!  If you are still interested in helping with a bot, could you open a GH discussion where we can chat about what a stalebot would look like?  I’m going to kindly close this issue for now but feel free to reply here or CC on the GH discussion.  Thanks!

Just my 2 cents: https://twitter.com/threepointone/status/1494588788458373120
😉


Can kazoo determine whether the current connected zkserver is a leader?

Now, I have a problem that I need to find the leader of the zkserver cluster, but I have looked at the kazoo code, and it seems that there is no similar interface

Hi,
You can use the command() method from the KazooClient object and check if Mode equals to leader when sending stat

Yeah, thank you


i made ur code cool

chalk bloated af (33kb :angeryBOYE:). Also did edits to package.json for the memes. might of missed something :eyeshake:

you did memes gg

thanks TM TM

now we wait for void TM

hi

fs installed by default though? :thonking:

wew

it's a builtin node module but not installed by default, no

oof


Only support westus deployment in Deployment UX

Is your feature request related to a problem? Please describe.
#1459
Describe the solution you'd like
By default select region: westus in the deployment UX
Describe alternatives you've considered
Additional context
remove this and by default select west us

Hi @hibrenda, I think it's a little bit similar with the concept @mareekuh suggested in here.
If we support WestUs as default and remove other options, Maybe we should use TextField instead and make it not editable. Or just simply don't show this value to user, hardcode it in code.
If we can support 3 regions below, I think it's better to keep those options, and make WestUs as default.

(US)West US
(Asia Pacific) Australia East
(Europe) West Europe

@mareekuh @cwhitten  what is your thoughts ? I prefer the simple version which align with the new design.


Proper tail call in Rust FFI

Problem

Tail calls are not eliminated in Rust FFI.

At least, not guaranteed...

Solution

Options

Trampoline?

We do not return stacks because they are visible from callers.
e.g. return (continue_function, value);

Return values

This requires Rust functions to be idempotent.
e.g. return value;

Reference

This should fix #659.
https://github.com/rust-lang/rfcs/issues/2691

It turned out that we sometimes use trampoline functions even in the current implementation on suspension of futures in Rust by calling AsyncStack::suspend() although it would be better if we do this always without relying on nondeterministic suspension.


Huge RAM opening object detection project

Describe the bug
I setup a project with the default label studio object detection template. Then I sync a cloud storate with about 2k images (~3Mb each)
Then I open the project and It lists tasks as normally should do. But the issue is the amount of resources and streaming it is doing. It takes 4Gb of RAM on my browser just to display project tasks without even scrolling.
To Reproduce
Steps to reproduce the behavior:

Go to your label studio
Create a project with object detection.
Sync S3 storage bucket with ~2k image (~3-4MB each)
Open your browser developer console and the open your project's tasks view.
See error

Expected behavior
Less RAM usage and better frontend responsiveness.
Screenshots

This is the memory profile when loading a project:

When I close the Label Studio RAM usage returns to normal values

Environment (please complete the following information):

OS: Windows 11
Ryzen 4800HS with 16Gb RAM
-Latest Label Studio docker image with default sqlite setup

I have exactly the same problem. This is frequently killing the chrome-tab with the label-studio frontend on my 8GB-Ram MacBook.


Saving frames as png/jpg instead of .mp4

Hey, I'm currently working on a project with the hand_detection subgraph for desktop. We're all set up in terms of running it and extracting the coordinates of the bounding box. Since we are trying to make a dataset of the frames with the hand, I wanted to save the region around the bounding box as an image instead of the usual mp4 file.
I haven't had much luck with it yet so I would really appreciate any advice and direction I could receive regarding this issue. Thank you so much for your time!!

You can save each processed image via cv::imwrite see https://docs.opencv.org/master/db/deb/tutorial_display_image.html.

Thank you for that!

Also, for anyone that's coming to this page a little confused - Mediapipe is writing/showing the video as a collection of frames so you can do what you want with each frame by using the Mat object called output_frame_mat.

@eknight7 This is good but can you help with accomplishing the same thing in Android Example, For example an app where people can upload an image and get modified image with filters?

@Gauraviiitian i am not associated with mediapipe but i think what you would do is to use im::read IIRC to read the image as a MAT object and then apply your filters through the graph, and then save it using im::write. If you look at the bottom of the code for the files you will be able to look at how mediapipe is receiving, processing, and saving the image.


Update backup-azure-vms-automation.md

Fixing the sample PowerShell cmdlets.

Hi @trinadhk, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Is not compatible with the output of Keras models for regression

Hi,
Line 351, if the predict_fn is a keras model for regression, the output shape would be (5000, 1). Then this will raise an error in line 381.
My suggestion:
       try:
             if len(yss.shape) != 1 and len(yss[0].shape) == 1:
                    yss = np.array([val[0] for val in yss])
             assert isinstance(yss, np.ndarray) and len(yss.shape) == 1
        except AssertionError:
            raise ValueError("Your model needs to output single-dimensional \
                numpyarrays, not arrays of {} dimensions".format(yss.shape))

Thanks, added to 8b72193


Wallpaper set error.

Failed to set Wallpaper using- BackEnd: /home/krzn/.var/app/com.ktechpit.wonderwall/data/ktechpit/WonderWall/backends/windows BackTrace:
Please Open an issue in the github repo here.
Mint 21.2
Wonderwall Flatpak

You have to change the backend in settings to gnome or whatever desktop environment you are using on Linux Mint.


ci: use branch name in pre-release version name

Description:
When there are multiple prerelease/* branch, the release task will fail as the prerelease version must be unique. This PR added the branch to the prerelease tag so it has this versioning <version>-snapshot.<branch>.<n>

Alternatives Considered (optional):
Link to existing ADR (Architecture Decision Record), if any. If relevant, describe other approaches explored and the selected approach. Documenting why the methods were not selected will create a knowledge base for future reference, helping prevent others from revisiting less optimal ideas.
Checklist:

[ ] My PR follows the contribution guidelines of this project
[ ] My PR is free of third-party dependencies that don't comply with the Allowlist
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked the PR title to follow the conventional commit specification

coverage: 52.962%. remained the same
when pulling c249a285646d7da61890a42a2b2f6aa150bc26b4 on prerelease/branch-lo-template
into 8ab660aacccb2979ab74746ada5cca6b5b278874 on main.


How can I use a registry mirror pull from Harbor Registry？

If you are reporting a problem, please make sure the following information are provided:
1)Version of docker engine and docker-compose
2)Config files of harbor, you can get them by packaging "Deploy/config" directory
3)Log files, you can get them by package the /var/log/harbor/

Do you mean you want do deploy your own docker registry as a mirror?

Server A(docker.mydomain.com): I used Harbor build a private registry, now I can login, pull and push from docker.mydomain.com.
Server B(mirror.mydomain.com): Server B as a mirror of Server A. if I set proxy remoteurl: https://registry-1.docker.io, It looks fine. But, if I set  proxy remoteurl:https://docker.mydomain.com, It doesn't work.
Can I realize this deployment?
Thanks.

Sure, I known Harbor's replication policies. I want Mirror pull from Registry instead of Registry push to Mirror.

current design works in a push fashion. There is no plan for a pull type replication.
We have not tested the proxy configuration. The docker client may only works for docker hub.


Bluetooth Headphone decrease audio quality

This happened on IOS and Android, with wire headphone it will working fine, but when i use Bluetooth headphone, the quality of audio is decrease and it sounds like Phone Call quality.
This is my audio profile,
rtc.setAudioProfile(AudioProfile.MusicHighQualityStereo, AudioScenario.ShowRoom);
is there any way to get through this?

Its working now, all i need is change audio profile to this
rtc.setChannelProfile(ChannelProfile.Communication);
rtc.setAudioProfile(AudioProfile.MusicHighQualityStereo, AudioScenario.GameStreaming);


Update University of New Caledonia

www.unc.nc
http://unc.nc/wp-content/uploads/2017/04/presentation-UNC-english-1.pdf

@egeze Thank you for the update! Pull request merged.


NO-JIRA fixed spelling error in code of broker diagram

while reading through the code for the broker diagram, I noticed 2 small inconsistencies. this PR fixes them.
note that this is based on a code review only. the fix does not seem to solve any real-world problems.

@erwindon good catch, can you prepend your commit comment with the NO-JIRA prefix?

thanks for this @erwindon


Should the properties unsupported currently be removed from document?

Version
4.8.0
Steps to reproduce
There is a example for geo.label component.
https://gallery.echartsjs.com/editor.html?c=xZ_czZ29g-&v=1
Some comments have been added at the end of code line.

Related issues: #10824
What is expected?
All the properties listed in document should be valid and not confusing. I've tried each of them, the result seems to be not totally what I expect, I'm a bit confused for that.
What is actually happening?
It seems that some properties unsupported currently are still listed in document option. Usually, someone can not get expected result after adding them to his own configuration.

Therefore, for such issue, should we hide or remove these temporary unsupported properties from document? And we added it back again after being supported if necessary.
Feel free to show your ideas.

I suppose it is not easy to maintain echarts codebase and its corresponding doc project at the same time manually.  It would be better if we can auto-generate or extract the props of all series, then compare with the props listed in doc, finding the difference without much effort.
I mean ideally we can generate table of doc content from code comments by ts-doc things to make sure the consistence. This would be a big project..


Add an attribute for renaming structs

I am trying to use this crate along with the tauri-specta crate to build an app that is attached to an SQLite database. I am using sea_orm as my ORM and it has an interesting quirk where every model struct must be named Model. For example:
#[derive(Debug, Clone, PartialEq, DeriveEntityModel, Serialize, Deserialize, Type)]
#[sea_orm(table_name = "tests")]
#[serde(rename_all = "camelCase")]
pub struct Model {
    #[sea_orm(primary_key)]
    #[serde(skip_deserializing)]
    pub id: i32,
    pub name: String,
    pub description: String,
}

This is resulting in type declarations that contain errors because the exported types have the same name. I am using this crate to replace a needlessly complicated type generation system using schemars::JsonSchema which allows me to use the attribute: #[schemars(title = "Test")] and the type declaration will have that name. It would make sense to add an attribute within this crate instead of going the route of parsing yet another crates macro attributes, especially if this would be the only attribute you'd borrow from schemars. I will see about implementing this myself and submitting a pull request, and I will update here if I feel it is beyond my capabilities to do so. Thank you :)

My goodness. Been using Rust + serde for years and only just realized I can rename an entire struct using serde. I apologize for my ignorance. The #[serde(rename = "Test")] attribute solved my problem.

Easily mistake to make! I need to make a one-stop location in the documentation for all supported attributes and their behaviors so they are easier to find.
Specta has done its best to copy serde's conventions because having compatibility is very nice for most usecases. You could have also done #[specta(rename = "Test")] in that case if you didn't want it to effect serde but I don't think that matters for SeaORM so either is fine.


fix(Datagrid): use selectedRowIds to set selected rows

Contributes to #3045
Addresses issue where items are preselected across multiple pages. The batch action toolbar would only render when you changed to the page containing the selected item/s. This behavior can be seen from the code sandbox provided by @K-Markopoulos (thank you!).
The story Select Items in All Pages also was breaking after the storybook 7 merge because onSelectAllRows was called before being defined.
What did you change?
packages/ibm-products/src/components/Datagrid/Datagrid.stories.js
packages/ibm-products/src/components/Datagrid/Datagrid/DatagridSelectAllWithToggle.js
packages/ibm-products/src/components/Datagrid/Datagrid/DatagridToolbar.js
packages/ibm-products/src/components/Datagrid/_storybook-styles.scss

How did you test and verify your work?
Storybook

@matthewgallo merging this to resolve the Storybook issue in Datagrid but I think we still need v1 for this one.

LOL. Never mind. You opened it as I clicked merge. 😂

Great timing! 😄


Remove extra empty line from Applied Visual Design Challenge

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

@thecodingaviator Go ahead and add this change for the non-English versions of this challenge as there would be nothing to be translated.  We will fix them all in one swipe

@RandellDawson Changes made

In a separate PR, it would not be a bad idea to go ahead an make the solution look like the rest (without the code =; `` It is not affecting the ability of the test runner to validate, but it does not look as clean as just having the full code (html and css together).

@RandellDawson I'm not sure I follow through. Where exactly are you referring to?

@thecodingaviator I was referring to the fact that the solution section should look like:
## Solution

```html
<style>
.center
{
  position: absolute;
  margin: auto;
  top: 0;
  right: 0;
  bottom: 0;
  left: 0;
  width: 100px;
  height: 100px;
  background-color: transparent;
  border-radius: 50%;
  box-shadow: 25px 10px 0px 0 blue;
}

</style>
<div class="center"></div>
```

instead of:
## Solution

```js
var code = ".center {background-color: transparent; border-radius: 50%; box-shadow: 25px 10px 0px 0 blue;}"
```

Basically, it should include the original challenge seed code plush the parts that need to be changed to pass the challenge.

@RandellDawson Got your point. The thing is, this would 99% surely would not be the only challenge that has this problem so we can go ahead and create an issue to track these changes IMO

I think we will just deal with these as they pop up instead of creating an issue for them.


Application insight custom logging - custom properties serialization bug

Description - Upon assigning circular JSON to custom properties of app insight log methods like trackTrace , trackException, etc..,
I experience a bug in the serialization and it's reflected in application insight blade table -> custom dimension column.
Runtime - NodeJSv12.18.3
lib version - applicationinsights@latest
Expected behavior - The JSON must be serialised (by removal of circular reference) and proper json is found in application insght blade table under column custom dimension
Actual behavior -
I showing error in custom dimension PFA.
Attachment -

Thanks for reporting the issue @MuhamedSalihSeyedIbrahim, all custom dimensions are send as strings by the SDK, not sure how we can fix this on SDK side, maybe a bug in UX because they are the ones trying to parse it as JSON, where specifically are you seeing this in the portal?, is it possible to validate the custom dimension is well formed on your side before sending it in the SDK?

I observed it exceptions table record in appinsights
when I spread the error obj of the axios in customproperties of track exception method
This is the code
log.trackException(axiosErrorInstance, {properties: {... axiosErrorInstance} });
My goal is to try manually the
proposal explained in issue #809
To capture addition information in error
Such as axios error object contain
Response , request ,config etc..,
Which is critical if stack doesn't provide enough detail.
Yes if proper simple json is provided
From node side the expected result is achieved but in my case the axios exception have cyclic response object of tcpSocket

@MuhamedSalihSeyedIbrahim Please ensure that you're sending correctly JSON serializable. There are several 3rd party NPM packages that can help clean up JSON for serialization. Thank you.


example: show "kitchen sink" of all events

We could have an example that shows a "kitchen sink" of all the possible events and what order they fire in.

Should be great


Motorola Z do not support "pm install"

Thanks for your awesome project.
My cell phone is Motorola Z (XT1650-05 / Android 8.0.0 / System OCC27.102-23), it will throw error "[[INSTALL_FAILED_TEST_ONLY: installPackageLI]]". Install commands as below:
adb push app-debug.apk /data/local/tmp
adb shell pm install -t -r /data/local/tmp/app-debug.apk
I found it's a OS BUG from Lenovo BBS, and the offical solution is:
adb install -t -r app-debug.apk
I found some adb commands (eg:logcat) do not implement in this project, and I wonder maybe "logcat" is same as "install",  can you share some advice?
BR

Sorry for late reply.
The interface of the install function
 def install(self, path,
                forward_lock=False,  # -l
                reinstall=False,  # -r
                test=False,  # -t
                installer_package_name=False,  # -i
                shared_mass_storage=False,  # -s
                internal_system_memory=False,  # -f
                downgrade=False,  # -d
                grand_all_permissions=False  # -g
                )

The  install -t -r app-debug.apk = device.install("app-debug.apk", test=True, reinstall=True)


docs(index): integrate v2 header into index

I’m concerned that since it’s not a part of a warning message with different styling it’s now hidden enough people won’t see it...

@bacongobbler and I discussed this and since we want to direct people to use v2 over v1, this awkward (but effective) header seems to do the trick quite nicely.


SD SFPU test failures

fixed: https://github.com/tenstorrent/tt-metal/actions/runs/9166519602

now fixed: https://github.com/tenstorrent/tt-metal/actions/runs/9166908302


Documentation for recurrent

I'm studying RNN's using jax so I'm currently investigating flax. I think the documentation in the RNN module is incorrect or out of date.
https://github.com/google-research/flax/blob/e7247d58e4f3460c03da5f935cb83d9c0883a97c/flax/nn/recurrent.py#L21-L23
Results in TypeError: apply() missing 1 required positional argument: 'inputs'
Also create builds and evaluates the model and returns a (y, model), so I feel like the design has changed and the recurrent examples should either initialise the state before calling create (but that wouldn't scan), or call create_by_shape?
Edit: I found a test which seems to confirm that the docstring is incorrect i.e. the code below creates an initial carry and passes to create
https://github.com/google-research/flax/blob/e7247d58e4f3460c03da5f935cb83d9c0883a97c/tests/nn_test.py#L461-L468
2nd Edit:
Also, I'm slightly confused by LSTMCell.initialize_carry() - it requires a batch_dim, and returns an  initialised (zero) state for each batch. I might be missing something but this seems to preclude using lax.scan() to process each batch sequentially using the state from the previous batch as the initial state for the next batch (or some other state estimator which is specifically what I'm attempting). For example I have 365 "trajectories" (timeseries) each consisting of 24 samples and 5 features. So the state should be size 5 and I want to scan each trajectory from some initial state I provide (the intent is to use another net to estimate the state).

Indeed there is a mistake in the example and the test is correct.
We also have a full LSTM example in the pipeline which should be merged soon.
As for your use case which if I understand correctly is the batch size 1:
I think you could try to initialize the carry with initialize_carry(rng, (1,), 5) and it should probably work when using initialize_carry(rng, (), 5) as well.

Thanks, you're correct both of those methods work for initialize_carry
I'm having issues with scan though. If I use the pure python definition of scan from the jax docs the code below works. But if I use the jax implementation I get an exception which I've quoted below.
I'm using jax 0.1.59 and jaxlib 0.1.39 with tensorflow 2.1.0
import jax
import flax
import numpy as onp
import jax.numpy as jnp

rng = jax.random.PRNGKey(0)

x = jnp.ones((365, 96, 2))
c0 = flax.nn.recurrent.LSTMCell.initialize_carry(rng, (), 2)
print(f"c0: {c0}")

(c, y), lstm = flax.nn.recurrent.LSTMCell.create(rng, c0, x[-1][0])
print(f"c: {c}")

def scan(f, init, xs, length=None):
    if xs is None:
        xs = [None] * length
    carry = init
    ys = []
    for x in xs:
        carry, y = f(carry, x)
        ys.append(y)
    return carry, jnp.stack(ys)

timeseries = x[0,:,:]
scan(lstm, c0, timeseries)

scan = jax.lax.scan
scan(lstm, c0, timeseries)

Traceback (most recent call last):
File "/home/david/dev/flax/examples/rnn/train.py", line 29, in
scan(lstm, c0, timeseries)
File "/home/david/.pyenv/versions/3.7.1/lib/python3.7/site-packages/jax/lax/lax_control_flow.py", line 791, in scan
jaxpr, consts, out_tree = _initial_style_jaxpr(f, in_tree, carry_avals + x_avals)
File "", line 2, in hash
TypeError: unhashable type: 'dict'

A Model instance currently fails when passed into a function like jit or scan. I have a fix for this but for now you could use scan(lambda c, x: lstm(c, x), c0, timeseries).
The reason this fails is because Jax tries to cache the functions it transforms but Model has parameters arrays which are not hashable. I will implement hash in a separate change and at a test for this.

Thanks yes that works.
PS I'm also seeing this XLA warning
2020-02-19 08:51:51.429909: W external/org_tensorflow/tensorflow/compiler/xla/service/hlo_pass_fix.h:49] Unexpectedly high number of iterations in HLO passes, exiting fixed point loop.
It's raised from LSTMCell.apply() as far as I can tell.
It could be due to me upgrading (unexpectedly) to tensorflow 2.1 [they've merged tensorflow and tensorflow-gpu 2.0 into a single package and released so now anything which depended on tensorflow now installs 2.1 and enables gpu support - not a bad thing long term but causes a few short term issues]

PS Would you recommend to use scan inside a Module instead?

Hi @david-waterworth -- can you please take a look at our seq2seq example and let us know if that help clarify how to use LSTMCell? https://github.com/google-research/flax/blob/prerelease/examples/seq2seq/train.py

@avital yes thanks, the use of nn.attention.scan_in_dim answers my question (although I probably wouldn't have expected to find it in the attention module?) -- I've been using a similar pattern with my plain jax implementation.

@avital there were a couple of minor questions I had after adapting the seq2seq example to for my purposes.

Why are you calling train_metrics = jax.device_get(metrics)? The annotated mnist example seems to imply this is automatic ("metrics are only retrieved from device when needed on host (like in this print statement)")?

The example doesn't evaluate a test loss during training. Again from the mnist example I discovered optimizer.target and using this I was able to compute the loss using my test set once every n iterations.

One more thing that really tripped me up.
In your example the Encoder.apply() has a default argument hidden_size and you never explicitly set this. I wanted to use a different value, so I made the following changes
class Seq2seq(nn.Module):
  """Sequence-to-sequence class using encoder/decoder architecture."""

  def apply(self,
            encoder_inputs,
            decoder_inputs,
            hidden_size=128,  # <- added here (used different default to demonstrate)
            train=True,
            max_output_len=None):
    """Run the seq2seq model."""
    # inputs.shape = (batch_size, seq_length, vocab_size).
    batch_size, _, vocab_size = encoder_inputs.shape
    carry = Encoder(encoder_inputs, name='encoder', hidden_size=hidden_size)

Then I modified create_model() i,e,:
def create_model():
  """Creates a seq2seq model."""
  vocab_size = CTABLE.vocab_size()
  _, model = Seq2seq.create_by_shape(
      nn.make_rng(), [((1, get_max_input_len(), vocab_size), jnp.float32),
                      ((1, get_max_output_len(), vocab_size), jnp.float32)],
      hidden_size=64)   # <- modified here (again different to defaults)
  return model

The issue is Seq2seq.create_by_shape() passes hidden_size=64 but the returned model doesn't have this parameter partially applied - so each time it is applied the default is used.
model = create_model()
model(batch['query'], batch['answer'])   # <- uses incorrect encoder hidden size?

What I had to do instead is create a partial Seq2seq.
def create_model():
  """Creates a seq2seq model."""
  vocab_size = CTABLE.vocab_size()
  model_def = Seq2seq.partial(hidden_size=64)
  _, model = model_def.create_by_shape(
      nn.make_rng(), [((1, get_max_input_len(), vocab_size), jnp.float32),
                      ((1, get_max_output_len(), vocab_size), jnp.float32)])
  return model

Should create_by_shape partially apply *args and **kwargs?

Finally it seems a little awkward that initialize_carry is stochastic? By default it uses initializers.zeros but because it calls random.split() you cannot pass None. That means that at inference time you seem to have to do:
model = train_model()

rng = jax.random.PRNGKey(0)
with flax.nn.stochastic(rng):
    Y_hat = model(X_test, XY_0_test)

I simply used jnp.zeros instead to create the initial state instead.

Hi David, thanks a lot for your comments! I'm also still relatively new to FLAX, so having someone taking a critical look at my code is definitely good for my understanding as well.

@avital yes thanks, the use of nn.attention.scan_in_dim answers my question (although I wouldn't have expected to find it in the attention module - maybe flax.jax_utils?)

I agree, we should move it to flax.jax_utils (not very high priority though)

Why are you calling train_metrics = jax.device_get(metrics)?

Hmm good point! I basically do this because this is done for the MNIST example, line 146, but I tested it and without that it also works. I will update the example.

The example doesn't evaluate a test loss during training. Again from the mnist example I discovered optimizer.target and using this I was able to do what I needed, ideally the examples would include this as it's fairly standard pratice?

Indeed, it doesn't report test loss. However the accuracy it reports is the same accuracy as during inference (without teacher forcing), which I explain in the comment on line 214: "Computes sequence accuracy, which is the same as the accuracy during inference, since teacher forcing is irrelevant when all output are correct."
I think adding test loss is nice but not crucial, since we report accuracy and some example decodings. Let me know if you feel otherwise, or whether you think my reasoning doesn't make sense.

For completeness should train_model not return the trained model? In the example you're not using it but it would still be useful, particularly as immutability means you have to return optimizer.target not model which might trap someone?

Yes that makes sense to me, I'll update it.

Should create_by_shape partially apply *args and **kwargs?

I think your solution of first using partial and then create_by_shape is fine, you could even concatenate them:
model_def = Seq2seq.partial(...).create_by_shape(...)
This is how I would do it as well, but if you find a better way, please let me know :-).

Finally it seems a little awkward that initialize_carry is stochastic?

I don't think I fully understand what you mean here. Could you clarify?

I think your solution of first using partial and then create_by_shape is fine,

What I've also started doing is to avoid having default values for apply *args like hidden_size. This way if I pass them to create_by_shape instead of partial I'll get an error later at train time. Otherwise it just silently creates with one value but applies using the default.

I don't think I fully understand what you mean here. Could you clarify?

What I mean by stochastic is if you do the following
model=train_model()
model(x)

It will throw an exception, you have to create a random context i.e.
model=train_model()
rng = jrandom.PRNGKey(0)

with nn.stochastic(rng):
  model(x)

This is because of: https://github.com/google/flax/blob/1022b9eb00e9dcac192003ad0e92fb5c63ad2598/examples/seq2seq/train.py#L117
Since whilst it's not used all the initialisers require an rng even the constant ones (i.e. initializers.zeros). It's not a big deal but I'm not sure there's a need to be able to randomly initialise the state which is why I simply replaced this line with jnp.zeros.

I think both your suggestions are good insights. I guess the stochastic initialize_carry is a bit less crucial (it is a most a bit more cumbersome in the current way), but the initialization problem seems like something users can run into more often. I will ask around how people usually handle this, and see if this can be improved, or if we can have clearer guidelines around how to best initialize such variables. I guess at least we should have some example. I'll let you know when I have an answer.

PS If you make hidden_size a flag (which if I understand absl correctly turns it into a command line parameter) and then modify the code so it's correctly passed to the Encoder you can demonstrate the need to use partial

I think it would be better to use a fixed seed to initialize the carry given that it is just zeros anyway. It seems odd to require nn.stochastic for something that actually isn't random. I created PR #85 for this

I'm closing this issue since most important questions seem to have been answered. David: if you have any other specific concerns / questions, can you please open a new issue? Thanks!

A Model instance currently fails when passed into a function like jit or scan. I have a fix for this but for now you could use scan(lambda c, x: lstm(c, x), c0, timeseries).

I plan to use Flax in my research on RNNs, but I'm struggling to understand some ideas behind Flax implementation for days. I really wanted to see an example of someone using the documented RNN cells, it would help me a lot!
Anyway, I tried  to use the lambda trick, but it ended up not working, I get an error like this: ValueError: Jax transforms and modules cannot be mixed. when the cell is being defined. The definitions are like bellow (I don't know if that is that way it is mean't to be done, but I made it to be run on a single sample (i.e. x is a vector)):
class LRNNCell(nn.Module):
    @nn.compact
    def __call__(self, h, x):
        nh = h.shape[0]
        Whx = nn.Dense(nh)
        Whh = nn.Dense(nh, use_bias=False)
        Wyh = nn.Dense(1)

        h = nn.tanh(Whx(x) + Whh(h))
        y = nn.tanh(Wyh(h))
        return h, y

class LRNN(nn.Module):
    ny: Any
    nh: Any

    @nn.compact
    def __call__(self, x):
        h = jnp.zeros(self.nh)
        cell = LRNNCell()

        h, y = jax.lax.scan(lambda h, x: cell(h, x), h, x)
        return y[-self.ny:]

What am I missing?

Hi @marcosrdac, it seems you replied on a very old issue that wasn't using Linen yet. Also, it seems your questions is better suited for a Github Discussion. Would you mind asking your question there? https://github.com/google/flax/discussions/new
Happy to help you further there! (then it is also easier for users to find the answer).

Hi @marcosrdac! I've copied your question to Github Discussion #1283 so it is easier accessible for other users. Let's continue the discussion there!


chore: update SCC profile_id to fe96bd4d-9b37-40f2-b39f-a62760e326a3

/run pipeline

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


Remove geometry selection from preset browser

In OSM, many feature types can be mapped using more than one geometry, but typically a single geometry is sufficient or most common. The iD 3 preset browser currently prompts the mapper to select a geometry type if there are multiple options.

This decision is burden for new mappers and we should remove it.
A simpler solution is to simply enter the "add mode" for the best geometry immediately after the user picks a feature type. Advanced mappers could then switch the geometry if needed. iD would remember the preferred geometry for each preset.

Here's a demo.
Remaining work:

Pick a keyboard shortcut to toggle the geometry type
Review presets and determine the preferred geometry type for each. iD interprets the array's item order as the preference order

@quincylvania For the keyboard shortcut, what about using 1/2/3 while in the "add mode"? Currently the numbers continue to change between presets, but since they aren't shown on screen in that mode I think it could be fine to override that (if someone did want to change preset they can hit escape first).

@SilentSpike Maybe. I like how you can still switch between quick presets after selecting one in case you accidentally select the wrong one.


Implement Freetrade parser

Supported events

Sell
Buy
Top up
Withdrawal
Interest on cash
Dividends
Free share

Freetrade doesn't provide stock splits events, I added a hacky support for few stocks borrowing code from another parser

The datetime.fromisoformat do not support dates like 2021-12-13T16:16:49.475Z in Python version before 3.11
CI uses Python 3.9 at the moment its EOL is planned in 2025
@KapJI Would you mind if I make a PR with Python version update?

Looks great! Can you also update the README.md with instructions for it?

Instead of using 3.12 you can do something like this maybe?
from datetime import datetime

iso_date = "2021-12-13T16:16:49.475Z"
# Replace 'Z' with '+00:00' to make it compatible
iso_date = iso_date.replace("Z", "+00:00")

# Parse the string with the adjusted format
parsed_date = datetime.strptime(iso_date, "%Y-%m-%dT%H:%M:%S.%f%z")

print(parsed_date)  # Output: 2021-12-13 16:16:49.475000+00:00

Thanks for the advice, I added a date parsing function based on your example and augmented README.md


No open tab Role in Service

Description
When browsing services you can no longer select and manually scale or expand the Role to see all
To Reproduce
instanciate service, go to "service" menu . select a service, open tab Role and select role.
Expected behavior
display role information
Details

Affected Component: [e.g. Sunstone, Scheduler or Storage]
Hypervisor: [e.g. KVM]
Version: [e.g. 5.4.12 or development]

Additional context
Add any other context about the problem here.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Resolved in the following PRs:

https://github.com/OpenNebula/one/pull/4421
https://github.com/OpenNebula/one/pull/4427

Doc:  https://github.com/OpenNebula/docs/pull/1010


Fixes Pause and Resume in home-song-item.

Fixes  Issue https://github.com/phanan/koel/issues/387

Silly me strikes again! Thanks for fixing!


question; How would you go about listening to Events.MEDIACONTROL_FULLSCREEN event ?

looked at node_modules/clappr/src/base/events.js:299
and It looks like both  Events.MEDIACONTROL_FULLSCREEN and
Events.CONTAINER_FULLSCREEN are exposed.
however,
window.player.on(Events.MEDIACONTROL_FULLSCREEN, () => {
           // never triggers after toggling fullscreen
});

neither event seems to work. Any clues ?
Thanks in advance!

I was able to get around this by asking the DOM  based on this isFullScreen function.
Closing issue.


PR: Allow an adjustable time difference for modification detection (Editor)

For CIFS (and perhaps other network file systems), where caching file attributes is used, expecting identical timestamps causes many false positives. This hacky patch allows for some deviation.

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #21877 hopefully, we'll see next week when much more testing is planed in production.
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: and1bm

Hi @ccordoba12,  I reverted the changes in the translatable string.

Thanks! I think this is ready, so we just need your confirmation that it actually works in your systems to merge it.

@and1bm, any news about this patch? Did it work for you?
We're planning to release 5.5.6 soon and we'd like to include it in that release.

@ccordoba12 sorry, we are still investigating, it seems not to fix all (?) problems. We have actimeo=600 set for our cifs mount of the homes, perhaps it's related to that number. However, students are reporting slowly and I am waiting for more detailed reports to understand the issue and cifs access times better.

Ok, no problem. Let us know when you have more info.


Question about resource collection mechanics of a CityTile

https://www.lux-ai.org/specs-2021

Workers cannot mine while on CityTiles. Instead, if there is at least one Worker on a CityTile, that CityTile will automatically collect adjacent resources at the same rate as a worker each turn and directly convert it all to fuel. The collection mechanic for a CityTile is the same as a worker and you can treat a CityTile as an individual Worker collecting resources.

What is the expected behavior of the above specifications?
Candidates are:

(a) Regardless of the number of workers on a CityTile, the CityTile is treated as a single worker.
(b) a CityTile is treated as a number of workers equal to the number of workers on that CityTile.
(c) depends on the situation.

In my understanding, (c) is the current behavior. But this comment says (a) is the expected behavior.

https://github.com/Lux-AI-Challenge/Lux-Design-2021/blob/26ff64b1d119c7ac582acb2b4df17c0d70cd819d/src/Game/index.ts#L676-L685
https://github.com/Lux-AI-Challenge/Lux-Design-2021/blob/26ff64b1d119c7ac582acb2b4df17c0d70cd819d/src/Game/index.ts#L26-L39

For example,
Case 1:

CityTile c_1 is on (5, 5). Wood Resource with enough resources is on (5, 6)
worker u_1 with 0 wood moves to (5, 5). r1 := ResourceRequest((5, 5), 20,  undefined, c_1)
worker u_2 with 0 wood moves to (5, 5). r2 := ResourceRequest((5, 5), 20, undefined, c_1)
r1.equals(r2) is true. So, r2 is ignored and CityTile c_1 collects only 20 woods

Case 2:

CityTile c_1 is on (5, 5). Wood Resource with enough resources is on (5, 6)
worker u_1 with 0 wood moves to (5, 5). r1 := ResourceRequest((5, 5), 20,  undefined, c_1)
worker u_2 with 90 wood moves to (5, 5). r2 := ResourceRequest((5, 5), 10, undefined, c_1)
r1.equals(r2) is false. So, r2 is NOT ignored and CityTile c_1 collects 20+10=30 woods

(a) is the expected behavior
you can treat a CityTile as an individual Worker collecting resources.

@StoneT2000 thank you for the explanation.
In Case 2, how many woods will be collected by the CityTile? 30?
If (a) is the expected behavior, then CityTile is subject to the same constraints as other Workers, and I think it cannot collect more than 20 wood (wood collection rate per turn).
Am I misunderstanding something?

Note that ResourceRequests are made by tile. basically if a tile already has a request for resources, other units on the same tile cannot generate more requests. See
https://github.com/Lux-AI-Challenge/Lux-Design-2021/blob/26ff64b1d119c7ac582acb2b4df17c0d70cd819d/src/Game/index.ts#L675-L685
Once resource requests are made, if a resource request's location is the same as a city tile, we don't use that 2nd amount argument anymore. See
https://github.com/Lux-AI-Challenge/Lux-Design-2021/blob/26ff64b1d119c7ac582acb2b4df17c0d70cd819d/src/Game/index.ts#L702-L709
So in the end, only r1 is created, and since on (5, 5) we have a CityTile, the CityTile collects 20 wood and converts that into 20 fuel. None of the workers u_1 or u_2 collect anything, and the worker u_2 drops all 90 wood into the CityTile for it to be converted into 90 fuel.

basically if a tile already has a request for resources, other units on the same tile cannot generate more requests
...
So in the end, only r1 is created,

In current implementation, if 2nd amount argument is different, multiple requests will be generated for units on the same tile.
https://github.com/Lux-AI-Challenge/Lux-Design-2021/blob/26ff64b1d119c7ac582acb2b4df17c0d70cd819d/src/Game/index.ts#L26-L39
In Case 2, 2nd amount argument between r1 and r2 is different, so both r1 and r2 are created.

I wrote a test to reproduce Case 2, and it looks like CityTile is collecting 30 woods instead of 20.
Could you please check this?
https://github.com/sash2104/Lux-Design-2021/pull/1

Hi, this is definitely a bug. However, our team has decided that it is too minor of a thing to ever get used as it's rare for this to occur.


Add version numbers to dependencies.

Also update the least php requirement since phpunit requires >= 5.3.3.
Added branch-alias to let developers install the latest version with 3.0.*@dev as version number.

Coverage remained the same at 99.3% when pulling 17bd8ac96fbc25db0cf0d25136c60b5dd4312709 on vinkla:composer into cab99997e468a0bc34860acaa8c6e9dcb2e7a536 on jenssegers:master.


Want new Package: Unciv

Here is its website. https://github.com/yairm210/Unciv
Add it is easy. The game just create few documents in its position.

Duplicate of https://github.com/Calinou/scoop-games/issues/484.


Wrong Case in

The path to the "jquery.dataTables.min.js" as a mistake :
it reads :
src="../bower_components/DataTables/media/js/jquery.dataTables.min.js"
but should be:
src="../bower_components/datatables/media/js/jquery.dataTables.min.js"
without any capital letter in "datatables"
Hope this help.

I'll hit this soon, thanks for pointing this out.

Should be fixed.


Unit 7

Make works
This is our unit 7 lesson about array lists

I liked how you incorporated images to help people understand the lesson. Something you could do is have the solutions to the hacks on a separate page that people can check their work. This will also make it so that the solutions don't have to be on the page with the hacks where people are working.

I like the styling you guys chose as the dark theme is easy on the eyes. However, I wish you used more capitals at the start of your sentences. P.S. I am still waiting for the subway surfers collab.


Auth Failure

Hi @smdahlen,
When trying to spin up a new digitalocean box, I'm currently receiving the following message:
There was an issue with the request made to the Digital Ocean
API at:

Path: /v2/droplets
URI Params: {}

The response status from the API was:

Status: 401
Response: {"id"=>"unauthorized", "message"=>"Unable to authenticate you."}

I've copied and pasted the example from the readme, replacing the appropriate values like my token. I've also confirmed the token is working using vagrant digitalocean-list images $token so I know that the issue is not with my token. I saw in another thread that someone else has had the same problem as well.
It looks to me as if the auth token is not being passed in the params which is odd. Unfortunately I don't know enough of ruby to debug this one more.
Vagrant 1.7.2::vagrant-digitalocean 0.7.4

I have the same problem.... =/

Same here.

same

Can you provide a paste of a terminal session showing this failure? (Make sure to obfuscate your API token, if present.) Also provide a brief example Vagrantfile that reproduces the failure.

@conorsch The issue itself has the output of the command, what else did you need from the term?

@jaitaiwan Trying to reproduce the error locally. I've seen a similar error before on a few projects, and it's generally caused by environment variables for the API token not being referenced. The terminal session would show precisely which commands were typed, in case it's an invocation error (e.g, placing the token as a command-line parameter when uping a new machine). Most valuable would be the Vagrantfile itself.

Here's the vagrantfile:
# -*- mode: ruby -*-
# vi: set ft=ruby :

# Vagrantfile API/syntax version. Don't touch unless you know what you're doing!
VAGRANTFILE_API_VERSION = "2"
DEPLOYMENT_ENV = ENV['env'] ||= 'development'
TOKEN = ENV['token']
KEYNAME = ENV['keyname']
HOSTNAME = ENV['host'] ||= 'default'

Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|

  config.vm.define :"#{HOSTNAME}" do |t|
  end

  # Base Box
  # --------------------
  config.vm.box = "vivid64"
  config.vm.box_url = "https://cloud-images.ubuntu.com/vagrant/vivid/current/vivid-server-cloudimg-amd64-vagrant-disk1.box"

  # Generic SSH Key location
  config.vm.provider :virtualbox do |provider, override|
    provider.customize ["modifyvm", :id, "--memory", "1024"]
    override.vm.network :private_network, ip: "<IP_ADDRESS>"
    override.vm.synced_folder ".", "/vagrant/", :mount_options => [ "dmode=777", "fmode=666" ]
    override.vm.synced_folder "./www", "/vagrant/www/", :mount_options => [ "dmode=775", "fmode=644" ], :owner => 'www-data', :group => 'www-data'
  end

  config.vm.provider :digital_ocean do |provider, override|
    override.ssh.private_key_path = '~/.ssh/id_rsa'

    # We're using Digital Ocean's pre-built images so set the box to something lightweight and genric
    override.vm.box = 'digital_ocean'
    override.vm.box_url = "https://github.com/smdahlen/vagrant-digitalocean/raw/master/box/digital_ocean.box"
    override.vm.hostname = HOSTNAME

    provider.name = HOSTNAME
    provider.token = TOKEN
    provider.image = 'ubuntu-14-10-x64'
    provider.region = 'nyc3'
    provider.size = '4gb'
    provider.ipv6 = true
    provider.ssh_key_name = KEYNAME
    provider.private_networking = true
  end

  # Provisioning Script
  # --------------------
  # Make sure it knows what env we want to deploy

  config.vm.provision "shell" do |s|
    s.path = "init.sh"
    s.args = [DEPLOYMENT_ENV]
  end

end

I checked and I'm seeing the ENV['token'] containing the digital ocean token. I'm fairly certain although not a stock-standard vagrant, that it follows the format set down in the examples on the home page.

The vagrant file hasn't changed in a long time, only moving devices has, so I assume its an update to the plugin but seems its working for others I'm not so sure.

Throw in a puts TOKEN, does this output anything?

Yup. Shows the token accurately.
On Mon, 27 Jul 2015 13:08 Seth Reeser<EMAIL_ADDRESS>wrote:

Throw in a puts TOKEN, does this output anything?
—
Reply to this email directly or view it on GitHub
https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125069476
.

Does this work? vagrant digitalocean-list images YOUR_TOKEN_HERE
On Sun, Jul 26, 2015 at 11:09 PM, Daniel Holmes<EMAIL_ADDRESS>wrote:

Yup. Shows the token accurately.
On Mon, 27 Jul 2015 13:08 Seth Reeser<EMAIL_ADDRESS>wrote:

Throw in a puts TOKEN, does this output anything?
—
Reply to this email directly or view it on GitHub
<
https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125069476
.

—
Reply to this email directly or view it on GitHub
https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125069752
.

--
Seth Reeser
Founder and Technology Director
devopsgroup.io

Yup! Get a lovely list of do images.
On Mon, 27 Jul 2015 13:15 Seth Reeser<EMAIL_ADDRESS>wrote:

Does this work? vagrant digitalocean-list images YOUR_TOKEN_HERE
On Sun, Jul 26, 2015 at 11:09 PM, Daniel Holmes<EMAIL_ADDRESS>wrote:

Yup. Shows the token accurately.
On Mon, 27 Jul 2015 13:08 Seth Reeser<EMAIL_ADDRESS>wrote:

Throw in a puts TOKEN, does this output anything?
—
Reply to this email directly or view it on GitHub
<

https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125069476

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125069752
.

--
Seth Reeser
Founder and Technology Director
devopsgroup.io
—
Reply to this email directly or view it on GitHub
https://github.com/smdahlen/vagrant-digitalocean/issues/200#issuecomment-125070379
.

were seeing this issue too, Vagrant 1.7.2::vagrant-digitalocean 0.7.7
and we also get a nice list of images using vagrant digitalocean-list images YOUR_TOKEN_HERE

try putting your token directly in your vagrant file

I'll echo @digcat's suggestion of hard-coding the token in the Vagrantfile temporarily. That should work if you paste the exact string used when invoking from the command line as vagrant digitalocean-list images YOUR_TOKEN_HERE.
@jaitaiwan Try upgrading vagrant and the plugin. You said you're running vagrant 1.7.2 and plugin 0.7.4, but current are 1.7.3 and 0.7.7. I'd definitely try the newer versions and see if that helps.


Invalid array length

//...
https.get(url, response => {
			// if --progress run the progressbar
			if (flags.progress) {
				const len = parseInt(response.headers['content-length'], 10);
				const bar = new ProgressBar(chalk `{yellow ↓} {red :percent} [:bar] :elapsed s`, {
					complete: '=',
					incomplete: ' ',
					width: 20,
					total: len,
					clear: true
				});

				// on data received fill the progressbar
				response.on('data', chunk => {
					bar.tick(chunk.length, {
						passphrase: 'Making something awesome'
					});
				});
			}

                  //... rest of the function

causes:
/Users/rawnly/code/NODE/splash-cli-2018/node_modules/progress/lib/node-progress.js:155
  complete = Array(Math.max(0, completeLength + 1)).join(this.chars.complete);
             ^

RangeError: Invalid array length
    at ProgressBar.render (/Users/rawnly/code/NODE/splash-cli-2018/node_modules/progress/lib/node-progress.js:155:14)
    at ontimeout (timers.js:471:11)
    at tryOnTimeout (timers.js:306:5)
    at Timer.listOnTimeout (timers.js:266:5)

Getting this as well

The reason for this problem is probably the total parameter except the problem, the value is NaN, you can check if this is the problem.

Currently experiencing this exact issue.

content-length isn't guaranteed so it could be 0 and make bug. set a value if content-length is 0.
      // content-length isn't guaranteed so it could be 0 and make bug. set a value if content-length is 0.
      const len = parseInt(response.headers["content-length"], 10) || 100000000;
      const bar = new ProgressBar("downloading [:bar] :rate/bps :percent :etas", {
        complete: "=",
        incomplete: " ",
        width: 20,
        total: len,
      });
      response.on("data", (chunk) => {
        bar.tick(chunk.length);
      });

it need validation:
#171

I confirm the bug in 2.0.3 in case total is 0 :
const ProgressBar = require('progress')
const format = 'progress: :percent [:bar] :current/:total (~:etas left)'
const options = {total: 0}
const progressBar = new ProgressBar(format, options)
// next line throw RangeError: Invalid array length at ProgressBar.render
// on the line `complete = Array(Math.max(0, completeLength + 1)).join(this.chars.complete);`
// because completeLength isn't a number
progressBar.tick(0)

The index of array must be integer. The input :total values is NAN or float, while the output array's index is not integer so than rendering error.

For a temp solution you can change node-progress.js
completeLength = Math.round(width * ratio)
with this one
completeLength = Math.round(width * ratio) | 0;
that works for me


⚠ Bofrost.lu has degraded performance

In 93fb66c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1030 ms

Resolved: Bofrost.lu performance has improved in 06f73c9 after 10 minutes.


Data URI instead of normal URI?

Hey, using the latest device_calendar, I'm trying to set the url param on Event with a URL directing to my custom deeplink, but it's not saved in calendar, I have no URL there. So I look into the plugin's code and I see this:
https://github.com/builttoroam/device_calendar/blob/6a9733c35fb693a5fad93ad641886e393a64de11/lib/src/models/event.dart#L246
Why would you accept only data: URIs? Should that be fixed and toString() any Uris? Do you accept PR for that?

Yeah feels like something we should take PR in


support for group by with totals

Summary
A suggestion on how to resolve group by with totals fails with 'handle packet: unexpected packet "Totals"'
With these changes, the totals row will be returned as the last row of the result.
Checklist
Delete items not relevant to your PR:

[V] Unit and integration tests covering the common scenarios were added
[V] A human-readable description of the changes was provided to include in CHANGELOG

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


WIP: Datepicker

Here's the first pass at the datepicker element, let me know what you think!

Here's a screenshot of the element for quick reference:

In that image, July 19 is selected, and July 21 is today's date (hence the outline).

@bartboy011 Looks great, I hope it gets mainlined! DatePicker is probably one of the hugest components to forms (who doesn't need a datepicker nowadays LOL?).
I am currently (myself) working on a min/max slider since Bulma/Buefy have none built-in (which I think is also a highly utilized component!). Thanks for your PR, I may cherry pick your patch for my own projects for now. : )

@bartboy011 well done!
I have some requests to complete the component:

i18n months and days (maybe is better 3 words then one);
month and year selectable (configurable by props);
footer buttons (today, clear, close, ..., configurable by props)

@jtommy Thanks! I'll start working on those edits soon!

This is going to be very nice.
Only a small suggestions: 😸

The possibility of disabling past dates.
Add a range of selectable years, for example start 2017 and ends in 2020.

To do list:

[x] i18n date names
[ ] month and year selectable
[ ] footer buttons
[ ] earliest and latest selectable dates

Hey all, just made the requested additions, take a look when you have a chance!
@jtommy the only thing I wasn't able to add was the ability to 'clear' the date out. Because so much of the component is built around the value prop being a Date, if the value is set to null it breaks the component. If you have a suggestion on a workaround, let me know!

@jtommy thanks for these!
Month and year selectable: I was addressing this through the passed in value prop. The date passed in is the date used for month and year. I think your suggestions for how to address the default date are better and I will adjust everything accordingly.
Today button: you're right, that is a better way for that button to work.
Do you think a second event is necessary? Your suggestions tend to revolve around removing the need for a v-model attribute on the component. Do we need to have another event emitted then that is something like select?

Month and year selectable: Default date it's ok but it isn't required (so you can implement Clear button). In general i think something to select year and/or month quickly (think about select your birthday).
About new event, I think it's ok at the moment if you follow my suggestions. I just noticed the issue into watch of isActive prop.

@jtommy I believe these changes address your entire review! Let me know if there's anything else you see.

@jtommy That's a good question. I'm not certain how I feel about the clear button also closing it. On one hand, not closing the modal on clear encourages the user to choose a date, but on the other, the clear button is an opt-in prop, so maybe I'm making decisions for the developer? What's your opinion?

@bartboy011 I think that clear button is a shortcut to reset the value. Now if you agree with me you could edit current behaviour and do reset+close (like today)

Great, sounds good to me. Changes will be up in a moment.

@bartboy011 Well done, great work!
The only thing to improve is about a quick way to select a month and year. What do you think about ?

@jtommy I think I've found an easy way to do this, via selects, however I'm encountering issues getting everything to fit on mobile. Here are a couple of screenshots:
Here's the new look with the selects on desktop:

And again on mobile:

Thoughts on how to make this work on mobile?

@bartboy011 Great!
About mobile, no touch o swipe behaviour at the moment. Ok, about remove arrow icons 👍
Question: How many elements into year select ? Automatic slide up and down by delta ?

Currently I have it set that, if there is a latest-date prop set, the year from that date is the highest the year selector will go. Otherwise, it offers up to 3 years forward. For the earliest date, same thing with the earliest-date prop for the year, otherwise it goes back to 1900 (a pretty standard earliest year I see in a lot of date pickers). Yes to the automatic slide.
Here's a screenshot:

Ok, when you are ok we review it. I think it's ready!

I just had a thought - with the new b-dropdown options (in v 0.5.0) it's possible to have the datepicker in a dropdown rather than a modal... Should this be given the option to be used as a standalone table, and not require the modal? Something like an as-modal prop?

Hey @bartboy011 thanks for contributing!
Looks like a great WIP, I have few observations:

As you said, I think it's better to use the dropdown and put the date on a b-input to really make it like a form control (like autocomplete), I don't think it's necessary an option to open like modal because the dropdown itself behaves like a modal when it's touchscreen.
The footer buttons labels/buttons could be a slot rather than prop, easier to stylize with icons and others (like the new upload component or dropdown trigger).
The selectable range functionality could also disable the prev/next buttons don't you think?

Other than that I think it's great, good work on doing without dependency 👍

@rafaelpimpa awesome notes, thank you! I'll do some overhauling and push commits soon!

Overhaul done! @jtommy @rafaelpimpa, the only thing I feel like that needs to be done now is allow developers to set the format in which the date is emitted/displayed. Currently if the value passed in is a date object, a date object is emitted, and the same with a string, but that doesn't address mm/dd/yyyy formatting. Any thoughts on how to best address this?

Only to say that it is a nice library. Good job ppl.

@bartboy011 Great job! Now it looks like a Datepicker for sure 😄
About the format, we can pass a function prop for that, and use the toLocaleDateString() (https://developer.mozilla.org/pt-BR/docs/Web/JavaScript/Reference/Global_Objects/Date/toLocaleDateString) as default.

@bartboy011 In addition to @rafaelpimpa you have to check formatValue function because there is issue about timezone. For example when i select a date i don't see the selected date but the previous day.
A possibile solution could be:
const toDate = new Date(value)
const date = new Date(toDate.getTime() - (toDate.getTimezoneOffset() * 60000)).toISOString()

@rafaelpimpa Good notes, thanks. I ended up not using b-dropdown as I was having some issues with the dropdown opening and then immediately closing when calling both this.$refs.dropdown.$data.isActive = true and this.$refs.dropdown.toggle(), not really sure why. It also was only occurring on desktop-sized browsers. Regardless, it ended up working out better to simply use standard HTML rather than a component.
I think the reason formatValue is being called so many times is that, in order to allow both a Date object and a String to be passed in as value, the value has to be passed through formatValue in order to appear properly in the input field. Currently, the b-input has :value="formatValue(value)" as a prop so that if a Date is passed in, it still is displayed as a string. Additionally, when the value is emitted, dateSelected is parsed to a string via formatValue if a string was passed in. So, it's being called once on the emitting of the selected value, once when the b-input component is re-rendered, and then once more when dateSelected is changed. Because it is used to compute the string version of more than one date, it can't be a computed value. So right now I don't really see a way to change the behavior to reduce usage.
I added the :date-formatter function prop with the .toLocaleString() fallback and it's working great! However, in my timezone, your fix @jtommy caused the same issues you were having without it. I'm not entirely sure how to address that issue at this time... I guess other than having developers pass in a dateFormatter() that handles this issue. I've gained a whole lot of respect for the work being done over at Moment.js after this!
Additionally, I went through and made the datepicker fully screen-reader friendly. The table is entirely tab-navigable, aria-labels have been added on any non-standard buttons, and all inputs have keydown listeners attached.

Thanks @bartboy011!
I've made a few tests using b-dropdown and it worked good, what I did was replacing every isActive to $refs.dropdown.isActive (dropdown ref being the b-dropdown itself) and remove the this.isActive = true from the focus event.
If you manage to do this then it's pretty much done ;)

I've got it working, but it requires moving the b-input into the b-dropdown itself, which causes styling issues with the input. The issue I was having previously, I believe, is that since I was not placing the b-input inside the b-dropdown so that the styling of the b-input was unaffected, the click event listener inside the b-dropdown was closing the dropdown menu when the b-input was clicked. I confirmed this by focusing on the b-input via tab, which opened the b-dropdown without an issue. If you're fine with the styling of the b-input being affected by the b-dropdown then I'll leave this as is and push the commit.

What kind of style was affected? I didnt't notice.

Mostly the input width since dropdown containers don't grow to fill the available width the way a standard input does. It's not the biggest issue in the world, but may annoy a few users.

We can fix that with this I guess:
.datepicker {
  .dropdown,
  .dropdown-trigger {
    width: 100%;
  }
}

Works for me! Committing now!

Thanks a lot for this @bartboy011!

@rafaelpimpa Absolutely, thanks for all the review! You as well, @jtommy!

Just a thought regarding the mobile version.... Wouldn't it be better to fallback to the native date input UI on touch screen devices? That's the direction that flatpickr have gone with and I feel that it makes a lot of sense.

@jgerbes We have just added that feature!

@rafaelpimpa It appears that the datepicker example is the docs isn't live: https://buefy.github.io/#/documentation/datepicker

The docs on the live site are only through v0.5.0 and the datepicker will be included in release v0.5.1, once that release is out they'll be live.

Is there a chance we could have a time element added to this? Sadly all of the other options with a time element all use bootstrap and I'd prefer to just use bulmar and buefy for everything inside my project.

@LiamDotPro We are working on it.


Use binaries from cmd/bin

We build subctl in cmd/bin, add that to the PATH instead of bin to
ensure we test the binary we're building.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

E2E (globalnet, 1.23, lighthouse)

Timed out, re-running


Implement anxcloud_vlan datasource

Description
Closes SYSENG-1281.
The schema is quite hacky and should definitely be reviewed. It is working, it's just not "clean". ^^
The VLAN data source test can be run with the following command:
# requires that the ANEXIA_TOKEN is set in the environment
TF_ACC=true go test ./anxcloud/ -run TestAccAnxCloudVLANDataSource

Checklist

[x] added release notes to Unreleased section in CHANGELOG.md, if user facing change
[ ] tackle codeclimate (sighs)
[x] merging schemas if possible

References

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

I refactored the code a bit and now it's ID > name. I think that makes the actual code a bit easier to understand, although Codeclimate is still not happy about the high number of returns. :/
Therefore I added just another commit which splits up the code a bit. Not sure whether that actually helps with readability, but if it satisfies the linter, well, yeah, let's do it then.

I think your changes in provider.go got lost while refactoring. That's why code lint fails.
Please also add an example and generate docs for that data-source.

I think your changes in provider.go got lost while refactoring. That's why code lint fails.

oh heck, sorry for that! And thanks for the suggestion with the diagnostics, now CodeClimate should be happy. :+1:

LGTM :+1:


Revise handling the application settings/configuration

[x] Fix not to use the singleton pattern
[ ] Implement INotifyPropertyChanged

[x] Fix not to use the singleton pattern

Done by #97.

[x] Fix not to use the singleton pattern

Done by #97.

[x]  Implement INotifyPropertyChanged

Done by #109.


High CPU on determinate progress bar

This page appears to consume between 75% and 100% CPU when determinate progress bar is used.

Do you mean the indeterminate progress bar?


Initial playwright setup

Reference Issues or PRs

First setup of #1653
What does this implement/fix?
Put a x in the boxes that apply

[x] New feature (non-breaking change which adds a feature)

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?

A big challenge that I don't have a great solution for is sometimes jlab opens with a popup already open (requesting a kernel for a notebook) and sometimes it doesn't. Since its a popup, the only thing we have access to are the popup dialog items. The only solution I can find is to wrap a try/except around getting a popup dialog item. If it doesn't exist, it will fail and that's the only way I have to know that the popup doesn't exist. Not great, but I tried many other options and nothing worked.

@kcpevey I'd like to split this PR into smaller PRs after we merge this initial PR. We should limit the scope to replacing only the cypress tests as an MVP. If possible too I'd like to minimize the number of abstrations we create. Then we work on adding each smaller feature. This PR as is just isn't well scoped and hard to finish in a certain amount of time.

@costrouc I completely agree. That's why I was hoping to get this first one merged before adding more tests. This just shows what I was hoping would be one "easy" integration test. I will add more details in the README on my vision of how the rest of the integration test suite would be filled out. I don't intend on removing cypress until we get a bunch of other PRs in and we're sure this is stable.

@costrouc @iameskild

When should the playwright tests run? I was thinking after a merge to develop or main?
Where should they run? I'm guessing after a test deployment of the current branch? Where can I find that? and the URL for it?

@kcpevey I'd like the playwright tests to run in https://github.com/nebari-dev/nebari/blob/main/.github/workflows/kubernetes_test.yaml#L165-L169

@kcpevey I'd like the playwright tests to run in https://github.com/nebari-dev/nebari/blob/main/.github/workflows/kubernetes_test.yaml#L165-L169 and remove https://github.com/nebari-dev/nebari/blob/main/.github/workflows/kubernetes_test.yaml#L138-L163.

Thanks! I think we should keep cypress for now and run both test suites until we have full coverage of the cypress tests with the playwright tests. Would you agree?

Need to add asserts after each major step (or more often) to make errors easier to track. I just had a failure on the My Server try/except but it was because of an incorrect username/password on the previous page.

I see that this is failing CI. It was passing ((see here)[https://github.com/nebari-dev/nebari/actions/runs/4755557238/jobs/8449860728]) before I made an unassociated change to azure tests, which means the test is still flaky 🤯. I will sort that out.

Ok, all the test pass except the k8s test failure which is also on develop (https://github.com/nebari-dev/nebari/actions/runs/4781817203/jobs/8511173379). Ready for review.

We need to at least resolve https://github.com/nebari-dev/nebari/pull/1665#discussion_r1190893467 before merge. Some documentation is missing and possibly some workflow improvements for better DevX. There are a few more open comments, but nothing blocking.

I'm going to try to carve out some time to get this wrapped up this week.

@kcpevey @pmeier @iameskild  @costrouc  I have changed the git clone to write file on the nebari instance via bash using cat and EOF to remove the dependency of having tests in a branch on git. I think this pull request is in good shape to be merged. I wanted to wrap this up as I might need this for creating some testing setup. Let me know if anyone have any objections/reviews/comments.

@aktech Thank you for taking this over! As for the reliance on a branch, that was just to show tests passing and not intended to be merged. Thank you improving things and pushing things through!


[CORE][BRANCH-1.2] Port #7861 to fix OOM in shuffle writer

What changes were proposed in this pull request?
Port #7861 to fix OOM in shuffle writer.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

I cherry pick GHA fix from https://github.com/apache/incubator-gluten/pull/8075 to tigger GHA. I will revert it later.

@ccat3z, the pr for fixing CI has merged. Please do a rebase.


[1Password] Possibility to copy username, password and OTP from Raycast

Extension
https://www.raycast.com/khasbilegt/1password
Description
It would be great is we could copy username, password and OTP for a specific 1P entry directly from Raycast instead of Raycast opening 1P to that entry. Would something like this be possible?
Who will benefit from this feature?
Everyone that uses 1P ;-) It will provide much more flow and speed!
Anything else?
No response

Duplicate of #4626


Postgres timestamp column type

Description

Steps to Reproduce

Create table with column with type timestamp
Use cli to generate struct
Insert a value

Behavior
Query Error: error occurred while decoding column "created_at": mismatched types; Rust type `core::option::Option<chrono::datetime::DateTime<chrono::offset::utc::Utc>>` (as SQL type `TIMESTAMPTZ`) is not compatible with SQL type `TIMESTAMP`

Reproduces How Often
Always
Versions
Postgres 14.1
├── sea-orm v0.6.0
│   ├── sea-orm-macros v0.6.0 (proc-macro)
│   ├── sea-query v0.21.0
│   │   ├── sea-query-derive v0.2.0 (proc-macro)
│   ├── sea-strum v0.23.0
│   │   └── sea-strum_macros v0.23.0 (proc-macro)

Hey @hf29h8sh321, thanks for the report! I guess you are on PostgreSQL database?

I could not reproduce the error.

PostgreSQL 13 table schema

create table satellite
(
    id              bigserial
        primary key,
    satellite_name  varchar                  not null,
    launch_date     timestamp                not null,
    deployment_date timestamp with time zone not null
);

SeaORM entity

use sea_orm::entity::prelude::*;

#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "satellite")]
pub struct Model {
    #[sea_orm(primary_key)]
    pub id: i32,
    pub satellite_name: String,
    #[sea_orm(default_value = "2022-01-26 16:24:00")]
    pub launch_date: DateTimeUtc,
    #[sea_orm(default_value = "2022-01-26 16:24:00")]
    pub deployment_date: DateTimeLocal,
}

#[derive(Copy, Clone, Debug, EnumIter, DeriveRelation)]
pub enum Relation {}

impl ActiveModelBehavior for ActiveModel {}

Insert operation

pub async fn create_satellites_log(db: &DatabaseConnection) -> Result<(), DbErr> {
    let archive = satellite::Model {
        id: 3,
        satellite_name: "Sea-00001-2022".to_owned(),
        launch_date: "2022-01-07T12:11:23Z".parse().unwrap(),
        deployment_date: "2022-01-07T12:11:23Z".parse().unwrap(),
    };

    let res = Satellite::insert(archive.clone().into_active_model())
        .exec(db)
        .await?;

    assert_eq!(archive.id, res.last_insert_id);
    assert_eq!(Satellite::find().one(db).await?, Some(archive.clone()));

    Ok(())
}

It seems like that column type is not compatible with struct field type

I've been looking at https://github.com/SeaQL/sea-orm/issues/586 and I've also encountered this error before, also using Postgres. If I remember correctly, the issue was that I declared the column type as a DateTime using ColumnDef::new(...).date_time(), which in turn threw that error during serialization of the entity. Changing that code to ColumnDef::new(...).timestamp_with_time_zone() fixed that error upon insertion. Could this possibly be related? I'm wondering if the underlying column type that gets created causes errors, either that or a weird bug when using the Option type with a DateTime
Originally posted by Galestrike#8814 on Discord

ColumnDef::new(...).date_time() will generate column type timestamp without time zone, aka timestamp in SeaQuery

https://github.com/SeaQL/sea-query/blob/5e72470e022719a3ce5df4f67fc9a8313c67355a/src/backend/postgres/table.rs#L64

Then, SeaSchema discover and write the column type as timestamp

https://github.com/SeaQL/sea-schema/blob/d4408a0cf4871ba66e6edc27f2528cc38730ab61/src/postgres/writer/column.rs#L108

Finally, SeaORM codegen write Rust type of timestamp column as DateTimeUtc

https://github.com/SeaQL/sea-orm/blob/0a26dee674a00cdda4cf3568f3d89a76fb5c8f57/sea-orm-codegen/src/entity/column.rs#L46

I believe the issue is related to a column without not null. Then, the type becomes Option<DateTimeUtc> which is Option<DateTime<Utc>>, which is probably not recognized as a timestamp without time zone.

I could not reproduce the error.

PostgreSQL 13 table schema

create table satellite
(
    id              bigserial
        primary key,
    satellite_name  varchar                  not null,
    launch_date     timestamp                not null,
    deployment_date timestamp with time zone not null
);

SeaORM entity

use sea_orm::entity::prelude::*;

#[derive(Clone, Debug, PartialEq, DeriveEntityModel)]
#[sea_orm(table_name = "satellite")]
pub struct Model {
    #[sea_orm(primary_key)]
    pub id: i32,
    pub satellite_name: String,
    #[sea_orm(default_value = "2022-01-26 16:24:00")]
    pub launch_date: DateTimeUtc,
    #[sea_orm(default_value = "2022-01-26 16:24:00")]
    pub deployment_date: DateTimeLocal,
}

#[derive(Copy, Clone, Debug, EnumIter, DeriveRelation)]
pub enum Relation {}

impl ActiveModelBehavior for ActiveModel {}

Insert operation

pub async fn create_satellites_log(db: &DatabaseConnection) -> Result<(), DbErr> {
    let archive = satellite::Model {
        id: 3,
        satellite_name: "Sea-00001-2022".to_owned(),
        launch_date: "2022-01-07T12:11:23Z".parse().unwrap(),
        deployment_date: "2022-01-07T12:11:23Z".parse().unwrap(),
    };

    let res = Satellite::insert(archive.clone().into_active_model())
        .exec(db)
        .await?;

    assert_eq!(archive.id, res.last_insert_id);
    assert_eq!(Satellite::find().one(db).await?, Some(archive.clone()));

    Ok(())
}

use timestamp without time zone type with sea-orm-cli would reproduce these problem

Hello!
I had this same issue:

I am using Postgres;
I have fields mapped with ColumnDef::new(..).timespamp();
The migration generate the correct field in the Database with type timespamp.

But then generated entities places DateTimeUtc and after I face this:
value: Query("error occurred while decoding column \"last_login\": mismatched types; Rust type `core::option::Option<chrono::datetime::DateTime<chrono::offset::utc::Utc>>` (as SQL type `TIMESTAMPTZ`) is not compatible with SQL type `TIMESTAMP`")

And I no longer can query those tables with timestamp.
I also had a LEFT JOIN that was returning Vec[] empty without crashing just because of that timestamp
A workaround I am testing is to change the DateTimeUtc to DateTime which is chrono::NaiveDateTime and I am filling that field with chrono::Utc::now().naive_utc() I still don't know it it causes any unwanted behaviour.
but now I can query those data, even the LEFT JOIN now works.
How is the status of this issue?

Hey @ricardoalcantara, please check @772


Jitsi recording fails when lobby is enabled

Bug Report
Expected behavior/code
I configure a live jitsi and I enable the lobby security. I want to start streaming and streaming should start
Actual Behavior
Jitsi recording fails. Jibri is enabled to join jitsi room.
Steps to Reproduce

Configure a jitsi live
Once jitsi initialized go to security options and enable lobby security
click "start streaming" button

This has been fixed directly on the Jitsi instance we are using.
For the records, the prosody configuration should whitelist jibri.
This is done by defining a muc_lobby_whitelist in the main virtualhost of the jitsi instance:
VirtualHost "{{ meet_fqdn }}"
        [...]
        muc_lobby_whitelist = { "recorder.{{ meet_fqdn }}" } -- Here we can whitelist jibri to enter lobby enabled rooms


Updated Contributors.md

Updating the contributors' markdown #5

@all-contributors please add @ravi-prakash1907  for content

@all-contributors please add @ravi-prakash1907 for content

@all-contributors please add @kraken-sudo for content

@all-contributors please add @ravi-prakash1907 for content, data, code, design


How can I check the classified result?

I have more questions. I also run the AdGraph in VMware(Ubuntu 16.04).
document.createCDATASection('NOTVEYUNIQUESTRING');
Does analysis including ml classifier end just by entering the above script into the chromium's dev console? Or is only the graph representation stored?
If feature extraction and ad/non-ad classification are completed, where can I check the results?
Thanks to, @UmarIqbal

It only stores the graph representation. I have attached the code to process the raw logs (rendering streams). You can use it to extract features for each webpage. Once you generate the feature file, you can use any ML framework/library to train the model.
I used WEKA to train the Random Forest model and I believe the attached folder has some commands to process the feature file with WEKA. However, feel free to use any library, e.g., Scikit, that you're comfortable with to train the model.
Link to code (it will expire in 2 days): https://www.dropbox.com/scl/fi/eg22gqprp5spisfa4jk4m/AdGraphAPI-master.zip?rlkey=k1cig4ctzt3gsj5maxib48g9l&dl=0
Feel free to also send me emails (I am more responsive on email and do not check the GitHub issues regularly). My email is<EMAIL_ADDRESS>


[2024-12-19T15:17:34 ERROR govee::commands::serve] while polling House Backyard Lights (DB:8E:C3:36:35:30:49:80 H705C): get_device_state: request https://openapi.api.govee.com/router/api/v1/device/state status 401: Unauthorized. Response body:

Govee Device SKU
H705C & H706C
Govee2MQTT Version
2024.07.13-82ddc6e9
Describe the issue
What is the subject error message referring to?
Startup Diagnostics
[2024-12-19T15:17:34 ERROR govee::commands::serve] while polling House Backyard Lights (DB:8E:C3:36:35:30:49:80 H705C): get_device_state: request https://openapi.api.govee.com/router/api/v1/device/state status 401: Unauthorized. Response body:
Additional Logs
No response
Home Assistant Logs
No response
Anything else?
I think this started after the 2024.12.3 patch but I'm not sure.  Everything was working fine at the end of November before I upgraded.  All of my other govee lights ( net lights, string lights, window lights, cone Christmas tree ) work fine.  It's just the permanent house lights giving me problems

Having the same exact issue as well that just started today with my Govee smart plugs


Corrupted thumbnails

Describe the bug
Sometimes, some thumbnails appear to be corrupted on the bluesky timeline.
To Reproduce
Steps to reproduce the behavior:

Go to https://bsky.app/profile/seriesbrasil.com.br/post/3l3qkyrb4l22s
Wait for thumb 1 to load

Expected behavior
Thumbails to load correctly.
Screenshots

Details

Platform: Windows 11, Firefox 130

Additional context
image url of example: https://cdn.bsky.app/img/feed_thumbnail/plain/did:plc:jt72xztklu3d2chvll2hizr6/bafkreihxox5g6hz457uf7j7rttptxrs34jnewtcto5eyystvhvmnaqrp7a@jpeg

I can't always reproduce this issue.

I think this was a issue with my connection, so I'm closing.


Move mock binary to objects crate

ref: https://github.com/0xPolygonMiden/miden-base/pull/227/files#r1325700327

Superseded by #250.


Use ISO 639-3 and ISO 15924 codes

Is your feature request related to a problem? Please describe.
It is mentioned at https://github.com/jzohrab/lute-v3/issues/116 that a plug-in system and potentially many more stuffs need to be done with unofficial support. Therefore a standarisation on the identifier of the language and writing system is needed.
Describe the solution you'd like
Use ISO 639-3 and ISO 15924 to indicate languages and writing systems.

Thanks for the note.  Can you provide good links for the above?  I'm also not sure where this info should be used.

https://iso639-3.sil.org/code_tables/639/data
https://www.unicode.org/iso15924/iso15924-codes.html


SBT incremental compilation bug if used with unmanagedResourceDirectories

steps
Automatically on Linux/Mac:
git clone https://github.com/vn971/sbt-incremental-bug
cd sbt-incremental-bug
./reproduce.sh
Or manually, check out README in the repo linked above.
problem
The problem appears to be with incorrect incremental compilation caches IF using the unmanagedResourceDirectories property in build.sbt file (see repo)
expected
As shown in README and ./reproduce.sh of the git repo, SBT should not be able to compile the project at the last step.
notes
The bug was discussed on IRC, confirmed (and reproduced) by "OlegYch|h" (thanks again for helping!).

// StackOverflow question: http://stackoverflow.com/questions/40765221/broken-sbt-compilation-bug-or-feature#comment68759140_40765221

I wonder if watchSource is getting confused.