Python sample code for Azure Functions/Even hubs throws exceptions

In the Python sample code for an event hub trigger:
import logging
import azure.functions as func

def main(event: func.EventHubEvent):
    logging.info('Function triggered to process a message: ', event.get_body())
    logging.info('  EnqueuedTimeUtc =', event.enqueued_time)
    logging.info('  SequenceNumber =', event.sequence_number)
    logging.info('  Offset =', event.offset)

    # Metadata
    for key in event.metadata:
        logging.info(f'Metadata: {key} = ', event.metadata[key])

Running this code causes the function to fail with an exception. It looks like the logging.info() lines are incorrect as they are passing parameters without formatting inside the msg.
See the docs on logging: https://docs.python.org/3/howto/logging.html#logging-variable-data

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

ID: a109c38e-b241-c9a2-7ef8-2e828659191c
Version Independent ID: 69842b0f-607f-5cf5-6e1a-4d312c98100c
Content: Azure Event Hubs trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-event-hubs-trigger.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@jimbobbennett Thank you for your feedback . We will investigate and update the thread.

Raised a PR in the azure-docs-pr repo with a fix, as requested in #71734

@craigshoemaker could you please review and confirm?

just signed off on PR 149511


Cedar: A Composable ML Accelerated Analog Circuit Simulator | G. Hertz, P. de Vos | JuliaCon 2021

Cedar: A Composable ML Accelerated Analog Circuit Simulator, https://www.youtube.com/watch?v=q8SzFTtgA60
This video was on my radar for the long time, I hope I made good timestamps for it.
00:00 Welcome!
00:20 Agenda
01:57 Technical computing is behind the times
02:27 We aim at matching technical computing with the times
02:46 Three simulation products from Julia Computing
03:20 JuliaHub, a supercomputer at your fingertips
03:55 A differential analog circuit simulator written in pure Julia
04:43 How to transform circuit model into equation?
06:52 Current state of parser and simulator
08:08 Using Symbolics.jl in designing of circuits
10:37 Differentiable circuit optimization
11:12 Our simulator is just a Julia library, so we can seamlessly use whole Julia ecosystem
12:43 ML is reshaping computing, including circuit design
13:00 Black box and white box surrogatizaton of models
13:32 Problem: surrogate models of circuits are stiff
13:56 Challenges of dealing with Continuous-Time Echo State Networks (CTSN)?
14:33 New approach to train oscillatory process (at the time of the talk, not published)
15:00 Analog design today is often the same as it was 1990s and have serious drawbacks
17:39 Julia Computing bring to analog circuits what digital community already do
19:43 Be we want to do it even better
22:48 Xyce.jl: wrapper for Xyce, a parallel circuit-simulator
23:38 Summary

Great, thank you!


Cleanup some invalidation code

To validate this pull request against other sbt modules, please visit this link.


useMutation infinite calls to fetch request when running test with jest.

The function used to call api is been called infinite times when running it inside jest

Can you provide an example?
Bear in mind that you can’t just call the mutate function during render as it will get called every render, which would cause state changes in react-query on every render. Make sure any calls to the mutation are only done once, like inside a useEffect


[Docs] Limit urllib3 version for docs building

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

@Xiangxu-0103 Please add motivation in the PR message.

@Xiangxu-0103 Please add motivation in the PR message.

Added


set the default of portalContainer to document.body

Not only because document.body is the default element used in react-dom's createPortal usage example, but because it's simpler and easier, anyone having anything against using document.body could change it to whatever they wanted using the now optional portalContainer prop.
Awesome library, by the way, works flawlessly! :)

Thanks for your PR


User interface issue, kind of...

Really happy with the code and its performance.  Have a user interface issue that I admit isn't really the codes problem, but hope I can receive some assistance.
The code I'm working on reads files into memory as a 32*32mb (1gb at a time) array.  I need to be able to hash for a member of that array (string(block[i][:]) works) and the array as a whole (unknown how).
Because of the quantity of data, all related code works via pointers and does not require me to ever copy any portion of that array.
I REALLY REALLY do not want to build a 1GB string from the array, both for memory usage and computation reasons.  Unfortunately string(block[:][:]) is not legal.  Any suggestions for how to reference block[32][32mb] as an argument to io.WriteString(hash, ??)?  Is there a different io function I should be calling?
Sincerely appreciate positive comments on this "out of the box" question.

You shouldn't use memory directly you should stream this.. using a Reader Writer combination and use io.Copy()

For inspiration you can look at https://github.com/s3git/bt2sum/

Again, my appologies for my beginner understanding of Golang, but given that hash, err := blake2b.New(nil) returns a Writer, what is wrong with using io.WriteString(hash, ).  The writer gets fed the same as if it was fed from an io.copy Reader does it not?  I could, of course, do an io.Copy(hash,strings.NewReader()) if there is a reason to do so, but would still suffer from a lack of method to convert [32][32gb]byte into the required string([1gb]byte) parameter.
My target application is designed for multigigabyte files, for which I already have 1GB chucks in memory, stored as [32][32GB] byte array.  Seems inefficient to have to read all that data a 2nd time, or to have to allocate another GB of memory to create a reformated copy (e.g. a [1gb]byte array)
I'm presuming the blake2b writer needs to receive all of its data at once, and everything I want to feed it is stored in the right order in memory in the [32][32mb] array.
The ancient C programmer in me would just grab the beginning address of the array and build the 3 word slice structure - but not aware of how to do that in GoLang that would satisify the compilers type-checking.

@goarchit can you paste a snippet we can help you simplify it ..

Just realized I likely have a bigger problem.  I'm not sure 2D arrays are stored without some additional data, since they are arrays of arrays, not true multidimensional arrays.  e.g. [x][y]byte, not [x,y]byte.  Golang really doesn't have multidimensional arrays from what I've read.
I had hopes with this snippet:
    log.Console("block[0]=",string(block[0][0]))
    p := unsafe.Pointer(&block[0][0])
    str := *(*string)(p)
    log.Console("str=",str)

The first log entries prints out what was expected (sanity check), but the second dies with a panic: runtime error: growslice: cap out of range error
I've come across some web posts about 2D arrays having excessive overhead as well, so perhaps I'll be better off just utilizing a 1d 1GB array and slice as needed.   That is well within my means.
Interested in any idea anyone might have, and wanted to thank you all for being kind for this out-of-the-box exchange.

Closing this issue as stale.


Get blame for the target branch of the pull request

Get blames for the target branch of the PR instead of assuming master

Also removed a number of trailing commas

If you fix the nits i'll pull it in, thanks!

if you add another argument, the blame looks something like

previousLine,
newLine

So you are now associated the blame for the previous line even though you did not change it. Also it clobbers the pull request because it highlights lines that did not change.
Finally, small thing but it's easier to move things around as you don't need to remove the comma from the last line and add it back to the other place.

We are also pushing trailing comma for function calls and arguments to be part of the javascript standard. It's currently in stage 2.

Thank you!


Deleting authors not working

Steps to reproduce:

Click "Deposit to this collection".
Select "Text" then  "Continue".
Populate an author using "https://sandbox.orcid.org/0000-0001-9303-6667".
Add another author using "Add another a author".
Populate additional author with "https://sandbox.orcid.org/0000-0003-0468-3673".
Click "Save draft"
Edit work.
Delete the second author.
Click "Save draft".

Expected result:
Second author is deleted.
Actual result:
Second author is not deleted.

Here's a slightly different scenario where deleting is also not working:

Create an item with two authors (https://sandbox.orcid.org/0000-0003-3437-349X with name text updated to Justin Littman and https://sandbox.orcid.org/0000-0002-5466-7797 with name text updated to Vivian Wong).
Deposit item
Edit to add a third author (https://sandbox.orcid.org/0000-0003-0962-2466 with name text updated to Naomi Dushay).
save as draft
edit and delete author #3. > author not deleted.
edit and delete author #2 > author not deleted.

Cannot be reproduced when affiliations are removed from the UI in #3287


Error to load controller after update to zend 2.6.0dev

Hello
We have custom CMS by zf2, I try update zend version to version 2.6.0dev, But after that I have A 404 error occurred

A 404 error occurred

Page not found.

Additional information:

The requested controller could not be mapped to an existing controller class.

Controller:
    system-index-index (resolves to invalid controller class or alias: index)

any important change happened to version 2.5.3 and 2.7.15?
can you help me to understand the problem?

Hey there,
It looks like something you'd have to debug (write an integration test) and then bisect: not something maintainers can help you with until you isolated the root cause.


chore: update referral api input payload

Review Checklist

Summary*

[ ] fix: Bug fix for Add the items tackled here as checklists
[x] feat: Completed task
[ ] chore: Incomplete task

[ ] test: Sub-task 1

Structure*

[ ] The Pull Request has a proper title that conforms to our MR title standards
[ ] The Pull Request has one commit, and if there are more than one, they should be squashed
[ ] The commit should have a proper title and a short description
[ ] The commit must be signed off
[ ] Unused imports are not present
[ ] Dead code is not present
[ ] Ensure dry running library to confirm changes

Tests

[ ] Proper and high quality unit, integration and acceptance(if applicable) tests have been written
[ ] The coverage threshold should not be lowered

Sign off*

[ ] All comments have been resolved by the reviewers
[ ] Approved by Czar {replace_with_czar_name}
[ ] Signed off by second reviewer {replace_with_name}
[ ] Ensure all checks are done before merging :warning:
[ ] All PRs needs to be signed before merging :warning:

N/B:

Add a checklist if more than one item is done.
Add screenshots and/or images where necessary
Indicate any breakages caused in the UI :exclamation:
Where necessary, indicate which issue the Pull Request solves (Closes #)
Any new files are updated in the folder structure in the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 81.742%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
7275

Relevant Lines:
8900

💛  - Coveralls


Windows 10 optimization with UWP

Describe the bug
UWP don't consume my laptop ram as task-manager reported, but always felt choppy and unresponsived. It's hardly animate anything on 60fps when on 3000x2000 screen (from 2000x1500 fine though)
Steps to reproduce the bug
Scale Windows 10 up to 3000x2000 or higher, then begin use any WinRT app like (old edge browser) I'm sure you will see the difference between 30fps and 60fps
Expected behavior
Animation choppy and lag, app sometime freeze
https://www.youtube.com/watch?v=u0Gx36d2VRQ&feature=youtu.be
Version Info

UWP (max: 18362, min: 17763)
Laptop (Surface Book 2016, Intel I5 63000U, GeForce 940M)
Windows 10 (10.0, Build 19013)
| Windows 10 version                  | Saw the problem? |
| :--------------------------------- | :-------------------- |
| Insider Build (xxxxx)              | Yes  |
| May 2019 Update (18362)            | Yes  |
| October 2018 Update (17763)        | Yes   |
| April 2018 Update (17134)          | Yes   |
| Fall Creators Update (16299)       | No Idea   |
| Creators Update (15063)            | No Idea   |

Surface book 2016 core i5, 940M
Additional context
If I resize my UWP app windows down very small (the point where UWP display only less than 2000 pixels) Windows 10 remain smooth and responsive. If switch to full screen mode, the app will end up like in the video,
Already put windows in high performance mode
Already update Nvidia Driver to the lastest
Already turn on High performance mode on every single UWP app in graphic setting

Is the app running on the Nvidia GPU or the integrated GPU? There's been a firmware issue that made Microsoft disable the Nvidia GPU. Could just be it can't keep up. Also are you using an external monitor and if so how's it connected?

Is the app running on the Nvidia GPU or the integrated GPU? There's been a firmware issue that made Microsoft disable the Nvidia GPU. Could just be it can't keep up. Also are you using an external monitor and if so how's it connected?

This happen to both OnBoard-Intel GPU and Nvidia 940M. And no, I connected to no external display

FYI my laptop is 3000×2000 (Matebook X Pro, Intel GPU) and I don't have this issue. It's probably the specific hardware configuration or install of Windows.

FYI my laptop is 3000×2000 (Matebook X Pro, Intel GPU) and I don't have this issue. It's probably the specific hardware configuration or install of Windows.
Try disable "Panel Self Refresh" in Intel settings as it can cause flickering on the intel gpu.

Downloaded Intel Graphics Command Center
Disable Panel Self Refresg + Maximum Performance
Still the same

Seriously you guys gonna ignore this ? all of the High Resolution laptop in my office are really suffer from Frames Drop and Freezing. All of them are WinRT app. Win32 don't have issue

Sorry, we've been on Thanksgiving holiday for the last few days. We'll need you to take some traces to diagnose further -- I've sent email to my colleagues and I'll get back to you.

Sorry, we've been on Thanksgiving holiday for the last few days. We'll need you to take some traces to diagnose further -- I've sent email to my colleagues and I'll get back to you.

If you want more evidence I will upload more of Surface devices to show you
I've been dig in and see what XAML element cause this and the Answer are

GridView
Acrylic

I thought I didn't applying it right but then I notice many big UWP app on the store all have the same issue

This is my app https://github.com/Strypper/OClubs
Awesome Tube
Microsoft Edge

@Strypper Can you capture a trace using this tool and share with us:
https://answers.microsoft.com/en-us/windows/forum/windows_10-update/windows-performance-recorder/a1648e8c-50c7-4243-9f1d-4216385c7ff3
The settings you'll want to turn on for the trace:
-Under Scenario Analysis:
Xaml App Responsiveness
Video Glitches
-Under Resource Analysis:
CPU activity
GPU activity
This should give us enough info to see exactly what is causing the animation to be glitchy.
Thanks!

Closing for now. Please reactivate if you have a trace for us to investigate.


Add property to IEditorOptions to disable tab trapping

Is it possible to add a property on IEditorOptions to permanently disable Tab Trapping? In our current design when the browser refreshes or a new window is opened the user would have to toggle the trapping off with Ctrl + M

You can do the following:
editor.createContextKey('editorTabMovesFocus', undefined).set(true);

Thank you for the response, editor.createContextKey('editorTabMovesFocus', true); worked for me!


Deep loader query objects are reserialized incorrectly

Hi @mdreizin!
We're having an issue over at stylus-relative-loader that I think I tracked down to webpack-config. We're on the latest, 6.1.2.
To summarize, this loader string goes in:
stylus-relative-loader?{"precacheImportVariables":[{"$tenant":"sams"},{"$tenant":"walmart"}]}

and after a WebpackConfig().merge(...), this comes out:
stylus-relative-loader?{"precacheImportVariables":[object Object]}

^ notice how it clobbered our array! 😱 The original nested object was a totally legit query.
I haven't dug into the webpack-config code yet, but I think it might be a bigger issue than just reserializing our object poorly: webpack says that queries can be any old arbitrary syntax and it's the loader's job to parse them – many just happen to use parseQuery from loader-utils which lets you put JSON5 there, but it's not a requirement. I think it's too presumptuous of webpack-config to think it can parse a stringified query. If the loader is already non-stringified (specified like { loader: "foo", query: { ... } }), then sure, but otherwise there's potential breakage.
I'll start digging into the code but would of course appreciate any help. :)
/cc @jmcriffey

@exogen Hello @exogen, I will take a look at this issue and try to fix it as soon as possible.

Nice timing @mdreizin! I just managed to put together a test case: #46.

Hello again @exogen @jmcriffey,
I have just fixed this issue and I would ask you to update to 6.1.3 version.
Could you please tell me does it work or not?

Preliminary testing shows the fix does indeed address our issue. I'm going to do a bit more testing to verify, but I think this does it. Thanks @mdreizin!

Confirmed that this is working in our full project now and I can close our workaround. Thank you @mdreizin!!


Customizable blog folder

Is there any reason why docs are outside the website folder but blog posts are inside? And there's no customization allowed either?
Is it because projects at FB already tend to have a docs folder, but not necessarily a blog? If that's the case, it would be great to have it customizable, for cases where the website is being set up as a first class citizen in its own repo.

At this time the blog folder is hard-coded. It would make sense to have it in the web-config like the docs does.
return siteConfig.customDocsPath ? siteConfig.customDocsPath : 'docs';
I believe this is as there are no markdown files inside the website folder. Everything inside the website folder is either a html template of a js React component.
This would be a small change to make as the default is sensible ATM, would just mean having to reference the variable in 3 to 4 places.
Time to implement is probably about an hour or two

@sudhirj @neilsutcliffe  Hi! We made a conscious decision to only have docs outside of the website folder. We did this to try to maintain a semblance of simplicity and approachability for Docusaurus. Generally all websites will have documentation. Much less will have a blog, and we didn't want to confuse the issue by appearing that a blog was required, etc.
We could possibly have a config for this, yes -- technically it wouldn't be that difficult -- but I would want to make sure with my team here that we want to stray away from that design principle.

We were so preoccupied with whether or not we could, we didn’t stop to think if we should.
~Jeff Goldblum (ish)

@neilsutcliffe Ha ha ha - nice quote 👍

Reminds me that I should watch the original Jurassic Park again.

Makes a ton of sense to do it this way when injecting Docusaurus into an existing project - so kudos for making that really well thought out and easy.
Docusaurus works great as a first class citizen as well, though, it would be a shame if the defaults look odd when initialized as a standalone project. Slash ought to be happy to be a strong independent saurus.

@sudhirj That's a fair point. Let me tag this as enhancement and look at seeing how we can make this happen so that is satisfies both our requirements. Thanks!

Yeah, we could probably do something similar like did with the docs folder where we allowed a customizable path.

bump - is there any update on this as it would be more ideal for us as different external parties work on docs and blogs. Then we can mount the two together for build.
Thanks

This issue is closed due to

Stale activity
It won't be addressed in current version but has been addressed in the next major version


Make downstreamSubject a weak property in SubjectSubscriber

This is how it is in Combine.
Fixes #186.

LGTM

  Generated by :no_entry_sign: Danger Swift against e73ab1202f0a7427ecf4594140e4e63da5d75580


Update secure-files.md

@deisterhold Thank you for your contribution, I am merging it now and it should be live shortly. Please accept our apologies for this initial delay while we work though the initial backlog of contributions.


Dynamic links not opening on older devices

Hello everybody!
I'm an iOS developer, and currently I needed to use Universal iOS links using Firebase's Dynamic Links platform, before upgrading to iOS 13, when we clicked on the link sent and generated from dynamic links, we were redirected to the application that we are developing, it worked perfectly.
After the update, when testing on iPhones 8 and 8 plus, we click on the link created and are redirected to the safari, and soon afterwards to the Appstore, but the correct behavior would be to be directed to our application. This behavior happens in newer devices like the Iphone X, and the Iphone 11. When using the Chrome mobile browser, we are directed to our application, as the chrome does not use Universal links, but the URL Schemes.
I found a topic on an apple developer forum addressing this problem but until February 16th it had not been resolved. https://forums.developer.apple.com/thread/123554

Hey @sagarcia31, this is a known issue and we're working on resolving it; see https://github.com/firebase/firebase-ios-sdk/issues/4914.

Thanks


Use PHP native json_validate in isJson function if available

PHP 8.3 introduces a json_validate function, which allows validating if a string contains valid json with less resource and memory consumption than using the json_encode function.
This PR uses the new function when it's available and thus is backwards compatible with older PHP versions, while taking advantage of the new function on PHP >= 8.3.
Note that Symfony has a polyfill for this, which is probably what will be used in practice by most existing Laravel installations using  PHP < 8.3 (as the polyfills are loaded with recent versions of symfony/http-foundation), which would allow removing the json_decode call sometime in the future.
See also the RFC.

@jnoordsij nice, thanks!


Mapping from a collection to another escapes the Collection API

(@ballPointPenguin raised this issue)
Here's a totally legitimate route handler:
this.get('/projects', (schema, request) => {
  let projectMembers = schema.projectMember.where({ userId: 12345 });
  let projects = projectMembers.map((member) => member.project);
  return projects;
});

but projects is no longer a Collection, so serializers break. How should we deal with this?

One option is
this.get('/projects', (schema, request) => {
  return schema.project.where(p => p.projectMember.userId == '12345');
});

which I think requires some changes to .where. (where no longer simply delegates to DB collection).

Another is to have Collection#map check if the returned array is an array of models of the same type, and to just make a new Collection at that point.

I need to see how Rails deals with this.

Collection#map check if the returned array is an array of models of the same type

I was thinking about same thing sounds reasonable to me

An issue with changing collection.map is that every array-like function that might return a homogeneous array of models would need the same treatment (e.g. reduce) and a user that decided to use build their own array with forEach and [].push(model) or using a lodash functions would be confused when it didn't work.

I need to see how Rails deals with this.

Rails ActiveRecord::CollectionProxy and ActiveRecord::Relation doesn't have any map method. Using map returns a plain array.
Because there are so many ways a user could create an array of models, it might be best to force users to keep in mind that they must return either a Model or Collection to get serialization behavior (the docs already mention this). This means for the original use case the user would need to wrap their response:
this.get('/projects', (schema, request) => {
  let projectMembers = schema.projectMember.where({ userId: 12345 });
  let projects = projectMembers.map((member) => member.project);
  return new Collection('project', projects);
});

I've felt the need for a general server.serialize method before, so that could be the other way of handling this case.
this.get('/projects', (schema, request) => {
  let projectMembers = schema.projectMember.where({ userId: 12345 });
  let projects = projectMembers.map((member) => member.project);
  return this.serialize('project', projects); // serialize could also take a Model or Collection without a type.
});

Another option would be to make the SerializerRegistry smart enough to detect the model type of a homogeneous array of models. I made an implementation of that here to see what that would be like.

The last solution seems best. The fact that I don't remember considering it makes me worry I forgot something, as it was the first thing that popped into my head as I was reading through your comment. When I'm not as tired I'll look back through some code and issues.
If that turns out to be a robust solution, we may not even need the notion of a Collection. Working with arrays is second-nature to JS devs, and people like to use different tools for that purpose (lodash, ramda etc). It would be great if they didn't have to even think about that decision affecting the work they're doing in route handlers.

I just realized that one issue with trying to guess the model type of the array is that it will be impossible for an empty array. If a user returns an empty array from a route then Mirage still needs to pass that to the correct serializer.
To make collections easier to work with, Mirage could use plain arrays and add some kind of property to to identify the model type. This could handle the empty collection case, but it still means that users would need to be aware that the arrays they return from their routes are special.
this.get('/projects', (schema, request) => {
  let projectMembers = schema.projectMember.where({ userId: 12345 });
  let projects = projectMembers.map((member) => member.project);

  // Wrapper needed in case array is empty.
  return withCollectionType(projects, 'project');
});

Using plain arrays and adding an explicit this.serialize method to be used at the end of routes would be my preference, but that does not handle the original issue posed here and is a pretty big API change.
At this point your where solution that accepts a function is looking pretty good. This matches lodash's _.filter function and should be pretty easy to implement. Basically where would just be _.filter that returns a Collection. What do you think of that? Would be happy to put together a PR.

Related to adding a serializer method: https://github.com/samselikoff/ember-cli-mirage/issues/367

#607 closes


Controlplaneoperator: Don't set status.kubeconfig before it exists

Currently, creating a cluster always results in a bunch of errors from
the hosted cluster reconciler because it got a 404 trying to fetch the
hosted controlplane kubeconfig. This happens because the hosted
controlplane operator sets status.kubeconfig before that secret actually
exists, whereas the hosted controlplane operator assumes that as soon as
that field is non-nil, it exists.

/test e2e-aws-pooled
/test e2e-aws

/test e2e-aws

/lgtm
/approve


[C++] Can't build arrow::util::Process with Emscripten

Describe the bug, including details regarding any error messages, version, and platform.
Boost.Process doesn't support Emscripten:
https://github.com/ursacomputing/crossbow/actions/runs/10821866436/job/30024731332#step:7:1985
FAILED: src/arrow/CMakeFiles/arrow_testing_objlib.dir/testing/process.cc.o
/usr/local/bin/sccache /root/emsdk/upstream/emscripten/em++ -DARROW_EXTRA_ERROR_CONTEXT -DARROW_TESTING_EXPORTING -DARROW_WITH_TIMING_TESTS -I/build/cpp/src -I/arrow/cpp/src -I/arrow/cpp/src/generated -isystem /arrow/cpp/thirdparty/flatbuffers/include -isystem /build/cpp/rapidjson_ep/src/rapidjson_ep-install/include -isystem /build/cpp/boost_ep-prefix/src/boost_ep -isystem /build/cpp/_deps/googletest-src/googletest/include -isystem /build/cpp/_deps/googletest-src/googletest -isystem /build/cpp/_deps/googletest-src/googlemock/include -isystem /build/cpp/_deps/googletest-src/googlemock -Qunused-arguments -fcolor-diagnostics -fPIC -fexceptions -Wno-error=deprecated-literal-operator  -Wall -Wextra -Wdocumentation -DARROW_WARN_DOCUMENTATION -Wno-shorten-64-to-32 -Wno-missing-braces -Wno-unused-parameter -Wno-constant-logical-operand -Wno-return-stack-address -Wdate-time -Wno-unknown-warning-option -Wno-pass-failed    -Werror -g2 -O1  -fPIC -std=c++17 -MD -MT src/arrow/CMakeFiles/arrow_testing_objlib.dir/testing/process.cc.o -MF src/arrow/CMakeFiles/arrow_testing_objlib.dir/testing/process.cc.o.d -o src/arrow/CMakeFiles/arrow_testing_objlib.dir/testing/process.cc.o -c /arrow/cpp/src/arrow/testing/process.cc
In file included from /arrow/cpp/src/arrow/testing/process.cc:27:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/io_context.hpp:25:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/wrapped_handler.hpp:18:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/bind_handler.hpp:20:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/handler_alloc_helpers.hpp:21:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/recycling_allocator.hpp:20:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/thread_context.hpp:20:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/call_stack.hpp:20:
/build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/tss_ptr.hpp:29:3: error: Only Windows and POSIX are supported!
   29 | # error Only Windows and POSIX are supported!
      |   ^
In file included from /arrow/cpp/src/arrow/testing/process.cc:27:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/io_context.hpp:44:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/scheduler.hpp:240:
In file included from /build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/impl/scheduler.ipp:25:
/build/cpp/boost_ep-prefix/src/boost_ep/boost/asio/detail/signal_blocker.hpp:27:3: error: Only Windows and POSIX are supported!
   27 | # error Only Windows and POSIX are supported!
      |   ^
2 errors generated.

Component(s)
C++

Issue resolved by pull request 44097
https://github.com/apache/arrow/pull/44097


Reorganization of requirements

Hi,
This is a small PR to reorganize the requirements and remove the GUI dependencies from the core.
It is mostly copied from AlphaTims. I think the removal of the strict requirements is very cool. This should also help to make some of the functions more reusable in other projects.
Let me know if I am missing something; otherwise, we could go this route.
@jalew188 After the PR, we could check if the compatibility works then as intended...

Side note. You call this a "small" PR, but I think it is exactly the right size. Anything bigger is quite a hassle to review in a single sitting and I much prefer working like this to be honest;)

Hi,
Thanks a lot for the review.
I made a couple of additional changes:

reduced the number of requirements and adapted the workflows. There are still a couple I want to reduce in the future.
the pythonnet is now part of install_requires - the notion is that Mac and Linux users will have to fiddle around for the full workflow (they need to install mono anyhow) but would help to install as a package.

As for the testing and pinned requirements: One of the main goals of the pinned requirements is to see when new packages will break the code. Therefore, I want to have them strict so that when having a dependabot-PR, the new versions are tested. To still have flexibility, I like the scheme that we can select how we want to install.
For testing, I only will test the pinned versions so that I can guarantee that the "stable"-version works. For now, I don't want to test the non-pinned version as in an ideal case if the dependabot system is up and running, the old versions should have been tested at some point. Moreover, running stable/not-stable tests on the integration tests would blow up the testing time quite a bit.
In an ideal world, we might be able to have a track on which versions work and which do not. I was searching now for a bit, but I haven't found anything yet, so happy to take suggestions.
@swillems : Agree on the size of the PRs.

Most looks fine, only major worry I have is that you do not use stable while you seem to want to enforce that...

Hi, thanks for the additional review. Cleared things up as I assumed alphapept[gui-stable] is using stable base requirements aswell.
As for the Pythonnet: It is required to run NET on Windows, Linux and macOS. For Linux and macOS we need Mono, and this caused problems for a lot of people that just want to install (e.g. and not use Thermo files).
So the way it is intended for now is to not install pythonnet on a macOS and Linux system per default -  and if you want Thermo support one needs to additionally install it. This is now reflected in the readme.


rtsp STREAM

Is posible to stream rtsp live stream?

Could you provide more detail and describe.

I have FOSCAM IP Camera and I want to stream video directly from camera.
For example: https://www.foscam.com/faqs/view.html?id=81
You can see rtsp link.

I think it's posible to stream rtsp live stream.

Thank youe for the reply.
Is this the rigth way to do that? Can I add url directly like this? Because, for me, it is not working.
<VlCPlayerView
source={{ uri: "rtsp://username:Password:@IP:port/videoMain" }}
/>

<VlCPlayerView
url={"rtsp://username:Password@IP:port/videoMain"}
/>

I allready tried that but not working.
Thank you for your help.
If you find solution, provide it.

No I can not, because it is posible to stream only in local network.
I have another question, how can I play youtube video from link?
I tried url="yt-link" and it is not working.

It's possible not support  url="yt-link".

xuyuanzhou, thank you very much.


How to specify IDs when using npy format?

Reading these docs it appears as though one can set the entry IDs when using parquet by setting –id_columns. How does one set entry IDs when using npy format?

If you use npy format you need to use sequential ids (and probably maintain
a metadata dataset in same order)
On Fri, Dec 16, 2022, 03:01 Samuel Ainsworth @.***>
wrote:

Reading these docs
https://criteo.github.io/autofaiss/getting_started/quantization.html it
appears as though one can set the entry IDs when using parquet by setting
–id_columns. How does one set entry IDs when using npy format?
—
Reply to this email directly, view it on GitHub
https://github.com/criteo/autofaiss/issues/146, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437QTW2SFPYFMJNVXPOLWNPEPHANCNFSM6AAAAAATAMJ3YM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

For future readers, don't forget also that the .npy files will be loaded in in alphabetical order.

And completely mess up your intended ordering.
Use str(my_file_num).zfill(3) to left-pad your file name indices with zeroes to avoid this behavior.

Normally the mapping between filename and index id, should be saved in a json. This is the responsible thing to do. Otherwise people may report erroneous scores if they do not load things properly. This shouldn't be left ambiguous.


Fall back to using OIDC Subject instead of Email

Email is not mandatory field, Subject is mandatory and expected to be unique. Might want to take a look at UserInfo first, however.
Issue: #56

This also needs a patch for the case of refresh ....

We will need to add a note to the changelog about this change

Not super sure about that. This will make this work on systems where it never worked before, but in a peculiar way. Once it is fully fixed correctly (as in email is requested separately from userinfo it is not present in the immediate reply) then the responses will change for these systems, but then also those systems will start working exactly the same as the systems that already work now.

We will need to add a note to the changelog about this change

Changelog added


Define Requirements: functionality outside of Scenario workflow

There are some workflows that currently exist outside of the repository pattern described in #539. These are exceptions to the repository pattern:

load_change_table in create.py to load change tables from outside of the scenario workflow.
export_case_mat can save case.mat files to the local filesystem for further investigation.

These functionalities will be preserved during the work done for this epic and the necessity of which can be reassessed at a later time if they need to be updated.


Feature Request: Have Summoned Token Use Your Default Token Settings

Right now if I have a character in pf2e summon a creature token, it’ll get summoned with settings such that it’s name and health never shown to anyone and it doesn’t get the dead condition on death. At the very least the first of these are settings the token spawns with that don’t match what I have set for new tokens I create myself when I drag a new actor into scenes and make it difficult for even the player controlling them to keep track of how they're doing.
It’d save people time and feel intuitive imo for summons from this to create tokens that follow your chosen ‘new token’ norms (but with allegiance set to the summoner's). But if that’s not how you want to handle things by default, could that at least be a setting toggle (“use token defaults” or something)?
Thanks for everything you’ve done either way!

Please note that if you enabled the module prior to changing the default token settings, you have to enable Debug mode and delete the already created actors.
Then refresh. You will see the new actors with the used default token settings.


Cannot load Editor integrated project

Where it can't load

Firefox: Staging + Private browsing
Firefox: Preview links

When viewing a integrated editor project, the webcomponent doesn't load.
Astropi mission zero does work.

I see this same problem in Firefox 125.0.1 on Linux. Visiting the blank python starter project in normal mode or private browsing mode fails to load the web-component.js.

We followed these Cloudflare instructions to fix this in Firefox 125.0.1 on Linux, in both normal and private browsing modes.


Doc update: order of content in section "Infrastructural Site Collection provisioning"

We've a cool script that does most of the work...but we only tell that at the end of the section...by then folks already followed the manual steps. Small thing, but will help improve the experience. Actually I'm wondering if even should explain the manual steps...as they're not complete.

I have the same feedback. Structure the doc in one way that leads to success. The scripts seem to be the way that's best aligned with the goal of getting a partner up and running quickly. The manual steps are interesting for troubleshooting if something goes wrong with the script (if they are indeed complete) but could be broken out into a separate doc.

Done. Thanks for the feedback.

Thx Paolo!


Fixed typo

I've fixed a typo in your README.md file : it was written "putput" instead of "output".
No need to fill out the automatic form that comes with PRs for this, afaik, since no code was affected.

@payne911 thanks for your contribution.


migrate DateDigitization to date_digitized in publication model

Migration
use the date originally digitized from the species descriptor file to fill the date_digitised field in the populations model

field name in csv -> database_date_digitization

population_date_digitization


SQLDelight plugin threw an uncaught NullPointerException

When looking at a source of the class by CTRL+mouse clicked.
Plugin Error: SQLDelight threw an uncaught NullPointerException. Disable Plugin
NullPointerException: Worker exited due to exception

Please submit the full stacktrace.

Android studio is not generating stack traces of the plugin. It only generates above exception.  How do i find stack trace of the plugin?

studio log file.
Worker exited due to exception
java.lang.NullPointerException
at com.squareup.sqldelight.intellij.lang.SqliteGotoDeclarationHandler.getGotoDeclarationTargets(SqliteGotoDeclarationHandler.kt:42)

Closing as we've replaced the gotodeclaration handler


Exceptions while trying to store secrets using environment variables option

Discussed in https://github.com/Kong/kong/discussions/12942

Originally posted by nboddu009 April 25, 2024
Hi Team,
Can someone help with below issue,
Issue : could not get value from external vault (no value found)
Description:
`Installed Kong enterprise version with database mode (kong-enterprise-edition_<IP_ADDRESS>_amd64.deb)
Trying use to storing secrets in environment variables(https://docs.konghq.com/gateway/3.6.x/kong-enterprise/secrets-management/backends/env/)
But It is giving me error like 'could not get value from external vault (no value found)'
Steps Produce the issue:
1. export SECRETS_PASSWORD=abc123
2. export KONG_VAULT_ENV_PREFIX=SECRETS_
3. Created Prefix using Admin API
curl -i -X PUT http://hostname:8001/vaults/my-env-vault
--data name=env
--data description="Store secrets in environment variables"
--data config.prefix="SECRETS_"
4. sudo kong reload
5. sudo kong vault get my-env-vault/secrets-password
Error:
User1:~$ export SECRETS_PASSWORD=abc123
User1:~$ export KONG_VAULT_ENV_PREFIX=SECRETS_
User1:~$ sudo kong reload
2024/04/25 16:10:38 [warn] ulimit is currently set to "1024". For better performance set it to at least "4096" using "ulimit -n"
2024/04/25 16:10:39 [warn] ulimit is currently set to "1024". For better performance set it to at least "4096" using "ulimit -n"
Kong reloaded
User1:~$ sudo kong vault get my-env-vault/secrets-password
2024/04/25 16:11:49 [warn] ulimit is currently set to "1024". For better performance set it to at least "4096" using "ulimit -n"
Error: could not get value from external vault (no value found)
  Run with --v (verbose) or --vv (debug) for more details
User1:~$ sudo kong vault get my-env-vault/password
2024/04/25 16:11:59 [warn] ulimit is currently set to "1024". For better performance set it to at least "4096" using "ulimit -n"
Error: could not get value from external vault (no value found)

  Run with --v (verbose) or --vv (debug) for more details`

It seems to be related to this: https://docs.konghq.com/gateway/changelog/#general

thank you for your reply @chronolaw,  do you have some time to connect via zoom

Sorry, I think that we can not do that (zoom meeting), you could add more details in this issue.

@chronolaw It is not an issue of ulimit.
@nboddu009 your last step is wrong, it should be kong vault get my-env-vault/password not my-env-vault/secrets-password.

@Water-Melon  I tried with kong vault get my-env-vault/password  as well and getting same error

Installed Kong enterprise version with database mode (kong-enterprise-edition_<IP_ADDRESS>_amd64.deb)

Hi @bungle , I verified the vaults on EE 3.6. and master branch according to the steps given by @nboddu009 (I changed secrets-password to password in the 5th step.), and everything is working fine. The password can be retrieved correctly. However the issue author is still unable to retrieve the value of password correctly. Could you take a look at this issue? Thanks.

@Water-Melon is there any chance to have a zoom call on this?

Could it be the sudo is not preserving your ENV?

By default, sudo is not passing environment variables to the process that it creates except for those that are explicitly configured.  The sudoers manual page has all the details.  You can use the -E switch to pass all environment variables.  Does that solve the problem by any chance?


Icons.adaptive uses a private type

Our public API shouldn't have any private types.

CC @guidezpl

#81578 fixes this


Custom Body in rest and http errors

In the process of migrating from Restify 4 to Restify 5, it seems that we are no longer able to override the body of the error.
You used to be able to do:
new restify.errors.BadRequestError({body: {whatever: 'you want'}})
After looking through the documentation and the source code, I can't see a way to accomplish this. Is this something that would fit in this project? I would be willing to open a PR if I knew it would be well received.
Thanks, Garret

Hi @gthicks , I've faced the same problem.
I've just realized an npm module in order to overcome to this.
It's called restify-errors-options.
https://github.com/simonepri/restify-errors-options
Your example simply becomes:
errorsOptions.add('whatever');
new restify.errors.BadRequestError({body: {whatever: 'you want'}});

@ltvolks @retrohacker  What do you think to add a section documenting this behaviour?
My suggestion is to add a reference to restify-errors-options while a better solution will be found.
restify-errors-options also allows users to provide defaults for custom values added.
Like that:
function defaultErrno(code) {
    return code.split('').filter(c => c === c.toUpperCase()).join('') + 'E';
}
errorsOptions.add('errno', defaultErrno);

const err = new errors.BadRequestError();
err.toJSON();
//=> {code: 'BadRequest', message: '', errno: 'BRE'}

err = new errors.NotFoundError();
err.toJSON();
//=> {code: 'NotFound', message: '', errno: 'NFE'}

err = new errors.InternalError({errno: 'IE.001231.231231'});
err.toJSON();
//=> {code: 'NotFound', message: '', errno: 'E.001231.231231'}

This allows users to create modularized options and publish them for others, like this one:
https://github.com/simonepri/restify-errors-options-errno

I can shed some context on this. In 4.x and previous versions of restify, each of the default formatters treated errors as a special entity. For example, in the JSON formatter, the code looked something like:
if (dataToSend instanceof Error) { ... } else { ... }

This was repeated across all the formatters. In 5.x we made the shift towards formatters not doing any type of magic based on inspection of payload. Instead, the JSON formatter now just does:
return JSON.stringify(dataToSend);

This means that to define the structure of the payload, one can define toJSON() for that object. What we wanted to achieve was separation of the construction of the object (in this case, the error) from how it would be serialized. In the case of any error or custom error created via restify-errors, that behavior is defined here:
https://github.com/restify/errors/blob/master/lib/baseClasses/HttpError.js#L131
With that in mind, the best way to do this today is to hook into the restifyError event that is emitted by the server:
server.on('restifyError', function(req, res, err, callback) {
    err.toJSON = function customToJSON() {
        return {
            name: err.name,
            mesg: err.message
        };
    };
    return callback();
});

server.on('restifyError', function(req, res, err, callback) {
    err.toJSON = function customToJSON() {
        return 'i just want a string';
    };
    return callback();
});

As I type this I noticed the documentation is actually incorrect - we'll have that fixed shortly!

One minor note - if you only want to change the values of the existing message, but not the structure, i.e., you are ok with the default toJSON() fields, you can simply pass your custom values into the constructor:
https://github.com/restify/errors#customizing-errors
Regarding the addition of arbitrary properties on errors, we facilitate that using the context option of the restify-error constructor object:
https://github.com/restify/errors#bunyan-support
You would just want to make sure you are using the context field as part of your custom toJSON() implementation.

Should have been shipped in #1439

Hey @simonepri,
Thank you for engaging so much on this repo! Want to be a maintainer? :smile:
If so, shoot me an email and we can chat<EMAIL_ADDRESS>

Glad to!
Thanks to all of you for the fantastic job with Restify!


(GH-1927) Fix powershell mkdir method

Fixes the powershell mkdir command to correctly handle paths with spaces in them. When passed to the command line, paths have to be quoted, the previous code did not handle this.
This uses double quotes instead of single quotes to allow string interpolation to happen when it is finally passed to PowerShell.
Closes #1927

CLA signed by all contributors.


Is it possible to get the rendered fixture temp dir path inside hooks or test modules?

It's not a bug rather a question. (if there is a better forum for this kind of questions pls suggest so that I won't open any issues for them later on)
Before my service started up by docker compose, I have to set up a directory structure which will be mounted into the container. I would like to create these directories in the temporary directory of the fixture, before the container starts.
I found that hook_setup_data.py can be a good place for this. But how would that hookl know the temp directory path?

When a fixture is started, the current working directory becore its tmp directory. You can access through os.gcwd.
Usually I setup the directory content directly in the fixture directory and set the volume on the containers. That's what is done in this repository for exemple.
The other one is to use the hook_setup_data.py or the hook_started.py. It's the best way to populate a database : https://fixtup.readthedocs.io/en/latest/handbook.html#populate-a-database-with-data-before-running-a-test
You can use discord to ask question.


Unable to open the application, .NET error

The application cannot be started, it shows a popup requesting me to install .NET. However, when I select "yes", nothing happens:

Also, I have all the possible .NET installations on my computer, hence I do not understand what more is required?

Thanks for your help.

I wonder if it detected the developer packages on your system and skipped the installation. Can you try installing the .net core 5.0.2 desktop x86 runtime from here: https://dotnet.microsoft.com/download/dotnet/thank-you/runtime-desktop-5.0.2-windows-x86-installer

I wonder if it detected the developer packages on your system and skipped the installation. Can you try installing the .net core 5.0.2 desktop x86 runtime from here: https://dotnet.microsoft.com/download/dotnet/thank-you/runtime-desktop-5.0.2-windows-x86-installer

Hi, thanks for your reply. I have uninstalled all the .NET packages that you see in my screenshot above, restarted computer, installed the x86 version of .NET 5.0.2 as suggested, installed Outlook on the Desktop, but the same issue persists. The same message and same no action after clicking on "yes" to install .NET. Any ideas?

Hi, thanks for your reply. I have uninstalled all the .NET packages that you see in my screenshot above, restarted computer, installed the x86 version of .NET 5.0.2 as suggested, installed Outlook on the Desktop, but the same issue persists. The same message and same no action after clicking on "yes" to install .NET. Any ideas?

If you've uninstalled all of them, you'll likely also need the x64 runtime as well: https://dotnet.microsoft.com/download/dotnet/thank-you/runtime-desktop-5.0.2-windows-x64-installer

If you've uninstalled all of them, you'll likely also need the x64 runtime as well: https://dotnet.microsoft.com/download/dotnet/thank-you/runtime-desktop-5.0.2-windows-x64-installer

Brilliant, it works now. Thanks a lot!

Brilliant, it works now. Thanks a lot!


Update docs for building on OSX/Linux

The old instructions were incorrect
I also expanded on the dependencies on OSX

@hughbe,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Thanks!


codehost: forward the error from the underlying git call

The loadRefs() method is called in several places in this file. The 3 line pattern is used in all other calls to loadRefs(), but for some reason this one has been missed, and not having that underlying error has made finding the true error much more difficult, because I have encountered several different reasons already.

@googlebot I signed it!

@googlebot I fixed it.


react-native-svg compatibility

A have this error on Android device:
Versions:

"react-native-svg": "^13.9.0"
"react-native-qrcode-svg": "^6.2.0"

The problem was in react-native-svg version. It works for version 13.4.0


On a dumb terminal that doesn't support reusing the line, it
only prints the status before starting the command, but prints
the output of the command when the command finishes, by which
point other commands may have started and printed their status
line.  This makes it impossible to determine what command
produced a line of output.
Modify BuildEdgeStarted to only print the status line if the
command is going to lock the console, or if ninja is running
on a smart terminal.  Modify BuildEdgeFinished to always
print the status line unless the command locked the console,
in which case the status was already printed and no other
command can have printed any lines.
  One disadvantage is
that dumb terminals won't show anything when starting a
command, making it harder to tell what commands are currently
running, but I expect most interactive uses of ninja will use
a smart terminal.

I'm tentatively merging this as discussed on the "[rfc] changing ninja's progress output order when writing to a file?" thread.

I think this might have regressed output on the Chrome bots. Dirk noticed un-nice output on https://build.chromium.org/p/chromium.linux/builders/Linux Builder (dbg)(32)/builds/52253/steps/compile/logs/stdio
where the failure happens and then a whole bunch of "completed" lines push the actual failure way back in the log. (Note that they're running -j50.)
Maybe because the jobs are finishing at that point? Seems like we might need a special case for failure if we want to keep this ordering of output.

(Should have been @dpranke above.)

Before this patch output from commands that were running concurrently with failed command would appear after the output of the failed command, with no status line to separate the outputs of the different commands.  Chromium probably never ran into this issue since your rules generally have no output.  After this patch every command output is preceded by its status line, but that means that failures are followed by status lines for all concurrent commands.
Hiding the status lines for concurrent commands that have no output doesn't seem right, since those rules ran and completed.  We could delay printing the error until all concurrent commands have finished, but then if you had an error concurrent with a long command you might get the error much later than you otherwise could have seen it.
On Android our commands are so noisy that identifying the build output from a failing command is impossible without this patch.

I think delaying the error is probably okay; I don't know how often you really need to see the failures in real time or the delay would be long enough to matter. I agree that the change is generally an improvement and I would like to avoid hiding the output from completed edges as well.

I think it'd be OK to delay the error, but only in dumb-mode; that would be annoying locally in smart-terminal mode where there's often a long unrelated link finishing.

sure, only in not-a-tty-mode is fine by me.

I just ran into this when 'ninja -v' did not print started jobs anymore (which run for many minutes, so there is basically no output at all). This is pretty annoying -- are there any plans to offer to revert to the previous behavior as an option?

FWIW I find this annoying in -v mode too, so I agree it'd be nice to do something there.


API allowing CodeActionProviders to reliably get the current editor selection

Fixes #49024
Fixes #49060
See #49024 for details on the problem. Adds an optional selection property to CodeActionContext. This property has the original editor selection

@jrieken Pushed an update that goes to the range | Selection approach instead. This required tweaking some other logic and tests so please let me know if you have any concerns about the changes


Improve cjson to support empty array

https://github.com/mpx/lua-cjson/issues/11

Hello, when would this be released? I'm using cjson and this feature would be awesome

Maybe after a month, you can start with the master version


Gh 26 - includes Francois changes in his fork

Description
Updated the osgi-fragment to osgi bundle and included all the dependencies, exports etc to get it to work as a bundle in AEM

Related Issue
https://github.com/adobe/aio-lib-java/issues/26

Motivation and Context
OSGI Fragment was not working correctly. So switched to this library being a proper osgi bundle instead.

How Has This Been Tested?
Tested on local AEM using the following command from within the /osgi folder:
mvn -Dlicense.header.path=../copyright_header.txt -P localInstall clean install sling:install

Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

replaced by https://github.com/adobe/aio-lib-java/pull/29


Random crash on Godot4.3 when picking up items

Hey! When running multiplayer on Godot4.3 , I sometimes get a random crash of my server when picking up a DroppedItem3D. It doesn't always happen. Usually it happens the first time I pick up a droppedItem3D when the hotbar is focused on the item I just picked up. The add-on works fine if I pick up the item with another hotbar cell being focused.
I haven't built the GDExtension from source yet but was going to take a look when I have time to see if I can pinpoint the exact line crashing through a debugger.
One thing to note is since I upgraded to 4.3, I get these messages when I start running the game. I think it's probably unrelated to the crash though:

I'm already aware of this issue, I was working on a new feature and received this, I'm currently working with high priority to resolve it. Still investigating something related to duplication of slots when adding items to inventory.


Deleted package detected

I'm a Cyber Security researcher and developer of PackjGuard [1] to address open-source software supply chain attacks.
Issue
During my research, I detected a deleted package in this repository.
Details
Specifically, the package vidiopy mentioned in file README at line 10 does not exist on the public PyPI registry. A bad actor can hijack this package to propagate malicious code.
Impact
Not only your apps/services using https://github.com/SohamTilekar/vidiopy repo code are vulnerable to this attack, but the users of your open-source Github repo could also fall victim.
You could read more about such attacks here<EMAIL_ADDRESS>Remediation
Please highlight this in file README and register a placeholder package for vidiopy on public PyPI soon to remediate.
To automatically fix such issues in future, please install PackjGuard Github app [1].
Thanks!

PackjGuard is a Github app that monitors your repos 24x7, detects vulnerable/malicious/risky open-source dependencies, and creates pull requests for auto remediation: https://github.com/marketplace/packjguard

Thanks, But I Already have Secure the vidiopy project name on pypi. There fore it Could not be used by another.
I Already Published Package on the pypi. then I deleted it due to some Problem.
I am going to Publish it again.

Deleted package names become available for use. An attacker can exploit this vulnerability and register a malicious package.

I Have a project name assigned on my account.

I only deleted the released version.

Oh ok. This makes sense. Thanks for providing the details! I will close this issue.


Windows development

I'm using a windows development environment and ran into the following issues

some the filenames are not valid in windows land
some of the scripts in package.json did not run

Hey @richardschneider - Thanks for the contribution :+1: It looks like your two PR's have gotten a bit mixed up, they both contain the same combined changeset. Could you separate them out?

Sorry about that.  I'm new to pull requests.  My mistake was making the 1st PR from master and not an issues branch.  This then pulled in 2nd PR when merged into master.
Not sure how to fix this?

I've added a few inline comments - if we can resolve them I'm happy to merge this as-is. Talking you through separating the two pieces of work would only prove frustrating for the both of us ;)
I'd recommend trying something like this to work through git branching: http://pcottle.github.io/learnGitBranching/
or maybe this (the nodejs workshopper modules are usually fantastic)
https://www.npmjs.com/package/git-it

Closing in favour of #41.


Improve Foundry documentation with explicit command parameters

What changed? Why?

Replaced placeholder "..." in Foundry commands with explicit parameters (<CONTRACT_NAME>, <CONTRACT_ADDRESS>)
Makes documentation more clear and prevents confusion about required command parameters
Helps developers understand exactly what values they need to provide

Notes to reviewers

No functional changes
Only documentation clarity improvements
All URLs, chain IDs and other technical details remain unchanged
Follows standard documentation practices for command-line examples

How has it been tested?

Verified all links still work
Confirmed command syntax matches official Foundry documentation
Checked that all original information is preserved

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

@cb-heimdall

Review Error for ericbrown99 @ 2024-12-02 14:57:08 UTC
User failed mfa authentication, see go/mfa-help

@cb-heimdall Hi! Is there anything I can help with regarding the MFA authentication? Let me know if you need any assistance with the review process.

@ericbrown99
Hi! I see that my pull request was approved, but the migration wasn't completed. How can I assist to make it happen, or what should I do on my end?

@brendan-defi  Hi! I see my PR was approved, but the migration hasn't been completed yet. What can I do to help move this forward? Since I noticed you're active in this branch, perhaps you could help us with this issue?

Hi @donatik27 thanks for your patience. I've approved and merged.


Create a embed code generator from provided username

https://surbhioberoi.github.io/github-widget/ should have a box to enter username and display the widget and the code that can be used to embed it on own websites.

I created a PR #17

@surbhioberoi are you accepting pull requests? I would like to work on this enhancement.


Update tf version check for 0.8.5

What
We've upgraded Terraform in the pipelines, so we should ensure we're
using the same version when running the local tasks for consistency.
How to review
Try to run one of the local terraform tasks with an older terraform version and verify it errors. eg:
ACTION=plan make dev pingdom
Who can review
Anyone but myself.

works, merging


Pydantic V2 JSON Schema lacks type attribute for Literals

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
Pydantic V1 inferred the types a Literal is compatible of and made them available in the JSON Schema. This is no longer the case with Pydantic V2.
Example Code
import json
from typing import Literal

from pydantic import BaseModel

class Health(BaseModel):
    status: Literal["ok", "error"]

# With Pydantic V2
print(json.dumps(Health.model_json_schema(), indent=2))
#{
#  "properties": {
#    "status": {
#      "enum": [
#        "ok",
#        "error"
#      ],
#      "title": "Status"
#    } <--- Notice properties.status has no type key
#  },
#  "required": [
#    "status"
#  ],
#  "title": "Health",
#  "type": "object"
#}

# When using V1
print(Health.schema_json(indent=2))
#{
#  "title": "Health",
#  "type": "object",
#  "properties": {
#    "status": {
#      "title": "Status",
#      "enum": [
#        "ok",
#        "error"
#      ],
#      "type": "string" <--- Notice properties.status.type is set to "string"
#    }
#  },
#  "required": [
#    "status"
#  ]
#}

Python, Pydantic & OS Version
pydantic version: 2.0b3
        pydantic-core version: 0.39.0 release build profile
                 install path: /home/***/.local/share/virtualenvs/***/lib/python3.10/site-packages/pydantic
               python version: 3.10.11 (main, Apr 20 2023, 13:40:31) [GCC 13.0.1 20230401 (Red Hat 13.0.1-0)]
                     platform: Linux-6.3.8-200.fc38.x86_64-x86_64-with-glibc2.37
     optional deps. installed: ['email-validator', 'typing-extensions']

The type key for enum is not required (see the examples at https://json-schema.org/). Is this causing any actual issues for you?

My primary use of Pydantic is through FastAPI. With the V1 behavior the OpenAPI schema was rendered as expected if I was to use Health as a path operation response model. A little more info in https://github.com/tiangolo/fastapi/discussions/9709#discussioncomment-6247094

Gotcha. Shouldn’t be hard to add, but I am also curious if the new swagger update fixes this. Of course for something like Literal[“foo”, 124] we can’t add a type.

Fixed in #6243


Bump lhapdf, yoda, rivet

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Hmm can the last commit be split in to two?

@grahamc Sure. I only bump those together because these are released together.

@GrahamcOfBorg build lhapdf yoda rivet

@GrahamcOfBorg eval


Always asking for transaction fee even I have resources.

always have fee every transaction even I have resources. I feel like it's stuck.

You sure? Try proceeding, and then look at the Fee Breakdown. Maybe it's RAM you're short on or something else?

If you want to share the account name in question, we can check exactly what's going on in our services. Feel free to email it to<EMAIL_ADDRESS>if you don't want to share publiclyh

it is my actual username here cryptoshark1. I was buying NFT last night then I just notice the gas fee. It usually pushes thru with my current resources. then even I'm adding/staking resources it is asking for a gas fee like on my first screenshot.

Ah, currently our system detects if you have less than a few milliseconds of CPU and then triggers, and since you are right on the border of "not having enough CPU" to actually perform the transaction, it's popping up.
You currently only have ~5ms of CPU total based on what you've staked. That's enough to do like 10 small transactions per day which you're running up against. If you don't want the above to pop up for transactions, I'd recommend staking even more so you have a surplus of CPU available.
We'll look into seeing if we can tone it down a bit when accounts are borderline out of resources as well on our end 👍

ohhh that's why. maybe you guys could also include the option to proceed for free on the anchor itself like the 2nd screenshot when I use bloks. I was just thinking if it is stuck or what because I can do some free transactions just fine using my resources if I choose to click that small link. anyways I really plan to stake some more to prevent this,  thanks @aaroncox !! your customer service is 200%!

We'll look into seeing if we can tone it down a bit when accounts are borderline out of resources as well on our end 👍
I'll gladly wait for this.


Add Page For Player to input player name

#16

Nice one, The text written in the input field is not seen.
Could you please implement in the computer mode too..
And increase the width of the input field.

@LyKhai2708  Any update on this PR?


fix: ttl not working

Hi, I upgraded to<EMAIL_ADDRESS>and TTL was not working.
Perhaps it was an omission in #193 updates.

In<EMAIL_ADDRESS>maxAge was renamed to ttl.
The set() function takes options objects instead of ttl integer.

See: https://github.com/isaacs/node-lru-cache/blob/main/CHANGELOG.md#specific-api-changes
Versions
cache-manager: 4.0.0
lru-cache: 7.10.1
node: 18.2.0
npm: 8.9.0

The some tests are failing, so I will fix them.

Test passed😀

Thanks.  I'll get this merged ASAP.

this is available in version 4.0.1

Thanks. I discover that cache get data was undefined a few days ago and i don't know why. I must fix this problem because app should be deploy today and i found ttl param was lost in lru which reason is it accept object options but not number.
Thanks your merge request!!!


Render outputs as soon as wlc is ready

This makes sure that the outputs are rendered when sway is launched, so
the user doesn't have to move the cursor before the background and bar
gets rendered on screen.
Fixes #509

Nice!


echo Is not an internal or external command, nor a running program or batch file.

When I try to run the start command it errors out right off the bat with
[nodemon] 1.9.2
[nodemon] to restart at any time, enter rs
[nodemon] watching: index.html
[nodemon] starting 'echo cp index.html build/index.html'
echo Is not an internal or external command, nor a running program or batch file.(the chinese mean)
[nodemon] app crashed - waiting for file changes before starting...

I think @Zipbug ran into this issue in #14 as well. I don't really know what the cause is.

I did indeed, I had to manually install all the dependencies for some reason and then it started working.

It seems to be the only way.


Add transaction index and timestamps to events

This helps with sequencing on the client side when there are multiple events fired in the same block.
The change also replaces the original block parsing logic that used filtered transactions from the Fabric library, with manually parsing the entire block using raw proto definitions. This allows us to get richer information out of the block.
Also added the timestamp (unix nano) to events:

for subscriptions listening on blocks (no chaincodeId specified), timestamp is added from the timestamp parsed out of the block
for chaincode events, since the event itself pushed by Fabric doesn't contain the timestamp, we must make a call to Fabric using the event's block number. This call is optional depending on the timestamps property in the stream specification

Codecov Report

Merging #67 (d68e711) into main (00f16ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   53.84%   53.84%
=======================================
  Files          29       29
  Lines        2598     2598
=======================================
  Hits         1399     1399
  Misses       1129     1129
  Partials       70       70

Impacted Files
Coverage Δ

internal/events/api/event.go
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 00f16ec...d68e711. Read the comment docs.

This is a critical piece to merging the onchain-logic code to main, so have also done a review to merge.


Receiver window minimized state

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When the user minimizes the receiver window, the window appears to be (just barely) visible at the bottom of the page

Expected Behavior
After minimizing the receiver window it should not be visible
Steps To Reproduce
minimize the receiver window in a conversation on react.relay.cc

If this ticket is going to take more than a couple hours, don't do it. We're going to do a big update to the Window component internals soon anyways.

https://github.com/relaycc/receiver/pull/57


PROFILE: interrupts/evictions/bytes = 0/0/64

Hi,
I'm attempting to use gperftools' CPU profiler on a couple binaries. I was getting seemingly no profiling data, and after reading some posts here mentioning here that it may be due to the binary executing too quickly for any profiling to actually be done (e.g. pprof --text binary prof produces nothing), I added a call to sleep(10) in my target binary. This didn't help either.
I'm asking here to inquire if there might be any other issues going on. I also cannot profile unless I use the LD_PRELOAD method described (adding -lprofiler to the compiler flags doesn't do anything, it seems).
Any ideas?

The sampling time of the tool is 10ms, and the result can be correctly analyzed only when the processing time of the program is more than 10ms.
I
example:
$ time ./main
PROFILE: interrupts/evictions/bytes = 0/0/64
real    0m0.006s
user    0m0.001s
sys     0m0.005s
time ./main
PROFILE: interrupts/evictions/bytes = 0/0/64
real    0m0.006s
user    0m0.001s
sys     0m0.005s
pprof --text main pprof.log
Using local file main.
Using local file pprof.log.
Removing killpg from all stack traces.
Total: 4 samples
2  50.0%  50.0%        3  75.0% std::__unguarded_partition
1  25.0%  75.0%        1  25.0% __gnu_cxx::__normal_iterator::base
1  25.0% 100.0%        1  25.0% __gnu_cxx::__normal_iterator::operator*
0   0.0% 100.0%        4 100.0% __libc_start_main
0   0.0% 100.0%        4 100.0% main
0   0.0% 100.0%        1  25.0% std::__final_insertion_sort
0   0.0% 100.0%        3  75.0% std::__introsort_loop
0   0.0% 100.0%        1  25.0% std::__unguarded_insertion_sort
0   0.0% 100.0%        4 100.0% std::sort

My program runs for about 2 minutes, still get PROFILE: interrupts/evictions/bytes = 0/0/64...
But after add strace to monitor syscall, weirdly it begin to sample PROFILE: interrupts/evictions/bytes = 150/3/9240
Have no idea why...

My program runs for about 2 minutes, still get PROFILE: interrupts/evictions/bytes = 0/0/64...
But after add strace to monitor syscall, weirdly it begin to sample PROFILE: interrupts/evictions/bytes = 150/3/9240
Have no idea why...

Closing then.


Speed up VEP by chunking VCF's

Description of feature
We could attempt to speed up the annotation by parallellizing the analysis.
cfr point 12 here https://www.ensembl.org/info/docs/tools/vep/script/vep_other.html

Fixed in #35


Make area selection block example

This PR exports addAreaMouseSelection() allowing it to be consumed as a lib function.
It also converts the Area Mouse Selection example to a block compatible version.

Thanks for the PR!  Looks good!


Language-specific guides request

Problem description

Problem location

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

@dandycheng110 Could you specify the language in which you'd like to see a new guide?

Ok, maybe Taiwanese can help me
Usha Mandya<EMAIL_ADDRESS>2021年2月9日 週二，下午11:27寫道：

@dandycheng110 https://github.com/dandycheng110 Could you specify the
language in which you'd like to see a new guide?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/issues/12287#issuecomment-776022622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQLZFSF72M3NL5QY42ZVB7DS6FH6RANCNFSM4XLCWWOA
.

@dandycheng110 Thanks for the reply. I meant the programming languages that we haven't covered in https://docs.docker.com/language/ . Unfortunately, translating our docs into other languages are out of scope. Therefore, I'll have to close this issue. Feel free to continue the conversation after I close. Thanks.


Less than sign in program details page is no longer announced for screendreaders

Tested
Manually used screen reader and confirmed that the less-than sign is no longer announced when focusing the link back to Programs & Services.
Release notes:
Less than sign in link back to Programs & Services is no longer announced for screenreaders.
Checklist

[X] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes #3589

Codecov Report

Merging #3594 (c2ce934) into main (5218331) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3594   +/-   ##
=========================================
  Coverage     80.47%   80.47%
  Complexity     2420     2420
=========================================
  Files           294      294
  Lines          9596     9596
  Branches        657      657
=========================================
  Hits           7722     7722
  Misses         1617     1617
  Partials        257      257

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Slice Add Malloc

Testing

****************************************************************

libgrpc.so

     VM SIZE                                                                FILE SIZE
 ++++++++++++++ GROWING                                                  ++++++++++++++
  +0.0%    +104 [None]                                                   +4.47Ki  +0.1%
  +3.6%    +160 src/core/lib/slice/slice_buffer.cc                          +160  +3.6%
      [NEW]    +146 grpc_slice_buffer_malloc_indexed                            +146  [NEW]
      +9.1%     +14 [Unmapped]                                                   +14  +9.1%

 -------------- SHRINKING                                                --------------
  -0.4%     -32 src/core/ext/transport/chttp2/transport/hpack_encoder.cc     -32  -0.4%
      [DEL]    -109 begin_frame                                                 -109  [DEL]

  +0.0%    +232 TOTAL                                                    +4.59Ki  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE              FILE SIZE
 ++++++++++++++ GROWIN ++++++++++++++

 -------------- SHRINK --------------
  [ = ]       0 [None]     -16  -0.0%

  [ = ]       0 TOTAL      -16  -0.0%

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE                                                                FILE SIZE
 ++++++++++++++ GROWING                                                  ++++++++++++++
  +0.0%    +168 [None]                                                      +568  +0.0%
      +0.0%    +136 [Unmapped]                                                  +568  +0.0%
      +2.9%     +32 [None]                                                         0  [ = ]
  +3.6%    +160 src/core/lib/slice/slice_buffer.cc                          +160  +3.6%
      [NEW]    +146 grpc_slice_buffer_malloc_indexed                            +146  [NEW]
      +9.3%     +14 [Unmapped]                                                   +14  +9.3%

 -------------- SHRINKING                                                --------------
  -0.4%     -32 src/core/ext/transport/chttp2/transport/hpack_encoder.cc     -32  -0.4%
      [DEL]    -109 begin_frame                                                 -109  [DEL]

  +0.0%    +296 TOTAL                                                       +696  +0.0%

****************************************************************

libgrpc++.so

     VM SIZE              FILE SIZE
 ++++++++++++++ GROWIN ++++++++++++++

 -------------- SHRINK --------------
  [ = ]       0 [None]      -8  -0.0%

  [ = ]       0 TOTAL       -8  -0.0%

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

No impact


Attribute observers result in endless output, even when script exited

Remove the comment on this line in small_demo.py to re-enable the mode.
So the code has
def mode_callback(attribute):
    print "Mode changed: ", v.mode

v.add_attribute_observer('mode', mode_callback)

When executed, the code runs. After the thread exits the mavproxy command prompt prints out the mode forever as shown below. This is a bug of some sort right?
AUTO>
AUTO> module load droneapi.module.api
AUTO> api start small_demo.py
AUTO> Mode: VehicleMode:AUTO
Location: Location:lat=-35.3632603,lon=149.1652287,alt=0.0,is_relative=False
Attitude: Attitude:pitch=0.00512621784583,yaw=-0.0500796176493,roll=0.00491848401725
Velocity: [-0.08, 0.08, 0.0]
GPS: GPSInfo:fix=3,num_sat=10
Armed: False
groundspeed: 0.0
airspeed: 0.0
Requesting 95 waypoints t=Wed Apr 29 14:56:15 2015 now=Wed Apr 29 14:56:15 2015
Home WP: MISSION_ITEM {target_system : 255, target_component : 0, seq : 0, frame : 0, command : 16, current : 0, autocontinue : 1, param1 : 0.0, param2 : 0.0, param3 : 0.0, param4 : 0.0, x : -35.3632621765, y :<PHONE_NUMBER>27, z :<PHONE_NUMBER>55}
Current dest: 1
Disarming...
Arming...
Overriding a RC channel
Current overrides are: {'1': 900, '4': 1000}
RC readback: {'1': 1500, '3': 1000, '2': 1500, '5': 1800, '4': 1500, '7': 1000, '6': 1000, '8': Got MAVLink msg: COMMAND_ACK {command : 400, result : 0}
1800}
Cancelling override
APM: Arm: Mode not armable
APIThread-0 exiting...
Got MAVLink msg: COMMAND_ACK {command : 400, result : 4}
Got MAVLink msg: COMMAND_ACK {command : 11, result : 0}
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
Mode changed X:  VehicleMode:AUTO
...

I need to confirm if this is a MAVProxy problem or DroneKit

Fixed by #255.


alt-L diagonostic doesn't work in a large project

In a small project, diagonostic  works well. But in a large one it doesn't ,though "GetType with hover" and "code completion" and "GoToDefinition with ctrl+click " all works OK.
vs code 1.7.2, you-complete-me 1.0.24

not an issue but a conf of ycmd server


fix: FunctionsClient constructor default options

What kind of change does this PR introduce?
fix FunctionsClient constructor
What is the current behavior?
FunctionsClient constructor requires to pass {} as options if you don't want to pass headers or fetch
What is the new behavior?
Add options default value {} in FunctionsClient constructor

:+1: headers = {} should work too, e.g. https://github.com/supabase/postgrest-js/blob/e612a5d2790a66d04a829c12cf6ceed1b5c293c6/src/PostgrestClient.ts#L21-L33


Male vs Female

When I get a list of my voices I see "English UK Male" and "English UK Female" ... how do I get Artyom to use the English UK Female voice?

Just change the order of voice, artyom takes the first voice available for a language, so the order to select the female first when available would be:
const myAssistant = new Artyom();

// Change voice order:
// original object property prefers male voice:
// "en-GB": ["Google UK English Male", "Google UK English Female", "en-GB", "en_GB"],

// Prefer Female voice
myAssistant.ArtyomVoicesIdentifiers["en-GB"] = ["Google UK English Female", "Google UK English Male", "en-GB", "en_GB"];

// Rest of your code

How can I pick a female voice on iOS? This solution does not work on iOS.


Abandoned Allow business entity editing

This adds methods for admins to see the current state of the business entities for
a given office, add a new business entity and update an existing business_entity.code and .name.
When updating only a BE.name, no new item is created.  If the code is amended, the existing BE has it's valid to date set and a new BE is created with the same start date and the new code.

This is a huge amount of code, sorry about all the comments. One thing I find missing though is some feature tests, to cover adding / editing of BECs.

Due to the scope and nature of the changes, along with the need to merge to the source BE branch, this PR will be closed.  A new branch has been created a new PR will be created for separate review.
I'll leave this here until the new version is merged, as a reference.


[Enhancement] Improve How Prepared Queries are Used

See https://www.reddit.com/r/PHP/comments/12axnwa/comment/jew6rcf/?utm_source=share&utm_medium=web2x&context=3

This feature was implemented by #74


Looks interesting... is the project still active?

This looks really interesting in combination with my Restic setup. However, the project didn't have any commits since about January 2023... is it still active / planned to go forward?

@mvdkleijn not really I'm afraid, following upstream Restic changes is very time consuming (very active project) and I had to abandon the idea for now.

OK, a shame but understandable. 👍


ACOM-771 Memorize  quiz component assets

When call questions and outcomes on a Quiz object, it requests same asset ids twice which should be memorized.

Updated, thanks!

Awesome :) Bump the version to 2.1.1 and it's ready to merge (patch version bump)

Had the version bumped to 2.1.1.

Looks great! Good job @hlcfan!

Waiting for Travis to finish before I can merge from my phone


Environment checking

I work with a lot of people that have very long living environments, so it would be quite nice if there was a way whether a current environment is compatible with a lockfile. Here I'm pulling inspiration from poetry check which will check if the versions currently installed are compatible/equal (I'm not actually sure which one) with those listed in the lockfile.

Sorry it took me so long to get back to you, that was a little rude of me. So let me try to illustrate what I mean with an example. We have a repo called X and on it work three people, A, B and C. To make sure they all have consistent envs with eachother they check in the lock file with the repository.

First A decides to run pixi add tomli which installs it to their env and write some tests for it. They add the lockfile and push to the repo.
When B now pulls the repo they have to reinstall/ update their environment, which it would be nice to be able to check without having to reresolve everything (if that's even possible, I assume that's what poetry does)
meanwhile C added polars to the project for some small automation but forgot to check in the lock file, so now when the CI tries to install from the lock file it breaks. For this one specifically, at a previous job we had a CI that checked if the lockfile and the pyproject.toml were still in sync which was required before merging to main.

Basically what I want is to have a lightweight way to check for sync errors between multiple actors. I hope this clears it up a bit, let me know if you still have questions

Hey @savente93! No worries at all :)
I think for pixi things are a little different. For every command that interacts with an environment (pixi run, pixi add, pixi shell, etc.) pixi will always update the lock-file (if it is not in sync). This check is relatively cheap. This ensures that the lock-file is always in sync with the project file. If someone forgot to check in the lock-file, the lock-file is simply generated on the next persons' machine.
I don't think the case for person C can happen. Worst case if the lock-file is not checked in, it is regenerated in CI before running any tests.
You can see this by doing:
pixi init bla
cd bla
pixi add python
pixi run python
rm pixi.lock # Now remove the lock-file
pixi run python
# The lock file is back!

I think a lot of package managers do have a --locked/--frozen-lockfile/--frozen flag though that is used in CI. This will cause the program to error out if the lock-file is not up to date with the project file instead of regenerating the lock.
Maybe I still misunderstand the use case, or does this help?

Yeah I think this does help I think I should be good then. Thanks for the explanation! :)


Show login/signup modal for unlogged user when they click on reblog link

Trello: https://trello.com/c/i2dwEj9r/91-show-login-signup-modal-for-unlogged-user-who-hit-action-button
Display login modal when clicking on reblog link, text on modal was based on comments in Trello.

I'm not sure that we should just start working on that straight away.
There is a lot we should take care of to handle this properly (likes, dislikes, comments, saving posts, reporting posts, following users). This needs to be implemented in different places with different specifics (we should handle redirect links that will allow user to finish that actions right after they log in, so we need data about that action (post to be liked/disliked, comment, post to be saved, user to be followed).
I think it's not as trivial as it seems (it might be, but it won't result in high quality code and leave us with tons of duplicated and unmaintainable code).
See #803

@Sekhmet good point, I'll close the PR for now? and move the trello card back over to in progress?

You can use Blocked label and Trello board.


Update winusb-installation.md

Fix typo

Learn Build status updates of commit 4a20e74:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-driver-docs-pr/usbcon/winusb-installation.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks Vincent!


Eliminate redundant partition stats update

The update_partition_stats_q function is being invoked from load_partition in the pypgstac. However, this function is also being automatically triggered by changes to the items table:
postgis=> \dS items;
...
Triggers:
    items_after_delete_trigger AFTER UPDATE ON items REFERENCING NEW TABLE AS newdata FOR EACH STATEMENT EXECUTE FUNCTION partition_after_triggerfunc()
    items_after_insert_trigger AFTER INSERT ON items REFERENCING NEW TABLE AS newdata FOR EACH STATEMENT EXECUTE FUNCTION partition_after_triggerfunc()
    items_after_update_trigger AFTER DELETE ON items REFERENCING OLD TABLE AS newdata FOR EACH STATEMENT EXECUTE FUNCTION partition_after_triggerfunc()

Reference to the trigger definitions.
Is this redundant? If so, we should consider removing the redundant call from load_partition.

There is a difference between these two invocations: in one case, the function is called with istrigger set to False, and in the other, it is set to True. This affects the execution of update_partition_stats because certain parts of the function are skipped when istrigger is True, while the rest of the function still runs. Therefore, the point that the same piece of code executes twice remains valid.
@bitner, could you please elaborate on why the "Modifying Constraints" part is skipped when update_partition_stats is called with istrigger set to True?


BigSur 11.1 issue: The authorization was denied since no user interaction was possible

If macOS asks password more than 1 it'll fail with the error message in the title.
For example if we want to save an x509 certificate into keychain. On Apple machines with BigSur 11.1 asks password twice. (who knows why..)
(I've tested with Intel mac and it works properly)
Reproduce:
Here is our command:
security add-trusted-cert -d -r trustRoot -k /Library/Keychains/System.keychain "/Users/user/ssl/mysite.local.crt"

If you run this in terminal with sudo prefix then terminal'll ask your password and after that an OS dialog will be prompted and asks your password again. (video attached)
If you'd like to run the command with sudo-prompt (video attached) then you will get the following error:
The authorization was denied since no user interaction was possible

Because sudo-prompt hides the second dialog.
UPDATE
On the first video you cannot see password dialog because screencast app hides it. After I typed my pass in terminal an OS dialog came up and asked my pass again.
https://user-images.githubusercontent.com/3911387/104630933-c5992600-569b-11eb-8bdb-514133fdf7ef.mov
https://user-images.githubusercontent.com/3911387/104630946-cdf16100-569b-11eb-94c5-7df21aa71432.mov

This is macOS’s issue, not sudo-prompt IMHO.
But to help out, maybe try unlocking the user’s keychain first via node child_process and then execute the sudo command after. Other relevant stuff in that SO thread.


size of PDF page and image are different

How to get an image with the same size (width and height) as the according pdf page?

This is likely caused by the (read) resolution for Imagick (PHP lib used by this package) being set differently than the PDF being converted. It would be great if one could publish a config file and change the resolution one self. Changing the resolution variable in the vendor files directly won't even work. See PR #58.

Does not work setting resolution.
            $pdf = new \Spatie\PdfToImage\Pdf($on_server.$filename_pdf);
            $pdf->setResolution(300);
            $pdf->setOutputFormat('png')->saveImage($on_server.$temp_filename);

File input
AmicMapPin.pdf
File output only 63x63 pixels

not work for me

@nitinjiniguru yes, i am comment this code
 //$this->imagick->mergeImageLayers(\Imagick::LAYERMETHOD_FLATTEN);
in \vendor\spatie\pdf-to-image\src\Pdf.php
in function getImageData($pathToImage)

and my full code

Thanks @pliasun  its work for me..
i found the place where to solve the issue and i change according i require


[active_job] Fix deserialization of jobs that are missing metadata

We may get jobs from applications that do not have OpenTelemetry instrumentation, such as during a rolling deployment. In such cases, jobs failed to deserialize with the following error:
Minitest::UnexpectedError: ActiveJob::DeserializationError: Error while trying to deserialize arguments: undefined method `map' for nil:NilClass
    ~/.gem/ruby/2.7.5/gems/activejob-<IP_ADDRESS>/lib/active_job/arguments.rb:42:in `deserialize'
    ~/.gem/ruby/2.7.5/gems/activejob-<IP_ADDRESS>/lib/active_job/core.rb:190:in `deserialize_arguments'
    ~/opensource/opentelemetry-ruby/instrumentation/active_job/lib/opentelemetry/instrumentation/active_job/patches/base.rb:30:in `deserialize'
    ~/opensource/opentelemetry-ruby/instrumentation/active_job/test/instrumentation/active_job/patches/base_test.rb:40:in `block (3 levels) in <top (required)>'

This pull request fixes this issue.

Good catch,

deserialize_arguments calls Arguments.deserialize without checking if the argument is nil, https://github.com/rails/rails/blob/3bfe4191a39f064dc5fd2fc37a2cc85f070ad4e9/activejob/lib/active_job/core.rb#L189-L191

and Arguments.deserialize then calls .map also making the same assumptions about the argument being an array. https://github.com/rails/rails/blob/3bfe4191a39f064dc5fd2fc37a2cc85f070ad4e9/activejob/lib/active_job/arguments.rb#L42

This PR should help us be more defensive here.


:sparkles: support override images in helm chart

Summary
Related issue(s)
Fixes #

/approve
/lgtm


bug: forge coverage not fully compatible with forge test commands

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
forge coverage
Operating System
Linux
Describe the bug
Expected to be compatible with:

[ ] --json (exits early)
[ ] --watch (exits, does not watch https://github.com/foundry-rs/foundry/issues/4303)

Not expected but should be marked as incompatible:

[ ] --junit
[ ] --debug (is ignored), same for --framegraph and --flamechart
[ ] --list (runs coverage but returns as 0% hitrate)

forge snapshot appears to be fully compatible
It is confusing that these options are available

There are multiple JSON coverage formats (see https://docs.codecov.com/docs/supported-report-formats), and I don't think we should invent one for our own, so at the very least it should be a --format <some JSON format> flag.

uhh, format --json would be quite nice.
For aave dao we're currently using lcov-parse node package to transform from lcov.info to json.