Generalize how targets can be provided to cross-targeting build

To get things off the ground, I'm having to put an Import of Common\ImportAfter*.NuGet.targets in CrossTargeting.targets. This is undue coupling to NuGet. We need to generalize, but can't import everything in that folder as the rest is not cross-targeting aware.

So the core problem is:

Build logic that is needed in both inner and outer builds.
That isn’t nuget-based
That can’t come in from an import in common.props/targets, because those are too specific for the outer build.

Right?
Should we promote the concept of being a cross-targeting capable target? That is, import $(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.CrossTargeting.targets\ImportAfter\*.targets from Microsoft.Common.CrossTargeting.targets?
Is it enough to have a generalized mechanism for the outer build only, or should these targets always also be included in the inner build? I know the NuGet targets are currently in both.

Should we promote the concept of being a cross-targeting capable target? That is, import $(MSBuildExtensionsPath)$(MSBuildToolsVersion)\Microsoft.Common.CrossTargeting.targets\ImportAfter*.targets from Microsoft.Common.CrossTargeting.targets?

Yes. I think this is the simplest solution that removes all coupling and it's perfectly consistent with the inner-build case.

Is it enough to have a generalized mechanism for the outer build only, or should these targets always also be included in the inner build? I know the NuGet targets are currently in both.

I think we should keep the CrossTargeting and Common (inner) ImportAfter separate. Often the outer target will just batch over the TargetFrameworks and dispatch to the inner builds, which is not something to repeat in the inner build. If something is written to handle both cases in the same file (as nuget is today) then they can go ahead an import the same file from both locations, but if we go the other way and do this on everyone's behalf, then we will force some to author tricky conditions to only run in the right cases.

@emgarten is also asking for a way to provide targets via CustomAfterMicrosoftCommonTargets or similar to outer build.

As @nguerrera mentioned, NuGet uses CustomAfterMicrosoftCommonTargets to recursively walk through project to project references. Each project is loaded and a set of targets are added in and executed using CustomAfterMicrosoftCommonTargets. This is done as part of the outer loop, NuGet handles evaluating the inner loop itself.

For @emgarten's scenario, I think a feature separate from actual common targets where you can provide a targets file to msbuild.exe invocation and it will import that at the very end would be better. Using CustomAfterMicrosoftCommonTargets has the potential to conflict with somebody else in the spaghetti of targets hijacking it from you.

@emgarten what if nuget.exe carried a tiny project file that just 1) imported some project by a variable passed in 2) imported your targets, then you build that and pass the user project to it.

That seems like it would give the same result, I'll try it out.

@nguerrera projects seem to be evaluated differently when using an import vs pass the project under the project attribute of an MSBuild element. I think the outer build will need to support CustomAfterMicrosoftCommonTargets or a new property to handle this, but inorder to work download level it would be nice to keep this as CustomAfterMicrosoftCommonTargets unless it causes other issues.

This is blocking NuGet in two ways currently

NuGet.exe isn't able to bring its own targets along and use them to recursively walk the p2p graph
Pack is unable to run in the outer build (buildCrossTargeting folder is unusable so far)

@emgarten Why does the p2p traversal need to have targets injected? Shouldn't they be installed in the global location (and thus provided to every project)?

Shouldn't they be installed in the global location (and thus provided to every project)?

That could work, but today NuGet.exe brings its own copy and runs standalone. For scenarios where NuGet.exe is newer or has bug fixes it would be nice to keep the option of using any targets instead of just what MSBuild has.

Ok, so the problem is that the outer build doesn't import $(CustomAfterMicrosoftCommonTargets) (because it doesn't import Microsoft.Common.CurrentVersion.targets), so you can't inject logic into the build.
Sounds like the easiest thing would be to provide a similar hook in Common.CrossTargeting.targets: a final import of $(CustomAfterCrossTargetingTargets). Then NuGet can set both properties and satisfy both the inner and outer builds. Right?
Tangentially: I'm confused by the mention of nuget.exe here. I thought we were going to great lengths to make NuGet restore be driven by MSBuild.

It sounds like we're approaching a consensus to mirror Microsoft.Common.targets hooks with analogous but distinct Microsfot.Common.CrossTargeting.hooks:

Microsoft.Common.CrossTargeting/(ImportBefore|ImportAfter}
$(Custom(Before|After)MicrosoftCommonCrossTargetingTargets

And anyone want to hook both can do so by using both paths or setting both variables?
Are we all on the same page? Let's get this in soon if so. I've accumulated a lot of test debt (hoping to pay it down over next few days) in  other changes that had to be rushed in to msbuild to unblock this or that,.  I would prefer not add more to my plate. Is this something someone on the msbuild team can drive with high priority?

Sounds good to me. I can work on it.

Reopening until the old mechanism is removed--but that's not urgent.

Not positive this fixes @rohit21agrawal's pack issues. Solves @emgarten's issues, we believe.

@emgarten @rainersigwald @rrelyea Is it possible to remove the old mechanism now. I am seeing duplicate imports of nuget targets warnings. This is a possible culprit. Or is that tracked elsewhere / caused by something else?

Before removing the old mechanism NuGet needs to use the new one: https://github.com/NuGet/Home/issues/3860

This didn't meet the RTW bar for NuGet, so we can't take it for RTW either.

I hope we'll be allowed to remove it later.


Context menu add to playlist

Use nested contextmenu for adding to playlist (instead of popup)

Implemented at 2.5.17


Fire programatically with pre-filled value

Is there an option to search and display results with pre-filled value, without the need to type anything into input field?
Here is an example of what I need.
// get autocomplete input
const siteUrlInput = document.querySelector('#autoComplete');
// get the search part of URL
const siteUrl = window.location.search.substring(1);
// assign the value to autocomplete
siteUrlInput.value = siteUrl;

After I assigned the value to AutoComplete input, I would like to get results for that query, but doing it programatically, without key presses etc.
Thanks

Hello @fluxxus,
Sure, you can do that after instantiating autoComplete.js as shown in the code below.
// Get autocomplete input
const siteUrlInput = document.querySelector('#autoComplete');
// Get the search part of URL
const siteUrl = window.location.search.substring(1);
// Assign the value to autoComplete.js
siteUrlInput.value = siteUrl;

// New Input Event
const event = new Event('input', {
    bubbles: true,
    cancelable: true,
});
// Event Dispatcher
input.dispatchEvent(event);

Try it and let me know how it went.
Cheers! :)

Hi @TarekRaafat and thanks for the quick response!! :)
Just one more thing please. How should I access the results now.
This clearly is not working now onSelection: feedback => {} . Any alternative approaches to accessing the results?

Sorry, I didn't get your question.
Could you please clarify what you mean by accessing the results not working, because in this demo accessing the results list and navigation works fine.

Ok, I will try to explain better this time.
I would like to have access to feedback without the need for user input. In your Codepen demo, I need to select one of the options to get the result displayed on the screen via feedback.selection.value.
Is there a way I can get the result displayed without the need for choosing anything?
Or better yet, I see in the console that CustomEvent is available on page load. It contains all of the information I need for the given query.
How can I access the info contained in that CustomEvent?
p.s. I hope I explained it better this time :)

Managed to do it on my own in the end. Thanks for the help! :)


draft toc

please review - there are some new files I just left at the bottom of
their nodes, and one whole new node - also it seems API vv3 is now in
two places in the repo

I'll merge and then make a couple of tweaks.


rate() is inaccurate for new label

Bug Report
What did you do?
I have a metric that shows calls to my app's "export" function. I use labels to signify what is exported, and in which format. I then plot this using sum(rate(export_total[1m])) by (what, extension)
What did you expect to see?
When someone calls the export function 5 times with a specific what/format pair, I see a spike in the graph to Y=5. If later it calls it again 3 times, I see a spike to Y=3.
What did you see instead? Under which circumstances?
Only the second spike appears.
This is because rate() doesn't treat the missing label as a value 0, and when it appears, rate assume that the current value is the baseline.
The workaround is to initialize all my counters with all possible label combinations to 0 (or any value) at application start, before users can increase them. This is annoying and difficult when those values are defined in a third-party library rather than my own app.
This behavior might be needed if an app already has non-zero counters when you start to scrape it, or when taking the range() of non-Counter metrics (such as Gauges) which don't necessarily start at 0. However it seems to me that a documented workaround is required to not have spikes silently disappear from graphs.
Related: https://github.com/prometheus/client_golang/issues/190
Related: https://www.robustperception.io/existential-issues-with-metrics (notes that this can lead to alerts not firing)
Environment

System information:
Linux 4.9.0-8-amd64 x86_64

Prometheus version:
prometheus, version 2.8.0 (branch: HEAD, revision: 59369491cfdfe8dcb325723d6d28a837887a07b9)
build user:       root@4c4d5c29b71f
build date:       20190312-07:46:58
go version:       go1.11.5
(from Docker Hub)

Prometheus configuration file:

global:
  scrape_interval: 15s
scrape_configs:
  - job_name: prometheus
    static_configs:
      - targets: ["localhost:9090"]
  - job_name: taguette
    static_configs:
      - targets: ["<IP_ADDRESS>:8900"]

Dupe of https://github.com/prometheus/prometheus/issues/1673, there's nothing we can do here I'm afraid.


Add Typescript support, export only components

Just some README updates on top of #20

@skoch thanks for the great feedback. I think I've addressed everything, so the only remaining issue is to get to the bottom of the CSS import situation. Then I think we can merge this in and ship a new version

Sounds good!
I'll take a peek later today and create a new project much like you did to confirm/deny.
Thanks for the discussion and looking forward to adding more.

I stand corrected.
In order to have the css bundled with the JS as an import, we should use vite-plugin-lib-inject-css
https://www.npmjs.com/package/vite-plugin-lib-inject-css
And with that the vite.config.js would need an update.
I'll update my PR (which will then be part of this?) I also see I neglected to add the types in package.json

I maybe should have put this comment here: https://github.com/czue/bluesky-comments/pull/22#issuecomment-2557990919 Pasting it below:
But, after merging #22 I'm now having issues with the CDN install:

Loading module from “http://localhost:4000/assets/bluesky-comments/main.css” was blocked because of a disallowed MIME type (“text/html”).

Seems like it's trying to load the css file from a relative path, but that file isn't present in the outputted build. Any ideas?
FYI, I'll be offline for the next couple days but should be able to turn this around on Monday or Tuesday before the holiday break.

Ahh, I see that now as well when testing that method.
I guess it could be removed and then users would be required to add the CSS in their project. Not ideal in my opinion but if you need to support that usage then I'm not sure how to proceed.
I'm wondering if there's a way to support both.
I'll be offline as well for a few days!

Ok, for now I've reverted e00857be4b8b54e54c3dd6955abb32aa5140d1af and will merge this all up and cut a release. Thanks again for the help! After this, we can decide if we want to do anything about the css stuff. I'll also add you as a collaborator on the project. :slightly_smiling_face:
Hope you have a good break!


accept integers in gates and instructions

This PR allows to have integers when gates and instructions are added to a circuit.
circ = QuantumCircuit(QuantumRegister(4))
circ.h(0)
circ.h(2)

I don't think the following instructions will work with this patch, as they don't use decorators.
barrier, snapshot, initialize.
Also I don't think conditionals will work. For example apply h on q0 if creg1 equals 3
circ.h(0).c_if(1, 3)
Also I'm not sure if ranges work:
circ.h(0:3)  # apply h on first 2 qubits
Can you add support and tests for these?

Good point!
Let's brake it down:

barrier

It seems supported. Test added in 0e2b3d1314fb605420fd47ca8ef60b6081bde9b8

snapshot

It does not support qubit arguments.

initialize

It seems supported. Test added in cdc39c1c (but it is not passing because #2128)

Also I don't think conditionals will work. For example apply h on q0 if creg1 equals 3
circ.h(0).c_if(1, 3)

Should not be circ.h(0).c_if(creg, 3)? I added a test in db85b385 and it seems to work.

Also I'm not sure if ranges work:
circ.h(0:3) # apply h on first 2 qubits

0:3 is not a valid expression in Python syntax.

0:3 is not a valid expression in Python syntax.
You are right. Let's make it work with range(0, 3) though

Can we also have the ability to create circuits without explicit registers:
circ1 = QuantumCircuit(3)  # 3 quantum wires (make it a flat quantum register internally)
circ2 = QuantumCircuit(3, 2)  # 3 quantum and 2 classical wires

Should not be circ.h(0).c_if(creg, 3)? I added a test in db85b385 and it seems to work.

Yeah let's keep conditionals only work with a creg for now. Seems hard to change in the short term.


Desert map?

Hello! Realease is comming and SO.. How to add map detection? Could you provide some offsets of current map?

@ivq  #37


enableAnalysis doesn't work with 1.5 galley

To reproduce, using Master, istioctl manifest generate --set values.galley.enableAnalysis=true | grep -i analysis.  There will be no --enableAnalysis output.
The Istiod deployment does not get the --enableAnalysis=true flag.
The Installer operator/pkg/vfs/assets.gen.go is generated from install/kubernetes/helm/istio/charts/galley/templates/deployment.yaml which does include the flag generation, but only for the istio-galley Deployment.  The Istiod Deployment does not have it.

--enableAnalysis was experimental in 1.4 and off by default. Since its still experimental (we didn't have sufficient time to benchmark and stress test at scale) it wasn't included in istiod in 1.5.x. Maybe we can get this into 1.6 which would require completing the necessary testing plus minimal integration of galley's analysis controller into istiod.

@ayj Is it possible to deploy Galley with 1.5?

You'll need to revert to the non-istiod micro-service deployments.

This feature was expected to be on by default in 1.6 after performance testing in off-by-default mode in 1.5... What needs to be done to re-add this feature to the project?  Were there any other features which have been removed for istiod?

Were there any other features which have been removed for istiod?

Yes, all of Galley was disabled by default, aside from validation which was moved over. As far as I know this means status/analysis and MCP source.
The feature is still around as off-by-default, you can deploy Galley with the flag enabled still.

Should we close this?  I was able to get Galley ready to deploy with analysis.
istioctl manifest generate --set components.galley.enabled=true --set values.galley.enableAnalysis=true | grep -i analysis

I couldn't actually deploy Galley.  See https://github.com/istio/istio/issues/21635 .

I was able to get Galley up and running using ./istioctl manifest apply --set components.galley.enabled=true --set values.galley.enableAnalysis=true --set values.global.controlPlaneSecurityEnabled=false
No statuses appear on the Istio resources, and although the logs for Galley say many analyzers are disabled some are running.
...
2020-02-28T20:12:50.604560Z	info	analysis	Skipping analyzer "annotations.K8sAnalyzer" because collection k8s/core/v1/namespaces is disabled.
2020-02-28T20:12:50.604578Z	info	analysis	Skipping analyzer "injection.JwtAnalyzer" because collection k8s/core/v1/services is disabled.
2020-02-28T20:12:50.604586Z	info	analysis	Skipping analyzer "auth.MTLSAnalyzer" because collection k8s/core/v1/pods is disabled.
...
2020-02-28T20:12:50.911633Z	info	source	Starting status controller...
2020-02-28T20:12:54.422861Z	info	processing	Publishing snapshot for group: default

After conversations with the networking team, it sounds like getting this feature working in istiod will be a priority over the next few weeks, and the plan is to release it with 1.5.1 as a bug fix.  For now, I am tracking the work under #21341

@therealmitchconnors can you please point to where this agreement was made?

@howardjohn this was in the UX Working group meeting on 2/25, but further conversations have cast doubt on the plan.  What do you think?  This is not a new feature, and it was not intentionally dropped from 1.5, so I can understand the argument that it is a bug fix.  Alternatively, it would be a fairly large change to 1.5.x...

You can still use analysis if you ship Galley, and we intentional made this decision to ship with Galley disabled by default given what features we ported over to Istiod in 1.5. Re-enabling is far too large of a change for 1.5.x in my opinion

Could anyone provide a doc about what does it actually analyze?

Background Analysis runs the same code as istioctl analyze and outputs its results to the status field of the relevant resource.   For more information on analyzers, see our announcement blog post, or the relevant istio.io page.


Add SparkSetup

Need a version of the current TalonSetup file for Spark Max motor controllers for 2023 drive and mechanisms
https://github.com/BitBucketsFRC4183/FRC2023-Charged-Up/blob/elevatorPrototype/src/main/java/org/bitbuckets/lib/vendor/ctre/TalonSetup.java

Did you look in vendor/spark

I didn't see this, but was looking at the wrong branch of the code. Consider renaming to vendor/rev. Should this be closed or are there still necessary features to add?

Limit switches, followers, are the only features missing
Smartmotion is also missing but not specified in my IMotorController contract

Sounds good, I'll leave the issue open for now.
Perhaps we should add another ISmartMotorController interface that implements SmartMotion/ Motion Magic


Fix encryption rules do not support uppercase fileds

Fixes #3309
Method rewrite of class EncryptInsertValueParameterRewriter get encryptor by logic column, which is got from SQLStatementContext and formated to lower case by orm frame(like mybatis) . So if encryption rule use upper case logic column, the rewrite method can not get encryptor by it. And then Encrypt-JDBC fail to rewrite the SQL.
Changes proposed in this pull request:
The method findPlainColumn and findShardingEncryptor use the lower case logic column as parameter, and  if the encryption rule  use upper case logic column, it  cause the problem.

Any unit test? And check the CI result.

Pull Request Test Coverage Report for Build 1202

16 of 17 (94.12%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 67.301%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/strategy/encrypt/EncryptTable.java
8
9
88.89%

Totals

Change from base Build 619:
0.02%

Covered Lines:
11310

Relevant Lines:
16805

💛  - Coveralls


Gem Not Loading

Ruby 3.2.2, this gem is not loading:
begin
  require 'gen-ai'
  puts 'Gem is loaded!'
rescue LoadError
  puts 'Gem is not loaded!'
end

Shows up in Gemfile and Gemfile.lock but it's not loading for some reason.

Works!   But...Bundler.require doesn't load it.

Fix is now on main branch.


Bad image format when loading C++ dll

Hi,
Thanks for this great piece of software, I'm devlopping a viewer plugin for a particular astroimage format called FITS. Here is the project link.
The core of my plugin is in C++ which is built as a DLL. The plugin.cs calls functions in it like
<EMAIL_ADDRESS>CallingConvention = CallingConvention.Cdecl, CharSet = CharSet.Unicode)]
        public static extern IntPtr FitsImageCreate(IntPtr path);

However at runtime, my plugin crashes and the exception log in QuickLook indicates bad image format.
As far as my little knowledge in the Windows world and some Googling goes, this means the plugin.dll is loading my viewer_core.dll which targets the wrong architecture. Here is some information that I gathered in this regard.

My machine is running 64bit windows. The plugin.cs project is using build configuration "AnyCPU", and my C++ project is built as x64. This produces the above exception.

Same machine. The plugin.cs project is using build configuration "x86", and my C++ project is built as win32. This causes QuickLook to crash at startup.

I have a standalone viewer in c# just for testing purpose, it shares the same code as plugin.cs. This viewer runs ok as x86 with a win32 C++ dll.

It seems to me that neither x64 nor win32 build works. So please advice on how to build a correct C++ DLL.
Thanks

My machine is running 64bit windows. The plugin.cs project is using build configuration "AnyCPU", and my C++ project is built as x64. This produces the above exception.

You can verify if QL is running in x64 by looking at the Task Manager. Also, you should check the arch of viewer_core.dll using dumpbin: https://docs.microsoft.com/en-us/cpp/build/reference/dumpbin-reference?view=msvc-160

Same machine. The plugin.cs project is using build configuration "x86", and my C++ project is built as win32. This causes QuickLook to crash at startup.

This is expected since 64-bit QL is trying to load your 32-bit DLL and failed.
Basically, you'll have to provide two viewer_core.dll as viewer_core_32.dll and viewer_core_64.dll, and compile your plugin.cs as AnyCPU plus perfer-32-bit OFF.

Thanks a ton, I finally got it working.
I think the lesson learnt is to check the cpp.dll that's actually packed with the plugin.dll. Mine kept copying an old x86 version. I was lost in the many output diretories that VS generates (x64, bin/release, bin/x64, etc)


need HDF5 flag for moab

Was setting up CAD_to_OpenMC in an isolated environment. Following the steps in the readme and got
print(f'ERROR: generated file {h5mfile} does not appear to be a hdf-file. Did you compile the moab libs with HDF enabled?')

I think the -DENABLE_HDF5=1 flag be added to the instructions in the readme maybe? This solved the issue for me.

This is in fact the same thing as #49. CAD_to_OpenMC should somehow detect the lack of hdf5-libraries, and at the very least issue a warning that the output file is in fact in vtk-format even though it is named h5m.

fixed


feat: allow options to be a fn in control pickers

this is required if one wants to use i18n labels for options

Sure, this is consistent with all of the others, so seems ok.

released


Crash in WAMR Android platform when enable simd

I add -DWAMR_BUILD_SIMD to build Android platform runtime iwasm. And when i use "wasm_application_execute_main" or "wasm_runtime_call_wasm" to call a wasm or aot funtion will going to crash. And detele the -DWAMR_BUILD_SIMD in runtime, no exception occurs during running. Is SIMD not supported on WAMR Android platform yet?

Hi, are you using ARM architecture? Currently, SIMD is only supported on the x86 platform.

Yes, i use aarch64 architecture. Expect the SIMD feature of WAMR on the Android platform.


08/08/2018 01:43 : Create Pull Request for branch prTesting_2018080801 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


Fix a bug in filtering + a few improvements

Hi, in this PR quite a bit of different stuff so I'll explain below.
First fixes, as mentioned https://github.com/softsimon/angular-2-dropdown-multiselect/pull/227 by default it was showing empty message now it's fixed.
All event handlers have now guarded access to event methods since some applications can run in webworker platform and calls like e.preventDefault() would fail.
A bit simplified template of dropdown, removed nested ngIf's.
Usage of filter pipe was moved from template into class to have results in a variable to be able to access it from more than one place.
Added new output onFilter which basically is a stream of search field values.
Added new setting closeOnClickOutside which by default is true. Adds ability to disable auto closing when user clicks outside of dropdown (required to use this in webworker mode since document click appears right after user clicks to open dropdown and it closes it immediately).
P.S. I merged with master and tried my best to keep all external changes but might have missed something when was resolving conflicts in dropdown.component.ts and dropdown.component.html so please check if everything is in place =)
Thanks

BTW I was working on an app running in webworker using your component and currently tracking one issue with clicks particularly happening  in this component but as far as I investigated comes from angular side.

Thanks again gund. New version is published.

@softsimon awesome! Thanks for quick merge =)
BTW it would be a good idea to introduce some e2e testing to run in CI to prevent this kind of bugs earlier for all PRs. What do you think?


Any chance of an update?

I am trying to build / install for node 9.11.1 and eventually would love to be able to install for 10, but there are depreciation errors. Any chance the dependencies or other requirements could be updated?
Thanks tons! This looks pretty promising for me.

Yeah you were probably having problems with the sqlite3 module, the version that botbouncer was using was very outdated.
I've updated the dependencies, I was able to get it installed and most tests working on node v10.6.0.
Please note that the Bitcoin payments checking isn't going to work since the blockr.io API isn't around anymore.

Thanks for the quick turnaround! Super appreciated.


[TEVA-1661] Review apps - make PR comment sticky

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1661
Changes in this PR:

Change the Post PR comment action in the review workflow to use sticky PR comment to ensure the PR conversation only has a single link to the review app

Review app deployed to https://teaching-vacancies-review-pr-2516.london.cloudapps.digital

😍

Good old Martin!


error with run1.bat: qemu-system-x86_64: failed to find romfile "vgabios-stdvga.bin

`set OSNAME=CustomOS
set WORKING_FILE=%0
set FIXED_WORKING_FILE=%WORKING_FILE:"=%
set BUILDDIR=%FIXED_WORKING_FILE%/../bin
set OVMFDIR=%FIXED_WORKING_FILE%/../../OVMFbin
qemu-system-x86_64 -drive "file=%BUILDDIR%/%OSNAME%.img" -m 256M -cpu qemu64 -drive "if=pflash,format=raw,unit=0,file=%OVMFDIR%/OVMF_CODE-pure-efi.fd,readonly=on" -drive "if=pflash,format=raw,unit=1,file=%OVMFDIR%/OVMF_VARS-pure-efi.fd" -net none
pause`
that is what I have for my run1.bat file but when I run it it gives me this error qemu-system-x86_64: failed to find romfile "vgabios-stdvga.bin"

I had the same problem. It looks like qemu is expecting the romfile in the current directory.
To fix that you can either

Copy the file vgabios-stdvga.bin from your qemu install folder to the kernel folder, or
Add this to your bat file before the qemu call: cd /D "C:\Program Files\qemu\" (Adapt the path to your qemu install folder)

Awesome Thanks! :)


Add hover effect on pick a color

Screenshots
If applicable, add screenshots to help explain your problem.

I would like to add some hover effect on the focus effect on the colors.

Can you please elaborate @gnikita432

are you working on this issue @gnikita432

No @arpit456jain you can take this issue.

No i had not commented it for taking it up😅 i was asking because we have a plan to migrate this project into django soon so i am waiting for all PRs got merged 🙂
@ankitapuri i think you can add label for up for grabs :)


How to Solve Phone Auth,verification failed:Invalid Token  Error

I am using Unity Firebase unity Sdk 6.16.1 and Unity latest quick master.
I have enabled Phone Authorization in Firebase Console and added team id and .p8 file in Firebase Console.
Actually when i run the  Auth project from Unity latest quick master. The SMS is sent to the desired no.
But it asks for Captacha and then only it sent sms to the desired no.
So i have enabled Push Notification,Backround Mode and Remote config options in Xcode to avoid Image capatcha..
After i enabled it  and  when i run it it says
Phone Auth, Verification failed: Invalid Token Firebase Error
So what i have to do.  I am using Xcode  12.01   Mac OS catalina  10.15.7
What i have to do get the Sms. I dont need capatcha in  my app. I have to bye pass them .  If i enabled pushnotification, backgroud mode.. then the  Sms is not sent.
What i have to do..

Hi @CLOUDFIRE91,
This is a great question for the backend team and not something that is being caused in the Unity SDK. Please contact Firebase Support who will attempt to route the question to the backend team directly. Am closing this ticket here but please feel free to refer support to this ticket.

Hi All
I found a Solution. There is nothing in error in unity. I did not add .p8 file in firebase console. Once i added the error solved.
Thanks vimanyu for your reply.


Add support for retries

Fixes #504.
This is just an initial draft to get feedback. There's no tests and it only has the retry in one (doWithSession) of several methods.
This would then need a second pr for spring cloud vault (here) so that we can use the spring.cloud.vault.something.retries=3 niceness.
I struggled a bit about where to setup the retry template. Is that setter a good idea? I'm a bit confused about the optional scope of some dependencies too, and even if spring retry is needed or, given how simple this is, we could just wrap this in some loop / try catch and not add extra libraries.

This change introduces a mandatory dependency on Spring Retry. Instead, it would make sense to have a ClientHttpRequestFactory that uses the Retry Template, ideally, within Spring Retry itself since HTTP retries aren't specific to Spring Vault.

This change introduces a mandatory dependency on Spring Retry. Instead, it would make sense to have a ClientHttpRequestFactory that uses the Retry Template, ideally, within Spring Retry itself since HTTP retries aren't specific to Spring Vault.

Thanks for the quick reply Mark!
I'm a bit confused by the last part of the comment, the "ideally within spring retry itself".
A new factory would look like this, no?

Thanks for the quick reply Mark!
I'm a bit confused by the last part of the comment, the "ideally within spring retry itself".
A new factory would look like this, no?

While it requires a new factory, users need to be able to specify a bit of configuration so we can't provide a context-less factory. It would make sense to plug the Retry integration on configuration-level together in the sense, if there's Spring Retry on the class path and a RetryTemplate is available, then we would pick up the ClientHttpRequestFactory and pass on the correct ClientHttpRequestFactory to RestTemplate and VaultTemplate.

While it requires a new factory, users need to be able to specify a bit of configuration so we can't provide a context-less factory. It would make sense to plug the Retry integration on configuration-level together in the sense, if there's Spring Retry on the class path and a RetryTemplate is available, then we would pick up the ClientHttpRequestFactory and pass on the correct ClientHttpRequestFactory to RestTemplate and VaultTemplate.

I took a stab at this but with the injection of the retry template things get a bit messy. I reckon folks using this project would be ok with a solution a bit more verbose like in your answer here.
So I wonder if it would make sense to move this to spring cloud vault instead. Imo it's where we'd get more return on investment by having a more concise solution (ie, we could use spring.cloud.value.retries=x).
There's still the issue of the dependency on retry that's not something that's straightforward to solve but this would be contained in a single project since we can create/override the ClientHttpRequestFactory there. Similar to this: https://github.com/spring-cloud/spring-cloud-vault/compare/master...worldtiki:retries

I took a stab at this but with the injection of the retry template things get a bit messy. I reckon folks using this project would be ok with a solution a bit more verbose like in your answer here.
So I wonder if it would make sense to move this to spring cloud vault instead. Imo it's where we'd get more return on investment by having a more concise solution (ie, we could use spring.cloud.value.retries=x).
There's still the issue of the dependency on retry that's not something that's straightforward to solve but this would be contained in a single project since we can create/override the ClientHttpRequestFactory there. Similar to this: https://github.com/spring-cloud/spring-cloud-vault/compare/master...worldtiki:retries

Thoughts on the last comment, @mp911de?  My organization is running into this also - if a PR to spring-cloud-vault is a better location for this I'm willing to submit something for feedback.

We couldn't convince ourselves that duplicating efforts regarding HTTP retries is a good idea across projects. Closing this PR in favor of https://github.com/spring-cloud/spring-cloud-vault/pull/577.
Leveraging retry functionality of the supported Apache HTTP Client would be a much better option (see https://memorynotfound.com/apache-httpclient-httprequestretryhandler-example/).

That's alright. Thanks for the update,
@pmv feel free to reuse bits from https://github.com/spring-cloud/spring-cloud-vault/compare/master...worldtiki:retries
I think the only thing missing is that it's using spring-retry and not HttpRequestRetryHandler. The example @mp911de shared looks simple enough, it would just be a matter of adding some sort of backoff.

Don't get me wrong, if you're able to use Apache HTTP Client, then that would be the easiest approach. If you cannot use Apache HTTP Client and are required to use the JDK Client or Netty, then there's no built-in retry functionality.


Vulkan Mobile backend: Warn about unsupported features in the editor and class reference

Godot version
4.0.dev (2a9dd654b)
System information
Fedora 34, GeForce GTX 1080 (NVIDIA 470.74)
Issue description
The following features have been tested and have no effect when enabled in the Vulkan mobile backend:

VoxelGI
WorldEnvironment > SDFGI
WorldEnvironment > Volumetric Fog

These features are unlikely to ever be supported in the Vulkan mobile backend, given its focus on low-end hardware such as mobile GPUs and integrated graphics.
We should emit node configuration warnings when an unsupported feature is enabled while using the Vulkan Mobile backend. The inspector should also be configured to hide properties that have no effect when using the Vulkan Mobile backend (especially in the Environment resource). On top of that, we should mention limitations in the class reference.
Steps to reproduce

Switch to the Vulkan Mobile backend in the project settings and restart the editor.
Try enabling the features listed above.

Minimal reproduction project
N/A

What about Depth of Field, Subsurface Scattering, SSIL and SSAO?

We also need this for our new compute shader logic though here it should be a "maybe", compute shaders do work on some mobile hardware but not on all. I think in theory the mobile renderer does allow the logic even though we no longer use compute shaders ourselves for this reason.

I will work on this before release.
Note to self to also clean up https://github.com/godotengine/godot/issues/54611 (its too late to remove, but I can hide it in the editor and emit a warning when used from a script)


feat(dim): wallet creation config #453

Description
configuration for dim wallet creation has been added
redundant configuration for onboardingserviceprovider has been removed.
Why
dim configuration is required for merge of https://github.com/eclipse-tractusx/portal-backend/pull/523
removal of redundant onboardingserviceprovider configuration enhances maintainability
Issue
https://github.com/eclipse-tractusx/portal-backend/issues/453
Link to pull request from other repository.
https://github.com/eclipse-tractusx/portal-backend/pull/523
Checklist
Please delete options that are not relevant.

[X] I have performed a self-review of my changes
[ ] I have successfully tested my changes
[X] I have added comments in the default values.yaml file with helm-docs syntax ('# -- ') if relevant for installation
[ ] I have commented my changes, particularly in hard-to-understand areas

LGTM regarding DIM config but I don't get the change for the onboardingserviceprovider in this PR. IMO this should be a separate change or at least mentioned in the PR description.

description of onboardingServiceProvider has been added to the PR description


Update translation.txt

Translated, maybe edit needed :)

There are some hiccups but I think it's good enough as it is, thank you!


fix orders on docker images

Binary market have orders on No, the UI will only put orders on Yes. Need to change.
I suspect this same issue for scalar markets.

@justinbarry @bthaile  assigning to@robert-e-davidson3 to do in parallel.
Feel free to move that around if it needs it -- but should be done this sprint along with Trading Integration.


Fix mounting volumes problem

This PR fixes volume problem especially  Downward API volume.
I've run conformance e2e tests.
Before modification:
Ran 94 of 293 Specs in 3553.798 seconds
FAIL! -- 57 Passed | 34 Failed | 0 Pending | 199 Skipped

After modification
Ran 94 of 293 Specs in 14656.223 seconds
FAIL! -- 65 Passed | 29 Failed | 0 Pending | 199 Skipped

When I removed -v /:/rootfs:ro \ I have similar result:
Ran 94 of 293 Specs in 14452.516 seconds
FAIL! -- 66 Passed | 28 Failed | 0 Pending | 199 Skipped

LGTM


Remove source api from schema reads

Use the value passed in the metadata and intercepted by the bridge.

@tatu-at-datastax Why closing this? It's not done :)


Prevent add-space after character insertion

Issue
When i go back into a word to correct it and insert a character, a space is automatically added after it on completion.
Solution
While the auto-space is useful, it should only happen when no other character is following the current cursor position.

Add space by default on completion
Don't add space when a character is present right after the cursor

Describe alternatives you've considered
No alternatives known to me. I just have to go along and delete the extra space.

One one hand this sounds helpful, on the other hand this creates an inconsistent behaviour if you do not want to insert just a character, but a whole word.
You know that you can avoid the auto-space by ending your stroke outside the circle?

You know that you can avoid the auto-space by ending your stroke outside the circle?

That's a nice tip, thanks!

One one hand this sounds helpful, on the other hand this creates an inconsistent behaviour if you do not want to insert just a character, but a whole word. You know that you can avoid the auto-space by ending your stroke outside the circle?

Good point. And awesome tip, thank you !
There should be a manual for such things ― but maybe it is the WIP wiki?

You know that you can avoid the auto-space by ending your stroke outside the circle?

Glad I read through the issues before submitting a new one, that works for me :)

You know that you can avoid the auto-space by ending your stroke outside the circle?

This is helpful, but honestly, I'd rather have a toggle to disable auto-space altogether. I only just found out about this style of keyboard, and while I'm still learning how to use it, I'd rather not have to deal with inputs I didn't want in the first place. (I don't like autocomplete either :P )

[Edit] It's also worth noting that ending outside the circle also triggers certain other gestures, such as tabbing between fields or moving the cursor, which makes this impractical in some apps--such as 8Vim's own options menu and test screen.

That’s a bug; and a fix has been done and released, but isn’t showing up in F-Droid yet because F-Droid is as slow as usual.


Authentication error on fresh arch install?

Hello, I have a freshly insatalled arch system, I installed ly from AUR with paru, then I did the systemctl commands. However doesn't matter how much I tried ly does not accept my password. I did the literally same thing few months ago on a VM and didn't get the same issue so I was wondering if this happened with the latest updates.

Update: O tried an non numeric password and it worked... Still my numeric password does not work. Also I tried to install lemur and had the exact same problem.

I did a fresh install of arch on my hardware. I used the archinstall script. ly does not authenticate at all for me. I can log in just fine to the console. I tried changing my password to one without numbers, like suggested above. It made no difference. Very frustrating. I tried this in a vm a few weeks ago and it worked.


[Function Expose] ReportSelections.SendEmailToCust

Please make function SendEmailToCust in Table 77 "Report Selections" External.
For the sake of completeness you might also want to make SendEmailToVendor External.

Are there any other built-in functions to achieve this? If not would be great to have back SendEmailToCust and SendEmailToVendor functions.


When will the API be accepting registration again?

Hello, I'd like to use the product lookup API to look up some product info for my site. I saw that you had planned to open sign ups again last year from https://github.com/walmartlabs/walmart-api/issues/9. When will it be available again?
I tried to use the affiliates API instead, but it's returning responses in binary format...

Hi did you find any solution yet  ? I also need this API

Walmart.io is the continuation of open labs. Api works almost the same way.

Walmart.io is the continuation of open labs. Api works almost the same way.

Commerce API?


Change in description + change of section titles

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the Portuguese Translation of FreeCodeCamp.
Hope you continue to help us in the future to improve our translation for millions of users to enjoy.


Are there any methods or any way to get different sections of the report individually?

For eg.
When I run the report on my dataframes, I want only the output under the 'Columns with Unequal Values or Types' columns.
I also want the output of 'Sample Rows with Unequal Values'.
These are the only part of the compare.report I need. Is there any way to capture only this part of the report?

@ZlatGod I think this is similar to https://github.com/capitalone/datacompy/issues/45 so I'll close this one and continue the conversation there.


CDL is not unified among adapters

Hi everyone,
and thanks for this great initiative!
I was wondering why the CDL data is stored in the cdl property of the metadata dictionary when using the EDL adapter while when using the ALE adapter it’s instead stored in the CDL property of the ALE property of the metadata.
Here's a sample of an ALE metadata:
{'ALE': {'CDL': '(0.8714 0.9334 0.9947) (-0.0870 -0.0922 -0.0808) (0.9988 1.0218 1.0101) (0.9000)'
And here's an example of an EDL metadata:
{'asc_sop': {'slope': [0.8044, 0.9091, 0.9958], 'power': [1.0282, 1.0281, 1.0275], 'offset': [0.0278, -0.0074, -0.0484]}, 'asc_sat': 1.0}
I'm writing a thin wrapper on top of OTIO for exporting CDLs, having a unified interface would greatly reduce the amount code that is adapter specific.
Is there a specific reason for this?
Maybe this issue can be linked to https://github.com/PixarAnimationStudios/OpenTimelineIO/issues/170 since having the cdl in the metadata leads to this kind of non unified approach.. (instead of storing it as a Clip Effect, for example)
Thanks!

That looks like an oversight. They should certainly be unified.
I would guess that the EDL code is more widely used than ALE, so conforming to that would likely be least disruptive.
In the long run, we would like to evolve CDL support into an official schema, and include support in all of the adapters.

I think the issue is that the ALE adapter doesn't seem to support ASC CDL explicitly. I think it's getting the metadata via this fallback:
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/6d3268764f6b6d84db2274271eabd0ea28033990/contrib/opentimelineio_contrib/adapters/ale.py#L121
It would be great to port what we have in the CMX adapter to the ALE adapter.

I see. I’d love to open a PR implementing the same CDL structure of the EDL, if needed.
I’ll also check if this slight inconsistency appears only using the ALE adapter.

@vvzen Do you happen to have an ALE with CDL data we could use for unittests?

For reference, here is the CMX reader code:
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/6d3268764f6b6d84db2274271eabd0ea28033990/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L408-L445
And the CMX writer code:
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/6d3268764f6b6d84db2274271eabd0ea28033990/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L408-L445

For reference, here is the CMX reader code:
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/6d3268764f6b6d84db2274271eabd0ea28033990/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L408-L445
And the CMX writer code:
https://github.com/PixarAnimationStudios/OpenTimelineIO/blob/master/src/py-opentimelineio/opentimelineio/adapters/cmx_3600.py#L1219-L1239

@reinecke I do have tons of ALE samples which I 'm currently using to unit test the small local patch that I made to the (pure python) ale adapter. I'll ask if it's fine sharing them since it's client material (or I'll just make them shareable). Where should I send them?
Today I'll run your tests, sign the contributing pdf and then I'll open a PR here so you can have a look.

Thanks for putting together a PR!
As long as you have one that is a representative example, you could include it in contrib/opentimelineio_contrib/adapters/tests/sample_data/ so you can reference it from the adapter unittests.
Are there different flavors of how the data comes across in these, or is it pretty regular? If they're all pretty much the same, including one sample case should be fine. If you have more files you don't mind sharing though, feel free to attach them to this issue.

Hi @reinecke !
I've opened the PR and included one ALE sample and test since they were all pretty similar.
I also see that in the EDL adapter you're providing a no-op CDL whenever you can't find the CDL data. Is this something we want to do also in the ALE? I feel that it may be more useful to signal that no CDL was found in the file/clip, rather than providing a default one. What do you think?
Thanks

I agree that it can be important to distinguish whether CDL data was missing versus default is likely to be important in some workflows.

I agree, I'd prefer it didn't exist in the OTIO if it doesn't exist in ALE or EDL.

This has been merged so I'll close it myself, thanks!


Migrate to Lucide icons

https://lucide.dev
Actively maintained fork of Feather icons, with a much improved selection of icons! (I am a major contributor).
https://lucide.dev/guide/packages/lucide-react

Lucide is built with ES Modules, so it's completely tree-shakable.

This is fantastic @danielbayley, thank you.
However, we're close to merging extra-large #265 which may result in conflicts. Would you mind rebasing from main once this has been merged?

@chrisvxd Sure, will do.

@danielbayley #265 has been merged, please rebase when you have a minute

Still failing! This time it's lucide

Suggest you need to add the dep to the demo app package.json. You should be able to test by running yarn build before pushing.

Still failing! This time it's lucide

@chrisvxd Ah ok cool, I was going off yarn dev vs yarn build… Fixed and build is succeeding for me locally now.

Nearly there! Build is working, but linting is failing because of formatting. yarn format should sort you out.
Thanks for the persistence!

Nearly there! Build is working, but linting is failing because of formatting. yarn format should sort you out.
Thanks for the persistence!

@chrisvxd Strange, as yarn lint inside of app/demo was passing for me locally… It was just a few quotes and semis… All green now!


Rename appveyor.yml to .appveyor.yml

This prevents it from showing up by default in ls, just like
.travis.yml

Hey GitHub, I'm disappointed 😞 @josephfrazier can you help? Thanks

Oh, it was the changes in https://github.com/OctoLinker/browser-extension/pull/244/commits/5863a79fb304a4cb8d9b599764bc1d01ee2d109d (https://github.com/OctoLinker/browser-extension/pull/244#pullrequestreview-14792919) that conflicted. I just re-did the git mv appveyor.yml .appveyor.yml on master and force-pushed.


Shadow Props Support & RSKDrawutil refactored

Supported Shadow Props ( offset, color, opacity & radius ) on ReactSkia
Re-factored RSKDrawUtils design to decouple it from RSKComponents

@shefayaparwin
This dev branch is been used for PR with upstream and upstream PR is not closed.
Create a separate dev branch based on main_otv -> add changes and create separate PR.
This PR may have to be closed; since this branch is rebased with upstream; so cannot be merged on main_otv.

Closing this review as it is based on upstream repo. Will Rebase to main_otv and re-raise the review


Change bootstrap_setting from filter to tag

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall first build on feature/bootstrap-setting at 98.502%

Totals

Change from base Build<PHONE_NUMBER>:
98.5%

Covered Lines:
1290

Relevant Lines:
1303

💛  - Coveralls


Session requirements

Work in progress

Food for thought (and might intro you to some new functions) https://github.com/bbatsov/clojure-style-guide

Partially merged.


Add usePkce flag to allow authorization_code flow without PKCE

I ran into an issue recently with an Authorization Server that supports integrations with the OAuth2.0 authorization_code flow both with and without PKCE (with no upgrade path for enabling PKCE on old integrations).
You could argue that it's a bug with the Auth Server not being correctly backwards compaptible, (clients SHOULD include PKCE parameters regardless of server support, as per https://tools.ietf.org/html/rfc7636#section-5).
But there is still a potential fix in this library by allowing to explicitly opt out of PKCE. The issue arises when an integration is set up without PKCE, but the client sends the code_challenge and code_challenge_method parameters along with the auth request. This is interpretted by the server as trying to connect incorrectly and returns with "invalid_grant".
The oidc-client-js library always includes the code_challenge and code_challenge_method parameters. This feature request is to add a flag to support opting out of the clause as the "SHOULD" in the spec allows.

I have the bulk of the code working locally and can submit it in a PR for review, but wanted to get the input of the maintainers before progressing further.

I believe a client would not know if the token server supported PKCE or not. IOW, it does not hurt to send the params.


[Draft] Introduce aggregate function MaxBy

The defination of MaxBy is listed here: https://apache-iotdb.feishu.cn/docx/OTcIdaogWoZdtmx2pY9cG9LPn5b

Codecov Report
Attention: 204 lines in your changes are missing coverage. Please review.

Comparison is base (c52da2b) 49.02% compared to head (a0031a8) 49.21%.
Report is 12 commits behind head on master.

Files
Patch %
Lines

...engine/execution/aggregation/MaxByAccumulator.java
40.36%
130 Missing :warning:

...b/queryengine/plan/planner/LogicalPlanBuilder.java
35.71%
18 Missing :warning:

...ueryengine/plan/planner/OperatorTreeGenerator.java
20.00%
16 Missing :warning:

...an/parameter/CrossSeriesAggregationDescriptor.java
60.00%
14 Missing :warning:

.../slidingwindow/SlidingWindowAggregatorFactory.java
61.11%
7 Missing :warning:

...tesian/ConcatExpressionWithSuffixPathsVisitor.java
44.44%
5 Missing :warning:

.../planner/plan/parameter/AggregationDescriptor.java
62.50%
3 Missing :warning:

.../main/java/org/apache/iotdb/tool/ExportTsFile.java
0.00%
2 Missing :warning:

.../db/protocol/thrift/impl/ClientRPCServiceImpl.java
0.00%
2 Missing :warning:

...db/db/queryengine/plan/analyze/AnalyzeVisitor.java
75.00%
2 Missing :warning:

... and 5 more

Additional details and impacted files
@@             Coverage Diff              @@
##             master   #11986      +/-   ##
============================================
+ Coverage     49.02%   49.21%   +0.19%
- Complexity    25887    26115     +228
============================================
  Files          2920     2932      +12
  Lines        182861   183874    +1013
  Branches      21928    22048     +120
============================================
+ Hits          89653    90501     +848
- Misses        93208    93373     +165

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


Fixed list filtering

Added missing filtering map to ApiBase.getList() method. It was ignored, and passed as null.

Thanks very much @catacraciun :-)


Calendar event not visible in my own calendar

From @HaakonME on June 1, 2014 8:5
I created an event on June 6th at about 10am as a test, the event started at 0000 and ends at 2359 on June 6th. I checked the adjust to viewer's timezone, then filled in the title and description and location fields and posted the event. Notifications about the event became available, but the event is not visible my calendar, unlike other events from other people. Am I doing something wrong, expecting something wrong, or is there something wrong here?
Copied from original issue: friendica/red#477

From @HaakonME on June 2, 2014 8:40
Thanks, Mike!
#fixed

From @HaakonME on June 1, 2014 8:8
I found my own event at my channel home, and when I used the Add event to calendar, it became visible. I suppose  there is now both an invisible calendar event which I created and a visible event that I added.


Resolve Issue #311 and #310

Issue #, if available:
#310 and #311
Description of changes:

Removed "Action": "Bedrock:InvokeMode" from lambda roles. Affected files:
a. 01-create-agent-with-function-definition.ipynb
b. 02-create-agent-with-api-schema.ipynb
c. 03-create-agent-with-return-of-control.ipynb
d. 06-prompt-and-session-attributes.ipynb
e. 07-custom-prompt-and-lambda-parsers.ipynb
f. 08-create-agent-with-guardrails/agent.py
g. 10-create-agent-with-memory.ipynb

Renamed Agent in agents-for-bedrock/features-examples/06-prompt-and-session-attributes/06-prompt-and-session-attributes.ipynb to hr-assistant-attributes to resolve conflicts.

Removed error in CRM agent.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please re do this PR following the new layout and the process for creating the required file . We request all files to have a presence on the Cookbook / Recipes Website which is fronting this github repo now. Some of the mandatory sections of the notebook need to include

Overview.
What are we demonstrating .
What use case .
What will you learn.
What is the architectural pattern and why we select this. With a diagram
What are the libraries to install
What model did we choose and why this model
Every cell needs to have a markup


Fix/null image validation

Closes #409
Stepts to test:

make run

@JustinCast @steph222 @Edgar097 @xavier506 I looks that I change two files but I didn't, this took some old Justin commits, I tried to make a git pull upstream master to update my local file and also a restart command and nothing helped me.


does not work with pandas v.21

AttributeError Traceback (most recent call last)
 in ()
----> 1 msno.matrix(dfa.asfreq('A'), freq='A')
~/anaconda3/lib/python3.6/site-packages/missingno/missingno.py in matrix(df, filter, n, p, sort, figsize, width_ratios, color, fontsize, labels, sparkline, inline, freq)
212 t.strftime('%Y-%m-%d'))
213
--> 214 elif type(df.index) == pd.tseries.index.DatetimeIndex:
215 ts_array = pd.date_range(df.index.date[0], df.index.date[-1],
216 freq=freq).values
AttributeError: module 'pandas.tseries' has no attribute 'index'

Welp, time for another new minor patch version.

Fixed in the new 0.3.8 patch release (changed to pd.DatetimeIndex). Thanks for reporting this!

Many thanks for fixing this, we are updating the on new version.


SAA-1353 work in progress for introducing user location description on scheduled events.

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

...esmanagementapi/service/InternalLocationService.kt
96.46% <100.00%> (+0.09%)
:arrow_up:

...ppsactivitiesmanagementapi/model/ScheduledEvent.kt
93.18% <80.00%> (-1.08%)
:arrow_down:

...tiesmanagementapi/service/ScheduledEventService.kt
87.93% <90.90%> (+0.43%)
:arrow_up:

...activitiesmanagementapi/util/TransformFunctions.kt
78.91% <50.00%> (-0.17%)
:arrow_down:

...agementapi/client/prisonapi/api/PrisonApiClient.kt
87.84% <0.00%> (-0.70%)
:arrow_down:

...tiesmanagementapi/util/PrisonApiTransformations.kt
94.32% <50.00%> (-1.91%)
:arrow_down:

... and 1 file with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


[5.3] --timeout option for queue:work command

Lack of timeout is something I found dangerous for our workers. For me, calling set_time_limit() is not an option:

Warning: This function has no effect when PHP is running in safe mode. There is no workaround other than turning off safe mode or changing the time limit in the php.ini.

In this PR, a solution based on alarm signal is proposed. At the moment, this is just for consideration and discussion. I will try to complete the tests and do any suggested fix or improvement.
Very thanks for your time.
Related issue: laravel/internal, #13372

I was not aware of how ticks work in php! I have written similar code in perl and it works fine. But in php, the script will never terminate if, for example, you call file_get_contents('http://a_server_that_listens_but_does_not_respond.com') inside your job.

I think it is better to implement the feature using idle timeout option of symfony/procss:

Run daemon loop inside a symfony process ($daemon), with $daemon->setIdleTimeout($timeout).
Inside the daemon process, print out a dummy line every time a job is handled or after sleep.

If I did not misunderstood, the symfony process will consider the $daemon to be late if it does not print a line every $timeout seconds. Then it sends signal SIGTERM the late $daemon, and after 10 seconds, if it does not respond and terminate, it kills the process by signal 9.

I think this feature is fine? StyleCI uses this for protecting against runaway analyses:

OK. But maybe I try using symfony/process in another PR.

The symfony process thing will kill performance because it'll have to boot a new app for each job.

That said, this code will interfere with and break code that already uses this stuff internally.

I guess there is no need to boot a new app for each job. Have you checked this: idle timeout option of symfony/procss

I think the idea here is pretty good, but one thing I don't see is how the timeout is actually being monitored? Can someone explain how the system knows if the timeout time has been exceeded. I just see you checking if $timeout > 0?

I can add tests if you want to merge. But, my plan is this:

Make changes to queue:listen in order to improve its performance. It will be one parent process, and one process that runs queue:work --daemon. Then have some documentations to discourage the direct use of queue:work command in production, with or without --daemon option.
Let queue:listen accepts these 3 options:

--memory: The memory limit used to detect leak inside the queue:work --daemon process (as of --memory option in queue:work --daemon of Laravel 5.2).
--maxMemory: The maximum memory limit of the daemon worker process at any given time (as of --memory option in queue:listen of Laravel 5.2).
--timeout: the timeout that will be checked via idle timeout option of symfony/procss. Again, no need to have one process per job.

If --force is actually a useful feature, fix it so that it will be applicable to queue:work --daemon and queue:work --listen as well. Otherwise, remove it.
[Possible maintenance utilities] Have some artisan commands to pause/resume all of the workers or workers of a single queue.

Currently changes in .env file does not affect the worker commands started by the queue:listen command. And the queue:restart command does not restart queue:listen workers. I think this is an issue.

Can I just point that this breaks if someone is already using this stuff inside an internal library. Say, the internal library might set pcntl_alarm back to 0 again inside the job...

@halaei I'd like to get something like this merged. Are you still working on it or no?

I add tests soon.

I am unable to trigger the alarm. I pushed a job with an infinite while(true) loop but the alarm never fires. I can confirm that the signal handler is being registered.
How do I test this on my machine?

During the test for exceeding timeout, I found that if the pushed job class is everywhere inside the /tests directory, alarm signal will never get produced. That maybe has to do with the magical stuff of phpunit.

Basic tests are added. Testing daemon is a little hard :(

Don't know exactly what is causing the fails in PHP 7.0 and the behavior I mentioned above. Can there be bugs in phpunit or php 7.0?

Sounds like the behavior of declare(tick=1) varies in different versions of PHP. I have another code that only works on HHVM.

@taylorotwell Sounds like a change of behavior in PHP 7.0.
Here is a bug report: bugs.php.net
Here is my test: git repo, travis-ci

According to<EMAIL_ADDRESS>the tests in this PR working on PHP 5.* is a PHP itself! So, this PR only works on HHVM and yet-to-come PHP 7.1.
So, I will go back to try the symfony solution! Please note that it may need the use of output buffering functions (e.g. ob_start() and ob_end_flush()).


It does not handle small differences of platform version

Ubuntu 14.10 is seems to be unsupported, because templates/default/2.4/sysvinit/ubuntu-14.10/apache2.erb does not exist.
The error message is appeared by test-kitchen: (sorry for very long message)
         * httpd_service[default] action start
           * template[default :create /etc/init.d/apache2-default] action create

             ================================================================================
             Error executing action `create` on resource 'template[default :create /etc/init.d/apache2-default]'
             ================================================================================

             Chef::Exceptions::FileNotFound
             ------------------------------
             Cookbook 'httpd' (0.2.6) does not contain a file at any of these locations:
        templates/ubuntu-14.10/2.4/sysvinit/ubuntu-14.10/apache2.erb
        templates/ubuntu/2.4/sysvinit/ubuntu-14.10/apache2.erb
        templates/default/2.4/sysvinit/ubuntu-14.10/apache2.erb
        templates/2.4/sysvinit/ubuntu-14.10/apache2.erb

             This cookbook _does_ contain: ['/tmp/kitchen/cache/cookbooks/httpd/templates/default/module_load.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.4.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/httpd.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/envvars.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/mods/rhel/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/magic.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/ubuntu-14.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-5/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/systemd/httpd.service.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/mods/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/debian-7/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-12.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-10.04/apache2.erb']

             Resource Declaration:
             ---------------------
       # In /tmp/kitchen/cache/cookbooks/httpd/libraries/provider_httpd_service_debian_sysvinit.rb

       20:             template "#{new_resource.name} :create /etc/init.d/#{apache_name}" do
       21:               path "/etc/init.d/#{apache_name}"
       22:               source "#{apache_version}/sysvinit/#{platform_and_version}/apache2.erb"
       23:               owner 'root'
       24:               group 'root'
       25:               mode '0755'
       26:               variables(apache_name: apache_name)
       27:               cookbook 'httpd'
       28:               action :create
       29:             end
       30:

             Compiled Resource:
             ------------------
             # Declared in /tmp/kitchen/cache/cookbooks/httpd/libraries/provider_httpd_service_debian_sysvinit.rb:20:in `block in <class:Sysvinit>'

             template("default :create /etc/init.d/apache2-default") do
        action [:create]
        retries 0
        retry_delay 2
        default_guard_interpreter :default
        path "/etc/init.d/apache2-default"
        backup 5
        atomic_update true

        cookbook "httpd"
        variables {:apache_name=>"apache2-default"}
        declared_type :template
        cookbook_name "webapps"
        owner "root"
        group "root"
        mode "0755"
             end

           ================================================================================
           Error executing action `start` on resource 'httpd_service[default]'
           ================================================================================

           Chef::Exceptions::FileNotFound
           ------------------------------
           template[default :create /etc/init.d/apache2-default] (/tmp/kitchen/cache/cookbooks/httpd/libraries/provider_httpd_service_debian_sysvinit.rb line 20) had an error: Chef::Exceptions::FileNotFound: Cookbook 'httpd' (0.2.6) does not contain a file at any of these locations:
             templates/ubuntu-14.10/2.4/sysvinit/ubuntu-14.10/apache2.erb
             templates/ubuntu/2.4/sysvinit/ubuntu-14.10/apache2.erb
             templates/default/2.4/sysvinit/ubuntu-14.10/apache2.erb
             templates/2.4/sysvinit/ubuntu-14.10/apache2.erb

           This cookbook _does_ contain: ['/tmp/kitchen/cache/cookbooks/httpd/templates/default/module_load.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.4.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/httpd.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/envvars.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/mods/rhel/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/magic.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/ubuntu-14.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-5/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/systemd/httpd.service.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/mods/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/debian-7/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-12.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-10.04/apache2.erb']

           Resource Declaration:
           ---------------------
           # In /tmp/kitchen/cache/cookbooks/webapps/recipes/default.rb

        10: httpd_service 'default' do
            11:     action [:create, :start]
            12: end

           Compiled Resource:
           ------------------
       # Declared in /tmp/kitchen/cache/cookbooks/webapps/recipes/default.rb:10:in `from_file'

           httpd_service("default") do
             action [:create, :start]
             updated true
             retries 0

             default_guard_interpreter :default
             declared_type :httpd_service
             cookbook_name "webapps"
             recipe_name "default"
             instance "default"
             timeout "400"
             log_level "warn"
             keepalive true
             maxkeepaliverequests "100"
             keepalivetimeout "5"
             hostname_lookups "off"
             listen_addresses ["<IP_ADDRESS>"]

           end

         templates/ubuntu-14.10/2.4/sysvinit/ubuntu-14.10/apache2.erb
         templates/ubuntu/2.4/sysvinit/ubuntu-14.10/apache2.erb
         templates/default/2.4/sysvinit/ubuntu-14.10/apache2.erb
         templates/2.4/sysvinit/ubuntu-14.10/apache2.erb

       This cookbook _does_ contain: ['/tmp/kitchen/cache/cookbooks/httpd/templates/default/module_load.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.4.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/httpd.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/envvars.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/mods/rhel/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/rhel/sysconfig/httpd-2.2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/magic.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/ubuntu-14.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-5/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/systemd/httpd.service.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.4/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/mods/mpm.conf.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/scripts/a2enmod.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/debian-7/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/el-6/httpd.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-12.04/apache2.erb','/tmp/kitchen/cache/cookbooks/httpd/templates/default/2.2/sysvinit/ubuntu-10.04/apache2.erb']

       Running handlers:
       [2015-03-16T02:35:37+00:00] ERROR: Running exception handlers
       Running handlers complete

       [2015-03-16T02:35:37+00:00] FATAL: Stacktrace dumped to /tmp/kitchen/cache/chef-stacktrace.out
       Chef Client failed. 81 resources updated in 42.035333268 seconds
       [2015-03-16T02:35:37+00:00] ERROR: Found 1 errors, they are stored in the backtrace
       [2015-03-16T02:35:38+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)
>>>>>> Converge failed on instance <default-ubuntu-1410>.
>>>>>> Please see .kitchen/logs/default-ubuntu-1410.log for more details
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: SSH exited (1) for command: [sh -c '
sudo -E /opt/chef/bin/chef-client --local-mode --config /tmp/kitchen/client.rb --log_level auto --force-formatter --no-color --chef-zero-port 8889 --json-attributes /tmp/kitchen/dna.json
']
>>>>>> ----------------------

I can imagine it properly works after copying templates/default/2.4/sysvinit/ubuntu-14.04/apache2.erb to templates/default/2.4/sysvinit/ubuntu-14.10/apache2.erb.
Is there any major changes in init script in Ubuntu 14.10?
Is this cookbook is only for ubuntu 14.04?
I think the main problem is this cookbook does not handle small differences of platform version like this.

Closing this one out now since 14.10 is EOL


tgpt: 1.7.5 -> 1.7.6

Diff: https://github.com/aandrew-me/tgpt/compare/refs/tags/v1.7.5...v1.7.6
Changelog: https://github.com/aandrew-me/tgpt/releases/tag/v1.7.6
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 253806 run on x86_64-linux 1

1 package built:

tgpt


Problem occurs when moving wsl

Problem:
PS C:\Users\Downloads\LxRunOffline-v3.5.0-msvc> .\LxRunOffline.exe move -n Ubuntu-20.04 -d E:\Ubuntu_wsl
[ERROR] Couldn't set the case sensitive attribute of the directory "\?\E:\Ubuntu_wsl\temp".
Reason: An invalid parameter was passed to a service or function.
I had done the following command:
PS E:> icacls E:\Ubuntu_wsl /grant "lyx:(OI)(CI)(F)"   #lyx is user name

I also reboot my computer after downloading LxRunOffline

I don't know much about it, but could it be that E: is formatted with a case-insensitive file system (like FAT, often used with USB sticks)? If so, try formatting it with NTFS. NTFS is case-sensitive.


Run miri in CI

You shouldn't need to manually install xargo; miri provides cargo miri setup specifically to do the first-time setup that would prompt for user input with just cargo miri.


Export well logs to json files

Basically entails writing the well log data to json files after they have been opened and imported from DAT files. Involves updating/creating necessary repositories and services. Involves placing files (summary, location, formation, and sample for each well log entry in each county DAT file) in dedicated folders (i.e. summary folder, location folder, formation folder, and sample folder) within each well log folder. The main well log folder will be titled after the well log number. Inside this folder will be the four component folders (summary, location, formations, and samples), and within each of these will be the json files titles with their UUID value. The use of the UUID in the filename allows for multiple reports to exist for a given well log (which is often the case).

Folder structure:
county name
--->county.dat (existing DAT file)
--->well number (root folder for json files)
------->summary
----------->uuid.json
------->location
----------->uuid.json
----------->map thumbnail image
------->formation
----------->uuid.json
------->sample
----------->uuid.json


fix: update open-api-spec and generator script and package.json

Description:
update open-api-spec and generator script and package.json
Checklist:

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

https://github.com/hyperledger/aries-framework-kotlin/blob/72c218c434f17a1f7b12bbb46c4ede0dbc4c0264/settings.gradle#L18
Example of publishing to the ghcr maven repo


After packaging, recorded video does not display UMG

Hello, after packaging successfully, the recorded video UMG is hidden, is there any way to make the recorded video UMG display in runtime mode

Well its not an issue of plugin not working you just have to provide the right texture to it that it can read from. Currently for UI We use
 	/** Callback on the render thread after slate rendering finishes and right before present is called */ 	DECLARE_MULTICAST_DELEGATE_TwoParams(FOnBackBufferReadyToPresent, SWindow&, const FTexture2DRHIRef&); 	FOnBackBufferReadyToPresent& OnBackBufferReadyToPresent() { return OnBackBufferReadyToPresentDelegate; }
Which should technically be final image+ UI. To solve this we have to find out where is right place to capture the final image texture of rendering .

I had the same problem. Did you find a solution in the meantime?


support for saxon 9 b

It would be nice to have support for saxon 9 b, for two reasons:

it is open source
it supports some second order functions, most importantly saxon:evaluate (with which xspec code would also be simpler)

As a workaround I have figured out that the following script as "saxon" works, with sacrifying coverage support (it just forgets the first three parameters, and I have saxon 9 b at ~/lib/saxon9.jar):
$ cat bin/saxon
#!/bin/bash
echo "this saxon script is specially crafted to make xpspec able to use saxon 9 b"
shift
shift
shift
java -cp /usr/share/java/tagsoup-1.2.1.jar -jar ~/lib/saxon9.jar ${*} targetdir=$PWD

Hi Árpád,
thanks for providing some sample code for integration, that's always useful.
As far as I can read on the Saxon official page, Saxon-B is an older product (bold is mine):

Saxon-HE does not offer all the capabilities that were present in Saxon-B. Most notably, support for Saxon extension functions and other extensions was dropped, as was the capability for writing extension functions that rely on dynamic loading of Java or .NET code (a new facility for "integrated extension functions" is however available). Users whose code relies on these features of Saxon-B should either purchase the Professional Edition product or stick with Saxon-B: the latest release of Saxon-B is <IP_ADDRESS>, and although there are no plans to develop it further or maintain it, it will remain available indefinitely.

I don't know if it helps, but I wonder if it is possible to achieve what you're doing with extension functions. This page describes how to write extension functions using Saxon 9.2 or better: http://www.saxonica.com/html/documentation/extensibility/integratedfunctions/). Alternatively XSLT 3.0 supports higher-order functions and xls:evaluate which is similar to saxon:evalute. However, XSLT 3.0 currently requires Saxon-PE or Saxon-EE which both need a license.
I can see how easy it is to extend the shell and batch scripts to support Saxon-B but you need to bear in mind that you're using an older product that is not maintained any more.
Sandro

Dear Sandro,
As far as I understand, extension functions and XSLT 3.0 are only supported by the PE or EE edition. Both of them closed source. I am writing open source software, so cannot and do not want to depend on closed source ones.
There are saxon-b variants still maintained (I am not the only one in the situation above). Actually the provided script is driving the https://github.com/orbeon/saxon variant for me.

Hi @magwas,
I submitted a pull request to add support for Saxon B (#82), could you check if it would fix this issue?
Regarding code coverage, this is currently broken also for SaxonHE and SaxonEE/PE, I submitted another pull request to fix it (#80) but it is unlikely that it is going to work for Saxon B as it requires Saxon extension functions.
I hope this can help maintaining your open source projects.
Sandro


SIGSEGV crash due to bad pointer dereference.

First observed on bom02 due to a very high container restart rate. Some evidence on other nodes but at much less frequency. This could be the result of client populations.
<IP_ADDRESS> - - [19/Jul/2019:07:56:21 +0000] "GET /ndt_protocol HTTP/1.1" 200 0
2019/07/19 07:56:31 ndt5.go:118: Handling connection <IP_ADDRESS>:3010<=WS(S),JSON=><IP_ADDRESS>:47570
2019/07/19 07:56:34 server.go:3012: http: TLS handshake error from [2600:3c02::17:d804]:47730: EOF
2019/07/19 07:56:37 web100_linux.go:24: Summarized data: &{0 14 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0}
2019/07/19 07:56:39 protocol.go:274: Connection is not a TCPConn
2019/07/19 07:56:39 ndt5.go:64: Wrote /var/spool/ndt/ndt5/2019/07/19/ndt-vdzbg_1563218806_00000000000044FE.json
2019/07/19 07:56:39 ndt5.go:104: Test failed, but we recovered: runtime error: invalid memory address or nil pointer dereference
2019/07/19 07:56:39 close.go:31: Could not close connection (close tcp <IP_ADDRESS>:3001-><IP_ADDRESS>:35898: use of closed network connection)
2019/07/19 07:56:41 c2s.go:99: Ended C2S test on <IP_ADDRESS>:43435<=WS(S),JSON=><IP_ADDRESS>:46416
2019/07/19 07:56:41 c2s.go:119: <IP_ADDRESS>:3010<=WS(S),JSON=><IP_ADDRESS>:47570 sent us 5242.633287687231 Kbps
2019/07/19 07:56:41 ndt5.go:216: You uploaded at 5242.6333 and downloaded at 0.0000
2019/07/19 07:56:41 ndt5.go:64: Wrote /var/spool/ndt/ndt5/2019/07/19/ndt-vdzbg_1563218806_0000000000004507.json
2019/07/19 07:56:41 close.go:31: Could not close connection (tls: use of closed connection)
<IP_ADDRESS> - - [19/Jul/2019:07:56:31 +0000] "GET /ndt_protocol HTTP/1.1" 200 0
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x82d8f2]

goroutine 12245 [running]:
github.com/m-lab/ndt-server/ndt5/protocol.(*netConnection).Close(0xc0001b2550, 0x9701f8, 0xd830a0)
        <autogenerated>:1 +0x32
github.com/m-lab/go/warnonerror.Close(0x7f5759bbdfb8, 0xc0001b2550, 0x95af09, 0x1f, 0xc0001b2550, 0x45ccc6)
        /go/src/github.com/m-lab/go/warnonerror/close.go:29 +0x35
github.com/m-lab/ndt-server/ndt5/c2s.ManageTest.func2.1(0x7f5759bbdfd8, 0xc0001b2550)
        /go/src/github.com/m-lab/ndt-server/ndt5/c2s/c2s.go:80 +0x7c
created by github.com/m-lab/ndt-server/ndt5/c2s.ManageTest.func2
        /go/src/github.com/m-lab/ndt-server/ndt5/c2s/c2s.go:78 +0x49

I believe the following sequence is happening:
• a plain server ServeOnce() returns a bad testConn
• a goroutine with 3sec timer to close bad testConn on c2s.ManageTest return is defer'd
• ServerIPAndPort() panics b/c the testConn is invalid (or testConn.LocalAddr() is not a *net.TCPAddr)
• first panic recovery happens in HandleControlChannel, defer'd close runs
• after 3 more seconds the defer'd sleep returns and attempts testConn.Close, which also generates a panic b/c testConn is invalid, and crashes the server.
Why or how is ServeOnce returning n invalid testConn? At least one case is when the ServeOnce ctx times out (>30sec).
https://github.com/m-lab/ndt-server/blob/655fc0954a59a7633841aa7460e8901d91417e77/ndt5/singleserving/server.go#L185-L190
B/c Accept never returned, err == nil, and for the same reason conn is also nil.

Reproducible by connecting to server and never running C2S test:
	fmt.Println("Connecting")
	d := net.Dialer{Timeout: time.Second}
	v, err := d.Dial("tcp", "<IP_ADDRESS>:3001")
	rtx.Must(err, "Failed to connect")

	// Write initial TLV with <MsgLogin (2)><2-byte length: 0,1><TestStatus | TestC2S (16 | 2)>
	v.Write([]byte{2, 0, 1, 18})

	// Expect the "kickoff" string -- "123456 654321"
	b := make([]byte, 100)
	v.Read(b)

	log.Println(string(b))
	time.Sleep(60 * time.Second)


SFTP - Connect Hangs

I'm interested in your library and  trying to run the example to test SFTP access - however after calling connect() the system just seems to hang and nothing resolves
Is this due to my host being an IP Address? And If so is there a correct way to do this?
    var client = new SSHClient(
      host: "<IP_ADDRESS>",
      port: 21,
      username: "user",
      passwordOrKey: "pass",
    );

    try {
      print("Attempting connection");
      String result = await client.connect(); //Never get past here!
      if (result == "session_connected") {
        result = await client.connectSFTP();
        if (result == "sftp_connected") {`
        ...

Hi,
Are you running it on iOS or Android? Can you try to debug in XCode or Android Studio and see where it got stuck?
The IP address <IP_ADDRESS> is internal. Make sure your device in the same internal network.

This was running on an Android device which was on the same internal network as the SFTP server
I'll hook it up to Android Studio and get back with some more info

Archiving the thread. Feel free to reopen if the issue persists.
Thanks,
Qian