Call can't get established - Terminated

Hello, trying to get my bot into a meeting.
I get all status working just fine, when it comes to actually join, it fails.
It goes from state "Establishing" to "Terminated" (checking on my ngrok dashboard: <IP_ADDRESS>:4040)
Error code 3002.
There is no documentation or help on this regard. How can I find and fix the source of the issue?

You might want to look at

#664

Sounds like the same problem. The solution provided there might help you as well. Otherwise, you'll have to provide more log details

You might want to look at

[PolicyRecordingBot]Server Internal Error. DiagCode: 3002#7106 #664

Sounds like the same problem. The solution provided there might help you as well. Otherwise, you'll have to provide more log details

There is no log anywhere other than that Graph Error.
The info on that other post you mentioned implies a certificate error. Which I strongly believe is not the root cause ( at least initially ). Without a valid certificate the call process wouldn't even start. Certificate is validated/loaded when the MediaChannel gets attached/created to the incoming/outgoing call.

You should be able to retrieve Call IDs / Call Chain IDs / Scenario IDs and timestamps as well. Then some of the Microsofties reading the issue can check their logs as well.
Also, the linked issue mentions the error during Establishing and going straight to Terminated, which sounds pretty close to what you are describing, so I wouldn't outright trust the certificate. You could also provide info on your setup, which of the samples you've used and so on.
The more info you provide, the more likely it is someone can look at it and identify the problem.

You should be able to retrieve Call IDs / Call Chain IDs / Scenario IDs and timestamps as well. Then some of the Microsofties reading the issue can check their logs as well.
Also, the linked issue mentions the error during Establishing and going straight to Terminated, which sounds pretty close to what you are describing, so I wouldn't outright trust the certificate. You could also provide info on your setup, which of the samples you've used and so on.
The more info you provide, the more likely it is someone can look at it and identify the problem.

Thanks for your patience on replying.
Maybe the root cause can be easily found if there is a place, doc, url, article, or whatever, that tells me what kind of certificate, I need to have to use when creating the MediaPlatformInstanceSettings.
If you have that information, kindly share, is there a certificate I can generate myself that works for local testing?
Kindly provide all possible details you remember on this, if you don't mind. If not, then kindly point me to the right direction. This whole thing is too abstract, poorly documented and even if you follow a tutorial from start to finish doing exactly what the person is broadcasting, it does not work.

Requirements:

Properly signed certificate (which means it must have a trust chain up to some generally trusted root authority. If you use a custom domain, you might get one from your domain provider. Otherwise, most samples refer to Let's encrypt certificates. Read up on ACME protocol in that case)
Afaik, CN must match the FQDN, with which your bot is addressed. It should match the FQDN you provide as service name in the media platform instance settings: example

[!Note]
I'm currently the lead maintainer on one of the Public samples soon to be merged into this repo. We are currently working hard on docs (https://github.com/LM-Development/aks-sample/pull/55). If you are struggling, maybe you can find some good info over there as well. Unfortunately, we haven't documented the local setup and usage of Let's encrypt yet.

Hi @MarceloSchoen . I faced the exact same issue some time ago. Try to run the same App on another machine

My most detailed info on this
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",
"value": [
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",
"changeType": "updated",
"resource": "/app/calls/811f5100-686a-45c5-a7be-b355b5d3d847",
"resourceUrl": "/communications/calls/811f5100-686a-45c5-a7be-b355b5d3d847",
"resourceData": {
<EMAIL_ADDRESS>"#microsoft.graph.call",
"state": "establishing",
"chatInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.chatInfo",
"threadId"<EMAIL_ADDRESS>"messageId": "0"
},
"meetingInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.organizerMeetingInfo",
"organizer": {
<EMAIL_ADDRESS>"#microsoft.graph.identitySet",
"user": {
<EMAIL_ADDRESS>"#microsoft.graph.identity",
"id": "ceff7fa8-b5a8-4776-afca-63b07fe42956",
"tenantId": "d8bde65a-3ded-4346-9518-670204e6e184"
}
}
},
"callChainId": "e7c65b69-98aa-4563-b300-0ff2fa0de413"
}
}
]
}
here call is 'establishing'
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",
"value": [
{
<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",
"changeType": "deleted",
"resource": "/app/calls/811f5100-686a-45c5-a7be-b355b5d3d847",
"resourceUrl": "/communications/calls/811f5100-686a-45c5-a7be-b355b5d3d847",
"resourceData": {
<EMAIL_ADDRESS>"#microsoft.graph.call",
"state": "terminated",
"resultInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.resultInfo",
"code": 500,
"subcode": 3002,
"message": "Server Internal Error. DiagCode: 500#3002.@"
},
"chatInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.chatInfo",
"threadId"<EMAIL_ADDRESS>"messageId": "0"
},
"meetingInfo": {
<EMAIL_ADDRESS>"#microsoft.graph.organizerMeetingInfo",
"organizer": {
<EMAIL_ADDRESS>"#microsoft.graph.identitySet",
"user": {
<EMAIL_ADDRESS>"#microsoft.graph.identity",
"id": "ceff7fa8-b5a8-4776-afca-63b07fe42956",
"tenantId": "d8bde65a-3ded-4346-9518-670204e6e184"
}
}
},
"callChainId": "e7c65b69-98aa-4563-b300-0ff2fa0de413"
}
}
]
}
here is terminated
on ngrock ( <IP_ADDRESS>:4040) I see:
All Requests
POST /api/calling 200 OK	48.86ms  ( <<< terminated )
POST /api/calling 200 OK	1.83s       ( estalishing )
my certificate:
E =<EMAIL_ADDRESS>CN = *.stephie.com.br
OU = TI
O = Stefanini
L = Sao Paulo
S = Sao Paulo
C = BR
Public Key, RSA 2048 bits
The thing that called my attention, and, by information provided here, the certificate must be ok ( Assuming is a trusted certificate, tell me if I'm wrong on my assumption )
Current status:
This CA Root certificate is not trusted because it is not in the Trusted Root Certification Authorities store.
I could not import to the Trusted Root Certification Authorities:
The import failed because the store was read-only, the store was full, or the store did not open correctly.

Hey buddy,
I had the EXACT same issue just now. The problem was NOT the certificate. Because it did work with this cert a couple of days ago. I tried regenerating the certificate many different times. Besides, if my cert is incorrect, I can't even initialize the media platform.
Anyway..
There was a mismatch in my DNS <-> Ngrok configuration. So make sure your DNS is configured correctly. The TCP forwarding address needs to match your cname record in your DNS config.
e.g. I had tcp://5.tcp.eu.ngrok.io:19416, while ngrok was forwarding for tcp://9.tcp.eu.ngrok.io:19416.
So if you look at the bigger picture, this error is probably related to microsoft's server not being able to reach your InstancePublicPort. So make sure your dns is cconfigured, the port is not being blocked by your firewall, etc.
Good luck! :)

Hey buddy,
I had the EXACT same issue just now. The problem was NOT the certificate. Because it did work with this cert a couple of days ago. I tried regenerating the certificate many different times. Besides, if my cert is incorrect, I can't even initialize the media platform.
Anyway..
There was a mismatch in my DNS <-> Ngrok configuration. So make sure your DNS is configured correctly. The TCP forwarding address needs to match your cname record in your DNS config.
e.g. I had tcp://5.tcp.eu.ngrok.io:19416, while ngrok was forwarding for tcp://9.tcp.eu.ngrok.io:19416.
So if you look at the bigger picture, this error is probably related to microsoft's server not being able to reach your InstancePublicPort. So make sure your dns is cconfigured, the port is not being blocked by your firewall, etc.
Good luck! :)

Thanks for your reply
But, that tcp from ngrok, what do you mean by checking my dns? im trying locally, nothing blocking or anything, besides, on a real scenario that would imply that I have like a neverending running ngrok, right? because If I restart it, that port it provides changes the number.
If you mean dns on my registered domain, I'll have to update the CName there ( in my case local.stephie.com.br ) then I'll be going nuts, this changes all the time and my domain provider takes about 1-2hs to apply changes.
the config ( appsettings ) do not use that, it uses the other address, in my case:
"BotConfiguration": {   "BotName": "*****",   "AadAppId": "d35df3e1-c974-4b4a-b3cc-828b614d227a",   "AadAppSecret": "******",   "ServiceCname": "86e6-200-180-169-254.ngrok-free.app",   "MediaServiceFQDN": "local.stephie.com.br",   "ServiceDnsName": "",   "CertificateThumbprint": "171a574199698646d4be5290f690902db685032a",   "InstancePublicPort": 15931,   "CallSignalingPort": 9441,   "InstanceInternalPort": 8445,   "PlaceCallEndpointUrl": "https://graph.microsoft.com/v1.0" },
the Cname matches the current ngrok instance, which properties are as bellow:
Session Status                online
Account                       Marcelo Schoen (Plan: Free)
Version                       3.8.0
Region                        South America (sa)
Latency                       30ms
Web Interface                 http://<IP_ADDRESS>:4040
Forwarding                    tcp://0.tcp.sa.ngrok.io:15931 -> localhost:8445
Forwarding                    https://86e6-200-180-169-254.ngrok-free.app -> https://localhost:9441                                                                                Connections                   ttl     opn     rt1     rt5     p50     p90
8       0       0.00    0.00    90.09   92.02

Assuming you generated your own self signed certificate, you did this against a domain. The DNS configuration of that domain needs the following two records (Not sure if no. 2 is mandatory, but this works for me. So it doesn't hurt to have it.)
Record 1: TYPE: CNAME, HOST: local, Value: 0.tcp.sa.ngrok.io (replace 0 with the number as shown in ngrok)
Record 2: TYPE CNAME, HOST: www, Value: [yourDnsName]
And yes you have to update the CNAME in your DNS records everytime your ngrok instance changes. Luckily for me it's instantaneously. But if it really takes two hours for you there's two approaches you can take.

Keep firing ngrok until you get the number (0.tcp.sa.ngrok.io) [0 = number] matches the one in your CNAME
You can get a paid ngrok account and configure this manually if I'm not mistaken

Don't forget update the InstancePublicPort and ServiceCName in the appsettings everytime you restart ngrok. This has cost me some headaches as well.

Assuming you generated your own self signed certificate, you did this against a domain. The DNS configuration of that domain needs the following two records (Not sure if no. 2 is mandatory, but this works for me. So it doesn't hurt to have it.)
Record 1: TYPE: CNAME, HOST: local, Value: 0.tcp.sa.ngrok.io (replace 0 with the number as shown in ngrok) Record 2: TYPE CNAME, HOST: www, Value: [yourDnsName]
And yes you have to update the CNAME in your DNS records everytime your ngrok instance changes. Luckily for me it's instantaneously. But if it really takes two hours for you there's two approaches you can take.

Keep firing ngrok until you get the number (0.tcp.sa.ngrok.io) [0 = number] matches the one in your CNAME
You can get a paid ngrok account and configure this manually if I'm not mistaken

Don't forget update the InstancePublicPort and ServiceCName in the appsettings everytime you restart ngrok. This has cost me some headaches as well.
Edit: In a 'real scenario' (e.g. running on production), there are alternatives to ngrok. Ngrok is just one of many proxy servers. But regardless of what you use, you will have to make sure everything is forwarded correctly. Ngrok just happens to constantly assign a new address for the TCP port. But I'm pretty sure you might be able to make this static when configured properly / using the software with a paid account.

Thanks! That is quite revealing.
I really did that part of setting up the cname on my domain, but in the case I'm on, the generated certificate had a lot of issues, the generated one had a public key of type ECC and this way my machine can't read the private key, I re-generated it with a type RSA and that solved that part of the issue. (certificate displays as valid)
I'll have a perpetual instance of ngrok running and probably will go for the paid service if that helps as well.

Assuming you generated your own self signed certificate, you did this against a domain. The DNS configuration of that domain needs the following two records (Not sure if no. 2 is mandatory, but this works for me. So it doesn't hurt to have it.)
Record 1: TYPE: CNAME, HOST: local, Value: 0.tcp.sa.ngrok.io (replace 0 with the number as shown in ngrok) Record 2: TYPE CNAME, HOST: www, Value: [yourDnsName]
And yes you have to update the CNAME in your DNS records everytime your ngrok instance changes. Luckily for me it's instantaneously. But if it really takes two hours for you there's two approaches you can take.

Keep firing ngrok until you get the number (0.tcp.sa.ngrok.io) [0 = number] matches the one in your CNAME
You can get a paid ngrok account and configure this manually if I'm not mistaken

Don't forget update the InstancePublicPort and ServiceCName in the appsettings everytime you restart ngrok. This has cost me some headaches as well.
Edit: In a 'real scenario' (e.g. running on production), there are alternatives to ngrok. Ngrok is just one of many proxy servers. But regardless of what you use, you will have to make sure everything is forwarded correctly. Ngrok just happens to constantly assign a new address for the TCP port. But I'm pretty sure you might be able to make this static when configured properly / using the software with a paid account.

Tried it,
Also, tried opening my router port on the given ones ( 9441, 8445, and whatever ngrok displayed ), no luck, same thing, goes from establishing to terminated with no additional info.
How can a Dev solve a problem that has no explanation on what is happening, and the request has a 200 response code? Like, its fine, I terminated your request with an error, but you got a 200, OK, all fine here.
Why is it so hard to get a straight answer? Like I've said a little earlier, the whole thing is poorly documented, if you follow microsoft's guidelines on implementing this, from start to finish, you either got stuck on the first few steps, or, you do things exactly as the 'documentation' tells you to, it does not work, and you have no chance on knowing what is wrong, and where.
You get a generic error message, that has no definition, or place to look up to. Just straight up, error 3002.
As if we could possibly guess what 3002 is about.

@MarceloSchoen
Please check that the InstancePublicPort value is correct. For the most recent call attempt on 19 April 2024, I see this error in telemetry (where  is the value given to MediaPlatformSettings.MediaPlatformInstanceSettings.ServiceFqdn):
Could not verify connectivity to the bot's media platform instance at :11703. Please verify that InstancePublicPort 11703 on the load balancer for  is mapped to InstanceInternalPort port 8445 on the local machine.

@MarceloSchoen Please check that the InstancePublicPort value is correct. For the most recent call attempt on 19 April 2024, I see this error in telemetry (where ServiceFQDN is the value given to MediaPlatformSettings.MediaPlatformInstanceSettings.ServiceFqdn):
Could not verify connectivity to the bot's media platform instance at ServiceFQDN:11703. Please verify that InstancePublicPort 11703 on the load balancer for ServiceFQDN is mapped to InstanceInternalPort port 8445 on the local machine. A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. [::ffff:146.112.211.xx]:11703
In previous call attempts the error indicated that ServiceFQDN was not a valid DNS name ("No such host is known").

Hi ssulzer, I'm testing locally, as I mentioned, even tried to open my router ports. And yes, the port was the same as ngrok displayed.
Your suggestion is that I create a LoadBalancer then? I believe I already have one in place on AWS.
For this scenario, all of the given ports must be set on AWS side?
I'm trying to be sure of this, because everytime I run ngrok the port change, same for the CName address I need to set up.
Can you kindly confirm?
Ports must be set on router?
Ports must be set on AWS LoadBalancer?
The mapping that you mentioned, is already performed by ngrok, I don't have to do anyting, right? When it runs it maps requests on the instance port to go to my localhost port (8445)

@MarceloSchoen Please check that the InstancePublicPort value is correct. For the most recent call attempt on 19 April 2024, I see this error in telemetry (where ServiceFQDN is the value given to MediaPlatformSettings.MediaPlatformInstanceSettings.ServiceFqdn):
Could not verify connectivity to the bot's media platform instance at ServiceFQDN:11703. Please verify that InstancePublicPort 11703 on the load balancer for ServiceFQDN is mapped to InstanceInternalPort port 8445 on the local machine. A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond. [::ffff:146.112.211.xx]:11703
In previous call attempts the error indicated that ServiceFQDN was not a valid DNS name ("No such host is known").

Hi ssulzer, I'm testing locally, as I mentioned, even tried to open my router ports. And yes, the port was the same as ngrok displayed. Your suggestion is that I create a LoadBalancer then? I believe I already have one in place on AWS. For this scenario, all of the given ports must be set on AWS side? I'm trying to be sure of this, because everytime I run ngrok the port change, same for the CName address I need to set up.
Can you kindly confirm? Ports must be set on router? Ports must be set on AWS LoadBalancer? The mapping that you mentioned, is already performed by ngrok, I don't have to do anyting, right? When it runs it maps requests on the instance port to go to my localhost port (8445)

It persists
{<EMAIL_ADDRESS>"#microsoft.graph.commsNotifications",     "value": [         {<EMAIL_ADDRESS>"#microsoft.graph.commsNotification",             "changeType": "deleted",             "resource": "/app/calls/431fd200-a430-4fbf-a335-f31ea5725f79",             "resourceUrl": "/communications/calls/431fd200-a430-4fbf-a335-f31ea5725f79",             "resourceData": {<EMAIL_ADDRESS>"#microsoft.graph.call",                 "state": "terminated",                 "resultInfo": {<EMAIL_ADDRESS>"#microsoft.graph.resultInfo",                     "code": 500,                     "subcode": 3002,                     "message": "Server Internal Error. DiagCode: 500#3002.@"                 },                 "chatInfo": {<EMAIL_ADDRESS>"#microsoft.graph.chatInfo",                     "threadId"<EMAIL_ADDRESS>                    "messageId": "0"                 },                 "meetingInfo": {<EMAIL_ADDRESS>"#microsoft.graph.organizerMeetingInfo",                     "organizer": {<EMAIL_ADDRESS>"#microsoft.graph.identitySet",                         "user": {<EMAIL_ADDRESS>"#microsoft.graph.identity",                             "id": "ceff7fa8-b5a8-4776-afca-63b07fe42956",                             "tenantId": "d8bde65a-3ded-4346-9518-670204e6e184"                         }                     }                 },                 "callChainId": "2784a1bd-09fd-412f-a665-48af0271cd60"             }         }     ] }

How can I review the progress as you have ( telemetry ) from my end, without the need to rely on someone checking it here?

I have started to get this error again for no reason. Everything, certificate, ngrok, local is working. But the bot is not able to join the meeting again.


Customizable Actions

Basically looking at similar issue as #139, we are trying to implement notification with customized action, something like this screenshot

Is there any way to do it on the js level or it has to come down to native android/ios code? Thanks for any response in advance.

Is there any information on this?


Fix Root Org Template Resolving Edge Case

Purpose

$Subject

Related Issues

https://github.com/wso2/product-is/issues/21214

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/12271554899

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/12271554899
Status: success


Consider emitting .d.ts files as output for TypeScript

Hi! @ryyppy showcased this in ReactAlicante last weekend, and it was very cool! 😎
After the talk, we had a very interesting conversation with @ForbesLindesay where he mentioned that it will increase performance if the TypeScript output were only declaration files and not actual source code (like the case with flow).
Maybe we can consider that? This issue is just to keep track of that conversation 😉

Some code is required to wrap components for ReasonReact, and to perform some type conversions at runtime.
One could in principle generate pairs of files, but perhaps the granularity of .d.ts files can be different.

Yes, the proposal was to generate a pair of files:

foo.d.ts - type definitions. Since these don't contain any implementation, they don't get full type checking by typescript
foo.js - the actual implementation, completely ignored by typescript
foo.js.flow - optionally also generate flow type definitions

In addition to being faster, typescript and flow are much less likely to break code written to type definitions when they add stricter type checking rules/change their inference engines.

@kutyel @ForbesLindesay
What are a couple of idiomatic projects with TypeScript and React? I'd like to look at them and see what the ideal conversion process would look like.
Ideally, we want to support the following conversion process:

There is a Parent reactjs component and a Child reactjs component.
Delete the Child component and re-write in Reason, with @genType annotations. The Parent component does not need to be changed at all.
Delete the Parent component and re-write in Reason. Delete the @genType annotations from the Reason Child component, as it's not used from JS anymore, so the wrapper artifacts generated become redundant and are now deleted.

The crucial step is 2, where the assumption is that what @genType generates is immediately consumable by Parent, i.e. is idiomatic.

I think @jaredpalmer can help with that 😉

Moving this to the more general issue https://github.com/cristianoc/genType/issues/95.

@cristianoc Issue #95 was closed without addressing how to emit .d.ts AFAICT so this issue should be reopened.


adds readme.md

Description
I have updated readme.md file
@thestarsahil pls see to the readme.md file.
if there is any change you want pls tell me
Screenshots

this is #74 solution.
thank you


Fix top-level doctest

The way the std::process::exit call in the top level doctest is currently commented out is somewhat confusing to readers (or at least does not flow well); this commit instead wraps it in a hidden if false { ... }, which will prevent it from running without affecting what the documentation looks like.

@dpc please merge 😉


Update

Update blackjack system pls

Update blackjack system pls

could you be more specific ? what exactly dyou mean

could you be more specific ? what exactly dyou mean

I want to say that if you could please update and review the blackjack system since I have been testing it and it does not work correctly. Thank you.

Can you reach out on discord ? Add me at kendrik2.0, its easier to talk there, you can make a temporary account i think. Im gonna need more info to know what exactly needs to be fixed.

There I sent you a request, my username is juan.cc02, when you accept me, talk to me and I'll tell you what happens with the code.


error

Xcode：14.2
MacOS：13.3.1 (a) (22E772610a)

Compare your Podfile with Podfile in example.
Then make sure pod install --repo-update could run without any error.
Maybe you need clean xcode cache before.


🌐 Add Chinese translation for docs/zh/docs/advanced/using-request-directly.md

📝 Docs preview for commit bfb103cfe50b741c95a1c326f0d9d37b7d0db4ea at: https://6499e0028604e00374442572--fastapi.netlify.app

Thank you @jaystone776! :rocket:
As you are one of the Chinese translations' champions, I'm gonna merge this right away. :tada:


Install the correct ServerSetup script

🗣 Description
This pull request fixes a bug where the most recent ServerSetup.sh script was being installed instead of the PCA-specific version that the PCA crew has historically been using.
It also adds a script for deleting PCA *.tfvars files from the S3 bucket where they are stored.  This is something I have been using for a while but kept forgetting to check in.
💭 Motivation and Context
The PCA crew wants to continue using the same version of the ServerSetup.sh file.  Switching to the new one would require testing to ensure identical functionality as well as updating their runbook.
🧪 Testing
I successfully built and deployed a new AMI using this code, and I verified that it contains the correct version of the ServerSetup.sh script.
🚥 Types of Changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (causes existing functionality to change)

✅ Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I believe the dependabot foo is misfiring because of this.


Change pbstream migrate to support new submap format.

This changes the migration support to:

Remove migration from old files from before version 1.0.
If this is still needed by someone, they can use the tool
as it was before this change.
Adds migration of maps without submap histograms. These
were so far migrated on-the-fly, but #1710 aims to remove
this.

See also #1709.
This was tested by converting a serialized 3D map and verifying
it can still be used for localization. The file changed a little
bit, but it seems to be good enough.
Signed-off-by: Wolfgang Hess<EMAIL_ADDRESS>

FYI @MichaelGrupp @schwoere

The code is adapted from MapBuilder::LoadState() at https://github.com/cartographer-project/cartographer/blob/master/cartographer/mapping/map_builder.cc.

There's also a documentation page: https://github.com/cartographer-project/cartographer/blob/master/docs/source/pbstream_migration.rst

@MichaelGrupp I updated the documentation to reflect the change in the tool. Even though it is now documented, it probably is best to point people towards the tool when loading a submap fails in #1710.

lgtm


Update haproxy.md

Purpose of this pull request
This pull request is to format the [[inputs.Haproxy]] to [[inputs.haproxy]].
Select the type of change

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions, updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

@JV0812, can this PR be merged? There isn't a "do-not-merge" tag on this PR.


Non-Wide Tables don't format correctly in pdf export

Description
When creating non-wide tables the exported pdf has a formatting issue.
Here is the table in the editor:

Here is the table in the pdf preview:

This only occurs when the table is placed after a column break (ie. it is on the right side of the page).

This is a known issue and a fix is on the way. See #318 for a temporary workaround.
Duplicate of #318


[Navigator]navigator transition,meantime setState,not smooth

if navigator transition (like push transition),meantime ,no matter what component setState,the transition will be not smooth.For example,A view get network data in componentDidMount,if the push transition was not finished ,the newwork data has gotten and the view setState,the transition will be bad

Now my solution is that the component adds a navigator onDidFocus life cycle.The setState handle is executed in the onDidFocus.But I think my solution is not very good.Any solutions?

Wrap your setState() code
InteractionManager.runAfterInteractions(() => {
    // ...your setstate here...
});

@ssssssssssss Thank you.That's useful


Diff revision

Add the ability to diff a specific revision of a file.
Usage: call the command on file.txt and enter 6 to diff with file.text#6

Excellent feature, thanks !


Theming / Styling

Is there a way on how to apply custom themes / styles to the popup components?

Sorry, theming is not supported yet.
You could overwrite the picker default in you own css files

@DanielYKPan If it's not supported yet, then why did you close it? Maybe it should stay open until this is implemented?


libz-ng.a and libz.a from zlib-compat mode have symbol conflicts

zlib-ng in native mode avoids symbol conflicts with stock zlib, but still has symbol conflicts with zlib supplied from zlib-ng built in zlib-compat mode.
This can cause an issue in a few different scenarios:

If a distribution uses zlib-ng in zlib-compat mode as its system zlib, and an application statically links native zlib-ng but also links a library (such as a system library) that links to zlib, expecting them not to conflict.
If an application uses native zlib-ng, but has many dependencies, one of which uses zlib-ng in zlib-compat mode (e.g. because it needs to link a C library that needs zlib symbols but wants the performance of zlib-ng).

One instance of this manifested as these link errors:
  = note: liblibz_ng_sys-fb58de474afbb4a4.rlib(inflate.obj) : error LNK2005: fixedtables already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(inflate.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(crc32_braid.obj) : error LNK2005: crc32_braid already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(crc32_braid.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: z_error already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zlibng_version already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zng_alloc_aligned already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zng_calloc already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zng_cfree already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zng_free_aligned already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: z_verbose already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)
          liblibz_ng_sys-fb58de474afbb4a4.rlib(zutil.obj) : error LNK2005: zlibng_string already defined in liblibz_sys-38cf6ff5b07be7a7.rlib(zutil.obj)

I think this could be fixed with three steps:

For the symbols like fixedtables and crc32_braid, prefix them with zng_ when not building in zlib-compat mode.
For symbols like zlibng_version, don't define them at all in zlib-compat mode.
For symbols like zng_alloc_aligned, prefix them differently (e.g. zngcompat_alloc_aligned) when building in zlib-compat mode.

This could also be addressed by hiding some of those symbols, if possible.
(Note that the issue above manifested on Windows, and I couldn't reproduce it on Linux, so it's possible it's specific to either the Windows build of zlib-ng or to behavior of the Windows linker.)

No new binary should ever link against two copies of zlib-ng with different compiler flags... It's expected to fail and is by design. Even if we prefix every single symbol, the result is still something we should discourage.

The scenario @joshtriplett describes is a quite common (and very annoying one) when building packages, and I can absolutely see myself hitting that one.
Lets say you want to build application NN with zlib-ng-native, and its dependency-tree has a ton of libraries, and one or more tightly system-integrated libraries depends on zlib-nb-compat.
Been there, done a very similar dance multiple times when wanting to build an updated application that requires a few updated libraries but somewhere in there you have a dependency for a non-updated library or otherwise conflicting one. Both ImageMagick and Curl comes to mind..
As for the suggested solutions:

Not a huge fan of prefixing everything with macros, but this would work.
I'd like the zlib-ng version to be available to the application somehow even in compat mode without breaking compatibility with existing applications. Not sure about this one, but perhaps a different prefix.
zngc_ prefix or something would work.

Hiding the symbols would be the ideal solution, I wonder why they are not already. zng_cfee (and others on the list above) use the Z_INTERNAL macro, that is supposed to use the internal hint or fall back to the hidden hint, effectively making sure this symbol is hidden from other applications. How was the errors above hit? Like described above an app linking zlib-ng-native and a library that already used zlib-ng-compat, both dynamically? Or are either library linked statically or both directly into the same application? It is my understanding that this scenario should not happen when doing dynamic linking (for the internal/hidden symbols that is).
Linking both directly into the same application is more likely to cause problems of course, and I'm not sure that is actually something worth doing (why would anyone need to? Even for benchmarking it does not really make sense to load both into the same binary at the same time).

Some of the symbols are specific to zlib 1.2.12 and will be fixed when we declare compatibility with the new API... Some might very well be due to bad compiler flags or incompatible compiler/linker versions.

Lets say you want to build application NN with zlib-ng-native, and its dependency-tree has a ton of libraries, and one or more tightly system-integrated libraries depends on zlib-nb-compat. Been there, done a very similar dance multiple times when wanting to build an updated application that requires a few updated libraries but somewhere in there you have a dependency for a non-updated library or otherwise conflicting one. Both ImageMagick and Curl comes to mind..

This is exactly the kind of scenario I'm running into. In my case, some of the libraries requiring zlib support are curl and libgit2, and I'd like both to use zlib-ng in zlib-compat mode for performance, but I'm also slowly (and incrementally) migrating to other dependencies that can natively use zlib-ng.

2. I'd like the zlib-ng version to be available to the application somehow even in compat mode without breaking compatibility with existing applications. Not sure about this one, but perhaps a different prefix.

Fair enough; would zngc_zlibng_version work for that?

3. `zngc_` prefix or something would work.

:+1:

Hiding the symbols would be the ideal solution, I wonder why they are not already. zng_cfee (and others on the list above) use the Z_INTERNAL macro, that is supposed to use the internal hint or fall back to the hidden hint, effectively making sure this symbol is hidden from other applications.

As far as I can tell, Z_INTERNAL exclusively uses __attribute__((visibility(...))), which doesn't work when compiling for MSVC. I don't know if there's an MSVC-compatible mechanism for hiding symbols in static libraries (other than combining source files and using static, which doesn't seem ideal).

How was the errors above hit? Like described above an app linking zlib-ng-native and a library that already used zlib-ng-compat, both dynamically? Or are either library linked statically or both directly into the same application? It is my understanding that this scenario should not happen when doing dynamic linking (for the internal/hidden symbols that is).

I was building a Rust application, which was statically linking in all its libraries. I pulled in multiple different crates; one could use zlib-ng in native mode because it had native support, while others (curl) needed zlib compatibility. I'd like to be able to incrementally migrate to using native support.
I'm one of the maintainers of libz-sys (which supports both zlib and zlib-ng in zlib-compat mode) and libz-ng-sys (which uses zlib-ng natively) in the Rust ecosystem. I'm trying to migrate some libraries from only supporting the former to also supporting the latter when possible. I'm expecting that migration to be incremental and gradual, and any crates providing C library bindings would have to wait for the underlying C library to support using zlib-ng natively (which curl and libgit2 don't yet). For that reason, I'd like to be able to ensure that 1) crates providing C library bindings that need zlib can offer the option of using zlib-ng-compat for performance, and 2) crates that support zlib-ng natively can use zlib-ng natively.

The correct way to handle is to use either statically or dynamically linked wrapper library that links dynamically to zlib-ng... This is how I do cross-language wrappers in my other projects... If there is still symbol conflicts, those should be handled with the existing symbol prefixing feature.
There will be a lot of intentional symbol conflicts when statically linking against multiple copies of zlib-ng.

Using the symbol prefix feature would make those libraries incompatible with C code that's expecting either the zlib API or the native zlib-ng API. And in any case, the symbol prefixing feature doesn't appear to prefix the symbols I encountered symbol conflicts with.

I have put together a GHA workflow that reproduces the issue:
https://github.com/nmoinvaz/zlib-ng/blob/improvements/dual-link-test/.github/workflows/link.yml
CI run:
https://github.com/nmoinvaz/zlib-ng/runs/6953216938?check_suite_focus=true
Output:
[100%] Linking CXX executable gtest_zlib
/usr/bin/ld: ../libz-ng.a(crc32_braid.c.o): in function `crc32_braid':
crc32_braid.c:(.text+0x60): multiple definition of `crc32_braid'; ../../compat/libz.a(crc32_braid.c.o):crc32_braid.c:(.text+0x70): first defined here
/usr/bin/ld: ../libz-ng.a(deflate.c.o): in function `read_buf':
deflate.c:(.text+0x1a50): multiple definition of `read_buf'; ../../compat/libz.a(deflate.c.o):deflate.c:(.text+0x1a40): first defined here
/usr/bin/ld: ../libz-ng.a(deflate.c.o): in function `fill_window':
deflate.c:(.text+0x1ae0): multiple definition of `fill_window'; ../../compat/libz.a(deflate.c.o):deflate.c:(.text+0x1ae0): first defined here
/usr/bin/ld: ../libz-ng.a(inflate.c.o): in function `fixedtables':
inflate.c:(.text+0x460): multiple definition of `fixedtables'; ../../compat/libz.a(inflate.c.o):inflate.c:(.text+0x450): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o): in function `zng_calloc':
zutil.c:(.text+0x0): multiple definition of `zng_calloc'; ../../compat/libz.a(zutil.c.o):zutil.c:(.text+0x0): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o): in function `zng_cfree':
zutil.c:(.text+0x60): multiple definition of `zng_cfree'; ../../compat/libz.a(zutil.c.o):zutil.c:(.text+0x60): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o): in function `zlibng_version':
zutil.c:(.text+0x70): multiple definition of `zlibng_version'; ../../compat/libz.a(zutil.c.o):zutil.c:(.text+0x80): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o): in function `zng_alloc_aligned':
zutil.c:(.text+0xb0): multiple definition of `zng_alloc_aligned'; ../../compat/libz.a(zutil.c.o):zutil.c:(.text+0xc0): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o): in function `zng_free_aligned':
zutil.c:(.text+0x160): multiple definition of `zng_free_aligned'; ../../compat/libz.a(zutil.c.o):zutil.c:(.text+0x170): first defined here
/usr/bin/ld: ../libz-ng.a(zutil.c.o):(.rodata+0x0): multiple definition of `zlibng_string'; ../../compat/libz.a(zutil.c.o):(.rodata+0x0): first defined here
collect2: error: ld returned 1 exit status
make[2]: *** [test/CMakeFiles/gtest_zlib.dir/build.make:516: test/gtest_zlib] Error 1
make[1]: *** [CMakeFiles/Makefile2:403: test/CMakeFiles/gtest_zlib.dir/all] Error 2
make: *** [Makefile:146: all] Error 2

What is strange to me is that it has a problem with crc32_braid but it does not have a problem with compare256_c. Both functions are defined very much the same.

When multiple object files need a symbol with same name, the lookup is done sequentially from the point where the symbol is used. This means any previous definitions will be ignored until the final phase of the linking. This is issue that happens only with static archives, as symbols in shared libraries are resolved at runtime.
For some symbols this is not an issue as they are required from either the startup object files inserted by compiler, or the first main function in the resulting executable (called from the startup object files).

@joshtriplett this should be merged into develop now.

Closing, should be resolved in develop.


As a user, I'd like to have an automatically populated 'Primary Donor' lookup field on Opportunity to Contact

This field should be the contact who is checked as the Primary Opportunity Contact Role. If the Contact Role is change to another contact, a job should check for that and update the lookup field.

Included in beta release version 3.41 (Beta 8) and higher

Included in production release version 3.41


Vendor in latest containers/storage

We need this to start testing metacopy up for podman.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

/approve

LGTM assuming happy tests


When a largeTextArea is the last required field, typing causes an insert

Since the largeTextArea posts an update every 100 characters, that can trigger an insert. The insert then sets the cursor position back to the start of the field.

Changed saveableChange boolean to changeType string, which gives the possibility of defining what the type of change is. This allows for not saving on changes, even significant changes.


Verify layer's digest when assembling layer

This Branch is based on https://github.com/docker/docker/pull/14067 but also includes verifying the layer's digest, if present.
Per @stevvooe comment --> https://github.com/docker/docker/pull/14067#discussion_r35064189

@vbatts LGTM

LGTM. Waiting for tests.

Quite a few failures
FAIL: docker_cli_by_digest_test.go:112: DockerRegistrySuite.TestCreateByDigest

docker_cli_by_digest_test.go:115:
    c.Fatalf("error setting up image: %v", err)
... Error: error setting up image: pushing the image to the private registry has failed: The push refers to a repository [<IP_ADDRESS>:5000/dockercli/busybox-by-dgst] (len: 1)
3221ebfb8a39: Buffering to Disk
3221ebfb8a39: Image successfully pushed
8c2e06607696: Image already exists
6ce2e90b0bc7: Buffering to Disk
failed digest verification of reassembled layer data
, exit status 1

Figures. That's why I wanted to merge #14067 first. I expected this digest bit to add confusion . the other PR passes everything...
I'm not in a place to jump on to fixing this right now

@dmcgowan again this is why I did not want to entangle digest verification with this tar assembly PR.

Maybe I shouldn't have closed the other one, sorry, but at the same time I understood @stevvooe was a nogo unless we take this one...

At a high level design: what if the Docker client supported telling the server where to find the origin registry for intermediate layers?  Then the server could download and verify the original.  In that model, we don't need client-side identical re-assembly.  The client should keep track of the difference between layers it built and layers it merely mirrored.
I can't think offhand of a case where a client downloads an image and is uploading it to a registry that doesn't have access to the intermediate layer.  For example, if an intermediate image requires entitlement, having this hard fail is actually a good thing, right?

we determined that the failures here are due to the digests being calculated on a compress layer, but this verification is occurring on an uncompressed layer.
Since layers are not unconditionally compressed or uncompressed (e.g. docker push compresses the layer, but docker save does not compress), doing verification at this point would be inconsistent.
Even though I'm looking forward to digest verification on push/save, I'm going to close this issue for now.


Introduce new parameter "ldap-admin-dn"

We want to restrict LDAP Updates to a single (super-)user. The new
commandline parameter defines the DN that is given write access to the
full database. All other users will have read-only access.

@longsleep  Here's another one ...

@longsleep Hi. Are you ok with this?


TF_ROCM_FUSION_ENABLE fails on official/resnet/cifar10_main.py

System information

Have I written custom code: No
OS Platform and Distribution: Ubuntu 18.04
TensorFlow installed from: binary
TensorFlow version: 1,13
Python version: 3.5
ROCm/MIOpen version: 2.4
GPU model and memory: gfx900/8GB

Describe the current behavior
When enabling TF_ROCM_FUSION_ENABLE  official/resnet/cifar10_main.py  model fails to run
Describe the expected behavior
TF_ROCM_FUSION_ENABLE should improve performan and do not fail
Code to reproduce the issue
$ docker run -it --rm --device=/dev/kfd --device=/dev/dri --group-add video  rocm/tensorflow:rocm2.4-tf1.13-python3
2326577ab9c8:~:$ cd /tmp/
@2326577ab9c8:/tmp$ git clone -b r1.13.0 --depth 1 https://github.com/tensorflow/models.git
Cloning into 'models'...
...
@2326577ab9c8:/tmp$ cd models/
@2326577ab9c8:/tmp/models$ TF_ROCM_FUSION_ENABLE=1 PYTHONPATH=`pwd`  python3.5 official/resnet/cifar10_download_and_extract.py
...
@2326577ab9c8:/tmp/models$ TF_ROCM_FUSION_ENABLE=1 PYTHONPATH=`pwd`  python3.5 official/resnet/cifar10_main.py

2019-05-15 02:28:48.760231: I tensorflow/core/graph/gpu_fusion_pass.cc:434] ROCm Fusion is enabled.
2019-05-15 02:28:48.767839: F tensorflow/core/graph/gpu_fusion_pass.cc:1346] Unexpected output connection on the BatchNormGrad node :id : 5696, name : gradients/resnet_model/batch_normalization/FusedBatchNorm_grad/FusedBatchNormGrad, type_string : FusedBatchNormGrad, num_inputs : 5, num_input_edges : 5, num_outputs : 5, num_output_edges : 6. output(idx, dest_node) :3, id : 5701, name : gradients/resnet_model/batch_normalization/FusedBatchNorm_grad/tuple/control_dependency_3, type_string : Identity, num_inputs : 1, num_input_edges : 2, num_outputs : 1, num_output_edges : 1
Fatal Python error: Aborted

Thread 0x00007ffe6affd700 (most recent call first):
  File "/usr/lib/python3.5/threading.py", line 293 in wait
  File "/usr/lib/python3.5/queue.py", line 164 in get
  File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/summary/writer/event_file_writer.py", line 159 in run
  File "/usr/lib/python3.5/threading.py", line 914 in _bootstrap_inner
  File "/usr/lib/python3.5/threading.py", line 882 in _bootstrap

Other info / logs
resnet from benchmarks actually runs with TF_ROCM_FUSION_ENABLE and shows nice performance boost.

Hi @TheCBaH , I can reproduce your observations. Please note TF_ROCM_FUSION_ENABLE is disabled by default, users should only enable it when it's known to work. We will refine the implementation of fusion on ROCm path to make it a more robust mechanism.
Adding @deven-amd here to comment if the failure for the resnet/cifar10_main.py sample is expected with Fusion enabled.

Hi @TheCBaH ,
Can you try out the docker container : devenamd/tensorflow:rocm2.4-tf1.13-python3-issue-456-fix
It is based on the `` image, and contains the fix for the issue.
The bug seems to be that the "reserved" output ports for one of the "BatchNormGrad" nodes have connections on them. This is not expected by the code that creates the fusion nodes, and hence the error.
The "fix" is to merely detect this while we are determining fusion eligibility for a node, and skip fusion for nodes that fit this scenario.
I will also be filiing a PR to push this change to the develop-upstream branch

Hi @deven-amd,

Can you try out the docker container : devenamd/tensorflow:rocm2.4-tf1.13-python3-issue-456-fix

Yes,  devenamd/tensorflow:rocm2.4-tf1.13-python3-issue-456-fix  fixes the issue, thank you or quick turnaround.

The "fix" is to merely detect this while we are determining fusion eligibility for a node, and skip fusion for nodes that fit this scenario.

Even, if fusion skipped for the batch normalization nodes, TF_ROCM_FUSION_ENABLE provides some boost for the entire network:

without TF_ROCM_FUSION_ENABLE

I0518 05:42:44.440500<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 14.024
I0518 05:42:44.441334<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.934088, learning_rate = 0.1, train_accuracy = 0.1953125 (7.130 sec)
I0518 05:42:44.441634<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.4139876, step = 200 (7.130 sec)
I0518 05:42:51.596385<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 13.9746
I0518 05:42:51.597278<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.8011167, learning_rate = 0.1, train_accuracy = 0.23242188 (7.156 sec)
I0518 05:42:51.597656<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.2656488, step = 300 (7.156 sec)
I0518 05:42:58.753165<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 13.9728
I0518 05:42:58.754460<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.6648593, learning_rate = 0.1, train_accuracy = 0.2625 (7.157 sec)
I0518 05:42:58.754843<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.1150098, step = 400 (7.157 sec)
I0518 05:43:05.922024<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 13.9492
I0518 05:43:05.923305<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.5323229, learning_rate = 0.1, train_accuracy = 0.29296875 (7.169 sec)
I0518 05:43:05.923877<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 1.9692287, step = 500 (7.169 sec)

with TF_ROCM_FUSION_ENABLE

I0518 05:44:23.737204<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.9406083, learning_rate = 0.1, train_accuracy = 0.1625 (7.068 sec)
I0518 05:44:23.737712<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.4758043, step = 400 (7.068 sec)
I0518 05:44:30.815824<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 14.1244
I0518 05:44:30.817555<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.8614838, learning_rate = 0.1, train_accuracy = 0.17578125 (7.080 sec)
I0518 05:44:30.818078<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.378029, step = 500 (7.080 sec)
I0518 05:44:37.909114<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 14.0978
I0518 05:44:37.910563<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.7252854, learning_rate = 0.1, train_accuracy = 0.19642857 (7.093 sec)
I0518 05:44:37.911145<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.2251897, step = 600 (7.093 sec)
I0518 05:44:44.992385<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 14.1178
I0518 05:44:44.993691<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] cross_entropy = 1.61365, learning_rate = 0.1, train_accuracy = 0.21484375 (7.083 sec)
I0518 05:44:44.994167<PHONE_NUMBER>32896 basic_session_run_hooks.py:247] loss = 2.097617, step = 700 (7.083 sec)
I0518 05:44:52.096229<PHONE_NUMBER>32896 basic_session_run_hooks.py:680] global_step/sec: 14.0767


Video Titles Generated Incorrectly

Ran unit test on titles for videos, for some reason the first word is not capitalized. (i.e. "test video 123" becomes "test Video 123")

@jonfroehlich, this is fixed in #511.


Elyra images as notebook-server for kubeflow?

This is more of a query: Are there any images provided for running Elyra as a notebook server on existing kubeflow cluster? Something I can use and customize as well ?
Also if not available do you forsee any issues creating a custom image for Elyra? I see this is a very useful plugin and something that can really be helpful in creating the kubeflow pipeline projects.
Thanks for your help, I went through the documentation and could not find any reference.

@kevin-bates thanks for the acknowledgement, I did some reading and looked into code, I am not fully aware of Elyra docker image creation and details of it, I did realize Kubeflow expects a different startup script, I am exploring that option to see if we can get Elyra as a custom image on  kubeflow, I think it would be a great addition for our DS team.

This sounds interesting, thank you for looking into things here.
The elyra project build and tests are driven via a Makefile.  As such, the command make docker-image should build the elyra image.  Once we figure out the startup requirements, we could either plumb appropriate parameters into start-elyra.sh or create an alternate CMD that could be used/overridden in these kinds of scenarios.
cc @akchinSTC for any possible corrections or comments.

I believe @kd303 is trying to use our Elyra image to launch a "notebook instance" when they click "notebooks" on the left panel, rather then talking about "Elyra runtime-images" which is used during component execution.
@kd303 We have extensively tested the image with JupyterHub and Jupyter related components, but I am aware Kubeflow departed from that and we might indeed have some divergency there, having said that, I believe our colleague @tomlyn has done this in the past and can help us here, otherwise, I can take a quick look during the week to see what we might need to update here.

Hi  @lresende I think you may have incorrectly tagged m your response above since I have no knowledge about this issue.

Sorry, @tomlyn.
@Tomcli should be the right one tagged on this issue.

@lresende @tomcli Thanks for your comments, would you have any blog reference or anything that I might refer to do further digging?

Sorry for the delay here, this is next on my priority list and I should get some updates in the next few days.

any update on this? want to use that in kubeflow as well.

@lresende in my simple mind (I am just dabbling with lot of things and nowhere near expert category), I am able to run my jupyter hub notebook on kubeflow, so if I install Elyra plugin as mentioned in github pages should it not work? (pip install --upgrade "elyra<2.0.0" && jupyter lab build)
The difference is the way Elyra's start-elyra.sh [https://github.com/elyra-ai/elyra/blob/master/etc/docker/elyra/start-elyra.sh] and the way Kubeflow expects it. I tried doing this on my local machine but jupyter lab build fails complaining npm is not installled.

@lresende in my simple mind (I am just dabbling with lot of things and nowhere near expert category), I am able to run my jupyter hub notebook on kubeflow, so if I install Elyra plugin as mentioned in github pages should it not work? (pip install --upgrade "elyra<2.0.0" && jupyter lab build)
The difference is the way Elyra's start-elyra.sh [https://github.com/elyra-ai/elyra/blob/master/etc/docker/elyra/start-elyra.sh] and the way Kubeflow expects it. I tried doing this on my local machine but jupyter lab build fails complaining npm is not installled.


Unable to install

After trying to install i got this error        C:\wafw00f-master>python setup.py install
running install
running bdist_egg
running egg_info
writing requirements to wafw00f.egg-info\requires.txt
writing wafw00f.egg-info\PKG-INFO
writing top-level names to wafw00f.egg-info\top_level.txt
writing dependency_links to wafw00f.egg-info\dependency_links.txt
reading manifest file 'wafw00f.egg-info\SOURCES.txt'
reading manifest template 'MANIFEST.in'
writing manifest file 'wafw00f.egg-info\SOURCES.txt'
installing library code to build\bdist.win-amd64\egg
running install_lib
running build_py
installing scripts to build\bdist.win-amd64\egg\EGG-INFO\scripts
running install_scripts
running build_scripts
copying wafw00f.egg-info\PKG-INFO -> build\bdist.win-amd64\egg\EGG-INFO
copying wafw00f.egg-info\SOURCES.txt -> build\bdist.win-amd64\egg\EGG-INFO
copying wafw00f.egg-info\dependency_links.txt -> build\bdist.win-amd64\egg\EGG-INFO
copying wafw00f.egg-info\requires.txt -> build\bdist.win-amd64\egg\EGG-INFO
copying wafw00f.egg-info\top_level.txt -> build\bdist.win-amd64\egg\EGG-INFO
zip_safe flag not set; analyzing archive contents...
creating 'dist\wafw00f-0.9.6-py2.7.egg' and adding 'build\bdist.win-amd64\egg' to it
Traceback (most recent call last):
File "setup.py", line 50, in
test_suite='nose.collector',
File "C:\Python27\lib\site-packages\setuptools_init_.py", line 129, in setup
return distutils.core.setup(**attrs)
File "C:\Python27\lib\distutils\core.py", line 151, in setup
dist.run_commands()
File "C:\Python27\lib\distutils\dist.py", line 953, in run_commands
self.run_command(cmd)
File "C:\Python27\lib\distutils\dist.py", line 972, in run_command
cmd_obj.run()
File "C:\Python27\lib\site-packages\setuptools\command\install.py", line 67, in run
self.do_egg_install()
File "C:\Python27\lib\site-packages\setuptools\command\install.py", line 109, in do_egg_install
self.run_command('bdist_egg')
File "C:\Python27\lib\distutils\cmd.py", line 326, in run_command
self.distribution.run_command(command)
File "C:\Python27\lib\distutils\dist.py", line 972, in run_command
cmd_obj.run()
File "C:\Python27\lib\site-packages\setuptools\command\bdist_egg.py", line 234, in run
dry_run=self.dry_run, mode=self.gen_header())
File "C:\Python27\lib\site-packages\setuptools\command\bdist_egg.py", line 497, in make_zipfile
visit(z, dirname, files)
File "C:\Python27\lib\site-packages\setuptools\command\bdist_egg.py", line 490, in visit
z.write(path, p)
File "C:\Python27\lib\zipfile.py", line 1125, in write
mtime = time.localtime(st.st_mtime)
ValueError: (22, 'Invalid argument')
Any idea how it could be fixed?
OS:Win10

I'm sorry I couldn't reproduce this issue on my test win10/python2.7. The stack trace indicates that something went wrong with the setuptools / zipfile.py.


Translations update from Weblate

Translations update from Weblate for mattermost-languages-shipped/mattermost-mobile_release-1.36.

Hello @weblate,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.


Travis fix

Revert back to precise. Bulld not working in trusty together with openjdk7

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bdd8c01). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #133   +/-   ##
=========================================
  Coverage          ?   58.16%
  Complexity        ?       82
=========================================
  Files             ?        9
  Lines             ?      306
  Branches          ?       24
=========================================
  Hits              ?      178
  Misses            ?      116
  Partials          ?       12

Continue to review full report at Codecov.

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


'scrollToGallery': false breaks plugin

Since you only call preventdefault in the scrollToGallery function, the preview will only work when scrollToGallery is enabled.
When set to false, the browser will just open the link.
SOLUTION: always call preventdefault on the link

Pieter, could you give an example of how to fix this? Thx.

So this is your code. It only works when scrollToGallery is set to true. When set to false, the preventDefault is never called, so the link keeps working as a link:
            /* Scroll to Top */
            if(settings.scrollToGallery) {
                $(this).find('li a').click(
                    function(e) {
                        e.preventDefault();
                    }
                );
            }

The easiest fix for this is just to call preventDefault on all links in the least gallery. I just did this at the bottom of my page, but it can be easily incorporated in the least.js file.
$('.least-gallery li a').click(function(){
    e.preventDefault();
})

Hey Guys,
sorry for the late answer and thank's for your' fix.
It's coming with the next version (2.3) very soon!

no problem, you're welcome!


Sorry why this window appears

Hello, you have updated the mod to the latest version, but not the game. Try open the game on steam, update and then open the mod.

ok

I know the reason. I have a lot of dack, so this happens. Can you create a foolder inside the game in which we place the groups? If we distribute them, this will not happen.

This will make it easier to find dack we are looking for

thank you

https://github.com/pixeltris/YgoMaster/blob/master/Docs/FileLoadError.md


Incorrect result for Expression Predicate

Per @Jackie-Jiang reported in PR #5070, query SELECT COUNT(*) FROM mytable WHERE ArrDelay > CarrierDelay LIMIT 26 result is not correct.
This could be reproduced by method testHardcodedSqlQueries() in class org.apache.pinot.integration.tests.RealtimeClusterIntegrationTest.
Sample exceptions:
java.lang.AssertionError: Value: 0 does not match, expected: 95455, got broker value: 95461, got client value:95461
PQL: SELECT COUNT(*) FROM mytable WHERE ArrDelay > CarrierDelay LIMIT 26
SQL: [SELECT COUNT(*) FROM mytable WHERE ArrDelay > CarrierDelay LIMIT 26]

Current thoughts is that issue should be in ExpressionFilterOperator.

Close this as fixed


The network address displayed is incorrect

Describe the bug
The network address displayed is incorrect
Incorrect prefix:

[x] Interlay
[x] Encointer
[x] Genshiro
[x] Kintsugi Testnet
[x] Composable Finance
[x] Automata

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./upgrade-ui-pr-1884-build-1.1.10-2023-09-06T11-13-43UTC.zip
Validation passed


Update recognition.md by adding link to badges

@qmacro or @je-hal please merge cause for me the option to merge is greyed out ...


Add option to configure URL clicking behavior

Description of the new feature/enhancement
The current behavior of the URL click option is to hold CTRL and left-click the URL. Please, please make this configurable such that we can choose our own method of opening URLs, e.g. left-click without CTRL, double-left-click, right-click, etc. Most importantly, I want to be able to single-left-click-without-holding-CTRL to open URLs.

Proposed technical implementation details (optional)

Sure, I'll tag this up as a part of #1553. Thanks!

Not sure if this a heavy request but how hard would it be to make it open links in incognito(maybe depending on some settings or Alt+left-clicking a link)?

Not sure if this a heavy request but how hard would it be to make it open links in incognito(maybe depending on some settings or Alt+left-clicking a link)?

That's probably impossible. We can only ask the OS to open your browser with a URL. We can't really make any assumptions about what browser that is, with which features.

I for one would be delighted to be able to right click on a link and get a menu to copy or open a link. Gnome terminal behaves this way.

I'd rather just be able to simply left-click URLs as I do in Putty and other terminals.

I use this: https://apps.microsoft.com/detail/9p7fsldkdxh3?hl=en-us&gl=US it asks before opening, and you can configure multiple browsers. LIFE SAVER!


Constructor auto mapping could fail when columnPrefix is specified in the parent resultMap

MyBatis version
3.5.0
Database vendor and version
H2
Test case or example project
contructor_mapping.zip
Steps to reproduce
Run FooMapperTest to find one object Foo with an immutable Bar  from DB
Expected result
get a Foo with a Bar constructed by autoMapping with columnPrefix "b_"
Actual result
An exception thrown with message:
...JdbcSQLSyntaxErrorException: Column "b_B_STR" not found ...

Thank you for the report & repro, @hiirage !
There seems to be a few issues indeed and I will try to fix them, but you should know that columnPrefix is not used in constructor auto-mapping.
Constructor auto-mapping is performed using constructor signature (i.e. the number and the types of arguments).
In your demo, for example, Bar's constructor takes two arguments String and Integer.
MyBatis checks the returned columns in the specified order and use the first two values if they can be converted to String and Integer respectively.
In other words, constructor auto-mapping relies on the column order of your query.
Personally, I recommend explicit mapping for constructor.

@hiirage ,
Both this issue and #1496 should be fixed in the latest 3.5.1-SNAPSHOT.
Let us know if you find any issue!
p.s.
Another issue I noticed is that the columns used for constructor auto-mapping are used again in applyAutomaticMappings().
This means that the values are set to the same properties twice (first via constructor, then via setter (or directly if it's 'final')).
There also is a corner case that causes unintentional mappings.
The failing test is here.

That’s a known issue @harawata. I thought we talked about it last time. I can’t remember if we had said it was necessary for that 2nd level mapping for some reason.

Thank you for the comment, @h3adache !
I must have missed the point and didn't realize the 'same property is set twice' part, then.
If it's known, that should be fine. :)


wanted: google/flatbuffers

Module location
https://github.com/google/flatbuffers
Link to bzlmod issue in the module's repository
https://github.com/google/flatbuffers/issues/7695
Any other context to provide?
No response
Fund our work

[ ] Sponsor our community's open source work by donating a feature bounty

Hi @fardream , I believe, this can be closed now ?

Yes, thanks.


Add a comment explaining that the file kallax.go is auto-generated.

Fixed in https://github.com/src-d/go-kallax/pull/40


Don’t compile in the kstore code introduced in dd822f7

That PR tripled the size of the nexd binary, so we decided to not  compile it in by default to rethink how to accomplish the same goal a  different way.
To build with the kstore code run: NEXODUS_BUILD_TAGS="kubernetes" make clean dist/nexd

I'm OK with this for now. I'm not excited about another dimension in the build matrix, though. I think I'd rather see it refactored into a separate binary eventually. What do you think?

Opened issue #1339 to remind us to drop the bloat at some point so we can enable the kstore by default again.


Underscore "_" in search do not work.

Try searching "register" in any repository
https://github.com/WordPress/WordPress/search?q=register&type=Code
There would be many results, this works fine.
Now search "register_"
https://github.com/WordPress/WordPress/search?q=register_&type=Code
No Result!!!

@valllabh What's odd is that it works from of the time, e.g. https://github.com/search?q=register_&ref=searchresults&type=Code&utf8=✓.
Sorry for posting this here, but this is the top Google search hit associated with this topic and nothing else is as close to as relevant.  Perhaps<EMAIL_ADDRESS>should use Github issues for tracking...

Modify your search like this:
register_
This is one of the ways in which Oracle supports escape characters. Here you define the escape character with the escape keyword. For details see this link on Oracle Docs.
https://github.com/WordPress/WordPress/search?q=register\_&type=Code


[RfR] Migration for DraftRegistration question.extra

Purpose
Having multiple files in a DraftReg requires that the extra field be a list but looking at data on staging most are still dicts. I didn't notice this problem locally which is why this migration wasn't in the original PR.
Changes
'q1' : {                         'q1': {
  'value': 'foo',      ->          'value': 'foo',
  'extra': {}                      'extra': []
}                                 }
'q2': {                          'q2': {
  'value': 'bar',                  'value': 'bar',
  'extra': {                       'extra': [{
    'file': 'data'       ->          'file': 'data'
  }                                 }]
}                                 }

Side effects
Shouldn't be any noticeable side effects other than fixing js errors that might be currently happening on staging.
Ticket
No ticket other than the parent

@samchrisinger Can you review this when you get a chance?

Note to whoever merges this--this will need to be merged into release/0.68.0.
@barbour-em Be sure to not merge develop into this branch.

Yes

One comment, otherwise lgtm

@nicipfeiffer how's this?

@barbour-em Looks good, thank you!

@barbour-em do we need to migrate metadata for existing Prereg registrations?

Thinking about displaying the registration supplement, yes. What should that query look like?
Node.find(
    Q('is_registered', 'eq', True) &
    Q(PREREG_CHALLENGE_METASCHEMA, 'in', 'registered_schema')
)

?

^ Maybe add a `&


Link to release announcement broken

When trying to access the site https://keras.io/keras_3/ from https://github.com/keras-team/keras/releases/tag/v3.0.0, it doesn't exist.
Should this instead link to https://keras.io/keras_core/announcement/?

https://keras.io/guides/migrating_to_keras_3/
this one also broken

Works for me, thanks

Looks good to me

Closing the issue since the issue is resolved. Thanks for confirming.


kubemark: benchmark optimize-utilization scheduler profile

What would you like to be added: Add kubemark benchmark to benchmark performance of the scheduler under the optimize-utilization profile (otherwise vanilla pod specs).
Why is this needed: We want to measure the latencies of relevant extension points (such as scoring) with the optimize-utilization profile in comparison to the default scheduler to quantify improvement.
/sig scheduling
/assign
/cc @ahg-g

/close


Table | character error

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
When using code tags in tables. If the content contains a '|' character, it will be recognized as a list separator
Reproducible demo
https://githubbzjyie-l5a2--3000--fc9e1a28.local-corp.webcontainer.io/docs/intro
Steps to reproduce
|  name   | desc |
| :-----: | ---- | ---- |
| `--help | -h`  | Help |

or

|      name      | desc |
| :------------: | ---- |
| `--help \| -h` | Help |

Expected behavior
Will not be parsed. At least the problem doesn't appear on other documentation engines
Actual behavior
.
Your environment

Public source code:
Public site URL:
Docusaurus version used:
Environment name and version (e.g. Chrome 89, Node.js 16.4):
Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

Hi, I'm not sure what "other documentation engines" you are referring to, but this is the exact behavior of GitHub, which is that you need to escape the pipe to avoid it becoming a column separator:
|      name     | desc |
| :-----------: | ---- |
| `--help | -h` | Help |

|      name      | desc |
| :------------: | ---- |
| `--help \| -h` | Help |

name
desc

`--help
-h`

name
desc

--help | -h
Help

We are using MDX, so any behavior is inherited from this parser and we have no control over it. Because Markdown tables is a GitHub Markdown extension syntax anyway, I think it's best for us to align behavior with GitHub.


Readmes rach kotlin2

Turns out the other ones I had planned for this one are already done! So just two readmes to look at here (just a few minor tweaks since these were recent samples). @hud10837 could you take the first review please?

great thanks! @sorenoid could you take a second review please?


fix: make maps compatible with kernel <= 4.14

In kernel 4.15 and additional parameter was added to allow maps to have names but using this breaks on older kernels. This change makes it so the name is only added on kernels 4.15 and newer.
Symptoms of the problem would show up like
error: map error: failed to create map `IPV4_EVENTS` with code -1: Invalid argument (os error 22)`

and in strace output would look like
bpf(BPF_MAP_CREATE, {map_type=BPF_MAP_TYPE_PERF_EVENT_ARRAY, key_size=4, value_size=4, max_entries=8, map_flags=0, inner_map_fd=0, map_name="IPV4_EVENTS", map_ifindex=0}, 128) = -1 EINVAL (Invalid argument)

After this patch the strace output looks like
bpf(BPF_MAP_CREATE, {map_type=BPF_MAP_TYPE_PERF_EVENT_ARRAY, key_size=4, value_size=4, max_entries=2, map_flags=0, inner_map_fd=0, map_name="", map_ifindex=0}, 128) = 9

Ended up just doing the easy stub to make this work since the kernel_version function isn't used extensively in areas that have tests.

Thank you! 🙏


Axis and scale indicator

I was wondering whether it would be possible to add the possibility of having in  a corner a little scale bar with orientation of the axes and some notion of length.
Maybe it's not required by most people but with the plotter i guess that's not too difficult to add.
Apologies in advance if it exists already and I missed it!

Thank you very much! This is very helpful and the options quite self-explanatory.
Thanks again!


xxd utility reports the same "version" regardless of actual version

When running xxd --version the response is always xxd V1.10 27oct98 by Juergen Weigert, regardless of actual version.
This caused me quite some confusion when trying to figure out why xxd -e file.dat was working fine on some machines but failing on others. The useful -e flag to swap endianness was introduced in more recent versions.
To Reproduce
Detailed steps to reproduce the behavior:

Run xxd -e file.dat on a machine with Vim version 8.1
As expected, a hexdump is produced with endianness swapped
Run xxd -e file.dat on a machine with Vim version 7.4
Usage instructions for xxd are printed, no hexdump is produced.
"Aha, maybe the xxd is an old version"
Run xxd -v on both machines
Both machines print the same version string.

Expected behavior
I expect the xxd utility to show different version strings for binaries that have different capabilities.
Environment (please complete the following information):

Vim versions 7.4 and 8.1 tested
OS: ArchLinux, RHEL 7.7
Terminal: urxvt

That definitly needs a facelift! Thank you @asmunder for raising the issue!
We failed to give proper credit to the contributors to xxd since 1998, my apologies.

The proposed change in #8475 looks good to me.
I agree that it makes sense to switch to using the date of latest change instead of version number.
And thanks for creating xxd ! It has been immensely helpful in many occasions. The most recent one that spurred this issue involves using xxd to process a ~3 TB scientific dataset generated by an older Fortran code, so that it can be read from Python (Numpy).

xxd seems to attract the most hillarous use cases.
Connect Fortran with Python at scale -- great success story! :heart:

Fixed by 8.2.3075.


Made the disabled state for nav more obvious.

Disabled pointer events and changed cursor to default as per #27380

https://github.com/twbs/bootstrap/blob/v4-dev/.stylelintrc#L40-L250

@XhmikosR @mdo I think I've done the requested changes. Sorry for the delay.


[as2_ign_gazebo_assets] Launch external objects with bridges

Feature Description
Update as2_ign_gazebo_assets so it can launch external models that also need to publish data to ROS2. (eg: windmill).
Implementation Considerations
as2_ign_gazebo_assets launch external objects implementation
Files to be changed

launch/launch_simulation.py: Pass models to function world_bridges(). Add launch_arguments into this function to get models to world_bridges.py file.
launch/model_bridges.py: change name to be specific for drones.
src/ign_assets/model.py: Make model class parent class, create two new model subclasses, one for drones, one for external objects. Each of the subclasses will contain specific bridges for each type of model. Also, register each definition of model for each subclass.
launch/world_bridges.py: make it launch so the bridges return by the call to object subclass of model launch with the specific object defined in it.

New simulation_config/ .json key to add object (eg: windmill)
    "objects": [
        {
            "model": "windmill",
            "name": "windmill_0",
            "xyz": [ 0.0, 0.0, 0.0 ],
            "rpy": [ 0, 0, 0.0 ]
        }
    ]

@javilinos is it solved in #145 ?

@RPS98 yes, solved in #145 . Some enhancements can be done in the future though.


Can't keep up!

Sorry everyone for the lack of updates and responses recently. I keep intending to put some time aside to clear the backlog but I never manage to get round to it and the issues keep stacking up, and the number of users keeps increasing. This package is now downloaded from pypi 12k+ times a month. I am very proud of this but unfortunately I have quite a busy and stressful job outside of programming which doesn't really afford me the time or emotional energy to manage this repo as well as it should be.
 This must happen all the time that a personal project grows beyond what one person can manage...?

Hey @ChrisKnott. I should have time to take this on, at least insofar as handling issues, managing releases and maintaining the project.

Thank you @ChrisKnott. This project has helped me a lot.


When will PU24 be available in LCS Shared Assets Library?

Or should I go somewhere else to get it?

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

ID: 4a8f30d9-db2d-d580-6c0d-37f9ac709701
Version Independent ID: e042d248-5402-e627-bb2d-20579ab2b3e8
Content: What's new or changed in Finance and Operations platform update 24 (March 2019) - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-and-ops/get-started/whats-new-platform-update-24.md
Service: dynamics-ax-platform
GitHub Login: @tonyafehr
Microsoft Alias: tfehr

@sericks007 can you review this one?

same question here.

I join to the question...
Thanks ¡

@meeramahabala, can you please answer this question.

I see it's now available in the asset library


examples/audio: The first note often includes noises on Windows

~72d8d7717381298f19f9832a3272030b70514b82 is the culprit?~

Simply, increasing the buffer size fixes this, but, hmm...


Sentence segmentation error case

This is an error case not to forget that causes some trouble with the sentence segmentation.
The document is not CC-BY, referenced here: https://dx.doi.org/10.1063/1.1874292
Here the delinquent paragraph:

With version 0.8.0 and the current master, the process fails:
ERROR [2024-06-11 06:22:00,602] org.grobid.service.process.GrobidRestProcessFiles: An unexpected exception occurs.
! java.lang.StringIndexOutOfBoundsException: begin 592, end 595, length 594
! at java.base/java.lang.String.checkBoundsBeginEnd(String.java:4606)
! at java.base/java.lang.String.substring(String.java:2709)
! at org.grobid.core.document.TEIFormatter.segmentIntoSentences(TEIFormatter.java:1900)
! at org.grobid.core.document.TEIFormatter.toTEITextPiece(TEIFormatter.java:1468)
! at org.grobid.core.document.TEIFormatter.toTEIBody(TEIFormatter.java:1015)
! at org.grobid.core.engines.FullTextParser.toTEI(FullTextParser.java:2648)
! ... 83 common frames omitted
! Causing: org.grobid.core.exceptions.GrobidException: [GENERAL] An exception occurred while running Grobid.

There are two problems (code https://github.com/kermitt2/grobid/blob/694f0ed055e8c9a5d5efdc314ebef78e5e2640cf/grobid-core/src/main/java/org/grobid/core/document/TEIFormatter.java#L2028):

String local_text_chunk = text.substring(pos+posInSentence, theSentences.get(i).end); may crash when the sentence is going over the text length
The if is completely ignored in certain cases, so all the accumulated nodes are dropped.  See below:

                C. dc field dependence of R "T , B rf , B dc , f…

                    <s>As mentioned in Ref.</s>
                    <s>31, properly annealed, bulk Nb TM-TE-mode cavities show large additional rf losses by frozen-in flux with, e.g., at 4.2 K and 2 GHz, R H Ӎ 2 ⍀ H dc / mT for RRRӍ 30, which is described in Eq. ͑3.9͒ by ␤ Ӎ 1 and ␤ Ͻ 10 for RRRտ 200.</s>
                    <s>Those large rf losses by the normal conducting cores of slow AF do not increase with rf field level.</s>
                    <s>,
                        <ref type="bibr" target="#b30">31</ref>
                    </s>

            ```

This is normally fixed in #1131.


Support dustjs v3.0.0

In the peerDependencies section it is a requirement that the version of dustjs can't be above 2.8:
https://github.com/LinkedInAttic/dustjs-helpers/blob/4370ddc27d6b079b6251ae8aaec6059dfdf1d746/package.json#L42
Last year v3.0.0 was released (https://github.com/linkedin/dustjs/releases/tag/v3.0.0), it would be nice if dustjs-helpers could support that.
Trying to install dustjs v3 and dustjs-helpers will now result in:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/dustjs-linkedin
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>- 2.8" from<EMAIL_ADDRESS>npm ERR! node_modules/dustjs-helpers
npm ERR<EMAIL_ADDRESS>from the root project

This is easily fixed by npm install --force or npm install --legacy-peer-deps, but it would be nice to avoid this.

also running into this issue, it would be good to upgrade the peerDependencies of this project to avoid this issue.

Same here, would love to see dustjs and dustjs-helpers to be maintained at least for peerDependencies and security fix !

@jimmyhchan Hello are you going to update this and dustjs ??

Since https://github.com/LinkedInAttic/dustjs-helpers/pull/149 is merged, the code is updated, but waiting for v1.8.0 to be published to NPM before closing this.


handleFetch not working

My sw-precache-config file is listed as below
module.exports = {
    staticFileGlobs: [
        'build/**.png',
        'build/vega-icons/**',
        'build/static/**'
    ],
    swFilePath: './build/service-worker.js',
    templateFilePath: './service-worker.tmpl',
    stripPrefix: 'build/',
    handleFetch: false
}

I am using the template file provided in the src code locally.
Even with the handleFetch flag set to false, I am still seeing the fetch event listener being added to the output service worker.

Can you try removing the output service worker and then regenerate ?

By the way, if you are starting out, then you should try Workbox

@mbj36 - The reason I am choosing the sw-precache over Workbox is because I am using service workers with the CRA build. I believe that is configured using sw-precache?

@mbj36 - By removing the output service worker, I assume you mean remove the swFilePath from the config file? If so, no that does not work. I get the same output as before.

I am facing the same issue. I have just started with CRA and PWA and sw-preache (as @redixhumayun stated) is the best option to customize CRA auto generated service worker.

By the way, if you are starting out, then you should try Workbox

I really hope this project will continue. Workbox is a kitchen-sink with everything in it, sw-precache creates a self-contained script of 11kB before minimizing for me.


Verify the incomplete chain from the first cert

When using the last cert, the following error is encountered from
www.musl-libc.org, currently using a certificate generated by
Let's Encrypt:
    transport_test.go:106: https://www.musl-libc.org/: err not expected but got: Get "https://www.musl-libc.org/": asn1: structure error: tags don't match (16 vs {class:0 tag:6 length:9 isCompound:false}) {optional:false explicit:false application:false private:false defaultValue:<nil> tag:<nil> stringType:0 timeType:0 set:false omitEmpty:false} tbsCertificate @2
--- FAIL: TestTransport (2.43s)

Using the first cert to verify the incomplete chain works correctly for
this case, and appears to do so for others as well.
Probably not great to add a permanent test on the www.musl-libc.org site, but it's the only example I have at the moment for this specific breakage.

Hey @jhuntwork thanks for the contribution! Sorry for the delay in getting back to you, unfortunately it looks like the site you've mentioned has since resolved their chain so I am unable to reproduce, however taking a look I see no harm in starting from the leaf instead (although on the rare chance where an incomplete chain contains its first intermediate but not the rest, this can result in a few extra network calls) I think the extra calls are worth ensuring that this library resolves the chain you've mentioned and will pull your fix in shortly.  I've added a few more test cases in #10, do those look to cover the issue you ran into? If not have you ran into this issue again? I'd love to make sure our test suite covers the scenario you've ran into.

@fcjr Nice, thanks for the more complete tests! Yes, those look like they cover the issue I observed. I haven't seen the issue since then, because I switched to using my forked copy in the meantime. Very happy to switch back once this is merged, though.


AVFoundation/Pipes?

This is a useful application.  I forked it because I need pipes support.  While I'm digging around, I noticed that QTKit is deprecated.  I'm just about done moving it to AVFoundation.  Do you want me to do a pull request when I'm done?

I've recently updated VideoSnap to use the AVFoundation framework and (finally) released v0.0.2 - I hope to have some time free to add new features to the command soon, will update here if I can get piped output to work...


UUID and MD5 generated

how to generate a new uuid or md5 random to save on postgresql, for example:

But the 'knex.raw('uuid_generate_v4()')' does not work, I wonder how can I do to generate a random hash and save automatic when creating a new record?
and it also seems that uuid-v4 can not generate random because of migration that is static and unchanged for each request.

Sorry I'm studying node due to own and my only help is google and github, I thank you for your help.

What is the reason why

But the 'knex.raw('uuid_generate_v4()')'

Doesn't work? Does that work with postgrsql when you try to use it with plain sql wirhout knex?
Try also running your code with DEBUG=knex:* evironment variable set to get more info whats going on with your queries.

@anzileiro, you should not use uuis-v4 package in migrations, but in your models, for example before saving a new model, set its property hash to the value of uuid() and then save it

You need to make sure the extension is available:
knex.raw('create extension if not exists "uuid-ossp"')

then you can do
t.string('hash').defaultTo(knex.raw('uuid_generate_v4()'))

Thanks @fl0w, case closed ☺

If you are using PostgreSQL 13 and want to use UUID V4, you no longer need to use uuid-ossp extension.
So you could write:
await knex.schema.createTable('Users', t => {
  t.uuid('id').defaultTo(knex.raw('gen_random_uuid()'));
  // ...
});

gen_random_uuid() function returns a version 4 (random) UUID.

@rhazegh Ayy thank you for this! The only thing you need to add is a .unique() if the table is going to be referenced elsewhere.
If you don't add the .unique(); you'll end up getting this error like this.

"There is no unique constraint matching given keys for referenced table < TABLE NAME >"

On the off chance anyone else stumbles across this thread, as of Knex 2.5.0 (08 July 2023) you can do
await knex.schema.createTable('Users', (tableBuilder) => {
  tableBuilder.uuid('id').defaultTo(knex.fn.uuid());
  // ...
});

and you can add things like primary() for primary keys


vbscript file to sort rdpwrap.ini

This is vbscript to sort and format an rdpwrap.ini. Use it to sort the different files you have, then use winmerge to compare. This will allow you to come up with one ini file having all appropriate info.
rdpwrap_edit.vbs.txt

This version has better consistency checking and sorts properly.
rdpwrap_edit_v2.vbs.txt

This is a special version that writes each section out as a chunk, substituting line breaks with an @ character. This causes winmerge to compare the entire section rather than trying to match lines between the two files. A composite is made more easily with winmerge, copying only entire sections. After a composite file is made the, line breaks can be put back by substituting for the @ characters.
rdpwrap_edit_group.vbs.txt

This is an ini updater I found in another thread.
rdpwrap_ini_updater.zip


How to build?

Help? I don't know how it should be built into a PHAR?

You can do http://php.net/manual/en/phar.createdefaultstub.phpor http://php.net/manual/en/phar.setstub.php.If you use the latter one, you need to remove the <?php at the start.

Thanks
On 27 Apr 2015 16:42, "Falkirks"<EMAIL_ADDRESS>wrote:

You can do http://php.net/manual/en/phar.createdefaultstub.phpor
http://php.net/manual/en/phar.setstub.php.If you use the latter one, you
need to remove the <?php at the start.
—
Reply to this email directly or view it on GitHub
https://github.com/Falkirks/StubUpdater/issues/1#issuecomment-96716370.


Fix filter fields on bills list

We need to try to adjust the size (side by side) for each field on web devices


Assign Permissions setting ignored on parameters

Describe the bug
When deploying a policy assignment for a definition that uses the assignPermissions attribute of parameters e.g. e920df7f-9a64-4066-9b58-52684c02a091 to set a role assignment on a secondary scope, this scope is ignored.
From documentation:
assignPermissions: (Optional) Set as true to have Azure portal create role assignments during policy assignment. This property is useful in case you wish to assign permissions outside the assignment scope. There's one role assignment per role definition in the policy (or per role definition in all of the initiative's policies). The parameter value must be a valid resource or scope.
https://learn.microsoft.com/en-us/azure/governance/policy/concepts/definition-structure-parameters
To Reproduce
Create a policy assignment for policy e920df7f-9a64-4066-9b58-52684c02a091 with a loganalytics workspace in a scope other than the network security group and the main scope of the policy.
Expected behavior
I would expect the contributor role to be assigned on both the network security group scope and the log analytics workspace scope, but the role is only assigned to the main scope of the policy assignment
Screenshots
N/A
EPAC Version
10.0.2

@techlake - how long has this been available for? I've never seen it, we could add this value as an additionalRoleAssignment?

#562 looks like a regression introduced in v10

Currently, use "additionalRoleAssignments" as the solution (work around). We are looking into fixing this soon.

Thanks @techlake, I indeed use the additionalRoleAssignments for now.

Many Policies do not set this; therefore, we will continue using the additionalRoleAssignments workaround


Add guide for Ubuntu 20.04

20.04 is the new LTS, we should have a guide in place to install it on this.

@svelle I am up for this

@fakela fantastic! Feel free to ping me here or on the community server if you need help at any point or additional info for any of the steps. :)


Font build types

Suggest that we use the following as the initial build artifacts:

unhinted ttf
hinted ttf
woff
woff2

woff and woff2 support is now available broadly across current browsers and my understanding is that it is not necessary to release other web font build types at this stage unless you are targeting old browsers.
woff: https://caniuse.com/#feat=woff
woff2: https://caniuse.com/#feat=woff2
I think that ultimately there will be a need to support:

unhinted otf
hinted otf

as many projects out there will be looking for an approach to build these file types.  Unhinted otf can be built from UFO with fontmake.  This is a simple modification that only requires an adjustment in the output file type flag on the command line.  I haven't come up with an approach to get from unhinted otf to hinted otf (including some level of granularity in the manual modification of any autohinting process) with a FLOSS hinting tool at this stage, so this would require a bit of research work.  For Hack, this has not been necessary because we are dropping otf build support to focus on ttf files.

Be interested in how you approach the file targets.  Will there be specific make target definitions for each build file type?


Multi Root workspaces: reopen after rename acts weird

When unsaved workspace, it seems to reopen an empty file
when saved workspace, it doesn't reopen at all

It seems it is related to renamewithgit because without that setting i don't have any issues.
Also i get this error message:
29.8.2019, 12:15:56 - * fallback: renaming without git from 1.al to Cod50996.GLTtestxx.al
29.8.2019, 12:15:56 - * you might want to set the setting "crs.RenameWithGit" to false for this workspace.
29.8.2019, 12:15:56 - ***
29.8.2019, 12:18:22 - *** Warning
29.8.2019, 12:18:22 - * fatal: not under version control, source=foundation/al/tables/1.al, destination=foundation/al/tables/Tab50996.GLTMyTablexyz.al

Could it be it only happens when the file is not in git yet?  When you would commit it, it does work?

I think so, i have to test that, but i rename on save an therefore this happens for new files all the time at the first save...
So i disabled renamewithgit because i dont realy need it.
But i would prefer to have it enabled.

Well, if you don’t have git, you will always have those “errors”, as it tries to rename with git first.  I didn’t find a way to test if it is a git-source-controlled workspace .. sorry.
From: Marco<EMAIL_ADDRESS>Sent: Thursday, August 29, 2019 1:32 PM
To: waldo1001/crs-al-language-extension<EMAIL_ADDRESS>Cc: waldo<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [waldo1001/crs-al-language-extension] Multi Root workspaces: reopen after rename acts weird (#128)
I think so, i have to test that, but i rename on save an therefore this happens for new files all the time at the first save...
So i disabled renamewithgit because i dont realy need it.
But i would prefer to have it enabled.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/waldo1001/crs-al-language-extension/issues/128?email_source=notifications&email_token=AC4HGTR6FRK3JAYI567IO53QG6XTBA5CNFSM4H4QO752YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5OFODI#issuecomment-526145293, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AC4HGTX7TMOTKQUC2F3IQ5DQG6XTBANCNFSM4H4QO75Q.

Of course we use git! But the renamewithgit is only useful wenn you rename many files in one commit... If you just have to rename a few (which is the most common scenario 99%) it is not needed sunce git figures it out itself.
When i have time i debug the issue and make a PR.

that would be awesome :-).
my time is limited at this moment ..

I created a PR #132  let me know what you think about it.

Thanks so much – I will try to have a look tomorrow evening!
From: Marco<EMAIL_ADDRESS>Sent: Tuesday, September 3, 2019 10:17 AM
To: waldo1001/crs-al-language-extension<EMAIL_ADDRESS>Cc: waldo<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [waldo1001/crs-al-language-extension] Multi Root workspaces: reopen after rename acts weird (#128)
I created a PR #132https://github.com/waldo1001/crs-al-language-extension/pull/132 let me know what you think about it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/waldo1001/crs-al-language-extension/issues/128?email_source=notifications&email_token=AC4HGTVZI5G7HRRSN4RSNRTQHYMRTA5CNFSM4H4QO752YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5XM7AY#issuecomment-527355779, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AC4HGTVQDCFNP2VWOAHFGCLQHYMRTANCNFSM4H4QO75Q.

at this point, I don't find a decent solution, but I have implemented a fix that makes it more stable.. .

From what i see there is no real fix but a warning if you use multiroot?
Thats a bit sad since we know its a async timing problem which can be solved...

I know, sorry, I tried, but I'm not good enough, apparently.  I really hoped your solution would have worked.. .
I do wonder, however, when a file is dragged and dropped in the explorer of VSCode - why isn't it "git mv"-ed either.. .  May be it just isn't possible?
Dunno .. but at least, it's somewhat more stable now.. .

But please - I'd really appreciate you or anyone else to have a go at it (again ;-))...


Create 0000-01-02-vinod-12.md

pull request for second slide modification.

changes approved merging the pull request

merge complete

changes updated and merging into main

verified the changes by reopening branch

reopen branch

successfully merged all changes to main


docs: fix typo

IFPS -> IPFS

nice! could you do add the step for DCO? the command to run is in the Details link by that step of the checks