Replication Issue [PLACEHOLDER]

Replication Issue [PLACEHOLDER]

If it's fair to presume that each EDV resource (e.g. JWE) has one resource owner (RO, the entity that gets the capability issued by the EDV), then:
Does EDV replication presume that all replicas are owned by the same RO or, that replicas may be accessed (and become owned by other end-users?

It could be one of a number of different scenarios: Here's one I proposed earlier today: https://hyperonomy.com/2021/03/04/trusted-content-storage-tcs-stack-secure-resource-sharing-srs-using-replicated-stubs-solution-concept/
BTW, I think you can think of someone being the resource owner but it might be better to think of them as Resource Controllers.

If one doesn't have an architecture reference model (ARM) in mind when talking or thinking about replication, it' hard/too abstract to have a fruitful discussion, here's the one I use: Generic Replication Pipeline:

Figure 20. Generic Replication Pipeline (Syntergy Replicator for SharePoint)
Outbound Processing Service
Outbound Processing Service is responsible for capturing and recording Replication Events that occur in the Source Repository. Outbound Processing Service is controlled by Replication Maps which determine what Events need to be captured, packaged, and transferred to the Target Repository. Groups of Replication Events are packaged into two types of messages or objects: Queued Items and Replication Packages.
Queued Item
A Queued Item is a unit of work to be transferred to a Target Repository for remote execution. The Replicator Web Service on a Target Repository is called to push a Queued Item from the Source Repository to a Target Repository.
Replication Package
A Replication Package is a collection of one or more Replication Events plus data about the changed information that is packaged in a format specific to the Replication Transport being used. When an Event is being processed, Outbound Processing Service calls the Source repository object model to extract the changed information from the Source Repository.
Package Transfer Service
The Package Transfer Service activity is responsible for the transfer of Queued Items and Replication Packages (Packages) from the Source Repository to the Target Repository. Package Transfer is the process that sits between Outbound Processing Service (on the Source Repository) and Inbound Processing Service (on the Target Repositories).
Inbound Processing Service
Inbound Processing Service is responsible for processing the Queued Items and Packages received and accepted by the Target Repository and applying them to the repository. The Queued Items and Packages are applied to the Target Repository content base by calling the Target Repository object model.

@mwherman2000 Naming things is hard and that applies to roles in various use cases. I use the names as defined in GNAP Section 1.2.
I PROPOSE that we incorporate these roles in our discussion. I know they are likely to change and SDS may find reason to diverge but for now, I will raise an issue that we start with the current GNAP roles.

Here's a few dimensions to consider in terms of an overall replication service ...based on the above Replication Pipeline...

Replication network scenarios
a.	Offline replication
b.	Air gap replication
c.	Slow, high-latency network connections (e.g. Satellite communications to Antarctica): Replicated stubs
d.	Bi-directional vs. Uni-directional
Outbound Replication Processing
a.	Real-time replication
b.	Replication filtering
c.	Selective Replication: Outbound filtering and Inbound package acceptance
i.	Content types: content, metadata, schema, container hierarchies, security permissions, security principles, roles, event handlers/workflow instances, configuration settings
Package Transfer
a.	Dynamic, multi-hop routing
b.	Cross-firewall replication
c.	Peer-to-Peer (P2P) topologies: Direct and Mediated
d.	Dynamic, multi-hop routing
e.	Cross-firewall replication
f.	Peer-to-Peer (P2P) topologies: Direct and Mediated
Replication Inbound Processing
a.	Multi-master replication
b.	Conflict management
i.	Conflict detection
ii.	Conflict resolution
Security
a.	Data at rest; data in motion (replication packages)
b.	Mapping security principles and roles
c.	Activity anonymity
Performance Management
a.	Bandwidth throttling
b.	CPU throttling
c.	Disk management
d.	Scalability
i.	Support for large objects (terabyte blob)
ii.	Support for large containers (petabytes)
iii.	High-frequency content changes
Failover/service recovery
a.	Disaster recovery: feasibility, fidelity, time/bandwidth/processing required

@agropper I think the Sovrin Glossary (and ToIP's clone) are in more common usage:  https://docs.google.com/document/d/1gfIz5TT0cNp2kxGMLFXr19x1uoZsruUe_0glHst2fZ8/edit#heading=h.3wjdvvstzjf0
e.g.
Controller
An Identity Owner that is responsible for control of another Entity—specifically the Private Keys needed to take actions on behalf of that Entity. For example, a Thing Controller has a Controller relationship with a Thing. It is one of three types of identity control relationships described in Appendix C.


After apply ExchangeMitigations.ps1 ECP access is lost

In Exchange Server 2016 CU11 and after applying the script ExchangeMitigations.ps1 ECP access is lost, can it be?
If I try to access the administration console from the local server -> "https: //localhost/ecp" I have an error "HTTP error 503 - The service is unavailable"
If I rollback the script I have the management available again.
I'm something wrong??

I attached the output of powershell script and the nmap scan before /after installed-

@eduzav this is by design and part of the mitigation. There is a switch for each piece.
ApplyAll, Apply OAB, Apply ECP, Apply UM.
Rollback all, rollback OAB, rollback, rollback ecp, rollback UM.
For more information visit: https://msrc-blog.microsoft.com/2021/03/05/microsoft-exchange-server-vulnerabilities-mitigations-march-2021/

Perfect!!!, other question, do you know if having the ecp disabled also serves as mitigation?
Ej -> Set-EcpVirtualDirectory -Identity "ecp (Default Web Site)" -AdminEnabled $false


make test command is giving error

When i run make test after running make build i get this error :
make: *** No rule to make target 'test'.  Stop.
When i checked in makefile it doesn't have test command and also when i run forge test i get this message
No tests found in project! Forge looks for functions that starts with test.
OS : Ubuntu 22
Branch : Master

Thanks for your response.


i18n(ko-KR): update error-reference.mdx and add new session error reference files.

Description (required)

update error-reference.mdx
add session-storage-init-error.mdx
add ``session-storage-save-error.mdx`
#10447

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

ko/reference/configuration-reference.mdx
Localization changed, will be marked as complete.

ko/reference/error-reference.mdx
Localization changed, will be marked as complete.

ko/reference/errors/session-storage-init-error.mdx
Localization added, will be marked as complete.

ko/reference/errors/session-storage-save-error.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Hey this is just test the app cod don't open the file

[go to GitHub app](follow yosif)
#Kill the iPhone or the Samsung
> And say: ha ha ha

The cod is end~~~~
😂😂😂

...


Updated urls for the readme

I noticed that the Author URL was broken, the rest are minor.

Thanks a lot and sorry for the long delay! I didn't got the notification for this PR.


Please annotate a glue class with some context configuration.

Hello,
When i launch my project in my local repo, everything work fine but on the team repo i have this error:
GRAVE: Exception while executing pickle
java.util.concurrent.ExecutionException: io.cucumber.core.backend.CucumberBackendException: Please annotate a glue class with some context configuration.

Hello, do you have an example branch with the problem ? Do you launch your test with a custom maven goal in your CI ?


A11y_Visual Studio Code Client_Search_ExtensionScoll to Top_Focus Visible: Keyboard focus is not visible on "Scroll to top" button when we are navigating using the tab key

[(Check out Accessibility Insights!)]- Identify accessibility bugs before check-in and make bug fixing faster and easier.
GitHubTags:
#A11yTCS; #A11ySev2; #DesktopApp; #Win32; #Visual Studio Code Client; #BM-VisualStudioCodeClient-Win32-Jan2024; #Keyboard; #WCAG2.4.7; #AIlimited; #FocusVisible;
Environment and OS details:
Visual Studio Code insider version 1.86.0
Microsoft Windows 11 Enterprise 22H2 Build 22621.3007
Repro Steps:

Open Visual Studio Code Insiders editor
Using "Ctrl+Shift+X" key navigate to the extension side view bar option.
Using Tab key navigate to the "More" option and select different options( "Show installed Extension, Show outdated extension, Show popular Extension,Etc")
Click on any installed extension
Tab till scroll to top button.
Verify whether the Keyboard focus is not visible on "Scroll to top" button or not.

Actual Result:
Keyboard focus is not visible on "Scroll to top" button when we are navigating using the tab key.
Expected Result:
Keyboard focus should be visible on "Scroll to top" button when we are navigating using the tab key.
User Impact:
Keyboard users will face difficulties is the focus is not visible on scroll to top button.
Attachment:

that button does not receive tab focus.

one can scroll using arrow keys, so closing this as it is keyboard navigable that way.


setCountryForPhoneCode does not set the countryCode to the desired countryCode

I am performing the following steps:

I have a button which opens a dialog and allows me to select the language or country.
On selection of a particular country, I set the country code in a textView on the calling activity. (It recreate the activity as changing the locale settings of app.)
onTextChangeListener, I set the country code in Country Code picker, but it always takes the default one instead of setting the desired one.
Is there something wrong here?

"setDefaultCountryUsingNameCode() or setDefaultCountryUsingPhoneCode() will not set default country as selected country in CCP view. To set default country as selected country in CCP view, call resetToDefaultCountry() method."


add USE_PRE_EXISTING_ACCOUNT_ROLES envvar

Patch to: https://github.com/communitiesuk/funding-service-design-authenticator/pull/155
Before this PR, when a user is running authenticator in a development environment, it would bypass the roles provided by AzureAD in favour of those pre-existing in the account store database.
This has the following issues:

If the account had no roles this route woulf fail
If the account had the wrong roles in the account store this would require manually changing the roles (instead of just inheriting the correct ones from AD)
The account store and AD for an account were not synchronized as they would be in dev and test
We could not replicate the authentication flow in development.

This patch introduces a new environment variable (USE_PRE_EXISTING_ACCOUNT_ROLES) that allows users the option to bypass role assignment via Azure AD in favour of the pre-existing roles in the account store, without forcing this logic path for anyone using the development environment.

@Adam-W1  - is this still needed?

@Adam-W1 - is this still needed?

i dont think it is @tferns so il close


Update index.rst

The directive for constructors should read as .. java:constructor:: constructor-signature. The current directive is truncated (it reads exactly as the role).

Thanks for the fix!


*: release v0.18.1

Blocked by https://github.com/operator-framework/operator-registry/pull/349
/hold

/hold cancel


glyphs file input VF 'wght' axis scale not as expected

The font available at fonts.googleapis.com/earlyaccess/nunitovfbeta.css was made by @m4rc1e using fontmake (via https://gist.github.com/m4rc1e/8b9a6e5678c6628eae7e682d4de63220#file-gf_vf-py-L120-L142) the scale of the 'wght' axis is not 100 to 900 as expected, but 50 to 160.

https://jsbin.com/lewikin/1/edit?html,output

I guess that is because those are the arbitrary interpolation values used in GlyphsApp which correspond to stem widths in UPM values to make the type designer's mind-arithmetic easier, and GlyphsApp uses for its own interpolation, and exports directly to the font.

This is a duplicate of #167.

Thanks @moyogo :)


Correct HandleCodeAction behavior for CodeAction

Both the CodeAction and Command interfaces contain the command member
but these should be handled differently for them. Corrected the
HandleCodeAction method to correctly handle these.
Tested with intelephense PHP language server and clangd.

How did you test the 'Command' interface with clangd?  Can you provide sample steps for that?  I can add a test for it.

#95


Use IO.popen instead of PTY (fix #42)

I think issue #42 comes from the control characters that Shenzhen emits to change the color in the terminal, because right after them a \n\r is inserted every ~5 characters. Using IO.popen instead of PTY fixes this issue.

Thanks :+1:


python312Packages.ndspy: 4.1.0 -> 4.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.ndspy is: Python library for many Nintendo DS file formats
meta.homepage for python312Packages.ndspy is: https://github.com/RoadrunnerWMC/ndspy
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.2.0 with grep in /nix/store/4jhi9w8x79hyv1vx76h5cciy24i29h8h-python3.12-ndspy-4.2.0
found 4.2.0 in filename of file in /nix/store/4jhi9w8x79hyv1vx76h5cciy24i29h8h-python3.12-ndspy-4.2.0

Rebuild report (if merged into master) (click to expand)

10 total rebuild path(s)

10 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.ndspy
python311Packages.skytemple-dtef
python311Packages.skytemple-files
python311Packages.skytemple-ssb-debugger
python312Packages.ndspy
python312Packages.skytemple-dtef
python312Packages.skytemple-files
python312Packages.skytemple-ssb-debugger
skytemple

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4jhi9w8x79hyv1vx76h5cciy24i29h8h-python3.12-ndspy-4.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.ndspy https://github.com/r-ryantm/nixpkgs/archive/7478ee57973cfa7f58612cd1f57d5e2f11a48921.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7478ee57973cfa7f58612cd1f57d5e2f11a48921#python312Packages.ndspy

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4jhi9w8x79hyv1vx76h5cciy24i29h8h-python3.12-ndspy-4.2.0
ls -la /nix/store/4jhi9w8x79hyv1vx76h5cciy24i29h8h-python3.12-ndspy-4.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

18 packages built:

python311Packages.ndspy
python311Packages.ndspy.dist
python311Packages.skytemple-dtef
python311Packages.skytemple-dtef.dist
python311Packages.skytemple-files
python311Packages.skytemple-files.dist
python311Packages.skytemple-ssb-debugger
python311Packages.skytemple-ssb-debugger.dist
python312Packages.ndspy
python312Packages.ndspy.dist
python312Packages.skytemple-dtef
python312Packages.skytemple-dtef.dist
python312Packages.skytemple-files
python312Packages.skytemple-files.dist
python312Packages.skytemple-ssb-debugger
python312Packages.skytemple-ssb-debugger.dist
skytemple
skytemple.dist

Maintainer pings
cc @marius851000 for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Tested SkyTemple. Some issue where found, but they also happen on master without this patch, and so are not caused by this.
LGTM


RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)

Traceback (most recent call last):
File "sru_expriment.py", line 181, in
train_loss = train(train_loader, criterion, optimizer, model, bsz_train, device)
File "sru_expriment.py", line 134, in train
output=model(inputs)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "sru_expriment.py", line 62, in forward
output,c= self.rnns(input)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/sru/sru_functional.py", line 613, in forward
h, c = rnn(prevx, c0[i], mask_pad=mask_pad)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in call
result = self.forward(*input, **kwargs)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/sru/sru_functional.py", line 376, in forward
U = self.compute_U(input)
File "/home/nzh/anaconda3/lib/python3.7/site-packages/sru/sru_functional.py", line 432, in compute_U
U = x.mm(self.weight)
RuntimeError: CUDA error: CUBLAS_STATUS_EXECUTION_FAILED when calling cublasSgemm( handle, opa, opb, m, n, k, &alpha, a, lda, b, ldb, &beta, c, ldc)
environment：python3.7+pytorch1.2

I faced with this problem multiple times and found out the version mismatch between CUDA and pytorch was the problem.
Try to update your torch and CUDA version.


Local video doesn't be rendered. It doesn't be sent to remote machine on IOS 13

I run the sample. It doesn’t send video to remote, but voice is ok.  How could I fix it?
Thanks

Any solution for this?

It's work perfectly to use GoogleWebRTC framework (https://webrtc.org/native-code/ios/).  You have to follow above steps to run the sample.

@linhongyi The problem is with iOS 13. Local video is showing only on landscape mode if I turn device to portrait that it is not working. I am testing on iPhone 7 plus device. On other devices working fine. Any help?

@NirajCapermint The video does not display because of libjingle_peerconnection. The framework I couldn't fix it.
I replaced libjingle_peerconnection by GoogleWebRTC to achieve our goal.

@linhongyi Replacing libjingle_peerconnection by GoogleWebRTC does not particularly solve the issue as some header files are different.

Any solution??

Hello @linhongyi could you please share how you migrated to GoogleWebRTC from libjingle_peerconnection? Thanks in advance

Hello @linhongyi could you please share how you migrated to GoogleWebRTC from libjingle_peerconnection? Thanks in advance

Try: https://github.com/sanaD13/SDWebRTC

Any solution??

https://github.com/sanaD13/SDWebRTC

Any solution??

https://github.com/sanaD13/SDWebRTC

Hello @sanaD13 your source code in github is a initial project. It's missing pod & webRTC source code. Please check again. Thank you.

I found a good sample https://github.com/crossle/AppRTCMobile

Hi. Check this again. Sorry about that
On Thu, Jan 2, 2020 at 5:29 AM maunb<EMAIL_ADDRESS>wrote:

I found a good sample https://github.com/crossle/AppRTCMobile
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ISBX/apprtc-ios/issues/153?email_source=notifications&email_token=AFHSZEWQPA645SY6765ZSZLQ3W6YNA5CNFSM4I2IRTVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH6BO3Y#issuecomment-570169199,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFHSZETM4DRT45VNSIFV3WLQ3W6YNANCNFSM4I2IRTVA
.

--
Regards,
Sana Desai
Software Developer (iOS Developer)

Hello All,
I managed to solve it by compiling the WebRTC code as someone previously recommended.
Follow the guide in https://webrtc.org/native-code/ios/ to download all the sources and compile the Xcode project. This project will generate the latest WebRTC framework and corresponding files along with sample application. Use these in your project and make necessary changes to fix errors/warnings to have local video be rendered properly on iOS 13.

Hello @linhongyi could you please share how you migrated to GoogleWebRTC from libjingle_peerconnection? Thanks

Hello @linhongyi could you please share how you migrated to GoogleWebRTC from libjingle_peerconnection? Thanks

Hey,
In this repo I used GoogleWebRTC framework https://github.com/sanaD13/SDWebRTC
Hope it helps.

Hello @linhongyi could you please share how you migrated to GoogleWebRTC from libjingle_peerconnection? Thanks

Hey,
In this repo I used GoogleWebRTC framework https://github.com/sanaD13/SDWebRTC
Hope it helps.

Your REPO is giving error.

@sunilju I migrated to GoogleWebRTC from libjingle_peerconnection base on this repo: https://github.com/crossle/AppRTCMobile/tree/master/webrtc

In popfile, remove pod 'libjingle_peerconnection' by pod 'GoogleWebRTC'
You need to update render local and remote video base on repo sample

@sunilju I migrated to GoogleWebRTC from libjingle_peerconnection base on this repo: https://github.com/crossle/AppRTCMobile/tree/master/webrtc

�In popfile, remove pod 'libjingle_peerconnection' by pod 'GoogleWebRTC'
You need to update render local and remote video base on repo sample

Can we solve issue by using  'libjingle_peerconnection' only?

Can we solve issue by using 'libjingle_peerconnection' only?
I mean we remove libjingle_peerconnection. We only use GoogleWebRTC.

Did you find any solution?


Pin git2 dependency to 0.14.2

In order to maintain a MSRV of 1.57 the git2 dependency was downgraded from 0.14.4 to 0.13.25 in [1].
This is problematic since versions prior to 0.14 are known to crash on systems with libgit2 >=
1.4.0. Luckily git2 only raised its requirements in version 0.14.3. Thus, pin to 0.14.2 instead
of going all the way back to 0.13.x.
[1] https://github.com/termoshtt/ocipkg/pull/66

works correctly! Thanks :)


autogrowinput

Hi.
Its it possible to make the autogrow function pluggable?
I'm using a different autogrow library http://jacklmoore.com/autosize/ which seems to cope well with older versions of IE
thanks very much!

Just had a look in the sources. Should be possible. Just place the autogrow function inside the options and use it in line 144. That should be all.
Unfortunatly I don't have the time for implementing and testing at the moment.


Question about the cookie

This is actually not an issue but a question instead.
May I ask how/when did the cookie got saved into browser? I was looking for a line such as cookies.set() in the client side but could not find it and the cookie was set correctly indeed somehow.

@cty945 so first of all in cookie-based authentication the cookie should be set by the server-side not by the client-side, since it's more secure and you can set a HTTP-only cookie that is not available for the client-side to edit. So the server sets the cookie on the response, and if you send cookies on HTTP responses the client automatically stores them.
Secondly, regarding your question, if you check the code you can find a comment in routes/api.js
// When using passport with callback, we have to manually call req.login to set the Cookie
    req.login(user, async () => {

So what's happening is that passport has a built-in cookie.set method that's being called on req.login

Thanks for the response! Yes I understand that the cookie has been set by the server. But the problem I encountered is actually on the client side. "the client automatically stores them" is not happening on my end when I try to reproduce it. Yes I did checked the response in Network tab, the cookies are there, but the client is not automatically storing them for some reason so that when the code checks whether the user has logged in it returned false.

Hello again.
Are you referring that it's not working in the context of this project, or in general?
In regards to this project, I just checked and ran a local setup called "Bla" node ./setup.js Bla and tested the login in the browser and the cookie is stored fine. See here a screenshot: https://i.imgur.com/Se2PwiS.png
Things for you to check

make sure you ran setup.js and have both the server and the client up and running (server must be connected to a mongodb instance)
login via the default credentials from _insertDefaultUsers or with the ones you set, if you changed them
make sure you're not using incognito, although it shouldn't have anything to do with it. Nor should adblockers, as I'm using adBlockers and it's working fine

If it's related to some other project I'm not sure I can help you much


Classes 2.0

https://github.com/neon-bindings/neon/issues/19

So the idea in general here is to have a coercion protocol (via traits) and to allow method definitions in classes to have type annotations that use the coercion protocol to abstract away all the conversion boilerplate. Moreover we should make it possible to define functions and modules declaratively with the macro syntax. So at that point we should have a nice "neon mullet" (JS business up front, Rust party in the back). Quick minimal example:
module! {
    export function foo(x: u32) -> u32 {
        Ok(x + 1)
    }
}

I'd probably use the word pub instead of export since we're writing Rust inside module!.

I'd probably use the word pub

I disagree: the idea is that the outside syntax is JavaScript, and the type syntax and function body syntax are Rust, because it's presenting a JS interface to the outside world while using a Rust implementation (including automatic conversions to Rust types) internally.

Some field experience with classes 1.0 (from https://github.com/matklad/fall/). Although the tone will sound slightly negative, I mightily admire the work done, which allows to bind Rust and nodejs at all! :) ❣
Using macro DSL is quite hard: it is opaque, you have to do contortions to impl classes for generic types with < in their names, and the only way to abstract over this macro is with another macro.
Here's how I try to reuse some functionality to declare similar classes: https://github.com/matklad/fall/blob/3d099c94cd3b25ca93e32d39851aaeefab3799ae/code/generic_backend/src/lib.rs#L52-L87. This felt bad to write.
What's more, this DSL forces huge swaths of Rust code to be inside of macro invocation, which kills a lot of IDE features (of course the IDEs are not mature yet, but it always would be true that the code inside the macro will be harder for IDEs to understand). It also effectively invents a (ad-hoc, informally-specified, bug-ridden, slow implementation) of new language withing Rust itself. the method "keyword" definitely feels alien.
There are also problems on the client side. I write my client code in TypeScript, and, naturally, I don't have any typechecks when calling Rust classes.
So, with all that, I find that the best pattern for my use-case was actually C-style OO:
On the Rust side, I declare a dumb class which is just a wrapper so that I can return a Rust type to JavaScript. I also declare a factory function and a bunch of "methods": functions, that take a wrapper as a first argument.
On the JS side, I wrap a pair of (my backend neon module, wrapper class) into a proper TypeScript class, which gives me typed wrapper and allows me to inspect, verify and massage arguments before they are handled to Rust.
So, for me personally, I would rather prefer https://github.com/neon-bindings/neon/issues/286 (a generic JsBox class to send data to JS) to be fixed, rather then a more elaborate macro-based DSL.
PS. Have I already told you that neon is just awesome? :)

@matklad Have I told you your feedback on neon is just awesome? ;)

Although the tone will sound slightly negative, I mightily admire the work done, which allows to bind Rust and nodejs at all! :) ❣

Please don’t worry—negative experience reports are extremely valuable! But I appreciate your kind words as well.

Using macro DSL is quite hard: it is opaque, you have to do contortions to impl classes for generic types with < in their names,

That part is basically a bug that should be fixed. Unless we run into some fundamental parsing limitation, I think it’s little more than an oversight on my part.

and the only way to abstract over this macro is with another macro

That’s definitely true.

What's more, this DSL forces huge swaths of Rust code to be inside of macro invocation, which kills a lot of IDE features (of course the IDEs are not mature yet, but it always would be true that the code inside the macro will be harder for IDEs to understand).

Yes, this is also true. Although it doesn’t have to be—I would like to see Rust push on having an “expand-for-IDE” API for macros, so they can do fast, side-effect-free expansions that may not generate correct code for executing at runtime but they can provide sufficient information for IDEs to understand the code. I’ve spoken with some core team members who are interested in this but it’s of course not something that can happen overnight. So for the meantime you’re absolutely right that the macro will sometimes confuse IDEs.

It also effectively invents (ad-hoc, informally-specified, bug-ridden, slow implementation of) a new language withing Rust itself. the method "keyword" definitely feels alien.

“Slow” is a bit of a stretch but hey, you got your Greenspun’s law reference in. ;P Seriously I don’t like method either and I wonder if we can eliminate it.

There are also problems on the client side. I write my client code in TypeScript, and, naturally, I don't have any typechecks when calling Rust classes.

Yeah, I’ve been wondering if there’s some IDL-like approach we could use to generate .d.ts type declarations.

On the JS side, I wrap a pair of (my backend neon module, wrapper class) into a proper TypeScript class, which gives me typed wrapper and allows me to inspect, verify and massage arguments before they are handled to Rust.

There’s definitely a lot of sense to doing the “polish” work of a class (eg constructor and method APIs, massaging arguments, etc) in JS instead of Rust.

So, for me personally, I would rather prefer neon-bindings/neon#286 (a generic JsBox class to send data to JS) to be fixed, rather then a more elaborate macro-based DSL.

I completely agree this is worth building, to see how far we can push it.
And I understand and largely agree with a lot of your concerns about macros.
At the same time, I’m not ready to give up on the macro DSL. In part this is because I think we can eliminate a lot of the boilerplate around argument parsing and conversion.
And in part it’s because I want to build as gentle an on-ramp for JS programmers just trying Rust for the first time. Yes, you can argue it’s not “real Rust” to use a macro DSL, but to a first time Rust programmer it makes no difference: they just want to get their “hello world” working. Something that looks like TypeScript syntax (function/method declarations with type annotations) is familiar enough to conceptually pattern match without really understanding.
That said, I think we just need to try both something like the simpler primitive you referenced and the macro DSL, and see what works best in what environments. I don’t see a reason we can’t explore both.
And we might find a less elaborate macro, like just a function macro with automatic argument parsing, in conjunction with styles like the one you describe, end up working best, and a class macro ends up being more trouble than it’s worth. You’ve definitely given me a good alternative mental model to chew on.

PS. Have I already told you that neon is just awesome? :)

💕

Although it doesn’t have to be—I would like to see Rust push on having an “expand-for-IDE” API for macros, so they can do fast, side-effect-free expansions that may not generate correct code for executing at runtime but they can provide sufficient information for IDEs to understand the code. I’ve spoken with some core team members who are interested in this but it’s of course not something that can happen overnight.

As an IDE writer myself, I can assure you that code inside of macro invocations will always be more annoying to write than the usual code :) There are two reasons for this (even if we assume support from compiler): performance and robustness.
Great IDEs make use of a true syntax tree when reacting to user's keystrokes. You need the tree to correctly indent lines after \n, add or remove trailing commas, glue or split string literals over multiple lines, that sort of stuff. That means that getting a syntax tree is a part of typing latency of at least certain characters. Usually, this is not a problem, because constructing a syntax tree is a very local operation: you need to know only about the current file, you can always relex it incrementally, and often times you can even reuse parsing results. However, with macros you can't parse code using purely local information: the code inside macros is formally a token tree, and to get a real parse tree you must resolve the macro to its definition, and it is necessary a global operation (I.e, you need to look at other  files, consult caches, etc).
Also it can happen that you can't resolve macro because the build is broken: internet is down and Cargo is acting up, or you are resolving a merge conflict and <<<<<< break everything, or you are editing a file but have not linked it to the rest of project by the mod foo; declaration, or you are editing a temporary in-memory buffer. In all of these cases, you simply can't resolve macros, and so IDE is forced to look at the code as a token tree, even if it looks like a Rust function to a human :)
Obviously, I am not trying to say that one should not use macros at all because IDEs are stupid, and I don't even try to say that macros will be too annoying for IDEs. They'll be somewhat more annoying :)

In part this is because I think we can eliminate a lot of the boilerplate around argument parsing and conversion.

It would be great to have some non-macro based solution for this as well. I would love to have something like
let (some_int, some_str, callback, data, obj): (i32, String, JsFunction, MySerializableData, MyJsClass)
    = parse_args(call.scope, call.arguments)?;

Currently I am using neon-serde for something similar, but it works only for serializable objects, and if I have a mixture of POD data and, say, js callbacks, I have to do manual unwrapping of arguments.

And in part it’s because I want to build as gentle an on-ramp for JS programmers just trying Rust for the first time. Yes, you can argue it’s not “real Rust” to use a macro DSL, but to a first time Rust programmer it makes no difference: they just want to get their “hello world” working. Something that looks like TypeScript syntax (function/method declarations with type annotations) is familiar enough to conceptually pattern match without really understanding.

Yeah, that's a very valid use-case for DSLs! Two thoughts on this topic:

For "hello world" use case, it's really important to emphasize functions and not classes, because the are simpler.

Marcos for beginners are a double edged sword. On the one hand, you can map rust code to a form, which resembles what you would have written in JavaScript. On the other hand, if you make an error when using macro, the error message won't be great.


homebrew_package fails on arm64 macOS w/ chef-workstation + "native" homebrew

Description
There are a couple issues at play:

Presently, Chef Workstation is only available as an x86_64 package so it contains binaries that must be run through Rosetta. When a binary running under Rosetta calls another binary, it too will be run under Rosetta even if the called binary is "fat" and contains a native arm64 version. So when brew is shelled out from a chef-client run installed via Chef Workstation, it will be x86_64 arch.

Homebrew natively installed on an arm64 Mac will be installed under /opt/homebrew. The path under x86_64 is /usr/local/Homebrew. Homebrew checks the path that brew was called from and enforces these paths with errors like:

Error: Cannot install in Homebrew on ARM processor in Intel default prefix (/usr/local)!
Please create a new installation in /opt/homebrew using one of the
"Alternative Installs" from:
  https://docs.brew.sh/Installation

And vice versa if not providing a /usr/local/bin/brew symlink. (See stacktrace below)
homebrew_package fails because of issue 1. homebrew_tap and homebrew_cask can fail if not provided additional properties because they have a default of /usr/local/bin/brew for their homebrew_path attribute and simply symlinking isn't possible because of issue 2 and homebrew_cask calls homebrew_tap without any properties if the install_cask property is true (and is by default), I think the easy fix there is to just specify the homebrew_path attribute on that to be the same as was specified for it. (Submitted https://github.com/chef/chef/pull/11607 for this)
Issue 1 can be worked around by wrappering brew with arch -64 to select the native 64-bit arch for the machine. At least being able to surface the native architecture in Chef Workstation's x86_64 package would be helpful for determining what actions to take inside the resource (e.g. /usr/local/bin/brew vs. /opt/homebrew/bin/brew). You can use arch with uname to accurately ascertain the native architecture like arch -64 uname -m. Maybe a helper inside Chef could handle this or be a part of ohai?
Chef Version
➜ chef --version
Chef Workstation version: 21.5.420
Chef Infra Client version: 17.0.242
Chef InSpec version: 4.36.4
Chef CLI version: 5.0.1
Chef Habitat version: 1.6.288
Test Kitchen version: 2.11.2
Cookstyle version: 7.10.1
Platform Version
macOS 11.3.1 on MacBook Pro M1
Replication Case

Install "native" homebrew via brew.sh standard install.
Install Chef Workstation from dmg.
Run a recipe with homebrew_package like

  homebrew_package 'coreutils' do
    action :upgrade
  end

Client Output
See stacktrace.
Stacktrace
* homebrew_package[coreutils] action upgrade[2021-05-20T15:40:14-07:00] INFO: Processing homebrew_package[coreutils] action upgrade (macos-workstation::base_software line 23)

    ================================================================================
    Error executing action `upgrade` on resource 'homebrew_package[coreutils]'
    ================================================================================

    Mixlib::ShellOut::ShellCommandFailed
    ------------------------------------
    Expected process to exit with [0], but received '1'
    ---- Begin output of ["brew", "install", "coreutils"] ----
    STDOUT:
    STDERR: Error: Cannot install under Rosetta 2 in ARM default prefix (/opt/homebrew)!
    To rerun under ARM use:
        arch -arm64 brew install ...
    To install under x86_64, install Homebrew into /usr/local.
    ---- End output of ["brew", "install", "coreutils"] ----
    Ran ["brew", "install", "coreutils"] returned 1

    Resource Declaration:
    ---------------------
    # In /Users/matt/.chef/local-mode-cache/cache/cookbooks/macos-workstation/recipes/base_software.rb

     23:   homebrew_package pkg do
     24:     action :upgrade
     25:   end
     26: end

    Compiled Resource:
    ------------------
    # Declared in /Users/matt/.chef/local-mode-cache/cache/cookbooks/macos-workstation/recipes/base_software.rb:23:in `block in from_file'

    homebrew_package("coreutils") do
      package_name "coreutils"
      action [:upgrade]
      default_guard_interpreter :default
      declared_type :homebrew_package
      cookbook_name "macos-workstation"
      recipe_name "base_software"
    end

    System Info:
    ------------
    chef_version=17.0.242
    platform=mac_os_x
    platform_version=11.3.1
    ruby=ruby 2.7.3p183 (2021-04-05 revision 6847ee089d) [x86_64-darwin18]
    program_name=/usr/local/bin/chef-client
    executable=/opt/chef-workstation/bin/chef-client

I mean the fix for #1 is to get workstation compiled natively for M1 mac, we're not going to patch things so that rosetta-run chef will be able to install native homebrew into /opt/homebrew.  And that needs to be addressed in chef-workstation (I know that's in flight, but have no idea when that might be delivered, but its pretty high priority, just a large yak shave to get everything building there).
The /usr/local -> /opt/homebrew change is something that we should do though so that native chef-client does the right thing there.

I'm going to decline this one in Chef Infra Client since the fix is actually to get Chef Workstation building on arm for macOS. There's a plan to do this once we get habitat building on arm.


The SMS Provider reported an error.

Hello,
Tried many different builds - tried re-installs but always failure with ConfigMgr - Driver Pkg
Error
The SMS Provider reported an error.

If I use ConfigMgr - Standard Pkg there is no issue when creating the package but I can't get Invoke-CMApplyDriverPackage.ps1 working with the AdminService account
Anyone please help me out with ConfigMgr - Driver Pkg error
Many Thanks
Greetings
Christoph

I've figured out that a clean install SCCM creates the driver packages - is there a way to keep existing driverpackages and create new one's with Driver Automation Tool ?
Next I've cleaned an existing deprecated driver package and tried to renew with Driver Automation Tool but still the error persist

I too am experiencing this issue but only on the new LARGE Dell EXE based driver sets.  Tried MANY things, still no resolution.  Will extract ok and imports anywhere between 10 and 50 before it pukes.


After upgrading from 1.1.3 to 1.3.0 DynamoDB client broken for real AWS DynamoDB service

Hi! After upgrade quarkus from 2.12.3 to 2.13.1 (with upgrading quarkus-amazon-services from 1.1.3 to 1.3.0) all ours DynamoDB client failed on real DynamoDB service (on mock server - all still work).
Explicitly we not provide any DynamoDB properties.
DynamoDB client call throws errors like this:
dynamodb-ssl-error.txt
If i set property quarkus.dynamodb.async-client.tls-trust-managers-provider.type to trust-all issue fixed, but this not look as solution for production.
Look like this broken by this changes.

@gsmet, on 1.2.0 i have same issue

Fixed and I started releasing 1.3.1.


Help text/tooltips

Add help text/tooltips. Wait until final design complete before adding.

Moved to main repo


Add new Resource of identity for service account agent

Category

[x] New feature
[ ] Bug fix
[ ] Improvement
[ ] Refactor
[ ] etc

Description
With the development of the 'Service Account Agent' mode, the corresponding Protobuf has been added. Please refer to the service_account.proto and app.proto files.

@yjinjo Please add "signed-off-by"


Network Partition Handling

As a user of Cloudstate, I would like the platform to "handle" network partitions for me, ensuring that I don't have data corruption or lowered availability as much as practical, so that I don't have to build this into my applications myself.
"Handle" in this context probably means detect, take action, and (if possible) alert (or log, so I may have my infrastructure alert if I wish)

What types of network partitions would you like the protocol to address? Between the sidecar and the entity function? Or between the Akka cluster?
I noticed in some tests that after the proxy connects to an entity and a discovery message exchange is successful, the proxy never tries to communicate with the entity again until there is an external request.
Perhaps the proxy should perform a heartbeat message from time to time, or what I think is best, monitor the Liveness Probe of the entity application and thereby feed some kind of metric.
I believe that the proxy can implement some treatment policy via RestartFlow in the event that a request for a user function fails. But I'm not sure about that, maybe @jroper or @viktorklang sis the best persons to answer that.

I was thinking only of Akka Cluster partitioning, but you make a good point, that's not the only failure point we could see.

All containers (entity, proxy, etc.) in a Pod are co-located and share the same network address, hence they are not affected by network partitions.
Failures like described might occur and the suggested treatment might be a good solution, but should be out of scope of this issue. @sleipnir, could you open a new issue please, if that makes sense?

@hseeberger I opened a new issue as suggested

but I think what he meant was about the possible network failures in the formation of the AKKA Cluster and not about the connections that occur over the same namespace. That's why I asked the previous question to disambiguate the question.
Am I right @michaelpnash ?

The split brain resolver automatically handles network partitions. The Lease strategy may be a good choice as we already have the k8s requirement and require a dynamic solution.

@edwardcallahan How many clusters & nodes will it be able to support?

Majority is the better, more conservative approach.
Enable for Majority.


pass command line args

docker build is failing, want to retry

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d19c4fe). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #63   +/-   ##
=========================================
  Coverage          ?   50.12%
=========================================
  Files             ?      129
  Lines             ?     8008
  Branches          ?        0
=========================================
  Hits              ?     4014
  Misses            ?     3602
  Partials          ?      392

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 d19c4fe...91110a4. Read the comment docs.


Add APIs to calculate and cleanup repository states

This makes it easy to determine that the repository is in the middle of a merge and abort it, for example.

The CI build timed out.

Looks legit :+1:


hello

Hello, I have used the Go-Twitter you wrote, and I think it is very useful. I hope to get to know you and jointly maintain this project with you. I hope you can give me your contact information， thinks

is still alive the project?


Why index error 300?

Code version (Git Hash) and PyTorch version
Dataset used
Expected behavior
Actual behavior
Steps to reproduce the behavior
Other comments
When I generate train.npy and pkl files using my dataset, I got indexerror:
'IndexError: index 300 is out of bounds for axis 1 with size 300'. Like this.
Where can I need to fix?

Oh! I found my error. One of my video files are over 300. That is why I got this error. Sorry for my mistake.


Interpretation of Multi-Class

Hello,
I would like to use the Multi-Class likelihood on the SVGP model. However I noticed that neither in the notebooks or the docs you precise how to recover the mapping from the different classes in the model to the labels of the data. The results of the prediction are therefore completely non interpretable for test data. Or did I miss something?

It's the robust max likelihood, and  more details can be found in the following:
[31] M. Girolami and S. Rogers. Variational Bayesian multinomial probit regression with Gaussian process
priors. Neural Comp., 18:2006, 2005.
[32] H. Kim and Z. Ghahramani. Bayesian Gaussian Process Classification with the EM-EP Algorithm. IEEE
TPAMI, 28(12):1948–1959, 2006.
[33] D. Hernandez-Lobato, J. M. Hern ´ andez-Lobato, and P. Dupont. Robust multi-class Gaussian process ´
classification. In NIPS, pages 280–288, 2011.
These references were copied from http://papers.nips.cc/paper/5875-mcmc-for-variationally-sparse-gaussian-processes.pdf, which is the basis for much of the gpflow code.

I should add that it's easy to interpret the probabilities, conditioned on the latent function: the probabilities are just 1-esp (for the largest) or e/(K-1) (all the others). Perhaps this doesn't seem all that nice compared to the softmax, where the probabilities are just the sortmax of the function values. When the integral is taken over the latent functions it's not so clear, however, as it depends on the mean and the variance of each function (as is also the case for the softmax: though in that case just using the mean for each function might give something sensible, whereas for the robustmax it certainly doesn't)

Thanks for the fast answer. My question is actually a lot more practical ^^. When using predict_y(X_test).
I will get arrays of probability (here computed through RobustMax). But how do I know which index of the array is attributed to which class? I tried to investigate the code, but I could not find this mapping anywhere.
For example, for a problem with 3 classes (let's say A B and C), when I compute p, _ = predict_y(x) a single test point. To which class refers p[0] ?

Oh I see: Sorry I totally misunderstood your question!
For the multiclass likelihood the classes must be labelled as 0, 1, ...K-1, which correspond to the indices of the output (i.e. p(y=k) = p(y)_k).  In a previous version of the code there was check to make sure that users didn't accidentally use 1, ..., K indexing, but the current code doesn't do this. If you pass A, B, C for the Y values it won't work properly (though might fail silently. I haven't tried)
Also, for Bernoulli the positive class must be labelled 1 and everything else is the other class.

Ah now I get it, the example is indeed doing it, but I did not understand it was a rule, thanks for the clarification.
Cheers,


Please document not to combine minimumExpectedTests and rerunFailedTests

I noticed our build was very unstable. I found out after checking a strange result, if you combine
rerunFailedTests with minimumExpectedTests (with more than 1), the whole thing fails if it needs the rerun 1 test.
I think this is bug, so also another user, but it's closed as not a bug
So please document this.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 16bd8dd1-7650-8c18-2553-38a8bc54e892
Version Independent ID: f9fa517a-7adc-e327-d12e-256295a1ed52
Content: Visual Studio Test task - Azure Pipelines
Content Source: docs/pipelines/tasks/test/vstest.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @shashban
Microsoft Alias: shashban

It's just documentation. Why is this taking ages? @shashban

This about documentation of a bug and even this is not working??

IMO it's a but, but it's closed as not a bug.
https://developercommunity.visualstudio.com/t/Combination-of-rerun-tests-and-minimum-n/1236219

@shashban For me this is also an issue, please fix or at least document.

Sad


HADOOP-17485. port UGI#getGroupsSet optimizations into 2.10

2631

 Quality gate: Failed
85 Issues

 12 Vulnerabilities
 73 Code issues

Details

 Quality gate: Failed
84 Issues

 12 Vulnerabilities
 72 Code issues

Details


Data preprocessing

Dear Authors:
I was trying to repeat your results. But I found the data/Datapre.py script did not fit the raw videos in CH-SIMI.zip (downloaded from Google Drive). I tried to wrote scripts to process the raw videos, but the train/val/test split index files did not fit the raw video index.
I am asking if it is right to use train_index.csv files to process the data.npz feature files? For example, when id '0' in the train_index.csv, if the '0'th feature in data.npz file belongs to the training set.
It would be very helpful if you can update the processed feature or the whole folder so that data/DataPre.py can process it properly.

@NicholasGote  Thanks for your attention!

The problem of data/Datapre.py
We updated our data/Datapre.py a few days ago, which may be not compatible with the old CH-SIMS.zip.  New data/Datapre.py has a more convenient way to extract features from raw videos, which needs a preprocessed label file and the root dir of raw videos. Please see our codes for details.

The index problem
Yes. It is right for "when id '0' in the train_index.csv, if the '0'th feature in data.npz file belongs to the training set."

The processed feature update problem
Sorry, we forgot to update our processed features in Google and Baidu Drives. Now, we have done that. You can download new features CH_SIMS_unaligned_39.pkl from Baidu Yun Disk [code: g63s] or Google Drive, which is compatible with our new code structure. The md5 code is a5b2ed3844200c7fb3b8ddc750b77feb.

It is worth noting that the CH_SIMS_unaligned_39.pkl is compatible with the feature structure.

Thanks a lot.


Bump s3cli to 0.0.45

Includes fix for Region being silently overridden if it didn't
match the provided AWS Hostname

#127485941

As part of merging this in, someone will need to do bosh add blob on https://s3.amazonaws.com/s3cli-artifacts/s3cli-0.0.45-linux-amd64 and remove the old blob from the blobs.yml.


Useless files such as demo.apk and images

Hello,
Is there any interest of putting the whole project files into the package download folder ?
We can notice the demo.apk file and lot of heavy image files.
Since we can download the demo app and see related images from your package home page, any reason to put them in package download folder ?
We do not need these files locally and their uselessly increase download size

these files is not useless,they need to be displayed in README.If you have good advice,you can tell me.


multiple definitions of swap from thread_pool.hpp

Greetings !
In boost/network/utils/thread_pool.hpp, line 101, there's global swap function that gets declared. The problem is, it's not inline, and the compiler complains about multiple definition for each target that includes thread_pool.hpp.
Seeing the history, this function used to be a template, 20 days ago or so. As a template, this issue wouldn't happen: but as a global function, we have to either declare it as inline, or define it away from the headers.
As far as I know, there's no other way to go around that.

Oops. You're right. I'm fine with making it inline. I'll get to this soon enough.

@Plaristote -- this is in the master branch, yes?


Make available on CDN.JS

I use this library quite a lot and it would be great if I could also leverage CDN.js's cloudflare enabled service as well.

Feel free to do so. I don't use CDNJS or know how to add it there.

Thanks - Here's the request https://github.com/cdnjs/cdnjs/issues/12348


Use 'fake' phone number in example

Your phone number being used (+44<PHONE_NUMBER>0) in the README shouldn't really be used in documentation.
Whilst the number range is quarantined by Ofcom, it can (and does from my UK mobile) connect somewhere unattended.
Instead, you can use Ofcom Drama Numbers, that are designed for use in films/TV etc., and can never be allocated. Calls to these destinations will fail.
The number used still validates in libphonenumber (for php) (which is what Laravel Phone uses).
Additionally, GB numbers are not formatted +44 0..., the leading zero is not used when converting it into an international format.

Awesome! Thanks for the contribution 🚀


Access-Control-Allowed-Origin header not exposed with string array using use in express and Router

Hi,
I've trying to usa a string array with origins url, but don't expose the Access-Control-Allowed-Origin header in response.
I've trying to use like
  app.use(
    cors({
      credentials: true,
      origin: ["http://www.urltest.com", "http://www.urltest2.com", "http://www.urltest3.com"]
    })
  );

and
var corsOptions = {
      origin: ["http://www.urltest.com", "http://www.urltest2.com", "http://www.urltest3.com"],
      optionsSuccessStatus: 200,
      allowedHeaders: ['Content-Disposition', 'content-disposition', 'Access-Control-Allow-Headers', 'access-control-allow-headers', 'authorization', 'Content-Type'],
      exposeHeaders: ['Content-Disposition', 'content-disposition', 'Access-Control-Allow-Headers', 'Access-Control-Allow-Origin', 'access-control-allow-headers' , 'authorization', 'Content-Type']
    }
router.use(cors(corsOptions));

But without success.
Can anyone help me?
I find some issues with this theme but all them open yet.

Are you sure the origin is http://www.urltest.com and not https://www.urltest.com?

Hi @git-xbits as far as I can tell, this module works fine when the calls are from one of the origins you listed above in your code. Perhaps what @abhijit-hota said would apply to you. But unfortinately we can only guess what the issue is you are having, especially without knowing where the requests are coming from, what headers are in the request, what the application looks like, etc.
Your post above may be more suited as a question on StackOverflow than this module's issue tracker. If you believe that there is an issue with this module, please let us know and we can re-open this issue. As a pre-requisite to re-open,please provide enough information for us to be able to reproduce your issue, or we won't be able to track down the case.


slweb: init at 0.5

Description of changes

slweb is a static website generator which aims at being simplistic. It transforms custom Markdown-like syntax into HTML.
Homepage: https://strahinja.srht.site/slweb/
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

In case you haven't noticed, one of ofborg's checks failed: slweb.passthru.tests on x86_64-darwin

Indeed. However, I do not own a mac and I do not really know what I should do...
Any idea ?

In case you haven't noticed, one of ofborg's checks failed: slweb.passthru.tests on x86_64-darwin

Indeed. However, I do not own a mac and I do not really know what I should do... Any idea ?

I think setting platforms to platforms.linux for now should sort it. (I'm not sure of the pros and cons of that vs setting broken = stdenv.isDarwin;)

Result of nixpkgs-review pr 205314 run on aarch64-linux 1

1 package built:

slweb

It seems to be OK to merge right ?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1548

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/755


confused about the type of metadata

I have some confused about the option type of metadata in tokenInfo, allow to mint a nft without metadata?

Yes, this is to allow you to mint a token first, reveal the metadata later

maybe it's not necessary  to complete it in two steps (i am not sure)

Just provide the option, you can also mint the token with metadata.


TextInput action doesn't support setting non-string property

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
4.11.1
Describe the bug
See a user comment here. Use TextInput to set a property with value extracted from LUIS recognized entities. If the recognized entities are an array, the TextInput action will refuse setting this array value to a property due to the string validation here https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Input/TextInput.cs#L51. This design seems not friendly. We think the string validation should only be applied to the raw user input rather than the property setting. Property should be able to take any valid object rather value limited to string.
To Reproduce
Steps to reproduce the behavior:

Use TextInput action
Set the Property with dialog.userInput
Set the Value with an expression that returns an array, e.g. =createArray('hello', 'world')
Send a response with the ${dialog.userInput}
You will see Object reference not set to an instance of an object.

Expected behavior
Non-string input can be set to property
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@tomlm @Stevenic can you chime in on this one, see if the restriction put on TextInput should be loosed or not.
A short version of above proposal is
Current restriction: "The final property set must be a string"
Proposed loosed restriction: "The value get from input must be a string, but the property eventually set don't have to be a string after post processing"

@tomlm @Stevenic can you chime in on this one, see if the restriction put on TextInput should be loosed or not.
A short version of above proposal is
Current restriction: "The final property set must be a string"
Proposed loosed restriction: "The value get from input must be a string, but the property eventually set don't have to be a string after post processing"

I made a fix in this commit. This is just a proposal, not a final fix. This fix is to allow users to set this.property with any objects evaluated from this.value. There is no validation or formatting on the result of this.value. If this.value is left null, then it will get the input from activity and perform recognition and validations on the activity input as what we do now. Do you have any insights? @boydc2014 @tomlm

I made a fix in this commit. This is just a proposal, not a final fix. This fix is to allow users to set this.property with any objects evaluated from this.value. There is no validation or formatting on the result of this.value. If this.value is left null, then it will get the input from activity and perform recognition and validations on the activity input as what we do now. Do you have any insights? @boydc2014 @tomlm

After checked current design with Dong, it should be by design that TextInput can only set text property currently, so will close this issue.


feat: add Array.concat and CI sanity check for base library compiler warnings

This PR addresses a compiler warning from using Array.append in the Principal.toLedgerAccount function. I also updated the CI to detect unintended compiler warnings when importing any base library module.

Closing (merged #630).


Disabling indentation detection is not enforced

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:57:00.652Z)
OS Version: Linux x64 4.13.8-1-ARCH
Extensions:

Extension
Author (truncated)
Version

xml
Dot
1.9.2

vscode-docker
Pet
0.0.20

python
don
0.7.0

Go
luk
0.6.66

hg
mrc
1.2.0

mssql
ms-
1.2.0

azure-account
ms-
0.1.3

erlang
pgo
0.1.1

java
red
0.12.0

vscode-apielements
vnc
0.6.7

Steps to Reproduce:

Define user settings for apiblueprint:

    "[apiblueprint]": {
        "editor.tabSize": 2,
        "editor.detectIndentation": false
    }

Open apib file
tabsize changes despite setting detect indentation to false.

Note: apib files contain varying different indentations for different parts of the spec. eg.
+ Model

    + Headers

contains a tabsize of 4 spaces (or rather think of it as 2 tabs)
where actual json contains tabsize of 2 spaces, so detect indentation doesn't really work. But if I turn that feature off it should stick to my defined tabsize

Reproduces without extensions: Yes/No
No

This works fine for me. I believe it got fixed in the meantime.
    "[plaintext]": {
        "editor.tabSize": 7,
        "editor.detectIndentation": false
    },

I create a file test.txt with the following content:
    4 spaces
    4 spaces
    4 spaces
    4 spaces
    4 spaces
    4 spaces
    4 spaces
    4 spaces
    4 spaces

The indentation guesser would typically guess that this file is indented with Spaces@4. When opening the file the bottom status bar shows Spaces@7.


Henrhoi/property-names-and-alias-change

Have changed names on parameters, as discussed on Architect-meeting. I have only added the one alias "tracelevel".
I do not exactly see where I can and should add the new aliases, for the other parameters that do not have a QueryProfileType. Perhaps you could help me, or come with some tips?
@bratseth

Considering this more, I also think we should also wait with merging this until Vespa 7.
That will happen this quarter in any case.
It won't break people using the search api over http but it will break any who set these parameters in an upstream searcher without using the static name variable (which they have not been able to in this case).
Anyway, it is good to complete it.


fix mvt versioning redux

Redux of #85 against latest master

Interesting, this is now working enough (after rebasing on #87) such that the tests are running. But there is this curious error:
# [composite] resolves polygon clockwise error in overzoomed V1 tiles
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
/home/travis/build/mapbox/vtcomposite/mason_packages/.link/include/boost/geometry/algorithms/detail/overlay/sort_by_side.hpp:525:60: runtime error: unsigned integer overflow: 0 - 1 cannot be represented in type 'unsigned long'
    #0 0x2b281df7838d in void boost::geometry::detail::overlay::sort_by_side::side_sorter<true, false, (boost::geometry::overlay_type)1, mapbox::geometry::point<long>, boost::geometry::strategy::side::side_by_triangle<void>, std::less<int> >::find_polygons_for_source<&(boost::geometry::segment_identifier::source_index)>(long, unsigned long)

I think the Skipping feature with malformed geometry (v1): expected command 2 but got 7 are likely okay/expected given this is a v1 tile. But the boost::geometry integer overflow is odd: is this a bug in boost::geometry? Would this produce incorrect results in release mode/production? /cc @artemp

But the boost::geometry integer overflow is odd: is this a bug in boost::geometry? Would this produce incorrect results in release mode/production? /cc @artemp

Looks like there is a std::size_t value of 0 in overflowing inside boost::geometry to become<PHONE_NUMBER>3709551615
I looks like there are multiple ways to trigger this problem in boost. One is simple: passing a zero-area polygon like [[8128,3600],[8224,3664],[8128,3600]]. This happens if I try removing the dupe filtering added in https://github.com/mapbox/vtcomposite/pull/82/files#diff-30bb0ced2eaed551287987a4af36957eR123. But there is another polygon that, even with the dupe filtering, triggers this error. It is:

[[-244,3348],[-252,3380],[-256,3392],[-264,3432],[-268,3448],[-268,3456],[-272,3464],[-272,3468],[-284,3476],[-284,3476],[-284,3480],[-288,3488],[-292,3492],[-296,3492],[-300,3496],[-304,3496],[-320,3500],[-320,3500],[-324,3504],[-328,3500],[-328,3496],[-336,3500],[-344,3508],[-352,3512],[-348,3512],[-348,3512],[-348,3516],[-348,3516],[-352,3516],[-352,3512],[-356,3512],[-356,3512],[-360,3512],[-364,3516],[-376,3520],[-380,3520],[-384,3520],[-388,3524],[-388,3524],[-388,3524],[-388,3524],[-392,3520],[-392,3520],[-392,3524],[-392,3524],[-392,3524],[-396,3524],[-396,3520],[-400,3520],[-400,3520],[-400,3520],[-408,3520],[-416,3520],[-424,3520],[-432,3516],[-448,3508],[-452,3504],[-456,3504],[-464,3500],[-476,3496],[-480,3496],[-484,3496],[-484,3492],[-488,3496],[-488,3496],[-488,3496],[-488,3500],[-488,3500],[-488,3504],[-488,3504],[-492,3500],[-492,3492],[-496,3492],[-496,3488],[-500,3484],[-504,3484],[-508,3484],[-512,3488],[-512,3488],[-512,5092],[-512,5092],[-508,5096],[-504,5100],[-500,5096],[-496,5100],[-492,5096],[-488,5096],[-484,5092],[-484,5092],[-480,5088],[-476,5076],[-472,5072],[-468,5068],[-452,5064],[-452,5068],[-444,5068],[-444,5068],[-440,5068],[-436,5064],[-432,5064],[-428,5064],[-424,5068],[-416,5072],[-412,5076],[-408,5080],[-404,5088],[-396,5088],[-392,5088],[-388,5088],[-384,5084],[-384,5080],[-380,5080],[-380,5080],[-380,5080],[-376,5076],[-376,5072],[-376,5072],[-372,5068],[-368,5064],[-368,5064],[-364,5052],[-364,5048],[-356,5044],[-356,5044],[-352,5044],[-348,5044],[-348,5040],[-348,5040],[-348,5040],[-348,5036],[-348,5036],[-348,5032],[-348,5032],[-344,5024],[-340,5008],[-336,5004],[-336,4996],[-336,4988],[-332,4980],[-328,4972],[-316,4964],[-312,4956],[-308,4952],[-308,4948],[-304,4944],[-296,4936],[-288,4932],[-284,4932],[-284,4932],[-280,4928],[-276,4928],[-276,4928],[-276,4928],[-272,4928],[-272,4928],[-272,4928],[-268,4920],[-268,4916],[-268,4916],[-268,4916],[-268,4916],[-264,4916],[-260,4912],[-260,4912],[-252,4904],[-248,4900],[-244,4900],[-236,4900],[-224,4896],[-224,4896],[-216,4900],[-216,4900],[-216,4900],[-216,4900],[-204,4904],[-200,4904],[-204,4904],[-204,4900],[-204,4900],[-200,4896],[-200,4896],[-196,4896],[-184,4896],[-176,4896],[-168,4900],[-168,4900],[-164,4900],[-160,4896],[-160,4896],[-152,4896],[-148,4892],[-144,4892],[-140,4892],[-136,4892],[-136,4892],[-136,4892],[-132,4892],[-132,4888],[-128,4888],[-128,4884],[-128,4880],[-132,4880],[-132,4884],[-136,4880],[-140,4880],[-148,4876],[-148,4872],[-156,4864],[-156,4860],[-156,4856],[-152,4856],[-148,4852],[-144,4840],[-140,4848],[-136,4844],[-140,4836],[-140,4832],[-132,4812],[-128,4808],[-124,4804],[-124,4804],[-120,4800],[-120,4796],[-124,4780],[-120,4772],[-120,4768],[-120,4764],[-116,4760],[-116,4756],[-112,4752],[-108,4752],[-100,4752],[-100,4752],[-96,4748],[-96,4728],[-96,4720],[-88,4720],[-88,4716],[-84,4716],[-84,4716],[-76,4716],[-76,4712],[-76,4712],[-76,4708],[-76,4704],[-68,4696],[-68,4696],[-64,4692],[-64,4692],[-44,4684],[-40,4680],[-36,4676],[-28,4676],[-20,4676],[-16,4672],[-12,4672],[-12,4668],[-12,4664],[-12,4660],[-8,4656],[-8,4656],[-4,4656],[0,4656],[4,4652],[8,4652],[8,4652],[12,4652],[12,4648],[12,4644],[16,4644],[16,4644],[20,4640],[20,4640],[24,4640],[24,4640],[24,4640],[28,4632],[28,4632],[32,4628],[36,4628],[40,4628],[40,4628],[40,4624],[40,4620],[36,4620],[36,4616],[32,4616],[32,4616],[32,4612],[32,4612],[28,4608],[20,4600],[12,4600],[4,4600],[0,4596],[-4,4592],[-12,4588],[-20,4580],[-20,4576],[-28,4564],[-36,4556],[-40,4544],[-44,4532],[-48,4528],[-44,4528],[-44,4524],[-44,4524],[-44,4520],[-44,4516],[-56,4492],[-60,4472],[-60,4464],[-60,4452],[-60,4444],[-60,4440],[-52,4432],[-52,4428],[-52,4424],[-48,4420],[-44,4416],[-40,4412],[-40,4408],[-36,4404],[-36,4400],[-36,4396],[-16,4364],[-8,4352],[0,4344],[0,4340],[0,4520],[108,4632],[0,4740],[0,5292],[-8,5296],[-16,5300],[-24,5304],[-28,5304],[-36,5300],[-48,5296],[-52,5292],[-56,5292],[-56,5288],[-56,5284],[-56,5284],[-56,5280],[-60,5280],[-60,5280],[-60,5276],[-64,5276],[-68,5276],[-76,5280],[-76,5280],[-80,5280],[-84,5280],[-84,5280],[-84,5280],[-88,5280],[-88,5280],[-88,5280],[-88,5284],[-88,5284],[-88,5284],[-88,5288],[-96,5304],[-96,5308],[-100,5308],[-100,5308],[-104,5308],[-104,5308],[-108,5312],[-112,5316],[-116,5316],[-116,5320],[-124,5320],[-128,5320],[-128,5316],[-128,5316],[-128,5312],[-132,5312],[-132,5312],[-136,5312],[-140,5312],[-140,5308],[-144,5308],[-144,5308],[-144,5304],[-152,5308],[-156,5316],[-160,5320],[-164,5320],[-164,5320],[-164,5316],[-168,5316],[-172,5320],[-176,5320],[-176,5324],[-180,5324],[-188,5328],[-188,5328],[-192,5332],[-196,5336],[-204,5340],[-208,5348],[-208,5348],[-212,5348],[-216,5352],[-216,5352],[-224,5368],[-224,5372],[-228,5376],[-228,5388],[-232,5392],[-232,5396],[-236,5396],[-240,5400],[-244,5400],[-248,5408],[-248,5408],[-248,5408],[-256,5408],[-272,5412],[-276,5412],[-296,5432],[-300,5436],[-300,5436],[-312,5440],[-316,5448],[-320,5448],[-324,5452],[-328,5452],[-328,5452],[-336,5452],[-352,5460],[-356,5460],[-356,5460],[-356,5460],[-360,5460],[-372,5460],[-380,5460],[-388,5456],[-392,5456],[-400,5460],[-400,5456],[-400,5456],[-400,5456],[-400,5456],[-404,5460],[-408,5456],[-416,5456],[-420,5452],[-424,5452],[-440,5444],[-444,5448],[-448,5448],[-452,5452],[-452,5456],[-460,5456],[-468,5456],[-484,5456],[-496,5452],[-504,5448],[-512,5436],[-512,5436],[-512,5440],[-508,5448],[-500,5452],[-512,5452],[-512,11372],[-488,11376],[-460,11384],[-428,11392],[-412,11400],[-408,11400],[-408,11400],[-408,11404],[-408,11404],[-404,11404],[-400,11408],[-392,11412],[-392,11416],[-392,11416],[-392,11416],[-388,11416],[-388,11416],[-384,11416],[-384,11420],[-384,11420],[-384,11424],[-380,11424],[-380,11424],[-376,11424],[-372,11424],[-360,11420],[-360,11420],[-356,11420],[-352,11412],[-348,11408],[-336,11404],[-320,11400],[-320,11400],[-320,11400],[-320,11400],[-320,11396],[-320,11396],[-312,11388],[-312,11388],[-300,11380],[-296,11380],[-296,11380],[-296,11380],[-296,11376],[-296,11376],[-296,11372],[-292,11372],[-288,11372],[-284,11372],[-284,11372],[-280,11368],[-264,11368],[-260,11364],[-256,11364],[-252,11364],[-252,11364],[-248,11364],[-248,11364],[-244,11360],[-240,11360],[-228,11360],[-224,11360],[-216,11356],[-212,11352],[-212,11352],[-212,11348],[-192,11340],[-188,11340],[-184,11340],[-184,11336],[-180,11336],[-180,11340],[-180,11340],[-176,11340],[-172,11340],[-168,11340],[-168,11340],[-164,11336],[-164,11340],[-160,11340],[-144,11336],[-144,11336],[-140,11332],[-128,11320],[-124,11320],[-116,11316],[-116,11316],[-112,11316],[-108,11312],[-108,11312],[-104,11312],[-92,11308],[-84,11304],[-84,11304],[-84,11300],[-80,11296],[-68,11288],[-64,11284],[-56,11284],[-16,11272],[-16,11272],[-12,11272],[-12,11268],[-8,11268],[-8,11268],[0,11264],[0,12288],[-512,12288],[-512,3308],[-244,3348],[-244,3348],[-244,3348]]

I think this is also likely containing zero-area slivers. I'm not sure that we can do anything about this. So it may make most sense to disable this test for the sanitizer run. Unless @artemp has other ideas?

Filed https://github.com/boostorg/geometry/pull/505 to see whether this should be fixed in boost::geometry, but for now I've skipped this test when using the sanitizers. I think this is ready to merge therefore.

Interestingly, the test above is passing for me:
# [composite] resolves polygon clockwise error in overzoomed V1 tiles
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 1 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
Skipping feature with malformed geometry (v1): expected command 2 but got 7
ok 1 v1 tiles with polygons composite successfully

@springmeyer @millzpaugh  -do I need to do anything extra to trigger it ?


Replace MovieClip with a more robust solution

Currently MovieClip is pretty simple, I'd like to replace it with a more robust animated sprite solution. More similar to what Phaser does for animations.
Ideally it would support:

Different display times per frame
Multiple different named animation sequences
play, pause, reverse, jump

It would also be great if there was a more precise means to specify animation speeds. This isn't very descriptive. 😄

Bumping to keep this open

Would we want a singleton animation manager of some sort like Phaser, or purely self contained animation controls like the current AnimatedSprite? Ideally, whatever API we come up with could also apply to pixi-spine with minimal changes (easier with a manager that can wrap the spine API).

I think a robust animation manager solution sounds great, but I'm not sure if it belongs in the main repo? Definitely would be easier to start as a plugin and could be promoted to live here later.

I agree. Something like this is likely to be complex and feature rich. A plugin is good approach to start. It could live in the main repo, and simply be opt-in plugin, like the unsafe-eval plugin we added recently.


Conflict with "Customizable Player Models" mod. stuck in a loop on menu.

it always trying to reload resourcepacks.
Conflict with "Customizable Player Models" mod.
https://www.curseforge.com/minecraft/mc-mods/custom-player-mode
log file
latest.log
https://user-images.githubusercontent.com/51751845/144784921-af579bf0-bc2b-4921-9e69-5beb5b6ea5fd.mp4

@SHsuperCM
Strongly related to #73. Could possibly hotfix using mixin priority until #73 is fixed properly.

Fixed for v0.8.2


fix: Blob.downloadTo() methods do not wrap RetryHelper$RetryHelperException

Fixes #100

I think we will need to make an exception for now but may be good to file
an issue to track it as future work.
On Wed, Apr 1, 2020, 11:57 PM Dmitry<EMAIL_ADDRESS>wrote:

@dmitry-fa commented on this pull request.
In
google-cloud-storage/src/test/java/com/google/cloud/storage/BlobTest.java
https://github.com/googleapis/java-storage/pull/218#discussion_r402090228
:

@@ -662,4 +661,25 @@ public Long answer() throws Throwable {
byte actual[] = Files.readAllBytes(file.toPath());
assertArrayEquals(expected, actual);
}

@Test

Is this rule applied to the tests within a single file or withing a
package?
Is it possible to make an exception?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/java-storage/pull/218#discussion_r402090228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAORXQB5KYTBLUVFXI4DNWDRKQZMBANCNFSM4LXMU2UQ
.


How to fit the text inside dropdown list?

Hello there,
I have a problem with the text length. I use ajax and got my data from server. Some elements are longer that width of dropdown ul and li elements. How can I use overflow-x: hide on that lines, which are longer the width of ul and li?
If it is possible give me example please.
Thank you.

Hi the width of the list is the same as the select.
You can add this to the css to overflow big lines:
.tokenize-dropdown > .dropdown-menu li > a {     text-overflow: ellipsis;     overflow-x: hidden; }

Thank you very much
Sent from Mail.Ru app for Android Friday, 28 April 2017, 05:42p.m. +06:00 from David Zeller<EMAIL_ADDRESS>:

Hi the width of the list is the same as the select.
You can add this to the css to overflow big lines:
.tokenize-dropdown > .dropdown-menu li > a { text-overflow: ellipsis; overflow-x: hidden; }
—
You are receiving this because you authored the thread.
Reply to this email directly,  view it on GitHub , or  mute the thread .


Added auth client for Zoho

Zoho is an enterprise business software provider. They have a massive user base. So it's a very much essential library for developers who want to integrate with Zoho

Is anyone here to review this PR?


Add character:  to font:

Character :
Font :

@komal7141, I am giving this to @ben-dh3 as he has already done the PR. @ben-dh3 request for assignment before initiating a PR for any further contributions.


Typo in comments of 0.5-dev

I find the commands in 0.5-dev uses hyphen to join words. However here uses underscore.
E.g. detect_content is used instead of detect-content.

Good catch, thank you.


retrieve 'latest' from another pipeline

Just so other don't block on that. If wanting to download the latest artifact from another pipeline, option "definition" not listed here has to be used. Option corresponds to the ID of the initial pipeline builder.
Example :
      - task: DownloadPipelineArtifact@1
        displayName: 'Download some artifact files'
        inputs:
          buildType: 'specific'
          project: 'f36b9048-dabc-4752-8626-cc67cd5a2a4f'
          artifactName: 'test-pipe'
          buildVersionToDownload: 'latest'
          definition: '1'
          downloadPath: $(System.DefaultWorkingDirectory)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b03d67d0-8738-72e6-dbe7-324c1784d4fc
Version Independent ID: bbb4d774-cd98-3ecb-7ccf-5f49e6f2b425
Content: Download Pipeline Artifact task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/download-pipeline-artifact.md
Product: devops
Technology: devops-cicd
GitHub Login: @willsmythe
Microsoft Alias: wismyth

Thank you rusher!!  Doc folks, can we get this updated please?

Also, this documentation needs to be updated for version 2 of this task.

@rusher, thank you for the feedback! I updated the task information a few days ago and am pushing an update today to cover the examples including this scenario here. 'definition' is an alias still available to use but is now named 'pipeline'. @randyram-at-ms this will be up to the latest version of the task. Thank you both again and feel free to reach out! (Leaving open until doc is pushed out)

@rusher @randyram-at-ms the doc is updated and live :) feel free to take a look! Closing this issue.

I just checked out the latest at https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/utility/download-pipeline-artifact?view=azure-devops, It’s awesome, thanks!!
Can you document the preferTriggeringPipeline parameter?
From: Chase Wilson<EMAIL_ADDRESS>Sent: Tuesday, June 25, 2019 11:19 AM
To: MicrosoftDocs/vsts-docs<EMAIL_ADDRESS>Cc: Randy Ramig<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/vsts-docs] retrieve 'latest' from another pipeline (#4530)
@rusherhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frusher&data=02|01|randyram%40exchange.microsoft.com|39ccaa17d8b74cf9b36808d6f9998a0c|72f988bf86f141af91ab2d7cd011db47|1|0|636970835173280792&sdata=x4JwEKOnyjVfXxgD1ent7L%2BOnxNv6lnzHRudp26GYEM%3D&reserved=0 @randyram-at-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frandyram-at-ms&data=02|01|randyram%40exchange.microsoft.com|39ccaa17d8b74cf9b36808d6f9998a0c|72f988bf86f141af91ab2d7cd011db47|1|0|636970835173290781&sdata=jE04B3eQWGqVCK90BV5A0%2B20dSP%2BgLCtCJxyavkWSVM%3D&reserved=0 the doc is updated and live :) feel free to take a look! Closing this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fvsts-docs%2Fissues%2F4530%3Femail_source%3Dnotifications%26email_token%3DAMFCZSJKMS5S62HOJ6USDWTP4JOPXA5CNFSM4HYUDE72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYREJUQ%23issuecomment-505562322&data=02|01|randyram%40exchange.microsoft.com|39ccaa17d8b74cf9b36808d6f9998a0c|72f988bf86f141af91ab2d7cd011db47|1|0|636970835173290781&sdata=gdHoQF0GqpLWfZwKJq6g71vNTa1PbQs1qqB%2Fw1E%2FkFM%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMFCZSO2C4UO3F4KJVCTXOLP4JOPXANCNFSM4HYUDE7Q&data=02|01|randyram%40exchange.microsoft.com|39ccaa17d8b74cf9b36808d6f9998a0c|72f988bf86f141af91ab2d7cd011db47|1|0|636970835173300776&sdata=XHsqAip0QR7tVfgw0HH7NV90zjqierGs0E1W7e9iqp0%3D&reserved=0.


Should I include single-header after my to_json and from_json custom functions declaration? Why not?

So I was going through the code and can not get how the library find user-defined functions for conversion (to_json and from_json)?
Let me explain with code.
simple example could look like this:
#include <json.hpp>

namespace mynamespace
{
    class myclass{};

    void to_json(json&, const myclass&);
    void from_json(const json&, myclass&);
}

// if I put it there then I understand how it finds my to_json and from_json
// #include <json.hpp>

void func()
{
    mynamespace::myclass val;
    json j = val;
    val = j.get<mynamespace::myclass>();
}

Before include of <json.hpp> there is no declaration of mynamespace::to/from_json so I expect any check in <json.hpp> like has_custom_to_json to fail. If not then how it works?
My guess is that all the magic is in  is_detected_exact  but I could not figure out how is it possible and what template trick is used?
P.S.: Sorry for multiple edits.

It's using ADL. The internals are from @theodelrieu - maybe they can share more insights.

Any comments from @theodelrieu ?

Hello, this is because of two-phase lookup, you can find a short but accurate overview here.
Basically, val = j.get<mynamespace::myclass>(); is the point of instantiation, i.e. the lookup's second phase. At that point, your from_json method is visible to the compiler, whereas it wasn't in the first phase.

@theodelrieu Is this behaviour required by C++ standard (which one?) or there could be some other approach used by compiler? As I see from RSDN and SO discussions many compilers fail to implement two-phase lookup properly, so I assume this library would not work in such case,

Two-phase lookup was always the expectation, even pre-C++11. Some compilers (eg, MSVC) didn't do it quite right but at this point, I would expect all the C++14 and later compilers to definitely have it down.


$group gives unexpected results when _id is defined as an object

FerretDB version
1.6.0
Backend
PostgreSQL
Environment
any
What did you do?
db.pampam.insertOne({"_id": 1})
db.pampam.aggregate([ {$group: {"_id": {"foo": "$_id"}}}])

See also #3088 for a bit more advanced example.
What did you expect to see?
[ { _id: { foo: 1 } } ]

What did you see instead?
[ { _id: { foo: '$_id' } } ]

Duplicate of #2165


Selects: Width is not correct

Describe the bug
It seems like the select field is not the default 250px width as the other fields are.
To Reproduce
Steps to reproduce the behavior:

Go to https://denali-design.github.io/denali-css/docs/#selects
See the first example

Expected behavior
Width should be 250px
Screenshots

Additional context
The width on .input element seems to be set to 100px when it should be 250px. Double-check this is not because of the theme.

Select input is already 250px.

Looks like this was fixed in an update. Thank you for pointing this out.


As a Open Data Hub Customer Care member I want the Stuart web interface online, to start using Stuart as helper to elaborate the tickets

Stuart is now live in testing https://stuart.testingmachine.eu
You all should have received the access credentials trough e-mail, if not please contact me.

@sseppi now it is also in passbolt

@dulvui super! thank you!

Now also available under https://stuart.opendatahub.com


Fix the y-axis to ensure that the 3 zones are visible

Testing a potential solution for #609
This solution is simply force the maximum value for the yAxis to be 2 * medium zone upper limit. This will make the upper limit of the medium zone to be always in the middle. Thoughts?

I added stories to simulate these 3 scenarios, Rt is too low, too high, and the case where it covers each region.
Notes

I need to fix the alignment for the current value for Rt when it's too close to the x-axis, will work on that now.
We might want to add a label to indicate that Rt goes well above the top y-axis, although it might not be necessary.
cc @igorkofman @mikelehen

I suspect this isn't exactly what we want, especially if we stop forcing the y-axis to start at 0 (the medium region will no longer end up in the middle necessarily).
Per Josh's feedback it sounds like maybe we want to move towards having the axes be a bit more static instead of dynamically determined based on the data.  Like maybe we need to be able to specify either specific limits for the yAxis or at least specific min/maxes for them.

How about we specify the lower and upper limits for each one of the zones here https://github.com/covid-projections/covid-projections/blob/develop/src/enums/zones.ts so the configuration of that value is centralized? If the boundaries of the charts are static, that would be the right place to put the values.
I can update this PR to do that, can @mikelehen confirm if that would work?

@pnavarrc Sounds good, let's give it a try.

Thanks for putting this together!  I didn't realize it would require quite so much code change. 😬
I kinda' think we should put a pin in this, since offhand I am guessing this needs more iteration still, but I'm not sure exactly where we're headed.  Just browsing https://covid-projections-git-pablo-fix-zone-chart-extreme-values.covidactnow.now.sh/ I notice that:

The test positive label for WA (https://covid-projections-git-pablo-fix-zone-chart-extreme-values.covidactnow.now.sh/us/wa) is falling out of view:

We end up with a lot of empty space in the charts, which I'm not really a fan of.

Are there any charts on http://staging.covidactnow.org/ that we think are really problematic right now?  Else, maybe we just leave things as-is and revisit?  I think working on the tooltips or the new chart for future projections would probably be a better use of your time. (Thanks / sorry!)

FYI- I merged develop in so we get the smoothed out data, and I tweaked the zone limits (start hospital usage at 60%, cap test positive at 100%) so that we get a better idea of what this looks like with real data.

We are leaving the charts as they are for now - closing!


Update char.md

Explain what happens with padding functions when original string has more than count characters.

Thanks!


Add property to get local endpoint of discovered NAT device

I added properties to get the local endpoint of NAT device and the IP address of the client which connected to the NAT device.
This is related to #37 .
I have added two properties as below to NatDevice class and implemented overrides to child classes.

HostEndPoint: the local endpoint of NAT device
LocalAddress:  the IP address of the client which connected to the NAT device

UpnpNatDevice class
I used UpnpNatDeviceInfo.

HostEndPoint: UpnpNatDeviceInfo.HostEndPoint
LocalAddress: UpnpNatDeviceInfo.LocalAddress

PmpNatDevice class
The meaning of LocalAddress property in PmpNatDevice class was same as the meaning of HostEndPoint.Address property in UpnpNatDeviceInfo class, so I changed LocalAddress property to HostEndPoint property and add new LocalAddress property.
I set the port of HostEndPoint as PmpConstants.ServerPort because old LocalAddress was used that way in other parts of codes.

This PR will not compile for c# 3.0 because the expression bodied members were introduced much much recently. I think it could be a good idea to migrate the projects to dotnet 3.1 and then introduce changes like these ones.

This PR will not compile for c# 3.0 because the expression bodied members were introduced much much recently. I think it could be a good idea to migrate the projects to dotnet 3.1 and then introduce changes like these ones.

Thank you for checking my PR.
I have changed codes to be compilable for C# 3.0.

Thank you for checking my PR.
I have changed codes to be compilable for C# 3.0.

I'm sorry. I forget to unify indentation.
I fixed indentation to tabs.

I'm sorry. I forget to unify indentation.
I fixed indentation to tabs.


Opacity slider for minimap

Like in the title

tricky to do in the current implementation, for two reasons;
First - we're using a Window to draw with, using the standard WindowStack system. I can't set opacity in there. We'd have to use ImmediateWindows I think (that's what floatmenus use).
Second - the minimap is really a series of textures for each overlay. Messing with 'overall' opacity levels would give some odd stacking results.

Maybe no need for 'overall' opacity levels, just the base terrain coloring. That way we can set all textures' opacity without regard for the stacking.

It should be possible to do a final composite texture without opacity and then render that individually with the opacity setting.  There are options to disable drawing the shadow and background for windows, this should be entirely doable with a couple minor changes.


Querystring redirection to a specific version

Is your feature request related to a problem? Please describe.
I'm always frustrated when I want to share a URL about a .Net Framework/.Net Core related topic because visiting one of those sites for example
https://docs.microsoft.com/en-us/dotnet/api/system.int32
Will always redirect to page with a querystring attached of the latest version at the moment I visit it, so today I will get this
https://docs.microsoft.com/en-us/dotnet/api/system.int32?view=netcore-3.1
In 5 days I will most likely get this
https://docs.microsoft.com/en-us/dotnet/api/system.int32?view=net-5.0
Basically everytime I don't want to share the URL with a specific version and would like to always share a URL that will show the latest version by default, so every freaking time I go and remove the querystring.
I am facing this problem quite often since I am more or less active on Stackoverflow and answer question and share the docs.micorosft.com links every now and than, and always when I do I remove the querystring.
Also when other users don't remove it and I follow an outdated page not looking that I am on an outdated page is frustrating aswell.
Describe the solution you'd like
The solution I would like to see to either not default redirect to the querystring attached solution or introduce a querystring for the latest version, so either one of those
https://docs.microsoft.com/en-us/dotnet/api/system.int32 (just don't include the undesired querystring at all)
https://docs.microsoft.com/en-us/dotnet/api/system.int32?view=latest (if you insist on doing a querystring, introduce a latest)
Describe alternatives you've considered
The only alternative I have found, which I described earlier is annoying.
Additional context
No additional context to share.

And there we go as predicted, the url
https://docs.microsoft.com/en-us/dotnet/api/system.int32
will now redirect to
https://docs.microsoft.com/en-us/dotnet/api/system.int32?view=net-5.0

One of our team members will get back to you shortly with additional information.

Any information and/or opinion on this topic?


Allow boolean values to be represented by numbers

Servers commonly represent boolean values as numbers even though this is
clearly wrong and they should be sending true/false.
Anyway, we can make it easier to deal with this by allowing numbers to
represent boolean values as well. This essentially fixes compatibility
with earlier versions of Argo, which did this anyway.
Fixes #358

:+1: do we have a test that can demo this? and possibly a number that fails (if that exists)

Test should be straightforward, just parse this JSON, both as true:
{
  "realBool": true,
  "numberBool": 1
}

As for failure, I can't really think of one for the number case, as any number this isn't 0 (false) will be cast to Bool as true. Aside from overflows, which can't be parsed into a number in the first place.

Added a success test and fixed the build by writing code that actually compiles (which seemed useful)


TextBox IsSpellCheckEnabled breaks after WinAppSdk 1.0

Describe the bug
After upgrading from 0.8.7-xxx to Windows App SDK 1.0, I have observed that my application with a spellcheck-enabled TextBox always ends up behaving as if IsSpellCheck = false after some time (i.e. it starts up with spellcheck support correctly performing auto replacements of typos and flagging misspelled words, but after leaving the app running overnight, the next day the text box is no longer doing spell check).
Steps to reproduce the bug
Create a Win32 WinUI 3 app w/ Windows App SDK 1.0, and add a <TextBox IsSpellCheckEnabled="true" /> UI element. Leave the app running overnight, let the system sleep or whatever, then randomly check in on it.
Expected behavior
Spell check should continue to work.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.0
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 10 (1909): Build 18363
Additional context
I've long reported that IsSpellCheckEnabled = true causes (handled) unmanaged Win32 exceptions. After upgrading to Windows App SDK 1.0, I had to disable unmanaged code debugging because the textboxes would trigger so many more spellcheck-related exceptions from time-to-time. I presume they are related and will provide some stack traces.

@codendone and @MikeHillberg FYI


Add powering support to CCZ and CCX

Fixes https://github.com/quantumlib/Cirq/issues/546 (by obsoleting it)

Automerge: started

Automerge done.


gradient test fixed.

Rewrote the gradient test.

passes in bot h float32 and float64
removed xfail

Looks great, many thanks. Merging...