Place the connection of the socket in the BinaryClient constructor

After testing, when Socket to establish a connection, Socket state will no longer change, unless it calls the appropriate method, if the network causes the Socket connection is disconnected or the server is automatically disconnected, Socket state will not change, So the following code can not detect whether the connection is available
socket != null && socket.isBound() && !socket.isClosed() && socket.isConnected()         && !socket.isInputShutdown() && !socket.isOutputShutdown()
That is, every time you send a command to detect the availability of the connection is not necessary, so intend to remove the detection code, and connect the Socket code to the construction method.

@iaiengxia IIUC according to your statement if we change the isConnected  logic to socket != null && true; then it'll be the same right?. As the socket state doesn't change per your observations.

Very sorry, now see this reply, you are right. Also in the actual development, I encountered a Socket disconnected situation, I think it should be added with broken cable reconnection function, do you think it is necessary

@iaiengxia well, I've tried changing the isConnected function and then some tests failed. So seems like the logic there is actually doing something. Have you tried this?

@marcosnils Can you look at the reason for the failure of the test, is it because the modified isConnected function

@marcosnils On the Socket state, I was tested. So I do not think there is a need to verify the availability of the socket every time I send a message (unless the heartbeat is sent, otherwise it is not verified), so I moved the Socket connection to the BinaryClient constructor.

If my understanding is right, the purpose of isConnected() is performing lazy-connection. So isConnected() can't determine disconnection or so (unless doing heartbeat, as you said earlier), but I guess it can determine at least socket is initialized. If then the code is implemented properly.

Maybe we could take another approach like flagging if you observe that isConnected() is a bottleneck.


Help,how can I get listeners by event name in v3?

When I use EM V2, I have a code in my project for implement isLastListener.
How can I implement in V3?
Does there have a solution?
use Zend\EventManager\AbstractListenerAggregate;
use Zend\EventManager\Event;
use Zend\EventManager\EventManagerInterface;

class TestListener extends AbstractListenerAggregate
{
    /** @var  EventManagerInterface */
    protected $events;

    public function isLastListener()
    {
        $listeners = $this->events->getListeners('event.name');
        $listeners = iterator_to_array($listeners);
        if (empty($listeners)) {
            return false;
        }

        $last = end($listeners);
        $call = $last->getCallback();

        return is_array($call) && $call[0] == $this;
    }

    public function attach(EventManagerInterface $events, $priority = 1)
    {
        $this->events = $events;
        $this->listeners[] = $lis = $events->attach('event.name', [$this, 'test']);
    }

    public function test(Event $e)
    {
        if ($this->isLastListener()) {
            //If is last listener, to do something.
        }
    }
}

I saw v3 code. Can change private getListenersByEventName to public?

Can change private getListenersByEventName to public?

No; we explicitly removed this use case from the public API.
One reason is because there are huge differences from v2 in how listeners are aggregated internally; keeping the method would have broken existing code in subtle, hard-to-fix ways, and providing a migration path would have been impossible. (Related: we actually went through 3 different iterations of aggregation before pinning the stable v3 release; having to change test expectations for each iteration was going to be a nightmare, so we removed them before the second iteration.)
The primary reason, though, is because it was a very infrequently used feature with a high maintenance burden. The number of use cases for it are quite slim, and the primary use case — retrieving CallbackHandler instances to allow detaching listeners — went away when we stopped wrapping callables in CallbackHandler instances.
Looking at your above code, there's a much better approach possible that will work with both v2 and v3 already. Register your listener at a really, really low negative priority to ensure it executes last. A good bet is that nobody is listening at - PHP_INT_MAX.

:ok_hand:
Thanks for your reply.
The priority of the listener is configured by the user.
So I must be write a class extension EventManager for implement  V2 getListeners.


Limit num endpoints in config fuzz test

Commit Message: Limit the number of endpoints in config_fuzz_test.
Additional Description:
A config_fuzz_test having more than 10 endpoints configured takes a
significant amount of time to run, which leads to a timeout in when
more then 1 minute is needed. Limiting the number of endpoints to 10
still allows the fuzzer to generate a sufficiently complex config.
Risk Level: Low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a

Rebased (and forced pushed) to hopefully fix the coverage problem in common/http, which this PR did not touch.

Please avoid force-pushing as it makes it difficult to track github comments (see here).
You should be able to merge the latest main instead.


support for custom formatting

i have already written a suggestion (#13), but you rejected it, which is completely understandable.
is it still possible to add custom formatting using the predefined colours provided by minecraft?

so you only want to change the color of the prefix if I understand that right?


Upgrade to Laravel 5.3

Include upgrading steps in the docs for future upgrades.

Do you think laravel will push to another version and call it LTS?

Not soon! maybe 6.1!! but some people are excited about features in 5.2 and 5.3, so I might have to keep upgrading it to the latest release. I believe upgrading should be simple and easy.

What version do you recommend running with Hello-API?

I really have no preference! 5.1 is great because of the LTS it has, but if you need some 5.2/5.3 features you might consider upgrading.

#39 w'll upgrade to 5.3 once it's released.


impl(otel): no spans if no sleeps

Motivated by #12959
I still like invoking the sleeper even when the duration is 0.

This change is

I still like invoking the sleeper even when the duration is 0.

I agree.  Even if there is no sleeping, it may have other side effects (despite this "sleeper" deciding that it won't).


PG 12 + PLPYTHON 3 compatibility

Adds PG12 to the matrix.
Adds some missing qualifications.
Since _CDB_Aggregable_Attributes and CDB_ColumnNames use sql_identifier which changed from text to name, it needs to be casted in several places so that it doesn't get truncated.
QuotaTest no longer depends on random(). Pending hiding the kb numbers since they seem to change depending on the run (after regenerating the DB, appears in all PG versions).
OAuth tests are no longer dependent on the PG version.
Until https://redmine.postgresql.org/issues/4868 is clarified, this changes PG12 build to depend on plpython3 instead of plpython (which defaults to plpython2). Multiple functions had to be adapted to work both with python2 and python3.

Let's make sure it works in our servers (covered by CI, staging for extra safety). At present I prefer your approach: being explicit about plpython3u.

Ups, wrong link. The correct one is https://redmine.postgresql.org/issues/4868. Right now I'm not too worried about what the decision would be, but it should be revisited when preparing the migration to PG12

I'm merging this since it doesn't involve any API changes and it shouldn't mean any changes for PG10 or PG11, but it's a good base to iterate over.


[Godot 4 Tilemap ] Can't erase/delete wrong placed bitmasks from terrains in paint property

Godot version
v4.0 alpha
System information
Windows 8 and 11
Issue description
I always set my bitmasks for terrains in godot 4 in one go, but today I was trying to erase wrong placed bitmasks and found out that there is no way to do this. I tried right click or look for a tool icon for this with no success. Is there a way to erase them?

Steps to reproduce
Create a tileset, create a terrain, place some bitmasks and try to erase them with right click.
Minimal reproduction project
No response

For now, I found Terrain -> No Terrain to erase.


Fixed LaTeX typo and maintained data/content consistency

Fixed one of the LaTeX that was not rendering as desired.

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

Hi @AgentJ08,
thank you for your contribution! 🙏 Don't forget to register here


More speed

This change is

Current dependencies on/for this PR:

main

PR #3820

PR #3822

PR #3830

PR #3831   👈

PR #3821

This comment was auto-generated by Graphite.


Clearly split config-time and request-time code in the mixer framework

Today there are two distinct code paths in the mixer:

request time, where adapters are executed according to config
config time, where we ingest and validate the configuration the user provided

These two code paths are distinct and do not interact. (The sole touch point in the mixer between the code paths two is in the adapterManager/manager class, which has a reference that's atomically updated to the latest config at the end of the config time code path.) Further, these two paths have very different requirements. In the request path we try to avoid allocations and optimize for low latency. In the config path we don't really care about latency or allocations and instead optimize for understandability (for maintainers) and good error messaging for the operator.
The problem today is that these two code paths are intermixed; this will almost certainly lead to pain down the line where we (or a third party adapter author) introduce high latency, allocation-full code to the request path unintentionally because of the lack of delineation between the two.
Proposal
We should clearly split the two code paths:

in //pkg/aspect, every manager implements the config.AspectValidator interface; this implementation has aspect specific logic but is completely unrelated to any other code in each aspect manager. These implementations should be moved into //pkg/config.

as a side effect of this the aspect parameters, defined in //pkg/aspect/config/<aspect name>.proto today, should be migrated to //pkg/config/proto/<aspect name>.proto
descriptor validation is not yet implemented; we're planning on implementing it as part of aspect validation. Under this proposal, it would also be implemented in //pkg/config too, likely in //pkg/config/descriptors, separating the concerns of descriptor validation from aspect param validation.

//pkg/adapter defines the adapter.ConfigValidator interface then embeds it in the adapter.Builder interface. The adapter.ConfigValidator interface should be split (so that adapter.Builder can still provide a default config) and removed from the adapter.Builder interface. Instead, we should require that adapter authors give us three things: a builder, an aspect implementation, and a config validator. (Today they give us a build and an aspect implementation.)

(I'm very convinced 1. needs to happen, I'm still thinking through 2. and how warranted it is.)
After these changes the two code paths will be very clearly split: mixer framework validation all lives in //pkg/config and adapter validation lives in a separate struct than the runtime adapter logic. Anything outside of //pkg/config must be written to the standards of request path code, and anything in //pkg/config can be implemented with more lax standards.

@geeknoid @guptasu @mandarjog @douglas-reid Thoughts on this? (I guess an @ mention is a better way to notify than assigning the issue.)

I like the overall approach. I've been worried about these blurred lines for a while, but never had a clear plan on how to address it.
This intersects with two other ideas I've been discussing:

I think the configs from pkg/aspect/config should move to the istio/api repo and integrated directly in the AspectConfig struct.

I want to get rid of the adapter.kind field from config. This requires introducing a new object in adapter code that represents the adapter as a whole, distinct from builders. We should factor this in as we consider rejiggering things.

Does aspect config structure belong in an istio/api repo? It seems like a detail that only mixer ever cares about (and thus is something that can be self-contained -- reducing friction related to adding new ones).

I think the shape of aspect config should be considered part of our API,
since callers can program against the APIs and use those protos.
On Mon, Apr 17, 2017 at 11:14 AM, Douglas Reid<EMAIL_ADDRESS>wrote:

Does aspect config structure belong in an istio/api repo? It seems like a
detail that only mixer ever cares about (and thus is something that can be
self-contained -- reducing friction related to adding new ones).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/mixer/issues/550#issuecomment-294548339, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHbKZqcI74znlaPM7uyTAr9y5pzpjks5rw6wHgaJpZM4M-GDs
.

We have not done what this issue proposes: there is still code that deals with config out of the request path that lives side-by-side with code in the request path that doesn't deal with config. All of the config validation in pkg/aspect, and in adapters themselves. All of this code should move around.
We probably need to update how exactly we want to structure things in light of the new adapter model, but the core issue persists and has not been resolved.

Addressed by new adapter model.


Fix bugs in tiling unpack ops for perfect tiling cases.

The perfect tiling case indicates that the tiling sizes is are multiple of inner_tile_size. In this context, The indices of input slice are all aligned to head. No extra data is needed when representing the tiled unpack op.

@chelini FYI

isPerfectTilingCase and isAlignedToInnerTileSize shadow each other. I think we can drop one for the other (potentially the latter for the former)

isAlignedToInnerTileSize is for checking if the dim-th tiling size is aligned to corresponding inner tile size. isPerfectTilingCase requires all isAlignedToInnerTileSize to be true.
Furthermore, there are cases that some isAlignedToInnerTileSize are true, but isPerfectTilingCase is false. For those isAlignedToInnerTileSize, we still can simplify the logic of inputSizes.


Ensure that the transpiler is installed separately...

I was reading the documentation for the Nth time, and noticed a line I never saw before:

Ensure that the transpiler is installed separately (npm install babel-core here).

Thing is, I never did this step and I never noticed because everything works. What am I missing here? One of my guesses is that, without installing the transpiler, my bundles are still ES6, and they have to do in-browser transpilation.
However, if that was true, and by installing the transpiler I'll now have ES5 bundles, then shouldn't I modify config.js to state that no transpiler is going to be used?

This is for a SystemJS-Builder specific Node workflow and doesn't apply to jspm.


SHASUMs via HTTPS?

Hi!
Shouldn't we be pulling the SHASUMs via HTTPS? Everything else feels like a substantial security issue to me which would make the checksum tests basically pointless.
Hope someone can proof me wrong :-)
Cheers

We should probably switch to https for everything. It's what they do for the docker-iojs image. I wouldn't say the current situation is pointless, but it's not great either.

This should be landing in the registry soon. See docker-library/official-images#847

Many thanks for the super fast change!
On Wed, Jun 24, 2015 at 3:47 PM -0700, "Christopher Horrell"<EMAIL_ADDRESS>wrote:
This should be landing in the registry soon. See docker-library/official-images#847
—
Reply to this email directly or view it on GitHub.


Appstudio update test-component-pac-ilfz

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-ilfz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
11 seconds
clone-repository

✅ Succeeded
9 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-ilfz-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
appstudio-init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
11 seconds
appstudio-configure-build

✅ Succeeded
30 seconds
build-container

✅ Succeeded
6 seconds
show-summary


chore(Refactor): Reorg of code base for modern 1 package structure an…

…d update deps
Resolves #576
Helps #565
QA:
Easiest way to test this, do normal regression tests. This should change NO functionality.

I'll run some regression testing on the browser to test this @joshuef @hunterlester. CC @safesurfer

Found https://github.com/maidsafe/safe_browser/issues/587 while doing some regression testing. We will be doing more extensive regression testing closer to the release date so I didn't spend too long on it today. No other issues encountered.


Use of variable for the current version number

Is your feature request related to a OpenMS API Reference Documentation? Please describe.
Would be nice if we used fixed version numbers in the docs, such that older docs still point to older releases in the install instructions.
This is only maintainable with variables. I bet this is easily possible with sphinx.

[ ] I have attached screenshot describing the problem, if applicable.

Describe the documentation enhancement you'd like

Describe alternatives you've considered

Additional context

https://sphinxcontrib-versioning.readthedocs.io/en/latest/tutorial.html

Maybe that is even overkill since we use readthedocs.
https://docs.readthedocs.io/en/stable/environment-variables.html
And then https://stackoverflow.com/questions/34006784/how-do-i-access-a-variable-in-sphinx-conf-py-from-my-rst-file

What is the use case of having {version} variable? What enhancement(s) does it bring in maintenance?


Demonstrate that America/Los_Angeles is not a valid timezone in tests

Hey @geerlingguy, this PR just adds a test to demonstrate that the timezone is not properly set on tests. Not sure if the error comes from the docker container we use to setup the test environment, as I consider this America/Los_Angeles timezone as a valid timezone value.
Thanks!

I can confirm, that executing it in a playbook against a Ubuntu 17.04 VM, it works as expected.
- name: "Set timezone to PST"
  become: yes
  timezone:
    name: "America/Los_Angeles"

@mdelapenya do you have an open issue for this too? I'm just trying to get more context around this.

Hi @geerlingguy, no I did not create it. This PR is a demonstration that current container used for tests does not accept changing the timezone.
I discovered that setting up the timezone in my ansible role for jenkins (using yours as base) fails because of this tiny problem.
Thanks!

Hey @geerlingguy, I created #178 as you suggested. Could you please take a look?
Thanks

This PR tries to set "America/Los_Angeles" as the timezone for the container that executes the tests. As you can check on Travis execution (https://travis-ci.org/geerlingguy/ansible-role-jenkins/jobs/278250070):

The timezone is not available.
Could you please check this?
Thanks


(Suggestion) 'Getting Started' Video Tutorial

First of all I am very impressed with this new tool, thank you for sharing your work @MarceloJacinto.
Even tho the Documentation is excellent, I believe that the community would benefit from a "getting started" video tutorial, where installation as well as the basic functionalities are explored and explained. This would serve as a guide for new users, and make it more accessible for the general public.
Thanks again and keep up the good work!!

I think the "getting started" video is a great idea. I will be working on it next!
Thanks for the suggestion

The getting started video was added to the installation tab in the documentation!


It is hard to detect QR code with some Desktop webcams.

0e7ca0f

Even using the largest resolution, It's impossible to detect on desktop webcams.
In addition, such devices allow only up to 1280x720 also for photo capturing; currently no idea to fix it.


Breaking change ? Unable to use String validate options on array of strings

An error occured after updating typegoose to last version : Type of phoneNumbers property is not a string.
Here is the errored property :
  @property({
    lowercase: true,
    required: [true, 'phone_numbers.error.required'],
  })
  public phoneNumbers: string[];

I have tried to use arrayProp :
  @arrayProp({
    items: String,
    lowercase: true,
    required: [true, 'api.model.agency.contact.phone_numbers.error.required'],
  })
  public phoneNumbers: string[];

But option lowercase is not defined in arrayProp options type.
I think that https://github.com/szokodiakos/typegoose/blob/master/src/prop.ts#L138 should accept array of string too

i know this is old, but i am working on it currently at https://github.com/Automattic/mongoose/issues/8012

Update from the mongoose issue:

it got merged into 5.7
5.7 got released
but there is are still some issues with it https://github.com/Automattic/mongoose/issues/8012#issuecomment-529763148

@Ben305 this can be closed in favor of https://github.com/typegoose/typegoose/pull/29


Ability to download private repos from github

When gvt fetching a private repo, I get the usual prompt:
Username for 'https://github.com':
Password for ...:
Is there a way to use ssh keys to fetch the repos?

I updated my git config as this, and now it is working for private repos:
git config --global url."git@github.com:".insteadOf https://github.com/

@canthefason worked for me. Thanks!


ImmutableDruidDataSource in more places

I'll clean this up after https://github.com/druid-io/druid/pull/5054 goes in.
This PR:

Uses ImmutableDruidDataSource in many more places
Removes the usage of locks in DruidServer
Stores ONLY the segment id to object mapping in DruidServer. Now many things take actual immutable snapshots of DruidServer objects in order to do various data processing on their state from the immutable snapshot. Most of these usages bubble up to http calls, which should ideally be faster. This may cause some extra extremely short lived objects for excessive http calls, but hopefully either has equal or lesser pressure on old gen by reducing the overall needed long-term object storage.
Moves lots of the looping to a more functional approach with java 8 sugar

Tests run: 5, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 67.953 sec <<< FAILURE! - in io.druid.curator.announcement.AnnouncerTest
testSanity(io.druid.curator.announcement.AnnouncerTest)  Time elapsed: 61.51 sec  <<< ERROR!
java.lang.Exception: test timed out after 60000 milliseconds
	at sun.misc.Unsafe.park(Native Method)
	at java.util.concurrent.locks.LockSupport.parkNanos(LockSupport.java:215)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedNanos(AbstractQueuedSynchronizer.java:1037)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.tryAcquireSharedNanos(AbstractQueuedSynchronizer.java:1328)
	at java.util.concurrent.CountDownLatch.await(CountDownLatch.java:277)
	at org.apache.curator.test.Timing.awaitLatch(Timing.java:120)
	at io.druid.curator.announcement.AnnouncerTest.testSanity(AnnouncerTest.java:102)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:47)
	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:44)
	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
	at org.junit.internal.runners.statements.FailOnTimeout$StatementThread.run(FailOnTimeout.java:74)

announcer test

@drcrallen would you fix the conflicts? I'll review soon.

I got caught up in other items. This has diverged a lot from master and I'm not sure it's worth catching up
Closing for now


Add installation instructions for openSUSE

magic-wormhole has been included in openSUSE since Leap 15.1, and is
also available in Tumbleweed:
https://software.opensuse.org/package/python-magic-wormhole
So document this explicitly.

thanks!


Does this package work with Laravel 5?

Hello,
I don't test with Laravel.
But it is a library that does not use any specific dependency.
You simply create objects and fill as desired.
Personally I use it with Symfony and in à simple php application for my tests.
I extract my medias in Doctrine and create all Media Object.
In futur version, a interface for media will be created.
This method for easy coupling with the entities.

Have you à feedback of your experience with Laravel ?


Page refers to options that are not provided by the product

There is no "Color saturation" well available, either in the Power BI web service or the current desktop client (Version: 2.69.5467.2151 64-bit (May, 2019))

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

ID: a47d7082-17fb-3b37-de23-020f49991e0c
Version Independent ID: 4d904a62-6e2e-b05f-b222-2500df079b76
Content: Filled Maps (Choropleths) in Power BI - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-filled-maps-choropleths.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @mihart
Microsoft Alias: mihart

Hi, @eldang -- thanks for your question! I've assigned it to the author to investigate.

Hi @eldang,
Thank you for your feedback.  Yes, the color saturation is now part of the Formatting pane.  I'm updating the article now so it should go live no later than next week.

@mihart but there's no color saturation option under formatting either!  As far as I can tell, there's currently no way to do a choropleth map with a continuous colour scale (as opposed to categories), which seems like kind of a basic need for an analytics package, and it's frustrating when the docs show examples.  Or have I missed something that is available somewhere?

Hi @eldang
Under the Format tab, expand "Data colors".  Then select the 3 dots and choose "Conditional formatting".  In the pane that appears, add the field you'd like to use for saturation.
I'm adding this to the doc today because it is rather hard to find.


cbt read failed immediately

This is the error information I got when try to use cbt read. I think it's related to the duploicate proto type. I have updated gcloud commands but the error is still there.
2017/07/18 10:55:17 proto: duplicate proto type registered: google.rpc.Status
2017/07/18 10:55:17 -creds flag unset, will use gcloud credential
2017/07/18 10:55:17 Retryable error: rpc error: code = Unavailable desc = grpc: the connection is closing, retrying in 47.77941ms

When you see this I think the operation is actually retried and will succeed. Otherwise you'd see different / additional logging. Can you confirm that's the case?

Please re-open if the read doesn't work after the transient error


Apicula now supports lutram

See https://github.com/YosysHQ/apicula/pull/111 and https://github.com/YosysHQ/nextpnr/pull/995


⚠ Bofrost.es has degraded performance

In 020027c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1659 ms

Resolved: Bofrost.es performance has improved in 33067f1 after 6 minutes.


release v0.11.1

What this PR does / why we need it:
Special notes for your reviewer:
Does this PR introduce a user-facing change?

LGTM


Remove ATS allows arbitrary loads.

ArcGIS Online has updated it's owning system URL. https://www.arcgis.com/sharing/rest/info?f=pjson
This change should be reflected in

[ ] XCode
[ ] README

Open PR for Xcode project: https://github.com/Esri/data-collection-ios/pull/121

Removes ATS settings from XCode project, changes merged into master.

Docs are updated. Will confirm after next dev site deploy.

These changes has been deployed. Closing.


Fix flickering of embedded note title when using linked mentions

Originally posted by @MichaelLepori in https://github.com/czottmann/obsidian-mononote/issues/19#issuecomment-2150484286

The same!
There is this issue, when your Plugin is active and I click on a linked mentions entry:
https://www.screenpresso.com/=W7P4b
Thank you.
Have a great day.
Mic.

Fixed in Release 1.2.1. Re-open the ticket if it isn't, please!


Move remaining value types to new types directory

Moves URI, CAL-ADDRESS, and RECUR
Issue #72

Codecov Report

Merging #99 (1f22e7e) into main (118f603) will decrease coverage by 0.39%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
- Coverage   96.83%   96.43%   -0.40%
==========================================
  Files          36       38       +2
  Lines        2115     2020      -95
==========================================
- Hits         2048     1948     -100
- Misses         67       72       +5

Impacted Files
Coverage Δ

ical/_types.py
90.45% <100.00%> (-4.31%)
:arrow_down:

ical/alarm.py
97.61% <100.00%> (-0.39%)
:arrow_down:

ical/event.py
96.24% <100.00%> (-0.03%)
:arrow_down:

ical/freebusy.py
98.33% <100.00%> (-0.28%)
:arrow_down:

ical/journal.py
100.00% <100.00%> (ø)

ical/timezone.py
96.09% <100.00%> (-0.39%)
:arrow_down:

ical/todo.py
98.41% <100.00%> (-0.03%)
:arrow_down:

ical/types/__init__.py
100.00% <100.00%> (ø)

ical/types/cal_address.py
100.00% <100.00%> (ø)

ical/types/parsing.py
92.00% <100.00%> (+4.50%)
:arrow_up:

... and 15 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove usage of makeSubStream from Type1Parser.extractFontProgram in src/core/type1_parser.js (issue 9735)

This avoids the initialization of, potentially thousands of, unnecessary Stream objects, by getting the required number of bytes directly instead.
Given the special behaviour, when length === 0, of the getBytes/skip methods, it's also necessary to handle that particular case to prevent errors when encountering empty CharStrings.

/botio test

Thank you! I took care of running all tests locally and everything passed.


TgBot instead of Bot, double cp fixed, authors added

closes #31

PR-Codex overview
This PR refactors the Telegram bot implementation by renaming and reorganizing classes and packages, adding author and version information, and replacing the use of generators with conversations.
Detailed summary

Renamed Bot to TgBot and moved it to io.blamer.bot.agents.tg package
Renamed BotInitializer to TgInitializer
Renamed BotConfiguration to ExtTg
Renamed BotTest to TgBotTest
Added author and version information to various files
Replaced the use of generators with conversations in TgBot
Moved package-info.java files to their parent packages
Deleted package-info.java file in io.blamer.bot.bot package
Updated TgBotTest to test the new changes

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@l3r8yJ maybe we should remove Bot interface?

or at least rename its method to init()

@rultor merge

@h1alexbel @l3r8yj Can't merge it. Some CI checks were failed. Apparently, the pull request is not ready to be merged since it has some problems. Please, fix them first.

@l3r8yJ WDYT?

@h1alexbel we can use @PostConstruct on interface method and say call it loadCommands/init/initCommands. I suggest to rename interface like to something likeBotInitialCommands

@l3r8yJ I don't think that hiding PostConstruct will help to improve maintainability

@l3r8yJ moreover we don't need to expose this method to others

@l3r8yJ yes, it's exactly what I'm suggesting

@h1alexbel can i try to merge?

@l3r8yJ I will up the coverage

@l3r8yJ ready for review

@rultor merge

@rultor merge

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

@rultor merge

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

@l3r8yJ chat not found in test:
https://stackoverflow.com/a/41291666/19147117
maybe we need to create fake client and call it on ITCase

I think we can find a ready-made solution to test the telegram api. The fake telegram API looks like a separate open source product, it is too big.
we can create a new idea issue, wdyt?

@h1alexbel thanks!

@l3r8yJ did you mean creating issue into blamer-io/ideas or what?

@h1alexbel yes

@l3r8yJ created. here it is


Decide how to supply simulation parameters

Right now the parameters are supplied in each Excel file. I'm trying to move everything towards plain text as much as possible, so I'd like a different way to supply the parameters. The parameters that need to be supplied are as follows:

number of pigments in the simulation
exciton transition bandwidth
wavenumber start, stop, and step

One option is to supply all parameters in a TOML file, but that seems like a pain for just 5 parameters. Another option is command line arguments to the simulation script.

Another element to this is having a record of the parameters used for a particular simulation. When you put the simulations in a file, you can store the inputs, outputs, and parameter file in a directory to save a reproducible simulation, and it's unambiguous which parameters were used to produce those outputs.
When you use command line arguments, there's no record of which parameters were used.


Sometimes, if ad is skipped, onAdComplete failed

chunk-vendors.js:formatted:16899 Uncaught TypeError: Cannot read property 'isLinear' of null
at T.onAdComplete (chunk-vendors.js:formatted:16899)
at rh (ima3.js?_=1576147226099:101)
currentAd is not an object
https://github.com/googleads/videojs-ima/blob/77da05d28922bb85e74e1cf4e13eea627c940bc7/src/sdk-impl.js#L477

Hi,
I was not able to reproduce the behavior that you're seeing with the onAdComplete event failing if the ad is skipped. Can you share the ad tag you were using when the issue occurred?

The is my vast source :
https://media-cloud-cdn.wedia-group.com/api/v1/media/roll/vastInformation.xml?DepartmentUid=1aac42b7-a847-4245-8bb3-4cc8b0752706&MediaGuid=5d9aed87-973c-4633-a078-fbe41c8b8b89

your ad linear duration is 00:00:00, i have an experience that bad duration value sometime makes strange error...

your ad linear duration should be 00:00:05 from MediaFile mp4 duration

Gonna test it, thanks

@ChoOo7 ok

Hi,
I’m closing this issue because it has been inactive for more than a month. Please reopen if you still encounter this issue with the latest stable version
Thank you!


Develop with ng serve and see everything works as expected.
Deploy with ng build --prod and see chart.piecelabel.js plugin not working anymore.

Observed behavior
As this is my first Angular project, my apology if it doesn't qualify an issue report.
My investigation is that the behavior difference (working in dev but not in prod) is caused by a different default compilation strategy: --aot is default to be on when building for production (ng build --prod). When I tried developing with AOT on (ng serve --aot), the same problem is observed.
Note that there is no compilation error. For the prod (AOT) build, chart.piecelabel.js complains not being able to locate the Chart object to inject new behaviors in the console log. But the chart is working properly, just that no label overlay is shown. I think the best guess is that both chart.piecelabel.js and chartjs are loaded, but the loading order is incorrect - the AOT compiler did some tricks causing chart.piecelabel.js being loaded before the Chartjs plugin, and thus the problem.
So it sounds like an import order issue. For now I am inclined to think this is not an expected behavior, so I am reporting here. Again, sorry if this is a false alarm. Any advise or guidance would be greatly appreciated. Thank you!

The actual cause of the problem is not that the imports in the file are being reordered but rather that the ChartsModule import no longer exists.  In AOT mode, the NgModule within app.module.ts that uses ChartsModule is compiled via the AOT compiler resulting in the import no longer being used within the file.  Unfortunately for this situation, this is working as intended.  Theoretically, the ng2-charts library could provide a mechanism to register chartjs plugins as it has assumed ownership of the lifecycle of the underlying chartjs library.

For what it's worth, I was able to work around the issue by manually importing the chart.js library and console.log it before importing the chart.piecelabel.js plugin, as suggested by @filipesilva in https://github.com/angular/angular-cli/issues/9058#issuecomment-355021360.
import {ChartsModule} from 'ng2-charts';
import * as Chart from 'chart.js'; // NEW
console.log(Chart); // NEW
import 'chart.piecelabel.js';

@tamcy you may be able to simplify that to:
import {ChartsModule} from 'ng2-charts';
import 'chart.js'; // NEW
import 'chart.piecelabel.js';

Closing as this is working as intended when using AOT mode.  The ng2-charts library is also marked as not having side effects.  If relying on side effects of the underlying chart.js package than importing it as indicated above would be the correct course of action.


Fix resource name bug and add logging

Resource names should not contain spaces 🌌 (Fixes #45)
Add logging 📃 (Fixes #47)

Coverage remained the same at 98.889% when pulling 686e9656ba0becf87d96d322b8353694e97f59dd on bugfix-and-logging into 193ba536ce8cd03cc49ab5e412c24e353e3755c7 on master.


Update fnv1a_xor_hashes.txt + fsorspclient<PHONE_NUMBER>7281069512

This is a great contribution. Thanks!

Hope you don't mind @roycewilliams - I added a link to your spreadsheet int the fnv text file. Great breakdown of the software each string corresponds.

Mind? Quite pleased, @jhsmith! If it helps anyone with their analysis, that's exactly what we wanted - and now people might actually be able to find it. ;)


[type:refactor] optimize response cors headers.

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3291 (810309c) into master (8cf209f) will decrease coverage by 0.13%.
The diff coverage is 61.90%.

@@             Coverage Diff              @@
##             master    #3291      +/-   ##
============================================
- Coverage     59.09%   58.95%   -0.14%
+ Complexity     4682     4675       -7
============================================
  Files           739      739
  Lines         20919    20938      +19
  Branches       1918     1921       +3
============================================
- Hits          12363    12345      -18
- Misses         7462     7489      +27
- Partials       1094     1104      +10

Impacted Files
Coverage Δ

...ugin/response/strategy/WebClientMessageWriter.java
72.13% <61.90%> (-6.45%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
87.50% <0.00%> (-12.50%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
54.25% <0.00%> (-9.58%)
:arrow_down:

...yu/plugin/logging/utils/LogCollectConfigUtils.java
66.66% <0.00%> (-8.89%)
:arrow_down:

...gin/logging/rocketmq/RocketMQLogCollectClient.java
68.00% <0.00%> (-8.00%)
:arrow_down:

...rg/apache/shenyu/plugin/rewrite/RewritePlugin.java
50.00% <0.00%> (-5.56%)
:arrow_down:

...in/response/strategy/NettyClientMessageWriter.java
69.56% <0.00%> (-4.35%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
8.00% <0.00%> (-4.00%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
74.79% <0.00%> (-2.44%)
:arrow_down:

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 8cf209f...810309c. Read the comment docs.


Error setting property 'color' of FAKIconImage

hi, i'm using react native 0.17 and react-native-icons 0.7.0. All is ok in android, but when running in the ios simulator, there is the problem:
2016-01-05 15:16:46.375 [warn][tid:main][RCTConvert.m:55] Error setting property 'color' of FAKIconImage with tag #29: JSON value '4282664004' of type NSNumber cannot be converted to NSString

seems like the color value is wrong, but i checked again and again, and sure they were set right.
anybody could help? thx

i tried to remove the color style and just keep color property, and it's ok now


[Feature] Allow adding dependencies and catalog in sql-gateway

Purpose of this pull request
This pr add a feature that allow user add dependencies to sql gateway and create catalog.
What's more
thiis is a backend feature, should be implemented in frontend also

+1


Document how to extract message body using firebase_messaging

There's some confusion w.r.t. whether onLaunch, onResume and onMessage are getting called on iOS or not. Many people are still complaining that one or more of these are not getting called (for e.g. see #18425).
How to reproduce?
Add firebase_messaging to your flutter project, configure for both iOS and Android as explained in the plugin's README. Then send a notification each to your iOS and Android apps using Firebase Console> Cloud Messaging by setting only the "Notification Text" in Step 1 (this is the only mandatory field in this step). Remember to set click_action to FLUTTER_NOTIFICATION_CLICK under Additional Options (optional, Step 5). In your flutter project, try to extract what you put in as "Notification Text" above using the following dart code to configure FirebaseMessaging:
onMessage: (Map<String, dynamic> msg) {
  _message = msg['notification']['body'];
  print(_message);
},
onResume: (Map<String, dynamic> msg) {
  _message = msg['notification']['body'];
  print(_message);
},
onLaunch: (Map<String, dynamic> msg) {
  _message = msg['notification']['body'];
  print(_message);
},

Result
All three messages are printed for Android. To be more clear, onMessage gets executed when the app is in the foreground when it gets a notification, onResume when the app is running in the background and you tap on the notification, onLaunch when the app isn't running and you tap on the notification. But on iOS, none of the print statements are executed. The notifications do arrive when the app is not in the foreground, but nothing happens when you click on them. No print outputs in XCode or VSCode. This leads to the idea that none of the three functions are getting called on iOS.
How I solved it
On looking more closely, I found out that all three functions are actually getting called. Only, execution was apparently getting stopped at _message = msg['notification']['body'];, so the print never got called on iOS. Flutter doesn't log any complaints either.
The issue is, the path to the "Notification Text" is different on iOS and Android, so I ended up using this code:
onMessage: (Map<String, dynamic> msg) {
  if (Theme.of(context).platform == TargetPlatform.iOS)
      _message = msg['aps']['alert'];
  else
      _message = msg['notification']['body'];
  print(_message);
},
onResume: (Map<String, dynamic> msg) {
  if (Theme.of(context).platform == TargetPlatform.iOS)
      _message = msg['aps']['alert'];
  else
      _message = msg['notification']['body'];
  print(_message);
},
onLaunch: (Map<String, dynamic> msg) {
  if (Theme.of(context).platform == TargetPlatform.iOS)
      _message = msg['aps']['alert'];
  else
      _message = msg['notification']['body'];
  print(_message);
},

Now everything works on both iOS and Android.
How this problem can be prevented
Going further, I request the Flutter/Firebase team to document how msg looks on iOS and Android. The firebase_messaging plugin's README is a good place. Also, why isn't Flutter throwing any error when it can't access msg['notification']['body'] on iOS?
Note
#23148 is somewhat different... that one is about keeping the structure of msg the same across onLaunch, onResume and onMessage. I actually find them to be the same! The issue I'm raising here is that it's not clear what the structures on Android and iOS are. It's desirable for the structures to be the same, but it's not as important as knowing what they are

Digging further...
When I send a message from the Firebase console, with only the mandatory "Notification text", that text appears as {aps: {alert: <notification text>}} in the notification payload on iOS. This needs to be accessed as msg['aps'][alert'].
If I add the optional "Notification title" in the Firebase console, it becomes {aps: {alert: {title: <notification title>, body: <notification text>}}. In this case, the notification text needs to be accessed as msg['aps']['alert']['body'] and the notification title as msg['aps']['alert']['title'].
On Android, it's consistently msg['notification']['body'] irrespective of whether I add a title or not. If I do, the title is accessible as  msg['notification']['title'].
This means we need different code not only for iOS vs Android, but also title vs no title!
I did fire a cloud function to implement all this. In the cloud function, the above behavior is reproducible by using {notification: {title: <Notification title>, 'body': <Notification text>} as the payload argument of  admin.messaging().sendToDevice().

I have tried all of these steps and still can't get any of the hooks to fire.
ATM I currently will only see the push notification if the app is in the background.
When the app is in the background and a push notification comes through, I can click the notification and it will open the app but I see
2019-02-21 08:13:35.652779+1100 Runner[312:8514] Warning: UNUserNotificationCenter delegate received call to -userNotificationCenter:didReceiveNotificationResponse:withCompletionHandler: but the completion handler was never called.
which leads me to believe my iOS setup is incorrect somewhere.

What version of firebase messaging are you using @kbrmimbyl

I have finally got the hooks to fire. I had to remove flutter_local_notfications from my project completely. I found this information here https://stackoverflow.com/questions/50970343/flutter-fcm-on-ios-not-calling-onmessage-and-onresume/51032413

Sorry for the delay, @nayanbhana. Glad you got it working. I had trouble even installing any local notification plugin, so I didn't use any. My firebase_messaging plugin is ^3.0.1

Note that #23148 is now fixed, as a breaking change in 4.0.0. It seems like this issue is mostly about documentation of the functionality of the plugin, especially on iOS.

@collinjackson "Now fixed"? I see 3.0.1 as the latest version on pub...

I also have found that the notification model is different on iOS and Android when the optional data attribute is utilized. See #29027

Anyone done a PR for the FCM plugin docs?

Has this been fixed by flutter/plugins#1728 or is there still something missing in the docs?

Has this been fixed by flutter/plugins#1728 or is there still something missing in the docs?
Thanks for the link - it points right to the part of README where this issue is explained. Technically speaking, this PR "fixes" the issue, but practically speaking – not really, as most users will probably copy-paste code from the _firebaseMessaging.configure(..) code block and assume that it's a good enough to start own data extraction from the message. It's unlikely to someone to assume that data structure will be different for iOS and Android, so 'Notification messages with additional data` part is skipped on the first read. Plus, people don't actually RTFM, as we know. :)
I, too spent many hours trying to understand what's wrong with my push notification handling logic, and found answer only in this thread, in the link to the PR, so maybe README can be improved.

@kbrmimbyl
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/427. Any further collaboration will be done there.


Add support for gzip compression

At present, there is no way to change the binary message before it is sent on the wire (in an extensible way), this makes it impossible to have plug in compressors, like gzip.
https://github.com/grpc/grpc/blob/master/doc/compression.md

For gRPC-web and Twirp, this is fine. The browser will negotiate the content encoding with the backend.
Is this about gRPC via @grpc/grpc-js?

That would only work with the server->client messaged, client server wouldn't be compressed.

That's true, but neither Twirp nor gRPC-web specify a mechanism for request compression. I'm also not aware of a browser API that gives us compression.
If you have a good idea to handle this, I'd say it should be added to the twirp / grpc-web transports.

Grpc supports per message compression, which would actually solve the
problem on the protocol side rather than the transport side.
On Tue, 14 Sep 2021, 23:11 Timo Stamm, @.***> wrote:

That's true, but neither Twirp
https://github.com/twitchtv/twirp/blob/main/PROTOCOL.md nor gRPC-web
https://github.com/grpc/grpc-web/blob/master/doc/browser-features.md#compression
specify a mechanism for request compression. I'm also not aware of a
browser API that gives us compression.
If you have a good idea to handle this, I'd say it should be added to the
twirp / grpc-web transports.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/timostamm/protobuf-ts/issues/153#issuecomment-919245187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA46MLMP6C3H4DOTRMMSP6DUB5QZLANCNFSM5D5MQ7WA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yes. Everything gRPC is run by @grpc/grpc-js, we're just generating simple clients that delegate to the runtime library. So this would be a job for @grpc/grpc-js. It's a nice library, written in TypeScript.
Closing this. Please reopen if you think something should be done on Twirp or gRPC-web.


Attempted to reset admin password, db migration multiple errors, restart.sh fails to start multiple components

Running the docker implementation of stroom, 173b
Followed this guide (sort of)
https://gchq.github.io/stroom-docs/latest/docs/user-guide/tools/command-line/
I couldn't find any "stroom-app.jar", looking in either the docker containers or other stroom directories, did find "stroom-app-all.jar" and used this instead.
We can't use email to reset password because there is no email. Current password is unknown, there are no other admin users besides 'admin'.
Also unable to copy in the error logs as we can't get anything off that system due to security policy.
After running the reset password command, the password did not reset. The command line process showed numerous java errors, most relating to database migration (why does database migration happen during password reset?). Guessing this is what broke everything.
After this issue attempting to run stop.sh, start.sh; or restart.sh, results in multiple components not starting. (nginx, proxy-local, proxy-remote, stroom-log-sender) showing status : stopped.
I then attempted to run the password reset command again, and it worked successfully with no errors.
However after this success, running stop.sh/start.sh or restart.sh still results in nginx, proxy-local, proxy-remote & stroom-log-sender still show status STOPPED.
Is there any hope of fixing or has the failed database migration broken things beyond repair?
I do have a full backup I could restore from - perhaps if anyone could provide guidance on what parts might have broken during the migration failure, I could do just a partial restore. (I'm not sure if the full backup restore would actually work, as we've only just setup the backups (luckily) and are yet to test a restore.
Also what would the correct command line password reset process be? I know I did it once a long time ago, but can't recall if I found stroom-app.jar, or did it from somewhere else like in the database manually (if that's where the password is even stored?)

The reset procedure assumes that you have a working and running Stroom.  If you are using a stack, you should exec into the running stroom container to run the reset_password command from within stroom directory.  stroom-app-all.jar is the correct jar , you must also specify the config.yml (found in the config folder).
Assuming you did this correctly I can't see why Stroom would be trying to perform migrations. If a later version of stroom-app-all.jar is used, then it will attempt to run any outstanding upgrades as per https://gchq.github.io/stroom-docs/latest/docs/user-guide/tools/command-line/#authentication-accounts-and-stroom-users
...
This command will also run any necessary database migrations to ensure it is working with the correct version of the database schema.
...
Not sure why nginx and stroom-log-sender are STOPPED tbh, they don't have a dependency on stroom.  Do the docker logs provide any clues as to why they are stopped.
Are the DB and Stroom containers up and working OK?

I will add a new command.sh script to the stack that will enable you to run any of the commands, e.g.
./command.sh reset_password -u foo -p bar
I terms of your broken stack, please can you explain exactly how you ran the reset_password command, e.g. exactly what command you ran and where from (e.g. inside the container or outside).  Where did you find the stroom-app-all.jar that you used?

why does database migration happen during password reset?
Every time stroom starts it verifies the state of the database to ensure all migrations it expects to be have been run.  This is the same for the commands like reset_password.  If the DB is up to date which it should have been the no migrations will happen.
This suggests that it was either pointing at a different DB or you were running reset_password using a jar file for a later version of stroom than you have in your stack.

Thanks again!
The stroom-app-all.jar was found inside the containers, in directories such as /stroom, /stroom-proxy, etc.
It seems like everything works now, The database rebuild required more space than was available, when I ran restart/start and it  tried to rebuild. (I guess, still figuring Stroom's finer details out).
(Unfortunately the physical server is lacking, there's not much more I can do about it except slowly learning where the logs are growing and setup tails/ tighten rotation as I discover new offending log locations in the containers)
So the password reset actually worked, everything else just broke when the migration was attempted with insufficient space I'm guessing.

Glad to hear it is working now.  Are you happy to close this now?

Glad to hear it is working now. Are you happy to close this now?

Yes, go ahead. thanks. - Just a quick addition before if you don't mind. stroom-app-all.jar seems to work, but where should I actually find stroom-app.jar? the documentation says "in the root directory" but I'm not sure what that means, given there are multiple containers, and I can't find it in the base stroom folder where start.sh, the config directory, etc are found.


Make block iteration and block parsing separate functions.

Describe your changes

Block iteration and block parsing are currently entwined concepts.
Currently, each iteration of the block paths results in parsing the file contents into a PrecomputedBlock.
This means that entire JSON of each block is serialized into memory upon each iteration, making ingestion slow.
In the future, we may want to avoid serializing the entire file into memory when ingesting. This helps.

Link issue(s) fixed
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have verified new and existing tests pass locally with my changes.
[x] I verified whether it was necessary to increment the database version.

@n1tranquilla,

Does this need a version bump?
Do you have any metrics to determine the impact on performance?

It should require a version bump, and there ought to be no impacts on performance. It's just a refactor that will facilitate future refactorings.


How to bind multi kong-plugin in ingress , separated with comma?

apiVersion: extensions/v1beta1
kind: Ingress
metadata:
   name: myapp-ingress
   annotations:
      plugins.konghq.com: http-svc-consumer-ratelimiting
spec:
   rules:
     - host: my.host.com
       http:
         paths:
           - path: /myendpoint
             backend:
               serviceName: myapp-service
               servicePort: 80

like below?

plugins.konghq.com: p1, p2

@WywTed Yes


Representing references as key tuples.

Fixes OpenNetworkingFoundation/TAPI#300

I haven't identified if it's a problem on ONOS yang-tool side issue or on the yang change, but
I am getting the following error:
Error in file: tapi-topology in<EMAIL_ADDRESS>at line: 127 at position: 20
YANG file error: Unable to find base leaf/leaf-list for given leafref path /tapi-common:context/tapi-topology:topology/tapi-topology:node/tapi-topology:owned-node-edge-point/tapi-topology:uuid

@y-higuchi Thanks for letting me know.

@y-higuchi I have validated files with pyang and everything looks fine. The line you are pointing to was not changed as a part of this patch. Moreover, the leaf  your tool is complaining about exists. However it might be some problem elsewhere but reported in a strange way. Is everything correct when you use original TAPI 2.0.1 yangs?

@bartoszm Yes, it was working fine with 2.0.1.
Since I saw textually similar reference got removed in the patch,
I was guessing may be the structure that is pointing to has changed or something,
and error part was left behind or something, but I guess that is not the case.
I'll try to see what is happening inside ONOS tool side a bit.

Hmm...wonder why this worked before since I believe this was NOT changed recently. But will rename the list.
Regards
Karthik
Sent from my Phone
On Mar 9, 2018, at 4:08 AM, Yuta HIGUCHI <notifications<EMAIL_ADDRESS>wrote:
@y-higuchi commented on this pull request.

In YANG/tapi-topology@2018-03-07.yanghttps://github.com/karthik-sethuraman/Snowmass-ONFOpenTransport/pull/1#discussion_r173396921:

@@ -333,10 +377,9 @@ module tapi-topology {
min-elements 1;
description "none";
}

       leaf-list node-rule-group {

           type leafref {

               path '/tapi-common:context/tapi-topology:topology/tapi-topology:node/tapi-topology:node-rule-group/tapi-topology:uuid';

           }

       list node-rule-group {

It was due to this.
the issue is parent (L366) and the child list has same name.
It is current limitation on onos yang tool side, but
would it be possible to rename one of them?
We had similar issue in 2.0 and following was done to avoid name clash.
OpenNetworkingFoundation/TAPI@73f936a#diff-1a7f8984b380a01f9a5e969f10b815c6R97https://github.com/OpenNetworkingFoundation/TAPI/commit/73f936a263485457dd650c5de03f6b3eab848d11#diff-1a7f8984b380a01f9a5e969f10b815c6R97

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/karthik-sethuraman/Snowmass-ONFOpenTransport/pull/1#pullrequestreview-102581681, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABymwha6nrFi89ymAepiRCZFXPRvLRJWks5tckaIgaJpZM4SgMnv.

@karthik-sethuraman with my change leaf-list was changed to list. thus maybe onos tooling was ok with names of attributes but it is not with the type names (I believe they map container/list nodes to types similarly to what ODL is doing)

Now it makes sense ☺
Regards
Karthik
From: Bartosz Michalik<EMAIL_ADDRESS>Sent: Friday, March 09, 2018 7:48 AM
To: karthik-sethuraman/Snowmass-ONFOpenTransport
Cc: Sethuraman, Karthik; Mention
Subject: Re: [karthik-sethuraman/Snowmass-ONFOpenTransport] Representing references as key tuples. (#1)
@karthik-sethuramanhttps://github.com/karthik-sethuraman with my change leaf-list was changed to list. thus maybe onos tooling was ok with names of attributes but it is not with the type names (I believe they map container/list nodes to types similarly to what ODL is doing)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/karthik-sethuraman/Snowmass-ONFOpenTransport/pull/1#issuecomment-371803319, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABymwj2RL29-ZMzUjUHOOcd2nFtEgQxCks5tcnnlgaJpZM4SgMnv.

Handled by ONF TAPI Pull Request 302


Invitation to Connect

Hi, I find your work remarkable! I have a potential opportunity I’d like to discuss. Can we connect?
Telegram: @the_vereckel
Discord: vereckel
WhatsApp: +44 7341 822373

Hi,
I just sent a friend request to your Discord account.
My Discord is techlead95

On Jun 15, 2024, at 4:26 AM, thebestfrag148 @.***> wrote:
Hi, I find your work remarkable! I have a potential opportunity I’d like to discuss. Can we connect?
Telegram: @the_vereckel
Discord: vereckel
WhatsApp: +44 7341 822373
—
Reply to this email directly, view it on GitHub https://github.com/adam5909099/mantine-date-time-picker/issues/7, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFIQYO3J4ENKQULNAWSFOTZHQCFPAVCNFSM6AAAAABJLRNDOWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TINRZGM3TMMQ.
You are receiving this because you are subscribed to this thread.


New connection to database.usgovcloudapi.net

New connection > someServer.database.usgovcloudapi.net > authentication type = Azure Active Directory - Universal with MFA support > Account = new account > sign in =<EMAIL_ADDRESS>> enter password  > account added > connect:
Microsoft.Data.SqlClient.SqlException (0x80131904): Login failed for user ''.
at Microsoft.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection, Action1 wrapCloseInAction)    at Microsoft.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)    at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)    at Microsoft.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.CompleteLogin(Boolean enlistOK)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean ignoreSniOpenTimeout, TimeoutTimer timeout, Boolean withFailover)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)    at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool, SqlAuthenticationProviderManager sqlAuthProviderManager)    at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)    at Microsoft.Data.ProviderBase.DbConnectionFactory.<>c__DisplayClass40_0.<TryGetConnection>b__1(Task1 _)
at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot)
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.SqlTools.ServiceLayer.Connection.ReliableConnection.ReliableSqlConnection.<>c__DisplayClass30_0.<b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ManagedBatchParser\ReliableConnection\ReliableSqlConnection.cs:line 314
--- End of stack trace from previous location where exception was thrown ---
at Microsoft.SqlTools.ServiceLayer.Connection.ConnectionService.TryOpenConnection(ConnectionInfo connectionInfo, ConnectParams connectionParams) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Connection\ConnectionService.cs:line 524
ClientConnectionId:25867c62-4ca2-4900-9968-1b3e94cdc954
Error Number:18456,State:1,Class:14

@aaomidi any ideas on this one?

Have you verified that this server connects via SSMS with AAD?

I have confirmed that AAD with MFA does work from SSMS.  The behavior has
changed a little bit since I first submitted (see attached).  At this
point, I have authenticated my AZGov AAD account via the browser popup and
the Azure Data Studio is just spinning after acknowledging the errors in
the screenshot.
On Thu, May 7, 2020 at 12:00 PM Amir Omidi<EMAIL_ADDRESS>wrote:

Have you verified that this server connects via SSMS with AAD?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/10023#issuecomment-625438686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGWE5BEX5AEFL54W4B76LITRQMAOHANCNFSM4MJ5GDRQ
.

I think this is going to be covered by #10314 but I'll keep this issue open just in case its different

Great, thanks Amir!
On Wed, May 13, 2020 at 1:46 PM Amir Omidi<EMAIL_ADDRESS>wrote:

I think this is going to be covered by #10314
https://github.com/microsoft/azuredatastudio/issues/10314 but I'll keep
this issue open just in case its different
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/10023#issuecomment-628236759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGWE5BD2VGQQVF2ZTT4AFALRRMBKNANCNFSM4MJ5GDRQ
.

Closing this issue for #10314, it seems to me that there is an underlying SqlClient issue, maybe we need to pass some more information.


Plugin crash at launch

Hello,
Please get log error from ULauncher v5.14.5 -v

Should be fixed if you update: aa9515e42487a7372fe7d72f7825d92a225f5a9b


chore(Java): add prime string

PR Checklist:

[x] My submission is formatted according to the guidelines in the contributing guide
[ ] My addition is on refer on the language README.md file
[x] My addition does not have a spelling problem
[x] My submission has a proper and user-friendly description of the algorithm
[ ] My submission has the time complexity of the algorithm
[x] My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] New algorithm
[ ] Optimization in previous algorithms
[x] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Other information:

Created a pr for issue #588

Also update readme.MD file.

Ok.I will do it

Hi @SamruddhiR,
Any udpated?

Hi @SamruddhiR,
This pull request will be closed, any updates please reopen this pull request.


Adding JavaScript outputmacro

I am trying to add a JavaScript outputmacro as described in the documentation and I am always getting an error from duktype. This is my macro code and a rule that uses the function:
outputmacro: ^inctest(^arg1) JavaScript permanent call int test int eval
function test(i) {
  return i+1;
}

topic: ~inc [inc increment]
u: (inc _~number) ^inctest(_0)

I am using the Linux executable file and the bot builds ok. But when I try to use that function I get a 'uncaught error' from duktype:
HARRY:  I don't know what to say.
juanmi: > inc 4
FATAL 56: uncaught error
PANIC 56: uncaught error (calling abort)
Aborted (core dumped)

Do you see something wrong in the code or do you know what could be happening? Thanks

which version of CS are you running. There had been a bug earlier.

Hi @bwilcox-1234 Thanks for the quick response, and thanks for chatscript, you have done a really got job, I am enjoying it a lot.
I am using version 7.111:
Finished compile

ChatScript EVSERVER Version 7.111 pid: 11066 64 bit LINUX compiled Jan 16 2017 04:54:53 host=local
Params:   dict:2097151 fact:800000 text:100000kb hash:100000
          buffer:80x80kb cache:1x5000kb userfacts:100 outputlimit:80000 loglimit:80000
WordNet: dict=214033  fact=85706  heap=12701684 Jan15'17-12:54:50
Build0:  dict=67756  fact=130446  heap=1146016 Compiled:Nov19'16-15:54:10 by version 6.9 "0"
Build1:  dict=175  fact=28  heap=214616 Compiled:Jan26'17-16:26:04 by version 7.111 "Orderflowers"
Used 60MB: dict 281,965 (24812kb) hashdepth 14/1 fact 216,180 (10376kb) heap 14185kb
           buffer (6400kb) cache (5000kb) POS: 918 (29kb)
Free 113MB: dict 1,815,186 hash 13580 fact 583,820 stack/heap 85,814KB

    Dictionary building disabled.

HARRY:  Hi, again.

Hi, I confirm I have same problem (using Linux Ubuntu: 14.04, with last ChatScript release:  7.111
I thought it was maybe a matter of parameters conversion (in the modified example here below Ireturn a string):
  outputmacro: ^inctest(^arg1) JavaScript permanent call string test int eval
  function test(i) {
    // return a string
    return '' + (i+1);
  }

  topic: ~inc [inc increment]
  u: (inc _~number) ^inctest(_0)

but the issue seems unrelated from the JS code. Maybe a question of duktype building?

Thanks @solyaris, I also tried an empty function and void return type
outputmacro: ^inctest(^arg1) JavaScript permanent call void test int eval
  function test(i) { }

And loading a js file with the code. But even trying to call the empty function returned the the same bug.

Hi Bruce,
there are news regarding the core dump caused by any function call to the CS-embedded javascript engine ?
BTW,  tried with last CS version both on Linux and on MacOs and I got everywhere the crash.
Here below more details on my test:
CS-program source code:
# macros ----------------------------------
outputmacro: ^inctest(^arg1) JavaScript permanent call string test int eval
function inctest(i)
  {
  // return a string
  return ('' + (i+1));
  }

outputmacro: ^fibonacci(^arg1) JavaScript permanent call void test int eval
// fib.js
function fib(n) {
    if (n == 0) { return 0; }
    if (n == 1) { return 1; }
    return fib(n-1) + fib(n-2);
}

// ...
function test(i) {
     var res = [];
     for (i = 0; i < 5; i++) {
     res.push(fib(i));
     }
 print(res.join(' '));
}

# topics ----------------------------------

topic: ~introductions keep repeat []
t: functions: fibonacci number, inc number

topic: ~test1 [fibonacci]
u: (fibonacci _~number) ^fibonacci(_0)

topic: ~test2 [inc increment]
u: (inc _~number) ^inctest(_0)

CS-Engine output:
>>:build giorgio
ChatScript Version 7.31  compiled Apr  8 2017 21:59:00

----Reading file simplecontrol.top
Reading outputmacro: ^giorgio
Reading table ^tbl:defaultbot
Reading topic ~control

----Reading file test.top
Reading outputmacro: ^inctest
Reading outputmacro: ^fibonacci
Reading topic ~introductions
Reading topic ~test1
*** Warning- line 31 of test.top: Word fibonacci only known in opposite case
Reading topic ~test2
*** Warning- line 34 of test.top: Word inc only known in opposite case

WARNING SUMMARY:
0 serious warnings, 0 function warnings, 0 spelling warnings, 2 case warnings, 0 substitution warnings

Finished compile

ChatScript EVSERVER Version 7.31 pid: 14272 64 bit LINUX compiled Apr  8 2017 21:59:00 host=local
Params:   dict:2097151 fact:800000 text:100000kb hash:100000
          buffer:80x80kb cache:1x5000kb userfacts:100 outputlimit:80000 loglimit:80000
WordNet: dict=214044  fact=85706  heap=12702032 Mar04'17-12:29:10
Build0:  dict=68042  fact=184643  heap=1148792 Compiled:Apr09'17-08:55:14 by version 7.31 "0"
Build1:  dict=12  fact=4  heap=126484 Compiled:Apr24'17-15:03:54 by version 7.31 "giorgio"
Used 63MB: dict 282,098 (24824kb) hashdepth 14/1 fact 270,353 (12976kb) heap 13977kb
           buffer (6400kb) cache (5000kb) POS: 918 (29kb)
Free 111MB: dict 1,815,053 hash 13577 fact 529,647 stack/heap 86,022KB

 Functions: fibonacci number, inc number

>> inc 2
FATAL 56: uncaught error
PANIC 56: uncaught error (calling abort)
Aborted (core dumped)
#

BTW, minor point:  I do not understand the warnings here:
*** Warning- line 31 of test.top: Word fibonacci only known in opposite case

What does it means?
Looking at the Warning message on engine source code: https://github.com/bwilcox-1234/ChatScript/blob/master/SRC/scriptCompile.cpp#L1925

I'm basically ignorant of javascript. I tried to look up syntax. For your inc function you do
// return a string
return ('' + (i+1));
But MY understanding is that a string concatenation would be "" + (i + 1).

Both those functions do the same thing, the type of quotes don't matter.
A better method would be (i + 1).toString(), but I don't think that's what's causing the issue anyway!

I'm javascript-ignorant me too; but '' + (i+1) appears a correct JS statement; in facts, double-checking with NodeJS :
# node -e 'i = 0; console.log('' + (i+1));'
# 1

or:
#  node -e 'i = 0; function test() {return('' + (i+1));}console.log(test())'
# 1

neither the problem seems related to the Duktape's  print()

Hi, I am getting the same error. Any update about it ?

getting the same error in version 7.73 . Tried with examples above and from the documentation. Javascript integration doesnt work.

I believe this issue should be addressed as a priority or removed from the ChatScript manual.

this issue should be addressed as a priority or removed from the ChatScript manual.

I quite agree with you, Matt.
As CS user, I spent some time in the past to investigate this CS crash that happens systematically on a linux OS. I'm not a C/ducktape expert and I was not able to solve myself.
So yes, I'd take off ducktape integration from CS.
But let me digress for a moment about a possible architectural open point:
In my opinion having, from a CS script, a fast access to "backend" functions made with any (external) programming language environment, is a fundamental value.
Javascript too is probably good idea, because is the most common programming language.
But I'am not sure about http://duktape.org/ as an "embbeded" JS environmewnt, inside the CS process (bug a part).
My preferred solution is to enable ChatScript to "call" external functions with some sort of RPC (Remote Procedure Call) IPC. Ands again nodeJS could be a perfect partner for that. What I have in mind is say super simple synchronous request/response (unix) socket call to an external server supplying these backend functions. If we are interesting I'd share a simple code (let say implementing the server with few lines of node).
Any other suggestion?

You can also use ^popen to execute a command and you can use Node.js  to create a command line command (or python or perl or whatever you like, this is a way to add any language to ChatScript). The inconvenient is that the node command will take some time to run, much more than any normal CS output-macro and this can become a bottlenet if you have many users for your bot.
Alternatively, you could have a node server and use ^jsonopen or ^tcpopen to send requests to the server. This way you don't have the server start-up time problem, but now you have to deploy and maintain one server more...
Anyway, a Javascript embedded environment would be great to have a script language more familiar an terse that the output-macros. Especially if you want to do some computation or take advantage of existing libraries, and that is why this thread keeps getting comments. Outputmacros can do the job technically, but JavaScript is  really popular programming language with access to a huge ecosystem of libraries, and having it inside CS would be a boost in productivity. But I am not very familiar with c++ and I could not fix the duktape it when I tried either.

Hi Juan
Yes, ^jsonopen or ^tcpopen are a solution now available. I was talking about the option to have a possible unix socket RPC to speedup a little bit runtime calls.  In this scenario there is a  need to setup a TCP server act to reply to function invocations.
Th embedded JS environment have pros but also minus (a part the bug ontopic), by example I see as a minus the fact backend functions are not decoupled ( my two cents ).
giorgio

Great conversation, and definitely highlighting certain things desired and required in CS.
I think, however, that we're looking at two separate things. The need to call external sources is already there in the form of jsonopen. Of course, this is limited by its synchronous nature. Under load, will this block the thread? If the endpoint is unavailable or slow, does it timeout? How do we deal with errors and the such? The volley nature of chatscript means that we need it to be sync, but has it been tested at scale?
The other aspect is just offloading some of the programmatic duties onto a more fully fledged language like JS. One of the reasons I'm looking into this is the idea of treating objects as arrays, and other things JS can do but CS is still a bit lacking in. Then again, just using Duktape completely limits all the things available in node packages.
I reckon that JS support will always be conceptually flawed and offloading is a better option. But with thousands of concurrent users, will it be a bottleneck, and has it really been tested outside of sandboxes?

Hi Matt,
you are extending  topics you too. That's not bad for me :)
The ducktape js integration (or disintegration) is related to the entire architecture of the CS engine.

The need to call external sources is already there in the form of jsonopen.

Right.  My minor note: I mentioned a unix socket IPC just as a possible alternative to a tcp socket, to speed up on a local host remote procedure call...

Of course, this is limited by its synchronous nature. Under load, will this block the thread?

I think so :(  If I well remember the http connection is managed by a thread containing a blocking curl request.  Besides, CS can scale "vertically" running multiple CS process instances "attached"  on a single IP port. See intro here:
https://github.com/bwilcox-1234/ChatScript/blob/master/WIKI/CLIENTS-AND-SERVERS/ChatScript-ClientServer-Manual.md
That's means you can mitigate the synchronous blocks... with multiple CS processes (one for each core) but I admit that's solve the waiting-mode architectural weakness :(

If the endpoint is unavailable or slow, does it timeout?

yes, see: https://github.com/bwilcox-1234/ChatScript/blob/master/WIKI/ChatScript-Json.md#web-json

How do we deal with errors and the such?

about errors:

http errors: I don't remember now, but see also:

https://github.com/bwilcox-1234/ChatScript/blob/master/WIKI/ChatScript-System-Functions-Manual.md#tcpopen--kind-url-data-function-

application errors: I reckon you can check the JSON reply of your remote procedure managing an err object, as we do in nodejs :)

The volley nature of chatscript means that we need it to be sync, but has it been tested at scale?

No shared testes, as far as I know / my side.

The other aspect is just offloading some of the programmatic duties onto a more fully fledged language like JS.

yes, is what i'd call "backend decoupling"

One of the reasons I'm looking into this is the idea of treating objects as arrays, and other things JS can do but CS is still a bit lacking in.

I agree and I understand difficulties managing  easily such simple data structure as an arrai in CS.Time ago I updated documentation exactly with an example that show how to manage an array of objects in CS:
https://github.com/bwilcox-1234/ChatScript/blob/master/WIKI/ChatScript-Json.md#arrays-of-objects

Then again, just using Duktape completely limits all the things available in node packages.

Agree!

I reckon that JS support will always be conceptually flawed and offloading is a better option.

I fair that... I agree.

But with thousands of concurrent users, will it be a bottleneck, and has it really been tested outside of sandboxes?

To be verified.

Hi, I just saw in chatbots.org forum that there is a workaround for this, I played a little with the sample code from Stephen and can confirm that this code works:

 outputmacro: ^declarejstest() javascript permanent eval
   function returnconstant () {
     return Math.PI
   }

outputmacro: ^jstest() javascript permanent call string returnconstant

topic: ~test-javascript KEEP REPEAT (test javascript)

u: TEST_ONE (one)
 ^declarejstest()
 Javacript says ^jstest()

@juanmirod Interesting. What's happening here that's making it not break then? Declare functions first and then call them later? Seems rather verbose!


Fix pre-commit typos, formatting and add admonitions

removing also old theme

I recall this PR from the other repo


fix(rowactionscell): isrowexpanded proptypes error being thrown for overflowmenu

An extra (invalid) prop was being passed to the OverflowMenu if a rowActions had an overflow

Summary

Fixes a propType error that was being thrown because the isRowExpanded prop was being passed all the way through to the <button DOM element if a table row had overflow actions

Change List (commits, features, bugs, etc)

fix(RowActionsCell): remove the isRowExpanded prop from the OverflowMenu

Acceptance Test (how to verify the PR)

Verify that for table rows with overflow that no proptype errors are shown

:tada: This PR is included in version 2.107.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ISPN-8850 NPE in ConsistentHashV2IntegrationTest.cleanUp

https://issues.jboss.org/browse/ISPN-8850
Always use @AfterClass(alwaysRun = true) instead of @AfterTest.
AbstractInfinispanTest.destroy() runs on @AfterClass and sets all
cache-like fields to null (ISPN-8478).

Test Result (81 failures / +74)

@tristantarrant Only 7 failures the second time, no idea why...

Merged, thanks !


[DEPRECATED] Replaces usages of zos.local.json with zos.dev-<...>.json

DO NOT MERGE
https://github.com/zeppelinos/zos/pull/348 is a cleaner version of this PR

@ajsantander there are some conflicts, could you please rebase/merge so I can review the latest version of your changes?

Closing in favour of https://github.com/zeppelinos/zos/pull/348


Adding Kill command

This is related to issue #375.
This is just a draft version of the kill command because it's not fully functional yet. The service.keep_alive? getter doesn't actually exist at this moment so I'll have to go add that to the Service class in the main brew repo. First, I just wanted to get some feedback that that was the right way to approach this before going over there and making that change.
Essentially the logic works as follows:
For each service to be killed ->

Check if it's running and print a message if it isn't.
Check if it's set to be kept alive and print a message if it is (meaning it can't be killed).
Stop the service.

Other details:

I renamed the original ServicesCli.kill() method to ServicesCli.launchctl_kill() to make room for the new method.
I decided to use the stop command with launchctl and systemctl to stop the services.

Thanks again @apainintheneck!


[BUTTON IDEA]: button shadow of familiar color of the button go into disco mode while hover.

button shadow of familiar color of the button go into disco mode while hover.


LLVM ERROR with any() + cache=True

Suppose we have a script file called "cache_test.py":
from numba import jit

@jit(nopython=True, cache=True)
def test(a):
    return (a < 0).any()

Run the function test:
First time:
>>> import numpy as np
>>> from cache_test import test
>>> a = np.arange(10)
>>> test(a)
False

Then quit Python.
Second time:
>>> import numpy as np
>>> from cache_test import test
>>> a = np.arange(10)
>>> test(a)
LLVM ERROR: Program used external function '__ZN5numba7targets9arraymath6np_any12$3clocals$3e12flat_any$242E5ArrayIbLi1E1C7mutable7alignedE' which could not be resolved!

>>> numba.__version__
'0.31.0'

The same error occurs with 0.30.1, where the error message is easier to read:
LLVM ERROR: Program used external function '_numba.targets.arraymath.np_any.<locals>.flat_any$2.array(bool,_1d,_C)' which could not be resolved!

I am able to replicate the error.
Btw, we borrowed the C++ mangling scheme.  So you can use c++filt to demangle the name:
$ c++filt '__ZN5numba7targets9arraymath6np_any12$3clocals$3e12flat_any$242E5ArrayIbLi1E1C7mutable7alignedE'
numba::targets::arraymath::np_any::$3clocals$3e::flat_any$242(Array<bool, 1, C, mutable, aligned>)

The error is due to the inner function (the impl for np.any) is not linked into the caller.  In fact, all @overload decorated method has this problem.


How can I change the name of namespaces created by LocallyRunOperatorExtension?

Hi. I wrote some tests using LocallyRunOperatorExtension, but I encountered the following error:
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: POST at:
https://xxxxxx.com/api/v1/namespaces. Message: namespaces "xxxxxxxxxoperatorsecuritye2e-xxxxxxxxxxxxxxxxxxxx"
already exists. Received status: Status(apiVersion=v1, code=409, details=StatusDetails(causes=[], group=null, kind=namespaces,
name=xxxxxxxxxoperatorsecuritye2e-xxxxxxxxxxxxxxxxxxxx, retryAfterSeconds=null, uid=null, additionalProperties={}),
kind=Status, message=namespaces "xxxxxxxxxoperatorsecuritye2e-xxxxxxxxxxxxxxxxxxxx" already exists,
metadata=ListMeta(_continue=null, remainingItemCount=null, resourceVersion=null, selfLink=null, additionalProperties={}),
reason=AlreadyExists, status=Failure, additionalProperties={}).

(I have obscured some data that reveals personal information).
I suspected it was because the class name + test method name I use as the namespace name was longer than 63 characters, and the UUID part that ensures uniqueness, was truncated, causing two tests running in parallel to create a namespace with the same name.
Looking at the java-operator-sdk code, it appears that there is no configuration to provide a custom namespace name, or at least modify the prefix.
Maybe I missed that configuration?
Besides, if there really is no such configuration, I'd like to add such a feature to the JOSDK, if you don't mind.

Hi @machenity , yes this would definetelly would make sense to improve.
What I can also think of is to always reserve space for UUID at the end of the name for the namespace. Would that fix you problem?
We could also make this configurable, if really necessary, but that seems to fix the problem nicely and generically. WDYT?

@machenity if you would volunteer to do this would be nice :)

Thanks for your answer :)
Actually first idea was adding an option accepts custom prefix string or string supplier for namespace.
It would be like this:
LocallyRunOperatorExtension.builder()
	.oneNamespacePerClass(false)
	.namespaceNamePrefix("NAMESPACE_NAME_PREFIX") // pass static string for prefix
	.withNamespaceNameSupplier(() -> "TEST" + new Random().nextInt().toString()) // pass Supplier<String> to call each time a namespace name is created
	.build()

But while reading your comments, your idea that truncating name earlier before postfix UUID seems nice.
Restrictive but quite simpler.
To be honest, I think having a 36-character UUID string in the name is overmuch for a 63-character limit. 🤔
Just wanna hear your opinion.

To be honest, I think having a 36-character UUID string in the name is overmuch for a 63-character limit.

Yes, definetelly, this can be a very short sequence of random chars.
I can also imagine something like this (very similar to your sample):
LocallyRunOperatorExtension.builder()
   .withNamespaceNameSupplier(ExtensionContext context -> context.getTestMethod() + "-"+randomString() )

The point is that extension context is passed to access information about the test case (method name). Maybe method name would be just enough in this case, but for sake of generality, this might be a way to go

So would be nice to do both, like able to specify namespace name supplier, but also fix this issue with to long names. (The late I would give priority, or in other words a better default supplier)


add support for criteo

@hguan6
Met an error when running this command: python data_processing.py -d criteo -n 10000000
tar: /home/ubuntu/netsdb/model-inference/decisionTree/experiments/dataset: Not found in archive
tar: Exiting with failure status due to previous errors
Any suggestions?

@hguan6
Met an error when running this command: python data_processing.py -d criteo -n 10000000
tar: /home/ubuntu/netsdb/model-inference/decisionTree/experiments/dataset: Not found in archive tar: Exiting with failure status due to previous errors
Any suggestions?

Sorry, the data processing script has not been fully tested. I manually downloaded the dataset and extract a portion of them to train a model. Will make it work soon.
Now the training and testing scripts work well.

@hguan6
Met an error when running this command: python data_processing.py -d criteo -n 10000000
tar: /home/ubuntu/netsdb/model-inference/decisionTree/experiments/dataset: Not found in archive tar: Exiting with failure status due to previous errors
Any suggestions?

Now the data_processing.py script should work. Note that the "-n" argument is not implemented in the current version.
I have a question concerning git commit. During development, I made many commits in order to push the changes to GitHub so that I can use the updated code in the agave environment. Shall I save the updated codes somewhere else, and delete the old branch and this old pull request, and then create a new branch and new PR?

This pull request has been merged to 41-decisiontree-gpu.


Update to Final ReactiveCocoa4

Can you make us a pull request?

#396 possible does this ¯_(ツ)_/¯

Nah, that's just Carthage. We need to update our podspec.

Moya updated to use release version of RC4.


Module Federation + SSR example

📖 Documentation
I have been trying to implement SSR with federated modules without much success. An example of server side rendering including bi-directional module sharing would be much appreciated.

SSR and federation won't work well. If you totally get what is going on, you can. If not this won't work.

tl;dr: federation is client-side


no light at all in iphone

hi , i noticed that the model that i upload in the local folder of my hass istance , work so well if i run from a imac 2015 , but , idk why  , when i try to open in my iphone12 i noticed that can control all the smart paired devices , but on light , no lights effect at all , no shadows , nothing , i've try to :

put extralightmode on
set the max lumens for each light spot
set diffent kinds of colors ( warm white , yellow )
try change a lots of lamps in sh3d
any suggest ?? im the only one that have this problem?

Hello @hashmaker1982 , I cannot reproduce it on my iPhone SE, I try to do some research. Are you using an iPhone browser or Home Assistant companion app ?

both , in iphone 12 but i found out my error right now , was the decay parameter that I had overlooked
btw ty


Keeping the text when no results are found (Selection)

Hey, I want to Implement the following functionality.

The user makes a search on a select

If there are no results I wan to keep the written text as the value of the select.

Is there a way to implement that using this library?

I think the link below has what you are looking for.
https://select2.org/tagging


Implement uv run --directory

Summary
uv run --directory  means that one doesn't have to change to a
project's directory to run programs from it. It makes it possible to use
projects as if they are tool installations.
To support this, first the code reading .python-version was updated so that
it can read such markers outside the current directory. Note the minor
change this causes (if I'm right), described in the commit.
Test Plan
One test has been added.
--directory
Not sure what the name of the argument should be, but it's following uv
sync's directory for now.
Other alternatives could be "--project". Uv run and uv tool run should
probably find common agreement on this (relevant for project-locked tools).
I've implemented this same change in Rye, some time ago, and then we went
with --pyproject <path to pyproject.toml file>. I think using
pyproject.toml file path and not directory was probably a mistake, an
overgeneralization one doesn't need.

A test had to be fixed for windows but is now done.


Blog: 11 Ways (Not) to Get Hacked

Blog and images /cc @kbarnard10
From the preview the images look a little large - LMK if I can do anything to constrain them or similar (web design not my forte!)

/assign @kbarnard10

/lgtm
/approve


pr template update

I would like to add a pr template in which it checks for screenshot compulsory

could u give this a level 2 tag

please review my pr


Allow "cargo install gluon" to work

When I run cargo install gluon to add the repl binary to my path and execute gluon -i I get:
$ gluon -i
No such file or directory (os error 2)

As I understand it, it's looking for .glu files from a relative directory here.
compiler.load_file_async(vm, "std/repl.glu").sync_or_error()?;

I would like to add to my vim plugin the ability to execute the repl from within vim on snippets of code or the whole file.
I'm not sure where the .glu files would be required to exist for them to be picked up.

I guess the way to fix this is to add a build script which copies all the files in std/ to the output directory. Then src/repl.rs can be modified to load the file relative to the executable.

from the cargo install --help docs:
This command manages Cargo's local set of installed binary crates. Only packages
which have [[bin]] targets can be installed, and all binaries are installed into
the installation root's `bin` folder. The installation root is determined, in
order of precedence, by `--root`, `$CARGO_INSTALL_ROOT`, the `install.root`
configuration key, and finally the home directory (which is either
`$CARGO_HOME` if set or `$HOME/.cargo` by default).

So, it looks like there would be multiple places to check if std/ is dropped right beside the binary. Would it be better to have a default, well-formed location? I, personally, don't like the thought of .glu files sitting in my bin/ dir.
Perhaps there could be a $GLUON_STD or $GLUON_LIB env variable that's set? build.rs could copy std there and then src/repl.rs would check there with precedence and fall back to relative.

cargo install gluon_repl (moved the app to its own crate) is working now since all gluon files are compiled into the program itself, avoiding the issue with directories.I think this issue can be closed for now but long term the library files should be distributed separately from the app better binary distribution should be fixed first (#194).
GLUON_PATH can be used to specify additional directories to lookup imports.


minor improvements

This PR includes a few minor improvements:

Update to alpine 3.3 and use its --no-cache.
Use py-pip instead of installing pip manually
remove xclip stuff - mitmproxy should just fall back and offer to save the file.

Thanks for the work here! :smiley:

Why would you use py-pip instead, it's older.

py-pip: The alpine package is 8.1.1, pypi's pip is 8.1.2. I think that's recent enough and I like the fact that we can uninstall it cleanly afterwards.
--no-cache: This avoids the need to use --update and remove /var/cache/apk/* when done installing packages.