can not shutdown when using RealtekCardReader.kext

I'm using using dell vostro 3578 laptop
CPU :I5 8250U
RAM:8GB
INTEL HD 620
When  I using RealtekCardReader.kext and RealtekCardReaderFriend.kext,  I shutdown my laptop bit it restart
If I remove this kext, It's normal
Can you help me ?

Can u please mention which card reader you have (and upload kernel logs)?

I don't know. Can you tell me a tool to check my model card reader
I'm newbie
Thanks you so much

Please go to system information and select "card reader" and send screenshot. Realtekcard reader kexts must be present for this

Thanks you very much
This is my screenshot

☺ U have Realtek RTS5129 card reader. I too have that reader, not faced the problem mentioned by you but still @0xFireWolf can help you with that

Thanks for your comment
Thank you very much

U can follow InsanelyMac and report it there too alongwith logs (check FAQ)

Thanks you so much

Hay @0xFireWolf  I using Realtek Card reader0.8.6, I shut down my computer, but computer restart
Can't you fix it ?


Mystery parameter

I'm using the latest angular (1.5.3) and the latest restangular (1.5.2) and am having the strangest issue. I have a simple request Restangular.one('Organization',5).get() that will always pass a mystery parameter that I'm not sure where it's coming from. It will always include "organization=123" (the 123 value is a valid number from my db - that is set in $scope elsewhere). If I set manually like so .get({organization:''}), then it does not get set. Otherwise the organization parameter is always there. Not sure why. Can you help?

Can you create a Plunk or fiddle to demonstrate this?

Much too complex of an app to create a plunk for it. Basically I don't know how/why the organization parameter is being set. It it set every time throughout the system where Restangular is used for "Organization" (both one and all). Somehow the "organization" parameter is being auto set on all of these requests, and I cannot find where or how. I've check the interceptors in the config, and nothing there. What am I missing?

Just found another place in the system where we're doing Restangular.all('job').getList() and this also appends the ?organization=123 to the request.

Found the problem. Another dev had used Restangular.setDefaultRequestParams on a single request, when in fact it affected every subsequent request. I have it under control now. Sorry for the false alarm.

Ah okay. Great! Glad to hear that


Add --layer-cache-disk-path arg to relevant services

Adds a --layer-cache-disk-path arg to rebaser, pinga and sdf to allow us to specify a non /tmp/ path when running in Production/Tools Production.
This should let us monitor + work with the services more easily.

bors merge

bors cancel

bors merge


React native 0.73 fails to build release

RN 0.73 upgraded using https://react-native-community.github.io/upgrade-helper/?from=0.72.4&to=0.73.1
AGP: 8.2.0
Gradle 8.3
react-native-vector-icons 10.0.3
The issue is similar to #1508 but this time the error is with other task generateReleaseLintVitalReportModel:
A problem was found with the configuration of task ':app:copyReactNativeVectorIconFonts' (type 'Copy').
  - Gradle detected a problem with the following location: '..../android/app/build/intermediates/ReactNativeVectorIcons/fonts'.

    Reason: Task ':app:generateReleaseLintVitalReportModel' uses this output of task ':app:copyReactNativeVectorIconFonts' without declaring an explicit or implicit dependency. This can lead to incorrect results being produced, depending on what order the tasks are executed.

    Possible solutions:
      1. Declare task ':app:copyReactNativeVectorIconFonts' as an input of ':app:generateReleaseLintVitalReportModel'.
      2. Declare an explicit dependency on ':app:copyReactNativeVectorIconFonts' from ':app:generateReleaseLintVitalReportModel' using Task#dependsOn.
      3. Declare an explicit dependency on ':app:copyReactNativeVectorIconFonts' from ':app:generateReleaseLintVitalReportModel' using Task#mustRunAfter.

Adding the following to fonts.gradle fixes it
        def generateReportTask = tasks.findByName("generate${targetName}LintVitalReportModel")
        if (generateReportTask) {
            generateReportTask.dependsOn(fontCopyTask)
        }

Could you please release a new release with that?

@tri-bao
To fix this ,follow these step

remove apply from: file("../../node_modules/react-native-vector-icons/fonts.gradle") from android/app/build.gradlew
go to node_modules/react-native-vector-icon/Fonts and copy all the fonts
go to android/app/src/main and create a folder named assets if not exist then inside then assets create one more folder named fonts and paste all the fonts from   node_modules/react-native-vector-icon/Fonts to here
now you can use the vector icon as usual and also you can create a build which won't failed

thanks @Vimal1464 while that works I had used patch-package to add the required task dependency into react-native-vector-icons/fonts.gradle

Welcome @tri-bao.
Can we follow each other ? as I am new to GitHub looking for some friends ...

Welcome @tri-bao. Can we follow each other ? as I am new to GitHub looking for some friends ...

sure, followed

@tri-bao  followed too. ❤
are you on discord ??
mine is vimal1464

In my case i just add
apply from:("../../node_modules/react-native-vector-icons/fonts.gradle");
in android\app\build.gradle then problem fixed for me

In my case i just add apply from:("../../node_modules/react-native-vector-icons/fonts.gradle"); in android\app\build.gradle then problem fixed for me

I was stuck in this for two day and this simple solution fixed for me as well, thank you @ahmeturganci

I am facing the same problem, no solution to this yet.

In my case I added
-keep class com.facebook.react.fabric.** { *; }
in /android/app/proguard-rules.pro and its started working

For my case, I've added the related dependency to build.gradle. For your code, it probably would be like:
tasks.configureEach {
        if (name == 'generateReleaseLintVitalReportModel') {
            dependsOn('copyReactNativeVectorIconFonts')
        }
    }

I hope it helps 🙏.
P.S: After implementing this method, the package seemed not to be working. I had to reset the cache.

I had this same error, but on the lintAnalyze${targetName} task.  I used @Jouskar workaround thanks muchly!

https://github.com/oblador/react-native-vector-icons/issues/1604
"I'm currently working on a React Native project (version 0.73.2) and encountering an issue with react-native-vector-icons integration. Despite following the installation steps (npm install --save react-native-vector-icons followed by pod install), the RNVectorIcons library does not seem to be added to my Podfile.lock or properly linked in my Xcode project. Auto-linking should work with my React Native version, but it appears to be missing. Is there a known issue or step I'm overlooking for React Native 0.73.2?"

Here is the patch patch for anyone that needs it
diff --git a/node_modules/react-native-vector-icons/fonts.gradle b/node_modules/react-native-vector-icons/fonts.gradle
index 951394b..ad15ebd 100644
--- a/node_modules/react-native-vector-icons/fonts.gradle
+++ b/node_modules/react-native-vector-icons/fonts.gradle
@@ -32,6 +32,11 @@ afterEvaluate {
         lintVitalAnalyzeTask.dependsOn(fontCopyTask)
         }

+        def generateReportTask = tasks.findByName("generate${targetName}LintVitalReportModel")
+        if (generateReportTask) {
+            generateReportTask.dependsOn(fontCopyTask)
+        }
+
         def generateAssetsTask = tasks.findByName("generate${targetName}Assets")
         generateAssetsTask.dependsOn(fontCopyTask)
       }

For my case, I've added the related dependency to build.gradle. For your code, it probably would be like:
tasks.configureEach {
        if (name == 'generateReleaseLintVitalReportModel') {
            dependsOn('copyReactNativeVectorIconFonts')
        }
    }

I hope it helps 🙏. P.S: After implementing this method, the package seemed not to be working. I had to reset the cache.

Thank you worked!

generateReleaseLintVitalReportModel

path ?


interaction with api for rest client

@Andrew-Sem provide details, why do you need crud for rest client?

@l3r8yJ sry, that's not what I meant
I meant just interaction with api, only read case
Should I rename issue or create new one for my case?

@Andrew-Sem qa-rest is only needed for read operations to interact with chat, for crud operations we have qa-admin.

@l3r8yJ yep, that's what I'm talking about

@Andrew-Sem then why do we need crud in qa-rest?

@l3r8yJ sry, that's not what I meant I meant just interaction with api, only read case Should I rename issue or create new one for my case?

@l3r8yJ We don't need as I corrected myself earlier

@Andrew-Sem we don't have crud operations in qa-rest, I don't undertand what you mean

@l3r8yJ Everything is very simple. I got a little confused with the name of the issue when I created it. You're right, we only need the read api. By crud I meant interaction with api - reading.
Finally, I'm just going to change the name of the issue now to make it more sense

@Andrew-Sem so, what’s wrong with interaction with api for rest client?

@l3r8yJ the problem is its absence. Here is a ticket for me

@Andrew-Sem got you, write something like i'm in or etc in future

@l3r8yJ k


Register weights as a non-persistent buffer of SinusoidalPositionalEmbedding

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
Discussed this in a GitHub issue of the pytorch repo.

[x] Did you read the contributor guideline?

[ ] Did you make sure to update the docs?
Not applicable

[ ] Did you write any new necessary tests?
No. But I've tested deeplearning/projects/fairseq-py:test_cpu in Meta's fbcode repo, and this diff does not introduce any new test failures.

What does this PR do?
The module SinusoidalPositionalEmbedding has the problem that its weights attribute is not moved to CPU or CUDA when the module is moved.
Registering weights as a buffer solves the problem.
This also eliminates the need for the buffer _float_tensor, which is used to keep track of whether the module is on CPU or CUDA.
Making weights a non-persistent buffer means it won't be saved to or loaded from a state_dict.
With the changes in this diff, the state_dict of a SinusoidalPositionalEmbedding module should contain neither weights or _float_tensor.
This diff ignores them by overriding the _load_from_state_dict method of the SinusoidalPositionalEmbedding module, instead of duplicating the code in many upgrade_state_dict functions.
TO DISCUSS: Is it OK for me to override _load_from_state_dict? It's a private function, but I see people have overriden it in many places, including in fairseq:
https://github.com/search?q=super()._load_from_state_dict&type=code
https://github.com/search?q=repo%3Afacebookresearch%2Ffairseq super()._load_from_state_dict&type=code
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Does this keep backwards compatability?

Does this keep backwards compatability?

Yes. The buffer weights is created upon construction of a SinusoidalPositionalEmbedding object; it doesn't need to be loaded from a state_dict.
With the changes in this PR, no matter whether a state_dict contains the keys weights and _float_tensor or not, these keys will be ignored.

BTW In the discussion on the pytorch issue, they've said it's OK to override the _load_from_state_dict method.


New pretty printing

The new pretty printer implemented in #8019 is great. Having type annotation in the repr is just awesome.
However, I noticed when the jaxper is small everything is squeezed in a single line. I understand it makes higher-order ops look saner, but I'm not sure it looks clearer in the simpler case.
def f(x):
  return x * jnp.sin(x)
jax.make_jaxpr(f)(1.)

# gives
# { lambda ; a:f64[]. let b:f64[] = sin a; c:f64[] = mul a b in (c,) }

Maybe a top-level jaxper should always use the multiline style? Or even lose the brackets? Something like
lambda ; a:f64[].
let b:f64[] = sin a
    c:f64[] = mul a b
in (c,)

We could do that. But are you sure you actually prefer it, as opposed to the new style being unfamiliar?
(By the way, if you are looking at jaxprs, try out the new but undocumented color and source context features:
print(jaxpr.pretty_print(use_color=True, source_info=True))

assuming you have an ANSI-compatible terminal.)

as opposed to the new style being unfamiliar?

That's a fair assumption. I will give the newer style some more time.

color and source context

Wow, just wow!
Shall I leave this issue open so people have a place to give their feedback about the new pretty printer, or do you prefer that I close?

Let's close the issue because there's no action to take. But feel free to reopen it if there's something to do or any comments to make!


WSDL Updates, Usernametoken authentication, multi network apdater support

This commit contains changes about how the WSDL stubs are generated: Maven will generate updated stubs if needed under target/generated-sources, so you can easily update the wsdl file and obtain the new stubs.
WSSE Usernametoken has been implemented in order to support this kind of authentication.
You can configure which network adapter you have to use (in case of multi network environment) to perform client-server communication (you have to use JVM Parameter -DnetworkInterface (default is eth0))

Can you update the README in order to explain how to use the library now that the client classes should be regenerated? I'm not sure whether this should be the default behavior or not..maybe we can provide different profiles with a set of pre-configured stubs.  It's quite uncomfortable to check all WSDL versions..

Hi @quonn77,  I've just merged another PR that improved a lot the current code. Could you try to rebase your code to the new master? Otherwise I'll try to rebase it by myself as soon I find the time..

Sorry @fpompermaier  but I have totally missed the message....it got just 2 years to read it.... Next week I will work to align with your PR.

Thanks @quonn77

Any update on this @quonn77 ?


Change package name

As published on npm, the package name is jsnet.

Coverage remained the same at 100.0% when pulling 399cf6dcfaf0ad323cb87cb336abdda34813a2fe on abhisheksoni27:patch-1 into 150199e3107564e45923cb884767ce07df112ac2 on DanRuta:master.

Nice, thanks for catching that! Can't believe I just now learned that npm is case sensitive

Haha! It happens. :laughing:


Apple Match-O Linker (Id) Error

I've done exactly what you wrote but I get this error (for FirebaseAuth):

Undefined symbols for architecture arm64:
"_FIRAuthStateDidChangeInternalNotification", referenced from:
___50-[FIRAuth possiblyPostAuthStateChangeNotification]_block_invoke in FirebaseAuth(FIRAuth_d3345993aebdf1a4605c6b15c3385d2b.o)
"_FIRAuthStateDidChangeInternalNotificationAppKey", referenced from:
-[FIRAuth possiblyPostAuthStateChangeNotification] in FirebaseAuth(FIRAuth_d3345993aebdf1a4605c6b15c3385d2b.o)
"_FIRAuthStateDidChangeInternalNotificationTokenKey", referenced from:
-[FIRAuth possiblyPostAuthStateChangeNotification] in FirebaseAuth(FIRAuth_d3345993aebdf1a4605c6b15c3385d2b.o)
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Same here

I gave up and used the released .framework files. It works fine.

What do you mean?

I'm getting the same issue, @maysamsh interesting too by your solution if you can explain it.

The solution was to add it without using this repo 😄

@tbaranes Just add it manually, follow this https://firebase.google.com/docs/ios/setup

Got it, I tried this as well, but still having the issue... 😑

@tbaranes What about my trick? delete the Info.plist, and drag it to your project from another project.

Not sure how that would help, but it didn't work. One thing that may be different from the usual cases, I'm adding Firebase to a framework target instead of the app one, but I don't see why it won't work this way.

I finally get it working by mixing the instructions from this repository and the manual one from firebase. Not really sure why is it working now and not before, but, well, it's working.
Closing since everyone seems to have find a way to make it work!


monthSelectPlugin - Dynamically setting minDate of second date input

My problem is that when I have 2 date inputs and when I use onClose event to set minDate of the second date input field, dates before my minDates are not visually disabled (greyed out), yes if I click on them no date will be picked. They become disabled only when I press a button (arrow) for next year and then go back and everything is disabled and it's working properly. I tried using true when I'm using setDate for it to trigger change event, but that didn't help.
Fiddle
https://jsfiddle.net/8rs3h4vu/1/
Snippets
Snip of how it should be:

Snip of how it is before I click on the arrow that is circled:

My Environment

flatpickr version used: v4
Browser name and version: Chrome
OS and version: Windows 10

got same issue. I also dynamic set min or max date in month plugin. is there any workaround like mimic go to next year and go back action? i tried changeYear(1),changeYear(-1). but didn't work

Any update on this ? I encountered the same issue. I also tried to find a workaround but without success. Let me know if you found anything that might help.

I got same issue.

I got same issue


Add option to scan a list of files

This PR adds the ability to scan only single files by passing the flag --scan-files FILE1 FILE2 FILE3 ... to the manage.py scan command.
This can be useful if another tool (such as inotifywait) is used to monitor the data folder for changes. When a change is detected, manage.py scan can be called with just the newly added photos and videos which significantly reduces the processing time needed to complete the scan.
Example usage:
sudo docker exec --user root backend python3 manage.py scan --scan-files /data/photo1.jpg /data/photo2.jpg /data/photo5.jpg

Sorry for the long wait :) Looks good to me :+1:


[device] VecDequeue instead of ringbuffer

Took your fix for drain, also removed one of the drains by just using .into::<Vec<u8>>()


java.lang.IllegalStateException: Skipping rule(s) which are depending on a rule which is not loaded

Observed Behavior
If you disable a rule such as:
[*.{kt,kts}]
ktlint_standard_multiline-expression-wrapping = disabled

It will cause execution to fail due to dependent rules requiring it to be loaded.
lint worker execution error
java.lang.IllegalStateException: Skipping rule(s) which are depending on a rule which is not loaded. Please check if you need to add additional rule sets before creating an issue.
  - Rule with id 'RuleId(value=standard:string-template-indent)' requires rule with id 'RuleId(value=standard:multiline-expression-wrapping)' to be loaded
        at com.pinterest.ktlint.rule.engine.internal.rulefilter.RunAfterRuleFilter.filter(RunAfterRuleFilter.kt:77)
        at com.pinterest.ktlint.rule.engine.internal.rulefilter.RuleFilterKt.applyRuleFilters(RuleFilter.kt:15)
        at com.pinterest.ktlint.rule.engine.internal.RuleExecutionContext$Companion.createRuleExecutionContext$ktlint_rule_engine(RuleExecutionContext.kt:192)

Expected Behavior
Disabling a rule should only prevent it from being evaluated, it shouldn't impact the executability of other rules which may share some behavior or logic.
Your Environment

Version of ktlint used: 1.0.0
Name and version (or code for custom task) of integration used: kotlinter-gradle 4.0.0

Well, practically speaking I found it impossible to configure ktlint in a way that would be workable for my organization. I'm not sure we'll ever be able to upgrade to ktlint 1.0, which will be a bit of an odd life experience for me given that separately I also maintain a gradle plugin for ktlint.
Here's the problem, and the example you show there is a great example, a rule like multiline-expression-wrapping is quite opinionated and would ripple through a codebase that had been previously consistent with ktlint. It's a bit jarring and I doubt every organization would find a rule like that desirable. On the other hand string-template-indent seems very sensible and desirable and I want that rule. I can't think of a functional reason why string-template-indent would also require multiline-expression-wrapping?

I ran into this same issue, and I also came across another similar rule dependency like this which makes even less sense to me.
  - Rule with id 'RuleId(value=standard:if-else-wrapping)' requires rule with id 'RuleId(value=standard:discouraged-comment-location)' to be loaded

Why should comment locations have anything to do with actual code blocks. 😕 I only want to disable the discouraged-comment-location rule so we can put comments where we like. Forcing us to disable other reasonable code rules to achieve it doesn't feel great.

I appreciate it's a very difficult problem and compromises will need to be made somewhere. Thanks for the explanation.

I can only 100% agree what Peter wrote. This really feels like a step backwards and not like something I'd consider 1.x

discouraged-comment-location is refactored and no rule depends on it anymore. Some logic is moved inside existing rules. 4 new rules with a smaller scope are introduced to disallow comments in for example a value argument (list). The smaller rules at least make it more understandable why there is a relation between rules. For more context, see #2371.

In general, I agree with the points raised here, I had to disable (at least at first) quite a few rules to be able to migrate to 1.x without many changes.
Thanks for the adjustments around discouraged-comment-location, doing
ktlint_standard_value-parameter-comment = disabled
ktlint_standard_value-argument-comment = disabled

is much better than
ktlint_standard_value-discouraged-comment-location = disabled
ktlint_standard_value-if-else-wrapping = disabled

As general feedback, multiline-expression-wrapping is by far the most controversial rule I needed to disable, followed closely by function-signature. I'll try deep diving on those and propose something that would allow me to re-enable them, but no ideas on how to approach that so far.

@jeremymailen wrote:

Here's the problem, and the example you show there is a great example, a rule like multiline-expression-wrapping is quite opinionated and would ripple through a codebase that had been previously consistent with ktlint. It's a bit jarring and I doubt every organization would find a rule like that desirable. On the other hand string-template-indent seems very sensible and desirable and I want that rule. I can't think of a functional reason why string-template-indent would also require multiline-expression-wrapping?

On this I replied with:

The multiline-expression-wrapping is required for string-template-indent to handle code like:
val foo = """
    some text
   """.trimIndent()

The string-template-indent has no idea how to calculate the identation of a multiline raw string for which the opening quotes are not aligned with the closing quotes.

The dependency on the multiline-expression-wrapping ensures that the opening """ are wrapped to a newline. I now see that it would be more convenient to let the string-template-indent do the wrapping of the opening """ which would mean that the rule dependency can be removed. Wrapping the opening """ functionally would fit in the string-template-indent rule as well.
@jeremymailen Now the question is whether you meant that wrapping of the opening """ is something which is not acceptable to your organization or that you only meant the generic multiline-expression-wrapping rule?

That sounds great. I only meant that adopting a different rule that affected all or many lines of code might become a sticking point. From a user point of view my opinion is that string template indent would ideally be independent of all other rules except the indent size. I totally admit this comes with a lot of implementation challenges and a rule needing some context input as to what formatting it's taking part in so that it's cooperative.

As far as I know, no further action is required at this moment. It the issue is not yet completely resolved, please add new information.


Update dependencies

Summary
terra-date-picker's DateInput never properly implemented react-datepicker's onFocus method which gets passed to custom inputs (which looking at react-datepicker's code might have been intentional) and focus became required for typing in input to work.
Side note: DatePicker grabs a method onInputFocus from its customProps to be called when DateInput gains focus. That method should probably be included as a prop instead of a custom prop.
Additional Details
Closes #1548
Thanks for contributing to Terra.
@cerner/terra

The terra-date-picker changes are blocked until Hacker0x01/react-datepicker#1374 is published or something with similar changes (specifically this one). As that could be an indefinite length of time, I am removing that portion of this PR for now.


Suggestions for accompanying Icon theme?

Just some feedback on this issue: We're looking into creating a Catppuccin-coloured fork of a popular icon set.
Please feel free to suggest sets which we could fork.

I would suggest Bearded Icons set.

I've used the ever-popular Material Icon Theme for the longest time, and to be honest it would be very difficult for me to use anything else. However, the only downside with Material is that the colors sort of clash when I use it with Mocha. With Latte, though, the set works amazingly well. If there was a variation that worked with all four palettes, this would probably be my endgame setup.

Here's an example of what it looks like on Latte:

I saw this in the Discord server, but figured I'd comment on this issue as well, but it looks like someone has started making their own Catppuccin VSCode icons here: https://github.com/prazdevs/catppuccin-vscode-icons

We've since moved the Icon pack over to the org, and I've linked to it in the README with the latest release.
Since Catppuccin now has multiple Extensions in the Marketplace, I've also created an Extension Pack (I'll make it look nicer later today).


Update ci-kubernetes-build-farm.md

golang version fix.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://657b23c27b7b6e2f780b6d29--harness-developer.netlify.app


Fixed and Addressed issue grid Creator - Web Design #339

Fixed and Addressed issue grid Creator - Web Design #339
added files
modules/grids/grid_NUM_Yaswanth_Mulakala.html
modules/grids/images_and_icons/img1.jpg
modules/grids/images_and_icons/img2.jpg
modules/grids/images_and_icons/img3.jpg

Heartfelt Thanks for Your Contribution to Our Open Source Project!
            Dear @ymulakala27,

            I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.

            A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.

            I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.

            Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.

            On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.

            Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.

            Best regards,

            Ntem Kenyor K.
            Lead
            Kwickerhub


After set integration between Jenkins and Zebrunner qps-pipeline branch shows previous version

Steps:

Update qps-infra from 4.8 to 4.9
Verify in managed jenkins that qps-pipeline branch shows 4.9
set integration between Jenkins and Zebrunner
$ ./Stop.sh
Update variables.env
$ docker-compose rm -fv
$ ./start.sh
Verify in managed jenkins that qps-pipeline branch shows 4.9
Actual result: in managed jenkins that qps-pipeline branch shows 4.8

Reverified and can't repro on 4.9


Use new API for the pseudo potential family installation.

@mbercx Do we still need this? https://github.com/aiidalab/aiidalab-docker-stack-demo/blob/768116a37ea8cf42ad903fdb25aed302de8f4627/Dockerfile#L12-L17

@mbercx Do we still need this?

No :). aiida-pseudo downloads the files needed. 👍


Viewports in OHIF V3 can be reduced to zero and unusable small widths

Describe the Bug
OHIF V3 can be resized such that the viewports can be reduced to zero and/or unusably small widths. Consideration should be made as to what is a tolerable small width. Furthermore consideration should be made as to how the OHIF app page behaves once a minimum width is decided upon. For example, should a horizontal scroll bar appear for the entire page once the minimum viewport width is reached and the page contents are too wide for the browser window? Or perhaps maybe just a horizontal scroll bar for the viewport area?
Steps to Reproduce:

Launch OHIF.
Resize the browser window such that its width becomes smaller (i.e. resize to a thinner window).
Notice that the viewports can become an unusable small size and can be reduced to zero width.

The current behavior
See the attached video.
https://user-images.githubusercontent.com/109477394/218104748-862504c0-bd35-4d36-9e2f-4257b3e1bed7.mp4
The expected behavior
Nothing proposed - this issue is being logged so that consideration is made for this.

:warning: Reports we cannot reproduce are at risk of being marked stale and
closed. The more information you can provide, the more likely we are to look
into and address your issue.

Let's revisit when we redo mobile design @dan-rukas

@dan-rukas assigned to take a look at this when we do mobile design and make sure this isn't an issue after the new design


Remove block statements for single-return arrow functions

This PR augments function-to-arrow to also remove block statements from existing single-return arrow functions, which are generated by other things and aren't currently being handled.

Should one line return-assignments stay inside a block?

@GoodForOneFare @fandy I think the codemod we have to handle return assignments should just handle block-less functions; it removes any order dependencies, and makes it more resilient to potentially unexpected decaf/ other transformations.


count number is much less than the one generated by HTSeq

I have been using hisat2-stringtie-DESeq2 pipeline for a while, and then I realized the sum of raw counts from all the genes only accounts for 20% of my mapped reads. That is a pretty low number and then I tried HTSeq and the sum gave me a number close to the number of mapped reads. I do not know the reason and after searching online I do not see comments like my situation. Anybody has a similar experience or knows what happened to my analyses?

Hi, I think, I am experiencing the similar situation: I used stringtie and prepDE to calculate the read counts for genes and HTSeq for the same. The number of reads for stringtie is much lower. It seems like stringtie filters out some reads. If it is the case, what are the criteria for such filtration? If not, what other reasons could be? Thank you!

Hi,
What I did to my data is to map one population to the reference genome generated from another population. I guess there are big genome differences between the two populations, and thus many reads got filtered out, because when I mapped the population to the corresponding reference the results are good. So I ended up using featurecounts (HTSeq is also ok) for my data analysis, which gave me a satisfying results.
Best,
Ning
On Nov 4, 2019, at 01:06, OlgaVT<EMAIL_ADDRESS>wrote:

Hi, I think, I am experiencing the similar situation: I used stringtie and prepDE to calculate the read counts for genes and HTSeq for the same. The number of reads for stringtie is much lower. It seems like stringtie filters out some reads. If it is the case, what are the criteria for such filtration? If not, what other reasons could be? Thank you!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_gpertea_stringtie_issues_223-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAMIUKPVHKNW36JWZ2MCEILLQR7QXRA5CNFSM4HVKQROKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEC6SWVQ-23issuecomment-2D549268310&d=DwMCaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=qstcNNQYmDDRoPOQLFDcVw&m=VuCYHQDRojrI0R1_RM7ZDRsMnZTsJV21iXZTecb-dGA&s=E6R7NfEbVzRsSGvQEEimmv9QOBV1u8BSLZJ9nDsT_48&e=, or unsubscribehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AMIUKPRMGTCBYRDI5YT5JVDQR7QXRANCNFSM4HVKQROA&d=DwMCaQ&c=clK7kQUTWtAVEOVIgvi0NU5BOUHhpN0H8p7CSfnc_gI&r=qstcNNQYmDDRoPOQLFDcVw&m=VuCYHQDRojrI0R1_RM7ZDRsMnZTsJV21iXZTecb-dGA&s=Nhu5m2j4AjB3srqGELKw6p2yF_2QgLOV-nh-Bm-_fWg&e=.


Add parameter validation for ECJPAKE module

Summary: Based on #1643, this PR adds parameter validation for the ECJPAKE module.

Typo spotted by check-names.sh fixed in f0af9f8.

Thanks @mpg for your review, I think I have addressed your comments in 185e516. Please re-review.

@gilles-peskine-arm Thanks for your review, I addressed your feedback. Please re-review.

CI passing, reviewed and approved, therefore labelling as 'ready to merge'.


Directory API - members get 400

The following request in java
directory.members().get(group, email).execute();
will sometimes get response 400 Invalid Input.
Both group and email belong to the same domain.
I was able to reproduce this with API explorer on this page
https://developers.google.com/admin-sdk/directory/v1/reference/members/get
This is not constant issue, sometimes it will not occur at all and 200 response would be returned.
It looks as if there is one (or more than one) faulty instance behind load balancer that returns 400 response.
We started to notice that issue on 31st July 2018.

@netp-dany Thanks for letting us know


allow publishing organization to be changed after the dataset is published

Hello,
We would like to request for this feature as we have some datasets with old publishing organizations that are no longer in use. We would like to transfer the datasets from the old publishing organizations to the ones that we are currently using but we cannot change that at the web interface.
Thanks a lot!
@Antonarctica

There are workarounds here: https://ipt.gbif.org/manual/en/ipt/latest/faq#how-do-i-change-the-publishing-organization-of-my-resource-the-drop-down-on-the-basic-metadata-page-is-disabled
but it may be something we should integrate into the UI.

Would this affect occurrence gbifIDs or they could remain stable after such a change?
Thanks
@abubelinha

@abubelinha
gbifIDs are a combination of datasetKey (UUID given by the registry) and occurrenceID, so that the publishing organization doesn't affect gbifIDs.

@MattBlissett Thank you very much! It will be super helpful if it can be integrated into the UI, we will very much appreciate it!
@abubelinha Thank you for asking!
@muttcg Thank you for clarifying - good to know!

In new IPTs publishing organization can be directly changed at the overview page. I'll update the manual


[Android] Use InvokeCommand and support nullable/optional for commands

Problem

Android clusters code lacks nullable/optional support

Change overview

Refactor to use InvokeCommand API
Support nullable/optional command args and command responses types

Testing

Manually call TestCluster

fast track: platform specific change made by platform owner


fix project paths change

fix #26 using onDidChangePaths event

pls merge. to fix issues with atom project plus


no 'local-forage-detect-blob-support' object store

Is it possible to not have an object store 'local-forage-detect-blob-support' in the database.
The reason is that we want to use localForage to put key value items in the object store. But we will probably be using idb to do more complex management. So for this we would like it to be as clean as possible.

If I look at the code it doesn't seem to be possible. It would be cool to have a localforage option to ommit this so the check is not done...

That object is required to determine support for blob storage so is required. Generally speaking we don't really "support" accessing the localForage database directly—it's possible but not really an intended use-case as localForage serialises values depending on support, stores things like the blob support bit, etc.
localForage does its best to limit how much data it stores but unfortunately I don't think it's on the roadmap to remove the little bits we do store. That said you should be able to check through the code and find any special stores we create and work around them in your own code 😄


MuJoCo header and bin path override fails

I tried to install using the following commands but the override does not seem to work. It would be great if there are working examples for these options:
mujoco lib location: $HOME/.mujoco/mujoco200
installation script:
git clone https://github.com/deepmind/dm_control.git

export MJLIB_PATH=~/.mujoco/mujoco200/bin
cd dm_control
pip install -e . --headers-dir=~/.mujoco/mujoco200/include

I have tried the following and non works:
python setup.py install --headers-dir=~/.mujoco/mujoco200/include

The only hit of the headers-dir in the code base are here: setup.py#L80
But they don't show up in the commands that are available in the setup.py file as if we run python setup.py -h shows no such options.

I suspect that this is because the ~ is not expanded by Bash in the --headers-dir= argument. Try --headers-dir=${HOME}/.mujoco/mujoco200/includes instead.
Also, as of yesterday we have uploaded dm_control to PyPI so you should be able to pip install dm_control as well. If you do this then there's no need to configure MuJoCo's headers path. (You will still need to export MJLIB_PATH though.)

thanks Saran, that was not the issue because ~ is interpolated by bash.
➜  ~ echo ~
/Users/ge

After pip install successfully, running dm_control.suite gives this error
OSError: Cannot find MuJoCo library at <my home>/.mujoco/mujoco200/bin.

whereas the files are infact there.
Merry Christmas!

The problem with bash expansion is that doesn't work if the argv doesn't begin with ~.
--headers-dir=~/foo```

This is a bug on our side, we should be applying `os.path.expanduser`to the path. I assume that you've got it installing successfully from PyPI in the end?

The `MJLIB_PATH` environment variable should be the full path of the actual `.so` file itself, not the directory containing the file. (See https://github.com/deepmind/dm_control/blob/master/dm_control/mujoco/wrapper/util.py#L123)

Thanks Saran! just resolved the second problem and indeed it is linked to this issue #6
The full installation guide that works is here:

First set the environments.

 31 # <<< conda init <<<
 32 export MUJOCO_GL="glfw"
 33 export MJLIB_PATH=$HOME/.mujoco/mujoco200/bin/libmujoco200.dylib
 34 export MJKEY_PATH=$HOME/.mujoco/mujoco200/mjkey.txt
 35 export LD_LIBRARY_PATH=$HOME/.mujoco/mujoco200/bin:$LD_LIBRARY_PATH
 36 export MUJOCO_PY_MJPRO_PATH=$HOME/.mujoco/mujoco200/
 37 export MUJOCO_PY_MJKEY_PATH=$HOME/.mujoco/mujoco200/mjkey.txt

install via pip: pip install dm_control. Not that installing with pip install -e . does not work because if the issue with dynamically generated ctypes #6

the MUJOCO_GL is important for mac os, otherwise you are going to run into a viewer error.

the MUJOCO_PATH is different for mac os vs linux, because the extension for the library files are difference.

In new OSX Catalina, you need to allow the library binary in the securities setting.

Thanks for the confirmation: I'll update the README to clarify that the MJLIB_PATH should point to the file rather than the directory.
Merry Christmas!

Thanks for your quick response :)

I done every but when I am trying to run to code example given with code name with Quick Start with Control suite, following error happen
dm_control.mujoco.wrapper.core.Error:Could not Register licence.

Now when I install from git installation fails. I think this is related to #6.
pip install git+git://github.com/deepmind/dm_control.git

Gives the following error:
.mujoco/mujoco200_macos/include/mjdata.h' does not exist.

Installation from PyPI works.


v1.19 buffer by default?

Sorry, not a feature request, but rather just a question.
Is the buffer introduced in v1.19 enabled by default?
Thanks.

No, you have to explicitly pass --display-buffer=XXX, where XXX is the buffer duration in milliseconds.

As not a programmer guy myself,
when you say “pass”, do you mean create a shortcut like this?:
"D:\scrcpy\scrcpy-noconsole.exe" --display-buffer=XXX

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#command-line-on-windows

Sorry just one more thing,
when I'm editing my copy of scrcpy-noconsole.vbs
where do I add the arguments?
strCommand = "cmd /c scrcpy.exe"
For Each Arg In WScript.Arguments
strCommand = strCommand & " """ & replace(Arg, """", """""""""") & """"
Next
CreateObject("Wscript.Shell").Run strCommand, 0, false


Flow errors pre existing when initiating react-native

I have created a react-native project from scratch.
react-native init AwesomeProject
cd AwesomeProject
react-native run-android

And when running flow from the command line I got the following 11 errors most of them related to the react packager.
node_modules/react-native/Libraries/Performance/Systrace.js:216
216:   require.Systrace = Systrace;
               ^^^^^^^^ property `Systrace`. Property not found in
216:   require.Systrace = Systrace;
       ^^^^^^^ object type

node_modules/react-native/local-cli/bundle/output/unbundle/as-indexed-file.js:74
 74: //$FlowIssue #14640206
     ^^^^^^^^^^^^^^^^^^^^^^ Error suppressing comment. Unused suppression

node_modules/react-native/local-cli/bundle/output/unbundle/as-indexed-file.js:116
116:   //$FlowIssue #14640206
       ^^^^^^^^^^^^^^^^^^^^^^ Error suppressing comment. Unused suppression

node_modules/react-native/packager/react-packager/src/Server/index.js:267
267:               if (deps.files.has(filePath)) {
                            ^^^^^ property `files`. Property cannot be accessed on possibly undefined value
267:               if (deps.files.has(filePath)) {
                       ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:268
268:                 deps.outdated.add(filePath);
                          ^^^^^^^^ property `outdated`. Property cannot be accessed on possibly undefined value
268:                 deps.outdated.add(filePath);
                     ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:560
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                    ^^^^^^^^^^^^^^^ property `dependencyPairs`. Property cannot be accessed on possibly undefined value
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                                                   ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:560
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                     ^^^^^ property `files`. Property cannot be accessed on possibly undefined value
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                                                   ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:560
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                            ^^^^^^^^^ property `idToIndex`. Property cannot be accessed on possibly undefined value
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                                                   ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:560
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                                       ^^^^^^^^ property `outdated`. Property cannot be accessed on possibly undefined value
560:         const {dependencyPairs, files, idToIndex, outdated} = deps;
                                                                   ^^^^ undefined

node_modules/react-native/packager/react-packager/src/Server/index.js:573
573:           deps.outdated = new Set();
                    ^^^^^^^^ property `outdated`. Property cannot be assigned on possibly undefined value
573:           deps.outdated = new Set();
               ^^^^ undefined

node_modules/react-native/packager/react-packager/src/node-haste/ModuleCache.js:117
117:       if (this._packageCache[packagePath]) {
                                  ^^^^^^^^^^^ undefined. This type is incompatible with
 36:   _packageCache: {[filePath: string]: Package};
                                  ^^^^^^ string

Found 11 errors

react-native-cli: 2.0.1
react-native: 0.40.0
flowtype: 0.37.4

I'm also seeing something similar after upgrading to 0.41.0-rc.0
node_modules/react-native/local-cli/bundle/output/unbundle/as-indexed-file.js:74
 74: const nullByteBuffer: Buffer = new Buffer(1).fill(0);
                                    ^^^^^^^^^^^^^^^^^^^^^ undefined. This type is incompatible with
 74: const nullByteBuffer: Buffer = new Buffer(1).fill(0);
                           ^^^^^^ Buffer

node_modules/react-native/local-cli/bundle/output/unbundle/as-indexed-file.js:115
115:   const table: Buffer = new Buffer(entryOffset(numEntries)).fill(0);
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ undefined. This type is incompatible with
115:   const table: Buffer = new Buffer(entryOffset(numEntries)).fill(0);
                    ^^^^^^ Buffer

+1

I don't know what the issue is but I think we need some eyes on the .flowconfig file:
For one, the file used to contain:
# Ignore unexpected extra @providesModule
.*/node_modules/commoner/test/source/widget/share.js

and now the same comment has this following it:
; Ignore unexpected extra "@providesModule"
.*/node_modules/.*/node_modules/fbjs/.*

Is that comment still accurate?
should the 2nd line have two node_modules?

In any case, changing it to .*/node_modules/fbjs/.* hasn't produced any noticeable change.
In the meantime, I added this to the [ignore] section to suppress all my node_module errors:
.*/node_modules/.*

currently 135 errors with

Flow 0.39.0
react-native 0.41.2

is there any way to suppress them?

Ongoing discussion in issue #11655 also.


Error: Unable to resolve module react-native/Libraries/Animated/src/Easing from D:\working_workspace\waterproof_project\6.code\newWaterproofApp\node_modules\teaset\components\ListRow\TouchableOpacity.js: react-native/Libraries/Animated/src/Easing could not be found within the project or in these directories:

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.

Here is the diff that solved my problem:
diff --git a/node_modules/teaset/components/ListRow/TouchableOpacity.js b/node_modules/teaset/components/ListRow/TouchableOpacity.js
index 8fc32cf..a4f1fb3 100644
--- a/node_modules/teaset/components/ListRow/TouchableOpacity.js
+++ b/node_modules/teaset/components/ListRow/TouchableOpacity.js
@@ -4,7 +4,7 @@ import React, {Component} from 'react';
 import PropTypes from 'prop-types';

 import {Platform, TouchableWithoutFeedback, Animated, ViewPropTypes} from 'react-native';
-import Easing from 'react-native/Libraries/Animated/src/Easing';
+import Easing from 'react-native/Libraries/Animated/Easing';
 import flattenStyle from 'react-native/Libraries/StyleSheet/flattenStyle';

 if (Platform.constants.reactNativeVersion.major === 0 && Platform.constants.reactNativeVersion.minor < 62) {

This issue body was partially generated by patch-package.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

I faced same issue,
Hope to get right answer as soon as possible.
Thanks.

The same problem

Add some configuration to babel.config.js
module.exports = {
  presets: ['module:metro-react-native-babel-preset'],
  plugins: [
    [
      require.resolve('babel-plugin-module-resolver'),
      {
        extensions: ['.js', '.png'],
        alias: {
          //
          'react-native/Libraries/Animated/src/Easing': 'react-native/Libraries/Animated/Easing',
        },
      },
    ]
  ],
};


new version error

I was using old version of ngx-soap..,everything was running fine.!! I just updated my angular project with "npm update"..,the ngx-soap module got updated too..now it is<EMAIL_ADDRESS>now I have so many errors..!!
How can I install my old version..?? any help regarding this ??

Just install the version you had before (npm install ngx-soap@) or follow the code changes as per readme.


Enhance task_output_view and workflow

Added some minor tweaks to the formatting of the logs and to the pipeline. Would be fine for me to revert the changes for the pipeline. The modifications to the task_output_view improves readability of the log files. Maybe the colors can be adapted for light mode, but in general this improves the readability a lot (using dark mode, so i'm not sure about that):

enhance task output view by colors and scroll instead of linebreaks
pushing artifacts to action summary

Thiis pr change whole line color. if a line has multi color or just some keyword is colored, will not be expected.

lgtm

Oh Thanks. I tested it with my usecase and it looked good for me too.  Maybe there can be some max line length in the future, but my logs are looking much better low.


No proposal wrench appears in Pharo snippet after package and class deleted

Create a new class and package in a Pharo snippet using the proposal wrench. Delete the new package (and the contained class). Note that the proposal wrench doesn't appear for the no longer existing class, in the Pharo snippet.
Windows 11, GT 0.8.2498

The snippet is only updated on editing at present time. We won't address this soon.

We won't address this soon.


Handle monetary amounts correctly

Currently f64 is used which is not ideal.
Look for keywords "price", "paid", "amount", "rate"... and everywhere "f64" is used.
Either:

Use String for prices
Convert+operate on integers (cents)
Add dependency on some decimal crate
Use serde_json::Number

Added rust_decimal dependency as it makes the most sense and is close to .NET


fix: display group if indexer unavailable

Closes: #134
This pull request updates how we handle the error from the graphql endpoint when fetching historical proposals. If the graphql endpoint is unavailable, group pages will load successfully without historical proposals and the error from the graphql request will be logged to the console rather than being thrown (preventing the page from being viewed without historical proposals).

LGTM! A tangential thought I have while looking at this code is that maybe we could open issue for integrating sentry as we have it for regen-web. I.e. In the try-catch block we have here now, we could also raise a sentry alert, which would give us notifications etc. and also sentry would capture any other unhandled exceptions as well

Definitely worth considering. Opened #144 for tracking. :+1:


Add DRF Benchmark

Awsome, thank you. What are the results?

trafaret   best=1.391s, avg=1.402s, stdev=0.013s
drf        best=6.006s, avg=6.153s, stdev=0.183s
pydantic   best=0.678s, avg=0.694s, stdev=0.014s
:)

wow, that's not great of drf. But having read its code in the past it, it does masses of checking that it's configured right and calls out to slow django standard methods.
I'll have a play with it later and merge.

ok, thanks

Looks good. I'm shocked by the performance gap.
I'll try to add a few more popular packages and then perhaps add a graph of relative performance to the docs.

This PR is a top result for a google search on "Pydantic + DRF" >> perhaps we can create a ticket to describe the benchmark on Pydantic's documentation? Additionally, it would be helpful to have some guidance on how the frameworks compare to each other.


Can I use an external camera with windows hello for business and how reliable it is?

No. Windows 10 currently only supports one Windows Hello for Business camera and does not fluidly switch to an external camera when the computer is docked with the lid closed. The product group is aware of this and is investigating this topic further.

Hello @rahulkj30 and thanks for posting feedback. This does not appear to be a documentation issue and it looks like it's resolved, so I am going to close this for now.


"fake" appears on LHN after flagging a content

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Start a new conversation from account A to account B (There should be no comment before the comment that is about to be flagged)
Flag the message sent from account B as either "harassment" or "Assault"
Observe the LHN

Expected Result:
"fake" should not appear on the LHN
Actual Result:
"fake" appears on LHN
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.29-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/1b10afc7-9f30-4a66-adc9-8e5a03f92e6f
https://github.com/Expensify/App/assets/43996225/21570101-bfa8-4bc8-b9a9-7ca7877bd2b8
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686941708297639
View all open jobs on GitHub

Thanks for the quick work Aldo. Let me know if I can help in any way.

During the PR https://github.com/Expensify/App/pull/20949 review we found quite a few other cases that were not working, see for more information:

Missing offline behaviour: https://github.com/Expensify/App/pull/20949#issuecomment-1595474711
Groups/Tasks not working correctly: https://github.com/Expensify/App/pull/20949#issuecomment-1595483433, https://github.com/Expensify/App/pull/20949#issuecomment-1595486497
Wrong comment author for last message (LHN): https://github.com/Expensify/App/pull/20949#issuecomment-1597612599
Wrong attachment last message: https://github.com/Expensify/App/pull/20949#issuecomment-1597674629
Flagged comment shows in LHN if you don't open the report loading report actions: https://github.com/Expensify/App/pull/20949#issuecomment-1597691483

Slack conversation: https://expensify.slack.com/archives/C03TQ48KC/p1687203338251969
Current plan to fix it: https://expensify.slack.com/archives/C07J32337/p1687204905234929?thread_ts=1687190085.248999&cid=C07J32337

FYI I also see __fake__ appear in LHN after creating a new group with a user I don't know yet / doesn't have a real account yet

Found while testing steps from https://github.com/Expensify/App/issues/21075

Gonna remove the blocker label as per this comment

To finish this, the following PR's will be needed:

Auth: https://github.com/Expensify/Auth/pull/8170
Web: https://github.com/Expensify/Web-Expensify/pull/37910
App: https://github.com/Expensify/App/pull/20949

Made backend PRs ready for review.

Waiting for the timings here to start merging: https://github.com/Expensify/Expensify/issues/294240

@aldo-expensify just beware that we're on a new feature freeze and we shouldn't merge anything risky until Jun 29th

ok!, thanks for reminding me :)

Not overdue, just on hold for the feature freeze.

I merged the Auth PR, after it gets deployed, I'll merge Web-E, and after Web-E gets deployed, we can tell the C+ to retest the App PR.

I believe this will also be fixed as a result of the PR: https://github.com/Expensify/App/issues/22386

I just update the Web-E PR and made it ready for review again: https://github.com/Expensify/Web-Expensify/pull/37910

I believe this will also be fixed as a result of the PR: https://github.com/Expensify/App/issues/22386

Maybe! I think we can check after the PRs for this issue have been merged

Hey guys, this issue has been fixed but payment has not been proceed for it. @lschurr @aldo-expensify @sakluger

True, I retested and this is working now:

@sakluger we have to handle the reporting bonus for @Nathan-Mulugeta

Bump @sakluger

Bump @sakluger again 😄

Sorry for the delay, just sent an offer to @Nathan-Mulugeta on Upwork.

Switching to a daily to reflect the priority.

Thanks @sakluger , just accepted the offer.

All paid up! Sorry again for the delay on payment @Nathan-Mulugeta.


python-tools error

python-tools experienced an unexpected exit.
Note: No error code generated just message above.

I got a similar one, with "exit with code 9009, signal null".


Function that finds the first non-missing element in a vector

Introduction
I would be good having in the module Explorer.Series a function similar to dplyr's coalesce or postgreSQL's coalesce.
In Pandas there's a function called DataFrame.combine_first that do something similar, but I'm not sure if it achieves the same results as the ones described above.
What coalesce's do is:
Having a pair of vectors of the same length, merge them, keeping the first value of each side of the pair that is not a :nil
Examples
iex> s1 = Explorer.Series.from_list([1, 2, 3, nil])
iex> s2 = Explorer.Series.from_list[nil, nil, nil, 4])

iex> Explorer.Series.coalesce(s1, s2)
#Explorer.Series<
  integer[3]
  [1, 2, 3, 4]
>

I think we can use Series::zip_with. Not what I think of when I think of zip_with -- more like vectorised if_else in dplyr. So given s1 and s2 it would be: s1.zip_with(s1.is_null(), s2).

Hmm, that looks like a fun one, but I'm currently busy doing #190.
What do you think of tagging this as a "Good First Issue"? It doesn't look like a hard one -- and I can be available to help, if needed.


Shwetha’s feedback to make ai, and main menu more distinguiable. Add exit, q (then ask for confirmation) to quit the cli application.

Shwetha’s feedback to make ai, and main menu more distinguiable. Add exit, q (then ask for confirmation) to quit the cli application.

addressed with https://github.com/VOLTTRON/dnp3-python/pull/39


External menu links are disfunctional

When clicking on governance or bancor x nothing happens

Once the governance item is clicked, other menu items also stop functioning

fixed, deploying to staging now.


Minor pyproject.toml fixes

requires-python = ">=3.8,<=3.11" doesn't allow the package to be installed with Python 3.11.3 for some reason.
fixed deps list
added missing deps
added LANDMark dep install from GitHub

@jrudar this PR is related to https://github.com/jrudar/LANDMark/pull/3 so that I could get TreeOrdination installed on my system with Python 3.11.3


WIP: add reporting

This likely not in its final, desired form, but I wanted to start thinking about how to report results. This PR generates tables that can be rendered in Markdown or HTML. Currently this is done via calling the test script as a regular python script, but that was just done for expediency and can be changed to some other desired way of generating these.
A question is how to most usefully present results. Currently, each format (zarr v2, zarr v3, N5) has a separate table where rows correspond to (writer, codec) and columns correspond to the reader with check marks indicating success and red X indicating failures. The Markdown output renders as follows (with option to avoid the emoji check/x marks if printing to the terminal):
zarr Results

z5py
zarr

js, blosc
:heavy_check_mark:
:heavy_check_mark:

js, gzip
:heavy_check_mark:
:heavy_check_mark:

js, raw
:heavy_check_mark:
:heavy_check_mark:

js, zlib
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, blosc
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, gzip
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, raw
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, zlib
:heavy_check_mark:
:heavy_check_mark:

z5py, blosc
:heavy_check_mark:
:heavy_check_mark:

z5py, gzip
:heavy_check_mark:
:heavy_check_mark:

z5py, raw
:heavy_check_mark:
:heavy_check_mark:

z5py, zlib
:heavy_check_mark:
:heavy_check_mark:

zarr, blosc
:heavy_check_mark:
:heavy_check_mark:

zarr, gzip
:heavy_check_mark:
:heavy_check_mark:

zarr, raw
:heavy_check_mark:
:heavy_check_mark:

zarr, zlib
:heavy_check_mark:
:heavy_check_mark:

N5 Results

pyn5
z5py
zarr

n5-java, bzip2
:heavy_check_mark:
----
----

n5-java, gzip
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

n5-java, raw
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

pyn5, bzip2
:heavy_check_mark:
----
----

pyn5, gzip
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

pyn5, raw
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

z5py, gzip
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

z5py, raw
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

zarr, gzip
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

zarr, raw
:heavy_check_mark:
:heavy_check_mark:
:heavy_check_mark:

zarr-v3 Results

zarrita

zarrita, blosc
:heavy_check_mark:

zarrita, gzip
:heavy_check_mark:

zarrita, raw
:heavy_check_mark:

zarrita, zlib
:heavy_check_mark:

An example of the table appearance when there is an error (corresponding to the error case mentioned in #29):
zarr Results

z5py
zarr

js, blosc
:heavy_check_mark:
:heavy_check_mark:

js, gzip
:heavy_check_mark:
:heavy_check_mark:

js, raw
:heavy_check_mark:
:heavy_check_mark:

js, zlib
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, blosc
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, gzip
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, raw
:heavy_check_mark:
:heavy_check_mark:

xtensor_zarr, zlib
:heavy_check_mark:
:heavy_check_mark:

z5py, blosc
:heavy_check_mark:
:heavy_check_mark:

z5py, gzip
:heavy_check_mark:
:x:: OSError: Not a gzipped file (b'x^')

z5py, raw
:heavy_check_mark:
:heavy_check_mark:

z5py, zlib
:heavy_check_mark:
:heavy_check_mark:

zarr, blosc
:heavy_check_mark:
:heavy_check_mark:

zarr, gzip
:heavy_check_mark:
:heavy_check_mark:

zarr, raw
:heavy_check_mark:
:heavy_check_mark:

zarr, zlib
:heavy_check_mark:
:heavy_check_mark:

Also, I forgot to mention that the reports generated currently get stored as an artifact via Github Actions.

As mentioned on the community call, these rock. I could certainly running the full Writer X Reader matrix and then having a list of known failures.

This is awesome!
I will have a closer look at the changes tomorrow to give some feedback.

The segfault in the most recent commit was debugged in #31 and seems to be a problem with xtensor 0.23.5 and/or xtensor-io 0.12.5.

Let me know if you want to implement anything else before merging, @grlee77.

I think it is okay to merge what is here and we can update as needed in follow up PRs.

But having a link to the md or html with the table would already be nice.

I am not very experienced with Github Actions, but my understanding is that the artifacts always get zipped. I will have to think about how we can link to the result from the README

I am not very experienced with Github Actions, but my understanding is that the artifacts always get zipped. I will have to think about how we can link to the result from the README

Ok, let's look into this in a follow up PR then. I will merge despite the failing tests, #32 should fix this.
Thanks a lot for this!

:tada: Thoughts on having this output go to http://zarr-developers.github.io/zarr_implementations for the main branch?


Changes in subcircuit  overflowing vertically

Describe the changes you have made in this PR -
I have added the jquery to set height and width for the subcircuit dialog and removed the  class which was not working in the past pr i made
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.223% when pulling fb417d9c4fe38fef4d05f450d410a7ef9e62c01e on ryuk156:changes-in-sub-circuit-vertically-overflowing into b14f86cbe15aa1b39d0fe2569eff463c8003d8b4 on CircuitVerse:master.


Set the cursor to the place where a user hits tab key

PR Summary
Addressed the issue https://github.com/PowerShell/PowerShell/issues/7022
For the PSReadLine part of issue, see PR https://github.com/lzybkr/PSReadLine/pull/708
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@iSazonov SHiPS is written in C#. It's a PowerShell provider. it calls into PowerShell scripts for fetching data. For example, Get-AzureRMWebApp.  Before calling PowerShell script from C#, it creates ProgressRecord and call WriteProgress, then Get-AzureRMWebApp get called which generates a few more write-progress from it's own cmdlet. When the cmdlet completes, SHiPS gets the data returned from Get-AzureRMWebApp. It then calls WriteProgress with ProgressRecordType.Completed from herenner.cs#L85.

You may write an app in C#, call WriteProgress(), then call the above PowerShell script from C#.
Or you may try the following:
On your Linux, install-module azurerm.netcore
install-module azurepsdrive
Login-AzureRmAccount
new-psdrive -name az -psprovider SHiPS -root 'Azurepsdrive#Azure'
cd az:
Get- then tab
you will the problem
cd az:\AutomationTeam\WebApps
Get- tab, you will the nested progress issue.


Test Discovery Issue With Class Constructors

Hey, this may be related to #576, and could be a Rider issue.
When using source an IClassConstructor to dependency inject stuff like in the example in the docs. Injecting a service that is not registered in the class constructor breaks the test discovery.
This is odd behavior and I only realized it was caused by the class not being in the service collection while making this example. This could be nice behavior if an error was raised at compile time.
For the example bellow if the .AddScoped<Class1>() line is commented out my discovery breaks.
Test classes;
namespace TUnit.Examples;

public class Class1 : IClass1
{
    public int Value => 1;
}

public interface IClass1
{
    int Value { get; }
}

[ClassConstructor<DependencyInjectionClassConstructor>]
public class ExampleTestFixture(Class1 c1)
{
    [Test]
    public Task PassTest()
    {
        return Task.CompletedTask;
    }
}

Class constructor;
using System.Diagnostics.CodeAnalysis;
using Microsoft.Extensions.DependencyInjection;
using TUnit.Core.Interfaces;

namespace TUnit.Examples;

public class DependencyInjectionClassConstructor : IClassConstructor, ITestEndEvent
{
    private IServiceProvider? _serviceProvider;
    private AsyncServiceScope _scope;

    public T Create<[DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors)] T>() where T : class
    {
        _serviceProvider = CreateServiceProvider();
        _scope = _serviceProvider.CreateAsyncScope();

        return ActivatorUtilities.GetServiceOrCreateInstance<T>(_scope.ServiceProvider);
    }

    public ValueTask OnTestEnd(TestContext testContext)
    {
        return _scope.DisposeAsync();
    }

    private static IServiceProvider CreateServiceProvider()
    {
        return new ServiceCollection()
               .AddScoped<Class1>() //Commenting this line out removes the test from the discovery
               .BuildServiceProvider();
    }
}

Yeah it was due to the thrown exception.
This should report better and not break test discovery in v0.1.1097

Cool thanks for getting back quickly.
Also really liking the library so far got some amazing features 👏

Thanks really appreciate it!


Forgot password auth0

Hello, first of all thank you for creating this awesome library.
I have a website where I implemented a loginWithRedirect using a custom login form. I want to add a forgot password page (where the users puts the email and he gets an email with a link after that). I'm not sure how to do that using this library, from my understanding I need to use a "changePassword" method, but it doesn't seem to be available.
Do I need to use auth-0.js for this? It's a bit confusing.

@ovidiumihaibelciug How did you solve this?

Hey @danactive,
I called the api endpoint directly
This is the endpoint:
POST https://${YOUR_AUTH0_DOMAIN}/dbconnections/change_password
and on that endpoint you have to send this data
{        client_id: YOUR_AUTH0_CLIENTID,       email: "the email from the input",       connection: "Username-Password-Authentication",       }
 And then the user should receive an email with a link to a page where he can reset the password.


Build image hangs on interactive prompt

AWS ParallelCluster version: 3.1.4
Bug description and how to reproduce:
Build image hangs on ubuntu because install of openssh-server, debconf brings up interactive prompt. It happens when UpdateOsPackages: True or UpdateOsPackages: False
Build command:
pcluster build-image --image-id XXX --image-configuration XXX.yaml --region XXX

Build config file:
Region: XXX
Build:
  InstanceType: m6g.large
  SubnetId: XXX
  ParentImage: ami-XXX
  Iam:
    InstanceRole: arn:aws:iam::XXXX
  Components:
    - Type: arn
      Value: arn:aws:imagebuilder:XXX
  UpdateOsPackages:
    Enabled: False
  SecurityGroupIds:
    - sg-XXX
Image:
  Name: cluster-ami
  RootVolume:
    Size: 35
    Encrypted: False

Image build log where process hangs:
    {
      "message": "Stdout: Setting up openssh-server (1:8.2p1-4ubuntu0.5) ...",
      "timestamp": "2022-05-23T17:16:52.804Z"
    },
    {
      "message": "Stdout: debconf: unable to initialize frontend: Dialog",
      "timestamp": "2022-05-23T17:16:53.076Z"
    },
    {
      "message": "Stdout: debconf: (TERM is not set, so the dialog frontend is not usable.)",
      "timestamp": "2022-05-23T17:16:53.076Z"
    },
    {
      "message": "Stdout: debconf: falling back to frontend: Readline",
      "timestamp": "2022-05-23T17:16:53.076Z"
    },
    {
      "message": "Stdout: Configuring openssh-server",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: --------------------------",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: ",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: A new version (/tmp/filefswXVy) of configuration file /etc/ssh/sshd_config is ",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: available, but the version installed currently has been locally modified.",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: ",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   1. install the package maintainer's version",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   2. keep the local version currently installed",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   3. show the differences between the versions",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   4. show a side-by-side difference between the versions",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   5. show a 3-way difference between available versions",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   6. do a 3-way merge between available versions",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout:   7. start a new shell to examine the situation",
      "timestamp": "2022-05-23T17:16:53.180Z"
    },
    {
      "message": "Stdout: ",
      "timestamp": "2022-05-23T17:16:53.180Z"
    }

Hi @popgendad
the AMI you're using as ParentImage is a public one?  If so, can you share the AMI id? In this way we can try to reproduce.
If the ParentImage is not public could you please execute the following commands on it:
apt-get -V -s upgrade openssh-server

apt show openssh-server -a

Thanks

Hi @popgendad
You shouldn't use a ParallelCluster AMI as ParentImage.
There are two ways to create Custom AMIs for ParallelCluster:

Build a Custom AWS ParallelCluster AMI
Modify an AWS ParallelCluster AMI

If you're using the 1st approach (pcluster build-image) you should use your personal AMI as ParentImage, not a ParallelCluster AMI, because the scope of the build-image process is to add ParallelCluster stuff on top of your AMI (with just your your software).
If you're using the 2nd approach, and you created a custom AMI starting from ami-05e7fea4bacc5792d and adding your software on top of it, you don't need to execute the build-image command, you can simply add it as CustomAmi in the cluster configuration.
Then let me also add that if your customization is a simple and fast one, the recommended approach is to use Custom Bootstrap Actions rather than building a custom AMI.
Could you please clarify which process are you following?
Are you passing the ami-05e7fea4bacc5792d as ParentImage?
Enrico

Hi @enrico-usai ok my bad, I had been using the parallelcluster AMIs with EC2-Image-Builder, so I see now how pcluster image-build will just re-install and re-configure the packages.


"preferenceId is required for starting MP Checkout"

I copy the example. Replace TestIDs and appear that:
"preferenceId is required for starting MP Checkout"

@maueli have you logged the preferenceId value that you're passing to the checkout? Maybe it's undefined.

I wonder if you could explain me how to set "server-side" react-native to run your example and get the "preferenceId".
Thanks!

The example has itself a README explain the mandatory steps in order to run the application correctly.
https://github.com/BlackBoxVision/react-native-mercadopago-px/tree/master/example
Also, we've a section of pre-requisites related to MP account setup:
https://github.com/BlackBoxVision/react-native-mercadopago-px#pre-requisites

Ok, I know I know. How can I get the preferenceid in a basic react-native App?. Do I have to install express to run the server-side?
How/where can I run "curl" ?. Could you give me a clue or a basic example of how can I run server-side and get preferenceid easily?
Thanksssss

In the example we go directly through MP Preference Creation API using the Access Token obtained from your account. That's a matter of simplicity in terms of the example.
If you want to add a server, you can check the px-nodejs repo: https://github.com/mercadopago/sdk-nodejs#-getting-started
I'll add a reference to the docs, but that's not related to our library setup.


Add mdl & yamllint and run delivery in circleci

Description
Add mdl & yamllint and run delivery in circleci
Issues Resolved
[List any existing issues this PR resolves]
Check List

[ ] All tests pass. See https://github.com/sous-chefs/apache2/blob/master/TESTING.md
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

This PR has been superseded.


nodal concentrated load and spring damper elements are not written to the input files correctly

Question regarding the right way is outstanding

new process should be added to the projectparameters.json which looks like:
{"python_module": "set_nodal_parameters_process",
"kratos_module": "StemApplication",
"process_name": "SetNodalParametersProcess",
"Parameters": {
"model_part_name":"choose_model_part_name"}
}


fix: hpa and variables

Closes #49
✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

thanks!


Stream not displayed when communicating over https.

built chatbot-ui on Docker.
After that, I published it externally with Nginx reverse proxy.

Received reverse proxy (own domain) and assigned to <IP_ADDRESS>:3000.

-With http communication, streaming display = 1-2 words at a time.
However,
-when using https (443), it is not displayed streaming, but is displayed after the full text is retrieved.
What settings are needed? Please let me know.

Streaming works fine for me over HTTPS using Caddy.
I used the prebuilt Docker container and Caddy as a proxy.  Just standard configuration for both.

@adamshand
Thank you!!
I changed the proxy to Caddy as you suggested and now I can view the stream.

@decolygs You just need to add these configuration by Nginx:
                        proxy_buffering off;
                        proxy_http_version 1.1;
                        chunked_transfer_encoding on;
                        proxy_pass http://localhost:3000/;


Housing units affordable to LI households

Description: Units annually priced lower than 30% of 51-80% AMI
Logic:
Source(s): NYC HVS
Year(s):

[ ] 2002
[ ] 2011
[ ] 2017

Geographies:

[ ] PUMA
[ ] Borough
[ ] City

Race and ethnicity breakdown: No
Denominator:
SME: HPD
Questions:
Roadblocks:

Changes
Source: ACS PUMS
Years: 2015-19
SME: Pop Div


Can we Ping a store?

Thanks for the great package. I just started using it and it'a exactly what I needed.
Is there an easy/idiomatic way to ping a particular cache store to ensure credentials and addresses are valid?
Many thanks.

Hi @ainsleyclark,
You have full access to the store objects (redis for instance), so you can use the already available ping command they provides.
I'm not sure I've provided you a good answer so please be free to elaborate more on your question if you need some other answers.

Hey thanks for your reply.
I figured out that we could the store, but a global Ping method on the StoreInterface I think would be really useful.
For go-cache it could just return nil and so fourth.
Many thanks.


simplify Dataset class

What I did is to move _InnerDataset.int into Dataset as _lazy_init, turn every _InnerDataset(...) into _lazy_init(...).

Hi @wxchan, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

I did not change any Dataset api.

@wxchan I found reset_parameters callback is not effect for cv.
Can you add support for this?
also can check parameter in the callback.

@wxchan Currently implementation of reset parameters is not efficient too.
https://github.com/Microsoft/LightGBM/blob/master/python-package/lightgbm/callback.py#L122-L130
it will reset many times if passed multi parameters.
I think you can join these key-value pair and call reset_parameters only one time.

@guolinke reset_parameters callback is not effect when callbacks=[reset_parameters(...)]? or you mean add another argument?

@wxchan the reason is we only reset parameters for env.model``, not for env.cvfolds```.

@guolinke can you see my comments at https://github.com/Microsoft/LightGBM/commit/65d55194bc984a850f0694653cd2f3f32562f5cc? I saw you reset HEAD.
self._is_construct came from old method self._is_construct(), I think it should not appear at set_label etc. now.

@wxchan I think using an additional variable to store _is_constructed less maintainable.
I change it back to a function now.


the selector added through soul-admin is not checked

Describe the bug

The survival of the divide selector added through soul-admin is not checked.
UpstreamCheckService#check
Environment

Soul version(s): [2.2.1]

Steps to reproduce

start soul-admin, soul-bootstrap
add divide selector  that cannot be connected through soul-admin
GET http://localhost:9195/http/order/findById?id=3

Expected behavior

The selector data table is updated and the selector update event is published.
Debug logs

Click to expand
replace this line with your debug logs

pr: https://github.com/dromara/soul/pull/991

pr: https://github.com/dromara/soul/pull/991


Add TensorDictKeysView

This PR adds a new object _TensorDictKeysView which is returned when accessing tensordict.keys() and holds a reference to the original tensordict. This allows us to modify the behaviour of key in tensordict.keys() and use tuples for nested key lookups. For example
import torch
from tensordict import TensorDict

td = TensorDict(
    {"a": TensorDict({"b": torch.rand(1, 2)}, [1, 2]), "c": torch.rand(1)},
    [1],
)

assert "a" in td.keys()
assert ("a",) in td.keys()
assert ("a", "b") in td.keys()
assert ("a", "c") not in td.keys()

This PR also disallows checks like item in tensordict except in the case of tensordict in lazy_stacked_tensordict where it will check if the tensordict is one of the tensordicts that will be lazily stacked when items are accessed.

@vmoens the behaviour of .keys on LazyStackedTensorDict and SavedTensorDict is quite different to TensorDict at the moment, they each produce generators of the top level keys. Would you prefer that they be brought in line with the behaviour of TensorDict? It seems a bit challenging:

In the case of SavedTensorDict, I guess we wouldn't want to load the data for every membership check, so does that mean traversing the contents at instantiation and capturing all keys (nested or otherwise) in some kind of data structure?

In the case of LazyStackedTensorDict if I understand directly the keys are those that are in the intersection of all keys in the list of tensordicts to be stacked. So again I'm wondering if one would need to traverse and extract keys at instantiation to properly support this...

Let me know whether you think this is ok as is or whether it's worth spending some time on these two cases.

SavedTensordict should be deprecated. You can use TensorDict with memmap and save it with torchsnapshot. I would not worry too much about it!

In the case of LazyStackedTensorDict if I understand directly the keys are those that are in the intersection of all keys in the list of tensordicts to be stacked. So again I'm wondering if one would need to traverse and extract keys at instantiation to properly support this...

Hmm that's gonna be a big overhead
Can we just put an error message if someone uses __contains__ with a tuples of str with LazyStacked?

I've addressed the outstanding bits:

not done anything with SavedTensorDict per your comment
added a _LazyStackedTensorDictKeysView class to inform the user that only strings are supported for now.

So I think this is ready for a final review.


Add ability to lock version if manual deploy

Ref:

trello card

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

susperseded


Inconsistent theme style across the scene tree, file tree, and project settings tree

Godot version
v4.0.dev.custom_build [09b0293fa]
System information
Windows 11, Vulkan Clustered
Issue description
We have borders for tree item in the scene tree and file tree but not in the project settings tree.

Steps to reproduce
Check out how tree items look in the scene tree, file tree, and project settings tree.
Minimal reproduction project
No response

Should probably close this.

Very well!


Add an in-app changelog

Add an in-app changelog that's triggered when opening the app after an upgrade (or series of upgrades) to let the user know what's new from the last time he/she used the app
Proposed library to use: https://github.com/TonnyL/WhatsNew
We can either hardcode it, or we can have a JSON file on the server, or somewhere in the code like @VaiTon suggested

Hey, I'm interested to work on this. Can I get some details on it?

@hmuhammadazeem we could create a file CHANGELOG.md and then display it after every update in-app or maybe fetch the details from github.

Oh my bad, the file CHANGELOG already exists. The second part of my sentence is still valid though.

Hello. I am new to open source and wanted to start my first project. Do you guys think I can take a crack at this

@rprager18 yes you are very welcome to do so!
@teolemon recommends this library (https://github.com/TonnyL/WhatsNew). Take a look!

@teolemon @VaiTon Hi!
Is it still TBD? Can I take it?

We can do something special with the format you'd like @naivekook

@teolemon @VaiTon Hi!

Hey hi @naivekook and welcome!

Is it still TBD? Can I take it?

Sure you can!

also do we have another source of changelog?
In current case we need to parse CHANGELOG.md via gradle task on each build to get the new changes and store them in application and it quite tricky because this file doesn't have stable formatting.

We could use the file used by the playstore maybe, I really do not know.


Update trace-enhancements demo to use transform processor

It seems most otel collector things are moving to transform processor + OTTL, we should do the same here.

We use the following processors across the repo which I believe could be replaced with OTTL:

filter
resource
attributes

We might want to wait until the transform processor/ OTTL reaches beta to migrate to it.  It is currently in alpha


Chat box - Formatted message to player, Formatted prefix, User UUIDs, Message Radius

Describe your idea
It's more the just one feature request but they should be mostly easy to implement.

A sendFormattedMessageToPlayer(string jsonObject, string user, string prefix) command, which would allow to send a formatted message to a player only.
A possibility to have formatted prefixes.
A possibility to use player's UUID instead of just using their player name (which can changed).
Ability to set a radius for the sendMessage() and sendFormattedMessage() functions (so it wouldn't send the message globally).

If accepted I could try to do them and do a pull request of them if needed, but can't promise anything.
Describe alternatives you've considered if you've any
No response
Additional context
No response
Linked Issues
No response

Added now the following
sendMessage(String message, int range[, String prefixOrJson, String brackets, String bracketColor]) (Use -1 for infinite range)
Added the same for ToPlayer with uuid and name support.

So I finally got time to do some code and create a pull request with some changes I made. If you don't like them, fell free to change them.

Improved in 0.7.7r


TypeError: Cannot read property 'id' of undefined

I have created a simple bot using the botmaster framework (using a Telegram bot type) and added botmaster-session-ware to store a simple count attribute across requests.
The incoming ware appears to create the session OK, but the outgoing ware throws a TypeError with the message: "Cannot read property 'id' of undefined"
I am using:

NodeJS v6.9.4
botmaster v2.2.7
botmaster-session-ware v0.1.0

Bot source files and logging trace attached.
source.zip
botmaster-session-ware-typeerror.txt

I have the same problem. Could you solve it?
Thanks! :)

It sounds like the recipient or bot id is missing in your setup.
how are you using session ware?


[HUDI-5241] Optimize HoodieDefaultTimeline API

Change Logs

rename the origin getInstants to getInstantsAsStream.
add a new getInstants that return a list.
make sure that only use getInstants interface when using this.instants.

Impact
LOW
Risk level (write none, low medium or high below)
LOW
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

@YannByron Just curious as to how much perf improvements does this change bring?

@YannByron Just curious as to how much perf improvements does this change bring?

It is not about perf improvements. It make api suitable for users' perception, and convenient to extend, like RFC-36(https://cwiki.apache.org/confluence/display/HUDI/RFC-36%3A+HUDI+Metastore+Server)


How to set a pop up style by click button?

Like popup style in ui-datepicker

Check out the example for Datepicker on the UI Bootstrap site:
http://angular-ui.github.io/bootstrap/#/datepicker


Extentions player (this Player)

I think to create an extension of an object we can modify is not required. These methods could just be moved into the Player object

And I think they should stay where they are

I added the "this" because I don't understand why, it's in the Extension class if it's not an extension.


Add Windows Service support for TCP tunnel

Summary
Adds support for running Pomerium CLI TCP tunnel as a Windows service.
When built for Windows, this adds two new modes for the TCP tunnel.
When executed as a Windows service, it will setup the appropriate event loop to work as a windows service.
When launched from the CLI with the flag -service it will run as if it was running as a service to allow for debugging service scenarios.
This allows for an idiomatic way to run a long-lived Pomerium CLI session. Naturally, if it is run as a system service it won't be able to launch a browser to sign-in so it probably only makes sense for routes that are unauthenticated.
The motivating use-case for this is to allow access to a service, behind a Pomerium server that requires mTLS but no further authentication, to clients that don't implement client cert authentication.
It does not change any existing functionality, neither on Windows or other platforms.
Code structure
First we move almost all code from tcp.go to tcp_impl.go.
Then we make a slight refactoring to create the context outside the runTcp function and pass it as an argument instead.
Then we break out the init function and Cobra command definition to platform specific files.
The non-Windows one is essentially unchanged, except that move back the common parts of init to tcp_impl.go.
The Windows variant adds code for handling the service and service emulation modes.
Related issues
None
Checklist

[x] reference any related issues
[ ] updated docs
[ ] updated unit tests
[ ] updated UPGRADING.md
[ ] add appropriate tag (improvement / bug / etc)
[x] ready for review

Thanks for the contribution. I will test this on a windows machine.
I am wondering if there's a more generic way of doing this so that the service handling isn't TCP-specific? Possibly we could run the service handling in cobra's PersistentPreRunE? I ask this because we're adding UDP tunneling support which is very similar but a separate subcommand.
I really appreciate the work you've done here, so if you don't have the bandwidth for this or what I'm suggesting doesn't actually make sense, we could use this PR as a first step and we can iterate from there.

I tested this and it seems to work, albeit, as mentioned, only on an unauthenticated route.

I am wondering if there's a more generic way of doing this so that the service handling isn't TCP-specific? Possibly we could run the service handling in cobra's PersistentPreRunE? I ask this because we're adding UDP tunneling support which is very similar but a separate subcommand.

I agree that it would make more sense to make this more generic. The reason I didn't was that I wanted to make the initial change as small as possible to make it more digestible and I also was not familiar with Cobra so I didn't know how to structure it.
If you like this change and would like to make it more generic I can look into that.

Ok, so the way I envision the structure is that you have a something, for example PersistentPreRun that creates a wrapper function which either executes the command normally or as a service. This wrapper function is then called from each specific command with the implementation of the command as a function pointer. So something like (pseudocode):
func createWrapper() {
  if serviceMode {
    wrapper = runInServiceMode
  } elif serviceDebugMode {
    wrapper = runInServiceDebugMode
  } else {
    wrapper = runCli
  }

  return wrapper
}

func tcpCommand() {
  wrapper(tcpCommand)
}

func udpCommand() {
  wrapper(udpCommand)
}

func runInServiceMode(commandFunction) {
  service setup stuff
  commandFunction(args, ...)
  service event loop
}

func runInServiceMode(commandFunction) {
  service debug setup stuff
  commandFunction(args, ...)
  service event loop
}

func runInServiceMode(commandFunction) {
  commandFunction(args, ...)
}


fix: expose stack variables

Description
Resolves: https://github.com/terraform-ibm-modules/stack-ibm-core-security-services/issues/11

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This PR is included in version 0.1.0 :tada:
The release is available on:

GitHub release
v0.1.0

Your semantic-release bot :package::rocket:


Vulnerability Data Source Landscape

Building out a landscape document with all of the current and historical sources of vulnerability data:
https://csaurl.org/gsd-landscape
Any/all contributions are welcome.
To get edit access, join the GSD working group at: https://csaurl.org/gsd-signup

A similar effort by the nexB folks: https://github.com/nexB/vulnerablecode/blob/main/SOURCES.rst