Modify checksum API

I'm not happy with the current API for the IP checksum implementation.  I have discussed the issues in the thread https://groups.google.com/forum/#!topic/snabb-devel/nlifqWTDqzc
This PR contains the following proposed changes:
The type of the initial argument is changed to uint16_t throughout and its endianness is now expected to be the same as the output of ipsum(). The old code used endian conversions in an asymmetrical manner, which I find very awkward.
The code for cksum_generic() is cleaned up a bit.
The documentation in the code has been extended to describe the full semantics of ipsum(). In particular, the output of ipsum() was previously described as being in network byte order, when, in fact, it is in host byte order, i.e. an endian conversion is required to write the checksum to memory in network byte order.
I think that the convention that the result of ipsum() and the initial argument are in host byte order is the natural choice. Conversions have to be performed when a checksum is read from or written to a packet buffer.

This API change makes sense to me.
The code looks nice too. I like the way you simplified the generic checksum routine. Is there any meaningful performance impact on that? If not then great. But we should remove the comment at the top of the file saying that the generic checksum is based on the DPDK one (http://dpdk.org/browse/dpdk/tree/lib/librte_net/rte_ip.h) because it will not be anymore.

The code looks nice too. I like the way you simplified the generic checksum routine. Is there any meaningful performance impact on that?

I didn't check, but I don't expect a difference, because of the inlining in the old code.

But we should remove the comment at the top of the file saying that the generic checksum is based on the DPDK one (http://dpdk.org/browse/dpdk/tree/lib/librte_net/rte_ip.h) because it will not be anymore.

Ok.

On first reading it sounded to me like we are saying that the caller needs to pad the input with a zero-byte if it is odd-length. Could be worth emphasising that this is not required?

Maybe it shouldn't be mentioned at all, since it's part of the algorithm.

Tony Rogvall who wrote the original SIMD checksum had an interesting idea for an API tweak for optimization btw. The idea is to actually insert zeros into the area following the payload so that no loop is needed to take care of the unaligned part at the end.

But how would one make sure that there is room for the padding bytes after the data?

But how would one make sure that there is room for the padding bytes after the data?

This seems doable in the special case that we are checksumming a struct packet: we could define a system-wide maximum packet data size and ensure there is some excess in the size of the struct packet.data array. Then every packet always has trailing padding. This would add complexity though so we wouldn't want to do it unless there was a meaningful payoff.

This change is merged onto the next branch now so I close this PR.
I made a small change to the comment in the merge (cc0ebb9). If you would like to rephrase that you can just push a new commit on this branch and reopen the PR.


create-proton-app fails to create app

Environment: Linux Ubuntu 18.04 on Surface Book 2 (Custom kernel, based on 4.17), Node 10.8, fresh update/upgrade. libgtk-3-dev  and build-essential are in place. Python3 is in place.
Trying to create my app folder, but got the following errors (sorry, long!):
> node-gyp configure build

gyp

ERR! configure error

gyp
 ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp
ERR! stack     at PythonFinder.failNoPython (/home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:492:19)
gyp ERR! stack
     at PythonFinder.<anonymous> (/home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:406:16)
gyp ERR! stack     at F (/home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/which/which.js:68:16)
gyp
 ERR! stack     at E (/home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/which/which.js:80:29)
gyp ERR!
 stack     at /home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/which/which.js:89:16
gyp ERR! stack
     at /home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/isexe/index.js:42:5
gyp ERR! stack     at /home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/isexe/mode.js:8:5
gyp
 ERR! stack     at FSReqWrap.oncomplete (fs.js:152:21)

gyp
 ERR! System Linux 4.17.3-surface-linux-surface

gyp
 ERR! command "/home/patrick/.nvm/versions/node/v10.8.0/bin/node" "/home/patrick/.nvm/versions/node/v10.8.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "configure" "build"

gyp ERR! cwd /home/patrick/mockingbird/node_modules/libui-node

gyp ERR! node -v v10.8.0
gyp
 ERR! node-gyp -v v3.7.0
gyp
 ERR! not ok

npm ERR!
code ELIFECYCLE

npm
 ERR! errno 1

npm ERR<EMAIL_ADDRESS>build: `node-gyp configure build`
npm
 ERR! Exit status 1
npm
ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm
 ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm WARN
 Local package.json exists, but node_modules missing, did you mean to install?

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/patrick/.npm/_logs/2018-09-20T00_10_22_092Z-debug.log

npm
WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN
 notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

npm
 ERR! code ELIFECYCLE

npm
 ERR! errno 1

npm ERR<EMAIL_ADDRESS>install: `libui-download && autogypi && npm run build`
npm ERR! Exit status 1

npm ERR!
npm ERR!
 Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm
 ERR!

Any idea on how to fix it?
Thanks in advance.

whereis python ==>
python: /usr/bin/python3.6m /usr/bin/python3.6 /usr/lib/python2.7 /usr/lib/python3.6 /usr/lib/python3.7 /etc/python2.7 /etc/python3.6 /usr/local/lib/python3.6 /usr/include/python3.6m /usr/share/python

I don't see a python 2.7 binary. You have the library path, but there's nothing in bin. Something is up with your distro.

Ok, I'll get into that a bit later. Thx a lot for your help!
On Wed, Sep 19, 2018, 8:37 PM Gustav Hansen<EMAIL_ADDRESS>wrote:

I don't see a python 2.7 binary. You have the library path, but there's
nothing in bin. Something is up with your distro.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kusti8/proton-native/issues/177#issuecomment-423001697,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AONBMj0Yr7aPLdvBLQ7ZiXKAdSv3snyeks5ucuM6gaJpZM4WxLBc
.


Add support for Windows

Updated version of https://github.com/conda-forge/ampl-mp-feedstock/pull/13, work in progress.
Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

The Windows build is failing with:
CMakeFiles/nl-example.dir/src/nl-example.cc.obj
nl-example.cc.obj : error LNK2019: unresolved external symbol "public: static unsigned int const * const fmt::internal::BasicData<void>::POWERS_OF_10_32" (?POWERS_OF_10_32@?$BasicData@X@internal@fmt@@2QBIB) referenced in function "unsigned int __cdecl fmt::internal::count_digits(unsigned int)" (?count_digits@internal@fmt@@YAII@Z)
nl-example.cc.obj : error LNK2019: unresolved external symbol "public: static unsigned __int64 const * const fmt::internal::BasicData<void>::POWERS_OF_10_64" (?POWERS_OF_10_64@?$BasicData@X@internal@fmt@@2QB_KB) referenced in function "unsigned int __cdecl fmt::internal::count_digits(unsigned __int64)" (?count_digits@internal@fmt@@YAI_K@Z)
nl-example.cc.obj : error LNK2019: unresolved external symbol "public: static char const * const fmt::internal::BasicData<void>::DIGITS" (?DIGITS@?$BasicData@X@internal@fmt@@2QBDB) referenced in function "void __cdecl fmt::internal::format_decimal<unsigned int,char>(char *,unsigned int,unsigned int)" (??$format_decimal@ID@internal@fmt@@YAXPEADII@Z)
nl-example.cc.obj : error LNK2019: unresolved external symbol "private: static class mp::internal::ExprInfo const * const mp::internal::ExprInfo::INFO" (?INFO@ExprInfo@internal@mp@@0QBV123@B) referenced in function "int __cdecl mp::expr::nl_opcode(enum mp::expr::Kind)" (?nl_opcode@expr@mp@@YAHW4Kind@12@@Z)
nl-example.cc.obj : error LNK2019: unresolved external symbol "private: static class mp::internal::OpCodeInfo const * const mp::internal::OpCodeInfo::INFO" (?INFO@OpCodeInfo@internal@mp@@0QBV123@B) referenced in function "class mp::internal::OpCodeInfo const & __cdecl mp::internal::GetOpCodeInfo(int)" (?GetOpCodeInfo@internal@mp@@YAAEBVOpCodeInfo@12@H@Z)
bin\nl-example.exe : fatal error LNK1120: 5 unresolved externals

It seems that ampl-mp has a vendored rather old version of the fmt library, as I guess that at the time the fmt main developer was working on AMPL.

It seems that ampl-mp has a vendored rather old version of the fmt library, as I guess that at the time the fmt main developer was working on AMPL.

I tried to tackle this, but it is not trivial to do so, see the issue that I opened upstream in https://github.com/ampl/mp/issues/131 .

Closing as I do not know how to solve this properly without tackling https://github.com/ampl/mp/issues/131, that would require some time.


Is there no landing page at all?

The official website qards.io does have a landing page. However, my blog is just a Not Found page. I tried to figure out if there is any way to get a landing page but I failed. So I come to ask if currently there is no landing page at all and not-found is the expected behaviour. If this is by design, a not-found page as a default landing page is just too...unfriendly, you know.
I understand this project is in early development and something may not be functional, so I have to come here and ask. Sorry for the trouble.

No trouble at all. There is a landing page in app. You just have to rename it. https://github.com/QardsJs/qards/tree/master/src/app/pages

Thanks. It's working now.


Theseus disturbs Brackets' "Highlight selected HTML item"

Steps to repro:

Open Getting Started\index.html
Start Live Preview
Set the cursor to the <img> tag
Switch back to the browser and look at the image (screenshot)

Result:
With (only) Theseus installed, the image looks just like before. Without Theseus (same settings), it has a blue highlighting.
CSS rules are highlighted correctly.
Env: Brackets development build 33 (but as far as I can remember this also in sprints before)
Windows 8.1

It still happens to me, did you get it work? I'm trying to supervive with a little trick: I've installed the "HTML Designer for Brackets" extension, and while you can debug the JS code in Live Preview, you can see and edit the HTML design in the DesignView or SplitView options from this extension...

No, sorry, I haven't developed this yet. :(


feat: show expose service with port and name in envs subcommand

Description
The expose service should be displayed for users to easily knows how to use them.

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

envd envs describe --env envd-quick-start
DEPENDENCY      TYPE
...,
....
SERVICENAME PORT HOSTPORT
nginx                80     8080

Looks promising.

@kemingy 圆梦大使感恩


As an editor I need to be able to add anchor points to pages and name/show these points in a menu so users can navigate the content on the page.

Acceptance criteria

Anchor point needs to feature in a URL too, so editors can point a user directly to that piece of content in the URL
If anchor links are added to the page by editor, an additional menu appears on the page for users to jump to the points of the page where the anchor(s) is added
Anchor link menu points should be highlighted/animated as user scrolls through/past each anchor point.
Anchor links menu hides when user scrolls up (short delay)
Investigate/discuss: Should anchor link menu also include links to other pages of interest or be restricted to anchor links for that page? Currently AEM is set up this way (https://www.equinor.com/en/careers.html)

In order to ensure good quality we should

[ ] Perform functional testing
[ ] Extra focus on usability
[ ] Test for portability
[ ] Test for accessibility
[ ] Create automated acceptance test/smoke test

Perhaps stick to anchor links within the page itself. Otherwise we might get ourselves into trouble 😱

Closing as this is a duplicate. Defined on #346


Add term and year to course titles in collection UI to disambiguate courses with the same title

Related to #391
We should disambiguate courses so we don't see duplicate listings, perhaps by adding a date to the title:

The OCW team is used to disambiguating courses using the term and the year.

Do we want to use short ids here instead of course+year, like in https://github.com/mitodl/ocw-studio/issues/1265 ?

Good idea. Will it fit?

I think so; we'll see. @pdpinch did you mean to move this out of to-do?

I did, just because I want to focus on publishing blockers today (Friday).
It’s OK to work on this, it’s just low priority.

On May 20, 2022, at 8:59 AM, Chris Chudzicki @.***> wrote:
I think so; we'll see. @pdpinch https://github.com/pdpinch did you mean to move this out of to-do?
—
Reply to this email directly, view it on GitHub https://github.com/mitodl/ocw-studio/issues/393#issuecomment-1132873078, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADJALRTO2KTNX2GBYQ33MTVK6EEXANCNFSM5AQGOIAA.
You are receiving this because you were mentioned.

@ChristopherChudzicki Are you working on this?

I am not. Thanks for checking... I'll unassign myself from this and other stuff for the moment.

I've been reviewing @MAbdurrehman12 PR for this (https://github.com/mitodl/ocw-studio/pull/1375). This turns out to be kind of hard to do well in OCW right now. Right now, it's feasible to get the short id into the "select" dropdown (since that's populated by OCW studio), but rather hard to get them in the sortable list:

The sortable area is populated by course list resource metadata. Fundamentally, I think our course lists aren't storing the right data.
WebsiteContent course list metadata look like this:
content_type: course-lists
courses:
- id: courses/11-165j-urban-energy-systems-and-policy-fall-2021
  title: Urban Energy Systems and Policy
- id: courses/4-401-environmental-technologies-in-buildings-fall-2018
  title: Environmental Technologies in Buildings
# ...

id is really website.url_path. This is used by Hugo to fetch website data from the static data.json files. (E.g., for making course cards in course lists).
-It is only used as a label for the items in OCW Studio. Theoretically hugo could use this, but it does not. It's also not great...if someone changes website Title, the value here will be out of date.

Ideally, it seems to me that the course lists would provide an identifier that Hugo can use (which we have url_path) and either an identifier Studio can use to fetch the relevant information (e.g., website.uuid) or the studio API could return extra information when fetching a course list.

@ChristopherChudzicki @pdpinch FYI: Had a call with @HussainTaj-arbisoft, we discussed the ticket and the work going on in the PR along with the issues in it. I'm gonna this ticket to Hussain and he can continue working on it from here.

https://github.com/mitodl/ocw-studio/pull/1715 will be reverted because short_id is not showing up on search results.

@HussainTaj-arbisoft I'm confused. Can you summarize what happened with #1375 and #1721 ?
What to do you think we need to do to get this to work? If it helps, it's not strictly necessary that we use the short_id -- we just need to display some data to differentiate courses with similar names. A term and year would be just as good. Even a year would help.

@pdpinch

Can you summarize what happened with #1375 and #1721 ?

The approach in https://github.com/mitodl/ocw-studio/pull/1375 made it challenging to sync two lists in the code. The details are in this comment https://github.com/mitodl/ocw-studio/pull/1375#discussion_r1124661620
https://github.com/mitodl/ocw-studio/pull/1721 was a rewrite of the same solution with a different approach. When I tested it on RC the short_id did not appear in the search results. It was my bad, I probably missed the part that renders the search results.

What to do you think we need to do to get this to work?

I'll take a look at this again next week. I'm hoping there will be a small change to https://github.com/mitodl/ocw-studio/pull/1721 and that should make it work.


Use TinEye instead of Google.

TinEye would be a preferred option to Google for reverse image searching, perhaps add a setting for choosing your search engine of choice, especially since I would rather that my search data not be directly collected by Google themselves.
Link to TinEye

My pull request adds a sub-menu, and has TinEye as an option. #1

Added by @ezist in #1


Profile with no pictures to be deleted in xx days

How can be add a function to auto delete a profile without a picture in xx days?
Some people create profiles but not upload pictures.
Cheers

The best would be to make the "Profile Photo" field required during the registration (Admin -> Setting -> General -> Registration tab, -> "Require photo to be uploaded". However, some users leave the registration process after the step one but don't go until step 4, so you might still have some users without a profile photo.

What you asked is doable, but it requires work! You will have to create a new cron job that runs every day for instance to delete all users without profile photos. You will have to ask it to a good PHP programmer (Toptal for example), or a very good one on UpWork, but I've heart that it's harder to find good ones there).


VM recreation with persistent disk failing after 49.0.0

Hey,
after the latest update we see errors by the recreation of vms.:
Task 1141468 | 07:57:30 | L executing post-stop: mongodb/434ab87d-62bc-4ea2-9ff4-a146d22fbc4a (1) (00:02:01)                         L Error: CPI error 'Bosh::Clouds::CloudError' with message 'update i-uf67phqg6lzvan0x2qlx failed and the vm has been deleted.: associate Persistent Disk error diskCid={{d-uf6gig6vlhx7hoxnrvde}}: SDK.ServerError ErrorCode: IncorrectInstanceStatus Recommend: https://api.aliyun.com/troubleshoot?q=IncorrectInstanceStatus&product=Ecs&requestId=15E2FBB4-41AF-5B2F-A41D-27D982DDB099 RequestId: 15E2FBB4-41AF-5B2F-A41D-27D982DDB099 Message: The current status of the resource does not support this operation.' in 'create_vm' CPI method (CPI request ID: 'cpi-773522') Task 1141468 | 07:59:22 | Error: CPI error 'Bosh::Clouds::CloudError' with message 'update i-uf67phqg6lzvan0x2qlx failed and the vm has been deleted.: associate Persistent Disk error diskCid={{d-uf6gig6vlhx7hoxnrvde}}: SDK.ServerError ErrorCode: IncorrectInstanceStatus Recommend: https://api.aliyun.com/troubleshoot?q=IncorrectInstanceStatus&product=Ecs&requestId=15E2FBB4-41AF-5B2F-A41D-27D982DDB099 RequestId: 15E2FBB4-41AF-5B2F-A41D-27D982DDB099 Message: The current status of the resource does not support this operation.' in 'create_vm' CPI method (CPI request ID: 'cpi-773522')
Can you please support us to fix this issue?

HI. @max-soe I have published a new release 50.0.0 to fix it. Please update it and try again.

Hi @xiaozhu36,
we will test the release and provide feedback


#1600 - Resolve issues with missing ./configure messages

Issue #1600

Codecov Report

Merging #1617 into master will decrease coverage by 1.22%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1617      +/-   ##
==========================================
- Coverage   70.48%   69.26%   -1.23%
==========================================
  Files         156      159       +3
  Lines       19028    20259    +1231
  Branches     2321     2487     +166
==========================================
+ Hits        13412    14032     +620
- Misses       4768     5324     +556
- Partials      848      903      +55

Impacted Files
Coverage Δ

src/ncp/ncp_base.cpp
34.86% <0%> (-12.98%)
:arrow_down:

src/core/thread/mesh_forwarder.cpp
78.21% <0%> (-0.49%)
:arrow_down:

src/core/thread/mle_router.cpp
77.61% <0%> (-0.33%)
:arrow_down:

src/ncp/hdlc.cpp
76.34% <0%> (ø)

src/ncp/ncp_base.hpp
0% <0%> (ø)

src/ncp/ncp_uart.cpp
71.6% <0%> (ø)

src/core/thread/network_diagnostic.cpp
4.86% <0%> (+0.88%)
:arrow_up:

src/core/thread/topology.hpp
90.24% <0%> (+1.21%)
:arrow_up:

src/core/thread/lowpan.cpp
93.9% <0%> (+1.43%)
:arrow_up:

src/core/thread/mle.hpp
94.91% <0%> (+1.69%)
:arrow_up:

... and 14 more

Continue to review full report at Codecov.

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

Thanks for addressing this issue!  See my comment above.

Done.


Release 1.9 blog post

First draft of release 1.9 blog post.

I will continue working on this on Wednesday,

@MarianRaphael This is ready for your review and approval.

Worth mentioning: HA is no longer in preview mode

Worth mentioning: HA is no longer in preview mode

Are we including HA in all tiers?

sumitshinde-84 made their first contribution in #2291 biancode made their first contribution in #2327

I added both these.

Worth mentioning: HA is no longer in preview mode
Are we including HA in all tiers?

Currently, we just have one tier in FF-Cloud so yes. It's still an ee feature.


Add a gRPC endpoint for fetching consensus state heights

Summary
The ibc-rs codebase would benefit from a gRPC endpoint that only fetches consensus state heights for a given client.
Problem Definition
At the moment, height information is extracted after querying for all consensus states via the QueryConsensusStatesRequest query, which is suboptimal. It would be nice to be able to query just for consensus state heights so that the extra step of extracting all of the heights can be eliminated.
Proposal
Add a gRPC endpoint (perhaps something like QueryConsensusHeightsRequest) that returns all consensus state heights for a given client.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Thanks for the rationale, @adizere. :)

Would a gRPC with the following request/response messages satisfy the requirements?

Yes, thank you!

The heights in the response should be in decreasing order.

This would be great. We usually sort the heights in descending order after we fetch them, so descending order is the semantic we'd like to achieve.
Note that the protobuf type PageRequest has already a reverse option. Our typical use-case is that we either want:

all consensus heights of a client, or
the last X (in descending order) consensus heights installed for a client.

For the first case we use PageRequest parametrized with limit: u64::MAX. For the second case, we would expect to use limit: X. In both cases we set reverse: true.

Thanks for the feedback, @adizere. So if I understand you correctly, the heights should be in descending order if reverse: true? Otherwise, we should return them in increasing order?

So if I understand you correctly, the heights should be in descending order if reverse: true?

Yes.
The new request should probably adhere to the same semantics as the others requests. The default ordering semantics are "ascending order". On the client side (i.e., relayer side), we will probably set reverse: true to pull the heights in descending order (or we will sort the heights in the response entirely on the client side, alternatively).

Otherwise, we should return them in increasing order?
Yep, I think this is the default that clients would expect, so I think it makes sense to retain the same expectations with this new request.

Anca pointed out that the default ordering in gRPC responses works on lexicographic value of Height, not on their numerical value.

Here is an example (trimmed from an actual query). If the chain has states for heights:
830, 821, 766, 222, 104, 76, 30, 25, 20, 10
The gRPC query returns:
10, 104, 20, 222, 25, 30, 76, 766, 8, 821, 830

Shall I open a separate issue for this? Or will this PR have the correct ordering implemented (i.e., adhering to numerical, not lexicographical order)?

Shall I open a separate issue for this? Or will this PR have the correct ordering implemented (i.e., adhering to numerical, not lexicographical order)?

@catShaark, can you please check this in #1336 and fix it and add a test if needed?

Shall I open a separate issue for this? Or will this PR have the correct ordering implemented (i.e., adhering to numerical, not lexicographical order)?

@catShaark, can you please check this in #1336 and fix it and add a test if needed?

I think this belong to the sdk. I can't find any ways to do numerical ordering as the store iterator can only iterate in lexicographical order.
Should I open an issue about this in the sdk repo ?

@crodriguezvega , I've opened an issue to propose my solution

This is now available in v3.1.0, v3.0.1, v2.3.0, v2.2.1, v2.1.1, v1.5.0, v1.4.1 and v1.3.1.


"Embrace the extensions mindset..." link leads to nowhere.

https://blogs.msdn.microsoft.com/axinthefield/embrace-the-extensions-mindset-with-dynamics-365-for-finance-and-operations/

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

ID: b7526dd1-39d1-a882-8560-52c530ef5649
Version Independent ID: 7dd2a62b-b310-595b-eeda-f746679bbf3c
Content: Extensibility home page - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/extensibility/extensibility-home-page.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @FrankDahl
Microsoft Alias: fdahl

@RobinARH

Fixed, and should go live tonight.


Add ioBroker.pixelit to latest

RE-CHECK!

Automated adapter checker
ioBroker.pixelit

:thumbsup: No errors found

[ ] :eyes: [W400] Cannot find "pixelit" in latest repository

Add comment "RE-CHECK!" to start check anew

Hi, some review comments:

https://github.com/o0shojo0o/ioBroker.pixelit/blob/master/io-package.json#L144 ... there is no js.controller 2.4 :-) I think you mean >= 2.0.0
normally you check for ack=true to decide if the adapter needs to react on a change (https://github.com/o0shojo0o/ioBroker.pixelit/blob/master/main.js#L98)

Rest is fine, please check these two things.

Thanks for your reviewe.

Hi, some review comments:

https://github.com/o0shojo0o/ioBroker.pixelit/blob/master/io-package.json#L144 ... there is no js.controller 2.4 :-) I think you mean >= 2.0.0

Oh yeah, that's what I mean ;) (fixed)

normally you check for ack=true to decide if the adapter needs to react on a change (https://github.com/o0shojo0o/ioBroker.pixelit/blob/master/main.js#L98)

Rest is fine, please check these two things.

In this case the ack=true has been set by the adapter itself after it has successfully sent the input to the API.
Therefore the event is not to be processed further by the adapter.

In this case the ack=true has been set by the adapter itself after it has successfully sent the input to the API.

Yes but what you do if a user sets ack=true? Then this should also not go out, but would because from then is not from your adapter

Yes but what you do if a user sets ack=true? Then this should also not go out, but would because from then is not from your adapter

I think I have understood it now... ;)

@o0shojo0o ready for another check?

@Apollon77 yes we can ;o)

Then welcome


Quantization: int8

Codecov Report
Attention: Patch coverage is 22.82282% with 257 lines in your changes are missing coverage. Please review.

Project coverage is 73.15%. Comparing base (70ebd3f) to head (4dcd42b).

Files
Patch %
Lines

...emb/infinity_emb/transformer/quantization/quant.py
20.00%
252 Missing :warning:

...y_emb/transformer/embedder/sentence_transformer.py
28.57%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #134       +/-   ##
===========================================
- Coverage   85.57%   73.15%   -12.43%
===========================================
  Files          28       29        +1
  Lines        1324     1650      +326
===========================================
+ Hits         1133     1207       +74
- Misses        191      443      +252

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix build on openSUSE.

Same as https://github.com/hyprwm/hyprland-plugins/pull/64 but for this plugin.

LGTM, thanks!


API Implementation

Is this actually implemented already somewhere?
I was trying to use this with https://github.com/DrDelay/pokemon-go but couldn't really figure out how to properly do it all. Are you working on something, or is that private?
Otherwise might be an idea to combine? /cc @DrDelay
This wat I was working on, based on https://github.com/ctrlaltdylan/pgo-php/blob/master/app/Models/Http/Client.php and https://github.com/tejado/pgoapi)
use POGOProtos\Networking\Envelopes\RequestEnvelope;
use POGOProtos\Networking\Envelopes\RequestEnvelope_AuthInfo;
use POGOProtos\Networking\Envelopes\RequestEnvelope_AuthInfo_JWT;
use POGOProtos\Networking\Envelopes\ResponseEnvelope;
use POGOProtos\Networking\Requests\Messages\GetPlayerMessage;
use POGOProtos\Networking\Requests\Request;
use POGOProtos\Networking\Requests\RequestType;

$requests = [];

$req = new Request();
$req->setRequestType(RequestType::GET_PLAYER);
$requests[] = $req;

$message = new DownloadSettingsMessage();
$message->setHash('4a2e9bc330dae60e7b74fc85b98868ab4700802e');
$req = new Request();
$req->setRequestType(RequestType::DOWNLOAD_SETTINGS);
$req->setRequestMessage($message->toProtobuf());
$requests[] = $req;

$token = new RequestEnvelope_AuthInfo_JWT();
$token->setContents($this->accessToken);
$token->setUnknown2(59);

$authInfo = new RequestEnvelope_AuthInfo();
$authInfo->setProvider('google');
$authInfo->setToken($token);

$envelope = new RequestEnvelope();
$envelope->addAllRequests($requests);
$envelope->setAuthInfo($authInfo);
$envelope->setRequestId(8145806132888207460);
$envelope->setStatusCode(2);
$envelope->setLatitude(51.123);
$envelope->setLongitude(5.123);
$envelope->setAltitude(0);

$client = $this->container->get(GuzzleClient::class);

$response = $client->request('POST', $this->apiUrl, ['body' => $envelope->toProtobuf()]);
$responseEnvelope = new ResponseEnvelope((string) $response->getBody());

But response seems to be just 'd"', not an actual proper something, or even an error message..
(Also saw that you already fixed the use ProtobufIO which I also ran into :))

I am working on a PHP API that uses this, I managed to get the API endpoint, but it requires some modifications to these classes. I will put up some code in https://github.com/jaspervdm/pogoapi-php that works upto now

I committed my progress to https://github.com/jaspervdm/pogoapi-php , I would be very open to a collaboration to develop a full API!

Looks very promising, I didn't deal with the whole Protobuf API communication at all until now, but will definitely try to understand and get some basic calls to work with this repo tomorrow. :+1:

I tried to integrate your protos into my project (see DrDelay/pokemon-go@bfd6f82997f4aaabf644701d2264dd08cf36f4c3), but am also only getting the 'd"' as response. Did anyone here solve that already :question:

Using pogoapi-php, I was able to get the API endpoint yesterday, but today it's giving me trouble.
Do get more then 'd"' though: https://github.com/jaspervdm/pogoapi-php/issues/4

When you fix the auth name, you should get more then d": https://github.com/DrDelay/pokemon-go/pull/2
(But then you get the same as the issue I referenced)

Should be fixed with https://github.com/jaspervdm/pogoprotos-php/pull/4

Can confirm this works for me

Only the results array is still empty (1 item with null,  because of the 0 length)

I don't know if it's obvious and everyone already did that, but when you decode the raw response this is empty aswell:
$ protoc --decode_raw < raw.bin
1: 53
2:<PHONE_NUMBER>230941184
3: "pgorelease.nianticlabs.com/plfe/193"
7 {
  1: "\033\363/\364C\17....\354\261"
  2:<PHONE_NUMBER>466
  3: "\303\2....;\353t\300"
}
100: ""

... -> Removed data from the AuthTicket, my PKMGO account is really precious :laughing:
So we are doing something wrong in the request?

I compared the requests from a working C#-Implementation with this one, and there is a difference that might be the problem:
The working one sends the location (in this example longitude) like this:<PHONE_NUMBER>467118047
"Ours" sends it like this: 0x40485e646c9122d3
The other one actually calls a function named FloatAsUlong to convert the data.
In a 64-bit PHP Environment you can implement this like that:
unpack('Q', pack('d', $float))[1]
I tried to implement this, but the Proto-file internally calls Protobuf::write_double that makes it wrong again. There is actually a comment saying BUG We need to convert from machine order to little order first next to this function.
This is how far I got, hopefully it helps someone to find a solution to this.

Okay, I was actually looking in to that this afternoon, why the Python API was doing that here https://github.com/tejado/pgoapi/blob/master/pgoapi/pgoapi.py
def set_position(self, lat, lng, alt):
        self.log.debug('Set Position - Lat: %s Long: %s Alt: %s', lat, lng, alt)

        self._position_lat = f2i(lat)
        self._position_lng = f2i(lng)
        self._position_alt = f2i(alt)

def f2i(float):
  return struct.unpack('<Q', struct.pack('<d', float))[0]

I tried passing the raw value from that, but I didn't check the actual sent data.

Also, the note seems to be about little/big endianness. Which they check here for example: https://github.com/drslump/Protobuf-PHP/blob/2201961982bc531a9ab22fc8a150ba815e9b6ef0/library/DrSlump/Protobuf/Codec/Binary/Writer.php#L287-L299
But that doesn't seem to matter for my case.
So should we fix Protobuf::write_double or is the type wrong?

I been working on a library for a while, have a look.
https://github.com/NicklasWallgren/PokemonGoAPI-PHP

Did you also run in to the above issue, or how did you solve that?

I did not actually, take a look at 'NicklasW\PkmGoApi\Handlers\RequestHandler' to see how I handle the API requests.

I noticed you create the RequestEnvelope in RequestHandler, but never fill in the location (latitude/longitude/altitude). Are you planning on implementing that?

In time, it's on the todo list :)

I feel a little dumb right know. You have to resend the request with the AuthTicket after receiving it.
He is doing it there: https://github.com/NicklasWallgren/PokemonGoAPI-PHP/blob/b1174471a1039c89a23260eedb7f6f45430d8f4d/src/Handlers/RequestHandler.php#L90-L95


Update toCamelCase snippet to use const instead of let

Update toCamelCase snippet to use const instead of let.

👍

👍

hello, do any of friends have the code for forward calculation for a magnetic field of a dyke which is under ground? thanks..

it is related to geophysics (geomagnetism)


EmailAuth.tsx ignores redirectTo

Bug report
Appears to be implemented correctly in svelte but not tsx. I've been running into this for weeks and thought I had a bug in my code
Describe the bug
See https://github.com/supabase/auth-ui/blob/2232e5f9d388648e0c37d84ba021518137579747/packages/react/src/components/Auth/interfaces/EmailAuth.tsx#L87
vs
https://github.com/supabase/auth-ui/blob/14245e3fa1df03e20a734252e317fb59e72b2d57/packages/svelte/src/lib/Auth/interfaces/EmailAuth.svelte#L48
the emailRedirectTo param is not passed to the auth.signUp function.
To Reproduce
Set up a Auth usage and specify a url to redirectTo

Probably the cause of https://github.com/supabase/auth-ui/issues/72 and https://github.com/supabase/auth-ui/issues/76


Can't remove yield p = np from a specific partial.

You can see in https://github.com/bullet-train-co/bullet_train-themes-light/pull/27 that I'm basically successful removing all the yield p = np calls in these theme library.
However, there is one that breaks: https://github.com/bullet-train-co/bullet_train-themes-light/pull/27/commits/771c315406aa958401feb99f982c3d51bff4f97e#
My guess is that this happens because it's testing another nice partial, but referencing the parent partial context within the block of the other one. That's the only thing I can think of.
Steps to reproduce:

Clone https://github.com/bullet-train-co/bullet_train
Get it up and running (e.g. bin/setup, bin/dev, etc.)
Run bin/develop and select the bullet_train-themes-light Ruby gem.
Open local/bullet_train-themes-light.
Edit app/views/themes/light/_page.html.erb
Remove <% yield p = np %>
Experience bug.

This is what the app looks like when this bug is triggered:

@kaspth I think partial.relay :title, :actions isn't the solution because it assumes the slots are named the same, which they might not be in other cases.
Looking at the code in the partial that breaks, I added some debug statements like so:
<% puts p.content_for(:title) # => "The Testing Team’s Dashboard" %>

<%= render 'account/shared/title' do |cp| %>
  <% puts p.content_for(:title) # => nil %>
  <% cp.content_for :title, p.content_for(:title) %>
  <% cp.content_for :actions, p.content_for(:actions) %>
<% end %>

<%= render 'account/shared/notices' %>

<div class="space-y-8 py-4 xl:py-8 xl:px-8">
  <%= p.yield :body %>
</div>

The fact that p is getting clobbered here or replaced with something else in the context of this block feels like a bug to me. Is there a way around it?


Adds AbsoluteUnixPath to represent Unix paths in absolute form.

Part of #992.
Followup PR will change LayerEntry#extractionPath to use this new AbsoluteUnixPath.

I think UnixPath can be useful. I can definitely some places where I want UnixPath instead of AbsoluteUnixPath. How about having isAbsolute() on UnixPath instead?

@chanseokoh Hmm, where would you see use of UnixPath? I think of just paths in the container, and those will always be absolute.

For example, https://github.com/GoogleContainerTools/jib/pull/984/files#diff-d4e5570720a7c83a526956eb06f31cffR27
So we will need to concat two UnixPaths.
UnixPath appRoot = ...;
UnixPath DEFAULT_CLASSES_PATH_ON_IMAGE = ...;

UnixPath finalPathInContainer = appRoot.resolve(DEFAULT_CLASSES_PATH_ON_IMAGE);

Technically that certainly works, but conceptually, RELATIVE_CLASSES_PATH_ON_IMAGE should be a Unix path, so if we can declare that it is a Unix path, that will convey the clear message. And since we will be introducing a new class to present a Unix path, it makes sense to apply that to any path expected to be a Unix path.

Okay that sounds reasonable. For toString though, we would probably want to only serialize absolute paths, so UnixPath may be better served as an interface with two subclasses - AbsoluteUnixPath and RelativeUnixPath, where AbsoluteUnixPath would be the only one that is serializable to string?

That can work too. (BTW, by "serializing", do you mean printing out a path in Unix-style? Every class will have toString() anyway.)

@chanseokoh Yep, serialization as in serializing to a Unix-style path string in absolute form.

I was thinking maybe we should limit this to our use case (container file systems) and call this PathInContainer?

Going to rework AbsoluteUnixPath to have it only parse Unix-style paths like RelativeUnixPath

Done. Ready for review again.


Navbar components are broken or overlapped when clicked

Hello @alshoja,
Thank you for your interest in working with our products.
It seems that you haven't respected our ISSUE_TEMPLATE, so we are going to close it.
Please add a new comment here, or open a new issue, by filling the mentioned ISSUE_TEMPLATE and please be sure that your issue title is a very small description of your issue as well.
All the best,
Stefan
Developer @ Creative-Tim.com

am sorry for issue template, I haven't seen any issue templates or maybe am not aware of that, can I get what to do for this issue? @groovemen

Please fill the issue_template and provide us more details about your issue.
Best,
Stefan


Global stats checkbox

Checkboxes don’t work on the global stats page.

Closed with version 1.16.0


ImportError for EigenSolvers

Thank you for this amazing work. I successfully installed everything with the help of Dockerfile. However, when I ran the main.py code, I got following error everytime. I tried recompiling everything but still got same error.
root@55d28786ec5a:/monodepth/code/python/src# python3 main.py
Adding '/monodepth/code/python/src' to sys.path
2023-02-10 08:01:56,106 - utility.depthmap_align - INFO - depthmapAlign python module installed!
Traceback (most recent call last):
  File "/monodepth/code/python/src/main.py", line 17, in <module>
    from utility import blending
  File "/monodepth/code/python/src/utility/blending.py", line 6, in <module>
    from EigenSolvers import LinearSolver
ImportError: /monodepth/code/cpp/lib/EigenSolvers.cpython-310-x86_64-linux-gnu.so: undefined symbol: _ZN6google10LogMessage6streamEv
root@55d28786ec5a:/monodepth/code/python/src#

Can you help me with this? Thank you.

Hey,
That symbol belong to glog I think. So it might be that glog did not installed or that it is not the exact version we tested with Glog 0.5.0.

@manurare Thanks, the glog seems to be of right version since I followed the instruction for docker. I suspect it may be caused due to python version. I will check and let you know.


Export-RubrikDatabaseBackup.ps1 does not handle Failover clusters correctly.

What happened:
When trying to export between 2 failover clusters the refresh fails because the New-RubrikHost cmdlet expects an actual host vs the clustered name.
Refreshing clusteredinstancename in Rubrik
Invoke-WebRequest : Host 'clusteredinstancename' conflicts with existing host 'testserver.rubrik.com' because they have the same identity.
What you expected to happen:
We should account for failover cluster names and their physical hosts.
How to reproduce it (as minimally and precisely as possible):
Try to use this script between 2 SQL Fail over clusters.
Anything else we need to know?:

This will be solved with the new cmdlet of Update-RubrikHost. You will run it like so.
Get-RubrikHost -Name "am1-sql16fc-1" | Update-RubrikHost
Please run update-module Rubrik to ge tthe new cmdlet.
I will close the issue, but feel free to reopen if the new cmdlet does not solve your issue.


Timeout on the docker logs reading only for max-file=1

What does this PR do?
Timeout on docker logs reading only for max-file=1
Motivation
When tailing containers that don't produce much log, we get a lot "tailer timeout error" -> "tailer stop" -> "tailer start" messages
After running for a while, the agent only logs things every 5 minutes.
Additional Notes
Anything else we should know when reviewing?

Will implement #2681


Rework Teams object to make unique Pokemon objects

When a team is created it should have each Pokemon be a new object. This points to weird errors where the same Pokemon could be edited multiple times in the same battle if there is 2x of the same one. Pointers are great!

See https://github.com/alan-turing-institute/p2lab-pokemon/issues/39 and https://github.com/alan-turing-institute/p2lab-pokemon/issues/36


[FIX] Fixed the URL parameter resetting in GItHub pages deployed app

Checklist

[ ] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[ ] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

For the reviewer: Discard the netlify deploy preview and check the GitHub pages deployment.


update Readme

@PiyushRaj927 can we add extensions as a table and we can also have the version of extension on readme itself.


Auto inject auto-approve when the function is apply and non-interactive is set

Pull Request Test Coverage Report for Build 329

4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 43.508%

Totals

Change from base Build 323:
0.04%

Covered Lines:
2322

Relevant Lines:
5337

💛  - Coveralls


EDU-269 Make app title configurable

Closes https://educandu.atlassian.net/browse/EDU-269

Nice, ELMU has officially left educandu 💯


fix(github): fix wait for pr checks to complete

:tada: This PR is included in version 1.0.0-beta.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update server name example to mandatory -sso URL

Hi @xkpd3, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Hi @xkpd3, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


Data studio is not able to authorize the kaggle app.

Data studio is not able to authorize the kaggle app.

having the same issue. Data studio will not allow me to authorize kaggle.

same here and getting the error message "Attention: This connector has not been verified or reviewed. It is recommended that you only use community connectors from trusted sources."

getting the same issue as above. I am posting this to show that this is not an isolated issue

Same here, Data studio will not authorize kaggle. Any inputs?

Same thing here. Any news?

Same here as well

For me the same.
Someone can explain why?
The same for google fit data source access

You can follow up this link for the latest updates from google community. In case of a found solution, feel free to post here.


Hey, I love the simplicity of Macy! I'm having a little trouble though getting it to work with lazy loading.
I'm using jQuery.lazy in a wordpress based gallery plugin that I'm building.
All my images end up overlapping, because it can't get the height for Macy.
 The documentation says that if waitForImages is set to false then it will run Macy every time images are added to the page. So it seems that as I have Macy on all default settings, then every time a new image loads through jQuery.lazyload, except it doesn't seem to be recalculating each time a new image loads. :/
Am I missing something? Do I need to have macy_instance.reInit(); everytime jQuery.lazy adds an image?

Okay I've delved into this some more and I think part of the problem might be that the dist folder is out of date. I'm still new to JS and I haven't yet started building out my projects etc, so I just took the code from macy.js in dist and used that. recalculate and other things mentioned in other threads don't seem to be working, but macy_instance.reInit is working when I type it into the console. I'm having trouble getting it to trigger on new images being loaded though. Any help here would be much appreciated.

Hi @thedonquixotic
Rather than call reInit I would recommend using macy_instance.recalculate() this would recalculate the images.
Hope this helps

@jrmd  I tried running recalculate as per your suggesetion in the other thread but it doesn't seem to do anything. reInit is working with the onImageLoad though, so that's good! I'm guessing it's not super performative though? I'm not having problems yet at least.

@thedonquixotic Could you show me an example so i can see how this is working but I have a feeling it will require true being passed as the first variable.
Like so, macy_instance.recalculate(true) this will force recalculate to reposition every element inside the container.

I'm using macy_instance.runOnImageLoad(function () { macy_instance.recalculate(); }, true); as per your suggestion in the other thread, but doesn't seem to work. I'm thinking this might just be because I have an old version of the dist file.

Have you tried passing true as a parameter to the recalculate
Like so macy_instance.runOnImageLoad(function () { macy_instance.recalculate(true); }, true);

Ay, there we go, yes that works. Thank you!

Awesome that's great news
If you are wondering how the issue is caused, it was caused due to many caching the elements as loaded from the first recalculate, this is obviously helps improved performance but sometimes we need to force update them all for instance when you lazy load the images or when you resize so you pass true as the first parameter to force Macy to ignore this cache and recalculate all elements :)
I will close this issue as I believe to have resolved the problem.
Let me know if you have any more problems

Excellent, thank you a bunch!
On Mon, Jun 19, 2017 at 6:26 PM Jerome Duncan<EMAIL_ADDRESS>wrote:

Awesome that's great news
If you are wondering how the issue is caused, it was caused due to many
caching the elements as loaded from the first recalculate, this is
obviously helps improved performance but sometimes we need to force update
them all for instance when you lazy load the images or when you resize so
you pass true as the first parameter to force Macy to ignore this cache and
recalculate all elements :)
I will close this issue as I believe to have resolved the problem.
Let me know if you have any more problems
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bigbitecreative/macy.js/issues/27#issuecomment-309601041,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJA4zqBnvBLSPJiuwvxRxmmU1BoTJ2Dyks5sFwONgaJpZM4N9E7M
.


sampleRate should be child of envelope instead of being added as tag

Description/Screenshot
When using sampling, the sampleRate currently is being added as tag to the telemetry-item. Although it is available in AppInsights this way, it is not correct. AppInsights expects this sampleRate to be in the telemetry-envelope, as sibling of time/name and tags.
In our situation we have enabled ingestion sampling (as fallback for non sampling applications in our landscape), and this bug is causing our telemetry to be sampled twice.  Also we are unable to extrapolate the actual statistics from the telemetry that is available.
Steps to Reproduce
Enable sampling and check the telemetry which is being tracked. It will look like this:
   {
      "time":"2019-09-24T12:09:48.240Z",
      "iKey":"x",
      "name":"y",
      "tags":{
         "sampleRate":"25"
      },
      "data":{ }
   }

while it should be something like:
   {
      "time":"2019-09-24T12:09:48.240Z",
      "iKey":"x",
      "name":"y",
      "sampleRate": 25,
      "tags":{ },
      "data":{ }
   }

OS/Browser: N/A
SDK Version [e.g. 22]: 22
How you initialized the SDK: NPM Setup

Expected behavior
sampleRate should get its own property on IEnvelope / Envelope.

Fixed in this PR #1061


Back button on home page does not work

Shouldn't work bc there's nothing to go back to 🤔 need to remove it? maybe?


adding axios.all method to getSearchToken() to call multiple apis

-added new variables for url values for api call
-added axios.all to begin refactoring getSearchToken()

program wont work with this recent merge, will finish function and merge later today


log_prob of blocked programs

Hello,
First, thanks for developing such an amazing package.
I am newbie to oryx and was playing around with its functionalities,
perhaps naively I had been attempting to evalute log_probs of blocker porblems, as below:
from jax.random import split
from oryx.core import ppl
import tensorflow_probability.substrates.jax.distributions as tfd

def latent_normal(key):
    z_key,x_key= split(key)
    z=ppl.random_variable(tfd.Normal(0,1),name="z")(z_key)
    return ppl.random_variable(tfd.Normal(z,1e-1),name="x")(x_key)

blocked=ppl.block(latent_normal,names=["z"])
ppl.joint_log_prob(blocked)({"x":10})

However, it returns:
{
"name": "ValueError",
"message": "Cannot compute log_prob of function.",
"stack": "---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[2], line 12
8     return ppl.random_variable(tfd.Normal(z,1e-1),name="x")(x_key)
11 blocked=ppl.block(latent_normal,names=["z"])
---> 12 ppl.joint_log_prob(blocked)({"x":10})
File /opt/conda/lib/python3.11/site-packages/oryx/core/interpreters/log_prob.py:71, in log_prob..wrapped(sample, *args, **kwargs)
67 flat_incells = [
68     InverseAndILDJ.unknown(trace_util.get_shaped_aval(dummy_seed))
69 ] + [InverseAndILDJ.new(val) for val in flat_inargs]
70 flat_outcells = [InverseAndILDJ.new(a) for a in flat_outargs]
---> 71 return log_prob_jaxpr(jaxpr.jaxpr, constcells, flat_incells, flat_outcells)
File /opt/conda/lib/python3.11/site-packages/oryx/core/interpreters/log_prob.py:128, in log_prob_jaxpr(jaxpr, constcells, flat_incells, flat_outcells)
118 _, final_log_prob = propagate.propagate(
119     InverseAndILDJ,
120     log_prob_rules,
(...)
125     reducer=reducer,
126     initial_state=0.)
127 if final_log_prob is failed_log_prob:
--> 128   raise ValueError('Cannot compute log_prob of function.')
129 return final_log_prob
ValueError: Cannot compute log_prob of function."
}
Am I missing something?
Thanks again
Very Best
Giovanni

Hi @jhn-nt. I'm not a contributor or maintainer but I think you might be missing the following. If you look at this description of block it says:
"The block transformation takes in a program and a sequence of names and returns a program that behaves identically except that in downstream transformations (likejoint_sample), the provided names are ignored."
When you run blocked=ppl.block(latent_normal,names=["z"]) you are telling downstream transformations (like joint_log_prob) to ignore the variable with name 'z'. Because the variable with name 'x' depends on the variable with name 'z', which you blocked, joint_log_prob is unable to "see" what the value of 'z' is to get the distribution over 'x' to then compute the log_prob of 10 under that distribution. That's why you're seeing the error.
In general, I would guess you are unable to compute samples or log probabilities of a program with respect to a variable that depends on variables you previously blocked. @sharadmv let me know if I missed anything. And @jhn-nt let me know if you have any further questions! Hope that helps.

Thank very much! this really helped
Giovanni


Improve Command Loader

Allow the command loader to load in commands to the base cli program recursivley, so that I can use a specific folder structure to display relationships between commands and their sub commands.
This is being worked on in branch: bridgeAndRoom

Command Loader looks like it is scaffolding the commander.js command objects correctly and output of hue bridge shows the command 'setup' as a sub command, however when running hue bridge setup -h, the output for hue bridge -h is displayed.
I have created a github issue, #857 at tj/commander.js, to help solve this issue.


fix: Settings - When the default contact is modified, the avatar for the share code is also changed.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/35842
PROPOSAL: https://github.com/Expensify/App/issues/35842#issuecomment-1927879981
Tests

Navigate to Settings > Share Code. Observe the profile avatar for the share code
Go to Settings > Profile > Change Contact Method. Return to check the share code avatar
Verify the avatar has not changed

[x] Verify that no errors appear in the JS console

Offline tests

Navigate to Settings > Share Code. Observe the profile avatar for the share code
Go to Settings > Profile > Change Contact Method. Return to check the share code avatar
Verify the avatar has not changed

QA Steps

Navigate to Settings > Share Code. Observe the profile avatar for the share code
Go to Settings > Profile > Change Contact Method. Return to check the share code avatar
Verify the avatar has not changed

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/85894871/6e657ece-93c0-4353-be19-ba8133513c1c

Android: mWeb Chrome
https://github.com/Expensify/App/assets/85894871/6ae6e879-ca21-457c-bfd0-2e5d5b8f736b

iOS: Native
https://github.com/Expensify/App/assets/85894871/469bc297-59a6-4e4d-9600-223eac4555bf

iOS: mWeb Safari
https://github.com/Expensify/App/assets/85894871/ac9e98ad-eda6-46f2-8ba1-a2130c09f630

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/85894871/8bea5118-c664-4efb-921e-07d6a67c41c7

MacOS: Desktop
https://github.com/Expensify/App/assets/85894871/50b13e04-bb6b-4812-a782-72f21913c686

Reviewer Checklist

[x] I have verified the author checklist is complete (all boxes are checked off).
[x] I verified the correct issue is linked in the ### Fixed Issues section above
[x] I verified testing steps are clear and they cover the changes made in this PR

[x] I verified the steps for local testing are in the Tests section
[x] I verified the steps for Staging and/or Production testing are in the QA steps section
[x] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[x] I checked that screenshots or videos are included for tests on all platforms
[x] I included screenshots or videos for tests on all platforms
[x] I verified tests pass on all platforms & I tested again on:

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

[x] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[x] I verified proper code patterns were followed (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I verified that this PR follows the guidelines as stated in the Review Guidelines
[x] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

iOS:Native & Android: Native
https://github.com/Expensify/App/assets/85645967/67df3070-b5c7-427c-8426-158459c051c4

Android: mWeb Chrome & iOS: mWeb Safari
https://github.com/Expensify/App/assets/85645967/ae484c64-2c23-43f6-83ba-c92d7163f523

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/85645967/32927db5-1376-46d3-ae04-ef205ad0c73e

MacOS: Desktop
https://github.com/Expensify/App/assets/85645967/24f7ff58-649d-4d7e-a257-a086173afeb3

So I originally agreed to the proposal and the code definitely seems to work but now that I'm reading it, the function is just kind of weird. It has this comment:
    // To ensure that the avatar remains unchanged and matches the one returned by the backend,
    // utilize an optimistic ID generated from the email instead of directly using the user ID.

But then we override that sometimes and it's not super clear why that is. I feel like this solution just kind of makes the function more confusing. Is there another way that we could get the avatar URL when a user already has an avatar set? I see that in Onyx it says the user has an avatar like "https://d2k5nsl2zxldvw.cloudfront.net/images/avatars/default-avatar_6.png". Can we just use the number 6 in there to get the NewDot URL?

Seem the issue was duplicate of 35560. Now its fixed we no longer need this PR.

Ah ok. I'll go ahead and close it out then. We'll still pay y'all out since the work was done.


[Suggestion]  Add support for importing coolors.co palettes.

Hi there,
First I just wanted to say thank you for all the work you've done on this program, it's a really amazing bit of software!
I was setting up some profiles today on my keyboard when I found your program, and in order to get everything looking nice I was using this site Coolors.co to find complimentary palettes.  It's really helpful, it's free, and it has an export feature that lets you save the colors to a .scss file.  And I got to thinking it would be cool if you could add a way to import those palette files directly.  It's not difficult to copy the hex over, but it would be awesome to have a couple preset profiles that are set up to use a palette of 5 colors that users could set up near instantly by just opening the exported .scss files.
Thanks again for this program, it's finally made me really glad I splurged on an RGB keyboard after several months.  The Equalizer layer is friggin amazing!

Here's an example of the contents of one of those SCSS files, I couldn't attach the file directly.
/* Coolors Exported Palette - coolors.co/2a2b2e-5a5a66-a4c2a8-aceb98-87ff65 */
/* HSL */
$color1: hsla(225%, 5%, 17%, 1);
$color2: hsla(240%, 6%, 38%, 1);
$color3: hsla(128%, 20%, 70%, 1);
$color4: hsla(106%, 67%, 76%, 1);
$color5: hsla(107%, 100%, 70%, 1);

/* RGB */
$color1: rgba(42, 43, 46, 1);
$color2: rgba(90, 90, 102, 1);
$color3: rgba(164, 194, 168, 1);
$color4: rgba(172, 235, 152, 1);
$color5: rgba(135, 255, 101, 1);


Python3 Support

Hey,
I want to use Python3 with this library.
I have made all the changes locally and it seems to work for me.
Can you give me access to push branches so I can make a pull request? @fedoco

You don't need special access to the repo in order to make a pull request. You just need to fork it.

Are you still interested?


Why are some read lengths not in frame?

Sorry if I'm asking naive questions. I'm new to ribo-seq analysis.
Q1. Most of the read lengths show a nice 3-nt periodic pattern in CDS, but some do not, like 31nt in my case. The graphs are attached ("frames_stratified_plots.pdf", "metaprofile_plots_readlen31.pdf", and "metaprofile_plots_readlen32.pdf"). What are those reads that are not in-frame? Should I only use the in-frame read lengths for downstream analyses?
Q2. While the metagene plot from all genes shows nice periodic pattern ("metaprofile_plots.pdf"), the pattern can go wild for a single gene, and there are peaks outside of CDS. Is this normal? ("metaprofile_plots_transcript_ENST00000412830.8_gene_PWP1.pdf", "metaprofile_plots_transcript_ENST00000541166.1_gene_PWP1.pdf", "metaprofile_plots_transcript_ENST00000547995.5_gene_PWP1.pdf" and "metaprofile_plots_transcript_ENST00000552760.5_gene_PWP1.pdf")
frames_stratified_plots.pdf
metaprofile_plots_readlen31.pdf
metaprofile_plots_readlen32.pdf
metaprofile_plots_transcript_ENST00000412830.8_gene_PWP1.pdf
metaprofile_plots_transcript_ENST00000541166.1_gene_PWP1.pdf
metaprofile_plots_transcript_ENST00000547995.5_gene_PWP1.pdf
metaprofile_plots_transcript_ENST00000552760.5_gene_PWP1.pdf
metaprofile_plots.pdf
Thanks.

Hi there.
A1. It is quite common - and, to me, perfectly reasonable - to observe reads that are in frame and reads that are not. You can clearly see the same pattern in this section of riboWaltz's ReadMe. I would say this is due to the action of RNAses and the standard size of ribosomes that make some populations of reads more "reliable" than others. Hopefully, lengths which are associated to in-frame reads are the most frequent and only a negligible amount of reads are >31 nts long and they are not going to affect downstream analyses. You can decide to keep or discard them to get better and more precise results, but it's up to you.
A2. Also in this case yes, it is normal to have good metaprofiles for multiple transcripts but not for single ones. This is because of the depth of the sequencing, that is usually too low for getting a nice coverage for single transcripts (especially if they are "underrepresented"; actin or tubulin are usually highly translated and might show something more). Also consider that metaprofiles are based on P-sites, that just cover one nucleotide. I think that even a whole sequencing run dedicated to a single sample wouldn't be enough for observing the 3-nucleotide periodicity for one mRNA. Given this, I would not suggest to generate P-site -based metaprofiles for single transcripts or even small subsets. It would be better to project the whole reads, i.e. their ~30-nts-long footprints, on the transcript of interest and average the replicates. So that you increase the signal at each mRNA position and can look for potential differences in ribosome coverage among multiple samples. This way, for sure you cannot detect defects at single nucleotide resolution for specific mRNAs separately, but at least you can have an hint on the "translational status" of single transcripts. It's a bit of manual work (riboWaltz doesn't include any function for this yet), but I think it can be done. Finally, don't worry about signal on the UTRs, it is a common feature. It can be due to upstream translation start site and alternative ORFs, but most of the time they are just random reads mapping outside the CDS.
Hope it helps.
Best,
Fabio

Thanks so much for the detailed answers. These are super helpful!
When I project the whole reads on the transcript of interest, should I shift the read positions (positions of all the bases in the read) using the P-site offset?

I wouldn't do that, otherwise you cannot really appreciate the real ribosome position. Plus, it's additional work that in my opinion is not necessary. Online there are several examples of similar plots, which should look like this for two conditions (note the signal upstream the gray area, which is the CDS):

Thanks!
I generated a coverage plot using the reads. There is a high peak in the 5' UTR. (yellow: UTR; blue: CDS
PWP1_ENST00000412830.8_coverage.pdf
)

Yes indeed. Might be the starting codon of an annotated isoform of the same transcript or an alternative and an unannotated translation start site for unknown ORFs. You can explore this hypotheses by checking the sequence of the transcript or by similar analysis.
Since you got the profile and the issue is not strictly related to riboWaltz, I'm going to close it.
Best
Fabio


Bug with manager notification

Hello
I found few bugs, when manager receive notification of user password expiration:

Every time logo inserted in template, even if i remove it from  $FormattingParameters
ManagerDisplayName  doesnt work.
Problem with EmailSentTo, if user has no email - in report Not available, if user have email - in report N/A
Here is logs from script:

[2020-09-17 15:43:47] [i] Starting processing Users section
[2020-09-17 15:43:47] [i] User USER1 expires in 1 days (09/19/2020 01:35:46). However user has no email address and will be skipped.
[2020-09-17 15:43:47] [i] User USER2 expires in 1 days (09/19/2020 01:46:32).
[2020-09-17 15:43:47] [i] Pretending to send email to<EMAIL_ADDRESS>...Success
[2020-09-17 15:43:47] [i] Ending processing Users section

Thank you!

Please insert your configuration here.

Hello dear Przemyslaw!
I almost use config from your example.
Here is it:
`
RemindersSendToManager = @{
Enable               = $true # doesn't processes this section at all if $false
RemindersDisplayOnly = $false # prevents sending any emails (good for testing)
SendToDefaultEmail   = $true # if enabled $EmailParameters are used (good for testing)
ManagersEmailSubject = "Manager Subject"
Reports              = @{
IncludePasswordNotificationsSent = @{
Enabled          = $true
IncludeNames     = 'DisplayName', 'DaysToExpire', 'EmailSentTo'
TextBeforeReport = '"Following users which you are listed as manager for have their passwords expiring soon:"'
        }
    }

    # You can use limit scope
    #LimitScope           = @{
    #    Groups = 'RecursiveGoup-FGP-Check'
    #}
    # SendCountMaximum     = 3
}

[ordered] @{
Enable                   = $true # doesn't processes this section at all if $false
RemindersDisplayOnly     = $true # prevents sending any emails (good for testing) - including managers
SendToDefaultEmail       = $true # if enabled $EmailParameters are used (good for testing)
Reminders                =  1, 2, 3, 4, 5 #50
#UseAdditionalField       = 'extensionAttribute13'
#SendCountMaximum         = 3
# this means we want to process only users that NeverExpire
#PasswordNeverExpires     = $true
#PasswordNeverExpiresDays = 30
# limit group or limit OU can limit people with password never expire to certain users only
LimitGroup               = @(
#'CN=GDS-PasswordExpiryNotifications,OU=Security,OU=Groups,OU=Production,DC=ad,DC=evotec,DC=xyz',
#'CN=GDS-TestGroup9,OU=Security,OU=Groups,OU=Production,DC=ad,DC=evotec,DC=xyz'
)
LimitOU                  = @(
'*MY_OU_HERE_1',
'*MY_OU_HERE_2'
)
}
`
this is my manager template:
`
TemplateForManagers     = "
Hello <>,
BLABLABLA
<<ManagerUsersTable>>

"

`

One quick fix: just add 'Manager' in to IncludeNames
But I doesn’t want show this field. Because those manager receive report.
 Reports              = @{
            IncludePasswordNotificationsSent = @{
                Enabled          = $true
                IncludeNames     = 'DisplayName', 'DaysToExpire', 'EmailSentTo', **'Manager'**
                TextBeforeReport = '"Following users which you are listed as manager for have their passwords expiring soon:"'

            }
        }

And I completely remove company-logo, but it still in email.

And thats why I got 'N/A' in manager report.
Users settings:
 [ordered] @{
                Enable                   = $true # doesn't processes this section at all if $false
                RemindersDisplayOnly     = $true # prevents sending any emails (good for testing) - including managers
                SendToDefaultEmail       = $false # if enabled $EmailParameters are used (good for testing)
                Reminders                =  1, 2, 3 #50

Managers settings:
 RemindersSendToManager = @{
        Enable               = $true # doesn't processes this section at all if $false
        RemindersDisplayOnly = $false # prevents sending any emails (good for testing)
        SendToDefaultEmail   = $true # if enabled $EmailParameters are used (good for testing)

if ($ConfigurationParameters.RemindersSendToManager.RemindersDisplayOnly -eq $true) {
                Write-Color @WriteParameters -Text "[i] Pretending to send email to manager email ", "$($m)", " ...", "Success" -Color White, Green, White, Green
                $EmailSent = @{ }
                $EmailSent.Status = $false
                $EmailSent.SentTo = 'N/A'
            }

So it's just misconfiguration on your part, not a bug. The bug you report is basically a problem with a logo, and nothing else, right?

So it's just misconfiguration on your part, not a bug. The bug you report is basically a problem with a logo, and nothing else, right?

Hm, not exactly. I doesn’t want to send email to users. I just want to notice managers .
Anyway, I got  'N/A' instead real user email.
Just try disable user notification and leave only manager report.
In addition , without adding 'Manager' in to IncludeNames - manager email would be without manager name.

That's for you to configure. YOu can configure your own reports.
ReminersDisplayOnly is not for what you want. it's for testing not preventing users from receiving emails.

That's for you to configure. YOu can configure your own reports.
ReminersDisplayOnly is not for what you want. it's for testing not preventing users from receiving emails.

Okay. I doesn’t want to inform users about password expiration , but I want to inform managers about that.
How can I do that?

Yeah. I can confirm: if mail was not sent - EmailSentTo = N/A
I should use different field with User Email?

UPD: just use  EmailAddress instead EmailSentTo
But script still put company-logo, even if I remove it from code


DOC: read_annotations also reads BDF and GDF

minor fix of the docstr, as BDF and GDF are also supported, but not mentioned, see:
https://github.com/mne-tools/mne-python/blob/0aae72dda030242d992866ce6d551206749474b2/mne/annotations.py#L1235-L1236

Thanks @sappelhoff !


In android the current location is not zooming in when loading the map initially

Hi,
I am using your module for the maps. In iOS it working fine and when initally loads its showing the current position in zoom.
But in Androd it showing the current location pointer but its not zooming in.
Here is my code:
For iOS:
 <MapView style={{height:height,width:width}}
         showsUserLocation ={true}
         followsUserLocation = {true} />

for Android:
here is the screenshot

    <MapView style={styles.map}
       showsUserLocation ={true}
       rotateEnabled = {true}
       zoomEnabled = {true}
       showsMyLocationButton = {true}
        />

For android the followUserLocationis not working
Can anyone help me to solve this.

https://github.com/react-community/react-native-maps/issues/209#issuecomment-350907665


Wake up sleeping reEnqueues when Success is called

It's a bit sad, but as written whenever a partition gets suspended once all future writes to that partition are also while it is suspended until it finally gets reset. This is because we re-enqueue based on what the result of the previous Produce call was, not based on the result of the call we're just about to make is. The code originally waited on the Produce so it would know whether to suspend or reset it, but this had the side effect that you could only have as many in flight requests as your partition count, which Bashar asked me to change. At the time that I implemented that request I didn't consider this optimization.


Update to Boost 1.64

replaced the /boost/ directory with files from 1.64 release
replaced the same folders from the /libs/ directory with those from the release

prerequisite for cctbx/cctbx_project#184


Protractor@4 promises.all didn't work

I installed new protractor@4 and faced the problem that my code didn't work because of renaming protractor namespace:
var flow = browser.controlFlow();         return protractor.promise.all(             actionPromises.map(function(promise) {                 return flow.execute(function() { return promise; });             })         );
And i get error: protractor.promise.all is not a function
I tried to rename protractor to browser and it didn't help.
Please help how should i change my code to get it worked?

Found this solution, it helped for me stackoverflow.
But i'm looking forward for adding back promise.all to protractor namespace


[BUG] bro your anchor sucks asf

if it was even supposed to work however it doesnt

works for me, usser issue

if it was even supposed to work however it doesnt

Config issue + robot issue, if u move too fast using step/strafe it wont anchor u to hole which is good


Tool to quantify activity in Slack

From @hellyeah on October 26, 2015 17:44
Copied from original issue: hackedu/hackedu#420

From @jonleung on October 28, 2015 21:45
@hellyeah I believe you did this already?

Closing this because hackclub/metrics exists


guest_os_hypervisor missing entries for XenServer 7.1.2

ISSUE TYPE

Bug Report

COMPONENT NAME

DB

CLOUDSTACK VERSION

4.11.2, 4.11.3

CONFIGURATION

N/A
OS / ENVIRONMENT

XenServer 7.1.2 (7.1 CU2) Hosts
SUMMARY

There are no entries in guest_os_hypervisor for XenServer 7.1.2 (7.1 CU2 - the current LTS XenServer) which means guests are booted using the "Other Install Media" template.  Amongst other things, this means that PV guests boot as HV, but also likely means guests won't get XenServer optimisations for their OS type.
Perhaps a better way rather than just inserting specific rows for this version would be to add logic to ignore the revision number in a version matching pattern x.y.z if there are no rows found, e.g. look for row with version = 7.1.2, fallback to row with version = 7.1.0 (would still work if XenServer is updated without CloudStack being updated).
Related: #3086
STEPS TO REPRODUCE

Install Fresh CloudStack instance with new XenServer 7.1.2 hosts.  Wait for CS to try to deploy system VMs.

EXPECTED RESULTS

System VMs should boot using the Debian Wheezy 7.0 (64-bit) XenServer template
ACTUAL RESULTS

System VMs should boot using the Other Install Media XenServer template (which fails to boot in a virtual environment using XenServer running on VMs, only capable of running PV).

@richardlawley You can see engine/schema/src/main/resources/META-INF/db/schema-41000to41100.sql as a reference on how to add support for a xenserver version. Is it fair to assume that every 7.1.x will support the same sets of xenserver guests? If yes, we can modify the business logic to take any existing mapping for any 7.1.x XenServer version to be used for any new 7.1.y version.

I'm fine with how to fix it on a one-off, but other people might be less comfortable.
Regarding future versions, I think it's possible that they could potentially introduce new templates, so there should still be the capability to add specific overrides, but modifying it to try 7.1.2, 7.1.1, 7.1.0 etc should take care of most situations with the ability to add new rows if needed.

@richardlawley I think we can deal with a large refactoring after 4.13, likely in 4.14, till then I've added the support here: https://github.com/apache/cloudstack/pull/3467/files Please review.


Border/Outline bars

It would be good if there was a way outline the bars and each stack within the bar.
In my use case I want each of the stacks within a bar to be the same color, and an outline border of black around each stack to differentiate each one rather than the color.
It may be good to have something like the following properties:
outline: boolean
outlineColor: hex color
outlineWidth: number in pixels

Hi @niallmc ,
Thank you for the suggestion. It was noted and I will make ASAP.
But for that moment, you could use some CSS properties to change the colors of the component.
For instance:
svg#hsbar > g > rect {
  fill: white;
  stroke: black;
  stroke-width: 3px;
}

@ricardodorosario Thanks for the CSS I can use for the moment.

Solved in #10


Why have to implement MS_Deform_Attn in cuda?

You have written "ms_deform_attn_core_pytorch(value, value_spatial_shapes, sampling_locations, attention_weights)" in pure pytorch. Why not use this pytorch version?

See #9.

Noted with thanks.


feature request: Use juliaup to customize which version of julia is being tested

If you have juliaup installed, you can do julia +1.8.2 [other arguments] to launch specifically the julia 1.8.2 version of julia. Benchmarking on specific julia version that way would be quite useful.

cc @kylebeggs
I guess all we would need to do is allow for a version spec here: https://github.com/MilesCranmer/AirspeedVelocity.jl/blob/493a0e01a32b2dbd0b81698bfd6259918e6ce5f6/src/Utils.jl#L246

Do you want to make a PR for this? I can review but probably can't code it myself for a while. I think we would also need an integration test with/without a version specification.

yeah I'll give it a shot


fix(ngRoute): extract path params containing hashes and/or question marks correctly

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This PR fixes a regression introduced by 840b5f0.
What is the current behavior? (You can also link to an open issue here)
If the path ($location.path()) contains hashes and/or question marks, path params are not correctly extracted.
For example, if you set $routeProvider.when('/foo/:bar', {}) and navigate to /foo/bar%23baz, you will get $routeParams.bar === "bar", not $routeParams.bar === "bar#baz".
What is the new behavior (if this is a feature change)?
Path params are correctly extracted ($routeParams.bar === "bar#baz").
Does this PR introduce a breaking change?
No.
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Fix/Feature: Docs have been added/updated
[x] Fix/Feature: Tests have been added; existing tests pass

Other information:

I signed it!

@gkalpak  Thanks for your review!
The previous parseRoute and pathRegExp functions have different types of arguments (urls and paths) so I thought they should be separate.
Anyway, I fixed it!

Awesome, thx!

The previous parseRoute and pathRegExp functions have different types of arguments (urls and paths) so I thought they should be separate.

I know, but they have almost the same logic, so it makes sense to share (see https://github.com/angular/angular.js/issues/14173#issuecomment-192649766).

I created #16672 which adds some refactorings on top of your commits (mainly to simplify the implementation and decouple the two slightly different usecases of ngRoute/ngMocks).
Thx for providing the main fix, @susisu 💯

Merged as https://github.com/angular/angular.js/commit/510404e5b34fcf47a4d0cc8094afe7431b0a0b63


Harbor should not display private repositories in the home page

Hello,
Harbor should not display private repositories in the home page, in the "Popular Repositories" Pane. Not only the name of the repository appears, but also the name of the owner, which is a potential login name to test (maliciously) elsewhere.
Unrelated: Harbor sounds very promising, thank you for your work !

@ecirtap thanks for the suggestion, it makes sense!

ETA on a fix for this, @reasonerjt? We have a publicly facing instance of harbor that is currently advertising these to the world after the update.

@jakerobinson We are actively looking into it now and a fix will be provided very soon. Thanks for reporting this problem - it is a major one that we want to tackle immediately.

@jakerobinson Hopefully a fix for this will pushed to master by the end of this week.

fixed by #627

Hello,
It is not fully fixed; private repository names are still displayed in the public home page (tested in 0.3.5). It would be wiser to not display anything about private repositories as their names can contain indications on internal (=private) entreprise activities.


Can't tell whether there was a response to a command

The network protocol for daliserver doesn't allow it to indicate whether or not a command returned a response.  Lack of response to a command is reported as response 0xff, status 0, which is the same as a response of 0xff.
This makes some DALI commands impossible to use.  For example, I can happily use the "QUERY ACTUAL LEVEL" command and receive valid data back because this command always causes a response from the ballast.  I can't use the "QUERY LAMP FAILURE" command because this command causes a response of 0xff for "yes" and a lack of response for "no" — daliserver squashes both of these to a response of 0xff.
In lib/usb.c you distinguish between the two cases, but you just call dali->req_callback() with a hard-coded 0xff when you're dealing with USBDALI_TYPE_NO_RESPONSE; a fix for this ought to be fairly simple.  The problem (and the reason I'm not sending a patch) is that the network protocol will have to be changed to fix this and I don't want to pre-empt whatever change you decide to make.

Thanks for pointing this out.
I've always been aware that the (very simplistic) protocol wouldn't stand the trial of time, but it was sufficient for the simple dimming control I use it for.
If it can make daliserver useful for you, I'll review and improve the protocol and condition handling.

That would be great!
If it won't affect your use of it, I suggest redefining the "status" byte of the response so that success/failure is reported in bit 0, and response or lack of response is reported in bit 1.  So we get:

Status
Meaning

0
success, no response (the most common case)

1
failure

2
success with response

I've improved the network protocol and added more status codes so you can differentiate the different types of messages. The Perl examples are updated as well and should be ready to go.
If you can test the changes, please report your results. I haven't had time for that yet.

Ok, that appears to work so far.  I get sensible replies to QUERY LAMP FAILURE.
I think it would be useful to expand the protocol further, though: it's able to report commands that originate on the DALI bus, but there's nowhere for it to indicate whether those commands had a response and what that response was.  I believe the hardware is capable of reporting responses to observed commands, because the manufacturer provides a DALI bus snooping application that does it.

I'll think about the proposed changes. Ah, what I'd give for proper documentation...
As for this bug report, I'll close it for now, since the feature seems to be working.


Result of mA is 0.81?

PETA
Why is there a big gap between my results and yours？

For PETA dataset, some simple baselines can achieve mA larger than 81%.
Please check your dataset and codes, e.g. correct weighted loss, correct dataset, ...

谢谢，我再多尝试发自我的华为手机-------- 原始邮件 --------主题：Re: [chufengt/iccv19_attribute] Result of mA is 0.81? (#3)发件人：Chufeng Tang 收件人：chufengt/iccv19_attribute 抄送：guanhuiyan ,Author For PETA dataset, some simple baselines can achieve mA larger than 81%.
Please check your dataset and codes, e.g. correct weighted loss, correct dataset, ...
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/chufengt/iccv19_attribute/issues/3?email_source=notifications\u0026email_token=AJOY63GW5EGFD2L66BIA5MDRGITTPA5CNFSM4LCX7CLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODWS7Y#issuecomment-596076927",
"url": "https://github.com/chufengt/iccv19_attribute/issues/3?email_source=notifications\u0026email_token=AJOY63GW5EGFD2L66BIA5MDRGITTPA5CNFSM4LCX7CLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEODWS7Y#issuecomment-596076927",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]