Disable picture in picture (PIP) on Android TV

I've tried all different display modes on different devices and it seems PIP is displaying as an option on Android TV and there is no way to disable it. Is this normal or are the player options not displaying correctly?
The reason I ask is that the button is easily pressed in error and there is no way to escape PIP on Android TV without force closing the player
If this is how it is intended, can you please consider including an option to disable PIP🙏
Thanks

The PiP functionality is a standard Android OS feature. Closing/dismissing may be altered by the device manufacturer. On my devices and the emulator, I can switch and dismiss the PiP from controls on the home screen. The PiP should just work the same as in other apps offering PiP functionality.

Thanks for the reply @moneytoo.  As you can see here I don't get any such controls on the home screen of the very common Android TV device NVIDIA Shield Pro 2019. The only option is to force close the app from the app settings menu

Switching the the app should also dismiss PiP. It should really be no different than with other apps (you may try for example PiP in Youtube/SmartTubeNext).

Switching the the app should also dismiss PiP

I already showed the app switcher. There is nothing there..

Stock Android TV doesn't include app/task switcher so normally would need to select it from home screen/launcher. What happens when you do that?

Stock Android TV doesn't include app/task switcher

NVIDIA Shield Pro 2019 definitely includes an app switcher but Just Player does not show. There is nothing there

you may try for example PiP in Youtube/SmartTubeNext

This is SmartTubeNext in the app switcher

select it from home screen/launcher. What happens when you do that?

Selecting the app from launcher/app settings will restore the player again but this is clumsy. A setting to turn off PIP would be a better option. Please consider adding this

Unfortunately I'm unable to reproduce this issue and therefore to do anything with it.


HUNTER_DOWNLOAD_SERVER for custom download directory location

Use HUNTER_DOWNLOAD_SERVER variable to specify custom location of directory with downloaded archives having same format as ${HUNTER_ROOT}/_Base/Download. Example:

default location: https://downloads.sourceforge.net/project/boost/boost/1.61.0/boost_1_61_0.tar.bz2

downloaded to ${HUNTER_ROOT}/_Base/Download/Boost/1.61.0/f84b1a1/boost_1_61_0.tar.bz2

if HUNTER_DOWNLOAD_SERVER=https://example.com/hunter-downloads then effective URL became https://example.com/hunter-downloads/Boost/1.61.0/f84b1a1/boost_1_61_0.tar.bz2

HUNTER_DOWNLOAD_SERVER with a local path can be useful for:

testing, multiple HUNTER_ROOT directories; one git-base, another "normal" one; both have shared Downloads directory
read-only network location

We need extract file name from given URL. Can have problems similar to https://github.com/ruslo/hunter/issues/455.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Fixed by https://github.com/ruslo/hunter/pull/1084


API.update_status() problem in example code

It seems that the API.update_status() function has changed however the example code hasn't.
And It causes a 400 Bad Request.

:+1:  Thanks


Develop

You can watch the presentation of our project here.
In this PR, we have:

Created Item class and all the properties given UML diagram;

Implemented can_be_archived? and move_to_archive methods for Item class;

Created a Book class in a separate book.rb file.

Implement can_be_archived? method for Book class by overloading method in superclass

Created a Label class with an association to the Item class (in a separate label.rb file).

Implement add_item method for Label class

Created a Game class in a separate game.rb file.

Implement can_be_archived? method Game class by overloading method in superclass

Created a Author class with an association to the Item class (in a separate author.rb file).

Implement add_item method for Author class

Created a Musicalbum class in a separate music_album.rb file.

Implement can_be_archived? method for Musicalbum class by overloading method in superclass

Created a Genre class with an association to the Item class (in a separate genre.rb file).

Implement add_item method for Genre class

Created PreserveData module for JSON to Ruby object conversion funcionailities

Created Console module for handling user input/output

Created a main.rb file that will serve as the console app entry-point.

Implemented startup actions:
- Presented the user with a list of options to perform.
- Let users choose an option.
- Gave a way to quite and save data in json files in the app.

Created Unit tests for Book, Game, Musicalbum, Label, Author and Genre classes

Added SQL script for creating the books, games, musicalbums, labels, genres, authors database tables with one-to-many relationships and adding indices to all foreign-keys in books, games, and music_albums tables

Hi @eri8-9 :wave:
thanks :pray:


[K12] Adding  All Application List View and Page Layout

Note: FOR INTERNAL USE ONLY. For support questions, please use the EDA Chatter group in The Power of Us Hub.
For feature requests, please use the Ideas tab in The Power of Us Hub.

Do you want to request a feature or report a bug?
feature
What is the current behavior?
Application object's page layout is defaulted to EDA
What is the expected behavior?
Application object's page layout should be defaulted to a K12 version
What is the motivation / use case for changing the behavior?
data integrity

**lurch: attach W-038892

Tracking W-038892


Glob vs regex for include/exclude-filepath options?

Currently, the exclude-filepath option uses regex. This mainly because this is how a contributor originally implemented it, and nobody has brought it up after that. After a user report/request, I am adding an include-filepath option to add local paths to the local file scanning for at least the sync scripts (#70). It's caused me to think about these options a bit, and I'm thinking that regex makes the simple cases more complex than they should be. So I'm considering switching to globbing for these options. This would also make a future change to using pathlib a bit easier.
My main question for anyone is, do you have a use for the exclude-filepath option or a future include-filepath option that couldn't be done with globs but can be done with regular expressions?

Sorry for the incredibly late reply. I'll say I'm OK with globs, but my use case is pretty basic.
Do you have any command examples you can post so I can get an idea of what to expect? My command is still the same:
gmdownload "/mnt/%albumartist%/%album%/%track2%. %title%"

Your use case wouldn't be affected. This is for the already existing --exclude option and the upcoming inverse of that for #70. The exclude option was added by someone else and used regex. I'm not sure if they actually needed regex or just happened to use it. Regex provides more power, but glob is the traditional thing to do with paths. I'm just wondering if anyone actually has a more advanced use case where regex is useful.


Youtube Adblock

Hello,
Can you please add this host file to your server for help block ads in youtube?
IPV4: https://ewpratten.github.io/youtube_ad_blocklist/hosts.ipv4.txt
IPV6: https://ewpratten.github.io/youtube_ad_blocklist/hosts.ipv6.txt
Thanks!

Hi @phanluchoacom , thanks for the report. I appreciate every user suggestion and feedback we get. There's always something to be gained.
I've given this some thoughts, and I think @Tomatoide is right in this case. I took a look at the list, and it seems to just list out various *.googlevideo.com domains. While they may contain youtube ads, they also potentially contain regular youtube videos or other google videos content.
I am worried that adding the list will break some regular content for users.
However, I'm willing to test this out! I'm setting up jp-dns3 server.
jp-dns3.bancuh.com
ipv4: <IP_ADDRESS>
ipv6: 2400:8902::f03c:94ff:fe29:1a27

Please try it out for a few days and let us know if it works!

I still see ads. I guess the DNS blacklist doesn't work for YouTube.
;rr3---sn-8pcv-u2xk.googlevideo.com.IN TYPE65
;rr3---sn-8pcv-u2xk.googlevideo.com.IN A
rr3---sn-8pcv-u2xk.googlevideo.com. 600 IN CNAME rr3.sn-8pcv-u2xk.googlevideo.com.
rr3.sn-8pcv-u2xk.googlevideo.com. 600 IN A <IP_ADDRESS>

Hi @phanluchoacom, @CasanierD , @Tomatoide
Thanks for all your contribution and testing efforts. I think we can conclude that this won't work very well.
I'll close this issue and shut down sg-dns3.


Feature/5967 extended analytics

What
Added extended analytics tracking when users download a file as part of the frontend-dataset-controller
https://trello.com/c/zkeB3QbA
How to review
Sense check
Tests pass
Who can review
Golang frontend. !me

I've decided not to do anything about the error - it was bubbling up from json.Marshal which since it is only marshalling a simple map won't fail


How to write the  .bmp images from tensors to disk after resizing ?

I am trying to read bmp images, resize them and write the image to disk using tensorflow. I have succeeded in reading it but unable to find a way write it to disk. Any idea how to do it ?
Here is the code below:
import tensorflow as tf

img_path = "D:/image01.bmp"

img = tf.read_file(img_path)

img_decode = tf.image.decode_bmp(img, channels=1) # unit8 tensor

IMG_WIDTH = 256

IMG_HEIGHT = 256

img_cast = tf.cast(img_decode,dtype=tf.float32)

img_res = tf.image.resize_bilinear(img_4d, (IMG_HEIGHT, IMG_WIDTH), align_corners=True)

The problem is I can't find a "encode_bmp" or any bmp related function that can be used to encode the image and save the resized image to disk.

@gadagashwini I tried reading, preprocessing images with tensorflow APIs. e.g. JPEG, PNG etc. I see a encode function for each formats but not the same for "BMP". Having that option to encode it back will help.

@mohapatras Thanks for finding this.

@mohamedadaly I started and contributed the bmp decoder so that I can test TensorFlow in command line programs on Android (Android image decoding APIs are Java APIs. Not easy to use them in command line C++ programs). So, yes, if nobody contributed a decoder for bmp, maybe you can start one :-)

@freedomtan I think "decoder for bmp" should be encoder.

@mohapatras yes, updated :-)

@ymodak Any solution yet on encoding bitmaps ?

To be honest, TensorFlow seems overkill for the flow described here. If you only want to programmatically resize images you can use ImageMagik or Python's PIL.
I think adding an encoder for bitmaps to TensorFlow is very low priority, but PRs are always welcome.

@mohapatras I added encode_bmp in tensorflow-io PR https://github.com/tensorflow/io/pull/539, you can take a look. If the API is stable enough later on we can push to TensorFlow core repo if needed.

@yongtang Sorry for the late reply. I will look into it. Thank you.

@mohapatras,
Can you please let us know if this PR has resolved this issue so that close it? Thanks!

@rmothukuru Yes the issue is resolved. Sorry for the late reply.


Bailout on undeclared types

Prerequisites

[x] I have checked the existing issues to ensure this is not a duplicate.
[x] I am running the latest version of Tailcall.

Describe the bug
At the time of schema validation, whenever there is a type that's not defined in the schema/config, tailcall server should bailout and display and error message with a proper trace.
Steps to reproduce
Use the following graphql config
schema @schema {
  query: Query
}

type Query {
  users: [User] @http(baseURL: "https://jsonplaceholder.typicode.com", path: "/users")
}

Perform a check command
tc check ./assets/jsonplaceholder.graphql
No errors found
N + 1: 0

Expected behavior

[ ] A validation error should be shown — `Undeclared type 'User' was found [@ Query, users]
[ ] tc start should not panic

Actual behavior

No validation error is thrown
Server start panics and fails.

Environment information:

All envs.

/bounty $100

/attempt #334

fixed in #338


Log sink and pub/sub topic not working when created in different projects

This is the same issue as issue reported in old repo.
Problem
In GCP console, when a log sink export is created, there is a way to choose custom destination where the destination can be a Pub/Sub topic in another project.
However, replicating the setup in deployment manager using logsink templates, does not create the pubsub in different project.
It only shows the configuration values as different project. Deploying them still creates everything in DM project.
Configuration
I tried using the project and projectId properties with another project ID but still it creates the sink and the pub/sub topic in the same DM project.
resources:
- name: test-pubsub
  type: logsink.py
  properties:
    project: pubsub-project-id
    projectId: sink-project-id # This seems to be used for getting value for sink's parent property.
    destinationName: pubsub-topic
    destinationType: pubsub
    # Rest of the config below like pubsubProperties.

Expanded config in deployment manager for log sink shows:
 parent: sink-project-id
 destination: pubsub.googleapis.com/projects/pubsub-project-id/topics/pubsub-topic

Executed using:
gcloud deployment-manager deployments create deployment-logsink --project=sink-project-id --config=config.yaml
I expected the pubsub topic to be created in pubsub-project. But it gets created in same DM project. Only the expanded config in DM UI shows different project values.
Thanks for responding in the old issue. However it seems to happen in the new version of code as well.

This is a kown pubsub bug, the patch is under internal review, I will update this issue when fixed.

This bug was fixed by the product team recently. It will be rolled out to the public during the coming days.


master does not compile

This is probably a consequence of #652.
Calling bin/lurk on master causes the error
error[E0277]: `bellpepper_core::num::AllocatedNum<F>` doesn't implement `Debug`
  --> src/circuit/gadgets/data.rs:30:5
   |
19 | #[derive(Clone, Debug)]
   |                 ----- in this derive macro expansion
...
30 |     pub thunk_tag: AllocatedNum<F>,
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `bellpepper_core::num::AllocatedNum<F>` cannot be formatted using `{:?}` because it doesn't implement `Debug`
   |
   = help: the trait `Debug` is not implemented for `bellpepper_core::num::AllocatedNum<F>`
   = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: `bellpepper_core::num::AllocatedNum<F>` doesn't implement `Debug`
  --> src/circuit/gadgets/hashes.rs:23:5
   |
20 | #[derive(Clone, Debug)]
   |                 ----- in this derive macro expansion
...
23 |     digest: AllocatedNum<F>,
   |     ^^^^^^^^^^^^^^^^^^^^^^^ `bellpepper_core::num::AllocatedNum<F>` cannot be formatted using `{:?}` because it doesn't implement `Debug`
   |
   = help: the trait `Debug` is not implemented for `bellpepper_core::num::AllocatedNum<F>`
   = note: this error originates in the derive macro `Debug` (in Nightly builds, run with -Z macro-backtrace for more info)

Removing Cargo.lork did the trick. Mine wasn't using an updated version of bellpepper


Same issue with Lenovo Yoga Pro 9 14IRP8

Hi.
I really appreciate your work. I have the same issue with the speakers on the 2023 model of the Pro 9. I already tried to fix it by applying https://github.com/PJungkamp/yoga9-linux/blob/main/config/etc/modprobe.d/snd.conf. But apparently, the speakers are wired a bit differently.
What steps do you recommend to get this working? I'm asking because you already fixed it before the fix was applied to the mainline.
[qdrop@qdrop-fedora-pro9i ~]$ neofetch
             .',;::::;,'.                qdrop@qdrop-fedora-pro9i
         .';:cccccccccccc:;,.            ------------------------
      .;cccccccccccccccccccccc;.         OS: Fedora Linux 38 (Workstation Edition) x86_64
    .:cccccccccccccccccccccccccc:.       Host: 83BU Yoga Pro 9 14IRP8
  .;ccccccccccccc;.:dddl:.;ccccccc;.     Kernel: 6.2.14-300.fc38.x86_64
 .:ccccccccccccc;OWMKOOXMWd;ccccccc:.    Uptime: 26 mins
.:ccccccccccccc;KMMc;cc;xMMc:ccccccc:.   Packages: 2129 (rpm), 32 (flatpak)
,cccccccccccccc;MMM.;cc;;WW::cccccccc,   Shell: bash 5.2.15
:cccccccccccccc;MMM.;cccccccccccccccc:   Resolution: 3440x1440
:ccccccc;oxOOOo;MMM0OOk.;cccccccccccc:   DE: GNOME 44.1
cccccc:0MMKxdd:;MMMkddc.;cccccccccccc;   WM: Mutter
ccccc:XM0';cccc;MMM.;cccccccccccccccc'   WM Theme: Adwaita
ccccc;MMo;ccccc;MMW.;ccccccccccccccc;    Theme: Adwaita [GTK2/3]
ccccc;0MNc.ccc.xMMd:ccccccccccccccc;     Icons: Adwaita [GTK2/3]
cccccc;dNMWXXXWM0::cccccccccccccc:,      Terminal: gnome-terminal
cccccccc;.:odl:.;cccccccccccccc:,.       CPU: 13th Gen Intel i9-13905H (20) @ 5.200GHz
:cccccccccccccccccccccccccccc:'.         GPU: NVIDIA GeForce RTX 4060 Max-Q / Mobile
.:cccccccccccccccccccccc:;,..            GPU: Intel Raptor Lake-P [Iris Xe Graphics]
  '::cccccccccccccc::;,.                 Memory: 5412MiB / 31818MiB

I captured the driver initialization sequence from a Windows VM. Then I replayed the initialization using the alsa-utils, bisecting the 2000 "verbs" (commands used in the HDA protocol) until I found the minimum required to get the speakers working. This took me two weeks of messing with qemu from random git repos online, reading the HDA standard to understand what I'm working with and a lot of trial and error. A lot of that was pretty useless in the end as there are a lot of similar cases documented online.
Did you check out this thread on the kernel bugzilla?
https://bugzilla.kernel.org/show_bug.cgi?id=208555
There are a lot of great hints buried between all those reports, mine included. (lately a lot about AMD versions of these laptops, just skim through it to find hints on Intel versions)
If you want to start tinkering check this post.
https://docs.kernel.org/sound/hd-audio/notes.html
Newer intel platforms use the snd-sof driver instead of the snd-hda-intel one, the sof driver is required for mic support but does not have all the debugging capabilities of the snd-hda-intel one.
Start by reading up on other peoples experiences, then check out documentation on the kernel driver and debugging facilities.
Try to use patch files and sysfs attributes to replicate the fixes for other laptops.
Pay special attention to the pin configuration and hda-verbs for initialization.
Only then start tinkering with the kernel when you've got some initial success. Search for the patch_realtek.c file in the kernel sources, where all the fixes for realtek codecs reside.
You can contact me by mail to get more details!

Hi!
I have faced the similar problem with Lenovo Yoga Pro 7 14IRH8. It was unable to control speaker volume using ALSA with sof-firmware. Sometimes volume control accidentally appeared after configuring Bluetooth headset but it was very unstable.
The final solution I used is quiet similar to this repo's. Just one line in snd.conf:
options snd-sof-intel-hda-common hda_model=alc287-yoga9-bass-spk-pin
I have found the solution here, and mine setup's behavior is the same. However I use newer 6.2 kernel version shipped with arch.

Nice to know that there's a way to apply the hda model parameter to the snd-sof based drivers. I should probably take this up into this repo. The legacy driver non-hda-sof driver was already unable to handle the microphone of my model. I don't know whether the standard snd-hda-intel driver is working at all for newer devices.
@glebpinigin Thank you for sharing!

Hi! thanks @glebpinigin for the tip! I got my mic and bass speakers working properly now!!

As another owner of the Lenovo Yoga 9 14IRPB, I want to add Herzlichen Dank!_ to @PJungkamp and the others who contribute this great work, and test.  (My Yoga died yesterday - some kind of no-charge/no-power/no-boot situation.  Until that is repaired, I won't be able to test.)  It had been running 23.04 quite well, excluding the issues discussed here.

I got my speakers working at Manjaro, with linux 6.6.

Use options snd-sof-intel-hda-common hda_model=17aa:38be (or whatever is your model, which you can check with alsa-info, but my guess is 17aa:38be for the Slim Pro 9i and 17aa:38bf for Yoga Pro 9i).
Move the firmwares from https://neo-zeon.de/~hiryu/tas/firmware-6.6.tar.bz2 to /lib/firmware/ and reboot, as described at https://bugzilla.kernel.org/show_bug.cgi?id=208555#c788 (I think these firmwares were already merged into linux, but I'm not sure if 6.7 will already have them)

I don't know if both are needed, so it would be useful to know what is exactly needed.
Also, I'm using KDE and the speakers only work if I first switch to "Play HiFi quality Music" profile, and I can only control the volume with the "Pro Audio" profile. Not sure how can I fix it and if this issue happens with Gnome too, so please tell me if you try it.
Finally, I think the repository https://github.com/darinpp/yoga-slim-7 contains useful things that would probably be helpful here (I'm almost sure the binary firmware there is from the same source).

The HPD sensor is a very simple HID sensor that I had been able to monitor using the hid-sensor-prox driver but a regression in Linux 6.6 an beyond broke hid-sensor-custom which is needed to find the sensor device... so with current kernels you're out of luck.
My goal was to have the HPD sensor be easily accessible using iio-sensor-proxy, but issues with the hid-sensor-prox code upstream and some internals in iio-sensor-proxy where a breaking change postponed MR had come to bite me.
I currently don't have too much time. Maybe I'll get back to it between semesters.

So the commits https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?h=sound-6.8-rc7&id=c1947ce61ff4cd4de2fe5f72423abedb6dc83011 and https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git/commit/?h=sound-6.8-rc7&id=0ac32a396e4f41e88df76ce2282423188a2d2ed0 that are gonna be in 6.8 (and maybe 6.6.X and 6.7.Y, not sure) should fix completely the sound issue with 14IRP8, including the weak sound and lack of volume control.
About the sensor, I think some patch was recently merged and your script find_sensors.fish is showing the sensor information strings again in 6.7.4 and 6.8. I still don't understand how to integrate iio-sensor-proxy, though. I enabled the one with the VL53L1_HOD sensor writing 1 manually to the enable_sensor file and succeeded to read the HPD data using your rust program. Is it the way to enable it or iio-sensor-proxy is supposed to do it? what about the other sensors in the list, like Lenovo ST HPD v201 Sensor and VL53L1?

Apparently after update to 6.8 kernel solution with editing snd.conf stopped working for me. Update to 6.10 didn't fix anything. My laptop version is Yoga Pro 7 14IRH8.

I have the Yoga Pro 9 16IMH9, the device in alsa-info is 17aa:3811 and unfortunately the solution described above doesn't work


Value.ReplaceType causes hang while using List.ParallelInvoke

Hi,
We are trying to use List.ParallelInvoke, in custom connector project, which calls Rest Api for fetching data.
We find out, that in case, when we use Value.ReplaceType
shared ParallelInvoke.Contents = Value.ReplaceType(ParallelInvokeImpl, ParallelInvokeType);  we get hang of our connector.
In case of  shared ParallelInvoke.Contents = () => NavTable() as table; it works fine.
Maybe you can help us, and tell, what can be the reason of this behaviour?
Please see attached example,  project where we reproduce our steps.
ParallelInvoke.zip
Thanks
@CurtHagenlocher  @cwebbbi  @fmms

Hi,
as I am working on this with @scorpionfly7 i played with it today as well and had several interesting findings that may help -- at least to me they were unexpected.
1. attachment of original comment
This works without any issue and after a second one gets 30 rows of result set.
2. adding a parameter to the Contents entry point
Just slightly modifying the example by adding a parameter:

and

This lets the query hang forever, without firing any HTTP requests.
3. removing the parallism
to make the connector work again, a one line code change is enough.

Very interested if this is by design and how we could still make it work,
regards
Felix

There's a lot going on here, but I'm not sure List.ParallelInvoke is the problem. Can you check what the web service you're using to test with returns when the connector hangs by using Fiddler?

There's a lot going on here, but I'm not sure List.ParallelInvoke is the problem. Can you check what the web service you're using to test with returns when the connector hangs by using Fiddler?

Hi @cwebbbi as soon as we pass a parameter to the constructor the connector hangs however, no HTTP request at all is issued. It seems to hang earlier.
Is it not reproducible for you?
Then we might have to share system configuration details. Though we can reproduce it on two machines. My machine details:

Win 10 1909
Visual Studio 2019 - 16.5.5
M Language 1.10

Anything else that might be helpful?

I've had a chance to look at this in more detail now, and I don't think it's anything to do with your system configuration either. I can reproduce the problem without using Value.ReplaceType or even the navigation table - I believe there's a deadlock happening somewhere. I'll let you know when or if I find a way to work around it.

@cwebbbi could you find any workarround? We will have to finalize this connector at the end of this week and would be happy to include support for parallelism

List.ParallelInvoke currently isn't supported for general consumption, and shouldn't be used by custom connectors.

@mattmasson Thanks for answer...Is there some alternative to do and manage parallel calls, that supported for custom connectors?


How to set page url outside app?

I'm sorry if this is covered somewhere, I can't find it.
I want to redirect my app to an authentication URL on user action, but when I use transitionTo or simply document.location.href=... then this gets intercepted and the app stays where it is. It works when I make it a simple link to click on.
How can I do the equivalent of clicking on a href?

That's what router.transitionTo is for. Where do you call it? Can you share some code or create an example where it fails?


Specify paths to migration files

This PR pulls in the awesome fix as suggested by @damnitjim for https://github.com/ajvargo/data-migrate/issues/18

any luck with this?

I think this PR would cover this case https://github.com/ajvargo/data-migrate/pull/25/files and add some more
Thanks for your PR @aakritigupta, it was from your changes that I figured out how to support mine. Could you try it out and see if that works for your cases as well?

When merged https://github.com/ilyakatz/data-migrate/pull/25/files, i believe this issue will be resolved. Hence, I'm going to close this. If there needs to be any followups, please open a new issue.
Thanks for reporting!


Fixed the markup extension for the SymbolIcon and FontIcon class not working at all.

Pull request type
Please check the type of change your PR introduces:

[ ] Update
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes

What is the current behavior?
Currently you cannot use a markup extension for symbolicons or fonticons because the constructor does not accept the arguments.

What is the new behavior?
The markup extension for the symbolicon and fonticon class are now working with some small issue.

Other information
So, I have now tested this extensively. I had previously closed this thread because for some reason the FontIcon markup extension randomly worked. However, I think that was due to VS (bugs and such).
I also confused the FontIcon class with the SymbolIcon class in the thread (sorry x.x).
So: The markup extension needs a standard constructor without arguments. Furthermore, the existing constructor does not accept 2 arguments.
Because of the bugs in VS, I wasn't sure before whether it was really a mistake in the markup extension
(that's why i closed the last pull request), but now I am. The SymbolIcon markup extension had the same problem.
The font size doesn't (forgot work), but it's not clear to me why.
Maybe you know why @pomianowski ?

@pomianowski I have no idea why this commit went into this pull request :s
Sorry for that, but i coulnd't even create another one.
So: As the arc class wasn't quite working so i thought "Why not? I do need it anyways.".
The arc class had a few problems including that the arc had to be re-inserted (as a control) when the size was changed because a resize was not taken into account when rendering.
That was the biggest problem that made me want to adapt these in the first place.
In the course of this, I thought it would be better for the user if he had the opportunity to edit both the sweepdirection and the penlinecap.
I also changed the starting position, from 3 a.m. to 12 a.m. I'll be happy to explain the reason for this:
Ultimately, it is important to the user that it is as easy to use as possible. This also applies to programmers. And what's easier than a clock that you've known since birth?
If you don't agree and/or are satisfied with it, it can always be changed^^

@m0lDaViA  I found a new "feature" while using the NavigationView in LeftFluent mode. If you are using a different Font like Montserrat (which is actually quite big), you end up with a cut-off string, even if you increase the item size manually. I think it might be the ContentPresenter...


fix: FileReader: Make a copy of the ArrayBuffer when returning partial results (3-0-x)

Backports https://chromium-review.googlesource.com/c/chromium/src/+/1492873

[x] PR description included and stakeholders cc'd
[ ] Patch information is added to appropriate .patches.yaml
[ ] script/update runs without error
[x] PR title follows semantic [commit guidelines]

Closing in favour or PR from electron/libchromiumcontent.


Sync components

Description

Adds the Synchronous Feature described here: #199

Type of change

New feature (non-breaking change which adds functionality)

How has this been tested?

Manual Tests (please write what did you do?)
I compared the data from the existing receiver with the data from this receiver and made sure they were the same.
I've made sure that only when the data were received the canvas become unlocked (and triggered downstream components)

Docs

Will be updated ASAP

Merged in! Thanks for you're hard work @psarras 🚀


Fix multipart parsing

When there's a LF format, and non-ascii characters the parsing doesn't
work.

This fixes #1438.

Older PR here https://github.com/mikel/mail/pull/1437

@mikel Hello? This is an obviously correct fix.

Thanks @felipec and @toofar and @sebbASF I've created a new PR against Master here https://github.com/mikel/mail/pull/1511 using the original PR that has the test coverage


Adding related challenges

Sorry, I didn't intend to do so many at once.  (I didn't realize how many I had worked on until I looked at the changed files.)
Updating/adding  related challenges for:
31,35,36,41,60,78,93,94,114,127,128,150,151,155,157,158,159,160,161,165,169.
Also edited ml5 language reference to be consistent and updated a couple of links.

Hey! Since #680 is now merged, can you also add 140-pi-leibniz to 21-mandelbrot-set-with-p5js?

Thank you!

Merging!


Path problem on Unix based FTP

General
On Unix based FTP servers using a path that contains '//' returns error code 553 :scream: While on Windows based FTP servers there is no error :clap:.
Solution
Using the following code to change the path to work with Windows and Unix :white_check_mark:

remotePath = remotePath.replace(/\\/g, '/').replace(/\/\//g, '/');

Enjoy :tada:

Why are you posting it this here? Are you having problems creating a directory? What kind of path does not work?

The path is something like this: aaaaa//bbbbb//ccccc path.
Windows based FTP handel this with no problem but Unix based FTP return error code 553.
causing jsftp-mkdirp to fail.
I posted it here so the issue will be known and either someone can use the info or solve the problem.

So why not just use a normal Unix path which from the looks of it will work on both?


Site down for Maintenance

Additional context
Server work is required for a rebuild of some systems, due to a hardware issue. This issue will be updated here as the matter progresses

System restore has been completed and initial networking is now up. Main services are still unavailable

Webserver restored - not all services for that proxy connected yet

Mail server is tentatively up - still testing

Mail push service is up

Webmail is up, but the full text indexing service is having issues there


Error: fields must be passed when initializing state

import React, { PropTypes } from 'react';
import { Field, reduxForm } from 'redux-form';

const Searchbar = ({ handleSubmit }) => (
  <form onSubmit={handleSubmit}>
    <Field name="q" component="input" type="text"/>
    <button type="submit" >Search</button>
  </form>
);

Searchbar.displayName = 'Searchbar';
Searchbar.propTypes = {
  handleSubmit: PropTypes.func
};

export default reduxForm({
  form: 'search'
})(Searchbar);

Saw no problems at all with what I thought was the minimum reproducible code: http://www.webpackbin.com/NyOCeUhSM
Can you elaborate? Include version numbers, and if possible enhance the above webpackbin with whatever you got in order to reproduce your issue.

thx. i installed the wrong version.

Out of curiosity @plexigras  what version did you have installed? We have started getting this error with 5.2.5

5.3.4 which i believe is or at least was the default provided by npm but the documentation was for 6.4.3
http://redux-form.com just redirected me straight to http://redux-form.com/6.4.3

This is some mess related to the latest releases.
Some fixes for people still using v5 were published as v5.3.4 after v6.4.3 was released.
I think @erikras recently tagged v6.4.3 as latest to avoid further problems with this.

I encountered this problem as of v5.3.4 up to v5.3.6, rolling back to v5.3.3 fixed it.


Image flows to the right and disappears [Laptops with touchscreen]

Steps to reproduce

Open incognito mode (no extensions)
Open imagelightbox demo page https://rejas.github.io/imagelightbox/
Open image in fullscreen
Move mouse
Image flows to the right and disappears

This bug seems to be happening only on laptops with touchscreen (tested 3 of them).
Can we expect a fix?

Question: How exactly do you move the mouse? And happens it with all demos on the page?

This is a issue with Touch-enabled Windows devices (maybe laptops?). Also for me.
If I hover on any of the images, the image will pop right. After hovering again, it will disappear.

@rejas I just got a chance to try it out with touch screen computer today.
Summary:

Filter images by extension part of https://rejas.github.io/imagelightbox/ - It does not open, however it seems to not work on my normal laptop too.
All example images tries to escape the mouse.
a) When you click on an image and if mouse is not hovered over image - image stays at the center of the screen. Once you hover over the picture with mouse - it tries to escape and runs to the right.
b) When you click on an image and mouse is already hovered over the image - image instantly runs to the right.

Touching the image on the screen (monitor) works ok.

Well, my laptop does not have touchscreen, but the problem occoured. It has something to do with latest Google Chrome versions or Windows 10 updates.
I fixed my issue with this:
https://github.com/osvaldasvalutis/imageLightbox.js/issues/3#issuecomment-269925816

Could you both try out v0.7.2 i just realeased?

@rejas I don't personally have touch screen computer, but user reported that problem is solved with v0.7.2.
Good job! Thank you.


[WIP/TESTING] docs: For Checksum filter

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

cc @ravenblackx testing here to get the ci - you can either pick this once its ready or i opened a pr accidentally on your pr, so i could update that

/docs


Javadocs are shown in an IFRAME. This breaks many functionalities.

What happened?
When viewing the javadoc of an artifact, the true javadoc page is displayed inside an IFRAME. This is presumably done so that Reposilite can put a banner above the javadocs.
The problem is that Oracle doesn't support this at all. The IFRAME breaks the search functionality of the javadoc because it tries to navigate the main window, which the page in the IFRAME has no access to (CORS). Fixed by building the javadoc using JDK19
IDE's which support html-importing of javadocs don't work with it because it expects a /index.html with the page contents. People can't copy the direct link to a relevant javadoc page to save for later or share with colleagues.
The current workaround is to visit the page, and then append .cache/unpack/index.html to the URL.
Bluntly put - Is it possible to turn this off? Ideally it would just put the javadocs directly at the URL, instead of a container page.
Weirdly, it seems javadocs including search do work for the reposilite project itself but I cant reproduce this behavior on my end.
Reposilite version
3.x
Relevant log output
No response

This is known Java bug that has been fixed lately in javadoc tool:

https://bugs.openjdk.org/browse/JDK-8268866

Custom index.html file was originally proposed by the old plugin, so we didn't change that, but we've left a possibility to request unpacked javadocs content just like you've said. We're using iframe for 2 reasons:

Add custom navigation header, so we can make navigation and extend it in the future with e.g. download link/version switcher
Embed javadocs in sandbox - there's a possibility that javadocs jar may contain malicious js script from e.g. mirrored repository. Of course it's not perfect, but provides basic protection for things like access to parent's localStorage etc.

Thanks, that does make a lot of sense. Im surprised it was fixed, because I picked the latest javadoc maven plugin version and the problem was still there. Then searched a bit and saw a bunch of oracle issue tickets relating to IFRAMES that were closed with 'WONTFIX'. Or is it not the javadoc maven plugin taht needs a version bump, but something else?
In that case I have two requests:

Some sort of direct link to this cache location on the javadoc page for people that are trying to import it into their IDE. Personally I just use the javadoc jar but apparently there is a need for this functionality.
Alternative to above: Why not put the true javadoc at /index.html and have / do the iframe? It seems that this is how the reposilite javadocs are hosted as this link also works
Some sort of #ref api in the page URL bar that shows where the IFRAME is navigated. But I'm afraid this isn't possible because reading of the IFRAME url is blocked by default for security reasons in browsers

Happy to report rebuilding the projects on Java 19 fixed the issues :+1:

Nice! Maybe we could do sth like:

Keep https://maven.reposilite.com/javadoc/releases/com/reposilite/reposilite/latest/ & add link to the raw version in header
https://maven.reposilite.com/javadoc-raw/releases/com/reposilite/reposilite/latest/ that'd be just the 1:1 jar content like with .cache/unpack/[...]

I think it could be relatively convenient for both cases.

I'm all for that. One problem I ran into was that the .cache directory isn't generated until the main / is called at least once. It also seems to have some background process to automatically remove it, maybe when a new snapshot is pushed. So in addition, it should generate the /javadoc-raw/ content automatically when a page under that url is first requested

I'm all for that. One problem I ran into was that the .cache directory isn't generated until the main / is called at least once. It also seems to have some background process to automatically remove it, maybe when a new snapshot is pushed. So in addition, it should generate the /javadoc-raw/ content automatically when a page under that url is first requested

This is required to obtain gav from request uri, so we can find related javadoc file. During first request for e.g. https://maven.reposilite.com/javadoc/releases/com/reposilite/reposilite/latest/a/b/c/d/file.html you're unable to say what's the groupid/artifactid/version.

IMO, having /latest/raw/index.html or /latest/unpacked/index.html might be a little cleaner though. Especially because to get there one first has to visit the /javadoc/ url anyway. (Unless you plan to add a second book icon in the repository for raw read? But dunno.)

The only problem with such approach is that it won't work if someone will have artifact called e.g. raw. It might be dumb to care about such niche, but this is something that we should be aware of. That's why in current impl, the location of this internal directory starts with .

What about /?raw=true?

Would you like to try to submit PR with such enhancement? :)

When I have the time for it I might. I'm a java developer primarily so the kotlin stuff is a bit new to me. Good enough at it to copy-paste stuff around and make tweaks, but not confident enough in it to redesign the network routing stuff.
For stability sake, probably better someone more knowledgeable picks that up. With search fixed, my primary concern is resolved so it has gotten less important

Sure, but just in case, it's open source and you can learn here, so if you'd decide someday to commit bigger changes in Kotlin to the repo, I'll try to guide you and highlight things we could improve :)
In the meantime, @Rollczi would you like to implement this?

I'm busy for a month. 😿


Non-string UserAgent can crash the app

Description

https://sentry.io/share/issue/e01331b2eb71498080208da9d62de7f1/
If either context.req.headers['user-agent'] or navigator.userAgent return a non-string value, the app crashes, because useragent.parse in 'express-useragent' expects a string argument and calls replace` on it.
This is the error you would see:
TypeError
Cannot read properties of undefined (reading 'replace')

Here is the code that causes the error:
https://github.com/WordPress/openverse-frontend/blob/5a3bed0dbab754e88749accc5a59477f7337583b/src/plugins/ua-parse.js#L3-L12
Reproduction

Add userAgent = null or userAgent = undefined to line 11 in the code above.
Run the app and try to open http://localhost:8443
See error.

Expectation

The app should not crash even if the user agent is undefined.
Additional context

I wonder why the user agent can be non-string?
Resolution

[ ] 🙋 I would be interested in resolving this bug.

I wonder why the user agent can be non-string?

My guess is some kind of privacy extension? :shrug:


bootstrap-select have a way to support manual input?

bootstrap-select have a method to support manual input?
Support drop-down selection and manual input at the same time？

This is a feature planned for a future update. Follow and 👍 #990 for more info.


Compilation.GetFullName assumes running on Windows

Hello there!
In my Godot C# project, I have the following (simplified) structure:
.../Alpha/MyObject.cs
.../Beta/MyObject.cs

When a TSCN file refers to Alpha.MyObject, then this logic comes into play.
It looks like this code is assuming we are always running on Windows, given that replaces "/" with "" in resource paths. We should probably use System.IO.Path.DirectorySeparatorChar instead.
Unfortunately I am not able to set up a PR right now, since a clean test build on Linux gives the following errors and I cannot verify that my fix won't break anything else:
  Godot 4 Tests failed with 4 error(s) (1.6s)
    /home/fed/gh/GodotSharp.SourceGenerators/Godot 4 Tests/.godot/mono/temp/obj/Debug/GodotSharp.SourceGenerators/GodotSharp.SourceGenerators.SceneTreeExtensions.SceneTreeSourceGenerator/GodotTests.TestScenes.SceneWithSameName.g.cs(42,21): error CS1520: Method must have a return type
    /home/fed/gh/GodotSharp.SourceGenerators/Godot 4 Tests/.godot/mono/temp/obj/Debug/GodotSharp.SourceGenerators/GodotSharp.SourceGenerators.SceneTreeExtensions.SceneTreeSourceGenerator/GodotTests.TestScenes.SceneWithSameName.g.cs(59,36): error CS0246: The type or namespace name 'root' could not be found (are you missing a using directive or an assembly reference?)
    /home/fed/gh/GodotSharp.SourceGenerators/Godot 4 Tests/.godot/mono/temp/obj/Debug/GodotSharp.SourceGenerators/GodotSharp.SourceGenerators.SceneTreeExtensions.SceneTreeSourceGenerator/GodotTests.TestScenes.Script.SceneWithSameName.g.cs(42,21): error CS1520: Method must have a return type
    /home/fed/gh/GodotSharp.SourceGenerators/Godot 4 Tests/.godot/mono/temp/obj/Debug/GodotSharp.SourceGenerators/GodotSharp.SourceGenerators.SceneTreeExtensions.SceneTreeSourceGenerator/GodotTests.TestScenes.Script.SceneWithSameName.g.cs(59,36): error CS0246: The type or namespace name 'root' could not be found (are you missing a using directive or an assembly reference?)

Thanks for that.  I'll have to setup a Linux environment for future testing.
For now, I've made the change.  Can you test on your side?
GodotSharp.SourceGenerators.2.5.1-241217-1202.Release.nupkg.zip

Ok, great.  I'll do a new pre-release nuget.  Cheers.


mac os x installation, can't access mysql.sock

after executing wp in wordpress directory:
Warning: mysql_connect(): [2002] No such file or directory (trying to connect via unix:///var/mysql/mysql.sock) in ~/Sites/wp/wp-includes/wp-db.php on line 1034
my mysqlsock is located in /tmp/mysql.sock
I run mysqld on-demand - open shell and run it for development, it is not always loaded
runing configuration is nginx-1.0.10/php-fpm-5.3.6/mysql-5.5.15

Ok, and do you expect wp-cli to automatically start mysqld, or what?

So, you have mysqld running.
WordPress works correctly in the browser, but not when using wp-cli?

yap, correctly in browser, not with wp-cli...
it says it can not connect to database.
though, got it working when I tell mysqld to create mysql.sock in /var/mysql/
mysqld --socket=/var/mysql/mysql.sock
could socket path be some kind of configuration option for wp-cli or something?

I think the problem is that your php5-fpm config differs from your php-cli config.
Besides, wp-cli already accepts a custom ini file to pass to php-cli:
https://github.com/andreascreten/wp-cli/blob/v0.3.0/src/bin/wp#L78

seems like you are right
mac os x Lion installation here
there is default /usr/bin/php (5.3.6) that uses /etc/php.ini
and there is /usr/local/Cellar/php/5.3.8/bin/php installed with brew (php-fpm goes through this one)
thx for help :)


Type Money adds zero to the left when you type fast

when you type fast in the input, the initial masked 00.0 allows to persist the zero to the left.
Depending of how fast you type numbers, 0s to the left are not totally removed.
like:
0154.54
00137.45

did you find a fix for this?

I have the same problem, was there any solution?


Implement Share Experiences: Experiences are stored in Django Database on Submission

Description
After a user has populated all the data fields for a Public Experience, submitted experiences should be stored on the Django database while adhering to the following requirements.
Definition of Done
When the user has clicked the Submit button on the Share Experiences page...

[x] Creation Date should be set to today's date
[x] Unique ID field should be populated with a randomly generated UUID.
[x] User ID field should be populated with the user's OpenHumans ID, which is obtained through the OpenHumans member model.
[x] Experience Text, Suggestion Text, Moderation Review Status, and Sharing permissions are all saved according to their current state on the web page.

Closed in #401


FIFT parameters are read as TONCLI parameters

.fif files can be configured via positional parameters + nominal parameters. Positional parameters work well.
Nominal parameters of FIFT dont work in toncli. Looking at the output, toncli thinks they're part of toncli nominal parameters while I want them to be FIFT parameters.
Example:
fift run .\fift\transaction.fif .\build\contract address 0 0 0.01 -C "test" Transaction
Gives
usage: toncli [-h] [-v] {start,deploy,get,send,tointeger,run_transaction,f,fc,lc,run,build,fift,lite-client,sendboc,wallet,run_tests,update_libs,func} ... toncli: error: unrecognized arguments: -C

Try to use --fift-args "..."

>toncli fift run .\fift\transaction.fif .\build\contract kQCMSwlKGQv5nQxGG5ci0M1i3_2yp4GiwjTEi6Mq7oNEhIWK 0 0 0.01 Transaction --fift-args "-v 4"
[ 4][t 0][2022-04-18 03:00:55.7492550][words.cpp:3115]  import_cmdlist_args(.\fift\transaction.fif,6)
[ 1][t 0][2022-04-18 03:00:55.7494717][fift-main.cpp:180]       Error interpreting standard preamble file `Fift.fif`: cannot locate file `Fift.fif`
Check that correct include path is set by -I or by FIFTPATH environment variable, or disable standard preamble by -n.

I use fift without problems but using --fift-args looks like requires me to configure FIFTPATH which i never did on windows. Idk if it is a bug or no so I report it

#14 duplication

#14 duplication

I can execute without problems toncli run_tests. It's probably a different thing

@SpeedK I guess this is global problem of FIFTPATH in windows version of toncli. As you said:
 --fift-args looks like requires me to configure FIFTPATH which i never did on windows


Use PEP426 environment markers for extras_require.

This pull request tries to address the issue #228 and introduces PEP426 compliant environment markers for the extras_require specification as proposed by @takluyver.

This looks correct to me.

Thanks so much, this looks something like what I expected too, although I really would like to get some automation before merging to make sure that this doesn't happen again. I can give that another shot hopefully tomorrow, unless you've got some ideas.
Also CC @dstufft just in case I can bother you into a consult for this, context is in #228.

I don't think the checking for bdist_ in args is needed: setuptools should handle the markers fine for egg_info, sdist and install too. I put up #231 before I knew this was here, it doesn't have the bdist_ checks.
The basic approach is here is fine, its exactly what the wheel package does.

So thanks again! I'm going to merge #231 only because the way that that looping over argv looks makes me uneasy. But I really appreciate everyone pitching in especially with the amount of time I have to think about this specifically :/


建议在鼠标实用工具增加鼠标外观的自定义功能

Description of the new feature / enhancement
比如使用自定义图片做为鼠标指针，给鼠标指针添加文字
Scenario when this would be used?
1、个性化
2、将作者名字添加到鼠标指针下。用于录制视频课程或Gif课程，每一帧都会显示作者名字，防止他人盗人。
Supporting information
No response

(English)
Description of the new feature / enhancement
For example, use a custom image as the mouse pointer to add text to the mouse pointer
Scenario when this would be used?

Personalization
Add the author's name to the mouse pointer. For recording video courses or Gif courses, the author's name will be displayed in every frame to prevent others from stealing.

/dup https://github.com/microsoft/PowerToys/issues/9008


TypeError: this.$options.events[key].bind is not a function

Hello,
I imported the semantic ui vue into my main.js but when I import a button or some component I get the following error in the console:
this. $ options.events [key] .bind is not a function
Battery Tracking:
beforeCreate @ webpack-internal: /// 16: 121: 25
callHook @ webpack-internal: /// 1: 2919: 9 .....
Thank you so much.

It's hard to give you an answer without a code. Please try to reproduce the error here so we can give you support.

Same error here. I think it's a conflict with vue-events. Can you confirm you're using it?

I think this error means, that you're trying to set non-function as a handler to the event, for example
@event="blablabla", when blablabla mthos isn't defined in the instance, or it's not a method, but prop, data or computed, that doesn't contain function.

I've the same issue. In my case, it's a conflict with vue-event-handler. I use this package a lot so it's a big problem for me.
You can see an example here : https://codesandbox.io/s/7m0q4w7zqq
Same issue in #157

Having the same issue, vue-events conflicting with semantic-ui-vue. The components seem to work despite the console warnings.

TypeError: this.$options.events[key].bind is not a function

Had the same error, while using Laravel Mix (npm run hot)
I resolved it just by stopping my npm watcher and relaunching. It seems webmix didn't seen I added the function !


Allow CBuzzFootBotController subclassing

Minor change to allow the creation of controllers based on the footbot with custom functions.
When RegisterFunctions() is private, it is not possible to call it from a subclass to add new functions while still registering the superclass (CBuzzFootBotController) functions.
Note that RegisterFunctions() is already protected in the spiri controller.

I see no problem here


CircleCI: run unit tests, and add test for Socket Permissions

For some reason, CircleCI only ran a build, but did not run any test. This patch adds a "test" step to CircleCI. root is required for some of the tests (doing chown/chmod).
Also adding a test to verify socket ownership/permissions

@tiborvass @tonistiigi @chris-crone ptal

bringing this one in, as we want tests to be run on PRs


Website 502 Bad Gateway Error for https://enzienaudio.com/

currently going to any url on the site generates a 502 Bad Gateway Error.
https://www.keycdn.com/support/502-bad-gateway/ explains some culprits for this.

For me the website works but my user data have seem to be vanished. Meaning I can't use online compiler at the moment.

It looked like it was back up yesterday, and then today I tried going back and I get:
This site can’t be reached enzienaudio.com refused to connect. ERR_CONNECTION_REFUSED
on all URLs for the site.
Who is your webhost?
Im a bit concerned, I had planned on using this in a bunch of projects going forward and now as I am about to finish some tests this happened.. I really hope it comes back up, its an awesome thing.

Hi all, I'm very sorry. We've had ridiculous server stability problems that we thought that we had solved. It's back up right now and I'm babysitting it. The server stability is an absolute priority for us right now as it's been shaky for too long.  Again, I apologise 😓

hehe yeah you got it back up minutes after my post :D . good work!
I feel you, are you running your own server then?  That would make sense and I could imagine how tricky that might be.

and were back to down :P

Looks to be back up again :D but one link is giving a 404, the pd objects list :
https://enzienaudio.com/docs/pdobjects.html

Hi @MacroMachines thanks for the patience, it seems like we've been outgrowing our initial server setup! We've been busy working on the backend to provide more stability going forward.


Working "Player not found" check

Return "Player not found" after check even if site returns status code 200

Looks good, I'd like to return a non-200 at some point for sure though


dcac-110-setup-structured-logging

Added Structured logging 1.9.16

Incorrect repository so closing.


[KBFS-2892] TLF Resets in the Row

This PR isn't yet useful on its own, but it sets up the scaffolding to support TLF resets in the Files tab. The next step is to implement the banners and allow the user to do something about the reset.

Ready for another look.

@chrisnojima I'd appreciate a final review on this now that we're out of code freeze.

@keybase/react-hackers I need another review on this.

@chrisnojima ping


Setting the translation parameter should not change the project configuration

Hey team, I don't really know if this exactly an issue, but it doesn't feel right to me.
I have a crowdin project with the folowing structure, with different source files on different directories:
root
- folder1
-- messages.json
- folder2
-- messages.json
- messages.json

The top level messages.json has the following export configuration:

On GH I have a workflow that uses the<EMAIL_ADDRESS>and I set the translation parameter with: /i18n/messages-%two_letters_code%.json. I'm doing this mapping because in my local project, the translations are located in a different directory.
The problem is that when I pass the parameter with this value, the project configuration is updated and if I go the Crowdin project to download it, it is using the last configuration provided by the github action, and not the one I manually add to the project.

I don't think that overriding the configuration should be the expected behavior, but is this a bug, or an intended feature? Thank you team, waiting for your feedback.

Hello @richardsd!
This is the expected behavior since the file would be exported according to the configuration option you've saved.  Otherwise, the Github Action for this file would not be functional.
The same file can't have two different export options within one project.
Hope this answers your question!

Hi @Gabez0r,
The Crowdin GH Action is based on the Crowdin CLI. CLI is closely connected to the translation pattern.
The possible solution is to use branches in Crowdin - files within each branch will have their own structure and translation pattern. The duplicated strings could be managed using the duplicates strategies.

Thank you for the response, @andrii-bodnar. So, if we want to have the very same translations and just use two different translation formats, i.e, export the files with a different pattern, but without having the config constantly switching between exports, we should have two branches duplicating the same content?

@Gabez0r could you please a more detailed example of your setup? Maybe there is another solution but I need to better understand your use case

Hi @richardsd, @Gabez0r,
There is a new alternative GH Actions solution that is not based on the Crowdin CLI and can be used without affecting the "Translations export pattern" - Crowdin Request Action
Maybe, it could help with your use cases as it gives more flexibility since you have more control and freedom on the Crowdin API usage

Hey @andrii-bodnar,
Thanks for getting back to us on this subject. We are not actively working on the topic anymore, as we based our solution on current behavior for now (and we are now in a new team), but I'll make sure that this information is passed on to the new team.
Thank you very much.


EditorConfig file and grammar fixes

fix some grammar by @0xSoul24
add an EditorConfig file that specify to the IDE how to format files

Wow, I collected quite some grammar mistakes over time xD
Thanks for fixing them.


openai.RateLimitError: Error code: 429 (API keys never used)

Hello,
I'm very much interested into this project but... cannot make it work.
I deployed it using portainer, adjusted everything and tried to configured with OpenAi then Gemini.
I still don't see any news coming.
Having a look to airflow logs, I'm seeing this :
[2024-08-27, 06:22:28 UTC] {subprocess.py:93} INFO - openai.RateLimitError: Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
Which doesn't make sense as you can see below:

Is there an hardcoded key some where ?
Especially in that case, where I specifically mentioned Gemini, so i don't understand how OpenAi is coming, and even though, it seems it doesn't reach my API Key.
Any clue ?

Just thinking about something:
I didn't put anything on variable below (as I don't know what to put):
AUTOGEN_GPT3_API_KEY=
AUTOGEN_GPT4_API_KEY=

But only filled next one:
OPENAI_API_KEY=
Is that correct or should i put same key everywhere ?

Hi @flowcool
There is no hardcoded key, it only rely on the .env config.
AUTOGEN_GPT3_API_KEY and AUTOGEN_GPT4_API_KEY are only for the experiment features; we can ignore them in your case.
Your above error log shows that your key is exceeding the quota. If this is a new key, it shouldn't be, but it may be that your account hasn't assigned enough budget for it. You may want to double-check.
The other way to check is to modify the LLM_PROVIDER=google and add a Google Gemini key, as it is free and has a good amount of quota for testing. ref

Hello,
thanks for the reply. As mentioned, I put (and let) google) during the whole night....and I still see next error, referring to OpenAi (and my openai API Key, still showing: Never used).
[2024-08-28, 06:21:22 UTC] {subprocess.py:93} INFO - [get_relevant] Fallback collection name: news_embedding__2024_08_27
[2024-08-28, 06:21:25 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (1/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:31 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (2/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (3/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [ERROR]: Score page failed, skip: Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - Traceback (most recent call last):
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/ops_reddit.py", line 323, in score
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     relevant_metas = op_milvus.get_relevant(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -                      ^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/ops_milvus.py", line 152, in get_relevant
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     embedding = emb_agent.get_or_create(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -                 ^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/embedding_agent.py", line 52, in get_or_create
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self.model.get_or_create(text, source, page_id, db_client, key_ttl)
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/embedding_openai.py", line 46, in get_or_create
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self.instance.get_or_create(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/embedding_openai_1x.py", line 105, in get_or_create
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     embedding = self.create(text[:EMBEDDING_MAX_LENGTH])
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/embedding_openai_1x.py", line 50, in create
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     emb = self.client.embeddings.create(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/resources/embeddings.py", line 114, in create
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._post(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1250, in post
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return cast(ResponseT, self.request(cast_to, opts, stream=stream, stream_cls=stream_cls))
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 931, in request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._request(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1015, in _request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._retry_request(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1063, in _retry_request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._request(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1015, in _request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._retry_request(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1063, in _retry_request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     return self._request(
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -            ^^^^^^^^^^^^^^
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -   File "/home/***/.local/lib/python3.11/site-packages/openai/_base_client.py", line 1030, in _request
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO -     raise self._make_status_error_from_response(err.response) from None
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - openai.RateLimitError: Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [EmbeddingAgent] provider: openai, model_name: text-embedding-ada-002
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - Initialized EmbeddingOpenAI 1x: 1.35.8
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - Initialized EmbeddingOpenAI Interface: openai version 1.35.8
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [get_relevant] collection_name: news_embedding__2024_08_28
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [MilvusClient] server.alias: default, host: milvus-standalone, port: 19530
[2024-08-28, 06:21:37 UTC] {subprocess.py:93} INFO - [get_relevant] Fallback collection name: news_embedding__2024_08_27
[2024-08-28, 06:21:41 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (1/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:47 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (2/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:53 UTC] {subprocess.py:93} INFO - [ERROR] RateLimitError during embedding (3/3): Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:53 UTC] {subprocess.py:93} INFO - [ERROR]: Score page failed, skip: Error code: 429 - {'error': {'message': 'You exceeded your current quota, please check your plan and billing details. For more information on this error, read the docs: https://platform.openai.com/docs/guides/error-codes/api-errors.', 'type': 'insufficient_quota', 'param': None, 'code': 'insufficient_quota'}}
[2024-08-28, 06:21:53 UTC] {subprocess.py:93} INFO - Traceback (most recent call last):
[2024-08-28, 06:21:53 UTC] {subprocess.py:93} INFO -   File "/opt/***/run/auto-news/src/ops_reddit.py", line 323, in score
[2024-08-28, 06:21:53 UTC] {subprocess.py:93} INFO -     relevant_metas = op_milvus.get_relevant(

I'm completely lost about what's going on. I even deleted everything and rebuild the overall, but doesn't change anything

Ok, so after (re)deleting everything again and recreating, it finally took gemini.... Does it mean once setup, we cannot change model ? (without deleting everything)

Oh, I see. Thanks for providing the detailed log. It's from the embedding generation path. I believe your open API key setup may have something wrong, which prevents your app from querying it.
Do you mind creating another open API key and ensure it has enough budget to be used? Then we can try again.
Steps:

Re-create the API key
Replace the OPENAI_API_KEY environment variable with the new key in your build/.env.
Restart service

Jeeeeeeeeeeez ! Finally, for everyone:
I tried:
user api key
service api key
You api key <- this one looks to work !!
Many thanks, I spent my day trying to read your scripts and understanding where it was crashing !! Now I'll waiting to have these processing and play with it tomorrow !!

Cheers! :+1: Glad to see that works for you, I appreciated your help in letting more people know it. Enjoy the peace of the day, and feel free to drop any issues/thoughts, we could make it better together.

I'll wait a few hour (tomorrow ) and close it, juste to make sure we're all good ;)

Actually no :( Same error coming back.
and same behavior on OpenAI: API KEY: Never used
I had a look in the code (again, spent my day) and noticed on src/embedding_openai_1x.py there's no mention of API key (in comparison to 0x). Could it be linked ?
very sorry for all this :(

Sorry to hear that. I found an OpenAI thread that discussed this issue https://community.openai.com/t/i-keep-getting-an-error-you-exceeded-your-current-quota-please-check-your-plan-and-billing-details-even-though-my-account-has-sufficient-credits/537053
It may be due to no available credit, the shared account already using all the query quota (not likely), or some other issue inside Openai.
Someone mentioned that creating a new API key may solve the problem. If it hasn't been done yet, suggest to try it as well. (By the way, prefer to create the project API key instead of the legacy user key.) We may be lucky on the next try :)

I finally put 10 bucks on OpenAi and here we go.... I thought free account would work but no.


Email function dont gets a email back and is stuck

So yeah the problem is that I cant use the
Email()
function anymore since today idk what happend but it doesnt work anymore something is wrong with the requests :( (i think)
Ps:It actually worked yesterday completly fine idk what happened please help me 👍

oh the api is under maintence xD


first, render last component

轮播刚开始，首先渲染最后一个组件，初始化，挂载的时候，如果loop === true; offset的x要有值，不然会先出现最后一个组件
this is bug!!!!!

This patch works for me, who can review it and merge?

It should be fixed with #572
Thanks


Allow hetrogeneous equality

I don't think its possible to give a GRead instance for GOrdering, it only seems possible via a newtype.
Diffs for dependent-sum and dependent-map are very straighforward.
https://github.com/wz1000/dependent-sum/commit/959aa4f93b9e1aeb816059311df8ef2fd2bf6243
https://github.com/wz1000/dependent-map/commit/
Downside: Can't be supported on anything before GHC 8.2

Closing because it turns out I actually needed something like the following:
class GEq (f :: forall t. k t -> *) where
  geq :: forall (a :: k t1) (b :: k t2). f a -> f b -> Maybe (a :~~: b)


render_template with partial option requiring filename extension?

Here is the gist.
I am working on a site with the following bug branch.  I'm brand new to RSpec and liking it so far.  I'm just trying to understand why this test is failing.  Every example I've found of "expect response to render template" with partial option specifies just the name of the partial with the leading underscore.  However, I'm getting failed tests with the above code.
In the error detail, it looks like the array is actually holding the value of the partial with the filename extension, which is how I thought to simply add the extension, which did fix it.  Contrary to the "if it ain't broke", or "if it hurts, don't do that" mantras, I'm looking to actually understand why I'm wrong here.  Can anyone shed some light?  On the off chance that this is a bug... I'm happy to have discovered it, but I doubt that's the case.
I also have the shoulda_matchers gem installed if that matters.  I thought it was part of native Rspec though.  If this is in the wrong place, I'll gladly move it.

As I tried to explain on rspec/rspec-dev#192 this matcher delegates to Rails own assert_template. The options accepted by it and used by it are subject to Rails conditions. Please see their documentation.


config: Require hook support on Linux and Solaris

This builds on #869, so review that one first.
Clarifying the language from #855 according to this comment.  Without something like this commit, there's no clear way for config authors to know if their runtime will support hooks or not, and there was previous agreement that that sort of ambiguity was not helpful.  This also gives normative Markdown grounding for the Go platform tag added in 28e8f68.

LGTM

LGTM


Create version of rule E2 to handle foreign_key based joins

Hello, when using version 3.0.0-beta.0, LAMS does not pick up that foreign_key uses the primary key in the view.
In this repo, I have created a minimal reproducible example: https://github.com/bendiktv2/lams-key-bug
It consists of 2 autogenerated views (Create View From Table), and then adding a refinement to each of them, to add pk-dimensions, and a count_distinct measure.
There are 2 explores, 1 which passes E2 with no linting errors (sql_on: ${view_1.pk1_item_id} = ${works.pk2_item_id} ;;), and 1 which fails (foreign_key: pk2_item_id).
It seems that LAMS is not picking up the use of the refined primary_key in the refined view.

Hi, thanks for the bug report!
Unless I'm mistaken, this seems to be a general issue with the E2 logic, and not specifically related to refinements?
I think the rule was just never considered under the usage of join>foreign_key since that is an older and often unused form of writing the join.
Code reference: Missing sql/sql_on defaults to empty string and proceeds: https://github.com/looker-open-source/look-at-me-sideways/blob/fabble/v3-extends-refinements/rules/e2.js#L28
Since the logic for foreign_key would be slightly different (the join is implicitly against the joined view's primary_key: yes dimension, rather than against a naming convention), my initial thought is to have foreign_key joins ignored by this rule, and then make a separate rule for foreign_key joins. Thoughts?

Pre-release 3.0.0-beta.1 should fix this.

foreign_key based joins should never error E2
In the near future, once 3.0 is properly release, it will be easier to add new rules as minor changes due to rules being opt it, so I can add rule E3 for foreign_key based joins

Thank you for filing this bug, and in advance for letting me know if this resolves things for you!

Hi @bendiktv2 - after sitting down and writing out this rule, and writing tests, only when I finally went to document the rule did I realize that this rule perhaps does not make sense? Or perhaps I don't understand it :) In any case, what I implemented is certainly not right...
Using this as an example:
explore: orders {
	join: users {
		relationship: many_to_one
		foreign_key: user_id
	}
}

The field that is to be provided in the foreign_key is not supposed to be a primary key, it's the foreign key on the orders view. So it wouldn't usually follow the PK naming convention. (It might when the FK happens to be a part of a composite PK, but that is more a coincidence than a rule)
Is there some rule to implement here afterall?

v3.1 implements rule E6, which ensures that foreign_key based joins declare the right relationship (usually many_to_one, and rarely one_to_one)
As it relates to my previous comment, this new rule is slightly different than E2 in that it does not use the primary key naming convention to enforce cardinality, but rather relies on the assumption that Looker will use the joined view's primary_key dimension, so it must always be a *-to-one join.


Disable a few mark_dependence sil combines in ossa

There are 2 reasons to disable this:

OSSA rauw creates copies for replacement, copies created in this case cannot be canonicalized away due to the presence of pointer escape. (See test test_mark_dependence_ossa_silcombine1 after -sil-combine -copy-propagation)

SILCombine of mark_dependence (enum can cause a potential miscompile.

ClosureLifetimeFixup specifically inserts enums to lifetime extend the closure lifetime until the end of the function. Replacing mark_dependence's base operand with a copy of the enum's operand can end up shortening the lifetime of the base. (See test test_mark_dependence_ossa_silcombine2 after -sil-combine -dce -semantic-arc-opts)
Fixes rdar://115139907 and rdar://114615539

@swift-ci test

@swift-ci apple silicon benchmark

@swift-ci test

@swift-ci smoke test


chore: Add Instruction::MakeArray to SSA

Description
Problem*
Resolves https://github.com/noir-lang/noir/issues/1733
Summary*
Adds a MakeArray instruction to explicitly create a new array. Now no other ValueIds can contain other ValueIds which simplifies a couple passes.
Additional Context
Still working on getting this working in brillig.
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Seems like --show-ssa fixes the compilation failures on the few programs I tried.

Disabling the normalization of valueids shows that we're deleting instructions in the DIE pass which shouldn't be removed. For example the load instruction when returning arr in quicksort is removed from array_sort.
This is happening because somehow the return value of this function (the loaded value) becomes a Value::Function rather than a Value::Instruction. We only mark return values as used if they're constants, instructions or block params so we end up removing the loaded instruction.

I've merged in master so that this branch is up to date so will be handing this PR back now.

This is passing all tests now and showing a surprisingly large improvement in the hashmap test but also large regressions in brillig bytecode sizes.

This is likely due to the fact that constant arrays are no long considered as "constants" by constant_allocation.rs. We're then not hoisting their usage up to avoid multiple initialisations.

@TomAFrench hmm I tried updating is_constant_value there but doing so just leads to many panics on ValueIds not being available.

Going to hold off on merging this for a bit due to the brillig size regressions

Going to hold off on merging this for a bit due to the brillig size regressions

Yeah would be good to avoid this regression.

This is likely due to the fact that constant arrays are no long considered as "constants" by constant_allocation.rs. We're then not hoisting their usage up to avoid multiple initialisations.

yes, I think that's the issue. For example, in slices, we have patterns like this:
    inc_rc v207
    v208 = make_slice [Field 0, Field 0]
    inc_rc v208
    v209 = eq v0, v1
    v210 = not v209
    jmpif v210 then: b18, else: b17
  b18():
    v212 = eq v0, Field 20
    v213 = not v212
    jmpif v213 then: b20, else: b19
  b20():
    v214 = make_slice [Field 0, Field 0, v1]
    v215 = make_slice [Field 0, Field 0, v1]
    v216 = make_slice [Field 0, Field 0, v1]
    jmp b21(u32 3, v216)
  b21(v10: u32, v11: [Field]):
    jmp b22(v10, v11)
  b22(v12: u32, v13: [Field]):
    inc_rc v13
    v217 = eq v12, u32 3
    constrain v12 == u32 3
    v218 = lt u32 2, v12
    constrain v218 == u1 1 '"Index out of bounds"'
    v219 = array_get v13, index u32 2
    v220 = eq v219, Field 10
    constrain v219 == Field 10
    v221 = make_slice [Field 0, Field 0]

Previously, these immediate slices would have been picked up by constant_allocation and hoisted to just 1 single initialization. Now, since array constants have been promoted to a instruction, we actually need to do at the ssa level what constant allocation was doing outside SSA. So in order to avoid any regression we need to identify payloads to make_array that are equal to each other and hoist the make_array to the common dominator of all the arrays that are equal.
Also, if all the values are immediates constant_allocation was hoisting them out of loops to avoid extra runtime cost. But that won't affect bytecode size

fold_2_to_17 	-130,344,860 ✅ 	-99.15%
bench_2_to_17 	-144,084,489 ✅ 	-99.58%

How about this for brillig improvements :sweat_smile:

Unfortunately the actual brillig bytecode size is still a bit bigger

Although doing so could potentially lead to more work if those else blocks were never executed and we hoist out of the if

Maybe we could hoist out of loops? that one should always lead to better perf I think

Ended up merging this with the smaller version of the "move instruction to common dominator block" change since the larger, recursive version was failing a few programs.
The diffs now are generally more positive. Executed opcodes can be better or worse depending on the test.


Make it easy to deploy Meltano and Airflow on Kubernetes

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2014
Originally created by @DouweM on 2020-05-07 14:45:00

View 10 previous comments from the original issue on GitLab


Vendor engine-api to 5d8739372315f8147ceb7bcc53576893ff96ffc6

This fix tries to update engine-api to 5d8739372315f8147ceb7bcc53576893ff96ffc6
This fix brings engine-api up to date, which includes changes from:
https://github.com/docker/engine-api/pull/256
https://github.com/docker/engine-api/pull/254
This fix is directly related to
#22566 Add config parameter to change per-container stop timeout during daemon shutdown
This fix is indirectly related to
#22489 Builder shell configuration
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

LGTM, thanks!
ping @vdemeester PTAL 👍

LGTM

Neat, thanks! Saves me having to revendor in #22489


markLine type 'average' is not assignable to type 'MarkerStatisticType'

What problem does this feature solve?
When using in TypeScript projects, a markLine with type: 'average' gets the following error:
Type '{ type: string; }' is not assignable to type 'MarkLine1DDataItemOption | MarkLine2DDataItemOption'.
  Type '{ type: string; }' is not assignable to type 'MarkLine1DDataItemOption'.
    Types of property 'type' are incompatible.
      Type 'string' is not assignable to type 'MarkerStatisticType'.ts(2322)

What is expected?
No type error.

This is a misuse.
Instead of
const a = {
    type: 'average'
};
const b: {type: MarkerStatisticType} = a;

it should be
const a = {
    type: 'average'
} as const;
const b: {type: MarkerStatisticType} = a;

to make a.type to be 'average' instead of string.

This is a misuse.
Instead of
const a = {
    type: 'average'
};
const b: {type: MarkerStatisticType} = a;

it should be
const a = {
    type: 'average'
} as const;
const b: {type: MarkerStatisticType} = a;

to make a.type to be 'average' instead of string.


k9 keeps sending failing messages

Every time I open k9, it tries to send a whatever message again and throws an error.
This seems to accumulate. I have now 3 messages that are trying to be send every time I connect to the WiFi. And of course every time the same errors.

How do I fix that/delete this cron?

Current version of k9 5.0006 with kitkat 4.4.4

This should already be fixed in master. In 5.006 you should be able to delete messages from the "Outbox" folder to stop K-9 Mail from trying to send them.


[docs] Remove redundant comments, duplicating info inferred from the definitions

Every class in our API docs has the class name in the class description section, which TypeDoc takes from the @class tag in the corresponding comment.
Here is a couple of examples.

As we can see from the screenshots, TypeDoc infers all the needed info about the class (including its base class) from the class definition. So, the information we provide with the @class and @extends tags becomes redundant. I suggest removing it.
This PR addresses it.
With this PR merged, the redundant info is gone.

What do you think, @LeaVerou?

Gotta love PRs that only remove code! ❤ Wrt why they're there, I think it was my futile attempt to make the previous API docs tooling work.

Agreed! 😊
And yes, I'm sure it's because of the previous tool we used to use.
I'll continue working on inline docs then. One day we'll get the API docs we'll be proud of. 🤞


Enable multicore support

Now that we're approaching Go 1.7 (hopefully released this week) and the language default for GOMAXPROCS has been the number of cores in a system for a while now, I figured it'd make sense to remove this restriction.

Is multicore really going to bring a tangible benefit to coreos-cloudinit?  It seems prudent to just leave it be, unless there's pressure to scale to multiple cores in this program.

I'm very concerned about all of the potential race conditions that are going to work their way out. This code is pretty fragile as is and we aren't planning on investing much more into this project (it's being superseded by Ignition). Thank you for the effort, but we are going to pass on this change.


Sinon Matchers don't work with the this.mock.expects("method").withArgs

The Matcher API (http://sinonjs.org/docs/#sinon-match-api) should be able to work with  Buster's mock.
Examples:
https://gist.github.com/zcicala/6772093

The noAsserts gives no asserts, because a sinon.mock is used directly, therefore the assertion is never registered - there's no way around it.
The justFails works just fine with latest buster.


Upgrade to hawkular-parent 18

@Jiri-Kremser could you please review?

@ppalaga travis failed with some deployment issues, I've re-run the build. Let's see what happens now. Perhaps the dependencies need to be changed also somewhere else (?)

@Jiri-Kremser I have seen that fail and wondered about it. It is passing here on my laptop locally.

@Jiri-Kremser we should perhaps turn agent off in itest profile to exclude one important source of non-reproducibility.

This hawkular branch depends on inventory-snapshot, so I think the issue is caused by this PR (https://github.com/hawkular/hawkular-inventory/pull/117), @pavolloffay could you please look into the itests?

There is a pull request for HWKINVENT-90 on hawkular/hawkular https://github.com/hawkular/hawkular/pull/327 .

I merged #327 and restarted Travis here.


replace readme image

Description
This PR fixes #114
Notes for Reviewers
Replaced the image with one provided in issue description
Signed commits

[x] Yes, I signed my commits.

Yay, @king-11! 🎈


Swiftlint corrections

Compiling with Swiftlint step produces issues and warnings. Fixing them by renaming most of too short characters and for try in Demo. Most warnings was whitespacing so swiftlint autocorrect fix most of them.
Now it has 0 swiftlint issues.

LGTM

Thanks a lot @JakubMazur and @yzhong52


Why model has XML definition?

The model section in the doc has XML definition, when I'm not using XML at all.
Any way to make it JSON?
xml: OrderedMap { "name": "Total" }
xml:
   name: Total

@aclowkey What do you mean "way to make it JSON"?
If you want to remove XML properties from swagger.json, yes, maybe we can change this in the future

Hi, is there any update for this?
I would also like to use json as a model definition.
Thanks

@eloo It's no problem to use JSON as model definition. This issus is about redundent XML definition in swagger.json, which I think is not your concern.

@eloo It's no problem to use JSON as model definition. This issus is about redundent XML definition in swagger.json, which I think is not your concern.

and how can i disable the XML?

and how can i disable the XML?


Started Plan-B, huh?

After executing A (https://github.com/WindowsGeekPro/workaholic-meow/issues/1), then comes B.
Think about C, but don't think too much unless B is executed.
It's sweet to dream on D, but why so?
Yet, it's way too much if it appears to think of E.
Sometimes, Thinking about F, G, H and I might not sound bad.
It's crazy to connect (I + G as IG), though it requires a B as a junction for all these stuffs.
B will be the beginning... And there is no end.. =P
This issue will be closed when B adventure begins after closing goodbye to A.... ;)

Waiting to finish A soon... :'(
But, you see that "learning is never ending".
If u are innovative, then u must learn so much and practice so much inorder to build something.

Why so?
Because so many geniuses have done so many stuffs and this learning part appears bigger especially when you were born in 1991 and you grow-up... Learning could have been easier if you were born in 1st century, but there won't be any foundations and you will be the source, which is hard task. Ok, why am i talking about this?  I came across many awesome topics and stuffs, but i couldn't anything because i need to finish A. There is no cheat-code here to finish this level especially the academics part. Academics is necessary and I will write a blog on this importance of academics or education after i clear all my papers and after i start earning. ;)

Facing worst things like 3 years ago... No use of mastering anything.. they only prefer only males and males only...
I thought that atleast one of my childhood dream will come true..


[core] Add warning for imperative built-in action calls

This PR warns for improper imperative calls of built-in actions, which is a very common problem.
https://stately.ai/docs/actions#built-in-actions

Ping @Andarist


chore(main): release 1.2.0

:robot: I have created a release beep boop
1.2.0 (2023-01-31)
Features

add model as submodule (3853fd3)
disable sanitizer (8804a93)
home: NavigationBar instead of SalomonBottomBar (e78a2bf)
load model (f9bd5fc)
log (3dd1004)
logging system (c9d4017)
mine: SettingsView (a1894f7)
model test (57837e0)
model: forward (b9542f0)
use material 3 (3257fa1)

Bug Fixes

deps: update dependency com.android.tools.build:gradle to v7.4.0 (1a24958)
hpp to cpp (f352b49)
no Werror (1645948)
set(CMAKE_CXX_STANDARD 20) (02d4462)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/CCXXXI/ecg_monitor/releases/tag/v1.2.0 :sunflower:


Leros behavior on branch instruction

Below is a small test program to test branch instructions. Leros should branch to A load 0 into the accumulator and make a system call with argument 0. This is also what Leros does, and it finishes the program with 0 in the accumulator. However when taking a look at the simulation waveform Leros also loads the instruction  loadi 0xFF (0x21FF), but it does not seem to execute the instruction. Is this the desired behavior of Leros?
start:
    loadi 1       // 0x2101
    brnz A        // 0xA003
    loadi 0xFF    // 0x21FF
    br start      // 0x8FFD

A:
    loadi 0       // 0x2100
    scall 0       // 0xFF00

Waveform:

To reproduce:
Add above test program under asm/test/
Run  LerosTest and open waveform generated waveform

As pcNext points into the instruction memory, it prefetches the next instruction. The combinational pcNext is used, as the ROM has an input register for the address. When there is a branch, the prefetched instruction is just ignored when the branch is taken.


Error: build error: contrib/openshift: no such file or directory

I was having trouble building the rhel7 version so I switched to the centos version.  Now I see this error:
Step 6 : COPY ./contrib/openshift /opt/openshift
928	F1014 19:21:03.004407       1 builder.go:204] Error: build error: contrib/openshift: no such file or directory

when building on openshift?  can you share your buildconfig?

I am using this build config:
https://gist.github.com/deanpeterson/75852e0a84cc69bfd9b3546836a9b2c5
And the Dockerfile in http://openshift@stash-d00
/projects/OP/repos/jenkins.git is here:
https://github.com/openshift/jenkins/blob/master/1/Dockerfile
On Fri, Oct 14, 2016 at 2:27 PM, Ben Parees<EMAIL_ADDRESS>wrote:

when building on openshift? can you share your buildconfig?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openshift/jenkins/issues/165#issuecomment-253898268,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGmtF3yHwKKWBcOu-D01PSwS12Y4AaOks5qz9eQgaJpZM4KXW39
.

does that stash contain the contrib directory?  your git repo needs to basically contain everything here:
https://github.com/openshift/jenkins/tree/master/1
(you don't need the test dir or dockerfile.rhel7)

That was dumb of me. I'm looking at about 20 things today and some basic
knowledge of Docker slipped my mind. Thanks!
On Fri, Oct 14, 2016 at 2:48 PM, Ben Parees<EMAIL_ADDRESS>wrote:

does that stash contain the contrib directory? your git repo needs to
basically contain everything here:
https://github.com/openshift/jenkins/tree/master/1
(you don't need the test dir or dockerfile.rhel7)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openshift/jenkins/issues/165#issuecomment-253902550,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGmtC5xXpXusuTkNeSyrzSkdjJyhHyPks5qz9yEgaJpZM4KXW39
.


Users template tasks

[x] form: is_active and is_present should be set to true by default
[x] list/detail/slide: add user title (getFullName() should contains title)
[x] list: Do not show group "Registered" (only for manager and its own groups).

all done with #1875


changes for bug 761

Changes proposed in this pull request
Incorporated the review comments in https://github.com/SnappyDataInc/snappy-store/commit/0d038d642c32ec079994b14353782d9f10ac149b
Also in ProjectionRow.toData moved common common code to serialize column into a function called for both "if (this.lobColumns == null)" and "else" part.
@sumwale please review.
Patch testing
Ran all store junit/dunit tests and randomPartitionTablesProcedureIndexWithAccessors.conf.
Other PRs
(Does this change required changes in other projects- snappyData, spark, spark-jobserver, aqp? Add the links of PR of the other subprojects that are related to this change)
…aInc/snappy-store/commit/0d038d642c32ec079994b14353782d9f10ac149b

looks good


Build with Go 1.17 / npm 7 / node 16

Update build to use Go 1.17.

Use golang-builder images for testing.
Update CircleCI go orb.

Signed-off-by: SuperQ<EMAIL_ADDRESS>

/prombench cancel

Actually there was https://github.com/prometheus/prometheus/pull/8956/files to attempt to ignore issue, but it's for func2 and not func1. Maybe we just need to add func2 as well? 😬

Yes we need to add func1

Thanks!


[docs] Demo UnstyledSelectMultiple crashes in mobile device

[X] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

On mobile device (in my case P20 Lite, Android 9, Chrome 104) visit https://mui.com/base/react-select/
Select an item from the select in Basics section
The demo crashes with the following error:

Your Environment

Tech
Version

MUI
v5.10.0

netlify deploy
https://62f1d4edc7ade70008b1594b--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Linux; Android 9; ANE-LX1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Mobile Safari/537.36

Device
P20 Lite, Android 9

I don't understand where it is crashing.
I opened codesandbox of UnstyledSelectMultiple from both of the links you provided in above description, but everything seems fine.

On mobile device (P20 Lite, Android 9, Chrome 104) open https://mui.com/base/react-select and under Basics section, select an item in the select (Ten, Twenty, Thirty). The following error occurs:

This error doesn't occur on PC (Windows 11, Chrome 104).

The error comes in mobile.
This should be the main highlight of your issue.
But you informed this later.
Please add full details instead of only headings.

I updated the title and the description of the issue.

Thanks for the report. I was able to reproduce it and will investigate it.

It got fixed in MUI Base version<EMAIL_ADDRESS>with https://github.com/mui/material-ui/pull/33882.


Dropdowns do not close when origin is clicked

Dropdowns do not close when origin is clicked and belowOrigin is true.
http://codepen.io/anon/pen/OyvmOM

So I went and fixed the issue then relized it was already fixed on the source file. I assume it will be apart of the next release?

yes it will!


DeveloperExceptionPage needs better text spacing on OSX/Safari

@kichalla when figuring this out, if it turns out we just have "too much" CSS then consider dramatically simplifying CSS. It could be that we went overboard at some early point and added complex styles that just don't work well.

This is also the case on Edge