seg fault in pio_syncfile

I have a case which runs on 4 cores with 4 pio tasks and blows up with 1 or 2 pio tasks. It's dying in the call to pio_syncfile. I get pages of this sort of output (see below), then the seg fault. The first active pio process is the one that complains from inside memcpy. Here's the stack trace:
 memcpy,                              FP=7fff8065b560
 ADIOI_NFS_WriteStrided,              FP=7fff8065b6f0
 ADIOI_GEN_WriteStridedColl,          FP=7fff8065bb00
 MPIOI_File_write_all,                FP=7fff8065bb70
 mca_io_romio_dist_MPI_File_write_at_all, FP=7fff8065bb90
 PMPI_File_write_at_all,              FP=7fff8065bbd0
 ncmpii_mgetput,                      FP=7fff8065bca0
 ncmpii_req_aggregation,              FP=7fff8065bdc0
 ncmpii_wait_getput,                  FP=7fff8065be30
 ncmpii_wait,                         FP=7fff8065bf30
 ncmpi_wait_all,                      FP=7fff8065bf70
 flush_output_buffer,                 FP=7fff8067bfb0
 PIOc_write_darray_multi,             FP=7fff8067c0b0
 flush_buffer,                        FP=7fff8067c0e0
 PIOc_sync,                           FP=7fff8067c130

mod_pio`netcdf_sync,                 FP=7fff8067c170
GPTLstop: GPTLinitialize has not been called
GPTLstart name=PIO:write_darray_multi_nc: GPTLinitialize has not been called
GPTLstop: GPTLinitialize has not been called
GPTLstart name=PIO:flush_output_buffer: GPTLinitialize has not been called
/archive/u1/uaf/kate/src/parallelio/src/clib/pio_darray.c 1362 2
/archive/u1/uaf/kate/src/parallelio/src/clib/pio_darray.c 1366 2
GPTLstop: GPTLinitialize has not been called
GPTLstart name=PIO:write_darray_multi_nc: GPTLinitialize has not been called
GPTLstop: GPTLinitialize has not been called
GPTLstart name=PIO:flush_output_buffer: GPTLinitialize has not been called
GPTLstop: GPTLinitialize has not been called
GPTLstart name=PIO:rearrange_comp2io: GPTLinitialize has not been called
mpirun noticed that process rank 1 with PID 150416 on node pacman3 exited on signal 11 (Segmentation fault).
Currently Loaded Modulefiles:

ncl/6.1.0                 10) proj/4.8.0.gnu-4.7.3
git/2.3.0                 11) pyngl/1.4.0
nco/4.3.1.gnu-4.7.3       12) totalview/8.12.0-0
gcc/4.7.3                 13) ncview/2.1.2
openmpi-gnu-4.7.3/1.4.3   14) matlab/R2014a
PrgEnv-gnu/4.7.3          15) hdf5/1.8.10-p1.gnu-4.7.3
proj/4.9.1.gnu-4.7.3      16) wgrib2/1.9.6a
gdal/1.10.0               17) jdk/1.8.0
python/2.7.4              18) panoply/4.0.4

pnetcdf is 1.6.1

Hi Kate,
We will be happy to look into this. Could you provide us with a case description for when you get this error? What kind of machine are you running on?
Thanks!

Hi Kate,
I'm trying to reproduce this on Yellowstone. I've been running on a Penguin
Computing Linux machine at ARSC.
Kate
On Thu, Sep 24, 2015 at 10:07 AM, Kate Thayer-Calder <
<EMAIL_ADDRESS>wrote:

Hi Kate,
We will be happy to look into this. Could you provide us with a case
description for when you get this error? What kind of machine are you
running on?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143007167
.

If I try "poe -np 4 ./oceanM ocean_solition.in", it asks for the name of
the command to run and I give it "oceanM ocean_soliton.in", but the oceanM
executable doesn't see the ocean_soliton.in argument (it reads a filename
there).
Kate
On Thu, Sep 24, 2015 at 10:33 AM, Katherine Hedstrom<EMAIL_ADDRESS>wrote:

Hi Kate,
I'm trying to reproduce this on Yellowstone. I've been running on a
Penguin Computing Linux machine at ARSC.
Kate
On Thu, Sep 24, 2015 at 10:07 AM, Kate Thayer-Calder <
<EMAIL_ADDRESS>wrote:

Hi Kate,
We will be happy to look into this. Could you provide us with a case
description for when you get this error? What kind of machine are you
running on?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143007167
.

Hi Kate,
It runs on Yellowstone, no problem.
Kate
On Thu, Sep 24, 2015 at 10:33 AM, Katherine Hedstrom<EMAIL_ADDRESS>wrote:

Hi Kate,
I'm trying to reproduce this on Yellowstone. I've been running on a
Penguin Computing Linux machine at ARSC.
Kate
On Thu, Sep 24, 2015 at 10:07 AM, Kate Thayer-Calder <
<EMAIL_ADDRESS>wrote:

Hi Kate,
We will be happy to look into this. Could you provide us with a case
description for when you get this error? What kind of machine are you
running on?
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143007167
.

To run on caldera in an interactive session you use
mpirun.lsf ./oceanM ...

If I try "poe -np 4 ./oceanM ocean_solition.in", it asks for the name of
the command to run and I give it "oceanM ocean_soliton.in", but the oceanM
executable doesn't see the ocean_soliton.in argument (it reads a filename
there).
Kate
On Thu, Sep 24, 2015 at 10:33 AM, Katherine Hedstrom <
<EMAIL_ADDRESS>wrote:

Hi Kate,
I'm trying to reproduce this on Yellowstone. I've been running on a
Penguin Computing Linux machine at ARSC.
Kate
On Thu, Sep 24, 2015 at 10:07 AM, Kate Thayer-Calder <
<EMAIL_ADDRESS>wrote:

Hi Kate,
We will be happy to look into this. Could you provide us with a case
description for when you get this error? What kind of machine are you
running on?
Thanks!
—
Reply to this email directly or view it on GitHub
<
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143007167>
.

—
Reply to this email directly or view it on GitHub
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143019048
.

--
Jim Edwards
CESM Software Engineer
National Center for Atmospheric Research
Boulder, CO

Hi Jim and Kate,
Since ifort was working for me on Yellowstone, I built everything with the
intel compilers on pacman. It's also dying in a call to pio_syncfile:
[pacman9:483725] *** An error occurred in MPI_Alltoallw
[pacman9:483725] *** reported by process [2399141889,2]
[pacman9:483725] *** on communicator MPI COMMUNICATOR 4 SPLIT FROM 0
[pacman9:483725] *** MPI_ERR_TYPE: invalid datatype
[pacman9:483725] *** MPI_ERRORS_ARE_FATAL (processes in this communicator
will now abort,
[pacman9:483725] ***    and potentially your MPI job)
[pacman9:483721] 3 more processes have sent help message
help-mpi-errors.txt / mpi_errors_are_fatal
[pacman9:483721] Set MCA parameter "orte_base_help_aggregate" to 0 to see
all help / error messages
Kate
On Thu, Sep 24, 2015 at 1:43 PM, jedwards4b<EMAIL_ADDRESS>wrote:

To run on caldera in an interactive session you use
mpirun.lsf ./oceanM ...

If I try "poe -np 4 ./oceanM ocean_solition.in", it asks for the name of
the command to run and I give it "oceanM ocean_soliton.in", but the
oceanM
executable doesn't see the ocean_soliton.in argument (it reads a
filename
there).
Kate
On Thu, Sep 24, 2015 at 10:33 AM, Katherine Hedstrom <
<EMAIL_ADDRESS>wrote:

Hi Kate,
I'm trying to reproduce this on Yellowstone. I've been running on a
Penguin Computing Linux machine at ARSC.
Kate
On Thu, Sep 24, 2015 at 10:07 AM, Kate Thayer-Calder <
<EMAIL_ADDRESS>wrote:

Hi Kate,
We will be happy to look into this. Could you provide us with a case
description for when you get this error? What kind of machine are you
running on?
Thanks!
—
Reply to this email directly or view it on GitHub
<

https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143007167>

.

—
Reply to this email directly or view it on GitHub
<
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143019048>
.

--
Jim Edwards
CESM Software Engineer
National Center for Atmospheric Research
Boulder, CO
—
Reply to this email directly or view it on GitHub
https://github.com/PARALLELIO/ParallelIO/issues/14#issuecomment-143058456
.

Hi Kate,
We have a local linux cluster where I'd like to try to reproduce this problem, but I need a little more information. How are you configuring PIO to start? When you say you are going from 4 tasks to 2, how are you making that change? Any information you can give me about how best to try to reproduce this error would be very helpful. If you have code that you can tar up and send to me, that would be REALLY great, but I understand that can be difficult. My email is<EMAIL_ADDRESS>if you want to directly contact me.
Thanks!
Kate

Update: It looks like some of the issues were caused by an incorrect configuration:
{{{
! PIO number of I/O tasks
 NIOTASKS = 1

PIO_STRIDE = 4
}}}
This caused a runtime seg fault but the reverse (PIO_STRIDE=1, NIOTASKS=4) worked. The question remains as to whether we want to support a configuration as above, which could, in theory, be done. However, it is not a high priority currently as this is not a common configuration. Issue will be moved to JIRA as a feature request.


Preffered Aspect Ratio on crop not working properly.

Hello.
I am trying to implement square crop. I have the following code in my void initState()
_controller
      ..initialize().then((_) {
        _controller.preferredCropAspectRatio = 1;
        _controller.updateCrop();
        _controller.video.play();
      });

This is the result
If I move the crop grid, it resizes properly:
I am using video_editor 1.1.0
Thanks in advance for any suggestions.

I think it doesn't work because _controller.updateCrop() doesn't notifyListeners. Try this and tell me if it works.
_controller.preferredCropAspectRatio = 1;
_controller.updateCrop();
_controller
    ..initialize().then((_) {
      _controller.video.play();
    });

Hello @seel-channel,
That way it doesn't work at all.


How to Unlock Samsung Galaxy S7 Edge forgotten password

Here you can use the best method how to unlock Samsung galaxy s7 edge by unlock Samsung galaxy s7 edge code generator available for downloading on this page bellow. Here you have all information and instructions how to complete the unlocking Samsung galaxy s7 edge process successfully without any problems.
Unlock Samsung Tool is the best software for Android devices. It can easily deal with many problems such as locked screen, data extraction, data backup ＆ restore and data recovery. But the removal locked screen function just support Samsung device currently.
MORE INFO:
Unlock the Screen Lock on Samsung
How to Remove Samsung Galaxy Note 4 Locked Screen Without Password

Samsung Lock Screen Removal can helps one click to unlock Samsung Galaxy S7 without losing data. Besides, you can use this tool to recover data from Samsung with broken screen.

Delete this spam

Samsung Lock Screen Removal, which is an all-in-one data recovery tool for almost all Samsung Galaxy devices. With the help of this program, you can not only restore the lost and deleted content like photos, videos, contacts, call logs, messages, WhatsApp chats and more from your Samsung Galaxy device including Samsung Galaxy S/Note/A/J/ON/Tab, but also one click to fix a bricked Samsung and unlock your Samsung Galaxy S/Note/Tab without losing data. So, you can easily Bypass Samsung Lock Screen Passcode, Pin Code, Pattern and Fingerprint by using this tool.
Video Guide:

very good and also I can suggest some batter ways for your Smartphone to unlock and use every time and free off cost.
SIM unlock  for samsung-galaxy-s7


User Telemetry & APM with Datadog (Intergen)

Ready for another overview around some extra tracing in the controllers.

@jonocairns awesome job !!
Can you just apply my last comment about the ASM / ARM mode script and then it will be ready for the merge :)
Thanks

Using the ARM mode is not about to replace just one word ;)
Please review the all procedure to deploy the webapp using the AZURE CLI with the ARM mode.
Thanks

@jonocairns One last effort and we should be good :)
Thanks

Thanks :) !


GUI: Support ordering of common source JavaScript files

This is for the Start panel.  The order in which we load JavaScript files can matter a lot...

The reordering widget should be similar to the Object Graphs controls, with "up" and "down" buttons having SVG icons, and placed at the end of each row, like https://github.com/ajvincent/es7-membrane/blame/master/docs/lab/index.html#L122-L124 but not inside a HTML table.  CSS grids are more appropriate.

Fixed, finally!


Sort uploads list by release date instead of last modified

Is your feature request related to a problem? Please describe.
I've noticed that when I upload videos, the videos are not sorted by the release date.  They are, instead, sorted by date of last modification.  Which means that if I edit a video's metadata, it places it at the top of the list of uploads.  I'm currently working on reuploading an entire channel which we've posted to other platforms, and I'm having to do it in strict chronological order so the videos don't show up in a different order on Odysee than on Rumble.
Describe the solution you'd like
I would like to see the Uploads list sorted by "release date" instead of the last modified time.
Describe alternatives you've considered
A dropdown for how to sort, with the default being "release date", would also work.

I should note that you can get the behavior I'm asking for if you enable filters.  But the default behavior is not to sort that way, which runs contrary to other platforms.  A sort dropdown that's there even if the user doesn't enable filters would work if the default was also in line with user expectations from other platforms.

Hey there, thanks for the feedback.  We're glad there's a way to access that feature if needed, we'll see if we can make this easier to access/understand for that particular case.  @keikari any thoughts?

Not sure what to do here to improve it. The claim_list only has sorting for 'name', 'height' and 'amount', so would need changes in lbrynet to make it default. (May or may not be doable by me)
The "Enable filter" and settings in there are sticky, so you should only need to change it once per login.

Yea, I recall the limitation.
Maybe just let's rename to that "Enable Filters and Sorting" -  that should give a better hint to people. Maybe add this setting to wallet preferences?

@tzarebczan @keikari Thanks for taking a look at this so quickly and getting back to me!
The problem that I'm seeing is that, if someone is logged out and is just looking at the channel casually, the default is not to sort by publication date like other platforms.  Logging in and setting a preference does sort of solve that for the subset of users who are logged in, but it doesn't cover the substantial portion of the potential user base who are not logged in.
Changing it to "Enable Filters and Sorting" would definitely be an improvement.  It still requires at least one extra user interaction to make Odysee behave more in line with what they'd be used to from other platforms, but it would be a little clearer.
Is there a way to possibly apply the sorting mechanics from having filters enabled as the default for when filters are not enabled?

Enabling it by default is bit more tricky, as way the things currently work it would have performance issues on larger accounts.
But it should be possible to permanently save the setting once it's changed, and can update the label.
I don't quite understand the issue while being logged out? That page is only accessible if one has logged in.

@keikari My apologies.  I thought the sorting of Uploads was the same as the sorting of the channel content pages when sorted by New.  I just ran another test, and that appears not to be the case.  The public channel pages do appear to sort correctly.  So the entire premise of this is kind of moot.
I do still like @tzarebczan's suggestion of renaming the button to "Enable Filters and Sorting", but my initial problem is solved by the realization of my faulty assumption, so feel free to close this or leave it open until the button's clarified.  Either way, I'm all set, and I'm sorry for bothering you with this.


Jobs says running forever in cases where the job fails to run

The heat equation demo is broken -- we are fixing that
But, the GUI should also be able to detect that. In this case the heat equation executable throughs an exception at runtime due to (i think) a bad schema. The GUI just marks the job as running forever -- we should be able to track that and show the error code, output, etc.

Turns out the heat equation exe was not broken (there were errors in the schema but they were not the source of the problem).
The real problem was that the heat equation was writing so much data that it was breaking the POpen Buffered Reader that was being assigned to the job when we started it.
The fix was to change the build script so that it writes the stdout to file (using > filename at the end of the command).
The ensures the stdout captured by popen is small. Once the job is finished we read the stdout file and use that to populate the GUI..
Bonus points: We can now update the gui stdout whenever we want without worrying -- we just read that file -- I will do that on monday and close this issue after that is implemented

Done


Management Groups diagnosticSettings 500 internalservererror on PUT

Description
I'm trying to send Management Group activity logs to a Log Analytics Workspace, following this info:

https://learn.microsoft.com/en-us/rest/api/monitor/diagnostic-settings/create-or-update?tabs=HTTP
https://learn.microsoft.com/en-us/rest/api/monitor/management-group-diagnostic-settings/create-or-update?tabs=HTTP
https://learn.microsoft.com/en-us/azure/azure-resource-manager/troubleshooting/common-deployment-errors
https://sameeraman.github.io/blog/2021/06/20/mgauditlogs/

I get StatusCode 500 "InternalServerError", returned header x-ms-failure-cause says service.
Happens both when using Az PowerShell module like so:
Invoke-AzRestMethod -Method 'PUT' -Path (
    'providers/microsoft.management/managementGroups/{0}/providers/microsoft.insights/diagnosticSettings/{1}?api-version=2020-01-01-preview' -f
        $Mg.'Name',
        $Log.'Name'
) -Payload $Body

And API endpoint directly like so:
$Headers = [ordered]@{
    'Authorization' = [string] 'Bearer {0}' -f (Get-AzAccessToken -ResourceTypeName 'Arm').'Token'
}
Invoke-RestMethod -Method 'Put' -Headers $Headers -Body $Body -ContentType 'application/json' -Uri (
    'https://management.azure.com/providers/microsoft.management/managementGroups/{0}/providers/microsoft.insights/diagnosticSettings/{1}?api-version=2020-01-01-preview' -f
        $Mg.'Name',
        $Log.'Name'
)

With a body / payload like so:
$Body = [string](
    [ordered]@{
        'properties' = [ordered]@{
            'workspaceId' = [string] $Log.'ResourceId'
        }
        'logs' = [array](
            [ordered]@{
                'category' = [string] 'Administrative'
                'enabled'  = [bool] $true
            },
            [ordered]@{
                'category' = [string] 'Policy'
                'enabled'  = [bool] $true
            }
        )
    } | ConvertTo-Json -Compress -Depth 3
)

Reproduce

PIM / add RBAC "Log Analytics Contributor" on both the Management Group in question, and the target Log Analytics Workspace.
Authenticate with PowerShell.
Find Log Analytics Workspace full resource ID, and Management Group name/id, either in the Azure portal, or with Az PowerShell.
Get info on them (so above code works) like so:

$Mg = Get-AzManagementGroup -GroupId '<management_group_name>' -WarningAction 'SilentlyContinue'
$Log = Get-AzResource -ExpandProperties -ResourceId '<log_analytics_full_resource_id>'

Run above code.

What I've tried for troubleshooting

Refresh auth/credentials.

Clear-AzContext -Force
Connect-AzAccount

Force TLS 1.2 only.

[System.Net.ServicePointManager]::SecurityProtocol = [System.Net.SecurityProtocolType]::Tls12

Questions

Response and doc on deployment errors points to a backend issue, rather than me goofing. Any known errors with mentioned endpoint?
Any obvious goofs in above logic / PowerShell? It should work according to the linked API endpoint doc and blog post as far as I can see.
Can "InternalServerError" / 500 also mean that I've done something wrong? If yes, this doc should be updated to reflect it.

Adding Service team to look into this.

I just found out that this was caused of me goofing with the $Body, by not putting logs inside properties.
Was:
$Body = [string](
    [ordered]@{
        'location' = [string] 'global'
        'properties' = [ordered]@{
            'workspaceId' = [string] $Log.'ResourceId'
        }
        'logs' = [array](
            [ordered]@{
                'category' = [string] 'Administrative'
                'enabled'  = [bool] $true
            },
            [ordered]@{
                'category' = [string] 'Policy'
                'enabled'  = [bool] $true
            }
        )
    } | ConvertTo-Json -Depth 3 -Compress
)

Became:
$Body = [string](
    [ordered]@{
        'properties' = [ordered]@{
            'workspaceId' = [string] $Log.'ResourceId'
            'logs' = [array](
                [ordered]@{
                    'category' = [string] 'Administrative'
                    'enabled'  = [bool] $true
                },
                [ordered]@{
                    'category' = [string] 'Policy'
                    'enabled'  = [bool] $true
                }
            )
        }
    } | ConvertTo-Json -Depth 3 -Compress
)

And now it works. :)


fix: Authorization headers not showing on kulala.inspect

As the title says, it fixes the issue where the headers were not shown on kulala.inspect, which would make kulala.run not work inside the inspect buffer.
The problem was described at https://github.com/mistweaverco/kulala.nvim/pull/122

The Authorization header is only removed from the header table when it should be handled by Curl itself (like for BASIC-User-Password / NTLM ....) but for other cases where there is no additional handling needed (like BEARER or BASIC where the Username and Passwort was already Base64-encoded) it keeps in the header list and still must be forwarded to Curl as header.
So the solution needs to be a little bit different, I'll have a look at it in the next days.


Update shavian.info/spelling/ to include info t vs d

It would be nice if https://www.shavian.info/spelling/ included information on T's that become D in various situations.
Water
Italy (vs Italian)

Flap-t (/d/ instead of /t/) often happens between vowel sounds or after a vowel and before a liquid.

If the T is a flap-t, it is still written with a T letter in Shavian?

I believe it is still phonemically /t/. The change you speak about occurs in US and Australian English but not in some other varieties (e.g. RP). It is not phonemically significant, i.e. it does not change the meaning, so both sounds in US and Australian English are represented by 𐑑.

Wouldn’t it be equally as phonemically significant as the past tense d changing to t?

It's a little different. The past tense and plural sound changes are pretty universal across all varieties of English. And the distinctions with the past tense and plural can be phonemically important, e.g the difference between 'canned' and 'cant', 'hence' and 'hens', etc.


[BUG] Add the External link icon to External links

Describe the bug
On Your care team screen, Add the External link icon to External links
To Reproduce
Steps to reproduce the behavior:

Install Safer app
Complete the onboarding steps.
Verify the role as University Student
Complete the Covid onboarding steps and proceed to the Safer Home screen
Tap on "Your car team". Tap on the "More about the OSF Oncall connect program" arrow.

Actual Result
If the link is external please add the External link icon to it.
Expected behavior
The external link icon is missing
Screenshots
Smartphone (please complete the following information):

Device: [e.g. iPhone and Android]

Version [e.g. 2.10.22]

Additional context
Add any other context about the problem here.

Retested this issue. This is fixed in version 2.10.25


experimental manipulations

Hi all -
Is there any part of NIF that deals with experimental manipulations?
For example, common experimental manipulations in developing animals are monocular deprivation by lid suture, enucleation (removal of an eye), deafening, etc. These might have a value associated with them, such as date of onset or offset.
As an example, one might want to express the following:
right eye lid suture onset at postnatal day 12
right eye lid suture offset at postnatal day 21
left eye lid suture onset at postnatal day 21
right eye lid suture onset at postnatal day 26
(these aren't the best experiment, but just an example of some events that one might want to specify for an experimental animal)
We have a ton of these we'd like to express for our lab.
The closest I found are these clinical read codes but I don't think this is what we want:
https://bioportal.bioontology.org/ontologies/RCD?p=classes&conceptid=72170
Thanks
Steve

These would be covered as part of the methods ontology. http://ontology.neuinfo.org/trees/query/subClassOf/ilxtr:technique
For example, there is blinding technique http://uri.interlex.org/tgbugs/uris/indexes/ontologies/methods/177 (resolves to http://uri.interlex.org/base/ilx_0739724), but we don't have some of the other ones. If you have a full list of these we can add them.
You can capture this information generically use the techniques.
subject was-participant-in-performance-of protocol-performance-1
protocol-performance-1 employs-technique blinding
protocol-performance-1 start-date (some date)
protocol-performance-1 start-time (some time)

Thanks Tom
I have a couple more questions.
Here is a list of putative techniques:

Sensory deprivation technique
Description

Dark-rearing
The raising and housing of animals in a completely light-free environment

Eyelid suture
The closing of an eye by surgical suture

Eyelid suture removal
The removal of eyelid sutures, allowing natural eye opening

Retinal silencing
Removal of neural activity of the retina without damage to retinal cellular structures

Retinal enucleation
Surgical removal of the eye

Question 1: some of these techniques can be applied to either the right or left eye. Would we develop a separate "technique" for each eye, or apply it to, say, UBERON:0004548 ("left eye") somehow? How would that be coded?
Question 2: Is there a list of the attributes of protocols, that includes the words "was-participant-in-performance-of", "employs-technique", "start-date", "start-time", etc?
Thanks
Steve


Take care of db exceptions

Every time we processing again the logs, DB constraint violations throw exceptions.
This should be handled more gracefully.

contained in #238 If is fine for you, you can close this issue


add documentation of how to best split files in preparations

A user pointed out that it will be great to have some documentation of how to best split prep info files. This can be super helpful in specific for per_sample_FASTQ as they can be ordered any way the user wants.
cc: @johannesbjork

It will be good to add some of this information to the documentation:

If we imagine the worst case scenario, that we have in all our columns the default 'not applicable' NULL value (https://www.ebi.ac.uk/ena/about/missing-values-reporting), and that XML adds 4 times the size due to its formatting (how EBI-ENA expects the submission); then we will have len('not applicable') * 50 * 4 = 2800 (2.8K) per sample; then we can divide that by the 10M (max submission for EBI-ENA) so a preparation can not have than 3.5K samples. Note that the more metadata columns, the less samples you can submit in a single go.


Non UTF-8 boot log crashes decoder

Describe the bug
Command Name
az vm boot-diagnostics get-boot-log
Errors:
'utf-8' codec can't decode byte 0xff in position 20023: invalid start byte
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 654, in execute
    raise ex
cli/core/commands/__init__.py, ln 718, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
...
storage/v2018_11_09/blob/baseblobservice.py, ln 2166, in get_blob_to_stream
    stream.write(blob.content)
cli/command_modules/vm/custom.py, ln 1438, in write
    content = str_or_bytes.decode('utf8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 20023: invalid start byte

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Deploy URN audiocodes:mediantsessionbordercontroller:mediantvirtualsbcazure:latest
az vm boot-diagnostics get-boot-log -n {} -g {}

Expected Behavior
Boot log to be downloaded.
Environment Summary
Linux-4.19.128-microsoft-standard-x86_64-with-debian-bullseye-sid
Python 3.6.10
Installer: DEB

azure-cli 2.13.0

Extensions:
azure-devops 0.18.0

Additional Context

hi @qwordy could  you pls help to look at this?

The error indicates that it is not UTF-8 encoding. I will support more encodings to fix this issue.

Thanks @qwordy! Believe me, I was quite surprised to find some VMs give a non-UTF-8 boot log, but so it goes.

hi @qwordy is it ok to close the issue now? thanks

hi @qwordy is it ok to close the issue now? thanks

May be we can support more encodings.

Mark it as a feature request: support more encodings of logs

@qwordy I wouldn't categorize this as a feature request but as a bug fix, as this results in a crash when using this functionality of az against images available in the Azure marketplace.

@qwordy I wouldn't categorize this as a feature request but as a bug fix, as this results in a crash when using this functionality of az against images available in the Azure marketplace.

Yes. I also think it is a bug fix. The feature-request label has a long story. There is a similar issue https://github.com/Azure/azure-cli/issues/16454 recently. I will prioritize this issue since more than one person reported it.

Yes. I also think it is a bug fix. The feature-request label has a long story. There is a similar issue https://github.com/Azure/azure-cli/issues/16454 recently. I will prioritize this issue since more than one person reported it.


Deploy to github pages

Deploying to github pages would allow people to install this theme by adding one line to their custom css:
/* Forgotten Keep */
@import url('https://awish-senpai.github.io/ForgottenKeep/index.css')

Deploying to pages is super easy:

go to the the pages settings of this github repo

change Branch from None to Main
click Save

Vouch

Thanks for this. Changed the settings as mentioned by you :). Good day!


hello author,I have a problem can you help me ?

I've started TORCS， then run the code pyclient.py on pycharm
and the system have a mistake as follows
didn't get response from server...

You should run both the server and client in separate terminals


Foundation 6: component mixin feedback

Hello friends,
In this thread we'd like to get your feedback on how we use Sass in Foundation, specifically mixins.
Starting with Foundation 4 we went mixin crazy with components. Instead of writing components in plain CSS, we built a series of mixins for each one, and then applied those mixins to classes. We continued this approach for Foundation 5, and Sass will remain a big element of Foundation 6.
However! We have a feeling that our use of Sass has gotten away from us. As we train new designers and developers in Sass, we find that it can create bad habits. The most obvious one is nesting: Sass encourages you to do deep nesting to match the hierarchy of your HTML. For a framework, this is a bad thing, as we want to keep specificity low. That's why for Foundation 6 we're better scrutinizing the CSS output of our Sass.
We'd like the community's feedback on the utility of component mixins. To determine their utility, we need to figure out why people use them. Why do people use mixins?
Benefits
Semantics. This is the big one, but you could argue it doesn't apply to everything. medium-6 large-3 columns? Okay, yeah, that's not semantic. And grid elements can easily have upwards of five classes, so being able to condense all that logic into one class with a mixin is very useful. In Foundation 6 we've made grid mixins even more robust, so we can be more competitive with the likes of Neat and Susy.
Now what about the rest of the components? It would be harder to argue that they're unsemantic. button, menu-bar, label, etc. are all fairly semantic. They describe what the component is, not what it looks like. Modifier classes venture into the unsemantic, like the sizing classes for buttons, or alignment classes, but the core classes themselves are semantic.
With that in mind, what is the utility of the mixins for those components? .button is a semantic class, so how does being able to replace the class using @include button; improve how developers use the framework?
This hits on a second, but possibly far less used, utility of Sass mixins: the ability to change class names.
Class naming. With the mixins, you can use our components within your own class structure. We're in the process of refining ZURB's own internal coding style, which we'll share with our developers, but it doesn't follow any popular style guide.
So this is the other main utility of Sass mixins, but we need to know: do people actually use this? We use Foundation every day here at ZURB, and in our own client work we find ourselves using mixins for the grid quite a bit, but less so for other components. We need to hear from our community if this is a feature worth investing in.
Problems
Sass mixins make our codebase complex and potentially messy. You can find examples in the existing codebase of complicated and messy Sass. When is something a mixin? Which elements do you make parameters? When do you calculate a value automatically or let the user define it?
Also consider complicated components like Orbit, which has a container, slides, captions, arrows, and navigation dots. If we did things right, every class needs a mixin. And some of them have absurdly tiny mixins. For example, in Foundation 6, the Orbit container is just one line of CSS. So do we write a whole orbit-container() mixins just for that one line of CSS?
This also hints at another issue that can crop up when using Sass and creating a framework: too much abstraction. We're already cutting back on mixins in some places. A mixin is an abstraction, but it needs to be a useful abstraction to make it worth the developer's time to learn how it works.
Summary
What we want to know is if component mixins are useful. Do you use them? Which components do you use them with? Here's a summary of our thinking on the matter:

Mixins allow us to write semantic markup, but it's possible that we over-extended the concept into areas that are already semantic.
Mixins allow developers to throw out our class names and write their own, but that's a time-consming process that requires thorough understanding of how our mixins work. It also requires our mixins to be well-written and well-organized, which is a challenge.
Further, it's likely that a very small portion of our developers are actually using component mixins for this purpose.
Sass makes our codebase complicated and more abstract for developers. Cutting back on the use of Sass where we can reduces the amount of time developers spend learning the framework, allowing them to spend more time actually using the framework.

First off, let me day I'm a big fan of Foundation and use it on pretty much all my projects.
That said, the nesting in Foundation is a real sore point for me, often leading to chaining classes to ridiculous levels to override specificity. I'm really glad this is being considered as a key factor in version 6.
In terms of mixins, we use these for a few components. At my day job we use namespacing and BEM which has led us to creating our own button classes. As a quick win we have used the button mixin but it's quite a simple mixin so this is on our list of things to refactor to our own fully custom code.
Another big one we use regularly is the block grid. It's great to be able to create custom block grid layouts and use media queries to set the number of list items wide the grid should be at each breakpoint.
These are the main mixins I can think of that we're using right now, but will have a think if there's any others over forgotten.

@fredrivett We're with you on the specificity front. We're really looking closely at our CSS output to ensure our Sass is written as cleanly as possible.
Also, good feedback on the mixins. Thanks!

@fredrivett The grid, block grid, and buttons are really good examples of where mixins can be used well. The grid and block grid are very verbose components, so abstracting away all those classes using mixins is real nice. Me personally, I don't always use the semantic method for the grid, but I always use block grid mixins, because those classes are so wordy.
And buttons are a good example because, even though they're a component, they're a simple component with a lot of configuration. So it's easy to make mixins for those elementary components: the base, the size, the style, etc.
It gets weird with more complicated components. Like I said, Orbit has like ten UI elements, and some of those elements only have a few lines of CSS, making the mixin feel like an unnecessary abstraction.  We do appreciate critical feedback, this is not the place. We'll remove them here.

Yes man, i'm sorry, I tried to correct the other user, I found it very rude comment.

At least allow toggling of classes for every group, if not every variation.

Can we get mixins for visibility?
http://foundation.zurb.com/forum/posts/633-visibility-sass-mixin

@michaelBenin Yep, we've already implemented these in 6!

Thanks for the feedback, all! We've looked it over and are really rocking away at getting Foundation 6 out very soon!


Changes to Create Statistic Popup

From #419:

Miro reference: https://miro.com/app/board/o9J_kj6tycQ=/?moveToWidget=3074457366776975026&cot=14
“Which variable would you like to use?”

[ ] Only show variables selected on the Confirm Variables page.
[ ] Allow selection of only one variable at a time
[ ] Add text: “Would you like to add another variable?” or similar

Link text back to the “Confirm Variables” page

workflow for later: Add shortcut from “Confirm Variables” page back to “Create the statistics” page,

"How would you like missing values to be handled?”

[ ] Remove “Insert random value”

Debug checks

[ ] Make sure editing works as well as selecting variables, etc.
[ ] What happens if a statistic is created with “EyeHeight” but the user goes back to Confirm Variables and de-selects “EyeHeight”, etc.

[ ] send list of stats and available var types (@raprasad)

[ ] For "Enter fixed value", do not have a default value or change default depending on data type

Current default is 0 (zero) but shouldn't be if variable is categorical, etc.
(Ideal, if possible given the time): Display the variable's min/max or categories under the fixed value box


MongoConnector fails upon update with $unset parameter

While working on #159 ticket I thought there was an error with my edit python doc_manager script. However, I tried it with a completely new fresh install of mongo-connector with the original doc_manager and the failure happens anyway, so this is a new issue.
To repeat the failure:

I start the mongo-connector like this:
mongo-connector -m <IP_ADDRESS>:27017 -t http://localhost:8983/solr -d C:\Python27\Lib\site-packages\mongo_connector\doc_managers\solr_doc_manager.py -n myDB.myCOLL
I run in the mongo shell this update: db.myCOLL.update({},{$unset:{someField:""}},{multi:true}).

MongoConnector runs correctly until it is supposed to work with this last command. It throws an exception like:
Exception in thread Thread-2:
Traceback (most recent call last):
 File "C:\Python27\lib\threading.py", line 810, in __bootstrap_inner
    self.run()
 File "C:\Python27\lib\site-packages\mongo_connector\oplog_manager.py", line 233, in run
    docman.update(doc, entry.get('o', {}))
 File "C:\Python27\lib\site-packages\mongo_connector\doc_managers\__init__.py", line 24, in wrapped
     return f(*args, **kwargs)
  File "C:\Python27\Lib\site-packages\mongo_connector\doc_managers\solr_doc_manager.py", line 187,    in update
    updated = self.apply_update(doc, update_spec)
  File "C:\Python27\Lib\site-packages\mongo_connector\doc_managers\solr_doc_manager.py", line 170, in apply_update
    doc.pop(to_unset)
KeyError: u'someField'

I tried it on 1.2.1. It is working on the older version, so it must be a new introduced bug.

Hope commenting won't reopen it! I just wanted to say a very late thanks. The project moved it's focus for a long while to a different development issues and I finally found time to try it out and it seems to work just fine. Thank you very much for your efforts!


Add docker container for php:cli for tests (phpunit and performance)

And update .travis.yml with composer.json


Draft: NFT Balance + NFT Card

closes #398 and adds more features

pipeline failed bro 🙈

pipeline failed bro 🙈

Its a draft bro 👀


Error: could not find template when using an enabled switch

Describe the bug
I have a template yaml that uses an enabled switch in the values.yaml that determines if the template should be rendered or not:
{{- if .Values.security.authorizationPolicy.enabled }}
---
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: {{ .Values.serviceName }}
# This continues with more yaml
{{- end }}

The values.yaml has security.authorizationPolicy.enabled which is set to false when I don't want to include an authorization-policy resource and it is set to true when I do want that resource.
When the enabled switch is set to false I get a exit status 1; Error: could not find template templates/authorization-policy.yaml in chart.
I was actually wanting the rendering to fail if I set the enabled switch to disabled and then I could assert that the render failed or that it produced an empty string or something similar, but this error is sort of wrong? It would indicate to me that the yaml template doesn't exist within the chart at all.
To Reproduce
Here's the part of my code where I set the enabled switch option and then attempt to get the Helm output for the template:
options := &helm.Options{ SetValues: map[string]string{ "security.authorizationPolicy.enabled": "true" }}

output := helm.RenderTemplate(GinkgoT(), options, helmChartPath, "authorization-policy", []string{"templates/authorization-policy.yaml"})

I imagine that this could be reproduced with any kubernetes resource, you just need to have an enabled switch at the top of the yaml template.
Expected behavior
When I realised why this was erroring, it sort of made sense, clearly the yaml template is not rendered because the switch is not enabled, but the error message is confusing. I think it would be better if this error message stated that the Helm template was rendered successfully, but the output was nothing, or empty, or something else indicating that the render itself worked, but no output was produced.
In the helm helm-unittest plugin it is possible to perform this kind of assertion using the hasDocuments assertion (see assertion types here).
For now, I am just going to use helm.RenderTemplateE and assert the error, because I know for a fact that the yaml template is there and my other tests will prove this when I set the switch to enabled, but I do think that a new error should be created to more effectively state what is wrong with the test, or this functionality should be implemented that allows you to assert that no documents were generated in the Helm output.
Here is some code and terminal screenshots to help show what I am describing:
Context("enabled switch", func() {
		It("should not render when set to 'disabled'", func() {
			options := &helm.Options{ SetValues: map[string]string{ "security.authorizationPolicy.enabled": "true" }} # This value is what is modified

			output := helm.RenderTemplate(GinkgoT(), options, helmChartPath, "authorization-policy", []string{"templates/authorization-policy.yaml"})
			outputNoSourceString := regexp.ReplaceAllString(output, "")

			helm.UnmarshalK8SYaml(GinkgoT(), outputNoSourceString, &authorizationPolicy)
		})
	})

When the enabled switch is true:

When the enabled switch is false:

Nice to have

[x] Terminal output
[x] Screenshots

Versions

Terratest version: v0.43.11
Environment details (Apple M1, macOS Ventura 13.4.1):

Additional context
Add any other context about the problem here.

Hi @Mo0rBy, the issue seems to occur because the authorization-policy.yaml template is skipped entirely when the flag --set security.authorizationPolicy.enabled=false is used. This causes Helm to throw an error, as it interprets a template with no output as if the file does not exist or was not processed.
To address this, we can either remove the conditional check or include an else block with a placeholder comment, ensuring the template renders successfully. Since the error stems from Helm's behavior, it cannot be resolved by modifying Helm directly.

Closing this issue as there's no activity on this for a while. Feel free to reopen this if you continue to face similar issue.

Hi @james03160927, yes I believe this is more on the Helm side to address, rather than the TerraTest side.
I've managed to work around it a long time ago by understanding why the "error" is coming up, so I just adjusted my assertions to ensure that I was getting this particular error + some other assertions, meaning I can effectively test the enabled flag for certain parts of my chart works correctly.
I don't think the TerraTest team should address this as, again, it's part of how Helm itself behaves. However, does this maybe deserve some documentation? Or at least an example? I'm not sure as it's a niche scenario, but then again, the enabled switch is a very common piece of functionality within many Helm charts and I would argue that it is an "industry standard".
Anyway, I'll leave my ranting here and there's no need to re-open the issue. Thanks!


Add Missing Readme.md & README.md.gotmpl content

Description
Due to a large code difference in #44 compared to main (over 1000 lines code), the changes in puris/charts/puris/README.md & puris/charts/puris/charts/backendREADME.md.gotmpl got deleted.
Solution
Put changes in another PR.

solved by #60


Repo sync for protected CLA branch

The pull request is created from main637995515414394755sync_temp to main to fix git push error for protected CLA branch

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


don't recoganize built-in procs

why is there a warning regarding openInputfile and such ?
the extension was working perfectly but suddenly it acted like this!
even thought the code runs successfully
P.s. : when i disable the extension the warning is gone

Hi,
AsmDude parses your file and searches for a definition for OpenInputFile, but couldn't find one, hence the error message. You can disable this by unchecking "show undefined labels in error task" and "Decorate undefined labels" in Tools>Options>AsmDude.
How did you define the memory operand OpenInputFile?

it's not an operand
it's a procedure in irvine32 library and it's working fine
AsmDude just SUDDENLY decided it is undefined!!
actually AsmDude auto-completed them before but now they are undefined!!!
the same goes for Crlf, writeChar, WriteDec, etc...

What version of AsmDude are you using. You must be including a file in which OpenInputFile is defined otherwise no assembler knows which memory offset to call. Are you sure you can assemble your file?

last version
yes it assembles and runs successfully and the output is correct

is that <IP_ADDRESS>?

<IP_ADDRESS>
there's no .6

Yes, something seems to go wrong with VS2017, I'll look into it.

Hi @Sherry-Ashraf,
Version <IP_ADDRESS> will correctly tokenize included files. Thanks for reporting this.

no problem ^_^
it's fixed now
great work
but it displays label clash warnings regarding the included file (it's a built-in file)
it's a warning  so it's not a big deal but i thought you should know  :)


Fix regression

Fixes #?
Context

Contributor Checklist

[ ] Review Contribution Guidelines
[ ] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[ ] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

This fixes https://github.com/gradle/gradle/issues/10347 right?


GeoPointField support needed

Updating to version 2.0.2, the framework fails with GeoPointField declaration:
u"<class 'mongoengine.fields.GeoPointField'> is not supported, yet. Please open a ticket regarding this issue and have it fixed asap.\nhttps://github.com/umutbozkurt/django-rest-framework-mongoengine/issues/"
With a previous version, we were able to use this field in a model or at least to use it in the declaration.
Thanks and keep the great work !!

Keep in mind, that GeoPointField is somewhat depricated ("legacy way") in favor of PointField which is able to make 2dsphere indexes.

@qwiglydee PointField doesnt seem to be supported yet either
<class 'mongoengine.fields.PointField'> is not supported, yet. Please open a ticket regarding this issue and have it fixed

Ok
Thanks a lot!
;)


No ripple effect in button and also error on clicking it

I have used the
<lx-button lx-type="raised">Button</lx-button>

above code from Component: Button . It is displayed correctly, but there is no ripple effect when I click that button.

Hello I had the same problem but I fixed it including first JQuery and after Angular


Restore the localpackages folder of nuget.config

Description
Restore the localpackages folder of nuget.config.  WinAppSDK automation places a freshly-built WinAppSDK nupkg in this folder to test it.
Target Release
WinAppSDK 1.6
Checklist

[ ] Samples build and run using the Visual Studio versions listed in the Windows development docs.
[ ] Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
[ ] Samples set the minimum supported OS version to Windows 10 version 1809.
[ ] Samples build clean with no warnings or errors.
[ ] [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
[ ] If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
[ ] I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

/azp run SamplesCI-DeploymentManager


[BUG] Conduit settings issues.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior

Expected Behavior
As shown, the conduits don't "work" until they're updated.
Steps To Reproduce

Place conduit.
Change conduit settings.
Leave the world.
Log back in.
Conduit stops working/returns to default.

Environment
- OS: Windows 11
- Java version: 19.0.1
- Minecraft version: 1.20.1
- Version: 6.0.5-alpha

Anything else?
If there's anything I forgot to mention/did wrong let me know, first time on this site.

was reproducable, when node had to be recovered, fixed with #296


Arduino Uno Example Millis

The arduino uno example for millis does not work with the upgraded version of avr-device. The interrupt seems to have been removed from the root avr-device crate.
#[avr_device::interrupt(atmega328p)]
This one specifically.

Hm, can you be a bit more specific about the problem you're encountering?  The TIMER0_COMPA interrupt we're using in the millis example is available when I look at the new avr-device sources: https://docs.rs/crate/avr-device-macros/0.5.0/source/src/vector.rs
Do you get a compiler error?  Or does the interrupt just not fire when it should?

When following along with the version upgrade that you did a few days ago of the avr-device and when I did I got a compiler error which I will include. It might be just me being a noob in rust still but I figured it was because it might have been removed since the version upgrade broke the build.
error[E0433]: failed to resolve: could not find `interrupt` in `avr_device`
  --> src/millis.rs:43:15
   |
43 | #[avr_device::interrupt(atmega328p)]
   |               ^^^^^^^^^ could not find `interrupt` in `avr_device`

This error dissapears when using 0.4.0.

Can you prepare a reproducer project somewhere?  It seems to be working in the examples here...

I might be just overlooking something, but in my attempt at reacreating the example it seems to be failing. In this repo. If you change the version to 0.4.0 it works.

Ah, that's a nasty one.  You were referencing an old version of arduino-hal which only enabled the interrupt-runtime-support for it's 0.4 version of avr-device while you were separately depending on avr-device 0.5.  So there were two versions of avr-device in your dependency graph, actually.  This is a quirk of cargo's dependency resolution algorithm, unfortunately.
You can fix this by depending on the latest version of arduino-hal instead:
diff --git a/Cargo.toml b/Cargo.toml
index 73c0bfc..10136fa 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -19,12 +19,11 @@ dht11 = "0.3.1"

 [dependencies.arduino-hal]
 git = "https://github.com/rahix/avr-hal"
-rev = "4170a773d4d76cc93433d2455ed8b14e573ebe70"
+rev = "4c9c44c314eb061ee20556ef10d45dea36e75ee4"
 features = ["arduino-uno"]

 [dependencies.avr-device]
 version = "0.5.0"
-features = ["atmega328p"]

 [dependencies.either]
 version = "1.6.1"

Now the version of arduino-hal will also depend on avr-device 0.5 and enable interrupt support for it.
I'll go and upgrade the version referenced in the avr-hal-template ASAP....


Adds split_set to datasets. Closes issue #57

This PR adds a split_set method to the Dataset class. It allows to take a given amount of a set and transfer it to a new dataset. Particularly useful when used with spearmint to create a validation set. Some considerations:

I wasn't sure about how to correctly get a random number, with regards to efficiency and respect to the random seed. (neon/dataset.py, line 264) The way I did it somehow feels wrong to me.
I am testing this functionality in test_cifar100.py, as I need to instantiate a dataset. I also tested its call in the fit function with yaml files on my machine.
It works batchwise, as this is called after dataset.format(). I think that changing it to work in load() would take a lot more changes, and is not worth the added complexity.

As mentioned in #56, I also think that having a way to have validation scores while training can be very useful. (This early stopping will help save a lot of time with spearmint) Do you want me to work on that and add it to model ? (ie, something like if print_validation then do it) Or is that a functionality you prefer to keep out of the framework ?
Thanks for your comments,
Séb

Hi Séb,
Thanks for taking the effort to implement this pull request, but I don't think we can accept it as-is unfortunately.
See my comments in #57 for additional detail, but we really don't want to be shuttling data around between train and validation sets after they have already been loaded and are sitting on device.  We currently have the ability to do this as part of the load() process for directories of images using the BatchWriter class and the validation_pct parameter.  What would be really useful for us is enabling this functionality at load time for non BatchWriter based datasets.  If that's something you could implement we'd be more than happy to take a look!
As for the validation scores while training, we're probably going to end up supporting functionality like this through asynchronous callbacks that users can specify to run at the end of each epoch, but we're still trying to determine exactly how that will look.  If you want to implement the print_validation (default to False) approach I think that would still have some use in the interim, so again we'd be happy to take a look at a pull request for that.

Hello,
Alright, I'll move the validation_split in the init() of the non BatchWriter datasets (and will rename it to validation_pct for consistency). I thought about the shuttling problem, but I assumed that it would only be the reference to memory that would be passed around. Is my assumption wrong, or is that kind of behavior outside of load that is problematic ? In the latter case, (and if having the batch approximation is not a problem) could we simply call the current split_set after format ? Otherwise, I'll work with the data before it is formatted.
I can also do the print_validation approach, directly in the models. Is that fine if they both come in the same pull request ?
Have a nice weekend,
Séb

Hi Séb,
The dataset.format() function that most datasets call at the end of the load() will actually copy the first batch of values from host and onto the device (ex. GPU memory for a GPU backend), so that's why we need to implement these changes prior to that.
I think you can adapt your split_set code to 1) have it be called before format(), 2) not use backend calls for your random sampling -- numpy should be ok since the data should be on host at that stage, 3) not take any parameters except for maybe to_set and from_set (everything else should be part of the Dataset class).  I think doing something fairly similar to sample_training_data could work (for smaller datasets at least).  Another minor gripe would be to utilize one of the synthetic datasets like UniformRandom or ToyImages for your testing (neither has validation out of the box so should be useful).  That way you would need to download CIFAR100 to run the tests -- I'd also rename the file to something like test_val_split.py instead of test_cifar100.py.
I'd put the print_validation stuff in a separate pull request if possible (easier to track independently).

Thanks, just implemented that in #60


Move $config empty condition check inside AbstractContainer::configure()

I've found many times this souces:
        if (!empty($config)) {
            \yii\di\AbstractContainer::configure($this, $config);
        }

For example, here:
https://github.com/yiisoft/db/blob/54e7a7ef9f7683bb641a13574c5c34cbd8c5dc61/src/Query.php#L1308
Should not be better that $config empty condition check was done inside AbstractContainer::configure, so we should have only:
\yii\di\AbstractContainer::configure($this, $config);

As far as I remember, it was performance optimization: empty() call is cheaper than method call. We can probably measure it with basic app and, if it's negligible, move it to the method.

You are saying that
        if (!empty($config)) {
            \yii\di\AbstractContainer::configure($this, $config);
        }

is cheaper that
class AbstractContainer
{
        function configure($object, $config = [])
       {
               if (!empty($config)) {
                       // ....
               }
       }
}

?
Anyway the question is that I think that if we check $config content inside ::configure is always a good thing, because developer could forget to do it.

There are two cases:

When config is empty we're just calling empty().
When config is not empty we're calling both empty() and a method.

With your change we'll call both all the time.
But overall I don't think that matters.

https://github.com/yiisoft/di/pull/65#discussion_r244886030
Can't we just remove these checks?

If my PR is approved, I'll remove $config content check from everywhere ::configure() is called,
that it is superfluous.

But the PR doesn't make sense to me. See comment.

Ok, but then the same check could be removed from everywhere and simple calling:
            \yii\di\AbstractContainer::configure($this, $config);
such as here: https://github.com/yiisoft/db/blob/54e7a7ef9f7683bb641a13574c5c34cbd8c5dc61/src/Query.php#L1308

Yes, I think so.

What if config is null?

There is already a typehint in method signature - null cannot be passed as iterable, so there is no need to check type again.
We could improve signature even more using object type for $object parameter:
public static function configure(object $object, iterable $config)

Yes.


Expand and reframe discussion of optionals

Reframe the discussion of optional unwrapping to not conceptually rely on force unwrapping.
Highlight the value of non-optional types never being nil, and how you can't forget to handle nil on an optional.
Explicitly list the ways you can handle a nil value, and briefly introduce the ?? operator.
Explain when you should consider using force unwrapping.
Reduce the comparison to C and Objective-C.
Fix some mis-use of note boxes, where they were highlighting information instead of asides that readers can choose to skip.

Fixes: rdar://41014749
Forum (post-hoc) pitch: https://forums.swift.org/t/65744

On Aug 18, 2023, at 4:53 PM, Chuck Toporek @.***> wrote:
@chuckdude requested changes on this pull request.
Please see edits and comments within.In TSPL.docc/LanguageGuide/TheBasics.md:

it returns an optional Int, rather than an Int.
-An optional Int is written as Int?, not Int.
-The question mark indicates that the value it contains is optional,
-meaning that it might contain some Int value,
-or it might contain no value at all.
-(It can't contain anything else, such as a Bool value or a String value.
-It's either an Int, or it's nothing at all.)

+To write an optional type,
+you write a question mark (?)
+after the name of the type that the optional contains ---
+for example, the type of an optional Int is Int?.
⬇ Suggested change -for example, the type of an optional Int is Int?.
+for example, Int?.

Stet; this is the first time the reader is seeing an optional type like Int? so we need to define it.

In TSPL.docc/LanguageGuide/TheBasics.md:

it returns an optional Int, rather than an Int.
-An optional Int is written as Int?, not Int.
-The question mark indicates that the value it contains is optional,
-meaning that it might contain some Int value,
-or it might contain no value at all.
-(It can't contain anything else, such as a Bool value or a String value.
-It's either an Int, or it's nothing at all.)

+To write an optional type,
+you write a question mark (?)
+after the name of the type that the optional contains ---
+for example, the type of an optional Int is Int?.
+An optional Int always contains
+either some Int value or no value at all.
+It can't contain anything else, like a Bool value or a String value.
⬇ Suggested change -It can't contain anything else, like a Bool value or a String value.
+It can't contain anything else, like a Bool or String value.

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1271,11 +1255,6 @@ serverResponseCode = nil

-->

-> Note: You can't use `nil` with non-optional constants and variables.
-> If a constant or variable in your code needs to work with
-> the absence of a value under certain conditions,
-> always declare it as an optional value of the appropriate type.
-
If you define an optional variable without providing a default value,
the variable is automatically set to `nil` for you:

⬇ Suggested change -the variable is automatically set to `nil` for you:
+the variable is automatically set to `nil`:

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1325,47 +1300,55 @@ if convertedNumber != nil {

-->

-Once you're sure that the optional *does* contain a value,
-you can access its underlying value
-by adding an exclamation point (`!`) to the end of the optional's name.
-The exclamation point effectively says,
-“I know that this optional definitely has a value; please use it.”
-This is known as *forced unwrapping* of the optional's value:
+You can't use `nil` with non-optional constants or variables.
+If a constant or variable in your code needs to work with
+the absence of a value under certain conditions,
+declare it as an optional value of the appropriate type.
+A constant or variable that's declared as a non-optional value,

⬇ Suggested change -A constant or variable that's declared as a non-optional value,
+A constant or variable that's declared as a non-optional value

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

-```swift
-if convertedNumber != nil {

print("convertedNumber has an integer value of (convertedNumber!).")
-}
-// Prints "convertedNumber has an integer value of 123."
-```
+This separation of optional and non-optional values
+lets you explicitly mark what information can be missing,
+and makes it easier to write correct code that handle missing values.

⬇ Suggested change -and makes it easier to write correct code that handle missing values.
+and makes it easier to write code that handles missing values.

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

-<!--

test: optionals
+When you access an optional value,
+your code always handles both the nil and non-nil case.
+There are several things you can do when a value is missing,
+which are described in more detail in the following sections:

⬇ Suggested change -which are described in more detail in the following sections:
+as described in the following sections:

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

Optional Binding
-You use optional binding to find out whether an optional contains a value,
+You use optional binding to find out whether an optional contains a value,
Should this be "an optional binding"?

Existing content omits the article before "optional binding" throughout.  I think the best explanation is that optional binding is a technique — for example, we'd write "use pattern matching to check the value".
I added an entry to the TSPL style guide to call this out.

In TSPL.docc/LanguageGuide/TheBasics.md:

and if so, to make that value available as a temporary constant or variable.
-Optional binding can be used with if and while statements
+Optional binding can be used with if, guard, and while statements

⬇ Suggested change -Optional binding can be used with if, guard, and while statements
+Use optional binding with if, guard, and while statements

Suggest stet — this is not an instruction, it's a statement of where optional binding is available.  An alternate rewrite:

Use optional binding with if, guard, and while statements

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1410,8 +1393,9 @@ If the conversion is successful,
the actualNumber constant becomes available for use within
the first branch of the if statement.
It has already been initialized with the value contained within the optional,

⬇ Suggested change -It has already been initialized with the value contained within the optional,
+It has already been initialized with the value contained within the optional,

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1410,8 +1393,9 @@ If the conversion is successful,
the actualNumber constant becomes available for use within
the first branch of the if statement.
It has already been initialized with the value contained within the optional,
-and so you don't use the ! suffix to access its value.
-In this example, actualNumber is simply used to print the result of the conversion.
+and has the corresponding non-optional type.
+In this case, the type of possibleNumber is Int?

⬇ Suggested change -In this case, the type of possibleNumber is Int?
+In this case, the type of possibleNumber is Int?,

Changed.
I think we were trying to avoid a "punctuation pile-up" especially for readers who are still getting used to ? as a postfix.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1448,11 +1432,11 @@ the value of a new constant named myNumber is set to that value.
Inside the body of the if statement,
writing myNumber refers to that new non-optional constant.
Before the beginning of the if statement and after its end,
-writing myNumber refers to the optional integer constant.
+writing myNumber refers to the original optional integer constant.

Suggest rewriting this sentence as:
Writing myNumber before or after the if statement refers to the original optional integer constant.

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

-> Note: Constants and variables created with optional binding in an if statement
-> are available only within the body of the if statement.
-> In contrast, the constants and variables created with a guard statement
-> are available in the lines of code that follow the guard statement,
-> as described in doc:ControlFlow#Early-Exit.
+Constants and variables created with optional binding in an if statement
+are available only within the body of the if statement.
+In contrast, the constants and variables created with a guard statement
+are available in the lines of code that follow the guard statement,
+as described in doc:ControlFlow#Early-Exit.

+### Providing a Fallback Value
+
+Another way to handle a missing value is to supply
+a default value using the nil-coalescing operator (??).
+If the optional on the left side of the ?? isn't nil,
⬇ Suggested change -If the optional on the left side of the ?? isn't nil,
+If the optional on the left of the ?? isn't nil,

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

-> In contrast, the constants and variables created with a guard statement
-> are available in the lines of code that follow the guard statement,
-> as described in doc:ControlFlow#Early-Exit.
+Constants and variables created with optional binding in an if statement
+are available only within the body of the if statement.
+In contrast, the constants and variables created with a guard statement
+are available in the lines of code that follow the guard statement,
+as described in doc:ControlFlow#Early-Exit.

+### Providing a Fallback Value
+
+Another way to handle a missing value is to supply
+a default value using the nil-coalescing operator (??).
+If the optional on the left side of the ?? isn't nil,
+that value is unwrapped and used.
+Otherwise, the value on the right side of ?? is used.
⬇ Suggested change -Otherwise, the value on the right side of ?? is used.
+Otherwise, the value on the right of ?? is used.

Changed.

In TSPL.docc/LanguageGuide/TheBasics.md:

-> let name: String? = nil

-> let greeting = "Hello, " + (name ?? "friend") + "!"
-> print(greeting)
<- Hello, friend!

+-->
+
+For more information about using ?? to provide a fallback value,
+see doc:BasicOperators#Nil-Coalescing-Operator.
+
+### Force Unwrapping
+
+When nil represents an unrecoverable failure,
+such a programmer error or corrupted state,
+you can access the underlying value
+by adding an exclamation mark (!) to the end of the optional's name.
⬇ Suggested change -by adding an exclamation mark (!) to the end of the optional's name.
+by adding an exclamation point (!) to the end of the optional's name.

Changed.
Good catch — searched the rest of TSPL for "exclamation mark" and this was the only one that came back after commit 03062511c7e284cc8fdcd10da31e47885a466b8e in 2020 fixed these up for Apple style.  This text came from a branch that diverged from 'main' before that fix.

In TSPL.docc/LanguageGuide/TheBasics.md:

+For more information about using ?? to provide a fallback value,
+see doc:BasicOperators#Nil-Coalescing-Operator.
+
+### Force Unwrapping
+
+When nil represents an unrecoverable failure,
+such a programmer error or corrupted state,
+you can access the underlying value
+by adding an exclamation mark (!) to the end of the optional's name.
+This is known as force unwrapping the optional's value.
+When you force unwrap a non-nil value,
+the result is its unwrapped value.
+Force unwrapping a nil value triggers a runtime error.
+
+The ! is, effectively, a shorter spelling of [fatalError(_:file:line:)][].
⬇ Suggested change -The ! is, effectively, a shorter spelling of [fatalError(_:file:line:)][].
+The ! is a shorter spelling of [fatalError(_:file:line:)][].

Let's find a different rewording.  The ! isn't literally or exactly a shorter spelling of fatalError(...) — it does the same thing, but the crash traceback will look different.

In TSPL.docc/LanguageGuide/TheBasics.md:

+```swift
+let possibleNumber = "123"
+let convertedNumber = Int(possibleNumber)

+let number = convertedNumber!
+
+guard let number = convertedNumber else {

fatalError("The number was invalid")
+}
+```

+Both versions of the code above depend on convertedNumber
+always containing a value.
+Writing that requirement as part of the code,
+using either of the approaches above,
+lets your code check that the requirement is true at run time.
⬇ Suggested change -lets your code check that the requirement is true at run time.
+lets your code check that the requirement is true at runtime.

Changed to follow ASG.
Someday we may need to consider an exception to this rule (as noted in the TSPL style guide) if we start documenting the Swift runtime and need to distinguish between things that are part of the Swift runtime environment, and things that happen when the code is running (at run time).

In TSPL.docc/LanguageGuide/TheBasics.md:

+let number = convertedNumber!

+guard let number = convertedNumber else {

fatalError("The number was invalid")
+}
+```

+Both versions of the code above depend on convertedNumber
+always containing a value.
+Writing that requirement as part of the code,
+using either of the approaches above,
+lets your code check that the requirement is true at run time.
+
+For more information about enforcing data requirements
+and checking assumptions at runtime,
+see doc:TheBasics#Assertions-and-Preconditions below.
⬇ Suggested change -see doc:TheBasics#Assertions-and-Preconditions below.
+see doc:TheBasics#Assertions-and-Preconditions.

Changed; the "below" is fairly typical of TSPL forward references within a chapter:
LanguageGuide/Concurrency.md:  as shown in <doc:Concurrency#Unstructured-Concurrency> below.
LanguageGuide/Generics.md:as discussed in <doc:Generics#Extensions-with-a-Generic-Where-Clause> below.
LanguageGuide/Generics.md:which is discussed in <doc:Generics#Associated-Types-with-a-Generic-Where-Clause> below.
LanguageGuide/Initialization.md:<doc:Initialization#Class-Inheritance-and-Initialization> below.
LanguageGuide/Initialization.md:as described in <doc:Initialization#Automatic-Initializer-Inheritance> below.
LanguageGuide/Initialization.md:> For more information, see <doc:Initialization#Automatic-Initializer-Inheritance> below.
LanguageGuide/StringsAndCharacters.md:  (Unicode is discussed in <doc:StringsAndCharacters#Unicode> below)
ReferenceManual/Statements.md:<doc:Statements#Continue-Statement> below.
ReferenceManual/Statements.md:and is discussed in <doc:Statements#Break-Statement> below.
ReferenceManual/Statements.md:Control transfer statements are discussed in <doc:Statements#Control-Transfer-Statements> below.
ReferenceManual/Statements.md:see <doc:Statements#Fallthrough-Statement> below.
ReferenceManual/Statements.md:<doc:Statements#Continue-Statement> below.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1570,6 +1623,11 @@ and can definitely be assumed to exist at every point thereafter.
The primary use of implicitly unwrapped optionals in Swift is during class initialization,

⬇ Suggested change -The primary use of implicitly unwrapped optionals in Swift is during class initialization,
+Use implicitly unwrapped optionals in Swift when initializing a class,

Stet.
This isn't an instruction — most class initialization doesn't use IUOs — it's a statement of the motivation for this language feature to exist.

In TSPL.docc/LanguageGuide/TheBasics.md:

+Trying to recover from an invalid state isn't possible ---
+because the assertion failed,
+you know that at least one piece of the program's data is invalid,
+but you don't know why it's invalid
+or what additional state is also invalid.

⬇ Suggested change -Trying to recover from an invalid state isn't possible ---
-because the assertion failed,
-you know that at least one piece of the program's data is invalid,
-but you don't know why it's invalid
-or what additional state is also invalid.
+Recovering from an invalid state is impossible.
+When an assertion fails,
+at least one piece of the program's data is invalid ---
+but you don't know why it's invalid
+or whether an additional state is also invalid.

Changed — I think this rewrite works.
TSPL often uses the phrasing "trying to do X" when describing something that you might attempt, but which doesn't work.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1293,13 +1272,6 @@ var surveyAnswer: String?

-->

-> Note: Swift's `nil` isn't the same as `nil` in Objective-C.
-> In Objective-C, `nil` is a pointer to a nonexistent object.
-> In Swift, `nil` isn't a pointer --- it's the absence of a value of a certain type.
-> Optionals of *any* type can be set to `nil`, not just object types.
-
-### If Statements and Forced Unwrapping
-
You can use an `if` statement to find out whether an optional contains a value
by comparing the optional against `nil`.

In TSPL.docc/LanguageGuide/TheBasics.md:

@@ -1308,6 +1280,9 @@ or the “not equal to” operator (!=).
If an optional has a value, it's considered to be “not equal to” nil:

⬇ Suggested change -If an optional has a value, it's considered to be “not equal to” nil:
+If an optional has a value, it's considered as not-equal-to, nil:

Suggest stet; this matches how operators are named throughout TSPL.


Difference behavior for system Bus Access in dbg.sv and RISC-V External Debug Support Version 0.13.2

In section 3.12.23 of "RISC-V External Debug Support Version 0.13.2", write "sbdata0" operation will trigger the system bus write access operation. But in dbg.sv, the write or read access operation is decided by sbcs_reg[15] or sbcs_reg[20] and this seems don't match with the specification.

Thanks for moving from Western Digital GitHub. The issue has been assigned, we will respond by early next week.

Hi Ya-Chau,
Thank you very much for bringing this issue to our attention.  We can confirm that this is a bug in our design and will be fixed soon.
Kind regards,
Thomas

glad to learn something new from SweRV and can contribute something to it at the same time.

Fixed in version 1.2.


Change style of selected photo?

Is it possible to change the styling like using a tick and removing borders?

Done.


Swift 3.0.1: /Carthage/Checkouts/Alamofire/Source/Request.swift:474:47: warning: expression implicitly coerced from 'URLSessionTask?' to Any

Building the framework with Carthage on Swift 3.0.1 produces following warnings:
/Carthage/Checkouts/Alamofire/Source/Request.swift:474:47: warning: expression implicitly coerced from 'URLSessionTask?' to Any

Fixed in https://github.com/Alamofire/Alamofire/pull/1735 .

Ok, I may just be a real noob but I have tried compiling via carthage with version 4.0.1 and I still get this warning. My project will not compile as a result. If this is fixed, which version do we need to target when building with Carthage in order to use it? I guess I am asking: what should my cartfile entry look like for this to pull down properly.
Thanks!

Just put
github "Alamofire/Alamofire"

it will take the last version.
and check what version of Swift you are using!
mine is:
swift --version

Apple Swift version 3.0.1 (swiftlang-8<IP_ADDRESS> clang-8<IP_ADDRESS>)
Target: x86_64-apple-macosx10.9

But @Erulezz  and @DanBurkhardt
The PR is not merged yet, so the warning is still there, we need to wait for #1735

I'm going to close this issue out since #1735 has been merged. The 4.1.0 release will be released shortly. Thanks for everyone's patience here.
Cheers. 🍻


Argon2 Support

Support for Argon2 is apparently coming in PHP 7.1 and will be the default in PHP 7.2.  It would be ideal if we could utilize this via Zend as well.
Not sure how multi-hashing functionality should be supported, perhaps that should be another ticket?

I'm personally interested in getting server relief supported.  This is a scheme in which the hashing happens client side, allowing extreme key stretching without opening one up to DoS attacks.  Argon2 has this functionality built-in, performing a final hash on the client input.  The API would need to send the salt to the client and then pass the result to Argon2...

@indolering we will support it for sure after the release of PHP 7.1. The idea is to create a specific class for Argon2 under the Zend\Crypt\Password namespace.

@ezimuel Any plans to include an interface for server relief?  It would represent an order-of-magnitude increase to security.

@indolering as I said, we will consider to support Argon2 when it will be available in PHP.


Yarn 6460: abitility to accelerate time in SLS

patch YARN-6460 not yet officially accepted by the community

replaced by https://github.com/criteo-forks/hadoop-common/pull/31


Highlight players who can attack you in the wilderness

This would help a ton when runecrafting

And when pking. I second this

Duplicate of #512


Fixed typo in tests/sitemaps_tests/test_http.py.

ommitted -> omitted

@eltociear Thanks :+1: Welcome aboard :boat:


[Release] sdk/resourcemanager/compute/armcompute/4.0.0

https://github.com/Azure/sdk-release-request/issues/3203

Hold for another compute breaking change.

Replaced by https://github.com/Azure/azure-sdk-for-go/pull/19244.


Added keystore password prompt

Adds a prompt when the Android keystore password is not set in config - for obvious reasons.

Hey @johannesberdin, thanks for your work.
What would happen if I have no intent to sign an APK? (keystore env vars being empty, see https://github.com/voydz/souyuz/blob/master/lib/souyuz/runner.rb#L16) Could you please verify that those checks are respected?
Thank you in advance. :)

You are right, the prompt only makes sense when keystore alias and path are configured.


Automated formatting for PHP code files

Looking for input.  Not disagreeing with people having a personal preference of one style over the other, but what do you think of implementing some sort of automated formatting that can be used to consistently (and automatically) apply a style to the code base? We can then also implement CI to lint against this.
Some potential solutions:

prettier-php (https://github.com/prettier/plugin-php) -- not sure how I feel about having to add yarn as a dependency in a php repo
php-cs-fixer (https://github.com/FriendsOfPHP/PHP-CS-Fixer)
phpfmt (https://github.com/kodeclan/phpfmt)

Thoughts? Opinions?

Sounds good, I’ve used php-cs-fixer and php CodeSniffer - both on the same project at times, although I wouldn’t recommend this because in edge cases they sometimes contradict each other and cause more headaches.
Whichever standard I recommend it a superset of PSR12.

On 28 Sep 2021, at 04:34, Tim Austin @.***> wrote:

Yea, I have no experience with things other than php-cs-fixer, but just wanted to put out there that open to alternatives if better things exist. If was have a start with php cs fixer, building on that might be an easy win to implement this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Since we have phpcs in our CI already, I spun up a draft pr with the laminas coding standard rules added.  Quite a few rules I think we would have to disable (just due to project requirements -- e.g. the example solution being named different from the inner class). But was super simple.
Draft PR here: #421

@neenjaw Awesome!
Gobally I exclude the PSR1.Methods.CamelCapsMethodName rule for test classes. Descriptive test method names can be rather long and I write them in snake_case for improved readability. Other than I only add exclusions for legacy code, where the coding standard can not be applied all at once.
As for this project, there are quite a few errors that could be fixed automatically. While it is possible to ignore or disable phpcs for specific lines in a file, I would suggest having all these rules in the phpcs configuration file thus not confusing users with irrelevant coding style directives.

It looks like this issue is solved with:

https://github.com/exercism/php/blob/main/.github/workflows/exercise-lint-phpcs-psr-12.yml
https://github.com/exercism/php/blob/main/phpcs-php.xml

@neenjaw Should it be closed ?

It looks like this issue is solved with:

https://github.com/exercism/php/blob/main/.github/workflows/exercise-lint-phpcs-psr-12.yml

https://github.com/exercism/php/blob/main/phpcs-php.xml

@neenjaw Should it be closed ?

As @arueckauer said above:
https://github.com/exercism/php/issues/418#issuecomment-952792656
This issue has not been fulfilled by the existing psr12 linting standard.
However since I haven't continued with this and no one else has, we can close it.

#418 (comment)

Oh sorry I did not read this comment. To me we should stick to PSR12 for as this is the only widely accepted coding standard.
Personally I like https://github.com/doctrine/coding-standard (with some tweaks)


Add missing tests

Codecov Report
Patch and project coverage have no change.

Comparison is base (14a0d39) 100.00% compared to head (ec75e8d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #163   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          806       806
  Branches       179       179
=========================================
  Hits           806       806

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update README.md

Fixed bug with ZipCodeRule (: for the place of =)

Codecov Report

Merging #184 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #184   +/-   ##
=======================================
  Coverage   80.82%   80.82%
=======================================
  Files          26       26
  Lines         485      485
=======================================
  Hits          392      392
  Misses         93       93

Continue to review full report at Codecov.

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

Thanks @wampir1408


How to add multiple http.HandleFunc?

I would like to add multiple handler for multiple endpoint but don't know how to do that.

Ghost bridge will only expose/configure/wrap a single http endpoint. You can run arbitrary http handlers there in Go however.


[MRTK3] Removing Segoe font family

Overview
Removing Segoe font family for license issues.
Changes
Removed all Segoe SDF font assets. We use Selawik in our UI set.

@cre8ivepark I'd really appreciate it if we can sort out the licensing issues ASAP....


Support run unit test, cucumber test, code format in osrm-backend-dev

Install node
Install clang-format

Testing related docs:

https://github.com/Project-OSRM/osrm-backend/blob/master/docs/testing.md
https://github.com/Project-OSRM/osrm-backend/wiki/Cucumber-Test-Suite

From https://github.com/Project-OSRM/osrm-backend/issues/5720, node should be at max version 10.

From https://github.com/Project-OSRM/osrm-backend/blob/1ba8aba4663590c5c53b3b0a9989e4fe605b45b8/scripts/format.sh#L33, the clang-format version has to be 3.8. Try apt-get install clang-format-3.8 on debian/ubuntu.

Testing related docs:

https://github.com/Project-OSRM/osrm-backend/blob/master/docs/testing.md
https://github.com/Project-OSRM/osrm-backend/wiki/Cucumber-Test-Suite

From Project-OSRM#5720, node should be at max version 10.
From Project-OSRM#5667, gcc 8 seems not work either, has to be gcc 6.

In my test, gcc 8 or node 14 still can not work. gcc 6 + node 10 work well.

Tested:
npm install --unsafe-perm
rm -rf build && rm -f lib/binding/*

./scripts/node_install.sh
ll lib/binding/

# run cucumber test via npm
npm test

# OR via cucumber directly
./node_modules/.bin/cucumber-js

# format code
./scripts/format.sh

npm install --unsafe-perm --build-from-source doesn't work for me. See more discuss in https://github.com/Project-OSRM/osrm-backend/issues/5776.
build api docs(npm run docs) doesn't work either, will be discussed in https://github.com/Project-OSRM/osrm-backend/issues/5867.


npc fishing and catching fish causes issues

version is 1.14.4 paper build 173
denizen version is Denizen-1.1.0-b4041-DEV
citizens version Citizens-2.0.25-b1743
when using the fish command from denizen in game and using /ex fish <player.selected_npc> <player.location> catch:fish
the npc will start fishing but never reel the line back in then after a few seconds it will cast an additional line and then repeat  untill you have a bunch of lines in the water

for refrence

 and using /ex fish <player.selected_npc> <player.location> catch:fish
Two things:
One, <npc> is all you need in /ex, as it automatically links the selected NPC.
Two: the fish command does not take an NPC as input, it uses the linked NPC, so you don't need that arg here all for this comand.
Also: Issue was caused by an error in NMS code, fixed via https://github.com/DenizenScript/Denizen/commit/ed69dcba4f4a030acf448916f45ecca98def02fe


waitForElementToHide not working as expected

Output of gemini --version:
4.16.0
Contents of .gemini.js file:
module.exports = {
    rootUrl: 'http://localhost:1337/#',
    gridUrl: 'http://localhost:4444/wd/hub',
    windowSize: '1600x1080',
    screenshotMode: 'fullpage',
    compositeImage: true,
    sessionsPerBrowser: 1,
    browsers: {
        chrome: {
            desiredCapabilities: {
                browserName: 'chrome'
            }
        }
    },
    system: {
        sourceRoot: './src',
        coverage: {
            enabled: false
        }
    }
};

Test source code:
gemini.suite('account', function (suite) {
    gemini.suite('reset password request', function (suite) {
        suite.setUrl('/')
            .setCaptureElements('md-dialog.container__dialog--password-reset')
            .capture('reset password dialog', function (actions, find) {
                actions.click(find('[ng-click="LoginCtrl.showResetPassword()"]'));
                actions.waitForElementToShow('md-dialog.container__dialog--password-reset', 5000);
            })
            .capture('close dialog with button', function (actions, find) {
                actions.click('[ng-click="ResetPasswordCtrl.cancel()"]');
                // Wait for dialog to close
                actions.waitForElementToHide('md-dialog.container__dialog--password-reset', 15000);
            });
    });
});

Command used to run the test:
gemini test --reporter flat --reporter html

Expected behaviour:
Dialog should be closed, but instead there is an error returned:
StateError: Area with css selector : md-dialog.container__dialog--password-reset is hidden
    at /usr/local/lib/node_modules/gemini/lib/browser/client-bridge.js:42:39
    at tryCatcher (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/promise.js:510:31)
    at Promise._settlePromise (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/promise.js:567:18)
    at Promise._settlePromise0 (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/promise.js:612:10)
    at Promise._settlePromises (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/promise.js:691:18)
    at Async._drainQueue (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/async.js:133:16)
    at Async._drainQueues (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/async.js:143:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/usr/local/lib/node_modules/gemini/node_modules/bluebird/js/release/async.js:17:14)
    at processImmediate [as _immediateCallback] (timers.js:383:17)

Sorry, this was a layer 8 problem. I set the dialog as element to capture which I then close. So it was not the waitForElementToHide which caused the error, but the capture. My bad.


Migrate to new s3 bucket

Hold: Don't merge this yet. Need additional testing, and to discuss my proposed strategy below. I'll apply this to other repos once we agree on how to move forward.
For those who don't know @finscn has been asking for a long time for us to use a CDN that is China-friendly. CloudFlare is China friendly, but there are some restrictions with how we will have to setup our S3 bucket data to migrate off of AWS CloudFront and onto CloudFlare. For some discussion see this issue.
My current idea is:

Merge a change similar to this for each of our repos we deploy.

Each which deploy to the old repo-specific bucket, but also to a new bucket for pixijs.download which has a folder for each repo. More details below.

Once merged, copy all the old values for each from the old bucket into the new one.
Bring down our CloudFront implementation and move our DNS over to CloudFlare

This will be the downtime which will last as long as it takes me to complete the CloudFlare setup and for DNS to propagate.

After we have confirmed the CloudFlare distribution is working:

Merge a change to each repo to stop deploying to the old buckets
Delete our CloudFront setup from AWS
Delete the old buckets from AWS

Details on the new single-bucket setup:
This single bucket strategy will make it much easier to scale and manage permissions. Currently adding a new project involves writing the travis scripts, adding a new bucket, and adding a CloudFront proxy. With this setup no S3/CloudFlare changes would be needed, all you need to do is write the travis config.
However, it does mean the cannonical url for some buckets will be pixijs.download/<repo> rather than <repo>.pixijs.download. I think I have have CNAME rules to make this not be a breaking change for most urls. The only URL that would be broken is the main pixi.js repo one, because it used no subdomain and no path.
An alternative to maintain backwards compatibility would be to have the new canonical url be files.pixijs.download/<repo> (or any subdomain) and have the root redirect to the new path.

Codecov Report

Merging #5862 into dev will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5862      +/-   ##
==========================================
+ Coverage   70.52%   70.53%   +<.01%
==========================================
  Files         206      206
  Lines       10407    10407
==========================================
+ Hits         7340     7341       +1
+ Misses       3067     3066       -1

Impacted Files
Coverage Δ

packages/ticker/src/Ticker.js
88% <0%> (+0.79%)
:arrow_up:

Continue to review full report at Codecov.

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


add test cases to 32bit unit test

Add the new unit tests for issue 3541 to the 32bit unit test file.

CI is green on my end.


[Feature]: including antd one time. we can use the libarary anywhere.

What problem does this feature solve?
I use antd everywhere including many programs in apps and packages. I will install antd everywhere that leadding to a problem of different antd versions. or exporting all antd components after installing in some package.
What does the proposed API look like?
we can install antd directly in global environment.  and use it anywhere.

You can use alias to force all antd resolved to a single version:
https://modernjs.dev/en/configure/app/source/alias.html


[Roadmap] AIOS Roadmap Q4 2024

Integration of the LLM Kernel into the OS Kernel (Om Raheja)

using the OpenBSD kernel or FreeBSD kernel as the OS kernel
create new abstraction in OS kernel for integrating LLMs
translation to C/C++ or Mojo language of LLM kernel implementation

Standard interface for agents and LLM kernels
Memory optimization low level handling
https://www.openblas.net for cuda

These would have to be completed before


Add DG draft for proposed sorting feature

Added implementation details of sorting to DG draft as per week 10 requirements.
Fix #112

Codecov Report

Merging #113 (3fd7eb7) into master (84c6d7b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 3fd7eb7 differs from pull request most recent head ba9230f. Consider uploading reports for the commit ba9230f to get more accurate results

@@            Coverage Diff            @@
##             master     #113   +/-   ##
=========================================
  Coverage     69.70%   69.70%
  Complexity      437      437
=========================================
  Files            80       80
  Lines          1436     1436
  Branches        155      155
=========================================
  Hits           1001     1001
  Misses          385      385
  Partials         50       50

Continue to review full report at Codecov.

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