fix: use KeyboardAvoidingView from react-native-keyboard-controller

Details
All possible solutions I listed in:
https://github.com/Expensify/App/issues/44514#issuecomment-2258226920
In this PR we are using 3rd approach. Before we already merged PR https://github.com/Expensify/App/pull/46513 but it caused some issues. This PR is a very similar to previous one, but in this PR I fixed some issues in react-native-keyboard-controller upstream and use version from main. If we agree to merge this PR (i. e. we don't find any new issues) then I can prepare a new release to npm and we will not use a version from GitHub directly.
This PR will help in implementation for https://github.com/Expensify/App/pull/42143
Fixed Issues

$ https://github.com/Expensify/App/issues/44514
PROPOSAL: https://github.com/Expensify/App/issues/44514#issuecomment-2258226920
Tests

Submit a couple of expenses in a workspace chat
Open the expense report
Tap on the first expense
Navigate back to the expense report
Be sure there is no layout jump/glitches
Test all other places in the application where we use a keyboard
Verify that https://github.com/Expensify/App/issues/46743 https://github.com/Expensify/App/issues/46744 https://github.com/Expensify/App/issues/46745 are not reproducible as well

Offline tests

N/A
QA Steps

Submit a couple of expenses in a workspace chat
Open the expense report
Tap on the first expense
Navigate back to the expense report
Be sure there is no layout jump/glitches
Test all other places in the application where we use a keyboard
Verify that https://github.com/Expensify/App/issues/46743 https://github.com/Expensify/App/issues/46744 https://github.com/Expensify/App/issues/46745 are not reproducible as well

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
Only native iOS affected.

Android: mWeb Chrome
Only native iOS affected.

iOS: Native
https://github.com/user-attachments/assets/30ac5fbe-46c0-4363-a5df-9d74fbf62090

iOS: mWeb Safari
Only native iOS affected.

MacOS: Chrome / Safari
Only native iOS affected.

MacOS: Desktop
Only native iOS affected.

@ishpaul777 did you compare debug and release variant? Basically yes, right now there will be some wobbling in debug builds and it should be less noticeable in release builds. Also after reanimated 3.15.0 there shouldn't be wobbling at all, because they optimized ShadowTree traverse algorithm a lot 👀

feel free to prepare a upstream release of package.

@ishpaul777 doing it now!

@ishpaul777 done, now using version from npm!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/marcochavezf in version: 9.0.22-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/marcochavezf in version: 9.0.22-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

This PR is failing for iOS app because of issue #47656

This PR is failing for iOS app because of issue https://github.com/Expensify/App/issues/47656

Okay, I think it's the same issue as https://github.com/Expensify/App/issues/44514
It seems like release iOS works faster and there could be a chance, that we switch enable variable faster, than needed and because of that we can see such micro-jump.
In development it wasn't the case, so I missed that 🙈

Hi, @kirillzyusko @ishpaul777  Could this PR be the reason for the ios bug on older OS versions here https://github.com/Expensify/App/issues/47679, can you please take a look into it?

🚀 Deployed to production by https://github.com/chiragsalian in version: 9.0.22-9 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/marcochavezf in version: 9.0.23-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/chiragsalian in version: 9.0.23-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


/lac rmpass not working

/lac rmpass always returns "Invalid username"
I think this is because you've swapped the true/false part in this line.

Fixed


Do the artifacts really work on osx and windows?

It seems that the different artifacts were all built on linux.

Tested and it works


please tag v0.16 and pushed to cocoapods

Hi @jsntghf . 0.16.0 has been released yet.

Hi @satya164 0.16.0 has been released, but the latest version in cocoapods is 0.13.0-rc.

@jsntghf React Native is not published to CocoaPods repository anymore. Use it like it's described in the docs - http://facebook.github.io/react-native/docs/embedded-app-ios.html#install-react-native-using-cocoapods

@satya164  I also wanna use react(0.16.0)  in cocoapod, how about you recently?

@Creemli the documentation explains how to use CocoaPods. React Native is published only to npm but you can still use CocoaPods.
For further questions if you're still stuck please ask on Stack Overflow. Thanks!

@ide  I have used Cocoapods for React Native, but it doesn't provide the function "react-native bundle". If React Native is not published to Cocoapods anymore, is there new document about how to integrating with existing apps? Or maybe we can help to convert RN npm package to Cocoapods package.

Please see the documentation here: https://facebook.github.io/react-native/docs/embedded-app-ios.html#install-react-native-using-cocoapods. There is no react-native bundle script so you should design a way to package up the JS with your app that works best for your custom setup.

Ok, thanks very much


GUI: Masternode tab doesn't refresh on MN IP address change

Hello,
In Dash Core GUI, Masternode Tab, Address sub-tab, after I changed my MN  IP address (masternode.conf), the GUI doesn't refresh and shows an old IP address.
My Masternodes were in 'MISSING' status, but the bug may follow other statuses also.
Restarting Dash Core doesn't solve the problem.
Version: Dash Core 12.1.4 on Windows 7 x64.
-Technologov, 29.03.2017.

Duplicate of #1338, closing


Support Icons on RecordActions

This PR allows Icons to be added to RecordActions.
There is a change in the logic of how RecordActions are displayed, previously any RecordActions added to a table would be displayed in addition to the default edit action.
To allow the default edit action to be replaced it will now only be shown if no RecordActions are supplied.

Hey @iantasker! I've rewritten this in #203 and merged into the develop branch. If you could test it (as well as the rest of the large refactor), that would be amazing. Thank you!


Chrome browser compatibility issues with latest release

Error message:
thread '' panicked at 'called Option::unwrap() on a None value', libcore/option.rs:335:21
Steps to reproduce:

Adapt example from fetchSync.js to run it in browser (file attached)
Run script in Chrome, using latest release (21 nov 2018)
Error is thrown when script fetches from MAM

My findings so far

In Firefox it works correct
Does not matter if mam.web..js or mam.web.min.js is used
Old releases (ones that uses IOTA library for init) worked correctly in chrome
Messages published in Firefox can be read in Chrome (publish in Firefox, save root, fetch in Chrome)

mam_test.zip

Hey,
there were some issues with Chrome like the one you reported above. We fixed them in the latest version.

Sounds great. I will test new library over the next few days across different platforms (windows, android, macOS).

Hi @sobolev-alexey , I have checked several browsers with the latest MAM release (as of 11 Feb 2019).
Default web build passed with all modern browsers on multiple platforms, only outdated browsers have failed. Many thanks for updating!
I have a proposition to increase adoption of MAM across web developers. It would be great to publish web libraries builds in /lib folder, so developers can start using them instantly, rather than setting up infrastructure (linux, node, yarn). mam.web.js libraries were published in the past and these helped me a lot to get started.
Current MAM library compatibility with browsers (default build):

WINDOWS 10:

Firefox Quantum 65.0 : PASSED
Chrome 72.0.3626.96 : PASSED
Edge 42.17134.1.0 : PASSED
Internet Explorer 11.523.17134.0 : FAILED 👎 (that's outdated browser)

MACOS HIGH SIERRA 10.13.6

Safari 12.0.3 (136<IP_ADDRESS>.1) : PASSED
Chrome 72.0.3626.96 : PASSED

ANDROID 4.4

Chrome 72.0.3626.96 : PASSED
Firefox 65.0 : PASSED
Samsung Internet 1.0 (Chromium 18) : FAILED 👎 (that's outdated browser)

IOS 12.1.4

Safari 12.0.3 (136<IP_ADDRESS>.1) : PASSED
Chrome 72.0.3626.74 : PASSED
Firefox 15.0 (13894) : PASSED

Hi @Core-pl
many thanks for the overview!
We tried to support as many older browsers as we can, but we are limited by browsers' support for WebAssembly
https://caniuse.com/#search=WebAssembly
The overview on the caniuse.com site is pretty much in line with the browsers that you listed above.
Internet Explorer (any version) does not support WebAssembly, same as Edge browser below version 16.

As for the mam.web.js, it was part of several releases, but caused issues with code minification tools, so I removed it.
If we see multiple requests from the community to add a pre-compiled version of mam.web.js, we will look into issues and release a patched version later.


Lorri is not able to access a file in KBFS

I have Lorri installed as a LaunchAgent in my user directory (i.e ~/Library/LaunchAgents/lorri.plist). I verified that Lorri is running as my user (same UID), but it somehow cannot access the same file that I can. Lorri is printing the following error:
error: opening file /Volumes/Keybase/team/team-name/path/to/file: Operation not permitted.

This is working fine on Linux. The problem is present only on Darwin. I'm running Catalina 10.15.4.
my log id: 6b5153e646fb9d8e0d34b91c

I'm not sure what Lorri is, but KBFS is only accessible to your own local macOS user for security reasons. So if Lorri is running as a different user (even root/admin), that's expected.

@strib it's running as my user.

Hmm.  Well I don't see the error in the log, which usually means that macOS itself returned the error to the program before it even came into KBFS.  And that usually means the file system request came through a non-you user.  It's possible Lorri itself farms something out to a different process or OS user (Finder does that sometime).  If you can read the file yourself, then that's definitely what's happening.  I'm sorry for the crapiness.

I'm not sure either how Lorri works.
CC some of the people who work on Lorri @Profpatsch @grahamc @curiousleo and @LnL7


Upgraded the footer of the Blog section and modified its buttons

##The footer section of the blog post
The footer of the blog posts section was not well decorated. So I modified them and added some colors, transitions, and effects to make it look more attractive.
I have made the footer section of the blog look like the main page (which comes under index.html)
##buttons of the blog post
I have also modified the blog section's default buttons (bootstrap default buttons) into new ones. I have added vibrant colors to make it look more eye-catching.

@kohi-noor Kindly add the Images of the output

My contribution: The footer section and all the yellow buttons in the blog section.
I hope this image finds you well.


TypeError [ERR_INVALID_ARG_TYPE]: The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received undefined[BUG]

Describe the bug
I am getting undefined when I try to upsert document using ElasticSearch in my chatflow. The same thing works properly with Pinecone. I have attached my chatflow below:

I was able to upsert successfully with the guide - https://docs.flowiseai.com/integrations/vector-stores/elastic

I have just checked the docs, but it shows the one with elastic cloud. I am using the docker one; the credentials I have are host, username, and password. I debugged my issue further and it shows the endpoint parameter is missing, I thought this might be the issue, so tried passing it a default value, and later totally removing that endpoint parameter. But I am still getting the same error. Putting some console log statements show that the Elasticsearch functions: prepareClientArgs, prepareConnectionOptions etc are properly working, and upsert results in error when building langchain..

I have debugged further, and it stops at:
               opts.node = `https://${cloudUrls[1]}.${cloudUrls[0]}`;
``` in file<EMAIL_ADDRESS>.
In opts.node line, I am getting undefined in URL. Then I hard-coded my url here, and got "Error: ConnectionError: self-signed certificate in certificate chain"


Execute a remote script

How cool would it be to be able to do:
conda execute https://raw.githubusercontent.com/pelson/conda-execute/master/example.sh

@pelson Is this closed by #19? Seems like I didn't format the magic words properly for github to auto-close it


Xcode 'dyld: lazy symbol binding failed: can't resolve symbol ___cxa_guard_acquire in'

Hello.
I have followed every step in the README and also tried to run the example app but I'm getting the following error when trying to run the app in XCode
dyld: lazy symbol binding failed: can't resolve symbol ___cxa_guard_acquire in /Users/gabriel/Library/Developer/CoreSimulator/Devices/E2983EEC-BB6B-4F04-A5A6-350B11DCF58B/data/Containers/Bundle/Application/B51B68CC-6238-434D-865B-B45D56168053/RNOpenALPRExample.app/RNOpenALPRExample because dependent dylib #1 could not be loaded dyld: can't resolve symbol ___cxa_guard_acquire in /Users/gabriel/Library/Developer/CoreSimulator/Devices/E2983EEC-BB6B-4F04-A5A6-350B11DCF58B/data/Containers/Bundle/Application/B51B68CC-6238-434D-865B-B45D56168053/RNOpenALPRExample.app/RNOpenALPRExample because dependent dylib #1 could not be loaded (lldb)
Anyone has any idea of what this could be related to or how to fix it ? Thanks in advance.

Can it be related to https://stackoverflow.com/questions/52331634/xcode-10-0-gm-dyld-lazy-symbol-binding-failed-cant-resolve-symbol-cxa-gu ?
What version of simulator or device are you running it on?

I saw this stackoverflow thread, Its a problem related to iOS 12 simulator. I ended up importing the OpenALPR framework directly in XCode. Thanks.


Summary statistics for Inspection Report is not displaying

After processing of inspection_master file. summary details are not displaying in te admin_console.

Re-tested working as expected

Closing this defect as it has been test validated successfully


Add Permissions Support

[x] Android
[x] iOS
[ ] Web

AFAIK permissions are handled automatically in Desktop so we don't need to worry about it.

In the current desktop implementation, desktop permissions are automatically denied, so it does not actually let the OS handle the permission (it should be Granted instead of Denied automatically).
Current implementation looks like this:
@ExperimentalPermissionsApi
@Stable
internal class MutablePermissionStateImpl(
    override val permission: Permission,
) : MutablePermissionState {

    override var status: PermissionStatus by mutableStateOf(getPermissionStatus())

    override fun launchPermissionRequest() {}

    override fun openAppSettings() {}

    override fun refreshPermissionStatus() {}

    private fun getPermissionStatus(): PermissionStatus {
        return PermissionStatus.Denied(false) // This should be PermissionStatus.Granted
    }
}


fasttext pybind11 wheel failure

I ran into this problem trying to install extra packages (eg. txtai[all]). It requires and tries to build fasttext, but it fails due to not finding pybind11. I have pybind11. I have fasttext, correct version and everything, but it insists on trying to build it and it fails.
For reference in case someone else runs into this, here's how I fixed it. I found the wheel file and edited the metadata to exclude fasttext.
Find the pip cache directory> "pip cache dir"
Go there, and create a temp dir> "mkdir a"
Change to that directory> "cd a"
Unzip the wheel> "unzip ../txtai-6.1.0-py3-none-any.whl"
Rename the wheel file in case something goes wrong> "mv ../txtai-6.1.0-py3-none-any.whl ../txtai-6.1.0-py3-none-any.whl.bak"
Go into the .dist-info directory in the unzipped directory structure> "cd txtai-6.1.0.dist-info"
Edit the METADATA file> "vim METADATA" (or nano or any text editor)
Find all instances of fasttext (I found 4), and delete the whole line wherever you find it.
Save the file.
Back in the temp directory, repack the zip file into a wheel> "zip -r ../txtai-6.1.0-py3-none-any.whl ."
Install the wheel directly> "pip install ../txtai-6.1.0-py3-none-any.whl"
Will someone verify that this is a workable procedure?

Do you have a C++ compiler installed? That is typically the reason fasttext installs fail.
I would never recommend an approach like this. The root cause should be addressed and a fix put into the project.

Thanks, @davidmezzetti . I know this is a total hack, and I very much desire clean corners boxing in all the code everywhere, but I had to do something, and I find that I'm nowhere near technical enough. So, this is what we have. And, if it helps someone like me, I'll call it worthwhile to report some workaround - just for practical purposes, you know?
Do you think there's a root cause to fix? I have gcc installed. Except for an upstream issue with fasttext, and they stopped dev a long time ago, I don't know any other solution.

What version of Python are you using?

This issue might be relevant: https://github.com/neuml/txtai/issues/572

Python == 3.11.6
I saw this solution in the fasttext repo, but I don't know how to apply it for this wheel. I can install fasttext with apt, but the txtai wheel tries to compile it anyway. If there's a clean solution, I'd think it was in getting the wheel to stop trying to compile things that already exist in the system.


All APIM Entities via ARM Template

Fixed NSG Rule for Cache
Add Support for Creating Backend, Certificate, API, Operation and Policies using ARM Template

@solankisamir,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@rwike77 @vturecek can you take a look ?

@rwike77  @vturecek  ping on this.

@rwike77 a gentle nudge on this. Should we add the template someplace else?

@solankisamir I'm starting to try out the new template.  I also need to update the associated article, will try to do that in the next day or two.  Sorry for the delay.

Closing the pull request, as changes have been incorporated.


Add donation options

I used material-table and material-table/core extensively in a project within the last year and am starting the conversation about open source contributions and support within the company I work for.
Could material-table/core profit from monetary support and how could this be done with it being a fork?
I cant promise anything but would like to use this project as an example for projects that deserve our gratitude.

Hi,
great that you like it so far.
We already talked about this and we were thinking about adding an open collective account.
The money would be used so that we can take time out of our schedule to work on features.
I just added an open collective for us: https://opencollective.com/material-table-core

I agree with @Domino987!

Closing since we now have options


Unknown crash on SWRevealViewController

Hi guys,
I 'inherited' a iOS project which is using the SWRevealViewController.
But in a particular reveal it always crashes with a InvallidArgumentException.
I traced it back to the _rearViewDeploymentForNewFrontViewPosition method. It fails on calling
[self _deploymentForViewController:_rearViewController inView:_contentView.rearView appear:appear disappear:disappear];
with the following exception:
Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: 'data parameter is nil'
I'am not familiar with the code, so help would be really appreciated.
Regards, Maarten.

rearViewController is probably nil. Can you check it?

Hi,
It is not nil, it doesn't jump into
if ( _rearViewController == nil && newPosition > FrontViewPositionLeft )
if I inspect the controller it looks like:


Remove enigmampc/btcutil dependency in go.sum. Update packages version.

Relates to https://github.com/crypto-com/chain-indexing/issues/661
To eliminate the https://github.com/enigmampc/btcutil dependency in chain-indexing/go.sum, we need to update this cosmosutils repo first.
Changes:

Follow cosmosSDK, using github.com/cosmos/btcutil v1.0.4, instead of github.com/btcsuite/btcutil v1.0.3-0.20201208143702-a53e38424cce
Update major packages version (ibc-go, CosmosSDK, chain-main).
Update the go module to 1.17.
Update instructions in readme.

Tests:

run the go test, passed
manually test the sample commands given in readme, expected output is generated

Important Reminder!
When you want to add/remove dependency through go mod, please use this command: go mod tidy -compat=1.17.
DO NOT RUN go mod tidy without the -compat=1.17, it will introduce the https://github.com/enigmampc/btcutil dependency in the repo's go.sum!

Codecov Report

Merging #14 (18c4f2a) into master (888d10f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   82.08%   82.08%
=======================================
  Files           4        4
  Lines          67       67
=======================================
  Hits           55       55
  Misses          6        6
  Partials        6        6

Impacted Files
Coverage Δ

types.go
100.00% <ø> (ø)

tmpubkey.go
53.84% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fullscreen Option

It would be very nice if there was a fullscreen option. As an adult I find this great but children might like it even more and with the program running full screen, either stretched or scaled, it might be easier for them to use.

Once 1.5.0 is out I'll probably get on that, but I'm already thinking about what should be done. I'd like a clean separation between UI and backend stuff. The only problem is I don't know what's the best way to achieve that with an application that needs to refresh its display while a background thread is working.
I'm just gonna put some pointers here for future reference:
https://www.reddit.com/r/JavaFX/comments/miij41/how_to_use_javafx_to_monitor_a_model_running_in_a/
https://stackoverflow.com/questions/22772379/updating-ui-from-different-threads-in-javafx
btw this issue is basically the same as #62
btw btw you can assign me to this @DC37

The more I think about it, the more I believe letting the GPU handle staff display is the way to go.
Every other idea I've had either involves awkward code dependencies or swarming the FXAT with redraw jobs.


[Feature Request]: Support bearer tokens

Description
Implement AuthRequest.validate() but for bearer tokens inside Authorization header instead of cookies

Done with v2 beta


Check value of frac_to_keep is admissible

The value for frac_to_keep should fall in the interval [0, 1]. If
this is not the case, an error is thrown and the function exited.
IMPORTANT: Please search among the Pull requests before creating one.
Title

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] New feature (non-breaking change which adds functionality)

Description

Checks that frac_to_keep lies within [0, 1].
How has this been tested?

Have release notes been modified?

Closes

Closes #659.

@michalk8, should we add a test checking that the error message is thrown when using inadmissible values for frac_to_keep?

@michalk8, any idea why codecov is failing? Also, all CI tests are shown to have passed but looking at the run here, for example, it says that test (macos-latest, 3.8, false, true) failed. In the test logs, it encounters a Segmentation fault.

@michalk8, any idea why codecov is failing?

It's just status check saying whether diff target has been reached or not - I've kept the defaults, we can change it later here.

test (macos-latest, 3.8, false, true) failed. In the test logs, it encounters a Segmentation fault.

I've sat macOS to allow failures, since in some cases, jax causes segfault on the CI (all the tests are related to 'stochastic' mode in VelocityKernel). If you want to, you can try tackling this (but I don't think you'd be able to reproduce this locally).
Otherwise, LGTM, thanks @WeilerP .


Remove start/stop services utility for the moment

Purpose
$subject.

Yes earlier when writing tests for a package "ab", we had to specifically start the services of that package, but with a recent change https://github.com/ballerina-platform/ballerina-lang/pull/9615/files we don't need to specifically start the services for that package,, this has some debates whether which is the correct behavior, so for now we will merge this PR and fix this properly later. Created an issue https://github.com/ballerina-platform/ballerina-lang/issues/9652 to track this


Implement read of 3D images with unnormalised sampler

We need to know where the sampler is coming from, so we need to inline every function containing a read image of a 3d image with non literal sampler.
This PR depends on KhronosGroup/SPIRV-Headers#377
We use the information from the sampler mask to know if you need to use the original coordinates or the one we have normalised.
Like for channel_image_order and channel_image_data_type we use metadata to pass information through the pipeline.

Please rebase this on top of the headers, I've merged the necessary changes.

Amber does not have the new spirv-headers yet.


logical inconsistency  in tRNA processing

charged-tRNA amino acid modification (GO:0019988)
Definition
The covalent alteration of an amino acid charged on a tRNA before it is incorporated into a protein, as in N-formylmethionine, selenocysteine or pyrrolysine.
is a descendent of  tRNA processing GO:0008033 which is defined:
The process in which a pre-tRNA molecule is converted to a mature tRNA, ready for addition of an aminoacyl group. [PMID:12533506]

Why is this inconsistent? Do you mean that the amino acid is not a part of the tRNA?

Ignore, I misread it.


Refactor genomic range sharing into utils-java.

Logic to shard genomic ranges should be all in one place to be reused by both dataflow and spark.  Its also important to ensure that the full collection of shards is shuffled so that parallel requests will be distributed across the dataset.
This pull request is part 1 of many parts to refactor some logic currently in dataflow-java down into utils-java so that it can be more widely re-used.  Since this code may change yet again, sending these pull requests to a branch for now instead of master.

@jean-philippe-martin PTAL

rearranged code per feedback and added integration tests
also refactored read sharding from dataflow-java to utils-java
please be sure to view https://github.com/googlegenomics/dataflow-java/pull/116 so that you can see where much of this code came from

Happy to rename anything to clean up/consolidate terminology.

Thank you! LGTM, aside from minor comments. Feel free to merge once ready.

Javadocs added.  Will update IntegrationTestHelper.API_KEY in both utils-java and dataflow-java in the next set of pull requests.


ZeroDivisionError  at 01_spindles_detection.ipynb

I download code and run 01_spindles_detection.ipynb , then I got an error:
ZeroDivisionError                         Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_28896/14777483.py in <module>
      1 # Apply the detection using yasa.spindles_detect
----> 2 sp = yasa.spindles_detect(data, sf)
      3
      4 # Display the results using .summary()
      5 sp.summary()

S:\Anaconda\envs\yasa_env\lib\site-packages\yasa\detection.py in spindles_detect(data, sf, ch_names, hypno, include, freq_sp, freq_broad, duration, min_distance, thresh, multi_only, remove_outliers, verbose)
    675             _, mcorr = moving_transform(x=data_sigma[i, :], y=data_broad[i, :],
    676                                         sf=sf, window=.3, step=.1,
--> 677                                         method='corr', interp=True)
    678         if do_rms:
    679             _, mrms = moving_transform(x=data_sigma[i, :], sf=sf,

S:\Anaconda\envs\yasa_env\lib\site-packages\yasa\others.py in moving_transform(x, y, sf, window, step, method, interp)
    192     if method in ['covar', 'corr']:
    193         for i in range(idx.size):
--> 194             out[i] = func(x[beg[i]:end[i]], y[beg[i]:end[i]])
    195     else:
    196         for i in range(idx.size):

S:\Anaconda\envs\yasa_env\lib\site-packages\yasa\others.py in func(x, y)
    183     elif method == 'corr':
    184         def func(x, y):
--> 185             return _corr(x, y)
    186
    187     else:

ZeroDivisionError: division by zero

Then I change sp = yasa.spindles_detect(data, sf) to sp = yasa.spindles_detect(data, sf, thresh={'rel_pow': 0.2, 'corr': None, 'rms': 1.5}) and the code can sucessfully run.
I want to know why would I  encounter this problem, hope your help.

I think maybe because jit . When I  comment out the jit in the numba.py
"""
This file contains Numba-accelerated functions used in the main detections.
"""
import numpy as np
from numba import jit

__all__ = []

#############################################################################
# NUMBA JIT UTILITY FUNCTIONS
#############################################################################

#@jit('float64(float64[:], float64[:])', nopython=True)
def _corr(x, y):
    """Fast Pearson correlation."""
    n = x.size
    mx, my = x.mean(), y.mean()
    xm2s, ym2s, r_num = 0, 0, 0
    for i in range(n):
        xm = x[i] - mx
        ym = y[i] - my
        r_num += (xm * ym)
        xm2s += xm**2
        ym2s += ym**2
    r_d1 = np.sqrt(xm2s)
    r_d2 = np.sqrt(ym2s)
    r_den = r_d1 * r_d2
    return r_num / r_den

#@jit('float64(float64[:], float64[:])', nopython=True)
def _covar(x, y):
    """Fast Covariance."""
    n = x.size
    mx, my = x.mean(), y.mean()
    cov = 0
    for i in range(n):
        xm = x[i] - mx
        ym = y[i] - my
        cov += (xm * ym)
    return cov / (n - 1)

#@jit('float64(float64[:])', nopython=True)
def _rms(x):
    """Fast root mean square."""
    n = x.size
    ms = 0
    for i in range(n):
        ms += x[i]**2
    ms /= n
    return np.sqrt(ms)

#@jit('float64(float64[:], float64[:])', nopython=True)
def _slope_lstsq(x, y):
    """Slope of a 1D least-squares regression.
    """
    n_times = x.shape[0]
    sx2 = 0
    sx = 0
    sy = 0
    sxy = 0
    for j in range(n_times):
        sx2 += x[j] ** 2
        sx += x[j]
        sxy += x[j] * y[j]
        sy += y[j]
    den = n_times * sx2 - (sx ** 2)
    num = n_times * sxy - sx * sy
    return num / den

#@jit('float64[:](float64[:], float64[:])', nopython=True)
def _detrend(x, y):
    """Fast linear detrending.
    """
    slope = _slope_lstsq(x, y)
    intercept = y.mean() - x.mean() * slope
    return y - (x * slope + intercept)

and use local yasa not the yasa installed by pip, the code can run successfully.
But the code can run successfully on the computer which has a GPU and  I don't have GPU on my computer. So maybe jit need GPU support, I guess.

Hi @Yifeng-J,
Thanks for opening the issue. I believe that there is something invalid with your data, my first guess is that there might be some flat data (e.g. no variance), which is not currently supported by YASA. Could you double-check for any flat period in your data? Or do you have any missing values? Are you using continuous EEG data? If so, how long is the data?
Thank you,
Raphael

The versions of numpy and numba are as follow:
numba=0.54.1
numpy =1.20.3

Thanks.


Firefox/LNBits: creating an invoice results in an error: Amount must be of number type

When creating a new invoice (Receive) I get the following error:

I use:

Firefox addon via the Extensions (no clue of the version)
LNBits as funding source
Data:

Description: Test
Amount: 10

hmm, interesting. thanks for reporting. What LNBits instance are you using?

You mean like, the funding source? It's LND. Or do you mean version?


POST params with file does not found if it posted after file (Tested in ubuntu )

If association key set  before file then it works fine as shown in image.

But any params set after file then they dont found in req.

Ok Try this....

set reqest in multipart

file     : image.jpg
name : Nishchit
age    : 22
In your controller console req.allParams() or req.params.all();
you will find " { } " printed out in console.
Now send this
name : Nishchit
age    : 22
file     : image.jpg
then re console request.
You will get
{
name : Nishchit
age    : 22
}
-ND

This is pretty well documented in the Skipper README.  Text parameters must be sent after the file.


Multi Config & Ability to use Different config for different actions.

Is your feature request related to a problem?
No response
Describe the solution you'd like
@Chralu Just saw your implementation in the latest merge. You have added a transport builder which can take in WSS and/or HTTP urls. Now Once we provide these all the traffic to the blockchain are passed via these transportbuilder only. but lets say we have a situation where lets say we want to .

Use default connected wallet for sending  call or sendTransaction action to the blockchain.
&
Use WSS transport url for watchContractEvent.

How do you propose we address this situation?
Additional context
No response
Epic
No response

Hi @Chralu
Can you please look into this https://github.com/0xMetaLab/wagmi_flutter_web/tree/TransportConfig and let me know your views?


Add git2_ureq transport

This is an alternate approach to #624 that uses an actual HTTP client (ureq) on top of rustls. In the end this came out significantly cleaner, and by using default_features = false I was able to keep the dependencies very minimal.
Note that this branch is built on top of the branch for #624 for convenience. If we decide to go this route I'll excise the git2_rustls files.

Also note this actually won't compile with ureq 1.4.1. I needed to make some local changes to add Send + Sync to the return value of into_reader, which I'll upstream now.

Nice! This seems pretty reasonable to me to add! I  haven't looked at git2-curl in a very long time though so some things could probably be improved there, but this seems like a great start at least!

Updated to refer to a released version of ureq with the needed Send marker. I also refactored significantly to better reflect my understanding of how things work. Definitely interested in feedback on code structure and style.

I'm actually going to close this for now. When I went deeper on integrating this with cargo, I realized the other fetches in cargo (the crate downloads) rely on HTTP/2 for a significant speedup. Since ureq doesn't currently support HTTP/2, it wouldn't be suitable as a general replacement for curl in cargo. I'm going to take a look at whether it's feasible to implement a lightweight subset of HTTP/2 in ureq for concurrent / pipelined fetches. But I may wind up coming back around to this with a different HTTP client.


Updating packages

This should update PF-React, PF-Next + some other stuff

Waiting for the patternfly-next package to be updated. I'll also update the hapijs / hoek package that has a vulnerability.


feat: Rename all SHA3 opcodes to KECCAK256

Closes: #513

@tungbq thank you for taking this!
There is one function sha3 that could use renaming.

@tungbq thank you for taking this!
There is one function sha3 that could use renaming.

Thanks @rakita for reviewing. I've updated the new commits to update sha3 function.
Please check if it meets the requirements. Thanks

there are failing test

Updated

Looks nice, will merge it after CI finishes


Github sign is does not appear in settings or the repo tool

Dev Home version
0.17 Canary 585
Steps to reproduce the bug
Log out of github.
Try to log-in via either settings or the repo tool.
Expected result
Prompt appears and I can log in.
Actual result
Settings is unresponsive.
The repo tool does do something, but the prompt does not appear.
Included Extensions Information
Did a test.
Logging in works with DevHome(Preview) and Github(Preview)
Logging in does not work with DevHome(Preview) and Github(Canary)

CantLogInLogsDhoehna.zip


typo: defintion -> definition

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/docs-conceptual/sharepoint-pnp/sharepoint-pnp-cmdlets.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-pnp/Add-PnPRoleDefinition.md
:white_check_mark:Succeeded
View (sharepoint-ps)

sharepoint/sharepoint-ps/sharepoint-server/Remove-SPSiteSubscriptionFeaturePackMember.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


refactor: add account nonce as input of run_validate and remove validation output

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#822  👈
#521
#519 : 2 other dependent PRs (#520 , #604 )
#672
#666 : 2 other dependent PRs (#694 , #701 )
#661
#660
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

crates/gateway/src/gateway.rs line 169 at r1 (raw file):
Previously, Yael-Starkware (YaelD) wrote…
Getting the nonce from the storage fails if the contract address doesn't exist, but in this case, the state reader will return the default value and not an error.
Other failures derive from communication, serde, internal papyrus error, etc, so I think it makes sense to classify it as an internal error.

If I understand @Yael-Starkware correctly, this should indeed be an internal error. No error should be handled differently, and indeed, some internal errors may arise.

Merge activity

Sep 22, 3:05 AM EDT: @ArniStarkware started a stack merge that includes this pull request via Graphite.


400 errors are not passed as a 'response'

need to interrogate 200 responses from server to see whether they are a JSON error and pass to callback as an error instead.

specifically, for some reason when an error is thrown by the metadata request to the service while initializing, the error callback isn't triggered at all...  (no idea why)

closed by Esri/esri-leaflet/pull/584


Return None when QueryResult::state is Errored on next_set()

Where I work we use this library quite a lot (thanks for the work!), but we stumbled against an issue that I can reproduce also on the latest version of this library (currently 21.0.0).
Basically when executing an invalid query the code hangs and it's fortunately reproducible 100% of the time. The code that triggers the issue looks like the following:
// 1. Create database
// 2. Create user with privileges

let opts = OptsBuilder::new()
    .ip_or_hostname(my_ip)
    .user(Some(my_user))
    .pass(Some(my_password))
    .db_name(Some(my_db));

let mut conn = Conn::new(opts).unwrap();

// NOTE: the last of the VALUES is missing!
let invalid_sql = r#"
CREATE TABLE IF NOT EXISTS `user_details` (
    `user_id` int(11) NOT NULL AUTO_INCREMENT,
    `username` varchar(255) DEFAULT NULL,
    `first_name` varchar(50) DEFAULT NULL,
    `last_name` varchar(50) DEFAULT NULL,
    PRIMARY KEY (`user_id`)
);

INSERT INTO `user_details` (`user_id`, `username`, `first_name`, `last_name`)
VALUES (1, 'rogers63', 'david')
"#;

let result_set = conn.query_iter(invalid_sql).unwrap();

After a bit of debugging I noticed that it hangs on QueyResult::drop, in particular at the 3rd call of QueryResult::next_set.
This MR is a first attempt to solve the issue, we are currently using this patch since it fixes the issue, however I do not have enough context to understand the consequences of this change, nor currently the time to implement some tests. Please feel free to merge if you think this is indeed a fix, to take it over, or to guide me over the following steps.

Hi, thanks for PR! I'm on a trip at the moment, so I'll look into it a bit later.

Hi, thanks. Please note that I also tried to return Some(Err(error)) instead (I had to create a dummy error since the current one cannot be cloned). But it also hangs in that case.

This is superseded by e5db79d. Closing.
Thanks!


Failed to batch download when "collect from frames recursively" is used

https://imgur.com/gallery/zrDjjA9
Uncaught ReferenceError: pickImages is not defined
    at <anonymous>:1:1

If the frame is dynamically created, the content script doesn't work on the page?

Probably related:
https://bugs.chromium.org/p/chromium/issues/detail?id=760954
I think it should be safe to ignore those dynamic iframes since they are ads most of the time.


Chrome Renderer crashes

I've been doing a few tests with marked, and I've found that if I feed it large documents with embedded HTML (or plain HTML alone), it will crash Chrome Renderer.
I've been able to reproduce this reliably by force-feeding it raw HTML - notably the contents of the container div at http://oss.sapo.pt/ (which I'm re-doing using marked, BTW - you'll notice the site currently fetches Markdown and parses it on the fly using showdown...).
Further testing reveals that Safari and Firefox are immune to this, BTW - I can toss in the same documents and they come out fine.

Very odd, could you link me to the exact markdown you're feeding it?

The html rule's regex tries to match infinitely. This appears to be a bug in v8 if it works with other browsers. I'll change it around and push a fix soon.
Anyway, I wouldn't recommend feeding that html to any markdown implementation. markdown.pl, discount, and upskirt all produce misnested tags for that piece of html. Marked might also produced misnested tags once I push the fix.

Great, thanks. My concern was that the loop might happen with smaller chunks of HTML - this was merely the one chunk I was having trouble with, and the one where I could reproduce the issue easily in Chrome (I was developing in Safari first).
On 24/01/2012, at 19:07, Christopher<EMAIL_ADDRESS>wrote:

The html rule's regex tries to match infinitely. This appears to be a bug in v8 if it works with other browsers. I'll change it around and push a fix soon.
Anyway, I wouldn't recommend feeding that html to any markdown implementation. markdown.pl, discount, and upskirt all produce misnested tags for that piece of html. Marked might also produced misnested tags once I push the fix.

Reply to this email directly or view it on GitHub:
https://github.com/chjj/marked/issues/23#issuecomment-3638293


ensure exit code is an int

Description of changes:
Ensure that the exit_code provided by the final exception handler provided is a valid int as expected by os._exit.
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

[x] I have read the CONTRIBUTING doc
[x] I used the commit message format described in CONTRIBUTING
[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[x] I have updated any necessary documentation (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

appreciated the unit tests on the corresponding change for v2, but since this branch is technically deprecated, I won't ask for unit tests here.


Oxygen update naming of Redhat family supplied packages to distinguish as from SaltStack

The following email thread captures the discussion, edited for brevity:
David Murphy<EMAIL_ADDRESS>Jan 16 (8 days ago)
to nicole, Mike, Thomas, bcc: me
Given I am touching the most of the dependencies for building Salt on Redhat 6 & 7 with the addition of Python 3, I thought this might be a good time to change the packaging names.
There is an issue already covering this that has been long standing and it would be good to finally get it addressed.
https://github.com/saltstack/salt/issues/29132
Proposal is to add and additional 'ss' to any package that we provide, including 'ius' packages
Current name:
python-cherrypy-5.6.0-1.el6.noarch.rpm
python27-2.7.13-2.ius.el6.x86_64.rpm
Renamed name:
python-cherrypy-5.6.0-1.ss.el6.noarch.rpm
python27-2.7.13-2.ius.ss.el6.x86_64.rpm
This would only affect the name of the package and help in identification of source, that is provide by SaltStack.
Can you please provide comments and/or approval to proceed.
David Murphy
Unix Engineer
<EMAIL_ADDRESS>|  www.saltstack.com
Nicole Thomas
Jan 16 (8 days ago)
to me, Mike, Thomas
Hi David,
That sounds good to me, but since I don't have much experience in this matter, I'd love to see if Mike or Tom have any other concerns about this approach.
Thanks,
Nicole
David Murphy<EMAIL_ADDRESS>Jan 16 (8 days ago)
to Nicole, Mike, Thomas, bcc: me
Just chatting with Daniel and he suggests instead of using 'ss' it would be better to use 'salt' and to add an additional 's' to the name similar to IUS.
For example:
python35u-mod_wsgi-4.5.24-1.ius.el6
we could use similarly in bold:
python35s-mod-wsgi-4.5.24-1.salt.el6
Thus altering the name slightly and showing the repo
Nicole Thomas
Jan 16 (8 days ago)
to me, Mike, Thomas
I like that better - it is more clear.
David Murphy<EMAIL_ADDRESS>10:30 AM (35 minutes ago)
to Nicole, Mike, Thomas, bcc: me
@Tom, @Mike
Any objections to doing the proposed above in the email thread ?
Thomas S Hatch
10:35 AM (30 minutes ago)
to me, Nicole, Mike
I think that is the right approach.
Thomas S. Hatch  |  Founder, CTO | SaltStack, Inc.

Blocked since yum update expects the package names to match and not on what the package provides, for example :
python-chardet-2.2.1-1.el7.noarch.rpm
pythons-chardet-2.2.1-1.el7.noarch.rpm
python-chardet-2.2.1-1.salt.el7.noarch.rpm
Are considered as three different items, even when they provide support for the same item.
And yum gets upset if they try to manipulate a file which one of the others controls.
Renaming everything is beyond belief, it would result in a tracking nightmare of CVE and updates.
Note that using IUS as an example in Python2.7 support for Redhat 6 was fine for them since Python 2.7 is not provided by Redhat or EPEL.  That is, IUS doesn't have any competing packages supplying Python 2.7, not the packages they provide compete with Redhat 7 or EPEL.
Need to consider a different approach

is this a valid issue @dmurphy18 ?


[Bug] Opening dev tools causes and exception

mainWindow isn't in scope in this block after I changed the quick exit checks. fix incoming
          {
            label: i18n._(/*i18n*/{id: "Developer"}),
            submenu: [
              {
                label: i18n._(/*i18n*/{id: "Developer Tools"}),
                accelerator:
                  process.platform === "darwin"
                    ? "Alt+Command+I"
                    : "Ctrl+Shift+I",
                click: () => mainWindow.toggleDevTools()
              }
            ]
          },

already merged


Remember me?

I'm Crandetan, do you remember me anymore? I used ur autofarm thing and the world got nuked lol

In ur world* u had 2magplants and more


Unary and Binary Operators combined with Functions produce incorrect indentation

Thanks for fixing #443, however I have found some more indentation problems with Unary and Binary operators combined with Functions. Return statement is irrelevant in all cases, could as well be an variable assignment.
'use strict';

// Unary Operators produce incorrect indentation with functions (return statement is irrelevant)
function aFn() {
    return [].reduce((p, c) => {
            return p + c;
        }, 0) / 1;
}

function aFn() {
    return 1 + (function() {
            return arguments;
        })();
}

function aFn() {
    return typeof function() {
            return arguments;
        } === 'function';
}

function aFn() {
    const b = [].reduce((p, c) => {
            return p + c;
        }, 0) / 1;
    return b;
}

// Binary Operators |, &&, instanceof and in, (possibly more?) produce incorrect indentation combined with functions  (return statement is irrelevant)
function aFn() {
    return 1 | (function() {
            return arguments;
        })();
}

function aFn(a) {
    return (function() {
            return arguments;
        })() && (function() {
            return arguments;
        })(); // same results if functions are not wrapped
}

function aFn(a) {
    return [function() {
            return;
        }] instanceof Array;
}

function aFn(a) {
    const b = [function() {
            return;
        }] instanceof Array;
    return b;
}

function aFn(a) {
    return a in [function() {
            return;
        }];
}

function aFn(a) {
    return function() {
            return;
        } instanceof Function;
}

aFn();

$ esformatter test.js > test_out.js
$ eslint test_out.js

D:\github\test\test.js
   6:13  error  Expected indentation of 8 space characters but found 12  indent
   7:9   error  Expected indentation of 4 space characters but found 8   indent
  12:13  error  Expected indentation of 8 space characters but found 12  indent
  13:9   error  Expected indentation of 4 space characters but found 8   indent
  18:13  error  Expected indentation of 8 space characters but found 12  indent
  19:9   error  Expected indentation of 4 space characters but found 8   indent
  24:13  error  Expected indentation of 8 space characters but found 12  indent
  25:9   error  Expected indentation of 4 space characters but found 8   indent
  38:13  error  Expected indentation of 8 space characters but found 12  indent
  39:9   error  Expected indentation of 4 space characters but found 8   indent
  46:13  error  Expected indentation of 8 space characters but found 12  indent
  47:9   error  Expected indentation of 4 space characters but found 8   indent
  52:13  error  Expected indentation of 8 space characters but found 12  indent
  53:9   error  Expected indentation of 4 space characters but found 8   indent
  60:13  error  Expected indentation of 8 space characters but found 12  indent
  61:9   error  Expected indentation of 4 space characters but found 8   indent
  66:13  error  Expected indentation of 8 space characters but found 12  indent
  67:9   error  Expected indentation of 4 space characters but found 8   indent
  73:13  error  Expected indentation of 8 space characters but found 12  indent
  74:9   error  Expected indentation of 4 space characters but found 8   indent
  79:13  error  Expected indentation of 8 space characters but found 12  indent
  80:9   error  Expected indentation of 4 space characters but found 8   indent

✖ 22 problems (22 errors, 0 warnings)

esformatter v0.9.5

it is a conflict with the ReturnStatement indentation, similar to what happened on #443 - will probably need to improve logic to actually check if nodes at left/right of the operator will add indentation or not (for BinaryExpression, UnaryExpression and LogicalExpression)
maybe we can abstract the logic somehow, so we don't need to implement it also for AssignmentExpression, VariableDeclaration and ObjectExpression... - I wish the whole indentation logic was simpler...


ENYO-426: Marquee shows ellipsis but Does Not start rolling in moon.InputHeader Enyo-DCO-1.1-Signed-off-by: Artem Krasnyi<EMAIL_ADDRESS>

…nputHeader

Closing PR as Jira has been closed.


Pass-Through : Send welcome message to client

Problem Statement
When using the passthrough fonctionnality, the passthrough server don't transmit the welcome message send by the OpenVPN Management Interface.
For manual connection, it can be a bit confusing.
For script connection, it can be blocking because of the script waiting the welcome message, or the end of it, and never receive it.
Proposed Solution
In internal/openvpn/passthrough.go
On connection to Management Interface, read input and send it to client
Additional information
Example :
Following script is used to send command to OpenVPN Management Interface (Component of https://github.com/d3vilh/openvpn-ui) :
https://github.com/d3vilh/openvpn-server-config/blob/main/server/mi/client.go#L98
At line 98, it read the welcome message before sending command, but never receive it from the PassThrough
buf := bufio.NewReader(conn) 	buf.ReadString('\n') //read welcome message 	err = SendCommand(conn, cmd)
Acceptance Criteria
No response

Thanks for the report. It should fixed in the next version

Additional fixes may required for the UI: https://github.com/jkroepke/openvpn-auth-oauth2/pull/225


Update #(mapped_)hmget to match redis-rb when passed an array

I ran into this mismatch incidentally by miscalling hmget. I assume
the goal here is to match redis-rb as closely as possible - these
changes bring mock_redis into line with redis-rb in these cases.

Thanks for the fix, @Empact. I've fixed the RuboCop warnings and merged in 7a0cc31.
Will push a release shortly.

0.15.1 has been released.


GUI tests quick fix due to matplotlib 3.4 API change

The tests in #152 and #153 were failing, but this was actually down to an API change in matplotlib 3.4 that affected the GUI. This provides a quick fix to the GUI tests to make them pass. However with some new deprecation warnings from matplotlib it looks like we will soon have to adapt the GUI section accordingly.
Checklist:

[ ] I have performed a self-review of my own code
[x] My code is PEP8 compliant (flake8 anesthetic tests)
[x] My code contains compliant docstrings (pydocstyle --convention=numpy anesthetic)
[x] New and existing unit tests pass locally with my changes (python -m pytest)
[ ] I have added tests that prove my fix is effective or that my feature works

Unsurprisingly the matplotlib API change in 3.4 is backwards incompatible and our current CI setup uses a mix of matplotlib 3.3 (tests fail with this fix) and matplotlib 3.4 (tests fail without this fix).
@williamjameshandley you are more familiar with the GUI. Do you want to make the GUI tests robust against this API change? Or should we update the requirements with matplotlib>=3.4?
Note, I am not even sure whether this issue might at all affect the GUI or whether it only affects the tests...

@williamjameshandley you are more familiar with the GUI. Do you want to make the GUI tests robust against this API change? Or should we update the requirements with matplotlib>=3.4?

Note that matplotlib version 3.4 has not made its way to conda, yet.
Note further that requiring matplotlib>=3.4 also means dropping python 3.6 support.

Ok, with the latest fix tests pass for both matplotlib versions, 3.3 and 3.4.
Merging this into #152, #153, #155, #157 should make the tests for those PRs pass as well.


Basic Implementation of Personal Scheduler feature

program scheduler that can view and sort according to time

add an empty tab and route it properly such that it is accessable by the command
schedule

be able to add and edit items (in text) in the schedule page that willl be sorted according to time

fix up GUI for program scheduler

Hi Gary, just a gentle reminder to close this issue once the objectives are achieved. We can then move on to focus on intermediate level implementations. Cheers!


Loading image bug

My app is embedding tweet using Twitter UI.
And loading image in tweet, I have shown gray square at the upper left.

Steps to reproduce the problem: While loading image.
Expected behavior: I have shown gray square.
Android API Version: 23
Android Device: Nexus 5(Android 5.1), Nexus 5X(Android 6.0)

Below is screen shot,

Can you provide the Tweet Id? Also we use a grey square for the profile image as a place holder. If the image fails to load we also have a image to indicate loading filed.

@efrohnhoefer Thank you for your replying.

Also we use a grey square for the profile image as a place holder.

Actually, this gray square is used for the profile image as a place holder.
And also for be used image in tweet as a place holder. I think users looks like bugs because gray square is small compared to image.
Below is screen shot by twitter-kit-android/samples (TWEET UI > SINGLE TWEETS),
Before loading:

After loading:

This issue will be resolved in the next release (v2.1.0) of Twitter Kit.
Turns out one cannot use the same ColorDrawable for Avatar and Image placeholder.


Merge dev-ru to main to launch Russian localisation

Describe your changes
The Russian version of Glossary is now ready to be merged into the main branch.
Our team (@kirkonru & @tym83 & @shurup) has translated 24 terms (and other minimum required content) into Russian.
I have checked that the localisation pages work, by building the website locally and via the preview feature in the previous PR (#3029).
We have also prepared a blog post to announce this localisation.
Related issue number or link (ex: resolves #issue-number)
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

LGTM

Checklist to make localization live.

[x] validate all files which are not in the localization directory (content/ru/*)
[x] check i18n/ru.toml exist
[x] check l10n configuration in config.toml
[ ] acknowledge to initial localization contributors list
[x] check Preview works

Thank you, @seokho-son! I proposed these changes in #3037 (not sure whether I could do it in this PR directly).

Thanks @shurup
I will take a look again once the https://github.com/cncf/glossary/pull/3037 merged. (I don't have to approve it)
FYI, we need to avoid working on upstream branches (this PR is to merge upstream/dev-ru into upstream/main)
So, using another PR to update upstream/dev-ru is appropriate approach. :)
This PR will be updated automatically, once #3037 has been merged in to dev-ru.

Great, thanks for your explanation, @seokho-son! We are ready to proceed, I guess :rocket:

Hi @seokho-son! Now, with your suggestions being applied, can remove the hold label and proceed? Or is something else stopping us (e.g. more reviewers, etc.)?

Based on https://github.com/cncf/glossary/pull/3031#issuecomment-2044005020
, LGTM!!!
Thanks to all contributors!!


⚠ Bofrost.be has degraded performance

In 2dfe673, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1564 ms

Resolved: Bofrost.be performance has improved in a6be2b6.


Configuration Incorrect

The configuration file does not use underscores for the command line arguments.

Hello @jiaaluo going to check into this, thank you.

@jiaaluo the issue is solved, I had to create an alternative _launch.sh that uses the argument as intended.


Login "Remember me" seems invalid under Firefox

To verify this issue, I created a new blank Firefox profile with Firefox v53.0.2, no extension installed, complete default config, and then in Firefox and Chrome, both login to huginn with "remember me" option checked, and then close Firefox and Chrome both.
After 4 days ( I don't know whether or not it can be reproduced when only wait 1 day, anyway I waited 4 days ), reopen Firefox and Chrome, visit huginn again.
Test result:
Chrome remains the login state
Firefox is redirected to the sign_in, and need to input username and password.
Did anyone come across this same problem?

That is odd, I have not noticed it on my instance (but mostly use Chrome to access it). I will do the same test to verify my Firefox behaves the same as yours.

Testing

Test

My test ran for a very long time 😄 . For me Firefox kept me logged in to Huginn.
I am closing this issue. Please feel free to reopen if you need further assistance!


Is the src folder out of date to the current release version in marketplace?

I am trying to do some tweaks to the extension (have this https://github.com/gencay/vscode-chatgpt/discussions/67 question too). Looks like the OpenAI API support is present in the released vscode extension but not in the source code of this repo. Most probably same with package.json too. is this correct? Am I missing something?

Are you looking into this release? https://github.com/gencay/vscode-chatgpt/releases/tag/v3.0.0

@ppipada @ncesar thanks for the engagement here folks. I am keeping the source code in my private repository at the moment due to various reasons (I explained it on this issue as well #37) To reiterate, after my initial implementation I saw various clones without respecting the license of this project, for which I am putting so much effort during my free-time for the community to utilize chatgpt in vs-code for free and all of a sudden vs-code marketplace was full of copy-pastes of my implementation/design. My plan is to update this public repository for browser-based autologin flow once it cools down and I've got enough features to carry on on this project with the help of the community. I believe in the power of open-source and I promise the source code will be live at some point, but I can't say when just yet.
I try to respond to feature requests/issues in my best ability here and address them in the extension as fast as possible.
I hope that addresses your question here. I will separately respond to your questions on your other issue #67 @ppipada.

Thank you for your response @gencay.
Thanks for your awesome work.

Makes senses @gencay , I feel you.
Let me know if there is something I can assist with your private repo.
I do have JS experience but not with VScode, yet.

I totally respect your work choice but this should be noted in the README to avoid further confusion and misunderstanding. The produced extension being not actually the same as what the source code builds is very dangerous for an app asking for passwords and API keys.

@Songkeys thank you, I updated the Readme.md -> Disclaimers section. I understand your concern, however I think it's obvious, but I should still note that, this extension is purely running on your vs-code client without any server-side code that could intervene with your prompts, settings etc. There is a telemetry reporter that uses the vs-codes default instrumentation flow, so you can also opt-out if you prefer not to send anything to application insights (This is also noted in the disclaimer). Btw I also wanted the extension to warn the user not to store PII (Password/API Key/Email etc.) in settings.json and instead use the in-memory option for remembering the API Keys. Since you may be sharing your vs-code/computer with somebody else and they may see those settings in plain text, which is not secure. I should also note that you don't have to provide email/password to use the browser autologin method, you could just enter all of that information manually on the browser. Since the browser will open in a fresh/incognito window, I added those options to reduce the friction with using the extension so that you don't always have to manually enter, still you could do that though.
This is a hobby project without any monetary expectation but since I put so much of my time in here, I think it deserves the respect of not stealing/copy pasting code elsewhere, check out marketplace it's full of clones, I even saw an extension that copied the whole repository without changing a single code/readme and released it on marketplace :/. Once it's calmer out there about the ChatGPT, the code will be out in open source for sure, since I think it's where it belongs to! 🙏

I can attest, when I was first using this extension months back, there were already clones copy-pasting the repo without attribution.
I'd normally be a little sus about a non-paid project going private, but remembering that and Ali working on this basically non-stop for months, it's understandable.


Add background-position-x and background-position-y

These sound very intuitive but don't work in latest firefox versions. I found these are not standard, rather, these where originally implemented by MSIE only

https://drafts.csswg.org/css-backgrounds-4/#background-position-longhands

Firefox doesn't yet support them.
According to http://snook.ca/archives/html_and_css/background-position-x-y , they work in all the other major browsers, and don't work in Presto.

:+1:

Duplicate of #989


Corrected spelling and grammar mistakes

This pull request addresses the following error in the documentation:
"verified functional" -> "verified to be functional"
I hope my corrections will be helpful and contribute to improving your work. Thank you for your dedication and effort.

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

@detrina Thanks! Since this is a small/trivial change, we're going to leave as is for now 🙏


[Bug]: When a table column name is created with an invalid name, an error occurs when disconnect the database

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
479f61e
Other environment information
No response
Actual behavior
When using () or 1 or 1.1 as the column name and diconnect database the error occurs.
Expected behavior
No response
Steps to reproduce
#connect
        infinity_obj = infinity.connect(REMOTE_HOST)
        db_obj = infinity_obj.get_database("default")
        db_obj.drop_table("my_table")

        try:
            tb = db_obj.create_table("my_table", {(): "int"}, None)
        except Exception as e:
            print(e)

        # disconnect
        res = infinity_obj.disconnect()
        assert res

Additional information
No response

It looks fixed.


Fix unavailable streams

Add extra check for not available streams

Should resolve mentioned issues with streams in #13
The approach is different, to explicitly fail for extra visibility.


JavaScript Vendor Folder not compiling

@unwiredtech wrote:

Javascript Vendor folder is not compiling, I tried to add to folders of JS libaries but they are not being uglified or minified.

This sounds similar to the problem described in https://github.com/salcode/bootstrap-genesis/issues/116#issuecomment-156474326
Can you provide more information on the steps to recreate this problem?  Thank you.

The issue where that the vendor;
/js/vendor/particle-js/particle.js
/js/vendor/swipebox/swipebox.js

are not combined with javacript.min.js, I noticed this when the grunt did not change, so I assume that it was not added to the main js, BUT to my surprise both particle-js and swipebox are working.

any thoughts?

When you run grunt it should include all of the .js files in /vendor in javascript.min.js, this is because of the line https://github.com/salcode/bootstrap-genesis/blob/develop/Gruntfile.js#L58
If these libraries (particle-js and swipebox) are working, it sounds like they are being added to javascript.min.js
If I'm misunderstanding and you still feel there is a problem, you may want to create Git repo of the problem and I'll take a look at it.

Hi Salcode, Apologies for this late reply.
I think it is included in the javacript.min.js but it seems it is not appearing in grunt when compiling?
I turned over the project and the functionality is working anyway, so no worries.
Kind Regards,

@unwiredtech I'm glad you got it sorted.
I'm going to close this issue since the code is appearing in the compiled file.
If you have further questions about how the grunt task works feel free to open a new issue specific to that.
Thanks.


The magic of parallel-execution (pylint)

http://pylint.pycqa.org/en/latest/user_guide/run.html#parallel-execution

Letting this through. It sounds nice to have this process go quicker; I'll wait and see if it causes problems for anyone.


Does auth-management work with feathers v5?

As the title states, does auth-management work with feathers v5?

Tested it in a v5 app and it works.

Released feathers-authentication-management v5 right now. It's fully compatible with feathers v5 dove :)


Revert "Add preinstall message to Rush projects"

The preinstall script gets packaged into the release which breaks the released package. It was a nice idea (to show a helpful message if a dev runs npm install inside a package directory) but there's no hook that ONLY triggers when installing locally.
The error that occurs when trying to install a released package with this commit is:
><EMAIL_ADDRESS>preinstall D:\Git\js\sdk\servicebus\service-bus\node_modules\@azure\amqp-common
> node ../../../common/scripts/rush-welcome.js

internal/modules/cjs/loader.js:584
    throw err;
    ^

Error: Cannot find module 'D:\Git\js\sdk\servicebus\service-bus\common\scripts\rush-welcome.js'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:582:15)
    at Function.Module._load (internal/modules/cjs/loader.js:508:25)
    at Function.Module.runMain (internal/modules/cjs/loader.js:754:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>preinstall: `node ../../../common/scripts/rush-welcome.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\mharder\AppData\Roaming\npm-cache\_logs\2019-05-09T17_31_15_375Z-debug.log

@KarishmaGhiya Looks like we're going to have to re-cut a release for amqp-common (1.0.0-preview.5) 😢


Docs: Frutiger Neue For Post

Check if https://fonts.post.ch/frutigerneueforpost/#/ could be migrated to the Storybook docs.

Closing as no longer relevant


ear

What kind of issue is this?

[ ] Question. This issue tracker is not the place for questions. If you want to ask how to do
something, or to understand why something isn't working the way you expect it to, use Gitter
or Stack Overflow. https://gitter.im/openzipkin/zipkin https://stackoverflow.com/questions/tagged/zipkin

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed and stay fixed. If you have a solution in mind, skip raising an issue and open a
pull request instead.

[ ] Feature Request. First, look at existing issues to see if the feature has been requested
before. If you don't find anything, tell us what problem you’re trying to solve. Often a
solution already exists! Don’t send pull requests to implement new features without first
getting our support. Sometimes we leave features out on purpose to keep the project small.

Can you describe what your actual issue is? Perhaps change the title also to reflect that.

https://github.com/antirez/redis/issues/3909

Closing due to lack of response. We can reopen if details of an issue are provided.


I can't run the code you provided.

I tried to run demo.py.
Here is the complete code: python demo.py -i ./demo -m ../fh02.pth
But the following error is prompted:
Traceback (most recent call last):
File "demo.py", line 280, in
fps_pred, y_pred = model_conv(x)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\nn\modules\module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\nn\parallel\data_parallel.py", line 141, in forward
return self.module(*inputs[0], **kwargs[0])
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\nn\modules\module.py", line 489, in call
result = self.forward(*input, **kwargs)
File "demo.py", line 210, in forward
roi1 = roi_pooling_ims(_x1, boxNew.mm(p1), size=(16, 8))
File "D:\Development\DevelopmentProject\Python\CCPD-master\rpnet\roi_pooling.py", line 73, in roi_pooling_ims
output.append(adaptive_max_pool(im, size))
File "D:\Development\DevelopmentProject\Python\CCPD-master\rpnet\roi_pooling.py", line 35, in adaptive_max_pool
return AdaptiveMaxPool2d(size[0], size[1])(input)
File "D:\Development\DevelopmentProject\Python\CCPD-master\rpnet\roi_pooling.py", line 18, in forward
self.backend = type2backend[type(input)]
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch_thnn_init.py", line 15, in getitem
return self.backends[name].load()
KeyError: <class 'torch.Tensor'>
I run: python wR2.py -i ./CCPD_2019/CCPD2019 -b 4
The following error occurred:
Traceback (most recent call last):
File "wR2.py", line 226, in
trainloader = DataLoader(dst, batch_size=batchSize, shuffle=True, num_workers=4)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\utils\data\dataloader.py", line 802, in init
sampler = RandomSampler(dataset)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\utils\data\sampler.py", line 64, in init
"value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integeral value, but got num_samples=0
I run python rpnet.py -i ./CCPD_2019/CCPD2019 -b 4 -se 0 -f ./wR2 -t ./demo
and error:
Traceback (most recent call last):
File "rpnet.py", line 406, in
trainloader = DataLoader(dst, batch_size=batchSize, shuffle=True, num_workers=8)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\utils\data\dataloader.py", line 802, in init
sampler = RandomSampler(dataset)
File "D:\Development\DevelopmentEnvironment\Miniconda3\lib\site-packages\torch\utils\data\sampler.py", line 64, in init
"value, but got num_samples={}".format(self.num_samples))
ValueError: num_samples should be a positive integeral value, but got num_samples=0
Can you help me solve this problem? I will appreciate your help very much.

File "demo.py", line 273, in
fps_pred, y_pred = model_conv(x)
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/modules/module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 123, in forward
outputs = self.parallel_apply(replicas, inputs, kwargs)
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/parallel/data_parallel.py", line 133, in parallel_apply
return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 77, in parallel_apply
raise output
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/parallel/parallel_apply.py", line 53, in _worker
output = module(*input, **kwargs)
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/nn/modules/module.py", line 477, in call
result = self.forward(*input, **kwargs)
File "demo.py", line 229, in forward
roi1 = roi_pooling_ims(_x1, boxNew.mm(p1), size=(16, 8))
File "/home/Masx/CCPD/rpnet/roi_pooling.py", line 73, in roi_pooling_ims
output.append(adaptive_max_pool(im, size))
File "/home/Masx/CCPD/rpnet/roi_pooling.py", line 35, in adaptive_max_pool
return AdaptiveMaxPool2d(size[0], size[1])(input)
File "/home/Masx/CCPD/rpnet/roi_pooling.py", line 18, in forward
self._backend = type2backend[type(input)]
File "/usr/local/pytorch-3/lib/python3.5/site-packages/torch/_thnn/init.py", line 15, in getitem
return self.backends[name].load()
KeyError: <class 'torch.Tensor'>

You guys  need to  modify  type(input) as input.type() in roi_pooling.py, line 18 ,then it works.
This error  caused  by  api changes  of  different pytorch  version

ninghongbo123  is  right  ,  up

@ninghongbo123 yes it solved one error but later got other error once we fix this, may i know the pytorch version in which it runs.

home/miniconda3/envs/pytorch/lib/python3.7/site-packages/torch/_thnn/utils.py", line 27, in getattr  raise NotImplementedError
NotImplementedError

@ninghongbo123 yes it solved one error but later got other error once we fix this, may i know the pytorch version in which it runs.

home/miniconda3/envs/pytorch/lib/python3.7/site-packages/torch/_thnn/utils.py", line 27, in getattr  raise NotImplementedError
NotImplementedError

Figured out that i was using latet pytorch 1.1 and the model has been built on 0.3 so downgraded the package it worked, for anyone trying this out I recomend them to have a environment with below versions strictly so that they wont face any issue.
python: pytorch(0.3.1), numpy(1.14.3), cv2(<IP_ADDRESS>).
system: Cuda(release 9.1, V9.1.85)

@Naveenkhasyap The same error occurred. Can't solve the problem without modifying or reducing the environment? Is there any other way?

@Naveenkhasyap
Hey, can you let me know how exactly did you install
python: pytorch(0.3.1), numpy(1.14.3), cv2(<IP_ADDRESS>).
system: Cuda(release 9.1, V9.1.85)
I am trying to use conda to install but facing a lot of issues.
Please let me know. I have been stuck on this for a week now.


externally_connectable, iframe+cookie discovery

how it would work.
some initial detection detail still needs work

closing as not planned


feat: subtree get/insert

This is an attempt to implement simple insert and get operations for GroveDb (inside subtrees without exposing to public API yet) to store binary data

Very good initial work guys!


Switch to good-web-game completely

Though historically good-web-game was created for the web, now it's powered by rapidly improving miniquad and works fine on linux/macos/win/wasm and Android.
Also, I would be so happy do get rid of this ugly crate swapping hack))
https://github.com/ozkriff/zemeroth/blob/b74f952b0d8b6ec2a2b4c632222209117279b087/src/main.rs#L3-L6
good-web-game is a kinda a half-measure: I'm planning to eventually migrate to miniquad completely.

Also, this should allow me to get rid of the massive nalgebra dependency.

https://github.com/not-fl3/good-web-game/pull/36 ("Add Conf::physical_root_dir")


feat: escape certain characters in file paths passed to tasks

This PR adds --escapefor escaping of certain characters from the staged file paths before being passed to tasks. Should fix https://github.com/okonet/lint-staged/issues/676.
Currently the escaped characters are as follows:
\s, ", ', `, $, *, \, [, ], (, ), {, }

Ah, Windows tests completely failed, so back to the drawing board.

You might need to check path.sep or other path methods that can help you making sure things are working correctly on windows too.

Any updates on this?

@hollandben Sorry, after finding out it's not that simple on Windows, I haven't took the time to update this. I think the scope of this 'fix' is worthy of being its own npm package.

Anyone want to take over this one?

Closing as stale.


fix: fixing nw.all() doesn't work in sum_horizontal

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[X] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #1071
Closes #1071

Checklist

[X] Code follows style guide (ruff)
[X] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.
Fixing bug for sum_horizontal (see #1071)

@MarcoGorelli we still have to find a way to make it work for Dask, but do you want to have a look at the implementation of pandas_like to see if it is what you was thinking?

yup, thanks @Cheukting ! this is great
if you wanted anything for the sprint, there's a few issues which would be good (#1000, #1004, ...)
i'll just add dask / pyarrow so we can make a release 🚀

Thanks @MarcoGorelli (ship it!!!!)

:thinking: why isn't the quokka gif showing

Ship it!


MIDI timestamping

Use a timestamp on MIDI events, which permits to operate nicely at higher latencies.

Implemented; need some testing.


PartialBooleanBlock rectangle line too thick?

The model
Modelica.Blocks.Icons.PartialbooleanBlock
contains a rectangle having:
lineThickness=5.0,
Is this value true? It seems too large ( I intend 5 logical millimetres, i.e. 5 mm on a PC with default 96 logical dpi and a screen having actually 96 DPI).
Dymola does not draw this rectangle at all (maybe  with a logic stating not to draw rectangles when
borderPattern=BorderPattern.Raised) OM, which starting from two days ago is DPI-aware also in drawing lines, draws the rectangle using a huge line.
What is the intended behaviour? Maybe the rectangle outline may just be omitted (as in Dymola's behaviour)?

I believe this is a specification issue if the rectangle line is to be drawn if the border pattern is note None. However, the MLS 3.3rev1 already states in <IP_ADDRESS>

The border pattern attributes  Raised, Sunken and Engraved represent
frames which are rendered in a tool-dependent way.

Thus it is not obvious if we should clarify the tool-dependent way. @HansOlsson Do you think this needs to be clarified more? If yes, we would need a Trac ticket then.

Even in the case in which no change in the MLS is needed (because of the tool-dependent nature of border pattern) there is a MSL issue, since 5 is a VERY large thickness, as I said.
If tool dependency means that a tool could can decide whether draw the rectangle or not, then the thickness should be wiser, e.g. 0.25 or  0.5  (mm).

While waiting for possibile specification modifications, is it possibile.to change the thickness in the MSL trunk? It is needed anyway.

Resolved by 4a80406f459d76d241bce4f5640d7fe3a9dfadc1 in master branch.
Sorry for the delay. I was hoping to receive some feedback from the library officer first, but he kept silent.

There are other model icons with 5mm thickness.
This is my list:
Blocks.Icons.PartialDiscreteBlock (fixed)
Blocks.Icons.DiscreteBlock
Blocks.Interfaces.PartialBooleanMISO
Blocks.Interfaces.PartialBooleanThresoldComparison
Blocks.Interfaces.PartialBooleanComparison
Blocks.Interfaces.PartialBooleanSISO_small
Blocks.Interfaces.PartialBooleanMISO
Blocks.Interfaces.PartialIntegerSISO
Blocks.Interfaces.PartialIntegerMISO
Blocks.Interfaces.DiscreteBlockIcon
Blocks.Interfaces.PartialBooleanBlockIcon
Blocks.Sources.RealExpression
Blocks.Sources.IntegerExpression
Blocks.Sources.BooleanExpression

Blocks.Icons.DiscreteBlock should also be fixed by 4a80406f459d76d241bce4f5640d7fe3a9dfadc1.

Resolved 4687753a886fb29b489f36f9587536f38abdb01d in master branch.


fix(tools): Bump cornerstone-tools to 3.9.0

Fix display function for elliptical and rectangle roi tools
Add tool definitions for freehand and circle roi

@allcontributors please add @evren217 for code


Introduce 'ExtraHeaders' capability across all Commands

Hey, I have a situation where I want to include a nonstandard HTTP header on Jenkins requests which authorize them to get past a firewalling proxy.
I didn't see a way to attach custom headers using this library so I added an extension point in this PR to accept arbitrary key/value pairs at the JenkinsClient level.
I'm not sure if there's a decent way to test this, as it seems the tests assume a localhost Jenkins, and Rider isn't hooking up to xUnit for me to begin with. Perhaps the best way to test this change would be manually constructing and passing an Authorization header instead of passing Username and ApiToken (so, Authorization: Basic Z3Vlc3Q6YmNiOTU0YTc3YWI0Nzc1MDIwMWE5ZjE4OGIxYjI1ZTg=)
Indeed, this PR offers a superset of the Username/ApiToken functionality.

A possible different approach could be allowing a global OnWrite callback. Even more flexible but also easier to go wrong

nice

Since this project is already using Basic Authentication through HTTP using the JenkinsClient.Username/Password, I'm not sure exactly what change your after; you should be able to use the existing Username/Password properties to create a basic authentication header. Are you using an alternate header name than Authorization?
This looks good though, but I'm not sure if it's the right approach. Instead I would prefer to add a "BeforeRequest" event that could be used to modify the HttpRequest object directly; this should be a simpler implementation that allows much more control.

I've implemented the above suggestion in #24 - please let me know if this meets your minimum requirements.

Hey, sorry for the delay, just got around to testing this on my side again. Turns out my PR is incomplete so I've switched to the implementation you generously implemented into the 1.0.5 release branch.
I had to make a small patch for full coverage, as RunAsync didn't get your OnBeforeRequestSend call added:
diff --git a/Jenkins.Net/Internal/JenkinsHttpCommand.cs b/Jenkins.Net/Internal/JenkinsHttpCommand.cs
index fc62312..9719e54 100644
--- a/Jenkins.Net/Internal/JenkinsHttpCommand.cs
+++ b/Jenkins.Net/Internal/JenkinsHttpCommand.cs
@@ -52,6 +52,8 @@ namespace JenkinsNET.Internal

             if (OnWriteAsync != null) await OnWriteAsync.Invoke(request, token);

+            client.OnBeforeRequestSend(request);
+
             using (token.Register(() => request.Abort(), false))
             using (var response = (HttpWebResponse)await request.GetResponseAsync()) {
                 token.ThrowIfCancellationRequested();

Other than that, #24 looks good from here! I'll go ahead and close this PR.

Oh man, nice catch. Thanks for getting that together and following up, I'll get another release out asap.


Add perfect-numbers exercise

Add perfect-numbers exercise

@abo64  - I pushed up some changes.

The example was modified to be more functional
I change the test suite to be property based. I have never written a ScalaCheck property based test, so please provide any feedback.

Thanks

Thanks @abo64 .
Implemented last set of suggestions..

thanks @ricemery


Update Nix module docs

Autogenerated by https://github.com/theCapypara/riptide-all/actions/runs/11011014324

Included in #6


dfu: flash_img: Constify flash_img_buffered_write() data parameter

The data passed has no reason to be changed and can therefore be passed
as read only.
Signed-off-by: Xavier Chapron<EMAIL_ADDRESS>

@XavierChapron could you please rebase this PR?

@XavierChapron could you please rebase this PR?

@ioannisg Done! :)


Deal with LoRaMacStop busy return code

In AT commands that stop the LoRa MAC via LoRaMacStop, we need to deal with cases where the function returns busy.

Done. There is only one command that invokes LoRaMacStop directly: factory_reset.


u8string

u8string is not able to convert to string anymore in new c++ standard.

Also this not support cyrillic


Better error checking for offline use.. Add in more reliability

If you are offline are trying to translate - or its too slow or some other error - the error capturing right now isn't nice. We should try and do things to capture and quietly present errors
Likewise with Azure TTS
We should have an option to turn on or off these easygui message boxes. SOme people who have disabilities wont be able to close them (or alternatively, a way of it autoclosing the dialog after a set time
In general I think the code could do with some refactoring to look nicer. It started off as a script but its getting quite hard to read now

If you are offline are trying to translate - or its too slow or some other error - the error capturing right now isn't nice. We should try and do things to capture and quietly present errors

We should have an option to turn on or off these easygui message boxes. SOme people who have disabilities wont be able to close them (or alternatively, a way of it autoclosing the dialog after a set time

https://github.com/AceCentre/TranslateAndTTS/commit/bb4c19a339634e64b6068f060ecf14361b2daaad
Replace easygui with pyside6 | Messagebox has auto close feature (Default is 10 seconds) | Add Error handling when there is no existing settings.cfg and user click yes on launch the "Configure TranslateAndTTS" but discard the set-up. Normally, It will raise some error since no config file was found, but using this fix, a message box will notify the user that there is no config file created and the app will exit | Remove Pygame welcome message


Build packages also for the Alpine linux

What do you think about building packages also for the Alpine linux? It seems that nfpms does support it, just by adding apk to the formats option.

I am alright with it, if you have some time and would like to open another PR i will merge it.
Also if you are opening another PR it will be great to remove the publisher block https://github.com/kha7iq/subvars/blob/master/.goreleaser.yaml#L92 .
Once these changes are merged i will create a new release


Add semantic class for reduced markup

By design, Bootstrap supports opt-in styling via explicit classes, rather than implicit styling via semantic HTML.  Opt-in styling is more robust, because implicit styling can be difficult to undo / opt out of, and may cause conflicts with third-party code.  However, styling entirely via explicit classes can result in more verbose markup.  The semantic class provides a middle ground between the two approaches, enabling implicit styling in limited scopes.  It changes nothing about the way Bootstrap currently works, but adds a way for users to style their content via semantic HTML.
For example, styling breadcrumbs via explicit classes:

Home

Library

Data

Versus styling breadcrumbs via the semantic class:

Home

Library

Data

The list of semantic implication rules in this commit is small.  It is intended to grow with community input.

TODO:

[ ] docs?
[ ] tests?
[ ] relocate Sass variables?
[ ] additional implication rules?

What you're after isn't about semantic vs explicit, it's strictly about implicit vs explicit.
Both options are 100% semantically correct—you have an list of items who's order matters. This is the correct markup regardless, and the selectors created for it—whether classes or elements—carry no semantic value. HTML has semantic value, CSS does not.
This is purely a difference in implementation and project-wide approach. To us, based on our experiences, an explicit set of classes is the wiser move for writing frontend code. Yes, there are options to reduce the number of classes, but flipping between them seems like it'd only add confusion for folks.

I agree with @mdo. We could provide options to change our code methodologies, but it ain't worth the additional complexity we'll introduce in our codebase.

What you're after isn't about semantic vs explicit, it's strictly about implicit vs explicit.

Yes, I was trying to convey explicit (which could be done with all div tags, if desired) vs implicit (which would only be done with semantic HTML).  I'm sorry that my post was worded poorly.  If the class was renamed from semantic to implicit, would that make this proposal worth more consideration?  Otherwise, could you expand on "there are options to reduce the number of classes"?
To be clear, I think it's valuable that Bootstrap can style any given markup, using explicit classes, because that ensures it can work with any third-party libraries.  But, on the other side of the coin, (optionally) fewer classes would make Bootstrap plug-and-play with many third-party libraries which generate semantic HTML.

What you are proposing is how Bootstrap before v4 worked: you've added, say, dropdown-menu to ul and all ul li a turned into a nicely styled dropdown menu links... but if you wanted to include li with something else, you've had to write extra CSS to reset styles that dropdown-menu added. If you wanted to reuse styling elsewhere, you had to copy-paste it over. It may not see like much, but this stuff was becoming tiresome real fast, so people started asking for more explicit approach that would make it easier to customize, reuse and extend Bootstrap.

@rafalp I believe this proposal is different than how Bootstrap previously worked, because here the cascading styles would be opt-in via the semantic class.  In other words, if you don't want ul > li automatically styled, simply don't include the semantic class on the ul.  (No need to do a reset.)


github workflow to update azure deployment

https://github.com/cs-discord-at-ucf/lion/commit/d5a3f15b7c169df677f7f02d524d976f822ffe28