Tweak the 'vV' replacing logic and mimic Bower and NPM behavior

Those 2 commits greatly improve this plugin. I'm pasting here the commit messages as they describe pretty well what each commit does:
commit 1
Currently the 'vV' chars are replaced with str_replace,
after a preg_match. Because of how str_replace works, this could
lead to some potential bugs. Replace with a single preg_replace
that will handle the job in exactly the same way.
As a side bonus, we're getting rid of a loop, which could speed
up things a little bit.
commit 2
Mimic the behavior of NPM and Bower when comparing versions
constructed by 1 or 2 groups of numbers.
Versions as "1", "7.9", "v4" and "v98.2" are perfectly valid
and both Bower and NPM expand those to, accordingly, "1.*",
"7.9.*", "v4.*" and "v98.2.*". Until now this plugin
failed at doing so, which causes quite some failures when
trying to install packages that use that syntax (npmconf,
just to name one).
This commit fixes that by adding a single regex replace
that matches 1 or 2 groups of numbers and appends an ".*".
Also, the regex keeps in mind that other type of versions,
such as "~2" or ">=4.5" don't need to be expanded.
Also note that this fix will not break current unit
tests.

@francoispluchino Thank you for merging this!
Do you plan releasing a patch version (maybe <IP_ADDRESS>?) including this fix? The issues described in #122 prevent this plugin from working correctly with a lot of packages and imho having a stable version of this plugin with this PR will benefit everybody.

@alexandernst The new version 1.0.2 is available.

Oh, nice! Thanks!


[owners] allow for special "*" rule on owners.yaml that makes files not need approvals

this is because top level files will be constanty requested reviews from root level OWNERS. we need a way to make the top level files easier to be edited and approved.

Related to/possibly dependent on #278

This may be rendered unnecessary once we support teams; at most, it will be syntactic sugar for the "everyone" team

Re-opening for the syntactic sugar/easier to use than teams.


No way to shut down a RemoteControl Animation.

This blocks https://github.com/ManiacalLabs/BiblioPixel/issues/364 as even if run.seconds is set, the remote thread continues to run after the animation itself has terminated.

Call cleanup() on RemoteControl...
How are you starting it? I assume with bp run?
If that's the case, capture the PID and the kill it after a bit.

Well, I'm in bp!
In the branch, bp run now lets you run multiple files sequentially - that
is, if you set run.seconds (which you can from a command line argument), or
if there's some other terminating condition.
Works great - all the animations shut down in an orderly fashion except
remote.
The idea that people have to kill -9 the program to stop an animation isn't
really very polished.  We should be able to switch between animations more
slickly!
Also, you would want commands for remote like "shutdown" or "move to next
animation page".
Anyway, I'll send out this review which gets a lot of the way there and
then figure out tomorrow a nice way to do this.
On Sat, Aug 19, 2017 at 1:47 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Call cleanup() on RemoteControl...
How are you starting it? I assume with bp run?
If that's the case, capture the PID and the kill it after a bit.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-323484214,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsgqoDdfrsAIm4taC--zgVtb2QavWks5sZiKEgaJpZM4O8Jbo
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Ok... so you are setting run.seconds on RemoteControl?
Ahh.... RemoteControl is being bad. It just sits in step() waiting for something on the queue and not caring about anything else. It's meant to be the main thread and never killed but by the user.
Theoretically we could probably modify the cleanup() function to somehow kill that queue (it already closes it)...  oh wait... cleanup() won't get called because seconds would only do anything if step() returns so it can check the number of seconds.
so... I may have to set the queue to timeout and loop back around. I didn't because it made things a little slower. I'll ponder it.

Let's not do it now.  We have this problem in general with any animation
that starts a new thread and we should do it right.
The megatest will run a series of non-remote animations and then end up on
a remote and hang there.  That'll be perfectly good.
On Sat, Aug 19, 2017 at 2:03 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Ok... so you are setting run.seconds on RemoteControl?
Ahh.... RemoteControl is being bad. It just sits in step() waiting for
something on the queue and not caring about anything else. It's meant to be
the main thread and never killed but by the user.
Theoretically we could probably modify the cleanup() function to somehow
kill that queue (it already closes it)... oh wait... cleanup() won't get
called because seconds would only do anything if step() returns so it can
check the number of seconds.
so... I may have to set the queue to timeout and loop back around. I
didn't because it made things a little slower. I'll ponder it.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-323485573,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsi9yziITyZSDuU97nV01uX5pu7L_ks5sZiZEgaJpZM4O8Jbo
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Oh... this solution is only for RemoteControl. The problem is that it's
step() method is not respecting it's elders and doesn't loop unless it gets
something on the queue. Otherwise it blocks indefinitely. I will fix that
but if another animation does similar it's out of our hands. Note: No other
animations do that that we wrote. RemoteControl just thinks it's a special
snowflake.
On Fri, Aug 18, 2017 at 8:15 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Let's not do it now. We have this problem in general with any animation
that starts a new thread and we should do it right.
The megatest will run a series of non-remote animations and then end up on
a remote and hang there. That'll be perfectly good.
On Sat, Aug 19, 2017 at 2:03 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Ok... so you are setting run.seconds on RemoteControl?
Ahh.... RemoteControl is being bad. It just sits in step() waiting for
something on the queue and not caring about anything else. It's meant to
be
the main thread and never killed but by the user.
Theoretically we could probably modify the cleanup() function to somehow
kill that queue (it already closes it)... oh wait... cleanup() won't get
called because seconds would only do anything if step() returns so it can
check the number of seconds.
so... I may have to set the queue to timeout and loop back around. I
didn't because it made things a little slower. I'll ponder it.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-
323485573>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsi9yziITyZSDuU97nV01uX5pu7L_ks5sZiZEgaJpZM4O8Jbo>

.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-323486680,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6m9AADTiHjw0W7wzzCm8Cmdm0HTYks5sZikfgaJpZM4O8Jbo
.

This is a common issue though - we can force people to use our own thread
spawning mechanism only, and have it run a loop that ends when the
animation does.
I have a sketch for a "generic threaded animation" with a built-in queue
and it'll be much easier to tell people to use that than to teach them how
to use threads and queues...
On Sat, Aug 19, 2017 at 2:18 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Oh... this solution is only for RemoteControl. The problem is that it's
step() method is not respecting it's elders and doesn't loop unless it gets
something on the queue. Otherwise it blocks indefinitely. I will fix that
but if another animation does similar it's out of our hands. Note: No other
animations do that that we wrote. RemoteControl just thinks it's a special
snowflake.
On Fri, Aug 18, 2017 at 8:15 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Let's not do it now. We have this problem in general with any animation
that starts a new thread and we should do it right.
The megatest will run a series of non-remote animations and then end up
on
a remote and hang there. That'll be perfectly good.
On Sat, Aug 19, 2017 at 2:03 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Ok... so you are setting run.seconds on RemoteControl?
Ahh.... RemoteControl is being bad. It just sits in step() waiting for
something on the queue and not caring about anything else. It's meant
to
be
the main thread and never killed but by the user.
Theoretically we could probably modify the cleanup() function to
somehow
kill that queue (it already closes it)... oh wait... cleanup() won't
get
called because seconds would only do anything if step() returns so it
can
check the number of seconds.
so... I may have to set the queue to timeout and loop back around. I
didn't because it made things a little slower. I'll ponder it.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-
323485573>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsi9yziITyZSDuU97nV01uX5pu7L_ks5sZiZEgaJpZM4O8Jbo>

.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-
323486680>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AA6a6m9AADTiHjw0W7wzzCm8Cmdm0HTYks5sZikfgaJpZM4O8Jbo>

.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-323486918,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPdsgbz3qUWxOSmwfxxPWWxAji_FXvzks5sZinLgaJpZM4O8Jbo
.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com

Ah... .yeah... after 3.3 ;) But sounds reasonable, just need to see it.
On Fri, Aug 18, 2017 at 8:27 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

This is a common issue though - we can force people to use our own thread
spawning mechanism only, and have it run a loop that ends when the
animation does.
I have a sketch for a "generic threaded animation" with a built-in queue
and it'll be much easier to tell people to use that than to teach them how
to use threads and queues...
On Sat, Aug 19, 2017 at 2:18 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Oh... this solution is only for RemoteControl. The problem is that it's
step() method is not respecting it's elders and doesn't loop unless it
gets
something on the queue. Otherwise it blocks indefinitely. I will fix that
but if another animation does similar it's out of our hands. Note: No
other
animations do that that we wrote. RemoteControl just thinks it's a
special
snowflake.
On Fri, Aug 18, 2017 at 8:15 PM, Tom Swirly<EMAIL_ADDRESS>wrote:

Let's not do it now. We have this problem in general with any animation
that starts a new thread and we should do it right.
The megatest will run a series of non-remote animations and then end up
on
a remote and hang there. That'll be perfectly good.
On Sat, Aug 19, 2017 at 2:03 AM, Adam Haile<EMAIL_ADDRESS>wrote:

Ok... so you are setting run.seconds on RemoteControl?
Ahh.... RemoteControl is being bad. It just sits in step() waiting
for
something on the queue and not caring about anything else. It's meant
to
be
the main thread and never killed but by the user.
Theoretically we could probably modify the cleanup() function to
somehow
kill that queue (it already closes it)... oh wait... cleanup() won't
get
called because seconds would only do anything if step() returns so it
can
check the number of seconds.
so... I may have to set the queue to timeout and loop back around. I
didn't because it made things a little slower. I'll ponder it.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/
issues/383#issuecomment-
323485573>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsi9yziITyZSDuU97nV01uX5pu7L_ks5sZiZEgaJpZM4O8Jbo>

.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-
323486680>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AA6a6m9AADTiHjw0W7wzzCm8Cmdm0HTYks5sZikfgaJpZM4O8Jbo>

.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-
323486918>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AAPdsgbz3qUWxOSmwfxxPWWxAji_FXvzks5sZinLgaJpZM4O8Jbo>

.

--
/t
https://tom.ritchford.com https://tom.ritchford.com
https://tom.swirly.com https://tom.swirly.com
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/383#issuecomment-323487684,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6qh5FzQx502TtvAu2OGGpdySHYyfks5sZiv2gaJpZM4O8Jbo
.


Created new Responsive Navbar

Fixes Issue #1727

 #1727
Changes proposed

Screenshots

https://github.com/codervivek5/VigyBag/assets/137484958/19e57b62-e28a-4f9b-9ba8-d1223b91feab

Note to reviewers

Summary by CodeRabbit

New Features

Added specific href links for "MyOrders", "Cart", "Help", and "Contact" in the Aside component.
Introduced a new SearchBar component with search functionality on the Home page.

Bug Fixes

Improved state management and user interaction within the Navbar component.

Style

Adjusted code formatting and styling in AuthButton, NavLogo, and SearchBar components for consistency.
Updated class names and import statements for better readability and maintainability.

Refactor

Simplified MobileMenu prop structure and menu link functionalities.
Enhanced layout and content structure in the Home page.

kindly solve merge conflict......accept upcommit changes


Correct the name of the DidChangeWatchedFilesParams field.

This PR corrects the name of the DidChangeWatchedFilesParams field to match the specification (to wit, changes and not params).
While I never had the similar initialized error described in #68, this fixes the error messages I described in https://github.com/alanz/haskell-lsp/issues/68#issuecomment-374049116.

Thank you!


Debug: Support MS DAP new extensions to improve "Step Into Target" user experience.

The MS DAP spec recently introduced updates to the "Step Into Target" request, to improve user experience when multiple targets are available on the same line in the source file.
Please update probe-rs-debugger to be able to support this request.

Closing this as 'no intention to implement - at this time', but will leave the issue around in case anybody searches on this topic.


Custom Resources List is Never Loading if version does not match regex "v.*?"

To Reproduce
Steps to reproduce the behavior:

Go to 'Custom Resources → Group → Your CRD'
Click on selected CRD
Wait until list of CRs will load
It will never load

Expected behavior
Expected to see list of CRs for CRD
Screenshots

Environment (please complete the following information):

Lens Version: 3.4.0
OS: OSX
Installation method: OSX Official Image

Logs:
When you run the application executable from command line you will see some logging output. Please paste them here:
/Applications/Lens.app/Contents/MacOS/Lens
info: Checking for update
info: Update for version 3.4.0 is not available (latest version: 3.4.0, downgrade is disallowed).
warn: PrometheusLens: failed to list services: [object Object]
warn: PrometheusLens: failed to list services: [object Object]
info: using operator as prometheus provider
info: using operator as prometheus provider

Thank you,

Looks like issue is caused by an assumption that version name matches pattern "v.*" at kube-api.ts#L36
In this particular CRD version is named as hub which causes regex to fail.
Changing regex to below fixes the problem:
static matcher = /(\/apis?.*?)\/(?:(.*?)\/)?(.*?)(?:\/namespaces\/(.+?))?\/([^\/]+)(?:\/([^\/?]+))?.*$/

Can it be accepted as a solution, or are any other side effects foreseen? Locally do not see any problems with adjusted regexp

I am trying to reproduce this issue. Would you be able to provide the crd that you are using and seeing this behaviour with. Thanks.

@Nokel81 thank you a lot!

You are most welcome.


Request delays

Hi Sammy,
Part of the solution to implementing ratelimiting (#154).
Let me know what you think.

I use both in my tests. One with mock and one without. Both get tested.

Ready to merge

Great work on this @anthonyvancauwenberghe. Thank you for contributing.


Fix context related bugs in orquesta

Fix access to st2 context in various locations of the workflow definition.  Fix access to api_user in the action context from the task action metadata.

Closes #4435.

I assume we need a changelog entry for this change?

@Kami @blag @cognifloyd Your individual concerns are addressed.


speed up tests

They're slow -

Really really slow, I agree.
The problem is the lack of the ability to do the startup and tear down after all the tests, rather than after each test so right now we have to cycle the Analysis Server, which is really expensive. I don't really like the counting hack, it seems like I'm likely to make mistakes. Hence the slowness.

There may be a way using groups - I'll investigate.

The new test package has a dependency on the analyzer. This is going to cause a lot of version lock issues for us, esp. as we'll need to be on bleeding edge versions of the analyzer.


noto color emoji compat

some emojis without varitation selector-16 are not displayed
like: ❄ vs ❄.

Can you please be a bit more specific as to where are they not displayed? Which operating system, application, browser, etc. Thanks

os: \a11 (ofc on a12 the compat won't have to change it or you won't be able to know)
application: \chrome or webview. ofc messages is also going to display old desgin when vs-15 unicode used, but it has no problems without vs-16. compat displays both with vs-16 and without vs-16 unicode on the apps ...like messages.


FixFixing  Kerberos Principal Not Mapping Correctly to a Local User (#40905)

Proposed changes
error log
http://<IP_ADDRESS>:8111/buildConfiguration/Doris_External_Regression/529960?buildTab=tests&status=muted&focusLine=NaN&suite=&package=external_table_p0.kerberos&class=test_single_hive_kerberos&expandedTest=build%3A(id%3A529960)%2Cid%3A2000000060
Changes

If there are multiple realms, it's best to configure hadoop.security.auth_to_local to correctly map Kerberos Principals to local users.
Thrift exceptions should use those from hive-shade, but to avoid future changes in hive-shade, I omitted part of the package name prefix.

(cherry picked from commit 902cf1e913f1b01dccb5978e9c9d521a2e8c9350)
#40905

run buildall

run p0


Use Rx5 by default

Started working on porting the library to use Rx5 by default.

Updated the getAdapter() and test setup
Changed peerDependency from rx to rxjs
Updated README
Updated blog example: changed to import local rxConnect instead of the version currently on NPM, and refactored the code to use Rx5 syntax
Updated wikistream example: changed to import local rxConnect and removed setting up the adapter

What's left to do:

Wikistream example is broken (as discussed). It seems wikipedia changed the API
Update the docs + codepen examples. I don't think I can do this because they refer to the version hosted on NPM which is still the latest version that uses Rx4 by default. Once the new version is released, I can change these.
I have a small wish to replace Codepen by Codesandbox ( https://codesandbox.io ) - it allows specifying the dependencies and pulls them straight from NPM, so it's easier to see what dependencies a project has.

BTW I am not sure if the adapter approach is really good. We found out that we were including both Rx4 and Rx5 in our bundle because current rx-connect has a peerDependency on Rx5 even though we wanted to use Rx5. It turns out we had Rx4 in our node_modules directory because it was a dependency of one of our devDependencies (chimp) and webpack decided to bundle it anyway...

Hi @jseminck,
Great job! And thanks for pointing me at CodeSandbox, it looks great indeed :)
I'll move code examples from docs to the repo and point CodeSandbox at them, most probably today, so that we can fix the docs together with this PR, ok?

@jseminck done :) Now you can update the docs with RxJS5, see https://github.com/bsideup/rx-connect/tree/master/examples/docs for sources (FYI you can run locally with yarn serve)

Tackled comments. Updated some examples. Also made examples on Code Sandbox with the current dependency but by changing the adapter: https://codesandbox.io/s/vn9qjnqovl
Some things...
Timestamp
Seems to work, but need to map to a plain object
@rxConnect(
  Rx.Observable
    .timer(0, 1000)
    .timestamp()
    .map(timestamp => ({ ...timestamp }))
)

JSONP
It seems RxJS5 doesn't have JSONP support out of the box, which is needed for the Wikipedia examples. See https://github.com/ReactiveX/rxjs/issues/2095 & https://github.com/ReactiveX/rxjs/issues/1223
As a workaround:
// RxJS5 doesn't support JSONP out of the box
function fetchFromWikipedia(search) {
  return fetchJsonp(
    `https://en.wikipedia.org/w/api.php?action=opensearch&search=${search}&format=json&callback=JSONPCallback`
  ).then(response => response.json());
}

function searchWikipedia(search) {
  return (
    Rx.Observable
      .from(fetchFromWikipedia(search))
      // Wikipedia has really weird response format o_O
      .map(([, titles, , urls]) =>
        titles.map((title, i) => ({ title, url: urls[i] }))
      )
  );
}

Maybe we can find a better example that doesn't need JSONP and we can use plain ajax?
getAdapter()
After changing the example to use webpack alias, I had to change getAdapter(). I think there's some issue in the build process for the blog example.
export function getAdapter() {
    const adapter = rxConnect.adapter || require("./rx5Adapter");
    return adapter.__esModule ? adapter.default : adapter;
}

I'll go over the examples once more later, probably still missed some stuff... The main differences are:

flatMapLatest to switchMap
debounce to debounceTime
.jsonp to ajax
The timestamp thing

@bsideup have you seen my comment @ https://github.com/bsideup/rx-connect/pull/9#discussion_r137447317 ?
Otherwise, what else do we need to do to get this in? Would love to have this 😄

@jseminck yeah, sorry for missing that. I left one comment re plain objects and minor one about pluck. Happy to merge once at least the first one is addressed :)

@jseminck nice! Please also update the examples (Timers) and we're ready to go 🎉

Just pushed this now to run it with our project & see if everything remains
green. Will update examples during the day.
On Thu, 21 Sep 2017 at 09:29, Sergei Egorov<EMAIL_ADDRESS>wrote:

@jseminck https://github.com/jseminck nice! Please also update the
examples (Timers) and we're ready to go 🎉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bsideup/rx-connect/pull/9#issuecomment-331064151, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJJI0U5cXXL_FCMfOuoXceeCRgW8soVrks5skgIugaJpZM4PLJH3
.

@jseminck merged! Thank a lot for your contribution :) Will release it in a few minutes

I will test it further, and update code sandbox to use the latest version
too!
On Mon, 25 Sep 2017 at 08:52, Sergei Egorov<EMAIL_ADDRESS>wrote:

@jseminck https://github.com/jseminck merged! Thank a lot for your
contribution :) Will release it in a few minutes
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bsideup/rx-connect/pull/9#issuecomment-331783484, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJJI0RBtDt-GljtxiXpNDUhJzbF5AGs9ks5slz-UgaJpZM4PLJH3
.

@jseminck released as 0.7.0 🎉
I'll update CodeSandbox and remove JSONP :)

@jseminck done :) Thanks again :)


Add support for SNI on ssl connection

I will have to rebuild my Client API library with another thign because andoris-async-hhtttp doesnt nsuppoer SNI so wrong certificate so no connection. really need this right now...

+1

SNI feature will be solved in #991, closing this as duplicate


Testing halted:
I don't seem to have permissions to access Limitless according to my logs and lack of connecting, I think I need to be whitelisted.

Testing.

@YuppyPlays this might be a rez rights issue, I will review how I add the food entities.

@misslivirose I'm switching the mousePressOnEntity call to the server script. I will have a version of it working in hifi://burrow soon (and will update you on that when it is ready). Until the user tests are finished today, I will not be updating the S3 scripts.


config-only package specialization

remove the config and build tools that expect js source, which doesnt exist in a config only package
some of the things that can be removed: https://github.com/traverson/semantic-release-config/commit/43bb33bff34a702f17e66670978b5c0599c0362e

also consider that main won't be built into lib/, so the package file shouldnt assume that main exists there (or that a module file exists to point to) https://github.com/traverson/semantic-release-config/commit/07ecd247e053031d2a78b96b5a982dcadb58b57b


Add a feature to the exists index

Hi,
How can I add a feature to exist my index in python?

Hi @zpmmehrdad if you are asking about increasing dimensionality of input vectors, this is not possible. Furthermore, I don't think it's worth a support. However, one can use sparse vectors with the large enough dimensionality. Please, use https://gitter.im/nmslib/Lobby?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge for these kind of discussions. Thank you!

Hi @searchivarius, Thanks for response in fact my problem is that i created a index with 10,000 features and now I wanna add another feature to the index.i don't want to create an index again.is it possible?

@zpmmehrdad unfortunately, it is not, sorry. It is also not clear how to include additional features. Adding a single feature can change a lot.

So what can I do?
I have a large dataset almost 50,000,000 records and every day it's getting big.it's mean I have to create an index every day!

Hi @zpmmehrdad I can feel your pain, but there is really no way to do this. Again, remember that adding a single feature can change everything. How do propose to deal this issue in a generic fashion?

Hi @searchivarius can I merge several indexes into one?
in your opinion what is the best solution for my problem?
Thnaks

@zpmmehrdad in principal it is possible, but we don't support it and doing so properly is an open research problem :-)

@zpmmehrdad I wonder, do you mean you what to append new data, not features (changing number of features =  changing dimensionality of the vectors)?

Hi @yurymalkov I mean, I have a large dataset of vectors about 50,000,000 (each record is 4096 dimension) and I want to create an index once and after that, I add another feature to the index because my dataset is growing every day. I understood that I should create an index every time.
what should I do?

Hi @zpmmehrdad we don't currently support ANY updates, except for SW-graph (which is slower than HNSW). However, we are both quite confused what you consider a feature. A feature is usually one vector dimension, but your understanding is, apparently, different.

record == row, feature == column, add row or add column ? Is it ok to add rows ? and how to do it?

@xxcheng0708 currently the index is static, you add rows and create an index.


Don't import pew package in setup.py

This fix duplicates the list of available commands by explict list in
the setup.py. The advantage is pew package doesn't need to be imported
at all in setup.py.
Setuptools setup.py imports project's package to get list of available
commands and create entry points. Pew module imported from package's
__init__.py imports pathlib package. In the end this setup.py cannot be
used without pathlib package installed first.

Thanks!
I'm actually thinking of removing the pew_* commands in the next release, since I never truly liked the added complexity they cause both in pew.py and in setup.py
I asked a couple of users, and apparently almost noone uses them (please let me know if you do!)
This means that the patch will be a bit different
(Again: so many (good!) things happened in my life that pew has really been neglected lately :/ but the next release is definitely on my TODO list )

I'm actually thinking of removing the pew_* commands in the next release

:+1:


Failed stream close of QUIC

`
Test quic send message:
AddressSanitizer:DEADLYSIGNAL
=================================================================
==1183514==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000018 (pc 0x555d5f62787a bp 0x7ffd955b6150 sp 0x7ffd955b6130 T0)
==1183514==The signal is caused by a READ memory access.
==1183514==Hint: address points to the zero page.
    #0 0x555d5f62787a in nng_stream_send /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5
    #1 0x555d5f6619b4 in stream_xfr_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:49:2
    #2 0x555d5f661a77 in nuts_stream_send_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:144:2
    #3 0x555d5f5e6335 in test_quic_send_message /home/jaylin/Projects/nanomq/nng/src/supplemental/quic/quic_api_test.c:56:6
    #4 0x555d5f5e8e26 in test_do_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1026:9
    #5 0x555d5f5e54c9 in test_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1122:23
    #6 0x555d5f5e2b59 in main /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1724:13
    #7 0x7faf98a29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #8 0x7faf98a29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #9 0x555d5f523704 in _start (/home/jaylin/Projects/nanomq/nng/build/src/supplemental/quic/quic_api_test+0x82704) (BuildId: 341cb2b5e180c52bf9ee1244052ca8614adb40bb)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5 in nng_stream_send
==1183514==ABORTING
Test quic recv message:
AddressSanitizer:DEADLYSIGNAL
=================================================================
==1183578==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000018 (pc 0x555d5f62787a bp 0x7ffd955b60d0 sp 0x7ffd955b60b0 T0)
==1183578==The signal is caused by a READ memory access.
==1183578==Hint: address points to the zero page.
    #0 0x555d5f62787a in nng_stream_send /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5
    #1 0x555d5f6619b4 in stream_xfr_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:49:2
    #2 0x555d5f661a77 in nuts_stream_send_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:144:2
    #3 0x555d5f5e688c in test_quic_recv_message /home/jaylin/Projects/nanomq/nng/src/supplemental/quic/quic_api_test.c:100:6
    #4 0x555d5f5e8e26 in test_do_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1026:9
    #5 0x555d5f5e54c9 in test_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1122:23
    #6 0x555d5f5e2b59 in main /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1724:13
    #7 0x7faf98a29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #8 0x7faf98a29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #9 0x555d5f523704 in _start (/home/jaylin/Projects/nanomq/nng/build/src/supplemental/quic/quic_api_test+0x82704) (BuildId: 341cb2b5e180c52bf9ee1244052ca8614adb40bb)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5 in nng_stream_send
==1183578==ABORTING
Test quic mqtt connection:
AddressSanitizer:DEADLYSIGNAL
=================================================================
==1183642==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000018 (pc 0x555d5f62787a bp 0x7ffd955b60d0 sp 0x7ffd955b60b0 T0)
==1183642==The signal is caused by a READ memory access.
==1183642==Hint: address points to the zero page.
    #0 0x555d5f62787a in nng_stream_send /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5
    #1 0x555d5f6619b4 in stream_xfr_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:49:2
    #2 0x555d5f661a77 in nuts_stream_send_start /home/jaylin/Projects/nanomq/nng/src/testing/streams.c:144:2
    #3 0x555d5f5e6ebc in test_quic_mqtt_connection /home/jaylin/Projects/nanomq/nng/src/supplemental/quic/quic_api_test.c:147:6
    #4 0x555d5f5e8e26 in test_do_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1026:9
    #5 0x555d5f5e54c9 in test_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1122:23
    #6 0x555d5f5e2b59 in main /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1724:13
    #7 0x7faf98a29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #8 0x7faf98a29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #9 0x555d5f523704 in _start (/home/jaylin/Projects/nanomq/nng/build/src/supplemental/quic/quic_api_test+0x82704) (BuildId: 341cb2b5e180c52bf9ee1244052ca8614adb40bb)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/jaylin/Projects/nanomq/nng/src/core/stream.c:263:5 in nng_stream_send
==1183642==ABORTING
Test quic multi-stream:
AddressSanitizer:DEADLYSIGNAL
=================================================================
==1183706==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000008 (pc 0x555d5f6277d6 bp 0x7ffd955b6270 sp 0x7ffd955b6260 T0)
==1183706==The signal is caused by a READ memory access.
==1183706==Hint: address points to the zero page.
    #0 0x555d5f6277d6 in nng_stream_close /home/jaylin/Projects/nanomq/nng/src/core/stream.c:249:5
    #1 0x555d5f5e75cb in test_quic_multi_stream /home/jaylin/Projects/nanomq/nng/src/supplemental/quic/quic_api_test.c:181:2
    #2 0x555d5f5e8e26 in test_do_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1026:9
    #3 0x555d5f5e54c9 in test_run_ /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1122:23
    #4 0x555d5f5e2b59 in main /home/jaylin/Projects/nanomq/nng/src/testing/acutest.h:1724:13
    #5 0x7faf98a29d8f in __libc_start_call_main csu/../sysdeps/nptl/libc_start_call_main.h:58:16
    #6 0x7faf98a29e3f in __libc_start_main csu/../csu/libc-start.c:392:3
    #7 0x555d5f523704 in _start (/home/jaylin/Projects/nanomq/nng/build/src/supplemental/quic/quic_api_test+0x82704) (BuildId: 341cb2b5e180c52bf9ee1244052ca8614adb40bb)

AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV /home/jaylin/Projects/nanomq/nng/src/core/stream.c:249:5 in nng_stream_close
==1183706==ABORTING
Summary:
  Count of all unit tests:        5
  Count of run unit tests:        5
  Count of failed unit tests:     5
  Count of skipped unit tests:    0
FAILED: 5 of 5 unit tests have failed.

`
Above is a failed CI run on my env.

The error was more likely to happen in aio or quic dialer start.


Unable to install to install Optimus

Hello all,
I'm really excited about what I've read about Optimus. Unfortunately, I'm coming across a number of issues getting it up and running.
I have followed the instructions here:
https://medium.com/hi-optimus/how-to-install-jupyter-notebook-4-4-0-and-optimus-on-ubuntu-18-04-92ff5ef30ea4
However, whenever i try to issue the following script to my notebook I get the following error:
import optimus as op
ModuleNotFoundError                       Traceback (most recent call last)
 in ()
----> 1 import optimus as op
ModuleNotFoundError: No module named 'optimus'
Im running python 3.6
I feel that this is a simple error that I have made, but I'm at a loss.
Any help will be greatly appreciated.
Cheers

Hi! Thanks for the interest. That post it’s a little outdated. You can run:
pip install optimuspyspark
And then check the examples from the master branch!
You have to do a different import now.
Please let me know if this work for you! :)

Hi FavioVazquez,
Thanks for reaching out. I'm sure I'm tried your suggestion and it didn't work out ... going to try it again now and let you know.

OK,
I'm not getting the following error from the terminal when I issue the following command after installing pip install optimuspyspark from https://github.com/ironmussa/Optimus
from optimus import Optimus
op = Optimus()
packt@ubuntu-c:~/.local/lib/python3.5/site-packages/optimus$ from optimus import Optimus
from: can't read /var/mail/optimus
packt@ubuntu-c:~/.local/lib/python3.5/site-packages/optimus$ op = Optimus()
bash: syntax error near unexpected token `('
packt@ubuntu-c:~/.local/lib/python3.5/site-packages/optimus$

Sorry, I meant to say I'm NOW getting the following error from the terminal when I issue the following command after installing pip install optimuspyspark from https://github.com/ironmussa/Optimus
from optimus import Optimus
op = Optimus()
packt@ubuntu-c:/.local/lib/python3.5/site-packages/optimus$ from optimus import Optimus
from: can't read /var/mail/optimus
packt@ubuntu-c:/.local/lib/python3.5/site-packages/optimus$ op = Optimus()
bash: syntax error near unexpected token `('
packt@ubuntu-c:~/.local/lib/python3.5/site-packages/optimus$

Any thoughts?

I tried to install on another unbuntu with the command sudo pip install optimuspyspark and I got the following error:
packt@ubuntu2:~$ sudo pip install optimuspyspark
The directory '/home/packt/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/home/packt/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Collecting optimuspyspark
  Downloading https://files.pythonhosted.org/packages/56/6a/93e4f3cedb245028820915173d932e2002cb6121a60c2cab6b013963cc91/optimuspyspark-2.1.3.tar.gz (82kB)
    100% |████████████████████████████████| 92kB 269kB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-ust1jm/optimuspyspark/setup.py", line 14, in <module>
        with open('requirements.txt') as f:
    IOError: [Errno 2] No such file or directory: 'requirements.txt'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-ust1jm/optimuspyspark/
packt@ubuntu2:~$

Any help will be greatly appreciated

Hi, I managed to get this up and running. However, when I go through the example shown here
https://github.com/ironmussa/Optimus
I run the command df.cols.replace("product","taaaccoo","taco")\ but there isn't any change to the table
Any reasons why?

hi @cpatte7372
It should work, I just execute
df =op.load.url("https://raw.githubusercontent.com/ironmussa/Optimus/master/examples/data/foo.csv")
df.table()

def func(value, arg):
    return "this was a number"

df\
    .rows.sort("product","desc")\
    .cols.lower(["firstName","lastName"])\
    .cols.date_transform("birth", "yyyy/MM/dd", "dd-MM-YYYY")\
    .cols.years_between("birth", "yyyy/MM/dd")\
    .cols.remove_accents("lastName")\
    .cols.remove_special_chars("lastName")\
    .cols.replace("product","taaaccoo","taco")\
    .cols.replace("product",["piza","pizzza"],"pizza")\
    .rows.drop(df["id"]<7)\
    .cols.drop("dummyCol")\
    .cols.rename(str.lower)\
    .cols.apply_by_dtypes("product",func,"string", data_type="integer")\
    .cols.trim("*")\
    .table()

Is this what you expect?

Hi ironmussa,
This worked perfectly, thanks.
I just have a few learning questions:

Can you explain what exactly the following command set out to achieve:
       def func(value, arg):

return "this was a number

Is there place where I can find all the switches e.g. remove_special_chars, remove_accents, years_between.

Thanks
Carlton
From: Argenis Leon<EMAIL_ADDRESS>Sent: 10 November 2018 23:56
To: ironmussa/Optimus<EMAIL_ADDRESS>Cc: cpatte7372<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ironmussa/Optimus] Unable to install to install Optimus (#391)
hi @cpatte7372 https://github.com/cpatte7372
It should work, I just execute
df =op.load.url("https://raw.githubusercontent.com/ironmussa/Optimus/master/examples/data/foo.csv")
df.table()
https://user-images.githubusercontent.com/37144/48307388-5353b680-e511-11e8-95bc-eec2aaf6bf09.png
def func(value, arg):
return "this was a number"
df
.rows.sort("product","desc")
.cols.lower(["firstName","lastName"])
.cols.date_transform("birth", "yyyy/MM/dd", "dd-MM-YYYY")
.cols.years_between("birth", "yyyy/MM/dd")
.cols.remove_accents("lastName")
.cols.remove_special_chars("lastName")
.cols.replace("product","taaaccoo","taco")
.cols.replace("product",["piza","pizzza"],"pizza")
.rows.drop(df["id"]<7)
.cols.drop("dummyCol")
.cols.rename(str.lower)
.cols.apply_by_dtypes("product",func,"string", data_type="integer")
.cols.trim("*")
.table()
https://user-images.githubusercontent.com/37144/48307404-91e97100-e511-11e8-91b5-971813dd4fc5.png
Is this what you expect?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ironmussa/Optimus/issues/391#issuecomment-437631421 , or mute the thread https://github.com/notifications/unsubscribe-auth/AXd82cCazDR7FAqYkVfjXYkeRu05AFGQks5ut2ergaJpZM4YXoY4 .  https://github.com/notifications/beacon/AXd82a1qwnz0omoyxC1gCOmuY3qXHlfWks5ut2ergaJpZM4YXoY4.gif

Hi argenisleon,
Any idea where I can find more cleansing examples?
Can you let me know where I can find a list of switches as shown below?
.cols.remove_accents
.cols.replace
.cols.rename
Thanks

@cpatte7372
About the first question:
def func(value, arg):
    return "this was a number"

Is a function applied all the integers in the 'product' column. More especicaflly in transform all  integers to the string "this was a number"
.cols.apply_by_dtypes("product",func,"string", data_type="integer")

You can find all the columns and row operacions here and here

You also can find a lot of examples and uses cases in the examples folder
Hope this help

@argenisleon
thanks for the information, that's great.
I'm trying to run the same commands from my Databricks workspace notebook (as you probably know Databricks is built on Spark and uses the same principle as Jupyter Notebook).
I issue the following commands:
!pip install optimuspyspark
!from optimus import Optimus
!op= Optimus()
!df =op.load.url("https://raw.githubusercontent.com/ironmussa/Optimus/master/examples/data/foo.csv")
!table()

However, there isn't any output. Do you have any idea why?
Cheers

@FavioVazquez can you please look at this?

@cpatte7372 Did you solve the problem?

@argenisleon this fixed the problem.
Thank you


Convert markdown to HTML

This is one markdown document currently, but would be nice to have an HTML document that looks good.  I don't know how to do this.
One suggestion from @bsipocz is the following example: https://epyc.astro.washington.edu/~bsipocz/fornax/html/#more-details-about-the-fornax-portal  (this link is not permanent).

this puts the documentation at the bottom of the website which houses the rendered notebooks under a heading of "More details about the Fornax Portal"
Do we want it to also be a standalone document?  if so, where?  Do we have main landing page already (similar to heliocloud.org)?
Do we have control over the number of headings that the page shows?  ie., currently there is a main heading of "Fornax Science Console" then under that there is "Getting Started", "Data access", etc.  Can we also have the individual questions listed under each section?  ie., under "Getting Started" list out also "How do I get an account/.", "How to login?" etc.,   I like to have a fuller table of contents so I can scan for my individual problem before being directed to the long document with everything in it (which makes scanning more difficult).  Scanning here means either by eye or with a text search.

this puts the documentation at the bottom of the website which houses the rendered notebooks under a heading of "More details about the Fornax Portal"

Location can be anywhere, we have full control over it, I just happened to add it to the bottom of the page.

One thing maybe worth to consider: cutting up the currenly one single md file into separate ones. The HTML can still show one or multiple of those as one page, but it would also allow better segmentation to sub pages.

This has been done. There is a temporary bug due to the duplicated #Documenation heading, but it will be fixed upsteam, so I'm closing this issue now as done.
Link to the section on the landing page will be https://fornax-navo.github.io/fornax-demo-notebooks/#documentation
with the content here: https://fornax-navo.github.io/fornax-demo-notebooks/documentation/README.html
(currently the landing URL is https://fornax-navo.github.io/fornax-demo-notebooks/#id1)


Added support for multilingual product name sorting

1. Why is this change necessary?
Sorting products by its name is not working properly when using translations on products to serve them in two different languages. When one sort products by name in a category listing they are ordered by their untranslated name.
2. What does this change do, exactly?
It joins the s_core_translations table for sorting by the suitable translation.
3. Describe each step to reproduce the issue or behaviour.
The common shopware installation serves a German product set with English translations. If you have an 'Igelmütze' in this shop it will be ordered wrong in an English listing as the translation says 'Hedgehog hat'. The listing displays 'Hedgehog hat' but it was sorted by 'Igelmütze'. This will be subtle as H and I are next to each other. But imagine a 'Strandtuch' product and its translation 'Beach towel'.
I recognized it in the Shopware 4 demo data in the category 'Genusswelten'/'World of Indulgence' with the product 'Wacholder Premium extra mild 32%' with its translation 'Juniper premium extra mild 32%'.
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-22265
Please use this issue to track the state of your pull request.


ADT location_category

Hi all, this is obviously a post-ATS consideration, but I would be interested in having the location_category mapped to distinguish different types of ICUs (Medical ICU (could combine with CCUs as well), Surgical ICU (could combine SICU, CTICU, Burn ICU), and Neuro ICU). I feel like practices differ considerably in approach to mechanical ventilation, sedation, etc in these settings so for certain types of studies, it may be helpful to distinguish between them.

@acortizmd great idea, this is the way MIMIC does it. @whiskey0504 we could just adopt the MIMIC careunit list of permissible value for location_category

for reference, this is the list of "care units" in MIMIC:

@acortizmd can you propose a expanded location_category CDE? Maybe use Penn and MIMIC. Our current location_category -> location_group

@08wparker I went through the different ICUs at Penn and have developed some general thoughts:

Over the course of time, ICUs often change affiliations, what is once a CCU may become a MICU later on, etc and if the EHR hasn't developed a new department name then the code may require a dttm flag to properly map hospitalizations to their type of ICU.

These are the types of ICUs that I would propose based off of what I have seen looking through our data. This is all from specific mapped ICU locations:

general_icu (this would be your typical community ICU that has a mix of surgical, medical, neuro, cardiac patients in it, most likely an open ICU model. I would favor the MICU/SICU type ICUs going in here as well)
medical_icu (this would be a totally/primarily medical ICU, closed ICU model, staffed by CCM, PCCM trained docs)
surgical_icu (this would combine SICU, burn ICU, trauma ICU, CVICU into one category - there may be utility in separating these further, especially when thinking about ECMO and how that may differentiate CVICU patients compared with other surgical ICU patients)
cardiac_icu (this would be a CCU without surgical patients)
neuro_icu (this would be a neuro ICU without surgical patients)
mixed_cardiac_icu (this is a combination of CCU and CVICU - we have several of these combinations in the Penn system and I imagine it may be similar at other institutions)
mixed_neuro_icu (this is a combination of neuro ICU and neuro SICU - similar to the cardiac one above, we have a couple of these in the Penn system)

The last thing I would say is that looking through our department names there are at least 3 or 4 ICUs that were created specifically for COVID surges. I favor lumping those in with the medical_icu category but it could be argued to separate them out if there was some desire to study how standard of care in these types of units during the surge differed from general MICUs.
I have not given much thought to the categorization of other wards at this time but I would think at least in that case it would make sense to follow the MIMIC model.


Add on issues

Hi,
I’ve installed your add on but I can’t seem to get the ring alarm integration working. It doesn’t give me any entities for the ring alarm.
I have MQTT installed and running. I’ve logged in to my ring account from the add-on but nothing. Please could you assist?

Well, you didn't provide any logs so all I can do is guess.  The logs will tell you whether the addon is working and, if the problem is on the addon or the Home Assistant side.  My first guess would be that you installed the Mosquitto addon but haven't configured the HA Mosquitto integration in Settings...Integrations but it's totally a blind guess since you provided no logs.

I think I might have a broken HA install as I’m not getting any logs at all!

Hmm, so when you start the addon and go to the log tab there's no logs at all there?  That's pretty weird since the addon is clearly running since you indicate you were able to authenticate and the web server for authentication also runs in the same addon and should definitely produce logs.  Is the Mosquitto addon displaying logs?

No logs showing anywhere! Ill do a fresh install tomorrow and try again


Rules for rewriting types

Rules for rewriting types enjoy a special property that functionality always holds. The usual rewrite rules are next to impossible to rewrite types in the real application. Already discussed in private emails, it seems there is no obvious way to generalize this, and so let's implement this special rule after #457 but before #383. This probably also needs syntactical support in #404.

@cangiuli @jonsterling When the conclusion is an equality judgment m = n in a, and you say rewrite eq in concl, what would you expect to happen? Which part should be rewritten?

Proposed design for the rewrite tactics:
a = b type
rewrite eq at left
rewrite eq at right

m = n in a
rewrite eq at left
rewrite eq at right
rewrite eq at type

a true
rewrite eq
(?) rewrite eq at type

@favonia I like your design. Re: if we have something like in concl, I would expect it to apply to the entire conclusion.

if we have something like in concl, I would expect it to apply to the entire conclusion.

Well, we need both rewrite blah at position in x and rewrite blah at position in concl. in is to pick the atomic judgment, and at is to specify the part. Maybe some native speaker (you) should step in and come up with something less confusing.

Sorry I made a mistake. The current parser will already fill in in concl for you if in is missing. So at least there's some natural way to rewrite the conclusion. However, to rewrite a hypothesis, it seems we need two prepositions. I picked in and at arbitrarily.

I think in and at are the right prepositions


feat: add roller shutter

@windkh Hey, is it possible to get this merged?

Yes ofcourse. Thanks alot! I will do this asap.

I am still waiting for my ordered hardware,... sorry for delay

I blindly merged without testing it.... thanks!!!

@gerdemann can you provide an example under the examples folder?


Smoothing/ Running Averages

Hi
I'm interested in producing a running average &/or smoothing via the average of a definable number of previous values (in a date series).  I could produce this via sql on the database that I'm getting the data from, but thought that this could be either a derived attribute or an aggregation function, but I'm a bit stuck on the best approach and also how to reference the values in previous cells.  Any guidance gratefully received.
Edit
It's amazing that just after posting I came across #140 which has given me something to work from.  I'll see what I can come up with.

OK, let me know how it goes! I think there's enough info in that other ticket to put you on the right track :)

Both of these are minor modifications of the code in #140  by philliproso
There are probably more efficient and elegant ways of doing this, but I've never used coffeescript before, so apologies in advance.
runningAverage takes account of whether some of the cells are empty or not,smoothedAverage does not
In the code for smoothedAverage, the value in a cell is the average of the current cell and the previous 3 cell values, in the earlier cells it's the average of the current cell and the previous cells.
runningAverage: (formatter=usFmt) -> ([attr]) -> (data, rowKey, colKey) ->
  sum: 0
  push: (record) -> @sum += parseFloat(record[attr]) if not isNaN parseFloat(record[attr])
  value: ->
    colKeys=data.getColKeys()
    counter = 0
    flat_col_key=colKey.join(String.fromCharCode(0))
    for item in colKeys
      flat_item=item.join(String.fromCharCode(0))
      if flat_item is flat_col_key
        itter=counter
      counter++
      prev_value = 0
      if itter >0
        denom = 1
        for i in [1...itter+1]
          aggregator = data.getAggregator(rowKey, colKeys[itter-i])
          if 'sum' of aggregator
            prev_value += aggregator.sum
            denom++
      return (@sum + prev_value) / (denom - 1)
  format: formatter
  numInputs: 1

smoothedAverage: (formatter=usFmt) -> ([attr]) -> (data, rowKey, colKey) ->
  sum: 0
  push: (record) -> @sum += parseFloat(record[attr]) if not isNaN parseFloat(record[attr])
  value: ->
    colKeys=data.getColKeys()
    counter = 0
    flat_col_key=colKey.join(String.fromCharCode(0))
    for item in colKeys
      flat_item=item.join(String.fromCharCode(0))
      if flat_item is flat_col_key
        itter=counter
      counter++
      for i in [1...itter+1]
        prev_value = 0
        if (itter -1) of colKeys
          aggregatorone = data.getAggregator(rowKey, colKeys[itter-i])
          if 'sum' of aggregatorone
            prev_value += aggregatorone.sum
        if (itter -2) of colKeys
          aggregatortwo = data.getAggregator(rowKey, colKeys[itter-(i+1)])
          if 'sum' of aggregatortwo
            prev_value += aggregatortwo.sum
        if (itter -3) of colKeys
          aggregatorthree = data.getAggregator(rowKey, colKeys[itter-(i+2)])
          if 'sum' of aggregatorthree
            prev_value += aggregatorthree.sum
        if itter > 3
          return ((@sum + prev_value) / 4)
        return ((@sum + prev_value) / (5 - (4 - itter)))
  format: formatter
  numInputs: 1

Both of these are minor modifications of the code by philliproso  in #140
runningAverage calculates a running average of non-blank cells
smoothedAverage calculates the average of the current cell and the 3 previous cells - it doesn't take account of blank cells, or if there have been fewer than 3 previous values, the average of the current cell and it's predecessors.
There are probably lots of more efficient and elegant ways of doing this in coffeescript, but this was my first time using it, so please indulge my inefficiencies (I got part of the way there in coffeescript, hacked the resulting javascript, then tried to work out the necessary coffeescript to get that js.
Anyway I've now got node.js / npm installed so may further refine things, but it's working sufficiently for me needs
runningAverage: (formatter=usFmt) -> ([attr]) -> (data, rowKey, colKey) ->
  sum: 0
  push: (record) -> @sum += parseFloat(record[attr]) if not isNaN parseFloat(record[attr])
  value: ->
    colKeys=data.getColKeys()
    counter = 0
    flat_col_key=colKey.join(String.fromCharCode(0))
    for item in colKeys
      flat_item=item.join(String.fromCharCode(0))
      if flat_item is flat_col_key
        itter=counter
      counter++
    prev_value = 0
    if itter >0
      denom = 1
      for i in [1...itter+1]
        aggregator = data.getAggregator(rowKey, colKeys[itter-i])
        if 'sum' of aggregator
          prev_value += aggregator.sum
          denom++
    return ((@sum + prev_value) / (denom - 1))
  format: formatter
  numInputs: 1

smoothedAverage: (formatter=usFmt) -> ([attr]) -> (data, rowKey, colKey) ->
  sum: 0
  push: (record) -> @sum += parseFloat(record[attr]) if not isNaN parseFloat(record[attr])
  value: ->
    colKeys=data.getColKeys()
    counter = 0
    flat_col_key=colKey.join(String.fromCharCode(0))
    for item in colKeys
      flat_item=item.join(String.fromCharCode(0))
      if flat_item is flat_col_key
        itter=counter
      counter++
    for i in [1...itter+1]
      prev_value = 0
      if itter-1 of colKeys
        aggregatorone = data.getAggregator(rowKey, colKeys[itter-1])
        if 'sum' of aggregatorone
          prev_value += aggregatorone.sum
      if itter-2 of colKeys
        aggregatortwo = data.getAggregator(rowKey, colKeys[itter-2])
        if 'sum' of aggregatortwo
          prev_value += aggregatortwo.sum
      if itter-3 of colKeys
        aggregatorthree = data.getAggregator(rowKey, colKeys[itter-3])
        if 'sum' of aggregatorthree
          prev_value += aggregatorthree.sum
      if itter > 3
        return ((@sum + prev_value) / 4)
      return ((@sum + prev_value) / (5 - (4 - itter)))
  format: formatter
  numInputs: 1

Again, another minor modification
Percentage change from previous value
percentageChange: (formatter=usFmt) -> ([attr]) -> (data, rowKey, colKey) ->
  sum: 0
  push: (record) -> @sum += parseFloat(record[attr]) if not isNaN parseFloat(record[attr])
  value: ->
    colKeys=data.getColKeys()
    counter = 0
    flat_col_key=colKey.join(String.fromCharCode(0))
    for item in colKeys
      flat_item=item.join(String.fromCharCode(0))
      if flat_item is flat_col_key
        itter=counter
      counter++
    if itter >0
      aggregator = data.getAggregator(rowKey, colKeys[itter-1])
      if 'sum' of aggregator
          return (((@sum - aggregator.sum)/aggregator.sum))
    return
  format: formatter
  numInputs: 1

I'm going to close this as a duplicate of #140 and leave a link there to here.


Update Needed

The instructions are a bit out of date, are for the old admin experience, and do not mention that it takes up to 12 hours for the app to be available to the organization.

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

ID: 445892a6-5948-33e4-ce60-72670829d8ef
Version Independent ID: a7407a6f-6563-9a4b-4a95-7ed908ab322b
Content: Enable the Report Message add-in
Content Source: microsoft-365/security/office-365-security/enable-the-report-message-add-in.md
Service: o365-seccomp
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@kjulson, I updated the topic and added a note about 12 hours.
Thanks for the feedback.
Closing


Use pub tracks + Cleanup on pub disconnect

Update to use pub tracks for subscribe (rather than passing them in). Properly cleanup pub ice disconnects too.
I removed the old examples and will open a PR with some new ones.

Going to merge this and keep working on it. Feel free to leave feedback and I can address it in a follow up PR


Allow adding custom loopback client in acquireTokenInteractive

This PR adds a loopback client interface to InteractiveRequest for devs who would like to run their own server for listening the authZ code response in acquireTokenInteractive API.

Codecov Report

Merging #5578 (30ec458) into dev (9384b58) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.50% <ø> (ø)

Carriedforward from 9384b58

msal-browser
86.28% <ø> (ø)

Carriedforward from 9384b58

msal-common
86.01% <ø> (ø)

Carriedforward from 9384b58

msal-core
82.84% <ø> (ø)

Carriedforward from 9384b58

msal-node
83.41% <100.00%> (+0.03%)
:arrow_up:

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from 9384b58

msal-react
94.68% <ø> (ø)

Carriedforward from 9384b58

node-token-validation
88.88% <ø> (ø)

Carriedforward from 9384b58

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-node/src/index.ts
100.00% <ø> (ø)

...ib/msal-node/src/client/PublicClientApplication.ts
77.50% <100.00%> (+1.18%)
:arrow_up:

lib/msal-node/src/network/LoopbackClient.ts
63.63% <100.00%> (ø)

Looks good, 2 nits and any docs or samples we can add to help illustrate usage?

done and done!

Looks good. Do we have a docs (and .md file) defining why someone would need a loop back server? If yes, please link here, if not, can we add it? Can be handled as a separate PR.

Thanks! Added comments for API documentation and also to an Electron sample for usage. Need to update/re-organize some of our .md docs for API usage -so probably best to tackle this separately in another PR


SAK-32386: force 'overview' tool to the top of the tool menu via sakai.property

https://jira.sakaiproject.org/browse/SAK-32386
There was a need at our institution to force the 'Overview' tool (formerly 'Home') to the top of the tool list in every site. We've implemented this behind the following sakai.property which defaults to true (portal.forceOverviewToTop).

Maybe true isn't the best default for this feature. I'm open to suggestions/opinions.

Yeah it should probably be disabled by default as there will be some who have used page order to order their tools and changing that should be local decision.

Done; default is now false


re-extend mocha timeouts

The timeout failures in Travis are annoying so hopefully this nice, long timeout will prevent them from happening again.
Also, I used this nifty little tool to get all of my dependencies up to date. I still think it may be a good idea to use some 3rd party package to help keep dependencies up to date but many of the automated ones seem a little obtrusive. Using this module every so often to check our dependencies would be sufficient for now I think.

Oo npm-check seems hype, good call


Error compiling - Cryptographic failure --Error signing assembly-- CS1548

I have already clone the repo, and using SharpDevelop to test the Phone.Net app, but I got an error about the sign. Is there a change about it or Am I missing something to compile the Atapi.dll?
Hope you can help me, I want to test it. Thanks

You will need to provide your own signing cert, or turn that off in the settings.


Update unicode_names2 to the latest for Unicode 12

This nets us the lobster emojo's name:
U+0001F99E, 🦞 0x0001F99E, \0374636, UTF-8: f0 9f a6 9e, UTF-16BE: d83edd9e
Width: 1, prints as 🦞
Quotes as \u{1f99e}
Unicode name: LOBSTER

bors r+

bors r+


⚠ AnimeBytes has degraded performance

In 86d9252, AnimeBytes ($AB) experienced degraded performance:

HTTP code: 200
Response time: 1506 ms

Resolved: AnimeBytes performance has improved in 6a55fb3.


Add support for Minitest::SharedExamples

This is adding support for Minitest::SharedExamples. In Minitest we can use SharedExamples to create extensions inside application, which would make the searching of the test_method_name  and test_path bad. This is  adding a new pattern of searching when the old one does not work.

Could you add shared example test in test_examples/fast/ directory.
In order to use shared example do we have to add another gem as dependency or it's just build in minitest gem?
Could you also update test in spec/knapsack/adapters/minitest_adapter_spec.rb to cover both context for regular test and shared example.

It's just build in minitest gem. I will try update the specs.

@ArturT I'm not so sure about the spec part could you give some guide? I just finished the base line of it. It's like a fake_test that include the SharedExampleSpec and should correctly return the test_path for that.

You could keep single describe for .test_path method. Here is example.
  describe '.test_path' do
    subject { described_class.test_path(obj) }

    before do
      parent_of_test_dir = File.expand_path('../../../', File.dirname(__FILE__))
      parent_of_test_dir_regexp = Regexp.new("^#{parent_of_test_dir}")
      described_class.class_variable_set(:@@parent_of_test_dir, parent_of_test_dir_regexp)
    end

    context 'when regular tes' do
      class FakeUserTest
        def test_user_age; end

        # method provided by Minitest
        # it returns test method name
        def name
          :test_user_age
        end
      end

      let(:obj) { FakeUserTest.new }

      it { should eq './spec/knapsack/adapters/minitest_adapter_spec.rb' }
    end

    context 'when shared example test' do
      class FakeSharedExampleUserTest
        # TODO
      end

      let(:obj) { FakeSharedExampleUserTest.new }

      it { should eq './spec/knapsack/adapters/minitest_adapter_spec.rb' }
    end
  end

You need to prepare proper FakeSharedExampleUserTest for your case. You could use binding.pry to bind in .test_path method and see what's the structure of the obj in case of shared example (the shared example you added in test_examples directory).
You can run tests from that directory with
KNAPSACK_TEST_FILE_PATTERN="test_examples/{,/*/}/*_test.rb" bundle exec rake "knapsack:minitest[--verbose]"

I noticed you removed your test for .test_path method. I can write test for that then.

I added missing spec for minitest adapter https://github.com/ArturT/knapsack/blob/master/spec/knapsack/adapters/minitest_adapter_spec.rb#L129
I've just released knapsack v1.12.0. I also updated pro version gem https://github.com/KnapsackPro/knapsack_pro-ruby ( https://knapsackpro.com )
@dingn1 Thank you for the PR and great work :)

@ArturT Thanks for your help.


Build a property extraction Regex

This issue records an attempt to build a regular expression which can parse/match all the properties within a message template according the "spec" (which doesn't really exist yet, but it will be basically the same as Serilog).
(?<start>\{)
(?<destr>\@|\$)?
(?<property>[\w\.\[\]]+)
(?<align>,[^}:]+)?
(?<format>:[^}]+)?
(?<end>\})+

Tests
I sat at<EMAIL_ADDRESS>{{UHOH}}
Just biting '{@Fruit}' number "#{@Count:abc}"
Income was {Income} at {@Da te:d}
Welcome, customer #{$CustomerId,-10}, pleasure to see you
Welcome, customer #{CustomerId,-10:000000}, pleasure to see you
{place}, {0}
{1}, {Place}
{{blahblah}}

Here is a link to an online (ECMAScript) version of the regex.
It looks like {{UHOH}} is a problem, as I understand the double-braces should be "escaped" into a single brace in the output.

Here is an updated Regex which matches {{ and }} so they can be replaced with { and }
([^{]
 (?<start_prop>\{)
 (?<destr>\@|\$)?
 (?<property>[\w\.\[\]]+)
 (?<align>,[^:}]+)?
 (?<format>:[^}]+)?
 (?<end_prop>\})+)
|
(?<double_open>\{\{)
|
(?<double_close>\}\})

with the input:
I sat at<EMAIL_ADDRESS>{{{{UHOH}}
Just biting '{@Fruit}' number "#{@Count:abc}"
Income was {Income} at {@Da te:d}
Welcome, customer #{$CustomerId,-10}, pleasure to see you
Welcome, customer #{CustomerId,-10:000000}, pleasure to see you
{place}, {0}
{1}, {Place}
{{blahblah}}

Replacement format: DO=${double_open}, DC=${double_close}, S=${start_prop}, D=${destr}, P=${property}, A=${align}, F=${format}, E=${end_prop}
Gives the output:
DO=, DC=, S={, D=@, P=Chair.whatever.whatever, A=, F=, E=}
DO={{, DC=, S=, D=, P=, A=, F=, E=
DO={{, DC=, S=, D=, P=, A=, F=, E=
DO=, DC=}}, S=, D=, P=, A=, F=, E=
DO=, DC=, S={, D=@, P=Fruit, A=, F=, E=}
DO=, DC=, S={, D=@, P=Count, A=, F=:abc, E=}
DO=, DC=, S={, D=, P=Income, A=, F=, E=}
DO=, DC=, S={, D=$, P=CustomerId, A=,-10, F=, E=}
DO=, DC=, S={, D=, P=CustomerId, A=,-10, F=:000000, E=}
DO=, DC=, S={, D=, P=place, A=, F=, E=}
DO=, DC=, S={, D=, P=0, A=, F=, E=}
DO=, DC=, S={, D=, P=1, A=, F=, E=}
DO=, DC=, S={, D=, P=Place, A=, F=, E=}
DO={{, DC=, S=, D=, P=, A=, F=, E=
DO=, DC=}}, S=, D=, P=, A=, F=, E=


Can not search option if try to type

I am using  component of this Library for display the list for Countries and as you can imagine, the list can be pretty long.
And If I try to select the option by typing as traditional select work, its not working. Is there any way to do that?

I'll take it a look to see how to do

Check out v1.3.1, now it is supported.

update to v1.3.3 because of some bugs.
In case you are interested, if you are coding some 'area code', please check this out https://github.com/edwardfhsiao/react-custom-flag-select

Hey @edwardfhsiao
Saved the day, Thanks a lot for this quick enhancement.

i'm fixing some IE10, IE9 compatibility issue now, maybe will release a new version soon

@lhuria94 just updated to v1.3.4 and v0.1.0 for react-custom-flag-select


Emoji 15.1 emojis do not show up as big emojis

The following emojis do not show up as big emojis: 🙂↔, 🙂↕, 🐦🔥, 🍋🟩, 🍄🟫, and ⛓💥.
Below is BB vs. iOS

My initial guess is because they are 4-byte Unicode characters, which are seen as multiple characters when you test the length, so it doesn't get picked up as <= 3 characters

Fixed with updated iOS font and emoji rejex


Call iOS native presentViewController fail when  visible.

RN code:
<View style={{flex: 1}}>
  <Modal visible transparent>
    <Button
      onPress={() => NativeModules.Tools.TestPresent()}
      title="Present"
    />
  </Modal>
</View>

Objective-C code:
RCT_EXPORT_METHOD(TestPresent) {
  dispatch_async(dispatch_get_main_queue(), ^{
     RCTLog(@"in function");
    TestController *controller = [[TestController alloc] init];
    UIViewController *rooterController = [UIApplication sharedApplication].keyWindow.rootViewController;
    UINavigationController *nav = [[UINavigationController alloc] initWithRootViewController:controller];
    [rooterController presentViewController:nav animated:YES completion:^{
      RCTLog(@"in blocks");
    }];
  });
}

When I click Present button, nothing happens. TestController not present.
But in Xcode console area only  RCTLog(@"in function"); logs, the  RCTLog(@"in blocks"); not log;
And yes, breakpoints will go to [rooterController presentViewController:nav animated:YES completion:^{ and rootController is right, and then nothing will happen, so the completion block won't run.
But If I change RN code this:
<View style={{flex: 1}}>
    <Button
      onPress={() => NativeModules.Tools.TestPresent()}
      title="Present"
    />
</View>

Everything back to successful without Modal. The TestController will present when press Present button.
Necessary Information
Environment:
OS: macOS High Sierra 10.13.4
Node: 9.5.0
Yarn: 1.5.0
npm: 5.7.1
Watchman: 4.9.0
Xcode: Xcode 9.3 Build version 9E145
Android Studio: 2.3 AI-162.4069837
Packages: (wanted => installed)
react: 16.0.0 => 16.0.0
react-native: 0.50.1 => 0.50.1

Because RN presenting its own view controller for modals see RCTModalHostView.m.
From this answer https://stackoverflow.com/a/39175552/2266462

A View Controller can only present one presented controller at a time. Indeed, there is a property: presentedViewController: UIViewController? that implies this.

@vovkasm
Thanks, I see.
It's not like react ReactDOM.createPortal creates a view on the same level of top view.
RN's Modal just a present vc instead of a view.
And what's the right way to present a controller on Modal visible using native function?

https://github.com/facebook/react-native/blob/26684cf3adf4094eb6c405d345a75bf8c7c0bf88/React/Views/RCTModalHostViewManager.m#L78
@FaiChou you can present a view controller from the topmost view controller, but presenting from other VCs will silently fail
If you make a native bridge you can use an approach like this to find the top-most VC:
- (void)present:(UIViewController *)newVC {
  UIViewController *rootVC = UIApplication.sharedApplication.delegate.window.rootViewController;

  while (rootVC.presentedViewController != nil) {
    rootVC = rootVC.presentedViewController;
  }

  [rootVC presentViewController:newVC
                       animated:YES
                     completion:nil];
}

But that is a code smell IMO, since you should already know the topmost VC

thanks all


Add prolog swipl linter

Added SWI-Prolog linter, one of implementations of Prolog language.
It tries to load current file within sandbox.
It checks (syntactic / semantic) (warnings / errors).

I found infinite loop (:- repeat, writeln('loop!'), fail.) causes ALE not to work, but I could not find the workaround for this.

infinite loop causes ALE not to work

Okay, finally it can be detected by setting SIGALRM.
and if current file tries to overwrite SIGALRM (:- alarm(1, writeln('hacked!'), _, []).) in any line, it is blocked by sandbox.

Also added help file. please review.

For easier review, here is the test script.
:- initialization main.
:- writeln(hello).

main :-
  fib(10, N), writeln(N),
  halt.

fib(0, 0).
fib(1, 1).
fib(N, M) :-
  N1 is N - 1, N2 is N - 2,
  fib(N1, M1), fib(N2, M2),
  M is M1 + M2.

:- repeat, writeln('loop!'), fail.

SWI-Prolog can be installed by apt install swi-prolog-nox on Debian.

If an infinite loop fails, does that mean that this command will execute the code you have written? If so, is it possible to write a command which stops execution immediately, and which finds syntax errors, so that no malicious code can be executed by viewing a file?

If an infinite loop fails, does that mean that this command will execute the code you have written?

No. Always the commands are executed until signal was received (3 seconds (prolog_swipl_timeout) after by default).
But it is executed in sandbox.
Please let me explain.

Prolog programs have two types of clauses, rule and query.
The format of rule is Head :- Body.. It is an definition of a function (it's called "predicate" in Prolog).
hello :- writeln('hello!').

The format of query is defined like :- Query. or ?- Query..
The following code calls above hello predicate.
:- hello.

So if you want to disallow running arbitrary code, you might think it's OK to disallow queries (like parsing viewing file, extract only definitions, save it to a temporary file, and run interpreter with the file).
But I want to execute queries because it is necessary to detect syntax errors.
In Prolog, a user can define arbitrary binary or unary operators.
:- op(501, xfy, 'by').

L by R :- write(L), write(' by '), write(R).

:- hello by john.

:- op(501, xfy, 'by'). defines binary operator by.
Without it, L by R :- write(L), write(' by '), write(R). causes syntax error.
interpreter also tries to execute :- hello by john. but it is blocked by sandbox.
ERROR: /home/tyru/config/home/volt/repos/github.com/w0rp/ale/test.pl:5:
        No permission to call sandboxed `write(_7730)'
        Reachable from:
                  user:A by B

Defining operators are commonly used in Prolog.
Without it, almost Prolog programs can not be parsed.

Okay, it runs in a sandbox. That's all I needed to know.

Cheers! :beers:

Thanks! :smile:


[feat] Add proxy config for frontend to talk to backend

added a proxy config to take /api/* paths made from the frontend to backend

i.e. /api/coolUrl => localhost:3000/api/coolUrl

@karnthis we'll need to make the backend api routes prefixed with /api in order for development to work.

Sure we can do that. 👍


Images / text not matching current Azure DevOps (online) services

The "project administrators" can (also) be viewed on the "Overview" page. The screenshot is slightly outdated.

The view the "Organization owner", you have to go to "Organization settings" as indicated in the screenshots and not "Admin settings" as indicated in the text. The second screenshot of the "Overview" page is outdated though.

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

ID: 20dce3f1-1133-3084-c5f4-7a5d6d0d6cf8
Version Independent ID: 31579324-f0e5-607a-caa1-ec31b444af79
Content: Find an admin or organization owner - Azure DevOps
Content Source: docs/organizations/security/lookup-organization-owner-admin.md
Product: devops
Technology: devops-security
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@Styxxy - Thank you for the feedback. I will update the topic within the next few days to reflect the current UI.


📦 Bump starlette to fix bad encoding issue #5646

Update starlette to v0.22.0 to fix issue with TestClient doing an extra unquote in the URL query
(https://github.com/encode/starlette/issues/1952 and https://github.com/encode/starlette/pull/1953), introduced with the (tests)-Breaking change in v0.21.0 with the swap from requests.Session to httpx.Client for the TestClient
closes #5646

This will be great for the fix on GZipMiddleware too!

Thank you @azogue ! I spent forever tracking back a broken test to this issue ! Is there something missing for this PR to be merged ?

Is there something missing for this PR to be merged ?

@lovasoa, I don't know :(
If there is, I could do it if pointed to 🥺
Release notes are filled with GH Actions, I think, and there is no 'milestone' tracking to add it for the next patch
I suppose it will be merged soon 🤞
We are using fastAPI in production with 0.85.1, waiting for the fix to update it, after adapting the test suite with the TestClient change.
The only thing I would do different is to unpin the starlette dependency, at least to ^0.22.0, instead of setting a fixed version each time, but I can understand the reasons behind, so I just changed the version, maintaining the hard link ==

Great job @azogue! Thanks for adding a test for it! :bow:
And thanks everyone for the interactions, especially @iudeen that I see you reviewed it, and added a comment letting me know you checked the code and test :wink: :cookie:
This is available in FastAPI 0.88.0, just released. :rocket:

at least to ^0.22.0,

@azogue I would be in favor of that too, but you know that means >=0.22.0 && < 0.23.0, right? If it was previously set to ^0.21.0 instead of ==0.21.0, it would still need the update anyway, because 0.22.0 is not covered by ^0.21.0.


Re-rendering of datagrid rows in case new  row items are added

Describe the bug
Is adding a new item to the data grid is resulting in re-rendering of all the rows even when trackBy function returns a unique identifier?
How to reproduce
Here is the link to reproduce - https://stackblitz.com/edit/clarity-custom-theme-v1-0-7bndff

Open Chrome Inspect > Elements tab to observe the behavior.
Click on "Add Data" button which will add a new data item to the grid.
When a new item is added, already added "clr-dg-row" elements are highlighted (chrome behavior when a change is detected for DOM elements) as well.

Expected behavior
Chrome "Elements" tab should not highlight elements that are already rendered in this case.
Versions
App

Angular: 8.1.0 and 7.2.15
Node: 12.6.0
Clarity: 2.0.2 and 1.1.5

Device:

Type: MacBook
OS: 10.13.6
Browser Chrome
Version 72.0.3626.121
--

Additional notes
I observed it is working fine with an unordered list - https://stackblitz.com/edit/clarity-custom-theme-v1-0-aqh8hh

This is an expected behavior because while we support trackBy in our custom iterator clrDgItems we also rebuild the current list of items each time for a few technical reasons. There could be an optimization some day to figure out how to reduce this, but we have to do calculations for size and end up always reassigning the new list of items in the process to trigger change detection in all the proper places.
TrackBy in this case is primarily used to help us identify which rows are selected as you paginate, but we do internally reassign models which causes a redraw of each element. This isn't a bug IMO but rather a potential optimization (though I'm not certain this could be changed).

I am worried in case the model bound to Datagrid grows bigger this might lead to performance issues which would be bad.
Is there a workaround if I am using Datagrid? I noticed there is a similar problem if I use "ngFor" instead of "clrDgItems".

There is no workaround, this is internal. The performance issue is only when you change the rows, which if that becomes too large we recommend using pagination. Regardless if you are changing the rows or contents of the datagrid quickly, I would wonder if you are using the right component for the job.

Okay, thanks for your response.
Is there any better component or guidelines from Clarity if the requirement is to add new rows as we fetch it from a backend? Just looking for suggestions.

I would just profile your case and see if there is anything to worry about. You don't want 1000 rows on the page, you want a smaller number anyways because users can't make sense of that much content at once. Don't worry about premature optimization unless you are getting performance drag, and I suspect this won't be the source if you are having slow pages.

Closing as I believe the question has been addressed and it is acting as expected.


Question about the learning objective mentioned in paper

"The generative process is the same as in auto-regressive language models: generation begins with an empty string, and at the 𝑖-th step a token 𝑧𝑖 is sampled"
Since the generative process is conducted token by token, I'm wondering about what is the meaning of calculating a reward for an incomplete sentence in the learning objective? Thanks if you can help me understand this :)

Hi @StarDewXXX, to compute the intermediate reward (i.e. after each token) we append an EOS token to the tokens generated so far and then compute evaluate the reward.


Something like GraphQL subscriptions?

Hi guys. I'm (again, years later) new to JSONAPI. I'm coming from GraphQL which I've never really been in love with.
In GraphQL there is the subscription concept that the major clients (Apollo, URQL) have created with websockets.
Is there something like this in the JSONAPI specification?
Can I use something similar with Orbit?
I saw that there are PR with "live queries" but I can find anything in the docs.
Thanks in advance.

Hi @frederikhors - Very sorry for the extremely late reply. Yes, live queries are orbit's answer to subscriptions. We are close to shipping orbit v0.17 and will include docs for live queries when we do, since they are now a part of orbit itself (and not just ember-orbit.
In the mean time, if you're feeling adventurous, the live query code and tests are in the @orbit/record-cache project.
I'll leave this open as a reminder to document live queries for v0.17.

Live queries are documented; I did just open https://github.com/orbitjs/orbit/pull/909 for a spelling update but I think this can be closed.


Suppress NumPy warnings for special case behaviour

The initial numpy.array_api PR https://github.com/numpy/numpy/pull/18585 had folks happy with keeping NumPy-proper's warnings when spec-compliant operations were being done (usually/always special case behaviour). Giving the flexibility of a separate package, could we instead suppress these warnings? Really my one use case here is array-api-tests, where right now our "daily driver" array module is numpy.array_api which deluges us with warnings that obfuscate other things... we could suppress them on our end tbf, just ugh.

Most of the functions in this module are just direct copies of the NumPy versions, i.e., we just do from numpy import * and export np.sin, np.exp, etc. I'm only wrapping the functions that have changed behavior. Doing this would require wrapping every single function.
Additionally, we don't wrap or subclass the array object, so even if we fix it for functions, it wouldn't be fixed for operators.
Instead, I would just disable the warnings when testing this module. I don't know if it can be done with an environment variable.


[BUG] Tableau intercepts 401 replies to JDBC driver after OAuth authentication

Describe the bug
This specifically relates to the interaction between Trino and its JDBC driver.
Trino and the JDBC driver have a protocol between themselves, wherein at certain times Trino replies with a 401 and a WWW_AUTHENTICATE header. The header contains a URL that the JDBC driver is expected to query in order to get an updated authentication token. This interaction works fine in tools like DBeaver where we use the JDBC driver as well. However, with Tableau something is interfering with the above protocol and the URL returned by Trino is never queried by the driver. This leads me to believe that Tableau may be intercepting 401 replies and isn't forwarding them to the driver.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10 Home
Tableau Version: 2023.1.0 (20231.23.0310.1044)

About you:
Name: Anton
Company: Starburst Data

Internal tracking: W-13546955

Just saw this. We are not intercepting HTTP requests for the driver. Is this still an issue?

I have started a conversation about all of these issues in our shared slack channel. If this is still an issue let's discuss it there. Resolving.


test arm install

right now we only test that the x86 binaries install successfully. Can we do the same with arm builds?
will be trivial when https://github.com/actions/runner-images/issues/2187 is resolved

Done in #64


Bug in Generic_Elongation_Solvers

Summary
When you use a ConstrictedBoxGroove using the follwing definition:
 groove=ConstrictedBoxGroove(                         r1=1.805e-3,                         r2=6.859e-3,                         r4=31.5875e-3,                         depth=4.9565e-3,                         indent=0.361e-3,                         usable_width=24.2e-3,                         ground_width=17.5e-3,                     ),                     nominal_radius=160e-3,                     rotational_frequency=1,                 ),                 gap=2e-3,
the solver yields a negative usable_width resulting in an Error of the mandatory_positive_or_zero list.
PyRolL Version and Platform
pyroll-core 2.1.3

Can you provide a drawing for this? I tried to draw the groove with your values in CAD and it seems to be geometrically impossible to me. So I suspect no bug, but invalid input...

Seem's that I used the wrong dimensions. No Bug!


Add Aorus Xtreme

Could we get Aorus Xtreme added to search?

Could we get Aorus Xtreme added to search?

What sites have the Xtreme up?

I saw it on listed www.alzashop.com, but I just noticed they haven't had any of the other cards price listed.


is this a joke?

Name one programming language that has a different interface for filter.
That is because filter by definition returns a subset of elements of the given data structure, for which a given predicate returns true.

Your tone reads as hostile, please remember the code of conduct.

Name one programming language that has a different interface for filter.

https://github.com/jridgewell/proposal-array-select-reject#related

That is because filter by definition returns a subset of elements of the given data structure, for which a given predicate returns true.

I'm not proposing we change filter. I'm saying its naming is problematic, because when I read array.filter(), I think it's filtering out the things that return true.

I apologize, but I really wasn’t sure if it was a joke.
But you cleared that up.
Let me get this straight, you’re proposing a name change of a method whose name and interface do not stem from the language in question, but from the functional paradigm itself - it’s ‘filtering in’, as you call it, in every other functional language (mind Ruby) -because You find it confusing?

you’re proposing a name change of a method

I'm proposing an alias. Array.p.filter won't change, this only adds Array.p.select which does the same thing. They'd literally be the same function instance: Array.prototype.filter === Array.prototype.select.

..in every other functional language (mind Ruby) -because You find it confusing?

I'm proposing it because there was some agreement on the TC39 IRC. Judging by the number for project stars (it's been less than 48 hours), I'm not the only one who finds it confusing.

Hum sorry to tell this, but this looks bad to me: introducing new methods for... nothing. I mean 'filter' is clear enough. You're adding a new layer of complexity...


Support for InfluxDB 0.9.x

Just a question, is InfluxDB 0.9 already supported? The HTTP API has changed significantly in 0.9.

Hi Matthias,
We haven't implemented compatibility with InfluxDB 0.9.x yet, we'll do it once this branch is released as stable.

Hi Marc, thanks for the info. Looking forward to have support for InfluxDB 0.9.x in facette. Btw, InfluxDB 0.9.0 will probably be released soon. Maybe it is worth thinking about how to support 0.8 and 0.9 in parallel? Or are you planning to drop support for 0.8 once 0.9 has been released?

Matthias
At the moment we haven't decided whether we'll support both 0.8.x and 0.9.x – my first guess would be to do so, since it's been a long time between 0.8.x and 0.9.x branches and many people would have deployed InfluxDB in the mean time, so there might be a significant user base for 0.8.x.
Cheers

FYI: Grafana already supports InfluxDB 0.9 as a datasource (as well as InfluxDB 0.8).

FYI: InfluxDB 0.9.0 has just been released today.

Hi Matthias,
We plan to implement the new support soon. But as the API and the client library changed, we will need to figure a proper way to offer access to both 0.8 and 0.9.

A new connector for InfluxDB 0.9 is on its way, see influxdb-0.9.
The old connector has been renamed to influxdb_legacy.

Were are currently waiting for a upstream issue to be fixed:
https://github.com/influxdb/influxdb/issues/3326
We will update this issue when we will be able to finalize its support.

This issue has been closed in the 0.9.3 release of InfluxDB :-)


beta6 to rc2 - scope problem

I have code like this:
  <ion-side-menus>
    <ion-side-menu-content>
        <ion-nav-bar class="bar-dark bar-header">
          <ion-nav-buttons side="left">
            <button class="button button-icon button-clear ion-navicon" menu-toggle="left">
            </button>
          </ion-nav-buttons>
          <ion-nav-buttons side="right" class="no-top" ng-if="user">
            <button class="icon-right ion-plus button">
              {{user.balance | currency }}
            </button>
          </ion-nav-buttons>
        </ion-nav-bar>
        <ion-nav-view animation="slide-left-right" name="menu-content"></ion-nav-view>

    </ion-side-menu-content>
    <ion-side-menu side="left">
          <ion-header-bar class="bar-positive" href="#/fantasy/home">
          </ion-header-bar>
          <ion-content class="has-header">
              <ion-list>
                <ion-item nav-clear menu-close href="#/home">
                  Home
                </ion-item>
                <ion-item nav-clear menu-close href="#/login" ng-if="!user">
                  Login
                </ion-item>
                <ion-item nav-clear menu-close href="#/games/all">
                  View All Games
                </ion-item>
                <ion-item nav-clear menu-close href="#/games/my">
                  My Games
                </ion-item>
                <ion-item nav-clear menu-close href="#/logout" ng-if="user">
                  Logout
                </ion-item>
              </ion-list>
            </ion-content>
    </ion-side-menu>
  </ion-side-menus>

Line:
          <ion-nav-buttons side="right" class="no-top" ng-if="user">
not works.
Line:
  <ion-item nav-clear menu-close href="#/logout" ng-if="user">
works.
In beta6 both line works. In rc2 - first line not working.
{{user.balance}} - works when removing if statement.
Where to dig?

Greetings @tigrus!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Looks like trouble in rc2 with this line:

I am having the same problem.

Me too - looks like ion-nav-buttons copies $scope and not updates it

Here is my bug: http://codepen.io/piernik/pen/NPQqgG
Click Go to list then first - title is default Item. Then click second title changes to Long first title
There is a bug I think

Directives copy their parent's scope in the OOP fashion where once they're instantiated they don't automatically update. Does using $parent.user help?

@perrygovier no. Only remove of ng-if helps. ng-if="true" also not working.

@perrygovier have You looked at my codepen? There is bug for sure. Look like navbar is refreshing scope too early - when scope isn't set yet.
I think @tigrus has the same problem.

@piernik  I don't think there's a bug in ionic,It all works for me here: http://codepen.io/Yuripetusko/pen/MYNVxz

@Yuripetusko even in You codepen it's not working well.
Click Go to list then first - title is ok
then second title is ok
then first again - title is wrong
navigate a little - title is not always changing as it should

@piernik  Yes you are right! I can see it now. I updated code a little bit so it's cleaner for ionic developers to see the issue. http://codepen.io/Yuripetusko/pen/MYNVxz
Basically if you go to list, then click first item, then click second item, and then click first one again the title doesn't update even though scope value is definitely updated

@tigrus As for the showing and hiding a button in the nav-bar, can you fork this codepen and replicate the issue: http://codepen.io/adamdbradley/pen/azereK?editors=101

@adamdbradley have You looked at mine and @Yuripetusko's examples? (http://codepen.io/piernik/pen/NPQqgG and http://codepen.io/Yuripetusko/pen/MYNVxz)
You have there bug on a plate.
I think problem with navbar when changing to the same state but with different params.

@piernik looking at your codepen, it should't even work.
At first glance, there shouldn't be a nested ion-nav-view in list.html. This could be the main cause of confusion with in the apps history logic. There could be a much simpler way of doing what you are trying to do.

@mhartington nested view are common thing in ui-router.
This is only an exmaple to isolate bug.
In real app I need to use nested view because of different data handling - data which is downloaded by app.item's resolve and then use different views to handle this data (ex. app.item.details, app.item.gallery, app.item.map). That way I don't have to resolve data for every view  - data is simply the same - I also don't have to work on difficult data cache.
Off course I would be able to do it differently (without nested view) but:

I didn't see in Your docs that I shouldn't use nested view
Only navbar is problem - everything works fine
Ad I said in ui-router nested view this in common thing.

Now is my question - should I rewrite my app or You can fix the problem?

Right, nesting views is common with ui-router, but for a mobile app paradigm it's not really advised unless you're making an abstract state. ion-nav-view is a container of many views that slide in and out, and take over the entire screen.
You should probably rewrite the structure and not use a nested ion-nav-view

I see that in router I can't use controller with abstract state. why? I thinks it's possible in ui-router
Here is my code
.state('app.item', {
	url: "/item/:id",
	abstract: true,
	views: {
		'page@app': {
			controller: "itemCtrl as vm"
		}
	},
	resolve: {
		itemData: ['psDane', '$stateParams', function (psDane, $stateParams) {
			return psDane.pobierzDaneAtrakcji($stateParams);
		}]
	}
})
.state('app.item.info', {
	url: "/info",
	views: {
		'page@app': {
			templateUrl: "app/atrakcja/atrakcja-info.html"
		}
	}
})
.state('app.item.map', {
	url: "/map",
	views: {
		'page@app': {
			templateUrl: "app/atrakcja/atrakcja-mapa.html",
			controller: "atrakcjaMapaCtrl as vm"
		}
	}
})

app.item state should be only for common controller for both child states but it is never triggered when going to app.item.info
Controller without view is not working either:
.state('app.item', {
	url: "/item/:id",
	abstract: true,
        controller: "itemCtrl as vm",
	resolve: {
		itemData: ['psDane', '$stateParams', function (psDane, $stateParams) {
			return psDane.pobierzDaneAtrakcji($stateParams);
		}]
	}
})

You can use controllers on abstract states. @piernik please post about this in the forum, it's not really relevant to the issue.

Played with codepen.
Working:
http://codepen.io/anon/pen/zGxYGz
Not working:
http://codepen.io/anon/pen/NqPWNP
If we move ng-if to ion-nav-buttons - it's ignored. If it is inside ion-nav-buttons - it works.
I think, ionic must throw warning about ion-nav-buttons directive.


Developers Package for V19

I'm doing some customizations and the last of the AzureCP.Developers.zip I can find is for version 17, June 2020. Is there a way I can build or source the latest AzureCP parent library to include in my project based on the 17 dev solution? Should extracting the .dll from the latest wsp work?
If not could you please add the developer package for version 19 or include in the upcoming 20 release?
Thank you!

Excellent, thank you!