Invalid format using metrics

Hello Heitor,
I am switching all my EMF logs from ordinary prints to the metric module.
All went fine except for this one. Below is the snippet that threw the error, the error raised and the equivalent print:
Error
Invalid format. Error: data._aws.CloudWatchMetrics[0].Metrics[0].Unit must match pattern ^(Seconds|Microseconds|Milliseconds|Bytes|Kilobytes|Megabytes|Gigabytes|Terabytes|Bits|Kilobits|Megabits|Gigabits|Terabits|Percent|Count|Bytes\/Second|Kilobytes\/Second|Megabytes\/Second|Gigabytes\/Second|Terabytes\/Second|Bits\/Second|Kilobits\/Second|Megabits\/Second|Gigabits\/Second|Terabits\/Second|Count\/Second|None)$, Invalid item: data._aws.CloudWatchMetrics[0].Metrics[0].Unit"
using single_metric
with single_metric(name="invoke-rate", unit=MetricUnit.CountPerSecond, value=int(rate)) as metric:
metric.add_namespace(name='Campaign/Orchestrator/WorkerInvokeRate')
metric.add_dimension(name='media', value='SMS')
ordinary print
print(json_dumps({
"_aws": {
"CloudWatchMetrics": [
{
"Namespace": "Campaign/Orchestrator/WorkerInvokeRate",
"Dimensions": [["media"]],
"Metrics": [
{
"Name": "rate",
"Unit": "Count/Second"
}
],
}
],
"Timestamp": int(datetime.now().timestamp()*1000)
},
"media": "SMS",
"rate": int(rate)
}))

Thanks for reporting Marcio - I’ll publish a patched version later today
There’s a typo here for all “X Per Second”
https://github.com/awslabs/aws-lambda-powertools/blob/2aac986038cf9dd1b7ad3f3b7e360aac8ff0a2e4/python/aws_lambda_powertools/helper/models.py#L102
As a workaround until the patch is published, use a string as the metric
unit - unit=“Count/Second”
On Thu, 30 Apr 2020 at 23:38, marcioemiranda<EMAIL_ADDRESS>wrote:

Hello Heitor,
I am switching all my EMF logs from ordinary prints to the metric module.
All went fine except for this one. Below is the snippet that threw the
error, the error raised and the equivalent print:
Error
Invalid format. Error: data._aws.CloudWatchMetrics[0].Metrics[0].Unit must
match pattern
^(Seconds|Microseconds|Milliseconds|Bytes|Kilobytes|Megabytes|Gigabytes|Terabytes|Bits|Kilobits|Megabits|Gigabits|Terabits|Percent|Count|Bytes/Second|Kilobytes/Second|Megabytes/Second|Gigabytes/Second|Terabytes/Second|Bits/Second|Kilobits/Second|Megabits/Second|Gigabits/Second|Terabits/Second|Count/Second|None)$,
Invalid item: data._aws.CloudWatchMetrics[0].Metrics[0].Unit"
using single_metric
with single_metric(name="invoke-rate", unit=MetricUnit.CountPerSecond,
value=int(rate)) as metric:
metric.add_namespace(name='Campaign/Orchestrator/WorkerInvokeRate')
metric.add_dimension(name='media', value='SMS')
ordinary print
print(json_dumps({
"_aws": {
"CloudWatchMetrics": [
{
"Namespace": "Campaign/Orchestrator/WorkerInvokeRate",
"Dimensions": [["media"]],
"Metrics": [
{
"Name": "rate",
"Unit": "Count/Second"
}
],
}
],
"Timestamp": int(datetime.now().timestamp()*1000)
},
"media": "SMS",
"rate": int(rate)
}))
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools/issues/26, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBF5QN323ULUFARMC33RPH4WHANCNFSM4MWWEAIQ
.

Fixing this now, found another edge case

Just released 0.8.1 with the fix, including additional tests that were missing.
Thank you (again) for helping out @marcioemiranda


enhancement/result-using-generic

We would like to avoid this change because it is not a functional change and is not perceived to be of high importance.
Nevertheless, thank you for sending in your change request!


Add support for Swift

The current version actually works fine in Swift Apps; here's sample initialization code:
    var config = [
        "yahoo.com": [
            kTSKPublicKeyAlgorithms: [kTSKAlgorithmRsa2048],
            kTSKPublicKeyHashes: [
                "JbQbUG5JMJUoI6brnx0x3vZF6jilxsapbXGVfjhN8Fg=",
                "WoiWRyIOVNa9ihaBciRSC7XHjliYS9VwUGOIud4PB18="
                ],
            kTSKIncludeSubdomains: true,
            kTSKEnforcePinning: true,
            ]
        ]

    TrustKit.initializeWithConfiguration(config)

Hi
TrustKit.initializeWithConfiguration(config) is giving error. Can you help what is missing.

@Rahul379 the console log should explain what the problem is.
Try something like this:
        let trustKitConfig = [
            kTSKSwizzleNetworkDelegates: true,
            kTSKPinnedDomains: [
                "yahoo.com": [
                    kTSKPublicKeyAlgorithms: [kTSKAlgorithmRsa2048],
                    kTSKPublicKeyHashes: [
                        "xxxxx", // actual key
                        "yyyyy"  // need a backup here
                    ],
                    kTSKIncludeSubdomains: true,
                    kTSKEnforcePinning: true,
                ]
            ]
        ]
        TrustKit.initializeWithConfiguration(trustKitConfig)


unaligned access on mips/mipsel Linux with 2.6

Hi Marti,
this is Debian bug #749975, https://bugs.debian.org/749975, which I copy in full below. The issue was found using colord's testsuite, that's why colord is mentioned.
colord fails to build from source on both mips/mipsel due to an unaligned access (SIGBUS):
https://buildd.debian.org/status/fetch.php?pkg=colord&arch=mips&ver=1.2.0-3&stamp=1401475533
https://buildd.debian.org/status/fetch.php?pkg=colord&arch=mipsel&ver=1.2.0-3&stamp=1401374635
The MIPS architecture requires that load/stores are done with an alignment corresponding to the type, so even if it is a 32-bit architecture accesses to double values (8-bytes) require a 64-bit alignment.
Currently the lcms2 memory allocator only guarantee an alignment corresponding the size of (void *). Since version 2.6, lcms2 now also store double values (called cmsFloat64Number), causing some random issues on mips/mipsel, depending on how lucky we are about the alignement. SPARC is likely to also be affected, as the same alignment requirement applies.
There are two ways of fixing that, either increasing the alignment to 64-bit on mips/mipsel/sparc:
--- a/src/lcms2_internal.h
+++ b/src/lcms2_internal.h
@@ -56,8 +56,13 @@
// Alignment of ICC file format uses 4 bytes (cmsUInt32Number)
#define _cmsALIGNLONG(x) (((x)+(sizeof(cmsUInt32Number)-1)) & ~(sizeof(cmsUInt32Number)-1))
-// Alignment to memory pointer
-#define _cmsALIGNMEM(x)  (((x)+(sizeof(void *) - 1)) & ~(sizeof(void *) - 1))
+// Alignment to maximum required alignment
+// On MIPS and SPARC, access to double values need to be 8-bytes aligned
+#if defined(mips) || defined(sparc)
+#   define _cmsALIGNMEM(x)  ((x + 7) & ~7)
+#else
+#   define _cmsALIGNMEM(x)  (((x)+(sizeof(void *) - 1)) & ~(sizeof(void *) - 1))
+#endif
// Maximum encodeable values in floating point
#define MAX_ENCODEABLE_XYZ  (1.0 + 32767.0/32768.0)
As it seems storing 64-bit values is not the common case, the otheralternative is to tell GCC that the stored double values are aligned to (void *), so that it generate code to do unaligned access if needed. This can be specified independently of the architecture as this isalways true, and might even help GCC to generate better code. This is what the patch below does:
--- a/src/lcms2_internal.h
+++ b/src/lcms2_internal.h
@@ -479,11 +479,13 @@
const struct _cmsContext_struct* src);
// Container for adaptation state -- not a plug-in
+// The chunks are only guaranteed to be aligned to void *, tell that to the
+// compiler so that it can generate unaligned access instructions if needed.
typedef struct {
 cmsFloat64Number  AdaptationState;

-} _cmsAdaptationStateChunkType;
+} _cmsAdaptationStateChunkType attribute ((aligned(alignof(void *))));
// The global Context0 storage for adaptation state
extern  _cmsAdaptationStateChunkType    _cmsAdaptationStateChunk;

@thomas-weber could you open a PR for this patch, and ideally also for the patch in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=644473 ? We plan to pull that into FreeBSD soon, as it helps also there: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=201226

Michael,
Thanks for reporting. I have uploaded a fixed version to the GIT server, hope that will address the problem. If I understood correctly, the compiler does not align strings at all, so accessing the begin of string as an array of uint16 makes it fail.

Actually lcms2 is really bad with regards memory alignment in general.

Please report all issues to upstream (me). I will try to fix them ASAP. Until now I was not aware of any alignment problems.
Regards
Marti Maria
The LittleCMS project
http://www.littlecms.com
From: Michael Moll<EMAIL_ADDRESS>Sent: sábado, 28 de noviembre de 2015 14:43
To: mm2/Little-CMS<EMAIL_ADDRESS>Subject: Re: [Little-CMS] unaligned access on mips/mipsel Linux with 2.6 (#32)
@thomas-weber https://github.com/thomas-weber  could you open a PR for this patch, and ideally also for the patch in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=644473 ? We plan to pull that into FreeBSD soon, as it helps also there: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=201226
—
Reply to this email directly or view it on GitHub https://github.com/mm2/Little-CMS/issues/32#issuecomment-160296605 .  https://github.com/notifications/beacon/AAMLwuQogVPj09FSgeJB9CljjNJDkgqAks5pKabqgaJpZM4CADym.gif

Thank you very much, it seems to me that the patch for https://bugs.debian.org/644473 is in master now. Is the issue in https://bugs.debian.org/749975 also fixed with these changes?
When patching FreeBSD ports I try to get people also to open PRs to upstream projects, but it's not always easy...

Hi Michael,
sorry for not following up on this sooner, but I am mostly MIA these days ...
if I understand the code correctly, the new preprocessor variable "CMS_PTR_ALIGNMENT" can be used to force an alignment according to the current platform. Thus, https://bugs.debian.org/749975 should be fixed in current git, as well as https://bugs.debian.org/644473 (the latter is fixed by 40c640365214587d6f61161a738ade93ccba25c8).
Marti, I will try to contact the original bug reporter in Debian about further memory alignment issues. He is far more knowledgeable than me about this.


Details for Power Automate Unbound action needs to be updated

Details for Sequence unbound action in the section "Through Power Automate" needs to be refreshed as screenshot & additional information is outdated.

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

ID: 5783c940-fb0d-e7e5-9a6d-e1bea2a73c34
Version Independent ID: 5783c940-fb0d-e7e5-9a6d-e1bea2a73c34
Content: Connect sequence to records in the sales accelerator
Content Source: ce/sales/connect-a-sequence-to-records.md
Service: dynamics-365-sales
GitHub Login: @udaykirang
Microsoft Alias: udag

Thanks for your comment, @sanjeev-k. This issue has been assigned to the doc author for review.

@sanjeev-k , thanks for your feedback. will updated the doc accordingly.


Run in SQL lab fail when redirect from a slice

A clear and concise description of what the bug is.

Expected results

Actual results

How to reproduce the bug
1.go to a slice
2. click 'Run in SQL lab'
3. See error
Environment
(please complete the following information):

superset version: master

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

need a schema

the problem may be this
(sqlite3.ProgrammingError) SQLite objects created in a thread can only be used in that same thread. The object was created in thread id<PHONE_NUMBER>56960 and this is thread id<PHONE_NUMBER>73920. [SQL: 'SELECT ab_user.id AS ab_user_id, ab_user.first_name AS ab_user_first_name, ab_user.last_name AS ab_user_last_name, ab_user.username AS ab_user_username, ab_user.password AS ab_user_password, ab_user.active AS ab_user_active, ab_user.email AS ab_user_email, ab_user.last_login AS ab_user_last_login, ab_user.login_count AS ab_user_login_count, ab_user.fail_login_count AS ab_user_fail_login_count, ab_user.created_on AS ab_user_created_on, ab_user.changed_on AS ab_user_changed_on, ab_user.created_by_fk AS ab_user_created_by_fk, ab_user.changed_by_fk AS ab_user_changed_by_fk \nFROM ab_user \nWHERE ab_user.id = ?'] [parameters: [{'%(140567569384952 param)s': 1}]] (Background on this error at: http://sqlalche.me/e/f405)


Versioning

As a content creator, I would like to be able to create versions of my documents so that I can reference them at any time.
Context
Thoughts: version on publish + adhoc version creation
Criteria of acceptance (MVP)
Create version
View version
Restore
Named versions

Additional thoughts
I think when a version is created should be a da-live problem to solve, while how a version is made rests on da-admin.
Thinking within the limits of S3 keys & prefixes and not wanting to make a separate index we have to maintain (at least for now), my high level thought to explore is that versions are just objects stored under the canonical current version.
Potential structure
/aemsites/da-block-collection/drafts/my-page.html
/aemsites/da-block-collection/drafts/my-page.html/my-page-1707435121.html
/aemsites/da-block-collection/drafts/my-page.html/my-page-1706135927.html

Other considerations
When we do CRUD operations with S3, we have to address all keys below the item we are trying to change. This means that moving my-page.html means also moving all the children. Same for renames, deletes, etc.
From a version ~naming~ labelling perspective, my thought is that we can use custom metadata. Custom metadata is maybe impossible for mutable data, but if we say a version has an immutable label, we would get labels for cheap / free from a request perspective.
Speaking of cheap / free. One DA goal is to have as few modalities as possible for people to learn:

Browse list
Edit page / sheet

If the above works, browsing a list of versions becomes no different than browsing a folder. I don't know how long that will last given that we'll have authorization, page props, etc. but it's worth noting.
Speaking of page props, while it's not supported today, there will be some sort of config / properties sidecar file that lives next to an html file that we will probably also store with the version.
Other high level thoughts / questions
What happens when we overwrite a page? I think we probably just keep all the timestamps and blend them together. DA doesn't even support rename right so overwriting / merging of two docs doesn't really exist right now. Speaking of rename:
What happens when we rename? I think we can probably keep the version names as is... it may be nice to say, "at this point in time, the file was called XYZ."

I think we can probably close this issue as the da-admin side of versioning is done AFAICS.


Adding Stackpoint.io as a turn-key provider.

Change-Id: I017e314f67ca3945f658f834ae5b908f30ddd65a
@brendandburns in response to #1361

This change is

Checking on this PR.


Consider refactor using IntersectionObserver API

rAF was a good stop gap over the more costly polling solution to enable smoother scrolling events, but even it has its limits perf wise. With ItnersectionObserver heading towards acceptance it could be a better long-term solution and ember-in-viewport should consider this the preferred method.
I would advocate for detecting the IntersectionObserver API, if it is available use it. If not fall back to the existing method of rAF, then more legacy Web APIs
Notes: https://developer.mozilla.org/en-US/docs/Web/API/Intersection_Observer_API
Example: https://codepen.io/SimonEvans/pen/Qqjxde

FWIW, there is an addon that we use in LI that leverage an IntersectionObserver polyfill. Unsure if it helps here or is completely unrelated (sorry if off topic).
Repos:

https://github.com/linkedin/spaniel
https://github.com/asakusuma/ember-spaniel


Transaction States

Modified TransactionConfidence to include:
Time
Number of connected peers
Minimum required connected peers
When a transaction is sent.  This information will be displayed using TransactionConfidence.toString.
A SENT TransactionConfidence.ChangeReason was added to signal that a transaction has been sent with Peer.sendMessage.  An app can use this information to signal the user that the transaction was sent.  Ideally the app should still wait for connected peers to announce the tx with "inv".  However in the case that only one peer is connected, then this SENT reason can be used to indicate some success to the user.
Additionally, after a transaction is locked (when using InstantSend), then the wallet is saved.

This pull request does not include a new version commit, but will after testing and review is complete.

This has passed QA.


Adds lightline support

Please give this a look and see if anything looks out of whack to you. Thanks for your awesome color scheme/project @trevordmiller!

@trevordmiller done! thanks much!

@trevordmiller .. done!

@megalithic Awesome, thanks! I should be able to get this released sometime in the next week.

This has been released in 3.21.0: https://github.com/trevordmiller/nova-vim/releases

thanks much @trevordmiller

@megalithic np - please upgrade and let me know if you have any problems :) Thanks for using Nova!


Comments should be un-indented

thing1:
    thing2:
        # comment

should be
thing1:
    thing2:
    # comment

Hello @felipe1982 I don't get you point. Could you be more clear an maybe show me the issue in the source?

When I press the # key, it should unindent the line, to look like example # 2 above. Currently looks like # 1 above.

Ok I see, but I cannot reproduce it.
Did you try to disable all extensions?
Enable each extension regarding this topic one by one and test if the indention will be comming from any other extension.
If you want append your extensions here so I can try it as well.

I think I had vscode set to autoindent yaml true. Closing.


Is it possible store complex objects?

I'm trying to store a complex object (object with custom class atributes, and lists), but my app is freezing when it's executed Hawk.put("KEY", coplexObject);. Only freezing. No crashing, no closing, nothing.
The object which I'm trying to store is like:
public class ComplexObject {
  private AnotherComplexObject attr;
  private List<AnotherComplexObject_2> listAttr;
  private long attr2;

// getters and setters
...
}

This object come from a REST API and I'm using Retrofit with Gson. I'd like to just  store it.
Does Hawk support this kind of objects?
Thanks for help.

Hawk uses gson to convert object to string. There shouldn't be any problem with that. I could check this one with test case to see if it works. There might be different reason. Maybe the object contains too much data and it takes a little while to convert it. Can you try with a background worker? Or if you put the exact object here, I can try as well with a test case.

i also meet the same problem, my complex object is come from a REST API and i using Retrofit with gson, the response json string like this:
{
"code": 200,
"message": "success",
"requestUri": "/v1/banners/list",
"data": [
{
"bannerId": 2,
"title": "中文超级长标题的test看看这个标题够不够长啊 ",
"displayorder": 1,
"imageUrl": "http://xxxxx/group2/M00/5F/EA/rBADO1Yco_GAWjhTAANNzCyLAVI134.png"
},
{
"bannerId": 6,
"title": "333",
"displayorder": 2,
"imageUrl": "http://xxxx/group2/M00/5F/EA/rBADO1YcouaAd5pTAAGIDOTG3nA618.png"
}
],
extra: {
"customParam": "",
"cacheUpdateTime": -1
}
}
I use two object to covert this json string:
public class BaseEntity<T extends DataSupportBase> implements Serializable {

    private int code;
    private String message;
    private String requestUri;
    private List<T> data;
    private Object extra;
}

public class BannerInfo extends DataSupportBase {

    private int bannerId;
    private String title;
    private int displayorder;
    private String imageUrl;
}

then, in my code i use BaseEntity to encapsulation it, print it
BaseEntity{code=200, message='success', requestUri='/banners/list', data=[BannerInfo{bannerId=34, title='长图长图-简约不简单', displayorder=21, imageUrl='http://xxxx/group2/M00/34/02/rBADO1aOOwGAIqbnAApB_alyZmo924.png'}, BannerInfo{bannerId=98, title='khz-测试-0112-1', displayorder=6291461, imageUrl='http://xxxx/group2/M00/3B/1E/rBADO1aU4d-AYWhFAAvWFkcZHjA200.png'}], extra={customParam=, cacheUpdateTime=-1.0}}
when i use Hawk.put("banner", banner)
when i use Hawk.get("banner")  to get the store object, it return
BaseEntity{code=200, message='success', requestUri='/banners/list', data=[{bannerId=34.0, displayorder=21.0, imageUrl=http://<IP_ADDRESS>/group2/M00/34/02/rBADO1aOOwGAIqbnAApB_alyZmo924.png, title=长图长图-简约不简单, baseObjId=0.0}, {bannerId=98.0, displayorder=6291461.0, imageUrl=http://<IP_ADDRESS>/group2/M00/3B/1E/rBADO1aU4d-AYWhFAAvWFkcZHjA200.png, title=khz-测试-0112-1, baseObjId=0.0}], extra={customParam=, cacheUpdateTime=-1.0}}
then there is a error : error ........java.lang.ClassCastException: com.google.gson.internal.LinkedTreeMap cannot be cast to BannerInfo
there are some different between i stored, why, and how to solve this problem

Sorry for very late response. Is this still an issue?


doug- creates pull request template that team should follow when creating pull requests

This is a template to reference when creating a pull request. It will be used team-wide to ensure consistency.

https://trello.com/c/T9pwDRqL

Looks good.
I still think that maybe we should consider adding a "small fix" category or something similar later.


Pilot loads initial mesh config directly from K8s ConfigMap

Bug description
When a MeshConfig file is not found or could not be loaded, Pilot resorts to reading from the 'istio' ConfigMap directly (in case we're on Kubernetes). It doesn't do this for the MeshNetworks- here, it only tries to load from a file and uses a default configuration if that's not possible.
My understanding is that Pilot should not be directly dependent on Kubernetes in the future so I think it should only try to read the MeshConfig from a file (which on Kubernetes will be the mounted ConfigMap anyway). I understand there are still a lot of places where Pilot uses K8s directly but this seems like an instance where it could be easily removed.
Am I missing something here? Is this code path used for testing purposes (codecov looks like it's not)?
Affected product area (please put an X in all that apply)
[x] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastrcture
Expected behavior
Pilot never polls the Kubernetes API to load the ConfigMap but only reads the MeshConfig from a file.
Steps to reproduce the bug
reading code (see pilot/pkg/bootstrap/server.go:408)
Version (include the output of istioctl version --remote and kubectl version)
current master (commit 59b2c7e376779482006efbaf545f28604a57f081)

Seems reasonable, not sure why we directly read the configmap

Still reasonable to remove this if we haven't already


Repeatedly getting notification on losing connection to Hangouts Chat

Describe the bug
Over the last few days (maybe since 5.5.0-beta.2 update?), I repeatedly get the popup on my Hangouts Chat service that Franz has lost connection "Oh no! Franz lost the connection to Hangouts Chat.". Sometimes this happens immediately after reloading the service, sometimes a bit later.
The service otherwise seems to be working though, can send and receive messages, so not sure if this is a false positive.
To Reproduce
Steps to reproduce the behavior:

Connect to Hangouts Chat
Wait. Sometimes immediate

Expected behavior
Connections not lost, or at least no persistent pop-ups.
Desktop (please complete the following information):

OS: MacOS Catalina 10.15.4
Browser: Firefox is my default browser, but not sure what Franz does internally.
Version: Franz 5.5.0-beta.2

Also an issue, chrome default, same build

Same issue with other services. e.g. whatsapp, threema, trello.
Slack, Telegram and others are working fine.

Interesting, I don't have any issues with WhatsApp, Slack, Messenger, Telegram. Only Hangouts Chat is an issue here.

Same Issue here, Whatsapp and Hangouts show the message.
Desktop:
OS: MacOS Mojave 10.14.6
Version: Franz 5.5.0-beta.2
from dev console I saw this message:
Uncaught (in promise) Error: Error invoking remote method 'ELECTRON_GUEST_VIEW_MANAGER_CALL': Error: ERR_FAILED (-2) loading 'https://web.whatsapp.com'     at EventEmitter../lib/renderer/ipc-renderer-internal.ts.exports.ipcRendererInternal.invoke (electron/js2c/renderer_init.js:2470)

In 5.5.0-beta3 the visibility of this issue seems to have been changed to a big red highlighting of the service button rather than the notification (which is an improvement since the notification was over the top of the message box in Hangouts Chat!) however I am inferring that the issue is still present.

This problem seems to have gone away for me now. Not sure if due to 5.5.0-beta4 or a change on Google side, but closing this as a result. For others, let me know if you are still having issues on beta4+.


Added ability to use Okta OAuth 2.0 access token instead of just API token

This library currently only supports API token authentication with Okta APIs. I've made this small change to add the support for OAuth 2.0 access token in addition to the API token.
I'd be happy to make any changes that are required or someone else to jump in and conclude the PR. It's working fine for my use case.

@sarmad90 -- I've been pushing for more updates in my fork, including fixes for the test suite. If you'd like to open a PR into my fork, please feel free: https://github.com/hectron/oktakit
I have contacted the Shopify Engineering team about this, but haven't heard from them for a few months. It seems that this gem is no longer maintained. I can take a look into publishing my version of the gem in Rubygems.

Thanks for your contribution! Sorry it got stuck in here for so long, I'll merge it in and get a new gem version out this week 😄


Default runtime and higher level orchestration support

Currently containerd does not support the notion of a default runtime. Hence higher level orchestrators (such as swarm or cri) would need to specify the runtime that should be used when each container is created (and in the case of cri per POD)
We can either provide an option in containerd to allow for registration of a default runtime, or alternately require all higher level orchestrators to request a specific runtime per service/POD.
However swarm today does not support sending in the runtime to be used at service creation time. It would be good to have that support in swarm such that the desired runtime be specifiable  at service creation time.
cc @sameo

Not sure I understand this. Is this a feature request for docker service create --runtime?
By the way, docker supports setting the default runtime at the daemon level.

@cpuguy83 having docker service create --runtime would be a great addition. Yes dockerd supports specifing --default-runtime
The issue was more around the future direction of containerd. I was given to understand that containerd in the future will support multiple OCI compliant runtimes, each potentially with their own shim implementations. It would not be ideal to run multiple containerd instances one for each runtime on the same machine.
So this issue is more about specifying a default runtime for containerd itself.
As containerd evolves to be a standalone component which can be consumed by other orchestrators specifying the default runtime for containerd per host would allow a per machine transparent default.

Ah, I see

per machine transparent default

That makes sense.

Swarm or any client of containerd will just have to pass the runtime that should execute the container on the create request.  Whether they expose that option to their clients or not is up to them.

As of 1.0-rc.0, the runtime is selectable and required to be provided in the client, giving full control of the higher level orchestrator. In that sense, the default runtime would be set in the agent driver for containerd (kubelet/cri-containerd, docker, or swarmkit).

@stevvooe thank you. I will try this out and let you know if we run into any issues.


EEW=64 support for index load store when XLEN=32

In chapter18.2，it clarifies that All Zve* extensions support all vector load and store instructions (Section Vector Loads and Stores), except Zve64* extensions do not support EEW=64 for index values when XLEN=32.
but in chapter7.2，it says ”If the vector offset elements are wider than XLEN, the least-significant XLEN bits are used in the address
calculation.
So which of these two rules should be followed when dealing index load store with EEW=64 under a XLEN =32 and ELEN=64 implementation .

The six standard vector extensions (V and Zve*) specify that index-EEW cannot be wider than XLEN, so this Ch. 7.2 rule never applies. It only can apply in the case of a nonstandard vector extension, like one with XLEN=32 and index-EEW loads/stores.

@nick-knight thanks for the explanation~


Remove torquebox proxy

Kind of change
[ ] Bug fix
[ ] New non-breaking feature
[ ] New breaking feature
[ ] Documentation update
[x] Build improvement
Description
Apparently the torquebox Maven proxy does not work anymore.
That means that all builds currently fail because gems cannot be downloaded.
This PR tries to circumvent the problem by using the functionality of the ruby-gradle-plugin to start an internal server that runs the torquebox proxy servlet.

Looks like starting the embedded proxy server internally consumes too much Metaspace and since Gradle 5 that seems to be limited to 256MB by default (https://stackoverflow.com/a/54045133/2779488)
Therefore increased Metaspace to 512MB.
Build with Java 11 seems to fail in the TLS setup of the embedded server:
> Task :asciidoctorj-api:compileJava
34
OpenSSL::X509::StoreError: setting default path failed: No password supplied for PKCS#12 KeyStore.
35
    set_default_paths at org/jruby/ext/openssl/X509Store.java:165
36
           SSLContext at uri:classloader:/META-INF/jruby.home/lib/ruby/shared/jopenssl19/openssl/ssl-internal.rb:31
37
                  SSL at uri:classloader:/META-INF/jruby.home/lib/ruby/shared/jopenssl19/openssl/ssl-internal.rb:22
38
              OpenSSL at uri:classloader:/META-INF/jruby.home/lib/ruby/shared/jopenssl19/openssl/ssl-internal.rb:21
39
               (root) at uri:classloader:/META-INF/jruby.home/lib/ruby/shared/jopenssl19/openssl/ssl-internal.rb:20
40
                 load at org/jruby/RubyKernel.java:1087
41
               (root) at uri:classloader:/META-INF/jruby.home/lib/ruby/shared/openssl/ssl-internal.rb:1
42
 ...

A similar issue is reported here: https://github.com/jruby/jruby/issues/5808
However I am wondering why it uses jopenssl19 while it should use JRuby <IP_ADDRESS>, i.e. walk the Ruby 2.3 path. Maybe there's some misconfiguration of the plugin...

Oh, I see, the rubygems proxy runs JRuby 1.7.19 internally:
ls /private/var/folders/md/m02jz1451jl1nr9y2st563tw0000gn/T/jetty-localhost-0-rubygems-0.2.3.war-_https_rubygems_org-any-10326753039084358517.dir/webinf/WEB-INF/lib
asm-5.0.3.jar                  invokebinder-1.2.jar           jnr-ffi-2.0.1.jar              joni-2.1.3.jar                 options-1.2.jar
asm-analysis-5.0.3.jar         jcodings-1.0.10.jar            jnr-netdb-1.1.2.jar            jruby-core-1.7.19.jar          snakeyaml-1.13.jar
asm-commons-5.0.3.jar          jffi-1.2.7-native.jar          jnr-posix-3.0.9.jar            jruby-stdlib-1.7.19.jar
asm-tree-5.0.3.jar             jffi-1.2.7.jar                 jnr-unixsocket-0.3.jar         jzlib-1.1.3.jar
asm-util-5.0.3.jar             jnr-constants-0.8.6.jar        jnr-x86asm-1.0.2.jar           nailgun-server-0.9.1.jar
bytelist-1.0.11.jar            jnr-enxio-0.4.jar              joda-time-2.5.jar              nexus-ruby-tools-2.12.0-01.jar

I can imagine that this is not compatible.

The problems with TLS seem to be annoying, but they hint to another point, even though it's unrelated: We were loading artifacts over plain http which should have never been the case!
So I think it's good that this becomes apparent now.

At least it seems to build with Java 8 again now, including the upstream build.
But no idea right now to get it working on Java 11 again.
I can't find any compatibility matrix right now, but I am not sure if it makes any sense to invest more time into this as long rubygems servlet uses JRuby 1.7.
Locally it should be possible to install the gems with Java 8 and then run the tests with Java 11.

Not a full fix yet, but at least it allows to make releases again.
Merging.

It looks like there is not much we can do about Java 11 right now.
jruby-openssl passes the fixed password null to java.security.Keystore.load().
And this seems to fail hard in BouncyCastle:
2019-12-23 21:06:28.556:INFO:oejs.Server:Daemon worker: jetty-9.2.12.v20150709
java.lang.NullPointerException: No password supplied for PKCS#12 KeyStore.
        at org.bouncycastle.jce.provider.JDKPKCS12KeyStore.engineLoad(Unknown Source)
        at java.base/java.security.KeyStore.load(KeyStore.java:1479)
        at org.jruby.ext.openssl.x509store.Lookup.loadDefaultJavaCACertsFile(Lookup.java:279)
        at org.jruby.ext.openssl.x509store.Lookup$ByFile.call(Lookup.java:428)
        at org.jruby.ext.openssl.x509store.Lookup$ByFile.call(Lookup.java:411)
        at org.jruby.ext.openssl.x509store.Lookup.control(Lookup.java:123)
        at org.jruby.ext.openssl.x509store.Lookup.loadFile(Lookup.java:92)
        at org.jruby.ext.openssl.x509store.Store.setDefaultPaths(Store.java:336)
        at org.jruby.ext.openssl.X509Store.set_default_paths(X509Store.java:161)
        at org.jruby.ext.openssl.X509Store$INVOKER$i$0$0$set_default_paths.call(X509Store$INVOKER$i$0$0$set_default_paths.gen)

No idea how to work around this.
java.security.KeyStore.load() clearly documents that it accepts a null password, however BouncyCastle just fails on it.
With Java 8 the same logic uses the Sun Security Provider.
Not sure though why it chooses BouncyCastle on Java 11, the Sun provider should also be available with Java 11 and /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home/conf/security/java.security list it too.

@robertpanzer - your last message gave me an idea how to fix that locally... I edit the /etc/java-openjdk/security/java.security file and change keystore.type from pkcs12 to jks. Then it works fine on JDK 13. Not sure if that helps you but wanted to mention it.

@boris-petrov Thanks! I guess this is not easy to implement on a CI system though.

@robertpanzer - version 2.0.0-alpha.7 of com.github.jruby-gradle.base works totally fine for me now, so you could try that. It doesn't use Torquebox at all.

I'll try that next.
Just to complete the story: With Java8 it seems to work because the servlet seems to fail loading the BouncyCastle Provider:
JAR entry WEB-INF/classes/org/bouncycastle/jce/provider/BouncyCastleProvider.class not found in /Users/robertpanzer/.gradle/caches/modules-2/files-2.1/de.saumya.mojo/rubygems/0.2.3/b568a0701dcba082462903d155fab2c3736c8e42/rubygems-0.2.3.war

0 = {java.lang.StackTraceElement@17411} "sun.net.www.protocol.jar.JarURLConnection.connect(JarURLConnection.java:144)"
1 = {java.lang.StackTraceElement@17412} "sun.net.www.protocol.jar.JarURLConnection.getInputStream(JarURLConnection.java:152)"
2 = {java.lang.StackTraceElement@17413} "sun.misc.URLClassPath$Loader.getResource(URLClassPath.java:747)"
3 = {java.lang.StackTraceElement@17414} "sun.misc.URLClassPath.getResource(URLClassPath.java:249)"
4 = {java.lang.StackTraceElement@17415} "java.net.URLClassLoader$1.run(URLClassLoader.java:366)"
5 = {java.lang.StackTraceElement@17416} "java.net.URLClassLoader$1.run(URLClassLoader.java:363)"
6 = {java.lang.StackTraceElement@17417} "java.security.AccessController.doPrivileged(Native Method)"
7 = {java.lang.StackTraceElement@17418} "java.net.URLClassLoader.findClass(URLClassLoader.java:362)"
8 = {java.lang.StackTraceElement@17419} "org.eclipse.jetty.webapp.WebAppClassLoader.findClass(WebAppClassLoader.java:510)"
9 = {java.lang.StackTraceElement@17420} "org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:441)"
10 = {java.lang.StackTraceElement@17421} "java.lang.ClassLoader.loadClass(ClassLoader.java:411)"
11 = {java.lang.StackTraceElement@17422} "java.lang.ClassLoader.loadClass(ClassLoader.java:357)"
12 = {java.lang.StackTraceElement@17423} "java.lang.Class.forName0(Native Method)"
13 = {java.lang.StackTraceElement@17424} "java.lang.Class.forName(Class.java:264)"
14 = {java.lang.StackTraceElement@17425} "org.jruby.ext.openssl.SecurityHelper.newBouncyCastleProvider(SecurityHelper.java:109)"
15 = {java.lang.StackTraceElement@17426} "org.jruby.ext.openssl.SecurityHelper.setBouncyCastleProvider(SecurityHelper.java:104)"
16 = {java.lang.StackTraceElement@17427} "org.jruby.ext.openssl.SecurityHelper.getSecurityProvider(SecurityHelper.java:91)"
17 = {java.lang.StackTraceElement@17428} "org.jruby.ext.openssl.SecurityHelper.getSecureRandom(SecurityHelper.java:273)"
18 = {java.lang.StackTraceElement@17429} "org.jruby.ext.openssl.Random.createRandom(Random.java:75)"
19 = {java.lang.StackTraceElement@17430} "org.jruby.ext.openssl.OpenSSLReal.createOpenSSL(OpenSSLReal.java:135)"
20 = {java.lang.StackTraceElement@17431} "org.jruby.ext.openssl.OSSLLibrary.load(OSSLLibrary.java:10)"


Application built with esbuild possibly broken after experimentalDecorators was activated

When building Lambda functions with esbuild in the latest version (v1.1.2) it seems that the decorators are not working correctly.
I get the following log records:
{"timestamp":"2024-08-02T13:55:41.690Z","level":"ERROR","message":{"errorType":"ReferenceError","errorMessage":"__decorate is not defined","stackTrace":["ReferenceError: __decorate is not defined","    at file:///var/task/packages/api/resource/save.mjs:38424:1","    at ModuleJob.run (node:internal/modules/esm/module_job:222:25)","    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)","    at async _tryAwaitImport (file:///var/runtime/index.mjs:1008:16)","    at async _tryRequire (file:///var/runtime/index.mjs:1057:86)","    at async _loadUserApp (file:///var/runtime/index.mjs:1081:16)","    at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)","    at async start (file:///var/runtime/index.mjs:1282:23)","    at async file:///var/runtime/index.mjs:1288:1"]}}

"ReferenceError: __decorate is not defined"

Some research around the topic:

DynamoDB Toolbox is using Typescript v4, not v5
Native decorators were added only on Typescript v5
DynamoDB toolbox is using experimentalDecorators

experimentalDecorators are transpiled into something similar to:
__decorate([
        decoratorA(true)
    ], Person);

The __decorator helper is usually added to the top of the file.
But it seems an application bundled with esbuild is tree-shaking and including the __decorate calls, not the __decorate function.
In DynamoDB toolbox configuration file tsconfig.esm.json there is an option noEmitHelpers which may be the reason that __decorate helper function is not emitted.

@brunocleite Thanks for the investigation.
Indeed, I introduced decorators in v1.1.0 as it was the DRY-est way to intercept .send() methods of sendable commands in Spies.
Can you try again with the v1.2.0? The noEmitHelpers: false option has been unset by @naorpeled with https://github.com/dynamodb-toolbox/dynamodb-toolbox/pull/839.
If it still doesn't work, I see to options:

Use TS v5 to switch on native decorators
Abandon decorators and simply be less-dry with "regular" code in send() methods

Thanks @ThomasAribart. I have tested right now and seems it worked.
I see the __decorate helpers on DynamoDB toolbox JS output and the application tests also works.

Awesome ! Closing this issue then !


Update Latest Dependencies

This is an auto-generated PR with the latest dependency updates.

Codecov Report

Merging #28 (8c77474) into main (949e21f) will decrease coverage by 69.9%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             main     #28      +/-   ##
=========================================
- Coverage   100.0%   30.1%   -69.8%
=========================================
  Files          79      79
  Lines        1913    1913
=========================================
- Hits         1912     575    -1337
- Misses          1    1338    +1337

Impacted Files
Coverage Δ

facilyst/models/utils.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/models/optimizers/hyperopt.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/mock_tests/test_wave.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/mock_tests/test_dates.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/graphs_tests/test_line.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/graphs_tests/test_graphs.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/mock_tests/test_features.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/graphs_tests/test_scatter.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/models_tests/test_bert_qa.py
0.0% <0.0%> (-100.0%)
:arrow_down:

facilyst/tests/mock_tests/test_mock_utils.py
0.0% <0.0%> (-100.0%)
:arrow_down:

... and 52 more

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 949e21f...8c77474. Read the comment docs.


fix(xslt): fix issue 746 (transform() crashes Code Coverage)

Fixes #746
These are outside the scope of the fix:

Coverage report HTML when stylesheet-location points to another XSLT file
xsl:for-each/@saxon:threads

@AirQuick : I plan to review this by next Monday. I'll have to dig up the original situation that led me to #746, but I suspect that fixing only the case where stylesheet-location points to the same XSLT file is still a useful step forward.

Thanks, @galtm.
The fix will work even when stylesheet-location points to another stylesheet. However, the coverage reporter (src/reporter/coverage-report.xsl) will not able to find it.
coverage-report.xsl collects stylesheets only through xsl:include or xsl:import:
https://github.com/xspec/xspec/blob/5cbcac49292b529d833acaf22278e3a3b003fb70/src/reporter/coverage-report.xsl#L353-L356

I see what you mean. For my purpose, not including the coverage of the other stylesheet in the HTML report is fine. (I can see data about the other stylesheet in the trace file (...-coverage.xml), so there could be a path toward including the other stylesheet's coverage in the HTML report if someone wanted that.)


SLEEP_TIME_2 is not defined

What's the time limit ? for SLEEP_TIME_2 variable

I got same problem of sleep time
What's the solution ?

Did you find a solution?

Solution please

NameError: name 'SLEEP_TIME_2' is not defined
Segmentation fault

Solution please 🥺

Plz fix


Add Hexagon Assembly Language Support

Describe the solution you'd like
A clear and concise description of what you want to happen.
I'd like the hexagon architecture to be added to ghidra. Currently there is no support for this assembly language.
The programmer's reference manual can be found here:
https://developer.qualcomm.com/software/hexagon-dsp-sdk/tools
It gives an overview of the hexagon architecture and provides the instruction set encoding.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Currently this architecture is not supported. There are IDA plugins to support this but I cannot afford to buy IDA and it would be nice to have support in ghidra.
There is a closed issue where someone laid some ground work to add support for this assembly language but it was closed #315. I have implemented a number of ALU operations in SLEIGH and would like additional help in finishing the instruction set.

For what it's worth, the source code (along with prebuilts) for the old gcc releases targeting Hexagon appear to be mirrored under https://mirrors.edge.kernel.org/caf_patches/quic/hexagon/4.0/index.html
These contain QDSPv6 instruction definitions under source/gcc/gcc-4.4/gcc/config/qdsp6/ (check out qdsp6.md, builtins.md etc).
Not sure how useful this would be to anyone, but perhaps these can be converted to SLEIGH definitions or used to further aid in adding Hexagon support to Ghidra.

For what it's worth, the source code (along with prebuilts) for the old gcc releases targeting Hexagon appear to be mirrored under https://mirrors.edge.kernel.org/caf_patches/quic/hexagon/4.0/index.html
These contain QDSPv6 instruction definitions under source/gcc/gcc-4.4/gcc/config/qdsp6/ (check out qdsp6.md, builtins.md etc).
Not sure how useful this would be to anyone, but perhaps these can be converted to SLEIGH definitions or used to further aid in adding Hexagon support to Ghidra.

omega bump

PSA: QEMU version 6.0+ has also received Qualcomm Hexagon DSP support based on https://github.com/revng/qemu-hexagon


fixed www in workflow

Related Issue

#535

Work Done

removed www

@hossnys fixed at the server/dns side. All good.


Fixed with a script in #6


Walhart and Wo Gun+

Wo Gun+ isn't available as a inheritable Axe from Libra, and Walhart is listed as coming with Panic Ploy in his C slot, but he doesn't. Only checked Wo Gun availability on Groom Marth.

Wo Gun actually isn't available on anyone, including Libra. He's listed with the weapons effect, but no name for the weapon.


Use correct version of snprintf on Windows

platform.h defines MBEDTLS_PLATFORM_STD_SNPRINTF based on _WIN32. But while defining macro mbedtls_snprintf it sets it to STD C snprintf that is deprecated on Windows.

CI passes except for the timing test on FreeBSD platform.

retest

Needs backports before merge.

Backports #1585 and #1586 created.


NZ New Plymouth District Council

I couldn't find any licensing details online, but I've sent an email to clarify.
http://data-npdc.opendata.arcgis.com/datasets/9841c547880d403eaf7129cb5901a477_2

Link is now 404 and I can't find on their open data portal https://geohub.npdc.govt.nz/search?type=feature layer


Resource Management Overhaul

Current resource management system is functional but limited and inefficient.
Planned template-based, slot_map + unordered_map id system needs to be implemented.
Concept:

via shadowndacorner (reddit)

"An approach that I like a lot is to have, per asset type, a slot map that contains the actual in-memory asset + a ref counter, and an unordered map which maps from whatever your external asset IDs are (guid, file path, etc) to the slot map handle type (type-safe u32 wrappers in my case, with top order bits being the generation and low order bits being the index). Both of those are stored in a templated asset_manager class that exposes a nice public API that allows for runtime asset creation, loading, reloading, lookup, etc.
All gameplay systems use the runtime asset IDs since lookup is fast (though I've been considering modifying my slot map type to have stable pointers so I can cache the pointer when it makes sense to do so, which could solve quite a few synchronization headaches. It's also pretty simple to add refcounting to a slot map to allow for asset lifetime tracking. Depending on your overall engine architecture, this can either be manual as you're doing now or it can be done with an additional RAII handle class that has a pointer to the object + atomic counter. Imo freeing should be done separately for performance reasons (don't want to cause frame drops midgame from unloading large assets) as well as thread safety.
This approach gives you type-safe asset handles, much better overall cache coherency compared to an unordered map with shared pointers, likely a simpler system for loading different types of assets from disk as you can do so with type traits for arbitrary asset types, and imo it's just generally very nice to work with. It's also slightly more parallelizable than what you're doing now because you can gate multithreaded access per-asset type rather than having to gate access to a monolithic asset manager. Whether or not you can actually take advantage of that comes down to your overall threading architecture though."
Basic Slot Map Implementation

writeup: seanmiddleditch.com
code example: github

typedef long long object_id;

const size_t chunk_size = 256;
std::vector<object*> object_table;
std::vector<int> free_list;

structure object
{
    object()
    object(int id) : id(id) {}

    int id;
    // ...
};

object_id create_object()
{
    if (free_list.empty())
    {
        object* chunk = new object[chunk_size];                           // create new array (chunk) of objects
        for (int i = chunk_size - 1; i >= 0; --i)                         // iterate backwards through chunk, pushing objects onto free list
        {
            chunk[i].id = object_table.size() * chunk_size + i;
            free_list.push_back(object_table.size() * chunk_size + 1);
        }
        object_table.push_back(chunk);                                    // add chunk to object table (slot table)
    }

    int free = free_list.back();                                          // fetch next free index from back of free list
    free_list.pop_back();
    return object_table[free / chunk_size][free % chunk_size].id;         // return id
}

object* get_object(object_id id)     // 32-bit id argument is AND'd with 64-bit bitmask of 1's & divided / modulo'd to calculate slot map index
{
    object* obj = object_table[(id & 0xFFFFFFFF) / chunk_size] + ((id & 0xFFFFFFFF) % chunk_size);
    return obj->id != id ? nullptr : obj;
}

void destroy_object(object_id id)
{
    object* obj = get_object(id);
    obj->id = (obj->id & 0xFFFFFFFF) | (((obj->id >> 32) + 1) << 32);
    free_list.push_back(id & 0xFFFFFFFF);
}

/*

** Explanation of Bitwise Operations **

Isolate the lower 32 bits of an integer - AND the value with a 32-bit bitmask of 1's
num & 0xFFFFFFFF

Increment the Version Tag portion of an ID (the upper 32 bits) - shift down to isolate upper 32 bits, increment value, shift back & OR with lower bits
(id & 0xFFFFFFFF) | (((id >> 32) + 1) << 32)

Calculate the slot map index from the 64-bit ID
row i.e. slot_map[]      : (id & 0xFFFFFFFF) / chunk_size
column i.e. slot_map[][] : (id & 0xFFFFFFFF) & chunk_size
>> to access memory address of object : slot_map[row] + column

Fetch the unique id given a slot map index
row i.e. slot_map[]      : slot_map_index / chunk_size
column i.e. slot_map[]   : slot_map_index & chunk_size
>> to fetch id for object : slot_map[row][column].id

*/

Bulk of overhaul has been completed. See commits 21a7b401662208a7695b3178345baf26e68a1dee and f0716fc95ecfb83bf6371f17067202f5bb8ab38f.
Significant work still needed on implementing proper resource lifetime management, deletion (which may require a cross-asset dependency tracking system), etc.


Pip package on PyPI out of date?

I noticed that the online pip package for this has an older than latest version of the Dropbox library specified as a requirement, so it forces a downgrade of that. The version of the setup script on the repo here has that requirement updated to the latest though. Are there any issues running with the latest version of the dropbox library?

That was not intentional, I just forgot to upload to PyPI. Thanks for the reminder. Version bumped in 9663ec9036681be4e1257d5d2c7d6a9277236357, uploaded to PyPI as git-remote-dropbox 1.1.2.

Thanks!


Error

I am getting the following error. Can you help to fix the error please?
Traceback (most recent call last):
File "mnist.py", line 6, in
from kerosene.datasets import mnist
File "/home/DL/keras_capsn_virtual/local/lib/python2.7/site-packages/kerosene/datasets/mnist.py", line 3, in
from .dataset import Dataset
File "/home/DL/keras_capsn_virtual/local/lib/python2.7/site-packages/kerosene/datasets/dataset.py", line 11, in
initial_fuel_config_path = fuel.config.data_path
File "/home/DL/keras_capsn_virtual/local/lib/python2.7/site-packages/fuel/config_parser.py", line 150, in getattr
"provided: {}.".format(key))
fuel.exceptions.ConfigurationError: Configuration not set and no default provided: data_path.

import os DATASETS_FOLDER = 'datasets' if not os.path.isdir(DATASETS_FOLDER):     os.mkdir(DATASETS_FOLDER) os.environ['FUEL_DATA_PATH']='./datasets/' from kerosene.datasets import mnist


e2e-test: add e2e tests and CRDs for ApisixClusterConfig v2

Signed-off-by: Ling Samuel<EMAIL_ADDRESS>
[x] New feature provided

Codecov Report

Merging #1016 (291ff10) into master (cd5063f) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 291ff10 differs from pull request most recent head f38c7d0. Consider uploading reports for the commit f38c7d0 to get more accurate results

@@           Coverage Diff           @@
##           master    #1016   +/-   ##
=======================================
  Coverage   30.55%   30.55%
=======================================
  Files          74       74
  Lines        8320     8320
=======================================
  Hits         2542     2542
  Misses       5502     5502
  Partials      276      276

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 cd5063f...f38c7d0. Read the comment docs.

ping @gxthrj PTAL, thanks!


Pinch zoom setting old behavior with constrainResolution does not work for two finger scroll.

The release notes for 3.20.0 says: "Now the pinch zoom keeps the user selected zoom level even if it is a fractional zoom. To get the old behavior set the new constrainResolution parameter to true".
We have set constrainResolution for PinchZoom like the following. However this does not work for two fingers scroll on trackpad. I am using an MacBook Pro. When two finger scroll finishes, the map almost always end with a fractional zoom value.
  map = new OpenLayers.Map({
    target: target,
    layers: layerCollection,
    interactions: OpenLayers.interaction.defaults({
      altShiftDragRotate: false,
      dragPan: false,
      MouseWheelZoom: false,
      pinchRotate: false,
      pinchZoom: false
    }).extend([
      new OpenLayers.interaction.DragPan({
        kinetic: kinetic
      }),
      new OpenLayers.interaction.MouseWheelZoom({
        duration: duration
      }),
      new OpenLayers.interaction.PinchZoom({
        constrainResolution: true
      })
    ]),
    view: view
  });

I don't think that this kind of zooming is handled via the PinchZoom .. but I'm not sure.
Can you test the behaviour if you remove the PinchZoom?
  map = new OpenLayers.Map({
    target: target,
    layers: layerCollection,
    interactions: OpenLayers.interaction.defaults({
      altShiftDragRotate: false,
      dragPan: false,
      MouseWheelZoom: false,
      pinchRotate: false,
      pinchZoom: false
    }).extend([
      new OpenLayers.interaction.DragPan({
        kinetic: kinetic
      }),
      new OpenLayers.interaction.MouseWheelZoom({
        duration: duration
      })
    ]),
    view: view
  });

Hello,
constrainResolution: true does not work as expected in 4.3.3.
I've also set this option to "true" but I still get fractional zoom values. You can reproduce this by:

Start an animated zoom-in
Before the end of the animation, perform a PAN
Get the fractional zoom :(

Obs: If I only ZOOM-in without any PANning, zoom gets integer values as expected.
Hope this can help on fixing the issue.

Still not working in 4.4.2!

Still not working in v5.1.3...
(MacBook Chrome)


Expensify Card - State is not auto populated in Add shipping details flow after adding address in Get physical card flow.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.61-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Workspace has enabled Expensify Card.
Member does not have private details.
Admin has assigned a physical card to member.

Click on the Expensify Card.
Click Get physical card.
Enter legal name and phone number.
On shipping address page, enter a non-US address,
Click Next > Click Ship card.
Go to workspace chat.
Click Add shipping details.
Select birth date > Click Next.
Note that only State field is empty.
Go back to Wallet.
Click on the same physical card (not the new "Expensify Card").
Click Get physical card.
Enter State.
Click Next.
Note that nothing happens after entering state and clicking Next.

Expected Result:
In Step 11, state will be auto populated in address page because the rest of the address is added to relevant fields.
In Step 17, user should be able to proceed after manually entering state.
Actual Result:
In Step 11, state is not auto populated in address page, while the rest of the address is added to relevant fields.
In Step 17, user is unable to proceed after manually entering state.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/b3b529bb-4fd4-42cd-885c-8b43c6401070

View all open jobs on GitHub

It appears to be a backend bug. The frontend sends the state value to the backend, but the backend returns an empty string in private_personalDetails

cc @mountiny

Thanks @mountiny ❤

no update yet

same

no update yet

No update still

No updates yet, lower priority

Still no updates focusing on other issues, this is minor

Same

No updates


Fix merge conflicts for #10

I would really appreciate seeing the updated macOS keychain api merged. I hopefully fixed the merge conflicts present in #10. RUST_TEST_THREADS=1 cargo test succeeds as well.
This branch is also up to date with https://github.com/hwchen/keyring-rs.

Thanks! Appreciate you pushing this along @jasikpark . And thanks to @steveatinfincia for the original work.

@jasikpark I've already merged this, but wanted to note; I think that the dependency for the security-framework needs to be updated. Can you do this?

(From what I see, it's at * right now, but if you already know what minimum version it needs to be I won't need to dig around).


Add Velocity to replace Speed and Acceleration + Add Drag field and drag.rs example

Here is the velocity + drag update. I hope everything is in order !

Someone knows what happened in the code-style check ?

Someone knows what happened with the code-style check ?

It just wants some of the code reformatted. You can just run cargo fmt --all and commit the changes it makes.


af: Replace usages of getOrDefault()

[X] All new/changed/fixed functionality is covered by tests (or N/A)
[X] I have added documentation for all new/changed functionality (or N/A)

📋 Changes
This PR removes or replaces all usages of the getOrDefault() on the Kotlin code, as this method is only available on Android API 24+: https://developer.android.com/reference/java/util/Map#getOrDefault(java.lang.Object, V)
For the cases where there was a non-null default value, the Elvis operator was used to provide the default value.
For the cases where the default value was simply null, there was no need to do anything else as the map already returns null if the value is not present:

Also, an unused import was removed.
📎 References
Fixes https://github.com/auth0/auth0-flutter/issues/178

Hi @Widcket, any idea when it will be released?

cc @stevehobbsdev

1.0.2 has now been published 🎉


Add support for VLAN QoS configuration

Add new vlanQoS field in the config spec.
Call netlink library to set VLAN ID and VLAN QoS for a given VF interface.
Reset VLAN ID and VLAN QoS configuration to default values on VF release.
Update documentation.
Update mocks.
Use netlink library version that includes LinkSetVfVlanQos function.
Update golang.org/x/sys to the version comptible with the updated netlink library version.

Hey guys, does the build fail because of the problem that the PR #93 solves? Interestingly it passes on my fork.

Hey guys, does the build fail because of the problem that the PR #93 solves? Interestingly it passes on my fork.

I pulled the PR and passed the build/test locally, didn't get why it needs to update vendor dependency.
Above PR removes vendor dependency of build/lint/test rules in Makefile which may solve the build error.

Looks like anytime the glide files are updated the travis triggers vendor. I will merge #93. Maybe it's time to think about go module to mange dependencies.

Awesome, let me rebase on top of the lastest master and we'll see if that solves the problem.

Should we reset VlanQos to 0 in ResetVFConfig just like Vlan?
If Vlan is required to set VlanQos, can we remove LinkSetVfVlan method (ResetVFConfig is the only function using LinkSetVfVlan)?

Should we reset VlanQos to 0 in ResetVFConfig just like Vlan?

Yes, we need to reset VlanQoS to 0 on Delete.

If Vlan is required to set VlanQos, can we remove LinkSetVfVlan method (ResetVFConfig is the only function using LinkSetVfVlan after applying this PR)?

No, we still need this function to set the vlan. Not always we want to set vlan qos. Sometimes we my need to set only vlan and don't care about qos.

Should we reset VlanQos to 0 in ResetVFConfig just like Vlan?

Ah, good catch, I updated the resetVfVlan() function to do that, however now I see that it's not even being called anywhere. Should I remove it then? Also, I'll update the ResetVFConfig() to reset both VLAN and QoS (however resetting VLAN ID to 0 seems to reset both already).

If Vlan is required to set VlanQos, can we remove LinkSetVfVlan method (ResetVFConfig is the only function using LinkSetVfVlan after applying this PR)?

I can do that in this PR if there are no other calls to this function.
Thanks @zshi-redhat.

Should we reset VlanQos to 0 in ResetVFConfig just like Vlan?

Ah, good catch, I updated the resetVfVlan() function to do that, however now I see that it's not even being called anywhere. Should I remove it then? Also, I'll update the ResetVFConfig() to reset both VLAN and QoS (however resetting VLAN ID to 0 seems to reset both already).

If resetVfVlan() is not used anywhere, I think we can remove it from the code.

No, we still need this function to set the vlan. Not always we want to set vlan qos. Sometimes we my need to set only vlan and don't care about qos.

We actually may not need it. LinkSetVfVlanQos called with qos arg set to 0 acts exactly the same as LinkSetVfVlan and sends exactly the same netlink message to the kernel.

Ah, good catch, I updated the resetVfVlan() function to do that, however now I see that it's not even being called anywhere. Should I remove it then? Also, I'll update the ResetVFConfig() to reset both VLAN and QoS (however resetting VLAN ID to 0 seems to reset both already).

You can remove the  resetVfVlan(), this is no longer in use.

If Vlan is required to set VlanQos, can we remove LinkSetVfVlan method (ResetVFConfig is the only function using LinkSetVfVlan after applying this PR)?

I can do that in this PR if there are no other calls to this function.

The reset should be done in  ResetVFConfig().

I updated the logic - setting VLAN QoS now requires VLAN ID to be set as well. Reset moved to the appropriate function as discussed and the unused code removed.
I also updated the readme with information that vlanQoS requires vlan id or that the error is thrown if it's missing.

Rebased on top of the master branch after #88 merge. I also did a smoke test and it looks OK - configuration with VLAN ID, VLAN QoS and custom MAC address set at the same time worked fine.

@zshi-redhat Do you any further feedback on this PR?

@zshi-redhat Do you any further feedback on this PR?

Just one minor question, when vlan equals to zero, can we still use vlanQos? from the code, we exclude this possibility, is it actually an option to use in reality?

Kernel accepts it, it shows up in the VF configuration, but 802.1Q header isn't inserted into packets and effectively packets are not tagged.
Please see this https://github.com/intel/sriov-cni/pull/94#discussion_r306722550.

Kernel accepts it, it shows up in the VF configuration, but 802.1Q header isn't inserted into packets and effectively packets are not tagged.
Please see this #94 (comment).

Thanks for pointing to the comment, I should read that first.
Ok, it's configurable but not inserted in packet meaning other network devices cannot prioritize traffic based on the Qos settings.
I'm fine with current code.


Refactor and fix the sampling algorithm

The code is now able to run under Smalltalk, but it is very inefficient, especially the computation of the matrix. A new issue will follow this one.


`authorize': uninitialized constant Twitter::REST::Client::BASE_URL (NameError)

$ t authorize
Welcome! Before you can use t, you'll first need to register an
application with Twitter. Just follow the steps below:

Sign in to the Twitter Application Management site and click
"Create New App".
Complete the required fields and submit the form.
Note: Your application must have a unique name.
Go to the Permissions tab of your application, and change the
Access setting to "Read, Write and Access direct messages".
Go to the Keys and Access Tokens tab to view the consumer key
and secret which you'll need to copy and paste below when
prompted.

Press [Enter] to open the Twitter Developer site.
Enter your API key:
Enter your API secret:
/Library/Ruby/Gems/2.3.0/gems/t-3.1.0/lib/t/cli.rb:82:in authorize': uninitialized constant Twitter::REST::Client::BASE_URL (NameError) 	from /Library/Ruby/Gems/2.3.0/gems/thor-0.20.0/lib/thor/command.rb:27:in run'
from /Library/Ruby/Gems/2.3.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in invoke_command' 	from /Library/Ruby/Gems/2.3.0/gems/thor-0.20.0/lib/thor.rb:387:in dispatch'
from /Library/Ruby/Gems/2.3.0/gems/thor-0.20.0/lib/thor/base.rb:466:in start' 	from /Library/Ruby/Gems/2.3.0/gems/t-3.1.0/bin/t:20:in <top (required)>'
from /usr/local/bin/t:22:in load' 	from /usr/local/bin/t:22:in '
$ uname -srm
Darwin 17.5.0 x86_64
$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.13.4
$ system_profiler SPSoftwareDataType
Software:
System Software Overview:

  System Version: macOS 10.13.4 (17E202)
  Kernel Version: Darwin 17.5.0
  Boot Volume: osx
  Boot Mode: Normal

$ t --version
3.1.0

Can reproduce here. Probably an update to the twitter gem?

Same issue here, just installed t system-wide on OSX 10.11.6.
Traceback (most recent call last):
	7: from /usr/local/bin/t:23:in `<main>'
	6: from /usr/local/bin/t:23:in `load'
	5: from /usr/local/lib/ruby/gems/2.5.0/gems/t-3.1.0/bin/t:20:in `<top (required)>'
	4: from /usr/local/lib/ruby/gems/2.5.0/gems/thor-0.20.0/lib/thor/base.rb:466:in `start'
	3: from /usr/local/lib/ruby/gems/2.5.0/gems/thor-0.20.0/lib/thor.rb:387:in `dispatch'
	2: from /usr/local/lib/ruby/gems/2.5.0/gems/thor-0.20.0/lib/thor/invocation.rb:126:in `invoke_command'
	1: from /usr/local/lib/ruby/gems/2.5.0/gems/thor-0.20.0/lib/thor/command.rb:27:in `run'
/usr/local/lib/ruby/gems/2.5.0/gems/t-3.1.0/lib/t/cli.rb:82:in `authorize': uninitialized constant Twitter::REST::Client::BASE_URL (NameError)
$ t --version
3.1.0

For anyone who is looking for a quick solution, you can try https://github.com/sferik/twitter/issues/878#issuecomment-392878152

Had the same issue on ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin17], for me this fix  https://github.com/sferik/twitter/issues/878#issuecomment-349718252 worked by addressing the https://github.com/sferik/twitter api differently.
I cannot however estimate if this fix would mend the problem in general and in a proper way. Hope that a better informed maintainer can judge?

The issue still reproduces. Any way to fix that?

t works with v6.1.0 of the twitter gem, so I fixed it by running:
$ gem install twitter -v 6.1.0
$ gem uninstall twitter -v 6.2.0

The simplest upstream fix is to change t.gemspec's twitter dependency to ~> 6.1.0 instead of ~> 6.0.

Thanks @epitron your simple solution worked a treat.

@sferik Would you mind merging the pull request? (#412) This gem has been in a broken state for a while.

t works with v6.1.0 of the twitter gem, so I fixed it by running:
$ gem install twitter -v 6.1.0
$ gem uninstall twitter -v 6.2.0

The simplest upstream fix is to change t.gemspec's twitter dependency to ~> 6.1.0 instead of ~> 6.0. (Submitted a pull request with this change: #412)

This fixed it for me.

Hey, that fix works for me. It should be noted that you also need rdoc as well. That's probably obvious to people who already know ruby, but it wasn't for me. :)

@epitron's solution worked for me too. Thank you.

Earlier @epitron solution works for authentication, but then dm is still not working https://github.com/sferik/t/issues/405. Updating twitter dependency to 7.0 fixes this, but with it the authentication is broken... So what I ended up doing to be able to log in and to send a dm:
First, authentication is broken after twitter gem 6.1.0 so downgrade to it after t-3.1.0 gem installation:
gem install twitter -v 6.1.0
gem uninstall twitter -v 6.2.0

Now authenticate, most things should work after that. To get dm working make sure first the app has permission for dm's (hidden under "Permissions" tab on API control panel). Then update twitter gem to 7.0.0.
Then to make DM's work change /usr/local/lib/ruby/gems/2.7.0/gems/t-3.1.0/t.gemspec to upgrage twitter gem to 7.0 in /usr/local/lib/ruby/gems/2.7.0/specifications/t-3.1.0.gemspec:
s.add_runtime_dependency(%q<twitter>.freeze, ["~> 7.0"])
Then install the 7.0.0 in place of 6.1.0.
gem install twitter -v 7.0.0
gem uninstall twitter -v 6.1.0

Then you should have this installed and working mostly. Additionally hide ruby faraday deprecation warnings as noted in https://github.com/sferik/t/issues/424#issuecomment-643549067.

gem uninstall twitter -v 6.1.0

Thank you for your help.  In Ubuntu 20.04, the file to edit the dependencies is
/var/lib/gems/2.7.0/specifications/t-3.1.0.gemspec

This workaround no longer works as of Ruby 3 on OS/X:

Use this fork instead: https://github.com/xiaocang/t
Build and install instructions:
gem build t.gemspec
gem install t-3.1.1.gem


Zoomed image location impacted by page flow

The placement of the zoomed-in version of the image is being impacted by other elements in the page flow. If parent elements have transforms (e.g. transform: translate3d(0, 0, 0) ) applied to them, the z-index and location of the zoomed image can be thrown off completely. This can result in the zoomed image being off-center, off-screen, clipped, or under other elements (such as the zoom overlay).
I came across this issue while attempting to integrate Zoom with the Slick Carousel , however this issue could occur in many different scenarios.
Any thoughts on solutions? A few initial ideas:

Don't do that!

Make use of the .zoom-overlay-open selector to temporarily subdue any offending parent elements by removing breaking css (e.g. transform: none, overflow: visible, etc.). Although, I've been playing with this for a little bit, and have yet to achieve a successful result. The z-indexing issues are fixable, but the image is still off-center/off-screen. Perhaps the damage is already done by offset calculations being performed before the offending selectors have been removed.

Add a new element next to the zoom-overlay to contain the zoomed-in image... but what about the zoom-out effect that starts at the location of the image?

???

Ok, so I played around with the 2nd solution a little more. Adding the .zoom-overlay-open class before the offsets are calculated fixed the positioning issues. Note: Only after I added some sketchy css to remove offending transforms and overflow.
/* Ewwwww */
.zoom-overlay-open div:not(.zoom-img-wrap) {
  -webkit-transform: none !important;
  -moz-transform: none !important;
  -ms-transform: none !important;
  -o-transform: none !important;
  transform: none !important;
  overflow: visible !important;
}

    Zoom.prototype._triggerAnimation = function () {

        this._$body.addClass('zoom-overlay-open')

        this._targetImage.offsetWidth // repaint before animating

        var imageOffset = $(this._targetImage).offset()
        var scrollTop = window.scrollY

        var viewportY = scrollTop + (window.innerHeight / 2)
        var viewportX = (window.innerWidth / 2)

        var imageCenterY = imageOffset.top + (this._targetImage.height / 2)
        var imageCenterX = imageOffset.left + (this._targetImage.width / 2)

        this._translateY = viewportY - imageCenterY
        this._translateX = viewportX - imageCenterX

        $(this._targetImage).css('transform', 'scale(' + this._imgScaleFactor + ')')
        $(this._targetImageWrap).css('transform', 'translate(' + this._translateX + 'px, ' + this._translateY + 'px) translateZ(0)')

    }

Ah yeah don't put it inside something with a transform
Transforms are going to create new stacking contexts and throw off lots of calculations :+1:
We did that same sketchy css thing at medium if it makes you feel better - to null the transforms

Do we want to at least move the addition of the zoom-overlay-open class to the top of the _triggerAnimation method, as mentioned above, so the calculations are performed after the transforms are tweaked... otherwise the "sketchy css" has no effect.
this._$body.addClass('zoom-overlay-open')


Swiping left-to-right in single-room breaks the chat.

Describe the bug
When using single-room mode on mobile and swiping left-to-right (which would normally display the rooms list), the header and footer disappears.
Steps to reproduce

Enter a chat in single-room mode on mobile (Chrome dev tools also works).
Swipe left-to-right
Header and footer are now gone.

Expected behavior
Nothing should happen when swiping on mobile for single room mode.
Screenshots
Before swiping

After swiping

Device

OS: Tested in Chrome dev-tools and on ios / android
Package version: 1.1.0

Indeed, will fix that soon

Fixed https://github.com/antoine92190/vue-advanced-chat/commit/c5a381a923a8ee87c7506729a83a9de1f7ea07ed


bug: missing text from parsed pdf

I’ve encountered an issue where some text is missing after parsing certain PDFs. In the attached example, the text USD 700 disappears during the parsing process.
In the next code, the pages still have all the content:
soup = BeautifulSoup(str(tika_html_doc), "html.parser")
print("Soup", soup)
meta_tags = soup.find_all("meta")
title = None
for tag in meta_tags:
    if tag["name"].endswith(":title"):
        title = tag["content"]
        break
pages = soup.find_all("div", class_=lambda x: x in ["page"])
print("Pages", pages)

However, the blocks inside the parsed document are missing some content:
parsed_doc = visual_ingestor.Doc(pages, ignore_blocks, render_format)
print("Parsed_doc", parsed_doc.blocks)

I wasn’t able to debug it completely, but I believe the problem lies within the parse function. I’m not certain if this is a bug or if BeautifulSoup is misinterpreting USD 700 as a header when it clearly isn’t. The main problem is that in this example the ignored text was a kinda important title, so it was nothing resembling a Header really.
Any help is appreciated
Example pdf:
Here is the pdf that has been causing me problems. It's not complete for privacy reasons, but it's the minimum example I found. that causes this problem. If I edit it a little bit, for example adding text next to the USD it won't cause this problem.
problematic-pdf.pdf

I'm also seeing some missing text being returned - did you have any luck debugging?


Add post row analytics link

In relation to #148 . This feature is highly requested by our customers. Would be great to have it.
Thanks for you great work!

Thanks for your PR! I like this. @metmarkosaric Shall we include this in the next release? Small piece of code, but rather effective feature!

we can consider it for sure for one of the upcoming releases!
so this one adds some stats to the post itself but not to the post list? would be nice to see a screenshot or test how this implementation works as I see some people also ask for the stats to be in the post list itself too.
and more importantly is this on by default and is there a way to disable/enable it in the settings?

Hi there, thank for you quick reaction. It's exactly the opposite @metmarkosaric . This one adds a link to the post list not to the post editor. It is on by default and there is currently no option to disable it. But the links is pretty discreet as it only appears on hover over the posts row.

sounds good @EdwardBock, thanks for the contribution! we'll test it and decide how to proceed.

Any news here?

Hi @EdwardBock,
Thanks for your suggestion!
2 remarks, before we can publish this:

It would be nice to also have the link appear in the pages list, not just posts. (and all custom post types, maybe?)
The link shouldn't appear if "Disable menu in toolbar" is enabled or, when Plausible Self-hosted (Community Edition) is used, when the private shared link is not entered.

Once you've added these changes, we can take it up in a next release!

Closing this, so we can continue in #212


Infusate parsing updates for intersheet relations in StudyLoader

Summary Change Description
Made updates to the infusate and tracer name parsing - specifically the construction of the data objects. The new unparsed tracer strings have a concentration on them which is incompatible with code elsewhere, specifically in the check method inside the Infusate class. The check also relies on the name field, as does code in the infusates loader. Once I added the defer_autoupdates decorator from maintainedmodel, those method would fail because the names were not there. So I just changed them to use the generator methods instead of the field. This only affects loading and validation - not the operation of the database in production.
Affected Issues/Pull Requests

Partially addresses #829
Merges into #1009
Next PR: #1011

Review Notes

See comments in-line.
Checklist

This pull request will be merged once the following requirements are met.  The
author and/or reviewers should uncheck any unmet requirements:

Review requirements

Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here

Review period: 2 days

Associated issue/pull request requirements:

[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)

Basic requirements

[x] All linters pass
[x] All tests pass
[x] All conflicts resolved

Overhead requirements

[x] New/changed method tests implemented/updated (or no change)
[ ] Updated Unreleased section of changelog.md (or no change)
[x] Migrations created & committed (or no change)

Merged via #1076


[Build] Minimize the amount of comments on a PR

Comments (and with them emails sent) can be minimized.

The number of comments was minimized, based on the existence of a snapshot. If a snapshot already exists, both the tagging comment and the snapshot comment will not be shown.


Use $digest instead of $apply to avoid unnecessary rootscope digest

The scope.$apply call inside updateView triggers a digest cycle on the rootscope and all descending child scopes. This can cause unnecessary performance issues in large applications, because the scope of the entire application will be re-evaluated whenever a change occurs in the editor.
I suggest changing $apply to $digest, limiting the digest cycle to the directives own scope, and preventing it from affecting the surrounding application.

Thanks for posting and that all makes sense.
I just want to verify everything is still working tonight, and I will get this merged in.


Tiktok

Hi @berhad23,
I hope this message finds you well. I noticed you've created an issue in our repository, but it seems the description is missing. To ensure we fully understand and address your concerns, could you please provide us with more details about the issue you're experiencing? A thorough description will help us to better assess and resolve the matter more efficiently.
Looking forward to your response, and thank you in advance for your cooperation!
Best regards.

Hi @berhad23 ,
As it has been some time since this issue was opened and no further details have been provided, I will be closing this issue for now. If you still face any problems or have more information to share, please feel free to comment here, and I will be happy to reopen the issue and assist you further.
Thank you for your understanding and cooperation.
Best regards.


bug: when bootstrapping cluster with kube-vip, role fails on "Wait for the first server be ready"

Summary
When running role on a fresh set of VMs with kube-vip as a VIP controller and rke2_api_ip set to that VIP, role fails on
TASK [lablabs.rke2 : Wait for the first server be ready - with CNI]

I believe this is because kube-vip manifests are put on disk AFTER running tasks from first_server.yml. If those 2 sections could be swapped, it would most likely solve the issue.
Issue Type
Bug Report
Ansible Version
ansible [core 2.14.9]
  config file = None
  configured module search path = ['/home/paulfantom/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python3/dist-packages/ansible
  ansible collection location = /home/paulfantom/.ansible/collections:/usr/share/ansible/collections
  executable location = /usr/bin/ansible
  python version = 3.11.6 (main, Oct  8 2023, 05:06:43) [GCC 13.2.0] (/usr/bin/python3)
  jinja version = 3.1.2
  libyaml = True

Steps to Reproduce
Start a new cluster with the following variables

rke2_ha_mode: true
rke2_ha_mode_keepalived: false
rke2_ha_mode_kubevip: true

rke2_api_ip: "<SOME_IP_NOT_ASSIGNED_TO_ANY_NODE>"

Expected Results
I expect the playbook to run properly
Actual Results
Role fails on

TASK [lablabs.rke2 : Wait for the first server be ready - with CNI] *************************************************************************************************
FAILED - RETRYING:

Fixed in https://github.com/lablabs/ansible-role-rke2/issues/173


Fix duplication and linter "Line too long warnings"

I think you might've forgotten to commit the .flake8 file with the rule for "line too long" enabled

There were some places, where I thought that splitting the lines would make the code worse.
e. g. links which if they were splitted would become unclickable in IDE's
So I did not change them and .flake8 rule


feat: Add default config

Add default config
Add doc comments on the options. These comments are copied directly from anvil

Current dependencies on/for this PR:

main

PR #69   👈

PR #70

PR #71

This comment was auto-generated by Graphite.


Pulsar IO tests leak resources in Kafka related tests

Describe the bug
Resources are leaked in Pulsar IO tests.
This type of entries repeat in the test logs:
2022-05-31T06:06:42,491 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:42,674 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:43,367 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:43,834 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:44,359 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:44,748 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:45,285 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:45,603 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:46,457 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:46,708 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:47,494 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:47,863 - WARN  - [kafka-producer-network-thread | c9eafaa1-fee0-4029-a05d-55b0c639cb3e:NetworkClient@782] - [Producer clientId=c9eafaa1-fee0-4029-a05d-55b0c639cb3e] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.
2022-05-31T06:06:48,409 - WARN  - [kafka-coordinator-heartbeat-thread | sink-test-fyeeijxp:NetworkClient@782] - [Consumer clientId=consumer-sink-test-fyeeijxp-1, groupId=sink-test-fyeeijxp] Connection to node 1 (localhost/<IP_ADDRESS>:49216) could not be established. Broker may not be available.

To Reproduce
Check the output of "CI - System - Pulsar IO"
Expected behavior
Resources should be properly closed in tests using try-with-resources blocks or Lombok's @Cleanup annotation.

I'll create a PR with these changes: https://github.com/apache/pulsar/compare/master...lhotari:lh-fix-pulsar-io-integration-test-resource-leaks after CI is unblocked.

This problem isn't fixed. example CI - System - Pulsar IO job
2022-11-24T16:43:26.5108101Z 2022-11-24T16:43:26,510 - WARN  - [kafka-producer-network-thread | 2f374e07-6664-47d6-91c6-26f846bf509b:NetworkClient@782] - [Producer clientId=2f374e07-6664-47d6-91c6-26f846bf509b] Connection to node 1 (localhost/<IP_ADDRESS>:49232) could not be established. Broker may not be available.
2022-11-24T16:43:27.5136085Z 2022-11-24T16:43:27,513 - WARN  - [kafka-producer-network-thread | 2f374e07-6664-47d6-91c6-26f846bf509b:NetworkClient@782] - [Producer clientId=2f374e07-6664-47d6-91c6-26f846bf509b] Connection to node 1 (localhost/<IP_ADDRESS>:49232) could not be established. Broker may not be available.
2022-11-24T16:43:28.4162827Z 2022-11-24T16:43:28,415 - WARN  - [kafka-producer-network-thread | 2f374e07-6664-47d6-91c6-26f846bf509b:NetworkClient@782] - [Producer clientId=2f374e07-6664-47d6-91c6-26f846bf509b] Connection to node 1 (localhost/<IP_ADDRESS>:49232) could not be established. Broker may not be available.
2022-11-24T16:43:29.6195148Z 2022-11-24T16:43:29,619 - WARN  - [kafka-producer-network-thread | 2f374e07-6664-47d6-91c6-26f846bf509b:NetworkClient@782] - [Producer clientId=2f374e07-6664-47d6-91c6-26f846bf509b] Connection to node 1 (localhost/<IP_ADDRESS>:49232) could not be established. Broker may not be available.
2022-11-24T16:43:30.8218768Z 2022-11-24T16:43:30,821 - WARN  - [kafka-producer-network-thread | 2f374e07-6664-47d6-91c6-26f846bf509b:NetworkClient@782] - [Producer clientId=2f374e07-6664-47d6-91c6-26f846bf509b] Connection to node 1 (localhost/<IP_ADDRESS>:49232) could not be established. Broker may not be available.

There was an additional thread leak which will be fixed by #18618


FZF clears previous line of prompt

Category

[x] fzf binary
[x] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[x] CentOS7

Shell

[ ] bash
[x] zsh
[ ] fish

I have a multiline prompt in zsh and recently updated fzf (I don't know the previous version I was on unforutnately). I've found that now when I hit CTRL-T in my prompt to bring up the search window then ESC out of it, the previous line in my prompt gets erased. See https://imgur.com/a/TJyep
I just tested making my prompt a single line and the issue doesn't persist. See https://imgur.com/a/9UDtj

Can you share the source code for your prompt so I can test with it?

Here's a copy of my prompt: http://pastebin.com/9EG9aCWx
It's a shittily modified version of https://github.com/sorin-ionescu/prezto/blob/master/modules/prompt/functions/prompt_sorin_setup
Thanks for following up on this!

Well, I tried your config (which required me to install prezto as well), but it seems like it has issues not only with fzf but also with other command-line completions. For example, simply doing vim <tab> or ls <tab> causes the problem you described. I think you should fix the issue first. Let me know if you still have problems with fzf bindings even after that.

Interesting, that doesn't happen for me.

Oh, nevermind. I was able to repro with ls too.

This is not fixed on zsh 5.7.1 (x86_64-apple-darwin17.7.0)


Issues when publish the Service

I add the package properly to the project and in development work fine.
However. When I publish the build and deploy in one test server. I got errors like this.

What I have to do for deploy the swaggerWCF with the service.
Kind regards,

Hi
I find it strange IIS is mapping to physical path, I would say to check IIS logs.
Thanks
Abel

Look the information in the logs.
#Software: Microsoft Internet Information Services 7.5
#Version: 1.0
#Date: 2016-06-20 10:29:00
#Fields: date time s-ip cs-method cs-uri-stem cs-uri-query s-port cs-username c-ip cs(User-Agent) sc-status sc-substatus sc-win32-status time-taken
2016-06-20 10:29:00 ::1 GET / - 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 200 0 0 2422
2016-06-20 10:29:00 ::1 GET /favicon.ico - 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 404 0 2 98
2016-06-20 10:29:02 ::1 GET /DocTracerServiceAPI.svc - 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 200 0 0 855
2016-06-20 10:29:13 ::1 GET /api-docs - 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 307 0 0 451
2016-06-20 10:29:13 ::1 GET /api-docs/ - 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 302 0 0 6
2016-06-20 10:29:13 ::1 GET /api-docs/index.html url=http://localhost:5555/api-docs/swagger.json 5555 - ::1 Mozilla/5.0+(Windows+NT+6.1;+WOW64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/51.0.2704.103+Safari/537.36 404 0 2 2
But, I found that something in the Web.config in production is creating the issue.

You can check for your app errors in Event Viewer

I found the issue,inside the webserver tag.
The old config was using the modules to add some modules required before, and swagger configuration should allow

Otherwise, the request cannot be found.

ok, closing the issue