When --offline flag specified for forc build, attempt to use local git checkouts for git dependencies before failing

Supporting forc build --offline is trivial for the case where the [dependencies] table in the Forc.toml manifest remains untouched, as forc can recreate the dependency graph entirely from the lock file. However, if the [dependencies] table is edited whatsoever, forc will try to recreate the whole graph by re-fetching dependencies.
This behaviour will be greatly improved by #1686 which makes more concise modifications to the package graph based on the difference between the dependencies read from the lock file and those read from the manifest. These means we can now at least support dependency removal and editing/adding path-sourced dependencies without failing in --offline mode.
We should also do our best to support editing/adding git dependencies in --offline mode.
Potential Approach
In the case that --offline was specified, we should attempt to find an existing git checkout for the specified dependency within ~/.forc/git/checkouts that adheres to the described git source.
For example, In the case that the following dependency was added:
foo = { git = "https://domain.com/owner/repo", branch = "bar" }

We should attempt to find all existing checkouts of the foo package whose commit is known to be of the bar branch from the https://domain.com/owner/repo repo and select the most recent one. Finding local checkouts for git sources that specify a tag or rev should be easier, as they should map to a specific commit.
We could possibly consider maintaining an index that maps from GitSource to the path of the latest known checkout for that git source. However, we should be weary that if we maintain an index it will need to handle the case where multiple processes or threads are attempting to read-from, add-to or edit it simultaneously. Ideally we shouldn't have to introduce a whole db crate just for this case.
Alternatively, when checking out local git repositories into ~/.forc/git/checkouts/, we can copy over the .git directory too (currently we omit it). Doing so would allow us to use the git2 crate to check whether the existing checkout is valid for the specified git source.
It's worth noting that searching and checking ~/.forc/git/checkouts without and index should not be a major performance hit, as we only require checking out directories that start with the same package name, and the search only needs to be performed once as consecutive calls to forc build will retrieve the dependency directly from the lock file.

I was planning to start this but moved formatter stuff to the front in my current work queue. Removing my assignment for now so that if someone wants to tackle it until we are done with formatter stuff. Otherwise, I can take a look after formatter as I am pretty excited about the offline support 😄

I implemented the logic for git sources other than branch as it seemed to be easier to start with and I am providing some benchmarks so that we can confirm the searching for a local version is manageable. For comparison i created an empty project which (implicitly) depends on std.
Building the empty project without searching for a local repo (and doing the fetch & pin stuff takes) 11.5 seconds on my system
Building the empty project with searching for a local repo takes 1.35 seconds on my system which feels like butter
I think it is worth searching for a local copy even if we are in online mode and if we cannot find one then we can fetch & pin (as a TODO suggests at the relevant part of the code).


Best practices for managing v0

Investigate and come up with a justification for the best practice for managing large datasets like v0

webdataset is looking pretty slick.
The specification is dead simple
https://docs.google.com/document/d/18OdLjruFNX74ILmgrdiCI9J1fQZuhzzRBCHV9URWto0/edit
The tar format ensures compatibility across platforms and allows for easy
creation, manipulation, and extraction of WebDataset files using standard tools.
The file naming conventions enable the grouping of related files into individual
data samples, identified by unique prefixes within the archive. This addresses
the "small file problem" common in deep learning, optimizing I/O and storage
utilization.

Benchmarking shows WDS just slightly behind TFDS.
https://github.com/huggingface/pytorch-image-models/discussions/1524#discussioncomment-4008520
But large datasets with TFDS seem a lot more complex. Requires using Apache Beam? https://www.tensorflow.org/datasets/beam_datasets#implementing_a_beam_dataset


added GA4 and GTM scripts

Description
Added scripts for Google Analytics 4 script and Google Tag Manager
Fixes # (issue)
Type of change.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

The Google Analytics tags were hard coded in by accident. I am submitting a new pr.


chore: polish cli login button & copy

📝 Changed routes:

/cli/login

(view full size)

Commit e0211671313bbd2bef1c4af4179c6ea93fc22ca5 (https://thirdweb-cuy7bgjki.thirdweb-preview.com).


IRGen: conform to the BlocksRuntime ABI

The BlocksRuntime ABI defines Block_layout_1 as <{ Int32, Int32 }> on
32-bit and <{ Int64, Int64 }> on 64-bit.  However, we were currently
treating it as <{ IntPtr, IntPtr }>.  This usually gets away with it as
it defined as IntPtrTy which matches this except on LLP64 targets.
Adjust it to match that.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

I think the ABI is wrong, honestly. Those fields are definitely both pointers. cc @rjmccall

Well.  I agree that the intent of the ABI is to declare those fields as pointer-sized rather than some sort of innate long-ish-ness that should be smaller on LLP64 targets.  They're just declared as unsigned long due to the longstanding UNIX-ism that unsigned long is pointer-sized.  However, Clang does indeed lay out this type using the target's unsigned long type, so in the short term, matching Clang is the right thing to do.
That said, as a Clang maintainer, I think we should just make a blocks-on-LLP64 ABI break to redeclare these fields as size_t / uintptr_t, and then reflecting that change in Swift will obviously be the right thing to do.

@rjmccall - cool; I think that I might try to take a look at changing clang for that.  I think that this will require a coordinated change in swift-corelibs-foundation, swift-corelibs-libdispatch, compiler-rt, and clang.  In the mean time, I think that we should go ahead and do this change to match clang's behaviour.


g5 instance types not yet added

AWS recently announced these new instance types, https://aws.amazon.com/ec2/instance-types/g5/

👋 @kcking - I believe G5 instance types are added. Are you not seeing them on your end?

 I had filtered the GPU Model column to nvidia but it looks like that column isn't yet filled out.

@kcking Ah - I think I know what that issue is. I believe GPU types for instances are filled in manually. Let me see if I can track this information down this weekend and get it added.

I've got all of this data ready to go with the exception of CUDA cores for each of the following instances. Trying to get someone from the AWS side to help out with this to get it finished up.
g5.xlarge
g5.2xlarge
g5.4xlarge
g5.8xlarge
g5.12xlarge
g5.16xlarge
g5.24xlarge
g5.48xlarge

Alrighty, thanks to Matthew Wilson from the AWS side I got this information and will roll this out later today: https://twitter.com/msw/status/1460325504045641729


Update with multicast limit note for vxlan

igmp_max_memberships must be increased to allow more than 20 mutlicast IPs (vxlan interfaces) per host

Did not verify, but LGTM.

Ah, believe me it IS legit, I confirmed it first hand (in production...) :(

@andrijapanic sure, I trust you :) Will merge this now, thanks.


Remove flickering for relayout actions

Every time we calculate the layout, the whole table flickers. Is there a way we can handle this? Maybe delay the rendering until the layout calculation is done? Or other ideas?
The perfect solution would of course be to have animated transitions explaining the changes. But I know that his will be difficult to achieve with the current implementation (also see #81).

@thinkh, you have the most web dev experience. What do you think?

See PR #337 for more information.

Does not happen anymore because of removed fisheye.


Warning about potential ambiguous conditional logic

Happend to see this code flagged...
https://github.com/visit-dav/visit/blob/408ac0bf09c1f74d5b0dc8565b3e9f5cfa157a8d/src/plots/Mesh/avtMeshPlotMapper.C#L294
on OSX with warning...
/Users/miller86/visit/visit/src/plots/Mesh/avtMeshPlotMapper.C:294:54: warning: '&&' within '||' [-Wlogical-op-parentheses]
            if (actors[i] != NULL && !labels.empty() &&
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
/Users/miller86/visit/visit/src/plots/Mesh/avtMeshPlotMapper.C:294:54: note: place parentheses around the '&&' expression to silence
      this warning
            if (actors[i] != NULL && !labels.empty() &&
                                                     ^

I believe the two lines or-ing the comparison of label strings need to be combined with parens () but I honestly wasn't sure.

Thanks Mark, that's the issue causing the regression failure.


Crash when request JSON SWIFT 3.0

xCode 8.1 beta, iOS 10.1 beta, swift 3.0, real device iPhone 5 (at simulator works great!)
crash when calling Alamofire.request(request).responseJSON{....}
Request.swift

// MARK: State

    /// Resumes the request.
    open func resume() {
        guard let task = task else { delegate.queue.isSuspended = false ; return }

        if startTime == nil { startTime = CFAbsoluteTimeGetCurrent() }

        task.resume()

        NotificationCenter.default.post(
            name: Notification.Name.Task.DidResume,
            object: self,
            userInfo: [Notification.Key.Task: task] <<<<=== EXC_BAD_ACESS
        )
    }

back trace
STAR thread #1: tid = 0x73404, 0x00a92bbe libswiftCore.dylibswift::metadataimpl::ValueWitnesses<swift::metadataimpl::OpaqueExistentialBox<0u> >::initializeWithCopy(swift::OpaqueValue*, swift::OpaqueValue*, swift::TargetMetadata<swift::InProcess> const*) + 12, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x2)     frame #0: 0x00a92bbe libswiftCore.dylibswift::metadataimpl::ValueWitnesses<swift::metadataimpl::OpaqueExistentialBox<0u> >::initializeWithCopy(swift::OpaqueValue*, swift::OpaqueValue*, swift::TargetMetadataswift::InProcess const*) + 12
frame #1: 0x00a8f9ba libswiftCore.dylibswift::OpaqueValue* tuple_initializeWithCopy<false, false>(swift::OpaqueValue*, swift::OpaqueValue*, swift::TargetMetadata<swift::InProcess> const*) + 38     frame #2: 0x008a6750 libswiftCore.dylibSwift._ArrayBuffer.getElement (Swift.Int, wasNativeTypeChecked : Swift.Bool) -> A + 112
frame #3: 0x008b7c30 libswiftCore.dylibSwift.Array.subscript.getter : (Swift.Int) -> A + 152     frame #4: 0x008b8d64 libswiftCore.dylibprotocol witness for Swift.Collection.subscript.getter : (A.Index) -> A.Iterator.Element in conformance  Swift.Array : Swift.Collection in Swift + 24
frame #5: 0x00a70548 libswiftCore.dylibprotocol witness for Swift.IndexableBase.subscript.getter : (A.Index) -> A._Element in conformance <A> Swift.Array<A> : Swift.IndexableBase in Swift + 24     frame #6: 0x0088fe18 libswiftCore.dylibSwift.IndexingIterator.next () -> Swift.Optional<A._Element> + 528
frame #7: 0x009091c4 libswiftCore.dylibstatic Swift._NativeDictionaryStorage.fromArray (Swift.Array<(A, B)>) -> Swift._NativeDictionaryStorage<A, B> + 436     frame #8: 0x00908f7c libswiftCore.dylibSwift.Dictionary.init (dictionaryLiteral : Swift.Array<(A, B)>...) -> Swift.Dictionary<A, B> + 80
STAR frame #9: 0x00443450 AlamofireRequest.resume(self=<unavailable>) -> () + 736 at Request.swift:205     frame #10: 0x00494e3c AlamofireSessionManager.request(urlRequest=, self=) -> DataRequest + 716 at SessionManager.swift:260
frame #11: 0x004948b4 AlamofireSessionManager.request(url=<unavailable>, method=<unavailable>, parameters=<unavailable>, encoding=<unavailable>, headers=<unavailable>, self=<unavailable>) -> DataRequest + 776 at SessionManager.swift:237     frame #12: 0x00423514 Alamofirerequest(url=, method=, parameters=, encoding=, headers=) -> DataRequest + 536 at Alamofire.swift:146
frame #13: 0x0013f4ec MyAppContentManager.(self=0x17653c70) -> ()).(closure #1) + 240 at ContentManager.swift:86     frame #14: 0x000d53d8 MyAppthunk + 56 at SELiveVC.swift:0
frame #15: 0x010e1d66 libdispatch.dylib_dispatch_call_block_and_release + 10     frame #16: 0x010e1d52 libdispatch.dylib_dispatch_client_callout + 22
frame #17: 0x010e65f0 libdispatch.dylib_dispatch_main_queue_callback_4CF + 1006     frame #18: 0x1e006a16 CoreFoundationCFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE + 8
frame #19: 0x1e004cfe CoreFoundation__CFRunLoopRun + 1422     frame #20: 0x1df54072 CoreFoundationCFRunLoopRunSpecific + 486
frame #21: 0x1df53e80 CoreFoundationCFRunLoopRunInMode + 104     frame #22: 0x1f700bfc GraphicsServicesGSEventRunModal + 156
frame #23: 0x2310e82e UIKit-[UIApplication _run] + 574     frame #24: 0x23108f60 UIKitUIApplicationMain + 150
frame #25: 0x0008abd8 MyAppmain + 172 at AppDelegate.swift:13     frame #26: 0x1d71e50a libdyld.dylibstart + 2

Sorry, we won't be able to look into this until we have devices with iOS 10.1. In the mean time, can you retest on the latest beta. This looks like it could be an underlying Swift issue.

Seeing this too on iOS 10.0.2

It doesn't happen consistently. Using the same params, it seems to only crash 50% of the time.

I am getting my app rejected due to this issue :C

On https:// challenge demand user client certificate. How can i pick certificate from profile (General/DeviceManagement/DigitalWorkSpace/Profile  and pass it to for?
The same certificate prompted on Safari Browser while accessing.

Any way we can help with this? iOS 10 has been out for quite some time. It'll be awesome if we can get this resolved soon. Our development is on a stand-still.

@AleyRobotics @pixelfreak @adrimarti I'm sorry, but I can't reproduce this issue at all. Our tests pass on a variety of simulator types with each optimization level. We're going to need to see the calling code that leads to this crash to be able to reproduce it. @pixelfreak You mentioned "using the same parameters", can we get a code sample with those parameters that causes this issue for you?

@jshier This does not happen in simulator. Only happens on a device.
There is nothing special with the call itself. It's just a vanilla request:
Alamofire.request("someHttpsURL", method: .post, parameters: params, encoding: JSONEncoding.default, headers: headers).responseJSON
{
    response in
}

@pixelfreak I'm sorry, but like I said, I tried to replicate on both device (iPhone 7, iPhone 4S) and the simulator. Can you put together a sample project that reproduces the issue, even if it's on device?

@AleyRobotics @jshier  I hava encountered a similar problem ,do you know how to fix it?

I have similar issue and due to this my app is rejected by Apple.The crash occurred on iPad devices while the app is working fine on iPhone devices. I am attaching the Apple crash log file.It seems that the error occurred in Request handler of Alamofire

Please try to fix this as soon as possible

Hey everyone,
I'm going to close this issue out because there's nothing for us to really go on here. We have been unable to reproduce this issue through numerous attempts and no one has posted a sample project that can reproduce the issue.
In the future, please provide more information so we're able to help you debug the issue. We want to help, but there's nothing here for us to actually help with.
Best of luck. 🍻

Anyone tried on iOS 9.1?


fix: denied user request to change textarea size

Users could easily deteriorate the entire website just by playing with message panel of contact page.
I fixed that by setting height of message panel to 8. Giving users a compatible area.
Also making sure your website integrity is maintained.
I do not believe raising issue was optimal for a 4 line of code. Just discuss here for any changes

I was actually working on responsiveness as per the issue from project.
But found this thing irritating me constantly.

we could also add only vetical sizing of textarea similar to what we have in github right here

I was actually working on responsiveness as per the issue from project.

Related to #14

we could also add only vetical sizing of textarea similar to what we have in github right here

Sure ! Sounds great .
Mention the issue #14 in your PR making it easier to understand what you are working on .

Thanks you @CodeTorso for the PR

Nice.


MINIFICPP-2318 Use libc++ in the clang CI build

We used to use libc++ as the standard library in the ubuntu_22_04_clang CI build, but this got accidentally lost in some recent change, and now we build with libstdc++.
This change reverts to using libc++.
https://issues.apache.org/jira/browse/MINIFICPP-2318

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

The ubuntu issue is because "ppa.launchpad.net" is down. https://status.canonical.com/


⚠ Binding element 'paginate' implicitly has an 'any' type

How do I get rid of the "Binding element 'paginate' implicitly has an 'any' type" error in VS Code for this line of code:
export async function getStaticPaths({ paginate }) {

I know I can set "noImplicitAny": false, but is there another solution?

Wrong place.


Internal Server Error with thumbs=true

The following request is producing an Internal Server Error:
Request:
https://api.nasa.gov/planetary/apod?api_key=[APY_KEY]&start_date=2020-09-12&end_date=2020-09-21&thumbs=true
Response:
{
  "code": 500,
  "msg": "Internal Service Error",
  "service_version": "v1"
}

Removing the 'thumbs=true' query parameter solves it

Same here.
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&date=&concept_tags=True&count=100&start_date=&end_date=&thumbs=true  500 (INTERNAL SERVER ERROR)

How about always return thumbnail_url, undefined if media_type == "image"?

@vikey89 is this still an issue? I'm attempting to reproduce and cannot get any error using &thumbs=true.

@JustinGOSSES I've been including &thumbs=true by default in an application I'm writing and I can't reproduce this. It's likely been fixed and this issue can be closed.
Link to the service that isn't erroring: https://github.com/HelixSpiral/apod-to-mqtt/blob/master/main.go#L34

I can't seem to find it in the documentation here anywhere, but when I include the count parameter and thumbs=true parameter, I am not getting thumbnail_url at all. Is there a way around this?


Sort prelude.cr

Add comments as to why order usually doesn't matter here.
Ref: https://github.com/crystal-lang/crystal/pull/2649#issuecomment-221891163

I'm not sure this should be placed in a comment, but rather in the language docs (that order doesn't matter much, and how the compiler works)

The comment is here for anyone else adding to this file - they shouldn't have to read the docs to know where to place the correct entry.
I don't think I'm qualified to document how the compiler works yet. :)

I agree that the comment is good here and that the change in general is good. It's specific to the compiler, a more general description should go to the language docs at the same time.
@asterite how strong do you feel abou this?

I think it's OK as it helps anyone wanting to add something to prelude. I'd also include a note to remember to add it to docs_main so it appears in the docs. But just that.

Let's merge this. I'll then add a reference to docs_main.


Logic inside config file

There is an if statement at the end of the config file, what is the purpose of that and can it be moved elsewhere? Generally you wouldn't want a config file to contain business logic, only variables and values.
In trying to parse the config file using Python everything I have tried is choking on this if logic.
I don't know enough about bash scripting to know, but I know that all I have ever seen is an empty.log file so I am wondering if this is running for anyone?
# Determine logfile name
# use the label of the DVD/CD or else use empty.log
# this is required for udev events where there is no media available
# such as ejecting the drive

if [ -n "$ID_FS_LABEL" ]; then
        LOGFILE=${ID_FS_LABEL}".log"
        elif [[ -n "$ID_CDROM_MEDIA_TRACK_COUNT_AUDIO" && $(which abcde-musicbrainz-tool) ]]; then
                LOGFILE=$(abcde-musicbrainz-tool --device "$DEVNAME" | cut -f1 -d ' ')".log"
        elif [[ -n "$ID_CDROM_MEDIA_TRACK_COUNT_AUDIO" &&  $(which cd-discid) ]]; then
                LOGFILE=$(cd-discid "$DEVNAME" | cut -f1 -d ' ')".log"
else
        LOGFILE="empty.log"
fi

# Set full logfile path
LOG=$LOGPATH$LOGFILE

I was able to work around this by only grabbing the lines with the variable assignment I need instead of the entire config file.

While not ideal to have logic in a config file, if no one else is concerned and I have worked around it reading from python, we can close this.

I think this is a valid issue, logic should be somewhere else.  Thanks.

Since it looks like it is dealing with naming the file differently for audio cds, this might be better placed int he identity.sh script. However, I just want to do a sanity check to make sure this logic is actually working? I ask because I have ripped all the different type and the only log file I have ever seen written to is empty.log
Is anyone else getting different log files? If not, we might just be able to remove it and just set the name. If so, we can find a better place I think.

Interestingly, I copied the whole log section of the config file to identify.sh just after it reads the config and the first DVD I inserted logged some stuff to empty.log and then created a logfile for the disc, something I have never witnessed before! It didn't log a bunch of stuff during the rip process (all the MKV stuff went to empty.log) but it was something...

Ok, I think I see how to make this work, the $LOG variable is being passed to the rip scripts, but it isn't being used, and I didn't yet comment out the parts of the config file that set it, so it is being set back to empty.log on each of these scripts.
I can't start on this now, but I think this should be a pretty straight forward PR, I'll try to get started on this soon. This should accomplish two things, get the logic out of the config file and get the separate log files per disc working correctly.


Feature request: Callback timeout

First of all, great module! +1
When migrating, there could be a chance when the done callback function is not called at all for whatever reason. It would be nice to have a timeout when migration up with CLI.
I have a bash script which runs on every deployment of my server. In that script I also have mm migrate. If for some reasons, the done function is not called, the mm migrate just waits.
Setting a timeout in the config file would an awesome feature!

Nice idea.
Unfortunately I can't guarantee I will have time for it in the nearest
future. If you're interested in sending a PR I'll be happy to review and
publish.
On Tue, Sep 20, 2016 at 5:13 PM Ali Yousuf<EMAIL_ADDRESS>wrote:

First of all, great module! +1
When migrating, there could be a chance when the done callback function
is not called at all for whatever reason. It would be nice to have a
timeout when migration up with CLI.
I have a bash script which runs on every deployment of my server. In that
script I also have mm migrate. If for some reasons, the done function is
not called, the mm migrate just waits.
Setting a timeout in the config file would an awesome feature!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/emirotin/mongodb-migrations/issues/20, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAgGCKbZSzloRLmyMTRa9Jqn-HZVjs1Gks5qr-nrgaJpZM4KBrP9
.

Published as 0.6.2. Thanks for the contribution again.


Update to latest Savi runtime (v0.20220912.0).

Resolves https://github.com/savi-lang/savi/issues/367

Looks like MacOS ain't happy with the -march=native -mtune=generic change.
I'll make a new runtime release with that disabled on MacOS...


tour: methods/25 fails

Context: https://tour.golang.org/methods/25
Run example and get error:
go: finding golang.org/x/tour latest
go: downloading golang.org/x/tour v0.0.0-20200201212631-8f38c9a8d074
go: extracting golang.org/x/tour v0.0.0-20200201212631-8f38c9a8d074
./prog.go:9:15: cannot use m (type Image) as type image.Image in argument to pic.ShowImage:
Image does not implement image.Image (missing At method)

Hi,
that page is an exercise. The title is "Exercise: Images" and it says:

Define your own Image type, implement the necessary methods, and call pic.ShowImage.

It won't compile until you solve the exercise correctly.
Closing here.


Add adamatch HF space and model

A little import fix and I added Hugging Face model and space links.

@sayakpaul Hi Sayak, do you have any suggestion on how to reproduce the original results?
Btw it may be beneficial to use the RandAugment version from KerasCV now that it's available.

Btw it may be beneficial to use the RandAugment version from KerasCV now that it's available.

Sure thing. I will try submitting a PR with that. Cc: @LukeWood

@sayakpaul Hi Sayak, do you have any suggestion on how to reproduce the original results?

I suspect this to be a version mismatch problem. But @johko mentions that even after 10 runs the performance stays there. Is it the same across different seeds?

Sorry for the late reply, I was out of office for a bit. I'll run some more experiments across different seeds, as @sayakpaul suggested and will report back on how it worked out.

Awesome. Thanks @johko! Appreciate your willingness.

cla/google Failing after 1s — ❌ Missing CLA from one or more contributors

Please sign the CLA.

@fchollet this one should be good now as well :)


Chewy DSL - how to do a geo-type filter?

How do I filter for objects within, say, 'x' km of a specific longitude-latitude pair? In JSON I would use something like
geo_distance: {         distance: "#{x}km",         coordinates: {           lat: "#latitude",           lon: "#longitude"         }       }
but how do I express this in the Chewy DSL?

Never mind - I figured it out. I can do something like this:
filter(geo_distance: {distance: "25km", coordinates: {lat: <latitude>, lon:<longitude>}})

Perfect :+1:


Correction in docs

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This is a docs update. This PR solves the mistake I found and reported in issue #629 . It's a very small mistake but reflects the prestige of the project, So, I hope you'll find this useful.

Why was this change needed? This PR is in context to issue #629 .

Other information:
Hope you find it useful and complying to the terms.

@holopin-bot @Raunakkumarr


doc page not found

http://PyMysqlPool.readthedocs.io/
broken url ?

sorry for that ,but will publish later.
thx for concern.


npm install fails with E401

When running npm install on the repo I'm getting an "Incorrect or missing password" error from npm.
Deleting the package-lock.json file resolves the issue and allows me to install the dependencies.
/code/ge/autocomplete-element main ❯ git log -1
commit 581c95f8d1ff6f81a9effb1fdf819afaf9b98599 (HEAD -> main, tag: v0.0.9, origin/main, origin/HEAD)

/code/ge/autocomplete-element main ❯ npm install
npm ERR! code E401
npm ERR! Incorrect or missing password.
npm ERR! If you were trying to login, change your password, create an
npm ERR! authentication token or enable two-factor authentication then
npm ERR! that means you likely typed your password in incorrectly.
npm ERR! Please try again, or recover your password at:
npm ERR!     https://www.npmjs.com/forgot
npm ERR!
npm ERR! If you were doing some other operation then your saved credentials are
npm ERR! probably out of date. To correct this please try logging in again with:
npm ERR!     npm login

npm ERR!      /Users/peter/.npm/_logs/2021-06-22T11_57_29_872Z-debug.log

Here's a docker one-liner which can be used to reproduce this issue:
docker run \
  --rm -it \
  -v "${PWD}:/code" \
  --entrypoint='bash' \
  node:latest \
  -c 'cd /code; rm -rf node_modules; npm install'

changing the last line to remove package-lock.json resolves the issue:
-c 'cd /code; rm -rf node_modules; rm package-lock.json; npm install'

something funky is going on with npm, possibly from back when either geocodeearth/autocomplete-element or geocodeearth/core-js were in development and were private.
I noticed for geocodeearth/ge that I needed to add the following settings to my package.json because npm appeared to publish org scoped packages as private by default.
 "publishConfig": {
    "access": "public"
  }

Thank you so much!!!  I tried so many other fixes that didn't work and this was the fix!  Thanks!


[APEX-209] Refactored monolithic deployInputStream function.

In an effort to better understand the flow of data within Apex, I have refactored the monolithic (200+ line long) deployInputStreams function to modularize it, add comments, and clarify operational flow.
This patch also includes formatting fixes per CodeStyle/CheckStyle guidelines for this function.
I would very much appreciate a review to confirm that method and variable names make sense and that comments appropriately capture what's going on. There has not been any modification of code (with the exception of duplicate instantiation of streamCodecIdentifier, streamCodec, sinkIdentifer, and sourceIdentifier in connectInputToBufferServer and connectInputToLocalSink. This was done for the sake of more concise method declaration. All other changes only involved method extraction.

Ilya, is splitting a method declaration part of our coding guidelines?

@243826 - this wasn't done explicitly based on the coding guidelines but was done in an effort to appeal to common best practices for coding. As an outsider, seeing this code base for the first time, it was difficult to make sense of it. Given the importance of building community and driving involvement from outside sources when expanding the number of contributors and committers, the more accessible and clear the code-base, the more likely it is to be maintained and improved in the future.

+1 for adding documentation and making things more modular

I think we need to be careful with code that is in the critical path of data transfer as we cannot sacrifice performance for better readability so we cannot make it a general rule. I need to look at this commit in a little bit more particular detail but it doesn't look like the code is in the critical path as it is around deployment of the operators.

@ilganeli the coding guidelines are there to leave no room for different interpretation of what commonly agreed best practice is. So before introducing personal flavors, please get it approved in the same fashion as the coding guidelines were approved.
jkl = xyz(a, b, c)

to
jkl =
  x(a
     b,
     c);

is very subjective in terms of value add.
You said it correctly that you are looking at the code for the first time so it's difficult. There is no expectation that people will get it right the first time they read it. After your reformatting, I am looking at the changes you made and now I find it difficult to comprehend what's going on :-)
Separately - please do not submit reformatting changes. There are  tools which can do it in one shot and probably save your and reviewers' time for better utilization elsewhere. May be if you want to work on these tools, I can share some code with you. Doing this piecemeal or without non format related changes just mucks with the history and context crippling the tools which help us utilize them. On top, it wrongly attributes it to a person who in some cases may not even understand the code.
If you are fixing a bug - then of course go ahead and reformat the newly written / modified code. Keeping the changes local will be nice as others working on the code will know what exactly changed. This will also make sure that we do not try to fix something that is not broken.

@243826 I think there is value in breaking up code into functions. At some points the original deployInputStreams function is nested 5 levels deep. That may be easy for you to read but for some people that is difficult to read. Maybe we can share these changes with a wider audience on the dev mailing list to get more opinions?
I also agree that it is important to preserve attribution. I know it is possible to change the author of a commit, maybe @ilganeli can provide a dummy commit which restores the original author of the code that has been moved to separate functions.
If you still feel this is not the place to be making these changes could you suggest a ticket for @ilganeli to work on, so that we can utilize his contributions?

@ilooner @243826 I think maintaining provenance of the code is definitely important, how would you recommend submitting a patch that maintains this?
With regards to formatting, the formatting in this patch is consistent with the style guide and check style as it stood when I submitted the patch. Since both of those have undergone significant changes since that point, I've just submitted an update which brings this code up to date.
@PramodSSImmaneni I agree that touching critical path code is risky, hence my ask for a comprehensive review. If it proves too difficult to analyze this patch, I could submit a series of smaller patches based on this work that does this in a series of smaller steps.

Hi,
There were a couple of bugs in the configuration settings of IntelliJ IDE and I apologize for that. I think a part of the discussion here arose because of these bugs. I have made fixes to IntelliJ settings in devel-3.
Please let me know if you see more discrepancies.

In general I would suggest to avoid auto-format in IntelliJ or any other IDE. It may be used for the new code, but IMO bulk/blind updates to existing code should be avoided due to possible bugs or unwanted behavior of IDE auto format feature (I am not even sure that all IDEs provide the same functionality). As an alternative I would recommend codestyle plugins available for all major IDEs. For IntelliJ I use CheckStyle-IDEA and it can identify all existing violations.

@ilganeli this can change the author of a commit
git commit --amend --author="User Name<EMAIL_ADDRESS>I think there are two options for doing this. You can attribute the entire change to the original author or you can attribute the original code sections to the original author and the new functions to you. This could be done by submitting two commits. The first commit will contain all your changes and something like a // at the end of each line of original code. The second commit would remove the // at the end of each line of original code and will be attributed to the original author. The second commit could have a message like "restoring original author"
I suggest posting this pull request to the dev mailing list to get more feedback about the readability and cleanliness of the change, since there seem to be some differing opinions.

I think the issue is not about author of the commit. It is more about coding guidelines. If there are approved coding guidelines that are being followed for this fix, then we need to follow same guidelines at couple of other places where there are monolithic functions

Any AI on this PR? or should this be closed?


Check In

@bryanmtl and @peterberkenbosch just checking in here to see if the kickstart of this looks ok. I am still trying to wrap my brain around the fosdick thing. I guess just start by testing that we can create a csv out of order data?

What do you think of grabbing one of the CSVs that Wombat makes available to Fosdick, and using it as the "finish line"? You could reverse-engineer the order data to arrive at the "starting line" and then drive out the in-between with specs.

@DanielWright Good idea. @braidn If you need one of these, I'll ship one to you this morning.

Absolutely need one! Thanks @bryanmtl

@braidn should be a fair amount of samples here: https://github.com/DynamoMTL/fosdick_integration


⚠ ClamAV has degraded performance

In 863429a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 447 ms

Resolved: ClamAV performance has improved in 5205c1a after 5 minutes.


[Substrate.io] Deterministic Runtime

The "Deterministic Runtime" page was migrated from Substrate.io:  https://docs.substrate.io/build/build-a-deterministic-runtime/
Note that the instructions in this page work as expected.

This PR is still relevant and can go under the new Deployment section of the IA


turms-v0.9.0-SNAPSHOT.20200406 can not run, ClassNotFound

envement:
centos7
jdk11
turms-v0.9.0-SNAPSHOT.20200406

[root@HK1003-B4 bin]# sh turms.sh
+ exec /usr/java/jdk-11.0.7/bin/java -Xms1g -Xmx1g -Dio.netty.allocator.type=unpooled -XX:MaxDirectMemorySize=536870912 -Dspring.config.location=/root/turms-v0.9.0/config/application.yaml -cp '/root/turms-v0.9.0/lib/*' im.turms.turms.TurmsApplication
Error: Could not find or load main class im.turms.turms.TurmsApplication
Caused by: java.lang.ClassNotFoundException: im.turms.turms.TurmsApplication

please help me ~~ !!, thanks everybody.

Short answer: you may need to add a parameter "-f"(means running turms as a fat jar).
BTW:

You may see another error later (something like "hazelcast.config is not found" ). If so, you need to specify the correct path of "spring.hazelcast.config"
(BTW: Hazelcast will be removed in the next release because of the reasons mentioned in https://github.com/turms-im/turms/issues/370 if you are interested)
It's better to use pooled direct memory instead of unpooled direct memory or you may notice frequent memory churn.


Logged-in Users are seeing incorrect call counts

A few complaints from iOS folks recently that they've signed in and their call counts have dropped from their previous totals.
At first I thought this was the backend code for uploading stats for users, but it appears that it works fine. For example, here's the stats I see on the backend for a user:
{
  "stats": {
    "contact": 316,
    "voicemail": 506,
    "unavailable": 27
  },
  "weeklyStreak": 35
}

They report that the impact screen shows 35 week streak (correctly), but only 3 calls made. Perhaps the stats are being displayed based on local data and not remote data?

Indeed that is the case. It's just looking at the saved pantry store. Looking at where this data is fetched / stored locally.

There's a call to https://api.5calls.org/v1/users/stats which is where the remote stats are coming from, so we should be getting the right data there.
I know we opted to keep some features relying on local data... if this isn't working correctly it might be worth using the remote data entirely.

On 2nd look, I take that back. It is merging these 2 types of data, but the server stats are not being fetched on this screen. I think I can get a quick PR up to address this.

Ah, it was fetching, but in a roundabout way (posting a profile changed notification manually).

Made some changes while looking into this #228, it's working for me so I'm not sure what else to do for now. Everything looks okay.

Merged #228 in for this. Thanks @subdigital


Failed to load the tool-bar package

[Enter steps to reproduce:]

.Open Atom

Atom: 1.52.0 x64
Electron: 6.1.12
OS: Mac OS X 10.15.7
Thrown From: tool-bar package 1.4.0
Stack Trace
Failed to load the tool-bar package
At '../dist/iconsets.css' wasn't found. Tried - /Users/dani/.atom/packages/tool-bar/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/base16-tomorrow-dark-theme/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/static/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/dist/iconsets.css,../dist/iconsets.css in /Users/dani/.atom/packages/tool-bar/styles/tool-bar.less:2:0

LessError: '../dist/iconsets.css' wasn't found. Tried - /Users/dani/.atom/packages/tool-bar/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/base16-tomorrow-dark-theme/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/static/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/dist/iconsets.css,../dist/iconsets.css
at /Users/dani/.atom/packages/tool-bar/styles/tool-bar.less:2:0

Commands

Non-Core Packages
compare-files 0.8.1
ide-json 0.2.1
language-docker 1.1.8
language-markdown 0.37.0
markdown-scroll-sync 2.1.2
markdown-toc 0.4.2
markdown-writer 2.11.10
pp-markdown 0.0.2
pretty-json 2.1.2
tidy-markdown 3.0.1
tool-bar 1.4.0
tool-bar-markdown-writer 0.5.1
xml-formatter 0.13.0

I created the same issue nearly at the same time. I've closed mine to relocate to add to this one:
[Steps to reproduce:]

Update to Atom 1.52.0, then
Restart Atom.
(Same issue after reinstalling tool-bar.)

Atom: 1.52.0 x64
Electron: 6.1.12
OS: Mac OS X 10.15.6
Thrown From: tool-bar package 1.4.0
Stack Trace
Failed to load the tool-bar package
At '../dist/iconsets.css' wasn't found. Tried - /Users/echan/.atom/packages/tool-bar/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-syntax/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/static/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/dist/iconsets.css,../dist/iconsets.css in /Users/echan/.atom/packages/tool-bar/styles/tool-bar.less:2:0

LessError: '../dist/iconsets.css' wasn't found. Tried - /Users/echan/.atom/packages/tool-bar/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-syntax/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/static/dist/iconsets.css,/Applications/Atom.app/Contents/Resources/app.asar/dist/iconsets.css,../dist/iconsets.css
at /Users/echan/.atom/packages/tool-bar/styles/tool-bar.less:2:0

Commands

Non-Core Packages
hyperclick 0.1.5
indent-detective 0.4.0
ink 0.12.5
julia-client 0.12.6
language-julia 0.19.3
latex-completions 0.3.6
save-workspace 0.5.0
Sublime-Style-Column-Selection 1.7.5
tool-bar 1.4.0
uber-juno 0.3.0

[Enter steps to reproduce:]

...
...

Atom: 1.48.0 x64
Electron: 5.0.13
OS: Linux Mint
Thrown From: tool-bar package 1.4.0
Stack Trace
Failed to load the tool-bar package
At '../dist/iconsets.css' wasn't found. Tried - /home/marcus/.atom/packages/tool-bar/dist/iconsets.css,/usr/share/atom/resources/app.asar/node_modules/one-dark-syntax/dist/iconsets.css,/usr/share/atom/resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/usr/share/atom/resources/app.asar/static/dist/iconsets.css,/usr/share/atom/resources/app.asar/dist/iconsets.css,../dist/iconsets.css in /home/marcus/.atom/packages/tool-bar/styles/tool-bar.less:2:0

LessError: '../dist/iconsets.css' wasn't found. Tried - /home/marcus/.atom/packages/tool-bar/dist/iconsets.css,/usr/share/atom/resources/app.asar/node_modules/one-dark-syntax/dist/iconsets.css,/usr/share/atom/resources/app.asar/node_modules/one-dark-ui/dist/iconsets.css,/usr/share/atom/resources/app.asar/static/dist/iconsets.css,/usr/share/atom/resources/app.asar/dist/iconsets.css,../dist/iconsets.css
at /home/marcus/.atom/packages/tool-bar/styles/tool-bar.less:2:0

Commands

Non-Core Packages
atom-ide-ui 0.13.0
atom-latex 0.9.1
autocomplete-bibtex 1.2.10
busy-signal 2.0.1
compare-files 0.8.1
file-icons 2.1.43
gitlab 0.1.3-7
highlight-selected 0.17.0
ide-python 1.5.0
intentions 1.1.5
language-latex 1.2.0
language-pfm 0.82.0
language-r 0.4.2
latex 0.50.2
linter 3.1.1
linter-chktex 1.4.0
linter-ui-default 2.1.2
linter-write-good 0.9.3
markdown-deluxe 1.0.0
markdown-fold 0.3.0
markdown-folding 0.3.1
markdown-img-paste 0.4.0
markdown-mindmap 0.6.2
markdown-preview-plus 4.8.2
markdown-table-formatter 2.9.2
markdown-toc 0.4.2
markdown-writer 2.11.10
minimap 4.29.9
multi-cursor 3.0.0
open-file 2.1.0
open-recent 5.0.0
open-unsupported-files 1.0.20
pandoc-convert 1.1.1
pdf-view 0.72.0
pen-paper-coffee-syntax 0.17.0
platformio-ide-terminal 2.10.0
process-palette 0.17.0
project-manager 3.3.8
sync-settings 5.0.5
tool-bar 1.4.0
tool-bar-markdown-writer 0.5.1
wordcount 3.2.0
Zen 0.18.0
zotero-citations 5.2.0

Oh sorry for causing this issue! I just released 1.4.1 that should fix it!


Failed to execute any maven operation to Alluxio

Alluxio Version:
What version of Alluxio are you using?
v2.1
Describe the bug
A clear and concise description of what the bug is.
Failed to execute any maven operations on Windows.
To Reproduce
Steps to reproduce the behavior (as minimally and precisely as possible)
I'm using Intellij IDEA2019.3.3 and Maven 3.6.3 on Win10.
I tried to run "maven clean" but failed. The error info is "Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.4.0:exec"
I found out there's a linux instruction "rm".
Expected behavior
A clear and concise description of what you expected to happen.
I checked out Alluxio's website and found this:

So I want to know if Alluxio could be build with maven on Windows and run on Linux.
Urgency
Describe the impact and urgency of the bug.
Additional context
Add any other context about the problem here.

If it's feasible to build alluxio on Windows, I also want to know how to fix the problem：
"Failed to execute goal org.codehaus.mojo:exec-maven-plugin:1.4.0:exec".

Hi @justKidrauhl, Alluxio does not currently support any windows platforms. You'll need to build and run Alluxio on a Linux or MacOS-based operating system.
Another option which you can try (but is not supported), is to compile directly through the command line using WSL
You can also try using VirtualBox to launch a CentOS/Ubuntu VM which you can then install IntelliJ and maven on.
Hope this helps. I'm closing this issue because we don't support windows, and I don't think that we will. If you still have issues compiling on other platforms you can re-open this issue


XP system - implement kind words rewards

View the below image for general acceptance criteria:

Verified on prod


Describe how there are dependencies in access rights

There should be some information about dependencies in access rights as described in this OLD doc:
Docs / Dynamics 365 / Customer Engagement (on-premises) / version 9 / Developer Guide / Use record-based security to control access to records / Dependencies between access rights
You must have READ access to do just about anything. Simply granting WRITE is not enough.

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

ID: 3c40f87c-1a3a-399d-d25f-ae348dc5e248
Version Independent ID: 4784510f-a482-f73a-85e2-f2e6d74168c3
Content: Create or edit a security role - Power Platform
Content Source: power-platform/admin/create-edit-security-role.md
Service: power-platform
GitHub Login: @paulliew
Microsoft Alias: paulliew

Security content rewrite is ongoing and will include this.


How to force a snapshot?

Currently snapshot will be taken at the configured snapshot distance. How can force a snapshot at will?
creatsnapshot() requires snapshot "s". How to get this?
void create_snapshot(snapshot& s,  async_result::handler_type& when_done)

Hi @kishorekrd
Currently, there is no way to create a snapshot at will. Your state machine can create snapshots anytime though, but those non-regular ones won't be used by NuRaft. It will be useful if there is an API like raft_server::create_snapshot(uint64_t log_idx). I will put it into the backlog.

I kind of solved it by setting snapshot distance to 1 and add my logic to chk_create_snapshot() to take decision on the snapshot at will. But API should be good. Thanks


When v-bind:center changes it forces v-bind:zoom to restart

When you bind the :center of the map to your vue data coordinates and you change it later (in order to pan the map, for example), the :zoom of the map also restarts to its original value, if it was changed.
This should be two separated processes.

Hi @fergardi,
For the moment only one way binding is supported. This means that if you change you values it will update the map but if you modify the map it will not update you values.
The only method in leaflet, as far as I know, to change center is to call setView that takes a center and a zoom. It currently use the value of the zoom you provide not the zoom the map is currently in.
I can think of 2 fix for this issue :

Make two way binding works on components.
Make the call to setView use the map zoom value instead of the zoom value provided.

@KoRiGaN could we support an event like "zoom:update" that pushes the new zoom level up to the parent. That would allow anyone that wanted to maintain parity to do so.
This is definitely a feature I am looking for.

So... right after I wrote that last comment, I remembered that most leaflet events actually are propogated up... and what do you know, this totally worked, thanks!
<v-map v-if="latlng && zoom" :zoom="zoom" :center="latlng" v-on:l-zoomend="({target: {_zoom: v}}) => zoom = v">

Hi @jteppinette,
Yes updating current zoom localy when a zoom event happen can also fix that issue.

Would this be related to the map re-centering when any data on the component changes? Like snapping back to the center value?

@fergardi, @jteppinette,
From v0.0.60 you can use .sync on the zoom value for it to be updated automatically from the map.
Hope this helps.
Micka

Closing as no comments for 3 weeks.
Please reopen this issue if it not solved yet.


[HELP] STM32 Macro Naming Conventions

Description
There seems to be two naming conventions when defining macros for STM32 boards:

Identifiers typically omit chip type, and start with STM32_

An example would be the STM32H7. In arch/arm/src/stm32h7/hardware/, and the board.h file for the nucleo-h743zi (boards/arm/stm32h7/nucleo-h743zi/include/board.h

Identifiers typically include chip type, and start with STM32XX_

An example would be the STM32L5. In arch/arm/src/stm32l5/hardware/, and the board.h file for nucleo-l552ze (boards/arm/stm32l5/nucleo-l552ze/inclue/board.h

I cannot figure out why some chips use the first style, and some use the second.
Are the differences between these naming conventions purely stylistic, or are there specific reasons to use one convention instead of the other?
When adding support for a new STM32 chip family, which naming convention would be preferred?
Verification

[X] I have verified before submitting the report.

Please use the shorter version: STM32_. The longer version comes from stm32l4 port and the ports using it are most likely based on stm32l4 port (or any of its later derivatives).
The shorter version is easier to maintain, helps when we have to update all stm32 chips at once and porting between stm32 boards is then easier.
I think STM32XX_ should be modifed to STM32_ someday.

Should function and file names follow this shorter version as well then?


[FIX] fix empty result for '--diff' and print style

Coverage remained the same at 71.721% when pulling 24a04eee41a551eb83c49496be9a8f357af04f59 on spenly:fix into 5707a39df693ddd243ddd7c47ff35ce66d25a6c6 on bndr:master.


Add message when widget script is not found on CDN

For #11247

Codecov Report

Merging #11249 into master will decrease coverage by 0.00%.
The diff coverage is 89.47%.

@@            Coverage Diff             @@
##           master   #11249      +/-   ##
==========================================
- Coverage   61.08%   61.07%   -0.01%
==========================================
  Files         601      601
  Lines       33085    33104      +19
  Branches     4671     4673       +2
==========================================
+ Hits        20209    20219      +10
- Misses      11858    11865       +7
- Partials     1018     1020       +2

Impacted Files
Coverage Δ

src/client/telemetry/index.ts
87.38% <ø> (ø)

...cience/ipywidgets/ipyWidgetScriptSourceProvider.ts
88.88% <88.23%> (-0.13%)
:arrow_down:

src/client/common/utils/localize.ts
95.35% <100.00%> (+0.02%)
:arrow_up:

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

...client/datascience/kernel-launcher/kernelFinder.ts
75.70% <0.00%> (-0.94%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
: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 acdc910...7b18c1f. Read the comment docs.


createResource onHydrated option not called during hydration

Describe the bug
The onHydrated option in createResource is not called during hydration.
Your Example Website or App
https://github.com/katywings/solid-on-hydrated-repro/blob/master/src/routes/index.tsx
Steps to Reproduce the Bug or Issue

Setup solid with ssr (e.g. using solid-start)
Change a route and add a createResource using the onHydrated option
Open up the route in the browser
onHydrated will not be called during client hydration

Expected behavior
The onHydrated option in createResource should be called during hydration (if I understand the usecase for this option properly)
Screenshots or Videos
https://user-images.githubusercontent.com/4012401/167256657-ee591909-bdd0-4260-9937-bad0265f4a85.mp4
Platform

OS: [Linux]
Browser: [Firefox]
Version: [1.3.17 and 1.4.0-beta.2]

Additional context
Potential solution:
During my debugging I noticed, that the variable p is empty during hydration. But the variable v is set and has the proper value. Replacing p with v at the following line, would solve the problem and make it so that onHydrated is actually called:
https://github.com/solidjs/solid/blob/1ceb6079c15f0db0af1323a0ee448ced460b5ab2/packages/solid/src/reactive/signal.ts#L523
But tbh. i don't understand the full createResource logic... changing p to v might break other things 😅.
Video of my proposed solution:
https://user-images.githubusercontent.com/4012401/167256750-0e72d4c8-83be-45ee-bf4a-af2c1c457b43.mp4

Sidenote: this is related to the discord discussion about TurboSolid: https://discordapp.com/channels/722131463138705510/722167424186843267/971362009184620584

P.S: @ryansolid Apart from above issue, I noticed a behaviour about onHydrated that maybe should be documented (when onHydrated is officially announced :D)
Signal setters don't work inside of onHydrated, you have to wrap them with raf or setTimeout:
https://github.com/katywings/solid-on-hydrated-repro/blob/master/src/routes/index.tsx#L14

Yeah it's a timing thing. It can be the value or the promise. So we should check either. Simple fix. It will be in 1.4. What is more interesting is the signal setting. It is due to still being hydration at that time. Which means it does trigger the graph, just rendering is skipped. I will fix that too I guess. I was thinking we needed to set this earlier but it's fine I think push it to the end of hydration.

It can be the value or the promise. So we should check either. Simple fix

Awesome ❤, thank you 🙏!

EDIT: Hmm.. yes it is more awkward if it can change data that hasn't be read yet in downstream portions not yet hydrated like during streaming. It would be better not to try to set signals here.

Yeah I think it makes sense like this, kinda similar to how batching works "glitch-free" 😉.


Svelte example

In addition to React and Vue examples, it would be helpful to have a Svelte one also (for those looking for an alternative to SvelteKit or to roll their own.)

Agreed & PR much welcome
On Sat 29. May 2021 at 18:42, Aral Balkan @.***> wrote:

In addition to React and Vue examples, it would be helpful to have a
Svelte one also.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/brillout/vite-plugin-ssr/issues/79, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHVQRQFWF3HDS2YV4UPN63TQEKQNANCNFSM45YOOVQQ
.

Done, see https://vite-plugin-ssr.com/svelte

Thanks @aral !


Replace enum package example with native enum

This took me quite a while before I understood to read the source code
Relevant code https://github.com/spatie/laravel-data/blob/5e170162c573085b7529cd1cd8881201f22b5110/src/Casts/EnumCast.php#L12-L15

Actually this seems to be bigger problem with this page since spatie/enum package isn't supported .... I'll write some example caster for that.

I decided to split this example to another PR #445 in case it is not needed

Would love to see this, as Spatie also serves as a great example on "how to" in the Laravel/PHP world :)

Thank you!


Linker command failed - library not found for -lPods-Responder

My ionic project called Responder compiles fine with:
ionic cordova build ios.
However, it will not compile in xcode due to a Pods linking issue.   Logs are at the bottom.  I tried removing the platform and re-adding like this "ionic cordova platform add<EMAIL_ADDRESS>but same issue.
I have no Pods scheme in my Responder.xcodeproj but I do in Responder.xcworkspace project and workspace builds.  I installed cocoapods and ran pod setup.
Any idea what step I am missing?

Cordova CLI version and cordova platform version
global packages:
@ionic/cli-utils : 1.2.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.2.0

local packages:
@ionic/app-scripts              : 1.3.7
@ionic/cli-plugin-cordova       : 1.2.1
@ionic/cli-plugin-ionic-angular : 1.2.0
Cordova Platforms               : ios 4.4.0
Ionic Framework                 : ionic-angular 3.0.1

System:
Node       : v7.4.0
OS         : macOS Sierra
Xcode      : Xcode 8.3.2 Build version 8E2002
ios-deploy : 1.9.1
ios-sim    : 5.0.12

Plugin version
cordova plugin version | grep phonegap-plugin-push   # e.g. 1.5.3

Sample Push Data Payload
Sample Code that illustrates the problem
Logs taken while reproducing problem
Ld /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Products/Debug-iphonesimulator/Responder.app/Responder normal x86_64
cd /Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios
export IPHONEOS_DEPLOYMENT_TARGET=9.0
export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -arch x86_64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator10.3.sdk -L/Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Products/Debug-iphonesimulator -L/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GGLInstanceID/Libraries -L/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GoogleCloudMessaging/Libraries -F/Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Products/Debug-iphonesimulator -F/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GoogleIPhoneUtilities/Frameworks -F/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GoogleInterchangeUtilities/Frameworks/frameworks -F/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GoogleSymbolUtilities/Frameworks/frameworks -F/Users/jordanmc/Documents/src/Responder/Microsoft.Responder/platforms/ios/Pods/GoogleUtilities/Frameworks/frameworks -filelist /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Intermediates/Responder.build/Debug-iphonesimulator/Responder.build/Objects-normal/x86_64/Responder.LinkFileList -Xlinker -rpath -Xlinker @executable_path/Frameworks -mios-simulator-version-min=9.0 -dead_strip -Xlinker -object_path_lto -Xlinker /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Intermediates/Responder.build/Debug-iphonesimulator/Responder.build/Objects-normal/x86_64/Responder_lto.o -Xlinker -no_deduplicate -Xlinker -objc_abi_version -Xlinker 2 -fobjc-arc -fobjc-link-runtime -ObjC -ObjC -lGGLInstanceIDLib -lGcmLib -lsqlite3 -lz -framework AddressBook -framework CoreGraphics -framework GoogleIPhoneUtilities -framework GoogleInterchangeUtilities -framework GoogleSymbolUtilities -framework GoogleUtilities -framework SystemConfiguration -Xlinker -sectcreate -Xlinker __TEXT -Xlinker __entitlements -Xlinker /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Intermediates/Responder.build/Debug-iphonesimulator/Responder.build/Responder.app.xcent /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Products/Debug-iphonesimulator/libCordova.a -weak_framework WebKit -lz -framework AddressBook -lsqlite3 -lPods-Responder -Xlinker -dependency_info -Xlinker /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Intermediates/Responder.build/Debug-iphonesimulator/Responder.build/Objects-normal/x86_64/Responder_dependency_info.dat -o /Users/jordanmc/Library/Developer/Xcode/DerivedData/Responder-gxmtnsbtcukdazexodqiycsumspa/Build/Products/Debug-iphonesimulator/Responder.app/Responder
ld: library not found for -lPods-Responder
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@jrmcdona open Responder.xcworkspace and build from there. As of Xcode 4 if you see a xcworkspace file you should open that in Xcode to build your project as it is the glue that holds all the sub-projects together.

@jrmcdona if that doesn't work for you please comment on this issue but for now I'm going to close it as I'm 99.44% sure that is your problem.

yes, that works and mentioned that in original post.  But it still does not build in .xcodeproj after that. I just tried that again.   Should I stop using .xcodeproject altogether now?
Here are the files that seem to have an issue in the .xcodeproj.   I have cleaned and rebuilt many times.

Yes, when using CocoaPods you have to use the .xcworkspace instead of the .xcodeproj

Ok thanks all


CORTX-27708: README docs in Hare should refer to RGW instead of legacy S3

As we moved from legacy S3 to RGW, Hare README documents you gets update.
Solution:
Replaced legacy S3 word with RGW in Hare README documents
Signed-off-by: pavankrishnat<EMAIL_ADDRESS>

@pavankrishnat Please include more details in commit message

@SwapnilGaonkar7  can we merge these changes now? are these looking good?

@pavankrishnat Please include more details in commit message

Added, Please confirm

@SwapnilGaonkar7 @vaibhavparatwar - Addressed review comments, please have a look.

@mssawant Addressed review comments, please have a look.

@pavankrishnat, there are some unresolved conversation from @SwapnilGaonkar7, kindly work it out with him.


Modularize it to able to compile it with AoT

Hi,
Would be possible to put it as a Module? It's in order to be importable from App Module and use AoT compilation at the same time.
Thanks in advance.

Will add it in version 0.7

Is it possible to add it to my project as a module? Not seen in the samples. Actually, it suggest to add it by the metadata "declarations".

Fixed in README.md. Yes you can import module


An error when I fitted the age and mass

Hello @jvines :
Happy new year! I'm trying to repeat the example that you show in the introduction in astroARIANDE, however, I meet a question. Can you help me to solve it? The detail as follows:
from astroARIADNE.star import Star ra = 75.795 dec = -30.399 starname = "NGTS-6" gaia_id =<PHONE_NUMBER>844840448 s = Star(starname, ra, dec, g_id=gaia_id, dustmap="Bayestar")

	#####################################
	##             ARIADNE             ##
	#####################################
		Author : Jose Vines
		Contact : jose . vines at ug . uchile . cl
		Star : NGTS-6
	*** LOOKING UP ARCHIVAL INFORMATION ***
		Gaia DR3 ID :<PHONE_NUMBER>844840448
		TIC : 1528696
		Gaia Effective temperature : 4975.000 +/- 104.390
		Gaia Stellar radius : 0.656 +/- 0.141
		Gaia Stellar Luminosity : 0.238 +/- 0.003
		Gaia Parallax : 3.306 +/- 0.013
		Bailer-Jones distance : 303.932 +/- 1.190
		Maximum Av : 0.015

	       Filter       	Magnitude	Uncertainty
	--------------------	---------	-----------
	     GALEX_NUV      	 21.9520 	  0.4090
	  GROUND_JOHNSON_B  	 15.1710 	  0.0140
	       SDSS_g       	 14.6390 	  0.0580
	    GaiaDR2v2_BP    	 14.3654 	  0.0040
	  GROUND_JOHNSON_V  	 14.0870 	  0.0210
	       SDSS_r       	 13.7030 	  0.0320
	    GaiaDR2v2_G     	 13.8020 	  0.0028
	       SDSS_i       	 13.3780 	  0.0570
	    GaiaDR2v2_RP    	 13.0970 	  0.0040
	        TESS        	 13.1686 	  0.0062
	      2MASS_J       	 12.2220 	  0.0330
	      2MASS_H       	 11.7670 	  0.0380
	      2MASS_Ks      	 11.6500 	  0.0320
	    WISE_RSR_W1     	 11.5550 	  0.0270
	    WISE_RSR_W2     	 11.6360 	  0.0270

from astroARIADNE.fitter import Fitter
out_folder = "/home/sdx/data/magnetic_variables/SED_data/test/"
engine = "dynesty"
nlive = 500
dlogz = 0.5
bound = 'multi'
sample = 'rwalk'
threads = 4
dynamic = False
setup = [engine, nlive, dlogz, bound, sample, threads, dynamic]
models = ["phoenix","btsettl", "btnextgen", "btcond", "kurucz", "ck04"]
f = Fitter()
f.star = s
f.setup = setup
f.av_law = "fitzpatrick"
f.out_folder = out_folder
f.bma = True
f.models = models
f.n_samples = 100000
f.prior_setup = {"teff":("normal", 5600, 100),                  'logg':('default'),                  'z':('default'),                  'dist':('default'),                  'rad':('default'),                  'Av':("fixed", 0.0)                 }
f.initialize()
		*** EXECUTING MAIN FITTING ROUTINE ***
			Selected engine : Bayesian Model Averaging
			Live points : 500
			log Evidence tolerance : 0.5
			Free parameters : 21
			Bounding : multi
			Sampling : rwalk
			N threads : 4
`f.fit_bma()`
			FITTING MODEL : phoenix
13039it [02:17, 94.55it/s, +500 | bound: 152 | nc: 1 | ncall: 300144 | eff(%):  4.518 | loglstar:   -inf < 393.231 <    inf | logz: 367.803 +/-  0.224 | dlogz:  0.001 >  0.500]
			FITTING MODEL : btsettl
13396it [02:23, 93.09it/s, +500 | bound: 158 | nc: 1 | ncall: 310992 | eff(%):  4.475 | loglstar:   -inf < 393.347 <    inf | logz: 367.112 +/-    nan | dlogz:  0.001 >  0.500]
			FITTING MODEL : kurucz
13839it [02:33, 90.21it/s, +500 | bound: 165 | nc: 1 | ncall: 321263 | eff(%):  4.470 | loglstar:   -inf < 393.767 <    inf | logz: 366.603 +/-    nan | dlogz:  0.001 >  0.500]
			FITTING MODEL : ck04
14482it [02:37, 92.06it/s, +500 | bound: 172 | nc: 1 | ncall: 335799 | eff(%):  4.468 | loglstar:   -inf < 392.879 <    inf | logz: 364.499 +/-    nan | dlogz:  0.001 >  0.500]
		*** AVERAGING POSTERIOR SAMPLES ***
100%|███████████████████████████████████████████| 26/26 [00:00<00:00, 61.64it/s]
		*** ESTIMATING AGE AND MASS USING MIST ISOCHRONES ***
/home/sdx/.local/lib/python3.8/site-packages/dynesty/dynamicsampler.py:446: RuntimeWarning: All the initial likelihood values are the same. You likely have a plateau in the likelihood. Nested sampling is *NOT* guaranteed to work in this case
  warnings.warn(
0it [00:00, ?it/s]
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
Input In [7], in <cell line: 1>()
----> 1 f.fit_bma()

File /usr/local/lib/python3.8/dist-packages/astroARIADNE-1.0.9-py3.8.egg/astroARIADNE/fitter.py:590, in Fitter.fit_bma(self)
    587 c = np.random.choice(self.colors)
    588 avgd = self.bayesian_model_average(outs, self._grids, self._norm,
    589                                    self.n_samples, c)
--> 590 self.save_bma(avgd)
    592 elapsed_time = execution_time(self.start)
    593 end(self.coordinator, elapsed_time, self.out_folder,
    594     'Bayesian Model Averaging', self.norm)

File /usr/local/lib/python3.8/dist-packages/astroARIADNE-1.0.9-py3.8.egg/astroARIADNE/fitter.py:1120, in Fitter.save_bma(self, avgd)
   1114     logdat_average = out_filler(sampw, logdat_average, 'AD', 'AD', out,
   1115                                 method='averaged')
   1117 # Add estimated age to best fit dictionary. This is done with the wider
   1118 # sampled distribution instead of the averaged one in order to save time
-> 1120 age_samp, mass_samp, eep_samp = self.estimate_age(
   1121     out['best_fit_samples'],
   1122     out['uncertainties_samples'],
   1123     c=choice(self.colors)
   1124 )
   1125 # Create new thingy for MIST samples. Sadly now everything done before
   1126 # this update will be incompatible :(
   1127 out['mist_samples'] = dict()

File /usr/local/lib/python3.8/dist-packages/astroARIADNE-1.0.9-py3.8.egg/astroARIADNE/fitter.py:1457, in Fitter.estimate_age(self, bf, unc, c)
   1454         params[b] = (m, e)
   1455         used_bands.append(b)
-> 1457 age_samp, mass_samp, eep_samp = estimate(used_bands, params, logg=False)
   1459 return age_samp, mass_samp, eep_samp

File /usr/local/lib/python3.8/dist-packages/astroARIADNE-1.0.9-py3.8.egg/astroARIADNE/isochrone.py:66, in estimate(bands, params, logg, out_folder)
     59 sampler = dynesty.NestedSampler(
     60     loglike, prior_transform, model.n_params + len(bands),
     61     nlive=500, bound='multi', sample='rwalk',
     62     logl_args=([model, params, bands]),
     63     ptform_args=([model])
     64 )
     65 try:
---> 66     sampler.run_nested(dlogz=0.01)
     67 except ValueError as e:
     68     dump_out = f'{out_folder}/isochrone_DUMP.pkl'

File ~/.local/lib/python3.8/site-packages/dynesty/sampler.py:952, in Sampler.run_nested(self, maxiter, maxcall, dlogz, logl_max, n_effective, add_live, print_progress, print_func, save_bounds, checkpoint_file, checkpoint_every, resume)
    950 try:
    951     ncall = self.ncall
--> 952     for it, results in enumerate(
    953             self.sample(maxiter=maxiter,
    954                         maxcall=maxcall,
    955                         dlogz=dlogz,
    956                         logl_max=logl_max,
    957                         save_bounds=save_bounds,
    958                         save_samples=True,
    959                         n_effective=n_effective,
    960                         resume=resume,
    961                         add_live=add_live)):
    962         ncall += results.nc
    964         # Print progress.

File ~/.local/lib/python3.8/site-packages/dynesty/sampler.py:778, in Sampler.sample(self, maxiter, maxcall, dlogz, logl_max, n_effective, add_live, save_bounds, save_samples, resume)
    773     old_blob = None
    775 # Sample a new live point from within the likelihood constraint
    776 # `logl > loglstar` using the bounding distribution and sampling
    777 # method from our sampler.
--> 778 u, v, logl, nc = self._new_point(loglstar_new)
    779 ncall += nc
    780 self.ncall += nc

File ~/.local/lib/python3.8/site-packages/dynesty/sampler.py:385, in Sampler._new_point(self, loglstar)
    382 ncall, nupdate = 0, 0
    383 while True:
    384     # Get the next point from the queue
--> 385     u, v, logl, nc, blob = self._get_point_value(loglstar)
    386     ncall += nc
    388     # Bounding checks.

File ~/.local/lib/python3.8/site-packages/dynesty/sampler.py:368, in Sampler._get_point_value(self, loglstar)
    366 # If the queue is empty, refill it.
    367 if self.nqueue <= 0:
--> 368     self._fill_queue(loglstar)
    370 # Grab the earliest entry.
    371 u, v, logl, nc, blob = self.queue.pop(0)

File ~/.local/lib/python3.8/site-packages/dynesty/sampler.py:311, in Sampler._fill_queue(self, loglstar)
    309     args = (np.nonzero(self.live_logl > loglstar)[0], )
    310     if len(args[0]) == 0:
--> 311         raise RuntimeError(
    312             'No live points are above loglstar. '
    313             'Do you have a likelihood plateau ? '
    314             'It is also possible that you are trying to sample '
    315             'excessively around the very peak of the posterior')
    316 else:
    317     args = ()

RuntimeError: No live points are above loglstar. Do you have a likelihood plateau ? It is also possible that you are trying to sample excessively around the very peak of the posterior

Hello @ShenDongxiang , I think your problem is:
'Av':("fixed", 0.0) on the line.
'Av':("fixed", 0) or 'Av': ('default') should be. I get the same error sometimes. Changing it with these values fixes your problem.

@ShenDongxiang
Hi, sorry, I didn't see this bug until now.
I'll try to check it out ASAP, but can't make any promises as I'm currently writing my thesis and that's got me quite busy.
What @denizcdursun says sounds weird to me, but it's worth a shot looking into. If that's indeed the source of the bug, please let me know so I can investigate further.

What @denizcdursun says sounds weird to me, but it's worth a shot looking into. If that's indeed the source of the bug, please let me know so I can investigate further.

I'll add the screenshot here when I get the similar error again.

Hello @ShenDongxiang , I think your problem is:
'Av':("fixed", 0.0) on the line. 'Av':("fixed", 0) or 'Av': ('default') should be. I get the same error sometimes. Changing it with these values fixes your problem.

Thank you very much for the method you provided. I modified my code according to the method you said, but the same error was still displayed

Hello, @denizcdursun :
Thank you very much for the method you provided. I modified my code according to the method you said, however, the same error was still occurred.

I figured it wouldn't fix anything. It doesn't make much sense. Does the error happen every time you run ariadne? with every star?

This has been dormant for a while so I'll close it. Feel free to reopen if needed.


HDDS-5777. Provide an option to dump table scan data to file.

What changes were proposed in this pull request?
Provide an option to dump entire table column family data to a file. This will be useful in HA when we want to compare DB data across SCM's/OM's for verification.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-5777
How was this patch tested?
Manually verified it on the cluster
ozone debug ldb --db=/var/lib/hadoop-ozone/scm/data/scm.db/ scan --column_family=containers --with-keys --out=/tmp/containers1 -l=-1
less /tmp/containers1
{
  "id": 1
} -> {
  "state": "OPEN",
  "pipelineID": {
    "id": "054f8658-81bf-4e81-a71e-4928130f5993"
  },
  "replicationConfig": {
    "replicationFactor": "THREE"
  },
  "usedBytes": 0,
  "numberOfKeys": 0,
  "lastUsed": {
    "seconds":<PHONE_NUMBER>,
    "nanos": 455000000
  },
  "stateEnterTime": {
    "seconds":<PHONE_NUMBER>,
    "nanos": 692000000
  },
  "owner": "om1",
  "containerID": 1,
  "deleteTransactionId": 0,
  "sequenceId": 0
}
{
  "id": 2
} -> {
  "state": "OPEN",
  "pipelineID": {
    "id": "054f8658-81bf-4e81-a71e-4928130f5993"
  },
  "replicationConfig": {
    "replicationFactor": "THREE"
  },
  "usedBytes": 0,
  "numberOfKeys": 0,
  "lastUsed": {
    "seconds":<PHONE_NUMBER>,
    "nanos": 471000000
  },
  "stateEnterTime": {
    "seconds":<PHONE_NUMBER>,
    "nanos": 998000000
  },
  "owner": "om1",
  "containerID": 2,
  "deleteTransactionId": 0,
  "sequenceId": 0
}

Fixed tests and added a test.


[1.18.2] Crash on world load - kotlin.UninitializedPropertyAccessException

Hi, I got this crash report on loading into a world I had recently started:
https://bytebin.lucko.me/iKku4MUszR

kotlin.UninitializedPropertyAccessException: lateinit property GATE_PLACEMENT has not been initialized

I hadn't placed any WR items yet, so I'm not immediately sure what could have led to this.

It looks like this error was encountered as NotEnoughCrashes was already generating a crash report for something else.
If you can give me any steps to reproduce, that would be much appreciated :)

From what I can tell, this error was caused by NotEnoughCrashes attempting to generate a crash-report that involved rendering the world for some reason. However it attempted to render a world before having fully loaded all client-side assets, meaning that the shader had not been initialized yet.
The shader-based system I'm using for fancy gate placement ghosts is deprecated and I'm planning to replace it with a system that's more Iris/Canvas shader-friendly that hopefully shouldn't be as vulnerable to this problem. I'll close this when that system is implemented. It shouldn't be too hard to back-port that system to the 0.3.x-1.18.2 branch either.

Got it, sure. Sorry for no initial response, I was trying to see if it would happen again, as there weren't any real steps leading to the initial crash. If it reoccurs, I'll see if it happens without NotEnoughCrashes and follow up.

Ok, the fix for this should be out in v0.3.10+1.18.2 and v0.4.13+1.19.2.


Restructure documentation examples

We currently use SassDoc's @example annotation by putting the Sass
within the first @example block, and then the output CSS in a second
@example block.
/// @example scss
///   .element {
///     @include border-top-radius(4px);
///   }
///
/// @example css
///   .element {
///     border-top-left-radius: 4px;
///     border-top-right-radius: 4px;
///   }

This is a bit confusing to follow input/output when a feature has
multiple examples.
/// @example scss
///   .first-element {
///     font-size: modular-scale(2);
///   }
///
///   .second-element {
///     margin-right: modular-scale(3, 2em);
///   }
///
///   .third-element {
///     font-size: modular-scale(3, 1em 1.6em, $major-seventh);
///   }
///
///   // Globally change the base ratio
///   $bourbon: (
///     "modular-scale-ratio": 1.2,
///   );
///
///   .fourth-element {
///     font-size: modular-scale(3);
///   }
///
/// @example css
///   .first-element {
///     font-size: 1.5625em;
///   }
///
///   .second-element {
///     margin-right: 3.90625em;
///   }
///
///   .third-element {
///     font-size: 3em;
///   }
///
///   .fourth-element {
///     font-size: 1.728em;
///   }

This also means that we can't describe individual examples very well
since SassDoc supports a description per @example block.
Since SassDoc doesn't currently have the concept of input/output, this
commit encapsulated each example in its own @example block, using a
CSS comment to indicate the output.
What does this PR do?
With this change, the documentation site can have a better understanding
of what an example is, and easily loop through multiple consistent
examples.
/// @example scss
///   .element {
///     font-size: modular-scale(2);
///   }
///
///   // CSS Output
///   .element {
///     font-size: 1.5625em;
///   }

Spoke with @whmii IRL…merging.


Empty build directories

What's wrong
I can't run the bootstrap / use-package. Adding additional use-package calls to an existing config seems to generate empty build directories.
Directions to reproduce
Pointing emacs at an empty directory (~/test-emacs), I've repro'ed the same thing with an empty .emacs.d but this was easier for debugging / testing.
(setq user-emacs-directory "~/test-emacs")

(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

(straight-use-package 'use-package)

(use-package el-patch
  :straight t)

$ emacs --batch --load init.el
Bootstrapping straight.el...
Bootstrapping straight.el...done
Building straight...
Building straight...done
Looking for use-package recipe → Cloning melpa...
Looking for use-package recipe → Cloning melpa...done
Cloning use-package...
Cloning use-package...done
Building use-package...
Building use-package...done
Symbol’s function definition is void: use-package

$ tree ~/test-emacs
tree.txt
Version information

Emacs version: GNU Emacs 26.3 (build 1, x86_64-apple-darwin17.7.0, NS appkit-1561.60 Version 10.13.6 (Build 17G65))
Operating system: macOS 10.15.1

I can't reproduce this issue. My steps were docker run -it --rm silex/emacs:26.3 bash to get a shell, then apt update && apt install -y git, then putting the code you provided into an init.el file in the current directory and running emacs --batch --load init.el as you suggested. Is there something else I might be missing?

I'd believe this is an issue with my local machine, but I can't seem to figure out what might be going on. Is there a way to turn on more verbose logging or tracing that I'm missing?

I'd believe this is an issue with my local machine, but I can't seem to find any failures or errors anywhere. Do you have any suggestions on ways to turn on verbose logging or tracing or similar?

(similarly, I see the same issue without batch mode / with the file in ~/.emacs.d)

I figured out the problem, and I'm not sure if this is a bug in straight or use-package:

I ran emacs using strace / dtruss and grepped around in the resulting sys call log to see anything suspicious. I specifically looked for places that touched the build directory. I noticed it was stat'ing a binary called /Users/mhj/bin/cmd, which seemed a little odd.
It looks like the binary was a test script I had created recently (no idea why I named it that)
I deleted that binary and use-package now works.

I guess the question is - what's looking for a cmd binary? Is this some well-intentioned windows compatibility code (eg cmd.exe or something?)?

It looks like this is the same issue as https://github.com/hlissner/doom-emacs/issues/1715

The problem is then already fixed. Since several people have run into the issue, I've backported the fix to master, so you should see an improvement immediately :)

This thread is being closed automatically by Tidier because it is labeled with "waiting on response" and has not seen any activity for 90 days. But don't worry—if you have any information that might advance the discussion, leave a comment and the thread may be reopened :)


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/113825

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/114168

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/114176


feat: python noarch

MNT: Re-rendered with conda-build 3.21.4, conda-smithy 3.14.3, and connda-forge-pinning 2<IP_ADDRESS>.06.04# We are using https://www.conventionalcommits.org/en/v1.0.0/

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch: python recipes are required to have a lower bound on the python version. Typically this means putting python >=3.6 in both host and run but you should check upstream for the package's Python compatibility.

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


request for event

Hello,
I would like to request the event 'fin-header-click'. So that when a column is sorted by the user this event should fire.
At the moment this is a workaround:
jsonGrid.addFinEventListener('fin-click', function(event) {
            if(event.detail.gridCell.y === -1){
                let x = event.detail.gridCell.x
                console.log(`sorting for column requested: ${colums[x+1]).name}`);
            }
        });

Maybe in the event details it should provide the information if the column is sorted asc or desc.
Also I want to overwrite the sorting functionality.
-----------update:
I would need the functionality to totally mute the default sorting behaviour and sort the table by providing new sorted data manually.

basically: how can I disable a feature: fin-hypergrid-feature-column-sorting ?

Actually the current implementation of fin-hypergird-feature-column-sorting isn't really about sorting but rather a general handler for header clicking; if you look at fin-hypergrid-behavior-base.js you will find a function fixedRowClicked which you can override on your own behavior object to alter or disable the sort behavior.

:+1:


neotest: coverage collection polishing

It'd be nice to know the reason for sorting from the commit message.

I'm writing an issue about that.


[Suggestion] Improve API around (single) unnamed return values

Component
contract
Describe the feature you would like
Follow up from conversation: https://github.com/alloy-rs/examples/pull/8#discussion_r1526788839
Consider the following:
 let Counter::numberReturn { _0 } = contract.number().call().await?;

Where Counter is implemented as follows:
contract Counter {
    uint256 public number;

    function setNumber(uint256 newNumber) public {
        number = newNumber;
    }

    function increment() public {
        number++;
    }
}

It would be preferable to either:

Be able to assign this return directly to a value (let ret = contract.number().call().await?;)
Or be able to rename the return value _0 into a user defined value

Additional context
Any changes will be reflected in alloy/examples

My apologies @prestwich, opened it in the wrong repo

having an unwrapped return type would be more convenient,
I think we can check this when analyzing the abi and check the outputs, if it's just a single type, use that

we can, but it breaks encoding symmetry for any case where (T,) and T have different abi encodings, which would need to be accounted for

We deemed the current API consistent and good enough. This change would require some specific workarounds and maybe hard to do in the current type system.
If you want return types to have a "nicer name" in functions, you can name them in the returns (...) tuple.
The example in the issue already has a named return value due to it being an elementary type getter: https://github.com/alloy-rs/core/blob/b4ca4fe483a8f5bee450a4ee627995b97249abbe/crates/sol-types/tests/macros/sol/mod.rs#L242-L255


Unable to get the Move property of the Worksheet class (760db9a3)

This issue was submitted by the add-in's exception handler.

Exception:
System.Runtime.InteropServices.COMException (0x800A03EC): Unable to get the Move property of the Worksheet class
   at System.Dynamic.ComRuntimeHelpers.CheckThrowException(Int32 hresult, ExcepInfo& excepInfo, UInt32 argErr, String message)
   at CallSite.Target(Closure , CallSite , ComObject , Object )
   at CallSite.Target(Closure , CallSite , Object , Object )
   at XLToolbox.Excel.ViewModels.WorkbookViewModel.DoMoveSheetDown() in XLToolbox\Excel\ViewModels\WorkbookViewModel.cs:line 445
   at XLToolbox.Excel.ViewModels.WorkbookViewModel.<get_MoveSheetDown>b__13(Object parameter) in XLToolbox\Excel\ViewModels\WorkbookViewModel.cs:line 184
   at Bovender.Mvvm.DelegatingCommand.Execute(Object parameter)

Message: Unable to get the Move property of the Worksheet class
User comment on submission:
NG 7.05 just installed. Before this error occurred I was trying to get worksheet manager to work. When I clicked on worksheet name it did not activate - unlike previous version. then I experimented with moving sheets up and down. Only half worked.
Environment

Parameter
Value

Version
7.0.5

OS
Microsoft Windows NT 6.1.7601 Service Pack 1, 64-bit

Excel
2010 SP2 (14.0.7015), 32-bit

CLR
4.0.30319.34209, 32-bit

VSTO runtime
10.0.40305

ClickOnce
False

Bovender
<IP_ADDRESS>

FreeImage
3.16.0

I think I solved this with bc575fc4f173aa8623695b5de8d32e091a95f276 (in v7.0.9)


Update libraries.

should fix #83

Hi @xanderim
Thanks for the PR! Sorry it took so long to get back to you.
Things were on pause for a while but version 2.0 should be out very soon.
I'm going to close this PR, as 2.0 is a complete re-write and the fix no longer applies (and lack of test suite in 1.0 means I can test open PRs easily).
This has been super useful though as - thanks to feedback like this - the new version avoids the pitfall of a custom babel config that this required.


Fixed init-order-fiasco for static slice table.

Fixes init-order bug affecting https://github.com/grpc/grpc/issues/19819 which
was first exposed by this commit:
https://github.com/grpc/grpc/commit/857375a1420df184963ed83f638dd21ea359aa37

Specific change: since we cannot be certain of the ordering of global object initialization across compilation units, it is dangerous to define the static slice table globally. Instead, we use the static-function variable method which is guaranteed to initialize the table before the first access.

Does this fix https://github.com/grpc/grpc/issues/19819?

There seem to be a lot of failures

There seem to be a lot of failures

Fixed, that's because of stuff in test/ that I didn't convert over.

@AspirinSJL I haven't tested it on mac, but Linux has the exact same failure and I just confirmed that this fixes the issue on Linux. My guess is that it will fix the mac issue as well

I'll review when the contents of test/ are changed over as well.

@hcaseyal Thanks for confirming!
@arjunroy Then let's mark "fix #19819" in the PR description. Thanks!


Typo on initial screen

The hover tooltip states "Rin as online MWC Node". It should state "Run as ...".

Edit: This is on line 486 of file /mwc-qt-wallet/windows/a_initaccount.ui link: https://github.com/mwcproject/mwc-qt-wallet/blob/c94480012320970b70465c7c284ea367a30950c0/windows/a_initaccount.ui#L486

It is already fixed in master branch.


Instance segmentation

@james-nvidia Thanks for giving the resource to understand the metrics. However, when I trained the model, I got mAP of 0.53 at maximum for (second number).
Now, I have another query if you could resolve:

Even with this metrics, I was able to visualize the results of bounding box detection on the test images, and it was looking pretty good. Now, I wanna extend it further to do instance segmentation.
I would be glad if you could guide me to proceed with the instance segmentation pipeline, provided that I have trained model model.pth, and bounding box detections in json format after running odtk inference.

Thanks in advance. Looking forward to hearing from you soon.
Originally posted by @nakul3112 in https://github.com/NVIDIA/retinanet-examples/issues/219#issuecomment-686843118

@james-nvidia 1) while using Nvidia ODTK, I changed the NMS factor, so my concern is the NMS factor would be just helpful during inference , right?
2) Where would I do padding, after training or during inference?
Little more elaboration on this would be appreciated, so that atleast I can be sure of my bounding box detection before proceeding with the instance segmentation.
Thanks,

NMS is applied during inference, not during training.
padding was just a suggestion. If you are feeding chips to a segmentation model, then you might want to provide more than just the bounding box, as there will be no context to the detection.
xmin, ymin, w, h could be modified to xmin - w/4.0, ymin -h/4.0, w * 3 / 2.0, h * 3 / 2.0, as a first suggestion.


base-compiler.js: acknowledge checkSource weakness

To potentially make the intent clearer for future inquiring minds, add a comment to checkSource function explaining that it's not meant to prevent arbitrary file inclusion, only to diagnose unsophisticated attempts.

Thanks!


ability to disable certain options?

Hi! I basically use only the icons.data.svg.css file. So it would be great if I had one or more options to  disable the other files generation :)
Thank you, regards!

In order to generate the icons.data.svg.css file, we use https://github.com/filamentgroup/directory-encoder (https://github.com/filamentgroup/grunticon/blob/master/tasks/grunticon.js#L187).
It's a pretty straight-forward library for this use and will cut out the rest of the cruft.
Thanks!


pd-ctl/API: get label information

We can use HTTP API and pd-ctl to get label information, like:
// List current labels and their names
label list
// Get the stores with the given label name and value, may support regular for name and value
label store name [value]

For HTTP API, may like:
GET /pd/v1/api/labels
GET /pd/v1/api/labels/store?name=xxx&value=xxx

/cc @huachaohuang any more functions?


migration component role

ROLE :   new feature (migration)
Description
The compoent has been migrated in components
Test cases have been created
Example
// Sample code

//check one role
 <Role  hasOne={['DEFAULT_ROLE']}} >
{children}
</Role>

//check many roles all are needed
 <Role  hasAll={['DEFAULT_ROLE', 'ADMIN_ROLE', 'OTHER_ROLE']} >
{children}
</Role>

//optional child element when roles are not sufficient
 <Role  hasOne={['DEFAULT_ROLE']}}  emptyBlock={
Insufficient rights
}>
{children}
</Role>

Done


Task list element adder

Moved enum class to different package, finish taskListCreator Fragment (add switches for switches for category, priorities and time priorities, begin date)

Commity z tego requesta są na branchu listEdit


chore : setup project for Innovision 2024

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your default branch!
[x] Make sure you are making a pull request against the default branch (left side). Also you should start your branch off default branch.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] I have added necessary documentation (if appropriate)

Description
Please describe your pull request.
❤Thank you!
Post merge checklist

[x] Follow steps from the guidelines for contributing to this repository.
[ ] If you are a new contributor, ping in the thread and one of the maintainers will add you to all-contributors list.

@Shurtu-gal , please review

@Srish-ty could you add the prettier:check and other scripts in package.json.

Done. https://github.com/dscnitrourkela/project-raisin/pull/117/commits/4f4aa61b393de674d1bd2eab8d48fb7e1b60d55f


org.redisson.client.codec.JsonJacksonMapCodec not found

question ：
Introduce redisson dependency in java project through mavan method，result is org.redisson.client.codec.JsonJacksonMapCodec not found
<dependency>
            <groupId>org.redisson</groupId>
            <artifactId>redisson</artifactId>
            <version>3.20.1</version>
        </dependency>

it was replaced with TypedJsonJacksonCodec

thank you

it was replaced with TypedJsonJacksonCodec

thank you


allow connecting to a mattermost instance without tls and specify a port

in matterbridge you can hook up to mattermost

without tls with "NoTLS"

to a different port other than 443 by specifying the port in the name of the mattermost server, for instance "Server=mattermost:8000"

please allow this in matterircd

It already does, use matterircd -mminsecure -mmserver mattermost:8000
 -mminsecure
        use http connection to mattermost
  -mmserver string
        specify default mattermost server/instance

or in the matterircd.toml
[mattermost]
#specify default mattermost server/instance (default "")
DefaultServer = "chat.mycompany.com:1234"

#use http connection to mattermost (default false)
Insecure = true

I would like to be able to do it while authenticating when messaging "slack"... but your solution works too.
thanks.


Iss hipcms 252 invite unregistered

Added User creation
working email on upb servers

something went terrible wrong here /close


ScaleBitmap: make use of createjs.promote()

Looks great.
Can you move the properties into the constructor? All properties defined as p.propertyName=value should instead be in the constructor as this.propertyName=value, including the JSDoc comments. This is part of the inheritance change.
If you can do this, I will accept the pull request. Thanks for your help!