Adding ALB forces launch template replacement

Description
When I add an ALB for the first time, terraform wants to replace my launch template, instead of updating it in place.
Versions
Terraform:

v0.12.25

Provider(s):

provider.aws v3.32.0
provider.kubernetes v1.11.1
provider.local v2.1.0
provider.null v3.1.0
provider.random v3.1.0
provider.template v2.2.0

Module version:

v13.0.0

Reproduction
Steps to reproduce the behavior:
Add a new ALB to  worker_groups_launch_template:
  {
    name                      = "general_purpose_xlarge"
    instance_type             = "t2.xlarge"
    asg_min_size              = 1
    asg_max_size              = 20
    asg_desired_capacity      = 1
    kubelet_extra_args        = "--node-labels=node-role=worker,ceph-client=true,node.kubernetes.io/lifecycle=normal"
    target_group_arns         = [aws_lb.lb_albtest.arn] # ADDED THIS!
    load_balancers            = [aws_elb.private_elb.name, aws_elb.public_elb.name]
    tags                      = local.autoscaling_tags
    protect_from_scale_in     = true
  },

The result is:
  # module.eks.aws_autoscaling_group.workers_launch_template[4] must be replaced
+/- resource "aws_autoscaling_group" "workers_launch_template" {
      ~ arn                       = "arn:aws:autoscaling:us-east-1:213814057860:autoScalingGroup:6b3cf96d-5d2d-4ead-a471-28883d3a1e03:autoScalingGroupName/ms-dev-apps-general_purpose_xlarge656765456789876500000014" -> (known after apply)
      ~ availability_zones        = [
          - "us-east-1a",
          - "us-east-1b",
          - "us-east-1c",
        ] -> (known after apply)
      - capacity_rebalance        = false -> null
      ~ default_cooldown          = 300 -> (known after apply)
      ~ desired_capacity          = 4 -> (known after apply)
      ~ enabled_metrics           = [] -> (known after apply)
      ~ force_delete              = false -> (known after apply)
      ~ health_check_grace_period = 300 -> (known after apply)
      ~ health_check_type         = "EC2" -> (known after apply)
      ~ id                        = "ms-dev-apps-general_purpose_xlarge656765456789876500000014" -> (known after apply)
      ~ load_balancers            = [
          - "ms-dev-apps-private-elb",
          - "ms-dev-apps-public-elb",
        ] -> (known after apply)
      ~ max_instance_lifetime     = 0 -> (known after apply)
      ~ max_size                  = 20 -> (known after apply)
        metrics_granularity       = "1Minute"
      ~ min_size                  = 1 -> (known after apply)
      ~ name                      = "ms-dev-apps-general_purpose_xlarge656765456789876500000014" -> (known after apply)
      ~ name_prefix               = "ms-dev-apps-general_purpose_xlarge" -> (known after apply) # forces replacement
      + placement_group           = (known after apply)
      ~ protect_from_scale_in     = true -> (known after apply)
      ~ service_linked_role_arn   = "arn:aws:iam::2133242340:role/aws-service-role/autoscaling.amazonaws.com/AWSServiceRoleForAutoScaling" -> (known after apply)
      ~ suspended_processes       = [
          - "AZRebalance",
        ] -> (known after apply)
      - tags                      = [] -> null
      ~ target_group_arns         = [] -> (known after apply)
      ~ termination_policies      = [] -> (known after apply)
      ~ vpc_zone_identifier       = [
          - "subnet-0267dd82dsfadf4b1",
          - "subnet-0a1969adsfasdfas230",
          - "subnet-0dadsfq3434296",
        ] -> (known after apply)
        wait_for_capacity_timeout = "10m"

      ~ launch_template {
          ~ id      = "lt-0b178007f556b286e" -> (known after apply)
          ~ name    = "ms-dev-apps-general_purpose_xlarge65676545675591110000000c" -> (known after apply)
          ~ version = "$Latest" -> (known after apply)
        }
      + mixed_instances_policy {
          + instances_distribution {
              + on_demand_allocation_strategy            = (known after apply)
              + on_demand_base_capacity                  = (known after apply)
              + on_demand_percentage_above_base_capacity = (known after apply)
              + spot_allocation_strategy                 = (known after apply)
              + spot_instance_pools                      = (known after apply)
              + spot_max_price                           = (known after apply)
            }

          + launch_template {
              + launch_template_specification {
                  + launch_template_id   = (known after apply)
                  + launch_template_name = (known after apply)
                  + version              = (known after apply)
                }

              + override {
                  + instance_type = (known after apply)
                }
            }
        }

      - tag {
          - key                 = "tag1" -> null
          - propagate_at_launch = true -> null
          - value               = "tag1value" -> null
        }
      - tag {
          - key                 = "tag2" -> null
          - propagate_at_launch = true -> null
          - value               = "tag2value" -> null
        }
      + tag {
          + key                 = (known after apply)
          + propagate_at_launch = (known after apply)
          + value               = (known after apply)
        }

It says that the name_prefix is forcing replacement?
Are you using workspaces?
No
Have you cleared the local cache (see Notice section above)?
Yes
Expected behavior
Should be updated in place
Am I doing something wrong? Thanks.

It looks like a terraform race condition issue. In case this is useful for someone, I managed to bypass the issue by creating the ALB first, and then attaching it (before I was creating it on the same terraform run...)

First, create the ALB and the target LB group
After the ALB is created, add the target_group_arns

Doing that in order (in two terraform runs) it is working for me.


chore: Remove flaky assert.

This is not a useful assert, and it's flaky.

Pull Request Test Coverage Report for Build 2783c21d-9417-4df9-83f1-d0e7005753ca

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 62.272%

Totals

Change from base Build 4a2f4ce8-fb4d-48f5-81ad-238622b76a69:
0.0%

Covered Lines:
2829

Relevant Lines:
4543

💛  - Coveralls


Crash when add category

res://addons/ggs/editor/category_panel/category_panel.gd:38 - Invalid call. Nonexistent function 'add_item' in base 'Tree ()'.

Please make sure the "GGS" is in the autoload list and enabled. If it is, restart your project and see if that'll solve the issue. Check out this for more info.
Let me know if that solved the issue.


Bug 2173525: Disk and net icons for boot order modal

📝 Description
Adding icons for boot order modal
🎥 Demo
Before

After

/retest


Need an option to generate documentation for all the endpoints, without adding tags: ['api'] to each of them

Feature request: ability to enable documentation for all routes
Currently, the only way to add an endpoint to the docs is by adding tags: ['api'] to the route options. Having many routes means:

a lot of manual work which is error-prone
reason of having this tag looks implicit in code. Imagine enableSwagger tag which is self-descriptive
this adds many identical lines of code for a single reason

Is this a maintainer's choice to prevent easy exposing of docs for all endpoints?
Or would you encourage submitting a PR adding an option to enable docs for all routes? Like allRoutes: boolean?

Similar https://github.com/glennjones/hapi-swagger/issues/543. IMO this feature would be awesome and PRs are welcome! 🤗

I've created a PR that does more than initial suggestion of adding allRoutes: boolean option.
Instead, I propose a solution where routeTag can accept a function with the route tags (tags: string[]) => boolean in the argument.
The resulting PR might be too complex, I can split it into smaller ones if needed

Closed via https://github.com/glennjones/hapi-swagger/pull/747


Topology import fails if the exported topology has more than one custom processor

https://github.com/hortonworks/streamline/pull/633


kata-runtime kata-check failed

Description of problem
I followed the instructions to install kata runtime.
 $ go get -d -u github.com/kata-containers/runtime
$ cd $GOPATH/src/github.com/kata-containers/runtime
$ make && sudo -E PATH=$PATH make install
I did these successfully. I found /usr/local/bin/kata-runtime and /usr/share/defaults/kata-containers/configuration.toml existed. Then I checked  the kata-runtime version. It showed,
 kata-runtime  : 1.10.0-rc0
commit   : 1296f6f1ad95cd7288f5e78e7c7dc39abe2d786a
OCI specs: 1.0.1-dev
However when I enter instruction sudo kata-runtime kata-check, it output as follows,
 ERRO[0000] /usr/share/defaults/kata-containers/configuration-qemu.toml: file /usr/share/kata-containers/vmlinuz.container does not exist  arch=amd64 name=kata-runtime pid=17608 source=runtime
/usr/share/defaults/kata-containers/configuration-qemu.toml: file /usr/share/kata-containers/vmlinuz.container does not exist
I found  /usr/share/kata-containers didn't exist.
Expected result
Maybe some information about device.
Actual result
 ERRO[0000] /usr/share/defaults/kata-containers/configuration-qemu.toml: file /usr/share/kata-containers/vmlinuz.container does not exist  arch=amd64 name=kata-runtime pid=17608 source=runtime
/usr/share/defaults/kata-containers/configuration-qemu.toml: file /usr/share/kata-containers/vmlinuz.container does not exist

Hi @Fvoiretryzig . Thanks for the report.
First, you get that error as you have not built and installed all the components of 'kata containers'. The runtime is just one of the necessary components. If you are installing from source, and have not already, take a look at the developers guide. There you find the instructions to build and install the other components as well such as the agent/filesystem and kernel images that are required.
Secondly, is that the only error line that was printed out? If so, I think that could be improved - the kata runtime is complaining that it cannot find one of its vital components, but I think it could/should also list all the information it can find (such as its own build/version information). Please let us know if that was the complete output?
thanks.

Hi @Fvoiretryzig . Thanks for the report.
First, you get that error as you have not built and installed all the components of 'kata containers'. The runtime is just one of the necessary components. If you are installing from source, and have not already, take a look at the developers guide. There you find the instructions to build and install the other components as well such as the agent/filesystem and kernel images that are required.
Secondly, is that the only error line that was printed out? If so, I think that could be improved - the kata runtime is complaining that it cannot find one of its vital components, but I think it could/should also list all the information it can find (such as its own build/version information). Please let us know if that was the complete output?
thanks.

Thanks for your help. The error messages  were complete output.
Because in the developers guide the command kata-runtime kata-check is just after the runtime installation, I thought I needed to check this before doing following things.


Create an sitemap.xml file

Tasks

[x] Create an sitemap.xml file

Writing the Issue

[x] Write each task for this issue using checkboxes
[x] Write all verbal instructions as tasks

Before Working on the Issue

[x] Assign yourself
[x] Share your screen on Discord
[x] Write which issue you are starting on Slack
[x] Git: Fetch the latest main
[x] Git: Create a branch with the name of the issue

Pre-PR Checks

[x] Insure that changes include only what is necessary for this PR
[x] Implement necessary Unit tests
[x] Implement necessary E2E tests
[x] Perform functionality checks

https://gmotech.jp/semlabo/seo/blog/robotstxt_method/


Question- Getting info for the sorce capabilitys

Hello,
Is it possible to read the information about the source capability and how to send it to UART?
I am working around project to power my old laptop with USB-PD.
My idea is to use part of debug to read info about selected PD profile and mainly how much W can draw from the source. with the second MCU i will simulate DS2505 which is a communication chip inside the charger containing info about power capability of the charger. with this configuration i never harm the PD charger.
I look the debug code, but i cant find any code providing data how much amperes A can provide the charger, i think the voltage can be easily read and send to UART,  when V and A are available the W can be calculated and sent to UART

This can be achieved with more or less the following steps:

Ensure debug output is not activated (by not defining the PD_DEBUG macro)
Change the code in pd_debug.cpp such that the UART code is always included
Modify on_source_caps_changed so it writes the desired information to UART using uart_transmit or uart_print. The information is available on the power_sink object, in particular in the source_caps array.

Note that on_source_caps_changed must immediately select a voltage. Otherwise it is likely that you run into a timeout of the USB PD charger and it will reset. Just select 5V (power_sink.request_power(5000);) until your second MCU has decided for another voltage.
If the ZYPDN12 is also supposed to receive commands via UART, you will have to extend the UART code in  pd_debug.cpp.
UART TX is on pin PA2, UART RX is on pin PA3.

If the ZYPDN12 is also supposed to receive commands via UART, you will have to extend the UART code in pd_debug.cpp.
UART TX is on pin PA2, UART RX is on pin PA3.

FWIW, the first commit on my apple-vdm branch enables both RX and TX and adds a stub for an UART debug loop.
Here's a STM32F030F4 quickref with pinouts.  I soldered a pair of enamel wires onto PA2 and PA3 those after removing the screw terminal on my board.

@manuelbl btw, really nice work on putting this OSS firmware (along with the detailed docs) together!

Thanks for the update on your project. Looks very promising.


Rename go-file-format-guide.md to go-ontology-file-format-guide.md

Maybe better not merge this one yet ?

In this case it doesn't matter.
The actual local url to the page is determined by the permalink parameter (on top of markdown files), not the filename


Update learn client app build pipeline

Have updated the dev pipelines with the required values for the HOME, FORUM, NEWS location, etc. we will test a handful of builds there before moving over to production.


Is it allowed to connect via Websocket

I have a running node in sync and can communicate with HTTP API. I need to connect to the node with WebSocket.
I don't see any configuration option for the binary to connect vi a websocket.
I see there is a port 3999 listening also but not sure where it came from.
How can I enable the WebSocket to the node ?

I the mainnet ok


pull request for e2e test at 01/27/2017 01:26:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


added ldconfig to Readme.txt

I started going through the exercises in the book. After compiling pangolin and following the instructions in the book I still couldn't detect the header from my cpp file. After running the command ldconfig it worked. I thought it may help someone to have that added into the instructions here as it took a bit of trouble shooting to figure out why it wasn't working.

also apologies for the trivial diff.

Thanks for PR!


GPU memory

Hi, thank you for your great job about parkour. I encountered some problems about my GPU memory.It said that an illegal memory access was encountered.

Hi, due to isaacgym overrides the cuda backend of pytorch, it probabily skips some validity check when you operating tenors.
Could you locate which line of code that triggers the illegal memory access error?

Besides that,it has shown CUDA out of memory before.I wondered if it may mean my GPU could not run this project?

Hmmm, line 330 in legged_robot_field.py is the process of translating obs_scale in to tensor if needed. This cannot be the code that triggers the error.
For the out-of-memory error, maybe there are too many short episodes (shorter than num_steps_per_env). Try checking the training procedure by remove the --headless command and decrease the num_envs in class env and num_rows, num_cols in class terrain. One possible reason that leads to "too many short episodes" is that, the robot is initialized to a terminate condition.
I run all my experience on Nvidia 3090 GPU with 23GB memory.
Maybe decrease the num_steps_per_env to 8 or 16 and try decrease num_envs when you start training.

OH!Thanks for your recommendations!I guess I could run this project now.By the way, my Gpu is Nvidia 3060 with 6GB memory.I wondered if it could run this project well? Actually I am finishing my undergraduate thesis, that could be important to me. At last thanks for your great work again!

The lack of GPU memory could lower your sample efficiency, thus increasing the time of your experiment iterations.
I recommend renting at least one 3090 GPU or applying for some old but functioning GPU cluster in your school, e.g. P40 or A40 GPUs.

I followed your recommendation then I run this project well. But now I meet another issue. When I played it, it said Segmentation fault (core dumped), actually I didn't do any changes based on your work. Thanks for your great work! I am doing some similar work, it really helps me a lot!

It seems you are using a 3060 laptop? What is your OS? It can't be windows with wsl or docker. I tried but gave up. lol
It probability because of using graphics that requires something like Vulkan, or simply leading to out of memory. Make sure your OS has proper display support (both hardware and software).

Yes but I rented a 3090 GPU. My OS is Linux, Ubuntu 20.04.

Is it a cloud service virtual machine? play.py requires a display. You can try the official example script, e.g. isaacgym_preview4/python/examples/joint_monkey.py

I used to play it at the very beginning. At that time I decreased the num_envs in class env and num_rows, num_cols in class terrain and played it by my own GPU. It could run but I found that my robots didn't learn anything. Then I followed another isuue and I successfully solved this problem. However now I cannot play it again.

Is it a cloud service virtual machine? play.py requires a display. You can try the official example script, e.g. isaacgym_preview4/python/examples/joint_monkey.py

On AutoDL, but not virtual machine.

 Can you successfully run the joint_monkey.py example from isaacgym?

Oh I cannot run the example. I run play.py on my own laptop

I think it is due to my GPU and my graphics card are unmatched?

Sorry for disturbing you so many times. Now I found out where the problem be. The server I rent hasn't been installed the GUI. So I could run isaacgym by --headless but could not play it. I wonder how to solve this problem. Thanks sooooooo much!

You may use Nvidia docker. Please refer to the docker folder in isaacgym.

I'm so sorry about disturbing u so many times but i found out a question right now. Actually I cannot find "A1FieldDistillCfgPPO.runner.pretrain_dataset.data_dir" in a1_field_distill_config.py. So how shuold I run data collection?

Oh, for data_dir, it is for training from a pre-collected trajectory. Maybe check out how scan_dir is used in the runner and the dataset.
For triggering the data collection process, please check scripts/collect.py, which is an example of how to start the data collection.
You can either collect the trajectory completely from teacher policy or collect the trajectory with action label in the DAgger manner.


Add PR template

I would like to add a pull request template for this repository. I believe that having a standardized template will help streamline the contribution process, ensuring that all necessary information is included and making it easier for maintainers to review and merge pull requests.
Could you please assign this issue to me under GSSOC'24.

Assigned @suhanipaliwal

Please add the label of appropriate level like level1, level2 or level3 as per gssoc guidelines on this issue.


Use Dial verb on outbound TwiML:

Add sales phone number field to view
Send sales phone number on the callback URL and dial it on outbound TwiML
Update tests

Thanks @hortega! LGTM

Thanks @acamino !


all removable part ?

I would like to know if it is possible to add all the breakable part on the dps because for example we have on the safi'jiva that the tail but I would like to add the horns ect ?

You can do that by editing MonsterData.json


Fix graph command with "simple" flag

The graph -s command was outdated and broken. I fixed it by copying code from graph.js to graphSimple.js.
There is a lot of opportunity to reduce code duplication in graphSimple.js, but I might be the only person who actually uses that feature.

LGTM!

LGTM!


Page Status Error

Leaving the page status blank when creating a new page generates an SQL error.
SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'status' cannot be null (SQL: insert into `pages` (`title`, `excerpt`, `body`, `slug`, `meta_description`, `meta_keywords`, `status`, `image`, `author_id`, `updated_at`, `created_at`) values (Test, test,
test
, test, test, test, , , 1, 2016-11-07 22:37:30, 2016-11-07 22:37:30))

... as does entering anything other than ACTIVE/INACTIVE.
Setting the field type to select dropdown and adding the following JSON fixes it:
{
    "default": "INACTIVE",
    "options": {
        "INACTIVE": "INACTIVE",
        "ACTIVE": "ACTIVE"
    }
}

@iBourgeois, it's fixed, I've made a PR #166.


Make a meaningful and much more complete README.md

By this do you mean the documentation for how to do cpan release or for how to use LaTeXML-Plugin-sTeX?

no, how to use it, how to install it, what it does, where to find documentation, ...

well done, I reworked this a bit. closing.


Question: subtitle track using three-letter language code

I'm using  video_transcoding to prepare my movies as followed:
transcode-video
--quiet
--no-log
--mp4
--target big
--max-width 1920  --max-height 1080
--main-audio audioTrackId --audio-width main=stereo
--crop detect  --fallback-crop minimal
--burn-subtitle subtitleTrackId
My objective is to convert my mkv files to the most compatible format, while preserving the quality as much as possible (I didn't give the --avbr setting a try yet..).
Everything is working great so far, but where I'm losing time is when I have to identify which track to keep. Source files tracks are ordered randomly, but I always want to keep the english audio track along the french non-forced subtitle one.
According to the documentation, I can select the first input audio track in a specific language using a three-letter code --main-audio fre, then find any embedded forced subtitle track that's in the same language as the main audio track --burn-subtitle scan.
This is not answering my need since what I would like is to pick the first english audio track (--main-audio eng) and the first non-forced french subtitle track (--burn-subtitle fre-non-forced).
I would like to avoid --scaning/using ffmpeg/mkvtoolnix to identify tracks and prepare configuration files for each file.
TL;DR: is there a way to select a non-forced subtitle track by using a three-letter language code, different from the main audio track?

Dear Jonathan Dana,
I use these to get English, Chinese and Japanese subs and audio when available although I prefer to have them as optional rather than burned into the video.
--add-subtitle eng,zho --add-audio eng,zho,jpn
Imagine you can easily change the language code for French.  Not sure about differentiating between forced and non-forced though.
Best,
Platonium
On Mar 26, 2019, at 7:08 AM, Jonathan Dana <notifications<EMAIL_ADDRESS>wrote:
I'm using video_transcoding to prepare my movies as followed:
transcode-video
--quiet
--no-log
--mp4
--target big
--max-width 1920 --max-height 1080
--main-audio audioTrackId --audio-width main=stereo
--crop detect --fallback-crop minimal
--burn-subtitle subtitleTrackId
My objective is to convert my mkv files to the most compatible format, while preserving the quality as much as possible (I didn't give the --avbr setting a try yet..).
Everything is working great so far, but where I'm losing time is when I have to identify which track to keep. Source files tracks are ordered randomly, but I always want to keep the english audio track along the french non-forced subtitle one.
According to the documentation, I can select the first input audio track in a specific language using a three-letter code --main-audio fre, then find any embedded forced subtitle track that's in the same language as the main audio track --burn-subtitle scan.
This is not answering my need since what I would like is to pick the first english audio track (--main-audio eng) and the first non-forced french subtitle track (--burn-subtitle fre-non-forced).
I would like to avoid --scaning/using ffmpeg/mkvtoolnix to identify tracks and prepare configuration files for each file.
TL;DR: is there a way to select a non-forced subtitle track by using a three-letter language code, different from the main audio track?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/donmelton/video_transcoding/issues/275, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOepbaNARQ9keMhMlKfEZvhqGUWssaR5ks5vainzgaJpZM4cLikL.

@Nakda  - Hi there! Hopefully we can get this working for you. It's always a little dicey when your input is random, but --main-audio eng --add-subtitle fra will get you most of the way there. What it won't do is burn the subtitle into the video. It will be set to passthrough if the original is PGS (I'm not 100% certain what happens if the input is non-image based).
I would recommend that you take a peek at the file first in MediaInfo or ffprobe to determine which track you want to burn and add that with --burn subtitle n. The ffprobe command to quickly view this is  ffprobe -select_streams s -show_entries stream_tags=language -v error -i "FILENAME"
What you're looking for would be more of a feature request to change the behavior of --burn-subtitle to be:
 --burn-subtitle TRACK|LANG|scan
                    burn track selected by number into video
                      or first with language code
                      or `scan` to find forced track in main audio language

@donmelton will have to speak to whether or not this is a change that would be considered, or even doable.
Hope this helps somewhat!

@Nakda Thanks for opening this issue with such a great question. And thanks to @khaosx and @platonium for responding so quickly, especially Kris with the best solution and that dandy feature enhancement.
OK, to officially answer your main question, Jonathan:

TL;DR: is there a way to select a non-forced subtitle track by using a three-letter language code, different from the main audio track?

Unfortunately, no. As Kris pointed out, this would require a feature enhancement. So I and the other project contributors will start discussing if, when and how we could do this.
I am so sorry I don't have a simple solution for you.

Thanks everyone for the feedback.
@khaosx I'm already using something similar to ffprobe to identify the tracks I want, but this is the time consuming part of the process I would like to skip.
Not sure the passthrough PGS will be handled by my Plex clients so I will stick with the old method for now unfortunately.
@donmelton Thanks, I'm now aware that it is not possible with the current state of the project. No big deal, I just wanted to be sure in case I was wasting time because I misread the manual :)
I'm no Ruby nor transcoding expert, as I'm using C# since I started programming, but I'm getting used to it and quickly adapted your batch example to Ruby (I find it more convenient since I'm running video_transcoding on Windows without using Linux subsystem).
So maybe I could investigate on the topic myself? If you don't mind, could you give me a quick insight on where to start?

@Nakda Sorry I took so long to reply! I left for a multiple-hour drive across state after my last comment.
Sure, please feel free to investigate and send a patch! The relevant code would be in the prepare_subtitle function of the transcode-video file.
Also, would you and/or @platonium like to join our Video Transcoding Slack? That's where me, @khaosx and other contributors/commenter hang out together to discuss development, usage, configuration and other random stuff.
If so, just send me an email (see: https://donmelton.com/contact/ for my email address) with your email address and I'll send you an invitation from Slack.

@donmelton No worries, thanks for the headsup.
I'm going to send you an email to join the Slack 👍
Do you want me to close the issue for now until I investigate more?

@Nakda Sure, you can close this issue since one of us is bound to open a new issue describing the feature that @khaosx proposed.

Dear Don,
Hope I’m doing this right.  Is there a slack digest?  We don’t use it at work so I would only follow along as I have been.
Best,
Plato
On Mar 27, 2019, at 8:04 AM, Jonathan Dana <notifications<EMAIL_ADDRESS>wrote:
Closed #275https://github.com/donmelton/video_transcoding/issues/275.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/donmelton/video_transcoding/issues/275#event-2233432288, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOepbXJDsuxVyytxbjNK77PfMH0PXk0Tks5va4hygaJpZM4cLikL.

@platonium Github is always acceptable to ask questions, and thanks for asking this one!
Sadly, there's no digest, as it's not really a mailing list type thing. It's more of a real-time chat. And that email would be gigantic - there's a fair amount of random chatter on no particular topic at all. If you ever have the ability to use it, please drop us a note for an invite!
Regards!

@platonium Yes, as @khaosx says, just send me your email address and I'll send you an invitation from Slack. You can even used their Web-based interface. You don't have to install the Slack application.


Create go workspace with cli, acceptance and tools

This allows different dependency trees between these modules while still allows them to be synced together. Also allows the modules not to be within the internal structure and removes the need for separating tests via build tags for acceptance tests.

This seems pretty straight forward. If using go 1.18 or newer, it doesn't seem to require any special usage. It just works.
Are there any downsides to this approach? I wonder if there will be any issues with dependabot.

Are there any downsides to this approach? I wonder if there will be any issues with dependabot.

Seems that dependabot doesn't support workspaces right now, so that's a downside -- I've seen go mod tidy in a module update the go.work.sum file.
Other than that, I was hoping that we could work with dependencies a bit more independent, but that is not the case, so having modules without workspace could be a bit more helpful at the moment. For example, the acceptance tests could run with a different version of docker go API than the CLI, which would help with Testcontainers update (#325). Downside of that, is that I don't think it's good for the projects to drift in versions, an example of that would be tekton/cli vs tekton/pipeline version issue that occurred in #324.
I haven't seen anything that would impede the day to day workflow. We don't really need workspaces, the modules don't depend on each other. An interesting development could be if we were to merge the https://github.com/hacbs-contract/enterprise-contract-controller and https://github.com/hacbs-contract/ec-cli into a single git repository, here workspace would be very beneficial, so if we would like to consider that, this would be a good first step.

I can see the "move acceptance tests to their own directory with their own go.mod file" change, but what part of this change is the "create go workspace"?

but what part of this change is the "create go workspace"?

In essence the go.work and go.work.sum files (https://github.com/hacbs-contract/ec-cli/pull/327/files#diff-70f7e3c361ae136c655130ab253927b5f6e5c987797a095db1bbb1e790930484). The repercussions are in the go.mod files. With go work sync the dependencies get hoisted (or that's my understanding) from modules and then the (newest?) version gets synced down to other modules -- resulting in some version changes that I did not expect when starting on this.
We don't inter-depend between modules so no other changes are visible. If we were, the replace in go.mod that would point to a local directory would be removed.

Aha, thanks.

Have we decided to drop this, at least for now?

Have we decided to drop this, at least for now?

I'll remove the go.work and go.work.sum files & update the title/commit message.


Scaffold needed?

Hi, is it needed to use a Scaffold to show flushbars?

doesnt follow issue pattern. Bug report or Feature request

Ok, where is it supposed that I have to ask that?


does rbac go into the auth bundle? how should it work?

Most simple first iteration:
use rbac bootstrap
receive admin groups from the MC/org to create admin bindings

Dex operator creates a configmap containing admin group info and management cluster name. This configmap can be consumed by athena and rbac-bootstrap so that configuration defaulted.
It would be better if dex-operator could be renamed into auth operator and would reconcile auth bundle apps in the future (not just dex apps)


hey i have made some tweaks on your's project

after seeing your  project  i thought that i can contribute on your project by giving your nav bar with  great UI so i hope you will like these tweaks and will accept my PR so i will wait for your response and feel free to give any  suggestions that will be a great opportunity to improve myself..

@Ashish9836 hey, thanks for the wonderful tweaks. Here are my thoughts on same.

The navbar is not matching with the current aesthetics of the Editpad. We can remove it.

The buttons are getting so prominent and snatching the focus from the actual text area. I was thinking that when user is using the main highlight or focus area should be on textarea, and the navbar button should not distract the user. Attaching the screenshot to show the same.

As you can naturally the eyes are getting distracted by the very prominent buttons.

Need to tweak the night mode as well. In the current night mode with this changes, the buttons are very hard to spot.

I really liked the new button colors and evenly spaced buttons and not getting pushed to left side. The animation on hower is awesome. The editpad is currently themed to look very plain, basic and give a very old age feel to user. Where application is taking a backseat and letting the user to use the editpad for taking notes. With this aesthetic of the editpad, I think we can remove some elements, and make it look something like below.

Notice the buttons are now much smaller. One thing will be great if the buttons start from left side. There is a lot of space on left hand size on the +new button

Feel free to comment if you doesn't agree with some points I mentioned or have some different idea.

@Ashish9836 Just check out the changes, looking really nice. Let me know once the PR is ready.

i have done all changes from my side
feel free to give any suggestion...
and you can accept the pr from my side

@Ashish9836 Merged. Thanks.


Add changelog

https://keepachangelog.com/en/1.0.0/
@AdamSulek @m-paz
From viadot v0.2.0, pls remember to add your changes to the changelog (viadot/CHANGELOG.md) following the above guidelines whenever making a pull request.

Pushed initial version; need to add the rest of updates as well as historical info


Handling Exceptions post outdated link and compilation errors

On the post about Handling Exceptions, where it links to the content on GitHub, it links to a branch/tag 'handling_exceptions' that no longer exists on the repository.
Further down, where it describes how to generate a breakpoint interrupt, my rust compiler tells me that x86_64::instructions::interrupts::int3() is not found, however, replacing it with x86_64's macro unsafe { int!(3) }; works. This would require that the crate definition be marked with #[macro_use].
I just upgraded to x86_64 version 0.1.2, which fixes the issue of finding the int3() function.

Sorry for the late reply.
Thanks a lot for reporting! I was short on time when I published “Handling Exceptions” and decided to postpone the creation of the GitHub tag to the next day. But I still did not have the time to do it :disappointed: .
I'll try to fix both things over the easter weekend.


[feature] add non-trivial optimization techniques to trading environment

such as

observation normalization

observation clipping

reward scaling

reward clipping

see pull request


This still doesn't work on the Mac Desktop Client. I have my own add in that does not work there either although it works in Office 365 and Windows desktop client.

This still doesn't work on the Mac Desktop Client. I have my own add in that does not work there either although it works in Office 365 and Windows desktop client.

Not working on my Outlook for Mac v16.40 (20073000) Desktop either.
Followed the side-loading instructions (and I'm a very experienced IT Pro), but it just hasn't worked.

@alex--thomas Can you check if you're on the new Mac UI? Contextual add-ins don't yet work on the new Mac UI.

@jabelman @alex--thomas For current support, see the documentation article for this feature.
Let us know if this still doesn't work for you.
Thanks.


Testing all the analysis results when testing save for T1

This is a very small PR. T1 returns two analysis results, where one of them contains all the fit parameters, and the other one is focused on the T1 value. Currently the test of save-load instructions for T1 verifies only that the second result before saving is the same as the one obtained after loading. This PR adds the same verification also for the first result.

@nkanazawa1989 There's no reason not to perform the test also with other experiments.
I see that the PR fails after merging with main, I'll check why.

There is no guarantee about the order of the analysis results. The code assumed that the order of the saved results is the same as the loaded ones, and this is what caused the CI failure. I've fixed it.


Updated recipe for pylexique version 1.3.2

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Release of pylexique version 1.3.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


[hmac] Change fifo_empty interrupt type to status

This resolves #21206.
This converts the HMAC Message FIFO empty interrupt from type event to status. Similarly to KMAC (#21657), the design is further changed to make the interrupt fire only if the message FIFO has run full previously. This can happen e.g. if the SHA2 engine is busy and cannot accept more data currently. In other cases, hardware will consume the data faster than software can provide it and there is no point in interrupting the software to tell it to work faster.
Also, the interrupt is not firing if 1) the block is running in HMAC mode and doing the second phase where the Message FIFO is managed entirely by the hardware, or 2) if software triggered the Process or Stop command already (no more input data is accepted).

Thanks for this work, @vogelpi 👍 Glad to see that the DV env can be updated to model and check this behavior without very intrusive modifications. I left two comments/questions, but otherwise this LGTM!

Thanks for your review @andreaskurth . unfortunately, it turns out the DV env changes are much more complicated. With what was in the PR before, the pass rate was not satisfactory and I currently don't have the time to bring it to an acceptable level. Since there are still other more important HMAC changes for which DV is in a better shape. I've now decided to disable verification for the FIFO empty status interrupt. This should be acceptable as no software is currently using this interrupt (previously, one test used the interrupt but the test would have to be refactored anyway to make use of the status type interrupt which is not a priority).
The verification of the status type FIFO empty interrupt is tracked in #21815.

SGTM :+1:


Fix misusage of test results from previous runs

Delete test-run-id.txt and test-result-codecoverage.json, after command
execution.

Codecov Report

Merging #660 (7c706b5) into develop (9c701f0) will decrease coverage by 0.18%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #660      +/-   ##
===========================================
- Coverage    64.58%   64.40%   -0.19%
===========================================
  Files           43       43
  Lines         1423     1427       +4
  Branches       240      242       +2
===========================================
  Hits           919      919
- Misses         504      508       +4

Impacted Files
Coverage Δ

.../core/src/sfpcommands/apextest/TriggerApexTests.ts
20.68% <0.00%> (-1.00%)
:arrow_down:

Continue to review full report at Codecov.

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


[Feature] addColumn move support before and last

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
flink cdc  ColumnPosition support: BEFORE、AFTER、FIRST、LAST
https://github.com/apache/flink-cdc/blob/fa6e7ea51258dcd90f06036196618224156df367/flink-cdc-common/src/main/java/org/apache/flink/cdc/common/event/AddColumnEvent.java#L53
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Is there any progress on this PR? If not, I can contribute this PR

@melin Can you cr please?


Feature Request: Ability to load API Spec from local string

Right now, as far as I can tell, you have to load the API Spec from a remote URL. I have a use case where I need to create a UI that someone can load / edit / create their API Spec in my app.
Is this a contribution you would accept? Or is there some other way to achieve this with the current implementation?

apart from remote URL, you can also load local JSON files.
core focus of this package is to explore and try the APIs.
A full fledged API editor will increase its overall bundle size
In my opinion you may create an API editor as a separate Web-Component and can use RapiDoc with in that to explore it. That way it will be more modular

Just to be clear, i'm not suggesting to bundle the editor into your component. The editor will live in my component/app, I just need to be able to load an api spec into it. Looking at the list of public attributes, and after trying a few things, I don't see how it would work currently.
Can you give me an example of how this could work starting from this:
<rapi-doc id="rd"></rapi-doc>

<script>
    const rapiDoc = document.getElementById('rd');
    const apiSpec = {
        // ... api spec here
    };

    // now what?
</script>

@stephantabor  great idea, looking forward to this feature

So i tried to turn my local JSON into variations of file, blob, and url objects and load them into rapidoc that way, and couldn't get it to work. I have added a first pass of the functionality i need to rapi-doc. If you are interested in contribution I can clean it up and open a PR, otherwise we'll just maintain a fork.

okay, I now get what you are looking for.
as of now there is no way to do this, but wont be difficult to provide it.
The functionality is interesting, and I would be more than happy to take an PR.
the way I envision it is to add couple of methods  (one for json and one for yaml)  to the component.
the method should validate the json/yaml and then further validate the spec and finally render it.

provided a method loadSpec() in release 5.1.0 which takes either a string (containing spec-url) or a spec defined in json object.
checkout this Example for the usage
Let me know if that takes care of your situation

@mrin9 thanks for adding it

Any plans to support the yaml format in loadSpec()?


Inline dice rolls in chat appear as fully black

This is odd, as I thought everything was cleanly targetted to only affect journals, but clearly there is something else at work here.
After some testing, it seems to always happen when Custom Journal is the only module, but not always when some other modules are activated. More testing is needed to see which one(s) is/are potentially "fixing" the issue.

Here is my list of enabled mods. None of these "fix" the issue, inline rolls were still showing up black 24 hours ago.
encounter-builder--v0.2.2;
spell-school-icons--v0.9.1;
resourcesplus--v1.5.5;
actually-private-rolls--v0.2;
adventuremusic--v1.0;
ambientdoors--v2.0.2;
animated-spell-effects--v0.2.1;
arms-reach--v1.1.1;
better-rolltables--v1.5.0;
betterrolls5e--v1.1.9;
bullseye--v1.0.3;
chat-images--v1.1.3;
chat-indicators--v1.0.0;
ChatPortrait--v0.1.4;
chat-scrolling--v0.0.7;
combat-enhancements--v0.1.2;
combatready--v1.1.8;
ct-effect-tooltips--v1.0.0;
combat-utility-belt--v1.1.3;
compendium-folders--v1.0.0;
conditions5e--v<IP_ADDRESS>;
cozy-player--v1.3.1;
cursor-zoom--v1.0.0-compatible-with-0.6.5;
DancingLights--v2.1.5;
daynightlights--v1.0.3;
dndAudioBundle-HammerHome--v1.0;
deselection--v1.4.4;
dice-calculator--v0.5.10;
dww-battlemaps--v1.0.0;
foundry-drmapzo-free--v0.10.0;
dynamiceffects--v0.5.94;
dynamic-illumination--v0.3.3;
ec-maps-remake-by-lios--v2.0.0;
find-the-culprit--v1.0.0;
followme--v1.0.0;
forien-quest-log--v0.4.4;
forien-unidentified-items--v0.2.3;
foundry_community_tables--v0.6;
workshop-welcome-screen--v1.0.3;
fpsmeter--v1.0.0;
gAudioBundle-1--v2.1;
gAudioBundle-2--v2.2;
gAudioBundle-3--v2.1;
gAudioBundle-4--v2.2;
token-drag-visibility--v1.1.1;
scaleGrid--v0.0.12;
grouproll--v0.6.3;
healthEstimate--v1.3.3;
inventory-plus--v0.2;
itemcollection--v1.3.15;
lazymoney--v0.9.7;
lessfog--v0.6.2;
lmrtfy--v1.2.2;
lib-wrapper--v1.0.1;
look-at-that--v0.0.1;
lootsheetnpc5e--v2.0.8;
magicitems--v1.18;
mergewalls--v0.0.11;
mess--v0.11.2;
michaelghelfi--v1.0;
miskasmaps--v1.0;
mountup--v3.0.1;
multilevel-tokens--v0.3.0;
mwpm--v0.4.0;
music-d20-sampler--v1.0.0;
navbar-tweaks--v1.0.3;
no-summon-vision--v1.0.0;
npcgen--v0.0.14;
tomcartos-ostenwold--v1.0.0;
permission_viewer--v0.7;
pick-up-stix--v0.7.9;
polyglot--v1.1;
popout--v0.8;
quicksceneview--v0.0.2;
response-times--v0.2.5;
right-click-enabler--v0.0.1;
sailors-funkyseas--v0.8.98;
smip-compendium--v0.3.0;
searchanywhere--v2.0.4;
ShowDragDistance--v1.0.9;
foundryvtt-show-notes--v0.1;
simplefog--v0.1.6;
skill-customization-5e--v2.1.0;
spellarena--v1.1.0;
furnace--v2.2;
mad-caverns-free--v1.0;
tidy-ui_game-settings--v0.1.15;
tidy5e-sheet--v0.2.17;
token-action-hud--v0.5.24;
token-auras--v1.4;
token-chat-link--v1.0.8;
token-info-icons--v1.1.5;
token-mold--v2.7.1;
trigger-happy--v0.7;
turnAlert--v1.4.0;
turnmarker--v2.6.9;
vtta-iconizer--v2.1.12;
vtta-party--v2.2.3;
vtta-tokenizer--v2.2.1;
wall-height--v2.0.1;

Hmm, that is odd. I forgot to mark this as closed, but I am rather certain I fixed exactly what was causing this (had a misplaced CSS rule that was the obvious culprit, once noticed). As for the other modules "fixing" it, I've mostly come to believe it was actually some caching issue or something of the sort, since the very same setup eventually came out as black as well. So just to make sure, have you updated and tried to refresh without caching (ctrl + f5)?
I won't close the issue just yet, in case this is still a problem, but I would be really surprised if it was.

It is working correctly now. Glad you found and fixed the issue so quickly (before I even reported), I am looking forward to using this module!

Awesome! Sorry for the initial bug, hope it didn't cause too much trouble on your end, and glad you're interested in my module! :) I'll properly close this issue now :P


Add experimental support for nvim

Fix for #4

Don't merge this yet

Ok

For reasons unknown, YankRing causes nvim to hang. Not sure if there is an equivalent plugin that supports both vim and nvim.


same Model hash after converting

hi,
Today i just noticed than after converting my 2.1 ckpt, it ended up with the same Model hash as the 2.0 model which is less than convenient when reloading image.

Hi, thank you for bringing this to my attention.
Please note that this behavior is caused by the safetensors library and is not related to my app.
I hope this helps clarify things for you. If you have any further questions or concerns, don't hesitate to let me know.


Build issue with VSTS (Part 2)

First of all thank you so much for reacting to my issue as fast as you did (#797)! I really appreciate it!
I am getting a similar error, but this time looking for 'Forms' in FFImageLoading instead. 'Error' after updating 2.2.23:
[error]Xamarin\SampleApp\obj\Debug\SampleApp.Views.PurchaseOptionButtonView.xaml.g.cs(18,40): Error CS0234: The type or namespace name 'Forms' does not exist in the namespace 'FFImageLoading' (are you missing an assembly reference?)
This time, the amount of warnings have been cut in half (see below) , is there a workaround for this issue as well?
2017-11-04T14:27:25.1058233Z ##[warning]C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(1988,5): Warning MSB3274: The primary reference "FFImageLoading.Platform, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL" could not be resolved because it was built against the ".NETPortable,Version=v5.0" framework. This is a higher version than the currently targeted framework ".NETPortable,Version=v4.5,Profile=Profile111".
2017-11-04T14:27:25.1058233Z C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(1988,5): warning MSB3274: The primary reference "FFImageLoading.Platform, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL" could not be resolved because it was built against the ".NETPortable,Version=v5.0" framework. This is a higher version than the currently targeted framework ".NETPortable,Version=v4.5,Profile=Profile111". [d:\a\1\s\Xamarin\SampleApp\SampleApp.csproj]
2017-11-04T14:27:25.1068227Z ##[warning]C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(1988,5): Warning MSB3275: The primary reference "FFImageLoading.Forms, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL" could not be resolved because it has an indirect dependency on the assembly "FFImageLoading.Platform, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" which was built against the ".NETPortable,Version=v5.0" framework. This is a higher version than the currently targeted framework ".NETPortable,Version=v4.5,Profile=Profile111".
2017-11-04T14:27:25.1068227Z C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(1988,5): warning MSB3275: The primary reference "FFImageLoading.Forms, Version=<IP_ADDRESS>, Culture=neutral, processorArchitecture=MSIL" could not be resolved because it has an indirect dependency on the assembly "FFImageLoading.Platform, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null" which was built against the ".NETPortable,Version=v5.0" framework. This is a higher version than the currently targeted framework ".NETPortable,Version=v4.5,Profile=Profile111". [d:\a\1\s\Xamarin\SampleApp\SampleApp.csproj]

This fix was also applied to Forms packages. Are you sure:

You have the same package version in every project
Removed all bin / obj directories, then cleaned & rebuild you solution?

I double checked and we have the same package version in every project. (2.2.23)
As far as cleaning the bin/obj. I will have to see if I can do that, I am using the hosted agent in VSTS (building locally has always been fine). I'll look into that and get back to you!

Was the fix also applied 'FFImageLoading.Platform' as well? This seems to be the one getting this warning this time around. I really appretiate the help!

Unluckily, missed this one. It's fixed now, new build soon.

Thanks for the quick update @daniel-luberda. Any idea when I should check back for the new build?

The update worked perfectly. Thank you @daniel-luberda 🎉


Can't install package into PCL Profile 111

Hi,
I'm having difficulty installing this into Profile 111. Which profile(s) do you support?

Could not install package 'Xam.Plugins.Forms.Svg <IP_ADDRESS>'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile111', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

I tried 259. Same issue.

Could you find a solution for this? or simply it doesnt support?

Problem looks like with windows phone 8.1 project. if you remove it in the target list, you can add it.

Are there any plans to provide support for profile 111? The windows phone 8.1 project is really required in this case. Alterntively does anyone know of an alternative for a rounded BoxView with a border?

Problem looks like with windows phone 8.1 project. if you remove it in the target list, you can add it.
I have tried to the instruction but the problem still occurs. Does anyone help?

Me too getting the same problem tried to remove Windows 8.1 but giving this error.


Update dials and gauges plugin UI

Per Aubrey's design

Triage discussion: @aubreydagoog is working on some more updates, will share with @miriamvu and the rest of the team later today.


Add tag prop to Flex/FlexItem + update markup in ProgressBar

These changes will help eliminate some React errors

I don't think this needs to be reviewed. Merging...


fix(openapi): add schema to generated parameters

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

When a required parameter does not have a schema, it cannot be validated by the form
Before:

After:

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thanks!


Invalid package format

Android Kiwi browser is unable to open dist.zip package format in version 0.6, but version 0.5.3 is valid.

I've tested commit 5365a5f and there is no problem in this commit

Unfortunately, Kiwi browser is unable to load the dist.zip file.

Fixed in 0.9.2


WIP: variables plugin

This will probably never be merged as the functionality properly belongs in the client application (e.g. CLUE), but for development and demo purposes it's convenient to be able to work on it here in the slate-editor repo where one can focus on the text-editing pieces of the problem without having to worry about publishing here and subscribing elsewhere.
Things that could potentially be worked on here without involving CLUE:

If text is selected when the toolbar button is pressed, prepopulate the name field with the selected text.
After inserting a chip (or image for that matter), the caret should be immediately after the chip/image so that the user can continue typing.
Enable cut/copy/paste of solo chips (already works for chips within embedded text).
Enable dragging of chips (already works for simple text).
Enable formatting of chip contents (e.g. bold/italic/etc.).
Add dialog controls for enabling/disabling rounding and specify # digits to which to round.

I think for #6 in the list above. This would be something handled by the variable shared model itself. The user would have a precision field and then the computedValue of the variable would take this precision into account.
3 other features that aren't specified well are, but could be tested out here:

handling selection. When the variable chip is selected in the text tile, the variable in the diagram should be highlighted. So we'll need to to have the slate plugin listen for this selection. I'm not sure how easy it would be to demo that here, but perhaps printing the selected variable below the slate editor would be sufficient.
showing more info about the chip on hover and single click. Variables will have descriptions so showing them on hover would be useful. Also it is likely we'll want to allow the user to customize which fields of the variable are shown in the chip. So if the value and unit are hidden, it would be useful to show those in the hover.
handling updates to the variable that happen outside of slate. The variable will change outside the slate form. The user might edit it in the diagram for example. When they are editing it or perhaps just when they are done editing it, the display of the chip in the text should update.


Docs: Add section to show using a store outside Vue.

This pull request adds a section to show how to use a Pinia store outside a Vue file to the Pinia docs.

What is this adding? I don't understand what new information it brings

Under Using a store outside of a component, the Pinia docs just mention that  ... if the pinia instance cannot be automatically injected, you have to manually provide it to the useStore() function but doesn't show how to actually do so.
This pull request adds an example to demonstrate how to inject a Pinia store outside Vue.

Yeah, that's explained in the SSR docs

@posva
What about client side rendered apps? Some people don't use SSR, they expect to find the info on the Using a store outside of a component page.

Your example is just showing how to pass a store as an argument of a function. Are you trying to show anything more?

I see. Thank you but I don't think this should be in the docs.


Add New wallet: Unable to enter amount in Auto transfer section.

https://github.com/bithyve/bitcoin-keeper/assets/62699947/89263022-ffba-44ed-9efd-7705bb659da1

Verified this issue on keeper dev app v 1.1.7(263)


parietal bone def and subterms

abnormal parietal bone morphology MP:0000109
AMENDED DEF: any structural anomaly of either of a pair of membranous bones of the roof of the skull located between the frontal and occipital bones that are large, curved and quadrilateral in outline, articulate with each other at the midline in the sagittal suture, and form most of the superior and lateral aspects of the cranium
SOURCES:
https://www.merriam-webster.com/dictionary/parietal bone
https://quizlet.com/222497251/cranial-bones-flash-cards/
short parietal bone
DEF: length reduction of the paired membranous bones located between the frontal and occipital bones that are large, curved and quadrilateral in outline, articulate with each other at the midline in the sagittal suture, and form most of the superior and lateral aspects of the cranium
SYN: decreased parietal bone length
SYN: reduced parietal bone length
SYN: short parietal bones
thin parietal bone
DEF: reduced thickness of the paired membranous bones located between the frontal and occipital bones that are large, curved and quadrilateral in outline, articulate with each other at the midline in the sagittal suture, and form most of the superior and lateral aspects of the cranium
SYN: thin parietal bones
SYN: decreased parietal bone thickness
SYN: reduced parietal bone thickness

Amended definition of the parent term (MP:0000109) and other child terms.
Added new child terms: short parietal bone MP:0030280, thin parietal bone MP:0030281


Usability tweaks for the canvas.

This adds a function called (draw-canvas ...) so that people don't need to include boilerplate code every time they want to draw a canvas. Also exports canvas-width, canvas-height, image-width and image-height, which I think are useful to have. Finally, it updates the canvas documentation. I left the "stars" example untouched in case it's helpful to see how to draw a canvas using lower-level functions.
Example:
(defsketch test
      ((title "Canvas test")
       (size-scale 1))
    (let ((cv (make-canvas 2 2)))
      (canvas-paint cv +blue+ 0 0)
      (canvas-paint cv +red+ 0 1)
      (canvas-paint cv +green+ 1 0)
      (canvas-paint cv +white+ 1 1)
      (canvas-lock cv)
      (let ((draw-size (* size-scale 2)))
        (draw-canvas cv :width draw-size :height draw-size
                     :min-filter :linear :mag-filter :linear)))
    (setf size-scale (mod (1+ size-scale) 50)))

(I'm not sure how the min & mag filters are supposed to work, but even when I pass :nearest, it still looks like a linear interpolation between the colours).

Woops, I accidentally included my documentation update in this. I'll fix that tomorrow.

I'm not sure how the min & mag filters are supposed to work

Those should be passed to canvas-lock, as they are not updated when image is locked.

Okay, I've removed the accidental commit. Also confirmed that passing :min-filter and :mag-filter to (canvas-lock ...) gets it to behave in the expected way, and updated the documentation of (draw-canvas ...) to make that usage more clear.

@Kevinpgalligan sorry for not asking earlier; it looks that we'll now have three ways of drawing things:
DRAW method that sketch classes are using (normally not overriden)
IMAGE for drawing images
DRAW-CANVAS coming from this PR
--- do you think all of these could use DRAW? (we could make it work for images in a separate PR and still support the old API for compat)

Hmm, I think I can de-duplicate the code that is shared between IMAGE and DRAW-CANVAS. I'm not sure that I see how DRAW is related, though? From my understanding, it's a method that each sketch implements that is called once per drawing loop. Within the DRAW method, i.e. the sketch body, we use functions like IMAGE, DRAW-CANVAS, RECT, LINE, etc. to draw actual shapes or objects. I'm not sure how they can be unified with DRAW or if it would make sense to do so, unless you're referring to a different DRAW interface.

@Kevinpgalligan DRAW is already defined with &allow-extra-keys. It's already used with figures, but figures need a big revamp before they are useful.

I like your suggestion, soDRAW is now used to draw the canvas. Updated the corresponding part of the docs. Also added a TODO so that we don't forget to refactor when adding a DRAW method for images. I'm happy to do that myself once this pull request has been merged.

Thanks!


Allow constraints to be provided in --upgrade-package

Summary
Allows, e.g., --upgrade-package flask<3.0.0.
Closes https://github.com/astral-sh/uv/issues/1964.

Thanks @charliermarsh !

No problem, sorry to steal it from you!

It was going to take me 52 years, it is better this way.
El mié, 10 de jul de 2024, 12:26 a. m., Charlie Marsh <
@.***> escribió:

No problem, sorry to steal it from you!
—
Reply to this email directly, view it on GitHub
https://github.com/astral-sh/uv/pull/4952#issuecomment-2219471763, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ3J7TO5MXIEXZNQ376PD3ZLSSXVAVCNFSM6AAAAABKT7I3U6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJZGQ3TCNZWGM
.
You are receiving this because you commented.Message ID:
@.***>


Can u add lat long?

It would be duplicate data, because you can get latitude and longitude from the "coor" field.
If you need that functionality you can do it easily by modifying this code of the placesCrawlerV2.py file

Original - Line 55 (mapScraper/placesCrawlerV2.py):
try:
    obj["coor"] = str(placesData[i][19][0])+","+str(placesData[i][19][1])
except:
    None

Modification - Line 55 (mapScraper/placesCrawlerV2.py):
try:
    obj["coor"] = str(placesData[i][19][0])+","+str(placesData[i][19][1])
    obj["lat"] = placesData[i][19][0]
    obj["long"] = placesData[i][19][1]
except:
    None

And you will get this output:
  {
    "id": "ChIJP0UWUA9sEg0RuJoxZuLavLs",
    "title": "Sevilla",
    "category": "Gym",
    "address": "C. Amor de Dios, 35, Casco Antiguo, 41002 Sevilla",
    "phoneNumber": "681 96 61 09",
    "completePhoneNumber": "+34 681 96 61 09",
    "domain": "instagram.com",
    "url": "https://instagram.com/sevillagym_oficial?utm_medium=copy_link",
    "coor": "37.3958503,-5.994440399999999",
    "lat": 37.3958503,
    "long": -5.994440399999999,
    "stars": 4.3,
    "reviews": 253
  }


Safari 14 will support webp

Hello,
It seems like Safari 14 will support webp.
The Safari 14 Beta release note :
https://developer.apple.com/documentation/safari-release-notes/safari-14-beta-release-notes
Media
New Features
Added WebP image support.

Thanks,
Eric

As far as I can see, Safari UA is not blacklisted for webp, so if the browser send image/webp as a navigational header, pagespeed will send the webp image

Hello,
It seems like Safari 14 will support webp.
The Safari 14 Beta release note :
https://developer.apple.com/documentation/safari-release-notes/safari-14-beta-release-notes
Media
New Features
Added WebP image support.

Thanks,
Eric

Great news!
@Lofesa  The pagespeed can server webp to any client but the problem is that the Safari cannot read webp. In other words you have to enable the UserAgent Append to ensure that your users can see your webp content successfully --as jpeg ofc and NOT as webp.

Well... you can configure pagespeed to serve webp to all clients.... or not.
pagespeed ServeRewrittenWebpUrlsToAnyAgent off; that is the default.
When this parameter is off, pagespeed only serve webp images to those browser that send a navigational header image/webp. If the browser don´t send ths header, no webp is served.
I don´t know how Safari does this.

I'm trying to understand this thread. I think, as Lofessa said, that if Safari can read webp, and sends Accept:image/webp in its request header, then PageSpeed will send webp to Safari. No customization on the PageSpeed side need to be done.
I did not understand @Poseidonas  response "Safari cannot read webp", which contradicts the premise of this issue.
But in any case I don't think there's any bug here.


`GLIBC_2.29' not found

Hi,
I ran:
near-sdk build contract/index.js

With the following contract:
import { NearBindgen, NearContract, call, view } from 'near-sdk-js';

// The @NearBindgen decorator allows this code to compile to Base64.
@NearBindgen
class MyContract extends NearContract {
  constructor() {
    //execute the NEAR Contract's constructor
    super();
    this.my_greeting = 'Hello Web3 World!';
  }

  // @call indicates that this is a 'change method' or a function
  // that changes state on the blockchain. Change methods cost gas.
  // For more info -> https://docs.near.org/docs/concepts/gas
  @call
  set_greeting({ message }) {
    env.log(`Saving greeting ${message}`);
    this.my_greeting = message;
  }

  // @view indicates a 'view method' or a function that returns
  // the current values stored on the blockchain. View calls are free
  // and do not cost gas.
  @view
  view_greeting() {
    env.log(`The current greeting is ${this.my_greeting}`);
    return this.my_greeting;
  }
}

With the sdk version 0.4.0-2 and Node.js version v17.9.1 I got this error:
> near-sdk build contract/index.js

Building contract/index.js contract...
Creating build directory...
mkdir -p build

Creating build/contract.js file with Rollup...
Near bindgen export done
Creating build/contract.h file with QJSC...
node_modules/near-sdk-js/cli/deps/qjsc -c -m -o build/contract.h -N code build/contract.js
near-sdk build [source] [target]

Build NEAR JS Smart-contract

Positionals:
  source  Contract to build                   [string] [default: "src/index.js"]
  target  Target file path and name    [string] [default: "build/contract.wasm"]

Options:
  --version  Show version number                                       [boolean]
  --help     Show help                                                 [boolean]

Error: Command failed: node_modules/near-sdk-js/cli/deps/qjsc -c -m -o build/contract.h -N code build/contract.js
node_modules/near-sdk-js/cli/deps/qjsc: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by node_modules/near-sdk-js/cli/deps/qjsc)

    at ChildProcess.exithandler (node:child_process:398:12)
    at ChildProcess.emit (node:events:527:28)
    at maybeClose (node:internal/child_process:1090:16)
    at Socket.<anonymous> (node:internal/child_process:449:11)
    at Socket.emit (node:events:527:28)
    at Pipe.<anonymous> (node:net:715:12) {
  code: 1,
  killed: false,
  signal: null,
  cmd: 'node_modules/near-sdk-js/cli/deps/qjsc -c -m -o build/contract.h -N code build/contract.js',
  stdout: '',
  stderr: "node_modules/near-sdk-js/cli/deps/qjsc: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.29' not found (required by node_modules/near-sdk-js/cli/deps/qjsc)\n"
}
 ELIFECYCLE  Command failed with exit code 1.

I am running this on WSL2

Note that if I run it with the version 0.3.0 it works;


feat(cluster) enable access log on cluster listener

Summary
Enables access logs on cluster listener and writes logs to admin access log.
Issues Resolved
Fix #6589

minor note: probably in the config file we should make a comment at the cluster_listen listener that the logs will be in admin access/error log files, to make sure we document this

@Tieske this was added in https://github.com/Kong/kong/pull/6599/commits/58e453cdb7ab264a24cf073b63691a1022473dea.

Thnx great doc comments ❤


Merge main into dev

READY FOR REVIEW
Summary

Release v2.2.0
Nojira: Remove stripe javascript and set images to load lazy
ADAPT-4099: Upgrade storyblok to v2 bridge

Co-authored-by: Yvonne Tang<EMAIL_ADDRESS>Co-authored-by: Shea McKinney<EMAIL_ADDRESS>Co-authored-by: Sherakama<EMAIL_ADDRESS>Co-authored-by: Yvonne Tang<EMAIL_ADDRESS>

cc: @wmui51 @forrestglick @yvonnetangsu @rebeccahongsf @buttonwillowsix


Update to Cadence v1.0.0-preview.32

Description
Automatically update to:

onflow/cadence v1.0.0-preview.32
onflow/flow-go-sdk v1.0.0-preview.34
onflow/flow-go c504b454e576
onflow/flow-emulator v1.0.0-preview.31
onflow/cadence-tools/test v1.0.0-preview.31
onflow/cadence-tools/lint v1.0.0-preview.31
onflow/cadence-tools/languageserver v1.0.0-preview.31
onflow/flixkit-go v1.2.1-cadence-v1-preview.17
onflow/flowkit/v2 v2.0.0-stable-cadence-alpha.23
onflow/flow-evm-gateway v0.15.0

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

Project coverage is 34.40%. Comparing base (c016b78) to head (c09d014).

Additional details and impacted files
@@                   Coverage Diff                   @@
##           feature/stable-cadence    #1638   +/-   ##
=======================================================
  Coverage                   34.40%   34.40%
=======================================================
  Files                          96       96
  Lines                        5917     5917
=======================================================
  Hits                         2036     2036
  Misses                       3604     3604
  Partials                      277      277

Flag
Coverage Δ

unittests
34.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Fix CI and tests by changing testnet node url and temporarily removing accounts tests

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] Commit messages follow the conventional commits spec
[ ] If this is a code change: I have written unit tests.
[ ] If this changes code in a published package: I have run pnpm changeset to create a changeset JSON document appropriate for this change.
[x] If this is a new API or substantial change: the PR has an accompanying issue (closes #1206) and the maintainers have approved on my working plan.

Motivation

This PR fixes CI actions allowing to reliably assert whether other changes are causing regressions. These changes are temporary and issues have been created.
Test Plan

Related issues/PRs

#1206

This is actually very dangerous, since you are literally removing tests just to see the tests not fail. However, I see why this was done: the CI RPC node was down.
I would instead recommend you to ping @frol or me and ask for the ci node to be put back online.
I would directly close this PR to make sure nobody can merge it.

@gtsonevv let’s use near-sandbox/near-workspaces to launch a local node instead of relying on an external node

@frol sure, but what about the failing accounts tests?

@gtsonevv Those needs to be fixed instead of disabled. It is not helpful to mask the test errors, we may as well just keep merging PRs with failing CI - at least this would be a conscious action.


[Candidate 3v3 Mojave Desert ZH v1] Supply Piles in center do not offer space for equal building opportunities

Observed behavior
Supply Piles in center do not offer space for equal building opportunities.
Expected behavior
Supply Piles in center offer space for equal building opportunities.
Screenshots

fixed


Pull request for issue #290

I modified the FormatDate and FormatDateTime methods in Clock.java and Dates.java to take another argument such that users can specify date and time format e.g. MM-dd-yyyy or MMM d, yyyy.  This implementation will accept any formats accepted by java.SimpleDateFormat (http://docs.oracle.com/javase/7/docs/api/java/text/SimpleDateFormat.html). The current default format is not changed.
To run "ant tests", I made some changes in DatesTest.java and ClockTest.java as well.
Here is a test instance (http://issue290-ai2.appspot.com/)
@jisqyv @halatmit @ram8647 @josmas

You need to increment the two component versions in YaVersion.java and increment the version of YOUNG_ANDROID_VERSION. You also need to provide an updater for projects that have older blocks. You need to replace any blocks that use the old versions to instead use your version, using an appropriate default pattern.

I incremented the version number for YaVersion and Clock component. The updater for the old version is also provided. For the blocks that use the older version will get to have a default pattern filled in as a default argument.
Also, another thing I changed is that the default pattern will be given to users like this so that people would get some ideas about how to utilize them correctly.

Here is a test instance (http://issue290-ai2.appspot.com/) -- same as the previous one
P.S.I was working in my branch called issue-#290 and merged it into review290 branch. That must be why I am getting so many commits for merging.
@jisqyv @ram8647 @halatmit @fturbak

Jeff,
Could you sort this out so Grace can issue a pull request?
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Wed, Jun 17, 2015 at 3:22 PM, Grace Ryu<EMAIL_ADDRESS>wrote:

I incremented the version number for YaVersion and Clock component. The
updater for the old version is also provided. For the blocks that use the
older version will get to have a default pattern filled in as a default
argument.
Also, another thing I changed is that the default pattern will be given to
users like this so that people would get some ideas about how to utilize
them correctly.
[image: screen shot 2015-06-17 at 3 18 58 pm]
https://cloud.githubusercontent.com/assets/8516676/8216380/36d461a2-1504-11e5-8cc7-89683a30214f.png
Here is a test instance (http://issue290-ai2.appspot.com/) -- same as the
previous one
P.S.I was working in my branch called issue-#290
https://github.com/mit-cml/appinventor-sources/issues/290 and merged it
into review290 branch. That must be why I am getting so many commits for
merging.
@jisqyv https://github.com/jisqyv @ram8647 https://github.com/ram8647
@halatmit https://github.com/halatmit @fturbak
https://github.com/fturbak
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/405#issuecomment-112919717
.

I just pushed another commit to fix a default pattern.

@graceRyu @jisqyv
Grace,
This is very nice work!   Nice job on getting the default args into place, and on doing the upgraders.
The only problem I see is that the unit tests are failing.  Please take a look.

Hi Hal,
I fixed the ClockTest.java I thought BUILD SUCCESSFUL meant passing the test, but apparently it's not. My bad. Now, you should be able to pass all the tests in TestClock.java
@halatmit @jisqyv @ram8647

@jisqyv Tests pass.
LGTM

Some comments. You need to update the Component Help String in OdeMessages.java (this is something that we should really fix). It is no longer sufficient to just update the description in the DesignerComponent annotation.
If you include clickable links in the documentation string, you must include a target="_blank" otherwise if someone clicks on the link they loose their App Inventor session.
Source code should not contain tab characters and should be indented properly.
I have taken care of this for you. I expect this change to be merged shortly (final tests running now).

@graceRyu MERGED into the "ucr" branch for the next Component release.

Hi Jeff,
Thank you for all your work.!I hope these changes are helpful to some
users. Also, thank you for the comments, I will keep in mind that in the
future.
Best,
Grace
On Tue, Jun 23, 2015 at 2:04 PM, Jeffrey Schiller<EMAIL_ADDRESS>wrote:

Closed #405 https://github.com/mit-cml/appinventor-sources/pull/405.
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/405#event-338106437.


feat(python-client): CRUD operations with custom molecular databases

The PR provides classes and methods in python-client for managing and querying molecular databases.

Default database config value is replaced with a database table field.
Split M\  database names into the name and version. A migration script provided. For backwards compatibility, handle use of the old database names in graphql and python-client.
New python-client data class MolecularDB.
Use better naming for the custom databases graphql queries.
Allow users to delete databases in their groups.
Allow database management mutations when authenticated via an API key.
Add multipart_upload function for handling S3 multipart uploads from python-client.
Add python-client tests for molecular database actions.
Python-client GraphQL queries better formatting.

@richardgoater I'm merging this PR before yours (https://github.com/metaspace2020/metaspace/pull/638). It'll likely cause merge conflicts later when you'll be merging master into your branch. Let me know and I'll help you with that.


Add assistant api

https://platform.openai.com/docs/api-reference/assistants

The Assistant / Threads API has a nested structured and I am currently trying to figure out, how we should access the API. Here are lists of some of the endpoints in three different variations:

Nested Resources

$client->assistants()->create(parameters: [/* ... */]);

$client->assistants()->files()->retrieve(assistantId: 'asst_123', fileId: 'file_123');

$client->threads()->create(parameters: [/* ... */]);

$client->threads()->runs()->create(threadId: 'thread_123', parameters: [/* ... */]);

$client->threads()->runs()->submitToolOutputs(threadId: 'thread_123', runId: 'run_123', parameters: [/* ... */]);

$client->threads()->runs()->steps()->retrieve(threadId: 'thread_123', runId: 'run_123', stepId: 'step_123');

$client->threads()->runs()->steps()->list(threadId: 'thread_123', runId: 'run_123');

Nested Resources with ID parameters

$client->assistants()->create(parameters: [/* ... */]);

$client->assistant(assistantId: 'asst_123')->retrieve(fileId: 'file_123');

$client->threads()->create(parameters: [/* ... */]);

$client->thread(threadId: 'thread_123')->runs()->create(parameters: [/* ... */]);

$client->thread(threadId: 'thread_123')->run(runId: 'run_123')->submitToolOutputs(parameters: [/* ... */]);

$client->thread(threadId: 'thread_123')->run(runId: 'run_123')->steps()->retrieve(stepId: 'step_123');

$client->thread(threadId: 'thread_123')->run(runId: 'run_123')->steps()->list();

Sibling Resources

$client->assistants()->create(parameters: [/* ... */]);

$client->assistantFiles()->retrieve(assistantId: 'asst_123', fileId: 'file_123');

$client->threads()->create(parameters: [/* ... */]);

$client->threadRuns()->create(threadId: 'thread_123', parameters: [/* ... */]);

$client->threadRuns()->submitToolOutputs(threadId: 'thread_123', runId: 'run_123', parameters: [/* ... */]);

$client->threadRunSteps()->retrieve(threadId: 'thread_123', runId: 'run_123', stepId: 'step_123');

$client->threadRunSteps()->list(threadId: 'thread_123', runId: 'run_123');

@nunomaduro What do you think? ATM I would probably go for variation 1.

I would pick from variation 1 or 2.

I'd go for version 2 simply because of the "RESTful" approach of it. You define what resource you're working with as you chain the functions together. I wonder if there's a non-chaining approach so that you basically go for variation 1 but with a specific "fetch" function. I.e ...->fetchAssistantStep(threadId, runId, stepId) (or something in that nature).
Just my 2 cents, I haven't deep dived into the beta API just yet :)
In any case, thanks so much for adding the feature!

@gehrisandro Thanks for kicking of :+1: Already running your PR locally and trying it out :clap:

@gehrisandro :clap: :clap: :clap:  Thank you

@gehrisandro 👏 👏 👏 Thank you

You are welcome!  And thanks for supporting me!

Nice work @gehrisandro !

Good job @gehrisandro 👏
Thank you !

Well done @gehrisandro 🎉

Thank you @gehrisandro, is it possible to upload the new API somehow? I see the docs show that you can, but don't see it in the PHP wrapper

Thank you @gehrisandro, is it possible to upload the new API somehow? I see the docs show that you can, but don't see it in the PHP wrapper

Can you explain by what you mean by "new API"

Thank you @gehrisandro, is it possible to upload the new API somehow? I see the docs show that you can, but don't see it in the PHP wrapper

Can you explain by what you mean by "new API"

Hey, sorry I meant - using the files API, how would we upload a file to OpenAI?
Is it possible to upload a temp file?

Thanks


Patch logging

1. Issue, if available:
Related to https://github.com/aws/karpenter/issues/1166 (won't solve it but will give a hand with future debugging)
2. Description of changes:
Atm,  there is a "black hole" in eviction routine. As long as karpenter thinks there are evictable pods, it will return on
	if !drained {
		return reconcile.Result{Requeue: true}, nil
	}

without any log, thus leaving human operator in the dark as to why nothing is happening.
This change adds a debug logging line covering this scenario (in case of a loop, at least operator will see whats exactly is stopping draining and termination process and will be able to investigate/intervene)
Also: fixed variable name colliding with pod name.
3. How was this change tested?
on local cluster
4. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[X] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I worry a bit on the spam of logging on each loop, as this will make the logs really noisy, even in the happy case.
	if !drained {
		return reconcile.Result{Requeue: true}, nil
	}

If we had a consistent reproduction we could add any verbose logging (or even a debugger) we wanted to and see the specific behavior that way.

I played with this a bit and saw
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.116Z	INFO	controller.termination	Cordoned node	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.117Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-2rhqw for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.117Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-q5lbg for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.117Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-h95r4 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.117Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-dwzvk for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.117Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-sgmk2 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.118Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-q5lbg for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.118Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-h95r4 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.118Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-dwzvk for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.118Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-2rhqw for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.118Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-sgmk2 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.135Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-q5lbg for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.135Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-h95r4 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.135Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-dwzvk for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.135Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-sgmk2 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.137Z	DEBUG	controller	Evicted pod default/inflate-68785b889b-2rhqw	{"commit": "4c1fb8a"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.157Z	DEBUG	controller	Evicted pod default/inflate-68785b889b-q5lbg	{"commit": "4c1fb8a"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.180Z	DEBUG	controller	Evicted pod default/inflate-68785b889b-h95r4	{"commit": "4c1fb8a"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.204Z	DEBUG	controller	Evicted pod default/inflate-68785b889b-dwzvk	{"commit": "4c1fb8a"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.217Z	DEBUG	controller.termination	Enqueuing default/inflate-68785b889b-sgmk2 for eviction	{"commit": "4c1fb8a", "node": "ip-192-168-82-251.us-west-2.compute.internal"}
karpenter-controller-5d979f9b9f-5lc9h manager 2022-02-08T19:07:04.229Z	DEBUG	controller	Evicted pod default/inflate-68785b889b-sgmk2	{"commit": "4c1fb8a"}

Odd to me that we're enqueuing multiple times.

@alekc, I'm not comfortable adding the debug logs since it looks like the controller idempotently (for good reason) continues to add evictable pods to the queue on every loop. This adds a huge amount of noise to the logs, which even at debug level, will muddy the waters beyond what I'm comfortable with. I'm thinking about alternatives to increase the visibility here.

@ellistarn fair enough, I am open to ideas, as long as we can close this black hole (karpenter taking inaction for any reason and operator not having any feedback).

uhm, is it something introduced recently? I am not sure that I ever saw eviction message for individual pod inside events.

Perhaps not. I was looking at 0s          Normal    Killing                   pod/inflate-b8fc9fdcc-fjpff                          Stopping container inflate.

Closing in favor of https://github.com/aws/karpenter/pull/1432


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added brick count statistics

#98 Feature request: displays counts of brick with colors
example:
legofy --stats --palette all image.jpg
{'transparent-flourescent-blue': 426, 'light-stone-grey': 9, 'transparent-green': 4, 'transparent-bluish-violet': 15, 'transparent-light-blue': 10, 'sand-green': 13, 'silver': 5, 'white': 35, 'dark-stone-grey': 6, 'transparent-brown': 6, 'transparent': 62, 'medium-stone-grey': 309}

Coverage decreased (-1.0%) to 94.558% when pulling 7a591a17a18e0da99bb26b1da880cb48f9931f27 on tytoons:master into 0154ef5275d98c5cc6ac5f52c936953d8632bc4c on JuanPotato:master.

Really like that you decided to make a pull request to this old project. Been dead for a while. Glad you took some interest to it.

Coverage decreased (-1.0%) to 94.558% when pulling 5db0a9730d4ce875921273783e973279622a5ead on tytoons:master into 0154ef5275d98c5cc6ac5f52c936953d8632bc4c on JuanPotato:master.

Some of these mistakes were from just last night coding.  This was a really fun project and I thought I would contribute to it.


Make maximum duplicate count configurable

Is your feature request related to a problem? Please describe.
Looking at https://github.com/NationalSecurityAgency/ghidra/blob/b3616a6831320daf71d299ab988a0bf13e052174/Ghidra/Features/Base/src/main/java/ghidra/app/plugin/core/symboltree/nodes/OrganizationNode.java#LL37C22-L37C22, it appears that the limit for duplicate symbols is hardcoded, which is currently frustrating me while navigating a C++ binary, which has stream operator overloads out the wazoo.
Describe the solution you'd like
The limit could be configurable or nonexistent altogether.
Describe alternatives you've considered
Not much else I can consider in this specific case.
Additional context

I'm having trouble understanding how it would be useful to have hundreds or thousands of tree nodes all with the same name and no real way to distinguish them. Can you explain your workflow and how that would be helpful?  I would recommend using the symbol table (with a filter) for these situations where you have many duplicate names. The extra info provided by the symbol table can help tell them apart. Trees become problematic when you have too many nodes under any one parent.

…You distinguish them by their parameters.

Anyway, I can probably use your suggested workaround.

So is your workflow to just click on each identical name and see more info about in the listing? So if you were looking for a specific item, you would just keep clicking them one at a time until you found the one you were looking for? I'm really interested in understanding how you use the symbol tree to accomplish some task. The general feedback for the symbol tree is that it isn't very useful as it is and we are considering redesigning it. So we are interested in understanding all the ways people are using it (or trying to use it).