WIP: special: Fix unchecked memory allocation in aswfa()

This is a proof-of-concept for fixing https://github.com/scipy/scipy/issues/21417.
Only one of the functions (aswfa()) with unchecked calls of calloc() is fixed.  If this looks like a reasonable approach, the same pattern can be used to fix all the other unchecked allocations in specfun.h.
Summary of the changes:

Add another SF error code, "SF_ERROR_NOMEM", that is used by ufuncs that require internally allocated memory.  Just like the other possible errors, how this error is handled can be queried and set using geterr() and seterr().  Unlike the other error conditions, the default behavior is to raise an exception if a memory allocation fails.

Modify the function aswfa() in specfun.h to return a status code that indicates when a memory allocation failed.

Modify the functions prolate_aswfa_nocv(), oblate_aswfa_nocv(), prolate_aswfa() and oblate_aswfa()  in sphd_wave.h to check the return value of aswfa(), and set the error state and return values appropriately if aswfa() returns NoMemory.

Ping @izaid, @ilayn, @steppi.

Just wanted to chime in here. Thanks @WarrenWeckesser for thinking about this. Comments below.
I'm a little concerned we are jumping ahead here. It's not that we don't need better error reporting for memory allocation (we do), it's just that the error handling you see now largely exists because we tried to make something compatible with what was there before. It is totally possible that the error handling you see will totally change in the near future when we separate out xsf and properly consider how to do this safely across all array backends.
For that reason, I also don't think we should be modifying the ufunc loops. Whatever you put in the ufunc loops will not translate to other array backends. I also put a lot of effort into getting them right. I'd much prefer we cleanly separate xsf into its own library (which @steppi is doing now) and get that in good shape, then worry about error reporting edge cases.
@WarrenWeckesser I really do appreciate the effort you are making, I'd just rather we all sync before you make changes as all of this is very fast moving. Is there some reason you want to do this now in particular? If you want to work on error handling, I'd suggest we discuss how to do it safely across backends.

@izaid, to clarify, I think this is only incidentally about improving error reporting. @WarrenWeckesser is trying to get rid of unchecked calls to malloc so malloc failing won't result in a segfault. The reason he's created a new error code is because raising when malloc fails seems like a more sensible default than ignoring it; the default for all other options is to ignore.

Also spurious malloc calloc per item is not the way this should work. Memory handling is definitely a sore point in these old functions which is independent from which backend is being used.

@izaid wrote

I'd just rather we all sync before you make changes as all of this is very fast moving.

Heh, I thought that's what we're doing!  I think there was some confusion over in gh-21417 about the change that I was suggesting, so to make it absolutely clear what I meant, I created this PR, marked "draft", and currently just for one function in specfun.h, namely aswfa().  This has been helpful, because I think the discussion has made clear to everyone what we can expect to achieve with a simple change like this.  Specifically, we can fix the unchecked allocations, but to eliminate the alloc/free churn (which is not the goal of this PR), further design work is needed.
Regarding the scipy.special error handling (i.e. geterr() & seterr()): there is no new design here.  I am just adding one new error code that is used to indicate that a memory allocation has failed.  This really should have been added as soon as any ufuncs started using dynamically allocated arrays.  I suspect the original set of error codes (the creation of which predates my time working on SciPy) did not include such an error because the original functions implemented in special didn't allocate memory dynamically.  We've had such functions for a while now, so we're overdue for adding the appropriate error status.
So, to be clear, the goal is to fix the immediate problem of unchecked memory allocations.  The changes I am suggesting are pretty straightforward.   If there is agreement, I'm happy to fix the rest of the functions in specfun.h.

Is there some reason you want to do this now in particular?

The problem with unchecked allocations was reported in August.  It would have been nice to fix it for the 1.15 release, but that didn't happen--so it goes, it's software.

So, to be clear, the goal is to fix the immediate problem of unchecked memory allocations. The changes I am suggesting are pretty straightforward. If there is agreement, I'm happy to fix the rest of the functions in specfun.h.

Is there some reason you want to do this now in particular?

The problem with unchecked allocations was reported in August. It would have been nice to fix it for the 1.15 release, but that didn't happen--so it goes, it's software.

I support moving forward with the general approach in this PR. Thanks @WarrenWeckesser. I guess we can at least try to get this in for the next bug fix release.

@izaid wrote

I'd just rather we all sync before you make changes as all of this is very fast moving.

Heh, I thought that's what we're doing! I think there was some confusion over in gh-21417 about the change that I was suggesting, so to make it absolutely clear what I meant, I created this PR, marked "draft", and currently just for one function in specfun.h, namely aswfa(). This has been helpful, because I think the discussion has made clear to everyone what we can expect to achieve with a simple change like this. Specifically, we can fix the unchecked allocations, but to eliminate the alloc/free churn (which is not the goal of this PR), further design work is needed.
Regarding the scipy.special error handling (i.e. geterr() & seterr()): there is no new design here. I am just adding one new error code that is used to indicate that a memory allocation has failed. This really should have been added as soon as any ufuncs started using dynamically allocated arrays. I suspect the original set of error codes (the creation of which predates my time working on SciPy) did not include such an error because the original functions implemented in special didn't allocate memory dynamically. We've had such functions for a while now, so we're overdue for adding the appropriate error status.
So, to be clear, the goal is to fix the immediate problem of unchecked memory allocations. The changes I am suggesting are pretty straightforward. If there is agreement, I'm happy to fix the rest of the functions in specfun.h.

Is there some reason you want to do this now in particular?

The problem with unchecked allocations was reported in August. It would have been nice to fix it for the 1.15 release, but that didn't happen--so it goes, it's software.

Happy with that, thanks! I think I just saw the comments on the ufunc loop and didn't want us diving into that just yet.


[Snyk] Upgrade rollup from 4.13.2 to 4.14.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade rollup from 4.13.2 to 4.14.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-04-03.

Release notes

Package name: rollup

4.14.0 - 2024-04-034.14.0
2024-04-03
Features

Display error causes in Rollup CLI (#5422)
Add basic support for explicit resource management via "using" and "await using" (#5423)

Pull Requests

#5422: feat: show all cause in Error (@ devohda, @ lukastaegert)
#5444: feat: support explicit-resource-management (@ TrickyPi)
#5445: docs: add @ shikiji/vitepress-twoslash (@ sapphi-red)
#5447: chore(deps): lock file maintenance minor/patch updates ( @ renovate[bot])
#5448: chore(deps): lock file maintenance (@ renovate[bot])

4.13.2 - 2024-03-284.13.2
2024-03-28
Bug Fixes

Ensure accessing module info is cached after the build phase for improved performance (#5438)
Support powerpc64le CPUs (#5350)

Pull Requests

#5350: Add support for ppc64le (@ pavolloffay, @ lukastaegert)
#5438: Cache module info getters before output generation (@ bluwy, @ lukastaegert)

    from rollup GitHub release notes

Commit messages

Package name: rollup

5abe71b 4.14.0
5301dec feat: show all cause in Error (#5422)
8521d29 feat: support explicit-resource-management (#5444)
c87901d docs: add `@ shikiji/vitepress-twoslash` (#5445)
1dd3c02 chore(deps): lock file maintenance (#5448)
a8e39eb chore(deps): lock file maintenance minor/patch updates (#5447)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Solved in #359


Enhancements and fixes

GRL can be used for creating image walls displayed in 1 draw call. This requires to add some code for dealing with UVs.

[x] get/set UVs
[ ] support appending UVs when in instance mode
[ ] fix instance support in ribbon mode

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14663/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14663/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14663/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14663/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14663/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14663/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14663/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14663/merge#BCU1XR#0

Happy new year @RolandCsibrei !!!
There is no need to start a draft PR when you start working on the improvements. Let me close it, and you can reopen when almost ready?
This way, it helps the team to focus on the PRs that are closer to go in and discuss the final details when necessary.

Hello and happy new year!!
I needed a snapshot. Is there any other way to create one?
Thanks!
On Tue, 2 Jan 2024 at 16:34, sebavan @.***> wrote:

Happy new year @RolandCsibrei https://github.com/RolandCsibrei !!!
There is no need to start a draft PR when you start working on the
improvements. Let me close it, and you can reopen when almost ready?
This way, it helps the team to focus on the PRs that are closer to go in
and discuss the final details when necessary.
—
Reply to this email directly, view it on GitHub
https://github.com/BabylonJS/Babylon.js/pull/14663#issuecomment-1874180085,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYKGHMD74BUWTHKJB4AWJ3YMQSGZAVCNFSM6AAAAABBJEKFGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZUGE4DAMBYGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

That is a pretty good reason then :-) feel free to reopen as draft whenever.

I needed a snapshot. Is there any other way to create one?

Why not testing in your local Playground?

I always test in my local PG ofcourse. I wanted to make a publicly available PG with the proprosed PR.

Feel free to reopen whenever :-) my bad for closing it yesterday, I was just doing new years cleanup lol

Sorry guys, I forgot about this one. I will close it now and open a new one when it will be ready.


Predicted Target Category does not matches with Imagenet Category

Hi @jacobgil ,
Thank you for sharing the code!
I was trying out on VOC dataset, every things works well, apart from one thing, which is; the predicted target category is not right when compared with official  Imagenet id. For example for following image:

Shows target category as 417, which corresponds to ballon rather than plane.(using the link https://gist.github.com/yrevar/942d3a0ac09ec9e5eb3a
Question:

Is it something which is generally seen,Since it depends upon what the network is seeing?
If not, is there a different interpretation of the same result

Regards,
Nitin Bansal

Hello,
In some cases the model could be just wrong. Maybe the baloon score scored higher than plane here.
Are there also VOC images the images is correct about?
Or is it wrong in most cases?

@jacobgil There are few other cases too, where the predicted category is wrong, but the region of interest (highlight region), is correct in all of them. For Dog and Cat Target Category is correct, but I also stumbled upon an image which consisted of Train, it was able to correctly zero down on the area of interest, but failed to predict the correct cateogry.

Hey @jacobgil,
I had an extra thought about the grad cam application. I was actually trying out GradCAM for CityScapes Dataset. So, In Cityscapes Dataset we can multiple instances of an object say car or people etc. So, we are not concentrating on classifying a single object as one of the class but could have multiple instances of multiple classes.
I find that, If I simply use GradCAM for cityscapes images, results do not look that informative per say. For example:

Now this could be because of two probable reason:

Resnet50 is pertained on ImageNet and not on Cityscapes
Resnet50  which is the underlying network used for capturing gradient and activation is a classification network and not an Object detection network.

Do, you think we can use a pertained version of say FasterRCNN or RetinaNet instead of Resnet50 for a dataset such as Cityscapes to get the grad/activation visualization for such a dataset.
Regards,
Nitin Bansal

Yes, but using grad-cam for object detection is more difficult and currently not supported in this repo (although I'm thinking of adding it in the future).
Try using a specific category like "Traffic light" (920 in imagenet). In the image above what was the target category? Since there are many objects, the classifier might have failed. You might want to focus it and try again.
To get the bounding box, use opencv findContours:
_, contours = cv2.findContours(binary_heatmap) for contour in contours: x,y,w,h = cv.boundingRect(cnt) #do something with it here..

If the classifier correctly classifies or not, is out of control of the visualization method. Many times the salient part might still appear in the incorrectly classified  target_category.
You might want to target specific categories instead of passing target_category=None.

@jacobgil Thank you for your prompt response. Please find my answers inline:

Yes, but using grad-cam for object detection is more difficult and currently not supported in this repo (although I'm thinking of adding it in the future).
Cool! May be I can start from my end and see if I able to get something good using any of the existing network. Will keep you posted.
Try using a specific category like "Traffic light" (920 in imagenet). In the image above what was the target category? Since there are many objects, the classifier might have failed. You might want to focus it and try again.
For me it is actually predicting 919 and 920 which is the traffic light. I can remove None and give it a try for a more specific class and see how that turns out to be.
To get the bounding box, use opencv findContours:
That makes sense! Thank you for your suggestion

Regards,
Nitin

Curious if for the image with the traffic-light above, you pass eigen_smooth=True, what happens?

@jacobgil I had already passed both the parameter as True which is eigen_smooth and aug_smooth.  BTW, I just stumbled upon a work which seems to be doing what we require for cityscapes dataset. Its a 2020 paper by name SEG_GRAD_CAM.
I have still not gone through this. But may be we could find something interesting here.

@jacobgil I just gave it an another try by setting the target_category as 817 which is class sport car in Imagenet set of 1000 classes. I get bit better results But I presume it is still not good.

Regards,
Nitin

Yes.
It might also be that the model isn't good enough in finding cars, since it was trained on ImageNet.
Curios if switching to one of the other methods like ScoreCAM or AblationCAM gives the same result?

I am getting pretty similar or worse results with other methods . GRADCAM++ provides a slightly better result. But I think methods based on Segmentation or Object detection model, would be better suited for dataset such as Cityscapes or KITTI. I will give it a try for SEG_GRAD_CAM work and see how the results pan out.


Update CI to use release 5.10 image

@swift-server-bot add to allowlist

5.10 is failing because strict concurrency checking and warnings as errors are enabled.
The concurrency checking emits a warning because of the generated test file which contains a static array of test classes, and XCTestCase isn't Sendable.
Do you have a preference for which option to disable?

contains a static array of test classes

Maybe we can just move it to be a fileprivate global let in the file, to avoid coupling it to the (non-Sendable) XCTestCase subclass?

I disabled strict concurrency checking for now and filed https://github.com/apple/swift-openapi-generator/issues/543 to re-enable it.
I also filed a radar against SwiftPM.


[Bug Report] conv test failure from discord community

Describe the bug
"Circular Buffer Config Error: Circular buffer size cannot be 0 B"
To Reproduce
Steps to reproduce the behavior:
`import torch
import ttnn
device_params = {"l1_small_size": 24576}
device = ttnn.CreateDevice(device_id=0, **device_params)
a = torch.rand((1, 1, 32, 32), dtype=torch.bfloat16)
b = torch.rand((1, 1, 5, 5), dtype=torch.bfloat16)
input_tensor  = ttnn.from_torch(a, layout=ttnn.TILE_LAYOUT, device=device)
weight_tensor = ttnn.from_torch(b, layout=ttnn.TILE_LAYOUT, device=device)
x = ttnn.conv2d(input_tensor=input_tensor,
weight_tensor=weight_tensor,
device=device,
in_channels=1,
out_channels=1,
batch_size=1,
input_height=32,
input_width=32,
kernel_size=[5,5],
stride=[1,1],
padding=[5,5],
dilation=[1,1],
groups=1)
res = ttnn.to_torch(x)
ttnn.close_device(device)`
Expected behavior
runs without the failure error
Please complete the following environment information:

OS: [e.g. Ubuntu 20.04]
Version of software: commit: 0d1eb7d95983ad7f42faadf86dc66e186f53f264

Additional context
this is a bug report from discord community.

still waiting on CI pipelines -- they are having machine issues :(

merged

thanks for the prompt fix @mywoodstock.


Fixed propagation of return values for some verb calls

Fixed some tear down handling of QPs, CM IDs, as well as other components which didn't have their return value propagated properly.
For QPs and CM IDs, the CM ID was improperly removed from the map in case of rdma_disconnect() and rdma_destroy_qp(). Instead, the code now removes the CM_ID only when it is destroyed, and removes the QP from the map upon rdma_destroy_qp().

Thanks!


Edit to types of storage accounts support ZRS

FileStorage --> File Share

@ajomathew : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ajomathew - Thanks for making a contribution! I have updated the account name to refer to "Premium file shares", which is the terminology we're now using for that account type. Since I already made that change, I'm closing out your PR.
#please-close


Add TTGlyphPointPen

See https://github.com/googlefonts/ufo2ft/issues/469: The TTGlyphPen may change the point indices. This adds a PointPen-based TTGlyphPointPen to avoid changes in the outline.
I'm not sure what to put in the tests; how strict must path handling be? E.g. What happens if beginPath() is called twice in a row, or addPoint() is not called on a path; or should the base PointPen take care of those edge cases?

I added some assertions to the pen so that beginPath can not be called twice, etc. which is more strict than the normal PointPen. I also adapted the tests from TTGlyphPen.

Quick comments:

consider adding it to the ttGlyphPen module (the recording pens also live in the same place)
can both tt pens be refactored to share most code?
could the segment pen be implemented in terms of the point pen?

Or perhaps they can have a common base class.

Maybe I can fuzz this in #2202 🤔 If you want to be anal about error handling (I would), check the changes I did to pens in that PR.

I moved the TTGlyphPointPen class into pens.ttGlyphPen, and also implemented both pens with a common base class, TTGlyphBasePen.
I also tried to subclass TTGlyphPen from SegmentToPointPen and use TTGlyphPointPen in there, but it turned out it gets more complex, so I kept the common subclass approach.

Maybe I can fuzz this in #2202 🤔 If you want to be anal about error handling (I would), check the changes I did to pens in that PR.

Looks useful, I could add the improved error handling after your PR is merged.

I think this is ready to be merged now. Any more comments @anthrotype @justvanrossum?

I just released https://github.com/fonttools/fonttools/releases/tag/4.24.0 with the new point pen


Add missing border to content boxes

i tried to keep the PR focused on just borders.. but did end up nudging a shadow or 2 while i was there 😬

@argyleink quick question before we merge… should these things have side borders when they're full width (as they are in the mobile view)? We could make the border just top & bottom in those cases.

no borders on mobile 👍
the pattern is essentially a light but deep box-shadow for desktop that includes a subtle border to help it pop a bit and simulate some minor lighting effects, but all that crap goes out the window for mobile where we pull lots of that depth out in favor of flat and single column layouts.
wow that was verbose... tldr; rule = no box shadow, no border

and cool, i'll wait for this to merge, cuz i'm updating the schedule page shadows right now. in fact, want me to put the borders on there in that PR?

goes well with #318

@argyleink

no borders on mobile

But they do in this PR. Eg https://deploy-preview-304--cds2019.netlify.com/devsummit/schedule/ - the "Every session is available through…" box has 4 borders. The top & bottom border looks good IMO, but don't think it should have side borders.

ooh yes, good call, i wanted to react with "no borders ONLY IF the content goes edge to edge," but i do like the borders in that deployed PR! thanks for calling it out. i'm convinced. let's keep the borders on top and bottom when content boxes goes edge to edge on mobile.
i'll pull latest and do a border sweep.
jake, i love your eagle eyes 🦅 👀 , would appreciate another review before merging

@googlebot I consent.
🤷♂

We still have side borders on content that's 100% width.
https://deploy-preview-304--cds2019.netlify.com/devsummit/schedule/

The box containing the "Every session is available…" content has 4 borders, top right bottom left. It seems weird to me that it has left & right borders right up against the edge of the viewport.

agree! lemme adjust!

check again! found a bunch of instances, and even fixed a tag that was missing a border! thanks nit picking 1px with me


insert snippet with clipboard variables test triggers permission dialog

When I run scripts/test-web-integration.sh --debug on my macOS, they eventually hang here asking for permissions for clipboard:

Works fine when I skip this test:
https://github.com/microsoft/vscode/blob/c258547b50e70486896a3dafcc1447fe28ea4400/extensions/vscode-api-tests/src/singlefolder-tests/editor.test.ts#L53
I think we need something like https://playwright.dev/docs/api/class-browsercontext#browser-context-grant-permissions if we wanted to avoid such a popup.
//cc @alexdima

https://github.com/microsoft/vscode/pull/170490/commits/9d2bdb850db5eb464318cd7f773244f6364112d0#diff-1254034211415436e2a808273dbb33090414834222490e8998a908bafe41b2e8R38 seems to make this work for me, please take a look

@bpasero What did change here? The test is already 3 years old. Is this a playwright or browser update?

Yeah its possible newer playwright versions caused this.

Then we are good with https://github.com/microsoft/vscode/commit/9d2bdb850db5eb464318cd7f773244f6364112d0#diff-1254034211415436e2a808273dbb33090414834222490e8998a908bafe41b2e8R38, right?

Oh yeah, forgot that this got merged already.


Better control of simplification and folding patterns via Pass Options

Request description
Per discussion in: https://github.com/openxla/stablehlo/pull/2586#discussion_r1799945650, we should consider what sorts of flags / options the simplification and folding pass should have.
Something like kFoldOpEltLimit should probably be controlled from the callsite, since some users will not want to fold at the expense of compile time, and others will want the smallest graph possible. And there are likely many other options like this i.e. OnlyFoldSplats.

from @mvpant

I think a broader design on "optimization / folding control" is needed

Sounds good. During my initial exploration of the MLIR framework, ONNX, StableHLO, TOSA, etc., I expected to find fine-grained control over the optimizations performed, similar to classic compilers. However, later discovered that most of these tools have passes that either apply all optimizations or none at all.

... took 1min+ for folding / DCE ...

I guess it is normal for model compilation that will run in production? While having settings to limit optimizations would be beneficial, based on experience with compiling ResNet and BERT models, it can take considerable time to simply reduce the number of computations. However, this effort is definitely worthwhile for achieving good performance.
One operator I’ve found to be somewhat problematic to fold is broadcast_in_dim (ignore transpose semantics). In certain scenarios, not expanding it into a constant can significantly reduce memory traffic on the accelerator, e.g. stablehlo.add(tensor, splat) where splat is result of broadcast_in_dim operation.


Compile Error of "example" in macos

Hi,
I got the following error:
/mobile_scanner/macos/Classes/MobileScannerPlugin.swift:4:8: error: no such module 'UIKit'
import UIKit
^
/mobile_scanner/example/macos/Pods/Pods.xcodeproj: warning: The macOS deployment target 'MACOSX_DEPLOYMENT_TARGET' is set to 10.11, but the range of supported deployment target versions is 10.13 to 13.1.99. (in target 'FlutterMacOS' from project 'Pods')
warning: Run script build phase 'Run Script' will be run during every build because it does not specify any outputs. To address this warning, either add output dependencies to the script phase, or configure it to run in every build by unchecking "Based on dependency analysis" in the script phase. (in target 'Runner' from project 'Runner')
** BUILD FAILED **
Exception: Build process failed

This should be the cause:
https://github.com/juliansteenbakker/mobile_scanner/commit/684634aadd3d5f09109193811a636c1a081cc2ec#r94007376
I have no problem with insufficient support for macOS, but the app itself should be kept able to run.

#434 fixes this.
Temporary solution:
  mobile_scanner:
    git:
      url: https://github.com/juliansteenbakker/mobile_scanner
      ref: bug/fix-macos

This is fixed in version 3.0.0.


readme update

If the contributer wishesh to create new category
we shoud also include the steps in readme.

Yes, that is needed. We just need to change the UI design to implement this, and we are working on the UI. Once it is fixed, we can add these steps.

@JasonDsouza212 still open?


fix for aws cli

Codecov Report

Merging #78 into master will decrease coverage by 1.36%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
- Coverage   25.19%   23.82%   -1.37%
==========================================
  Files          11       11
  Lines         262      277      +15
==========================================
  Hits           66       66
- Misses        196      211      +15

Impacted Files
Coverage Δ

R/s3cache.R
2.04% <0%> (-0.24%)
:arrow_down:

R/s3delete.R
0% <0%> (ø)
:arrow_up:

R/s3exists.R
0% <0%> (ø)
:arrow_up:

R/s3.put.R
31.7% <0%> (ø)
:arrow_up:

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 2ea4c87...7b1890b. Read the comment docs.


s3Connector changes for regions apart from aws default regions

(cherry picked from commit a3c61be5b7755e5537613899124d8825a9309206)

Change has been committed to SVN: http://svn.apache.org/viewvc?rev=1879390&view=rev


WebGL2 returns null when accessing lime.graphics.RenderContext [Desktop, Windows]

Lime.software's Docs says this should be available, but upon running program it returns null. Tested for C++ and HashLink.
Using Lime 7.7.0
import lime.app.Application;
import lime.graphics.RenderContext;
import lime.graphics.WebGL2RenderContext;
import lime.graphics.opengl.GLProgram;

class Main extends Application
{

	public function new()
	{
		super();
	}

	var gl:WebGL2RenderContext;
	var program:GLProgram;

	public override function render(context:RenderContext):Void
	{
		switch (context.type)
		{
			case OPENGL, OPENGLES, WEBGL:

				gl = context.webgl2; //context.webgl2 is null
				program = gl.createProgram(); //crashes here

				//none of this will obviously run
				gl.viewport(0, 0, window.width, window.height);
				gl.clearColor(1.0, 1.0, 1.0, 1.0);
				gl.clear(gl.COLOR_BUFFER_BIT);

			default:
		}
	}

}

WebGL2RenderContext.zip

OpenGL ES 3.0 compatibility is required to support a WebGL 2 compatibility context.
This is disabled on current versions for macOS and Windows due to issues linking at compile-time:
https://github.com/haxelime/lime/blob/develop/project/src/graphics/opengl/OpenGL.h#L76
It is enabled for Linux and iOS. Either something may change in the underlying OS SDKs we use to define all symbols, or changes to SDL's OpenGL headers, or we move to a different system (with dynamic runtime linking) instead of the SDL GL approach. A dynamic system would be ideal (long-term) for supporting a broader variety of OpenGL desktop features beyond GLES3.


Encountered two children with the same key

Android
/ReactNativeJS: 'Warning: Encountered two children with the same key, `%s`. Keys should be unique so that components maintain their identity across updates. Non-unique keys may cause children to be duplicated and/or omitted — the behavior is unsupported and could change in a future version.%s', 'NaN', '
  in AndroidHorizontalScrollContentView (at ScrollView.js:1007)
   in AndroidHorizontalScrollView (at ScrollView.js:1147)
    in ScrollView (at HomeHeader.js:165)
    in RCTView (at View.js:35)
    in View (at HomeHeader.js:164)
    in RCTView (at View.js:35)
    in View (at HomeHeader.js:85)
    in HomeHeader (at Sample.js:21)
    in RCTView (at View.js:35)
    in View (at react-native-scrollable-tabview-collapsible/index.js:337)
    in RCTView (at View.js:35)
    in View (at ScrollView.js:1007)
    in RCTScrollView (at ScrollView.js:1147)
    in ScrollView (at react-native-scrollable-tabview-collapsible/index.js:322)
    in ScrollableTabView (at Sample.js:19)\n    in CollapsibleExample (at SceneView.js:9)\n    in SceneView (at StackViewLayout.tsx:888)\n    in RCTView (at View.js:35)\n    in View (at StackViewLayout.tsx:887)\n    in RCTView (at View.js:35)\n    in View (at StackViewLayout.tsx:886)\n    in RCTView (at View.js:35)\n    in View (at createAnimatedComponent.js:151)\n    in AnimatedComponent (at StackViewCard.tsx:93)\n    in RCTView (at View.js:35)\n    in View (at createAnimatedComponent.js:151)\n    in AnimatedComponent (at screens.native.js:59)\n    in Screen (at StackViewCard.tsx:80)\n    in Card (at createPointerEventsContainer.tsx:95)\n    in Container (at StackViewLayout.tsx:971)\n    in RCTView (at View.js:35)\n    in View (at screens.native.js:83)\n    in ScreenContainer (at StackViewLayout.tsx:383)\n    in RCTView (at View.js:35)\n    in View (at createAnimatedComponent.js:151)\n    in AnimatedComponent (at StackViewLayout.tsx:379)\n    in PanGestureHandler (at StackViewLayout.tsx:372)\n    in StackViewLayout (at withOrientation.js:30)\n    in withOrientation (at StackView.tsx:103)\n    in RCTView (at View.js:35)\n    in View (at Transitioner.tsx:267)\n    in Transitioner (at StackView.tsx:40)\n    in StackView (at createNavigator.js:61)\n    in Navigator (at createKeyboardAwareNavigator.js:12)\n    in KeyboardAwareNavigator (at createAppContainer.js:429)\n    in NavigationContainer (at JiForum/index.js:108)\n    in Provider (at JiForum/index.js:106)\n    in RCTView (at View.js:35)\n    in View (created by Portal.Host)\n    in Portal.Host (created by Provider)\n    in ThemeProvider (created by Provider)\n    in LocaleProvider (created by Provider)\n    in Provider (at JiForum/index.js:105)\n    in App (at renderApplication.js:40)\n    in RCTView (at View.js:35)\n    in View (at AppContainer.js:98)\n    in _class (at react-native-root-siblings/index.js:80)\n    in RCTView (at View.js:35)\n    in View (at react-native-root-siblings/index.js:79)\n    in RootSiblingsWrapper (at AppContainer.js:112)\n    in RCTView (at View.js:35)\n    in View (at AppContainer.js:115)\n    in AppContainer (at renderApplication.js:39)'

So how is this related to this lib?


Changing Language doesn't work

Hi
Nice middleware!!
Question.
Trying to set the current language through this
app.get("/lang/:lan?", function(req, res) {
  let lan = req.params.lan;
  req.i18n.changeLanguage(newlan);
...

But it doesn't work?
This how I initialise i18n
i18next
  .use(i18nextmiddleware.LanguageDetector)
  .use(filesystemBackend)
  .init({
    // debug: true,
    saveMissing: true,
    lngs: ['en', "pt", "pt-BR", "en-US"],
    load: 'all',
    fallbackLng : "en",
    backend: {
      loadPath: __dirname + '/locales/{{lng}}/{{ns}}.json'
    }
});

what am I missing?

you might not need to use the languagedetector if setting it yourself - or remove you setting it and let the detector do its job: https://github.com/i18next/i18next-express-middleware#detector-options (enabling the path in order array)

I was misunderstanding the logic, and thought that
req.i18n.changeLanguage(newlan);

would set the language and keep it in a session, which is not the default behaviour, and the
  req.i18n.changeLanguage(newlan);

needs to be called directly before rendering any output


New User, Phone Number

Comment from Darcy...
does phone number format matter? I did not receive any text messages from the site, even with 3 admin/volunteers having my number for the testing. (And I know my number will text to itself!) This is hindering my ability to robustly test some of the functionality.

I’m looking into message sending on the staging server. It currently works in development but not on the server. I’ll update here when I have more info.

@SarahOMD this should now be working. I created a new need and assigned Darcy to a shift after I fixed the issue.

@SarahOMD I'm going to close this issue now. Feel free to re-open or open a new issue if any other messaging problems arise.


Update style.css

changed style.css to use font awesome instead of background images

Hello, @garek007 !
It is a very cool PR, but seems, like this project was abandoned...
But there is a good news: here is a alive fork of this project!

Great thanks for that @maxkoryukov ! I will submit my PR over there!


[Insights] Create basic insights dashboard with from-the-box data

Problem Statement
As ReportStream, we would like to see basic metrics in a dashboard, created from the available App Insights data out of the box (no custom insights).
Notes
Refer to the analytics as outlined in the Product Brief https://cdc.sharepoint.com/:p:/r/teams/USDSatCDC/Shared%20Documents/PRIME%20ReportStream/Team%20Documents/Experience%20Team/Artifacts/ReportStream%20Experience%20Team_Product%20Brief_draft%20v3%202021.10.21.pptx?d=w951bdeb553ad410b8149e95f463f3a59&csf=1&web=1&e=eaB5nI.
Refer to shared scratchpad "notes on tools to consider for gathering metrics" https://cdc.sharepoint.com/teams/USDSatCDC/_layouts/15/doc.aspx?sourcedoc={da429e45-5f42-4399-a2b1-70ccc066a718}&action=edit
Related to https://app.zenhub.com/workspaces/experience-607d9d5e68b95200150fec37/issues/cdcgov/prime-reportstream/2834
Acceptance Criteria

[ ] Assume we are just setting up out-of-the-box metrics
[ ] Create an Azure Application Insights dashboard to show the following metrics

Per @kevinhaube - an Engineer does NOT need to pick this ticket up :)

Determined 11/28 with Jewel and Jeremy that the "otb" data is not quite enough to form anything actionable. Site reliability is already handled by the current dashboard. Case closed!


Gulp tasks fail on Windows 10

Description
Gulp tasks error on Windows 10 for all project types tested (angular 1, angular 2, with TypeScript, with SystemJS, with Webpack, TodoMVC, or just with basic application)
Error Message & Stack Trace
[09:43:41] Finished 'styles' after 1.14 s
[09:43:42] 'systemjs' errored after 2.6 s
[09:43:42] Error on fetch for app/hello.ts at file:///[USER-PATH]/Projects/Test/Fountain/app/hello.ts
Loading src\index.ts
Error: ENOENT: no such file or directory, open 'C:[USER-PATH]\Projects\Test\Fountain\app\hello.ts'
at Error (native)
[09:43:42] 'build' errored after 2.65 s
[09:43:42] 'default' errored after 2.66 s
Config
{
"generator-fountain-angular1": {
"version": "1.0.0-rc1",
"props": {
"framework": "angular1",
"modules": "systemjs",
"js": "typescript",
"ci": [],
"css": "scss",
"resolved": "C:\[USER-PATH]\AppData\Roaming\npm\node_modules\generator-fountain-webapp\node_modules\generator-fountain-angular1\generators\app\index.js",
"namespace": "fountain-angular1",
"argv": {
"remain": [],
"cooked": [],
"original": []
},
"sample": "hello",
"router": "uirouter"
}
}
}
Environment
Node.js v6.6.0
win32 10.0.14393
NPM: 3.10.3
Yeoman: 1.8.5

I'm having the same problem and i think i have identified the issue (though i don't know how to solve it at the moment). The task systemjs
function systemjs(done) {
  const builder = new Builder('./', 'jspm.config.js');
  builder.config({
    paths: {
      "github:*": "jspm_packages/github/*",
      "npm:*": "jspm_packages/npm/*"
    },
    packageConfigPaths: [
<EMAIL_ADDRESS>      'npm:*.json',
      'github:*/*.json'
    ]
  });
  builder.buildStatic(
    `${conf.path.src('index.ts')} + ${conf.path.tmp('templateCacheHtml.ts')}`,
    conf.path.tmp('index.js'),
    {
      production: true,
      browser: true
    }
  ).then(() => {
    done();
  }, done);
}

checks index.ts which imports from ./app
import {TodoService} from './app/todos/todos.ts';
import {App} from './app/containers/App.ts';
import {Header} from './app/components/Header.ts';
import {MainSection} from './app/components/MainSection.ts';
import {TodoTextInput} from './app/components/TodoTextInput.ts';
import {TodoItem} from './app/components/TodoItem.ts';
import {Footer} from './app/components/Footer.ts';
import 'angular-ui-router';
import routesConfig from './routes.ts';

so when building gulp the root folder is the project folder, not the src folder (which contains app), therefore it will look for "projectname/app/files" rather than "projectname/src/app/files" when importing from index.ts

I have found this: https://github.com/FountainJS/generator-fountain-webapp/issues/81 the last comment has a solution for Angular 1 + systemjs


Can't run within a Job due to version check

I'm trying to run this as part of a job (just locally for now, but eventually remotely), but the minimum version check is failing.
Importing the module like this:
Import-Module FSWatcherEngineEvent

as part of the job fails with this message:
The current Windows PowerShell host is: 'ServerRemoteHost' (version <IP_ADDRESS>). The module 'C:\Users\...\Documents\WindowsPowerShell\Modules\FSWatcherEngineEvent\1.5.0\FSWatcherEngineEvent.psd1' requires a minimum Windows PowerShell host version of '5.1' to run.
    + CategoryInfo          : ResourceUnavailable: (C:\Users\rtheob...ngineEvent.psd1:String) [Import-Module], InvalidOperationException
    + FullyQualifiedErrorId : Modules_InsufficientPowerShellHostVersion,Microsoft.PowerShell.Commands.ImportModuleCommand
    + PSComputerName        : localhost

If I import the module outside the job, I get a different error:
The 'New-FileSystemWatcher' command was found in the module 'FSWatcherEngineEvent', but the module could not be loaded. For more information, run 'Import-Module FSWatcherEngineEvent'.
    + CategoryInfo          : ObjectNotFound: (New-FileSystemWatcher:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CouldNotAutoloadMatchingModule
    + PSComputerName        : localhost

Is this because I only installed the module using scope "CurrentUser"? I'm on a CORP box and don't have admin privileges.
I came across this thread on reddit, but I'm not sure if it is applicable.
https://www.reddit.com/r/PowerShell/comments/s1fcvc/startjob_ps_version_inconsistency/

Hi  @randallt,
sorry for the late response, I was AFK for some days.
About the loading outside of the job: The module was found but couldn't be loaded. Please try to load the module using Import-Module FSWatcherEngineEvent -Verbose. This should give us more details why the module wasn't loading. If this doesn't work a job can't do it either.
About the version check in the remote host: I don't have an env currently to try this out. What you could try is to remove the version requirement in the module manifest FSWatcherEngineEvent.psd1': Comment the line PowershellHostVersion="5.1" out by adding # at the beginning or simply remove the line and try again. Maybe this helps with the remote host.

I think the root cause ist what you have described with the second part of the original issue: the module isn't found.
This can be fixed by adding the parent directory of the module to the $env:PSModulePath . To inspect what is in the module path when starting a job (locally or remotely) you my just start the job code with the line $env:PSModulePath to add the current value of the module path to the output. I tried the following:
# content of a script file c:\temp\test-fsw-job.ps1
$env:PSModulePath
Import-Module -Name FSWatcherEngineEvent
New-FileSystemWatcher -SourceIdentifier "myevent" -Path c:\temp\files
Register-EngineEvent -SourceIdentifier "myevent" -Action { $event | ConvertTo-Json | Write-Host }

# now start the script file as job and display the output (this is PS7)
Start-Job -FilePath .\test-fsw-job.ps1 | Receive-Job  -Wait -AutoRemoveJob -WriteEvents

The output shows at first the content of the psmodule path followed by the created engine event related to the FS watcher. If the modules parent path isn't in the list for dirs to search for modules you may add the path to the profile the job is reading on startup or you load the module by specifying the path to the Import-Module command.


Prometheus metrics

This is a bit of a stretch but it would be great if there was a way to scrape prometheus metrics. Grabbing the number of watches, strikes and removals for grafana would make it a lot easier to monitor or even alert on.

Hi @disconn3ct
This is indeed a bit of a stretch, but maybe in combination with #53 so that it'll output json which can then be interpreted for use-cases like these.
You'd need to build your own adapter to work as a broken between Swaparr and prometheus though.
But to built in native-support for prometheus is not something I currently have the time for - feel free to contribute.
Might not be the answer you were hoping for but JSON, for now, is the best thing I can offer currently.
^ Thijmen


how to understand use_bert_emb?

I change the bert model to Albert model.
But when use_bert_emb is set to true, the model reports an error: RuntimeError: The size of tensor a (128) must match the size of tensor b (768) at non-singleton dimension 2.
if use_bert_emb is set to false, the model works normally.
How can I solve this problem?

I have the same problem with you. Have you solved it?

Sorry, the problem has not been solved for the time being.
| |
Shuai Zhao
|
|
<EMAIL_ADDRESS>|
Signature is customized by Netease Mail Master
On 08/21/2020 15:16, GuanNiPiShi123 wrote:
I have the same problem with you. Have you solved it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Check chapter 14 of "Introduction to algorithms" by CLRS to augment the RBT ds

The BST has a rank method, which works in linear time instead of the possible optimal logarithmic time...

Add support for

Order statistics tree (OS-RANK, OS-SELECT) in log(n) time by augment the RBT.

Add support for interval trees


Add linters

Add linters
Signed-off-by: Shayegan Hooshyari<EMAIL_ADDRESS>Co-authored-by: James Peach<EMAIL_ADDRESS>

@jpeach Can I get a PR review?

@Glyphack will try to review and land this over the Christmas break.

Thanks for tackling this @Glyphack!
Most of my comments are nits, so if you prefer, we can land this PR and fix them in follow up PRs. Let me know your preference :)

I think we can resolve them now, I'll do it.


Add a doc about how to upgrade to v0.2.

So that we have something to link against from docker.

LGTM after comments, thanks so much for starting this! :smile:

LGTM


Add Transactions reference

Added a transactions reference which simplifies the API page and explains other undocumented functionality.
Clarified schema cache and errors page using https://hemingwayapp.com/.

Closes https://github.com/PostgREST/postgrest-docs/issues/511.
Closes https://github.com/PostgREST/postgrest-docs/issues/384

Builds fail bc of
ImportError: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with OpenSSL 1.0.2n  7 Dec 2017. See: https://github.com/urllib3/urllib3/issues/2168

https://github.com/urllib3/urllib3/issues/2168
https://github.com/psf/requests/issues/6432


IOS 14.4 | secureTextEntry makes app really slow

Description
As soon as my app contains a TextInput with "secureTextEntry" prop set to true, the app becomes REALLY slow when typing in any TextInput present on the screen.
React Native version:
Run react-native info in your terminal and copy the results here.
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Create a TextInput with the prop "secureTextEntry" set on true
Try typing in any TextInput present in the component
Observe how slow it is

Expected Results
Describe what you expected to happen.
It shouldn't be slow obv.
Snack, code example, screenshot, or link to a repository:
<TextInput secureTextEntry />

Fixed on IOS 14.4.1

Fixed on IOS 14.4.1


Instructions and config to run in AWS EKS

I have tested all steps described here in this blog post: https://blog.pythian.com/using-el-carro-operator-on-aws/
Did minor changes from that post to create these instructions, making it easier to follow and using fewer pre-created files.

Awesome, thank you so much Nelson!

/ok-to-test

Thank you Ash for the review and suggestions, all make sense.
I have made those changes and committed, please check.


liked songs only showing 10k out of the total 43k

🧐 Have you fully read the README and viewed similar issues?

[X] I have read the README
[X] I have read the FAQ
[X] I have checked previous issues, open and closed
[X] I have completed all troubleshooting steps and the issue is not present on stock but returns after re-applying SpotX-Bash

Сountry of your account
USA
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify version: Spotify for macOS (Apple Silicon) <IP_ADDRESS>5
- OS: macOS Sonoma 14.5
- Shell: zsh

📝 Description

Go to liked songs
Expect to see all 43,213 songs
I only see 10,000 songs and the list ends
...

🖥 Terminal Input/Output
I didnt use any commands. The only command I ever use is the main one for updating spotX each time spotify updates.
📸 Screenshots
No response

so as soon as you uninstall SpotX while using the exact same Spotify version number, all ~43k songs show up again?

correct!!

Hmm. I've no way to attempt to test or correct this as I have no accounts with liked songs even close to that number. SpotX custom patches shouldn't be doing anything to affect this... so I would assume it's a toggled ON experimental setting causing an issue.

Confirm the issue is related to experimental settings, use the -e flag to exclude all experimental settings.

bash <(curl -sSL https://spotx-official.github.io/run.sh) -e

This installs SpotX but doesn't enable any experimental features.

Do all liked songs show up? yes/no

Testing with previous versions can help narrow down when this issue started (using the -V versionnumberhere flag with the command). Versions can be found here - https://docs.google.com/spreadsheets/d/1wztO1L4zvNykBRw7X4jxP8pvo11oQjT0O5DvZ_-S4Ok/ you'd be using the 1.2.xx.xxx number only version.. leaving off the .gXXXXXXXX

bash <(curl -sSL https://spotx-official.github.io/run.sh) -B -V <IP_ADDRESS>6

Do all liked songs show up? yes/no
If no, what about 1.2.46.xxx, 1.2.45.xxx.. etc etc.. at what downgraded version do all liked songs show up again?

yes they do!! thank u so much omg

yes they do!! thank u so much omg

v<IP_ADDRESS>9 came out today on macOS... I'm curious if you have all ~43k liked songs show up WITHOUT using the -e flag when running SpotX-Bash now?

hey!! so im still pretty new to coding/programming stuff, and so the way
i've been using it since i first installed spotx bash is: every few weeks
or so when i finally see an ad in my spotify i'll just close spotify, copy
the main line of code from the github into my terminal and run it, and then
spotify works ad free for a good month before i need to paste and run the
code again. I thought this was "updating" it but from reading the github
more thoroughly I see that its supposed to auto update? either way, i'm not
entirely sure how to check what you want to know, cuz I went to go paste
and run the code since seeing this but I get "spot X is already installed"
or something.
On Mon, Oct 21, 2024 at 12:04 PM jetfir3 @.***> wrote:

yes they do!! thank u so much omg
v<IP_ADDRESS>9 came out today on macOS... I'm curious if you have all ~43k
liked songs show up WITHOUT using the -e flag when running SpotX-Bash now?
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/SpotX-Official/SpotX-Bash/issues/63*issuecomment-2427498299__;Iw!!Ka_JY85zDv0FFw!jf_6bq1o_MOr4jOCcGJzzSifzgnIIMLm_QQP18kikYCgyumFTqd-TyBO4gliz06pJhuhjwQKYaYhoqF_hd244Gd8Y0U$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A5OHJNL35U7VQ2Z5TJHDNLDZ4VF3VAVCNFSM6AAAAABQEMFHVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRXGQ4TQMRZHE__;!!Ka_JY85zDv0FFw!jf_6bq1o_MOr4jOCcGJzzSifzgnIIMLm_QQP18kikYCgyumFTqd-TyBO4gliz06pJhuhjwQKYaYhoqF_hd24eb2YrAc$
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

hey!!

You have to run SpotX-Bash again every few weeks or so because you aren't blocking Spotify updates (this requires adding the -B flag to the end of the command)... so Spotify automatically updates itself at some point which overwrites the SpotX-Bash patches.
It's perfectly fine to continue doing things that way.. letting Spotify update itself when it needs to -- just know it's not a matter of "SpotX-Bash stopped working out of nowhere" but that new Spotify updates erase all patches made since the client is updating all its files.
Alternatively, you can have SpotX-Bash update your Spotify to the latest version instead by adding the --installmac flag to the command... this will install the latest version then patch it after installation. You could also add the "block updates" option.. but up to you.
Example command would be:
bash <(curl -sSL https://spotx-official.github.io/run.sh) --installmac -B

or if you don't want to worry about remembering flags.. you can always use the -i (interactive) flag to run through things with you...
bash <(curl -sSL https://spotx-official.github.io/run.sh) -i

and you will be asked which features you want to apply.. including if you want to update to the latest version.

all my liked songs show up now without using the "-e" :)
On Thu, Oct 24, 2024 at 5:42 AM jetfir3 @.***> wrote:

hey!!
You have to run SpotX-Bash again every few weeks or so because you aren't
blocking Spotify updates (this requires adding the -B flag to the end of
the command)... so Spotify automatically updates itself at some point which
overwrites the SpotX-Bash patches.
It's perfectly fine to continue doing things that way.. letting Spotify
update itself when it needs to -- just know it's not a matter of
"SpotX-Bash stopped working out of nowhere" but that new Spotify updates
erase all patches made since the client is updating all its files.
Alternatively, you can have SpotX-Bash update your Spotify to the latest
version instead by adding the --installmac flag to the command... this
will install the latest version then patch it after installation. You could
also add the "block updates" option.. but up to you.
Example command would be:
bash <(curl -sSL https://spotx-official.github.io/run.sh https://urldefense.com/v3/__https://spotx-official.github.io/run.sh__;!!Ka_JY85zDv0FFw!iV9VkqBJkFKKUMAyrjwzoqr-6Pi8dvL4tATdVb0eua2LgBbeFEUZ6UvvqsepcxYobYmE7ftUZD38v0WIWmM7ejvPwcw$) --installmac -B
or if you don't want to worry about remembering flags.. you can always use
the -i (interactive) flag to run through things with you...
bash <(curl -sSL https://spotx-official.github.io/run.sh https://urldefense.com/v3/__https://spotx-official.github.io/run.sh__;!!Ka_JY85zDv0FFw!iV9VkqBJkFKKUMAyrjwzoqr-6Pi8dvL4tATdVb0eua2LgBbeFEUZ6UvvqsepcxYobYmE7ftUZD38v0WIWmM7ejvPwcw$) -i
and you will be asked which features you want to apply.. including if you
want to update to the latest version.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/SpotX-Official/SpotX-Bash/issues/63*issuecomment-2435190896__;Iw!!Ka_JY85zDv0FFw!iV9VkqBJkFKKUMAyrjwzoqr-6Pi8dvL4tATdVb0eua2LgBbeFEUZ6UvvqsepcxYobYmE7ftUZD38v0WIWmM7HilGFQE$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/A5OHJNJMX62OB3FSOGICAMLZ5DTLRAVCNFSM6AAAAABQEMFHVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZVGE4TAOBZGY__;!!Ka_JY85zDv0FFw!iV9VkqBJkFKKUMAyrjwzoqr-6Pi8dvL4tATdVb0eua2LgBbeFEUZ6UvvqsepcxYobYmE7ftUZD38v0WIWmM7SGLxM-M$
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Update to .NET 8.0 & Update Packages

As far I can tell, this replaces the existing in-process examples with the new out-of-process examples.
Since the in-process model is still supported and widely used in .NET 6.0, and is going to be supported at some point soon in .NET 8.0, I don't think we should be removing support for that.
I think it would make sense for this library to support both styles of function for as long as both are supported (which will presumably be at least until November 2026, since .NET 8.0 is going to support in-proc).
So instead of just modify the existing example projects to use out of proc shouldn't we keep those as they were, and add new ones using the isolated model so that we support both for the time being?

#427 now covers these changes


Does kosmos-2 supports finetune on new scene datasets with a different input size ?

Of course, you can tune it on custom datasets.
As for the input size, we downsample the image tokens to 64 in the resampler module.

Of course, you can tune it on custom datasets. As for the input size, we downsample the image tokens to 64 in the resampler module.

I'd like to finetune kosmos-2 with my own dataset. But the input size too small is not suitable for my task(many tiny object and text), so i'd like to know whether kosmos-2 supports bigger input image size, such as 448、560?

I'd like to finetune kosmos-2 with my own dataset. But the input size too small is not suitable for my task(many tiny object and text), so i'd like to know whether kosmos-2 supports bigger input image size, such as 448、560?

Of course it is possible, but there are two points to note:

Kosmos-2's vision encoder is built on clip-l/14, and its pre-training input is 224,
Kosmos-2 use a resampler, no matter how many the input tokens are, we will downsample to 64, I am not sure if this meets your requirements.

I'd like to finetune kosmos-2 with my own dataset. But the input size too small is not suitable for my task(many tiny object and text), so i'd like to know whether kosmos-2 supports bigger input image size, such as 448、560?

Of course it is possible, but there are two points to note:

Kosmos-2's vision encoder is built on clip-l/14, and its pre-training input is 224,
Kosmos-2 use a resampler, no matter how many the input tokens are, we will downsample to 64, I am not sure if this meets your requirements.

Thans for your kind reply！I will try to do that.

I'd like to finetune kosmos-2 with my own dataset. But the input size too small is not suitable for my task(many tiny object and text), so i'd like to know whether kosmos-2 supports bigger input image size, such as 448、560?

Of course it is possible, but there are two points to note:

Kosmos-2's vision encoder is built on clip-l/14, and its pre-training input is 224,
Kosmos-2 use a resampler, no matter how many the input tokens are, we will downsample to 64, I am not sure if this meets your requirements.

Where can I modify this resampler? Thank you so much

Hi, I'm interested in this topic too, but I cannot figure out how to use the code you provided (neither if I can) for fine-tuning kosmos2 on my downstream task. Could you point me out to something? Or should I use the Trainer provided by the Huggingface library?
Thank you

Hi all, I have the same question too, wanna finetune KOSMOS-2.
Could someone provide a tutorial about finetune KOSMOS-2 plz? (I am totally a beginner)
Best

Hi all, I have the same question too, wanna finetune KOSMOS-2. Could someone provide a tutorial about finetune KOSMOS-2 plz? (I am totally a beginner) Best

Any update on this issue?


Fix URL path in documentation link to ensure correct format

Checklist

[x] A link to the issue/user story that the pull request relates to
[x] How to recreate the problem without the fix
[x] Design of the fix
[x] How to prove that the fix works
[x] Automated tests that prove the fix keeps on working
[x] Documentation - any JSDoc, website, or Stackoverflow answers?

Issue/User story

The pull request addresses an issue where the documentation link on the landing page generates a URL with an incorrect path, specifically introducing double slashes (//) after the version number, leading to inconsistent URL formatting.
Steps to Reproduce

Navigate to the landing page.
Hover over the documentation link.
Observe the URL preview at the bottom left of the browser window, which incorrectly shows // after the version number.

Existing issues

[x] GitHub Issues
[ ] Stack Overflow issues
[ ] Discord history

Design of the fix

The fix involved removing the additional slash from the href attribute in the anchor tag for the documentation link. This approach was chosen because it directly addresses the root cause of the issue - an extra slash introduced in the URL path, ensuring that URLs are generated correctly without manual intervention or further processing.
Validation of the fix

Manual testing was conducted to confirm that the URL is now correctly formatted without double slashes.
Automated Tests

No new automated tests were added as the fix is straightforward and does not introduce new functionality. Existing tests ensure that the documentation links are accessible and lead to the correct pages.
What documentation has been provided for this pull request

N/A - The change does not introduce new functionality or concepts that require additional documentation beyond the code change itself.

@PrathamSikka24 you fixed this in #1532 so I'm closing this PR


The password inserted into the database when adding a user is a hash value, but it is plaintext when updating a user.

I ran into this problem when I created a demo for testing using this project.Maybe it's a bug, so submit this issues.Thanks for checking out.
The password inserted into the database when adding a user is a hash value, but it is plaintext when updating a user.
post -> /api/v1/users     Add new user.The password inserted into the database when adding a user is a hash value.
PUT -> /api/v1/users/2  Update user info.The password for inserting the database is in plaintext

I analyzed it for perhaps the following reasons.
The encryption process is done here, and the  __init__  method is executed when adding a user, but not called when updating. Maybe that's the reason.

Related Codes：
https://github.com/karec/cookiecutter-flask-restful/blob/d2441a22dd67801f21e7c7e06049051da326d71a/{{cookiecutter.project_name}}/{{cookiecutter.app_name}}/api/resources/user.py#L152-L159
https://github.com/karec/cookiecutter-flask-restful/blob/d2441a22dd67801f21e7c7e06049051da326d71a/{{cookiecutter.project_name}}/{{cookiecutter.app_name}}/api/resources/user.py#L87-L94
https://github.com/karec/cookiecutter-flask-restful/blob/d2441a22dd67801f21e7c7e06049051da326d71a/{{cookiecutter.project_name}}/{{cookiecutter.app_name}}/models/user.py#L4-L19

Hello @LANVNAL and thanks for reporting this.
This is indeed a bug and you pin-pointed precisely the issue: __init__ is never called by marshmallow-sqlalchemy when updating and I've hadn't added necessary logic to handle that case.
To fix this and avoid future issues with this, I've put the password hashing logic in an hybrid_property with an associated setter instead of putting the logic inside the endpoint itself.
I've also added a check inside the test_put_user
Related commit

Issue fixed on master, I'm closing this issue. Thanks again for reporting !


CII best practices

I've been listening to ChangeLog podcast recently about Core Infrastructure Initiative. In short, they propose a list of things a project should follow. This was mostly for core projects, but I thought that we may find some interesting things to look at. The list of criteria is here.
One of the things they recommend is doing static and dynamic analysis of the code. We don't do that, I think.

We can probably add cppcheck to our CI.
As far as dynamic analysis, valgrind is already in our Docker image.  Another option would be to add option flags to the compiler to use the address sanitizer.

Will do dynamic analysis with sanitize flags and valgrind, each in a separate build on master with frequency off builds to be determined. Will also do dynamic analysis of test coverage.
For static analysis we will look at cppcheck but will need a plan to address errors in output; also known to have false positives. Will also look at coverity
@Rombur will take a look at adding sanitize flags to build and report back on errors and run times
@ssslattery will look at how coverage testing is setup in cap and report back
@sslattery will run a few tests with valgrind and will look at CASL valgrind suppression file. Will also look at how valgrind tests are currently run in CASL. There may be a test prefix capability.

I really like the coverage output - I think this would be really helpful

Ran a few of the operator tests in the docker with valgrind. Clean so far except for MPI...

See #153 for @Rombur results on sanitize

@rombur will open new issue on sanitize builds in CI

Just giving up on this.


chore(deps): update to github.com/blinklabs-io/gouroboros v0.89.1

Several of the functions related to gOuroboros from this repository are now available within gOuroboros upstream. As such, this updates the dependency to the correct upstream version and removes the migrated code.

Thank you very much to the Gouroboros team. I will conduct a local test to verify the functionality of the code. At first look, it appears that the code simply relocates functions into the Gouroboros library. If possible, could you please review this file: https://github.com/cardano-foundation/cardano-ibc-incubator/blob/d4d8ec9b933b1fcd57f2a82c488e9da6ec625f81/relayer/relayer/chains/cardano/types.go#L16
The unmarshal function from Gorobouros works well, but the marshaling part kind of doesn't work for me. I have cloned the structs and modified some properties in those structs.

The code is moved to gOuroboros nearly verbatim (only a couple variable changes and some nil pointers avoidance by error checking and failing fast). Here, I only changed a couple functions to take UTxOs as an argument so it could consume the gOuroboros objects. I'll take a look at the relayer code, too.

Thank you very much to the Gouroboros team. I will conduct a local test to verify the functionality of the code. At first look, it appears that the code simply relocates functions into the Gouroboros library. If possible, could you please review this file:
https://github.com/cardano-foundation/cardano-ibc-incubator/blob/d4d8ec9b933b1fcd57f2a82c488e9da6ec625f81/relayer/relayer/chains/cardano/types.go#L16
The unmarshal function from Gorobouros works well, but the marshaling part kind of doesn't work for me. I have cloned the structs and modified some properties in those structs.

I passed this along to the team and we will make it a priority to test marshal and unmarshal on the objects going forward. We will take these changes and put them into a future gOuroboros release and will submit a PR here for those changes once it's resolved.

I updated this against the latest changes and resolved the conflict.

OK. I merged your PR, then resolved the conflicts that occurred in here. After that, I merged main after syncing with your upstream main here. Finally, I updated gOuroboros to the latest release.

Note: we've updated the various types, such as BabbageTransaction so they should serialize correctly now... also, I added PaymentKeyHash and StakeKeyHash functions to ledger.Address which returns a Blake2b224 (so you can .String() it) which should be able to replace the echovl/cardano-go functionality in the relayer's func (cc *CardanoProvider) MsgTransfer


datapy create table

write a datapy create table functionality similar to:
datapy create_table -s <schema_name> -t <table_name>

Along with comment on #4 I'd like to mirror a bash/sql like inutitive syntax
datapy --create --table <schema_name>.<table_name> feels more intuitive
would also like to accept datapy -ct <schema_name>.<table_name>

default schema would be appreciated as well so that tables can be created without schema name.


Changelog automation

Closes #14

This changelog file is used to organize and categorize the changes made to a project in a clear and structured manner. It helps users, contributors, and developers quickly understand the project's evolution by segmenting changes into different categories:

User Stories 👑: Groups major directives and initiatives of the project, often represented by epics.

New Features 🎉: Lists enhancements and user-visible features that enrich the user experience.

Bug Fixes 🐛: Records resolved issues to ensure better stability and performance of the application.

Other Changes: Captures all other modifications that don't fall under the previous categories, ensuring no update is overlooked.
The labels associated with each category allow for automatic classification and ease the management of versions and the communication of updates.
Additionally, this structure facilitates the automation of release creation, streamlining the process of preparing and publishing new releases.

This file needs to be added to each repository, for example: fertiscan-backend/.github/release.yml. Therefore, I'll consider changing this PR to include an automation script to add the release.yml inside every repo

Won't de. The current workflow for the release creation is using mikepenz/release-changelog-builder-action@v5. Wich enable the possiblity to automate the changelog


⚠ Bofrost.hr has degraded performance

In 3a493ac, Bofrost.hr (https://www.bofrost.hr) experienced degraded performance:

HTTP code: 200
Response time: 12945 ms

Resolved: Bofrost.hr performance has improved in 0850698.


Update README URLs based on HTTP redirects

Created with https://github.com/dkhamsing/frankenstein
GitHub Corrected URLs

Was
Now

https://github.com/DABSquared/omnipay-cybersource-soap
https://github.com/Klinche/omnipay-cybersource-soap

HTTPS Corrected URLs

Was
Now

http://getcomposer.org/
https://getcomposer.org/

Other Corrected URLs

Was
Now

https://poser.pugx.org/omnipay/omnipay/d/total.png
https://poser.pugx.org/omnipay/omnipay/d/total

https://poser.pugx.org/omnipay/omnipay/version.png
https://poser.pugx.org/omnipay/omnipay/version

Thanks!


[FEATURE] Inclusion of  API response for the audit logs having audit_category as INDEX_EVENT to infer the result of the INDEX_EVENT happened.

Is your feature request related to a problem?
Audit logs  having audit_category as INDEX_EVENT does not provides sufficient information to determine the outcome of the performed INDEX_EVENT.
For instance, when an index deletion is attempted, the audit logs may record two scenarios:
Case 1: Failure - Due to the absence of the index:
{
    "audit_cluster_name": "viv1-enhan",
    "audit_node_name": "opensearch-client-55d98b9456-99drf",
    "audit_trace_task_id": "_Fcuu8R9Rt-jQZdSA:357",
    "audit_transport_request_type": "DeleteIndexRequest",
    "audit_category": "INDEX_EVENT",
    "audit_request_origin": "REST",
    "audit_node_id": "_Fcuu8R9Rt-jQZdSA",
    "audit_request_layer": "TRANSPORT",
    "@timestamp": "2024-02-12T13:43:32.653+00:00",
    "audit_format_version": 4,
    "audit_request_remote_address": "<IP_ADDRESS>",
    "audit_request_privilege": "indices:admin/delete",
    "audit_node_host_address": "<IP_ADDRESS>",
    "audit_request_effective_user": "admin",
    "audit_trace_indices": [
        "akhj"
    ],
    "audit_node_host_name": "<IP_ADDRESS>"
}

Case 2: Success - The index exists and is successfully deleted:
{
    "audit_cluster_name": "viv1-enhan",
    "audit_node_name": `"opensearch-client-55d98b9456-99drf",`
    "audit_trace_task_id": "_Fcuu8R9Rt-jQZdSA:1169",
    "audit_transport_request_type": "DeleteIndexRequest",
    "audit_category": "INDEX_EVENT",
    "audit_request_origin": "REST",
    "audit_node_id": "_Fcuu8R9Rt-jQZdSA",
    "audit_request_layer": "TRANSPORT",
    "@timestamp": "2024-02-12T13:50:47.608+00:00",
    "audit_format_version": 4,
    "audit_request_remote_address": "<IP_ADDRESS>",
    "audit_request_privilege": "indices:admin/delete",
    "audit_node_host_address": "<IP_ADDRESS>",
    "audit_request_effective_user": "admin",
    "audit_trace_indices": [
        "akhj"
    ],
    "audit_trace_resolved_indices": [
        "akhj"
    ],
    "audit_node_host_name": "<IP_ADDRESS>"
}

We notice that the audit logs produced for these cases are quite similar and do not provide sufficient information to deduce the outcome of the INDEX_EVENT.
What solution would you like?
Failure Case
curl   -k -uadmin:admin -XDELETE https://<IP_ADDRESS>:9200/akhj
API returned response:
{"error":{"root_cause":[{"type":"index_not_found_exception","reason":"no such index [akhj]","index":"akhj","resource.id":"akhj","resource.type":"index_or_alias","index_uuid":"_na_"}],"type":"index_not_found_exception","reason":"no such index [akhj]","index":"akhj","resource.id":"akhj","resource.type":"index_or_alias","index_uuid":"_na_"},"status":404}

Success case
curl   -k -uadmin:admin -XDELETE https://<IP_ADDRESS>:9200/akhj
API returned response:
{"acknowledged":true}

The audit logs would offer greater significance if they included either the complete API response or pertinent segments of it, providing sufficient information to infer the outcome of the INDEX_EVENT.
What alternatives have you considered?
No alternatives have been considered.
Do you have any additional context?
None.

[Triage] Thanks for filing this issue @vp2312. This issue has clear exit criteria, marking it as triaged.


Scientific/exponential format silently behaves unexpectedly

What happened?
Possibly a duplicate of #1118 (though this issue seems different in that these inputs were rejected before).
Entering numbers with scientific notation silently produces the wrong number.
I think instead such inputs should either be rejected or passed through to sql.
PRQL input
from numbers
select [ exponential = 1e9 ]

SQL output
SELECT
  19 AS exponential
FROM
  numbers

Expected SQL output
SELECT
  1E9 AS exponential
FROM
  numbers
-- Or the prql compiler rejects the input

MVCE confirmation

[X] Minimal example
[x] New issue

Anything else?
This was on the playground. Perhaps it is different in other versions.

Hi, thank you for reporting this!
I can reproduce with the latest dev version.

Some options for the implementation:

Convert to float in rust, then convert to scientific when writing. But not trivial, likely requires heuristics on when to use scientific notation, and some float approximation.

We could also store smaller numbers as integers, up to 2^32 for i32...

Store as string
Raise an error

Probably string is easiest, float is best. Any thoughts from others

Actually converting to a float was easiest, in #2865
It currently will write the whole number out, which isn't ideal with huge numbers, but fixing that is a bit more work, as it requires writing a heuristic for when to express fully vs in scientific notation. (Contributions welcome if anyone wants to take a swing)


Added group Detection Functionality with unit test

Added a function in template class that prints the group of a team member based on the name they used when initializing with the constructor.

Please create a new branch for this. Please also submit an issue describing this new functionality. Then submit a new PR.
As far as the changes, let's have the function return a string instead of printing to console. You can then adjust the test appropriately. Lmk if you have questions on this.


mod needed?

Description
the mod is asking for a Cumilation mod.
but I can't find it downloaded and it's not showing a website to download it
To Reproduce
Steps to reproduce the behavior:

Start Rimworld
click mods
select RJW-Genes
check the dependency box.

Screenshots
none

https://github.com/vegapnk/Cumpilation
I found it from the RJW Loverslab page. You can find pretty much everything with any level of integration with RJW there.  It's more like a full list of every 18+ mod for Rimworld.

thank you for the help.
oddly, Git didn't list it with a link as a dependency.


Icon request: NAVER

URL of the site or app (required): https://www.naver.com/
Logo resources for the icon (optional, but recommended): main icon: PNG logo
Additional resources to see for logo: https://m-logoproject.naver.com/logonaver

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/NAVER.svg
❓ How to add icon(s) to Aegis?
P.S. Next time, please do not copypaste issue template to "blank issue". Use the issue template as is, copypasting removes labels.


Local IP to Domain

Hello,
I installed Postal on my Proxmox server. But I can connect with the local IP address. Connection with domain name is not possible.
Domain screen:

ERR_TOO_MANY_REDIRECTS

I updated postal.yml and restart container (LXC) but I still not connect via domain.
What could be the reason?
Thank you.

It might be that your nginx is redirecting you into a loop, I'm not sure how LXC is involved but presumably theres some confusion at the proxy end.


Bug #7 appeared again on the latest version

Hey @kenmoore
The latest plugin got an old issue again. Take a look at #7
Thanks in advance!
Best!
Ivan

@kenmoore any ideas ?

I'm also facing this issue.

Same here :(

Any updates?

Sorry everyone, it's been a long time since I've written Sketch plugins and I'm not up to date on the latest API changes. It would be great if Bohemian could proactively determine which plugins they were going to break before a new release, and notify the developers with clear guidance on how to prevent breakages. However, each time this type of thing happens all I can do is ask @bomberstudios (Ale at Bohemian) for his direct assistance.
Ale...help, please?

still waiting...


[FEATURE]  Flint SOP

What is index state in query_requet_index is deleted?

Closing. Reopen if needed.


Invalid image URL crashes the client app

Step 1: Describe your environment

Xcode version: 12.5.1
Firebase SDK version: 8.4.0
Installation method: CocoaPods
Firebase Component: IAM

Step 2: Describe the problem
Steps to reproduce:

Compose a new in-app message in the Firebase Console, and leave a trailing whitespace in the image URL;
Make the message appear in the app (send a test message or publish the campaign).

Why the crash happens:

In FIRIAMFetchResponseParser’s convertToMessageDefinitionWithMessageDict:, [NSURL URLWithString:imageURLStr] returns nil because of the whitespace;
The msgData object is created with nil in imageURL;
nil URLs produce no image data;
The imageData parameter in FIRInAppMessagingDisplayMessage initializer is not nullable, so the entire displayMessage object becomes nil;
FIRInAppMessagingDisplay’s displayMessage:displayDelegate: method assumes messageForDisplay is not optional, so when a nil is received in the Swift runtime, a crash occurs.

Next Steps
In addition to the check introduced in the associated pull request, a new method could be added to FIRIAMFetchResponseParser to sanitize URL strings by handling common errors, such as whitespaces.
Images

Stack Trace
Crashed: com.apple.main-thread
0  libswiftCore.dylib             0x1b68d45b8 swift_getObjectType + 60
1  <REDACTED>                     0x10358a344 closure #1 in <REDACTED>.displayMessage(_:displayDelegate:) +<PHONE_NUMBER> (<compiler-generated>:4390806340)
2  <REDACTED>                     0x1021a89b0 thunk for @escaping @callee_guaranteed () -> () +<PHONE_NUMBER> (<compiler-generated>:4369959344)
3  libdispatch.dylib              0x1a8de69a8 _dispatch_call_block_and_release + 24
4  libdispatch.dylib              0x1a8de7524 _dispatch_client_callout + 16
5  libdispatch.dylib              0x1a8d995b4 _dispatch_main_queue_callback_4CF$VARIANT$mp + 904
6  CoreFoundation                 0x1a909f6bc __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 12
7  CoreFoundation                 0x1a909a590 __CFRunLoopRun + 1724
8  CoreFoundation                 0x1a9099ba8 CFRunLoopRunSpecific + 424
9  GraphicsServices               0x1b3209344 GSEventRunModal + 160
10 UIKitCore                      0x1ad1d53e4 UIApplicationMain + 1932
11 <REDACTED>                     0x10215cb2c main + 15 (<REDACTED>.swift:15)
12 libdyld.dylib                  0x1a8f218f0 start + 4

It'd be nice as well if we can check for common errors when the IAM is being created in console.


Chart.js bar chart with big data

Hi,
i trying to using the bar chart with a lot of data
but its look like very bad.
i using version 3.2.0
this is my code:
https://codepen.io/kfir-swisa/pen/xxgMzgO
why the bars very small?
why there is no scroll or something?
and how can i working with large data on the chart?
tnx

The reason the bars are small is because otherwise they dont fit in the chart area, if you want  it to become scollable please take a look at this plugin: https://github.com/chartjs/chartjs-plugin-zoom


Internal Server Error make test fail

Hey guys, running check and getting this:
→ ./node_modules/.bin/nsp check
(+) Debug output: "Internal Server Error"
(+) Error: Got an invalid response from Node Security, please email the above debug output to<EMAIL_ADDRESS>
What's going on here?
Thanks

We're currently in the middle of some maintenance, I'm working hard on getting things finished so I can get services back up! Sorry for the interruption!


Added an activator for CLOSE passivator option

If we specify a CLOSE passivator, the connection is closed after use, but no one opens it. This results in exception and false login failure. I propose to set an activator which will open a closed connection. I am not sure though, which credentials will be used in such a case.

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


check Boolean constructor with isBoolean

couldn't pass false as value for [Boolean]
> var Type = require('union-type')
> Type({ Exists: [Boolean] }).Exists(true)
[ true,
  of: { Exists: [Function], case: [Function], caseOn: [Function] },
  name: 'Exists' ]
> Type({ Exists: [Boolean] }).Exists(false)
TypeError: wrong value false passed to location 0 in Exists

That is an accidental omission! Thanks you for catching it.


JSON Query Support

With for example jq, which could run in the browser (jq-web)

That's a great idea, I will definitely implement this. To make it easier to use, I would merge all three json files into one and then remove all fields people don't want to see (like UUIDs).


Design grass LOD shaders

Tweak for a specific planar scale on heightmap that looks good.
Maybe add LOD3 with even less grass.

Closed as of PR #122


Integration with ODIM's services part #6

Changes:

Implemented GET action for all resources under System resource.

Looks good.


vEthernet (WSL) doesn't appear in Network Adapters

Windows Build Number
Version 10.0.22000.100
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Debian (lsb_release -r command doesn't work either!!)
Other Software
No response
Repro Steps
Go to Control Panel -> Network and internet -> Network and sharing centre -> Change adapter settings.  vEthernet (WSL) does not appear there as it should with WSL and Hypervisor Platform features enabled
Expected Behavior
vEthernet (WSL) adapter should exist.
Actual Behavior
vEthernet (WSL) adapter is not present
Diagnostic Logs
No response

The adapter will only appear after you have run WSL once (per boot).

In my case, that wsl vethernet interface doesn't appear after booting wsl2 buy appears in ipconfig /all output.

In my case, that wsl vethernet interface doesn't appear in network control panel after booting wsl2. But appears in ipconfig /all output and in device manager as hidden device.

Same here. I have not that adapter listen in windows 11 beta channel. Could not tweak it.

In my case, that wsl vethernet interface doesn't appear in network control panel after booting wsl2. But appears in ipconfig /all output and in device manager as hidden device.

Mine is the very same.. The linux subsystem shows as:
IP address:    <IP_ADDRESS>/20
Gateway:       <IP_ADDRESS>
I noted a Windows update for WSL was installed today, but nothing changed regarding this issue.
I tried the suggestion (Accessing a WSL 2 distribution from your local area network (LAN)) on this page:
https://docs.microsoft.com/en-us/windows/wsl/compare-versions#accessing-network-applications
setting the proxy port to 60081, but still can't access the WSL virtual machine through, say, Remote Desktop Connection at <IP_ADDRESS>:60081.
So the WSL 2 virtual machine is isolated from the network.

I have a similar issue. Restart-NetAdapter -Name "vEthernet (WSL)" gives me the following error:
+ CategoryInfo          : ObjectNotFound: (vEthernet (WSL):String) [Restart-NetAdapter], CimJobException
+ FullyQualifiedErrorId : CmdletizationQuery_NotFound_Name,Restart-NetAdapter

In addition to that I cannot see vEthernet (WSL) in the Get-NetAdapter 's list,

Same here. Fresh windows 10 install, fresh wsl install. I haven't any WSL network adapter on windows.

Same here, recently installed latest windows 11 update and the adapter Ethernet (WSL) shows on ipconfig, but it doesn't show on Get-NetAdapter

Same for me, but if I use the -All flag it does show up:
Get-NetIPConfiguration -All

EDIT: It shows up with Get-NetAdapter -IncludeHidden

This should be the answer to the issue. 😉

Someone posted elsewhere, disabling ipv6 fixed it for them, and it did for me as well, at least so far.

Works for me like this:
Disable-NetAdapterBinding -Name "vEthernet (WSL)" -ComponentID ms_tcpip6 -IncludeHidden

WSL2, Win11
I was able to fix the internet speed running these commands on powershell as admin
Disable-NetAdapterChecksumOffload -Name "vEthernet (WSL)" -TcpIPv4 -UdpIPv4 -IncludeHidden
Disable-NetAdapterChecksumOffload -Name "vEthernet (WSL)" -TcpIPv6 -UdpIPv6 -IncludeHidden
After that i needed to restart the windows to work properlly
PS: Before I restart Windows I tried to restart the network adapter with this command, but not worked
Restart-NetAdapter -Name "vEthernet (WSL)" -IncludeHidden

same for me, even with -IncludeHidden
build 19044.2604

I had a similar incident after installing then uninstalling VMware, which left my Hyper-V Extensible Virtual Switch entirely broken.  I was unable to load WSL2 for about 10 days due to the missing virtual network adapter.
What fixed it for me was removing this registry key which VMware left behind:
HKEY_LOCAL_MACHINE\SOFTWARE\Classes\CLSID\{3d09c1ca-2bcc-40b7-b9bb-3f3ec143a87b}

After removing this registry key manually, then clicking Network Reset on the Networking Control Panel page, my system came back up with fully working networking, including my WSL2 launching again!
Note: Network Reset made me wait 5 minutes before it rebooted the system.
Hope it helps other people who are stuck!
Other error codes and error messages I experienced:
Windows is still setting up class configuration for this device. (Code 56)
0x80071779
0x80041002

To create a vEthernet (WSL) interface install a new distro from appstore and then you can remove.
For me enabling mirrored networking mode solved the connectivity issues:
Edit %UserProfile%\.wslconfig to include:
[wsl2]
networkingMode=mirrored

Then restart distributions with wsl --shutdown
https://superuser.com/a/1826530/1060778


TypeError: object of type 'IndexFlatIP' has no len()

Hi, I trained roberta-large with my own data by using run_sup_example.sh.  But when I build index for senteced and use model.search, I encountered the error. The building process has no problem, the error is in model.search.
/site-packages/simcse/tool.py", line 64, in encode
    total_batch = len(sentence) // batch_size + (1 if len(sentence) % batch_size > 0 else 0)
TypeError: object of type 'IndexFlatIP' has no len()

I modify the default value of use_faiss to True, then it works. Maybe something wrong

Hi Maydaytyh,
Thanks for spotting our the problem! I'll look into it


Fixed "Start timer" not showing on Remember The Milk

Resolves issue #770
The button is now right to the task title as it worked before.

Thanks very much for this!
Expect a new release of Toggl Button including this change within a week. (We've got to get the Asana fix out, and we can piggyback on that release.)


Publish substrate-test-runtime-client so we can write tests for custom RPC

Right now substrate-test-runtime-client is not published.
This means we can't use it write tests for custom RPC.
We should either publish test related crates or provide alternative solution for builders to write tests.

This is a Substrate specific test runtime, that should not be published.
I don't understand why you not just write your own test runtime?
I think we could publish substrate-test-client, as this is some generic abstraction, but not more.

Maybe I am not asking for the right thing. My real request is be able to write tests for our custom RPC and by looking examples from Substrate, substrate-test-runtime-client is used and hence my request.
Maybe I should write my own test runtime and my own test client? It will be helpful if there are some examples.

Can you not use your runtime?
At the beginning substrate-test-runtime was some very stripped down runtime. Nowadays it already contains a lot more code, probably too much.
Maybe a better question, why do you need a full runtime for your RPC? Can you not just pass some mock object into your RPC to test it?

It is probably me need to read more code to understand how it suppose to work... But some example will always be helpful.
This is the code I want to write test for https://github.com/AcalaNetwork/Acala/blob/master/modules/evm/rpc/src/lib.rs#L95-L101
We have defined our runtime API which is used by our RPC code.
Just wasn't sure the best way to create mocks that will get everything compile.

There is https://docs.rs/sp-api/3.0.0/sp_api/macro.mock_impl_runtime_apis.html that will probably be helpful for you :)

I tried to implement RPC test like this:
https://github.com/paritytech/substrate/blob/a8c2bc66ea8667a1dcbafa13ce184b020bd6f84b/client/consensus/babe/rpc/src/lib.rs#L292-L301
The TestClientBuilder of substrate-test-runtime-client is required to generate the client, but the client does not implement EVMApi.
Is there any way to implement the RPC unit test?

I don't know what your rpc takes as arguments etc.
And as I already said above, you should not use the substrate-test-runtime-client outside of substrate.


Bump OCP to v4.8

Signed-off-by: Manuel Hutter<EMAIL_ADDRESS>
Checklist

[x] Keep pull requests small so they can be easily reviewed.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog
[x] Link this PR to related issues.

Reading htrough the release nodes, some things stick out that might be potential issues. Before updating the default, I would like to check those points first. Internal ticket reference: https://ticket.vshn.net/browse/OCP-206.

No, feel free to merge


Upgrade to Scading 0.13.1 and Scala 2.11.6

Next sprint (27/4) we are planning on upgrading to Scading 0.13.1 and Scala 2.11.6.

The new version of scalding is has a number of bug fixes and improvements
Scala 2.11 has improvements for Macros amongst other things.

The impact this will have as of yet is unknown. However, macro code might be heavily affected.
@anandsiva @dg2 @jamindaw @quintona please make the teams in your portfolios aware of this.
Status:

[x] uniform
[x] omnia-test
[x] omnitool
[x] permafrost
[x] humbug
[x] cascading-hive
[x] thermometer
[x] parlour
[x] ebenezer
[x] edge
[ ] answer
[ ] maestro

Ran into a problem with the lack of scrooge-bijection for Scala 2.11. I have raised an issue against bijection to figure out where it is since there are other Scala 2.11 bijection projects.

Unless we use scrooge-bijection a lot in some code I haven't seen, it could be replaced with a handful of 3-4 line utility methods.

Added status update to issue description.
I am currently stuck on Thermometer, something really weird is happening where the tests seem to run but no test information is displayed and the tests are marked as failure without any information.
@anandsiva at this stage I don't think we will be ready for any loads to be tested this sprint.

Currently struggling with the different source generators from scrooge, humbug and thrift interacting weirdly as part of the upgrade in enbenezer.

Now stuck untangeling Parlour. What ever was done with regards to how parlour does configuration does not work for Scala 2.11.

Ebenezer is now done as well. It was going slow because we had to run the tests differently and travis kept running out of memory.

There is a 2.11 branch of Maestro that is being tested right now.

All done.


ambient: fix a regression in pod update handler

Please provide a description of this PR:
This is a regression in https://github.com/istio/istio/pull/47843
IsPodInIpset does not work when eBPF redirection is enabled

@dddddai: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
5c5d59721571f14b2f10ac64838b00dcfd666f53
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc @bleggett


Fix differences in effective date and or time

There is a difference in effective dates. This could cause implications – users submitting in BCROS, not COLIN, there is a difference in time (2 minutes) in ledger vs actual filing time (please let me know if you need more info) - Elizabeth has confirmed that there could be possible implications of this and could result in additional phone calls to Joshua’s unit.

This one, issue with date and time – filed in BCROS on Aug 20 at 8:57pm but reflecting in COLIN as August 21 at 8:00am. It was also filed before the anniversary date.

@nsmenon8890 - fyi

Each filing gets 3 dates:

Date and time when AR is submitted (before paying)  ----> Filing Date
Date and time when AR is paid (successful payment)  ----> Payment Completion Date
Date and time when AR is processed (AR completion) ----> Completion Date

It seems the issue here was that there was a delay between Payment Completion Date and Completion Date  due to the time needed to sync with Colin.
I changed it so that these two dates and times are identical.  I kept the filing date as is as that doesn't seem to be a problem? This can easily be changed so that all three are the same though. I tested it with this business and the times are now in sync.

Before these changes the data in the database looked like this (notice the time delay between completion date and payment_completion_date:

@nsmenon8890 - FYI

This one is quite tricky.
I updated the data in Google Cloud so that all of the data fields are consistent when an AR is filed. Here is an example below.

Even with the data being consistent on the Business AR side of things, Colin still displays the effective date to be something different - usually a couple of minutes in the future, but sometimes as much as an entire day.

It seems as if something is happening to the effective date in the post request that gets sent to Lear  which is located here.
https://github.com/bcgov/lear/blob/f045c27b19792ea63e9596cf2313148f6432063f/colin-api/src/colin_api/resources/filing.py#L90-L93
Further work needs to be done on this issue, specifically finding out why Lear is changing the effective date.

We believe the issue is in COLIN_API in lear:

It seems like the problem with this one is how Colin is displaying the effective date and date fields.
In Business AR, a filing is sent to the Colin API that looks like this;
2024-09-13 11:12:00,479 - process_paid_filings.job - INFO in job:job.py:99 - send_filing: Filing details to be sent:
2024-09-13 11:12:00,480 - process_paid_filings.job - INFO in job:job.py:100 - send_filing: {
    "filing": {
        "annualReport": {
            "annualGeneralMeetingDate": "2024-09-12",
            "annualReportDate": "2010-09-26",
            "unanimousResolutionDate": "",
            "votedForNoAGM": false
        },
        "business": {
            "adminFreeze": "False",
            "corpState": "ACT",
            "corpStateClass": "ACT",
            "email": "",
            "foundingDate": "2008-09-26T05:07:54-00:00",
            "goodStanding": false,
            "hasFutureEffectiveFilings": false,
            "homeCompanyName": "",
            "homeJurisdictionNumber": "",
            "homeRecognitionDate": "",
            "identifier": "0835905",
            "jurisdiction": "BC",
            "lastAgmDate": "2024-09-11",
            "lastArDate": "2009-09-26",
            "lastLedgerTimestamp": "2024-09-12T19:26:32-00:00",
            "legalName": "LAMA PASS FUEL CO. LTD.",
            "legalType": "BC",
            "nextARYear": 2010,
            "status": "Active",
            "taxId": "123"
        },
        "documents": [
            {
                "name": "Receipt",
                "url": "https://business-ar-api-dev-dy4loprnwa-nn.a.run.app/v1/business/0835905/filings/142/documents/receipt"
            }
        ],
        "header": {
            "certifiedBy": "",
            "certifiedByDisplayName": "",
            "colinIds": [],
            "completionDate": "",
            "date": "2024-09-13T18:10:36.984605+00:00",
            "filingDateTime": "2024-09-13T18:10:36.984605+00:00",
            "filingYear": 2010,
            "id": 142,
            "name": "annualReport",
            "paymentAccount": "3495",
            "paymentStatus": "COMPLETED",
            "paymentToken": 40656,
            "status": "PAID",
            "submitter": "",
            "learEffectiveDate": "2024-09-13T18:10:36.984605+00:00",
            "source": "BAR"
        }
    }
}

In the above example, both date and affective date are the exact same. The result for this filing in Colin ends up looking like this:

Date and time filed is 5 minutes in the future. After looking and playing around with the endpoint in Lear it seems this date and time filed field is coming from elsewhere - changing it has no effect on what is displayed in Colin. This date and time filed field seems to be coming from the event table which has the date defined as SYSDATE. Therefore when a record is added the date and time filed is always different because it is using the time of the transaction which wiill always be different than the time stamp in Business AR before sending the data over.

In the above example, the date and time filed was at 11:15. This is coming from CDEV from the field event_timestamp.

@ozamani9gh what do you think?

@Maribeth-Wilson This needs to go to home team to fix.  I can't tag Michael Kagis.  can you send this to home team, and CC Thayne.
"Date and time filed is 5 minutes in the future. After looking and playing around with the endpoint in Lear it seems this date and time filed field is coming from elsewhere - changing it has no effect on what is displayed in Colin. This date and time filed field seems to be coming from the event table which has the date defined as SYSDATE. Therefore when a record is added the date and time filed is always different because it is using the time of the transaction which wiill always be different than the time stamp in Business AR before sending the data over."


Bottom nav bar should depend on flag

Fixing an issue where settings showNavigation to false didn't do anything.
Similar bug on forked repo https://github.com/CrossPT/flutter_plugin_pdf_viewer/issues/57

this would be great ;) I struggled to find the problem untill i saw the condition in the code, and this issue.

@dimitri-bret There is a workaround you can use, until this is merged. pass navigationBuilder: null in the widget call.

Added the requested change @lohanidamodar


Bug Fix and Error Page

Show Error 404 page if url not available
Fix the dark mode bug

I will take up this issue assign it to meeeeeee.