Issue with version of event-stream (flatmap-stream) packages in package-lock

So i just started with your egghead tutorial on hooks (https://egghead.io/lessons/react-refactor-a-class-component-with-react-hooks-to-a-function). (thanks for doing those!)
When trying to install the dependencies via npm i get the following error:

npm ERR! notarget No matching version found for<EMAIL_ADDRESS>
which is related to npm removing that package https://blog.npmjs.org/post/180565383195/details-about-the-event-stream-incident
just a heads up for you or anyone running into this issue.
quick solutions:

install an older version of event-stream (npm i<EMAIL_ADDRESS>remove package-lock and try again
yarn

Fixed!


Add target directory

Once we start a download we must be able to tell the Downloader where to save the files. This should be passed via CLI too (e.g. in the structure described in #12) and can have a meaningful default set to the cwd.

I'm interested on doing this!

@danielfireman does it worth it to wait until #12 is covered? Or this might be tackled within #12 itself?

Awesome! Roger that! Working on it! Can you assign this issue to me?


Extend string types

Implemented Extend<char>, Extend<&'a char>, FromIterator<char>, and FromIterator<&'a char> for String and WString.
Of note here is that with this change, String and WString now have interior mutability so they are no longer able to implement Sync safely.
Original Issue: https://github.com/ros2-rust/ros2_rust/issues/246

Thanks for making these changes!

Thanks for all the guidance :)


Test failure in FeignHttpClientTests

Works on command line for me but not in IDE so probably a race condition.

Could not reporoduce. It seems this issue is no longer there.


Mipnerf OOM during first initialization

Thanks for sharing this excellent work.
I've tried nerfacto which worked fine.
After 8x downscale,the resolution of images is only 480x270:
ns-train mipnerf nerfstudio-data --downscale-factor 8 --data ./
logging events to: outputs/mipnerf/2022-11-27_222917
[10:29:25 PM] Skipping 0 files in dataset split train.                                       nerfstudio_dataparser.py:96
              Skipping 0 files in dataset split val.                                         nerfstudio_dataparser.py:96
Setting up training dataset...
Caching all 155 images.
Setting up evaluation dataset...
Caching all 17 images.
No checkpoints to load, training from scratch
Printing profiling stats, from longest to shortest duration in seconds
Traceback (most recent call last):
  File "/home/horry/anaconda3/envs/torch/bin/ns-train", line 8, in <module>
    sys.exit(entrypoint())
  File "/home/horry/Nerf/nerfstudio/scripts/train.py", line 248, in entrypoint
    main(
  File "/home/horry/Nerf/nerfstudio/scripts/train.py", line 234, in main
    launch(
  File "/home/horry/Nerf/nerfstudio/scripts/train.py", line 173, in launch
    main_func(local_rank=0, world_size=world_size, config=config)
  File "/home/horry/Nerf/nerfstudio/scripts/train.py", line 88, in train_loop
    trainer.train()
  File "/home/horry/Nerf/nerfstudio/nerfstudio/engine/trainer.py", line 151, in train
    loss, loss_dict, metrics_dict = self.train_iteration(step)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/utils/profiler.py", line 43, in wrapper
    ret = func(*args, **kwargs)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/engine/trainer.py", line 315, in train_iteration
    _, loss_dict, metrics_dict = self.pipeline.get_train_loss_dict(step=step)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/utils/profiler.py", line 43, in wrapper
    ret = func(*args, **kwargs)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/pipelines/base_pipeline.py", line 255, in get_train_loss_dict
    model_outputs = self.model(ray_bundle)
  File "/home/horry/anaconda3/envs/torch/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/models/base_model.py", line 138, in forward
    return self.get_outputs(ray_bundle)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/models/mipnerf.py", line 120, in get_outputs
    field_outputs_fine = self.field.forward(ray_samples_pdf)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/fields/base_field.py", line 104, in forward
    density, density_embedding = self.get_density(ray_samples)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/fields/vanilla_nerf_field.py", line 96, in get_density
    encoded_xyz = self.position_encoding(gaussian_samples.mean, covs=gaussian_samples.cov)
  File "/home/horry/anaconda3/envs/torch/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/horry/Nerf/nerfstudio/nerfstudio/field_components/encodings.py", line 153, in forward
    encoded_inputs = expected_sin(
  File "/home/horry/Nerf/nerfstudio/nerfstudio/utils/math.py", line 193, in expected_sin
    return torch.exp(-0.5 * x_vars) * torch.sin(x_means)
RuntimeError: CUDA out of memory. Tried to allocate 384.00 MiB (GPU 0; 15.74 GiB total capacity; 13.18 GiB already allocated; 287.00 MiB free; 13.41 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF
wandb: Waiting for W&B process to finish... (failed 1). Press Control-C to abort syncing.
wandb: Synced sweet-violet-28: https://wandb.ai/belkahorry/nerfstudio-project/runs/hahkjmlw
wandb: Synced 6 W&B file(s), 0 media file(s), 0 artifact file(s) and 0 other file(s)
wandb: Find logs at: outputs/mipnerf/2022-11-27_222917/wandb/run-20221127_222919-hahkjmlw/logs

I knew original mip360(multinerf) got OOM bug during computing SSIM in eval step. But may fixed by #81. So how can i fix this one?

LPIPS requires a fair amount of memory, you could try removing all references to it in the model file, ie https://github.com/nerfstudio-project/nerfstudio/blob/aac7aa4786f6a6a7ba5da7acfc79db73359d6b28/nerfstudio/models/mipnerf.py#L89
You can also decrease the batch size using the --pipeline.datamanager.train-num-rays-per-batch argument.


Update criteria.md

Added a suggestion of cryptographic agility.
The security period of a cryptographic algorithm is evaluated based on the current computing and cipher analysis capabilities. The security period of the algorithm cannot be fulfilled if the computing capability is tremendously enhanced or any design vulnerability is identified. When an algorithm or the key length is found to be insecure, it is necessary to be replaced. Therefore, cryptographic agility should be considered during project design to avoid update failures or high costs due to incompatibility.
Signed-off-by: Wang Anyu<EMAIL_ADDRESS>

Current coverage is 98.42% (diff: 100%)

Merging #530 into master will decrease coverage by 0.02%

@@             master       #530   diff @@
==========================================
  Files            39         39
  Lines          1028       1013    -15
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1012        997    -15
  Misses           16         16
  Partials          0          0

Powered by Codecov. Last update ed4f0da...f0420f1

Believe it or not, this is controversial.
I personally agree that cryptographic agility is important.  We originally had such a criterion.
However, several experts argue that you're better off selecting a really good algorithm, and forcing it.  For more info: https://lists.coreinfrastructure.org/pipermail/cii-badges/2016-February/000101.html


merge: dev to stage

:tada: This PR is included in version 0.48.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Release embedded-hal-async v0.1.0

Fixes compliation on latest nightlies, so we'd like to get this out soon.
I've named the version 0.1.0 and not 0.1.0-alpha.3 because there's not much point in using the alpha if the main version is 0.1 which already indicates "API is WIP". Next breaking change can be released as v0.2 (probably the switch to AFIT), etc.

r? @therealprof
(rust-highfive has picked a reviewer for you, use r? to override)

Another rationale for the -alpha version is the dependency on an -alpha version of embedded-hal.
For this crate I am fine with either, though.
Do you want to go ahead with 0.1.0?

Another rationale for the -alpha version is the dependency on an -alpha version of embedded-hal.

Oh... I didn't think about that at all. :rofl:  That's not semver-stable at all, indeed.
-alpha.3 it is then.


The Emperor's New Groove

tonyhax version: Which version are you running? Please ensure you are running the latest stable, or a newer beta. 1.4.3
Game name: The Emperor's New Groove
Game code: Game's serial number, eg "SCES-12345" SLUS-94571
Executable hash: SHA-1, SHA-256 or MD-5 of the game's main executable or the disc. This is important since there could be multiple revisions of the same game.
Triggered at: Please explain when is the screen shown, eg "after the Sony logo", or "after the first video" While loading to first level
Is the console chipped?: Yes or no. No
Is it an original disc or a burned copy?: Self-explaining. Burned

Fixed in 86b34c71b667930c379c58c1777f4ce616197d9b. Will be available on next v1.4.4 release.

Available in tonyhax v1.4.4


concatenate_datasets throws error when changing the order of datasets to concatenate

Hey,
I played around with the concatenate_datasets(...) function: https://huggingface.co/docs/datasets/package_reference/main_classes.html?highlight=concatenate_datasets#datasets.concatenate_datasets
and noticed that when the order in which the datasets are concatenated changes an error is thrown where it should not IMO.
Here is a google colab to reproduce the error: https://colab.research.google.com/drive/17VTFU4KQ735-waWZJjeOHS6yDTfV5ekK?usp=sharing

Hi,
this bug is related to Dataset.{remove_columns, rename_column, cast, flatten} not propagating the change to the schema metadata when the info features are updated, so this line is the culprit:
common_voice_train = common_voice_train.remove_columns(['client_id', 'up_votes', 'down_votes', 'age', 'gender', 'accent', 'locale', 'segment'])

The order is important because the resulting dataset inherits the schema metadata of the first dataset passed to the concatenate_datasets(...) function (pa.concat_tables docs). I'll try to fix this ASAP.


give terminator arnies curse action + refactor

About the PR
adds an action that exterminators can use up to 1 minute after spawning, gives them gear at the large cost of giving them an accent that spews out terminator quotes (and pizza shop :trollface:) which along with the gear makes them ultra valid, better get that kill quickly
also able to break out of cuffs since strong
fixes #24330
Why / Balance
like warops but for exterminator, discard any stealth for some extra gear
if you are sick of being stuck in maints or cant be bothered to make a bat, you get:

passenger id to get outta maints
bike to be quick, le lore
clothes (and headset so you can declare war on common), le lore
a kammerer

ingame screenshot :trollface:

action expires after 1 minute (tested) so you cant stealth for 30m then randomly spawn in a shotgun infront of the target
Technical details
@keronshb pls look at my action shitcode
quotes are intentionally not translated since that would be hell to translate :trollface:
the gear + accent prototype ids are stored on the event so if its made to support managing multiple actions you dont need to hardcode ActionOneGear ActionTwoGear etc and switch on the action name or some shit
Media
the action

the bird is the word
https://github.com/space-wizards/space-station-14/assets/39013340/89d21541-ca4d-4d15-a2d8-a4d2188299f1

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
exterminator components are in shared and renamed to Exterminator
Changelog
:cl:

add: Exterminators can now opt in to receiving Arnie's Curse, get sick gear in exchange for only speaking in terminator quotes.

extremely based

Probably needs a different motorbike, but that can come later.
If your open to bikesheds the idea seems nice and reusable if the loadout and ftl were able to be set in yml somehow.

can't wait for maintainers to 1984 close this because vehicle code is shitty
jokes aside here are some quick and shitty lever action shotgun sprites

I want this except sometimes you hit your own vehicle when shooting while riding

Based

This is fucking insane I love it

If your open to bikesheds the idea seems nice and reusable if the loadout and accent were able to be set in yml somehow.

they are but it can only have 1 action
ideally it would be more generic so you could have 5 of them and they all get deleted when you use one but lazy

shitty lever action shotgun sprites

99% sure its a pump action shotgun

99% sure its a pump action shotgun

the one he uses on the bike isn't a pump action

eh, he uses a few different shotguns, and making this pr run into the weapons freeze seems unnecessary. Could always PR the shotgun later and try to add it to the loadout like I plan to do with a motorbike.

Maybe add ammo drum (.50 shells)?

would be unstoppable then, just having a few shells means you can't waste them OR you have to raid security, both need you to be robust

I love this

It would be cool if you could also control what quote you say, like replacing certain letters with a certain quote.

It would be cool if you could also control what quote you say, like replacing certain letters with a certain quote.

levenshtein replacement accent which finds string with least levenshtein distance and says it

do arnie's curse should spawn in a time travelling rebel that guards the target :trollface:

@metalgearsloth what change i have renamed everything internally to Exterminator

@metalgearsloth what change i have renamed everything internally to Exterminator

ExterminatorCursed needs removing.

the entire curse related stuff?

@metalgearsloth merg

Can you give exterminator welding protection?

they should already be immune to temporary blindness not sure what proper blindness is

When i tried to deconstruct wall with welder i got blindness effect, i was exterminator

Aw, why was the curse removed? I was looking forward to talking in nothing but quotes

le copyright

Re-merge after newmed stuff ig.


Upgrade StarUML to 2.4.0

Bumping the version to the latest available on the official website

Thank you for the contribution. It was merged directly as f34172eca6a9fb166498bd0a2d2b3aae0d843210 to keep commit history cleaner. Your contribution is still credited to you.


SSC incorrectly refuses to use high-resolution weather data

Describe the bug
When running a pvsamv1 simulation with two years of 1-minute weather data, SSC produces the following error:
exec fail(pvsamv1): invalid number of data records (1051200): must be an integer multiple of 8760.  This is triggered by this line of code in shared/lib_pv_io_manager.cpp: https://github.com/NREL/ssc/blob/develop/shared/lib_pv_io_manager.cpp#L129
	//total number of records in the weather file (i.e. 8760 * timestep)
	numberOfWeatherFileRecords = weatherDataProvider->nrecords();
	stepsPerHour = numberOfWeatherFileRecords / 8760;
	dtHour = 1.0 / stepsPerHour;

	if (stepsPerHour < 1 || stepsPerHour > 60 || stepsPerHour * 8760 != numberOfWeatherFileRecords)
		throw compute_module::exec_error(cmName, util::format("invalid number of data records (%zu): must be an integer multiple of 8760", numberOfWeatherFileRecords));

As 1051200 is indeed an integer multiple of 8760, I believe that either this error message or the surrounding logic should be revised.  In particular, I believe the issue lies in the line stepsPerHour = numberOfWeatherFileRecords / 8760;, which does not actually calculate steps per hour for weather files covering more than one year -- eg a 2-year simulation with 5-minute weather data will calculate 24 steps per hour, not 12.  Note that if SSC requires weatherfiles to cover only a single year, then this check will still permit the 2-year, 5-minute data example.

Hey @janinefreeman , thanks for the reply!  I did end up getting around this issue by running two years separately and combining the output results.
Is it really true that SSC does not handle >1 year weatherfiles correctly?  I ask because I frequently run simulations with weatherfiles that are two years long with 5-minute resolution, and the results have seemed reasonable so far.  My purposes are primarily centered on high-resolution timeseries weather data which is almost never exactly one year in length.  I have been padding the weather files with zeros to fill them out to be an integral number of years long so that SSC will accept them, eg my data runs from June 2017 through August 2018, so I fill in zeros for Jan-June 2017 and Aug-Dec 2018 and run the whole padded weatherfile in a single simulation.  In case it matters, I only use the pvsamv1 module, and in particular, only the gen output array -- no financial model or anything.

@kevinsa5 It is true that SSC does not handle >1 year weatherfiles correctly, however in your specific case, you might not be too far off, if you're processing the gen output correctly.
In pvsamv1, the weather file timestamp is used to calculate sun position, so that is working correctly. However, sunrise/sunset calculations are dependent on timestep, so your 2-year simulations are probably missing generation from both sunrise and sunset hours every day. Fortunately, not a lot of power is produced at those times, so the difference is likely relatively small. The attached PNG shows output from SAM when I take the default Phoenix TMY2 weather file and run it as is (1 yr), versus copying and pasting the entire year to make a 2-yr file and running that. You can see that most of the daylight hours are the same, but sunrise and sunset hours are missing from the 2-yr file.

The part where the outputs really start to diverge for multi-year weather files is when SAM calculates energy versus power. The gen output represents power (kW), so it is independent of timestep length. However, many SAM outputs downstream of that represent energy (kWh), which is dependent on how long SAM thinks the power is produced for. So the annual energy value that SAM would calculate, for example, would be half of what it should be for a 2-yr simulation. Any downstream financial model results would be disastrous, but since you are not using those, you might be ok. If you're processing gen correctly, assuming that each power value persists for 1 hour instead of 30 minutes, then you are also probably getting correct energy values.
Hope that helps! We'll be adding better warnings about this case for our next release.

@janinefreeman this is extremely helpful -- thank you!  The sunrise/sunset phenomenon is good to know about, and I am indeed handling the power-energy conversion correctly (I prefer to work with power specifically for ease of timeseries resampling).  I suppose I got lucky that my misguided multi-year use case can happily ignore any inaccuracy during low-light conditions at dawn and dusk without ill effect.
Many thanks to you and the team for this tool.  It is the basis for a "mission-critical" tool at my company in utility-scale solar, and we use it for many custom analyses that would be otherwise impossible as well.

Modified the check in pv_io_manager to return error specific to stepsPerHour. Also implemented checks in weatherfile reader to check that the stepsPerHour found by subtracting two consecutive records is the same as totalRecords/8760.


Add AES GCM/CCM implementation and remove AES CTR/CBC modes

Added support for the following AES modes:

AES128GCM
AES256GCM (default)
AES-CCM-64-128-128
AES-CCM-64-128-256
One of these can be selected using the flag AES_MODE=gcm/ccm.
The key-length is selected based on the key exchange KEX=ecdh/ecdh384
and DA=ecdsa256/ecdsa384. The higher crypto level is the default.

Removed support for the following legacy AES modes:

AES128/CTR/HMAC-SHA256
AES128/CBC/HMAC-SHA256
AES256/CTR/HMAC-SHA384
AES256/CBC/HMAC-SHA384

Additionally, the storage HAL layer for linux now uses one of the AES
modes selected, instead of previous default AES128GCM.
Signed-off-by: Chandrakar, Prateek<EMAIL_ADDRESS>

Added support for the following AES modes:

AES128GCM
AES256GCM (default)
AES-CCM-64-128-128
AES-CCM-64-128-256
One of these can be selected using the flag AES_MODE=gcm/ccm.
The key-length is selected based on the key exchange KEX=ecdh/ecdh384
and DA=ecdsa256/ecdsa384. The higher crypto level is the default.

Considering we now only have ECDH supported in our codebase, would there be a situation where ECDH and ECDSA384 is chosen? Would that be a valid scenario?
If not, consider removing KEX parameter and use DA for choosing the KEX.

Removed support for the following legacy AES modes:

AES128/CTR/HMAC-SHA256
AES128/CBC/HMAC-SHA256
AES256/CTR/HMAC-SHA384
AES256/CBC/HMAC-SHA384

Additionally, the storage HAL layer for linux now uses one of the AES
linux => Linux

modes selected, instead of previous default AES128GCM.
Signed-off-by: Chandrakar, Prateek<EMAIL_ADDRESS>

Updated all review comments, except 1 that has been replied separately

Additionally, yes, we can streamline the usage of KEX and DA parameters. Please create an issue that will be fixed in another PR.


CreateBugFromPenTest - Bug is already active

When scanning my web app in front & behind an application gateway, the CreateBugFromPenTest task does not create new bugs, but identifies with the existing created bugs from the first scan.
Scanning behind waf
2019-08-09T17:45:24.3478958Z ##[command]D:\a\r1\a/_CI-OWASP/drop/owasp-zap-vsts-tool/bin/Release/owasp-zap-vsts-tool.exe createbugfrompentest collectionUri="https://mydevops" teamProjectName="ProjTeam" team="MyTeam" releaseUri=vstfs:///ReleaseManagement/Release/1766 releaseEnvironmentUri=vstfs:///ReleaseManagement/Environment/4890 filepath=D:\a\r1\a\issues.xml  prefix="No WAF" targetUrl="https://mywebapp.azurewebsites.net" failOnHigh=False personalAccessToken=""
2019-08-09T17:45:25.6780361Z Issues(2 found.
2019-08-09T17:45:26.0890536Z Bug created
2019-08-09T17:45:26.4454036Z Bug created
2019-08-09T17:45:26.5751597Z ##[section]Finishing: Behind WAF - OWASP ZAP Create Azure DevOps Bugs from Issues
Scanning in front of waf
2019-08-09T17:50:39.5100575Z ##[command]D:\a\r1\a/_CI-OWASP/drop/owasp-zap-vsts-tool/bin/Release/owasp-zap-vsts-tool.exe createbugfrompentest collectionUri="https://mydevops" teamProjectName="ProjTeam" team="MyTeam" releaseUri=vstfs:///ReleaseManagement/Release/1766 releaseEnvironmentUri=vstfs:///ReleaseManagement/Environment/4890 filepath=D:\a\r1\a\issues.xml  prefix="WAF" targetUrl="https://test.mydomain.com" failOnHigh=False personalAccessToken=""
2019-08-09T17:50:40.5332947Z Issues(2 found.
2019-08-09T17:50:40.8327729Z Bug is already active
2019-08-09T17:50:41.1550887Z Bug is already active
2019-08-09T17:50:41.2870901Z ##[section]Finishing: Infront of WAF - OWASP ZAP Create Azure DevOps Bugs from Issues

The prefix is being ignored when determining if a bug is a duplicate or not.


Workspace release tags do not work

I generated tags with https://github.com/crate-ci/cargo-release as suggested in the readme.
But unfortunately, this doesn't trigger the release because it doesn't match the v* format: https://github.com/domoritz/arrow-tools/tags.

Nm, I should read the readme.
I added


[4.0][Possible BUG]Operations in controller needed to order correcly

Sorry English is not my first language but I'll try my best to explain this.
What I did:
I put CreateOperation after ShowOperation in use statement in my controller
...
class MyController extends CrudController
{
    use ShowOperation,CreateOperation;
...

What I expected to happen:
When I go to admin/my/create. I expected it to show create form.
What happened:
it say "No query results for model"
What I've already tried to fix it:
...
class MyController extends CrudController
{
    use CreateOperation,ShowOperation;
...

The problem is from Route order that laravel alway use first route that match condition.(admin/my/create vs admin/my/{id})
In this case admin/my/create is working correctly
The problem is. If I want to override function setupShowRoutes I have to override function setupCreateRoutes(even I dont need to change anything with it) and I have put it before setupShowRoutes in my controller. Is this a normal way to fix this?
Backpack, Laravel, PHP, DB version:
Backpack 4.0
Laravel 6.0
PHP 7.2.4
MYSQL 8.0.13

Hi @reindeeruz ,
Indeed - it's a known issue, and the current solution is to put the Show operation after Create.
To fix this long-term, we're considering is #2039 - changing the route of the "show" operation from monster/{id} to monster/{id}/show. But this would make it a little different from how Laravel does its own RESTful controllers.
Let me know what you think about v4 so far - not many people have tried it yet.
Cheers!


Service to fetch Assemblies by accession

When given an accession there currently exists functionality in the project to:

Open an FTP connection (using FTPClient)
Construc the path of required directory (using NCBIBrowser)
Navigate to the path of the *assembly_report.txt of given accession (using NCBIBrowser)
Download the report into an InputStream (using NCBIBrowser)
Read the InputStream using BufferedReader (using AssemblyReportReader)
Parse the report into AssemblyEntity and ChromosomeEntity Java objects (using AssemblyReportReader)

This functionality is presently being harnessed in various test cases. The goal is to expose this functionality to the user using an API. Doing this will require building these components:

AssemblyDao interface to define all functionality to be implemented.
FTPClientAssemblyDaoImplement that implements AssemblyDao.
AssemblyService is a Spring service that makes use of AssemblyDao.
ClientAPI is a Controller that lets user access AssemblyService through HTTP requests.

Completed and merged. Please move to done.


[feat] Add module types and modify nodejs, html5, grunt and java builds

Description
Add module types and modify nodejs, html5, grunt and java builds.
The new module types are: com.sap.portal.content, hdb, sitecontent, dwf (all run the npm builder).
html5, grunt and java builds were aligned to the old MBT behavior.
Added missing "npm install" to nodejs (it now runs the npm builder).
Checklist

[x] Code compiles correctly
[ ] Relevant tests were added (unit / contract / integration)
[ ] Relevant logs were added
[x] Formatting and linting run locally successfully
[x] All tests passing
[ ] UA review
[ ] Design is documented
[ ] Extended the README / documentation, if necessary
[ ] Open source is approved

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


DS-4107 Represent DSO metadata as a map in REST

REST Contract: https://github.com/DSpace/Rest7Contract/pull/39
This modifies the DSpace 7 REST API implementation to model embedded DSO metadata as maps, as described here: https://jira.duraspace.org/browse/DS-4107
Most of these changes were lifted from the POC PR: https://github.com/DSpace/DSpace/pull/2175  ...where representing metadata as a map was identified as useful keep.
Overall lines of code were reduced because the change provided an opportunity to share code in some places.
ITs have also been updated to expect the new form.

Thanks for taking a look @terrywbrady and @abollini . I have rebased on master and believe I have implemented all suggested improvements (mainly javadocs).
Note that the angular side of this was done a few weeks ago. I will shortly be rebasing that as well, after which I will link here. I do recommend this only be merged when the related angular changes are also ready to be merged.

Angular implementation has been rebased and is ready for review: https://github.com/DSpace/dspace-angular/pull/347

Rebased to latest master

Merging as this is at +2, and looks good to me too!


Read text from stdin

I added functionality to enable piping text into typer instead of having to rely on babble or an input file, as specified in #2

I also changed default -c and -p flags functionality to work more intuitively.

Thank you for the changes! I really appreciate the stdin functionality, it's awesome! (Maybe we should add some docs in the README.md about it).
For the flag changes, I'm not so sure the changes are intuitive, could you explain it to me? monkeytype.com and keybr.com both start off with all lowercase and no punctuation for their typing tests so I think typer should do the same. Then, if people want punctuation or capitalization they can opt-in.

For the flag changes, I'm not so sure the changes are intuitive, could you explain it to me?

Since we are using stdin now it seems strange to me that by just using typer without any flags input from stdin is stripped from any punctuation and swapped to lowercase, when it could be intended that way. I thought that input to the program should be consicered as user intended it to be.

For the flag changes, I'm not so sure the changes are intuitive, could you explain it to me?

Since we are using stdin now it seems strange to me that by just using typer without any flags input from stdin is stripped from any punctuation and swapped to lowercase, when it could be intended that way. I thought that input to the program should be considered as user intended it to be.

I think it's okay to filter the input in this case (for a typing test) since the user will probably use stdin to curl text from the web and cat files, etc... so it still makes sense for typer to use this input and remove punctuation and capitalization. If the user wants unchanged input then they can use the -cp flags. Currently it works this way for files as well with the -f flags. If users complain, then we can change this to your suggested solution, but I'd lean to keeping it as it currently is for now.

Then the changes can be merged without  commit e351378

Merged in #15


Movement?

Hi, I wonder if you've done any more work on this, or know of the existence of another project that might do this?

nope. :) sorry, i was just playing around, I have no intention to keep working on this.

Ah ok thanks :) Can I ask why?

just not using datomic, and have plenty of other things on my to-do list :/ Sorry I couldn't be more helpful - @bitemyapp is the only person I can think of who knows both datomic and Haskell, but I suspect he would tell you to run.

That makes perfect sense :) Was just curious, is all.
Haha yeah, he rather hates Datomic, I noticed. :) Not entirely sure why. I'm building trees, and in this case the standard relational table structure isn't really what I'm after. Also, I kind of fell in love with the idea that the db client is part of your codebase - it makes huge sense to me rather than use stored procedures or something ugly like that.
Thanks!

@JulianLeviston love doesn't make for a working database.

@bitemyapp indeed!

@bitemyapp by the way, if you feel like saying more, please do. If not, well that's fine, too :)


[MINOR]: fix min max accumulator nan bug

Which issue does this PR close?

Closes #13415
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Thanks @akurmustafa


Thanks for your consideration

https://github.com/comfusion/after-dark

Hello @jhabdas,
thank you for sharing your theme with the community :+1: . To take the last mile please update the screenshots in images with the required dimensions.
Thanks

All set @digitalcraftsman. I noticed the pixel density of the images was sitting at 144ppi so I bumped it down to 72ppi when resized.

Thinking of making better use of layout inheritance in the homepage index file to be more DRY. Should we hold off on the addition until I try the update?

Meh. Never mind. Fire when ready.

Great. I've merged your theme in 79dce826df0739374b4a060ce0cfe913c03b5c86. The theme site should list it soon.

That's great news! I had a blast creating it, and I appreciate your efforts
@digitalcraftsman
On Tue, Sep 20, 2016 at 7:44 AM digitalcraftsman<EMAIL_ADDRESS>wrote:

Closed #162 https://github.com/spf13/hugoThemes/issues/162.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spf13/hugoThemes/issues/162#event-795583341, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAa36hFqSkig6czDo1qq6dkFY284L8_sks5qr9VFgaJpZM4J_U6J
.

womp womp https://github.com/gohugoio/hugoThemes/commit/ef50a91d200fbe6820f0ba7146999337789dd1b8


Remove the JS module export

This: https://github.com/chris-pearce/backpack.css/issues/27 wasn't done properly, basically it was a bit of rushed job, therefore, it needs more care. This task is about removing it until it can be revisited.

Fixed in: https://github.com/chris-pearce/backpack.css/pull/31.


More Robust Handling of Mutes

The AcceptEvent() function in cmd/realy/app/implementation.go currently blocks events sent to the relay based on whether the event author's pubkey is in a mute list of one of the relay owners.
Given that mute lists are often inadvertently deleted as users move between various clients, this is not the most robust way to ensure mutes are correctly handled.
It would probably be better to allow owners of the relay to edit a custom mutelist (which may or may not itself be a Nostr event) that is used only by the relay.  This keeps a strong separation of concerns between the owners' mutes and the npubs which should be rejected by the relay.
Likewise, only accepting events from npubs on the owners' follow lists presents similar concerns.  Allow lists should be unique to the relay, rather than shared with the relay owners' npubs.

it would be possible to prevent the owner mute events being deleted and satisfy the same need here
the allow list only controls read/write access with auth enabled, it does not prevent writing of events from any other npub except the ones in the mute lists
i will make a patch to prevent the deletion of active owner mute lists in the accept event handling of deletes as this is where i can check the npub of the event

this has turned out to be a really important thing to check on, as the delete implementation is not enforcing safe invariants
it was:

allowing users to delete other users events
allowing the deletion of delete events
allowing the deletion of future event matches for the case of a tags
allowing a tags to match on non-parameterized replaceable event kinds

i have declined to implement the "k" tag delete feature as i think that is too indistriminate to be safe for users, they will not realise just how much it might destroy and a malicious party may do this for the lulz
the actual point of the issue though, i am resolving this by having the event IDs stored and a tags with follow and mute kind in a delete from the owners will not be honored either for the same reason as the issue

just to be clear, k tags are annotations, they don't mean "delete all this kind"


Timeout: Incomplete Dataset Export Download

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

So I've been trying to export one of my datasets for the past several days but in vain. Dumping annotations seems to be working fine, but exporting the dataset always results in a timeout halfway through the download. I've attempted this on two different networks and browsers with no success.
Expected Behaviour
Trying to export a dataset (4.1 GB) should download successfully, without any timeouts or errors.
Current Behaviour
Download timeouts after 2GB have downloaded and the resulting file is unusable.
Possible Solution
I've tried circumventing the timeout in multiple ways, which have all resulted in failure.
1 - I tried using wget in Colab to get around what I thought was slow internet, but the downloaded file is empty and of Length = 0 .
!wget --no-check-certificate  --load-cookies cookies.txt \
'https://cvat.org/api/v1/tasks/###/dataset?format=TFRecord%201.0&action=download'

2- Update: This only worked once and now times out at 2GB as well.
I tried using Selenium on Colab and it was successful:
from selenium import webdriver

options = webdriver.ChromeOptions()
options.add_argument('--headless')
options.add_argument('--no-sandbox')
options.add_argument('--disable-dev-shm-usage')

options.add_experimental_option("prefs", {
  "download.default_directory": r"/content/",
  "download.prompt_for_download": False,
  "download.directory_upgrade": True,
  "safebrowsing.enabled": True
})

wd = webdriver.Chrome('chromedriver',options=options)
wd.get("https://cvat.org/auth/login")

username = wd.find_element_by_id("username")
password = wd.find_element_by_id("password")

username.send_keys("username")
password.send_keys("password")
element = wd.find_element_by_css_selector('.login-form-button')
wd.execute_script("arguments[0].click();", element)

from selenium.webdriver import ActionChains
from selenium.webdriver.support.ui import WebDriverWait
from selenium.webdriver.support import expected_conditions as EC
from selenium.webdriver.common.by import By

wait = WebDriverWait(wd, 10)
actions = ActionChains(wd)

open = wd.find_element_by_css_selector("#root > section > main > div > div:nth-child(3) > div > div:nth-child(1) > div:nth-child(4) > div:nth-child(1) > div > a")
wd.execute_script("arguments[0].click();", open)

action_button = wd.find_element_by_css_selector("#root > section > main > div > div > div.ant-row-flex.ant-row-flex-space-between.ant-row-flex-middle.cvat-task-top-bar > div:nth-child(2) > button")
actions.move_to_element(action_button).perform()

export_data = wd.find_element_by_css_selector("body > div:nth-child(9) > div > div > ul > li:nth-child(3) > div")
actions.move_to_element(export_data).perform()

data = wd.find_element_by_xpath("//*[@id='export_task_dataset$Menu']/li[1]")
wd.execute_script("arguments[0].click();", data)

Steps to Reproduce (for bugs)

Create Task
Annotate a few things
Export dataset
Download timeouts before successful download.

Context
I can't export my data for use in research.
Your Environment
CVAT.org

Hi, can you download the dataset with web UI manually?
Also please specify the task id.

I can't download with the web UI manually which is why I resorted to my workarounds. The manual download results in truncation after 2GB have been downloaded and results in a file named 'dataset' that's empty. The task ID is 17431

Thanks for the report!

I can't download with the web UI manually which is why I resorted to my workarounds. The manual download results in truncation after 2GB have been downloaded and results in a file named 'dataset' that's empty. The task ID is 17431

I have the same issue when trying to export a dataset.  Reports downloading a 4GB file via Chrome, but the resulting 'dataset' file is empty.

@kevincolyar yes I still have the same problem in 2021. did you manage to find the solutoin?

@kevincolyar yes I still have the same problem in 2021. did you manage to find the solutoin?

@azhavoro is there any temporary workaround to download dataset?

@azhavoro is there any temporary workaround to download dataset?

I have the exact same issue, but with 1 GB. It downloads until 1 GB and then fails leaving a incomplete 1 GB file. I thought this might be related to https://github.com/openvinotoolkit/cvat/issues/956 but I could not find any info in the logs.
Temporary workaround could be to mount the docker volume and copy created zip by hand (before cache cleaner deletes it again). Going to try this now.

I have the exact same issue, but with 1 GB. It downloads until 1 GB and then fails leaving a incomplete 1 GB file. I thought this might be related to https://github.com/openvinotoolkit/cvat/issues/956 but I could not find any info in the logs.
Temporary workaround could be to mount the docker volume and copy created zip by hand (before cache cleaner deletes it again). Going to try this now.

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
 39 4200M   39 1667M    0     0  20.0M      0  0:03:29  0:01:23  0:02:06 20.5M
curl: (18) transfer closed with<PHONE_NUMBER> bytes remaining to read

Bug is still there. Even with blazing internet speed I'm only able to download 39%

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
 39 4200M   39 1667M    0     0  20.0M      0  0:03:29  0:01:23  0:02:06 20.5M
curl: (18) transfer closed with<PHONE_NUMBER> bytes remaining to read

Bug is still there. Even with blazing internet speed I'm only able to download 39%

I have also experienced this issue. I'm trying to export a 5.6GB zipped dataset and the download timeout is at 1GB always. Is there a workaround for this?

I have also experienced this issue. I'm trying to export a 5.6GB zipped dataset and the download timeout is at 1GB always. Is there a workaround for this?

@JobCollins Hi, try to adjust --limit-request-body value https://github.com/openvinotoolkit/cvat/blob/develop/supervisord.conf#L65. In the near future we plan to remove any data size limitations by default.

@JobCollins Hi, try to adjust --limit-request-body value https://github.com/openvinotoolkit/cvat/blob/develop/supervisord.conf#L65. In the near future we plan to remove any data size limitations by default.

@azhavoro thanks for this but my annotation job was done online (cvat.org). I resolved to use online after the local installation kept failing. Or could I sign in to my online profile via local? That would be great.

@azhavoro thanks for this but my annotation job was done online (cvat.org). I resolved to use online after the local installation kept failing. Or could I sign in to my online profile via local? That would be great.

Thanks a lot, @azhavoro . Using your advice,
1.I downloaded the annotations online (which has no issues downloading) then
2.uploaded the annotations to the offline installation and job
3.changed the local supervisord.conf code line 65
And I got my full dataset exported. Quite a workaround! 😁😅

Thanks a lot, @azhavoro . Using your advice,
1.I downloaded the annotations online (which has no issues downloading) then
2.uploaded the annotations to the offline installation and job
3.changed the local supervisord.conf code line 65
And I got my full dataset exported. Quite a workaround! 😁😅

Any updates on this? I have 6 datasets to export and they all expire...

@cyrilzakka Have you tried set --limit-request-body to 0 https://github.com/openvinotoolkit/cvat/blob/develop/supervisord.conf#L65 here as a workaround?

@azhavoro I'm using CVAT online so I don't think that's an option for me. And I don't have access to the dataset in order to do what @JobCollins proposed in the offline installation... so I'm basically stuck.

Is there a way of downloading an uploaded video from CVAT? In other words, when first creating the dataset, I uploaded an MP4 file. Since I can't export any of my data with more than 7K annotated frames, is it possible to download the original MP4?

@azhavoro I'm using CVAT online so I don't think that's an option for me. And I don't have access to the dataset in order to do what @JobCollins proposed in the offline installation... so I'm basically stuck. Any help would be greatly appreciated.

What do you mean "I'm using CVAT online"?
Do you have access to server where your CVAT instance is running?

@azhavoro I'm using CVAT online so I don't think that's an option for me. And I don't have access to the dataset in order to do what @JobCollins proposed in the offline installation... so I'm basically stuck. Any help would be greatly appreciated.

What do you mean "I'm using CVAT online"?
Do you have access to server where your CVAT instance is running?

I used https://cvat.org/ - so I don't have access to the server to be able to set --limit-request-body to 0

@azhavoro I'm using CVAT online so I don't think that's an option for me. And I don't have access to the dataset in order to do what @JobCollins proposed in the offline installation... so I'm basically stuck. Any help would be greatly appreciated.

What do you mean "I'm using CVAT online"?
Do you have access to server where your CVAT instance is running?

I used https://cvat.org/ - so I don't have access to the server to be able to set --limit-request-body to 0

I feel for you. Guess the only way is to try to contact owners of cvat.org and ask them for help. If i understand correctly - developers of CVAT (github issues here) are not the same persons as the owners of the website. Wish you good luck with your issue.

@lgg, we discussed internally and I'm going to disable this limit for cvat.org today

Great, thank you so much and I do apologize for all the trouble! Will you let me know when you disable the limit so I can download my datasets asap?

@azhavoro that would be great! Thanks @cyrilzakka for pushing this!

How to solve this problem in a running Docker hosted on a server. I have the access to the hosted server and i could end up loosing a lot of labelled images

Sorry to bring this up again, but has this issue been fixed? As of today, I'm still  receiving this error. Always around 46-48% of this dataset:

@cyrilzakka Yea, me too. Even when I set:  --limit-request-body 0 as suggested in [(https://github.com/openvinotoolkit/cvat/blob/develop/supervisord.conf#L65)]
My downloading annotation file is stopped when it reaches about 1GB.

Even tried -C option on curl but still can't get my data. At this point I've exhausted all of my options :(

Is there a real solution for this issue? I cannot download my dataset which is above 1GB.

@memirerdol I've been waiting for a solution for 10 months now while my data is being held hostage and my research paper is on hold because of it. I would've understood had they mentioned data above a certain size couldn't be downloaded but they failed to mention it anywhere... It's super frustrating but there's nothing to be done

@memirerdol @ConstantSun For some odd reason, curl worked for me today - I was able to download 64 GB worth of my annotated data. Regular downloads using the GUI still times out after 1GB

Was having the same issue on my local setup using the default docker-compose setup and I discovered that the problem seems to be with the cvat_proxy container which is running an nginx http proxy.
proxy_buffering is enabled in which case it will (try to) cache large reponses to a temporary local file in the cvat_proxy container, and the default setting for the maximum buffered file size is 1GB. I wasn't sure if I should completely disable buffering to a file because I didn't want to stall any worker threads in the backend, so I just bumped the setting to 10GB.
Index: cvat/cvat_proxy/nginx.conf
===================================================================
--- cvat.orig/cvat_proxy/nginx.conf
+++ cvat/cvat_proxy/nginx.conf
@@ -15,4 +15,5 @@ http {

     include /etc/nginx/conf.d/*.conf;
     client_max_body_size 0;
+    proxy_max_temp_file_size 10240m;
 }

Until this issue is resolved, you can use this cURL command to download datasets from projects (as COCO):
curl 'https://cvat.cargocult.tech/api/projects/<project_id>/dataset/?action=download&filename=export&format=COCO%201.0&location=local&use_default_location=true' \
  -H 'accept: application/vnd.cvat+json' \
  -H 'Authorization: Basic cnlhbjpOaWhvbkcwIQ==' -u <user:password> -o out.zip

Hi all,
a possible workaround to this problem would be to download only the annotations and then extract the corresponding frames from the mp4 files outside CVAT when needed. This is possible if we know how the frames are extracted by cvat to create the tasks.
Can someone link the exact command used to extract the frames?


Uncaught TypeError: string is not a function

Uncaught TypeError: string is not a function
calendar.js line 432
this.context.append(this.options.templatesthis.options.view);

I am having the same issue here, and I double checked my tmpls path and it is the right path


How to force the video quality?

Hi,
I use your player and it work really well!
I just have an issue with the video quality, when I'm on wifi the video start with high quality and when I'm on my mobile network (4G+ better than my wifi) it start with low quality, really low quality. Is it possible to force the quality when I launch the video?
Thanks

I have the same problem....any solution for that? Maybe it is a solution to append ?vq=hd720 to the Youtube ID...will check it out...


Allow changing default install location of apps

To my surprise, this seems to be a common request :)
Premise: allow users to change the install location of applications
Current Workaround: use appget install <package-id> -i to launch the installer in interactive mode and change the location manually.
Installers that seem to support changing the pack:

MSI INSTALLDIR="PATH"
Inno /DIR="PATH"

Needs Research

Wix
InstallShield
NSIS
InstallBuilder
AdvancedInstaller
Squirrel
SetupFactory

This article may be helpful in the case of the Wix installer https://wrightfully.com/allowing-the-user-to-select-the-install-folder.