Remove IDisposable from ICacheEntry

https://github.com/aspnet/Caching/blob/65228be1d9aba6f2c360553868010f18706ee302/src/Microsoft.Extensions.Caching.Abstractions/ICacheEntry.cs#L13
IDisposable recommends a using pattern as follows, where Dispose commits the entry to the cache:
using (var entry = MemoryCache.CreateEntry(cacheKey))
{
  // The result is processed inside an entry
  // such that the tokens are inherited.
  entry.Value = CreateValue(params);
  entry.SetOptions(options);
}

However, if CreateValue throws then the entry is still committed to the cache as null. Thus the using pattern should not be used with ICacheEntry, and we should remove IDisposable to prevent it. Replace it with a Commit method.
https://github.com/aspnet/Mvc/issues/6522

Hi @muratg is there any plans to proceed with the breaking change?

@anurse

As I understand it, to "fix" this we'd be introducing a breaking change. It's too late to do that kind of thing in 3.0, so there is no change planned here in 3.0.

Sorry @anurse, I wasn't completely explicit.
My question was for a major after v3 because, since this issue is closed, was it added to a backlog for contracts revision?

No, it was closed back in January. If you'd like to request this change be reconsidered, we can do that. What's the scenario that's broken for you right now?

Nothing is currently broken per se, since you can work around even in older versions (just create the value before requesting the cache entry), but is about consistency when implementing the disposable pattern, like @Tratcher said, while also being explicit that a new entry have been added to the cache.
The thing is, you request an entry from the cache and not create one by yourself, so it is not clear that disposing the entry will also add it to the cache (as far as what concerns the developer, it is implementing IDisposable to release resources) and if an exception is thrown inside the using it will still save the entry in whatever state it may be.
To ensure some kind of atomic behavior during the entry setup, adding a commit method would made the approach more explicit and less prone to misinterpretations. The simple fact comments to this behavior must be added to the extensions methods or it may be forgotten are a clear indication of an implementation problem.
The IDistributedCache doesn't have this problem because it only exposes contracts enabling explicit atomic behavior (arguments are the key, value and entry options).

The thing is, you request an entry from the cache and not create one by yourself, so it is not clear that disposing the entry will also add it to the cache

I do agree this feels odd and counter-intuitive. I don't have background on why this pattern was used, but it feels confusing to me. So I'm with you here :).
The main thing we think about when considering breaking changes (as with any changes) is if the benefits outweigh the costs. The cost for a breaking change starts out fairly high since users are forced to react if they upgrade. Generally we do this by marking an API as obsolete in one version (so that users get a warning, but the API still works) and then removing it in the next version. It seems like that would be a reasonable thing to consider in the 5.0 time frame (which is the next release open for this kind of change).
I'll open this up and put it on the backlog for consideration as we do 5.0 planning.

Perhaps the shorter-term solution is to implement an alternate way to add to the cache, and go ahead and mark the current versions of ICacheEntry and .CreateEntry() as deprecated?


Incertae sedis AGAIN -- Edit taxon name

In spite of the issue with incertae sedis is marked as resolved, I still have problems with it:
(1) it should be inCCCCCCCCCCertae with -c not -s.
Look here:
https://opiliones.fandom.com/wiki/Incertae_sedis
(2) I have a genus Gjellerupia which doesn't belong to any family in the infraorder Grassatores. Therefore it is a incertae sedis taxon.

But whenever I set the incertae sedis button here:

I get the msg:

What is happening?

Just a comment: I manage to set a genus in a superfamily as incertae sedis, but NOT in an infraorder.

I will check the model. Please check the task, it could be a label issue:

That label is fine, it maches exactly with https://opiliones.fandom.com/wiki/Incertae_sedis.
Where exactly do we see insertae?  Screenshot?

Ah, I see..

Dmitry: anyway the big problem is not the typo (because it is only in a hidden part), but the inability to use incertae sedis with infraorder, although it works fine for family.


Add FlowType support

Hi! Are there any plans to add FlowType declarations for this lib? :trollface:

Hi @ryyppy! it wasn't on the roadmap but, now that you said I'll actually love to add FlowType support. Can you help me with that or send me any good article that explains how to add it?

Done in https://github.com/zzarcon/gh-emoji/pull/14


Fix order of glShader arguments in README

Error compiling shader unless you pass the vertex shader before the fragment shader in the arguments list.

Ah good catch, thanks very much @rippedspine! :)


Make file filters more intuitive of bundle download (#207)

Add two new options:

--no-data and
--no-metadata

and rename --metadata-files and --data-files to
--metadata-filter and --data-filter respectively.

Codecov Report

Merging #360 into master will increase coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
+ Coverage   84.98%   85.26%   +0.27%
==========================================
  Files          39       37       -2
  Lines        1785     1764      -21
==========================================
- Hits         1517     1504      -13
+ Misses        268      260       -8

Impacted Files
Coverage Δ

hca/dss/__init__.py
92.73% <100%> (+2.56%)
:arrow_up:

hca/version.py
100% <100%> (ø)
:arrow_up:

hca/cli.py
58.77% <0%> (-0.72%)
:arrow_down:

hca/upload/lib/client_side_checksum_handler.py
91.17% <0%> (-0.5%)
:arrow_down:

hca/util/__init__.py
85.98% <0%> (-0.38%)
:arrow_down:

hca/query/cli.py

hca/query/__init__.py

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 8805027...a7db10f. Read the comment docs.

Also addresses #338.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 79 ms

Resolved: ClamAV performance has improved in 50de0bb.


Revert rust support

We've changed our strategy for enabling IDE support for Bazel Rust projects, and the code changes we'd originally made to bazel-intellij to support Rust are no longer needed.

PR Review Checklist
Do not edit the content of this comment. The PR reviewer should simply update this comment by ticking each review item below, as they get completed.

Code

[ ] Packages, classes, and methods have a single domain of responsibility.
[ ] Packages, classes, and methods are grouped into cohesive and consistent domain model.
[ ] The code is canonical and the minimum required to achieve the goal.
[ ] Modules, libraries, and APIs are easy to use, robust (foolproof and not errorprone), and tested.
[ ] Logic and naming has clear narrative that communicates the accurate intent and responsibility of each module (e.g. method, class, etc.).
[ ] The code is algorithmically efficient and scalable for the whole application.

Architecture

[ ] Any required refactoring is completed, and the architecture does not introduce technical debt incidentally.
[ ] Any required build and release automations are updated and/or implemented.
[ ] Any new components follows a consistent style with respect to the pre-existing codebase.
[ ] The architecture intuitively reflects the application domain, and is easy to understand.
[ ] The architecture has a well-defined hierarchy of encapsulated components.
[ ] The architecture is extensible and scalable.

Unnecessary as we've just hard-reset the main branch.


Looking for information on how to use molecule with vagrant

I've been a long time user and fan of Molecule 1 and I am trying to make Molecule 2 use the vagrant driver rather than a container to test my role. I've looked for examples everywhere on how to do this but I can only find Molecule 1.25 examples.
I've followed the guide and created a role using:
molecule init role -r my_role

Then I changed the driver in molecule.yml to use vagrant and the platform to use a box rather than an image.
molecule test

It calls the create.yml playbook which basically ignores anything to do with vagrant and just uses containers (which fails).
Firstly is using vagrant with molecule 2 currently supported? Am I doing something wrong? Am I missing something in the documentation?
Secondly, I've been unable to find an example of an open source project that implements vagrant with molecule2. If you know of a project could you point me in it's direction?

You need to initialize the role with the proper driver.
molecule init role -r my_role -d vagrant

Thank you!


Support of SLURM

Support of SLURM 20.11.7
New CycleCloud template added for SLURM using the 2.5.0 cyclecloud-slurm project
Accounting managed mySQL database provisionned, this is optional. Turned on by default, false if missing

close #24
close #656
close #649

We still need to rename the queues to avoid underscores in the hostnames

We still need to rename the queues to avoid underscores in the hostnames

I will do it in a separate branch and PR in order to track it as it will change the documentation as well as tutorials

note that Slurm project 2.5.1 was pushed to fix a name resolution issue in 2.5.0

Yes, but the project cyclecloud/slurm:2.5.1 has not been pushed yet in the public cycle lockers
From: Jerry Morey @.>
Sent: mercredi 8 décembre 2021 14:01
To: Azure/az-hop @.>
Cc: Xavier Pillons @.>; State change @.>
Subject: Re: [Azure/az-hop] Support of SLURM (PR #658)
note that Slurm project 2.5.1 was pushed to fix a name resolution issue in 2.5.0

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Faz-hop%2Fpull%2F658%23issuecomment-988794968&data=04|01|Xavier.Pillons%40microsoft.com|74e391163c004ab2726108d9ba4ad224|72f988bf86f141af91ab2d7cd011db47|1|0|637745652838359776|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=%2Fbo5VQC%2F5ZB6Wz4hKJgrA6UiAtTgdMUJo3fCIP8WCx4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABYCJISD44OHNIR5RW6BJ6LUP5JJXANCNFSM5JOG4ZPA&data=04|01|Xavier.Pillons%40microsoft.com|74e391163c004ab2726108d9ba4ad224|72f988bf86f141af91ab2d7cd011db47|1|0|637745652838359776|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=QfEmVpnL77cohnEW1%2Fdi54DBlq5mRViEVR6lGMXSYkU%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Xavier.Pillons%40microsoft.com|74e391163c004ab2726108d9ba4ad224|72f988bf86f141af91ab2d7cd011db47|1|0|637745652838359776|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=XabnWG4D%2FRUAaDyEfW4vqMaMjq2Clvs%2FjRdW2meel4E%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Xavier.Pillons%40microsoft.com|74e391163c004ab2726108d9ba4ad224|72f988bf86f141af91ab2d7cd011db47|1|0|637745652838359776|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=xTV9HgtIbliFaQbQMIazf7er%2B0KfY0kzpdVeorVoKS4%3D&reserved=0.


Add support for futures-rs

Using this library in a server environment working with futures-rs would be amazing.

What do you mean by working with futures-rs for protobuf?
I'm currently working on gRPC implementation in rust: https://github.com/stepancheg/grpc-rust.

Currently Coded{Input,Output}Stream are based on blocking IO. Therefore I need to read data asynchronously into a buffer before converting it into the according message. It would be nice to be able to use Coded{Input,Output}Stream directly with futures without needing to manually reading data into a buffer.

That would make protobuf parser and serializer enormously complex and hard to implement. Neither C++ nor Java protobuf implementation do it. So I think rust-protobuf shouldn't try to do it either. Until at least rust implements async/wait.
If your messages are large, you can split them into a sequence of small messages. gRPC protocol supports streaming of messages, and it is implemented in grpc-rust.


Make parseNumbers and parseBooleans options work with arrayFormat

fix #199

Awesome, this solves an issue I just came across. Can you release a new version with this soon?

Done


typeof expressions should return an equivalent of a guard

Search Terms
typeof guard array filter

Suggestion
Currently, typeof expressions return a boolean as expected. However, they are also special in that they narrow the types of their operands. In a similar vein, we can use type guards as the return value of functions to narrow(or assert) the types of their parameters:
declare const broad: string | number | object;
if(typeof broad === 'number') {
    // broad is now a number, not a string nor an object
}

declare function isNumber(param: any): param is number;

if(isNumber(broad) {
  // broad is now a number, not a string nor an object
}

This is all fine, but suppose you write a very specific function:
// Current return type: boolean
// Wish it was `param is number`
const isNumber = (param: any) => typeof param === 'number'

if (isNumber(broad)) {
   // param is possibly a string or object?!
}

This is IMO a special case that should be handled; particularly, when a typeof expression(or any expression that narrows a parameter's type) is the return value of a function, TS should infer that the function is a guard.
Notice that, unlike the discussion in #6015, this issue is about the inferred type of a function which has a return value of a narrowing-expression. We could go further:
const isObject = (o: unknown) => typeof o === 'object';
// type for isObject == (o:unknown) => o is object

const isElement = (o: object) => o instanceof Element;
// type for isElement == (o:object) => o is Element

const unknownIsElement = (o: unknown) => isObject(o) && isElement(o);
// type for unknownIsElement == (o:unknown) => o is Element

const numberIsElement = (o:unknown) => isNumber(o) && isElement(o); // error: o is number so isElement can't be called

(by the way, I searched but couldn't find a similar one. If this is a duplicate I apologize!)

Use Cases
The most immediate use case is for array filter:
// error: (string|object)[] not assignable to string[]
const stringElements: string[] = elements.filter(e => typeof e === 'string');

// Works, but repetitive
const stringElements: string[] = elements.filter((e): e is string => typeof e === 'string');

I'm sure this type of change would make guards far more powerful as well, since we could create functions combine JS-checks(typeof, instanceof) or even TS-checks(like discriminant variable checking) without having to explicitly write the guard expression(which, IMO is as good as a type assertion at the moment).
Examples
(see above)
Checklist
My suggestion meets these guidelines:

[ ] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

This is possibly a breaking change, but IMO a good one as it would highlight actual programming errors for code that breaks

I also think this would be an extremely useful convention, but would like to take it a step further and say that type guards should be extended to be able to create typeof-like guards, where one function can definitively state the type. For example:
function(obj: any): obj is TypeGuardAdv<{
    type1: Type1,
    type2: Type2,
    type3: Type3,
}> {
    if(obj.prop1) return "type1";
    if(obj.prop2) return "type2";
    if(obj.prop3) return "type3";
}

I just used a generic TypeGuardAdv to demonstrate the concept, as I'm not entirely sure what the resulting syntax would be.
I'm not sure if this next suggestion is in scope for this issue or not, but it would also be nice to be able to store these type annotations to variables. Consider the following example:
interface BaseType {
    name: string;
}
interface MyType extends BaseType {
    myProp: string;
}
declare const isMyType: (obj: any) => obj is MyType;

const obj = { name: string, myProp: string };
const objIsMyType = isMyType(obj);

if(objIsMyType) {
    doSomethingExtraWith(obj.myProp);
}
doSomethingWith(obj);
if(objIsMyType) {
    extraFinalization(obj.myProp);
}

If a type guard function requires extensive checks, then it would be inefficient to run them more than once if obj is constant.


[HOLD] Implement Android conversation notifications

In Android 11, conversation notifications were added that offer a richer notification experience for messaging.

Conversation notifications are placed highest in the notification shade and prominently display avatars and names with the app's logo in the corner. Individual notifications can be marked as a priority allowing them to cut through Do Not Disturb settings. This is very similar to communication notifications which we implemented for iOS here.

Android docs

FYI this is a copy of the old internal issue here: https://github.com/Expensify/Expensify/issues/258528

From the docs, we need to implement these to get conversation notifications

[x] The notification uses MessagingStyle.
[ ] (Only if the app targets Android 11 or higher) The notification is associated with a valid long-lived dynamic or cached sharing shortcut. The notification can set this association by calling setShortcutId() or setShortcutInfo(). If the app targets Android 10 or lower, the notification doesn't have to be associated with a shortcut, as discussed in the fallback options section.

Which means we should just need to implement sharing shortcuts. I'm going to look for a contributor to work on this.
@robertKozik you worked on our last issue for clearing notifications on Android. Would you be interested in working on this as well?

Hey @arosiclair! I can take care of this feature, because I work currently on the HybridApp, where we will refactor notifications in order to make sure the work for both OldDot, and NewDot. However, could we slightly wait with implementation of this feature? I just want to make sure that the notifications work fine after the refactor, and then, as a follow-up, I will add the conversation notifications to Android.

Makes sense I'll assign you and put this on hold. Keep us updated 🙏

Still held @staszekscp?

Still held I believe. cc @staszekscp for any updates

Hey! Sorry, I missed the previous comment - the PR with the refactor is open, I just have to find one fix for a bug that I noticed. After merging I will work on this issue - I have already done some small testing, and I think I know what needs to be done 😄

Still held on https://github.com/Expensify/Mobile-Expensify/pull/12610

Remains held

Stil held on https://github.com/Expensify/Mobile-Expensify/pull/12610

Asked for an update on the PR. Still held for now.

Still held.

Still held, but the latest comment in that PR indicates we're getting closer:

Seems like we need to merge main, but probably also fix New Arch + iOS still, so this is close to being ready for a merge

Still held.

Haven't been able to make progress in the last couple months. Dropping this to monthly.

Looks like https://github.com/Expensify/Mobile-Expensify/pull/12610 was merged! @staszekscp what's the plan for this issue?

Bump @staszekscp

Sorry, I haven't seen the last bump! Actually I could try to open a PR tomorrow with the change, if nothing critical comes up for the HybridApp release. If something happens, I'll pass the task to someone in the SWM team for next week in order to finish it. Again, sorry for the delay, I was swamped by the HybridApp issues 😢

No worries this is definitely lower priority compared to core functionality for the HybridApp

I had a bit of time to work on that, and the "styling" part is done - the notification appears as the conversation type, and works just fine 😄 The thing that I need to work on, is to implement the conversation shortcut. It's a feature to open a conversation, after selecting it from a menu that appears after long press on the app icon. The shortcut is there, but it doesn't open any chat yet
https://github.com/user-attachments/assets/67c6e8e6-7161-49ef-9ae2-f74ea186fa9a

Nice!

It's a feature to open a conversation, after selecting it from a menu that appears after long press on the app icon. The shortcut is there, but it doesn't open any chat yet

Lemme know if you need any help figuring this part out

Thanks! Actually I've just gotten it working 😄 I still need to do a major cleanup, but on Monday I'll open the PR! (There is a bank holiday in Poland, and we'll be OOO on Thursday and Friday!
https://github.com/user-attachments/assets/a7700951-e056-42ab-80d8-a57258517d22

I am heading out of office until September 21st, so assigning a buddy to watch over this in my absence.
Current status: PR being worked on in https://github.com/Expensify/App/pull/47626

Awaiting payment

I'm back from OOO, and can take this one back over - thanks for keeping an eye on it @garrettmknight

Regression steps proposed in https://github.com/Expensify/Expensify/issues/430593
Payment complete. Closing this one out!


Sequential.set_weights did not work with BN

amazing this was not found before.
Looks like there is no test unit for set_weights.

This is now fixed.


Duplicated variable names in output of multiple choices

It seems that Trealla's output for multiple choices, e.g.:
Trealla v0.1alpha, 64-bits (int=128-bits), Dec  5 2016
?- member(X, [1,2,3]).
X: 1 (abort=a,trace=t,next=;):
X: 2, X: 2,  (abort=a,trace=t,next=;):
X: 3, X: 3,  (abort=a,trace=t,next=;):

(0.000 s) no
?-

Variable names in lines of X: 2 and X: 3 seem to be duplicated.

See Issue #37.


Update required ovirt_metrics version

Purpose or Intent
The ovirt_metrics required version was updated to v1.3.0
Signed-off-by: Shirly Radco<EMAIL_ADDRESS>

@miq_bot add_label darga/yes

:+1: LGTM, the test failures are already corrected upstream.  Please close / reopen or re run the javascript suite.

@sradco FYI, the bot is named with a dash (@miq-bot) not an underscore, which is why your command didn't get recognized.

Darga Backport details:
$ git log
commit c948cbfb9d02e31248548906b47853ef4f922483
Author: Jason Frey<EMAIL_ADDRESS>Date:   Wed Aug 31 17:25:04 2016 -0400

    Merge pull request #10893 from sradco/update_ovirt_metrics_version

    Update required ovirt_metrics version
    (cherry picked from commit b8351d90ca2c828053383d28709c74d293ebcee7)

https://bugzilla.redhat.com/show_bug.cgi?id=1375340


mails filter by labels

as in Gmail there are many labels e.g. social, primary, promotions etc. ...
How to filter mails by labels .
$f->query()->all()->paginate(10, $request->page);

Hi @zu007 ,
please head over to the main repository https://github.com/Webklex/php-imap.
In general: if these flags are accessible depends on the mail provider. I'm not aware that google supports the search for labels. However please prove me wrong :)
Best regards,


[Managed LU] Design for serialization & deserialization for dialog.lu file

Lu parser support for section (CRUD)

as new bf lu (ludown) is implemented . so close it.


Script to update code displays

Requested Feature
Given the recent change to tx.fhir.org that a code display not matching is now an error instead of a warning (ref: https://chat.fhir.org/#narrow/stream/179294-committers.2Fannounce/topic/Coding.2Edisplay.20change/near/357558577 ), we should try to better keep up with changes to terminology, so that someone validating a Synthea patient doesn't get inundated with errors. Right now this is a painfully manual process of running some sample patients through the validator and seeing what errors we get. We should create a script or some other mechanism that will:

scan all codes used in the modules
check the display text against the preferred text for the code system
either update the module directly if it doesn't match, or provide a list of necessary changes that a user can apply

Implemented in #1407


Added the "alt" Property for Images

🛠 Fixes Issue
Issue - #1455
👨💻 Changes proposed
Added the "alt" attribute to all images on the website.
In this Pull Request, I have addressed the issue #1455 by adding the "alt" attribute to all images present on the website. The "alt" attribute provides alternative text for images, making the website more accessible for users with visual impairments.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Hello @rohanbabbar04 , @SauravMukherjee44 and team,
I'm excited to contribute to the project and look forward to your review of this Pull Request. Let me know if any further adjustments are needed.
Thank you for considering these enhancements to improve website accessibility!

Some images seem to be broken do correct it :arrow_up:

Hello @rohanbabbar04 ,
I hope you're doing well. I wanted to provide an update on my Pull Request. I have addressed the issues raised during the review and fixed the broken images as suggested.
Thank you for your valuable feedback, and I appreciate your time in reviewing my contributions. I'm looking forward to your re-evaluation of the updated Pull Request.

Hello @rohanbabbar04,
I hope you're doing well. I have made the necessary changes to address the broken images in the project. Could you please review my updated Pull Request?

Hello @rohanbabbar04 ,
I have made the necessary changes to address the broken images in the project. Could you please review my updated Pull Request?

Still I don't think its done

Wheras it should be

Hello @rohanbabbar04 ,
I hope you're doing well. I have addressed the issues with broken images in my Pull Request. Could you kindly review my changes and consider merging the PR if everything looks good?
I've tested the changes, to ensure that all images are now loading correctly. Your feedback is greatly appreciated.
Thank you for your time and support!

Hello @rohanbabbar04 ,
I sincerely apologize for the inconvenience caused by the broken images in my previous Pull Request. I have revisited the issue, and I believe I have now correctly fixed the images.
Could you kindly review my updated changes and consider merging the PR if everything looks good?
Thank you for your understanding, and I appreciate your time and consideration.

Hello @rohanbabbar04  ,
I have made the necessary corrections to fix the broken images and cleaned up the code by removing the commented sections, as per your feedback.
Could you please review my updated Pull Request and consider merging the PR if everything looks good?
Thank you for your understanding, and I appreciate your time and consideration.

Looks great....
Thanks

I hope all is well. I have restored the necessary code snippet and made the related changes accordingly. The code is now complete and ready for review.
Thank you for your continued support and guidance throughout this process.

Thanks it looks good....


Issue with k8s.io/docs/reference/command-line-tools-reference/kube-proxy/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

PLZ point out the problem

/close


[Refactoring] DatabaseMetaData databaseMetaData is passed in almost every method

DatabaseMetaData databaseMetaData is passed in every method of
org\sqlcomponents\core\crawler\Crawler.java
Better make it object vartiable to make it avaible to all by default.

We can not store as instance variable as this is a resource. has to be closed.

    _`DatabaseMetaData databaseMetaData= lConnection.getMetaData();`_

databaseMetaData is coming from connection as above, yes, connection  is a resource that needs to be closed properly.
Most Database objects depend on databaseMetaData

not sure if we is a good idea to pass this   databaseMetaData to construct database itself and if any needs databaseMetaData  then they can use retrieve from database reference.
That way at least we don't have to pass both database and databaseMetaData as method parameters.


I see 5.1.1 on NPM, but no tag in the repo?

I was trying to investigate some code in node_modules, but it differs from what I see here in the repo. I see that I have skatejs 5.1.1 installed, but I don't see that tag here in the repo. Where's it at?

I don't see it in the GitHub releases either.

Ah, this is in the v6 branch too! I spent too much time reading the old code, loaded up node_modules version to find it was totally different. 😆

Yeah, I've done a few releases from there. I'm still getting used to the monorepo.


Fix: 1388 Encode Markup in TX ID Field

Fixed 1388
Added function to convert characters to escaped versions to reveal attempts at users including markup in the transaction id fulfillment field.

Ready for review/merge @rmisio @jjeffryes.

Looks good to me.

This is pending a couple of changes I ping J So Hot directly with.

@rmisio fixed.

ok @jjeffryes - if latest changes are ok with you, this is ready to merge.


API Endpoint to get debug report

endpoint calls ssh debug report and outputs them for the GUI to display
more details on backend API
needs token protected endpoint

Related issues:

https://github.com/fusion44/blitz_api/issues/160
https://github.com/cstenglein/raspiblitz-web/issues/49

@rootzoll Take care about this: https://github.com/fusion44/blitz_api/issues/160

OK @fusion44 & @cstenglein fix waiting to be merged.
https://github.com/fusion44/blitz_api/pull/175

Done, can be tested

Tested fine with v1.9.0rc4 - closing for release.


Type definitions and tests for constant-case

https://github.com/blakeembrey/constant-case
https://www.npmjs.org/package/constant-case

@stpettersens Thanks mate!


Solution for Prevention

Hi @davidokeyode ,
Any solution for prevent the demo in Microsoft Defender for cloud?
For Example, Prevent Azure Password Spray

@cheahengsoon Sorry I missed this. Yeah detection exists in MDC. There are ways that an attacker could look to work around by doing things like varying the source IP and implementing backoff techniques.


I cannot change to an env

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
When I try amplify status, I see that I use dev, when I try amplify env list I see that I have an qa env as well, when I try amplify checkout qa, I get the following error: Please pass in a valid environment name. Run amplify env list to get a list of valid environments.
When I try amplify env get --name qa, I do get the info of the qa env.
To Reproduce
Steps to reproduce the behavior:

amplify env add (follow the input steps and let's call it dev)
amplify status (this shows we use dev)
amplify env list (this shows dev)
amplify env add (follow the input steps and let's call it qa)
amplify status (this shows qa)
amplify env list (this shows dev and qa but qa with an * so we use qa)
amplify env checkout qa (this shows: Please pass in a valid environment name. Run amplify env list to get a list of valid environments.)

Expected behavior
Switch th the qa env with the amplify env checkout qa command.
Additional context
I use Cloud9

Which version of the CLI are you using?

@ben-elsen We've reproduced the issue. To unblock, please use the following flow:
f45c89966b0d:mybetatest kaustavg$ amplify env add
Note: It is recommended to run this command from the root of your app directory
? Do you want to use an existing environment? Yes
? Choose the environment you would like to use: (Use arrow keys)
dev
❯ qa

Thanks!


fix: Can not set frequency_penalty and presence_penalty to -2 and 2

Fix: You can set frequency_penalty and presence_penalty to -2 and 2 now.
Fix: The workflow will not continue to show a Flet window if an error occurs.
Fix: auto focus on streamed result not working
Fix: former fix guarding assistant_output should use sth other than an empty string.
Feature: cmd+w to close Flet window

Thanks for this nice PR

No problem!

Do you want to add more improvements or do you want that we release it?

I was gonna say that these minor tweaks don't deserve a release, but I recalled that a user requested the system prompt feature, #92, implemented in #98 and has not yet been released either. I think that deserves a small release 😁

I just invited you as a collaborator on this project. You already worked so much on it.

Thank you!

Can I ask you to merge develop into main?

I can't be sure of anything but I will certainly take a look!


Benchmark Infra

Added a benchmark executable.
Initial results:
Transfer Updates Operations: Duration: 60000 Interval: 1000μs Queries: 59758 TPS: 995.967 Average: 14925μs p50: 14360μs p99: 25667μs

High throughput:
Transfer Updates Operations: Duration: 30000 Interval: 100μs Queries: 72062 TPS: 2402.067 Average: 53282μs p50: 51102μs p99: 93265μs

But Audit procedure could yield inconsistent results due to read committed isolation.

Looks good to me.


Chapter files whose name contains a colon don't work with chapter-make-read.lua

Hi, I've noticed that if a filename contains colons then it doesn't work work with chapter-make-read.lua. E.g. I have Megadeth: Tiny Desk Concert.mkv and Megadeth: Tiny Desk Concert.chp. chapter-make-read.lua ignores the chapters until I rename both files replacing the colon with something else. Is it something that can be fixed in the script?

On Windows, special characters such as \/:%? are illegal characters that cannot be used in filenames. This was specially handled in the previous code. Unexpectedly broken usage on Unix platforms, fixed.

Great, thanks!


Unwanted Bottomsheet padding on android12 +

there is unwanted padding for the bottom padding that occurs only if you are on android 12 and above!
is this a bug or am i doing something wrong?

This seems to be a bug. Can you run the sample app on your device and open the text field bottom sheet to check if you also get the same issue?
If this occurs, can you provide more information, such as the device, the system, and the keyboard app?
If not, can you provide the information above and extra information, such as your composable parameters and the reproduction steps?
Extract it to get the apk:
sample-debug.zip

sample app works fine on my device.
here is a snippet of my composable
BottomSheet(
state = state,
modifier = modifier.imePadding(),  //might be causing the issue ?
shape = configs.shape,
backgroundColor = configs.background,
skipPeeked = configs.skippedPeek,
peekHeight = configs.peekHeight,
dragHandle = configs.handle,
behaviors = configs.behaviors
) {
content(this)
}

| modifier = modifier.imePadding(), //might be causing the issue ?
Yes, this is expected behavior, when the whole sheet needs to show above the keyboard, you can combine the SOFT_INPUT_ADJUST_RESIZE with imePadding()
BottomSheet(
    state = state,
    modifier = modifier.imePadding(),
    behaviors = DialogSheetBehaviors(
        dialogWindowSoftInputMode = WindowManager.LayoutParams.SOFT_INPUT_ADJUST_RESIZE
    ),
) {
    TextFieldSheetContent(state = state)
}

Will add this to the README

Yes, this is expected behavior, when the whole sheet needs to show above the keyboard, you can combine the SOFT_INPUT_ADJUST_RESIZE with imePadding()

showAboveKeyboard was added to simplify this.
Check #94

Sorry for being late, The issue has been resolved using the suggested method.


[k8s-extension] release new cli version-1.4.3

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

k8s-extension


npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.

This package was working before. Now when I removed my node_modules and package-lock.json, I get the following error:
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@remix-pwa%2fpush - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR!      /Users/vahid/.npm/_logs/2024-04-12T13_36_18_124Z-debug-0.log

These are my versions:
"dependencies": {
  "@remix-pwa/cache": "^2.0.12",
  "@remix-pwa/client": "^2.0.1",
  "@remix-pwa/push": "^1.0.9",
  "@remix-pwa/strategy": "^2.1.9",
  "@remix-pwa/sw": "^2.1.12",
  "@remix-pwa/sync": "^2.0.1",
  "remix-pwa": "^3.0.19",
},
"devDependencies": {
    "@remix-pwa/dev": "2.0.31",
    "@remix-pwa/worker-runtime": "^2.0.8",
}

@ShafSpecs Apparently the @remix-pwa/push is completely gone on npm:
https://www.npmjs.com/package/@remix-pwa/push?activeTab=readme

My bad, I deleted it last night. I'm re-writing the push as a whole to integrate into the new v4 ecosystem. What was meant to be a simple deprecation turned into a full blown deletion

I would try to republish, but it would be under a different name

@ShafSpecs Then I would appreciate to inform me about the name, whenever it's done. Thank you

@vahidvdn Package now available at: https://www.npmjs.com/package/@remix-pwa/old-push

Thanks @ShafSpecs
Now I get this error:
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>- Not found

Thanks @ShafSpecs
Now I get this error:
npm ERR! 404 Not Found - GET<EMAIL_ADDRESS>- Not found

Oh boy. Peer dependency issues. Would fix and republish soon

@vahidvdn Re-published the fix. Should be installable

@ShafSpecs I'm still getting the same error. You can check this out by browser:
<EMAIL_ADDRESS>

@ShafSpecs This version worked fine: "@remix-pwa/client": "^2.1.0-dev.1",

@ShafSpecs Now I can't build my app. I get the following error:
🏗 Building Service Worker in development mode...

💿 Built Service Worker in: 8.599ms
👀 Watching for changes in the service worker file...
✘ [ERROR] Could not resolve "virtual:entry-sw"

    node_modules/@remix-pwa/worker-runtime/dist/src/service-worker.internal.js:3:23:
      3 │ import * as build from 'virtual:entry-sw';
        ╵                        ~~~~~~~~~~~~~~~~~~

  You can mark the path "virtual:entry-sw" as external to exclude
  it from the bundle, which will remove this error and leave the
  unresolved path in the bundle.

@vahidvdn what are you installing? It's the push package you need right? That's available via: @remix-pwa/old-push

@ShafSpecs Yes, it's fixed. Now I'm still getting the same error for @remix-pwa/client". As I mentioned earlier, this version doesn't exist. You can also check with browser:
<EMAIL_ADDRESS>

You would need the latest version for that 👍

@ShafSpecs As I mentioned, the latest version is not working for me, because I don't want to upgrade to vite. For now, I just want to simply be able to build as before.

@ShafSpecs As I mentioned, the latest version is not working for me, because I don't want to upgrade to vite. For now, I just want to simply be able to build as before.

Oh, that's why. Didn't know that info. The new client should still work regardless though

@ShafSpecs Sorry, but it doesn't. I'm getting these errors with new versions:
✘ [ERROR] Could not resolve "virtual:entry-sw"

    node_modules/@remix-pwa/worker-runtime/dist/src/service-worker.internal.js:3:23:
      3 │ import * as build from 'virtual:entry-sw';
        ╵                        ~~~~~~~~~~~~~~~~~~

  You can mark the path "virtual:entry-sw" as external to exclude
  it from the bundle, which will remove this error and leave the
  unresolved path in the bundle.

/Users/vahid/casenio/dnw/dnw-ticketing/node_modules/esbuild/lib/main.js:1651
  let error = new Error(text);

@ShafSpecs Now I can't build my app. I get the following error:
🏗 Building Service Worker in development mode...

💿 Built Service Worker in: 8.599ms
👀 Watching for changes in the service worker file...
✘ [ERROR] Could not resolve "virtual:entry-sw"

    node_modules/@remix-pwa/worker-runtime/dist/src/service-worker.internal.js:3:23:
      3 │ import * as build from 'virtual:entry-sw';
        ╵                        ~~~~~~~~~~~~~~~~~~

  You can mark the path "virtual:entry-sw" as external to exclude
  it from the bundle, which will remove this error and leave the
  unresolved path in the bundle.

I guess I need to be able to install the same version

Now ik where this error is coming from. You would need to install an older version of worker runtime. Can't remember which, but if you check the npl version tab, I think it was published about 2-3 months ago

@ShafSpecs Starting from 2.1.0-dev.1 is possible to install. But getting the error in build time.
My version was 2.0.1 (right one version before 2.1.0-dev.1), but it's not available on npm (404 error)

@ShafSpecs
ok, first thing is, there is no 2.0.8 at all for @remix-pwa/client. Or maybe I misunderstood you.
Second thing is, I guess previous versions should be there for installation. I mean, it's not backward compatibility. I don't want to upgrade remix-pwa with old version of remix. I just want to keep my app up & running with the same version that was working before.
I hope it was clear and please let me know if I am missing anything

@ShafSpecs
ok, first thing is, there is no 2.0.8 at all for @remix-pwa/client. Or maybe I misunderstood you.

I meant for @remix-pwa/worker-runtime. The error you get about an absence of a virtual:entry-sw is because virtual:entry-sw is exposed for Vite. For use with esbuild, you would need to use an older version. @remix-pwa/client version doesn't matter at all as it isn't based of vite or esbuild

@ShafSpecs oh, sorry. but my worker version is: "@remix-pwa/worker-runtime": "^2.0.8"
I guess I found the issue. My client version is: "@remix-pwa/client": "^2.0.1", but the error says there is no version 2.0.2. I don't know why this happening. Any idea?


How to submit a BASH_IT plugin?

I'm currently writing a bash plugin for book writers and thought it would be worthwhile to share it with BASH_IT community.  It does a few simple things such as setting up a book project, adding pages, removing pages,  re-serializing directory, setting up repository etc.
Should I just submit a PR or there is something else I that should do/read/evaluate with before submitting?
Thanks!

Great - looking forward to your contribution!
Take a look at the other plugins, try to stick to the general format and naming conventions used in the other files, e.g. using dashes for function names instead of underscores (we're not consistent, but new submissions should try to stick to dashes).
There are also some meta tags like group or about, please use them for describing your plugin and its functions.
Ideally your PR is rebased against the current master and squashed into a single commit.
Once you have submitted, we'll review the PR and then provide comments.


let the suffix "" when concurrency is 1

when concurrency=1, do not  add suffix to clientId in a container.
the situation is that,when enable the  kafka's ACL and the clientId  produced by app automatic  and  only one partition in kafka topic，
add the suffix can result the wrong clientid,so when concurrency=1, just let the suffix "".

@happyapple668 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

This would be a breaking change so we can't accept it; sorry (I can't even look at it until you sign the CLA.
We could add an option dontSuffixConsumerZero, which would be acceptable if false by default.
I am releasing 2.5 today so will try to squeeze that in.

@happyapple668 Thank you for signing the Contributor License Agreement!

Thanks for pointing this out; superseded by https://github.com/spring-projects/spring-kafka/pull/1486


Nightly e2e drinking

Description and Motivation

adds codegen script
adds drinking nightly e2e

Has this been tested? How?

tests pass locally

Screenshots (if appropriate)

Types of changes

Refactor / chore

New frontend preview link is below in the Netlify comment 😎

Can you be sure to link and close the related issue using the "closes #9999" syntax in this pr description? Thanks!! @alinix1


Updated reactor core, reactor netty and netty versions to latest

Updated reactor core to 3.3.8.RELEASE - release notes : https://github.com/reactor/reactor-core/releases/tag/v3.3.8.RELEASE

Updated reactor-netty to 0.9.10.RELEASE - release notes : https://github.com/reactor/reactor-netty/releases/tag/v0.9.10.RELEASE

Updated netty to 4.1.51.Final - release notes : https://netty.io/news/2020/07/09/4-1-51-Final.html

Updated netty tcnative to 2.0.31.Final - release notes : https://github.com/netty/netty-tcnative/releases/tag/netty-tcnative-parent-2.0.31.Final

Updated netty tcnative boring ssl to 2.0.31.Final

The reason for these updates is the reactor-netty issue that azure-cosmos is facing : https://github.com/reactor/reactor-netty/issues/1165

This fixes these issues : https://github.com/Azure/azure-sdk-for-java/issues/7349 and https://github.com/Azure/azure-sdk-for-java/issues/7348

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/check-enforcer override


Use irmin 0.12.0

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "new-irmin" git@github.com:samoht/datakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353314136
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Can not run libuvc_installation.sh normally

Hello,When I'm running realsense-viewer,it shows like this:

21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video0 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.0/video4linux/video0 21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video1 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.0/video4linux/video1 21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video2 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.0/video4linux/video2 21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video3 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.0/video4linux/video3 21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video4 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.3/video4linux/video4 21:12:39 [Info] /tmp/sandbox/src/linux/backend-v4l2.cpp:753 - Enumerating UVC video5 realpath=/sys/devices/platform/5fb80000.pcie/pci0000:00/0000:00:00.0/0000:01:00.0/0000:02:00.0/0000:03:00.0/usb1/1-2/1-2:1.3/video4linux/video5 21:12:39 [Info] /tmp/sandbox/src/context.cpp:128 - Found 0 RealSense devices (0xfe requested & 0xff from device-mask in settings)
Then I search online for the problems,I try to run libuvc_installation.sh,but it didn't run normally
`
linaro@bm1684:/data$ sudo ./libuvc_installation.sh

exec
exec
'[' 0 -ne 0 ']'
lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04 LTS
Release:	20.04
Codename:	focal
++ uname -r
echo 'Kernel version 5.4.217-bm1684-g70a3e20060f6-dirty'
Kernel version 5.4.217-bm1684-g70a3e20060f6-dirty
sudo apt-get update
Hit:1 http://ports.ubuntu.com/ubuntu-ports focal InRelease
Hit:2 https://librealsense.intel.com/Debian/apt-repo bionic InRelease
Hit:3 https://librealsense.intel.com/Debian/apt-repo focal InRelease
Reading package lists... Done
cd /root/
sudo rm -rf ./librealsense_build
mkdir librealsense_build
cd librealsense_build
++ sudo swapon --show
++ wc -l
'[' 0 -eq 0 ']'
echo 'No swapon - setting up 1Gb swap file'
No swapon - setting up 1Gb swap file
sudo fallocate -l 2G /swapfile
sudo chmod 600 /swapfile
sudo mkswap /swapfile
mkswap: /swapfile: warning: wiping old swap signature.
Setting up swapspace version 1, size = 2 GiB<PHONE_NUMBER> bytes)
no label, UUID=8ea18b91-7bd9-4148-9e59-7aa2566480ba
sudo swapon /swapfile
swapon: /swapfile: swapon failed: Invalid argument
`

for more information
linaro@bm1684:/data$ lsb_release -a No LSB modules are available. Distributor ID:	Ubuntu Description:	Ubuntu 20.04 LTS Release:	20.04 Codename:	focal linaro@bm1684:/data$ uname -r 5.4.217-bm1684-g70a3e20060f6-dirty
Can you give me some advice ,please

I am using D435i

Hi @DiopWang  This error does not usually occur when using the libuvc_installation.sh script.  My research of non-RealSense Linux cases indicates that when it does occur, it is caused by the instruction sudo fallocate -l 2G /swapfile.  In the libuvc script it can be found on line 26.
Replacing the sudo fallocate command with the one below can resolve the error.
sudo dd if=/dev/zero of=/swapfile bs=1024 count=1048576

Hi @DiopWang This error does not usually occur when using the libuvc_installation.sh script. My research of non-RealSense Linux cases indicates that when this error does occur, it is caused by the instruction sudo fallocate -l 2G /swapfile. In the libuvc script it can be found on line 26.您好，使用 libuvc_installation.sh 脚本时通常不会发生此错误。我对非 RealSense Linux 案例的研究表明，当确实发生此错误时，它是由指令 sudo fallocate -l 2G /swapfile 引起的。在 libuvc 脚本中，可以在第 26 行找到它。
Replacing the sudo fallocate command with the one below can resolve the error.将 sudo Fallocate 命令替换为以下命令可以解决该错误。
sudo dd if=/dev/zero of=/swapfile bs=1024 count=1048576

I really thank you for your response,however,I follow your advice to replace sudo fallocate -l 2G /swapfile as sudo dd if=/dev/zero of=/swapfile bs=1024 count=1048576,but it did not work.I have met this problem before also
like this,it shows swapon: /swapfile: swapon failed :Invalid argument,Can you tell me what can I do next.Thank you so much

Hi @DiopWang This error does not usually occur when using the libuvc_installation.sh script. My research of non-RealSense Linux cases indicates that when this error does occur, it is caused by the instruction sudo fallocate -l 2G /swapfile. In the libuvc script it can be found on line 26.
Replacing the sudo fallocate command with the one below can resolve the error.
sudo dd if=/dev/zero of=/swapfile bs=1024 count=1048576

@MartyG-RealSense Hi,I am really thank you for your response.However,I did what you told me,it still show swapon: /swapfile: swapon failed: Invalid argument. Can you tell me what can I do in other way?Thank you so much

If you comment out the entire swapfile mechanism in lines 26 to 30 then is the libuvc script able to complete sucessfully?
https://github.com/IntelRealSense/librealsense/blob/master/scripts/libuvc_installation.sh#L26-L30

If you comment out the entire swapfile mechanism in lines 26 to 30 then is the libuvc script able to complete sucessfully?
https://github.com/IntelRealSense/librealsense/blob/master/scripts/libuvc_installation.sh#L26-L30

@MartyG-RealSense Yes,I comment out it,and it can work,and Librealsense script completed
But when I am running rs-enumerate-devices,it still did not work.
To be honest,I have tried almost every methods in last 4 days

If you comment out the entire swapfile mechanism in lines 26 to 30 then is the libuvc script able to complete sucessfully?
https://github.com/IntelRealSense/librealsense/blob/master/scripts/libuvc_installation.sh#L26-L30

@MartyG-RealSense Yes,I comment out it,and it can work,and Librealsense script completed But when I am running rs-enumerate-devices,it still did not work. To be honest,I have tried almost every methods in last 4 days

术语库
Could this be the cause of the system kernel, such as no corresponding drivers

When using a libuvc installation of librealsense the kernel should not matter as librealsense bypasses the kernel.
Are you using the official 1 meter USB cable supplied with the camera or a longer USB cable of your own choice?
What is the computer that you are using your D435i on, please (Raspberry Pi, Nvidia Jetson, desktop / laptop PC, etc)
Is the cable plugged into a built-in USB port on the computer or a USB interface such as a hub or PCIe?

When using a libuvc installation of librealsense the kernel should not matter as librealsense bypasses the kernel.当使用 librealsense 的 libuvc 安装时，内核应该无关紧要，因为 librealsense 会绕过内核。
Are you using the official 1 meter USB cable supplied with the camera or a longer USB cable of your own choice?您使用相机随附的官方 1 米 USB 线还是您自己选择的更长 USB 线？
What is the computer that you are using your D435i on, please (Raspberry Pi, Nvidia Jetson, desktop / laptop PC, etc)请问您使用 D435i 的计算机是什么（Raspberry Pi、Nvidia Jetson、台式机/笔记本电脑等）
Is the cable plugged into a built-in USB port on the computer or a USB interface such as a hub or PCIe?电缆是否插入计算机上的内置 USB 端口或 USB 接口（例如集线器或 PCIe）？

I am using official cable,and plugged USB3.0 port,but ubuntu shows it only used USB2.0 port,
It is using a development board called "Shaolinpai" which designed by sophon company

Are you using Sophon's SLKY01 Shaolin Pai board at the link below?
https://en.sophgo.com/product/introduce/slky01.html
It appears to use a Sophon chip called BM1684 as its CPU.  The librealsense SDK is intended for use with x86 / x64 CPUs (like those in desktop and laptop PCs) or Arm CPUs (such as those in Raspberry Pi and Nvidia Jetson boards).  If the BM1684 CPU is neither x86/x64 or Arm chip architecture then librealsense may not know how to make use of it.  It looks as though the BM1684 chip was designed to specialize in processing deep learning activities rather than general computing activities.

Are you using Sophon's SLKY01 Shaolin Pai board at the link below?您是否使用下面链接中的 Sophon SLKY01 少林牌板？
https://en.sophgo.com/product/introduce/slky01.html
It appears to use a Sophon chip called BM1684 as its CPU. The librealsense SDK is intended for use with x86 / x64 CPUs (like those in desktop and laptop PCs) or Arm CPUs (such as those in Raspberry Pi and Nvidia Jetson boards). If the BM1684 CPU is neither x86/x64 or Arm chip architecture then librealsense may not know how to make use of it. It looks as though the BM1684 chip was designed to specialize in processing deep learning activities rather than general computing activities.它似乎使用名为 BM1684 的 Sophon 芯片作为 CPU。 librealsense SDK 适用于 x86 / x64 CPU（例如台式机和笔记本电脑中的 CPU）或 Arm CPU（例如 Raspberry Pi 和 Nvidia Jetson 主板中的 CPU）。如果 BM1684 CPU 既不是 x86/x64 也不是 Arm 芯片架构，那么 librealsense 可能不知道如何使用它。看起来 BM1684 芯片的设计目的是专门处理深度学习活动，而不是一般计算活动。

yes,it is a TPU chip.Thanks for your help

Hi @DiopWang  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


remove cgi module usage, which causes deprecation warnings on Python 3.11

On Python 3.11, as soon as botocore.utils is imported there are deprecation warnings due to the import cgi, a module which is slated to be removed from stdlib. PEP 594 recommends replacing the cgi.parse_header with this email.message substitute.

Codecov Report
Base: 93.57% // Head: 93.53% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (5e14dd4) compared to base (72df8fa).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2794      +/-   ##
===========================================
- Coverage    93.57%   93.53%   -0.04%
===========================================
  Files           63       63
  Lines        13300    13302       +2
===========================================
- Hits         12445    12442       -3
- Misses         855      860       +5

Impacted Files
Coverage Δ

botocore/utils.py
79.02% <100.00%> (-0.04%)
:arrow_down:

botocore/compat.py
92.00% <0.00%> (-0.67%)
:arrow_down:

botocore/httpsession.py
91.43% <0.00%> (-0.39%)
:arrow_down:

botocore/credentials.py
98.44% <0.00%> (-0.21%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@nateprewitt  Just saw you move CI to Python-3.11, could we get some eyes on this PR please?

Hi @wimglenn, thanks for checking in. It's on our radar but we're currently looking at other options than email to replace this behavior. Once we have a conclusion on that, we'll revisit here with a decision. For the time being, there's no immediate risk of removal until October 2023 at the earliest with the release of 3.12.

Interesting, is there some problem with the stdlib email recommendation from the PEP?
I'm not concerned about the removal in 3.13, but about the warning itself (some of escalate warnings to errors in CI)


Bug/6.3/tdi 37635 fix conflict

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
What is the new behavior?
please see the comment in https://jira.talendforge.org/browse/TDI-37635.
adjust the tdataprepdbinput component to meet the requirement of https://jira.talendforge.org/browse/TCOMP-344
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information:
https://jira.talendforge.org/browse/TDI-37635
https://jira.talendforge.org/browse/TCOMP-344

rebuild the test in jenkins as the issue is about the env, not about the current changeset.

rebase it to master again and now some tests will fail :
please see org.talend.components.jdbc.dataprep.JDBCInputTestIT, need to consider it. And will discuss with Seb and chenbin.

have fixed the issue by a workaround.

Pull request test result: success :star:

General comments:

Seems, here we make special tJDBC component adoption for DataPrep product needs.
It's not what we want in TCOMP. TCOMP idea was to have unified component implementation, which is compatible with all Talend products. If you see that something is missed and disallow you to have unified implementation you should notify about it in Components/TCOMP chat.

Please, write javadoc for your code. It is hard to understand code without javadoc.

I've added junit transitive dep exclusion.
I'll merge after successful rebuild

@gonchar-ivan thanks for the review.

Pull request test result: success :star:


CreateHome fails

Describe the bug
Sometimes ™ the CreateHome grpc request fails on first user login. As a result the logged in user sees a Resource not found page, which is yielded by a 404 from PROPFIND on the user home.
What I noticed in the filesystem is that the following folder structure was created, with a symlink to a non-existing folder:
.
├── blobs
├── nodes
│   └── root
│       └── ddc2004c-0977-11eb-9d3f-a793888cd0f8 -> ../ddc2004c-0977-11eb-9d3f-a793888cd0f8
├── spaces
│   ├── personal
│   └── share
├── trash
└── uploads

Steps to reproduce
Steps to reproduce the behavior:

unknown, seemingly random

A hunch that I thought I had, but still couldn't reproduce it in a stable way:

delete ~/.ocis
start ocis single binary (in my case it was with a WEB_ASSET_PATH set to a locally build web): WEB_ASSET_PATH=/path/to/web/dist OCIS_INSECURE=true ./bin/ocis server
try to login via https://localhost:9200 (failing)
have a 2022/01/20 10:23:35 [ERR] mdns: Failed to bind to udp port: listen udp4 <IP_ADDRESS>:0: socket: too many open files listen udp6 [::]:0: socket: too many open files
restart ocis with same command as above
try to login again (succeeding)
see the Resource not found page as described above
see the ocis error log as quoted below

Again, I don't
Expected behavior
User home gets provisioned reliably
Actual behavior
User home sometimes doesn't get provisioned
Setup
oCIS single binary from current master at fdb2f55b77532c1ea52452664d8f7577e2bf8805
Additional context
oCIS error log
2022-01-18T10:48:44+01:00 ERR failed to create storage space error="Decomposedfs: root has no parent" pkg=rgrpc request={"name":"Admin","opaque":{"map":{"space_id":{"decoder":"plain","value":"ZGRjMjAwNGMtMDk3Ny0xMWViLTlkM2YtYTc5Mzg4OGNkMGY4"}}},"owner":{"display_name":"Admin","gid_number":30000,"groups":["users"],"id":{"idp":"https://localhost:9200","opaque_id":"ddc2004c-0977-11eb-9d3f-a793888cd0f8","type":1},"mail":"admin@example.org","mail_verified":true,"opaque":{"map":{"roles":{"decoder":"json","value":"WyI3MTg4MTg4My0xNzY4LTQ2YmQtYTI0ZC1hMzU2YTJhZmRmN2YiXQ=="}}},"uid_number":20004,"username":"admin"},"type":"personal"} service=storage status={"code":15,"message":"error listing spaces","trace":"00000000000000000000000000000000"} traceid=00000000000000000000000000000000
2022-01-18T10:48:44+01:00 ERR error when calling Createhome error="gateway: grpc failed with code CODE_INTERNAL" service=proxy

OS
macOS 11.5.2
I only have 256 file descriptors configured, but then again, I'm not even sure that the too many open files mdns error is related in any way. Of course oCIS is not fully functioning when I have the error, but I'm not sure if this is the thing that leads to the error with the failing CreateHome.

Also happens on continuously deployed demo instances (eg. https://ocis.ocis-web.latest.owncloud.works/).
I did a dump of the user storage (see attached zip), that is similar to @kulmann s example, that can be use to reproduce the state oCIS is in after the CreateHome didn't succeed... Though it doesn't bring us futher to the answer why this happens at all.
tree ~/.ocis/storage/users/
/home/kloucek/.ocis/storage/users/
├── blobs
├── nodes
│   ├── 9bde6eee-1537-4228-a699-0a2afb6accc7
│   ├── ddc2004c-0977-11eb-9d3f-a793888cd0f8
│   └── root
│       ├── 4c510ada-c86b-4815-8820-42cdf82c3d51 -> ../4c510ada-c86b-4815-8820-42cdf82c3d51
│       └── ddc2004c-0977-11eb-9d3f-a793888cd0f8 -> ../ddc2004c-0977-11eb-9d3f-a793888cd0f8
├── spaces
│   ├── personal
│   │   └── ddc2004c-0977-11eb-9d3f-a793888cd0f8 -> ../../nodes/ddc2004c-0977-11eb-9d3f-a793888cd0f8
│   └── share
├── trash
└── uploads

12 directories, 1 file
``
[storage-users.zip](https://github.com/owncloud/ocis/files/7904194/storage-users.zip)

@butonic can you please have a look again?

@wkloucek did you see any too many open files log lines on the deployment examples? Would like to get a feeling if that's related or unrelated.

Possible fix: https://github.com/cs3org/reva/pull/2457

Side note: Started the WOPI deployment example today and ran into this issue multiple times

Side note: Started the WOPI deployment example today and ran into this issue multiple times

which version of oCIS did you use? (if latest, could you please post your docker image digest?)

Side note: Started the WOPI deployment example today and ran into this issue multiple times

which version of oCIS did you use? (if latest, could you please post your docker image digest?)

sha256:55f2b4d71249d06d45241b83780a396eea83aa6c1d906fbbd1e5f90bf9e373e9

55f2b4d71249d06d45241b83780a396eea83aa6c1d906fbbd1e5f90bf9e373e9

did you start from scratch (no volumes present before) or did you have pre-existing volumes? You can check that by running docker inspect ocis_web_ocis-data (which should also be name of the volume on your side). It will stat the creation date in the CreatedAt field.
Background: It was potentially fixed in https://github.com/cs3org/reva/pull/2457. It is already included in your image but if you have a old volume, the bug will be persisted forever...

55f2b4d71249d06d45241b83780a396eea83aa6c1d906fbbd1e5f90bf9e373e9

did you start from scratch (no volumes present before) or did you have pre-existing volumes? You can check that by running docker inspect ocis_web_ocis-data (which should also be name of the volume on your side). It will stat the creation date in the CreatedAt field.
Background: It was potentially fixed in cs3org/reva#2457. It is already included in your image but if you have a old volume, the bug will be persisted forever...

Had some issues because volumes seemed not to be deleted the way I expected them to...now they're all fresh but the problem persists, yes

Just set up a new machine (and both pulled fresh docker images & compiled oCIS locally), still an issue

I have the issue with current owncloud/ocis:latest build (image id: 70d9f96b99e6) - seems to occur inconsistency (eg every 3 or 4th time when you provision a new docker image, even when deleting volumes).
After creating the container and logging in, (I was using admin:admin), every 3rd or 4th time the API request fails with a 404 (proxy makes request to storage-frontend which fails).
Screenshot

Error Log (in container)
{"level":"debug","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","user-agent":"grpc-go/1.44.1-dev","from":"tcp://<IP_ADDRESS>:48818","uri":"/cs3.storage.registry.v1beta1.RegistryAPI/GetStorageProviders","start":"11/Feb/2022:01:30:07 +0000","end":"11/Feb/2022:01:30:07 +0000","time_ns":103300,"code":"OK","time":"2022-02-11T01:30:07Z","message":"unary"}

{"level":"error","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","error":"Decomposedfs: root has no parent","status":{"code":15,"message":"error creating space","trace":"00000000000000000000000000000000"},"request":{"opaque":{"map":{"space_id":{"decoder":"plain","value":"ZGRjMjAwNGMtMDk3Ny0xMWViLTlkM2YtYTc5Mzg4OGNkMGY4"}}},"owner":{"id":{"idp":"https://ocis.owncloud.test","opaque_id":"ddc2004c-0977-11eb-9d3f-a793888cd0f8","type":1},"username":"admin","mail":"admin@example.org","mail_verified":true,"display_name":"Admin","groups":["users"],"opaque":{"map":{"roles":{"decoder":"json","value":"WyI3MTg4MTg4My0xNzY4LTQ2YmQtYTI0ZC1hMzU2YTJhZmRmN2YiXQ=="}}},"uid_number":20004,"gid_number":30000},"type":"personal","name":"Admin"},"time":"2022-02-11T01:30:07Z","message":"failed to create storage space"}

{"level":"debug","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","user-agent":"grpc-go/1.44.1-dev","from":"tcp://<IP_ADDRESS>:43778","uri":"/cs3.storage.provider.v1beta1.ProviderAPI/CreateStorageSpace","start":"11/Feb/2022:01:30:07 +0000","end":"11/Feb/2022:01:30:07 +0000","time_ns":127600,"code":"OK","time":"2022-02-11T01:30:07Z","message":"unary"}

{"level":"debug","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","user-agent":"grpc-go/1.44.1-dev","from":"tcp://<IP_ADDRESS>:48750","uri":"/cs3.gateway.v1beta1.GatewayAPI/CreateHome","start":"11/Feb/2022:01:30:07 +0000","end":"11/Feb/2022:01:30:07 +0000","time_ns":1073000,"code":"OK","time":"2022-02-11T01:30:07Z","message":"unary"}

2022-02-11T01:30:07Z ERR error when calling Createhome error="gateway: grpc failed with code CODE_INTERNAL" service=proxy

2022-02-11T01:30:07Z DBG director found path=/remote.php/webdav/ policy=ocis prefix=/remote.php/ routeType=prefix service=proxy

{"level":"debug","service":"storage","pkg":"rhttp","traceid":"00000000000000000000000000000000","time":"2022-02-11T01:30:07Z","message":"access token is already provided"}

I did not see any error relating to too many open files in my case, below is the full error log since starting the container.
See https://github.com/owncloud/ocis/issues/3152 for original (probable duplicate) bug report.

@nealey IIRC tmpfs does not support extended attributes in the user.* namespace, which is required by the decomposed filesystem ocis uses by default: https://cateee.net/lkddb/web-lkddb/TMPFS_XATTR.html Try using a different filesystem, or use a docker volume.

Reading this thread it seems to either be the default (low) ulimit on macos or conservative oses, or a filesystem without extattr support. There have also been a lot of fixes with regards to file locking and atomicity, so this should no longer be a problem. Closing. Please open a new issue if you think you are still encountering this.


Feature/grid related fixes

Currently we are working on a nuxt integration for the aem spa editor which is based on your vue integration (thanks for your effort).
Due to issues with the AEM Grid we created an internal fork of your package and fixed the missing grid css classes.
However, you were a little bit faster and already released a fix yourself, but we added a few improvements in which you might be interessted, this includes:

minor code style fixes and cosmetic changes
we replaced Utils.modelToProps calls with super method calls like in the react package
minor fix in Container.ts (wrong use of Array.map)

I hope this helps you a little bit, keep up your good work!
If you like, we can also get in touch to work together on a better vue support.

8bd2ceb This achieves the same goal as the cqItemsOrder.map function call. The difference between this implementation and ours is that you are declaring a local variable to store the childComponents while we are just returning a new array with the map function.

I think the result is not exact the same. The Array.prototype.map function is designed to return an array with the same size like the original array. But in your code you are also apply some kind of filter. In some cases you are not having any return statement. I am not sure what map is doing in these cases but probably it will be equal to return undefined.
So I think it is better to explicitly push elements into the array. This combines mapping and filtering and you don't have null or undefined values in your resulting array.


ROX-13710: Enable ROX_SEARCH_PAGE_UI to replace search modal with page

Description
Follow up #2689
Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed
Manual testing
Click Search on top nav and then

Click CLI button to see menu

Click Help button to see menu

Click User button to see menu

See left nav (above) until you click collapse button (below)

Integration testing

Verify that classic modal tests are skipped

/test gke-qa-e2e-tests


MaxStdio

I have very long loading screen before menu. MaxStdio = 2048. Is this is a fix for it?

no, the only fix for that is Private Profile Redirector.

no u do not understand me correctly. i mean i have too long game initialization (before i get control in main menu and can start new game or load save). If i set MaxStdio = -1 (disable), game initialization i fast ( around 2-4 seconds) if MaxStdio = 2048 or higher game initialization is about 1-1,5 minutes! i have a very powerfull pc and can understand why does thsi happen((

That is what Private Profile Redirector fixes.


Dialog was forced to close by an unknown source

I'm submitting a ...
[ x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
A bug

What is the current behavior?
If I open a dialog and then I click the back button of my browser I get an exception (see title)

What is the expected behavior?
I expect a function the dialog provides to catch this action. I could for example do the same like if somebody just aborts the dialog

I see this too.

I am having the same issue. Any updates?

Not from my end.

I have same problem. My dialog bails out randomly now and then and makes the screen freeze with a modal dialog. Impossible to use with this unpredictable behaviour ....
EXCEPTION: Uncaught (in promise): Error: Dialog was forced to close by an unknown source.
Error: Dialog was forced to close by an unknown source.
at new DialogBailOutError (http://localhost/MyApp/libs/angular2-modal/bundles/angular2-modal.umd.js:340:32)
at DialogRef.bailOut (http://localhost/MyApp/libs/angular2-modal/bundles/angular2-modal.umd.js:421:45)
at ModalOverlay.exports.ModalOverlay.ModalOverlay.ngOnDestroy (http://localhost/MyApp/libs/angular2-modal/bundles/angular2-modal.umd.js:1208:32)

So the problem here is that the dialog has a context that it was opened under on the current page. When you navigate away from that page, the context is lost, causing this error to occur.
I've got a simple workaround as well. Use a subscription to watch for navigation events. If you receive a close event and you want your dialog to stay open, redirect back to the current page. Make sure to call unsubscribe when the dialog closes otherwise you will always be redirected back to your current page.
myDialogWatcher = router.events.subscribe((event) => {
processing code here
});
And then when you close your dialog, call:
myDialogWatcher.unsubscribe();
This will force the dialog to keep its context

I have now switched to nx-modal and made it work in my context, but thanks for your reply!

I have the same issue. I didn't understand when it occures and when it doesn't. Any help ?

From https://github.com/shlomiassaf/angular2-modal/blob/master/src/lib/overlay/overlay.component.ts:
 ngOnDestroy(): void {
    super.ngOnDestroy();
    if (this.dialogRef.destroyed !== true) {
      // if we're here the overlay is destroyed by an external event that is not user invoked.
      // i.e: The user did no call dismiss or close and dialogRef.destroy() did not invoke.
      // this will happen when routing or killing an element containing a blocked overlay (ngIf)
      // we bail out, i.e gracefully shutting down.
      this.dialogRef.bailOut();
    }

The dialog belongs to the page that launched it. When the parent page changes, destroy is called on both the component that opened the dialog and the dialog itself. This leaves the dialog in an undefined state because the user has not called dialog.close() yet so there is no defined behavior of what to do with whatever content the dialog has. The current action is to throw an exception notifying that the dialog was closed by an unknown source.
My suggestion was to cancel the navigate while the dialog is open so that we don't encounter such undefined states. I did this by intercepting the navigate event generated by angular and using it to redirect back to the current page. This does cancel the user's ability to navigate while a dialog is open, but it means that the user doesn't lose any data present in the dialog.


Please emit a Signal for the progress

In the class stdf2ph5 is the possibility to disable the progress. I use this in a plugin for spyder, because the used tqdm Progress function generate a exception in spyder.
But in large files, the converting to h5 takes a long time, and the user get now message,
Could you create a Signal(int,int)  and emit this Signal in  you loop, with (progresscounter, self.records_count)?
Than I can connect to this Signal and can display this values in a own function.

Hello,
There is a new argument of the SHP constructor - enable_qt_update.
By default is disabled (False). If you want to enable the new functionality, please set it with value True.
Let me know if this resolve your issue.
Regards,
Seimit


Tapo T310 Support

Describe The Bug
Is it possible to add support for the Tapo T310 temperature & humidity sensor?
Expected behavior
Display info from the sensor.
Logs
N/A
Plugin Config
N/A
Plugin Version
N/A
Homebridge Version
N/A
Node.js Version
N/A
NPM Version
N/A
Operating System
N/A

Hello @parryg i don't have the device so i can add support for it


Bump version to 0.4.1 to reflect re-licensing.

rust-url would like to depend on a relicensed version of heapsize. (https://github.com/servo/rust-url/issues/317).
The re-licensing has already happened, but a version bump (at least minor version) and re-publishing on crates.io is required for that.
This PR simply bumps the version to 0.4.1. Once this get's published to crates.io, other crates can depend on heapsize with MIT/apache.

This change is

Thanks @theduke !
Looks like the test failures are probably a result of upstream nightly breakage.
ping @mbrubeck @Manishearth maybe one of you can get this through.

That failure is tracked in https://github.com/servo/heapsize/issues/86.


Incorrect base_degree for lamport aggregate

In the following code degree for state[0] and state[1] is set to 2, however, I don't see the value for state[0] and state[1] is derived by multiplying value of any other state and so the degree must be 1
https://github.com/novifinancial/winterfell/blob/main/examples/src/lamport/aggregate/air.rs#L51
https://github.com/novifinancial/winterfell/blob/main/examples/src/lamport/aggregate/air.rs#L52
Let me know if I am missing something.
Thanks

I see there is is_binary check for constraint, that's why it's degree is 2.


Add frontend UI of Dashboard (will show Pending Requests from Datastore)

This looks the same as the other pull request whats the difference?

This looks the same as the other pull request whats the difference?

I think both my changes combined into the previous branch, but I basically created a new page called Dashboard to display all of the Pending Bookings!


Please consider add "Typings" npm package to "Prerequisites"

OS - Mac OSX (El Capitan)
Version -
angular-cli: 0.0.39
node: 5.0.0
os: darwin x64
Repro steps -
Install angular-cli without Typings npm package.
The log given by the failure -
"Typings" folder missing
"browser.d.ts" file missing.
Please consider add "Typings" npm package to "Prerequisites"

I have a request which might the sense be the opposite of this one:  Don't require or ask users to "npm install -g typings". That so easily leads to incorrect results because of version skew. Rather, the user should only need to do that if they explicitly want the convenience of running the tool themselves at the command line without "node_modules/.bin/typings". And even then it only works correctly if the tool (which I believe this one does) has code in it to ignore the global installation (making all those extra bits wasted anyway) and run the project local one.
When something like CLI needs to call something like typings, it should only always ever used the locally installed one, never use a globally installed one even if it is present. Therefore, if it has a dependency, it does not need to describe it as a prerequisite, rather it needs to simply declare it as a devDependency in the normal way.... Which it already does.
So if there is an actual bug here, then it is some case by which CLI, when it calls typings, erroneously relies on a global install.

That was old (and erroneous) behavior that has since been fixed, newer versions of the CLI have a local typings install.
There is a bug in beta.6 that requires a global typings install, but that is also unintended and were are looking to fix it: https://github.com/angular/angular-cli/issues/1092


ELEVATE Improvement-Project/Unnati Microservice with PostgreSQL

Product Name
Unnati
Description
Implement a new microservice or a backend system that has all the features and capabilities enabled by Project Service (https://github.com/shikshalokam/ml-projects-service) with PostgreSQL as a database layer instead of MongoDB using NodeJS stack while adhering to design and architecture principles followed by other Micro-service under the ELEVATE project (https://github.com/ELEVATE-Project/)
Database layer change will help build an implementation around survey/assessment that will become a DPG(Digital Public Good)
Reuse as much code as possible from the existing service.
Acceptance Criteria

[ ]  Consume and submit the project

Project
Unnati/Improvement-Project Backend Implementation
Organization Name:
Shikshalokam
Domain
ALL
Tech Skills Needed:
NodeJS,PostgreSQL, MySQL
Mentor(s)
@ankitpws
Complexity
High
Category
Feature
Sub Category
API,Database,Refactoring,Backend,Technical Debt

Hey @aks30, I have worked with NodeJS and relational databases and would love to contribute to this project. Please let me the detailed requirements for it

Hey @aks30, I have worked with NodeJS and relational databases and would love to contribute to this project. Please let me the detailed requirements for it

Thank you for your interest in this project. I suggest you join the Discord channel for the same here - https://discord.gg/g6CZvQDY and let's connect there to take it further.

@starks-and-wolves did you get started with the migration? if not, I'd like to take a stab at it.

@starks-and-wolves did you get started with the migration? if not, I'd like to take a stab at it.

Hey, yes, I am looking into this issue

Hi
@starks-and-wolves @Deekshithrathod
Thank you for showing your interest in contributing to this project. I am delighted to be your mentor on this journey. I would appreciate it if you could reach out to me on Discord (@hi_shanky) to facilitate our future discussions. Looking forward to connecting with you there!

Hey @ankitpws great to hear from you. I have texted you on Discord, can you please confirm if it is the correct user?

Hey @ankitpws great to hear from you. I have texted you on Discord, can you please confirm if it is the correct user?

yes, its correct.


update banner in README.md

put together this simple banner:

Render the markdown and you'll see it's not correct. Also, you accidentally removed the link to the website.

oops


wired up the ingredients JS to the UI

Wired up the inital display of stored ingredients, adding a new ingredient and removing an existing ingredient functionality.

Nice one


not working

`PS C:\Users\user01\Downloads> .\InMemoryNET.exe https://github.com/Flangvik/SharpCollection/blob/master/NetFramework_4.7
_Any/Rubeus.exe triage
~ InMemoryNET ~
[+] Opened connection to github.com!
[+] Sent GET to /Flangvik/SharpCollection/blob/master/NetFramework_4.7_Any/Rubeus.exe!
[!] WinHttpQueryHeaders: 12150
Nothing more.Compiled without problems. Tested on WS2016-2019, firewall was off. Any suggestions?

Hey, fwiw, i dont maintain these repositories so ill be closing this issue.
It works for me, 12150 seems to be ERROR_HTTP_HEADER_NOT_FOUND. Not too sure whats going on, but the repos works for me.

As i said, i dont maintain these poc repos so ill be closing, good luck debugging!


[Feature Request]: Signer util for node wallet api

We need the signer util that can be used in DA Nodes to open up use-cases where multiple users can submit blobs via 1 DA node.
Ref: https://github.com/celestiaorg/celestia-node/discussions/3022

Ref: https://github.com/celestiaorg/celestia-app/issues/3259


Initial redux scaffolding

This PR is intended to start the discussion around what patterns we want to settle on for redux. Questions that need answering:

What middleware do we plan on using? (redux-saga, redux-logger, connected-router, redux-persist, etc)
What directory structure would we feel comfortable moving forward with?
directory structure suggestions
Where do we plan on centralizing types for redux? Where would we like to keep our types.ts files & how to we remain consistent with this?
Do we plan on using union types to infer the type of redux actions? examples in this PR are pulled from
typescript redux example

What middleware do we plan on using?

redux-saga: sagas or async/await? Sagas brings on-the-fly cancel feature.
redux-logger: Is it mainly used for developments only? If yes then arent't devtools (redux-toolkit) enough?
connected-router: I suppose it's connected-react-router?
redux-persist: What are the use cases ? Can't they be addressed using local storage (JS)?
etc...

reduxsauce: If using redux-toolkit then won't be needed.

For several reasons, we have decided to use react-query instead of react-redux. Closing this PR as we start to ramp up with react-query.


Injecting into remote process: Unable to remotly allocate memory Failed Press any key to continue . . . . .

Injecting into remote process: Unable to remotly allocate memory
Failed
Press any key to continue . . . . .
Something like that happening to me even when my drive has 1000gb

The game either has anti-cheat protection measures against DLL injections, you must run the injector as administrator, or the game itself may be x86/x64 bit and you are using the wrong injector version(the x86/x64 must be the same as the game you are targeting. If it says 32-bit then you must use the x86 version of the injector).


expected struct try_future::TryFuture, found struct futures::AndThen

Hello, do you know how to solve this problem?

Hi! Can you try to chain .into_future() onto the end of your last expression? (E.g. state.db. (...) . into _future())

It works perfectly! Thanks. My last question is how to return early a resolved value.
like in this situation:

Closed, the solution is to replace impl Future by Box<Future> and package FutureResult and AndThen, Thank you!


Bonding Curve Contract Requirement

[ ] Should be a pure function utility that takes parameters and returns price
[ ] Should be steeper for first 1,000 NFTs and flatten out afterwards, ideally in the $100-$500. range
[ ] Should re-use a battle tested curve, such as SoundSwap, audit, source

Architecture Possibilities:
Option 1

Bonding curve is deployed by factory at the same time as each creator token
Each creator token is tied to exactly one bonding curve contract
Optionally: Each creator token can update to point to a new bonding curve contract in the future if they want
Optionally: the parameters on the bonding curve contract can also be updated (by admin? and creator?)

Option 2

A single bonding curve is deployed at the same time the factory is deployed
Each creator token is given that bonding curve at the time of deployment
Optionally: Each creator token can update their bonding curve in the future independently
Optionally: The global parameters on the shared bonding curve can be updated by admin

Option 3

A single bonding curve is deployed at the same time the factory is deployed
Each creator token asks the factory contract for the current bonding curve, i.e. beacon pattern
Optionally: The admin can update the global bonding curve for all creator tokens
Optionally: The admin can update the parameters on the global bonding curve

After a call with the showtime team, we are going with Option 1 with no updateability.


Consider using OrderedDictionary<Key, Value> for maps

https://github.com/apple/swift-collections contains OrderedDictionary<Key, Value>.  If it were adopted, then parsing/serializing a message with a map would be less likely to change the order of the binary produced.
The downside is it would be an api change since they are different types and don't immediately convert from one to the other.
I believe some of the other languages use something like an ordered map under the hood to get the behaviors it would enabled here.

Thinking about this some more… Go (and maybe some other languages?) have started looking at ensure things are actually more random - i.e. - not only do they not worry about map ordering, but when generating TextFormat, Binary, or JSON, the very the field order to help ensure folks aren't making assumptions.  So similar to how Swift changed hashing to get more random values, it might make sense to never do this (and maybe even talk about doing some of the things other languages are considering).

Going to close this since I don't think we should do this any more given what some of the other languages are doing, we can always revisit in a new issue if things change.


Port Issue

Not sure if this is a bug or a system issue.
(debug=True, port=3000)
yet my app still runs on 5000 (default port)

You're presumably referring to passing arguments to app.run. But the flask command doesn't execute that function. Use the --port option: flask run --port 3000.

thank you.

i usually dont build with flask, i just do make


AppResource reloads when toggling reveal all resources

It's happening for Storage Accounts, but not for Functions. Haven't tested all types yet.
https://user-images.githubusercontent.com/12476526/184420829-cf7a05f7-873d-498c-8851-37a76adc3da1.mov

NOTE: Verify this when we change to the new api.


[Feature] Provide option to only update specific columns in incremental model

Describe the Feature
When building data models using dbt-oracle, currently for an incremental model, there is no optional way to update only the columns I want. The adapter will update all columns except unique_keys. (I just checked out the code and found it, so if it has been satisfied before, please let me know).
For example, my data model follows the Fact-Dimension model, and here is an example Dimension table:
Table: D_MODEL
Columns: ID, MODEL_NAME, UPDATED_DATE
As we can see, the unique_key ID in the Dimension table is just used to join the Fact table. But, the merge on columns is MODEL_NAME, it is not the primary key of Dimension tables.
So, in this case, I only want to update the column UPDATED_DATE, but there is no config to finalize them. Dbt-oracle update column ID and UPDATE_DATE, and the whole Fact-Dim model seem broken because it can not join again.
Describe alternatives you've considered
Recently, I have worked with dbt-bigquery and they have the option merge_update_columns docs to update only columns they want.
I also read the merge_update_columns source code and found they customized the BaseConfig with BigqueryConfig, then use them on Jinja code too to filter out targeted columns.
So, our problem can be resolved by just creating an additional argument and logic, seem easy!
Who will this benefit?
I thought it is very useful for a lot of users who built special data models with unexpected behavior and make dbt-oracle more fluently.
Anything else?
Maybe if I have time, I will experiment on them. Also, can I create a pull request, or contribute to this project?

Thank you @viplazylmht for raising this feature request. This is now addressed in dbt-oracle==1.1.1
You can use the config merge_update_columns in incremental models
{{config(materialized='incremental',
         unique_key='user_id',
         merge_update_columns=['user_name', 'income', 'last_login_date'])}}
SELECT * FROM {{ ref('seed') }}
{% if is_incremental() %}
    WHERE last_login_date > (SELECT max(last_login_date) FROM {{ this }})
{% endif %}

Please test this with dbt-oracle==1.1.1 and let me know if you have any questions.
If you want to check the changes please review the PR https://github.com/oracle/dbt-oracle/pull/35


Admin videos manager last page is broken when there are no videos

This is issue also valid for front-end website


Modality fusion implementation question

hello. I am studying 2DPASS with code.
It seems that the modality fusion implementation is in network/arch_2dpass.py from line 100 to line 105.
In the thesis, bitwise add is specified, but I do not see it in the code, so I ask a question.
Below is the code.
modality fusion
feat_learner = F.relu(self.leanersidx)
feat_cat = torch.cat([img_feat, feat_learner], 1)
feat_cat = self.fcs1idx
feat_weight = torch.sigmoid(self.fcs2idx)
fuse_feat = F.relu(feat_cat * feat_weight)
I think that [fuse_feat = F.relu(feat_cat*feat_wieght) + img_feat] implements the formula in the paper as a code.
Isn't it?

i have the same question
thanks in advance

This issue is code implementation, not performance. I think open code is not implemented as described in the paper.

So, do you mean that some implementations mentioned in the paper can not work in this released code? Or it may be missing in the code. Therefore, it is impossible to reproduce the model as reported performance.

Missing in the code.
modality fusion
feat_learner = F.relu(self.leaners[idx](pts_feat))
feat_cat = torch.cat([img_feat, feat_learner], 1)
feat_cat = self.fcs1[idx](feat_cat)
feat_weight = torch.sigmoid(self.fcs2[idx](feat_cat))
fuse_feat = F.relu(feat_cat * feat_weight)

I think that [fuse_feat = F.relu(feat_cat*feat_wieght) + img_feat] implements the formula in the paper as a code.
feat_learner = F.relu(self.leaners[idx](pts_feat))
feat_cat = torch.cat([img_feat, feat_learner], 1)
feat_cat = self.fcs1[idx](feat_cat)
feat_weight = torch.sigmoid(self.fcs2[idx](feat_cat))
fuse_feat = F.relu(feat_cat * feat_weight) + img_feat

Thanks for your reply, I will further check this issue.

I just trained the modified version as you said, and the performance did improve a little bit around 2 on mIoU. I believe there may be some other wrong implements or missing in the resealed code. And thank you again.

I just trained the modified version as you said, and the performance did improve a little bit around 2 on mIoU. I believe there may be some other wrong implements or missing in the released code. And thank you again.

@kjwkch @LiXiang0021
After reviewing the current implementation, I noticed that besides the fusion modification, the point feature pass through the 2D learner needs to be added to the original point feature before passing through multihead_3d_classifier, in order to match the model architecture outlined in the paper as below:

        feat_learner = F.relu(self.leaners[idx](pts_feat))
        # feat_learner -> voxel-wise feature after 2D learner

        pts_pred_full = self.multihead_3d_classifier[idx]((pts_feat+feat_learner))
        # pts_feat+feat_learner -> voxel-wise Enhanced 3D Features

        # correspondence
        pts_label_full = self.voxelize_labels(data_dict['labels'], data_dict['layer_{}'.format(idx)]['full_coors'])
        pts_pred = self.p2img_mapping(pts_pred_full[coors_inv], point2img_index, batch_idx)

        # modality fusion

        feat_learner = self.p2img_mapping(feat_learner[coors_inv], point2img_index, batch_idx)
        # feat_learner -> point-wise feature after 2D learner and img_mapping

        feat_cat = torch.cat([img_feat, feat_learner], 1)
        feat_cat = self.fcs1[idx](feat_cat)
        feat_weight = torch.sigmoid(self.fcs2[idx](feat_cat))
        fuse_feat = F.relu(feat_cat * feat_weight) + img_feat

Currently, the implementation takes the point feature as input directly for multihead_3d_classifier instead of adding the point feature after the 2D learner.
https://github.com/yanx27/2DPASS/blob/80b8646bbb0dd46ddcee31f531d6f1c45baf35fe/network/arch_2dpass.py#L89-L93
@yanx27, I would appreciate any suggestions you may have regarding this matter.

@jaywu109  does the new script changes work for you ?


Fix on change data

Case:
initPlugin action doesn't work properly in case when you didn't define matcher in options. So when you try change value without defined matcher it won't work.
Related issues:
https://github.com/NejcZdovc/ng2-select2/issues/100
https://github.com/NejcZdovc/ng2-select2/issues/104
https://github.com/NejcZdovc/ng2-select2/issues/105
https://github.com/NejcZdovc/ng2-select2/issues/108

@NejcZdovc can you review this?

@NejcZdovc please review this. It fixes a bug


[Request For External] Report 840 Suggest Worksheet Line

Please make the following variables protected in Report 840 Suggest Worksheet Line to allow Extensibility
protected var
        Window: Dialog;
        ConsiderSource: array[50] of Boolean;
        SourceType: Enum "Cash Flow Source Type";
        CFWorksheetLine2: Record "Cash Flow Worksheet Line";
        Customer: Record Customer;
        Vendor: Record Vendor;
        CFSetup: Record "Cash Flow Setup";
        SelectionCashFlowForecast: Record "Cash Flow Forecast";
        Currency: Record Currency;
        CurrExchRate: Record "Currency Exchange Rate";
        TotalAmt: Decimal;

Thank You!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


old ODR reference to be updatable. Planio#29054

This is to add in the ability to edit the ODR reference on the system.

I've now merged update_application_log into main for 2022_sprint_12, as well as develop.


Broken in Opera

The better question is probably why on Earth am I using Opera as my web browser.... Regardless though I decided to try it out and Tilde is just completely broken.
Used this addon to access the page whenever I open a new tab. Basically when hitting enter nothing happens. Then input box is cleared but it doesn't take me anywhere. Since it's built off of chromium I'd assume the javascript wouldn't act any different so maybe it's the addon.
I don't expect work to be put into this but I figured I'd at least document it and if I find a workaround I'll update the issue.

Interesting. Are you getting any console errors?

 Made an issue without even thinking to check that. Turns out I had settings on that denied sites from using localstorage... Thanks for the help!

No worries! Enjoy 😄


test pages should point to local qunit resources as opposed to the remote cdns

Currently we point qunit.js and .css to their cdn locations. This is done for the purpose of running tests via TravisCI. We should revert to having these resources locally to allow for running tests offline.

This has been fixed. Test pages now look like https://github.com/bitovi/canjs/blob/master/component/test.html using steal-qunit. We just have to make sure that we add those to the GitHub pages otherwise the tests won't run.