[fix] [broker] Fix incorrect recent joined position of consumers in Key_Shared mode

Motivation
Background-1: After all consumers are removed, the Dispatcher will call cursor.rewind and reset all attributes in memory, making the Dispatcher like a new Dispatcher.
Background-2: there have a variable recentlyJoinedConsumers records the read position of the cursor when a new consumer registers.
If there has an in-flight reading, Dispatcher does not call cursor.rewind immediately but waits for the in-flight reading complete, it makes the recentJoinedPosition of new consumers in the wrong position.

the last consumer is removed, mark shouldRewindBeforeReadingOrReplaying is true

the read position is 5:0, and the mark deleted position is 3:-1.

add new consumer(C1), skip cursor.rewind if shouldRewindBeforeReadingOrReplaying is true
add new consumer(C2), put C2 into recentlyJoinedConsumers, the max read position of C2 is 5:0
add new consumer(C3), put C3 into recentlyJoinedConsumers, the max read position of C3 is 5:0
the in-flight reading is complete, rewind read position to 3:0.

at this time, the max read position of all the consumers should be null.

Modifications
After rewind, clear recentJoinedPosition
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

x

@codelipenghui

Could you please provide an example? If I understand correctly, it should be
All consumers are disconnected
There is an inflight read entry operation
A new consumer connects to the broker
If all of the 3 operations happened concurrently. The new consumer might get a chance to use the updated read position by the inflight read operation as the recently joined position? But it should use the position after rewinding.

If all of the 3 operations happened concurrently.

no race condition, all executed sequentially.

Could you please provide an example?

I add an example to the section Motivation

The new consumer might get a chance to use the updated read position by the inflight read operation as the recently joined position?

Yes, you can see the line-165 of https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L163-L169

@poorbarcode Oh, I got your point now.
So, from the user's perspective, all the consumers might get stuck if the problem occurs since the messages that dispatch to consumers are before the recently joined position of all consumers.
I have a question about:
https://github.com/apache/pulsar/blob/a26cf3eb78858ef5638650b6dbd7dab1f76a8c61/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L563-L570
Should we also skip the message delivery for readType == ReadType.Replay?
And the redelivery messages should also be clear for all rewind operations? since we are going to read the messages from the next position of the mark delete position. I noticed we have cleared at
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L170
https://github.com/apache/pulsar/blob/master/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/PersistentDispatcherMultipleConsumers.java#L209
But we have more than 2 places to rewind the cursor.

This PR will continue after waiting for PIP-269 to complete.


how to define the rule for 3rd party module?

Hi. thank you for your excellent work!
My research area is 3D point clouds, including shape classification and semantic segmentation.  In my pytorch project, there are some 3rd party libs, such as ball query algorithm in PointNet++, which is customized CUDA function. In this situation, how could i define the rule for calculating the macs and params? I would appreciate it if you can give me some advice.

For newly defined modules (no matter in CUDA or C or Python), there should be a corresponding python class wrapper. You can register counting functions with that wrapper.
As for reference, you can check how THOP counts for PyTorch modules
https://github.com/Lyken17/pytorch-OpCounter/blob/master/thop/profile.py#L21-L65


Log elapsed time for a successful celery task

Build: https://ocd.appscale.com:8080/job/Daily Build/2751/


Rendering subsystem TODOs

Copy-on-write lists
struct everything (basically all drawing ops could be represented by a single struct with Type field
schedule update on scene consume
actual layers
pause timers when nothing happens
separate render thread for each window to avoid blocking other windows on v-sync

Animations should also tick on scene update
to avoid triggering animators multiple times between rendered frames
i. e. the render cycle would be:
while(true)
{
        if(ConsumeSceneAndTriggerUpdate())
              Render();
}

Which would schedule an update to the UI thread for that particular toplevel. The update routine should trigger animation tick, run layoutmanager pass for that particular toplevel and only then build the scene. That way we'll be preparing the next scene graph version while the render thread renders the current one. And won't run any unnecessary computations if renderer can't keep up with the UI thread for some reason.
The only problem would be animations that tick slower than the render timer, so we need to somehow switch the animation timer to something else once the render one gets paused.

@kekekeks i wonder if they can be independent, both timers updating at their own pace.

Any updates on this issue?


Update Czech translation

Credits to Christine Vasta (@kiwiistnichtda) =)

Dík!
Note that this translation is currently at 97% and only 3 strings remain. If it makes it easier, you can also translate at https://www.localeapp.com/projects/377

@JasonBarnabe Thanks! Christine used our own tool to update the translation, but I see the pluralization is handled a bit differently, so it makes sense to just use Localapp anyway. I've just asked her for translating the remaining phrases =)

Hey @JasonBarnabe, Christine has translated the remaining phrases using Localeapp. Any way for this being released somehow soon? =)

Hey!
Any news on releasing the modified translation?
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Bez
virů. www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
2018-02-26 14:07 GMT+01:00 Jonatan Kłosko<EMAIL_ADDRESS>
Hey @JasonBarnabe https://github.com/jasonbarnabe, Christine has
translated the remaining phrases using Localeapp. Any way for this being
released somehow soon? =)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tigrish/devise-i18n/pull/227#issuecomment-368498642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AivAixTsobOtQ9NxySuL_IRsw2s-iiU1ks5tYqyggaJpZM4SLDKg
.

It was released yesterday in 1.6.1.

@JasonBarnabe Lovely, thanks a lot! :tada:
@kiwiistnichtda Thanks for the reminder, I'll update our website to the latest version soon!


Compiling errors in GlobalVariable.mqh

'::' - unexpected token
200      if(!m_cs.isValid()) m_ref=HandleManager::create();
'create' - function not defined
200      if(!m_cs.isValid()) m_ref=HandleManager::create();
'::' - unexpected token
211      m_ref=HandleManager::create();
'create' - function not defined
211      m_ref=HandleManager::create();
'&' - parameter passed as reference, variable expected
206      InterpretZmqMessage(&pushSocket, components);

Same here. I used to had it running flawlessly. Did a modification / upgrade today then I'm getting the same message.

Same here. I used to had it running flawlessly. Did a modification / upgrade today then I'm getting the same message.
Edit _ This solution worked:
7iW commented 4 days ago
I was able to resolve this issue by replacing the Darwinex Zero MQ implementation with the Zero MQ implementation from https://github.com/dingmaotu/mql-zmq.

That worked thanks.

Hi,
thank you for raising this issue. Indeed the update required some changes in the dependencies. The file mql-zmq-master.zip was updated in our repo, so it should work again now without the need to download it from the original source.


cloudprovider: specify cloud provider in ExternalID

Why do you want to rename GetExternalID to GetInstanceID? It currently corresponds nicely with the API field of the same name.

Similarly, why do you want to prefix the cloud provider ID onto the API objects' ExternalID's? What's the motivation for the change?

@a-robinson I probably addressed all your comments. The answers to your questions should be now in the commit messages or the 3 patches. Thanks!

@a-robinson this should be ready to be merged

Sorry @simon3z, I was pulled away from my desk most of yesterday afternoon.
I think this looks good now, but cc @bgrant0607 about the NodeSpec.ExternalID change since it effectively changes the V1 API.

I am not in favor of this change. Changing the format of external ID is a breaking change. Also, I'd like a full URL to be a valid external ID.
@simon3z What's the motivation? There is no PR description.

@bgrant0607 the description is in each commit.

Our node identifiers are a mess.

name -- this should just be the resource name in API (and key in etcd)

Instance name on GCE
Hostname or address on bare metal

externalID

Instance id number on GCE
Same as name on bare metal

addresses

ExternalIP and InternalIP
Hostname is representable, but not filled in, at least on GCE

systemUUID

OS-generated UUID

I would like the object name to be just that. It might coincide with other fields, but should not be abused for other purposes other than to interact with the Kubernetes API.
Similarly, addresses should contain the authoritative means of determining how to contact the node. We probably need more information there, such as which address(es) can be used for authentication.
The systemUUID is presumably used as a unique identifier on the system itself. I'm not sure that's too relevant to Kubernetes, but it's fine to return it.
We totally lack a principled mechanism for relating a Kubernetes node to a VM in a cloudprovider or physical machine or VM in a machine registry/DB. That's what this PR is about. I'd like it to work for people maintaining their cloud providers.
Identity of the cloudprovider and/or information about how to contact it are not specified, nor is information like the project, zone, region, etc. That's all implicit and internal, fixed when the cluster is created. For example:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/cloudprovider/gce/gce.go#L54
Although clusters are single-zone today, I don't want to bake that into the API.
The GCE instance ID can't actually be used to look up the instance, so I don't know why we care about that.
So what information would be useful?
I think a URL to look up information about the node in the cloud provider would be useful. That's what I thought externalID should contain. We could rename that field to providerID or somesuch.
Alternatively, we could split this into multiple fields, such as the name (and/or address?) of the cloudprovider, project/zone (which might be empty in some scenarios), and instance name/id (which might be empty if there is no external registry or cloudprovider).
cc @davidopp @quinton-hoole @thockin @justinsb @a-robinson @roberthbailey @rjnagal @kelseyhightower @smarterclayton

@bgrant0607 For AWS, we're using the instance name, but AWS has a much more natural instance key - the AWS Instance ID.  It would be great if that were the identifier which the cloudprovider worked with, rather than (or in addition to) the name, which often requires an extra lookup.
When you say a URL, do you mean an http:// URL that a user could reach, or do you mean something URL-like that the cloud-provider defines?  For AWS, I believe it would make sense to use aws://<zone>/<instanceid> - instances naturally live in a zone, the zone determines the region, the region determines the endpoint.  I believe instance ids are globally unique, but I'm not sure whether this is guaranteed, so it seems safer to qualify the instance id with the zone.  Also, including the zone means we can immediately resolve it without scanning every region, and we can avoid some lookups entirely e.g. to know whether EBS volume X is in the same zone and therefore could be attached to instance Y.
If someone was running a private cloud that used the EC2 API, we could probably tolerate aws://<endpoint>/<zone>/<instanceid> as well.
I am +1 on name being irrelevant to the cloudprovider, and externalID / providerID being the reference for the provider.  I think a URL makes more sense than multiple fields, with the scheme determining the cloud provider, and the remainder containing whatever additional information is required by the cloud provider in whatever format makes sense for that cloud.

I meant a usable web URL that RESTy higher-level orchestrators or users could use without needing to understand how to construct such URLs for any particular cloud provider.
I could see that some kind of custom URI that included provider, zone, etc. could be useful for enlightened clients.
Those don't have to be mutually exclusive -- we could have multiple fields.

systemUUID
OS-generated UUID

@bgrant0607 actually systemUUID is the system uuid provided by the bios (I think from /sys/class/dmi/id/product_uuid). In many hypervisors it's very similar to the externalID since it's pushed as system bios uuid. The os-generated one is machineID.
I think we should split ExternalID in two parts:

ProviderID: fully under control of the cloud provider, it will generate a unique id that is meaningful to identify the provider (e.g. aws://<endpoint>/<zone>, or ovirt://<hostname>:<port>)

ExternalID: instance id as used by the cloud provider to identify the instance (basically this remains the same as today so we don't break api v1).

So for example:
{
  "providerID": "ovirt://<hostname>:<port>",
  "externalID": "fb544333-e2c0-4a51-a37a-175f88ad1636"
}

For clarity ProviderID could be further broken into for example:
{
  "providerName": "ovirt",
  "providerID": "https://<hostname>:<port>",
  "externalID": "fb544333-e2c0-4a51-a37a-175f88ad1636"
}

{
  "providerName": "aws",
  "providerID": "/<endpoint>/<zone>",  // or whatever makes sense here
  "externalID": "fb544333-e2c0-4a51-a37a-175f88ad1636"
}

Proposal:
I suggest we add "providerID", which would use URL-like syntax as proposed in this PR, starting with provider://. The exact format would be provider-specific. GCE and AWS would include info such as project and zone. The ID should be resolvable. Therefore, GCE should use the instance name rather than the id.
We can then just deprecate externalID and remove it when we remove v1beta1.
We could add a usable http URL later, if needed.
WDYT? Any other objections, opinions, requirements, etc.?

By "ID should be resolvable", I mean that it should be easily/efficiently usable when looking up the instance with the cloud provider, not that it needs to be resolvable via DNS.

If we are going to make providerID URL-like, why force a specific prefix? GCE has a RESTful URL that represents the instance resource, and it would be nice to avoid having to do s/provider:/http:/ if we wanted to use it directly. I kinda liked having the providerName as well, because then you don't need to know how to parse all of the (opaque) providerID fields to know which cloud provider is hosting a node.

@bgrant0607 this is probably far from being final but it should be a good approximation of what you proposed.

Thanks, @simon3z.
Requirements:

Don't change the contents of externalID. Just deprecate it.
Make provider explicit
Create an ID that's sufficient, and ideally convenient and efficient, for looking up info from the cloudprovider
Use the term "provider" in new fields, for consistency with the concept in the system

Other than that, I don't have strong opinions and delegate the final decision to @roberthbailey.

@simon3z this now also needs a rebase.

@bgrant0607 @justinsb @a-robinson @roberthbailey can you review? Thanks!

@bgrant0607 @justinsb @a-robinson @roberthbailey ping?

@simon3z I'd love to take another look but you need to rebase first.

@roberthbailey it is now rebased.

I think this LGTM, at least as far as AWS is concerned.
Is the long-term goal here that we will eventually pass the ProviderId (and not the name) to the cloudprovider?  That would make me very happy, particularly if at least the API changes can make v1.

@davidopp @roberthbailey can you review? This is conflicting with kubelet-apiserver registration, I'll rebase when that part will be stabilized (extremely painful to rebase).
On the other hand I don't want to delay this PR any further, so can you review what you can, thanks.

I'm hopefully that https://github.com/GoogleCloudPlatform/kubernetes/pull/8164 will get merged soon, which will hopefully make your rebase less painful.

#8164 was merged and looks like it's going to stick this time. @simon3z can you rebase (hopefully for the last time) so that this PR can be reviewed? Thanks for your patience.

@bgrant0607 @justinsb @a-robinson @roberthbailey can you review? Thanks!

Thanks for rebasing. Today is a holiday for most of us but I'll take a look first thing tomorrow.

LGTM. We can fix/improve this in subsequent PRs. Also, we should document the purposes of the various identifiers in docs/node.md.


WDC simulator not accessible ERR_CONNECTION_REFUSED

I am having issues reaching the WDC Simulator on my MAC.  I am using chrome and have tried clearing the history, changing proxy/firewall settings, restarting web browser, etc but I am still having issues connecting to the WDC Simulator for Tableau.  It always says ERR_CONNECTION_REFUSED in chrome.  While tyring this website below in Safari, it says Safari can't find this server.  Here is the address I am trying to reach:
http://localhost:8888/Simulator/index.html
I have installed the necessary software per the instructions here:
http://tableau.github.io/webdataconnector/docs/

You may have another application that is using that port.
By default, the test web server runs on port 8888. To change the port number, edit the http-server script in the package.json file. For example, you might enter the following to change the port to 8000:

"http-server": "node node_modules/http-server/bin/http-server -p 8000"

From docs

@boconne27 did that suggestion solve your problem?  I agree with @tagyoureit's assessment.   I am closing this issue for lack of a response, please reopen if you are still experiencing problems.


[Orders] Employee can not edit paid order

Environment: all
Reproducible: always
**Build found:**24/09/2023
Preconditions

Log as employee
Employee has to have permission to edit orders
Order must be formed and paid
Confirmed amount on "Textile waste" in the "Order detais" section must be 0
Go to https://www.greencity.social/#/ubs-admin/orders

Steps to reproduce

Choose the formed and paid order.
In the "Order detais" section in confirmed amount add more units of the product to "Textile waste"
Click on "Save" button

Actual result
After clicking on "Save" button error message "Bag does not exist by id: 6" appears. Response 400 in devtools appears.
Expected result
After clicking on "Save" button message "Your changes have been saved" appears. Response 201 in devtools.
User story and test case links
https://github.com/ita-social-projects/GreenCity/issues/5538
https://github.com/ita-social-projects/GreenCity/assets/128155213/3a26669a-423b-4d8b-bb37-6d2910ff968b

This bug is not reproduced


Update the StaticModel

Changes

move to the package metamodel
give the option to use it on both run time and build time
Remove the condition to look for the StaticMetamodel as mandatory. Because at the Java Annotation Processor, we will generate those classes from the Entityannotation, e.g.: Hibernate
Allow dynamic `StaticMetamodel ' search using the Metamodel interface.

I read a lot of the Jakarta Persistence Spec. What I don't understand is the way to allow a mutable entity instead of create once it reads from the annotations at build time.
Is there anything that must happen in real-time?
Update: I find the reason:

When the entity manager factory for a persistence unit is created, it is the responsibility of the persistence provider to initialize the state of the metamodel classes of the persistence unit. Any generated metamodel classes must be accessible on the classpath.

https://jakarta.ee/specifications/persistence/3.0/jakarta-persistence-spec-3.0.html#bootstrapping

@njr-11 I will close mine, once you created this one here: https://github.com/jakartaee/data/pull/302


capacity/limit is misdocumented

client_reference.rst:

[ ] Still documents the old limit parameter, not the new capacity one
[ ] Lists the default value for all the connector subclasses as None, rather than 20.

I'd PR a fix, but as I noted in https://github.com/KeepSafe/aiohttp/issues/1601#issuecomment-279258716 I disagree with @fafhrd91's change to the limit/capacity functionality so I'll wait for that argument to be resolved first - no sense documenting something that I'm currently arguing you guys should revert!

i agree with your reasoning, but i still think we need capacity feature.
i suggest rename capacity to max_connections and add max_connections_per_host

Let don't invite very_long_names_in_java_style.
This is documentation issue.
capacity is good name, if we document limit as obsolete and deprecated the meaning of attribute is obvious. limit is also perfect name but already used.
@ExplodingCabbage regarding to your specific case (web scrapper throttling) -- please use explicit semaphores or something like this. Or, maybe, create a ClientSession instance per site.
aiohttp should have perfect defaults for generic cases.
I believe it means that ClientSession should behave like browser.
Most users don't care about connection limits but they are very sensitive to memory/resource leaks.

i think we should support per host limit.
only reason is, limit per host it is not that trivial to implement.

good point.
Then let's choose capacity and capacity_per_host as alias for deprecated limit.
They are shorter than max_connections and add max_connections_per_host.
I believe total limit is much more wider used than limit per host.

but maybe we should just leave limit in place, just change it's meaning.
and add limit_per_host for old limit meaning.

i renamed capacity back to limit and added limit_per_host


Add meetup.com group info to apiops.net

Add information about meetup service to apiops.net. Encourage people to create groups in other cities.

I'll add the info to apiops.net

I'll use this https://github.com/user3581488/Meetup

Added without any APIs just as link for now. http://apiops.net/#localmeetups


Trie.java delete does not correctly work

Try to add two words like "ciao" and "ciaone" and delete "ciaone"
The Tries will be empty

https://gist.github.com/sbilello/76a61ae45487edf516f559f9f40b0e8d


Warn with proper guidance for trying to use Vue without the plugin installed

Is your feature request related to a problem? Please describe.
#1365
Currently, we are assuming Vue file as JSX without @vitejs/plugin-vue. This is potentially confusing.
Describe the solution you'd like
Show the warning and guidance for users to install Vue's plugin

Most users should create a new Vite 2 app with @vitejs/create-app, and will be prompted to select proper templates, so they won't actually run into this problem. This only happens to people who don't read the docs.

Most users should create a new Vite 2 app with @vitejs/create-app, and will be prompted to select proper templates, so they won't actually run into this problem. This only happens to people who don't read the docs.


Usage of NoCommentsYamlLine, unit test and fixes + puzzle

PR for #98

Coverage decreased (-0.4%) to 86.35% when pulling 12331260e0f4874a76c25f53deb68f70ae93132a on 98 into 3b27e3d74c08c2decc040451d05b9584378ea100 on master.

@rultor merge pls

@rultor merge pls

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge pls

@amihaiemil Done! FYI, the full log is here (took me 1min)


Init unit test

Description
Init unit test
Type of change

Unit tests (no changes in the code)

Testing steps
Done on CI
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report

Merging #28 (9d717c6) into main (ce41141) will increase coverage by 4.47%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   83.76%   88.23%   +4.47%
==========================================
  Files          17       17
  Lines         425      425
==========================================
+ Hits          356      375      +19
+ Misses         69       50      -19

Impacted Files
Coverage Δ

ccx_messaging/engines/sha_extractor_engine.py
60.00% <0.00%> (+60.00%)
:arrow_up:

ccx_messaging/engines/__init__.py
100.00% <0.00%> (+100.00%)
:arrow_up:

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 ce41141...9d717c6. Read the comment docs.


Throwing import error

I'm very sorry for posting such a simple bug but for the life of me I can't figure out why its throwing this error, I have tried reinstalling numpy and have all the required packages. Numpy even works when running other python files but not here for some reason.
Error:
Traceback (most recent call last):
  File "pretrain_DAMSM.py", line 3, in <module>
    from miscc.utils import mkdir_p
  File "/books-nn/T2I_CL/DM-GAN+CL/code/miscc/utils.py", line 4, in <module>
    from torch.nn import init
ModuleNotFoundError: No module named 'torch'

This was thrown as a result of me running:
sudo python2.7 pretrain_DAMSM.py --cfg cfg/DAMSM/book.yml --gpu 0

I think the problem is the environment. You'd better create an Anaconda virtual environment to install the required packages. Take it as a reference: https://github.com/huiyegit/APLC_XLNet

It seems that switching from a python 3.7 conda environment to a python 3.6 conda environment solved the problem, thanks!


Mock Service Worker not intercepting requests in loader

What is the location of your example repository?
https://github.com/rmiller61/hydrogen-oxygen-msw
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
@shopify/remix-oxygen 2.0.2, @shopify/hydrogen 2023.10.3
What version of Remix are you using?
2.1.0
Steps to Reproduce

Clone https://github.com/rmiller61/hydrogen-oxygen-msw.git
Ensure you are using Node 18.8.0 or higher so you can use --import in NODE_OPTIONS
npm run dev

Expected Behavior
MSW should intercept requests (that match the handlers specified in tests/mocks/handlers.ts) on the server so their responses can be mocked.
I have a GraphQL query named "Header" defined in my root.tsx that should be intercepted by MSW.
Actual Behavior
MSW doesn't pick up any requests in the loader functions. This is evident by the lack of lifecycle events logged to the console, as well as the responses themselves. It does, however, intercept requests outside the loader, such as the initial fetch request Hydrogen sends to pull my environment variables from Shopify. It's specifically the requests in the loader.
It's worth noting that MSW can't intercept 3rd party APIs, either. I initially thought this might be a problem with the Storefront client, but it appears to be any fetch request inside the loader.

@frandiox thank you - this actually does work. Very strange, though. Didn't even occur to me to use the browser package since I'm trying to intercept on the server. But all's well that ends well I guess. I'll mark this as closed.


Crashes on some other hardware

The following crash can be triggered when pressing the brightness keys on some systems:
INFO:gabelstaplerwm::wm::window_system: executing binding for KeyPress { code: 233, mods: 0, mode: Normal }
thread 'main' panicked at 'wait() should either return Ok or panic', /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/sys/unix/process/process_unix.rs:74
stack backtrace:
   1:     0x5559793389ba - std::sys::imp::backtrace::tracing::imp::write::h1d59ca58eb86a1e2
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/sys/unix/backtrace/tracing/gcc_s.rs:42
   2:     0x555979337d01 - std::panicking::default_hook::{{closure}}::hc8550e2dc230bf9b
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/panicking.rs:351
   3:     0x55597933720c - std::panicking::rust_panic_with_hook::h319375f6b98710b0
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/panicking.rs:367
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/panicking.rs:555
   4:     0x555979336a51 - std::panicking::begin_panic::h8462d4bdb88c73cc
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/panicking.rs:517
   5:     0x55597933f847 - std::process::Command::spawn::h174a0ef6a697e968
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/sys/unix/process/process_unix.rs:74
                        at /buildslave/rust-buildbot/slave/nightly-dist-rustc-linux/build/src/libstd/process.rs:559
   6:     0x5559792e8044 - gabelstaplerwm::wm::config::exec_command::h3ed28e821b936b9b
   7:     0x5559792fe4f1 - gabelstaplerwm::wm::config::setup_wm::{{closure}}::hb4bda0dcbc2dc40b
   8:     0x5559792f470f - gabelstaplerwm::main::h403f0e00ab78e9c3
   9:     0x5559793005ef - main
  10:     0x7f44d98354e0 - __libc_start_main
  11:     0x5559792dad79 - _start
  12:                0x0 - <unknown>

This is due to some code outside of the master branch. The spawn functionality should be reworked there.


info.1pt.co

Display metadata on links such as:

when the link was created
how many clicks it has
which country it was created in

https://github.com/1pt-co/info.1pt.co


[wip] integrating fluxible-router

Still to do:

[x] fix styling issues

For some reason the nav links are not floating right and the logo text isn't always hiding when it should.

Depends on update to fluxible-router: https://github.com/yahoo/fluxible-router/pull/12

CLA is valid!

:shipit:

Because we're moving away from https://github.com/yahoo/fluxible-router/pull/12 in favor of a less opinionated router, I'll be updating this PR to follow the ideas in https://github.com/yahoo/fluxible-router/pull/14.

Depends on yahoo/fluxible-router#14 before we can merge.

:+1:

:+1: once the demo links are moved.

Coverage increased (+2.12%) to 92.94% when pulling 1ca0b188c3ba0faded3482ad477d4debc780cf01 on jedireza:fluxible-router into e7d1656a1e7765b988a6e62aaf5d23ff76093e5c on yahoo:master.

Coverage increased (+2.12%) to 92.94% when pulling 1ca0b188c3ba0faded3482ad477d4debc780cf01 on jedireza:fluxible-router into e7d1656a1e7765b988a6e62aaf5d23ff76093e5c on yahoo:master.

:+1:

Coverage increased (+2.12%) to 92.94% when pulling 361af7ec452b6c3e06289ff59d390be7e6b3b3d2 on jedireza:fluxible-router into e7d1656a1e7765b988a6e62aaf5d23ff76093e5c on yahoo:master.


Camera Permission on devices aksed every page reload

Hello,
I've integrated this component (Thanks) and  what I see today is :

On Desktop, permission is asked first time, then camera is working after each reload
On mobile (I'm testing with Iphone SE, IOS 11 on Safari), everytime I'm reloading the page, same question over again "asking for permission".
Don't really know if it's a ngx-scanner question or a "permission handling by browser" question. Sorry if I'm mistaken.

Thanks
Ivan

Different devices (browser) have some different behaviors. e.g.: Samsung Internet Browser asks for permissions everytime you choose a different camera. Chrome Browser in most Androids asks just the first time.
This is a difficult thing to track, so I will not close until @werthdavid gives his point.
Thanks for reporting. 😁

I did some research and it seems that you can't prevent Mobile Safari from asking, sorry for that.
But: iOS 11.3 has several changes regarding WebRTC, maybe the behavior changed there. I'll close the issue

Try to turn off livereload


SETHOOK and DEL event

I believe, that it would be nice idea to have event subscription mask setup during SETHOOK
Currently del, drop and set events causes hook to fire.
In my case I am using ~20000 web hooks, and del is not interested. Moreover, it produce unneccessary traffic to endpoint.
I would be nice to specify that 'set' is the only interesting event during SETHOOK declaration

Sure, this is doable.

I just pushed an update that adds the COMMANDS keyword. It works like the DETECT keyword.
NEARBY fleet FENCE COMMANDS set,del POINT 33 -115 10000

Will only send set and del commands.
NEARBY fleet FENCE COMMANDS set POINT 33 -115 10000

Will only send set commands.
When the COMMANDS keyword is not provided, all notifications are sent.
Let me know if this works for you.

Nice job @tidwall  - thanks - It will be super useful for my Tile38 demo tomorrow.
Hope to join your dev process soon

That's great to hear! Thanks a bunch for your help and good luck with tomorrow's demo.


Need react version of providing user info

This is an enhancement

[ x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Documentation issue or request

Description
DV needs access to the userinfo.    @gashcrumb has pointed to the following that could be ported over:
https://github.com/syndesisio/syndesis/blob/master/app/ui/src/app/core/providers/user-provider.service.ts#L24

@seanforyou23 FYI we'll need this also for showing the right user in the nav bar.

I'd probably create a WithUser component that just makes the response from /users/~ available.  It'll be a lot like this one as the API just returns a simple object that looks like { username: "foo" } afaik.
For logout eventually I'd probably create a WithUserHelpers and it'd look similar to this file and put the code for logout in there ultimately.  But I'd say focus on WithUsers for this issue.

Also we could probably just set this user in the AppContext using WithUser by adding it around here, that way we don't need to invoke that API, instead code that needs the user name can use an AppContext.Consumer.

Also we could probably just set this user in the AppContext  using WithUser by adding it around here , that way we don't need to invoke that API, instead code that needs the user name can use an AppContext.Consumer.

+1


v4.0.0 - Portal EmailSettings - Credentials not working

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request

Module Version

4.0.0

Affected Resource(s)

ArcGIS_PortalSettings
ArcGIS.psm1

Configuration Files

        "EmailSettings": {
            "SMTPServerAddress": "smtp.test.com",
            "From"<EMAIL_ADDRESS>            "Label": "MyPortal",
            "AuthenticationRequired": true,
            "EncryptionMethod": "TLS",
            "SMTPPort": "325",
            "UserName": "mydomain\\myuser",
            "Password": "mypassword"
        },

Expected Behavior

PortalEmail settings with credentials should have been set.
Actual Behavior

The credentials are not set and an error is thrown. The credentials are not handed over from ArcGIS.psm1 to the ArcGISPortalSettings.ps1. The variable is wrong.
Steps to Reproduce

Just use an email settings with credential for portal base deployment.
Important Factoids

On-Prem Azure (westeurope)
References

ArcGIS.psm1 - Line 1378 should be:
$PortalSettingsArgs["EmailSettingsCredential"] = $EmailSettingsCredential

@scma-esrich fyi

Hello @lorenzmeyer77  and @scma-esrich this issue has been addressed in v4.0.1.


Slow GUI on 4K monitor

Description
What steps will reproduce the problem?

Important update to what's below: This only happens when I launch Spyder from the Ubuntu command line in Windows 10. When I run Spyder from Windows itself, the problem is not reproduced. This makes it more likely that the issue lies in something to do with my settings, or with Windows 10's implementation of Ubuntu. Still, though, I'd be curious to know what's going on.
I just upgraded to Spyder 4.0.0 on Windows, and the GUI is extremely sluggish on 4K screens. More specifically, any graphical things I do (using the menu bar at the top, changing Preferences, etc.) are very slow (I have to wait a couple of seconds each time I use the backspace key in defining my own syntax coloring, for example). The actual editor window isn't sluggish at all, and neither is iPython.
This problem only happens on my 4K monitor; I have a normal 1080p monitor and the sluggishness disappears if I simply drag Spyder over to it. I have also tried several different DPI scalings (Normal, Enable auto high DPI scaling, and a couple custom scalings), to no avail.
Reproducing the problem might be immediate on a different 4K monitor. Just open Spyder and navigate to some graphical menu (for example, Preferences). It becomes immediately obvious for me.

If you find a way to solve this, please let us know with a comment below.


Test: new default file watcher

Refs: https://github.com/microsoft/vscode/issues/132483

[ ] macOS @TylerLeonhardt
[x] linux @lramos15
[x] windows @joaomoreno

Complexity: 5
Create Issue

The file watcher (vscode-nsfw) that was previously only used in multi-root workspaces is now enabled by default on all platforms even when opening a single folder. On top of that, the watcher service that leveraged it in VSCode was rewritten.
Setup: best to test this on a larger code base (like VSCode itself) and doing changes not only in root but also in deep hierarchies. In order to diagnose issues you can:

run code-insiders --verbose
filter the Console by putting nsfw into the filter
monitor the output: you will see every raw file event reported from the nsfw library and a normalised event signalling the event that is actually being sent to VSCode

IMPORTANT: over the years we have built a lot of measures to workaround missing file events making it actually a bit harder to test the functionality of these events. As such:

never do a file modification in the same VSCode window that you observe because when e.g. deleting a file using VSCode explorer, we remove it instantly knowing that it was deleted
even when you do a change outside of the VSCode window, make sure to observe the change in the window without moving focus into the window - we refresh the explorer and opened editors when that happens assuming there was a change

Testing Events

basic operations are reflected in the VSCode explorer when done from a VSCode explorer in another window

file/folder create
file/folder delete
file/folder move
file/folder rename
file/folder copy

the same is true when doing these operations from the native OS explorer (e.g. Windows explorer, macOS Finder)
changes to files are reflected in the editor if the file is opened in the editor

Testing Robustness

even under large pressure the watching never stops working (e.g. large npm install or git operation) - monitor devtools output for information in case it stops working
macOS/Linux: delete the folder that you opened as workspace and bring it back and verify the file watcher continues to work without window reload
Linux: configure a low fs.inotify.max_user_watches (instructions), restart and open a large folder with VSCode and verify you see a warning notification telling you how to increase that limit

Testing New Settings

ensure you have a folder in your workspace that is a symbolic link to a folder outside the workspace and use the new files.watcherInclude setting to enable it for watching and verify that works
ensure you can force to use our old watcher when configuring files.legacyWatcher. if you run with --verbose and search for "watcher", the name of the watcher will be printed (chokidar for macOS, Linux and C# for Windows)

ensure you can force to use our old watcher when configuring files.legacyWatcher. if you run with --verbose and search for "watcher", the name of the watcher will be printed (chokidar for macOS, Linux and C# for Windows)

I only see [File Watcher (node.js)] does that imply chokidar? It does say [File Watcher (nsfw)] before the change.

ensure you have a folder in your workspace that is a symbolic link to a folder outside the workspace and use the new files.watcherInclude setting to enable it for watching and verify that works

So does this mean I could add a ton of paths to this setting that are not opened in VS Code and nsfw will still watch them regardless? Could I add / as a path in my workspace settings and slow down the user's machine?

@TylerLeonhardt you should see [File Watcher (chokidar)] but I forgot to mention that for the file watcher to be used, you need to open a folder or multi-root workspace and not be in an empty window with loose files. And even when you are in a folder or workspace: any file that is outside of that workspace is watched by a custom node.js based watcher that is only watching that particular file. So maybe that happened here? You can try again by opening a folder and observing files of that folder?

So does this mean I could add a ton of paths to this setting that are not opened in VS Code and nsfw will still watch them regardless? Could I add / as a path in my workspace settings and slow down the user's machine?

No, we only allow paths that are inside the workspace:
https://github.com/microsoft/vscode/blob/24b3b589bbfd5115f36f0eb90063299466e96b18/src/vs/workbench/contrib/files/browser/workspaceWatcher.ts#L132-L144


Modelling ListView's ViewCell navigation to a Page and back

I am trying to figure out Exrin approach to model Container(Region), Stack etc in a variety of cases: MasterDetail, TabbedPage, CarouselPage.
=> Please let me know if the diagram is consistent with your concept, where it is wrong?
=> Question: For a SingleContainer where (region = container) it contains a stack. Instead of hosting navigationPage with many pages, how about hosting a ListView with many ViewCells with each ViewCell (tapping) leading to navigation to other page: Is the diagram consistent with your concept?
=> Suggestion:
Is it possible to navigate from other page to a specific ViewCell of a ListView?
Is it possible to navigate SPECIFICALLY to a particular ViewCell of a ListView? For example, when the navigation is triggered to a particular ViewCell (e.g. ViewCell5), The ListView will adjust the scrolling so that ViewCell5 is position top of the screen. Can this be done? How to model that in Exrin?

Yes, this is mostly correct. The ListView's though, are inside ContentPage's, not directly in a Navigation Page.
Hence if you want to navigate from a ViewCell to another page, or directly to a ViewCell in a page, you just need to navigate to that page, then in that ViewModel of the Page, use the ScrollTo method to bring the exact ViewCell into view.

@adamped  Congratulation of Exrin 2.0 beta. Clearly Logical!


[Feature] Ability to move nodes/applications.

Is this a bug or a feature request?
Feature
What did you expect?

Possibility to move nodes from an application to another as admin or as user from an organization
Possibility to move applications with nodes from an organization to another as admin.

This should probably be asked at the lora-app-server repo: https://github.com/brocaar/lora-app-server/issues.

Yes, please create this issue there.
(Please note that this feature might not be possible, as nodes of application 1 could be on a different network-server than the nodes of application 2.)


Add sourcemap support for testing

Fixes #195

No problem!
On Wed, May 2, 2018 at 8:41 PM Alex Eagle<EMAIL_ADDRESS>wrote:

@alexeagle approved this pull request.
Nice, thank you! Too bad we don't have a test case for this but I imagine
it's pretty hard
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_typescript/pull/196#pullrequestreview-117114303,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADs5098ssYAh6c8GW1iH2cHv4L7RBVSks5tulIpgaJpZM4Tvy4k
.


[carbon-dashboard] Upgrade linguiJS dependency

Currently React throws this warning (at least in the carbonmark-data project) : Trans: Support for defaultProps will be removed from function components in a future major release
This has been solved by linguiJS development team: https://github.com/lingui/js-lingui/issues/1493
We should upgrade lingui to benefit from this

Fixed by https://github.com/lingui/js-lingui/pull/1494


Construct evaluation chains

These have been in the works for a while. This is useful for multi-step tasks where validation at one stage is useful. It'll prevent instances of irrelevant context being passed to a chain with the LLM hallucinating or failing at a user-facing step.
This will require a bit of architecture that I've gotten comfy with in the notebooks. It will need few shot examples, as well as a schema validation. In many of these cases, a boolean output is sufficient, but enforcing json will be better so I can handle nuanced cases/errors in the future.

I've gotten much more practice with making eval chains and composing them into more nuanced workflows! I want to experiment with routing a bit more, then we'll start making some proper multi-step chains that call specified functions.

A demo has been implemented in get_eval_chain() in chains.py


winston transport console set level=silly, but it only error or debug.

const testLogger = winston.createLogger ({
    transports :[
        new winston.transports.Console({
                level: 'silly'
          })
    ]
})

testLogger.log('debug',"Log debug");
testLogger.log('error', "Log error");
testLogger.log('info', "Log info");
testLogger.log('warn',"Log warn");
testLogger.log('verbose', "Log verbose");

The console out only show
{"level":"debug","message":"Log debug"}
{"level":"error","message":"Log error"}
The problem seems only happens on Console, other transport seems working properly.
I am using 3.0.0-rc4

excellent observation, in case of VS Code;
"outputCapture": "std"
helped to get all the messages to console. @indexzero your post is a great hint, that for spending your time on this.


feat(request): Introduce matchedRoutes and routePath for HonoRequest

app.use('*', async function logger(c, next) {
  await next()
  c.req.matchedRoutes.forEach(({ handler, method, path }, i) => {
    const name = handler.name || (handler.length < 2 ? '[handler]' : '[middleware]')
    console.log(
      method,
      ' ',
      path,
      ' '.repeat(Math.max(10 - path.length, 0)),
      name,
      i === c.req.routeIndex ? '<- respond from here' : ''
    )
  })
})
app.use('*', async function namedMiddleware(c, next) {
  await next()
})
app.get('/*', async (c, next) => await next())
app.get('/hi/:name', (c) => c.json({ name: c.req.param('name'), path: c.req.routePath }))

% bun run --hot app.ts
Started server http://localhost:3000
ALL   /*          logger
ALL   /*          namedMiddleware
GET   /hi/*       [middleware]
GET   /hi/:name   [handler] <- respond from here

% curl http://localhost:3000/hi/hono
{"name":"hono","path":"/hi/:name"}

Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

@usualoma
Could you resolve the conflicts?

@yusukebe Oops. Resolved!

Hi @usualoma
This is great. I like the minimal changes to the code and the features are implemented, Context and routers are not changed.
I made only one comment. Check it. Thanks!

@yusukebe Thank you for your review. These changes would improve DX.
Fixed problem in 4c596ac!

Thanks!
Okay, merge now. Maybe I'll release the next minor version "v3.11.0" soon.


New Doc entry

Description:
Clarification on documentation about the x509 certificate component
Steps to reproduce:
Log in with a secondary user store user following the X509Certificate validation (https://is.docs.wso2.com/en/latest/learn/x509certificate-authenticator/#adding-a-claim-mapping-for-the-certificate)
PR: https://github.com/wso2/docs-is/pull/3142

Closing since this part is already added to the doc with https://github.com/wso2/docs-is/pull/3125/.


Jdbc context run action returning result is type as single value instead of list

Version: v3.8.0
Module: quill-jdbc
https://github.com/getquill/quill/blob/876cae2c6b56317f373426b6049a40273d8301d0/quill-jdbc/src/main/scala/io/getquill/context/jdbc/JdbcContext.scala#L24
Expected behavior
Delete returning should return a list as many value could have been deleted.
Actual behavior
Return only the first deleted value, same behavior on update. (may be induce by insert returning?)
Steps to reproduce the behavior
Define a quote using returning on delete or update, result is a single value
Workaround
Unkown
@getquill/maintainers

I'm having the same problem.  I'd offer to fix it myself but I'm going to be busy with personal matters for the next few weeks.
Hopefully I'll have some time to take a look and the issue is simple enough that a novice like me can fix it.
This seems especially strange to me because update/delete should be able to return zero results without error so this behavior seems simply incorrect.

Yeah, it definitely sounds incorrect. @tflucke if you want to look into it I can try to provide some guidance.

Perhaps consider making it "returningList" or similar and possibly deprecating and renaming "returning" to "returningOne"?
I believe the root issue is JdbcRunContext.scala:65 just needs to not wrap extractResult in handleSingleResult:
  def executeActionReturning[O](sql: String, prepare: Prepare = identityPrepare, extractor: Extractor[O], returningBehavior: ReturnAction)(info: ExecutionInfo, dc: Runner): Result[O] =
    withConnectionWrapped { conn =>
      val (params, ps) = prepare(prepareWithReturning(sql, conn, returningBehavior), conn)
      logger.logQuery(sql, params)
      ps.executeUpdate()
      handleSingleResult(extractResult(ps.getGeneratedKeys, conn, extractor))
    }

I started chasing this down (#2461). It doesn't look like it's too bad, but since executeActionReturning is part of the top level quill API it's a pretty impactful change. I think we should decide whether or not to change the existing api before chasing down all the implementations.
Implementing a new "returningList" or similar with more appropriate semantics would make it possible to limit the blast radius of the change, maintain backwards compatibility and possibly even maintain backwards binary compatibility.

What would be the semantics of "returningOne"?  What happens specifically when:

There are zero results?
There are two or more results?
There is an error?

It seems contrary to the design philosophy of Quill to start introducing new semantics like this.  Is the goal just to maintain backwards compatibility?  If so, I would leave "returning" as-is deprecated and not introduce anything to replace it.
If a user needs the old functionality, they can use ".headOption" or a match statement and write their own error handling (which should have been how this was originally handled).

I think it should be returningMany and there should be a corresponding executeActionReturningMany. @easel if you'd like to try to do implement this (or anyone else) I can try to sketch out what needs to be done. Please let me know.


Error on torch functions which are not part of the ltorch

As per title. We error and recommending using thunder.examine.examine for checking support.

@carmocca , I have addressed your comments.  Let me know if you have more to say.

Thank you @nikitaved!


ECS cookbook?

I'm a beginner when it comes to ECS architecture and it would really help if there were people kind enough to share their techniques or common patterns in ECS framework (like multiple features sharing the same component, handling Game objects with multiple heirarchies etc.)

@mzaks Has created this all though it's not complete yet.

We do have a FAQ in the wiki which can serve a similar purpose, but why is that cookbook a separate repository instead of a part of the wiki? The official wiki seems like a much better, more visible and much more easily editable place for this type of content.

Having read it all, it would awesome if he continues writing it. It would be awesome to have a book that goes more in depth with ECS as there aren't any book (that i could find) written solely for it.

Yeah, yeah, I just wrote another two chapters. Sorry for being so slow, it's just a side project so I return too it only when being kicked 😀. Keep kicking me, other wise I will digress into doing some other things 😅.


test: Support running the test suite without oauth2client

Make it possible to run the test suite without oauth2client, by skipping the few tests that specifically require it.  Given that oauth2client is deprecated, we are aiming towards removing it entirely from Gentoo and the unconditional imports in the test suite are a blocker for that.
I've also added running without oauth2client as an additional nox scenarios but I can remove that if you prefer.

Thanks a lot!


[armadillo] build failure

Operating system
Linux
Compiler
GCC
Steps to reproduce the behavior
`vcpkg new --application`
`vcpkg add port armadillo`

add some files like CMakeLists.txt, main.cpp...

`cmake --preset=default`

Failure logs
Preset CMake variables:

  CMAKE_TOOLCHAIN_FILE="/opt/vcpkg/scripts/buildsystems/vcpkg.cmake"

-- Running vcpkg install
Detecting compiler hash for triplet x64-linux...
Compiler found: /usr/sbin/c++
The following packages will be built and installed:
<EMAIL_ADDRESS>  * blas:x64-linux@2023-03-25
  * lapack:x64-linux@2022-02-22#2
  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  *<EMAIL_ADDRESS>  * vcpkg-cmake:x64-linux@2023-05-04
  * vcpkg-cmake-config:x64-linux@2022-02-06#1
Additional packages (*) will be modified to complete this operation.
Restored 6 package(s) from /home/olioolii/.cache/vcpkg/archives in 84.6 ms. Use --debug to see more details.
Installing 1/9<EMAIL_ADDRESS>Elapsed time to handle vcpkg-cmake-config:x64-linux: 466 us
vcpkg-cmake-config:x64-linux package ABI: 49ad0cd6a7940645e5d23a5ff430ec635809b643920acdc4bd1881c3853bd0db
Installing 2/9<EMAIL_ADDRESS>Elapsed time to handle vcpkg-cmake:x64-linux: 222 us
vcpkg-cmake:x64-linux package ABI: e25d4f83bebb2f875d1558903ae17bbf5c61f29df2ca70da2d33e6aac16976f2
Installing 3/9<EMAIL_ADDRESS>Elapsed time to handle pthreads:x64-linux: 192 us
pthreads:x64-linux package ABI: c5f5f65e1fa7d4265daa9d863af8343e9fa96fe07ad4616cb68056be7740f009
Installing 4/9<EMAIL_ADDRESS>Elapsed time to handle pthread:x64-linux: 197 us
pthread:x64-linux package ABI: 7961449c8a107bef496b1b853da579a9e222943d09f28f17de36600b78df3a68
Installing 5/9<EMAIL_ADDRESS>Elapsed time to handle openblas:x64-linux: 943 us
openblas:x64-linux package ABI: 380a254c69f09dbf3c1bc893e03ae18b642f0f282970b565160af7581b34a86d
Installing 6/9<EMAIL_ADDRESS>Elapsed time to handle blas:x64-linux: 204 us
blas:x64-linux package ABI: 6bf4243424a1442e2403c072594ba295b66d4002a4d5ef06756d213e3acd8b9d
Installing 7/9<EMAIL_ADDRESS>Building<EMAIL_ADDRESS>-- Using cached Reference-LAPACK-lapack-v3.11.0.tar.gz.
-- Cleaning sources at /opt/vcpkg/buildtrees/lapack-reference/src/v3.11.0-5e6de67bfb.clean. Use --editable to skip cleaning for t
he packages you specify.
-- Extracting source /var/cache/vcpkg/Reference-LAPACK-lapack-v3.11.0.tar.gz
-- Applying patch cmake-config.patch
-- Applying patch lapacke.patch
-- Using source at /opt/vcpkg/buildtrees/lapack-reference/src/v3.11.0-5e6de67bfb.clean
-- The Fortran compiler identification is unknown
CMake Error at scripts/cmake/vcpkg_find_fortran.cmake:62 (message):
  Unable to find a Fortran compiler using 'CMakeDetermineFortranCompiler'.
  Please install one (e.g.  gfortran) and make it available on the PATH!
Call Stack (most recent call first):
  ports/lapack-reference/portfile.cmake:54 (vcpkg_find_fortran)
  scripts/ports.cmake:175 (include)

error: building lapack-reference:x64-linux failed with: BUILD_FAILED
Elapsed time to handle lapack-reference:x64-linux: 1.7 s
Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+lapack-reference
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?title=[lapack-reference]+Build+error+on+x64-linux&body=Copy+issue+body+from+%2Fho
me%2Folioolii%2FProjects%2Fuim-copy%2Fbuild%2Fvcpkg_installed%2Fvcpkg%2Fissue_body.md

-- Running vcpkg install - failed
CMake Error at /opt/vcpkg/scripts/buildsystems/vcpkg.cmake:899 (message):
  vcpkg install failed.  See logs for more information:
  /home/olioolii/Projects/uim-copy/build/vcpkg-manifest-install.log
Call Stack (most recent call first):
  /usr/share/cmake/Modules/CMakeDetermineSystem.cmake:146 (include)
  CMakeLists.txt:3 (project)

CMake Error: CMake was unable to find a build program corresponding to "Ninja".  CMAKE_MAKE_PROGRAM is not set.  You probably nee
d to select a different build tool.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!

Additional context
Here are my source files and configuration files.
main.cpp:
#include <armadillo>

int main() {
    arma::mat matrix{{1, 2, 3}, {4, 5, 6}};
    matrix.print();
    return 0;
}

CMakeLists.txt:
cmake_minimum_required(VERSION 3.20)

project(test)

find_package(Armadillo CONFIG REQUIRED)
add_executable(main main.cpp)
target_link_libraries(main PRIVATE ${ARMADILLO_LIBRARIES})

CMakePresets.json
{
  "version": 2,
  "configurePresets": [
    {
      "name": "default",
      "generator": "Ninja",
      "binaryDir": "${sourceDir}/build",
      "cacheVariables": {
        "CMAKE_TOOLCHAIN_FILE": "$env{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake"
      }
    }
  ]
}

vcpkg.json
{
  "dependencies": [
    "armadillo"
  ]
}

I think it's an issue with this package armadillo, as I was able to build the project properly when I switched to other package like fmt.
Here are tools installed.

Sorry. I found that I forgot to install gfortran with sudo pacman -S gcc-fortran.


feat: Implement hiding credentials used with digest security scheme

Description

Implements hiding of new security scheme digest and its credentials. Also resolves #27.
Motivation and Context

Possibility of digest security scheme use.
Types of changes

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

Checklist:

[ ] I have updated the documentation accordingly. If you are changing code related to user secrets you need to really make sure that security documentation is correct.
[x] I have read the CONTRIBUTION_GUIDE document.
[x] I haven't repeated the code. (DRY)
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@martinalbert why wasn't this merged? Do you plan it?


deleting a stack page

Description of the change

Checklist
Please make sure that the proposed change checks all the boxes below before requesting a review:

[x] I have reviewed the guidelines for contributing to this repository.
[x] The preview looks fine.
[x] The tests pass.
[x] The commit history is clean and meaningful.
[x] The pull request is opened against the main branch.
[x] The pull request is no longer marked as a draft.
[x] You agree to license your contribution under the MIT license to Spacelift (not required for Spacelift employees).
[x] You have updated the navigation files correctly:

[ ] No new pages have been added, or;
[ ] Only nav.yaml has been updated because the changes only apply to SaaS, or;
[ ] Only nav.self-hosted.yaml has been updated because the changes only apply to Self-Hosted, or;
[ ] Both nav.yaml and nav.self-hosted.yaml have been updated.

If the proposed change is ready to be merged, please request a review from @spacelift-io/solutions-engineering. Someone will review and merge the pull request.
Spacelift employees should request reviews from the relevant engineers and are allowed to merge pull requests after they got at least one approval.
Thank you for your contribution! 🙇

Since this also applies to self-hosted, make sure to update its navigation. See https://github.com/spacelift-io/user-documentation/blob/main/CONTRIBUTING.md#self-hosted.


Review asserts() all over the code and enable assert() for release build

If possible try to return errors and handle them instead of triggering an assert().
It is better to have release build with assert() enabled. For that need to investigate how to undef NDEBUG, CMAKE defines NDEBUG by default for release build.

I have changes for the second part; but we will still need to review our assert()s.


NIFI-2635 - Reopen Cluster Two Phase Commit

PR for the last two commits that were lost when merging PR-333. I've updated the commits for the current state of master.

Reviewing

+1
Went through code in this commit and compared to the code missing from my rebase of PR #933 and it looks like it is inclusive of all the changes. Did a contrib check build and verified the features that were missing. Sorry for messing up #933 @mcgilman, I will merge this in.


Adds Persian translations

I've added the Persian translations to the project.

Thanks Faraz, merging :)


DONE!

That's it! I've played through every adventure currently available with a single character. And created a complete set of maps.
I've logged lots of reports, but overall I have to say that the system works very well, is a lot of fun to play and is getting better all the time.  Thanks for bringing Eamon to the internet.  You've done an amazing job.  I look forward to may more hours of fun and profit.
Damien

I've played through every adventure currently available with a single character. And created a complete set of maps.

That's cool! Would you be willing to share the maps you made? I edit a wiki dedicated to Eamon (eamon.wiki) and the maps sound like they'd be a great addition to the articles.
Just a thought; thanks!
~Huw

I've played through every adventure currently available with a single character. And created a complete set of maps.

That's cool! Would you be willing to share the maps you made? I edit a wiki dedicated to Eamon (eamon.wiki) and the maps sound like they'd be a great addition to the articles.
Just a thought; thanks!
~Huw


fix: update Vite peer dependency version

Sets 5.0.3 as the minimum because earlier versions are broken with SvelteKit

wouldn't it make more sense to add it to kit? technically this is q breaking change.


armel: run on ubicloud

It's important to note that this breaks armel builds on forks. I suspect (but have not tried) that those forks can make them work by creating a runner group called ubicloud-standard-2-arm that will then run on GH runners and use qemu.

those forks can make them work by creating a runner group called ubicloud-standard-2-arm that will then run on GH runners and use qemu.

I was wrong about this.

perhaps https://stackoverflow.com/questions/71961921/specify-runner-to-be-used-depending-on-condition-in-a-github-actions-workflow

second commit makes fork use qemu instead of failing


Provided String, expected DateTime BUG

Bug description
Hi, I have problem with DateTime attribute.
Schema:
model Project {
  id          String     @id @default(cuid())
  name        String     @unique
  description String
  manager     Profile   @relation(fields: [managerId], references: [user])
  managerId   String
  startDate   DateTime
  endDate     DateTime?
  activities  Activity[]
}

JSON:
{
        id: '1',
        name: 'projectA',
        description: 'Lorem Ipsum dolor sit amet',
        manager: {
            connect: {user: '2'}
        },
        startDate: "2020-03-19T14:21:00+0200",
        endDate: "2021-03-19T14:21:00+0200",
}

When I want to create project, it says
Argument startDate: Got invalid value '2020-03-19T14:21:00+0200' on prisma.createOneProject. Provided String, expected DateTime.
What am I doing wrong?
Environment & setup

OS: MacOS Catalina
Database: PostgreSQL
Prisma version:  2.0.0-beta.2
Node.js version: v10.15.3

You will need to pass an ISO 8601(https://en.wikipedia.org/wiki/ISO_8601) compatible date string if you want it to get automatically parsed.
Otherwise, you can pass Date objects to the client so that client can internally serialize them by calling toISOString on the date object.
startDate: new Date("2020-03-19T14:21:00+0200"),
endDate: new Date("2021-03-19T14:21:00+0200"),


Add cattrs benchmark

Change Summary
Adds a benchmark against attrs+cattrs.
I checked, and each of the test cases in the benchmark has the same outcome in this cattrs benchmark as in the original pydantic benchmark, so I think I set up all appropriate validations.
The benchmark has only been tested to work with Python 3.7.
On my computer, I get the following results:
...
pydantic time=2.071s, success=50.60%
pydantic best=1.728s, avg=2.189s, stdev=0.294s

...
cattrs time=0.834s, success=50.60%
cattrs best=0.533s, avg=0.700s, stdev=0.133s

attrs+cattrs is obviously lacking a lot of built-in functionality compared to pydantic, so even if it is faster I'm not sure it would make sense to use as a substitute (although I was able to replicate all the validations used in the benchmark without too much effort), but based on these numbers maybe there are some tricks attrs/cattrs are using that could help speed up pydantic?
Related issue number
https://github.com/samuelcolvin/pydantic/issues/542
Checklist

[ ] Unit tests for the changes exist
[ ] Tests pass on CI and coverage remains at 100%
[ ] Documentation reflects the changes where applicable
[ ] HISTORY.rst has been updated

if this is the first change since a release, please add a new section
include the issue number or this pull request number #<number>
include your github username @<whomever>

have you seen the discussion on #513?

Ah, yeah looks like an almost perfect duplicate! Sorry about that; I thought I had searched for it but clearly not!


wrap lon can lead to much slower masking

masking with rasterio requires equally spaced lat/ lon coordinates. However, wrapping the longitude can lead to coordinates that are no longer equally spaced (i.e. if the regions go from -180...180 and lon from 0..360):
import numpy as np
import regionmask
lat = np.arange(89.5, -90, -1)
lon = np.arange(0.5, 360)

lon_wrapped = regionmask.core.mask._wrapAngle(lon)

print(regionmask.core.mask.equally_spaced(lon, lat))
print(regionmask.core.mask.equally_spaced(lon_wrapped, lat))

True
False

Unfortunately masking with shapely is quite a bit slower:
countries_50 = regionmask.defined_regions.natural_earth.countries_50
%timeit countries_50.mask(lon, lat)
%timeit countries_50.mask(lon[:180], lat); countries_50.mask(lon[180:], lat)

1.08 s ± 13.2 ms per loop
116 ms ± 2.23 ms per loop

Solution: split lon into two parts and rasterize them seperately with rasterio.
xref #121

lon2 = np.arange(-179.5, 180)
%timeit countries_50.mask(lon2, lat)

57.7 ms ± 536 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)

After #132 the timings are now
# can be re-arraged
%timeit countries_50.mask(lon, lat)
# is split into two parts and masked
%timeit countries_50.mask(lon[:-1], lat)

%timeit countries_50.mask(lon, lat, method="shapely")

63.6 ms ± 624 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)
130 ms ± 832 µs per loop (mean ± std. dev. of 7 runs, 10 loops each)
1.19 s ± 5.2 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)


failed to install

trying to install the package i get:
PM> Install-Package NugetPackageRestore
Installing 'NugetPackageRestore 1.0.3'.
Successfully installed 'NugetPackageRestore 1.0.3'.
Adding 'NugetPackageRestore 1.0.3' to zuers_geo_gp_tools.
Uninstalling 'NugetPackageRestore 1.0.3'.
Successfully uninstalled 'NugetPackageRestore 1.0.3'.
Install failed. Rolling back...
Install-Package : Could not install package 'NugetPackageRestore 1.0.3'. You are trying to install this package into a project that targets
'.NETFramework,Version=v3.5', but the package does not contain any assembly references or content files that are compatible with that
framework. For more information, contact the package author.
At line:1 char:1

Install-Package NugetPackageRestore

  + CategoryInfo          : NotSpecified: (:) [Install-Package], InvalidOperationException
  + FullyQualifiedErrorId : NuGetCmdletUnhandledException,NuGet.PowerShell.Commands.InstallPackageCommand

trying to install the package via the ui fails, since i cannot find it there using the search

This package targets .Net Framework version 4, if you could upgrade your project to version 4 the you will be able to install the package.

Ah ok. Unfortunately i cannot upgrade since that library works with 3.5
only. Are there major differences between 3.5 and 4 from ths msbuild
perspective, or would it be easy to recompile the task for 3.5?
Thanks!
On Thu, Feb 12, 2015 at 6:42 PM, Alberto Rechy<EMAIL_ADDRESS>wrote:

This package targets .Net Framework version 4, if you could upgrade your
project to version 4 the you will be able to install the package.
—
Reply to this email directly or view it on GitHub
https://github.com/deeplight77/NugetPackageRestore/issues/3#issuecomment-74116263
.


Post Data Encoding?

why Post method need a map for form data and the key and value not encoding ?
e.g # not replace with %23

@CrazyMouse I believe the map is very simple and flexible to work with.  It could be done with something like url.Values but even this is just a wrapper around a map[string][]string.  To be honest most people use the form values as a simple map so it simplifies things.
As for the value encoding, can you submit an example of it not working?  I have not found your issue to be the case.

Hi Smith，
I am going sorry for the issue I submited 。after that I found It’is my web
server’s issue .
i hope you can understand my bad english
best wishes for you
Vincent Smith<EMAIL_ADDRESS>周五下午10:52写道：

@CrazyMouse https://github.com/CrazyMouse I believe the map is very
simple and flexible to work with. It could be done with something like
url.Values but even this is just a wrapper around a map[string][]string.
To be honest most people use the form values as a simple map so it
simplifies things.
As for the value encoding, can you submit an example of it not working? I
have not found your issue to be the case.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gocolly/colly/issues/183#issuecomment-404856920, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABH1Ht-BbWnjkJ14h3cUUm2bkX0pgtQJks5uGLQzgaJpZM4VCe3_
.

Lol no problem
你的英语不如我的中文差
(Google translate: "Your English is not as bad as my Chinese")


Events starting late

Hi. Thanks for releasing the code.
I noticed that the timestamp for office_zigzag dataset start from<PHONE_NUMBER>67, but the simulated events start from 19208. I tested with other image sequences and got similar issue. Can you tell me why this is happening?

Sorry for the late reply, hasn't been on github for a while. Will check and get back to you later. Thank you for your interest in our work.

I think the "late" emerging time might attribute to 2 factors. The first is that the difference between the first and second frame is kind of small (the biggest intensity change is 11, especially in the log domain). The second is that our algo will initialize a "clean" cmos residual voltage, which makes it not sensitive to noise at the beginning.


fix LimeSDR-Mini oversampling

Tested oversampling of 2 and 4 with sample rate of 40M.
May relate to https://github.com/myriadrf/gr-limesdr/issues/3 (i have only tested this PR with SoapySDR)

LimeSDR-Mini default oversampling settings are intentionally different from LimeSDR-USB. LimeSDR-Mini it is a smaller board that is more likely to be affected by heat issues, so lower frequencies are selected by default.

i am giving sample rate 500khz and oversampling is in default. so finally the sample rate i am seeing is (500khz * 32) 16Mhz. the oversampling factor is changing(32,16,8,4,2,1) depends on the sample rate which we given. why the oversampling is needed? for me 500khz sample rate is enough but the limesdrmini doing some inbuilt operation based on the input sample rate.can anyone please tell me about this oversampling operation in limesdrmini.

Oversampling factor changes signal sample rate at DAC/ADC. It is not changed at base band interface.

hi @ztamosevicius thanks for your answer. if i am giving 500khz sample rate and over sample factor as default(it takes 32) then what is the actual sample rate?. and if you don't mind, kindly elaborate the previous answer.

Hi @vetrivelmnv13,
Considering transmit case: you supply base band signal of 500kHz and this is a sample rate you supply to LMS7002M transceiver at digital interface. Inside of LMS7002 you set interpolation ratio of 32. In this case signal sample rate supplied to the DAC will be 500kHz * 32 = 16MHz.
Vice versa in case of receive.

In case of transmit:
Increasing the sample rate at the digital-to-analog-converter(DAC) allows for better TX filter roll-of. Essentially you get cleaner signal and less inference to neighboring bands.
In case of receive:
Oversampling is used to increase the sampling quality. Oversampling by 4 , and then decimating by 4 (basically averaging), gives you 3dB better SNR. For example: Your sample-rate is 1Mhz -> Signal is oversampled with 4Mhz -> Signal is decimated back to 1Mhz -> You just gained 3dB of SNR (won back from reducing the quantization noise, if your SNR is already close to thermal noise you  wont win anything)
So, you would always want to have maximum oversampling possible.

hi @gasparka,
as per my experiment the minimum sampling rate is 0.96Mhz can i reduce it to 0.5Mhz by using this decimation and interpolation?.why i am trying to reduce means, my signal bandwidth is just 250khz only.

Should be possible to set samplerate 1Mhz and then decimation by 2. But i dont know the details. Maybe @ztamosevicius  can help.

Thank you @gasparka

Hi @vetrivelmnv13,
The lowest frequency CGEN can generate is about 3.7MHz. By using division by 4, the lowest DAC/ADC sample rate will be 3.7MHz/4 = ~0.928MHz. If you engage decimation/interpolation by 2, your base band signal sample will be 0.928MHz/2 = 0.464MHz.

thank you @ztamosevicius,
one more doubt i have, i developed my code using LimeSuite api's. in that i didn't seen any api for interpolation/decimation. but in LMS7002M api the interpolation/decimation all those things present. so along with the Limesuite api's can i use LMS7002M api's ?

hi @gasparka,
as per my experiment the minimum sampling rate is 0.96Mhz can i reduce it to 0.5Mhz by using this decimation and interpolation?.why i am trying to reduce means, my signal bandwidth is just 250khz only.so i want to know can i reduce it below 0.96Mhz?

Have you tried using LMS_SetSampleRate() with oversampling set to 0 (auto)? It should set interpolation/decimation values automatically. If I remember correctly,  it used to be possible to set sample rate as low as 30 kHz.

hi @IgnasJarusevicius ,
i got it.it working fine.one small question i have, is there any good way to reduce the buffer size?. now it is (1024*1024)


I would like to be able to rerun my own travis jobs.

Feature Request
To manually restart jobs in travis, you need admin privileges on the linked github repo. It would be great if non-admin developers (such as myself), had a mechanism to rerun tests in the event of a flake or what have you. I know the main kube repos have a git robot that can do this if you comment '/retest'.
Is your feature request related to a problem? Please describe.
I can't rerun my own tests without forcefully changing the commit SHA somehow. I cannot rerun tests on others PRs at all.
Describe the solution you'd like
Some way to kick travis tests.

There are some options here that need to be investigated:

We can move to Red Hat OpenShift's prow instance, but this would require us to port our e2e test setup, which is not trivial due to the fact that prow jobs run in containers that lack access to docker sockets. We would need alternate ways to spin up clusters and build images used in the e2e tests.
We may be able to move to GitHub Actions. We would need to investigate what PR authors and repo approvers/reviewers would be able to do (ideally with slash commands)
Same as 2, but stay on travis.

I did some investigation around Github Actions. It seems a bit kludgy, but possible. Defining a github action basically consists of some light scaffolding around a Dockerfile. So we'd have to define a Dockerfile with the scripts and such for each test we want to run baked in. I don't see anything about unrestricted access to the docker socket, and assume we can't get that unless we host our own Github Actions server, which is an option apparently. Poking around a bit in my fork of OperatorSDK vs here, it looks like you require write access to be able to manually rerun Actions, and it's away in some weird menu that's not really anywhere near your PR. There looks to be github robots to let you do it with comment commands on the PR itself, though.

Here’s an example of something we could do: https://github.com/joelanford/helm-operator/blob/plugin/.github/workflows/ci.yml
A good chunk of that could be refactored into scripts in the repo, but I don’t think we’ll need any custom Dockerfile-based action implementations.

Bunch of discussion from the meeting. The decision was made to investigate Github actions at this time instead of Prow at this time as we know converting to Prow would be a bunch of work, and we can always switch to Prow later if Github actions don't work.
This issue is now tracking progress on:

try to write a github action to rerun my travis jobs while changing as little as possible
1a. investigate the possibility of implementing /commands via github actions to do other things such as adding labels to Github issues.

Things that are related and have been created as their own issues:
2. #4110 make CI jobs run in Github actions
3. #4112 make the state of the master CI jobs more obvious to developers


Add more information/links regarding Bluetooth 5 Long Range support

I found several links interesting links regarding Bluetooth 5 Long Range/LE coded PY support in smartphones.
A manufacturer that provides a list. (It seems to be a bit outdated as iPhones do not support LE coded PHY, but they are listed here):

https://support.global.mytracmo.com/portal/en/kb/articles/list-of-smartphones-that-support-bluetooth-5-long-range Perhaps we can extend the list with supported phones from this list and have a remark that it is unverified information?

A method to look up LE coded PY on the Bluetooth website for any smartphone/device:

https://docs.silabs.com/bluetooth/3.2/miscellaneous/mobile/finding-the-bluetooth-features-of-smartphones

I think that even with a free membership you can look up the SIG Qualification details to see if it is qualified for LE coded PY. @friissoren Intel is a Bluetooth member, would it be possible for you to extract a list of phones with LE coded PY support?
Interestingly most Apple iPhones seem to support LE coded PHY (first link), but it looks that Apple has dropped it after iOS 14 https://developer.apple.com/forums/thread/665542. So it is a software issue, that Apple could potentially enable again in the future.

Thanks. This is very interesting and those sites are new to me.

Perhaps we can extend the list with supported phones from this list and have a remark that it is unverified information?

I will try do some updates based on this information.
I don't know the details of how exactly they are doing their testing (most likely they don't need the Extended Advertising feature supported) or whether this info is just scraped from e.g. Google Playstore info about which devices supports which Android feature flags.

A method to look up LE coded PY on the Bluetooth website for any smartphone/device:

I tried to look-up the OnePlus 6T, which I know do have support for receiving the long range signals (at least part of the time). However, it was listed to not have LE Coded PHY support. So a clear contradiction there.
Looking up each phone via this method would clearly take some time and I wonder if it is worth the effort if the information can't be relied on anyway. Let's see.
(I apparently did have some Bluetooth SIG credentials, maybe via Intel?, don't remember when I got them, and was able to log into the site).

Apple has dropped it after iOS 14

I have read similar discussions earlier. If I understood correctly, even when they did have Coded PHY support, they had not implemented the Extended Advertising support to support more than some 100+ bytes, which meant it still wouldn't have been able to pick up all the pings, since they can be up to 255 bytes.
The lack of support in iOS (BT5, NaN, Beacon) has all the time been a SW issue. Apple really likes to do their own thing and not play ball with anyone else. Let's see what the future brings.

I tried to look-up the OnePlus 6T, which I know do have support for receiving the long range signals (at least part of the time). However, it was listed to not have LE Coded PHY support. So a clear contradiction there.

Of course a manufacturer can decide not to declare that the device supports LE Coded PHY officially. I guess if a manufacturer mentioned LE Coded PHY support for a phone, it will work for sure. If it is not listed, it may work like in the OncePlus-6T case.

would it be possible for you to extract a list of phones with LE coded PY support?

The problem is such a list only gives you a list of possibles. I am about to return a tablet (Samsung A7 Lite) that has a column of YESs on nRF Connect, but in reality only picks up a coded message every few seconds.

Consider making a PR for the list of supported devices: https://github.com/opendroneid/receiver-android/blob/master/supported-smartphones.md#list-of-devices-and-their-capabilities
But yes I agree that Bluetooth Long Range support is rare for smartphones and tablets and the situation does not seem to improve. The suggested links, but also the nRF Connect app are a first check to look for Bluetooth Long Range support for a particular model. As a second step users should verify that there is indeed good/continuous reception of those signals.

I'll probably do a PR when I find a tablet that works.
The problem is that there is no killer app that needs BT LR coded, so there is no incentive for the manufacturers to prioritise it. I just want the manufacturers to put it in their FAQ and/or in the script that they give to their 1st/2nd line support people.


Added preliminary JSON and MsgPack serialization and deserialization

Falk, thanks for the PR. I hope you don't mind being our guinea pig for the TA1/TA2 collaborative development process :)
This may already be on your radar, but for any PRs:

Please run our Python auto-linter. We use the PEP8 standards for the most part. See our documentation here: https://github.com/NextCenturyCorporation/MCS/blob/development/machine_common_sense/DEV.md#linting
Please add unit tests. We use the Python unitttest library. Our tests are located in the tests/ folder. See our documentation here: https://github.com/NextCenturyCorporation/MCS/blob/development/machine_common_sense/DEV.md#testing
If you introduce any new dependencies, please add them to our pip requirements file.

@deanwetherby I think had some ideas about some of this and python packages, so we should make sure he chimes in before we merge this as well.

@ThomasSchellenbergNextCentury Do the code changes I just pushed address your concerns? As aforementioned in response to your comments I would rather hold off with generic enhancements for now, since serialization can depend a lot on order, esp. for the flags we set, so I would prefer doing this iteratively. The class refactoring should be done, simple tests are added and my code should always be PEP8 compliant. Please let me know if there is anything you need to approve this.

Thank you very much for your review and helpful explanations. I have applied all suggestions and am looking forward to a final check and hopefully merge of this PR. Please just let me know if there is anything else I can improve.


Keep track of the number of card plays

It would be interesting to keep a tally of the number of times each White Card is played/winner, e.g. "Sean Penn won 18 of its 26 plays".

Should the stats be per-game (channel) or bot-wide?
How should the information be accessed?

22:53:44 <+dutchie> would have to be chan specific surely
22:54:11 < zuzak> I just want to know if anybody has actually won with "A
                  gassy antelope"
22:54:11 <+dutchie> unless you want to run a griffer master db
22:54:58 <+dutchie> guess a "report stats centrally" option might not be so
                    evil
22:55:40 <+dutchie> but then is that chan specific or instance wide?
22:55:44 < zuzak> I was thinking centrally
22:56:33 <+dutchie> somebody not on a phone should probably paste all this
                    into github
22:56:39 <+Dee> the bot spports multiple games iirc
22:56:52 <+Dee> bot/db wide makes sense to me
22:57:24 < zuzak> unless you want mildly creepy stats like "zuzak is a
                  great fan of 'Auschwitz'!"
22:57:59 <+ziphy> now i want that

Perhaps a playlogs table with columns…

id
game
user
text
was_winner
tsar_user
time

Primary key
Game the card was played in
User who played the card
Text of the card
Was this card the winner
User who was the Tsar
Time the round ended

which gives us a lot of stats about who played what, who chose what, what won, both all-time and per-game.
A bit lame storing the card text multiple times, but the card text isn't currently in the database anywhere so can't be refered to by id. Don't really want to make a table just for card text for logs…

Let's have two tables actually.

playlogs

id
roundlog
user
text
time

Primary key
Round this belongs to in roundlogs table (roundlogs.id)
User who played the card (users.id)
Text of the play
Time the play was made

roundlogs

id
game
winplay
tsaruser
starttime
endtime

Primary key
Game this round was for (games.id)
Which play from playlogs won this one (playlogs.id)
User who was the Tsar (users.id)
Time the round started
Time the round ended


reference: divide status-checking from other commands, add git graph

We should emphasize the commands which are always safe to use, and
can be used to figure out what's going on.

git graph is a useful command which is easy to miss, if you don't
see the one place we define it.  Make it and the definition easy to
find.

This PR depends on and includes my PR of reformatting the bold to
monospace.

Looks ok


Support boolean properties in LDAP filter creation. Fixes #224.

Codecov Report

Merging #229 into development will increase coverage by 0.05%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##           development     #229      +/-   ##
===============================================
+ Coverage        82.82%   82.87%   +0.05%
===============================================
  Files              186      186
  Lines            13190    13197       +7
===============================================
+ Hits             10924    10937      +13
+ Misses            2266     2260       -6

Impacted Files
Coverage Δ

framework/include/cppmicroservices/LDAPProp.h
100% <ø> (ø)
:arrow_up:

framework/test/driver/ServiceFactoryTest.cpp
97% <100%> (ø)
:arrow_up:

framework/src/util/LDAPProp.cpp
72.52% <100%> (+1.26%)
:arrow_up:

framework/test/driver/LDAPFilterTest.cpp
83% <100%> (+0.34%)
:arrow_up:

framework/include/cppmicroservices/detail/Log.h
100% <0%> (+8.33%)
:arrow_up:

...ramework/include/cppmicroservices/detail/Threads.h
97.77% <0%> (+8.88%)
:arrow_up:


Fix keyboard crashes in BrowserFragment

Task/Issue URL: https://app.asana.com/0/414730916066338/1208339084893367/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#5033  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mikescamell and the rest of your teammates on  Graphite


Show list of bookshelves that a book belongs to on a book view

Closes #1598
I put these underneath actions because there could potentially be many that push the actions off-screen. But you will likely want to change the placement of this if this is a feature you wish to merge 👍
Another part of this PR to look out for is the heading above the list. I went with Bookshelves because I think it should be clear enough from the context (a book) that it is referring to the bookshelves that the book can be found on.
Thanks

Apologies for the massive delay in review @cw1998 & thanks for offering this.
Code looked great. Also thanks for writing a test to cover it.
I performed an couple of changes upon your implementation:

Moved the list to the left sidebar - This is just to align with the other navigation/traversal elements shown in BookStack, which are also shown on the left.
Instead of showing some empty state text ("Shelves this is added to will show here...") I instead just hid the list. Via permissions, it is possible to currently completely hide shelves within BookStack, so this accommodates that scenario where someone may not use shelves otherwise they will have an always-empty list.

Again, Massive thanks. Will be part of the next release (v0.29).


Implement goto_location

It would be very useful to have goto_location command (as available in c++ API)

Is anything willing to contribute this action? Would greatly appreciate :)

@vivh3 @JonasVautherin @avillanueva I'll see what I can do, I'm trying to get more involved with implementing more features into MAVSDK-Python, but I'm a bit short on time as school work is high around this time of year, and I don't want to take on too much. I already have a couple other issues im working on for MAVSDK, I'll keep you posted!

It is now in release 0.7.0!


Make inspect try an image on failure if type not specified

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>When doing "buildah inspect {imageID}" one would receive an error saying the container was not found.  If you did "buildah inspect --type={imageID}" the output for the image would be found.  This change will have inspect check to see if the id that was passed in is an image if the lookup for the container fails AND the type parameter is not provided.
This aligns the functionality to the man page documentation and addresses issue #212

LGTM

@rh-atomic-bot r+

@TomSweeneyRedHat Actually golint is not happy with you PR

lint error, running it down.

@rh-atomic-bot r+


(polish) Filled out missing Steam Store and Community files, fixed workshop_description_polish.vdf and storepage_81731_polish.json.

Thanks! Can you reply with a link to your Steam profile so I can give you a Translator medal?

Here you go!
http://steamcommunity.com/profiles/76561198081779107


Add generic 'status' map to BaseService

Very useful if services can mark their current status in such a map.  LastRefreshed, entity counts, etc. are all useful things to record.
Can be easily displayed in Hoist Admin client.

Should also research monitoring / observability build into latest Spring w/Grails 5

Closing this!  We have it!  Score.


I would like to update my README.md profile

@ZachyDev submit ur PR :)

Hey, will you please assign this issue to me?

@ZachyDev please create another issue:)


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=576540545
https://github.com/Azure/azure-cli-extensions/commit/dcc792708535b6bd4827dfbca216c229e9b04a43

Docs Build status updates of commit 0321aa7:
:clock10: Full build: the average full build time is 6 min(s) 41 sec(s), this is based on the last 177 build(s) for this repository.

Docs Build status updates of commit 0321aa7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

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


DJV 2.0 Feature request: Add option to have audio when jogging/scrubbing

At present clips with audio play audio only when "default" (native frame) rate is used.
For finding audio match-points, i.e. when having separate audio/video tracks with a clapperboard, it is however essential to "jog" to the precise beginning of an audio event to sync it to your video track.
This also applies to scrubbing (fast forward/reverse or using the playhead).

This would be a nice option, though I'm not quite sure how to implement it yet...

Not sure either, but i.e. kdenlive features this,


Eslint

This resolves #121
DCO 1.1 Signed-off-by: Kenneth Skovhus<EMAIL_ADDRESS>

I'll leave this for you @Snugug with the 'it was like this when I got here' :smile:

I like the current way as it lets us tie it in to our normal testing suite, but I'm not opposed to this either.

@skovhus In the future, let's have a discussion in the issue queue before you open a PR

Per our Contributing Guidelines

@Snugug ok. :+1:

Thanks!


Race condition when creating GCP service accounts for Fulcio

@therealnb reported (in Sigstore Office Hours today) an issue where the Fulcio service account wasn't ready on the initial run. On a re-run it succeeded.
This points to a race condition or need for retries.
@therealnb, please chime in with any details you can provide 🙂

Error from server (Forbidden): error when creating "STDIN": pods "redis" is forbidden: error looking up service account rekor-system/default: serviceaccount "default" not found

When I looked - it was there. Retry worked.
I think I followed the instructions. This machine never had kind on it before. I did create one default cluster then deleted it before I ran the scaffold scripts. I don't think that would have mattered.
$ kind version
kind v0.18.0 go1.20.3 darwin/amd64


Set proper rails generator defaults

To avoid the requirement to provide additional flags while using the
builtin rails generators I have added some flags to the application
configuration. Now the generator always uses rspec, slim and
factory_girl by default, and test_unit as a fallback if any generator
doesn't support rspec.
Signed-off-by: Thomas Boerger<EMAIL_ADDRESS>

Thanks :clap:


podman: 1.6.3 -> 1.7.0

Motivation for this change
https://github.com/containers/libpod/releases/tag/v1.7.0
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Thanks!


Replace links to elasticsearch.org with elastic.co

This commit replaces the remaining links to elasticsearch.org with their elastic.co counterparts.
Closes #3915

ported to 7.4 and master


[Bug] Reciprocal op requires 2 inputs instead of 1

Seems like reciprocal is defined as eltwise binary, instead of eltwise unary on TTIR to TTNN level. Therefore, we're enforcing 2 input requirements instead of one.
Error sample:
E       RuntimeError: TT_FATAL @ /localdev/nvukobrat/tt-forge-fe/third_party/tt-mlir/runtime/lib/ttnn/program.cpp:494: program->inputs()->size() == inputs.size()
E       info:
E       Program expects 2 inputs, found 1 in input tensors vector

Can you show us what test you are trying to run?
Reciprocal op is defined as element-wise unary in both TTIR and TTNN dialects:

TTIR Reciprocal Op
TTNN Reciprocal Op

Can you show us what test you are trying to run?
Reciprocal op is defined as element-wise unary in both TTIR and TTNN dialects:

TTIR Reciprocal Op
TTNN Reciprocal Op

I'm having issues connecting to VPN to provide detailed results, but as recall the issue could be caused by this part:
third_party/tt-mlir/lib/Conversion/TTIRToTTNN/TTIRToTTNN.cpp, more specifically: populateTTIRToTTNNPatterns.

Also, test that I'm running is on FFE side, still waiting for approval in order to be merged. But for quick check, feel free to cherry pick these two commits:

git cherry-pick c86a1f64612bec0a7f095a08a8c657d508e70c13 (ref)
git cherry-pick a8bac9fad837d094a4324d0bd978cce57e1ecbcd (ref)

Can you provide the reportify graph for this test case?
I have a suspicion that this test translates to addition with a constant, and it seems that constants aren't yet supported in the forge-fe->tt-mlir lowering, as you've discovered. This is probably why you're getting the message because the model you've defined has 2 inputs, but we need to verify this on the Reportify graph.

As per our offline discussion, the Forge Compiler must support the passing of the constant inputs. I will close this for now. In order to fully work with constants, we need to support a constant op in TT-MLIR. The task already exists: https://github.com/tenstorrent/tt-mlir/issues/231