Add the 3th generation iPad pro to the device name list

Description
I've added the 3rd generation Ipad pro to the list with available devices. This fixes a problem where it just returns "iPad" for these devices.
I've specifically added 3th generation to the name of the device. This way you can actually tell the difference between the multiple versions of the iPad Pro. This is a must have because it's the only way to tell the difference between the iPad pro with physical home button (generation 1 and 2) and the iPad pro without physical home button (generation 3). I kept the name of generation 2 the same for now but I can rename it for consistency.

Thank you for the contribution!
This looks like the right thing to do given the current style, but I have to admit, it seems silly that we are maintaining this database - it doesn't seem like the best use of our time trying to keep up with apple product launches.
How would you feel, as someone clearly consuming this information, about converting to a not-written-yet parallel API that just gave you the unfiltered device information, as described here: https://github.com/react-native-community/react-native-device-info/issues/585#issuecomment-478345235
It seems like the logic you'd have to do on the friendly names we are returning to handle edge cases would not be that much different if you were just doing the lookup in your own table, then we could stop trying to keep up with Apple.
If you think that could work I would take a PR for that and merge/release it immediately. Otherwise I guess it's more table updates here, forever ;-)

Yes I agree, I'm actually fine with a function that just returns the device id. I can maintain the list with device name in my own project if I need it (although I highly doubt it, since you can check on the device ID as well).  The only downside is that I think that some users still want to get the device name from this library.

It appears that device id is already available as getDeviceId() so you might be set then. I understand users might want to get the model mapping here but it seems like a race we can't win. I need to think about it a bit more to see who is actually consuming the info or if there is a better way. There may be no better way and this is how we need to do it

I slept on it and here's what I'm thinking:

the docs for getModel should contain a warning for iOS that it is community-maintained, it might lag releases, and getDevice is the most reliable, but we accept PRs
I will freely merge getModel PRs but won't update it myself, I think this is a community benefit and doesn't take a lot of effort as long as the community understands what they are getting with the API

I think that's a good balance
So for this one I need a changelog entry (just make it "next" for the version) and if you could add the warning on the getModel API like above, that would help. Then I'll merge this in and do a release

I've added a note to the changelogs and readme. The same problem is with getNotch(), which is also maintained by the community.
I can also add a console.warn when using getModel(), but I'm not sure if we want that now. Maybe in the future when you want to deprecate the function.


Added the config template json file and updated readme

Signed-off-by: sayantan1413<EMAIL_ADDRESS>Issue Number
fixes #154

Describe the changes you've made

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)

Additional context (OPTIONAL)

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

Provide a Deployed link of route/page that needs to review
Preview: Deploy preview link here with the appropriate route
<!-- Please add deployed link to here text >

Hey, @vinayaksh42  please review this PR.


No notification if app is terminated

Notifications are not displaying when the app is terminated. I tried creating the notification from FCM onBackgroundMessage method. works fine when app is minimised.

exact same problem


Fix SwizzleArray broadcasting

Hi @willow-ahrens,
In the eager mode, I think there's an issue with broadcast(fn, ::SwizzleArray, ::SwizzleArray). It complained about missing functions, but with them in place it fails when performing copyto!.
The test/test_issues.jl change reproduces the issue.

The issue here is more that the swizzle wasn't declaring a FinchStyle broadcast style. I added a definition for that.
In general, most of the "interface" overloads for Tensor (i.e. getindex, setindex, map, broadcast, reduce, copy), should also be defined for SwizzleArray, PermissiveArray, OffsetArray, etc. I think the right way forward on that is to add an abstract class like CompiledArray that everything can inherit from for the purposes of defining interfaces in terms of the Finch compiler.
The fixes I've added so far should be good for now though!

Also, I've chosen not to define length on Finch tensors because most of the tensors require cartesian indexing to work properly, and length tends to be used for linear indexing. If at some point we want to do a linear indexing + length PR we could add them though!

Thank you! In this version broadcast with SwizzleArrays works for me!
I think the error comes from copyto!(A, A_sw) line.

Oops! Yeah I forgot how julia Array cannot be resized


Prefer the local image name of the same subscription to global image alias such a "RHEL" to avoid unintended  charge.

When creating a VM from an image, we can use aliases defined here.
https://github.com/Azure/azure-cli/blob/4c98204f3baf42a021c02f691e310e651e5ce546/src/command_modules/azure-cli-vm/azure/cli/command_modules/vm/_actions.py#L70-L93
(test code) https://github.com/Azure/azure-cli/blob/011e1eaa0902ac7da47d473b989417f00730cf47/src/command_modules/azure-cli-vm/tests/aliases.json
However, we also create an image with the same name of these aliases. So I can create my customer image with name "rhel".Then I intend to create a new VM from this customer image with the command.
$ az vm create -n testvm  -g rhel-base -l japanwest --image rhel --storage-sku Standard_LRS --size Standard_A1 --authentication-type password --admin-password clouduser

Though I intended to create VM from my custom image, but the VM is created with RHEL VM on the marketplace. It means it costs extra unintended marketplace charge.
I know I can specify the resource id of an image to create new VM with my custom image. But, users may think they can specify the image name if the location and Resource group of the VM is same as those of the custom image.
I have some ideas to improve this situation. Could you tell me your thought?

Azure CLI 2.0 always prefer the image name when the local image name has the same name of aliases
Azure CLI 2.0 outputs the warning message, when the user create a VM with one of aliases and the user, has the image whose name is same of the alises.
Azure CLI 2.0 does nothing about this issue as it's not CLI specific issue. But the document should be improved. (in this case I'll create a documentation fix PR to azure docs repo)

Hi @tanaka-takayoshi, I think your suggestions make a lot of sense.  The order

Most VM/VMSS create use images defined in the alias doc. The number I was told is 80%+. Hence, I am not convinced we have to search managed images first, which incurs one extra rest call (mostly not useful and hurts the command perf). Also, it is possible other people prefer aliased common images, so this change will break them. Last, using a managed image is till very minority scenario, so I am not convinced we have to pull the trigger right now.
If using a resource id is inconvenient, I suggest use a new image name when you create it, be more specific is always good. Please let me know if picking a new image name is not applicable as well.
Agreed, we should update doc to make the logic clear.

I have already added debug statements in the VM/VMSS create command (visible with --debug) which show how the storage profile is resolved (i.e. managed image vs. marketplace URN).

Given the discussion above and that the "storage profile" of the image is emitted in debug output, I will be closing this issue.
I would also like to add that If you really need to disambiguate between custom images and managed images, you can use a different name from the alias. Moreover, you can ask your customers to confirm that the result of az vm show -n exampleVM has the correct storage profile. If the vm was created from an image, the imageReference under storageProfile would show details for a managed image. Users can run:
az vm show -n exampleVM-g ova-test-1 --query "storageProfile.imageReference"
If the current documentation for --image can be improved, feel free to let me know how it could be improved or to make a change to it yourself.


fix log level for async queries

https://github.com/biothings/biothings_explorer/issues/751

Superseded by #10


Add kubernetes-mixin runbooks

fixes #8 and a lot of other dead urls.
todo:
fix references between files.

I guess certain sections should be extracted and added to general section, especially Pod debugging.

@paulfantom ping

After slack talk it is better to close it and split into smaller commits.

yeah, AFAIR it was not merged yet

Hi @paulfantom is there any update on this?


Fix connection timeout check to broker

I spend around 15min trying to find a better solution to fix this and failed.

Strange. The original implementation looked nice, this feels like a downgrade. Do you consider this an upstream bug in hyper? Would you consider opening an issue there?

I imagine it has to do with all the different connectors we are using and one of them does not tell hyper that it is infact a timeout.

It also really does not help that this error type from hyper is not really public. So it is difficult to match on it.

While cursed, this solution nicely encapsulates the cursed parts and gives sufficient context on why this code exists.


Update Xenko.Assets.Presentation.es.po

Now is perfect, at least the gamestudio part. Some parts of the UI stil remains in english.
PR Details
Fixing typos at the spanish translation.
Description
Changed "ajustar transformaciones a este valor" to a much accurate "ajustar traslación a este valor" & "ajustar rotación a este valor" gizmo option.
Related Issue

Motivation and Context

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Thanks for your contribution.
Can you please do those changes on weblate?
Link: https://hosted.weblate.org/projects/xenko/
More details in https://github.com/xenko3d/xenko/blob/master/docs/localization.md


Creating the Webhook and creating intent and entities in dialogflow

@kmehta2228 and @nivetha-kuruparan
Please work with Backend Team and create new intents.
And we have @marwa-khafagy in our team. You can explain dialogflow to her and I believe she can also help with creating and updating more intents.


Can't delete some characters in the editor

List of known bad characters:
🔥
These are windows specific "emojis".  Find out how to disallow or make benign.

Looks like it could be related to the Microsoft Edge browser.


Theme Editor Improvement Suggestions

As mentioned by a lot of people in various places(IRC, Discord). The Theme Editor needs some serious bit of work to be a proper tool for creating Themes and editing them.
I will list some of my ideas here to do the same,

Fix the usability bugs like #29059, #29056, #29038, #29034
Preview and Editing options in the Add Dialog too, currently the process of adding new Theme Options is convoluted, add a new Item, go to the settings section, adjust its settings, preview. This could be simplified if we do a quick preview and editing tool inside of the Add Dialog itself.
More templates for the Editor to help quickly generate various types of GUIs. Such as creating something that uses Texture StyleBoxes, which is ridiculously painful to do at the moment.
Make editing sub-resources like Stylebox much easier.

This is just some of my ideas feel free to push forward more suggestions.

-> Also made some PRs to fix a few of the issue will probably use some suggesttions to fix more issues and do some feature requests when I have time.

See also #29035

See also #20123
This is not related to Theme Editor itself but is also related to Theme feature.

@williamd1k0 Aah that's pretty annoying will look into it.

Closing this as feature proposals and enhancements are now tracked in the proposals repository. https://github.com/godotengine/godot-proposals/issues/1438 can be used for further discussions.


Parsing log output

Steps to reproduce the problem:

mitmdump --mode reverse:https://<IP_ADDRESS>:443 -p 8082 -v -w ./w1.log --rawtcp --tcp-host <IP_ADDRESS> --ssl-insecure -s ./tcp_message.py > stdout1.log

Any other comments? What have you tried so far?
I would like to write a program to parse the output in stdout1.log (the redirected file named above). But I noticed this anomaly (seen here and elsewhere in the log): the initial 'P' as in 'POST' is interrupted by 1 line of output from mitmdump and 1 line from your example script tcp_message.py. The following line continues the 'OST' from 'POST'.
To simplify parsing this output, is there a way to prevent this behavior, e.g., flushing log after each write?
Thank you.
P
<IP_ADDRESS>:60377 -> tcp -> <IP_ADDRESS>:443
[tcp_message] from client to server:
OST /sdk HTTP/1.1\r\nSOAPAction: urn:vim25/5.5\r\nContent-Type: text/xml; charset=utf-8\r\nCache-Control: no-cache\r\nPragma: no-cache\r\nUser-Agent: Java/1.7.0_80\r\nHost: <IP_ADDRESS>:8082\r\nAccept: text/html, image/gif, image/jpeg, *; q=.2, */*; q=.2\r\nConnection: keep-alive\r\nContent-Length: 439\r\n\r\n
<IP_ADDRESS>:60377 -> tcp -> <IP_ADDRESS>:443

System information
Mitmproxy: 4.0.3
Python:    3.6.5
OpenSSL:   OpenSSL 1.1.0h  27 Mar 2018
Platform:  Linux-2.6.32-642.11.1.el6.x86_64-x86_64-with-centos-6.9-Final

<IP_ADDRESS>:60377 -> tcp -> <IP_ADDRESS>:443
[tcp_message] from client to server:
OST /sdk HTTP/1.1\r\nSOAPAction: urn:vim25/5.5\r\nContent-Type: text/xml; charset=utf-8\r\nCache-Control: no-cache\r\nPragma: no-cache\r\nUser-Agent: Java/1.7.0_80\r\nHost: <IP_ADDRESS>:8082\r\nAccept: text/html, image/gif, image/jpeg, *; q=.2, /; q=.2\r\nConnection: keep-alive\r\nContent-Length: 439\r\n\r\n
<IP_ADDRESS>:60377 -> tcp -> <IP_ADDRESS>:443

Hi,
I'd definitely recommend against parsing mitmproxy's stdout output. You can write your own addon that just dumps things into a file in the format you prefer. Here's the addon that generates the stdout output: https://github.com/mitmproxy/mitmproxy/blob/master/mitmproxy/addons/dumper.py


fix the path of kubeedge-web-controller-app.yaml

some file paths in the instructure are incorrect. now fix them.

@fisherxu please help to review this pr. thanks.


Cannot translate complex object filter on the right of where clause in aggregation sets

Not atm @imansafari1991 - sorry I've been fairly busy and haven't had the bandwidth to get to this.


[release-1.5] Make leader election a top level pilot component, use it for singleton components

This is an automated cherry-pick of #20809

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.5
e973b417d16104372b72f420fdfca6bb8e54d477
link
/test lint_istio_release-1.5

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

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.5
a36d1ed0fbc8e351f8e9407133f0e6a51803358a
link
/test unit-tests_istio_release-1.5

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

/retest


Ebola Healthcare Facilities in West Africe needs Access and Use Constraints

Ebola Healthcare Facilities in West Africa id=d1a00d57cea2458a887d3a6b4052952b needs Access and Use Constraints
@karlterrey can help with demo content wording; however, we still need to cite data

I have sent @karlterrey a quick look document with metadata and access and use constraints for this service. We shall see what legal has to say about it.

I have deleted all ebola related content.


feat: length, reduced words in Coxeter groups

Add new file GroupTheory/Coxeter/Length.lean. Define the length function and reduced words of Coxeter groups. Prove their basic properties.
I decided to split #11408 (now closed) into two pull requests. This is the first one.

[x] depends on: #11406

bors d+

bors r+


Tong

changed the link

@TongHere please rename the PR name at the top so we know what is the content 😄 thx!


[Feature request] QR-Code scanning for importing MFA app passwords

No idea how easy such a thing would be to implement, but just an idea.  Nextcloud has that feature to turn the app password into a QR code.   I know you have that checkbox under advanced to help facilitate mfa users, but I wasn't really sure how it was suppose to work.  I would think the user would simply just enter in the generated app password from their nextcloud which was I ended up doing.

Hello
I think you used the functionality correctly.
If you have MFA active you need to create the appPassword yourself and select the specific option.
If you do not use MFA the app will generate an appPassword on its own on the first login, and from then on only uses the generated appPassword to continue.
I like the idea behind the QR code scanning but it will be very low prio, as users with MFA can already log in and it is not that big of a deal. But i will keep it on here as a feature request.

Hello
I think you used the functionality correctly.
If you have MFA active you need to create the appPassword yourself and select the specific option.
If you do not use MFA the app will generate an appPassword on its own on the first login, and from then on only uses the generated appPassword to continue.
I like the idea behind the QR code scanning but it will be very low prio, as users with MFA can already log in and it is not that big of a deal. But i will keep it on here as a feature request.

Oh I see.  that is neat, and a great idea.
As for the QR code, agreed, totally low priority.

Where I was confused,
It's the phrasing for the checkbox:
"Use self generated App Password
Needed for accounts with 2FA"
You of course mean that self generated app passwords are needed for 2FA users.  In my mind, I read that as the checkbox was needed for 2FA users and was confused as I couldn't see how that checkbox would help a 2FA user.  It doesn't really clarify that the feature is really meant to improve security for non 2FA users.

@joleaf already implemented that feature in his NC Passwords Mobile App, a nextcloud app also written in flutter. Perhaps you can use this as reference implementation or even copy&paste:

issue: https://gitlab.com/joleaf/nc-passwords-app/-/issues/43 (title is a little bit misleading :-) )
merge-request: https://gitlab.com/joleaf/nc-passwords-app/-/merge_requests/83


SharePoint Provisioning Failure

provisioning action correlation ID: 2ac27b21-e387-4f09-a8e7-346fd3f2a7ff !

Hi,
Thanks for using the Look Book site. Based on the event log, it looks like there was an issue uploading/updating the SPPKG of the solution in the App Catalog. Please, verify that you have proper permissions to access the App Catalog, and eventually remove the already existing SPPKG solution if you are upgrading the Learning Pathways site on your target tenant. Once you have done that, please try again and, in case of any failure, share the updated Correlation ID (by creating a new issue) for us to double-check the flow. Thanks.


Dining: OceanView location is missing from app

This location is now coming across the HDH feed but not making it into the feed consumed by the app. Investigate campus-eateries.json

Emailed HDH group on how to handle this. Waiting to hear back.

@a6wu This turned out to be an issue on our end. This commit has been pushed and is ready for deployment to AWS Prod. The updated file in question is: campus-eateries.json
https://gitlab.com/ucsd-wts-services/dining-service/tree/521efe36e903daaceae80b5dc5db66e45adba2c2

Fix pushed to AWS Prod and verified, closed.


Uncaught Error: ENOENT: no such file or directory, lstat '/private/var/folders/pg/5zpnd23s2gxfwqm...

[Enter steps to reproduce:]

...
...

Atom: 1.23.0 x64
Electron: 1.6.15
OS: Mac OS X 10.13.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT: no such file or directory, lstat '/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app'
At fs.js:902

Error: ENOENT: no such file or directory, lstat '/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app'
    at fs.lstatSync (fs.js:902:18)
    at Object.fs.lstatSync (ELECTRON_ASAR.js:244:16)
    at realpathSync (fs.js:1542:21)
    at Object.fs.realpathSync (ELECTRON_ASAR.js:331:24)
    at toRealPath (module.js:130:13)
    at tryFile (module.js:126:22)
    at tryExtensions (module.js:138:22)
    at Function.Module._findPath (module.js:190:20)
    at Module._resolveFilename (module.js:468:25)
    at Module._resolveFilename (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.get_Module._resolveFilename (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/app/src/module-cache.js:354:58)
    at Module.require (/app.asar/static/index.js:40:43)
    at require (internal/module.js:20:19)
    at Object.get [as shell] (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/electron.asar/common/api/exports/electron.js:9:18)
    at get_shell (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/app/src/application-delegate.js:20:44)
    at ApplicationDelegate.module.exports.ApplicationDelegate.openExternal (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/app/src/application-delegate.js:398:20)
    at WindowEventHandler.handleLinkClick (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/app/src/window-event-handler.js:240:38)
    at HTMLDocument.innerHandler (/private/var/folders/pg/5zpnd23s2gxfwqm9ntmmq3nm0000gp/T/AppTranslocation/18C0E688-1745-41B4-ACC2-500541381175/d/Atom-2.app/Contents/Resources/app/src/delegated-listener.js:10:17)

Commands

Non-Core Packages
pinegrow-atom 0.3.1

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #10774. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

I went to the page/issue you referenced, but it appears there is no resolution of the glitch. So do I just uninstall Atom until this is resolved?
From H

On Dec 12, 2017, at 4:52 PM, Robert Sese<EMAIL_ADDRESS>wrote:
Closed #16363.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey @githubH - go ahead and try uninstalling and then reinstalling and let me know if you still have trouble after that.


Fixes memory leaks in CT code

Two leaks in ct_test.c and one in SCT_validate().

I think there may be more CT-related memory leaks. When building with enable-crypto-mdebug and running:
./ssltest -s_key keyU.ss -s_cert certU.ss -c_key keyU.ss -c_cert certU.ss -bio_pair -tls1 -requestct

I get a lot of errors from the internal memory leak detector, but if I remove the -requestct option I don't get any error anymore.
Unfortunately the memleak detector output is not very user-friendly so I've been unable to pinpoint the cause of this, and it may be that the cause isn't the CT code after all.

The problem described above seems to be what is causing most of the Travis build failures right now btw.

Yes, this doesn't fix all of it:
https://travis-ci.org/openssl/openssl/builds/114229318
I removed crypto-mdebug from sanitizer builds; looking at the logs from those shows that the leak is somewhere in ASN.1, but I couldn't at a glance figure out where it was coming from.
http://clang.llvm.org/docs/AddressSanitizer.html#id4 explains how to get a more meaningful stack trace in a local build.

From what I can make out from the Travis failures, those are the same leaks that valgrind picks up but I disregarded because CT code doesn't appear in the stack trace. If these are the ones that you tracked back to https://github.com/openssl/openssl/commit/dd696a55a2554cc8c89dd64f7e1171ce211dfc5c, then perhaps the way the functions in crypto/ct/ct_oct.c and registered with the x509 code (in crypto/ct/ct_x509.c) obscures them from valgrind and ASan's stack traces. I'll comb through them and see if I can spot any leaks.

Yes, I suspect the ASN.1 indirection is hiding the relation to CT. These failures only show up in CT tests, and git bisect blames the commit that added these tests.

I suppose these never turned up sooner because, while parts of that code have been in for almost a year, I don't think there were ever any tests exercising it. I'll let you know what I find.

OPENSSL_NO_UNIT_TEST guarding, apparently, is needed if you need to access library internals that aren't normally exported. (Because that used to break Win builds otherwise.) You probably don't need it.

OPENSSL_UNIT_TEST exposes a function table that jumps to internal fucntions.  It was back from when Mike Bland was really really interested in testing OpenSSL.  He's moved on to other things.  We should seriously consider throwing it out, but the heartbleed stuff uses it to test,and currently only the heartbleed.  (I didn't answer your -team email, @ekasper because I thought you knew about that unit-test stuff.)
We need a unit-test framework.  Don't ahve one.  Post 1.1 we should think of something.

I'll create a new PR that removes the OPENSSL_UNIT_TEST guard from ct_test.

PR #808 removes the OPENSSL_NO_UNIT_TEST guard from ct_test.c.

+1

+1

pushed thanks.


Stack level too deep error after upgrading to elasticsearch-model 0.1.8

After upgrading to elasticsearch-model 0.1.8, the error stack level too deep will occur if I use records.records when searching multiple models:
Elasticsearch::Model.search("hello", [User, Post]).records.records

SystemStackError:
  stack level too deep
# ~/.rvm/gems/ruby-2.0.0-p643/gems/elasticsearch-model-0.1.8/lib/elasticsearch/model/adapters/multiple.rb:40

It's okay if I downgrade to elasticsearch-model 0.1.7.

I'm going to close this, especially as it references gem version 0.1.7. Please open a new issue if you have further questions.


FIX: open deeplink on iOS

Just removed all unnecessary #import.
RCTLinkingManager is needed by React Native to handle deeplinks: https://reactnative.dev/docs/0.67/linking


Set execution_state when executing a cell

Necessary but not sufficient to fix https://github.com/datalayer/jupyter-server-nbmodel/issues/13
Still needs:

https://github.com/jupyter-server/jupyter_ydoc/pull/259
https://github.com/jupyterlab/jupyterlab/pull/16651

The test failure appears unrelated as it is also failing on main branch (here).

The upstream PRs are all merged now :)

@echarles / @fcollonval any thoughts here?

@krassowski Thank you for the ping. We have been running on many different tracks and will be able to take this back later next week only.

I understand, thank you for leading this! Let me know how I can help.


regex class - need more work

I've realised flagging some things as help wanted works. 2 tickets closed recently by someone else.
So I will make effort to let people know what I will probably be slow getting too.
Regex is a class low down my radar and I've only done what's needed as I go. if anyone needs this or wants to take it on, then go ahead. Realistically i wont get to it otherwise for a while yet as still busy on other areas.

the class RegExp() is in javascript module. and would probably need tests etc. good luck.
lol.
I did toy with the idea of it doing both python and regex type regexes if you set a flag. and using a lib by someone else.


fixes php8 notices in the new php8 branch

Add the thomas' patch in the devphp8 branch.

Hey I don't have much experience with the developer documentation system yet. I've checked out this branch and the php8 branch on the developer documentation repository.
When I run generate.sh I get quite a few warnings and errors. I'm not sure if this is expected:
Could not open input file: ../app/composer.phar
Could not open input file: ../app/composer.phar
There was an error during the initialisation but we try to proceed anyway. FYI: error was
The configuration file {/var/www/development/developer-documentation/piwik/config/config.ini.php} has not been found or could not be read.
» Please check that /var/www/development/developer-documentation/piwik/config/config.ini.php is readable by the user 'lance'.
PHP Warning:  Constant K_PATH_MAIN already defined in /var/www/development/developer-documentation/piwik/plugins/ScheduledReports/config/tcpdf_config.php on line 16
... lots more of these errors
Then a bunch of these ones:
Unresolved link: "DataTable" in Class Piwik\DataTable\Filter\AddColumnsProcessedMetrics

Please install composer in your app project.
The config.ini.php error can be ignored.
Never had the constant error. which is you piwik branch?
You can ignore the Unresolved link errors.

Thanks, looks like it is working. Should the travis.yml file be updated rather than deleted?

Lance, It has been discussed with Thomas


Environment state

I'm working on the game snake, which extends the Environment class, but cannot make it 100% compatible...
See: https://github.com/bitwise-ben/X/blob/feature/snake/x/environment.py
That's because a simplified representation in terms of position of fruit / basket is not the same as the representation of a snake, which contains a location for each segment. Thus, observe and state return matrices (i.e., a frame/screen) in my implementation.
Probably, in the future when there are games that have more than two in-game objects it's easier to return a frame/screen as state, isn't it?
My question is, what should I do with my current implementation?

Tkx for looking into this @bitwise-ben
So, the exposed and internal states can be different. For example the exposed state could be the entire picture of the game and the internal state is everything you need to keep the game running. The fruit game (optionally) returns pixel locations just to make the table based model work. But we don't expect that all the environments have such a simple representation. You can make the snake game image based only. No problem.
Let us know how we can help in your implementation.

Thanks for explaining what state and observe should return. I think that my feature branch meets the requirements, and I've submitted a pull request.

Cool! Merged! We will work on this further! Let me know if you want to do any other PR


Vanilla UJS as ES6 modules

I needed to include Vanilla UJS via webpack in our product and couldn't figure out how to stop webpack tree shaking it all away, so I converted Vanilla UJS to use module syntax and have an init method when it's imported.
Might bee too big of a change for a pull request and I needed it pretty quick, so I made a fork and an npm module. (https://github.com/sakamies/modular-ujs)
Is this something you'd want to integrate to this repo?

@sakamies thanks for doing that, and yes, this is a thing I need too.


fix: metric recover

Related issue (if exists)

Summary

🤖 Generated by Copilot at 6f20188
This pull request improves the webpack test performance and reduces the bundle size by enabling the Update step in the reusable-build workflow and fixing a bug in the pr-diff.js script.

Walkthrough

🤖 Generated by Copilot at 6f20188

Enable Update step in reusable-build workflow to run webpack-test-metric-diff action and post a comment on the pull request with the webpack test metrics comparison (link)
Add console.log statement to pr-diff.js script for debugging the main branch data fetching in the webpack-test-metric-diff action (link)

Test Compatibility Diff
main	pr	diff
0.177%	0.199%	✅ ⏫ 0.022

This metric diff is out of date, it is generated because I add a willNotSupport case, which will increase test compatibility.


Same protein, same substrate, but with different affinities in docking results

Could it be that the diffusion process is a little stochastic and landed on a slightly different conformation for your protein? I have also seen inconsistent results for the same protein and substrate -- I think the key is that DynamicBind is not deterministic


! in .gitignore isn't taken into account.

$ cd /tmp; mkdir agt; cd agt
$ echo a > f1 > f2
$ echo f1 > .gitignore
$ ag --nocolor --nogroup a
f2:1:a
$ (echo '*'; echo !f2) > .gitignore
$ ag --nocolor --nogroup a
Should be 'f2:1:a' here!!
$ rm -rf /tmp/agt

To add to @austinpray's comment, any file that is gitignored using star-globbing is not ignored by ag:
# ag will not ignore files in build/ =(
build/*

# but this works
build/

Would love to see this bug fixed.

As commented on a couple of related issues, this could be solved if ag got it's git ignore list from git status --ignored

@jasonkarns
Using git's interface is a good idea, but I think calling git would be bad in regards to performance, especially on Windows probably.
Instead, it could / should probably be retrieved using libgit2 or something similar.

Any status on this? Workaround? It's annoying.

I hit this issue when solving another problem which requires an exclude-and-re-include solution.
I'm wondering is there any progress here since several years has passed :)

https://github.com/ggreer/the_silver_searcher/pull/1101 was merged a while ago but it doesn't seem to have solved this issue 😞  (or maybe it was reverted at some point 🤔 )

I just compiled commit 965f71d, and tested with the following .gitignore file:
*.d
!*.d/

But ag still ignored my directory ./rootfs/packages.d.

Oh dear. An almost 8 year old issue is never good 🙂
Seems for me the solution is to use -U --skip-vcs-ignores, hopefully that helps someone else who stumbles across this.

Oh dear. An almost 8 year old issue is never good 🙂
Seems for me the solution is to use -U --skip-vcs-ignores, hopefully that helps someone else who stumbles across this.

Seems for me the solution is to use -U --skip-vcs-ignores, hopefully that helps someone else who stumbles across this.

If the ! PATTERN will never work, then I agree with you that -U --skip-vcs-ignores is the "solution", at least better than nothing.

Seems for me the solution is to use -U --skip-vcs-ignores, hopefully that helps someone else who stumbles across this.

If the ! PATTERN will never work, then I agree with you that -U --skip-vcs-ignores is the "solution", at least better than nothing.

ripgrep is a comparable tool that supports ! in .gitignore.

ripgrep is a comparable tool that supports ! in .gitignore.

Just discovered this issue myself and I agree that it is a major annoyance.


"Endpoint - Anomalous New Services -Rule" fires for service="unknown"

The "Endpoint - Anomalous New Services -Rule" should filter out services that are "unknown".
There are many Windows EventIDs which are tagged with service and report in the endpoint data model without having a value for "service" They will end up being set to service="unknown".
Looking at the EventIDs (over 60 at my customer!). There's no easy way to do a service mapping. Maybe it's better to exclude events with service="unknown" from alerting for the moment.

@my2ndhead : This.  search is from SA-EndpointProtection (Enterprise Security ships it) and not from ESCU. Probably creating a SOLNESS ticket is more relevant here


change odm string annotation to field with type string

This changes are fixed error, which appears in php v. > 7 - [RuntimeException]
An error occurred when executing the "'cache:clear --no-warmup'" command:
PHP Fatal error:  Cannot use 'String' as class name as it is reserved in

thanks ;)

Hey @fredeastside and @l3pp4rd ! Coming from this issue:

https://github.com/stof/StofDoctrineExtensionsBundle/issues/332#issuecomment-235898693

Dou you think there is a chance to push the fix into a 2.4.x release?


local walrus stops after reboot

Steps to reproduce

Install local walrus.
Reboot
Check local walrus container

Result
It stopped.
Expected behavior
It starts again after reboot
Environment

Walrus version: v0.5.1

Environment

Walrus version: main 0a3779654c43dfb1a5d3299c4862502a909f2878
UI version: dev-c3de3bf

Test Result: Reboot the host and check the Walrus container, pass


Don't default to the first service defined

The way that the lookup was previously fetched, setting a lookup to nil would result in using the first entry in the services list: https://github.com/alexreisner/geocoder/compare/master...seejohnrun:master#diff-e9708023fb6f4a972c59b374f21a0ad4R37
This was pretty surprising to us and probably not the intended behavior. We'd expect that either:

Fail when there is no valid lookup for a given type
Disable that type of lookup when set to nil.

This PR goes with the second approach, which also gives us the ability to disable ip_address lookup by just setting the configuration value to nil.
Thank you for all your work on this gem, and please let me know any feedback! Ultimately our goal is to be able to turn off IP address lookup so if you prefer another way I'd be happy to make up a PR

Very interesting. Thanks for the PR. However, an empty lookup feels like overkill, and potentially confusing. Could you maybe provide an example of application code where this is an issue? (Also note that you can test whether a string is an IP address by calling IpAddress.new(str).valid?.)

Yeah the empty lookup is mostly an implementation detail since there are so many places that depend on a lookup being returned in all cases from #get. I could certainly chase them down if you prefer!
As far as a use case, we're looking to use geocoder for its geocoding abilities and don't want to allow lookups via IP. I suppose we could pre-check if something is an IP before passing it in, but that feels pretty awkward especially since the implementation inside geocoder where it's decided whether to use IP or address geocoding might change in the future and our check might fall out of line with it.
Looking forward to your feedback!

I think the right way to do this is actually to do a pre-check, which is possible via the public method Geocoder::Query.new('...').ip_address?. Wouldn't you want to show an error message anyway?

I think probably a lot of people would be surprised to find that IP lookup is being performed by default when it hasn't been set up, so I'd think a first-class feature to disable that lookup might be nice, but I understand and appreciate not wanting to incorporate something like this. Going to close this PR for now, and thank you for your time!


refactor caching (more robust API for caching functions)

We have discussed updating or improving the approach to caching in this repo - would be good to keep track of the spec for this in an issue.

related issue #214

Looks like a dup of https://github.com/hammerlab/cohorts/issues/175?

yeah - could find that one for some reason


How can I dynamically add Labels

Hi, I'm trying to make a Pie Chart, initially empty and then with AJAX I want to fill with data and it will be updated at real time.
I tried to add labels and data to dataset and after make the .update() but there's nothing on the canvas, it still in blank.. So how can I add a label to data.labels and when I call the update method the new values appear to canvas?
Thanks in advance

@didix16 what version are you using? This works in this sample https://github.com/chartjs/Chart.js/blob/master/samples/pie.html

I'm using the latest version. I tried that example but is not the exactly example which I'm looking for.
What I do is just initialize an empty pie chart and then I want to add labels or 'pieces' with its corresponding data.
I mean, initialize the pie chart with an empty labels array.
Could you help me with some codepen example or which function should I use to add new pieces dynamically or which steps must I follow? Thank you so much.

If it works to have an empty labels array initially, then I would do that. Does having no labels array cause a crash?

Initially, if you have an empty array there is not any crash but obviously, there's nothing to paint. However if you have an empty array and after you add a label by making this: chart.labels.push("Some label"); chart.datasets[0].data[0] = 10 (I think this is the object order, I dont have the documentation here right now) and you call .update() then an error is thrown on console that you want to access a property of an undefined (If I remember the property was a length).

Can you create a fiddle with this so i can debug?

Ok, let me a few minutes!

which version are you using from github? the latest?

Yes, this one: https://github.com/chartjs/Chart.js/blob/master/dist/Chart.bundle.js
Is the 2.1.4

weird, I will try and look into this and see if I find anything

I tested this locally and it works fine :/

Mmm it's weird.. Today I'll do it again at my job. I'll try what I did in jsfiddle. If still don't work, I'll make screenshot to the error.

Ok nevermind, at job I didn't access to chart.config.data property, I was accessing directly to chart.datasets and chart.labels....
I'm sorry dude. Now is working perfectly. It was great doing it in jsfiddle since I can view where I make the mistake.
Anyway, thanks for you support :+1:

Hi


FIX: Set UseHighDpiPixmaps only for PyQt5 and PySide2

Background
In https://github.com/mne-tools/mne-nirs/pull/510 we migrated to PySide6.
When calling setAttribute(Qt.AA_UseHighDpiPixmaps) there is a warning as this attribute was deprecated in QT6 and it has no effect as HiDPI is enabled by default.
What does this implement/fix?
One way of solving the issue, as suggested by @larsoner in https://github.com/mne-tools/mne-nirs/pull/510#discussion_r1179883423 is to check the QT version and set the attribute conditionally.

Contributed with ❤ by AE Studio

Thank you @larsoner!

🎉 newcontrib 🎉


Fair demo upsert

Add FAIR Data demo upsert fileview. This is a separate demo project that uses schematic model submit with the upsert flag.

Merging without review because this only changes configuration files.


Browser(chrome) is crashed while uploading large file size

Chrome Browser (Version 80.0.3987.106 (Official Build) (64-bit)) is crashed when uploading a large document file(100MB+).It is working fine for smaller files.I am on windows 10(16GB RAM)I am following this article and using

Code example 2: Upload a file in the same domain by using the REST API and jQuery

https://docs.microsoft.com/en-us/sharepoint/dev/sp-add-ins/upload-a-file-by-using-the-rest-api-and-jquery
Browser is crashed as soon as POST method is sent from addFileToFolder is called.Nothing in console and network tab when  addFileToFolder  is called.

function addFileToFolder(arrayBuffer) {
   // Get the file name from the file input control on the page.
   var parts = fileInput[0].value.split('\\');
   var fileName = parts[parts.length - 1];

   // Construct the endpoint.
   var fileCollectionEndpoint = String.format(
       "{0}/_api/sp.appcontextsite(@target)/web/getfolderbyserverrelativeurl('{1}')/files" +
       "/add(overwrite=true, url='{2}')?@target='{3}'",
       appWebUrl, serverRelativeUrlToFolder, fileName, hostWebUrl);

   // Send the request and return the response.
   // This call returns the SharePoint file.
   return jQuery.ajax({
     url: fileCollectionEndpoint,
     type: "POST",
     data: arrayBuffer,
     processData: false,
     headers: {
       "accept": "application/json;odata=verbose",
       "X-RequestDigest": jQuery("#__REQUESTDIGEST").val(),
       "content-length": arrayBuffer.byteLength
     }
   });
 }

Not surprising, but need additional context about your environment.

Agreed. In my experience with large file uploads, 100MB is a lot of data to load into a browser. Most browsers, including Chrome and Firefox both will crash when you hit 100MB limit.
With large files, I would recommend using the chunked method of file uploads. This is supported both in REST and CSOM, but you may have a better experience if you use PnPJs chunked file uploads.
This is my preferred method for uploading large files to SharePoint, using PnPJs.
https://github.com/SharePoint/PnP-JS-Core/wiki/Working-With:-Files#adding-files

Furthermore, Look at the Graph API for resumable uploads... well documented and intended for files > 4MB


Java define return types

Added ability to define return types in Java more easily when creating a function. Previously, the return types would always default to void. Now, you can (optionally) decide to specify the return type of a function during creation. The previous way of defining functions still works too.
Example: saying "pub static String funky my function" will return
public static String myFunction()
This works for all of the code types defined in java.py

Just realized that this breaks every language that uses the code_functions tag that isn't Java. Will update if I come up with another solution.

I have finished testing this and it looks like it works well.
Some final thoughts: There are two main ways I thought of doing this. The first way is the way that I did it in my first commit which involved changing all the code_* functions in functions.py to allow them to accept return values as arguments and the way that I did it in my last commit where the correct function is chosen based on whether the user dictates a return type or not. Both ways have pros and cons.
Any feedback is appreciated!

From the community backlog session — we apologize for taking so long to review this! @AndreasArvidsson plans on some pretty major refactoring of the language modes (you can see some of the discussion in #1597) which would use a Talon list for modifiers such as private, static, etc. rather than many separate Talon actions for each combination of modifiers. So unfortunately we did not feel it was helpful to merge at this time.


Different IPC model?

We  are looking for a replacement for WebBrowser from ExcelDNA.  This looks cool but the idea of running an http server on all of our clients is terrifying.
Does anybody know if communicating with the CefSharp process using WCF over named pipes (will|will not) work?
I might see if I can get it working.

In the past I've used CefSharp protocol handlers to route requests without HTTP. This workaround makes it more difficult to get info between browser and the add-in, however I have found a way to get messages from the client to the server... I've been meaning to expand the example with this option. As far as I remember it would allow a bound call from JS in the browser to the host add-in. I need to check again.

I got it working with named pipes.  I forked your project and pushed my proof of concept.  It's a disaster, it still has all the Nancy stuff, but it works on my machine.
Thank you for putting this up.

I've pushed some changes showing strings being passed between processes. I haven't thought it out very far but I believe this could be useful. I was thinking about JSON-formatted messages as these can be easy to serialise and de-serialise to objects. Another way to go perhaps. Also likely to be a bit messy.

Our ExcelDNA/CefSharp integration is in production.  I wrote a re-cap @ https://groups.google.com/forum/#!topic/exceldna/PflxmqgvMkw.  Thanks again.

Thanks for that. I've followed up on your modification and removed the HTTP server to provide a working example based on your approach. I will do some more work to remove the polling that I had adopted for shutting down the browser I think, and I'd like it to be easier to integrate with extending the browser class perhaps. However, it still serves as a good example. I wonder if administrator rights are required for the named-pipe, as that is true for the HTTP port reservation, which was not ideal anyway.

We fretted about the permissions related to opening a named pipe but it has not been a problem.


feat(object-code): compatible with versions below typescript 5.2.0 (WeakKey)

For more compatibility, hope to remove the use of WeakKey
WeakKey is referenced after typescript 5.2.0
https://github.com/microsoft/TypeScript/pull/54195

thanks!


Breaking commenting for all other files

I don't know why but this plugin is causing my commenting from vim-commentator to break. It puts the -- instead of the file's syntax commenting characters. This seemingly happened after I upgrade vim to 7.4 with patches 1-1786. I don't have a good enough understanding of vim to figure out where to even begin with this problem.

Same problem here, running vim 7.4 with tpope's comment plugin

I started a conversation about this issue on /r/vim with the hopes that it'd lead to me submitting a patch.

Redditor violet pointed out that this is likely caused by ftplugin being symlinked to plugin. I can confirm that removing the symlink solves the problem. However, I don't know if that breaks Vundle in some way (per the commit message).

I've also experienced this.

Same, its overriding commenting on all other filetypes. When I remove the plugin->ftplugin symlink, it goes away. I don't have the issue described in #14
$ vim --version
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Jun  3 2015 09:55:01)
MacOS X (unix) version
Included patches: 1-712
Compiled by Homebrew
Huge version with MacVim GUI.


mv delete all plugins

hi,
there is a big bug . it is possible with / mv import plugins to import the order plugin . if you do then /mvdelete plugins the entire plugin folder is deleted . Hackers exploit this on my server

I dunno how people are getting the ability to use those commands unless you're doing something wrong but in any case the latest dev build on http://ci.onarandombox.com should prevent this behavior.

Very Nice! Yes, it was a fail.. Bungeecord.. -.-

it would be possible in the settings /mvdelete to disable?

Don't give them permission.

Normally no one has that. But in the last 2 years 2 people managed to log in with my account. The first one used a Bukkit bug. The second time Bungee wans´t secured enough. I'm sure someday it's going to happen again.  It gives me a better feeling when they can't delete the world.  Especially I don't even use mvdelete. Therefor I use ftp instead.

Learn how to secure your systems. It's the easiest and safest way. Yo do need do to that anyway.
Otherwise you should disable every plugin command...


Color preview in settings.html

Is your feature request related to a problem? Please describe.
I'm always frustrated if I can't preview the color in the web GUI, especially if there is a potential preview field on the right side of the input field.
Describe the solution you'd like
Use the field to the right and show a preview of the color entered in the input field
Describe alternatives you've considered
none
Additional context
n/a

Works as expected -> add to the next release


[Workspace Feeds] Web - Card - Added bank account isn't showing for Expensify Card

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

Version Number: 9.0.19-0
Reproducible in staging?: Y
Reproducible in production?: Can't check
Issue was found when executing this PR https://github.com/Expensify/App/pull/46860
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
Action Performed:

Navigate to https://staging.new.expensify.com/
Log in with a new Expensifail account
Create a workspace
Navigate to Workspace settings - More features
Enable "Expensify Card"
Enable "Workflows"
Navigate to Workflows - Connect bank account
Add Regions bank with plaid
Navigate to Workspace settings - Expensify Card
Click on the "Issue new card" button

Expected Result:
I should be able to see and choose the existing bank account.
Actual Result:
Added bank account isn't showing for Expensify Card. I can add the missing bank account if I go a step further and click on "Add a new bank account". The bank account also shows up after a relog.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/c13b8fa0-1d09-4ced-8a79-2baa8918e1ce

View all open jobs on GitHub

We think that this bug might be related to #wave-collect - Release 2

@IuliiaHerets Can you please check if this is reliably repro by other testers or across platforms?
cc @koko57 @VickyStash for viz

Happy to take over Robert if you want

@mountiny As soon as the team has checked, I'll get back to you with an answer

looks like the same root cause like in this one: https://github.com/Expensify/App/issues/45729
so either we don't receive a properly formatted data or the account is not eligible (but in the second case we should be redirected to VBBA, it should have been fixed by my last merged PR https://github.com/Expensify/App/pull/46564)

@mountiny QA Team still can reproduce the issue on different platforms (Web, iOS app, Android app). Build - v9.0.19-9
https://github.com/user-attachments/assets/7fa29e98-7a0b-4762-9256-cd2a9e596005
https://github.com/user-attachments/assets/805a71bd-3a9a-4e28-9f90-299e30a1fdaa
https://github.com/user-attachments/assets/c438afb9-6171-4495-b3a4-4e0c136c9a0d

The problem is definitely with the format we get from AcceptACHContractForBankAccount and OpenPaymentsPage API calls  - we don't get a bank account with type and allowDebit properties - that's why we get an empty list. Before we've filtered out eligible bank account we had the problem described in this issue https://github.com/Expensify/App/issues/45729
here's how we get the data for AcceptACHContractForBankAccount

and for OpenPaymentsPage

We could possibly call OpenPaymentsPage when we click Issue New Card, but it would be much better to unify how the data is sent from the backend and maybe change the onyx method in the OpenPaymentsPage from set to merge

I will have a look thanks!

PR  is up here https://github.com/Expensify/Auth/pull/12043 that should start returning these properties

@koko57 @DylanDylann @allgandalf could you please retest this? Now the AcceptACHContractForBankAccount should return the bank account with the type and allowDebit too

I will retest it today

@mountiny
So this issue is solved - we get type and allowDebit  and the account is showing in the account list.

We still have the https://github.com/Expensify/App/issues/45729 problem, because we still don't get accountNumber in the AcceptACHContractForBankAccount. But this can be solved by using mask property from additionalData.
cc @s77rt

@koko57 The backend changes for this are deployed now, can you check if you can still repro your issue?

@mountiny accountNumber is sent now but not the same way it is sent in OpenPaymentsPage - we get this in accountData in OpenPaymentsPage when in AcceptACHContractForBankAccount in the bankAccount object

@koko57 damn ok updating this, I must have mixed it up in the previous screenshots.

@mountiny please let me know when it's ready to retest

@koko57 This is ready to test, can you please have a look?

Confirming it's working - bank account is showing after adding it.

So I believe we can close this one now, thank you for confirming


Disable Row

Hi,
I want to disable a row until the promise return.
I try this.
$scope.gridApi.selection.clearSelectedRows();
$scope.gridOptions.enableRowSelection = !$scope.gridOptions.enableRowSelection;
$scope.gridApi.core.notifyDataChange( uiGridConstants.dataChange.OPTIONS);
The row selected comes unSelect but the disable not work,.
Thanks

I did a plunker to demonstrate it, but it now won't save.  I think the key here is that enableRowSelection deals only with clicking in the body of the grid, not with selecting by the rowHeader.  So if you turn off enableRowSelection the expected result is that clicking in the body of the grid won't select the row, which means you have to have first turned off enableRowHeaderSelection.
The code was:
var app = angular.module('app', ['ngTouch', 'ui.grid', 'ui.grid.selection']);

app.controller('MainCtrl', ['$scope', '$http', '$log', '$timeout', 'uiGridConstants', function ($scope, $http, $log, $timeout, uiGridConstants) {
  $scope.gridOptions = {
    enableRowHeaderSelection: true,
    enableRowSelection: true,
    enableSelectAll: true,
    selectionRowHeaderWidth: 35,
    rowHeight: 35,
    showGridFooter:true
  };

  $scope.gridOptions.columnDefs = [
    { name: 'id' },
    { name: 'name'},
    { name: 'age', displayName: 'Age (not focusable)', allowCellFocus : false },
    { name: 'address.city' }
  ];

  $http.get('https://cdn.rawgit.com/angular-ui/ui-grid.info/gh-pages/data/500_complex.json')
    .success(function(data) {
      $scope.gridOptions.data = data;
      $timeout(function() {
        if($scope.gridApi.selection.selectRow){
          $scope.gridApi.selection.selectRow($scope.gridOptions.data[0]);
        }
      });
    });

    $scope.gridOptions.onRegisterApi = function(gridApi){
      //set gridApi on scope
      $scope.gridApi = gridApi;
    };

    $scope.disableFirstRow = function() {
      $scope.gridApi.selection.clearSelectedRows();
      $scope.gridOptions.enableRowSelection = !$scope.gridOptions.enableRowSelection;
      $scope.gridApi.core.notifyDataChange( uiGridConstants.dataChange.OPTIONS);
    };
}]);

and
<!doctype html>

    <link rel="stylesheet" href="http://ui-grid.info/release/ui-grid-unstable.css" type="text/css">
    <link rel="stylesheet" href="main.css" type="text/css">

  <button ng-click="disableFirstRow()">Disable First Row</button>

Thanks,
I don't know what's the problem, on this plunker work good.
http://plnkr.co/edit/Iu4gbqWno4UrIJi2kVRn?p=preview
But in my application with another plugins not work.
:(

OK.  I also don't know what the problem is because I can't see your application, but the usual approach would be to add things to the plunker, making the plunker more and more like your application, until it stops working.

right, I'll tell you when discovering the problem, thank you.

Closing: solution proposed that should resolve the issue, no further comments for 2 weeks, assuming solution has resolved issue.


Markdown Export

I want to be able to export a full, organized, plaintext copy of a book like @XVilka suggests in #1717.
Status

[x] Export single .md files
[x] Convert WYSIWYG HTML into Markdown on export
[x] Make the export button show in the UI
[x] Export zip files with proper directory trees

I know this is a cold PR, but hopefully there's something usable in here. General guidance for how/where to implement these features is appreciated.
Help Needed

[ ] Export media in the zip file
[ ] Since this is a heavy operation, gate .zip export behind a permission

These are things that might be good to also add... not sure where to start on them though.
Benefits

Adopting BookStack is easier knowing that all the data stored there is in a standard format.
A lot of people like git sync (hi @dkess), so this makes that possible with external software.

Played around with some things but I'm not really sure how to make codeclimate happy... :(
This PR also now introduces a dependency which could be used to properly do conversions between MD <--> HTML. After testing, the only thing that breaks are the "Success, Warning, Danger" boxes, and the code blocks use nonideal (but still working) syntax.

If this is tested properly and get's through it would be such a biggie for me. @nikhiljha Thank you for putting in the effort.
A Markdown Import function would also be awesome somewhere down the road.

Hi @nikhiljha, sorry for my very late response and thank you very much for offering this.
I've merge this in and followed this up with commits 57ea2e92ec07d4682466857d27bd706a30e39652 and 992f03a3c01759653594fc383917b9aa3affb5a2.
Within those commits I've addressed some of the points mentioned on your previous comment, to get the callout boxes working (Or at least left as HTML) along with making some content tweaks so that code blocks function as expected.
I've also removed the ZIP solution for now to get the core functionality moving and merged. In regards to the ZIP component, it's a good idea but I think will need to be thought about more universally, as you'd maybe then expect ZIP options for the other formats. Maybe it'd be an option/parameter on the existing endpoints, rather than whole new endpoints.
The core markdown export functionality will be part of the next feature release.


Pattern1 using DART programming

triangle pattern using DART programming

@nmortha I'm closing the PR as it is not fulfilling the requirement...you can send a PR again after maintaining all the steps and eligibility criteria for a valid PR...


Custom constraint with LP relaxation

Hi,
First, I would like to thank you for providing a Python API to SCIP. I think the community needs more open-source codes for optimization.
I have just started experimenting with the API by implementing a separator. The Gomory cuts example was helpful and the separator seems to work as expected.
As my next project I was planning to implement a custom constraint type on the form:
f(x) = y,
where f is a complicated function that can be evaluated and bounded, but not coded explicitly using the supported operators. I was planning to use the bounds to create an LP relaxation of f. I am not sure if I should use a constraint handler or a relaxator abstraction for this. I could not find any examples of the latter.
I would need the ingredients to incorporate such constraints in the spatial branch-and-bound scheme of SCIP:

Evaluation of f(x) and, if possible, its gradient at x (for upper bounding problems)
Specification of LP relaxations of f (for lower bounding problems)
Specification of x as (spatial) branching variables

Look forward to hear from you. And again, thank you for your contributions.
Kind regards,
Bjarne

Hi
A constraint handler should do. There is an "example" in test_alldiff. You can also have a look at https://scip.zib.de/doc/html/CONS.php for some of the terms and what each callback is supposed to do.

Thank you. The constraint handler seems to provide the needed functionality.
I will report back if I get around to implementing a useful example.

Hi.
I have a couple of follow-up questions. With reference to the documentation you linked to:

Is it true that branching is performed in the CONSENFOLP function via the return flag SCIP_BRANCHED? Does this mean that the constraint handler is responsible for branching? In that case, are branching priorities handled outside of the constraint handler?
I cannot seem to find a way to return the gradient of the nonlinear constraint at some point x. How can I supply constraint gradients when solving NLP relaxations?

Kind regards,
Bjarne

Hi
Usually branching is performed by the branching rule, but in some cases, a constraint handler might branch, either because it might know a better way of branching (e.g. sos1 constraints https://scip.zib.de/doc/html/cons__sos1_8c.php#aa2bded9b22b1dd6022827b1e01e571f8 )
or out of desperation, (e.g. cons_quadratic ).
Usually, though, nonlinear constraint handler suggests which variable to branch on with the function SCIPaddExternBranchCand. I couldn't really find documentation on adding external branching candidates, but you can have a look at cons_quadratic.c. The documentation of SCIPaddExternBranchCand is here https://scip.zib.de/doc/html/group__PublicBranchingMethods.php#gaf8f111a4eba5256e52bbf5d75af4c456
This function is not yet part of the interface, but adding it is no problem.
Regarding the gradients, this is tricky. I do not think this is currently possible. SCIP uses CppAD to compute gradients, but I think it might support adding a specific user expression for which one might be able to specify the derivative directly. However, none of this is supported at the moment in pyscipopt and I don't think it is going to be supported since we are rewriting the way scip handles nonlinear expressions. Maybe @mueldgog  or @svigerske might know a trick to get around this?
Note, however, that SCIP uses gradient information only for primal heuristics, so if you know any decent solution in advance, it might not be too bad.


[admin] Publish

Docs Build status updates of commit 366f0f7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/power-automate-integration.md
:white_check_mark:Succeeded
View

docs/images/automate-a-task-button.png
:white_check_mark:Succeeded
View

docs/images/automate-a-task-choices.png
:white_check_mark:Succeeded
View

docs/resources/samples/excel-samples.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Cannot connect

Version of the custom_component
0.2.2
Configuration
N/A. Could not complete initial setup.
Describe the bug
I enter the API key and click submit then get this:

My API key is correct and the internet connection is working.
Debug log
N/A. Could not complete initial setup.

same. hope someone is still working on this.

I am having this issue too.

I've noticed a pattern.
I actually got it to work again today when I put in my API key again.
but then I decided to reboot my server, and I lost it again. I'm going to try to reload it again tomorrow. It seems like something get garbled up when/ if you do a full reboot of your server.
I'll report back again tomorrow.

I can confirm, after waiting 24 hours, the integration works again.
I'm not sure why, or what's happening... some kind of caching mechanism....?
but after 24 hours it seems to reset itself. I'll try not to reset my HA for the next 24 hours and see if the setting hold.

... and it's broken again. Going to try to disable the Polling option for updates...??:

after disabling the two option in my post above, I've not had any issues with the Integration!
I think the "Enable polling for updates" might have been adding to the polling issue and using up the limited amount Govee does provide for the API.

+1 unavailable device

Same here, made the mistake of removing it to readd integration. Now I cannot readd, and try anything else...

ok seems like i hit a rate limit? custom_components.govee.config_flow.CannotConnect: API: API-Error 429: {"message":"rate limited! the limit is 10000 requests every 24 hours. X-RateLimit-Reset in Http Response Header shows the reset time."}

ok seems like i hit a rate limit? custom_components.govee.config_flow.CannotConnect: API: API-Error 429: {"message":"rate limited! the limit is 10000 requests every 24 hours. X-RateLimit-Reset in Http Response Header shows the reset time."}

did you try my trick? unchecking those options in the configuration?
You'll probably have to wait 24 hours before you can try to add the integration again.

I'm facing the same issue
I try to reload the integration once is not working, and after this still not working e removed And I try again to added this integration with a new API key requested and I'm facing the same issue as mentioned in this post
Is there any thing to solve this?
Thank you and best regards

did you try my trick? unchecking those options in the configuration?
You'll probably have to wait 24 hours before you can try to add the integration again.

@MiAutomations Give https://github.com/wez/govee-lan-hass a look.

+1 Unavailable device

+1


My Calculator

Description 📜
Add a project of My calculator  build using html,css and javascript  which performs all arithmatic operations and few others easily.
Fixes #1505

Type of change 📝

[x] Add a new project
[] Modified or enhanced the existing projects
[] Improve the readme of any project

Domain of Contribution 📊

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

@gurjeetsinghvirdee please review

Why r you creating same project?🙂🙂🙂 it's already there.
https://github.com/khushi-purwar/WebDev-ProjectKart/tree/master/08.Calculator

I am CLosing this


Add footer in child

how to add footer in below of the child in ExpandableDraggableWithSectionExample.
How to return footer view in bind view holder it gives me arrayIndexOutOfBoundException.
The probleme statem is explained in the image given below.

hello navneetjain01 any success in above task ??

not till now.


helm/cosigned: remove helm charts

This PR removes the ci infra and the Helm Chart Cosigned and move that to the official sigstore helm repo (https://github.com/sigstore/helm-charts)
The PR that adds the chart is: https://github.com/sigstore/helm-charts/pull/10

i'm going to clean up the gh-page branch and remove it


SyntaxError: Unexpected token 'export'

I keep getting this error when using the library in Typescript:
node_modules\iso-datestring-validator\dist\index.js:1
exportfrom"./is-valid-date/is-valid-date.function";exportfrom"./is-valid-iso-datestring/is-valid-iso-datestring.function";exportfro
m"./is-valid-time/is-valid-time.function";exportfrom"./is-valid-year-month/is-valid-year-month.function";
^^^^^^
SyntaxError: Unexpected token 'export'

Is it node project or a frontend project with a bundler? I need more info to reproduce and look into it.

I'm using it in a backend NodeJS (Typescript) project and I'm using Babel.

Got it, will investigate, thanks. Most likely it's because version 2.1.3 uses ES6 modules.
You could switch to version 2.1.2 until I fix the issue, it's commonjs so there shouldn't be any problems. The only difference between 2.1.2 and 2.1.3 being ES6 and bugfix for datestring without time.

I went back to commonjs module in recent version 2.1.4, you should be cool now.


fix(controller): use django HttpResponse for logs

Django Rest Framework will wrap the string with quotes because
of the JSONRenderer. Django's HttpResponse class handles the
application logs as we expect it to.
fixes #4657

Example response before and after:
><> curl http://deis.<IP_ADDRESS>.xip.io/v1/apps/foo/logs -H 'Authorization: token 2f680997d314e37f61386c9660a94b809eb7f770'
"2016-04-01T15:56:26UTC foo[deis-controller]: bacongobbler created initial release\n2016-04-01T15:56:36UTC foo[deis-controller]: bacongobbler deployed deis/example-go\n2016-04-01T15:56:53UTC foo[deis-controller]: bacongobbler scaled containers cmd=1\n"

><> curl http://deis.<IP_ADDRESS>.xip.io/v1/apps/foo/logs -H 'Authorization: token 2f680997d314e37f61386c9660a94b809eb7f770'
2016-04-01T15:56:26UTC foo[deis-controller]: bacongobbler created initial release
2016-04-01T15:56:36UTC foo[deis-controller]: bacongobbler deployed deis/example-go
2016-04-01T15:56:53UTC foo[deis-controller]: bacongobbler scaled containers cmd=1

I tested this against Deis v1 master. Looks good with the updated client + controller:
$ deis logs
2016-04-04T16:26:36UTC classy-question[deis-controller]: matt created initial release
2016-04-04T16:27:43UTC classy-question[deis-controller]: matt deployed 6b142e5
2016-04-04T16:31:14UTC classy-question[deis-controller]: matt scaled containers web=3
2016-04-04T16:32:29UTC classy-question[deis-controller]: matt added POWERED_BY
2016-04-04T16:33:04UTC classy-question[web.1]: Server listening on port 5000 in production mode


skipping non-upgradable packages

When I am running the script as non-root some packages fail to update:
pip3 list -o
Package     Version        Latest Type
----------- -------------- ------ -----
dbus-python 1.2.16         1.2.18 sdist
duplicity   <IP_ADDRESS>       0.8.20 sdist
gpg         1.13.1-unknown 1.10.0 sdist
pycups      1.9.73         2.0.1  sdist
PyGObject   3.36.0         3.42.0 sdist

I suspect these are OS-installed and need root permission. Perhaps we can have a flag to filter
out such packages (perhaps by type=sdist)?

Maybe #4 fixes this?


Crash Course on Climate & Energy

Resource URL
https://www.youtube.com/watch?v=dhFlDkES6Ws&list=PL8dPuuaLjXtMx8ZIQV9NduU_HFjDwykuj
Add a short description for the resource.
Welcome to Crash Course Climate and Energy! Over the next 12 episodes, we’re going to take a look at the ways climate change is affecting our planet, untangle the evolving energy landscape, and learn to think critically about how we, as individuals and as a society, can affect the outcomes.
Why do you believe this to be a good resource?
M Jackson makes a complex subject like climate change easy to understand for anyone, in my opinion. She's been right in the middle of drastic weather changes personally, and so has a unique perspective on the matter.
Any additional information you want to share?
In the video description, you can find the links to all the research she did for the videos. It's all neatly organised in a Google Doc! https://docs.google.com/document/d/1rRJ-L9TLNfPwPfzn3LdjDEw-wHtThwTfDUe2rDtFXQQ/edit

Thank you for your contribution, @JaiNeethling 🎉 My apologies for taking so long to reply. I will review these resources today.


C++ QASM Simulator as a Debian Package

In order to use the simulator, the only option available right now is to download this entire repository and compile it. However, not in all cases we are also interested in using the Python SDK, for example, if we want use another framework like the one for Swift. So, it would be nice to have another way to install the QASM simulator ... for Linux (other platforms, like macOS, we can discuss later :) ).
Expected Behavior
apt-get install qasm-simulator-cpp
./qasm_simulator_cpp input.json

Current Behavior
git clone https://github.com/QISKit/qiskit-sdk-py.git
cd qiskit-sdk-py
make depends
make sim
cd out/src/qasm-simulator-cpp
./qasm_simulator_cpp input.json

Possible Solution
I have already prepared the package and it is available here: https://github.com/indisoluble/qasm-simulator-cpp-debian-pkg.
Now I have to find a sponsor but, for this to be possible, I need to confirm a couple of things:

Do you agree with packaging this software?
If the answer is yes, I would be the maintainer of this package, would this be OK?

Context
I opened the following issue in the QISKit repository for Swift to discuss how we could integrate the QASM simulator in the Swift framework: Plan to integrate C++ simulator. I suggested the first step should be to provide an easy way to install the simulator but this plan can not go through unless you agree ... hopefully you will :)

Hello @indisoluble - and thanks for the suggestion and for the effort so far! We actually had some internal conversations about packaging qiskit (this Python SDK) for Debian, as we currently have a couple of people involved with Debian in the team (that would be me, as a low activity maintainer mostly at the DPMT, and @1ucian0 , as a fully fledged Debian developer!) and we would be happy to have it in the Debian repositories. As a matter of fact, a RFP was issued by Luciano, although it was not followed up with high priority as the idea was to work on it after the 0.5 release.
The lowering of the priority is also partly related to the C++ simulator - as you noticed, currently we are building the C++ as a single binary that is invoked when needed, but in the short term we are planning to include it as a proper compiled extension more integrated with Python (which should also simplify the packaging effort). We still do plan to keep the option for compiling it as a standalone binary, as it might be useful in some cases - for example, (and me not being familiar with the Swift ecosystem), it seems it would fit that case as well if indeed Swift packages that mix languages are not possible.
As for the packaging of the simulator itself: yes, we would love to have it packaged! Being a maintainer yourself shouldn't be a problem at all, however, could you look into the possibility of joining an existing team that might be suitable, and allow for maintaining it in a collaborative manner (Debian Science comes to mind)? Hopefully Luciano will be able to elaborate - it's not mandatory at all (and if no team seems appropriate, there are other options, such as multiple maintainers or NMUs), it's mostly for sharing the workload and ensuring we are able to keep the package in sync with the latest releases eventually. Please let us know if any help is needed navigating the Debian procedures, but usually all starts with an ITP :)
In the technical front, after peeking a bit at your repository, please note we currently have a cmake-based system in place that we actually use when building the Python packages (and compiling the binary as as a step). While the existing plain Makefile should also do the trick, it might be a good idea to try to make use of the cmake build system, as one of the benefits is that it should cope better with differences in architectures and compilers, and ease the amount of work needed when packaging for multiple platforms.
TL;DR: looking forward to it :+1:

Hi @diego-plan9 & @1ucian0,
Thanks for answering so soon :)
I just got the ID for the ITP bug report: 899261.
About joining a team, I am looking into it right now. DebianScience seems OK or even Debian Scientific Computing Team.
Also, as you suggest, I will see how how I can modify the package to use cmake instead of Makefile.
Regards.

Hi @indisoluble,
Probably the best course of action is to move the packaging code to salsa.debian.org (I agree, the DebianScience place seems like the correct place) and work on packaging qiskit for Debian in multiple binaries, one of them can be the simulator. I can sponsor it (I'm luciano@d.o).
Since the issue is tracked in the BTS and is not a QISKit (as upstream) issue. I'm closing it.
Thanks for pushing the effort, luciano


08/31/2018 01:45 : Create Pull Request for branch prTesting_2018083101 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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.


Implement coupon-less rods

Could be as simple as having a zero-length coupon, which must be paired with one of the new, coupon-less rods.

This is effectively done by making a zero-length coupon stand-in


mark_as_read doesn't trigger the receipt timestamp when read_at? is updated

Hello,
I'm wondering if this is a feature or not, but when I trigger mark_as_read (on receipt or conversation), the timestamp of the receipt (receipt.updated_at) isn't updated at the same time of is_read?. This is due to the usage of update_all.
I encounter a case where it would be useful to get the last X updated receipts of a user to perform key caching, simply by using current_user.receipts.map(&:updated_at).last(X)

Which gem and Rails versions are you using?


[Bug]-[62]:RTE error highlight includes the label element

Is there an existing issue for this?

[X] I have searched the existing issues

Description
The RTE widget error highlight is on both the component and label. Whereas it should only be on the input component.
While fixing this we need to find the same for any I/O widgets with label.

Steps To Reproduce

D&D RTE
Set to required
check the error highlight; it must be only on the input component

Public Sample App
No response
Version
Cloud - v1.7.0

Fix - Only show the red border on the editor and not the label
Stats

Stat
Values

Reach
62

Effort (months)
0.5

@somangshu @dilippitchika
From the screenshot below, the Date picker doesn't have the red border to signify that it's required.
Should this be fixed in this PR?
Also, the UI for Checkbox Group and Switch Group looks odd, was this intended compared to a single checkbox

Hey Tolu, no i don't think it was intended. We need some design for these.

@dilippitchika
This PR would only solve the RTE issue, everything else would be outside this PR

Hey @Tooluloope yes that is the expectation

Issue has been verified in Release and it is working as expected.


import 'vue-flux/dist/vue-flux.css' missing?

When I try importing css as per the doc it threw me an error that it is not found and the file is physically not present anywhere in the package as well, am I missing something?

Hello, CSS is not needed to be imported anymore since some versions ago.
I guess you found it in the wiki, I will review and remove the import.


chore(store-ui): Remove '@storybook/addon-storysource'

What's the purpose of this pull request?
Remove the Storybook add-on @storybook/addon-storysource. We are already using the @storybook/addon-docs add-on, which is already capable of generating the same code snippets we need, and these two packages being used at the same time caused some sort of conflict, as described in this issue: https://github.com/storybookjs/storybook/issues/13362.
This issue came to light after @igorbrasileiro spotted a bug in the Table component stories: https://github.com/vtex/faststore/pull/987#issuecomment-942640022.
How to test it?
The bug mentioned above should be solved in the deploy preview below. And the source code for all other stories should be working the same way as before.
References
https://github.com/storybookjs/storybook/issues/13362

@igorbrasileiro Oooh! I think it won't show up because the base branch here is not master 😅
But here's how the table looks now (running locally):

@igorbrasileiro Oooh! I think it won't show up because the base branch here is not master 😅

The other stories are also working as they should, right?

Yes! Nothing changed in the others :)


Xgboost GPU ignores USE_SPREAD_STRATEGY=0

in cluster env, set USE_SPREAD_STRATEGY=0. Run xgboost train with GPU, i can still see these being printed and indicates to me there is a placement group with spread strategy.
Demands:
 {'GPU': 1.0, 'CPU': 2.0} * 4 (SPREAD): 1+ pending placement groups
2022-03-02 13:56:50,578 WARNING resource_demand_scheduler.py:746 -- The autoscaler could not find a node type to satisfy the request: [{'CPU': 2.0, 'GPU': 1.0}, {'GPU': 1.0, 'CPU': 2.0}, {'GPU': 1.0, 'CPU': 2.0}]. If this request is related to placement groups the resource request will resolve itself, otherwise please specify a node type with the necessary resource https://docs.ray.io/en/master/cluster/autoscaling.html#multiple-node-type-autoscaling.
2022-03-02 13:56:50,578 WARNING resource_demand_scheduler.py:746 -- The autoscaler could not find a node type to satisfy the request: [{'CPU': 2.0, 'GPU': 1.0}, {'GPU': 1.0, 'CPU': 2.0}, {'GPU': 1.0, 'CPU': 2.0}]. If this request is related to placement groups the resource request will resolve itself, otherwise please specify a node type with the necessary resource https://docs.ray.io/en/master/cluster/autoscaling.html#multiple-node-type-autoscaling.

cc @xwjiang2010

@ZhijieWang it's RXGB_USE_SPREAD_STRATEGY - the XGBoost section in the Ray Docs is a bit outdated and in the process of being updated

Closed by https://github.com/ray-project/ray/pull/22783


Allow searching by browse_node or search_index alone

Problem (Actual behavior)
In the latest version, v3.3.0, the following search requests raise amazon.exception.AmazonException: ValueError: At least one of the following args must be provided: keywords, actor, artist, author, brand,title.
amazon.search_products(search_index='Books')
amazon.search_products(browse_node='2275256051')

Expected behavior
The above requests should returns search results. Though the API document is ambiguous, the SearchItems API returns expected results when we specify only SearchIndex or BrowseNodeId (at least in country JP).
In fact, requesting SearchItems without additional parameters in the Scratchpad, error message says as follows:

Changes
This PR allows searching by browse_node or search_index alone.

@sergioteula  This claim seems legit. I just checked in the scratchpad as well.

Thanks for the confirmation!


6.10.20 RN

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Adam-Dupaski:x: jds-mendixYou have signed the CLA already but the status is still pending? Let us recheck it.


Doesn't work with `.jpe extension

Describe the bug
When adding an attachment to a note and then syncing to Obsidian, I noticed that the images do not show up in Obsidian's file explorer. For some reason, the file extension is being truncated. The file in question is a jpg on my phone but in the source property in the note on my pc, it shows up as .jpe which is not a valid extension. When I change the extension to .jpeg it shows up in the explorer
To Reproduce
Steps to reproduce the behavior:

Add this to source: https://yixcweyqwkqyvebpmdvr.supabase.co/storage/v1/object/public/attachments/c0378068-0eca-41d8-a058-665e255f6a66.jpe
Try to sync with obsidian plugin
Attachment doesnt' sync

Expected behavior

Attachment should sync with all common attachment types
https://discord.com/channels/910294799188385885/1148630648408518689/1148630648408518689

When I was testing, I observed that the same image successfully synced to the designated folder. However, despite the successful sync, I couldn't locate the image within Obsidian. This issue seems specific to images with a '.jpe' extension. In contrast, I can see other files with '.jpeg' and '.png' extensions without any problems.

Great find @gonzalovelasco . I'm going to close this ticket


"Vulturești", "Bârzești", "Huluba" are listed in the drop-down menu

Description:
When user writes "Vulturești", "Bârzești" and "Huluba" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Vulturești"

Expected results
The location appears in the drop-down menu as "Sat Vulturești, ARGEȘ (Vulturești)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bârzești"

Expected results
The location appears in the drop-down menu as "Sat Bârzești, ARGEȘ (Vulturești)"

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Huluba"

Expected results
The location appears in the drop-down menu as "Sat Huluba, ARGEȘ (Vulturești)"


Questions not loading

company questions are not getting loaded after clicking on the company name. it's just getting loaded infinitely.

#99


CC Search extension page on mozilla site says it's not a recommended extension.

Is your feature request related to a problem? Please describe.
CC search extension page on Mozilla says it's not a recommended extension.

Describe the solution you'd like
There are 2-ways to get through this.

We can request users to email amo-featured [at] mozilla [dot] org with a link to its AMO listing page.
All extension must follow some best security practices , so we can work to improve on this.

Describe alternatives you've considered
To go with present scenario.
Additional context
If an extension is tagged not recommended by Mozilla then it will not be promoted or featured by Mozilla other than on the extension's original page. So if we work on this we can improve the visibility.

From the discussion on Slack:

A large number of extensions on the firefox addon store has this warning. In fact, very few addons have the "recommended" badge and are featured on the Firefox addons front page. The good thing about cc search extension is that it does not have any serious permissions in manifest or content scripts (i.e scripts that can be injected into the pages that users visit), therefore when it is installed the user is not shown warnings like "this extension requires your permission to access your data for all websites, read and modify privacy settings)...

The warning can make people doubt the credibility and make them vary of installing the extension making a dent in the impact


Drop prompt from output is determined in an unclear manner in HFModel

Dropping the prompt from the model output is necessary to be able to correctly retrieve an output from the Prediction object, but this is only done in HFModel when a ValueException is thrown upon model creation for unclear reasons. It would be useful to either have this property setable by the class user, or have the reason for it's current set up be explained?
Line 79 onwards in hf.py for reference.

Just to clarify, are you saying there's an issue in HFModel? Or HFModel is correct and the rest aren't?
(In general, HFModel is very clunky and needs work. For local models, we usually recommend TGI clients for now.)

That there's an issue in HFModel. If that's the case I can accept that! Will maybe make a PR with my local changes to get it usable.

That would be really useful! We'd love to merge your fixes.
That said, I recommend looking into using TGI. It's a lot faster, more customizable, and much more stable than HFModel.


[DigitalOcean/Azure driver] Intermittent tls Handshake errors on create

A few times now I've gotten DigitalOcean errors on create:
$ machine create -d digitalocean --digitalocean-access-token token foo
INFO[0000] Creating SSH key...
INFO[0001] Creating Digital Ocean droplet...
INFO[0003] Waiting for SSH...
FATA[0094] tls Handshake error: dial tcp <IP_ADDRESS>:2376: connection refused

If I set that host to active and run docker commands against it it works as expected... it's just the create output that's blowing up.
Anyone else run into this?

I've been seeing some odd behavior today as well - like SSH never making it, droplets saying they weren't created etc.  Will investigate more.

Added "Azure" to the tag in title.

@ahmetalpbalkan thanks for the report -- I'm digging to see if I can find what might be causing this for Azure as well.

@ehazlett one thing I noticed with Azure is, if you set --azure-port something other than default 2376, it still keeps using 2376.
The failure above doesn't prevent installation to complete (I guess docker is installed and started by the time we get that failure). However using the wrong port causes bad port mapping on Azure VM and you can't reach Docker daemon using the port you specified at --azure-port. I think this hardcoding causes that.
Not sure if these issues are related, though.

@ahmetalpbalkan ok could you open a separate issue for that?  I'm getting some odd behavior with Azure itself as well (A hosted service with the specified name already exists. with no services).
/cc @jeffmendoza any ideas on that one?
DigitalOcean has been fine for me today.

I've had this on virtual box today.

Ok this is starting to sound like a possible race issue perhaps.
On Wed, Jan 7, 2015 at 1:18 PM, Simon Thulbourn<EMAIL_ADDRESS>wrote:

I've had this on virtual box today.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/229#issuecomment-69093684.

hosted service names are used for DNS and must be globally unique.

@jeffmendoza i had no services provisioned.  this issue went away for azure yesterday.  not sure what happened.

@jeffmendoza thanks for explaining :)

I'm seeing this same issue in VirtualBox as well. The create errors. But if I run with --debug to keep the machine around, after it is created, I can use the Docker client against the TCP socket just fine. Here is the end of the create debug output for both Digital Ocean and VirtualBox showing the issue:

Digital Ocean: https://gist.github.com/andyshinn/cfb0ffd350cee35b3d92
VirtualBox: https://gist.github.com/andyshinn/4b91d344cdeb52e4959d

@andyshinn @sthulb @ahmetalpbalkan could you test the fix in #266?  I haven't had this issue anymore after that fix.

I can't reproduce this particular issue. But am having a bunch of other failures which seem to be slowness inside the DO network (failing sometimes on docker stop, curling the binary with identity, etc.).

Yeah I was having a bunch of issues on DO today as well :(
On Mon, Jan 12, 2015 at 11:40 PM, Andy Shinn<EMAIL_ADDRESS>wrote:

I can't reproduce this particular issue. But am having a bunch of other
failures which seem to be slowness inside the DO network (failing sometimes
on docker stop, curling the binary with identity, etc.).
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/229#issuecomment-69694566.

@ehazlett just had a chance to try with #266. It works for Azure –unless I specify the --azure-port parameter. There's some hard coding in the code for port 2376 and this disallows custom ports from being used. Nonetheless, it doesn't give the TLS handshake error anymore.

Thanks.  Yeah we should open a separate issue for that.  We have a couple
of drivers that hard code some things.
On Wed, Jan 14, 2015 at 12:04 PM, Ahmet Alp Balkan<EMAIL_ADDRESS>
wrote:

@ehazlett https://github.com/ehazlett just had a chance to try with #266
https://github.com/docker/machine/pull/266. It works for Azure –unless
I specify the --azure-port parameter. There's some hard coding in the
code for port 2376 and this disallows custom ports from being used.
Nonetheless, it doesn't give the TLS handshake error anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/229#issuecomment-69950754.


Fixes #298: Add some notes to documentation

Fixes #298

PR updated.

LGTM @moxious if there are no further comments I can merge this

LGTM, merge!


Fix dynamic configuration watch implementation current value not null when the config is deleted.

Fix

[ ] Add a unit test to verify that the fix works.

[X] Explain briefly why the bug exists and how to fix it.

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.

[X] Update the CHANGES log.

In ConfigWatcherRegister.singleConfigsSync(), when config is deleted and watcher.value() is not null, core would notify watcher every syncPeriod time.
ConfigChangeWatcher watcher = holder.getWatcher();
                String newItemValue = item.getValue();
                if (newItemValue == null) {
                    if (watcher.value() != null) {
                        // Notify watcher, the new value is null with delete event type.
                        watcher.notify(
                            new ConfigChangeWatcher.ConfigChangeEvent(null, ConfigChangeWatcher.EventType.DELETE));
                    } else {
                        // Don't need to notify, stay in null.
                    }

@JaredTan95 when you do merging in the next time, please make the commit logs clean.
Fix dynamic configuration watch implementation current value not null…
… when the config is deleted. (#7606)

* Fix dynamic configuration watch implementation current value not null when the config is deleted.

* add changes.md

* fix LoggingConfigWatcher watch.vale, and when value = null ,config not reset.

* fix

* fix style

* Update oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/UninstrumentedGatewaysConfig.java

Co-authored-by: kezhenxu94<EMAIL_ADDRESS>
* use Strings.nullToEmpty

Co-authored-by: 吴晟 Wu Sheng<EMAIL_ADDRESS>Co-authored-by: kezhenxu94<EMAIL_ADDRESS>
This is how it looks like now. Too many unnecessary things.