Yosys Verilog frontend: module parameters

It looks like Yosys frontend for Verilog is not fully compatible with the Verilog output of GHDL synth when it comes to module parameters.
It chokes on the following:

module ODDRX1F
#(
GSR)
(input  SCLK,
input  RST,
input  D0,
input  D1,
output Q);
endmodule

with the following error:

Test.v:3: ERROR: In pure Verilog (not SystemVerilog), parameter/localparam with an initializer must use the parameter/localparam keyword

I'm not good enough at Verilog to know what Verilog standard the above is compliant with, or if it's even standard Verilog. But GHDL synth generates this kind of module definitions for VHDL components that have generics.

You can try using yosys in SystemVerilog mode, via the -sv option to the Verilog frontend (or via naming the input file something.sv).
Better yet, I recommend using the yosys GHDL plugin instead of going from GHDL to yosys via Verilog.

I tried having Yosys use the SV mode, but then it issues another error on the line that instantiates the above module:

ODDRX1F sdram_clock #(
.GSR("01000101010011100100000101000010010011000100010101000100"))
(

with the following error:

Test.sv:689: ERROR: syntax error, unexpected '#', expecting '(' or '['
Parsing SystemVerilog input from `Colorlight_i5_Test2.sv' to AST representation.

I'll have to try the yosys GHDL plugin, I assumed there was no real difference. Does the plugin not use Verilog as an intermediate?

Now that looks like plain old incorrect code (the #(...)param assignment is supposed to come before sdram_clock instance name).
The GHDL plugin goes directly from GHDL IR to yosys IR, it doesn't involve Verilog.

Also the value given for the parameter is definitely incorrect — it somehow converted an "ENABLED" string into its binary representation, then emitted that as a string, there's no way this could possibly work.

Thanks! I'll have to report this to the GHDL team I guess.
Using the yosys GHDL plugin, yosys completes successfully.


Update index.md

thus is very well put toghthter, better than mine in may ways and I will rephrase many of the things I have typed so far

yeah, add in your degree, and i think its great!


Local storage

Local storage genre now saves on the page after refresh.

tried to merge to wrong branch.

resolved conflict with HTML


MSID: 89210 Version: 1 DOI: 2<IP_ADDRESS>9681

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=89210)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/89210/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[ ] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9681)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/89210/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/89210)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/89210

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/89210

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/89210/v1/89210-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/89210)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Semi-colon on the end of the first affiliation. OK to leave


BLS24/Frobenius

Refactor Fp24 Frobenius (inlined constants and computation).

Do not merge, as we might only need Frobenius,  FrobeniusSquare and FrobeniusQuad following https://eprint.iacr.org/2020/875.pdf

Ready for review


F# can't produce default interface methods

The remark section on the page about F# interfaces seems to allude to a capability that I don't think it has:

However, you can provide a default implementation by also including a separate definition of the member as a method together with the default keyword. Doing so is equivalent to creating a virtual method in a base class in other .NET languages. Such a virtual method can be overridden in classes that implement the interface.

I may be misinterpreting this; I was surprised to see the git blame for this paragraph is quite old. F#5 can use C# default interface methods, but it cannot provide them.
Related: https://github.com/fsharp/fslang-suggestions/issues/679
If the existing remark is in fact wrong, I would of course like feedback on the wording of the clarification (in PR).

@dsyme For review


An object-as-map's value type is ignored when returning the object from a function

Flow version: 0.157.0, 0.154.0
Expected behavior
This should fail to typecheck, as the returned object's property value is not a string:
function makeMap(key: string): {|[string]: string|} {
  return {
    [key]: 999
  }
}

Actual behavior
No errors.

Link to Try-Flow or Github repo:
https://flow.org/try/#0GYVwdgxgLglg9mABAWwIYGsCmBZVAHACiwE8AuRAZygCcYwBzASnIG8AfAbStoYF1zudemwC+iFgChEiapighqSSdOkcS-RAE5tUxCIn6gA

https://flow.org/try/#0CYUwxgNghgTiAEA3W8DWICeAueBnALjAJYB2A5gNwBQAFAN4Da6GAujgJycC+OjBx5NnkKkyXAJQUgA minimized repro


Pageres still not showing new options

Dear developer,
Thanks for enhancing pageres. But i could not see new feauture after install. How to get new feauture show in command line.
Still its showing old options
Options
-v, --verbose            Verbose output
-c, --crop               Crop to the set height
-d, --delay=    Delay screenshot capture

After installation, use pageresi as cmd name. It's a doc bug.
As a temporary modfication for our using, this repository hasn't that good quality. Maybe Issues at Pageres(https://github.com/sindresorhus/pageres-cli) is better choice. Sorry for that.


Closing a scene tab using shortcut always re-saves the scene

Godot version
3.4
System information
macOS
Issue description
Closing a tab by clicking on the X will close it immediately if the scene is already saved. Using the shortcut Shift + Command + W for Close Scene will save the scene again, even if there are no changes, making the process much slower
The problem increases when you are closing multiple tabs since you have to wait for the saving dialog for each tab, so closing several tabs requires more time than it should
Steps to reproduce
Open multiple scenes and close them using the Close Scene shortcut
Minimal reproduction project
No response

The Save Scene editor action always attempts to save the scene, even if the scene isn't marked as modified (* next to the tab name). This is because the modified marker isn't 100% reliable in all cases (e.g. with tool scripts or modified resources), and we prefer erring on the side of caution.

This suggests two solutions:

Command + W becomes the equivalent of clicking the tab's X to close it immediately while Shift + Command + W (Close Scene or Save Scene editor action) remains the same calling save always
Fix the problem of the modifier marker not being reliable. I'd imagine an easy solution would be to hash the file and compare it, if the contents have been changed (hashes are different) then save again (tscn files are saved as text, so there should be no problem hashing the files)

Fix the problem of the modifier marker not being reliable. I'd imagine an easy solution would be to hash the file and compare it, if the contents have been changed (hashes are different) then save again (tscn files are saved as text, so there should be no problem hashing the files)

Resources referenced in scene files can be saved to external files, so this won't always work in practice.

Fix the problem of the modifier marker not being reliable. I'd imagine an easy solution would be to hash the file and compare it, if the contents have been changed (hashes are different) then save again (tscn files are saved as text, so there should be no problem hashing the files)

Resources referenced in scene files can be saved to external files, so this won't always work in practice. When the external resource file changes, the reference doesn't change in the scene file.

What do you mean with 'the reference doesn't change in the scene file'?
It could still be a problem to solve, to correctly detect changes. But I think the focus of this issue is that the shortcut is slower than clicking the tab's X button.
I'm not sure then when or how the engine decides when to save scenes or not when closing tabs (it becomes confusing and non-intuitive), what happens if I close the whole project, what happens if I just click the tab's X.
In the end, I'd prefer to close tabs with the standard shortcut Command + W rather than having to use the mouse. Especially for closing multiple tabs

This was resolved in 4.0 by #40525


Error: Vault is already initialized

I have a vault server running at localhost:8200
going through the example code is giving me the Error: Vault is already initialized
var vault = require("node-vault")();

vault.init({ secret_shares: 1, secret_threshold: 1 })
.then((result) => {
  console.log(result);
  vault.token = result.root_token;
  const key = result.keys[0];
  return vault.unseal({ secret_shares: 1, key });
})
.catch(err => console.error(err));

this also doesn't work for me
var vault = require("node-vault")();

vault.initialized()
.then((result) => {
  console.log(result); // { initialized : true }
  return vault.init({ secret_shares: 1, secret_threshold: 1 });
})
.then((result) => { // doesn't resolve because vault is already initialized
  console.log(result);
  vault.token = result.root_token;
  const key = result.keys[0];
  return vault.unseal({ secret_shares: 1, key });
})
.then(console.log)
.catch((err) => console.error(err.message));

how do i get the root_token?
I am using boostport/kubernetes-vault
so i do have access to the vault-token.
{
   "clientToken":"91526d9b-4850-3405-02a8-aa29e74e17a5",
   "accessor":"476ea048-ded5-4d07-eeea-938c6b4e43ec",
   "leaseDuration":3600,
   "renewable":true,
   "vaultAddr":"https://vault:8200"
}

is there a way i can read vault secrets with the clientToken?
var vault = require("node-vault")({
  token : clientToken,
  endpoint : vaultAddr
});

vault.initialized()
.then((result) => {
  console.log(result); // { initialized : true }
  return vault.read('database/config/prod-pg');
})
.then(dbConfig => {
  console.log('DB Config', dbConfig);
})
.catch((err) => console.error(err.message)); // Error: permission denied

returns Error: permission denied

For the first issue you mention, seems like you are trying to initialize an already initialized vault instance. Check the vault status with the vault cli to assert this.
For the second issue regarding using the client token to retrieve a secret, that seems like an issue of the client not having the needed permissions. You can also check this by using the vault cli again, authenticating with the client token and trying to retrieve the secret.

I'm closing this for now as it seems like the problem is not node-vault related.

yes you were right. the clientToken did not have sufficient access to get that secret.
i updated the policy so the appRole is authorized, and tested with the cli as you said.
everything works now. Thanks!
i do think the docs and examples could be a bit more descriptive.


Fixed ts_utils.get_node_text deprecation

Hi @ItsSunnyMonster,
I've been needing to remove this deprecation notice for a while but didn't have time. Thanks for taking it up. But I've noticed that some of the unit tests are failing. Could you please take a look at it as well? Otherwise, I'll check on it on my free time

Hi,
I looked over the unit tests and it seems like the failing test cases were def_constructor, destructor and copy_constructor. In all 3 of these cases, it fails because the return_node passed into get_node_text is nil (at line 158 in my fork and 148 in this repo). That does make sense since constructors and destructors don't have return types.
I don't fully understand how Neovim plugins work, but my guess is that the old function handled the case where node is nil slightly differently from how the new function handles it? What was the return value of the old function when node is nil? I could perhaps add a check for nil and return this value in my wrapper around the new function.

Alright, I found the implementation of the old function online and it turns out it simply returns an empty table if node is nil. I have added that to my branch and if that was correct all the unit tests should pass.

Thanks for help and the fix


Problem with Docker.build Vapor project with qemu

Hello, I'm having problems building the Vapor project.
Relation: https://github.com/vapor/template/issues/96
I would like to clarify whether you know about such a problem, is there a solution, perhaps I didn’t see it, please help, I can’t move on.
I also tried Swift:latest and also have problems building the docker image.
Thank you, I hope for an answer

Log trace:

[build  7/12] RUN swift build -c release --static-swift-stdlib     -Xlinker -u -Xlinker _swift_backtrace_isThunkFunction:
32.16 Building for production...
33.94 [0/1285] Compiling _NumericsShims _NumericsShims.c
33.94 [1/1285] Compiling errno.c
34.08 [2/1285] Compiling a_d2i_fp.c
34.21 [3/1285] Compiling a_utctm.c
34.24 [4/1285] Compiling a_type.c
34.37 [5/1285] Compiling a_time.c
35.06 swift-runtime: unable to suspend thread 231
35.06 swift-runtime: unable to suspend thread 231
35.06 qemu: uncaught target signal 4 (Illegal instruction) - core dumped

My Dockerfile
Dockerfile.txt

This solve my problem


Better query metadata errors (#837)

@kyleconroy this pull request also includes tests, let me know if there is a problem, happy to fix, thanks!

@kyleconroy this pull request also includes tests, let me know if there is a problem, happy to fix, thanks!


Is WSL2 on windows server 2022 21H2 available for gpu access?

Env: win serve r2022 (21H2) + WSL2（5.4） + ubuntu 20.04 + cuda11.0
When I use "nvidia-smi" , it shows:
Failed to initialize NVML: GPU access blocked by the operating system
Failed to properly shut down NVML: GPU access blocked by the operating system

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

ID: 0fdfa117-e801-0df5-3c81-3151a97f431d
Version Independent ID: ee755d20-17c2-3d18-0c5c-b9ed31135218
Content: Enable NVIDIA CUDA on WSL 2
Content Source: docs/directml/gpu-cuda-in-wsl.md
Product: machine-learning
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @joshuia, and thanks for filing this issue.
I believe that there's nothing incorrect in this specific topic, so this doesn't appear to be a docs issue.
You could try sharing feedback on NVIDIA's support via their Community forum for CUDA on WSL.
The feedback channel you're using here (Submit and view feedback for > This page) is for feedback about the documentation in this topic itself, as published. For example, if the content is incorrect or incomplete.
For questions (or requests for help) regarding the way a Microsoft software product behaves, one option is to click Submit and view feedback for > This product, and you'll be taken to Microsoft Q&A (more info about that below).
Here are all of the developer support resources, for completeness.

Microsoft Q&A is the right place to lobby for a change or a new feature, to make a suggestion, or to ask a product question.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

Thanks!
-Steve


Bump AWS and OrderedCollections versions

Codecov Report
Patch coverage has no change and project coverage change: -9.13 :warning:

Comparison is base (a2f0054) 95.78% compared to head (5e7f580) 86.66%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   95.78%   86.66%   -9.13%
==========================================
  Files          35       35
  Lines        4036     3930     -106
==========================================
- Hits         3866     3406     -460
- Misses        170      524     +354

see 26 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Asset Index Pagination (closes #178)

Signed-off-by: Dominik Pinsel<EMAIL_ADDRESS>

I think you need to rebase against main because InMemoryAssetIndex changed a bit.
For the rest, introducing AssetStorage seems like a bit of unneeded indirection here.
I'm still dubious about the need for the Cursor thing, wouldn't the offset/limit pattern just enough?

Offset would only work with storage systems where all items are in some kind of "list". E.g. if the items are stored in a tree an offset cannot be used. There may also be a problem with the offset when new items are not happened at the end of the list.
Regarding the AssetStorage. I assume it's cleaner if the Asset items are not stored in the AssetIndex itself. Like this one objects does the storing of the assets, and the other one creates the index and makes them queryable by the EDC. I assume in a productive implementation this both cases would be put in separated classes, too.


Unable to find 1.0.0 build

Attempting to pull in this recent build 1.0.0, does not find anything, resolves to 0.0.1.

Sorry about that! Everything should be good to go now.


Only check REST arguments if the file does not exist

Description
Fix for #4084. Will now check if the given filename exists before checking for REST parameters.
Tests
I did not look into test cases for this, but I ran the following command before and after the change:
Before:
bin % ./oiiotool \?.png -o \?-output.png
oiiotool ERROR: read : "?.png": ImageInput::create() called with malformed filename
Full command line was:
> oiiotool ?.png -o ?-output.png

After:
bin % ./oiiotool \?.png -o \?-output.png

No output, ?-output.png created.
Checklist:

[x] I have read the contribution guidelines.
[ ] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[ ] If I added or modified a C++ API call, I have also amended the
corresponding Python bindings (and if altering ImageBufAlgo functions, also
exposed the new functionality as oiiotool options).
[x] My code follows the prevailing code style of this project. If I haven't
already run clang-format before submitting, I definitely will look at the CI
test that runs clang-format and fix anything that it highlights as being
nonconforming.

The solution looks right to me!
Can you please amend to fix the DCO signoff? The easiest way would be
git commit --amend -s
(just save, you don't need to change anything, the -s adds the sign-off)
git push origin fix/imageinput-rest-check  --force

Should be done!


STM32: Runtime GPIO clock gating

Based on https://github.com/zephyrproject-rtos/zephyr/pull/32390
This PR demonstrate a runtime GPIO clock gating solution.
With this solution GPIO clocks are only on when they are required and released as soon as there are no more active clients.
This is the case when gpio clients are calling:

gpio_config with GPIO_DISCONNECTED flag
gpio_interrupt_configure with GPIO_INT_MODE_DISABLED
For pinmux, gpio ports are clocked on only during configuration and released once done.

While this PR allows to test impact of gpio clocks on power saving, it is not advised to be used in production
in multithreaded context. A safer implementation would make use of on/off api (https://github.com/zephyrproject-rtos/zephyr/pull/21090).
Basic tests showed 1600uA savings on blinky application on nucleo_l476rg.

@pabigot I'm looking at how to convert this to pm_device. Looking to what is available in device_pm, the sys on/off api (#21090) feels a bit redundant.
Is that the correct understanding that device_pm would be enough for proper handling of clocks in multi-threaded context ?

@pabigot I'm looking at how to convert this to pm_device usage. Looking to what is available in device_pm, the sys on/off api (#21090) feels a bit redundant.
Is that the correct understanding that device_pm would be enough for proper handling of clocks in multi-threaded context ?

From my perspective there's been no validation that what device_pm currently does is what Zephyr needs.  But certainly turning devices on and off is what sys on/off was intended to support, so if device PM were to be a wrapper around it it that would make sense.
I hope device PM isn't going to continue to use a separate solution.

Closed in favoir of https://github.com/zephyrproject-rtos/zephyr/pull/32788


reward_trainer.py having RuntimeError: Tensors must be contiguous errors during evaluation

Hi, I am running the reward_trainer.py example, and was trying to run with the command torchrun --nproc_per_node=8 reward_trainer.py I only modify the eval_split to 'test' so that it will have the eval data available. I did not change the dataset. I set it to run 3 epochs, and the evaluation strategy is set to epochs, so I expect evaluation will happen after each epoch after training.
The training is successful, however the evaluation always crashed due to the RuntimeError: Tensors must be contiguous errors mentioned above. More detailed error logs are as follows.
  File ".../venv/lib/python3.10/site-packages/accelerate/utils/operations.py", line 289, in gather
    return func(data, *args, **kwargs)
  File ".../venv/lib/python3.10/site-packages/accelerate/utils/operations.py", line 266, in _gpu_gather_one
    return _gpu_gather(tensor)
    torch.distributed.all_gather(output_tensors, tensor)  File ".../venv/lib/python3.10/site-packages/accelerate/utils/operations.py", line 269, in _gpu_gather

  File ".../venv/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 1451, in wrapper
    return func(*args, **kwargs)
  File ".../venv/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 2448, in all_gather
        output = eval_loop(return recursively_apply(_gpu_gather_one, tensor, error_on_other_type=True)

  File ".../venv/lib/python3.10/site-packages/transformers/trainer.py", line 3147, in evaluation_loop
  File ".../venv/lib/python3.10/site-packages/accelerate/utils/operations.py", line 128, in recursively_apply
    return func(data, *args, **kwargs)
  File ".../venv/lib/python3.10/site-packages/accelerate/utils/operations.py", line 266, in _gpu_gather_one
    return func(*args, **kwargs)
  File ".../venv/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 2448, in all_gather
    torch.distributed.all_gather(output_tensors, tensor)
  File ".../venv/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 1451, in wrapper
    work = default_pg.allgather([tensor_list], [tensor])
RuntimeError: Tensors must be contiguous

I searched online and see this can be changed by adding the tensor.contiguous in the package before the line that has errors. However, I wonder is there ways to fix it in the reward_trainer.py file instead of go fix the package. Thank you for reading this.

updating accelerate package solves it for me

Hi @jltchiu
Can you try to upgrade accelerate package and re-run your script with accelerate launch xxx.py (after calling accelerate config) and selecting "multi-GPU"

closing the issue for now, feel free to re-open if you think this is not addressed


Could you publish fanuc-driver on nuget.org?

It might be a good idea to publish fanuc-driver on nuget.org for easier installation.
At the same time, you might want to create GitHub Releases too. :wink:

Would I be versioning the entire codebase?

Well, I’d version fanuc-driver itself.
However, now that I think about it, IMHO you created collectors/strategies/vendors/handlers (or at least some of them) as they could/should be configurable by fanuc-driver user. If that is the case, fanuc-driver they should be removed from the codebase.
Actually, even nlog.xml and config.yml should not be in the codebase (examples could be there, e.g. in an example/ folder or with .example extension).
I suggest to publish it on nuget.org so that we could simply install it without manually building it (unless we want to modify some code). Versioning would help us install a specific version (or even latest stable, dev, rc).
As I see it, fanuc-driver is an adaptor (it adapts all the different FOCAS libraries, as well as makes them working even on platforms not supported by Fanuc) and a microservice (it gathers, processes and provides some data from machines). Maybe the code should be split; although you already have base-driver, however I think the code should be split as follows:

fanuc-adaptor:

it could include current base-driver, all FOCAS libraries;
it would provide low-level stuff like get machine data and small amount of pre-processing (like transforming array of letters into a single string);
think of this adaptor like a database driver;

a microservice:

it would use fanuc-adaptor;
it would include current higher level processors (veneers and strategies?), run sweeps, process the data more intensively (like diffing two consequent sweeps), puplish the data via MQTT;
I think that the entire configuration (like what should be gathered and/or published) should be outside of it, so that the user could change it either using a config file or runtime using a CLI command or using some kind of API call (or publishing a particular topic with a specifically structured message);
think of this like an ORM.

Note that is how I’d like fanuc-driver to work. It does not talk about the implementation. You can dismissed it at will.

I agree on all of the above.  Even MQTT should be removed from the adapter.

Even MQTT should be removed from the adapter.

It should be used as an external library/package, but IMHO it should be utilised by the microservice (not fanuc-adaptor).

I was thinking about the different consumers of the data.  Some people will want it in SQL, some will want an HTTP endpoint, some will want something else.  I think MQTT, although external to the adapter, should be the primary mode of transmission of the data from fanuc-driver to another process.... unless a pattern is defined for 'output plugins'.
I reviewed the concepts of base-driver across the FactoryIO and OPCXMLDA drivers.
https://docs.ladder99.com/en/drivers/base-driver
I guess I meant for the handler to maintain whatever type of connection to an external system.

Even MQTT should be removed from the adapter.

It should be used as an external library/package, but IMHO it should be utilised by the microservice (not fanuc-adaptor).

https://github.com/Ladder99/fanuc-driver/commit/b9679448a203f04f216bee73058bc99a3cbb6777


install instructions for specific versions

Draft 1.0.0 release here https://github.com/CITCOM-project/CausalTestingFramework/releases/edit/untagged-09a6985599a8f8f7aa2d should generate zip and tar.gz on publish. We would still expect people to install from github clone rather than these files. Instructions for this are in this PR. They're too clunky so we need to get on PyPI soon.

If you’re worrying a lot about backwards compatibility, you should probably already be 1.0.0.

I am 😄


fixed invalid credentials

The issue was that our backend had a load balancer.
The authentication begins when the user is redirected from the Relying Party (RP) to the OP (OpenID Provider). The OP then authenticates the user and redirects the user back to the RP with an authorization code. The problem that I was experiencing was that the OP was sending the user back to the server IP in the redirect_uri. However, the server IP is not the same as the RP address. This is because the reverse proxy is the one that is receiving the incoming requests.
As a result, when the user is redirected back to the server IP, the server does not know where the OIDC authentication was initiated. This is because the server does not have the session cookie that was set by the RP. Hence we got a authorization_request_not_found error.
Ibrahim Iqbal — 03/10/2023 15:44
To fix this problem I just added the following 2 lines in my application.yml
server:
forward-headers-strategy: native


Could this runner be integrated with GitHub Actions?

Hi @adrienbrown2020, are you using Hydra Lab to test an Android app or an iOS app?
For Android app test only, we have an Gradle plugin ready for use, see details here: Gradle plugin README.
You may add a new "Gradle Build Action" task in your Git Actions workflow, a sample of the yml content as below:
- name: Gradle Build Action
  uses<EMAIL_ADDRESS>  with:
    # Gradle command line arguments (supports multi-line input)
    arguments: [your gradle arguments after command "gradle"]

If you are using Azure DevOps pipeline, you may directly add the gradle task within it, see details here: Trigger a test task run in the Hydra Lab test service.
An alternative Azure DevOps extension is also available for ADO pipeline, if you want to test an iOS app (or Android app as well), you may search for extension "Hydra Lab" in ADO marketplace, and involve the task accordingly.


Releases should be created...

For every version of the software (10.3.0, 10.3.1, etc.) there should be a corresponding release in this repository. The downloads on the download page should have the same code.

:+1:

Same from me.  I saw the adapter is on the product page, but no processor?

FYI @nfeuerstein


issue with exercise 4.1

My code gives the correct answer but I got failed the OK test:(
Could anyone help me with that?
My code:
from itertools import islice
import re
def read_constants(file_path):
    with open(file_path,"r") as infile:
        constants = {}
        for line in islice(infile,2,None):
            words = re.split('\  +',line)
            k = words[0]
            value = float(words[1])
            constants[k] = value

Error messages:
Running tests
---------------------------------------------------------------------
question 4.1 > Suite 1 > Case 1
d1 = read_constants('data/constants.txt'); d2 = {'gravitational constant': 6.67259e-11, 'Boltzmann constant': 1.380658e-23, 'speed of light': 299792458.0, 'Planck constant': 6.6260755e-34, 'electron mass': 9.1093897e-31, 'Avogadro number': 6.0221367e+23, 'elementary charge': 1.60217733e-19, 'proton mass': 1.6726231e-27}
>>> len([1 for k in d1 if k in d2 and d1[k]==d2[k]]) == len(d2)
TypeError: 'NoneType' object is not iterable
# Error: expected
#     True
# but got
#     Traceback (most recent call last):
#       ...
#     TypeError: 'NoneType' object is not iterable
Run only this test case with "python3 ok -q exercise-4_1 --suite 1 --case 1"
---------------------------------------------------------------------
Test summary
    Passed: 0
    Failed: 1
[k..........] 0.0% passed

@marijanbeg 😞

Hi @gems-27da458d, thank you for your question. Could you please show us the output of your function?
read_constants('data/constants.txt')

Hi @gems-27da458d, thank you for letting us know. Okpy tests pass, which means that your solution is correct. Unfortunately, we cannot include more PyBryt references which is the reason PyBryt is not happy. You should not worry about this. :)


I cannot delete a key

@mafintosh I am excited to finally have started to play around with hyperdb. It's fun.
I have begun to write a simple graph database on top of it (using levelgraph's triple store spec as a starting point - https://github.com/e-e-e/hyperdb-graph). Although I have almost immediately hit the problem that delete is not implemented yet in hyperdb. On the plus side - I have gotten hyperdb-graph passing almost all the core triple store tests - excluding deletion and levelgraphs filtering options.
Also - It looks like hyberdb.batch() ignores the type property entirely and just call the put method -
https://github.com/mafintosh/hyperdb/blob/master/index.js#L94. Is the intention that in the future we will be able to specify del as a valid type? Or should I not rely on this for now.
Whats the plans for implementing deletion generally - anything I can do to help?

I think @mafintosh is working on deletions for v2.

@noffle - I just realised that this is not a real issue. Obviously there can be no real deletions as we are dealing with append only logs. So I managed to implement delete just by putting null values. Then when reading back from stream its just a matter of checking if the latest node for that key has a value of null. If so - that key has been deleted.
You can close this issue, unless @mafintosh wants to track the feature still.

deletes are in master


Cannot achieve the reported accuracy 0.844 with wide_n_deep_tutorial

After running the code in "wide_n_deep_tutorial.py", I achieved accuracy 0.834 for wide-only model, but the achieved accuracy for wide-and-deep model was only about 0.82.
I didn't change anything in the source code and just couldn't achieve the accuracy 0.844 as reported on the website. Can anyone help me to tell where could sth be wrong?

@ispirmustafa Is that within noise?

@Sean-Git I tried that and I get about 84.7% accuracy when train steps = 200, and increase train steps to 800, 1000, or larger, it even can up to 85.5%. so, I think you might check your code and ensure there are no modifications on key code.

closing based on answer from @whatbeg


Prevent activating an already activated loans market

The activate_market extrinsic should fail if the market is already active, so as not to re-emit the ActivatedMarket event.

Solutions
Solution 1: Pass is_active flag to mutate_market
Pros:

Ensures the activation status is explicitly passed as a flag to the mutate_market function.

Cons:

Every mutate_market query will now need to pass a flag, potentially increasing complexity and maintenance overhead.

Solution 2: Add an is_active flag when updating mutate_market
Pros:

Allows updating the mutate_market function to include an is_active flag.
Only emits an event if the flag is not set, reducing unnecessary event emissions.

Cons:

If additional conditions need to be checked in the future, adding more flags can cause issues and make the code more complex.

Solution 3: Add the check_market_state_before_activating function
Pros:

Introduces a separate function, check_market_state_before_activating, to ensure all necessary checks are performed before updating the market.
Provides flexibility to modify the helper function in the future if more ensures are needed.

By implementing Solution 3, the code can benefit from a dedicated function that handles the market state checks, ensuring proper validation before activating the market. This approach offers flexibility for future modifications and enhances code readability.


Resources can not be found at compile time when added as dependency (Scala 3)

Scala 3 version of Dumbo.withResourcesIn is not able to list resources that are added as dependency at compile time (mentioned in https://github.com/rolang/dumbo/issues/79#issuecomment-2410443413 ).

:point_up: cc @taig it should be fixed now from v0.5.2.
Let me know if you encounter any other issues.

Unfortunately, it doesn't work for me in a real world scenario:
//> using repositories sonatype-s01:snapshots
//> using dep dev.rolang::dumbo:0.5.2
//> using dep de.hellobonnie::swan-mock-server:0.0.0+259-010141f9-SNAPSHOT

import _root_.dumbo.ResourceFilePath

ResourceFilePath.fromResourcesDir("migration/swan")

scala-cli dumbo.sc
Compiling project (Scala 3.5.2, JVM (21))
[error] ./dumbo.sc:7:1
[error] Exception occurred while executing macro expansion.
[error] java.nio.file.NoSuchFileException: [...]/Library/Caches/Coursier/v1/https/s01.oss.sonatype.org/content/repositories/snapshots/de/hellobonnie/swan-mock_3/0.0.0%252B259-010141f9-SNAPSHOT/swan-mock_3-0.0.0%252B259-010141f9-SNAPSHOT.jar
[error]
[error] ResourceFilePath.fromResourcesDir("migration/swan")
[error] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The problem seems to come from getClass().getClassLoader().getResources which returns swan-mock_3-0.0.0%252B259-010141f9-SNAPSHOT.jar, but the correct path / file name is swan-mock_3-0.0.0%2B259-010141f9-SNAPSHOT.jar. I failed to find out why it's doing that.

@taig Ok I see. Thanks for providing the example, it helped investigation.
TL;DR It seems to be caused by double URL encoded + sign in the name. Removing the + from the version tag should also make it work.
Looks like coursier is storing the file name URL encoded. So a file name like swan-mock_3-0.0.0+259-010141f9-SNAPSHOT becomes 0.0.0%2B259-010141f9-SNAPSHOT on the file system. The + sign gets encoded to %2B (for encoding reference).
getClass().getClassLoader().getResources returns a collection of URLs, I guess when it get a file name like  0.0.0%2B259-010141f9-SNAPSHOT it applies URL encoding again and the % becomes %25, that's how it ends up with %252B instead of just %2B.
Can't tell for sure whether the file name should not have been URL encoded in the file system or getResources() shouldn't encode it again. I guess one could add a small workaround for that in dumbo to get it working :thinking: .
In any case it doesn't seem to be related to the usage of macros, I get the same behavior when applying it at runtime, the resources can't be found due to double URL encoded path.

Should be working now.
//> using repositories sonatype-s01:snapshots
//> using repositories sonatype:snapshots
//> using dep dev.rolang::dumbo:0.5.2-4-247449e-SNAPSHOT
//> using dep de.hellobonnie::swan-mock-server:0.0.0+259-010141f9-SNAPSHOT

import _root_.dumbo.ResourceFilePath

println(ResourceFilePath.fromResourcesDir("migration/swan"))

scala-cli dumbo.sc
Compiling project (Scala 3.5.2, JVM (21))
Compiled project (Scala 3.5.2, JVM (21))
List(/migration/swan/V0001__initial.sql)

Thanks, I successfully migrated to 0.5.3 and could get rid of my custom resource scanner 👏
-def migrate(px: SxPool[IO], logger: Logger[IO])(database: String, migrations: String): IO[Unit] =
+def migrate(px: SxPool[IO], logger: Logger[IO])(database: String, migrations: List[ResourceFilePath]): IO[Unit] =
   given Console[IO] = Consoles(logger.append(Scope.Root / "dumbo" / database))
-    for
-      resources <- Resources
-        .list(migrations)
-        .filter(_.extName === ".sql")
-        .map(path => ResourceFilePath(s"/$path"))
-        .compile
-        .toList
-      _ <- Dumbo.withResources[IO](resources).withSession(px).runMigration
-    yield ()
+  Dumbo.withResources[IO](migrations).withSession(px).runMigration.void

Had to revert due to a different issue that emerged 🙃
https://github.com/scalacenter/scalafix/issues/2137

@taig  One thing I occasionally need to do with sbt is to run copyResources beforehand. Usually happens when I have sbt shell open and run cleanup and then re-compile. It actually never happens in CI. Also closing / reopening sbt shell seems to fix it in most cases.
Have also this setting in some places to ensure that resources are copied beforehand.
(Compile / compile) := ((Compile / compile) dependsOn (Compile / copyResources)).value

The order in which resources are copied and the compilation is executed seem to vary in sbt depending on how you run the tasks... :thinking:

Thank you, that does indeed fix the problem 🙏


Fixes transformation of lowercase 1-char protein to 3-char protein and uses mocking to test

This PR has a small bugfix, but the main content is to show a way of use mocking to remove the randomness of hypothesis' draw() function so that we can fix what is drawn, and have deterministic tests, allowing for e.g. to test some corner cases. I am also struggling to know how to do good tests in this project, since this randomness factor seems to complicate things, and we are more interested in the strategy producing output that conforms to certain properties, rather than specific outputs. But mocking in such a way might be helpful to some tests. This is just an example, we could try this to test other parts of this function and other functions.
For full details, see https://github.com/Lab41/hypothesis-bio/issues/36

Thanks for finding the bug!
Turns out the tests that you have written are something that can be written using minimal, thus eliminating the need for Mocking code. (Which in turn eliminates a dependency)
Can you refactor your tests based on the example here: https://github.com/Lab41/hypothesis-bio/blob/master/tests/test_rna.py

I think this can be closed, as it is outdated, as https://github.com/Lab41/hypothesis-bio/commit/a2a02bcfe8fa077dafdea8565629886e03865ad7 seems to implement the changes addressed here (sorry for the slow response, I could not do it earlier). Tell me if I am wrong and should still implement the changes requested and do the PR.

Yea sorry @leoisl !
I am closing this PR


When is wrapper.setProps + setTimeout needed?

Hi,
Thanks for this package!
I don't understand when I need to use setProps (and therefore make assertions wrapped in setTimeout).
The following code does not work, children is always empty:
const wrapper = mount(<Render if={false}>Hello</Render>)

This is needed:
    const wrapper = mount(<Render />)
    wrapper.setProps({if: false, children: Hello})

Could you help me understand why?

Maybe this is caused by preact. I'm not sure. And please give me a real code to debug it.

Thanks for the fast response!
You can clone git@github.com:oskarhane/neo4j-browser.git and checkout the 3.0-bring-back-enzyme branch and run yarn install and then yarn dev to test in watch mode.
The relevant test is in https://github.com/oskarhane/neo4j-browser/blob/45654bfcfc46055d7a7f8a21e63d6bfecbd007a5/src/browser/components/Render/index.test.js#L29-L56

Sorry, I got a error when running yarn dev

Weird. What if you try npm run dev?
You installed the dependecies?
Do you have a global old version of jest installed?

Try npm run dev got the same result. I have installed dependencies using yarn install. I'm not sure it related to node version. My node version is 6.4.0.

Hi. Thanks for this
I have the same problem. Any news about this ?

I've found that wrapper.update() after wrapper.update.setProps removes the need for async assertions.
Does that work for you too @mparpaillon?

@oskarhane Yes. Just a typo in your solution. It's
wrapper.setProps({bla: 'blabla'});
wrapper.update();

@windyGex This can be closed I guess

Sorry, typing on the phone :)


GetIt Problem in Delphi 10.3.3

I'm using Delphi 10.3.3. I formerly installed SVGIconImageList 2.x using GetIt. Today, I saw SVGIconImageList 3.0 is listed in GetIt. So I uninstalled the previous 2.x and tried to install 3.0. However it fails. Please see the screenshot.

Any idea is appreciated.

The problem appears that GetIt don't compile SVGImage32Package, that contains the new SVG engine Image32...
I will write to Embarcadero asking to redo the installer but it will take days.
I recommend that you download the project directly from GitHub and install it by hand following these simple instructions.
https://github.com/EtheaDev/SVGIconImageList/wiki

All right. Thank you!


Add Landing page content

🚀 Feature
Add description of the website (introduction) and misc. data about its work

Hi! Can I get this issue?

Assigned you @Alahoz , will assign @Yash-RajSingh if the issue won't get resolved within 1 day.

Has this issue been attended to now or can i take this up?

Assigned you @Yash-RajSingh

I've added the best suited information and details about the website on the landing page in the PR #46 . Kindly check and merge.


Migrate project from kci org

Since the main project seems to be discontinued, it would be nice to migrate it somewhere, so it can keep living.
There are a lot of things to be done

[x] Organization must be configured correctly
[ ] All the repos that belong to the operator should also be here
[ ] Packages should be migrated with all the history (or maybe without)
[ ] Actions should be configured
[ ] There must be a notice that project is not maintained in the old repo anymore
[ ] Maybe, we can now use main branch everywhere instead of master

I've invited you couple of days ago, you sure you haven't got it? I think it must be  resent now as well


Ensuring that the SolrDeleter handles cases in which a HTTP network error is encountered when requesting the deleting of a batch of IDs

Resolves #1389

@hackartisan I think in your comment you meant to type: we are not ready to close issue #1389 (I didn't want to edit your comment)

Yes. thanks @christinach; I fixed that typo.


Added permissive security flags fixes #3444

Added permissive security flags fixes #3444
Description:
New permissive security flag, to test run the game and enter a world, when the permissive security("-permissiveSecurity") flag is onsecurity messages similar to this: "[main] ERROR o.t.m.s.WarnOnlyProviderFactory - Non-permitted permission..." will appear in the console.

Can one of the admins please verify this patch?

Can one of the admins please verify this patch?

https://github.com/MovingBlocks/Terasology/pull/3494 This should be the fixed formatting version, it works fine as far as I could test, pretty sure Cervator also took a look at it.

Closing as replaced by #3494


Update Puppeteer to version 21.2.1

This release fixes the upstream bug that caused old Firefox revisions to not be removed from the cache.
Fixes #16904.

/botio test


Tabs Bug: The next and prev buttons display abnormal

Version
1.3.0
Environment
ng-zorro-antd 1.3.0
Reproduction link
https://ng-zorro-antd-start-qrygqd.stackblitz.io/
Steps to reproduce
I need to hide the pre and Next buttons without exceeding the scope.
What is expected?
The next and prev buttons automatically hide
What is actually happening?
The next and prev buttons are not hidden automatically.
Other?

https://stackblitz.com/edit/ng-zorro-antd-start-qrygqd?file=src%2Fapp%2Fapp.component.ts


Feat: Implement Email Notification on Subscribing to newsletter

Describe the feature
Currently, clicking the "Subscribe" button on our website's subscription form does not trigger any action, leading to a poor user experience. To enhance functionality and improve user engagement, we propose implementing a back-end feature that sends an automatic email notification to users upon clicking the "Subscribe" button. This email will confirm their subscription with a message such as "Thanks for subscribing."
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


How to write custom delegate method viewForHeaderInSection

How to write custom delegate method viewForHeaderInSectionfor UITableView using Bond?
I'm a newbie to Bond

Bond does not set up a delegate, only the data source. You can set the table view's delegate as you normally would.


Provide tasks for analyzing the health of core components

Much of this functionality is currently implemented in https://github.com/samvera-labs/core-dependency-report, but I would be curious as to whether that entire repository could be deprecated in favor of this.

@jrgriffiniii I believe the fixtures could be part of Huborg the repository, but not distributed as Huborg the gem. I definitely think the behavior would be reasonable to move into Huborg.

This morning I noticed that Github added a "Used by" button. This shows who's using the repository. It does not have version number checks.


Change cursor on Mac when inside splitter

Hey, I think it wold be really cool if you could get the same cursor on Mac in this that you get in the finder or Xcode when hovering over the splitter.
Maybe I can dig in and figure it out, but am new to Swift UI so wanted to ask about it.  Maybe it's simple somehow?
Thanks!

If you look in Sources/SplitView/Splitter.swift you will see some commented-out code I was using for a while on Mac Catalyst.
/*
 TODO: This seems to work okay in simple situations, but is kinda janky in others, so removing for now
.onHover { inside in
 // Perhaps should consider some kind of custom hoverEffect, since the cursor change
 // doesn't work on iOS
    #if targetEnvironment(macCatalyst)
    if inside {
        layout.isHorizontal ? NSCursor.resizeLeftRight.push() : NSCursor.resizeUpDown.push()
    } else {
        NSCursor.pop()
    }
    #endif
}
*/

There might be a good SwiftUI way to do this now, but in the version I was using at the time I had to resort to NSCursor. I don't remember exactly what the jankiness I referred to in the TODO comment was. I think it was that the cursor didn't change properly back to normal in some conditions. Hopefully it works okay or will put you on the right path. Happy to take a pull request if you like your result, but in any case, please let me know what worked for you. Thanks!

I'll try this out and tell you how it goes.  Thanks Steven

please share any progress using NSCursor because native without catalyst is still great to have, thanks

The changes in https://github.com/stevengharris/SplitView/pull/16 will add native cursor changes for Mac Catalyst and MacOS. The jankiness I found earlier when I went back to look at this in more detail was all around nested split views. I was not popping the cursor unless onHover was triggered and the cursor was outside of the Splitter. I believe the right thing to do is to always pop the cursor, and this is particularly needed when you transition from one splitter to another when using nested split views.
I can still detect some issues when you carefully drag your cursor along a splitter and move to another splitter when you have them nested. This is possible to see in the "Nested adjustable" demo. What happens is that occasionally the onHover in the splitter you're moving into is just not triggered as you transition from one splitter to the next. However, since the resizing cursor has been popped as it exits from the first splitter, the cursor will just be the normal arrow style and not transition to the different resizing style. It's pretty difficult to cause this to occur, and as a result, I just don't think I'm going to care about it happening very infrequently. And since the cause is that the onHover is not being triggered when the cursor transitions from one splitter to another in the nested case, I really don't know what I could do about it anyway. It's possible that continuousOnHover would avoid the problem, but frankly it's a mess to sort out how to deal with the version dependencies to use it, and I don't want to foist a required version change on people consuming the library.
I updated the DemoSplitter used in the "Custom splitter" demo to show how to avoid having the resizing cursor change take place when hovering over embedded content in a custom splitter, too.


Add web workflow to build Repl with Emscripten

Fixes #177
The build artifact produced after this change can be attached to releases; https://luau-lang.org/demo references a specific pinned release (0.505 at the moment) to fetch the JS binary.
We can probably change the JS code later to try to find the latest release using a GitHub API or binary search...

We can probably change the JS code later to try to find the latest release using a GitHub API or binary search...

There's a relatively simple way to get info about the latest release in json format:
curl -H 'Accept: application/json' https://api.github.com/repos/Roblox/luau/releases/latest

{
  // snip
  "tag_name": "0.505",
  "target_commitish": "master",
  "name": "0.505",
  "draft": false,
  "prerelease": false,
  "created_at": "2021-11-19T16:10:07Z",
  "published_at": "2021-11-19T16:14:25Z",
  // snip
  assets: [
    // snip
    {
      "url": "https://api.github.com/repos/Roblox/luau/releases/assets/50013249",
      "id": 50013249,
      "node_id": "RA_kwDOCN7VCM4C-yRB",
      "name": "Luau.Web.js",
      // snip
    }
  ]
}

@tiffany352 Nice! Is this something the browser can just query, or does this run into rate limiting et al?
We could probably also use this as a default and fall back to the hardcoded version.
PRs welcome in either case ;-)


enable current-dir inclusion

This change enables CMAKE_INCLUDE_CURRENT_DIR when building Triton.

Additional Info
The CMAKE_INCLUDE_CURRENT_DIR option is enabled when building LLVM and i found that my local builds failed to find a few headers if this isn't set. I'm not sure why others aren't hitting this, but if this doesn't break anyone else, i'd like to enable this.
In particular, the table-gen generated header files TritonCombine.inc and TritonGPUCombine.inc cannot be found without this option in my local builds.


How to login the dashboard

I hava set up successful but i found a problem that i can not login the background。就是登录不了后台 有什么办法登录吗？ 进入后台管理 。Waiting for your reply!

use /register and /login

register is not run successful. register but no records add in the db:users

我也很感兴趣

@xienima did you run php artisan db:seed?
If so, you can login using the credentials printed in the console.

Can't login... I did the "/login" but it doesn't redirect me into the admin panel, and i used the<EMAIL_ADDRESS>and "123pass" for password. And i did use php artisan db:seed.
And also why don't you make things easy by adding the login/register link into the Nav?

Yes, I login i used the<EMAIL_ADDRESS>and "123pass", but I can't post articles. @m2ura
Is there any solution?

@rshuzz It did login using those credentials to me... the i went to route http://localhost:8000/dashboard. that took me into the dashboard.

you can  search admin name in mysql  ,there are three items !But their password:pass123

use
php artisan migrate
php artisan db:seed
user<EMAIL_ADDRESS>       password:123pass

先自己注册一个,然后更改数据库users表中的is_admin字段为1就可以登录了


Outline per page on existing PDF

Question
Description
hello everyone !
I am new to this but I am trying to add outlines (bookmarks) to some pages of an existing PDF that I would import. I did succeed adding one outline on the first page of a document I created but I really need to add on selected pages of a document that already exists created without pdf kit).
Do you know how to do that please?
Thanks in advance and have a great day!
Code sample
None
Your environment

pdfkit version: last
Node version: last
Browser version (if applicable): on electron (chromium)
Operating System: windows 10

I am doing this:
  const doc = new PDFDocument;
  const { outline } = doc;
  const mainOutline = outline.addItem(title, { expanded: true });
  doc.do things
  doc.addPage().do things
  mainOutline.addItem(page title);

This gives me one top level bookmark on the first page and then each page I create also shows up.


Adds experimental support for cancelling individual inputs

Using sigusr1, an exception is thrown into the user(main) thread in case of sync code, and a CancelledError is thrown in case the user code is async
Also adds some new container tests that check that signal handling works as expected for sigusr1 (might be nice to add tests for sigint/sigterm as well
There is no code in the worker yet to actually send these signals (there are some non-trivial prerequisites there), so we could either merge this and have it be unused (albeit tested) functionality for a bit, or keep the PR until we have the necessary support on the worker?

Nice! Might make sense to ship this so that we have it present in the client even without the worker support. That way we have the optionality to implement the worker side of it later.

Doing some PR cleanup; I'm guessing this has been superseded by the work you're currently doing to going to close.


Add ARIA: list role

The ARIA list role can be used to identify a list of items.
https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/List_role

I would like to take this up!


bug with react-native link in android and windows

Environment

React Native Environment Info:
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-4770K CPU @ 3.50GHz
Memory: 2.66 GB / 15.89 GB
Binaries:
Yarn: 1.13.0 - ~\AppData\Roaming\npm\yarn.CMD
npm: 6.5.0 - C:\Program Files\nodejs\npm.CMD
Watchman: 4.9.4 - D:\servers\watchman\watchman.EXE
IDEs:
Android Studio: Version  <IP_ADDRESS> AI-182.5<IP_ADDRESS>99772
Description

When run react-native link in windows and android
the url use \ and not /
for example:
project(':react-native-gesture-handler').projectDir = new File(rootProject.projectDir, '..\node_modules\react-native-gesture-handler\android')
and to fix to:
project(':react-native-gesture-handler').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-gesture-handler/android')
Reproducible Demo

react-native link in windows

same error for me


New Pipelines landing page

Build the page as per the design
Split the current Overview page into:

UI walkthrough
Understand the architecture pages under Introduction
Create new Configure Pipelines Overview page

Screenshots

Rebased recent changes, new screenshot

Perhaps a follow up change. I reckon it would be great if these whole tiles were hoverable / clickable.


Warning with refEq

We are testing something that uses OkHttp interceptors and came across an issue that we are not sure how to fix. We have this:
verify(chain).proceed(refEq(expectedRequest))

To make sure the Chain is called with the right request. However, this gives us a Type Mismatch warning because It expects a Request and not Request? as refEq returns.
The only thing that we have tried to do is to add !! at the end to remove the warning but that has woese consequences as it makes the tests fail with a KotlinNullPointerException. Which is understandable since Mockito returns null in their implementation.
Would it be as simple as using ?: createInstance() as it is done on other matchers?

Fixed in #329


Does netty support java 10?  If not, when?

I have tried to compile netty in jdk 10,but there are to many errors. Then i change to jdk1.6, it worked.

You're confusing compiling with a given JDK, and supporting one as a runtime.
Netty can run with Java 10.

The errors you speak of (Logging) have been posted as issues likely. You can find them in the close and open issues with the search box. This question in general is overly broad and probably is already answered.
Java 9 Issues Open/Closed
Javga 10 Issues Open/Closed


navigation to narrowed value

Given code like:
Object x = "";
if (is String x) {}

Clicking on the second occurrence x takes me nowhere. It should take me to the declaration.

Fixed.


[Docs] Hover effect in Bouncy number checker

Category of documentation update
there is much more transition due to which it looks unutractive
https://github.com/Rakesh9100/CalcDiverse/assets/168094736/212f4f03-a719-4f42-9cc3-24bb97da1a10
Describe the change you think might work
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@puneet426 what change do you have in mind?

@jessicachawla I wanted to change the UI

What kind of change? Please elaborate

i want to change the duration of transition and font size after transition and also change page colors

Okay.
@Rakesh9100 this issue can be assigned

Go ahead @puneet426

@jessicachawla can i add explaination in it ??

Provide your PR @puneet426


[Feature] Create a keyboard shortcut or a button to select all points loaded in napari GUI

🚀 Feature
Create a keyboard shortcut to select all points loaded in napari GUI
Motivation
I am using cellfinder-napari which allows visualization and annotation of cells within 2d brain images. Using napari GUI, I can only select specific points within a specific plane. I cannot generalize my modifications to all the points in all the planes.
Following snapshot shows few points selected after clicking on "select points"

But notice at the bottom right corner that this is for 131st (out of 549) plane. Any modifications I make to these points will not be reflected in the points that show up in any other plane (once I start going from left to right). So It can be frustration to individually select points again and again and make modification for each selection.
Pitch
A keyboard shortcut or a button that selects every single point so that the modifications (increase/decrease point size, change color, change shape) can be applied to everything and not just few selected points.
Alternatives
@sofroniewn has mentioned here that this can be done via CLI using:
layer.selected_data = list(range(len(layer.data)))
But I am not sure if I can use this technique as I am using an existing plugin. I haven't yet looked into writing a plugin myself but if this feature is hard to implement at the moment, then I will definitely look into it.

A great suggestion, thanks for raising it @rdbcasillas!
One thing to consider is whether user would want to also have a shortcut to select all visible points. So we could do e.g. "ctrl-a" - select all visible points, "ctrl-shift-a" select all points in the data. (Or vice versa).
A potentially useful reference is Gmail: select all selects the current page of emails, with a message asking whether you want to select all emails matching the current filter. Probably we have to up our messaging game in our UI before we can be this sophisticated, though. 😬 We'll get there!

One thing to consider is whether user would want to also have a shortcut to select all visible points.

All points in the current slice is A https://github.com/napari/napari/blob/v0.3.8/napari/layers/points/_points_key_bindings.py#L54-L59

I think https://github.com/napari/napari/pull/4809 closes this?
It implemented the ability to select all points in a layer vs. just the ones in a current view:

Works in 3D mode too!

@psobolewskiPhD one thing I would like us to add before closing is an "Edit" menu with a "select all" option.

I'm all for an edit menu:
https://github.com/napari/napari/issues/5080

haha very good, I guess we can close this then?

I think so, I mean the OP asks for keyboard shortcuts, which do exist.
I think we should probably look into better documentation for 1. existing shortcuts and 2. the ability to change them.
https://github.com/napari/docs/issues/13


Remove version: CrystalDewWorld.CrystalDiskMark version 8.0.3

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Duplicate;


document std.stdio.byRecord

follow-up to #4075 / #4053 - this PR just adds documentation for byRecord (as suggested by @ntrel)

Ping @greenify

Sorry lost track of it - I found the reason why REF_ALTTEXT wasn't working. format is a parameter and thus put in italic - using the underscore magic for format works :)

Auto-merge toggled on


"Maximum path length limitation in Windows" Link Not Found

"Maximum path length limitation in Windows" Link Not Found.

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

ID: 39fffb76-5378-fe8a-62fc-0c82a6d03d6d
Version Independent ID: 102ff357-ae01-8222-8c14-64e2a5e78a0b
Content: Portals support for Power Platform CLI - Power Apps
Content Source: powerapps-docs/maker/portals/power-apps-cli.md
Service: powerapps
Sub-service: portals
GitHub Login: @neerajnandwana-msft
Microsoft Alias: nenandw

Thank you for bringing this to our attention.  The link has been fixed and will be updated on docs in a few hours.  The link is https://docs.microsoft.com/windows/win32/fileio/maximum-file-path-limitation


remark-wiki-link: Obsidian's shortened paths to images are not resolved properly?

Obsidian style shortened paths in our markdownbased project seem not to be resolved to their full paths properly. It works for .md files though.
### Tasks
- [ ] investigate

Obsidian style ![[]] embeds do work. The format used was wrong, i.e. sb tried using GFM ![]() format with shortened path, which we don't support.


Error building hlo_verifier_test.cc

tensorflow/compiler/xla/service/hlo_verifier_test.cc:719:31: error: 'StrReplaceAll' is not a member of 'absl'
   return ParseHloString(absl::StrReplaceAll(
                               ^~~~~~~~~~~~~

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes - but no changes to this file.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): ubuntu 18.04
TensorFlow installed from (source or binary): source build
TensorFlow version (use command below): master
Python version: master build
Bazel version (if compiling from source): 0.24.1
GCC/Compiler version (if compiling from source): gcc (Ubuntu 7.3.0-27ubuntu1~18.04) 7.3.0

When building from source including XLA I receive the error above.

Just to verify did you get chance to follow instructions from TensorFlow website .Please, let us know. Thanks

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


New stats collection

The collect functions all seem to work: https://github.com/psiinon/zap-mgmt-scripts/runs/1296920608?check_suite_focus=true
The github 'daily' function also seems to work based on local testing, the docker and bitly ones are still to do.
I've added the right secrets so that the new action runs successfully in my fork. This means that we'll start collecting the daily stats.
When it passes review and merged then I'll delete them and put them in this repo.
Signed-off-by: Simon Bennetts<EMAIL_ADDRESS>

Oh, looks like we dont have the WIP check - so this just a reminder to NOT merge this PR yet...

The bitly monthly stats should be good now.

All planned changes done - working fine here and pushing to the S3 backets.
No longer WIP so ready for review.

Thanks @kingthorin - comments hopefully addressed


Develop

Codecov Report

Merging #7 (42c17a0) into master (069eaa9) will decrease coverage by 2.33%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
- Coverage   96.11%   93.78%   -2.34%
==========================================
  Files          15       16       +1
  Lines         541      579      +38
==========================================
+ Hits          520      543      +23
- Misses         21       36      +15

Impacted Files
Coverage Δ

dataloader/preprocessor.py
92.79% <57.14%> (-1.60%)
:arrow_down:

dataloader/image_generator.py
85.96% <62.50%> (-2.72%)
:arrow_down:

utils/plotter.py
81.66% <81.66%> (ø)

dataloader/circle_generator.py
83.33% <100.00%> (ø)

dataloader/image.py
100.00% <100.00%> (ø)

...unctional_tests/test_image_generator_functional.py
100.00% <100.00%> (ø)

tests/utils/test_coordinates_converter.py
100.00% <100.00%> (ø)

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 069eaa9...42c17a0. Read the comment docs.


Remove non-header files from include folder

The include directory previously contained all files copied over, but only *.h and *.hpp files are required. This commit removes all files that do not have a .h or .hpp extension, as well as any empty directories that may result from this cleanup.

@pablogs9 any updates on this?

Reopening to have CI passing

@mergify backport rolling jazzy

@meetgandhi-dev thanks for the contrib!


Parse edit masks in DefineDataParser

Currently edit masks are just added as descendants, e.g. in INIT.
I'm not sure if they're interesting for analyzers. But they should at least be parsed correctly to show errors for invalid masks.

[ ] Check masks TODO category

I think we can close this now. I don't believe EM is interesting to keep as an attribute on the variable.


feat: Episode 11: WAYAHDYGH: Joey

Another week, another (kinda late sorry) episode! Thanks to @joeyAghion for joining me on this one.
Still took a few hours, but definitely the fastest edit I've done so far. Feedback welcome and appreciated!
Thinking I'll try to ship this tomorrow if things sound OK to folks.

Got a thumbsup from Jon on Slack and Anna's out today, so I'm going to go ahead and merge myself!


$ ionic build --prod failure

Short description of the problem:
Trying to run that command and it fails with this error:
Can't resolve all parameters for MyApp in /Users/rafa/natal-food/imoto-motoboys/src/app/app.component.ts:
            ([object Object], [object Object], [object Object], ?).

See:
➜  imoto-motoboys git:(master) ✗ ionic build --prod
Running app-scripts build: --prod
[01:34:30]  build prod started ...
[01:34:30]  clean started ...
[01:34:30]  clean finished in 2 ms
[01:34:30]  copy started ...
[01:34:30]  deeplinks started ...
[01:34:30]  deeplinks finished in 58 ms
[01:34:30]  ngc started ...
[01:34:34]  typescript error
            Can't resolve all parameters for MyApp in /Users/rafa/natal-food/imoto-motoboys/src/app/app.component.ts:
            ([object Object], [object Object], [object Object], ?).

Error: The Angular AoT build failed. See the issues above
    at /Users/rafa/natal-food/imoto-motoboys/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:237:55
    at step (/Users/rafa/natal-food/imoto-motoboys/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:32:23)
    at Object.next (/Users/rafa/natal-food/imoto-motoboys/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:13:53)
    at fulfilled (/Users/rafa/natal-food/imoto-motoboys/node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:4:58)
    at <anonymous>
[01:34:34]  copy finished in 3.44 s

But I have my provider normally setup on app.module.ts.
Relevant code
# app.component.ts

import { Component, ViewChild } from '@angular/core'
import { Nav, Platform } from 'ionic-angular'
import { StatusBar } from '@ionic-native/status-bar'
import { SplashScreen } from '@ionic-native/splash-screen'

import HomePage from '../pages/home/home'
import LoginPage from '../pages/login/login'
import Motoboy from '../services/motoboy'

@Component({
  templateUrl: 'app.html'
})
export class MyApp {
  @ViewChild(Nav) nav: Nav

  rootPage: any

  pages: Array<{title: string, component: any}>

  constructor(
    public platform: Platform,
    public statusBar: StatusBar,
    public splashScreen: SplashScreen,
    public motoboy: Motoboy,
  ) {}

# app.module.ts

import Motoboy from '../services/motoboy'
...
  providers: [
    Motoboy,

Which @ionic/app-scripts version are you using?
3.1.8
Does anyone have a clue how to proceed here? Spent the entire day trying to figure out how to fix that 😞

I managed to get this to work, it's very difficult to address this issue due to the really bad error message it throws.
For more info, see my answer here:
https://forum.ionicframework.com/t/issue-running-ionic-build-prod/120435/16
Copy/paste
Apparently I can't export default class, this was the entire ******* issue.
My steps:
Step 1
Create a new ionic project and make check whether $ ionic build --prod works.
It did work.
Step 2
Try to make the build fail.
I achieved that creating a service and trying to use it in app.component.ts
import { Injectable } from '@angular/core'

@Injectable()
export default class Service {
  constructor() {}
}

Step 3
Try to make the build pass.
Updated my service to do an export class Service instead of export default class Service
import { Injectable } from '@angular/core'

@Injectable()
export class Service {
  constructor() {}
}

It passed.
Conclusion
Both errors were solved by that.
1- app.component.ts -> Can't resolve all parameters for MyApp [...]
2- app.module.ts -> Unexpected value 'null' declared by the module 'AppModule in [...]


ID bhyzub055q7mzop: Add influxdb to Databases and Administration

{
"services": {
"influxdb": {
"name": "influxdb",
"available": 1,
"total": 1,
"uris": [
"uiotn9ohoh9q9ebvpterr7gitk.provider1.akashdev.net"
],
"observed-generation": 0,
"replicas": 0,
"updated-replicas": 0,
"ready-replicas": 0,
"available-replicas": 0
}
},
"forwarded-ports": {}
}

I resolved the conflicts and merged.  Thank you!


[structure] public SDK should be generated on build

Currently, sources directory provides special SDK directory, which contains all public headers available for host and target build environment.

these headers should be moved to proper libraries / images
SDK directory should be installed on target file system when building image
for project crosscompile, headers should be available from sources
for self hosted compilation, these headers should be in a system-wide directory

Effective change is to:

move sources/sdk/selment to sources/kernel/headers/ as public headers (as opposed to sources/kernel/kernel/headers, which are private headers)
move sources/sdk/crt to sources/libraries/crt/headers

Solved by moving all headers into separate /source/public/ directory.


Catching

Codecov Report

Merging #3 (17d0b28) into master (d91a8fa) will increase coverage by 1.38%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   88.82%   90.20%   +1.38%
==========================================
  Files           8        9       +1
  Lines         170      194      +24
==========================================
+ Hits          151      175      +24
  Misses         13       13
  Partials        6        6

Flag
Coverage Δ

unittests
90.20% <100.00%> (+1.38%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

catch.go
100.00% <100.00%> (ø)

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 d91a8fa...17d0b28. Read the comment docs.


feat: bump fuels to next version

Bump fuels to next version

This PR was created only for testing purposes. Closing it.

re-opening to test a quick TS SDK release


[FEATURE] To add google tracking id for user profile.

Description
It would be helpful for a user to see the analytics and if we support adding google analytics tracking id in json file.
It would be helpful.
Please confirm if this is a valid feature.
I would love to do this.

By Eddie

I think the field gAnalytics can be added to people's profiles
if gAnalytics is available in the profile, then add to their profile page

Can I know more about this feature you suggested?

Hey @Shubhdeep12, I wanna work on this issue! I am new to opensource contribution, so it'd be more helpful to me if some more info is provided.

Hi @anirudhp26 @AgniveshChaubey
Thanks for the interest in the feature.
I also want to do this but want to confirm whether this is valid or not.
Explanation - In order to create a linkfree page, we need to create a json file, so what i am suggesting is the support to have a google analytics tracking id in the json file, so that user can have the analytics of his/her linkfree page.
@eddiejaoude sorry to mention.
But any comments from your end would be helpful here.

@Shubhdeep12 ohh so you mean to track info like whose profile is viewed more, all ?
This can be a great feature as well

We now have stats for the profile page and links
Can we have individual google analytics for each page?

Hey @eddiejaoude sorry for late reply. Seems like it is considered as valid feature. Should i start work on this? Or anyone else is working on this right now?
Here is my plan:

As you also mentioned above,  in json file there would be a new key gAnalytics where user can add his/her GTM tag.
And we will add that particular tag in head tag for the particular page.

Please confirm, then I'll start work on this.
Thanks

No problem. I have now assigned it two you @Shubhdeep12

We removed nextjs-google-analytics as we are using Vercel analytics. I think this feature will come as part of the Premium version on LinkFree, so I will block it for a few more weeks until that part is finished

I have changed the label to status: blocked given the the discussion on the Pull Request (quoted above): https://github.com/EddieHubCommunity/LinkFree/pull/5724

Going to close this for now as it is not a highly requested feature plus has some legal complications


Add Swift 4 Support

Updating code to support Swift 4.

Hey @chrisiKern , thanks for your work! I'm currently addressing open issues and will make an update somewhere this or next week. I'll then merge your update in a separate swift4 branch such that people can still use the latest version of the framework with Xcode 8.x too, but can opt-in for swift4 when desired. I'll keep you posted!

I have merged this into a separate swift4 branch. Thanks again @chrisiKern!


Enhance the maps screen functionality

[ ] Add a chat button for each business flashpoint that takes the user to a new chat window with the business
[ ] Reference BitcoinJungle for Maps improvements


Bind ONLY to <IP_ADDRESS>

Security should be addressed here, we should provide a way to provide binding arguments:
In my case I want to bind only to <IP_ADDRESS> and not exposing my app to public ( only nginx is exposed ).
Ofcourse we can use the firewall to prevent direct access to my app - but Nex.js should provide a basic mechanism to bind to a specific address.

@mtrabelsi now dev --help now start --help. We provide a --host option to set the binding host. I believe that's what you're looking for?

I think you're talking about now which is not my case :- ) I'm putting a issue on next.js repo

Well, that's sad : https://learnnextjs.com/basics/deploying-a-nextjs-app
Looks like it's not yet decided how we deploy next apps.

@mtrabelsi whoops it was still early, I'm talking about next.

next dev --help

    Description
      Starts the application in development mode (hot-code reloading, error
      reporting, etc)

    Usage
      $ next dev <dir> -p <port number>

    <dir> represents where the compiled folder should go.
    If no directory is provided, the folder will be created in the current directory.
    You can set a custom folder in config https://github.com/zeit/next.js#custom-configuration.

    Options
      --port, -p      A port number on which to start the application
      --hostname, -H  Hostname on which to start the application
      --help, -h      Displays this message```


Support hypergraphs and hypergraph algorithms

This issue is for brainstorming what would be involved in generalizing SPRAS to support hypergraphs and hypergraph algorithms. One possible goal could be showing how converting hypergraphs to graphs and running the graph algorithms in SPRAS is ineffective.
There are no short term plans to work on this.

Are any of the datasets naturally a hypergraph that have instead been converted into a graph for graph algorithms? I know that PPIs are typically graphs, and may be difficult to convert into hypergraphs, but any reaction network can be naturally converted to a hypergraph.
I suspect that many evaluation metrics are likely directly applicable to hyperpaths, as long as they refer to the original nodes and reactions within the reaction network.

We don't have many datasets in SPRAS yet and are making a push to add them this summer. We will likely add pathway databases, so those (e.g. Reactome) would support hypergraph representations.

That would be great. Do you already have a set of metrics to compare different pathway reconstruction methods? If so, could you point me to them?

We don't have anything written up yet. @ntalluri is working on that. We will likely use fairly standard machine learning classification metrics. The question is what to use for ground truth and when we will have edges versus only nodes for ground truth, which is likely when working with real data.

Sounds good. I would love to test our hyperpath and factory methods on pathway reconstruction benchmarks once they are available.


Should loadTheme use loadEssentialThemeData?

Looking at #1945 I started wondering why loadEssentialThemeData is not called from loadTheme in order to reduce code duplication, in fact what is loaded in loadEssentialThemeData is the "mandatory" part of the theme data we need, while loadTheme adds more stuff.

The queries appear to be pretty different.


feat: add gitsignsupdatedone autocmd

how do you feel like add an event after update done ?  my situation is my statusline using coroutine and  refresh when event trigger. when the GitSignsUpdate triggered the b:gitsigns_status_dict may not set or updated so I can't get any useful data from it . add this event will more useful. or move the GitSignsUpdate position ?

Does the GitSignsUpdate not work?

right because this event triggered before variable set or update.  so i can't get any data when it triggered in my stl with coroutine. this pr can work because it trigged after variable set.

So instead of creating an entirely new event. Just move the old event later.

okay

Read https://github.com/lewis6991/gitsigns.nvim/blob/main/CONTRIBUTING.md

emm why closed?

I'm not submitting this in its current form. I told you changes I wanted and you said.

in there works for me these days. i don't look whole project sry.

So I closed the PR.

in there works for me these days. i don't look whole project sry.

what i mean is triggerd in this file it works fine. I can get the correct data in coroutine.  i don't look  the whole project .just find this update function from Lspsaga outgoing_calls.

I told you changes

you don't tell me where is correct pos to put this event ?  you  trigger  before updating  👍  ?
fine.  is your plugin.


Add K8s for kids

Awesome resource, thank you! There’s a slight issue with the markdown link though.

Fixed!

Thank you!


R: update CRAN package set

Motivation for this change
Update CRAN snapshot
Things done
Following instructions in doc/languages-frameworks/r.section.md:
$ cd pkgs/development/r-modules
$ nix-shell generate-shell.nix
$ Rscript generate-r-packages.R cran  > cran-packages.nix.new
$ mv cran-packages.nix.new cran-packages.nix
$ nix-build test-evaluation.nix --dry-run

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Pushed de589cd1ce8185910c927034fa774ebf4c2dce75 to r-updates  for test building at https://hydra.nixos.org/jobset/nixpkgs/r-updates.

Looks like hydra finished. Do I need to investigate those three failures more closely?
Also, since hydra is building the packages anyway, is there a way we can enable substitutions for R packages so users don't have to rebuild from source? If storage space is a concern, we could just do it for a small whitelist. Some popular packages (Rcpp, dplyr, etc) take a decent amount of time to build locally, especially on older laptops.

The substitution should already happen when hydra is building the packages.

@Mic92 I don't think I've ever had any R package substituted. And as far as I know I'm not doing anything that would require them to be rebuilt.

I don't think I've ever had any R package substituted.

R packages aren't usually built by Hydra.

@peti Thanks. My question was though whether it's possible to enable that, at least for a subset of R packages. Some are very popular (Rcpp, dplyr, etc) and take a while to build locally. Other distros (e.g. Debian) do offer a subset of CRAN packages as binaries.

I don't know. We used to build rPackages but had to disable that a while ago because the sheer number of packages overloaded Hydra's evaluator. It's possible that problem would still exist today, but I am not in a position to tell with any authority. We could probably build a subset without any problems ... but that poses the question of what that subset would be? Who defines that?
Personally, I don't feel stronly about any of this, to be honest. I compile my R packages locally and it seems to be no big deal. Maybe I'm just lucky with my choice of dependencies, though. Who knows.

Sure, that makes sense.
I don't think it would be very hard to come up with a reasonable whitelist. Perhaps the tidyverse + the X number of packages that have the highest number of reverse dependencies? Perhaps a few select others too based on the amount of compile time? rstan is one package (for me) that is annoying to compile locally because of the amount of time it takes.


Not issues~

i'm too lazy to PR
Here is my tips
modify the tagname according to the parameter
    const Wrap = ({ tagName, content }) => {
      const Tag = `${tagName}`     // variable name must begin with capital letters
      return <Tag>{content}</Tag>
    }

@wangcongyi Added a small note to the Weapper components section. Let me know if it looks fine.

@vasanthk  It looks great
I apologize for my laziness
Happy coding