TypeError: Cannot read property 'forEach' of undefined (Heroku logs)

Tried this example:
https://github.com/Charca/bootbot/blob/master/examples/persistent-menu-example.js
but getting this on logs (Heroku):
TypeError: Cannot read property 'forEach' of undefined
at data.entry.forEach (/app/node_modules/bootbot/lib/BootBot.js:427:23)

My index.js
const disableInput = false;

bot.setPersistentMenu([{
		"title": "More about us",
		"type": "nested",
		"call_to_actions": [{
				"title": "Who are the founders?",
				"type": "postback",
				"payload": "FOUNDERS_INFO"
			},
			{
				"title": "Why was Zamphyr founded?",
				"type": "postback",
				"payload": "WHY_INFO"
			}
		]
	}
], disableInput);

// Persistent menu
bot.on('postback:FOUNDERS_INFO', (payload, chat) => {
	chat.say('Lorem ipsum...')
})

bot.on('postback:WHY_INFO', (payload, chat) => {
	chat.say('Lorem ispum...')
})

bot.start(process.env.PORT || 3000);

When it comes to
bot.setPersistentMednu( [ {...} ] );

I even tried using "Postman" to send a POST request with same JSON data (https://developers.facebook.com/docs/messenger-platform/send-messages/persistent-menu), but still getting
TypeError: Cannot read property 'forEach' of undefined
at data.entry.forEach (/app/node_modules/bootbot/lib/BootBot.js:427:23)

UPDATE
Ok, after couple of seconds the "Persistent menu" works, but Heroku still logs the same error. Can you check that out, if you have time?

Looks like BootBot.js has a handler that assumes entry.messaging exists:
            handleFacebookData(data) {
              // Iterate over each entry. There may be multiple if batched.
              data.entry.forEach((entry) => {
                // Iterate over each messaging event
:( ===>         entry.messaging.forEach((event) => {
                  ...
                });
              });
            }

Capturing the input, I get an object which has no messaging property:
[ { id: '146158229413601',
    time:<PHONE_NUMBER>761,
    standby: [ [Object] ] } ]

I think this problem occurs because you're subscribed to events that the bot isn't equipped to handle by default. It just assumes everything will be one kind of event. Adding if (!entry.messaging) return; at the top of that forEach seems to fix things. Not sure what side effect it has though.

@VladimirDev93 @secretrobotron This seems to be the same issue that's described in https://github.com/Charca/bootbot/issues/86 . The workaround to stop receiving the error is mentioned in that issue as well.

🎉


"Get" and "Put" example

Is there any way to pull out "Get" and "Put"  module ?
I want to control Led through "PUT" and acquire fixed sources data through "GET".

Of course you can do this. I have no time to write an example (sorry about that), but I will explain how to do it. You can do GET, PUT and all the other REST requests with the coapclient:
https://github.com/Tanganelli/CoAPthon/blob/master/coapclient.py
while in the led or where you want to listen to any REST request you have to run the coapserver:
https://github.com/Tanganelli/CoAPthon/blob/master/coapserver.py and edit the resources in order to tell to the program what to do when a rest request ask for a specific resorce.
I hope that this was useful

Hi @MarcoIeni / @Tanganelli ,
Is there any way to dynamically add resource to server using PUT when server is running ?
Thanks,
Venkat


PHP: persistent channel (backport to v1.4.x branch)

This is to backport the PHP Persistent Channel PR #11878 to the v1.4.x branch so that we can release this as PHP release 1.4.3.

Reviewed in #11878 LGTM


Rails/InverseOf false positive

Same as #5235 but for InverseOf.
  with_options inverse_of: :user do
    has_many :portfolio_items, through: :portfolios
  end
  has_many :portfolio_items, through: :portfolios, inverse_of: :user

=>
app/models/user.rb:11:5: C: Rails/InverseOf: Specify an :inverse_of option.
    has_many :portfolio_items, through: :portfolios
    ^^^^^^^^

I'm reading this and it states:

There are a few limitations to :inverse_of support:
They do not work with :through associations.
They do not work with :polymorphic associations.
They do not work with :as associations.

So I'm wondering why do rubocop tells me to add inverse_of on has_many :through ?

Same issue as https://github.com/bbatsov/rubocop/issues/4751 . A fix was made here : https://github.com/bbatsov/rubocop/commit/0ec9f6e646a64d34138dd58b28cc6023bf356818
So we could add a similar check for :inverse_of

I tried verifying some patterns, but it seems that :inverse_of works with :as associations correctly.
class Picture < ApplicationRecord
  belongs_to :imageable, polymorphic: true
end

class Employee < ApplicationRecord
  has_many :pictures, as: :imageable, inverse_of: :imageable
end

class Product < ApplicationRecord
  has_many :pictures, as: :imageable
end

irb(main):001:0> employee = Employee.first
irb(main):002:0> picture = employee.pictures.build
irb(main):003:0> employee.equal? picture.imageable
=> true

However, it does not seem to work with :through or :polymorphic. I'm planning to fix this.

Worth noting that inverse_of is automatically inferred from as in Rails 5.2 so this can be relaxed a bit for >= 5.2. https://github.com/rails/rails/pull/28808

That was already taken care of in the original implementation of the cop - if it doesn't work it's a bug.


qubes-core-vchan-xen: xen_4_10 -> xen

Description of changes
Trivial renaming, without fixing the package.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build qubes-core-vchan-xen


Spells?

Magic, do we speak it?

Probably yes (at least because of the jumps). Which means we also need mana.

We also need it for healers. Therefore it is mandatory.
The spell strength should be partially determined by the wand or staff


Support JPQL positional parameters

The format is ?n where n is an integer > 0: https://docs.oracle.com/html/E24396_01/ejb3_langref.html#ejb3_langref_pos_params
So far in our use cases JSqlParser understands HQL/JPQL just fine, even if it's not its goal.

This should be no problem. I will look into it.


🚀 Feature: Switch from npm-package-json-lint to eslint-plugin-package-json

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Splitting out of #834: I'd like to reduce the number of linters and scripts used in this project. I recently started working on eslint-plugin-package-json. I hope to get it to be a reasonable replacement for npm-package-json-lint.
Additional Info
Blocked on eslint-plugin-package-json getting to feature parity with npm-package-json-lint. Currently tracked in https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/28; will be split out soon.

I'm getting pretty close on the eslint-plugin-package-json end. https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/milestone/1 tracks it having a fleshed out initial recommended config. Once that milestone is done, I'll switch over here.
There are a few require-* style rules from npm-package-json-lint not captured by that milestone, but that's ok. I think the really impactful stuff is in.

Per https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues/42#issuecomment-1902291972, this is now unblocked. Hooray! 🙌


[Bug]: [AutogenStudio] Agents no longer have access to previous conversations within a Session

Describe the bug
This started happenning after I upgraded AutogenStudio from 0.0.44a0 -> 0.0.47.
I then downgraded again, but the behavior persists.
Tried removing the current python virtual envs, and installing autogenstudio in new venv, but the bug persists.
Steps to reproduce

install autogenstudio pip install autogenstudio==0.0.44a0 (this was the last version which I know worked)
start the server via autogenstudio ui --port 8084 --appdir /path/to/ui (I made sure to clean the ui artifacts)
setup a basic workflow with 1 proxyuser and 1 primary assistant agent
in the Playground, create a new session with the workflow from step 3.
input a prompt which outputs a list (e.g give me 10 uses for ginger)
ask a follow up question on any of the 10 points, without specifying exactly the text from the point (e.g pleae elaborate on point 6)

Model Used
I tried gpt 3.5 turbo, 4 turbo, mistral medium, claude 3 opus
Expected Behavior
When I ask the follow up question, a request is made to openai with all previous messages from the conversation.
So considering the previous example, when I ask pleae elaborate on point 6, the model knows exactly which point I am refering to.
Screenshots and logs
userproxy (to primary_assistant):
give me 10 uses for ginger

primary_assistant (to userproxy):
To provide you with 10 uses for ginger, I will combine my knowledge with some research. Let's start by listing some common uses for ginger:

Culinary Purposes: Ginger is widely used in cooking and baking to add flavor to dishes such as stir-fries, curries, soups, and desserts.
Digestive Aid: Ginger is known for its ability to help with digestion and alleviate digestive issues like nausea, bloating, and indigestion.
Anti-Inflammatory Properties: Ginger contains compounds that have anti-inflammatory effects, which can help reduce inflammation in the body.
Immune System Booster: Ginger is rich in antioxidants and has immune-boosting properties that can help strengthen the immune system.
Relief from Motion Sickness: Ginger is often used to alleviate symptoms of motion sickness and nausea.
Sore Throat Relief: Ginger tea with honey can help soothe a sore throat and provide relief from coughing.
Pain Relief: Ginger has been used traditionally to help alleviate pain, including menstrual cramps and muscle soreness.
Blood Sugar Regulation: Some studies suggest that ginger may help regulate blood sugar levels.
Skin Benefits: Ginger has antioxidant properties that can help improve skin health and reduce signs of aging.
Hair Care: Ginger can be used in hair masks or shampoos to promote hair growth and improve scalp health.

Now, let's summarize these uses in a list format using Python:

userproxy (to primary_assistant):
TERMINATE

Modified files:  0
INFO:     <IP_ADDRESS>:55332 - "POST /api/messages HTTP/1.1" 200 OK
userproxy (to primary_assistant):
give me 10 uses for ginger

primary_assistant (to userproxy):
TERMINATE

userproxy (to primary_assistant):
pleae elaborate on point 6

primary_assistant (to userproxy):
TERMINATE

Modified files:  0
INFO:     <IP_ADDRESS>:55336 - "POST /api/messages HTTP/1.1" 200 OK
userproxy (to primary_assistant):
give me 10 uses for ginger

primary_assistant (to userproxy):
TERMINATE

userproxy (to primary_assistant):
pleae elaborate on point 6

primary_assistant (to userproxy):
TERMINATE

userproxy (to primary_assistant):
pleae elaborate on point 6

primary_assistant (to userproxy):
To elaborate on point 6, ginger can be used in various desserts such as gingerbread cookies, ginger snaps, ginger cake, ginger ice cream, ginger pudding, ginger scones, ginger muffins, ginger-flavored frosting, and ginger-infused custards. Ginger adds a warm and spicy flavor to these sweet treats, balancing out the sweetness and providing a unique taste profile. It can be used in both baked and frozen desserts to enhance the overall flavor and aroma. Ginger can be grated, minced, or powdered depending on the recipe requirements.

userproxy (to primary_assistant):
TERMINATE

Additional Information
AutoGenStudio Version: 0.0.44a0
Operating System: MacOS with M2 pro
Python Version: 3.10.12 / 3.11.8

Hi @dragosMC91
Thanks for noting this ...
The short answer is to consider

update autogenstudio pip install -U autogenstudio ..v0.0.48
change the summary_method field in your agent workflow to llm instead of last.

Background

The ability for agents to reference previous conversations is impacted by the type of information stored in the session history.  In AGS, we store the final response from a workflow of agents as history ( not the agent conversations itself).

The final response  is determined by the summary_method field for a workflow configuration. Supported options include

last:  The default is last which saves the last message in the agent conversation. The drawback of last is that if the last message does not contain the actual answer, then that information is not committed to history.
llm (recommended): This uses an LLM  to summarize the agent conversation before it is added to history (recommended).
none: No final message is created.

In your case, the summary message is likely set to last hence the inability to reference previous messages.
I am happy to get some feedback here too.
https://github.com/microsoft/autogen/assets/1547007/1165bbce-51ac-45d9-a319-e85f558c41cf

Hello @victordibia, thank you for the reply and apologies for the delayed response, I just got to testing this and indeed it works as expected.
The details you provided about summary_method are really useful, I initially thought the role of the attribute is just cosmetic. Maybe this should be mentioned in the official docs (https://autogen-studio.com/) I am sure other people had this problem as well.
Also, I now noticed with the latest version of autogenstudio (0.0.54), with summary_method == llm, that you get a neatly formatted summary of the complete answer. So, for this, an additional API call is made to the llm service, maybe this should also be explicitly mentioned in the docs because it will incur extra costs (I think). I am not sure which option is more expensive:

(from previous versions) commit the full response to the session history and thus submit more input tokens on next messages
(current versions) make 1 additional request to summarize the response and only commit that to the session history
Not sure if the above is a valid concern and this is actually how the setup works but I thought 2 might be more expensive since output tokens are 3x (openai models) to 5x (anthropic models) more expensive than input tokens

Feedback on AGS as someone who has been fiddling with the ui tool since it's early days (autogen assistant 0.0.1x ui)

the ability to export user created resources (models, agents, workflows etc) is invaluable - it happened a few times after an AGS version update that I had to purge the DB because nothing was working anymore, now I can just export my resources and import them if this happens again. Suggestion: maybe have an export all resources feature, which downloads a small zip with all relevant folders and their corresponding resources
usability in general was greatly improved - with current versions I can’t think of any real issues i've encountered, I would like more control over how responses are displayed in the UI, maybe have this decoupled from 'summary_method' feature
I would like the ability to be able to customize the proxy user more, currently it executes code locally which is scary, I would like to tell it to use docker. Also when using docker, it would be nice to be able to execute all code within a target container (not specify the image and have the framework create the container) but I guess this part is not related to the AGS ui project, its related to the root project

Overall I think you guys are doing an incredible job and I love this project and will continue to use it

Hi @dragosMC91 ,
Thanks for all of this feedback and suggestion!!
I am still thinking about the right was to store history - in addition to cost, full history can be noisy (imagine an agent conversation with 30 turns and many errors becoming part of the history which can degrade future performance). I am sure we will all learn and converge on some standard, or just make that an option that the user/dev can modify.
Thanks for the feedback on the value of being able to export entities. I like the idea of export all (possibly recreate the db sort of). There is currently some work to rewrite/improve the db capabilities in AGS #1694 . Once that is concluded, we should enable such a feature. We are also trying to improve the overall UI/UX and being able to specify docker container should be integrated here.
Finally, https://autogen-studio.com/ IS NOT OFFICIAL DOCUMENTATION.


Extract the logics of create/delete topology into topo.go

This is the first step of refactoring kne_cli library by making
the code reusable for the Topology manager service.
Test

Unit tests are added.
Compiled and ran the refactored kne_cli in the GCE VM
(with the order of create/show/delete), compared its output
with the same parameters/steps with current kne_cli, and
the output were consistent.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

28 of 51 (54.9%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-4.2%) to 60.364%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

topo/topo.go
28
51
54.9%

Totals

Change from base Build<PHONE_NUMBER>:
-4.2%

Covered Lines:
1724

Relevant Lines:
2856

💛  - Coveralls


Release 1.0.4

List of bug fixes:

Fix ovirt-image-template parameters. rhbz #1489454
Improve shutdown of non-migrable VMs. rhbz #1488526
Require ansible 2.3.1.
Vm infra fix defaults. rhbz #1489893
Add memory_guaranteed parameter to ovirt-vm-infra. rhbz #1490852
Add sockets variable to ovirt-vm-infra role. rhbz #1489888
Add nics to ovirt-vm-infra role. rhbz #1490934
Add timeouts for ovirt-vm-infra role. #1490838
Improve reinstalling of the hosts.
Don't hot-plug disk. rhbz #1490930
Don't run with cloud-init if not specified.
Add suffix to modules with bug fixes. rhbz #1487082
Rebase ovirt_vms module with Ansible fix #27382. rhbz #1491010

@mwperina

ci build please


wxtoproxy fails to open

Installed wxtoproxy per directions, but when I double-click wxtoproxy, it fails to open. First time I did so windows defender threw an error, and I clicked "run anyway". Now when I double click it nothing happens at all, nothing visible in task manager either.

You need to drag it into your wxtoimg installation folder first.


DotNetCoreInstallerV0 task has a grammar mistake

If it doesn't find the requested version of .NET Core from the cache, it says this:
"Cache does not contains this particular .NET Core. Will be downloaded and installed."
The "contains" should be replaced with "contain", but while we are at it, I would suggest to replace the whole string with the following, but since I am no linguist, feel free to ignore the suggestion:
"The cache does not contain the requested version of .NET Core. Downloading and installing it now."

Thanks akunaatrium for the suggestion, we shall update the string.

We have merged the PR for this, hence closing the issue now.


Missing IP Ranges / Documentation for IP Ranges for Self-Hosted Integration Runtime - UK South

[Enter feedback here]
We are attempting to allow the ADF Integration Runtime running on an on prem VM through our Checkpoint firewall. However, the IP ranges specified on this page are not the same as what is being called. We are currently using UK South for our ADF instance.
Here are some examples of the IPs we are seeing (which are currently being blocked by our firewall, which we are having to manually trace and whitelist) which are not documented.

<IP_ADDRESS> (Shows as the same location as UK South but not on the list)
<IP_ADDRESS>
<IP_ADDRESS>

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 33fc045d-cec5-9c08-0eee-8af574b9e76f
Version Independent ID: 94e41877-46e8-f5fa-f1f7-14714db3a447
Content: Azure Integration Runtime IP addresses - Azure Data Factory
Content Source: articles/data-factory/azure-integration-runtime-ip-addresses.md
Service: data-factory
GitHub Login: @nabhishek
Microsoft Alias: abnarain

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Hello @timmo001  and thank you for your question.  You mention

ADF Integration Runtime running on a VM

If the integration runtime is running on your vm, then it is a self-hosted integration runtime, not an Azure (managed) integration runtime.  The IP addresses listed on this page are in reference to the Azure (managed) integration runtimes.  Since you are hosting this runtime on your own vm, the IP address you are seeing would belong to your vm.
Does this help, or did I misunderstand?

So is there a definitive list of

@timmo001  Are you using Data Flows, or only regular pipelines?
I have looked up the IP adresses you provided, and they are Microsoft addresses, some in UK West rather than UK South.

@timmo001  I will assign the issue to author for clarification.
@nabhishek Can you give comment?

@timmo001 We have updated the IP ranges for Self-Hosted IR - UK South region.

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Error upon loading file from hdfs

I encountered this error upon reading a file located in hdfs. Do you guys have some idea why this is happening and how we can solve this?
Py4JJavaError: An error occurred while calling o41.load.
: org.apache.spark.SparkException: Job aborted due to stage failure: Task 1 in stage 2.0 failed 4 times, most recent failure: Lost task 1.3 in stage 2.0 (TID 11, <IP_ADDRESS>): java.lang.ClassNotFoundException: com.databricks.spark.csv.CsvRelation$$anonfun$tokenRdd$1
Thank you so much!

What version of Spark and spark-csv are you using? How are you loading spark-csv package?

Hi Hossein
I am using spark 1.5.2, java
   coalescedSubSetToSave.write()

        .format("com.databricks.spark.csv")

        .option("header", "true")

        .save(outputFile);

This is the version info from my gradle build file. (gradle is a replacement
for maven)
 compile     group: 'com.databricks',   name: 'spark-csv_2.10',

version: '1.3.0'
    testCompile group: 'com.databricks',   name: 'spark-csv_2.10',

version: '1.3.0'
From:  Hossein Falaki<EMAIL_ADDRESS>Reply-To:  databricks/spark-csv
<reply+00725a2f12f2e9b189c8031e6d565d92438fb09e7865cca492cf00000001129aa0e29
<EMAIL_ADDRESS>Date:  Tuesday, December 29, 2015 at 11:28 AM
To:  databricks/spark-csv<EMAIL_ADDRESS>Subject:  Re: [spark-csv] Error upon loading file from hdfs (#223)

What version of Spark and spark-csv are you using? How are you loading
spark-csv package?

Reply to this email directly or view it on GitHub
https://github.com/databricks/spark-csv/issues/223#issuecomment-167860225 .

spark-csv depends on apache commons csv and univocity jar files. I suggest you load the package using spark-submit or spark-shell --packages flag as suggested in the README.

This is now solved by using the --packages option in starting the kernel. Thank you so much for your replies.

I reopen this issue since spark-csv is now working however, it does not infer the schema. All fields are treated as string. I use Spark-1.5 hadoop 2.6 and the latest version of spark-csv. I use IBM spark-kernel to process this: https://github.com/ibm-et/spark-kernel. What I did is that I add spark-csv as environment variable under Spark options:
{
"display_name": "Spark 1.5.1 (Scala 2.10.4)",
"language_info": { "name": "scala" },
"argv": [
"/src/spark-kernel/dist/spark-kernel/bin/spark-kernel",
"--profile",
"{connection_file}"
],
"codemirror_mode": "scala",
"env": {
"SPARK_OPTS": "--driver-java-options=-Xms1024M --driver-java-options=-Xmx4096M --driver-java-options=-Dlog4j.logLevel=trace --packages=com.databricks:spark-csv_2.11:1.3.0 ",
"MAX_INTERPRETER_THREADS": "16",
"SPARK_CONFIGURATION": "spark.cores.max=4",
"CAPTURE_STANDARD_OUT": "true",
"CAPTURE_STANDARD_ERR": "true",
"SEND_EMPTY_OUTPUT": "false",
"SPARK_HOME": "/opt/spark",
"PYTHONPATH": "/opt/spark/python:/opt/spark/python/lib/py4j-<IP_ADDRESS>-src.zip"
}
}


[Feature Request] Ability to hide / minimise filter, view, preset panel

Since I can't seem to find a way to do it my need is probably a feature request. I need to be able to hide or minimise all map controls, specifically the panel that filters, views etc. My use case is follow note map in a narrow window in the sidebar. Too much is hidden by the panel.

It's a very good and worthwhile idea. Just to be honest, I'm not sure I'll be able to get to it soon :-/

As long as it is on a ToDo list I am happy to wait

Same issue as tariquesani, there is a workaround in issue https://github.com/esm7/obsidian-map-view/issues/256 but preferable would be to move it all somewhat lower on the map view.


3.0.1 shape(), size(), ndim(), dType(), itemSize(), nBytes()

These methods will calculate the basic properties of the array like

shape() -  represents the shape of the array.
size()-represents the size of the array.
ndim()- represents the dimensions of the array.
dType() - represents the data type of the data stored in NDArray.
itemSize()  - represents the total number of items stored in the array.
nBytes() - represents the bytes consumed by the array to store the data

points 1-5 are completed


New data model for agent -> control plane communication

Description of the Change
An initial proposal of a new data model for agent / TitusMaster communication.

@fabiokung Yes, we should move task launches to the new model as well.


Data serialisation and transformation

auto-management of file format variants

use shims to automatically generate / interconvert alternative formats (XML, JSON, YAML) - assuming this can be done in a lossless way - again, as part of CI


FUNCTIONS_EXTENSION_VERSION value should be updated

hi team,
based on the discussion here: https://github.com/Azure/app-service-announcements/issues/266
if cx set the value of FUNCTIONS_EXTENSION_VERSION as ~2, the Function runtime will be V3, not V2

if cx want to set it as V2, cx should set the value as ~2.0
I understand you updated in the following, but we may need to update the table as well.
Please update the document as soon as possible to avoid any misunderstanding.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1df5e2bd-489d-19b2-87d3-d993ebb82021
Version Independent ID: 9e9d9ded-8f40-22be-4654-48c87f2c59e8
Content: Azure Functions runtime versions overview
Content Source: articles/azure-functions/functions-versions.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@MrWBQ Thank you for your feedback! We will review and update as appropriate.

@ggailey777 Can we have a confirmation on this? I couldn't replicate this at my end.

Both ~2 and ~3 are no longer supported, as documented in this article. #please-close


Add SignalR to the metapackage

It's not there and it should be by now.
/cc @JunTaoLuo @muratg @moozzyk @natemcmaster

Yes, only on the dev branches though (for 2.1).

I think we will need to do this for preview.

Note that we are moving from packages.csv to artifacts.props pending https://github.com/aspnet/Universe/pull/566. Instructions will be updated but the jist of it is to mark signalr packages as Category="ship" Metapackage="true"


add support for const length arrays ([u8; N])

minimal addition to implement prost::Message over [u8; N] (alongside Vec<u8> and Bytes<u8>.
i've added a helper Newable trait because there's not (yet/always) a blanket Default on [u8; N] and the macro generated code has a lot of, constraints... it's a wonky solution for a problem that hopefully won't exist as things are refactored.

the problem is that there is no way to express that in protobuf.
if you have [u8;20] and the field you are parsing only contains 10 bytes? what do you do? you'd need to add some parameter (len) to track how much of that buffer is filled, etc... at which point you want heapless::Vec<T;N> (which does also implement Default), which the maintainers did not want to add...

the problem is that there is no way to express that in protobuf.

if you have [u8;20] and the field you are parsing only contains 10 bytes? what do you do?

it seems pretty reasonable to me to encode a fixed length field as variable length in the proto, and to fail on the decoding of a mismatched length, as one would with any other invalid or unmatched message / field (though this PR does not yet have that behaviour).

at which point you want heapless::Vec<T;N>

this addresses an issue with prost::Message not functioning over otherwise readily serializable type with explicitly sized fields, i specifically do not want to replace these with variable length versions that don't make sense from the application side. whether or not this is the ideal construction of application and serialisation objects, it is a problem that exists for users of core language primitives.

updated PR to propagate errors, expose BytesAdapter. it'd be great to know if there was any interest in merging something like this prior to spending more effort on it?


anychart cpu 100%

all timer ref in anychart-bundle.min.js should prefix by window. to ensure jsdom will remove all listener after window.close()

Hello!
Thank you for the information.
At the moment our developer team is busy with pre-release work, we will approach this question right after the release.
Kind regards, Roman.

How do you connect/use/configure image magick with anycharts? Any guides? Mine seems not working.

@denghongcai
the AnyChart NodeJS module has been just updated. Our dev team made an effort to fix all known issues. So, we highly recommend you to update the module and please let us know if it won't solve your problem.


NIFI-11757 Upgrade Google Cloud Libraries from 26.15.0 to 26.17.0

Summary
NIFI-11757 Upgrades Google Cloud libraries from 26.15.0 to 26.17.0 for GCP extension components.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 LGTM, thanks for the upgrade! Merging to support/nifi-1.x and main


Support DOI URLs in update_ezid_doi management command

If a URL is detected in the arguments of the update_ezid_doi command,
parse out the DOI from the URL and use that to find the required
preprint

Wonder if it makes sense to have a separate parameter for doi url? Need to update the help text for the command. Also, wonder about 'short_name' - isn't there only one repository for now. Is the identifier to search for repository needed?

Wonder if it makes sense to have a separate parameter for doi url?

It might, but I prefer to just "do the right thing" with whatever is provided.

Need to update the help text for the command.

Good catch, I'll update it.

Also, wonder about 'short_name' - isn't there only one repository for now. Is the identifier to search for repository needed?

We have to look up the repository object somehow, and I didn't want to assume we'd only ever have one.


Summer Meetings Planning

Dear WG Participants,
In order to manage summer meetings. I would like to propose some dates for next meetings, aiming at having at least one meeting per month.
Have synched with Ludovic and He is off the first middle of August and I will have limited availability. So for August is proposed 23th
Proposed Dates

Wednesday, 19th July, 16:00 - 17:00 CEST
Wednesday, 23th August, 16:00 - 17:00 CEST
Wednesday, 06th September, 16:00 - 17:00 CEST

Let me know if that fits
@bigludo7, @rartych , @alabajnaid, @eragaji, @ShutingQing, @FahadTanwir-STC

Actually i'll be available. If you don't mind, let me help on this in case there are new friends coming. I'll collect new comer's proposal if someone have, and bring back to you and ludovic, and then we discuss.

In the next meeting, you can explain with more detail your view on that.
Regarding meetings is hardly to have with more frequency that bi-weekly. Some people attend other tracks and it is also needed to review issues, comments,...
Please connect to the next meeting so we can understand better about your comment

Yeah understood. I know that you and ludovic are maintaining not only carrier billing.
What I'll suggest is:

Let's see is there any volunteers in current maintainers and participants can help on maintain the project.
Pedro and Ludovic can leave some guidance on what need to do, and how we can proceed.

If no one volunteer to maintian, then we consider to lower down the meeting frequency.

Already managed


Data Visualization with D3: Work with Data in D3

Challenge work-with-data-in-d3 has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.95 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
Works in Atom Text Editor - does not pas test in FCC

<body>
<script>
  const dataset = [12, 31, 22, 17, 25, 18, 29, 14, 9];

  // Add your code below this line

const data =  d3.select("body").selectAll("h2").data(dataset).enter().append("h2").text("New Title")

  // Add your code above this line
</script>
</body>

    ```

Closing in favor of #12762


APK availability

Would you consider making the APK of your app available outside Google's walled garden – for those having no access there? Thanks in advance!

I always wanted to also release this on F-Droid, jsut never had the time or patience to work through the process.I'll happily look into creating releases here though, but can't promise anything - very limited time!
But you can also easily compile this yourself. Just clone the repo, run ./gradlew assembleRelease  and find your apk in app/build/outputs/apk/release/ and you should be set.

I always wanted to also release this on F-Droid

Great to hear, and I gladly help you with that! (disclosure: I'm one of the F-Droid maintainers) I also run my own "simple binary repo", so with APKs attached at releases I could quickly add your app there and automatically keep it up-to-date.

But you can also easily compile this yourself.

Unfortunately not: I'm no Android dev and don't have a build environment. My intention was rather making your app available to other F-Droid users via my repo, and then see if the app meets F-Droid's stronger criteria to suggest bringing it there :smile:


React intl filter by in provider details views

issue: https://issues.redhat.com/browse/COST-1372
https://user-images.githubusercontent.com/57504257/129579412-f9ab9051-bc52-4f0e-9a77-d280aa33fa8b.mov

Codecov Report

Merging #2087 (68e8236) into react-intl_changeover (5124cbb) will decrease coverage by 0.09%.
The diff coverage is n/a.

:exclamation: Current head 68e8236 differs from pull request most recent head 52cfa46. Consider uploading reports for the commit 52cfa46 to get more accurate results

@@                    Coverage Diff                    @@
##           react-intl_changeover    #2087      +/-   ##
=========================================================
- Coverage                  70.17%   70.08%   -0.10%
=========================================================
  Files                        407      410       +3
  Lines                       7484     7511      +27
  Branches                    1580     1447     -133
=========================================================
+ Hits                        5252     5264      +12
- Misses                      2205     2222      +17
+ Partials                      27       25       -2

Impacted Files
Coverage Δ

src/locales/messages.ts
100.00% <ø> (ø)

src/api/exports/exportUtils.ts
27.50% <0.00%> (-1.08%)
:arrow_down:

src/api/providers.ts
85.71% <0.00%> (ø)

src/components/charts/costChart/costChart.tsx
10.00% <0.00%> (ø)

...omponents/charts/dailyCostChart/dailyCostChart.tsx
8.44% <0.00%> (ø)

src/api/exports/azureOcpExport.ts
60.00% <0.00%> (ø)

src/api/exports/awsOcpExport.ts
60.00% <0.00%> (ø)

src/api/exports/gcpOcpExport.ts
60.00% <0.00%> (ø)

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 5124cbb...52cfa46. Read the comment docs.


resolve the four points mentioned in GitHub issue 1122.

Self-explanatory. @SteveDiamond let me know if any changes are needed before merging.
(I'll note the commit message is incorrect. The issue is 1122, not 1142.)

This all looks great!


Support batch for all operations

Closes #7

Pull Request Test Coverage Report for Build 055da48ed9e90b7a7f4a112987267c5d3402a7ed-PR-11

111 of 111 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 0bf0369b83c7466839dfe8c36dca3cbebe0e949f:
0.0%

Covered Lines:
190

Relevant Lines:
190

💛  - Coveralls


Laravel 10.x Shift

This pull request includes the changes for upgrading to Laravel 10.x. Feel free to commit any additional changes to the shift-80568 branch.
Before merging, you need to:

Checkout the shift-80568 branch
Review all pull request comments for additional changes
Run composer update (if the scripts fail, add --no-scripts)
Clear any config, route, or view cache
Thoroughly test your application (no tests?, no CI?)

If you need help with your upgrade, check out the Human Shifts. You may also join the Shifty Coders Slack workspace to level-up your Laravel skills.

:warning: Shift upgraded your configuration files by defaulting them and merging your true customizations. These include values which are not changeable through core ENV variables.
You should review this commit for additional customizations or opportunities to use new ENV variables. If you have a lot of customizations, you may undo this commit with git revert 4e6333fa and make the config file changes manually.

:information_source: Shift updated your dependencies for Laravel 10. While many of the popular packages are reviewed, you may have to update additional packages in order for your application to be compatible with Laravel 10. Watch dealing with dependencies for tips on handling any Composer issues.

:information_source: Laravel 10 added PHP type hints to all user-land code included in a new Laravel application. In an effort to modernize your code, Shift added type hints to any method which is used by Laravel, as well as any method which had types defined in its PHP DocBlock.

:information_source: Now with type hints in your code, defining types within PHP DocBlocks is redundant. Laravel has removed all of the @param and @return tags from its DocBlocks where types are defined with PHP. Similarly, Shift removed these tags from any DocBlock where the code now has equivalent type hints.

:information_source: Shift understand developers have different preferences when it comes to type hints. All of Shift's automation is done in nice, atomic commits. This makes it easier to undo any of the changes Shift makes.
If you wish to undo any of these changes, you may first revert the DocBlock changes by running git revert 7ef511f. Then undo the added type hints by running git revert 96006d4.

:information_source: All of the underlying Symfony components used by Laravel have been upgraded to Symfony 6.2. Shift detected references to Symfony classes within your application. You should review the Symfony change log for any additional changes.

:tada: Congratulations, you're now running the latest version of Laravel!
Next, you may optionally run the following Shifts to ensure your application is fully upgraded, adopts the latest Laravel conventions, and easier to maintain in the future:

Upgrade Checker ensures your application is fully upgraded by detecting any outdated code.
Laravel Fixer automatically updates your code to the latest Laravel conventions.
Tests Generator intelligently generates model factories, HTTP Tests, and configuration for your application.
CI Generator intelligently generates CI jobs to lint PHP, check code style, and run tests, including Dusk.

You may also use the Shift Workbench to automate common tasks for maintaining your Laravel and PHP applications.

:alembic: This Shift is still being refined. Please report any issues or suggestions to<EMAIL_ADDRESS>Your feedback is what helps improve the experience for everyone.

:x: PHP syntax errors were detected after running your Shift. Often these are simply differences between the PHP version on the Shift server (8.1) and your project. Occasionally they are misplaced lines or duplicate import statements.
You may quickly check the PHP syntax locally by running php -l on the following files:

[ ] app/Http/Kernel.php


[WIP] [DEBUG] Run CI on DistributedClusterTest only

DON'T MERGE!
Try to fix issue #184 , since the tests passed in local env but failed in github CI. Here we remove other tests first for debugging quickly.

Close it because the reason is found in https://github.com/streamnative/kop/issues/252


settings for video time limit

settings for video time limit.
based on top of #231 Implement time limit to recordings

This looks great. Thanks! A few thoughts for next steps:

Once Fix tests which don't specify explicit URI #265 is merged, you should add a test for video recordings.
We need some UI affordances. We need something to tell the user that a duration limit is currently enabled before they start the recording, and also some kind of UI that tells the user how much time they have remaining before reaching the limit. These can be in follow-up PRs.

Thanks! I'll file some bugs to track these


[Snyk] Security upgrade express from 4.16.4 to 4.17.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

768/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 7.5
Prototype Pollution SNYK-JS-QS-3153490
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: express
    The new version differs by 173 commits.

3d7fce5 4.17.3
f906371 build: update example dependencies
6381bc6 deps<EMAIL_ADDRESS>a007863 deps<EMAIL_ADDRESS>e98f584 Revert "build: use<EMAIL_ADDRESS>for Node.js < 4"
a659137 tests: use strict mode
a39e409 tests: prevent leaking changes to NODE_ENV
82de4de examples: fix path traversal in downloads example
12310c5 build: use nyc for test coverage
884657d examples: remove bitwise syntax for includes check
7511d08 build: use<EMAIL_ADDRESS>for Node.js < 4
2585f20 tests: fix test missing assertion
9d09762 build<EMAIL_ADDRESS>43cc56e build: clean up gitignore
1c7bbcc build<EMAIL_ADDRESS>9cbbc8a deps<EMAIL_ADDRESS>6fbc269 pref: remove unnecessary regexp for trust proxy
2bc734a deps<EMAIL_ADDRESS>89bb531 docs: fix typo in res.download jsdoc
744564f tests: add test for multiple ips in "trust proxy"
da6cb0e tests: add range tests to res.download
00ad5be tests: add more tests for app.request & app.response
141914e tests: fix tests that did not bubble errors
bd4fdfe tests: remove global dependency on should

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Prototype Pollution

Checkmarx AST – Scan Summary & Details – 683f4a10-5797-4848-b1e3-f1cbdfef54b3
New Issues

Severity
Issue
File / Package
Scan Engine

HIGH
CVE-2020-7751
Npm-pathval-1.1.0
CxSCA

HIGH
CVE-2021-23369
Npm-handlebars-4.5.3
CxSCA

HIGH
CVE-2021-23383
Npm-handlebars-4.5.3
CxSCA

HIGH
CVE-2021-32820
Npm-express-handlebars-3.0.0
CxSCA

HIGH
CVE-2021-44906
Npm-minimist-0.0.8
CxSCA

HIGH
CVE-2021-44906
Npm-minimist-0.0.10
CxSCA

HIGH
CVE-2022-3517
Npm-minimatch-3.0.4
CxSCA

HIGH
Cx89601373-08db
Npm-debug-3.1.0
CxSCA

HIGH
Cx89601373-08db
Npm-debug-2.6.9
CxSCA

HIGH
Cx89601373-08db
Npm-debug-3.2.6
CxSCA

HIGH
Cx8bc4df28-fcf5
Npm-debug-3.1.0
CxSCA

HIGH
Cx8bc4df28-fcf5
Npm-debug-3.2.6
CxSCA

HIGH
Cx8bc4df28-fcf5
Npm-debug-2.6.9
CxSCA

HIGH
Cxb3ca64d2-9cd1
Npm-mocha-5.2.0
CxSCA

HIGH
Cxdca8e59f-8bfe
Npm-inflight-1.0.6
CxSCA

HIGH
Passwords And Secrets - Generic Password
/azuredeploy.json: 122, 150, 280, 308
CxKICS

HIGH
Web App Not Using TLS Last Version
/azuredeploy.json: 50
CxKICS

HIGH
Website Not Forcing HTTPS
/azuredeploy.json: 84
CxKICS

HIGH
Website with Client Certificate Auth Disabled
/azuredeploy.json: 79
CxKICS

MEDIUM
CVE-2007-2379
Npm-jquery-3.3.1
CxSCA

MEDIUM
CVE-2014-6071
Npm-jquery-3.3.1
CxSCA

MEDIUM
CVE-2019-11358
Npm-jquery-3.3.1
CxSCA

MEDIUM
CVE-2020-11022
Npm-jquery-3.3.1
CxSCA

MEDIUM
CVE-2020-11023
Npm-jquery-3.3.1
CxSCA

MEDIUM
CVE-2020-7598
Npm-minimist-0.0.8
CxSCA

MEDIUM
CVE-2020-7598
Npm-minimist-0.0.10
CxSCA

MEDIUM
Cx435a6fda-ca38
Npm-commander-2.20.0
CxSCA

MEDIUM
Cx435a6fda-ca38
Npm-commander-2.15.1
CxSCA

MEDIUM
Cx65603961-769c
Npm-debug-3.1.0
CxSCA

MEDIUM
Cx65603961-769c
Npm-debug-3.2.6
CxSCA

MEDIUM
Cx65603961-769c
Npm-debug-2.6.9
CxSCA

MEDIUM
Cxbd6f2b91-dd38
Npm-debug-3.2.6
CxSCA

MEDIUM
Missing_HSTS_Header
/routes/contact.js: 6
CxSAST

LOW
Missing_CSP_Header
/routes/contact.js: 6
CxSAST

LOW
Website Azure Active Directory Disabled
/azuredeploy.json: 22
CxKICS

LOW
Website with 'Http20Enabled' Disabled
/azuredeploy.json: 54, 75
CxKICS


Click 'Enter the globally unique name for the new Function App' result in undefined function id

We are using commands to invoke 'createFunctionApp'.
The code we use:
const functionAppId: string|undefined =
await vscode.commands.executeCommand(
'azureFunctions.createFunctionApp', subscriptionId);
The following item would show

When clicking 'Enter the globally unique name for the new Function App' , the above function would return functionId = undefined

@lirenhe just curious what inspired you to click that? I agree its weird that it closes the inputbox, but I never would've thought to click it in the first place.
Secondly, you should definitely handle the case where we return undefined for the functionAppId (if you're not already). A return value of undefined signifies that the user canceled the operation (usually by hitting the 'escape' key while in the inputbox - not like how you described above).

@EricJizbaMSFT , this issue comes from our bug bash. I agree that people should not click that but we could not avoid it.
From user experience prospective, the expected behavior should be which it is clicked, that click event should be ignored instead of return 'undefined' function id so that user could still type in the box for the function app name.

Yes I completely agree with you. My question was more related to the priority of the bug and I think you've answered that. Thanks


CHEF-13:  remove most of supports API

still there on service (where it makes some sense)
also still on mount (because i have no idea if that is actively being
used or if it makes any sense at all).
converts it to a property on mount + service as well.
bonus bug fix:  the old array-style code was mutating the default values, which it should not do now.

well guess i can't do that -- integration tests for the win

@chef/client-core i think this is going to be ready for review -- travis should be green modulo sunspots.


decrease FSharpChecker's internal cache to 50 projects

With old value 200 when I found all refs of a symbol used in ~90 projects, VS occupied > 2GB of RAM  and became unusable due to continuous GCing.
With the cache of size 50 subsequent launches of "Find all refs" are as slow as the very first one, but VS private set stays at ~1.6GB, which is acceptable.

Is there any way we can make the cache more adaptive to memory usage rather than using a fixed number?

@dsyme I'm not sure how we can do it. We have many cache layers in VFPT, memory usage is a bit unpredictable. Moreover, only a single FSharpChecker instance is used once it is created.

@dsyme About adaptive cache. It would be great if FSharpChecker extends the ctor like this:
let checker =
    FSharpChecker.Create (
        projectCacheSize = 50,
        keepAllBackgroundResolutions = false,
        isMemoryPressure = fun _ -> get_current_process_private_set() > 2GB)

It could call the callback before adding a new entry to the cache and, if it return true, remove the oldest / less recent used entry. However, if new entries hold more memory than old ones, then this algorithm won't work well. FCS could repeat calling the callback + GC.Collect() until the callback returns false, but I suspect that GC.Collect() may take several seconds or more.


Standard breaking Math Constants Definitiion

Why do you require _USE_MATH_DEFINES, along with the inclusion of math.h or cmath? Who do you think you are, requiring that, when no other C/C++ compiler works that way?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f9bc6b17-58bd-4505-5abf-dbe93fda1560
Version Independent ID: 5417cb08-4448-9d2c-d457-ad77ac8b2700
Content: Math Constants
Content Source: docs/c-runtime-library/math-constants.md
Product: visual-cpp
GitHub Login: @corob-msft
Microsoft Alias: corob

@DavidChipman This is a compromise between support for the POSIX/XOPEN standard and the C standard for <math.h>. The C standards (incorporated by reference in the C++ standards) do not define these constants, which means they'd be polluting the global namespace if they were available by default. Way back something like 20 years ago, someone decided that the Microsoft CRT should only expose the POSIX/XOPEN extensions under various guard macros, so the default was strictly conforming to the C standard. Some other runtime libraries were not as careful. Rather than cruft up your code with VC-specific defines, use the /D_USE_MATH_DEFINES command line option to enable the constants.


Cann't build the image_Capture for unable to find the sl_zed_interface.dll

it is like that the sl_zed_interface.dll is created in C , and cann't import in C#.   can  any one  solve it ? thank you very much

sl_zed_interface.dll is a C wrapper library of the C++ SDK. It must be in C so that C# can import the function (through dllimport).
Please follow the instructions on the ReadMe to build the C# Net wrapper that will install sl_zed_interface in the appropriate location.

sl_zed_interface.dll is a C wrapper library of the C++ SDK. It must be in C so that C# can import the function (through dllimport).
Please follow the instructions on the ReadMe to build the C# Net wrapper that will install sl_zed_interface in the appropriate location.

thanks for quick reply.     i build the  stereolabs.zed,and  there are the sl_zed_interface.dll and stereolabs.zed.dll in the  "bin" ,but however when i build the tutorials, it doesn't work well,  and show the error that cann't import the sl_zed_omyerface.dll  because it is a C wrapper.
I am sure that there is no problem before this, so  should i write the code to import this c  wrapper by myself(using dllimport) ,or this project already comes with it ?

sl_zed_interface.dll is a C wrapper library of the C++ SDK. It must be in C so that C# can import the function (through dllimport).
Please follow the instructions on the ReadMe to build the C# Net wrapper that will install sl_zed_interface in the appropriate location.

sl_zed_interface.dll is a C wrapper library of the C++ SDK. It must be in C so that C# can import the function (through dllimport).
Please follow the instructions on the ReadMe to build the C# Net wrapper that will install sl_zed_interface in the appropriate location.

thanks for quick reply. i build the stereolabs.zed,and there are the sl_zed_interface.dll and stereolabs.zed.dll in the "bin" ,but however when i build the tutorials, it doesn't work well, and show the error that cann't import the sl_zed_omyerface.dll because it is a C wrapper.
I am sure that there is no problem before this, so should i write the code to import this c wrapper by myself(using dllimport) ,or this project already comes with it ?

sl_zed_interface.dll is a C wrapper library of the C++ SDK. It must be in C so that C# can import the function (through dllimport).
Please follow the instructions on the ReadMe to build the C# Net wrapper that will install sl_zed_interface in the appropriate location.

thank you for your reply. I get it .   I remove the sl_zed_interface  in the picture and replace it using stereolabs.zed.  and it work well.


Increase Default ReadTimeout to prevent TLS sycall timeout error

Default ReadTimeout of 5s is not sufficient on clients with high load average. TLS syscall errors are seen for petasos and talaria.

Codecov Report

Merging #252 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #252   +/-   ##
=======================================
  Coverage   67.65%   67.65%
=======================================
  Files         156      156
  Lines       10437    10437
=======================================
  Hits         7061     7061
  Misses       2672     2672
  Partials      704      704

Impacted Files
Coverage Δ

server/webpa.go
82.97% <ø> (ø)
:arrow_up:

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 5ab7c20...31b3562. Read the comment docs.

Will do this change specific for comcast hence closing


[Bug]: Data Flushed/Deleted in local Docker setup after restarting the containers

What component(s) are affected?

[X] Python SDK
[ ] Opik UI
[X] Opik Server
[ ] Documentation

Opik version

Opik version: x.x.x

Describe the problem
While i setup Opik locally and use it ,the Traces ,Logs and all are being stored and shown up in the Dashboard but after stopping the Opik running containers and after restarting them the Data is been Deleted/flushed
please sort this this out is there any command or code predefined to delete them or i need to do some modification to solve this persistent memory issue!!
Reproduction steps
No response

@ikharsha I've run the following and wasn't able to reproduce:
cd deployment/docker-compose

# Start Opik
docker compose up -d

# Logged some traces and prompts

# Stopped Opik
docker compose down

# Restarted Opik
docker compose up -d

# -> Data was still available

Can you share a bit more about your setup (Windows vs Mac machine for example) and some code to reproduce using the latest code on the main branch ? The data should definitely be persisted so we might have a bug depending on the setup

Hey @jverre, I deployed Opik on an Azure VM instance and can access it through the VM's URL on PORT:5173. I’ve configured the Opik URL the same way on my personal laptop. I’ve logged some traces, and everything is visible on the dashboard, working perfectly.
However, my VM automatically shuts down at midnight (12:00 AM). When I restart the VM in the morning, I manually restart the Opik dependency containers (Redis, MySQL, Clickhouse, backend, frontend) that were stopped due to the system shutdown. But now, I see nothing on my dashboard—everything appears new, and I’ve lost all the traces/logged data.
`
cd deployment/docker-compose
#Start Opik:
docker compose up -d
#Log some traces and prompts:
#Verified that traces and prompts were visible on the dashboard.
#VM automatically shuts down at midnight (12:00 AM):
#All Opik dependency containers (Redis, MySQL, Clickhouse, backend, frontend) stop due to the system shutdown.
#Restart the VM in the morning:
#Manually restart the containers:
docker start
#Dashboard appears empty, and all previously logged traces and data are lost.
`
I hope this scenario makes sense. Should I run docker compose down to stop the containers before the shutdown and then start them again after the restart? Will this prevent the data loss?

Hi @ikharsha, all data is configured to stay on the local volume. Can you please print here docker-compose.yaml that you use?

`name: opik
services:
mysql:
image: mysql:8.4.2
hostname: mysql
environment:
MYSQL_ROOT_PASSWORD: opik
MYSQL_DATABASE: opik
MYSQL_USER: opik
MYSQL_PASSWORD: opik
healthcheck:
test: [ "CMD", "mysqladmin", "ping", "-h", "<IP_ADDRESS>", "--silent" ]
timeout: 1s
interval: 1s
retries: 300
ports:
- "3306"
volumes:
# Mounted on your $HOME folder
- ~/opik/mysql:/var/lib/mysql/
redis:
image: redis:7.2.4-alpine3.19
hostname: redis
command: redis-server --requirepass opik
ports:
- '6379'
healthcheck:
test: [ "CMD", "nc", "-z", "localhost", "6379" ]
interval: 2s
timeout: 4s
retries: 20
start_period: 30s
clickhouse:
image: clickhouse/clickhouse-server:<IP_ADDRESS>-alpine
hostname: clickhouse
environment:
CLICKHOUSE_DB: opik
CLICKHOUSE_USER: opik
CLICKHOUSE_PASSWORD: opik
# Enables SQL-driven Access Control and Account Management:
# https://clickhouse.com/docs/en/operations/access-rights#enabling-access-control
CLICKHOUSE_DEFAULT_ACCESS_MANAGEMENT: 1
ports:
- "8123" # HTTP default port
- "9000" # Native Protocol port
volumes:
# Mounted on your $HOME folder
- ~/opik/clickhouse/data:/var/lib/clickhouse/
- ~/opik/clickhouse/logs:/var/log/clickhouse-server/
healthcheck:
test: [ "CMD", "wget", "--spider", "-q", "http://<IP_ADDRESS>:8123/ping" ]
interval: 1s
timeout: 1s
retries: 300
backend:
image: ghcr.io/comet-ml/opik/opik-backend:${OPIK_VERSION:-latest}
build:
context: ../../apps/opik-backend
dockerfile: Dockerfile
args:
OPIK_VERSION: ${OPIK_VERSION:-latest}
platform: linux/amd64
hostname: backend
command: [ "bash", "-c", "./run_db_migrations.sh && ./entrypoint.sh" ]
environment:
DOCKER_BUILDKIT: 1
STATE_DB_PROTOCOL: "jdbc:mysql://"
STATE_DB_URL: "mysql:3306/opik?createDatabaseIfNotExist=true&rewriteBatchedStatements=true"
STATE_DB_DATABASE_NAME: opik
STATE_DB_USER: opik
STATE_DB_PASS: opik
ANALYTICS_DB_MIGRATIONS_URL: "jdbc:clickhouse://clickhouse:8123"
ANALYTICS_DB_MIGRATIONS_USER: opik
ANALYTICS_DB_MIGRATIONS_PASS: opik
ANALYTICS_DB_PROTOCOL: "HTTP"
ANALYTICS_DB_HOST: "clickhouse"
ANALYTICS_DB_PORT: 8123
ANALYTICS_DB_USERNAME: opik
ANALYTICS_DB_DATABASE_NAME: opik
JAVA_OPTS: "-Dliquibase.propertySubstitutionEnabled=true"
REDIS_URL: redis://:opik@redis:6379/
ANALYTICS_DB_PASS: opik
OPIK_OTEL_SDK_ENABLED: false
OTEL_VERSION: 2.9.0
OTEL_PROPAGATORS: "tracecontext,baggage,b3"
OTEL_EXPERIMENTAL_EXPORTER_OTLP_RETRY_ENABLED: true
OTEL_EXPORTER_OTLP_METRICS_DEFAULT_HISTOGRAM_AGGREGATION: BASE2_EXPONENTIAL_BUCKET_HISTOGRAM
OTEL_EXPERIMENTAL_RESOURCE_DISABLED_KEYS: process.command_args
OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE: delta
OPIK_USAGE_REPORT_ENABLED: ${OPIK_USAGE_REPORT_ENABLED:-true}
ports:
- "8080"
- "3003"
depends_on:
mysql:
condition: service_healthy
clickhouse:
condition: service_healthy
frontend:
image: ghcr.io/comet-ml/opik/opik-frontend:${OPIK_VERSION:-latest}
build:
context: ../../apps/opik-frontend
dockerfile: Dockerfile
platform: linux/amd64
hostname: frontend
ports:
- "5173:5173"
extra_hosts:
- "apihost:host-gateway"
volumes:
- ./nginx_default_local.conf:/etc/nginx/conf.d/default.conf
depends_on:
backend:
condition: service_started
networks:
default:`
volumes are not reciding at the home folder location specified in the yaml file
- ~/opik/mysql:/var/lib/mysql/
but they found to recide in the parent folder of Opik repo clone

My machine gets shut down everyday so every morning i am starting my containers manually with the following command
docker start
and i am not using the below command
docker compose down
the containers are stopping with the machine being auto shut down by EOD
Could you check out the above comment and help out with the folders - ~/opik/mysql:/var/lib/mysql/

Hi @ikharsha
Thanks for all the details. We implemented the following changes in relation to Docker compose volumes:

https://github.com/comet-ml/opik/pull/626

From your docker-compose print out, I believe you aren't using the latest version. Could you please update your repository and try again? There a some upgrades that affect data storage.
In the meantime, we'd need the following information about you Azure VM:

Operating system and version.
What type of storage you're using:
-- Ephemeral instance storage?
-- Attached volumes?

Please provide as much details as you can.
Thank you very much.

Hi team,
I wanted to know if I update the opik to latest version and do docker compose up -d. The existing data in the local which appears in the dashboard under various projects will be deleted ?

@NaniNorris The data will not be lost 👍 I'll update the docs tomorrow to make it clearer as well

Hi @ikharsha
Hopefully you have been able to resolve your issue, I'll close this for now. Let me know if you still are having any issues and we can see how to best support


DBZ-815 Ensuring latest offset is propagated also if change events re…

…late to non-whitelisted tables
https://issues.jboss.org/browse/DBZ-815

Best reviewed commit by commit. The second one isn't strictly necessary, but I felt it makes things much more straight-forward.

@gunnarmorling Applied, thanks!


The program just dies when I am trying to read a large .xlsx file.

I'm trying to read a large .xlsx file (118 MB, 0.5 million rows, between 20-30 columns and just print the rows to the console.
Here's my code:
#include <OpenXLSX.hpp>
#include<iostream>
using namespace std;
using namespace OpenXLSX;

int main() {

  XLDocument doc;

  doc.open("./my_large_file.xlsx");

  auto wks = doc.workbook().worksheet("Sheet1");

  for (auto& row : wks.rows()) {
        for (auto& value : std::vector<XLCellValue>(row.values())) {
            cout << value << " ";
        }
        cout << endl;
    }

  return 0;
}

The code simply dies. There are no errors thrown during the compilation/build. It simply dies.

Taking this back. This works if I'm running it in a shell outside my editor. I think it's not running the app within the shell inside the editor (VS Code).


Great work!, does it have an "auto play" for the images?

Hi this is a great work, does it have a property or method to play all te images automaticaly?
Thanks a lot.

@MarcosGalaviz it doesn't look like there's a method built in for doing that. You can do something like this though.
setInterval(function() {
  document.querySelector('cat-slider').goNext()
}, 5000);

This means every 5 seconds go to the next slide.


Class composition not loading

Hi,
I am getting setup using css-loader to enable CSS Modules. However, the file that gets served to the browser still contains the compose: ... code.
ie. this is what I get when I look at the CSS being served to the browser.
.item-count__label___1fSMp {
  compose: spanNormal;

  margin-bottom: 25px;

  text-transform: lowercase;
  font-style: italic;
  font-size: 90%;
  line-height: normal;

  compose: lightTextColor from "../shared/colors.css";
}

What could be causing css-loader to not replace the compose code with the code from the external file?
Thanks,
Jason

You should write composes instead of compose as per docs.
On Sun, Sep 27, 2015, 04:13 Jason Silberman<EMAIL_ADDRESS>wrote:

Hi,
I am getting setup using css-loader to enable CSS Modules. However, the
file that gets served to the browser still contains the compose: ... code.
ie. this is what I get when I look at the CSS being served to the browser.
.item-count__label___1fSMp {
compose: spanNormal;
margin-bottom: 25px;
text-transform: lowercase;
font-style: italic;
font-size: 90%;
line-height: normal;
compose: lightTextColor from "../shared/colors.css";
}
What could be causing css-loader to not replace the compose code with the
code from the external file?
Thanks,
Jason
—
Reply to this email directly or view it on GitHub
https://github.com/webpack/css-loader/issues/148.

Oh wow. I really should have read that closer...


Fix joining channels from the find channels screen

Summary
In the Find Channels screen, we have two different sets of channels: those coming from the database (that we have already joined) and those coming from the server (that we haven't joined yet). We make this distinction in the render by sending different onPress functions, whether they need to switch or join the channel.
But we were passing both type of channels to the same function that was only trying to switch, not to join also the channel. This PR fixes that issue.
Ticket Link
https://mattermost.atlassian.net/browse/MM-56349
Release Note
Fix issue where it was not possible to join a channel from the "Find channels" screen.

/update-branch

/update-branch

/update-branch

/cherry-pick release-2.13

Cherry pick is scheduled.


Store git_ssh_key in Juju Secret if using Juju 3

Enhancement Proposal
Following the work on https://github.com/canonical/cos-configuration-k8s-operator/pull/67, we should store the git_ssh_key in Juju secrets when using Juju 3

Example implementation: https://github.com/canonical/traefik-k8s-operator/pull/142


Feature/energy balance init

This PR adds the files for the energy balance submodule and the setup() of variables that are required for the energy balance. The setup is copied from the simple abiotic model and there is an open to do to make the handling of constants neater, I would like to leave this for now as we are still moving constants around. I also deleted a few unneccessary bits here and there.
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Only thing I might suggest is making the open TODO into a Github issue, basically so that the issue is recorded somewhere that other people check


XSS attack in webview

Hi. I'll cut to the chase. We are using version 5.7.2+1 of your plugin. Our security department found a vulnerability in your plugin. Reflected XSS vulnerability. The security department recommends adding character filtering to the url. For example, whitelist or blacklist. Especially the symbols “<”, “>”, ' " ' .
Below is an excerpt from the report. Can we expect a fix in the next releases? Thank you.

I think that user input validation/filtering/whatever should be done at the application level (by you on Flutter/Dart side), not by the plugin.
If you need to validate or filter user data, do it with Dart before sending it to WebView.
You would have done the same

The plugin, in some cases, just exposes the native methods, in others, adds new features.
For example, postUrl is the native method you would use if the app was written in Java/Kotlin for Android.
In that case, you would do your user data/input validation in Java before calling the method.
As we are using Flutter/Dart, you will do the same from Flutter/Dart side. That's the same.


Is it possible to delete all data (for example when user logs out) of core data? Not the models but data in the models

Using SugarRecord with Swift 3. Is it possible to delete all data (for example when user logs out) of core data? Not the models/entities but data in the models/entities?
My SugarRecordManager.swift
  class SugarRecordManager
  {
  static let sharedInstance = SugarRecordManager()
   private init(){

   }

// Initializing CoreDataDefaultStorage
func coreDataStorage() -> CoreDataDefaultStorage {
    let store = CoreDataStore.named("db")
    let bundle = Bundle(for: type(of: self))
    let model = CoreDataObjectModel.merged([bundle])
    let defaultStorage = try! CoreDataDefaultStorage(store: store, model: model)
    return defaultStorage
}

// other functions for fetching and storing objects

// want to write a function here to wipe out all data
}

Hey @mabdullahsaeed, it's possible to delete all the data calling the method removeStore() on your storage. Let me know if that helps.
https://github.com/carambalabs/SugarRecord/blob/master/SugarRecord/Source/CoreData/Storages/CoreDataDefaultStorage.swift#L104


Add Footer

Toy with the idea of adding a footer

A footer has been added, but it should be anchored to the bottom of the page.


Problem starting with bochs - "connection refused"

I compiled bochs from the provided repo and got the following message:
Connection failed: Connection refused
telnet: Unable to connect to remote host: Connection refused
   0: Trying ::1...
   0: Trying <IP_ADDRESS>...
00000000000i[      ] LTDL_LIBRARY_PATH not set. using compile time default '/usr/local/lib/bochs/plugins'
00000000000i[      ] debugger using rc file './bochs/dbg_command.txt'.
00000000000i[      ] BXSHARE not set. using compile time default '/usr/local/share/bochs'
00000000000e[SIM   ] get_param_bool(unmapped) could not find a parameter
Connection failed: Connection refused
telnet: Unable to connect to remote host: Connection refused
   0: Trying ::1...
   0: Trying <IP_ADDRESS>...
Connection failed: Connection refused
telnet: Unable to connect to remote host: Connection refused
   0: Trying ::1...
   0: Trying <IP_ADDRESS>...
Connection failed: Connection refused

This connection failure repeated forever until I killed the script.
Any idea what may be causing this?

Seems that I'm somehow managing to segfault bochs...?
❯ Bochs/bochs/bochs -q -unlock -rc tests/bochs/dbg_command.txt -f tests/bochs/linux_amd.bxrc
00000000000i[      ] LTDL_LIBRARY_PATH not set. using compile time default '/usr/local/lib/bochs/plugins'
========================================================================
                      Bochs x86 Emulator 2.7.svn
               Built from SVN snapshot after release 2.7
                  Compiled on Feb 15 2023 at 11:02:57
========================================================================
00000000000i[      ] debugger using rc file 'tests/bochs/dbg_command.txt'.
00000000000i[      ] BXSHARE not set. using compile time default '/usr/local/share/bochs'
00000000000e[SIM   ] get_param_bool(unmapped) could not find a parameter
zsh: segmentation fault (core dumped)  Bochs/bochs/bochs -q -unlock -rc tests/bochs/dbg_command.txt -f ``` (here bochs is the executable built from your patched gcc branch)

Turns out I wasn't accounting for libraries that would be installed using make install - I solved this by using CONFIGURE_ARGS=--prefix=$HOME/bochs_install sh .conf.linux when building so it would install to a discrete directory

I got a similar error yesterday in class.
[CPU0  ]i| RDMSR: Read 00000000:fee00900 from MSR_APICBASE
[CPU0  ]i| RDMSR: Read 00000000:fee00900 from MSR_APICBASE
[CPU0  ]i| RDMSR: Read 00000000:fee00900 from MSR_APICBASE
[CPU0  ]i| RDMSR: Read 00000000:fee00900 from MSR_APICBASE
telnet: Unable to connect to remote host: Connection refused
   0: Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
   0: Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
   0: Trying <IP_ADDRESS>...
telnet: Unable to connect to remote host: Connection refused
   0: Trying <IP_ADDRESS>...

        Sl   00:17   0:20 /init
root         7  0.0  0.0   1820    88 ?        Ss   00:17   0:00 /init
root         8  0.0  0.0   1820   104 ?        S    00:17   0:03 /init
wenyi        9  0.0  0.0   6328  5368 pts/0    Ss+  00:17   0:00 -bash
wenyi    27577 99.9  1.4 1149456 119248 pts/0  R    02:51  60:41 bochs -q -unlock -rc ./bochs/dbg_command.txt -f ./bochs
root     28094  0.0  0.0   2688   460 ?        Ss   03:19   0:00 /init
root     28095  0.0  0.0   2688   460 ?        S    03:19   0:00 /init
wenyi    28096  0.0  0.0   6200  5428 pts/2    Ss+  03:19   0:00 -bash
root     29031  0.0  0.0   2824   604 ?        Ss   03:51   0:00 /init
root     29032  0.0  0.0   2824   604 ?        R    03:51   0:00 /init
wenyi    29033  0.1  0.0   6200  5080 pts/1    Ss   03:51   0:00 -bash
wenyi    29046  0.0  0.0   7780  3172 pts/1    R+   03:51   0:00 ps -aux
wenyi@LAPTOP-KO2DFVQ8:~$ kill -9 27577

After that I was able to successfully build:
UEFI Interactive Shell v2.2
  12: EDK II
  12: UEFI v2.70 (EDK II, 0x00010000)
  12: Mapping table
  12:       FS0: Alias(s):F0a:;BLK0:
  12:           PciRoot(0x0)/Pci(0x1,0x1)/Ata(0x0)
Press ESC in 1 seconds to skip startup.nsh or any other key to continue.
  14: Shell> fs0:
  14: FS0:\> copy -q fs1:rhv.efi rhv.efi
  14: cp: File not found - 'fs1:rhv.efi'
  14: FS0:\> rhv.efi snapshot.img snapshot_patch_empty.json corpus

Hope this helps :wink:


Fix proxy to work with Russian test user

Fix proxy to work with Russian test user.
Add router function to proxy config to work with Russian data localization.
When target changes logging looks like this:

If the unit tests aren't wanted, I can remove them. They helped me make sure I covered all the cases but it didn't seem like the package was setup to handle them. I ran them directly inside the folder with npx jest ./getDataLocalizationRouter.test.js.
See https://familysearch.slack.com/archives/C06FR9QDBRB/p1716312966290559 and https://fhjira.churchofjesuschrist.org/browse/RUSSIA-463 for background.

@redbugz does this pull request look good? Or do you or @joeycozza want to review and let me know?

OK. With keeping the test file I had to omit it from the package.files so that it doesn't get published. So just pushed that change.


README: Add info re updates to the app loaders

Merge or wait, both are fine 🙂

Cool, let's give them a little while and then merge if we don't hear back, maybe by the weekend

Ok let's merge this


A Warning，No result

imagededup-0.1.0-py3.7.egg/imagededup/utils/plotter.py:66: UserWarning: tight_layout : falling back to Agg renderer

Hi the warning is normal as matplotlib is using agg as backend. Can you please give more details? For example, what kind of data you use, what kind of methods and where e.g. in CLI or in a Jupyter notebook. Thanks.

Hi the warning is normal as matplotlib is using agg as backend. Can you please give more details? For example, what kind of data you use, what kind of methods and where e.g. in CLI or in a Jupyter notebook. Thanks.

from imagededup.methods import PHash
phasher = PHash()

encodings = phasher.encode_images(image_dir='./photo/')

duplicates = phasher.find_duplicates(encoding_map=encodings)

from imagededup.utils import plot_duplicates
plot_duplicates(image_dir='./photo/', duplicate_map=duplicates, filename='dup.jpg')

Test data: one of the two pictures is a copy of the other, only the name is different.

Hi, it would be great if you can share the following info:

Your OS
Are you generally able to plot using matplotlib in your current settings?
The output of import matplotlib; matplotlib.get_backend() in your current settings
Are you working in a dedicated virtual environment? (if yes, using virtualenv or conda?)
Your IDE

The issue might be related to: https://www.pyimagesearch.com/2015/08/24/resolved-matplotlib-figures-not-showing-up-or-displaying/ OR https://github.com/matplotlib/matplotlib/issues/1852

Some of the following information, thanks.
OS: CentOS 8 64bits;
First time using matplotlib;
>>> import matplotlib;
>>> matplotlib.get_backend()
 'agg'

Python 3.7.4 (default, Aug 13 2019, 20:35:49) [GCC 7.3.0] :: Anaconda, Inc. on linux

It seems that the issue is unrelated to this package. The link to the problem I shared earlier points to the issue to be specific to .nix OSes. (Here is the link again.).
So, I would suggest first ensuring that matplotlib works generally on your system. If you still face problem with matplotlib with this package only, then we can have a look at it.

Thanks,As you said,matplotlib figures not showing up or displaying generally on my system,this problem has been solved and can be used normally.


update Enas mutator.py

fix the issue #2091

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

@marsggbo , thanks for your contribution, could you merge the latest update of master branch to your branch?


Block PRs from merging if the link checker fails in the documentation repo

Is your feature request related to a problem? Please describe
The link checker in the documentation repo runs on every pull request and subsequent commit. Even when the link checker fails, the merge button is still enabled and the PR can be merged. So broken links sometimes seep through to the codebase.
Describe the solution you'd like
We'd like to disallow merging when the link checker has found broken links and is failing. The other workflow (vale) is set up as non-blocking and should not prevent the PR from being merged.
Describe alternatives you've considered
The alternative relies on individual diligence which is hard to enforce.
Additional context
No response

@CEHENKLE for visibility

@bbarani  what do you think?  Can we make this a blocking check?

I have enabled the mandatory status checks before merging in main branch.

@kolchfa-aws Let us know if this works now.

I see that above check is only enabled on main branch. For enabling on all branches you can modify the branch specific rules here: https://github.com/opensearch-project/documentation-website/settings/branches
Closing this issue as no action item is pending from our end.
Thanks!


Peerless Wind God times out even with timelock enabled

As seen here

The video is gone

Iirc to replicate it you can turn on invincibility and infinite timer, wait until way after the spell would've ended normally, then turn invincibility off and miss

I did exactly that and I couldn't replicate the issue


Enhancements: Postgres, use /ui/api to fetch backend status, refactor to io.trino.gateway, add timeout to prepareStatement operation

Add routing for requests to healthy backends
Deactivate clusters with zero workers
Add PostgreSQL support for backend database
Rename and repackage from Presto to Trino

I'd like to put everything in one commit. Let me know if there are anything else to fix!

Can you please just do fixup commits and stick with the four we have
so you just do
git commmit --fixup=hashofcommittofix
and then once you have them all in
git rebase -i HEAD~5 --autosquash

I see. Np.


Deploying builds from AppVeyor

I'm using AppVeyor for PR builds and would like to have this handle my production builds as well since it's already building my master branch. From what I've read I should setup AppVeyor to produce a web deploy package which can then be pushed to my app service using the settings in the publish profile. I'm assuming if I switch over to web deploy that kudu needs to be disabled and isn't part of the process anymore, is that correct? Are there any other steps that need to be taken to switch my deployment process from kudu to web deploy?
I was also wondering if web deploy was the right option for deploying from a build server like this?

Hi @xt0rted - I've done heaps of AppVeyor work -> to -> Azure App Services.
If what I think you're asking is: Can I use KUDU for everything except production builds?? I'm not sure (sowwy) but what you will need to look at is if KUDU can only response to specific branches or any branch except a specific branch.
There is some Kudu Deployment documentation that suggests you might get kudu to only kick off on a specific branch. Bottom of that file it mentions having a deployment_branch app setting.  This means you can't have any branch name, but only a specified one ... which could be a bit of a PITA for your usecase? To me, this all feels a bit clunky (if this is the way to do it).
My preference would be to use AV for build/test/deploy but if your repo is private, then you will have to pay.
If you do go down 100% AV, basically this is what I've been doing (OSS and private repo's)

Turn off any KUDU stuff in the App Service
Download the Publish Profile for the App Service. (see image below)
Create an appveyor.yml file (case sensitive. yes. i just said that).
Build my code, test my code, zip contents into a single zip and then web deploy to azure (either main slot or staging slots, if you use staging slots).
💰 profit.

The hardest part is making sure you

figure out which parts of the Publish Profile you need to copy/paste
encoding your secret password using AppVeyor.

So first up, here's two examples of 2x AV files I have used for OSS:

.NET Core project
.NET Full Framework project

To encode a secret password for AppVeyor, click this...

And to get more information about how to do all of this, I referenced this AV wiki page for my detailed help.
I'm happy to help with any Q's. No, I don't work for AV but just love it :)

Thanks @PureKrome. I've been using AppVeyor for a few years now for all of my projects but the only deployments it's doing are publishing NuGet packages for me. I'm pretty sure your high praises for it on Jabbr were why I switched to it from TeamCity in the first place.
Currently I have AV setup to build & test my PRs along with build any commits/merges to master. That's where it stops though. Those commits to master also trigger the kudu deploy which does its own build. This is what I'd like to get rid of since AV is already doing the build, it's just not being deployed.
I've had a continuous delivery process since the start of this project but there's a number of rough edges to it. The unused AV release build is one. The others are not using a slot to warm up the deploy, and running my EF migrations automatically on startup. None of that is ideal but it got me going so now I want to improve on it.
My dream setup is to have AppVeyor do the following:

Build the site, compile my views, run gulp, do ✨ things
Deploy to a slot on Azure
Deploy static assets to the CDN
Migrate my dB
Warm up the deployment slot
Swap to production
Ping search engines that things have changed

I've read through a bunch of docs & blog posts on doing all of this but what seems to be tripping me up is how to handle EF and its migrations which throws a hissy fit the moment the schema changes. I've read plenty on setting up auto swap slots and deploying from AV, but nothing seems to cover how to properly handle dealing with EF migrations while keeping the site live.
For now my AV deploy changes are sitting in a dusty PR since I couldn't spend any more time on this. I'd love to pick it back up and get it done once and for all. I know moving away from EF could solve some of my problems, but for now it's too useful in such a young project that I don't want to move away until things become more stable (ripping out asp.net identity....that's another story and I can't wait to rip that 💩 out).

I only ever saw EF migrations with JabbR's source code which Fowler setup. We've always done it manually and made our live deployments a 3 step process

code (AV CI/CD to staging slot)
DB (if required, most of the time this is skipped)
Flip switch

so that's our weakness in our CI/CD link.

After describing the process and thinking about it some more it might be fine to deploy to a slot, warm it up, then trigger the migrations and swap. The window for an EF schema error is small enough that it shouldn't be an issue at the moment (most updates while someone is on the site don't touch the dB anyway) and any that do come up should be gracefully handled anyhow. I have a test environment that mirrors production and deploys at the same time, so I could try switching the deploy process on that environment first then move it over to production when ready. Maybe later in the week I'll have to try dusting off that PR.
Thanks for bringing this back up and listening to me while I work through this.

👍 👍

@PureKrome I started setting up my web deploy step and it's generating the correct output, but I noticed it's not including any .pdb files aside from the one for the web project. Is there a magical setting I'm missing somewhere to include those? My kudu based deploy script includes them and the msbuild call is the same aside from including /t:Package.

@xt0rted Does this SO question/answers help?

I didn't follow this whole thread, but please do note that if you are deploying to App Service via msdeploy (whether from VS, AppVeyor, Octopus, ...), you are not using Kudu. Given that, maybe this issue is not actually Kudu related?

@PureKrome that was one of the pages I found while looking into this, but I already have that option unchecked. This is why it's annoying, running msbuild against my solution or projects does produce the .pdb files, but when building the web deploy package only the target project's .pdb is included. I guess it's off to SO with that question.
@davidebbo in general what's being discussed now isn't kudu related. My original question was about what needs to be done to disable kudu in order to migrate to the web deploy process. That sort of went off the rails as we started talking about how to get the deploy package being built so I could then get to the point of disabling kudu. The only reason I've kept the conversation on here is because once I was finished I wanted to provide clear steps of what I needed to do for anyone else coming across this trying to do the same.

Understood. I'll close it since there is no Kudu issue, but feel free to continue discussing here.


[GHSA-8xw3-8jcr-ch76] The acpi_ds_create_operands() function in drivers/acpi...

Updates

Affected products
Source code location
Summary

Comments
https://github.com/acpica/acpica/pull/295

same as #5082


SchedulerSuite.TestDeployController: timed out waiting for the deploy to complete

FAIL: test_scheduler.go:452: SchedulerSuite.TestDeployController

++ 18:55:09.970 got deployment event: scheduler starting
++ 18:55:09.970 got deployment event: scheduler starting
++ 18:55:09.970 got deployment event: scheduler starting
++ 18:55:11.560 got deployment event: scheduler up
++ 18:55:11.650 got deployment event: scheduler up
++ 18:55:11.650 got deployment event: scheduler up
++ 18:55:11.670 got deployment event: scheduler stopping
++ 18:55:11.670 got deployment event: scheduler stopping
++ 18:55:11.680 got deployment event: scheduler stopping
++ 18:55:12.370 got deployment event: scheduler down
++ 18:55:12.450 got deployment event: scheduler down
++ 18:55:12.600 got deployment event: scheduler down
++ 18:55:12.650 got deployment event: web starting
++ 18:55:14.310 got deployment event: web up
++ 18:55:14.320 got deployment event: web stopping
++ 18:55:14.443 reconnecting deployment event stream
++ 18:55:14.499 got deployment event: scheduler starting
++ 18:55:14.499 got deployment event: scheduler starting
++ 18:55:14.499 got deployment event: scheduler starting
++ 18:55:14.499 got deployment event: scheduler up
++ 18:55:14.499 got deployment event: scheduler up
++ 18:55:14.499 got deployment event: scheduler up
++ 18:55:14.499 got deployment event: scheduler stopping
++ 18:55:14.499 got deployment event: scheduler stopping
++ 18:55:14.499 got deployment event: scheduler stopping
++ 18:55:14.499 got deployment event: scheduler down
++ 18:55:14.499 got deployment event: scheduler down
++ 18:55:14.499 got deployment event: scheduler down
++ 18:55:14.499 got deployment event: web starting
++ 18:55:14.499 got deployment event: web up
++ 18:55:14.499 got deployment event: web stopping
test_scheduler.go:510:
    t.Fatal("timed out waiting for the deploy to complete")
... Error: timed out waiting for the deploy to complete

https://ci.flynn.io/builds/20150312184659-c9263595

Fixed in c54a751.


Multiple batteries support

This changes the battery widget to support multiple batteries. It will display a bar for each battery and a single summarised text description.

Overall percentage is just the mean, a useful improvement would be to weight it based on individual battery capacity but I get the impression that might not be something we can rely on for all batteries

Overall time remaining is the sum, if all batteries provide this information. Only one of my two batteries does so I'm not sure if this is the right behaviour generally speaking, but I just use percentage anyway. I would guess this is right for discharging, when one is discharged at a time, and wrong for charging when both are charged simultaneously, maybe someone with two good batteries can confirm.

Despite these shortcomings I find it useful for my two battery machine.
The first three commits add the new behaviour. I verified the new functions will work for a single battery so the fourth commit removes old functions. Fifth commit renames new functions to match old so that config will still work -- I can drop either or both of these if you want to preserve the old single-battery functions too but I think they're redundant.

Nice, thanks!


Hide UpdateTemplatePermissions button when the template is owned by a project.

ISSUE TYPE

Bug Report
Improvement Request

COMPONENT NAME

UI

CLOUDSTACK VERSION

master

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

If a template is owned by a project it cannot be shared. User gets the following error:
Update template permissions is an invalid operation on template project1tmpl. Project owned templates cannot be shared outside template.
It makes sense to hide the button.
STEPS TO REPRODUCE

EXPECTED RESULTS

Action should be hidden

ACTUAL RESULTS

Action is available

ping @anuragaw

This has been fixed in Primate


ignore flag doesn't work

What version of Turborepo are you using?
1.1.4
What package manager are you using / does the bug impact?
pnpm
What operating system are you using?
Mac
Describe the Bug
I try :
  turbo run build --ignore="**/*"
  # or
  turbo run build --ignore="*"
  # or
  turbo run build --ignore="**"

Expected Behavior
I expect turbo should not execute build command, do nothing.
But turbo still executed build
To Reproduce
...

Same issue with Yarn.

--ignore command is for gitginore paths with --since flag. you can use ! with --scope to ignore based on name

@jaredpalmer
turbo run build --scope=!ignoredPackage still builds the ignored package.
Based on what you said, are the docs not up to date?
https://turborepo.org/docs/reference/command-line-reference#--scope
https://turborepo.org/docs/reference/command-line-reference#--ignore
Thanks

For others having this issue, the ignored package was a dependency for me, so adding the --no-deps flag resolved this for me.


Add BeeFore build trigger.

I had a discussion with @IanLee1521 at PyCon over dinner about adopting to a common configuration system for Python's linting tools (pycodestyle, flake8, pylint, isort, ...). We two came to the conclusion that a single file used by all projects (e.g. setup.cfg as already supported by most of them) might be a good idea. I adopted my packages to follow that pattern. See https://github.com/MarkusH/django-dynamic-forms/blob/master/setup.cfg for an example.

Having a proliferation of config files in different formats doesn't help anyone; adopting a common format sounds like a great idea.

In addition, I think a common library to handle the parsing and locating of such files would be great too.
For instance, a common way across packages to handle discovery and merging of: system (/etc), user (~/), local (./) and command line configuration files and options would also be great. Specifically, which take precedence and where they should be located (and how to name them).

It seems this PR is no longer relevant @freakboy3742, should we close it?

Yes, this has now been implemented.


Post 2.3.2 upgrade, fails to import block 13280607

Summary
We upgraded two of our environments to algorand-indexer 2.3.2 and go-algorand 2.5.4. One has been running for over a day, and the other has been running for 6 hours. They both failed to import block 13280607 from our node with the following error.
{"level":"info","msg":"Starting block importer.","time":"2021-04-17T21:12:16Z"}
{"level":"info","msg":"will start from round >13280607","time":"2021-04-17T21:12:16Z"}
{"error":"update account asset, pq: null value in column \"frozen\" violates not-null constraint","level":"error","msg":"failed to commit round accounting","time":"2021-04-17T21:12:16Z"}

Our non-upgraded environments did not have a problem with this block. (algorand-indexer 2.2.1 and go-algorand 2.4.1)
Scope/Requirements
Looking for information on how to get the algorand-indexer to accept this block and progress. We followed the upgrade steps (mainnet to mainnet) but if we missed a step that solves this issue please let us know.
Urgency/Relative Priority
With the importance of go-algorand 2.5.4 protocol upgrade, we believe this is a medium-to-high level of priority.

Closing. Restoring the database that progressed through this block and going through the migration again allowed it to progress. I don't know what it was about this block but I'll submit an issue if it happens again.