Update ssdp and allow specification of bind ports

I did a PR into node-ssdp to allow specifying which port to bind to for discovery as I need to be able to run these libraries behind a firewall as the default 0 picks a random available port.
This PR adds ability to specify the ports to wemo-client to use for event listening, and also for discovery.
Thanks!

Current coverage is 89.96% (diff: 100%)

Merging #40 into master will increase coverage by 0.07%

@@             master        #40   diff @@
==========================================
  Files             2          2
  Lines           267        269     +2
  Methods          57         57
  Messages          0          0
  Branches         40         43     +3
==========================================
+ Hits            240        242     +2
  Misses           27         27
  Partials          0          0

Powered by Codecov. Last update 2a0fd44...a2e9cc2

Nice feature, thank you!

You're welcome :) It'll be great to see this in a release soon 👍 Thanks for looking after this great project! :)

Thanks for the follow-up. It's now in released in v0.11.0


Restart and Update doesn't work

Version
4.4.2-67841
Windows 10
Describe the bug
Emulator shows a notification that a new version is available (4.4.2-68352) but clicking Restart doesn't do anything.
Expected behavior
The emulator to close, install and restart.
Screenshots

[bug]

I've seen this bug before, but I was unable to repro it reliably. I'll keep an eye out for it, and see if I can track down what's causing it.
If you click either one of the buttons, nothing happens, correct?

Dismiss works - restart doesn't do anything.

Ok, thanks. I'll see if I can get a reliable repro for it.

I've got a repro if you need @tonyanziano

This is still an issue with Version 4.8.0 --> 4.8.1
I get a similar behaviour as in #2096, only for me after this step:

closing / reopening, clicking on Check for updates --> Updates downloading --> notification, clicking Restart this pops up:


Katex fonts not included in extension.

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[ ] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[X] Markdown.  Markdown features, such as syntax support and features

Describe the bug
The extension doesn't include Katex fonts in its release, this leads to incorrect rendering of mathematical expressions (See #1747).
I believe this line is supposed to copy the font folder to the build output but it doesn't seem to be present on my local installation of the extension. Katex css file expects the fonts folder to be inside the css\themes folder (as it's loaded from dark.css or light.css)
https://github.com/dendronhq/dendron/blob/67a84c9185cb3def45cc83d5921158b308d9c404/packages/common-assets/scripts/buildStyles.js#L70
I'm not sure if the fonts were removed manually (to decrease extesion size?) but it breaks Katex so I'm assuming this is a bug.

Thank you for reporting.
We also have a regression where Katex isn't rendering properly when published, and I think this has to be related to that as well.
I'll let you know when the fix for this gets pushed.

This issue has been resolved with v85. Please let us know if there are still outstanding concerns related to this issue.
I will mark this to automatically close in 4 days if there are not further comments.


Fix bug that changes disappear after context menu

Signed-off-by: Ruiyun Xie<EMAIL_ADDRESS>Summary of changes
Fixed bug that changes for one attribution disappear after delete or confirm another attribution via context menu in audit and attribution view.
Context and reason for change
Changes should not be discarded
How can the changes be tested
The changes can be tested in the UI

We also need to fix the same bug for the case that we delete multiple selected attributions via the context menu "Delete selected globally".


[Feature Request]: Config to disable item requirements/add valid items for styles

Is your feature request related to a problem?
I use a lot of mods that add tools, including in unconventional ways (like Tetra, that does not add unique items, bit rather a modulable tool system). Because vanity seems to only allows a few precise items to be modified in the stylist table, I cannot apply Vanity styles to my gear.
Solution(s)
Maybe there could be a config option to either disable specific item requirements for styles (may be a bit chaotic for players with lots of styles, but can be remedied by temporarily taking styles out of the stylist table and generally allows more player freedom), or to add item ids that can be stylized?
Describe alternatives you've considered
I'm not going to lie, I'm asking mainly so I can reconcile both Vanity and tetra, but a config seems easier to implement than tetra compatibility.
Mod Version
2.1.1
Mod Loader Version
1.20.1 - 47.2.0
Mod Loader
Forge
Additional context
Lovely mod! Love it so far. I'm not sure how feasible what I'm suggesting is considering allowed items seem to be defined on a per-pack basis but I figured it cost nothing to ask, have a nice day

It shouldn't be hard to fix that for yourself with a datapack, but yeah the thing is that whoever made the vanity packs you are using probably just didn't set up mod compat by using tags, so you could say it is "intended".
I can see how this would be easier for players though, but it might be hard to implement considering styles can have multiple different models for different items, so the game wouldn't know which to chose, and if all were displayed it'd end up leading to many broken models.
I personally tend to just make my styles compatible with tags, so they should work with anything, but many Vanity developers opt into only letting certain tool tiers have a design or make unique textures for different materials, which isn't really compatible with this approach.


many-to-one inverse relationship with 0.102.0

Hey team,
I can see great improvement in version 0.102.0 and thank you very much for your great effort on making Realm better.
Meanwhile, I noticed a change on inverse relationship.
With version 0.98.6, for a relationship like 1 Person has many Dogs and 1 Dog has one owner, I can setup the relationship like:
Class Dog {
  var owner: Person? {
    return linkingObjectsOfClass(String(Person), forProperty: "dogs").first as? Person
  }
}

but linkingObjectsOfClass method in deprecated now, and how could I setup an inverse relationship without breaking the existing API - owner returns a Person, not a RLMResult?
Thanks.

The simplest way is to just have a private LinkingObjects property which is then exposed with a computed getter:
class Dog {
  private let owners = LinkingObjects(fromType: Person.self, property: "dogs")
  var owner: Person? {
    return owners.first
  }
}

Cool, many thanks.


Laravel v8.4 & PHP v8

Hello there.
Below error is what i get when trying to install this package for Laravel v8.4 & PHP v8 (downgrade & upgrade not working too).

Your requirements could not be resolved to an installable set of packages.

Problem 1
- saeedpooyanfar/zarinpal[v3.0.0, ..., v3.0.3] require guzzlehttp/guzzle ^6.3 -> found guzzlehttp/guzzle[6.3.0, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).
- Root composer.json requires saeedpooyanfar/zarinpal ^3.0 -> satisfiable by saeedpooyanfar/zarinpal[v3.0.0, v3.0.1, v3.0.2, v3.0.3].
Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.
Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Hi, thanks for your contribute!
Please checkout<EMAIL_ADDRESS>


Bugs of ry

When I use the function to read the yields table, it shows that there is no ry. I check the code from NuPyCEE, there is no definition of ry. However, it use it and its function to read the table. Obviously, it cannot work.

Thanks for reaching out.
"ry" is defined in NuPyCEE in chem_evol.py (import NuPyCEE.read_yields as ry). If you look further into chem_evol.py, you will see that NuPyCEE calls the classes "ry.read_yields_Z(...)" and "ry.read_yields_M_Z(...)". These two classes inherit from the class "ry.read_yields(...)".
If you want to read yields tables by your own, without using the chemical evolution codes (i.e. SYGMA or OMEGA), you need to import NuPyCEE.read_yields as ry, and then either use ry.read_yields_Z or ry.read_yields_M_Z, depending on whether your yields are metallicity-dependent only, or mass- and metallicity-dependent.
If you encounter a problem, or have questions, please let me know.  If I did not understand properly your issue, could you please provide more details so I can reproduce the problem?
Have a nice day.

Dear Benoit Côté
Thanks for your help!
I've been busy these days and I just found that you have sent this
email. Sorry for replying late. Now, I can run the ry correctly. If I
encounter any problem later, I will let you know.
Thanks for your patience! That's very kind of you.
Best regards,
Jinning Liang
Benoit Côté @.***> 于2021年6月30日周三 下午11:45写道：

Thanks for reaching out.
"ry" is defined in NuPyCEE in chem_evol.py (import NuPyCEE.read_yields as ry). If you look further into chem_evol.py, you will see that NuPyCEE calls the classes "ry.read_yields_Z(...)" and "ry.read_yields_M_Z(...)". These two classes inherit from the class "ry.read_yields(...)".
If you want to read yields tables by your own, without using the chemical evolution codes (i.e. SYGMA or OMEGA), you need to import NuPyCEE.read_yields as ry, and then either use ry.read_yields_Z or ry.read_yields_M_Z, depending on whether your yields are metallicity-dependent only, or mass- and metallicity-dependent.
If you encounter a problem, or have questions, please let me know. If I did not understand properly your issue, could you please provide more details so I can reproduce the problem?
Have a nice day.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Crash when viewing macOS address book database table MODELCACHE

Having seen you mention it over in the fediverse I thought I'd take this for a spin. Looks great! Not really being much of a user of SQLite I needed any database quick to view so I thought I'd dive into the Apple address book data.
If I try and view the table MODELCACHE I get a crash deep in the guts of Rich, down in markup.py, showing a MarkupError: closing tag ... at position 111721 doesn't match any open tag (where ... in there is a load of binary data).
It's possible that this is a Rich issue, or perhaps a Textual DataTable issue, but I wanted to flag it up for you first in case it's more a case of needing to be defensive about some kinds of data pulled from the database (in this case it's some unholy binary blob by the looks of things -- I would post the full information here but I'm not even sure if it might include sensitive data or not; hence the more vague report to start with).
Happy to move this over to the Textual issues if we feel that the DataTable might need to be a bit more defensive about content that might cause errors.

Thanks for this - it's definitely something I should handle gracefully. Check the data type of the column, and if it's a binary blob type, let the user can see there's data, but not try to show it or allow editing. Or maybe trying to edit a binary field could just ask to load the data from a file.
Not sure if you want to harden DataTable too, but it could be nice if it threw a more descriptive error message when it gets unexpected content.

On further investigation, the specific error is raised because DataTable is trying to render rich content in the cell - it encountered a string it interpreted as a closing format tag, something like [/bold], but didn't see an opening tag. Inevitably, the [/***] pattern just happened to show up in several kilobytes of binary image data converted to a string that I threw in there.
@davep , is there a way to disable Rich rendering in the DataTable? I should still fix it to not try to show a bunch of binary gunk, but that would prevent the crash at least.

@cdelker Off the top of my head I don't know of a way of turning off RIch rendering in the table, and thinking about it at the moment I can see how this could possibly be a source of some issues -- I might have to bring that up with Will and Darren and see what they think (in part Textual is about building on top of Rich so supporting that "out of the box" is going to be really useful for most folk, so having it "just work" by default feels like the right thing to do).
Perhaps the best thing for your app is sanitise the data being fed to the DataTable using rich.markup.escape?

My pleasure. And thanks for working with Textual. Looking forward to seeing how this application grows.
Also reminded me I should use SQLite for some stuff.


CHANGELOG release wrangling

Changelog for 0.6 & 0.7 do not have ticket information, which makes if difficult to trace backwards.
Whoever is wrangling the next release, could they please add ticket details, as we start to have more formalized testing, they will trace backwards from the CHANGELOG.
@jbeda | @vishh your names are last on the commit log for those branches.

What do you mean by "ticket details"?  What is missing?

Closing this, I'm keeping CHANGELOG up to date with my release notes.  @timothysc if you need other info, let me know.
Best
--brendan


Use Scale via Transform

Change the current way in which scale is done (using CSS Zoom) to use the CSS3 transform: scale(x).
Defaults to zoom if transform is unusable
Adds in a helper for checking whether a specific CSS property is useable within the current browser.

Changes Unknown when pulling b565621483227ec82e0d4389fe9f5d769990d715 on nick/scale-with-transform into ** on master**.


Partial fix for #34. npm v2 now works as long as hot mode is disabled.

This is a simple partial fix for #34, adding two libraries as peer dependencies for react-server-cli. In both npm 2 and npm 3, I was able to do the following:

clone example-reactserver
npm install
(for npm 3 only) npm install any uninstalled peer dependencies.
./node_modules/react-server-cli/bin/react-server-cli --hot=false
open localhost:3000

And things worked. Huzzah.
I still have dependency work to do to make hot reload mode work with npm v2, so #34 should stay open.
(Also, note that in order to get example-reactserver to serve up its page correctly, this bug fix PR needs to be pulled.) (PR was merged.)

I found a better way to do this that fixes both hot and non-hot mode, so I'm going to close this PR to make it clearer.


feat: adds ability to provide redirect uri

WIP PR for discussion: aiming to provide the ability to include a redirect URI, client ID, and client secrets to facilitate the migration away from "out of band" OAuth authentication.
@tswast
See also changes in these repos:

https://github.com/googleapis/python-bigquery-pandas/pull/595
https://github.com/googleapis/google-auth-library-python-oauthlib/pull/259
https://github.com/pydata/pydata-google-auth/pull/58

These failing tests are due to the fact that pydata-google-auth also needs a matching update and that PR (noted above) has not been merged into the main project yet. When that is merged, the expectation is that these tests will pass.


WatchableMutable History

Add Methods and Props to watchable mutable to add history to all plexus instances that inherit this primitive. This should have configurable history length.

Released with v1.3.5


ignore self signed certificate issue in request-promise

I am using request-promise module for my node app to make some API call. https://www.npmjs.com/package/request-promise
import request from 'request-promise';
let options = {
                method: GET,
                json: true,
                uri : "https://" +this.urls + endpoint,
                body: payload,
                rejectUnauthorized: false // This doesn't work
            };

let response = await request(options)

SInce the API what I am trying to use is insecure (having self signed certificate), the conncetion is failing with this error:
Error: connect ECONNREFUSED
I know with "request" module, we could pass rejectUnauthorized: false , to handle such case. I am not sure how can I pass such option with request-promise module.

Add insecure: true to your options. So it looks like
let options = {
                method: GET,
                json: true,
                uri : "https://" +this.urls + endpoint,
                body: payload,
                insecure: true
            };

@ecdeveloper Thanks for helping out! Quick question: Where did you find this option? Is it one that request forwards to the internal node libraries and which is documented in the node.js docs?
@jkjha Just in case insecure: true doesn’t work have a look at the section about agentOptions in the request README. It explicitly mentions the case of using self-signed certificates. Btw, all options for request work for request-promise as well.

+1

Having the same issue, insecure true didn't solve it

So I need to implement access to an mTLS protected resourcese, that means in neeed to provide the the CA which signed the service i am trying to access, as well as my own certificate and privateKey. what would use for that?

"rejectUnauthorized: false" doesn't work.. but am unable to understand why.. ?
It's a part of options correct?

Having the same issue, insecure true didn't solve it but rejectUnauthorized: false did

does nothing for me, i just get a different kind of error - still saying self signed

Adding both options seems to work for me :)

strictSSL: false
worked for me
And also making sure that json: true was taken out... or commented out

This works for me in request.get
request.get(url, {rejectUnauthorized: false}, function(err, res, body)


Cannot import name 'float32_io' from 'albucore'

Hi,
I'm running into this dependency import issue when trying to import Remover on transparent_background==1.3.3
>>> from transparent_background import Remover Traceback (most recent call last):   File "/usr/lib/python3.8/code.py", line 90, in runcode     exec(code, self.locals)   File "<console>", line 1, in <module>   File "/opt/chairisher/env/lib/python3.8/site-packages/transparent_background/__init__.py", line 1, in <module>     from transparent_background.Remover import Remover, console   File "/opt/chairisher/env/lib/python3.8/site-packages/transparent_background/Remover.py", line 15, in <module>     import albumentations as A   File "/opt/chairisher/env/lib/python3.8/site-packages/albumentations/__init__.py", line 6, in <module>     from .augmentations import *   File "/opt/chairisher/env/lib/python3.8/site-packages/albumentations/augmentations/__init__.py", line 1, in <module>     from .blur.functional import *   File "/opt/chairisher/env/lib/python3.8/site-packages/albumentations/augmentations/blur/__init__.py", line 1, in <module>     from .functional import *   File "/opt/chairisher/env/lib/python3.8/site-packages/albumentations/augmentations/blur/functional.py", line 9, in <module>     from albucore import clipped, float32_io, maybe_process_in_chunks, preserve_channel_dim ImportError: cannot import name 'float32_io' from 'albucore' (/opt/chairisher/env/lib/python3.8/site-packages/albucore/__init__.py)
When I revert back to transparent_background==1.3.2, the issue is resolved. Was this commit meant to address this issue: https://github.com/plemeri/transparent-background/commit/70b851d3abee30d025f4c006eda7ab43a44f7555?

Hi @kwokster10, it seems the problem is happening outside of our package. albumentation was introduced by https://github.com/plemeri/transparent-background/pull/56 to support cv2, numpy input directly. Maybe I have to look into it. Thanks for the report!


How do I pass restrictfilenames in a python script?

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.04.16. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.04.16

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

How do I pass restrictfilenames in a python script?
According to this Stack Overflow answer, the restrictfilenames option doesn't require an argument. The example provided, though, is for the command line execution. How do I pass restrictfilenames in ydl_opts?
I have tried both
ydl_opts = {
    'restrictfilenames': True
}

and
ydl_opts = {
    'restrictfilenames': None
}

But when I download the test video I get non-ASCII characters, which is not expected.
ydl_opts = {
    'retrictfilenames': None,
    'outtmpl': "%(title)s-%(id)s-%(resolution)s.%(ext)s",
}
with youtube_dl.YoutubeDL(ydl_opts) as ydl:
    meta = ydl.extract_info(link, download=False)
    print(ydl.prepare_filename(meta))

Output: youtube-dl test video ''_ä↭𝕐-BaW_jenozKc-1920x1080.mp4
The same output occurs with
ydl_opts = {
    'retrictfilenames': None,
    'outtmpl': "%(title)s-%(id)s-%(resolution)s.%(ext)s",
}
with youtube_dl.YoutubeDL(ydl_opts) as ydl:
    meta = ydl.extract_info(link, download=False)
    print(ydl.prepare_filename(meta))

Output: youtube-dl test video ''_ä↭𝕐-BaW_jenozKc-1920x1080.mp4
How do I pass restrictfilenames in a python script?

restrictfilenames works fine. Incorrect spelling in your code, missing s letter.


Describe what you want
    id bigint GENERATED BY DEFAULT AS IDENTITY PRIMARY KEY

There is also, ALWAYS but that means the ORM can never insert ID unless they override, so I am not sure if that should be supported from day one.
column_name type GENERATED { ALWAYS | BY DEFAULT } AS IDENTITY[ ( sequence_option ) ]

Can also support a few parameters, like sequence_option
id bigint GENERATED BY DEFAULT AS IDENTITY PRIMARY KEY (START WITH 10 INCREMENT BY 10),

Identity should be considered the default as best practice, as opposed to big/serial.

duplicate of https://github.com/drizzle-team/drizzle-orm/issues/261


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-29-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4857

Questions ?
Please refer to the Backport tool documentation


Arabic and RTL languages

Hi there,
As everybody knows by now that the Cube engine is being discontinued and a new OCR engine based on LSTM will be introduced, even-though neither the Cube training tools or it's documentation been published.
One can only see that there is purposely neglect and sabotage to the  "RTL" Right to Left language community.
Hindering the developer community of training the Tesseract on RTL languages.
Issues such as that Tesseract while training considers all the letters and words as a single word, and the training is conducted as training a single word, along with many other issues while training RTL languages have been neglected for years and years, Tesseract showing no interest in solving any of the RTL languages issues.
And yet we ask Ray to publish the documentation and tools to train Cube before it is fully removed from tesseract, or at least assure the community that the RTL languages (Arabic, Hebrew...) will be considered in future version releases of Tesseract and future tools will be published helping the developer community in the training process of RTL languages.
List of some issues in the RTL training:

Tesseract consider all words as a single 1 Word while training.
"Generated training data for 1 words"

Tesseract fail to add some letters while training.
"APPLY_BOXES: boxfile line * : FAILURE! Couldn't find a matching blob"

List of some issues in the RTL recognition:
3) Tesseract combines all the letters together while recognition.
4) Tesseract reverse the direction of the words while recognition.
Suggested solution for some of these problems:
2) The "box file" must be created with a criteria inwhich the boxes for each letter must not conjoin or merge with any other box, meaning that each box is an entity for a single glyph and does not have any part of it enter another box of another glyph.

Adding "ara.config" file in the folder you wish to train while modifiying it by setting "tessedit_ocr_engine_mode 0" instead of "1", or removing the entire line.

Left-to-right languages are incredibly important. Arabic,  for example, has hundreds of
millions of speakers and is the fifth most widely used language when including all variants.
I recommend looking at the DAS tutorial slides, they are interesting reading. The last
set 'Building a Multi-Lingual OCR Engine' talks about in-progress numbers with Farsi
and other languages that currently perform poorly with Tesseract 3. Machine learning
has advanced quite a bit recently, and that's where the energy is.
http://www.vistawide.com/languages/top_30_languages.htm
https://github.com/tesseract-ocr/docs/tree/master/das_tutorial2016

Take a look at DAS 2016 slides

but what about solving the issues?

@christophered No one is hindered or attempting to sabotage RTL languages. Get over yourself. You can use and develop a fork of Tesseract 3.0 until the cows come home if you so choose.
The Arabic trained data is available in the tessdata repo, and if you want to submit patches to improve the LTSM engine for Arabic, you can. If not, that's fine, maybe someone else familiar with both the language and Tesseract will come around. In an open source community, you are not entitled to anything...if you want things fixed, submit patches -- you're already pretty close having already identified some problems.

You replied to a 9 month old post.
Since then, LSTM code landed in the repo, and the OP opinion had changed.
https://github.com/tesseract-ocr/tesseract/issues/630#issuecomment-271392038

Hi, im using tesseract 4.00 and leptonica 1.75.3 i used eng.traindata for some image and it worked very well . now im trying to use the same code for arabic i used ara.traindata but it gives a weird characters. is it due to getUTF8Text(); or that has nothing to do with it

Hi, I am facing the same issue Arabic text extracted using pytesseract.image_to_string(filename,'lang=ara') is from left to

right.
original image


Changing rooms - hardcoded popup text

Describe the bug
When in an active room, changing to another room, a popup appers with hardcoded english text in it. Its a problem because then we cant translate it in the locales.
Screenshots

Dup of #1234


Add dependency review workflow

Add workflow to review dependencies - documentation.

Codecov Report

Merging #545 (1ef3ff6) into main (06e8fea) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #545   +/-   ##
=======================================
  Coverage   97.39%   97.39%
=======================================
  Files          15       15
  Lines         846      846
=======================================
  Hits          824      824
  Misses         22       22

Flag
Coverage Δ

linux
97.39% <ø> (ø)

macos
97.39% <ø> (ø)

windows
?

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[GUI-173] Add labels with storage driver type.

Description
Storage driver type should be displayed as a label within run table's row.
Related JIRA's issue: https://mongoose-issues.atlassian.net/projects/GUI/issues/GUI-173


Make better app icon

The current icon isn't a good resolution and is also a total placeholder. Doesn't look great on the Fitbit device.

Done.


C++ server bind to IPv6 address but not IPv4

I'm running examples/cpp/helloworld on ubuntu, in greeter_server.cc there is an IPv4 address to be used as server address:
std::string server_address("<IP_ADDRESS>:50051");

But in fact the IPv6 address is used:
root@vultr:~/grpc/examples/cpp/helloworld# netstat -lnt
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN
tcp6       0      0 :::22                   :::*                    LISTEN
tcp6       0      0 :::50051                :::*                    LISTEN

My fault, :::50051 is both IPv4 and IPv6 :smile:


no_priority: reduce to bugs created less than 1 year ago

Coverage remained the same at 31.236% when pulling 14a7fdc8cabf65e6445eaa08a0ab035ca53377c6 on calixteman:workflow_1_year into 508b438b19d451f15723a83b1fe73de02864e883 on mozilla:master.


CORS doesn't work with other request methods than GET & POST

I was trying express-zod-api and found errors with CORS.

For request with methods different of GET and POST, it looks like the CORS middleware doesn't exits, similar to #514
Reproduction repository https://github.com/leosuncin/express-zod-api-cors-error

Hello @leosuncin ,
I'm sorry, but your repo is relatively big... so I'm not sure I figured out what's happening there.
But, do I understand correctly, that on your screenshot you're calling a path that is not present in the routing?
I see you have /health and /todo specified, but you're calling localhost:3000/... just with some hash or ID. Right?
So, could you please clarify how this request you expect to be handled? Since it appears to me as you're requesting the path that does not listed in the routing, so therefore it does not have any supported methods at all.
Looking forward to your reply.
P.S. It would be great if you could provide me with exact cURL command to reproduce the issue. So I could clone your repo, start it and check it out.

Sorry for the mess, I simplified and reduced the code a lot, please check it again.
A cURL command won't help to debug it since the CORS issue only happens when the browser does a Cross-Origin Request, that's the reason why the reproduction code is composed by a server and a front-end app running on different ports.

Furthermore, I encountered other errors: the Authorization header is not accepted because the default configuration only allow Content-Type
https://github.com/RobinTail/express-zod-api/blob/2385f16040116f40c4972c080f9f75577dc0a25c/src/endpoint.ts#L201
This also cause problems with cookies because the header Set-Cookie is not available in the client side.
I'd like to recommend to use cors instead of defining the headers
https://github.com/RobinTail/express-zod-api/blob/45f34af4649ca0814bba14b430601afd765766ec/src/endpoint.ts#L193-L203

@leosuncin

Furthermore, I encountered other errors: the Authorization header is not accepted because the default configuration only allow Content-Type
This also cause problems with cookies because the header Set-Cookie is not available in the client side.

It's already possible.
For that you need to change the config setting from cors: true to a function that returns headers.
Please check out the following article of the documentation:
https://github.com/RobinTail/express-zod-api#cross-origin-resource-sharing

I'd like to recommend to use cors instead of defining the headers

Check out the following example on how to connect the cors express middleware:
https://github.com/RobinTail/express-zod-api#using-native-express-middlewares

@leosuncin

A cURL command won't help to debug it since the CORS issue only happens when the browser does a Cross-Origin Request, that's the reason why the reproduction code is composed by a server and a front-end app running on different ports.

Any request is possible to reproduce.
In particular I'm asking you to copy OPTIONS request as cURL from your browser or a tool you're using.
This option is available in a context menu.

The OPTIONS request
curl 'http://localhost:3010/hello' -X OPTIONS -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:107.0) Gecko/20100101 Firefox/107.0' -H 'Accept: */*' -H 'Accept-Language: es-MX,es;q=0.8,en-US;q=0.5,en;q=0.3' -H 'Accept-Encoding: gzip, deflate, br' -H 'Access-Control-Request-Method: PUT' -H 'Referer: http://localhost:3020/' -H 'Origin: http://localhost:3020' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: same-site' -H 'Pragma: no-cache' -H 'Cache-Control: no-cache'

The PUT request
curl 'http://localhost:3010/hello' -X PUT -H 'User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:107.0) Gecko/20100101 Firefox/107.0' -H 'Accept: */*' -H 'Accept-Language: es-MX,es;q=0.8,en-US;q=0.5,en;q=0.3' -H 'Accept-Encoding: gzip, deflate, br' -H 'Referer: http://localhost:3020/' -H 'Origin: http://localhost:3020' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: same-site'

Both works in the terminal, but not in the browser
Download CORS_error.zip, unzip it and upload the HAR file to https://toolbox.googleapps.com/apps/har_analyzer/

Thank you @leosuncin .
For some reason your setup of the /hello endpoint returns the following header in the response to OPTIONS request:
Access-Control-Allow-Methods: GET, OPTIONS

Investigation continues.

Yeah, this is a very nice bug, @leosuncin .
Thank you very for finding this!
The reason is the way of processing the DependsOnMethod instance. In your case each method technically has a separate Endpoint instance. Each of them is for one method only. The current implementation creates several OPTIONS handlers for the same path, but only the first one being called (GET). So it responds with the wrong list of methods actually supported by the path.
This is bug. I'm going to find the way to fix it.

The exact location of the issue:
https://github.com/RobinTail/express-zod-api/blob/45f34af4649ca0814bba14b430601afd765766ec/src/routing.ts#L66

The issue with allowed methods is fixed in recently released v8.4.1 🚀
Thank you again for finding this, @leosuncin
I'm also going to fix it in v7.
It should work in your repo now, but without authorization.
In order to allow Authorization header, consider changing cors config option into a function. See the example above.


why libprofiloextapi.so not exits ?

Thank you for this job.
I try to assemble it in my app, but it crashed when launch, say

could't find libprofiloextapi.so

I unzipped the release aar and found nothing with name "libprofiloextapi.so"
For help

Sorry about that, that's a legitimate bug. Fix incoming.


Adding a fr translation for contribute-tools

Yet another file to help on https://github.com/rust-lang/rust-www/pull/501

👍


iOS 16.4 printing - background graphics support

iOS 16.4 add property to allow background graphics when printing the web page. Default value is false, so I need to do this change to allow background graphics.
There is a link to Apple change log.

Thank you for finding this! It was driving me mad and I'd not discovered the cause myself. 😄


Create an encrypted note from selection

Explore the idea of having the ability to select text, then choose 'create encrypted note' from the command pallet.
This would create a new whole-encrypted note with the contents of the selection.
The original selected text becomes a link to the new encrypted note.

Hey, as I commented on Issue 150, maybe this would also work really well from the context menu of the selection:

There could be multiple commands in the Meld Encrypt section, including "Create Encrypted Note from selection".


Added apiserver/bundlestore stats.

Codecov Report

Merging #219 into master will increase coverage by -0.31%.

@@            Coverage Diff             @@
##           master     #219      +/-   ##
==========================================
- Coverage   57.53%   57.22%   -0.31%
==========================================
  Files          86       86
  Lines        4119     4157      +38
==========================================
+ Hits         2370     2379       +9
- Misses       1512     1541      +29
  Partials      237      237

Impacted Files
Coverage Δ

snapshot/bundlestore/groupcache.go
0% <ø> (ø)
:white_check_mark:

snapshot/bundlestore/server.go
66.66% <91.66%> (+4.66%)
:white_check_mark:

Continue to review full report at Codecov.

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


Container error thrown when using shadow DOM

Trying to use F2 within the shadow DOM but an error is thrown by container.js when checking that the root has been added to the page.
App root for ' + appId + ' was not appended to the DOM. Check your AppHandler callbacks to ensure you have rendered the app root to the DOM.')
The code is checking that the root has a parent node that is the document. In order to allow an F2 app to be used in the shadow DOM, this check should include if there is a parent node in the shadow DOM.

@mgaughan, Do you have an example? I am assuming you are talking about thecheck in elementInDocument.
I think we could use something like this to detect if it is in the ShadowDOM
if(parent.toString() === "[object ShadowRoot]") {
return true;
}
from http://stackoverflow.com/questions/27453617/how-can-i-tell-if-an-element-is-in-a-shadow-dom

@qrider - correct, that's the code I've been using locally. I have forked and made the change - just need to add a unit test.


Ghostscript error

When I tried to run ghostscript.m I received this error:
Error in ghostscript (line 50)
[varargout{1:nargout}] = system([gs_command(gs_path()) cmd]);
Does anyone know what happened? Thank you for your help!

no response received, so closing this issue


Add curled and dotted underline for diagnostics

I'm wondering if support for underline styles (helix#3015, superseded by helix#4064) is planned. This would bring the themes in this repo on par with the default themes in terms of diagnostic support.
There are currently 4 underline styles:

curl
dashed
dotted (docs mention dot but that is not correct on master)
double_underline

These should ideally feel familiar, so the color should match the gutter icon's color, the style should match what other popular editors do.
Other Editors
Diagnostic differences:

helix/lsp
intelij-based
vscode

error
error
error

warning
warning
warning

info
hint
info

hint
suggestion
info (different styling)

IntelliJ-based IDEs

severity
underline type
color
styling

error
undercurled
vibrant red
-

warning
undercurled
soft red
-

info
underdotted
vibrant cyan
short span, nothing we have control over

hint
undercurled
soft yellow
-

Visual Studio Code

severity
underline type
color
styling

error
undercurled
vibrant red
-

warning
undercurled
vibrant yellow
-

info
underdashed
vibrant cyan
short span, nothing we have control over

hint
underdashed
soft yellow
-

Most themes for these editors don't interfere with the default styling of the diagnostics. The changes are simple
+ "diagnostic.hint" = { underline = { color = <hint>, style = "dotted" } }
+ "diagnostic.info" = { underline = { color = <info>, style = "dotted" } }
+ "diagnostic.warning" = { underline = { color = <warning>, style = "curl" } }
+ "diagnostic.error" = { underline = { color = <error>, style = "curl" } }

As well as removing the deprecated option for the fallback style and changing it to curl (same as the default theme):
- "diagnostic" = { modifiers = ["underlined"] }
+ "diagnostic" = { underline = { style = "line" } }

I've tried this with gruvbox_original_dark_hard only, but I figure this Is simple enough that it should work for all of them.
It's up to you if prefer curl for all of them or dashed over dotted, I thought the distinction from color only is a bit hard to see, and the dashed underline looked kind of weird to me.

Using gruvbox original dark hard in alacritty:

Using gruvbox original dark hard in kitty:

The same with dashed as reference:

I see this on wezterm, are you up to date?

Looks great! Interestingly, I'm using the same wezterm version but the styled underlines don't show up for me:

I'll have to check why that happens. Regardless, the changes and visuals look good to me, so unless there's other things you want to address I'd go ahead and merge this.

Perhaps you're still on helix 22.08, I'm running master because the arch package is always a bit late. So while it is released, it may not be available on your system either.

If you're ok with dotted over dashed then it's good to merge.

Thanks!


Sample function of MixtureDistribution class errors for (d,)-shaped distributions when num_samples>1

Description
I came across a bug in the MixtureDistribution class, where sampling is prohibited by an incompatible shapes broadcast (see MWEs 1 and 2 below).
To Reproduce
MWE-1 demonstrates where sampling is prohibited by an incompatible shapes broadcast
import mxnet as mx
from mxnet import nd
import numpy as np
gaussian = Gaussian(mu=nd.array([0.]), sigma=nd.array([1.]))
gamma = Gamma(alpha=nd.array([9.]), beta=nd.array([2.]))
mixture = MixtureDistribution(mixture_probs=nd.array([0.9, 0.1]), components=[gaussian, gamma])
mixture.sample(num_samples=1) # WORKS
mixture.sample(num_samples=10) # DOESN"T WORK

MWE-1 Error message or code output
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/IPython/core/interactiveshell.py", line 3417, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-30-52d279ea9fbf>", line 9, in <module>
    mixture.sample(num_samples=10) # DOESN"T WORK
  File "/Users/eeehrlic/Documents/projects/capstone/gluon-ts/src/gluonts/mx/distribution/mixture.py", line 155, in sample
    # idx = idx.broadcast_like(samples_list[0])
  File "/usr/local/lib/python3.8/site-packages/mxnet/ndarray/ndarray.py", line 2333, in broadcast_like
    return self.broadcast_to(other.shape)
  File "/usr/local/lib/python3.8/site-packages/mxnet/ndarray/ndarray.py", line 2290, in broadcast_to
    raise ValueError(err_str)
ValueError: operands could not be broadcast together with remapped shapes[original->remapped]: (10,) and requeste

MWE-2 Incompatible shapes broadcast also prohibits sampling from mixture of multivariate distributions.
from gluonts.mx.distribution import MixtureDistribution
from gluonts.mx.distribution.multivariate_gaussian import *
mu1_bi = mx.nd.array([0., 0.])
mu2_bi = mx.nd.array([2., 2.])
sigma_bi = mx.nd.array([[0.01, 0.], [0., 0.01]])
mvgaussian1 = MultivariateGaussian(mu=mu1_bi, L=sigma_bi)
mvgaussian2 = MultivariateGaussian(mu=mu2_bi, L=sigma_bi)
p = mx.nd.array([0.6])
mix_prob_tensor = mx.nd.stack(p, 1.0 - p, axis=-1)
mixture = MixtureDistribution(mixture_probs=mix_prob_tensor, components=[mvgaussian1, mvgaussian2])
samples = mixture.sample() # DOESN'T WORK
samples = mixture.sample(1) # DOESN'T WORK
samples = mixture.sample(10) # DOESN'T WORK

MWE-2 Error message or code output
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/IPython/core/interactiveshell.py", line 3417, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-5-6d163038db74>", line 1, in <module>
    samples = mixture.sample() # DOESN'T WORK
  File "/Users/eeehrlic/Documents/projects/capstone/gluon-ts/src/gluonts/mx/distribution/mixture.py", line 155, in sample
    idx = idx.broadcast_like(samples_list[0])
  File "/usr/local/lib/python3.8/site-packages/mxnet/ndarray/ndarray.py", line 2333, in broadcast_like
    return self.broadcast_to(other.shape)
  File "/usr/local/lib/python3.8/site-packages/mxnet/ndarray/ndarray.py", line 2285, in broadcast_to
    raise ValueError(err_str)
ValueError: operands could not be broadcast together with remapped shapes[original->remapped]: (1, 1) and requested shape (2,)

Environment

Operating system: MacOs 10.14.6
Python version: Python 3.8.5
GluonTS version: 0.5.1.dev92+g14a1ec8
MXNet version: 1.6.0

Hey @elenaehrlich,
I think this is not a bug, but we should maybe catch this with an assertion... The mixture_probs have to be of shape (1,2). This will work:
import mxnet as mx
from mxnet import nd
import numpy as np
gaussian = Gaussian(mu=nd.array([0.]), sigma=nd.array([1.]))
gamma = Gamma(alpha=nd.array([9.]), beta=nd.array([2.]))
mixture = MixtureDistribution(mixture_probs=nd.array([0.9, 0.1]), components=[gaussian, gamma])
mixture.sample(num_samples=1) # WORKS
mixture.sample(num_samples=10) # WORKS

The reason is that the one dimensional batch shape is a special case of higher dimensional batch shapes:
gaussian = Gaussian(mu=nd.array([0., 0.1]), sigma=nd.array([1., 1.1]))
gamma = Gamma(alpha=nd.array([9., 1.]), beta=nd.array([2., 3.]))
mixture = MixtureDistribution(mixture_probs=nd.array([[0.9, 0.1], [0.1, 0.9]]), components=[gaussian, gamma])
mixture.sample(num_samples=1) # WORKS
mixture.sample(num_samples=10) # WORKS

Never mind, I just saw your PR regarding this and I think this can be closed.


Fix for minidump support

This allows minidump support to compile and be setup correctly when the checkbox is checked. The VS lib templates were missing TORQUE_RELEASE in the preprocessor definitions which caused minidump support to not be enabled regardless of a checkbox or not.

This is in response to #1191 and #1192

Do we need to do anything about the TORQUE_MINIDUMP define or is that one okay after all?

TORQUE_MINIDUMP is being set correctly, this fix is just for adding TORQUE_RELEASE for release builds which is required along side the MINIDUMP define, and also to get a file to compile when its included from the 2 defines.

Thanks for the fix!

@adambeer could you sign the OSA agreement in your GG.com account? I should have checked before merging :).

Done. I knew about this and probably should have done it before submitting ;)

Thanks!


Create a github workflow to test pull requests

For now we only lint typescript code.

@ChrisJamesC This is great thank you 🙌🏼

@ChrisJamesC Thanks for creating the PR!! I'm seeing some issues when running this workflow:
https://github.com/Shopify/react-native-skia/runs/5033854401?check_suite_focus=true
Could you see if there is something missing?

Thanks for the approval.
It passes now. We can still discuss if you would like to regroup common dependencies (eslint, react) in the parent package or keep importing them at the 3 different places. Or we can move on with this, since it works for now.

We'll merge it since it works for now :) :) Thanks so much!!


ember-getowner-polyfill depreciated

[X] I am on the latest ember-intl version
[X] I have searched the issues of this repo and believe that this is not a duplicate

Environment

Ember Version: 2.10.0
Ember CLI Version:2.10.0
Ember Intl Version: DOWNLOADED TODAY
Browser(s): tested in Chrome
Node Version:6.9.2

Steps to Reproduce

DEBUG:
ember.debug.js:5732DEBUG: Ember      : 2.10.0
ember.debug.js:5732DEBUG: Ember Data : 2.10.0
ember.debug.js:5732DEBUG: Firebase   : 3.6.3
ember.debug.js:5732DEBUG: EmberFire  : 0.0.0
ember.debug.js:5732DEBUG: jQuery     : 3.1.1
ember.debug.js:5732DEBUG: -------------------------------
ember.debug.js:5563DEPRECATION: ember-getowner-polyfill is now a true polyfill.

Dear Jason,
many thanks for your fast answer.
I think the ember-getowner-polyfill as a direct dep is depreciated since Ember 2.3.
And thanks for your great job !
All the best
Joachim Bäurle

Am 13.12.2016 um 07:46 schrieb Jason Mitchell<EMAIL_ADDRESS>Thanks, I plan to remove ember-getowner-polyfill as a direct dep soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jasonmit/ember-intl/issues/406#issuecomment-266660456, or mute the thread https://github.com/notifications/unsubscribe-auth/AVx-5N1OZpSYBeN1M9u2vksfRsKj5N6hks5rHj80gaJpZM4LLMWj.

@jbpresident should be resolved in 2.16.1.  Feel free to reopen if you still see it.
Thanks for opening the issue!

Dear Jason,
many many thanks from Europe ! Solved ! And so fast !
ALL THE BEST !
Joachim Bäurle

Am 13.12.2016 um 23:59 schrieb Jason Mitchell<EMAIL_ADDRESS>@jbpresident https://github.com/jbpresident should be resolved in 2.16.1. Feel free to reopen if you still see it.
Thanks for opening the issue!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jasonmit/ember-intl/issues/406#issuecomment-266889366, or mute the thread https://github.com/notifications/unsubscribe-auth/AVx-5Cimsr6zs8My4c_Uzaz9GADI4VpOks5rHyM5gaJpZM4LLMWj.

Dear Jason,
we still have on problem with ember-intl on Safari.
If we try to load the polyfill from cdn polyfills.io http://polyfills.io/ as described in your documentation we get the following return :
/* Disable minification (remove .min from URL path) for more info */
and no translation of the language.
Our settings are in environment.js are :
    // ember-intl - Polyfill - Load if needed !!
    // https://github.com/jasonmit/ember-intl
    'polyfill-io': {
      features: [
        'Intl.~locale.en-US',
        'Intl.~locale.de-DE',
        'Intl.~locale.es-ES'
      ]
    }

If we do an call directly from the browser without .min we get the following return :
/* Polyfill service v3.15.0

For detailed credits and licence information see https://github.com/financial-times/polyfill-service.

UA detected: safari/10.0.0
Features requested: Intl.~locale.de-DE,Intl.~locale.en-US,Intl.~locale.es-ES
*/

/* No polyfills found for current settings */
and no translation of the language.
ANY IDEA ?
best regards
Joachim Bäurle

Am 14.12.2016 um 08:51 schrieb Joachim Bäurle<EMAIL_ADDRESS>Dear Jason,
many many thanks from Europe ! Solved ! And so fast !
ALL THE BEST !
Joachim Bäurle

Am 13.12.2016 um 23:59 schrieb Jason Mitchell<EMAIL_ADDRESS><EMAIL_ADDRESS>@jbpresident https://github.com/jbpresident should be resolved in 2.16.1. Feel free to reopen if you still see it.
Thanks for opening the issue!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/jasonmit/ember-intl/issues/406#issuecomment-266889366, or mute the thread https://github.com/notifications/unsubscribe-auth/AVx-5Cimsr6zs8My4c_Uzaz9GADI4VpOks5rHyM5gaJpZM4LLMWj.

@jbpresident could you open a separate issue.  Also, when you do that if you could throw together a quick dummy app, would speed up the debug process.

It sounds like the polyfills just aren't needed in Safari. If you want to force them to be loaded anyway, you can do this in the config instead:
'polyfill-io': {
  features: [
    'Intl.~locale.en-US',
    'Intl.~locale.de-DE',
    'Intl.~locale.es-ES'
  ],
  flags: [
    'always'
  ]
}


Is the 6.0 NewArea not yet supported?

you can only select the following three options...
Gridania
Limsa
Uldah
...

I'm in the process of creating an explanation.
The teleport button is for when you want to do a world visit.
In the new version(0.9.5), the name has been changed,
but if you put a checkmark at the top and go to the EW area's Aetheryte, you can select an instance with one button.


setup for resizable windows, drop weight

as it says, it drops the weight method for calculating size,
and instead uses previous width/height.

How does this behave, in plain english?

it sums up the total width of child containers,
(if a child container has width/height 0 it is given a default value of to fit in container nicely)
it then sets the scaling to be parent->width / total_width
if it is unchanged, scaling is 1, and widths/height are unchanged.
if total width is smaller (as a result of removing / resizing container), the scaling is 1.n, and the widths/height of all the children are increased to fill the parent container.
if total width is larger (as a result in inserting / resizing container), the scaling is 0.n, and the widths/height of all the children are decreased to fill the parent container.

Looks good from my testing.


Missing prerequisites.txt file

Hi
Your prerequisites.txt file is missing, could you please provide the file.
"
To install the repo in your virtual environment, run the following:
Clone the repo
git clone https://github.com/maragraziani/interpretableWSItoRNAseq.git
Install NPM packages
python3 -m pip install -r prerequisites.txt
"

Hi!! I added it now! Thanks for spotting the missing file.
Cheers
Mara


Belarusian life and maxiphone carrier

Country: Belarusian BY
Example number(s) and/or range(s): +375 25 333 33 33
Number type ("fixed-line", "mobile", "short code", etc.): mobile
For short codes, cost and dialing restrictions:
Where or whom did you get the number(s) from: http://life.com.by
Authoritative evidence (e.g. national numbering plan, operator announcement):
Link from demo (http://libphonenumber.appspot.com) showing error:

Hello. There a phone numbers +375 25 xxx xx xx (BY - region) Belarus.
25 - code "life" mobile carrier - does not recognized.
http://life.com.by
and +375 24 xxx xx xx
24 - code "mixaphone" mobile carrier - does not recognized.
http://www.maxiphone.by

@mikhailyaskou,
Hi,
Thank you for contributing to libphonenumber!
We can't connect to +375 25 333 33 33. Did you have a typo?
As per official evidence of Belarus http://www.beltelecom.by/en/subscribers/phone-codes we support only 25[5679] prefixes in Mobile and 24[9] prefix as VOIP https://www.beltelecom.by/chasto-zadavaemye-voprosy.
We need valid evidence to support the remaining prefixes i.e 24[0-8] and 25[0-48]. Please share any supporting evidence (for example, a national numbering plan, announcements from mobile carriers, news articles) you may have. See our guidelines.

@mikhailyaskou,
This issue is blocked on your or other GitHub contributors' input. Please help us find evidence that 24[0-8] and 25[0-48] are valid, or provide an example number from a business website or similar. Otherwise we don't have enough information to fix the issue and may have to close it as infeasible.

I'm closing as this issue is infeasible for us to fix at this point (we need more information). Please re-open if you respond so we pick up on it.


Installation

Hello!
I have been trying to install the library as per your instructions.

When using:
pip install molgrid
I received the following message:
ERROR: Could not find a version that satisfies the requirement molgrid (from versions: none)
ERROR: No matching distribution found for molgrid

When using:
conda install -c jsunseri molgrid
I received the following error:
PackagesNotFoundError: The following packages are not available from current channels:

molgrid

I tried this in a freshly created environment (python 3.7).  Could you please advise how I should proceed to get it installed? In the future, I am interested in integrating it with the following repository: https://github.com/fimrie/DenseFS
Best wishes,
Kadi

Thank you @dkoes. It's:
WARNING: Package(s) not found: molgrid
Yes, having a conda package would certainly be helpful.

Hello @Luc1100 and @dkoes,
Do you have an estimate when updated conda package may be available?
Thank you and best wishes,
Kadi

Hello @Luc1100 and @dkoes,
Do you have an estimate when updated conda package may be available?
Thank you and best wishes,
Kadi

@kadiliissaar The package can be found here: https://anaconda.org/gnina/molgrid
I have not done much testing other than checking that it imports. Let me know if you have an issues.

Thank you @Luc1100! I see the anaconda page only lists Linux, are you planning on making it MacOS compatible as well?

@kadiliissaar I uploaded one, however, I cannot say if it works at all as I have an M1 Mac and libmolgrid requires CUDA.

Fair enough! I am just trying conda install on a linux machine but the environment resolving has been going on for 10+ mins. Do you know why this may be?

@kadiliissaar Are you working in a clean environment? I tested on another linux machine and it installed pretty fast.

Thank you @Luc1100. I tried now in a clean environment (with python 3.9) and the installation went fine.
However, when trying to import molgrid, I received the following message:
from .molgrid import *
ImportError: libboost_python39.so.1.73.0: cannot open shared object file: No such file or directory
Do you know why this may be?

Hi,
any chance of getting the PyPi wheel updated to support MacOS too?
Thanks!


Public path and output path do not work together?

I'm using the latest version of file-loader. All I want to do is pull some images from a node_modules package and emit it to my output directory. But I've been struggling with this for hours.
I have the following directories involved in this:

/assets/css/
/assets/misc/

When I find example.png in /assets/css/main.css, I want file-loader to copy that file to /assets/misc/hash.png and generate the URL to the CSS file as ../misc/example.png.
loader: 'file-loader',
options: {
	outputPath: 'assets/misc/',
	publicPath: '../misc/'
}

When I try the above configuration, the generated URL is correct, but no files are actually emitted anywhere - outputPath is ignored! If I remove the publicPath line, then the files are correctly emitted to /assets/misc/, but the generated URL is incorrect (assets/misc/example.png instead of ../misc/example.png.
I've tried alternative solutions as well:
loader: 'file-loader',
options: {
	name: baseWebsiteUrl + '[path][hash].[ext]',
	outputPath: 'assets/misc/',
}

When I try the above configuration (with baseWebsiteUrl being a string containing my website's base URL), the files are again correctly emitted to /assets/misc/, but the generate URL is still wrong - this time it's assets/misc/MYWEBSITEURL/node_modules/NODEPACKAGE/example.png.
Why can't I simply use publicPath and outputPath together? Surely this is a bug?

Seems like it's working now. No idea why.

@HunterGraubard
So which config can work?
Can you post the final config setting?
I am confused with similar problem like this, too.

Sorry, I don't really understand it myself. I've found this plugin very confusing to use.
test: /some-filename-from-node-package\.png$/,
use: [{
	loader: 'file-loader',
	options: {
		publicPath: '/assets/misc/',
		outputPath: 'assets/misc/',
	}
}]

I ended up using this configuration. It writes the files to the assets/misc directory, and writes the URL into the stylesheet with a leading / to make it root-relative. Using a baseWebsiteUrl variable at the start of publicPath is working too now (e.g. publicPath: baseWebsiteUrl + 'assets/misc/' writes an absolute URL into the stylesheet).
My main problem before was that when I used publicPath, no files would be emitted, making it pointless. But today it's suddenly working, so the problem is gone.

@HunterGraubard Can you create minimum reproducible test repo and write in readme what you have and what you expected?


If you use a Noutlist's Ploy on a Scroll and Quill it can't be opened until you write something else to it.

Screencast from 2024-09-12 16-37-33.webm
You can't see it on the recording but I am trying to open the scroll, right clicking it.

This is a known issue, though I've yet to look into its cause.

Fixed in commit https://github.com/enjarai/trickster/commit/b0bc63bf94b29d17c216e6743e641d431c1d3821.


fix broken links in template creation guide

Description
Fixes broken links in the template creation guide.
Related issues
Addresses [issue #].
Testing
Verified that the updated links in the template creation guide are working as expected and link to the correct pages.

Thanks for helping to refine to documents!

Thanks for the review @BoyaWu10 - are you able to merge this in for me?

Sure!


Incorrect result when comparing floats and ints in Expresso script

Hello :) We are running Articy with Unreal, using this plugin, and we have run into an issue that seems rather... disturbing.
Consider the following Expresso condition:
getProp(getObj("Player"), "PlayerProperties.Intimidation") < 3
PlayerProperties.Intimidation is a float, and is compared against an integer value. I would expect the condition to evaluate to true if the float value is larger than 3.0, and false otherwise. The result, however, is undefined behaviour.
After debugging, I managed to track down the issue. The generated C++ looks like this:
return ConditionOrTrue(
	getProp(getObj("Player"), "PlayerProperties.Intimidation") < 3
)

This causes an implicit cast - the value 3 is cast to ExpressoType with type set to Int. Then, ExpressoType::operator< is evaluated; however, this operator assumes that both expressions are of the same type; it just performs GetFloat() < Other.GetFloat(). However, in this case Other is an Int; so we are reading undefined values from a union.

Thanks for notifying us.
While this "same-type" assumption is made throughout the Unreal expresso script system, we will include a fix for the most common comparison operators regarding the float and int mismatch, including <, >, >=, <=, == and !=.


improved --merge subcommand

Change the behavior of slipcover --merge as follows:

human-readable output of the merged result is printed to stdout, unless --silent is passed
in json output mode, honor the --pretty-print flag
support XML output mode via --xml flag
honor --fail-under flag

Motivated by https://github.com/plasma-umass/slipcover/issues/62


Add fantasy-land integration for valid pointfree functions

Now that we have the types working with fantasy-land functions from other libs, makes sense to make the integration both ways. All of the pointfree functions that have corresponding fantasy-land definitions should check for the namespace. These functions should check if the fantasy-land methods exist first and then fall back to the "public" if it does not exist.

Got the last bit for this done (except for traverse and ap).
closing.


[Form-Recognizer] ContainerRegistryContentClient (service version 2021-07-01) can upload Docker manifest failing in nightly runs

Form-Recognizer nightly test runs are failing with:

Error message:
{"errors":[{"code":"MANIFEST_BLOB_UNKNOWN","message":"blob unknown to registry","detail":"sha256:feb5d9fea6a5e9606aa995e879d862b825965ba48de054caab5ef356dc6b3412"},{"code":"MANIFEST_BLOB_UNKNOWN","message":"blob unknown to registry","detail":"sha256:2db29710123e3e53a794f2694094b9b4338aa9ee5c40b930cb8063a1be392c54"}]}
Stack trace:
RestError: {"errors":[{"code":"MANIFEST_BLOB_UNKNOWN","message":"blob unknown to registry","detail":"sha256:feb5d9fea6a5e9606aa995e879d862b825965ba48de054caab5ef356dc6b3412"},{"code":"MANIFEST_BLOB_UNKNOWN","message":"blob unknown to registry","detail":"sha256:2db29710123e3e53a794f2694094b9b4338aa9ee5c40b930cb8063a1be392c54"}]}
at handleErrorResponse (/Users/runner/work/1/s/sdk/core/core-client/src/deserializationPolicy.ts:255:17)
at deserializeResponseBody (/Users/runner/work/1/s/sdk/core/core-client/src/deserializationPolicy.ts:159:43)
at processTicksAndRejections (node:internal/process/task_queues:95:5)

For more details check here:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2768086&view=results

Fixed in #25841


#1712 react readonly textarea

Before (the change)
textarea react component does not expose readonly prop, like the textarea web component does.
After (the change)
textarea react component does expose readonly prop
Make sure that you've checked the boxes below before you submit the PR

[x] I have read and followed the setup steps
[x] I have created necessary unit tests
[x] I have tested the functionality in both React and Angular.

Steps needed to test
Add textarea react component with readonly prop to react playground, then as a user try to change its input value. Expected result: Textarea can receive focus but won't accept keyboard input.

:tada: This PR is included in version 1.17.0-alpha.129 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 3.0.0-alpha.10 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Two key releases at once causes one to not be reported

Running 3.1.2 on windows, compiled as a cmake reference from another project with mingw x86. When I hit two keys ('r' and 'v') and release them both at once in the "events" tool, the release of one of them is not picked up until I either press another key (like Alt), or defocus the window.
Below is an example of pressing r & v and releasing them both immediately, then using the mouse and focusing on a different window. You can see that while 'r' is released immediately, the 'v' release does not show up until the window defocuses:
Library initialized
Creating windowed mode window 1 (640x480)
Main loop starting
00000000 to 1 at 0.481: Window refresh
00000001 to 1 at 2.962: Key 0x0052 Scancode 0x0013 (R) (with no mods) was pressed
00000002 to 1 at 2.988: Character 0x00000072 (r) with modifiers (with no mods) input
00000003 to 1 at 3.008: Character 0x00000072 (r) input
00000004 to 1 at 3.013: Key 0x0056 Scancode 0x002f (V) (with no mods) was pressed
00000005 to 1 at 3.018: Character 0x00000076 (v) with modifiers (with no mods) input
00000006 to 1 at 3.022: Character 0x00000076 (v) input
00000007 to 1 at 3.029: Key 0x0052 Scancode 0x0013 (R) (with no mods) was released
00000008 to 1 at 18.242: Cursor position: 585.000000 436.000000
00000009 to 1 at 18.260: Cursor entered window
0000000a to 1 at 18.269: Cursor position: 428.000000 324.000000
0000000b to 1 at 18.281: Cursor position: 355.000000 275.000000
0000000c to 1 at 18.288: Cursor position: 314.000000 250.000000
0000000d to 1 at 18.296: Cursor position: 269.000000 222.000000
0000000e to 1 at 18.304: Cursor position: 221.000000 193.000000
0000000f to 1 at 18.312: Cursor position: 169.000000 161.000000
00000010 to 1 at 18.320: Cursor position: 118.000000 129.000000
00000011 to 1 at 18.328: Cursor position: 70.000000 100.000000
00000012 to 1 at 18.334: Cursor position: 22.000000 71.000000
00000013 to 1 at 18.342: Cursor left window
00000014 to 1 at 20.964: Cursor position: 267.000000 0.000000
00000015 to 1 at 20.981: Cursor entered window
00000016 to 1 at 20.994: Cursor position: 397.000000 46.000000
00000017 to 1 at 21.008: Cursor position: 446.000000 65.000000
00000018 to 1 at 21.024: Cursor position: 551.000000 101.000000
00000019 to 1 at 21.038: Cursor left window
0000001a to 1 at 23.398: Window defocused
0000001b to 1 at 23.414: Key 0x0056 Scancode 0x0000 (V) (with no mods) was released

Cannot reproduce with 3.1.2 on Windows 7. Are you running Windows in a VM?

Is it possible hardware is a confounding variable here? I know different keyboards have different handling of detecting presses/releases of multiple keys at the same time. See https://www.microsoft.com/appliedsciences/antighostingexplained.mspx.

@shurcooL, thanks, I tried it with another keyboard and I couldn't reproduce, so it must be a hardware issue.


Update ML.NET Azure Functions Guide

Summary
This PR replaces #32973
Fixes #32840
Preview Link

@IEvangelist This one's ready for review when you get a chance. Thanks!


[BUG] Curve Limit Orders fail to execute. No errors or logs

Unclear that Curve Limit Order back-end is functioning.

Curve Limit Order does not appear to execute. Are there any bugs on the back-end to indicate failure?
User attempted to withdraw, but was unable to withdraw. withdraw_type (uint256) is required, but none is specified in the contract.
Are there any bugs with this bot?
Please see Discord discussion here: https://discord.com/channels/987730379182051408/1259668833115836518/1259672789468582000

This is fixed.


Cross compile for the appropriate hosts when building a Darwin toolchain

Addresses rdar://98787335

@swift-ci Please smoke test

@swift-ci Please Build Toolchain

@swift-ci Please Build Apple Silicon

I wonder if we should make cross-compile-hosts dynamic. Users can use the cross-compile boolean flag, and it will automatically map the correct supported architectures.
For example:

On macOS host (x86_64) cross compile (arm64)
On macOS host (arm64) cross compile (x86_64)

Does this fix https://github.com/apple/swift/issues/60348 ?

@keith yes, it should -- thanks for bringing that up

As per feedback above, closing this in favor of adding an option to autodetect the correct hosts to cross compile for
Duplicate of #60664

Duplicate of #60664


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
16e53c8 -> 7241dd8

github.com/gruntwork-io/terratest
require
minor
v0.46.16 -> v0.47.0

Release Notes

gruntwork-io/terratest (github.com/gruntwork-io/terratest)
v0.47.0
Compare Source
Modules affected

packer
aws
helm
azure

Description

Added support for Packer 1.10
Fixed error checking in Azure
Fixed tests and documentation for helm, aws, packer modules

Related links

https://github.com/gruntwork-io/terratest/pull/1395
https://github.com/gruntwork-io/terratest/pull/1419

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 1.9.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Start using semantic versioning

Hey @matryer, first of all thank you, love your work here!
Have you thought about semver tagging the project for enhanced control over versions?

I would like additionally to propose official releases of moq for easier installation. This process can be automated pretty well with https://github.com/goreleaser/goreleaser.

Yeah this is a good point—I’ll do this
On Wed, 2 Sep 2020 at 09:08, Lucas Bremgartner<EMAIL_ADDRESS>wrote:

I would like additionally to propose official releases of moq for easier
installation. This process can be automated pretty well with
https://github.com/goreleaser/goreleaser.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/matryer/moq/issues/137#issuecomment-685430519, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAY2GYOWOONRUHO6OVEJDTSDX4PNANCNFSM4QSNW7OQ
.

OK @Streppel @breml Check that out... we have some releases now https://github.com/matryer/moq/releases

@matryer Thank you very much, this is really appreciated.


[T4A2][W13-B2] Albert Yeoh Ji Bin

Ready for review.

i see that you haven't submitted part 2 of this assignment. as long as you know how to make methods as class level methods, it is fine, no need to submit another PR for that.

@bertojo
Some comments added. Please ack & close the PR after reading comments.


Add support for booting on Intel NUC (tested on Kaby Lake)

@ebail reworked as suggested.

@ebail reworked as suggested.

No problem...


"+" format adds a plus sign to zero

Just like the title says - the "+" formatting adds a plus sign to 0, turning it into "+0" (https://jsfiddle.net/b7g9r739/). I'm using it to display the plus sign for positive values. Is this the correct behaviour?
Here it says:

a plus sign for positive and a minus sign for negative.

On a side note for another question regarding intended behaviour. Some time ago I ran into this (leading zero for si prefix), which I wasn't expecting.

Yes, this is the expected behavior. I’ll make a small clarification in the README.
Regarding your second comment, what behavior did you expect? Your format is ,.2s, which means you’ll get two significant digits with SI-prefix formatting. Therefore 1.01 is rounded to 1.0e0 and displayed as 1.0, and 0.01 is rounded to 1.0e-2 and displayed as 10m. If you think you’ve found a bug, please file a separate issue and specify both the expected and the observed behavior in the description (along with a link to live code to reproduce the observed behavior, as you’ve done).
Also, if you want to ask for help, please use Stack Overflow tag d3.js. If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


RYA-172 Fixed CopyTool unit test from not clearing tmp hadoop directo…

…ry between tests.
Description
Fixed CopyTool unit test so it clears out the hadoop temp directory it uses between each test.  Most of the changes are from save actions.  Search for the importFilesToChildTable method to see actual changes.
Tests
Ran unit tests multiple times
Links
Jira
Checklist

[x] Code Review
[ ] Squash Commits

People To Review
@amihalik

looks good -- about to merge


chore(master): release 7.0.0

:robot: I have created a release beep boop
7.0.0 (2024-06-02)
⚠ BREAKING CHANGES

merge api (#920)

Features

merge api (#920) (a7b68be)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ethersphere/bee-js/releases/tag/v7.0.0 :sunflower:

Release v7.0.0 addresses this.


only works in reading mode, not preview or source. By design??

In sandbox vault, only works in reading mode, not preview or source. By design??
It seems to work if the element is transcluded, though.
https://github.com/user-attachments/assets/42f7928d-a467-4edf-9c28-3dbb52881d50

I see. My hope was to use it while editing, to see what the link is about without having to open it.


Can't scroll when input is in focus

I have set scrollEnabled={true} so the whole form can scroll while you are filling it up. If I set focus on some TextInput scroll works, but if I type something and then I try scrolling while TextInput is in focus scroll doesn't work. This was working in some older versions of the library.

some question

I think I have the same question about this life-saving module, but I'm not sure.
I have a dropdown search bar that presents a scrolling Flatlist of autocompleted results. If I wrap it in the Keyboard-Aware-Scroll-View, I can no longer scroll the results...any ideas?

@johngoren my problem gets solved when I rollback to v0.3.0, maybe it will work for you too.

I also rolled back to v0.3.0. The issue we had was that the input would not scroll to focus unless we had already focused that input previously. So on initial load, when I would focus next input it only scrolled when I got to the part of the window where it would have to scroll to show the field itself.
However if I refocused the top input and tried again, it would scroll to each individual input like normal. Version 3.0 scrolls to input on initial load, it doesn't require dirty-ing the textInput first.
Hope this helps explain the problem better.

Hi!
Back to the office, sorry for the delay. Can you set up a minimum example reproducing the issue???
Thanks!

Tap on some input filed (screen was scrolling, and content offset apply - it's ok)
tap on another input field
tap on empty space
4)result - keyboard doesn't hidden

When i am using a textinput field which has multiLine prop and the whole thing is surrounded by the KeyboardAwareScrollView , the keyboard overlaps the textfield as the text size increases and cannot see what is typed and focus.

Here a code sample:
import React, {Component} from 'react';
import { TextInput, Text } from 'react-native';
import PropTypes from 'prop-types';
import { KeyboardAwareScrollView } from 'react-native-keyboard-aware-scroll-view';

class LinkAccount extends Component {
    constructor(props) {
        super(props);
        this.state = { text: '' };
    }

    render() {
        return (
            <KeyboardAwareScrollView
                keyboardShouldPersistTaps="handled"
                resetScrollToCoords={{ x: 0, y: 0 }}
                keyboardOpeningTime={0}
                scrollEnabled={true}
            >
                <Text style={{ marginBottom: 400 }}>
                    Lorem ipsum dolor sit amet.
                </Text>
                <TextInput
                    style={{height: 40, borderColor: 'gray', borderWidth: 1}}
                    onChangeText={(text) => this.setState({text})}
                    value={this.state.text}
                />

            </KeyboardAwareScrollView>
        );
    }
}

LinkAccount.propTypes = {};
LinkAccount.defaultProps = {};

export default LinkAccount;

The issue I have is: Can't scroll when there is any content in TextInput (when it's dirty). When its empty, no problem. This was not the case in previous versions of this lib.
Here is a video of issue https://screencast.com/t/Acrsuby6ZewG

I have the same problem. I Wrapped my contents in KeyboardAwareScrollView.
When I 'm going to type in textinput, the whole thing goes fine (scroll...) but when I typed something , the contents height?? suddenly changed and the scroll doesnot work.

@zarcode I can reproduce this also.
Btw. From @zarcode video, it looks like "KeyboardAwareScrollView" has two different ScrollViews inside depending on the keyboard status. To spot the difference keep your attention on the scroll indicator. When the keyboard is disabled there is no scroll indicator on ScrollView, but when the keyboard is enabled – ScrollView changes to a new one with the scroll indicator.
The problem occurs, when typing something inside textInput changes the second ScrollView to the first one without scroll indicator.
Hopefully, it will help to fix this issue.

I will check this out today, thanks for reporting!

@malcevicius this module doesn't have two ScrollViews.
I will check this issue today.

From what I can gather, KeyboardAwareHOC resets the state var keyboardSpace during componentWillReceiveProps, which gets fired due to potential changes from text input events in the text input fields (and potentially other events), and it is keyboardSpace which controls the vertical offset due to a visible keyboard.
I've managed to have a workaround by removing the componentWillReceiveProps code from KeyboardAwareHOC. I'm not suggesting that as the long term fix, as I'm not sure what the 'keyboard spacer reset' behaviour was meant to do...

Hmm, wondering if anyone has had any breakthroughs on a long-term fix for this. Is the problem all down to that keyboardSpace reset?

I still have this issue. Can't scroll on Android when the keyboard is open. Works fine on iOS tho.


Python extensions do not work after updating to the latest version of VS Code Insiders

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.75.0-insider (system setup) (4acf2d9b46b75748ae687cf3b2952a0799679873)
OS Version: Windows 11 Pro Insider Preview (Dev Channel) Build 25267.1000

Steps to Reproduce:

Install the latest version of VS Code Insiders
Install Python extension (or official extension pack) and switch to Pre-Release
Extension will try activating automatically, and stuck at Python extension loading...
Open Developer Tool, it will show Activating extension 'ms-python.python' failed: this.a.sendEventData is not a function. and all Python-related extension can not activate correctly.

Other attempts:

Latest version of VS Code Insiders + Python extension ( latest Pre-Release): Failed to load.
Latest version of VS Code Insiders + Python extension ( latest Release): Failed to load.
Latest version of VS Code + Python extension ( latest Release): OK.
Latest version of VS Code + Python extension ( latest Pre-Release): OK.

Screenshot:

Can confirm this occurs on my machine. Parallel to issue microsoft/vscode-python#20456 on the Python extension's repo.

I am having the same issue, both in local and remote sessions.
OS Version: Windows 11 Enterprise Build 22621

Can confirm this occurs on my machine also.

I'm still seeing this
Version: 1.75.0-insider (system setup)
Commit: 727d2d32aea02baf7d13fa38787e8ed64dd858d0
Date: 2023-01-06T05:21:40.528Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes


After setting media parameters to senders and flows how to activate the sender?

Hello,
I hope this message finds you well. My name is Gururaj, and I am a new user of the Sony NMOS-CPP library.
I am currently working on a setup where main application which is generator, providing media attributes to sony/nmos-cpp library. In the nmos library node_implementation.cpp, I'm extracting Media Attribute values (such as Image Height, Width, etc.) from the main generator board. and successfully setting these attribute values into NMOS resources, specifically Senders and Flows, and subsequently calling model.notify(). After performing this operation, I can observe the updated attributes under the endpoints:
/x-nmos/node/v1.3/sources/
/x-nmos/node/v1.3/flows/
However, I have encountered an issue where these updated values do not seem to be reflected in the manifest files endpoints:
/x-nmos/node/v1.3/senders/
/x-nmos/connection/v1.1/single/senders/{senderID}/transportfile/
I initially expected that invoking model.notify() would automatically update the values in the library. Unfortunately, it appears that this is not happening in my implementation.
I would greatly appreciate it if someone could provide insights into what might be missing or incorrect in my implementation, preventing the update of these attributes in the SDP object.
Thank you for your assistance.
Best regards,
Gururaj

Have a look at make_node_implementation_transportfile_setter which shows how Senders' SDP data can be updated based on Flow and Sender resources. However, this function is no longer called directly by the example node implementation code. Instead it just issues an activation to have nmos-cpp update it automatically. See: https://github.com/sony/nmos-cpp/blob/25240c43e4953c9202dfd372759daed5cbdf4c72/Development/nmos-cpp-node/node_implementation.cpp#L478-L488


Flink 3750 fixed

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[X] General

The pull request references the related JIRA issue
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message

[X] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[X] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

New cleaned PR aligned with the current master

Thanks for the PR @fpompermaier.
I think the new format is a bit too much tailored towards certain query templates (BETWEEN predicate on integer column). Also modifying queries that users provide, is a bit risky, IMO.
To make it more general I would propose to:

Accept query templates with markers, similar to parameter makers in prepared statements: SELECT address FROM people WHERE name = ? AND birthday BETWEEN ? AND ?.
Let users explicitly provide bounds. Users should know their data best and can provide bounds which take skewed distributions into account. Parameter values can be provided as Object[], one array for each parameter. We can provide some utility methods to help users generating uniformly distributed parameter values.
Let InputSplit not provide two bound values but the index for the parameter value array. So each instance can build the query by substituting the parameters by values.

What do you think?

Thanks for the update @fpompermaier. I added a few comments and suggestions.

Thanks for the update @fpompermaier. Overall the PR looks good. I added a few comments and suggestions. Thanks, Fabian

Hi @fpompermaier, I added one more comment. I'm not sure if you noticed the other two comments I made a fews day back to JDBCFullTest and JdbcInputFormat.nextRecord() after your last update of the PR. Thanks, Fabian

Thanks for the update @fpompermaier. Just had a few minor comments. After that this PR should be good to merge.

Thanks for the update. Looks good. Will merge it later :-)

Merging


MSSDK-1301 bank payment methods

create and embed two drop-ins: one for standard payment methods and one for bank once
fix 'drop in recreate' logic
adjust styling

add copy for bank payment methods

I think with these changes we should change one thing in ThankYouPage.js
{totalAmount &&
           currencySymbol &&
           t(
             'thank-you-page.sub-text-with-price',
             `You have been charged {{currencySymbol}}{{totalAmount}}.`,
             {
               currencySymbol,
               totalAmount
             }
           )}

Code above is working when both totalAmount and currencySymbol are undefined, but now when totalAmount can be 0 we will return 0 😛

Maybe we could change it to:
  const chargedMessage = totalAmount
    ? currencySymbol &&
      t(
        'thank-you-page.sub-text-with-price',
        `You have been charged {{currencySymbol}}{{totalAmount}}.`,
        {
          currencySymbol,
          totalAmount
        }
      )
    : null;


Fix login without sign up.

Description
login was sucessfull even before signup.
Related Issues
username was not appearing after sign in .
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #700

Type of PR

[x ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist

[ x] I have gone through the contributing guide
[ x] I have updated my branch and synced it with project main branch before making this PR
[ x] I have performed a self-review of my code
[ x] I have tested the changes thoroughly before submitting this pull request.
[ x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]
firebase rules should be changed and google auth provider should be enabled.

pls review this request

@Subhams-GIT Please resolve conflicts


spec: clarify legality of non-ASCII graphic characters in import paths

About import paths, the spec (https://golang.org/ref/spec#Import_declarations) says:

Implementation restriction: A compiler may restrict ImportPaths to non-empty strings using only characters belonging to Unicode's L, M, N, P, and S general categories (the Graphic characters without spaces) and may also exclude the characters !"#$%&'()*,:;<=>?[]^`{|} and the Unicode replacement character U+FFFD.

but the Go 1.16 release notes (https://golang.org/doc/go1.16#go-command) say

In module mode, the go command now disallows import paths that include non-ASCII characters or path elements with a leading dot character (.). Module paths with these characters were already disallowed (see Module paths and versions), so this change affects only paths within module subdirectories.

The latter wording appears to be a stronger restriction than allowed by the spec. Does the spec need to be changed?

The spec deals with the language. Modules isn't part of the language, it's part of the toolchain.
So basically this line The interpretation of the ImportPath is implementation-dependent applies

CC @jayconrod, @bcmills.

Hey Alan, hope all is well. We miss you over here.
I think it would be reasonable to note within the spec that clarify the build tool (cmd/go, Bazel, Blaze) is responsible for interpreting import paths and may impose additional restrictions. The restrictions for module paths are documented in Module paths and versions.
Incidentally, in 1.16.2, we rolled back some of the changes in 1.16 because they were affecting more users than we anticipated. Import paths are now validated separately from module paths, and import paths may begin with a leading dot and may contain + characters.
cc @bcmills @matloob

The spec is wrong. At the least, an implementation needs to be able to insist on Unicode normalization that would exclude some of those code points. But further restrictions should be allowed as well, along with name-specific restrictions (we don't allow a package named "com1" for example).

@rsc It's a pity that Go will do normalization since package ʕʘϖʘʔ is fine with spec and go tool now.
https://pkg.go.dev/github.com/mengzhuo/moe
import (
        "fmt"
        ʕʘϖʘʔ "github.com/mengzhuo/moe"
)

func main() {
        fmt.Println(ʕʘϖʘʔ.Ꮷಠ_ಠ("ʕʘϖʘʔ powered"))
}

This is in the Go1.18 milestone.  Is it likely to happen for 1.18?  Thanks.
@rsc You assigned this to yourself back in April.
@griesemer see https://github.com/golang/go/issues/44970#issuecomment-814343089

@griesemer  This is in the 1.18 milestone; time to move to 1.19?  Thanks.

Not urgent. Moving to 1.19.

As a case study for this, #53954 was an issue relating to macOS's handling of Unicode file names.
The test involved the character Ä, which can be represented either as the composed character c3 84 or the decomposed character 41 cc 88. In Git, we had file paths and source files using the composed character (c3 84).
However, on macOS, the file names are rewritten to use the decomposed characters (41 cc 88). This caused problems with the test, because test/run.go used the file name to construct cmd/compile's -p flag; so it invoked cmd/compile with -p=test/\x41\xcc\x88foo.
Then when compiling the second package, cmd/compile saw import "./\xc3\x84foo". The macOS file system layer silently translated this into opening ./\x41\xcc\x88foo.a, but that tripped a recently added assertion that ensures the -p flag used to compile a package (i.e., "test/\x41\xcc\x88foo") matches the import path string used to import it (i.e., "test/\xc3\x84foo").


Sending bidirectional Quiver invites results in odd state

Sending an invite to a friend and getting the invite from the friend (and clicking both) results in an odd state.

This appears to be fixed in the latest version of Quiver, we can re-open if we see the issue again


getting some information

Hi,
I really love the basic idea of helm templating. So I decided to use it for a project. But some things are still really hard to get.
For instance, after you parse my templates, where do you put the parsed results? It's so important if you have a parser for anything that you can view the results to see what the parser did maybe different than one expected.
Where can I see what http requests helm is sending (url+body)? I'm not having a default kubernetes installation here and need to make sure that others can also reach the tiller service. Also maybe seeing some http responses may help debug things.
What is the difference between the argument --namespace and  the environment variable $TILLER_NAMESPACE? For some reason I can only use helm when I do the second version. And I had nothing tell me about the problem.  Would be really nice if there would be a smarter way to debug.
And why does a helm delete --purge <name> not remove everything? For instance in my environment there is always a serviceaccount that still exists after.
Thanks!

@erikbgithub Thank you for using Helm. Hopefully we can get your issues taken care of. To see parsed results of templating you may add the --debug flag to see the final output. Also adding the --dry-run flag together with --debug will let you see the output without actually installing.
Helm uses gRPC not REST so you will not be able to see requests to Tiller as you might be used to in conventional REST applications.
--namespace directs Helm to install your chart into the specified namespace.
TILLER_NAMESPACE refers to the namespace that the backend componnent Tiller is installed into ("kube-system" by default)
Hope that helps. Please let us know if you have any more questions.

Hi jascott1, thanks for responding.
I'm not sure the namespace question is complete. Because I do both in a separate session after helm init already succeded another day.
Example. Let's assume you start the day with a new bash session in which no helm environment is set. Tiller is already deployed. Now you do just a helm ls --namespace=foobar and it fails. Now you set export TILLER_NAMESPACE=foobar  and now helm ls without a namespace flag succeeds. Would you agree that situation is not explained by your answer?

nope, I can understand @jascott1's instructions quite clearly (but then again, I'm a helm dev).

helm list with --namespace shows charts that were released in that namespace.

TILLER_NAMESPACE=foobar helm list tells helm to communicate with the tiller instance installed in the foobar namespace to list all releases installed across all namespaces.

I can understand the confusion, though. TILLER_NAMESPACE is documented here:

Helm will look for Tiller in the kube-system namespace unless --tiller-namespace or $TILLER_NAMESPACE is set.

Does that help clear things up? If not, I highly suggest taking a look through the Using Helm section of the docs to get a clearer picture.

Ah now I understand. Thanks!


hotel-api: hotel groups python example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingAds/hotel-service/code-example-hotel-groups.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.


Add default avatar for chatty kathy

We want to default this to the bullhorn icon, so that chat topics show up looking like chat topics to staff members

Closed with #47


Roll back LoggerAPI dependency to 1.7

...to avoid SPM resolution problems with other projects that contain an existing dependency on 1.7

Description

Motivation and Context

SPM fails to resolve in reasonable time with any other project (such as Kitura) that still has a dependency on LoggerAPI 1.7.
How Has This Been Tested?

Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

Merging #243 into master will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #243      +/-   ##
==========================================
- Coverage   74.85%   74.61%   -0.24%
==========================================
  Files          31       31
  Lines        4171     4171
==========================================
- Hits         3122     3112      -10
- Misses       1049     1059      +10

Flag
Coverage Δ

#CHTTPParser
57.46% <ø> (ø)
:arrow_up:

#KituraNet
74.61% <ø> (-0.24%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/KituraNet/HTTP/HTTPServer.swift
78.21% <0%> (-2.8%)
:arrow_down:

Sources/KituraNet/IncomingSocketHandler.swift
78.53% <0%> (-2.29%)
:arrow_down:

Continue to review full report at Codecov.

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


ClassNotFound Exception While Building

I added the extension to my project and I got this exception while building.
Failed to build quarkus application: java.lang.RuntimeException: Failed to load steps from class
   io.quarkiverse.loggingmanager.deployment.LoggingManagerProcessor:
io/quarkus/vertx/http/deployment/NonApplicationRootPathBuildItem:
    io.quarkus.vertx.http.deployment.NonApplicationRootPathBuildItem

My Quarkus platform and Quarkus plugin versions are 1.10.5.Final

Please use 1.11.0.Final (Non application came in that version).

Please use 1.11.0.Final (Non application came in that version).

Or use the one of the previous versions of this extension, which are going to be compatible with you version. Formerly this extension was called logging-ui, so you will find the available versions here:
https://mvnrepository.com/artifact/io.quarkiverse.loggingui/quarkus-logging-ui

Or use the one of the previous versions of this extension, which are going to be compatible with you version. Formerly this extension was called logging-ui, so you will find the available versions here:
https://mvnrepository.com/artifact/io.quarkiverse.loggingui/quarkus-logging-ui