UEFI spec version vs a minimal subset

UEFI spec section 2.6 lists the various boot/runtime services and platform specific elements. If we mandate a specific version in BRS spec, it may mean that all of the things specified in the spec has to be implemented by the firmware.
For EDK2, it probably won't be an issue but it makes harder for any other firmware/boot loader to comply.
There are many platform specific requirements which may not be required.
Example:
If a platform includes USB bus support, then EFI_USB2_HC_PROTOCOL and EFI_USB_IO_PROTOCOL
must be implemented. An external device can support USB by producing a USB Host Controller Protocol.
If a platform includes an NVM Express controller, then EFI_NVM_EXPRESS_PASS_THRU_PROTOCOL must
be implemented.
I think we should add a UEFI conformance profile for each recipe to make it easier.  EBBR already defines a one for the its own UEFI subset.
@adurbin-rivos @andreiw

Thanks for the pointer. Presumably, we would put a profile in that would indicate minimum support. Essentially 2.6.1 Required Elements. And punt on the following as optional:

2.6.2 Platform-Specific Elements
2.6.3 Driver-Specific Elements

Looks like appropriate section for conformance profiles is 4.6.5 EFI_CONFORMANCE_PROFILE_TABLE. There's only currently 1 listed (to adhere to all requirements encompassed in  2.6 requirements).

EBBR publishes its own GUID: https://github.com/ARM-software/ebbr/blob/main/source/chapter2-uefi.rst#efi-conformance-profile-table
I missed the original suggestion about associating a conformance profile for each recipe. Question I have is what happens when we extend it? Looks like EBBR uses semantic versioning and any value in the patch level of the version number is deemed compliant.

EBBR decided to go down the route of listing required pieces as well as exceptions to those other line items.

EBBR decided to go down the route of listing required pieces as well as exceptions to those other line items.

Yeah as few of those are largely unused in modern system. If we can relax the requirement section in UEFI spec itself, we won't have to do anything. I am not sure if that's something feasible keeping our timeline.

EBBR publishes its own GUID: https://github.com/ARM-software/ebbr/blob/main/source/chapter2-uefi.rst#efi-conformance-profile-table
I missed the original suggestion about associating a conformance profile for each recipe. Question I have is what happens when we extend it? Looks like EBBR uses semantic versioning and any value in the patch level of the version number is deemed compliant.

For BRS-B, follow conformance profile defined by EFI_CONFORMANCE_PROFILE_EBBR_2_1_GUID For BRS-I, define a new conformance profile with a 2.6.1 (required) and subset of 2.6.2 & 2.6.3. We also need to define required/optional bootime/runtime services. ARM base boot requirement can be a good starting point for this.

The original draft BRS doc (https://docs.google.com/document/d/1X0TSbheEJjRGWhG2nzUnfIl_QcHWSuvd/edit#heading=h.38op2zyxb220) talked about defining a conformance profile (EFI_CONFORMANCE_PROFILE_OS_A_SEE_1_0_GUID).
...I think a conformance profile is a good idea. We can refer to the the laundry list in SBBR as a start (I kinda duplicated that in the draft doc) and define the subset of that for BRS-I.
Like I mentioned, this doc needs a BRS-I-S or BRS-SI defined as well (the server fw reqs on top of BRS-I), and that ought to have a separate conformance profile that is a superset of the BRS-I one.

So the question is 2.6.2 Platform-Specific Elements (EFI_CONFORMANCE_PROFILES_UEFI_SPEC_GUID) too restrictive for the BRS-I (not BRS-B aka EBBR) use case?
I took a long hard look at 2.6.2 and it's common sense stuff for a non-embedded implementation.

Complete?


Search functionality

Implement search functionality, and utilize user input in geo api.
Attach lat/long from geo api to weather api.

Added countries as a selection in the search form

Completed search functionality. Will still need to add error catches later.


Philadelphia

I just changed "D.C." to "DC" so it would fit in with the other states and entered a template for philly.

Since we already have a philly letter with similar content, I'm going to close this one out.
Feel free to make a PR against the current philly template, or if you want to add a different one for philly that works to - e.g., defund12.org/philly-somethingelse. Check out Louisville as an example, as we have multiple letters for it!
Thanks so much!


Verify the size of data received based on tsize option

The tsize option can be used to verify the size of the received file after the transfer is complete. If the tsize does not match the number of octets received then the error packet can be sent as a response to the last data packet.
This is a second verification apart from ACK that the data received is complete.
I can contribute the changes if you think it is appropriate to include this.

Hello and thank you for your interest in rs-tftpd! Of course you can contribute for this. I think this would be a great addition.


Add new user and card functions for frontend, enhance backend api and fix bugs

my butt is stuck in the toilet, pls call for help, and a pizza!


Update Kiota version on main to 1.14.0

Updating Kiota version to 1.14.0. Verify there are no breaking changes.

This was a test for https://github.com/Hyperfoil/horreum-client-golang/pull/19


Touch (click) events are drifted (shifted incorrectly) for PortalTargets

Fixing it.
Possibly related: https://github.com/fzyzcjy/flutter_portal/issues/21

reproduced.

caused by #65


modify md format based on change of grpc md format

Hi, @treetty , could you provide the reference of the change?

https://github.com/grpc/grpc-go/blob/master/metadata/metadata.go

type MD map[string][]string
line 76

It is better to use this format(https://github.com/grpc/grpc-go/blob/master/metadata/metadata.go#L76)

LGTM

sure.
you may also want to review the pr that i made in bwmf.
also, i m not quite sure about how to run a bwmf task on etcd yet.
if you can help me with that, it would be great! @plutoshe
@xiaoyunwu

thank you for commenting on this! @plutoshe


Reduce test kernel set by over 400MB, down to < 20MB.

spiceypy/tests/gettestkernels.py

Replace de421.bsp (16MB) with de405s.bsp (1.4MB; DE-405 short ca. 1997 to 2010).
DE405S:  Needs to be <native>-IEEE for testing, typically LTL-IEEE for Intel.
DE405S:  Wrote script make_de405s_native.py to convert from BIG-IEEE kernel at JPL/NAIF.
DE405S:  Integrate make_de405s_native.py script into gettestkernels.py.
DE405S:  Shift many time-lookup tests into range of de405s.bsp.
DE405S:  Add step to spiceypy/tests/gettestkernels.py to convert BIG-IEEE de405s.bsp from NAIF to native byteorder.
Shift Cassini test kernel coverage from 2004 to 2013, to use smaller kernels.
CASSINI/2013:  Change ABCORR to LT from LT+S, to NONE from S.
Drop MER and MRO kernels; use Phobos instead of MRO/MER; hardcode MER location.
Retrieve smaller Phobos DSK (59k) than original NAIF DSK (16MB).
Retrieve smaller, and static, Earth rotation state binary PCK.
Do not retrieve latest Earth binary PCK; it changes too often.
Remove MarsKernels class.
Remove obsolete cleanup and download (get*Kernels) routines and calls.
Remove remove redundant kernelFile.close().
Improve readability of global downloadTestKernels() routine.

spiceypy/tests/make_de405s_native.py

New script to convert de405s.bsp from NAIF to native byte-order.
Does functionally the same thing as SPICE utility BINGO.
Downloads NAIF de405s.bsp as de405s_bigendian.bsp.
Converts de405s_bigendian.bsp to def05s.bsp.
Asserts with checksum to ensure success.

spiceypy/tests/test_wrapper.py

Adjust assertion expected values to match returns from new kernels
Adjust parameters in test_dskw02_dskrb2_dskmi2()
Remove obsolete cleanup calls
Replace MGS/Mars tests with Cassini/Enceladus tests; adjust expected values
test_ckcov():  simplify assertion, improve test
Change ['%s%f%d' % tuple(...)] format statements to ['{}{}{}'.format(...)
Use Mercury transit of Sun instead ofr MRO transit of Mars for test_occult().

Coverage remained the same at 100.0% when pulling d067908a24cdd1740063d300aadfb4fcc7dae0aa on drbitboy:improvement/smaller-kernel-set into 64fb6e428155df6c38f18e3f747a8a3818e63f90 on AndrewAnnex:master.

Coverage remained the same at 100.0% when pulling d067908a24cdd1740063d300aadfb4fcc7dae0aa on drbitboy:improvement/smaller-kernel-set into 64fb6e428155df6c38f18e3f747a8a3818e63f90 on AndrewAnnex:master.

ok so the BE/ LE thing...  Is there a major BE OS I am forgetting? I guess you could count stuff running on ARM potentially?
As per my other post about the phobos DSK we could just host the LE kernel on Github instead of converting it. This would be less code to manage.

[Combining a few of the comment subjects above into this one comment for brevity (not my strong suit;-)]
Issues:
0.1. Non-https DSK:  meh, as you say; still it would be breaking the CONtRIBUTING guidelines.  I am not sure https does not work it; it might just be a typo.  In any case, does six.urllib2 actually validate the certificates against an actual authority?  Or is it like the commonly-used [--no-check-certificate] option in wget?
0.2. Non-NAIF DSK:  that page is my old Cornell University Astronomy (CUAstro) Dept. Account<EMAIL_ADDRESS>~carcich/public_html/, still active five years after I retired.  I agree it is not ideal, but NAIF's DSKs are may be no more stable (see 1.2 below).
0.3. BE/LE thing:  no, there is no BE OS (yes, we got no bananas;-), but CONTRIBUTING.md makes "keeping things platform-neutral" a Responsibility.
Alternatives (two in addition to your three):
1.0. Non-NAIF DSK;  If nothing else, I could ask Tom Shannon, the CUAstro admin, to host it somewhere more permanent in that domain; I am pretty sure he would do that.  Or maybe you have someplace (other than Github; Google Drive?) to host this file; I think that could also use https.  Still, it's ugly.
1.1. Use 10Mb NAIF DSK:  no if at all possible; I really do not want to give back 50% against the fruits of my hard labor.
1.2. Ask NAIF to host it:  been there, done that, was turned down; Boris said "the generic_kernels/dsk area is very much in flux", so maybe even the current wasteful Phobos test DSK is unstable.  I re-asked about possibility of a_older_versions/ sub-directory; we'll see what he says.
1.3. Use github to host it:  probably the most robust option.
1.3.1 At 20Mb for all of the kernels, why not go all the way and put a .zip and a .tar.gz in a separate SpiceyPy support repo (although compression will probably be minimal on the binary kernels, which make up the bulk of the test kernels).  I think we can use the raw interface to urllib just the file, or use the git Python interface to extract the contents of the support repo (SpiceyPySupport?  SpiceyPyTestKernels?).
1.4. Nat suggested generating kernels on the fly.  We already do that in some cases (test_getfov(); test_gfsntc(); test_srfc2s(); so one could argue that, in effect, the precedent of having a kernel in the SpiceyPy repo is already broken).
2.1. BE/LE:  IIRC, the byte order requirement on the SPICE side comes from deprecated, but backward-compatible, DAF routines that pre-date the introduction of LTL-IEEE/BIG-IEEE in binary kernels, and the introduction of endian-blind SPICE routines.  So a better solution might be to drop the SpiceyPy interfaces to any endian-sensitive deprecated routines, along with their tests, at which point we no longer need to supply at least one native binary test kernel, we can just take whatever byte order NAIF supplies.

after reading the above:

Seems like making another github repo just for the two test kernels is the best option
By platform neutrality I meant macOS/Windows/Linux compatibility. Like using having forward slashes vs back slashes for file paths and the like. BE OS support is not important for the project at the moment.

I can make a public repo for the two kernels soon. An extra advantage is that the repo would be clone-able so we could have backups hosted by gitlab/bitbucket.

I agree with using Github for those test kernels, but if we do that then we put all of the kernels into the github repo:  once we have a single test kernel on Github, it makes no sense to bother with another domain (NAIF/JPL) which we cannot control.

https://github.com/drbitboy/SpiceyPyTestKernels

Okay, it's done, using my test kernel repo for now.  We should get some new checks soon.
Obviously at some point we will move it to AndrewAnnex@github, and I can also shrink the binary kernels some more, but this works now (at least it passes in my [linux+python2|3] environment.
Also, I think it may even be faster (I don't think NAIF/JPL has bandwidth that can compete with github).

ok great, I can just fork the repo and then we can just update the links. I think less than 20Mb is a huge win so for now we can just stick with that for now.

Ok @drbitboy, I made the fork and I have a pr for adding the LE de405s.bsp kernel see https://github.com/AndrewAnnex/SpiceyPyTestKernels/pull/1 . Let me know if that is ok, since that conversion script isn't part of the spiceypy codebase you can choose to license it under your name as well, just let me know and I will update that pr before merging.

Okay, that is great.
So there are a few more changes to make here:

Remove [assert len(...) == 3] and re-do ets list
Remove commented line [#handle = ...]
Re-home those kernel links to @AndrewAnnex

Wait, are you are putting the make_de405s_native.py script into the kernels repo?  It belongs in the SpiceyPy repo, under tests so will not be confused with the codebase.

I added the LE kernel in that PR I made against my fork of the test kernel repository (I didn't delete the BE one). So as far as I can see there is no need for it to be in this PR once I merge in that PR (I was waiting for you to take a look at it). This will also simplify a number of line changes in the current pr once I merge that other PR into the master branch. If you are very concerned about being able to run the tests with the BE version of the kernel then we could have an if check in the gettestkernels code that downloads the correct version if needed.  Otherwise, I don't understand why it is critical here.

I am confused; not sure which pr and which "it" you are referring to in several places in that last comment.
can we deal with this tomorrow?  I have to take a break.
Thanks,
Brian

sure thing, feel free to wait until later before you read this. I will try to be more clear here:

I forked your repository for the Test Kernels (https://github.com/drbitboy/SpiceyPyTestKernels)
I made a PR against my fork of the Test Kernels at (https://github.com/AndrewAnnex/SpiceyPyTestKernels/pull/1) that includes the following:

de405s.bsp, which is the LE (aka PC) version of the de405s_bigendian.bsp kernel
make_de405s_native.py, that I used to make the new de405s.bsp kernel
Updates to the README to include the md5 hash for the new de405s.bsp kernel

what I am suggesting is:

with your OK I will merge that PR (https://github.com/AndrewAnnex/SpiceyPyTestKernels/pull/1) into my fork of SpiceyPyTestKernels.
Then we can remove make_de405s_native.py from this PR.
make_de405s_native.py would now be located in (https://github.com/AndrewAnnex/SpiceyPyTestKernels)
Then we can simplify the code in this PR a bit more as we do not need to convert the de405s_bigendian.bsp kernel within the tests here anymore, as we can directly download the new de405s.bsp kernel.

Coverage remained the same at 100.0% when pulling ff7ce308278ab620562e6e4715dba270d581ce27 on drbitboy:improvement/smaller-kernel-set into 64fb6e428155df6c38f18e3f747a8a3818e63f90 on AndrewAnnex:master.

If we put both LE and BE de405s SPKs into the SpiceyPyTestKernels repo, the make_de405s_native.py script is no longer needed.

Okay, it looks like appveyor and travis are or will be soon showing success, with the usual ignoreable travis osx/py2.7 hiccup.
The script [make_de405s_native.py] is obsolete and gone, and I am pretty sure all review issues have been addressed.
When you re-pull or re-fork the test kernels repo, then you can change all seventeen occurrences of  [drbitboy] to [AndrewAnnex] in spiceypy/tests/gettestkernels.py and do your internal PR, or just push.
We could make the repo owner an envvar that defaults to AndrewAnnex a la
ghOwner = os.environ("TEST_KERNELS_REPO_OWNER","AndrewAnnex')
...
blah_url = "https://raw.githubusercontent.com/{}/SpiceyPyTestKernels/master/...".format(ghOwner)

to make modifying the testing easier in the future, but the tests will eventually stabilize, so the slight awkwardness in updating tests without the envvar is not enough to compensate for the loss in code clarity.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>09f30f16e01c12595af90a3d60605b on drbitboy:improvement/smaller-kernel-set into 64fb6e428155df6c38f18e3f747a8a3818e63f90 on AndrewAnnex:master.

ok great @drbitboy, this looks good so I will merge soon

https://ci.appveyor.com/project/drbitboy/spiceypy/build/1.0.68/job/88jvt1uweykp7nub
hmmph.

Coverage remained the same at 100.0% when pulling 95220475f5a91cf9a20a71b5c7f1c6897fa13caa on drbitboy:improvement/smaller-kernel-set into 64fb6e428155df6c38f18e3f747a8a3818e63f90 on AndrewAnnex:master.

@drbitboy interesting... but it didn't fail in my appveyor builds? maybe there is a more memory efficient way to unpack the list... although it should be a numpy array at that point I think?

Yes, very odd.  Also, it was a memory error but not a segfault, so whatever caused the problem was caught and an exception thrown.  Maybe the VM or the VM host ran out of memory.

I will merge this once I can get the last travis macos build to pass, so don't worry about this for now.

yes, that's what I figured.

woo hoo!


DSA approve/reject - the Reservation Status doesn't update to "Approved" after updating a pending reservation. Also checked in Calendar... No change of status

Testing Design:
I created a reservation "draft" using Dimple STO user.
Tested the updated SAF print. (it works) and clicked "Submit to Property Custodian". it works... store successful.
However, when I opened Admin (DSA user type) and update reservation to Approval.
It says store successful but the status label in the reservations list and calendar didn't change. Also checked in database phpmyadmin. it didn't change.

also the same case with Reject Option


Can Additional data be modified?

Question
Can additional data that already have a value (not NULL/Nonebe modified?
If yes, they probably should have historization.
If no, what happens if update is tried?

Very good question, happy to talk about it on a call if needed.
But on the first glance it looks like additional data potentially can contain various things and maybe even some business logic can be applied to it, which might mean that having a history of it can bring some value, i.e. won't limit anybody (unlike no history, just raw updates).
So I would be inclined to say this: yes they can be modified, but using 'immutability' principle. A brand new record would be created, retiring an old one.

Implemented as part of #58 in #143


Refactor Show.js to be a stateless functional component

Hey @simeydk, could you combine this and https://github.com/wesbos/Syntax/pull/206 into one stateless functional update?

Hi @416serg
I'm happy to combine #201 and #206. However, you have closed both. Should I open a new PR that combines both?

Hey Simey,
That would be great! 🔥 Make sure to pull in a fresh copy of master and
open up a new PR for that 😄
Looking forward to it!
On Tue, Oct 23, 2018 at 3:17 AM Simey de Klerk<EMAIL_ADDRESS>wrote:

Hi @416serg https://github.com/416serg
I'm happy to combine #201 https://github.com/wesbos/Syntax/pull/201 and
#206 https://github.com/wesbos/Syntax/pull/206. However, you have
closed both. Should I open a new PR that combines both?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wesbos/Syntax/pull/201#issuecomment-432124406, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADlhn40k627XvM7aFa_tMQZExHdDHswxks5unsKGgaJpZM4XbHUS
.


[QUESTION] When will be Roadmap item will be out in the new version

may i know that when the next version of this package will get release with support of Imports feature any date for the completion of this feature.

See our blog: https://medium.com/@maatwebsite/laravel-excel-lessons-learned-7fee2812551 We don't have a planned date for 3.1. We currently have no time to develop this and thus cannot promise any release date.


Use result of AssignTargetPath to calculate output paths

Instead of performing path mangling, we'll use the established convention of using AssignTargetPath to calculate relative paths \ output paths.
This resolves https://github.com/aspnet/Razor/issues/1847 https://github.com/aspnet/Razor/issues/1829

🆙 📅

Build seems to be failing though

🆙 📅
I'll fix up the xplat tests after this test run

Tests fixed on Travis


check node version on postinstall & while initiating a new project

Fixes #169

Travis checks are failing because lib/lib/check-engine.js is not available.
If we update the build on travis, that would fix the tests.

Either way process.version seems super nice here, since it makes the whole check synchronous.

@developit Are you opposed to moving this check-engine.js file to the root & adding it to the NPM files?
This way, we can let the tests run without doing any fancy side stepping. Currently, this will be fine with downloads since a lib/ will already be built.

Yup, I'm fine with that.


add job to test against curves repo in CI

Description

closes: #34
Add new CI job test_against_curves which does the following steps (as per issue description)

Clone curves repo
Clone algebra repo
Patch curves Cargo.toml to force the dependency tree to use the local version of the algebra crates.
Run all tests in curves

Additional Notes
Running all the tests for curves taken around 22 min which leads to an increase of CI runtime from 10 min to 25 min on average.
But if an additional 15min can prevent merging changes that break downstream (curves), I'd say it's well spent

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (master)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

Hi, so is the idea to make sure that new PRs don't break curves? But how about the ones that do?

Hi, so is the idea to make sure that new PRs in algebra don't break curves? But how about the ones that do?
I guess the idea is that at least it would be caught? I think this makes sense given that it's a separate job.

Yeah, that's the idea, ones that break curves should ideally cause CI to fail.

Although this makes some sense, do note that many other downstream components make use of generic APIs in algebra, so this doesn't rule out downstream breakage outside of curves.

The job in PR only concerns with detecting breakage in curves. For other components, we can try to add similar jobs if the resulting CI runtime is within the respectable range.

This is great, thanks!


Replace existing annotations

Currently, if SVScore will not run on a file with existing SVScore annotations. It would be nice if it were able to do so, replacing both the vcfanno and SVScore annotations with new ones.

Fixed in e13e52dcc0144f713ff95a022160fc087cca8bc7


Automatically set screen resolution?

        WindowManager wm = (WindowManager) context.getSystemService(Context.WINDOW_SERVICE);
        Display display = wm.getDefaultDisplay();
        Point size = new Point();
        display.getSize(size);
        subject.setScreenResolution(size.x, size.y);

A user is doing the above - is this better/different from what we are already doing? If so, how/why?

We are doing the same thing except that we are accounting for the fact that this only works on API 19 and up.  If we have a different target api it uses a different approach.

Cool, closing


Layer issue on tooltip and x-axis tooltip

Codepen
https://codepen.io/jimfilippou-the-animator/pen/aRpxdGe
Explanation

What is the behavior you expect?
Tooltip not overriding the x-axis tooltip

What is happening instead?

What error message are you getting?
No error message

Oh! So, the tooltip still gets in the way of x-axis tooltip. Wondering, why my last fix didn't work.
Your codepen link doesn't work. Can you please send a new one?

Whoa... why 404 on my Codepen?
investigating... 🔍

Will be shipped with tomorrow's release 2.1.5


E2E tests for multiple worker node groups

Description of changes:
Expands current e2e testing framework to support multiple worker node
groups. It allows to add/remove/update worker node groups to the initial
spec and during upgrades.
Adds one simple vSphere tests that starts with two worker node groups.
During the upgrade, it scales down one of them to 1 replica, it deletes
the other one and creates a new one.
This will need to be refactored eventually. The framework api is becoming a bit convoluted and expanding the framework itself is quite difficult. We will create tickets to refactor this in the near future. We don't do it now because it requires to rethink a bit the framework internals and in particular the way we apply api object updates and that will take some time.
We are postponing that refactor because at least this adds the ability to test multiple worker node groups and unblocks subsequent work.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/lgtm

/cherrpick release-0.7

all of the checks have passed, and then were re-trigged by commentary. Overriding.
/override eks-anywhere-attribution-files-presubmit
/override eks-anywhere-e2e-presubmit
/override eks-anywhere-mocks-presubmits
/override eks-anywhere-presubmit


Robot Localization Example Tweaks

Our robot localization example now works as described in PR #247 but as the screenshot shows, Rviz is placing the WAM-V very far away from the reference grid:

Can we center the grid?
Also, running
roslaunch wamv_gazebo localization_example.launch

still produces a disconcerting stream of warnings. I realize this is a topic of hot discussion, but until it's resolved can we do something to suppress them? I don't want our users to think they've done something wrong if this the expected behavior. If suppressing the warnings causes problems let's at least add some documentation to the tutorial.

@crvogt Tagging you since this is relevant to the tutorial you're reviewing.

@M1chaelM I'll look at re-centering the grid. With regards to the warnings, I can add a line to the launch that should suppress the warnings.

@crvogt Great! Thank you.

@M1chaelM I'm noticing that running rostopic echo /wamv/robot_localization/odometry/filtered in the rviz example that the pose is not only very large, but suggests the wamv is moving. Have you come across this/is this expected?

@M1chaelM After restarting my machine, the pose is fine. I'll do a PR with the grid updated to match the location of the wamv.

A slightly more targeted error (but still very clumsy) suppression line is:
2> >(grep -v TF_REPEATED_DATA)

I think we can add it to the command line when needed, rather than baking it in to anything. Since the grid location is fixed with PR #267 I will close this issue.


Allow adding custom fields to enroll form

I would like to add a custom radio field to enroll form so that user should select that before enrolling. It seems this is not possible and it is only possible for registration.

Is this possible?

I would like this as well? any way to achieve this?


Expert merging: c-BTM

We would like to create a script for creating a merged model by using the C-BTM method.
The script would take as input:
List of experts models from the [MDEL HF repo](https://huggingface.co/Multi-Domain-Expert-Layers).
Name of the output model

The averaged model would be uploaded to the MDEL HF repo. It's model card should contain the names of the experts it was created from.

I would also work on this too.
@NourFahmy  There are two steps that we could split 😃

Clustering of original training data used in different expert
Inference code (that weight next token prediction logits according to proximity of input to cluster centroid)

@NourFahmy @kenhktsui
Check out Minho's adapation of the clustering step from the cBTM repo:

Hi @kenhktsui - happy to take on inference and support where need be on clustering, and to fill any gaps from Minho's efforts.
I've put up a PR here
I've made the following assumptions I can easily fix:

both the embedded context & current token will be passed to the script
only the conditional probability of the token at time t given the context is needed, as per formula 2 in the paper

kindly inform if anything else is needed!
cc: @mrcabbage972


Focus other pane after sync

This adds an option which, when set, focuses the other tmux pane after tmux_cwd_sync_now is executed.
To activate this, add the following to ranger's rc.conf file:
set tmux_cwd_sync_now_focus true

Tested, it's working well. At the moment, i don't see any issue...


#163989809 Fix: Fix some server files

refactor the controller and the validation handling the controller codes

[Finishes Fix]

Codecov Report

Merging #58 into develop will increase coverage by 0.8%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           develop      #58     +/-   ##
==========================================
+ Coverage    93.64%   94.45%   +0.8%
==========================================
  Files           14       14
  Lines          598      595      -3
==========================================
+ Hits           560      562      +2
+ Misses          38       33      -5

Impacted Files
Coverage Δ

server/middleware/validateRecord.js
95% <ø> (+2.69%)
:arrow_up:

server/controllers/record.js
93.75% <100%> (+10.41%)
:arrow_up:

server/routes/index.js
96% <100%> (ø)
:arrow_up:

server/controllers/user.js
85.71% <100%> (+3.66%)
:arrow_up:

server/middleware/validateUser.js
85.18% <80%> (-1.18%)
:arrow_down:

Continue to review full report at Codecov.

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


Drift net + Support (expert) cape not counting graceful

🐛 Bug Report
Write a description of the bug here, give all the information you can about what it is, how it happened.
Support cape should be giving the effect of full graceful everywhere, drift net fishing doesn't have that check included yet.
Untested - Master Agility Cape (probably same issue but not tested)

Master agility cape was reported as a bug a while ago which I made a patch for.
Im primarily an OSB player and aren't aware of all the capes and specific perks from rs3 content.
Will be a quick thing to patch

This was fixed


chore: add issues automatically to devprod project

I'll need to test it in prod @penalosa but I think it'll appear in the untriaged tab yeah


Autoload on .yuck files and add better instructions for doom emacs users

Thanks for your help!

You're very welcome


Gtk theming

On some systems, the current way to handle Gtk themes seems to be working great because the applications look native. Here is the Drawing app on Xubuntu 18.04, looks native.

However, in other situations, theming is entirely broken.
Maybe the way to go would be to bundle the theme from Flatpak, and then decide at runtime whether to use the system one or the bundled one. E.g., if the system one is much older than the bundled one, then use the bundled one; use the system one otherwise.
What do you think?

Is a very good idea, but i don't know how to implement this, maybe checking glibc version?

These applications (if the conversion is successful) runs even on Ubuntu 12.04

Is a very good idea, but i don't know how to implement this, maybe checking glibc version?

I was thinking of checking the Gtk version.

I'm added this option on https://github.com/sudo-give-me-coffee/flatpak2appdir/commit/4eb33ec4fae1639b07cf9a402c118b1ca45a0e52


split/merge encapsulated packet content

Example:
{"seqNumber":7,"encapsulatedPackets":[{"reliability":3,"hasSplit":16,"length":1432,"messageIndex":6,"orderIndex":1,"orderChannel":0,"splitCoun
                t":12,"splitID":0,"splitIndex":4

https://github.com/PocketMine/RakLib/blob/master/server/Session.php#L284

So that's actually fairly easy to solve.
For the parser : just create an new parser stream instance, write data to it and listen to its "data" event, that automatically merge packets.
For the serializer/write : split the buffer in n parts of size mtuSize and create n encapsulated packets to put them in.

The fact that this is needed make #11 kind of impossible (which is not that much a problem)
In a kind of weird way, that "merge data over several packets" look like what's done in minecraft classic to encode the map (see https://github.com/mhsjlw/minecraft-classic-protocol/blob/master/examples/client.js#L23 )

basically, it's a stream in a stream.

done


Make labels on the topology legend clickable

@miq-bot add_label ui, enhancement, providers/containers

@himdel @martinpovolny please review. thanks!
cc @stefwalter @jonnyfiveiq @serenamarie125 @dclarizio

:+1: (except for whitespace in CSS)

(except for whitespace in CSS)

what whitespace? line num please

@abonas Lines 16 & 17 - missing space after colon. (Also, 53 & 54 have double semicolons..)

@abonas Lines 16 & 17 - missing space after colon. (Also, 53 & 54 have double semicolons..)

@himdel done!

@miq-bot add_label ui, enhancement, providers/containers

@himdel can you please merge it?

@abonas sorry, I don't have the rights..

@chessbyte ?
On Oct 19, 2015 6:02 PM, "Martin Hradil"<EMAIL_ADDRESS>wrote:

@abonas https://github.com/abonas sorry, I don't have the rights..
—
Reply to this email directly or view it on GitHub
https://github.com/ManageIQ/manageiq/pull/4957#issuecomment-149240893.

@miq-bot add_label topology


Recommended CPU and RAM for ~1000 requests/s

Hello,
I will need to handle 500+ concurrent users per second => ~1000request/s. Right now I have 2 cpu and 15 GB of ram (aws r3.large instance) and I am not able to handle more than 30 request/s (via blazemeter.com or jMeter locally) with simple queries like following: (product contains 7 string properties)
SELECT expand(OUT('HasProduct')[0-99]) FROM Category WHERE id = 'recommended' fetchplan delivery:1
This query is fast, when it's executed only once, but on concurrent connections it's really slow. Doesn't matter if I use differente query (select, insert), I am not able to handle more than 30-50 request/s.
I am using default settings, OrientDB 2.1.16 and OrientJS driver.
Thank you

hi @janjilek
How much does this query takes?
SELECT expand(OUT('HasProduct')[0-99]) FROM Category WHERE id = 'recommended' fetchplan delivery:1

Elapsed in explain: 26.697037
Explain execution in the web console:  0.072 sec
Query executed in the web console: 0.282 sec

Can you post  code you are using for the connection?

It's strange, when I try it on really slow machine (1 virtual cpu, 512MB ram) i have following:
Elapsed in explain: 25.986715
Explain execution in the web console: 0.07 sec
Query executed in the web console: 0.233 sec
So it looks like some default configuration issue.

Actually I tried a lot of different approaches:

1 db instance
var server = OrientDB({});
export var db = server.use({});
Use of ODatabse
var db = new ODatabase({});
Use of Connection pool
var pool = new Pool({
min: 1,
max: 100
});
pool.acquire().then((db) => {});
Create N (>=10) connections polls (max = 50) and round robin between them

The option number 4 is the fastest one, but the difference is about 10-15%

When I execute it on my MacbookPro 13 2015 (i5 2.7, 16GB ram) i have same results.

@maggiolo00 probably we should discuss that with @tglman  ?

Can you monitor the Cpu of OrientDB while running this test?

how many request with number 4?

I run jMeter with this setup:
Number of threads (users): 100
Ramp-up period: 1s
Loop count: 5
Results:
Samples: 500
Throughput: 7.7requests/s
Min: 235ms
Max: 13314ms
Average: 11579ms
Top commnad on server instance
VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND
3736m 936m  16m S 16.0  6.1   2:40.68 java

how many request with number 4? // Sorry i don't understand your question.

I updated results with different approaches. The approach 1 is normally not so bad, because I have more NodeJS instances. This results are only from 1 backend instance.

@janjilek  Im very interesting in your approach I also have the same configuration but without high traffic. Good luck. I will try to help you when I find something.

@janjilek
with approach 4 is more than 15%.
This is because with approach 1 yes you will have only one connection opened and all the request are serialized.

Yes, sure, I had error in my code, because I tested it with more backend instances before, so I had 5+ connections to the DB.

Is 35.6 requests/s expected performance for 2 CPUs and 12 GB of RAM? Is more important to have more CPU cores? Or how many CPUs and GB of memory would you suggest for that kind of requests/s?
Thank you

@janjilek
do you have numbers with 2 instances?

perhaps related cause db switch overhead
https://github.com/orientechnologies/orientjs/issues/169

I will setup 2 instances and i will write here results.

Results for approach 1 with 2 backend instances
Samples: 500
Throughput: 13.9 requests/s
Min: 254 ms
Max: 7471 ms
Average: 6072 ms
Top commnad on server instance
VIRT RES SHR S %CPU %MEM TIME+ COMMAND
3898m 984m  16m S 32.3  6.4   6:10.84 java
Results for approach 1 with 4 backend instances
Samples: 500
Throughput: 25.6 requests/s
Min: 222 ms
Max: 4317 ms
Average: 3197 ms
Top commnad on server instance
VIRT RES SHR S %CPU %MEM TIME+ COMMAND
4092m 1.0g  16m S 66.2  7.0   6:35.14 java
Results for approach 4:
Basically the same like with 1 instance

@janjilek
strange with 4 instances and approach 4 you should go better.
Which is the size of your custom pool?

I tried 50 pools with max = 5 in 4 instances => 1000 connections. Now i will try 5 pools with max = 50 and maybe 25 pools with max = 25

So with 2 instances and 5 pools with max = 50 performance drops to 15 requests/s (don't understend why) and CPU usage is only on 35%.  With 50 pools with max=5 the performance is +- the same like before.
With 4 instances and 5 pools with max = 50 performance drops to 30 requests/s and CPU usage is on 50%.  With 50 pools with max=5 the performance is +- the same like before.

I will try to add some CPUs and will write you the result with approach 4.

Results for approach 4 and db server with 4 CPUs:

backend server with 4 instances
Samples: 500
Throughput: 73.1 requests/s
Min: 208 ms
Max: 2350 ms
Average: 1024 ms

Top commnad on server instance
VIRT RES SHR S %CPU %MEM TIME+ COMMAND
4938m 872m  16m S 297.0  2.8   2:09.32 java

backend server with 1 instance
Samples: 500
Throughput: 56.5 requests/s
Min: 262 ms
Max: 3109 ms
Average: 1428 ms

Top commnad on server instance
VIRT RES SHR S %CPU %MEM TIME+ COMMAND
4667m 929m  16m S 256.0  3.0   3:27.61 java

I have tried now again (5 times) with 4 instances and the average result is about 62 requests/s. So not so big difference.

With 8 CPU on DB machine:
Samples: 500
Throughput: 75.8 requests/s
Min: 202 ms
Max: 2278 ms
Average: 912 ms
Top commnad on server instance
VIRT RES SHR S %CPU %MEM TIME+ COMMAND
6862m 885m  16m S 217.7  5.9   2:05.43 java

I will try some more stuff tomorrow. It's not using all the db cores, so maybe it's something wrong with load balancing.

So, I tried today a lot of instance types for backend and database and best these are the best results.
I was able to handle 181 requests/s with c4.2xlarge (8 CPUs, cpu usage 560%) instance on database and 2x m4.xlarge instances (8 CPUs in totall). I tried also more m4.xlarge instance but the result was same like with 2.
When I descreased number of returned products from 100 to 50, I was able to handle 360 request/s but the cpu usage was again about 560%. So there is probably some aws throughput limit and I have to intestigate it more.
The last try was with 10Gbps instances. 1 c4.8xlarge (36 CPUs, 132 GB RAM) for database and 2 m4.10xlarge (40 CPUs,	124.5 GB RAM). The result was 120 request/s (with 1200% of cpu usage), so there is probably some db configuration error.
Do you have any suggestion (configuration), how to test maximum concurrent performance for different queries? Is normal, that OrientJS driver consumes fully 8 CPUs for 200 requests/s?
PS: t2 instances were bottleneck for yesterday testing.

hi @janjilek
about orientjs a lot of cpu time is consumed on deserializing the results set. In 2.2 we implemented the native serializer/deserializer to gain more speed.

@janjilek
Not sure if this could be relevant but you might lower latency and jitter on ec2 instances with the right drivers, enabling enhanced network capabilities (http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/enhanced-networking.html).

Thank you, I will try new version of driver next week and enhanced network capabilities sounds also good. Will write you back results next week.

Interesting @fabriziofortino . You would think the best networking would be standard and not an option.
@janjilek - I also know in the past OrientJS had issues with, I believe it was, marshaling and there was discussion on improving the OrientJS driver. It seems it is why ODB did so badly compared against Arango. That was some time ago too. I didn't follow if there were improvements made to the driver or not. If there wasn't, the performance issues are is most likely in the OrientJS driver.
Scott

@smolinari I can try to implement test case also with Java driver and compare results :-)


Add support to compile with VS2013?

Hi Roland,
I appreciate your work on Sqlpp11 library, quite impressed with the concept and would like to use it in my project but unfortunately your code does not compile with VS2013. Can you please add support to get this compile with VS2013?
I know you mentioned somewhere you have used C++11 features that were not supported by VS 2013 compiler but there are alternatives you can use bu substituting with equivalent boost library implementation.
Can you please add support to get this compile using MSVC 2013?
Thanks
Praveen

Hi Praveen,
As discussed in issue 34 (https://github.com/rbock/sqlpp11/issues/34), not even the CTP6 of MSVC-2015 has sufficient C++11 support. Back in August 2014, the current build of MSVC still had troubles with static_assert (this did not compile then: https://gist.github.com/rbock/b573ee3d024902998e3e).
I see no (reasonable) way to replace these things with boost equivalents. And quite frankly, even if I did, I would not want to do it. I'd rather spend my time improving the library and hoping for the MS compiler to finally catch up.
Of course, if you believe to be able to rewrite those parts of the library that stop it from compiling with MSVC 2013, please feel free to do so. I'll gladly link to your project.
Best,
Roland

Seems MS fixed it (https://gist.github.com/rbock/b573ee3d024902998e3e)
Try the latest online version of the compiler (updated on Jun 1 2015)
http://webcompiler.cloudapp.net/

The question was about MSVC-2013 (not 2015). There is some hope left for MSVC-2015, but as of now, that also seems problematic.
struct A
{
  static constexpr const char _literal[] =  "hello_world";
};

This does not compile even with the June-1 version. The compiler accepts the constexpr outside the struct though. See also: https://github.com/rbock/sqlpp11/issues/43#issuecomment-107643085

struct A
{
static constexpr const char* _literal =  "hello_world";
};
compiles fine :)

That does not help me, I think. Take a look at
include/sqlp11/alias_provider.h or tests/Sample.h
If that can be done with const char*, then we're getting somewhere :-)
On 2015-06-02 14:53, Alexey wrote:

struct A
{
static constexpr const char* _literal = "hello_world";
};
compiles fine :)
—
Reply to this email directly or view it on GitHub
https://github.com/rbock/sqlpp11/issues/36#issuecomment-107943672.


[Backport] [2.x] [Concurrent Segment Search]: Add support for query profiler with concurrent aggregation (#9248)

(cherry picked from commit https://github.com/opensearch-project/OpenSearch/commit/a7374460aa0e91b5af4859458e771e5b2525b9e2)

Description
At present, the query profiler is the total timing obtained by summing all segments. However, this does not accurately represent the detailed timing of the query, including queue time and gap time, during concurrent execution. This PR will address the problem.
Related Issues
Resolves https://github.com/opensearch-project/OpenSearch/issues/8330 https://github.com/opensearch-project/OpenSearch/issues/7354

Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@sohami @ticheng-aws holding the merge, we have NPE here #9815 and another NPE variation in this pull request, we cannot let it sneak into the release:
java.lang.NullPointerException: Cannot invoke "org.opensearch.action.search.SearchResponse.getFailedShards()" because "profileResponse" is null
	at __randomizedtesting.SeedInfo.seed([E68DEFD659967F87:B55838D408DE9DFB]:0)
	at org.opensearch.search.profile.query.QueryProfilerIT.testProfileMatchesRegular(QueryProfilerIT.java:195)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:568)
	at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1750)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:938)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:974)
	at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:988)
	at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)

@reta We looked into this bug. The NPE is happening because of an issue observed in concurrent search path of profile where rewrite is happening during execution phase as well (this use case was unknown). This issue existed even before these new changes were done but it is showing up now since we enabled the QueryProfilerIT with concurrent search flag enabled as part of these changes. So the plan is to merge these new profile changes and disable the failing test for concurrent segment search for now. The fix will be worked on as part of the issue you created and we will enable the test with that fix. Given only some specific queries (with profile option enabled) in concurrent segment search path is impacted and is behind experimental flag, we should still be fine back porting this PR for experimental release in 2.10.0. Wdyt ?

Just realized this is a separate issue and is happening in the new code path. We will not back-port the new changes to 2.10 in that case.

Closd this PR as it's been backported from https://github.com/opensearch-project/OpenSearch/pull/10898.


Raise error instead of writing to log

Coverage dropped because I've added the pytradfri directory rather than only testing the tests-folder.

Coverage remained the same at 100.0% when pulling b6675269d39ea42aa25c8524ddebbde06aed752f on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.

Coverage remained the same at 100.0% when pulling b6675269d39ea42aa25c8524ddebbde06aed752f on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.

Coverage decreased (-80.8%) to 19.194% when pulling dbacc67ed34c99fb9e92b9fe07567cd2973859b6 on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.

Coverage decreased (-37.5%) to 62.545% when pulling 24a2cfa794e023f8f3952838c58aa91493dd79f2 on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.

Coverage decreased (-37.5%) to 62.545% when pulling 24a2cfa794e023f8f3952838c58aa91493dd79f2 on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.

@balloob Thanks! I'll squash and merge as soon as Travis starts working again and tests pass.

Coverage decreased (-37.5%) to 62.545% when pulling 738ae0b9eef6c845af96cbc94c04188937846387 on ggravlingen-patch-1 into c9f0ed9767a4a778fd0ae5277b061ba509b17bd2 on master.


XCode Build Fails: Duplicate Symbols for Architecture x86_64

Hi, I'm getting this error from your modules when building for iOS
ld: 186 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
** BUILD FAILED **
This might have something to do with the $(inherited) Linker Flag from Cocoapods.
How do we fix this?

Hi, if like me you use Cocoapods.
For make it build I unlink with run "react-native unlink react-native-maps" and now everything works fine.
I think pod + automatically link create this duplicate issue

@christopherdro can be close. @noboddee's solution works

Nope, still getting that Duplicate Symbols.
Please Re-Open this Issue.

@ProteanDev wait for https://github.com/react-community/react-native-maps/issues/1862 (soon, waiting for a Testflight Apple review to complete the tests) where I'll set some better instructions on how to install it. If you're in a hurry ping me and I'll share it with you before making the PR

@ProteanDev https://github.com/alvelig/react-native-maps-pods-example check this example project

Right now experienced that on my own project example LOL unlink solved.

same issue here, i just not make react-native link but getting this error

@abrajeethan have a look in Xcode if you have any map libs linked -  https://github.com/react-community/react-native-maps/blob/master/docs/installation.md#on-ios
Also try to clean the project form Xcode


Bump Flow version to v0.75.0

Currently we are on flow 0.71.0, this change moves that version to 0.75.0. Luckily no new errors arise from this bump 🙌

Deploy preview for gestalt ready!
Built with commit e6dfb60c26860ccd6350e4be0908c59490116881
https://deploy-preview-268--gestalt.netlify.com

Deploy preview for gestalt ready!
Built with commit 77226012c471b5307f18a3aeca8bee8b527add1e
https://deploy-preview-268--gestalt.netlify.com

Codecov Report

Merging #268 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   70.24%   70.24%
=======================================
  Files          46       46
  Lines         820      820
  Branches      189      189
=======================================
  Hits          576      576
  Misses        172      172
  Partials       72       72

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 cb4c3fc...7722601. Read the comment docs.


ArrowIOError: Invalid argument

I get an ArrowIOError when reading a specific file that was also written by pyarrow. Specifically, the traceback is:
>>> import pyarrow.parquet as pq
>>> pq.ParquetFile('gaz2016zcta5distancemiles.parquet')
---------------------------------------------------------------------------
ArrowIOError                              Traceback (most recent call last)
<ipython-input-18-149f11bf68a5> in <module>()
----> 1 pf = pq.ParquetFile('gaz2016zcta5distancemiles.parquet')

~/local/anaconda3/lib/python3.6/site-packages/pyarrow/parquet.py in __init__(self, source, metadata, common_metadata)
     62         self.reader = ParquetReader()
     63         source = _ensure_file(source)
---> 64         self.reader.open(source, metadata=metadata)
     65         self.common_metadata = common_metadata
     66         self._nested_paths_by_prefix = self._build_nested_paths()

_parquet.pyx in pyarrow._parquet.ParquetReader.open()

error.pxi in pyarrow.lib.check_status()

ArrowIOError: Arrow error: IOError: [Errno 22] Invalid argument

Here's a reproducible example with the specific file I'm working with. I'm converting a 34 GB csv file to parquet in chunks of roughly 2GB each. To get the source data:
wget https://www.nber.org/distance/2016/gaz/zcta5/gaz2016zcta5distancemiles.csv.zip
unzip gaz2016zcta5distancemiles.csv.zip

Then the basic idea from the pyarrow Parquet documentation is instantiating the writer class; looping over chunks of the csv and writing them to parquet; then closing the writer object.
import numpy as np
import pandas as pd
import pyarrow as pa
import pyarrow.parquet as pq
from pathlib import Path

zcta_file = Path('gaz2016zcta5distancemiles.csv')
itr = pd.read_csv(
    zcta_file,
    header=0,
    dtype={'zip1': str, 'zip2': str, 'mi_to_zcta5': np.float64},
    engine='c',
    chunksize=64617153)

schema = pa.schema([
    pa.field('zip1', pa.string()),
    pa.field('zip2', pa.string()),
    pa.field('mi_to_zcta5', pa.float64())])

writer = pq.ParquetWriter('gaz2016zcta5distancemiles.parquet', schema=schema)
print(f'Starting conversion')

i = 0
for df in itr:
    i += 1
    print(f'Finished reading csv block {i}')

    table = pa.Table.from_pandas(df, preserve_index=False, nthreads=3)
    writer.write_table(table)

    print(f'Finished writing parquet block {i}')

writer.close()

Then running this python script produces the file gaz2016zcta5distancemiles.parquet, but just attempting to read the metadata with pq.ParquetFile() produces the above exception.
I tested this with pyarrow 0.8 and pyarrow 0.9. I assume that pandas would complain on import of the csv if the columns in the data were not string, string, and float64, so I think creating the Parquet schema in that way should be fine.

To make sure that the schema creation wasn't the issue, I rewrote the loop to be:
i = 0
for df in itr:
    i += 1
    print(f'Finished reading csv block {i}')

    table = pa.Table.from_pandas(df, preserve_index=False, nthreads=3)
    if i == 1:
        writer = pq.ParquetWriter('gaz2016zcta5distancemiles.parquet', schema=table.schema)

    writer.write_table(table)

    print(f'Finished writing parquet block {i}')

writer.close()

but I still get the same exception when trying to read the metadata as above.

Could you please open a JIRA with this information? Thanks

Closing as fixed (see JIRA ticket).


Invalid include in servedocs

I believe this is not valid:
https://github.com/asprionj/LiveServer.jl/blob/18170d003e732402b045ca90f5ad8748a88c0aab/src/utils.jl#L77 since make.jl typically using packages that are not available from within LiveServer.jl.

Hrm, what do you mean exactly?

packages that are not available from within LiveServer.jl

I'm not completely into the entire environments / projects stuff. When servedocs is invoked from a REPL using a certain environment where all the packages are available, there's no problems, are there? In what use cases could there be problems? How could these be circumvented? (Or, what would be the "correct" / clean solution?)

When servedocs is invoked from a REPL using a certain environment where all the packages are available, there's no problems, are there?

Yea there is. The include above is LiveServer.include, which is only allowed to load things from LiveServer/Project.toml. E.g. try this branch: https://github.com/fredrikekre/LiterateLive.jl/tree/fe/issue
which results in
julia> using LiveServer; servedocs()
┌ Warning: Package LiveServer does not have Literate in its dependencies:
│ - If you have LiveServer checked out for development and have
│   added Literate as a dependency but haven't updated your primary
│   environment's manifest file, try `Pkg.resolve()`.
│ - Otherwise you may need to report an issue with LiveServer
└ Loading Literate into LiveServer from project dependency, future warnings for LiveServer are suppressed.
✓ LiveServer listening on http://localhost:8000/ ...
  (use CTRL+C to shut down)

Or, what would be the "correct" / clean solution?

Not sure, three solutions:

Have the user input "their" include, e.g. servedocs(f = include). (Pretty ugly.)
Make servedocs a macro, which could then use the __module__ macro argument to determine the callers include. (Probably the most stable solution, although maybe a bit ugly with a macro).
Change include to Main.include. (Correct in 99% of cases or more I think, since users will call servedocs from the REPL)


(DEV-3688) Upgrades Expo SDK

Fixes [issue-number]
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

Error on pod install:
~/Documents/Projects/Apollos/ios (upgrade-expo) $ rm Podfile.lock
~/Documents/Projects/Apollos/ios (upgrade-expo *) $ pod install --repo-update
Updating local specs repositories
  $ /usr/local/bin/git -C /Users/michael.neeley/.cocoapods/repos/master fetch origin --progress
  remote: Enumerating objects: 62, done.
  remote: Counting objects: 100% (62/62), done.
  remote: Compressing objects: 100% (19/19), done.
  remote: Total 44 (delta 29), reused 36 (delta 22), pack-reused 0
  From https://github.com/CocoaPods/Specs
     696a6f15def..d005bd35e52  master     -> origin/master
  $ /usr/local/bin/git -C /Users/michael.neeley/.cocoapods/repos/master rev-parse --abbrev-ref HEAD
  master
  $ /usr/local/bin/git -C /Users/michael.neeley/.cocoapods/repos/master reset --hard origin/master
  HEAD is now at d005bd35e52 [Add] VietThucSwift 0.1.4

CocoaPods 1.7.0.beta.3 is available.
To update use: `gem install cocoapods --pre`
[!] This is a test version we'd love you to try.

For more information, see https://blog.cocoapods.org and the CHANGELOG for this version at https://github.com/CocoaPods/CocoaPods/releases/tag/1.7.0.beta.3

Analyzing dependencies
Fetching podspec for `DoubleConversion` from `../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`
Pre-downloading: `ExpoKit` from `http://github.com/expo/expo.git`, tag `ios/2.4.4`
Fetching podspec for `Folly` from `../node_modules/react-native/third-party-podspecs/Folly.podspec`
Fetching podspec for `React` from `../node_modules/react-native`
Fetching podspec for `glog` from `../node_modules/react-native/third-party-podspecs/glog.podspec`
Fetching podspec for `yoga` from `../node_modules/react-native/ReactCommon/yoga`
Removing GLog
Removing boost
Downloading dependencies
Using Amplitude-iOS (3.14.1)
Installing Analytics 3.6.10 (was 3.6.9)
Installing AppAuth 0.95.0 (was 0.92.0)
Using Bolts (1.9.0)
Using Branch (0.14.12)
Using CocoaLumberjack (3.2.1)
Installing Crashlytics 3.12.0 (was 3.10.1)
Using DoubleConversion (1.1.5)
Installing ExpoKit 2.4.4 (was 2.3.3)
Installing FBAudienceNetwork 4.99.3 (was 4.28.0)
Installing FBSDKCoreKit 4.44.1 (was 4.31.1)
Installing FBSDKLoginKit 4.44.1 (was 4.31.1)
Installing FBSDKShareKit 4.44.1 (was 4.31.1)
Installing Fabric 1.9.0 (was 1.7.6)
Installing Folly 2<IP_ADDRESS>
Using GPUImage (0.1.7)
Using Google-Mobile-Ads-SDK (7.22.0)
Using GoogleAppUtilities (1.1.2)
Using GoogleAuthUtilities (2.0.2)
Using GoogleMaps (2.2.0)
Using GoogleNetworkingUtilities (1.2.2)
Using GoogleSignIn (3.0.0)
Using GoogleSymbolUtilities (1.1.2)
Using GoogleUtilities (1.3.2)
Using JKBigInteger2 (0.0.5)
Installing React 0.54.2 (was 0.52.0)
Installing boost-for-react-native (1.63.0)
Installing glog (0.3.4)

[!] Error installing glog

――― MARKDOWN TEMPLATE ―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

### Command

/usr/local/Cellar/cocoapods/1.6.1/libexec/bin/pod install --repo-update

### Report

* What did you do?

* What did you expect to happen?

* What happened instead?

### Stack

   CocoaPods : 1.6.1
        Ruby : ruby 2.3.7p456 (2018-03-28 revision 63024) [universal.x86_64-darwin18]
    RubyGems : <IP_ADDRESS>
        Host : Mac OS X 10.14.3 (18D109)
       Xcode : 10.2 (10E125)
         Git : git version 2.20.1
Ruby lib dir : /System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib
Repositories : master - https://github.com/CocoaPods/Specs.git @ d005bd35e52d1d837d92e87c579f2fc5222fded1

### Plugins

cocoapods-deintegrate : 1.0.4
cocoapods-plugins     : 1.0.0
cocoapods-search      : 1.0.0
cocoapods-stats       : 1.1.0
cocoapods-trunk       : 1.3.1
cocoapods-try         : 1.1.0

### Podfile

source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '9.0'

target 'newspring' do
  pod 'ExpoKit',
    :git => "http://github.com/expo/expo.git",
    :tag => "ios/2.4.4",
    :subspecs => [
      "Core",
      "CPP",
      "GL"
    ]

  pod 'React',
    :path => "../node_modules/react-native",
    :subspecs => [
      "Core",
      "ART",
      "RCTActionSheet",
      "RCTAnimation",
      "RCTCameraRoll",
      "RCTGeolocation",
      "RCTImage",
      "RCTNetwork",
      "RCTText",
      "RCTVibration",
      "RCTWebSocket",
      "DevSupport",
      "CxxBridge"
    ]
  pod 'yoga',
    :path => "../node_modules/react-native/ReactCommon/yoga"
  pod 'DoubleConversion',
    :podspec => "../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec",
    :inhibit_warnings => true
  pod 'Folly',
    :podspec => "../node_modules/react-native/third-party-podspecs/Folly.podspec",
    :inhibit_warnings => true
  pod 'glog',
    :podspec => "../node_modules/react-native/third-party-podspecs/glog.podspec",
    :inhibit_warnings => true

  post_install do |installer|
    installer.pods_project.main_group.tab_width = '2';
    installer.pods_project.main_group.indent_width = '2';

    installer.pod_targets.each do |target|

    if target.pod_name == 'ExpoKit'
      target.native_target.build_configurations.each do |config|
        config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= ['$(inherited)']
        config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] << 'EX_DETACHED=1'

        # needed for GoogleMaps 2.x
        config.build_settings['FRAMEWORK_SEARCH_PATHS'] ||= []
        config.build_settings['FRAMEWORK_SEARCH_PATHS'] << '${PODS_ROOT}/GoogleMaps/Base/Frameworks'
        config.build_settings['FRAMEWORK_SEARCH_PATHS'] << '${PODS_ROOT}/GoogleMaps/Maps/Frameworks'
      end
    end

    if ['Amplitude-iOS','Analytics','AppAuth','Branch','CocoaLumberjack','FBSDKCoreKit','FBSDKLoginKit','FBSDKShareKit','GPUImage','JKBigInteger2'].include? target.pod_name
      target.native_target.build_configurations.each do |config|
        config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '9.0'
      end
    end
    # Build React Native with RCT_DEV enabled
    next unless target.pod_name == 'React'
    target.native_target.build_configurations.each do |config|
      config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= ['$(inherited)']
      config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] << 'RCT_DEV=1'
    end

    end
  end
end

### Error

Errno::ENOENT - No such file or directory @ rb_sysopen - /Users/michael.neeley/Documents/Projects/Apollos/ios/Pods/Local Podspecs/glog.podspec.json
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-core-1.6.1/lib/cocoapods-core/specification.rb:568:in `read'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-core-1.6.1/lib/cocoapods-core/specification.rb:568:in `checksum'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader/request.rb:62:in `slug'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader/cache.rb:114:in `path_for_spec'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader/cache.rb:140:in `cached_spec'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader/cache.rb:125:in `cached_pod'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader/cache.rb:33:in `download_pod'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/downloader.rb:42:in `download'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer/pod_source_installer.rb:104:in `download_source'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer/pod_source_installer.rb:62:in `install!'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:426:in `install_source_of_pod'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:377:in `block (2 levels) in install_pod_sources'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/user_interface.rb:86:in `titled_section'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:376:in `block in install_pod_sources'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:359:in `each'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:359:in `install_pod_sources'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:189:in `block in download_dependencies'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/user_interface.rb:64:in `section'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:188:in `download_dependencies'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/installer.rb:137:in `install!'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/command/install.rb:48:in `run'
/Library/Ruby/Gems/2.3.0/gems/claide-1.0.2/lib/claide/command.rb:334:in `run'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/lib/cocoapods/command.rb:52:in `run'
/usr/local/Cellar/cocoapods/1.6.1/libexec/gems/cocoapods-1.6.1/bin/pod:55:in `<top (required)>'
/usr/local/Cellar/cocoapods/1.6.1/libexec/bin/pod:22:in `load'
/usr/local/Cellar/cocoapods/1.6.1/libexec/bin/pod:22:in `<main>'

――― TEMPLATE END ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

[!] Oh no, an error occurred.

got through it! moving again...

@vinnyjth

@IsaacHardy any thoughts on why Netlify is failing?

closes #799


🔧 fix: safe conversion for union (#775)

Checklist

[ ] The title of my PR starts with a Conventional Commit prefix (fix:, feat:, docs: etc).
[ ] I have read the Contributor Guide and followed the process outlined for submitting PRs.
[ ] I have updated/added tests for new/updated/fixed functionality.
[ ] I have updated/added dartdoc comments with ///.

Breaking Change

[ ] Yes, this is a breaking change.
[ ] No, this is not a breaking change.

Related Issues

Codecov Report

Merging #776 (0961c25) into main (7717bd1) will increase coverage by 6.94%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #776      +/-   ##
==========================================
+ Coverage   80.61%   87.56%   +6.94%
==========================================
  Files         233        7     -226
  Lines        2884      193    -2691
==========================================
- Hits         2325      169    -2156
+ Misses        559       24     -535

see 226 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


The link does not work as expected

Hello,
the link Microsoft Cloud Red Teaming does not work as expected, it takes you to gallery sample code with a redirected URL: https://docs.microsoft.com/en-us/samples/browse/?redirectedfrom=TechNet-Gallery

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

ID: 4b783cd7-b822-5273-2f07-b53a762fd6aa
Version Independent ID: 2439ae01-ad2a-e790-5118-439c9f72f625
Content: Azure Security Control - Penetration Tests and Red Team Exercises
Content Source: benchmark/azure/security-control-penetration-tests-red-team-exercises.md
Service: security
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@SQLtattoo Thank you for pointing this, let me investigate and revert back.

@SQLtattoo Worked with the content author and fixed the hyperlink issue related to [Microsoft Cloud Red Teaming]. Thank you once again for pointing this out.


KeywordPrompt not working

The keyword prompt is throwing exception when run on my side and the package already exists. I don't have the chance to input a keyword.

V3 sample no longer uses keyword prompt.


Switched to a VPS

I switched to a VPS today instead of running it on my on computer. (Every weekend when I travel I would turn off my pc and people would get angry :P )
Since I'm not experienced with python I google how to let the script running after I closed the ssh connection.
I ended up starting it with nohup python nabbot.py
Is that the best/correct approach?

That should work according to this:
https://unix.stackexchange.com/a/4006
I don't know if there's a better way to do it.
What VPS are you using?

I'm using Digital Ocean with Ubuntu 17.04, $5 month


Sidebar nav collapses at mobile viewports

On mobile screens, the sidebar navigation is being displayed at the top of the page (right underneath the top nav) and is pushing content off-screen.
This should be either included into main navigation, or as a separate collapsable nav. (we need to define this before starting)

can you attach a picture of what you see and on what pages you see this? thanks!

@sigalsax I think we can delete this... this was about the subnav items in the documentation pages, but they're moving to Flare, so I think we can close this.

sure thing!


Themes appears to be broken on 2.7.1

This is  a bug.
Prerequisites

[x ] Are you running the latest version?
[x ] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[x ] Did you check the documentation?
[x ] Did you perform a cursory search?

Description
Themes are no longer being applied
Steps to Reproduce
Apply a new theme
Refresh the application
Expected behavior
Selecting a new theme should apply the new theme
Actual behavior
When you apply a theme, the href of the theme link is being set to undefined
When you refresh the application the chosen theme is not applied.

closed with https://github.com/laravel-enso/Core/commit/6bc31bfb0f72d1f6a7d8ccc1372423a8f5f75767


Regression in BLEU code

It looks like #1319 has caused a new set of regressions.
@josiahwang, @alvations – any input would be appreciated, thanks.

@stevenbird I'm recoding the BLEU from scratch with a lot more unit tests. I think that's a more maintainable solution than trying to improve the old one.

@stevenbird I'm checking the cpython code and it seems like the _normalize parameter in fractions.Fraction() is a new feature that's only available in Python >3.5, https://hg.python.org/cpython/file/3.5/Lib/fractions.py#l187
The default behavior before python 3.5 is to always normalize: https://hg.python.org/cpython/file/2.7/Lib/fractions.py
And it seems like this feature is not backported in older python versions. We've to come up with something else such that the bleu, smoothing and fractions can handle fringe cases.

I see, thanks @alvations – sorry to hear this is complicated!

I've got the _normalize parameter to work by overloading the class. And some try-except tricks will be needed after the imports to use the overloaded class.
try:
    fractions.Fraction(0, 1000, _normalize=False)
    from fractions import Fraction
except TypeError:
    class Fraction(fractions.Fraction):
        def __new__(cls, numerator=0, denominator=None, _normalize=True):
            self = super(Fraction, cls).__new__(cls, numerator, denominator)
            if not _normalize:
                self._numerator = numerator
                self._denominator = denominator
            return self

But I think it's best to revert to the BLEU without the _normalize first, then write the necessary tests first and patch the code with the overloaded Fraction to validate the tests.

@stevenbird The CI tests are failing for BLEU again because I think i accidentally merged the code that contains the new nltk.compat.Fraction at https://github.com/nltk/nltk/commit/9b74f56ebe61372cb30bcecf25b28c7b25d5dc87
We should either revert https://github.com/nltk/nltk/commit/9b74f56ebe61372cb30bcecf25b28c7b25d5dc87 or move it forward with #1332. Sorry about the mistake.

#1330 and #1488 should resolve this.


s7.govt m13.ar(aug, aur) m30.prose pira.ly iframe...'(27') xx0x90../

#7100 #7701 #5752 So, can you explain to me how to ig.no.ir aray.se/ce interspace.sh(;el')
def P.aram.sil [sylica] k.isis.gov(t)
curl jalalabad.h   @martins-vds @microsoftCSE  @jel.ly(1...msv4x)
ran.sum1 note.sh7, 8.0.2
mm
msvt.xxxx

Duplicate of #1111

it doesn't seem actionable.

t.sota ../a..ray.cc1 #h1tabletx
Hey maria cantwel. ⭐
On Wed, Nov 2, 2022, 7:43 PM Maira Wenzel @.***> wrote:

it doesn't seem actionable.
—
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/7920#issuecomment-1301554051, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A25PHC7ATYYI3C7VY62LYUTWGMKDTANCNFSM6AAAAAARVU2USA
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Bug][Relay] fix relay frontend pytorch op addmm bug

fix relay frontend pytorch op: addmm
calculation formula error.
bug:
out = input + alpha * beta * mat1 @ mat2
fix bug:
out = beta * input + alpha * mat1 @ mat2

@yelite

@shingjan @Hzfengsy

Please add a regression test for it

Please add a regression test for it

ok

https://ci.tlcpack.ai/blue/organizations/jenkins/tvm-lint/detail/PR-15294/5/artifacts
I don't know what the reason for his mistake is, and I don't know how to modify it。
I hope your can help me
@jikechao @Hzfengsy

---------Error Log----------
[2023-07-14T19:17:09.367Z] + cat /tmp/5893.logclean.txt
[2023-07-14T19:17:09.367Z] ----------------------------
[2023-07-14T19:17:09.367Z] + echo ----------------------------
[2023-07-14T19:17:09.367Z] + grep --quiet -E 'warning|error'
[2023-07-14T19:17:09.367Z] + cleanup
[2023-07-14T19:17:09.367Z] + rm -f /tmp/5893.log.txt /tmp/5893.logclean.txt
[2023-07-14T19:17:09.367Z] Checking ASF license headers...
[2023-07-14T19:17:09.367Z] + echo 'Checking ASF license headers...'
[2023-07-14T19:17:09.367Z] + tests/lint/check_asf_header.sh --local
[2023-07-14T19:17:12.651Z] NOTE: --local flag present, filtering untracked files
[2023-07-14T19:17:13.217Z] No files violate ASF header check
[2023-07-14T19:17:13.217Z] Linting the C++ code...
[2023-07-14T19:17:13.217Z] + echo 'Linting the C++ code...'
[2023-07-14T19:17:13.217Z] + tests/lint/cpplint.sh
[2023-07-14T19:17:13.217Z] Running 2 cpplints (VTA and TVM)...
[2023-07-14T19:17:13.217Z] All passed!
[2023-07-14T19:19:04.666Z] =====619/619 cpp-header files passed check=====
[2023-07-14T19:19:04.666Z] =====923/923 cpp-source files passed check=====
[2023-07-14T19:19:04.666Z] All passed!
[2023-07-14T19:19:04.666Z] + echo 'clang-format check...'
[2023-07-14T19:19:04.666Z] + tests/lint/git-clang-format.sh
[2023-07-14T19:19:04.666Z] clang-format check...
[2023-07-14T19:19:04.666Z] Ubuntu clang-format version 15.0.7
[2023-07-14T19:19:04.666Z] Running git-clang-format against all C++ files
[2023-07-14T19:19:51.337Z]
[2023-07-14T19:19:51.337Z] ---------clang-format log----------
[2023-07-14T19:19:51.337Z] clang-format did not modify any files
[2023-07-14T19:19:51.337Z] Rust check...
[2023-07-14T19:19:51.337Z] + echo 'Rust check...'
[2023-07-14T19:19:51.337Z] + tests/lint/rust_format.sh
[2023-07-14T19:19:51.337Z] Docker check...
[2023-07-14T19:19:51.337Z] + echo 'Docker check...'
[2023-07-14T19:19:51.337Z] + tests/lint/docker-format.sh
[2023-07-14T19:19:51.337Z] + cleanup
[2023-07-14T19:19:51.337Z] + rm -rf '/tmp/3764.*'
[2023-07-14T19:19:51.345Z] [Pipeline] }
[2023-07-14T19:19:51.359Z] [Pipeline] // withEnv
[2023-07-14T19:19:51.365Z] [Pipeline] }
[2023-07-14T19:19:51.379Z] [Pipeline] // timeout
[2023-07-14T19:19:51.385Z] [Pipeline] }
[2023-07-14T19:19:51.399Z] [Pipeline] // ws
[2023-07-14T19:19:51.404Z] [Pipeline] }
[2023-07-14T19:19:51.423Z] [Pipeline] // node
[2023-07-14T19:19:51.427Z] [Pipeline] }
[2023-07-14T19:19:51.445Z] [Pipeline] // parallel
[2023-07-14T19:19:51.453Z] [Pipeline] }
[2023-07-14T19:19:51.466Z] [Pipeline] // stage
[2023-07-14T19:19:51.476Z] [Pipeline] End of Pipeline
[2023-07-14T19:19:51.498Z] ERROR: script returned exit code 1
[2023-07-14T19:19:52.450Z]
[2023-07-14T19:19:52.451Z] GitHub has been notified of this commit’s build result
[2023-07-14T19:19:52.451Z]
[2023-07-14T19:19:52.869Z] Finished: FAILURE

Change the code format as follows:

[2023-07-14T19:08:15.007Z]  @tvm.testing.uses_gpu

[2023-07-14T19:08:15.007Z]  def test_addmm():

[2023-07-14T19:08:15.007Z]      def test_fn(alpha, beta):

[2023-07-14T19:08:15.007Z] -        return lambda inp, batch1, batch2: torch.addmm(

[2023-07-14T19:08:15.007Z] -            inp, batch1, batch2, beta=beta, alpha=alpha

[2023-07-14T19:08:15.007Z] -        )

[2023-07-14T19:08:15.007Z] +        return lambda inp, batch1, batch2: torch.addmm(inp, batch1, batch2, beta=beta, alpha=alpha)

@jikechao 3Q


Add reporter for format summary

Description
Add reporter for a format summary of one line per file. This reporter support colored output in same way as the plain reporter. Like other reporters it allows it content to be written to a separate file. The reported proceduces output when running in format and lint only mode. See examples below.
With lint only:
$ ktlint-dev --experimental --reporter=format --relative --color
Foo3.kt: Format required (2 violations need manual fixing)
Foo2.kt: Format required (1 violation needs manual fixing)
Foo1.kt: Format not needed (no violations found)
Foo4.kt: Format required (all violations can be autocorrected)

With format:
$ ktlint-dev --experimental --reporter=format --relative --color -F
Foo3.kt: Format not completed (2 violations need manual fixing)
Foo2.kt: Format not completed (1 violation needs manual fixing)
Foo1.kt: Format not needed (no violations found)
Foo4.kt: Format completed (all violations have been fixed)

Closes #821
Checklist

[X] PR description added
[X] tests are added
[X] CHANGELOG.md is updated

In case of adding a new rule:

[ ] README.md is updated
[ ] Rule has been applied on Ktlint itself and violations are fixed

Duplicate of #

@


webflux can not get the traceid in the logback

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?
SkyWalking 8.0.1
OS: linux
JRE: 1.8

Which company or project?

What happened?
i use webflux  can not get the traceid in the logback plugin

Requirement or improvement

Please describe your requirements or improvement suggestions.

If this thread has not been instrumented, then you can't get trace id from the context.

It is better to use getTraceId after the Spring Controller.


Add isSubsetOf

Data.Set has a fuction isSubsetOf :: Ord a => Set a -> Set a -> Bool. I needed the corresponding function for Data.HashSet, and it should be possible to implement it more efficiently than can be done through the public interface of Data.HashSet currently.

I'm also interested in having a isSubmapOf function for HashMap, just like the one from Map and IntMap. I have a few ideas but not sure which one is the best way to implement it. @tibbe

Implement isSubmapOf recursively over two HashMap, perform case-by-case analysis like current implementation of unionWith. This is complicated to implement and quite error-prone. I also have concern regarding commit https://github.com/tibbe/unordered-containers/commit/df6c50e1369d0ce76ff1b34776a0f5dc5edcc88c. What's the current status of the issue in this commit? Is BitmapIndexed guaranteed to have more than 1 and less than full children now?

Implement isSubmapOf like current implementation of (==), transform two HashMap into two lists of Leafs and Collisions with toList' (see https://github.com/tibbe/unordered-containers/commit/df6c50e1369d0ce76ff1b34776a0f5dc5edcc88c). The left is linear scanning and case analysis with only Leaf and Collision. The problem with this approach is that in order to run in strictly linear complexity, we need to change the bits in top layer from least-significant to most-significant, this way the keys in the list returned by toList' is sorted in ordinary Word order. This might hurt the performance if the range of hash key is not a full Word. But in exchange, we can also implement intersect and difference in linear time easily.

I would very much like a subset-testing function too.

Yes, this seems like a good idea. Let's do it the fastest way. I've been wanting to add mergeA for a while; can we do that and use it for this? I don't remember if it has the right shape to do the job efficiently.

isSubsetOf was added in #282 and released v<IP_ADDRESS>.


feat(substrait): add wildcard handling to producer

Rationale for this change
Current roundtrip tests for wildcard expression pass only because tests themselves optimize the plan beforehand and "optimize" the wildcard away before actually doing a roundtrip. Producer should be able to convert the plan to substrait even if the plan isn't optimized beforehand.
What changes are included in this PR?

Changes the wildcard test not to optimize the query beforehand
Injects ExpandWildcardRule at the top of the producer. Substrait doesn't have anything analogous to the wildcard, so the easiest solution I found was to run ExpandWildcardRule to work around that. The main thing is that it's run as part of the producer so the producer no longer assumes that full optimizer has been run before.

Are these changes tested?
yes
Are there any user-facing changes?
yes, producer can convert wildcard expression independently of the optimizer.

@alamb thanks. does the ci failure seem flaky to you? Or should I actually run docs update script?

Thanks @tokoko  and @vbarua for the review


Add .hidden class to discloseVisible selector

Fixes #1234 and build errors from recent builds. https://travis-ci.org/ExactTarget/fuelux/builds/58018262

Look alright


[CP] Fix of git

Commit(s) to merge
https://dart-review.googlesource.com/c/sdk/+/254923
Target
beta
Issue Description

depending on git Uri's of the form<EMAIL_ADDRESS>will crash pub.
dart pub global activate --source git URL crashes if the url is on the form<EMAIL_ADDRESS>
This is because Uri.parse throws on such uris - problem was introduced in https://github.com/dart-lang/pub/pull/3305 when we started using package:path/prettyUri to present Uri's in the output.
What is the fix
We catch the exception from Uri.parse and just present the Uri as is.
https://github.com/dart-lang/pub/commit/8ff63a8e7e36e70ceb96819c4dd9b3594b572302
We already did a fix of this issue in: https://github.com/dart-lang/pub/commit/a949b329b1b51f5f3973a790e0a0a45897d837de but missed two instances of path.prettyUri.
Why cherry-pick
This is a low-risk fix re-enabling users to depend on git packages via ssh.
Risk
low
Issue link(s)
https://github.com/dart-lang/pub/issues/3497, https://github.com/dart-lang/pub/issues/3520
Extra Info
No response

LGTM

SGTM

The linked changelist https://dart-review.googlesource.com/c/sdk/+/254921 is bad. I believe you meant https://dart-review.googlesource.com/c/sdk/+/254923 instead.

The linked changelist https://dart-review.googlesource.com/c/sdk/+/254921 is bad. I believe you meant https://dart-review.googlesource.com/c/sdk/+/254923 instead.

Yes indeed

This commit has been cherry-picked in 2.18.0-271.7.beta (0283de5506a5bd7d0e63e2092bf53ec93adddbd1).


Save audio from Telegram to Deep

This package should download audio from telegram and save it to deep

@Prinz-Eugen-Deep what is the name of your package?

https://www.npmjs.com/package/@deep-foundation/telegram_save_audio

Done at https://habr.com/ru/companies/deepfoundation/articles/801137


Can't convert Ch. 3

I'm getting this error. https://i.imgur.com/TvrTwZN.png . Any solution?

Please nevermind, it was a problem with the game. Worked it out already, thanks.

glad to hear it


Update Catalan translation

Update Catalan translation with latest strings file.

Coverage remained the same at 24.394% when pulling 69927117b22a2bb62783b65bbcc76f34ee122ae3 on Ereza:dev into a6d0e945fabab109fb133c59c21edc0e2adfb0b3 on NyaaPantsu:dev.


chore: updating jest to solve npm security issues.

I updated jest to solve over a 100 security concerns after npm installing.
I had to update the regex because for some reason it didn't seem to work with the previous one.

Codecov Report

Merging #39 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #39   +/-   ##
======================================
  Coverage    99.5%   99.5%
======================================
  Files          14      14
  Lines         204     204
  Branches       35      35
======================================
  Hits          203     203
  Partials        1       1

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 7a2236b...2b4eaa1. Read the comment docs.

I'll re-install with yarn tonight np

Oops I installed with my work's NPM registry 🤦♂

@JoviDeCroock Comments should be fixed, haha


SegsAdmin not working past the Splash screen.

I'm not sure exactly what breaks this, and when... but anyways, I cannot run SegsAdmin right now. I got to the Splash screen where the big SEGS logo is showing with a 'Loading' at the bottom, and then nothing happens.
Debug Log below, off a freshly cloned fork from my afk branch.
Debugging starts QMetaObject::connectSlotsByName: No matching signal for on_add_user() QMetaObject::connectSlotsByName: No matching signal for on_add_admin_user() QMetaObject::connectSlotsByName: No matching signal for on_generate_config_file() Checking for existing databases... qt.network.ssl: QSslSocket: cannot call unresolved function SSLv23_client_method qt.network.ssl: QSslSocket: cannot call unresolved function SSL_CTX_new qt.network.ssl: QSslSocket: cannot call unresolved function SSL_library_init qt.network.ssl: QSslSocket: cannot call unresolved function ERR_get_error qt.network.ssl: QSslSocket: cannot call unresolved function ERR_get_error Api retrieval and stored success ASSERT failure in QVector<T>::operator[]: "index out of range", file C:/Tools/Qt/5.10.1/mingw53_32/include/QtCore/qvector.h, line 432 QWaitCondition: Destroyed while threads are still waiting Debugging has finished
Tested on QtCreator 4.5.2 and Win10, might be necessary to add since the latest QtCreator I did made things wonky (the clang stuff responsible for that?) and I reverted back to avoid that problem. And of course, the OS might play a role as well. (Right now, I'm really not in the mood for Windows bashing, so let's leave it at that)

Looks like this is caused by a banjaxed Mingw32. Reinstalling it resolved the issue for me, so we can close this one down.


Allow custom prefixes for generated Python modules.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

This is ok to test

Hmm, any guidance on these failed tests would be welcome.  Some of the output is not super helpful.  Some of them read like server errors?

We have some flaky tests. CC @dgquintas.

Known issues: #10002 #9980 #9804 #9976 #9956 #9609
This list should be shorter very soon :/
But as far as this PR goes, consider testing to be OK.

Closing as superseded by pull request 10141.


import sky  - Anaconda No module named 'sky.view'; 'sky' is not a package

Anaconda distribution in python 3 does not find module sky. How to fix it? I install module today, I checked issues the version is newest.
(py36) C:\Users\User\Anaconda3\envs\py36\Lib\site-packages\sky>python
Python 3.6.4 |Anaconda, Inc.| (default, Jan 16 2018, 10:22:32) [MSC v.1900 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

import sky
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\User\Anaconda3\envs\py36\Lib\site-packages\sky\sky.py", line 5, in
from sky.view import view
ModuleNotFoundError: No module named 'sky.view'; 'sky' is not a package

Please try again, should be fixed as of 0.0.201

Installing collected packages: sky
Found existing installation: sky 0.0.199
Uninstalling sky-0.0.199:
Successfully uninstalled sky-0.0.199
Successfully installed sky-0.0.201


Should enable visualization of data specific portion of landing page for KBaseFBA.FBAModelSet object

This is a minor change that has been tested.  Note that this request pulls this fix into narrative-dev, not yet the develop branch.


Connecting to server session / auth cookie

Hi guys,
Do you have any solution for getting a current user or any info from session in the root value or in the context object?
Cheers

@simon2k I'm not sure what exactly you're asking for, but we recommend to do auth outside of GraphQL and simply adding the current user to the context. If you're looking for an example: here's how we do it in GitHunt-API.

@helfer Are there any examples of adding the user to the context for hapi?

I found an answer.
Turns out the graphqlOptions can be function.  Here is an example.
server.register({
    register: graphqlHapi,
    options: {
      path: '/graphql',
      graphqlOptions: (request) => {
        return {
          schema: schema,
          context: {user: request.auth.credentials},
          debug: true,
        }
      },
      router: { cors: true },
      route: { auth: 'session' },
    }
  })

correct. :)


MTLS disabled does not work with headless service

Bug description
MTLS is enabled globally on my cluster. I have a headless service with a statefulset of Cassandra nodes and they can't communicate even if I disable MTLS. #12551 says It doesn't work with MTLS but even without MTLS I can't get it to work. I followed https://github.com/arielb135/RabbitMQ-with-istio-MTLS and also https://itnext.io/musings-about-istio-with-mtls-c64b551fe104 but no success.
Expected behavior
Cassandra nodes should be able to communicate on internodes port. When I try to netcat to the port 7000  returns immediately. When Istio is disabled the connection stays open until I send characters.
Steps to reproduce the bug
You can deploy https://pastebin.com/raw/pgDSH0RN
and see that cassandra-1 fails cause it can't handshake with cassandra-1. But the issue is because it can't connect to itself.
You can exec into cassandra-0 and run :
apt-get update
apt-get install -y netcat net-tools
nc cassandra-0 7000

When Istio is disabled the last command does not return and accepts characters
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
client version: unknown
control plane version: 1.4.0
Client Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.8", GitCommit:"211047e9a1922595eaa3a1127ed365e9299a6c23", GitTreeState:"clean", BuildDate:"2019-10-15T12:11:03Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.3", GitCommit:"2d3c76f9091b6bec110a5e63777c332469e0cba2", GitTreeState:"clean", BuildDate:"2019-08-20T18:57:36Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
How was Istio installed?
Using helm
Environment where bug was observed (cloud vendor, OS, etc)
Docker for Mac, Kind
https://pastebin.com/raw/XPVNfZif (istioctl authn tls-check)
https://pastebin.com/raw/BzBf8Qpk (istioctl proxy-config cluster)

@hzxuzhonghu here is the ticket with the information you asked.

Sorry @cscetbon, i didn't get a chance to reproduce until now. The cause i found is that the cassandra bind 7000 port on pod ip , but envoy will proxy traffic only to <IP_ADDRESS>. So you should change config i think. But since i am not familiar with it, can not help on that.
$ netstat -pan |grep 7000
tcp        0      0 <IP_ADDRESS>:7000        <IP_ADDRESS>:*               LISTEN

Embrassed, https://istio.io/faq/applications/#cassandra

Same as #19280


Add support for Fairing

🚀 Feature

Find the link there: https://github.com/facebookresearch/fairring

Motivation

Pitch

Alternatives

Additional context

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging Pytorch Lightning, Transformers, and Hydra.

cc @borda

I've thought about this as well. IMO, the process group backend should be a constructor argument on the relevant strategies.
The logic here is pretty hidden, especially with the environment variable. Making this a constructor argument would be much more explicit: https://github.com/PyTorchLightning/pytorch-lightning/blob/478c6085dfc2163f13dd36fd9fcdb8c1f12f5f54/pytorch_lightning/strategies/parallel.py#L101-L106

I can work on this! (Making process group backend a constructor argument and then integrating with fairring)

The previous mechanism with the env var had the advantage of making the code hardware agnostic, i.e., you could set PL_TORCH_DISTRIBUTED_BACKEND="gloo" as fixed and freely choose the devices and hardware though e.g. CLI. Now, you would need to change the code and instantiate the strategy yourself, or register the custom strategy in the Registry.
I hope existing users will not be impacted by this.

@awaelchli - if there is pushback from users regarding the enviroment variable, we can always undeprecate it and continue supporting it. At the very least, this logic is now isolated to a single utility function under distributed.py


Improve isolation between test suites

Our main test script runs npm test scripts of all packages against the same instance of ganache-cli (with contracts deployed after start).
If some test suite doesn't manage snapshot correctly, another test suite can fail since the blockchain state isn't the expected one.
Another problem that can occur is related to the accounts that are shared between test suites: Nonce issues can occur if some test case doesn't handle that correctly.
An idea that could help to solve this issue is to start a ganache instance for each test suite, but it'll slow the test script a lot.
@pcowgill any idea?

Another idea: Try to force snapshot / nonce management thru mochaSetup.js using global hooks at all packages.

@marcelomorgado This is a good idea.
Haha I was just typing what I think is the same idea you just posted!
"Are there any commands to restore ganache to its default starting state at the start of any test suite? That might be faster than starting a new ganache instance"

A fix to be made: Ensure that blockchain will be reverted to the correct snapshot beforeEach test case.
Now, when an exception occurs inside a hook and before revertFromSnapshot it's breaking the test suite and demands ganache-cli restarting.

@marcelomorgado What's that quote from?

Oh, sorry I've just edited the comment above with the reference.


Makefile: add uninstall

Signed-off-by: Zhou Hao<EMAIL_ADDRESS>

LGTM, thought we'd need to remove the completions too, but they're apparently not installed with make install

updated.

LGTM and happy green test buttons.

@rh-atomic-bot r+


Directline webchat not working in React Strict Mode

I have a question
Minimizable web chat example from samples is not working in React Strict Mode.
conversations API is getting cancelled
React Version: 18.2.0
NodeJS: 18.19.1

Could you use the bug form?