Milestone 1. Setup Overpass on own Azure instance

@dakotabenjamin @ramyaragupathy the MM leaderboard deployment will need to be updated when this changes (it currently points at export.hotosm.org).

Sub-tasks:

[ ] Set up CloudWatch monitoring for overpass replication state, disk and memory usage.
[ ] Switch other HOT applications to new Overpass

@mojodna other than MM Leaderboard, what other applications use Overpass? Is it OK if the only exposed CGI scripts are /interpreter and /augmented_diffs ? Also, I think we'll use HTTP instead of HTTPS for the new instance - the other public instances seem to be fine with HTTP and I don't see our instance being consumed directly from a browser (let me know if you know of otherwise)

Just MM and the occasional augmented diff-related experiment I run on an ad hoc basis.
/{interpreter,augmented_diff_status,augmented_diff} please.
I don't know of any browser-based uses.

@bdon notes

new overpass is running at http://overpass.hotosm.org/api/interpreter - gradually catching up, still about a day behind

@mojodna can you try switching to http://overpass.hotosm.org ? it should support those 3 endpoints now.

Working for my ad hoc tests.
@dakotabenjamin / @ramyaragupathy can you change OVERPASS_URL in the MM workers, test, and deploy?

sure @mojodna - we will test this out

@dakotabenjamin deployed the containers with the updated url. New overpass is about 14 diffs behind the main overpass. Otherwise rest of the updates on the DB is working as expected

There's a CloudWatch metric called ReplicationSecondsBehind in the HOT AWS account that tracks how far behind it is - looks like it can spike to around 10-15 minutes behind. I'm comparing this to the previous Overpass instance at export.hotosm.org and it's about the same.

I noticed the overpass instance doesn't have https enabled. Tracking it here in case it was forgotten.

@willemarcel see above https://github.com/hotosm/osm-export-tool/issues/315#issuecomment-509436286


'k6' is not recognized as an internal or external command.

Hi,
I followed the steps to configure k6 on Windows 10 and getting the error: 'k6' is not recognized as an internal or external command.
I have installed k6.exe (x64) and executing the steps in the mentioned order
git clone git@github.com:loadimpact/postman-to-k6.git
cd postman-to-k6
npm install
#On windows
node bin/postman-to-k6.js examples/postman/v2/echo.json
k6 run k6-script.js
Getting error at the last command: k6 run k6-script.js

@lobmails Sorry for the confusion, but the instructions expect k6 to be installed on your machine.
https://k6.readme.io/docs/installation
Please, let me know if the installation solves the issue.

@ppcano the k6 installation is giving error:
"Error: cannot load such file -- language/node"
any idea what this is about?

@rubal-jain
If the problem is the k6 installation, I recommend you:

ask for help in the Slack Channel
if it is a bug; open an issue in the k6 project with a detailed description of your environment and the steps to reproduce it


Custom keybindings

I prefer PlugUpdate to create new buffer instead of a new window, so I changed g:plug_window to "enew" but now pressing q after update closes the vim instead of wiping the buffer. Is it possible to rebind q to :bw or :bd? I want to rebind it only for this buffer, not globally.

You can bind keys with autocmd FileType vim-plug, but in this case bd seems to be a better default, so I updated the code. Thanks.


Make periodical wait of monitoring graceful shutdown configurable

Motivations

Current graceful shutdown takes 100ms at minimum if not completed
requests are exists when start to shutdown.

Changes

Make periodical wait configurable to make a chance to stop server
with shorter period

Is there any noticeable effect on total time taken for shutdown? I guess the difference would be less than 100ms, no?

Is there any noticeable effect on total time taken for shutdown because of this change? I guess the difference would be less than 100ms, no?

No, and correct, less than 100ms. Let me add more metrics and will come back here if needed.


fix(@angular-devkit/build-angular): wrong budgets

WARNING in budgets exceeded message showing wrong number
Fixes angular#16871.

Can you update the commit message to describe the change?
Maybe something similar to correct maximum budget size message

Done

Also merged to 9.0.x patch branch.


[Bug]: Docker Container keep crashing whenever started

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
b967513
Other environment information
Ubuntu 20.04.1 LTS
Docker 20.10.11

Actual behavior
I had pulled older version of the image in Feb and tried using the database. Currently, to update the image, I've pulled the latest docker image and started the container with docker run, but the container just keep exiting with error code 139.
docker logs shows the following message
Load base FULL catalog json from: /tmp/infinity/data/catalog/META_CATALOG.88.json
terminate called after throwing an instance of 'nlohmann::json_abi_v3_11_2::detail::type_error'
what():  [json.exception.type_error.302] type must be number, but is number
Expected behavior
No response
Steps to reproduce
Just run a container with the latest image.

Additional information

Don't know if these helps solving the problem.
Please mention if any other info is needed.

Compared to the February version, the metadata structure of the current version has been updated, which means that the Docker of the current version may encounter errors when reading the metadata of the previous commit. You can delete all the data in the database directory and restart the new version of the program. Or, you can still use an older version of Docker so that you can read the current data.
The default data directory is:/tmp/infinity, but it should be noted that deleting its contents will result in the loss of previously imported data.

Fixed in latest version and if the problem raise again, please reopen the issue.


RESTEASY-1296

Changed treatment of Weld proxies in GeneralValidatorImpl in resteasy-validator-provider-11.

Some kind of intermittent failure in RESTEASY-923-WF8.


docs(docker): fix release fail

修复 Dockerfile release 问题

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


Unable to find FQDN for resource

I am trying to configure Release Management for a project I currently have hosted in Visual Studio Online.  The target for the release are Azure VM's.  I'm following the instructions in this document https://msdn.microsoft.com/en-us/Library/vs/alm/Release/examples/net-to-azure-vm but every time it executes the Azure Deployment: Select Resource Group task, it fails with error messages saying that it is  Unable to find FQDN for resource for (VM name here).  In my resource group, I have 4 VM's and all have Public IP addresses along with DNS entries matching the format vmname-eastus.cloudapp.azure.com.  Unless I'm mistaken, that is the FQDN for the servers.  Also, the IP address have bee assigned as Static.  Does that make a difference?  Any help resolving this issue would be greatly appreciated.

I've figured out why it's unable to resolve the FQDN for my servers.  I have an Azure Load balancer set up in the same resource group and I only have one VM associated with the load balancer.  Beyond that, I have no Inbound Nat Rules associated with them either.  So it is not finding my VM's when it's building the $fdqmMap hashtable in the Powershell script.  This is where my VM's would be added to the map:
        #Find out the NIC, and thus the corresponding machine to which the NAT rule belongs         foreach($nic in $networkInterfaceResources)         {             foreach($ipc in $nic.IpConfigurations)             {                 foreach($rule in $ipc.LoadBalancerInboundNatRules)                 {                     $fqdn = $fqdnMap[$rule.Id]                     if(-not [string]::IsNullOrEmpty($fqdn))                     {                         $fqdnMap.Remove($rule.Id)                         if($nic.VirtualMachine)                         {                             $fqdnMap[$nic.VirtualMachine.Id] = $fqdn                         }                     }                 }             }         }
I feel like this is a bad assumption to make that if there is a Load Balancer then all VM's in that Resource Group must not only be associated with a Load Balancer, but must also have Inbound NAT rules as well.  These scripts should be able to find the FQDN whether there is a load balancer in place or not.

Thanks for the feedback. We will track and address this bug.

Fix will be available in next deployment


Update DownloadPipelineArtifact@2 inputs

Changes as referenced here: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/download-pipeline-artifact?view=azure-devops

@a2441918 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks for your contribution!
#sign-off


Remove deprecated flag ISTIO_GATEWAY_STRIP_HOST_PORT.

This is kind of clean up.

@SpecialYang: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
c30a0c86c239f5ce747516434fe635f7b1825b11
link
true
/test unit-tests-arm64

unit-tests_istio
c30a0c86c239f5ce747516434fe635f7b1825b11
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


How to change default position of UICompositeManipulator3D in SharpDx

I am trying to use UICompositeManipulator3D in WPF SharpDx. It always gets created at the center of the ViewPort3DX.
I need to draw a 3D box and UICompositeManipulator3D togather and bind manipulator to the box.
In helix there was property called Position was available to set the location of the manipulator.
How I can do the same in SharpDx??

I think this should be post under HelixToolkit repo


Show only active courses in Course Delegates and Course Setup by default

test comment - ignore

test comment - ignore


Rename files so the case matches the module name

This pull request modifies the npm run transform script so the module file names match the case of the default export.

@ahocevar, great to see this change.
Do you need the script part? I would expect the names to be correct after your change in the transform.

Will this mean that users of the ol package have to change their imports? If so we should tell people this, right?

@marcjansen I need to change this pull request. It was meant to accompany #7181, and not make changes to the ol package for now. For changes people need to make once what we've outlined in #7181 is in place, we are going to create a roadmap page with instructions on the wiki.

Pull request updated.

Strange Travis issue when building api docs. I'm sure it's unrelated to this pull request, but am unable to investigate this now.

See https://github.com/jsdoc3/jsdoc/issues/1438 for the build failure.

Thanks @tschaub for investigating the Travis issue. I've set the node version to 8.4 in .travis.yml for now.

Will this mean that users of the ol package have to change their imports? If so we should tell people this, right?

@marcjansen - you were right.  We should have left out the changes to transforms/module.js until 5.0.  I published<EMAIL_ADDRESS>with this change before realizing that it is breaking (on case sensitive filesystems).  I put up #7313 to fix and will get out a patch release when that is in.

In the meantime<EMAIL_ADDRESS>is out (I can't wait until the whole release process is as easy as publishing the ol package is now).


Include and render HTML into existing adoc file using asciidoctor

So I have an exisiting adoc template in which I want to include and render HTML file and save it to output PDF file.
The template file includes the following code:
include::header.adoct[] include::../owasp_report.html[]
PDF is generated using gradle command.
I've tried to use .md file as an input but generated PDF file does not render the way I would expect - e.g. markdown <p>  is displayed as raw markdown and its values are not rendered.

This converter does not support the inclusion of HTML content*. You might be interested in trying asciidoctor-pdf.js, which is an alternative HTML and browser-based PDF converter.
* It only supports a small subset of inline HTML, which is really more of a pseudo-language designed for inline formatting.


back port "fix TS-4195: double free when stop trafficserver" to 6.2.x

Conflicts:
proxy/Main.cc

@zizhong Could you include a reference for the original commit in your commit message?
https://cwiki.apache.org/confluence/display/TS/Git#Git-Cherry-Pick

@maskit Thanks! Fixed.

[approve ci]

Intel CC build failed! See https://ci.trafficserver.apache.org/job/icc-github/40/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1502/ for details.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1608/ for details.

The build issue is fixed by this commit https://github.com/apache/trafficserver/commit/6bdc94e2747594e17161d3ed6a4c691d0fd49670

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/172/ for details.

backport 6bdc94e to fix the build issue.

[approve ci]

Is the double free related to ICC build issues too?

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/freebsd-github/1669/ for details.

Intel CC build successful! See https://ci.trafficserver.apache.org/job/icc-github/101/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/linux-github/1565/ for details.

clang-analyzer build successful! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/233/ for details.

@PSUdaemon No. It's not related. When it got built, there were some ICC build issues, so I backport zwoop's fix too.

So, my bad, but we reverted this on master, because of issues raised on the use of _exit(). Also, not sure I agree that the ICC changes should be rolled into this (if we were to back port this).


(WIP) info-overlay: Improve styling of hotkeys.

Follow up to PR https://github.com/zulip/zulip/pull/12401.
Testing Plan:
Tested manually to see that everything looks okay.
GIFs or Screenshots:
First commit: Replaces all the "Up", "Down", "Left", "Right" with "↑", "↓", "←" ,"→".

Second commit: Increases size of arrow-keys. (WIP)

@rishig thoughts on which version is better?  I think I like the second one better than the first (it's more clear).  I could imagine replacing with an actual icon from an icon font and that being nicer, but this seems like an improvement in any case.

@timabbott could you clarify on this question on the previous pr? (Not sure that you saw it https://github.com/zulip/zulip/pull/12401)

Would it be easier to put the class="rendered_markdown" on the td blocks themselves instead of on each of the element tags? Or alternatively, to put span tags with that class inside the td blocks?

I think putting it on the td tags makes the most sense (answered in #12401 as well).

yeah, agreed the second one is better.

Looks great, merged, thanks @YashRE42!


Issue 91

Fixed #91. Some of the tests were configured with the wrong namespaces, which was hidden by the fact that namespaces were ignored. I fixed the other tests, but you should check to make sure they're still testing what you intended.

Looks nice!

Thanks a lot!


panic on graceful shutdown

Hello, when using the Shutdown method, I get a panic:

panic: send on closed channel

This is because the removeClient channel is closed before all HTTP handlers got closed.
I am a beginner, but to me it looks like we should never close removeClient.
But it will allow a goroutine leak, since goroutines trying to send on this channel will remain blocked.
I'll try to post a PR.

Share the code that you have, there are tests covering this area, it should work.

Hello, thanks for answering so quickly. In my PR the test reproduces the panic. I'm new to all this git stuff, sorry, I should have committed the test separately from the fix.

Hello, hope u are doing well
Is there eventually any update about this ? I am getting each time i do a ctrl+c a panic when calling shutdown like above stated
Regards

Yes, there's going to be a fix, I was working on it, but right now I don't have time to continue it.


@app.middleware("http")

@app.middleware("http")
I quote middleware to calculate the process time, while I find the requests per second (RPS) to be 60% of without middleware. That means 4000rps to around 2500rps. Is it right?
Could you please give some advice to improve the performance of middleware?
@app.middleware("http")
async def add_process_time_header(request: Request, call_next):
    start_timestamp = time.time()
    request.state.timestamp = start_timestamp
    response = await call_next(request)
    process_time = time.time() - start_timestamp
    response.headers["X-Process-Time"] = str(process_time)
    logger.info({"status_code": response.status_code,
                 "url": request.url,
                 "client": request.client.host,
                 "timestamp": start_timestamp,
                 "process_time": process_time})
    return response

It's a known issue on Starlette side.
When you create a middleware using the decorator @app.middleware('http') it creates a BaseHTTPMiddleware based middleware. Precisely the class that comes with the issue (https://github.com/encode/starlette/issues/919).
You should create a pure ASGI middleware. I'm going to recommend this middleware tho: TimingMiddleware.
The FastAPI @app.add_middleware is inherited from Starlette, so you should be able to do as: https://github.com/steinnes/timing-asgi#usage.
Note:
There's an issue on Starlette side to improve the documentation about middlewares. So, I think a PR will be gladly accepted by @tiangolo (those are my words, not his, jfyk) if someone wants to improve the documentation about it on Advanced Middleware section on FastAPI itself.

Thank you, Kludex！Maybe an wrapper will works well. Hope to hear good news soon.

Thanks for the help @Kludex! 🙇
If that solves your question, you can close the issue @littleforce163 ☕


Create codecov.yml

Disable annoying GitHub comments, allow for some % decline in codecov to pass status checks

bors r+


tauri-webpack is deprecated

read in your README - sorry, we should have deprecated that ages ago

Thanks for the tip :) I'll remove it.


chore(main): release 0.1.3

:robot: I have created a release beep boop
0.1.3 (2024-07-23)
Miscellaneous Chores

release 0.1.3 (65a0c70)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.3
:sunflower:


fix(datatable): Fix looking up nested keys in data for column

fix(datatable): Fix looking up nested keys in data for column

:+1:

:+1:


As preserve the sound in the video?

Thank you for this work. However, is there any way to keep the sound (the voice for exemple) in the generated video?

You can try to do this with ffmpeg
# Extract sound from driving video to 'sound.aac'
ffmpeg -i "assets/driving.mp4" -vn -acodec copy "./sound.aac"
# Composite 'sound.aac' into the generated video
ffmpeg -i "./generated.mp4" -i "./sound.aac" -c copy -map 0:v:0 -map 1:a:0 "new_generated.mp4"


[web] Flutter web using Cloud firestore

Is there anyway  for me to use cloud firestore in a flutter web project?

Duplicate of https://github.com/flutter/flutter/issues/32637
See discussion there!


Getting rate limited by OpenAI on ingestion

Getting the following error when ingesting my documentation:

openai.error.RateLimitError: Rate limit reached for default-global-with-image-limits in organization org-Wge... on requests per min. Limit: 60.000000 / min. Current: 70.000000 / min. Contact<EMAIL_ADDRESS>if you continue to have issues. Please add a payment method to your account to increase your rate limit. Visit https://platform.openai.com/account/billing to add a payment method.

Additional context:
Number of Tokens = 538,902
I can't find any way to specify a backoff. I have a paid account, and apparently the rate limit is raised over time. However, I would like to be able to specify a rate limit.

Think that vectorising data in batches will help
Check this issue out #54

chunk_size is how much text a single request, turn it smaller only will make it worse,
problem should site in the request side

Solved in #54 Will help avoid rate limits and save progress if llm provider has issues

@dartpain probably not fix, see https://github.com/arc53/DocsGPT/pull/85

I use some method that seems too tricky: add some sleep in this function https://github.com/hwchase17/langchain/blob/b7747017d72eaeabfa65edb7eec413d3fd006ddb/langchain/embeddings/openai.py#LL72-L76C87

Another workaround is to use retry: #91 , it works pretty well : )


Add tags as columns in containerImageChargeback

Same as #10940 but for Container images
This adds tags as available fields\columns in Container image chargeback reports

@gtanzillo Please review
cc @simon3z
@miq-bot add_label providers/containers, chargeback

This is an issue in the field, please consider this for Euwe.

@mikedahlgren @zeari do we have a BZ marked for Euwe for this?

@mikedahlgren @zeari do we have a BZ marked for Euwe for this?

@simon3z I dont think so...

I was under the impression that this was needed to resolve this bug:
https://bugzilla.redhat.com/show_bug.cgi?id=1354029
Which is under Q/A right now for Euwe.

@zeari @simon3z I was under the impression that this was needed to resolve the bug:
https://bugzilla.redhat.com/show_bug.cgi?id=1354029
Which is under Q/A right now for Euwe.

@mikedahlgren no, as the title states it's about Chargeback for Container Images

@zeari are you planning on re-opening this or go another path?

@mikedahlgren This commit was merged in https://github.com/ManageIQ/manageiq/pull/12851


error on creating project out of swagger spec

I go to http://editor.swagger.io and load the default example file. Then I download both spec.json and spec.yaml. When I try to create a new Project in SoapUI Pro by using the downloaded spec.json as Swagger Definition the import fails with
Error java.lang.RuntimeException: Unknown Swagger Version: null

When I try to import spec.yaml instead, the import fails with
Error java.lang.NullPointerException: Cannot get property 'basePath' on null object

The swagger spec is version 2.0. I'm using SoapUI Pro 5.1.2 with soapui-swagger-plugin-0.3.1-plugin-dist installed.

Same problem here (basepath) ... with the latest version of the plugin directly downloaded from Ready! API.

I am also facing the same issue with ReadyAPI1.2.1 any help much appreciated
Thanks in Advance

@olensmar I am using the v2.0 and ready API 1.2.1 could you please help me
Thanks in Advance

thanks guys - an update of the plugin is in the making - if any one of you could share your swagger specs with me so I can make sure they work that would be grand - please send to<EMAIL_ADDRESS>if possible!
/Ole

@olensmar Thank You very much currently we have local instance which you can't access
if you fixed please let me know i will check and let you know
Thanks in Advance
karthik
I will mail you as well

The sample specs @ http://editor.swagger.io/#/ fails to load (I tried others, same error).
Thanks !
(P.S. working for a multinational bank right now and about to include SoapUI as a main testing component)

thanks for your input @pvdyck

@pvdyck I've sent you an update of the plugin - please let me know how it works so I can either fix or publish. Thanks all!

ok - the plugin seems to work now - anyone else want to give it a try before we publish it?
Thanks in advance
/Ole

@olensmar YGM. Included an example I'm working on right now. Same errors when using the Petstore examples. Willing to test the plugin, let me know.
great work,
cheers,
Martijn

Guys - the new version of the plugin (2.1) is live in the plugin repository - if you have any issues please let me know! Thanks for your help!
/Ole

@olensmar I just tried, i'm not able to load a 2.0 spec.
The spec is correct, i checked in http://editor.swagger.io/

darn!
can you share the spec?

@olensmar this is it.
http://pastie.org/private/kzvoxwn2eh5xr4anfdth1a

@olensmar any development on this issue, how can I help?

not yet - i'll fix it over the weekend - sorry to keep you waiting!

Thank you!
On Fri, Apr 17, 2015 at 12:09 PM, Ole Lensmar<EMAIL_ADDRESS>wrote:

not yet - i'll fix it over the weekend - sorry to keep you waiting!
—
Reply to this email directly or view it on GitHub
https://github.com/olensmar/soapui-swagger-plugin/issues/3#issuecomment-94028926
.

--
Sebastián Ortiz Vásquez.
2015

ok - i've dug in - the cause of this is that the Swagger parsers internally only support string values for the "default" property in your definitions - you have
...
"billingAddress": {
"default": null,
"$ref": "#/definitions/BillingAddress"
}
...
and later on
...
"installments": {
"type": "number",
"description": "The amount of payments to divide the charge amount.",
"minimum": 1,
"maximum": 36,
"default": 1
},
...
unfortunately the swagger parser doesn't currently support anything but string values for default (see https://github.com/swagger-api/swagger-spec/issues/229 regarding null). If you remove default or give it a string value your definition loads ok. I'll follow up with the swagger team to see if I've understood this correctly.
/Ole


Add repo.altSource to provide alternative source to fetch the helm chart from if repo.source failed

Signed-off-by: Mike Ng<EMAIL_ADDRESS>

It looks good to me, the AltSource is optional. so backward compatibility should be fine

/test test-unit

/lgtm


Docker support

Updated config.example.ts to support environment variable configuration (used in docker-compose).
How to use:

ensure you have Docker desktop installed on your machine;
navigate to project root;
run command docker-compose up (to run as a background process use docker-compose up -d)
wait until service boots up and check http://localhost:3030

Not sure if it should be MySQL 5.7 or MySQL 8. This also worked with MariaDB 10.4.17.

This is awesome! Though, I think we should also add phpmyadmin so admins/owners can change stuff when needed.

This is awesome! Though, I think we should also add phpmyadmin so admins/owners can change stuff when needed.

Added phpmyadmin to docker-compose. Open on localhost:8080 to test it out.

This is the only error I'm getting:

I didn't change anything, for the rest it's all good.

I think your db host is defined as <IP_ADDRESS> or localhost, thus it's not able to connect. Check the introduced changes in config.example.ts and modify your config.ts accordingly (it adds environment variable support). If you check docker-compose.yml you can see that DB_HOST is overwritten to use database.

Check the introduced changes in config.example.ts and modify your config.ts accordingly

That's my bad lmao. Forgot about this

Alright, All seems to be working! I'll merge this in and update the guides. Thanks a lot!

Cool! In case of updating artifact I think docker compose down --rmi=local to docker-compose up should suffice.

Alright, this is what I've got the docs, anything that would need to be changed?

I think it's fine.

Alright, awesome. I'll merge this into main too so others can start using it if they want to. Thanks a lot for this PR 🎉!


ansible: devtoolset-4: Split installation into two steps

Split up to allow installation in two steps.

Same problem:
PLAY [cowfish.openmicroscopy.org] **********************************************************************************************************************************************************************************************************************************************

TASK [Gathering Facts] *********************************************************************************************************************************************************************************************************************************************************
ok: [cowfish.openmicroscopy.org]

TASK [devtoolset-4 : devtoolset-4 | Enable SCL repository] *********************************************************************************************************************************************************************************************************************
changed: [cowfish.openmicroscopy.org]

TASK [devtoolset-4 : devtoolset-4 | Install devtoolset-4] **********************************************************************************************************************************************************************************************************************
fatal: [cowfish.openmicroscopy.org]: FAILED! => {"changed": false, "failed": true, "msg": "No package matching 'devtoolset-4' found available, installed or updated", "rc": 126, "results": ["No package matching 'devtoolset-4' found available, installed or updated"]}

PLAY RECAP *********************************************************************************************************************************************************************************************************************************************************************
cowfish.openmicroscopy.org : ok=2    changed=1    unreachable=0    failed=1

@joshmoore Thanks.  As suspected, the original task was file and this is functionally equivalent.
I suspect there's something about this host which needs tweaking independently; may be related to spacewalk?  Or a yum config change on the node iteself?  We probably made the same tweak to gretzky35, but I can't see any record of what that change was on the gdoc for gretzky35 or on trello.
@kennethgillen Any ideas what we might need to do here?

@joshmoore Would it be possible to retry with -vvvv… to see if it reports anything informative?

TASK [devtoolset-4 : devtoolset-4 | Enable SCL repository]
changed: [cowfish.openmicroscopy.org] => {
    "changed": true,
    "changes": {
        "installed": [
            "centos-release-scl"
        ],
        "updated": []
    },
    "invocation": {
        "module_args": {
            "conf_file": null,
            "disable_gpg_check": false,
            "disablerepo": null,
            "enablerepo": null,
            "exclude": null,
            "install_repoquery": true,
            "installroot": "/",
            "list": null,
            "name": [
                "centos-release-scl"
            ],
            "skip_broken": false,
            "state": "latest",
            "update_cache": false,
            "validate_certs": true
        }
    },
    "msg": " Warning: Due to potential bad behaviour with rhnplugin and certificates, used slower repoquery calls instead of Yum API.",
    "rc": 0,
    "results": []
}

@rleigh-codelibre can use infra-testpr to debug why this doesn't work on LSC-standard VM images.


update trigger for imagetest

paths parameter takes globs, not regex
build-container-image task assumes input is named guest-test-infra, add mappings

/lgtm


Connection issue with custom sidebar

Hi, I'm experiencing an issue with the custom sidebar. If I look at the console in Chrome, I see that the custom sidebar is the reason why I cannot log in remotely. I also have difficulty sometimes on my Android device, but this only happens when I'm accessing remotely. On my PC, I do manage to get in sometimes remotly, but most of the time, I'm not able to log in. I delete the cookies and reset all permissions, and then I manage to get in, but not every time. This is a bit random but very frustrating most times.
custom-sidebar-yaml.js?v4.6.0:1 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'addEventListener')
custom-sidebar-yaml.js?v4.6.0:1 Uncaught (in promise) TypeError: Cannot read properties of null (reading '__route')
custom-sidebar-yaml.js?v4.6.0:1 Uncaught (in promise) TypeError: Cannot read properties of null (reading 'querySelector')
custom-sidebar-yaml.js?v4.6.0:1 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'config')

Hi @freddiemare,
I also access remotely to my Home Assistant instance on my PC and Android device and I don't have any issues, so I suspect that there is something different in your setup. But before going there, let's do something. Uninstall comletely custom-sidebar and check that you can access correctly your devices remotely, this is a JavaScript plugin and if it has error it should affect the functionality of the plugin itself but it should not impeed you to work with your Home Assistant instance.
Do this test first and we can continue later on to see if we can spot where is the issue.
Regards

Hi @elchininet, thank you so much for your kind message I do appreciate it, and you are absolutely correct I will do a test by uninstalling custom-sidebar and give you feedback I will also check my setup again.
Kind regards
Freddie

Hi @freddiemare,
Have you been able to test your instance with the plugin uninstalled?
Regards

Hi @freddiemare,
Closing this issue. Please, open a new one if you still have issues.
Regards


quality filtering of paired end reads

I ran
qiime quality-filter q-score --i-demux artifacts/samples.qza --output-dir quality-filtered.qza
samples.qza has two paired end samples

S1_R1.fastq
S1_R2.fastq
S2_R1.fastq
S2_R2.fastq

the quality-filtered.qza file has 2 files only
S1_R1.fastq
S2_R1.fastq
I expected four files.
How can I solve this problem

Please post user support questions to the QIIME 2 Forum. Thanks!


Chores: Reduce Code Duplication

There are some cases of unnecessary Code Duplication, i marked few that i made with @code-duplication in a comment.

Note many of them are in the branch #359


Can't rename files and folders that are already selected

Godot version
v4.1.2.stable.mono.official [399c9dc39]
System information
Kubuntu 22.04
Issue description
Since Godot 4.0 beta there has been a bug where I can't rename files that are already selected.
I thought it was a basic bug and it would be fixed but turns out I got used to it and it still exists.
Steps to reproduce

Select a file or folder in the file system tab
Right click
Try to rename

Minimal reproduction project
This has nothing to do with godot projects, it happens in all of them.

I reproduced this in 4.1, but only on the second try, after I renamed the file/folder and tried it again without switching the selection. In master the bug is not reproducible for me. Please test this in 4.2 beta 4 and let me know if the bug is fixed.

it's not reproducible for me in 4.2.

Thanks, closing as fixed in 4.2.


Some bad ts performance cases

Steps to reproduce
I write many details in the readme of https://github.com/tjx666/ts-perf-issue
Current behavior
The cases described in https://github.com/tjx666/ts-perf-issue/tree/main still bad ts performance
Expected behavior
Improve these cases performance.
Context
https://github.com/tjx666/ts-perf-issue
Your environment

npx @mui/envinfo
 System:
    OS: macOS 14.5
  Binaries:
    Node: 20.15.0 - ~/.local/state/fnm_multishells/22122_1719478157230/bin/node
    npm: 10.7.0 - ~/.local/state/fnm_multishells/22122_1719478157230/bin/npm
    pnpm: 9.4.0 - ~/.local/state/fnm_multishells/22122_1719478157230/bin/pnpm
  Browsers:
    Chrome: 126.0.6478.127
    Edge: Not Found
    Safari: 17.5
  npmPackages:
    @mui/lab: 5.0.0-alpha.170 => 5.0.0-alpha.170
    @mui/material: ^5.15.20 => 5.15.20
    @types/react: ^18.3.3 => 18.3.3
    react: ^18.3.1 => 18.3.1
    typescript: ^5.5.2 => 5.5.2

Search keywords: typescript, suggestion, slow, bad, performace, sxProps, createTheme, @mui/system

Thank you so much for the detail explanations!

I just discovered that in my case, i had performance issues completely disappear when removing the @mui/icons-material package.
Then i tried downgrading to an older version and that seem to have improved my performance by a lot.
Version 5.11.11 seem to be much more stable in performance, but it requires the use of the --legacy-peer-deps flag.
"@mui/icons-material": "^5.11.11"
I noted the issues happening here. (I have not tried other versions)
"@mui/icons-material": "^6.1.4"
My Package.json
  "dependencies": {
    "@emotion/cache": "^11.13.1",
    "@emotion/react": "^11.13.3",
    "@emotion/styled": "^11.13.0",
    "@fontsource/roboto": "^5.1.0",
    "@mdx-js/loader": "^3.0.1",
    "@mdx-js/react": "^3.0.1",
    "@mui/icons-material": "^5.11.11",
    "@mui/material": "^6.1.4",
    "@mui/material-nextjs": "^6.1.4",
    "@next/mdx": "^14.2.15",
    "gray-matter": "^4.0.3",
    "mdast-util-to-string": "^4.0.0",
    "next": "14.2.15",
    "react": "^18",
    "react-dom": "^18",
    "rehype-autolink-headings": "^7.1.0",
    "rehype-slug": "^6.0.0",
    "remark": "^15.0.1",
    "remark-footnotes": "^5.0.0",
    "remark-frontmatter": "^5.0.0",
    "remark-gfm": "^4.0.0",
    "remark-mdx-images": "^3.0.0",
    "unist-util-visit": "^5.0.0"
  },
  "devDependencies": {
    "@types/mdx": "^2.0.13",
    "@types/react": "^18",
    "@types/react-dom": "^18",
    "eslint": "^8",
    "eslint-config-next": "14.2.15",
    "typescript": "^5"
  }

I started seeing degraded code completion performance from 6.1.0, downgrading to 6.0.2 fixed the issue.


where is the key_stats.csv?

hi, thanks for your great job, very grateful, but I wonder where the key_stats.csv file goes?

Sorry, which file are you trying to run?

hello Vaibhav, thanks for your response, the file I want to run is Regression/linear-svc-machine-learning.py

I saw your code, the key_stats.csv may contains many factors like PE,PB,PEG and so on

I actually had the path set to my backup hard drive as the key_stats.csv file I had was very large. If you open up the file, just change the path variable to wherever you have the file saved.
You can download it from here: https://github.com/sbsar6/python-data-mining/blob/master/key_stats.csv
~Merry Christmas & Happy Holidays ~

Thank you so much for your great help, and also thanks for your selfless dedication about the code, the data file like a Christmas gift. This is a blessing from a Chinese friend: Merry Christmas and happy new year

LOL, thank you for your kind words! Happy new year!


Tensorflow 1.6

With new tensorflow(1.6) it looks like the histogram values that I recorded acts weird.
1.5

1.6

Does https://github.com/lanpa/tensorboard-pytorch/wiki work?

of course; didn't know about the wiki before


ValueError: could not convert string to float

Hello,
Thank you for putting this code.
I try a lot of times , however I always got the errors like:
Traceback_* (most recent call last):
File "qiime_to_lefse.py", line 126, in
main()
File "qiime_to_lefse.py", line 122, in main
append_otu_info(taxafile, outfile, sample_dict, categories, clean)
File "qiime_to_lefse.py", line 53, in append_otu_info
abundance = [float(a) for a in abundance.split('\t')]
ValueError: could not convert string to float: O1
I'm confused by this. Can you explain a little bit about this? thx a lot
Regards,
yujia

Hi @atease1506
The error message you see is from the script is trying to convert the numbers in otu-table from text-format (a string) into a float-format (a number with a decimal)
The otu table should be a separated text file that looks something like this
Taxon   Sample1 Sample2 ...
None;Other;Other        0.0     0.0     0.0 ...
k__Bacteria;Other;Other 0.000885822182852       0.00125591582301        0.000211148648649 ...
...

So lets look at the following code snippet where we process the file and where your error is coming from
taxa, abundance = line.rstrip('\n').split('\t', 1)
abundance = [float(a) for a in abundance.split('\t')]

The first line of code is going to take taxa line (all lines but the first since those are the sample names) and split it into two parts, the taxa-classification (a variable named taxa) and a list of abundances (a variable named abundance)
The next step of the code tries to take the list of abundance values (which are in text format right now) and convert them all to float values. It should be able to do that since everything after the taxa-classification is the abundance value for each sample. In your case it ran into something other than an abundance or possibly a malformed number.
You error message says it couldn't convert the string O1 to float.  Since the letter O is not a number, the conversion failed.
I hope that this helped explain the error. Now lets see if we can fix it!

Have you manipulated any of the text in the otu-table by hand? Should that be 01 instead? That is the number zero, instead of the letter 'O'?
Does the otu-table that you are inputing look similar to the example I have pasted above?

If thats not the case, I may need to see a snippet of it to help me solve it
Best,
Gene Blanchard


Update troubleshooting-required-status-checks.md

remove extra space

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

@mic-max Thanks so much for opening a PR! I'll get this triaged for review :zap:


How to fix - Attempt to de-reference a null object - While trying to Import Campaign Members

Hello,
I'm trying to import campaign members via npsp data import object, but keep running into the Attempt to de-reference a null object.

Hi @DariusDixon We do not provide support via GitHub issues. Please could you log a support case for this. In the case creation form if you select the following it should ensure it gets assigned to the correct case queue

Product: NonProfit, Education and Power of Us
Topic: Non Profit Cloud


Short solution needed: "How to set variable in Lua" (nginx-lua)

Please help us write most modern and shortest code solution for this issue:
How to set variable in Lua (technology: nginx-lua)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to set Nginx variable


Add ABI Generation for KV Tables

Change Description

This change introduces abi generation for KV tables as well as a host of refactorings to abigen and codegen in general.
API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Note: This is currently targeted at the kv-wrapper-mega-merge branch so the diff looks correct. However, it should really just be merged to develop once all of the underlying KV work is merged.


!!/approve on github sometimes doesn't

What problem has occurred? What issues has it caused?
I added an !!/approve comment on several open PRs here on GitHub. Still PullApprove says "Pending: approval required by one of ...."
I deleted the failed comments and proceeded to approve from chat instead:
#2963 #2957 #2955 #2952
Weirdly, #2951 looks like it was successfully approved, but still not merged. I will proceed to merge that manually.
What would you like to happen/not happen?
Approving on Github should work just like approving from chat.

I agree that it would be nice if a !!/approve comment on GitHub did the same job of approving and merging, with resolving merge conflicts, that a !!/approve PR# does in chat. However, I've never seen it work fully the same (i.e. I've never seen it resolve the merge conflicts we typically get when another !!/watch or !!/approve has been done after the PR was created). It used to automatically merge when there were no conflicts. However, I haven't seen it do that in quite some time.

If I remember correctly, before !!/approve on GitHub broke, it was using a merge strategy which automatically resolves most of the normal merge conflicts that we get (as a result single-line additions). The word onion rings a bell, but I feel like that's not right (a quick search of the git docs doesn't show up any results. Octopus maybe?)

When !!/approveing in GitHub by group 'code-admins' as set in .pullapprove.yml, then, a check named 'code-review/pullapprove' will be marked as 'passed'. However, the PR is not merged automatically (cc @makyen @angussidney), so it isn't broken at all.
It was suggested to trigger an automatic merge if it had a certain number of approving reviews, but unfortunately rejected. It was issue pullapprove/support#9. It was mentioned that a bot named automerge could do that (GH repo) after adding an 'automerge' tag. However, I don't really like this idea.
I also found https://github.com/dgmltn/EggTimer from this Stack Overflow answer which may help, I haven't tested it, though.

@double-beep metasmoke should automatically merge PRs once they're approved by PullApprove. Sounds like there's a bug there somewhere.

I have a feeling that a past PR (from me) that removed the creation of .git/attributes on MS side might be the cause, but my Git-fu told me this .gitattributes file should always work.


feat: implement support for more well-known-types

google.protobuf.Struct (mapped to a JSON STRING)
google.type.TimeOfDay (mapped to TIME)
google.type.Date (mapped to DATE)
google.type.LatLng (mapped to GEOGRAPHY)

The reason for using GEOGRAPHY as default for LatLng is because of the
clustering opportunities for tables that store locations as GEOGRAPHY.
Future SchemaOptions may override to store LatLng as a record instead.
Integration tested against public BigQuery datasets. The intention is to
identify datasets that exercise the features of this library, then
perform a load/marshal/unmarshal sequence and validate that the original
result is preserved.

Why not map proto struct to a BigQuery struct as well (I think they call it a record in Go)?

Why not map proto struct to a BigQuery struct as well (I think they call it a record in Go)?

@alethenorio google.protobuf.Struct is an untyped struct without a schema - can't be mapped 1:1 with a BigQuery RECORD.

@alethenorio google.protobuf.Struct is an untyped struct without a schema - can't be mapped 1:1 with a BigQuery RECORD.

Ah I see, it's just a map. 👍

Ah I see, it's just a map. 👍

@alethenorio Thanks! :raised_hands:

@alethenorio Thanks! :raised_hands:


Developing with Slate results in a 503 timeout error on website

I can't be certain that it's Slate's pushing that's causing it but the past few days whilst developing with Slate, I've occasionally been given a 503 error page with the following error message:

This page is temporarily unavailable because a device from your location is sending large amounts of web requests. Visitors from other locations can still view the page.

Screenshots attached.

@ConduciveMammal is this occurring when you push to Github and then your Github tool pushes the latest version to your test store? I suspect that tool might be pushing files faster than our servers allow.
Otherwise, are you still experiencing this bug? Could you please provide more details on how to replicate it?

Nah, this was just using default Slate to push directly to the store without GH.
Oddly enough, I've not actually seen this happen for a while, so perhaps there was just a glitch with Shopify’s servers?

Cool. Let us know if you see it again and we'll take another look!

I've gotten this error several times tonight while using slate v1.0.

I'm getting this error after updating to the latest package versions.


Implement solving separable problems in parallel

If problem is separable, we split it into smaller problems and solve them in parallel.
To enable the option, run problem.solve(parallel=True).
I cache the separable problems (as a list) by decorating problem._separable_problems with lazyprop. I'm still not sure how I should detect if the objective or constraint has changed to recompute these separable problems.

This is really excellent code! Sorry for not getting to it sooner.
You should make sure the cache is invalidated if the user changes the objective or constraints. This is done currently in problems/solvers/solver.py in the validate_cache function.
You could factor some version of this function out into solvers/utilities and reuse it.

Sorry for taking so long to fix this PR. I've added code to check if the problem has changed before solving in parallel. However, I didn't use the validate_cache function since it only works if the cache is of type SymData. I just keep a cache for the parallel solver consisting of the objective and constraints (as LinOps).
I ran into some problem with CVXcanon version 0.0.21, but upgrading to version 0.0.22 fixes it. So I'm also adding the requirement of CVXcanon >= 0.0.22.
After putting get_separable_problems into cvxpy.transforms, I ran into circular imports. Problem needs to import cvxpy.transforms, and cvxpy.transforms needs to import Problem. I resolved it by just importing cvxpy in problem.py. I'm not sure if this is the best way to solve it, but I couldn't think of anything else.

This is beautiful code! Committing now, and sorry for the delay.


save: fsync data safely to disk

On modern Linux systems, it can take 30 seconds for the data to actually hit the disk (check
/proc/sys/vm/dirty_expire_centisecs).
If the computer crashes in those 30 seconds, the user may end up with an empty file as seen here:
https://github.com/neovim/neovim/issues/9888
This is why editors like vim and nano call
the fsync syscall after they wrote the file.
This syscall is available as file.Sync() in Go.
Running strace against micro shows that fsync is
called as expected:
$ strace -f -p $(pgrep micro) -e fsync
strace: Process 3284344 attached with 9 threads
[pid 3284351] fsync(8)                  = 0

Also, we now catch errors returned from w.Flush().

Maybe we should have this functionality guarded behind an option called fsync which is off by default? That seems to be the behavior vim/neovim have as well. I don't think we should be calling fsync after every save by default, since that will hurt performance and is unnecessary in the common case.

Hi, no, vim has fsync on by default.
How about running the fsync in the background?

Hi, no, vim has fsync on by default.

Indeed.
However I don't think we should just follow that in micro without considering the consequences. It's a question of tradeoff between safety and speed, both are important (and perhaps not just speed but also things like SSD longevity).
How many other editors do fsync by default?

How about running the fsync in the background?

That would defeat the purpose of fsync (which stands for "synchronous"), wouldn't it?

I tested three more editors, so now we have (use of fsync verified by strace):

vim: fsync
nano: fsync
sublime text: fsync
gedit: fsync
emacs: fsync
neovim: nothing
micro: nothing

Looks like the newcomers (neovim and micro) have a different attitude to user data.

That would defeat the purpose of fsync (which stands for "synchronous"), wouldn't it?

No. fsync tells the kernel "get this data to disk asap". But that does not mean that the UI has to be blocked.

Thanks for checking.
Found the change in neovim which made fsync disabled by default: https://github.com/neovim/neovim/pull/8304

Okay, seems reasonable to call fsync in the background. Another option is to only call fsync on backups (which already run in the background). Seems like the best approach might be to call fsync on both though (with fsync running in the background for the normal buffer).


stable_softmax

Hi - I assume by the name stable_softmax is more stable than regular softmax. less inf or nan? if so, is this generally applicable to other libs? i've seen people convert to torch32 for more stability than convert back to original dtype.

@ontocord oh, that was actually because i had a misconception about pytorch's softmax (did not know they subtract out the max)
removed it for clarity!
and yes, you are right that some people like to do everything in float32. as we move towards bfloat16, this will become a non-issue

Awesome! I am testing the memorizing transformer code now. I'd like to incorporate some of it into the longllama code since they didn't implement the knn stuff.

nice! i prepared the tanh gating for that exact reason 😄 good luck!


Linux arm32 build only has the JDK

The Linux arm32 builds only have the JDK. Should there and/or will there be a Linux arm32 JRE build.

Hi @jkinross . We are now producing Arm32 JRE's for JDK8 and JDK11+.

Hi @jkinross . We are now producing Arm32 JRE's for JDK8 and JDK11+.

Closing


make compatible with React 16 by upgrading dependency react-motion

react-motion already has a fix for the issue, but react-scrollbar installs an outdated version of react-motion

Thank you @mallorybulkley :)


Invalid URLs in Railsbytes output

Congrats on the new gem!
I noticed an issue with the URLs in the Railsbytes output:
A simple "hello world" example of using Rails app templates
┃ x7msKX - https://railsbytes.com/api/v1/accounts/2/templates/x7msKX

That URL results in a 401 (I expect it's only intended for API requests). It's valid if changed to https://railsbytes.com/templates/x7msKX
I checked the feed and it doesn't seem to currently contain a 'public' URL in the response so I expect that will need to be added on the server-side. I'm happy to open a PR to update the client once that's available.

Yep, good catch. 👍
Fixed!

btw it's now returning https://railsbytes.com/public/templates/x7msKX.json rather than https://railsbytes.com/public/templates/x7msKX.

Whoops done.
I want to make that command interactive so you can navigate pages and select a specific template from the returned list (rather than copying the ID).


error: Failed to prepare distributions, Caused by: Failed to fetch wheel: sentence-transformers==2.2.2

I'm currently setting up a new project using uv following using
uv init
uv version = 0.4.5
python version = 3.11
windows 11
When trying to add graphrag
uv add graphrag
I get the following error:
$ uv add graphrag
Resolved 242 packages in 785ms
error: Failed to prepare distributions
Caused by: Failed to fetch wheel: sentence-transformers==2.2.2
Caused by: Build backend failed to build wheel through build_wheel() with exit code: 1
Some further information printed out:
********************************************************************************
Usage of dash-separated 'description-file' will not be supported in future
versions. Please use the underscore name 'description_file' instead.
    By 2024-Sep-26, you need to update your project and remove deprecated calls
    or your builds will no longer be supported.

    See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
    ********************************************************************************

!!
opt = self.warn_dash_deprecation(opt, section)
error: could not create 'build\bdist.win-amd64\wheel.\sentence_transformers\cross_encoder\evaluation\CEBinaryAccuracyEvaluator.py': No such file or directory
Any ideas on the issue and how I can resolve it?

Are you able to pip install sentence-transformers==2.2.2 --use-pep517? Or does that also show an error?

I was able to use pip install to get it working and it would appear in the uv.lock
Since then everytime I use uv add for another depenency the same error will come up as before, as it tries to add sentence-transformers to the dependency of the pyproject.toml?

I'm also having a similar issue with many other packages. Anything else you can suggest here @charliermarsh?

Can you elaborate on the issue you're seeing? The commands you're running, the error you're seeing, how I might reproduce it, etc.

I solved my issue. Sorry for the delayed reply. I didn't want to hijack the thread, as it was a slightly different issue.


feat(badge): add HTTP/3

Pull Request Test Coverage Report for Build 1692

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

Totals

Change from base Build 1691:
0.0%

Covered Lines:
101

Relevant Lines:
152

💛  - Coveralls


Implement node crypto HKDF, except for KeyObject inputs

Corrected the copyright messages, also switched to a different approach to do HKDF that works with older bssl versions. It turns out that this approach is also much simpler as bssl's HKDF functions handles edge cases that OpenSSL does not, which allows us to greatly simplify the C++ code ☺
@jasnell Feel free to take another look, although the fixes should be pretty straightforward.


Piskel Background Transparency Not Working

I import a picture with a transparent background, animate it in Piskel, export it and the transparent background turns white.
This is my first time experiencing this and I've used Piskel multiple times prior with no issue.
Here's the animation: http://www.piskelapp.com/p/agxzfnBpc2tlbC1hcHByEwsSBlBpc2tlbBiAgID22vqjCww/view
But when I downloaded it and uploaded it to another site (deviantart), the background turns white: http://sta.sh/01kks0nvt9v4
I need to get this commission fixed soon. I hope this can be resolved.
Thanks!
-Chris

I reproduced the issue. In the meantime, you could use GIMP to edit the gif to get output like the following:

Thanks for taking a look @epicabsol !
This sprite has ~1400 different colors, and I had to disable transparency in GIF exports for sprites that have more than 256 colors. (see issues #357 and #358)
The only workaround at the moment is to do what @epicabsol suggested.
I think we have a few options to improve this though:

display an error message when exporting a sprite with transparent pixels & more than 256 colors (with link to a wikipage explaining the issue, workarounds etc...)
let the user choose to keep transparency, even if it might screw up the output (with a checkbox in the export controller?)
perform a color reduction on our side before sending the sprite to gif.js, and submit 256 colors (including the transparent color)

Thanks so much! :'D The quick fix and immediate responses are really great. ♥


Microphone not work in background mode at some devices.

Hi @pedroSG94
I have a problem with the microphone. In background, the app was muted on some devices.
And this is my code. I set it in ScreenService
Please help me. Thanks!!!
fun handleMicrophone() {
        val microphoneSource = (genericStream.audioSource as MicrophoneSource)
        if (isMute) {
            microphoneSource.mute()
        } else {
            microphoneSource.unMute()
        }
// Update UI
    }

Hello,
If you can't get audio from microphone in background check that your service has this permission:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/AndroidManifest.xml#L64
In this case you need add microphone permission like:
        android:foregroundServiceType="mediaProjection|microphone"

Hi @pedroSG94
If I want to mute only the microphone while still recording audio from the app (like game audio), how can I do that? When I mute the microphone, it also mutes the app audio.

Hello,
If you are using audio and you can hear app audio that means that microphone capture audio from speakers. That is the reason about your case.
You can change to use directly audio from the app instead of mute microphone. For this you need use GenericStream to stream or any class that extends from StreamBase (screen stream example with GenericStream):
//I think that you can reuse mediaprojection from ScreenSource but you should test it
genericStream.changeAudioSource(InternalAudioSource(mediaProjection))

And instead of unmute you should return to Microphone source:
genericStream.changeAudioSource(MicrophoneSource())

Hi @pedroSG94 ,
I'm encountering an issue where the code crashes after several calls to the handleMicrophone function. I was wondering if you could take a look when you have a moment.
 fun handleMicrophone(isMute: Boolean) {
        if (isMute) {
            mediaProjection?.let {
                if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
                     genericStream.changeAudioSource(InternalAudioSource(it))
                }
            }
        } else {
            genericStream.changeAudioSource(MicrophoneSource())
        }
    }

Hello,
I will check it.
Are you re using mediaprojection when you call to that method? Or are you using a new instance each time?
Try to use a new instace each time meanwhile I test it

Hi @pedroSG94 ,
We re-using mediaprojection. We will try your suggestions
Thanks

Hi @pedroSG94 ,
We have tried both cases but it still crash after several calls. Could you please help us check it out?
Thanks so much.

Hello,
I fixed the error.
Now, you should re-use mediaprojection class.
This error is because no matter if you request a second instance of mediaprojection. If you have a mediaprojection working (in this case from ScreenSource), the mediaprojection is same instance until you stop all sources that depend of that mediaprojection. As proof, I never see a modal when I request the second mediaprojection.
I think that share this mediaprojection and call stop while other source is using it produce this crash. So I think that mediaprojection should be stopped after stop all sources that depend of it.

Fix uploaded to version 2.4.8. Update the library to solve it.

Hi @pedroSG94
Thanks for quick response, we are really appreciate it.
After updating to version 2.4.8, the recording works fine the first time. However, when we stop and then start recording again, the app crashes. Additionally, switching microphones is not working. Could you please assist us with this? I’ve attached a file for your reference.
Thank you so much.

Hello,
I'm not able to reproduce it in my app example.
I added this code to Service:
  var muted = false

  fun toggleMute() {
      if (muted) {
        genericStream.changeAudioSource(MicrophoneSource())
        muted = false
      } else {
        mediaProjection?.let {
          if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
            genericStream.changeAudioSource(InternalAudioSource(it))
            muted = true
          }
        }
      }
  }

And added a new button to the activity and test it:
    val test = findViewById<ImageView>(R.id.b_test)
    test.setOnClickListener {
      val service = ScreenService.INSTANCE
      service?.toggleMute()
    }

In xml:
  <ImageView
      android:src="@drawable/record_icon"
      android:layout_width="98dp"
      android:layout_height="98dp"
      android:id="@+id/b_test"
      app:layout_constraintBottom_toBottomOf="parent"
      app:layout_constraintStart_toStartOf="parent"
      android:layout_marginBottom="20dp"
      />

Can you tell me how to reproduce your case in my app example?

Ok, I detected the problem and fixed it. Use version 2.4.9
The only problem is that you will need stop mediaprojection class manually when you finish working with it. I added this feature to toggle audio sources to test it in the screen example.
Example of how to release mediaprojection class:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/screen/ScreenService.kt#L141
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/screen/ScreenService.kt#L148

Hi @pedroSG94 ,
Thank you very much for your help. We'll review it and let you know if it's working properly.
Best regards

It works. Thank you so much

Closing issue as completed


Field for decimals within DataObject are not constructing with the correct value

Field is a Numeric Field however is excluding the decimal value. The field should obviously display 24.73 but is only displaying 24
'Days'          => 'Decimal',

ss4

Assuming you're using NumericField you can ->setScale(2) to allow two decimal places when you scaffold your CMS field

I think decimal needs to scaffold the default field with a suitable scale.

Fix at https://github.com/silverstripe/silverstripe-framework/pull/7456

I'm seeing this issue again
SS 4.4.1

hmmm the $db is using 'Decimal' not 'DBDecimal' which is probably the cause.
Using Decimal in $db is correct. SilverStripe's Injector translates Decimal to DBDecimal when it creates an instance of the database field class. It may seem confusing, though.

So if you still have this issue, I think it needs more inspecting. :) Have you tried SS 4.5.1?


CPack tweakage

Couple changes to make CPack work on Fedora 14 (and I'd imagine Debian/Ubuntu too)
CPACK_PACKAGE_CONTACT (needed for .deb) also probably needs a valid email.
Patch:
http://www.pasteall.org/21173/diff

This has gone without comment for a long time and the link showing the patch no longer works, so I'm going to close.
I'm just going to cop to having dropped this one on the floor. If it's something that anyone cares about, please refile or better, submit a PR that makes the fix that this would involve.


How to locate the problem with the error stack?

I am using the obfuscated file now, and there is an error, but the reported stack information is after obfuscation, how can I find the location information in my original file? I found that have sourceMap, how can I achieve my needs, how can I use it, I read the document, but I am still very confused, can you give an example? Thank you very much！！

Hi. As a start point describe how you're using obfuscator (CLI, node js API, webpack plugin)

i use javascript-obfuscator source.js --output out.js --config obfuscator.json，And then out.js use on my website. but i use window.onerror accept the error message，but the reported stack information is after obfuscation how can I find the location information in my original file?

Using a source map on production anyone can get the original source code from this source map.
So basically, you just may add --source-map true --source-map-mode separate and the file with the source map will be generated. But as I mentioned before when you will serve this source map file, keep it unreachable (with help of server configuration) for all external users.

Maybe this is not what I want. I want to capture the user’s error message (using the onerror function to listen for Javascript errors), so I can’t restrict external users. Then I want to use this error message to know what is wrong with my code, but The current error message is after confusion

There are ways to map the error stack trace using the source map to its original positions, on NodeJS side, for example. But this is a different topic not related to obfuscator.

I will close this issue. If you have other questions feel free to create a new issue.

ok thanks!!!to find


set exactOptionalPropertyTypes: true, improved types

Description
Please include a summary of the change and which issue is fixed. Please also include relevant
motivation and context. List any dependencies that are required for this change.
Related https://github.com/ardatan/whatwg-node/issues/391

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take
action on it as appropriate
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can
reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Environment:

OS:
package-name:
NodeJS:

Checklist:

[ ] I have followed the
CONTRIBUTING doc and the
style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests and linter rules pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose
the solution you did and what alternatives you considered, etc...

@ardatan can you please follow up?

@ardatan can you please follow up this week?

@JustFly1984 I'll do when I am available

@ardatan resolved merge conflicts

@ardatan please follow up

@ardatan please follow up. I really do not want to disable exactOptionalPropertyTypes in the project I have to use fets.

@ardatan please follow up


Brochure: Re-structure cross-reference text for Resolutions, Recommendations,  etc. (French version)

In relation to https://github.com/metanorma/bipm-si-brochure/issues/189 and https://github.com/metanorma/bipm-si-brochure/issues/190
From BIPM:
Number | Page | Clause/paragraph | Comment |
-- | -- | -- | -- | --
24 | 58 | CIPM 1967 | "CIPM recommandation 2" should be replaced by "Recommandation 2 du CIPM" | general rule, to be applied systematically
For example, this markup:
[[cipm1967]]
[%unnumbered]
== CIPM, 1967 (((masse))) index-range:kilogramme_multiples[(((kilogramme,multiples et sous-multiples)))] index-range:multiples_kilogramme[(((multiples du kilogramme)))]

[[cipm1967r2]]
[%unnumbered]
=== {blank}

[.variant-title,type=quoted]
*Multiples et sous-multiples décimaux de l’unité de masse* (<<PV35_2>> et _Metrologia_, 1968, *4*, 45)

[[cipm1967r2r2]]
==== Recommandation 2

renders:

Here, CIPM recommandation 2 should be replaced by Recommandation 2 du CIPM
Same goes with CCTF Recommandation 2. It should be replaced by Recommandation 2 du CCTF.
In summary, these are the cases that we must address:

CIPM recommandation 2 should be Recommandation 2 du CIPM
CCTF Recommandation 2 should be Recommandation 2 du CCTF
CGPM Résolution 1 should be Résolution 1 de la CGPM
(CGPM Résolution xx = Résolution xx de la CGPM). Same for CIPM.
CGPM Réunion 9 should be 9e réunion de la CGPM
CIPM Réunion 51 should be 51e réunion du CIPM

This ticket is dependent on:

https://github.com/metanorma/bipm-si-brochure/issues/193
https://github.com/relaton/relaton-bipm/issues/34

I will take on the ticket, but it should just flow through as a result of the relaton updates.

@ronaldtse The wording of https://github.com/relaton/relaton-bipm/issues/34 indicates that "Recommandation 2 du CIPM" is not to be used, and "CIPM -- Recommandation 2 (1967)" is to be used instead. If that is the case, then both this issue and https://github.com/metanorma/bipm-si-brochure/issues/193 are completely misleading, since what they're requesting is out of date—and in fact, the rendering they will end up with is pretty much what there is now already.
So really, both this issue and https://github.com/metanorma/bipm-si-brochure/issues/193 are obsoleted by https://github.com/metanorma/metanorma-bipm/issues/216 , and should be closed.
@ronaldtse Kindly confirm.

This issue is pending on BIPM feedback here:

https://github.com/metanorma/bipm-si-brochure/pull/210

Please keep this on hold for now.

@opoudjis @andrew2net BIPM has approved the format documented in https://github.com/metanorma/bipm-si-brochure/pull/210 , which is now merged into this repository. It's a go!

The document to follow is as attached.
bipm-outcomes-en.pdf

Awaiting relaton work

Now awaiting https://github.com/relaton/relaton-bipm/issues/50

@ronaldtse As far as I can tell, the long French citation, Résolution 1 de la CGPM (1889), has not been implemented, and has not been provided for in the Brochure. I assume this is not a problem any more?

@andrew2net can you help answer this? We do need this reference. Thanks.

@andrew2net when using the language independent identifier:
$ bundle exec relaton fetch "CGPM RES 1 (1889)"
[relaton-bipm] ("CGPM RES 1 (1889)") fetching...
[relaton-bipm] ("CGPM RES 1 (1889)") not found.
No matching bibliographic entry found

These options are presumably to be realised as alternate docidentifiers, each with its own type. In order to permit the user to specify one docidentifier over another in a citation, I am adding the attribute style to erefs. (It is already realised for xrefs). The attribute value for style includes the docidentifier type to be used. It is not limited to that, and future implementations may realise more values.
(This of course is the kind of feature bloat that continues to make Metanorma unlearnable and unmanageable, and I am introducing this feature bloat because Metanorma policy is to provide flexibility to SDOs.)
So assume ref1 = CGPM RES 1, which has a bibitem that looks like:
....
<docidentifier type="BIPM" primary="true">CGPM Resolution (1889)</docidentifier>
<docidentifier type="BIPM" primary="true" language="fr" script="Latn">CGPM Résolution (1889)</docidentifier>
<docidentifier type="BIPM-long" primary="true" language="fr" script="Latn">Résolution 1ᵉ du CGPM (1889)</docidentifier>

Note that I'd rather not introduce formatting into docidentifier, such as:
<docidentifier type="BIPM-long" primary="true" language="fr" script="Latn">Résolution 1e du CGPM (1889)</docidentifier>

Then

<<ref1>> in a French document will automatically resolve to the primary (BIPM) identifier in the right language, CGPM Résolution (1889).
<<ref1,style=BIPM-long%>> will resolve instead to Résolution 1ᵉ du CGPM (1889)

It turns out that text citations use the long identifier (which we now call just BIPM for that reason); but the references listing uses the short identifier. I'm coming up with the rule:

In other flavours, BIPM documents are both cited and referenced with the BIPM identifier
In BIPM itself, documents are referenced (in references) with the BIPM identifier, but cited (in text) with the long identifier.

I'm not obeying that distinction outside of BIPM, outside of BIPM we will just use the one identifier consistently. I will realise BIPM long citations within BIPM documents as a code override (at least for now).

The precise long French identifier will be debugged. The citation with both short and long French identifier will be fixed when @andrew2net removes the "primary" tag from the long French identifier. This will result in citation by long, referencing by short identifier.


Enabling Sharing of the feature service fails if original is not in root folder of My Content

The enableSharing function produces an error and fails if the original feature service has been moved to a folder other than the root folder in My Content of ArcGIS Online.
If the feature service is moved back to the root folder and the script is re-run, it will complete successfully.
Regardless of which folder the feature service is in, the item will still be found by the script and the ID printed to the screen during execution.
"failed:"
{u'error': {u'messageCode': u'CONT_0005', u'message': u'Item does not exist in this folder.', u'code': 400, u'details': []}}

Is there any update on when this will be resolved?

See my pull request on the arcpy repository which fixes this.

Since posting I have found that solution, and it works...

Fixed by https://github.com/khibma/update-hosted-feature-service/pull/7


Character like companion profiles

Is your feature request related to a problem? Please describe.
I've got an Etheral Soul-Trader companion pet. This is a human sized humanoid, with humanesque intelligence. When creating a profile for this companion, it feels weird being forced to treat it as a pet. There are many fields I wish I could add for this companion, to make it appear as a person in its own right. Birth place, residence, family members, class, personality traits, a page with description and history, the same stuff as a character gets.
Describe the solution you'd like
An option to create profiles for companions that have all the same features as character profiles.
Describe alternatives you've considered
Written a restricted profile, and utilized the "At first glance" buttons.
Additional context
N/A

Sorry I forgot to reply to you before. Currently this isn't something we are looking to do. It is a lot of work required for a very specific use case, as most pets & mounts would still fall in the current category. Such information can still be manually written in the description part of the pet instead.


verification

I realized that I can't use this plugin, the captcha code is entered where to click on the verification

ok，已经解决了 往下拉

有问题随时联系


Provisioning Profile "com.company.app AppStore" doesn't include signing certificate iPhone...

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject

I'm having codesigning issues when using gym for an iOS app; in particular the codesigning_identity (Signing Certificate) is not being set by the build_app command.
Question Description

I expect cert and sigh to check certificates and provisioning profiles and then be able to set these profiles in my XCode project using build_app (gym), but the following Factfile returns the error. Code Signing Error: Provisioning profile "com.CompanyName.myCompanyName AppStore" doesn't include signing certificate "iPhone Distribution: Company Name (XZXZXZXZXZX)".
default_platform(:ios)

platform :ios do
  desc "Push a new beta build to TestFlight"
  lane :beta do |options|
	  build_number = increment_build_number(
		  build_number: latest_testflight_build_number + 1
	  )
	  version_number = increment_version_number(
		version_number: options[:version_number]
	  )
	  automatic_code_signing(use_automatic_signing: false, team_id: "XZXZXZXZXZX")

	  sh(
		  "echo ##teamcity[buildNumber '#{version_number}(#{build_number})']"
	  )
	  cert
	  uuid = sigh(
		force: true,
		filename: "CompanyName.mobileprovision"
	  )
	  update_project_provisioning(
		target_filter: "CompanyName",
		profile: "CompanyName.mobileprovision",
		build_configuration: options[:configuration]
	  )

	  build_app(
		  scheme: "CompanyName",
		  configuration: options[:configuration],
		  clean: true,
		  silent: true,
		  output_directory: "/usr/local/Repos/OLS/artifacts/",
		  output_name: "CompanyName.ipa",
		  export_method: "app-store",
		  codesigning_identity: "iPhone Distribution: Company Name (XZXZXZXZXZX)",
		  export_options: {
			  provisioningProfiles: {
				  "com.CompanyName.myCompanyName" => "com.CompanyName.myCompanyName AppStore",
				  "com.CompanyName.myCompanyName" => "com.CompanyName.myCompanyName Development"
			  }
		  }
	  )

	  if options[:pushToStore]
		  upload_to_testflight(
			  ipa: "/usr/local/Repos/OLS/artifacts/CompanyName.ipa",
		  )
	  end
  end
end

After running fastlane and recieving the error. I'm left with an XCode project that states:
Code Signing Error: Provisioning profile "com.CompanyName.myCompanyName AppStore" doesn't include signing certificate "iPhone Developer: Company Name (XYXYXYXYXYX)".

If I manually then reselect the Provisioing Profile for the "Signing (Release)" back to the same one that was set. The Signing Certificate then changes to the one specified iPhone Distribution: Company Name (XZXZXZXZXZX)
I'm aware this is a code signing issue, and maybe not a Fastlane issue, but any help is appriciated to narrow down this issue.
Thank you
Dean Meehan

Got the same problem, I can sign fine using Xcode directly, but not through fastlane

I actually got this sorted. Although it's possibly not a fault with XCODE and I'd love to sit down and write a detailed guide /procedure to fix all coffee signing issues.
I fixed it by:

Backing up all your certificates by exporting them
Delete all your iphone certificates
Try and use cert and sigh to generate them.
Restart the Mac
Re-add your certs 1 by 1 while trying build_app again and again.

Try them steps to see how far you get. If you get an error fffffffffffff, I just reverted from svn to a working status.
Good luck
On 11 Aug 2018 20:31, LasseSLambertsen<EMAIL_ADDRESS>wrote:
Got the same problem, I can sign fine using Xcode directly, but not through fastlane
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/fastlane/fastlane/issues/13079#issuecomment-412296826, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEwwjyryAUlGQwCOxhOGwlsaYz5pQqEvks5uPzEQgaJpZM4V38nV.

Managed to finally fix my problem. It was a permission problem, somehow I had corrupted some... Fixed it by running:
diskutil resetUserPermissions / id -u
In a terminal


How to reuse existing template literay when updating code in 'do action'

Is there any way to pass the dynamic expression e.g.
<Action
  do={() => {
     const newCode = ``;
     code.update`async function animate() {
       // New code here ...
       ${newCode}
     }`;
  }}
/>

I don't wanna repeat the same code again and again. I want to reuse the existing code because I have many lines of code. So how can we achieve this ? Any thoughts on this ?

I thought about it but I turns out it's more complicated because tagged templates accept strings and expressions as arguments:
function update(strings, ...expressions) {
  // ...
}

So we could just concatenate everything but in this example where you have expressions inside:
async function animate() {
  ${newCode}
 }

The output would be this because the expressions are added after the strings:
async function animate() {

 }

 // new code...

You could use a function but I would have to think about it more.

The strings and expressions array seem to match so we could merge them together. 🤔

Yeah it seems to work but the formatting needs work.

I think we should make it as a normal function and call it instead passing template strings. If we do so, user has to do code formatting by himself.

Sounds good. Looking forward to it. BTW, thanks for quick reply.

Issue closed by https://github.com/animotionjs/animotion/commit/27982ad42e107a6d84083b93e79fdc4a9e1a4ea5.


Only one paragraph extracted

 Issue by JohnChu101
Tue Aug 20 01:39:11 2019
Originally opened as https://github.com/codelucas/newspaper/issues/726

Yonhap news is one of the largest news agency in south korea.
And i found newspaper doesnt support it much.
For example, this news post has about 9 paragraphs. But newspaper extracts only the second paragraph:
https://newspaper-demo.herokuapp.com/articles/show?url_to_clean=https%3A%2F%2Fen.yna.co.kr%2Fview%2FAEN20190820002300325

Can i fix the problem? Thanks

Works in v 0.9.2


Longest Common Subsequence | C

Longest Common Subsequence -
Want to add the algorithm to this problem under the DP folder in Algorithm -
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding!

I like to contribute assign me

I would like to contribute


AttributeError in GET /clan/CQQJJCRJ/stats.json

Error in ClashLeaders
AttributeError in GET /clan/CQQJJCRJ/stats.json
'NoneType' object has no attribute 'to_data_frame'
View on Bugsnag
Stacktrace
clashleaders/views/clan.py:92 - clan_stats

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
AttributeError in GET /clan/L8U9RVJJ/stats.json

An error linked to this issue has been reopened in Bugsnag
AttributeError in GET /clan/VYPPG9RU/stats.json


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_23bf320c-6d5d-4d08-847f-f981b95c6fe0/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_23bf320c-6d5d-4d08-847f-f981b95c6fe0.git

Docs Build status updates of commit 8267e81:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source-1' (branch: Release_Archive_main_2022-01-25-02-33-33) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Generic class for attune_delay like autonomics

A stripped down class similar to AttuneDelay but initializes with more instructions from a config file, attune catalog, and/or data file.   The attune_delay looks really good for an autonomic feature that changes an input angle to achieve a certain expected phasematching condition for a given set of frequency inputs,  but the current terminologies (_kind="attune-delay" , class AttuneDelay, specific traits) are specific to the word "delay".
Alternately,  separate kinds for each type of attune ( like "attune_theta",  "attune_theta_corrector" (a possible kinematic mirror to adjust errors between expected angle changes and an adjustment to get spatial overlap again, for example)), and just a series of streamlined code that generalizes the entire class constructor.
This class may be necessary in order to generate constant phasematching protocols for four-wave mixing work particularly at lower visible wavelengths where phasematching is more critical  (ie., cobalamins, metal carbonyls, paddlewheel complexes)

@kameyer226 it's my current understanding that active phasematching does not fit into the attune data model because the offsets are not separable. As an example, attune assumes that the offset calculated from w1 and the offset calculated from w2 can be added together to calculate the offset for d1, for any possible w1, w2, combination.
It's for this reason that @ksunden and @emkaufman developed the ndinterp daemon, which I believe is already used.
Again, this is just my understanding. I could be wrong!

The difference being that there may be more than one wrapped_daemon if an extra motor is needed for aligning a mirror kinematically during the procedure.  Mainly this issue just calls into question whether or not the phasematching should be considered an attune with some ndinterp potentially used, whereas it is my understanding that the phasematching currently stands as simply ndinterp with no attune involved.   Am I right?

phasematching currently stands as simply ndinterp with no attune involved

I believe you are correct.

Mainly this issue just calls into question whether or not the phasematching should be considered an attune with some ndinterp potentially used.

My vote is "phasematching should not use attune". On a longer timescale perhaps the attune data model could be extended to support multidimensional correction factors, but that seems like a large amount of effort when we already have a working solution in ndinterp.
Just my vote! It's not my project, just trying to give some historical context.

I'll close this issue in a few days, is there still reason to keep ndinterp in yaqd-wright?   It seems more useful now in general after this was all brought up...

ndinterp is kind of a hack, it emulates enough of the interface expected of an attune-delay to fool the attune daemon in to treating it exactly as it does delays, but most of those methods don't actually do anything.
I'm of the firm belief after dealing with cosets from PyCMDS and implementing attune-delay that these kind of domain-specific "correction" factors are much better as to-task implementations rather than trying to do something overgeneralized.
As such, I feel that there needs to be a use case in mind and not just "I want all the autonomics".
That said, I could imagine breaking out the "what does the 'control' object need to know" bits into a trait.
ndinterp is in yaqd-wright because that is kind of the catch-all for "really specific things for the wright group".

I would advise against anyone outside of the Wright Group using it. Thus I don't want it to be its own package.
Its closest relatives are the attune daemons, though it does not itself use attune, so it doesn't seem right to have it here.
So it kind of defaults to yaqd-wright at that point

For context, I put ndinterp together in about a day or two so that specific experiments could be conducted. I never really looked back at it.
I'd consider making something more specialized to the task after calibration procedures, tests of repeatability, and comparison of empirical data to theory are concluded.